Get rid of goog.isArrayLike

With this change, it's no longer possible to extend an array with
an "array like" object (an object with a 'length' property).
We were not using this feature in the library.

Very nice, please merge!


Invalid Field Status - UI

Affected module
Does it impact the UI, backend or Ingestion Framework?
UI
Describe the bug
A clear and concise description of what the bug is.
Invalid field name status
To Reproduce
Go into Service Details page

Ingesting Sample Data
http://localhost:8585/service/databaseServices/bigquery_gcp

Screenshots or steps to reproduce

Hi, Kindly assign this to me!

Block by #3581
Even after removing status field from API call, we are not getting status from API for more details please refer above issue.


ImportError: Traceback (most recent call last) while importing the tensorflow

import tensorflow as
I am getting following error error. Importing at jupyter notebook. Python version is 3.8.3.

ImportError                               Traceback (most recent call last)
~\anaconda3\lib\site-packages\tensorflow\python\pywrap_tensorflow.py in
63   try:
---> 64     from tensorflow.python._pywrap_tensorflow_internal import *
65   # This try catch logic is because there is no bazel equivalent for py_extension.
ImportError: DLL load failed while importing _pywrap_tensorflow_internal: The specified module could not be found.
During handling of the above exception, another exception occurred:
ImportError                               Traceback (most recent call last)
 in
