When does upsertGraph attempt to insert a model, as opposed to relate to it?

Hi, I was curious as to why upserting this object with relate: true:
{
   "status":"1",
   "identifier":"Team",
   "users":[
      {
         "id":89,
<EMAIL_ADDRESS>         "role":{
            "id":2,
            "display_name":"Disc Jockey",
            "name":"dj"
         },
         "team_admin":false
      },
      {
         "id":2,
<EMAIL_ADDRESS>         "role":{
            "id":1,
            "display_name":"Admin",
            "name":"admin"
         },
         "team_admin":true
      }
   ]
}

generates queries that insert the "users" rather than relates to them.
I know that in the absence of a reproduction, your ability to diagnose the specific issue is limited. I was just wondering why the presence of the "id" property doesn't automatically prevent insertions.
Team relations, for reference:
  static get relationMappings() {
    return {
      users: {
        relation: DatabaseModel.ManyToManyRelation,
        modelClass: User,
        join: {
          from: 'team.id',
          through: {
            from: 'team_membership.team_id',
            to: 'team_membership.user_id',
            extra: ['team_admin']
          },
          to: 'users.id'
        }
      },
    };
  }

If there is no explanation for it within the objections framework, I'll continue to hunt for the cause within my own code rather than check the documentation, but I did want to inquire here quickly. Thank you.

If you aren't using insertMissing: true, I don't think this should generate any inserts other than the root (assuming identifier isn't the primary key, or is it?) and the pivot table rows. There could be a bug here. Could you try to reproduce this using the reproduction-template.js file or some other method?

@GreatSchool2 Did you solve this?

Yeah, the issue had to do with how IDs are detected on an object. I traced the logic to:
https://github.com/Vincit/objection.js/blob/32a90c1c1e1ab808fa4562635417006a53ccfc36/lib/model/modelColPropMap.js#L5
and observed that since my constructors (which took an optional copy argument that defaulted to an empty object) were just assigning this.id = copy.id, the id property never showed up in the difference. I also had implemented my own $parseDatabaseJson which used these constructors.

Oh, ok. Objection doesn't really use constructors for anything. If you want, you can override and use the fromJson static method instead.


From private repo

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


Nonexistent signal after adding StaticBody to a MeshInstance from editor

Godot version:
master
OS/device including version:
Fedora 28: Linux 4.17.7-200.fc28.x86_64
Mesa 18.0.5
VGA compatible controller: Intel Corporation 3rd Gen Core processor Graphics Controller
Kernel driver in use: i915
VGA compatible controller: NVIDIA Corporation GF117M [GeForce 610M/710M/810M/820M / GT 620M/625M/630M/720M] (rev a1)
Subsystem: Acer Incorporated [ALI] GeForce 710M
Kernel driver in use: nouveau
Issue description:
After launching the project, the Debugger shows the following warning
0:00:00:0287 - Nonexistent signal: changed
----------
Type:Error
Description: Nonexistent signal: changed
Time: 0:00:00:0287
C Error: Condition ' !s ' is true.
C Source: core/object.cpp:1465
C Function: disconnect

I shows one for every MeshInstance with StaticBody and collisions created from the editor (Convex or Trimesh).

Steps to reproduce:

Create new project.
Add Spatial base node for the scene.
Add a MeshInstance with a CubeMesh, select the mesh Instance and add collisions from the editor (Create Convex Static Body).

I've added the --version

I'm glad I found this, I was going mad trying to locate the offending signal/connection in my projec,

Closing as Fixed by #20381


Add Loading Spinner to CatchAll routes fallback, and show 404 on Release page error

📝 Changed routes:

…Address]/[...catchAll]

(view full size)

