fix($compile): turn link[href] into a RESOURCE_URL context.

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix.
What is the current behavior? (You can also link to an open issue here)
There's no $sce context for link[href], meaning a  tag will have no $sce protection (might have link sanitization though, I'm not sure of that). That's fine for several rel types, but at least stylesheet and import can run script in your origin, which corresponds to a RESOURCE_URL context.
What is the new behavior (if this is a feature change)?
Turn it into a RESOURCE_URL context. Note that this is an overcorrection, sadly, but the security level of this attribute depends on the rel attribute, and I don't know if we can make context-aware decisions in this type of situation.
RESOURCE_URLs are allowed to be interpolated if the value is $sce-blessed, or for plain strings if they're from the same origin.
Angular2 is being fixed to do the same with https://github.com/angular/angular/pull/8869/files#diff-5ffbf51559339aaa2db503c37cc3a658R60.
Does this PR introduce a breaking change?
Most apps keep the default whitelist, so it will break apps with dynamically-set link[href]s pointing to other origins.
Please check if the PR fulfills these requirements

[X] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[X] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

No comprehensive list of contexts in the docs to update, they just say the $sce.RESOURCE_URL type is "For URLs that are not only safe to follow as links, but whose contents are also safe to include in your application. Examples include ... ".

@mprobst can you give a view on this - since you are working on the A2 version?

Yes, this is correct. Thanks for the fix Raphael!

Isn't this a BC ?

This is a breaking change. LGTMs though.

LGTM2 - I'm just saying we should update the commit message with a BC notice (including instructions on how to update old code).


✏ Style Edits to the Home Page

🫤 Description

I'm offering a style correction, not a typo issue. There are no problems with the grammar from what I saw, but some style issues to make it sound better for the below lines.

📍 Location

File name: /index.html

Line number: 17, 33
URL: https://divyanshudhruv.github.io/Minifolio/

🧪 Correction

Line 17: "Learn more"
Line 33: I design websites. That's the long and short of it. I design UI and UX with Sketch, Design, and pretty decent HTML and CSS, while dabbling with Javascript and Java.

📦 Additional Information

Add any other relevant information here.

📌 Related Issue

Link to other issues related to this problem

I also have another question. I don't really do frontend design. Are "sketch" and "design" tools that people use to design websites? If so, it might be useful to link or bold them so they stand out more. Design especially makes me just think of the verb "to design".

Yes, true. Thank you for noticing the error. If you want, you can replace the error. But the correction in line:33 doesn't sound suitable. Try to make more better correction to the sentence . If you want may I assign the work to you ? You can contribute(open pull request after 1 oct for hacktoberfest) during hacktoberfest.

Yes, assign it to me! I'll think about how to reword line:33. Right now I'm thinking it needs more active verbs (i.e. from this list).

Ok , Assigning to you... Good luck

hi, i'd like to participate in fixing this issue (hacktoberfest)

@vcavanna  ?

Hello, I would like to help contribute to your project in this issue.

Hi @divyanshudhruv it'll be done end of day


Getting raw depth values that are aligned to color

