User Agent is VLC/3.0.21 LibVLC/3.0.21

For an STB, verification and channel list works great.
When i want to open a channel from the list i get HTTP 459 which is a custom error code from my streamer.
[00007f9b4485adb0] main demux debug: looking for access_demux module matching "http": 14 candidates
[00007f9b4485adb0] main demux debug: no access_demux modules matched
[00007f9b44863fc0] main stream debug: creating access: http://*********/play/live.php?mac=XX:XX:XX:XX:XX:XX&stream=12061&extension=ts&play_token=---------
[00007f9b44863fc0] main stream debug: looking for access module matching "http": 24 candidates
[00007f9b44863fc0] http stream debug: resolving imaxhd.eu ...
[00007f9b44863fc0] http stream debug: outgoing request:
GET /play/live.php?mac=XX:XX:XX:XX:XX:XX&stream=12061&extension=ts&play_token=--------- HTTP/1.1
Host: *******:80
Accept: */*
Accept-Language: en_US
User-Agent: VLC/3.0.21 LibVLC/3.0.21
Range: bytes=0-

[00007f9b44863fc0] http stream debug: incoming response:
HTTP/1.1 459
Date: Thu, 04 Jul 2024 10:51:30 GMT
Content-Type: text/html; charset=UTF-8
Connection: close
Access-Control-Allow-Origin: *

[00007f9b44863fc0] access stream error: HTTP 459 error
[00007f9b44863fc0] http stream debug: connection failed
[00007f9b44863fc0] http stream debug: querying proxy for http://*********/play/live.php?mac=XX:XX:XX:XX:XX:XX&stream=12061&extension=ts&play_token=---------
[00007f9b44863fc0] http stream debug: no proxy
[00007f9b44863fc0] http stream debug: http: server='*********' port=80 file='/play/live.php'
[00007f9b44863fc0] main stream debug: net: connecting to imaxhd.eu port 80
[00007f9b44863fc0] main stream debug: connection succeeded (socket = 15)
[00007f9b44863fc0] http stream debug: sending request:
GET /play/live.php?mac=XX:XX:XX:XX:XX:XX&stream=12061&extension=ts&play_token=--------- HTTP/1.0
Host: *********
User-Agent: VLC/3.0.21 LibVLC/3.0.21
Icy-MetaData: 1

[00007f9b44863fc0] http stream debug: HTTP answer code 459
[00007f9b44863fc0] http stream error: error: HTTP/1.1 459
[00007f9b44863fc0] main stream debug: no access modules matched
[00007f9b3f5134f0] main input error: Your input can't be opened
[00007f9b3f5134f0] main input error: VLC is unable to open the MRL 'http://*********/play/live.php?mac=XX:XX:XX:XX:XX:XX&stream=12061&extension=ts&play_token=--------- Check the log for details.

Since 459 is a custom error code and we don't know what is causing it, i thought maybe sending requests as User-Agent: VLC/3.0.21 LibVLC/3.0.21 might cause this problem because there is no difference then a mag device.
I saw that we are setting User Agent in channel_list.py and config_manager.py but i am still seeing VLC agent as User Agent.

Hi @mkursadulusoy
I don't think that is caused by the useragent, probably you are using a shared account or a bad provider. I guess the other players will not play that either. But still, just to be sure, I made the changes, you can try the following and let me know if this works:
https://github.com/ozankaraali/QiTV/releases/tag/v1.4.5

Closing the issue due to inactivity, keep me updated if the issue is solved or continuing with the prerelese.

I have tried it but apperantly, even the order of the parameters are matter when defining vlc.instance.
User agent was still seem like VLC/3.0.21 LibVLC/3.0.21
i could set user agent succesfully by self.media.add_option(f'http-user-agent={user_agent}') in line 175.
Also i realised my problem was not user agent like you said. My provider is filtering connections by IP. So it is your call.


Update README.md

last PR for now. Next one will again be a big one, maybe when master thesis is finished.


Centralize jest + babel configs

Work on top of:
https://github.com/ReactTraining/react-router/pull/6922
Tests can be run through:
"test": "jest --projects ./packages/*/"

and everything maps to respective modules directories etc, so you don't have to build anything before running tests.
There is still a possibility to run TEST_ENV=cjs npm test but it requires to run build task first (as this setup maps to dist files, so we need to - well - build them 😉 )
I think this work is complete now - if you find anything not working, let me know so I can fix it. If this is also something different from what you have imagined - then please specify what you find to be odd and let me know what would you like to have improved.

Thanks, @Andarist! Really appreciate this. I learned a lot by reading through your Jest config. Using the react-native/jest-preset was new to me, and mapping through the transforms to make the babel-jest transform use rootMode: upward was just what we needed. Again, thank you!

No problem - if u have any problems with a tooling setup you can always ping me so I can at least try to help you out.


[Suggestion] The ability to insta write something in the chat

Hello! This would be very nice and it is, adding the ability to add what I would type in chat once I enter the champ selection.
For example: Instant Message: Mid
When the queue is accepted, it will immediately write what I put in earlier.
regards,

I was thinking about it when creating it but I couldn't figure out how to create a "ui" for it so I skipped it.
I will probably add it at some point when I feel like working on it again, it just didn't feel necessary when creating the app because instalocking champions in blind picks wasn't the goal of it

The thing that is a super useful tool for blind picks for new champion release days. I decided to use this one because the interface is simpler and I thought that something is missing, the idea that I proposed to you!

Yes that would be a great addition to the app it works so great just calling the lanes would be awesome for blind pick !


Add pg_idkit extension

Motivation
Extension pg_idkit should be added.
Why? Neon has an entire page to show support for prisma. My assumption is it wants to work well together with prisma.
Prisma has support for cuid. Great, however if you want to do a custom migration where you want to populate a new table, postgres has no support for cuid (or cuid2 for that matter). If there was support for the extension pg_idkit, this would be an easy fix, and even able to upgrade to cuid2 before prisma does.
I don't know from a neon's perspective what needs to happen for this, but the installation from pg_idkit's side is quite simple.
I've put it as an epic, because it's not a bug I suppose.

I also think that pg_idkit would be a great addition

@sammyolo  thanks for creating this! I would also love to use this! 🙏


remove deprecated methods

These are deprecated at EzXML.jl 0.6 and I think nobody use them anymore.

Codecov Report

Merging #84 into master will increase coverage by 2.24%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #84      +/-   ##
==========================================
+ Coverage   95.53%   97.78%   +2.24%
==========================================
  Files           7        7
  Lines         739      722      -17
==========================================
  Hits          706      706
+ Misses         33       16      -17

Impacted Files
Coverage Δ

src/EzXML.jl
100% <ø> (+94.44%)
:arrow_up:

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 335ab41...4756d8a. Read the comment docs.


Update on Request Header documentation for clarification

Could do with an update on the image for the Request Header Section.
I went off the image first as it was more prominent (used es-ThreadId).  Then looking at the top section I could see that the intended prefix was X-es-backend.

Resuming work on the project: I will take a look at this


Change gcr project to official google_containers

This update the gcr project to the official one.
I've already upload an image node-problem-detector:0.1 to google_containers, will send a PR to kubernetes to add it as an add-on pod soon.
@dchen1107

Manually merged.


Dev

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 3 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-SDK/97/rdkcentral/Lightning-SDK

Commit: ffa4847dbb9734fa86bfd40e168918d1bcea14ad

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 3 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-SDK/97/rdkcentral/Lightning-SDK

Commit: f1299391e0bb6f19eca241e0d8da5f4e1950ce0d

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 3 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-SDK/97/rdkcentral/Lightning-SDK

Commit: 13641c746f0642c91742b750df5954266d88c421

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 3 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-SDK/97/rdkcentral/Lightning-SDK

Commit: 9346456c5fe2a720e86676da4d34e3410cabf058

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 3 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-SDK/97/rdkcentral/Lightning-SDK

Commit: f839b618cb7d62c4e737ddc20a14a5f896c84875

Report detail: gist

WARNING: A Blackduck scan failure has been waived
A prior failure has been upvoted

Upvote reason: matching to itself

Commit: f839b618cb7d62c4e737ddc20a14a5f896c84875

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 3 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-SDK/97/rdkcentral/Lightning-SDK

Commit: 7573cf8342a169f63ce2031f3bb44de6d01d7d37

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 3 files pending identification.

Protex Server Path: /home/blackduck/github/Lightning-SDK/97/rdkcentral/Lightning-SDK

Commit: 7573cf8342a169f63ce2031f3bb44de6d01d7d37

Report detail: gist


How can I use MLR model?

Please refer to the FAQ in doc and search for the related issues before you ask the question.
Describe the question(问题描述)
I want to use the Piece-wise Linear Model to train a dataset organized by time. I want the dataset to be divided by the "matchclock" column, so I wrote the following codes
base_columns = list(set(x_train.columns) - set(['matchclock']))
model = MLR(x_train[['matchclock']], base_feature_columns=x_train[base_columns], region_num=4, l2_reg_linear=1e-5,init_std=0.0001, seed=1024, task='binary')
model.compile("adam", "binary_crossentropy",
          metrics=['binary_crossentropy'], )
history = model.fit(x_train, y_train.values,
                    batch_size=256, epochs=10, verbose=2, validation_split=0.2, )

However it prints out some errors
Traceback (most recent call last):
  File ".\smooth_on_deepctr.py", line 202, in <module>
    model = get_compiled_model_mlr("")
  File ".\smooth_on_deepctr.py", line 147, in get_compiled_model_mlr
    model = MLR(x_train[['matchclock']], base_feature_columns=x_train[base_columns], region_num=4,l2_reg_linear=1e-5,init_std=0.0001, seed=1024, task='binary')
  File "E:\Programs\Anaconda2\envs\py3\lib\site-packages\deepctr\models\mlr.py", line 43, in MLR
    features = build_input_features(region_feature_columns + base_feature_columns + bias_feature_columns)
  File "E:\Programs\Anaconda2\envs\py3\lib\site-packages\pandas\core\ops\__init__.py", line 1488, in f
    other = _align_method_FRAME(self, other, axis)
  File "E:\Programs\Anaconda2\envs\py3\lib\site-packages\pandas\core\ops\__init__.py", line 1458, in _align_method_FRAME
    right = to_series(right)
  File "E:\Programs\Anaconda2\envs\py3\lib\site-packages\pandas\core\ops\__init__.py", line 1419, in to_series
    msg.format(req_len=len(left.columns), given_len=len(right))
