Add a Decoder

Hello Markus!
I found your library while searching for OPUS bindings for Haskell. Your encoder has been really useful.
As I see it, the code only has an Encoder, but I'd love to decode some OPUS audio as well. So here's a pull request!
Changes in this PR

Added Codec.Audio.Opus.Decoder, which mirrors the Encoder variant for the most part, but also all specs in the OPUS API PDF
Added Codec.Audio.Opus.Decoder.Conduit, which mirrors the Encoder variant closely.
Added test vector tests for the new decoder, which mirrors the run_vectors.sh file in the OPUS repo

Decoder tests
The tests require opus_compare executable in the project root (which can be obtained by compiling this file), and the new opus test vectors (https://opus-codec.org/static/testvectors/opus_testvectors-rfc8251.tar.gz), but once those are in place, passes properly:

I would've loved to add similar tests for the Encoder, but the original run_vectors.sh only had decoder testing so I haven't pursued further.
I hope you can take a look/review it, and hopefully merge it in!

Whoops, noticed something, converting to a draft temporarily.

Fixed


PVRCNN tested on my dataset, got a lot of false postive

How to fix this problem?
I am using PVRCNN to test on my 32 beam dataset, pretrained by Pandaset 64 beam.

Hello, sorry to ask this, but could you tell me which tool did you use to visualize the results?

How to fix this problem?
I am using PVRCNN to test on my 32 beam dataset, pretrained by Pandaset 64 beam.

Hello if you are ok, Could i get pretrained by Pandaset 64 beam?
<EMAIL_ADDRESS>-> this is my email address.
I tried train own my dataset. But it keeps failing.
If you can help me, it will be a great help to me :)
Thk

Not so sure about what problem you may have. But can refer to my successful achievement on training my own dataset and inference correctly https://github.com/OrangeSodahub/CRLFnet


readme doc correction

Just spotted something
the issue link has the old project name url which is incorrect,it should be:
https://github.com/googlesamples/android-testing-templates/issues

Thanks!


Message reset renamed to clearHint

Already done. See https://github.com/ikr/react-star-rating-input/commit/d5a5ba344ecd2d348771b222a1dd556768c3879b


Go Client Example?

Hi
I'm just wondering if there is a Go client example (not HTML+JS) anywhere?
Thanks
Ken

@kpfaulkner There is no go client example, sorry for that.

Am I right in saying that if I find a go client that is pomelo  compatible, then in theory it would work?

@kpfaulkner Yes, and the examples contain a tough implementation, maybe you can refine it and make it ready for production.


always add trailing slash to base path

fixes #9341.
In #9220, we changed the logic for resolving asset paths to accommodate the relative: true option, and included special handling for the case where you're rendering the root page when base is set — if the base path is /a/b/c and you render /a/b/c, then the relative path (somewhat counter-intuitively) needs to be ./c for links to work.
Turns out that was a mistake, because the root should always end with a trailing slash — if you prerender that page, it will be output as <output>/index.html, and relative paths will thus be incorrectly converted to <output>/c/....
The fix is to remove the special case handling, and ensure that /a/b/c always redirects to /a/b/c/. This is much simpler and more correct. It arguably falls into the semver gray zone where someone somewhere might be relying on the buggy behaviour, but I think it's vanishingly unlikely, and in any case the worst that will happen is a URL that used to render something gets turned into a redirect.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

If you're on /base/, do like this fix does, if you're on /base, keep the old behavior.

/base/ was already treated correctly, it was only /base where things (quite reasonably, in retrospect) got messed up

Dear @Rich-Harris,
I tried the PR with my code, now.
Unfortunately, this PR only partly solves the problem (at least for me).
It works fine when I call the base path directly, in my case "http://localhost/build"
It also works following links to routes inside the already loaded SPA.
It does not work when calling a route below the base path in the browser directly, in my case "http://localhost/build/Termine" or "http://localhost/build/Termine/" (with or without trailing slash)
Am I missing something?
I'll attach the created scipt tag in index.html in different versions:
URL "http://.../build/Termine"
Sveltekit version 1.8, working correctly:
<script>
    {
        __sveltekit_rs4xzv = {
            env: {},
            assets: "/build",
            element: document.currentScript.parentElement
        };

        Promise.all([
            import("/build/_app/immutable/entry/start.3791b6e2.js"),
            import("/build/_app/immutable/entry/app.e98f8777.js")
        ]).then(([kit, app]) => {
            kit.start(app, __sveltekit_rs4xzv.element);
        });
    }
</script>

Sveltekit version 1.11, neither working on base URL nor on path under base URL:
<script>
    {
        __sveltekit_1h6aion = {
            env: {},
            assets: "/build",
            base: new URL("./build", location).pathname,
            element: document.currentScript.parentElement
        };

        Promise.all([
            import("./build/_app/immutable/entry/start.6980a169.js"),
            import("./build/_app/immutable/entry/app.35650362.js")
        ]).then(([kit, app]) => {
            kit.start(app, __sveltekit_1h6aion.element);
        });
    }
</script>

Sveltekit version PR 9343, working correctly on base URL, not on route under base URL:
<script>
    {
        __sveltekit_inylau = {
            env: {},
            assets: "/build",
            base: new URL(".", location).pathname.slice(0, -1),
            element: document.currentScript.parentElement
        };

        Promise.all([
            import("./_app/immutable/entry/start.a989781b.js"),
            import("./_app/immutable/entry/app.3bd555c8.js")
        ]).then(([kit, app]) => {
            kit.start(app, __sveltekit_inylau.element);
        });
    }
</script>

My paths / base setting is '/build', but could be anything.
The problem seems to be that the path "./_app/immutable/entry/start.a989781b.js" is evaluated by the browser to
/build/Termine/_app/immutable/entry/start.a989781b.js
But only the file
/build/_app/immutable/entry/start.a989781b.js
was created
My .htaccess file is as follows
RewriteEngine On
RewriteCond %{REQUEST_FILENAME} !-f
RewriteRule .* index.html [NC,L]

I was not able to find a working RewriteRule to work around the new behavior.
Would it be the best to always use an absolute path when using paths/base?
Thank you already!

It sounds like you're rewriting all paths to index.html? If so, there's no reason to expect that to work! You need to use a fallback page (which must use absolute paths) if you're building an SPA.


Same constant hardcoded in multiple places

The percentage of bid value, the bidder receives for unsealing valid non-winning bid (currently 99.5%) is hardcoded in multiple places:

https://github.com/ethereum/ens/blob/master/HashRegistrarSimplified.sol#L339
https://github.com/ethereum/ens/blob/master/HashRegistrarSimplified.sol#L339
https://github.com/ethereum/ens/blob/master/HashRegistrarSimplified.sol#L339
https://github.com/ethereum/ens/blob/master/HashRegistrarSimplified.sol#L339

This makes changing this constant very error-prone.

Good point, thanks.

+1 Related #146 #147


Yosemite fix

Summary of fixes which should resolve #4

Changed the CUDA check to look for nvcc
Explicitly remove cunn/cmake if we're on OS X 10.10 (ideally there'd be a PR to cunn which obviates this)
Force the use of clang, because CUDA does not work with recent GCC versions.

thanks a lot. getting the most uptodate packages was a design decision. i'm open to splitting it into update.sh and install.sh separately.

Thanks for the speedy merge.
Re: pulling from master, understood, I think it's fine behavior, was just curious.
I will be working on my fork for awhile, and I'll check back in when I have a distribution with the "kitchen sink" install working well on OS X and Linux.
Also, it's a little unclear to me if this repo requires itself to be installed in torch-distro. It would be advantageous if this directory name could be flexible, and if the directory was moved, if the only updates required to keep it working were changes to PATH.

due to rpath weirdness in the way it's done so far, once you install it, moving the directory will break shared object rpaths, and you'll have to reinstall after moving the directory.
if you always use run.sh to actually run torch, then you can move the directories around, but I usually dont use run.sh, I add torch-distro/install/bin to my PATH

Is it supposed to install cunn now?

Hrm, I suppose I'll think about how best to make it more flexible. I also
have been adding the bin directory to my PATH, so that i can run th and
itorch from the shell.
On Fri, Jan 16, 2015 at 2:11 PM, Sergey Zagoruyko<EMAIL_ADDRESS>wrote:

Is it supposed to install cunn now?
—
Reply to this email directly or view it on GitHub
https://github.com/soumith/torch-distro/pull/5#issuecomment-70305446.

I mean it's not installed for me now, cutorch is, but cunn gives error:
th> require('cunn')
.../rocks/torch-distro/install/share/lua/5.1/trepl/init.lua:319: loop or previous error loading module 'cunn'
stack traceback:
	[C]: in function 'error'
	.../rocks/torch-distro/install/share/lua/5.1/trepl/init.lua:319: in function 'require'
	[string "require('cunn')"]:1: in main chunk
	[C]: in function 'xpcall'
	.../rocks/torch-distro/install/share/lua/5.1/trepl/init.lua:588: in function 'repl'
	...rch-distro/install/lib/luarocks/rocks/trepl/scm-1/bin/th:185: in main chunk
	[C]: at 0x010f805630

This is using the most recent merge?
On Fri, Jan 16, 2015 at 2:18 PM, Sergey Zagoruyko<EMAIL_ADDRESS>wrote:

I mean it's not installed for me now, cutorch is, but cunn gives error:
th> require('cunn')
.../rocks/torch-distro/install/share/lua/5.1/trepl/init.lua:319: loop or previous error loading module 'cunn'
stack traceback:
[C]: in function 'error'
.../rocks/torch-distro/install/share/lua/5.1/trepl/init.lua:319: in function 'require'
[string "require('cunn')"]:1: in main chunk
[C]: in function 'xpcall'
.../rocks/torch-distro/install/share/lua/5.1/trepl/init.lua:588: in function 'repl'
...rch-distro/install/lib/luarocks/rocks/trepl/scm-1/bin/th:185: in main chunk
[C]: at 0x010f805630
—
Reply to this email directly or view it on GitHub
https://github.com/soumith/torch-distro/pull/5#issuecomment-70306428.

I've just checked 2f2f2c3b795c26ab4d1da474dc3fb5378ec88d73, gives the same error. If I go to cunn and manually call luarocks make it doesn't work either. Something is broken.

Oh this might be the recent SpatialAverageMaxPooling problem in the master
of nn. If nn breaks then that's why.
On Fri, Jan 16, 2015 at 2:46 PM Sergey Zagoruyko<EMAIL_ADDRESS>wrote:

I've just checked 2f2f2c3
https://github.com/soumith/torch-distro/commit/2f2f2c3b795c26ab4d1da474dc3fb5378ec88d73,
gives the same error. If I go to cunn and manually call luarocks make it
doesn't work either. Something is broken.
—
Reply to this email directly or view it on GitHub
https://github.com/soumith/torch-distro/pull/5#issuecomment-70310746.

