Adding invalid implementation for a function creates error messages in other parts of the code

I've been running into a weird error while implementing a decoder. Below is the code simplified as far as I can while still triggering the error:
interface Foo
    exposes [validFn]
    imports []

invalidFn : {} -> *
# invalidFn = \_ -> validFn {} (\_ -> Foo)

validFn : a, (a -> [Foo]) -> a
validFn = \s, toMaybe ->
    h : a -> a
    h = \x ->
        when toMaybe x is
            Foo -> invalidFn {}

    h s

This compilers. Now uncomment the invalidFn line, and you get the following errors:
>   roc test Foo.roc

── TYPE MISMATCH in package/Rocon.roc ──────────────────────────────────────────

This expression is used in an unexpected way:

6│  invalidFn = \_ -> validFn {} (\_ -> Foo)
                      ^^^^^^^^^^^^^^^^^^^^^^

This validFn call produces:

    {}

But you are trying to use it as:

    *

Tip: The type annotation uses the type variable * to say that this
definition can produce any type of value. But in the body I see that
it will only produce a record value of a single specific type. Maybe
change the type annotation to be more specific? Maybe change the code
to be more general?

── TYPE MISMATCH in package/Rocon.roc ──────────────────────────────────────────

This 1st argument to toMaybe has an unexpected type:

12│          when toMaybe x is
                          ^

This x value is a:

    a

But toMaybe needs its 1st argument to be:

    a

────────────────────────────────────────────────────────────────────────────────

2 errors and 0 warnings found in 28 ms

That first error is helpful, but the second one I don't think is:

The error message is saying Roc is expecting an a and getting an a, so what's the problem?
It's weird that adding an implementation for one function creates a compiler error in an entirely different function.
In my original case I got 34 extra errors in various seemingly unrelated parts of the code instead of just the one here, and those 34 burried the useful error 😅.

Thanks for this minimal reproduction @jwoudenberg! This is an interesting one :)

The error message is saying Roc is expecting an a and getting an a, so what's the problem?

The a is scoped to one definition, not to the entire function or file, so those are not inherently the same a. That being said, I don't know why that mismatch occurs :smile: I'll open a new issue for that.

It's weird that adding an implementation for one function creates a compiler error in an entirely different function.

Note that you did not just add an implementation here. The type does not match the implementation, so I can see how this changes things when the compiler tries to make the implementation work.

In my original case I got 34 extra errors in various seemingly unrelated parts of the code instead of just the one here, and those 34 burried the useful error

I also think these were caused by the compiler trying to make the implementation work with a different type then before.

Can we close this in favor of #6595 @jwoudenberg?

Sure, thank you!

Hey @Anton-4 , quick follow up question to your comments:

Note that you did not just add an implementation here. The type does not match the implementation, so I can see how this changes things when the compiler tries to make the implementation work.

I also think these were caused by the compiler trying to make the implementation work with a different type then before.

Do you mean to say that the compiler will infer the correct type for invalidFn which is different from the one I specified, and that inferred type then causes a type-error in validFn?

Do you mean to say that the compiler will infer the correct type for invalidFn which is different from the one I specified, and that inferred type then causes a type-error in validFn?

I believe that is the case, I'd have to do some investigating to confirm that.

If I try with this simple code sample:
interface Foo
    exposes [foo]
    imports []

foo : U8 -> U8
foo = \_ -> {}

bar : U8
bar = foo 4

Then I just get a compile error for foo. The compiler doesn't give an additional compiler error in bar, which uses foo correctly according to its type signature, but not according to its implementation.
This behavior is matches my expectations too. I often add type signatures to contain type errors in a bit of code, making sure that if I make an error in one place it doesn't result in a type error in a completely different bit of code. That does appear to be happening here.
Going to reopen in that light, but please let me know if you disagree!

Interesting! Yeah, this warrants further investigation.


Issue #893: Adds api on local client for getCallset and searchCallset

In addition to adding these methods, I went through and standardardize the way callSet was referenced -- in some places it was callset and in others it was callSet. It seems like the apis are usually capitalizing Set, so I went with that convention.

+1. Nice catch on the call(S|s)et stuff.

+1


Support for PBR

Blender now supports PBR extensions as per: https://en.wikipedia.org/wiki/Wavefront_.obj_file#Physically-based_Rendering
Sure would be nice to have them here too.

LGTM


Add Support for Changeable Font Sizes

This PR adds a Boolean flag, 'customizableFontSize', which allows the Badge to optionally be resized with a call to UIFont *countLabelFont. Additional customizable feature, while not affecting default library behavior.

I like this idea (didn't realize it wasn't default), but the implementation is a little complicated.  I'm just going to let the user fully change the font on line 209.  Thanks!

Sounds good - Thanks


I want to know how long to train a model. I have only a GPU 3090ti .

So I want to know time to train a model. For example , trained 50 epoch may spend how much time.

I am not sure. You can have a try.


fixed bug in unfolding

Fixed an unfolding bug in some Beethoven Sonatas. Extended musicxml import support of "fine" encodings. Added part.segments property.

Codecov Report
Patch coverage: 83.33% and project coverage change: -0.03 :warning:

Comparison is base (5810f2a) 84.83% compared to head (0a78217) 84.80%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #209      +/-   ##
===========================================
- Coverage    84.83%   84.80%   -0.03%
===========================================
  Files           69       69
  Lines        12204    12215      +11
===========================================
+ Hits         10353    10359       +6
- Misses        1851     1856       +5

Impacted Files
Coverage Δ

partitura/score.py
79.33% <82.35%> (-0.17%)
:arrow_down:

partitura/io/importmusicxml.py
73.85% <100.00%> (ø)

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

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


Change source of data in check-ram - closes #14

