Adding N parameter to rise/set/transit functions

tl;dr: you can speed up rise/set times by an order of magnitude in exchange for a few minutes of rise/set time precision, without changing anything fundamental about how astroplan/astropy compute rise/set times.
This PR implements the changes I suggested in #423, which I summarize briefly below. There are also several PEP8 compliance fixes throughout the docstrings on the Observer object. If merged, this PR supersedes #327.
The primary goal of this PR is to expose the N parameter, which has been lurking within the guts of the Observer object since its inception. Rise/set/transit times are computed using a grid search, which scans for horizon crossings over N=150 times spread over 24 hours relative to the reference time supplied by the user. By throttling that parameter up or down, the user can simultaneously change the precision and speed of rise/set/transit calculations. N~100 gives precision better than 1 min from the true rise/set/transit time, N~10 gives precision better than 5 minutes and is about 10x faster. This fix is designed to address the common complaint that rise/set times are slow (#402, #423).
The default behavior of the functions remains unchanged. When not supplied by the user, N=150 by default for all functions. Thus none of the tests need to change, and introducing this new optional kwarg won't change people's results in legacy code.
Still to do:

add narrative docs explaining the update
include plots in docs comparing precision at different N

Tagging some users who might have opinions about this implementation: @wtgee, @StuartLittlefair, @iwellaway, @eteq.

@bmorris3 I like this approach a lot, and it does indeed make things a lot faster, and is simple to understand for the end user.
For a single time/target I find the speedup between N=150 and N=10 is only x2, but this obviously improves with grids of times and targets, since N broadcasts against those grids, so the speedup will be more impressive if doing large numbers of calculations.
One thing I've often wondered is why we couldn't use a 2-pass scenario, where we use a small N to get an initial rough estimate of the event time, and then re-grid 10--15 mins either side of this estimate (using the same N) to refine it, i.e use a few lines like the ones below at the end of these functions...
    t = self._two_point_interp(jd1, jd2, al1, al2, horizon=horizon)
    # normally we'd return here, but let's use a finer grid,
    # 15 mins either side of rough time
    times = _generate_24hr_grid(t, -0.01, 0.01, N)
    altaz = self.altaz(times, target, grid_times_targets=grid_times_targets)
    altitudes = altaz.alt
    al1, al2, jd1, jd2 = self._horiz_cross(times, altitudes, rise_set, horizon)
    return self._two_point_interp(jd1, jd2, al1, al2, horizon=horizon)

This is roughly the same speed for a single time/target for N=10 vs N=150 (as the two calculations offset the x2 speedup), but is still 5x faster for a grid of 300 times and agrees with the N=150 case to within 1 second.
Or - giving the option to the user, you could switch a two-pass behaviour on or off using an argument, and allow the user to tune N and two_pass for their own accuracy requirements...

@StuartLittlefair I seem to remember that it was originally our intention to use a two-pass solution, but looking back through the PR history, I can't find evidence that we ever actually did a two-pass solution.
I think something @eteq and I discussed casually a few times is that atmospheric pressure and humidity along the line of sight can probably change rise/set times by a few minutes here or there, so we wondered whether it would be physically meaningful to provide rise/set times that are accurate to the minute anyways. I suppose I'm making the arguments that (1) perhaps the default N could be smaller, and (2) a two-pass solution is probably overkill.

I don't really disagree with any of that. But I would make the following counter-arguments:

for any given accuracy requirement, a two-pass solution is quicker than just using a finer grid over 24 hour

It seems like a simple change to the code, I think the lines pasted above work fine

I've updated the PR to include narrative docs, which render like so:

Nice. I don't think my comments should hold up merging this PR BTW - I could always submit a separate PR for a two-pass approach at a later date.

this looks good and definitely helps in cases where precision isn't too important (like twilight times). i think there's still more to be gained from either a two-pass approach as suggested by @StuartLittlefair or an iterative approach like skyfield uses (see find_discrete in https://github.com/skyfielders/python-skyfield/blob/master/skyfield/almanac.py). those can be subjects for future PRs, though, and shouldn't hold this up...


How to get the selected value of a cell?

Hi,
In my grid, there is a drop-down column, so I want to change a value of another cell (same row) according to selection of drop-down cell.
I checked the documentation but I could not able to find the method/event that using to get the value of dropdown item change,
and how to change another cell's value within that.
if anyone can help me with that would be very much appreciate.
Thanks

any helpwould be appreciate

Hi @tabalinas
I've implemented both onItemUpdating and onItemUpdated functions but those are not executing when changing a cell value,
In my grid, I dont have a separate column for update, i want to update other column's value when switching to cell, hence I need to get the value on cell click,
a little help please?

@thilankascentic, are you talking about something related to https://github.com/tabalinas/jsgrid/issues/929#issuecomment-333538846?


Always include to_one linkage

This PR reenables the always_include_to_one_linkage_data configuration option. This joins in the to_one relationship in order to retrieve the linkage data without additional database requests.
The path parsing logic for includes, filters and sorts was moved to a Path class that parses into Relationship and Field path parts.
Filtering is extended across polymorphic relationships.
Enables the a new join_left ponyfill to extend left_joins to rails 4 and works around an issue with nested joins not aliasing correctly on rails 5 and 5.1.
Streamlines the JoinTree and Joins handling of polymorphic relationships in paths. Filters can now specify the specific resource type of a polymorphic relation using the format relationshipname#resource_type. For example pictures.imageable#document.name
All Submissions:

[ ] I've checked to ensure there aren't other open Pull Requests for the same update/change.
[ ] I've submitted a ticket for my issue if one did not already exist.
[ ] My submission passes all tests. (Please run the full test suite locally to cut down on noise from travis failures.)
[ ] I've used Github auto-closing keywords in the commit message or the description.
[ ] I've added/updated tests for this change.

New Feature Submissions:

[ ] I've submitted an issue that describes this feature, and received the go ahead from the maintainers.
[ ] My submission includes new tests.
[ ] My submission maintains compliance with JSON:API.

Bug fixes and Changes to Core Features:

[ ] I've included an explanation of what the changes do and why I'd like you to include them.
[ ] I've provided test(s) that fails without the change.

Test Plan:
Reviewer Checklist:

[ ] Maintains compliance with JSON:API
[ ] Adequate test coverage exists to prevent regressions

Let's deprecate Resource#find_records properly to discourage its use.

@joegaudet, @dgeb Thanks for the reviews. I think I've got all the suggestions implemented or addressed, with the exception of deprecating the find_records method. I'm still looking for a good way to handle that so people who have it overridden can be notified they may need to change it. As it is now it will at least fail so I haven't renamed the method. Any suggestions?


⚠ RPC server - Conflux has degraded performance

In 5692296, RPC server - Conflux ($CONFLUX_RPC_URL) experienced degraded performance:

HTTP code: 200
Response time: 1285 ms

Resolved: RPC server - Conflux performance has improved in 3b5ad68 after 14 minutes.


Add Firebase notifications

Simply added Firebase push notifications functionality.

Thanks @Matttd. Can you try to fix this minor issues in your code?

Thanks @Matttd. Can you try to fix this minor issues?

Thank you again :+1:
Merged


No BDD100K format jsons generated after running convert_to_bdd.sh

No BDD100K format jsons (for MOTS challenge submit) generated after running convert_to_bdd.sh.
After running convert_to_bdd.sh, In pcan-main/converted_results/seg_track dir, png masks generated but I can not find json files.
my convert_to_bdd.sh:
python ./tools/to_bdd100k.py configs/segtrack-frcnn_r50_fpn_12e_bdd10k_fixed_pcan.py --res eval_result_pcan_test.pkl
--task 'seg_track' --bdd-dir converted_results

The benchmark used to also accept the png masks files but it has changed recently. To get the json, install the latest bdd100k api (https://github.com/bdd100k/bdd100k) and run following command:
python -m bdd100k.label.to_rle -i dir/to/bitmasks -o output.json -m seg_track

The benchmark used to also accept the png masks (bit masks) files but it has changed recently. To get the json, install the latest bdd100k api (https://github.com/bdd100k/bdd100k) and run following command:
python -m bdd100k.label.to_rle -i dir/to/bitmasks -o output.json -m seg_track
"-i", "--input", help=("directory of bitmasks to convert") "-o", "--output", help="path to save scalabel (bdd100k) formatted label file with RLEs"
You can reference to this file.

Thank you for your reply! I run the bdd100k.label.to_rle command and successfully get the output.json (one JSON file of the entire validation set). I want to submit it to ECCV2022 BDD100K Multiple Object Tracking and Segmentation Challenges(https://eval.ai/web/challenges/challenge-page/1834/overview) , but i get error as follow:
Traceback (most recent call last):
File "/code/scripts/workers/submission_worker.py", line 491, in run_submission
submission_metadata=submission_serializer.data,
File "/tmp/tmp2tp1i0z9/compute/challenge_data/challenge_1834/main.py", line 103, in evaluate
nproc=1,
File "/usr/local/lib/python3.7/site-packages/scalabel/eval/seg_track.py", line 46, in evaluate_seg_track_all
hota_result = evaluate_seg_track_hota(gts, results, config, nproc)
File "/usr/local/lib/python3.7/site-packages/scalabel/eval/hotas.py", line 237, in evaluate_seg_track_hota
assert len(gts) == len(results)
AssertionError

Hi have you solved this problem?

videoName

Could you tell me how to solve this problem? What's wrong with 'videoName' and how you modify it? Thanks very much.

Could you tell me how to solve this problem? What's wrong with 'videoName' and how you modify it? Thanks very much.

Could you tell me how to solve this problem? What's wrong with 'videoName' and how you modify it? Thanks very much.

yes, I fixed the "videoName" in the json and solved this problem.

Could you tell me how to solve this problem? What's wrong with 'videoName' and how you modify it? Thanks very much.


Fix for ML 0.6.0 and TLD 2.04+

Had to rework the core method, see source code for notes/comments

Tested on existing saves and new games.
Tested dropping & picking up - matches, antibiotics, sticks from inventory (of same % and different)
Tested adding from console - matches, antibiotics, sticks, curedRabitPelts (console condition is wierd, seems to default to 100%)

Will resubmit once I have a more complete solution


Non-smooth behavior when typing text

We should add a small (150ms or so) delay between typing text in the textfield and updating the qr code. Right now it "stutters" when typing quickly because rendering the QR code takes quite some time.

This problem came with 81a3078dd5b4de88d32ca1871614e5bf0b926fe7


[macOS] Compile errors when building with C#

Godot version:
Godot 3.2.x from commit 7b4b83e9dcb91b44b39e8cc87df3ddecdb29b689
OS/device including version:
macOS 10.14.6
Issue description:
Trying to follow the instructions to build with C# support.
I'm using the following options to scons:
target=debug -j 8 dev=yes use_llvm=yes tools=yes module_mono_enabled=yes mono_glue=no

This produces many build errors because -Werror and -Winconsistent-missing-override are both specified.
Example build errors:
In file included from modules/mono/csharp_script.cpp:31:
modules/mono/csharp_script.h:70:2: error: 'get_class' overrides a member function but is not marked 'override' [-Werror,-Winconsistent-missing-override]
        GDCLASS(CSharpScript, Script);
        ^
./core/object.h:271:17: note: expanded from macro 'GDCLASS'
        virtual String get_class() const {                                                                                \
                       ^
./core/script_language.h:101:2: note: overridden virtual function is here
        GDCLASS(Script, Resource);
        ^
./core/object.h:271:17: note: expanded from macro 'GDCLASS'
        virtual String get_class() const {                                                                                \
                       ^
In file included from modules/mono/csharp_script.cpp:31:
modules/mono/csharp_script.h:70:2: error: '_get_class_namev' overrides a member function but is not marked 'override' [-Werror,-Winconsistent-missing-override]
        GDCLASS(CSharpScript, Script);
        ^
./core/object.h:274:28: note: expanded from macro 'GDCLASS'
        virtual const StringName *_get_class_namev() const {                                                                                \
                                  ^
./core/script_language.h:101:2: note: overridden virtual function is here
        GDCLASS(Script, Resource);
        ^
./core/object.h:274:28: note: expanded from macro 'GDCLASS'
        virtual const StringName *_get_class_namev() const {                                                                                \
                                  ^

Example compile command
clang++ -o modules/mono/mono_gd/gd_mono_method.osx.tools.64.o -c -std=gnu++14 -Wctor-dtor-privacy -Wnon-virtual-dtor -g3 -arch x86_64 -mmacosx-version-min=10.9 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Werror -DDEBUG_ENABLED -DDEBUG_MEMORY_ENABLED -DDEBUG_MEMORY_ALLOC -DDISABLE_FORCED_INLINE -DTYPED_METHOD_BIND -DOSX_ENABLED -DUNIX_ENABLED -DGLES_ENABLED -DAPPLE_STYLE_KEYS -DCOREAUDIO_ENABLED -DCOREMIDI_ENABLED -DGL_SILENCE_DEPRECATION -DPTRCALL_ENABLED -DTOOLS_ENABLED -DGDSCRIPT_ENABLED -DMINIZIP_ENABLED -DZSTD_STATIC_LINKING_ONLY -DGLAD_ENABLED -DGLES_OVER_GL -DMODULE_ASSIMP_ENABLED -DMODULE_BMP_ENABLED -DMODULE_BULLET_ENABLED -DMODULE_CAMERA_ENABLED -DMODULE_CSG_ENABLED -DMODULE_CVTT_ENABLED -DMODULE_DDS_ENABLED -DMODULE_ENET_ENABLED -DMODULE_ETC_ENABLED -DMODULE_FREETYPE_ENABLED -DMODULE_GDNATIVE_ENABLED -DMODULE_GDSCRIPT_ENABLED -DMODULE_GRIDMAP_ENABLED -DMODULE_HDR_ENABLED -DMODULE_JPG_ENABLED -DMODULE_JSONRPC_ENABLED -DMODULE_MBEDTLS_ENABLED -DMODULE_MOBILE_VR_ENABLED -DMODULE_MONO_ENABLED -DHAVE_CXX11_THREAD_LOCAL -DGD_MONO_HOT_RELOAD -D_REENTRANT -I/Library/Frameworks/Mono.framework/Versions/Current/include/mono-2.0 -Ithirdparty/libpng -Ithirdparty/glad -Ithirdparty/zstd -Ithirdparty/zlib -Iplatform/osx -I. modules/mono/mono_gd/gd_mono_method.cpp

Regression from 0da84b5 (cherry picked from commit 51e1614, without cherry picking 26fcf2b first).

Welp, we'll have to be carefully when cherrypicking from now on unless we decide to use override in 3.2 (but it would break third-party code so I'm not in favour of doing it).

(cherry picked from commit 51e1614, without cherry picking 26fcf2b first)

I don't think that's correct - you don't want override in 3.2 code.
Removing override seems to be the correct fix.
PR incoming...

Fixed by #40573.

I'm still getting this issue when compiling mono with a custom module (sg_physics_2d). I'm using the 3.5.stable source code, and I just move the custom module under the modules folder
In file included from modules/sg_physics_2d/register_types.cpp:34:
In file included from modules/sg_physics_2d/./scene/2d/sg_fixed_position_2d.h:27:
modules/sg_physics_2d/./scene/2d/sg_fixed_node_2d.h:35:2: error: 'get_class' overrides a member function but is not marked 'override' [-Werror,-Winconsistent-missing-override]
        GDCLASS(SGFixedNode2D, Node2D);
        ^
./core/object.h:270:17: note: expanded from macro 'GDCLASS'
        virtual String get_class() const {                                                                                \
                       ^
./scene/2d/node_2d.h:37:2: note: overridden virtual function is here
        GDCLASS(Node2D, CanvasItem);
        ^
./core/object.h:270:17: note: expanded from macro 'GDCLASS'
        virtual String get_class() const {                                                                                \
                       ^
In file included from modules/sg_physics_2d/register_types.cpp:34:
In file included from modules/sg_physics_2d/./scene/2d/sg_fixed_position_2d.h:27:
modules/sg_physics_2d/./scene/2d/sg_fixed_node_2d.h:35:2: error: '_get_class_namev' overrides a member function but is not marked 'override' [-Werror,-Winconsistent-missing-override]
        GDCLASS(SGFixedNode2D, Node2D);
        ^

This is an issue with that module, you should report it on that module's repository. It's 3.x branch shouldn't use override.


[Improvement][scheduler]When configuring the Cron Manage, I think that Start and stop time is not necessary.

Describe the question
When configuring the Cron Manage, I think that Start and stop time is not necessary.
I only config the cron expression，and the Start and stop time is Optional.

Which version of DolphinScheduler:
-[1.3.4-release]

In my opinion, the start and stop times do not affect the performance. The default time is used for tasks that need to run all the time, and the time can be selected according to your own needs

In my opinion, the start and stop times do not affect the performance. The default time is used for tasks that need to run all the time, and the time can be selected according to your own needs

I think the start and stop times and the cron expression position exchange are more friendly, and the start and stop times is made into optional configuration (default is no end time). The current design is ambiguous.

I think the start and stop times and the cron expression position exchange are more friendly, and the start and stop times is made into optional configuration (default is no end time). The current design is ambiguous.

The current end time is 100 years after the current time, which is also equivalent to an infinite end time.
Of course, you can also wait for feedback from other users. If everyone thinks it's not good, it can be optimized

The current end time is 100 years after the current time, which is also equivalent to an infinite end time.
Of course, you can also wait for feedback from other users. If everyone thinks it's not good, it can be optimized

I think the start time is important, since the task will run at once after cron manage online , i think it's a bug, but can be avoid by set the start time.


Add docker-compose and link to wiki

Adds a docker-compose.yaml for a convenient single-host deployment starting point. The wiki has been updated to include minimal setup and run instructions using standard tooling.
Draft until the dependencies can be pinned with a fully working solution. Until then this branch can be considered a test for acceptance of the complete solution.

I am getting a lot of errors locally on first try. I need to double-check my postgre configuration but somethings seems a bit off with the volumes :thinking:

I had staged this for the nix developers to use it as acceptance criteria. We cannot couple the service state with the node socket in a volume, so the Docker images need to change. I'm working on this

Yeah, I've been hacking around yesterday and there are a few annoying things with the images as they are right now:
a) The node.socket can't just be easily shared across services because it is a file and mounting files as volumes is kinda tricky with Docker. As a work around, I mounted the whole /data and within which happen to also be /data/node.socket. Ugly, but does the job for now.
b) Logs from cardano-db-sync are not really user friendly. Forces us to mount a /tmp volume in order to access them... when there are logs. I ran into 2 or 3 errors that were simply very cryptic and occurred before the log handlers were even in place.
c) The pgpass file also has to be mounted as a volume, though ideally, I'd rather have this one manage as a secret since in the end, it also contains the postgres credentials.

Improvement proposal https://github.com/input-output-hk/cardano-db-sync/issues/26


404 at: https://docs.docker.com/sorry/#https://docs.docker.com/articles/https/

Incoming Link from Azure Docker on Ubuntu Server compute server.
https://docs.docker.com/sorry/#https://docs.docker.com/articles/https/
Can not find the documentation  Running Docker with https to configure Docker to be reachable via the network in a safe manner linked via azure.

Fixed in 5389e6cb122077d705692dec43555bc7a74ea5e0


[Adaptive Card > OrderDelivery.Json]: Voice over does convey the incorrect information as 'Dimmed' for the 'Delivery' button.

Test Environment:
Device: iPhone 11
iOS Version: 16.7.1
App Version: 1.0 (2.4.0-beta.<IP_ADDRESS>)
Screen Reader: Voice Over
Repro Steps:

Launch Adaptive Cards iOS application.
Home screen will appear.
Navigate to "V1.5" control and select it.
Navigate on "V1.5" screen and select "Scenarios" button.
'Scenario' screen will appear.
Navigate to "OrderDelivery.json" control and double tap to activate it.
Navigate to the 'Delivery' button and observe the issue.

Actual Result:
Voice over does convey the incorrect information as 'Dimmed' for the Delivery button and on double tapping on it no action is getting performed.
Also, Delivery button does not looks like a disabled control.
Expected Result:
Voice over should convey the correct information for the 'Delivery' button and when focus comes on it then it should announce as 'Delivery button'.
User Experience:
It will impact the screen reader user if the voice over does convey the incorrect information for the non-disabled control.
MAS Reference:
https://liquid.microsoft.com/Web/Object/Read/MS.Accessibility/Requirements/01.03.01
"Have feedback to share on Bugs ? Please help fill Trusted Tester Bug Feedback (office.com) “
https://github.com/microsoft/AdaptiveCards-Mobile/assets/95913748/b3f1d276-f829-4fcb-a5d5-c67f805e56e1

Currently we don't have any specific appearance when the button is not enabled (like the grey color for "destructive" sentiment components). By default, disabled system buttons doesn't change at all. Any change is easy to add but I have doubt of what is expected in this case (As an example I changed the alpha value when is not enabled in the screenshot). Note, this could change diff tests. cc @VikrantSingh01

@GabrielMedAlv see this image for Android


Metrics data behind load balancer

Question
Hi everyone. I assume that you are using the client in several use cases. My question is regarding the metrics api on the prometheus format. If I have many tusd-server replicas behind the same load balancer, how can I scrap metrics from specific servers? Prometheus only support statics targets, If I use an Amazon Autoscaling Group, or a Kubernetes pods environment... How can I configure metrics to be scraped? Also prometheus has a pushgateway utility but it is no recommended for service layer scrapping.
A workaround may be to export metrics in a json format and scrapping with telegraf and influxdb to save.
Is there any way to get those insights? Is there any metrics on json format?
I may probably upload this question in stackoverflow but I decided to try here first.
Regards

Hi there, this sounds like a general question about Prometheus and not tusd directly. I am not a Prometheus expert, so I recommend you to get support from them directly. However, from a brief research it looks like you need to somehow target the servers inside your autoscaling group directly: https://stackoverflow.com/a/49034770 Beside that I cannot provide you with more help.

Is there any metrics on json format?

No, the metrics are not currently available in the JSON format.

Hi @Acconut
Thanks for the answer.
I did my own driver to get the same metrics on a Json format using runtime, os, and prometheus/procfs.
Regards

Glad to hear you found a solution!


Allow package to filter {,out} specific paths

Now the (unrelated to this PR) error with Julia v1.8.1
Compat functions: Test Failed at /home/runner/work/BinaryBuilderBase.jl/BinaryBuilderBase.jl/test/compat.jl:33
  Expression: stdlib_version(gmp_jll, v"1.8") == v"6.2.1+2"
   Evaluated: v"6.2.1+1" == v"6.2.1+2"

is quite funny: this was passing with Julia v1.8.0.


Added option to provide audiences to the k8s service account auth

enable audiences example
Example of the spec
spec:
  vault:
    path: pki/sign/sandbox
    server: https://vault.net
    auth:
      kubernetes:
        role: sandbox-issuer
        mountPath: /v1/auth/kubernetes
        serviceAccountRef:
          name: sandbox-issuer
        audiences:
        - vault

Pull Request Motivation
Motivation is elaborated in #6666
Kind
feature
Release Note
Added option to provide additional audiences in the service account auth section for vault

/ok-to-test
/kind feature

Thanks for raising this @andrey-dubnik  - we'll try to take a look after the 1.14 release has been completed 😁

I assume this will not make it into 0.14.
Do you assume this will make it into a 0.14.x patch release or will it most likely land in 0.15 (all under the assumption: "if accepted and merged") ?

I assume this will not make it into 0.14.
Do you assume this will make it into a 0.14.x patch release or will it most likely land in 0.15 (all under the assumption: "if accepted and merged") ?

Yeah this was raised after the code freeze for 1.14 so no chance it gets in there. This is a new feature so we wouldn't normally backport it. We only backport bug fixes and things like doc updates / build improvements normally
This should make it into 1.15!

I'm going to reopen as can't sort the DCO


Enable Cosmos HTTPS tunneling through a proxy secured by basic auth (for master)

See https://bugs.openjdk.java.net/browse/JDK-8168839
High Level Description
Fixes DCOS-15542. In release 8u111 of the JDK, basic auth was disabled for HTTPS tunneling. Since DC/OS 1.9 shipped with a more recent JDK, it prevented some customers from accessing the package Universe from their cluster. This change removes basic auth from the disabled list.
Related Issues

DCOS-15542. Cosmos fails any request that queries Universe through a proxy with authentication

Checklist for all PR's

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: this is for a DC/OS component. Test is in the component project.
[X] Read the DC/OS contributing guidelines
[X] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and Review Process
What is the review process and when will my changes land?
All PRs require 2 approvals using github's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot label Ready For Review

@jsancio @larioj @jgehrcke @lingmann please review, thanks!

@mesosphere-mergebot bump-ee


How to detect bounding boxes at the back of the vehicle?

This is a dataset/general query.
I have trained an object detector with "centerpoint_rnn.cfg" config and the model has trained and is working well. But I noticed just now that the KITTI dataset has gt bbox information just for objects which are in the FOV of the cameras. So the cars at the back of the vehicle are not detected. This has caused problems for me since I am working on Moving Object Segmentation and I want to detect vehicles that are at the back as well. Any suggestion on how that can be achieved.

In the image, it can be seen that  the cars in front of the vehicle bearing laser scanner are detected but there is a car just behind the vehicle which remains undetected

Hi ,
I'm having the same issue but somehow changing the point cloud range didn't effect .
The detector doesn't recognise the objects in the right part of the point cloud and even when I change the point cloud range and make sure the point coordinates are inside the point cloud range - objects there are still not detected at all.
And I also noticed that not every number there is accepted, some cause bugs , didn't understand yet what causes it .
Did you change anything else other than the point cloud range ?
Thank you !


Correction of TextFormField colors on CustomDialog

- Changed the TextFormField focus border color to the design color instead of the the default color of a TextFormField
- I added a colorScheme property to the ThemeData widget and set the primary and secondary properties to the Design color
- Tested it with my mobile device
- I added a colorScheme property to the ThemeData widget and set the primary and secondary properties to the Design color

Correction of TextFormField colors on CustomDialog


Generate random whole numbers challenge not evaluating

Challenge http://www.freecodecamp.com/challenges/waypoint-generate-random-whole-numbers-within-a-range has an issue.
I ran this Code, which i believe its correct, but the editor is not evaluating it::

function myFunction() {
// Make myFunction return a random number between zero and nine instead of a decimal
// Only change code below this line.
return Math.floor(Math.random() * (9 - 0 + 1)) + 0;
}

@jamesagwa thanks for posting this issue!
I guess you didn't pass third test? Yes your code is definitely correct. But you can see your test's assert function contents and according to those you have to use min and max variables minimum twice. This is the situation when the JavaScript instructions for processing challenge can help you to solve the challenge. (check Waypoint: Comment your JavaScript Code - the last paragraph of the challenge description clarifies this.)
Please review the Guidelines for Contributing, thank you!.


WIP: Bring back the webcam for profile submission

Actually, the webcam video got stored as .mkv for me. This happened in Chrome. Mozilla correctly stores it as .webm.

Should be fixed now 🙌🏻


cpptools Integration - trying to to understand

Brief Issue Summary
I have just started using VSCode and CMake Tools (CMT).  I came across some problems with include statements in header files.  I have looked at issue #481 as it seems close to what I am seeing.

I used CMake: Edit usr-local CMake kits to add my toolchain otherwise my toolchain did not show even though it is in my environmental PATH.
Question - Shouldn't CMT automatically detect my compiler if it was in my PATH?
Started new project and selected my toolchain kit and is listed in bottom menu. When popup came up "CMake would like to configure intelliSense for this folder" I clicked allowed.
If I opened a header file in my project 1st I would receive a bunch of green squiggle lines on includes.

But if I open Greetings.cpp 1st and then Greetings.h most green squiggles go away.

Reading issue #481 states you need to open the cpp file first before the header file which the OP thought was fixed.
Question - Do we need to open up the cpp file first before the header file?

I want to include header file from external library (esp-idf) whose components are part of the build directory but their CMakeLists.txt files do not contain the *.h files.  In picture below, the build folder shows the driver folder and the freetos folder but I still get the green squiggly lines trying to include headers from their folders.

If I take the suggestion and add these to IncludePath a new c_cpp_properties file is created and then all the green squiggly lines come back.

Question - How do I add these includes and not get the green squiggly lines

If I look at the newly created c_cpp_properties.json file is lists the wrong compiler path.

Question - Why is the wrong compiler path listed?
Platform and Versions

Operating System:  Linux Mint 18
CMake Version:  3.12.4
VSCode Version:  1.29
CMake Tools Extension Version:  1.1.2
Compiler/Toolchain:  xtensa-esp32-elf-gcc

Other Notes/Information

Shouldn't CMT automatically detect my compiler if it was in my PATH?

It should. What is you compiler path, and what is the output from --version?

Do we need to open up the cpp file first before the header file?

For an unknown header, cpptools will use the configuration information from a source file which immediately includes that header. If you haven't yet opened a source file that includes the header, it has no choice but to guess. If the header file is associated with a target, then cpptools will not have to guess the configuration.

How do I add these includes and not get the green squiggly lines?

For this, I do not know. cpptools has a debug more where it will print the how/why it is choosing the configuration for a file. That's you best bet to find out.

Why is the wrong compiler path listed?

That file is generated by cpptools. I do not know how its default content is decided.

Here is my path.
/home/ed/bin:/home/ed/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/home/ed/esp32-smooth/esp-idf/tools:/home/ed/esp32-smooth/xtensa-esp32-elf/bin

Here is the version.
ed@ed-ideacentre-AIO-910-27ISH ~/esp32-smooth/xtensa-esp32-elf/bin $
xtensa-esp32-elf-gcc-5.2.0 --version
xtensa-esp32-elf-gcc-5.2.0 (crosstool-NG crosstool-ng-1.22.0-80-g6c4433a)
5.2.0
Copyright (C) 2015 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

If the header file is associated with a target, then cpptools will not
have to guess the configuration.
ATask.h and BTask.h are in target in the include folder of target and
smooth/core/Application.h, smooth/core/Task.h and
smooth/core/task_priorites.h are in target in the externals folder but
still get the green squiggly lines if I open header file first.

It sounds like the cpptools integration isn't as useful as I thought it
would be and will revert back to c_cpp_properties.json file where I place
all my includePaths and browse paths so my includes will be found.  Other
that this issue CMT is very good.

Thanks
On Tue, Nov 27, 2018 at 12:39 PM vector-of-bool<EMAIL_ADDRESS>wrote:

Shouldn't CMT automatically detect my compiler if it was in my PATH?
It should. What is you compiler path, and what is the output from
--version?
Do we need to open up the cpp file first before the header file?
For an unknown header, cpptools will use the configuration information
from a source file which immediately includes that header. If you haven't
yet opened a source file that includes the header, it has no choice but to
guess. If the header file is associated with a target, then cpptools will
not have to guess the configuration.
How do I add these includes and not get the green squiggly lines?
For this, I do not know. cpptools has a debug more where it will print the
how/why it is choosing the configuration for a file. That's you best bet to
find out.
Why is the wrong compiler path listed?
That file is generated by cpptools. I do not know how its default content
is decided.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vector-of-bool/vscode-cmake-tools/issues/599#issuecomment-442189547,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Apd4__WK-HZiJGDQibP09jEwAL4_kvgjks5uzZTngaJpZM4Y2K44
.

When you open the file in which the green underlines appear, do you receive a warning about missing configuration for that file?

@vector-of-bool @enelson1001 Hi guys,
It looks like this issue is very similar to mine: https://github.com/vector-of-bool/vscode-cmake-tools/issues/604
Based on my experience, I have just found that CLion works fine with ESP-IDF, KDevelop also fine for ESP-IDF with pure C code but cannot recognise C++ at all. It actually partially works with your plugin and VSCode but there are a lot of annoying false alarms. So I'm suspecting if it can be something wrong with Espressif themselves who designed their CMake build system.
@vector-of-bool, If you need me to test something for solving these issues, please reply me and I will assist you as much as I can.

The goal is that you should not need to edit c_cpp_properties.json at all when you are working with a CMake project. If you are finding yourself needing to specify options here, then either there is 1) a CMake Tools bug, 2) a cpptools bug, or 3) your project is misconfigured.
I've seen a lot of instances of (3), and its the easiest thing to fix. I also see instances of (1) and (2), but they are more difficult. Trying to rule out what has gone wrong is the best I can do from where I am.

vector-of-bool
I created a new project and allowed CMT to configure intellisense.
When I open header files in my project I am receiving the following
notifications.  This time I am not getting any green squiggly lines
under the includes but just these notifications that pop-ups.
[image: image.png]
But I can peek at the Definition.
[image: image.png]
No errors in build.
This is the folder structure.
[image: image.png]
This is the CMakeLists.txt under main folder
[image: image.png]
Why are the notifications popping up?
Thanks
On Sun, Dec 2, 2018 at 1:31 AM vector-of-bool<EMAIL_ADDRESS>wrote:

The goal is that you should not need to edit c_cpp_properties.json at all
when you are working with a CMake project. If you are finding yourself
needing to specify options here, then either there is 1) a CMake Tools bug,
2) a cpptools bug, or 3) your project is misconfigured.
I've seen a lot of instances of (3), and its the easiest thing to fix. I
also see instances of (1) and (2), but they are more difficult. Trying to
rule out what has gone wrong is the best I can do from where I am.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vector-of-bool/vscode-cmake-tools/issues/599#issuecomment-443490744,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Apd4_0MtT-Gk3OCO-Xiksto9P0_3OyBaks5u04_xgaJpZM4Y2K44
.

@enelson1001 hey mate, your images are corrupted

Lets try this again
Notifications

Peek Definition

Folder Structure

CMakeLists.txt under main folder

Kit being used

Thanks

This has actually come up often enough that I've written a section for it in the documentation just yesterday. Take a look and see if it fixes your issue. In your case, you will need to add your headers to the COMPONENT_SRCS variable.

Added the includes in the CMakeLists.txt file as shown below and now the pop-ups have disappeared.

Could you please bold this statement in the new section you wrote yesterday.
"If the file which receives this message is outside of your project, it is safe to ignore it."
I redid another previous project and removed c_cpp_properties.json and the build directory, updated the CMakeLIsts.txt file and restarted the project and  was getting green squiggly lines on the includes outside my project but everything built and ran OK. I reread your new section and it would be helpful if that sentence is bold and stands out.
Thanks for helping me out on this problem.
This issue can be closed.

One more thing maybe you should include in write-up.
I noticed that when I tried converting project to use cpptools integration it did not always make the settings.json file.
I assume that CMT when using cpptools integration should always have  the following settings file?

I ended up copying the settings.json file to the .vscode folder and everything worked OK.
Thanks

I'll update the docs as noted.
When you configure your project with CMake, you should have a pop-up message from cpptools asking if you want to enable the integration. Confirming this will add the noted settings to the settings.json file without any manual intervention.


Revert "add greenwave test check in prepare release"

Reverts openshift/aos-cd-jobs#3485
Needs https://github.com/openshift/elliott/pull/503 to merge first

/lgtm


You should update that readme file. There are lots of typos and errors in it.

Package Name: any
Package Version: any
Operating System:any
Python Version: any

Describe the bug
There are a lot of typos in this doc and it is difficult to get through. https://github.com/Azure/azure-sdk-for-python/tree/master/sdk/digitaltwins/azure-digitaltwins-core
Here are the things I have noticed so far:

MSFT is aware of a typo in the code and is apparently considering potential fixes.
You do not have to load the client secret into your environment variables. Skip all that and use the AzureCliCredential if you are developing locally and not building an app.
There are no import statements in the code examples. You will have to figure out where those methods are on your own. Some things are from other packages such as json and uuid.
serviceClient and service_client are used interchangably as well as some other naming issues.
The doc references the query language that is at review.docs (and inaccesible to non MSFT employees), Here is the public doc.

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Hello Bill,
Thank you for the review!
I am checking in the fixes for your comment except for the followings:

I don't think using the import statements in the Readme file is appropriate. We are providing code snippets in the Readme and not runnable code. Also in the Readme we stated clearly which sample is the snippet from. Obviously if you want to start coding you should open the related sample file and you can find everything there. All the samples are runnable after you set up the environment variables.
We can debate what approach we should prefer in the samples but we choose the environment variable setup. I think that is the most convenient in all cases but especially if you are using VSCode.

Thanks,
Zoltan

Hey, thanks Zoltan,
My two cents (but it's totally an opinion):

I think the import statements show the user from which imported element. Generally it is common to show the import statements (for example the azureml sdk always shows import statements). It's just saves time for the person who is following along with your examples. Part of the Readme is showing how the methods are organized in the SDK.
environment variables require you to hard code access keys into flat files on your local machines, using AD login is more secure and allows MFA. It also requires a lot less setup for people using the code for the first time.

Moreover, thanks for this SDK! It is the gateway to ADT for data scientists.


Auto clean null values

I am updating from version 1.7 and the main problem I am running into is that I was dependent on "null" being automatically removed up on an insert/update and that no longer happens.
Is there a way to extend the clean for all my collections so that I don't have to change my code everywhere to fix that?
I couldn't find anything obvious in the api.

I don't have the time to make a full change with tests and stuff but I think it would be great if this packages supported something like this
https://github.com/ToyboxZach/simpl-schema/blob/374971786e0a35582e339f4aa00a8d7decfa996e/src/clean.ts#L82


pyenvutils build with flit

https://github.com/dougransom/pyenvutils build primarily with flit and a tiny amount of python, to provide information about the  python environment.  i think could be generally useful for those who live in a shell like bash, cmd, or powershell.  I am not sure how to get the word out.  I use it on some projects to push files to the site-packages folder, which could change depending on the python environment.
As an example, pye_executable
would produce something like:
c:\python38-32\python.exe
It is in pypi  but I am not sure how to get some more interest.

Looks neat! Maybe you could try posting on the Python discourse forum, or sites like the Python subreddit, or tweeting about it. :slightly_smiling_face:


Create Project “virtual-tour-museum-kotagede”

Automatically generated by Netlify CMS

👷 Deploy Preview for ibnuraffi processing.

Name
Link

🔨 Latest commit
a1c58bc011902043afda89b04678de7916843d65

🔍 Latest deploy log
https://app.netlify.com/sites/ibnuraffi/deploys/6266c5ba0bf36100087353eb


Cannot edit geometry

Context
In organization users, using the builder, you cannot edit geometries, and the table view in the builder is readonly, due to a conflict with query defaults.
Steps to Reproduce

In an org user, create a map
Go to the layer, and try to edit geometry

Current Result
Readonly (query not default)
Expected result
Be able to edit it
Additional info
Likely related to #10510

Can we prioritize this?

Only waiting for CR, we'll probably deploy tomorrow :)


Apple-Music-Electron hijacked my iTunes !

Describe the bug
Since I ran Apple-Music-Setup-2.5.0.exe, the application auto-installed into a predetermined location and ever since then, I cannot use my regular iTunes. The "For You", "Browse" and "Radio" sections in iTunes no longer load any content! I have uninstalled Apple-Music-Setup, I have reinstalled iTunes, I have rebooted the PC, I have even attempted to use system restore (although that seems to have failed) and I just can't get those three sections back in my iTunes!
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment Information

OS * Windows 7 64-bit Ultimate
Application Version (e.g. 2.5.0):

Additional context
Add any other context about the problem here.
Have you followed the troubleshooting guide and checked for any outstanding/pinned issues that can help?
Troubleshooting guide can be found here.
Yes/No
Log File
Follow this to find your log file.
Attach your log file to the post but make sure to remove any LastFM keys that are in the file.

Wat? We embed the music.apple.com site. This might be a network issue on your end.


AppService: Changing AppSettings after initial launch throws error

Set up:
Use one of these azure app service examples:
(Although probably any of the azure-XX-appservice examples will show the same issue.)

azure-cs-appservice, or
azure-ts-appservice

Launch the stack.

Go to Azure console, select the resource group that was created by the stack, select the "App Service" that was created by the stack.
In the console panel along the left, find Settings/Configuration and select that.
In the Configuration view, you will see a section named "Application settings" and in that section you will see the 4 settings created by the pulumi code. (e.g. APPINSIGHTS_INSTRUMENTATIONKEY, APPLICATIONINSIGHTS_CONNECTION_STRING, etc).

How to cause the error

Go to the pulumi code and add an AppSetting, it can be any key:value pair, so for example add
MY_NEW_SETTING: "new setting
pulumi up to apply the change

This will cause an error like that shown at the bottom.
However, if you refresh the azure console for the App Service (click the refresh button and the continue button in the console), you will see the new setting was added.
Furthermore, if you had made other changes as part of the update, and those changes were not executed before this error, those changes are not made.
Any pulumi up from this point forward will fail until you do a pulumi refresh.
WORK AROUND
Run pulumi refresh after the error.
This will synchronize the AppSettings and allow subsequent updates (for non-AppSettings) to work.
Subsequent AppSettings changes will, naturally, throw the error on pulumi up and will require another pulumi refresh
pulumi up error after changing AppSettings
Diagnostics:
  azure:appservice:AppService (mitchts-as):
    error: transport is closing

  pulumi:pulumi:Stack (azure-appservice-dev):
    panic: interface conversion: interface {} is nil, not map[string]interface {}
    goroutine 48 [running]:
    github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/web.expandAppServiceLogs(0x5310fe0, 0xc0014cb3e0, 0x4, 0x5310fe0, 0xc0014cb3e0, 0x1)
        /home/runner/go/pkg/mod/github.com/pulumi/terraform-provider-azurerm@v1.41.1-0.20201016142117-857b1cde95f3/azurerm/internal/services/web/app_service.go:1382 +0xcad
    github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/web.resourceArmAppServiceUpdate(0xc000a76770, 0x549e2c0, 0xc000164b00, 0x0, 0x0)
        /home/runner/go/pkg/mod/github.com/pulumi/terraform-provider-azurerm@v1.41.1-0.20201016142117-857b1cde95f3/azurerm/internal/services/web/resource_arm_app_service.go:487 +0x1c40
    github.com/hashicorp/terraform-plugin-sdk/helper/schema.(*Resource).Apply(0xc0006e9950, 0xc0005e8550, 0xc0015be300, 0x549e2c0, 0xc000164b00, 0x1, 0xffffffffffffffff, 0x0)
        /home/runner/go/pkg/mod/github.com/hashicorp/terraform-plugin-sdk@v1.13.1/helper/schema/resource.go:316 +0x263
    github.com/hashicorp/terraform-plugin-sdk/helper/schema.(*Provider).Apply(0xc00019cd80, 0xc0019375e0, 0xc0005e8550, 0xc0015be300, 0x0, 0x0, 0x0)
        /home/runner/go/pkg/mod/github.com/hashicorp/terraform-plugin-sdk@v1.13.1/helper/schema/provider.go:294 +0x99
    github.com/pulumi/pulumi-terraform-bridge/v2/pkg/tfshim/sdk-v1.v1Provider.Apply(0xc00019cd80, 0x5c05520, 0x13, 0x65cddc0, 0xc0005e8550, 0x65e8be0, 0xc0015be300, 0x0, 0x65e8be0, 0xc0015be300, ...)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/v2@v2.11.0/pkg/tfshim/sdk-v1/provider.go:97 +0xe5
    github.com/pulumi/pulumi-terraform-bridge/v2/pkg/tfbridge.(*Provider).Update(0xc000a4e380, 0x65ca540, 0xc0008c0930, 0xc0011a2000, 0xc000a4e380, 0x550e201, 0xc000bea300)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/v2@v2.11.0/pkg/tfbridge/provider.go:957 +0x959
    github.com/pulumi/pulumi/sdk/v2/proto/go._ResourceProvider_Update_Handler.func1(0x65ca540, 0xc0008c0930, 0x5911d40, 0xc0011a2000, 0x58f92e0, 0xa074d00, 0x65ca540, 0xc0008c0930)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi/sdk/v2@v2.11.3-0.20201009201355-249140242ebb/proto/go/provider.pb.go:2280 +0x89
    github.com/grpc-ecosystem/grpc-opentracing/go/otgrpc.OpenTracingServerInterceptor.func1(0x65ca540, 0xc0008c0000, 0x5911d40, 0xc0011a2000, 0xc001926240, 0xc001926260, 0x0, 0x0, 0x654c7c0, 0xc00027f840)
        /home/runner/go/pkg/mod/github.com/grpc-ecosystem/grpc-opentracing@v0.0.0-20180507213350-8e809c8a8645/go/otgrpc/server.go:57 +0x2eb
    github.com/pulumi/pulumi/sdk/v2/proto/go._ResourceProvider_Update_Handler(0x59c8e80, 0xc000a4e380, 0x65ca540, 0xc0008c0000, 0xc00141e000, 0xc000549760, 0x65ca540, 0xc0008c0000, 0xc0000bc600, 0x12b5)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi/sdk/v2@v2.11.3-0.20201009201355-249140242ebb/proto/go/provider.pb.go:2282 +0x14b
    google.golang.org/grpc.(*Server).processUnaryRPC(0xc000a4e1c0, 0x6611320, 0xc000782900, 0xc0001aa100, 0xc001132c90, 0xa0378b8, 0x0, 0x0, 0x0)
        /home/runner/go/pkg/mod/google.golang.org/grpc@v1.30.0/server.go:1171 +0x50a
    google.golang.org/grpc.(*Server).handleStream(0xc000a4e1c0, 0x6611320, 0xc000782900, 0xc0001aa100, 0x0)
        /home/runner/go/pkg/mod/google.golang.org/grpc@v1.30.0/server.go:1494 +0xccd
    google.golang.org/grpc.(*Server).serveStreams.func1.2(0xc000dcc050, 0xc000a4e1c0, 0x6611320, 0xc000782900, 0xc0001aa100)
        /home/runner/go/pkg/mod/google.golang.org/grpc@v1.30.0/server.go:834 +0xa1
    created by google.golang.org/grpc.(*Server).serveStreams.func1
        /home/runner/go/pkg/mod/google.golang.org/grpc@v1.30.0/server.go:832 +0x204

    error: update failed

This error is killing me 😭 Even a refresh doesn't help. I'm pretty much stuck.
Diagnostics:
  pulumi:pulumi:Stack (srm-dev):
    panic: interface conversion: interface {} is nil, not map[string]interface {}
    goroutine 449 [running]:
    github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/web.expandAppServiceLogs(0x47c1120, 0xc001e41260, 0x4, 0x47c1120, 0xc001e41260, 0x1)
        /home/runner/go/pkg/mod/github.com/pulumi/terraform-provider-azurerm@v1.41.1-0.20201016142117-857b1cde95f3/azurerm/internal/services/web/app_service.go:1382 +0xcb4
    github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/web.resourceArmAppServiceUpdate(0xc000d72770, 0x494d740, 0xc0002c8580, 0x0, 0x0)
        /home/runner/go/pkg/mod/github.com/pulumi/terraform-provider-azurerm@v1.41.1-0.20201016142117-857b1cde95f3/azurerm/internal/services/web/resource_arm_app_service.go:487 +0x1c47
    github.com/hashicorp/terraform-plugin-sdk/helper/schema.(*Resource).Apply(0xc001573560, 0xc0007bf180, 0xc00190b940, 0x494d740, 0xc0002c8580, 0x1, 0x0, 0x0)
        /home/runner/go/pkg/mod/github.com/hashicorp/terraform-plugin-sdk@v1.13.1/helper/schema/resource.go:316 +0x26a
    github.com/hashicorp/terraform-plugin-sdk/helper/schema.(*Provider).Apply(0xc000718c00, 0xc0022175e8, 0xc0007bf180, 0xc00190b940, 0x0, 0x0, 0x0)
        /home/runner/go/pkg/mod/github.com/hashicorp/terraform-plugin-sdk@v1.13.1/helper/schema/provider.go:294 +0xa0
    github.com/pulumi/pulumi-terraform-bridge/v2/pkg/tfshim/sdk-v1.v1Provider.Apply(0xc000718c00, 0x50b3c6a, 0x13, 0x5a6b320, 0xc001f26830, 0x5aada80, 0xc00190b940, 0x0, 0x5aada80, 0xc00190b940, ...)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/v2@v2.11.1-0.20201020163502-64cff1e50894/pkg/tfshim/sdk-v1/provider.go:97 +0x11c
    github.com/pulumi/pulumi-terraform-bridge/v2/pkg/tfbridge.(*Provider).Update(0xc0009d6380, 0x5a67aa0, 0xc001bc6060, 0xc000396c80, 0xc0009d6380, 0x49bcf01, 0xc0018ded00)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/v2@v2.11.1-0.20201020163502-64cff1e50894/pkg/tfbridge/provider.go:970 +0x96f
    github.com/pulumi/pulumi/sdk/v2/proto/go._ResourceProvider_Update_Handler.func1(0x5a67aa0, 0xc001bc6060, 0x4dbf7e0, 0xc000396c80, 0x4da7140, 0x9521b20, 0x5a67aa0, 0xc001bc6060)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi/sdk/v2@v2.12.0/proto/go/provider.pb.go:2280 +0x90
    github.com/grpc-ecosystem/grpc-opentracing/go/otgrpc.OpenTracingServerInterceptor.func1(0x5a67aa0, 0xc001c6b830, 0x4dbf7e0, 0xc000396c80, 0xc000c4b600, 0xc000c4b620, 0x0, 0x0, 0x59e9a80, 0xc0001cd780)
        /home/runner/go/pkg/mod/github.com/grpc-ecosystem/grpc-opentracing@v0.0.0-20180507213350-8e809c8a8645/go/otgrpc/server.go:57 +0x2f2
    github.com/pulumi/pulumi/sdk/v2/proto/go._ResourceProvider_Update_Handler(0x4e76920, 0xc0009d6380, 0x5a67aa0, 0xc001c6b830, 0xc0007bc3c0, 0xc000c10200, 0x5a67aa0, 0xc001c6b830, 0xc000213500, 0x1360)
        /home/runner/go/pkg/mod/github.com/pulumi/pulumi/sdk/v2@v2.12.0/proto/go/provider.pb.go:2282 +0x152
    google.golang.org/grpc.(*Server).processUnaryRPC(0xc0009d61c0, 0x5aaefe0, 0xc001702480, 0xc00005f800, 0xc0009e0b40, 0x94e17f8, 0x0, 0x0, 0x0)
        /home/runner/go/pkg/mod/google.golang.org/grpc@v1.30.0/server.go:1171 +0x511
    google.golang.org/grpc.(*Server).handleStream(0xc0009d61c0, 0x5aaefe0, 0xc001702480, 0xc00005f800, 0x0)
        /home/runner/go/pkg/mod/google.golang.org/grpc@v1.30.0/server.go:1494 +0xcd4
    google.golang.org/grpc.(*Server).serveStreams.func1.2(0xc000de3cf0, 0xc0009d61c0, 0x5aaefe0, 0xc001702480, 0xc00005f800)
        /home/runner/go/pkg/mod/google.golang.org/grpc@v1.30.0/server.go:834 +0xa8
    created by google.golang.org/grpc.(*Server).serveStreams.func1
        /home/runner/go/pkg/mod/google.golang.org/grpc@v1.30.0/server.go:832 +0x20b

    error: update failed

  azure:compute:LinuxVirtualMachine (srm-dev-docker-vm):
    error: transport is closing

  azure:appservice:AppService (srm-dev-app):
    error: transport is closing

Is this the same issue as these? The stack looks pretty similar... https://github.com/pulumi/pulumi-azure/issues/383 https://github.com/pulumi/pulumi-azure/issues/454
BTW, I'm using Pulumi v2.12.1 and Pulumi.Azure 3.27.0.

Huh, just saw there was a new 3.28.0 version released 4 days ago ✨ I tried updating and it looks like it magically solved everything 😅🤷♂

It also fails for me:
Callstack: https://pastebin.com/TGWDP7bG
Versions:
 <PackageReference Include="Pulumi.Azure" Version="3.30.0" />
 <PackageReference Include="Pulumi.AzureAD" Version="2.6.0" />

Pulumi cli: v2.13.2

For the web.expandAppServiceLogs panic, the simplest workaround is to specify the httplogs setting:
ie, for C#:
               Logs = new AppServiceLogsArgs
                {
                    HttpLogs = new AppServiceLogsHttpLogsArgs
                    {
                        FileSystem = new AppServiceLogsHttpLogsFileSystemArgs { RetentionInDays = 14, RetentionInMb = 35 }
                    }
                }

This problem has been around for a long time now and really needs to be fixed.

@Hawxy Thank you for providing this workaround.  This is a great way to buy some time before this issue is resolved without needing to resort to a temporary downgrade of the @pulumi/azure package.

Is any fix planned for this issue?
At the moment we are waiting for a fix several weeks now. As a workaround we are thinking of setting app settings using the Azure CLI after the pulumi script as part of our CI/CD pipeline. But we would rather avoid it.

Is any fix planned for this issue?
At the moment we are waiting for a fix several weeks now. As a workaround we are thinking of setting app settings using the Azure CLI after the pulumi script as part of our CI/CD pipeline. But we would rather avoid it.

@bascoder doesn't the workaround above work for you?

@bascoder doesn't the workaround above work for you?

@bascoder doesn't the workaround above work for you?

Do you mean the workaround from Hawxy?
I'm not just trying to configure app logs. But also a bunch of other app-specific custom settings.

@bascoder doesn't the workaround above work for you?

Do you mean the workaround from Hawxy?
I'm not just trying to configure app logs.

Do you mean the workaround from Hawxy?

yes

I'm not just trying to configure app logs.

My understanding is that if you set this property, you can proceed with updating all other settings too.

Do you mean the workaround from Hawxy?

yes

I'm not just trying to configure app logs.

My understanding is that if you set this property, you can proceed with updating all other settings too.

@bascoder Just need to add that Log property to your current AppService description. I did this to a project I'm working on and it works; weeks later, a colleague had this issue and we fixed it with the same workaround.

@bascoder Just need to add that Log property to your current AppService description. I did this to a project I'm working on and it works; weeks later, a colleague had this issue and we fixed it with the same workaround.

I've had to use the workaround to this problem too. Is this likely to get looked at at some point soon?

It should be noted that when a failure takes place, if a resource was created in that same process it will not be logged and but still exists in Azure.

It should be noted that when a failure takes place, if a resource was created in that same process it will not be logged and but still exists in Azure.

Does the Pulumi refresh bring your resource list back in sync at least?

I deleted them manually after realising, so no need to run that now.

Does the Pulumi refresh bring your resource list back in sync at least?

Just tested: No. Sadly not (and does not make the up-error disappear, either)

This issue has been fixed in the 4.0.0 version of the provider. Thanks to everyone for their patience and apologies it took so long.


get_status is not precise

get_status does check the status respectively process of an OUD instance. The grep command does also allow truncated strings. should be limited to the real OUD instance names.

add check for instance name. get_status now check's the OUDTAB before it check's the status. all grep's are using -w to check the word rather than the substring


Upgrade

Hello,
Please confirm that when performing Agent and Proxy upgrade, no need to re-run configuration.
For Proxy seems to be clear as it is written "Registration of the proxy service for password protection is necessary only once in the lifetime of the service".
But regards DC Agent, simply installing on top of existing too?

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

ID: 23e000f0-84ce-7ec7-e5ed-4bd9c49f5d49
Version Independent ID: e636b9d1-be81-0f00-97c4-ff4fa9078be3
Content: Deploy Azure AD password protection - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-password-ban-bad-on-premises-deploy.md
Service: active-directory
Sub-service: authentication
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@lightupdifire Thank you for your query . We will investigate and update the thread.

@shashishailaj, you can update article, we did installation on top and went fine.

Assigning this to @MicrosoftGuyJFlo to look into your feedback. Maybe the same sentence can be added in the DC Agent instructions?

@jay98014 Thoughts on this feedback?

Hi @lightupdifire,
Short answer to your question is that yes, you can just run the DC agent installer without uninstalling the old version, and it will do an in-place upgrade.
In the near future I will be adding a new doc section to document all of the various upgrade considerations.

@jay98014 ,
Thanks! Will update asap. If documentation planned, then i guess it is up to you to complete it ;)


Allow Neural Networks to reuse parameters

Feature Request
Is your feature request related to a problem? Please describe.
The current neural network implementation in ASReview is training a new model from scratch every time a new record is labeled (this affects both the lab and simulation modes). This seems unnecessary - the model architecture is the same every time, and the goal of the model is the same too. The only change between consecutive models is the input data.
Describe the solution you'd like
The NN models in ASReview could be considered a transfer learning problem: the model weights of a previously trained model can be used to initialize the new model. The amount of epochs trained per model can then be reduced significantly. This could lead to an increase in performance (since the model weights will still be trained over more epochs eventually) and to a reduction in computing time required. Reducing computing time this way opens up options that might otherwise be too computationally expensive (like using a deeper network).
As far as I can tell this would not be easy to implement within the current structure of ASReview, and it would also require additional research to figure out how optimally implement this (including hyperparameter settings like the amount of trained epochs) and how it affects performance. For these reasons it's not possible for me personally to implement this as part of my thesis.
Teachability, Documentation, Adoption, Migration Strategy
This change does not affect how users interact with ASReview, though some documentation explaining how the neural network works could be useful.

I really like the idea. Such a mechanism would be very useful.
Not 100% sure if the results will be identical because of the batch size used in Neural Networks, but it will not affect the performance I guess (@bagheria ?).
I can't oversee the challenges of this request at the moment. The trained model needs to be saved on disk. Sounds not very complicated, although I don't know how much IO this requires. I'm open to ideas and prototypes.

Prototype: https://github.com/asreview/asreview/pull/636


ResourceWarning and lost functionality.

sys:1: ResourceWarning: unclosed <ssl.SSLSocket fd=7, family=AddressFamily.AF_INET, type=SocketKind.SOCK_STREAM, proto=6, laddr=('<IP_ADDRESS>', 52014), raddr=('<IP_ADDRESS>', 443)>
/usr/local/Cellar/python3/3.4.3/Frameworks/Python.framework/Versions/3.4/lib/python3.4/importlib/_bootstrap.py:2150: ImportWarning: sys.meta_path is empty

This is an error I got when i closed rtv, also some functions didn't seem to work like pressing i to check for unread messages.

I looked into the ResourceWarning and it seems to be a common issue on python 3. I'll see if I can silence it, but it shouldn't cause any negative side effects for the time being. My guess is that the other issues you're seeing are unrelated. Can you elaborate any further?

Fixed the ResourceWarning. 8eee962c6bc9fa1a740465ca105bdd82ce9b23cc


Principal unable to enable DyanmoDB auto scaling

Version information

DCE v0.23.0
Describe the bug
DCE IAM principal does not have permissions to enable auto scaling in DynamoDB.
If I try to create a table with auto scaling enabled from the web console, I see:

Relevant SO:
https://stackoverflow.com/questions/51236686/you-do-not-have-the-required-role-to-enable-auto-scaling
Looks like it's some special IAM role property, not part of the policy itself.

We've made some changes to include application-autoscaling:* in the policy (see pull requests #327 and #328). However, this specific informational message is a known issue when the AWS-generated role does not exist. See https://docs.aws.amazon.com/autoscaling/application/userguide/application-auto-scaling-service-linked-roles.html. The service-linked role is created automagically by AWS when you create a table for the first time. As such, this message only appears once per account so long as the service-linked role is not deleted.


Laravel 11.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 11.x.
Before merging, you need to:

Checkout the l11-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: Shift detected a dependency for the laravel/framework. Unless you have a special requirement for depending on the entire Laravel framework, it's recommended to add dependencies for the specific illuminate packages instead.

:warning: Shift detected GitHub Actions which run jobs using a version matrix. Shift attempted to update your configuration for Laravel 11. However, you should review these changes to ensure the desired combination of versions are built for your package.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/breadcrumbs.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "tabuna/breadcrumbs": "dev-l11-compatibility",
    }
}