|---------------------------------|------------------------------------------- |
| Camera Model                       |  D415 |
| Operating System & Version |   MacOS  |
| Platform                                 | PC |
| Language                            |  C#            |
Issue Description
Hello,
I am using the align-advance sample and am trying to get the depth values that correspond to each pixel but it seems that there is misalignment. How do I get the depth values for each pixel? (I don't need to save it in .csv or anything by the way, I just need to use this data for another process).
Thank you!

As you can see, even after using the align-advanced sample, there seems to be a misalignment. You can see that there is a bigger space between my thumb and the edge of the screen on the color image versus the depth. Is there a way to properly align them and get the depth values?

@aalarcon96 hello, at closer examination it is possible that you misinterpret the data due to the blend of the overlayed depth with the background RGB image, and do not account for the blank lines at the bottom of depth image, which are integral part of if (the black strip).

@aalarcon96 Any luck to get it through with the suggestion of ev-mp? Looking forward to your update. Thanks!

Ev-mp was right, thank you for following up.


Set remoteHost and remoteAddr on Coyote request

Related: #81
When TomcatServiceInvocationHandler converts a Netty HTTP request to a
Coyote request, it should set the remoteAddr() and remoteHost() property
of the Coyote request.

@yfinkelstein Could you please confirm this PR fixes your problem?

Hmm thought I had replied to this but don't see my comment, let me try again - just wondering can we add unit tests for this or too hard to mock?

@anuraaga I can write a test case for this using my JSP-fu for sure. Stay tuned.

@anuraaga Updated.


Next Permutation Problem

Hello my name is Shailesh Yogendra and I want to implement this question in C++ . I am a competitive coder
Question name = Next Permutation.
Please assign me this task
@tarun26091999  @rudrakshi99  @plazzy99 @nakul-19

Hey @rudrakshi99 @tarun26091999
I would like to do it in Java.
Please assign me this issue.

Hello @tarun26091999 @dheerajkotwani @esheetaparulekar @anubhavitis @jeelnathani, I think this is an important question and this should definitely be added in the Algo-Tree and I will start working on it as soon as I get assigned!! as I am a part of GSSoC'21, so I would like to implement this in c++.

@jagruthi13 @snowpanther6 do it for array.


--counter does not work with Facebook

Description
I made a native wrapper for facebook.com with the --counter option. The titles of Facebook have (X) Facebook, but nothing happened when I receive a notification.
Steps to Reproduce Issue
$ nativefier --counter --disable-dev-tools --name "Facebook" "https://www.facebook.com"

Specifications

Version of Nativefier: v7.0.1
Version of Node.js: v6.7.0
OS: MacOs Sierra 10.12.1
Architecture x64

I have the same problem ! Do you fix it ?

Hey @kcampion @nansthomas, https://github.com/jiahaog/nativefier/pull/256 was recently release as part of Nativefier 7.1.0, could you try it and see if --counter works now? Thanks :)

The problem seems to be always here @ronjouch after updating Nativefier and regenerated the app.

Closing as I have not heard a response back. We can reopen if anyone is still experiencing this.


Expose WMS Layer and WFS FeatureType MetadataURLs

Applications may want to provide links to available metadata of layers and feature types shown. For this to be possible it is necessary to expose the relevant elements in the API.
Note that some changes to test fixtures were required in order to harmonise the MetadataURL elements included in different WFS versions.

@jahow I've rebased this one on to the current state of main to deal with the conflicts

All good!


New release including Spectrum class

I've found the new Spectrum class to be super useful, but right now I can only use it if I build sncosmo from source or pip install from Github. It'd be nice to have it installable through the standard pip/conda channels, as well as to be able to indicate a dependency on this version down the line.
Can we get a new release that includes the rewritten Spectrum class?

sncosmo v2.2.0 was just released and contains those commits. It is on pypi now and should be on conda shortly.


Expose public Device interface

The first commit removes the __exit__ method from Device which has no use without the corresponding __enter__ method anyway (see https://www.python.org/dev/peps/pep-0343/). If this was intended to be a destructor, it should be called __del__, but that is not needed either, since default destructor should delete the file cache and close the file handles automatically.
The second commit exposes methods of the Device class as public, and makes the names compatible with boostc version of python bindings. If you don't like the new method names, we can discuss it, but I think it should be possible for the user to use raw Device class (hence, the methods should definitely be public).
The commit also gets rid of leading double underscores in private member names. See https://www.python.org/dev/peps/pep-0008/#method-names-and-instance-variables.

I also found a bug in my initial implementation of Device._matches(). ef8c019 fixes that.

Thanks!


[BUG] Failed to load data files with ITK when using Clara Train docker image

Describe the bug
Loading data files with the DataDefinition class which is using ITK fails when using Clara Train docker image.
Steps/Code to reproduce bug
/clara-viz/notebooks$ docker run --rm -it --network host -e NVIDIA_DRIVER_CAPABILITIES=graphics,video,compute,utility -v $PWD:/notebooks nvcr.io/nvidia/clara-train-sdk:v4.0
...
# pip3 install clara-viz
# cd /notebooks
# python3
>>> from clara.viz.core import DataDefintion
>>> d = DataDefinition()
>>>d.append('data/syn3193805/img0066.nii.gz', 'DXYZ')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/opt/conda/lib/python3.8/site-packages/clara/viz/core/datadefinition.py", line 92, in append
    itk.CommonEnums.IOComponent_FLOAT: itk.F,
  File "/opt/conda/lib/python3.8/site-packages/clara/viz/core/importutil.py", line 154, in __getattr__
    raise self._exception
  File "/opt/conda/lib/python3.8/site-packages/clara/viz/core/importutil.py", line 118, in optional_import
    raise AssertionError
clara.viz.core.importutil.OptionalImportError: import itk.

For details about installing the optional dependencies, please visit:
    https://docs.monai.io/en/latest/installation.html#installing-the-recommended-dependencies

Expected behavior
File should be loaded without error
Environment details (please complete the following information)

OS/Platform: Linux
Python Version: 3.8.5
Clara Viz Version: 0.1.2

Additional context

Fixed with 0.1.3.


shift select

Would be useful to select/unselect a whole range by clicking one checkbox as the start and hold shift when clicking on the end checkbox. The start can also be below the end.
This is commonly used in e.g. gmail, github, excel

I didn't make that before, maybe I'll get some help from someone interested but first I need to change the webview html management.

With v1.4.0 you should select rangee with Shift+Click 👍🏻


Examples crash under Mac OS X

This is a pristine checkout on e8bb880c42df99a095e19a16912a77dd5e11fe0d (latest master). No other audio programs are running right now.
$ cargo run --release --example basic
    Finished release [optimized] target(s) in 0.0 secs
     Running `target/release/examples/basic`
-> exit code 1

$ target/release/examples/basic
zsh: segmentation fault (core dumped)  target/release/examples/basic

This is a MacBookPro Retina 15 Inch, Early 2013 with Intel Core i7 running OS X El Capitan.

Thanks for the report, but please open an issue for cpal instead: https://github.com/tomaka/cpal
Since rodio doesn't contain any unsafe code, it can't be the cause.

Done: https://github.com/tomaka/cpal/issues/148
CPAL does not crash though (see the bug report). But it also does not play a sound. Maybe CPALs interface does not fulfill the safety guarantees/is unsound?


feat: Extract initContainer.udf.command as values from clickhouse chart

Closes #431

@Sinhyeok Thanks for the PR 🎉

@Sinhyeok Thanks for the PR 🎉

Welcome!

@prashant-shahi
Could you please distribute the clickhouse chart to reflect this fix, the current version is v23.13.0 which was released 4 months ago.


"getHostByRequestOptions" assumes "host" and "hostname" are the same thing

That's incorrect and given a host (hostname + port), it then concludes that it's an IPv6 address and tries to wrap it in brackets:
new URL('/resource', 'http://[<IP_ADDRESS>:1234]')

Which results in an invalid URL being constructed.
The root cause for this is here:
https://github.com/mswjs/interceptors/blob/beb1561770403e6175e24e7948da53418cf50ddc/src/utils/getUrlByRequestOptions.ts#L60-L62
host must be removed from here.

Released: v0.22.3 🎉
This has been released in v0.22.3!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/interceptors@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


neonavigation: 0.10.9-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository neonavigation to 0.10.9-1:

upstream repository: https://github.com/at-wat/neonavigation.git
release repository: https://github.com/at-wat/neonavigation-release.git
distro file: melodic/distribution.yaml
bloom version: 0.10.2
previous version for package: 0.10.8-1

costmap_cspace

No changes

joystick_interrupt

No changes

map_organizer

No changes

neonavigation

No changes

neonavigation_common

No changes

neonavigation_launch
* neonavigation_launch: add isolated room to demo map (#591 <https://github.com/at-wat/neonavigation/issues/591>)
* Contributors: Atsushi Watanabe

obj_to_pointcloud

No changes

planner_cspace
* planner_cspace: abort A* search on continuous timeout (#592 <https://github.com/at-wat/neonavigation/issues/592>)
* Contributors: Atsushi Watanabe

safety_limiter

No changes

track_odometry

No changes

trajectory_tracker

No changes

Holding for Melodic sync.

New version is released: https://github.com/ros/rosdistro/pull/28678


Main

Description
Please summarise the changes.
Related issues
Please mention any github issues addressed by this PR.
Checklist

[ ] I have performed a self-review of my code.
[ ] I have commented hard-to-understand parts of my code.
[ ] I have made corresponding changes to the public API documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have updated the changelog with any user-facing changes.

I would like to merge this PR without the tests working because of the issue adresse in https://github.com/CQCL/pytket-quantinuum/pull/390, would that be fine with you @cqc-alec ?

I would like to merge this PR without the tests working because of the issue adresse in #390, would that be fine with you @cqc-alec ?

Yes


Added word and sentence(s)

This is simply a word and sentence regex for text inputs. It allows for custom messages. The sentence validation requires actual sentence punctuation which could eventually be made as an optional feature. Word basically requires that the input is letters only.

Sorry also built a test for each validator.

Cool will update when I get a chance.


Npm Run Dev error

Describe the bug
When I am running "npm run dev" - This error pops up, -  SyntaxError: Unexpected token u in JSON at position 0
at JSON.parse
Screenshots

Hey! Did you add all the necessary env vars? Also you need to run the backend api.
Furthermore, I'm focusing more on https://github.com/homanp/superagent. The hosted versions is here superagent.sh

Same issue as the guy above, this is my env:
DATABASE_URL="URL of my database"
NEXTAUTH_SECRET=""
NEXTAUTH_URL=""
NEXT_PUBLIC_GITHUB_ID=""
NEXT_PUBLIC_GITHUB_SECRET=""
OPENAI_API_KEY=""
METAL_API_KEY=""
METAL_APP_ID=""
METAL_INDEX_ID=""
METAL_CLIENT_ID=""
NEXT_PUBLIC_AMAZON_S3_BUCKET_NAME=""
NEXT_PUBLIC_AMAZON_S3_SECRET_ACCESS_KEY=""
NEXT_PUBLIC_AMAZON_S3_ACCESS_KEY_ID=""
NEXT_PUBLIC_LANGCHAIN_UI_API_URL=""
I'm running a supabase postgre database, which you failed to mention as a prerequisite.
The API is up and running, i checked it, the error pops up when i try to access the site by running the ip in my browser.
Can you be a bit more specific about what else we need to add to .env?
Also another thing, this is the error that pops up when i try to build the project:
Failed to compile.
./app/user/login/login.js
Module parse failed: Bad character escape sequence (2:99)
File was processed with these loaders:

./node_modules/next/dist/build/webpack/loaders/next-flight-loader/index.js
./node_modules/next/dist/build/webpack/loaders/next-swc-loader.js
You may need an additional loader to handle the result of these loaders.
| import { createProxy } from "next/dist/build/webpack/loaders/next-flight-loader/module-proxy"

const proxy = createProxy("C:\Users\User\Desktop\DSC\work\langchain-ui\langchain-ui\app\user\login\login.js")
|
| // Accessing the __esModule property and exporting $$typeof are required here.

Import trace for requested module:
./app/user/login/login.js
./app/user/login/page.js

Build failed because of webpack errors

I'd also like a detailed explanation of the 6th 7th and 8th step, are these supposed to be ran in different terminals and should they be running simultaneously along with a terminal running the API?

Hey @PavleVidanovic
This repo needs to be configured as well and ran separately https://github.com/homanp/langchain-ui-api
I'm not actively developing this repo any more so things might be out of date etc. Focusing on building https://github.com/homanp/superagent atm.


go-ipa: use more performant Inverse

This PR shows how the optimized go-ipa Inverse(...) code change impacts benchmarks in this repo.
The raw Inverse(...) benchmark has the same expected speedup improvement as shown in the go-ipa benchmark:
name                  old time/op    new time/op    delta
FpInverse/go-ipa-16     1.66µs ± 5%    1.12µs ± 2%  -32.34%  (p=0.000 n=15+13)
FpInverse/go-blst-16    1.42µs ± 1%    1.42µs ± 2%     ~     (p=0.955 n=13+15)

Regarding our other benchmark to specifically measure the performance of trie key generation:
name                  old time/op    new time/op    delta
HashAddress/pedersen_hash-16    6.71µs ± 1%    6.08µs ± 2%  -9.40%  (p=0.000 n=12+15)
HashStorageSlot/pedersen_hash-16    8.45µs ± 3%    7.88µs ± 3%  -6.75%  (p=0.000 n=15+15)

So, we got around a ~1.09x speedup.
 This makes sense as shown in my previous document where the flamegraph shows that the cost was dominated by Inverse(..). We also got a slight improvement in full Pedersen Commitment, but that operation is dominated by Mul(...).
Note: I'll keep this as a draft PR until we have an idea if https://github.com/crate-crypto/go-ipa/pull/26 will get merged.

results on my arm64 machine:
name                  old time/op    new time/op    delta
FpInverse/go-ipa-12     2.82µs ± 0%    1.93µs ± 0%   ~     (p=1.000 n=1+1)
FpInverse/go-blst-12    2.47µs ± 0%    2.53µs ± 0%   ~     (p=1.000 n=1+1)

So the speedup is similar.


Update typings

When using RMA in a recent project, I found that the types weren't particularly robust and so took a look at refining them in order to make the Animated component a little safer.
Take a look and feel free to merge or expand on my initial revisions 👍

Also added support for mountTimingFunction and unmountTimingFunction props

@nathnhughes your pull is so great, thanks! Would you like me to put you as a collaborator?

@mijim thanks, that would be great!


Output a value in a label

I have a very simple form, and i'd like to output a variable, passed to the form, inside a label.
For example, in a form to input a weight measurement, i'd like to write the unit of measure next to the input box (es. Kg). The unit of measure would be the variable i pass to the form.
Is it possible?
Thank you very much!!

not sure which UI do you use:

for bootstrap use addons https://formly.dev/examples/bootstrap-specific/input-add-ons
for material use suffix https://formly.dev/examples/other/material-prefix-suffix


fix UseIdentityServerAuthentication extension method to avoid exception

The exception info:
An unhandled exception of type 'System.InvalidOperationException' occurred in Microsoft.AspNetCore.Http.Abstractions.dll
Additional information: Unable to resolve service for type 'Microsoft.AspNetCore.Builder.IApplicationBuilder' while attempting to activate 'IdentityServer4.AccessTokenValidation.IdentityServerAuthenticationMiddleware'.
Because of lack of a parameter in "app.UseMiddleware" function,  just add it then work well

Hi @huoshan12345, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

thanks - I missed that.


Revert "Merlin ask a pull request of my branch into masters"

Reverts Wormkil/firstProject#22

This is not what you should do.


Add files via upload

🛠 Issue (Number) -- #17
Description
Type of change

Checklist:

[✅ ] I have made this from my own
[✅ ] I have taken help from some online resourses
[ ✅] I have commented my code, particularly in hard-to-understand areas

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Shall I proceed ?


My flutter web application is not running, I've tried everything.

From flutter MASTER, to BETA, to STABLE. I've shifted channels everywhere, but the error I always get while executing flutter run -d chrome is:
Launching lib\main.dart on Chrome in debug mode...
Syncing files to device Chrome...
18,003ms (!)
Failed to compile application.

I've ran flutter doctor multiple times, upgraded flutter multiple times. Still the error persists.
Want to see my flutter doctor result? Here it is:
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel master, 1.20.0-1.0.pre.91, on Microsoft Windows [Version 10.0.18363.900], locale en-IN)

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[√] Chrome - develop for the web
[√] Android Studio (version 4.0)
[!] IntelliJ IDEA Community Edition (version 2018.3)
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
[√] IntelliJ IDEA Community Edition (version 2019.2)
[√] Connected device (4 available)

! Doctor found issues in 1 category.

Want to see my flutter devices reslult? Here it is:
POCO F1    • 488f7b1    • android-arm64  • Android 9 (API 28)
Web Server • web-server • web-javascript • Flutter Tools
Chrome     • chrome     • web-javascript • Google Chrome 83.0.4103.106
Edge       • edge       • web-javascript • Microsoft Edge 83.0.478.54

What shall I do?

Can you show us what steps you've followed? also, are you on mac?

I am on Windows.

Can you show us what steps you've followed? also, are you on mac?

I just created an app for android phone. But then I wanted to test it out on web after it was established well for android phone. I followed all the traditional steps of
flutter channel beta
flutter configure --enable-web
flutter upgrade --force
flutter run -d chrome
and it fails to run it on chrome.
I have tried on all other channels as well, not just beta.

Could you provide the output of flutter analyze

Could you provide the output of flutter analyze

Ok. There was some weird import dart.fft in one of the dart files somewhere. I removed that and it made the connection. But now the problem is that the chrome is giving a blank white screen. I think it requires a different issue, so I'm going to close this now. Thanks for your help everyone.


Reproduce mock server patch generation increase issue

This PR is just to show an issue with MockServer, that generation increases when status is patched.

cc @manusa


connect to gtps

is there an easy way to connect to a GTPS? I'm not familiar with c# and not sure how I could do it - thanks

is there an easy way to connect to a GTPS? I'm not familiar with c# and not sure how I could do it - thanks

later i make it support gtps, and btw, this is not C#, this is C Programming

is there an easy way to connect to a GTPS? I'm not familiar with c# and not sure how I could do it - thanks

later i make it support gtps, and btw, this is not C#, this is C Programming

when :trollface:

is there an easy way to connect to a GTPS? I'm not familiar with c# and not sure how I could do it - thanks

later i make it support gtps, and btw, this is not C#, this is C Programming

when :trollface:

idk, lazy to add 😂

is there an easy way to connect to a GTPS? I'm not familiar with c# and not sure how I could do it - thanks

later i make it support gtps, and btw, this is not C#, this is C Programming

when :trollface:

idk, lazy to add 😂

💀

oke done


Allow volunteers to actually sign up

Resolved #131
Volunteers now see a "Sign Up" and "Un-Sign Up" button under event details now

This merge commit will also take care of #134 due to my inability to use git correctly


Can't compile for MacOS (EDIT: Linux too)

Forgive me if this is a zig issue, but I've tried everything I can think of to make this work on my end.
zig version 0.13.0
 .@"raylib-zig" = .{
            .url = "https://github.com/Not-Nik/raylib-zig/archive/devel.tar.gz",
            .hash = "1220c1b7e14d6e41479e7f8c0a99b99c4ee953da58ce3987c1ab5840bf5266d81e67",
        },

I've tried on Linux, Monterey 12.7.6, and Sonoma (through a VM), all x86.
No matter which module I add a framework path to in my build.zig (Raylib, root_module, etc.), the searched paths in the error is always "none". I don't know where in the chain this search is happening but I haven't managed to get this to change.
I tried SDL and RGFW backends via "addCMacro".
To reproduce:

use the setup script
run zig build -Dtarget=x86_64-macos (or -Dtarget=aarch64-macos)

When the target is not defined and I'm on macOS, it compiles, though I don't know how compatible it is with other OS versions. I'm unable to try compiling natively for aarch64 as I don't have access to any apple silicon devices. My goal is to compile for both via github actions so manually compiling on native hardware is not ideal
The error:
install
└─ install raylib
   └─ zig build-lib raylib Debug x86_64-macos failure
error: error: unable to find framework 'Foundation'. searched paths:  none
error: unable to find framework 'CoreServices'. searched paths:  none
error: unable to find framework 'CoreGraphics'. searched paths:  none
error: unable to find framework 'AppKit'. searched paths:  none
error: unable to find framework 'IOKit'. searched paths:  none

Sorry it seems that I never tried explicitly setting the target as x86_64-linux. I get an error when I do that as well. Seems like only windows lets me set the target explicitly. I'm on NixOS but I can try a different distro when I have a chance,
The error:
/home/gregturbo/.cache/zig/p/1220f655fd57d8e10b5dbe7d99c45a0b9836a13cea085d75cd4c15f6e603a1fcb058/src/external/glfw/src/x11_platform.h:36:10: error: 'X11/Xcursor/Xcursor.h' file not found
#include <X11/Xcursor/Xcursor.h>
         ^~~~~~~~~~~~~~~~~~~~~~~~
/home/gregturbo/.cache/zig/p/1220f655fd57d8e10b5dbe7d99c45a0b9836a13cea085d75cd4c15f6e603a1fcb058/src/rglfw.c:73:10: note: in file included from /home/gregturbo/.cache/zig/p/1220f655fd57d8e10b5dbe7d99c45a0b9836a13cea085d75cd4c15f6e603a1fcb058/src/rglfw.c:73:
#include "external/glfw/src/init.c"
         ^
/home/gregturbo/.cache/zig/p/1220f655fd57d8e10b5dbe7d99c45a0b9836a13cea085d75cd4c15f6e603a1fcb058/src/external/glfw/src/init.c:28:10: note: in file included from /home/gregturbo/.cache/zig/p/1220f655fd57d8e10b5dbe7d99c45a0b9836a13cea085d75cd4c15f6e603a1fcb058/src/external/glfw/src/init.c:28:
#include "internal.h"
         ^
/home/gregturbo/.cache/zig/p/1220f655fd57d8e10b5dbe7d99c45a0b9836a13cea085d75cd4c15f6e603a1fcb058/src/external/glfw/src/internal.h:325:10: note: in file included from
 /home/gregturbo/.cache/zig/p/1220f655fd57d8e10b5dbe7d99c45a0b9836a13cea085d75cd4c15f6e603a1fcb058/src/external/glfw/src/internal.h:325:
#include "platform.h"
         ^
/home/gregturbo/.cache/zig/p/1220f655fd57d8e10b5dbe7d99c45a0b9836a13cea085d75cd4c15f6e603a1fcb058/src/external/glfw/src/platform.h:81:11: note: in file included from
/home/gregturbo/.cache/zig/p/1220f655fd57d8e10b5dbe7d99c45a0b9836a13cea085d75cd4c15f6e603a1fcb058/src/external/glfw/src/platform.h:81:
 #include "x11_platform.h"

For reference, this is the nix shell I'm building it with:
let
  unstable = import (fetchTarball https://nixos.org/channels/nixos-unstable/nixexprs.tar.xz) { };
in
{ pkgs ? import <nixpkgs> {} }:
  pkgs.mkShell {
    nativeBuildInputs = with pkgs.buildPackages; [
      unstable.zig #0.13.0
      SDL2
      pkg-config
      xorg.libX11
      xorg.libXcursor
      xorg.libXrandr
      xorg.libXinerama
      wayland
      pulseaudioFull
      unzip
      libxkbcommon
      glfw
    ];
}

trying on popOS without nix-shell gives me the same error as #139

I was getting similar errors. I'm on Windows and I don't have a Mac, so I don't have a straightforward solution for it. However, for Linux, I could get it to work by supplementing some libraries for X11. I believe GLFW needs them. So, I modified the zig-raylib\build.zig and my project's build.zig.
Make sure to have a local copy of zig-raylib. First, you will need the system libs and headers from this link: https://github.com/zig-gamedev/zig-gamedev/tree/main/libs/system-sdk
Then, in raylib-zig\build.zig, add these:
exe.addLibraryPath(b.path("../system-sdk/linux/lib/x86_64-linux-gnu")); // add this, line 80s

exe.linkSystemLibrary("GL");
exe.linkSystemLibrary("rt");
exe.linkSystemLibrary("dl");
exe.linkSystemLibrary("m");
exe.linkSystemLibrary("X11");

const lib = raylib.artifact("raylib");

lib.addLibraryPath(b.path("../system-sdk/linux/lib/x86_64-linux-gnu"));  // add, line 110s
lib.addSystemIncludePath(b.path("../system-sdk/linux/include"));	 // add

And in your project's build.zig, add this line:
const exe = b.addExecutable(.{});

exe.addLibraryPath(b.path("libs/system-sdk/linux/lib/x86_64-linux-gnu")); // add this too

You will also need to build with GNU: zig build -Dtarget=x86_64-linux-gnu
Making these modifications allows me to build on Linux. I test it through VMs (PopOS), WSL (Ubuntu), and cross-compile to Linux on Windows.
There are also Mac libraries and frameworks in the link above. I have not tested them, though.

Thanks, that works for me on Linux. Unfortunately the problem with macos is that I still can't get it to search any paths at all for the sdk

ok well it compiles with x86_64-linux and x86_linux-gnu but now I get this error trying to launch it:
INFO: Initializing raylib 5.5-dev
INFO: Platform backend: DESKTOP (GLFW)
INFO: Supported raylib modules:
INFO:     > rcore:..... loaded (mandatory)
INFO:     > rlgl:...... loaded (mandatory)
INFO:     > rshapes:... loaded (optional)
INFO:     > rtextures:. loaded (optional)
INFO:     > rtext:..... loaded (optional)
INFO:     > rmodels:... loaded (optional)
INFO:     > raudio:.... loaded (optional)
WARNING: GLFW: Error: 65544 Description: X11: Failed to load Xlib
WARNING: GLFW: Failed to initialize GLFW
Segmentation fault at address 0x0
???:?:?: 0x0 in ??? (???)
Aborted (core dumped)

Never seen it before, I had a fully working and finished app when it was running without an explicit target but I'm about ready to give up on raylib/raygui at this point lol

I feel you. I have been experimenting with raylib for the last month. I think I have seen that error once. I have not used raygui, but my guess is that the problem is related to raygui.

@GregTurbo, this is my shell.nix for a z-raylib project on NixOS and Darwin (note the shell hook for Darwin)
{ pkgs ? import <nixpkgs> {} }:
let
  zig_0_13 = pkgs.callPackage ./nix/zig/0.13 { };
in
with pkgs; mkShell {
  nativeBuildInputs = [
    pkg-config
    glfw
    zig_0_13
  ]
  ++ lib.optionals stdenv.isDarwin [
    darwin.apple_sdk.frameworks.AudioToolbox
    darwin.apple_sdk.frameworks.AudioUnit
    darwin.apple_sdk.frameworks.Cocoa
    darwin.apple_sdk.frameworks.CoreAudio
    darwin.apple_sdk.frameworks.CoreMIDI
    darwin.apple_sdk.frameworks.CoreVideo
    darwin.apple_sdk.frameworks.GLUT
    darwin.apple_sdk.frameworks.IOKit
    darwin.apple_sdk.frameworks.OpenGL
  ]
  ++ lib.optionals stdenv.isLinux [
    libGL
    libGLU
    xorg.libX11
    xorg.libXrandr
    xorg.libXi
    xorg.libXcursor
    xorg.libXinerama
  ];

  shellHook =
  ''
  ''
  + lib.optionalString stdenv.isDarwin ''
    # https://github.com/ziglang/zig/issues/18998
    unset NIX_CFLAGS_COMPILE
    unset NIX_LDFLAGS
  '';
}

Just to recap, a normal zig build works for you on Linux and macOS? But passing -Dtarget=x86_64-linux or -Dtarget=x86_64-macos respectively breaks the build? In that case the issue is probably with Zig.

Stale


can someone help to create version 0.5.2?

Hello the team here, can someone give help to create the version 0.5.2 to include the latest changes ? I plan to use the API like EnqueueUniqueInByKey. Thanks a lot !

Hello, is the project still active? why there is no upgrading and support anymore?

Same question for me... v0.5.1 doesn't have the latest changes, which really sucks as I'm also needing to use EnqueueUniqueInByKey()

For future users needing this, here's how I got it to work:

I updated my go.mod file have the latest master commit hash, instead of the v0.5.1, ie:
github.com/gocraft/work 5959e69ad211c5ca37ffdf3ede02e35a5ae41d98

I then ran go mod tidy to confirm it worked, and it changed the git hash to:
github.com/gocraft/work v0.5.2-0.20200429214246-5959e69ad211

I don't know how/why go mod is smart enough to change this to the missing v0.5.2 version, but I'm happy it worked.
I think you could technically do github.com/gocraft/work master, but then that will probably grab unexpected changes for when/if anything new commits to master (which I don't want any unexpected surprises).

Looks like the maintainers are trying to replace this with https://github.com/taylorchu/work, which I think is a mistake because there are no plans to replicate the Web UI in that project the same way its done here. I believe that makes this current version far superior that the new version they are building.

Bummer. I'm in the process of looking for an alternative since this project is super dead... I've found problems with the EnqueueUniqueInByKey() not working as it should.

@skplunkerin , have you make the EnqueueUniqueInByKey() work?


update: add recovery path upon apiserver errors

What type of PR is this?
/kind bug

What this PR does / why we need it:

CRI can get stuck in transient state if apiserver is not available when trying to update CRI state after cloud operation.
When apiserver is not reachable from the controller, the controller process should be killed so that other controller that might have apiserver acces can take over.
This PR also includes a CRI recovery that reverts CRI state to non-transient state upon controller start. This will trigger controller reconciler to kick in and handle the rest.

Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.721%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
179

Relevant Lines:
419

💛  - Coveralls


Incorrect exercise: [courseId=grammar-intro_eng][lessonNumber=12][exerciseId=3466]

Description
user should not be marked incorrect for specifying tenses, assuming they match the prompt's tenses.
Diagnostics information (do not remove)

Exercise identifier: [courseId=grammar-intro_eng][lessonNumber=12][exerciseId=3466]
Exercise payload: {"sentences":[{"text":"The future buyer is happy.","lojbanic":false}],"title":"Translate this sentence","type":"typing"}
Attempted solution: {"text":"lo ba te vecnu ku ca gleki"}
Attempted solution was classified as correct: no
Validation payload: {"canonicalAnswer":"lo ba te vecnu cu gleki","correct":false}

Agree. Unfortunately, this is difficult to implement witthout negatively impacting things like "lo pu vecnu" as well (where I do want tenses to be mandatory).
I will add a remark in the lesson about tenses that, unless explicitly requested in the prompt (e.g. "Translate specifying tenses"), tenses should never be specified.


Initial split to public/private w/ 1ES PT

1ES conversion. Basic build seems okay. Test build at https://dev.azure.com/dnceng/internal/_build/results?buildId=2399802&view=results - we need to redirect the public pipeline to the new pipeline filename post-merge

I merged this and updated the pipeline and triggered a build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=604822&view=results

Brilliant. Thanks Eric

Looks like official build is broken after this, we had one good build, then it failed after this went in.
It's in the same step as job as was failing before, but it's a different step that's failing.  Easy to miss since the dang thing was failing before.
Seems to be ReleaseConfigs aren't created.  Let's see why not.


Write a PHP program to print numbers from n to 1

Description
Write a PHP program to print numbers from n to 1
Input  : 5
Output : 5 4 3 2 1

How to contribute

Save the solution in program/print_numbers_from_n_to_1.php file
Add print_numbers_from_n_to_1.php file in print-numbers-from-n-to-1 folder

!assign


bugfix find blas

closes #233

I can't review this unfortunately as OSX is case-insensitive (which is possibly why this slipped in in the first place).
To check that everything is OK, I would have a look at the Travis logs for the master branch and search for the warning in #233. If your fix is OK, the same warning will have disappeared in this PR's Travis log.

Or I suppose confirming that the warning message is no longer on your machine is good enough.

The warning disappeared, check here for example. You see that BLAS was found.
I'm not sure it's a Apple issue. CMake is case sensitive on both Apple and UNIX.


[WIP]update Branch 2.5 mrs

Why I'm doing:
What I'm doing:
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.19 out of 20 committers have signed the CLA.:white_check_mark: gengjun-git:white_check_mark: decster:white_check_mark: luohaha:white_check_mark: Astralidea:white_check_mark: amber-create:white_check_mark: banmoy:white_check_mark: DanRoscigno:white_check_mark: satanson:white_check_mark: evelynzhaojie:white_check_mark: chaoyli:white_check_mark: yongbingwang:white_check_mark: Smith-Cruise:white_check_mark: letian-jiang:white_check_mark: yandongxiao:white_check_mark: liuyehcf:white_check_mark: packy92:white_check_mark: nshangyiming:white_check_mark: hellolilyliuyi:white_check_mark: kevincai:x: mergify[bot]You have signed the CLA already but the status is still pending? Let us recheck it.


Update on-demand CSS and JS again

Primarily to ensure that the U.S. flag CSS image URLs are root-relative (see cfpb/cfgov-refresh#3129).

You can confirm the intended change by expanding the header.css and footer.css diffs and Ctrl+Fing for us-flag.


Organize tests

@tothtamas28 let's get in the habit of putting a summary of changes in the main commit body (even a short one), especially regarding the rationale for the changes. This message is what's included in the commit log on squash merge, and while it's not always useful, when it is useful it's very useful.


re-enable edit tests

fixes #6

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

Merging [#9][cc-pull] into [master][cc-base-branch] will increase coverage by 1.17%

@@             master         #9   diff @@
==========================================
  Files            12         15     +3
  Lines          3415       3485    +70
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           2198       2284    +86
+ Misses         1217       1201    -16
  Partials          0          0

Powered by Codecov. Last updated by [3b8a881...80de4b0][cc-compare]
[cc-base-branch]: https://codecov.io/gh/SexualHealthInnovations/django-wizard-builder/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/SexualHealthInnovations/django-wizard-builder/compare/3b8a881060158f5c3b19a56fffd35ff92547b175...80de4b005cf7ab28de812a39af4257c02c4e080d?src=pr
[cc-pull]: https://codecov.io/gh/SexualHealthInnovations/django-wizard-builder/pull/9?src=pr


[GENERIC viewer] Skip the iframe-case when checking if the container div, on BaseViewer-instances, is absolutely positioned (PR 12354 follow-up)

Given that getComputedStyle only works on visible elements, the result of PR #12354 is that if the viewer is placed in a hidden iframe the viewer will now be broken. This obviously wasn't the intention of that PR, hence I believe that we should limit the position: absolute; check slightly to avoid this.

/botio-linux preview

Looks good!


no results for infohash lookup

The following code snippet yields no peers for the lookup query. no peer is printed to stdout. IF i use the same DHT object for announce and lookup, the correct IP is found. However when creating a new DHT object, the IP is not found.
Am i misusing the library or is this expected behavior and why?
Thanks!
var DHT = require("bittorrent-dht")

var ih = "46f9e9a430f839e7acd76733d649b885985cff4b"
var dht =  new DHT()

dht.listen(20000, function () {
  console.log('now listening')
})

dht.on('ready', function () {
  console.log("dht is ready")

  dht.announce(ih, 6666, function (r) {
    var otherDHT = new DHT()

    console.log("announce result is " + r)

    otherDHT.listen(20001, function () {
    })
    otherDHT.on('ready', function () {
      console.log("other dht is ready")

      otherDHT.lookup(ih, function (err, r) {
      })
      otherDHT.on('peer', function (addr, ih, from) {
        console.log("got peer " + addr)
      })
    })
  })
})

Hm, this should work. Not sure why it doesn't.

This issue was caused by a bug introduced in @mvayngrib's PR #52, and fixed in #54.
Will release a new version of bittorrent-dht to fix the issue.

Released as 3.0.0.

nice work, thanks!


Leveling tracker shows success on import but doesn't show guide info

I downloaded the new version from today, ran the script, and in the Leveling Tracker section, clicked Generate Guide. I imported my build (Pohx RF Jugg from his website) and checked off my progress on the website, before exporting and then importing with the LLK Import Guide button. It shows success, but the progress percentage that said 0% disappears, and I then get a 'no imported guide' message when left clicking the book icon, and the attached error message when right-clicking it.

placing the cursor behind the r in line 550 of leveling tracker.ahk and hitting backspace twice to just remove the r will let me right-click the book icon, but no characters are found, and still cannot get the guide to import correctly. After doing this I completely wiped the LLK folder and reinstalled/re-started the script and am having the same issue as originally posted.

I do run Path of Building and Awakened POE Trade. So I tried shutting down PoE, LLK, and both of those other tools, then renamed the "C:\Program Files (x86)\Steam\steamapps\common\Path of Exile\logs\Client.txt" file to Client_backup.txt.
After those steps I started up PoE, then LLK, and tried to enable the Leveling Tracker with the default route (no imported build, but disabled league start and library checkboxes), I still get 'no imported guide' and the error about the r on line 550 in leveling tracker.ahk
route-info.txt
pob-data.txt

Yeah, that's the build guide I tested earlier. Could you open the ini folder of the script and open the 'leveling guide.ini' file? That's where the individual steps and the progress are saved. Is there anything unusual about the contents of that file?
I also have to hit the bed right now, but I'll check back first thing in the morning.

I'm having the same exact issue. There is no leveling guide.ini file anywhere in the folder. Looks like maybe that file/data is not getting written for some reason.

I'm having the same exact issue. There is no leveling guide.ini file anywhere in the folder. Looks like maybe that file/data is not getting written for some reason.

Some people have write-permission issues that prevent them from setting up the image-checks. Maybe running the script as administrator would help, I'm not sure. A temporary fix could be creating said file manually as a blank ini-file and see if that fixes it. Though that would be just a temporary fix, and the problem would certainly come up again somewhere else.

It must be something with folder security. I also could not find that file, and ended up trying to run LLK from C:\temp instead of Program Files and now it is actually importing the build successfully. Characters for gear tracker (right-click book icon) are still not importing after restarting LLK multiple times so I'm not sure what to check for that yet.

Characters for gear tracker (right-click book icon) are still not importing after restarting LLK multiple times so I'm not sure what to check for that yet.

Shoot, I got distracted by that other problem. The script scans the client.txt for recent characters by looking for the level-up message that is logged there. You probably have to level up once and then restart the script to make it find the character. Sorry about that, things are pretty hectic right now due to higher traffic.

Add a check for write permissions on the directory and raise an error to the user if the check fails.

Yes, it's included in the next release. Thanks for confirming, I couldn't replicate it whatsoever.


feat: add openapi-lint pre-commit-hook

Just leaving the comment here since the PR is still open: I have an RFC open that advocates adopting redocly for Swagger and OpenAPI linting. https://team-turo.atlassian.net/wiki/spaces/EN/pages/2828501164/RFC-252+Deprecate+node-types-dunlop+repo+turo+types-dunlop+package+in+favor+of+API+generation+from+dunlop-api + https://github.com/turo/dunlop-api/pull/136


restartroundnow no longer restarts the round now

It goes to lobby but it's supposed to just reset the round and bypass the lobby.

I think bypassing the lobby would cause confusion. It wouldn't be friendly on a live server anymore.

restartround is the one that goes straight to lobby but restartroundnow is mainly for debugging.

This is most definitely a feature to me, not a bug. Whenever I need to go to the lobby, I set the CVar game.lobbyenabled to true and then run restartroundnow to get to the lobby fast. If I had to wait for the restartround timer to run out every time I would go insane.
In my opinion you should just set game.lobbyenabled to false to get this behavior.

Also 2nd part of the issue is you can't start the game anyway from said lobby you get to.

How long has that been an issue? I swear that works fine for me.

Only noticed it yesterday.
Also latejoining isn't working for me in debug either. I tried using the respawn command on myself and tried joining but it didn't work (even with role timers off).

This is most definitely a feature to me, not a bug. Whenever I need to go to the lobby, I set the CVar game.lobbyenabled to true and then run restartroundnow to get to the lobby fast. If I had to wait for the restartround timer to run out every time I would go insane.
In my opinion you should just set game.lobbyenabled to false to get this behavior.

I just learned golobby exists which does this, so it makes more sense to have restartroundnow bypass the lobby, and have use golobby if you want to immediately go to the lobby.

Could not reproduce.

Supersceded by #11550 which correctly describes the issue.


how does viewer.exe works? any open source for that?

I believe this was just copied from the provided halite game files

Do you have any of other files they provided with you bcoz the site is not there now.

Hm not that I remember. Everything I needed at the time should probably be in bin.


Assignment 3 - 6ITA - Pratik patel

Solved all the questions
#175

@Pratik00531  Please redo the PR for the new 6ita-task3 branch


Add cssnano to end of postcss plugin chain to compress compiled css

Add cssnano to end of postcss plugin chain to compress compiled css when NODE_ENV=production only.
Split postcss plugins into an explicit base set concatenated with a dev or prod set to achieve this behaviour. This follows the pattern used by the general webpack plugins.
Connected to rangle/rangle-starter#55

:shipit:
By the way - the 'connected to' thing doesn't work when there's a colon (:) there.

OK I can fix those
On Apr 21, 2016 4:08 PM, "Seth Davenport"<EMAIL_ADDRESS>wrote:

[image: :shipit:]
By the way - the 'connected to' thing doesn't work when there's a colon
(:) there.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/rangle/react-redux-starter/pull/148#issuecomment-213094259


Adding Pokemon Mystery Dungeon DX

Hi, could you add Pokémon Mystery Dungeon: Rescue Team DX? Thanks very much!

I have added the game(s) you requested in the most recent release: https://github.com/hobby-grade/Nintendo-Switch-Discord-Status/releases/tag/v1.1.1
Thank you for checking out the project, reach out if you have any more suggestions!

Thank you!!


Synchronous API awkward in node

I'm trying to implement a language server in Node, based on the lsp example. This language server is a bridge between VS Code and a remote compile server - it fulfils requests for, e.g. hover or code completion by making requests to the compile server and then awaiting the results.
It seems to be pretty much impossible to do this due to the mismatch between the synchronous onHover/onCompletion function call from VS Code, and the asynchronous nature of the request to the compile server - VS Code expects a result to be returned immediately from the IConnection.onHover callback, but Node expects to call back with results at some point in the future.
Is there any possibility of implementing asynchronous alternatives to onHover, onCompletion etc. that allow for hover and completion results to be returned to VS Code asynchronously, rather than immediately?

Realised  this is an issue with the node implementation rather than with the protocol.

@NotRelevant You can return results asynchronously with vscode-languageserver if you want. I have some TypeScript code that looks like this and it works fine.
connection.onCompletion((textDocumentPosition: TextDocumentPositionParams): CompletionItem[] | PromiseLike<CompletionItem[]> => {
  /* ... */
});

Thanks, that's exactly what I need.

Each method on the server can either return the value or a promise resolving to the value (which allows async calls to other servers)


chore(CategoryTheory/Sites): remove unused lemmas and simplify some proofs in Sites.Discrete

Why not using the name IsConstant? Is "discrete" a terminology that is specific to condensed objects?

I guess "discrete" is specific to topological objects such as condensed sets, and IsConstant is better for general sheaves (note that the constant sheaf is in general not with this terminology though...). I'll rename it.

I messed up the merge, closing in favour of #16385


Write a Dart program to convert fahrenheit to kelvin

Description
Write a Dart program to convert fahrenheit to kelvin
(1°F − 32) × 5/9 + 273.15 = 255.928K
Input  : 1
Output : 255.928

Contributed by - @KyleMcIndoe

How to contribute

Save the solution in convert_fahrenheit_to_kelvin.dart file
Add convert_fahrenheit_to_kelvin.dart file in program/convert-fahrenheit-to-kelvin folder

I would like to work on this @harshraj8843, assign it to me.

Hey @gantanikhilraj
You can comment !assign on any good first issue you want to contribute to, to be auto-assigned
!assign

!assign


Rename and restructure Locust/TaskSet

Some of locusts core concepts have a naming style that makes it unnecessarily confusing. Also, some of the code design doesn't make a lot of sense.
I'd like to:

Merge the Locust and TaskSet classes into one. Separating the two never made sense to me. The user's Locust class almost never contains a significant number of things, and the things that ARE there are mostly things that should actually be set via command line parameters/config file "orthogonally" to the user behaviour (like wait times & target host). Another benefit of this is that it will make "the most basic locust script you could ever write" a lot shorter & easier to understand, so onboarding new users will be easier.
Rename this Locust/TaskSet to User
Change some command line parameters to use industry-standard names (avoid stuff like "hatch rate". Maybe this is also the time to drop the master/slave terminology)

Of course these things should be done carefully and, if possible, leaving the old functionality in place but marked deprecated for a while. I'm not sure if I'll ever have time to implement any of this, but I'd like to talk about it :P

Merge the Locust and TaskSet classes into one

I'm not in favor of merging them. I think they are conceptually different. In the beginning of Locust's development TaskSet didn't exist. Instead tasks were declared directly on the Locust class, and one could nest SubLocust classes. I think the introduction of the TaskSet class made it much better.

The user's Locust class almost never contains a significant number of things

For most larger test codebases I've worked on the Locust classes has contained a bunch of stuff. Almost always wrapper methods for making HTTP requests, and often some state for the simulated users.

Rename this Locust/TaskSet to User

I wouldn't mind renaming Locust to User.

Change some command line parameters to use industry-standard names (avoid stuff like "hatch rate". Maybe this is also the time to drop the master/slave terminology)

Sounds like a good idea!

Merge the Locust and TaskSet classes into one

I'm not in favor of merging them. I think they are conceptually different. In the beginning of Locust's development TaskSet didn't exist. Instead tasks were declared directly on the Locust class, and one could nest SubLocust classes. I think the introduction of the TaskSet class made it much better.

To me, a TaskSet represents a user behaviour (or "scenario" if you will). What does a Locust represent?

The user's Locust class almost never contains a significant number of things

For most larger test codebases I've worked on the Locust classes has contained a bunch of stuff. Almost always wrapper methods for making HTTP requests, and often some state for the simulated users.

Oh. I just put helpers & state directly in the TaskSet. Hmm... I guess if you put helpers in the Locust class you can inherit them without restricting the type of TaskSet you can inherit from etc. Are there any other differences/benefit of having them in the Locust? It seems a bit weird to separate actual tasks from helper methods into different classes, but maybe that is just me.

Rename this Locust/TaskSet to User

I wouldn't mind renaming Locust to User.

I guess another way of scratching my itch would be to make TaskSets optional somehow (without having to declare an internal class, as I think you suggested once before). Like putting @task method declarations directly on a Locust (and having that be the "normal" use case, and separately defined TaskSets be the "advanced" case)

Change some command line parameters to use industry-standard names (avoid stuff like "hatch rate". Maybe this is also the time to drop the master/slave terminology)

Sounds like a good idea!

I guess what I'm trying to say is that there are two main things that define a load test.

The behaviour of the users
The load profile (how fast users run, how many of each user type, for how long, rampup etc)

I think it makes a lot of sense to define those orthogonally. Right now they kind of bleed into eachother and they dont have a clear object/class representation.
I think a Locust (in the current implementation) is more like a "thread" or "TaskSet executor". Someone/thing that can perform user tasks, but isnt a user itself.

+1 on using industry-standard names

To me, a TaskSet represents a user behaviour (or "scenario" if you will). What does a Locust represent? (it isnt really a "user", it just contains a user/scenario)

A Locust represents a simulated user while a TaskSet can represent the user behaviour, or part of the user behaviour when TaskSets are nested (e.g. a sub section of a web app).

I just put helpers & state directly in the TaskSet. Hmm... I guess if you put helpers in the Locust class you can inherit them without restricting the type of TaskSet you can inherit from etc. Are there any other differences/benefit of having them in the Locust?

If you have state or helpers that are global to the simulated user, and you use nested TaskSets, you can always access that state, or those helpers, using self.locust.my_state. (I guess this could be partly solved by introducing a root attribute on TaskSet instances).
Overall I'd say that the separation of Locust and TaskSet makes sense when you consider cases with nested TaskSets. Also, if we rename the Locust class to User, it'd feel very weird to nest User classes under other User classes.

I guess another way of scratching my itch would be to make TaskSets optional somehow

I do remember considering this when introducing TaskSet (previously tasks could be declared directly under Locust and SubLocust). The only benefit from this would be that the minimal locust example would be slightly smaller (for any real world scenario, having to declare one User class on a few lines shouldn't matter). I can see two drawbacks which I think exceeds the benefit:

It feels like a violation of the There should be one-- and preferably only one --obvious way to do it.
It would complicate the implementation a lot. I imagine that we'd construct a TaskSet dynamically under the hood (because the alternative - to bring TaskSet logic into User/Locust - seems like a worse idea) which would make the code more "magic", and the developer would know less about the environment that their code would be running in.

I think a Locust (in the current implementation) is more like a "thread" or "TaskSet runner". Something that can perform/trigger user tasks, but isnt a user itself. Renaming it "User" wouldnt really fix that.

I don't understand why you don't think Locust/User in the current implementation doesn't represent a user. To me it's exactly what it represents. It is run in a separate thread/greenlet, yes, but that's what I'd expect (how could it represent a simulated user without being run a separate thread?).

To me, a TaskSet represents a user behaviour (or "scenario" if you will). What does a Locust represent? (it isnt really a "user", it just contains a user/scenario)

A Locust represents a simulated user while a TaskSet can represent the user behaviour, or part of the user behaviour when TaskSets are nested (e.g. a sub section of a web app).

Ok, I think I understand your thinking now. The Locust represents a "client", not really a "user" (because a user, in my parlance, has more than just state and utility functions, it also has an objective or task that it wants to perform).
 Do you agree?
However, I think there are some "code smells" that indicate that they are not really separate things in the current implementation (wait times are on both objects, the "client" is present in TaskSet as well as Locust, etc) - basically a lot of things indicating that TaskSet is really just a subtype of Locust.

I just put helpers & state directly in the TaskSet. Hmm... I guess if you put helpers in the Locust class you can inherit them without restricting the type of TaskSet you can inherit from etc. Are there any other differences/benefit of having them in the Locust?

If you have state or helpers that are global to the simulated user, and you use nested TaskSets, you can always access that state, or those helpers, using self.locust.my_state. (I guess this could be partly solved by introducing a root attribute on TaskSet instances).
Overall I'd say that the separation of Locust and TaskSet makes sense when you consider cases with nested TaskSets. Also, if we rename the Locust class to User, it'd feel very weird to nest User classes under other User classes.

Hmm... yes, nested TaskSets make it necessary to introduce some more complexity I guess. Tbh, I never felt the need to do that, because "it is just python" :) If a task needs to perform another task in sequence then I just define a function and then call it. I think this is solving a case that is more relevant for other load gens that dont have the inherent flexibility of running code in your load test.
My tests end up being a little less random than they would if I had used locusts built in task weighting but to me it never mattered.
Anyway, I think nested TaskSets do make sense in some cases, and I'm not against them.

I guess another way of scratching my itch would be to make TaskSets optional somehow

I do remember considering this when introducing TaskSet (previously tasks could be declared directly under Locust and SubLocust). The only benefit from this would be that the minimal locust example would be slightly smaller (for any real world scenario, having to declare one User class on a few lines shouldn't matter). I can see two drawbacks which I think exceeds the benefit:

It feels like a violation of the There should be one-- and preferably only one --obvious way to do it.
It would complicate the implementation a lot. I imagine that we'd construct a TaskSet dynamically under the hood (because the alternative - to bring TaskSet logic into User/Locust - seems like a worse idea) which would make the code more "magic", and the developer would know less about the environment that their code would be running in.

Agreed, that is not a good solution within the current framework.

I think a Locust (in the current implementation) is more like a "thread" or "TaskSet runner". Something that can perform/trigger user tasks, but isnt a user itself. Renaming it "User" wouldnt really fix that.

I don't understand why you don't think Locust/User in the current implementation doesn't represent a user. To me it's exactly what it represents. It is run in a separate thread/greenlet, yes, but that's what I'd expect (how could it represent a simulated user without being run a separate thread?).

What I mean is a Locust is lots of things (a client, wait time definitions, a greenlet, etc) and it contains a set of tasks (which is the most important thing for a simulated user). I guess we're kind of getting into semantics here, but a User shouldnt just reference a set of tasks, it is a set of tasks (that is why I like Client better - which also matches what we already use in the command line :) )
I have to go now, but will happily debate more at a later time :)

A client could be dumb and just have an http client (plain HttpLocust) or contain utility functions specific to your site, to allow the TaskSet to be implemented on a higher level of abstraction. Do you agree?

Yes! Though I actually like the User as terminology better than Client, since Client can be confused with something like HTTP/websocket/thrift client. I think of User as in "virtual person" :).
I guess one argument for keeping the Locust terminology is that people might have preconceptions of what a User is, and it might feel weird to declare a User when you're simulating just some program speaking to another program over an API. Locust doesn't come with such preconceptions, and it can just be some entity that will be attacking your system. (Though I still think User is a better terminology).

yes, nested TaskSets make it necessary to introduce some more complexity I guess. Tbh, I never felt the need to do that, because "it is just python" :) If a task needs to perform another task in sequence then I just define a function and then call it.

A lot of apps are structured in a kind of hierarchical structure. Nested TaskSets can be very useful to simulate a person using such an app. Imagine a webapp such as GitHub. Then you could have a bunch of TaskSets and tasks that are structured in a way similar to the hierarchy of the app:

Repo

Issues

View issue

Preview comment
Add comment
Like comment
Follow issue
Add label
Close issue

Create issue

PR

View PR

Add comment
Like comment
Merge PR

Create PR

Browse code

Blame code

User page

Follow user
Repositories

Index page

...

Issues

...

Explore

...

Ok, I think I kind of follow you now. I would still argue that the TaskSet is a User (because it mainly accesses things on the User/Locust object (like .client) to which it belongs.
But I guess defining the locust class with an in-line definition of the TaskSet seems a viable workaround (maybe we can work that into the documentation examples?)
I have tried a couple of times to put my thoughts into words but end up getting stuck :)
What if we did something radical, like make tasks functions (not object methods) that take a Locust as a parameter? So we hold no state at all on the TaskSet and all state on the Locust/User? That would get rid of the need to copy lots of properties from the Locust to the TaskSet. Just a thought...

The more I think about it the more I like my idea :) TaskSets would then become basically just lists of weighted (and possibly nested) task methods. The TaskSet object would no longer contain things like wait_time and references to a Locust/User.

The scheduling logic (execute_next_task() etc) would have to be moved to Locust/User.

What if we did something radical, like make tasks functions (not object methods) that take a Locust as a parameter?

That's possible in the current implementation. This works:
def task1(l):
    pass
def task2(l):
    pass

class User(Locust):
    wait_time = constant(1)
    class task_set(TaskSet):
        tasks = [task1, task2]

or this in order to give different wieights:
class User(Locust):
    wait_time = constant(1)
    class task_set(TaskSet):
        tasks = {task1:1, task2:5}

The TaskSet object would no longer contain things like wait_time

I see it as a feature to be able to specify different wait_times for different task sets. The average wait time for some parts of an application could be different from the other parts.
I also think it's useful to be able to store temporary state on the TaskSet instance, related to a certain part of the application.

because it mainly accesses things on the User/Locust object (like .client) to which it belongs.

The TaskSet.client is just a shortcut for TaskSet.locust.client. Though I guess one could argue that having that shortcut might diffuse the individual roles of TaskSet and Locust.

What if we did something radical, like make tasks functions (not object methods) that take a Locust as a parameter?

That's possible in the current implementation. This works:
def task1(l):
    pass
def task2(l):
    pass

class User(Locust):
    wait_time = constant(1)
    class task_set(TaskSet):
        tasks = [task1, task2]

or this in order to give different wieights:
class User(Locust):
    wait_time = constant(1)
    class task_set(TaskSet):
        tasks = {task1:1, task2:5}

The TaskSet object would no longer contain things like wait_time

I see it as a feature to be able to specify different wait_times for different task sets. The average wait time for some parts of an application could be different from the other parts.
I also think it's useful to be able to store temporary state on the TaskSet instance, related to a certain part of the application.

Sure, you can do it, but I'm not trying to add a new feature, I'm trying to clean up the design and remove stuff that adds complexity internally and gives too many ways to do things for a user.
"There should be one-- and preferably only one --obvious way to do it", right? :)
By no longer instantiating TaskSets we make it so that there is only one place where the test plan stores user state (the Locust/User)
Being able to specify different wait times for different task sets is a niche feature imho, and you can always just use sleeps to achieve that. But if you think that is important, fine, the main thing is that there should be no instances of TaskSet.

The main thing is that there should be no instances of TaskSet.

Hmm, I don't really see the big issue with having instances of TaskSets (though maybe it's Stockholm syndrome :)).
I think a class works pretty well to encapsulate the code for testing some part of an application, and I can't see a better solution at the moment. If you declare the tasks as module functions, then you have to an extra reference to each of those functions from your TaskSet/User, which seems redundant.

They can be class functions. Like this:
class WebTask(TaskSet):
    @task
    def my_task(user):
        user.client.post(...)

class WebLocust(FastHttpLocust):
    task_set = WebTask
    wait_time = constant_pacing(1)
    ...

They can be static methods, just not object methods

Hmm, yes, that could work. Though I suspect that it would complicate the implementation a bit.

I'm assuming decorators work the same way for class functions :)

Yep!


[Windows] Remove invalid call in cleanup script

"LOG" function is not defined in script Clean-AntreaNetwork.ps1, an invalid call may cause unexpected failure.

/skip-all


Add 'shouldNotReceive' expectation

I think it would be consistent to have a 'shouldNotReceive' expectation.
Currently, the way to go is writing $m->shouldReceive('foo')->never(). This could then be written as $m->shouldNotReceive('foo').

I have no real objections. @davedevelopment ?

:+1:

:+1:

Fixed in #391


Explain Windows installation

I just tried to install vty on Windows under cygwin. The unix package (a dependency) won't even install. It says in the README that Windows is cygwin only. Is that something that anyone has ever got working? If so, what is the magic? If not, perhaps update the README.

Thanks for pointing this out! Having never tried it and being a new maintainer, I'm not entirely clear on the status of Windows support (although I do welcome contributions in this regard). There is an unfinished pull request that Corey (former maintainer) updated periodically for a while, in case you'd like to read up on the context:
https://github.com/jtdaugherty/vty/pull/1
In the mean time, though, based on your experience, I'll definitely update the README to avoid false advertising.

Now that the README has been updated, please close this issue if you think it has been resolved. Otherwise please let me know what else I can do.

Thanks, that makes it clear and matches my experience.


[Feature Request]: Feedback backend.

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Assign it to me for Feedback backend.
Use Case
The all my approach and implementation I discussed in that ISSUE #305
I add this feedback schema along with Joi validation.
Benefits
No response
Add ScreenShots
@Soujanya2004   Please assign it  to me under label-3. I want to implement this.
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Soujanya2004    One clarification should visitor allow to delete their feedback?
Or they only add the feedback. Rest functionality like delete operation and what to show, this permission only allow for admin? Because this approach follows in modern website.

@Soujanya2004  And also I think there is need authentication/ logged in to give feedback about the platform. Because if there is no authentication or not allow after logged in, we did not track or prevent for spamming, which may cause database overflow issue. Because then unauthourized user can gave unlimited feedback.   So there is need authentication.     Still logged in I hide this feedback button. After that this button appear, for feedback input.

@Soujanya2004 One clarification should visitor allow to delete their feedback? Or they only add the feedback. Rest functionality like delete operation and what to show, this permission only allow for admin? Because this approach follows in modern website.

feedback once given cannot be deleted by user, only admin will have access

@Soujanya2004 And also I think there is need authentication/ logged in to give feedback about the platform. Because if there is no authentication or not allow after logged in, we did not track or prevent for spamming, which may cause database overflow issue. Because then unauthourized user can gave unlimited feedback. So there is need authentication. Still logged in I hide this feedback button. After that this button appear, for feedback input.

just like reviews user can give feedback only if they are logged in

@Soujanya2004
I thought exactly that same thing .
You assign me this.

@Soujanya2004 I can work on it if you want...
You can assign me...
Can make a PR by EOD

@NK-Works  I am in

@NK-Works appreciate ur interest but @Suvadip-sana is already working on it
@Suvadip-sana make sure u finish it by today?


bpf: host: use security identities in to-netdev's trace notifications

For some types of traffic, to-netdev derives precise security identities. Consistently use these values in the trace notifications.

/test


Use URL encoding with /moveTablet

Fixes https://discuss.dgraph.io/t/ratel-does-not-url-encode-predicate-in-move-tablet-url/9067.
The tablet variable in the URL is a string, and it has been inserted into the URL without proper encoding, causing issues. I've fixed that by passing the string through encodeURIComponent.

This change is

Ratel Preview:

Thank you for fixing this, @ajeetdsouza


Create the UnitType (GSIM: UnitType) Domain/Entity Class

Create the interface IUnitType that extends the IConcept interface in the Common library.
Create the UnitType object that implements IUnitType by extending AbstarctIdentifiableArtefact, this object should be placed in Presentation.Domain under StructuralMetadata/Gsim/Concept folders.

it was not finished

Concept class has been removed, thus description is not up to date, I think.

I have updated the description.
Please try to remove every unused reference from each class you create.

Changes have been made
Commit: 101fe363a8821a370f37bb6985639cc180905ff1 [101fe36]
Parents: a9d1774f00
Author: Veaceslav Chirilov<EMAIL_ADDRESS>Date: Friday, July 2, 2021 12:48:00 PM
Committer: Veaceslav Chirilov

UnitType should be under StructuralMetadata/Gsim/Concept folder and please remove all unused references from each class @vchirilov


http://raphaeljs.com/ is down.  Looks like a domain issue

+1

I make a clone http://sikuda.ru/raphaeljs.com/
from https://github.com/DmitryBaranovskiy/raphaeljs.com

Duplicated https://github.com/DmitryBaranovskiy/raphael/issues/1022
Please search for issues before creating a new one.
Thanks.


Is this project still maintained?

This looks exactly like what I need. I'm going to be using something like this in development, since I need to sync with a development server. The lack of activity on this PR is a little concerning though. Is this project still actively maintained?

@Stratus3D.  Looks like the owner is AWOL but I've been using it for awhile now.  It's so simple (just two scripts) that one can fork and modify it easily (like I did for local sync).  I use it to mirror a copy of a data partition to a USB drive and to do development on raspberry pi's by syncing code on my dev computer to the RPi for running.   Easy to invoke at startup.   The watch script uses inotify.  https://techarena51.com/index.php/inotify-tools-example/

@dkebler Thanks for comment. That's an interest use case for this. I ended up not using syncd, but if I ever have a need for it in the future I will use it. Thanks!


would you help me solving Q382?

my code is as below：
    n = 0
    while self.head:
        if random.randint(0,n)==0:
            res = self.head.val
        n += 1
        self.head = self.head.next
    return res

yes，for test  ["Solution","getRandom","getRandom","getRandom"]
[[[3]],[],[],[]]
it raised UnboundLocalError: local variable 'res' referenced before assignment
but, if I name new var to make it equal to self.head:  ss = self.head ,    AC!!!!!
    n = 0
    ss = self.head
    while ss:
        if random.randint(0,n)==0:
            res = ss.val
        n += 1
        ss.head = ss.next
    return res

I am so confused!

@lypro09539 if ur res does not go into if statement then return res will cause an error
define res before the while loop so that it won't cause this issue

3Q


How can one enforce tied weights in Keras?

I have built an auto-encoder consisting of a single hidden layer (hence, two weights and two biases to be learned). I want to impose weight tying, that is to force the network to learn the same weight matrix for input-to-hidden and hidden-to-output transformations.
I wonder how this can be implemented in Keras.
thanks,
Hadi

If the layers come back-to-back, you can combine them into one custom layer that uses the same weights. Otherwise, you have to instantiate the first layer, and pass that instantiation as an argument to the second layer where you will just use the weights from the first.  Either way, you have to use a custom layer. There are many different examples out there..

I've been working on Convolutional Auto-encoders and wanted tied weights, so I tried what @ncullen93 suggested.
It was easier than expected :)
Here is a Gist that implements special layers for Conv1D and Conv2D.

