Improve precision of float multiple_of validation

Change Summary
Improved precision of float multiple_of validation and added test cases.

Refine the validation logic for better handling of floating-point rounding errors

Implement a more robust comparison using both absolute and relative error thresholds

Expand test cases to cover a wider range of scenarios

Add comprehensive tests for multiples of 0.5, 0.1, 2.0, and 0.01

Related issue number
fix https://github.com/pydantic/pydantic/issues/9871

Checklist

[x] Unit tests for the changes exist
[ ] Documentation reflects the changes where applicable
[ ] Pydantic tests pass with this pydantic-core (except for expected changes)
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review

I'm ok with changing this, but I think before we accept this PR we should discuss a bit what we want multiple_of for floats to do (and if we already document any behaviour).
Really, I think users expect this to match Python, so I assume that broadly we want the Python expression value % multiple_of == 0 to be true in Python for all value and multiple_of which we validate here.
Due to accuracy limitations of floating point, there are probably limitations on the ranges we can accept. Do the absolute and relative error thresholds give us enough control here? Do we need to give users any config levers to pull, if we're changing this?

Thank you for your feedback. Indeed, I agree that we should first clarify what multiple_of should do.
If the result of the multiple_of validation should be the same as Python's value % multiple_of == 0, then the modification I made is inappropriate. Naturally, this is because it doesn't match the results of Python's value % multiple_of == 0, which doesn't allow for rounding errors. (The following test uses the value of 0.01 for multiple_of, which was raised as an issue)
def is_multiple_of(value, multiple_of):
    return value % multiple_of == 0
multiple_of = 0.01
test_values = range(-10, 10)
print(f"Testing for multiple_of = {multiple_of}")
print("-" * 45)
print("value | is_multiple_of | value % multiple_of")
print("-" * 45)
for value in test_values:
    result = is_multiple_of(value, multiple_of)
    remainder = value % multiple_of
    print(f"{value:5} | {str(result):14} | {remainder}")

❯ python test.py
Testing for multiple_of = 0.01
---------------------------------------------
value | is_multiple_of | value % multiple_of
---------------------------------------------
  -10 | False          | 2.0816681711721685e-16
   -9 | False          | 1.8735013540549517e-16
   -8 | False          | 1.6653345369377348e-16
   -7 | False          | 1.457167719820518e-16
   -6 | False          | 1.249000902703301e-16
   -5 | False          | 1.0408340855860843e-16
   -4 | False          | 8.326672684688674e-17
   -3 | False          | 6.245004513516506e-17
   -2 | False          | 4.163336342344337e-17
   -1 | False          | 2.0816681711721685e-17
    0 | True           | 0.0
    1 | False          | 0.00999999999999998
    2 | False          | 0.009999999999999959
    3 | False          | 0.009999999999999938
    4 | False          | 0.009999999999999917
    5 | False          | 0.009999999999999896
    6 | False          | 0.009999999999999875
    7 | False          | 0.009999999999999854
    8 | False          | 0.009999999999999834
    9 | False          | 0.009999999999999813

However, in the existing specification, let threshold = float.abs() / 1e9; is implemented as a rounding tolerance, so I thought there might have been some intention, such as value % multiple_of == 0 alone being too strict.

Due to accuracy limitations of floating point, there are probably limitations on the ranges we can accept. Do the absolute and relative error thresholds give us enough control here? Do we need to give users any config levers to pull, if we're changing this?

Indeed, due to the precision limitations of floating-point numbers, there are restrictions for extremely large or small values. However, we believe the threshold settings we've incorporated can cover most common cases. (If necessary, we can implement more extensive test cases.)
Also, depending on user requirements, it might be worth considering adding an option for users to set the threshold themselves if they need to strictly control extreme edge cases.
In any case, I cannot decide on my own what result the multiple_of validation should return, so I would appreciate your judgment on this matter.

@davidhewitt  @sydney-runkle
I was wondering if you've had a chance to review this yet?
I would greatly appreciate your feedback when you have the opportunity.

Ah, after reviewing again, I do think we need @davidhewitt re:

In any case, I cannot decide on my own what result the multiple_of validation should return, so I would appreciate your judgment on this matter.

Your tests look good to me, but let's defer to DH regarding this logical choice. Will follow up with you tomorrow 👍

Thank you for checking! I'll be looking forward to response.

Sure thing. DH is out today, but should be back on Monday and able to respond 👍

This is high on my list of priorities and I've been doing some thinking on it, but I also have some fairly pressing stuff to complete this week. Will do my best to give a full writeup of my thoughts ASAP.

(and sorry for the very long delay, btw)

@K-dash,
Hope you're well! Any interest in continuing here? Thanks for your work thus far!


Preliminary wrapping of Randoop

@martinschaef, this is just a preliminary work. feel free to review it. Check the Randoop.java class. This is the main wrapper.

@martinschaef , I improved the overall test coverage of my classes; used less fancy collections, cleaned unnecessary code, etc, but checkstyle is still complaining :(. It seems I have to make my code compatible with Java 7 :(. if that is the case, then I will try to do that later today.

achievement unlocked!!! All checks have passed!!!

Many thanks :)


Render time only filter decorator

Added the @render_time_only decorator for filters and tests. It marks filters to not be resolved at compile time even if the arguments are constants. This can be useful as described in #1752 for time or memory intensive filters and tests, filters with global counting variables and changing global states, or filters that cause external actions such as printing, file operations or server requests.
The feature is tested by a new pytest, documented in changelog and the API documentation, passes pytest and tox (with a small tweak because mypy doesn't register set attributes).

Relating to issue #1752, @davidism can you review this MR ? The checks are good, the changelog and documentation are also aligned. Thanks


Stop using a developers personal access key to github for checkpoint.

It currently uses a personal github API token to authenticate with github. All API requests need API tokens even if they are for public repos.
Priya has currently put in her own accounts github API access token (read only access). In the long term, we have to create either a bot user for github and use it's access token OR register checkpoint as a OAuth application (shenanigans, implement an oauth callback etc.) on the dotmesh-io github organisation.

This is around 'Stop using a developers personal access key to github for checkpoint'. Github supports oauth, personal dev tokens etc on a single user level but NOT on organisations. Github at the organisation level only supports webhooks.
I've implemented a webhook from github so that when a release gets published, an event is fired to an endpoint in checkpoint http://checkoint.dotmesh.com/v1/release and that checks for signatures (i.e that it's really from github) and secrets and then updates it's notion of what the latest release is. This means that we no longer have to re-deploy checkpoint when a github release happens.
However, we can't only use the webhooks, because when the service restarts, it would need to get the version from github and not wait for a release to happen. So we can't really get away from using developer tokens, unless we store the lastest version in a local db somewhere so it persists between restarts. Even if we did that, we still have to wait for the first release to occur so it can be persisted.


Increment version for hdinsight releases

Increment package version after release of Azure.ResourceManager.HDInsight

API change check
API changes are not detected in this pull request.

/check-enforcer override

Failures are unrelated to these changes.   Merging.


[Bug] MarkLine tooltips are not shown when tooltip.trigger is axis

Version
5.4.3
Link to Minimal Reproduction
https://codesandbox.io/s/stacked-line-chart-forked-jfpn79?file=/index.html
Steps to Reproduce

Open the link
When you hover on the MarkLine component, the tooltip is not displayed but only the ones of the line series

Current Behavior
When you hover on the MarkLine component, the tooltip is not displayed but only the ones of the line series
Expected Behavior
I expect the markLine to be shown even on markLine. This happens when you set the tooltip. trigger equal to item for example.
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

not-a-bug
The Expected Behavior is not confirmed by the documentation.
There is no statement saying that trigger: "axis" should output the same info as trigger: "item".
Therefore you are asking for a new [Feature].

Any news?


避免OpenStack未设置默认值,导致认证失败

这个 PR 实现什么功能/修复什么问题:
避免OpenStack未设置默认值,导致认证失败
是否需要 backport 到之前的 release 分支:

release/2.10.0

Codecov Report

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

@@          Coverage Diff           @@
##           master   #1076   +/-   ##
======================================
  Coverage     7.9%    7.9%
======================================
  Files         356     356
  Lines       56477   56477
======================================
  Hits         4464    4464
  Misses      51618   51618
  Partials      395     395

Flag
Coverage Δ

#aFlag
7.9% <ø> (ø)
: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 9e1e8f5...2596b70. Read the comment docs.

/lgtm
/approve


The system cannot find the path specified

I tried to download Adele Hello, and another song by the link and I got this:

Enter a song/cmd: Adele Hello

Adele - Hello
5,827,520 Bytes [100.00%] received. Rate: [ 164 KB/s].  ETA: [0 secs]
Converting Adele_-_Hello.m4a to mp3..
The system cannot find the path specified.

Enter a song/cmd: https://play.spotify.com/track/4xkOaSrkexMciUUogZKVTS

Eminem- Till I Collapse Lyrics
4,773,227 Bytes [100.00%] received. Rate: [ 174 KB/s].  ETA: [0 secs]
Converting Eminem-_Till_I_Collapse_Lyrics.m4a to mp3..
The system cannot find the path specified.
Fixing meta-tags..
Traceback (most recent call last):
File "spotdl.py", line 250, in
Main()
File "spotdl.py", line 238, in Main
audiofile = eyed3.load("Music/" + Title + '.mp3')
File "C:\Python27\lib\site-packages\eyed3\core.py", line 81, in load
raise IOError("file not found: %s" % path)
IOError: file not found: Music/Eminem-_Till_I_Collapse_Lyrics.mp3

@benado This seems a problem with avconv..
you could try to install it again by purging it using sudo apt-get purge libav-tools and installing it again using sudo apt-get install libav-tools.
This should help maybe. Let us know how it goes!

I'm using windows and Ubuntu, It works fine in Ubuntu, the problem is in Windows.

@benado okay, that narrows it down.
Can you tell me the output of the following command by running it in python interpreter:
import os
print os.name

btw what windows are you on?

I tested on my windows and the problem seems to come up when you are not running the script from python's directory. To fix it, just cd C:\python27 and then run python spotdl.py.


Add KNX fan component

Description:
Related issue (if applicable): fixes #
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#6202
Example entry for configuration.yaml (if applicable):
fan:
  - platform: knx
    name: living-room-fan
    speed_address: '1/3/7'
    speed_state_address: '1/3/8'

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Hi @itineric,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Whats that last check thats not passing ? How can I fix it ?

You can try rebase to make coveralls pass. But it's not required. These modules are already excluded from coverage calculation, so it's ok to have coveralls fail. It's flaky.

This PR seems to have gone stale. Closing it.


Respect --cache parameter for receipt caching

          This should not be hard-coded. Also it might make sense to derive it from the cache argument, maybe `/receipts` or something.

Originally posted by @Wollac in https://github.com/risc0/zeth/pull/69#discussion_r1479979922

Related: https://github.com/risc0/zeth/issues/40
In the current system, if the cache_zkp directory does not already exist, then the receipt will fail to save.


Dont drop frames on non-user_errors

          1 Warning

:warning:
Please provide a changelog summary in the Pull Request description @ohayon

  Generated by :no_entry_sign: Danger

Any tests we can add for this?


Use getindex directly in Base.values

Optional, as the generated code is identical, but thought this would be clearer as the meaning is more direct than using an anonymous function.
Unrelated:
What does Base.indexed_iterate do in this package?

I was also wondering if there was  a better way to obtain the field names than using findfirst as in https://github.com/MasonProtter/MutableNamedTuples.jl/blob/master/src/MutableNamedTuples.jl#L30
I was thinking something like, the following, but it seems to allocate:
function _setfield!(mnt::MutableNamedTuple, s::Symbol, x)
          nt = getfield(mnt,:nt)
          setindex!(nt[s], x)
       end

@MasonProtter  should we close this ? (clearing out old PRs 😄 )

Hi @musm, my mistake I had forgotten about this!


Exception thrown by application class

See the error above --- perhaps this has to do with my environment variables?
What do i need to name the user-defined environment variables?
Thanks

The environment variables are referenced in the TwitterAnalysisServlet.java file ...
consumer_key = env.get("TWITTER_CONSUMER_KEY");
consumer_secret = env.get("TWITTER_CONSUMER_SECRET");
access_token = env.get("TWITTER_ACCESS_TOKEN");
access_key = env.get("TWITTER_ACCESS_KEY");
Those are the names of the vars needed.


Rudimentary support for dynamic trait objects

Fixes #655
Totally ignores all the bound-vars stuff for TraitObject because I haven't understood it at all, but wanted to start iterating.

This pr already helps a ton, although there are still some dyn trait related crashes when I try to apply it to Tock. So far, I've seen one main crash that I've minimized it down to:
struct Cow {}

trait CowContainer<'a> {
    fn get_cow(&self) -> &'a Cow;
}