ValueError: Unable to coerce to Series, length must be 217: given 0

Additional context
Add any other context about the problem here.
Operating environment(运行环境):

python version 3.6.9
tensorflow version 1.15.0
deepctr version  0.7.0

Is this problem related to how to generate x? My x is a 2d pandas matrix with columns representing features and rows representing records

It compiles when I change x's format


Update mill-main to 0.11.4

About this PR
📦 Updates com.lihaoyi:mill-main from 0.11.2 to 0.11.4
📜 GitHub Release Notes - Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.lihaoyi", artifactId = "mill-main" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.lihaoyi", artifactId = "mill-main" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #105.


Suggestion: add sweep function for potential stucked tokens

This is from experience with current version of vesting contract.
Some vested tokens could be accidentally claimed to same vesting contract and the lack of sweep function doesn't allow to recover them.
Also can be useful in case other tokens are sent to the contract accidentally.
This sweep can be called by the recipient of the vesting, since it should not have access to locked balance, only balanceOf of erc20 provided through argument address.

added in #9


Preparing for 1.0.0-beta.4 release

Current coverage is 100% (diff: 100%)

Merging #49 into master will not change coverage

@@           master   #49   diff @@
===================================
  Files          21    21
  Lines         473   473
  Methods        56    56
  Messages        0     0
  Branches       52    52
===================================
  Hits          473   473
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 4f4a227...d5c0b68


Define a type/table does not exist error code

When calls to the riak_ql helper module throws an undef then return a specific bucket type does not exist error. It is likely to be a common error and not catching it returns a horrible (and mostly uninteresting) stack trace.
This needs to be supported in the clients.

+1 0f16b61

@borshop merge


Fix DOS time conversion

I have made few small fixes and split them into two PRs.
status of tests\test_PyFatFS.py after applying both fixes:
23 failed, 67 passed, 1 skipped

Awesome, thank you!


Missing Some Types

Environment details

OS: macOS 13.5.
Node.js version: 18.16.0
npm version: 9.5.1
@google-cloud/storage version: 6.10.1 / 7.0.0

Steps to reproduce

Try to use any of the lifecycle related types from bucket.ts, e.g LifecycleRule or LifecycleAction.
Get type error saying it doesn't exist.

It appears we're not exporting these in the index file: https://github.com/googleapis/nodejs-storage/blob/main/src/index.ts

Thanks @hspak raising a PR to fix this.


Gpl 216 edit study validation

N.B. Most of the changes in this pull request will disappear once the following PR is merged in: https://github.com/sanger/sequencescape/pull/2454
Validation in this form is weird:

radio buttons are always made required in the front end (form_builder.rb, radio_select method)
other fields are never made required in the front end
asterisks are added based on whether the field is required on the model, which doesn't work for some which are conditionally required (e.g. data access group)

This means that if you submitted the form without having filled out a (non-radio) field that is required on the model, it threw a server side error. This error should have been displayed in a nice Rails way, but -
Rails.logger.warn "Failed to update attributes: #{@study.errors.map(&:to_s)}"
The above line was throwing an exception due to the .map(&:to_s) syntax not working inside a 'rescue' block.
Changing it to the .map { } syntax allows it to get to the 'render' line and means the validation errors are displayed more nicely.

Updated this with a change to the 'attach_option_updater' function in the related_fields file.
This fixes the following issue:

The 3 'prevention' fields were not being saved to the database
This is because of 'if: :never_release?' in the study model, which checks if the 'timing' field is set to 'never'
The form allowed you to fill these fields in when 'timing' was not 'never'
Looks like the form intended this to be prevented by restricting the value in 'timing' to only 'never', when 'strategy' was 'not applicable'
But, this functionality was broken - now fixed


lottieconv documentation not available

The documentation of the lottieconv release is not available at https://docs.msrd0.de

Yes, there is a delay of about a day after publishing on crates.io. They are available now: https://docs.msrd0.de/lottieconv/0.1.0/lottieconv/index.html
Also, you might want to look at https://github.com/msrd0/rlottie-rs/blob/lottieconv-0.1.0/lottieconv/bin/lottie2gif.rs#L63 for an example how to use the crate.


tmplock must be closed before removal

From https://github.com/tus/tusd/pull/117
Quite dirty because I based my modification on the tag v1.0.0 (and added the tests from master, while disabling the non relevant ones)

