Implement CoroutineScope instead of declaring it as a property

I think it's better to make MainViewModel to implement CoroutineScope insead of declaring it as a property, so every uiScope.launch { .. } can be replaced by launch { ... }.

I think the property is better because what if I'd like to use another dispatcher other than the Main? withContext might help but we'd like to get a reference to the Job to maybe cancel it later on.

@CodeZilla7 In this case, I prefer to use async and keep the reference of Deferred<T>. It's a non-blocking cancelable future.

Do you guys get an IDE complaint when using uiScope.launch? In my case case, AS tells me "Unsupported [cannot use release coroutines with api version less than 1.3]". Gradle and AS seem to be configured fine and using kotlin version 1.3.

Do you guys get an IDE complaint when using uiScope.launch? In my case case, AS tells me "Unsupported [cannot use release coroutines with api version less than 1.3]". Gradle and AS seem to be configured fine and using kotlin version 1.3. The app still runs though.

I don't have any error in my end. Try Invalidate Cache and Restart. But first double check if you're using the stable versions.

@CodeZilla7 invalidating got rid of the issue. Cheers! :)

That has the side effect of exposing launch as part of the public API of the ViewModel, hence why it prefers to use composition in this context.


Refactored merge sort to use array pointers instead of .shift()

The previous implementation of merge sort used the array.shift() method to merge elements from leftArray and rightArray into the sorted array. This is a bad practice, because JavaScript's implementation of the Array.prototype.shift() method is, at worst, O(n) runtime complexity, where n is the length of the array being sorted (in order to move n elements of an array over by 1, all n elements of the array must typically be touched).
To work around this, I refactored merge sort to use array pointers instead. Now, after an element has been added to the sorted array, rather than delete it from the old array and shift the entire old array over, the array index pointer is just incremented to the right by one to avoid the old element. This will always require O(1) runtime complexity, which is a significant performance boost.
I also added two simple concat() statements to the returned sortedArray value, because the length of the left and right arrays can no longer be used to determine if an element is still "dangling" (needing to be added to the sorted array).
All tests passed after my changes.
Quick Stack Overflow info on JavaScript array method runtime complexities

@austintheriot , thanks for the contribution!


Trying to run Conversion to Swift 4.2

I get this error when trying to run the Conversion to Swift 4.2 and I am using pod install to install.
:0: error: module map file 'build/Release-iphoneos/FeedKit/FeedKit.modulemap' not found
:0: error: module map file 'build/Release-iphoneos/FeedKit/FeedKit.modulemap' not found
:0: error: missing required module 'SwiftShims'

Hi, @dbadmin.
Project as been converted to Swift 4.2 in commit.
Thanks

I ran the pod update but still can not convert my project to Swift 4.2 because of the error I get.

@dbadmin have you tried deleting your derived data folder? 🤔

I removed everything in the derived data folder.  I have removed all pods and all files associated then reinstalled all of them.

@dbadmin how about this way, remove FeedKit completely, migrate to 4.2 and then add FeedKit again.
That will either work, or at least give a glimpse of what’s the issue.

I was able to run the convert after commenting out FeedKit and SwiftSoup pods. Then I ran pod update to remove them and then commented out all code that used the pods and was able to run the update.
My Pod file.
platform :ios, '11.2'
target "Test App" do
pod 'Firebase/Core'
pod 'Fabric'
pod 'Crashlytics'
#pod 'FeedKit'
#pod 'SwiftSoup'
end

Good to know!
Cheers


Release/v1.4.4 back into master.

Just merging release back into master

nvm, we are not even using this view anymore.


How to run finetuning on multiple GPUs?

Currently, the code does not support multiple GPU training. However, it is possible to modify the code using DataParallel to achieve it. You can refer to this tutorial on PyTorch's official website for guidance: https://pytorch.org/tutorials/beginner/blitz/data_parallel_tutorial.html.
I plan to create a pull request myself in the future to support multi-GPU training. Thank you for the suggestion!

@jumon Any update on this ?


HDFS-15966. Empty the statistical parameters when emptying the redundant queue

2894

 Quality gate: Passed
 Everything looks good
Details

 Quality gate: Passed
 Everything looks good
Details


Rename field so that GCP stackdriver logs are captured at the right level

Description
This PR will have the json log formatter in diagd.py have its levelname field renamed to severity so that GCP stackdriver logs are properly annotated with the right severity.
Related Issues
List related issues.
Testing
A few sentences describing what testing you've done, e.g., manual tests, automated tests, deployed in production, etc.
Checklist

[ ] Does my change need to be backported to a previous release?

What backport versions were discussed with the Maintainers in the Issue?

[ ] I made sure to update CHANGELOG.md.
Remember, the CHANGELOG needs to mention:

Any new features
Any changes to our included version of Envoy
Any non-backward-compatible changes
Any deprecations

[ ] This is unlikely to impact how Ambassador performs at scale.
Remember, things that might have an impact at scale include:

Any significant changes in memory use that might require adjusting the memory limits
Any significant changes in CPU use that might require adjusting the CPU limits
Anything that might change how many replicas users should use
Changes that impact data-plane latency/scalability

[ ] My change is adequately tested.
Remember when considering testing:

Your change needs to be specifically covered by tests.

Tests need to cover all the states where your change is relevant: for example, if you add a behavior that can be enabled or disabled, you'll need tests that cover the enabled case and tests that cover the disabled case. It's not sufficient just to test with the behavior enabled.

You also need to make sure that the entire area being changed has adequate test coverage.

If existing tests don't actually cover the entire area being changed, add tests.
This applies even for aspects of the area that you're not changing – check the test coverage, and improve it if needed!

We should lean on the bulk of code being covered by unit tests, but...
... an end-to-end test should cover the integration points

[ ] I updated DEVELOPING.md with any any special dev tricks I had to use to work on this code efficiently.

[ ] The changes in this PR have been reviewed for security concerns and adherence to security best practices.

@alexionescu - Thanks for this PR, would you mind writing up an issue first and/or finding an existing issue so that we can capture discussion and reasoning for a fix.
The key thing in this one is to think about whether it will have breaking changes on other users (maybe not but something to keep in mind).

@LanceEa Done, thank you for your feedback!


Add fluid typography, clean up existing type styles

Right now we sort of fudged things just to get something live! It'd be nice to use postcss or manually write some CSS clamps to build out a typography system.

Did some touch ups, this will be an ongoing process!


ARROW-15398: [Integration-tests] Made integration tests be built in parallel and increase cache hit

This is a draft PR to gauge interest in splitting the build of integration tests into separate stages, so that a change in a language does not trigger a rebuild in all of them.
The idea is to use docker multi-stage builds to build the integration test binaries for each language, which are then packed in the same docker image prior to running all the tests. This has 2 main consequences:

the build of each of the languages is done in parallel, since docker parallelizes independent stages
the build of a language is only done when something changed on it

Overall, the hope is to significantly reduce the time it takes to build and run the tests, as we only need to build the language that changed in a given PR, as opposed to all of them. It also makes it easier to understand what dependencies each language depends on.
There are some rough edges in the crossbow, but before handling them, I would like to gauge whether this is something that others would be interested in.

@raulcd @assignUser Does one of you want to take a look at this?

I would like to gauge whether this is something that others would be interested in.

💯 Improving the (docker & other) caching for our builds is something I am currently looking at so this comes just at the right time :D

I would like to gauge whether this is something that others would be interested in.

💯 Improving the (docker & other) caching for our builds is something I am currently looking at so this comes just at the right time :D


Module 'fs' has no default export

I'm receiving Module '"fs"' has no default export. error when trying to compile and deploy script containing twitter-api-v2 package, is there any specific requirement so this error won't appear?

Adding --allowSyntheticDefaultImports to the command solved it for me

Hello,
allowSyntheticDefaultImports is supposed to be true by default, maybe it's manually set to false in you tsconfig.json file?
Indeed we could set it to false ourselves to maximize compatibility :)

Now merged and available in 1.4.0 :)


Grpc healthcheck not working properly

Description:
I have been testing the grpc health check with a simple health check example here: https://github.com/grpc/grpc-go/tree/master/examples/features/health. But the health check configured through envoy is not working properly. When the service status is NO_SERVING, envoy still sends requests to the server.
Below are the only changes I made to the grpc-go example:
in service/main.go, update: system = "Echo"
in client/main.go, delete grpc.WithDefaultServiceConfig(serviceConfig), update Addr: "localhost:9911"
here is my envoy config:
static_resources:
  listeners:
  - address:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 9911
    filter_chains:
    - filters:
      - name: envoy.http_connection_manager
        typed_config:
          "@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
          codec_type: auto
          stat_prefix: ingress_http
          route_config:
            name: greeter_route # configure the localclus routes
            virtual_hosts:
            - name: servicegreeter
              domains:
              - "*"
              routes:
              - match:
                  prefix: "/"
                  grpc: {}
                route:
                  cluster: grpc1
          http_filters:
          - name: envoy.filters.http.router
            typed_config:
              "@type": type.googleapis.com/envoy.extensions.filters.http.router.v3.Router

  clusters:
    - name: grpc1
      connect_timeout: 1.25s
      type: STATIC
      lb_policy: ROUND_ROBIN
      dns_lookup_family: V4_ONLY
      typed_extension_protocol_options:
        envoy.extensions.upstreams.http.v3.HttpProtocolOptions:
          "@type": type.googleapis.com/envoy.extensions.upstreams.http.v3.HttpProtocolOptions
          explicit_http_config:
            http2_protocol_options: {}
      load_assignment:
        cluster_name: grpc1
        endpoints:
        - lb_endpoints:
          - endpoint:
              address:
                socket_address:
                  address: <IP_ADDRESS>
                  port_value: 50052
        - lb_endpoints:
          - endpoint:
              address:
                socket_address:
                  address: <IP_ADDRESS>
                  port_value: 50051
      health_checks:
        - timeout: 1s
          interval: 1s
          no_traffic_interval: 0.001s
          unhealthy_threshold: 1
          healthy_threshold: 1
          grpc_health_check:
            service_name: "Echo"
          event_log_path: ./health_check.log
          always_log_health_check_failures: true
          reuse_connection: false

log:
server 50051:
8350 ± : go run server/main.go -port=50051 -sleep=10s                                         ⏎ [154d9h10m] ✹ ✭
Update server to SERVING
Update server to NOT_SERVING
Update server to SERVING
Update server to NOT_SERVING

server 50052:
8349 ± : go run server/main.go -port=50052 -sleep=20s                                         ⏎ [154d9h10m] ✹ ✭
Update server to SERVING
Update server to NOT_SERVING