You can tie weights using a Lambda layer:
model = Sequential()
model.add(Dense(10, input_dim=10, bias=False))
W = K.transpose(model.layers[-1].W)
model.add(Lambda(lambda x: K.dot(x, W)))

Here is a sample solution for people who are still wondering about an auto-encoder with tied weights.http://www.pythonexample.com/snippet/python/autoencoder_extrapy_njellinas_python


settings are getting uploaded repeatedly

Visual Studio Code Version : 1.18.1
Code Settings Sync Version : 2.8.5
Operating System : macOS 10.12.6
Occurs On: Upload
Proxy Enabled: Yes
Gist: https://gist.github.com/iwinux/2b45bc4f70aa42fc75482ec15f44e047
It seems that every time the sync.lastUpload field changes, Settings Sync sees it as an update - thus upload it again and again and again....
Look at the last 7 revisions of the gist posted above for example.

can you provide the steps to reproduce in details

@shanalikhan
Steps:

open VS Code settings page (Command + ,)
edit some options (e.g.: editor.fontFamily)
wait for Settings Sync to upload
after uploading, sync.lastUpload in settings.json is updated
and thus triggers Settings Sync to upload again (repeat step 3-4)

This happens when the autoUpload option is enabled. It stops after a half dozen updates but seems unnecessary. It makes it hard to see how/what has changed if you need to figure out where syncs went wrong.