Commit acd56a4557b074ec36426ee56980d16ae9bc1467 (https://thirdweb-lun8s416c.thirdweb-preview.com).


Begin refactoring ndt-cloud

This change begins to refactor the content of ndt-server.go into separate packages.
Specifically, the TestResponder implementation is renamed to the legacy.Responder in the legacy package.
legacy/c2s.go and legacy/s2c.go contain the logic necessary for each operation and follow a similar pattern each.

{C2S,S2C}TestHandler - the handler used by the test server started in the control channel that runs the actual test.
{C2S,S2C}Controller - the complementary logic to the test handler called from the control channel.
run{C2S,S2C} - simple timeout wrappers around {send,recv}Until
{send,recv}Until - the transfer loops underlying the S2C and C2S tests.

As well, this change adds the ndt/const.go file for constants shared across packages.
And, the content of ndt-server.go is minimized. the NdtServer handler for the control channel should be moved in a future PR.

This change is

Do please resolve the conflicts, though.

The changes here are still a good idea, but the PR is unrecoverable at this point due to subsequent changes.


python311Packages.elastic-apm: 6.16.1 -> 6.16.2

Diff: https://github.com/elastic/apm-agent-python/compare/refs/tags/v6.16.1...v6.16.2
Changelog: https://github.com/elastic/apm-agent-python/releases/tag/v6.16.2
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 240807 run on aarch64-linux 1

4 packages built:

python310Packages.elastic-apm
python310Packages.elastic-apm.dist
python311Packages.elastic-apm
python311Packages.elastic-apm.dist

Result of nixpkgs-review pr 240807 run on aarch64-darwin 1

4 packages built:

python310Packages.elastic-apm
python310Packages.elastic-apm.dist
python311Packages.elastic-apm
python311Packages.elastic-apm.dist


Missing LICENSE file

As per https://github.com/andersson/pil-squasher/blob/e8573fe/pil-squasher.c#L1-L30 the source code is licensed under a BSD-3-Clause, yet the repository root is missing a LICENSE file, which is required to be distributed along with installed binaries.

@andersson could you please add the LICENSE file?


JSONRPC - error calling getbalance, sendtoaddress

Sorry my mistake - there was corrupted block and wallet runs in Safe Mode

Hi,
I'm getting errors calling getbalance, sendtoaddress RPCs.
Other like getinfo() are OK.
Also I'm unable to receive money to my wallet

ok

@michalzahor
new
https://github.com/withu2018/infinitecoin


creating a cloudwatch logs group and giving explicit access to only it

Description
The resources deployed by this module mimics the experience when creating Lambda functions in the console. The function gets created with an IAM role attached which allows it to put logs and create groups/streams anywhere; even places it's not supposed to. When the function creates the log group with it's built-in permissions this happens outside of Terraform. When you destroy the module a log group will remain and become stale because it's never tracked by Terraform's state in this way.
This PR adds a Cloudwatch Logs group and adds an explicit dependency on the function so that it will only be created when the log group is made (to prevent a race condition). It also adjusts the policy of the function to explicitly allow only log events on the Terraform managed log group.

Now that I think about it we could probably do away with the CreateLogGroup permission since that will be done by Terraform...

Thanks, @lalanza808 and @mwarkentin for this PR!
I've just released v2.3.0 with this feature.
Please let me know if something is not working as expected.


How to create strict date and time input?

Which package(s) are the source of the bug?
@maskito/kit
Playground Link
https://maskito.dev/kit/date#min-max
Description
I'm trying to create a date input field and a time input field that can not be empty and always has a valid date or time.
I found #1217 which states:

This behavior is already implemented in our mask.
In strict mode, if a user tries to enter a non-valid date, the mask will either prevent the user from entering such a date, or if it is possible to turn it into a valid date, it will change it to a valid date. Textfield's value NEVER contains invalid date value.

If this behavious is already implemented, how do i activate this?
Even with min & max set, i can still just input any number into the date field, for example just a 1.

If this is not possible by default, is there an easy way to abort a user input in a pre or postprocessor?
Maskito version
3.0.1
Which browsers have you used?

[ ] Chrome
[X] Firefox
[ ] Safari
[ ] Edge

Which operating systems have you used?

[ ] macOS
[X] Windows
[ ] Linux
[ ] iOS
[ ] Android

The mask will turn the fully entered date into a valid date, meaning if you entered a date earlier than min — it would be converted or if you entered 13th month etc. But it will not disallow you typing intermediary states. For your situation, maybe a good idea would be to add a plugin that sets the value on blur?

I dont want to allow any intermediary state at any time, i start with a valid date and i just want to cancel every user input that would lead to an invalid date. Same with time.

@movedoa it cannot be done even with plain integers, much less for dates. To be specific, it can be implement done for max, but it cannot be done for min and the reason is of a logical nature. Funerals start with fun (pun intended).
To enable typing in some string value xxx, each intermediate state (state.substring(0, n) where n = 1, state.length) must be allowed as well. For "Funerals", that amounts to ['F', 'Fu', 'Fun', 'Fune', 'Funer', 'Funera', 'Funeral']. Furthermore, when it comes to integers, the intermediate states are not linear, since:

Length is not the only variable anymore, now we have content that produces n permutations that satisfy the given constraint
And even the above is only the case if there's a maximum as well. If there's no maximum, any valid integer is a valid intermediate state, since regardless of what it is, appending enough or more than enough (up to infinity) digits will make it satisfy min.

❌ 1 > 900
❌ 10 > 900
❌ 100 > 900
✔ 1000 > 900

Technically you can do it the way you want, it's just going to be frustration simulator for your users. Imagine they want to type in 31 of January while the input contains 28 of February. You would force them to go change the month first then go and change the date and no other way to do it. It's much easier to just erase the whole value and type in 31.02 — especially on the phone with numbers keyboard, than to try and move caret around an undeletable string value to edit the parts in the only order that works.


More quotes

I am interested in making a contribution to this repo. Is it open to adding more quotes?

yes it's open to adding more quotes

Fork => add quotes => make a pr


Unexpected issue with package import, using cython and numpy

Details

Read the Docs project URL: https://freud.readthedocs.io/
Build URL (if applicable): https://readthedocs.org/projects/freud/builds/8431508/ and https://readthedocs.org/projects/freud/builds/8452348/
Read the Docs username (if applicable): bdice

Expected Result
Build passes, documentation shows module info.
Actual Result
Build passes but doesn't actually contain module documentation. Builds should show as failing.
Expected result is on stable: https://freud.readthedocs.io/en/stable/box.html
Failing result is on latest: https://freud.readthedocs.io/en/latest/box.html
The package appears to build correctly, but we think there is something going wrong with Cython or NumPy and we can't get enough info to debug it ourselves. When RTD installs on top of our environment.yml file, it changes some package versions, specifically numpy from the openblas to mkl variants. I don't think that openblas/mkl is the problem, but changing numpy versions may very well invalidate something we rely on. We've seen two different error messages but it's not clear what causes either one.
This failure began appearing about simultaneously with our switch from Bitbucket (https://bitbucket.org/glotzer/freud.git) to GitHub (https://github.com/glotzerlab/freud.git) so we're not sure what to blame. However, I think I ruled out the repo URL change as a cause. I tried changing the URL back to Bitbucket and the issue can be reproduced. Also I created a fresh RTD project with the GitHub URL and it fails in the same way.
Most recent error (https://readthedocs.org/projects/freud/builds/8452348/):
/home/docs/checkouts/readthedocs.org/user_builds/freud/checkouts/awesome-readme/doc/source/box.rst:7: WARNING: failed to import freud.box.Box
/home/docs/checkouts/readthedocs.org/user_builds/freud/checkouts/awesome-readme/doc/source/box.rst:7: WARNING: failed to import freud.box.ParticleBuffer
WARNING: autodoc: failed to import module 'box' from module 'freud'; the following exception was raised:
Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/freud/conda/awesome-readme/lib/python3.7/site-packages/sphinx/ext/autodoc/importer.py", line 154, in import_module
    __import__(modname)
  File "/home/docs/checkouts/readthedocs.org/user_builds/freud/conda/awesome-readme/lib/python3.7/site-packages/freud/__init__.py", line 5, in <module>
    from . import box
  File "freud/box.pyx", line 37, in init freud.box
AttributeError: type object 'freud.box.Box' has no attribute '__reduce_cython__'

Previous error from earlier in the month (https://readthedocs.org/projects/freud/builds/8431508/):
/home/docs/checkouts/readthedocs.org/user_builds/freud/checkouts/latest/doc/source/box.rst:7: WARNING: failed to import freud.box.Box
/home/docs/checkouts/readthedocs.org/user_builds/freud/checkouts/latest/doc/source/box.rst:7: WARNING: failed to import freud.box.ParticleBuffer
WARNING: autodoc: failed to import module 'box' from module 'freud'; the following exception was raised:
Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/freud/conda/latest/lib/python3.6/site-packages/sphinx/ext/autodoc/importer.py", line 154, in import_module
    __import__(modname)
  File "/home/docs/checkouts/readthedocs.org/user_builds/freud/conda/latest/lib/python3.6/site-packages/freud/__init__.py", line 7, in <module>
    from . import common
  File "/home/docs/checkouts/readthedocs.org/user_builds/freud/conda/latest/lib/python3.6/site-packages/freud/common.py", line 7, in <module>
    import numpy as np
  File "/home/docs/checkouts/readthedocs.org/user_builds/freud/conda/latest/lib/python3.6/site-packages/numpy/__init__.py", line 142, in <module>
    from . import core
  File "/home/docs/checkouts/readthedocs.org/user_builds/freud/conda/latest/lib/python3.6/site-packages/numpy/core/__init__.py", line 59, in <module>
    from . import numeric
  File "/home/docs/checkouts/readthedocs.org/user_builds/freud/conda/latest/lib/python3.6/site-packages/numpy/core/numeric.py", line 3093, in <module>
    from . import fromnumeric
  File "/home/docs/checkouts/readthedocs.org/user_builds/freud/conda/latest/lib/python3.6/site-packages/numpy/core/fromnumeric.py", line 17, in <module>
    from . import _methods
  File "/home/docs/checkouts/readthedocs.org/user_builds/freud/conda/latest/lib/python3.6/site-packages/numpy/core/_methods.py", line 158, in <module>
    _NDARRAY_ARRAY_FUNCTION = mu.ndarray.__array_function__
AttributeError: type object 'numpy.ndarray' has no attribute '__array_function__'

Continuing to test...
5. Minimal (tbb + cython) conda environment (conda-forge)
It might be possible to use only TBB and Cython from conda-forge and rely on pip to install numpy correctly. I made a new Python 3.6 environment with only tbb tbb-devel cython installed from conda-forge.

environment.yml
cython 0.29.4 (conda-forge)
pip install . fails to find Cython?! I thought this part was working before, and I'm not sure what changed. 🤷♂

Result: https://readthedocs.org/projects/freud/builds/8519670/
Processing /home/docs/checkouts/readthedocs.org/user_builds/freud/checkouts/fix-rtd
  Installing build dependencies: started
  Installing build dependencies: finished with status 'done'
  Getting requirements to build wheel: started
  Getting requirements to build wheel: finished with status 'error'
  Complete output from command /home/docs/checkouts/readthedocs.org/user_builds/freud/conda/fix-rtd/bin/python /home/docs/checkouts/readthedocs.org/user_builds/freud/conda/fix-rtd/lib/python3.6/site-packages/pip/_vendor/pep517/_in_process.py get_requires_for_build_wheel /tmp/tmpipwqys9e:
  Traceback (most recent call last):
    File "setup.py", line 168, in <module>
      from Cython.Build import cythonize
  ModuleNotFoundError: No module named 'Cython'

... # and then our setup script catches this and raises a user-friendly exception

@stsewd I think I need to know more about how the actual ReadTheDocs image works to fix this problem. Using a Docker container (docker run -it readthedocs/build) and the following script adapted from the RTD build output, I am able to build my package successfully. I'm not sure what RTD does differently than this script. Can you give any insight?
Specifically, when does the conda environment get activated? It doesn't show that step in the build output, but surely it has to happen.
cd ~
git clone --no-single-branch --depth 50 https://github.com/glotzerlab/freud.git
cd freud
git checkout --force origin/fix-rtd
git clean -d -f -f
git submodule update --init
conda env create --name fix-rtd --file environment.yml
conda install --yes --name fix-rtd mock pillow sphinx sphinx_rtd_theme
source activate fix-rtd
python -m pip install -U recommonmark readthedocs-sphinx-ext
python -m pip install --ignore-installed .
cd ~  # so that we don't import the local, un-built package
python -c "import freud"  # succeeds

We don't call activate, we just execute the commands with the python binary from conda.
https://github.com/rtfd/readthedocs.org/blob/a472cb939c65b8765ea3eaaa710fd26d22afdd45/readthedocs/doc_builder/python_environments.py#L386-L387
https://github.com/rtfd/readthedocs.org/blob/a472cb939c65b8765ea3eaaa710fd26d22afdd45/readthedocs/doc_builder/python_environments.py#L122-L132
Maybe we are losing something when skipping the activate step?

@stsewd I tried installing pip into the environment, but that didn't help. Previously, the environment.yml I provided worked fine and Cython was found from the conda installation.
The first failing build was Jan. 22: https://readthedocs.org/projects/freud/builds/8431508/
The last working build was Jan. 9: https://readthedocs.org/projects/freud/builds/8368585/
I see some commits (by you) over that time span that change the handling of pip, I wonder if that is related to the problem? The build outputs show that the change from pip to python -m pip coincide with the introduction of this failure.
https://github.com/rtfd/readthedocs.org/commit/50035a40adcb111b98581a58cc466363bfa8be00#diff-31f8391d8643c47c6caedf53930c0b7e

I tried enabling the option to use setup.py, but it didn't seem to change the build output commands. That feature appears to be disabled for conda builds, but that isn't documented on the settings page...
I also tried creating a new doc/requirements-rtd.txt file that includes cython for ReadTheDocs, but this is unfortunately disabled for conda environments (also not documented on the settings page). https://github.com/rtfd/readthedocs.org/blob/50035a40adcb111b98581a58cc466363bfa8be00/readthedocs/doc_builder/python_environments.py#L418-L421
https://readthedocs.org/projects/freud/builds/8525999/

We have this issue https://github.com/rtfd/readthedocs.org/issues/2776, isn't clear for me why we don't install requirements in conda

How should we proceed with testing the pip vs python -m pip issue? Do I need to install RTD locally and test RTD before/after that change was made? I looked at the installation guide and could probably do this if you aren't able. We're holding on the 1.0 release of our package until docs can be fixed, so fixing this is pretty high on our team's priority list. Your help is sincerely appreciated, @stsewd!

Sorry, wrong button, I'll try to replicate this locally in the next days. If it was working before, I have some guess

Awesome, @stsewd. I got pretty far with installing RTD locally. I got it to start running Docker containers to build the software but the containers fail when they get to conda commands for some reason. I have not had time to debug that yet. I look forward to your tests, hopefully we can identify the cause and get this resolved soon. Thanks again for your help.

I didn't read the whole thread. Why do you need to install conda dependencies from environment.yml and requirements.txt file?
If you want to install things with pip inside your conda env, you can use dependencies.pip in your YAML file: https://conda.io/projects/conda/en/latest/user-guide/tasks/manage-environments.html#creating-an-environment-file-manually

@humitos Good idea, I hadn't considered using that approach in the conda environment. My current understanding of the problem(s) is that conflicting versions of numpy were being installed (>=1.16 and <1.16 can't coexist, it seems). This may have been due to a change in the way packages are installed and detected in the environment (RTD switched from pip to python -m pip). I have been experimenting with different ways to install packages, using pip or install_requires in setup.py instead of the conda environment, hoping to find a configuration that works as a short-term solution. But if this is due to the changes in RTD (which coincide with the problem's introduction, but we haven't yet definitively shown), it would be good to address that root cause instead of combining various environment/package management hacks.

@stsewd @humitos It seems that I've been able to solve the problem by circumventing pip entirely. I discovered that I was running v1 of the readthedocs.yml schema, so I upgraded that to v2. (I'm not sure if upgrading this was necessary.) I created a fresh conda environment with all requirements and installed the package with setuptools, and it seems to build. I may have been able to fix this by just using setuptools with config v1, but I'm not sure. I had tried using setuptools via the web interface earlier but I think I must not have realized that the web interface buttons didn't affect my build at all (the config file is the primary source of truth, in my understanding).
I think there is still an underlying bug with how pip was being used by RTD, since using pip instead of setuptools should still detect all the packages I installed (namely cython and the proper version of numpy). This used to work fine with pip until the recent changes mentioned above. I'll close this issue for now - thanks again to both of you for the helpful suggestions and support.

@bdice thanks for the extensive testing and the final update on this! Appreciated!


[opentelemetry-operator] allow to override repositories without need to overwrite tag

We would like to be able to use own docker registry for example for metricsAllocator. Now it we want to do it, we need to specify tag as well, which is problematic as it has to be updated every time we would like to update the chart and use the default version. I think that it should use appVersion as default tag value.
Here is the code which requires both tag and repository values.

Although typically appVersion corresponds directly with the thing being installed, I think since the operator version and target allocator version and auto-instrumentation versions are so closely related I'd be ok using appVersion across all those images.
I do not want to use it for the collector tag.

@TylerHelmuth I have also the question a bit related to that. Would it be fine to move collectors tag and repo to CRD? I imagine there may be some scenarios when users would like to have multiple collectors/distros and one operator 🤔

The OpenTelemetryCollector CRD allows you to specify the collector image it uses.  The collector image in the chart sets the Operator's default collector image, but the CRD allows overriding. The operator chart does not install OpenTelemetryCollector CRs.

@TylerHelmuth I started working on that and after all I see two possible solutions:

write everything explicitly in values.yaml (preferred by me, but may be harder to maintain)
use appVersion as default version and .Values.manager.image.repository as default repository (less overrides if customer wants to change repository)

As there are two parts which may be overrided, both of them needs defaults or both of them needs to be always set. I think it's more clear and transparent to set both of them with values.yaml


Packfile Virtual File System

Change the existing class PackfileVFS to provide an interface for packfiles similar to the standard IO functions. Should be able to enumerate packfiles like folders and read/write data from paths.
For the moment this will be read only. I expect the initial version to support data folder packfile access via //data/. So, for example, iterating the files in misc.vpp_pc might look like this in code:
for (var entry in VFS.Enumerate("//data/misc.vpp_pc/"))
{
    //Do something with entry. Entries can be directories or files. vpps and str2s are directories.
}

Some other improvements I'd like to make post v1.0.0:
#56

Completed by 5c1e7661ba0cdfd64699735bb8cfc3fa57624bc3


Feature request: Processing on multiple sets of FASTQ files

I frequently work with single-cell multi-omics data (mostly scRNA-seq + CITE-seq, but scATAC-seq is also expected), and it is known that different modalities for the same samples requires separate sequencing libraries for each modality, which means there will be multiple sets of FASTQ files per sample. What I'm looking for in splitcode is the ability to reassign barcodes for several sets of FASTQ files uniformly such that multi-omics data can seamlessly integrate. Is it possible to do this?

Hmm, not sure I understand.
Are you talking about the final barcodes that are created when you use --assign? You'd like those barcodes to carry over from, say, your scRNA-seq data to your CITE-seq data?
Or, do you mean, supplying scRNA-seq data and your CITE-seq data in a single splitcode command?
Perhaps a simple example would help.


add cache control header option

Adds an optional header field to allow the browser to cache the responses.

@itowlson I will add docs to developer once this gets in and closes in on a release.

Ah, of course, the docs are in a separate repo, as I perfectly well knew.  I feebly plead Monday.


Removing description strings from boolean flags

Boolean flags dont have a description. This will change the help messages for flags from:
  -d, --debug bool  (default false)

to
  -d, --debug

@goyox86

Current coverage is 95.04% (diff: 100%)

Merging #37 into master will increase coverage by 0.02%

@@             master        #37   diff @@
==========================================
  Files            28         28
  Lines          2288       2299    +11
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           2174       2185    +11
  Misses          114        114
  Partials          0          0

Powered by Codecov. Last update 0ae5aaf...3eb9d66

I will merge this, when you are back we will have a walkthrough the code


Problems Compiling with IL2CPP

Hi, I've been trying to compile my App for iOS, since a few weeks now iOS is forcing everyone to upload the new apps and updates with 64bit support, so, Unity has a new way to compile for iOS called IL2CPP. If you want support for both 32 and 64 bits, you must check "Universal".
The thing is that I am Using Unity 4.6.6f1 (pro) with the latest cardboard plugin. If I compile with Mono 2.x it works like a charm, everything work perfectly, even on iPad 2, it renders smaller to fit the cardboard. I can even run the app with iOS 7.1. I tried to upload to the appstore and apple rejected immediately the submission because it didn't have 64bits support.
Then I just went to the player settings, compiled with IL2CPP Universal, disabled the striping level (just in case) and went to Xcode to test. The result: 1 error and I can't find ANYTHING about it ANYWHERE. Google shows 0 results. I don't really know if this is related to Unity only (something they have done wrong) if I'm missing something (maybe some Xcode stuff I didn't do correctly) or related to the plugin... anyways, if anyone here can help me solve this I'll gladly appreciate it.
More info:

I followed the steps just like the instructions said (added CoreText, libc++, added resources.bundle)
The project doesn't have any other plugins (except only for one to reproduce videos, "mobile movie texture" latest version)
The error is this:

ld: 64-bit LDR/STR not 8-byte aligned: from l005 (0x100D1D540) to l036@0x00001304 (0x100ED4774) in 'l005' from /Users/nosmirck/Desktop/build/Libraries/libplatform.a(RotateToLandscapeViewController.o) for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Thanks in advance guys!
Best Regards,
Luis.

So... I've had a similar issue with libPhone.a so maybe you could try this:
check that libplatform.a is in fact in the project's linked frameworks and
libraries. If it's already there, remove it and add it back again.
El sáb., jun. 27, 2015 1:25 AM, Luis Pulido<EMAIL_ADDRESS>escribió:

Hi, I've been trying to compile my App for iOS, since a few weeks now iOS
is forcing everyone to upload the new apps and updates with 64bit support,
so, Unity has a new way to compile for iOS called IL2CPP. If you want
support for both 32 and 64 bits, you must check "Universal".
The thing is that I am Using Unity 4.6.6f1 (pro) with the latest cardboard
plugin. If I compile with Mono 2.x it works like a charm, everything work
perfectly, even on iPad 2, it renders smaller to fit the cardboard. I can
even run the app with iOS 7.1. I tried to upload to the appstore and apple
rejected immediately the submission because it didn't have 64bits support.
Then I just went to the player settings, compiled with IL2CPP Universal,
disabled the striping level (just in case) and went to Xcode to test. The
result: 1 error and I can't find ANYTHING about it ANYWHERE. Google shows 0
results. I don't really know if this is related to Unity only (something
they have done wrong) if I'm missing something (maybe some Xcode stuff I
didn't do correctly) or related to the plugin... anyways, if anyone here
can help me solve this I'll gladly appreciate it.
More info:

I followed the steps just like the instructions said (added
CoreText, libc++, added resources.bundle)
The project doesn't have any other plugins (except only for one to
reproduce videos, "mobile movie texture" latest version)
The error is this:

ld: 64-bit LDR/STR not 8-byte aligned: from l005 (0x100D1D540) to l036@0x00001304 (0x100ED4774) in 'l005' from /Users/nosmirck/Desktop/build/Libraries/libplatform.a(RotateToLandscapeViewController.o) for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Thanks in advance guys!
Best Regards,
Luis.
—
Reply to this email directly or view it on GitHub
https://github.com/googlesamples/cardboard-unity/issues/42.

Hi lesmo, thanks for the help, I just tried what you said, same problem appears, same error, same library...
The libplatform.a was in fact in the linked libraries, I removed as you suggested and added it again, compiled and the same error came again.
What else can I do?

UPDATE: I moved the whole project to Unity 5, I had to fix some issues (skybox missing) and again compiled under the same settings as before (IL2CPP, Universal, no striping) opened in Xcode and tried to compile, the first error that came up was that it couldn't find the LibraryPlugin folder, I checked the libraries paths and there was a backslash "" between Libary and Plugin (I don't know why) but I just changed it to a forward slash "/" and that error is gone.
Now, trying to compile again and after all the almost 400 files where compiled, it failed on linking, saying that libbase.a has not 8-bit aligned for architecture arm64.
This is the error:
ld: 64-bit LDR/STR not 8-byte aligned: from __ZNSt3__118condition_variable8wait_forIxNS_5ratioILl1ELl1000000000EEEEENS_9cv_statusERNS_11unique_lockINS_5mutexEEERKNS_6chrono8durationIT_T0_EE (0x100602620) to l007@0x00000F04 (0x10127B944) in '__ZNSt3__118condition_variable8wait_forIxNS_5ratioILl1ELl1000000000EEEEENS_9cv_statusERNS_11unique_lockINS_5mutexEEERKNS_6chrono8durationIT_T0_EE' from /Users/nosmirck/Desktop/iosbuildu5/Libraries/Plugins/iOS/libbase.a(task_loop.o) for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

So I tried what @lesmo suggested (removing and adding the library) but it didn't work, the same error appeared.
I'd really like to receive an answer from the developers of this plugin, maybe something is missing and I can't figure it out.
Thanks in advance for any help you could bring.

I am able to build and run an IL2CPP Universal version of the SDK's demo scene, starting from a fresh project and importing the SDK.  This worked in both Unity 4.6.4 and 5.1.0.  I left most of the settings like stripping and Mono level.
Could you try this step, and see if you get the same results?  (Either I missed something you did or there is some other issue at work here.)