Hi,
I edited check to fix issue I reported (#14). It works on Linux very well. I tried using lib for vmstat - but I cannot get with it values of buffers/cache ;(

@11mariom I will take a look at this but I will say I am not inclined to use proc unless absolutely necessary.
@tas50  you agree ?

@mattyjones well - I had to do something to fix failing nodes. I know it is not best way to do it, but... it is better than using free -m | grep buffers - as it will not fail on CentOS 7

@11mariom Can you rebase this and I will merge
Thanks

@mattyjones what do you want me to rebase?

I think he meant rebase the PR.

Bump for rebase.

I think this can be closed as the vmstat rewrite has landed.

Closing as check-ram uses vmstat now.


Some examples please?

Hello - thanks for this repository. I'm coming from Adafruit'x NeoPixel and GFX libraries and I'd like to use your RGBMatrix libraries, but I'm finding it difficult to start without examples.
Could you please post the code corresponding to the sample screenshots / gifs?
thanks

Hi!
This is simply an emulator for the existing rpi-rgb-led-matrix library, so if you have questions about how it works or how to begin development for it, I'd start there.
With that said, that library does include sample files that you could install the emulator into according to instructions in the README: https://github.com/hzeller/rpi-rgb-led-matrix/tree/master/bindings/python/samples
I also have some projects with the emulator pre-installed:
https://github.com/ty-porter/mlb-led-scoreboard-emulated
https://github.com/ty-porter/nfl-led-scoreboard-emulated
The emulator is also compatible with other projects such as https://github.com/riffnshred/nhl-led-scoreboard

Thanks for the suggestion, I just needed time to get the samples together, working, and documented.


Batcher should sort data in addition to batching

The batcher has an opportunity to sort potentially-slightly-unordered-data (e.g. a user logging to the same entity path from multiple threads) at basically no extra cost, saving a lot of compute on the backend (OOO ingestion is painful for the store, the main cache, the secondary caches...).
We need to extract the swap-sorting logic from the store and expose it on DataTable so everything can benefit from it with no effort.

Blocked on https://github.com/rerun-io/rerun/issues/1981


WIP: new CSAF (don't merge yet)

Tracking PR for the new CSAF, currently not ready for a merge yet

Closing as this was superseded by #132


Virtual sound device on Windows Instance

Description
I'm writing tests for Sonic Pi, using Github Actions.
On Linux I can do 'jackd -d dummy &' to get a virtual audio device.
On Mac, Soundflower is automatically installed (a virtual audio device)
But I'm stuck on Windows. Is there a way for my github action to run a test which finds an audio device on a windows instance? It doesn't need to make a sound, obviously, just register as an audio device that the software can connect to.
Question, Bug, or Feature?:
Question
Virtual environments affected

[* ] Windows Server 2016 R2
[ *] Windows Server 2019

Expected behavior
Some kind of valid sound device is installed
Actual behavior
No sound device is found
Repro steps
Launch an executable as part of an action that requires a sound device (virtual or real).

Hello, @cmaughan
Could you please try to install in runtime [https://github.com/duncanthrax/scream]?
jobs:
  compile:
    runs-on: windows-2019
    steps:
      - name: Audio device
        run: Get-CimInstance Win32_SoundDevice | fl *
      - name: Install Scream
        shell: powershell
        run: |
          Start-Service audio*
          Invoke-WebRequest https://github.com/duncanthrax/scream/releases/download/3.6/Scream3.6.zip -OutFile C:\Scream3.6.zip
          Extract-7Zip -Path C:\Scream3.6.zip -DestinationPath C:\Scream
          $cert = (Get-AuthenticodeSignature C:\Scream\Install\driver\Scream.sys).SignerCertificate
          $store = [System.Security.Cryptography.X509Certificates.X509Store]::new("TrustedPublisher", "LocalMachine")
          $store.Open("ReadWrite")
          $store.Add($cert)
          $store.Close()
          cd C:\Scream\Install\driver
          C:\Scream\Install\helpers\devcon install Scream.inf *Scream
      - name: Audio device
        run: Get-CimInstance Win32_SoundDevice | fl *

Hello, @cmaughan
Could you please try to install in runtime [https://github.com/duncanthrax/scream]?
jobs:
  compile:
    runs-on: windows-2019
    steps:
      - name: Audio device
        run: Get-CimInstance Win32_SoundDevice | fl *
      - name: Install Scream
        shell: powershell
        run: |
          Start-Service audio*
          Invoke-WebRequest https://github.com/duncanthrax/scream/releases/download/3.6/Scream3.6.zip -OutFile C:\Scream3.6.zip
          Extract-7Zip -Path C:\Scream3.6.zip -DestinationPath C:\Scream
          $cert = (Get-AuthenticodeSignature C:\Scream\Install\driver\Scream.sys).SignerCertificate
          $store = [System.Security.Cryptography.X509Certificates.X509Store]::new("TrustedPublisher", "LocalMachine")
          $store.Open("ReadWrite")
          $store.Add($cert)
          $store.Close()
          cd C:\Scream\Install\driver
          C:\Scream\Install\helpers\devcon install Scream.inf *Scream
      - name: Audio device
        run: Get-CimInstance Win32_SoundDevice | fl *

This works great :) Thanks so much for the effort; I hope it didn't take too long!
https://github.com/cmaughan/sonic-pi/runs/1764226203?check_suite_focus=true#step:22:217

This works great :) Thanks so much for the effort; I hope it didn't take too long!
https://github.com/cmaughan/sonic-pi/runs/1764226203?check_suite_focus=true#step:22:217

@dechamps I use VBCable in the Windows part of sound-ci-helpers (which I use by running powershell sound-ci-helpers/windows/setup_sound.ps1 on Azure after a clone)
https://github.com/LABSN/sound-ci-helpers
I think it creates an input and an output device, feel free to try it.
I should probably make this a proper GitHub Action step, I don't think it would be too difficult!

On the new windows-2022 environment (which is currently being rolled out as windows-latest - see #4856), workflows using audio devices might break because the Windows Audio engine is not started by default anymore. This can be remedied using the following step:
- run: net start audiosrv

@al-cheb can we revive the discussion? Scream cannot afford the certificates, which is more than understandable. GitHub as part of Microsoft may can contribute or support, so that audio testing can become available again on windows:
https://github.com/duncanthrax/scream/issues/202#issuecomment-1649858065
Thank you :)

@Gustl22 it looks like you are talking about driver code signing, whereas this issue is about setting up a virtual sound device on CIs (which does seem possible using two solutions mentioned above: https://github.com/actions/runner-images/issues/2528#issuecomment-1050295010, https://github.com/actions/runner-images/issues/2528#issuecomment-1037547655)

@larsoner https://github.com/actions/runner-images/issues/2528#issuecomment-1050295010 is no solution (I'm pretty sure at least), it just is an additional step to let Scream add a virtual device, see this run.
And your mentioned action also fails, see this run.
Yes, it is not about the specific solution, but it is about a working solution supported by GH Actions.

And your mentioned action also fails, see this run.

Should be fixed now, feel free to try again :+1: Hopefully it works for now (until GH officially supports something, if ever!)

Thanks for fixing @larsoner, but even in your solution you propose to use scream once, vb cable driver breaks.
For me, the issue is not solved here.
@al-cheb please reconsider opening again. How does Microsoft solve this problem internally for their tests? Are they using real sound cards for that? Thx.

Looks like the Scream-based solution stopped working at some point on windows-latest (windows-2022 20240514.3.0) - it hangs on the devcon step:
Scream\Install\helpers\devcon-x64.exe install Scream\Install\driver\x64\Scream.inf *Scream

Device node created. Install is complete when drivers are installed...
Updating drivers for *Scream from …\Scream\Install\driver\x64\Scream.inf.
(…hangs forever…)

Same if I upgrade to latest Scream 4.0.
Example run: https://github.com/dechamps/FlexASIO/actions/runs/9238174920/job/25415967004
Not sure what broke it?

It took some heroics, but I was able to capture a screenshot:
      - shell: powershell
        run: Install-Module -Name PShot -AcceptLicense -Force
      - shell: powershell
        run: |
          Import-Module -Name PShot
          Start-Job { Get-PShot -Directory . -Delay 30 }
          Scream\Install\helpers\devcon-x64.exe install Scream\Install\driver\x64\Scream.inf *Scream
        continue-on-error: true
        timeout-minutes: 1
      - uses: actions/upload-artifact@v2
        with:
          path: 'PShot*'

The result:

Looks like the Import-Certificate -FilePath Scream\Install\driver\x64\Scream.cat -CertStoreLocation Cert:\LocalMachine\TrustedPublisher command is not sufficient for it to work anymore.

And I guess this is why:

I found duncanthrax/scream#215 about this. Looks like it's not going to be fixed, so we're gonna have to find some way to work around this or use a different driver.

…and here's how I worked around it:
      - run: 'Invoke-WebRequest https://github.com/duncanthrax/scream/releases/download/4.0/Scream4.0.zip -OutFile Scream4.0.zip'
      - run: 'Expand-Archive -Path Scream4.0.zip -DestinationPath Scream'
      # To work around https://github.com/duncanthrax/scream/issues/215, create our own self-signed certificate for the Scream driver.
      # makecert.exe insists on interactively asking the user for a password (sigh...), so use OpenSSL instead.
      # `-extensions v3_req` is a trick to make sure the resulting cert has basic constraint CA:FALSE (the default is CA:TRUE which is problematic here) without having to create an OpenSSL config file.
      - run: 'openssl req -batch -verbose -x509 -newkey rsa -keyout ScreamCertificate.pvk -out ScreamCertificate.cer -nodes -extensions v3_req'
      - run: 'openssl pkcs12 -export -nodes -in ScreamCertificate.cer -inkey ScreamCertificate.pvk -out ScreamCertificate.pfx -passout pass:'
      # This is just to make sure signtool.exe is in the PATH
      - uses: ilammy/msvc-dev-cmd@v1
      # Sign the driver with the self-signed certificate we just made.
      - run: 'signtool sign /v /fd SHA256 /f ScreamCertificate.pfx Scream\Install\driver\x64\Scream.cat'
      # Tell Windows to trust the self-signed certificate we just made.
      # (For some reason it has to be added to both stores for it to work.)
      - run: 'Import-Certificate -FilePath ScreamCertificate.cer -CertStoreLocation Cert:\LocalMachine\root'
      - run: 'Import-Certificate -FilePath ScreamCertificate.cer -CertStoreLocation Cert:\LocalMachine\TrustedPublisher'
      # Finally, install the driver.
      - run: 'Scream\Install\helpers\devcon-x64.exe install Scream\Install\driver\x64\Scream.inf *Scream'
        # For convenience, make sure we fail fast if for whatever reason the install gets blocked on some GUI prompt.
        timeout-minutes: 5

All good now: https://github.com/dechamps/FlexASIO/actions/runs/9242970185/job/25426485555
You'll then likely want to follow that with net start audiosrv for the Windows audio stack to actually work.

For anyone trying to do what I was doing, install a virtual sound device on macos, windows, and ubuntu (all latest), see this run which works for scream on windows and also the aforementioned as of the date of this comment.


NavigationView - Automatically wire up the back button if content is Frame

Today NavigationView's back button doesn't do anything by default. It will be better if it automatically goes back to the previous page when the content is a Frame and IsNavigationStackEnabled is true unless the developer wants to handle it on their own. This will make NavigationView easier to use.

@YuliKl Can I implement this feature?


[Python 3.12] Build issue - No module named 'Cython'

Preparing metadata (pyproject.toml): finished with status 'error'
  error: subprocess-exited-with-error

  × Preparing metadata (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [15 lines of output]
      Traceback (most recent call last):
        File "/opt/hostedtoolcache/Python/3.12.3/x64/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 3[53](https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/8896934501/job/24438570724#step:8:54), in <module>
          main()
        File "/opt/hostedtoolcache/Python/3.12.3/x64/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/opt/hostedtoolcache/Python/3.12.3/x64/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 149, in prepare_metadata_for_build_wheel
          return hook(metadata_directory, config_settings)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/opt/hostedtoolcache/Python/3.12.3/x64/lib/python3.12/site-packages/setuptools/build_meta.py", line 366, in prepare_metadata_for_build_wheel
          self.run_setup()
        File "/opt/hostedtoolcache/Python/3.12.3/x64/lib/python3.12/site-packages/setuptools/build_meta.py", line 311, in run_setup
          exec(code, locals())
        File "<string>", line [60](https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/8896934501/job/24438570724#step:8:61), in <module>
      ModuleNotFoundError: No module named 'Cython'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/8896934501/job/24438570724

A workaround for this: pip install cython. After explicitly installing cython i was able to compile triton, however there were segfaults in the core tests, see #1009.

Please use stock PyTorch XPU source build and test to remove this dependency. BKC has sent.

This is an IPEX issue, #925 makes it obsolete.


Fix typos in renaming message for Errno

There were some mismatches between the renamed:  suggestion and the actual variable names

@swift-ci please test


test: add tests for locale updates

Tests aria-invalid attribute on the DatePicker's input element when the locale is updated and the date becomes invalid through incorrect text input.

Add test test for #1895 (https://github.com/dmtrKovalenko/date-io/pull/406)

Am I the only who cannot see the changes of this PR?

@dmtrKovalenko, During a revert to an earlier commit my local git became out of sync and would not pull in changes from the next branch.  I was unable to merge and ran a reset.
While running tests with luxon and moment the test for valid after invalid fails with locale de.  I can look into this further if you'd like.


not working anymore with kubernetes 1.22

I have been using this until kubernetes 1.21; and it works perfectly. thanks alot.
But it seems that it doesnt support 1.22 anymore.
And I think its because of multiple reasons :

you cant use cert-manager 1.2 anymore with k8s 1.22 because they removed the beta api of apiservices.apiregistration.k8s.io, so  the minimum cert-manager version is 1.5 for k8s 1.22.
It seems that the new cert-manager version, requires extra rbac config for any custom webhooks; I am seeing alot of errors in cert-manager that its service account is forbidden to access the custom api that duckdns deployment created.

Did you guys manage to test this with the newer k8s version and the new cert-manager versions ?

My fork is working on v1.24.4+k3s1 with cert-manager 1.9.1.
https://github.com/joshuakraitberg/cert-manager-webhook-duckdns

@joshuakraitberg : I am having problems with your repo regarding rbac. The rbac does not seem to define the sufficient rights. I am using v1.24.6+k3s1" and cert-manager v1.10.0. Can you also include "issues" at your repo so that I can report this to proper repo?
The error is shown in challenge events when the cert order is stuck on pending state:
"Warning  PresentError  22m (x28 over 10h)  cert-manager-challenges  Error presenting challenge: duckdns.mydomain.duckdns.org is forbidden: User "system:serviceaccount:cert-manager:cert-manager" cannot create resource "duckdns" in API group "mydomain.duckdns.org" at the cluster scope"

@attiolli Done.


Some compilers do not accept .mp_obj_type in 1.20

I'm finding that Emscripten's clang version is refusing to accept the way some structs are being initialized now. This doesn't happen before the 1.20 merge so I assume it's related to upstream changing how object types are defined. Any ideas?
build/lvgl/lv_mpy.c:1138:20: error: initializer element is not a compile-time constant
    .mp_obj_type = mp_lv_LV_RES_type_base,
                   ^~~~~~~~~~~~~~~~~~~~~~
build/lvgl/lv_mpy.c:1184:20: error: initializer element is not a compile-time constant
    .mp_obj_type = mp_lv_ENUM_LV_LOG_LEVEL_type_base,
                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
build/lvgl/lv_mpy.c:1246:20: error: initializer element is not a compile-time constant
    .mp_obj_type = mp_lv_LV_ALIGN_type_base,
                   ^~~~~~~~~~~~~~~~~~~~~~~~
build/lvgl/lv_mpy.c:1294:20: error: initializer element is not a compile-time constant
    .mp_obj_type = mp_lv_LV_DIR_type_base,

Code near line 1246
STATIC MP_DEFINE_CONST_OBJ_TYPE(
    mp_lv_LV_ALIGN_type_base,
    MP_QSTR_LV_ALIGN,
    MP_TYPE_FLAG_NONE,
    print, LV_ALIGN_print,

    attr, call_parent_methods,

    locals_dict, &LV_ALIGN_locals_dict
);

STATIC const mp_lv_obj_type_t mp_lv_LV_ALIGN_type = {
#ifdef LV_OBJ_T
    .lv_obj_class = NULL,
#endif
    .mp_obj_type = mp_lv_LV_ALIGN_type_base,
};

Could you try to enable MICROPY_PY_THREAD in ports/webassembly/mpconfigport.h and see if you are getting the same error on upstream micropython?

I don't get the error when I do that.

What is the clang version that you are using? Is it possible to switch clang version on Emscripten?

The previous version of Emscripten I was using included Clang 13. I'll update to the latest version (which uses Clang 17) and see what happens.

Could you try branch fix/282?
I changed mp_obj_type to pointer, I think it should be anyway.

Seems to have moved the problem to a different location. Progress?
build/lvgl/lv_mpy.c:15952:13: error: initializer element is not a compile-time constant
 15952 |     parent, mp_lv_obj_type.mp_obj_type,
       |             ^~~~~~~~~~~~~~~~~~~~~~~~~~
../../py/obj.h:783:250: note: expanded from macro 'MP_DEFINE_CONST_OBJ_TYPE'
  783 | #define MP_DEFINE_CONST_OBJ_TYPE(...) MP_DEFINE_CONST_OBJ_TYPE_EXPAND(MP_DEFINE_CONST_OBJ_TYPE_NARGS(__VA_ARGS__, _INV, 12, _INV, 11, _INV, 10, _INV, 9, _INV, 8, _INV, 7, _INV, 6, _INV, 5, _INV, 4, _INV, 3, _INV, 2, _INV, 1, _INV, 0)(mp_obj_type_t, __VA_ARGS__))
      |                                                                                ^~~~~~~~~~~
../../py/obj.h:749:415: note: expanded from macro 'MP_DEFINE_CONST_OBJ_TYPE_NARGS_7'
  749 | #define MP_DEFINE_CONST_OBJ_TYPE_NARGS_7(_struct_type, _typename, _name, _flags, f1, v1, f2, v2, f3, v3, f4, v4, f5, v5, f6, v6, f7, v7) const _struct_type _typename = { .base = { &mp_type_type }, .name = _name, .flags = _flags, .slot_index_##f1 = 1, .slot_index_##f2 = 2, .slot_index_##f3 = 3, .slot_index_##f4 = 4, .slot_index_##f5 = 5, .slot_index_##f6 = 6, .slot_index_##f7 = 7, .slots = { v1, v2, v3, v4, v5, v6, v7, } }
      |                                                                                ^~
../../py/obj.h:770:44: note: expanded from macro 'MP_DEFINE_CONST_OBJ_TYPE_EXPAND'
  770 | #define MP_DEFINE_CONST_OBJ_TYPE_EXPAND(x) x
      |                                            ^

STATIC MP_DEFINE_CONST_OBJ_TYPE(
    mp_lv_canvas_type_base,
    MP_QSTR_canvas,
    MP_TYPE_FLAG_NONE,
    print, canvas_print,
    make_new, canvas_make_new,
    binary_op, mp_lv_obj_binary_op,
    attr, call_parent_methods,
    buffer, mp_lv_obj_get_buffer,
    parent, mp_lv_obj_type.mp_obj_type, // <---- this is the line that errors
    locals_dict, &canvas_locals_dict
);

Seems to have moved the problem to a different location. Progress?

Maybe...
I've updated the branch, could you try again?

Unfortunately looks about the same to me.
build/lvgl/lv_mpy.c:15429:34: error: initializer element is not a compile-time constant
 15429 |     { MP_ROM_QSTR(MP_QSTR_CTRL), MP_ROM_PTR(mp_lv_LV_BTNMATRIX_CTRL_type.mp_obj_type) },
       |                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../py/obj.h:348:23: note: expanded from macro 'MP_ROM_PTR'
  348 | #define MP_ROM_PTR(p) (p)
      |                       ^~~
build/lvgl/lv_mpy.c:15462:13: error: initializer element is not a compile-time constant
 15462 |     parent, mp_lv_obj_type.mp_obj_type,
       |             ^~~~~~~~~~~~~~~~~~~~~~~~~~
../../py/obj.h:783:250: note: expanded from macro 'MP_DEFINE_CONST_OBJ_TYPE'
  783 | #define MP_DEFINE_CONST_OBJ_TYPE(...) MP_DEFINE_CONST_OBJ_TYPE_EXPAND(MP_DEFINE_CONST_OBJ_TYPE_NARGS(__VA_ARGS__, _INV, 12, _INV, 11, _INV, 10, _INV, 9, _INV, 8, _INV, 7, _INV, 6, _INV, 5, _INV, 4, _INV, 3, _INV, 2, _INV, 1, _INV, 0)(mp_obj_type_t, __VA_ARGS__))
      |                                                                                ^~~~~~~~~~~
../../py/obj.h:749:415: note: expanded from macro 'MP_DEFINE_CONST_OBJ_TYPE_NARGS_7'
  749 | #define MP_DEFINE_CONST_OBJ_TYPE_NARGS_7(_struct_type, _typename, _name, _flags, f1, v1, f2, v2, f3, v3, f4, v4, f5, v5, f6, v6, f7, v7) const _struct_type _typename = { .base = { &mp_type_type }, .name = _name, .flags = _flags, .slot_index_##f1 = 1, .slot_index_##f2 = 2, .slot_index_##f3 = 3, .slot_index_##f4 = 4, .slot_index_##f5 = 5, .slot_index_##f6 = 6, .slot_index_##f7 = 7, .slots = { v1, v2, v3, v4, v5, v6, v7, } }
      |                                                                                ^~
../../py/obj.h:770:44: note: expanded from macro 'MP_DEFINE_CONST_OBJ_TYPE_EXPAND'
  770 | #define MP_DEFINE_CONST_OBJ_TYPE_EXPAND(x) x
      |                                            ^
build/lvgl/lv_mpy.c:15771:13: error: initializer element is not a compile-time constant
 15771 |     parent, mp_lv_obj_type.mp_obj_type,
       |             ^~~~~~~~~~~~~~~~~~~~~~~~~~

Unfortunately looks about the same to me.

Forgot to push, sorry.... Try now?

Much better, now only the MP_ROM_PTR one remains.
build/lvgl/lv_mpy.c:12731:34: error: initializer element is not a compile-time constant
 12731 |     { MP_ROM_QSTR(MP_QSTR_FLAG), MP_ROM_PTR(mp_lv_LV_OBJ_FLAG_type.mp_obj_type) },
       |                                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../py/obj.h:348:23: note: expanded from macro 'MP_ROM_PTR'
  348 | #define MP_ROM_PTR(p) (p)
      |                       ^~~

Much better, now only the MP_ROM_PTR one remains.

Fixed now?

Seems to have carried on compiling now! Thank you!

When it's all good - we can merge https://github.com/lvgl/lv_binding_micropython/pull/284


Minio port change

Summary
Im deploying minio with the sorry-cypress chart and I realized my screenshots are not being saved to the bucket. I think its because port 9000 is hard coded in director and my minio runs through an ingress on port 80. There does not seem to be a way to override the MINIO_PORT in the values.yaml.
How to reproduce
Deploy using helm on a kubernetes cluster with metallb as the load balancer.
Environment

sorry-cypress version: latest
cypress version: 8.7.0
helm chart version: 2.5.2

@thowd22 You can update this by setting the servicePort option on minio config
minio:
  service:
    port: 80


Always getting skipping message

Only run install-peer-deps after \install` command. Skipping.`
No matther what I do. The instructions are pretty straightforward.
I get the same message both after normal npm install or npm run install-peers.
Windows 10
Node 11

You probably would want to include install-peers in postinstall instead of running it by itself. If not, you can also use -f to force the install (only available >= 2.1.1).

Hi @juliaqiuxy, is it possible to use from prepare script? I want to use it to install my peer dependencies in order to debug my package without put these as devDependencies too


Group selection not possible

If I enabled the "Allow group selection." option, I can't select all in the marked section.
Can this be solved

by clicking on the name (in this case the feature of #81 will be disabled),
or the arrow will be replaced with the checkbox and only by clicking on the name the section will be opened.

I support the question of @PhilippDehner and can add, that I have one tree where the groups can be selected, and another one where there is no checkbox. Both within the same Variables panel in different tabs, so the option "Allow group selection" should be valid for both. All variables behind share the same pattern (type "query", "Multi-value" and "Allow All option" activated), so this should not be the cause. The only difference I see is that the tree without checkboxes is much longer (51 nodes, each with 5 to 20 leaves) while the other tree is much shorter (3 nodes, 30 leaves in total).

Resolved in https://github.com/VolkovLabs/volkovlabs-variable-panel/pull/139


config.status: error: cannot find input file: `Makefile.in'

On CentOS 7 having an error after running ./configure command:
# ./bootstrap.sh
aclocal: warning: couldn't open directory 'm4': Toks failas ar aplankas neegzistuoja
configure.ac:7: installing './install-sh'
configure.ac:7: installing './missing'
Makefile.am:6: error: Libtool library used but 'LIBTOOL' is undefined
Makefile.am:6:   The usual way to define 'LIBTOOL' is to add 'LT_INIT'
Makefile.am:6:   to 'configure.ac' and run 'aclocal' and 'autoconf' again.
Makefile.am:6:   If 'LT_INIT' is in 'configure.ac', make sure
Makefile.am:6:   its definition is in aclocal's search path.
Makefile.am: installing './depcomp'
parallel-tests: installing './test-driver'
autoreconf: automake failed with exit status: 1

# ./configure
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking how to run the C preprocessor... gcc -E
checking for grep that handles long lines and -e... /bin/grep
checking for egrep... /bin/grep -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking minix/config.h usability... no
checking minix/config.h presence... no
checking for minix/config.h... no
checking whether it is safe to define __EXTENSIONS__... yes
./configure: line 3402: LT_INIT: command not found
checking for a BSD-compatible install... /bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking for style of include used by make... GNU
checking whether make supports nested variables... yes
checking dependency style of gcc... gcc3
checking whether to enable maintainer-specific portions of Makefiles... yes
checking for gcc... (cached) gcc
checking whether we are using the GNU C compiler... (cached) yes
checking whether gcc accepts -g... (cached) yes
checking for gcc option to accept ISO C89... (cached) none needed
checking for gcc option to accept ISO C99... -std=gnu99
checking for gcc -std=gnu99 option to accept ISO Standard C... (cached) -std=gnu99
checking sys/fsuid.h usability... yes
checking sys/fsuid.h presence... yes
checking for sys/fsuid.h... yes
checking for setfsuid... yes
checking for security/pam_appl.h... yes
checking for security/pam_modules.h... yes
checking for pam_get_user in -lpam... yes
checking whether certain PAM functions require const arguments... yes
checking for library containing dlopen... -ldl
checking that generated files are newer than configure... done
configure: creating ./config.status
config.status: error: cannot find input file: `Makefile.in'

Has anyone resolved this kind of an issue?

Well, automake failed, so that's probably it.
Do you have libtool installed?

I thought that pam-devel would suffice. I've installed libtool and it's working now. Thanks!


Comparing UUID7 doesn't seem to produce the expected results

I am attempting to sort a list of UUIDs and when I use Uniq.UUID.compare to do that I am not getting the list of UUIDs not sorted properly.
Here is the code used to produce and sort the UUIDs and debugging output.
Produce the UUIDs:
uuids_with_index =
  1..10
  |> Enum.to_list()
  |> Enum.map(fn i ->
    {i, Uniq.UUID.uuid7()}
  end)

Output:
uuids_with_index #=> [
  {1, "018dd112-d7d4-79b7-9c3f-cd8f5a268bad"},
  {2, "018dd112-d7d4-7963-ac79-96d0ba13f0e3"},
   {3, "018dd112-d7d4-7348-8dd5-e7a8fed42a22"},
   {4, "018dd112-d7d4-7a48-a433-fcb3294b9fb3"},
   {5, "018dd112-d7d4-72fb-9650-558ff08be388"},
   {6, "018dd112-d7d4-7f7a-80fd-7bf8daeab8ce"},
   {7, "018dd112-d7d4-7b67-8c66-f0b665a19e4f"},
   {8, "018dd112-d7d4-7a80-bad6-2d0378fc51c5"},
   {9, "018dd112-d7d4-74f9-a64f-2ab48c5f1ee8"},
   {10, "018dd112-d7d4-74a4-804b-686439c83abc"}
]

Create the list and reverse it:
uuids = Enum.map(uuids_with_index, fn {_i, uuid} -> uuid end) |> Enum.reverse()

Output:
uuids #=> ["018dd112-d7d4-74a4-804b-686439c83abc", "018dd112-d7d4-74f9-a64f-2ab48c5f1ee8",
 "018dd112-d7d4-7a80-bad6-2d0378fc51c5", "018dd112-d7d4-7b67-8c66-f0b665a19e4f",
 "018dd112-d7d4-7f7a-80fd-7bf8daeab8ce", "018dd112-d7d4-72fb-9650-558ff08be388",
 "018dd112-d7d4-7a48-a433-fcb3294b9fb3", "018dd112-d7d4-7348-8dd5-e7a8fed42a22",
 "018dd112-d7d4-7963-ac79-96d0ba13f0e3", "018dd112-d7d4-79b7-9c3f-cd8f5a268bad"]

Attempt to sort them:
sorted =
  Enum.sort(uuids, fn a, b ->
    case Uniq.UUID.compare(a, b) do
      :eq -> true
      :lt -> true
      :gt -> false
    end
  end)

Sorted output:
sorted #=> ["018dd112-d7d4-7f7a-80fd-7bf8daeab8ce", "018dd112-d7d4-7b67-8c66-f0b665a19e4f",
 "018dd112-d7d4-7a80-bad6-2d0378fc51c5", "018dd112-d7d4-7a48-a433-fcb3294b9fb3",
 "018dd112-d7d4-79b7-9c3f-cd8f5a268bad", "018dd112-d7d4-7963-ac79-96d0ba13f0e3",
 "018dd112-d7d4-74f9-a64f-2ab48c5f1ee8", "018dd112-d7d4-74a4-804b-686439c83abc",
 "018dd112-d7d4-7348-8dd5-e7a8fed42a22", "018dd112-d7d4-72fb-9650-558ff08be388"]

It's because the resolution is 1ms and all your uuids are generated in the same millisecond. I'm facing the same problem, it cases flaky tests. You need to either add 1ms delay before generating the next uuid.
It would be good to have a "special" version that has bigger resolution for use in tests. It could be named unsafe_uuid_v7.
https://github.com/beam-community/ex_machina/issues/463


Expands Data Metering

This commit expands the data usage tracking by including authentication responses in usage tracking.
Signed-off-by: Lui de la Parra<EMAIL_ADDRESS>

Coverage remained the same at 100.0% when pulling cb10bcb6b1aa3688ba11c8d46b93fbf7e0c15367 on tracking-upgrade into 01859fca07901af517379a5a89e669bd9a203340 on master.


SkyWalking cluster problem

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

Bug

Which version of SkyWalking, OS and JRE?

Which company or project?

What happen?
If possible, provide a way for reproducing the error. e.g. demo application, component version.

Requirement or improvement

Hi，SkyWalking builds a cluster with three machines, one of which is down, the other ones will report an error, not available, is there a problem with the configuration, or the SkyWalking cluster itself is strongly associated？

That one down means no available process? SkyWalking cluster will wait if the target registered in zk/consule or detected by k8s API server.


Accept Header Not Working

I tried following the instructions in the README to get Halcyon to output plain json if the accept header is simply application/json instead of application/hal+json, but it doesn't seem to be working. No matter how I fiddle with it, it always returns the HAL formatted results (with a content-type header of application/json in the response).
I tried cloning the Halcyon project and running the test project Halcyon.Tests.SelfHost.Mvc, which appeared to be setup the way the README described, but I got the same results as with my own project. I feel like I'm missing something important about getting Halcyon to handle the Accept header correctly. Any help would be greatly appreciated.
Thanks.

I tried setting a breakpoint in Halcyon.Web.HAL.Json.JsonHalOutputFormatter line 50, which is in the WriteAsync method and running a couple requests against the Halcyon.Tests.SelfHost.Mvc test project.
What I found was that if I set the accept header to application/hal+json, the breakpoint was hit and the code worked as expected.
When I set the accept header to just application/json, the breakpoint never got hit. It would seem that the media type application/json is being handled by some other output formatter, and so the logic that checks the media type and chooses which way to format the output is never hit for plain json and thus the response never gets reformatted, I guess?

@justintconroy It looks like the README is slightly outdated.  In order to achieve this behavior I had to explicitly remove the default JsonOutputFormatter:
        services
            .AddMvc()
            .AddMvcOptions(c => {
                c.OutputFormatters.RemoveType<JsonOutputFormatter>();
                c.OutputFormatters.Add(new JsonHalOutputFormatter(
                    new string[] { "application/hal+json", "application/vnd.example.hal+json", "application/vnd.example.hal.v1+json" }
                ));
            });

Oh yay. That works. Thanks! :-)


Add scripts for datasets

How were the following h5ad files generated? It shouldn't be sufficient to post an h5ad file on figshare to have it included in openproblems.

https://figshare.com/ndownloader/files/36509385 from allen_brain_atlas.py
https://ndownloader.figshare.com/files/36086786 from immune_cells.py
https://figshare.com/ndownloader/files/36872214 from mouse_blood_olssen_labelled.py
https://ndownloader.figshare.com/files/36088649 from mouse_hspc_nestorowa2016.py
https://ndownloader.figshare.com/files/36086813 from pancreas.py
https://figshare.com/ndownloader/files/37593188 from tnbc_wu2021.py

@dbdimitrov

https://figshare.com/ndownloader/files/37593188 from tnbc_wu2021.py
https://figshare.com/ndownloader/files/36509385 from allen_brain_atlas.py
@LuckyMD
https://ndownloader.figshare.com/files/36086813 from pancreas.py
https://ndownloader.figshare.com/files/36086786 from immune_cells.py
@mvinyard
https://ndownloader.figshare.com/files/36088649 from mouse_hspc_nestorowa2016.py
@mvinyard or @lazappi ?
https://figshare.com/ndownloader/files/36872214 from mouse_blood_olssen_labelled.py

I'm not sure exactly what the concern is but I'm pretty sure the mouse blood was uploaded by the authors as part of the supplementary to their paper.

@lazappi @scottgigante-immunai - if I remember correctly, I actually put these on figshare during an early version of populating the repo with a work-able set of data / tasks, etc. @rcannood @scottgigante-immunai - how do you suggest we document these scripts? Happy to follow your guidelines!

Ideally we would have a reproducible python script that fetches the data from wherever the authors posted it, but failing that at least a description of what was done (if it's not exactly the same as how it was done last time because nobody remembers, we can always regenerate it according to best practices.)

@scottgigante-immunai - thank you! This sounds great. I will migrate these datasets to this in the coming week or so. Thanks again!

This is great news!
Indeed, it'd be great if there could be a script with a set of instructions on which file was downloaded, processed in which way, in order to get the same object as what's on figshare.
In openpipelines this is a set of bash scripts, but Python is fine as well, as long we have some way of reproducing those files ;)

The immune and pancreas examples are from scib. There is a detailed set of notebooks both datasets including download and processing here.


Couldn't find build tools in your program.

zhuan@zhuan-HP:~/zhuan/ML-KWS-for-MCU/Deployment$ mbed new kws_simple_test --mbedlib
[mbed] Creating new program "kws_simple_test" (git)
[mbed] Adding library "mbed" from "https://mbed.org/users/mbed_official/code/mbed/builds" at latest revision in the current branch
[mbed] Unpacking library build "5aab5a7997ee" in "/home/zhuan/zhuan/ML-KWS-for-MCU/Deployment/kws_simple_test/mbed"
[mbed] Updating reference "mbed" -> "https://mbed.org/users/mbed_official/code/mbed/builds/5aab5a7997ee"
[mbed] ## Couldn't find build tools in your program. Downloading the mbed 2.0 SDK tools...
anybody comes across this problem "Couldn't find build tools in your program."? Look forward to your comment, thank you!

@zhuanaa
Its not an error, its just a message that it couldn't find tools and hence downloading them.  See this

Closing the issue due to inactivity, please reopen it if you still face the issue.


correct updates

I changed it as per your desired requests.

first, I removed the .ps1 script and the .exe file.
I added a github script where it does the following.
captures, verifies, downloads and extracts the latest version for the user without the need to change the URL manually, it already does this automatically.
and finally I converted the script.ps1 to exe with powershell converter.

That seems a lot better now, much appreciated!
However, there's still no source code for the exe file.
This is an open source project, so all the code for any included exe files needs to be here.
What does it use administrator privileges for?
For directory selection, it might be best to use the current directory by default, rather than asking every time it's launched.
And finally, one potential issue.
If future release zips have this updater in it, what happens when it tries to overwrite itself while extracting the zip?
Sorry for all the extra questions and work, I just want to make sure we get it right before including this.

I will answer your questions one by one.

First, administrator privilege is necessary so that when saving, download and extract it to a directory on the c: disk, where depending on the directory it does not allow you to save the mod.
When overwrite/replace, it will overwrite forcefully/forcibly.
I was thinking about putting the steam directory
"C:\Program Files (x86)\Steam\steamapps\common\Plants Vs Zombies"
I asked the user to choose the directory because he can decide where to place the mod, I myself place the mod elsewhere, and I imagine that many users do the same, so he can decide where to place it. whether in Program Files, or elsewhere
I will send the .ps1 source code to you

Okay, I believe everything is ok now...
I sent the ps1 in the commit

@CG8516  and now? Ok, ALL right?

Yep, all good!
I made a few adjustments to handle some more edge-cases, and run without administrator privileges by default.
I was a bit wary of including an exe file without a way to build it in a reproducble way, so I've built a small program that will automatically extract and run the script, instead of requiring a 3rd-party tool like ps2exe.
That way anyone with the project files can build a perfect 1:1 copy of the exe file.
It's been moved to this repo, and will be included in the next release.


Error on Account receivable with reverse charge

in document,  it is written If the sales tax code is reverse charge, then sales tax direction is Sales Tax Receivable.
I reverse charge is setup, no tax direction is modified.
from PM feedback, the document is wrong.
can you please update the document to avoid customer confusion and misused on reverse charge?

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

ID: 3aaea6b3-d996-eb5c-0a4e-82454d5a1bf0
Version Independent ID: 523a07d8-d3b5-eba1-4a5d-f5c6c3f90448
Content: Sales tax calculation on general journal lines - Finance | Dynamics 365
Content Source: articles/finance/general-ledger/sales-tax-calculation-on-general-journal.md
Service: dynamics-365-finance
Product: ****
Technology: ****
GitHub Login: @EricWangChen
Microsoft Alias: wangchen

@anboudin, thanks for the feedback. @EricWangChen, please review and make needed updates to this topic.

Bug 660437 created to track this issue.

Hi, the related statement (and diagram) has been removed via PR 13216


Empty folder 04_pts_vis

Thanks for the great code!(Still learning how it works xD) I just want to test your model on my pointcloud(.npy file) so I ran
python make_pc_dataset.py

with including my npy file in 04_pts and adding the name on testset.txt. Then the folder "04_pts" was created but it was empty.
Am I doing right? Thanks!

Hi @MinchoU,
please see the section "Dataset from Point Clouds for Reconstruction" in the readme: https://github.com/ErlerPhilipp/points2surf#dataset-from-point-clouds-for-reconstruction
To use this script, you're supposed to place you point clouds in 00_base_pc. However, that's only meant for point clouds stored in mesh-file types, e.g. PLY, OBJ, STL anything that Trimesh can load. Since you already have your point clouds as NPY files, that's unnecessary effort.
The easiest solution would be to hijack the included abc_minimal dataset. Simply put your point clouds into 04_pts and modify the testset.txt. You will also need some minor changes in the dataloader to skip the usual point cloud files and use the NPY files directly.
In source/data_loader.py modify these spots to load your NPY files directly:

Replace the contents of load_pts() (line 285) to load your NPY files: https://github.com/ErlerPhilipp/points2surf/blob/master/source/data_loader.py#L285
There might be some other places where the ending (.npy) is attached to the file name, possibly with '.xyz' in the middle.

Be aware that P2S performs poorly on large point clouds (maybe sub-sample to ~50k points), some partial scans and scans from non-solids (e.g. landscapes).
I'm happy to accept a pull request (to an extra branch). Let me know if you need more help.

Thanks for your help. But I'm still struggling with the command python make_pc_dataset.py
I've understood that this command converts point clouds to meshes that are in 04_pts and listed in testset.txt. And I have tested this command on abc_minimal without including my point clouds(haven't modified anything) but still nothing was created in folder 04_pts_vis.
Am I missing something?

I've understood that this command converts point clouds to meshes that are in 04_pts and listed in testset.txt. And I have tested this command on abc_minimal without including my point clouds(haven't modified anything) but still nothing was created in folder 04_pts_vis.

