Plugins colliding with each other?

I just downloaded OpenIAB into a new project and attempted to make a build of the 'OpenIAB-demo' scene, but got the following errors:
Plugin 'OpenIAB_W8Plugin.dll' is used from several locations:
Assets/Plugins/WP8/OpenIAB_W8Plugin.dll would be copied to <PluginPath>/OpenIAB_W8Plugin.dll
Assets/Plugins/OpenIAB_W8Plugin.dll would be copied to <PluginPath>/OpenIAB_W8Plugin.dll
Please fix plugin settings and try again.

Plugins colliding with each other.

This is using the latest <IP_ADDRESS> release, with Unity 5.0.0f4, and set to the Windows Phone 8 target.

Same here, the problem exists for quite some time

There was an issue with Unity 5. They added plugin inspector, so plugin folders don't matter now.
Anyway I pushed new settings to master. Please try if it works for you.

I would ask you to build a for me plugin? I have eternal problems with Gradle :/
thanks a lot;) super project.

Looking for it as well, as I was unlucky with Graddle too. Thanks :)

Is it fixed?

No news :-(

Hi my solution was to set the editor DLLs to "Editor" instead of "Any Platform" and the WP8 DLLs to "WP8Player".
I really hate it when a big name like unity don't provide documentation on such simple stuff.

I have same problem. Current anyone know how to fix it ?

I have the same issue. How can this be fixed? In which release?
@GrimReio

Try to download ZIP (all files) and then replace from zip the files to your project:
https://s.mail.ru/2ckdi8BUjBkd/img-2015-11-20-20-41-52.png
That worked for me.

Try to be out of this dll(OpenIAB_W8Plugin.dll) in folder of debug,and I solve in this way.


LR200 is not found

When I connect RealSense LR200 camera, no device is found. A RealSense error will be thrown with index out of range (see below). I am aware that LR200 is not supported with this library. However, as far as I understand, the difference to R200 is minimal. Is there a quick solution to get it to work?

File "realsense.py", line 11, in <module>
    pyrealsense.stream.DepthStream(fps=60)])
  File "/Users/vinkoo/.virtualenvs/cv30r/lib/python2.7/site-packages/pyrealsense/core.py", line 80, in Device
    _check_error(e)
  File "/Users/vinkoo/.virtualenvs/cv30r/lib/python2.7/site-packages/pyrealsense/utils.py", line 34, in _check_error
    pp(lrs.rs_get_error_message, e))
pyrealsense.utils.RealsenseError: rs_get_device(context:0x7fea0e222d40, index:0) crashed with: out of range value for argument "index"

Required Info

Camera Model
LR200

Firmware Version
?

Operating System & Version
OS X El Capitan 10.11.6

librealsense version
1.12.1

pyrealsense version
2.0

hello,
did you managed to use the LR200 with librealsense?
If not, pyrealsense will not be able to connect any better, since it is just wrapping the code from librealsense.
see core.py:start:
    lrs.rs_create_context.restype = ctypes.POINTER(rs_context)
    ctx = lrs.rs_create_context(RS_API_VERSION, ctypes.byref(e))
    _check_error(e)

    n_devices = lrs.rs_get_device_count(ctx, ctypes.byref(e))
    logger.info("There are {} connected RealSense devices.".format(n_devices))

Good point, I will give it a try. Haven't tried yet.

any news ? Or can this be closed ?

Just came back from holidays and now back to the equipment. According to the documentation the librealsense should support LR200. However, I have not managed to test it yet. I try to organise time and report back in a week.
Thanks for following up.

please reopen if needed.


Converted to swift 4 and rxswift 4

Fixes #58

I'm not sure there's a need for a 6.0 for this version... I think it can be a minor patch and not necessarily major

@freak4pc 5.1.0 would be ok?

Codecov Report

Merging #59 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #59   +/-   ##
=======================================
  Coverage   90.26%   90.26%
=======================================
  Files           2        2
  Lines         113      113
=======================================
  Hits          102      102
  Misses         11       11

Codecov Report

Merging #59 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #59   +/-   ##
=======================================
  Coverage   90.26%   90.26%
=======================================
  Files           2        2
  Lines         113      113
=======================================
  Hits          102      102
  Misses         11       11

@esttorhe yes, I'd love to be a part of organisation, thanks!

@AYastrebov I'm going to close this PR in favor of #60
Feel free to follow and comment on it; I'll send the invite in a few.
Thanks


⚠ ClamAV has degraded performance

In 6bfae84, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 93 ms

Resolved: ClamAV performance has improved in 5a24daa.


Capture docs from abstract base classes?

SelectOracle / PrepareOracle / BlockEncoding / ... potentially more bloqs have docs which aren't captured anywhere which could be helpful for developers. Should they be captured?

Have you tried adding a BloqDocSpec without any bloq examples? it will pull in / render the docstrings which I assume is the extent of the documentation you were imagining

This already worked as per #988 and #967


Fixed answers for the question about Twig classes

The answer 'a combination of a and c' makes no sense because there is no a and c, there are answers from 0 to 5.
The answer 'none of the above' is also incorrect because answers are shuffled and 'above'-answer can be the first option.

Hi @AAstakhov,
you're right.
Thank you for your contribution.


Pointing browser to http://localhost:3000/weatherforecast needs clarification for Windows 7 users

Problem encountered on https://dotnet.microsoft.com/learn/aspnet/microservice-tutorial/run-docker
Operating System: windows 7
It will probably be useful to have an additional note on the tutorial about the IP address that the docker machine is using.  I had to run the command:
$ docker-machine ip default
This command revealed the IP address that was actually being used for hosting the example WebAPI, <IP_ADDRESS>.
After knowing that IP address I was able to reach to my example docker-hosted Web API with:
https://<IP_ADDRESS>:3000/weatherforecast
This resource sorted out some of that info for me. Hope it may be useful for others.

@shirhatti this seems like good feedback.

@shirhatti any thoughts on this?

Also pinging @nishanil

Per system requirements stated on Docker website, it requires Windows 10. Ref: https://docs.docker.com/docker-for-windows/install/#system-requirements

So we probably need to update our intro to add the Windows version required in the prereqs, right?
https://dotnet.microsoft.com/learn/aspnet/microservice-tutorial/intro

I am still facing the issue.

Hi @laxmikanth-d, just checking if you are still facing this issue. If yes, could you please share some more details? Thank you.

Support for Windows 7 ended. I'm wondering if we can just add vague note about supported OS @mairaw. Adding and maintaining full list seems excessive.


adding ability to set checked state for menu item at registration

resolves #58422
resolves #56885
resolves #14986
word wrap has a transient state which is what is controlled by the action (not the setting), so I haven't done that one yet.
Can also look into #46807 as well.

@sbatten @jrieken can extensions use this?

not yet


Proposal: Tooltip bindings also on classes

As the headline implies this is regarding the mdl tooltips component. (Tooltips design spec)
Sometimes it would useful to be able to assign the same tooltip to multiple elements. If you had e.g. multiple mdl-card elements where some mdl-card__menu buttons are identical you would not want to give each button a unique id if you could just give them all one class that is associated to the tooltip.
Since the for tag is reserved for associations with id attributes I implemented a MaterialTooltip.prototype.init() function which also resolves a custom attribute for-class . This approach also prevents the degradation of performance of the current implementation relying on the id.
My proposal for material-design-lite/src/tooltip/tooltip.js:MaterialTooltip.prototype.init
/**
 * Initialize element.
 */
MaterialTooltip.prototype.init = function() {
  'use strict';

  if (this.element_) {
    var forElId = this.element_.getAttribute('for');
    this.forElements_ = [];

    if (forElId) {
      this.forElements_.push(document.getElementById(forElId));
    } else {
      var forElClass = this.element_.getAttribute('for-class');
      if (forElClass) {
        this.forElements_ = document.getElementsByClassName(forElClass);
      }
    }

    for (var i = 0; i < this.forElements_.length; i++) {
      this.boundMouseEnterHandler = this.handleMouseEnter_.bind(this);
      this.boundMouseLeaveHandler = this.handleMouseLeave_.bind(this);
      this.forElements_[i].addEventListener('mouseenter', this.boundMouseEnterHandler,
        false);
      this.forElements_[i].addEventListener('click', this.boundMouseEnterHandler,
        false);
      this.forElements_[i].addEventListener('mouseleave', this.boundMouseLeaveHandler);
    }
  }
};

Since I only intended to be a user and not a developer of MDL neither have I set up anything to run the needed tests and create a clean pull request for this, nor have I studied the Google JavaScript Style Guide or signed the CLA yet. As such I am leaving this here for discussion (maybe inclusion by someone that does have his/her environment set up properly) and as a possible remedy for others coming across this limitation.

This seems to break the idea that  one component is an isolated item. The idea of components is they only touch what is within their elements bounds. The suggestion here is to make the tooltip touch the entire document which is a big no-go in my opinion.

Closing; superseded by #4495
We are still in the very early stages of v2 and are currently working towards an alpha. Our master branch is where all v2 work is happening if you are interested in keeping up with its progress. 😄


Importing PDI_C fails from C++ only cmake project

The c_std_99 feature is not available when the C language is not enabled

mentioned in commit e1466d577fc7744be7d8e60c853e82674f9e8e37

mentioned in commit 0aaf5c9dbd997175cfc5b40d853f54b5f3b16126


Ghost images on stage after resize

I have a carousel with items that consist of a  with 2 children, an  and a . The carousel is configured to loop and to show different numbers of items going from 6 down to 2 as the viewport size gets smaller.
In Chrome, if the number of items is reduced, I get funny artefacts: copies of the images showing behind the items. The images have no counterpart in DOM, so it seems to be a repaint issue. If I change one of the CSS styles of the stage in Developer Tools, a repaint happens and the image disappears.
This doesn't happen in IE or Firefox.
Below you see my carousel after resizing. The image of 'Kavi Koyote' is copied on the stage between the 2 items on the right.

While this might be classified as a Webkit bug, I think it's serious enough to consider a workaround in Owl Carousel. Perhaps something after a resize that will force a repaint of the stage.

seems the clones effect appearing behind the original items.

I'm having the same issue, has someone found a workaround ?

I have a rather unelegant workaround for this:
$('#my-carousel').on('resized.owl.carousel', function() {
    var $stage = $(this).find('.owl-stage');
    var c = $stage.css('background-color');
    c = (c == '#fffffe' || c == 'rgb(255, 255, 254)') ? '#ffffff' : '#fffffe';
    $stage.css('background-color', c);
});