I also have autoDownload enabled which may be making it worse.


(PUP-5583) Simplify acceptance tests

These commits remove support for packages and passenger types of acceptance, bumps our beaker dependency, and uses beaker's updated install_puppet_agent_dev_repo_on method to install puppet-agent on all agent-only platforms. Unfortunately, we can't remove puppet's install_repos_on and its related helper methods, because beaker can't install latest repo configs from nightlies due to BKR-673.

@er0ck @johnduarte @MikaelSmith @james-stocks Unfortunately I couldn't remove as much helper code from puppet's acceptance directory as I would have liked due to BKR-673.

Verified when running the usual way:
$ bundle exec rake ci:test:aio TEST_TARGET=debian6-64a SUITE_VERSION=<IP_ADDRESS>.g5c59c61 SHA=5c59c61a4d02ffb32135c9eb327136c7ab64bbca TESTS=tests/ticket_7101_template_compile.rb
...
* Install puppet-agent...
  Fetching: http://builds.delivery.puppetlabs.net/puppet-agent/5c59c61a4d02ffb32135c9eb327136c7ab64bbca/repo_configs/deb/pl-puppet-agent-5c59c61a4d02ffb32135c9eb327136c7ab64bbca-squeeze.list
...
fa1uy8hvgt7hpu1.delivery.puppetlabs.net (debian6-64-1) 09:56:10$ apt-get install --force-yes  -y puppet-agent
...
* Install puppetserver...

  uk5w2loryn41tge.delivery.puppetlabs.net (redhat7-64-1) 09:56:13$ curl -o /etc/yum.repos.d/pl-puppetserver-latest-el-7-x86_64.repo http://nightlies.puppetlabs.com/puppetserver-latest//repo_configs/rpm/pl-puppetserver-latest-el-7-x86_64.repo