Thank you again for this PR. I actually made the necessary changes to the master branch so we can use it for tusd (and get all updates from @nightlyone's original repository). And there I noticed that this problem should already be addressed: https://github.com/Acconut/lockfile/blob/0a9809d62f141721dc3cd5819f7c708c9d3eaa09/lockfile.go#L101-L105

I also want to look into how we can possible merge this fork back by adding a setting to the TryLock function. Maintaining a fork is not the best solution.

by adding a setting to the TryLock function

I think you could add an unexported boolean to the Lockfile structure preventOwnerReLock (which by default will be false) and a NewWithoutOwnerReLock function to initialize it.

I thought about that, too, but Lockfile is not a structure, it's just an alias for a string (see https://github.com/nightlyone/lockfile/blob/master/lockfile.go#L19). This makes your solution sadly not possible without breaking backwards-compatibility.

I thought that something like that might work: https://play.golang.org/p/PXRr9hsVff, but no...
Other possibility: add a PreventOwnerReLock global variable which can be set to true globally.


Update alerts-troubleshoot.md

fix discrepancies around webhook retry for AG

@AbbyMSFT this is in response to this issue. The text now matches verbatim what it says in the AG docs that I own. https://github.com/MicrosoftDocs/azure-docs/issues/116611

#sign-off

@AbbyMSFT

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thanks, @AbbyMSFT! I committed your suggestions.
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#sign-off


Call for maintainer(s)

I'm looking for a maintainer for this project. My focus is elsewhere right now, but I know many people still rely on the DocBook toolchain. I'd like to see this project continue to evolve under different leadership.
If you're considering becoming the maintainer of this project, know that you'd have the freedom to take a different approach to integrating the DocBook / Apache FO toolchain other than Gradle. Using Gradle is not a hard requirement. The only requirement is to fill the original mission of the project, which is to provide a way to to run the DocBook toolchain without having to rely on a local install. In short, it's a more portable and modern version of a2x from AsciiDoc Py.

Maybe @bsmith-n4 is interested? Commits · NumberFour/asciidoctor-fopub

As no one has come forward, I'm archiving this repository. If you would like to see it continue, please join the project chat at https://chat.asciidoctor.org and make your intentions known.


Fixed the Typographical error on the gitpod.mdx component

What does this PR does
This PR Correct the typographical error on pages/docs/environments/gitpod.mdx component
Fixes Issue #9540

@Taki7980 This issue is in triage status and not yet assigned nor ready for dev.  This PR will be closed.
Please read our contributing guide before completing any work on issues. How To Contribute


[SCHEMA] Define YAML tables for PET common metadata fields and radiopharmaceutical data v2

References #1014 and #1002
Aped from tsalo's #1017, but just PET fields.
Changes proposed:
Add YAML definitions for PET page's common metadata sections.
Fixes ugly history view from previous (cancelled) PR #1026

I saw that here as well: https://github.com/bids-standard/bids-specification/pull/1017/files#r814328092
I ended up dropping the quotes but @effigies proposed (https://github.com/bids-standard/bids-specification/pull/1017/files#r824140193) dataset.modalities contains "PET"

Switched PET in to -> dataset.modalities contains "PET" per your suggestion. Went right over my head if you mentioned this yesterday.

@bendhouseart can you move your proposed changes into a new src/schema/rules/sidecars/pet.yaml file to deal with the conflict?

@tsalo done per your request.

Squashed and merged manually to ensure minimal destruction.
Compare e332798 to validate.


Update README

Add VS Code extension to related projects
Remove 知乎 link from related projects, because the link is 404

Codecov Report

Merging #104 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #104   +/-   ##
=======================================
  Coverage   94.97%   94.97%
=======================================
  Files           1        1
  Lines         179      179
=======================================
  Hits          170      170
  Misses          9        9

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 fb78e96...01770e2. Read the comment docs.


feat(parser): support short-option groups

This is a proposal for supporting short option groups that is quite simple (which I like), and at the same time agrees with an edge-case documented in yargs-parser -- which make me think it's a reasonable approach (I at least haven't noticed many folks complain about this edge-case in the past).
The implementation
Quite simply, -foo expands out to -f, -o, -o, and each individual value is handled by the next step in the parser.
The interesting edge-case of this is that if withValue is configured for the terminal value in the short option list, and the short-option list is followed by a positional, it will be consumed:
const passedArgs = ['-rf', 'foo'];
const passedOptions = { withValue: ['f'] };
const expected = { flags: { r: true, f: true }, values: { r: undefined, f: 'foo' }, positionals: [] };
const args = parseArgs(passedArgs, passedOptions);

As mentioned, this behavior agrees with unit tests in yargs-parser:
// from yargs' test suite:
it('should set the value of the final option in a group to the next supplied value', function () {
    const parse = parser(['-cats', 'meow'])
    parse.should.have.property('c', true)
    parse.should.have.property('a', true)
    parse.should.have.property('t', true)
    parse.should.have.property('s', 'meow')
    parse.should.have.property('_').with.length(0)
})

TODO: document this decision in README.
Fixes #2

@shadowspawn take a look at the updated FAQ, if you don't mind.

LGTM again.


Button?

Hey it's Ryan from FBLA! Nice job for the win!
About this issue, your "Lets get in touch" button doesn't link to anything. You might want to fix that or remove the hover cursor.

It's a mailto: link so you may not have a mail client configured, but thank you for your feedback!
I'm still in the process of migrating to GitHub pages from shared Apache hosting, so I still have a few kinks to work out but I'll be fully transitioned soon!


TypeExtensionTests - Implement

Add tests for the remaining TypeExtension methods
What is the current behaviour?
TypeExtension does not have test coverage of IsStatic() and CreateDefaultValue()
What is the new behaviour?
TypeExtension has full test coverage
What issues does this resolve?
None
What PRs does this depend on?

#130

Does this introduce a breaking change?

[ ] Yes
[x] No

Let's talk about test naming! There are many conventions, and no single standard, so it's down to personal preference and priorities.
In general the preference seems to be individual tests for each case, rather than one per function. This means that any UI that summarizes test results lets you see immediately which case failed, instead of only which function.
A common format is MethodName_StateUnderTest_ExpectedResult()[^1] (or Given_When_Then), ex. IsStatic_WhenNonStaticClass_False(), though some argue that including the name of the function being tested means manually updating tests when said function is renamed.
One suggestion is to add a separate test class (fixture) for each method[^2], grouping them together and removing the need to include the method name in every test name. This groups them nicely in test UIs as well.
Finally, a more recent standard goes for plain human readability[^3], for example NonStaticClassIsNotStatic() or StaticClassTypeDefaultValueIsNull() (note that names are definitive, not using "should"). This avoids including tested function names, leans into the idea that unit tests are also documentation, and may be more flexible and easier to write with NUnit's attributes (ex. multiple test cases).
Also, I arbitrarily created static classes for the first test fixtures, but it is recommended to create non-static classes[^4], if only because it's good practice, and to encourage test inheritance and reuse.
[^1]: Naming standards for unit tests
[^2]: Structuring unit tests
[^3]: You are naming your tests wrong!
[^4]: Static or non static test fixture in NUnit?


Patch 1

Correction of config not taken into account and also cleanly stop the server (saving)

@Naarin thanks for the hint on the servermap, i fixed that in 1.2 + added the quit on exit ( i didn't see your pull request until i added it, but you came up with that first, so all credit goes to you <3.


Handle updates in Rocket web framework

I am not using handle_updates function, and I receive update using webhook like this (it is rocker framework) :
#[post("/bot/telegram/test-dev", format = "application/json", data = "<update>")]
pub fn telegram_webhook(update: Json<tgbot::types::Update>, db_conn: RustyDbConn) -> String {
    "".into()
}

So the question is, how can I send one reply or more? And not only reply, sometimes bot need send notifications.. I am trying code:
let method = SendMessage::new(chat_id, "Test msg:");
tokio::run(api.clone().execute(method).then(|x| {
    Ok::<(), ()>(())
}));

and it works. But code after tokio::run is unreachable.
Looks like long polling is turining on? I am not sure..
So, the question is how can I send messages not using tgbot's handlers? Forward thank you!

Hi! Unfortunately I have no idea why it is unreachable. @mexus can you help?
Also I guess it's better to send update to a thread where you can spawn a future. Same for notifications. So you don't need to create a tokio runtime on every request which is a bit expensive.

Hi! Unfortunately I have no idea why it is unreachable. @mexus can you help?

I have simplified an example:
extern crate futures;
extern crate tgbot;
extern crate tokio;

use futures::Future;
use tgbot::{
    Api,
    Config, methods::SendMessage,
};

pub fn get_telegram_api() -> Api {
    let token = "_________YOUR_TELEGRAM_TOKEN_________".to_string();
    let config = Config::new(token);
    Api::new(config).expect("Failed to create API")
}

fn main() {
    let api = get_telegram_api();

    let method = SendMessage::new(1, "Test");
    let f = api.clone().execute(method).then(|x| {
        println!("{:?}", x);
        Ok::<(), ()>(())
    });
    tokio::run(f);

    println!("This code is unreachable");
}

Read it like it is the case when bot want send a notification (not a reply)
As I understand, tokio::run blocks the rest of thread. But unfortunately api::spawn  or tokio::spawn instead of tokio::run not works. And my_future:wait() not working also
P. S. this example works, but program doesn't shut down after sending a message, either println unreachable

Try block_on_all: https://docs.rs/tokio/0.1.22/tokio/runtime/current_thread/fn.block_on_all.html

I've added an example for notifications:
https://github.com/tg-rs/tg-rs/blob/cd33892ee3419ed6bfbdbdcefac2103af045f349/tgbot/examples/notify.rs
In Rocket you can do something like this: tx.send(Notification::Update(update))?;

Note that rx.recv() is blocking. You can write your own stream for this.
https://docs.rs/futures/0.1.28/futures/stream/trait.Stream.html
https://doc.rust-lang.org/stable/std/sync/mpsc/struct.Receiver.html#method.try_recv
https://doc.rust-lang.org/stable/std/sync/mpsc/enum.TryRecvError.html

Try block_on_all: https://docs.rs/tokio/0.1.22/tokio/runtime/current_thread/fn.block_on_all.html
Yes, not sure about block_on_all, but block_on works for me!))

use tokio::runtime::current_thread::Runtime;
let mut rt = Runtime::new().unwrap();
rt.block_on(future);
println!("String will be printed!!");

I've added an example for notifications:
https://github.com/tg-rs/tg-rs/blob/cd33892ee3419ed6bfbdbdcefac2103af045f349/tgbot/examples/notify.rs
In Rocket you can do something like this: tx.send(Notification::Update(update))?;

This example also looks nice!) with channel, really good one.
The only thing it require some work to implement it like fairings or like request guards to have copy of tx in route. but any way it is already much possible. And example suits great, works, and have correct logic!)


use markdown-doclet

I loved being able to write doc comments in Markdown in Ceylon, and it's such a bummer to go back to the awful quasi-HTML-ish format that's native to Java.
Fortunately, someone has written a doclet for this, so let's try it out.

For info, we also experimented and used the asciidoc doclet in Quarkus. I forgot if that’s still in place or if we gave up @gsmet
I remember I liked it

I've already written most of the API doc in Javadoc now, and can't be bothered rewriting it. Closing this issue.


running the debugger on an open SOMns file now runs SOMns with that file

sorry for the whitespaces, and I hadn't noticed the import. everything should be fixed now!


Add support for Directory.Packages.props file as entrypoint

This PR will enable dependabot to use a Directory.Packages.props file as entry point, allowing the setup below:
Repo:
src/
├─ Backend/
│  ├─ Project1/
│  │  ├─ Project1.csproj
│  ├─ Project2/
│  │  ├─ Project2.csproj
│  ├─ Directory.Packages.props
├─ App/
├─ Solution.sln

dependabot.yml:

version: 2
updates:
  - package-ecosystem: "nuget"
    directory: "/src/Backend"
    schedule:
      interval: "daily"
    open-pull-requests-limit: 5
    target-branch: master

Note: I have never written ruby before and I'm a bit out of my depth
This is working locally towards our repo, but I'm guessing there is a bunch of things I am missing.
I'm happy to get any feedback

Please add tests for this change

The issue is discussed in the issue: https://github.com/dependabot/dependabot-core/issues/5635
I will look into adding tests!
On another note, as seen in the PR two files have identical functions, is this intentional?
Should the function "project_files" be broken out?

The issue is discussed in the issue: https://github.com/dependabot/dependabot-core/issues/5635
I will look into adding tests!
On another note, as seen in the PR two files have identical functions, is this intentional?
Should the function "project_files" be broken out?

Refactoring it sounds reasonable to me

I now added some tests.
I used this repo to create the fixtures
The repo has drifted from what the PR originally described and looks more like this:
[tobias@tobias-expertbook nuget-directory-packages-props-example]$ tree -P "*.sln|*.csproj|Directory.Packages.props" --prune
.
├── App1
│   ├── App1.sln
│   └── src
│       └── Backend
│           ├── Project1
│           │   └── Project1.csproj
│           └── Project2
│               └── Project2.csproj
└── Directory.Packages.props

The intended dependabot.yml would look like this:
version: 2
updates:
  - package-ecosystem: "nuget"
    directory: "/"
    schedule:
      interval: "daily"
    open-pull-requests-limit: 5
    target-branch: master

The reason for the drift is entirely selfish, it is more like our private repository. Still, the repo in the PR should also work.
For testing I added

Simple test for getting only directory.packages.props if it is found in the root
Test to make sure the new code does not override the old logic for sln and all files are still fetched.

I wanted to do some refactoring but quickly felt out of my depth so I fell back onto doing the least work possible.
Happy to hear any feedback, in particular it would be helpful if we can figure out more tests cases.

@TobiasLaving just to clarify, did you see 2⃣ in my comment above? Can you add those two simple tests?

@TobiasLaving What is the status on this?
@jeffwidman How long we should expect this PR to hold of for https://github.com/dependabot/dependabot-core/pull/6031

@DkSkydancer This PR is ready in my opinion

@TobiasLaving  @jeffwidman  ping. we are still waiting for this!

Seems @jeffwidman is out (?)
Do you know the next steps @yeikel ?

Hey folks, Dependabot maintainers here - we've had a bit of a deploy freeze and have fallen behind on reviews. Sorry about that!
I took a look at the changes and I agree that this PR is in a good state. We should be able to take a look at merging it some time this week.
In the meantime, would you be able to bring it up-to-date with the base branch @TobiasLaving? The CI tests should be able to run now

@TobiasLaving Can you the update as requested above pl?

@DkSkydancer @Nishnha
PR is now rebased and all checks has passed :+1:

@Nishnha the PR should be ready now

@Nishnha  @jeffwidman Can we get a maintainer review please?

@TobiasLaving it looks like this branch is out of date again I'm afraid :(