Basically, by swapping the background color between 2 indiscernable shades of white, the browser is forced to repaint. I tried other things, like inspecting the inner or outer height, changing the display CSS property to none and back, but they all got optimized away.

I'm also having an identical issue, though it's not just images - the item borders and sometimes text too, often just parts of an item.  SanderKruger's workaround above does not fully fix my problem unfortunately.  Been trying to get somewhere with this for hours but not feeling optimistic now.  This is practically the only place I have found mention of this problem so I'm guessing it does not affect a lot of people.  Wish I knew why

I have run into a similar problem. The only way I got around it was removing [-webkit-backface-visibility: hidden;] under .owl-carousel .owl-item.

Ooh you superstar!  Thanks so much for posting that solution - looking good so far

I have the same problem with artifacts on Chrome. Solution from dhenry-adpear is worked for me. Thank you bro.

.owl-carousel .owl-item {
  backface-visibility: hidden;
  transform: translate3d(0,0,0);
}

(add prefix if you want, I'm using autoprefixer)

Use this to redraw the images on the carousel's onResized event https://coderwall.com/p/ahazha/force-redraw-on-an-element-jquery
onResized: function(){           $(this).find('img').redraw();  }


Conv fails in OSA_Module if input tensor is in Row Major

Describe the bug
In the OSA_Module of VovnetCP module of PETR model, a conv fails with L1 issue if the input to the conv is in Row major layout:
E       RuntimeError: TT_THROW @ ../tt_metal/impl/program/program.cpp:747: tt::exception
E       info:
E       Statically allocated circular buffers in program 31 clash with L1 buffers on core range [(x=0,y=0) - (x=7,y=5)]. L1 buffer allocated at 491520 and static circular buffer region ends at 508416

If the input is converted to Tile layout, conv passes but pcc drops from 0.99 to 0.0627.
To Reproduce
Steps to reproduce the behavior:

Checkout to the branch harini/petr_vovnetcp_osa_conv
Run the command: pytest /home/ubuntu/harini/tt-metal/tests/ttnn/integration_tests/petr/test_vovnetcp.py::test_vovnetcp_osa_module

Expected behavior
Expected to resolve the issue and get 0.99 pcc for the conv.
Please complete the following environment information:

OS: [e.g. Ubuntu 20.04]
Device: WH n150

cc: @punithsekar

@HariniMohan0102 Can you rebase this on origin/main? I'm unable to run this.

Hi @sankarmanoj-tt ,
I have tested the branch after rebasing it to latest main. Resolved the issues that were faced in the convolution.
The above-mentioned  Statically allocated circular buffers issue is occurring in the latest main.
So, We can close this issue.
CC: @saichandax @HariniMohan0102


Changed Microsoft username but old one still appearing

Same to #4340 but with Microsoft account. Is there any method to address it?

@olegoid, do you know how to fix this?

It showes undefined now. But with the same problem.

The two are both wrong.

Closing as a duplicate of #4309.


[QA] Flickering Trees When Facing North

Explorer Alpha build versions:

v0.29.0

Issue Description:
When facing north at coordinates (50, 110), trees flicker as the camera is moved.
LOGS:
Flickering issue.zip
STR:

Launch the explorer.
Use the command /goto 50,110.
Observe the trees while moving the camera.

❌ Actual Result
Trees flicker when the camera is moved while facing north.
https://github.com/user-attachments/assets/32acfe1b-c79c-494f-9f2a-f07d20853f2c
✅ Expected Result
Trees should render smoothly without flickering when the camera is moved.
Additional Notes:
Windows 11. 12th Gen Intel(R) Core(TM) i7-12650H 2.30 GHz. nvidia geforce RTX
Macbook M1 Pro Sonoma 14.5

it would seem to be this scene:

More specifically this asset:


Metamask send token failure

when using the test account from here with metamask, https://github.com/AcalaNetwork/Acala/blob/66b83277f203b1515fac7cdfa2db4da8d8cfb6e0/node/service/src/chain_spec/mandala.rs#L77-L80, sending token will fail due with this error:
request error
    methodName: "eth_sendRawTransaction"
    params: [
      "0x02f87482053980843b9aca008477359400825728940085560b24769dac4ed057f1b2ae40746aa9aab6888ac7230489e8000080c080a0158ba84828b01dfaf67b5a2e17360346316ccb12f4fcebfa459178ce4be41915a05a63dae5dfc9e32618405fd7ef60a2d11e19251e641cfcd517aca14473c09f4d"
    ]
    err: {
      "type": "Error",
      "message": "1010: Invalid Transaction: Transaction is outdated",
      ...
    }

Have you changed the gasPrice?
gasPrice:<PHONE_NUMBER>10000, // storage_limit = 100000, validUntil = 10000, 100000 << 32 | 10000

We can’t really support the default send token feature without asking user to enter some seemingly ridiculous gas price (and tx fee) so it is just a won’t support thing.

does it mean user have to manually enter a super high gas price every time they send transaction through metamask? In this way metamask will block user from sending, saying insufficient fund (due to super high estimation of transaction fee)

@ianhe8x

Yeah so it is just something we cannot support.
The recommended way will be perform token transfer in our (to-be-built) dApp.

close it since it is not a bug then

We can’t really support the default send token feature without asking user to enter some seemingly ridiculous gas price (and tx fee) so it is just a won’t support thing.

So is it an issue only when send native token and won't be same case for contract calls?

The issue exists if the transaction is created by Metamask. If the transaction is created by dApp that understand Acala EVM then it will be all good.

what's the difference? does the dapp just need to suggest a proper gas price?

It should ask for EIP-712 signature instead of the standard Ethereum signature


update to 0.0.6

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/satmad-feedstock#9.

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I've added bot automerge as requested, but nothing actually changed.

@conda-forge-admin, please rerun bot


acknowledgements

Remove the double star line.
Make sure that the Canadian people are still associated and helping with hake.
And, make sure you are OK with what I wrote about Ian.

I'm ok with what's written here
On Mon, Feb 8, 2016 at 4:19 PM, allanhicks<EMAIL_ADDRESS>wrote:

Remove the double star line.
Make sure that the Canadian people are still associated and helping with
hake.
And, make sure you are OK with what I wrote about Ian.
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/102.

sounds good to me
On Mon, Feb 8, 2016 at 4:28 PM, minnowslayer<EMAIL_ADDRESS>wrote:

I'm ok with what's written here
On Mon, Feb 8, 2016 at 4:19 PM, allanhicks<EMAIL_ADDRESS>wrote:

Remove the double star line.
Make sure that the Canadian people are still associated and helping with
hake.
And, make sure you are OK with what I wrote about Ian.
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/102.

—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/102#issuecomment-181639327
.

--
Aaron M. Berger, Ph.D. Research Mathematical Statistician
Fishery Resource Analysis and Monitoring Division
Northwest Fisheries Science Center
2032 SE OSU Dr. Bldg. 955
Newport, OR 97365-5275
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Just edited...


Add conference-call-like features (audio menu, caller pin, dtmf menu)

Important: These commits depend on https://github.com/slomkowski/mumlib/pull/14
These conference-call-like features include:

autodeaf: When mumsi connects to mumble, the user is auto-deafened. When the SIP call is answered, the user is auto-undeafened/unmuted. This allows mumble users to see the mumsi status in the overview, even when they are in a different channel.

dtmf menu: A basic menu structure for DTMF commands is included, which will eventually support things like '*5' for self-mute, etc. For now, it supports the caller pin entry.

caller pin entry: Allows the SIP caller to be forced to enter a pin before being allowed to enter the mumble channel. This helps keep eavesdroppers out!

playAudioFile(): This plays WAV files to either the caller or the mumble channel. The wave files are configurable in the config.ini. Currently, it is used for the welcome message, pin prompt, and the goodbye message when hanging up after wrong pin was entered. It also plays a "caller entering chat" message to make sure other users notice when someone new joins.

@mrscotty  hello hope you are doing good and these are some great effort to make changes. where i can find the master file with all the changes made.
Regards,
salik

Hi Salik,
I'm not sure what you mean by "the master file with all the changes made".
Take a look at the pull request [https://github.com/slomkowski/mumsi/pull/18], which containes a series of the 12 Git commits I made during development.
In README.md, see the sections "Multi-Line Support", "Caller PIN", "Autodeaf" and "Audio Files". The file config.ini.example also contains some example entries.
Best regards,
Scott

On Dec 18, 2017, at 11:57, satti001<EMAIL_ADDRESS>wrote:
@mrscotty hello hope you are doing good and these are some great effort to make changes. where i can find the master file with all the changes made.
Regards,
salik
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Improvement

The circles behind the PageTitles in the site structure are a nice way to encourage users to make the SEO good for a page. However, it is not always clear why a red/yellow circle remains. Even if all the fields are filled and no hints are given, the circles are still not all green.
Proposed solution:

repeat the circles in the 'Meta Data & SEO' tab and use words to explain how the score can get to 100%

Or it is a bug that 100% cannot be achieved. But I can't check what's missing, so either way it would be a nice improvement.

there was a bug on the code thats being sorted on dev master @baukezwaan


Error raised when version without patch compare

Error raised when version without patch compare. Such as
semver.compare("9.0", "9.0")
ValueError: 9.0 is not valid SemVer string

Thanks @wahello for your report. However, semver is true in this case. You need to a major, minor, and a patch number. You provided only the first two. This is not a valid semver version.


Use time.Duration for timeout

Hey! Thanks for the PR. Sorry I don't develop in go for a while so I'd like to ask whether or not it breaks library API?  Will it break existing projects using the library? Do I need to release a new version for this?


Dialog and Alert roles can host keyboard listeners

Add a config change to allow dialog and alert roles to host keyboard interaction handlers.

Coverage remained the same at 99.442% when pulling 011f8d9c1ce5565492925a4529fe1017d2f32079 on jessebeach:dialog-is-kind-of-interactive into 6596435cb1d873c100c7f193f2881ffcaefe96ac on evcohen:master.

Coverage remained the same at 99.442% when pulling 011f8d9c1ce5565492925a4529fe1017d2f32079 on jessebeach:dialog-is-kind-of-interactive into 6596435cb1d873c100c7f193f2881ffcaefe96ac on evcohen:master.

Coverage remained the same at 99.442% when pulling 011f8d9c1ce5565492925a4529fe1017d2f32079 on jessebeach:dialog-is-kind-of-interactive into 6596435cb1d873c100c7f193f2881ffcaefe96ac on evcohen:master.

Coverage remained the same at 99.442% when pulling 011f8d9c1ce5565492925a4529fe1017d2f32079 on jessebeach:dialog-is-kind-of-interactive into 6596435cb1d873c100c7f193f2881ffcaefe96ac on evcohen:master.

Coverage remained the same at 99.442% when pulling 011f8d9c1ce5565492925a4529fe1017d2f32079 on jessebeach:dialog-is-kind-of-interactive into 6596435cb1d873c100c7f193f2881ffcaefe96ac on evcohen:master.


Q: log to string?

For testing purposes, it would be convenient if I could temporarily set something to use a logger that simply writes to a String or StringIO object, so I can check what was logged in my tests. Instead of to a file. Or to the standard stdout/stderr.
Is this possible with Yell, can I create a logger that writes to a StringIO or String instead of to a file?

Yes sure. You can either read the tests for yell in order to see how it is done there, or you can write your own adapter for it. Like so:
require 'yell'

# out write stream
require 'stringio'
$stringio = StringIO.new

# our new adapter that only assigns the global variable to the @stream
class StringAdapter < Yell::Adapters::Io
  open { @stream = $stringio }
end

# do the logging stuff
logger = Yell.new(StringAdapter.new)

Yell::Severities.each do |level|
  logger.send level.downcase, level
end

# inspect your string
puts $stringio.string

You only need to take care to rewind the $stringio after every test. Hope that helps.

thanks!


Deploying to heroku is broken after upgrading to yarn 4

Pushing to heroku fails with this error:
remote: -----> Installing dependencies
remote:        Installing node modules (yarn.lock)
remote:        Unknown Syntax Error: Invalid option name ("--production=false").
remote:
remote:        $ yarn install [--json] [--immutable] [--immutable-cache] [--refresh-lockfile] [--check-cache] [--check-resolutions] [--inline-builds] [--mode #0]

A complicating factor here is that Heroku recommends putting the heroku/nodejs buildpack before heroku/ruby but that's causing problems because our build step in package.json assumes that it can use the bundle command (for some reason that I don't fully understand at the moment) which isn't installed yet if we follow Heroku's recommendation.

A complicating factor here is that Heroku recommends putting the heroku/nodejs buildpack before heroku/ruby but that's causing problems because our build step in package.json assumes that it can use the bundle command (for some reason that I don't fully understand at the moment) which isn't installed yet if we follow Heroku's recommendation.

I can confirm this works as on 25th Jul 2024

Thanks for the confirmation @deepakmahakale. 🤘


change language_detection_threshold type to float

language_detection_threshold can't be supplied as None as detect_language compares it with a float.
Change parameter type in WhisperModel to reflect this.
Added test to confirm this behavior.

There's no use case for None, so the Optional has been removed. Furthermore, supplying None raises a TypeError as the test shows. So, we shouldn't allow None to be supplied for type safety.

I see your point, thanks for noticing
for consistency can you also review other occurrences of this case? these are some examples
https://github.com/SYSTRAN/faster-whisper/blob/8327d8cc647266ed66f6cd878cf97eccface7351/faster_whisper/transcribe.py#L298
also I don't think the test is needed, the number of invalid input combos is too large to have tests for all of them, what do you think?


Use 'StringBuilder.Append(char)' instead of 'StringBuilder.Append(str…

…ing)'
see #792

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

