Feature: Add the ability to store state

Whenever BananaBot restarts the announcements are reset to the factory defaults and a new password is generated. It would be good if BananaBot was able to store state to avoid needing to reconfigure custom announcements after a restart.
This task is to implement a mechanism to store the current state of the system (announcements and any other important variables). The preference is to store announcements in a file using a format such as JSON or perhaps YAML depending on which best suits our needs. While a database would be a viable solution, BananaBot is intended to be as lightweight as possible so the preference is to store data in a file as the state of the system should never be too massive. I would like to explore using YAML but am open to other suggestions.
This task should add the ability to store the announcements and any other required data. The storing of the password however will be part of another issue as it needs to be done following best practices. Including hashing (which already occurs) and salting the password.
Done:

BananaBot now stores the announcements and other required data so it can recover and eliminate the need to reconfigure the system after reboots
Unit tests for the new functionality

After almost 6 months I have finally returned and finished (I think) this feature. The only remaining things to do are:

Add validation for the config file once it is detected
Fix the old unit tests and create new ones for the additional code

The app now checks for a config file on startup and loads the config if there is one, or creates one using the defaults if there isn't one. If the app was active when it was closed workers are created for the announcements. The config file is updated anytime a change is made to the state of the system (new announcement, banana time or text updated, days changes etc.) to ensure that in the event of a crash, the most recent state should have been saved.

Validation was added and everything was working as intended from my testing. I began fixing the unit tests which was going well however I began to run into issues when trying to create a clean test environment for each test. Configuration seemed to be persisting across tests.
I tried a number of different solutions to fix the issues but in the end didn't have much luck. This to me highlighted a pretty major design flaw with how I was handling the app state. As I was already unhappy with the design (which I came up with ~6 months prior) it didn't take much to convince myself it needed to change.
I decided to refactor the AppConfig class into an AppState class using the Singleton Pattern. The Singleton Pattern is perfect to store the application's state as there should only ever be one, and it needs to be accessible throughout the app.
As part of this refactor, I am also making an effort to further ensure separation of responsibilities, and also fix any inconsistencies in the code (camel case vs snake case etc.)

The work mentioned in my last comment has now been completed. Specifically:

I have refactored AppConfig to AppState which utilises the Singleton Pattern to share state around the application
I have refactored the logic from endpoints.py that modified the application state to utilities.py where possible
I have fixed inconsistencies in the code, ensuring snake case is used throughout the codebase
I have re-implemented all the test cases to work with the refactored code

All the unit tests pass and as far as I can tell in my testing of the app, everything is working as intended. Ideally, we would have a better test suite, however as this is a small personal project we are not always going to be able to have what would be ideal.
I think this task might FINALLY be done after letting it sit on a branch for 6 months. I will be making a PR shortly :)


Running "abi-types-generator" command fails on ubuntu

Hey there! First of all thanks a lot for this package.
On to the issue! I've tried to run abi-types-generator on an existing project and it failed, I was not sure if the problem was on my end so I cloned the repository and tried to run the examples and I got the same error:
> yarn web3-example
yarn run v1.22.5
$ abi-types-generator './abi-examples/fake-contract-abi.json' --output='./web3/fake-contract-example/generated-typings' --name=fake-contract
/usr/bin/env: ‘node\r’: No such file or directory
error Command failed with exit code 127.

After a quick google search of that error message, and since I'm working on Ubuntu, I tried this solution and it fixed my problem.
Let me know if this makes sense and if there's anything I can help with!

Hey thanks a lot for raising this issue. Yeah this package was written on windows so that error makes sense. Saying that npm fixed this problem on version 5.4.0 but yarn I’m afraid have not. Can you confirm if you use npm to run the script it works? I can have a look at putting that /r fix in the bin file as well and then test it on Ubuntu and windows to make sure it’s not broken anything.
I think you can still use the package right? Or is this a blocker for you?

Hey @joshstevens19 thanks for the quick reply!
I've double checked and if I run the script with npm it still fails because we install our packages with yarn. Just for testing, I tried removing the project node modules and installing them with npm and running the script with npm, and this problem I'm reporting was fixed. However swapping to npm is not an option for us currently.
We have the above hack to fix the issue so we can still use the package, but a fix at the root would be awesome! :)

Have not forgot about this i promise! sorry been very busy its on my TODO list

Hey so now the compiler will make the new line be an LF which seems to work fine on windows as well. Cant check on Linux so if you can confirm this has fixed it that be great > version 1.1.8 :+1:

@joshstevens19 nicely done! Can confirm that it works on version 1.1.8. Keep up the good work :)


Enable Preview

Please put a sample gif or images

@rajaasim I will surely do that. Meanwhile, you can check out the sample app.
Here is the latest app link:
https://github.com/Dhaval2404/ImagePicker/releases/download/v1.5/ImagePicker-v1.5.apk


layers get re uploaded

Hi, I'm hitting a bug where i push the same image twice to the registry and I get layers re uploaded since they get a 404 from the docker registry.
Building and registry version:
docker -v
Docker version 1.7.1, build 786b29d/1.7.1
I attach the Dockerfiles and the script I'm using to build and push.
docker run -d -p 5000:5000 --restart=always --name registry -v /opt/docker/certs:/certs -v /opt/docker/data:/var/lib/registry -v /opt/docker/auth:/auth -e "REGISTRY_AUTH_HTPASSWD_REALM=Registry Realm" -e REGISTRY_AUTH_HTPASSWD_PATH=/auth/htpasswd -e REGISTRY_HTTP_TLS_CERTIFICATE=/certs/domain.crt -e REGISTRY_HTTP_TLS_KEY=/certs/domain.key -e REGISTRY_AUTH=htpasswd registry:2
Registry log:
time="2015-11-02T11:16:53Z" level=error msg="response completed with error" auth.user.name=autobot err.code="BLOB_UNKNOWN" err.detail=sha256:2607274acb71aaabc54f3c0c2cf34f8228636f0c2b6465755b17a42d7c955ff8 err.message="blob unknown to registry" http.request.host="<IP_ADDRESS>:5000" http.request.id=f385373a-7b6f-46d5-bc34-212cba007e38 http.request.method=HEAD http.request.remoteaddr="<IP_ADDRESS>:5496" http.request.uri="/v2/taringa_base_php/blobs/sha256:2607274acb71aaabc54f3c0c2cf34f8228636f0c2b6465755b17a42d7c955ff8" http.request.useragent="docker/1.7.1 go/go1.4.2 kernel/2.6.32-573.7.1.el6.x86_64 os/linux arch/amd64" http.response.contenttype="application/json; charset=utf-8" http.response.duration=6.635495ms http.response.status=404 http.response.written=157 instance.id=a634580a-1388-4f10-b624-309e4aeadb3d vars.digest="sha256:2607274acb71aaabc54f3c0c2cf34f8228636f0c2b6465755b17a42d7c955ff8" vars.name="taringa_base_php" version=v2.1.1
<IP_ADDRESS> - - [02/Nov/2015:11:16:53 +0000] "HEAD /v2/taringa_base_php/blobs/sha256:2607274acb71aaabc54f3c0c2cf34f8228636f0c2b6465755b17a42d7c955ff8 HTTP/1.1" 404 157 "" "docker/1.7.1 go/go1.4.2 kernel/2.6.32-573.7.1.el6.x86_64 os/linux arch/amd64"

REGISTRY=<IP_ADDRESS>:5000
IMAGES='taringa_base
taringa_base_nginx
taringa_base_php'
for IMAGE in '${IMAGES}'
docker build -t taringa_base taringa_base
Sending build context to Docker daemon 2.048 kB
Sending build context to Docker daemon
Step 0 : FROM centos:centos6
---> 3bbbf0aca359
Step 1 : RUN yum -y install "http://pkgs.repoforge.org/rpmforge-release/rpmforge-release-0.5.3-1.el6.rf.x86_64.rpm"
---> Using cache
---> fab9afbebec5
Step 2 : RUN yum -y install "http://dl.fedoraproject.org/pub/epel/6/x86_64/epel-release-6-8.noarch.rpm"
---> Using cache
---> 86ecac389f5f
Step 3 : RUN yum install -y python-setuptools python-meld3 	&& easy_install pip
---> Using cache
---> 8345d756cee9
Step 4 : RUN pip install supervisor
---> Using cache
---> 9c98fed78644
Step 5 : CMD supervisord -n
---> Using cache
---> 4e7f6d56ff46
Successfully built 4e7f6d56ff46
for IMAGE in '${IMAGES}'
docker build -t taringa_base_nginx taringa_base_nginx
Sending build context to Docker daemon 2.048 kB
Sending build context to Docker daemon
Step 0 : FROM taringa_base
---> 4e7f6d56ff46
Step 1 : RUN yum --disableplugin=fastestmirror install -y 	curl 	gcc 	GeoIP 	GeoIP-devel 	openssl-devel 	patch 	pcre-devel 	tar
---> Using cache
---> 4ceb48f326a7
Step 2 : EXPOSE 8080
---> Using cache
---> 53cc1a878804
Successfully built 53cc1a878804
for IMAGE in '${IMAGES}'
docker build -t taringa_base_php taringa_base_php
Sending build context to Docker daemon 2.048 kB
Sending build context to Docker daemon
Step 0 : FROM taringa_base_nginx
---> 53cc1a878804
Step 1 : RUN yum -y install "http://rpms.famillecollet.com/enterprise/remi-release-6.rpm"
---> Using cache
---> 3f662e98f33b
Successfully built 3f662e98f33b
docker login -u autobot -p password -e user@domain <IP_ADDRESS>:5000
WARNING: login credentials saved in /var/lib/jenkins/.docker/config.json
Login Succeeded
for IMAGE in '${IMAGES}'
docker tag -f taringa_base <IP_ADDRESS>:5000/taringa_base
docker push <IP_ADDRESS>:5000/taringa_base
The push refers to a repository [<IP_ADDRESS>:5000/taringa_base] (len: 1)
4e7f6d56ff46: Image already exists
4e7f6d56ff46: Image already exists
9c98fed78644: Image already exists
8345d756cee9: Buffering to Disk
8345d756cee9: Image successfully pushed
86ecac389f5f: Image already exists
fab9afbebec5: Image already exists
3bbbf0aca359: Image already exists
fea77d2fd61e: Image already exists
91e6f84b8fe8: Image already exists
2c2557968d48: Image already exists
47d44cb6f252: Image already exists
Digest: sha256:f9bb4c341f02deb58f8d3a41911e65c7b804d70e025b6b3ca824d8901bfcf0ce
for IMAGE in '${IMAGES}'
docker tag -f taringa_base_nginx <IP_ADDRESS>:5000/taringa_base_nginx
docker push <IP_ADDRESS>:5000/taringa_base_nginx
The push refers to a repository [<IP_ADDRESS>:5000/taringa_base_nginx] (len: 1)
53cc1a878804: Image already exists
53cc1a878804: Image already exists
4ceb48f326a7: Buffering to Disk
4ceb48f326a7: Image successfully pushed
4e7f6d56ff46: Image already exists
9c98fed78644: Image already exists
8345d756cee9: Buffering to Disk
8345d756cee9: Image successfully pushed
86ecac389f5f: Image already exists
fab9afbebec5: Image already exists
3bbbf0aca359: Image already exists
fea77d2fd61e: Image already exists
91e6f84b8fe8: Image already exists
2c2557968d48: Image already exists
47d44cb6f252: Image already exists
Digest: sha256:e87d7dad819d7539a3b5b577ecb5b9d041f3b08453bfdbec1d76e098b81ce452
for IMAGE in '${IMAGES}'
docker tag -f taringa_base_php <IP_ADDRESS>:5000/taringa_base_php
docker push <IP_ADDRESS>:5000/taringa_base_php
The push refers to a repository [<IP_ADDRESS>:5000/taringa_base_php] (len: 1)
3f662e98f33b: Image already exists
3f662e98f33b: Image already exists
53cc1a878804: Image already exists
4ceb48f326a7: Buffering to Disk
4ceb48f326a7: Image successfully pushed
4e7f6d56ff46: Image already exists
9c98fed78644: Image already exists
8345d756cee9: Image already exists
86ecac389f5f: Image already exists
fab9afbebec5: Image already exists
3bbbf0aca359: Image already exists
fea77d2fd61e: Image already exists
91e6f84b8fe8: Image already exists
2c2557968d48: Image already exists
47d44cb6f252: Image already exists
Digest: sha256:dc1ef52e328ff27fecd37bbaf7c929eaf5fa7c6ee693a874cd6761e2add3e727