Thank you for this contribution! I left a comment on the regex capture that I would like to see addressed before approving.
We can bring this PR up-to-date before we merge it, so it's fine that it has become out-of-date with the base branch.
@jeffwidman should we fork https://github.com/TobiasLaving/nuget-directory-packages-props-example into the dependabot org since it's referenced in the specs?

Good points, I have made the changes locally sadly the dev container seems to be broken for nuget atm so I cannot run it locally: https://github.com/dependabot/dependabot-core/issues/7438
Will push once I can validate locally

@Nishnha
Fixed your comments, thanks for finding and fixing the docker issue!

@Nishnha There, the linter is now happy :+1:
Struggeled a little bit with running the lint script in the dev container, sorry about the delay

@Nishnha
For my understanding, who does the actual merge?

We have a deploy-then-merge process so it's not as straightforward as pressing the merge button. That said, this PR has been merged now 😄
Thank you for the contribution!

@Nishnha hi is there please any way of tracking when this will go to full release across Github?


Issues with tooling on windows 10

Hi
I just tried Dart for the first time.
I have 2 major issues which are stopping me from even starting out;

When i try pub global activate angel_cli, the process gets stuck at Installed executable angel..
I am assuming that Angel was indeed instaled but i am not sure if any further steps were not performed.

aqueduct create is timing out. I have tried it with different networks so it doesnt seems to be a network issue.

PS C:\Users\Mockingbird\DevSpace\Dart> aqueduct create testproj
-- Aqueduct CLI Version: 2.3.0
    Aqueduct directory is: C:\Users\Mockingbird\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\aqueduct-2.3.0
    Template source is: C:\Users\Mockingbird\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\aqueduct-2.3.0\example\te
mplates\default\
    See more templates with 'aqueduct create list-templates'
-- Copying template files to new project directory (C:\Users\Mockingbird\DevSpace\Dart\testproj)...
    Copying contents of C:\Users\Mockingbird\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\aqueduct-2.3.0\example\te
mplates\default\.analysis_options
    Copying contents of C:\Users\Mockingbird\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\aqueduct-2.3.0\example\te
mplates\default\.gitignore
    Copying contents of C:\Users\Mockingbird\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\aqueduct-2.3.0\example\te
mplates\default\.travis.yml
    Copying contents of C:\Users\Mockingbird\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\aqueduct-2.3.0\example\te
mplates\default\bin
    Copying contents of C:\Users\Mockingbird\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\aqueduct-2.3.0\example\te
mplates\default\config.src.yaml
    Copying contents of C:\Users\Mockingbird\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\aqueduct-2.3.0\example\te
mplates\default\lib
    Copying contents of C:\Users\Mockingbird\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\aqueduct-2.3.0\example\te
mplates\default\pubspec.yaml
    Copying contents of C:\Users\Mockingbird\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\aqueduct-2.3.0\example\te
mplates\default\README.md
    Copying contents of C:\Users\Mockingbird\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\aqueduct-2.3.0\example\te
mplates\default\test
    Generating config.yaml from config.src.yaml.
-- Fetching project dependencies (pub get --no-packages-dir )...
*** Timed out fetching dependencies. Reconnect to the internet or use --offline.
*** TimeoutException after 0:00:20.000000: Future not completed

I am on windows 10, 64 bits and have installed Aqueduct version 2.3.0
Dart is Dart VM version: 1.24.2 (Thu Jun 22 08:55:56 2017) on "windows_x64"

It's timing out on pub get, which is the Dart command to fetch dependencies. What happens when you run pub get in that project directory?

On Mac i found same error.
If i do pub get inside the folder
I got Got Dependencies!

👍 The timeout is too short, will increase and not fail the command.


Fail to run trex gui on Ubuntu 18.04

When I try:
java -jar trex-stateless-gui.jar
I get:
Error: Could not find or load main class com.exalttech.trex.application.TrexApp
Caused by: java.lang.NoClassDefFoundError: javafx/application/Application
My Java is set as following - java --version:
java 12.0.1 2019-04-16
Java(TM) SE Runtime Environment (build 12.0.1+12)
Java HotSpot(TM) 64-Bit Server VM (build 12.0.1+12, mixed mode, sharing)
What am I missing?

Hi, You should use older java version
egor@blagov-home:~$ java -version
java version "1.8.0_201"
Java(TM) SE Runtime Environment (build 1.8.0_201-b09)
Java HotSpot(TM) 64-Bit Server VM (build 25.201-b09, mixed mode)

OK, Oracle JDK 8 did work fine.
Maybe its worth to update the error message.
Currently it says: "Please download and install latest Oracle Java Runtime environment(JRE)"
While it should say something like  "Please download and install Oracle Java 8 Runtime environment (JRE)"
Anyway, thanks!

OK, Oracle JDK 8 did work fine.
Maybe its worth to update the error message.
Currently it says: "Please download and install latest Oracle Java Runtime environment(JRE)"
While it should say something like "Please download and install Oracle Java 8 Runtime environment (JRE)"
Anyway, thanks!

Thanks you for your input, will do that.

Hi, error message is edited in v4.5.5


[snapshot] Update fastlane_core dependency

👍

I'm single and ready to bring me along with you and your thighs pumped for pleasure in the area of reality of finding love! Maybe you can get it together for the rest of my efforts in temporary basis of common interest to stand up and get a want and need spotted with the best eyes are handed down from ww2 squadron leader in the raaf flying in newly formed mans design of a great way to get a follow up, pa the fish sucks that closely guarded memories are not a pretend threat life and civilian casualties were caused by the greedy and grandiose thought a bit more than too much for anyone. Doug was Adelaide's smartest student and dreams of being a potential pm! Wasn't a little bit more than a dream! Then taking my pas life into jeopardy and shot out of the sky casualties 2 crashes behind enemy lines and the last time he saw very much a mate co polite good bye to two Austalian life's lost forever and always will be in our hearts. To salvage humanitarian assistance from across the globe are not going to only trouble with the best part of four years flying planes wiped out cities out of rage and end the reports that the best way for me to swear in alliance between us men are that the man showed love you so much better for the hero of mine and family anywhere else


select updates

#676

@MikeTschudi console.logs are again to try and understand something that only seems to occur on devext


Reporting for OS version for Windows and OS X by overriding the user …

…agent

Implemented using a mixture of code from the following sources

https://github.com/googleanalytics/google-analytics-plugin-for-unity/pull/78
1.1 For user agent override
https://github.com/googleanalytics/google-analytics-plugin-for-unity/issues/94
2.1 For the user agent string for Windows and OS X.
2.2 Includes a bit of extra code to fix a bug in Unity 4.6 where it reports Windows 10 as Windows 8.1 (6.3.10XXX), where the build number is correct

I've just signed the CLA

I've just signed the CLA
On Thursday, 14 April 2016, 17:04, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). Please visit https://cla.developers.google.com/ to sign.Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
If you signed the CLA as a corporation, please let us know the company's name.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub


fix: connections and threads failing to close

Hi there, great library, thanks for developing.
I noticed that connections were staying open even after closing them. Preventing threads and connections from properly ending.
It would appear the connection close function is setting the ready state to closed after emitting the close event.
https://github.com/Bugs5382/node-hl7-client/blob/51ae5bd58d98287189b875f83bfd05f2ea047175/src/client/connection.ts#L150-L152
This causes a failure on the listener trigger by the close event
https://github.com/Bugs5382/node-hl7-client/blob/51ae5bd58d98287189b875f83bfd05f2ea047175/src/client/connection.ts#L334-L338
It will always run into this exception state as the readystate has already been set to closed.
Having tested locally a fix would be to remove the line
https://github.com/Bugs5382/node-hl7-client/blob/51ae5bd58d98287189b875f83bfd05f2ea047175/src/client/connection.ts#L152
Even with this fix the process thread is kept open until connectiontimer timeout is next triggered. Root cause is here
Adding the following clearTimeout(this._connectionTimer); just before or after the below line would resolve this issue
https://github.com/Bugs5382/node-hl7-client/blob/51ae5bd58d98287189b875f83bfd05f2ea047175/src/client/connection.ts#L150
With these two fixes the connections successfully close and the process thread terminates.

@seanjb Do you want to submit a PR against the develop branch?

 this._readyState = ReadyState.CLOSED

It will have to stay in the metood of 'close()' method is called again, I don't see why it would, but just in case, it just stops the whole method from running. On the 'checkSend()' inside the 'sendMessage()' would sometimes fail if this is not set in close.
https://github.com/Bugs5382/node-hl7-client/blob/9d9f2965906417b4027640950e63d8d9ca7f5314/src/client/connection.ts#L150-L154

:tada: This issue has been resolved in version 2.1.1-beta.3 :tada:
The release is available on:

npm package (@develop dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@seanjb - Please try the beta version above before I release it to the wild.

Thanks @Bugs5382 this seems to have resolved the second part of the issue, however the first part still persists. The race condition.
Coming into the socket.on('close') method the connection ready state is already set to closed.
https://github.com/Bugs5382/node-hl7-client/blob/9d9f2965906417b4027640950e63d8d9ca7f5314/src/client/connection.ts#L336-L340
Having debugged the code I can see the below line is reached before the above method.
https://github.com/Bugs5382/node-hl7-client/blob/9d9f2965906417b4027640950e63d8d9ca7f5314/src/client/connection.ts#L154
Causing a 'Socket closed unexpectedly by server' error.
So perhaps a fix might be to add an extra check into the first if conditional e.g.
(this._readyState === ReadyState.CLOSING || this._readyState === ReadyState.CLOSED) or this._readyState === ReadyState.CLOSING || !this._connectionTimer
Not sure of any other functionality this could break.

@seanjb Looking into it.

I think...
this._readyState === ReadyState.CLOSING || !this._connectionTimer

It will be the better choice. No connection timmer means it should have (in theory) already been closed manually—I plan on releasing a new build tomorrow. I have a test script that runs for about 24 hours with about a million hl7 messages for testing and simulated disconnects, etc., to make sure.

@seanjb If you want to give it a shot, there's a new branch running with this code. As I said above, I have local testing going on right now. By the time I get up in the morning, I should have the results.

#106 PR

:tada: This issue has been resolved in version 2.2.0-beta.1 :tada:
The release is available on:

npm package (@develop dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.3.0-beta.1 :tada:
The release is available on:

npm package (@develop dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@seanjb Will be released into the wild by the end of the day. My massive testing didn't find any issues.

:tada: This issue has been resolved in version 2.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Sorry @Bugs5382 based in Sydney so didnt get a chance to test the beta when it became available. Have just checked 2.3.0 now and still have the issue.
It looks like when clearing the timeout the this._connectionTimer is not set to null. So my suggested fix was insufficient (sorry about that). A better fix is as follows to check the connection timer if it exists has been destroyed - which is true if the cleartimeout method was invoked:
(this._readyState === ReadyState.CLOSING || this._readyState === ReadyState.CLOSED) or this._readyState === ReadyState.CLOSING || (this._connectionTimer == null)) || || (this._connectionTimer?.destroyed)) )
Tested the above through debugging and works.
It may also be safer to clear the timeout before emitting the close event also. However my testing did not reveal this to have any impact if it is before or after the emit.

@seanjb Can you submit a 2nd PR?

:tada: This issue has been resolved in version 2.3.1-beta.1 :tada:
The release is available on:

npm package (@develop dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@seanjb Added to develop. Use the developed version for now for testing. Tomorrow (25th my time) when I wake up, I'll check to see if everything is good. Thanks for your help, and be sure to ⭐  it up!  👍

Cheers - just tested and Beta confirmed working

Ok. I know I am up.. hah... but I have a few more things to merge into main tomorrow. So just going to hold off.

:tada: This issue has been resolved in version 2.3.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Implement Easter calculation like dateutil?

Will support foe calculating Easter eventually be added similar to the functionality of date-util?

Since we already require dateutil to be present maybe it could be as simple as making the easter module available from there.


Could not update the profile

when i try to edit my profile photo.it said:"Could not update the profile"
Smartphone
Device: huawei Honor V10
OS: Android 8.0.0
RAM: 6 GB

@Shadyzpop I can't reproduce this.

Lets wait for the issue owner otherwise we close it in 7 days

@alexsirbu04 @Shadyzpop @ChristBKK  how to help ? the problem still there

@sysulsb We will further test this. Thank you!


Allow node-sass instead of sass for legacy apps

Clear and concise description of the problem
My company has dozens of apps that area all unfortunately using legacy versions of SASS via node-sass, importing this (recently) outdated scss from third-party modules. This means that I get compilation/syntax errors when trying to use Vite because vite only works with sass.
As it stands that means my dreams of moving builds over to Vite are DOA, unfortunately.
Suggested solution
Allow users to opt into legacy sass transformations using node-sass to give companies like mine an opportunity to use Vite and to (hopefully, someday, probably not) update our Sass files.
In general, it could be good if these sorts of pre-processors were completely configurable, because what happens when someone writes the much-better "rust-sass" or whatever, and people want to switch?
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

As @Niputi already wrote in Discord, I also think you are really on your own with this legacy dependency and I think you will not get support in Vite core. So maybe you could try to reach something with a plugin.
But for now Vite (in core) will not support such legacy dependencies. Vite is aiming for modern and fast code. If you are really stuck with such old deps, you may want to use something like webpack, cause there you have much more configurability but sadly not that fast dev-server speed.


Adding IIIF view of Solr record

Closes #58
@tampakis Here is a pretty simple implementation of generating IIIF-style JSON from the Solr doc.  This doesn't handle some of the complications we've talked about with multiple titles in different languages.  Want to point me to some sample records? Or punt on that for another ticket?

I've updated this PR to change how authors are handled, to just have creator and contributors.
So far, there isn't any attempt to figure out the language/script of the values.  I've got some sample data to look at, so I'll see what I can come up with tomorrow morning.  I'm fine including that in this PR or doing that in a separate issue.

I think it makes sense doing the language script stuff in a separate issue.


add attribute toclass gKDR for obtaining the eigenvalues

One line change
Add attribute to the gKDR class for obtaining the eigenvalues.

Codecov Report

Merging #251 (5fd4a80) into devel (e56d167) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##            devel     #251   +/-   ##
=======================================
  Coverage   76.77%   76.78%
=======================================
  Files          55       55
  Lines        9491     9492    +1
=======================================
+ Hits         7287     7288    +1
  Misses       2204     2204

Files Changed
Coverage Δ

mogp_emulator/DimensionReduction.py
90.38% <100.00%> (+0.09%)
:arrow_up:

Thanks @yyimingucl, merged into devel


cryptomator: 1.11.0 -> 1.11.1

Description of changes
Update to latest version: https://github.com/cryptomator/cryptomator/releases/tag/1.11.1

Things done

Built on platform(s)

[x] 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
[x] 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.

Closing in favor of https://github.com/NixOS/nixpkgs/pull/279754


Persistence system

The Show, UnShow and Persist system. This will be an ongoing issue.

This will be closed work will continue in #174.


How to get DateTimeOffset from DateTime object when importing a CliXml file

I have a script that collects a lot of data from remote machines in different time zones.  I serialize all of the data using Export-CliXml, and then deserialize the data using Import-CliXml to parse it into a SQL database.
I'd like to be able to convert the DateTime objects I collect into DateTimeOffset objects, in order to store the DateTime in the machine's local time.  Unfortunately, it looks like the offset information is lost during the deserialization.  I know I could get the content of the CliXml file and convert the string to a DateTimeOffset, but I'm wondering if there's any way to preserve the information when using Import-CliXml.  I know I could also use Get-TimeZone to get the offset information, but I'm wondering if there's a way I can do this without collecting it.
Thank you!
Example: Converting string from CliXml file to DateTimeOffset

Example: Importing CliXml and converting to DateTimeOffset does not preserve TimeZone offset

Can you please explain this issue a little more clearly???
In the original XML file, the date is stored in UTC time, so the time is displayed as 13:13 with an offset of 0.
After the import, the date is stored in Central Time with an offset of -5 hrs.  That's why it is showing up as 8:13.

I want to preserve the offset of the DateTime as it's stored in the XML file, so I can convert the DateTime back to the local time it was created.
The offset is stored in the XML file, and I can grab the string out of the XML file's contents and convert it to a DateTimeOffset to get that local time with the correct offset.  When I use Import-CliXml though, the offset is lost during the deserialization back to DateTime, and gets automatically converted to my machine's local time.
I sort of expected the answer to be "get the timezone offset and calculate the local time", but I was hoping there might be some way to preserve the offset during deserialization without needing to collect it.

Not within DateTime's type structure, no. It's not designed for it. Had it been originally stored as DateTimeOffset, I think this might be possible, however. 🤔

Here's a different example using Alaska time (-8):
Manual conversion of the string:

Importing the CliXml

I'm probably just going to end up collecting it as a DateTimeOffset.  I was just hoping I didn't have to :)

it looks like the offset information is lost during the deserialization

. ----no
centos7 + ps 6.21
$a = (get-date) -as [datetimeoffset]
Export-Clixml -InputObject $a -LiteralPath /tmp/abc.xml

$b = import-Clixml -LiteralPath /tmp/abc.xml
$b.gettype().fullname
$b | gm
$b.Offset

Yes, if you initially convert to [datetimeoffset] and serialize that you will retain the information.
@tloveland1 was serializing [datetime] and attempting to rehydrate it as [datetimeoffset] in order to maintain the offset information.


Replaced HandleQueueTable with DeleteButton

DeleteButton component will make it so the whole table doesn't rerender on toggle, just the components
Added getAllUsers endpoint to QueueTableDelete
Added handleNextUser to queueDashboard

@jchen42703


Google Drive: Prevent downlading very big files

Description
Add a check on the file size of gdrive documents before downloading them.
This is to prevent the pod to be OOMKilled when trying to download a very big file.
Risk
Can be rolled back.
Deploy Plan
Deploy connectors.

Cool thanks @PopDaph !


[1.21] Add COMPLIANCE_POLICY to Istio

Backport of https://github.com/istio/istio/pull/49385

hold now, will merge after 1.21.0 released.


How to catch this errors