client: (here both 50051 and 50052 keep responding to client even though they are NO_SERVING)
UnaryEcho:  hello from localhost:50052
UnaryEcho:  hello from localhost:50051
UnaryEcho:  hello from localhost:50052
UnaryEcho:  hello from localhost:50051
UnaryEcho:  hello from localhost:50052
UnaryEcho:  hello from localhost:50051
UnaryEcho:  hello from localhost:50051
UnaryEcho:  hello from localhost:50051
UnaryEcho:  hello from localhost:50051
UnaryEcho:  hello from localhost:50052

I found out I should add panic_threshold. This is working now.
      common_lb_config:
        healthy_panic_threshold:
          value: 0


FormData

Why aren't using FormData when post?

You is trying do a upload files using AJAX?

Nope, I'm trying to do a POST with https://github.com/typicode/json-server and seems that is not working if I don't user Formdata for data

Ok was not that... the problem is that on docs you say:
"Default is "application/x-www-form-urlencoded; charset=UTF-8""
but on code I think is application/json no?

Yes I can confirm that the error was about docs, now is working
phonon.ajax({
            method: method,
            url: url,
            data: data,
            contentType: 'application/x-www-form-urlencoded; charset=UTF-8',
            success: (res) => resolve(res),
            error: (res) => reject(res)
        });

@Qathom please docs looks like incorrect because default contentType is json.

Hi @CKGrafico and @John-Henrique,
Thank you for opening this issue. It will be fixed.


Fix dot notation for if_exist

Description
Fixes #4521
This is a patch fix only for the if_exist rule and DOES NOT yet cover other rules.
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPdocs
[x] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Other than Abdul's comment this looks good!
Out of curiosity... does @internal really do anything on a final class?

Other than Abdul's comment this looks good!
Out of curiosity... does @internal really do anything on a final class?

Actually, I copied this rule from php-cs-fixer. @internal means the class should not be used in users' code. This can be applied to anything where BC promise is not guaranteed. final classes may or may not be for @internal use. final classes only limit their inheritance.


.lycheeignore can not exclude files / directories

I would like to exclude some files / directories form being scanned by lychee.
Example:
$ mkdir -pv /tmp/test/dir

$ cd /tmp/test/

$ cat > dir/README.md << EOF
https://google.com
EOF

$ cat > .lycheeignore << EOF
^./dir/
EOF

$ lychee --verbose .
✔ [200] https://google.com/

🔍 1 Total ✅ 1 OK 🚫 0 Errors

I excluded the dir directory in .lycheeignore but it was scanned by lychee :-(
Is there a way how to use .lycheeignore with files/directories ?
Thank you...

That's a common misconception that keeps popping up unfortunately:
the .lycheeignore lists URLs that should be excluded not paths.
To exclude the path you can create a .lychee.toml file with the following content:
exclude_path = ["^./dir/"]

Example config file: https://github.com/mre/endler.dev/blob/50d8d5f90dbafa445c9455e420a40f8866f3e1c7/lychee.toml#L28
Hope that helps.

Very welcome. Thanks for the pull request for adding more details about the behavior.

Closing this and moving the discussion over to your PR.

FYI: Added a section to the lychee docs as well. https://lychee.cli.rs/#/recipes/excluding-paths
Also fixed a little typo that slipped through code review. 😄
https://github.com/lycheeverse/lychee/commit/fbf0e9faeafbe3bcaa6b246249a2cc5e59596402

Thank you... and sorry for the typo...


location services can no longer be switched on

That quite likely is not a UnifiedNlp issue - but hopefully somebody here has an idea what to do...
After 2 weeks of happily using a gapps-free UnifiedNLP I suddenly can not any longer switch on location services at all. The slider just sweeps back - that's all.  I re-installed google play services - but that doesnt change anything.
What could I do? - I've got titanium backups of everything - what components should I restore?

Did you install any relevant updates or software since it worked? maybe some energy saver tool? are you using UnifiedNLP with the Xposed plugin? Additional system details would be helpful

No, nothing at all. No new software, no updates, no energy saver. The last
bit of software to be installed was Mozilla Stumbler, and location services
worked nicely after the install.
I'm using slimkat 4.4.4 (9.0) on a Huawei y300; dont have a xposed plugin
or any other xposed component installed.


not working

the commands are not working

You need to remove the channel check as by default the commands will only work in 1 specific channel


Replacing text in Pipeline Build is not working

After live testing, Ingrid is still unable to validate the key from syncfusion and Ingrid version is still the same as in development phase.

commit aaff3c1 should fix this issue


Consider renaming flatMap to chain

Personally, I prefer flatMap, but in recent discussions, TC39 seems to be preferring to use the name chain, and fantasy-land uses chain, too.

This doesn't seem documented yet. See pull/95


(SimBrief) Wrong cruising ALT and wrong speed integer on prefile (VATSIM)

Recent dev Version of phpVMS

@FatihKoz is this something with simbrief?

Speeds are always reported with trailing zeros in flight plans, this is what simbrief reports and we pass that info to VATSIM prefile form too without a change but seems like they want it in their style. Anyway we can fix it and make the VATSIM prefile fully compatible with VATSIM.
For the altitude part i need to check what simbrief returns and what we use. What is the problem or the expectation of VATSIM here ? Should it be 23000 or something else like FL230, need more info to process


Support for AdonisJS

I quickly read the "getting started" documentation and it looks like AdonisJS is not supported.  Am I correct? If yes, will it be supported in the future?
It looks to be a pretty popular MVC server written in Node and I was sad not to find support for it.
Thanks :)

cc @krzysztofstudniarek

Hey @pinkynrg
Currently we're focusing on different aspects of AdminJS. Adding AddonisJS is not on our roadmap currently.
Kind regards,
Krzysztof Studniarek


[Feature Request] Allow user logins with usernames

Describe the feature you'd like
Allow usernames to be used to login to BookStack, as well as emails.
Describe the benefits this feature would bring to BookStack users
Saves users time when logging in, typing out a full email address takes far longer than a regular username.
Additional context
N/A

Thanks for your request @Skyler-Altol.
Since this is effectively already requested in #1498 I'll be closing this off as a duplicate.


Numpydoc

resolves #381
added missing documentation to the readthedocs autogenerator
moved param specifications from init to the class definition so that it is visible in readthedocs
modifications to the documentation to make it compliant with numpydoc

Pull Request Test Coverage Report for Build 2701

8 of 8 (100.0%)  changed or added relevant lines in 5 files are covered.
11 unchanged lines in 5 files lost coverage.
Overall coverage increased (+0.01%) to 87.391%

Files with Coverage Reduction
New Missed Lines
%

flow/core/params.py
1
93.99%

flow/envs/base_env.py
1
74.71%

flow/core/util.py
2
95.08%

flow/controllers/velocity_controllers.py
2
13.39%

flow/renderer/pyglet_renderer.py
5
31.28%

Totals

Change from base Build 2699:
0.01%

Covered Lines:
7423

Relevant Lines:
8494

💛  - Coveralls


Ctrl + L(Clearscreen shortcut) doesn't works correctly.

Steps to reproduce

Open Poweshell.

Type ping <IP_ADDRESS>, This is kind of necessary to show my problem. Any command that prints >1 line of output'll do just fine.

Now, Type "some text, Doesn't matters what it is".

Expected behavior
It should clear the output printed by the ping command and move the prompt to the top of screen without deleting the text "some text, Doesn't matters what it is".  This is the behaviour shown by pretty much all shells on linux.
Actual behavior
It just prints ^L to console. Sometimes it does what I mentioned in the "Expected behaviour" section but most of the time it just prints ^L in console which is quite annoying.
I am a begninner in C# and if someone can give me some pointers on what's causing this problem I can try fixing it.
Regards
Ishan Jain
PS:
I searched Github issues, Known Issues page with terms like "Clear screen", "Ctrl+L" and "^L" but didn't found anything relevant.

What version of PowerShell are you using and what version of PSReadline?  This doesn't repro for me

I updated Windows 10 on my laptop from 1503 to 1703 and it seems to work fine now. Powershell Version is 5.1.1 and I couldn't find a way to get the version of PSReadline. I'll close this issue for now and reopen if this happens again.

This was a Windows Powershell related bug so I should have read the Readme correctly and raise this issue on UserVoice. Apologies for wasting time.


Add a shortcut for focussing the search bar

Is your feature request related to a problem? Please describe.
Adding a shortcut for focussing the search bar input would be nice

Describe the solution you'd like
On a keypress for example: '/' should focus the search bar

@akzain please follow the contributions guidelines !!
https://midouwebdev.gitbook.io/mumble-docs/contributing/contributing
You didn't add your issue to the project board !
It is necessary for contributors to see which issue is available !!
Please take a look at https://midouwebdev.gitbook.io/mumble-docs/contributing/contributing

@akzain
i added it this time but try to do it next time
Thanks !


Regenerates last prompt when script gets restarted

Every time I restart the script, it'll regenerate the last prompt which'll @mention the last person who used the bot. It's really frustrating and makes someone confused every time it happens