Finally, run: composer update

@tabuna thanks for the merge! Would you mind tagging a new release with this compatibility?

@mbardelmeijer Done


Make relative path interpretation for markdown compatible with pandoc

Currently, the relative paths for images in markdown are interpreted differently in pandoc and VS Code, which limits the way for project organisation considerably. Therefore it would be great if VS Code would interpret:
![Image Caption](images/SomeImageFile.jpg)
as relative to the working directory like pandoc does and not relative to the directory where the current file is stored.

VSCode uses markdown-it which doesn't transform src attribute in any way, and changing default logic will be breaking change.
Most likely you'll need to create simple VSCode extension that adds markdown-it plugin that alters renderer for images.

Thanks @IllusionMH ,
but this would only be the last resort, as we want to run intermediate builds as well. We're multiple authors working on cloned GitHub repos, so interested to keep the relative path syntax in the files.


Add CI Jobs for Conda

Currently we are only testing our create_template_X workflows with Docker on Github Actions. This is easier since we know that CUDA is setup there and things will just sail smoothly.
I would like to evaluate whether runs with Conda also work on GA and if so to add a job for the create_mlflow_x workflows.
@Emiller88 this issue could be one for you?

So TLDR - won't work, right @Emiller88 ?
The I would close this for now and reopen when we have GPU CI somewhen