Well, I forgot to mention I did this all from PC (windows 7) Unity 5.1.1p1. I mentioned I'm using a plugin (aside from cardboard) that is mobile movie texture, latest version, but the issues are always with random libraries from the cardboard. I'm going to try now installing Unity on the Mac (I didn't before because I have no space on hard drive, but I just got an external drive to do this) I'll report back when I try this.
Anyways I did what you said just when I moved the project to Unity 5. I removed and deleted everything related to cardboard and mobile movie texture and reimported from scratch both plugins.
The project has nothing else, strange thing is that it is showing me the same linking error I got the first time (when I was using 4.6.6) but with a different library now on 5.1.1.
I'll let you know how it went.

OK.  I recommend trying with a new blank project for the "nothing but cardboard" experiment.  Trying to scrub stuff out of a Unity project is tricky because all kinds of changes lurk in various places.  Then if that works, add in the other plugin to this new project and try again.

@smdol Ok, I just tried "nothing but cardboard" and I got the same error with the same library (libbase.a(task_loop.o) linker error).
So I guess the error is because I'm compiling from windows and copying the result folder to my mac...
I'm going to try compiling the whole project on the mac directly. I'll update in a few hours.

I just compiled the whole project from the mac on Unity 4.6.3 (the one I have installed for now, 5 is downloading) and I got the same error with libplatform.a.
I created a new project from scratch on the mac with Unity 4.6.3, added only cardboard (v0.5) and tried to compile with the example (treasure) and I got the same error with the same library (libplatform.a).
So, I guess there is something wrong with the compiled libraries for iOS arm64 or I'm doing something wrong...

When you created your nothing-but-cardboard project, what build settings did you change?  I'll try the same.  Also, did you unpack the unity package on the Mac, or copy the files from Windows?  Make sure to try unpacking on the Mac.  (Let's get Windows completely out of the equation.)

UPDATE:
Still failing.
What I've done so far:

Moved the project to my Mac, installed Unity 5.1.1f1 and opened there (I had a Unity5 backup before moving to 4.6.6)
1.a) Simply I built the project, opened in XCode, tried to build = same error with libbase.a
1.b) Deleted the Plugin from the Unity Project and imported the plugins again (re-downloaded from here and the other from Asset Store), Built, opened in XCode, build, = same error with libbase.a

Created a new project from scratch, added ONLY cardboard plugin, added the demoscene to the build
2.a) Built, opened in XCode, build = same error with libbase.a
2.b) Deleted everything (empty project) created a new scene, added a cube, built, opened in XCode, build, success.

My configuration:

Unity 5.1.1f1
Xcode 6.2
iOS SDK 8.2

How I am building from Unity5 (Mac OS X 10.10.2):

IL2CPP, Universal, .Net 2.0 subset, stripping: microslib

How I am building from XCode:

Nothing special, just setting the signing account. CoreText is being added by default, and adding the resources.bundle causes error for duplicated files (they seem to be added by default on Unity 5 builds)
I hit Build or Archive, the same error happens (libbase.a on Unity 5)

So, I still don't think there is a problem with cardboard plugin, I'm currently upgrading my Xcode to 6.4 to see if this solves my problem, still, I'd like you to check this out, please use the same .unitypackage we are all using from the releases tab.
I'll come back here with updates when I upgrade XCode.

Ok, after upgrading XCode to 6.4, even the first build from windows worked just fine... I want to close this thread now that the issue is solved, but you should mention in the instructions that it requires XCode 6.4 because it doesn't say anywhere that it needs to be built from an specific version of XCode. Thanks anyways for the help!! you are doing a great job!


feat: export oak fieldset component

How to review this PR
Leave this text block for the reviewer

Check component hierarchy is followed correctly
Check the design Heuristics have been followed
Check naming conventions have been applied
Check for these gotchyas:

Missing exports for Oak components
Accidental export of Internal components
Snapshots of unexpected components have been modified
Circular dependencies
Code duplication (via not using base components)
Non-functional storybook for this or related components
Sensitve files changed eg. atoms, or style tokens
Relative imports
Default exports

Exports fieldset component

we really need a guard against this 😆 . I've done this so many times myself.

Every time 😂


 Have you tried using Pe-Bear to analyze the dll, looking for its exports?
This behaviour that you are commenting happened to me when I tried to run the tool over a DLL with no exports (for example, a dll used to store static resources like images and so), maybe that's what is going on in your case as well.

Hi @CheetoTrump, Don't think you are retarded, probably it's the tool failing for some reason xD What dll are you trying to trace/proxy? Have you tried using Pe-Bear to analyze the dll, looking for its exports?
This behaviour that you are commenting happened to me when I tried to run the tool over a DLL with no exports (for example, a dll used to store static resources like images and so), maybe that's what is going on in your case as well.

I tried different dlls and added checks etc, ill send the upate
also the dll had 3 exports

elo.json
couldnt upload .rs directly so just rename to .rs

Is your dll present on a default windows installation so I can try to run the tool on it? May you send me the exact command you are typing to generate the trace/proxy dlls?

generator.exe -m trace -p C:\Windows\System32\adhapi.dll
dumpbin export of the dll
Dump of file C:\Windows\System32\adhapi.dll
1000 .pdata
File Type: DLLrdata
1000 .reloc
Section contains the following exports for AdhApi.dll
7000 .text
00000000 characteristicsest> du
8AC114AE time date stamp
0.00 version
1 ordinal base
7 number of functions
7 number of names
ordinal hint RVA      name

      1    0 000019A0 AdhEngineClose
      2    1 00001790 AdhEngineOpen
      3    2 00002570 AdhGetConfig
      4    3 00002800 AdhGetEvidenceCollectorResult
      5    4 00001EC0 AdhStatusEventSubscribe
      6    5 000022F0 AdhStatusEventUnsubscribe
      7    6 00001120 DllMain

Summary
    1000 .data
    1000 .pdata
    2000 .rdata
    1000 .reloc
    1000 .rsrc
    3000 .text

    after running no files were written

> generator.exe -m trace -p C:\Windows\System32\adhapi.dll
[+] Process completed.

It is working perfectly on several VMs that I have. Do you manage to reach the "Process completed" message when you run the command?
Any particularities about your system (SO version, special restrictions...) that might give me a clue?

Any update on this issue?


How do you pronounce jose?

Am I the only one curious? Can we throw that in the readme? Hope it's not the Spanish pronunciation, that'd be a bit confusing.

I totally pronounce it the Spanish way :)
This is a question best suited for the IETF / JOSE working group. You can find their contact at https://datatracker.ietf.org/wg/jose/charter/.


Problem in pathogen statues graph when increase population

Dear Ira,
First I would like to thank you for your tool which is amazing and very useful. I would like to use it to record a video to raise awareness in my community about the importance of staying home. However, I am trying to set the population number from "this.peopleTally  = 27000;" when I do so and run the simulation, I face a problem with the graph where it draws the colores just at the very top of the graph.
sorry, I am not really a programmer but I have the basic knowledge. so I appreciate your help.
Many thanks

Hi Jihad. I hope you're well and thanks for your efforts to help your community understand the situation.  I understand that you are trying to set the total population in the model to be 27,000 people? The short answer is that I don't think the model can support that many people. However, you might explain that each person in the current model could represent 10 or 20 actual people.

Firstly, any changes to the initial conditions of the model should be made in the file "A_ConfigModel.pde". Variables in this file, as well as "A_ConfigView.pde", are the only ones that are designed to be changed.
Currently, there is no direct way to set the total population, since the total population is calculated from the number of households and their sizes. Households are randomly generated each time the simulation is run. If you would like to adjust the total number of households, or the distribution of people in each household, consider changing the following values in 'A_ConfigModel.pde':

'int N = 1;' on line 103 [i.e. changing to N = 2 will roughly double the population]
'int minDwellingSize = 1;' on line 124 [i.e. min number of people in a household is 1]
'int maxDwellingSize = 4;'  on line 125 [i.e. max number of people in a household is 4]

Unfortunately, though, I don't think the simulation can handle many more people than I'm already generating by default. It's a limitation of Processing.

good luck!


(no ticket) Public typedef of ServiceIf called IfaceType was added into generated ServiceProcessor

Public typedef of ServiceIf called IfaceType was added into generated ServiceProcessor
Client: cpp

This change might be convenient for doing some template magic,
for instance making some kind of ServiceManager or something similar.
A generated SomeServiceProcessor declaration will be look as following:
class SomeServiceProcessor : public ::apache::thrift::TDispatchProcessor {
 // bla-bla-bla
 public:
  typedef SomeServiceIf IfaceType;