Docker with Debian Bullseye and Node 18
even with both images rendered and sent into the channel, the last entry in dbQueue.json is:
  "id": 1258,
  "status": "new",
  "cmd": "Jurassic World diorama sculpture by Hugh Hancock\r",
  "userid": "141959308521766912",
  "username": "Arragon",
  "discriminator": "2062",
  "timestampRequested": "2023-05-18T13:52:10.404Z",
  "channel": "1053350555541585941",
  "attachments": [],
  "seed":<PHONE_NUMBER>,
  "number": 1,
  "width": 768,
  "height": 768,
  "steps": 30,
  "prompt": "Jurassic World diorama sculpture by Hugh Hancock",
  "scale": 5.5,
  "sampler": "k_euler_a",
  "renderer": "localApi",
  "strength": 0.7,
  "threshold": 0,
  "perlin": 0,
  "gfpgan_strength": 0,
  "codeformer_strength": 0,
  "upscale_level": "",
  "upscale_strength": 0.5,
  "variation_amount": 0,
  "with_variations": [],
  "results": [],
  "model": "kotosmix",
  "invert_mask": false,
  "seamless": false,
  "hires_fix": false,
  "cost": 0
}```

Maybe it forgets to flush the file?

Nailed it, I added a dbWrite() call after it marks the job done, should resolve this :)


Make >=> right associative

Fixes #56
Do you have any objections to also reducing the other 100 level notations so they bind more strongly than =?

No, please do.


Investigate using Node.js from Docker image

To reduce the number of software needed on the users machine to get started. We could increase the success rate for newcomers this way, and just declare the Node.js version needed in the README if they want to run it locally.
If we want to use Docker images, we need to ensure that it mounts in the volume properly so changes are picked up (e.g. by docker-compose up --build).

Since it is targeting Qlik Core developers, the requirement to have Node.js and Docker on the dev machine seems pretty light. I'm not fully confident that running the example Node.js code in Docker makes things much easier for newcomers.


add support PostCSS 5.x

Hi! npm could not install this package to my project with postcss 5.0.3:
npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>

Hi, I tried to update it to the 5 version yesterday, but looks like plugins should be updated first :)
So I downgraded it :)
Anyways, all tests pass with the version 5 and if you don't mind about warnings - you can use it.

I'll link it to that Issue https://github.com/css-modules/css-modules-loader-core/issues/21

Thank you, @joeybaker
I have published it in 1.0.5

Thank you


setup core-controllers before e2e-test

Some e2e tests cannot work correctly for there's no oam-core-controller reconciling resources.
e.g. vela comp status  need to get reconciled AppConfig object.
Signed-off-by: roy wang<EMAIL_ADDRESS>

solved in #329. closed.


add .asf.yaml

Adding the .asf.yaml file allows control over a few things.

Setting the project name
Setting the project homepage
Auto-linking issues (like FREEMARKER-224 which would automatically turn into a link after this is merged)
Auto-linking pull requests into the correspinding JIRA issues
Dependabot alerts (for developers with the respective karma only)
enabling/disabling of specific merge buttons.

I am open to any changes, I would just like to add a little bit of convenience using the jira options and auto links.
We could also add labels to the github repository if wanted.

@ddekany I forgot to set issues to false. You should do that before people start opening issues here.

Fix:
https://github.com/apache/freemarker/pull/101


Sketch-n-sketch example on home page crashes in Chrome / Mac OS

When I follow the link on the elm-lang.org home page to the Sketch-n-Sketch application using Chrome on MacOS I quickly get a stack overflow error before the app even draws anything.
I reported the error: https://github.com/ravichugh/sketch-n-sketch/issues/84
Working around the error requires running Chrome with a large stack. The error is not planned to be addressed until around December.
Sketch-n-Sketch is a great application and a nice example of what can be done with Elm, but I wonder if it should be featured on the Elm front page when some (many?) users are going to get a fatal runtime error as soon as they try it.

Can you suggest some alternatives for the spot in the meantime?

On the Elm slack user noahzgordon points out that version v0.5 works fine. It works for me too. Maybe we could link to that specific version for now rather than to "latest".

Yeah, that sounds great to me, let's do that! Can you send a PR that does the change?


Coefplot: Plotting the same regression equation using different dataframes

I am trying to use  for a list of models that fit the same equation (with the same variable names) but using different data frames. For example, I would like to plot one (or two) variables for each of those models to get something like this:

However, as far as I can tell, this can only be achieved if the model specification is not exactly identical across models. If it is, <coef_plot> generates what seems like a strange repeating pattern:

Hence, it seems like  gets confused if the models in the list use the same formula.
Is it currently possible to plot the same variable from the same model specification estimated on different different frames _as though they were different models?
Thank you.

Yes - the problem arises because coefplot loops over the provided models and uses the model name. I see two options:
Option 1: You could work with the split argument:
import pyfixest as pf

df = pf.get_data()
fit = pf.feols("Y ~ X1", split = "f1", data = df[df.f1.isin([1,2])])
fit.coefplot(coord_flip=False, keep = "X1")

Alternatively, you could overwrite the _model_name attribute by hand:
fit1 = pf.feols("Y ~ X1", data = df[df.f1.isin([1])])
fit2 = pf.feols("Y ~ X1", data = df[df.f1.isin([2])])

fit1._model_name
# 'Y~X1'
fit2._model_name
# 'Y~X1'

fit1._model_name += ", sample f1 = 1"
fit2._model_name += ", sample f1 = 2"
pf.coefplot([fit1, fit2], coord_flip=False, keep = "X1")

which also produces

Maybe it would be convenient to add a model_name argument to pf.coefplot() that would allow users to pass custom model names to avoid duplicates?

Thank you for this tip! Accessing and modifying the model name did the trick!

#Hi @escherpf I would reopen this as
I'd like to implement a more user friendly solution here (that is less hacky), and if we close this, I'm afraid I might forget about this 😀

Hi @escherpf , this has now gotten a proper solution with #748 . Thanks for reporting!


Unclear reference to "Device Connection String"

At:
https://github.com/Azure-Samples/iot-hub-c-m0wifi-getstartedkit/blob/master/README.md#17-modify-the-remote-monitoring-sample
(and multiple other places)
It states:
Replace the placeholders with your WiFi name (SSID), WiFi password, and the device connection string you created at the beginning of this tutorial
However, as someone who is new to Azure, it is not clear if you mean:

Azure Solution
Device ID
Device IOT Hub Hostname
Device Key

Hi @lastcoolnameleft, we changed this document, can you check if is it clear now?


Update angular.d.ts to fix IHttpPromise.

IHttpPromise does not extend IPromise directly.  It always transforms it with the response (of type T) as the data member.  See $httpProvider.$http.sendRec.resolvePromise(response, status, headers, statusText).  This matters when exploiting the inheritance to IPromise (for example in a return).

@ngbrown thanks mate!

note
https://docs.angularjs.org/api/ng/service/$http#usage


Fix missing wasm_bindgen attribute

The generated JS code in the following code snippet has a constructor. The corresponding Rust code did not have the attribute marking the new as the constructor.

Thanks!


Error: Building package plplot:x64-windows-static failed with: BUILD_FAILED

Hi,
I'm trying to get started with plplot in conjunction with the wxwidgets library. I'd like to use static linkage. From the documentation I understand I should therefore use the following command:

PS D:\Source\VS19\vcpkg> .\vcpkg.exe install plplot[wxwidgets]:x64-windows-static --recurse

I used the --recurse flag as the plplot library was first built without any wxwidgets dependency. This went flawlessly both for dynamic and static linkage.
Now I get the following error message:

CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:136 (message):
Command failed: D:/Source/VS19/vcpkg/downloads/tools/cmake-3.14.0-windows/cmake-3.14.0-win32-x86/bin/cmake.exe --build . --config Debug --target install -- -v
Working Directory: D:/Source/VS19/vcpkg/buildtrees/plplot/x64-windows-static-dbg
See logs for more information:
D:\Source\VS19\vcpkg\buildtrees\plplot\install-x64-windows-static-dbg-out.log

Call Stack (most recent call first):
scripts/cmake/vcpkg_build_cmake.cmake:96 (vcpkg_execute_build_process)
scripts/cmake/vcpkg_install_cmake.cmake:24 (vcpkg_build_cmake)
ports/plplot/portfile.cmake:48 (vcpkg_install_cmake)
scripts/ports.cmake:94 (include)

Error: Building package plplot:x64-windows-static failed with: BUILD_FAILED

Not sure what I am doing wrong here..

@MrBeee, thanks for posting this issue, I have submitted a PR for fixing this issue.


docs: Use backticks to format file name in working-with-the-npm-registry#installing-a-package.

Closes #35772

@Piyush-r-bhaskar Thank you so much for opening a PR with this fix! ✨ We are currently in a repo freeze, but we'll get this merged as soon as the freeze ends on Jan 6th 💛
Now that you are veteran of the OS Docs repo, take a look at our help wanted section for more issues you can work on 💛


Is this mantained?

I'm wondering if this is mantained, because I'm planning on mantaining a fork of this.

If anyone is interested. I've implemented basic github checks and updated the project to modern dependencies here https://github.com/yknx4/opencov

Hi and thanks a lot for working on this!
I am not actively working on this but would be very happy to merge PR with functionalities you added!


Spark line chart appendData logs "dfsfgdgdgdf" and is not updating the chart

I took the spark1 from the spark line demo and tried to append data by adding the following:
setInterval(function() {
  spark1.appendData({
    data: [ 20 ]
  });
}, 300);

The data is not appended and 'dfsfgdgdgdf' is printed to the console every 300:th ms. Looking in the source code there is a random console log there: https://github.com/apexcharts/apexcharts.js/blob/8281a078618c8330f291305321b83fce6b74a192/src/modules/ZoomPanSelection.js#L203

In addition to that appendData() documentation is inconsistent: https://apexcharts.com/docs/methods/#appendData

Name
Type
Default
Description

newData
Array
[ ]
The data array to append the existing series datasets

While the example suggest that you send an object to appendData()

The documentation for https://apexcharts.com/docs/methods/#updateSeries is also wrong
chart.updateSeries([
  data: [32, 44, 31, 41, 22]
])

The argument in the example is an array with a named key which is invalid javascript, it is supposed to be:
chart.updateSeries([{
  data: [32, 44, 31, 41, 22]
}])

updateSeries actually works for the spark lines though.

Thanks for bringing this to attention.
The documentation was indeed wrong for the updateSeries() and appendData() method, and I updated it on the website.
Also, removed the random console.logs.
I will check the appendData() with sparklines

The appendData() method works correctly with the updated doc. It was missing the array parenthesis.
The correct format would be this as updated in the website docs recently
setInterval(function() {
      spark3.appendData([{
        data: [20]
      }]);
    }, 1000);

Here's the result

@junedchhipa Thank you for the quick response, this solves my issues! 👍


IGNITE-15382 Decouple API and implementation for Tuple and Schema

https://issues.apache.org/jira/browse/IGNITE-15382

In general it looks good to me. But is it possible to avoid using of ServiceLoader for TupleFactory and SchemaBuilder?


Send picture to Whatsapp Camera

Hi.
I want to automate the whatsapp qrcode scanner (Adding Devices).
So what I need is send to whatsapp camera the content of qrcode (which i already have).
I have both (image and string content).
Is it possible using pict2cam?
Thanks.

Hello @duduccosta!
Unfortunately not, Pict2Cam can't hook into camera inputs.
WhatsApp doesn't use the Android Intent system for taking pictures (which Pict2Cam can hook into) or video: they bundle their own camera implementation that doesn't rely on Intents, but uses the device's camera input directly without third party apps.
You would need a way to replace the device's camera input with your own images or video in order for that to work, most likely needing to root your device (so you can manipulate the I/O of the OS).
I'm sorry that I can't provide you with much more information about this, I've tried to find a way to do this from some time to no avail, so if you ever find something, please let me know :)
Regards


Protect "fixup" postprocessors from user config

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
If the user specifies postprocessor_args they will interfere with the postprocessors for automatically fixing containers and stretched image (FFmpegFixupStretchedPP, FFmpegFixupM4aPP, FFmpegFixupM3u8PP). This pull request corrects this.
Before:
PS C:\Users\rasmu\Desktop\youtube test\ffmpeg test> youtube-dl --format 'bestaudio' --postprocessor-args "-acodec libopus -b:a 32K" -v _yDZY5_u8FQ
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--format', 'bestaudio', '--postprocessor-args', '-acodec libopus -b:a 32K', '-v', '_yDZY5_u8FQ']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dl version 2020.01.24
[debug] Python version 3.8.1 (CPython) - Windows-10-10.0.18362-SP0
[debug] exe versions: ffmpeg 4.2.1, ffprobe 4.2.1
[debug] Proxy map: {}
[youtube] _yDZY5_u8FQ: Downloading webpage
[youtube] _yDZY5_u8FQ: Downloading video info webpage
[debug] Invoking downloader on 'https://r4---sn-uqj-j2iz.googlevideo.com/videoplayback?expire=1580179293&ei=_EovXrG5O9Hn7QSb35DADg&ip=<IP_ADDRESS>&id=o-AGl-QaqCwM23zMS0NKBGIs7O8zB-MmAcR6uTNL-2r1Ja&itag=258&source=youtube&requiressl=yes&mm=31%2C29&mn=sn-uqj-j2iz%2Csn-5goeen7y&ms=au%2Crdu&mv=m&mvi=3&pl=24&initcwndbps=1635000&vprv=1&mime=audio%2Fmp4&gir=yes&clen=3266848&dur=67.370&lmt=1579640280116448&mt=1580157610&fvip=4&keepalive=yes&fexp=23842630&c=WEB&txp=1311222&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=ALgxI2wwRAIgCYXccnAAqM5H-Nuyp9Mocbc9wYeVG_VxAuv--vNa-o0CIBbLhvFyrqVZpaerSAqSTIIRU6wK1JHnJ65rMMAZBqU-&lsparams=mm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AHylml4wRAIgFSmlCUrtYya1rYxt95jN2EtNgFm1QcovQxmaSdzvxPcCIFp0RS-QjqkCl11uf80kIEPEM7pds9TmS0VhH1tDswqE&ratebypass=yes'
[download] Why Does Space Inspire Us Elon Musk-_yDZY5_u8FQ.m4a has already been downloaded
[download] 100% of 3.11MiB
[ffmpeg] Correcting container in "Why Does Space Inspire Us Elon Musk-_yDZY5_u8FQ.m4a"
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:Why Does Space Inspire Us Elon Musk-_yDZY5_u8FQ.m4a" -c copy -f mp4 -acodec libopus "-b:a" 32K "file:Why Does Space Inspire Us Elon Musk-_yDZY5_u8FQ.temp.m4a"
ERROR: Conversion failed!
Traceback (most recent call last):
  File "c:\users\rasmu\appdata\local\programs\python\python38\lib\site-packages\youtube_dl\YoutubeDL.py", line 2064, in post_process
    files_to_delete, info = pp.run(info)
  File "c:\users\rasmu\appdata\local\programs\python\python38\lib\site-packages\youtube_dl\postprocessor\ffmpeg.py", line 561, in run
    self.run_ffmpeg(filename, temp_filename, options)
  File "c:\users\rasmu\appdata\local\programs\python\python38\lib\site-packages\youtube_dl\postprocessor\ffmpeg.py", line 239, in run_ffmpeg
    self.run_ffmpeg_multiple_files([path], out_path, opts)
  File "c:\users\rasmu\appdata\local\programs\python\python38\lib\site-packages\youtube_dl\postprocessor\ffmpeg.py", line 235, in run_ffmpeg_multiple_files
    raise FFmpegPostProcessorError(msg)
youtube_dl.postprocessor.ffmpeg.FFmpegPostProcessorError: Conversion failed!

After:
PS C:\Users\rasmu\Desktop\youtube test\ffmpeg test> youtube-dl --format 'bestaudio' --postprocessor-args "-acodec libopus -b:a 32K" -v _yDZY5_u8FQ
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--format', 'bestaudio', '--postprocessor-args', '-acodec libopus -b:a 32K', '-v', '_yDZY5_u8FQ']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dl version 2020.01.24
[debug] Python version 3.8.1 (CPython) - Windows-10-10.0.18362-SP0
[debug] exe versions: ffmpeg 4.2.1, ffprobe 4.2.1
[debug] Proxy map: {}
[youtube] _yDZY5_u8FQ: Downloading webpage
[youtube] _yDZY5_u8FQ: Downloading video info webpage
[debug] Invoking downloader on 'https://r4---sn-uqj-j2iz.googlevideo.com/videoplayback?expire=1580179322&ei=GksvXriNE4mI7AS_4LegAg&ip=<IP_ADDRESS>&id=o-AGIFJj5fP5ZWL3s9sLF1x9ERbLGlWQS5FDB8Qbh-TxqS&itag=258&source=youtube&requiressl=yes&mm=31%2C29&mn=sn-uqj-j2iz%2Csn-5go7yne6&ms=au%2Crdu&mv=m&mvi=3&pl=24&initcwndbps=1635000&vprv=1&mime=audio%2Fmp4&gir=yes&clen=3266848&dur=67.370&lmt=1579640280116448&mt=1580157610&fvip=4&keepalive=yes&fexp=23842630&c=WEB&txp=1311222&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=ALgxI2wwRgIhAMTlCOhhdF7W9WDJzgOIrn6QbSYIC38NhnK-b928VSbMAiEAn5pzWfcMwtN0wCKu244e9aFoFZ1AvMod_nNyzDAkBmg%3D&lsparams=mm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AHylml4wRAIgAbOcrNz5CxLxiakDR5hKO_RmVkNVnelpxo6F4R-G1UECIDbRxMJjuXD1QfYUaaKk87HBVqK-vPwMl1voL1XN7Jwt&ratebypass=yes'
[download] Why Does Space Inspire Us Elon Musk-_yDZY5_u8FQ.m4a has already been downloaded
[download] 100% of 3.11MiB
[ffmpeg] Correcting container in "Why Does Space Inspire Us Elon Musk-_yDZY5_u8FQ.m4a"
[debug] ffmpeg command line: ffmpeg -y -loglevel "repeat+info" -i "file:Why Does Space Inspire Us Elon Musk-_yDZY5_u8FQ.m4a" -c copy -f mp4 "file:Why Does Space Inspire Us Elon Musk-_yDZY5_u8FQ.temp.m4a"

It seems I am running into this exact issue with YouTube-dl. Has this fix been released in the current version? Thank you.

Not as far as I know. I closed this pull request because I, being new at git, merged the patch to my own master branch which prevented me from making seperate pull requests to this project. Never got around to making a new one, and I'm not entirely sure this is the best solution to the problem either.


Should =copy call extract?

Need benchmarks.

Unless I find a way to make a correct estimate of the BiTable size there is no point in discussing.

Absolutely not. (No hidden gotchas)


Error while installing with recomended setting

I got following error right after running:
npx create-strapi-starter projectFolder next-ecommerce
[develop:frontend] Starting: develop:frontend
[develop:backend ] internal/modules/cjs/loader.js:888
[develop:backend ]   throw err;
[develop:backend ]   ^
[develop:backend ]
[develop:backend ] Error: Cannot find module '/home/myuser/projectFolder/develop:backend'
[develop:backend ]     at Function.Module._resolveFilename (internal/modules/cjs/loader.js:885:15)
[develop:backend ]     at Function.Module._load (internal/modules/cjs/loader.js:730:27)
[develop:backend ]     at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12)
[develop:backend ]     at internal/main/run_main_module.js:17:47 {
[develop:backend ]   code: 'MODULE_NOT_FOUND',
[develop:backend ]   requireStack: []
[develop:backend ] }
[develop:frontend] internal/modules/cjs/loader.js:888
[develop:frontend]   throw err;
[develop:frontend]   ^
[develop:frontend]
[develop:frontend] Error: Cannot find module '/home/myuser/projectFolder/develop:frontend'
[develop:frontend]     at Function.Module._resolveFilename (internal/modules/cjs/loader.js:885:15)
[develop:frontend]     at Function.Module._load (internal/modules/cjs/loader.js:730:27)
[develop:frontend]     at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12)
[develop:frontend]     at internal/main/run_main_module.js:17:47 {
[develop:frontend]   code: 'MODULE_NOT_FOUND',
[develop:frontend]   requireStack: []
[develop:frontend] }
Done in 0.72s.

Before that I've run custom Strapi installation without problem with PostgreSQL database on same server/account.

Error happened because my web server proxy wasn't configured properly.
If you ran across this problem than it's important that you configure backend and frontend server block in your web server configuration.
In my case I was using nginx as web server. Following this answer:
https://serverfault.com/a/792769/336745
helped me configure it properly.
After that installation with PostgresSQL and my own domain for frontend and subdomain for backend went ok.


LaunchAgents

[ ] Move LaunchAgents to ~/Library/LaunchAgents
[ ] LA to update brew
[ ] LA to update GPG keys

See https://github.com/herrbischoff/awesome-osx-command-line/blob/master/launchagents.md


Update Android.gitignore

Reasons for making this change:
TODO
Links to documentation supporting these rule changes:
TODO
If this is a new template:

Link to application or project’s homepage: TODO

I think those files ignored in global/JetBrains.gitignore.

I think those files ignored in Global/JetBrains.gitignore.

☝


Redesign place

Update place pages

Also fixed sidebar scroll issue
Layout updates for different window widths
Shrink "brand" link in header
Default to USA page instead of Mountain View
Fixed fixed-sidebar scroll sometimes overflowing into the footer. Pin it to the main chart container.

This is the last change in the branch before I merge back in to master.


GSOC2016 - Idea #3: Implements SASL mechanisms

Hi @jeanparpaillon
Just to get an understanding of what would this entail, is the idea to implement all the SASL mechanisms  (https://en.wikipedia.org/wiki/Simple_Authentication_and_Security_Layer#SASL_mechanisms)?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/31744361-gsoc2016-idea-3-implements-sasl-mechanisms?utm_campaign=plugin&utm_content=tracker%2F2587299&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F2587299&utm_medium=issues&utm_source=github).

I suppose the title sould be refined as, while D-Bus specifications tells every SASL mechanism should be supported, they are not all implemented by reference implementation (C one) and non-SASL authentication mechanism are implemented (nonce-TCP transport...).
In order to be fully compatible with D-Bus existing implementations, it would be preferable to:

identify all required mechanisms (SASL and non-SASL)
identify all required transports (I suppose transport and authentication mechanisms are not 100% decoupled)
Check which combination should be prioritized and doable in a GSoC.

I consider solving 1 and 2 is a prerequisite for being a good candidate for 3 ;)

Ah, i see, i thought the ideas were separate projects that could be picked independently. I find the whole project interesting but i have doubts on wether i could allocate enough time for it, what would be your estimate of the workload for the whole thing?

I don't have a precise idea of the workload as I have not yet identified what mechanisms are really required.
That's why I propose, as a first step, to identify these mechanisms (which shouldn't take more than a half day...) then discuss which ones are feasible as GSoC.

Got it, closing issue, hopefully will open a new one with a better analysis

Great, thanks !


Add more guardduty support

Added support for AWS_API_CALL action type.

I'm not a AWS GuardDuty user, so I'll take your word for it.  Thanks @stevemac007!

Thanks @homeyjd - I've got a few more changes in flight at the moment, just doing some testing on a number of real accounts to confirm it all works, should have another pull in the next couple of weeks.

Sounds great @stevemac007
FYI, fixed a couple ESLint issues to guardduty.js here: https://github.com/arabold/aws-to-slack/commit/665d50319340139a80a245b48e5d25558aad151e


AVG function - attempt to divide by zero

If there're no files in a directory AVG function fails with an error:
thread 'main' panicked at 'attempt to divide by zero', src/function.rs:382:20
note: Run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

Also MIN function returns -1 in such case, which looks confusing.

Nice catch! Thank you very match for opening this issue, and for the PR as well. Aggregate functions nasty behavior has been fixed in master.


Auto-refreshing realms and notifications possible race condition

I think I ran into a race condition regarding auto-refreshing realms and notifications.
When I have a notification on a realm it causes it to be cached for that thread, which means that it follows the auto-refreshing rules when reading from it instead of always being the latest data (Something I hadn't encountered before I used notifications, since I throw away my realm objects after reading from them). When the notification comes through it causes it to auto-refresh, so the latest write data is available.
The way that I have been using realm is to write in a background thread, then post a block on the main thread to do something with the written data. Before registering for notifications my realm object would always have the latest data, because we would be on the next run loop and the realm object was fresh. Now with notifications I would not find my data available, since my block would run before the notification was posted to the cached realm object.
For now I've fixed this by calling refresh on any realm that I plan on reading from. This is not ideal, so I was hoping that there would be a way to ensure that the auto-refreshing happens before any block of mine is run. I assumed that after the commitWriteTransaction or transactionWithBlock any dispatch_async on the main thread would run on an updated realm.

Brentley,
There are two potential solutions to the issues you're running into. We cannot guarantee that the auto-refresh will happen on any queue before your own block runs, or even that the refresh will happen on the exact next spin of the run loop. That being said, the following should work for you:

Using -[RLMRealm addNotificationBlock:] on your main queue RLMRealm instance. That block will be guaranteed to be called once that thread's realm instance is refreshed.

Do exactly what you're doing, which is dispatching your own notification and manually refreshing the realm on the target thread (that refresh will guarantee that you're dealing with the most recently-committed data).

1 won't work, because it's way too broad, and this is my abstraction layer between my cached data and the client (we don't consume Realm objects in the app).
2 is fine, I just thought it was non-performant, otherwise it would already be done for auto-refreshed realms when I called [Realm realmWithPath].
Also, it's not any queue, it's the main queue, which is the same queue that I have the notification setup on. By calling dispatch_async after the write commit, I am already after the point that realm itself should have done the same thing, so my block should be run after it (the blocks are queued serially). I believe the issue is that Realm is using performSelector, which is causing the realm notification to happen after block dispatches, since I read somewhere that the run loop will interleave two different types of events on the run loop?
Anyway, maybe there is another way for me to queue so I'm guaranteed to run after the notification, if realm won't change to update sooner.

 Calling refresh with nothing to do is very close to free, so calling it before the autorefresh happens just means that the work happens at a different time.
The notifications being unordered due to the mixing on GCD and performSelect:onThread: does sound plausible to me, since they do pretty different things.

@BrentleyJones I just want to follow up and make sure you have refreshing working in your app?

Calling it manually is working.

Glad to hear it!


Stop Order

Hi,
I am fairly new to this, so please excuse any mistakes.
i am currently trying to build a test application for my strategy. The strategy needs stop orders, E.g. a buy order that executes after a price that is higher then the current price is reached, which works in the R Trader. is that also possible with your application?

Yes this is possible with some python code.


uBlock Origin breaks deploy previews

No idea why, but it needs to be turned off for the deploy previews right now.

Could be related to rollbar, or specific class names? Interesting that we're tripping anything at all.

Maybe it could be related to Netlify itself? There are two filters that include this domain but I'm not sure yet how to exactly interpret these rules
netlify.com,scrims.io##+js(bab-defuser.js)
or
||netlify.com/$script,redirect=noopjs,domain=duellinksmeta.com

This seems to have been resolved upstream in uBlock Origin. Thanks @mapx and @MarkTielemans!


Is there any plans for UI?

^

^
I think this would be great, especially because make easier to set buttons
Also I considered important suggest, a waybto provide a simple package as appimage, flatpak or snap ; for users not so professionals for use packages or the terminal.

Well this script started very small for my own needs and I had no plans for a UI. But I'll think about it.


App not Working on Huawei Devices

Despite  #474 /  #484  there is also to be noticed, that the App is not useable on devices with HMS.
All newer Huawei devices are banned from Goolge/PlayStore due to a presidential order. Without an apk or a upload to the Huawei store, there'll be approx. 1M Users not able to use this app.

Same problem for me, although I have installed Google Play Services besides Huawei Mobile Service: https://github.com/corona-warn-app/cwa-app-android/issues/478#issuecomment-644553948

@Huawei implimented an own framework called HMS Core Contact Shield. This could be may implemented for users with Google Serivce baned Huawei devices.

In fact the app should be available as .apk in https://github.com/corona-warn-app/cwa-app-android/releases
That version should later be extended to update itself without the need for a store or google.
It could also be added to other stores like i.e. https://f-droid.org/. That is where true FOSS goes ... not "don't be evil"

+1
Otherwise people will download your standalone APK from sources like https://apkpure.com/de/corona-warn-app/de.rki.coronawarnapp/download?from=details and you probably don't want that.

https://github.com/corona-warn-app/cwa-documentation/issues/5

The respective information, especially the issue https://github.com/corona-warn-app/cwa-documentation/issues/5 has already been posted. We are closing this issue now...
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team

@SebastianWolf-SAP will there be an implementation of HMS services in your app? Currently it means all phones of Huawei are not supported at all. By closing this issue, all Huawei, Honor, and so on will be left out to participate in the app pogamme and therefore in taking care of our health.

@mat-l As @SebastianWolf-SAP  linked the decision is to use the API from Google and Apple.
 You are for now in pity as well as all other non iOS/Google Service Android users.
Welcome to the walled garden called monopoly!

@Nokius yes and we the tax-payers of Germany payed about 20 Million Euros to SAP (and apparently Telekom) for this solution.  But we are obligated to clap our hands that this stuff is open source.

@SebastianWolf-SAP @Nokius yeah its the trade war, I commented as well over here: https://github.com/corona-warn-app/cwa-app-android/issues/510#issuecomment-644693694
In my opinion there must be support for HSM as well!

@SebastianWolf-SAP Stop closing issues with unrelated references.
A link to the documentation is not sufficient. Please update your FAQs and general public facing documents to mention which phones are not supported rather than deeplinking complicated documentation.
This issue was about Huawei, not F-Droid. In your link I can't find a Huawei mention.

@Nokius yes and we the tax-payers of Germany payed about 20 Million Euros to SAP (and apparently Telekom) for this solution.  But we are obligated to clap our hands that this stuff is open source.

Ok, how would you expect that SAP or German government would resolve the ongoing trade conflict between USA and China (Google and Huawei, respectively) for a tiny Corona warn app?

@s-martin they can't stop it, but they should provide an solution that fits all, that means also to implement HMS Core services, which I wrote here: https://github.com/corona-warn-app/cwa-app-android/issues/510#issuecomment-644693694

The Exposure Notification API is provided by Google via Play Services, so there's no workaround at this point.

@carlos22  I agree with you it's tax payers money used here.
Personally I'm very happy to see that public money has turn into public code.
But 20 Million are not covering corner cases.
Like:

non smartphone users with feature phones
gray market devices with no Google Services
alternative OS/ROM's
Android devices with no Google Services
...

As we both found out it's open source so you or anyone else can modify the code and open a PR. What about you @Huawei wanna get you users under the corona-warn-app umbrella?

Huawei has a market share of 16% in Germany - that is quite significant. It's more than half of Apple's. So it can be considered worth it's own app - if the changes required aren't huge. Marginal costs per marginal benefit could be lower than for Apple with only 30% market share.
https://gs.statcounter.com/vendor-market-share/mobile/germany

@corneliusroemer is it now 16% or 17%. Anyway I don't thinks all of them are without Google Service. If so it's still a user choose and not the fool of the app developer(s).

@Nokius Not sure how many devices are affected. But Huawei has significant market share. Same can be said about Apple, why doesn't everyone use Android. Sure Huawei doesn't offer the API, so that's Huawei's fault. Not sure if there's a workaround possible.

Huawei will offer an API as @Nokius mentioned above.

Did anyone bother telling google about the problem and maybe ask them to make that code available?
That would allow for a truly free implementation, which would suddenly also cover huawei devices and not just lineageOS and friends. Who knows, they might be willing to help out.

The Exposure Notification API is provided by Google via Play Services, so there's no workaround at this point.

Yes but that is the essential part. Which I would have expected to be implemented by this app. But software development today means you pay a lot of money for people making nice UIs and really using FAANG frameworks. Thats like giving up.

Did anyone bother telling google about the problem and maybe ask them to make that code available?
That would allow for a truly free implementation, which would suddenly also cover huawei devices and not just lineageOS and friends. Who knows, they might be willing to help out.

They sure will not. Google Play Services are essential to what keeps them having full control over Android. Very clever people at Google found this loophole to make sure everybody still likes Android because it is open source but still are able to make contracts with OEMs and ship code to every smartphone. They will never give you anything that is inside of Play Services.g that is inside of Play Services.

@SebastianWolf-SAP I think you should reopen this issue here - it's unrelated to F-Droid. It asks specifically for compatibility with a Huawei API that is TBA.
There should be special regards for Huawei while the issue is not resolve - if ever. Doesn't harm to have one issue that acts as a reminder that Huawei support is know to be limited.

@corneliusroemer As I linked earlier today, currently the implementation is only for Apple and Android ( Google Service API). Here again the link to the developer mentioning it.
If you like to see support for an minority group of users, create fork, read the Huawei documentation, code, create a PR -> provide profit for the group of users. Looks like you are effected and you have the ability to solve it it's open source and you can make it happen.

@henning-schild come on google is aware of this 'issue' and they do it (from my point of view) on purposes. Google will never give up on users data as they are an ad company.  So the google services are imported for them.

Dear @corneliusroemer,
we have stated repeatedly, also in our FAQ and also in the official communication that only iOS and Android devices running Google Play Services are supported and we are not aware of additional plans. Moreover, there is currently no sign that the German Federal Government and/or the Robert Koch Institute will contract us to implement the app on other platforms.
We cannot state more clearly than what we've already done. And to be even more precise: We're the wrong contacts for such questions. Requests to implement the app on other platforms need to be ordered by the government or implemented by the community as @Nokius said.
So either somebody starts doing the work here (and announce that accordingly such as the folks in #75 for a free implementation of Google Play) or we will keep this issue closed with the label 'wontfix'.
Thank you for your understanding!
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team

I personally also believe that google is aware and they are locking things down on purpose. But that does not rule out talking to them! Until somebody does that we are just working on assumptions here.
And with "somebody" i am thinking of SAP and Telekom with the COVID health mission, they might have some leverage. They should at least ask instead of refusing/ignoring true FOSS ... maybe based on the same assumptions that others already mentioned here.
To be honest i could not care less about the app ... i just see the leverage as a potential to open up android. That would be a good tax invest with or without the virus

@henning-schild For sure the Federal Republic of Germany in person as RKI can try to contact Google so they may will debundle the Proximity Tracing API from the Google Services. BUT how Android will get the API then? To add it directly to the AOSP base or the deviates OEM/Manufacture create will take month/years or never happens as this will not push out Updates anytime soon.
Also the User who not install this OS Updates are may bigger then User who avoid Application Updates which in fact Google services are.
From my point of view Europe should wake up now and start an European driven mobile OS. So we [Europe / Federal Republic of Germany] could find solutions and not try to bargain with 'ruler'.

@Nokius i would not day "Europe" because there is no reason to limit the scope of a liberation, maybe the pressure and funding could initially come from there.
The media seemed to focus more on availability on older devices, which just boils down to the same problem. Even if things will take a lot of time and this app will not get its fast solutions ... getting google to open up and loose control would be a step in the right direction. Maybe for the next pandemic or simply the next federal effort to release FOSS apps, which will hopefully happen after this one.


add linux resolvers to settings

This is adding existing resolvers for linux

But it might be a good idea to give app_name a better user hint. (As I understand it, app_name is assembled from Applications addon Hostname + Variants_label.) This is not clear when just looking at the settings page.

Yeah, I think this is something temporary, it needs more user friednly mapping, maybe even dynamic enum of applications. I plan to address this in documentation.


Application singlePageApplication redirectUris incorrectly validated

Describe what happened
I have the need to create a brk-multihub redirect uri for an Application under singlePageApplication.redirectUris as part of NAA for Outlook (see https://learn.microsoft.com/en-gb/office/dev/add-ins/develop/enable-nested-app-authentication-in-your-add-in).
I can set these via a separate azuread.ApplicationRedirectUris resource, validation passes. I cannot use this approach though because the main Application is constantly updated (see #1685) and these redirect uris are being wiped out.
If I instead use the properties of the main resource I get validation errors:
azuread:index/application:Application resource 'main-application' has a problem: URI must have a trailing slash when there is no path segment for "single_page_application.0.redirect_uris.0". Examine values at 'main-application.singlePageApplication.redirectUris)
or, with the trailing slash:
azuread:index/application:Application resource 'main-application' has a problem: unexpected URI scheme for "single_page_application.0.redirect_uris.0", expected one of: http, https, ms-appx-web. Examine values at 'main-application.singlePageApplication.redirectUris'.
I'm not sure if this is actually an underlying problem in Terraform or here in Pulumi
Sample program
import * as azuread from '@pulumi/azuread';

new azure.Application('app', {
    displayName: 'Test',

    singlePageApplication: {
        redirectUris: [
            // Required for Outlook Add-In NAA
            pulumi.interpolate`brk-multihub://my-add-in-domain.com`,
        ],
    },
});