Yep I think you're right. There won't be the necessary stuff on the GitHub runners to use conda.


Updated message when redcarpet not installed.It's required to run tha…

Users should run redcarpet install command in sudo mode, otherwise will fail
hfli@CNhfli ~/Documents $ /usr/bin/gem install --user redcarpet pygments.rb
WARNING:  You don't have /Users/hfli/.gem/ruby/2.3.0/bin in your PATH,
	  gem executables will not run.
Building native extensions.  This could take a while...
ERROR:  Error installing redcarpet:
	ERROR: Failed to build gem native extension.

    current directory: /Users/hfli/.gem/ruby/2.3.0/gems/redcarpet-3.4.0/ext/redcarpet
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/bin/ruby -r ./siteconf20180206-4334-x7jsup.rb extconf.rb
mkmf.rb can't find header files for ruby at /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib/ruby/include/ruby.h

extconf failed, exit code 1

Gem files will remain installed in /Users/hfli/.gem/ruby/2.3.0/gems/redcarpet-3.4.0 for inspection.
Results logged to /Users/hfli/.gem/ruby/2.3.0/extensions/universal-darwin-17/2.3.0/redcarpet-3.4.0/gem_make.out
Successfully installed pygments.rb-1.2.1
Parsing documentation for pygments.rb-1.2.1
Done installing documentation for pygments.rb after 0 seconds
1 gem installed