  SomeServiceProcessor(::std::shared_ptr<SomeServiceIf> iface) :
  // bla-bla-bla

instead of just
class SomeServiceProcessor : public ::apache::thrift::TDispatchProcessor {
 // bla-bla-bla
 public:
  SomeServiceProcessor(::std::shared_ptr<SomeServiceIf> iface) :
  // bla-bla-bla

[ ] Did you create an Apache Jira ticket?  (not required for trivial changes)
[ ] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[ ] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, add  [skip ci] at the end of your pull request to free up build resources.

To be honest I need an access to iface object too...so maybe it's better just to add something like
class SomeServiceProcessor : public ::apache::thrift::TDispatchProcessor {
 // bla-bla-bla
 public:
  auto getHandler() {
    return iface_;
  }

?


Add override for IncludeXmlComments for non-file-path

Add override for IncludeXmlComments to allow for adding xml comments derived from location other than file path.

Hi @smichtch,
Would you mind accepting the PR to include in your PR back to domaindrivendev?
Thanks.


Generator integration

Combustion works really well for me so far. There are some situations though where I would like to run a generator inside the dummy app and was wondering whether turning this into a Combustion feature might be helpful. For example:

When testing an engine that depends on another engine being installed (i.e. Active Admin).
When the tested engine requires custom setup that can be performed via an install generator.

In both cases, I would prefer not to commit the changes made by the generator, but rather regenerate on every Travis run to ensure the steps lead to a working setup with the current versions of the gems. In development mode, on the other hand, I'd prefer to generate once and reuse git ignored files on subsequent test runs to speed things up.
I could imagine passing a list of generators to Cumbustion.initialize!. But I'm not sure whether those should actually make changes to spec/internals or maybe some other ignored directory.
Do you think a workflow like that might make sense in the context of combustion?

Hi @tf.
I have the engine which depends on active admin.
And in every build i recreate application. But i not use combustion in this engine.  It`s may be useful for you:
https://github.com/artofhuman/activeadmin_settings_cached/blob/master/spec/spec_helper.rb#L19-L21
https://github.com/artofhuman/activeadmin_settings_cached/blob/master/spec/support/rails_template.rb

Hi @artofhuman,
thanks for the example. I am also doing something similar in another non Combustion project. But I was wondering if we might be able to simplify things by having Combustion take care of it.

Tim, this certainly sounds useful. I wonder if it's better to keep it separate from Combustion.initialize, but likely to be invoked directly after that? Combustion.generate perhaps?
The one thing I'm wary of is if these generators require user input. Granted, I'd expect this to generally not be the case, but would be great to handle these situations in case people make mistakes or a generator's behaviour changes unexpectedly.

Hi Pat,
thanks for your answer. Most generators that I'm aware of allow to replace manual input with command line flags of some sort. So that should not be a big issue.
I'm mainly not sure yet, where those generators should output files. spec/internal is almost like a dummy application, but often a lot more minimal - which is good, but might trip up generators which expect to be run in the context of a standard Rails app. Also this would mean mixing generated files, which should best be git ignored, with manually edited files. That does not really feel right.

Hmm, where to output the generated files is a tricky thing. I was thinking in spec/internal, but if you're not committing those files, then yes, that could get confusing. No better solution comes to mind, though…

There could be an option to have an ignored spec/dummy directory which is bootstrapped with the files from spec/internal. That way generators could create files there. I'm not sure though if that could mean a too fundamental departure from the core ideas of Combustion.

It does feel like a stretch - if you're wanting a full app, then maybe the non-Combustion path is the easier one?
That said, I'd be open to Combustion working with both the current approach (minimal app, only what's necessary in spec/internal) or a full app in spec/internal and Combustion just provides a unified interface.
Given you've got some gems that take the non-Combustion path: how do you manage testing against multiple versions of Rails in those situations? Do you generate spec/dummy every time?

Yes. spec/dummy is ignored. Additionally I generate to spec/dummy/rails-4.2.0 etc to allow for multiple environments in development. I do some custom environment variable magic at the moment, where I probably should have used Appraisal instead.
Since everything works at the moment, I would be hesitant to port the app to Combustion even if full app support was a thing. And for the cases where I am using Combustion, I do not have an urgent need for generator integration.
So maybe the best idea is to keep Combustion focused and look into extracting something new from my existing app once I again have the need for the full app scenario.
Thanks for the helpful discussion. Feel free to close this issue.

Great to have the feedback :)


Feature Request: Yeasu Style Mic and speaker

Hello, this is great project. Could you add Yeasu style mic and speaker support? here is the link to it. https://pinoutguide.com/PortableDevices/yaesu_vertex_microphone_pinout.shtml

From what I can tell it functions quite differently. Not sure if it's adaptable that easily..

@skuep actually it's very close to the 2 pin Kenwood connector which has Ground, MIC,SPK,PTT and RX/TX. the Pin 2 (Clone) in the diagram is TX/RX which need a diode to split it to two pins (smart). And Yeasu has package TNC cable CT-91 which convert the one pin to two pins. I built a programming cable using the diagram below with diode and it works perfect.

CT-91


Signing authority models

#288
I apologize in advanced. I decided to do some refactoring so hence the massive file changes. The serializer being in one place was starting to become unruly so I decided to split them by models.
Changelog:

Added SigningAuthorityAssertion Table
Added SigningAuthorityConfirmation Table
Created model for SigningAuthorityAssertion
Replaced the hard-coded assertions with the ones from the table
Checkbox should now call a function to automatically add the field into the state
Refactored Serializer

 Signing Authority | Manage versions of assertions |3|


Settings Rendering Issues of Plugins Not Working Properly

Describe The Bug
I have a content blocker on my machines (StopTheMadness) and have found in 4.50.5 settings for Homebridge MyQ no longer show.  This bug does not exist in 4.50.4.  Issue exists in any version of Homebridge MyQ on 4.50.5.
Disabling StopTheMadness or running in private browsing renders the page correctly.  Also a rollback to 4.50.4 solves the issues as well.
Logs
No logging errors - just UI glitches.

Config
n/a

Homebridge UI Version
4.50.5
Homebridge Version
1.6.1
Node.js Version
18.17.1
Operating System
macOS
Environment Info

[ ] Using Docker?
[ ] Using Hyper-V?
[ ] Using hb-service?

Raspberry Pi Model
None

@dnicolson, any thoughts? seems like this might be from changes in homebridge/plugin-ui-utils.

Sure... Find Movies attached...
This is normal behavior before 4.50.5 and all content blockers disabled.  (Video taken using 4.50.4 and content blockers on)
https://drive.google.com/file/d/1v_qIu4RE8Q7u8e2PZ9tN6rK-FzK4lWPz/view?usp=drive_link
This is the behavior on on 4.50.5 with content blockers on
https://drive.google.com/file/d/1EL09yBfA6Q4LkaS_uf5LAHIJ8OtA292S/view?usp=drive_link

It looks like a JavaScript error or a JavaScript network request is blocked.
Do you see anything in the DevTools/Web Inspector console or any blocked requests in the network tab?

Yes, on the 4.50.5 release I get a JavaScript error: Error while parsing the 'sandbox' attribute: 'allow-downloads' is an invalid sandbox flag.
This error is not generated on 4.50.4

That's odd because that error is apparently benign and also that attribute hasn't been changed in years:
https://github.com/homebridge/homebridge-config-ui-x/blame/b81fa60a2c2c5bc56a940f2426e914cad7df7cea/ui/src/app/core/manage-plugins/custom-plugins/custom-plugins.component.html#L16
Was anything blocked in the network tab? Do you see different results in different browsers?

There is nothing being blocked.. But in 4.50.5 I also get an error in the console log.
[Error] Did not parse stylesheet at 'http://xxx.xxx.xxx.xxx:8080/webkit-masked-url://hidden/' because non CSS MIME types are not allowed in strict mode.
This does not happen in 4.50.4

@dnicolson Thoughts on what would cause this with the changes you made?

Nothing comes to mind, I see that some of the changes are reverted in the beta branch. Can you get @simplytoast1 to test releases with the changes reverted?

I reverted but then reverted that revert.

I wonder if it is a dependency?

It could be. The issue doesn't appear to be reproducible so creating betas with incremental changes might be the only way forward.

reverted to<EMAIL_ADDRESS>@simplytoast1 once the next beta is released can you test it out so we can see if this is related to changes done on that repo?

Any updates on this? @simplytoast1

The issue is still occurring in 1.6.2b0

Hi. I'm having the same problem with the settings of Homebridge UniFi Protect. The settings do not show up when I click settings. I first noticed it on Chrome IOS and have also had the same problem on Chrome on MacOS. Switching to incognito fixes the problem. I also managed to access the settings through Safari. I noticed that if I downgrade below HBUP V13, the problem goes away and comes back if I update to HBUP V13 and up. I also tried downgrading HBUI all the way down to 4.43.1 but it made no change. Hope this helps.

Hi @maximuswallis
Just tried a combination myself of latest macos (14.0), chrome (117.0.5938.149), unifi-protect (6.15.1) and homebridge-ui (most recent beta).
Although I don't have devices for this plugin, I downloaded it out of interest and the settings screen loads for me with no issue.
Are you still having trouble with this? Could you try updating to the most recent beta of the Homebridge UI to check if you still get the issue?

https://github.com/homebridge/homebridge-config-ui-x/wiki/How-To-Test-Upcoming-Changes

@bwp91 or @donavanbecker can you please release a new beta that includes https://github.com/homebridge/plugin-ui-utils/pull/14?
Thanks!

@dnicolson Will do

@dnicolson 4.50.7-test.18 should have changes.

@jarz have you tried the latest beta of Homebridge UI?

Thanks for the pointer @donavanbecker -- the beta release fixes it.

Hi @maximuswallis
Just tried a combination myself of latest macos (14.0), chrome (117.0.5938.149), unifi-protect (6.15.1) and homebridge-ui (most recent beta).
Although I don't have devices for this plugin, I downloaded it out of interest and the settings screen loads for me with no issue.
Are you still having trouble with this? Could you try updating to the most recent beta of the Homebridge UI to check if you still get the issue?

https://github.com/homebridge/homebridge-config-ui-x/wiki/How-To-Test-Upcoming-Changes

Hi, I tried updating to the latest beta '4.51.0-test.2' and also 'v4.50.7-test.18', but unfortunately neither fixed the problem for me. Wish I had better news for you! (By the way, I am running Mac OS Catalina - not sure if that is relevant but throwing it out there since you mentioned you are running Sonoma)

Is this still an issue? on the most recent beta version of the UI?

Closing due to no response


Back to top button

Add back to top into htmlcss guide file

@googlebot I signed it!

Thanks for the suggestion, but I’m unfortunately unable to accept this pull request because the markup is automatically generated from a shared template we maintain internally.

Is there any way to help you with maintenance?

Not with the markup used by the style guides, but thanks for showing an interest!


[RESTEASY-1680] Making testMultipleDataFields more robust

@jimma , this is to prevent issues with events sometimes not delivered in order

@jimma , I'm merging this, considering it's just a test and it's affecting (failing) many other PRs. Feel free to comment anyway, if you have anything to add.


Fix clusterInfo in development_mysql.yaml

for https://github.com/uber/cadence/issues/1498

Coverage increased (+0.02%) to 68.97% when pulling 6941b0a2365b9eb9ad9a118fe52510090bb43eb3 on longquanzheng:clusterInfo into 12f051efcd63fc6b25bcc21fd3679bbb5ac86260 on uber:master.


bin/test doesn't properly interrupt on CTRL+C

Description
Reproduction steps
Interrupting tests with CTRL+C does not always result in tests stopping executing: Here's a video of what I'm talking about:
Screencast_20240311_170109.webm
Even though I terminated very early during the test run it still ran all tests: 15371 tests, 6490 assertions, 0 failures, 0 errors, 8 skips. Sometimes it properly exits but most of the time it doesn't.
Thread.new and Thread::Queue.new seem suspicious but I'm still getting this when I remove these usages.

I've noticed this too, but it's rarely a problem so haven't spent time looking into it. If you're able to narrow down the cause then it's probably an easy fix.


No --load-from in training [options]

Hi, Thanks for your work
I have read getting start carefully.
It writes that --resume-from is different with --load-from.
However, I cannot find the '--load-from' in the training options. And it would meet error when using --load-from
Thanks a lot for your help

Ok，it can be solved by add load-from in the train.py manually


Scrolling inertia does not work

Most mobile apps support scrolling with "inertia" so that the page keeps scrolling if you flick it up or down.  This doesn't work on the 5calls app.

On it! Thanks.

@dektar I got this done. I am making a PR now.

Oh, I just did it!

Fixed in https://github.com/5calls/android/commit/04b2941c792bd9c61dd9b717223c6d85e3685ea8. @levibostian let me know if you want to work on something else!


Tooltip model argument of custom callback is undefined

Expected Behavior
Configuring the custom callback on the tooltips options and logging the tooltip model, which is the argument of callback, I'm expecting to see the object.
Current Behavior
When I log the tooltip model instance, passed as argument to custom callback, the instance is undefined.
Steps to Reproduce (for bugs)
https://codepen.io/stockinail/pen/mdVrzWQ
Environment

Chart.js version: dist/master
Browser name and version: FF 77.0.1

I also noticed this. You can access the toolTipModel from this though.

I also noticed this. You can access the toolTipModel from this though.

@kurkle this is a workaround, isn't it? Or does the custom callback receive the model from this and not by the argument anymore?
The this does not look like the tooltip model, described into the doc, even if there are all properties (some are inside options).


[@angular/cdk] feature request. Native inputs wrappers

Bug, feature request, or proposal:
Feature request.
What is the expected behavior?
Quite often devs need to create custom checkboxes, radios etc. with own styles. Angular material has own wrappers for native inputs, it allows to customize it, add animation, have accessibility and so on. I think many devs create own custom form controls for this, and this is really exhausting.
I had own wrappers but it was difficult to make it work like native input, so I downloaded @angular/material and extended my wrappers with material's form controls, but with own templates and styles. It gave me my own customized form controls which support API from material. But it requires to have installed @angular/material and some redundant dependencies into my code like material animation etc. So I completly removed angular/material, copied only needed code from the material's form controls without any animation, utilized only @angular/cdk and it works fine, with full API and accessibility.
Can we have some generic input wrappers, which will allow devs to easily customize it? So devs won't have headache when develop custom form controls for native inputs, like preventing DOM events, accessibility etc. which is common
What is the current behavior?
To create own wrapper for native form control with accessibility and events, devs have to write almost the same code as in the angular material's wrappers.

It sounds like something I would like to do, but I probably won't have time to work on it for a while

Closing as a dup of https://github.com/angular/components/issues/7891 which has a lot of discussion on the topic


An error occured while server rendering /index.html:  require is not defined

Please provide the environment you discovered this bug in.
Windows Subsystem Linux (WSL2)
Which area/package is the issue in?
platform
Description
I have just created a brand new NX application, and running it first time is giving me the following error:

Please provide the exception or error you saw
[vite] Error when evaluating SSR module /<EMAIL_ADDRESS>|- ReferenceError: require is not defined
    at eval (/@fs/home/mw/projects/allthingstypescript/node_modules/.pnpm/xhr2@0.2.1/node_modules/xhr2/lib/xhr2.js:121:3)
    at eval (/@fs/home/mw/projects/allthingstypescript/node_modules/.pnpm/xhr2@0.2.1/node_modules/xhr2/lib/xhr2.js:1246:4)
    at instantiateModule (file:///home/mw/projects/allthingstypescript/node_modules/.pnpm/vite@4.3.1_@types+node@16.11.7_less@4.1.3_sass@1.62.1_stylus@0.55.0_terser@5.17.1/node_modules/vite/dist/node/chunks/dep-24daf00c.js:54351:15)

Other information
No response
I would be willing to submit a PR to fix this issue

[X] Yes
[ ] No

Try disabling SSR and see if that helps, or adding xhr2 to the vite config

How do I disable SSR, the second option not working

@mainawycliffe looks like you're using pnpm. I was too, and had the same issue. The error does not occur with npm.

You can disable SSR in the analog plugin
https://analogjs.org/docs/features/server/server-side-rendering#disabling-ssr

@mainawycliffe looks like you're using pnpm. I was too, and had the same issue. The error does not occur with npm.

Yeah, I am. Seems like the look of all my problems, will try without and report back.

Re-open if you're still having issues


ipp-usb only for systemd?

Hello,
I was successful to use ipp-usb when I boot Linux with systemd, but not when I boot with sysVinit.
Does ipp-usb not support sysVinit?
Thank you.

Hi!
It should work with sysVinit, but somebody needs to write appropriate scripts.
ipp-usb has two modes of operations:

udev mode

ipp-usb udev

In this mode ipp-usb assumes that it will be started every time when compatible device is plugged into the system, and automatically exits when all compatible devices are unplugged.

standalone mode

ipp-usb standalone

In this mode, ipp-usb runs forever, and automatically handles PnP events by itself, serving all connected compatible devices.
Both of these modes can be used with -bg flag, which tells ipp-usb to deamonize itself.

Thank you.
Do I understand right, that until now both the "udev mode" and the "standalone mode" do not work with sysVinit, because appropriate scripts do not exist until now?
Or is there a combination of "mode" and "flag" that works without "the missing appropriate scripts"? If yes, which combination works without "the missing appropriate scripts"?
Thank you.

Other question:
When I have run the following command:
sudo ipp-usb standalone -bg
Then driverless printing is active and it works.
Problem: When I then start my VM I have no access to this printer.
Question: How can I then deactivate my printer in the host? I am looking for something like the opposite of the command "sudo ipp-usb standalone -bg".
I have not found anything for this in
man ipp-usb
There I have only found options to activate it, but not how to inactivate it.
Thank you.
Edit: If there is another way to free up the printer for the VM, then I am also open for that solution.


Add tests and a git pre-push hook

The tests don't cover much of the code, they're mostly there just as a starting point.
Note that each developer has to manually edit the hook in .git/hooks/pre-push if they want pre-push.sh to be run before pushing.

@riej All done! Not sure if the GitHub action configuration will work, though...

I figured it out thanks to your gradle.yml file from 4 months ago. I had to add pull requests to it, along with Java 11.


Injecting/Mentioning player name via [playerNum0] in the in-game chat doesn't work

During developing a plugin that offers Player Username even on LAN lobby, after testing compatibility with BiggerLobby, I noticed that my plugin and even the game can't refer/mention the lobby hoster (Player #0) in the chat, this issue is still valid on setup with only LC_API + BiggerLobby enabled.
Here are screenshots showing what happened if I try to mention Player #1, Player #0, and Player #2 on the chat system:

As visible on the picture, it seems like the Player #0 is not evaluated to an actual player username, which in my case is TIRTAGT.

After forking and checking BiggerLobby's source code, the issue seems to be just because of wrong starting array Index on Patches/InternalPatches.cs line 38
I patched the code, and here's the result doing the above test again:

Issue fixed, As the needed changes to fix the issue is minimal, I'm not sure whether or not to open a Pull Request for this, so I've open up an issue instead.
Let me know if I need to open a Pull Request for the fix.

appreciate it, I went ahead and fully rewrote the method since it was causing problems with LethalExpansion
should be fixed in 2.7.0


BUG - archive name with timestamp producing archive with timestamp appended

Description
Found when testing https://github.com/SAP/open-ux-tools/issues/687
Appending a timestamp to the archive name should be a env/config variable. Archives generated with the timestamp will cause issues for existing clients who are not expecting or wanting to generate archives with a new naming convention.
Steps to Reproduce
Generate a project for ABAP deployment
./bin/deploy --name ZJLONG0205 --description 'Deployment Cataog' --package '$TMP' --archive-folder /Users/testprojectjl0205/dist --url http://testsite.com:50000 --client 100 --verbose true --test true

Expected results
The archive generated is archive.zip.
Actual results
Generated archive is archive-1684321871711.zip.
Screenshots
If applicable, add screenshots to help explain the problem.
Version/Components/Environment
Add any other context about the problem here
OS:

[x] Mac OS
[ ] Windows
[ ] Other

Root Cause Analysis
Problem
{describe the problem}
Fix
{describe the fix}
Why was it missed
{Some explanation why this issue might have been missed during normal development/testing cycle}
How can we avoid this
{if we don’t want to see this type of issues anymore what we should do to prevent}

Root Cause Analysis
Problem
Then the keep flag was enabled, the generated archive appends a timestamp.
Fix
Remove the timestamp from the name of the arhive name.
Why was it missed
Issue with existing flow which assume the generated archive is always archive.zip. This will support existing behaviour in third party tools.
How can we avoid this
Continue testing and ensuring existing behaviour is maintained.


SafeMath to use Outvars Again

Changes
I measured a regression in add and sub when updating SafeMath from openzeppelin-solidity version 1.9.0 to 2.0.1.
The regression went away when I reverted the outvar change and made c an outvar again.
The regression was measured on solc 0.4.24 for an upcoming TrustToken smart contract, and seemed to amount to 5 gas per call.
This patches the regression by switching back to outvars.
Reviewers @frangio @nventuro

I should also note that our solc runs parameter was 20000 instead of the default 200.

I have verified the optimization with solc 0.4.24 on Rinkeby.
Contract
Using Outvars
Using return
For 100 calls, a difference of 478 gas.

Also verified on solc 0.5.5
Contract
Using Outvars
Using return
For 100 calls, a difference of 878.

Profiling also indicates a regression in return gas for solc 0.5.* compared to 0.4.24, yet another reason not to update.

Wow nice

Hello @wjmelements, thank you for your thorough testing! It is not often that we get actual verifiable measurements done in a proper way when discussing gas costs. Indeed, using outvars as opposed to declaring a local variable and returning it does seem to produce less efficient code.
However, I'm afraid our response will be the same as in https://github.com/OpenZeppelin/openzeppelin-solidity/pull/1635: there's no clear reason why that method yields better result than ours, and why the codegen emits bytecode that the optimizer isn't able to make as succinct as in the outvars case. It may very well be the case that a Solidity update changes the codegen internals, causing outvars to become more expensive, forcing us to revert this change, and we don't want to go through that.
Instead, we should forward this feedback to the Solidity development team: the issue I mentioned previously derived in https://github.com/ethereum/solidity/issues/6075, which led to https://github.com/ethereum/solidity/pull/6213: a new compiler optimization rule that was included in v0.5.6, released a couple hours ago.

You might consider keeping this change for the 2.0.* branch since it will keep tracking 0.4.24. If you aren't planning to support 0.4.24, or optimizations in general, we will fork.

I understand your reasoning though. Thanks for filing the report to solc.


Add partial upgrade script

Works for consul and nomad-server.
Basically it terminates N - simple_majority(N) each time, and then check that the new instances are able to connect back to their respective services (by checking the set of AWS instances and service specific instances to be the same), before continuing.

Superseded by #213


strange exhaustion with packet buffer enqueue?

I'm receiving two alternating messages (start and stop) on a UDP socket. The third cannot be processed. I added some logging to PacketBuffer::enqueue, seen in the debug output here:
[DEBUG] full: false, window: 6000, contig_window: 6000, size: 5
[DEBUG] full: false, window: 6000, contig_window: 5995, size: 4
[DEBUG] full: false, window: 6000, contig_window: 5996, size: 5
[DEBUG] cannot process ingress packet: buffer space exhausted

The third packet triggers the condition here:
        if window < size || (window != contig_window && window - contig_window < size) {
            return Err(Error::Exhausted)
        }

because window - contig_window == 4 < size == 5. But as far as I understand, having a contiguous window larger than size cannot be an exhaustion condition?

Which version of smoltcp?

@dlrobertson Can you please take a look? Something's screwy with that condition.

Yup, that logic seems off...

FWIW, it's revision 216c393 (the switch to managed 0.7 breaks with my pegged nightly).

@birkenfeld thanks for pointing this out. The linked PR fixes this.


Implement sample from ModeResult

Finally found the source of lppd mismatch with the book - during sampling I did independent sampling, but need to use full covariance matrix. It lead to different samples and, as a result, discrepancy in LPPD values computed in chapter 7.
This code implements sampling from ModeResult and produce Chains object.

Yes, this is the evolution of this code :). Doing sampling in the loop was a big mistake, though.


Draft: Template scrap news example

First iteration for example that it scrap news and save in  jina as a retrieve search engine.

It is a little weird this example, can you clarify the intention of it? I see there are like 2 ways to index? I would prefer to focus it in one single problem (maybe this Article link can be interesting)

It is a little weird this example, can you clarify the intention of it? I see there are like 2 ways to index? I would prefer to focus it in one single problem (maybe this Article link can be interesting)

Thanks for your example @carlosb1!
Since we are restructuring our examples repo now, do you think it would be better if @carlosb1 hosted this in his own repo @JoanFM ? I'll add a section to our readme for community examples and link it from there

Thanks for your example @carlosb1!
Since we are restructuring our examples repo now, do you think it would be better if @carlosb1 hosted this in his own repo @JoanFM ? I'll add a section to our readme for community examples and link it from there

Thanks for your example @carlosb1!
Since we are restructuring our examples repo now, do you think it would be better if @carlosb1 hosted this in their own repo @JoanFM ? I'll add a section to our readme for community examples and link it from there

I did some modifications (I removed not used functions). However, if you prefer move my minimal example, ok for me.

Thanks for your example @carlosb1!
Since we are restructuring our examples repo now, do you think it would be better if @carlosb1 hosted this in their own repo @JoanFM ? I'll add a section to our readme for community examples and link it from there

I did some modifications (I removed not used functions). However, if you prefer move my minimal example, ok for me.

Thanks for your example @carlosb1!
Since we are restructuring our examples repo now, do you think it would be better if @carlosb1 hosted this in their own repo @JoanFM ? I'll add a section to our readme for community examples and link it from there

I did some modifications (I removed unused functions). However, if you prefer move my minimal example, ok for me.

Yes we should move it and link it in our repo.
As per the content, is it just indexing the title of the articles?

Thanks for your example @carlosb1!
Since we are restructuring our examples repo now, do you think it would be better if @carlosb1 hosted this in their own repo @JoanFM ? I'll add a section to our readme for community examples and link it from there

I did some modifications (I removed unused functions). However, if you prefer move my minimal example, ok for me.

Yes we should move it and link it in our repo.
As per the content, is it just indexing the title of the articles?

Thanks for your example @carlosb1!
Since we are restructuring our examples repo now, do you think it would be better if @carlosb1 hosted this in their own repo @JoanFM ? I'll add a section to our readme for community examples and link it from there

I did some modifications (I removed unused functions). However, if you prefer move my minimal example, ok for me.

Yes we should move it and link it in our repo.
As per the content, is it just indexing the title of the articles?

yes.... my idea is the creation of an incremental example while I understand better the library.

Thanks for your example @carlosb1!
Since we are restructuring our examples repo now, do you think it would be better if @carlosb1 hosted this in their own repo @JoanFM ? I'll add a section to our readme for community examples and link it from there

I did some modifications (I removed unused functions). However, if you prefer move my minimal example, ok for me.

Yes we should move it and link it in our repo.
As per the content, is it just indexing the title of the articles?

yes.... my idea is the creation of an incremental example while I understand better the library.

Hey @carlosb1, feel free to make PR to our examples repo README when you're done. Just add your example under community examples

Hey @carlosb1, feel free to make PR to our examples repo README when you're done. Just add your example under community examples


Create a way to add items to a playlist.

This is going to rock! Currently it kind of sucks in Plex, because you have to find the item, and then right click to it add to it a playlist
We could have it so that you select a playlist, and then have a 'quick add' button which allows you to search for a track to add.
We should also have a 'Add to Playlist' option in the track context menu.

There is currently a context menu on list items to 'Add to Playlist...' and also 'Remove from Playlist'.
I still want to build a quick add option - as well as a way to drag/drop playlist items to sort them.


Do Unifying Owner Telescope

Are you sure this will work

For code:
open data Nat | O | S Nat
open data Vec (n : Nat) (A : Type)
| O, A => vnil
| S n, A => vcons A (Vec n A)

def what => vnil {Nat}
def how => vcons {Nat} O vnil

Before fix:

After fix:

Also, CtorTelescope wasn't used anywhere except ExprTycker, so it is safe to modify it.

bors r= @ice1000 @imkiva
Proof accepted!!


Clarification for page content

Dears,
I find it hard to understand the information provided in this page and what actually benefit I can gain from it from planning perspective.

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

ID: a4b9ed32-e3d4-109a-130b-f6cb12f9cbfb
Version Independent ID: ab9abc48-1f0f-330e-0153-27205cffca88
Content: Approaches to digital estate planning - Cloud Adoption Framework
Content Source: docs/digital-estate/approach.md
Service: caf
Sub-service: caf-methodology-plan
GitHub Login: @martinekuan
Microsoft Alias: martinek

Can you provide additional context?  If you still require assistance, please re-open with additional context.


Feature: Support Presto

Support for Presto as datasource.

Thanks! This is amazing and much appreciated.


Dami/rpc update

Codecov Report

Merging #200 (d7f5f2b) into main (d6eb7d8) will increase coverage by 0.04%.
The diff coverage is 6.25%.

@@            Coverage Diff             @@
##             main     #200      +/-   ##
==========================================
+ Coverage   63.38%   63.43%   +0.04%
==========================================
  Files         104      104
  Lines        7476     7447      -29
==========================================
- Hits         4739     4724      -15
+ Misses       2737     2723      -14

Impacted Files
Coverage Δ

node/src/rpc.rs
0.00% <0.00%> (ø)

pallets/linkable-tree/rpc/src/lib.rs
0.00% <0.00%> (ø)

pallets/mt/rpc/src/lib.rs
0.00% <ø> (ø)

standalone/node/src/chain_spec.rs
0.00% <ø> (ø)

standalone/node/src/rpc.rs
0.00% <0.00%> (ø)

standalone/runtime/src/lib.rs
0.00% <ø> (ø)

pallets/linkable-tree/src/lib.rs
91.15% <100.00%> (ø)

node/src/main.rs
0.00% <0.00%> (ø)

primitives/src/utils.rs

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 d6eb7d8...d7f5f2b. Read the comment docs.


Added the support of debug commands in the remote controller.

Hello,
I've just added a debug command in the remote controller that would allow us to send 'sc_pb.RequestDebug' requests and execute debug actions. We use this command extensively for our recent paper published at ICML 2018 (https://arxiv.org/pdf/1803.11485.pdf). This command would allow us to have our code consistent with changes from your side. Also, others may also find this feature useful.
Many thanks,
Mikayel Samvelyan

Merged in 30ba837


New implementation parallel joins

This changes the way fetches are inspected to parallelize independent fetches. It does this by adding a new Join construct to the Fetch structure. This Join is parallelized in a new intermediate interpreter which inspects the actual Free constructs and replaces cached fetches by Free.Pure.
It does play a bit fast and loose with the laws with this fiddling with Free under the hood, but it now executes code in a fetch.map(...) or a fetch.flatMap(...) only once.
(The other PR updating cats and adding Scala 2.12 support should be merged first).

Current coverage is 76.03% (diff: 78.94%)

Merging #84 into master will increase coverage by 0.15%

@@             master        #84   diff @@
==========================================
  Files             8         10     +2
  Lines           228        242    +14
  Methods         226        240    +14
  Messages          0          0
  Branches          2          2
==========================================
+ Hits            173        184    +11
- Misses           55         58     +3
  Partials          0          0

Powered by Codecov. Last update 45ce3fa...d5187cd

I'll rebase this on the current master.

LGTM :shipit:


Add full JSON validation instead of sanity check

I think these two lines may become common source of errors if white space included in beginning or end of the JSON string:
eosio_assert(json[0] == '{',             "payload must be json");
eosio_assert(json[json.size()-1] == '}', "payload must be json");
Since you are not doing true JSON validation - you may as well remove those lines.
It would be good to include full JSON validation - however that will require adding a new dependency to your contract file:
There are many JSON validation libraries available. We need to check if EOSIO code is already using one and reuse the same one.

White space shouldn't be included in front or behind the payload anyways. Otherwise you have a pretty poorly generated JSON file. Can you give me an example of a JSON generator that includes whitespace in front or behind the {}?
The checks are simple ones to remind the user if the payload is clearly not a JSON file. Doing a full JSON parse of the data would consume a fair amount of CPU on-chain. This is not meant to be a stringent check, only a reminder -- validation of conforming the BP standard JSON should be done client side, as different entities may have relatively strict or loose requirements for validity.
I'll look into a full JSON validation, and see how much CPU that uses.

That library is not a contract library, it's part of the nodeos and cleos code, by the way. A full JSON parser would need to be included in the contract.
Note that the eos forum contract is doing a similar sanity check, and not a full JSON parse as well:
https://github.com/eoscanada/eosio.forum/blob/master/src/forum.cpp#L34

I agree, closing this issue.


latest master: 6 tests fail - ubuntu

Operation System: ubuntu 17.04
Beaker Version: 0.6.1 ... master: https://github.com/beakerbrowser/beaker/commit/49bf3502ddf985ec51d41aebb1d9f00ef724e9e7
[mario:~/git/beaker/tests] master(810)+ 2h31m14s ±
-> npm test

><EMAIL_ADDRESS>test /home/mario/git/beaker/tests
> ava ./*-test.js -s

[
  "User data path set by environment variables",
  "userData: /tmp/beaker-test-UXI95u"
]
WARNING: the "keys" command will be depcrecated soon. Please use a different command in order to avoid failures in your test after updating WebdriverIO.
WARNING: the "keys" command will be depcrecated soon. Please use a different command in order to avoid failures in your test after updating WebdriverIO.
WARNING: the "keys" command will be depcrecated soon. Please use a different command in order to avoid failures in your test after updating WebdriverIO.
WARNING: the "keys" command will be depcrecated soon. Please use a different command in order to avoid failures in your test after updating WebdriverIO.
(node:14498) DeprecationWarning: DEPRECATION NOTICE: ok() has been renamed to truthy() and will eventually be removed. See https://github.com/avajs/ava-codemods to help upgrade your codebase automatically.
WARNING: the "keys" command will be depcrecated soon. Please use a different command in order to avoid failures in your test after updating WebdriverIO.
[
  "User data path set by environment variables",
  "userData: /tmp/beaker-test-0WwWdy",
  "(node:14789) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 append listeners added. Use emitter.setMaxListeners() to increase limit",
  "(node:14789) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 error listeners added. Use emitter.setMaxListeners() to increase limit"
]
WARNING: the "keys" command will be depcrecated soon. Please use a different command in order to avoid failures in your test after updating WebdriverIO.
WARNING: the "keys" command will be depcrecated soon. Please use a different command in order to avoid failures in your test after updating WebdriverIO.
WARNING: the "keys" command will be depcrecated soon. Please use a different command in order to avoid failures in your test after updating WebdriverIO.
WARNING: the "keys" command will be depcrecated soon. Please use a different command in order to avoid failures in your test after updating WebdriverIO.
WARNING: the "keys" command will be depcrecated soon. Please use a different command in order to avoid failures in your test after updating WebdriverIO.
[ 'User data path set by environment variables',
  'userData: /tmp/beaker-test-I1xt4u' ]

   36 passed
   6 failed

   1. dat-archive-web-api-test › archive.writeFile

  t.falsy(res.value)
              |
              Object{name:"QuotaExceededError",quotaExceeded:true}

      Object{name:"QuotaExceededError",quotaExceeded:true}
        _callee13$ (dat-archive-web-api-test.js:386:7)
        tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
        GeneratorFunctionPrototype.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
        GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
        process._tickDomainCallback (internal/process/next_tick.js:135:7)

   2. dat-archive-web-api-test › archive.writeFile writes to subdirectories

  t.falsy(res.value)
              |
              Object{name:"QuotaExceededError",quotaExceeded:true}

      Object{name:"QuotaExceededError",quotaExceeded:true}
        _callee19$ (dat-archive-web-api-test.js:496:5)
        tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
        GeneratorFunctionPrototype.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
        GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
        process._tickDomainCallback (internal/process/next_tick.js:135:7)

   3. dat-archive-web-api-test › archive.writeFile doesnt overwrite folders

  t.deepEqual(res.value.name, 'EntryAlreadyExistsError')
                        |
                        "QuotaExceededError"

      "QuotaExceededError"
        _callee20$ (dat-archive-web-api-test.js:526:5)
        tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
        GeneratorFunctionPrototype.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
        GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
        process._tickDomainCallback (internal/process/next_tick.js:135:7)

   4. dat-archive-web-api-test › archive.commit does allow you to exceed the quota, but subsequent writes will fail
   write file accepted
  t.falsy(res.value, 'write file accepted')
              |
              Object{name:"QuotaExceededError",quotaExceeded:true}

      Object{name:"QuotaExceededError",quotaExceeded:true}
        _callee25$ (dat-archive-web-api-test.js:674:5)
        tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
        GeneratorFunctionPrototype.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
        GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
        process._tickDomainCallback (internal/process/next_tick.js:135:7)

   5. dat-archive-web-api-test › archive.createFileActivityStream

  t.deepEqual(res.value, ['/a.txt', '/b.txt', '/a.txt', '/a.txt', '/b.txt', '/c.txt'])
                  |
                  []

      []
        _callee33$ (dat-archive-web-api-test.js:1261:5)
        tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
        GeneratorFunctionPrototype.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
        GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
        process._tickDomainCallback (internal/process/next_tick.js:135:7)

   6. dat-archive-web-api-test › archive.createNetworkActivityStream

  t.deepEqual(res.value.content.all, true)
                                |
                                false

      false
        _callee34$ (dat-archive-web-api-test.js:1316:5)
        tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
        GeneratorFunctionPrototype.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
        GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
        process._tickDomainCallback (internal/process/next_tick.js:135:7)
npm ERR! Test failed.  See above for more details.

Looks like some issues with file-watching and disk-size calculation. If anybody wants to submit PRs, that'd be great.

Stood up a linux box and did some debugging. I've pushed updates which resolve the disk-size problem. Turns out that OSX was reporting real size, and linux was reporting size-on-disk. Both now report size-on-disk.
Still problematic:

The linux tests are extremely flakey. There are random timeouts in multiple different areas.
File-watching is not fixed yet

thx for the info. .. did run the updated version. .... only 2 tests fail. ...
  42 passed
   2 failed

   1. dat-archive-web-api-test › archive.createFileActivityStream

  t.deepEqual(res.value, ['/a.txt', '/b.txt', '/a.txt', '/a.txt', '/b.txt', '/c.txt'])
                  |
                  ["/a.txt","/a.txt","/b.txt","/b.txt","/a.txt","/a.txt","/a.txt","/a.txt","/b.txt","/b.txt","/c.txt","/c.txt"]

      ["/a.txt","/a.txt","/b.txt","/b.txt","/a.txt","/a.txt","/a.txt","/a.txt","/b.txt","/b.txt","/c.txt","/c.txt"]
        _callee35$ (dat-archive-web-api-test.js:1187:5)
        tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
        GeneratorFunctionPrototype.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
        GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
        process._tickDomainCallback (internal/process/next_tick.js:135:7)

   2. dat-archive-web-api-test › archive.createNetworkActivityStream

  t.deepEqual(res.value.content.all, true)
                                |
                                false

      false
        _callee36$ (dat-archive-web-api-test.js:1242:5)
        tryCatch (node_modules/regenerator-runtime/runtime.js:65:40)
        GeneratorFunctionPrototype.invoke [as _invoke] (node_modules/regenerator-runtime/runtime.js:303:22)
        GeneratorFunctionPrototype.prototype.(anonymous function) [as next] (node_modules/regenerator-runtime/runtime.js:117:21)
        process._tickDomainCallback (internal/process/next_tick.js:135:7)
npm ERR! Test failed.  See above for more details.

If you pull latest and burnthemall, you should have all tests passing except for one (the createNetworkActivityStream test, which fails for all platforms). If so, I think we can close this.
Thanks for reporting!

tested. -> as you wrote,  I'll close the issue, thx


Cookies on response from POST

Hi,
Is is possible with your library to actually get cookies from a response? See here for an example of the problem I'm trying to solve.
https://github.com/Alamofire/Alamofire/issues/946
Thanks

Hi,
With a simple edit in the

class CookieInterceptor: NSObject, ResponseInterceptor {

    func process<ReturnType>(endpoint: Endpoint<ReturnType>, response: NSHTTPURLResponse?, responseObject: ResponseObject) -> ProcessResults {
        print(response)

        for cookie in NSHTTPCookieStorage.sharedHTTPCookieStorage().cookiesForURL(response!.URL!)! {
            print(cookie)
        }
        return ProcessResults(true, nil)
    }
}

An interceptor like this should print the cookies, you would need to be pointing your build against the dev branch however.
This is tested against https://httpbin.org/cookies/set?k2=v2&k1=v1


writeTodos() should ensure .lint-todo dir does not exist if no errors are present

writeTodos() current behavior is to always create a .lint-todo dir.
Proposal: if no errors are present:

it shouldn't create a .lint-todo
it should also ensure .lint-todo is deleted.

Closing since .lint-todo dir should actually be expected once authors have opted in to use todos.