I use Brotli and when for some reason container not loaded I want catch erros like this
Uncaught SyntaxError: Invalid or unexpected token
build.loader.js:1 Uncaught ReferenceError: unityFramework is not defined
at HTMLScriptElement.r.onload (build.loader.js:1)
to render another content
unityContext.on('error' ... not catching this errors and unityContext.on('progress' stuck on 0.9.
Can I some how catch it with unityContext or i must check another services for this?

Hi! If you're still running into this problem, could you try again using the latest version? I've recently implemented a wider error boundary.


HBASE-25676 Move generic classes from hbase-server to hbase-common

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 24s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2 Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 57s
branch-2 passed

+1 :green_heart:
compile
4m  5s
branch-2 passed

+1 :green_heart:
checkstyle
1m 43s
branch-2 passed

+1 :green_heart:
spotbugs
2m 58s
branch-2 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 11s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 41s
the patch passed

+1 :green_heart:
compile
4m  6s
the patch passed

-0 :warning:
javac
0m 48s
hbase-common generated 9 new + 159 unchanged - 0 fixed = 168 total (was 159)

-0 :warning:
checkstyle
0m 25s
hbase-common: The patch generated 41 new + 0 unchanged - 0 fixed = 41 total (was 0)

-0 :warning:
checkstyle
1m 14s
hbase-server: The patch generated 1 new + 52 unchanged - 58 fixed = 53 total (was 110)

-0 :warning:
whitespace
0m  0s
The patch has 11 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

+1 :green_heart:
hadoopcheck
12m 54s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
3m 15s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 23s
The patch does not generate ASF License warnings.

49m  1s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3060/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3060

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux c229b3647674 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 72496272aa

Default Java
AdoptOpenJDK-1.8.0_282-b08

javac
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3060/1/artifact/yetus-general-check/output/diff-compile-javac-hbase-common.txt

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3060/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-common.txt

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3060/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

whitespace
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3060/1/artifact/yetus-general-check/output/whitespace-eol.txt

Max. process+thread count
86 (vs. ulimit of 12500)

modules
C: hbase-common hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3060/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 17s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 40s
branch-2 passed

+1 :green_heart:
compile
1m 22s
branch-2 passed

+1 :green_heart:
shadedjars
6m  5s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 58s
branch-2 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 18s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 21s
the patch passed

+1 :green_heart:
compile
1m 23s
the patch passed

+1 :green_heart:
javac
1m 23s
the patch passed

+1 :green_heart:
shadedjars
6m  0s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 58s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 38s
hbase-common in the patch passed.

+1 :green_heart:
unit
147m 20s
hbase-server in the patch passed.

177m 29s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3060/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3060

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 6e015dd68137 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 72496272aa

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3060/1/testReport/

Max. process+thread count
3783 (vs. ulimit of 12500)

modules
C: hbase-common hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3060/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 42s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+0 :ok:
mvndep
0m 18s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
5m  2s
branch-2 passed

+1 :green_heart:
compile
1m 44s
branch-2 passed

+1 :green_heart:
shadedjars
7m 32s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 11s
branch-2 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 18s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 43s
the patch passed

+1 :green_heart:
compile
1m 52s
the patch passed

+1 :green_heart:
javac
1m 52s
the patch passed

+1 :green_heart:
shadedjars
8m  1s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 12s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
2m 42s
hbase-common in the patch passed.

+1 :green_heart:
unit
199m 50s
hbase-server in the patch passed.

238m 27s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3060/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3060

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 6f946e259e32 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 72496272aa

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3060/1/testReport/

Max. process+thread count
2648 (vs. ulimit of 12500)

modules
C: hbase-common hbase-server U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3060/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

The purpose here is to make hbase-server module smaller? That's good I think.
But considering from another dimension, if the classes are only used by hbase-server, we'd better put them in hbase-server. If we want to use them in other modules like hbase-http, then we move them to a more generic module like hbase-common.
What do you think sir?

`if the classes are only used by hbase-server, we'd better put them in hbase-server. If we want to use them in other modules like hbase-http, then we move them to a more generic module like hbase-common.
Agree. Let me review these moves again w/ above in mind; I was all about shrinking size in previous refactor
`

Let me just close this. Most of the classes here are used by hbase-server only. Even though a bunch of theses classes are generic and could be used elsewhere other than by hbase-server AND even though a good portion of the content of hbase-common is currently only used by hbase-server, lets favor coherent, contained modules.


[Feature Request] Pipelines/Jobs integration

Description
Pipelines/Jobs integration to check on jobs status and logs, also run/cancel/retry pipelines/jobs.
Also for checking jogs logs
API:
https://docs.gitlab.com/ee/api/pipelines.html
https://docs.gitlab.com/ee/api/jobs.html

Hi,
Thank you for the suggestion. But unfortunately due to my workload I'm unable to implement new requirements at this time. I'm sorry about that but I would appreciate it if you implement this feature into the app. Since it's open source, anyone can contribute.
In case of any questions, let me know.
Thank you for understanding

I would, but my programing skills aren't good.
Maybe someone see this issue untie future and help with it.

I'm starting to implement new Features over at thelooter/gitplus_for_gitlab since this Repository seems to be dead. If you still have a need for this feature, feel free to raise a issue over there

@Generator initial support for pipelines landed in the most recent release https://github.com/thelooter/labplus_for_gitlab


Debugging output for COLMAP

Currently, COLMAP failures are hard to debug since running the script fails to provide output on each step in COLMAP's running.
To mitigate this the following should be implemented:
1.) Detailed printouts should happen on each step of the COLMAP worker's execution aswell as the ffmpeg.
2.) Avoid COLMAP's output dumping to the terminal: COLMAP's output should be piped to an output file and saved locally under each job id to the worker for debugging purposes (maybe a flag to enable this as well?).
3.) Automatically run the to_cam.py and save the resulting figure (hopefully in a way that allows interactive rendering rather than a png) to the job id folder.
4.) Use the extrinsic matrices generated to print out general statistics about the camera locations (After NDC and all coordinate conversions). Use the formula below to get the camera point and run basic statistics like min max for each coordinate, average point across all cameras, maybe standard deviation across each axis x,y,z, and anything else that may be useful.

^ source https://en.wikipedia.org/wiki/Camera_resectioning

Finished


Mixin applications should consistently {,not }have implements clause

The dartk and rastak compilers handle mixins differently in terms of the implements clause.
class Base { }
class Mixin { }
class Foo extends Base with Mixin {}
main() {
  print(new Foo());
}

dartk:
  class Base extends core::Object {
    constructor •() → void
      : super core::Object::•()
      ;
  }
  class Mixin extends core::Object {
    constructor •() → void
      : super core::Object::•()
      ;
  }
  class Foo extends mix::#class1 {
    constructor •() → void
      : super mix::Base::•()
      ;
  }
  abstract mixin #class1 = mix::Base with mix::Mixin;
  static method main() → dynamic {
    core::print(new mix::Foo::•());
  }

rastak:
  class Base extends core::Object {
    constructor •() → mix::Base
      : super core::Object::•();
  }
  class Mixin extends core::Object {
    constructor •() → mix::Mixin
      : super core::Object::•();
  }
  class Foo extends mix::#class1 {
    constructor •() → mix::Foo
      : super mix::#class1::•();
  }
  abstract mixin #class1 = mix::Base with mix::Mixin implements mix::Mixin {
    constructor •() → mix::Base
      : super mix::Base::•();
  }
  static method main() → dynamic {
    core::print(new mix::Foo::•());
  }

/cc @ahe @asgerf  Do we think adding the implements clause is the right thing to do?

I don't see a compelling reason to put the mixed-in type in the 'implements' clause.
Can we close this issue or does anyone want to act on it?


How to get real playlist when player is in shuffled mode

i want to snap left and right to switch prev/next song，i can't get real playlist in shuffled mode , so it  don't know how to handle it
when snapping i need display prev/next song's name, but i only have playlist in repeated mode, cause display song's name is not  right when snapping, after snapping finish  i use skipToNextMediaItem/skipToPreviousMediaItem to switch song.

Thanks for your question.
You can iterate over the timeline and say whether to use shuffling mode or not. This works in the very same way except the boolean that indicates whether to apply shuffling or not:

    val window = Timeline.Window()
    var windowIndex = timeline.getFirstWindowIndex(true)

    // iterate with shuffle mode off
    while (windowIndex != C.INDEX_UNSET) {
      // do something with the media item
      timeline.getWindow(windowIndex, window).mediaItem
      // next window index without shuffling
      windowIndex =
        timeline.getNextWindowIndex(
          windowIndex,
          Player.REPEAT_MODE_OFF,
          /* shuffleModeEnabled= */ false,
        )
    }

   // iterate with shuffle mode on
    while (windowIndex != C.INDEX_UNSET) {
      // do something with the media item
      timeline.getWindow(windowIndex, window).mediaItem
      // go to next window in shuffle mode
      windowIndex =
        timeline.getNextWindowIndex(
          windowIndex,
          Player.REPEAT_MODE_OFF,
          /* shuffleModeEnabled= */ true,
        )
    }

thx，i test it's good!，thx


Fix wait functionality in svcat register command

Never actually made this work before, this should fix that.

I think this is barfing on a data race that I've included (intentionally) to test the wait functionality. Is there some way to fix that?

/lgtm


Google CSE Unauthorized access to internal API

使用搜索时,显示
Unauthorized access to internal API. Please refer to https://support.google.com/customsearch/answer/4542055
并且在chrome64里进行搜索会提示不安全的脚本

hi


question: relevance scores in Qrels

Why is the relevance score mandatory in Qrels?
I don’t see where you use it to compute the metrics https://github.com/AmenRa/ranx/blob/master/ranx/metrics.py
Is there any way to make it optional? If not, would a filler value like 0 for all documents be appropriate?

Hi, if you only have a list of relevant document ids but no relevance judgment, you should use 1 as relevance judgment score.
0 relevance judgment scores are removed before computing metric scores to prevent miscomputations.
Among the implemented metrics only NDCG uses the relevance judgment scores. You can see it here.
By definition, all the other metrics consider relevance judgment scores to be binary even if they are not.

Got it, thanks !


cache factor levelCounts in MetaDataHandler

Description and related tickets, documents
cache factor levelCounts in MetaDataHandler so DDF don't have to recompute
Reviewers: @ubolonton
PR Progress
Make sure all checkboxes below are checked before merged

[ ] Branch is in format prefix/description (see this)
[ ] Merge check has no conflicts. PR checks passed.
[ ] Code review is done.

looks good.


fix: Monaco editor loader CDN js path

Description
The current CDN path seems not be able to reach( not sure for all region or just in China )

the loader's default  CDN path:

The fix is using an API provide in @monaco-editor/loader reset it.

Task Item
close #7965

Screenshots

Let's make sure all is working before we merge this, I remember we were using 0.19.3 (it's now updated to 0.21.2)

Let's make sure all is working before we merge this, I remember we were using 0.19.3 (it's now updated to 0.21.2)

The reason of upgrading is we need the integrated @monaco-editor/loader from<EMAIL_ADDRESS>and it requires<EMAIL_ADDRESS>I reviewed the changelog, no breaking changes in our usage.
https://github.com/suren-atoyan/monaco-react/blob/master/CHANGELOG.md
https://github.com/microsoft/monaco-editor/blob/main/CHANGELOG.md
Sure, Let's wait for the customer's confirm.