when install in sudo mode:
hfli@CNhfli ~/Documents $ sudo /usr/bin/gem install --user redcarpet pygments.rb

Password:
Fetching: redcarpet-3.4.0.gem (100%)
WARNING:  You don't have /Users/hfli/.gem/ruby/2.3.0/bin in your PATH,
	  gem executables will not run.
Building native extensions.  This could take a while...
Successfully installed redcarpet-3.4.0
Parsing documentation for redcarpet-3.4.0
Installing ri documentation for redcarpet-3.4.0
Done installing documentation for redcarpet after 0 seconds
Fetching: pygments.rb-1.2.1.gem (100%)
Successfully installed pygments.rb-1.2.1
Parsing documentation for pygments.rb-1.2.1
Done installing documentation for pygments.rb after 0 seconds
2 gems installed

This is not correct; you don't need sudo to use --user. Your previous error was not a permissions error.


Update InstaScale Custom Resource to allow users to specific memory and cpu resources

Ted will be picking this one up

@anishasthana My understanding is, currently the InstaScale CRD does not allow specifying resources limit and requests for the InstaScale controller deployment. The asking is to update the CRD to include:
    resources:
      requests:
        memory: "64Mi"
        cpu: "250m"
      limits:
        memory: "128Mi"
        cpu: "500m"