/azp run Steeltoe.All


[DOCFIX] Use correct punctuations in docs/cn/Configuration-Settings.md

#345

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/792/
Test PASSed.


[SQL] Aggregations refactor

Detangling and documenting some code related to aggregations.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.


Make _INITIAL_MMAP_SIZE configurable and/or part of public API

We tend to create a lot of processes leading to lot of metrics files lying around (around 30k pids used in a week).
With each file taking initially 1MB, this leads to 30GB of used space, mainly containing zeroes.
An easy fix was to monkey-patch prometheus_client.core._INITIAL_MMAP_SIZE and set it to 1kB instead.
It works perfectly but apparently the import moved and is now prometheus_client.mmap_dict._INITIAL_MMAP_SIZE.
We'll update it (and add an assert to check for its next move ^^) but it would be cleaner if this was officially configurable.

That's not how the client works, you should have no more than a few MB of vitrual memory used at a time.

That's not how the client works, you should have no more than a few MB of vitrual memory used at a time.

I'm not sure I follow. We are using multiprocess mode which write mmap to files (as far as I understand), and on one of our server that's what I see:
$ ls -sh /tmp/prometheus_metrics/ | wc -l
52446
$ ls -sh /tmp/prometheus_metrics/ | head
total 205M
4.0K counter_10022.db
4.0K counter_10037.db
4.0K counter_10062.db
4.0K counter_10098.db
4.0K counter_1011.db
4.0K counter_10160.db
4.0K counter_10161.db
4.0K counter_10174.db
4.0K counter_10217.db

Ah, you mean disk space rather than memory. I'd expect those files to be sparse, you can check with du.

Ah, you mean disk space rather than memory. I'd expect those files to be sparse, you can check with du.

My bad, you are right, I had only checked with ls:
$ du -h test
4.0K    test
$ ls -lh test
-rw-r--r-- 1 xfernandez users 2.0M Jun 13 15:01 test

So my monkey-patch and this issue were useless, sorry for the noise :-/


Minor code correctness fixes

As I'm currently AFK for a few days, thought I'd put a PR forward with what little I've got so far. The branch naming is a play on my OCD and is meant to consolidate what I've learned from the K5's code structure and apply it to the 890.
While I'm more interested in correctness, I found 1KB of space savings in total with these first few changes. I don't expect my understanding to be flawless but I hope it's a start.

Your doing great! I'm 100% happy with your code, and, thank you for helping the fans of this cool radio.


Getting events for large midi's (black midis) takes FOREVER

I'm a gentledev i:

[x] fully read the README recently
[x] searched for existing issues
[x] checked i'm up to date with the latest version of the project

Expected behavior
It doesn't take 5-15 minutes to load. A reasonable time, like many other apps, is 1 minute. VanBasco's took 2 seconds.
Actual behavior
It takes FOREVER to load.
Steps to reproduce the behavior
Load up a black midi with lots of notes (insane, like 5-10 million) and run midi.getMidiEvents()
Debugging informations

node -v result:

9.0.0

npm -v result:

5.5.1

If the result is lower than 6.9.5, there is
poor chances i even have a look to it. Please,
use the last NodeJS LTS version.

Discovering the concept of Black MIDI.
This project was not created with heavy performances in mind and the result you're getting is not surprising me.
Despite I think supporting that kind of MIDI file could be achieved, I won't do it since it would take a lot of time I currently do not have.
This project was just a PoC, it accidentally begun to be used by people despite not actively maintained. I'm still wondering why tbh ;).
For those unfamiliar with black MIDIs:
https://thump.vice.com/en_us/article/8q74wb/meet-the-15-year-old-prince-of-black-midi

Thanks for introducing me to the "black MIDI" phenomenon. (Summary: So many notes that the sheet music view is mostly black.)
I think we should clarify our README's claim "What it does: • Read MIDI files" by adding "(very slow for huge/"black" files)".
What do you think?

I think it should be noted that when run in an administrator command prompt and manually allowing more ram using the --max_old_memory thing makes it load much, much faster. In fact, I noticed without it the application actually ran out of memory as it used 2.6 GB to load it.

Oh, that's good to know. In that case the slowness most probably wasn't a problem with this module, but just swapping on a slow disk.

I actually have an SSD on this laptop :joy: But either way allowing more RAM fixed the issue. Here's a direct link to one of the Midi's.


Lotto24 adaptions

the initial greeting message is configurable
sort teams by number of the key instead of reverse-ASCII sort
allow to set a TITLE_x env var which is different from the Splunk On-Call team/routing key and use that as spoken name in the menu
couple of other small adaptions

See https://github.com/Lotto24/twilio-live-call-routing/pull/2


Feature/fix differentiable conformance

For the moment, headers, footers (and supplementaryViewModels) are not tested when a section view model is asked if its content is similar to another.
This patch fixes the bug.

Merged into the development branch.


EKS ? yes

works like a charm on 2 clusters i tried.
using Version 0.3.0 (0.3.0) :)
much faster than Lens

How to authenticate with eks? it ask for token, while Lens can do without asking for a token.

The inability to easily use kubeconfig w/EKS AWS_PROFILE is a showstopper for us (unlike Octant, Lens, OpenLens, Aptakube, etc.)

So whats the solution for this ?? how do I connect to the clusters which are listed in my kube config file ?

bump

is there anybody who was able to access EKS cluster using the token generated from aws eks get-token

When the aws CLI adds a cluster to my kubeconfig, it adds an eks get-token command exec to the config.
Headlamp reads the kubeconfig and lists the cluster.
However, when I try to connect then headlamp asks me for a token. This is odd because the command to fetch a token is right there in the kubeconfig. Furthermore, since the eks get-token expires relatively quickly, there's no use in me entering a token manually. (Furthermore, Lens and Octant work as I expect.)

FTR, in the case of @joebowbeer this has to do with running when kubeconfig is set to use the exec plugin for getting the token from aws. We do need to make sure Headlamp is playing well with this.

Hi @joaocc , what version and flavor of Headlamp are you using? (Mac/Linux/Windows)

Desktop-wise I am having issues with macOS and latest headlam.
We are also having some issues on k8s deployment (via helm) with keycloak SSO and multicluster (but it may be keycloak specific, not sure yet)


Doesn't work with dates

I am trying to query by a date range, as follows:
if (dateLogFrom) {
    query.filter(f => f.ge('DateLog', dateLogFrom));
}

if (dateLogTo) {
    query.filter(f => f.le('DateLog', dateLogTo));
}

This does not work, because the resulting filter places single quotes around the dates:
?$filter=DateLog ge '2022-02-01' and DateLog le '2022-02-03'
I then also tried to see what would happen if I passed in a JavaScript date object instead:
if (dateLogFrom) {
    query.filter(f => f.ge('DateLog', Date.parse(dateLogFrom)));
}

if (dateLogTo) {
    query.filter(f => f.le('DateLog', Date.parse(dateLogTo)));
}

