diarization capabilities

Describe the feature
Hi whiteeagle, I have another request. Is it possible to implement pyannote for speaker diarization instead of Sherpa-ONNX? I saw mention of this in other contexts, but comparing both, I noticed that diarization using pyannote was much more accurate in separating the speakers and transcribed a bit more effectively. I had tried around with playing with the speaker detection threshold and transcription model (downloaded large model [not turbo]) and compared to a medium model on another software, known as 'aTrain' on github (which uses pyannote and fast whisper, medium model), the difference was very noticeable, and I tested this with almost a dozen files. Please let us know if this is possible. I love the interface of Vibe and the fact that it can handle batch transcriptions, but compared to the transcription/diarization, aTrain seemed to outperform- maybe something to look into. Will continue to use Vibe since it supports multiple files, which is what I directly need (aTrain only does one at a time) and the interface is much nicer, but may be worth exploring their configurations. Keep up the good work!

Hey @mussaj,
Thanks for bringing this up. I’m aware of the diarization inaccuracies in Vibe. Currently, we don't use sherpa-onnx; instead, we use a Rust library I created called pyannote-rs. That said, it's possible that sherpa-onnx could offer better accuracy.
I’ve also worked on a C++ project that uses sherpa-onnx for diarization, and it might be worth comparing the results with the same files to see if it's more accurate. If so, we can look into integrating sherpa-onnx into Vibe.
I have ready-to-use binaries available in that project. You can find it here: https://github.com/thewh1teagle/loud.cpp
Let me know if you want to give it a try!