Your point clouds go into 00_base_pc as mesh files with only vertices. This processing step (normalization and subsampling) turns them into Numpy files, which will appear in 04_pts. So you would need to convert your NPY files to e.g. PLY. The 04_pts_vis are optional side-products of this processing step.
So the structure is like this:
    00_base_pc
         |
    processing
         |
        /\
04_pts  (04_pts_vis)

There is file_utils.call_necessary() before every call that checks if the outputs are up-to-date. Assuming the files come with correct timestamps from the repo, it will see that the files in 04_pts are fine and do nothing.


undefined symbol:  _ZN6caffe26detail37_typeMetaDataInstance_preallocated_32E

when running the command "./scripts/dist_run_single.sh 4 vcr/train_end2end.py ./cfgs/vcr/base_q2a_4x16G_fp32.yaml ./"
I got this error
ImportError: /data/workspace/VL-BERT/vcr/../common/lib/roi_pooling/C_ROIPooling.cpython-36m-x86_64-linux-gnu.so: undefined symbol: _ZN6caffe26detail37_typeMetaDataInstance_preallocated_32E
Traceback (most recent call last):
File "./scripts/launch.py", line 200, in
main()
File "./scripts/launch.py", line 196, in main
cmd=process.args)
subprocess.CalledProcessError: Command '['/root/anaconda3/envs/vl-bert/bin/python', '-u', 'vcr/train_end2end.py', '--cfg', './cfgs/vcr/base_q2a_4x16G_fp32.yaml', '--model-dir', './', '--dist']' returned non-zero exit status 1.
(vl-bert) root@6cbe3808c2b8:/data/workspace/VL-BERT# Traceback (most recent call last):
File "vcr/train_end2end.py", line 8, in
from vcr.function.train import train_net
File "/data/workspace/VL-BERT/vcr/../vcr/function/train.py", line 26, in
from vcr.modules import *
File "/data/workspace/VL-BERT/vcr/../vcr/modules/init.py", line 1, in
from .resnet_vlbert_for_vcr import ResNetVLBERT
File "/data/workspace/VL-BERT/vcr/../vcr/modules/resnet_vlbert_for_vcr.py", line 7, in
from common.fast_rcnn import FastRCNN
File "/data/workspace/VL-BERT/vcr/../common/fast_rcnn.py", line 10, in
from common.lib.roi_pooling.roi_pool import ROIPool
File "/data/workspace/VL-BERT/vcr/../common/lib/roi_pooling/init.py", line 1, in
from .roi_align import ROIAlign
File "/data/workspace/VL-BERT/vcr/../common/lib/roi_pooling/roi_align.py", line 8, in
from . import C_ROIPooling
ImportError: /data/workspace/VL-BERT/vcr/../common/lib/roi_pooling/C_ROIPooling.cpython-36m-x86_64-linux-gnu.so: undefined symbol: _ZN6caffe26detail37_typeMetaDataInstance_preallocated_32E
Traceback (most recent call last):
File "vcr/train_end2end.py", line 8, in
from vcr.function.train import train_net
File "/data/workspace/VL-BERT/vcr/../vcr/function/train.py", line 26, in
from vcr.modules import *
File "/data/workspace/VL-BERT/vcr/../vcr/modules/init.py", line 1, in
from .resnet_vlbert_for_vcr import ResNetVLBERT
File "/data/workspace/VL-BERT/vcr/../vcr/modules/resnet_vlbert_for_vcr.py", line 7, in
from common.fast_rcnn import FastRCNN
File "/data/workspace/VL-BERT/vcr/../common/fast_rcnn.py", line 10, in
from common.lib.roi_pooling.roi_pool import ROIPool
File "/data/workspace/VL-BERT/vcr/../common/lib/roi_pooling/init.py", line 1, in
from .roi_align import ROIAlign
File "/data/workspace/VL-BERT/vcr/../common/lib/roi_pooling/roi_align.py", line 8, in
from . import C_ROIPooling