That results in the following:
?$filter=DateLog ge<PHONE_NUMBER>000 and DateLog le<PHONE_NUMBER>000
No quotes this time, but obviously OData doesn't understand that at all..
The filter builder needs to either detect Date objects and write them out in the format OData expects.. example:
?$filter=DateLog ge 2022-02-01T00:00:00.00Z and DateLog le 2022-02-03T00:00:00.00Z
or perhaps have some way of detecting that the string passed in is a date and just not enquote it..

Any update on this? I noticed the same issue for GUIDs as well. I get an error like this:
ODataException: A binary operator with incompatible types was detected. Found operand types 'Edm.Guid' and 'Edm.String' for operator kind 'Equal'.

This is because a GUID should not be wrapped in quotes like a string. In order to solve the issue for both dates and GUIDs, I have a suggestion; could you add another parameter to the functions? Example for eq:
query.filter(f => f.eq('SupplierProfileId', supplierProfileId, 'guid'));

If the 3rd arg is not present, then the default behavior of string should be applied (wrap in quotes). If the 3rd arg is guid, then don't wrap in quotes... and for datetime, we could have this:
query.filter(f => f.ge('DateLog', dateLogFrom, 'datetime'));

And that would then parse the value as a date and format the result the way that OData expects.
Thoughts?

Thanks for the suggestion! Hopefully this solves your problem.

It's working great, thanks!


[Snyk] Fix for 13 vulnerabilities

Snyk has created this PR to fix 13 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):

package.json
yarn.lock

Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Server-side Request Forgery (SSRF) SNYK-JS-AXIOS-7361793
  801

Incomplete List of Disallowed Inputs SNYK-JS-BABELTRAVERSE-5962462
  786

Uncontrolled resource consumption SNYK-JS-BRACES-6838727
  696

Improper Input Validation SNYK-JS-FOLLOWREDIRECTS-6141137
  686

Cross-site Request Forgery (CSRF) SNYK-JS-AXIOS-6032459
  676

Information Exposure SNYK-JS-FOLLOWREDIRECTS-6444610
  646

Regular Expression Denial of Service (ReDoS) SNYK-JS-ADOBECSSTOOLS-6096077
  589

Prototype Pollution SNYK-JS-AXIOS-6144788
  589

Inefficient Regular Expression Complexity SNYK-JS-MICROMATCH-6838728
  589

Regular Expression Denial of Service (ReDoS) SNYK-JS-AXIOS-6124857
  586

Uncontrolled Resource Consumption SNYK-JS-GRPCGRPCJS-7242922
  559

Improper Input Validation SNYK-JS-POSTCSS-5926692
  479

Regular Expression Denial of Service (ReDoS) SNYK-JS-ADOBECSSTOOLS-5871286
  464

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)
🦉 Cross-site Request Forgery (CSRF)
🦉 Prototype Pollution
🦉 More lessons are available in Snyk Learn

👷 Deploy Preview for resonant-biscochitos-ee6470 processing.

Name
Link

🔨 Latest commit
f4879c41b9777658b7dfa50af578c40d60d48e5c

🔍 Latest deploy log
https://app.netlify.com/sites/resonant-biscochitos-ee6470/deploys/66daaa23b898dd00085a9256


record screen + audio in a single file

Hi Muaz,
Is it possible to record screen + audio into a single file?

https://github.com/muaz-khan/Chrome-Extensions/tree/master/screen-recording

Using above extension, go to options page and enable the microphone.
Hints:

Capture screen on Chrome or Firefox.
Capture audio using a separate getUserMedia request.
Use screenStream.addTrack(audioStream.getAudioTracks()[0]). Supported both in Chrome and Firefox.
Record (the resulting) screenStream using RecordRTC and it will record both audio+screen in single WebM.

@muaz-khan your answer helped me with this: https://github.com/electron/electron/issues/8589


improve singleton macro error message

before
scala> import shapeless._, syntax.singleton._
import shapeless._
import syntax.singleton._

scala> (true: Boolean).narrow
<console>:20: error: exception during macro expansion:
java.lang.NullPointerException
    at shapeless.SingletonTypeMacros.extractResult(singletons.scala:318)
    at shapeless.SingletonTypeMacros.mkSingletonOps(singletons.scala:393)

       (true: Boolean).narrow
            ^
<console>:20: error: value narrow is not a member of Boolean
       (true: Boolean).narrow
                       ^

after
scala> import shapeless._, syntax.singleton._
import shapeless._
import syntax.singleton._

scala> (true: Boolean).narrow
<console>:23: error: Expression (true: Boolean) does not evaluate to a constant or a stable reference value
       (true: Boolean).narrow
            ^
<console>:23: error: value narrow is not a member of Boolean
       (true: Boolean).narrow
                       ^

[Current coverage][cc-pull] is 75.35%

Merging [#611][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #611   diff @@
==========================================
  Files            68         68
  Lines          3543       3543
  Methods        2899       2900     +1
  Messages          0          0
  Branches        121        120     -1
==========================================
  Hits           2670       2670
  Misses          873        873
  Partials          0          0

Powered by Codecov. Last updated by [3a02b7c...5912487][cc-compare]
[cc-base-branch]: https://codecov.io/gh/milessabin/shapeless/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/milessabin/shapeless/compare/3a02b7cb3f645077ad16caecbd07a74ff01813cf...5912487f98e03a490ab6006b5274929969f5a4b1
[cc-pull]: https://codecov.io/gh/milessabin/shapeless/pull/611?src=pr

LGTM! Many thanks ... merging :-)

Hmm ... actually, before I do merge this ...
It should be possible to strip off the outer type ascription and see the underlying term ... would you like to have a try at that?

Am I dreaming, or has this PR (and some others) mysteriously come back from the dead?

I'm not sure how this slipped through the cracks ... many thanks, merging :-)

cherry-picked as 3735f547ec7652087b41c119b690e3e480d38be4.


Updated text to reflect the role of MO, Inc.

Also included the articles of organization, bylaws, and determination letter for MO, Inc. off a separate support/governance page that is liked to from the intro and donation pages (as well as the current version of the baner).

Coverage increased (+0.03%) to 86.281% when pulling 7a737f1743e37453b84a102b6e2615b5cff8a0e1 on governance into 3d636cd23adf0d87b3f68517c549bbea49b5609c on master.

Coverage increased (+0.03%) to 86.283% when pulling 9b559258f88fb7af0bdff461638605f8a2a5f388 on governance into 3d636cd23adf0d87b3f68517c549bbea49b5609c on master.

Coverage increased (+0.03%) to 86.283% when pulling 9b559258f88fb7af0bdff461638605f8a2a5f388 on governance into 3d636cd23adf0d87b3f68517c549bbea49b5609c on master.


changinge "add" to "multiply"

Changing the 213rd line code from “merged = layers.add([encoded_sentence, encoded_question])” to "merged = layers.multiply([encoded_sentence, encoded_question])"  can improve the accuracy of the model.
Summary
Related Issues
PR Overview