...
  uk5w2loryn41tge.delivery.puppetlabs.net (redhat7-64-1) 09:56:13$ curl -o /etc/yum.repos.d/pl-puppet-agent-5c59c61a4d02ffb32135c9eb327136c7ab64bbca-el-7-x86_64.repo http://builds.puppetlabs.lan/puppet-agent/5c59c61a4d02ffb32135c9eb327136c7ab64bbca/repo_configs/rpm/pl-puppet-agent-5c59c61a4d02ffb32135c9eb327136c7ab64bbca-el-7-x86_64.repo
...
uk5w2loryn41tge.delivery.puppetlabs.net (redhat7-64-1) 09:56:13$ yum -y  install puppetserver
...
    --> Processing Dependency: puppet-agent >= 1.3.0 for package: puppetserver-2.2.2.stable-0.1SNAPSHOT.2016.01.14T0324.el7.noarch
...
    Installing:
     puppetserver        noarch 2.2.2.stable-0.1SNAPSHOT.2016.01.14T0324.el7
                                                       pl-puppetserver-latest  40 M
...
     puppet-agent        x86_64 <IP_ADDRESS>.g5c59c61-1.el7
                                                       pl-puppet-agent-5c59c61a4d02ffb32135c9eb327136c7ab64bbca

Alternatively, you can specify a non-redhat master:
$ bundle exec rake ci:test:aio MASTER_TEST_TARGET=centos7-64m TEST_TARGET=debian6-64a SUITE_VERSION=<IP_ADDRESS>.g5c59c61 SHA=5c59c61a4d02ffb32135c9eb327136c7ab64bbca TESTS=tests/ticket_7101_template_compile.rb
...
* Install puppetserver...

  cd8cx3b6ehzpw7q.delivery.puppetlabs.net (centos7-64-1) 10:01:34$ curl -o /etc/yum.repos.d/pl-puppetserver-latest-el-7-x86_64.repo http://nightlies.puppetlabs.com/puppetserver-latest//repo_configs/rpm/pl-puppetserver-latest-el-7-x86_64.repo