Log output
No response
Affected Resource(s)
azuread.Application
Output of pulumi about
CLI
Version      3.140.0
Go Version   go1.23.3
Go Compiler  gc
Plugins
KIND      NAME          VERSION
resource  azure         4.37.0
resource  azure-native  2.72.0
resource  azuread       6.0.2
resource  cloudflare    5.43.1
resource  docker-build  0.0.7
resource  kubernetes    4.15.0
resource  newrelic      5.10.0
language  nodejs        unknown
resource  random        4.0.0
resource  tls           4.0.0
Host
OS       Microsoft Windows 11 Pro
Version  10.0.22631 Build 22631
Arch     x86_64
This project is written in nodejs: executable='C:\Program Files\nodejs\node.exe' version='v22.1.0'
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hi @barclayadam, I'm afraid this is a limitation in the upstream TF provider. brk-multihub is not among its allowed URL schemes.
I don't see an issue for it yet in their issue tracker, so filing one would be the best course of action.

I also noticed that the Pulumi docs for ApplicationRedirectUris omit an important note that the TF docs have:

This resource is incompatible with the azuread_application resource, instead use this with the azuread_application_registration resource.

Thank you @thomas11, an issue has been raised with the Terraform provider.
Interestingly I've been using ApplicationRedirectUris successfully with Application for many months now. It's the only way to create some of the redirect URIs required for Outlook add-ins, as they need the client ID, available in the Application instance itself to create the redirect URIs.
That is for PublicClient. It's SPA type that is not working for me only.