Coverage decreased (-0.001%) to 50.361% when pulling 9c7c721fb122c3955787094ec9c9365236780931 on zhixzhan/monaco-loader-fix into 56801686c41a21630be2955a54b992dd59d8f91a on main.

How about we fix this issue by copy the load.js locally? A user also says he cannot access the cdn from a VM. https://github.com/microsoft/BotFramework-Composer/pull/5547

@zhixzhan let us know whenever you feel confident about this PR not breaking monaco API due to the version bump and we can merge it.

@hatpick I test more scenario on 'Property Editor', 'All up view', 'Form Editor', 'JSON Editor', all looks good. I think we are ok to merge it.


Update rancher pipeline images

@deniseschannon  I can update Minio. I don't know if we need to update pipeline-tools since I am not familiar with it. Kube-api-auth is updated with base image alpine:3.11.3 in this PR https://github.com/rancher/kube-api-auth/pull/7

@deniseschannon kube-api-auth image updated.


How to use this?

npx licence-report works perfectly, however additionally need licences of sub dependencies. I've tried a few variations to get this to run but not having much luck.
npx license-report-recursive --recurse
npx --package license-report-recursive -- license-report --recurse
npx --package=license-report-recursive -c 'license-report --recurse'
npx --package=license-report-recursive -c 'license-report --recurse --output=csv --csvHeaders'
Nothing outputs to stdout

oh, somehow I goofed the publishing to npm. Working on the problem and report here as soon as everything is ok again.

ok, I fixed.
The problem was that a one of the last actions, I added a super clever postinstall script that updates the version number of license-report in the readme.md file. But this script also runs, when you try to install 'license-report-recursive' or run it via npx. Ups 😃
So now I added a build script that creates a dist forlder used for publishing to npm. Here I only added files needed for using the package and dropped the scripts from the package.json. Now the package should run.