...
  cd8cx3b6ehzpw7q.delivery.puppetlabs.net (centos7-64-1) 10:01:34$ curl -o /etc/yum.repos.d/pl-puppet-agent-5c59c61a4d02ffb32135c9eb327136c7ab64bbca-el-7-x86_64.repo http://builds.puppetlabs.lan/puppet-agent/5c59c61a4d02ffb32135c9eb327136c7ab64bbca/repo_configs/rpm/pl-puppet-agent-5c59c61a4d02ffb32135c9eb327136c7ab64bbca-el-7-x86_64.repo
...
  Installing puppetserver

  cd8cx3b6ehzpw7q.delivery.puppetlabs.net (centos7-64-1) 10:01:35$ yum -y  install puppetserver
...
      Installing : puppet-agent-<IP_ADDRESS>.g5c59c61-1.el7.x86_64                 6/9
      Installing : libjpeg-turbo-1.2.90-5.el7.x86_64                            7/9
      Installing : 1:java-1.8.0-openjdk-headless-<IP_ADDRESS>-2.b17.el7_1.x86_64    8/9
      Installing : puppetserver-2.2.2.stable-0.1SNAPSHOT.2016.01.14T0324.el7.   9/9
...

And you can add an agent on the master node (like puppetserver CI does):
$ bundle exec rake ci:test:aio MASTER_TEST_TARGET=centos7-64ma TEST_TARGET=debian6-64a SUITE_VERSION=<IP_ADDRESS>.g5c59c61 SHA=5c59c61a4d02ffb32135c9eb327136c7ab64bbca TESTS=tests/ticket_7101_template_compile.rb
...
* Install puppetserver...

  ddlfypjq6wwghod.delivery.puppetlabs.net (centos7-64-1) 10:11:02$ curl -o /etc/yum.repos.d/pl-puppetserver-latest-el-7-x86_64.repo http://nightlies.puppetlabs.com/puppetserver-latest//repo_configs/rpm/pl-puppetserver-latest-el-7-x86_64.repo