Read configs from a json serialized env var

Feature Request
In Sys Integ tests, configs could be loaded from a json serialized env var. Would be nice to have an extension method on IConfiguartionBuilder to read from env var and deserialize it into a POCO.
This helps with setting up test env such as in Travis CI when we need to worry about only about 1 env var.

kind of workaround: https://gist.github.com/kr/6161118 converts json to env variables. you only have to change contents of json file. giving more thought to it - how is that different from using application.json?
Or you suggest using whole configuration in single env var?

@karb0f0s I am looking for a way to have settings in an env var. This might need some IConfigurationBuilder extensions to read that env var and convert back from JSON.
export CI_BUILD_SETTINGS='{ "Tests": { "Execute":true } }'

var testSettings = Configuration.GetSection("Tests").Get<TestSettings>();


Use observable for OS proposal timeout

Description of the Change
Reworked&reopened feature of using observable for making proposal timeout in Ordering Service (from #1196)
Benefits
Testable code, and no even more rare RC-bugs
Possible Drawbacks
None
Usage Examples or Tests [optional]
ordering_service_test

Is it ok that ordering_gate_service_test fails?

Is it ok that ordering_gate_service_test fails?

nope


fix(alpine) add a missing golang build step required for alpine

:tada: This PR is included in version 4.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add notification metrics

It adds notification related prometheus metrics, including:

total count for events/success/failure/error
total count for notification per each status code
gauge of the pending notification queue

Signed-off-by: tifayuki<EMAIL_ADDRESS>

Same as the other PR.
Can we see the output?
Thanks.

An sample output is:
# HELP registry_notifications_events_total The number of total events
# TYPE registry_notifications_events_total counter
registry_notifications_events_total{type="Events"} 8
registry_notifications_events_total{type="Successes"} 8
# HELP registry_notifications_pending_total The gauge of pending events in queue
# TYPE registry_notifications_pending_total gauge
registry_notifications_pending_total 0
# HELP registry_notifications_status_total The number of status code
# TYPE registry_notifications_status_total counter
registry_notifications_status_total{code="200 OK"} 8

@stevvooe @dmp42

@stevvooe any chance to take a look at this? would be great to have this change in.

@dmcgowan @stevvooe  friendly ping on this one :)