[ ] This PR requires new unit tests [y/n] (make sure tests are included)
[ ] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[ ] This PR is backwards compatible [y/n]
[ ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Thanks for the PR. Can you please change the header of this file according to the accuracy increase?

I don't think the numbers are convincing enough to justify a modification of this file. I would suggest making a separate GitHub repository for this.


Documentation issue for pipe method

PR #132 created a problem for generating documentation.
@RebeccaStevens You changed pipe signature from function to const, and typedoc cannot pick it up correctly anymore, the pipe is altogether skipped from documentation, which in turn results in unresolved links.
You can check for errors by running npm run doc.

Also, I noticed that for some methods, you copied inline documentation for sync + async. I'm not sure I see the benefit of it, duplicating documentation. Plus that extra category Async, in addition to already sufficient Async-Only.
I think it is simpler to keep just one documentation block per operator, and leave as Sync+Async category by default.

This has been resolved, closing now.


Blank lines in View code

I replace VIEW in Teradata with code:
REPLACE VIEW BASE.V_TEST
AS
SELECT
    1       AS ID,
    'TEST'  AS NAME

When execute command SHOW VIEW BASE. V_TEST, get view text with blank lines:
REPLACE VIEW SNAPSHOT.V_TEST_KSS

AS

SELECT

    1       AS ID,

    'TEST'  AS NAME

Why  empty lines in code are present?

I can't say (seems to be some Taradata-specific glitch) but it doesn't look like a DBeaver issue.

Do you see extra line feeds in other tools (for views created in DBeaver)?
This may happen if your view source contains "broken" line feeds like \n\r  (this bug will be fixed in the next version).
Are you typing view source manually or copy-paste it from some external program? If copy-paste then could you check what line feeds it has?
In any case - please check the fix in 5.2.1 or in EA (https://dbeaver.io/files/ea)


NameError: name 'logging' is not defined

With cadquery patched with cq_warehouse.extensions, if cq_warehouse.extensions is not imported, a "NameError: name 'logging' is not defined" error occurs.
Quick fix is to import cq_warehouse.extensions into your model.

Fixed by mocking logging:
class logging:
    def debug(self):
        pass
    def warn(self):
        pass

in the patched version of occ_impl/shapes.py - the only one with logging.


Clean file_sources

file_sources keeps growing.
Clean unused file_sources elements after memory_cleanup.

https://www.youtube.com/watch?v=-ilXc0j1iHA

fixed

fixed

Now remote.id stays the same.

Now remote.id stays the same.

good

good


Make copied images public-read

To test, import from s3 (eg. s3://sample-tiffs/356f564e3a0dc9d15553c17cf4583f21-7.tif) and a local file and check that the copied files are publicly accessible in the AWS console.
Connects #266

New commit makes bucket publicly readable.

:clap: Verified this works for both uploaded files and S3 copied files.

+1


Type-Hint Backward Compatibility

The library should now work with 3.6+.
TypedDict and Final are both common "new" types (3.8+).

Don't you need to add the requirement for that library?

Don't you need to add the requirement for that library?

It's a built-in library like typing

I think that's only true for later python versions:
dschultz@tide3:~$ podman run --rm -it python:3.6
Python 3.6.15 (default, Sep  8 2021, 02:11:58)
[GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import typing_extensions
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'typing_extensions'

I spoke too soon, should be good now.

Actually, I'm not sure typing_extensions is built in at all.  It just happens to be a dependency of something else in most cases.

right, it actually wouldn't make sense to have it as a built-in. If it were, then the python developers should've just put everything into typing


Add Ops/cancel role on queue

Add role to Armada queues to allow users in this role to cancel jobs in a queue, but not to submit them.

https://github.com/G-Research/armada/pull/840 fixes this


Fix build_resource when model name is Action

Fix #125

This PR also fixes the issue for me, although I get a different error message than the one in the original issue. The error message I get is When assigning attributes, you must pass a hash as an argument.. In any case, this PR fixes it 😄


Fix on Extended math extension

Fix review from https://github.com/4ian/GDevelop-extensions/pull/57

I took 4ians advice and placed each constant in its own expression.
ExtendedMathSupport.zip

Thanks! Should we rename each expression so that it is in PascaleCase (Pi, etc...)?
For consistency with what we do for GDevelop expressions.

I'm like 99% sure that everything is in PacaleCase now
ExtendedMathSupport.zip

Your zip don't use the lastest content from this PR.

Oh your right I forgot to swap by branch from master to fix extended math. I fixed some minor spelling errors.
ExtendedMathSupport.zip

Thank you everyone, the version in this PR is same that's yours.
I merge the fix, all will be available directly in the extension list on GDevelop!
No download required ;)


chore: fix pre-commit config and adapt code to make all checks pass

PR Type

Improvement

Description
This PR is a preliminary step to

#161

and makes pre-commit pass on the current codebase.
This involved:

fixing the pre-commit configuration (and that of its hooks):

revert prettier version from 4.0.0-alpha.8 to 3.1.0
disable prettier on yaml and json files (as they are handled by yamllint and clang-format respectively)
add clang-format config for json
add exception in yamllint for .clang-format as its format is required to be non-standard

implementing the code changes flagged by pre-commit hooks

whitespace changes
import changes
copyright notices in cpp/hpp source files
quoting changes in yaml files
added/removed qualifiers like explicit, override, etc.
...

pre-commit is now passing.
Review Procedure
Make sure to check "hide whitespace" and click apply/reload in the "changed files" view's settings when reviewing:

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[ ] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets

CI Checks

Build and test for PR: Required to pass before the merge.

Codecov Report
Attention: Patch coverage is 5.17241% with 55 lines in your changes missing coverage. Please review.

Please upload report for BASE (develop@46171bf). Learn more about missing BASE report.

Files
Patch %
Lines

nebula_ros/src/robosense/decoder_wrapper.cpp
0.00%
32 Missing :warning:

...src/nebula_decoders_robosense/robosense_driver.cpp
0.00%
4 Missing :warning:

nebula_ros/src/hesai/hesai_ros_wrapper.cpp
0.00%
3 Missing :warning:

...s/nebula_decoders_robosense/decoders/bpearl_v3.hpp
0.00%
2 Missing :warning:

...s/nebula_decoders_robosense/decoders/bpearl_v4.hpp
0.00%
2 Missing :warning:

...ders/nebula_decoders_robosense/decoders/helios.hpp
0.00%
2 Missing :warning:

...a_decoders_robosense/decoders/robosense_packet.hpp
0.00%
2 Missing :warning:

nebula_ros/src/robosense/robosense_ros_wrapper.cpp
0.00%
2 Missing :warning:

nebula_ros/src/velodyne/velodyne_ros_wrapper.cpp
0.00%
2 Missing :warning:

nebula_common/src/nebula_common.cpp
0.00%
1 Missing :warning:

... and 3 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             develop     #162   +/-   ##
==========================================
  Coverage           ?   14.13%
==========================================
  Files              ?       82
  Lines              ?     7585
  Branches           ?     1179
==========================================
  Hits               ?     1072
  Misses             ?     5679
  Partials           ?      834

Flag
Coverage Δ

differential
14.13% <5.17%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


BREAKING CHANGE: Release V7

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

GitHub release
v3.0.0

Your semantic-release bot :package::rocket:


DoubleEntry::UnknownAccount: account: spending scoped?: true

On a Rails 5.1.4 app installing double_entry using Postgresql, with the following initializer:
require 'double_entry'

DoubleEntry.configure do |config|
  config.define_accounts do |accounts|
    user_scope = accounts.active_record_scope_identifier(User)
    accounts.define(:identifier => :savings,  :scope_identifier => user_scope, :positive_only => true)
    accounts.define(:identifier => :checking, :scope_identifier => user_scope)
  end

  config.define_transfers do |transfers|
    transfers.define(:from => :checking, :to => :savings,  :code => :deposit)
    transfers.define(:from => :savings,  :to => :checking, :code => :withdraw)
  end
end

and unchanged migrations (beyond specifying ActiveRecord version 5.1).
I get the following error in console:
=> account = DoubleEntry.account(:spending, :scope => User.first)
  User Load (0.3ms)  SELECT  "users".* FROM "users" ORDER BY "users"."id" ASC LIMIT $1  [["LIMIT", 1]]
DoubleEntry::UnknownAccount: account: spending scoped?: true

Is this possibly related to #115?  I've tried every combination I can think of for passing in a scope here and it's all failing.
My app already has an Account and Transfer model defined, but surely this shouldn't be of issue given double_entry's module namespacing?  Any help much appreciated.

Hi @KelseyDH,
It looks like you haven't defined a 'spending' account in your configuration block.
Try something like this:
accounts.define(:identifier => :spending,  :scope_identifier => user_scope)

@orien Whoops, thanks for noticing that mistake.  However, I've tried many combinations here, including with properly declared accounts and I still get the same error.
E.g.:
With a configuration block correctly loading a savings entry:
require 'double_entry'

DoubleEntry.configure do |config|
  config.define_accounts do |accounts|
    user_scope = accounts.active_record_scope_identifier(User)
    accounts.define(:identifier => :savings,  :scope_identifier => user_scope)
    accounts.define(:identifier => :checking, :scope_identifier => user_scope)
  end

  config.define_transfers do |transfers|
    transfers.define(:from => :checking, :to => :savings,  :code => :deposit)
    transfers.define(:from => :savings,  :to => :checking, :code => :withdraw)
  end
end

calling it still fails:
 account = DoubleEntry.account(:savings, :scope => User.first)
  User Load (0.7ms)  SELECT  "users".* FROM "users" ORDER BY "users"."id" ASC LIMIT $1  [["LIMIT", 1]]
DoubleEntry::UnknownAccount: account: savings scoped?: true
from /Users/username/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/double_entry-1.0.1/lib/double_entry/account.rb:44:in `find'

Does
account = DoubleEntry.account(:savings, :scope => User.first.id.to_s)
Work?
On Wed, 21 Feb 2018 at 21:01, Kelsey Hannan<EMAIL_ADDRESS>wrote:

@orien https://github.com/orien Whoops, thanks for noticing that
mistake. However, I've tried many combinations here, including with
properly declared accounts and I still get the same error.
E.g.:
With a configuration block correctly loading a savings entry:
require 'double_entry'
DoubleEntry.configure do |config|
config.define_accounts do |accounts|
user_scope = accounts.active_record_scope_
identifier(User)
accounts.define(:identifier => :savings,  :scope_identifier => user_scope)
accounts.define(:identifier => :checking, :scope_identifier => user_scope)
end
config.define_transfers do |transfers|
transfers.define(:from => :checking, :to => :savings,  :code => :deposit)
transfers.define(:from => :savings,  :to => :checking, :code => :withdraw)
end
end
calling it still fails:
account = DoubleEntry.account(:savings, :scope => User.first)
User Load (0.7ms)  SELECT  "users".* FROM "users" ORDER BY "users"."id" ASC LIMIT $1  [["LIMIT", 1]]
DoubleEntry::UnknownAccount: account: savings scoped?: true
from /Users/username/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/double_entry-1.0.1/lib/double_entry/account.rb:44:in `find'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/envato/double_entry/issues/142#issuecomment-367242940,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAuo0_dHZQGtLi6ISznfc9x5iU6O2P9ks5tW81ogaJpZM4SGf0x
.

@eadz   Nope, doesn't work.
account = DoubleEntry.account(:savings, :scope => User.first.id.to_s)
  User Load (0.5ms)  SELECT  "users".* FROM "users" ORDER BY "users"."id" ASC LIMIT $1  [["LIMIT", 1]]
DoubleEntry::UnknownAccount: account: savings scoped?: true
from /Users/username/.rbenv/versions/2.4.2/lib/ruby/gems/2.4.0/gems/double_entry-1.0.1/lib/double_entry/account.rb:44:in `find'

I got it working on a fresh install of Rails 5.1.5 with devise.  So I suspect there must be something in my project that's overriding the user scope.
Of note, I already have ActiveRecord Account and Transfer models, and on the user side, I'm using devise_token_auth and papertrail which also do their own things to activerecord internals.
No idea where to start on debugging this, however.

Interesting to note:  after I added a plain Transfer and Account model to my sample rails 5.1 double_entry installation, I got errors on this same command, but in a different way.
E.g.:
irb(main):012:0> account = DoubleEntry.account(:savings, :scope => User.last)
  User Load (0.6ms)  SELECT  "users".* FROM "users" ORDER BY "users"."id" DESC LIMIT $1  [["LIMIT", 1]]
DoubleEntry::AccountScopeMismatchError: Expected instance of `User`, received instance of `User`
	from (irb):12
irb(main):013:0> account = DoubleEntry.account(:savings, :scope => User.last.id.to_s)
  User Load (0.6ms)  SELECT  "users".* FROM "users" ORDER BY "users"."id" DESC LIMIT $1  [["LIMIT", 1]]
=> #{Account account: savings scope: 2 currency: USD}

Whereby after I followed your debugging tips with the string conversion, it worked.  So basically this gem breaks by default if there is an Account or Transfer model already present, and in addition to that breaks in other places in ways I'm struggling to understand.
On the primary app I'm using I've commented out my User, Account and Transfer models... and even tried to switch the main scope over to a model to something else more neutral, and it still fails.
Is there a way we can pull this scoping out of double_entry?

Edit: Whoops.  Solved the original issue:  Turns out my double_entry.rb initializer was in config/ not config/initializer
I'm no longer getting DoubleEntry::UnknownAccount anymore, which was obviously just triggering because my initializer wasn't getting picked up.
Instead I'm now just getting variations of DoubleEntry::AccountScopeMismatchError instead, which I believe relates to #115 where I post the details, and close this issue.


resolver can't deal effectively with old Ruby versions

[Apologies if this is a duplicate, but I heard that as of bundler 1.6 all resolver bugs are supposed to be fixed, and when I searched the issues for resolver it found 151 closed and 2 open, neither of which relate to this report.]
If I install Ruby 1.8.7 via rvm, and then create a fresh Gemfile:
source 'https://rubygems.org'
gem 'guard'

then I get the following:
$ bundle
Fetching gem metadata from https://rubygems.org/...........
Fetching additional metadata from https://rubygems.org/..
Resolving dependencies...
Using timers 1.1.0

Gem::InstallError: celluloid requires Ruby version >= 1.9.2.
An error occurred while installing celluloid (0.15.2), and Bundler cannot continue.
Make sure that `gem install celluloid -v '0.15.2'` succeeds before bundling.

AFAICS this is because guard requires listen, and newer versions of listen require celluloid which requires Ruby >= 1.9.2, but unfortunately bundler's output does not make this at all obvious.  The only way I have found to debug this is to re-run bundler on a newer Ruby and then manually inspect the resulting Gemfile.lock, and even then presumably there is no guarantee that a Gemfile.lock generated on a newer Ruby can accurately represent the incomplete dependency tree which caused the above error on 1.8.7.
Assuming my analysis above is correct, why did bundler even mention celluloid?  Shouldn't it have noticed that listen requires 1.9.3 even before trying to solve listen's gem dependencies?  So at worst I would have expected it to output:
Gem::InstallError: listen requires Ruby version >= 1.9.3.
An error occurred while installing listen (2.7.1), and Bundler cannot continue.
Make sure that `gem install listen -v '2.7.1'` succeeds before bundling.

and by not prematurely iterating further down the dependency tree, it would have been easier to debug.
At best, there would be some way of getting bundler to dump the partial dependency tree it was using at the point the error was generated.  For example, it could offer an option to dump Gemfile.lock.broken or similar.
However, whilst it would be great if bundler made it easier to debug this kind of scenario, that's not the main issue I want to report here (let me know if you would prefer a separate issue for that).
The main issue is that bundler's resolver can't figure out for itself that an older version of listen would be able to satisfy the dependencies.  If I add the following hint to my Gemfile:
gem 'listen', '< 2.0'

then bundler successfully resolves all dependencies:
Fetching gem metadata from https://rubygems.org/............
Fetching additional metadata from https://rubygems.org/..
Resolving dependencies...
Installing ffi 1.9.3
Installing rb-fsevent 0.9.4
Installing rb-inotify 0.9.3
Installing rb-kqueue 0.2.2
Installing listen 1.3.1
Using bundler 1.6.1
Your bundle is complete!
Use `bundle show [gemname]` to see where a bundled gem is installed.

but it really should not need the hint, because it already has the necessary metadata available to figure this out.
BTW this is not an isolated case.  Another example is attempting to install chefspec.  Again, even though chefspec's metadata clearly states Ruby >= 1.9 is required, bundler attempts to install it, only failing when it reaches the fauxhai leaf in the dependency tree.

Hi there, three years later I'm still having those issues.
I'm stuck with a broken build and the unability to know which gem is dependent on the failing one in this process.
How should I proceed ? bundle install --verbose is not helping, bundling on a newer platform doesn't pull the specific gem in my dependency tree, I'm basically stuck.

@Jerskouille please open a new issue with the information requested in https://github.com/bundler/bundler/blob/master/doc/contributing/ISSUES.md


Update selectNodeVersion.js to use WEBSITE_NPM_DEFAULT_VERSION. Fixes…

… #1772
This matched this commit 7b32c189b50a26ab4289f7052c19bd12bfd11e6d

I wonder if we can get to a point where we don't need this selectNodeVersion.js script and deploy.cmd simply relies on whatever is in the path.   The path will be determined at only one code path say in NodeJSBuilder.

@suwatch doing this in NodeJSBuilder is not possible, as that would break any custom deployment script. It needs to either happen for all deployments, regardless of Node (may be tricky), or needs to exist inside the script. BTW, note that despite its name, selectNodeVersion also generates the web.config if there isn't one.


Convert MSTest tests to NUnit tests

There was not enough time to run all tests. I'll try to optimize test running to speed up the process.

@xplicit I ran the test from your branch locally and in my VS the NUnit tests take the same amount of time to run as MSTests before. We were already close to the limit on AppVeyor so maybe it was just a fluke. I kicked off another build on the PR.

@xplicit On AppVeyor the NUnit tests run significantly slower (comparing the longest running tests in last successful build and PR build it seems to be 4x slower).

@sapek I have a couple of ideas how to speed up NUnit runs, I want to try them. When I get the results, I'll write them here

@xplicit I was wondering that maybe NUnit test runner tries to execute tests from all assemblies that are passed on the command line concurrently and on CI VM which has only one virtual processor this end up taking more time than executing them serially? Maybe try call the runner for each test assembly on it own?

@sapek I tried several options (by the way did not try to run files one by one), but could not isolate issue and find workaround yet :( They should run the tests one by one (at least at my local machine nunit-console works this way).  That's a very strange issue for me, because I have different behaviour of test runs for nunit-console on my machine and appveyor machine. I am still trying to find the solution.

@xplicit Another possibility to test is that the problem is in the custom AppVeyor logger. Try running it w/o it. We would loose tests reporting but that is not such a big deal because the result is still in the console output.

@xplicit Separately from working with @CharliePoole on figuring out why nunit-console is so slow running Bond tests, would it be an option to run the tests in appveyor using nulite? Or is it not available as an executable anywhere?

It's possible to run nunitlite with tests, the package is available on nuget. It's easy to use it, what is needed just to add new console project and reference NUnitLite nuget package.
One thing, that current version of nunitlite on nuget has two limitations compared to nunit-console:

NUnitLite can run only one test dll at once, so it's needed to split the test command on the several commands.
NUnitLite loads assembly by their assembly name, not by filename. This requires one additional step of coping all testing assemblies (or nunitlite exe file with referenced assemblies) to one folder and run it from there.

The last thing is already fixed in master branch of NUnit, but recent NUnitLite package has not been uploaded to nuget yet. I wait new version of NUnitLite on nuget (hope to see it soon) or resolving the issue with long running of nunit-console - but the last one is not an easy thing (even to detect what is the cause of the issue).
New version of NUnitLite would protect the test script from the noise like copy path/to/test/dlls somefolder commands, but it's possible even now to create the test script with friendly output of the test results to the console and creating tests artifacts on appveyor.

Specifying each test assembly is not a big deal. However having to add a command line tool for NUnitLite is a bigger problem, that pollutes the repository, not just AppVeyor config. @CharliePoole, could NUnit ship with a console app for NUnitLite?

@sapek The concept of NUnitLite is that your test assembly is a console app and references both nunit.framework and nunitlite. Your Main calls the text-based UI in nunitlite and nunitlite executes the tests in the calling assembly. There's no "runner" in the sense you mean.
If you use nuget to add nunitlite to the project, an appropriate main will be added. If not, it's easy enough to create manually following the info in the readme.
The notion of loading an assembly by name is a remnant of an older version of nunitlite. In the current code, it also works by file name. However, we haven't yet decided whether to make nunitlite a full-fledged runner that loads files and runs tests or to leave it as just a library used by self-executing tests, which is what it is right now.
[Note that NUnitLite runs on platforms where there is no such a thing as a console. I've tailored my explanation here to running under desktop .NET or mono.]

@sapek I have a look into NUnitLite once more and can say that it's possible to get an exe file without creating additional projects in a bond solution.  It needs to change appveyor script file only. This could be used as a temporary solution while nunit-console will be fixed for slow execution on appveyor.

I consolidated the unit tests in @40f3888 which should give us a little breathing room in appveyor CI (after switching to open source plan build was already timing out if the cached items expired and had to be rebuilt). If you can use the nunitlite w/o adding a project that would work.

I have found a resolution for long-running tests in nunit-console on appveyor, now I'm checking the build. If all is OK, I'll make new PR

@sapek Here are new PR https://github.com/Microsoft/bond/pull/34
NUnit tests now have the similar execution time as MSTest.

Looking forward to it!
On Wed, Feb 11, 2015 at 1:47 PM, Sergey Zhukov<EMAIL_ADDRESS>wrote:

I have found a resolution for long-running tests in nunit-console on
appveyor, now I'm checking the build. If all is OK, I'll make new PR
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/bond/pull/23#issuecomment-73971803.

When I tried to compile nunitlite.exe with csc, I've noticed that compiled with csc version executes slow as nunit-console on appveyor.  What was interested  that msbuild - compiled works much faster on appveyor. Investigating the command line I've found a parameter which increases performance a lot. It was /platform:anycpu32bitpreferred. I tried to switch nunit-console in bond project to nunit-console-x86 (or nunit-console.exe --x86 for version 3.0) and it worked with near the same speed as nunitlite on appveyor.

Have any of the previously reported results been from versions of NUnit you
built yourself? That's a potential difference we should probably explore.
On Wed, Feb 11, 2015 at 3:00 PM, Sergey Zhukov<EMAIL_ADDRESS>wrote:

When I tried to compile nunitlite.exe with csc, I've noticed that compiled
with csc version executes slow as nunit-console on appveyor. What was
interested that msbuild - compiled works much faster on appveyor.
Investigating the command line I've found a parameter which increases
performance a lot. It was /platform:anycpu32bitpreferred. I tried to
switch nunit-console in bond project to nunit-console-x86 (or nunit-console.exe
--x86 for version 3.0) and it worked with near the same speed as
nunitlite on appveyor.
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/bond/pull/23#issuecomment-73983867.

Merged these changes in #35. Thanks.


UI broken on Android since upgrade to Capacitor 4

Since upgrade to Capacitor 4 and version 3 of this plugin, the previous custom UI does not show up again on Android.  The complete WebView is hidden (not even accessible through the Chrome debugger).  Besides, the scanner now shows some information about the scanned code already while scanning.
On iOS, the behaviour is the same as before the upgrade.
Tested on Galaxy Tab A8 with Android 11

Unfortunately I am not able to reproduce this issue...
Can you share a repo where I can reproduce your problem?
Without an implementation to debug it won't be possible for me to help you....

As for me, splashscreen (probably the new Android 12 one) is showing between the scanner UI and the camera view.

You should be able to remove the splashscreen with some ionic plugins, have you already tested this?

Please check out this: https://github.com/capacitor-community/barcode-scanner#ionic-css-variables
If you still have issues, feel free to reopen this issue :)

I couldn't remove the new splashscreen with the splashscreen capacitor plugin. But I was able to revert to the old splashscreen by edtiting the android/app/src/main/res/values/styles.xml file, doing the contrary described here : https://capacitorjs.com/docs/updating/4-0#optional-use-the-new-android-12-splash-screen-api


Make example sketch filename match folder name

The Arduino IDE requires the sketch folder name to match the filename of the primary sketch file. This change causes the example sketch to be accessible via the Arduino IDE's File > Examples > LIBRARYNAME menu after the library is installed.
Reference:
https://github.com/arduino/Arduino/wiki/Arduino-IDE-1.5:-Library-specification#library-examples

So what you are saying is, for example, the sequences folder should not really exist, because there should be a folder for each example sketch, right?

Each sketch must be in its own folder. The name of that folder must match the file name of the primary .ino file exactly:
examples
|-- DCmotor
|   `-- DCmotor.ino
|-- blink
|   `-- blink.ino
|-- blinkAll
|   `-- blinkAll.ino
`-- trafficLight
    `-- trafficLight.ino

You are welcome to organize the examples in arbitrarily named subfolders if you like, which will cause the File > Examples > Sparkie menu in the Arduino IDE to follow the same structure. For example:
examples
|-- DCmotor
|   `-- DCmotor.ino
|-- blink
|   `-- blink.ino
|-- blinkAll
|   `-- blinkAll.ino
`-- sequences
    |-- LarsonScanner
    |   `-- LarsonScanner.ino
    `-- trafficLight
        `-- trafficLight.ino

Oh I see. Thank you so much!
Adding this change for future release

@EddyBer16 why did you close this pull request?
If you agree with the improvement I have proposed here, then you should reopen and merge the pull request. I'm trying to make a contribution to your library in the open source spirit but its ending up to be quite a lot of work for me.
For more information, see:
https://help.github.com/articles/merging-a-pull-request/

Omg that's true. Totally my bad.
I reviewed the file changes and I didn't notice anything until I checked the new file paths.
Again, thank you so much for your contribution and patience!


make_tracks on asap7 generates an error x_offset -270 not >= 0

Describe the bug
running make_tracks on asap7 generates [ERROR IFP-0039] x_offset -270 not >= 0
Expected Behavior
Either improve the error message to use correct units and indicate the offending metal layer or correctly generate the routing tracks.
Environment
OpenROAD v2.0-16896-g1531ad983
Features included (+) or not (-): +Charts +GPU +GUI +Python

To Reproduce
make_tracks.tar.gz
tar xvf make_tracks.tar.gz
cd off-track
openroad run.tcl

Relevant log output
[INFO IFP-0001] Added 5999 rows of 29999 site asap7sc7p5t.
[ERROR IFP-0039] x_offset -270 not >= 0
Error: run.tcl, 9 IFP-0039

Screenshots
No response
Additional Context
No response

correctly generate the routing tracks.

I don't think we want tracks outside the die area

@maliberty I think you could create the tracks, but skip the ones that would fall outside the diearea. I don't feel strongly about it, but I would prefer to error message to be in um and reference the metal layer.


meta: move GetPaths to baseMeta

Codecov Report

Merging #2711 (9de8730) into main (cdccfe8) will decrease coverage by 0.12%.
The diff coverage is 68.71%.

@@            Coverage Diff             @@
##             main    #2711      +/-   ##
==========================================
- Coverage   59.58%   59.46%   -0.13%
==========================================
  Files         138      138
  Lines       23626    23725      +99
==========================================
+ Hits        14077    14107      +30
- Misses       7716     7779      +63
- Partials     1833     1839       +6

Impacted Files
Coverage Δ

cmd/fsck.go
66.07% <0.00%> (ø)

pkg/meta/interface.go
45.45% <ø> (+0.06%)
:arrow_up:

pkg/meta/base.go
65.37% <49.05%> (-1.59%)
:arrow_down:

pkg/chunk/cached_store.go
73.80% <50.00%> (-0.42%)
:arrow_down:

pkg/utils/logger.go
63.93% <66.66%> (+1.67%)
:arrow_up:

pkg/chunk/disk_cache.go
73.99% <79.76%> (-4.22%)
:arrow_down:

cmd/flags.go
93.96% <100.00%> (+0.15%)
:arrow_up:

cmd/mount.go
60.49% <100.00%> (+0.12%)
:arrow_up:

pkg/utils/logger_syslog.go
42.30% <100.00%> (ø)

pkg/vfs/backup.go
60.67% <100.00%> (-0.44%)
:arrow_down:

... and 9 more

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


Only able to open OpenRCT2 title sequences

I have both RCT1 and RCT2 assets available:

And the title sequences are available:

And I can confirm that when I select, for example, RCT1 for my title sequences it runs through those.
And in the "Title Sequences" menu I can see all available options:

And in the add/load menu the ORCT2 parks are available:

But none of the RCT1/2 sequence maps show up:

Is there perhaps a certain requirement I'm not meeting? Or is this a technical limitation with how ORCT2 formats and ingests the maps?

I don't think the original sequences could ever be edited because they are special cases which reference the original parks but don't actually include them in the .titleseq.


[envrc] run devbox install to ensure createWrappers is called

Summary
Our .envrc today calls shellenv but that doesn't run createWrappers.
If the package's binary may change, then the .envrc environment will not get updated.
How was it tested?
in examples/flakes/php, I did:

devbox generate direnv
php -m | grep ds which prints ds.
edit examples/flakes/php/my-php-flake/flake.nix to remove ds extension
re-run php -m | grep ds. This still prints ds.

Now, for the test steps:

cd out of the direnv
re-enter the direnv-enabled directory (examples/flakes/php)
re-run php -m | grep ds.
BEFORE: This still prints ds.
AFTER: no ds is printed.

Note, in Step 4, ideally we would not print ds but for that the direnv would need
to check local flake files for changes. Today, it just checks devbox.json for changes.
Not addressing in this PR (workaround: cd out and back in to the directory)

Current dependencies on/for this PR:

main

PR #1092   👈

This comment was auto-generated by Graphite.

@mikeland86 yeah, that makes sense to me. I agree.
I think its okay to land this since its in the code we generate, and not user-visible .envrc file.


Remove retired repos

widdershins is still being used in govuk-content-api-docs.  Could you add it back into the list of repos?

I don't think we should do that — it's still an unmaintained fork so it ought to stay archived unless you're willing to become its maintainer and keep it up to date.
There's no immediate problem with depending on an archived repo — it just makes it clearer that you have a (pre-existing) problem to address sometime in the not too distant future.

Otherwise we're just sweeping serious security risks under the rug.

Otherwise we're just sweeping potentially serious security risks under the rug.

Surely we're doing that by archiving the repo and therefore hiding the security alerts associated with it?

Not really, we're making it clear that people are depending on unmaintained and insecure software, which is a problem that you already have right now.
You're free to become a maintainer of the fork if you wish.


Add virtletctl gen

This PR includes a commit from #645 which needs to be merged first.

[x] bump k8s libs to 1.9 so MountPropagation field can be set in k8s objects
[x] embed virtlet-ds.yaml using bindata
[x] check bindata consistency during build/cmd.sh build
[x] implement virtletctl gen command
[x] update build/cmd.sh start-dind to use virtletctl gen
[x] embed image tag in virtletctl when doing release builds on CircleCI
[x] update demo.sh to use virtletctl from mirantis/virtlet image (pass selected image tag to it using --tag)
[x] remove jq from the build container
[x] fix style issues
[x] split the part that updates k8s libs into a separate PR
[x] update docs
[x] verify build/cmd.sh release which no longer includes yamls
[x] verify raw_devices setting in virtlet config
[x] verify VIRTLET_DISABLE_KVM=1 build/cmd.sh start-dind

This change is

Review status: 43 of 44 files reviewed at latest revision, 1 unresolved discussion.

docs/networking.md, line 105 at r2 (raw file):
Previously, jellonek (Piotr Skamruk) wrote…
There was a typo - missing space after setting. So that should be "setting `sriov_...".

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Linux Command to compare text files

share some commands which can be used to compare to text files in linux.
give examples so its understandable.


Android app crashes on startup

Hello!
As soon as I add this plugin and try to run the app with both debug and release mode (tried in real device and simulator) it just crashes instantly. I've added this to the AndroidManifestaswell:
<meta-data
  android:name="com.google.android.gms.ads.APPLICATION_ID"
  android:value="ca-app-pub-3940256099942544~3347511713"/>

Flutter version = 1.6.3 (currently latest beta version)
Dart version = 2.3.2
Any help or input appreciated, thanks!

Are you using Android X? This plugin requires it. If you don't want to use Android X then downgrade versions.

Hello! @kmcgill88
Yes I am using Android X

I saw another open issue where it says "do not support for flutter 1.6"
Is that why it won't work for me maybe? I doesn't support flutter version 1.6 and above? (I use version 1.6.3)

@matte5031 could be! I haven't tried the plugin on 1.6.3. I've only used the stable 1.5.3. If you debug more and figure something out please post back to this issue.

Okay so the root of the problem was that the meta-data was in the wrong place, it must be within  and  tags, this can be closed. thanks

Glad you figured it out!

Are you using Android X? This plugin requires it. If you don't want to use Android X then downgrade versions.

Hi,
I'm facing the same problem here, so how to downgrade from AndroidX, it is really makes me too many problems with all AdMob dependencies. I would appreciate your help.
Thank you in advance.


Ubuntu & Windows autoinstallers tests are not finished

Tests for checking installation of operating systems were not finished. The main problem is that the installer images must be present on the pikvm partition and must be simulated as flash memory. Windows autoinstaller was successful to boot, but it was failing during the installation. Ubuntu image wasn't bootable for some reason. Below is the code produced for these tests:
Test cases:
OSA001.001 Windows 11 autoinstaller
    [Documentation]    This test mounts windows autoinstaller image, runs it and
    ...    tests if installation has run correct
    Mount image on PiKVM    windows11-autoinstaller.bin
    Power On
    Enter Boot Menu Tianocore
    Enter submenu in Tianocore    PiKVM Composite KVM Device
    # Wait for the installation to end
    Telnet.Set Timeout    60 minute
    Read From Terminal Until    ${tianocore_string}
    Sleep    60s
    Login to Windows
    Pass Execution    System installed successfully

OSA002.001 Ubuntu 22.04 auto installer
    [Documentation]    This test mounts windows autoinstaller image, runs it and
    ...    tests if installation has run correct
    Mount image on PiKVM    ubuntu_auto.bin
    Power On
    Enter Boot Menu Tianocore
    Enter submenu in Tianocore    PiKVM Composite KVM Device
    # Wait for the installation to end
    Telnet.Set Timeout    60 minute
    Read From Terminal Until    ${tianocore_string}
    Pass Execution    System installed successfully

Keywords:
Check if image is present on PiKVM
    [Documentation]    Checks if specified image is available on PiKVM
    [Arguments]    ${image_name}    ${pikvm_username}=admin    ${pikvm_password}=admin
    ${auth_credentials}    Evaluate    base64.b64encode("${pikvm_username}:${pikvm_password}".encode('utf-8')).decode('utf-8')
    Create Session    pikvm    https://${pikvm_ip}/api
    ${headers}    Create Dictionary    Authorization=Basic ${auth_credentials}
    ${response}=    GET On Session    pikvm    /msd    headers=${headers}
    ${response}=    Evaluate    json.loads('''${response.text}''')
    ${images}=    Set Variable    ${response['result']['storage']['images']}
    Should Contain    ${images}    ${image_name}

Mount image on PiKVM
    [Documentation]    Mounts selected image as flash memory
    [Arguments]    ${image_name}    ${pikvm_username}=admin    ${pikvm_password}=admin
    Check if image is present on PiKVM    ${image_name}    pikvm_username=${pikvm_username}    pikvm_password=${pikvm_password}
    ${auth_credentials}    Evaluate    base64.b64encode("${pikvm_username}:${pikvm_password}".encode('utf-8')).decode('utf-8')
    Create Session    pikvm    https://${pikvm_ip}/api
    ${headers}    Create Dictionary    Authorization=Basic ${auth_credentials}
    ${response}=    POST On Session    pikvm    url=/msd/set_connected?connected=0    headers=${headers}
    ${response}=    POST On Session    pikvm    url=/msd/set_params?image=${image_name}&cdrom=0    headers=${headers}
    ${response}=    POST On Session    pikvm    url=/msd/set_connected?connected=1    headers=${headers}

Instead we work on:

https://github.com/Dasharo/open-source-firmware-validation/pull/338
https://github.com/Dasharo/open-source-firmware-validation/pull/274


About resourcemanager port

version 1.3.1
How to change this port.
HA Rresourcemanager.
if resourcemanager HA enable, please type the HA ips ; if resourcemanager is single, make this value empty
yarn.resourcemanager.ha.rm.ids=<IP_ADDRESS>,<IP_ADDRESS>
if resourcemanager HA enable or not use resourcemanager, please keep the default value; If resourcemanager is single, you only need to replace ds1 to actual resourcemanager hostname.
yarn.application.status.address=http://yarnIp1:8088/ws/v1/cluster/apps/%s

我添加了配置并重启,没有效果

yarn.application.status.address
This configuration also needs to be changed
is ok


Log files in web gui shows log message in Date column

Describe the bug
When looking at any log file in the web gui, the Date column contains the date and time plus the log message. The Message column contains part of the date plus the log message.
Expected behavior
The Date column should only contain the date. The Message column should only contain the log message.
Screenshots

Relevant log files
Additional context
Environment
OPNsense 19.1.8-amd64
FreeBSD 11.2-RELEASE-p10-HBSD
OpenSSL 1.0.2r 26 Feb 2019

Fixed tomorrow in 19.1.9, see https://github.com/opnsense/core/issues/3493#issuecomment-494119420

Ok, thanks, sorry for the duplicate!

No worries. 😊


Decouple web interface and create a separate package for it

Web interface is now in the voltron-web package. It can be installed from PyPI with pre-packed JS or built (requiring npm and webpack) from the voltron-web repo


Annotations attributes

What do you think about adding annotations attributes ?
https://www.jetbrains.com/help/resharper/Reference__Code_Annotation_Attributes.html

I like the idea, but it would take an incredible amount of time since I would have to read the help for each of the 10,000 functions and annotate each one. Thoughts on automating that?

Not sure what is the best way.
If you look in SDK headers, they have several attributes specifying whether pointer is nullable or not for instance:
WINBASEAPI
BOOL
WINAPI
CreateProcessW(
    _In_opt_ LPCWSTR lpApplicationName,
    _Inout_opt_ LPWSTR lpCommandLine,
    _In_opt_ LPSECURITY_ATTRIBUTES lpProcessAttributes,
    _In_opt_ LPSECURITY_ATTRIBUTES lpThreadAttributes,
    _In_ BOOL bInheritHandles,
    _In_ DWORD dwCreationFlags,
    _In_opt_ LPVOID lpEnvironment,
    _In_opt_ LPCWSTR lpCurrentDirectory,
    _In_ LPSTARTUPINFOW lpStartupInfo,
    _Out_ LPPROCESS_INFORMATION lpProcessInformation
    );

I have used the [Optional] attribute on parameters that could be omitted by passing null in the 2.0 release. Using the JetBrains attributes are the next step, but I wanted to get 2.0 out.


Fix bug calculating scroll position when unloading elements

Great library. Do you guys use this in production? Seems like you would have caught this one... scroll position was calculated wrong so the page jerked around when unloading elements.

Nevermind I'm still working on figuring this out but the fix is bad :(

Btw while you are looking at this... if you could offer any tips why I'm seeing jerkiness during Page loads/unloads... I'll let you know when I figure out what the actual problem was...


fix: e2e concurrency

Type of change:

[ ] Bugfix
[ ] New feature provided
[ ] Improve performance
[ ] Backport patches

What this PR does / why we need it:

Pre-submission checklist:

[ ] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?
[ ] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

Codecov Report

Merging #1078 (cbe50af) into master (8d76428) will decrease coverage by 0.01%.
The diff coverage is 29.54%.

:exclamation: Current head cbe50af differs from pull request most recent head a30d813. Consider uploading reports for the commit a30d813 to get more accurate results

@@            Coverage Diff             @@
##           master    #1078      +/-   ##
==========================================
- Coverage   30.83%   30.81%   -0.02%
==========================================
  Files          77       77
  Lines        9140     9174      +34
==========================================
+ Hits         2818     2827       +9
- Misses       6019     6038      +19
- Partials      303      309       +6

Impacted Files
Coverage Δ

pkg/ingress/controller.go
0.83% <0.00%> (-0.01%)
:arrow_down:

pkg/ingress/endpoint.go
0.00% <0.00%> (ø)

pkg/ingress/gateway_httproute.go
0.00% <0.00%> (ø)

pkg/kube/translation/translator.go
43.51% <14.28%> (-2.56%)
:arrow_down:

pkg/kube/translation/gateway_httproute.go
63.87% <45.00%> (-3.53%)
:arrow_down:

pkg/kube/translation/util.go
33.73% <75.00%> (+1.01%)
: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 2af39c9...a30d813. Read the comment docs.


Only go async when flush async

@dougbu I saw this in your traces from  #5320 but can't seem to get the async paths to trigger to measure the difference
What tests were you running? The .ps1 ones?

Ahh... debug...

FYI @benaadams later images in #5320 use Release bits.

Does this PR actually reduce allocations in any current Performance scenario when using Release bits?

No, which is why I was confused 😁

What tests were you running? The .ps1 ones?

Generally yes. For the BasicViews scenario, I use .\loadtest.ps1 -rps 20 -variation htmlhelpers b/c the default 100 RPS leads to more noise in the profile and the other scenarios I've been using also focus on HTML helpers. (My changes in #5320 should result in similar reductions but less percentage-wise since tag helper scenarios use more memory.)

No, which is why I was confused 😁

So should we close this PR?


Save sorting selection on exit

Sorting appears to default to Ascending despite being changed to Descending before exit.
Can this be changed so it saves the User's selection on exit?

Done with https://github.com/Rinnegatamante/yoyoloader_vita/commit/1268a60b0006e67a9cf07f8e98c6181b7d773485


docket: fix requestTreaty

This fixes app refs by exposing the @urbit/api#subscribeOnce to the useDocketState hook.

closed in favor of #1343


Console errors: "..record has no field with key 'address'"

Halin v 0.12.0
Neo4j 4.0.0-beta03mr03
Seeing many of these errors on the console. Only with a Neo4j 4.0 db (4.0.0-beta03mr03 to be exact).
This also seems to be preventing listing of the multiple databases
index.js:1437 WARN ClusterManager#getDatabases() returned unexpected error Neo4jError: This record has no field with key 'address', available key are: [name,status,default].
    at newError (http://localhost:3000/static/js/0.chunk.js:177748:10)
    at Record.get (http://localhost:3000/static/js/0.chunk.js:192108:37)
    at getValueFromRecord (http://localhost:3000/static/js/main.chunk.js:9356:24)
    at http://localhost:3000/static/js/main.chunk.js:9406:19
    at Array.forEach (<anonymous>)
    at http://localhost:3000/static/js/main.chunk.js:9404:29
    at Array.map (<anonymous>)
    at Object.unpackResults (http://localhost:3000/static/js/main.chunk.js:9395:26)
    at http://localhost:3000/static/js/main.chunk.js:1063:64

@aldrinm Halin v0.12 just came out, which supports neo4j 4.0.   I'm not sure at this point what's going on with beta03mr03, because there were internal surface changes on some of the temporary testing builds.  Could you please try v0.12 against Neo4j 4.0 or 4.0MR01 and report back?   I'm not seeing this error right now with 4.0.
Going forward, because 4.0's release is right around the corner, the older MR, beta, and alpha builds won't be supported by Halin because they contain a lot of intermediate changes that didn't make it into the final product, so I'm hoping this is just a temporary hiccup, but standing by for your confirmation.

Fair enough. I just tried it with 4.0 RC01 and it works just fine.


RuntimeError: shape '[800, 800, -1]' is invalid for input of size 6220800

I trained the tutorial model stonehenge, and got some varying results (more successful when the distance is shorter and less obstacles). Afterwards I decided to go on with my own Blender model to give it a test. After many trial and error, I managed to find a setting which worked fine until I receive a RuntimeError at 7% while simulating it. The log file is attached but the error goes like:
RuntimeError: shape '[800, 800, -1]' is invalid for input of size 6220800
Is anyone else having the same problem simulating their own model or does anyone know how to solve this issue?

Many Thanks!

In case someone faces the same problem, I solved it by applying the following:

Turn of the "render" option of the environment.
Take the camera out of the environment.

and save your blender file before simulating.

Hope it helps!