I think it may be caused by ROI_pooling.xx.so
But I've compiled C_ROIPooling.cpython-36m-x86_64-linux-gnu.so  with "./scripts/init.sh"

I also met this error in some environment, it seems that some python dependency makes the compiling failed, I solved it by creating a new conda environment and install dependencies as described in README. Could you try to create a new environment and try again? Thanks!

Also, you can try the new requirements.txt that I just updated.

Also, you can try the new requirements.txt that I just updated.

I have tried the new requirements.txt. But it doesnot work as well

@search-opensource-space Could you please check the version of nvcc and gcc by:
nvcc -V
gcc --version

Thanks!

i have the same error, the version of gcc and nvcc  as same as @search-opensource-space ,my system is centos7.5, python 3.6.10, pytorch 1.10, torchvision 0.2.1

@wangshuwch I only test the code under Ubuntu 16.04.

I have the same issue with this. Does anyone figure out what is the problem?

In my case, I resolved this by upgrading gcc 4.9.3 to 5.4.0 :)
And, I don't know why successfully building roi_pooling using "./scripts/init.sh"  instead of building in the exact directory.

@becxer Actually in the './scripts/init.sh', it would enter the directory firstly and then compile the operators.

@jackroos Yes I know, It sounds weird but doing it manually gives the same error :(


Move pick-seq attributes PR to prod

@PozhidayevaDarya has requested that the changes made in #269 are released to production.
I have a draft release here: https://github.com/ncihtan/data-models/releases/edit/untagged-510dc78bef0e4f34def2

PR for mtDCA https://github.com/Sage-Bionetworks/data_curator_config/pull/86
Not going to deploy to shinyapps given imenent decomissioning


reopening #111 (ZFS)

reporting that the kernel module build and loads:
[tru@mba15-m2-nixos:~]$ uname -a
Linux mba15-m2-nixos 6.6.0-asahi #1-NixOS SMP PREEMPT_DYNAMIC Tue Jan  1 00:00:00 UTC 1980 aarch64 GNU/Linux
[tru@mba15-m2-nixos:~]$ modinfo zfs|head
filename:       /run/booted-system/kernel-modules/lib/modules/6.6.0-asahi/extra/zfs.ko.zst
version:        2.2.3-1
license:        CDDL
license:        Dual BSD/GPL
license:        Dual MIT/GPL
author:         OpenZFS
description:    ZFS
alias:          zzstd
alias:          zcommon
alias:          zunicode
[tru@mba15-m2-nixos:~]$ lsmod|grep zfs
zfs                  4440064  6
spl                   131072  1 zfs

/etc/nixos/configuration.nix
...
# ZFS
boot.supportedFilesystems = [ "zfs" ];
boot.zfs.forceImportRoot = false;
networking.hostId= "007f0200";
#        error: Package ‘zfs-kernel-2.2.3-6.6.0-asahi’ in /nix/store/r944d4j7p39b769x7ykbsiz5wm3q8agf-nixos-23.11/nixos/pkgs/os-specific/linux/zfs/generic.nix:213 is marked as broken, refusing to evaluate.
nixpkgs.config.allowBroken = true;

oops forgot to report that I was using https://github.com/tpwrules/nixos-apple-silicon/releases/tag/release-2024-03-24

Original issue: #111

AIUI the main pain point is:

Asahi patches track a recent kernel
ZFS module tracks a more stable kernel

So with a single linux-asahi kernel package:

either a more stable kernel is tracked and Apple Silicon support via Asahi ends up being dragged down by ZFS
or there's a risk of ZFS not building anymore against the too-recent kernel required by the latest Asahi patchset

Could there be another strategy? I'm thinking that it ties in with issues such as #234, where intentionally targeting a stable channel conflicts with the kernel version matching the Asahi patchset requires a Rust compiler more recent than what's provided by 24.05.
So, what if there were two linux-asahi kernel packages? One that would track the latest Asahi patches and one that would track a more stable kernel with a non-latest Asahi patchset but compatible with stable NixOS channels?


Cull unnecessary tasks

Consider the following dask
In [2]: def inc(x): return x + 1

In [3]: d = {'x': 1, 'y': (inc, 'x'), 'z': (inc, 'x'), 'out': (add, 'y', 10)}

If we go ahead and get out the out key
>>> from dask.threaded import get
>>> get(d, 'out')

We end up doing excess work computing 'z'.
It would be nice to have a function that takes a dask and a key or list of keys and removes all tasks not necessary to compute those keys.
>>> cull(d, ['out'])
{'x': 1, 'y': (inc, 'x'), 'out': (add, 'y', 10)}

Where should functions like these live?  core.py?  Some sort of analysis.py?

analysis.py seems good. The answer here would be:
{ ancestor(out) for out in outs }

with ancestor(out) in the DAG sense, right?

Hrm, what does ancestor return?
The result should be another dictionary, so one might write it in a similar form to what you've written above as the following:
{k: v for k, v in dsk.items() if depends_on(outs, k)}

For some definition (and very efficient implementation) of depends_on.

ancestor would essentially be the items for which depends_on(outs, k) returns True. I like yours better since I was implying a union of results which likely isn't necessary. Going through everything in the graph is probably fine because it's unlikely you would have a huge graph with only a small subset of the nodes surviving the cull.

Ah, I see, you're saying that we should start a traversal from just the out nodes.  That seems like a good idea.
I think that a common use case for this optimization would be relevant is to compute some giant blocked array expression X and then only ask for a small corner of that array (e.g. the first ten rows)  X[:10, :10, :10].

I think a more generic function like subgraph (or subdask, etc) that takes a dask, a node or list of nodes, and boolean keywords successors (or descendants, etc) and predecessors (or ancestors, etc) would be useful.
We can still add cull to the API, which would call the generic function.  If we wanted to get fancy (which, generally, we shouldn't get fancy unless there is a good reason too), we could have the node or list of nodes as optional input.  If not provided, the default nodes will be the nodes with no children.

Closed via #10.


Commit working unit test

This PR contains an example of a working unit test for ensureDaemonSet. The Unit tests will aim to cover all ensure functions and Reconcilers within CollectorReconciler. This should cover more than 50%. A Mock client has been set up to record basic Kubernetes API Client functionality and this will be used to determine whether the resources made are expected and contain the right fields. For testing reconcilers as a whole, we can test to see whether the desired resources have been created (as the ensure functions will be unit tested individually to ensure each resource is correct). Thus, I would like to also include TypeMeta Kind field when defining the resources to help distinguish between different resources.

Unit tests now easily cover 50-80% of working code. An issue I have encountered related to the Main Reconciler is that the Mock client Get does not update the obj struct pointer. Thus, when calling client Get in the main reconciler for the Kepler Instance, it will never work. I am not sure how to update the client.Object obj with the new fields as the interface seems to be lacking the needed methods. Any recommendations?

@KaiyiLiu1234 can you rebase the PR to re run the tests?

Unit tests use kubernetes Fake Client for mocking kubernetes api. Currently, the Main Reconciler and all sub reconcilers are covered by unit tests.


nested dom-repeat with sort attribute shows duplicate entries when adding new items.

When pushing new items into a nested array rendered by a dom-repeat with a sort attribute, duplicate items are rendered:
http://jsbin.com/fazazujamu/2/edit
(Click 'go' once, you should see items 11 and 21 duplicated in the output)
from my testing, this requires the array to be  nested inside another array which is rendered with dom-repeat, and it requires the items to be added one by one, AFTER the parent item is already rendered, and must be added using this.push.

I fixed this in your last jsBin also:
Original:
        go: function () {
            for (var i=0; i<DATA.length; i++) {
                var items = [];
              var cat = {'num': DATA[i].num, items: items};
                this.push('categories', cat);
                for (var j=0; j<DATA[i].items.length; j++) {
                    // `i` here is only ever 0 or 1, so you're always pushing new items
                    // into categories[0].items or categories[1].items
                    this.push('categories.' + i + '.items', DATA[i].items[j]);
                }

            }
        }

In the inner loop, i is only ever 0 or 1, so you're always pushing into the this.categories[0].items or this.categories[1].items.  On the second run, you push 2 more categories, so for those you want to be pushing into this.categories[2].items and this.categories[3].items:
Fixed:
        go: function () {
            for (var i=0; i<DATA.length; i++) {
                var items = [];
              var cat = {'num': DATA[i].num, items: items};
                this.push('categories', cat);
                for (var j=0; j<DATA[i].items.length; j++) {
                    // push onto the MOST RECENTLY ADDED category's items
                    this.push('categories.' + (this.categories.length-1) + '.items', DATA[i].items[j]);
                }

            }
        }

This works as expected:
http://jsbin.com/jicoxe/2/edit

@kevinpschaaf this does NOT resolve the issue.  if you look at your 'working as expected' version, you'll still see a duplicate entry at the end of each category. THAT is the issue I'm reporting.  yes, clicking the button twice does have unexpected results, but that is not relevant. don't click it twice, the bug is present after 1 click.
it's caused by pushing onto a nested array, and is still evident in your version. please re-open this ticket.

Got it-- sorry that was obscured by the other issue.

Confirmed the problem, will land a fix for this soon.  Thanks guys!

Thanks for the quick turn around, I am both the guys. :D

lol, I hadn't noticed the similarity in user names... :)

Issue will be resolved by https://github.com/Polymer/polymer/pull/1795
Confirmed it fixes the reported issue here:
http://jsbin.com/jicoxe/4/edit


Make Pixel Vision 8's 16 color palette the default colors in the Color Chip

Need to replace black and white with the official 16 color palette for Pixel Vision 8's Color Chip.

This has been added.


Delete study functionality

Depends on #1589
This adds the deletion of studies to the new layout.

@squirrelo could you pull from the branch and fix the conflicts? Thanks.

done.

:+1:

ping @josenavas

Few comments


ESP32 - Update advertising data to be closer to the spec

Problem
This PR update the ble advertising data to get it closer to the spec.
Summary of Changes

Update the advertising data on the esp32
Change the connection delegate to use the content of the advertising data instead of the connection name to connect to the peripheral
Change the code in examples/chip-tool/ to pass the discriminator instead of a name
Change the code in src/darwin to pass the discriminator instead of a name

@saurabhst @jelderton @BroderickCarlin ?

Will merge since my comments were more nit/meh. Would love an answer, however if we need to adjust code we can do it so in a separate PR.


add support for async/await

So currently i do something like this if i have an async rule:
app.post('/auth/signup', (req, res) => {
  const validation = new Validator(data, rules)

  function onSuccess () {
    return res.json({ message: 'user successfully created!' })
  }

  function onFailure () {
    return res.status(400).json(validation.errors)
  }

  validation.checkAsync(onSuccess, onFailure)
})

In my opinion this is very hard to look at and messy. Instead with proper async/await we could do something like this which is much more readable and cleaner:
app.post('/auth/signup', async (req, res) => {
  const validation = new Validator(data, rules)

  if (await validation.fails()) {
    return res.status(400).json(validation.errors)
  }

  return res.json({ message: 'user successfully created!' })
})

Are there any plans for implementing something like this?

I am sorry you feel the current version makes your code look hard to read and messy.  Your desired “solution” could be handled if you were to create a simple wrapper method (which is how I do it in my own apps)
Due to existing code, returning a promise from validator / fails (and / or passes) can’t be easily implemented as it would be a breaking change for all our existing users.
For now, my recommendation would be to create the aforementioned wrapper method.   We have considered the async/await approaching for a future version, but for now the current API will need to remain the same.

@higoka FYI, here is how I am using it
Note: The Validation class I am referencing is something new in Validator 3.5 (which will be released soon) but in short, it is a class that wraps around the existing validatorjs library and is something that could be created today (it does not need to be part of validatorjs package).
/*-------------------------------------------------------------------------------------------
 * Example using async/await validation module
 * the "Validation" module is custom class, which executes validator methods under the hood
 * -----------------------------------------------------------------------------------------*/

const Validation = require("../dist/Validation");

const data = { fname: "Mike", lname: "Erickson" };
const rules = { fname: "required", lname: "required|min:3" };
const fieldAliases = { fname: "First Name", lname: "Last Name" };

async function testValidator() {
  let result = { status: "pass", messages: {} };
  await new Validation().validate(data, rules, null, null, fieldAliases).catch((error) => {
    result = { status: "fail", messages: { ...error } };
  });

  console.log(result);
}

testValidator();

In this example, the console output would contain the default result.
{ status: 'pass', messages: {} }
If one of the rules were to fail, say not including the `lname` value, the response would be

```bash
{
  status: 'fail',
  messages: { lname: [ 'The Last Name field is required.' ] }
}


feat: replace the 'fast-url-parser' module with the 'node:url' module

closed #204

@gweesin You probably already know this but I think @leo was previously merging on this repo but has left Vercel (if his twitter is any indication). Not so sure this is getting merged unless someone else from over there has the ability to approve. Maybe @rauchg knows who hits the buttons on this repo now?

I indeed don't have access to this repo anymore, but I just pressed "View git blame" on the line that imports fast-url-parser, which took me to this commit. That commit replaces the native url module with fast-url-parser.
I was wondering why (I don't remember the reason), but looking at the readme of the fast-url-parser package and considering it has 4 million weekly downloads, it seems that the only reason why the package exists is to make the native url module 25x faster (e.g. for parse()). That's why it has the same API too.
So unless the native url module got a lot faster over the years (which is probably the case, but we'd have to test to what extend that is the case), using it might make serve-handler a lot slower in production.
But agreed that, if the owner of fast-url-parser is not responsive, replacing it in server-handler might be the only option. I think it is very likely that the native url module has gotten faster over the years.

I've run fast-url-parser benchmarks (higher is better) with Node v22: fast-url-parser is 2.5 times faster than node:url.

October 15, 2024 / Node 22.9.0
fast-url-parser
node:url
Diff

parse()
509333.95
216544.43
2.35

format()
567429.07
187700.54
3.02

resolve("../foo/bar?baz=boom")
579789.47
193241.46
3.00

resolve("foo/bar")
579329.57
194313.53
2.98

resolve("http://nodejs.org")
605089.97
190518.45
3.18

resolve("./foo/bar?baz")
607911.60
187227.32
3.25

For your information, here is the benchmark displayed in fast-url-parser's README dating back 11 years. fast-url-parser was 10 times faster.

December 29, 2013 / Maybe Node 0.10.x
fast-url-parser
node:url
Diff

parse()
509333.95
16459
24.4

format()
567429.07
15978
15.9

resolve("../foo/bar?baz=boom")
56701.419
6837.7
8.29

resolve("foo/bar")
80059.500
7038.6
11.4

resolve("http://nodejs.org")
118566.13
6491.1
18.3

resolve("./foo/bar?baz")
62778.648
6968.4
9.01

That seems like a reasonable tradeoff to me, considering that using the native module would also make the package lighter.
Great to see that Node has gotten a lot faster. I'll try to ping someone at Vercel to merge this PR.

Thanks for the PR @gweesin and thanks for the benchmarks @regseb!
I'll try to cut a release later today with those changes

Thank you all for hopping on this. Really appreciate it.


Import Failure Handling

I am testing much of the functionality here and I noticed that if I import records using the csv format, the tool fails on the whole import.
Would it be useful for it to succeed on what it can and report out status of failed (with the current error details) vs successful entries?

CSV import is intentionally all-or-none. Imagine you're importing a hundred interface connections. NetBox gets halfway through and finds an error. If it didn't roll back the transaction, you'd be stuck with half of your connections created and the other half still missing. This would require you to delete all of the successfully imported connections before attempting another import.
It's a lot simpler IMO to just fix the error and try re-importing the whole batch again, especially when you're exporting to CSV from Google Sheets or Excel.

Thanks for the insight into your thought process!


docs: fix Official External Commands section

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[N/A] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Work around ronn's inability to nest indents within list items by modifying its man page output, avoiding the need for the extra line breaks that was causing lines with pipes to become tables in the HTML output. This PR relies on Homebrew/homebrew-bundle#377 and Homebrew/homebrew-services#174, which adds colons to their help texts' command specs, matching those in core brew commands.

@EricFromCanada https://github.com/Homebrew/brew/pull/4383 has now been merged

Rebased. Also added commits that fix & clarify the changes made by #4383.

Thanks again @EricFromCanada, great work as usual!


Pagination

For accounts orders, fills, deposits, withdrawals and more.

added for most the calls where an array is returned (where it makes sense)

Should be addressed after PR #17


Install on shared hosting

Hello
I'm going through the installation instructions but it seems that I need to have root access to be able to install Miniflux. Is it possible to install without root access on shared hosting (Dreamhost in this case).
Many thanks for your advice : )

You need to have your own server or use a PaaS like Heroku.

Heroku did the trick. Thanks for your reply and for this fantastic app!


Can't be opened on macOS Monterey 12.0.1

After upgrading to macOS Monterey 12.0.1 Nightfall can no longer be opened.
The following pop-ups show up when trying:

The application "Nightfall" can't be opened

"Nightfall" will damage your computer. You should move it to the Bin.

Thanks for reporting this.
I was able to reproduce this issue with one of my Macs running Catalina, but not from a machine running Monterey. I'll release an updated version as soon as I'm able to identify the cause of the issue.

Yes, I can no longer open this either—tried installing through Brew and direct download.
Sounds like the app has been flagged as malware by Apple.

It seems to be a code signing problem— the codesign utility is showing the error CSSMERR_TP_CERT_REVOKED.
Interestingly, it seems to only affect the Intel version of the app.

I've re-signed the app here, please let me know if this resolves the problem.
Nightfall.zip

I've adjusted the app's signing here, please let me know if this resolves the problem.
Nightfall.zip

Yes, this version does fix this error for me. Thank you!

Should be fixed in version 2.1.3

@r-thomson faced same issue with version 2.1.3 on intel mac with catalina.

Version 2.1.3 still has signing issues. Gatekeeper is rejecting it because its considered an Ad-hoc signature.

Version 2.1.3 still has signing issues. Gatekeeper is rejecting it because its considered an Ad-hoc signature.

Gatekeeper will never accept this app because I don't have a Developer ID to sign with (it's not worth the $100/year for me just to distribute a free app).
If there's a better way to configure the signing, please let me know.

Indeed it won't open on MacOS 12.0.1 by default:

However, right-clicking the .app and selecting Open now reveals the usual option to override Gatekeeper. I also had to re-grant screen recording permission.

Thanks Ryan, I love this app!


JSONDecodeError occurs while trying to fetch 9gag memes

Hello,
I noticed, that an error occurs when trying to fetch content from /9gag and /9gagnsfw endpoints.
This is thrown while requesting one of endpoints written above.

Can you fix this err?
Thanks,
Matt.
PS. Kawał dobrej roboty 😄

Hi, thanks for reporting this issue!
I noticed that the error is caused by Cloudflare anti-spam protection, it's an HTML page so it's not a valid JSON. Unfortunately for us, that might be complicated to get around 😞
But I see hope as https://api.12345.pl/9gag is still working for some reason - maybe because they let Google scrape their site and this instance is hosted in Google Cloud Platform.
I will do some experiments, maybe it's time to pull out the big guns like puppeteer

Yep, it seems that Cloudflare let us crawl the site only because of the Google Cloud IP address. If you want to crawl 9gag do it from Google IP. You can host your instance with App Engine (app.yaml is already created) or set up a simple VPN on Compute Engine for easier development (OpenVPN is quite simple to do).
Changing User-Agent doesn't seem to change anything.
Something very interesting is that Puppeteer seems to get past that protection when not run in headless mode 🤔
I might close that issue and rephrase the problem, but I am not sure atm.


'You' omitted after tracking expenses in self DM despite adding Display Name

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.85-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Go to self DM, click the plus sign
Choose Track Expense, add amount
Complete the flow
Go to profile, add Display Name. Note 'you' is omitted after adding Display Name

Expected Result:
'You' should display even after adding Display Name
Actual Result:
'You' is omitted after tracking expenses in self DM and adding Display Name
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/a1fa8881-0f96-405c-86a4-37d0381c2efa
View all open jobs on GitHub

@lschurr  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

I think this should be held until this issue is resolved: https://github.com/Expensify/App/issues/43641

@sobitneupane - do you agree we should hold? https://github.com/Expensify/App/issues/43944#issuecomment-2178133095

Yup @lschurr. This looks related to https://github.com/Expensify/App/issues/43641 issue and should probably be solved by the backend change being made in https://github.com/Expensify/App/issues/43641 issue.

@lschurr are you sure this belongs in #newdot-quality? Is it a bug that happens consistently with reproducible steps?

Ah, let's see. It was on hold for https://github.com/Expensify/App/issues/43641 but that looks fixed now. @sobitneupane can you check?

@lschurr I could not reproduce the issue any longer.

Ok, going to close this one out since it's been fixed. I don't think there are any payments needed on this issue.


Angular Project - "node node_modules/webpack/bin/webpack.js --env.prod" exited with code 2

I'm trying to create a WebDeploy package to handoff to my system admins for the netcore2.0 angular 4 project.  but I keep getting the following error when I run the Publish command on the project in VS 2017...

The command "node node_modules/webpack/bin/webpack.js --env.prod" exited with code 2.

Attached in the error log from npm-cache...
2017-10-09T14_36_28_220Z-debug.log
It is basically not looking in the correct directory for the package.json file...
This command works...
node node_modules/webpack/bin/webpack.js --config webpack.config.vendor.js --env.prod
This command does not:
node node_modules/webpack/bin/webpack.js --env.prod
Any thoughts?

I might be off the mark here but I found your question while trying to fix the same error message I got after starting up the ReactRedux template. I also found this post, https://stackoverflow.com/questions/45764501/angular-dot-net-core-2-0-build-issue, so I went along and found that Visual Studio, Tools -> Options -> Projects and Solutions -> Web Package management -> External Web Tools has my path as a path. And that in my path variable I had two paths to nodejs, the actual one 'c:\program files\nodejs' and an old empty one 'c:\program files (x86)\nodejs'. I removed the old stinky path and bing.

I only have the $(PATH) variable itself... No other references to node...the system path only has 1 node reference...

Added piece of info... I think that it has something to do with TFVC and how it branches... I have recreated the app from scratch and used git instead and it builds the deploy package for me. I noticed in the log it was looking in the wrong folder for the package.json file... It wasnt looking down into the branch folder for it.  I thought I would be better off just using Git than trying to figure out the TFVC issue...let me know if you guys want to explore this further and I can try to help on the side... Just didn't have time this week...projects need to get done.

When you run webpack, you need to be sure that its current working directory is the root directory of your project (the directory containing package.json). That's the only thing I can think that might be causing it to look in the wrong place for package.json.
If you investigate further and believe there's a problem in any of the code in this repo, please let us know the details!

Still trying to fix this problem. I get this problem while deploying my app to Azure through VSTS. In local it runs fine however in VSTS build it gives me this problem:
Module not found : error : Can't resolve './../$$_gendir/ClientApp/app/app.module.server.ngfactory' in 'd:\a\1\s\ClientApp' [d:\a\1\s\MenuRushWeb.csproj]
@ ./ClientApp/boot.server.ts 7:0-94
EXEC : ERROR in error : Cannot determine the module for class UserDetailComponent in d:/a/1/s/ClientApp/app/components/aa-todelete-appseed/user-detail/user-detail.component.ts! Add UserDetailComponent to the NgModule to fix it. [d:\a\1\s\MenuRushWeb.csproj]
Cannot determine the module for class ControlMessages in d:/a/1/s/ClientApp/app/components/control-messages/control-message.component.ts! Add ControlMessages to the NgModule to fix it.
Cannot determine the module for class UserBasicInfoInputComponent in d:/a/1/s/ClientApp/app/components/user-basicinfo-input/user-basicinfo-input.component.ts! Add UserBasicInfoInputComponent to the NgModule to fix it.
Cannot determine the module for class UserPhoneInputComponent in d:/a/1/s/ClientApp/app/components/user-phone-input/user-phone-input.component.ts! Add UserPhoneInputComponent to the NgModule to fix it.
Cannot determine the module for class CounterComponent in d:/a/1/s/ClientApp/app/containers/aa-todelete-appseed/counter/counter.component.ts! Add CounterComponent to the NgModule to fix it.
Cannot determine the module for class NgxBootstrapComponent in d:/a/1/s/ClientApp/app/containers/aa-todelete-appseed/ngx-bootstrap-demo/ngx-bootstrap.component.ts! Add NgxBootstrapComponent to the NgModule to fix it.
Cannot determine the module for class UsersComponent in d:/a/1/s/ClientApp/app/containers/users/users.component.ts! Add UsersComponent to the NgModule to fix it.
at Error (native)
at syntaxError (d:\a\1\s\node_modules@angular\compiler@angular\compiler.es5.js:1699:2)
at analyzeAndValidateNgModules (d:\a\1\s\node_modules@angular\compiler\bundles\compiler.umd.js:24166:15)
at AotCompiler.analyzeModulesAsync (d:\a\1\s\node_modules@angular\compiler\bundles\compiler.umd.js:23797:46)
at CodeGenerator.codegen (d:\a\1\s\packages\compiler-cli\src\codegen.ts:41:10)
at Function.NgTools_InternalApi_NG_2.codeGen (d:\a\1\s\packages\compiler-cli\src\ngtools_api.ts:112:26)
at _donePromise.Promise.resolve.then (d:\a\1\s\node_modules@ngtools\webpack\src\plugin.js:430:58)
at process._tickCallback (internal/process/next_tick.js:103:7)
at Module.runMain (module.js:606:11)
at run (bootstrap_node.js:394:7)
at startup (bootstrap_node.js:149:9)
at bootstrap_node.js:509:3
d:\a\1\s\MenuRushWeb.csproj(73,5): error MSB3073: The command "node node_modules/webpack/bin/webpack.js --env.prod" exited with code 2.
Error: C:\Program Files\dotnet\dotnet.exe failed with return code: 1
Any help is appreciated

@suhaibqaiser This is an Angular usage question - some aspect of your application is not configured correctly for Angular's ahead-of-time (AoT) compilation. It's not specifically to do with publishing to Azure or VSTS. Please see Angular docs or check with the Angular community how to track down what's wrong.

I have same problem that happens when I try to publish to a folder. It runs find.. But something happened along then way that messed up publishing. Ive looked for hours for a fix.. So I guess Ill try doing the project again and separate the angular web site from the server

I know this is a bit of a hack but here is what i did just to get it working real quick to do some live testing
in your .csproj there is likely a build event named "PublishRunWebpack"
it will execute the webpack config and webpack.config.vendor.js
if you have not configured --env.prod correctly (which you likely haven't because its a pain) you can simply remove the --env.prod from the command.
this will execute the configs the same way they are executed locally
if you are having success with your application locally this shouldn't throw an error.
I successfully published after making this change but obviously ill be configuring --prod here soon and I suggest everyone else do the same lol

I started a new asp.net/angular project and got this error, running npm install did the trick.

@AESTechnology it's work for me.

@AESTechnology it's work for me.

This error occurred in my code as well. The fix for me was that the path for app.shared.module.ts file in my webpack.config.js file was incorrect. Corrected the path and error gone.


IndexLevel filter not working correctly

I've been porting over the IndexLevel code to React Native, and ran into a bug with the range filters used by the IndexLevel class.
An iterator is used to find what values are "less than" or "greater than" the target value. This breaks down, because the LevelDB database sorts these keys in byte order.
Here's an example of a failing test (index-level.spec.ts)
it('supports range queries with variable digit count', async () => {
  await index.put('1', { digit: 1 });
  await index.put('2', { digit: 2 });
  await index.put('10', { digit: 10 });

  const resp = await index.query({
    digit: {
      gte: '10'
    }
  });

  expect(resp.length).to.equal(1);
});

Instead of just one record being returned, we instead see that all three records are returned to the user. This is because the byte order of these keys are actually: [10, 1, 2]

This bug can also be seen with the EqualFilter as well with the following test:
it('supports equality queries', async () => {
  await index.put('1', { digit: 1 });
  await index.put('2', { digit: 2 });
  await index.put('10', { digit: 10 });

  const resp = await index.query({
    digit: 1
  });

  expect(resp.length).to.equal(1);
});

In this test, both the "1" and "10" records are returned, even though only one of those actually has a digit equal to 1.

Closing, thanks @amika-sq!


Why Extended Journaled instead of APFS?

Why do you advise to use macOS Extended Journaled to format the hard disk instead of APFS?

Also curious.

I used APFS without problem ;-)

I'm actually not sure the technical reasons of why. It just seems to work and APFS just doesn't. Sorry for the wait for this not very helpful comment.


How to embed multiple YouTube videos?

What question would you like to ask?
I created an asset field to add videos to an entry:

I want to include in the template not just one video but all videos which have been added to the field in the entry. When I use the following code, I get an error:
    {% set videos = craft.embeddedAssets.get(entry.videos.all()) %}

        {% for video in videos %}

        {% endfor %}

Twig error message:
TypeError
spicyweb\embeddedassets\Variable::get(): Argument #1 ($asset) must be of type craft\elements\Asset, array given, called in /v1/vendor/twig/twig/src/Extension/CoreExtension.php on line 1635
When I use one() instead of all(), I don't get an error but it will only include one video. Do you have a suggestion?
I also noticed the option in the plugin configuration to use YouTube Nocookie. With the above template setup it is not working. How does this configuration work? I also tried the variable {{ video.getUrl() }} to get the full YouTube URL but it didn't work.

video.getUrl() returns exactly the url value stored in the JSON file. To apply the nocookie domain, you'll need to use video.getIframeSrc([]).
I'm unsure why getIframeCode() is not working for you. Could you please post the video URL and I'll try it on my machine.
craft.embeddedAssets.get() doesn't support operating on an array of assets, although that's a good idea and something I'll look at when I get a chance. For now, you could do something like:

    {% for video in entry.videos.all() %}

            {% set video = craft.embeddedAssets.get(video) %}
            {# the rest of your code here #}

    {% endfor %}

Many thanks for your feedback. I think the main issue was that I was looping through the embedded assets instead of the entries containing the asset. With your reversed code it worked. I ended up with this solution:
        {% if entry.videos|length %}

Videos

            {% for video in entry.videos.all() %}

                {% set video = craft.embeddedAssets.get(video) %}
                {{ video.getIframeCode(
                    ['autoplay=0', 'controls=0', 'playsinline=1'],
                    ['loading=lazy'],
                    ['frameborder=0']
                ) }}

            {% endfor %}

        {% endif %}


Bicep or Terraform?
Bicep
Module Classification?
Resource Module
Module Name
avm/res/insights/scheduled-query-rule
Module Details
This module was previously owned by @emichellecarter.
Note: when someone volunteers to be the owner of this module, please reach out to @emichellecarter for

"un-orphaning " the module by performing step 5, here: When a New Owner is Identified
changing the ownership of the existing GitHub teams as well.

Additional links:

Module proposal: https://github.com/Azure/Azure-Verified-Modules/issues/376
Initial PR: https://github.com/Azure/bicep-registry-modules/pull/712
Code: https://github.com/Azure/bicep-registry-modules/tree/main/avm/res/insights/scheduled-query-rule

Do you want to be the new owner of this module?
No
Module Owner's GitHub Username (handle)
No response
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

@emichellecarter, please perform the steps under point 5. of this guide:. Thanks!


添加评论时候报错 Board not found

[2016-12-14T09:39:13.331Z] POST /api/comments/wiki/00148170165015348eed4f326694f0faeaddd40146689f1000
[API Request]
{
"tag": "wikiffffffffffffffffffffffffff",
"name": "asdfas",
"content": "大是打发斯蒂芬"
}
17:39:13.332 [Warp@ca6d3618] CONNECTION: opened from pool: 1
17:39:13.332 [Warp@ca6d3618] SQL: select id, cover_id, content_id, views, name, tag, description, created_at, updated_at, version from wikis where id='00148170165015348eed4f326694f0faeaddd40146689f1000'
17:39:13.332 [Warp@ca6d3618] CONNECTION: released to pool: 0
17:39:13.333 [Warp@ca6d3618] CONNECTION: opened from pool: 1
17:39:13.333 [Warp@ca6d3618] SQL: select id, topics, locked, tag, name, description, display_order, created_at, updated_at, version from boards  order by display_order
17:39:13.333 [Warp@ca6d3618] CONNECTION: released to pool: 0
X-Execution-Time: 3
[Error] error when handle url: /api/comments/wiki/00148170165015348eed4f326694f0faeaddd40146689f1000
undefined
[API Response]
{
"error": "entity:notfound",
"data": "Board",
"message": "Board not found."
}

board和wiki的tag要对上


Plot of Equity Curve normalized by Gross Leverage timeseries

This would provide a hypothetical view of how an algo would peform if the algo implemented order logic such that it was always invested at 1.0 leverage every day
The Gross Leverage timeseries is available in the zipline daily perf packet I believe

I just realized this is more difficult than anticipated unless the gross leverage normalization is done at the minutely level.  Case being is that the zipline daily perf packet gross leverage timeseries is simply the gross leverage at the end of the day, which might be a really small value even if the portfolio had a really high leverage during the day and created a lot of profit. So you wouldn't want to normalize using this really small end-of-day gross leverage value and unrealistically inflate the profits by a large factor

I think we do that now.


gracefully fallback when fontawesome not available

I think this is just a nice-to-have now, not necessary. But, in the React app without fontawesome available, it looks like this:

I was thinking to drop the fontawesome class on the wrapping template, and add a styling option that can take a fontawesome value.. that should give you default icons..

It now no longer uses fontawesome by default. Make sure to link both vis.css and ml-visjs-graph.js.less in your wrapper component though.

I may have failed to thoroughly test this. Maybe this is a good reason for a short screen share tomorrow?

The mis-alignment of orb-labels is not related to fontawesome. I opened #34 for that, and closing this ticket..

Moved it back to original milestone..


Fix issue that no valid semantic version tag found when installing from source via personal repo

For the multi-backend-refactor branch, if we git clone the source code from our personal repository and install it, we will get ValueError: No valid semantic version tag found.
This PR fixes this problem.
cc @Titus-von-Koeller @matthewdouglas

Thanks @statelesshz, makes sense. We appreciate your initiative in providing a fix!


Refactor swe book

Added chapters on below areas, its code to collect the data, processing. Added scientific and code explanation

fsca
snotel
DEM

@ZihengSun removed the build files


Will be update?

Real need option to disable closing modal by click on modal overlay

Content with big height vot visible in this modal.

Hi @Poldan, thank you for the comment.

The first one, plainmodalbeforeclose event may help you.
See: https://github.com/anseki/jquery-plainmodal/issues/3, https://github.com/anseki/jquery-plainmodal/issues/9
The second one, more details please.

Hi @anseki
Window screen has height 600px. Modal content has height 900px.
How to show hidden contents?

Typically, a long content is shown by scrolling frame such as overflow: scroll/overflow: auto.

http://getbootstrap.com/javascript/#modals
Can i do something, like bt modal with your plugin?
If modal block will be inside wrapper(now his place near wrap), then i can do:

wrapper is fixed, overflow:auto
modal is absolute, and we have scroll on the right side of the screen

p.s. sorry, my english is so bad

Do you want the Bootstrap style?
Or, do you want to make a scrollable element?
For example, implement both:
https://jsfiddle.net/L0cug7vn/show/ (Click "Open Modal" button)

See http://anseki.github.io/jquery-plainmodal/ for the style.
See the .modal-body class in the example for the scrollable element.

BTW, was the first question (disabling closing the modal window) in your comment already solved?
I change the title of this issue. Please make a new issue if you want. You should make one issue by one topic because this is issue tracker system.

I have added new option : {'closeOnModal':boolean}, and, when it's true, don't create click events on modal window.
Scrollbar inside window is working, but creating some bad thing (max-height must be setted).
But if it works, i can use it :+1:
Thanks for answersm @anseki

The closeOnModal option is unnecessary because those events are cancelable.
A scrollable element and its style like max-height are unrelated to the jQuery-plainModal. It is CSS style specification.
You can learn at here: https://developer.mozilla.org/en-US/docs/Web/CSS/overflow
The jQuery-plainModal should do nothing about the style.
Anyway the issues were solved? I close this issue.


css Update introjs-nassim.css

Checklist

[x] I have followed the Contributor Guidelines.
[x] The code has been thoroughly tested in my local development environment with flake8 and pylint.
[x] The code is Python 3 compatible.
[x] The code follows the PEP8 styling guidelines with 4 spaces indentation.
[x] This Pull Request relates to only one issue or only one feature
[x] I have referenced the corresponding issue number in my commit message
[x] I have added the relevant documentation.
[x] My branch is up-to-date with the Upstream master branch.

Changes proposed in this pull request
Your development environment

OS: x
OS Version: x
Python Version: x

Closing this out as inactive, feel free to reopen when it's ready.


Modification in Makefile of sw/ directory, for spmv-fp to work

Hello.
Thank you for your work. I would like to suggest a little change to your Makefile (located here).
For the implementations of floating-point spmv (float_pob, float_stall), you have to add -DFP_POB or -DFP_STALL accordingly to the CXXFLAGS, so that the compilation of host or benchmark completes succesfully.
These flags are already set here, but not visible in the previously referred "main" functions in sw/ directory.
Thank you

Thanks for your comments! Could you please show the error message you get? The host.cpp and benchmark.cpp programs are designed to work for all designs so they don't need to see the flags. The compilation works well on our end.

Closed for being inactive.