pub struct CowCell<'a> {
    inner: &'a dyn CowContainer<'a>,
}

When you run flux on this code, it errors with:
error: internal compiler error: crates/flux-fhir-analysis/src/conv/mod.rs:444:13: unexpected! conv_poly_dyn_trait_ref
Which points to this code: https://github.com/flux-rs/flux/blob/23c745369bec99cf6bee49dd84c8547fbeac39bc/crates/flux-fhir-analysis/src/conv/mod.rs#L438-L450
Comparing this code to other code that passes, it appears that that the problem is that the the trait being parameterized by the dyn trait has a lifetime parameter. Let me know if you need any other info

I'll probably have some time to look at this tomorrow

@enjhnsn2 are you sure about the second one, it seems to work fine for me:
rjhala@pozole ~/r/flux (dyn)> more tests/tests/todo/dyn02.rs
pub trait Animal {
    fn noise(&self);
}

#[flux::trusted]
pub fn get_animal() -> &'static dyn Animal {
    unimplemented!()
}
rjhala@pozole ~/r/flux (dyn)> cargo xtask run tests/tests/todo/dyn02.rs
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.02s
     Running `target/debug/xtask run tests/tests/todo/dyn02.rs`
$ export FLUX_BUILD_SYSROOT=1
$ cargo build -p flux-rs
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.02s
$ cargo build
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.06s

@enjhnsn2 - the above two things work now, see

https://github.com/flux-rs/flux/blob/dyn/tests/tests/pos/surface/dyn02.rs
https://github.com/flux-rs/flux/blob/dyn/tests/tests/pos/surface/dyn03.rs

pub trait Animal {
fn noise(&self);
}
#[flux::trusted]
pub fn get_animal() -> &'static dyn Animal {
unimplemented!()
}

Whoops, I think I may have messed up the minimization for that one. When I was minimizing I started with tock code that pulled in the tock kernel and I think the error was in the dependency, not the minimized code. Let me try to get a better example

Alright, I found a minimized example that triggers the error. Unfortunately its a little more complicated:
pub trait Animal {
    fn noise(&self);
}