Hi again! the development of sherpa-rs is exciting! However, as the author mentioned, I think pyannote-rs is still much better (it's already using vibe as you mentioned). I still use pyannote for speech detection and speaker ID with whisper-rs(thanks to Vulkan support). I think the main problem with Speaker ID is more related to parallel speech or some "unbalanced" audio data (see discussion). Because if we use bigger speaker ID models, the result is pretty much the same because the segmentation for parallel speech is not good for dealing with the such scenarios. This may be a wrong conclusion, but this is what I came to after my tests. 🙂 I tried some of Gstreamer's complex audio normalization pipelines (experimentally), but in some cases, it made the results better, while in other cases, it made them worse. So I concluded that we need a better speech detection model to deal with parallel speech/noises so feeding the Speaker ID model with better segmented audio will give better results.
In short, if you're working on a specific audio file, I'm afraid you need to do some work on audio normalization. I still haven't come across a general solution. I mean, I haven't seen a general solution on how the best audio normalization should be. But vibe's method works very well for many situations: https://github.com/thewh1teagle/vibe/blob/cd05a7eba808e462e131564a51de4dcbd50f73aa/core/src/audio.rs#L54 If you know or have read somewhere, please share. Because in the current situation, audio normalization has a significant effect on detection ACC. 😊

You may be right. I am a novice at all of this, but I noticed when the speakers spoke around the same time or briefly after one another, there were issues detected different speakers. May be related to segmentation/normalization as you said. Hoping for a fix down the pipeline!


Image not clear and I don’t know buttons confusing

Feedback from GISCorps:

The not clear and the I don't know in meanings almost the same, I think it would be enough to use only one of those.
Gergely G. Puskas<EMAIL_ADDRESS>

"I don't know" and "photo not clear" seem interchangeable in the tutorials  (crop identification tutorial says the answer is "photo not clear" for the question about method of cultivation where you can't tell from the photo. However, in the tour at the beginning, you are asked to select this if the photo is blurry, poor quality, or not clear.) This could be very confusing and result in poor data.
Emily Hultman<EMAIL_ADDRESS>

The user needs a clear definition of the "Image not clear" button. I was not certain whether that literally meant that the 'image' was not clear as in its resolution, or 'not clear' as in the feature that we were being asked to identify could not be determined from the image.
Melanie Harris<EMAIL_ADDRESS>

@supranove change tour text to say "Select this option is the image is not clear enough for you to answer the question"

or "Select this option if you cannot answer this question given what you can see in the image"

@supranove is this done? if so we can close it

@elliec Yep, this should be fine. I left it open in case we got feedback about the distinction still not being clear enough however that hasn't happened so I think the tour text's clear enough now.


Do not trigger workflow if a related issue is found and it's closed

This mainly affects testing, but we should refrain from starting a workflow against an existing issue that's been closed.

This hasn't really come up as an issue in long enough that I'm ok closing. Can re-open if this becomes an impediment.


StatusCode.UNIMPLEMENTED

Hi,
When trying the immudb. I can not get it to work. I run the db on docker with: sudo docker run -it -p 3322:3322 -p 9497:9497 --name immudb codenotary/immudb:latest and this is my client code:
import immudb
from immudb.client import ImmudbClient

ic = ImmudbClient("<IP_ADDRESS>:3322")
ic.login("immudb","immudb")

if "testdb" not in [o.databasename for o in ic.databaseList().dblist.databases]:
    ic.databaseCreate(b"testdb")
print(ic.databaseList().dblist.databases)

ic.databaseUse(b"testdb")
print('using testdb')
key="Hello".encode('utf8')
value="Immutable world!".encode('utf8')

# set a key/value pair
ic.set(key,value)

I do get a read out of the databases. But then I get the following error:
    raise _InactiveRpcError(state)
grpc._channel._InactiveRpcError: <_InactiveRpcError of RPC that terminated with:
        status = StatusCode.UNIMPLEMENTED
        details = "unknown method SetSV for service immudb.schema.ImmuService"
        debug_error_string = "{"created":"@1606746880.907274017","description":"Error received from peer ipv4:<IP_ADDRESS>:3322","file":"src/core/lib/surface/call.cc","file_line":1061,"grpc_message":"unknown method SetSV for service immudb.schema.ImmuService","grpc_status":12}"
>

I'm even running the code in docker and still get the same error. here the Dockerfile:
FROM python:3.7.2-slim

RUN apt-get update && apt-get remove -y --purge man-db && apt-get dist-upgrade -y && apt-get autoremove -y
RUN apt-get update && apt-get install gcc wget git python3-dev g++ -y
RUN apt install python3-pip git -y
RUN pip3 install --upgrade pip

RUN python3 -m pip install --upgrade setuptools
RUN pip3 install --no-cache-dir  --force-reinstall -Iv grpcio #==1.32

RUN pip3 install immudb-py

WORKDIR /app
COPY run_immudb.py /app/run_immudb.py

CMD python3 run_immudb.py

Any help would be much appreciated

Hi takotab.
Latest versions of immudb have deprecated and removed some functions that are still used in main branch of immudb-py.
That is going to be fixed. For the moment, I suggest you to try branch "feat/nosv" of immudb-py that is coping with that and will be merged soon.

I've merged feat/nosv branch into master, you can use that now


Autogenerate source directory

I believe it'd be nice to have an option / autogenerate source tree root when you're defining new target. Am I missing an existing option?

Maybe use a template?
I have one globally available on my machine for Swift Packages:
targetTemplates:
  SwiftPackage:
    type: framework
    platform:
      - iOS
      - macOS
      - tvOS
      - watchOS
    platformSuffix: " (${platform})"
    sources:
      - path: "../Sources/${target_name}"
        excludes:
          - ReadMe.md

The takeaway point is the use of ${target_name} in templated sources.


Fixed demo completions

Rel: #834 Added MenuNext as a menu event upon pressing tab to the demo example. Fixes the currently broken completions on the example.

Thanks!


Switching Block editor to Python failed in "Traffic Panic"

Describe the bug
Switching Block editor to Python failed in "Traffic Panic"
Steps to reproduce the behavior
1.Navigate to https://arcade.makecode.com/beta#
2.Game Jam----->Traffic Panic------>Open Example
3.Switch Block editor to Python editor
Actual behavior
Switching Block editor to Python failed in "Traffic Panic"

Additional context
1.OS: Windows(rs6)
2.arcade version: 1.3.30
3.Microsoft MakeCode version: 6.8.15

@Sheila111 does this work in live? Or is this a regression in /beta

@Sheila111 does this work in live? Or is this a regression in /beta

Is this a duplicate of: https://github.com/microsoft/pxt-arcade/issues/2471 ?

Is this a duplicate of: https://github.com/microsoft/pxt-arcade/issues/2471 ?

Still happened in live.

Still happened in live.


nixos/swap: prefer 'umask' over 'chmod'

Description of changes
part of #121293
tested:

nixos/tests/swap-file-btrfs.nix
nixos/tests/swap-partition.nix
nixos/tests/swap-random-encryption.nix

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks!


chore: update stable to 2.2.0

what about the helm chart @eliobischof do we need to change something there as well?

what about the helm chart @eliobischof do we need to change something there as well?

I created a PR: https://github.com/zitadel/zitadel-charts/pull/35


gerrit-331601-8

Current coverage is 62.83% (diff: 88.88%)

No coverage report found for master at 10b17d4.

Powered by Codecov. Last update 10b17d4...86693d4


Implement Reset by Instruction

http://www.microchip.com/forums/m223037.aspx
Goto 0 seems to be enough

http://www.microchip.com/forums/m396677.aspx => SoftReset() works :)


⚠ Mobitopup has degraded performance

In f0747c1, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 4433 ms

Resolved: Mobitopup performance has improved in af566d4.


uninitialized constant Spree::Theme::CURRENT_THEME_PATH (NameError)

run: rails db:migrate
brought the error
app/controllers/spree/store_controller_decorator.rb:1:in `<top (required)>': uninitialized constant Spree::Theme::CURRENT_THEME_PATH (NameError)

Hi @99cm
After adding the gem and bundle install,  try
bundle exec rails g vinsol_spree_themes:install.
I hope this works.

I will try. Thank you very much. If the steps work, I'll close this issue.

after 'bundle install', tried
'bundle exec rails g vinsol_spree_themes:install'
terminal displayed a similar error:
/spree_themes-0b8801fe8f35/app/controllers/spree/store_controller_decorator.rb:1:in `<top (required)>': uninitialized constant Spree::Theme::CURRENT_THEME_PATH (NameError)

Hi @99cm
Can you please specify the spree version and ruby version you are using??

We are using Ruby 2.5.0 and Spree 3.4.4.
Thank you.

Hi @99cm
Can you please send the complete gemfile of your application so that we can replicate the issue and help you further.
Also please tell the steps you have done after bundle the vinsol_spree_themes gem.

Hi @99cm
I used your gemfile and it's working fine for me so i couldn't replicate the issue.
Please create a new spree app and then try to install the extension. I hope that would work.
Thank you


elm-make cannot find modules in other folders

This may well be something that I have done rather than a fault with elm-vim, but in the project I am currently working on elm-make cannot find modules which are defined in files stored outside the directory that the file in the buffer is in.
The file structure I have is as follows:
--root (contains elm-package.json)
----src
------elm
--------models
--------views
If, for instance, I edit a file in views that imports a module defined in its parent directory (elm) then the module cannot be found. My elm-package.json lists src/elm in source directories. Symlinking that file into src/elm helps in that dependencies listed there are found by elm-make (they weren't until I set up the symlink), but it still seems strange that the modules defined locally are not found. I am running vim from the root directory.
Is this normal behaviour?

OK, after some digging about I have managed to work out the source of the error, though I have no idea why it is there.
In the above directory listing, the elm-package.json file is in root, but for reasons I do not understand the make command was being run in root/source/elm. I do not know why, because echoing getcwd() at line 115 of the file autoload/elm.vim printed the root directory. However, if the call to getcwd() is moved after the call to shellescape(a:input) in the same file then the problem is resolved. Here's the relevant snippet for those more knowledgable than me to use to work out how to fix this elegantly (though my hack seems to work):
fun! elm#Build(input, output, show_warnings)
  let s:errors = []
  let fixes = []
  let rawlines = []

  let bin = "elm-make"
  let format = "--report=json"
  let input = shellescape(a:input)
  let bugfix = getcwd() "!!!! Extra line
  let output = "--output=" . shellescape(a:output)
  let command = bin . " " . format  . " " . input . " " . output
  let reports = s:ExecuteInRoot(command)

  for report in split(reports, '\n')
    if report[0] == '['
      for error in elm#util#DecodeJSON(report)
        if a:show_warnings == 0 && error.type == "warning"
        else
          call add(s:errors, error)
          call add(fixes, {"filename": error.file,
                \"valid": 1,
                \"bufnr": bufnr('%'),
                \"type": (error.type == "error") ? 'E' : 'W',
                \"lnum": error.region.start.line,
                \"col": error.region.start.column,
                \"text": error.overview})
        endif
      endfor
    else
      call add(rawlines, report)
    endif
  endfor


use localhost server for interface request. will appear always error

Q
A

Bug?
don't know

New Feature?
yes

Version
"guzzlehttp/guzzle": "~6.0",

Actual Behavior
What is the actual behavior?
use localhost server for interface request. will appear always error, nginx error 504 Gateway Time-out  :
ex: use <IP_ADDRESS> deed request url server address. looping HBL thread.
2018/03/16 13:26:00 [error] 2848#13436: *15 upstream timed out (10060: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond) while reading response header from upstream, client: <IP_ADDRESS>, server: tv.cc, request: "GET /aaa/cccc HTTP/1.1", upstream: "fastcgi://<IP_ADDRESS>:9000", host: "tv.cc"
But, use products no Exception
Expected Behavior
What is the behavior you expect?
use localhost address request o exception
Steps to Reproduce
What are the steps to reproduce this bug? Please add code examples,
screenshots or links to GitHub repositories that reproduce the problem.
code:
<?php
$client = new \GuzzleHttp\Client();
$response = $client->request('GET', '<IP_ADDRESS>', [
    "timeout" => 3000
]);

echo $response->getStatusCode(), "\n";
echo $response->getBody();

php5.6
nginx:1.11.5
Possible Solutions
If you have already ideas how to solve the issue, add them here.
(remove this section if not needed)

Thank you for this issue.
However, this does not look like there is anything wrong with Guzzle. If you get a proper HTTP response (like 504) I would start to investigate your server and gateway.
Im closing this issue. Please refer to other resources like StackOverflow if you need help.

wait , wait.
but, The same code. merely request url address unlikeliness.  And, localhost url  is normal.
so, I dont'know , are Guzzle or me a problem of.
thx.

Again, it is not Guzzle. It is your server settings on localhost that is the issue.

The returned response indicates that your server responded, this the request itself is successful. The response code indicates that something went wrong on server side (most probably your nginx server couldn't connect to php-fpm).
The fact that it works locally is irrelevant in this case as the problem is on the server side.

@sagikazarmark But The problem is my local server . If direct use local curl visit ,this can be.
curl http://localhost/q101/code?uid=990006 {"code":200,"message":"","data":{"code":"lL170620"}}
If I put this url in there.  e.g.
$client->request('GET', '<IP_ADDRESS>', [     "uid" => 3000 ]);
Just keep asking , until nginx 504 Time out

@sagikazarmark oh, yes. problem solved .  causes of problem is nginx config of fastcgi_pass port the same....   cause a blockage.  thanks very much

@ishwy can you share the nginx config?

@ivalsdtech it's gone, the main thing is that  fastcgi_pass


Does it support RDBMS?

I'm interested to use this software or perhaps contributing to it, does it capability to support RDMS?
How CRUD interface created?

Nevermind, we found out that it supports sequelize.


UDP receive buffer size on OpenBSD?

Hey there,
I'm running into an issue on OpenBSD where I get the following output:
failed to increase receive buffer size (wanted: 2048 kiB, got 0 kiB). See https://github.com/lucas-clemente/quic-go/wiki/UDP-Receive-Buffer-Size for details.

Unfortunately the linked documentation doesn't describe how to do so. Also, resources that I found (e.g. https://comp.unix.bsd.openbsd.misc.narkive.com/X8fbptCR/how-to-raise-the-maxsockbuf-in-openbsd) doesn't seem to provide a feasible solution to this.
Is there a way quic-go can circumvent this issue?
Thank you!

I don’t have any experience with OpenBSD. A PR would be appreciated.

I just tested this on OpenBSD 7.2 and have taken the following notes:
The correct sysctl to change is:
sysctl net.inet.udp.recvspace=...

While the maximum value supported by the kernel is<PHONE_NUMBER>, resolving DNS hostnames via dig fails when the value is set beyond 2048000.
The maximum value doesn't seem to be affected by allocating more memory to the system (8GB vs 16GB available to the host at boot time).


OS X Install - Double Click on makesymlinks throws "unidentified developer" error

Hi, During OSX install, when I double clicked on the makesymlinks icon I got the following error dialog: "can’t be opened because it is from an unidentified developer."
Had to hit control+click and add an exception to install the script.
Everything else was very smooth, but It would be great if this error didn't happen and I didn't have to do exception. The warning dialog is not reassuring.
Thanks!

Hello that warning because of permissions settings of osx and that package installer is not on appstore. if you change that settings to trust 'from anywhere' that warning will not come again.


Don't lookup the hostname during initialisation

This commit defers DNS lookups until submission of UDP packets. It removes
:inet.gethostbyname from the configure function and passes the unresolved
hostname to :get_udp.send as a char list. This allows gelf_logger to react
to DNS changes at runtime. Prior to this commit an IP address change of the
destination host would cause us to lose all subsequent log lines.
Fixes #7 by allowing gelf_logger to be started despite the target hostname
being unresolvable.
Let me know what you think 😀. Thanks a lot for the awesome lib!

I'm honestly a little ashamed that I hadn't noticed that before. Thank you for the pull request!


Impersonate Action

Hi,
Action for impersonate user in UserResource it's not visible.
I'm looking into source code and i found (row 22 FilamentAuthentication class)
protected bool $impersonate = false;
I'm not found configuration fon change this behaviour

Per image - make sure you have latest config file


it can't go " if(btn_state == TBN_DOWN) { " switch.

I have wacom driver and tablet,but it can't go " if(btn_state == TBN_DOWN) { " switch. Can you help me?

Sorry for your inconvenience. But This project is pretty much obsolete and I don't think I can really help you. Last time I looked at the source is about 3 years ago.
You may want to check out official Qt source code to  get a better understanding of how Qt handle this.

<EMAIL_ADDRESS>谢谢

<EMAIL_ADDRESS>


Upgrade to OpenSSL 1.0.1q

Please, release ruby-build after merging this if possible. Thank you!

Thanks always!


[Task] Update dependencies and clean version catalog

Hi team,
In this PR, I have cleanup and versions catalog file, introducing [bundles] block packaging together group of dependencies of the same flavour offering me dependency readability.
Please review
Thanks

Thanks for the contribution, this looks like multiple unrelated changes grouped together which means we wouldn't be able to merge it.

There are what look to be test changes in the HomeScreen file
I don't think we need the gradle file reformatting
The Portuguese strings file

We would love adding the Portuguese strings to the app, could you please create a new PR just with those changes? The rest I don't think we could accept unfortunately as I don't see the benefit.

Thanks for the contribution, this looks like multiple unrelated changes grouped together which means we wouldn't be able to merge it.

There are what look to be test changes in the HomeScreen file
I don't think we need the gradle file reformatting
The Portuguese strings file

We would love adding the Portuguese strings to the app, could you please create a new PR just with those changes? The rest I don't think we could accept unfortunately as I don't see the benefit.

Thanks @bentrengrove for the feedback.
About the multiple unrelated changes, my bad on that, I actually branched from my old PR which is #655 for the Portuguese translation I did a while back. Kindly take a look on it.
Now, reformatting the version catalog file to introducing bundles, make the gradle dependencies more readable and combining related group libraries together.
Thanks


Feat/improve css compatibility

Great news, a friend of mine has lent me a Windows with a licensed Outlook. So I can fix the Outlook compatibility issue.
So this PR contains code for:

Fix outlook missing button #59
Fix outlook quotes display
Add CSS inlining by default with https://github.com/vanng822/go-premailer/premailer (now Outlook displays the text as centered)

=> Replaces PR #48
Hey @kauffj @loeffel-io @vjeantet @dmuriel @elliots  🙌
Can you test this PR on your side ?
It worked for me on a Windows 10 with latest version of Outlook. I think it's still perfectible, but it does the work.
Edit: the size of the button now adapts to content in default theme.

outlook 2016 windows10 has problem

flat and default has the same problem ,can not show button

I find a workaround. success show button
DisableCSSInlining: true,

	h := hermes.Hermes{
		// Optional Theme
		// Theme: new(Default)
		Theme:              new(hermes.Flat),
		DisableCSSInlining: true,


Update regex for name to reject numerals

Closes #165

Codecov Report

Merging #231 (49843e8) into master (a4fed9f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #231   +/-   ##
=========================================
  Coverage     63.89%   63.89%
  Complexity      826      826
=========================================
  Files           138      138
  Lines          3124     3124
  Branches        407      407
=========================================
  Hits           1996     1996
  Misses          991      991
  Partials        137      137

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/model/student/Name.java
80.00% <ø> (ø)

Continue to review full report at Codecov.

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


⚠ Bofrost.ch has degraded performance

In 1014d1f, Bofrost.ch (https://www.bofrost.ch) experienced degraded performance:

HTTP code: 200
Response time: 2888 ms

Resolved: Bofrost.ch performance has improved in 8127f5d.


Check webview is focued when setting active webview

This PR fixes #131289

Would it better to compare activeInput.webview against this._webviewService.activeWebview to match with this
https://github.com/microsoft/vscode/blob/69eba4e0582c82a1c6efa9a08fa3fffbfffae4c3/src/vs/workbench/contrib/webviewPanel/browser/webviewWorkbenchService.ts#L208
instead of using isFocused?


[PL-48711] Document that 'Secret Identifier' will now allow - (hyphen…

PL-48711
https://harness.atlassian.net/wiki/spaces/CDNG/pages/736200458/Entity+Identifier
Referenced from: https://harness.atlassian.net/browse/PL-48009
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screenshot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://660a706bb604842f42712751--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://660bbe79833e1297a1d809e5--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://660c70eb4d6f1f2c189d971b--harness-developer.netlify.app

@brian-f-harness Review request please


EPMCIR-1326 Provide staging only endpoint allowing spaces to seed users

https://jira.epam.com/jira/browse/EPMCIR-1326
Why:
Spaces like testIO app needs to be able to seed users when the staging/dev env is setup.
Cirro needs to provide an endpoint which is only accessible in staging/dev environment POST /users which allows spaces to create user + app_user with email, password, firstname, lastname etc.
What:

Provide wrapper functionality in cirro-ruby-client gem
Cover with some tests

@vladveterok would you be so kind to update the readme with the description of your new feature ;) also we need to let everybody know that params are completely optional.

@vladveterok would you be so kind to update the readme with the description of your new feature ;) also we need to let everybody know that params are completely optional.

@mesmerze  done!


ecapture 在大于 5.2 的内核版本抱怨内核版本不大于 5.2

Describe the bug
在我个人的使用过程，甚至是 #595 #598 中的日志的内核版本已经来到 6.1.*，但仍然提示 WRN Kernel version is less than 5.2, Process filtering parameters do not take effect such as pid/uid. kernel=6.1.0
To Reproduce
Steps to reproduce the behavior:

# ./ecapture tls -m text -u $uid
waiting the log

Expected behavior
应该不会在大于 5.2.0 的内核提示内核版本低的 bug
Screenshots
None, but output: Kernel version is less than 5.2, Process filtering parameters do not take effect such as pid/uid. kernel=5.15.123
Linux Server/Android (please complete the following information):

Env: N/AKernel version is less than 5.2, Process filtering parameters do not take effect such as pid/uid. kernel=5.15.123`
OS: HYperOS?
Arch: arm_aarch64
Kernel Version: 5.15.123-android13-8-00008-g3ca6a2912c7e-ab11087001
Version: androidgki_arm64:v0.8.5:6.5.0-1025-azure

Additional context
看起来这是一个普遍问题，并不特定于设备

It seems there is a kernel version recognition error.
I don't have a device called HYperOS and I'm not sure how to reproduce this issue. Does this problem also occur on Linux kernels that are greater than version 6.0?

In #595，linux kernel version > 6.0


V2 - Corporate Console - requestcorporatesignature endpoint fails when authority name has digits

Observed that requestcorporatesignature endpoint fails when authority name has digits. User service accepts digits, however authority name does not accept digits
https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/request-corporate-signature

Allowing digits is fine.

Executed the automation script and observed that the issue is fixed


Error When Updating

When trying to update novel info I got this toast message and the info is not renewed

I have tried disabling VPN and AdBlock but the problem persists.

which version do you use? try to disable/enable Use Key Store in Settings --> Update and Notification.
Possible duplicate with #270

I use version 1.1.46 from Play Store.
Thank you. Your solution works.
You're correct. It's a duplicate of #270. Sorry:disappointed:. I didn't find it before.


Add Vision safe search detection.

This adds Safe Search detection to the Vision API.
Safe Search isn't an EntityAnnotation based call so a new SafeSearchAnnotation was created.

@tseaver this one should be good to take a look at.
Then the last API addition is #2320.


Ensure that wtpython can work on 3.7 through 3.10

Rich is ^3.6 and textual is ^3.7.

This issue should be closed.

Closed by #120.
@pyxiis Next time I'll link the PR which should auto-close it.


Self contained apps do not have a runtime graph

Describe the bug
The .NET Core SDK trims the RID graph for self-contained apps. This means self contained apps cannot correctly interpret deps.json and RIDs.
To Reproduce

Create a host app that is self contained.
Create a plugin which uses a dependency that has rid-specific assets, such as System.Drawing.Common

Expected behavior
Plugins do not need to know about the RID used by the host, and vice versa.
Additional context
The .NET Core SDK makes the assumption that self-contained applications know about their dependencies at build-time, and it does some trimming to remove information it thinks won't be needed at runtime. Some of the information trimmed is the runtime identifier fallback graph. This means when this section of code runs, it cannot find a runtime graph so fallbackGraph = { "any" }.
https://github.com/natemcmaster/DotNetCorePlugins/blob/abf20cdaacebb1d691df3ed5aa18d22892f5a48a/src/Plugins/Loader/DependencyContextExtensions.cs#L108-L116
For example: when it is reading the deps.json file, the runtime does not know how to handle the "win" runtime identifer.
          "runtimes/win/lib/netcoreapp2.0/System.Drawing.Common.dll": {
            "rid": "win",
            "assetType": "runtime",
            "assemblyVersion": "<IP_ADDRESS>",
            "fileVersion": "4.6.26919.2"
          }

Options:

Embed a copy of the RID graph in McMaster.NETCore.Plugins.dll (acquired from Microsoft.NETCore.Platforms)

Downside: everytime the RID graph changes, we'd need a new version of this library. For example, every time macOS releases a new verison, or a new type of Linux gains support for .NET Core

Dynamically find the RID graph at build time and embed in the app.

Downside: Since the SDK doesn't do this, we'll have to include some build magic in the targets for this library to preserve the RID graph into a resource embedded in the app

Add API and let users specify the RID graph programmatically.

Downside: this is just cruel.

See also these related issues in .NET Core https://github.com/dotnet/sdk/issues/1046 and https://github.com/dotnet/core-setup/issues/1846


Update stripe_lang.php. Was throwing an error in the log.

When using it today the 'text_exp' was throwing a lang error. Updated to 'text_exp_month' and 'text_exp_year' and that fixed it.

kindly merge your PR to the develop branch instead.


useApplyAttributeChange() performance issue

Version :
"@shopify/ui-extensions": "2024.4.x",
"@shopify/ui-extensions-react": "2024.4.x"
Describe the bug
Hello,
I'm having a problem with a checkout V2 extension for which I'm using the useApplyAttributeChange() hook.
This hook generates multiple renderings of my component.
I go from a few renderings to dozens or even hundreds.
Would it be possible to launch the update of multiple control attributes (instead of having to do them one by one)?
Steps to reproduce the behavior:
const applyAttributeChange = useApplyAttributeChange();
const saveMyAttributes = async () => {
  const result =
      await applyAttributeChange({
        type: 'updateAttribute',
        key: myAttribute1,
        value: value1,
      })
      &&
      await applyAttributeChange({
        type: 'updateAttribute',
        key: myAttribute2,
        value: value2,
      })
      && await applyAttributeChange({
        type: 'updateAttribute',
        key: myAttribute3,
        value: value3
      })
      && await applyAttributeChange({
        type: 'updateAttribute',
        key: myAttribute4,
        value: value4
      })
      && await applyAttributeChange({
        type: 'updateAttribute',
        key: myAttribute5,
        value: value5
      })

    // Update ok ?
    if (result.type === 'success') {
     // Do stuff...
    }
}
useEffect(() => {

    const toUpdate = MY_CONDITION_TO_UPDATE_CODE
    if (toUpdate) {
      saveMyAttributes()
    }
  }, []);

This code isn't called on every reload, but via a useEffect to keep component updates to a minimum.
Expected behavior
Multiple updates on several control attributes at the same time

i had same scenario just ran all the updates at once without await and it seemed to be working fine for my case all attributes get applied at same time

Wahooo! What a difference! Indeed, it generates far, far fewer renderings!
Thanks a lot @hashemHP


List extractor accepts Seq parameters

(Vector(1, 2, 3): Seq[Int]) match {
  case Nil     => "empty"
  case x :: xs => "non empty"
}

This code compiles and fails at runtime with a match error: https://scastie.scala-lang.org/mRhIPpDDTsCP04gava4SFw
The :: extractor should not accept a Seq as parameter, this should be a compile error.

:: is a case class, not an extractor. But Seq is a non-sealed base class so there is no exhaustiveness checking. If anything, this should be a ticket on scala/bug but I doubt we can improve this without lots of false positives. The current behavior is the same as in 2.12.


flutter home path

i see your assuming that a flutter path exists in the env variables.
not everyone has it this way.
why not make it possible to pass in the dir as a variable when this is run ?
that way the developer can pick the folder where they want flutter to be installed.
For example here is my .bash_profile

#git clone -b dev https://github.com/flutter/flutter.git
export FLUTTER_ROOT=$HOME/workspace/flutter/standard/flutter
echoLog "FLUTTER_ROOT (exported)" $FLUTTER_ROOT
export PATH=$PATH:$FLUTTER_ROOT/bin

fvm does not find it:
x-MacBook-Pro:go-flutter apple$ fvm
No Flutter versions installed

Ok i worked it out by using where fvm wants to put flutter.
went with that :)


Why not use Maven or Gradle to mange the dependencies as well as the project?

Why not use Maven or Gradle to mange the dependencies as well as the project?

It is on the TO-DO list


[fmt] build failure on MinGW

Host Environment

OS: Windows 10 x64 1909
Compiler: (MinGW.org GCC-8.2.0-5) 8.2.0

To Reproduce
Firstly install fmt: ./vcpkg install fmt:x64-windows
Add configuration to CMakeLists.txt:
find_package(fmt CONFIG REQUIRED)
target_link_libraries(main PRIVATE fmt::fmt)

Failure logs
CMake Error at C:/vcpkg/scripts/buildsystems/vcpkg.cmake:288 (_find_package):
  Could not find a configuration file for package "fmt" that is compatible
  with requested version "6.0.0".

  The following configuration files were considered but not accepted:

    C:/vcpkg/installed/x64-windows/share/fmt/fmt-config.cmake, version: 6.0.0 (64bit)

Additional context
After I removed these line in %VCPKGDIR%\share\fmt\fmt-config-version.cmake it passed the build, but failed to run with these error message: .../ld.exe: CMakeFiles\PS.dir/objects.a(main.cpp.obj):main.cpp:(.text.startup+0x57): undefined reference to _imp___ZN3fmt2v66vprintENS0_17basic_string_viewIcEENS0_11format_argsE
# check that the installed version has the same 32/64bit-ness as the one which is currently searching:
if(NOT CMAKE_SIZEOF_VOID_P STREQUAL "8")
   math(EXPR installedBits "8 * 8")
   set(PACKAGE_VERSION "${PACKAGE_VERSION} (${installedBits}bit)")
   set(PACKAGE_VERSION_UNSUITABLE TRUE)
endif()

If you use gcc in your project, You should ./vcpkg install fmt:x64-mingw

Oh, my mistake 😂 Thank you!

ld.exe: CMakeFiles\PS.dir/objects.a(main.cc.obj): in function `ZN3fmt2v65printIA12_cJRxELi0EEEvRKT_DpOT0_':
%VCPKGDIR%/include/fmt/core.h:1409: undefined reference to 'fmt::v6::vprint(fmt::v6::basic_string_view<char>, fmt::v6::format_args)'

Just tried MinGW (x86 and x64 and both failed with this error. Any ideas?

@BasixKOR Any news/update on this long standing issue?

@BasixKOR I can't repro this currently, seems this has been fixed at some point.
Computing installation plan...
The following packages will be built and installed:
    fmt[core]:x64-mingw-dynamic -> 7.1.3
Detecting compiler hash for triplet x64-mingw-dynamic...
Could not locate cached archive: C:\Users\nghuu\AppData\Local\vcpkg\archives\93\93099fa56318cb133dd416a7f907bb6c95d34e5d.zip
Starting package 1/1: fmt:x64-mingw-dynamic
Building package fmt[core]:x64-mingw-dynamic...
-- Using community triplet x64-mingw-dynamic. This triplet configuration is not guaranteed to succeed.
-- [COMMUNITY] Loading triplet configuration from: C:\vcpkg\triplets\community\x64-mingw-dynamic.cmake
-- Downloading https://github.com/fmtlib/fmt/archive/7bdf0628b1276379886c7f6dda2cef2b3b374f0b.tar.gz...
-- Extracting source C:/vcpkg/downloads/fmtlib-fmt-7bdf0628b1276379886c7f6dda2cef2b3b374f0b.tar.gz
-- Applying patch fix-warning4189.patch
-- Using source at C:/vcpkg/buildtrees/fmt/src/2b3b374f0b-8ea71ee8ea.clean
-- Configuring x64-mingw-dynamic
-- Building x64-mingw-dynamic-dbg
-- Building x64-mingw-dynamic-rel
-- Installing: C:/vcpkg/packages/fmt_x64-mingw-dynamic/share/fmt/copyright
-- Using msys root at C:/vcpkg/downloads/tools/msys2/c809757c94447846
-- Fixing pkgconfig file: C:/vcpkg/packages/fmt_x64-mingw-dynamic/lib/pkgconfig/fmt.pc
-- Fixing pkgconfig file: C:/vcpkg/packages/fmt_x64-mingw-dynamic/debug/lib/pkgconfig/fmt.pc
-- Installing: C:/vcpkg/packages/fmt_x64-mingw-dynamic/share/fmt/usage
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: C:\Users\nghuu\AppData\Local\vcpkg\archives\93\93099fa56318cb133dd416a7f907bb6c95d34e5d.zip
Building package fmt[core]:x64-mingw-dynamic... done
Installing package fmt[core]:x64-mingw-dynamic...
Installing package fmt[core]:x64-mingw-dynamic... done
Elapsed time for package fmt:x64-mingw-dynamic: 4.386 min

Total elapsed time: 5.879 min

The package fmt provides CMake targets:

    find_package(fmt CONFIG REQUIRED)
    target_link_libraries(main PRIVATE fmt::fmt)

    # Or use the header-only version
    target_link_libraries(main PRIVATE fmt::fmt-header-only)

Inspection with nm also shows that fmt::v7::vprint(fmt::v7::basic_string_view<char>, fmt::v7::format_args) (_ZN3fmt2v76vprintENS0_17basic_string_viewIcEENS0_11format_argsE) is present.

@BasixKOR I can't repro this currently, seems this has been fixed at some point.
Computing installation plan...
The following packages will be built and installed:
    fmt[core]:x64-mingw-dynamic -> 7.1.3
Detecting compiler hash for triplet x64-mingw-dynamic...
Could not locate cached archive: C:\Users\nghuu\AppData\Local\vcpkg\archives\93\93099fa56318cb133dd416a7f907bb6c95d34e5d.zip
Starting package 1/1: fmt:x64-mingw-dynamic
Building package fmt[core]:x64-mingw-dynamic...
-- Using community triplet x64-mingw-dynamic. This triplet configuration is not guaranteed to succeed.
-- [COMMUNITY] Loading triplet configuration from: C:\vcpkg\triplets\community\x64-mingw-dynamic.cmake
-- Downloading https://github.com/fmtlib/fmt/archive/7bdf0628b1276379886c7f6dda2cef2b3b374f0b.tar.gz...
-- Extracting source C:/vcpkg/downloads/fmtlib-fmt-7bdf0628b1276379886c7f6dda2cef2b3b374f0b.tar.gz
-- Applying patch fix-warning4189.patch
-- Using source at C:/vcpkg/buildtrees/fmt/src/2b3b374f0b-8ea71ee8ea.clean
-- Configuring x64-mingw-dynamic
-- Building x64-mingw-dynamic-dbg
-- Building x64-mingw-dynamic-rel
-- Installing: C:/vcpkg/packages/fmt_x64-mingw-dynamic/share/fmt/copyright
-- Using msys root at C:/vcpkg/downloads/tools/msys2/c809757c94447846
-- Fixing pkgconfig file: C:/vcpkg/packages/fmt_x64-mingw-dynamic/lib/pkgconfig/fmt.pc
-- Fixing pkgconfig file: C:/vcpkg/packages/fmt_x64-mingw-dynamic/debug/lib/pkgconfig/fmt.pc
-- Installing: C:/vcpkg/packages/fmt_x64-mingw-dynamic/share/fmt/usage
-- Performing post-build validation
-- Performing post-build validation done
Stored binary cache: C:\Users\nghuu\AppData\Local\vcpkg\archives\93\93099fa56318cb133dd416a7f907bb6c95d34e5d.zip
Building package fmt[core]:x64-mingw-dynamic... done
Installing package fmt[core]:x64-mingw-dynamic...
Installing package fmt[core]:x64-mingw-dynamic... done
Elapsed time for package fmt:x64-mingw-dynamic: 4.386 min

Total elapsed time: 5.879 min

The package fmt provides CMake targets:

    find_package(fmt CONFIG REQUIRED)
    target_link_libraries(main PRIVATE fmt::fmt)

    # Or use the header-only version
    target_link_libraries(main PRIVATE fmt::fmt-header-only)

Inspection with nm also shows that fmt::v7::vprint(fmt::v7::basic_string_view<char>, fmt::v7::format_args) (_ZN3fmt2v76vprintENS0_17basic_string_viewIcEENS0_11format_argsE) is present.

I just checked on my machine and although it still requires the manual patch it works as you said.

I just checked on my machine and although it still requires the manual patch it works as you said.

You haven't updated vcpkg yet, run git pull and .\bootstrap-vcpkg.bat and try again.

You haven't updated vcpkg yet, run git pull and .\bootstrap-vcpkg.bat and try again.

Update: I was using MinGW 32-bit version to compile fmt x86_64, so after noticing that I switched to MinGW-w64 (in MSYS2) and now it works very well.
Sorry for the hassle and thank you all for generous help!

Update: I was using MinGW 32-bit version to compile fmt x86_64, so after noticing that I switched to MinGW-w64 (in MSYS2) and now it works very well.
Sorry for the hassle and thank you all for generous help!


Refresh user data if mutation fails

I want to merge this change because it fixes bug when user tries to do anything with the non-existent address, refreshing the cache and removing it.
Pull Request Checklist

[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] The changes are tested.
[x] The code is documented (docstrings, project documentation).
[x] Changes are mentioned in the changelog.

Test Environment Config

API_URI=https://master.staging.saleor.rocks/graphql/

Task linked: CU-6ym7u7 Invalidate cached data if mutation throws error


Type transformation modifiers for v-model

From @Morgul's comment in #1713:

<!-- Cast to number -->
<input type="number" v-model.number="foo">

<!-- Cast to boolean -->
<input type="text" v-model.boolean="bar">

It would also seem useful to be able to register custom transformations. eg, datetime transformations for Moment.js objects.

@rpkilby Thanks for creating this issue, I was planning on tackling it to day, but work got in the way. :)

Curious - couldn't this just be done with custom two-way filters? Maybe I'm missing something.

@davidkhess Indeed, it seems possible: https://jsfiddle.net/posva/8whzuL16/1/

@davidkhess Yes, it's possible. This issue, however, is about adding built-in support.
The problem I've had with this approach is that it's unexpected, coming from other frameworks. All of them 'have the magic built in' to handle number inputs. Writing your own filter to handle this, while possible, is very sub-optimal. The compromise between what @yyx990803 envisions and what I'd like seems to be adding transforms for v-model.

@Morgul completely understand – I tend to favor less magic.

@rpkilby @Morgul Do you have a real use-case about any modifier you think should exist?
There is already a number parameter for text inputs: http://vuejs.org/guide/forms.html#number
Isn't it enough?

@posva Not really, no. As discussed in #1713, it's surprising when you're getting a number out of a number input. While vue has a way of handling that (the number directive), we discussed the fact that it could be better implemented as a transform, because then vue's syntax for handling these sorts of issues is consistent. This is the exact type of thing transforms were made for; number should be one.

@posva - It seems like a date transform for the date/datetime input types would be useful. However, I'm more interested in being able to register transforms. eg,

momentjs transform instead of a potentially built in date transform
JSON transform

@rpkilby I think two way filters work out pretty well for this. The guide actually talk about this http://vuejs.org/guide/custom-filter.html#Two-way-Filters
Maybe a more explicit example showing v-model="when | date" in the docs help out. I don't think a modifier is needed because two way filters already address this issue

@posva - yes, and we currently do this. I guess my problem is that I have vague (and I want to emphasize, very minor) dissatisfactions with the API for dealing with various data types. I have some thoughts that haven't really formed into anything cohesive.

Boolean values and or array selection are handled magically for inputs of type="checkbox" (and to be fair, this is kind of necessary).
Number values are not handled magically for inputs of type="number", and require a number paramater.
Any other data type requires a two-way filter.

The result of all of these leaves us with:
<input type="checkbox" v-model="foo" />
<input type="number" v-model="bar" number />
<input type="date" v-model="baz | date" />

Given that checkbox has magical behavior, it's understandable that someone would expect that number would have magical behavior as well.
Forgetting to add the number parameter is an easy source of bugs.
Why does number get special treatment anyway? Should it be replaced by a filter?
Filters seem like they should be used for value transformations, while this request is more about type transformation.
I also don't like the idea of polluting the filter namespace with type transformation filters. Out of a form context, they don't make much sense. At the very least, it's great that Vue has global and component-local registration, so this isn't really an issue.

Example:
<input type="text" v-model="title" />
<input type="text" v-model="slug | lower | kebab" />
<input type="date" v-model.date="publishOn" />
<input type="number" v-model.number="priority" />

@rpkilby very nice analysis and writeup. Agreed, things are inconsistent.

@rpkilby thanks for the thoughtful input - we'll definitely improve this in 1.1 :)

@yyx990803 any new thoughts on this given the deprecation of filters outside of interpolated text in Vue 2.x?

Ditto, removing filters seems to beg for some hook on v-model to accomplish type/value transformations.

The problem with other type transformations other than .number is that the conversion from the value back to string is not straightforward. e.g. for a date transformer, if you programmatically set the bound value to a Date object, there's no way for Vue to know how to display it as text in the input box. Then again we are back to similar concepts as two-way filters.
Type transformations also by definition makes the view out of sync with the underlying state.
My take on this use case is that instead of the implicit magic conversions, let v-model just sync what the user actually inputs with an underlying value (the source of truth), and then build derived values separately based on that value.
For example, you can perform these transformations only when you need to send it to the server for persistence. Or, use computed properties based on the bound value if you need to display it elsewhere in a different format.

My take on this use case is that instead of the implicit magic conversions, let v-model just sync what the user actually inputs with an underlying value (the source of truth), and then build derived values separately based on that value.

Preface: upon rereading, it's unclear if you're talking about removing the number directive or not, but my argument can be applied to any sort of transformation between a form's data and what's stored in a model, not just numeric inputs.
For simple use cases, this may be sufficient (and in some cases preferred). However, I have a use case with about 60 <input type="number"> fields. A rare case, admittedly, but it's what I've got. Since I need to perform math operations on the user input, I would end up with significantly more than 60 computed properties for those operations, and then the code to save to the server would be an annoying (but not complex) merger of the computed properties and the underlying data object.
I agree in principal with "let v-model just sync what the user actually inputs", however my contention is that when a user is presented with a <input type="number"> field the user's intention is to input a number, so Vue should store it as a number. The user doesn't know that HTML5's spec for number inputs is weird; They were asked for a number, so they input a number.
I'm fine if I have to do some work somewhere (specifying a transform, a two way filter, or something like that), but I don't want to have to define 60+ computed properties and then write code to try and merge those computed properties with another object just so I can turn it into JSON to send to my server. It feels like an inelegant solution, compared to other frameworks.
My use case is numeric inputs, but being a little more pedantic, you could apply the same logic to the values for check boxes (true/false vs string values), date fields, radio buttons, etc. Any time you have a large number of properties that need special handling, you'll run into this problem. I just don't think that 'use a computed property' scales well.
I'd be a lot more sanguine if it felt like there was a clear path forward that had the same elegance I'm used to in the rest of the framework.

@Morgul .number is still available because the conversion from a number to a string is very straightforward. That is not the case for other arbitrary types, e.g. Date.

For argument:

Our API uses the iso 8601 format to serialize dates.
Client-side data model deserializes date strings into native Date objects.
The application code expects to interact with Date objects.

you can perform these transformations only when you need to send it to the server for persistence. Or, use computed properties based on the bound value if you need to display it elsewhere in a different format.

For a one-off type transformation this is fine, but as @Morgul said, this is problematic at scale. In our latest project we dealt with a lot of dates. In lieu of type transforms, we used two-way filters. At some point, we made the switch to using moment.js. Updating the filter was incredibly simple.

No template changes were required (still just v-model="someDate | date")
The filter is DRY - just had to change the read/write functions.

In contrast, using computed properties would have been more painful:

Each new date input would require additional boilerplate for setting up a two-way computed prop.
Computed properties aren't DRY. Updating from Dates to moments would require globally updating each computed property. Probably not a huge issue in practice, but definitely more prone to error.

Other thoughts:

Type transformations also by definition makes the view out of sync with the underlying state.

I would disagree - the view is a representation of the underlying state. The type transformation simply transforms an unrepresentable type (Date object) into something that is representable (date string).

if you programmatically set the bound value to a Date object, there's no way for Vue to know how to display it as text in the input box. Then again we are back to similar concepts as two-way filters.

Type transformations are definitely similar to two-way filters, however they seem like they would be much more simple. An intentionally minimal API makes sense.

Transforms are a v-model directive modifier and do not otherwise complicate the template syntax.
The modifier syntax does not permit additional arguments in the template, which would keep transforms simple. You get an input value, produce an output value.
Similar to filters, I'd agree that Vue shouldn't provide builtin transforms (except for number).

Possible API:
Vue.typeTransform('date', {
  render(value) {
    return value ? value.isoformat() : '';
  },

  parse(value) {
    return value ? new Date(value) : undefined;
  },
});

This would be in line with the coerce functionality in component props.

@rpkilby Opened https://github.com/vuejs/vue/issues/3666

A boolean modifier would have been great.

This works perfect, indeed a boolean modifier would also be a nice addition 👍

Az
On Wed, 1 Aug 2018, 18:31 Jerry Jacobs<EMAIL_ADDRESS>wrote:

This works perfect, indeed a boolean modifier would also be a nice
addition 👍
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vuejs/vue/issues/2002#issuecomment-409656290, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AOYJoQCojZNAjrbZxY_4QLHVHItaRZCqks5uMeYLgaJpZM4Gy4Xw
.


Proposal: Warn about unused values in values file

When using a values file during a chart installation like helm install --values values.yaml my-chart, Helm doesn't tell me if I've specified values in values.yaml that are not consumed by the installation. For example, if I have the following values.yaml:
db:
  username: "admin"

and my Helm chart is looking for {{ .Values.db.user }}, the install continues without using what's in my values.yaml. Although this is expected behavior, it's been very difficult for me to track these errors down for charts with larger values files or subcharts.
Having Helm notify me if there were values in my values.yaml that were not consumed would help make the error more apparent. Using the example above, outputting something like:
Warning: value db.username not consumed

It would also be really nice if there was an option to fail the install if there were unused values. Again, using the example above:
helm install --strict-consume --values.yaml my-chart

would fail the installation.
Output of helm version:
Client: &version.Version{SemVer:"v2.14.3", GitCommit:"0e7f3b6637f7af8fcfddb3d2941fcc7cbebb0085", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.14.3", GitCommit:"0e7f3b6637f7af8fcfddb3d2941fcc7cbebb0085", GitTreeState:"clean"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.3", GitCommit:"2d3c76f9091b6bec110a5e63777c332469e0cba2", GitTreeState:"clean", BuildDate:"2019-08-19T11:13:54Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.2", GitCommit:"f6278300bebbb750328ac16ee6dd3aa7d3549568", GitTreeState:"clean", BuildDate:"2019-08-05T09:15:22Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):
AKS, Minikube

@karuppiah7890 yes I would! Okay, I'll do it for v3. Awesome, thanks 😃

This feature would be really nice. Ideally, it should have a strict mode that makes the command fail instead of just warn.

Is there any progress here?

@kurczynski Are you working on this? Or someone else could pick it up :)

I've looked into it but haven't had the time to get started. If anyone would like to work on it, have at it :-D

I would have expected the values.schema.json feature to deal with this problem. Unfortunately, it doesn't complain about extra values either - which is very surprising given that unexpected field values should count as a schema violation.
Having configuration accidentally exist at the wrong place in a values.yaml file is a fairly common mistake and without careful inspection of the generated output it's very easy to miss.
Please consider adding this feature to the schema validator and ideally as an option for schema-less values.

@seanhoughton You can use the values.schema.json to get errors when some unknown / unexpected / extra fields are provided as input. Example
{
    "$schema": "http://json-schema.org/schema#",
    "type": "object",
    "additionalProperties": false,
    "properties": {
        "affinity": {
            "type": "object"
        },
        "autoscaling": {
            "type": "object",
            "properties": {
                "enabled": {
                    "type": "boolean"

@seanhoughton You can use the values.schema.json to get errors when some unknown / unexpected / extra fields are provided as input. Example
{
    "$schema": "http://json-schema.org/schema#",
    "type": "object",
    "additionalProperties": false,
    "properties": {
        "affinity": {
            "type": "object"
        },
        "autoscaling": {
            "type": "object",
            "properties": {
                "enabled": {
                    "type": "boolean"

cc @kurczynski

cc @kurczynski

Thanks @karuppiah7890 , that's why I edited my post and mentioned that it's a reasonable alternative.

Thanks @karuppiah7890 , that's why I edited my post and mentioned that it's a reasonable alternative.

Ah okay. I thought you were proposing something else as an alternative

Ah okay. I thought you were proposing something else as an alternative

If user provided schema can help, can you elaborate what feature you want to add to Helm? I didn't get that part

If user provided schema can help, can you elaborate what feature you want to add to Helm? I didn't get that part

Creating a schema is a reasonable solution for helm charts you control. However, there are many existing helm charts that don't have schemas and are used either directly or as dependencies. The external dependencies sometimes contain poor documentation or complex value hierarchies and it's very easy to make mistakes due to unfamiliarity and lack of error checking. It would be helpful to have some basic linting features for these helm charts without schemas.
One alternative might be to auto-generate a schema for everything internally (or in our CI pipeline somehow) using the existing default values.yaml file and lint against that. However, if the author has a lot of commented out or missing example values this won't work well.
Ultimately all charts should get schemas which would make this issue moot.

Creating a schema is a reasonable solution for helm charts you control. However, there are many existing helm charts that don't have schemas and are used either directly or as dependencies. The external dependencies sometimes contain poor documentation or complex value hierarchies and it's very easy to make mistakes due to unfamiliarity and lack of error checking. It would be helpful to have some basic linting features for these helm charts without schemas.
One alternative might be to auto-generate a schema for everything internally (or in our CI pipeline somehow) using the existing default values.yaml file and lint against that. However, if the author has a lot of commented out or missing example values this won't work well.
Ultimately all charts should get schemas which would make this issue moot.

Makes sense. I'll check more into this. And auto generating schema makes sense but yeah, like you said, it might not be exactly a complete solution depending on the chart. I've worked on schema auto gen before (https://github.com/karuppiah7890/helm-schema-gen) . I'll see if some basic issues can be caught with the help of plugins. But I cannot comment on bringing this feature to the core of helm. I'll let the core maintainers comment on it

Makes sense. I'll check more into this. And auto generating schema makes sense but yeah, like you said, it might not be exactly a complete solution depending on the chart. I've worked on schema auto gen before (https://github.com/karuppiah7890/helm-schema-gen) . I'll see if some basic issues can be caught with the help of plugins. But I cannot comment on bringing this feature to the core of helm. I'll let the core maintainers comment on it

This feature would still be useful, without having to generate a schema.

👋 Hey folks. I'm going to try taking a whack at this, but I'll need some help. I'll look for y'all in slack

👋 Hey folks, I have https://github.com/DataDog/k8s-platform-resources/pull/5604 ready for review and was hoping someone in this thread could take a look.

Hey guys, just dropping this germ for your values.schema.json:
"additionalProperties": false
works like a charm for us


Fix issue #2136

This PR fixes issue #2136

This issue has been fixed by the following commit. Therefore closing the PR
https://github.com/ballerinalang/ballerina/commit/8b1c2a9ce905d5b4c7233d717b7646ef95ff331a


Having CSS as part of a module

I just found about mithril and it looks pretty sweet so far!
What do you think of having a CSS file for each module lives in the same folder where the module is. Since a module represent part of the UI in our app, why not having the styling for this component at the same place where the rest of the module? (increase the cohesiveness of our modules)
Here are a few options for implementing this - https://gist.github.com/substack/68f8d502be42d5cd4942#css
insert-css is one of those options:
var fs = require('fs');
var insertCss = require('insert-css');
var css = fs.readFileSync(__dirname + '/style.css');
insertCss(css);

It uses browserify and injects the style.css into the header.

Yeah, you could probably do this, but it's not really in scope for Mithril. Might be a good suggestion for boilerplate projects though.

thanks a lot. i'll look at those projects.


Update U.S. employment norm peak plot to June 4 data

This PR updates the U.S. employment normalized peak plot to the June 4, 2021 data release (May 2021 employment data).

Codecov Report

Merging #24 (ee35479) into master (5708078) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #24   +/-   ##
=======================================
  Coverage   95.12%   95.12%
=======================================
  Files           2        2
  Lines         164      164
=======================================
  Hits          156      156
  Misses          8        8

Continue to review full report at Codecov.

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


PR: Refactor use of subprocess.run

Fixes #13

Codecov Report
Base: 83.14% // Head: 79.88% // Decreases project coverage by -3.25% :warning:

Coverage data is based on head (c5a3c53) compared to base (493b098).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
- Coverage   83.14%   79.88%   -3.26%
==========================================
  Files           7        7
  Lines         534      527       -7
  Branches       75       69       -6
==========================================
- Hits          444      421      -23
- Misses         56       65       +9
- Partials       34       41       +7

Impacted Files
Coverage Δ

envs_manager/api.py
68.57% <100.00%> (+6.50%)
:arrow_up:

envs_manager/backends/conda_like_interface.py
71.63% <100.00%> (-7.83%)
:arrow_down:

envs_manager/backends/venv_interface.py
84.90% <100.00%> (-2.26%)
:arrow_down:

envs_manager/tests/test_manager.py
92.59% <0.00%> (-3.81%)
:arrow_down:

envs_manager/tests/test_cli.py
100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[CI][BYORTL] add Verilator regression test to CI

@tqchen @tmoreau89

Thanks @vegaluisjose , looking at the files, looks like that libverilator.so needs to be shared cross stages. Merging this now would create a problem.
It would be great to not do so, and build the library before test

Hey @tqchen ,
I don't think I am following, could you elaborate on what the problem will be?
The library is currently built when the CMAKE macro USE_VERILATOR_HW is ON. Also, during this stage the library is linked, so tvm knows where to find such library when selecting that backend in Python.

The main problem would be right after we merge the PR，if there is no library named libverilator(because it is not yet in the rules) the CI on the main will fail. Until we add a PR to build the libverilator hw, given that the hw itself is not hard to build, would be great if we build it at the test stage instead of caching it after build

It would really be nice if the hw part can be built independently, and loaded on demand.

+1

Hey guys,
The CI is updated now with the new version of Verilator backend. CI (Jenkins) seems to be working.

Hey guys,
The CI is updated now with the new version of Verilator backend. CI (Jenkins) seems to be working.

Thank you @liangfu @tqchen @vegaluisjose ; the PR has been merged.

Thank you @liangfu @tqchen @vegaluisjose ; the PR has been merged.


Get Started with the SAP BTP Command Line Interface (btp CLI)

Tutorials: https://developers.sap.com/tutorials/cp-sapcp-getstarted.html
Hi,
I have installed BTP latest version (client v2.8.0) on windows and tried running few steps locally.
After every two or 3 command executions, it's  asking to re-login.
I am facing this issue today repeatedly. Please check on this.

Thanks
Mounika

Hi Mounika,
thanks for reporting this issue not only here, but also as a bug. As you have probably seen, the issue has been resolved. My apologies on not answering more quickly here! Unfortunately, I was away from the office for a long time and my process of forwarding issues failed.
Hope you are again enjoying working with the btp CLI.
Kind regards,
Anna


feat: Set default memory limit relative to available OS memory

This is a draft on how setting relative memory limits could work using github.com/pbnjay/memory.
Note that running this on a machine with 32 GiB RAM shows that this simple relative limit does not make too much sense, as it would allow to use 24 GiB - which is probably not a reasonable assumption in a multi tenancy environment.
In something like a container with 1-2GiB memory, setting a soft limit 25% below the hard limit of the container would make sense though.
I think this level of intelligent setting of memory limits is better left to endusers who can configure what they need using GOMEMLIMIT.

Won't do - as described above, a lower default limit and docs on how to tune it to your usecase is preferrable


wait for client.admin key exists hangs in Debian

./roles/ceph-mon/tasks/ceph_keys.yml hangs in Debian.

name: wait for client.admin key exists
wait_for: path=/etc/ceph/ceph.client.admin.keyring

I have sent a PR for this. https://github.com/Abhishekvrshny/ceph-ansible/pull/1

can you send your PR or ceph-ansible and not on your fork? :)

Closed by #380


[Bug Fix]: CMS association not detected when API form is rendered

Addresses: https://github.com/restarone/violet_rails/issues/1321
DEMO
https://user-images.githubusercontent.com/25191509/216809971-c0b7d411-9b0f-4479-a4ed-263f07cb4429.mov

Acceptance Criteria:

When the "Form rendering snippet" Code is added to any page it should be associated with the namespace of whose the snippet code is added.
Gap or any html tag should not block the association of the Namespace with the CMS.


Natural language semantic code search at Github

Article demonstrating the implementation of search feature within Github

Can this be merged?


(FACT-900) Use GetUserProfileDirectory

Previously, we queried environment variables to find the User's profile
directory. This followed logic used in Puppet to do the same. That was
more error-prone and required querying multiple environment variables.
Use GetUserProfileDirectory to get the path instead, which should work on
all platforms. In particular this will be better supported in future
Windows versions.

/cc @Iristyle

Verified on Windows Server 2012r2:
As Admin
2015-04-08 14:49:56.729314 DEBUG puppetlabs.facter - searching "C:/ProgramData\PuppetLabs\facter\facts.d" for external facts.
2015-04-08 14:49:56.729314 DEBUG puppetlabs.facter - checking execution on C:/ProgramData\PuppetLabs\facter\facts.d\foo.bat
2015-04-08 14:49:56.729314 DEBUG puppetlabs.facter - resolving facts from executable file "C:/ProgramData\PuppetLabs\facter\facts.d\foo.bat".
2015-04-08 14:49:56.729314 DEBUG puppetlabs.facter - executing command: C:/ProgramData\PuppetLabs\facter\facts.d\foo.bat
2015-04-08 14:49:56.744955 DEBUG | - zebra=C:\Program Files (x86)
2015-04-08 14:49:56.744955 DEBUG puppetlabs.facter - fact "zebra" has resolved to "C:\Program Files (x86)".
2015-04-08 14:49:56.744955 DEBUG puppetlabs.facter - process exited with exit code 0.
2015-04-08 14:49:56.744955 DEBUG puppetlabs.facter - completed resolving facts from executable file "C:/ProgramData\PuppetLabs\facter\facts.d\foo.bat".

As Standard User
2015-04-08 14:52:50.174710 DEBUG puppetlabs.facter - searching "C:/Users\michael\.puppetlabs\opt\facter\facts.d" for external facts.
2015-04-08 14:52:50.174710 DEBUG puppetlabs.facter - checking execution on C:/Users\michael\.puppetlabs\opt\facter\facts.d\foo.bat
2015-04-08 14:52:50.174710 DEBUG puppetlabs.facter - resolving facts from executable file "C:/Users\michael\.puppetlabs\opt\facter\facts.d\foo.bat".
2015-04-08 14:52:50.174710 DEBUG puppetlabs.facter - executing command: C:/Users\michael\.puppetlabs\opt\facter\facts.d\foo.bat
2015-04-08 14:52:50.190342 DEBUG | - zebra=C:\Users\michael
2015-04-08 14:52:50.190342 DEBUG puppetlabs.facter - fact "zebra" has resolved to "C:\Users\michael".
2015-04-08 14:52:50.190342 DEBUG puppetlabs.facter - process exited with exit code 0.
2015-04-08 14:52:50.190342 DEBUG puppetlabs.facter - completed resolving facts from executable file "C:/Users\michael\.puppetlabs\opt\facter\facts.d\foo.bat".

Verified on Windows Server 2003:
As Admin 2015-04-08 14:47:05.000000 DEBUG puppetlabs.facter - skipping external facts for "C:\Documents and Settings\All Users\Application Data\PuppetLabs\facter\facts.d": No such file or directory
As Standard User 2015-04-08 14:51:35.828125 DEBUG puppetlabs.facter - skipping external facts for "C:\Documents and Settings\michael\.puppetlabs\opt\facter\facts.d": No such file or directory

Other than above, :+1: provided the Windows folks are :+1: .

CLA signed by all contributors.

:+1:

I broke the misc changes into a bunch of separate commits.

Looks great... :shipit:


Fix failing workflow

Update baseline
Test workflow run
https://github.com/priyanshuone6/PyBaMM/runs/3799185566?check_suite_focus=true

Thanks Priyanshu!


Spectra plot improve

This moves the SpectrumLike plotting to a member function and allows for over plotting of other models (such as was seen in Vianello et al 2017 ;) ).
The old global function was kept, but is simple a convience wrapper that allows for selecting color sets, etc.
A few other features were added to improve plotting.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@5cdadb1). Click here to learn what that means.
The diff coverage is 80.57%.

@@           Coverage Diff            @@
##             master    #224   +/-   ##
========================================
  Coverage          ?   74.3%
========================================
  Files             ?     132
  Lines             ?   14767
  Branches          ?       0
========================================
  Hits              ?   10973
  Misses            ?    3794
  Partials          ?       0

It looks like you are still working on this. Please tell me when you are done and I'll merge it.
Does it change anything regarding the documentation? Please have a look and make sure that the documentation still works.

Ah, I just added some docs.
Did you add 3 docs for this?
Sent from my iPhone

On 17 Jul 2017, at 20:10, Giacomo Vianello<EMAIL_ADDRESS>wrote:
It looks like you are still working on this. Please tell me when you are done and I'll merge it.
Does it change anything regarding the documentation? Please have a look and make sure that the documentation still works.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Documentation was added. In the process, I added .from_function class methods to the SpectrumLike classes following the XYLike approach. This should be useful for testing the background modeling in the near future.

@giacomov I am done with this for now. I'll continue to work on the documentation but add that directly .

@giacomov I removed the duplicated code and added some tests to check fitting of the SpectrumLike classes now that we have generators.

Thanks! Merging it now


PoC: Skip transparent objects and optimize per-frame material updates.

work-in-progress
This is "no chains" fork of #1271 to experiment with directions on how to efficiently update materials, TextElements and objects.
Skip completly transparent objects, so then don't land in render
pipeline if opacity===0.

Tile: add updateDynamicObjects callback that shall be called by
MapView to update dynamic objects explicitly

MapView: Call updateDynamicObjects on all rendered tiles and then don't add
completly transparent objects to scene

TileGeometryCreator Refactor most of onBeforeRender callbacks so they are executed
in controlled (in scope of Tile.updateDynamicObjects)

TextStyleCache to use Tile.addUpdater to support full interpolation some of TextElement props

Refactor fading render helper to use Tile.addUpdater Don't install fading render helper at all if fading is disabled

Codecov Report

Merging #1274 into master will decrease coverage by 0.08%.
The diff coverage is 27.46%.

@@            Coverage Diff             @@
##           master    #1274      +/-   ##
==========================================
- Coverage   52.94%   52.86%   -0.09%
==========================================
  Files         260      249      -11
  Lines       23390    23144     -246
  Branches     5177     5131      -46
==========================================
- Hits        12385    12234     -151
+ Misses      11005    10910      -95

Impacted Files
Coverage Δ

@here/harp-mapview/lib/poi/PoiRenderer.ts
11.25% <0%> (ø)
:arrow_up:

@here/harp-mapview/lib/text/TextElement.ts
87.17% <100%> (+0.69%)
:arrow_up:

@here/harp-mapview/lib/text/Placement.ts
83.87% <100%> (ø)
:arrow_up:

@here/harp-materials/lib/MapMeshMaterials.ts
40.61% <100%> (+1.97%)
:arrow_up:

@here/harp-mapview/lib/DecodedTileHelpers.ts
51.47% <100%> (ø)
:arrow_up:

...e/harp-mapview/lib/geometry/TileGeometryCreator.ts
28.15% <19.14%> (+1.74%)
:arrow_up:

@here/harp-mapview/lib/text/TextStyleCache.ts
52.74% <19.23%> (-4.13%)
:arrow_down:

@here/harp-mapview/lib/Tile.ts
64.88% <57.14%> (-0.19%)
:arrow_down:

@here/harp-mapview/lib/MapView.ts
59.86% <80%> (+0.08%)
:arrow_up:

...re/harp-text-canvas/lib/rendering/TextMaterials.ts
42.85% <0%> (-34.07%)
:arrow_down:

... and 36 more

Continue to review full report at Codecov.

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


Allow for click to enlarge images on the data portal

The channel is narrow to support readability but this can make images small and hard to read.
Lets use one of the gatsby plugins to pop open the images to a larger frame on click.

There are additional comments regarding this in this closed dupe:
https://github.com/HumanCellAtlas/data-browser/issues/824

@MillenniumFalconMechanic resolved by PR https://github.com/HumanCellAtlas/data-portal/pull/513/files


Smoketest failure "hides the toc when searching depending on the search behavior"

https://dev.azure.com/monacotools/Monaco/_build/results?buildId=236635&view=logs&j=4801dce2-64f3-53d6-b366-d49a1977c639&t=91c4c8b0-4e34-5230-16e7-5d930ac20e2c
I wonder if this is worth maintaining a smoketest for. Smoketests just tend to be flaky

The setting was toggled properly, but I'm not sure whether the search was done so quickly after that the ToC continued to show. I'll skip this test for now, but I think it's worth repairing in a later iteration.


When using LoadFromBrowser, a popup window appears if the content is in json.

When using LoadFromBrowser, a popup window appears if the content is in json.
Can I disable the pop-up window?
Language: C#
HtmlAgilityPack ver: 1.11.38
Point of Popup
HtmlAgilityPack.HtmlDocument doc = web.LoadFromBrowser(strurl);

Hello @ogosengi ,
Thank you for reporting, we will look at it.
Best Regards,
Jon

Sponsorship
Help us improve this library
Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Hello @ogosengi ,
The v1.11.39 has been released.
Could you try it and let me know if everything is fixed

Is this library useful to you? Please help us by becoming a sponsor to keep it alive and supported.

Thank you for Your fast reply.
But It hasn't been fixed yet.
I am coding in VisualStudio2019(WindowsForm, .NET Framework 4.7.2)
example code:
string str1 = "https://infuser.odcloud.kr/oas/docs?namespace=15074473/v1";
HtmlWeb web = new HtmlWeb();
HtmlAgilityPack.HtmlDocument doc = web.LoadFromBrowser(str1);
The pop-up window still appears.

Hello @ogosengi ,
Unfortunately, I don't get a popup anymore on my side even with the URL and config you provided. So that makes it pretty much impossible to fix it on my side.
Could you check if you still get the popup with this code:
string str1 = "https://infuser.odcloud.kr/oas/docs?namespace=15074473/v1";

var webBrowser = new WebBrowser();
webBrowser.ScriptErrorsSuppressed = true;
webBrowser.NewWindow += WebBrowser_NewWindow;
webBrowser.Navigate(str1);

private void WebBrowser_NewWindow(object sender, CancelEventArgs e)
{
	e.Cancel = true;
}

If yes, try to fix it if you can and we will apply your change. As said, we cannot reproduce it so we cannot go anything more on our side.
Best Regards,
Jon

Hello @JonathanMagnan
It is same problem.
From googling, It seems that it is probably a problem with IE Setting on my computer.
Perhaps htmlagilityPack was made based on IE.
If I try the URL in IE, it will try to download the file in the same way.

I Try Selenium Chrome Driver (very slow ㅜㅜ)
private void button2_Click(object sender, EventArgs e)
{
ChromeOptions options = new ChromeOptions();
ChromeDriverService driverService = ChromeDriverService.CreateDefaultService();
driverService.HideCommandPromptWindow = true;
options.AddArgument("headless");
ChromeDriver driver = new ChromeDriver(driverService, options);

driver.Navigate().GoToUrl("https://infuser.odcloud.kr/oas/docs?namespace=15074473/v1");
textBox1.Text = driver.PageSource;

}
It is not Popup

Hello @ogosengi ,
Unfortunately, we cannot add a reference to selenium to our project.

I solved it in another way.
https://stackoverflow.com/questions/6114360/ie-prompts-to-open-or-save-json-result-from-server
I try Registry Update, It work.
JSON mime type

Windows Registry Editor Version 5.00
;
; Tell IE to open JSON documents in the browser (rather than always demanding that it saves
; the JSON data into an external file first).
; 25336920-03F9-11cf-8FD0-00AA00686F13 is the CLSID for the "Browse in place" .
;
; Taken from:
; http://stackoverflow.com/questions/2483771/how-can-i-convince-ie-to-simply-display-application-json-rather-than-offer-to-do
[HKEY_CLASSES_ROOT\MIME\Database\Content Type\application/json]
"CLSID"="{25336920-03F9-11cf-8FD0-00AA00686F13}"
"Encoding"=hex:08,00,00,00
[HKEY_CLASSES_ROOT\MIME\Database\Content Type\text/json]
"CLSID"="{25336920-03F9-11cf-8FD0-00AA00686F13}"
"Encoding"=hex:08,00,00,00

这是来自QQ邮箱的假期自动回复邮件。

您好，你的邮件已收到。
亮亮


Implements the ProfileStatsView and friends

Test Suite 'All tests' passed at 2016-10-10 17:12:05.784.
   Executed 2249 tests, with 0 failures (0 unexpected) in 226.101 (227.078) seconds

good to merge?

Yep, I marked it "approved" in GitHub land but forgot to change the label. GTG.


ADO to GitHub - ADO Work Item links

Description
I've performed a trial run of migration from Azure DevOps Repos to GitHub and have a query around Work Item links. According to https://docs.github.com/en/migrations/using-github-enterprise-importer/migrating-from-azure-devops-to-github-enterprise-cloud/about-migrations-from-azure-devops-to-github-enterprise-cloud#data-that-is-migrated
Work items links on pull requests are migrated. This is a pretty key feature as we have many pull requests with the linked to many different work items with Azure DevOps Boards.
However when I ran the test work items ended up like this:

The pull request not found here is because ado2gh has disabled the ADO repo. But with the Work item links on PRs being migrated to me implies I should see GitHub PR links on this work item, like https://learn.microsoft.com/en-gb/azure/devops/boards/github/link-to-from-github?view=azure-devops#view-or-open-links-from-the-development-section

I've poked around a bit but cant see what I may have done wrong. Is this an issue with the migrator?
Reproduction Steps

Install ado2gh cli v1.8.0
Run generate-script pointing ado2gh at an ADO project where you have at least one repo, with completed and potentially in progress PRs that have links to ADO Boards work items.
Run the migration powershell script
Inspect GitHub PRs migrated, they link back to ADO work items
Inspect the ADO work item, it shows errors for development links to pull requests (referring to the disabled ADO repo) and they dont show any links to GitHub PRs.

I was able to reproduce this. It looks like GEI is generating links in the PR description back to the AzDO work items, but the format of the link is not what the AzDO Boards <-> GitHub Integration expects.  If the link format in the GH PR was AB#123 then the Boards Integration would kick in and automatically create the link on the AzDO side.

Thanks both for your input. Ive had a look but don't see where its putting those PR comments in. I assume it happens inside the GitHub CLI migration logic?
I did try to see if I could work around this limitation by querying all the PRs in GitHub and then calling ADO REST APIs to add a link to the GitHub PR, but from the ADO side its not possible to patch work items with a GitHub link due to that being controlled by the ADO GitHub app (which has a token for the connection to GitHub).
https://github.com/github/gh-gei/blob/main/src/ado2gh/Commands/ConfigureAutoLink/ConfigureAutoLinkCommandHandler.cs looks like it runs after the migration of the repo in order to configure the auto link for future PRs right? So I don't think modifying/adding to that would help unless I'm wrong.
Is this an issue that needs to be raised against the GitHub CLI itself?

👋 @WillEllis,
Yes, this needs to be fixed in the backend since the actual repo migration happens in the GEI backend and the CLI just initiates it. I will create an issue to address this.

Thanks @ArinGhazarian. In the meantime if anyone else has this issue, I have managed to work around this limitation right now by writing a script that after migration goes through all the migrated repos PRs, finds any that have "AzureDevOps/#" in the body and updates that piece of text to be "AB#". As pointed out by Dylan above, this is what the Azure Boards integration is expecting and it then does its thing and links all the PRs so they show in the ADO work items correctly.

Hi @WillEllis, can you share this script?

@evervondel here you go
# Define GitHub repository information
$githubOrg = ""
$githubRepo = ""
$githubToken = ""
$pageSize = 100

# Function to call GitHub API to get pull requests
function Get-GitHubPullRequests {
    $url = "https://api.github.com/repos/$githubOrg/$githubRepo/pulls?state=all&per_page=$pageSize"
    $headers = @{
        "Authorization" = "token $githubToken"
    }
    $pullRequests = @()
    do {
        $response = Invoke-WebRequest -Uri $url -Headers $headers

        # Parse JSON response
        $responseData = $response.Content | ConvertFrom-Json

        # Output pull requests
        $pullRequests += $responseData

        # Check if there's another page
        $nextPageUrl = $response.RelationLink.next
        if ($nextPageUrl) {
            $url = $nextPageUrl
        } else {
            # No more pages, exit loop
            $url = $null
        }
    } while ($url)

    return $pullRequests
}

# Function to update PR description in GitHub repository
function Update-GitHubPRDescription {
    param(
        [string]$prNumber,
        [string]$updatedDescription
    )

    $url = "https://api.github.com/repos/$githubOrg/$githubRepo/pulls/$prNumber"
    $headers = @{
        "Authorization" = "token $githubToken"
        "Content-Type" = "application/json"
    }
    $body = @{
        "body" = $updatedDescription
    } | ConvertTo-Json

    Invoke-RestMethod -Uri $url -Method Patch -Headers $headers -Body $body
}

# Get all pull requests from GitHub repository
$pullRequests = Get-GitHubPullRequests

# Loop through each pull request
foreach ($pullRequest in $pullRequests) {
    $prNumber = $pullRequest.number
    $prBody = $pullRequest.body

    # Check if the PR body contains the string "AzureDevOps#"
    if ($prBody -like "*AzureDevOps/#*") {
        # Update the PR body by replacing "AzureDevOps#" with "AB#"
        $updatedDescription = $prBody -replace "AzureDevOps/#", "AB#"
        Write-Host "Updating description for PR $prNumber"

        # Call function to update the PR description
        Update-GitHubPRDescription -prNumber $prNumber -updatedDescription $updatedDescription
    }
}

kudos !!
On Fri, Sep 20, 2024 at 9:24 PM Will Ellis @.***> wrote:

@evervondel https://github.com/evervondel here you go
Define GitHub repository information
$githubOrg = ""
$githubRepo = ""
$githubToken = ""
$pageSize = 100
Function to call GitHub API to get pull requests
function Get-GitHubPullRequests {
$url = "https://api.github.com/repos/$githubOrg/$githubRepo/pulls?state=all&per_page=$pageSize"
$headers = @{
"Authorization" = "token $githubToken"
}
$pullRequests = @()
do {
$response = Invoke-WebRequest -Uri $url -Headers $headers
    # Parse JSON response
    $responseData = $response.Content | ConvertFrom-Json

    # Output pull requests
    $pullRequests += $responseData

    # Check if there's another page
    $nextPageUrl = $response.RelationLink.next
    if ($nextPageUrl) {
        $url = $nextPageUrl
    } else {
        # No more pages, exit loop
        $url = $null
    }
} while ($url)

return $pullRequests

}
Function to update PR description in GitHub repository
function Update-GitHubPRDescription {
param(
[string]$prNumber,
[string]$updatedDescription
)
$url = "https://api.github.com/repos/$githubOrg/$githubRepo/pulls/$prNumber"
$headers = @{
    "Authorization" = "token $githubToken"
    "Content-Type" = "application/json"
}
$body = @{
    "body" = $updatedDescription
} | ConvertTo-Json

Invoke-RestMethod -Uri $url -Method Patch -Headers $headers -Body $body

}
Get all pull requests from GitHub repository
$pullRequests = Get-GitHubPullRequests
Loop through each pull request
foreach ($pullRequest in $pullRequests) {
$prNumber = $pullRequest.number
$prBody = $pullRequest.body
# Check if the PR body contains the string "AzureDevOps#"
if ($prBody -like "*AzureDevOps/#*") {
    # Update the PR body by replacing "AzureDevOps#" with "AB#"
    $updatedDescription = $prBody -replace "AzureDevOps/#", "AB#"
    Write-Host "Updating description for PR $prNumber"

    # Call function to update the PR description
    Update-GitHubPRDescription -prNumber $prNumber -updatedDescription $updatedDescription
}

}
—
Reply to this email directly, view it on GitHub
https://github.com/github/gh-gei/issues/1228#issuecomment-2364439891,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEVNO32XNVEM422AQKABGJ3ZXRY7PAVCNFSM6AAAAABDTP3BYKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRUGQZTSOBZGE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Firefox Quick Find triggers when typing '

Firefox has a feature called "Quick Find" which lets the user open a link using purely the keyboard. It gets triggered by '. Normally, you can still type ' just fine in text fields and so forth without opening up the Quick Find menu every time but that doesn't seem to apply to your site.
For reference, this option can be disabled globally by setting accessibility.typeaheadfind.manual to false in about:config.
(Note: Tridactyl's equivalent jump to links feature, bound to fF still works even if that option is set to false)
How to reproduce:

Open Firefox
Make sure accessibility.typeaheadfind.manual is set to true in about:config
Press '

Pressing / also pops up Quick Find but that's unlikely to appear in the list of words.

Thanks for reporting this, should be fixed now. :)


Bug: hamburger menu button seems broken

The hamburger menu does not cause anything to happen for me, should it? Or should it be removed?

Small oversight on my part. The hamburger is not the same as the un..hamburgered menu, and I missed that and thus it doesn't have any content right now. Should be very fixable!

This was fixed in the finalization of the design! :eyes:


API route for querying version information

Describe the maintenance task
We currently have plugins and dependencies that both have versions. There presently no way to query version from the API.
Describe why this task is necessary
Report versions to the user. This is currently done through a Django template in CE-UI.
Additional context
The suggestion is to have a single route in the manager app, e.g.
/manager/api/versions
that returns a list of all modules with respective version, license and homepage information.

Implemented in c32eda9


SSL certificate verify failed

Connecting to my room in vim with FlooJoinRoom https://floobits.com/r/nilbus/my_room fails with:
Error connecting:[Errno 1] _ssl.c:504: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed

Connecting with http instead of https works, even though I selected Require SSL (which is perhaps a separate issue)

I am currently having this issue, don't mean to necropost but it's currently an issue for me on sublime text 3 and vim/neovim.


why it cost much more time why it cost much more time between finish epoch and save checkpoint when run multimachine when loading much more data?

Running on sync mode.
About loading 1days, it will only cost 10s betwwen finish epoch and save checkpoints.
About loading 10days,it cost 87s ,and loading 15days data to memory,it will cost 152s
It seem there has relevant with larger data.Is it reasonably?

can you specify your question specifically?

i add some code on python about nd.save.
toc3 = time.time()
nd.save(param_name, save_dict)
toc4 = time.time()
logging.info('nd[%d] nd save param cost=%.3f', epoch, (toc4 - toc3))
@starimpact

the size>

what u mean?
As I said, the much more data loaded,it cost much more time to save
@starimpact

it is normal.
the data transfer from gpu and cpu is slower as your data is bigger.

but in fact is it only up to batchsize?
the batchsize is equal.
What i means is that loading more data before start trainning.
@starimpact

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Add example output in README?

It would be nice to also get some example output in the README. That would explain better what this repo/application does.

Hi! I've tried adding an example to the README now, hope it's easier to understand what uno does.

Nice! 🙏


Duplicate zip entry [com/crashlytics/android/answers/shim/R.class

I am getting the following error due to the new release 2.6.0.

Warning:Exception while processing task java.io.IOException: Can't write [/Users/rishabhharit/Documents/GitHub/myproject-android/myproject/build/intermediates/transforms/proguard/release/jars/3/1f/main.jar] (Can't read [/Users/rishabhharit/Documents/GitHub/myproject-android/myproject/build/intermediates/transforms/retrolambda/release/folders/1/1/retrolambda(;;;;;;**.class)] (Can't read [com] (Can't read [crashlytics] (Can't read [android] (Can't read [answers] (Can't read [shim] (Can't read [R.class] (Duplicate zip entry [com/crashlytics/android/answers/shim/R.class]))))))))

Wasted my whole day to figure this out because I was using
compile 'io.branch.sdk.android:library:2.+'
in my build.gradle.

Same here, tried to update to 2.6.0 and got ZipException: duplicate entry: com/crashlytics/android/answers/shim/R.class.
We're using fabric answers 1.3.12 and crashlytics 2.6.7.
@rishabh876 that's one of the reasons why + in dependency versions is a bad idea.

I wouldn't have used '+' if their documentation didn't recommend it.

Hey guys, sorry for the headache. Please try including Branch via the following in your build.gradle file:
compile ('io.branch.sdk.android:library:2.+') {
  exclude module: 'answers-shim'
}

And let me know if this resolves your dependency issue.

did not work for me because proguard is missing references

@EvangelosG tried excluding answers-shim, worked for me!

Thanks @vsochinsky . @friedger The above exclusion resolves the duplicate entry. Proguard is another adventure. Please contact<EMAIL_ADDRESS>with all details possible (build.gradle(s) & proguard rules) if you require additional assistance.

Hey @EvangelosG I have the problem still here are the libraries that I'm using
compile('com.digits.sdk.android:digits:2.0.6@aar') {
        transitive = true;
    }
compile('com.crashlytics.sdk.android:answers:1.3.12@aar') {
        transitive = true;
    }
    compile('com.crashlytics.sdk.android:crashlytics:2.6.7@aar') {
        transitive = true;
    }
    compile('com.crashlytics.sdk.android:crashlytics-ndk:1.1.2@aar') {
        transitive = true;
    }
    compile('io.branch.sdk.android:library:2.+') {
        exclude module: 'answers-shim'
    }

and this is the error when building the APK
Error:Execution failed for task ':app:transformClassesWithJarMergingForProdRelease'.
> com.android.build.api.transform.TransformException: java.util.zip.ZipException: duplicate entry: com/crashlytics/android/answers/shim/R.class

@leodeleon22 It looks like the you get multiple answers-shim from digits and  answers . Can you please try to exclude answers-shim from digits also (The same way you do for Branch)

@sojanpr I did but I receive an exception because of the missing methods
´´´
Fatal Exception: java.lang.NoClassDefFoundError: com.crashlytics.android.answers.shim.AnswersOptionalLogger
at com.digits.sdk.android.DefaultAnswersLogger.loginBegin(DefaultAnswersLogger.java:48)
at com.digits.sdk.android.DigitsEventCollector.authImpression(DigitsEventCollector.java:90)
at com.digits.sdk.android.AuthClient.startSignUp(AuthClient.java:83)
at com.digits.sdk.android.Digits.authenticate(Digits.java:103)
´´´

@leodeleon22  Sorry that you still face this issue. I need to check  which config in your dependencies exporting the Answers multiple time. Can you give it a  try by excluding the answers-shim form crashlytics . At this point this is not some thing related to Branch SDK

Might not be relevant to most, but for me I had the segment analytics branch integration enabled, so I modified my build.gradle as follows:
    compile ('io.branch.segment.analytics.android.integrations:branch:+') {
        exclude module: 'answers-shim'
    }

And it works.

None of the proposed answers worked for me with
  // https://fabric.io/kits/android/crashlytics/install
  compile('com.crashlytics.sdk.android:crashlytics:2.6.7@aar') {
    transitive = true;
  }

Downgrading to Branch 2.5.8 from 2.6.0 fixed my problem for now, will check back here for updates (if any)

@kevin-ingenuity could you please help to debug. Can  you please share you complete dependency list in build.grade

@friedger for proguard you could add the following rule
-dontwarn com.crashlytics.android.answers.shim.*