While we don't currently guarantee that re-uploads will not happen, this issue has been largely mitigated in 1.8 by storing metadata which can be used to reproduce a same upload content. If you still have issues with 1.8+ please re-open up the issue so we can attempt to reproduce test cases around this.  Thanks for your report.


Use html.EscapeString()

We can use html.EscapeString() instead of changing some symbols (section Security)

Frontend is ready - 444dbf6

Done with 985d800


[Question] Installation question

Hi!
I saw in previous questions/bugs that Pytorch CuDA and NVCC has to be the same, which is the case for me, however, I am still not able to run any step after nndet_example. What am I doing wrong?
This is my nndet_env:
----- PyTorch Information -----
PyTorch Version: 1.8.0+cu111
PyTorch Debug: False
PyTorch CUDA: 11.1
PyTorch Backend cudnn: 8005
PyTorch CUDA Arch List: ['sm_37', 'sm_50', 'sm_60', 'sm_70', 'sm_75', 'sm_80', 'sm_86']
PyTorch Current Device Capability: (7, 5)
PyTorch CUDA available: True
----- System Information -----
System NVCC: nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2020 NVIDIA Corporation
Built on Mon_Oct_12_20:09:46_PDT_2020
Cuda compilation tools, release 11.1, V11.1.105
Build cuda_11.1.TC455_06.29190527_0
System Arch List: 5.2 6.0 6.1 7.0 7.5 8.0 8.6+PTX
System OMP_NUM_THREADS: 1
System CUDA_HOME is None: True
System CPU Count: 40
Python Version: 3.8.5 (default, Sep  4 2020, 07:30:14)
[GCC 7.3.0]
----- nnDetection Information -----
det_num_threads 6
det_data is set True
det_models is set True

Dear @rooskraaijveld ,
thank you for your Issue, which error message do you receive?
Best,
Michael

Hmm, on first sight your env looks correct.

Try deleting the build folder and *.so file and re-install it
Try setting the CUDA HOME path explicitly

PS: always do (1) when running a new install, otherwise it won't recompile


Office Add-ins Community Call Blog - April 2024

Category

[ ] Content fix
[x] New article

Contents of the Pull Request
Office Add-ins developer platform community call blog – April 10, 2024
Topic included:
esign Best Practices for Office Add-ins presented by Skylar Pan, Product Manager2 at Microsoft,
Add-in authentication and access to Exchange APIs presented by Ben Siler, Senior Product Manager,
Implementing an integrated spam reporting add-in for Outlook presented by Eric Legault, Microsoft MVP.
Call hosted by Mingjia Liu, Product Manager at Microsoft. Recorded on April 10, 2024.

do you need help publishing this one?


 We can than omit the apply{} or with{}.
Current Behavior
TaskProvider.configure don't return itself but void.
Context
I want to configure a lazy task directly and use the returned TaskProvider in another task (as dependency).
First register all the task and later call configure on them feels wrong somehow:
val donwloadStrings = tasks.register("updateStrings")
val updateStrings = tasks.register("updateStrings")

downloadStrings.configure {
   // Configure
}
updateStrings.configure {
  dependsOn(donwloadString)
}

Your Environment
Gradle 4.10-rc-3
IntelliJ IDEA 2018.2.2 (Community Edition)
Build #IC-182.4129.33, built on August 21, 2018
JRE: 1.8.0_152-release-1248-b8 x86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
macOS 10.13.6

Although I see this could be valuable, I fail to see how it's valuable in this specific use case. You can pass a configuration closure/lambda/action during the registration like:
val downloadStrings = tasks.register("downloadStrings") {
   // Configure
}
val updateStrings = tasks.register("updateStrings") {
    dependsOn(downloadStrings)
}

The question would be why did you feel you needed to register all the task and later configure them? That would be a failure to properly communicate on our part.

Hey @lacasseio
that is true. I missed that fact the we can configure it directly in the register() method 🙃
I just replace "everything" to that code. Works fine 👌
The only thing where it's missed is the overload where you can add arguments to the task.

At lest for this it would be valuable.

The question would be why did you feel you needed to register all the task and later configure them?

Maybe i described it wrong. I don't want to do that 🙃 But that is a "workaround" to don't add apply or with.
So everything is good here 👍
Feel free to close this issue if you don't want to return the TaskProvider on configure. I'm fine with the current API :)

That would be a good solution, yes 👍 😀

BTW, if you go that way, I'd suggest providing a Kotlin extension using vararg:
fun <T: Task> TaskContainer.register(name: String, type: KClass<T>, vararg constructorArgs: String, configuration: T.() -> Unit): TaskProvider<T> =
    register(name, type.java, constructorArgs).apply { configure(configuration) }

that could be called as:
tasks.register("someTask", CustomTask::class, 42, 52.0) { /* … */ }