and update the InstaScale controller accordingly.

Yep that's correct!

Hi, I'm having trouble with passing the k8s v1 core resourceRequirements struct object into the config manifest
https://github.com/project-codeflare/codeflare-operator/blob/28ac54c44e06cd8eb4e06453619466e5aee94644/controllers/instascale_controller.go#L53
It gives me this error when I try to map the yaml template from the resourceRequirements struct object into json.
2023-03-07T16:19:40-08:00	ERROR	Reconciler error	{"controller": "instascale", "controllerGroup": "codeflare.codeflare.dev", "controllerKind": "InstaScale", "InstaScale": {"name":"example","namespace":"default"}, "namespace": "default", "name": "example", "reconcileID": "bf55d3b9-2b22-4043-ad3e-5901b17ba011", "error": "error loading template yaml: error converting YAML to JSON: yaml: line 21: did not find expected alphabetic or numeric character"}

I'm wondering what's the right way to map struct type object into the deployment yaml template.

Here the current changes https://github.com/project-codeflare/codeflare-operator/pull/58


fix(NotificationPanel): add missing role

Closes #5448
Add missing a role to the Notification Panel.
What did you change?
Added role="dialog" to the panel and focus trap for it.
(Had to disable jsx-a11y/no-noninteractive-element-interactions for dialog element to allow the use of Escape key for closing the panel)
How did you test and verify your work?
Storybook