fn apply_closure_to_animal<F>(closure: F, animal: &'static dyn Animal)
where
    F: FnOnce(&dyn Animal),
{
    closure(animal);
}

Thanks, on it!

Ranjit.

On Wed, Jul 24, 2024 at 10:33 AM enjhnsn2 @.***> wrote:

Alright, I found a minimized example that triggers the error.
Unfortunately its a little more complicated:
pub trait Animal {
fn noise(&self);}
fn apply_closure_to_animal(closure: F, animal: &'static dyn Animal)where
F: FnOnce(&dyn Animal),{
closure(animal);}
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/flux-rs/flux/pull/664*issuecomment-2248555787__;Iw!!Mih3wA!HMp_f3DiAmkkZyGYTraImuODudwhdIKDCrIJrm2GsTIaWArQDMIXCnEPBLprCdOC47H0ZBEGXZ2Wk2KUKDXHXJnO$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AAMS4OCINCS2FSTVUEW3ZTTZN7QPNAVCNFSM6AAAAABLK6BQ4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBYGU2TKNZYG4__;!!Mih3wA!HMp_f3DiAmkkZyGYTraImuODudwhdIKDCrIJrm2GsTIaWArQDMIXCnEPBLprCdOC47H0ZBEGXZ2Wk2KUKF0HazE_$
.
You are receiving this because you authored the thread.Message ID:
@.***>

I prefer if

Alright, I found a minimized example that triggers the error. Unfortunately its a little more complicated:
pub trait Animal {
    fn noise(&self);
}

fn apply_closure_to_animal<F>(closure: F, animal: &'static dyn Animal)
where
    F: FnOnce(&dyn Animal),
{
    closure(animal);
}

This bug isn't too frequent and is possible to work around, so I don't think you should block this pr on it.

This feels it'll fail for various reasons not just dynamic objects. I'd prefer if we don't fix it in this PR.

This bug isn't too frequent and is possible to work around, so I don't think you should block this pr on it.

This feels it'll fail for various reasons not just dynamic objects. I'd prefer if we don't fix it in this PR.

@nilehmann  -- it turns out that for @enjhnsn2 example with closure and dyn above (see issue #666) -- just filling out the todo! case for Dynamic in the to_rustc(...) gets this to not crash, so I just tacked it onto this PR since its very much related to the whole conversion to/from ExistentialPredicate/ExistentialTraitRef that is the bulk of this PR...

Ok I think I addressed all the issues above -- except the

We are putting Dynamic in BaseTy which means they can be indexed (by unit as returned by this function),

But I'm pretty certain that's (for now at least) the sensible choice - as in dyn you have to forget all about the underlying concrete type; I suggest we revisit this later if we have reason to (or if we think there's some use-case for indexing Dynamic values...)


Informer send notification after closed

As discussed in https://github.com/megaease/easegress/pull/79#discussion_r660204660, it is possible for the current mesh informer implementation to send notifications after it is closed, this is unexpected and may cause bugs.
To fix this, we can add a wait operation in informer.Close to ensure all notifications are sent before the function returns, the change itself is simple but we need to check the existing code to ensure the wait does not cause a deadlock.
If the above solution is not possible, we can document the behavior and let client code handle the delayed notifications.

@suchen-sci


Update labels for teacher training years

Trello card
Trello-1725
Context
We want to update the labels fo the teacher training years, which come back from the CRM as '2021', 2022', etc. Instead, if the year is the current year we want to show the format:
<year> - start your training this September

We only show the current year up until September. If the year is a future year we want to use the format:
<year> - start your training in September <year>

Changes proposed in this pull request

Update labels for teacher training years

Guidance to review
https://review-teacher-training-adviser-760.london.cloudapps.digital/teacher_training_adviser/sign_up/start_teacher_training

I would guess on mobile it might crop the right-most text on a select picker? cc @MylesJarvis

@ethax-ross think I managed to break this trying to see what this option would look like, any idea what I've done wrong?


map - click to search as a tool instead

All too often i find myself clicking the map not with the intention of starting a search but to navigate. Search should probably be a tool instead

it is now a tool


Issue Saving CSVs in Python

Here is a sample CSV file:
apple,orange
1,2
3,4

Now I want to load it in, and save it as a test:
df = sqlContext.read.format('com.databricks.spark.csv').options(header='true').load('test.csv')
df.select('apple', 'orange').write.format('com.databricks.spark.csv').save('new_data.csv')

This method however, does not output a CSV but rather a folder containing 3 files: part-00000, part-00001 and  _SUCCESS.
I assume this is an error? I'm using:
Spark 1.4.0
pyspark --packages com.databricks:spark-csv_2.10:1.1.0

No this is intended behavior. Spark writes one file per partition and default number of partitions is two. You can change that using repartitioning.

Seems odd though that the API is not designed in a way that the output of save() is valid input for load()?

You can pass a directory to load() function. The output of save() is valid for load(). Does it not work for you?

Actually it's working perfectly fine - sorry about the confusion!

Works perfectly fine - sorry for the confusion


Reuse existing identity key on startup

I looked around the docs and issues list for a way to reuse an existing master key on the hyperdrive startup, but found nothing. Is it possible? And if so, how to do it?
Use case for that would be using the same identity while accessing your data on the network in different devices. Imagine an hyperdrive-based "Twitter". I want to post from several different devices (computer, phone, etc.) but using the same identity, so I won't need to have a separate feed of things I posted for each device I am using.

Did you ever find an answer?


Remove public imports

This PR removes the import public construct used in spine/users/identifiers.proto as this syntax does not work well with JS generated code.
See this issue for the status of the bug.

@armiol, PTAL.


iPhoneHTTPServer Sample working on simulator but not on device.

Greetings I have implemented this in an iPhone App but neither my App nor the Sample iPhoneHTTPServer work for the device, they do work in the simulator though.
When I use the wifiIP:port on my Mac(using any browser) which is on the same network of the iPhone it simply won't open the Page, but it doesn t give a 404 error either. Its like the page is there but it can't connect.
"Unable to connect" "Firefox can't establish a connection to the server at <IP_ADDRESS>:52395."
"The site could be temporarily unavailable or too busy. Try again in a few
moments.
If you are unable to load any pages, check your computer's network
connection.
If your computer or network is protected by a firewall or proxy, make sure
that Firefox is permitted to access the Web."
Any idea what might be causing this? I know I have to disable my Mac OS X firewall when I want to share a website on the local network is there something similar for the iPhone App?
thanks

I had the same problem and I have tried all the steps mentioned in this issue, but it still doesn't work. I'm sure i have config the server correctly.


Add authentication to Inventory service

The provider inventory contains information about the provider layout and this can be considered sensitive by some users.
It would be better to add an authentication layer to the inventory service to limit access to authenticated users. The UI impersonates the logged in user and can then access the inventory.
RHBZ: https://bugzilla.redhat.com/show_bug.cgi?id=1929140

For GA, let's just do a TokenReview for authentication and defer authorization.

Done. https://github.com/konveyor/forklift-controller/pull/203


Can't set state, bad token?

i keep getting this error saying bad token does anyone know what wrongs?

Unless there is something deeper, I recall an error like this that I had.  I found that the token sitting in /tmp/myqtoken.json was old and expired.  I deleted the token file and all worked.  Hate to say this is it as I have not tested this for a few weeks and memory is foggy.  Try and see


Don't update the deployment if we didn't start it

Fixes #4993

Proposed Changes

Don't update deployments if we didn't start them. This avoids errors when trying to mark a deployment inactive without knowing the references to that deployment, because it doesn't exists.

Readiness Checklist
Author/Contributor

[x] I included all the needed documentation for this change.
[x] I provided the necessary tests.

Reviewing Maintainer

[x] Label as breaking if this is a large, fundamental change.
[x] Label as either: automation, bug, documentation, enhancement, infrastructure.

@zkoppert going ahead with this simple change so we can also check if it fully works as expected given the upcoming changes to the cd workflow, in case of issues.


Collaborate with Paul on early Service Delivery Model/Blueprint

Collaborate with Paul on early Service Delivery Model/Blueprint
Related Milestone: Develop alpha version of Managed Cloud Services Blueprint
Definition of done
Participate in session with Brittany to review exemplar Service Blueprint, then initiate ours along similar lines.

Replaced by more granular tickets #239, 242, 244


Correct Amblight Setup

I'm having issues with my Fushicai frame grabber and just need some clarification on my setup.
My setup is for multiple HDMI sources so I am using a HDMI splitter and HDMI2AV convertor with a Rpi model B running Rasbian.  With this setup I am unable to get my framegrabber to take screenshots or work with my TV. I am pretty sure I have tried everythung and it is still not working. So i just want to check something so i know what i should be aiming for.
With this setup up would I be correct in thinking that if my framegrabber is working correctly if i plug in another TV using a HDMI cable from the Rpi to the TV I should see whatever is on my source input? For example my Blu-ray player / laptop / sky box? Or will the display output from the Rpi be XMBC?
Please help?

@Batmanan still need help?

Closed due to age. Please open a new one, if problems still exists.
Thank you


Missing a clearDatabase() function that would casacade-delete all the tables in a database

I am running tests with a database inside a docker-composed environment.
Why? Wanna make sure my service classes do exactly what they're supposed to do.
Currently, I am running a
  beforeEach(async () => {
    await userRepository.nativeDelete({});
  });

for every test where I delete the data that was created during the test.
A clearDatabase() function would really help, since I want to have a clean database in the beginning of every test.
Also: do you provide a functionality for creating seed data so that I don't have to add things manually?

I guess it makes sense to have a helper method for this, although don't expect too smart things, as each driver behaves a bit differently.
If you want cascading using postgres, you can use QueryBuilder.truncate(), other drivers do not offer cascading deletes - this is a database feature, no plans to implement app level cascading outside of entity manager. In MySQL we can just disable FKs via foreign_key_checks.
Note that doing nativeDelete() (or in general simple delete query) will be usually faster than truncate, especially when cascading. I would say it should be preferred in tests.
You can also wipe the whole schema via SchemaGenerator.dropSchema() and just recreate it.

Also: do you provide a functionality for creating seed data so that I don't have to add things manually?

It's already implemented in v5, don't expect this to land to v4 either, only bugfixes are being backported.
https://mikro-orm.io/docs/next/seeding
And lastly a regular reminder, if you want to open issues, please don't ignore the issue template.

FYI v5 now has SchemaGenerator.refreshDatabase(), it will:

ensure the database exists
drop the schema if exists
create the schema from scratch


make error

Issue summary
nvlink fatal   : Could not open input file '/usr/lib/x86_64-linux-gnu/libpthread.a' (target: sm_86)
Steps to reproduce
I followed the installation guildline, it worked fine untill make
I got these output:
`
[  0%] Performing update step for 'libmolgrid'
[  0%] No patch step for 'libmolgrid'
[  0%] Performing configure step for 'libmolgrid'
-- Current git revision is b9ae032f52fc7a8153987bde09c0efa3620d8bb6
-- Found Boost: /usr/lib/x86_64-linux-gnu/cmake/Boost-1.74.0/BoostConfig.cmake (found version "1.74.0") found components: regex unit_test_framework program_options system filesystem iostreams
-- Found Boost: /usr/lib/x86_64-linux-gnu/cmake/Boost-1.74.0/BoostConfig.cmake (found version "1.74.0") found components: system filesystem python39
-- Found Boost: /usr/lib/x86_64-linux-gnu/cmake/Boost-1.74.0/BoostConfig.cmake (found version "1.74.0") found components: unit_test_framework system
-- Configuring done
-- Generating done
-- Build files have been written to: /home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build
[  0%] Performing build step for 'libmolgrid'
[ 29%] Built target libmolgrid_static
[ 59%] Built target libmolgrid_shared
[ 65%] Built target molgrid
[ 68%] Built target test_grid_cpp
[ 71%] Built target test_transform_cu
[ 75%] Built target test_grid_cu
[ 78%] Built target test_gridinterp_cpp
[ 81%] Built target test_transform_cpp
[ 84%] Built target test_gridmaker_cu
[ 87%] Built target test_mgrid_cpp
[ 90%] Built target test_gridmaker_cpp
[ 93%] Built target test_coordinateset_cpp
[ 96%] Built target test_mgrid_cu
[100%] Built target test_quaternion_cpp
[  0%] Performing install step for 'libmolgrid'
[ 29%] Built target libmolgrid_static
[ 59%] Built target libmolgrid_shared
[ 65%] Built target molgrid
[ 68%] Built target test_grid_cpp
[ 71%] Built target test_transform_cu
[ 75%] Built target test_grid_cu
[ 78%] Built target test_gridinterp_cpp
[ 81%] Built target test_transform_cpp
[ 84%] Built target test_gridmaker_cu
[ 87%] Built target test_mgrid_cpp
[ 90%] Built target test_gridmaker_cpp
[ 93%] Built target test_coordinateset_cpp
[ 96%] Built target test_mgrid_cu
[100%] Built target test_quaternion_cpp
Install the project...
-- Install configuration: "Release"
-- Up-to-date: /home/glinttsd/gnina/build/external/lib/libmolgrid.so
-- Up-to-date: /home/glinttsd/gnina/build/external/lib/libmolgrid.a
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/libmolgrid.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/grid.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/managed_grid.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/quaternion.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/transform.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/atom_typer.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/example.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/coordinateset.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/exampleref_providers.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/example_extractor.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/example_provider.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/example_dataset.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/grid_maker.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/grid_interpolater.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/coord_cache.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/common.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/grid_io.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/cartesian_grid.h
-- Up-to-date: /home/glinttsd/gnina/build/external/include/libmolgrid/config.h
/usr/lib/python3/dist-packages/setuptools/dist.py:461: UserWarning: Normalizing 'v1.0.1.dev53+gb9ae032f' to '1.0.1.dev53+gb9ae032f'
warnings.warn(tmpl.format(**locals()))
running install
Checking .pth file support in /home/glinttsd/gnina/build/external/lib/python3.9/site-packages/
/usr/bin/python3.9 -E -c pass
TEST FAILED: /home/glinttsd/gnina/build/external/lib/python3.9/site-packages/ does NOT support .pth files
bad install directory or PYTHONPATH
You are attempting to install a package to a directory that is not
on PYTHONPATH and which Python does not read ".pth" files from.  The
installation directory you specified (via --install-dir, --prefix, or
the distutils default setting) was:
/home/glinttsd/gnina/build/external/lib/python3.9/site-packages/
and your PYTHONPATH environment variable currently contains:
Here are some of your options for correcting the problem:

You can choose a different installation directory, i.e., one that is
on PYTHONPATH or supports .pth files

You can add the installation directory to the PYTHONPATH environment
variable.  (It must then also be on PYTHONPATH whenever you run
Python and want to use the package(s) you are installing.)

You can set up the installation directory to support ".pth" files by
using one of the approaches described here:
https://setuptools.readthedocs.io/en/latest/easy_install.html#custom-installation-locations
Please make the appropriate changes for your system and try again.
running bdist_egg
running egg_info
writing /home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build/python/molgrid.egg-info/PKG-INFO
writing dependency_links to /home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build/python/molgrid.egg-info/dependency_links.txt
writing requirements to /home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build/python/molgrid.egg-info/requires.txt
writing top-level names to /home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build/python/molgrid.egg-info/top_level.txt
reading manifest file '/home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build/python/molgrid.egg-info/SOURCES.txt'
writing manifest file '/home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build/python/molgrid.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_py
creating build/bdist.linux-x86_64/egg
creating build/bdist.linux-x86_64/egg/molgrid
copying build/lib/molgrid/init.py -> build/bdist.linux-x86_64/egg/molgrid
copying build/lib/molgrid/molgrid.so -> build/bdist.linux-x86_64/egg/molgrid
copying build/lib/molgrid/torch_bindings.py -> build/bdist.linux-x86_64/egg/molgrid
byte-compiling build/bdist.linux-x86_64/egg/molgrid/init.py to init.cpython-39.pyc
byte-compiling build/bdist.linux-x86_64/egg/molgrid/torch_bindings.py to torch_bindings.cpython-39.pyc
creating build/bdist.linux-x86_64/egg/EGG-INFO
copying /home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build/python/molgrid.egg-info/PKG-INFO -> build/bdist.linux-x86_64/egg/EGG-INFO
copying /home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build/python/molgrid.egg-info/SOURCES.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying /home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build/python/molgrid.egg-info/dependency_links.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying /home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build/python/molgrid.egg-info/not-zip-safe -> build/bdist.linux-x86_64/egg/EGG-INFO
copying /home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build/python/molgrid.egg-info/requires.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying /home/glinttsd/gnina/build/libmolgrid-prefix/src/libmolgrid-build/python/molgrid.egg-info/top_level.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
writing build/bdist.linux-x86_64/egg/EGG-INFO/native_libs.txt
creating 'dist/molgrid-1.0.1.dev53+gb9ae032f-py3.9.egg' and adding 'build/bdist.linux-x86_64/egg' to it
removing 'build/bdist.linux-x86_64/egg' (and everything under it)
Processing molgrid-1.0.1.dev53+gb9ae032f-py3.9.egg
removing '/home/glinttsd/gnina/build/external/lib/python3.9/site-packages/molgrid-1.0.1.dev53+gb9ae032f-py3.9.egg' (and everything under it)
creating /home/glinttsd/gnina/build/external/lib/python3.9/site-packages/molgrid-1.0.1.dev53+gb9ae032f-py3.9.egg
Extracting molgrid-1.0.1.dev53+gb9ae032f-py3.9.egg to /home/glinttsd/gnina/build/external/lib/python3.9/site-packages

Installed /home/glinttsd/gnina/build/external/lib/python3.9/site-packages/molgrid-1.0.1.dev53+gb9ae032f-py3.9.egg
Processing dependencies for molgrid==1.0.1.dev53+gb9ae032f
Searching for pytest==6.0.2
Best match: pytest 6.0.2
Installing py.test script to /home/glinttsd/gnina/build/external/bin
Installing pytest script to /home/glinttsd/gnina/build/external/bin
Using /usr/lib/python3/dist-packages
Searching for pyquaternion==0.9.9
Best match: pyquaternion 0.9.9
Processing pyquaternion-0.9.9-py3.9.egg
Using /home/glinttsd/gnina/build/external/lib/python3.9/site-packages/pyquaternion-0.9.9-py3.9.egg
Searching for numpy==1.19.5
Best match: numpy 1.19.5
Installing f2py script to /home/glinttsd/gnina/build/external/bin
Installing f2py3 script to /home/glinttsd/gnina/build/external/bin
Installing f2py3.9 script to /home/glinttsd/gnina/build/external/bin
Using /usr/lib/python3/dist-packages
Finished processing dependencies for molgrid==1.0.1.dev53+gb9ae032f
[  0%] Completed 'libmolgrid'
[  1%] Built target libmolgrid
[  1%] Built target caffeproto
[  1%] Linking CUDA device code CMakeFiles/caffe.dir/cmake_device_link.o
nvlink fatal   : Could not open input file '/usr/lib/x86_64-linux-gnu/libpthread.a' (target: sm_86)
make[2]: *** [caffe/src/caffe/CMakeFiles/caffe.dir/build.make:2787: caffe/src/caffe/CMakeFiles/caffe.dir/cmake_device_link.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1387: caffe/src/caffe/CMakeFiles/caffe.dir/all] Error 2
make: *** [Makefile:160: all] Error 2
`
It seems like I have a python related problem (indicated in the middle of output info) and a nvlink problem (indicated in the last of output info)
I also noticed the same problem in #140
Your system configuration
Operating system: Ubuntu 21.10
Compiler: gcc 11.2.0
CUDA version (if applicable):11.6
CUDNN version (if applicable): not installed
Python version:3.9.7

Duplicate of #140


[css] Using [alt]+arrow to shift blocks could result in not working CSS

Issue Type: Bug
I shifted a CSS code block upwards above another code block. The CSS file looked ok, but the class wasn't used, because there seamed to be an invisible char included.
After deleting the blank between the class name and the { braces - it worked.
VS Code version: Code 1.21.1 (79b44aa704ce542d8ca4a3cc44cfca566e7720f1, 2018-03-14T14:39:09.335Z)
OS version: Darwin x64 17.4.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6920HQ CPU @ 2.90GHz (8 x 2900)

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.29GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (25)

Extension
Author (truncated)
Version

ng-template
Ang
0.1.9

vscode-markdownlint
Dav
0.14.1

EditorConfig
Edi
0.12.1

vscode-docker
Pet
0.0.25

code-settings-sync
Sha
2.9.0

html-css-class-completion
Zig
1.17.1

npm-intellisense
chr
1.3.0

tslint
eg2
1.0.28

vscode-npm-script
eg2
0.3.3

prettier-vscode
esb
1.2.2

vscode-power-mode
hoo
2.2.0

Angular2
joh
2.15.0

vscode-scss
mrm
0.6.2

vscode-azureappservice
ms-
0.6.1

vscode-azurefunctions
ms-
0.7.0

vscode-azurestorage
ms-
0.2.0

vscode-cosmosdb
ms-
0.5.1

azure-account
ms-
0.3.3

azurecli
ms-
0.4.1

vscode-node-azure-pack
ms-
0.0.2

azurerm-vscode-tools
msa
0.4.0

debugger-for-chrome
msj
4.2.2

angular2-inline
nat
0.0.17

vscode-icons
rob
7.22.0

stylelint
shi
0.35.0

Reproduces without extensions

Any idea how the invisible character got inserted. Is it related to the 'shift up' (did you use the 'move up' command?
A sample with detailed steps would be helpful.


using gray image to predict

I wanna ask if it is possible to use 1 channel images data to train and test
I already finish the training part, but when I run the predict.py it always problem
I only changed yolo.py line 232 from
" input_image = Input(shape=(None, None, 3)) "  to
" input_image = Input(shape=(None, None, 1)) "
plz anyone help me did I miss anything THX a lot

There was a response from the author answering this for yolo2, so are we sure that this can't be done in this version?


filling in missing family/given names for VA legislators.

I'm not sure why these are missing?

thanks for this, merging with a fix for the lint issues that arose w/ retirements

Thanks!


Possible implementation for background-image sizing

Just a thought but...
If in .hyper_main if you set background-size: contain; the image looks much sharper and better in terms of resolution.

I think I tried that originally and found that the background image started tiling if the dimensions of the terminal were greater than the dimensions of the image, and I wasn't sure if that was the effect I wanted. If you want to provide some screenshot samples of background-size: contain; for a few different size windows, we can choose what suits best.
We could also provide it as a config option to let people choose how they want to display the background

Well. If you used background-repeat: none; that would prevent the image from tiling when the terminal windows exceeds the image dimensions.
I will test this more thoroughly when I get home from work as my monitors at home are larger than those at work.
I think in the end though, adding a config option would be better, rather than "forcing" the user to have background-size: contain;


arm: stm32f7: add support for the stm32f765xx soc

The stm32f765xx series is similar to stm32f767xx series but doesn't include an LCD-TFT peripheral. I have updated the device tree files to have a separate one for the stm32f765 which is then included by the stm32f767 to add lcd-tft display controller node.

One last issue: https://github.com/zephyrproject-rtos/zephyr/actions/runs/6026405699/job/16349554945?pr=62055
To update the author name, see https://www.git-tower.com/learn/git/faq/change-author-name-email


ERROR INSTALLING COUCHDB!!

Description
I install couchdb on a weekly basis and today i ran the same code as it do
sudo apt update && sudo apt install -y curl apt-transport-https gnupg
curl https://couchdb.apache.org/repo/keys.asc | gpg --dearmor | sudo tee /usr/share/keyrings/couchdb-archive-keyring.gpg >/dev/null 2>&1
source /etc/os-release
echo "deb [signed-by=/usr/share/keyrings/couchdb-archive-keyring.gpg] https://apache.jfrog.io/artifactory/couchdb-deb/ ${VERSION_CODENAME} main" \
    | sudo tee /etc/apt/sources.list.d/couchdb.list >/dev/null

not i receive an error
Steps to Reproduce
just run
sudo apt update && sudo apt install -y curl apt-transport-https gnupg
curl https://couchdb.apache.org/repo/keys.asc | gpg --dearmor | sudo tee /usr/share/keyrings/couchdb-archive-keyring.gpg >/dev/null 2>&1
source /etc/os-release
echo "deb [signed-by=/usr/share/keyrings/couchdb-archive-keyring.gpg] https://apache.jfrog.io/artifactory/couchdb-deb/ ${VERSION_CODENAME} main" \
    | sudo tee /etc/apt/sources.list.d/couchdb.list >/dev/null

Your Environment
I installed a new vps and installed it right away.

CouchDB version used:
Browser name and version:
Operating system and version: ubuntu 22.04

Additional Context
I noticed a push was made yesterday. maybe this is the problem. It is now causing issues on my other cluster coucdb servers.

the issue was that i installed it on ubuntu 22.04 and the package is ubuntu 20.04


Adds support for querying packages

Makes a system call to a import.sh script that finds the distro than does the import if possible and if not than everything occurs normally...
Tested for archlinux but if you like the implementation than I could do testing for other distros too...and also add support for may-be open-suse.
There is a possibility of  a feature that you could revert, even, from the installation prompt(but that I don't think would be of much practical value as...how many valid python scripts match package names?)
So views...

This looks good. I'll try to put it to test in a few days.


fix(@angular-devkit/schematics): include logger in testing context

SchematicTestRunner.callRule was not using the schematicTestRunner's logger requiring the user to pass it in the parent context themselves. This should be default.

Nevermind. Guess this is more complex than I thought :\ Doesn't seem like callRule functions that well right now...


fix error: Use .Site.Config.Services.GoogleAnalytics.ID instead.

Purpose
Fix for hugo 0.133.0+ error
ERROR deprecated: .Site.GoogleAnalytics was deprecated in Hugo v0.120.0 and will be removed in Hugo 0.134.0. Use .Site.Config.Services.GoogleAnalytics.ID instead.

Closing as a previous PR in #3137 was already made to support this breaking change in Hugo.


Removes aad options. Closes #5823

Removes aad options. Closes #5823 and #5676

Thanks @nanddeepn!

hey @nanddeepn may I kindly ask you to rebase with latest v9 and resolve the conflicts before we proceed 👍.
Let me know if you need any kind of help.
Thank you for your constant work and contributions to the community 👏. You Rock 🤩

@nanddeepn this change is almost same as #6131. Lets concat those two PR's to a single change closing both issues.
Please let's leave the upgrade guidance from this PR since it is really helpful but we also need to add this new page to sidebar.ts 👍

Hi @Adam-it
Concating two PRs to close #5823 and #5676

Hi @Adam-it
Concating two PRs to close #5823 and #5676

Awesome 🤩😍

Due to an upcoming breaking change connected to CLI for Microsoft 365 login experience we will need to do a v9 release a bit sooner than we anticipated. Due to that we had to change the target of this PR to v10 which will be the next regular major release.

@nanddeepn I am terribly sorry for a long hold up from my side due to other priority activities.
May I kindly ask you to rebase this branch with latest main so that we may proceed with the review?

Ready to merge 🚀

Merged manually. Thank you for your awesome contribution 👏.
You Rock 🤩


Affix - affix.bs.affix doesn't fire on page load

An element is affixed when scrolling more than 100px.

If you scroll to 200px it gets affixed
affix.bs.affix event fires
if you reload the page the element is affixed
but affix.bs.affix doesn't fire.

Tested on FF 27 and Chrome 33 on Mac.
View the behavior:
http://jsbin.com/lasiqema/1/
View the code:
http://jsbin.com/lasiqema/1/edit?html,js,output

Had the same problem. nightkon's code fixed it for me too.
The key take away though is that specifying a static integer for top causes the plugin to not fire on page load as shown below:
    $('#abc').affix({
        offset: {
            top: 400
        }
    });

The fix is to have a function call that gets re-evaluated internally by the affix plugin like so:
    $('#abc').affix({
        offset: {
            top: function() { return 400; }
        }
    });


(federation-guid-optimization) Test to reproduce reverse synchronization issue

closing after discussion, this is not a bug


type expressions for Accessibility options

In preparation for doing code reviews of a11y instrumentation...
Accessibility.js contains ACCESSIBILITY_OPTION_KEYS, which documents the options available.   It would be helpful to have type expressions for all of these.

Sounds good, Ill add em. Documentation pattern was borrowed from NODE_OPTION_KEYS in Node, and I see similar lists of options in
Accessibility.js:
AlignBox.js:
Circle.js:
DOM.js:
Image.js:
LayoutBox.js:
Line.js:
Node.js:
Paintable.js:
Path.js:
Rectangle.js:
RichText.js:
Text.js:
WebGLNode.js:

@jonathanolson should we add type expressions to these as well? I am happy to do it, I see it is already done for AlignBox.

Added to Accessibility.js, to @jonathanolson before adding to the others.

👍 This looks very nice, thanks @jessegreenberg.

@jonathanolson should we add type expressions to these as well? I am happy to do it, I see it is already done for AlignBox.

It sounds like a lot of duplication (a pain to maintain), but would be nice to have presumable when scanning. I'm fine with them being added, thanks!

Sounds great, type expressions were added to lists of option keys for files listed in https://github.com/phetsims/scenery/issues/977#issuecomment-497881972. Closing.


Don't rely on implicit const char* -> bool conversion.

While here only used in a test, this is an error-prone pattern.
Also fix slightly changed dependencies needed with the recent move
of using random identifier generation closer to the specific
verilog tool.
Signed-off-by: Henner Zeller<EMAIL_ADDRESS>

Codecov Report

Merging #1240 (8a2b00b) into master (c14b8f5) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 8a2b00b differs from pull request most recent head ee2f071. Consider uploading reports for the commit ee2f071 to get more accurate results

@@           Coverage Diff           @@
##           master    #1240   +/-   ##
=======================================
  Coverage   92.75%   92.75%
=======================================
  Files         338      338
  Lines       23259    23259
=======================================
  Hits        21574    21574
  Misses       1685     1685

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 c14b8f5...ee2f071. Read the comment docs.


bootstrap: goes to "registering" state after several hours

@dbaba san,
I encountered one potential bug,
I configure device to use bootstrap, client bootstrapping and registration is successful (in it Connected).
Then, everything is working for some hours (I don't know for how long exactly),
then, client node is going to permanent "registering" status - it doesn't change to Registered

If I re-deploy, it is working fine in most cases,
sometimes it goes to Disconnected immediately: changing Listen port to new one helps,
Restarting Node-Red also helps.
Could you please suggest how to troubleshoot the issue?

@djDuff Thanks for the report! Let me check the issue.

@djDuff I've released v2.2.0. I ran the client a couple of nights and got no errors. Of course, it didn't fall into "registering" state during the time.
However, for the safety, I added the watchdog for the lwm2m client process, which will restart it if the process gets halted.


Clarify relationship with Disqus, the company

In the package.json description, this package is described as the official Disqus React package. However, this is not mentioned on the GitHub repo description, nor in the README file, and the package is not published under the Disqus GitHub Organization https://github.com/disqus.
I think it would be useful to clarify how official this package is, and make it easier for developers to trust it out of the other Disqus React components that are out there.

Yep! We'll be transferring it shortly.

Hi! Is there any update on this issue? Also, could you kindly update package.json and create a new release on npm? Thanks in advance!

+1

Repository is now part of the Disqus org, as is the NPM package!


Moved table_entity_mapping in container as parameter

Description
Please describe your pull request:

Why is it necessary?
Easier editable with plugins.
What does it improve?
Plugins could do add a new CompilerPass which modifies the container parameter "shopware_attribute.table_entity_mapping"
Does it have side effects?
Nope

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?

How to test?

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-18295.
Please use this issue to track the state of your pull request.

Thanks @shyim. Can you provide an example plugin which utilizes your change? Maybe we can use it as an example for the devdocs.


Document SQL Server aggregation pushdown

Follows https://github.com/prestosql/presto/pull/5196 / https://github.com/prestosql/presto/issues/4139

See https://github.com/prestosql/presto/issues/5111#issuecomment-696183351 for input.

https://github.com/prestosql/presto/pull/5245


Need YAML support for accessing elements from saved lists

Feature description
Need list element operations from the saved List we need to access one of the elements from that list.. something like "savedListVariable[0]" also need support for all operations can be done on List eg: List contains particular element, saving one of the value from the list, checking List fileds constraints.
Effected tests:
TC_AUDIOOUTPUT_7_2
TC-BR-2
TC-BR-4
TC-CADMIN-1.13
TC_CNET_4_2
TC_CNET_4_3
TC_CNET_4_4
TC_CNET_4_9
TC_CNET_4_10
TC_CNET_4_11
TC-DD-3.14
TC-DD-3.17
TC_DGGEN_2_1
TC_FLABEL_2_1
TC_LTIME_3_1
TC_MEDIAINPUT_3_13
TC_PSCFG_2_2

Platform
all
Platform Version(s)
No response
Anything else?

TC-MOD-1.2 and TC-LCFG 2.1 scripts also requires List filed constraint check

High Priority for automated tests

From python yaml runner we can now save lists and will unblock us @kvikrambhat


Known limitations

There are a few limitations in the initial release but we are working day and night to resolve these and add them as future features.

Doesn't support Image / Audio data
Doesn't support - quote delimited CSVs( TFX doesn't support qCSV yet )
Classification only supports integer labels from 0 to N

Support for Quoted CSV added in 1.3.1


3D Box  orientation Regression problem

hi,
in your code , global object orientation is regressed by CNN , from only the contents of the 2D bboxs result,
but , I found that the author of the paper[https://arxiv.org/abs/1612.00496] suggested that ,local orientation is regressed, not the global object orientation.
Did I get it wrong?

Hi @weisili2016 ,
The CNN will predict the local object orientation, not the global object orientation.
So in testing phase, θlocal will add with θray.
https://github.com/smallcorgi/3D-Deepbox/blob/c409d46648afc241930972bae5f1a2c700816862/main.py#L252

data_processing.py
line 14.
def compute_anchors(angle):
....
so, in training phase ,the input angle is the local object orientation?

@weisili2016
Yes.


Implement PLE0206

#970

Rock! Nice work.

Oh wait, I think this should be PLR0206? Since it's a "refactor" rule?

@charliermarsh I misunderstood the naming rule. Yes, it should be PLR0206.

@harupy - Sorry, I wasn't super clear. What do you think of that convention? I wanted to preserve the R0206 part to match Pylint.

That convention sounds good to me. If we use PLE, W1301 and E1301 would conflict.

Oh hah, true!

I'll rename this check and push it straight to main.

Oh nm! Thank you!


added accelerate upgrade in ollama UI quickstart

accelerate version specified in bigdl-core-cpp is fallen behind and will result in incompatibility with transformers when running open-webui for ollama, therefore requires an upgrade before running start_windows.bat

@rnwang04 pls kindly review :)

Is it specific to Open WebUI, or shall we upgrade [cpp] dependency?
Instead of using the latest version, we should probably specify the particular versions that we have tested
We should add these versions into requirements.txt?

Is it specific to Open WebUI, or shall we upgrade [cpp] dependency?
Instead of using the latest version, we should probably specify the particular versions that we have tested
We should add these versions into requirements.txt?

OK, we will do more test with transformers / accelerate version, and evaluate if we can directly upgrade and specify their version in [cpp] dependency.

Is it specific to Open WebUI, or shall we upgrade [cpp] dependency?
Instead of using the latest version, we should probably specify the particular versions that we have tested
We should add these versions into requirements.txt?

OK, we will do more test with transformers / accelerate version, and evaluate if we can directly upgrade and specify their version in [cpp] dependency.

You should always specify the particular versions that you have tested; using latest versions will break the compatibility in future.

Is it specific to Open WebUI, or shall we upgrade [cpp] dependency?
Instead of using the latest version, we should probably specify the particular versions that we have tested
We should add these versions into requirements.txt?

OK, we will do more test with transformers / accelerate version, and evaluate if we can directly upgrade and specify their version in [cpp] dependency.

You should always specify the particular versions that you have tested; using latest versions will break the compatibility in future.

Got it, we will pay attention to it later.


An official, community driven Thanos Helm Chart in thanos-community organization.

Now doc provide two helm chat, but none is official.
Uses have no idea to choose one to play with.
Some discussion is in slack.
I would like to thanos host an official helm charts like loki does, so that users can easily get official and stable version of helm chart to deploy.
As an official version, not need to update all the time, update it when release(not cover rc) is enough i think.
What do you think? @bwplotka
BTW, it is good to add a group which covert all maintainers to ping in vote case.
@thanos-io/thanos-maintainers

cc @thanos-io/thanos-maintainers

As an official version, not need to update all the time,

What's wrong with the unofficial version? What problem are we trying to solve here? Is it only the choice for users; if that's the case what if we would recommend officially one of those? (:
None of the maintainers use helm, so it's hard and not effective for us to maintain such a chart. We are already overloaded. We are definitely happy for someone to help in doing this, but not sure if having same repository helps here.

Unofficial version means maintainer by person, maybe outdated or something wrong or not support as much as official one does. The official version means most authoritative i think.
Feel free to host a chart or put it in helm/charts as stable/thanos.
I know that many maintainers used to use jsonnet, while more users use helm in fact. I would like to do this if possible.

Prometheus also doesn’t have a chart maintained by prometheus maintainers for the same reasons. I’ve had very bad experiences with having helm “official charts“ in the prometheus operator repository so I’m rather hesitant as well. I think if there are one or more maintainers willing to maintain it then having a separate repo for a chart should be fine.

I would say that unless it is part of anyone's core workflow for rolling out Thanos with that specific image it would be a burden and become unloved therefore giving less value to the community than one maintained elsewhere.
However, if someone does have Helm as their core workflow and wants to maintain it they should but it would be hard to expect everyone to learn the intricacies of Helm to support it.

Any input for this? @thanos-io/thanos-maintainers
Maybe not host by this repo, i would like to add thanos into helm/charts if possible.

I am not against having an "official", community-supported thanos chart hosted at helm/charts, just that I don't think the maintainers here will be able to offer support.

What about having it under a new repo in the org? With a core set of maintainers that are Helm advocates and use it as part of their flow. @daixiang0 do you use helm as part of your daily workflow?
As much as personally do use Helm if users are wanting it it would be better to have something to offer people.

@domgreen Yep, i use helm as a part of daily work.
Create a new repo seems a good choice.

cc @thanos-io/thanos-maintainers
Any discuss result for this?

AFAIK there's still no consensus here because

We can discuss it on our team sync.

still hasn't happened. My 2 cents: I only deploy certain components of Thanos on K8s so it would be a bit harder for me to maintain them.

I am afraid we negative on this side.

I personally also don't see the harm in having 2 helm charts. The two authors could join forces and effectively make it the default choice, without the thanos maintainers having to bless anything when we don't even use it.

update

Cool, I will put this on the agenda for next Thanos Maintainers Sync.

any process about this?

Hi @daixiang0
Yes! We discussed this together with the team on our maintainers meeting 12th Feb. We decided together that we can consider maintaining the official helm chart IF there would be a person that will be happy to maintain it AND will meet a bar of being official Thanos maintainer. Since for now, we don't have such a person, we can't reliably maintain it.
Another alternative is to use jsonnet to generate helm charts. This option was for now discarded as it puts another API restrictions on the jsonnet itself which we would like to avoid.
In the meantime, there are many very good helm charts available made by community. It would be awesome if the community would join forces to merge many charts into one. We are happy even to organize single repo for it e.g https://github.com/orgs/thanos-community org that will be maintained by the community, not us.
If you really want to improve helm + Thanos world @daixiang0, do you think you could reach the authors of those (at least 3) Thanos helm chart with this proposition (to combine forces)? You can reference this comment here. (:

@bwplotka Oh, it is a really big challenge for me, there are there helm chart repos for thanos and one for thanos-operator(check by https://hub.helm.sh/charts?q=thanos), and i am familiar with none of them. I would file an issue for each one and reference your comment.
Maybe we can have an initial version and invite them as maintainer?

The issue with clear explanation is enough. (:
Let's see what their suggestions are first, if they are happy to merge
their version into one under community project. They have to be independent
and motivated to work together. (:
As Thanos maintainers we are happy to be moderators/mentors and advices in
terms of deployment etc if needed.
On Wed, 19 Feb 2020, 08:16 Xiang Dai<EMAIL_ADDRESS>wrote:

@bwplotka https://github.com/bwplotka Oh, it is a really big challenge
for me, there are there helm chart repos for thanos and one for
thanos-operator(check by https://hub.helm.sh/charts?q=thanos), and i am
familiar with none of them. I would file an issue for each one and
reference your comment.
Maybe we can have an initial version and invite them as maintainer?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thanos-io/thanos/issues/1820?email_source=notifications&email_token=ABVA3O7DZCJAQESMEFW56NLRDTTFJA5CNFSM4JTPSZTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMGZDLY#issuecomment-588091823,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVA3O7TU2EDLLTCFIMAPTDRDTTFJANCNFSM4JTPSZTA
.

Awesome! Thanks for this. :heart:
Looking forward for response from other helm charts as well. :muscle:

Maybe not host by this repo, i would like to add thanos into helm/charts if possible.

Helm/charts will be deprecated soon. See:
https://github.com/helm/charts/blob/master/README.md#status-of-the-project

Hi!
First of all thanks for the effort to help unify the deployment of Thanos. I'm always glad to see when a project organizes itself to be more productive. ;)
As it is already well known the official stable Helm repository will be decommissioned at the end of the year. To be honest we were not a big fan of that monolith repository because it slowed down the collaboration of individual charts. That is one of the reasons why we started our own Banzai Cloud's stable repository as well.
Banzai Cloud provides Kubernetes solutions (as a platform) where amongst other things monitoring is included as well. To be more transparent we are doing as many things as possible as an open-source project. Basically, our charts already work as community charts (with focusing on testing, upgradability, etc...). We are more than happy to help the effort of the official chart, however, we have to consider our customer's requirements as well.
To add a bit more information regarding the Thanos deployment we already started another project called Thanos Operator. We believe the operator approach is more consistent and flexible in the long run.
As @carrodher proposed we can craft a proposal or schedule a call and ping me anytime if you have questions.

From my POV it would be great to have a single chart or somthing like a operator to install thanos and from kiwiwgrid side(my side) we can also support to to reach this goal.
From my side it would be great to support you.

Ok, let's decide something then. Wonder if some common Google Docs make sense. (: I understand it will be hard or even impossible to merge different helm charts... but maybe it's worth to try? :thinking:
We can definitely create a new project and add anyone who wants to help maintain, but we might need to have first some agreement/design of how merge can happen.

I'd be happy to help with any helm-chart work :+1:

Is there already a repo available for it?

Not yet, happy to create ASAP
On Fri, 17 Apr 2020 at 15:08, André Bauer<EMAIL_ADDRESS>wrote:

Is there already a repo available for it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thanos-io/thanos/issues/1820#issuecomment-615265164,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVA3O2IT7FP2ITOJR5OC4DRNBPHNANCNFSM4JTPSZTA
.

But we might need some idea how we go forward technically with the content
of such repo. Any ideas?
On Fri, 17 Apr 2020 at 16:45, Bartłomiej Płotka<EMAIL_ADDRESS>wrote:

Not yet, happy to create ASAP
On Fri, 17 Apr 2020 at 15:08, André Bauer<EMAIL_ADDRESS>wrote:

Is there already a repo available for it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thanos-io/thanos/issues/1820#issuecomment-615265164,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVA3O2IT7FP2ITOJR5OC4DRNBPHNANCNFSM4JTPSZTA
.

the banzai chart is the most comprehensive and up to date chart ive seen for thanos so far, that might provide a good base?

Hmm... I guess we should compare the charts and use the one with the richest futureset as base?
If we deprecate the other charts some automatic upgradepath would be cool too. If not possible we should at least document how to do it manually.
I'd also like to see Github actions ci in the later repo.
Maybe we should start with the repo and the ci pipeline?

@monotek you mean creating the repo (under thanos or thanos-community?) and start designing the CI pipeline?
We could document the comparison between charts in repo

Yes, that's what I meaned.
Documenting the decision makes sense too :-)

Amazing!
Let's start up then :muscle:

I started placeholder repo: https://github.com/thanos-community/charts
.. and Google Doc for collaborations: https://docs.google.com/document/d/18GXxwOm9c2fDK20LV1Sekte0xvK4qXpIGwBcuw_ZYSA/edit#
I described initial information and goals and at this point, I am totally removing myself from this doc. I will only suggest things if needed.
This doc is meant to be community-driven*, so I added @spencergilbert @tarokkk as editors (active on this issue). Looks like @monotek you have some nice ideas, so please let me know your email as well (could not find), will add you if you want.
Please DM us if you want to be added as editor and help to shape this project as well! :heart:
Also: Anyone can SUGGEST changes to this doc as well. :hugs:
Help wanted!

Thanks! :)
I would call the repo helm-charts, just to make clear, what kind of charts are saved there...

Good feedback, if there are no objections will rename today. (:
On Mon, 20 Apr 2020 at 11:39, André Bauer<EMAIL_ADDRESS>wrote:

Thanks! :)
I would call the repo helm-charts, just to make clear, what kind of charts
are saved there...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thanos-io/thanos/issues/1820#issuecomment-616464380,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVA3O22FGLLXOTKNJHPDTTRNQQ7PANCNFSM4JTPSZTA
.

Nice and specific :+1: looks good to me

pipeline is merged into the repo: https://github.com/thanos-community/helm-charts/pull/1
theoretically we cant start adding the chart.
any preferences which chart should be used as the base for the new chart?
if so, why?

This is exactly the thing we need to discuss. Since no one wants to start, just propose something @monotek and the doc, and let's see how we can iterate over it :hugs:
cc @spencergilbert @tarokkk @carrodher

Sorry @monotek @bwplotka personal life has been crazy for me the past couple days. I think if we could start a proposal in the doc we can start listing pros/cons of charts and see what matches our expectations the best

We rely heavily on community charts for our Thanos setup. Unfortunately, none of the charts I've evaluated fits our needs perfectly. So I would be happy to share some thoughts as well as to help build the new one-chart-fits-all thing.

@alexgaganov-fiverr could you add your thoughts to this google doc? https://docs.google.com/document/d/18GXxwOm9c2fDK20LV1Sekte0xvK4qXpIGwBcuw_ZYSA/edit#
I rolled my own chart to get our thanos usage off the ground, so I haven't worked with the existing charts much. It would be great to get more user feedback before we start implementing anything

@spencergilbert are you happy to moderate that doc? 🤗 I think you have edit access.

I'll do my best - hopefully we can grab a few more people though :smile:

Hi, :wave:
As per discussions on our public Thanos meeting I created #thanos-helm-charts slack channel in CNCF Slack.
Anyone is welcome to join! @tarokkk @alexgaganov-fiverr @spencergilbert @monotek @rpahli @carrodher @daixiang0 :hugs:
We are slowly moving with our Google doc content for official Thanos helm chart, so feedback is welcome!

Question
I saw that prometheus community tend to use jsonnet(vs helm) to deploy in k8s, so they created the kube-prometheus project as the main community deployment method.
So I wonder what is the official deployment method of thanos on k8s? is it kube-thanos or this new helm chart repo?

There is no single official one deployment model. It's really up to
whatever your generally use for your other workloads. It's just nice if
every popular deployment way has some good support from Thanos community.
(:
On Sun, 17 May 2020, 14:13 shay-berman<EMAIL_ADDRESS>wrote:

Question
I saw that prometheus community tend
https://github.com/coreos/prometheus-operator/issues/3169#issuecomment-624554249
to use jsonnet(vs helm) to deploy in k8s, so they created the
kube-prometheus https://github.com/coreos/kube-prometheus project as
the main community deployment method.
So I wonder what is the official deployment method of thanos on k8s? is it
kube-thanos https://github.com/thanos-io/kube-thanos or this new helm
chart repo?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thanos-io/thanos/issues/1820#issuecomment-629796044,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVA3O24XVRN3UTHIDD47FDRR7PITANCNFSM4JTPSZTA
.

We need to get back to this (:
On Tue, 16 Jun 2020 at 21:44, stale[bot]<EMAIL_ADDRESS>wrote:

Hello 👋 Looks like there was no activity on this issue for last 30 days.
Do you mind updating us on the status? Is this still reproducible or
needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity for next week, this issue will be closed (we
can always reopen an issue if we need!). Alternatively, use remind command
https://probot.github.io/apps/reminders/ if you wish to be reminded at
some point in future.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thanos-io/thanos/issues/1820#issuecomment-645002022,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVA3O765ULAEY7V4HJR233RW7KULANCNFSM4JTPSZTA
.

Sorry for playing dead.
Not so much time for this at the moment.

Yeah, as soon as I got out from under a bunch of job work - sudden personal life problems pop up :( I'm still around and want to help

still in process.

Still in progress, we are slowly adding more stuff to
https://github.com/thanos-community/helm-charts
Help wanted (:
On Wed, 19 Aug 2020 at 04:00, stale[bot]<EMAIL_ADDRESS>wrote:

Hello 👋 Looks like there was no activity on this issue for last 30 days.
Do you mind updating us on the status? Is this still reproducible or
needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity for next week, this issue will be closed (we
can always reopen an issue if we need!). Alternatively, use remind command
https://probot.github.io/apps/reminders/ if you wish to be reminded at
some point in future.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thanos-io/thanos/issues/1820#issuecomment-675822771,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVA3O5ZDA3IFS3T2N2RV5TSBM56DANCNFSM4JTPSZTA
.

What's blocking us from starting with one of the charts available now, using the most recent thanos version, by voting for it?

@bwplotka I haven't been able to dedicate a lot of time on this lately, and there doesn't seem to be any other activity. Is the support/desire still there?

I think there's still interest generally speaking. :)

The desire is there and would be happy to help support.  🙂

:+1: Good to hear, I was feeling lonely :smiley:
I've swung back around and started to update the query PR I had opened https://github.com/thanos-community/helm-charts/pull/4

As there is no real progress i guess it's not that important which chart we would use to start.
Still think we should just vote.

Honestly I think that's probably for the best @monotek @bwplotka. I've been busier than I expected and haven't been able to spend tons of time on it, and there hasn't been too much work from anyone else.
At this point, if there is still a desire to have an endorsed set of charts - it seems like it should come from an existing chart.

Thanks, all @spencergilbert @monotek for your amazing work!
Overall this project was some kind of experimentation.

Can we create together charts that are placed in "official' place?
Does community have time and focus to get on this together? Is it important enough?
Does maintainers of other charts have enough interest and time to help move their chart into "official place". Does it even matter if those works?

We tried and especially kudos to @spencergilbert who already spent a lot of time on this attempt and produce few good charts.
I personally have some learnigs from this:

2020 is hard

Packaing open source software in 2020 is still incredible hard and important. Helm charts are not solving this problem because:

OS projects has to literally reimplement (and maintain!) most of their configuration flags + kubernetes flags to produce chart. Document it etc. So double work.
Even if this is done, this does not solve all use cases: People who don't use Helm (majority), People who don't use K8s, people who needs custom options.

While Helm community did amazing work, I don't think it's fair to force every project to produce and maintain it's chart. It's ok if they have $$$ and time to support this, but Thanos does not have. I think we tried enough to encourage community to collaborate one if it's needed, but looks like it's not an important enough thing 🤗
Thanks again, and still, let's collaborate together on  https://github.com/thanos-community/helm-charts if anyone has time. For now I will close this issue unless there are objections. (:

Personally I think we should work together with other CNCF project on some other tooling that will be:

takes simple, understandable, typed, documented, ideally autogenerated from flags input
extensible
can output:

pure YAML to apply
generate helm chart
generate k8s operator
generate salt, ansible, chef, terraform etc

Business idea for 2021 🔥

Personally, I think we should work together with other CNCF project on some other tooling that:

takes simple, understandable, typed, documented, ideally autogenerated from flags input

will be extensible (if something is not in the input, you can add some custom flags yourself)

can output:

pure YAML to apply
generate helm chart
generate k8s operator
generate salt, ansible, chef, terraform etc

Business idea for 2021

Definitely a million dollar idea :smile:


Add files via upload

@iamvanca Thank you for your patience while waiting for a review and processing your request. Pull request merged.

@philipto Thank you!


The name of PR is too short if use 'kotlin()'

Searched with: jvm, kotlin

Package manager/ecosystem gradle gradle.kts

Manifest contents prior to update build.gradle.kts

Updated dependency "org.jetbrains.kotlin.jvm" (in PR, "jvm")

What you expected to see, versus what you actually saw
expected "Bump org.jetbrains.kotlin.jvm from 1.4.21 to 1.4.30", but was "Bump jvm from 1.4.21 to 1.4.30"

Images of the diff or a link to the PR, issue or logs

PR: https://github.com/anatawa12/bintray2central/pull/1
You may think it's expected behavior, but I think full name is better.

I see it both ways, as it's a kotlin plugin, the name is just jvm, but the message is not clear it's a kotlin plugin, and as you said, it is updating org.jetbrains.kotlin.jvm behind the scenes. 🤷
Can always do it the long way and swap it to id('org.jetbrains.koltin.jvm') and it'll be the long name. :D

well it should support it in my opinion.
Maybe doesn't have to be the WHOLE plugin id, but at least something like kotlin-jvm would be helpful.

This is a tricky one, as @busches notes there's tradeoffs here.
I don't think it's something we're going to invest time into changing, but if you feel strongly enough to open a PR happy to review/have a broader discussion about it...


Investigate using Circe instead of spray-json

This includes looking at Rapture support for Circe which seems to be available in version 2.

FYI Looks like Circe now has a bridge for Spray:
https://github.com/travisbrown/circe/releases/tag/v0.4.0

@ctataryn, as far as I can tell Circe supports JSON marshalling for spray the HTTP library – not to be confused with spray-json. That's not interesting for this project which uses Akka HTTP.

@hseeberger Ahh I see, sorry! I arrived here after a google search for circe + spray somehow, saw your name and just assumed this was spray related :)


Allow updating docker digests without rebuilding

This is useful when the digests somehow get out of sync, as happened in
https://github.com/iree-org/iree/pull/9983.
Also updates quote style to be consistent with all our other Python
stuff.

Better names for the flag welcome. The first commit updates quote style to be consistent with basically all our other Python files. I can split that into a separate PR if people want

This change doesn't affect CI. Merging past it (man we really need a way to disable this...)


Does not handle Mac OS X / macOS cleanly due to incompatible getopt

Ideally we should test for getopt flavour, and provide a solution using e.g. the homebrew-installed version of gnu-getopt, with a prompt to tell the user how to install?

Happy to work on this in due course :)


add clobber option to load function

Default behavior is to set all environment variables in dotenv file. This would make it possible to not overwrite existing variables.

I support this!
Would like to see my existing PR merged first since one of these PRs will now have to deal with resolving merge conflicts

@cmc13 please rebase and I will merge :)

You can also flatten all the commits into one, once you fix the last thing

@rogusdev no need for the whole flatten commits thingy. You can squash during merge

@tonerdo yes but if the author does it first, then they get to pick the commit message they want -- if I do it, it just uses the PR title which might be less ideal sometimes.  Either way is fine, I just encourage the flatten be done first.


Maor/lbyaml v2

@maor-lb just making sure you are aware, this repo is public and open to the world.
I'm not opposed to the lbyaml v2 changes, but we have to make sure the discovery-client remains buildable without it as well for non-LB consumers.

@maor-lb should this PR remain open? wip? draft? closed?

@maor-lb should this PR remain open? wip? draft? closed?

I'm closing and opening new PR which remove dockerize dependency for build


Market tag separator implies hierarchy

@kevinreilly commented on Thu Aug 16 2018
I propose we change the separator in between market tags from a / to a ,. The tags currently look like they are hierarchical when they are actually siblings.
Current separator:

Proposed separator:

@bthaile commented on Thu Aug 16 2018
Discussing this.
https://app.clubhouse.io/augur/story/15534

@sharkcrayon commented on Fri Aug 17 2018
@matt-bullock Opinions on this? I agree that the slash may be implying a hierarchy that's not there. But I'm not a fan of the comma, at least with the current design.

@bthaile commented on Mon Aug 20 2018
How about a dash -
from: TAGS / BITCOIN / BTC
to: TAGS / BITCOIN - BTC
or I like a dot, I dig the dot!
to: TAGS / BITCOIN ∙ BTC

@matt-bullock commented on Mon Aug 20 2018
Tags need more thought. There are better ways to visually improve things here.
Do we really need to use the word 'Tags' and 'Category'? We could use visual indicators to show they are tags or categories. Could be icons, could be the way they are styled. This results in less clutter.
I'd rather not do a quick change here as there should be a correlation between the way the tags and categories are shown in the menu and how they are shown on the market cards. And of course our whole menu design needs some rethinking.

Closing as a dupe of https://github.com/AugurProject/augur/issues/298


Upgrade Crossplane AWS provider Go SDK for EKS pod identity support

Description of your changes

This pull request updates the Crossplane provider-aws project to use the latest version of the AWS SDK for Go v2, specifically v1.23.0 or later. This change is necessary to leverage the new EKS pod identity feature introduced in the AWS Go SDK V2. By upgrading to the latest AWS SDK for Go v2, the Crossplane provider-aws project can take advantage of the new EKS pod identity feature.
EKS Pod identity launch link
Related Issues #1981 #1972
I have:

[x] Read and followed Crossplane's contribution process.
[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

any chance to get this merged and integrated for podidentity to work?

any chance to get this merged and integrated for podidentity to work?

We should be able to merge this once the linter issues are fixed.

Superseeded by https://github.com/crossplane-contrib/provider-aws/pull/2095


contain function might not suitable to assert that "local node is in partitionTables"

Describe the bug
After updating 0.12.4 to 0.13.0, I make a mistake and run start-server rather than start-node.
Then I see an error: All nodes in partitionTables must contain local node in start-server mode.
Which appeared in line 416 https://github.com/apache/iotdb/blob/ac51b0802ac5d016f028fbfe829f8b8ab530740e/cluster/src/main/java/org/apache/iotdb/cluster/ClusterIoTDB.java#L411-L423
I checked the error log and find that the node has only one parameter which is different from nodes in metaGroupMember, nodeIdentifier.
In https://github.com/apache/iotdb/blob/ac51b0802ac5d016f028fbfe829f8b8ab530740e/cluster/src/main/java/org/apache/iotdb/cluster/utils/ClusterNode.java#L56-L62
I got nodeIdentifier is not parameter which should be the same. And I cannot find any place to change it, it's used for thrift. But is it necessary to keep the parameter nodeIdentifier the same?
If not, contain function might be rewritten。
To Reproduce
Am not really make sure this process will reproduce but am sure to get this error with the following procedure.

run start-cluster, then close it
run start-server, then close it
run start-cluster
See error

Expected behavior
This node should have been contained in all nodes, although nodeIdentifier is different.
Desktop (please complete the following information):

OS: CentOS7
Version 0.13.0

Hi, thanks for the detailed report and diagnosis.
We are refactoring the cluster module and this issue will be solved in the new implementation.
(If you are interested, welcome to join us to the new implementation, or welcome to have a try to fix this issue in v0.13)


Selected environment takes time to get reflected in the status bar

From @bamurtaugh in https://github.com/microsoft/vscode-python/issues/17110#issuecomment-908534842

Going from the timestamps in the logs, it took 1.5 minutes for the extension to go from the Python path in python.defaultInterpreterPath to auto-selecting the virtual environment in the workspace
Seems like it took even longer to update the status bar with the newly selected environment

It's very likely because discovery was blocking the activation of the rest of the extension, this is no longer a problem with https://github.com/microsoft/vscode-python/issues/19102.


Get a shorter name for the pages

In mobile devices, the long name usually looks really ugly.

It is more important that it is readable and memorable than anything else.

I consider this completed.


Cow: Watch +1-888-236-1460

Cow requests the watch of the watch_number +1-888-236-1460. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ


Enrich documentation for inference

What does this PR do?

This PR adds clearer explanations about how to run inference.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Show favourites or pinned bookmarks on Home screen

I think a way to pin bookmarks, so that they are shown as symbols icons when a new tab is opened would be a great workflow addition

Have you solved this problem? What do you mean "as symbols". Site icons?

Not a problem, a feature request ... Yes i mean icons, symbols may be a misleading term. Of course with the site name as text. Opera for android e.g. does it in a very convenient, customisable way

Have you solved this problem? What do you mean "as symbols". Site icons?

Not a problem, a feature request ... Yes i mean icons, symbols may be a misleading term. Of course with the site name as text. Opera for android e.g. does it in a very convenient, customisable way

Thanks @neatwheat I think the idea you are suggesting is pretty similar to the concept of Favourites we have on our iOS app.
On the iOS app, we show a grid showing user's favourites on a new tab.

is that what you wanted to describe on your feature request? Thanks!

@cmonfortep
Yeah, that looks like it's pretty much what I meant! I think it would be a great feature that could improve usabilty a lot - especially if someone can choose their own favorites!

We created a task to explore this internally: https://app.asana.com/0/715106103902962/1198217253840085/f

This one is coming soon. We are currently working on a project to add favorites in the same way as the iOS works.
Closing this one as it's currently in progress.

What's the status here? In DDG v5.102.3 this feature seems still missing.


Fix the tcnative lib loading problem in OSGi

Motivation:
As the issue #5539 say, the OpenSsl.class will throw java.lang.UnsatisfiedLinkError: org.apache.tomcat.jni.Library.version(I)I when it is invoked. This path try to resolve the problem by modifying the native library loading logic of OpenSsl.class.
Modifications:
The OpenSsl.class loads the tcnative lib by NativeLibraryLoader.loadFirstAvailable(). The native library will be loaded in the bundle netty-common's ClassLoader, which is diff with the native class's ClassLoader. That is the root cause of throws UnsatisfiedLinkError when the native method is invoked.
So, it should load the native library by the its bundle classloader firstly, then the embedded resources if failed.
Result:
First of all, the error threw by native method problem will be resolved.
Secondly, the native library should work as normal in non-OSGi env. But, this is hard. The loading logic of Library.class in netty-tcnative bundle is simple: try to load the library in PATH env. If not found, it falls back to the originally logic NativeLibraryLoader.loadFirstAvailable().
BTW: this path depend on netty-tcnative-1.1.33.Fork19 version (with Bundle-NativeCode manifest entry added).
Signed-off-by: XU JINCHUAN<EMAIL_ADDRESS>

Actually, in my personal view, the NativeLibraryLoader.load will NOT work properly in OSGi. Because the ClassLoader which to load the native library is belong to netty-common bundle, NOT the ClassLoader of the native library.
So, if the netty would be compatible with OSGi, we have to customize the ClassLoader of native bundles (I have no idea about that), or we have to modify the NativeLibraryLoader -- rather than calling the System.load() or System.loadLibrary() directly, it should return the native library absolute path and let the caller invoke the System.load() with the absolute path. Something like this:
// Find/extract the embedded native lib. Or return the libname if not found.
String libPath = NativeLibraryLoader.prepareLibrary(libname);
System.load(libPath);  // Load the native lib into current ClassLoader.

@xsir I think the second idea would be ok... Are you interested to provide a PR ?

@normanmaurer I have considered the second idea for a while. The solution has a drawback: the callers of NativeLibraryLoader have to be modified to satisfy the new approach, and the NativeLibraryLoader signature will be changed too.
I have a new idea, but it has a little bit tricky:

Defines a new a simple helper Class, which is a wrapper for System.load() and System.loadLibrary(), will be injected into the native library's ClassLoader in step 2(a);
When loading the native library, will find the helper Class in the native library's ClassLoader at first:
a) if not found, turn the helper Class into a byte[] and define in the native library's ClassLoader, then will get a NEW helper Class instance;
b) else, use the helper Class instance;
Use the helper Class instance to load the native library.

That is, the native library will be loaded in propitiate ClassLoader! Ant this approach has a little side effect to non-OSGi environment application (in the step 1, the helper will be found cause it has been loaded).
I am not sure it is a good solution. What do you think?
I will submit a PR soon, thank you!

You might be able to convince @normanmaurer changing method signature in NativeLibraryLoader is ok because it is internal :)

@johnou Thank you, I just trying to make a better solution, make the NativeLibraryLoader working properly in both OSGi and non-OSGi env.
@normanmaurer I just submit a new commit, please review it and thanks.

The new patch leave the caller of NativeLibraryLoader untouched, such as OpenSsl.class in handler module.

@Scottmitch @normanmaurer The new commit trying to make the NativeLibraryLoader working properly in both OSGi and non-OSGi env. That will add a little bit overhead: find class, define the helper class into the library's ClassLoader if NOT found.
Another solution is to modify the caller process of NativeLibraryLoader, just as OpenSsl.class do in my first commit -- the modification is up to the library's loading helper, the Library.initialize(name) in tcnative for example.
So, as you see, I think the former solution is better: it hidden the detail of platform into itself, this make the caller simpler. Thanks!

@xsir - few more comments and lgtm ... good work.
@normanmaurer - assigned to you ... plz pull in when satisfied

@xsir can you please explain to me again why we need to use reflection hacks for this ? is there no other way ?

Actually, there has another solution to this issue, I have explained to @Scottmitch (forgive my poor English, btw):

netty-tcnative case, which provided a loading helper class org.apache.tomcat.jni.Library. Just like #9f2a213, we should modify OpenSsl, instead of calling the NativeLibraryLoader, trying the helper class first (Depends on a properly Bundle-NativeCode manifest entry). The helper class Library in netty-tcnative does NOT support loading library by absolute path.
netty-transport-native-epoll case, we should modify Native, instead of calling the NativeLibraryLoader, trying the System.load() first (Depends on a properly Bundle-NativeCode manifest entry).

This solution is more simple, but there has a drawback: the library loading process is diff between OSGi and non-OSGi. In the OSGi, the library in $PATH has a higher priority than the one in the bundle. The non-OSGi is just the reverse. That's why I submitted a more complicated solution:)
Thanks for all of the comments!

Got it... I will wait for @jasontedor to comment :)

@normanmaurer ACKing your ping, still catching up after being on vacation last week. I'll try to take a look on Friday if that's acceptable to you?

@jasontedor sure thing... take your time and thanks!

Squashed and cherry-picked into 4.1 (00f74b92fa4e10e26ea8b99f2bb95681085a3dc8) and 4.0 (cca236a8b22095b887e0877d77a4b274805f0d0d)
@xsir thanks a lot!


Fixing paddings in program settings page and cleaning up html

Critical Changes
Changes
Issues Closed
New Metadata
Unpackaged Metadata
Deleted Metadata
Testing Notes

Tested this by building a scratch org and the "New" button for auto enrollment mappings under program enrollment is properly aligned now. QA passed


MinTrace fails for zero-inflated Time Series with res_methods in ['wls_var', 'mint_cov', 'mint_shrink']

What happened + What you expected to happen
Applying MinTrace with a res_method will fail due to the following error:
ValueError: Insample residuals close to 0, zero_residual_prc=1.0. Check Y_df``
How can you overcome this? It's probably because some time series has many zeros.
Versions / Dependencies
0.3.0
Reproduction script
# !pip install -U numba statsforecast datasetsforecast
import pandas as pd

# compute base forecast no coherent
from statsforecast.core import StatsForecast
from statsforecast.models import AutoARIMA, Naive

#obtain hierarchical reconciliation methods and evaluation
from hierarchicalforecast.core import HierarchicalReconciliation
from hierarchicalforecast.methods import BottomUp, TopDown, MiddleOut, MinTrace

#obtain hierarchical datasets
from datasetsforecast.hierarchical import HierarchicalData

# Load TourismSmall dataset
Y_df, S, tags = HierarchicalData.load('./data', 'TourismSmall')
Y_df['ds'] = pd.to_datetime(Y_df['ds'])
Y_df['y'] = 0.0

# Compute base level predictions
sf = StatsForecast(df=Y_df,
                   models=[AutoARIMA(season_length=12), Naive()],
                   freq='M', n_jobs=-1)

forecasts_df = sf.forecast(h=12, fitted=True)

# Reconcile the base predictions
reconcilers = [
    MinTrace(method="mint_shrink")
]

hrec = HierarchicalReconciliation(reconcilers=reconcilers)

reconciled_forecasts = hrec.reconcile(Y_df=sf.forecast_fitted_values(), Y_hat_df=forecasts_df, S=S, tags=tags)

Issue Severity
Medium: It is a significant difficulty but I can work around it.

Hey @baniasbaabe,
You might want to force it using a bit of Gaussian noise data augmentation.
See the FavoritaHierarchicalDataset.load_item_data example:
https://github.com/Nixtla/hierarchicalforecast/blob/main/experiments/hierarchical_baselines/nbs/run_favorita_baselines.ipynb
We have specialized hierarchical methods on zero inflated processes here:
https://nixtla.github.io/neuralforecast/examples/hierarchicalnetworks.html
The paper of a HINT-related method recently got accepted in the International Journal of Forecasting.
Kin G. Olivares, O. Nganba Meetei, Ruijun Ma, Rohan Reddy, Mengfei Cao, Lee Dicker (2023).”Probabilistic Hierarchical Forecasting with Deep Poisson Mixtures”. International Journal Forecasting, accepted paper. URL https://arxiv.org/pdf/2110.13179.pdf.

Many thanks for the fast reply, I will look into it!