(note that as expressed above, it doesn't even use the proposed new API, so could be added to kotlin-dsl right now; otherwise it could use the proposed new API by wrapping the vararg into a list; you would probably want to add the corresponding registering delegate, and maybe a reified variant)

I opened issue https://github.com/gradle/gradle-native/issues/835 to track the proposed work for a register overload.

I'm confused. Why have you moved that to the native repo? 🤔
Isn't that a "Gradle core" thing and needs to implemented in this repo?


Added prop for passing config object to babel

Allows configuring specific features that should or should not be
transpiled, and lets you enable stage: 0 features like class property
initializers

Looks great, let me test this before merging

Awesome! and thanks again for the excellent component

any chance any of these will ever get merged? No rush of course, but we are trying to figure out whether to just publish our forked version.


Update types in types/index.d.ts

Description
The current types trigger an error:
node_modules/react-sortable-hoc/types/index.d.ts:108:11 - error TS2694: Namespace 'React' has no exported member 'SFC'.

108   | React.SFC<P>
              ~~~

Indeed, React.StatelessComponent has been removed from the react types (see https://stackoverflow.com/questions/44375759/how-should-i-declare-a-stateless-functional-component-with-typescript-in-react)
This PR fixes this error.

Hello @clauderic! Could you take a look at this PR? 🙂

I also just stumbled across this problem.
(Explicitly passing the prop type to SortableElement<P> and SortableContainer<P> helps as a workaround)
Having this update would still be nice though!

Hi @clauderic, any chance you can have a look at this PR?

For users with skipLibCheck: false, it will allow using this library
For users with skipLibCheck: true, it will catch misuses of the library! Else component props are typed as any 😞

🙏


[FEA] support GpuHashJoin on structs containing floats

Is your feature request related to a problem? Please describe.
We can not support GpuHashJoin on structs who contain child of float type, because the catalyst rule NormalizeFloatingNumbers transforms join expressions, whose data type is struct type containing float children, to pattern KnownFloatingPointNormalized(If(IsNull(expr), Literal(null, struct.dataType), struct)). And the expression Literal(null, struct.dataType) inside the pattern can not run on GPU, since cuDF scalar of StructType is yet supported.
Describe the solution you'd like
This issue is relied on cuDF issue #7990.

This is join key with structs? the linked issue is Scalar support so is this just with Scalars? please clarify and if possible add example.

This is join key with structs? the linked issue is Scalar support so is this just with Scalars? please clarify and if possible add example.

Hi Tom, I added an example to illustrate the problem.


Applies .gitignore too much

I have a .gitignore that is ignoring a few specific files a the root of my repo, as in:
foo.js
bar.js

These are named the same as files under src/. Given this .gitignore, Git will ignore the files at the root but not those under src (which is what I want). Git Plus however greys out the files under src/ by matching them too eagerly.

The problem is actually with treeview, sorry!


Rust support/sdk

The readme said to make an issue requesting a language, so here it is.
Is this a maybe/yes/no?

I quickly added the compiled rust flatbuffer files: https://github.com/open-meteo/sdk/blob/781b2078cd0d1c79d14a10afb5951ae0140a54a8/rust/weather_api_generated.rs
The API returns multiple length-prefixed flat buffer messages. To decode a message, take the first 4 bytes to get the length of a message, then decode it using flatbuffers and repeat:
https://github.com/open-meteo/sdk/blob/main/swift/Sources/OpenMeteoSdk/OpenMeteoSdk.swift#L84-L94
Do you have experience in using HTTP clients in Rust and could give it a try? Afterwards this could be bundled into a crate.


Add guide on how to read the DG at the top of the DG

-[ ] What do _ italics_ mean.
-[ ] What does AddCommand#undo() mean.


EUI Organ Info Y3 cosmetic update

EUI Organ Info Y3 cosmetic update
Two layouts in HuBMAP and HRA styles are currently under review:

Organ Info Y3

Ready for dev: ccf-organ-info
Our cosmetically redesigned organ-info component is ready for dev. Please reach out with questions as needed!

Figma: Approved Specification

Updated Interaction Details

A menu with the same listings that we currently have appears on click of the HRA Resources button:


Stack traces

Currently it's quite hard to know what bit of code causes a failure in Umlaut's tracing functionality.
For example, I have the following:
ERROR: AssertionError: Expected Pair{IRCode,...}, but got Pair{Method, DataType} instead for f=arrayref with argtypes=(Expr, Vector{Any}, Int64)
Stacktrace:
  [1] getcode(f::Any, argtypes::Any)
    @ Umlaut ~/ml/ad_playground/Taped.jl/dev/Umlaut/src/trace.jl:247
  [2] trace!(t::Umlaut.Tracer, v_fargs::Any)
    @ Umlaut ~/ml/ad_playground/Taped.jl/dev/Umlaut/src/trace.jl:478
  [3] trace_call!(::Umlaut.Tracer{TapedContext}, ::Any, ::Vararg{Any})
    @ Umlaut ~/ml/ad_playground/Taped.jl/dev/Umlaut/src/trace.jl:297
  [4] trace_block!(t::Umlaut.Tracer, ir::Core.Compiler.IRCode, bi::Integer, prev_bi::Integer, sparams::Any)
    @ Umlaut ~/ml/ad_playground/Taped.jl/dev/Umlaut/src/trace.jl:334
  [5] trace!(t::Umlaut.Tracer, v_fargs::Any)
    @ Umlaut ~/ml/ad_playground/Taped.jl/dev/Umlaut/src/trace.jl:487
  [6] trace_call!(::Umlaut.Tracer{TapedContext}, ::Any, ::Vararg{Any})
    @ Umlaut ~/ml/ad_playground/Taped.jl/dev/Umlaut/src/trace.jl:297
  [7] trace_block!(t::Umlaut.Tracer, ir::Core.Compiler.IRCode, bi::Integer, prev_bi::Integer, sparams::Any)
    @ Umlaut ~/ml/ad_playground/Taped.jl/dev/Umlaut/src/trace.jl:334
  [8] trace!(t::Umlaut.Tracer, v_fargs::Any)
    @ Umlaut ~/ml/ad_playground/Taped.jl/dev/Umlaut/src/trace.jl:487
  [9] trace(::Any, ::Any, ::Vararg{Any}; ctx::Any, deprecated_kws::Base.Pairs{Symbol, V, Tuple{Vararg{Symbol, N}}, NamedTuple{names, T}} where {V, N, names, T<:Tuple{Vararg{Any, N}}})
    @ Umlaut ~/ml/ad_playground/Taped.jl/dev/Umlaut/src/trace.jl:609
 [10] trace_shapes(f::Function, x::Vector{Any})
    @ Taped ~/ml/ad_playground/Taped.jl/src/tracing.jl:46
 [11] top-level scope
    @ REPL[30]:1

It tells me that arrayref is the problematic function, with a particular set of argument types. It would be really helpful if this kind of error generated an additional bit of information that shows where in the code that Umlaut is interpretting we currently are.

Perhaps you want the (undocumented) function Umlaut.print_stack_trace(), which prints the stack trace so far. Or, if you want to dive right to the middle of tracing, you may try Umlaut.get_latest_tracer_state(), which returns an instance if Tracer, IR of the currently traced function and a tuple of argument Variable's.
In any case, this is a tracing bug and shouldn't happen in normal circumstances. I can reproduce it using exactly the information from the error message:
julia> f = Base.arrayref
arrayref (built-in function)

julia> argtypes=(Expr, Vector{Any}, Int64)
(Expr, Vector{Any}, Int64)

julia> getcode(f, argtypes)
ERROR: AssertionError: Expected Pair{IRCode,...}, but got Pair{Method, DataType} instead for f=arrayref with argtypes=(Expr, Vector{Any}, Int64)
Stacktrace:
 [1] getcode(f::Any, argtypes::Any)
   @ Main ~/work/Umlaut.jl/src/trace.jl:247
 [2] top-level scope
   @ REPL[19]:1

Apparently, Base.code_ircode(f, argtypes; optimize_until="slot2reg") for Base.arrayref returns Pair{Method, ...} instead of Pair{IRCode, ...}. Maybe, I can extract the IR from that Method, but I wonder why you even got to tracing the Base primitive, because usually functions in Base are recorded to the tape as is. Do you use a custom context that doesn't re-use BaseCtx? Can you share a bit more details about your use case?

Do you use a custom context that doesn't re-use BaseCtx? Can you share a bit more details about your use case?

Exactly -- I'm intentionally playing around with what I treat as a primitive. Re my application -- I'm not too worried about being able to solve this bug per-se, I'm more concerned about it being tricky generally to know where in code tracing has broken (if it breaks).
Thanks for pointing me towards Umlaut.print_stack_trace() -- I'll take a look at it

The main issue as I see it is that you try to trace through a built-in function. Such functions don't behave as usual - the are implemented in C or a low-level subset of Julia and often don't have IR representation that we can trace. Unless you want to dive deep into the Julia compiler and runtime, the only was to avoid such issues is to mark these functions as primitives. Which is as simple as:
Umlaut.isprimitive(YourCtx(), Base.arrayref, args...) = true


Clarification on how to get a list of domains associated with fingerprinting

I've been working through the various data options trying to compile a list of domains that classify as fingerprinting. I'm getting mixed results and wondering if you can clarify, what you think of as the canonical approach.
Apologies if I'm just misreading the documentation. I'm happy to submit a PR to docs if you think it would be useful.
I can use the data source, and get a list of tracker ids as follows
fp_trackers = set()
regions = {'de', 'eu', 'fr', 'global', 'us'}
for region in regions:
    who_tracks_data = DataSource(region=region)
    who_tracks_fp = who_tracks_data.trackers.df[who_tracks_data.trackers.df.bad_qs > 0.1]
    fp_trackers.update(list(who_tracks_fp.tracker.values))

This gives me 193 trackers. I can then map this to domains using the map from create_tracker_map.
could_not_find = []
domains = set()
for tracker in fp_trackers:
    try:
        domains.update(tracker_info['trackers'][tracker]['domains'])
    except KeyError:
        could_not_find.append(tracker)

This will give me 326 domains.
If I take a different route, and read in all the csv files under assets folders labeled domains.csv, I can get a list of domains like this
domains_df = pd.concat([
    pd.read_csv(file, parse_dates=['month'])
    for file in asset_paths['domains'] # I have previously assembled all the paths
])
fingerprinting_trackers = domains_df[domains_df.bad_qs > 0.1].host_tld.unique()

But this gives me a list of 292 domains.
I can think of an explanation for this - not all host_tld's might have a bad_qs that meets the threshold but they've been added to the tracker map for other reasons.
However, given that the other csv files may also be relevant, I was starting to lose confidence and so wanted to check in.
Many thanks in advance for your help.

The domains.csv and trackers.csv files represent different aggregations of the same data. If we consider the fingerprinting case:

domains.csv counts the proportion of times when each hostname (at TLD+1 level) was seen sending a fingerprint (or suspected fingerprint) in a third-party context on a page.
trackers.csv counts the proportion for any of the hostnames associated with a tracker - from the mapping in the tracker database.

For the majority of trackers the relationship between domains and trackers is one-to-one. For others the domains files will show to which domains fingerprinting data is sent, while the trackers view shows a more aggregated picture of what the tracker is doing.
For example, Facebook uses facebook.net as a CDN, and we can see from the stats little evidence of tracking on this domain. The tracking requests are aimed at facebook.com where they have the user's login cookie. In the tracker view we report the aggregate view of both domains, which shows the aggregate view of Facebook's third-party traffic.
I hope that clears things up a little for you. From your use-case it looks like the domains.csv data view would fit better.

Hi @birdsarah, many thanks for the PR and issues raised. domains.csv is currently not exposed via the API. If you'd find this useful, you can add this to loader.py, extending our API.  Here's one way to do it:
class Domains(PandasDataLoader):
    def __init__(self, data_months, region="global"):
        super().__init__(data_months, name="domains", region=region)

then add this to class DataSource, still on loader.py
       ...
        self.domains = Domains(
            data_months=self.data_months,
            region=region
        )

Now you can consume domains via the DataSource:
data = DataSource(region="global")
domains = data.domains.df

where domains would be a pandas dataframe of all months for which domains.csv is available.

Thanks so much for this feedback @sammacbeth @ecnmst. This is extremely helpful.
I'll leave this open and plan to make the addition to loader.py that @ecnmst proposes.

But if, on reflection, you don't want the update to loader.py feel free to close the issue.


Looking to use OpusMSEncoder with baresip for multichannel audio call

Any hints on how I could implement the multistream/multichannel version of the opus codec in baresip so that I can configure number of channels and other parameters relevant to the OpusMSEncoder?
I've tried to find any related info here and elsewhere, but have not found anything.

hi @tonylangley
what is your use-case for Opus Multistream ?
the opus module supports normal opus, but not opus multistream.
the libopus API is different so we would have to implement it either in
opus.so or opusms.so
I think the FFmpeg project supports Opus Multistream.
is there a large interest for this? if so we could add it to the Roadmap.
Alfred

Hi @alfredh
Thanks for having a look at my query.
I'm interested in enabling at least a 4-channel networked music performance bi-directional, session-initiated "call" between compatible parties.
I've had some success with integrating OpusMSEncoder into liblinphone, but that framework is much heavier than what I'm looking for (on a Beagleboard-xM Beaglebone Black, or Beagleboard-X15).
For my use-case I need all 4 channels to be closely sample-locked, i.e. no significant drift between channels 1&2 and 3&4.
I can't determine whether there's large interest in this capability. But I do advocate for more than just stereosonic interactions, particularly when there are many people involved in a conference call and directional audio is important for intelligibility.
I'm guessing that more than just the opus module would need to be updated to enable 4-channels to work end-to-end through baresip - right?
Tony

This sounds like a fun project :)
Personally I have tested channel count of 1 and 2 in baresip, which is working.
The channel count is just an 8-bit unsigned integer, meaning that in theory up
to 255 channels can be supported (is that enough for Frank Zappa and Pink Floyd ?)
For example here is the audio-source parameters:
/** Audio Source parameters */
struct ausrc_prm {
	uint32_t   srate;       /**< Sampling rate in [Hz]      */
	uint8_t    ch;          /**< Number of channels         */
	uint32_t   ptime;       /**< Wanted packet-time in [ms] */
	int        fmt;         /**< Sample format (enum aufmt) */
};

Channel mapping is not supported.
For example if your sound-card supports 4 channels, you could just test
the current code and see how it works:

audio source with 4 channels (e.g. alsa)
audio codec with 4 channels (e.g. opus or L16 hacked)

static struct aucodec opus = {
	.name      = "opus",
	.srate     = 48000,
	.crate     = 48000,
	.ch        = 2,               /* <--- change this to 4 */
	.pch       = 2,
	.fmtp      = fmtp,
	.encupdh   = opus_encode_update,
	.ench      = opus_encode_frm,
	.decupdh   = opus_decode_update,
	.dech      = opus_decode_frm,
	.plch      = opus_decode_pkloss,
};

hope this helps!
Alfred

hi @tonylangley
I have added the multistream info to the roadmap:
https://github.com/alfredh/baresip/wiki/Roadmap
does the previous post answer your question ? if so please be so kind
to close the ticket.

Hi @alfredh
Thanks for your reply. Your suggestions of how to get this working have helped :)
I'm currently attempting to integrate an opus_multistream module, ideally so that it can be enabled/disabled via the config. I'm basically copying modules/opus to modules/opus_multistream, refactoring all the opus calls to the multistream variant and updating mk/modules.mk to include this new module. The opus_multistream API requires additional parameters for streams, coupled_streams and channel_mappings in (at least) the opus_multistream_encoder_create and opus_multistream_decoder_create calls, so I'm not yet sure how best to incorporate these into the config (as they're only needed for multistream i.e. > 2 channels) and make them available to modules/opus_multistream.
Happy to close, as I think I have all I need to try and get this working.
Thanks again :)


V2 - Company Not Found - Internal Server error displayed due to CORS issue while identifying CLA Manager with LFID and associated to another company

Prerequisite

Create a CLA group with CCLA enabled (I tried the flow where CLA Group is for standalone project which does not have any parent or any child projects)
Add github repository for the CLA group