hi @sangeethababu9223 ,
few things i observed

When the dialog opens, the focus should automatically move to the dialog.(which announces to screenreader that the dialog is open)
when dialog is closed with escape key, the focus should return to the triggering button
closing with escape key isn't showing exit animation

the code looks good! one last thing though, can you provide some testing for this functionality?

Thanks @davidmenendez.
I've added test cases for esc key and focus trap.


Feature request: cast with seek

Feature request: a new --seek flag for catt cast, so that initialization of the cast, and setting the seek position to start playing from, can be done in a single command.

Yeah, that would be nice. It's on my todo list.

Can we tell the Chromecast to start from a specific timestamp, or do we have to do a play and a seek internally?

Can we tell the Chromecast to start from a specific timestamp

Yep, the default receiver supports this. As for YouTube, this was implemented in casttube recently, but the pychromecast controller hasn't been updated yet.


[WIP] test galaxykit command refactor

Just making sure https://github.com/ansible/galaxykit/pull/47 won't break our CI use of galaxykit.

green: https://github.com/ansible/ansible-hub-ui/runs/6553850868?check_suite_focus=true


Jarring restart

It's a bit jarring when the game is ended and then restarted (and I'm not sure that the refresh is very smooth at that point). I suggest a 5-10 second count-down before the next game is initialized.

