[UIKit] Configuration of Conversation View

I'm aggregating the settings of Conversation View here.
I think we need to design a way so that configuration is easy to set. Top of my head it could be done by:

Constructor configurations + Setter
Callback Function

I think different config might be better to use a different way to configure? Or we should just use app delegate? (However, I think it is weird to use app delegate to set simple things like the color?)
Also, don't assume all Conversation View in the apps have the same setting. (e.g. Users might want Read by enabled only for certain users).
Anyway, here are a list:
Feature Enable

Disable / Enable Voice Message
Disable / Enable Sending Attachment
Disable / Enable Sending Camera Images
Disable / Enable Read Status
Disable / Enable Typing Indicator

UI Customization

Bubble background color
Bubble text color
Conversion background color
Conversion background images
Show / Hide avatar
Text Area Background Color
Icons of Attachment, Camera, and Voice Call
Copywriting Customization

Data Customization

User avatar #69
Display name of user #69

Behaviour Customization

Show Group Title or Other Username's as Group name

Event Hook Callback

New Message in Conversation View
Updated Message in Conversation View
Before / After Message Send
Message Send Failed
Disconnected
Connected

I will try to build the UIKit using 3rd party ChatKit.

[ ] Prepare a list of configurable items based on chatkit

@carmenlau
I have prepared a list of features implemented in Chat UIKit on iOS. Most of them are connecting UI with Skygear Chat. For the UI configurable items, I think they can implemented in ChatKit.
I think the following list can be the specification of the Android UIKit for Skygear Chat.
Conversation View
UI

Message Bubble Color
Message Avatar
Typing Indicator
Accessory Button Should Show
Accessory Button Action

Skygear Chat Features

Message Status (all read / some read / sent / delivering)
Typing Indicator
Participants Fetching
Messages Fetching
Message Change Subscription

Hooks

Before Message Send Hook
After Message Send Hook
Failed to Send Message Hook
After Fetch Participants Hook
Failed to Fetch Participants Hook
Before Fetch Message Hook
After Fetch Message Hook
Failed to Fetch Message Hook

Conversation List View
UI

Conversation Avatar
Pull Down Refresh

Skygear Chat Features

Conversation Change Subscription
Conversation Unread Count Badge
Conversation Last Message

Participant List View
UI

Conversation Avatar

Skygear Chat Features

Query Participants (by username, by name, by email)

The design of chat ui
https://www.dropbox.com/s/dmvuxxon49fo646/SkygearChatApp(Android)-130917.png?dl=0
https://www.dropbox.com/s/y72t5pwm8xr7dlg/Android-SkygearChatApp-170817.sketch?dl=0

Feature Enable

Disable / Enable Voice Message
Disable / Enable Sending Camera Images
Disable / Enable Sending Attachment
Disable / Enable Read Status
Disable / Enable Typing Indicator

UI Customization

Bubble incoming background color

Bubble incoming text color

Bubble incoming voice play button color

Bubble incoming timestamp color

Bubble outgoing background color

Bubble outgoing text color

Bubble outgoing status text color

Bubble outgoing voice play button color

Bubble outgoing timestamp color

Conversion background color

Conversion background images

Show Group Title or Other Username's as Group name #80

Text Area Background Color

Icons of Attachment, Camera, and Voice Message

Behaviour Customization

Copywriting Customization #39

Event Hook Callback

New Message in Conversation View
Updated Message in Conversation View
Before / After Message Send
Message Send Failed
Disconnected
Connected

New Message in Conversation View
Updated Message in Conversation View
Before / After Message Send
Message Send Failed
Disconnected
Connected

Hi, Disconnect and Connect cannot be done in Android and iOS because there is no listener to hook with PubsubContainer, only JS has such feature.
skygeario/skygear-SDK-Android#173
skygeario/skygear-SDK-iOS#155


Improve decoupling

This addresses the decoupling concerns in #2 by moving state evolution "laws" into separate functions.  Maybe this is a clean approach?

Reasonable given the constraint of using arbitrary model attributes.


SetText doesn't work from a different func.

For some reason, SetText on a Label does not work from different files, even if they are global vars in the same package. Something like:
Main.go
package main
MyLabel = tui.NewLabel("doe")
Ui, err = tui.New(tui.NewVBox(MyLabel))
if err := Ui.Run(); err != nil {
panic(err)
}
MyLabel.SetText("ray")
foo()
other.go
package main
foo() {
MyLabel.SetText("me")
}
go run main.go other.go
MyLabel is still "ray"
I've made all my vars exportable. I'm not sure what I'm doing wrong?

The call to ui.Run will block the main goroutine, meaning foo will be run when you exit the application. You'll need to call SetText from another goroutine. Checkout the chat example for some ideas on how you can do this.

Now I feel really dumb. I just looked at the chat example and I don't see any go routines?

Don't. You're not the first one to ask about it. Actually, I realized that in the chat example the goroutine is managed by the ui.
You can supply a function to be run every time the user presses enter in the input field.
input.OnSubmit(func(e *tui.Entry) {
	history.Append(tui.NewHBox(
		tui.NewLabel(time.Now().Format("15:04")),
		tui.NewPadder(1, 0, tui.NewLabel(fmt.Sprintf("<%s>", "john"))),
		tui.NewLabel(e.Text()),
		tui.NewSpacer(),
	))
	input.SetText("")
})

If you want to update the UI from one of your own goroutines, you can use the ui.Update method: https://github.com/marcusolsson/tui-go/blob/master/example/audioplayer/main.go#L66-L71

Thanks! I'll check it out.


add message argument to wait_until-while_present methods

Pull Request Test Coverage Report for Build 1666

4 of 6 (66.67%)  changed or added relevant lines in 1 file are covered.
223 unchanged lines in 12 files lost coverage.
Overall coverage decreased (-0.7%) to 95.553%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/watir/wait.rb
4
6
66.67%

Files with Coverage Reduction
New Missed Lines
%

lib/watir/elements/select.rb
1
91.58%

spec/watirspec/elements/element_spec.rb
1
95.85%

lib/watir/browser.rb
2
96.12%

lib/watir/elements/element.rb
3
94.42%

lib/watir/wait.rb
4
83.52%

spec/watirspec/window_switching_spec.rb
4
96.64%

spec/watirspec/alert_spec.rb
7
100.0%

spec/watirspec/relaxed_locate_spec.rb
15
100.0%

lib/watir/legacy_wait.rb
18
75.56%

lib/watir/capabilities.rb
19
84.81%

Totals

Change from base Build 1665:
-0.7%

Covered Lines:
10272

Relevant Lines:
10750

💛  - Coveralls

Rebased and merged: a70ce65ed1faa312c3245febc586f5918544aa1f
(I updated the default messages with the added info).
Thanks for the commit!


FIX: bugs in aggregator loaders

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@30b2ea4). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #174   +/-   ##
=======================================
  Coverage        ?   88.04%
=======================================
  Files           ?        4
  Lines           ?      393
  Branches        ?        0
=======================================
  Hits            ?      346
  Misses          ?       47
  Partials        ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add integration test tools for MORYX modules

Adding a MoryxTestEnvironment as well as a tutorial on how to use it for integration testing of modules.

I removed the class generic as it makes things unncessarily complicated. It was only used to extract the module from the service collection via its facade registration. Since modules are registered under their type this could be simplified.

I like it, adjusted the tutorial to match the non generig class


adding permute function for reordering tensor products of quantum objects

This hopefully resolves issue #95, it doesnt handle super operators, but perhaps that can be added in the future on top of this.

Sorry that we have re-organized the source code structure yesterday. So there's a conflict compare to the main branch.
Could you put these new functions in src/qobj/arithmetic_and_attributes.jl.

Could you also put the docstring of permute in to the documentation API list (in the section Qobj arithmetic and attributes)


[R2] multipart bindings

Still need to check if I need to implement gcvisitor on the MultipartUpload object, since it contains a jsg::Ref.

Still need to check if I need to implement gcvisitor on the MultipartUpload object, since it contains a jsg::Ref.

Since it is unlikely that the R2Bucket and the MultipartUpload object will have a circular reference with each other, the gcvisitor impl would be optional. Personally I'd add it still but it shouldn't be required in this case.

I think I've adressed all feedback now, I'll squash the commits once all reviewing is done.

Real quick question, Since this is the the OSS repo do we want to have CF specific API's in here?
Ping: @jasnell @Frederik-Baetens @vlovich

@Warfields that kind of question might be better suited for an internal chat channel, but if you look in the readme you can see that one of the goals of open source workerd is for it to be used "As an application server, to self-host applications designed for Cloudflare Workers."
Applications designed for Cloudflare Workers will generally make use of Cloudflare specific APIs. In order to be able to self host them, they do need to be able to use the bindings. That means that the bindings code also needs to be included here. In the future we, or the community might provide alternative backends for the bindings to tie into, so that applications making use of the bindings can seamlessly run on a self hosted workerd and on the Cloudflare Workers platform.

Sounds Good @Frederik-Baetens

Closing because I lost permissions to update this branch


fix: Speed up the block round trip proptest

Reduce the number of cases run by the block round trip proptest, to
speed up the Zebra tests.
Closes #803.

After this change, using this script:
for d in zebra*; do
    pushd "$d"
    cargo --quiet test --no-run
    time cargo --quiet test
    popd
done

The crates with test times over 1 second are:
zebra-chain
real    0m8.927s
user    0m39.625s
zebra-consensus
real    0m1.716s
user    0m2.761s
zebrad
real    0m3.837s
user    0m1.772s

@yaahc is 15-20 seconds fast enough for all the Zebra tests, or would you like me to tweak them some more?

I'm ok with configuring the number of proptests run, I wonder if we can use the env vars it looks at to make it 'faster' where needed (local before push) and 'slower' where capacity allows (CI, where we can require even more test cases to pass if we want).