@tifayuki Could you add endpoint label also to this PR?

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "notification_metrics" git@github.com:tifayuki/distribution.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354473512
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@manishtomar I added the endpoint label. Can you take a look at it?

@tifayuki Thanks! Code LGTM. Could you merge latest master and push it? I wonder if it will fix travis build.

@manishtomar
I merged the upstream master to this PR. But on Travis, it always shows:
The command "export CGO_ENABLED=$TRAVIS_CGO_ENABLED" exited with 0.
0.23s$ DCO_VERBOSITY=-q script/validate/dco
/home/travis/gopath/bin/git-validation
 99ce26e - FAIL - has whitespace errors. See `git show --check 99ce26efeda8fc8661974a1d3edaab0740c2abc3`.
1 commits to fix
The command "DCO_VERBOSITY=-q script/validate/dco" exited with 1.

Any idea what error it is?

@tifayuki can you rebase on master and see if the build still fails?

@mrueg it worked

@dmcgowan any chance to get this in for 2.7.1?

LGTM
If you are going to open up a backport to 2.7 for this, please provide strong justification and show how it is not impactful for compatibility. Such a change in a patch release is very irregular, but I understand there is no 2.8 currently on the horizon.

Can you squash this before merging? Will make cherry-picking cleaner

If you are going to open up a backport to 2.7 for this, please provide strong justification and show how it is not impactful for compatibility.

It does not impact compatibility. I originally wanted it in 2.7 release as this change is being used in Hub registry but missed the train. I thought we could get it in 2.7.1|2 release but I that was assuming that 2.7.x release will be based on master branch which is incorrect. Hence, please ignore my request to put this change in patch release. It can go in next release. I just wanted to get this merged. Also I apologize I just realized a mistake when looking at the code and have requested these. We can merge it after @tifayuki makes those changes. Thanks for your time!

@tifayuki Thanks for the changes. However build is failing due to unsigned commits which was GH fault when accepting the suggestion. Could you please rebase with one signed commit? Also, you will need to put .Inc(float64(len(events))) to fix the lint error.

@dmcgowan Can this be merged now please?

do we want some of the commits to be squashed before merging?

@tifayuki Can you please squash commits?

Actually since this only needs to be merged and not cherry-picked it should be ok to squash during merge @dmcgowan . If you can you or @caervs please merge this?

@caervs Can you please merge this?


Windows Support

What are the additional dependencies or configurations I would need to build and run this on a Windows based platform? If the SDK needs more work to support Windows, we are happy to contribute. Any resources or pointers would be appreciated.

Hi @nipunasudha
The project needs modifications before being able to talk to the new Algorand API. Unfortunately I don't have time to spend on porting the SDK to the new API.
As for the Windows support, I don't think it would need that many modifications but I never tried compiling for Windows. I think it's worth trying to see what kind of error can arise.
If you wish to contribute, please fork the repo and start working on it, I would be happy to merge anything that you do on this repository! 🙂

Thank you very much for the information, we will start working on porting this package to new API. First step is looking for what changed from the old API 😁 If you have any Algorand API migration guide or change log resources, please do share with us.


Can't pip install on Mac

hey, when I run the pip command it gives me the following error:
ERROR: Could not find a version that satisfies the requirement pdftotext (from versions: none)
ERROR: No matching distribution found for pdftotext
is there another way to install it, or solve this way?

Can you provide more detail? What command did you type? What was the full output?

simply typed pip install pdftotext on terminal, and received the following error