I like that suggestion. I didn’t put much polish on it since it’s more of a demo for the article but what you suggest will be a nice and simple improvement. 👍 At the moment I’m vacationing with family and without laptop but I’ll add it when I’m back.

Also and maybe more importantly, if you sit on the board and just watch it seems like it stops getting updates esp at the end of the game. I was watching the websocket and it switches to only seeing pings and nothing else. I'd debug but I'm probably supposed to be doing actual $work right now haha
(Very fun demo)

I'm glad you're finding it fun! I'm actually now seeing quite a few things to improve now. Normally I would not launch something aday before going on vacation , but it was "just a demo". :sweat_smile:

@awwaiid added the countdown. :)
I couldn't figure out why ActionCable is not correctly listening to the updates on the new game after doing a JS refresh. I've now switched to a full page reload and noted it down for checking when I have time again. I'm interested to see what it is and if it's something to do with the Morphing refresh. We'll see.
Thanks for the suggestions!

It seems a lot better!!


GDScript Typed Autocompletion

Godot version:

3.1 master

Issue description:

Some collaborators have proposed a great idea from another I had to make it easier and faster to write code in GDScript, but that, however, could not be implemented. It was about adding a shortcut for typed nodes. You could not implement a shortcut that works in the analysis of the code, but you can do something similar only in the editor: Autocomplete the syntax of the already typed nodes. Then another idea came: Now that we have Typed GDScript, can we improve code autocompletion in more ways?

Autocompleting the types of the nodes in relation to the current scene
Autocompleting the resource preload
Other things that you can think of

And once this is done you have to think about how to implement it. An option in Project Settings that activates Typed Autocompleter? A special syntax? I would prefer the first, since it does not alter the existing ease of use.
Credits: @Zylann @vnen @OvermindDL1

#21189

Autocompleting types of virtual functions (arguments and return type)

This is already done and there's an editor setting for it. It also enable types in the signal connection functions that the editor generates.

Is this still an issue?

Indeed, features in this issue have been implemented. If you have other proposals to make, please open an issue on the Godot Proposals repository.


Is this project alive?

Just trying to figure out if I should keep it building / patching in Fedora or just retire and recommend people to use starship :)

Yes, it's alive and I'm using it daily, but I no longer do any development since it's feature complete for me.
Also my January and February were so crazy that I did not have any time to spend on my projects, that's why I'm replying right now.
I hope to look into Fedora builds and dependencies in coming days/weeks.
Thanks for taking care of it Fedora!


Importerror

I followd the instructions for install but there's import error from mmcv
The problem is as follows
python3.8/site-packages/mmcv/_ext.cpython-38-x86_64-linux-gnu.so: undefined symbol: _ZNK2at10TensorBase8data_ptrIdEEPT_v
I installed mmcv-full==1.4.0 and all the version of packages are same with the recommendation.

I followd the instructions for install but there's import error from mmcv The problem is as follows
python3.8/site-packages/mmcv/_ext.cpython-38-x86_64-linux-gnu.so: undefined symbol: _ZNK2at10TensorBase8data_ptrIdEEPT_v
I installed mmcv-full==1.4.0 and all the version of packages are same with the recommendation.

I met the same problem, how did you finally solve it? @oliver0922

I followd the instructions for install but there's import error from mmcv The problem is as follows
python3.8/site-packages/mmcv/_ext.cpython-38-x86_64-linux-gnu.so: undefined symbol: _ZNK2at10TensorBase8data_ptrIdEEPT_v
I installed mmcv-full==1.4.0 and all the version of packages are same with the recommendation.

I met the same problem, how did you finally solve it? @oliver0922

Problem was solved. It is caused by mismatch of cuda versions between mmcv and pytorch. My nvcc version is 11.7, so mmcv should be installed by precompiled wheel rather than compiled locally. In a nutsheel, the steps are:

pip uinstall mmcv-full
pip cache remove mmcv-full*
pip install mmcv-full==1.4.0 -f https://download.openmmlab.com/mmcv/dist/cu111/torch1.9.0/index.html
cd <PATH TO MMDET3D> and pip install -e . -v

so: undefined symbol:

Hi, so after re-install mmcv-full, the mmdet3d need to be recompiled again? I meet the same problem and I have finished all install steps but get the bug when importing the dataset.


Implement CMake/packaging support

This PR implements initial CMake support. Packaging with CPack is also available, and can be triggered with sudo make package

Thanks for the patch! I'm a bit unclear on what's the intention here. Do you want to get rid of Makefile and use CMake instead? Why is it needed, besides the packaging capability?

Thanks for the patch! I'm a bit unclear on what's the intention here. Do you want to get rid of Makefile and use CMake instead? Why is it needed, besides the packaging capability?

Hello @Dor1s!
My main goals were:

Add packaging support
Have a CMakeList.txt file that I can open with an IDE (QtCreator, CLion, etc)
Allow projects to add_subdirectory(afl) (where afl is a git submodule pointing here)
Add helpers so that from a CMake project a user can create afl targets

I can close the PR if this does not align with the guidelines/roadmap.
Thanks!

@alessandrogario thanks for the clarifications! This definitely makes sense, but I'd rather hold on for some time and consider doing this later as a planned effort (e.g. maybe with or after some other refactoring / code structure changes).
I'm sorry for rejecting this PR right now, but let me open an issue for CMake support and mention this as a starting point. Hopefully we (and/or you) can get back to this later :)


Razor server fails to launch with only .net9 installed

Copied from comment here - https://github.com/dotnet/vscode-csharp/issues/6980#issuecomment-2006418064
Looks like the Razor server does not roll forward (currently the dotnet runtime installer allows any dotnet greater than 8).
This issue also reproduces on the .NET9.0 SDK with VSCode v 1.87.2, C# Dev Kit v 1.4.29, C# v 2.22.5.
The 'Razor Logs' output window logs show as below:

Using dotnet configured on PATH
[Client - 3:15:55 PM] Razor language server path: /Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/.razor/rzls
[Client - 3:15:55 PM] Server arguments --logLevel 2 --projectConfigurationFileName project.razor.vscode.bin --DelegateToCSharpOnDiagnosticPublish true --UpdateBuffersForClosedDocuments true --telemetryLevel all --sessionId 3cf33dfc-3ae6-4b08-a51f-a57b103873fb1710832553170 --telemetryExtensionPath /Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/.razortelemetry/Microsoft.VisualStudio.DevKit.Razor.dll
[Client - 3:15:55 PM] Starting Razor Language Server...
You must install or update .NET to run this application.

App: /Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/.razor/rzls
Architecture: x64
Framework: 'Microsoft.NETCore.App', version '8.0.1' (x64)
.NET location: /usr/local/share/dotnet

The following frameworks were found:
  9.0.0-preview.3.24162.31 at [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]

Learn more:
https://aka.ms/dotnet/app-launch-failed

To install missing framework, download:
https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=8.0.1&arch=x64&rid=osx-x64&os=osx.12
[Error - 3:15:55 PM] Server initialization failed.
  Message: Pending response rejected since connection got disposed
  Code: -32097
[Info  - 3:15:55 PM] Connection to server got closed. Server will restart.
true
[Error - 3:15:55 PM] Razor Language Server client: couldn't create connection to server.
  Message: Pending response rejected since connection got disposed
  Code: -32097
You must install or update .NET to run this application.

App: /Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/.razor/rzls
Architecture: x64
Framework: 'Microsoft.NETCore.App', version '8.0.1' (x64)
.NET location: /usr/local/share/dotnet

The following frameworks were found:
  9.0.0-preview.3.24162.31 at [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]

Learn more:
https://aka.ms/dotnet/app-launch-failed

To install missing framework, download:
https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=8.0.1&arch=x64&rid=osx-x64&os=osx.12
[Error - 3:15:55 PM] Server initialization failed.
  Message: Pending response rejected since connection got disposed
  Code: -32097
[Info  - 3:15:55 PM] Connection to server got closed. Server will restart.
true
[Error - 3:15:55 PM] Razor Language Server client: couldn't create connection to server.
  Message: Pending response rejected since connection got disposed
  Code: -32097
[Client - 3:15:55 PM] (Error) Failed when activating Razor VSCode.
Pending response rejected since connection got disposed
Stack Trace:
Error: Pending response rejected since connection got disposed
  at Object.dispose (/Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/dist/extension.js:2:2098108)
  at Object.dispose (/Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/dist/extension.js:2:2152884)
  at S.handleConnectionClosed (/Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/dist/extension.js:2:2153290)
  at S.handleConnectionClosed (/Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/dist/extension.js:2:2311358)
  at /Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/dist/extension.js:2:2153033
  at i.invoke (/Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/dist/extension.js:2:2099821)
  at o.fire (/Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/dist/extension.js:2:2100586)
  at te (/Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/dist/extension.js:2:2086231)
  at i.invoke (/Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/dist/extension.js:2:2099821)
  at o.fire (/Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/dist/extension.js:2:2100586)
  at v.fireClose (/Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/dist/extension.js:2:2108439)
  at Socket.<anonymous> (/Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/dist/extension.js:2:2110033)
  at Socket.emit (node:events:526:35)
  at Pipe.<anonymous> (node:net:323:12)
You must install or update .NET to run this application.

App: /Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/.razor/rzls
Architecture: x64
Framework: 'Microsoft.NETCore.App', version '8.0.1' (x64)
.NET location: /usr/local/share/dotnet

The following frameworks were found:
  9.0.0-preview.3.24162.31 at [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]

Learn more:
https://aka.ms/dotnet/app-launch-failed

To install missing framework, download:
https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=8.0.1&arch=x64&rid=osx-x64&os=osx.12
[Error - 3:15:55 PM] Server initialization failed.
  Message: Cannot call write after a stream was destroyed
  Code: -32099
[Error - 3:15:55 PM] Razor Language Server client: couldn't create connection to server.
  Message: Cannot call write after a stream was destroyed
  Code: -32099
