[Flaky Test] ConnectionFailTest

example failure
logs:
9_Run unit tests.txt

It seems that this flaky test exposes the memory leak.
$ valgrind --leak-check=full ./ConnectionFailTest
# ...
==7133== LEAK SUMMARY:
==7133==    definitely lost: 6,168 bytes in 3 blocks
==7133==    indirectly lost: 42,511 bytes in 36 blocks
==7133==      possibly lost: 156,667 bytes in 76 blocks
==7133==    still reachable: 11,474 bytes in 149 blocks
==7133==                       of which reachable via heuristic:
==7133==                         multipleinheritance: 328 bytes in 2 blocks
==7133==         suppressed: 0 bytes in 0 blocks

The memory leak happens for my own branch, so just ignore it.

It seems also a flaky test for my branch. Here is the failure

@RobertIndie Yes. I have figured out the reason and will push a fix soon.

Another failure logs:
2023-03-14 02:57:33.014 INFO  [140281[225](https://github.com/apache/pulsar-client-cpp/actions/runs/4402928356/jobs/7730474705#step:9:226)117888] ClientImpl:589 | Closing Pulsar client with 0 producers and 1 consumers
2023-03-14 02:57:33.014 INFO  [140281225117888] ConsumerImpl:1206 | [persistent://public/default/test-connection-fail-51678762652-partition-4, reader-097f2a19ff, 9] Closing consumer for topic persistent://public/default/test-connection-fail-51678762652-partition-4
2023-03-14 02:57:33.014 INFO  [140281225117888] ConsumerImpl:1206 | [persistent://public/default/test-connection-fail-51678762652-partition-3, reader-097f2a19ff, 8] Closing consumer for topic persistent://public/default/test-connection-fail-51678762652-partition-3
2023-03-14 02:57:33.014 INFO  [140281225117888] ConsumerImpl:1206 | [persistent://public/default/test-connection-fail-51678762652-partition-2, reader-097f2a19ff, 7] Closing consumer for topic persistent://public/default/test-connection-fail-51678762652-partition-2
2023-03-14 02:57:33.014 INFO  [140281225117888] ConsumerImpl:1206 | [persistent://public/default/test-connection-fail-51678762652-partition-1, reader-097f2a19ff, 6] Closing consumer for topic persistent://public/default/test-connection-fail-51678762652-partition-1
2023-03-14 02:57:33.014 INFO  [140281225117888] ConsumerImpl:1206 | [persistent://public/default/test-connection-fail-51678762652-partition-0, reader-097f2a19ff, 5] Closing consumer for topic persistent://public/default/test-connection-fail-51678762652-partition-0
2023-03-14 02:57:33.027 INFO  [140281191507520] ConsumerImpl:1192 | [persistent://public/default/test-connection-fail-51678762652-partition-4, reader-097f2a19ff, 9] Closed consumer 9
2023-03-14 02:57:33.027 INFO  [140281191507520] ConsumerImpl:1192 | [persistent://public/default/test-connection-fail-51678762652-partition-3, reader-097f2a19ff, 8] Closed consumer 8
2023-03-14 02:57:33.027 INFO  [140281191507520] ConsumerImpl:1192 | [persistent://public/default/test-connection-fail-51678762652-partition-2, reader-097f2a19ff, 7] Closed consumer 7
2023-03-14 02:57:33.027 INFO  [140281191507520] ConsumerImpl:1192 | [persistent://public/default/test-connection-fail-51678762652-partition-1, reader-097f2a19ff, 6] Closed consumer 6
2023-03-14 02:57:33.027 INFO  [140281191507520] ConsumerImpl:1192 | [persistent://public/default/test-connection-fail-51678762652-partition-0, reader-097f2a19ff, 5] Closed consumer 5
2023-03-14 03:55:52.953 ERROR [140281191507520] ClientConnection:615 | [<IP_ADDRESS>:51534 -> <IP_ADDRESS>:6650] Read operation failed: End of file
2023-03-14 03:55:52.953 ERROR [140281191507520] ClientConnection:1198 | [<IP_ADDRESS>:51534 -> <IP_ADDRESS>:6650] Connection closed with ConnectError
2023-03-14 03:55:52.953 INFO  [140281191507520] ClientConnection:262 | [<IP_ADDRESS>:51534 -> <IP_ADDRESS>:6650] Destroyed connection

It's still a flaky test, see here. https://github.com/apache/pulsar-client-cpp/pull/211 fixes the race condition in Client::close, but there is another reason that might make this test fail. The root cause is the Boost resolver might stuck in async_resolve. We need to add a timeout to this method.


Fix start script path so photon builds! 🙌🏻

This resolves the issue in #9.
See image of app after fix:

Whoops! Good catch. I just left one comment but otherwise that looks good.

@connors Just updated the PR, again. Works 100% on my end.

Helllll yeah


add IUA(Abidjan, Côte d'ivoire) to the request

@bakidev Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@bakidev Please provide us with:
* the school official website URL

* the school street address, including city and country

* an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course

* a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@philipto
1- School official website https://www.iuaci.org/
2 - School street adress: II Plateaux, 7ème Tranche, Rue L40,
Carrefour Cascades, Abidjan, Côte d’ivoire
3  -the official website where the school offers an IT-related long-term (>1 year) course: https://www.iuaci.org/faculte-des-sciences-et-technologiques/
4 - This is my own student email<EMAIL_ADDRESS>. Let me know if you want some additional informations

@bakidev  Thank you for the clarifications. Pull request merged. Please start requesting the licenses in about an hour, to let the changes to propagate through our system.

@bakidev Thank you for the clarifications. Pull request merged. Please start requesting the licenses in about an hour, to let the changes to propagate through our system.

@philipto thank you very much

@bakidev Thank you for the clarifications. Pull request merged. Please start requesting the licenses in about an hour, to let the changes to propagate through our system.

@philipto thank you very much


Extend PR template (for new talktorials)

Description
We had a PR template tailored for refactoring talktorials; extend this template to be helpful when adding new talktorials or extending existing talktorials.
Todos

[x] Update PR template with some details on added packages, data, etc.

Status

[ ] Ready to go

Hi @AndreaVolkamer, could you please check if I covered relevant PR template TODOs?
@gorostiolam and @jesperswillem, sharing this also with you already for your submissions.
Please let me know if you think something's missing or unclear, thank you!


feat: add ordinals addresses in receive modal

Try out this version of the Hiro Wallet - download extension builds.

This pr closes https://github.com/hirosystems/stacks-wallet-web/issues/3229

@kyranjamie I fixed comments mentioned in https://github.com/hirosystems/stacks-wallet-web/pull/3259

This element needs some styling fixes and subtitle can be removed

Could you also fix the font size for the section? I updated to body 2 (14px)

The ability to quickly copy a BTC address can be removed from the modal as well and from the header. This is beacuse the user needs to see the notification before.

Save to say we can remove the ability to copy STX address from the header as well on this PR?

Yep!

@mica000  I removed copy icons, and added separator. but do these truncated addresses really make much sense to user?

I think @mica000 might mean both the copy buttons and the address values entirely, showing just the total fiat value of BTC and STX per https://github.com/hirosystems/stacks-wallet-web/issues/3132#issuecomment-1433124048

We can remove these addresses:

We can also remove the subtitle from the BTC receive step

On the same step, the style of the address seems to be removed:

The popover doesn't seem to contain the changes of this PR?

@mica000 I removed subtitle from the BTC receive step

We can remove these addresses:
I will do it in separate pr

The popover doesn't seem to contain the changes of this PR?

hm, weird, for me it's ok

probably you checked some other version?

@alter-eggo Sorry, my bad then! Everything is looking good now! Thank you!
Is it correct that the ability to copy BTC from the header will be removed on another PR, correct?

This looks good to ship!
A few nitpicks here to resolve afterwards as needed:

Should say "Do not deposit Ordinal inscriptions here"
The entire second line should be linked, inclusive of "Use the" since otherwise it seems a bit weird?
We can remove the period from the end of the second line

cc @mica000

done


the image can't download

there is a way to set proxy config for kitematic?
in most cases can't download images

Are you behind a firewall or VPN?
See our wiki for possible solutions:
https://github.com/docker/kitematic/wiki/Common-Proxy-Issues-&-Fixes


Export JPG/PNG as blob

I have a use case where I want to use CCapture to export the Canvas frames to use with an external AVI Encoder.
In order to do this most effectively, I'd like to export each Canvas frame as an individual file blob so that it can be passed directly to the encoder.
I'm about to embark on this work, so any thoughts on how this might work in the context of PR would be helpful to allow me to contribute it back.

You can follow the structure defined in CCFrameEncoder (https://github.com/spite/ccapture.js/blob/master/src/CCapture.js#L128) to build your own frame encoder.
Are you storing the frames in memory, sending them to a server...?

Thank you. I'll be storing them in memory as some kind of array of blobs. I was planning to pass this array of blobs to the AVI Encoder (a fork of this).
So all I really need to achieve is to obtain the array of blobs from the animation. This is trivial using canvas.toDataURL() & requestAnimationFrame, but doesn't come with the smooth animation timing advantages of CCapture.
I can think of two routes to adding this functionality:

Adding a new Encoder that allows access by the calling code to a blob from each captured frame. Ideally it could be configured to handle the blobs on a per frame basis at the time of capture (via callback or event emit) or access to all frames at the end of the capture via capturer.save() where the data returned is an array of blobs.

I did wonder if it might be possible to achieve this objective more simply by expecting the calling code to use capturer.save(function(blob){ frames.push(blob) }) after the every call to capturer.capture()? I'm guessing the caller would have ensure that they extract only the last item from the array of blobs in this case.


Adding an agent is very slow

Everytime we call simulator._add_agent(agent), it takes ~40s. Is this expected? Is there anyway to speed this up?

Wow that is not expected. Does this happen when the world is large? (i.e. been heavily explored) Does the world have a lot of agents?

No, the world only has a single agent and has not been explored at all. We create a Simulator and then immediately add the agent to it.
In case it helps with diagnosing, this is our agent class- it doesn't do anything interesting:
class _DummyAgent(jbw.agent.Agent):
  """Minimal implementation of the agent interface."""

  def __init__(self, simulator, load_filepath=None):  # pylint: disable=useless-super-delegation
    super().__init__(simulator, load_filepath)

  def do_next_action(self):
    raise NotImplementedError()

  def save(self, filepath):
    # This method is called by the simulator when saving.
    with open(filepath, 'wb') as f:
      pickle.dump((self._id, self._position, self._direction, self._scent,
                   self._vision, self._items), f, pickle.HIGHEST_PROTOCOL)

  def _load(self, filepath):
    # This method is called by the simulator when loading.
    with open(filepath, 'rb') as f:
      (self._id, self._position, self._direction, self._scent, self._vision,
       self._items) = pickle.load(f)

Simulator config:
'agent_field_of_view': 6.283185307179586,
'allowed_movement_directions': [<ActionPolicy.ALLOWED: 0>, <ActionPolicy.ALLOWED: 0>, <ActionPolicy.ALLOWED: 0>, <ActionPolicy.ALLOWED: 0>],
'allowed_turn_directions': [<ActionPolicy.DISALLOWED: 1>, <ActionPolicy.DISALLOWED: 1>, <ActionPolicy.DISALLOWED: 1>, <ActionPolicy.DISALLOWED: 1>]
,
'collision_policy': <MovementConflictPolicy.FIRST_COME_FIRST_SERVED: 1>,
'decay_param': 0.4,
'deleted_item_lifetime': 2000,
'diffusion_param': 0.14,
'max_steps_per_movement': 1,
'mcmc_num_iter': 10000,
'no_op_allowed': False,
'patch_size': 64,
'vision_range': 8,
'seed': 84}```

Simulator kwargs

Callback:

def _on_step(self):
self._t += 1

We're then doing:

simulator = jbw.Simulator(
sim_config=simulator_config,
on_step_callback=self._on_step)
agent = _DummyAgent(self._simulator)

The slowdown is in the call to [`sim_handle->add_agent`](https://github.com/eaplatanios/jelly-bean-world/blob/master/api/python/src/jbw/simulator.cpp#L1543).

Oh this is actually expected. The simulator doesn't actually start generating the world until the first agent is added. In addition, when a new agent is added, the simulator performs 10 times more MCMC iterations to make sure the initial patches of the world are well-mixed (since there are no previous patches to provide a good initialization).
Adding agents in the future will be much faster since we currently only support adding them at the position (0,0), which will have been already explored.

Ok- thanks for clarifying. That makes sense- all of the time is spent in get_fixed_neighborhood.


Font size is different from Photoshop

Hi. thanks for the tool
I have a several psd files which contains a text with font size that is different from Photoshop
For example:
In engine_dict font size is 25
In Photoshop font size is 6pt
If we assume that 25 is pixels and translate into points,
25 / 300 * 72 = 6pt
Document DPI = 300
Then it turns out what Photoshop shows.
Can PSD file contains a PX for font size and how we can determine it?
Or any other property may affect on the font size and we should recalculate it?
Thanks.

I have encountered the same problem. Have you solved this problem? How to deal with it?


Buttons

Alternative style for buttons :

Hi @r0ms, thanks for your PR. It's looks very nice.
Please have a look of my comments.
Don't hesitate to contact me if you need any informations.


fix(deployment): Clean up orphaned deployments

Signed-off-by: Vu Dinh<EMAIL_ADDRESS>

/test e2e-aws-olm

/test e2e-aws

/test e2e-aws

/retest

@ecordell Ready for final review.

Looks great, thanks for wrapping this up!
/lgtm
/retest

/retest

/retest

/retest

/retest

/lgtm
/retest


Add Remark keyword-matching to find command

Codecov Report
Base: 57.56% // Head: 57.62% // Increases project coverage by +0.05% :tada:

Coverage data is based on head (80b10b8) compared to base (7283b00).
Patch coverage: 80.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #304      +/-   ##
============================================
+ Coverage     57.56%   57.62%   +0.05%
- Complexity      889      893       +4
============================================
  Files           178      179       +1
  Lines          3933     3943      +10
  Branches        500      501       +1
============================================
+ Hits           2264     2272       +8
- Misses         1555     1557       +2
  Partials        114      114

Impacted Files
Coverage Δ

.../model/remark/RemarkContainsKeywordsPredicate.java
75.00% <75.00%> (ø)

...model/person/PatientContainsKeywordsPredicate.java
91.66% <100.00%> (+0.75%)
:arrow_up:

...u/uninurse/model/person/PatientMatchPredicate.java
95.23% <100.00%> (+0.23%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Assertions for CreateMachineAndExecute and SendEventAndExecute

We should fail an assertion if CreateMachineAndExecute or SendEventAndExecute calls Receive.

Done in #166


compileProtos fails open google/rpc//.proto

service-control introduced v2 protos: https://github.com/googleapis/googleapis/tree/master/google/api/servicecontrol/v2
service_control.proto import google/rpc/context/attribute_context.proto code
Nodejs-service-control fails on generated new PR due to npm run compile-protos fails on error:
(node:20193) UnhandledPromiseRejectionWarning: Error: ENOENT: no such file or directory, open 'protos/google/rpc/context/attribute_context.proto'

It's ok to import google/rpc/*.proto but not subdirectory.

cc @SurferJeffAtGoogle  and @noahdietz

Should fixed in typescript-gapic-generator. Created issue there.


Add a PageObject class to generated app

It can use Capybara methods like visit or find directly; it can use t instead of I18n.t; it has all the Formulaic helpers like fill_form; it has all the FactoryGirl helpers.
Example:
class UserPage < PageObject
  def initialize
    @user = create(:user)
  end

  def visit_profile
    visit user_path(@user)
  end
end

Should we close this PR now and focus on developing a library instead?

Yep.


[MS IE 11] Object doesn't support property or method 'assign'

Description
[MS IE 11] Object doesn't support property or method 'assign'
Steps to Reproduce

Open any MS Internet Explorer (I used IE 11)
Open playground https://mozilla-services.github.io/react-jsonschema-form/

Expected behavior
Form is shown
Actual behavior
SCRIPT438: Object doesn't support property or method 'assign'
Version
branch gh-pages

possible solution https://www.npmjs.com/package/object-assign

The build files ensure a polyfill for Object.assign through babel. If you're producing your own custom builds, it's up to you to install the right polyfills targeting the right platforms in your project.

Wait. You say branch gh-pages, that means the playground doesn't work on IE 11? I'm reopening.

I'm really surprised as the bundle.js file clearly shows a polyfill being defined in the source.

Can you confirm this is happening with the playground hosted at http://mozilla-services.github.io/react-jsonschema-form/?

We could also load polyfill.io polyfills by default in the playground.

https://developer.microsoft.com/en-us/microsoft-edge/tools/screenshots/#https://mozilla-services.github.io/react-jsonschema-form/
Confirm, it's for github playground only
On Wed, May 18, 2016 at 2:36 PM -0700, "Nicolas Perriault" <notifications<EMAIL_ADDRESS>wrote:
We could also load polyfill.iohttp://polyfill.io/ polyfills by default in the playground.

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://github.com/mozilla-services/react-jsonschema-form/issues/206#issuecomment-220165629

Looks like some cases were not transformed

I got the same issue as well as problems with find and findIndex methods for arrays. What should be my code in IE instead?

As mentioned in https://github.com/mozilla-services/react-jsonschema-form/issues/206#issuecomment-220165629 you could load specific polyfills for IE. Didn't that work for you?

I am not sure how should I do that and will it work for other people? In the meantime I already implemented workaround.

Just add <script src="//cdn.polyfill.io/v2/polyfill.min.js"></script> to your html document.

n1k0,
Adding the CDN polyfill above causes:
SCRIPT5022:  Exception thrown and not caught
polyfills.bundle.js (830,34)

Does this happen on https://mozilla-services.github.io/react-jsonschema-form/ as well?

try this https://github.com/aspnet/JavaScriptServices/wiki/Supporting-Internet-Explorer-11-(or-older)


Mock commands change

spongemock = /sm (no more /set fails)
sneakyspongemock = /ssm
miakomock = /mm
sneakymiakomock = /smm

#17


WIP: [ci] use R 4.2 in most R-package CI jobs

Fixes #4881.
Attempts to add support for R 4.2 in most of the R package's CI jobs.
Notes for Reviewers
#4881 refers to creating a comment-triggered workflow for the UCRT toolchain for Windows builds, but I don't think that's necessary any more now that R 4.2.0 has actually been released and Rtools42 contains the UCRT build tools.
References for some concerns about Windows builds:

Rtools42

https://cran.r-project.org/bin/windows/Rtools/rtools42/rtools.html

CRAN still requires 32-bit builds for r-oldrel (which is R 4.1)

https://cran.r-project.org/web/checks/check_flavors.html#r-devel-windows-x86_64

different paths for different versions, per r-lib/actions code

https://github.com/r-lib/actions/blob/8e8f7edba0e5bda9318f9377c0dc9c2f5d0fb786/setup-r/src/installer.ts#L431-L457

I'd like to propose checking passed R version to the FindLibR.cmake module from the logs
https://github.com/microsoft/LightGBM/blob/f1328d5c5fee1da5907371e292dda3e8344b4b63/cmake/modules/FindLibR.cmake#L76
similarly to
https://github.com/microsoft/LightGBM/blob/f1328d5c5fee1da5907371e292dda3e8344b4b63/.ci/test_r_package.sh#L237-L253
That will prevent situations like https://github.com/microsoft/LightGBM/pull/5129#issuecomment-1088190413.

ooooo, I think that the inet_pton detection check in configure.win might be part of the problem!
I tried running the following shell script in the CI builds for the Windows jobs.
echo "------ testing test program ----"

cat > conftest.cpp <<EOL
#include <ws2tcpip.h>
int main() {
  void* p = inet_pton;
  return 0;
}
EOL

g++ -std=gnu++11 -I"c:/rtools42/x86_64-w64-mingw32.static.posix/include" -o conftest conftest.cpp || exit 123
echo "------ done compiling ----"

./conftest || exit 123
rm -f ./conftest
rm -f ./conftest.cpp
echo "------ done running ----"

And got the following
------ testing test program ----
conftest.cpp: In function 'int main()':
conftest.cpp:3:13: error: invalid conversion from 'INT (*)(INT, LPCSTR, PVOID)' {aka 'int (*)(int, const char*, void*)'} to 'void*' [-fpermissive]
    3 |   void* p = inet_pton;
      |             ^~~~~~~~~
      |             |
      |             INT (*)(INT, LPCSTR, PVOID) {aka int (*)(int, const char*, void*)}
Building R package

To me, that looks like inet_pton was FOUND, but was incorrectly detected as NOT FOUND because this code can't be compiled.

This also seems relevant for the CMake side: https://cmake.org/cmake/help/latest/module/CheckCXXSymbolExists.html

CheckCXXSymbolExists command is unreliable when  is (potentially) an overloaded function. Since there is no reliable way to predict whether a given function in the system environment may be defined as an overloaded function or may be an overloaded function on other systems or will become so in the future, it is generally advised to use the CheckCXXSourceCompiles module for checking any function symbol (unless somehow you surely know the checked function is not overloaded on other systems or will not be so in the future).

Closing this in favor of #5503 , which includes the fixes discovered through this investigation.


create a group "ziti" when installing ziti-edge-tunnel

The Linux package should create group "ziti" so that ziti-edge-tunnel can detect the group exists and set file owner root:ziti and filemode 0770 on the IPC socket. Related issue: https://github.com/openziti/ziti-tunnel-sdk-c/issues/552

this was done in #614


created a vcap-local.json file

contains info on the Database

From the ReadMe doc you add the db by creating this file


The example will crash

it crash, the demo not works, because the permission thing

you need to ask permission to the user to "read data" before calling this class. You can check this out https://github.com/pankaj89/PermissionHelper

There is no need to ask permission before, in the new version, library will do that


Rename small_function to function_ref

Closes #112.

Seems like I don't have the rights to merge. There's no "Merge pull request" button in the UI, and this is my cmd output:
$ git push -vv
Pushing to github.com:cschreib/snitch.git
Enter passphrase:
ERROR: Permission to cschreib/snitch.git denied to tocic.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

I see, looks like you'd need to be formally invited as collaborator before you are able to do this. I've opened a ticket to discuss the topic (https://github.com/cschreib/snitch/issues/132), please chime in there if you have an opinion. In the meantime I will merge this myself, thanks for your contribution!


Adding Group in cluster on TS004F

If binding the remote to one group i not working.
Was testing adding the group cluster and ZHA is showing it in group and is possible being added in groups.
Then added its sending light commands (OnOff Dimm up and down) to the group without problems !!
OnOff cluster is deleted for not getting one light in the device card and looks being OK and shall not being added.
One PS for user if pressing right upper and lower bottom long its changing mode to scene and one more time to commands as the mode attribute on OnOff cluster but in scene mode its only using EP1 do not so useful until we have getting the INIT working of the 3 new endpoints in ZHA.

Codecov Report

Merging #1276 (b2bec1a) into dev (9a14013) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #1276   +/-   ##
=======================================
  Coverage   80.28%   80.28%
=======================================
  Files         207      207
  Lines        5696     5696
=======================================
  Hits         4573     4573
  Misses       1123     1123

Impacted Files
Coverage Δ

zhaquirks/tuya/ts004f.py
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

Is this good to go?

Its working on my TS004F in light mode i can "binding" / adding it to one light group and its controlling the lights. 👍


experiment: allow persistent? actor to switch default and transient as synonym for flexible

This is the simplest solution with least grammar irregularities and probably least required doc changes.

adds keywords persistent and transient.
transient is a synonym for flexible.
persistent? actor (class) ... flips the default from flexible to stable.

Parses nicely because there is no any longer ambiguity between stable/flexible uses as object modifier or visibility. That means we don't need fancy and confusing grammar factoring.

Thank you for this PR!

Nice and simple PR. Let's get this merged! Just two things that we can address in follow-up PRs:

We should also streamline documentation to this syntax. I'll ask @jessiemongeon1
We could deprecate the old actor syntax with warning in a next step. Or ask users to explicitly specify stability modifier on old actor fields.

Maybe let me take a pass first before getting @jessiemongeon1 involved. She has tons on her plate and it might be quicker for me to revise the relevant bits.

@Mergifyio update


Support DnD from fetch structure to filter panel to create a new logical group

Summary

Support DnD from fetch structure to fitler panel to create a new logical group. Fixes #1707
How did you test this change?

[ ] Test(s) added
[x] Manual testing (please provide screenshots/recordings)
[ ] No testing (please provide an explanation)

https://user-images.githubusercontent.com/73408381/202579313-ca8bc1af-0ffa-473c-a927-da7de619969d.mov

Can we add this video to the filter part of the query docs?


Includes the maps page basics

Basics of map page includes the fundamental layout of the Google maps on the maps.html page, with the map defaulted to a marker pointing to Google headquarters.

bash-profile(.save) and factorypath shouldn't be committed.

Should probably git pull then create new branches as to not have old commits from master show up as new changes here and prevent small inconsistencies like the same code on different lines


Publish to npm?

These hooks look cool, but did you consider publishing them to npm as libraries? So it will be possible to update dependencies, instead of having a constant copy inside our own repo?

P.S. The only important note here I'd like to mention, - to make possible to use these type of imports:
import useAuth from 'usehooks/useAuth'
Rather than:
import { useAuth } from 'usehooks'
Afaik, tools like NextJS optimize their buindles based on from values (well, it's not just NextJS, it's specific configuration for Webpack). I.e. by describing:
from 'usehooks/useAuth'
It will bundle only this useAuth part and its dependencies, rather than whole usehooks library.

P.P.S. npmjs.com/usehooks is free for now btw

Main reason I haven't is that these are meant to be a good starting point for writing your own custom hooks, often requiring some tweaking depending on the use-case, and I don't have a lot of time these days to manage issues / pull-requests. Will think on it in the future if it looks like I have the time.


RunProxy should return void - not chainable IApplicationBuilder

RunProxy should return void, because after proxy you should not chain anything.

Hi @mkosieradzki, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Good catch 😄
I'll merge when @Tratcher give the 👍

There's precedent https://github.com/aspnet/HttpAbstractions/blob/87cd79d6fc54bb4abf07c1e380cd7a9498a78612/src/Microsoft.AspNetCore.Http.Abstractions/Extensions/RunExtensions.cs#L19


Whether the handle of the drag can be distributed according to the slot

hi
Whether the handle of the drag can be distributed according to the vue slot ?
I need to customize the style of draggable objects...

Not sure if I understood the question, in the demo there is a raw pane which can be dragged by starting anywhere in the pane, would that be your goal?

I made some changes, any element containing pane-drag as attribute will be able to drag the view, updated demo as well, I'm not sure if this will be the final solution, but let me know if works well for you

            testing a drag handle

DRAG HERE

            test

...

Thank you for your support. It really helped me. Thank you very much.


Implement some index management

meta data management
alias management with reindexing support & atomic swap of aliases between old and new index
document the above

Currently mapping dsl adds metadata as well. May need some additional features for updating & querying that.

This will likely be post 1.0 as I've identified some additional topics related to this (e.g. ILM) that I may want to address as well


Update F# on Azure Docs - Package Management

1.Update package-management page to use the new packages.
2.Remove the reference to CloudConfigurationManager from the 4 other pages.
@jongio for notification.

//cc: @tg-msft,  @christothes

@isaacabraham - Could you please have a look?  We'd like to get this pushed for the .NET Conf.  Thank you.

Aside from the async bit of code - which doesn't appear to me to use any asynchrony at all - this looks more or less fine to me. Main comments are:

Package management. Consider removing the links to paket and nuget  and the "package manager" page (which looks out of date to me) and simply replace with a simple "how to add a package in a script" e.g. "Install the xxx and xxx packages using #r "nuget:package name" or similar.

Executing code. I don't know if there's anywhere that tells people how to execute scripts (ideally, piecemeal, section by section)? e.g. "highlight a line and press ALT + ENTER to execute it; the results are shown in the FSI console. (Rider uses a different key combo, by VS and Code both use ALT ENTER by default).

Aside from the async bit of code - which doesn't appear to me to use any asynchrony at all - this looks more or less fine to me. Main comments are:

Package management. Consider removing the links to paket and nuget  and the "package manager" page (which looks out of date to me) and simply replace with a simple "how to add a package in a script" e.g. "Install the xxx and xxx packages using #r "nuget:package name" or similar.
Executing code. I don't know if there's anywhere that tells people how to execute scripts (ideally, piecemeal, section by section)? e.g. "highlight a line and press ALT + ENTER to execute it; the results are shown in the FSI console. (Rider uses a different key combo, by VS and Code both use ALT ENTER by default).

@jongio - It seems that “package manager" article is no longer needed in 4 ReadMe files, anywhere else references the package-management.md file? If not, do we need continue to update it?

@jsquire - Thanks for your guidance! Completed all changes according to your comments. Let me know if you have any other suggestions, thanks!

//cc: @tg-msft.   Do you mind giving it a quick look to ensure that the Storage patterns are correct?

@isaacabraham and @CamSoper - Can you please have another look when you get a chance?

LGTM, let me know when you'd like me to merge it

We are good to merge.  Thank you Cam

@tg-msft and @isaacabraham - Can you please take a look and approve if good?

@tg-msft Thanks for your review, updated according to your comments.

Everyone good for me to merge?

@CamSoper Please help merge this PR.

@tg-msft - Are you good?  If so, can you please approve?  Thanks


onDispose not getting called and webparts persisting between single app pages

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Windows
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[X] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version: Version 103.0.5060.66 (Official Build) (64-bit)
SPFx version: 1.15.0
Node.js version: 14.19.1
created from @microsoft/generator-sharepoint

Describe the bug / error
Repo here
When navigating between two app pages the webpart does not seem to be disposed properly. This occurs only after navigation from one app page to another, and not if the webparts are contained in a "normal" modern page (with sections/columns).
In the gif below I have setup 2 app pages from the repo above (which is just out of the box from the generator + alert when rendered and alert when disposed). Each app page uses the same web part 'ondispose3' with a different description ('Instance 1', 'Instance 2').
When navigating between these using site navigation it seems onDispose is not being called. This leaves the render still happening even in a page like "Documents".
Works: 'Documents' -> 'App Page 1' -> 'Documents' -> 'App Page 2' -> 'Documents'
Does not work:  'Documents' -> 'App Page 1' -> 'App Page 2' -> 'Documents' ('App Page 1' still seems to be trying to render)

#6246 #6786 and #5227 all seemed somewhat related but do not seem to be directly applicable/solve the issue.
Steps to reproduce

Clone/build the repo, or create a new webpart from yo generator and add alert/console.log on render and on dispose
Add 2 new app pages with the webpart
Change the description to be unique for each
Add links to the pages in site navigation
Navigate between each like 'Documents' -> 'App Page 1' -> 'App Page 2' -> 'Documents'

Expected behavior
Webparts in the app pages should be disposed correctly between navigation and scenarios like: 'Documents' -> 'App Page 1' -> 'App Page 2' -> 'Documents'  should not cause additional renders.

Hello @AJIXuMuK...If you have had a chance to take a look at this could you let me know if this is indeed a bug or perhaps an unsupported scenario/something I am doing incorrectly?

Hello @Greg-Hitchon - yes, the issue has been fixed internally and will be rolling out WW during the next couple of weeks.

This should be rolled out WW by now.


Trunas-CSP 2.5.0

Hi Michael,
Question: is there a version 2.5.0 coming for Truenas-CSP to align to the HPE CSI 2.5.0?
Is it possible to use TruenasCSP 2.4.2 with HPE CSI 2.5.0?
Thanks!!
MS

Is it possible to use TruenasCSP 2.4.2 with HPE CSI 2.5.0?

There's been no changes to the CSP spec so you can deploy TrueNAS CSP 2.4.2 (with the YAML manifest) and the bare HPE CSI Driver Helm chart.
However, there are many improvements made in CHAP and volumeMode: Block (RWX in particular) that needs to be addressed in the TrueNAS CSP. It will take a while to implement unfortunately but I'm setting up an environment for this as we speak.

There's been no changes to the CSP spec so you can deploy TrueNAS CSP 2.4.2 (with the YAML manifest) and the bare HPE CSI Driver Helm chart.

I'll take that back. I tried this and there are issues I can't quite isolate yet so please sit tight.

There's been no changes to the CSP spec so you can deploy TrueNAS CSP 2.4.2 (with the YAML manifest) and the bare HPE CSI Driver Helm chart.

I'll take that back. I tried this and there are issues I can't quite isolate yet so please sit tight.

Ok, thank for your feedback! I'll wait for further instructions :)

Just a short update that I've made some strides in this department and have a stable build. There's still a lot to do but I'll hopefully get more time this coming looooong weekend to wrap things up.


Footer: Make localizable thanks and add support for copyright param

I speak Spanish very badly, so it is a semi automatic translation, do not hesitate to improve it ;-)

@birros Thank you for your help!


libmatroska: 1.5.0 -> 1.5.2

Motivation for this change
Fixes the build, broken by https://github.com/NixOS/nixpkgs/pull/62459
(libmatroska 1.5.0 expects libebml 1.3.7)
Currently broken on
installing 'libmatroska-1.5.0'
these derivations will be built:
  /nix/store/wyn86b98pya06xs9xw06fkikd5crxp41-libmatroska-1.5.0.drv
building '/nix/store/wyn86b98pya06xs9xw06fkikd5crxp41-libmatroska-1.5.0.drv'...
unpacking sources
unpacking source archive /nix/store/i3wq66dndcvkm5jrkhzcsfx7c6d7sgdi-source
source root is source
patching sources
configuring
fixing cmake files...
cmake flags: -DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON -DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON -DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON -DCMAKE_BUILD_TYPE=Release -DCMAKE_SKIP_BUILD_RPATH=ON -DBUILD_TESTING=OFF -DCMAKE_INSTALL_INCLUDEDIR=/nix/store/g7471azh71jvbdz5szm8yd782imh5iqz-libmatroska-1.5.0/include -DCMAKE_INSTALL_LIBDIR=/nix/store/g7471azh71jvbdz5szm8yd782imh5iqz-libmatroska-1.5.0/lib -DCMAKE_INSTALL_NAME_DIR=/nix/store/g7471azh71jvbdz5szm8yd782imh5iqz-libmatroska-1.5.0/lib -DCMAKE_POLICY_DEFAULT_CMP0025=NEW -DCMAKE_OSX_DEPLOYMENT_TARGET= -DCMAKE_OSX_SYSROOT= -DCMAKE_FIND_FRAMEWORK=last -DCMAKE_STRIP=/nix/store/2dfjlvp38xzkyylwpavnh61azi0d168b-binutils-2.31.1/bin/strip -DCMAKE_RANLIB=/nix/store/2dfjlvp38xzkyylwpavnh61azi0d168b-binutils-2.31.1/bin/ranlib -DCMAKE_AR=/nix/store/2dfjlvp38xzkyylwpavnh61azi0d168b-binutils-2.31.1/bin/ar -DCMAKE_C_COMPILER=gcc -DCMAKE_CXX_COMPILER=g++ -DCMAKE_INSTALL_PREFIX=/nix/store/g7471azh71jvbdz5szm8yd782imh5iqz-libmatroska-1.5.0 -DBUILD_SHARED_LIBS=YES
-- The C compiler identification is GNU 7.4.0
-- The CXX compiler identification is GNU 7.4.0
-- Check for working C compiler: /nix/store/ds1prvgw3i3ic8c7axyrw4lwm3d0gqab-gcc-wrapper-7.4.0/bin/gcc
-- Check for working C compiler: /nix/store/ds1prvgw3i3ic8c7axyrw4lwm3d0gqab-gcc-wrapper-7.4.0/bin/gcc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /nix/store/ds1prvgw3i3ic8c7axyrw4lwm3d0gqab-gcc-wrapper-7.4.0/bin/g++
-- Check for working CXX compiler: /nix/store/ds1prvgw3i3ic8c7axyrw4lwm3d0gqab-gcc-wrapper-7.4.0/bin/g++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at CMakeLists.txt:8 (find_package):
  By not providing "FindEbml.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "Ebml", but
  CMake did not find one.

  Could not find a package configuration file provided by "Ebml" (requested
  version 1.3.7) with any of the following names:

    EbmlConfig.cmake
    ebml-config.cmake

  Add the installation prefix of "Ebml" to CMAKE_PREFIX_PATH or set
  "Ebml_DIR" to a directory containing one of the above files.  If "Ebml"
  provides a separate development package or SDK, be sure it has been
  installed.

-- Configuring incomplete, errors occurred!
See also "/build/source/build/CMakeFiles/CMakeOutput.log".
builder for '/nix/store/wyn86b98pya06xs9xw06fkikd5crxp41-libmatroska-1.5.0.drv' failed with exit code 1
error: build of '/nix/store/wyn86b98pya06xs9xw06fkikd5crxp41-libmatroska-1.5.0.drv' failed

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc @worldofpeace

cc maintainer @spwhitt

@ofborg build cantata libmatroska libsForQt5.phonon-backend-vlc libsForQt5.vlc libsForQt511.phonon-backend-vlc libsForQt511.vlc libsForQt512.phonon-backend-vlc libsForQt512.vlc linphone mediastreamer mediastreamer-openh264 megaglest minitube mkvtoolnix mkvtoolnix-cli msilbc obs-linuxbrowser obs-studio phonon-backend-vlc tomahawk tribler vlc vlc_qt5

@GrahamcOfBorg build cantata libmatroska libsForQt5.phonon-backend-vlc libsForQt5.vlc libsForQt511.phonon-backend-vlc libsForQt511.vlc libsForQt512.phonon-backend-vlc libsForQt512.vlc linphone mediastreamer mediastreamer-openh264 megaglest minitube mkvtoolnix mkvtoolnix-cli msilbc obs-linuxbrowser obs-studio phonon-backend-vlc tomahawk tribler vlc vlc_qt5

Might be reasonable to merge and unbreak master instead of waiting for ofborg to start, libmatroska and vlc and megaglest built fine on my machine.

@GrahamcOfBorg build libmatroska

Thanks for fixing this @ivan so quickly


Feature/issue 347 accept header

These commits update Reader.open to attempt to find a concrete reader before calling Util::File.open_file and sets the Accept header narrowly if it can. This fixes a problem when a specific format is selected, but a request is made which ends up returning a different content-type due to content negotiation. This narrows the acceptable content types.
Fixes #347.

Coverage increased (+0.6%) to 91.517% when pulling b7085814fde9d0cb6c6c9b7c52e6ac1a6381fae4 on feature/issue-347-accept-header into 186dd2d04bcbdf38b7d04a7a4f8ddabf0c32b450 on develop.


Detect cedar-14 stack, or fail

I'm unsure why this is necessary considering how old the cedar stack is now. Has this been an issue that you've encountered?

@sorentwo It's not. But it's something I've done on other buildpacks. It might be extra cautious, and perhaps unwarranted. Feel free to close if you like.

It seems sane, and I appreciate the PR.


React Recipe Application

Prerequisites

[X] I am a GSSoC-22 Participant
[X] I checked to make sure that this issue has not already been filed

Description
A Recipe Application , which will give you chef's abilitites, to cook your favourite food
Tech Stack
ReactJS

@heet-kumar @manisha-27 Please Assign the Task

closed with #440


Problems saving / restoring multi-word adaptations

Adapt It Mobile Version: 0.5.9 (pre-beta release for Graeme)
Operating System: iOS
OS Version: 10.1
Severity: 3
Description:
On testing the pre-beta release on an iPad Mini, Graeme noticed some instances where single word -> multiple word adaptations and muti-word (phrase) adaptations were not persisting properly. From Graeme -
"On restarting AIM later, I noticed that all the single word substitutions were retained and applied, but that “write” -> “am writing” and “are forgiven you” -> “are forgiven” were not applied again:-
But “are forgiven you” - which I had selected to be a phrase - was simply copied to the target phrase rather than being substituted with “are forgiven”.
So perhaps AIM is not handling multiple word phrases properly?"
Steps to Reproduce:
[EB] I did see this behavior once on my Android tablet with a merged phrase, but was unable to reproduce it on subsequent tries with either single or multi-word adaptations. I am seeing this on iOS more frequently. Steps to reproduce:

Launch AIM and start adapting
On a single word source phrase, enter in a target text with multiple words.
Select several words and create a phrase from them. Type in something other than the source text as the adaptation.
Exit to the main screen of AIM, and go back to adapt again.

--> You should see the adaptations you entered earlier if it's working correctly. If the bug is still there, you'll see either blank entries for the target text, or the same text copied from the source to the target.

Moving this up to the 1.0 beta milestone.

Marking this issue as closed:
There was an underlying problem in the database schema upgrade mechanism that might have been the issue for this. I cleaned up the DB schema check / upgrade code, and I have not seen this behavior in either Android or iOS (on a simulator!) since.
I'll be kicking off a "real" iOS build of the AIM beta shortly, so this can be verified on a real device. If this is still an issue, we can reopen an address as a post-beta fix.


Recent breaking change now ignores SlotName and deploys straight to Production!

This recent change breaks existing behaviour in quite a dangerous way, by silently removing the SlotName setting if the DeployToSlotOrASEFlag setting is not supplied.
The documentation is not super clear on the relationship between these settings, and up until this change a specified SlotName was respected if supplied, regardless of the value of DeployToSlotOrASEFlag
Consider an existing pipeline yaml script which supplies a SlotName but no DeployToSlotOrASEFlag, in order to deploy to a staging slot as a prelude to final testing before swapping that slot into production:
  - task: AzureFunctionApp@1
    inputs:
      azureSubscription: ${{parameters.subscription}}
      appType: functionApp
      appName: ${{parameters.app}}
      slotName: ${{parameters.slot}}
      package: '$(System.ArtifactsDirectory)/drop/netcoreapp${{parameters.netCoreVersion}}'

This change means that the same script will now have the SlotName parameter silently cleared and the deployment will now be direct to production!
My strong preference, if it is felt that there is a need to enforce that DeployToSlotOrASEFlag is supplied along with SlotName, would be to have the script fail with a validation error, rather than silently ignore SlotName. This is especially because silently altering the behaviour of scripts that previously deployed to a slot, to deploy direct to production, is likely to have serious implications.

Sorry to say but this document clearly specifies that slot Name is required field when deployToSlotOrASE is true

this fix was made to solve this issue

When using a Function App Deployment task if the user is activating and customizing the “Deploy to Slot or App Service Environment” setting to target a different slot to the default one(production) saves the config and runs the pipeline if afterwards he disables the option, saves the pipeline configuration, and performs a run again the pipeline will still target the configured disabled slot rather than returning to the default behavior which would imply targeting the production slot.

What you're saying is correct, but that's not really the point I'm making here. You're correct that the docs say slotName is required if deployToSlotOrASE is set to true, but this implies that slotName is optional if deployToSlotOrASE is not set to false, and indeed this has been the behaviour of the service up to this point.
Nowhere in the docs does it say that the flag is only allowed if deployToSlotOrASE is true, nor does it state that the slot name will be silently ignored if deployToSlotOrASE is false or not supplied.
As I've pointed out on the ticket, this is a breaking change vs existing behaviour, and a very dangerous one because it can result in an existing script with the slotName set, silently changing to deploy direct to production.
My suggestion here would be that if this is the intended behaviour, the docs should certainly be made clearer (also incidentally resourceGroupName is not required if deployToSlotOrASE is true).
But really, if slotName is not supported when deployToSlotOrASE is not supplied (given especially that it was supported up until now), that this condition (slotName supplied but deployToSlotOrASE not supplied) should result in a validation error.

We will update documentation.

...and a validation error if slotName is supplied but deployToSlotOrASE is not?
Changing from respecting the slot name to ignoring it, without so much as emitting a warning to logs, still feels wrong to me, especially since we're agreed the docs are not clear that the previous behaviour is unsupported.

I ran into this as well - previously just setting the slotName deployed to the slot but with this change it silently just deploys to the Production slot.  Luckily we noticed it in our development environment versus production.  When I setup the pipeline it definitely deployed to the slot with just the slotName specified, so I never expected another property needed to be set.

For classic pipeline, slot name field is not visible on UI unless you select deploy to slot option

Even if you unselect Deploy to slot option value was persisting and user who was expecting it to deploy to prod was getting deployed to slot.
Due this buggy behaviour it was working for yaml based pipelines. So we won't set any validation error if slotName is supplied but deployToSlotOrASE is not this will unnecessarily break pipeline for classic pipeline users.
we will update document to make it more explicit.

Right, so imo the fix here ought to be that the UI should be clearing the value of slotName when the screen is saved with "deploy to slot" unchecked. That would avoid the breaking change you've made here to the behaviour with yaml pipelines. It also just seems like a bad idea from a UI or elsewhere to persist a setting that you know you're going to ignore at runtime - why not just not persist that setting?
Hopefully I've made this point clearly now, but it's clear that:

The actual behaviour has changed in a dangerous, breaking way
The mode of usage setting slotName without deployToSlot is not prohibited by the docs
Others have also been relying on this behaviour.

It seems to me you can fix the reported issue with Classic Pipelines without making a breaking change to yaml pipelines. That's what I would like to ask that you do here.

This change has made the behaviour of AzureWebApp@1 and AzureFunctionApp@1 inconsistent.
I fear that a change will be made to AzureWebApp@1 without warning as well.


[BUG] core, in RetryPolicy.attemptSync, delayDuration is not used

Describe the bug
code, attemptSync in azure-core: https://github.com/Azure/azure-sdk-for-java/blob/775e676/sdk/core/azure-core/src/main/java/com/azure/core/http/policy/RetryPolicy.java#L188-L199
Here, the delayDuration is calculated and logged, but never actually used.
Exception or Stack Trace
Add the exception log and stack trace if available
To Reproduce
Steps to reproduce the behavior:
Code Snippet
Add the code snippet that causes the issue.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):

OS: [e.g. iOS]
IDE: [e.g. IntelliJ]
Library/Libraries: com.azure:azure-core:1.45.0
Java version: [e.g. 8]
App Server/Environment: [e.g. Tomcat, WildFly, Azure Function, Apache Spark, Databricks, IDE plugin or anything special]
Frameworks: [e.g. Spring Boot, Micronaut, Quarkus, etc]

If you suspect a dependency version mismatch (e.g. you see NoClassDefFoundError, NoSuchMethodError or similar), please check out Troubleshoot dependency version conflict article first. If it doesn't provide solution for the problem, please provide:

verbose dependency tree (mvn dependency:tree -Dverbose)
exception message, full stack trace, and any available logs

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Bug Description Added
[ ] Repro Steps Added
[ ] Setup information Added

This was fixed by https://github.com/Azure/azure-sdk-for-java/pull/37788


style(footer): Improved footer on phones

PR related to this trello task --> https://trello.com/c/3WQhvxeK

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


"Database already closed" error in unit test expect()

Hey guys,
I'm having a hard time to do some unit tests.
If this is more suited for the discussions page, feel free to say so.
We are currently using moor 3.4.0 and moor_generator 3.4.1.
Basically, what I try to do is the following:

Doing a unit test
Create a new DAO object, using a in memory SQLite database
Insert data into the DAO using an input method
check, if everything runs without throwing an exception

This is some sample code (I'll omit the DAO class itself. if needed, just tell me):
test('Test insertFeedback of FeedbackDao on sane input', () async {
    FeedbackDao testDao = FeedbackDao(testDb);
    final testId = 'uuid';
    final testUserName = 'Some Name';
    final testFeedback = 1;
    final testTimestamp = DateTime.parse('1969-07-20T20:18:04');
    final testFeedbackText = 'Some Feedback!';

    var userFeedback = new UserFeedback(
        id: testId,
        userName: testUserName,
        feedback: testFeedback,
        timestamp: testTimestamp,
        feedbackText: testFeedbackText);
    expect(() async => await testDao.insertFeedback(userFeedback), returnsNormally);
  });

When I run this test, I'll run into a Bad state: This database has already been closed error in the debug console.
Full output is:
DatabaseImpl._ensureOpen
package:sqlite3/…/impl/database.dart:295
DatabaseImpl.execute
package:sqlite3/…/impl/database.dart:90
_VmDelegate._runWithArgs
package:moor/…/ffi/vm_database.dart:160
_VmDelegate.runCustom
package:moor/…/ffi/vm_database.dart:170
_ExecutorWithQueryDelegate.runCustom.<fn>
package:moor/…/helpers/engines.dart:79
_ExecutorWithQueryDelegate._synchronized
package:moor/…/helpers/engines.dart:25
_ExecutorWithQueryDelegate.runCustom
package:moor/…/helpers/engines.dart:76
QueryEngine.customStatement.<fn>
package:moor/…/api/query_engine.dart:301
QueryEngine.doWhenOpened.<fn>
package:moor/…/api/query_engine.dart:98
===== asynchronous gap ===========================
dart:async                                                               _asyncErrorWrapperHelper
InsertStatement.insertOnConflictUpdate
package:moor/…/statements/insert.dart:89
FeedbackDao.insertFeedback
package:somePrivatePackageName/…/user_feedback_curd/user_feedback_curd.dart:18
main.<fn>.<fn>
test\database\user_feedback_curd_test.dart:33
_ReturnsNormally.typedMatches
package:matcher/src/core_matchers.dart:153
FeatureMatcher.matches
package:matcher/src/feature_matcher.dart:16
package:test_api                                                         expect
main.<fn>
test\database\user_feedback_curd_test.dart:33
main.<fn>
test\database\user_feedback_curd_test.dart:17
2

✖ Test insertFeedback of FeedbackDao on sane input
Exited (1)

It seems to be a problem with the except. When I switch the last line of the first code block to await testDao.insertFeedback(userFeedback);, everything runs perfectly.
For the sake of it, I tried reading from the DAO in the same method later on and it works perfectly.
I believe there is something elementary, what I'm missing.
Any hint is much appreciated.
Thanks,
Matthias

How do you close the database? In a tearDown block? Basically, I think what's happening here is

Your test code completes (since expect is a synchronous function)
Your tearDown handler is called, which closes the database
insertFeedback is called after an asynchronous delay

That schedule would explain the error you're seeing. In general, it's always a good practice to make sure that your test function doesn't leave some further async work behind. For asynchronous expectations, you can use expectLater for that:
await expectLater(() async => await testDao.insertFeedback(userFeedback), returnsNormally);

Thanks for the fast reply!
Yes, you are right. I do it with tearDown:
    testDb = AppDatabase.inMemory(VmDatabase.memory());
  });

  tearDown(() async {
    await testDb.close();
  });```

I had the strong believe, that tearDown waits.

I've switched my `expect()` line to your `expectLater()`, but it emits the error as well:

```Bad state: This database has already been closed
DatabaseImpl._ensureOpen
package:sqlite3/…/impl/database.dart:295
DatabaseImpl.execute
package:sqlite3/…/impl/database.dart:90
_VmDelegate._runWithArgs
package:moor/…/ffi/vm_database.dart:160
_VmDelegate.runCustom
package:moor/…/ffi/vm_database.dart:170
_ExecutorWithQueryDelegate.runCustom.<fn>
package:moor/…/helpers/engines.dart:79
_ExecutorWithQueryDelegate._synchronized
package:moor/…/helpers/engines.dart:25
_ExecutorWithQueryDelegate.runCustom
package:moor/…/helpers/engines.dart:76
QueryEngine.customStatement.<fn>
package:moor/…/api/query_engine.dart:301
QueryEngine.doWhenOpened.<fn>
package:moor/…/api/query_engine.dart:98
===== asynchronous gap ===========================
dart:async                                                    _asyncErrorWrapperHelper
test\database\user_feedback_curd_test.dart                    main.<fn>.<fn>
test\database\user_feedback_curd_test.dart:1
_ReturnsNormally.typedMatches
package:matcher/src/core_matchers.dart:153
FeatureMatcher.matches
package:matcher/src/feature_matcher.dart:16
package:test_api                                              expectLater
main.<fn>
test\database\user_feedback_curd_test.dart:33
main.<fn>
test\database\user_feedback_curd_test.dart:17
2

✖ Test insertFeedback of FeedbackDao on sane input
Exited (1)

I had the strong believe, that tearDown waits.

You're right, it does. The problem is that expect does not, so parts of your test and tearDown would run concurrently.
I was wrong about expectLater though, the problem is the returnsNormally matcher. As you can see here, it does not actually check for asynchronous errors. So it just sees that the async function returns a future and is happy with that. Meanwhile, the future keeps running and eventually hits the error. The completes matcher can be used to match a future that doesn't complete with an error, but it's a bit awkward to use with functions. I'd probably use it like this:
await expectLater(Future(() async => await testDao.insertFeedback(userFeedback)), completes);

IIRC, you don't even need expectLater in this case, since the test runner waits until all async matchers reported a result.

You are my true hero of the week!
Honestly, this is amazing and makes me very happy. Wouldn't have found it on my own.
Everything works fine now, cross-tested it with:
  Future insertFeedback(UserFeedback feedback) =>
      throw new FormatException();

iam workin with sqflite: ^2.2.8+4
i have erro in near "ON": syntax error
E/flutter ( 9325): [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: DatabaseException(near "ON": syntax error (code 1): , while compiling: INSERT OR REPLACE INTO paycode
pleass hellp me!!!!


Docs? API calls available?

What functions are available on an Index object? I don't see any docs.
I was unable to answer this even after console-logging an Index object:
Index {
  indexName: 'whatever',
  as:
   AlgoliaSearchNodeJS {
     applicationID: '<snipped>',
     apiKey: '<snipped>',
     hosts: { read: [Object], write: [Object] },
     _timeouts: { connect: 2000, read: 5000, write: 30000 },
     _hostIndexes: { read: 0, write: 0 },
     _timeoutMultiplier: 1,
     _shuffleResult: [ 1, 2, 3 ],
     extraHeaders: [],
     cache: {},
     _ua: 'Algolia for Node.js 3.22.1',
     _useCache: false,
     _useFallback: true,
     _setTimeout: undefined,
     _Agent:
      Agent {
        domain: null,
        _events: [Object],
        _eventsCount: 1,
        _maxListeners: undefined,
        defaultPort: 443,
        protocol: 'https:',
        options: [Object],
        requests: {},
        sockets: {},
        freeSockets: {},
        keepAliveMsecs: 1000,
        keepAlive: true,
        maxSockets: Infinity,
        maxFreeSockets: 256,
        maxCachedSessions: 100,
        _sessionCache: [Object] } },
  typeAheadArgs: null,
  typeAheadValueOption: null,
  cache: {} }

i.e. how do I update an object by ID? I can see what REST calls to make for this in Algolia's docs, but it isn't clear even by looking through this codebase, what function to call.

Have a look here: https://www.algolia.com/doc/api-client/javascript/indexing/#update-objects


NoMethodError: undefined method `extname' for "/.../../asset.gif":String

Hi,
the newest version has in lib/eslint-rails/runner.rb line 31 Rails.application.assets.each_file.to_a.select { |pn| pn.extname == '.js' }   .. Rails.application.assets.each_file.to_a returns an array of strings, not files, therefore string.extname trhows an error .. any suggestions?
Thanks :)

@lubosch
I've had a same error on my setup (not sure if it is related to Ruby/Rails version), have a fork for this fix:
gem 'eslint-rails', github: 'filippovG/eslint-rails', branch: 'fix-extname-undefined'

Hmm.. I'm certain this was working before. I'll fix it.

Looks like it changed in Rails 4.2. Fixing.

Released version 1.2.1 with the fix. Thank you for reporting this.

Thank you, everything is working now


Fix Actions Click Behavior for ApplicantBadge Component

Currently, the ApplicantBadge component is not focusing the correct card when clicking an action.

To Do
Focus the correct applicant card when clicking an action.
So there's no delay when using the dropdown...
Acceptance Criteria

[ ] The correct applicant card must be focused when clicking an action.

@relztic Thanks for raising that, as per our discussion 👍


animation does not show

If you do something like this:
var ch = document.createElement('div');
var ctx_div = document.createElement('canvas');
ch.appendChild(ctx_div);
var ctx = ctx_div.getContext('2d');
var chart = new Chart(ctx, chart_options);
document.body.appendChild(ch);

Then the chart shows but does not animate.
The reason is because the canvas element was added to ch then the chart was instantiated.  After that the ch element was added to document.body.
It works if you add the ch element to the document body before instantiating the chart, like this:
var ch = document.createElement('div');
var ctx_div = document.createElement('canvas');
ch.appendChild(ctx_div);
document.body.appendChild(ch);
var ctx = ctx_div.getContext('2d');
var chart = new Chart(ctx, chart_options);

The problem with this is that there is no reason in JS that it should not work with both and many times when using the library in projects you need to structure things like that for certain conditions to be met.

And most important, it would make it easier for people so we can see some new things.

That is for 2.9.3.
Situation is worse in master:
Chart.js:8230 Uncaught RangeError: minimumFractionDigits value is out of range.
    at new NumberFormat (<anonymous>)
    at LinearScale.numeric (Chart.js:8230)
    at callback (Chart.js:783)
    at LinearScale.generateTickLabels (Chart.js:8699)
    at LinearScale._convertTicksToLabels (Chart.js:8855)
    at LinearScale.update (Chart.js:8592)
    at fitBoxes (Chart.js:7013)
    at Object.update (Chart.js:7124)
    at Chart._updateLayout (Chart.js:7745)
    at Chart.update (Chart.js:7723)

https://codepen.io/kurkle/pen/MWaowYj

It looks like the tick value is NaN. We could add some kind of default value in that case. @benmccann thoughts?

I guess we could return empty string or something. But the real issue seems to be before the tick formatter. I don't think the tick value should be NaN should it?

I don't think so either. I'm guessing something measured is undefined and we've silently converted to NaN instead of assuming 0. I'll see if I can trace it.

I've also run into this issue, the cause seems to be _calculatePadding here returning NaN instead of 0.
@etimberg your commit should resolve the the NaN issue.
This resolves the runtime error on master - but the animation issue is still present.

The issue with the animation is that it runs before the node is added to the DOM as shown in https://codepen.io/etimberg/pen/xxwQdJd.
I'm not sure what the solution to this is. If this case could be detected reliably, we could block the animations until the node is attached but I'm not sure if that would break any existing use cases. @kurkle thoughts?


Remove go 1.7 support

It looks like the go 1.7 build is failing with this output:
../../mjibson/esc/embed/embed.go:181: undefined: sort.Slice
The command "go get github.com/mjibson/esc" failed and exited with 2 during .
The most recent commit on mjibson/esc is using a function which is not available on go1.7.
Would you prefer trying to pull a specific version of mjibson/esc or to drop support for go1.7?

Coverage remained the same at 96.358% when pulling 0e4b51c93440099f83f4b28b0372f433da118ffb on zacatac:remove-go17-support into c51312f508e7040f06d1eb174da68364155b1fc0 on cweill:master.

Coverage remained the same at 96.358% when pulling 0e4b51c93440099f83f4b28b0372f433da118ffb on zacatac:remove-go17-support into c51312f508e7040f06d1eb174da68364155b1fc0 on cweill:master.

Coverage remained the same at 96.358% when pulling 0e4b51c93440099f83f4b28b0372f433da118ffb on zacatac:remove-go17-support into c51312f508e7040f06d1eb174da68364155b1fc0 on cweill:master.

Thanks Charles


Android 6.0.1 TextInput: Spacebar nor swipe nor autocomplete works?

I'm having a number of issues with TextInput on Android 6.0.1 (stock Nexus device) using python3crystax.  My .kv just is a standard TextInput... Nothing fancy.  Nonetheless:

The space bar does not insert anything.
The first letter does not default to a capital letter.
The keyboard shown does not have word predictions or allow swiping to enter words; only a single letter at a time.  It also has the numeric row showing by default, which is weird.

I've tried Config.set('kivy', 'keyboard_mode', 'system') to no avail.  Any ideas?  I find it very weird that Kivy's default doesn't seem to use the system keyboard in text mode on a mobile device.  I've also tried different input_type values, and none of them seemed to change anything.  I'm using 1.9.2dev from commit 1b4e49b6f1789cc74efa325aa74c749f132755fe.
I'm using the stock keyboard on 6.0.1: English (US) Google Keyboard.    Normally this keyboard has swipe, word completion, etc.  None of that seems to be working.  Also, interestingly, I can press "enter" for newlines, but no spaces.
Since this isn't a known issue and it's super weird, I've went ahead and deleted both ~/.buildozer and the local .buildozer folders in my project, then rebuilt.  This did not fix the issue.

@akshayaurora Any thoughts?

PS, This is also using CrystaX NDK 10.3.2.  Basically, I just started a new Kivy project and so am using the latest version of everything on my device.  It's great how well everything works on the device... except for the TextInput.

I have seen the same issue with the new toolchain. The new toolchain does override the IME methods but does not account for the IME... we need to hook into sdl2 for this. Will look into this tonight as I need it for my own app too now.

I have a similar problem. I describe it here #4611

Any ideas on this?  Or an ETA?  Just wondering

No ETA, it will depend on when someone has time to look at it. That
could be qua-non if he has a direct problem with it already, or I've
been looking at the keyboard for a different reason and will fix it on
the way if I come across it (although I can't immediately duplicate the
spaces problem).
On 16/09/16 18:13, Walt Woods wrote:

Any ideas on this? Or an ETA? Just wondering
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kivy/kivy/issues/4605#issuecomment-247655789, or
mute the thread
https://github.com/notifications/unsubscribe-auth/ABNQmzqxU6XtfUj5pzJDRJjFa3jcdlEoks5qqs4jgaJpZM4J7HZR.

Same Problem here, keyboard spacebar is not working. The spacebar only contains the language label "Deutsch" for my swiss/german keyboard on Android 6.0.1. and kivy 1.9.2 dev. Does the label catch the touch event and prevent inserting  space?

I've tracked the space bar not working to (I think) a problem in kivy's sdl2 event handling - I don't think it's responding correctly to the spacebar scancode, but most keyboards pass a different event type that does work.

I've was debugging a bit. There is something wrong with the TextInputEvent in sdl2.pxi for an empty space. It does not get fired for 32L (an empty space) e.g ' '. The keycode mapping seems to be correct and the TextInput widget will receive the key_down event within the correct keycode and text. But nothing handles this usecase.
sdl2.pxi
    cdef struct SDL_TextInputEvent:
        Uint32 type                               # ::SDL_TEXTINPUT */
        Uint32 timestamp
        Uint32 windowID                     # The window with keyboard focus, if any */
        char *text                                 # The input text */

Does "char *text" include a string with only an empty space?
The issue is related to the sdl2 Event handling. The TextInput widget does nothing and will leave the empty space handling from spacebar in TextInput.keyboard_on_key_down at the line 2401 to the WindowSDL  Event handling in mainloop :
textinput.py
if EventLoop.window.__class__.__module__ == \
    'kivy.core.window.window_sdl2':
    return

The WindowSDL does not receive a TexInputEvent with the event action 'textinput' and does not handle the text Input in WindowSDL._mainloop at line: 582
window_sdl2.py
elif action == 'textinput':
    text = args[0]
    self.dispatch('on_textinput', text)

The event setup for keycode 32L a string with only an empty space does not happen. But I don't know where this Event Setup should happen: I guess that's the SDL2 lib right?
My current hack to support empty space ' ' in a custom TextInput widget is the following:
    # XXX: remove if issue is resolved https://github.com/kivy/kivy/issues/4605
    def keyboard_on_key_down(self, window, keycode, text, modifiers):
        old = self.text
        # I hope super call this is not asynchron ;-)
        rv = super(WidgetTextInputBase, self).keyboard_on_key_down(
            window, keycode, text, modifiers)
        if text == ' ' and self.text == old:
            # add space if not added yet
            # review if space will not get added twice on other device or on
            # pygame because of cloked text insert
            self.text = self.text + text
        return rv

I hope that helps a bit, have a nice Weekend, Roger

I've submitted a PR at https://github.com/kivy/kivy/pull/4712 that works around the widespread space input problem and discusses the reasons for it. I'm not sure if it's safe though - as noted in the link, it could cause more problems if there are any devices/keyboards with a particular different behaviour.

I wondered if a safe, if inconvenient solution would be to log all the textinput and keydown events each frame, then handle them all at the end of the frame when it's apparently safe (I'm not sure if this is guaranteed) to assume any textinput and keydown events will all have been fired.

Can we have keyboard in kivy applications that has word-predictions feature by not using system keyboard and some other software keyboard ?

Any solutions for this issue ? I hope that someone knows what's happening with textinput in android :/

@rlimaeco I'm not aware of anyone having looked into it, although it's near the top of my things to look at.

Could you give some informations about the modules that I have to understand, I'll need to study pyjnius and how things really work, to find a way to help if it's possible hehehe. I'm learning kivy since 1.8 and I really like this framework and how it's evolving. Thanks for everything


Flexible Content similar?

Hi. Is there a control similar with ACF Flexible Content? Where we can reorder layouts / fields? Thanks

Hi,
there is no such field currently. You might be able to achieve this with the existing control types & some custom logic.
Maybe the repeater field or sortable field might come in handy here.
Best,
David


Fix advanced SQL breaking bugs

Related to: https://github.com/CartoDB/cartodb/issues/13603, https://github.com/CartoDB/cartodb/issues/9974.
Changes:

Fix infinite loop bug introduced in #12916.
Fix blank panels bug introduced in #13324.
Fix Fix them redirection bug. Related to #13090.
Better error handling: the_geom, the_geom_webmercator, cartodb_id.

Acceptance
BEFORE

SQL
UI message
Blank panels
Inf. loop

1
SELECT x FROM
Error in SQL query: column "x" does not exist
Yes
No

2
 SELECT the_geom FROM
The column "the_geom_webmercator" does not exist
No
No

3
 SELECT the_geom_webmercator FROM
-
No
Yes

4
SELECT the_geom, the_geom_webmercator FROM
lineas_metro_madrid: Postgis Plugin: ERROR: column "cartodb_id" does not existLINE 1: ...e_geom_webmercator",0.001),0.001,true),3) AS geom,"cartodb_i... ^in executeQuery Full sql was: 'SELECT ST_AsTWKB(ST_Simplify
No
No

5
SELECT cartodb_id FROM
-
Yes
Yes

6
SELECT 1 as x FROM
-
Yes
Yes

7
SELECT * FROM wrong_table
Error in SQL query: relation "wrong_table" does not exist
Yes
No

AFTER

SQL
UI message

1
SELECT x FROM
Error in SQL query: column "x" does not exist

2
 SELECT the_geom FROM
The column "the_geom_webmercator" does not exist

3
 SELECT the_geom_webmercator FROM
The column "cartodb_id" does not exist

4
SELECT the_geom, the_geom_webmercator FROM
The column "cartodb_id" does not exist

5*
SELECT cartodb_id FROM
The column "the_geom_webmercator" does not exist

6*
SELECT 1 as x FROM
The column "the_geom_webmercator" does not exist

7
SELECT * FROM wrong_table
Error in SQL query: relation "wrong_table" does not exist

NOTE
The cases 5* and 6* have the blank panels bug and infinite loop bug FIXED, but a review is needed in the returned error, because now it returns an Error in SQL query error and they should return a The column does not exist warning. This may be related with the Maps API responses. New ticket: https://github.com/CartoDB/cartodb/issues/13642.

Tested in staging, looks pretty good, I've found a problem when having an analysis applied, if you're in the data tab and reload the page, it renders nothing:

If then you click on the map <-> data switch it renders, so I think it's missing a listener somewhere to trigger a render after something loads.

Acceptance
Good job. Now nothing breaks badly as before and the user can continue working and she's notified on what's wrong (the SQL query).
However, blank panels are still happening on 2, 3 and 4. I don't consider them critical because now it's shown blank, but it doesn't break the layout and at least there is an infobox telling the user that something is wrong with the query.
Bandera 🇯🇲

The icon of empty layer shows up but it'll be tackled in another issue. Don't find it critical neither because the layer is broken in any case.

Hi @rubenmoya. Good catch! I'll take a look.
Hi @ivanmalagon. The blank panels bug is not happening anymore. The behavior that you saw in cases 2,3,4 is the expected behavior. When there is a Warning message (the_geom_webmercator or cartodb_id) the DATA tab should be empty until the SQL query is updated and the rest of the tabs should work without any error message.
Thanks for the acceptance!

I'm going to deploy to staging the fix for the empty DATA on reload.


something about mpeg testdata

I noticed that the MPEG test data includes three sequences from Ford, but only 1/50th of the sample is in the repository. please ask me how to apply for mpeg membership in order to download the full dataset, which is mentioned at https://box.nju.edu.cn/f/82b25a75e63c49fabfb0/. Thank you very much for your help!

Hi,
Sorry, I don't know how to apply for MPEG membership.
This is the link of Ford_01_q1mm dataset in NJU box: https://box.nju.edu.cn/f/7c1f3416ad4a4f7b8d85/
Jianqiang


Store

Hola, This tool is effective and very powerful. I love it. But I have a problem that I hope you can help me solve. Microsoft store does not work correctly after applying the script. Could you tell me how to make it work again? I try to check for updates in the store and a triangle appears with an s Exclamation mark and when searching for the apps I use the store closes. Please help me get it back

To fix Microsoft Store, you will have to enter powershell (as admin) and enter the following:
Get-AppxPackage -allusers Microsoft.WindowsStore | Foreach {Add-AppxPackage -DisableDevelopmentMode -Register "$($_.InstallLocation)\AppXManifest.xml"}
If that doesn't work, try running a widget command from the command prompt (as admin):
winget install 9WZDNCRFJBMP

To fix Microsoft Store, you will have to enter powershell (as admin) and enter the following:
Get-AppxPackage -allusers Microsoft.WindowsStore | Foreach {Add-AppxPackage -DisableDevelopmentMode -Register "$($_.InstallLocation)\AppXManifest.xml"}
If that doesn't work, try running a widget command from the command prompt (as admin):
winget install 9WZDNCRFJBMP

Thank you. I solved it by starting a service that the script disabled. But I have 2 more problems. Hello. Please help me, after using the script. The key prtsc does not capture the screen like it used to. I pressed fn+ptrsc and the snipping tool opened to take a screenshot.How can I solve that?
The other problem is that the Bluetooth stopped working. I loved the tool. I only have these two problems.

Solved


Dreamview is blank in browser

System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04):
Linux Ubuntu 18.04
Apollo installed from (source or binary):
source
Apollo version (3.5, 5.0, 5.5, 6.0):
master
Output of apollo.sh config if on master branch:
[INFO] Apollo Environment Settings:
[INFO]     APOLLO_ROOT_DIR: /apollo
[INFO]     APOLLO_CACHE_DIR: /apollo/.cache
[INFO]     APOLLO_IN_DOCKER: true
[INFO]     APOLLO_VERSION: master-2021-11-16-a379b21230
[INFO]     DOCKER_IMG: dev-x86_64-18.04-20210517_1712
[INFO]     APOLLO_ENV:  STAGE=dev USE_ESD_CAN=false
[INFO]     USE_GPU: USE_GPU_HOST=1 USE_GPU_TARGET=1

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:

1.when run bash ./scripts/bootstrap.sh start in docker, the output is as follow.
dreamview module is launched，but when open localhost:8888 in browser, it's blank.
nohup: appending output to 'nohup.out'
[ OK ] Launched module monitor.
nohup: appending output to 'nohup.out'
[ OK ] Launched module dreamview.

acording to FAQ of dreamview, I change server_ports to 5555， but dreamview is still blank，

when run "ps aux | grep dreamview" in docker, the output is

/usr/bin/python3 /apollo/bazel-bin/cyber/tools/cyber_launch/cyber_launch.runfiles/apollo/cyber/tools/cyber_launch/cyber_launch.py start /apollo/modules/dreamview/launch/dreamview.launch；
/apollo/bazel-bin/modules/dreamview/dreamview --flagfile=/apollo/modules/common/data/global_flagfile.txt；

when run "lsof -i :8888" in localhost, it display:

COMMAND PID USER FD TYPE DEVICE SIZE/OFF NODE NAME
dreamview 18356 ****    3u IPv4 3020969 0t0 TCP *:8888 (LISTEN)

And the building of cyber_visualizer is failed, so I comment out it.

Supporting materials (screenshots, command lines, code/script snippets):

Closed due to inactivity. If the problem persists, pls feel free to reopen it or create a new one and refer to it.


⬆ upgraded downshift to 7.0.1

resolves #2633

Need to update snapshots for the components changed 📸

Might I suggest its maybe time to remove MultiSelect and SingleSelect? 🙂

Might I suggest its maybe time to remove MultiSelect and SingleSelect (in a separate issue ofc)? 🙂

We literally discussed this yesterday, and that is indeed the plan (in a seperate issue) 😀

Downshift should be updated in eds-labs-react too since it is used by TimePicker


Format with rustfmt.

The world's most trivial patch, but this will allow us to update the lightbeam submodule in the Wasmtime tree.

I think rustfmt must have updated, because I just ran rustfmt locally and it ended up with a different result

Quite possible. The wasmtime CI uses "stable", so that recently changed to Rust 1.37.


fix resolv.conf is empty , cannot get dns

Proposed Changes

Types of Changes

Verification

Testing

Linked Issues

User-Facing Change

fix resolv.conf is empty , coredns cannot get dns

Further Comments

Can you describe the problem you're trying to solve here? It looks like you're running into problems running k3s on hosts without an upstream DNS server configured? Is using the <IP_ADDRESS> default upstream a functional choice in this environment?

Closing due to inactivity


New version of up-enhsp supporting the new optimisations problem kinds plan_length and action_costs

…ptimisation problem kind

Codecov Report

Merging #203 (6c42eb2) into master (8517250) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #203      +/-   ##
==========================================
- Coverage   91.78%   91.74%   -0.05%
==========================================
  Files         122      122
  Lines       14499    14499
==========================================
- Hits        13308    13302       -6
- Misses       1191     1197       +6

Impacted Files
Coverage Δ

unified_planning/environment.py
94.82% <0.00%> (-5.18%)
:arrow_down:

unified_planning/engines/factory.py
81.68% <0.00%> (-1.10%)
:arrow_down:

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 8517250...6c42eb2. Read the comment docs.


Add EqualConcentration instead of confusing FixedVolume

Every person who has tried to understand it seems to have found the equal_conc parameter to FixedVolume confusing.  So this splits the action into two:

FixedVolume does exactly what it says.  It transfers a fixed volume of each component.  It doesn't care about concentrations, or anything else.
EqualConcentration transfers an equal (but not fixed/set) concentration of each component.  equal_conc is now method.  The class can just check that transfer concentrations will be equal ("check"), can have a set minimum volume ("min_volume"), can have a set maximum volume ("max_volume"), etc.

This is a breaking change that will break quite a bit of current code.
This is no longer a major breaking change, as there is code to allow the old behavaviour, mostly with a DeprecationWarning.  However, note that now by default FixedVolume does not check that concentrations are equal, so this may break some code.

Codecov Report

Merging #9 (8f01aa6) into main (776b3c7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #9   +/-   ##
=======================================
  Coverage   99.60%   99.60%
=======================================
  Files           3        3
  Lines         250      250
=======================================
  Hits          249      249
  Misses          1        1

Impacted Files
Coverage Δ

tests/test_mixes.py
99.50% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Note this also fixes some automatic code coverage calculation bugs, which in turn causes the coverage change to look horrible.  The previous high coverage was wrong.


relation "revisions" does not exist

I'm getting relation "revisions" does not exist with version 2.6.1
'INSERT INTO "revisions" ("id","model","document","operation","document_id","revision","created_at","updated_at") VALUES (DEFAULT,\'User\',\'{"last_email_validation_at":"2017-07-11T20:27:30.778Z","should_update_password":false,"login_attemps":0,"terms_n_conditions_accepted_at":"2018-03-28T14:38:59.976Z","email":"someone1@avi-on.com","password":"483483a29358524790249a3d219c4377","enabled":true,"first_name":"Count Dooku","last_name":"Evildoer","marketing_emails":true,"app_version":null,"last_sign_in_at":null,"phone_number":null,"latest_version":"0.0.0","reset_password_token":null,"reset_password_expires":null,"confirmation_token":null,"confirmed_at":null,"last_phone_verification_at":"2017-07-11T20:27:33.572Z","opt_in":false,"sms_authentication":true,"source_app":null,"disabled_at":null,"modified_at":null}\',\'create\',1,1,\'2019-05-02 18:39:04.060 +00:00\',\'2019-05-02 18:39:04.060 +00:00\') RETURNING *;' },
  original:
   { Error: ERROR:  relation "revisions" does not exist```

Can you try it with version 3.0.0-rc.6 if you're using Sequelize v4 or 3.0.0-rc.7 when using Sequelize v5?


Pester Task could contain the current release of Pester - 4.3.1

VSTS Extension you are using
Pester
Where are you running it?
VSTS
Version of Extension/Task
Current
Expected behaviour and actual behaviour
The included versions of Pester within this task are currently 3.4.3 and 4.0.8. The most recent version of Pester is 4.3.1, which contains a few fixes.
We could update the included versions to be 3.4.3 and 4.3.1.
Steps to reproduce the problem
Load extension, notice that the two versions available are 3.4.3 and 4.0.8.

This has now been shipped as 7.0.1


When using native modules, the app only works on the computer it was packaged

I'm having problems loading native modules (robotjs). The error is this:

It's trying to load "robotjs.node" from the path "/Users/RUBI35900/..etc, which is from the computer I package the app (RUBI35900 is my development machine). And this error throws on a different machine. So it seems that webpack stores the full path instead of using a relative one, so it only works on the machine that was used for packaging the app.
I forked the repo to reproduce the issue:
https://github.com/pancho111203/electron-react-boilerplate
To reproduce:

clone the repo
run 'yarn'
run 'yarn package'
move the created release file to another computer
install and execute in that computer

The error should appear

Found issue: https://github.com/webpack-contrib/node-loader/issues/12

@amilajack I don't know why you close this, considering it stills affects the repo.


CRITICAL: [Marketing] [$250] Create training interstitial when you click Track expense

Problem:
We are about to promote Track expense to 10MM+ existing users, most of whom have never used NewDot.  If we are lucky, they will see our marketing message and click through to check it out.  It's unlikely they have an actual expense to track at that moment, so it's more likely they are going to just sorta click around to see what's available.  Some unknown number of users will intuitively figure out our UI.  However, some number of users won't find it intuitive by itself, and will abandon the product.
Note: We are working on Stage 2 onboarding.  However, it doesn't exist today.  Additionally, even if it did, it only targets new signups -- not historical signups.  Furthermore, it only educates a single use case.
Solution:
Show "inline/on-demand training" the first time you use the Track expense, to:

Explain what this feature does from a functional basis
Preview how it works, to make it less scary and encourage them to give it a shot
Highlight the major use cases they should be using it for

It would work like this:
<gif combo roll>

[ ] Don't show me this again
(   Tell me more!    )
(     Got it!        )

The components are:

A modal interstitial that shows when you tap it
A looping Lottie animation walking through all the animations we've already made
A checkbox to cause this to be hidden in the future (which sets dismissedTrackTraining=1 in Onyx locally)
A button that links to HelpDot for Track
A button to continue

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01978ee6c0535caae9
Upwork Job ID:<PHONE_NUMBER>608776704
Last Price Increase: 2024-04-12
Automatic offers:

tienifr | Contributor | 0

🎀 👀 🎀
Let's assign @tienifr to implement this feature

@yuwenmemon is OOO until 16th, so I've asked someone else to assign themselves to this urgent issue

Taking this over @yuwenmemon

@tienifr's proposal looks good to me as well. Assigning

Hey @tienifr, for this point in your proposal:

and potentially call an NVP endpoint to mark this in the back-end

I created an NVP called HAS_SEEN_TRACK_TRAINING = "hasSeenTrackTraining" in the backend that you can set using SET_NAME_VALUE_PAIR in the frontend. Similar to how it is done here.

cc @dubielzyk-expensify since I believe you were working on the video.

I'll add myself here so I can help with PRs and I'll upload the video when it's done. It's posted to TCW here

Thanks Jon!

Hey @tienifr, do you think you could give an ETA for when the PR will be ready for review? Thanks!

Hey @tienifr! Friendly bump on the above :)

Update that TCW came back with videos. Only minor feedback so should get final files early next week :)

Hey @tienifr! We're trying to get the PR out by end of day Monday or early Tuesday. If you could mention me whenever you push the PR, I can help with an initial review along with @rushatgabhane :)

We should show this training every single time unless Don't show me this again is checked. Which means that we should only set the HAS_SEEN_TRACK_TRAINING when the checkbox is checked and Got it is pressed.

I also need this because I couldn't find it in HelpDot. Is it this one?

Hm, that looks like it's only for mileage. I'll check and get back to you on that but you can use that link for the initial PR for now.

Here's the video:
onboarding-track-video.mp4.zip

Hey! Yes, I believe so. I'm not exactly sure either how to do this but I checked internally. I'll upload the asset and give you the url.

Hi, the video is ready and available.
You can use the path ${CLOUDFRONT_URL}/videos/guided-setup-track-business.mp4 from the code.

Hi @rushatgabhane and @tienifr, this PR introduced this bug which we fixed with a 300ms timeout. I'm still not sure if this is the best way to do this so I wanted to ask if you guys had any other ideas.

@srikarparsi please attach payment summary for this issue 🙇

Hey @rushatgabhane! Since this was a pretty big PR and it only caused one small regression, we can pay $250 for you and tienifr each without the deduction. It would be great though if you could take a look at this comment and check if there’s a better approach for fixing this regression. More information about the fix is here and we’re essentially looking for a good way to queue the next animation once the previous animation is complete.

Waiting for BZ checklist to be complete before approving payment, based on @srikarparsi's comment above.

The PR that introduced the bug has been identified. Link to the PR:

The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

Determine if we should create a regression test for this bug.

If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again

I'm still not sure if this is the best way to do this so I wanted to ask if you guys had any other ideas.
@srikarparsi no, i don't know a better way yet. I spent time on this yesterday, but couldn't find much

Ok sounds good, @tienifr did you have a chance to look? The current solution is okay for me since we do the same thing in other places but just want to check. If not, I believe we can proceed with payment @muttmuure since the regression steps look good to me.

Cool sounds good! I think this solution should work fine as well then. @muttmuure do you think you could help with payment or is there anything else that needs to be done? The regression steps here look good to me.

Still waiting for a payment summary so I can approve.

Any updates here?

I flagged with Matt Moore via DM but he's OOO until Monday

I don't see the High Priority label on this issue, right?

In any case, @muttmuure I need a summary on this issue when you're back from OOO.

Going to give the benefit of the doubt that this is at least a HIGH issue, since it has CRITICAL status in the project
@rushatgabhane - C+ $500
@tienifr - C $500

$500 approved for @rushatgabhane


OpenCover returns with no Coverage

I am running the following in my cake build script.
if(IsRunningOnWindows())
{
	Action<ICakeContext> testAction = tool => {
		tool.DotNetCoreTest(project.GetDirectory().FullPath, new DotNetCoreTestSettings {
			Configuration = configuration,
			NoBuild = true,
			Verbose = false,
			ArgumentCustomization = args =>
				args.Append("-xml").Append("./build/" + project.GetFilenameWithoutExtension() + ".xml")
		});
	};

	OpenCover(testAction,
		"./build/OpenCover.xml",
		new OpenCoverSettings ()
			.WithFilter("+[*]* -[xunit.*]* -[*.Tests]* ")
			.ExcludeByFile("*/*Designer.cs;*/*.g.cs;*/*.g.i.cs"));
}

Currently only have 5 active Unit Tests that will be run. But OpenCover says:
========================================
Run-Tests
========================================
Executing task: Run-Tests
Executing: C:\Program Files\dotnet\dotnet.exe
xUnit.net .NET CLI test runner (64-bit .NET Core win10-x64)
  Discovering: NGitLab.Tests
  Discovered:  NGitLab.Tests
  Starting:    NGitLab.Tests
  Finished:    NGitLab.Tests
=== TEST EXECUTION SUMMARY ===
   NGitLab.Tests  Total: 5, Errors: 0, Failed: 0, Skipped: 0, Time: 0.172s
SUMMARY: Total: 1 targets, Passed: 1, Failed: 0.
Committing...
Visited Classes 0 of 59 (0)
Visited Methods 0 of 135 (0)
Visited Points 0 of 281 (0)
Visited Branches 0 of 175 (0)

==== Alternative Results (includes all methods including those without corresponding source) ====
Alternative Visited Classes 0 of 94 (0)
Alternative Visited Methods 0 of 245 (0)

I have no idea what the problem is. I checked the XML that was created. The modules are not skipped.
Any idea what the issue could be?

Trick is to add the OldStyle = true flag. Now the coverage report gets created 👍
if(IsRunningOnWindows())
{
	Action<ICakeContext> testAction = tool => {
		tool.DotNetCoreTest(project.GetDirectory().FullPath, new DotNetCoreTestSettings {
			Configuration = configuration,
			NoBuild = true,
			Verbose = false,
			ArgumentCustomization = args =>
				args.Append("-xml").Append("./build/" + project.GetFilenameWithoutExtension() + ".xml")
		});
	};

	OpenCover(testAction,
		"./build/OpenCover.xml",
		new OpenCoverSettings {
			OldStyle = true
		}
			.WithFilter("+[*]* -[xunit.*]* -[*.Tests]* ")
			.ExcludeByFile("*/*Designer.cs;*/*.g.cs;*/*.g.i.cs"));
}


[DNM]coprocessor/decimal: add error info to debug decimal encode

This PR is used to debug, Please do not merge it

/release

Can this PR being closed?


修改客户端背景色

为何会overwrite我的背景色...

It's a feature

On the one hand I am really pleased with your exellent answer.  On the other hand, you just like Microsoft who tells me darkness is one part of the standard...

There are restricted predefined colors such as RED, BLUE,  YELLOW, BLACK, CYAN etc in curses.COLOR_*, the are the same as colors in your ~/.dir_colors or /etc/DIR_COLORS.
Just like you cannot change your dir color freely in shell, our choices of color are limited due to the curses library.

I am pretty sorry what i just said to you. But I have already changed the terminal profile and the background never changed. And you should know there are lots of terminal that support 256 colors.
That means the terminal can display more than 8 colors(without any restriction)

Definitely terminal can display lots of colors such as text or background colors, however, you cannot change color of your directories as you want.
Try google "terminal directory color"

Thanks for your patiently answering ... Time will really solve my problem..


solve_Lt() does not work right

When I use the solve_Lt() to calculate L'x=b, I use the x returned to do L'x, and this does not equal to b, also when I use the dense matrix to do this calculation, the x I got from cholmod is different from the dense version. How can I get the right x?

You must use solve_Lt(b, use_LDLt_decomposition=False) and at least version 0.4.3 of scikit-sparse.

After I put the parameter use_LDLt_decomposition, I got a result which is of the permutation of the original A, how can I get a result without permutating the original matrix? For example, how can I get a result same as what I get from dense version Cholesky decomposition?

The ordering_method parameter in the decompose functions allows you to disable permutation using the value natural. For example cholesky(A, ordering_method="natural").

Got it! It works! Thank you so much for your quick reply!