Owners-MacBook-Pro:~ owner$ pip install pdftotext
You are using pip version 6.1.1, however version 19.2.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
Collecting pdftotext
Retrying (Retry(total=4, connect=None, read=None, redirect=None)) after connection broken by 'ProtocolError('Connection aborted.', error(54, 'Connection reset by peer'))': /simple/pdftotext/
Retrying (Retry(total=3, connect=None, read=None, redirect=None)) after connection broken by 'ProtocolError('Connection aborted.', error(54, 'Connection reset by peer'))': /simple/pdftotext/
Retrying (Retry(total=2, connect=None, read=None, redirect=None)) after connection broken by 'ProtocolError('Connection aborted.', error(54, 'Connection reset by peer'))': /simple/pdftotext/
Retrying (Retry(total=1, connect=None, read=None, redirect=None)) after connection broken by 'ProtocolError('Connection aborted.', error(54, 'Connection reset by peer'))': /simple/pdftotext/
Retrying (Retry(total=0, connect=None, read=None, redirect=None)) after connection broken by 'ProtocolError('Connection aborted.', error(54, 'Connection reset by peer'))': /simple/pdftotext/
Retrying (Retry(total=4, connect=None, read=None, redirect=None)) after connection broken by 'ProtocolError('Connection aborted.', error(54, 'Connection reset by peer'))': /simple/pdftotext/
Retrying (Retry(total=3, connect=None, read=None, redirect=None)) after connection broken by 'ProtocolError('Connection aborted.', error(54, 'Connection reset by peer'))': /simple/pdftotext/
Retrying (Retry(total=2, connect=None, read=None, redirect=None)) after connection broken by 'ProtocolError('Connection aborted.', error(54, 'Connection reset by peer'))': /simple/pdftotext/
Retrying (Retry(total=1, connect=None, read=None, redirect=None)) after connection broken by 'ProtocolError('Connection aborted.', error(54, 'Connection reset by peer'))': /simple/pdftotext/
Retrying (Retry(total=0, connect=None, read=None, redirect=None)) after connection broken by 'ProtocolError('Connection aborted.', error(54, 'Connection reset by peer'))': /simple/pdftotext/
Could not find a version that satisfies the requirement pdftotext (from versions: )
No matching distribution found for pdftotext

already solved it , needed to disable ipv6 while using pip due to my wi-fi router I think

:+1:

Now I'm getting this error

Owners-MacBook-Pro:~ owner$ pip install pdftotext
You are using pip version 6.1.1, however version 19.2.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
Collecting pdftotext
Using cached https://files.pythonhosted.org/packages/a6/a7/c202adb0bcd3adc3030b0c5f7f0e21f62a721913e93296e6c4ddc305cbd3/pdftotext-2.1.2.tar.gz
Installing collected packages: pdftotext
Running setup.py install for pdftotext
Complete output from command /usr/bin/python -c "import setuptools, tokenize;file='/private/var/folders/vh/ktk4_qrn5k7_043z22q7y2p00000gn/T/pip-build-mzgEx9/pdftotext/setup.py';exec(compile(getattr(tokenize, 'open', open)(file).read().replace('\r\n', '\n'), file, 'exec'))" install --record /var/folders/vh/ktk4_qrn5k7_043z22q7y2p00000gn/T/pip-S6ghdk-record/install-record.txt --single-version-externally-managed --compile:
/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/distutils/dist.py:267: UserWarning: Unknown distribution option: 'long_description_content_type'
warnings.warn(msg)
running install
running build
running build_ext
building 'pdftotext' extension
creating build
creating build/temp.macosx-10.14-intel-2.7
cc -fno-strict-aliasing -fno-common -dynamic -g -Os -pipe -fno-common -fno-strict-aliasing -fwrapv -DENABLE_DTRACE -DMACOSX -DNDEBUG -Wall -Wstrict-prototypes -Wshorten-64-to-32 -DNDEBUG -g -fwrapv -Os -Wall -Wstrict-prototypes -DENABLE_DTRACE -arch i386 -arch x86_64 -pipe -DPOPPLER_CPP_AT_LEAST_0_30_0=0 -I/usr/local/include -I/System/Library/Frameworks/Python.framework/Versions/2.7/include/python2.7 -c pdftotext.cpp -o build/temp.macosx-10.14-intel-2.7/pdftotext.o -Wall -mmacosx-version-min=10.9
pdftotext.cpp:3:10: fatal error: 'poppler/cpp/poppler-document.h' file not found
#include <poppler/cpp/poppler-document.h>
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
error: command 'cc' failed with exit status 1

----------------------------------------
Command "/usr/bin/python -c "import setuptools, tokenize;__file__='/private/var/folders/vh/ktk4_qrn5k7_043z22q7y2p00000gn/T/pip-build-mzgEx9/pdftotext/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /var/folders/vh/ktk4_qrn5k7_043z22q7y2p00000gn/T/pip-S6ghdk-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /private/var/folders/vh/ktk4_qrn5k7_043z22q7y2p00000gn/T/pip-build-mzgEx9/pdftotext

Did you install the dependencies? brew install pkg-config poppler is the usual way to do it on macOS.

Hi @R470R, did you solve the problem ? I have the same issue...

Still having the same issue in 2022, on a M1 Macbook Pro. Anyone else have this issue? I tried all the things mentioned on this post.

Ping! It's March 14, 2024! Happy Pi Day! Has anyone figured out a way around this?


[BUG] MERGE plugin errors with pass+drop+no-psi-merge+ignore-conflicts

Bug description:
File1.ts is a SPTS with only PAT+PMT+VIDEO+AUDIO (from ffmpeg)
File2.ts is a regular MPTS with multiple services
I'm merging to "File1.ts" the PSI/SI tables from "File2.ts", the result fails as it contains less pids or more pids than wanted.

Then video (512) and audio (4096) pids are translated to the pids 201,202 used in the File2 using the remap plugin.
After that the merge plugin wants to include with -p 200 the PMT table, with -p 0-20 the PSI/SI tables, and with -d 201-202 it will want to block the original audio and video.
Expected behavior:
The result is:

With this exact line, all A/V pids are merged and no PSI/SI tables are added.
If I remove the "pass" values and it remains the "drop", then only the PMTs are added, but the PSI/SI (0-20) are missing.

Errors and logs:
No errors printed.
Environment:
Not relevant, but last commit compiled in Linux.
Additional information:
The bug is then that with the merge plugin when using --no-psi-merge --ignore-conflicts it's not possible to clean pass the 0-20 pids. Futhermore, when mixing --drop and --pass parameters then inconsistent results are obtained.