Thanks @BePo65. Without --recurse it now works. However, getting a callstack error now:
RangeError: Maximum call stack size exceeded
    at addDependenciesRecursive (file:///Users/.../.npm/_npx/4e25a78b136288d0/node_modules/license-report-recursive/lib/addDependenciesRecursive.js:53:34)
    at async addDependenciesRecursive (file:///Users/.../.npm/_npx/4e25a78b136288d0/node_modules/license-report-recursive/lib/addDependenciesRecursive.js:52:82)
    at async file:///Users/.../.npm/_npx/4e25a78b136288d0/node_modules/license-report-recursive/index.js:67:78

license-report v6 changed the way the license data is acquired and required a major rework of this program here.
Looks like I am not yet finished :-( I will try to solve this issue asap, but I am afraid it will take more than a few hours.

Fixed the problem by switching from recursion to loops and added tests for dependency loop. The new version (6.2.0) was also released on npmjs.
I hope this will make this little program usable for you.


[BUG] Add scripts to pull CSP prices

Describe the bug
Currently, for some CSPs like DB, the prices are static and hardcoded into the resources folder
We need to add scripts in resources/dev to parse the web-pages of DB/Azure to pull the price information.
Those scripts should be triggered:

prepackage manager needs to trigger those scripts to get the latest pricing information while building a single artifact
in the non-fat mode, the runtime should run those scripts and cache the output in the cache folder.

Pricing not supported anymore


Weird behavior that leads to a crash

Device Specifications:

Redmi Note 10 Pro (Android 13)
Windows 10.0.19044
Android Debug Bridge version 1.0.41 / Version 34.0.1-9979309
App Version: Beta 0.8.23050

Steps to Reproduce:

Connect to Android device and click 'Browse'

'Root' and 'Internal Storage' appears, but not the external SD card

Click on 'Internal Storage'

Click the back button

SD card appears, the app freezes and crashes eventually

https://github.com/Alex4SSB/ADB-Explorer/assets/55659978/5e4cb035-7f03-4094-9a51-cdb6ceb6c5cc

Ok, so I see a lot of weird things here:

The app was not able to get device name ("Redmi" etc.) from adb devices -l. It would help if you could give me the result you get (you can hide IP / mDNS ID).
The app was not able to get device Android version. It executes adb shell getprop, and searches for the key ro.build.version.release to get major Android version. It would help if you could find which key in the getprop contains the major Android version on your device.
The app was not able to get the drives on the first try - you can see that Root and Internal Storage are displayed without capacity information. It executes adb shell df, where the displayed drives can be found. It would help if you could find how the drives are displayed there. Usually Internal Storage drive is the /storage/emulated partition.
The real issue here seems to happen when defining the SD drive:

Drive name is not displayed yet - should be "µSD Card"
Drive icon is not displayed - should be \uE7F1 in Segoe MDL2 Assets, judging by your other icons
Drive usage percentage is not displayed, but that's probably because it was not loaded yet (the used size is displayed)

You can try to disable Polling for Drives in the app settings, but I'm not sure whether this will help.

Ok, so we'll have to pinpoint the issue.
I've compiled a version for you, where I disabled external drive type update.
I hope that will prevent the app from crashing for you.
ADB_Explorer_0.8.23050_x64_noDriveTypeUpdate.zip

I compiled another version, to help us better understand the underlying issue.
I added three buttons:

After browsing the device, click these buttons one at a time, a message will pop up with the result.
Each button represents a (possible) step during SD discovery.
Hopefully, the results will help us identify the problem.
ADB_Explorer_0.8.23050_x64_SD_debug.zip

Ok, this time it shouldn't crash, but instead show a message with the exception details when trying to execute getprop.
Please click the "prop" button after browsing the device. this time the message will also be automatically copied to clipboard.
ADB_Explorer_0.8.23050_x64_getprop_debug.zip

Well, I think I solved the problem, and the fix is now in the latest commit.
Now the app will not crash when receiving from getprop multiple entries with the same key, which seems to be the issue on your device. This should also display your Android version at the right bottom corner of the app.
You can try the fixed build:
ADB_Explorer_0.8.23050_x64_fixed_getprop.zip
I will soon release an official version, and you'll be able to delete this one and use the official one, whether from GitHub or Microsoft Store.
I will close the issue once you confirm the problem is fixed.

Fixed in c957f29b7c894e0d81170dcc42f71deafaed5d18


OTAUpdateManager

@ArduinoBot

Hello @raghulrajg. The automated system here only operates on open pull requests. Since your submission was already accepted and the pull request merged, the bot won't pay any more attention to this pull request.
But this human is happy to provide assistance. Was there something you needed from the bot?

Hello @raghulrajg. The automated system here only operates on open pull requests. Since your submission was already accepted and the pull request merged, the bot won't pay any more attention to this pull request.
But this human is happy to provide assistance. Was there something you needed from the bot?

I try find my library in Arduino library manager, i can't find out

The job that indexes the libraries runs hourly. After that, the update must propagate through the cache, which takes some time. We give a conservative estimate of availability within one day's time.
I see that the library is already available in my Library Manager:

Please check your Library Manager again.

Thank you for response, yes I find my library but Ardu-badge not be found https://www.ardu-badge.com/OTAUpdateManager

The job that indexes the libraries runs hourly. After that, the update must propagate through the cache, which takes some time. We give a conservative estimate of availability within one day's time.
I see that the library is already available in my Library Manager:

Please check your Library Manager again.

Thank you for your response, yes I find my library but Ardu-badge not be found https://www.ardu-badge.com/OTAUpdateManager

"ArduBadge" is a 3rd party service. Arduino does not have any involvement in the maintenance of that project so reports of problems with ArduBadge are off topic here.
Please report the problem via the issue tracker of the ArduBadge repository:
https://github.com/gilmaimon/ArduBadge/issues
I did a quick check and it looks like a long-standing problem. For example, the "EspNowCam" library was added to Library Manager 2024-01-29:
https://github.com/arduino/library-registry/commit/1058d61903363ed256371e9b55be847321cccc38
Yet it has not been picked up by ArduBadge:
https://www.ardu-badge.com/EspNowCam
Conversely, the "Logistics" library was added to Library Manager 2024-01-04:
https://github.com/arduino/library-registry/commit/3cde734c48fbd062a5118b7fac24b481dd014905
And it was picked up by ArduBadge:
https://www.ardu-badge.com/Logistic
I didn't investigate further, but this indicates the problem might have started sometime between 2024-01-04 and 2024-01-29. This matches with the timing of this report of previously submitted libraries which were picked up, but are now not receiving version updates: https://github.com/gilmaimon/ArduBadge/issues/28

@ArduinoBot

Hello @raghulrajg. The automated system here only operates on open pull requests. Since your submission was already accepted and the pull request merged, the bot won't pay any more attention to this pull request.
But this human is happy to provide assistance. Was there something you needed from the bot?


[kimchi][circuits] fix most clippy lints

ran cargo clippy in circuits/kimchi and fixed most of the links. The rest of the lints are a bit more subtle as they complain about functions that either take too many arguments or return too many values. I'm happy that clippy lints these, but we have to think a bit more about how to transform these to types.

Codecov Report

Merging #210 (81a61f9) into master (23dc19e) will increase coverage by 0.01%.
The diff coverage is 69.44%.

@@            Coverage Diff             @@
##           master     #210      +/-   ##
==========================================
+ Coverage   88.94%   88.95%   +0.01%
==========================================
  Files          74       74
  Lines       18237    18236       -1
==========================================
+ Hits        16220    16222       +2
+ Misses       2017     2014       -3

Impacted Files
Coverage Δ

circuits/kimchi/src/gates/endosclmul.rs
6.25% <0.00%> (ø)

circuits/kimchi/src/gates/varbasemul.rs
3.18% <0.00%> (ø)

circuits/kimchi/src/polynomials/endomul_scalar.rs
85.84% <91.30%> (+0.27%)
:arrow_up:

circuits/kimchi/src/expr.rs
81.30% <100.00%> (-0.04%)
:arrow_down:

circuits/kimchi/src/gate.rs
87.30% <100.00%> (ø)

circuits/kimchi/src/gates/generic.rs
89.18% <100.00%> (ø)

circuits/kimchi/src/gates/poseidon.rs
92.30% <100.00%> (ø)

circuits/kimchi/src/nolookup/constraints.rs
94.66% <100.00%> (-0.02%)
:arrow_down:

circuits/kimchi/src/polynomials/chacha.rs
95.84% <100.00%> (ø)

circuits/kimchi/src/polynomials/complete_add.rs
98.27% <100.00%> (ø)

... and 8 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 23dc19e...81a61f9. Read the comment docs.


Error parsing vsftpd logs when day of the month is a single digit

The parsing of vsftpd logs when the day of the month is a single digit is failing. This is because there is an extra space in the timestamp field, so the HTTPDERROR_DATE format does not work.
Here are syslog logs in rfc5424 format for testing purposes :
<13>1 2022-08-08T10:01:37.377+00:00 {"name":"test"} LOGSTASH - - - 2022-08-08T10:01:37.377Z {name=securecodebox-test} Mon Aug  8 10:01:37 2022 [pid 28368] [test] FAIL LOGIN: Client "::ffff:<IP_ADDRESS>"
<13>1 2022-08-08T10:01:37.377+00:00 {"name":"test"} LOGSTASH - - - 2022-08-08T10:01:37.377Z {name=securecodebox-test} Mon Aug 18 10:01:37 2022 [pid 28365] [test] FAIL LOGIN: Client "::ffff:<IP_ADDRESS>"

The fllowing output is returned with the command cscli explain --file output.txt --type vsftpd --verbose :
line: <13>1 2022-08-08T10:01:37.377+00:00 {"name":"securecodebox-test"} LOGSTASH - - - 2022-08-08T10:01:37.377Z {name=securecodebox-test} Mon Aug  8 10:01:37 2022 [pid 28368] [test] FAIL LOGIN: Client "::ffff:<IP_ADDRESS>"
        ├ s00-raw
        |       ├ 🟢 crowdsecurity/non-syslog (first_parser)
        |       └ 🔴 crowdsecurity/syslog-logs
        ├ s01-parse
        |       ├ 🔴 crowdsecurity/apache2-logs
        |       ├ 🔴 crowdsecurity/haproxy-logs
        |       ├ 🔴 crowdsecurity/nginx-logs
        |       ├ 🔴 crowdsecurity/sshd-logs
        |       └ 🔴 vsftpd-logs
        └-------- parser failure 🔴

line: <13>1 2022-08-08T10:01:37.377+00:00 {"name":"securecodebox-test"} LOGSTASH - - - 2022-08-08T10:01:37.377Z {name=securecodebox-test} Mon Aug 18 10:01:37 2022 [pid 28365] [test] FAIL LOGIN: Client "::ffff:<IP_ADDRESS>"
        ├ s00-raw
        |       ├ 🟢 crowdsecurity/non-syslog (first_parser)
        |       └ 🔴 crowdsecurity/syslog-logs
        ├ s01-parse
        |       ├ 🔴 crowdsecurity/apache2-logs
        |       ├ 🔴 crowdsecurity/haproxy-logs
        |       ├ 🔴 crowdsecurity/nginx-logs
        |       ├ 🔴 crowdsecurity/sshd-logs
        |       └ 🟢 vsftpd-logs (+7 ~2)
        |               └ update evt.Stage : s01-parse -> s02-enrich
        |               └ create evt.Parsed.source_ip : <IP_ADDRESS>
        |               └ create evt.Parsed.timestamp : Mon Aug 18 10:01:37 2022
        |               └ create evt.Parsed.user : test
        |               └ update evt.StrTime :  -> Mon Aug 18 10:01:37 2022
        |               └ create evt.Meta.log_type : ftp_failed_auth
        |               └ create evt.Meta.program : vsftpd
        |               └ create evt.Meta.source_ip : <IP_ADDRESS>
        |               └ create evt.Meta.user : test
        ├ s02-enrich
        |       ├ 🟢 crowdsecurity/dateparse-enrich (+2 ~1)
        |               ├ create evt.Enriched.MarshaledTime : 2022-08-18T10:01:37Z
        |               ├ update evt.MarshaledTime :  -> 2022-08-18T10:01:37Z
        |               ├ create evt.Meta.timestamp : 2022-08-18T10:01:37Z
        |       ├ 🟢 crowdsecurity/geoip-enrich (+9)
        |               ├ create evt.Enriched.ASNNumber : 0
        |               ├ create evt.Enriched.ASNOrg :
        |               ├ create evt.Enriched.ASNumber : 0
        |               ├ create evt.Enriched.IsInEU : false
        |               ├ create evt.Enriched.IsoCode :
        |               ├ create evt.Enriched.Latitude : 0.000000
        |               ├ create evt.Enriched.Longitude : 0.000000
        |               ├ create evt.Meta.ASNNumber : 0
        |               ├ create evt.Meta.IsInEU : false
        |       └ 🔴 crowdsecurity/http-logs
        ├-------- parser success 🟢
        ├ Scenarios
                └ 🟢 crowdsecurity/vsftpd-bf

In fact I'm facing the same issue with logstash. I ended up with this solution for the timestamp field :
# Use separate pattern to HTTPDERROR_DATE for single-digit day-in-month ("Jul  1")
CUSTOMTIMESTAMP %{DAY} %{MONTH}  %{MONTHDAY} %{TIME} %{YEAR}
(%{HTTPDERROR_DATE:timestamp}|%{CUSTOMTIMESTAMP:timestamp}) \[pid %{NUMBER}\] ...

You could also use an optionnal space:
CUSTOMTIMESTAMP %{DAY} %{MONTH} (?:\s?)%{MONTHDAY} %{TIME} %{YEAR}
(%{CUSTOMTIMESTAMP:timestamp}) \[pid %{NUMBER}\] ...

We will integrate this fix as soon as the 1.4.2 is live.
The reason being that "timestamp" will sometimes get an extra space and this could be problematic for replays among other things.
(spoiler alert)
1.4.2 will allow us to define a static: StrTimeFormat with a similar pattern as value, this way StrTime won't with this extra space won't be an issue.

We will integrate this fix as soon as the 1.4.2 is live.
The reason being that "timestamp" will sometimes get an extra space and this could be problematic for replays among other things.
(spoiler alert)
1.4.2 will allow us to define a static: StrTimeFormat with a similar pattern as value, this way StrTime with this extra space won't be problematic.


JBDS-4505 Fuse on Karaf does not have a public facing download URL

Codecov Report

Merging #831 into master will decrease coverage by 0.13%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #831      +/-   ##
==========================================
- Coverage   96.48%   96.35%   -0.14%
==========================================
  Files          35       35
  Lines        3187     3374     +187
  Branches      267      480     +213
==========================================
+ Hits         3075     3251     +176
- Misses        112      114       +2
- Partials        0        9       +9

Impacted Files
Coverage Δ

browser/model/jbossfusekaraf.js
95.5% <100%> (+1.12%)
:arrow_up:

browser/model/helpers/downloader.js
96.94% <0%> (-1.52%)
:arrow_down:

browser/model/jbosseap.js
98.8% <0%> (-1.2%)
:arrow_down:

browser/pages/about/controller.js
21.05% <0%> (-1.17%)
:arrow_down:

browser/directives/componentPanel.js
14.28% <0%> (-1.1%)
:arrow_down:

browser/pages/confirm/controller.js
89.71% <0%> (-0.93%)
:arrow_down:

browser/services/data.js
97.03% <0%> (-0.92%)
:arrow_down:

browser/model/jbossfuse.js
99.21% <0%> (-0.79%)
:arrow_down:

browser/model/cdk.js
99.22% <0%> (-0.78%)
:arrow_down:

browser/model/jdk-install.js
97.29% <0%> (-0.45%)
:arrow_down:

... and 22 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 2634a20...48ba934. Read the comment docs.


Setting the font-family is not explicit in documentation

It took a little time to figure out that the typeset() mixin does not actually set font-family. It would be nice to include a note in the documentation.
Here's an example:
h2 {
    font-family: font-family-of($sans);
    @include typeset($sans, 40px, 46px, 2, 2);
}

Fixed in 1.0.8
3253df3b7331129dc1d6e464e9d3b7e756f69e00


Fix issue where tooltip popups persist on mobile

John says we had a fix for this in the Carina mini, so apply it here too.


Switch to machine-parsable output

TL;DR
Currently, we rely on parsing the text output from the gcloud command to extract the URL and other outputs. This isn't guaranteed to be stable. We should instead use --format=json and parse the resulting output as JSON to extract desired values. This also would enable use to expose more outputs about the revision that we currently do not.
Detailed design
Append `--format=json` to all gcloud commands, parse output as JSON.

Additional information
No response

I am going to take a crack at this....


Fix / breadcrumb-admin-subpaths

fixes #2279

"fixes" comment needs to be in the description, not a comment.  I updated it.

This has been reverted.  It caused more issues (e.g. #2376)


Extend Usage to print values

This way Usage() can be used after processing to print applied configuration as a better-looking alternative to fmt.Printf("%+v\n", config).

This seems like a debugging use case? Applications may reasonably need to:

parse configuration from the environment
print helpful output documenting supported environment vars and their types

I'm trying to wrap my head around the need for this or find precedent anywhere else. It would be like if someapp -t -a -N 2 -h printed help output informing the user that they activated the "t" and "a" flags, and that the "N" value is 2.

I think @AlekSi might be trying to solve the problem of "What the heck actually got applied from the env vars I passed" because it can be challenging to figure out which were actually picked up. I wrote a thing which we use to solve that: https://github.com/relistan/rubberneck
Configure your app with envconfig, and then on startup have it dump the config it's running from to the logs. You could also do this from a CLI switch if needed.

@relistan that package looks great! I agree it's not really the job of envconfig -- I'll be pointing people to your package next time this comes up.

Thanks @teepark!