Steps to Follow

Login to Github as corporate contributor
Navigate to the repository used in prerequisite
Create PR
CLA check fails
Click on CLA NOT SIGNED
Contributor is redirected to Contributor Console
Click on Proceed as a Corporate Contributor
8.Click on building icon
Click on Organization not listed? + Click Here
Enter new company details
Click Next
Click Proceed
Click No
Enter name and email of user has LFID and has contributor role for another company and project
Observed that Internal Server error is displayed

Observed that CORS error is  not displayed. Will reopen the bug If issue reoccurs again


uninitialized constant RSpec::Support (NameError)

I have upgraded from rspec-rails version 2.99.0 to 3.0.1. but when ever I try to run any command, it is giving following error.

/home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/rspec-core-3.0.2/lib/rspec/core/formatters/documentation_formatter.rb:1:in <top (required)>': uninitialized constant RSpec::Support (NameError) 	from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/stopwatch_formatter-1.0.0/lib/stopwatch_formatter.rb:1:in <top (required)>'
from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/bundler-1.6.2/lib/bundler/runtime.rb:76:in require' 	from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/bundler-1.6.2/lib/bundler/runtime.rb:76:in block (2 levels) in require'
from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/bundler-1.6.2/lib/bundler/runtime.rb:72:in each' 	from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/bundler-1.6.2/lib/bundler/runtime.rb:72:in block in require'
from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/bundler-1.6.2/lib/bundler/runtime.rb:61:in each' 	from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/bundler-1.6.2/lib/bundler/runtime.rb:61:in require'
from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/bundler-1.6.2/lib/bundler.rb:132:in require' 	from /home/rahul/workspace/org/sample_app/config/application.rb:8:in <top (required)>'
from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/railties-4.1.4/lib/rails/commands/commands_tasks.rb:79:in require' 	from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/railties-4.1.4/lib/rails/commands/commands_tasks.rb:79:in block in server'
from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/railties-4.1.4/lib/rails/commands/commands_tasks.rb:76:in tap' 	from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/railties-4.1.4/lib/rails/commands/commands_tasks.rb:76:in server'
from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/railties-4.1.4/lib/rails/commands/commands_tasks.rb:40:in run_command!' 	from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/railties-4.1.4/lib/rails/commands.rb:17:in <top (required)>'
from /home/rahul/workspace/org/sample_app/bin/rails:8:in require' 	from /home/rahul/workspace/org/sample_app/bin/rails:8:in <top (required)>'
from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/spring-1.1.3/lib/spring/client/rails.rb:27:in load' 	from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/spring-1.1.3/lib/spring/client/rails.rb:27:in call'
from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/spring-1.1.3/lib/spring/client/command.rb:7:in call' 	from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/spring-1.1.3/lib/spring/client.rb:26:in run'
from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/spring-1.1.3/bin/spring:48:in <top (required)>' 	from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/spring-1.1.3/lib/spring/binstub.rb:11:in load'
from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/spring-1.1.3/lib/spring/binstub.rb:11:in <top (required)>' 	from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in require'
from /home/rahul/.rbenv/versions/2.1.2/lib/ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in require' 	from /home/rahul/workspace/org/sample_app/bin/spring:16:in <top (required)>'
from bin/rails:3:in load' 	from bin/rails:3:in '

Related issue (for googlers following the trail): https://github.com/grosser/rspec-instafail/issues/16


Increase default memory limit for the prometheus container

What kind of request is this (question/bug/enhancement/feature request): enhancement
Steps to reproduce (least amount of steps as possible):
Deploy prometheus in a cluster with a number of nodes / workloads
Result:
The prometheus container is often the highest memory consumer, the current 1GB memory limit for the container can introduce OOM kill and crash loop backup issues for the pod intermittently, or constantly if the prometheus container memory needs are high for the environment.
Other details that may be helpful:
We do document that resources assignments are a guide, however, with higher defaults, this could reduce the surprise as clusters grow organically and require a higher limit.
A higher default memory limit, while keeping the same memory request will reduce the likelihood of prometheus being killed (limit), while maintaining the same memory needs for scheduling (requests) and deploying monitoring. Additionally, a tip or warning in the UI may help with this, if the current memory limit is to be kept.
gz#11501
gz#12420

https://github.com/rancher/rancher/issues/27865

@aiyengar2 What do you think? ^

FYI - I can share the same experience with rancher-monitoring deployments on top of RKE2 where prometheus runs into OOM due to "too small limits" probably during higher memory requirements during retention. Also the prometheus-node-exporter has this problem i.e. in case the deployment is on bare-metal - assuming the number of local hardware components play a role in memory usage, there.
For the moment I have doubled the defaults in rancher-monitoring v14.5.100
prometheus:
  prometheusSpec:
    resources:
      limits:
        memory: 3000Mi
    requests:
        memory: 1500Mi

prometheus-node-exporter:
resources:
limits:
memory: 100Mi
requests:
memory: 60Mi
P.S. a similar problem exists with rancher-logging where we had to increase the memory for fluentbit
P.P.S. the clusters I am dealing with are very small - most often 3 masters and less than 20 workers.


chore(repo): Deprecate node@{14,15,16,17} versions

Description
Some information about changes for all those node versions
Node 15 : https://platform.sh/blog/nodejs-15-could-make-your-app-leaner-and-meaner/
- Logical assignment operators
- String.prototype.replaceAll
- Promise.any
- AggregateError
- AbortController
- timers/promises
- npm 7
- Workspaces
- New package-lock.json format

Node 16 : https://medium.com/the-node-js-collection/node-js-16-available-now-7f5099a97e70
- Stable Timers Promises API
- Experimental implementation of the standard Web Crypto API
- npm 7 (v7.10.0 in Node.js v16.0.0)
- Node-API version 8
- Stable AbortController implementation based on the AbortController Web API
- Stable Source Maps v3
- Web platform atob (buffer.atob(data)) and btoa (buffer.btoa(data)) implementations for compatibility with legacy web platform APIs

Node 17 : https://blog.appsignal.com/2021/10/27/what-s-new-in-nodejs-17.html
- New Promise-based APIs (dns, fs, stream, and timers)
- Stack Traces Now Include the Version of Node.js
- OpenSSL 3.0 Support
- V8 Upgraded to v9.5 : https://v8.dev/blog/v8-release-95
    - Intl.DisplayNames
    - WebAssembly Exception Handling

Node 18 : https://nodejs.org/en/blog/announcements/v18-release-announce
- V8 Upgraded to 10.1
    - Array: findLast and findLastIndex
- Built-in Test Runner
- New Browser-compatible APIs
    - fetch
    - Streams
    - Blob - https://nodejs.org/api/buffer.html#class-blob
    - BroadcastChannel - https://nodejs.org/api/worker_threads.html#class-broadcastchannel-extends-eventtarget

Checklist

[x] npm test runs as expected.
[x] npm run build runs as expected.
[x] (If applicable) JSDoc comments have been added or updated for any package exports
[x] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[x] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/clerk-js
[ ] @clerk/clerk-react
[x] @clerk/nextjs
[x] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[x] @clerk/backend
[x] @clerk/clerk-sdk-node
[ ] @clerk/shared
[x] @clerk/fastify
[ ] @clerk/chrome-extension
[x] gatsby-plugin-clerk
[ ] build/tooling/chore

It would be good to add the engines field to all packages, e.g. shared is missing that information
For our reasoning why we chose Node 18 we should use https://github.com/nodejs/release#release-schedule for our changelog. We'll just follow the LTS plan

The reason we chose Node 18 for the v5 release is outlined here: https://nodejs.org/en/blog/announcements/nodejs16-eol?mkt_tok=MDAxLUxSRy0xOTQAAAGMUL0csiUbl5CDLnZeh6VUFzAi-55IRjUjdJFK08bCgi-8udk1gxib7nWjYJZzdt-f7Y8YRS_MxsvPa9_gml4IeZxVmqc2TDPcO-KNkEyr

@LekoArts : I will update the @types/node, engines and the README to ^18.18.0 everywhere since the v18 has entered maintenance mode and there probably won't ever be a v18.19.

@LekoArts changes applied, ptal.


connectionHandler$tbl() compatible with DatabaseConnector dbplyr

Should ConnectionHandler create dbplyr tables in the same way that DatabaseConnector
Replacing
https://github.com/OHDSI/ResultModelManager/blob/e513ba2d9bc4dbe6460b2f793628379cecd782ed/R/ConnectionHandler.R#L67
with
table <- DatabaseConnector::inDatabaseSchema(databaseSchema, table)

Also, wouldn't it make more sense to have connectionHandler object in DatabaseConnector ???
i think  connectionHandler is very handy

Thanks Javier - yes the table could use the DatabaseConnector functionality. Note that these classes actually pre-date the DatabaseConnector dbplyr support and so some code will look like this. I would gladly accept a PR if someone wished to write one ;) to change something like this to use the DatabaseConnector standard.
In terms of the use of connectionHandelers in DatabaseConnector, this is really up to Martijn who maintains that package. Though I imagine that they could be useful they are still quite immature and the main principle of DatabaseConnector is to maintain stability and interoperability first.  Whereas this package is may change more frequently. Though I don't intend to break the API a lot of the stuff included in here are utilities for shiny apps and markdown reports I have worked on, rather than analytics packages that we need to send to sites to run.
As these utilities mature we can look at moving them into DatabaseConnector.


a lot of commits to reconcile dev & main

there are a lot of commits here to catch up main with dev.
i didnt know you couldnt easily (relative term) pick individual commits from a pull-request .
If you can quickly review and approve this monster, in the future , i will work off a different branch and also make smaller , more frequent pull -requests so it's easier to manage.

@olanderb I would advise for the future each indicator under development gets its own branch and therefore PR.
I will work to define a workflow... But I get we (or at least I) also need to prioritize vs. perfectionism...


CORE-2542 added support for specifying clob / blob in loadData

Merged into 3.5 since it's more than I'd like to do in a 3.4 patch release. Thanks!


Terminal Output disappears after switching tabs

When switching the tabs and then back to terminal the output is gone. I also saw this in Codel

Could I take the task?

I prepared a proposal of fixing the issue: #604


Review of the Status API

This issue has two goals:

Draft developer-oriented documentation about what an ophyd Status object is and the intent behind its design.
Propose possible extensions of that API to be tackled in one or more PRs.

As we build out the documentation, it may be useful to revisit these early design choices and understand them better, even if we elect not to change or extend anything. This could be also be useful for future work around handling failure (e.g. alarm states) better.
Comparison between Status and Future
Ophyd Status objects are like concurrent.futures.Future objects but with some semantics specific to controlling physical hardware. How do they compare?