[Error - 3:15:55 PM] Restarting server failed
  Message: Cannot call write after a stream was destroyed
  Code: -32099
[Error - 3:15:55 PM] Restarting server failed
  Message: Cannot call write after a stream was destroyed
  Code: -32099
[Info  - 3:15:55 PM] Connection to server got closed. Server will restart.
true
You must install or update .NET to run this application.

App: /Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/.razor/rzls
Architecture: x64
Framework: 'Microsoft.NETCore.App', version '8.0.1' (x64)
.NET location: /usr/local/share/dotnet

The following frameworks were found:
  9.0.0-preview.3.24162.31 at [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]

Learn more:
https://aka.ms/dotnet/app-launch-failed

To install missing framework, download:
https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=8.0.1&arch=x64&rid=osx-x64&os=osx.12
[Error - 3:15:55 PM] Server initialization failed.
  Message: Pending response rejected since connection got disposed
  Code: -32097
[Info  - 3:15:55 PM] Connection to server got closed. Server will restart.
true
[Error - 3:15:55 PM] Razor Language Server client: couldn't create connection to server.
  Message: Pending response rejected since connection got disposed
  Code: -32097
You must install or update .NET to run this application.

App: /Users/vs/.vscode/extensions/ms-dotnettools.csharp-2.22.5-darwin-x64/.razor/rzls
Architecture: x64
Framework: 'Microsoft.NETCore.App', version '8.0.1' (x64)
.NET location: /usr/local/share/dotnet

The following frameworks were found:
  9.0.0-preview.3.24162.31 at [/usr/local/share/dotnet/shared/Microsoft.NETCore.App]

Learn more:
https://aka.ms/dotnet/app-launch-failed

To install missing framework, download:
https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=8.0.1&arch=x64&rid=osx-x64&os=osx.12
[Error - 3:15:55 PM] Server initialization failed.
  Message: Pending response rejected since connection got disposed
  Code: -32097
[Error - 3:15:55 PM] The Razor Language Server server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.
[Error - 3:15:55 PM] Razor Language Server client: couldn't create connection to server.
  Message: Pending response rejected since connection got disposed
  Code: -32097
[Error - 3:15:55 PM] Restarting server failed
  Message: Pending response rejected since connection got disposed
  Code: -32097
[Client - 3:16:04 PM] File '/Users/vs/RazorNoAuth/Pages/Privacy.cshtml' didn't exist in the Razor document list. This is likely because it's from outside the workspace.
[Client - 3:16:04 PM] (Warning) razor.provideDynamicFileInfo failed with CodeExpectedError: cannot open virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/Privacy.cshtml__virtual.cs. Detail: Unable to resolve resource virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/Privacy.cshtml__virtual.cs
[Client - 3:16:04 PM] File '/Users/vs/RazorNoAuth/Pages/Error.cshtml' didn't exist in the Razor document list. This is likely because it's from outside the workspace.
[Client - 3:16:04 PM] (Warning) razor.provideDynamicFileInfo failed with CodeExpectedError: cannot open virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/Error.cshtml__virtual.cs. Detail: Unable to resolve resource virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/Error.cshtml__virtual.cs
[Client - 3:16:04 PM] File '/Users/vs/RazorNoAuth/Pages/Shared/_ValidationScriptsPartial.cshtml' didn't exist in the Razor document list. This is likely because it's from outside the workspace.
[Client - 3:16:04 PM] (Warning) razor.provideDynamicFileInfo failed with CodeExpectedError: cannot open virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/Shared/_ValidationScriptsPartial.cshtml__virtual.cs. Detail: Unable to resolve resource virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/Shared/_ValidationScriptsPartial.cshtml__virtual.cs
[Client - 3:16:04 PM] File '/Users/vs/RazorNoAuth/Pages/_ViewStart.cshtml' didn't exist in the Razor document list. This is likely because it's from outside the workspace.
[Client - 3:16:04 PM] (Warning) razor.provideDynamicFileInfo failed with CodeExpectedError: cannot open virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/_ViewStart.cshtml__virtual.cs. Detail: Unable to resolve resource virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/_ViewStart.cshtml__virtual.cs
[Client - 3:16:04 PM] File '/Users/vs/RazorNoAuth/Pages/_ViewImports.cshtml' didn't exist in the Razor document list. This is likely because it's from outside the workspace.
[Client - 3:16:04 PM] (Warning) razor.provideDynamicFileInfo failed with CodeExpectedError: cannot open virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/_ViewImports.cshtml__virtual.cs. Detail: Unable to resolve resource virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/_ViewImports.cshtml__virtual.cs
[Client - 3:16:04 PM] File '/Users/vs/RazorNoAuth/Pages/Index.cshtml' didn't exist in the Razor document list. This is likely because it's from outside the workspace.
[Client - 3:16:04 PM] (Warning) razor.provideDynamicFileInfo failed with CodeExpectedError: cannot open virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/Index.cshtml__virtual.cs. Detail: Unable to resolve resource virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/Index.cshtml__virtual.cs
[Client - 3:16:04 PM] File '/Users/vs/RazorNoAuth/Pages/Shared/_Layout.cshtml' didn't exist in the Razor document list. This is likely because it's from outside the workspace.
[Client - 3:16:04 PM] (Warning) razor.provideDynamicFileInfo failed with CodeExpectedError: cannot open virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/Shared/_Layout.cshtml__virtual.cs. Detail: Unable to resolve resource virtualCSharp-razor:/Users/vs/RazorNoAuth/Pages/Shared/_Layout.cshtml__virtual.cs

@ryzngard is this fixed by https://github.com/dotnet/razor/pull/10064 ?

Yes, it should be

@dibarbet This issue still reproduces on the latest .NET 9.0 Preview 3 SDK (install from the https://github.com/dotnet/installer) 9.0.100-preview.3.24175.24 . No update version  for C# Dev Kit, C# and VSCode. Could you help me to take a look？

@dibarbet This issue still reproduces on the latest .NET 9.0 Preview 3 SDK (install from the https://github.com/dotnet/installer) 9.0.100-preview.3.24175.24 . No update version for C# Dev Kit, C# and VSCode. Could you help me to take a look？

@ryzngard was this merged into VSCode last week?  If it was it should be fixed in the prerelease we did this week.

Nevermind, I see it is in 2.23.14 - https://github.com/dotnet/vscode-csharp/releases/tag/v2.23.14
@v-Judy try the latest prerelease of the C# extension (2.23.14)

@v-Judy try the latest prerelease of the C# extension (2.23.14)

@dibarbet Thanks for your reply! I'm sorry so late to give the results. This issue does not repro on the latest version v2.23.15 for C# Extension.

@v-Judy do you mind checking the runtimesettings file on your machine? Locally for me it was located at ~\.vscode\extensions\ms-dotnettools.csharp-2.24.17-win32-x64\.razor\rzls.runtimeconfig.json (this is the latest prerelease I had installed) and the contents were as follows
{
  "runtimeOptions": {
    "tfm": "net8.0",
    "rollForward": "LatestMajor",
    "framework": {
      "name": "Microsoft.NETCore.App",
      "version": "8.0.0"
    },
    "configProperties": {
      "System.Reflection.Metadata.MetadataUpdater.IsSupported": false,
      "System.Runtime.Serialization.EnableUnsafeBinaryFormatterSerialization": false
    }
  }
}

The key is that the "version": "8.0.0" is present. If you have "version": "8.0.1" or any other value for "rollForward": "LatestMajor" then you still have an extension without the fix

They said it does not repro on the latest version, so I think we're all good :)

They said it does not repro on the latest version, so I think we're all good :)

omg... ☕ needed. It's been a long day already 🤣
Everything has been falling apart so I just assumed this too. Yay for a win!

@v-Judy do you mind checking the runtimesettings file on your machine?

The runtimesettings file on my machine contents were as follows:
{
  "runtimeOptions": {
    "tfm": "net8.0",
    "rollForward": "LatestMajor",
    "framework": {
      "name": "Microsoft.NETCore.App",
      "version": "8.0.1"
    },
    "configProperties": {
      "System.Reflection.Metadata.MetadataUpdater.IsSupported": false,
      "System.Runtime.Serialization.EnableUnsafeBinaryFormatterSerialization": false
    }
  }
}


⚠ Bofrost.be has degraded performance

In 503fbd4, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1886 ms

Resolved: Bofrost.be performance has improved in 5259dbb after 2 hours, 3 minutes.


issue #38 made manage address screen

Related Issue

Fixes #38 .

Proposed Changes

Made manage address screen

Checklist

[x] Tested
[x] No Conflicts
[x] Change In Code
[ ] Change In Documentation

https://user-images.githubusercontent.com/66346161/111302184-15cf3d00-8679-11eb-96a8-529cbbd93ad7.mp4

@anotherwebguy All looks good, but can you add a screen instead of directly going to add the address. Like, the screen displays the current addresses and there is a button to add a new one.

@anotherwebguy All looks good, but can you add a screen instead of directly going to add the address. Like, the screen displays the current addresses and there is a button to add a new one.

Okay i will do that @himanshusharma89

made the changes as you asked
https://user-images.githubusercontent.com/66346161/111420310-fedc2980-8710-11eb-9846-f6ea9414b893.mp4


blocking_queue.cpp:50] Data layer prefetch queue empty with Digits 2.0

Hi,
My system:

Cuda 7.5
CudNN V3
2x Titan X's

I am transitioning from Caffe master from BVLC to Digits mainly for the ease of use.

When I train the reference_caffenet model with Imagenet 1000 using Caffe master (1 Titan X), training 20 iterations take about 6sec to 6.5 seconds.
I used the web installer of digits (which I believe comes with Cuda 7.0 and cudnnv3). If I start training on the same reference_caffenet model with Digits (1 Titan X), training 20 iterations takes about double the time (13 to 14 seconds). I also notice a lot of log messages like the one below:

18:29:17.241222 11321 blocking_queue.cpp:50] Data layer prefetch queue empty
I tried recompiling NVCaffe with USE_CNMEM := 0 and USE_CNMEM := 1 (as that was the only difference between NVidia's Caffe and BVLC's Caffe makefiles) but there was no difference in performance.
Any tips on how to debug this? I really want to be able to use Digits with 2 GPUs. I don't see the above problem  with the Caffe master from BVLC.
Siddharth

Filed at https://github.com/NVIDIA/caffe/issues/55.

I slove the problem by changing the batches number to 16 from 32


ordered pipeline exec

Changed to do ordered pipeline exec. Needed so that schema inference is always executed first - that way last step status can be put in resp. Without this, it is possible that schema inference pipeline gets executed first and so would cause the incorrect status set on the response.
Given that a lot of changes need to occur on the server - maybe there's a better way to do this.
Need to discuss with @blinktag

COPY/PASTE:
Arrived at the conclusion that we need to establish pipeline order on server. Without that, we will never be able to 100% say that X should execute before Y - at best, they are "it will probably happen".
Closing this PR in favor of the "pipeline refactor" PR.