yeah sorry that's the bug we introduced :( fixing it now

This might be an argument for having submodules point to specific commits?
On Fri, Jan 16, 2015 at 3:33 PM Sergey Zagoruyko<EMAIL_ADDRESS>wrote:

yeah sorry that's the bug we introduced :( fixing it now
—
Reply to this email directly or view it on GitHub
https://github.com/soumith/torch-distro/pull/5#issuecomment-70317699.

tested master works! cunn imports normally.
regarding cmake folder in cutorch and cunn, I see two possibilities, one is to remove it and set CMAKE_MINIMUM_REQUIRED(VERSION 2.8) instead of 2.6, another one is to remove the line
SET(CMAKE_MODULE_PATH "${CMAKE_CURRENT_SOURCE_DIR}/cmake" "${CMAKE_MODULE_PATH}")

for OS X build. According to this http://www.cmake.org/Wiki/CMake_Life_Cycle_Considerations 2.8 will break the support of some old distributions. @soumith what do you think?

If my vote counts, I would push for 2.8 (or higher). Ubuntu 13 supports 2.8, and it itself is EOL'd. Most folks walking down this Lua/Torch road are going to be doing some intensive stuff, making it likely that they're using modern machines with modern OS's.
Also, interest in Torch seems to be ramping up (with the press and all), so any breaking changes should happen earlier than later. If future-proofing is possible now, now is a good opportunity.

i am completely in, will open a PR to do so.

done, made cutorch and cunn mainlines have 2.8 as minimum CMake version

ok, also removed the FindCUDA files. Turns out, torch7 was already on cmake minimum version 2.8, cutorch/cunn were lagging behind on the update.

Thanks soumith! we can revert 277e98cca317acf3268fc964dba5feab42dad447 as it's not needed any more. Also @fmassa pointed that it fails if nvcc is not in PATH

We'll need to then return an empty string from that command, because it
would fail for me when doing $(which nvidia-smi).
On Mon, Jan 19, 2015 at 10:17 AM, Sergey Zagoruyko<EMAIL_ADDRESS>
wrote:

Thanks soumith! we can revert 277e98c
https://github.com/soumith/torch-distro/commit/277e98cca317acf3268fc964dba5feab42dad447
as it's not needed any more. Also @fmassa https://github.com/fmassa
pointed that it fails if nvcc is not in PATH
—
Reply to this email directly or view it on GitHub
https://github.com/soumith/torch-distro/pull/5#issuecomment-70509770.

let me know if this commit works for you guys:
https://github.com/soumith/torch-distro/commit/a236eedd2f093ee46445dd0a2eea71d4d06954db
@alexbw giving you commit rights to torch-distro, feel free to push minor fixes as you please, i like your earlier PR.


Use  instead of background-image for persona previews

Fixes #1426

Looks like there's a test that needs updating to look for the img

Indeed, just fixed it, thanks!

Closing for now, we don't need to get rid of unsafe inlines for the moment.


[Feature] Option to Execute Command Upon Entering State

As the title denotes, you could designate a command to run for when the state switches to a particular state; it would run once and that would be it until it switches to a different state (assuming the config. file has any command set for said new state).
I'm thinking this could be useful in, say, the battery module to execute a notification to dunst when entering state full or even run a command to suspend when entering the state critical.
I'm assuming it wouldn't involve much more as waybar has to keep track of the state, anyway, so it's just executing a user command along with changing the icon, etc. but I always could be mistaken.

I'm thinking this could be useful in, say, the battery module to execute a notification to dunst when entering state full or even run a command to suspend when entering the state critical.

I was looking for exactly the same feature.

me too!

This sounds like a duplicate of https://github.com/Alexays/Waybar/issues/670


Sway segfault in flutter::ELinuxEGLSurface::MakeCurrent()

Hi,
I am using an i.MX8MP board and Sway.
When I use Weston, I can launch the app gallery, but when I use Sway, I get a segmentation fault.
Using GDB, I can trace back to the following:
Starting program: /home/user/Gallery/gallery -b .
...
Thread 6 "io.flutter.io" received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0xffffdd9ee0c0 (LWP 1083)]
0x0000fffff7a87a84 in wl_proxy_set_queue () from /usr/lib/libwayland-client.so.0
#0  0x0000fffff7a87a84 in wl_proxy_set_queue () from /usr/lib/libwayland-client.so.0
#1  0x0000fffff79fc664 in ?? () from /usr/lib/libEGL.so.1
#2  0x0000fffff79ed774 in ?? () from /usr/lib/libEGL.so.1
#3  0x0000fffff7f6c9b4 in flutter::ELinuxEGLSurface::MakeCurrent() const () from /home/user/Gallery/lib/libflutter_elinux_wayland.so
#4  0x0000fffff704929c in ?? () from /home/user/Gallery/lib/libflutter_engine.so
#5  0x0000fffff758b218 in ?? () from /home/user/Gallery/lib/libflutter_engine.so
#6  0x0000fffff707828c in ?? () from /home/user/Gallery/lib/libflutter_engine.so
#7  0x0000fffff707e3cc in ?? () from /home/user/Gallery/lib/libflutter_engine.so
#8  0x0000fffff707813c in ?? () from /home/user/Gallery/lib/libflutter_engine.so
#9  0x0000fffff707dc48 in ?? () from /home/user/Gallery/lib/libflutter_engine.so
#10 0x0000fffff7b902a8 in start_thread (arg=0xffffffffddf7) at pthread_create.c:442
#11 0x0000fffff7bf7bdc in thread_start () at ../sysdeps/unix/sysv/linux/aarch64/clone.S:79

Also the following command yield :
>FLUTTER_LOG_LEVELS=TRACE ./gallery -b .
[TRACE][operator()(171)] presentation info: clk_id = 1
[INFO][operator()(573)] Display output info: width = 1920, height = 1080, refresh = 60000
[INFO][operator()(606)] Display scale for output(10): 1
[TRACE][CreateRenderSurface(1148)] Created the Wayland surface: 1280x720

Thanks for the bug report.
Can this bug be reproducible on desktop Linux PC (e.g. Ubuntu 22)?

im going to try to setup a vm ubuntu jammy jellyfish arm64. Ill get back to you soon!

Im back with some news, I have built a VM with fedora aarch64, installed sway + the flutter app and they were working fine.
Turn out the driver imx-gpu-viv form NXP (libEGL.so.1) doesnt fully support wlroot .
You can have more info: https://community.nxp.com/t5/i-MX-Processors/GLFW-Vulkan-applications-crash-on-imx8mp-with-wlroots-based-WM/td-p/1572600

You may try to use the open-source mesa etnaviv driver instead.


BuildError Raspberry Pi 4 aar64 armv8-a