Both have methods for adding callbacks that get called once upon completion (success or failure); Status.add_callback(f) and Future.add_done_callback(f). A Status objects calls f() with no arguments; a Future calls f(future) with a reference to itself.
Futures can be canceled until the executor begins the associated work and marks them as un-cancel-able. Status objects can never be cancelled.
Many (not all) Status objects have a reference to the Device that they are moving. This enables stopping an action when it is already in progress, status.device.stop(), which is something a Future cannot do.
Futures have results which are set by the executor via Future.set_result(result) and collected by the caller via Future.result(timeout=None). In the event of failure, an exception is set via Future.set_exception(exc), which can be collected via Future.exception(timeout=None). Calling Future.result() on a failed Future raises the exception.  They can log exceptions but do not provide a way to access them programmatically. They provide a boolean success property.
Status has a done read-only property; Future has a done() method with the same meaning.
Both have a concept of timeout but with different meanings. The Status object takes in timeout at __init__ time and keeps its own clock starting from __init__. A Future leaves it up to the caller of result() or exception() to provide an optional timeout, and the clock starts when the method is called.
Status also has a concept of settle_time, a constant time added after the underlying task has completed and before it reports being done.
Status has special methods for reporting incremental progress in order to support progress bars.
Status supports __and__.
Future takes no arguments at __init__. Status takes in timeout and settle_time, which is needs to do in order to support its different meaning of timeout. It also takes in done and success at __init__.

Possible Changes

Should we change the expected signature to callbacks from f() to f(status)? We made an analogous change in https://github.com/caproto/caproto/pull/547, and the result was nice. Crucially, it can be done in a backward-compatible way by inspecting the signature of f when it is registered. If we done this, the bluesky RunEngine can be simplified because it will not need to keep a separate record of the mapping between callbacks and status objects.
Does cancellation make sense? I think it could be done: just as with Futures, there is point past which the action of a Status cannot be canceled, but it could be canceled up until the Write[Notify]Request is sent to the IOC. But I think it would not be as useful in our context as it is in the general Futures context. The difference is, Futures typically spend time waiting in a queue for a resource-bound Executor, whereas Status objects have their underlying messages sent quickly. At most, we should wait until we have a compelling reason to need this.
Should we add set_exception(exc) and exception(timeout=None)? Seems pretty useful! We might slowly deprecate the method _finished(success=True) in favor of new methods set_finished() (akin to set_result(result)) and set_exception(exc)) so that failure always comes with an exception giving information about the cause.
Currently, the caller has to check success and done to know the state of the Status. (The success attribute is initialized to False, "guilty until proven innocent". This has been a point of confusion in the past.) Would it be useful to add a method called wait(timeout=None) either returns None (successfully completed), raises WaitTimeoutError (didn't complete within the timeout in the method call but still running), or raises an exc if an exception has been set? This would be analogous to Future.result(timeout=None) but without any result to return.
The __and__ method is an interesting case. Why doesn't concurrent.futures need something like this? I think it's because we require Device.set to return exactly one Status object. Therefore, in order to communicate, "Wait for this and that and also that," we need to build a Status object that composes several status objects with logical AND, and return that. If we allowed Device.set to return a list of Status objects, we could do something like executor.gather on the caller side, and there would be no need for __and__. It would also give progress bars more ability to understand what is going on. (This could be even more important for the telemetry work.) Should we extend the expected interface of set (and friends) to accept list of Status objects as well as just one?

I think (1), (3), and (4) may be worth doing now. I'm not convinced we ever need to do (2). I think (5) could be important but needs careful thought because it interacts with may possible future plans and needs.

Food for thought @MikeHart85 @klauer @hhslepicka @ZLLentz @tacaswell and anyone else interested....I have bounced these ideas off of anyone yet, just got thinking about the Futures API this morning. :- )

:+1: to changing the signature, would make state management much easier
:-1: to adding a 'cancel' to the status object, cancellation is better handeled else where on the devices (who then mark the sattus objects as failed)
:+1: to adding set_exception and 75/25 on moving to set_finished
+.75 on the wait.  We talked about that and rejected it early on (there is wait helper function though iirc?) .  My vague memory is that it was something about blocking semantics, but I don't recall clearly and we understand all of this stuff much better now so worth re-considering
There are external functions for "wait for all of these futures" and "wait for the first of these futures" that do AND and OR.  I am skeptical of expanding the return of set to be "one or a list" as the type stability seems a high cost to pay for slightly better progress bars (I suspect we could get the improvement on progress bars by tweaking that API to always be a list-like rather than scaler like).

More than once I've wished we had something like numbers 3 and 4 in that possible changes list.

Closed by #836 and #837


BuiltinHint: implement print

We need to implement print utils in the context of our builtin hint processor.
Rust reference: https://github.com/lambdaclass/cairo-vm/blob/main/vm/src/hint_processor/builtin_hint_processor/print.rs

@tcoratger @lana-shanghai Hi, could I work on this issue?

@tcoratger @lana-shanghai Hi, could I work on this issue?

@tcoratger @lana-shanghai Hi, could I work on this issue?

assigned


prerender and CORS preflight check

After adding prerender_, the CORS preflight check errors began appearing. I would rather not have to disable CORS checks. I presume to be not an outlier.

@janat08 Can you provide more details around this issue? It would help us a lot if could detail the steps to reproduce the issue.

I might have been working in cloud ide to trigger cors, as they proxy local host.

Closing due to lack of reproducible steps.

@janat08 In case you feel that this is still an issue, do let us know. We'll be more than happy to investigate.


Adding coverage flag in Makefile