Using different input files (I don't have yours) and the following command
tsp -I file file1.ts \
    -P filter -s -n -p 0-20 \
    -P merge --no-psi-merge --ignore-conflicts -d 201-202 -p 200 -p 0-20 "tsp -I file file2.ts" \
    -P psi \
    -O drop

The output exactly contains the PSI of file2.ts (PAT, PMT, SDT).
There must be something wrong in the way you interpret that the PID's 0 to 20 are not present in the output. Maybe the result is something wrong with the overall structure of the output due to incorrect manipulation. Check that PID's 0-20 are actually present in File2.ts and in output.ts.

Hi @lelegard ,
Sorry but the bug is here (I just repeated now). Please, retry with this command:
tsp -I file file1.ts \
    -P filter -s -n -p 0-20 -p 4096 \
    -P remap -n -u 256=201 \
    -P remap -n -u 257=202 \
    -P merge --no-psi-merge --ignore-conflicts -d 201-202 -d 301-302 -p 200 -p 0-20 "tsp -I file file2.ts" \
    -P analyze \
    -O drop

Please, check this:

The file1.ts has video pid 256 and audio pid 257, PMT in 4096 (default output from ffmpeg).
The file2.ts has multiple services. Pids 301,302 are A/V of another service. Pids 201,202 are A/V of the "replaced" service. PMT 200 is the one of this service.

Then the analysis indicates:

Pids 301 and 302 are present in the output, even the -P merge -d 301-302;
Pids 201 and 202 have Unexpected and Duplicated packets, event the -P merge -d 201-202;

So you can see that the cause of this is because all pids from the merged stream are inserted, even the --drop parameters.
And that's the first part. If you remove every --pass then all is as expected, except that you don't have the PSI tables (obviously).
The error appears when MIXING "drop" and "pass". In this case the result is indeterminated. In fact, it's in this way: pass all. But this is not the expected behaviour.
I hope you can reproduce now, and fix it.
Thank you!

OK, you initially described the problem as "it's not possible to clean pass the 0-20 pids". This is why I responded that was not the case.
There is in fact an issue (I just fixed it) with options --drop and --pass. For some reason, they used an old manual way of processing ranges of integers which was incorrect. The issue is with -p 200, one value, not a range. It is incorrectly interpreted as "pass PID 200 and subsequent", canceling the previous -d 201-202 -d 301-302 by re-allowing those PID's.
Just as a test, you will notice that -p 200-200 will work because this is a range (of one value). This is the bug I just fixed.
But notice that -p 200 is useless anyway. The default is to drop PID's 0-31 and pass 32-8191. So, -d 201-202 -d 301-302 -p 0-20 make sense because they change the default. But -p 200 is useless because PID 200 is already passed by default.
Finally, please note the option:
  -t
  --transparent
      Pass all PID's without logical transformation. Equivalent to
      --no-psi-merge --ignore-conflicts --pass 0x00-0x1F.

So, your command can be simplified as -P merge -t -d 201-202 -d 301-302 "tsp -I file file2.ts".

Hi @lelegard ,
Thank you for fixing it! That's the typical error not related with what we think, and created by ancient code.
Before closing the issue, please add this comment to the plugin help and documentation: "The default behaviour is to pass all pids": https://github.com/tsduck/tsduck/blob/3d4091d310980198d188b0af9e33f1002286bcf8/src/tsplugins/tsplugin_merge.cpp#L338
Regards.


Can't add password redis

i connect redis is okie but when usenodejs-health-checker for check integration i don't see where to put password
and rep always return false

Hello @minhkhoa5899 !
you can use password in the object you use in the list of integrations

Should be something like this:

await HealthcheckerDetailedCheck({
    name: "My node application",
    version: "my version",
    integrations: [
      {
        type: HealthTypes.Redis,
        name: "redis integration",
        host: REDIS_HOST,
        auth: {
          password: REDIS_PASSWORD
        }
      }
    ]
});


Game Server Console - Terraria Server Console Like Behavior

Request
Instead of the input always being at the bottom, it should always be right under the last input entry.
Alternatives
I've considered leaving it the way it is, but it would be nice to make it feel more like a Minecraft / Terraria console application.

Solved in upcoming commit.


Changing avatar image results in "files[normalizedPath] is not a function" Error

I'm trying to change the avatar image, but when using an image source other than the demo, this error shows:
 files[normalizedPath] is not a function
at misc/ImageWrapper.astro:23:38
and I can't figure out how to solve this.
this theme looks amazing btw!

Thank you for your support!
This error usually occurs because the image cannot be found at the given path. Please check if the image path you entered is correct.

If the path starts with /, then this path is relative to the /public directory.
Otherwise, the path is relative to the /src directory.

I hope this will solve the problem.

hi again!
my image is in /src/images/avatar.png
src/assets/images/avatar.png = error
assets/images/avatar.png = error
i have tried moving the image to the public directory, this is what happens
/public/avatar.png = error
/avatar.png = error

this is weird, everything seems to work fine if i clone the theme again and start over. guess ill just do that


i cant access my fuel wallet in my chrome extension it keeps loading and when i try to update it,it just loads

What version of Wallet are you using?
0.42.3
What is happening?
when i try to login in it just keeps loading after inputing the password and i try to update the extension but its not working
steps to reproduce
it should please be resolved
How it should behave?
it should come on
What os are you seeing the problem on?
Windows Desktop
What browser?
Chrome
What is your browser version?
131.0.6778.109

Hello @mubii1
We apologize for the inconvenience you’re experiencing. For personalized assistance, please contact our support team directly.
Ticket ID:FUE678
Fuel support
To start a chat with a live support agent, click the live chat icon at the bottom corner of the page.


Failing test case - Usage of deprecated constant via subclass is not marked as deprecation call

The same problem is with deprecated constant defined in interface and used via subclass.
See e.g. https://github.com/nette/application/blob/v2.4/src/Application/IRouter.php#L22 used with SimpleRouter:
Code \Nette\Application\Routers\SimpleRouter::SECURED throws no error

Hi, I don't get it, you just changed some tests but the tests are passing? Either fix the code, or provide a failing test with expected results...

Hi, sorry, it is failing on my env. I have to find differencies. First is my php version 7.4.14

Hi, I can't isolate the problem in this repository for unknown reason, so I've created very simple app to reproduce my problem. If you will have some time, please check it on https://github.com/lulco/deprecated-constant
It is simple Nette 2.4 "application" with RouterFactory where deprecated flag IRouter::SECURED is used. Once directly via IRouter and once via class SimpleRouter (which implements IRouter).
I believe there should be two errors (on lines 15 and 16) in build:
https://github.com/lulco/deprecated-constant/runs/2116992714

Fixed in PHPStan: https://github.com/phpstan/phpstan-src/commit/6d523028e399c15dc77aec3affd2ea97ff735925


Testing dataset likely poorly represents the set of images that a Firefox description writer would be used for

https://github.com/mozilla/distilvit/blob/9c301fd5ba1f62ab407ca0a342642666a1ec13c5/docs/fighting_bias.md?plain=1#L113
Having scanned through the photos being used in testing the AI being developed, they seem similar in important ways to the dataset I used in testing my options around direct comparison algorithms for very close but not equal images, think the same image resized and/or compressed, explained here: https://github.com/alt-text-org/image-algo-testing.
I realized in my exploration of the results that the image data I used, snagged from some facebook algorithm invention event, poorly represented the set of photo categories that were posted by users of social media, my prioritized userbase. I prioritized those users in part because I believed the vast majority of usages of the tool I was building would be pictures being sent to social media. I believe that an AI description engine would be used by a very similar population and dominated by people posting images to social media. That belief is based on my last 3 years of deep involvement in and creation of alt text writing tools, but is worth more in depth and documented exploration and research into classes of image likely to be fed to the AI for description.
After some consideration I decided that given the nature of my tests mentioned above that the dataset used was not reliable, but given the still clear results of that testing and the existence of several other "in future testing..." items I did not seek to re-run with a more representative collection at that time. I assert that the image type is much more important here due to the nature of the algorithms being tested and their likely usage.
I think it is important to make decisions on this project based not just on my claims here but active research and questions to community about how such a tool is likely to be used, and then integration of that into design and testing of this tool. I assert that a public release prior to said research and communication is not responsible if the goal is increasing web accessibility.

Thanks a lot for your feedback. We'd love to engage more with the community to make sure we do the right thing. Do you have some tips on how we can improve the training? or some folks we can reach out?

Thanks a lot for your feedback. We'd love to engage more with the community to make sure we do the right thing. Do you have some tips on how we can improve the training? or some folks we can reach out?

I just emailed, but want to say here that I now know that my understanding of Mozilla plans was mistaken, and so while the concerns above are valid I do not think this issue applies to this repo and will close it. As I said in the email, I still think it's worth discussing with y'all and hope to do so.


chore : remove kernel headers mount if btf found with initDeploy = true

Purpose of PR?:
Fixes #
Does this PR introduce a breaking change?
If the changes in this PR are manually verified, list down the scenarios covered::
Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Checklist:

[ ] Bug fix. Fixes #
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[ ] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Commit has unit tests
[ ] Commit has integration tests

Failing CRI-O tests are not relevant to the changes here. They'll be handled separately.


Remove the istensor local in nesting.lua.

This enables support for other Tensor implementations, which may provide
a new version of torch.isTensor but only be required after nngraph.

Thanks!


Add streets URL to slack app post

Summary
Pretty straightforward change to link out to Delivery Needed map from the Slack request post.
It'll link out to something like https://crownheightsma.herokuapp.com/delivery-needed?request=FDNNBQ.
Example from my dev Slack app:

Hover on link:

Closes #78
Open questions
The map link causes an attachment to appear in the post. Is there a way to tell Slack not to add attachments for the post? I tried looking up ways to do this but couldn't find anything. We can block attachments for any URL domain, but unsure if that's the best way forward.
Slack post attachment I'm referring to:

Thanks! There is an option for the postMessage we make to "unfurl" media or not.
Docs here. Think we want  both unfurl_links and unfurl_media set to false

@mjmaurer excellent, exactly what i was looking for! see screenshot for currents state without unfurled media:


[AMBARI-23918] [Logsearch UI] 'Component' option missing in the autofill in include filter

What changes were proposed in this pull request?
The typeahead library should look in the labels not in the values.
How was this patch tested?
Manually and by unit tests.
Please review Ambari Contributing Guide before opening a pull request.
@aBabiichuk please review it. Thanks.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2399/
Test PASSed.


BUG: MacOS-specific NWJS Menu command prevents Windows app from running JS

Tested on v0.2.0
Error: extensions::sendRequest:148 "Uncaught".

Line 19 of settings.js: Menu.createMacBuiltin(nw.App.manifest.name);

Here's some information about detecting OS: https://adamlynch.com/platform-detection/

Re-opening—this would require additional logic to check within the NW.js loop for specific OS'.


Video freeze while switching

The Video freeze while loading the content, when switching from a playlist with video to a playlist with AF.
Tested with Raspberry Pi:
UID (for REST API)
2e3dce4da4d3176c63a219bdd63b8588b81ddf6e50ae52bf08cc4
DUID
D9d7bf621f111f187c730e37f95fa83a05ea7e70552c71bb01
Version 1.4.2_1.7.0100
The smil with the video: https://deploy.screenfood.com/lightplayer/smil/rspivid/asdf and
the smil with the af: https://deploy.screenfood.com/lightplayer/smil/rspiaf/asdf

Hello,
are you switching playlists manually, or does this issue happen when you update your smil file and smil player updates automatically?

Hello
We push the new playlist directly from the CMS to the player.

Hi @rst303, I think that the visual mechanism is that we push the changes from the CMS, but what happens underneath is this:
we update the smil file and smil player updates automatically after reaching the refresh interval

Hello,
thanks for the clarification. I will look into it.

@PetrDusek89 : I am sorry, the playlist https://deploy.screenfood.com/lightplayer/smil/rspivid/asdf  was changed, now it has that one which creates the issue.

Hello,
I am having hard time reproducing your issue. If it is possible could you please be more specific which video freezes for you and when exactly does this happen?
Thanks Petr

Hello,
this issue should be resolved in latest release. Please upgrade your SMIL player to the latest master (1.8.0).


lambda deploy is not working

Env: Linux Ubuntu 22
Below is a script of commands to set up environment and run deploy
#!/bin/bash

mkdir lambda_func2 && cd lambda_func2
# python3 -m venv ./.venv
# source ./.venv/bin/activate
# pip install python-lambda
lambda init
# supply aws access key and secret access key to config.yaml
# change the region to your region inside config.yaml
# optional: change the function name inside config.yaml
lambda invoke -v
lambda deploy

Using the script above will output an error message
botocore.exceptions.ClientError: An error occurred (UnrecognizedClientException) when calling the GetFunction operation: The security token included in the request is invalid.

Here's the full StackTrace
Bundling: 'install.sh'
Bundling: 'event.json'
Bundling: 'service.py'
Traceback (most recent call last):
  File "/home/adminUser/Downloads/examples/design/Examples/lambda_func2/.venv/bin/lambda", line 213, in <module>
    cli()
  File "/home/adminUse/Downloads/examples/design/Examples/lambda_func2/.venv/lib/python3.10/site-packages/click/core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "/home/adminUser/Downloads/examples/design/Examples/lambda_func2/.venv/lib/python3.10/site-packages/click/core.py", line 696, in main
    rv = self.invoke(ctx)
  File "/home/adminUser/Downloads/examples/design/Examples/lambda_func2/.venv/lib/python3.10/site-packages/click/core.py", line 1060, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/adminUser/Downloads/examples/design/Examples/lambda_func2/.venv/lib/python3.10/site-packages/click/core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/adminUser/Downloads/examples/design/Examples/lambda_func2/.venv/lib/python3.10/site-packages/click/core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "/home/adminUser/Downloads/examples/design/Examples/lambda_func2/.venv/bin/lambda", line 117, in deploy
    aws_lambda.deploy(
  File "/home/adminUser/Downloads/examples/design/Examples/lambda_func2/.venv/lib/python3.10/site-packages/aws_lambda/aws_lambda.py", line 136, in deploy
    existing_config = get_function_config(cfg)
  File "/home/adminUser/Downloads/examples/design/Examples/lambda_func2/.venv/lib/python3.10/site-packages/aws_lambda/aws_lambda.py", line 825, in get_function_config
    return client.get_function(FunctionName=function_name)
  File "/home/adminUser/Downloads/examples/design/Examples/lambda_func2/.venv/lib/python3.10/site-packages/botocore/client.py", line 553, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/home/adminUser/Downloads/examples/design/Examples/lambda_func2/.venv/lib/python3.10/site-packages/botocore/client.py", line 1009, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.exceptions.ClientError: An error occurred (UnrecognizedClientException) when calling the GetFunction operation: The security token included in the request is invalid.

Hi Wondering whether you've solved this as I'm having the same issue.

Didn't receive a reply from the author. Closing it.