I also submitted this issue to sassc-rails (https://github.com/sass/sassc-rails/issues/138).
How to reproduce:
$ bundle add sassc-rails on a raspberry pi 4 (balena 64bit OS 2.44.0+rev3)
Observation:
...
[main]     Fetching sassc 2.2.1
[main]     Installing sassc 2.2.1 with native extensions
[main]     Gem::Ext::BuildError: ERROR: Failed to build gem native extension.
[main]         current directory: /usr/lib/ruby/gems/2.5.0/gems/sassc-2.2.1/ext
[main]     /usr/bin/ruby -r ./siteconf20191111-1-1oooxp3.rb extconf.rb
[main]     creating Makefile
[main]     current directory: /usr/lib/ruby/gems/2.5.0/gems/sassc-2.2.1/ext
[main]     make "DESTDIR=" clean
[main]     current directory: /usr/lib/ruby/gems/2.5.0/gems/sassc-2.2.1/ext
[main]     make "DESTDIR="
[main]     compiling ./libsass/src/cencode.c
[main]     cc1: error: unknown value 'armv8-a-march=armv8-a' for -march
[main]     cc1: note: valid arguments are: armv8-a armv8.1-a armv8.2-a armv8.3-a armv8.4-a
[main]     native
[main]     make: *** [Makefile:243: cencode.o] Error 1
[main]     make failed, exit code 2
[main]     Gem files will remain installed in /usr/lib/ruby/gems/2.5.0/gems/sassc-2.2.1 for
[main]     inspection.
[main]     Results logged to
[main]     /usr/lib/ruby/gems/2.5.0/extensions/aarch64-linux/2.5.0/sassc-2.2.1/gem_make.out
[main]     An error occurred while installing sassc (2.2.1), and Bundler cannot continue.
[main]     Make sure that `gem install sassc -v '2.2.1' --source 'https://rubygems.org/'`
[main]     succeeds before bundling.
[main]     In Gemfile:
[main]       sassc-rails was resolved to 2.1.2, which depends on
[main]         sassc

My guess
Note these two lines:
[main]     cc1: error: unknown value 'armv8-a-march=armv8-a' for -march
[main]     cc1: note: valid arguments are: armv8-a armv8.1-a armv8.2-a armv8.3-a armv8.4-a

It seems that a wrong option is generated:
Actual: -march=armv8-a-march=armv8-a
Expected: -march=armv8-a

Possibly related to #150 ?

Also I should add that I am performing the build in a docker image using balena.io cloud build infrastructure.
Docker image tag: balenalib/aarch64-alpine:latest-build

You can set BUNDLE_BUILD__SASSC=--disable-march-tune-native to disable -march=native

https://github.com/sass/sassc-ruby/issues/222#issuecomment-857092485
Current work around: gem install sassc -- --disable-march-tune-native


[CS2103T-W13-1] Networkers

Networkers helps ISP technicians who visit their clients often remember the details of their clients and the procedures done during the visit to the client. It is optimized for CLI users, assisting them in storing important information quickly.

Codecov Report

Merging #40 (9ce30ac) into master (4f7660a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #40   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

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


Setting Ski Profile - Type of Skiing Datapoint

Describe briefly your feature.
The issue is that currently there is no dedicated subsection in the settings page for managing ski profile information, such as the type of skiing and others.
Describe the solution you'd like
The solution is to create a new page within the settings menu specifically for Ski Profile. Within this, an input field named as Type of Skiing will be added in the dropdown format which takes values -Downhill, Touring, and Snowpark.
The users should be able to easily view and edit their preferred type of skiing.
Also to ensure that this information is accessible through appropriate class/public functions and remain consistent.

Successfully added the data point for Ski type in Ski Profile subsection.


[ContainerRegistry] Fix SDKType for mgmt plane

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

/azp run net - containerregistry - mgmt

same as https://github.com/Azure/azure-sdk-for-net/pull/31299, please just close this one


How to Optimize app.js?

I think what he meant was that the app size is large. I am working with inertiajs. As the system grows larger the size becomes larger as well. How to maybe optimize it so that it only loads needed js.
In this picture you can see that my app.js is really large, when I am just rendering a new page with hello world p tag on it.

You can split the coee with dynamic import
https://inertiajs.com/client-side-setup here

Yes, code splitting is an option, but 12.1MB is a HUGE file. I have a very large Inertia app, with hundreds of pages, and it's still only 1.9MB, and it compresses down to 398kB. I'd recommend reviewing all of your dependencies to see which ones you can trim.
Closing this because it's not an issue with the Laravel adapter.
Good luck! 💪


fix(navbar): fix props for Navbar

Exclude won't remove setOpen, Omit will do.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Header: use the new calcite navigation component

Use the new calcite navigation component in the header.
https://developers.arcgis.com/calcite-design-system/components/navigation/

This has been added


feat (mapfilter) - Add mapFilter operator + docs

I added the mapFilter operator proposed on #34
after that I'll create a separate PR for debug operator.
The commit also added the mapfilter.md for docs, don't found the astro config for link it, maybe now is it all automated!?

@nartc about the failing CI I need to write the @tsignore inside the TEST to prove that the mapFunction correctly handle the case when someone don't produce/return any value for some code-path.
Locally the test pass, but not in CI can you please help me or suggest a valid workaround? Thanks

We'll revisit the PR after observer-spy is in. At the moment, can you fix the rebase since your PR contains code that's already in main?

@nartc I'll close this PR and tomorrow will redone all works from ground up in a new PR correct naming mapSkipUndefined, and simplifying test using toArray() as suggested by @tomalaforge hoping this will make everyone happy and then will be approved.
I need to go sleep tonight, tomorrow I'll done it, promised!


We should not always trigger CI/CD

Task description:
CI/CD like go cicd or e2e test will take lots of time. Sometimes we just modify the docs, it will trigger some meaningless workflows.
I suggest adding path filters for some worklfows: https://github.com/dorny/paths-filter
Solution:
Who can join or take the task:
The good first issue is intended for first-time contributors to get started on his/her contributor journey.
After a contributor has successfully completed 1-2 good first issue's,
they should be ready to move on to help wanted items, saving the remaining good first issue for other new contributors.
How to join or take the task:
Just reply on the issue with the message /assign in a separate line.
Then, the issue will be assigned to you.
How to ask for help:
If you need help or have questions, please feel free to ask on this issue.
The issue author or other members of the community will guide you through the contribution process.

https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#onpushpull_requestpull_request_targetpathspaths-ignore  Github actiopn provide such functionality


tag_list attribute is nil

Hello!
Rails 6.0
acts-as-taggable-on: 6.5.0
I have a taggable model, "Person", which I want to serialize to json. when I access tag_list as an attribute on an instance of "Person" seems to be empty, even though both the instance has been tagged and the cached_tag_list is up to date.
i.e.:
@person=Person.new()
@person.tag_list.add('foobar')
@person.save
attributes = @person.attributes
nil == attributes.tag_list # this is true
['foobar'] == @person.tag_list  # this is also true

Is this the expected behavior?
I'm a huge fan of this gem. Use it in almost all of my projects. Truly appreciate all of the work that goes into it.
Thanks!

Have you tried upgrading to 7.0.0?

Have you tried upgrading to 7.0.0?

Same problem

Just confirming that this is an issue with v9 as well. I could possible find some time to look into it, assuming that these attributes should be present.


npm audit fix does not work due to packages being listed as bundled

Not sure why these are listed as bundled dependencies. Could it be changed, or dependencies updated? Atm I can't patch xml2js vulnerability.
npm WARN audit fix<EMAIL_ADDRESS>node_modules/cdk-nextjs-standalone/node_modules/xml2js
npm WARN audit fix<EMAIL_ADDRESS>is a bundled dependency of
npm WARN audit fix<EMAIL_ADDRESS><EMAIL_ADDRESS>at node_modules/cdk-nextjs-standalone
npm WARN audit fix<EMAIL_ADDRESS>It cannot be fixed automatically.
npm WARN audit fix<EMAIL_ADDRESS>Check for updates to the cdk-nextjs-standalone package.
npm WARN audit fix<EMAIL_ADDRESS>node_modules/cdk-nextjs-standalone/node_modules/aws-sdk
npm WARN audit fix<EMAIL_ADDRESS>is a bundled dependency of
npm WARN audit fix<EMAIL_ADDRESS><EMAIL_ADDRESS>at node_modules/cdk-nextjs-standalone
npm WARN audit fix<EMAIL_ADDRESS>It cannot be fixed automatically.
npm WARN audit fix<EMAIL_ADDRESS>Check for updates to the cdk-nextjs-standalone package.

@edvinerikson, this is the nature of a construct package that is packaged for multiple programming languages. Dependencies must be bundled. If you'd like a dependency's version to be updated, please test it out locally, then submit a PR. Please re-open if you think this issue is not addressed. Thank you!


Purge-protection flag

Some comments to documentation, by default and if the parameter is not specified the "enablePurgeProtection" is show as "null" what's it means? off / on?

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

ID: 1c0af272-74a9-6fb9-6079-8d6d0b800a6a
Version Independent ID: 79354b48-c4f0-6361-cf6b-8afc9d70c97b
Content: Azure Key Vault soft delete
Content Source: articles/key-vault/key-vault-ovw-soft-delete.md
Service: key-vault
GitHub Login: @BryanLa
Microsoft Alias: bryanla

@lestaat Thanks for your feedback! We will investigate and update soon on it.

@lestaat Please note that as stated in Purge protection flag section of the document - Purge protection(--enable-purge-protection in Azure CLI) flag is off by default. When this flag is turned on, a vault or an object in deleted state cannot be purged until the retention period of 90 days has passed.

@lestaat We will now proceed to close this thread. If there are further questions regarding this matter, please open a new issue and we will gladly continue the discussion.


Add failing tests for concat/unconcat ops

Added some failing tests for something that I expected to work and I need to build a LSTM layer. One of the errors I get is:
=== RUN   TestUnconcatConcatOpSequence
--- FAIL: TestUnconcatConcatOpSequence (0.00s)
    op_tensor_test.go:449: "Reshape" not yet implemented for non-contiguous views
        gorgonia.org/tensor.(*Dense).Reshape
        	/home/markkremer/go/pkg/mod/gorgonia.org/tensor@v0.9.6/dense.go:146
        gorgonia.org/gorgonia.reshapeOp.UnsafeDo
        	..../Go/src/github.com/MarkKremer/gorgonia/op_tensor.go:1205
        gorgonia.org/gorgonia.(*execOp).exec
        	..../Go/src/github.com/MarkKremer/gorgonia/vm_tape_nocuda.go:52
        gorgonia.org/gorgonia.(*tapeMachine).runall
        	..../Go/src/github.com/MarkKremer/gorgonia/vm_tape.go:236
        runtime.goexit
        	/usr/local/go/src/runtime/asm_amd64.s:1357
        Failed to carry UnsafeDo()
        gorgonia.org/gorgonia.(*execOp).exec
        	..../Go/src/github.com/MarkKremer/gorgonia/vm_tape_nocuda.go:53
        gorgonia.org/gorgonia.(*tapeMachine).runall
        	..../Go/src/github.com/MarkKremer/gorgonia/vm_tape.go:236
        runtime.goexit
        	/usr/local/go/src/runtime/asm_amd64.s:1357
        PC 12. Failed to execute instruction Reshape(2, 3, 1)	[CPU3]	CPU3	false	true	false
        gorgonia.org/gorgonia.(*tapeMachine).runall
        	..../Go/src/github.com/MarkKremer/gorgonia/vm_tape.go:237
        runtime.goexit
        	/usr/local/go/src/runtime/asm_amd64.s:1357
        PC: 12
        gorgonia.org/gorgonia.(*tapeMachine).RunAll
        	..../Go/src/github.com/MarkKremer/gorgonia/vm_tape.go:220
        gorgonia.org/gorgonia.TestUnconcatConcatOpSequence
        	..../Go/src/github.com/MarkKremer/gorgonia/op_tensor_test.go:448
        testing.tRunner
        	/usr/local/go/src/testing/testing.go:909
        runtime.goexit
        	/usr/local/go/src/runtime/asm_amd64.s:1357
FAIL

What I noticed is:

The concat op's SymDiff method doesn't always return a gradient with the same shape as the input nodes. This will cause problems when using that gradient in other ops.
The compiler (compile.go -> codegenerator.addNode(...)) determines that the reshape can be done using the UnsafeDo method. But Unconcat returns a view which isn't supported by that method. I'm not entirely sure if we could just copy the if v.IsView() { val = v.Materialize() } code from the Do() method. Or should the compiler have a check instead that uses the safe Do() method if one or more of the inputs are views?

This is slightly related to #404 in that I need it for the same model and that something with the shapes is going wrong. But now that I have found the issues it doesn't have much overlap that is of interest for Gorgonia.

I think this is fixed in #391

.... which I just noticed is not merged.

That would be great. I'll check if my tests work with that.

It fixes one of my test and I'm very happy that that part works now. TestConcatOp does still fail. And I keep getting this error in my own project:
ReduceAdd failed during differentiation: Operation failed: Failed to infer shape. Op: + false: Shape mismatch: (30, 128, 1) and (30, 128)

After TestConcatOp is fixed this one might be resolved too. But I'll try to create another test which actually produces the error above. Extra info: ReduceAdd is called in differentiation.go.
What needs to happen before #391 can get merged?

I added a test that gives the error I mentioned. ReduceAdd used here doesn't support adding tensors of different shapes. Something else should be used or the grads have to be broadcasted at some point before reaching this code. But this is only necessary if there are more that 2 gradients to be applied, and those gradients have incompatible shapes.
My TestConcatOp checks the gradient shapes but now that I think of it, I'm not certain it will cause any actual problems if the other problems get fixed. It depends on if the other pathways that apply the gradients can deal with slightly different shapes (e.g. (3, 3, 1) vs (3, 3)). I don't know if that's the case.

I just merged #391 . Should I tag this version?

I don't care about it personally but it may be nice for the issue about "Convnet isn't learning".

Small update: I'll try to get my model working completely before finishing up this PR so I know it works as intended.


A BUG: can not find "cxxabi.h"

There are maybe a mistake in file ./toolchains/android.toolchain.cmake at line 1039.
set( ANDROID_ABI_INCLUDE_DIRS "${ANDROID_CXX_ROOT}/llvm-libc++abi/include" )
and I find the correct dir of ANDROID_ABI_INCLUDE_DIRS is  "${ANDROID_CXX_ROOT}/llvm-libc++abi/libcxxabi/include"
so the line 1039 must be set( ANDROID_ABI_INCLUDE_DIRS "${ANDROID_CXX_ROOT}/llvm-libc++abi/libcxxabi/include" )

Good eye. Did you encounter an error related to this while building the library in a particular configuration?

I don't think it happens in a particular configuration. In fact, the file cxxabi.h in the path ${ANDROID_CXX_ROOT}/llvm-libc++abi/libcxxabi/include correctly.

So it sounds like this isn't actually causing a problem with the build process. Is that correct?
The android.toolchain.cmake file was written by another open-source project, not by us, so I don't want to modify it unless we are very confident that it is causing a problem.

It's up to you.
Can you tell me the android.toolchain.cmake file was written by which open-source project?

Hi @Oliver001 , I remember adding the support for c++ shared a while back https://github.com/tangrams/tangram-es/commit/67f04114b818f6bee392727d00a68d17e26b74c9 (and the cmake file was originally taken from the project you mentioned).
Unfortunately, this part might not work for all the android NDKs since it's highly adapted for our needs, we will be more specific in our README and specify that we currently develop under android-r10e NDK (https://github.com/tangrams/tangram-es#android).
We are also waiting for the new android studio which has CMake support before investing time in adapting this toolchain for all configurations (Refer https://github.com/tangrams/tangram-es/issues/807).

We got the toolchain file from https://github.com/taka-no-me/android-cmake but then had to make some additions and corrections, and actually that line you pointed out was one of the things we added! After realizing that and testing the include path, I went ahead and made the change you suggested: https://github.com/tangrams/tangram-es/commit/b979ac754cd83bb8566dbfc6e1e09a6d506ff28b

Hi @karimnaaji,  the new stable android studio which has CMake support maybe release after two month or later.
Thanks.


Input nulls cause queries to not execute

Steps To Reproduce
Navigate to https://mdsinabox.com/nba/teams/
view the query filtered_summary_by_team
Inspect the SQL

Expected Behavior
The user has not selected a button yet, so the input result is null, and returns the expected query results for if '${inputs.my_input}' was null
Actual Behaviour

The query does not return data and so remains in a loading state
Workarounds
Create an {#if} block before the results are available, returning different data. See the source code

Duplicate of #1479

oops missed your issue


Return a 401 error when auth info is not provided in endpoint requiring default auth

I created a fresh repo here that enables demonstration of the issue: https://github.com/gabrielliwerant/meteor-restivus-bug.
When attempting to use authentication with a GET route, we are presented with the following stack trace:
TypeError: Not a string or buffer at Hash.update (crypto.js:240:17) at Object.Accounts._hashLoginToken (packages/accounts-base/accounts_server.js:577:1) at [object Object].Restivus.config.auth.user (packages/nimble:restivus/lib/restivus.coffee:15:27) at Route.Route._authenticate (packages/nimble:restivus/lib/route.coffee:159:34) at Route.Route._authAccepted (packages/nimble:restivus/lib/route.coffee:146:8) at Route.Route._callEndpoint (packages/nimble:restivus/lib/route.coffee:124:9) at [object Object].Router.route.action (packages/nimble:restivus/lib/route.coffee:46:31) at boundNext (packages/iron:middleware-stack/lib/middleware_stack.js:251:1) at runWithEnvironment (packages/meteor/dynamics_nodejs.js:108:1) at packages/meteor/dynamics_nodejs.js:121:1

Thanks for providing a test repo! That's always greatly appreciated.
You need to login and then pass a user ID and auth token to any authenticated routes (unless you provide custom auth). The repro steps you provided in your test repo cause an expected error (although I admit there's room for some much clearer error handling there). You are declaring the route as requiring authentication, but are not providing the user ID and auth token in the headers. This is the intended behavior. Please read more on authentication in Restivus. I'm going to close this issue, but if for some reason I've misunderstood you and you still feel this is unresolved, please feel free to reopen it and continue the discussion. Thanks!

Actually, I'm going to reopen this until I return a 401 Unauthorized error from the default auth when no auth info is provided.

Thanks for the quickness of your response to this issue.
The problem may be a little deeper, or is perhaps a documentation issue. I seem unable to generate a userID and auth token (which I did attempt before I opened this issue).
According to the docs, I make a request to the login endpoint like so:
`curl http://localhost:3000/api/login/ -d "password=testpassword&user=test"
but I receive the following response:
{ "status": "error", "message": "Unauthorized"}
I also get the response above if I use a REST client or query params.
My understanding from the instructions ("The response will look like this, which you must save (for subsequent authenticated requests):") is that I should be able to make this request by submitting new credentials, and then get the generated authToken and userId, which I subsequently pass in the header to authenticate my requests. However, I receive the above error instead. So I am either not following the instructions correctly, or there is another issue.
By the way, I appreciate the work you've been doing on this meteor package. Other than the above issue, I have found the API to be very clean and easy to use.

Ah, I think I see the issue. I will certainly review the authentication section of the docs and try to provide some more details there and make things clearer. The docs are assuming that you have already created that user. You can do that by just adding a POST collection endpoint for your Meteor.users collection (or by any other means of creating a user with the accounts packages). Please check out the section of the docs on generating endpoints for the Meteor.users collection. At the very least, I should put that link at the beginning of the "Authentication" section of the docs, and make it clear that you have to create a Meteor.user separately. Sorry for the confusion.

Ah, yes, that makes sense now that you point that out. I apologize for my confusion, but I agree it's an opportunity to make the docs more idiot-proof. Thank you.

No problem! I agree. I'll definitely update that section of the docs. Thanks for reporting the issue!

Resolved in 5bf092bc2211bb105c511d74431f4e1484b897ba

Thanks for the good work, however i have the same issue as gabrielliwerant reported
{ "status": "error", "message": "Unauthorized"}
, so perhaps you could clarify:

when you write "Works alongside the alanning:roles package" this implies that restivus does not include this packages and this should be added to the project ( as well as meteor add accounts-password) right?
There is no "user" field by default when using the Accounts.createUser() function, should it be added so that
curl http://localhost:3000/api/login/ -d "password=testpassword&user=test" would work ? If i try to use the email and password to authenticate i get :
curl http://localhost:3000/api/login/ -d "password=<EMAIL_ADDRESS>Cannot call method 'indexOf' of undefined
at [object Object].addRoute.post [as action] (packages/nimble:restivus/lib/restivus.coffee:276:29)
at Route.Route._callEndpoint (packages/nimble:restivus/lib/route.coffee:126:25)
at [object Object].Router.route.action (packages/nimble:restivus/lib/route.coffee:46:31)
at boundNext (packages/iron:middleware-stack/lib/middleware_stack.js:251:1)
at runWithEnvironment (packages/meteor/dynamics_nodejs.js:108:1)

I can see also that you "Return a 401 error when auth info is not provided in endpoint"  in 0.8.0 branch, would it be soon available through the standard update procedure meteor update nimble:restivus ?
Thanks

Hi @thierrysikora. To answer your questions:

Yes, that's correct
No, you don't have to add any field to the Meteor.user document. Just put the email in the user field of the request, and Restivus will (very naively) determine if it is an email and validate against the proper field in the Meteor.user document. It won't recognize the email field that you provided, hence the error.
YES! Restivus v0.8.0 is now available!

Hello! It's not worth re-opening this issue for this, but I am just commenting to say that I had a similar problem as above, but what I was doing wrong was using the user ID rather than the email address of the user I had created. It was not obvious from the docs that it would guess from the email which user I intended, rather than using the ID of the user object. I am new to Meteor though, so perhaps this is obvious to people usually. Otherwise, perhaps a recommendation for future docs updates would be that the "user" request parameter can be the email address. Thanks for the great repo by the way!

@markmacgillivray I completely agree. This ambiguous field name was a carry-over from the original RestStop2 API that Restivus was inspired by. I just opened #121 to resolve this. Thanks for reporting this, and an even bigger thanks for the compliment!


null safety applied⚡

_ No description provided. _

I want to use your project how can I access to your repo?


[SR-6347] Added "SEE ALSO" to --help for Swift commands

Nice, perfect!

@swift-ci please smoke test

@aciidb0mb3r I'd be great if SwiftTool itself was a protocol, so we could set the toolName as one of its requirements.

Congrats @MrNickBarker for this PR!

I guess, we could have added this property in SwiftTool also (and fatalError if subclasses didn't override).

Please let me know If you want me to make this a property of SwiftTool. Not a problem 👍


mime package is getting esm only

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the issue has not already been raised

Issue
We depend on mime. Some people would like to see the world break. Maybe we should avoid that...

We can stay on the old version for as long as we want.

Mime 3.0 is broken right now, package managers are complaining about it, should update to 4.0

What are they complaining about?
All is fine on my end

With TS:

You have to override @types/mime to version 3.0.4 to stop this error

I guess it sounds a good idea for a fork and embed the types 🫠😅

I can do this, but after Wednesday
So someone else who is willing can do it before

I don't understand the problem here.
We do not use @types/mime inside this codebase or not even reference the @types/mime.
@thelinuxlich
Where is the problem of @types/mime linked to us?

You don't but TS checks for it when interacting with the mime sub dependency

Then this is the author of using this library should install the proper types.
Not actually our problem, isn't it?

Here is the minimal check to see if missing @types/mime causing problem.
No complain of the problem you shown, and no need to install @types/mime.
index.ts
import { send } from '@fastify/send'
import { createServer } from 'node:http'

createServer(function onRequest(req, res) {
  send(req, './index.ts')
    .pipe(res)
}).listen(3000)

package.json
{
  "name": "repo",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "dev": "ts-node index.ts",
    "build": "tsc index.ts"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "@fastify/send": "^2.1.0"
  },
  "devDependencies": {
    "@types/node": "^20.12.2",
    "ts-node": "^10.9.2",
    "typescript": "^5.4.3"
  }
}

tsconfig.json
{
  "compilerOptions": {
    "strict": true
  }
}

Another one: https://github.com/firebase/firebase-admin-node/issues/2512


Remove gcloud auth activate-service-account from Kokoro Mac build

Fixes #12056

Only affects kokoro tests, merging

Let's doublecheck that the results are still getting uploaded to bigquery and flaky tests are fetched correctly after merging this.


template_matching for ios

The matrix on the README currently states that the template_matching using KNIFT is available on Android only. Is this because there is no example for it on iOS is yet or because it is not possible on iOS? In any case is there a plan to support iOS?

It is possible on iOS. But we don't have an example for iOS.

Thanks for the answer. Good to hear that it would work on iOS as well. Hope to see an example soon as well.

Thanks for the answer. Good to hear that it would work on iOS as well. Hope to see an example soon.

HI, @mpost. The matrix you mentioned above is the pose matrix of object? I can't figure out how to get the pose matrix of object from the solution KNIFT. Can you give me some idea if possible~ Thanks in advance.


E2E Integration

Test-NetStack has been re-enabled in the lab as of 1/11/22 (with a timeout in WTT to work around the hanging issues we were seeing on stages 2 & 4). We are still seeing some failures in stages 2-6, but I will make separate issues for those.


save_plane keeps figures open (memory issue)

I noticed something when I use the save_quality_control in a loop of many images. After a few exports I get a warning message from matplotlib stating that I have more than 20 figures open currently. In the loop I exported about 100 images which in the end ate up about 3 GB of memory.
After a look at it I think it can easily be resolved by closing the figure after saving it, as mentioned in this stackoverflow thread.
In the code, I think it would refer to line 1001 in msct_image.py, function save_plan
        try:
            fig = plt.figure(figsize=(n_lines*10, n_col*20))
            for i, slices in enumerate(slice_list):
                slice_im, slice_seg = slices
                plot = fig.add_subplot(n_lines, n_col, i+1)
                plot.imshow(slice_im, cmap=cm.gray, interpolation='nearest')
                if index[i] is None:
                    title = 'mid slice'
                else:
                    title = 'slice '+str(index[i])
                plot.set_title(title)
                if seg is not None:
                    slice_seg[slice_seg < thr] = 0
                    plot.imshow(slice_seg, cmap=cmap_seg, interpolation='nearest')
                plt.axis('off')
            # plt.imshow(slice, cmap=cm.gray, interpolation='nearest')
            # if seg is not None:
            #     plt.imshow(slice_seg, cmap=cmap_seg, interpolation='nearest')
            # plt.axis('off')
            fname_png = path_output + self.file_name + suffix + format
            plt.savefig(fname_png, bbox_inches='tight')

I think a plt.close(fig) after the plt.savefig() would solve this.

Yes it makes sense !

Does this fix the issue ?

Should fix it. I'll try! Thanks Sara!

is it fixed? can we close this issue?

fixed


Sample MyCore implementation

Could you please provide a sample MyCore implementation? Reason I'm asking is that I've done all other configuration/setup (as explained in the readme) and I'm able to see started plug <IP_ADDRESS>:5060/udp logged message when I start the application, however none of the log messages from my custom core implementation appear when I send a sample INVITE request.
For the reference - my custom core implementation:
defmodule ZCore do
  require Logger
  alias Sippet.Transactions
  alias Sippet.Message

  @behaviour Sippet.Core

  require Logger

  @impl Sippet.Core
  def receive_request(req, serverTxKey) do
    Logger.debug(fn -> "#{inspect self()} receive_request:: req => #{inspect req}; serverTxKey => #{inspect serverTxKey}" end)
    req |> Message.to_response(100) |> Transactions.send_response(serverTxKey)
  end

  @impl Sippet.Core
  def receive_response(resp, clientTxKey) do
    Logger.debug(fn -> "#{inspect self()} receive_response:: resp => #{inspect resp}; clientTxKey => #{inspect clientTxKey}" end)
  end

  @impl Sippet.Core
  def receive_error(reason, txKey) do
    Logger.debug(fn -> "#{inspect self()} receive_error:: reason => #{inspect reason}; txKey => #{inspect txKey}" end)
  end

end

As you might have guessed - an Elixir beginner here so any other relevant comments/suggestions are welcome.

Hi @IvanRibakov .
In principle I don't see any problem with your implementation. By the way, I would check your configurations in config/config.exs. In special, double check if you connected your Core to the solution. You should have the following on your configuration file:
# Configures the sippet core
config :sippet, Sippet.Core, ZCore

As an alternative, you can check https://github.com/balena/elixir-sippet-proxy . It is another library that uses elixir-sippet to build a SIP proxy. It contains a Sippet.Core implementation and that could be a good start point for your implementation.

Closing this issue due to inactivity. Please open a new issue if the problem persists.


Add gapic config adviser

Codecov Report

Merging #1513 into master will increase coverage by 0.08%.
The diff coverage is 90.34%.

@@             Coverage Diff             @@
##             master   #1513      +/-   ##
===========================================
+ Coverage     86.91%     87%   +0.08%
- Complexity     4112    4237     +125
===========================================
  Files           404     414      +10
  Lines         16574   17009     +435
  Branches       1762    1862     +100
===========================================
+ Hits          14406   14799     +393
- Misses         1683    1701      +18
- Partials        485     509      +24

Impacted Files
Coverage Δ
Complexity Δ

...gen/configgen/transformer/LanguageTransformer.java
88.73% <ø> (ø)
4 <0> (ø)
:arrow_down:

...degen/configgen/transformer/ConfigTransformer.java
96.29% <ø> (ø)
12 <0> (ø)
:arrow_down:

...gle/api/codegen/advising/LanguageSettingsRule.java
100% <100%> (ø)
7 <7> (?)

...google/api/codegen/advising/LicenseHeaderRule.java
100% <100%> (ø)
6 <6> (?)

.../java/com/google/api/codegen/CodeGeneratorApi.java
21.27% <57.14%> (+2.88%)
3 <0> (ø)
:arrow_down:

...google/api/codegen/advising/PageStreamingRule.java
73.07% <73.07%> (ø)
13 <13> (?)

...ava/com/google/api/codegen/advising/FieldRule.java
82.35% <82.35%> (ø)
10 <10> (?)

.../java/com/google/api/codegen/advising/Adviser.java
88.88% <88.88%> (ø)
6 <6> (?)

...om/google/api/codegen/advising/CollectionRule.java
89.18% <89.18%> (ø)
15 <15> (?)

...ava/com/google/api/codegen/advising/RetryRule.java
89.23% <89.23%> (ø)
19 <19> (?)

... and 13 more

Continue to review full report at Codecov.

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

Actually, this PR is uncomfortably large. I'm going to split it into 8 parts.


[Feature suggestion] turn this into a github action

It would be cool to have a github action that each time you commit scans for latex in md files and automatically substitutes them with images

I use the texify app for this: https://github.com/apps/texify

texify was working well in general. It's difficult to debug, though. Sometimes I push a change and never see a followup commit from texify. I'm not sure if it's an error on my end or if texify is "down". A github action that I could control and debug the output of, would be much more maintainable. Does anyone know how to do it?

Looks like someone did that a few days ago. See readme2tex-action and its source code.

Check my repo https://github.com/tuyenld/readme2tex-action

Add action.yml file to your repo at .github/workflows/action.yml.
Change branch name main if it is necessary.


Spec out the MVP!

So there's a few bits and pieces in the repo about what it is you're working on, but it would be nice if you could flesh out the MVP in an issue. I humbly suggest creating a new issue for discussion, and keep editing the first post so that there's always an up-to-date summary at the top of the thread.
Kind of like https://github.com/xobust/Project-Broban/issues/1 :)

See #15 hopefully I understand what MVP stands for, ehm Mac what does it really stand for? xD

@EE87 Minimum Viable Product, as referenced in Mac's inda guidelines.


Error running tutorial FinRL_MultiCrypto_Trading.ipynb

Describe the bug
I get the following error when I am trying to run the tutorial:
AttributeError: 'AgentPPO' object has no attribute 'traj_list'.
To Reproduce
Steps to reproduce the behavior:

Open https://github.com/AI4Finance-Foundation/FinRL/blob/master/tutorials/3-Practical/FinRL_MultiCrypto_Trading.ipynb in Google Colab
Run all the cells until the "Training" cell.
Run the "Training" cell
See error

same error here

Updated ElegantRL to address this issue. Hope it is fixed.

redownload ElegantRL already. Still the same error.


Add workaround to .Net install issue June 2023

Works around an issue introduced in June's updates that prevent the installation of .Net Framework 3.5 or 2.0 after running security patches.

/assign bkatyl


BLOCKED UNTIL GA: Update README for GA

This commit updates the README docs as a result of the GA release of the CLI

@sigalsax I have not reviewed the SDK documentation as it is out of scope for me currently


Dev env - always patch even if up & cache yarn packages

What this PR does / why we need it:

Which issue(s) this PR fixes:

Does this PR require a test?

NONE
Does this PR require a release note?

NONE

Does this PR require documentation?

NONE

Merging as this only affects the dev env.


Is there any method to get a private key from a registar which has been enrolled?

Description

Now  I  have a register client to register a new user with membersrvc API .I need to get the private key of a enrolled registar.Is there any gRPC api that I can use?

After I enrolled a registar and register a new user.I don't konw how to use the same registar to register another user.


⚠ Samsung has degraded performance

In 4eb0b77, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2794 ms

Resolved: Samsung performance has improved in 2f92683.


No Pop-Up Menu for Revision Module View

When right-clicking the Revision Module View, a pop-up menu with the following options should appear:
1. Add revision.
2. Delete selected revision.
3. Calculate selected revision.
4. Save selected revision.
5. Save all revisions.

Added pop-up menu to Revision Module View in commit 71d6507b17120f635d93ecf789e6ebbed151ada2
Did not implement the calculate function as that is associated with the Work View in the new design scheme.


Where is the compute part or NXT

Compute are important in nowadays

There are two compute examples in the examples directory, including this one:
https://github.com/google/nxt-standalone/blob/master/examples/ComputeBoids.cpp


acs-engine / kubernetes upgrade "getsockopt: connection timed out"

We are currently using acs-engine 0.10.0 to deploy kubernetes 1.8.4.
Under this setup the service.beta.kubernetes.io/azure-load-balancer-internal: "true" does not appear to work correctly.  When I upgraded a cluster on AKS from kubernetes 1.8.10 to 1.9.6, that annotation does work.
Therefore, I am now upgrading our acs-engine setup.  Using acs-engine v0.18.3 and changing to "orchestratorVersion": "1.9.6", the cluster is created as expected.
However, I'm unable to get azure-vote working in this new cluster as the frontend cannot seem to connect to the redis backend despite a normal azure-vote-back service configuration.  Also, I cannot connect to the kubernetes web ui, which fails with Error: 'dial tcp <IP_ADDRESS>:8443: getsockopt: connection timed out' Trying to reach: 'https://<IP_ADDRESS>:8443/'.
Any pointers on how to debug this further or tips for upgrading from 1.8.x to 1.9.x?

What exactly "does not appear to work correctly" ? Do you have any error messages or log files?

FWIW I am able to create a working internal LB with Kubernetes version 1.8.4.

How did you test it?  From within the kubernetes cluster or from the vnet/subnet?  In my observations, attempting to curl a service exposed to the vnet via an internal lb fails.
Here are general steps to reproduce:

Create a typical aks cluster az aks create --kubernetes-version 1.8.10.
Install azure-vote application as normal.  Port-forward azure-vote-front and curl from localhost to confirm it's working.
Change the service type to LoadBalancer, add the annotation service.beta.kubernetes.io/azure-load-balancer-internal: "true".  External IP from kubectl get svc -o wide is <IP_ADDRESS>.
Curl again via kubectl port-forward, still working.  So far, so good.
Add a "regular" ubuntu vm to the same vnet as the aks cluster (using portal.azure.com).
SSH to the ubuntu machine and try to curl -v <IP_ADDRESS> the service from (3) using the subnet private IP.  Curl will hang, does not work.
Upgrade kubernetes to 1.9.6
SSH back into the ubuntu machine, curl -v <IP_ADDRESS> again, now works as expected.

How exactly did you perform step 3 ?
I didn't change the service type but tested with a new one.

Step 3 was to say, "Deploy azure-vote in standard fashion (without internal lb), santiy check it works, then update the service to use internal lb".
If you just skipped over the "standard fashion" part and went directly to internal lb, that's fine.  So you're saying you could SSH and curl the internal IP?


rename_with instead of rename

The pivot example cell with rename fails. To pass the rename function down, it should be rename_with I guess.
My humble support for this completely awesome library, and the livebook example just makes it crystal clear.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


[nextcloud] Cache Error

i get these Messages
Info	cli	Memcache \OC\Memcache\APCu not available for distributed cache
Info	cli	Memcache \OC\Memcache\APCu not available for local cache

thats in my config.php
  'memcache.local' => '\OC\Memcache\APCu',
  'memcache.locking' => '\OC\Memcache\Redis',
  'redis' =>
  array (
    'host' => '/tmp/redis.sock',
    'port' => 0,
    'timeout' => 0.0,
  ),

Try adding this line:
'memcache.distributed' => '\OC\Memcache\APCu',

What's the output of php7 -m ?

I have the same issue since yesterday. (daily build)
Updating permissions...
Updating permissions in /nextcloud...
Permissions in /data are correct.
Permissions in /config are correct.
Permissions in /apps2 are correct.
Updating permissions in /etc/nginx...
Updating permissions in /etc/php7.1...
Updating permissions in /var/log...
Updating permissions in /var/lib/nginx...
Updating permissions in /tmp...
Updating permissions in /etc/s6.d...
Done updating permissions.
PHP Warning: PHP Startup: Unable to load dynamic library '/usr/lib/php7.1/modules/apcu.so' - Error loading shared library /usr/lib/php7.1/modules/apcu.so: No such file or directory in Unknown on line 0
Nextcloud is already latest version
PHP Warning: PHP Startup: Unable to load dynamic library '/usr/lib/php7.1/modules/apcu.so' - Error loading shared library /usr/lib/php7.1/modules/apcu.so: No such file or directory in Unknown on line 0
[25-Feb-2017 13:38:51] NOTICE: PHP message: PHP Warning: PHP Startup: Unable to load dynamic library '/usr/lib/php7.1/modules/apcu.so' - Error loading shared library /usr/lib/php7.1/modules/apcu.so: No such file or directory in Unknown on line 0

This will be fixed in the next build. https://github.com/Wonderfall/dockerfiles/commit/814c4bb67932efe87adedd4e9aa964b814e151fa
Somehow pcre-dev now needs to be explicitly installed otherwise the apcu module doesn't build.
However I don't think this is related to the OP.


Everything is going great

Things are so easy, the other problems in my life are now more apparent.

Few issues


How to run?

Hello!
I am new to python as in, I know nothing. I was able to follow your instructions all the way through and successfully installed everything as instructed.
My question is what now? How do you use this tool in Photoshop? Thanks in advance!

Thank you so kindly for responding! That makes much more sense. So I’ve followed their steps completely and terminal returned no errors. So hopefully I can proceed with the next step, whatever it may be.


Unit tests

Add unit tests, change default target to run test-short, skip integration tests when run with test-short.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ea710fd). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   78.94%
=========================================
  Files             ?        1
  Lines             ?       57
  Branches          ?        0
=========================================
  Hits              ?       45
  Misses            ?        7
  Partials          ?        5

Continue to review full report at Codecov.

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


Fixed table markup

Hi Alex and Jan,
this is just the smallest contribution ever, but upon reading the documentation I saw this list that wanted to be a table, and quickly fixed it.
Kind regards
Florian

Thank you!! 🎉


ott: 0.31 -> 0.32

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/ott-lang/ott/releases.
meta.description for ott is: "A tool for the working semanticist"
meta.homepage for ott is: "http://www.cl.cam.ac.uk/~pes20/ott"
meta.changelog for ott is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/pc2ync7yd0nnrqksxsx5i73zvs51vkjy-ott-0.32/bin/ott had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.32 with grep in /nix/store/pc2ync7yd0nnrqksxsx5i73zvs51vkjy-ott-0.32
found 0.32 in filename of file in /nix/store/pc2ync7yd0nnrqksxsx5i73zvs51vkjy-ott-0.32

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
ott

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pc2ync7yd0nnrqksxsx5i73zvs51vkjy-ott-0.32 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ott https://github.com/r-ryantm/nixpkgs/archive/b3cadfd9e6bf298574f8334f335289e4d0693232.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pc2ync7yd0nnrqksxsx5i73zvs51vkjy-ott-0.32
ls -la /nix/store/pc2ync7yd0nnrqksxsx5i73zvs51vkjy-ott-0.32/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

ott

Maintainer pings
cc @jwiegley for testing.

Result of nixpkgs-review pr 163885 run on x86_64-linux 1

1 package built:

ott


Allow passing file-contents via stdin

I know this question has been asked before (#465) but my use-case is different. I am from the AtomLinter team and we're using this project in our linter-scss-lint package. We support both lintOnFly (lint as you type) and lint on save. To support lintOnFly for that provider, we'll have to get the current editor text and save it in a temp file and give that temp file to scss-lint to lint and then return those results, which means thousands of temp file would be created every few hours (AtomLinter/linter-scss-lint#72).
All of those temp files can be avoided if scss-lint supports passing file contents as stdin, if you would do so, You won't be the only one doing it, currently eslint, coffeelint, jshint and a lot of other linters provide similar options. In case you want to search your rcfile there or something, You could also add a parameter to specify path for the stdin file.

Hey @steelbrain, thanks for your work on integrating scss-lint with the Atom editor.
Passing files via STDIN introduces a couple of edge cases, which I'm skeptical saving even thousands of temporary files every few hours is worth addressing. (that's not a lot of writes in my mind)
For example, one important question is what file path is the file passed via STDIN treated as having? scss-lint uses the file path in order to apply the correct configuration based on your .scss-lint.yml. I could easily see confusion from users piping a particular file to scss-lint and being surprised that linters they've disabled for that file in their configuration start reporting warnings.
This is a problem in general when you lint against a file with a different path than the original file you're linting—you're bound to run into situations where configuration you expect to be applied doesn't apply because of differences in the path.
I'm open to suggestions, but I think the pragmatic answer here is to stick with linting the file directly.

@sds I had specified the scenario about the file paths, jshint has a --filename argument, You can write to stdin and specify the path of the file there. It'll resolve the configuration files and all from there on automatically. so Yes, if you use that file path, You can read the .scss-lint.yml file automatically so no more linter re-enabling when they are disabled from the configuration file.

Ah, sorry, I missed your last sentence. That makes sense. Happy to accept a pull request!

@sds I would love to send a PR but I don't know enough ruby to do that :(

I also work on the scss-lint plugin for atom and know ruby reasonably well.  Maybe I can take a crack at this :smile_cat:

Just for the record, Flycheck (a linting extension for Emacs) would profit from this feature as well :relaxed:

Implemented in 8ad337a1a9d38f890b56ac355e44aadd36b7aa80. Thanks for the feedback!

@sds Thank you.  That is great news!  😍
Mind if I ask whether there is an ETA for a release that includes this change?

:clap: @sds you are awesome!

@lunaryorn 0.43.0 has just been released, but I would probably give it some time to settle given the number of new features going out. There tend to be a lot of bug reports, especially with new features whose behavior isn't fully comprehended until they're out in the wild.

@sds I hate waiting for the cool things 😊
How long do you think will it take for the dust to settle?  Should we wait until the first bug fix release of 0.43?  Or should we wait for the next minor release?

@lunaryorn I anticipate we'll push out a bugfix release by the end of this week.


2023.02 review

Coverage: 92.924% (+0.0008%) from 92.923% when pulling 9244bb9581ac22f98b428593aba6c60e2982b534 on antgonza:2023.02-review into e1ff4dcc7512fd1e7a2969bb9bbbd110d72076e0 on qiita-spots:dev.

Looks good to me!


Change Target Framework from .net core to .net standard

As I understand your package mostly aimed for enterprise app. It's good point to keep client libs based on .net standard framework instead of .net core framework

Thx! 😊 , We will look into this!


Add time type

Adds support for time.Time values, default in RFC3339, can be set.

10/10 PR. Thank you very much!


Operation set by index

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Additional documentation:

Checklist:

[ ] Added relevant tests or not required
[ ] Didn't break anything

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

What is intended behavior for invalid index?
I added negative index validation for json array methods in https://github.com/yorkie-team/yorkie/pull/985/commits/9de45cfd4d3d1ba79acc563b64df978819a9084f. But it still seems like there could be a potential 'nil pointer dereference' error.
In json array, GetObject, GetArray, ... , Delete method validate index itself and call *crdt.Array.Get directly. This is ok.
But, InsertIntegerAfter(and MoveAfterByIndex, SetInteger) method call *json.Array.Get which validates index and returns nil or *crdt.Array.Get(idx).
It can be executed nil.CreatedAt() if *json.Array.Get returns 'nil' in following statements.
func (p *Array) InsertIntegerAfter(index int, v int) *Array {
	p.insertAfterInternal(p.Get(index).CreatedAt(), func(ticket *time.Ticket) crdt.Element {
		primitive, err := crdt.NewPrimitive(v, ticket)
		if err != nil {
			panic(err)
		}
		return primitive
	})

	return p
}

I initially created the hand-written simple test cases in func TestArrayConcurrency. To ensure all cases are covered, I also added tests for all operation pairs in func TestArrayConcurrencyTable.
The former can be reproduced in the latter. Would it be better to delete the former tests? And would it be better to create a new test file for array concurrency, similar to tree_concurrency_test.go?

If both tests are testing the same case, it may be good to remove first one.


Integration tests workflow fails

Tests need to be fixed

[x] private name/fqdn gateways test :arrow_right:
"Bad Gateway" does not contain "Directory listing for"
dial tcp [300:170b:469a:2ed8:6f7e:59d0:e50:e41d]:22: connect: connection timed out
[x] name gateways test :arrow_right:
couldn't schedule request gateway: couldn't find a node satisfying the given requirements
dial tcp [300:164c:759a:efe6:1655:fb4b:6652:377e]:22: connect: connection timed out
[x] fqdn gateways test :arrow_right: could not start ssh connection
[x] mattermost/peetube/presearch/taiga test :arrow_right:
couldn't schedule request gateway: couldn't find a node satisfying the given requirements
could not deploy network networkk: error sending deployment to node 2: global workload with the same name 'networkk' exists: conflict
[x] qsfs and zdb tests
couldn't schedule request gateway: couldn't find a node satisfying the given requirements
trace error
Error: couldn't deploy deployment with error: error waiting deployment: workload meta1 within deployment 531958 failed with error: failed to create zdb namespace: failed to connect to 0-db: a5b6bd02-***: dial unix /var/run/zdb_a5b6bd02-***/zdb.sock: connect: connection refused
[x] nomad test
couldn't schedule request gateway: couldn't find a node satisfying the given requirements
order issue
[x] k8s test :arrow_right: workers should be found
[x] k8s invalid names test :arrow_right: couldn't deploy k8s cluster with error: k8s workers and master must have unique names: w0 occurred more than once
[x] vm test cases

tests timeout

add workflow to test example using the latest version

Testing

tried to deploy 10 vms (2 with public ips, 8 without)

File
terraform {
  required_providers {
    grid = {
      source = "threefoldtech/grid"
    }
  }
}

provider "grid" {
}

locals {
  name = "multinodedeployment"
}

resource "grid_scheduler" "sched" {
  requests {
    name             = "node1"
    cru              = 2
    sru              = 1024 * 2
    mru              = 1024 * 2
    public_ips_count = 2
    public_config    = true
  }
  requests {
    name = "node2"
    cru  = 2
    sru  = 1024 * 10
    mru  = 1024 * 10
  }
}

resource "grid_network" "net1" {
  nodes       = [grid_scheduler.sched.nodes["node1"], grid_scheduler.sched.nodes["node2"]]
  ip_range    = "<IP_ADDRESS>/16"
  name        = local.name
  description = "new network"
}

resource "grid_deployment" "d1" {
  name         = local.name
  node         = grid_scheduler.sched.nodes["node1"]
  network_name = grid_network.net1.name
  vms {
    name       = "vm1"
    flist      = "https://hub.grid.tf/tf-official-apps/base:latest.flist"
    cpu        = 2
    memory     = 1024
    publicip   = true
    entrypoint = "/sbin/zinit init"
    env_vars = {
      SSH_KEY = file("~/.ssh/id_rsa.pub")
    }
  }
  vms {
    name       = "vm2"
    flist      = "https://hub.grid.tf/tf-official-apps/base:latest.flist"
    cpu        = 2
    memory     = 1024
    publicip   = true
    entrypoint = "/sbin/zinit init"
    env_vars = {
      SSH_KEY = file("~/.ssh/id_rsa.pub")
    }
  }
}

resource "grid_deployment" "d2" {
  node         = grid_scheduler.sched.nodes["node2"]
  network_name = grid_network.net1.name
  vms {
    name       = "vm3"
    flist      = "https://hub.grid.tf/tf-official-apps/base:latest.flist"
    cpu        = 1
    memory     = 1024
    entrypoint = "/sbin/zinit init"
    env_vars = {
      SSH_KEY = file("~/.ssh/id_rsa.pub")
    }
  }
  vms {
    name       = "vm4"
    flist      = "https://hub.grid.tf/tf-official-apps/base:latest.flist"
    cpu        = 1
    memory     = 1024
    entrypoint = "/sbin/zinit init"
    env_vars = {
      SSH_KEY = file("~/.ssh/id_rsa.pub")
    }
  }
  vms {
    name       = "vm5"
    flist      = "https://hub.grid.tf/tf-official-apps/base:latest.flist"
    cpu        = 1
    memory     = 1024
    entrypoint = "/sbin/zinit init"
    env_vars = {
      SSH_KEY = file("~/.ssh/id_rsa.pub")
    }
  }
  vms {
    name       = "vm6"
    flist      = "https://hub.grid.tf/tf-official-apps/base:latest.flist"
    cpu        = 1
    memory     = 1024
    entrypoint = "/sbin/zinit init"
    env_vars = {
      SSH_KEY = file("~/.ssh/id_rsa.pub")
    }
  }
}

resource "grid_deployment" "d3" {
  node         = grid_scheduler.sched.nodes["node2"]
  network_name = grid_network.net1.name
  vms {
    name       = "vm7"
    flist      = "https://hub.grid.tf/tf-official-apps/base:latest.flist"
    cpu        = 1
    memory     = 1024
    entrypoint = "/sbin/zinit init"
    env_vars = {
      SSH_KEY = file("~/.ssh/id_rsa.pub")
    }
  }
  vms {
    name       = "vm8"
    flist      = "https://hub.grid.tf/tf-official-apps/base:latest.flist"
    cpu        = 1
    memory     = 1024
    entrypoint = "/sbin/zinit init"
    env_vars = {
      SSH_KEY = file("~/.ssh/id_rsa.pub")
    }
  }
  vms {
    name       = "vm9"
    flist      = "https://hub.grid.tf/tf-official-apps/base:latest.flist"
    cpu        = 1
    memory     = 1024
    entrypoint = "/sbin/zinit init"
    env_vars = {
      SSH_KEY = file("~/.ssh/id_rsa.pub")
    }
  }
  vms {
    name       = "vm10"
    flist      = "https://hub.grid.tf/tf-official-apps/base:latest.flist"
    cpu        = 1
    memory     = 1024
    entrypoint = "/sbin/zinit init"
    env_vars = {
      SSH_KEY = file("~/.ssh/id_rsa.pub")
    }
  }
}

output "node1_zmachine1_ip" {
  value = grid_deployment.d1.vms[0].ip
}
output "node1_zmachine_computed_public_ip" {
  value = grid_deployment.d1.vms[0].computedip
}

output "node2_zmachine1_ip" {
  value = grid_deployment.d2.vms[0].ip
}
output "node2_zmachine_computed_public_ip" {
  value = grid_deployment.d2.vms[0].computedip
}

Result
Apply complete! Resources: 5 added, 0 changed, 0 destroyed.

Outputs:

node1_zmachine1_ip = "<IP_ADDRESS>"
node1_zmachine_computed_public_ip = "<IP_ADDRESS>/24"
node2_zmachine1_ip = "<IP_ADDRESS>"
node2_zmachine_computed_public_ip = ""


Cannot add Crouton Integration Extension to 43.0.2357.52 (Official Build) beta (64-bit)

I have been using Crouton In A Tab (criat) for a few months now.  I really love it.  Today, I booted up my Chromebook and couldn't get to my chroot.  It seems that the chroot boots in the background (as intended) but the extension is broken.

I attempted to remove and reinstall the extension, but I get the following error

I powerwashed the device, but still am unable to install the Extension.  The link refrence I am using to attempt to install the Extension is https://chrome.google.com/webstore/detail/crouton-integration/gcpneefbbnfalgjniomfjknbcgkbijom
My Chrome Version is 43.0.2357.52 (Official Build) beta (64-bit)

You might be out of disk space...


Add citext filter support

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#480  👈
#479
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @acco and the rest of your teammates on  Graphite


StateEditor.cs 40 line causing "ArgumentException: Getting control 1's position..."

Describe the bug
StateEditor.cs 40 line causing "ArgumentException: Getting control 1's position in a group with only 1 controls when doing repaint
Aborting
UnityEngine.GUILayoutGroup.GetNext () (at <82a3a4e0a0a94de5afddd345f6cf50d1>:0)
UnityEngine.GUILayoutUtility.BeginLayoutGroup (UnityEngine.GUIStyle style, UnityEngine.GUILayoutOption[] options, System.Type layoutType) (at <82a3a4e0a0a94de5afddd345f6cf50d1>:0)
UnityEditor.EditorGUILayout.BeginVertical (UnityEngine.GUIContent content, UnityEngine.GUIStyle style, UnityEngine.GUILayoutOption[] options) (at :0)
UnityEditor.EditorGUILayout.BeginVertical (UnityEngine.GUIStyle style, UnityEngine.GUILayoutOption[] options) (at :0)
StatefulUI.Editor.ReferenceInspectors.StateEditor.OnGUI (UnityEngine.Rect rect, UnityEditor.SerializedProperty property, UnityEngine.GUIContent label) (at Assets/Plugins/StatefulUI/Editor/ReferenceInspectors/StateEditor.cs:40)"
To Reproduce
Steps to reproduce the behavior:

Go to StatefulComponent inspector view
Open state editor
Scroll down to make top of the field is not visible
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Package Version 1.0.1

I "solved" this by removing the BeginVertical/EndVertical calls, and I can't seem to notice any difference in the inspector.

I "solved" this by removing the BeginVertical/EndVertical calls, and I can't seem to notice any difference in the inspector.

You're welcome to make a Pull Request 🙂

Thanks for the report! Please try v1.0.7


Installation - added sudo commands

Added sudo commands to install istioctl

This change is no longer needed, because the installation archive will contain istioctl binary. I removed those instructions completely, let's close this.


Credential manager

Hey, is there a way to store credentials via an electron API of some kind?
Im just asking before I look at doing my own.
The only thing I have found so far - https://github.com/atom/node-keytar

I think it would be best to write your own credential manager with a hash of your choice.

Yeah thanks man, ended up doing that. However we aren't using Electron.NET anymore, far from a production ready solution unfortunately.


fix(rpc): setup swarm connection on next tick to fix a bug with dht-relay

what about skipped tests and lock file change?

what about skipped tests and lock file change?

It is important to note that lock file doesn't get published on NPM, it is merely for development purposes. I often delete it and reinstall everything, just to make sure I am developing on latest versions of everything. Maybe we should remove it.
Skipped tests are mostly dealing with deprecated API (stuff that doesn't use core-data module) and are breaking now mostly because they were using Drivestore and that was using an earlier version of Corestore. I can update Drivestore to work with new versions of Corestore, but I was hoping to finish migration to core-data soon and skip support for Drivestore.

it is merely for development purposes.

It is important to note that lock file doesn't get published on NPM, it is merely for development purposes.

for publishing there is a shrinkwrap

I often delete it and reinstall everything, just to make sure I am developing on latest versions of everything.

Haven't you had a problem because of it recently?

Skipped tests are mostly dealing with deprecated API (stuff that doesn't use core-data module) and are breaking now mostly because they were using Drivestore and that was using an earlier version of Corestore. I can update Drivestore to work with new versions of Corestore, but I was hoping to finish migration to core-data soon and skip support for Drivestore.

Does it mean that they can be safely removed?

Does it mean that they can be safely removed?

No, any code that is still using Drivestore, but updates Corestore will get errors. Luckily for now Bitkit is the main user, and lockfiles are helping there.
After opening this issue (and delivering this fix manually in my local attempt to move core-data api to Bitkit) other issues appeared in Bitkit anyways, so there is a chance we will have to wait for bare.js, because bugs in corestore and hyperswarm seem endless in Bitkit right now, while in nodejs everything is much more robust.
Up to @rbndg, maybe we can close these two PRs and wait for bare.

To make that a bit more clear: If you want to break compatibility (maybe deliver a major version of Slashtags) then yes we can safely remove Drivestore and everything related to it, it just needs to be a major version.

Updated after merging #94, it doesn't solve https://github.com/synonymdev/slashtags-core-data/pull/19 so not sure if it is worth merging now. Maybe after solving that it wouldn't be needed.


git lfs pull 失败

报错如下
batch response: This repository is over its data quota. Purchase more data packs to restore access.
error: failed to fetch some objects from 'https://github.com/openatx/atxserver2-android-provider.git/info/lfs'

我也是。。

遇到同样的问题。。。

欠费，大文件无法下载


Crashed when network rebuilds(StackOverFlow)

Issue description:
I replaced a controller in a network without it and it crashed.
What happens:
It crashed. Stack overflow error. Detailed crash report is available below.
What you expected to happen:
The network could run normally.
Steps to reproduce:
1.Build a simple network.
2.Build another network in a different dimension(With controller).
3.Create a link between these two networks(Both of the networks should have network receiver and transmitter).
4.One of the controller disappears and then break the remaining controller.
5.Place the controller to the place where the previous controller was.
6.Then it crashed (QAQ
Version (make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: <IP_ADDRESS>1
Refined Storage: 1.2.25

Does this issue occur on a server?
Yes. Though the game is single player, the crash occurred on the server side.
If a (crash)log is relevant for this issue, link it here:
https://gist.github.com/yesterday17/501eb8e086c61469c5934dd73b6d4f89

It occurred again when i restarted the game and then entered the world.
Another crash report:
https://gist.github.com/yesterday17/a15ed646620c5da65f59ad9a13232c5b


fix(blog): falco graduation image metadata

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug
Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog
What this PR does / why we need it:
Fix the metadata link
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 77199bed72a28c84f0267898dba05d24d41c2076

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [LucaGuerra,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif, leogr, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [Issif,LucaGuerra,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Correction for raw affiliation Gembloux, 5030, Belgium; Microbial Processes and Interactions laboratory, UMRt BioEcoAgro 1158 INRAE, TERRA teaching and research centre, Gembloux Agro-Bio Tech, University of Liège

Correction needed for raw affiliation Gembloux, 5030, Belgium; Microbial Processes and Interactions laboratory, UMRt BioEcoAgro 1158 INRAE, TERRA teaching and research centre, Gembloux Agro-Bio Tech, University of Liège
raw_affiliation_name: Gembloux, 5030, Belgium; Microbial Processes and Interactions laboratory, UMRt BioEcoAgro 1158 INRAE, TERRA teaching and research centre, Gembloux Agro-Bio Tech, University of Liège
new_rors: 00bmzhb16;00afp2z80;032v67m28
previous_rors: 00bmzhb16;00afp2z80
works_examples: W4396217759
contact: 9abeaf74b69bd078137a2bcb67df168f:29d31012c3b2619dfe25b6f0278e7c68 @ u-picardie.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Feat/build

@rajpalc7, I've also rebased your commits to clear up some changes that shouldn't have been included in the PR.  You might need to reset your working copy to the latest commits on the feat/build branch before you continue development.  We can review that together when we look at the reusable workflow.

@rajpalc7, slight change in plan.  I had a quick chat with @cvarjao and we agreed we should clean up and merge this PR before making any enhancements.  I'll work on that and I'll create the tickets describing the further enhancements separately.

Sounds good Wade, I like the latest plan as this will let me focus on the current sprint tickets and other outstanding tasks.


Feedback: Week 4 Slick

Please provide your feedback by posting a comment against this issue.

I'm trying to follow along, but it can't seem to find my implicit versions.  I checked out the github repo, and get the same errors when trying sbt compile against your copy:
[error] /Users/Lewis/Downloads/BookCode/52-technologies-in-2016/04-slick/tasky/src/main/scala/datamodel/DataModel.scala:25: could not find implicit value for parameter tt: slick.ast.TypedType[java.time.LocalDateTime]
[error]     def createdAt = column[LocalDateTime]("createdAt")
[error]                                          ^
[error] /Users/Lewis/Downloads/BookCode/52-technologies-in-2016/04-slick/tasky/src/main/scala/datamodel/DataModel.scala:27: could not find implicit value for parameter tt: slick.ast.TypedType[java.time.LocalDateTime]
[error]     def dueBy = column[LocalDateTime]("dueBy")
[error]                                      ^
[error] /Users/Lewis/Downloads/BookCode/52-technologies-in-2016/04-slick/tasky/src/main/scala/datamodel/DataModel.scala:29: could not find implicit value for parameter tt: slick.ast.TypedType[Set[String]]
[error]     def tags = column[Set[String]]("tags")
[error]                                   ^
[error] three errors found
[error] (compile:compileIncremental) Compilation failed
[error] Total time: 3 s, completed 14-Feb-2016 15:01:43

Small issue.  The blogpost part doesn't describe the beforeEach method that you've implemented in the code example to recreate the database for each test.  As it is written, following along leads to having four entries in the database when you run the test to insert 3 items.  If you try to re-run it, then the database isn't created yet.
Seems easy enough to fix by looking at the code example with beforeEach/afterEach, but even with the afterEach doing a db.shutdown I am getting an error about tasks already existing when it tries to create the table that causes it not to run any test after the first...  Maybe it will make more sense when I read the next blogpost.

@llcawthorne thanks for the feedback. Below I have replied to your comments.

The code was not compiling because I had to explicitly set the column mapper in the column definition https://github.com/shekhargulati/52-technologies-in-2016/blob/master/04-slick/tasky/src/main/scala/datamodel/DataModel.scala#L25. I have pushed the changes so code should now compile fine and tests should pass.
I will add section on beforeEach. I think the reason db.shutdown was not working for you is that db.shutdown runs asynchronously so I have to wrap that in Await.result as well. It was working on my machine so I didn't noticed that behaviour. I have pushed the changes. Please try and let me know if you still face so issue.

Also, I would be happy to accept any pull request that helps to improve the content.


Add a menu item to .nib

First, let me say a huge "thank you" to @sveinbjornt for all his hard work on this most excellent and versatile application. I've already used it to create a bunch of useful Mac apps and I'm blown away. Wow! Thank you!
I know the focus of Platypus is on simple script execution and not complex interaction, but I've put together a pretty good preferences window for my .app using Pashua, and was hoping to be able to invoke it somehow.
The ideal way to do this of course would be to add a "Preferences" menu item by editing the .nib file. I can add the menu item, but have no way to link it to any kind of action. I'm a complete Xcode novice and generally find it baffling. I've been able to link the menu item to a few predefined actions but that's about it.

I have a script in my .app bundle that invokes the Pashua window--I just can't figure out how to link it to this menu item or to get it to launch from inside my Platypus app. I know there's a myriad of ways of gathering user input using AppleScript and other types of dialogs, but as the concept of preferences is not really a part of the app workflow, I was hoping for some sort of a solution. Re-running my main script with an argument, launching an external .app, launching a script, anything at all would allow me to open the prefs window.
Many thanks in advance for help/advice.

I'm still hoping someone out there has made a Platypus app with a preferences window and might be able to point me in the right direction!

Something like Pashua might do the trick. Caveat: Haven't checked the state of that project in a while.
https://www.bluem.net/en/projects/pashua/

But you're not going to be able to hook into that Preferences menu item unlesss the ScriptExec binary and its assets are modified. How would the interaction with Platypus even work? A GUI window within Platypus where you can add modifiable preferences to the app? How would the Prefs window look? What kind of controls should be on offer? What kind of values? String, boolean, multiple choice? Complexity starts to scale quickly here.
That being said, you could always alter the ScriptExec binary according to your needs.

I've tried, and in fact it does not (that project is also sort of in "open source doldrums"). The workaround is that I can use an external tool to detect a keypress on launch, and open a cocoaDialog window that way, but this only works on launch, and is far from elegant. I'm assuming there's no way to create a simple hook from that menu item to invoke a script or action? I really love Platypus and have been using it like crazy, but it would be a tremendous help to have any kind of method for invoking an additional script or app after the primary script has started running. I know this is not the paradigm, but for something like a preferences window I can't see another solution that would make sense.

I actually got a really good idea from thinking about this. What if I modify the ScriptExec binary to have a generic IBAction called menuItemSelected: or something, which isn't hooked to anything, but when called, runs the script with the menu item title (much like in Status Menu mode). Power users would then be able to add menu items in the nib and attach it to the menuItemSelected: action.

Oh my god ... This is EXACTLY what I was hoping for!!

I am over the moon! I'm not sure if I will be able to build Platypus from source though, as in the past I had an issue with developer code signing. I'm very sorry but I'm pretty bad with XCode (which is a bit of a catch-22 because if I could figure out I might actually make some native apps). I'm on 10.13.6 because Mojave introduced some new sandboxing stuff that broke many of my Platypus apps and I haven't had time to sort it out. If there is some way to work with a compiled app with the modification, I will dive into it tonight and let you know how it goes!

OK, just pushed this to master. You should be able to build Platypus without problem by just disabling code signing on the Platypus and platypus_clt targets. Then cd to the project directory, and run ./build_release.sh. Should do the trick. If you run into trouble, ping me and I'll send you binaries.

I'm so sorry, but I wasn't able to get it to build.

Binaries here:
https://sveinbjorn.org/files/bin.zip

Thanks and sorry I'm such an xcode dunce. I'll let you know how it goes!

No worries. This Apple signing business sucks, really clamps down the platform. I'm leaving soon, probably when my creaking 2012 rMBP kicks the bucket, Linux on generic x86 after that.

Sorry to hear, and hope Platypus lives on for a long time to come. I've been able to create GUIs for a whole lot of Mac-specific software that folks typically run in a terminal window with the PHP cli, and it has been invaluable.
Yeah, Apple is not developer-friendly these days, esp where desktop software is concerned. This kind of stuff is driving me nuts:
https://www.felix-schwarz.org/blog/2018/06/apple-event-sandboxing-in-macos-mojave
It does seem like Ubuntu keeps getting more viable as a full service desktop OS, but there's always stuff I miss. At least with the 2018 Macbook Air there's finally a reasonable dev machine being manufactured again.

Oh wow, I built a test app and it is truly glorious! So many thanks for this incredible addition! I'm going to push a bunch of updates to my Platypus apps and hopefully make some people very, very happy!
https://drive.google.com/open?id=1Zl-vrexu5ZN3NZWIa9Wu7MR9MBphU63z


Remove the sidebar/drawer from Android.

In the new design, the hamburger icon has been removed but the navigation drawer is still there and can be accessed by swiping from the left. This needs to be removed.

Implemented in final redesign


k8s client go update

@kyessenov: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/pilot-presubmit.sh
0c8bc0ab008069f800d2eeb5ba9d6258a21b6f42
link
@istio-testing bazel test this

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Codacy-test-with coordinate-ordering

A test to see how codacy reacts to certain changes

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Complexity decreasing per file
==============================
+ lib/improver/utilities/cube_manipulation.py  -3

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Complexity decreasing per file
==============================
+ lib/improver/utilities/cube_manipulation.py  -3

See the complete overview on Codacy


nlp: wrong entity suggestion

try a sentence that is analyzed as an intent with entities
create a new intent for this sentence
-> there should be no entity suggestions, but those of the first intent are used

fixed