refering #67
Next step is to add codecoverage indicator in README using an external tools to have a global view (similar to https://codecov.io/gh/kubernetes/minikube)
This PR aim to improve developer experience by be able to measure code coverage locally.

/lgtm


Softban avoid

This commit includes a workaround for soft bans by spinning pokéstops repeatedly.

This PR is completely messed up.

Yeah going to resubmit it just noticed its mess


Replace os.date os.time

Any 3rd library?

no strong need for this replacement. os.date is not used for runtime for now.


Should global queue state documents be held in a specific queue state table?

Should queue state documents be held in a specific table?  This way you can index on queue name and not hard-code the document id.  Also, if someone does delete() on a queue/table then the worker processes will lose changefeed to the state document and subsequent reviews will not trigger queue processing.  Could be avoided if the state document is held in it's own table.

PK can be the queue name to restrict duplicates.  Is there a big reason to keep this document in the individual queue table?

All valid concerns @TomKaltz, however they are all irrelevant.
The Queue object change feed is for the whole queue table, not the state document. Change events are processed within the Queue object.
If I didn't have a fixed id on the state document, then I would need to put an identity hash or field on the state document. Why do that when you can use the id?
The state document is never read. It is only ever written to.
The state document write module is using an insert statement with a  { conflict: 'replace' } option. This means it will create the document if it does not exist, and if it does it replaces the current document.
State document deletions are ignored.
Read more here: https://github.com/grantcarthew/node-rethinkdb-job-queue/blob/master/src/queue-state.js
And here: https://github.com/grantcarthew/node-rethinkdb-job-queue/blob/master/src/queue-change.js
Nice work though @TomKaltz, keep it up. You have found a few design issues since you started pocking around. Keep on poking.


Upgrade all the CUBE .loc file to the latest version of cube

https://github.com/Quicr/hactar/commit/fb63b7ada19c87fa005101680e4ceedf8c9e7fd1


 No matter what respawn time I set. With or without sentry trait.

I doubt this is exclusive to the hurt command.

What traits does the NPC have? What are these traits set to?

Is this still an issue?

This should be fixed by using the cause argument.


add test case 4 soul-admin vo

// Describe your PR here; eg. Fixes #issueNo
add test case 4 soul-admin vo #653

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

It`s my first PR, so happy~


Fix #150: Update demo

Still work in progress. Hopefully, will merge it this weekend.
Planning to update docs also.

Coverage remained the same at 91.69% when pulling f5bfcb514d6724c56901190438afdfebb23f60c0 on update-demo into a0d61004a2d99c8206f1311d399644c49cfdfa72 on master.

Coverage remained the same at 91.69% when pulling f5bfcb514d6724c56901190438afdfebb23f60c0 on update-demo into a0d61004a2d99c8206f1311d399644c49cfdfa72 on master.

Coverage remained the same at 91.05% when pulling aa5d359c84094ae48ea95a53b47af24925a0ddff on update-demo into 02756beaa1ab679965cdb29df5dc8ef04c77d92c on master.


[ALLUXIO-2749] Make AlluxioWorkerInfoTest final

To avoid accidental subclass extension

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15623/
Test FAILed.

Merged build finished. Test FAILed.


The redirect is set in the mobile version

Good day.
In the script pH7Builder (formerly pH7CMS) when entering from a mobile phone, it will be redirected to another site. Perhaps a redirect is hidden in the site code.
Tell me how to check? The temporary address of the site jobdelivery.ru

Hi @obmen86
Hope you are doing well :smiley:
Your issue looks really similar to this one https://github.com/pH7Software/pH7-Social-Dating-CMS/issues/395

So, it looks you enabled the Adult Disclaimer (normally, it's disabled by default), which is a third-party service and may gives some kind of ads/redirection as mentioned on the Settings Page.

Once you log in, please disable it in order to avoid this issue again (Admin Panel -> Settings -> Adult Disclaimer).

Hope it helps!
By the way, if you like the software, do you mind to Star this Github repository

Thanks in advance @obmen86! 💯


Hotfix/sg 000 pretrained models fixes

Closing down as new release already has these problems fixed.

closed as new release fixed these issues.

Already fixed.


Translation efforts url in the readme returns a 404 page

Hello, @neicore,  thanks for checking out the project. We will be looking at how to put an appropriate link there. Also feel free to make suggestions based on what other translation projects have done.

Also @neicore, do you know of other folks in Tz who use react and are willing to help with translation to Swahili?

Hello @orama254
I have shared and will keep sharing this in my network to get people to contribute. I currently have my hands full will another project but will definitely contribute when I get a chance.

Hello @orama254 I'm Kenyan and very willing to help with the translation to Swahili. However, I'm still experiencing the same issue @neicore had with the translation efforts URL returning a 404 error. I'm open to your suggestions.

Hello @Lynnmwova,
The link should point to issue #1 . Feel free to make a pull request that fixes this and also let me know if you need any assistance with it.
cc @neicore

I will do exactly that. Thank you!

Closing this as resolved, feel free to contribute to the translation or to refer others as well to contribute.
cc @neicore @Lynnmwova


Use autoload when possible to lazily require internal dependencies

I recently came across bumbler as a tool to track down slow-requiring dependencies in a Rails app.
The default threshold is 100ms, and bumbler reports it taking:
❯ bumbler
[###########################################################################################################################################################                                                                                ]
(189/301)
189 of 301 gems required
Slow requires:
    118.90  plaid
    162.70  xero-ruby
    266.35  datadog_api_client
    434.79  package_protections

After this PR, it doesn't even show up with 10ms as the threshold 👀

I don't know much about auto-load... can you share any good links/documentation you know of that might be valuable (if you have) and what this means in the context of package protections?

I don't have any, but some that come up that talk about it the benefits

https://medium.com/rubycademy/the-autoload-method-in-ruby-11fd079562ef
https://www.rubyguides.com/2019/08/autoloading-in-ruby/

Zeitwerk is built on top of autoload, and the Railsconf keynote talks about it quite a bit: https://www.youtube.com/watch?v=DzyGdOd_6-Y

Is the idea that this should create zero behavioral change since we're just lazily loading constants once they're referenced?

Correct, there is not any functional changes to how it's used. The ruby files will just not load until the constants are loaded.

is autoload a vanilla ruby API?

Yes.

Is the additional inline module (i.e. opening up Rubocop::Cop) required, or can we autoload Rubocop::Cop::packageProtections (i.e. a namespaces constant) alongside the other autoload?

Yes, it's required. It's autoload is method being called on the current object/module and registers constants to load. It needs to be inside the module it's autoloading for.

If we are running code when something is required that we depend on (which we probably shouldn't be doing), this would change that behavior right? e.g. we do this in gusto packwerk today, but I don't think we do that in package protections!

That's right, it's best to avoid calling a method at require time that would end up loading all the constants or it removes the benefits of autoloading.
One strategy is to make a method which handles registering it, instead of calling it directly. Another is having a different require endpoint that triggers that. A good example of that it is require 'bundler/setup' which you use to pull in to require a gemfile's dependency, rather than calling Bundler.setup.

@alexevanczuk have you had a chance to consider this?


Image overlaps on company name on Multiple project create page.

is this still open?

Now its proper.


add default branch arg when building flatpak

We need to set the default branch when building the flatpak or weird stuff happens.
https://github.com/elementary/appcenter-reviews/runs/2587751116?check_suite_focus=true#step:4:127

this change is not enough, you need to run ncc build index.js -o ./dist


README: Test PR for CI

This is a test PR to see how
CI on Power8 machine is running.

/test

/test

All tests passed but CI failed due to leaky pods issue. The journalctl logs are here:https://github.com/kata-containers/runtime/pull/1267#issuecomment-468313710

/retest

For the record:

this still needed?  If not, can you please close @nitkon?


Upgraded welcome message

Added some flavor to the message, open for changes.

Codecov Report

Merging #134 into master will increase coverage by 0.05%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #134      +/-   ##
==========================================
+ Coverage    90.9%   90.96%   +0.05%
==========================================
  Files          80       81       +1
  Lines        1023     1029       +6
  Branches      162      162
==========================================
+ Hits          930      936       +6
  Misses         77       77
  Partials       16       16

Impacted Files
Coverage Δ

packages/warriorjs-cli/src/Game.js
53.2% <0%> (ø)
:arrow_up:

...ackages/warriorjs-cli/src/ui/printWelcomeHeader.js
100% <100%> (ø)

Continue to review full report at Codecov.

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

Hey, I tried to get the docs right and help a bit on the project :)
I'm actually totally new to javascript, node, npm, yarn and github and just managed to clone from github and get a synced working folder where I could launch the game.
Let me know if something is wrong.

Hey @Eveneit! Thanks a lot for this, it looks nice! After the changes introduced in #147, I started playing with your design to make it a little more subtle and I arrived to this:

If you like it, I prefer that you're the one that makes the change because the original idea was yours. Let me know!
BTW, the text version is "++~~~+$#$+~~~++" padded with "~", everything in dim gray, as in #147.

Hello @Eveneit, just checking in to see if you're still interested in working on this.

Resolved by #191.


keyboard-descriptor: add restore all function and check unexpected event

Add restoreAll for easy restore all registered key codes and events.
Throw error when input event name is unexcepted.

Checklist

[x] npm test passes
[x] tests and/or benchmarks are included
[x] documentation is changed or added

Sorry for these mistakes, descriptor's fn must return promise for now, just resolved all and LGTM :)


Dose VALI support jetson devices？

Dose VALI support jetson devices? Thanks.

Hi @zhjunqin
Latest VALI doesn't support Jetson.
There was a PR for VPF which did add Jetson support. It's not difficult from programmer's perspective.
However, HW acceleration for video on Jetson requires a special FFMpeg version from Nvidia. It's available from a package manager but I didn't do any kind of research if that's compatible with VALI licensing. So the build procedure for Jetson may become complicated. Also I don't possess a Jetson board and would have to purchase it for development and in order to deploy a GitHub self-hosted runner for CI tests.
Long story short: it looks like a good feature request for VALI v.4. Please start a feature request conversation.


Error while starting the bot

 node index.js
[Discord] › ℹ  info      Logging in...
internal/fs/utils.js:269
throw err;
^
Error: ENOENT: no such file or directory, open 'private.pem'
at Object.openSync (fs.js:462:3)
at Object.readFileSync (fs.js:364:35)
at Object.main [as run] (/home/runner/ward/server.js:56:21)
at Object. (/home/runner/ward/index.js:5:11)
at Module._compile (internal/modules/cjs/loader.js:999:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
at Module.load (internal/modules/cjs/loader.js:863:32)
at Function.Module._load (internal/modules/cjs/loader.js:708:14)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12)
at internal/main/run_main_module.js:17:47 {
errno: -2,
syscall: 'open',
code: 'ENOENT',
path: 'private.pem'
}
exit status 1
this is logged in the console

Try setting https in config.json to false

i also don't know what to put in the secret key andpublic key

i also don't know what to put in the secret key and public key

i also don't know what to put in the secret key and public key

You must use an SSL certificate.


Implement AOT

We need to implement AOT for deployments. This will improve performance.
https://angular.io/guide/aot-compiler

Complete.


Error during training

When {CUDA_VISIBLE_DEVICES='0' catalyst-rl run-trainer --config configs/config.yml} is executed during run-training.sh, catalyst.utils.tools.registry.RegistryException: No factory with name 'CoppeliaSimEnvWrapper' was registered
what is the problem and which version of catalyst should I use?
Traceback (most recent call last):
File "/workspace/mazhengyu/anaconda3/envs/catalystenv/bin/catalyst-rl", line 8, in
sys.exit(main())
File "/workspace/mazhengyu/anaconda3/envs/catalystenv/lib/python3.6/site-packages/catalyst/rl/main.py", line 44, in main
COMMANDS[args.command].main(args, uargs)
File "/workspace/mazhengyu/anaconda3/envs/catalystenv/lib/python3.6/site-packages/catalyst/rl/scripts/run_trainer.py", line 69, in main
env = ENVIRONMENTS.get_from_params(**config["environment"])
File "/workspace/mazhengyu/anaconda3/envs/catalystenv/lib/python3.6/site-packages/catalyst/utils/tools/registry.py", line 244, in get_from_params
return self.get_instance(name, meta_factory=meta_factory, **kwargs)
File "/workspace/mazhengyu/anaconda3/envs/catalystenv/lib/python3.6/site-packages/catalyst/utils/tools/registry.py", line 216, in get_instance
f = self.get(name)
File "/workspace/mazhengyu/anaconda3/envs/catalystenv/lib/python3.6/site-packages/catalyst/utils/tools/registry.py", line 187, in get
f"No factory with name '{name}' was registered"
catalyst.utils.tools.registry.RegistryException: No factory with name 'CoppeliaSimEnvWrapper' was registered

Hi，I have the same problem. Have you resolved it yet？


svg elements disappeared when wheel or drag

i import svg.panzoom.js in vue2.0 webpack 4.0 environment,but all svg elements disappeared when mousewheel or drag.

make sure to set a viewbox at start. It can be as simple as 0 0 ${width} ${height}


After new VS install, unable to clean

With a new PC, I've been reinstalling my dev environment including VS. For UB, I've been following the README as with my old PC, but I am having the same issue described in #1728 after pulling and trying to clean:
Clean started...
1>------ Clean started: Project: SolastaUnfinishedBusiness, Configuration: Debug Install Any CPU ------
1>Zipping directory "C:\dev\Solasta\SolastaUnfinishedBusiness\SolastaUnfinishedBusiness\Translations\" to "C:\dev\Solasta\SolastaUnfinishedBusiness\SolastaUnfinishedBusiness\Resources\Translations.zip".
1>C:\dev\Solasta\SolastaUnfinishedBusiness\SolastaUnfinishedBusiness\SolastaUnfinishedBusiness.csproj(28,5): error MSB4036: The "Publicise" task was not found. Check the following: 1.) The name of the task in the project file is the same as the name of the task class. 2.) The task class is "public" and implements the Microsoft.Build.Framework.ITask interface. 3.) The task is correctly declared with <UsingTask> in the project file, or in the *.tasks files located in the "C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\amd64" directory.
1>Done building project "SolastaUnfinishedBusiness.csproj" -- FAILED.
========== Clean: 0 succeeded, 1 failed, 0 skipped ==========
========== Clean started at 7:54 PM and took 00.403 seconds ==========

It's some sort of NuGet issue, I will post the solution here if it doesn't involve waiting a few days for it to automagically correct itself.

Ok fixed, I did the following steps to the default VS 2022 install:

Added NuGet Package Source: Tools > NuGet Package manager > Package Sources > New > https://api.nuget.org/v3/index.json
Installed .NET 4.7.2 SDK from https://dotnet.microsoft.com/en-us/download/dotnet-framework/thank-you/net472-developer-pack-offline-installer
Restarted, NuGet installed the missing package, clean successful, build successful

Perhaps README should be tweaked?


[unmanaged-cluster] Provide meaningful exit error codes

Feature Request
When executing the plugin from within a script it is almost impossible to know what is the reason of a failure without parsing the output file and processing it. It'll be ideal that every error would provide a different error exit code (!=0) and that these are documented so that embedding "unmanaged-cluster" functionality in other tools can be easier.

Love this!
Wondering if this needs it's own proposal or if @stmcginnis's proposal around improving error handling and visibility during bootstrapping incorporates this. But regardless, this probably warrants it's own proposal so we can have some buy in from the community on what the codes might mean.

I agree, this is good as its own proposal. And also agree, this would be a good thing.
The only thing I am not sure of is the way cobra handles error returns and if there is a way to tell it to provide different exit codes. But from what I've seen, the RunE method of implementing commands rarely gives what we want, and we should probably just use Run and print out our own errors and exit directly so we have control over this. Just some thoughts for the dev side.

Happy to write this proposal unless @jorgemoralespou is passionate about it! 👀

@jpmcb I'm never passionate about anything ;-) so you can go ahead, but TBH I think this could just be part of @stmcginnis's proposal around improving error handling and visibility during bootstrapping. I could understand not including it there if the proposal would extend to error codes across all the plugins and commands. But then, maybe this should be moved to tanzu-framework, or defined here and then discussed for incorporation in TF, as in that case, would be generic to all plugins and commands.

@jorgemoralespou can you provide a few examples of what errors you're thinking of? Doesn't need to be a proposal, but it'll help us understand what categories you've got in mind.

Sure. When creating a cluster (I'm only thinking about unmanaged now), it goes through multiple steps (prereqs check, pulling down image, parsing tkr, pulling down kapp-controller, installing package repository,...). What we would like is to have a different error code for every error in these steps so that if e.g. we get error code -13 we know it means that kapp-controller image could not be pulled, if we get -2 it means that prerequirements (docker memory) could not be satisfied,... and so on, as my understanding is that stderr and stdout provide already human readable output and this would need to be parsed to be understood by a machine scripted process.
These error code should be well documented and not change.


ping/pong

Ping/pong the currently connected websocket connection.
I found that even if all the connections are gone, a canceled get request is sent and new connection 1 is created.

https://stackoverflow.com/questions/10585355/sending-websocket-ping-pong-frame-from-browser

https://github.com/websockets/ws/blob/master/lib/sender.js#L225-L241

ping/pong frame is not available, so we use string message to handle { type: "ping" } { type: "pong" }.


Update Networking API Specs for NRP

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

@bganapa , @deathly809 , @martabarillas
I am submitting a PR to fix the api scan issues in the NRP repo.
This scan seems to be failing. public.rest-api-specs (lintdiff). Looking at the details, there doesn't seem to be any additional information. Do you know why it is failing?

@weidongxu-microsoft Could you please help why the CI gate is failing? There is not much details in the job links. Thanks!

You can click the pipeline error for detailed message.

abc831203@TSDM'[{\n "Channel": "error",\n "Text": "Syntax error: duplicate key",\n "Source": [\n {\n "document": "file:///home/vsts/work/1/s/specification/azsadmin/resource-manager/network/Microsoft.Network.Admin/preview/2015-06-15/Network.json",\n "Position": {\n "line": 124,\n "column": 12\n }\n }\n ],\n "Range": [\n {\n "document":

You can click the pipeline error for detailed message.

abc831203@TSDM'[{\n "Channel": "error",\n "Text": "Syntax error: duplicate key",\n "Source": [\n {\n "document": "file:///home/vsts/work/1/s/specification/azsadmin/resource-manager/network/Microsoft.Network.Admin/preview/2015-06-15/Network.json",\n "Position": {\n "line": 124,\n "column": 12\n }\n }\n ],\n "Range": [\n {\n "document":

@zhiyuanxia  could you pls rename the PR title to a meaningful title?

@yungezz I updated my commit and title with a more meaningful title

@weidongxu-microsoft I updated the changes so that all of the checks are now passing. Could you take a look to see if we can complete the PR?
Thank you.

Thank you Weidong.
I updated the changes so that all of the checks are now passing. Could you take a look to see if we can complete the PR?
Michael
From: Weidong Xu<EMAIL_ADDRESS>Sent: Monday, August 12, 2019 5:08 PM
To: Azure/azure-rest-api-specs<EMAIL_ADDRESS>Cc: Michael Xiao<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/azure-rest-api-specs] Users/zhixiao/nrpapispec (#6933)
You can click the pipeline error for detailed message.
abc831203@TSDM'[{\n "Channel": "error",\n "Text": "Syntax error: duplicate key",\n "Source": [\n {\n "document": "file:///home/vsts/work/1/s/specification/azsadmin/resource-manager/network/Microsoft.Network.Admin/preview/2015-06-15/Network.json",\n "Position": {\n "line": 124,\n "column": 12\n }\n }\n ],\n "Range": [\n {\n "document":
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F6933%3Femail_source%3Dnotifications%26email_token%3DAMQAEFNAXKF6C36Q6FSXMWTQEH3MLA5CNFSM4ILEDI42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4EFJFA%23issuecomment-520639636&data=02|01|zhixiao%40microsoft.com|b42d1fed9c9a4ed6853908d71f823de6|72f988bf86f141af91ab2d7cd011db47|1|0|637012516550872645&sdata=tHmv1ZD2FYFfWHOu%2FnNqHS9Ja%2B8kdONHgQeFYFlatqg%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMQAEFKVENECUGQYIVBDYGDQEH3MLANCNFSM4ILEDI4Q&data=02|01|zhixiao%40microsoft.com|b42d1fed9c9a4ed6853908d71f823de6|72f988bf86f141af91ab2d7cd011db47|1|0|637012516550882639&sdata=dmCvuw5q8kY%2Bs3i5bnb%2FqPCjSBuZn564EIuxCwLkFV4%3D&reserved=0.

@zhiyuanxia
I mainly focus on details of swagger definition and autorest extensions etc. (I am more concerned about the code generated via autorest on the API)
ARM will review the API change. That could take a couple of days.
And you seems to have a breaking change in Operation definition. (doc https://microsoft.sharepoint.com/:w:/t/azureresourcemanagerteam/EWXsAQ1yx25KkyYCeeWGUgwBSBxEdUDEbHi6FZ__U8EOQw?rtime=2ef6LVcg10g)

Thank you for taking a look at the pull request, Weidong.
I have made some additions based on your comments and removed the breaking change. Please let me know if there is anything I can do to help the ARM team with their review.
Thanks,
Michael
From: Weidong Xu<EMAIL_ADDRESS>Sent: Tuesday, August 13, 2019 6:36 PM
To: Azure/azure-rest-api-specs<EMAIL_ADDRESS>Cc: Michael Xiao<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-rest-api-specs] Update Networking API Specs for NRP (#6933)
@zhiyuanxiahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fzhiyuanxia&data=02|01|zhixiao%40microsoft.com|960aca085035443703e008d72057bf3d|72f988bf86f141af91ab2d7cd011db47|1|0|637013433547919548&sdata=4tI88MbxG8h6oF65QdSss0vNj9E%2BsDdL4fXLFuhfCxM%3D&reserved=0
I mainly focus on details of swagger definition and autorest extensions etc. (I am more concerned about the code generated via autorest on the API)
ARM will review the API change. That could take a couple of days.
And you seems to have a breaking change in Operation definition. (doc https://microsoft.sharepoint.com/:w:/t/azureresourcemanagerteam/EWXsAQ1yx25KkyYCeeWGUgwBSBxEdUDEbHi6FZ__U8EOQw?rtime=2ef6LVcg10ghttps://nam06.safelinks.protection.outlook.com/ap/w-59584e83/?url=https%3A%2F%2Fmicrosoft.sharepoint.com%2F%3Aw%3A%2Ft%2Fazureresourcemanagerteam%2FEWXsAQ1yx25KkyYCeeWGUgwBSBxEdUDEbHi6FZ__U8EOQw%3Frtime%3D2ef6LVcg10g&data=02|01|zhixiao%40microsoft.com|960aca085035443703e008d72057bf3d|72f988bf86f141af91ab2d7cd011db47|1|0|637013433547919548&sdata=eFiHPbZyvVIQ%2FBm1%2FrYwHt401FD4EEpSZlCS5zKiywM%3D&reserved=0)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F6933%3Femail_source%3Dnotifications%26email_token%3DAMQAEFNCUZS4FRW5HL3FVADQENOPTA5CNFSM4ILEDI42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4HOEBI%23issuecomment-521069061&data=02|01|zhixiao%40microsoft.com|960aca085035443703e008d72057bf3d|72f988bf86f141af91ab2d7cd011db47|1|0|637013433547929504&sdata=ie2rwaOTaRc7GZnQ30eVWB6I52i71bmcb646v1fmDus%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMQAEFJWQJF442G5GQ5JLATQENOPTANCNFSM4ILEDI4Q&data=02|01|zhixiao%40microsoft.com|960aca085035443703e008d72057bf3d|72f988bf86f141af91ab2d7cd011db47|1|0|637013433547929504&sdata=Z1mq1kKinslcTa8OJsIg1e3ADJ8hioRslCUrJ%2FMvVUI%3D&reserved=0.

@zhiyuanxia please address my comments.

Raised to<EMAIL_ADDRESS>still waiting for feedback.

Hi Marta,
Can you clarify which comments you are referring to? I might be missing something but I don’t see any comments on the PR. Do you mean the codeflow comments? I have addressed them in a later iteration. I will add my new changes to the codeflow review.
Thanks,
Michael
From: martab-msft<EMAIL_ADDRESS>Sent: Friday, August 16, 2019 9:39 AM
To: Azure/azure-rest-api-specs<EMAIL_ADDRESS>Cc: Michael Xiao<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-rest-api-specs] Update Networking API Specs for NRP (#6933)
@zhiyuanxiahttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fzhiyuanxia&data=02|01|zhixiao%40microsoft.com|146ba1da25ee43b22dff08d7226840b7|72f988bf86f141af91ab2d7cd011db47|1|0|637015703465114491&sdata=5e7IzqXALDuI7js%2F5gCW9ybVZ7MJN5vxiHhdp%2FSji5c%3D&reserved=0 please address my comments.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F6933%3Femail_source%3Dnotifications%26email_token%3DAMQAEFL2BZBUD7LXTE2XK7LQE3J2TA5CNFSM4ILEDI42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4PDHMQ%23issuecomment-522073010&data=02|01|zhixiao%40microsoft.com|146ba1da25ee43b22dff08d7226840b7|72f988bf86f141af91ab2d7cd011db47|1|0|637015703465114491&sdata=PpIBK15a%2Fmm3tHMDeDQ2EWE8sYAYDDkJqTTriUMzWSc%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMQAEFLP4NFZGDOEXW4VWJ3QE3J2TANCNFSM4ILEDI4Q&data=02|01|zhixiao%40microsoft.com|146ba1da25ee43b22dff08d7226840b7|72f988bf86f141af91ab2d7cd011db47|1|0|637015703465124486&sdata=cCfxJFUkmrudZMtGdu37saWN2oB%2FM8bUBmNkLkNBFtI%3D&reserved=0.

@zhiyuanxia

Hi Marta, Can you clarify which comments you are referring to? I might be missing something but I don’t see any comments on the PR. Do you mean the codeflow comments? I have addressed them in a later iteration. I will add my new changes to the codeflow review. Thanks, Michael

She probably forgot to submit her review.  You can't see review comments unless the review has been submitted :(


add me on discord

Add me on discord Exclusive Leaks#0001 I have a job offer for you

Add me on discord Exclusive Leaks#0001 I have a job offer for you

Invalid username/tag

How can I contact you?

add me on discord: dvnt#9260

How can I contact you?

@anyidwg @VulcanBirdy
add me on discord: dvnt#9260

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.


API configuration

Hi there
Thanks so much for building and maintaining this plugin, I just got my flush mount intercom yesterday and I'm super excited! I've managed to get the DoorBird to show up in HomeKit but each accessory (lock, light, motion) is displaying "No Response" and my guess is I didn't configure the API notifications properly.
How do I configure the API within the DoorBird app? Within Administration >>> HTTP Calls, what is the actual URL that I put there?
I've currently just put the following in my DoorBird app (note I'm running Homebridge on my Mac):

http://<doorbird_IP>/bha-api/notification.cgi?http-user=<admin_username>&http-password=<admin_password>&event=doorbell&subscribe=1&url=http://<mac_IP_address>:5005>/doorbell.html

Terminal log shows the following error when running Homebridge:

[2018-6-2 21:46:09] DoorBird is running on port 49618.
DoorBird is listening on port 5005
Snapshot -i http://<DoorBird_IP>:80/bha-api/image.cgi?http-user=<admin_username>&http-password=<admin_password> -t 1 -s 480x270 -f image2 -
events.js:183
throw er; // Unhandled 'error' event
^
Error: spawn ffmpeg ENOENT
at _errnoException (util.js:1022:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:190:19)
at onErrorNT (internal/child_process.js:372:16)
at _combinedTickCallback (internal/process/next_tick.js:138:11)
at process._tickCallback (internal/process/next_tick.js:180:9)

Is ffmpeg installed?
https://github.com/KhaosT/homebridge-camera-ffmpeg/wiki/Raspberry-PI

Yep, I installed v4.0.
I tried removing the DoorBird plugin and just running both DoorBird configs in the tested configurations link below but both gave the same error. Seems like something's wrong with the ffmpeg component instead. Any clues?
https://github.com/KhaosT/homebridge-camera-ffmpeg/wiki/Tested-Configuration

Is ffmpeg in $PATH, seems like it cannot be started. Can you start in the shell?

Yep, you are correct. I've now installed ffmpeg properly and the video stream is working - progress! Thank for your help.
The light switch and lock is working but pressing the doorbell button doesn't trigger a notification, nor is the motion sensor triggering HomeKit notification. Do you see anything wrong with what I've put as the HTTP call?  Should the Homebridge IP address at the end of the string be my Mac's IP address?

For future reference and for other newbies like myself, you have to install ffmpeg 'properly.' It's not as simple as downloading ffmpeg and double clicking the file and running it (this is what I did initially...). I installed it through terminal via home-brew which you can easily google how to. Once this was installed, the Homebridge plugin works without errors!

Your DoorBird needs to have registered the endpoint for your homebridge - motion and doorbell separately.
The doorbird api will call them on the event..
Sent from my iPhone

On 3 Jun 2018, at 07:35, BillS11<EMAIL_ADDRESS>wrote:
Yep, you are correct. I've now installed ffmpeg properly and the video stream is working - progress! Thank for your help.
The light switch and lock is working but pressing the doorbell button doesn't trigger a notification, nor is the motion sensor triggering HomeKit notification. Do you see anything wrong with what I've put as the HTTP call? Should the Homebridge IP address at the end of the string be my Mac's IP address?
For future reference and for other newbies like myself, you have to install ffmpeg 'properly.' It's not as simple as downloading ffmpeg and double clicking the file and running it (this is what I did initially...). I installed it through terminal via home-brew which you can easily google how to. Once this was installed, the Homebridge plugin works without errors!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I've added two HTTP Calls in the iOS DoorBird app. Is this what you mean by registered endpoint?
HTTP call for Doorbell is:
http://<doorbird_IP>/bha-api/notification.cgi?http-user=&http-password=&event=doorbell&subscribe=1&url=http://<mac_IP>:5005/doorbell.html

Hmm you can test them directly by calling the url in a browser it will fire the event in homekit. Then you’ll know if it’s on the plugin setup side or the doorbird side, if it works then plugin is correct and what you did on doorbird is wrong. I found using wget in a terminal was better than the doorbird app if you wanted to ensure it’s all good. Do check the app afterwards as there are time slots against the api which you need to make all blue in the app - make sense when you see it.
Sent from my iPhone

On 3 Jun 2018, at 08:12, BillS11<EMAIL_ADDRESS>wrote:
I've added two HTTP Calls in the iOS DoorBird app. Is this what you mean by registered endpoint?
HTTP call for Doorbell is:
http://<doorbird_IP>/bha-api/notification.cgi?http-user=&http-password=&event=doorbell&subscribe=1&url=http://<mac_IP>:5005/doorbell.html
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks brownad, didn't know about the schedule step where you have to make it all blue. I've since done this and it still doesn't work. What's interesting is I tested the line in my web browser as you suggested and got some interesting results. I feel I'm very close so hoping you can help me get across the line here.
The entire line I'm using is:

http://<DoorBird_IP>/bha-api/notification.cgi?http-user=<Login>&http-password=<password>&event=doorbell&subscribe=1&url=http://<IP_ADDRESS>:5005/doorbell.html

When I test this entire line in my browser, I receive the following output but no HomeKit notification on my iPhone.

{"BHA": { "RETURNCODE": "1", "NOTIFICATIONS": [{"event": "doorbell1","subscribe": "1","url": "http://<IP_ADDRESS>:5005/doorbell.html","user": "","password": "","relaxation": "10"}]}}

However, when I tested the back end of the line (as below) in my browser, I receive a HomeKit notification on my iPhone.

http://<IP_ADDRESS>:5005/doorbell.html

Happy days, enjoy 😎
It’s a bit of a pain to setup but worth it.  I hope one day it’s all native... promises promises
Sent from my iPhone

On 3 Jun 2018, at 15:25, BillS11<EMAIL_ADDRESS>wrote:
Closed #21.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Downloading from mail.ru seems broken

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.03.24
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

youtube-dl --proxy socks5://<IP_ADDRESS>:10808 --no-cache-dir -f "1080p"--merge-output-format mp4 "https://my.mail.ru/community/gotivim.mm/video/_groupvideo/288.html" -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--proxy', 'socks5://<IP_ADDRESS>:10808', '--no-cache-dir', '-f', '1080p', '--merge-output-format', 'mp4', 'https://my.mail.ru/community/gotivim.mm/video/_groupvideo/288.html', '-v']
[debug] Encodings: locale cp936, fs utf-8, out utf-8, pref cp936
[debug] youtube-dl version 2020.03.24
[debug] Python version 3.7.4 (CPython) - Windows-10-10.0.18362-SP0
[debug] exe versions: ffmpeg 4.0.2, ffprobe 4.0.2
[debug] Proxy map: {'http': 'socks5://<IP_ADDRESS>:10808', 'https': 'socks5://<IP_ADDRESS>:10808'}
[mailru] community/gotivim.mm/_groupvideo/288: Downloading webpage
[mailru] community/gotivim.mm/_groupvideo/288: Downloading video meta JSON
[debug] Invoking downloader on 'http://cdn7.my.mail.ru/hv/62465402.mp4?slave[]=s%3Ahttp%3A%2F%2Fvideo-cephgw1.i%3A8003%2Frados3%2F62465402-hv&p=f&expire_at=1585504800&touch=1499087450&reg=188&sign=db04c5f100a3c50f30e86d49a6ab5caa1b32f314'
ERROR: unable to download video data: HTTP Error 403: Forbidden
Traceback (most recent call last):
  File "c:\python37\lib\site-packages\youtube_dl\YoutubeDL.py", line 1926, in process_info
    success = dl(filename, info_dict)
  File "c:\python37\lib\site-packages\youtube_dl\YoutubeDL.py", line 1865, in dl
    return fd.download(name, info)
  File "c:\python37\lib\site-packages\youtube_dl\downloader\common.py", line 366, in download
    return self.real_download(filename, info_dict)
  File "c:\python37\lib\site-packages\youtube_dl\downloader\http.py", line 341, in real_download
    establish_connection()
  File "c:\python37\lib\site-packages\youtube_dl\downloader\http.py", line 109, in establish_connection
    ctx.data = self.ydl.urlopen(request)
  File "c:\python37\lib\site-packages\youtube_dl\YoutubeDL.py", line 2238, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "c:\python37\lib\urllib\request.py", line 531, in open
    response = meth(req, response)
  File "c:\python37\lib\urllib\request.py", line 641, in http_response
    'http', request, response, code, msg, hdrs)
  File "c:\python37\lib\urllib\request.py", line 569, in error
    return self._call_chain(*args)
  File "c:\python37\lib\urllib\request.py", line 503, in _call_chain
    result = func(*args)
  File "c:\python37\lib\urllib\request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 403: Forbidden

Description

It seems like mail.ru has updated their online player recently and you can't access their cdn directly to download video. Accessing CDN through browser directly returns permission denied errors as well. I've tried adding user agent and cookies but doesn't help. Is there any way to bypass this restriction? Thanks in advance.

You should see the #23458:
Let"s try:
youtube-dl --add-header 'Referer: https://my.mail.ru/' [DASH_URL]

Yes, with --add-header "Cookie:video_key=da498ac65a8271fe4e5b489afff638e.........." and even without any referers download works fine.
Cookie I've got from Google Chrome for site my.mail.ru

Yes, with --add-header "Cookie:video_key=da498ac65a8271fe4e5b489afff638e.........." and even without any referers download works fine.
Cookie I've got from Google Chrome for site my.mail.ru
This way works fine.


Tracy: Add version 0.9.1

Tracy is a profiler that I personally use for analyzing video game performance. The client library can already be found on vcpkg so making the graphical tool available on scoop is a big help.

[✔ ] I have read the Contributing Guide.

/verify

/verify


Support for Ruby 2.4.1 Absent Operator

See Absent Operator.
Basically it matches anything not within the parens:
>> "John Doe" =~ /\A(?~John) Doe\z/
=> nil
>> "Jane Doe" =~ /\A(?~John) Doe\z/
=> 0

Kind of weird they released this in a patch version.

Excellent. Thanks.
Also need to update Unicode properties to Unicode 8.0, which I believe was added to ruby in 2.4.0. It looks like it only adds new script names though.

Addressed in release 0.4.4


Implementing find_workspace_root

In #706 I've used a workaround, but it assumes projects won't be structured like monorepos where the root pyproject.toml needs to be searched for.
I want to look at how other projects do it first. I'd think it's easier to just implement a workspace marker in the pyproject.toml
[workspace]

This was resolved with find_root and find_first from huak-workspace.


Unable to detech hash salt and rounds in minimal docker install

Hi,
I tried to set up a test environment with Authelia, to try it out before committing it to my homelab. I have set it up and can correctly navigate to the webpage, and get the login page, on http://home-ip:8080.
However, when I try to login, using a different hashed password than "mypassword", I get an authentication failure and the following error in my logs:
error: date='Sun Dec 16 2018 20:47:29 GMT+0000 (UTC)' method='POST', path='/api/firstfactor' requestId='xxx' sessionId='xxx' ip='desktop-ip' message='Reply with error 200: Unable to detect the hash salt and rounds. Make sure the password is hashed with SSHA512.'
So it can find the file, but somehow not detect if the hash is correct. I have the following in my users_database.yml:
###############################################################
#                         Users Database                      #
###############################################################

# This file can be used if you do not have an LDAP set up.

# List of users
users:
  test_user:
    password: "{CRYPT}$6$rounds=500000$W/8aYO.7Nb2SqIgd$TOXDeaMicSWZ.9BU/VafTxjVICwFl77zvR2NT0AV/tJ7pscfEgr7omY/CRN.Gdqbr7XKb6DsKnHbWNrRSFgtm0"
    email<EMAIL_ADDRESS>    groups:
      - admins
      - dev

Hope you can help me?
TLDR: Error is thrown when using a different password to the default mypassword.

Hello @Svarto , what method to you use to generate the hash? Do you follow the doc?

Yes, I literally copy and paste the code from the docs using python3 etc. I use python version 3.7. The weird thing is that when I leave it as "mypassword" it works fine, but when I change that to "testtest" I get the error that the hash can't be read.
Am I assuming correctly creating the hash for another password I just change the "mypassword" to my own password string within the " " and change nothing else in the command string?
-------- Original Message --------
On 21 Dec 2018, 10:43, Clément Michaud wrote:

Hello @Svarto , what method to you use to generate the hash? Do you follow the doc?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@Svarto , you're doing the right thing. I will test on my side to see if I reproduce. I'll let you know.
In the meantime what you can do to generate a valid hash is to use the reset password feature. It will update the conf with the hash of the new password and this one should work.

it worked with the password reset, I will close this now. Thanks a lot for the help!

I have the very same problem.  I also cannot even use the password reset as it somehow returns {"error":"An error occurred during password reset. Your password has not been changed."}

Hello @Roemer, do you have more info from the logs?
Anyway, I will release a new major in a few days, I will include a tool to generate a hash given a password. Stay tuned.