...
  ddlfypjq6wwghod.delivery.puppetlabs.net (centos7-64-1) 10:11:02$ curl -o /etc/yum.repos.d/pl-puppet-agent-5c59c61a4d02ffb32135c9eb327136c7ab64bbca-el-7-x86_64.repo http://builds.puppetlabs.lan/puppet-agent/5c59c61a4d02ffb32135c9eb327136c7ab64bbca/repo_configs/rpm/pl-puppet-agent-5c59c61a4d02ffb32135c9eb327136c7ab64bbca-el-7-x86_64.repo
...
  ddlfypjq6wwghod.delivery.puppetlabs.net (centos7-64-1) 10:11:02$ yum -y  install puppetserver
...
      Installing : puppet-agent-<IP_ADDRESS>.g5c59c61-1.el7.x86_64                 6/9
      Installing : libjpeg-turbo-1.2.90-5.el7.x86_64                            7/9
      Installing : 1:java-1.8.0-openjdk-headless-<IP_ADDRESS>-2.b17.el7_1.x86_64    8/9
      Installing : puppetserver-2.2.2.stable-0.1SNAPSHOT.2016.01.14T0324.el7.   9/9

:+1:
Passed on AIX 5.3


E2E golden tests cardano-wallet vs. cardano-addresses account xpub (including purpose)

Issue Number
ADP-950
Overview

0ded5ef0626a4efb3b06eb15a6d9032363dfca41
E2E golden tests cardano-wallet vs. cardano-addresses account xpub (including purpose)

Comments
While testing purpose addition to https://input-output-hk.github.io/cardano-wallet/api/edge/#operation/postAccountKey endpoint I have extended existing e2e tests to assert if the returned keys are the same as produced by cardano-addresses.

bors r+

bors r+

bors r+


Use Homie Settings before Homie Setup?

Is it possible to use homie settings before homie is setup.
Id like it use this to set a parameter used fro advertising a node attribute (which has to be done before homie setup...).
Thanks Tim.