----> 1 import tensorflow as tf
~\anaconda3\lib\site-packages\tensorflow_init_.py in
39 import sys as _sys
40
---> 41 from tensorflow.python.tools import module_util as _module_util
42 from tensorflow.python.util.lazy_loader import LazyLoader as _LazyLoader
43
~\anaconda3\lib\site-packages\tensorflow\python_init_.py in
38 # pylint: disable=wildcard-import,g-bad-import-order,g-import-not-at-top
39
---> 40 from tensorflow.python.eager import context
41
42 # pylint: enable=wildcard-import
~\anaconda3\lib\site-packages\tensorflow\python\eager\context.py in
33 from tensorflow.core.protobuf import config_pb2
34 from tensorflow.core.protobuf import rewriter_config_pb2
---> 35 from tensorflow.python import pywrap_tfe
36 from tensorflow.python import tf2
37 from tensorflow.python.client import pywrap_tf_session
~\anaconda3\lib\site-packages\tensorflow\python\pywrap_tfe.py in
26
27 # pylint: disable=invalid-import-order,g-bad-import-order, wildcard-import, unused-import
---> 28 from tensorflow.python import pywrap_tensorflow
29 from tensorflow.python._pywrap_tfe import *
~\anaconda3\lib\site-packages\tensorflow\python\pywrap_tensorflow.py in
81 for some common reasons and solutions.  Include the entire stack trace
82 above this error message when asking for help.""" % traceback.format_exc()
---> 83   raise ImportError(msg)
84
85 # pylint: enable=wildcard-import,g-import-not-at-top,unused-import,line-too-long
ImportError: Traceback (most recent call last):
File "C:\Users\ssahu\anaconda3\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 64, in
from tensorflow.python._pywrap_tensorflow_internal import *
ImportError: DLL load failed while importing _pywrap_tensorflow_internal: The specified module could not be found.
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

@ssahu2498,
Installation issues within the Anaconda environment are tracked in the Anaconda repo.
Could you please submit a new issue using this link and fill in the template, so that the issue can be tracked there. Thanks!

I come cross the same problem

I come cross the same problem

@chaizhongming,
Could you please submit a new issue from this link and fill in the template, so that we can track the issue there. Thanks!

Having the same issue currently, it was working yesterday with no issues and today it started giving errors even though I changed nothing. Why's this happening?

@Khezkid,
Could you please submit a new issue from this link and fill in the template, so that we can track the issue there. Thanks!

ON "ANACONDA.NAVIGATOR" MAIN PAGE CHANGE THE setting of  "APPLICATION ON TENSORFLOW OR TF"


socket.io - JS and CPP Server/Client Cooperation

This may be a dumb question, but would it be possible to have socket.io JS running as a server, and socket.io CPP as the client, and they work together?

Yep. This is the objective of the socket.io-client-cpp :)


Use custom github deploy feature in ICT to test BluePic.

Thanks to @tfrank64 for testing this out. Here's the outcome of using the custom github deploy feature in ICT on BluePic:

Though the different services were created, they were not bound to the BluePic app.
The populator scripts were not executed.

Hence, it seems the behavior that existed before in ICT when provisioning BluePic did not execute when using  the custom github deploy feature.

ICT has now added the necessary logic to have BluePic as a featured app. Closing this issue.


Documentation updated

Signed-off-by: Kestutis Gudynas<EMAIL_ADDRESS>

@denyeart are you okay with this as the 2.x change?

Beyond the signature issue that Brett mentioned, would be nice to have a descriptive commit/PR title and message.
After we merge this in main, we'll also backport to release-2.3 and release-2.2.

@Mergifyio backport release-2.3

@Mergifyio backport release-2.2


refactor: moving API response to constants

creates TARBALL_UPLOADED & BAD_PACKAGE_DATA constant

Type:
The following has been addressed in the PR:

There is a related issue?
Unit or Functional tests are included in the PR

Description:

@ashishsurana do you want to help me to improve the functional testing system? I can help you along the process.

@ashishsurana do you want to help me to improve the functional testing system? I can help you along the process.

Yes, I would love to do. Please tell me how I can I contribute


Fix broken link in CONTRIBUTING.md

What does this PR do?
Fix a broken link.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


fix: default missing values to zero on area chart

CATEGORY
Choose one

[x] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
Area chart currently fails to render properly if there are missing groupby values in the time series. This PR makes it possible to specify NULL handling behavior per NVD3 chart, and defaults values to zero for the Area chart. Note: other charts are left unchanged, as defaulting to zero is most likely not expected behavior for all charts (at least line charts).
BEFORE

AFTER

TEST PLAN
Visual inspection.
ADDITIONAL INFORMATION

[x] Has associated issue: closes #8671
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
@etr2460

Codecov Report

Merging #8678 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #8678      +/-   ##
==========================================
+ Coverage   65.77%   65.77%   +<.01%
==========================================
  Files         482      482
  Lines       23834    23836       +2
  Branches     2594     2594
==========================================
+ Hits        15676    15678       +2
  Misses       7985     7985
  Partials      173      173

Impacted Files
Coverage Δ

superset/viz.py
71.33% <100%> (+0.03%)
:arrow_up:

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 75c1378...4b43395. Read the comment docs.


Check if layout name is nil before attempting to perform actions

I had an issue where it would sometimes cause a crash on page load. This was the fix.

Thanks for this. I will merge this in and work on a release soon.


optimize (is horribly slow)

Due to the way cleanco currently works, quite intensive operations are taking place every time a name is cleaned (a class is instantiated every time a name is cleaned; see what happens in __init__).
This should be optimized so that the operations only take place once.

Implemented in release 1.2 pushed to PyPI


<EMAIL_ADDRESS>Update persist

Total Commander: update persistent files.

plugins: default installation directory for plugins (e.g. if you open plugin archive, totalcmd by default installs it in this directory)
VERTICAL.BAR, vertical.br2: vertical (between panels) button bar
contplug.ini, fsplugin.ini, lsplugin.ini, pkplugin.ini: default config files for plugins settings (https://www.ghisler.ch/wiki/index.php/Ini-settings)

Unfortunately it's not the all possible files. Various plugins can use their own config files. For example SFTP plugin from Total Commander developer is using sftpplug.ini, his WebDAV plugin uses tcwebdav.ini and so on. It's the same problem as in foobar2000: #2884. So it could be better to split it to normal and portable version. But this PR should cover most cases.

Toolbars will not work with precreation. They will be created empty and then total commander will make all toolbars empty instead of defaults.

Are you sure? It works in current version of manifest for DEFAULT.BAR. And it works for VERTICAL.BAR also based on my testing.

In case of clean installation without any persisted data?
I remember one of the file managers is having these issues. But it could be unreal or doublecmd or similar

Yes. I've tested a clean installation. Then I changed some settings, buttons, installed some plugins. Then uninstall, install. It worked for me.

It works because DEFAULT.BAR and VERTICAL.BAR are already in an installer archive. An install script doesn't create those files (New-Item silently fails). So they are either defaults (from installer) or are copied from persist directory by scoop. default.br2 and vertical.br2 are created empty, but total commander overrides them with something after first startup.
If I empty those files manually after installation, then indeed a toolbar would be empty. But it shouldn't happen (files are already there; if not, then toolbar would be empty anyway).

Probably default.br2 and vertical.br2 aren't needed at all. As I understand it's just an icon cache. It should be regenerated if not exist. 1, 2


Fixed Cancel Generation Issue

#952 Solved
Now the website build is canceled on pressing the cancel button. There also should not be any problem with multiple users.

Codecov Report

Merging #1031 into development will decrease coverage by -1.26%.
The diff coverage is 1.95%.

@@               Coverage Diff               @@
##           development    #1031      +/-   ##
===============================================
- Coverage        32.28%   31.02%   -1.26%
===============================================
  Files                7        7
  Lines             1112     1170      +58
  Branches           172      196      +24
===============================================
+ Hits               359      363       +4
- Misses             753      807      +54

Impacted Files
Coverage Δ

src/app.js
44.28% <ø> (-1.31%)
:x:

src/backend/generator.js
13.66% <1.97%> (-1.5%)
:x:

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 9d62043...2de1325. Read the comment docs.

Thanks @SparshBansal But can you please mention it is referred too.

@aayusharora  , I am sorry , i didn't get what you meant.

@SparshBansal Please refer the Issue number in the description of this PR.

Issue #952 Solved

Ok Cool Thanks

@SparshBansal Can you please squash the commits.

@SparshBansal Hey. Thanks for the request. To make reviewing easier, could you please host the test application on Heroku, please?

@aayusharora , I have squashed the commits and will send a new PR

@Princu7 , I will host it on heroku for review

@Princu7 I have deployed it on heroku and here is the link : https://safe-sea-55983.herokuapp.com/

Nice Work @SparshBansal

@SparshBansal Before sending a PR please let us know that you are working on the issue.
I checked the demo link, the generation does not stop immediately if X is clicked.
You can use a variable in socket object to keep the status.
Please look into it.

@SparshBansal There are bugs in this PR. @geekyd Thanks for reporting.
Reverting this :(

@aayusharora
@geekyd
It doesn't stop immediately because when the user issues a cancel request it is send to the back end server and if the server has started some asynchronous operation in the asynchronous series then I can't cancel it in between. I cancel it between 2 asynchronous steps.
That's why it doesn't immediately cancel the build process.
Also I notify the frontend client of cancellation only when it has actually cancelled it in the backend. That is between 2 asynchronous steps.
I don't know how to cancel an asynchronous operation in between. Could you please guide me on that.

@SparshBansal We can hide the progress bar and the status bar as soon as the user hits cancel, and the rest will be handled by the backend.

Yeah we can do that

@geekyd I think we also disable the upload zip element till the build gets cancelled in the backend and we receive an acknowledgement at the client side

@geekyd  Also I have noticed a bug while running the dev version hosted on heroku , If I cancelled uploading in the middle and retried again , it started from the same point it left off thus suggesting that the upload was not cancelled.

@SparshBansal The zip file upload is definitely canceled. It might be starting from middle maybe because of caching issue :)
To be more precise, it is because of this line
uploader.resetFileInputs = false
If you set it to true, it should be starting from beginning.
Will check the test server soon. Net speed a bit slow now :smiley:


How to use in .Net MAUI Blazor (Blazor Hybrid)

Would be great help if some one can share code how to use in .net maui blazor app(blazor hybrid).
Will Zxing.Net.Maui will support all previous version of android like form 7 onward to 12? or only some specific version.

I solved the problem Guys.
Blazor Code
<button class="btn btn-warning" @onclick="Openwindow">Scan Now
@code{
async Task Openwindow()
{

    App.Current.MainPage.Navigation.PushModalAsync(new NewPage1());
}

}
Add a new MAUI Xaml Page

<zxing:CameraBarcodeReaderView x:Name="barcodeReader"
WidthRequest="300" HeightRequest="300"
IsDetecting="True" IsTorchOn="False" BarcodesDetected="barcodeReader_BarcodesDetected_1" />
    <Label
            x:Name="barcodeResult"
            Text="Hello, World!"
            SemanticProperties.HeadingLevel="Level1"
            FontSize="32"
            HorizontalOptions="Center" />
    <Button Text="🔄" Grid.Column="0" BackgroundColor="#aa000000" CornerRadius="8" BorderColor="Black" Clicked="Button_Clicked" />

</VerticalStackLayout>

public NewPage1()
{
InitializeComponent();
barcodeReader.Options = new BarcodeReaderOptions
{
Formats = BarcodeFormats.OneDimensional,
AutoRotate = true,
Multiple = true
};
}
private void barcodeReader_BarcodesDetected_1(object sender, ZXing.Net.Maui.BarcodeDetectionEventArgs e)
{
    Dispatcher.Dispatch(() =>
    {
        barcodeResult.Text = e.Results[0].Value.ToString();// $"{e.Results[0].Value}";
        Globaclass.BarcodeID = barcodeResult.Text;
        Navigation.PopAsync();
    });
}

private void Button_Clicked(object sender, EventArgs e)
{
    barcodeReader.CameraLocation = barcodeReader.CameraLocation == CameraLocation.Rear ? CameraLocation.Front : CameraLocation.Rear;
}

and problem solved

@rajeshsws: I love to do your workaround and implement it. But I can't follow your Code Example. What goes Where? I would be delighted if you could take the time help clear it up.

Add in Pages/index.razor:  <button class="btn btn-warning" @OnClick="Openwindow">Scan Now</button>  ...
...

Thx you for your time.

@ragininternet I've uploaded a quick test app here

The navigation back ( Navigation.PopAsync(); ) in Xaml page doesn't work, it never comes back to orignal page (blazorview), any idea ? The same with the test app here.

Gerald Versluis has a video doing exactly what you guys are asking:
https://www.youtube.com/watch?v=2dllz4NZC0I&t=6s
Believe me, i was also struggling for quite a while.
I've implemented his solution in a .net MAUI Blazor mobile application and it works perfectly.

Let me give my two cents: https://pavlodatsiuk.hashnode.dev/implementing-maui-blazor-with-zxing-qr-barcode-scanner


voluntarily yield?

I'm quite new to Rust so I could be wrong, but is there some voluntarily yielding I can do? That is, the task is still runnable, but put at the back of the runnable queue.

RTFM tasks have "run to completion" semantics -- they are not time-sliced threads like you normally see in OSes like Linux. So, no; you can't yield, or suspend, a task. You can only return from it and have something else (an interrupt or another task) invoke it (spawn / schedule) again -- tasks are event driven after all.
That being said you can implement cooperative multitasking which is closer to "straight-line threaded code" in idle (which is not a task and runs forever / never returns) using generators, futures, or what have you. Any such implementation needs no additional support from RTFM and there are many possible implementations with different trade offs so we leave end users go at it in whichever way they want (i.e. we provide no particular implementation in this crate).

Thanks @japaric for the explanation.
I found this: https://github.com/japaric/no-std-async-experiments
It looks very promising - may I ask when there will be a release?

@riaqn I have no plans (time) to work on that any time soon.
If you have no more questions related to RTFM could you please close this issue?

Thanks. I'm closing the  issue.

Did this change at some point?  I have two tasks that look like
  #[task(priority = 1)]
  async fn task1(_cx: task1::Context) {
      defmt::info!("Hello from task1!");

      loop {
          defmt::info!("1");
          Mono::delay(500_u64.millis()).await;
      }
  }

  #[task(priority = 1)]
  async fn task2(_cx: task2::Context) {
      defmt::info!("Hello from task2!");

      loop {
          defmt::info!("2");
          Mono::delay(1000_u64.millis()).await;
      }
  }

Despite never returning, they interleave as I'd expect, which suggests that they can yield execution to other tasks and then resume.  Which brings me to the question I was initially looking for - can you yield execution without specifying e.g. a delay or other condition, just "let other tasks run, then come back"?
(More confusingly, a loop I forgot in idle that only iterated a variable and occasionally printed also kept running, and I haven't the foggiest how it'd do that without either using multiple cores or proper threads.  Only idle seems to be capable of arbitrary interruption, though; other tasks just hog execution.)


Monaco editor encodes URL when doing Ctrl + click to follow link

See this code pen link https://codepen.io/nikhilexe/pen/RYwveN
If you ctrl + click on the URL, browser will open with this URL https://abc.com/id//def/ghi basically it encodes %2F to /
If you paste this URL directly into the browser https://abc.com/id/%2Fdef%2Fghi nothing changes.

+1
We are seeing a similar issue.

duplicate of https://github.com/Microsoft/vscode/issues/25852


Headless rendering on Mac?

I'm currently trying to run something on a headless Mac server that is hanging due to a call to Physics.render()
Relevant code excerpt:
from dm_control.suite import base

class MyTask(base.Task):

  ...

  def get_observation(self, physics):
    image = physics.render(...)
    ...

my_task.get_observation(self, physics)

I face the same issue if I try to follow the 'getting started' steps outlined here: https://github.com/google-deepmind/dm_control/tree/main/dm_control/mujoco
From a few existing issues and the description on the main readme.md, I can't quite tell if headless rendering on a MacOS system is supported or not - it seems like it might only be supported for Nvidia systems. If so, are there any known workarounds?

@kevinzakka you render on Mac, right? Any thoughts?

In theory the CGL code path allows an OpenGL context to be created without a window. In practice the call to create a CGL context hangs until a graphical login session for the current user is created. I don't believe this is actually documented anywhere (and the entire CGL API is marked deprecated by Apple several years ago).
I've spent several days in the past trying to figure out how to open a graphical login session via SSH but have come to a conclusion that this isn't possible (or at least not through any documented means).
For our CI server the "workaround" is to VNC into the login screen and log in to the CI user after each reboot. In other words, it doesn't work headlessly.


Unresolved dependencies (sbt version 0.13.13)

steps

Installed sbt <IP_ADDRESS> (windows installer)
Executed sbt run on project
SBT_HOME is set correctly, sbt sbtVersion returns <IP_ADDRESS>

problem
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
[warn]  ::          UNRESOLVED DEPENDENCIES         ::
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
[warn]  :: com.typesafe.play#sbt-plugin;2.5.10: not found
[warn]  :: com.typesafe.sbt#sbt-digest;1.1.1: not found
[warn]  ::::::::::::::::::::::::::::::::::::::::::::::
Installation previously was working, only when new developer joined and tried fresh installation it failed (two month ago issue was not present, fresh installation was working).
sbtError.txt
expectation
notes
sbt version: <IP_ADDRESS>

Your project is using sbt 1. You have to upgrade to Play 2.6.6 and sbt-digest 1.1.4.


Joint wavelet regularization not supported in pics

Currently not implemented. Requires using prox_unithresh_create along with a wavelet linop

See my attemp for wavelet3 in joint-wavelet2 ... completely untested but I think it should work...

Is your wavelet3 competitive in speed with wavelet2 on CPU and GPU?
If not I still prefer to have the special case of wavelet2 + unitary
threshold for 3D, since I use it in a clinical setting. If it is then we
should get rid of wavelet2 completely.
Jon
On Wed, Aug 10, 2016 at 6:24 AM, Martin Uecker<EMAIL_ADDRESS>wrote:

See my attemp for wavelet3 in joint-wavelet2 ... completely untested but I
think it should work...
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/mrirecon/bart/issues/54#issuecomment-238865135, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIUBa7nHyPgcB7h4Rr_cBCc31zFcZz3Kks5qedEDgaJpZM4JKs-u
.

No, for the CPU it depends on circumstances and the GPU case is slow. Otherwise I would have removed the wavelet2 code a long time ago...

Actually, I did some tests and it seems fast even on the GPU. Maybe because newer ones have a cache and do not rely as much on the explicit use of the shared memory. Are you OK with removing wavelet2?

I will check when I have a chance. @frank thoughts?

I'm ok if it's faster. Wavelet2 has served its time

On Aug 12, 2016, at 5:35 AM, Jon Tamir<EMAIL_ADDRESS>wrote:
I will check when I have a chance. @frank thoughts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

What's the latest on this? I rebased the wavelet branch. Do we have any performance regression tests?

Not sure what branch you are talking about...
the old wavelets are not used by default anymore. My plan is to make another release soon, and if nobody complaints too loudly, remove wavelet2 in the release after this.
For joint thresholding, somebody needs to test whether joint wavelet thresholding in the joint-wavelets2 branch works as intended and contribute a test. Then we could merge that feature.

Okay, I will take a look. That's the branch I'm talking about

Implemented on master. Still needs testing


refactor ppa improve internal documentation

Codecov Report

Merging #296 (2a84538) into main (35e979f) will increase coverage by 0.16%.
The diff coverage is 4.34%.

@@            Coverage Diff             @@
##             main     #296      +/-   ##
==========================================
+ Coverage   80.94%   81.11%   +0.16%
==========================================
  Files          34       34
  Lines        4041     4034       -7
==========================================
+ Hits         3271     3272       +1
+ Misses        770      762       -8

Files
Coverage Δ

preliz/predictive/ppa.py
12.50% <4.34%> (+0.94%)
:arrow_up:


Update_wrapper

Please merge this PR and open a new PR against Bjorn repo in order to include that tool @DenisGrim

I opened a PR on the main repo https://github.com/bgruening/galaxytools/pull/1289


fix(ui yaml): ui run as nginx user

What this PR does / why we need it:
ui container run as nginx user
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Codecov Report

Merging #1081 (dbe3f8a) into main (2aa97a9) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main    #1081   +/-   ##
=======================================
  Coverage   40.80%   40.80%
=======================================
  Files          24       24
  Lines        1642     1642
=======================================
  Hits          670      670
  Misses        900      900
  Partials       72       72

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

LGTM


where can i backend

awesome job ,  where can i backend plz?

hey @minbokang  this is the backend repo of this project https://github.com/themohitbharti/Hive_backend


Update kernel shap

Fine-tune kernel shap to explain binary mnist model.

Minor changes to the notebook. This PR will be merged directly.


Apple TV component stop to work after upgrade to 0.55.1

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.55.1
Python release (python3 --version):
Python 3.4.2
Component/platform:
Apple TV
Description of problem:
After upgrade from 0.44 to 0.45.1 the Apple TV component stop to work.
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):

Additional info:
2017-10-20 19:48:12 ERROR (Thread-12) [homeassistant.util.package] Unable to install package pyatv==0.3.5: Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-9whdcb_n/cryptography/
2017-10-20 19:48:12 ERROR (MainThread) [homeassistant.setup] Not initializing apple_tv because could not install dependency pyatv==0.3.5
2017-10-20 19:48:12 ERROR (MainThread) [homeassistant.setup] Setup failed for apple_tv: Could not install all requirements.

You need to have the development packages specified in the documentation, otherwise cryptography won't compile: https://home-assistant.io/components/apple_tv/


do not record user agent

it's tested in tests/test_whispyr_client_basic.py (test_user_agent_is_set)
also it will make new diff of new releases easier to read

Pull Request Test Coverage Report for Build 37

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

Totals

Change from base Build 35:
0.0%

Covered Lines:
254

Relevant Lines:
262

💛  - Coveralls


Wrong popper width with overflow: auto

CodeSandbox demo

https://codesandbox.io/s/pedantic-dijkstra-nl3xi
Steps to reproduce the problem

Create a long list in the popper and set max-height to it
Set popper style "overflow-y: auto"
In Windows/Chrome87 resizing viewport makes the popper width count correctly. In Firefox remains unchanged.

What is the expected behavior?
As I know this is a browser problem, the best solution I expect is a consistent and predictable popper behavior. But not resizing popper in Chrome when resizing viewport will be fine as well. Or maybe you have an idea of how to prevent this behavior on the user side.

What went wrong?
The biggest problem is the wrong width in the modifiers.

Any other comments?

Can be fixed quickly visually for Chrome. The fix for chrome is simple by setting e.g. popper min-width. But it's not work in Firefox.
It's a known browser bug:
https://bugzilla.mozilla.org/show_bug.cgi?id=764076
it looks like the problem doesn't exist on macOS due to a different scrollbar
Windows/Chrome:

MacOS/Chrome:

On a closer look this doesn't seem to have anything to do with Popper, the position is correctly computed, it's just that the browser decides to give the popper a size rather than another. There's nothing we can do for this.


Feature Request: option to prevent navigating to the same destination

Suppose you have a simple button, when clicked it calls navController.navigate(destination).
Now, if you double-tap it or press it quickly multiple times, multiple navigate() calls will be invoked.
I'd love to prevent this kind of behavior, an option to prevent navigating to the same destination would be great.
Another option would be to expose some kind of property to let apps know navController is currently performing a navigation, so they could disable screen events during the transition.

@bugger15 I don't think the feature should be included into the core navigation library. It seems quite logical to me that multiple navigate calls execute multiple navigation transitions. Maybe you could eliminate the multiple navigate calls in the first place.
Usually, the problem of multiple or simultaneous clicks are resolved with debounce (basically, a reasonable delay between click handling). This way a button cannot be spammed with clicks.
Alternatively, you could always create a wrapper class around NavController and add your own custom logic for accepting/rejecting navigation actions.
As for some inTransition property - that is a great idea, I was thinking to add it myself. Right now it is possible to detect the transition through Lifecycle of a NavHostEntry. During the transition all entries are paused. This follows the behaviour of the official AndroidX Navigation Component library.

You are right, I realized that only once posted this. NavController should not handle this, this would be a possible responsibility for AnimatedNavHost. It would be nice to know when its animating, though. Feel free to close this issue.


Seed Ansible Roles for Vmdb Plugins

Allow plugins to add a requirements.yml file to specify roles from
galaxy/github/etc... and seed them locally.
If I have an example requirements.yml:
cat content_tmp/ansible/requirements.yml
- src: yatesr.timezone

And I seed the roles
bundle exec rake evm:ansible:seed
** override_gem: manageiq-providers-vmware, [{:path=>"/home/agrare/src/manageiq/manageiq-providers-vmware"}], caller: /home/agrare/src/manageiq/manageiq/bundler.d/plugins.rb
Seeding roles for ManageIQ::Providers::Vmware...
Seeding roles for ManageIQ::Providers::Vmware...Complete

ls -l ../manageiq-providers-vmware/content_tmp/ansible/roles/
total 0
drwxr-xr-x 1 agrare agrare  78 Jul 30 11:08 yatesr.timezone

cc @gtanzillo @Ladas

We should probably extract the path to ansible to core, if this script expects it in some location, right?
And rename the content_tmp :-)

Yeah I was trying to think of a good place to put this constant since eventually it'll be shared by both Ansible::Runner and EmbeddedAnsible but figured I'd put the PR in and we could discuss here :)

Okay @gtanzillo @Fryguy this is ready to review
We still need to figure out what we're doing with the content/ansible collision with playbook seeding.  Don't need to do that in this PR just pointing it out :smile:


[PORT] Fixes for QnA / Recognizer Adaptive telemetry

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/3715

Adds UseTelemetry extension method for DiaogManager which adds IBotTelemetryClient to turn state for use by adaptive recognizers.
Adaptive recognizers will now use the IBotTelemetryClient on the turn state unless the TelemetryClient property on the recognizer has been set.
Adds LogPersonalInformation property to QnAMaker dialogs and LUIS / QnA adaptive recognizers. Adaptive LUIS / QnA will look in settings.telemetry.logPersonalInformation by default.
Ensures QnAMaker client is always passed a TelemetryClient if set, along with the LogPersonalInformation flag (Fixes #3472)
LUIS Adaptive recognizer: Ensure telemetry properties passed in can override default properties

@cwhitten @vishwacsena Once these changes have been approved, I can raise an issue for composer to detail the changes needed in the runtime for telemetry.
Changed projects

Microsoft.Bot.Builder.AI.QnA
Microsoft.Bot.Builder.Dialogs.Adaptive
Microsoft.Bot.Builder.Dialogs

[adaptive]

This depends on #1855


'Module' Object is not callable

I am trying to train the model according to the instruction given in the readme but I am getting this error can you tell me whats wrong.

I need more information.

I need more information.

Brother How can I change the training and testing from cuda to cpu

Brother How can I change the training and testing from cuda to cpu

With this implementation, you can't. The NMS procedure is implemented only for CUDA.

With this implementation, you can't. The NMS procedure is implemented only for CUDA.

ok thanks for letting me know

ok thanks for letting me know

大佬您好 我也遇到了同样的问题
Traceback (most recent call last):
File "main.py", line 59, in
main()
File "main.py", line 52, in main
runner.train()
File "/mnt/zdm/LaneATT-main/lib/runner.py", line 51, in train
outputs = model(images, **self.cfg.get_train_parameters())
File "/mnt/obj_detect_env/env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/mnt/zdm/LaneATT-main/lib/models/laneatt.py", line 108, in forward
proposals_list = self.nms(reg_proposals, attention_matrix, nms_thres, nms_topk, conf_threshold)
File "/mnt/zdm/LaneATT-main/lib/models/laneatt.py", line 129, in nms
keep, num_to_keep, _ = nms(proposals, scores, overlap=nms_thres, top_k=nms_topk)
TypeError: 'module' object is not callable
显示的错误如上，请问您有啥解决方法吗~？谢谢啦

大佬您好 我也遇到了同样的问题
Traceback (most recent call last):
File "main.py", line 59, in
main()
File "main.py", line 52, in main
runner.train()
File "/mnt/zdm/LaneATT-main/lib/runner.py", line 51, in train
outputs = model(images, **self.cfg.get_train_parameters())
File "/mnt/obj_detect_env/env/lib/python3.6/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/mnt/zdm/LaneATT-main/lib/models/laneatt.py", line 108, in forward
proposals_list = self.nms(reg_proposals, attention_matrix, nms_thres, nms_topk, conf_threshold)
File "/mnt/zdm/LaneATT-main/lib/models/laneatt.py", line 129, in nms
keep, num_to_keep, _ = nms(proposals, scores, overlap=nms_thres, top_k=nms_topk)
TypeError: 'module' object is not callable
显示的错误如上，请问您有啥解决方法吗~？谢谢啦


Feature: install the CA into Firefox's store

https://github.com/FiloSottile/mkcert/blob/master/truststore_nss.go
https://support.mozilla.org/en-US/kb/setting-certificate-authorities-firefox

Firefox now automatically trusts certificates from system stores which makes the feature redundant. More at:
https://support.mozilla.org/en-US/kb/automatically-trust-third-party-certificates


Simple fix for "Apply changes to fixtures of same type" & dimmers. Fi…

…xes #457.

Uhmm...I'm not sure if this is what theater guys want.
If I read the code right, you are selecting every channel of every dimmer fixture.
I believe what they want is: I have 10 fixtures with 6 channels each. If I select channel 2, every channel 2 of the other 9 fixtures should be selected.

@janosvitok  After 2999f40690dc195c5c581b49917062522c33aa17 can we abandon this ?

Let's forget this :)


*: add Vincent Batts (vbatts) to MAINTAINERS

Per discussion on the appc-dev mailing list [1], and according to the current
governance policy, Vincent has been sucessfully nominated as a new maintainer
of the appc spec.
[1] https://groups.google.com/d/msg/appc-dev/AuAjfN6EB1w/kccuVV_-l3MJ

lgtm


#import data

iris.data file is missing
The path is incorrect and is not mandatory.
It worked for me this way with iris.data from other sources:
data=pd.read_csv('iris.data', names=['f1','f2','f3','f4','f5'], skiprows=1)

what error did you encounter?


Update isomorphic to 1.8.0 in commerce-sdk-react

Description

Upgrading the version of commerce-sdk-isomorphic we are utilizing in commerce-sdk-react to support the use of the channel_id parameter
Types of Changes

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Documentation update
[ ] Breaking change (could cause existing functionality to not work as expected)
[x] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

Removing a public function or component or prop
Adding a required argument to a function
Changing the data type of a function parameter or return value
Adding a new peer dependency to package.json

Changes

(change1)

How to Test-Drive This PR

(step1)

Checklists

General

[ ] Changes are covered by test cases
[ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance
You must check off all items in one of the follow two lists:

[ ] There are no changes to UI

or...

[ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
[ ] Changes comply with WCAG 2.0 guidelines levels A and AA
[ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

[ ] Changes include a UI text update in the Retail React App (which requires translation)

Hey Yuna! Thanks for creating the PR, I'm wondering have you verified that the channel_id is now being sent?
You probably have already done this but if in any case that you haven't, you could probably verify it by running the test project and inspect the network requests related to SLAS. We want to make sure the channel_id is the same as the site id and is being sent for both guest and registered user flows.

Thanks @kevinxh, definitely agree, let's include a screenshot or GIF or something that shows the change working as expected!

@kevinxh @bfeister Thanks for the comments! Ran the test project and verified that channel_id is being sent by the SLAS login helpers. Included a screenshot for reference.

Oh and just noticed a nuance of @kevinxh 's comment - be sure to double check for both guest as well as registered user flows

@bfeister Yup! Just verified both. Will merge the PR.


[QUERY] DefaultAzureCredential fails to re-authenticate when debugging locally

Library name and version
Azure.Identity 1.5.0
Query/Question
We are using DefaultAzureCredential to authorize az-f() calls. In particular, we use the following code:
var blobServiceClient = new BlobServiceClient(new Uri(_options.StorageServiceUri),
       new DefaultAzureCredential());

It works perfectly as described. However, for one of the older projects that I have't used for awhile, I now get this error when I debug my az function locally:
ResponseBody: {"error":"invalid_grant","error_description":"AADSTS700082: The refresh token has expired due to inactivity. The token was issued on older date and was inactive for 90.00:00:00.\r\nTrace ID: dd2f0ca1-8033-4a28-9383-20b08d4f0901\r\nCorrelation ID: 87a5c127-edf2-43d1-89f0-3f6eb3b5082b\r\nTimestamp: 2022-02-03 22:13:22Z","error_codes":[700082],"timestamp":"2022-02-03 22:13:22Z","trace_id":"dd2f0ca1-8033-4a28-9383-20b08d4f0901","correlation_id":"87a5c127-edf2-43d1-89f0-3f6eb3b5082b","error_uri":"https://login.microsoftonline.com/error?code=700082","suberror":"bad_token"}
Here is more of a callstack:
[2022-02-03T22:13:25.593Z] Correlation ID: 87a5c127-edf2-43d1-89f0-3f6eb3b5082b
[2022-02-03T22:13:25.593Z] Timestamp: 2022-02-03 22:13:22Z
[2022-02-03T22:13:25.593Z]    at Microsoft.Identity.Client.OAuth2.OAuth2Client.ThrowServerException(HttpResponse response, RequestContext requestContext)
[2022-02-03T22:13:25.593Z]    at Microsoft.Identity.Client.OAuth2.OAuth2Client.CreateResponse[T](HttpResponse response, RequestContext requestContext)
[2022-02-03T22:13:25.593Z]    at Microsoft.Identity.Client.OAuth2.OAuth2Client.ExecuteRequestAsync[T](Uri endPoint, HttpMethod method, RequestContext requestContext, Boolean expectErrorsOn200OK, Boolean addCommonHeaders)
[2022-02-03T22:13:25.594Z]    at Microsoft.Identity.Client.OAuth2.OAuth2Client.GetTokenAsync(Uri endPoint, RequestContext requestContext, Boolean addCommonHeaders)
[2022-02-03T22:13:25.594Z]    at Microsoft.Identity.Client.OAuth2.TokenClient.SendHttpAndClearTelemetryAsync(String tokenEndpoint, ICoreLogger logger)
[2022-02-03T22:13:25.594Z]    at Microsoft.Identity.Client.OAuth2.TokenClient.SendHttpAndClearTelemetryAsync(String tokenEndpoint, ICoreLogger logger)
[2022-02-03T22:13:25.594Z]    at Microsoft.Identity.Client.OAuth2.TokenClient.SendTokenRequestAsync(IDictionary 2 additionalBodyParameters, String scopeOverride, String tokenEndpointOverride, CancellationToken cancellationToken)
[2022-02-03T22:13:25.595Z]    at Microsoft.Identity.Client.Internal.Requests.ByRefreshTokenRequest.ExecuteAsync(CancellationToken cancellationToken)
[2022-02-03T22:13:25.595Z]    at Microsoft.Identity.Client.Internal.Requests.RequestBase.RunAsync(CancellationToken cancellationToken)
[2022-02-03T22:13:25.595Z]    at Microsoft.Identity.Client.ApiConfig.Executors.ClientApplicationBaseExecutor.ExecuteAsync(AcquireTokenCommonParameters commonParameters, AcquireTokenByRefreshTokenParameters refreshTokenParameters, CancellationToken cancellationToken)
[2022-02-03T22:13:25.595Z]    at Azure.Identity.AbstractAcquireTokenParameterBuilderExtensions.ExecuteAsync[T](AbstractAcquireTokenParameterBuilder 1 builder, Boolean async, CancellationToken cancellationToken)
[2022-02-03T22:13:25.595Z]    at Azure.Identity.MsalPublicClient.AcquireTokenByRefreshTokenCoreAsync(String[] scopes, String claims, String refreshToken, AzureCloudInstance azureCloudInstance, String tenant, Boolean async, CancellationToken cancellationToken)
[2022-02-03T22:13:25.595Z]    at Azure.Identity.MsalPublicClient.AcquireTokenByRefreshTokenAsync(String[] scopes, String claims, String refreshToken, AzureCloudInstance azureCloudInstance, String tenant, Boolean async, CancellationToken cancellationToken)
[2022-02-03T22:13:25.596Z]    at Azure.Identity.VisualStudioCodeCredential.GetTokenImplAsync(TokenRequestContext requestContext, Boolean async, CancellationToken cancellationToken)
[2022-02-03T22:13:25.596Z]      StatusCode: 400
I searched everywhere, but I could not find where Visual Studio for Mac keeps the tokens.
Does anybody know how I can force Visual Studio for Mac to re-authenticate or delete the refresh token? Thank you!
Environment
Visual Studio for Mac, netcoreapp3.1, Azure Function project

Thank you for your feedback.  Tagging and routing to the team members best able to assist.

Hi @dmitrymal I suspect that you are getting the credential from another source. This can be validated by enabling logging on the credential. You can find details on how to enable it here.


Clarify a little bit how to use

given your how to use, consider this scenario:
-Search by title but not by body
-When a title is found, pull the body data
The way I've been using it, is searching all data, and that had a severe performance downgrade in the lighthouse test.

Is what I'm asking the index plugin options?

Hi @decimoseptimo, yes, index is what you're looking for.
Your normalizer function should return data that should either be indexed, stored, or both.
Typically you would want to index as few fields as possible to keep the index size small. Your store can contain the larger fields that are needed to display the results, but may not be necessary to index.

Glad to hear that, I'm currently researching performance bottelnecks, and this was one of them, now it seems fixed.
One last thing, does ref: 'id' have to match graphql id? i ended up using ref: '_id' since i use _id. Is not a problem, just wondering.

Good question! No, it does not need to match. If you set ref to _id, just make sure your normalizer function returns an _id property.
It defaults to id just as a convention but you can set anything as your ref.


Handle Returndatacopy and additional fixes

PR is fine. But I miss something. The title is returndatacopy and I cannot see anything specific about returndatacopy in code.

the bumped evmone submodule adds the identity precompiled contract, allowing us to CALL and later RETURNDATACOPY in our programs.


Enhanced Logging

Re-structure the logging framework such that it can be configured at run time.

Allow multiple log output handlers with different levels.

Add custom log file output to the command line.

Switch to Commons Logging for all logging output?

Better define the log levels so a saner amount of logging is produced. (I've seen gigabyte log files)

Check all info level log output. In particular insure that each task displays a "starting" and a "completed", and that all parameteristation is output at some point. Also the memory info should be produced on debug level periodically.

Commons logging now implemented throughout

Mostly done in develop branch:

All project logging uses commons logging.
Log4j will be picked up at run time if the jar is in the libs folder. Otherwise it falls back to JDK1.4 logging.
The output has been re-fined somewhat. There is still a LOT of output, but it's not so much and it's more helpful.

Over all this issue hasn't been entirely resolved, but then it was quite vague. Closing until there are some more specific problems to solve.


Decorate Request and Reply with a function to get the label

Why is this not closed?

I think it will be automatically closed when we do a prerelease or a release

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add logic to help with checking on hooks

This will help us make sure that pull request builds will provision correctly.
It checks if ros-pull-request-builder has appropriate permissions. It requires the environment ROSGHPRB_TOKEN to be set to check. But it will skip the test if that's unset.

It is great to test this automatically rather on requiring us to manually do so. Is there anything in the output which indicates that the new checks have been run?

It prints a pass notice here: https://github.com/ros/rosdistro/pull/16040/files#diff-2fc07dfe4e98c3fa7008712936aabe4cR91
Maybe it would make more sense to explicitly invoke this as a separate travis test so for the ambiguous cases we know if it's just a hook issue and won't effect local users running tests that they cannot reproduce.

It prints a pass notice here: https://github.com/ros/rosdistro/pull/16040/files#diff-2fc07dfe4e98c3fa7008712936aabe4cR91

I didn't see the output in the log anywhere. I guess it only gets printed if the patch actually changes a repository entry so it isn't visible for this PR?

Correct, it only checks for hooks inside a changed source entry.

Updated for comments

2 more fixups.

All error messages should be printed to stderr.

This comment is still pending.

Fixed stderr output. I missed that in the general comments.


Add audio to Recording

Fixed in https://github.com/rampadc/NDISenderExample/commit/348cc9b9d881947068b700b49d32250b1cb4c49a


Not booting on Raspberry Pi 4 Model B (4GB RAM)

I'm not exactly sure what the issue is as the Pi doesn't boot at all, but when I flash the latest release to the microSD card and power on the Pi, the LED blinks a couple of times to indicate it's reading the disk and then goes off and only the power LED remains. The Pi does not connect to ethernet or appear to be processing at all.
Obviously this is a very new Pi. Just documenting that it doesn't work :)

OK. Hypriot (official) isn't booting neither, so it doesn't appear to be 64-bit specific. I'm sure I'd seen somebody else post a video in which they installed hypriot onto the Raspberry Pi 4 the day it was released. I guess not, or there's something changed with the boot process that needs updating for it to work.
Raspbian official is working fine, so no issue with the device or power etc.

Currently I don’t have a Raspberry Pi 4, will take some time to get one as they are sold out everywhere.
Typically we have to install the latest boot files first, which should support the RPi4 for the initial boot process. Then it will identify the RPi4 board/cpu and tries to load the correct kernel file and the RPi4 related device tree file - now you know, these files are specific for the RPi4 and should be also provided on the SD card. Best way to find this out is installing the boot files first, connect the RPi4 over the UART and watch the boot logs seen there.

Raspberry Pi 4 doesn't provide currently a good 64bit kernel. Therefore no support for the Pi4 until the kernel is available. Closing this issue for now.

Yeah, noticed that. It works, but only if memory is capped at 1GB, AFAIK. Currently my setup is a cluster of Pi3B's running Kubernetes. They were running HypriotOS (32-bit), but the master node as of about 1 year ago fails to initialize or upgrade without some hacks as it's under resourced. I switched to Hypriot 64-bit largely because there were documented performance optimizations in Go targeting Arm64 and this would be applicable to k8s. However it didn't actually help with the master initialization in reality, though I've stuck with it. I was hoping Pi4 w/ 4GB RAM would be my savior, and it seems like it is (On official Raspbian buster k8s initializes in what looks like < 60s), though it makes sense for me to either switch the whole cluster back to Raspbian and do it that way, or wait for better 64 bit support, rather than run a 32-bit master and 64-bit nodes. That said mixed-architecture k8s clusters do work just fine.

@d11wtq have you looked into using k3os?

Interesting. Will have a :eyes:.


[Azure.Core] Adding suppression

This PR adds the same warning suppression for the DiagnosticScope.Failed(...) overload on L141 to the new overload that was added in https://github.com/Azure/azure-sdk-for-net/pull/39617
cc/: @vitek-karas

API change check
API changes are not detected in this pull request.


back end failed to launch on Huggingface due to unexpected griffe version

Try to launch mindsearch/app.py on Huggingface as part of gradio app.
After installed the required modules as mentioned in requirements.txt.
The app failed to launch and got the following log in container:
Traceback (most recent call last): File "/home/user/app/app.py", line 11, in  from lagent.schema import AgentStatusCode File "/usr/local/lib/python3.10/site-packages/lagent/init.py", line 2, in  from .actions import * # noqa: F401, F403 File "/usr/local/lib/python3.10/site-packages/lagent/actions/init.py", line 3, in  from .action_executor import ActionExecutor File "/usr/local/lib/python3.10/site-packages/lagent/actions/action_executor.py", line 4, in  from .base_action import BaseAction File "/usr/local/lib/python3.10/site-packages/lagent/actions/base_action.py", line 16, in  from griffe.enumerations import DocstringSectionKind ModuleNotFoundError: No module named 'griffe.enumerations'
I tried to append griffe==0.48 to requirements.txt can solve this issue.

The Pull Request https://github.com/InternLM/MindSearch/pull/174 is target to fix it.

@BetterSnail @LCorleone @zehuichen123 @Harold-lkk please help to review this PR.

We've fixed this in toolkit lagent, check this.

OK


Updated bitstream patch validation

Description
Earlier this week I noticed that I could modify UploadConfigurations for the UI, but it's not possible to use a custom bitstream metadata submission form. You can create a new UploadConfiguration bean that references a custom bitstream metadata form and add this to the UploadConfigurationService bean in access-conditions.xml (as described in the documentation). This can be mapped to a submission process by handle and the metadata form appears in the UI. But when you submit the changes to the backend the PATCH operation fails.
This is because the backend is hard-coded to use the default "bitstream-metadata" form in submission-forms.xml for validation. In fairness, this seems consistent with the documentation.
The changes in this PR allow for custom bitstream metadata forms in addition to the default "bitstream-metadata" form.
Instructions for Reviewers
List of changes in this PR:

Patch validation changed to use the UploadConfigurationService to read the metadata used for validation.

I believe current IT's provide test coverage.
Testing requires a submission configuration that uses an alternative bitstream metadata form. I suppose it's possible to define the new form in submission-forms.xml and then just use this form
name in the  uploadConfigurationDefault "metadata" property. A more complete and realistic test is to create a new submission process, but that's probably not necessary for testing.
Question: The dc.title field is required so it seems like we could add a check for the required title field during validation. I wanted to see what others think first.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[ ] If my PR includes new libraries/dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[ ] If my PR modifies REST API endpoints, I've opened a separate REST Contract PR related to this change.
[ ] If my PR includes new configurations, I've provided basic technical documentation in the PR itself.
[ ] If my PR fixes an issue ticket, I've linked them together.

Thanks @abollini. I will take a look at the DSpace-CRIS code.
Just wondering, did you also work on the bitstream edit form for existing bitstreams (under Item edit)? I've had time recently to modify it to use the bitstream form that is configured for the collection and might have a working proposal soon. If this has already been done by someone else who can share that would be great!  The dynamic forms are tricky. (It's mostly Angular with possibly a minor REST addition).

Hi @abollini , sorry for the delay. I ported your dspace-cris code and tested locally. It works as expected. I didn't notice any ITs that cover this feature. Presumably, if they do exist they prove that the change didn't break anything.

I've also verified the previously failing ITs in this PR were already fixed on main.  They are unrelated to this PR.


WIP - Bayes factor charts

I think this is all the code that needs to change in this repo to address #115
Happy to take opinions on:

Colour scales
Axis ranges

currently automatic for all of the iteration history charts so they don't align, but are easier to interpret
domain set to [-10,10] for the main chart to cover a reasonable range (up to m/u = 1000)

Chart titles to be more specific about interpretation of Bayes factor?

This is brilliant.
A couple of minor comments/thoughts:

Would be good to label the x axis of the 'Distribution of (non-null) 𝛾 values' to something like 'proportion of rows'

While we're at it label the x axis of the probability_distribution_chart as 'proportion of non-matches' and 'proportion of matches'

I would probably be tempted to completely delete the 'Probability distribution of comparison vector values by iteration number' chart. With your new bayes factor iteration chart, I don't think it adds anything and ends up just being too much information

Oh - also - before you merge increment
version = "0.2.2"
to version = "0.2.3"
in pyproject.toml
Then give me a shout and I can run you through how to release on pypi (or just go ahead and do it if you already know how)


jest-haste-map: @providesModule naming collision

after run 'create-react-native-app my-app', and then run 'npm run ios'，I got error：
jest-haste-map: @providesModule naming collision:
Duplicate module name: glob
Paths: /Users/guhaoxin/Desktop/react-native-workspace/gank/node_modules/react-native/node_modules/rimraf/node_modules/glob/package.json collides with /Users/guhaoxin/Desktop/react-native-workspace/gank/node_modules/react-native/node_modules/glob/package.json
This warning is caused by a @providesModule declaration with the same name across two different files.

This seems to be a react-native and<EMAIL_ADDRESS>issue. The same issue is affecting me when upgrading in my standard react-native app.

This also happens with 'react-native init' and 'run-android'

similar issue but Duplicate module name: fb-watchman , Duplicate module name: bser
Loading dependency graph...jest-haste-map: @providesModule naming collision:
  Duplicate module name: bser
  Paths: ...

This warning is caused by a @providesModule declaration with the same name across two different files.
jest-haste-map: @providesModule naming collision:
  Duplicate module name: fb-watchman
  Paths: ...

This warning is caused by a @providesModule declaration with the same name across two different files.
jest-haste-map: @providesModule naming collision:
  Duplicate module name: bser
  Paths: ...

having same exact problems.. this is my package.json
{
  "name": "xxx",
  "version": "7.0.0",
  "private": true,
  "devDependencies": {
    "babel-jest": "17.0.0",
    "babel-preset-react-native": "1.9.0",
    "chai": "^3.5.0",
    "jest": "17.0.0",
    "jest-expo": "~1.0.1",
    "jest-react-native": "17.0.0",
    "mocha": "^2.5.3",
    "react-native-scripts": "0.0.30",
    "react-test-renderer": "16.0.0-alpha.6",
    "remote-redux-devtools": "^0.3.3",
    "remote-redux-devtools-on-debugger": "^0.4.6"
  },
  "main": "./node_modules/react-native-scripts/build/bin/crna-entry.js",
  "scripts": {
    "postinstall": "remotedev-debugger",
    "start": "react-native-scripts start",
    "eject": "react-native-scripts eject",
    "android": "react-native-scripts android",
    "ios": "react-native-scripts ios",
    "test": "node node_modules/jest/bin/jest.js --watch"
  },
  "jest": {
    "preset": "jest-expo"
  },
  "dependencies": {
    "@expo/vector-icons": "~5.0.0",
    "expo": "^17.0.0",
    "lodash": "^4.13.1",
    "moment": "^2.13.0",
    "native-base": "2.1.4",
    "react": "16.0.0-alpha.6",
    "react-native": "github:exponent/react-native#sdk-17.0.0",
    "react-native-camera-roll-picker": "^1.1.9",
    "react-native-gifted-chat": "0.1.3",
    "react-native-modalbox": "^1.3.7",
    "react-native-multi-slider": "^0.3.5",
    "react-native-router-flux": "3.38.0",
    "react-native-scrollable-tab-view": "^0.7.2",
    "react-native-swiper": "^1.5.4",
    "react-redux": "^4.4.5",
    "redux": "^3.5.2",
    "redux-persist": "^3.2.2",
    "redux-thunk": "^2.1.0"
  }
}

:+1:

same problem.

+1

https://github.com/facebook/react-native/issues/13364

if you use jest-expo, don't install other versions of jest :) additionally, this is a problem with the react-native packager / haste module system that there is work in progress to attempt to resolve. it's not due to create-react-native-app itself so I'll close the issue here and you can track in the issue mentioned above by @ajostergaard

+1

same issue. how can i resolve it please?.

Same issue ?? +1

+1

For me Problem was, watchman path in windows environment variables. Removing that unused path fixed problem for me. Hope this helps to someone.

@ajostergaard ,
Is the issue solved? can I know the solution?

Let me know in details. Is this the first time you are starting that project entirely? that will make me know where we could really pin point where the problem is coming from.

I am facing the issue on RN 0.56 and xcode 9.4.
I tried giving the path for React in my pod file to node_modules/react-native but still it installs a React folder to pods/ .
Can someone please explain how to fix this bug

I had this same problem as well but was able to fix it by removing pod files installed in ios/Pods
And then clearing cache

@ameshkin
rm -rf ios/Pods
pod install
how to clear cache?

ios/Podfile
I commented out it. I didn't wrote the line but it automatically wrote...
  # pod 'RNVectorIcons', :path => '../node_modules/react-native-vector-icons'
``

and `pod install` again. it works.


Remove the nonce cache.

#434
@HaoK @PinpointTownes @brentschmaltz
cc: @Eilon @muratg

:shipit:

If you really want to remove the server-side cache, I won't fight for it. But you have to realize that this nonce feature offers absolutely no protection against replay attacks... it's basically pointless here.

@PinpointTownes http://openid.net/specs/openid-connect-core-1_0.html#NonceNotes
The use of the cookie is specifically recommended to keep a 3rd party from resubmitting the same token. There's also the TVP.TokenReplayCache.

The use of the cookie is specifically recommended to keep a 3rd party from resubmitting the same token.

Yes, but if you don't check the nonce against already-used nonces, you can't detect replays. Again, preventing replay attacks is not mandatory (for both the code and the implicit flows), but clearly recommended by the specs.

There's also the TVP.TokenReplayCache.

The problem with TokenReplayCache is that there's no default implemention (unlike WIF, that had a default in-memory implementation). Do you really think someone will ever implement it? Setting a bool to true was much easier :smile:

You forget the part about setting up a distributed cache.

Ah yeah, services.AddCaching() or (services.AddRedisCache() for a real distributed implementation) :smile:

The more I think about it, the more I realize that keeping IDistributedCache is not harmful. But I think it needs a better name to clearly indicates what it can be used for. What about renaming CacheNonces to DetectReplayedTokens (the name used by WIF) or DetectReplayAttacks?

@brentschmaltz? This still seems redundant with TokenReplayCache, no?

I think we're still going to remove it for now.


Create script to summarize and collate info on user suspiciousness

Possible indicators to help determine whether user is flagged as suspicious:

graph giving network (interested in all other indicators for other users on graph)
Gittp sign-up date
Gittip bank/CC attachment dates
Elsewhere sign-up dates (GitHub, Twitter, etc)
Elsewhere activity
github ssh keys

cc: @bruceadams

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Closing in light of our decision to shut down Gratipay.
Thank you all for a great run, and I'm sorry it didn't work out! 😞  💃


Decimal on Mac OSX

Hi,
I'm a Mac OSX v. 10.8.4 user.
When i am trying to compil scss files containing variables with decimal value into css file, it compils this way (except the first time):
scss:
$font_large: 1.4em;
$color: rgba(#345, 0.7);
span#logo {
font-size: $font_large;
background-color: $color;
}
css:
span#logo {
font-size: 1em;
background-color: rgba(#345,0);
}
Note:
why there's no auto prefix option in kind of prefixr ?
thank you
Jérôme, Swiss based webdesigner and webdeveloper.

Closing this ticket. Download the latest version of Scout-App. If the problem still exists, create a new issue.

Scout-App.io


Use Math.trunc in toString() instead of bitwise operator.

The current implementation of toString() uses a bitwise operation to find the whole number part of a fraction:
str+= N / D | 0;

This has a side-effect of converting the result to a signed 32-bit integer value, which results in numeric overflow when formatting larger values.
More information about this here.
Using Math's built-in truncation will allow the code to work for larger values supported by Javascript's number type.
A test was added that demonstrates this. Before this change, (new Fraction(1e12)).toString() returned "-727379968". After this change, the same code correctly returns "1000000000000".

Hi @infusion, could you take a look at this PR when you get a chance? Thank you!

Could you please use Math.floor everywhere instead of introducing the trunc-function? This also helps gzip compression when these function calls are repeated.

Could you please use Math.floor everywhere instead of introducing the trunc-function? This also helps gzip compression when these function calls are repeated.

Done, thanks for taking a look!

This looks good, thanks for your work!!


Configuration of the linter

The actual linter is Black. There is not a configuration provided to it.
We need to check if it's enough or not?
For the future, it can be interesting to look at Ruff for being compatible with scitkit-learn.

+1 for ruff: also because it gives you single tool to replace black, isort and whole bunch of other things...


add Multiple Signatures on a Single PDF

I want to add multiple signatures on a single Pdf. My code as follows  `System.Drawing.Rectangle rect = new System.Drawing.Rectangle(100, 100, 100, 100);
        PKCS1 signature = new<EMAIL_ADDRESS>"20180201");
        PdfFileSignature pdfSign = new PdfFileSignature(new Document(@"input.pdf"));
        pdfSign.SignatureAppearance = "test.png";
        pdfSign.Sign(1, "Signature Reason", "Contact", "Location", true, rect, signature);
        pdfSign.Save(@"output.pdf");
        pdfSign.Close();
        rect = new System.Drawing.Rectangle(260, 260, 100, 100);

        PKCS1 signature2 = new<EMAIL_ADDRESS>"20180201");
        pdfSign = new PdfFileSignature(new Document(@"output.pdf"));
        pdfSign.SignatureAppearance = "test.png";
        pdfSign.Sign(1, " Reason2", "Contact", "Location", true, rect, signature2);
        pdfSign.Save(@"output.pdf");
        pdfSign.Close();`

There is a problem that  my reader  (Adobe Acrobat Reader DC)tells me that Signature verification statusThe signature is invalid.The document has been altered or corrupted since the signature was applied.

@bluetianx , Would you please share all the resources including source and generated PDF file, PFX file and a screenshot displaying the problem in Adobe Acrobat DC. So that we may try to reproduce and investigate your scenario exactly, to help you out. Before sharing request data via Google Drive, Dropbox etc, please ensure using Aspose.PDF for .NET 18.8 in your environment.
PS: I work with Aspose as Developer Evangelist.

sorroy, i cannot user Google Drive,I upload resouces in my github github,  You can run method TestMethod1() of UnitTestConsole project,then  you can the output pdf file in AsposePDFTest\TechLab\UnitTestConsole\bin\Debug\netcoreapp2.0.

The version of acrobat that I use is 18.011.20058.33888

@bluetianx , Thank you for sharing requested resources. We have logged a ticket with ID PDFNET-45257 in our issue management system for further investigation and resolution. We will update you as soon as some significant updates will be available.
We are sorry for the inconvenience.

Alright 。thanks


Duplicate and unused Json merge methods.

There are two methods: JsonMergeUtil.mergeOnto (unused method) and Config.merge which are not implemented in the same way, but they do almost the same thing.
Which of them can we remove?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29675231-duplicate-and-unused-json-merge-methods?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github).

@msteiger what do you think? The JsonMergeUtil.java version is the only thing in the class so we could probably just delete the whole class. Especially if it is showing as unused anyway.


Should be able to use /dev/tty instead of stdin

I think the Stdin module should read from /dev/tty instead of stdin (or else the user should be able to choose). Otherwise writing interactive programs that read input from stdin (e.g. fzf in cat ... | fzf) will have problems.
Specifically, when I tried doing that with minttea, I got an ENOTTY error message (although the program still seemed to function as expected). Changing the stdin module to start with let stdin_fd = Unix.descr_of_in_channel (In_channel.open_bin "/dev/tty") got rid of it.

Hi @roddyyaga! This makes perfect sense. Also would be kinda cool to "replay" by going cat file | tui. Should I make the change or would you prefer opening a PR?

You can make the PR :)


Switching to next script often doesn't switch the focus

Godot version
v4.1.rc2.official [46424488e]
System information
Windows 11
Issue description
More often than not when using shortcut to switch to next script, even though opened script is switched, focus (cursor) is not moved to that file so one has to click to be able to actually work on that script. No problems observed when switching to previous script or using history next or history previous shortcuts.
https://github.com/godotengine/godot/assets/15647621/083464d5-9bd3-465a-988f-9d6a5b28cd0a
Steps to reproduce

Open couple of scripts.
Use shortcut to switch to next script (default shortcut: Ctrl+Shift+.).
Actual behaviour
Open file is switched. Writing or navigating with arrows, etc. is not working. There is one line highlighted but there is no blinking cursor there.
Expected behaviour
File is switched. Cursor is in the last position it was in that file. Editing and navigation with keyboard is possible immidietly.

Minimal reproduction project
N/A

On Linux (KDE, X11) it works correctly. It retains the last cursor position in each file and the focus is in the editor. I've tried it on 4.1 RC2 as well, using Ctrl+Shift+. and Ctrl+Shift+,, see the gif below.
Probably it's a Windows-only bug. But it would be nice to check it on MacOS as well.


[next] js-Files do not get imported

Version
@nuxtjs/i18n: 8.0.0-alpha.0-27596800.2df94e2
nuxt: 3.0.0-rc.4
Nuxt configuration
Please change to [x] if relevant for this issue:

[ ] Applies to a site deployed to a static server (site generated with nuxt generate)
[ ] Applies to a site deployed to a server with a Node backend

Steps to reproduce
Create a js file for a language, eg. en.js that returns the messages.
What is actually happening?
It seems like these files are mostly ignored right now, because the resolveLocales method in the utils only resolves json,json5,yaml,yml. It does work when I just add js there. Is there some more implementation work needed, or would that solve the issue? Or am I missing some configuration on my part?

@timgoeller , js file is not available for the moment.
See this file :
https://github.com/nuxt-community/i18n-module/blob/next/TODO.md


ODK-X Documentation - Relocate Translation Button To Improve Usability

The language translator button is located at the end of the page, this may not be a good user experience for non-English speaking users as they may have initial confusion trying to locate the translator button, and they may leave the entire page thinking there is no translation option on the page. To improve usability, we may need to move this button to the navigation bar or place the button at the top of the page.
https://docs.odk-x.org/

Hi, mentors
Can I please be assigned to this issue? I was also wondering if it can be merged with the initial issue I raised.

Thank you for your reply @maprehensive. I understand.

This means issues raised should be mostly focused on the mobile applications, right?


Feature/fix for building using clang

For more information w.r.t. to the default initialization issue in UartWindows.hpp, see: https://godbolt.org/z/E7qGE375K

Closed because I selected the wrong target branch


Release v1.39.0

Proposed Change
Release v1.39.0 prep.
Checklist

[ ] Changes are tested
[ ] CI has passed

Any chance we can squeeze the sort by mtime feature flag in this release?

This requires OTel v0.89.0, which isn't released fully just yet.


Leaning subdocuments virtuals with select does not work on last mongoose version

Hi !
With mongoose 5.12.2 and mongoose-lean-virtuals 0.7.6, just found that the next query only lean the root _id value :
 Project.find(match)
            .select('_id name')
            .populate('company', '_id name')
            .lean({ virtuals: ['id'] });  // we only need the id virtual

// Return :
"{
  "_id": "5cade6a33c27940010c01de2",
  "company": {
    "_id": "5cadc7773c27940010c00298",
    "name": "SOME COMP"
  },
  "name": "A PROJECT",
  "id": "5cade6a33c27940010c01de2"
}"

result.company haven't any "id" property.
From the past, with mongoose 5.9.1 and mongoose-lean-virtuals 0.5.0, the same query return :
"{
  "_id": "5cade6a33c27940010c01de2",
  "company": {
    "_id": "5cadc7773c27940010c00298",
    "name": "SOME COMP",
    "id": "5cadc7773c27940010c00298"
  },
  "name": "A PROJECT",
  "id": "5cade6a33c27940010c01de2"
}"

Virtual worked fine with subdocument.
However, It work well using  .lean({ virtuals: true });.
But we have some virtuals that we dont want to return everytime, and deleting them from each document/subdocument is not suitable.
Is it intended ?

This behavior is by design. In order to get company.id to show up with lean-virtuals, you need to do .lean({ virtuals: ['id', 'company.id'] })


bwa mem

Why is bwa-mem used with the pacbio option instead of the ont2d?

This was in a version of the pipeline before ont2d came along. The marginAlign pipeline is the more updated and supported version.


Memory leak caused by ChunkExtractor

🐛 Bug Report
Server side rendering in combination with specifying the stats file path for the ChunkExtractor seems to cause a memory leak. It might be in connection with streaming the response, but I'm not sure about this yet. Lucky wise I found a better solution and think we should adapt the documentation to preload the file only once anyway. Whats the point of loading it for every request again? Do you agree and maybe know the issue of the memory leak?
To Reproduce
Example for Fastify:
const chunkExtractor = new ChunkExtractor({ statsFile: path });
  const jsx = await createReactTree(chunkExtractor);
  const resStream = new PassThrough();
  reply
    .headers({})
    .status(200)
    .send(resStream);
  resStream.write(buildHeader(chunkExtractor));
  const stream = renderToNodeStream(jsx);
  await new Promise((resolve, reject) => {
    stream.pipe(resStream, { end: false });
    stream.on("error", err => {
      resStream.end();
      reject(err);
    });
    stream.on("end", () => {
      resStream.write(buildFooter(client));
      resStream.end();
      resolve();
    });
  });

Expected behavior
ChunkExtractor loads stats and releases all allocated memory after response is completed.
Used solution
I was able to solve it in my case by simply loading the file once during startup. If the file otherwise gets loaded for every request, then the examples and documentation should also be changed, as it will decrease performance and doesn't seem to be necessary:
const statsFile = resolve(__dirname, "loadable-stats.json");
const stats = JSON.parse(readFileSync(statsFile).toString("utf8"));

const doAnswer = async (req, res) => {
   const chunkExtractor = new ChunkExtractor({ stats });
   // more code
}

Link to repl or repo
Unfortunately, this is private project. But as I already found a better solution, I just like to inform you in case you might be aware of the underlying issue.
System:

OS: Linux 5.5 Fedora 31 (Workstation Edition) 31 (Workstation Edition)
CPU: (8) x64 Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz
Memory: 23.52 GB / 31.08 GB
Container: Yes
Shell: 5.0.11 - /bin/bash

Binaries:

Node: 10.16.0 - ~/.nvm/versions/node/v10.16.0/bin/node
Yarn: 1.22.4 - /usr/bin/yarn
npm: 6.9.0 - ~/.nvm/versions/node/v10.16.0/bin/npm

npmPackages:

@loadable/babel-plugin: ^5.11.0 => 5.12.0
@loadable/component: ^5.11.0 => 5.12.0
@loadable/server: ^5.11.0 => 5.12.0
@loadable/webpack-plugin: ^5.7.1 => 5.12.0

Yes, it is a memory leak, hopefully only in dev - here
https://github.com/gregberge/loadable-components/blob/4e99a97c5ae958c4f18ac82b7e1a3566bc81c912/packages/server/src/util.js#L3-L6
The module cache got cleared, so file could be required again, but it does not clean reference from the parent module, so there is more and more copies stored in the "parent module" (which is util.js)
Open questions:

which way - stat or statFile - is better for dev? For production there is no difference
why stats has to be updated so "forceably"? With HMR you don't have to remove something from the cache
should one keep util module cache clean? Ie wipe data from the require[module].children

"Great" :)
I'm not sure about the correct way yet. I don't use HMD but reloading the server with Webpack in watch mode together with Nodemon and stat seems to work fine.
Somehow I'm confused that there is an ENV check. I thought it should already be executed with NODE_ENV set to production. I will check this on my side.

Refs:

https://github.com/theKashey/wipeNodeCache/issues/4
https://github.com/theKashey/rewiremock/issues/116


Compilation fails with Intellij 14.1.5

Context:

Intellij 14.1.5
Lombok plugin: <IP_ADDRESS>

Issue:

everything is fine in the editor view :

no warning
lombok processing is fine (for instance, @Getter produces usable getter in other class, ...)

but compilation fails (build project/compile class) as if the annotation processing is not taken into account in the compile phase

Sample class:
package net.devlab722;

import lombok.RequiredArgsConstructor;

@RequiredArgsConstructor
public class Lombi {

    private final String first;
    private final String second;
}

The logs:
/home/cfurmaniak/workspace/SANDBOXES/grokking-algorithms/src/main/java/net/devlab722/Lombi.java
Error:(8, 26) java: variable first not initialized in the default constructor
Error:(9, 26) java: variable second not initialized in the default constructor

Did you enabled annotation processing in the options of intellij?

Yes, with or without it, same failure result.

What is your jdk version? Are you using javac or eclipse compiler or something else?

Information:Using javac 1.8.0_60 to compile java sources

└> java -version
openjdk version "1.8.0_60"
OpenJDK Runtime Environment (build 1.8.0_60-b27)
OpenJDK 64-Bit Server VM (build 25.60-b23, mixed mode)

Can you verify it with oracle jdk please?

ok, this is weird. I checked twice on 2 distincts setup before submiting the issue. And it failed on both setup.
On top of that, someone reported the same issue (twitter).
And now, on one of my 2 setups...it works.
I will check on my second setup tonight.
stay tune

Any new results? It should simple work:)

@looztra

It works on one setup and still doesn't work on my second setup. Still
investigating what is the difference in the environment.
Is it working for everybody else?
On Mon, 2 Nov 2015, 12:04 Jan Marian Meyer<EMAIL_ADDRESS>wrote:

@looztra https://github.com/looztra
—
Reply to this email directly or view it on GitHub
https://github.com/mplushnikov/lombok-intellij-plugin/issues/146#issuecomment-152989107
.

All my colleagues doesn't have any problems with compilation (both MacOs and Windows)
Please verify your compiler settings and the option "Enable annotation processing". If somebody has any issues, this was the root cause.

I had the same issue as @looztra .. enabling annotation processing fixed it for me

Just tried with the oracle JDK: same problem.
Tried with a fresh install of intellij 15: same problem (by fresh I mean I cleared the .IntelliJIdea15 directory in my home).
(and yes, annotation processins is enabled)

ok, I think I get it, and I'm a bit ashamed I did not realize earlier that it was related to the project that I used for testing.
I used the same class during all my tests, but not always in the same project :(
The answer is that in the non working project there are these dependencies:
<dependency>
            <groupId>org.openjdk.jmh</groupId>
            <artifactId>jmh-core</artifactId>
            <version>${jmh.version}</version>
        </dependency>
        <dependency>
            <groupId>org.openjdk.jmh</groupId>
            <artifactId>jmh-generator-annprocess</artifactId>
            <version>${jmh.version}</version>
            <scope>provided</scope>
        </dependency>
       <dependency>
            <groupId>org.openjdk.jmh</groupId>
            <artifactId>jmh-generator-annprocess</artifactId>
            <version>${jmh.version}</version>
            <scope>provided</scope>
        </dependency>

Without these dep, it works, with it does not work and generates the errors mentionned in my bug report.

Interesting problem! JMH benchmark generator is based on annotation processor. It's possible, that they interfere each other.
I will close this issue now, because it seems to work now for you.

Was the same problem. I used gradle like this:
dependencies {
...
implementation 'org.projectlombok:lombok'
}
But I needed to use:
dependencies {
...
compileOnly 'org.projectlombok:lombok:1.18.24'
annotationProcessor 'org.projectlombok:lombok:1.18.24'
}
Correct lombok dependency for you can find here: https://projectlombok.org/setup/gradle


Skip serializing docker default network.

Summary:
The Mesos agent determines the default docker network mode by checking
if the ContainerInfo::DockerInfo.network() field is set and setting
the OS specific default. So, we don't serialize the network if it's
the default (HOST in this case).
JIRA issues:
MARATHON-8056

@alenkacz @zen-dog Strongly disagree with this patch. Trying to second guess the user's intent by translating HOST networking to "use the agent default network" is completely inappropriate. While I can empathize with the operator in the scenario illustrated by the OP, I'm very disinclined to approve this commit.
Instead, let's consider an alternative approach: maybe Marathon could support a --default-network-mode flag that allowed a user to override the currently hard-coded default of HOST networking. This flag would, of course, apply for both UCR and Docker containers. However, this approach could prove tricky to get right because of assumptions scattered throughout the code base. And it could be argued that this also breaks API because the API currently states that the default network mode is HOST -- and many app and pod definitions have been written to assume that.
I don't have a good solution for the problem described by the OP. This PR isn't a good solution, sorry.

Like jdef said, translating HOST networking to agent default network automatically is an extreme assumption which would break a lot of existing customer services. Luckily marathon can be easily extended via plugins to achieve what you want. Take a look for example at this plugin - you an update the resulting proto setting the networking to what you need it to be.

Thanks for the comments. I do think something needs to be done even if this PR isn't the correct solution. At the very least, it needs to be documented that the default marathon settings do not work on Windows, and users need to explicitly set the docker networking mode to container or container/bridge for Windows workloads.
@zen-dog Could you give a concrete example on how it would actually break any existing services? The code doesn't check for HOST mode at all; it simply doesn't distinguish "default" in the protobuf from "unserialized" (similar to how proto3 works). The only case that it would ever break if mesos-agent does not honor the default specified in mesos.proto, which never happens on Linux. The only weird thing is that if a user deliberately sends HOST to a Windows container (an invalid configuration), it would end up working due to the HOST -> deserialized behavior.

@gupta-ak I think the right things to do here are:
(a) close out this PR; asking Marathon internals to subvert the intent of the API is not the right approach.
(b) file a JIRA (so Mesosphere and community can track) or else a PR (ditto) for a docs update that calls out potential problems for Windows users and redirects them to the Windows documentation for Mesos (http://mesos.apache.org/documentation/latest/windows/). I suspect that this isn't the only "wall" that Windows users will hit.
(c) file a Mesos JIRA to update the known issues documented by Mesos (http://mesos.apache.org/documentation/latest/windows/). I don't see the HOST networking problem mentioned on this page, and it probably should be.

Sounds good.


Creates CONTRIBUTOR and MAINTAINERS files

Created to create a standard review expectation for potential contributors and a clear representation of maintainers.
The MAINTAINERS file will help avoid issues like those created in https://github.com/ruby-git/ruby-git/pull/342#issuecomment-368945651
closes #349

The failure of jruby-<IP_ADDRESS> in Travis CI seems to be a random occurrence.  My change doesn't even change how the code works but it failed the test.
I would propose that we allow failures of jruby-<IP_ADDRESS> in Travis.  What do you think @perlun?

The failure of jruby-<IP_ADDRESS> in Travis CI seems to be a random occurrence. My change doesn't even change how the code works but it failed the test.

You're right, it seems to be an intermittent error. I restarted the job now.

I would propose that we allow failures of jruby-<IP_ADDRESS> in Travis. What do you think @perlun?

I am hesitant to do that, since it would easily allow us to miss real JRuby breakage (i.e. things that for whatever reason doesn't work there.) I'm a long-time JRuby user myself, and I know how annoying it is with gems that only work on MRI, so for the time being, I think it would be much better to look into the root cause for that intermittent error. There is an issue about it: https://github.com/ruby-git/ruby-git/issues/319

I will rebase the branch when I get home.

@tarcinil I merged this now. Please add the tool in GitHub that checks for DCO so we don't have to check that manually.

@perlun I would love to but I believe I would need to be added to the organization so that I can configure integrations.
Please see the integration located here:
https://github.com/probot/dco
Installation can be done through:
https://github.com/integration/dco

@tarcinil Sure, I'll add you - we need all the help we can get. Visit this page and you should be able to see your invitation: https://github.com/orgs/ruby-git/invitation

@perlun A request has been sent to install it.

A request to install DCO has been submitted on the @ruby-git account.

@tarcinil Great. I approved the request now, thanks for your efforts.


Can't debug version above 0.30.8 on WebStorm or VSCode using remote debugger

It keeps connecting but never connects. It happens on OSX and Linux. I can`t test on Windows.

Its to debug the main proccess

I'm merging this to #3378.

I disagree because I can debug v34 with node inspector

at least on mac...


Add action notifying certain users depending on labels added

Hopefully we will get a notification when an issue is created.

The action file is missing the .yml extension. GitHub is not recognizing the action as such.


Tweak 2210 for ClimaRad

Still not clear what the 3 values in 2100 mean

@zxdavb can you review this?

@silverailscolo I am sorry I have not reviewed your PRs - I am under the hammer with work at the moment, and another repo is consuming all my spare capacity  - please bear with me.

I support your prios. We'll get to it


Update README: Change "Scenarios" header to fix link

The link here didn't do anything because the header name was "Scenario":

For other scenarios, see scenarios.

Updating the header name should fix that link.

Thanks for letting me know! I actually prefer to change the link to be #scenario though. I'll fix it in another PR.


fix: suppress header access with symbol key deprecation warning in Racecar Instrumentation

Fix the Rdkafka deprecation warning for header access with symbol key in the Racecar instrumentation.
This copies the approach taken for the Rdkafka instrumentation here.
Both of these can be cleaned up if/when support for rdkafka 0.12.0 is removed.
@chrisholmes

Hi @ewhorton! Thanks for your submission! Would you mind signing the CLA? We can't accept your submission until you do.

Hi again @kaylareopelle, sorry for the delay, we have just been able to get this signed now 🙂


Refine merging strategy for arrays in app.config

I believe the array should be overwritten instead of merged for the app.config.
Example: https://stackblitz.com/edit/github-jwwim7?file=app.config.ts,layers%2Fbase%2Fapp.config.ts,nuxt.config.ts,app.vue
I expect it to be:
{
  "array": [
    "hello from main"
  ]
}

Closing now that we have function mergers that can be configured by layers.

Upstream tracker for schema refeniement: https://github.com/unjs/untyped/issues/121


Update to gulp 4 and clean up pipelines

Update to gulp 4 to get rid of some npm audit stuff

gulp.task is deprecated, so switch to functions
Leverage ts-node so that we can use typescript for gulpfile

Clean up pipelines

Use common/test.yml for all OS's. I think it's easier to read the steps if they're in the same file. Plus there's less copied code.
Always use Node 8. Again - this is just to keep our builds consistent (previously I think macOS was defaulting to Node 6)

@StephenWeatherford FYI I've been working on doing this in the docker repo as well. I think it will help a few things there


xwm: missing dependency xcb-util-keysyms

@mcpcpc , the build fails without xcb-util-keysyms.
xwm.c:6:10: fatal error: xcb/xcb_keysyms.h: No such file or directory
    6 | #include <xcb/xcb_keysyms.h>

addressed by https://github.com/kisslinux/community/pull/1537


Error while running actions

Hi,
I forked the project and tried running the "Any branch: Build APK" action on the "kiwi" branch.
The build fails with this error

The URL returns 404
I'm probably missing something very simple.
Any help would be greatly appreciated

Lmao

Lmao

Would you please explain?

To run the actions you need special permission from the owner of the original action as building requires access to GitHub build system. kiwi access keys are not cloned for security and privacy reasons.
In theory you can also build kiwi manuallyif you local linux box is equipped with fast CPUs and much RAM.
Basically it's check out original chromium source code from Google and apply the files from this repo on top.

In theory you can also build kiwi manually if your local linux box is equipped with fast CPUs and much RAM.

In practice, you can't. #411

Yes, for now it's not possible (it was at some point), except with https://github.com/kiwibrowser/src the classic repo. The new repo still needs some work again

To run the actions you need special permission from the owner of the original action as building requires access to GitHub build system infrastucture. kiwi access keys are not cloned for security and privacy reasons.
In theory you can also build kiwi manually if your local linux box is equipped with fast CPUs and much RAM. Basically it's check out original chromium source code from Google and apply the files from this repo on top.

Thanks for the detailed explanation

Yes, for now it's not possible (it was at some point), except with https://github.com/kiwibrowser/src the classic repo. The new repo still needs some work again

So there is no way to download this repo and build it?

Any news about building the project localy?

@AviFix, Doing this would require allot of time to explain as there's a few different avenues in approaching this.
The following links should help you in locating information regarding this topic.
https://docs.github.com/en/get-started/quickstart/fork-a-repo
https://frontend.turing.edu/lessons/module-1/github-repo.html
There's no "easy" way to do this in on itself. It all depends on a few factors of what you currently know, how quick of a study you can learn and the knowledge/know how of the language in which it's written (C+, Java, etc...).
From here, you will have to do a little homework/research of your own (just 1 optional hint... Google is your friend... LOL!).
I wish you the best of luck! :-))
~Ibuprophen

https://frontend.turing.edu/lessons/module-1/github-repo.html

Hi
Thanks for your reply
The previous kiwi repo had a manual for building it locally and it worked fine for me
I'm looking for a similar manual that will explain how to build this ripo and get an APK file

Yes, for now it's not possible (it was at some point), except with https://github.com/kiwibrowser/src the classic repo. The new repo still needs some work again

Is it possible now?
Is there any way for me to do some private changes and build a custom APK?

Seems like bunch of changes were just merged today: https://github.com/kiwibrowser/src.next/commit/543d5c3b5be0dd6bd84b7f2fb05618093d0494f8
Did anyone try to compile it? Does it finally have all missing resource files or it's still in non-buildable state?
Thanks.

I will try to compile it as it looks good on a first glance.

Unfortunately, it still misses the java source files specific to kiwi.