I don't want to globally override the number of cases by default, because only the block rests are slow right now.
But I could use the cases env var where it exists, and then default to 16 block cases. That way, we can increase the number of cases in CI, or decrease them even further right before push.
(Most of the time when I'm developing, each change takes me longer than 20 seconds to write.)

But I could use the cases env var where it exists, and then default to 16 block cases. That way, we can increase the number of cases in CI, or decrease them even further right before push.

See #828 for this change.


KAFKA-8857; Don't check synonyms while determining if config is readOnly

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@omkreddy Thanks for the review, merging to trunk.


Hover on anchors leveraging dynamic values in a span are bad

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
I have a "helper" MDX file:
### {props.title}

...

When this is rendered, the HTML looks like:

Reproducible demo
https://stackblitz.com/edit/github-pqvkar?file=docs%2F_helper.mdx
Steps to reproduce
(See description)
Expected behavior
My title looking like the displayed value
Actual behavior
Tooltip uses the expression
Your environment
No response
Self-service

[ ] I'd be willing to fix this bug myself.

I have a "helper" MDX file:
### {props.title}

...

When this is rendered, the HTML looks like:

Not really, this is rendered as:

foo

Which is the intended behavior
I understand "propstitle" does not look like a good looking value, but it is the only reasonable default we can infer here, because we must be able to compute the anchor without executing the underlying MDX/React code.
If you want another value, you can use the heading {#headingId} syntax instead to override the default value that we infer.

For the same reason, this value can't be dynamic.

If you want to use a helper for headings, maybe this would work better?
## <Helper title="foo" /> {#foo}

You can also provide your own custom h2/h3/hx component to the MDX provider map: https://docusaurus.io/docs/markdown-features/react#mdx-component-scope


get_account_right sometimes can crash the chef-client

Description
The get_account_right method in Chef::ReservedNames::Win32::Security can randomly crash the chef client. It occurs when getting the rights for both a local user and a system account, i.e. NT AUTHORITY\LOCALSERVICE.
However, I would expect that when get_account_right is called it would not error.
Chef Version
13.6.4
Platform Version

Windows 2012R2
Windows 2016 (No logs attached, but it has the same inconsistent behavior)

Replication Case
The easiest way to replicate is using kitchen, however I can replicate locally on a VM too.
In a recipe I have the following:
user 'dummy_user' do
  password 'SomePassw0rd!'
  action :nothing
end.run_action(:create)

def add_account_right_with_check(account, se_rights)
  puts "checking #{account} has #{se_rights}"
  has_right = Chef::ReservedNames::Win32::Security.get_account_right(account).include?(se_rights)
  puts "has right: #{has_right}"
  return if has_right
  Chef::ReservedNames::Win32::Security.add_account_right(account, se_rights)
  puts "Set #{se_rights} on #{account}"
end

def add_account_right_without_check(account, se_rights)
  Chef::ReservedNames::Win32::Security.add_account_right(account, se_rights)
  puts "Set #{se_rights} on #{account}"
end

1.upto(10) do |i|
  puts "Attempt: #{i}"

  add_account_right_without_check('dummy_user', 'SeServiceLogonRight')
  add_account_right_without_check('NT AUTHORITY\LOCALSERVICE', 'SeServiceLogonRight')
end

1.upto(10) do |i|
  puts "Attempt: #{i}"

  add_account_right_with_check('dummy_user', 'SeServiceLogonRight')
  add_account_right_with_check('NT AUTHORITY\LOCALSERVICE', 'SeServiceLogonRight')
end

puts 'finished'

Since I found the problem with get_account_right, I logged onto a VM, and used puts to find out exactly where was the error.
I found that calling LsaFreeMemory did not work correctly, so I updated the code to:
puts 'before LsaFreeMemory'
LsaFreeMemory(privilege_pointer)
puts 'after LsaFreeMemory'

(See https://github.com/chef/chef/blob/master/lib/chef/win32/security.rb#L201)
Fail on local user
This failed on the 1st call to get_account_right
Client Output
      Chef Omnibus installation detected (13)
      Transferring files to <default-win2012r2>
      Starting Chef Client, version 13.6.4
      resolving cookbooks for run list: ["test_cookbook::default"]
      Synchronizing Cookbooks:
        - test_cookbook (0.1.0)
      Installing Cookbook Gems:
      Compiling Cookbooks...
      Recipe: test_cookbook::default
        * windows_user[dummy_user] action create
          - create user dummy_user
      Attempt: 1
      Set SeServiceLogonRight on dummy_user
      Set SeServiceLogonRight on NT AUTHORITY\LOCALSERVICE
      Attempt: 2
      Set SeServiceLogonRight on dummy_user
      Set SeServiceLogonRight on NT AUTHORITY\LOCALSERVICE
      ...
      # all succeeded
      ...
      Attempt: 10
      Set SeServiceLogonRight on dummy_user
      Set SeServiceLogonRight on NT AUTHORITY\LOCALSERVICE
      Attempt: 1
      checking dummy_user has SeServiceLogonRight
      before LsaFreeMemory
>>>>>> ------Exception-------
>>>>>> Class: Kitchen::ActionFailed
>>>>>> Message: 1 actions failed.
>>>>>>     Converge failed on instance <default-win2012r2>.  Please see .kitchen/logs/default-win2012r2.log for more details
>>>>>> ----------------------
>>>>>> Please see .kitchen/logs/kitchen.log for more details
>>>>>> Also try running `kitchen diagnose --all` for configuration

Kitchen Log
I, [2017-11-16T13:52:51.331454 #2120]  INFO -- default-win2012r2: Attempt: 9
I, [2017-11-16T13:52:51.333455 #2120]  INFO -- default-win2012r2: Set SeServiceLogonRight on dummy_user
I, [2017-11-16T13:52:51.335582 #2120]  INFO -- default-win2012r2: Set SeServiceLogonRight on NT AUTHORITY\LOCALSERVICE
I, [2017-11-16T13:52:51.337697 #2120]  INFO -- default-win2012r2: Attempt: 10
I, [2017-11-16T13:52:51.339744 #2120]  INFO -- default-win2012r2: Set SeServiceLogonRight on dummy_user
I, [2017-11-16T13:52:51.341824 #2120]  INFO -- default-win2012r2: Set SeServiceLogonRight on NT AUTHORITY\LOCALSERVICE
I, [2017-11-16T13:52:51.343958 #2120]  INFO -- default-win2012r2: Attempt: 1
I, [2017-11-16T13:52:51.346138 #2120]  INFO -- default-win2012r2: checking dummy_user has SeServiceLogonRight
I, [2017-11-16T13:52:51.348391 #2120]  INFO -- default-win2012r2: before LsaFreeMemory
E, [2017-11-16T13:52:51.411780 #2120] ERROR -- default-win2012r2: Converge failed on instance <default-win2012r2>.
E, [2017-11-16T13:52:51.411870 #2120] ERROR -- default-win2012r2: ------Exception-------
E, [2017-11-16T13:52:51.411886 #2120] ERROR -- default-win2012r2: Class: Kitchen::ActionFailed
E, [2017-11-16T13:52:51.411896 #2120] ERROR -- default-win2012r2: Message: WinRM exited (-1073740940) for command: [$env:http_proxy = "http://my.proxy.com"
$env:HTTP_PROXY = "http://my.proxy.com"
$env:https_proxy = "http://my.proxy.com"
$env:HTTPS_PROXY = "http://my.proxy.com"
$env:TEST_KITCHEN = "1"
$env:PATH = try {
[System.Environment]::GetEnvironmentVariable('PATH','Machine')
} catch { $env:PATH }

& $env:systemdrive\opscode\chef\bin\chef-client.bat --local-mode --config $env:TEMP\kitchen\client.rb --log_level auto --force-formatter --no-color --json-attributes $env:TEMP\kitchen\dna.json --chef-zero-port 8889 ; exit $LastExitCode]

Fail on NT AUTHORITY\LOCALSERVICE
This failed on the 2nd call to get_account_right
Client Output
      Chef Omnibus installation detected (13)
      Transferring files to <default-win2012r2>
      Starting Chef Client, version 13.6.4
      resolving cookbooks for run list: ["test_cookbook::default"]
      Synchronizing Cookbooks:
        - test_cookbook (0.1.0)
      Installing Cookbook Gems:
      Compiling Cookbooks...
      Recipe: test_cookbook::default
        * windows_user[dummy_user] action create (up to date)
      Attempt: 1
      Set SeServiceLogonRight on dummy_user
      Set SeServiceLogonRight on NT AUTHORITY\LOCALSERVICE
      Attempt: 2
      Set SeServiceLogonRight on dummy_user
      Set SeServiceLogonRight on NT AUTHORITY\LOCALSERVICE
      ...
      # all succeeded
      ...
      Attempt: 10
      Set SeServiceLogonRight on dummy_user
      Set SeServiceLogonRight on NT AUTHORITY\LOCALSERVICE
      Attempt: 1
      checking dummy_user has SeServiceLogonRight
      before LsaFreeMemory
      after LsaFreeMemory
      has right: true
      checking NT AUTHORITY\LOCALSERVICE has SeServiceLogonRight
      before LsaFreeMemory
>>>>>> ------Exception-------
>>>>>> Class: Kitchen::ActionFailed
>>>>>> Message: 1 actions failed.
>>>>>>     Converge failed on instance <default-win2012r2>.
E, [2017-11-16T13:49:02.006787 #1857] ERROR -- default-win2012r2: ------Exception-------
E, [2017-11-16T13:49:02.006802 #1857] ERROR -- default-win2012r2: Class: Kitchen::ActionFailed
E, [2017-11-16T13:49:02.006812 #1857] ERROR -- default-win2012r2: Message: WinRM exited (-1073740940) for command: [$env:http_proxy = "http://my.proxy.com"
$env:HTTP_PROXY = "http://my.proxy.com"
$env:https_proxy = "http://my.proxy.com"
$env:HTTPS_PROXY = "http://my.proxy.com"
$env:TEST_KITCHEN = "1"
$env:PATH = try {
[System.Environment]::GetEnvironmentVariable('PATH','Machine')
} catch { $env:PATH }

& $env:systemdrive\opscode\chef\bin\chef-client.bat --local-mode --config $env:TEMP\kitchen\client.rb --log_level auto --force-formatter --no-color --json-attributes $env:TEMP\kitchen\dna.json --chef-zero-port 8889 ; exit $LastExitCode]

We also hit this in production on Win 2016 when using get_account_right.

Got this on 2k12 R2 as well.  The resource invoked looks like so:
ruby_block 'Assign logon as service rights' do
      block do
        Chef::ReservedNames::Win32::Security.add_account_right(new_resource.username, 'SeServiceLogonRight')
      end
      not_if { Chef::ReservedNames::Win32::Security.get_account_right(new_resource.username).include?('SeServiceLogonRight') }
    end

This will pass when a domain user does not have the access rights enabled but fail for every subsequent converge

A colleague captured the following from the failed run:
PS C:\Windows\system32>
PS C:\Windows\system32> $lastexitcode
-1073740940
PS C:\Windows\system32>

which indicates a heap corruption

Hello All,
I am hitting the same error intermittently. I am using Add_account, do we have any solution for this.

@SATYA995 please open a separate issue and include your version of chef, version of windows, and the stacktrace output of the failure.


Support for Internet Explorer 11

Currently the demo page is not working on IE11. Will this library get support for IE11 in the future?

This library will primarily target modern browsers, however there will be a "legacy" bundle which will support IE11 at least. We are planning to have the support in place by the time we release our first stable version.

Is there an ETA for when that might be?

This may be related. I'm getting the following error in Microsoft Edge:
SCRIPT438: Object doesn't support property or method 'ToString'
draggable.js (1506, 11)

Is anyone aware of this error and/or if the legacy bundle will resolved this?

@heyitsjhu, you have the code on that line that's failing? It looks like it's trying to call ToString, but typically that kind of function is camel case like toString...

Thanks for the quick response @AStoker.
Unfortunately, Edge is only providing me with the line number to the line of code in draggable that's throwing the error. I've tried the same code in Chrome and Safari, where it executes without issues.
I also thought the ToString part was weird but I'm not defining or calling that anywhere in the code that I'm writing (the three lines in my post above). So I search the draggable.js file, but even then nothing showed up in the search. So I'm wondering if it has to do with how Edge is interpreting some of the code? I'm not entirely sure.

@heyitsjhu I ran into the same issue you're seeing in IE 11 with Object.assign. I was able to get a basic example with Sortable working by adding babel-polyfill.

@heyitsjhu, Object.assign is part of the ES6 syntax which isn't supported by IE11, womp womp 😢
You got a reproducible gist we can look at?

Thanks for the bug report 👍 We definitely want to fully support Edge (I may regret this statement 😆 ).
@heyitsjhu Could you state which Edge version you were using? That way I'll try and reproduce, so we can get a fix out for the next beta.
For the IE11 issue with Object.assign, we will change to use spread syntax here, which should get transpiled differently/correctly by babel. This may or may not solve all the IE11 bugs, but IE11 support
I hope that may have answered some more questions. I will keep this issue open until the Edge bugs have been fixed in the next release. Thanks for reporting!

Thanks all!
Yep, turns out the assign() error was resolved with a babel-polyfill script, so everything works fine on IE11 now.
As for Edge, I think it's worth noting that with the polyfill script, I no longer get an error message running the code. However, I am still having issues dragging and dropping rows from a table with touch, but that might be another issue. Yay polyfills!
I'm testing this on Edge 38.14393.0.0.

@AStoker will a jsfiddle link work? http://jsfiddle.net/pjofjg1h/2/
If you open this in Edge, and I'm pretty sure in IE11 too, and then inspect, you should see the error in the console. I hope this helps you guys!

@heyitsjhu ( @tsov, you might want to know this as well ) , it's breaking on Edge for me based off an iterable error.
When you call document.querySelectorAll('.ui-grid-canvas'), this returns a NodeList. Normally, this should be iterable (even though it's not an array), however, Edge and IE don't support an iterable node list. See below:
Compatibility: https://developer.mozilla.org/en-US/docs/Web/API/NodeList#Browser_compatibility
Bug report: https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/5998615/)
Where this comes down to is when we're trying to loop through our containers (which comes from your query selector).
for (const container of this.containers) { //This requires `this.containers` to be iterable
      container.addEventListener('drag:start', this.dragStart, true);
      container.addEventListener('drag:move', this.dragMove, true);
      container.addEventListener('drag:stop', this.dragStop, true);
      container.addEventListener('drag:pressure', this.dragPressure, true);
}

Since this.containers is not iterable, we throw errors and everything breaks. To make it work, containers just needs to be iterable.
That could be done in the source code by using something like Array.from(this.containers) (turns the node list into an array which we can iterate over).
Another approach (until it's fixed in the library) would be to pass in something iterable yourself (like an array).
let iterableContainers = Array.from(document.querySelectorAll('.ui-grid-canvas'));
let swappable = new Draggable.Sortable(iterableContainers, {
    draggable: '.ui-grid-row',
  });

Side note, ES6 doesn't work for IE, so you can use an ES5 way of converting to Arrays like this
Array.prototype.slice.call(document.querySelectorAll('.ui-grid-canvas'));

Hope this helps.

If a PR would help, I can make one. But it seems pretty trivial to solve (and easy enough to work around until then)

@AStoker thanks for explaining the issue 👍 should help anyone who is currently facing this issue!
We are aware of the NodeList issues in IE, but haven't had the time to address it yet.

If a PR would help, I can make one.

🙏 would be greatly appreciated! You can point to the v1.0.0-beta.3 branch

@tsov, getting a branch set up to make this fix. There is a failing test for Swappable.
$ jest --config config.json
 PASS  src/Draggable/Sensors/MouseSensor/tests/MouseSensor.test.js
 PASS  src/Draggable/tests/Draggable.test.js
 FAIL  src/Swappable/tests/Swappable.test.js
  ● Swappable › triggers `swappable:swap` event with `swappedElement`

    TypeError: Cannot read property '0' of undefined

      at Object.<anonymous> (src/Swappable/tests/Swappable.test.js:44:55)
          at Promise (<anonymous>)
      at Promise.resolve.then.el (node_modules/p-map/index.js:42:16)
          at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

Test Suites: 1 failed, 2 passed, 3 total
Tests:       1 failed, 1 skipped, 7 passed, 9 total
Snapshots:   0 total
Time:        5.535s
Ran all test suites.

That was from a fresh download. Is that expected? Want to make sure I'm starting from a valid place.

@AStoker thank you for looking into it! I am guessing this is on the v1.0.0-beta.3 branch? Because it's currently failing, I am planning to address the current test failures today.
I did however include a commit for beta.3 that introduces a very simple legacy bundle to use with older Browsers and earlier Edge versions. I was able to get Draggable working in IE11 on beta.3, but haven't tested for IE10 yet.
Here the commit that introduces core-js for index.legacy.js: https://github.com/Shopify/draggable/commit/4b9e8cc2e2812689697ccafc751765c7c83495ee

@Tsov, did core-js solve the issue with the NodeList? Wouldn't have expected that. If so, no need for me to make a PR and this issue could be closed. Just want to make sure first.

Actually, just took a look at the beta.3 branch, looks like everything should work in regards to the containers and node lists
https://github.com/Shopify/draggable/blob/4b9e8cc2e2812689697ccafc751765c7c83495ee/src/Draggable/Draggable.js#L78
No need for a PR

We now include a legacy bundle that supports IE11:
import {Draggable} from<EMAIL_ADDRESS>
<script src="https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>
Closed via: https://github.com/Shopify/draggable/releases/tag/v1.0.0-beta.3

Hate to drag this back up but i think an update has broken IE 11 support maybe? I cant use anything higher than Beta 4. I can use Beta 5 if I also use babel-polyfill, which isnt ideal but i can't get 6 or 7 working no matter what I do.
With Beta 6 I get SCRIPT1010: Expected Identifier - draggable.bundle.legacy.min.js (1,15223)
Withe Beta 7 I get SCRIPT1010: Expected Identifier - draggable.bundle.legacy.min.js (1,15405)

@Harry-Harrison, looks like
const f={"sortable:sorted":function({dragEvent:e}){...
____________________________________^^

is your breaking line.
From the source, it's this constant definition: https://github.com/Shopify/draggable/blob/9f54956d24da310f7c002682fd120738096c6a7c/src/Sortable/Sortable.js#L38
which calls this function:
https://github.com/Shopify/draggable/blob/9f54956d24da310f7c002682fd120738096c6a7c/src/Sortable/Sortable.js#L15
Do you have a fiddle that demonstrates this? I can't be certain off the top of my head, but I think it's trying to use an ES6 object definition as the function parameter.

@AStoker, here is a jsFiddle demonstrating the issue in IE 11: https://jsfiddle.net/znm1d8vc/1/embedded/result,css,html,js
Let me know if you need anything else that will help; we are running into the same issue!


Regarding different hash value getting in two different method of API

Please see the below screen shot.

Please give an answer as soon as possible. i really need it urgently.

Please give us a solution as soon as possible. Any help is worth for us.

Please give us a solution as soon as possible. Any help is worth for us.

Ok, but can you tell me how to Start or Build Javascript Repo in Windows environment.

IotaApi.cs, line 592
foreach (var tryte in trytesResponse.Trytes) trxs.Add(new Transaction(tryte, _curl));
to
foreach (var tryte in trytesResponse.Trytes) trxs.Add(new Transaction(tryte));


Build Error

I am using the below command to build, as specified in the Readme.
"./gradlew :library:build"
But I am facing the below error. Attached are the logs.
CMakePlugin.cmakeConfigure - ERRORS:
CMake Error in CMakeLists.txt:
Target "anjay" contains relative path in its INTERFACE_INCLUDE_DIRECTORIES:
"MBEDTLS_INCLUDE_DIR-NOTFOUND"
Please let me know how to resolve this error and successfully build the jar file and the native shared library.
Log.txt

Hi,
following line from the log file you provided:
-- Could NOT find mbedTLS (missing: MBEDTLS_INCLUDE_DIR MBEDTLS_LIBRARY MBEDTLS_CRYPTO_LIBRARY MBEDTLS_X509_LIBRARY MBEDTLS_LIBRARIES MBEDTLS_VERSION)

suggest that you don't have mbedTLS installed.
Please install it by executing the command appropriate for your operating system, as described here: https://github.com/AVSystem/Anjay#dependencies

Hi, following line from the log file you provided:
-- Could NOT find mbedTLS (missing: MBEDTLS_INCLUDE_DIR MBEDTLS_LIBRARY MBEDTLS_CRYPTO_LIBRARY MBEDTLS_X509_LIBRARY MBEDTLS_LIBRARIES MBEDTLS_VERSION)

suggest that you don't have mbedTLS installed. Please install it by executing the command appropriate for your operating system, as described here: https://github.com/AVSystem/Anjay#dependencies

Hi. What if I do not want to include mbedTLS in my module? What all changes will be required in that case?

Hi,
Anjay-java was designed to be compiled with security enabled by default. You can compile-in the TLS library, but keep using basic NoSec communication in the runtime. What holds you back from including mbedTLS into the library?

Thank you for your reply. Ya currently I am using NoSec communication.
Can I get some information on Anjay Java's documentation, if any? I see that in Anjay C, there is a complete document. It would be really helpful if we get all 1:1 API mappings and other details in one document.

Hi, I am trying to compile "libanjay-jni.so" separately. And I am seeing this error.
_[100%] Linking CXX shared library libanjay-jni.so
/usr/bin/ld: CMakeFiles/anjay-jni.dir/src/native_access_control.cpp.o: relocation R_X86_64_PC32 against undefined symbol `ZZN3jni16MakeNativeMethodIZNS_17NativeMethodMakerIMZNKS_16NativePeerHelperI19NativeAccessControlS3_FSt10unique_ptrIS3_St14default_deleteIS3_EER7JNIEnv_RNS_6ObjectI11NativeAnjayEEEE15MakeInitializerERKNS_5FieldIS3_lEEPKcPSE_EUlS9_RNSA_IS3_EESD_E_KFvS9_SO_SD_EJEEclISP_EEDaSL_RKT_EUlPS8_PNS_7jobjectESZ_E_EEDaSL_SL_SW_PNSt9enable_ifIXsrSt8is_classISU_E5valueEvE4typeEE6method' can not be used when making a shared object; recompile with -fPIC
/usr/bin/ld: final link failed: Bad value
collect2: error: ld returned 1 exit status
CMakeFiles/anjay-jni.dir/build.make:511: recipe for target 'libanjay-jni.so' failed
make[2]: *** [libanjay-jni.so] Error 1
CMakeFiles/Makefile2:81: recipe for target 'CMakeFiles/anjay-jni.dir/all' failed
make[1]: *** [CMakeFiles/anjay-jni.dir/all] Error 2
Makefile:151: recipe for target 'all' failed
make: *** [all] Error 2
Can you please help me address this issue?

Hi Kushma,
We cannot reproduce your issue. The library should compile cleanly when configured through just cmake . (or cmake -DJAVA_HOME=<path_to_your_jdk> . if your JDK isn't available in its default location) and it does so on my system.
The CMAKE_POSITION_INDEPENDENT_CODE variable is set in the project's CMakeLists, which should set the -fPIC flag mentioned in the warning appropriately - so it looks that CMake isn't handling your system properly for some reason. This is probably an OS-specific issue.
What operating system are you using? If it's a Linux system, please tell us what distribution in which version it is. Please also tell us the exact command you're using to invoke cmake, and the versions of CMake and JDK if they're not the default ones for the distribution in question.

Hello, thank you for the reply.
I am sticking to using " ./gradlew -Pandroid :library:build" command to build.
I observed that when I change the DTLS_BACKEND to another value other than mbedtls in "native-library/deps/anjay/CMakeLists.txt", and give a build. The  DTLS_BACKEND value is not updating in my CMakeCache.txt.
I also observed that the DTLS_BACKEND variable is not getting updated, confirmed from the gradle logs.
Can you pls explain why this is happening? Is there a way I can give a clean build using gradlew?

Hi Kushma,
The values specified in CMakeLists.txt for so-called "cache variables" are just defaults, see CMake documentation for `set(... CACHE ...)](https://cmake.org/cmake/help/latest/command/set.html#set-cache-entry) and [`option()`.
The specific rules for variable handling in CMake are rather complex, but the general rule is that variables specified on the command line (i.e. cmake -DVAR=value - within the context of Anjay-java this is specified in the Gradle scripts) have the highest priority, then the values already in CMakeCache.txt, if any, then the defaults in CMakeLists.txt. This is designed so that re-running cmake will not change the configuration that you have already set.
Removing CMakeCache.txt (note that there may sometimes be more than one in different directories) is the way to go if you want to reset to the default settings.

Hello, thank you for the clarification.
From Anjay Java to Anjay C++ there is a JNI Iayer and I was working on writing a callback from native back to java. For this, I will need both "JNIEnv *" and "jobject" in arguments but I only see JNIEnv.
Can you please let me know how I can go ahead with writing this JNI callback? And is there a callback in the existing code for reference?

Hi,
I have added another comment today. Can you please check and get back to me?
https://github.com/AVSystem/Anjay-java/issues/3
Looking forward to hearing from you.
Thank you,
Kushma
On Mon, Oct 17, 2022 at 6:27 PM kFYatek @.***> wrote:

Hi Kushma,
The values specified in CMakeLists.txt for so-called "cache variables"
are just defaults, see CMake documentation for set(... CACHE
...)](https://cmake.org/cmake/help/latest/command/set.html#set-cache-entry)
and [option()` https://cmake.org/cmake/help/latest/command/option.html.
The specific rules for variable handling in CMake are rather complex, but
the general rule is that variables specified on the command line (i.e. cmake
-DVAR=value - within the context of Anjay-java this is specified in the
Gradle scripts
https://github.com/AVSystem/Anjay-java/blob/c8f8c5e0ac5a086db4ca183155eed07374fc6585/library/scripts/native.gradle#L30)
have the highest priority, then the values already in CMakeCache.txt, if
any, then the defaults in CMakeLists.txt. This is designed so that
re-running cmake will not change the configuration that you have already
set.
Removing CMakeCache.txt (note that there may sometimes be more than one
in different directories) is the way to go if you want to reset to the
default settings.
—
Reply to this email directly, view it on GitHub
https://github.com/AVSystem/Anjay-java/issues/3#issuecomment-1280819189,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACWPSBKZNHUZK3X7T3IYYXLWDVEKRANCNFSM55JYZH3Q
.
You are receiving this because you authored the thread.Message ID:
@.***>


Fix control message decoding and add support for ScmCredentials

While https://github.com/nix-rust/nix/pull/918 fixed the encoding, the decoding done by the CmsgIterator still remained broken when multiple messages are received. However, since nix didn't support any control message that could reliably be sent twice in one sendmsg call, this couldn't be tested properly.
This PR addresses this by adding SCM_CREDENTIALS support and testing all of this by passing both an SCM_CREDENTIALS and a SCM_RIGHTS message at the same time.
I've also verified that the resulting encoding is the same as for roughly equivalent C code.

Any idea why Travis didn't build this PR?

I added [ci skip] to the last commit as it was just updating the changelog. This causes Travis to ignore the commit which saves CI time.

Uh, now the old test fails? That's odd.

I think the issues are in qemu. I haven't managed to get any socket tests failing on my Raspberry Pi 1 (armv6) or 2 (armv7). Additionally, I wrote a test program in C (https://gist.github.com/jonas-schievink/cb6e6584a055539d2113f22d91068e2d) that works fine. The control message encoding for nix and the test program is identical, and it's the kernel (or qemu) that strips out the second message, not our decoder doing weird stuff.
I'm going to build a smaller reproducer and report the bug to qemu, then #[ignore] the tests here.

@asomers Alright, after a long journey this might be finally done (except the commit message for
386cd04 is wrong - but I'll fix that when squashing later). Care to take another look?

@jonas-schievink please squash this; then I'll merge it.

@asomers This all look good? You asked for it to be squashed, which is has been. We could split this up into two commits with the other one added the as_raw implementations for Pid and friends, but it's a minor nit, so I'm fine not worrying about it.

bors retry


ARROW-7054: [Docs] Enable spoofing project version when building Sphinx docs

This is a workaround for ARROW-7053, but it can also help with adding new sections to the docs in between releases when needed.

Codecov Report

Merging #5766 into master will decrease coverage by 23.29%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5766      +/-   ##
==========================================
- Coverage      89%   65.71%   -23.3%
==========================================
  Files        1006      543     -463
  Lines      137493    77118   -60375
  Branches     1501        0    -1501
==========================================
- Hits       122381    50675   -71706
- Misses      14747    26443   +11696
+ Partials      365        0     -365

Impacted Files
Coverage Δ

cpp/src/arrow/util/memory.h
0% <0%> (-100%)
:arrow_down:

cpp/src/parquet/internal_file_decryptor.h
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/filesystem/path_tree.h
0% <0%> (-100%)
:arrow_down:

cpp/src/gandiva/date_utils.h
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/util/memory.cc
0% <0%> (-100%)
:arrow_down:

cpp/src/gandiva/decimal_type_util.h
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/compute/logical_type.h
0% <0%> (-100%)
:arrow_down:

cpp/src/arrow/compute/kernels/add.h
0% <0%> (-100%)
:arrow_down:

cpp/src/parquet/hasher.h
0% <0%> (-100%)
:arrow_down:

cpp/src/gandiva/basic_decimal_scalar.h
0% <0%> (-100%)
:arrow_down:

... and 730 more

Continue to review full report at Codecov.

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


Add new JSON for wioTerminal project

Thank you for contributing to the Azure IoT Plug and Play Models repository
This project has adopted the Microsoft Open Source Code of Conduct. For more information see the Code of Conduct FAQ or contact<EMAIL_ADDRESS>with any additional questions or comments.
The repository pipeline will ensure minimum validation of incoming DTDL models.

PR validation steps are described in the tools Wiki.
The same validation can be run locally via the dmr-client command line tool.

Requested info template for model(s) submission
When submitting models to the repository we ask that you provide as much of the following meta information around your models and related devices as possible. This info will be used to improve Plug and Play.
:star2: Please replace the markdown comment examples with your own values.
Company Info
Seeed Studio

IoT Plug and Play Device Info
Wio Terminal

Model Submission Goals
Add new JSON for Wio terminal project

Code Owners & Reserved Names
N/A

Hi @inesying,
Can you resolve the conflicts?

Hi @inesying,
Can you resolve the conflicts?

Hi rido
Sorry for long waiting, I will solve it.


pause / stop broken

fixed. regression from Argus widget rework
other: stopping plannar throws exception resulting in "Planner thread crashed" message. Not best UX we should indicate this exception was expected


:bug: fix storage record pagination with post-filter query params

Related to: https://github.com/hyperledger/aries-cloudagent-python/issues/3001#issuecomment-2206110085
Marking as draft to first validate things work as expected (testing here: https://github.com/didx-xyz/aries-cloudapi-python/pull/850)

Oof, I was just about to commit a change to make it more readable 😅

No prob, I'll make another PR with some minor amendments, also improving the error handling which is prob worth it

Ok. Sorry about that. Another PR will be good.

All good, I was slow 😄 included the changes in #3083


Empty API Mock editor

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v18.14.0
Amplify CLI Version
10.7.0
What operating system are you using?
Ubuntu 22.04 on WSL2 on Windows 10
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
Yes, the changes are extensive.
Describe the bug
I ran apt-get install java-11-amazon-corretto-jdk to install Java.
Running amplify mock api launches Amplify GraphiQL Explore on localhost:20002, but it's empty saying "No Schema Available".
The CLI output is:
vscode ➜ .../panel-api/project/amplify/backend (features/VCD-504-ecv-api ✗) $ amplify mock api
GraphQL schema compiled successfully.
Edit your schema at /workspaces/gene-panel-api/project/amplify/backend/api/genepanelapi/schema.graphql or place .graphql files in a directory at /workspaces/gene-panel-api/project/amplify/backend/api/genepanelapi/schema
Mock does not handle CloudFormation resource of type AWS::IAM::ManagedPolicy. Skipping processing resource AuthRolePolicy01.
Mock does not handle CloudFormation resource of type AWS::IAM::ManagedPolicy. Skipping processing resource AuthRolePolicy02.
Mock does not handle CloudFormation resource of type AWS::IAM::ManagedPolicy. Skipping processing resource AuthRolePolicy03.
Data source of type RELATIONAL_DATABASE is not supported by local mocking. A NONE data source will be used.
Data source of type HTTP is not supported by local mocking. A NONE data source will be used.
Data source of type HTTP is not supported by local mocking. A NONE data source will be used.
Data source of type HTTP is not supported by local mocking. A NONE data source will be used.
Could not find ref for "RdsPanelDbName". Using unsubstituted value.
Could not find ref for "RdsPanelDbName". Using unsubstituted value.
Running GraphQL codegen
✔ Generated GraphQL operations successfully and saved at ../../src/graphql
AppSync Mock endpoint is running at http://<IP_ADDRESS>:20002
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Re-Trying with a new shard
Mock DB deletion detected. Clearing the OpenSearch indices...
Re-Trying with a new shard
Re-Trying with a new shard
Failed to start API Mocking. Running cleanup tasks.
MockProcessFault: Failed to start API Mocking.. Reason: null
at APITest.start (/snapshot/repo/build/node_modules/amplify-util-mock/lib/api/api.js:151:13)
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:95:5)
at async start (/snapshot/repo/build/node_modules/amplify-util-mock/lib/api/index.js:18:5)
at async Object.run (/snapshot/repo/build/node_modules/amplify-util-mock/lib/commands/mock/api.js:21:5)
at async Object.executeAmplifyCommand (/snapshot/repo/build/node_modules/amplify-util-mock/lib/amplify-plugin-index.js:47:3)
at async executePluginModuleCommand (/snapshot/repo/build/node_modules/@aws-amplify/cli-internal/lib/execution-manager.js:142:5)
at async executeCommand (/snapshot/repo/build/node_modules/@aws-amplify/cli-internal/lib/execution-manager.js:40:9)
at async Object.run (/snapshot/repo/build/node_modules/@aws-amplify/cli-internal/lib/index.js:117:5) {
classification: 'FAULT',
options: {
message: 'Failed to start API Mocking.. Reason: null',
link: 'https://docs.amplify.aws/cli/graphql/troubleshooting/'
},
downstreamException: undefined,
toObject: [Function (anonymous)],
details: undefined,
resolution: undefined,
code: undefined,
link: 'https://docs.amplify.aws/cli/graphql/troubleshooting/'
}
Expected behavior
I expected to see the API in the explorer.
Reproduction steps
Run 'amplify mock api'
Project Identifier
9c19ea5526c1afd0a3e1987ea44f96af
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey folks I'm able to reproduce this issue using the Todo schema. Marking as a bug

Hi, there is an ETA for the resolution of this bug?

@josefaidt. We are still experiencing this issue. Any update on the fix?

We are experiencing the same issue on 11.0.5.

Checking for any updates on this bug. Still exists in 12.1.0

Hey folks I'm able to reproduce this issue using the Todo schema. Marking as a bug

Hi Josef, are you able to resolve the bug or provide a walkaround? My team has been stuck here for too long, any help would be greatly appreciated!
Best regards!

Hi josefaidt.
Thank you for your response.
You have transfered this issue to here.
But the code has probrem is not in this repository. It is in amplify-cli repository.
Please check my PR on amplify-cli repository. https://github.com/aws-amplify/amplify-cli/pull/12906/commits/ab4acb731c625b9578cbe8a9b90a8cc055dcf560
Best regards.

@josefaidt Hi josefaidt.
Thank you for your response.
You have transfered this issue from amplify-cli repository to here.
But the code has probrem is not in this repository. It is in amplify-cli repository.
Please see my PR in amplify-cli repository.
amplify mock is very useful and important function for developers. A lot of developers are stuck in this bug for months.
I would appreciate it if you could respond promptly.
Best regards,

I found that the error directly comes from adding Cognito User Pool auth to the api, no matter if API or IAM is also enabled. Working with amplify -v 12.0.0 since amplify mocks fail completely on m2 Mac using the latest version of the cli.

Im using -v 12.2.0 and seeing the same issues. I am seeing the following error in the console:
"Error: Error when generating OIDC token: Right-hand side of 'instanceof' is not an object"

This was working fine earlier today, not sure what changed recently.

Any updates ? Same issue with Amplify CLI version 12.3.0.

I have confirmed that it also occurs in v12.4.0

Still occurs in v12.7.0.

It's been months and still not able to see anything on Amplify GraphiQL Explorer.
There is this Uncaught (in promise) Error: Error when generating OIDC token: invalid 'instanceof' operand g error that everybody mentioned on my browser console.
Also, when I try to execute a query, I always get the unauthorized error:
{
  "errors": [
    {
      "errorType": "UnauthorizedException",
      "message": "UnauthorizedException: Missing authorization"
    }
  ]
}

As mentioned by others developing and testing locally is very important and without the development process is far slower. @josefaidt Are there any updates?

Hi everyone, this issue was opened almost 1 year ago. It is NOT a MINOR bug. There are teams around the world that are not able to use the latest version of Amplify just because of this problem. Could you help us, @josefaidt, @alharris-at ?

I would like to share my experience, maybe will help someone.
Yes, I'm having the same issue as everyone. But, given that I use nvm to control my node versions, I "manage" to deal with this problem installing different amplify-cli version on different node version as mentioned here.
nvm use 20
npm install npm install -g @aws-amplify

nvm use 18
npm install -g<EMAIL_ADDRESS>// working version

Also, I do agree with @kmy90. It should not be treated as a minor thing.

Hey folks, this is a serious issue here impacting everyone and disrupting the development process. Using workarounds, like downgrading the Amplify version, isn't the most convenient solution. Personally, I struggled with testing my projects locally using the explorer, and it really bothered me that such an important issue had been left unattended for a year.
I fixed it by creating a PR (https://github.com/aws-amplify/amplify-cli/pull/13555). Can you please support it? We need the Amplify team to review and approve the fix. Thanks!

Still occurs in v12.10.1.

Fixed in version 12.11.0


Manual-test: Change height to 1200px height in iframe-apprtc.html

Currently the height is set to "90%" which does not work, need to specify in pixels instead.

samdutton, PTAL.


#723: [Bug] Unable to get Group epoch

#723: [Bug] Unable to get Group epoch

@raphaelrobert  Done this diff just have one bug fix.

Still getting used to rust ;p thanks for the review and prompt replies.


Enable full linting with errors. Use SuppressWarning explicit where n…

…eeded
@hmusum or @bjorncs PR
Contains the safe parts of #30836

Indeed, accident due to cleaning branch first. Fixed.

@bjorncs Fixed


docs: email documentation example for SMTP setup

The SMTP code example had a typo and was using the wrong environment variable to set the port for the email plugin
Description

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] Chore (non-breaking change which does not add functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Change to the templates directory (does not affect core functionality)
[ ] Change to the examples directory (does not affect core functionality)
[ ] This change requires a documentation update

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Existing test suite passes locally with my changes
[x] I have made corresponding changes to the documentation

Thanks @BraianPita


Use a consistently seeded Rng for benchmark stability

I was seeing a lot of variance in benchmark results, and I noticed that
we were using SmallRng::from_entropy(). Now we use a fixed seed value,
so we test the same input data every time.

This only affects the benchmarks, and CI passed, so I'll go ahead and merge.

Thanks, good catch.


How can I export all image from this extension?

Hello,
I would like to export all images from this extension for backup, is there any solution for this requirement? thanks.

you can open the teams in  browsers ,
then right click the stikcer icon,
next open settings page , and save the images one by one.

Hi NewFuture,
Thanks for the reply, I found a quick solution base on your suggestion.
After opening the settings page from the web browser, we can press ctrl+s to store full html & resources, the all images and html, css will be stored on the target folder.
btw, if the image is added from other people sticker rather than import by yourself, the store image file name will be imgo, imgo(1), imgo(2)..., need to rename it manually (add .jpg)


Autocomplete for cities #6

Added auto complete for cities search. User will be shown suggestions based on what city name he wants to provide.
https://user-images.githubusercontent.com/61612477/180254007-309373e1-b47a-4d6b-9430-32eb3f2209ae.mp4

@gibbonhug can you please follow this PR, give feedback and merge when ready?


Homepage search box bugs

The homepage search box at the very top of https://www.openclimate.network/ has these three bugs (I can split these up if preferred):

If I type something and hit Enter it says "Thank you! Your submission has been received!" (not what I expect when searching. I wasn't entering a "submission"! Is there a database slowly filling with search nonsense from people like me?!)

If I type something and hit "Explore" green button on the right, nothing happens. No error in the js console.

(Minor) If I start typing more than three letters then it begins showing search-as-you-type matches below the box, but the first search I tried was "UK" (Awkward I know, but maybe some super-short matches could be pre-programmed)

@hwood-vivup thank you so much for the feedback, we will definitely review them!


Component's name should be automatically generated by route.name for keep-alive

Description
It was inspired by https://github.com/vuejs/core/issues/4993
I want to create a new plugin to implement it for this I need to export the 'getRouteMap' api.
Or implement it directly in the current plugin. like this: https://github.com/hannoeru/vite-plugin-pages/pull/323 .
What do you think?

Names are already generated. You can also override them in multiple ways shown in the readme
I don’t understand

In my application, I must manual specify component's name use unplugin-vue-macros  :
<script setup>
defineOptions({
  name: '/blogs'
})
</script>

but, when I update the file structure, I also need to multiple update component's name.
so I want create a plugin to  automatically generate component's name.

aah, usually vue already generates the name of the component based on the filename, if you want to do something different, I believe this should be declared locally, in each component.
If you want, you can create your own vite plugin to automatically generate this part for you but keep in mind this problem is simple for common cases but gets more complicated with others (already declared export default (), etc) and the complexity added in the end is probably not worth it given how easy it is to add a name to a component

so how use keep-alive includes field without define component name?
in this case we need a component name

defineOptions({
name: 'home',
})
definePage({
name: 'home',})

I have to do this. otherwise router cache is no work.


Bjerksund-Stensland 1993 option pricing model

Motiviation/Rational
While the intent is to add both of the Bjerksund-Stensland option pricing models for american exercise options, this pull request is for the 1993 model, with the 2002 model to follow shortly after.
Testing
The primary source used is the description of the 1993 model in the paper on the 2002 model here.  Included in the paper are four tables with values for the 1993 model (referenced as "flat boundary approximation").  A test was run checking all of the values (source code can be provided upon request):
Closed Form Valuation of American Options, 2002
Table 1, pg. 9 - 1993 model
Section 1, Calls:  [ 0.03  0.57  3.49 10.32 20.  ]  Puts:  [20.41 11.25  4.4   1.12  0.18]
Section 2, Calls:  [ 0.03  0.57  3.46 10.29 20.  ]  Puts:  [20.22 11.14  4.35  1.11  0.18]
Section 3, Calls:  [ 1.05  3.25  7.37 13.47 21.23]  Puts:  [21.44 13.91  8.27  4.52  2.29]
Section 4, Calls:  [ 0.21  1.34  4.65 10.94 20.  ]  Puts:  [20.95 12.63  6.37  2.65  0.92]
Table 2, pg. 10 - 1993 model
Section 1, Calls:  [ 0.05  0.85  4.44 11.66 20.9 ]  Puts:  [20.   10.19  3.51  0.78  0.11]
Section 2, Calls:  [ 0.05  0.84  4.4  11.55 20.69]  Puts:  [20.   10.17  3.49  0.77  0.11]
Section 3, Calls:  [ 1.29  3.82  8.35 14.8  22.71]  Puts:  [20.53 12.91  7.42  3.93  1.93]
Section 4, Calls:  [ 0.41  2.18  6.5  13.42 22.06]  Puts:  [20.   10.7   4.7   1.71  0.52]
Table 3, pg. 11 - 1993 model
Section 1, Puts:  [20.   10.01  3.16  0.65  0.09]
Section 2, Puts:  [20.   10.    2.86  0.54  0.07]
Section 3, Puts:  [20.28 12.48  7.04  3.66  1.77]
Section 4, Puts:  [20.   10.24  4.11  1.37  0.39]
Table 4, pg. 12 - 1993 model
Section 1, Calls:  [ 2.3   4.71  8.44 13.74 20.85]  Puts:  [25.61 20.04 15.47 11.78  8.87]
Section 2, Calls:  [ 3.95  7.2  11.64 17.24 23.93]  Puts:  [22.12 16.14 11.64  8.31  5.89]
Section 3, Calls:  [16.75 22.25 28.31 34.84 41.77]  Puts:  [28.42 23.97 20.34 17.34 14.86]
Section 4, Puts:  [20.   11.67  6.9   4.12  2.48]

All of the values match, with the only exception being those in Table 4, Section 3.  Confirming the values using online calculators, such as this one here, led to the conclusion that the values in Table 4, Section 3 are not correct - but I would appreciate the input of others.
This PR is in draft status as I still need to add tests to american_option_test.py.

Tests based on the first 3 out of four tables in the reference paper have been added and passed when run in bazel.  Some minor adjustments needed to be made to the way in which comparisons are done in the main function, as TensorFlow doesn't allow tensors to be compared like python bools when not in eager execution mode.
As everything has been added, the PR will be taken out of draft mode as it is ready for review.

Adjusted the notation to precisely reflect the reference document, added comments and added tests for the 4th table in the reference document, but commented out the tests for section 3 as the values in the reference document are suspect.

Hi Cyril,
Resolved most of the comments, only two I have questions regarding.
I squashed my commits together and pushed.  Let me know if I did it wrong, as this is not something I have done before.

Pushed a commit with just the changes to modified_boundary, but can squash together with the other commits if you prefer.

Sorry for the delay. Please notice I have removed cost_of_carries arg as it is ambiguous in general and for our needs is expressed via dividend_rates and discount_rates.


[Tests] - Ecommerce Module

The ecommerce modules needs tests adding

Hello,
I'll like to work on this. Could I get assigned?
Thanks.

Hello,
For this endpoint, I noticed that the openapi definitions are different from what the endpoint returns.
Should I update the openapi definitions or update the endpoint to return what the openapi definitions specify?
Thanks.

Hi @edwardsmoses, thank you for the tests you added, If you want to do another pull for updating the OpenAPI definitions. I would really appreciate the help. 👍

Hi @ageddesi ,
Thanks - I'll get on updating the OpenAPI definitions.

Hello @ageddesi ,
is this issue resolved, if not i would like to contribute on this.

The ticket seems to be open still so please feel free to tackle this.

Thanks  @ageddesi , could you assign me this issue, i'll submit a PR. Also  one question does this count as a hacktoberfest contribution


Port gateway updates from notebook (PRs 5317 and 5484)

This pull request brings over the changes relative to the following Notebook PRs:

https://github.com/jupyter/notebook/pull/5317: Gateway only: Ensure launch and request timeouts are in sync
https://github.com/jupyter/notebook/pull/5484: Added functionality to allow filter kernels by Jupyter Enterprise Gat…

Thanks, @kevin-bates


⚠ Samsung has degraded performance

In af6595a, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3013 ms

Resolved: Samsung performance has improved in 5873493.


i have question about dpo training process (just simple question)

When using ppo, ref_model was frozen.
Should I also freeze ref_model when using dpo?
I checked DPOTrainer, and when ref_model == None, it was frozen after model deepcopy.
When I was training before, I accidentally entered ref_model as a variable in DPOTrainer and found that it trained well without required_grad=False. (Strange, but it trained well).
This time, I train without inputting ref_model as a variable to DPOTrainer (input None), and the loss does not drop.
What could be the problem?
Is it correct that dpo does not freeze ref_model?

It sounds weird, but we use the ref_model to keep the output logits close to the ref_model. If we update the parameters in the ref_model, it's hard to determine whether the model has learned better, or if the ref_model's output is simply closer to the dpo model , rather than what we proposed.


filePath = N/A when deploying multiple files (with errors) using source:deploy -p

Summary
When deploying multiple files with force:source:deploy and at least one file has a deploy error, the filePath is N/A in the error results.
Steps To Reproduce:

Create two apex files (Apex1.cls, Apex2.cls)
Change one file (Apex2.cls) so that is will not compile
Deploy using force:source:deploy -p Apex1.cls,Apex2.cls --json

Expected result
The result filePath should indicate which file has the compile error (Apex2.cls).
Actual result
filePath is N/A and fullName is missing. There is no indication of which file the error is related to.
Additional information
So here's how I'm using sfdx. I have a plugin that finds all changed files via git status and deploys them to my org using sfdx force:source:deploy -p file1,file2,etc --json. When I deploy a file with a compile error I'd like to know which file has which error so I can fix it quickly. As a workaround, the best I can do is compare a list of all files deployed to those in the result.partialSuccess array and output the complement set. This at least tells me which files I need to check. It would be much better if I knew which error is related to which file.
Screenshot of JSON when deploying multiple files.

The filePath and fullName are available when only a single file is deployed.
Screenshot of JSON when deploying a single file.

SFDX CLI Version: sfdx-cli/6.56.0 (darwin-x64) node-v11.7.0
SFDX plugin Version: salesforcedx 45.5.0 (core)
OS and version: OSX Mojave 10.14.3 (18D109)

@gbutt can you please upgrade to the most recent version of the CLI? We just updated to V7

@clairebianchi I can confirm that this is still an issue on sfdx-cli/7.93.1-762bca056d linux-x64 node-v14.15.4.
Can we re-open this issue? I've tried the workaround that @gbutt suggests, but it does not seem to do the trick. We have a generated LWC library with 50+ components that we push to Salesforce and debugging errors without filePath, type, columnNumber nor lineNumber is quite a challenge.
sfdx force:source:deploy --sourcepath salesforce/main/default/lwc/ --json
{
   "status":1,
   "result":[
      {
         "columnNumber":"0",
         "lineNumber":"0",
         "error":"lightningCommunity__Default is not a flow related target! (0:0)",
         "type":"",
         "filePath":"N/A",
         "problemType":"Error"
      }
   ],
   "name":"DeployFailed",
   "message":"Deploy failed.",
   "exitCode":1,
   "actions":[

   ],
   "commandName":"SourceDeployCommand",
   "data":[
      {
         "columnNumber":"0",
         "lineNumber":"0",
         "error":"lightningCommunity__Default is not a flow related target! (0:0)",
         "type":"",
         "filePath":"N/A",
         "problemType":"Error"
      }
   ],
   "stack":"DeployFailed: Deploy failed.\n    at /home/node/.local/share/sfdx/client/7.93.1-762bca056d/node_modules/salesforce-alm/dist/lib/source/sourceApiCommand.js:70:31\n    at async SourceDeployCommand.execLegacyCommand (/home/node/.local/share/sfdx/client/7.93.1-762bca056d/node_modules/salesforce-alm/dist/ToolbeltCommand.js:149:29)\n    at async SourceDeployCommand.run (/home/node/.local/share/sfdx/client/7.93.1-762bca056d/node_modules/salesforce-alm/dist/commands/force/source/deploy.js:32:16)\n    at async SourceDeployCommand._run (/home/node/.local/share/sfdx/client/7.93.1-762bca056d/node_modules/@salesforce/command/lib/sfdxCommand.js:81:40)\n    at async Config.runCommand (/home/node/.local/share/sfdx/client/7.93.1-762bca056d/node_modules/@oclif/config/lib/config.js:173:24)\n    at async Main.run (/home/node/.local/share/sfdx/client/7.93.1-762bca056d/node_modules/@oclif/command/lib/main.js:27:9)\n    at async Main._run (/home/node/.local/share/sfdx/client/7.93.1-762bca056d/node_modules/@oclif/command/lib/command.js:43:20)\n    at async Object.run (/home/node/.local/share/sfdx/client/7.93.1-762bca056d/dist/cli.js:121:21)",
   "warnings":[

   ]
}


Makes the MapIs type alias in "collections/persistent"  be an alias to the persistent HashMap class, not the mutable one.

Thanks!


[feature request] optional button

please add the feature of a Button that can be option on any slide.
like "for more info press" -> Button
with onClickListener
something like this "get started" button:

This won't happen anytime soon -- since the library already allows you to create your own fragments, so there are no limits to how many button you can have on your slides...
If you feel that its inclusion is a must, submit a PR.

i am not sure why i would need to use the library, if i need to code everything myself manually?

@zoff99 the app intro library is a bare-bones library that allows you to pass text and images to it to create a simple introduction to your app. If you desire custom layouts and custom buttons, the library supports that too. The library makes it easier to do such things, so I don't see why you don't see the need for it.

i have done it totally manual now, without any library. which was less complicated and also works down to API9 (2.3.x)
i used this page. takes 10 minutes and you're set.

http://www.androidhive.info/2016/05/android-build-intro-slider-app/

@zoff99 if your intention was to prove this library useless, I'd say that's kind of a dick move. If you find it easier to code everything yourself, fine. If you feel the need to come on here and make this library (and the hours of work that went into it) look pointless, that's not fine.

the exact opposite. i tried this library and before that another lib. i put work into it, but both did not really work. so i was forced to do it manually.
if your read something evil here, thats your perspective.

@zoff99 if you did work manually, consider making your own library instead of telling us that our work isn't needed.


Feature: Skip exsitings/translated keys

We are using various Locale files, and some of them already contain many translations. It would be great if we could control via a property whether the keys that are already translated should not be translated again, but only the keys that do not exist or are empty.
This would save API requests, as the already translated keys could be skipped.

Will this be implemented? @Quilljou

@WeslleyDeSouza same issue, so I raise a PR to improve it: https://github.com/Quilljou/transmart/pull/9, cc @Quilljou please help review.

Version upgraded to 0.4.0. Give it a try.
Thanks @Germey

@Germey  since v4 im getting:

For response_json params, I think OpenAI API should support it, what model does you use?
And yes, every file will create a cache file, so that to remove the multi-write issue at same time.
For docs, @Quilljou could you please help update?

@Germey im using the default model.

@WeslleyDeSouza Updated the default model to the latest. Now would be fine at 0.4.1

@Quilljou thx but im having the same error.

it works, when i remove the attribute.

Hard to know which model supports the json_object response. I removed this attribute in version 0.4.2 @WeslleyDeSouza


Task/test peer service

Create Transaction Command
./issue_transaction [COMMAND] [DATAMODEL] [DATAMODEL args...]
Can use DataModel

peer

ip
publickey
trustScore
isActive (true or false)

domain

name
value

[WIP] Can not use DataModel now

account
asset

Test add peer
Manual
$IROHA_HOME/docker/build.sh -f
docker-compose up
docker-compose scale iroha=4
docker exec -it iroha_iroha_1 sh -c "/usr/local/iroha/bin/issue_transaction add peer [ip] [publicKey] 0.0 false"
But, circle.yml happened error.
Pelase check someone. circle.yml.
( notes: I overwrite config-server.py )

WIP

docker exec -it iroha_iroha_1 sh -c "/usr/local/iroha/bin/issue_transaction add peer [ip] [publicKey] 0.0 false"

This means that node iroha_iroha_1 (which is already in a network), invite peer with ip=[ip] and public key = [publicKey]?
0.0 is a trusted score?
What is false in the end?

@Warchant
Yes.
iroha_iroha_1 is already in a network. ( In other words, iroha_iroha_1 is master peer.)
invite peer with ip=[ip] and public key = [publicKey] is correct.
In particular, [ip] = iroha_iroha_i's ip , [publicKey] = ioroha_iroha_i's publickey. ( "i" is integer that is upper than 1. )
0.0 is trustScore. But, it is not used.
false is which active? or non active? but, it is not used.
So, 0.0 and false is dummy data.

The trustScore will be part of Hijiri, the reputation system, once it is done.

trustScore is fixed value when add peer.
In other words, every peer has the same trustScore at the beginning.

How do I know that new node is connected?
I tried this with 2 and 3 nodes, only master node reacts on issue_command, slave node (which I am trying to connect) says nothing

@Warchant
Currently under investigation.
Happened my case.
iroha_1            |<PHONE_NUMBER>[sumeragi] commit
iroha_1            |<PHONE_NUMBER>[sumeragi] commit count:1
iroha_1            |<PHONE_NUMBER> INFO [WorldStateRepositoryWithLeveldb] LoadDB
iroha_1            |<PHONE_NUMBER> INFO [WorldStateRepositoryWithLeveldb] NotFound:

After this, Verify sendall should be called. However, the program does not seem to call this.

To simplify your development:
0. create iroha network docker network create --subnet=<IP_ADDRESS>/16 iroha-network (you should do it only once)

prepare sumeragi.json for 2 nodes (for each node separate config)
run docker run -dit --network=iroha-network --ip=<IP_ADDRESS> --name iroha1 -v sumeragi1.json:/usr/local/iroha/config/sumeragi.json hyperledger/iroha-docker for each node (change iroha1 to iroha2 and sumeragi1.json to sumeragi2.json, ip from <IP_ADDRESS> to <IP_ADDRESS>)

at this step you have two separate fully configured nodes

then, attach to 'master' peer docker exec -it iroha1 bash
issue necessary commands

https://github.com/hyperledger/iroha/pull/209#issuecomment-286394220
Correct this problem.
sendAll method don't send my peer.
However, Sumeragi do to issue sendAll after get consensus. Then, Sumeragi don't exec execute function.  Sumeragi execute transaction in Verify after get consensus.
So, sendAll method must send mine.
https://github.com/hyperledger/iroha/pull/209/commits/2b91bd06c2f3a46841e9d098620e5e22ef9dcf21

(venv-system) ubuntu@box791:~/iroha$ sudo docker exec -it iroha1 /bin/bash
open /proc/self/oom_score_adj: no such file or directory
(venv-system) ubuntu@box791:~/iroha$ docker exec -it iroha1 /bin/bash
open /proc/self/oom_score_adj: no such file or directory
(venv-system) ubuntu@box791:~/iroha$ docker ps
CONTAINER ID        IMAGE                      COMMAND             CREATED             STATUS              PORTS               NAMES
9b3f740176e2        hyperledger/iroha-docker   "/run.sh"           11 minutes ago      Up 11 minutes                           iroha2
57e5946dfbc2        hyperledger/iroha-docker   "/run.sh"           11 minutes ago      Up 11 minutes                           iroha1

Is docker exec * * unused in circleci?
I got an error. open /proc/self/oom_score_adj: no such file or directory

Correct Local Test.

Could you check review.
issue_transaction.cpp  can use add peer or add domain. Now, other than that is no compatible. But, it is don't use  urgent. So, please ignore tools/helper/*.

I've removed redundant commits
D: but a bit missed with commit hash at rebasing so all commits' hashes have been changed, so either push old branch so I can properly remove those, or just download new branch


diagnostics floats are really noisy

Currently I see:
[clang(Semantic Issue) ovl_no_viable_function_in_call] [E] No matching function for call to 'foo'
The diagnostic text is the most important but it starts on column 59, and is the first thing to get truncated/wrapped.
A big contributor is clang/clangd which use long strings for the code. I think they have some value but display needs to be done carefully.
Another contributor is the (Semantic Issue) category, which I'm not sure is pulling its weight.
I'd like to try making the source/code styled differently, placed at the end of the diagnostic, and dropping the [E] in upstream coc.

The diagnostic format is defined [source code] [severity] message in https://github.com/neoclide/coc.nvim/blob/3c462c6486725f7e3a82900e75c671d5d31bf093/src/diagnostic/manager.ts#L306. coc needs to add diagnostic format support.

btw, we might also consider moving the full diagnostic info to the command line and only show diagnostic message in the floating window.
I also think it would be better if diag(floating window) hasn't moved with the cursor, it could pop-up once the user enters diag range and stay there until user exits it.

I also think it would be better if diag(floating window) hasn't moved with the cursor, it could pop-up once the user enters diag range and stay there until user exits it.

Yeah, I don't know if this is easy/hard to do.
I do find myself wishing it was pinned to column zero instead of anchored at the cursor, because it tends to truncate or wrap.

I also think it would be better if diag(floating window) hasn't moved with the cursor

If you are using Neovim, add "diagnostic.virtualText": true to coc-settings.json, the diagnostic message will pinned with virtual text at the end of line.

Thanks for the pointer. I think that's the one for preview window, and this one is for floating windows:
https://github.com/neoclide/coc.nvim/blob/3c462c6486725f7e3a82900e75c671d5d31bf093/src/diagnostic/manager.ts#L458
(Some of the concerns apply to both, but space is much less of a concern in the preview window for instance)

If you are using Neovim, add "diagnostic.virtualText": true to coc-settings.json, the diagnostic message will pinned with virtual text at the end of line.

This is nice - I tried it out, but one of our goals is to support a bunch of internal users who are using a particular version of vim (fairly recent: 8.1.2269).

So while customizability (and highlighting) is nice, playing with this makes me more convinced that the defaults can be improved first. Does this seem more readable
Before:

After:

@kadircet points out that floating windows use a different color per severity, as well as the gutter marker, and the Error:  is mostly noise:

https://github.com/neoclide/coc.nvim/pull/1564

Changed on coc by adding diagnostic.format, without change the default style. User can define format they want, eg drop the source part.

Thanks for doing that!
Having it customizable means ~all power-users will be happy, but IME most non-power-users won't work out how to change it even if it bothers them.
How do you feel about changing the default style? I think either "%message\n[%source%code]" or "%message" would be a clearer default, with more info available to those who want it.

I don't like to change default style. Current style [source code] [E] message in most case is ok, for example:

[pylint E0104] [E] Return outside function
[json 515] [E] Colon expected
[tsserver 6133] [I] 'a' is declared but its value is never read.

The code part is needed and useful for users. On clangd side, the noisy is because too-long-error-code. We can overwrite the default diagnostic.format in coc-clangd, I make it default %message [%source].

@fannheyward overwrite default configuration by extension doesn't seem to be a good idea, the user install an extension and the behavior unexpected changed.

@chemzqm the overwrite configurations has been removed.


Bulk upload countersigned agreements

This should be a one-time- (or few-times-at-most)-use script to upload the first batch of countersigned G7 framework agreements when we get them from CCS.
See: https://www.pivotaltracker.com/story/show/109718484
Depends on:

[x] https://github.com/alphagov/digitalmarketplace-utils/pull/230

Just had a conversation with @cathrooney - looks like the files will probably be coming in from CCS in the format Sanitised_Supplier_Name-123456_Signed_Framework_Agreement.pdf - but I think we might as well hold off updating the script until we have a bunch of files to update.
This is rebased up to date and hopefully ready to go, barring putting in the correct filename format when we get the files.

This has been run to upload the first batch of agreements to prod now, so hopefully ready to go :)


[PWA] Service Worker

Hello,
I noticed that the Service Worker file was deleted from the update v1.2.0 is it normal or a mistake ?

yes. CRA 4 made in totally optional. If you want you can add it (check the CRA documentation)

yes. CRA 4 made in totally optional. If you want you can add it (check the CRA documentation)


ldap: Fix test_registration_through_ldap.

Fixes #6663

@timabbott, please review.

Merged, thanks @umairwaheed!  Sorry for the delay reading this.


Resumable plugins rebased

This is a rebase & some cleanup to #495.  Will squash commits down before merge.
I'd like to do some more manual testing before merge, but my main feedback points have been addressed and we need to get this in now-ish, so any further issues that come up can be addressed in subsequent PRs.
FYI @nrb @carlisia. Don't expect full reviews since it's a lot of code.

Recording a couple of potential action items:

[ ] this includes breaking changes to the protobuf for plugins. We should reach out to all of our known plugin developers to make sure they're not surprised by this.
[ ] backup/restore controller - this PR logs the per-backup & restore logs to both a file in object storage AND to stdout, so there's info available if the obj storage upload fails. Do we want to log to stdout in all cases? Can we do this only if the upload fails?
[ ] eliminating the backup service and moving to pkg-level functions resulted in some annoying wrapper structs being created in order to enable mocking in unit tests. Can we use a monkey-patching-like approach here to simplify?
[ ] the sample plugins repo will need to be updated

backup/restore controller - this PR logs the per-backup & restore logs to both a file in object storage AND to stdout, so there's info available if the obj storage upload fails. Do we want to log to stdout in all cases? Can we do this only if the upload fails?

I can't think of an easy way to do that. Is there any harm in always uploading to the ark server log too?

eliminating the backup service and moving to pkg-level functions resulted in some annoying wrapper structs being created in order to enable mocking in unit tests. Can we use a monkey-patching-like approach here to simplify?

We can bring it back

Re: backup service, interested in opinions on the last commit that I just pushed as an approach that allows us not to have the backup service and also doesn't require a bunch of interface/struct definitions for testing purposes.

@ncdc re: logs, I was just concerned about the server logs getting super-verbose.  Couldn't we just copy the contents of the log temp file into the server log if/only if the call to upload it to obj storage fails?

I guess it's worth a try

Or you could revert that part of the PR

eh, i'm gonna leave the logs as-is. we can always change if it's an issue down the road.

A question - does this require any changes to the https://github.com/heptio/ark-plugin-example/ repo for 3rd party contributors?

Yes, and we should start tagging and/or releasing versions of the example repo to match tagged/released versions of ark (and master=master)

@skriss Specifically addressing the backupDownloader replacement, +1 there. Makes sense to me.

@nrb did the rest of the refactoring work, commit per type. at this point I think i'm ready to merge (once I squash everything down)

The refactored work LGTM. Looks like the rebase work isn't terrible, either.

  I'll plan to squash later today and then we can get it merged.

Testing looked good to me. The plugin process restarted and I never saw more than one.

squashed and ready to merge.


feature: translate numeric primary keys to text in PostgreSQL

PostgreSQL dialect does not support numeric primary keys. We instead translate the numeric components of the primary key to text and warn users.

Tested with live migration in SMT. Job succeeded.


Pause exam when page loses focus

If allowed by exam settings, the exam should be paused when the student switches away from the exam window. This could make reported exam durations a bit more accurate.

On reflection (6 years!), this is a bad idea: it 's not like the student won't be thinking about the exam while they're looking at something else.


Add custom cache key to support dynamic url images

Please add this feature because it is a vital one for many apps like mine. Thank you.

something like this ？
https://github.com/fluttercandies/extended_image_library/issues/20

something like this ？
https://github.com/fluttercandies/extended_image_library/issues/20

No, something that overrides the the url passed and identifies an image from a unique key, like this:
ExtendedImage.network(
url: 'whatever it is',
cacheKey: 'Image 1' // The image is retrieved from cache using this key as an identifier!
);

store key with url+md5 now

https://github.com/fluttercandies/extended_image_library/blob/e67006f75cdcdfabfe4890b9b9dd067cc17da48c/lib/src/extended_network_image_utils.dart#L14

But the url is dynamic, it changes every time it is fetched. Would not hashing it produce a different key every time the image is fetched?
One more thing, where do I insert the hashed url in the parameters of  extendedImage. Can you please elaborate more using an example

I don't want to make this library more complex.  we store key as md5 now, you can override image provider and make cache logic by yourself.

i  will do this later

Hi, Firstly, thanks for this feature. However, i am realising that when i have the same cacheKey but different url, the widget blinks every-time there is a rebuild.
However when using cache network image, this does not occur.
ExtendedImage.network
https://user-images.githubusercontent.com/40727575/150145874-f70246c8-7cda-4193-a728-826e296422e1.mov
ExtendedImage.network(
widget.file.source,
cacheKey: widget.file.fileId,
retries: 5,
timeRetry: const Duration(seconds: 2),
enableMemoryCache: true,
loadStateChanged: (state) {
switch (state.extendedImageLoadState) {
case LoadState.loading:
return _ImageByState.loading();
case LoadState.failed:
return _ImageByState.failed();
case LoadState.completed:
return _ImageByState.successFolderFile(
image: state.imageProvider,
file: widget.file,
privateFolderController: widget.privateFolderController);
default:
return const SizedBox();
}
},
),
CachedNetworkImage
https://user-images.githubusercontent.com/40727575/150146184-4e71615b-9484-4e93-8c44-1ce531159dda.mov
CachedNetworkImage(
cacheKey: widget.file.fileId,
imageUrl: widget.file.source,
progressIndicatorBuilder: (context, url, downloadProgress) =>
CircularProgressIndicator(value: downloadProgress.progress),
errorWidget: (context, url, error) => Icon(Icons.error),
),

Hi, Firstly, thanks for this feature. However, i am realising that when i have the same cacheKey but different url, the widget blinks every-time there is a rebuild.
However when using cache network image, this does not occur.
ExtendedImage.network
Screen.Recording.2022-01-19.at.10.00.42.PM.mov
ExtendedImage.network( widget.file.source, cacheKey: widget.file.fileId, retries: 5, timeRetry: const Duration(seconds: 2), enableMemoryCache: true, loadStateChanged: (state) { switch (state.extendedImageLoadState) { case LoadState.loading: return _ImageByState.loading(); case LoadState.failed: return _ImageByState.failed(); case LoadState.completed: return _ImageByState.successFolderFile( image: state.imageProvider, file: widget.file, privateFolderController: widget.privateFolderController); default: return const SizedBox(); } }, ),
CachedNetworkImage
Screen.Recording.2022-01-19.at.10.01.11.PM.mov
CachedNetworkImage( cacheKey: widget.file.fileId, imageUrl: widget.file.source, progressIndicatorBuilder: (context, url, downloadProgress) => CircularProgressIndicator(value: downloadProgress.progress), errorWidget: (context, url, error) => Icon(Icons.error), ),

https://github.com/fluttercandies/extended_image/blob/646c3f04701125598e0458babeb3a3e0634079d1/lib/src/extended_image.dart#L747   it will help

Hi, Firstly, thanks for this feature. However, i am realising that when i have the same cacheKey but different url, the widget blinks every-time there is a rebuild.
However when using cache network image, this does not occur.
ExtendedImage.network
Screen.Recording.2022-01-19.at.10.00.42.PM.mov
ExtendedImage.network( widget.file.source, cacheKey: widget.file.fileId, retries: 5, timeRetry: const Duration(seconds: 2), enableMemoryCache: true, loadStateChanged: (state) { switch (state.extendedImageLoadState) { case LoadState.loading: return _ImageByState.loading(); case LoadState.failed: return _ImageByState.failed(); case LoadState.completed: return _ImageByState.successFolderFile( image: state.imageProvider, file: widget.file, privateFolderController: widget.privateFolderController); default: return const SizedBox(); } }, ),
CachedNetworkImage
Screen.Recording.2022-01-19.at.10.01.11.PM.mov
CachedNetworkImage( cacheKey: widget.file.fileId, imageUrl: widget.file.source, progressIndicatorBuilder: (context, url, downloadProgress) => CircularProgressIndicator(value: downloadProgress.progress), errorWidget: (context, url, error) => Icon(Icons.error), ),

https://github.com/fluttercandies/extended_image/blob/646c3f04701125598e0458babeb3a3e0634079d1/lib/src/extended_image.dart#L747
it will help

just tried
         gaplessPlayback: true,
does help abit that the LoadState.completed will remain even when i refresh, however it still blinks.
gaplessPlayback: false,
https://user-images.githubusercontent.com/40727575/150246580-c3b25f41-d456-4296-97fd-17dbaa3afff3.mov
gaplessPlayback: false,
https://user-images.githubusercontent.com/40727575/150246598-7a3f0f4d-8a78-49dd-ac89-d1d0da78ebdc.mov

I also realised this is the case only when the url is different.
      ExtendedImage.network(           "https://www.planetware.com/wpimages/2020/03/best-hot-air-balloon-rides-cappadocia-turkey.jpg",           cacheKey: file.fileId,           retries: 5,           timeRetry: const Duration(seconds: 2),           enableMemoryCache: true,           gaplessPlayback: false,           loadStateChanged: (state) {             switch (state.extendedImageLoadState) {               case LoadState.loading:                 return _ImageByState.loading();               case LoadState.failed:                 return _ImageByState.failed();               case LoadState.completed:                 return _ImageByState.successFolderFile(                     image: state.imageProvider,                     file: file,                     privateFolderController: privateFolderController);               default:                 return const SizedBox();             }           },         ),
This code even with gaplessPlayback: false, it does not blink.
Its only when the url is dynamic that the images blink.

i didn't get what is that url is dynamic? maybe a simple runnable demo is better

`import 'dart:math';
import 'package:extended_image/extended_image.dart';
import 'package:flutter/material.dart';
class TestPage extends StatefulWidget {
const TestPage({Key? key}) : super(key: key);
@override
_TestPageState createState() => _TestPageState();
}
class _TestPageState extends State {
@override
Widget build(BuildContext context) {
return Scaffold(
body: Column(
children: [
Expanded(
child: GridView.builder(
padding:
const EdgeInsets.symmetric(horizontal: 15, vertical: 10),
gridDelegate: const SliverGridDelegateWithFixedCrossAxisCount(
crossAxisCount: 4,
childAspectRatio: 1,
crossAxisSpacing: 10,
mainAxisSpacing: 10),
itemCount: 50,
itemBuilder: (BuildContext context, index) {
return ExtendedImage.network(
"https://www.planetware.com/wpimages/2020/03/best-hot-air-balloon-rides-cappadocia-turkey.jpg?index=" +
Random().nextInt(1000).toString(),
cacheKey: index.toString(),
retries: 5,
timeRetry: const Duration(seconds: 2),
enableMemoryCache: true,
gaplessPlayback: true,
loadStateChanged: (state) {
switch (state.extendedImageLoadState) {
case LoadState.loading:
return Container(color: Colors.yellow);
case LoadState.failed:
return Container(color: Colors.red);
case LoadState.completed:
return Container(
decoration: BoxDecoration(
borderRadius: BorderRadius.circular(5),
color: Colors.black12,
image: DecorationImage(
image: state.imageProvider,
fit: BoxFit.cover,
),
),
);
default:
return const SizedBox();
}
},
);
}),
),
TextButton(
onPressed: () {
setState(() {});
print('Tapped');
},
child: Text('set state'))
],
),
);
}
}
`
https://user-images.githubusercontent.com/40727575/150248566-ab2ba8ba-d5c5-44fb-845a-d869e1f4bafa.mov

"https://www.planetware.com/wpimages/2020/03/best-hot-air-balloon-rides-cappadocia-turkey.jpg?index=" +
Random().nextInt(1000).toString(),
ImageSource is changed every time, i can't find any solution. you can use CacheImageProvider instead of ExtendedImageProvider.  i didn't do any thing about the dynamic url

hopefully there can be an update in the near future. this is quite common usage for when urls comes with different tokens everytime.  for example
`https://domain.s3.ap-southeast-1.amazonaws.com/users/0dc26ab8-0d4d-403f-b6ae-be0fd329b3c5/privateFolders/8c0af424-3359-4f5f-a860-8ae42be44d5c/77d86382-84e2-431c-b017-621e199c0e5c.webp?AWSAccessKeyId=ASIAQ4SPHJ5HL7ULXFV5&Expires=1642644676&Signature=UQo6r3rOf2XuNGv4gxP4VmqtzJU%3D&x-amz-security-token=IQoJb3JpZ2luX2VjEOL%2F%2F%2F%2F%2F%2F%2F%2F%2F%2FwEaDmFwLXNvdXRoZWFzdC0xIkgwRgIhAOjLUYct0vLXgCnM4kpW57X0FMA2M3VXIbA20JVyjZBwAiEApZh33s6XgzRH1%2Fs4k1bqpk5tjB%2BF2DSJ%2FlbYRsRHtIsquwII%2B%2F%2F%2F%2F%2F%2F%2F%2F%2F%2F%2FARABGgwwNjEzNjk0MzgwMzAiDNC4Q%2BjpVlU1WUMl8CqPAt8hz6cnXJfeamLSa%2B4v3%2BcjOf053zWUqk3xl0aadd1LI73SNfcJs%2BdLiNPqPYIh6MPivILk%2BlR6KWxunsewxcQgswdvM2w7gELNFW3JWRX1Y%2BHurmxJigXnHYBlROIAvyiQwkR8X5yC3XCjWSvl%2BiqCR%2F4xmgiG2guvC58T%2BNP0d3YvWlGzEjR4mkqrbLjq8vxBK9vCtoCruKRC16cUIGHREb2KMSkFR8MIPQyCWhIK7zqH8q3wtht045aUBj2uBoWARMNz6KNNdjuTotDQsXEU5U2Irnu5B3Cfk72tc%2BEJD5TmaiB6fdVFx1YAcUFgIgjpZmdJao8QeenvBUaWUcyh2Thtlsv%2BVim30Sw%2FU7Aw3%2FWijwY6mQHU38v20vpMvI9kEhEFYFY74mQyaYdSLMXxRyGK9
`
the token is used on the backend to determine if the file access has expired or not.  I am using aws s3 presigned url which uses this feature.

hopefully there can be an update in the near future. this is quite common usage for when urls comes with different tokens everytime. for example `https://domain.s3.ap-southeast-1.amazonaws.com/users/0dc26ab8-0d4d-403f-b6ae-be0fd329b3c5/privateFolders/8c0af424-3359-4f5f-a860-8ae42be44d5c/77d86382-84e2-431c-b017-621e199c0e5c.webp?AWSAccessKeyId=ASIAQ4SPHJ5HL7ULXFV5&Expires=1642644676&Signature=UQo6r3rOf2XuNGv4gxP4VmqtzJU%3D&x-amz-security-token=IQoJb3JpZ2luX2VjEOL%2F%2F%2F%2F%2F%2F%2F%2F%2F%2FwEaDmFwLXNvdXRoZWFzdC0xIkgwRgIhAOjLUYct0vLXgCnM4kpW57X0FMA2M3VXIbA20JVyjZBwAiEApZh33s6XgzRH1%2Fs4k1bqpk5tjB%2BF2DSJ%2FlbYRsRHtIsquwII%2B%2F%2F%2F%2F%2F%2F%2F%2F%2F%2F%2FARABGgwwNjEzNjk0MzgwMzAiDNC4Q%2BjpVlU1WUMl8CqPAt8hz6cnXJfeamLSa%2B4v3%2BcjOf053zWUqk3xl0aadd1LI73SNfcJs%2BdLiNPqPYIh6MPivILk%2BlR6KWxunsewxcQgswdvM2w7gELNFW3JWRX1Y%2BHurmxJigXnHYBlROIAvyiQwkR8X5yC3XCjWSvl%2BiqCR%2F4xmgiG2guvC58T%2BNP0d3YvWlGzEjR4mkqrbLjq8vxBK9vCtoCruKRC16cUIGHREb2KMSkFR8MIPQyCWhIK7zqH8q3wtht045aUBj2uBoWARMNz6KNNdjuTotDQsXEU5U2Irnu5B3Cfk72tc%2BEJD5TmaiB6fdVFx1YAcUFgIgjpZmdJao8QeenvBUaWUcyh2Thtlsv%2BVim30Sw%2FU7Aw3%2FWijwY6mQHU38v20vpMvI9kEhEFYFY74mQyaYdSLMXxRyGK9
`
the token is used on the backend to determine if the file access has expired or not. I am using aws s3 presigned url which uses this feature.

https://github.com/fluttercandies/extended_image_library/issues/20


SCM progress indicator does not stop

VSCode Version: Code - Insiders 1.18.0-insider (e6a76e4bd3f52ab07452bb181e861f5a9bfb6596, 2017-10-27T05:12:19.123Z)
OS Version: Darwin x64 16.7.0
Extensions:

Extension
Author (truncated)
Version

autofilename
Jer
0.1.3

regex
chr
0.1.0

ssh
chr
0.0.2

vscode-eslint
dba
1.4.0

ghlink
dt
0.0.6

tslint
eg2
1.0.13

azure-account
ms-
0.1.3

azurecli
ms-
0.3.0

github-issues-prs
ms-
0.6.0

debugger-for-chrome
msj
3.4.0

vetur
oct
0.10.1

vscode-versionlens
pfl
0.19.1

vscode-open-in-github
sys
1.6.0

Not sure what triggers it, but it keeps going in one of my multi-root workspaces.

Seems to be the same as in #36754. Should already be fixed, can you try to update to the latest version?

@chrmarti Open htop and tell me what the git processes are doing.

@joaomoreno There is no git process running. I'll check if it still occurs with the latest version or if maybe #36754 fixed it.


[VS Code] The user code is invalid or expired. Try signing in again.

#326 What can I do when keep failing signing in?

We've seen issues where peoples system clock has drifted from the current time, so it's no longer correct. Can you check your clock?
If it is correct, could you share your logs with us -- if you open the command palette, and then run "Live Share: Export logs", it should generate the logs for you.

@augielopez Version 0.3.98 and 0.3.144 have some fixes in it to try to help with this kind of situation.  Can you update and see if it solves the problem for you?

@hyounoo We haven't heard back so I am assuming one of our recent releases has solved the problem for you.  We also just released 0.3.125 which has further fixes in this area.  Please feel free to re-open if you are still having trouble.

Sorry for late reply.
It turned out that my company firewall.
The signing to MS is blocked when connected to company network.
Without vpn, this works just fine. I will have to ask IT support team for the issue. Thank you.