Also wondering if its possible to set settings through code during running of homie with out going through mqtt?

Yes, it is possible. But be careful when you use settings within a constructor (or anything called from a constructor) of a global object. See PR https://github.com/marvinroger/homie-esp8266/pull/335 for a solution to use HomieSetting in a constructor.

@marvinroger, ideas?

The problem is we're populating the settings objects at setup time. Introducing another method might be a solution, but we would have to refactor a bunch of things, but that's definitely a valid use case. Let's put this on 2.1.0 roadmap (as 2.0.0 is not even published final.. school, again and again 😕).
Sent from my Google Nexus 5X using FastHub

Ah, yes. My PR #335 just makes it possible to add items to the vector  IHomieSetting::settings before main() (and thus the 'global' setup()) , but the value of the HomieSetting is read during Homie::setup().

school, again and again

I get that, in the same boat.
TBH dont know much about the internals of homie, ill slowly learn how it all works so i can make more meaningful contributions. Thus dont really know what you guys are talking about sorry :P

Could we have a Homie.loadSettings() which loads the config? then use Homie.setup() like normal, but the load config part is optional thus, not interfering with the current workflow? (Homie.setup() would call Homie.loadSettings() if not been called, obviously...).
Can you extract that part of the settings config code and use the settings objects like that, all before Homie.setup()?

BTW: @marvinroger Can you merge my PR soon? It just changes a undefined (or random) behaviour to a defined one.
I need the PR for my HomieNodeCollection, so for now I use my own fork, but this means to not update that often. It also complicates using platformio / travis.

I've beed struggling with this a lot. I'm subclassing Homie nodes which have custom properties. Example a Weather Underground Node that needs an API key, location etc.
When I set the properties' default values in SubclassedNode.setup() or in SubclassedNode.setuphandler() , which is called via Homie.setSetupFunction(), they override the settings that Homie has already read from the config.
See log:
• OTA - Display
• OTA - Setup done
💡 Firmware display (1.0.0)
🔌 Booting into normal mode 🔌
{} Stored configuration
  • Hardware device ID: a020a6117237
  • Device ID: a020a6117237
  • Name: Display
  • Wi-Fi:
    ◦ SSID: onklan
    ◦ Password not shown
  • MQTT:
    ◦ Host: <IP_ADDRESS>
    ◦ Port: 1883
    ◦ Base topic: homie/
    ◦ Auth? no
  • OTA:
    ◦ Enabled? no
  • Custom settings:
    ◦ WuApiKey: xxxxxxxx (set)                       <--- Yeah correct properties are read
    ◦ WuLanguage: DL (set)
    ◦ WuCountry: DE (set)
    ◦ WuCity: MyTown (set)
WundergroundNode Setup                               <--- called from Homie.setup()
↕ Attempting to connect to Wi-Fi...
✔ Wi-Fi connected, IP: <IP_ADDRESS>
Triggering WIFI_CONNECTED event...
↕ Attempting to connect to MQTT...
Sending initial information...
✔ MQTT ready
Triggering MQTT_READY event...
Calling setup function...
Setuphandler
WundergroundNode Setuphandler                         <--- called from Homie.setupHandler()
〽 Sending statistics...
  • Wi-Fi signal quality: 70%
  • Uptime: 11s
Requesting URL: /api/YOUR_API_KEY/conditions/lang:EN/q/UK/YOURTOWN.json        <-- uses wrong defaults, because they are overwritten in setup() setuphandler()
start document
Requesting URL: /api/YOUR_API_KEY/forecast10day/lang:EN/q/UK/YOURTOWN.json

My solution is that I have introduced a beforeSetup() method, that I call from main setup before Homie.setup(). The log then looks like this.
• OTA - Display
• OTA - Setup done
WundergroundNode Before Setup                    <--- called from main setup() before Homie.setup()
💡 Firmware display (1.0.0)
🔌 Booting into normal mode 🔌
{} Stored configuration
  • Hardware device ID: a020a6117237
  • Device ID: a020a6117237
  • Name: Display
  • Wi-Fi:
    ◦ SSID: onklan
    ◦ Password not shown
  • MQTT:
    ◦ Host: <IP_ADDRESS>
    ◦ Port: 1883
    ◦ Base topic: homie/
    ◦ Auth? no
  • OTA:
    ◦ Enabled? no
  • Custom settings:
    ◦ WuApiKey: xxxxxxxx (set)
    ◦ WuLanguage: DL (set)
    ◦ WuCountry: DE (set)
    ◦ WuCity: MyTown (set)
    ◦ MqttTopic: homie/a020a611f230/indoor/# (set)
StatusNode Setup
MqttNode Setup
Topic:homie/a020a611f230/indoor/#
WundergroundNode Setup
↕ Attempting to connect to Wi-Fi...
✔ Wi-Fi connected, IP: <IP_ADDRESS>
Triggering WIFI_CONNECTED event...
↕ Attempting to connect to MQTT...
Sending initial information...
✔ MQTT ready
Triggering MQTT_READY event...
Calling setup function...
Setuphandler
WundergroundNode Setuphandler
〽 Sending statistics...
  • Wi-Fi signal quality: 72%
  • Uptime: 11s
Requesting URL: /api/xxxxxxxx/conditions/lang:DL/q/DE/MyTown.json   <--- Success :)
start document
Requesting URL: /api/xxxxxxxx/forecast10day/lang:DL/q/DE/MyTown.json
start document

Any progress on this ?

Added int v2.1 (not yet merged)
Example of use (also in docs)
void SetupHomieSettings() {
	percentageSetting.setDefaultValue(50).setValidator([](long candidate) {
		return (candidate >= 0) && (candidate <= 100);
	});

	Homie.getLogger() << "percentageSetting Value: " << percentageSetting.get() << endl; // should be default

	Homie.loadSettings();

	Homie.getLogger() << "percentageSetting Value: " << percentageSetting.get() << endl; // should be config value
}

Seems to be stale issue.
Will be closed.
Feel free to reopen if there are more information.


Webhook call returns 403

Bug description
I use Portainer Business Edition 2.16.2 and configured a webhook for a stack, which I use in docker hub to be called after a new image was pushed. The docker hub webhook history shows that the URL was called, but the return code is 403 and so the webhook failed. The docker image is not re-pulled, the stack is not re-deployed. I've also inspected the logs of portainer, but did not find an error message.
If I call the webhook from my local computer (curl -X POST <url>) all works as expected, the new image is pulled and the container restarted with this new image. So, I suppose it has something to do with usage from docker hub.
Expected behavior
The docker image should be pulled from docker hub and the stack should be re-deployed.
Portainer Logs
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF main.go:582 > encryption key file not present | filename=portainer
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF main.go:601 > proceeding without encryption key |
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF github.com/portainer/portainer-ee/api/database/boltdb/db.go:125 > loading PortainerDB | filename=portainer.db
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF github.com/portainer/portainer-ee/api/datastore/backup.go:151 > creating DB backup |
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF github.com/portainer/portainer-ee/api/datastore/backup.go:74 > copying DB file | from=/data/portainer.db to=/data/backups/EE/portainer.db.072.20230115073427
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF github.com/portainer/portainer-ee/api/database/boltdb/db.go:125 > loading PortainerDB | filename=portainer.db
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF github.com/portainer/portainer-ee/api/datastore/migrate_data.go:121 > migrating database | migrator_version=72 portaineree_db_version=72
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF github.com/portainer/portainer-ee/api/datastore/migrate_data.go:139 > migrator_edition=EE portainer_edition=EE store_edition=EE
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF github.com/portainer/portainer-ee/api/cloud/cluster_setup.go:77 > starting cloud cluster setup service |
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34:28 server: Reverse tunnelling enabled
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34:28 server: Fingerprint 0b:08:cf:9d:f7:00:cd:b7:61:18:e9:c9:8a:92:b2:b1
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34:28 server: Listening on <IP_ADDRESS>:8000...
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF main.go:558 > instance already has defined environments, skipping the environment defined via CLI |
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF main.go:887 > starting Portainer | build_number=25296 go_version=1.19.3 image_tag=linux-amd64-2.16.2 nodejs_version=18.12.1 version=2.16.0 webpack_version=5.65.0 yarn_version=1.22.19
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF github.com/portainer/portainer-ee/api/http/server.go:392 > starting HTTPS server | bind_address=:9443
portainer_portainer.1.kfjmpfk91sas@justso    | 2023/01/15 07:34AM INF github.com/portainer/portainer-ee/api/http/server.go:377 > starting HTTP server | bind_address=:9000

Technical details:

Portainer version: Business Edition 2.16.2
Docker version (managed by Portainer): 20.10.12, build 20.10.12-0ubuntu4
Kubernetes version (managed by Portainer): N/A
Platform (windows/linux): linux
Command used to start Portainer: see my stack
Browser: N/A
Use Case (delete as appropriate): Using Portainer at Home
Have you reviewed our technical documentation and knowledge base? Yes

Additional context
I run portainer behind a reverse proxy which handles TLS and maps port 80 to 9000.

@jschirrmacher I believe your Portainer instance is exposed on the internet so that the Docker Hub can interact with your webhook URL?
If so, I would suggest using an open-source project called diun (https://crazymax.dev/diun/). This way, you will be able to run the diun service inside your cluster, and when there is a new image has been pushed to your Docker Hub repo, it will trigger the webhook call.
You can take a look at some example diun config files below:
diun.yaml
watch:
  workers: 10
  schedule: "* * * * *"

providers:
  swarm:
    watchByDefault: false

notif:
  script:
    cmd: "sh"
    args: "/webhook.sh"

webhook.sh - this is using Portainer as an example
#!/bin/sh
echo "Installing curl"
apk add curl --no-cache --update

echo "Updating Portainer image to ${DIUN_ENTRY_IMAGE}"
echo "DIUN_PORTAINER_SERVICE_UUID: $DIUN_PORTAINER_SERVICE_UUID"

DIUN_PORTAINER_IMAGE_TAG=$(echo ${DIUN_ENTRY_IMAGE} | cut -d : -f 2)

curl -X POST \
  "http://portainer:9000/api/webhooks/${DIUN_PORTAINER_SERVICE_UUID}?tag=${DIUN_PORTAINER_IMAGE_TAG}"

Hope this helps :)

Thanks, @stevensbkang ! This seems like a good solution to mitigate the problem.
I still wonder, why it is possible to call the webhook from the outside of my cluster via curl but not from dockerhub. But this seems more like a problem of what dockerhub is sending exactly, maybe some header or something in the payload which makes the difference. In fact, when calling curl, I don't send any payload or extra headers. I'll try to figure out how dockerhub's call looks like and try to thin that out.

Same issue here with Portainer BE 2.19.4 (Docker Swarm 25.0.4): I added the Portainer webhook URL to my repos in Docker Hub but nothing happens on Portainer side and Docker Hub gets a 403 as response.
I am exposing Portainer via a simple nginx reverse proxy.
To be honest, the Diun workaround seems an over-complication for something that should just work