ModuleNotFoundError: No module named 'official'

I'm trying to train a mbt model. However, I got the following error:
Traceback (most recent call last):
File "/home/eftekhar/anaconda3/lib/python3.9/runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/home/eftekhar/anaconda3/lib/python3.9/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/home/eftekhar/models/scenic/scenic/projects/mbt/main.py", line 47, in
app.run(main=main)
File "/home/eftekhar/models/scenic/scenic/app.py", line 65, in run
app.run(functools.partial(_run_main, main=main))
File "/home/eftekhar/anaconda3/lib/python3.9/site-packages/absl/app.py", line 308, in run
_run_main(main, args)
File "/home/eftekhar/anaconda3/lib/python3.9/site-packages/absl/app.py", line 254, in _run_main
sys.exit(main(argv))
File "/home/eftekhar/models/scenic/scenic/app.py", line 100, in _run_main
main(rng=rng, config=FLAGS.config, workdir=FLAGS.workdir, writer=writer)
File "/home/eftekhar/models/scenic/scenic/projects/mbt/main.py", line 34, in main
dataset = train_utils.get_dataset(
File "/home/eftekhar/models/scenic/scenic/train_lib_deprecated/train_utils.py", line 280, in get_dataset
dataset_builder = datasets.get_dataset(dataset_name)
File "/home/eftekhar/models/scenic/scenic/dataset_lib/datasets.py", line 144, in get_dataset
return DatasetRegistry.get(dataset_name)
File "/home/eftekhar/models/scenic/scenic/dataset_lib/datasets.py", line 107, in get
importlib.import_module(module)
File "/home/eftekhar/anaconda3/lib/python3.9/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1030, in _gcd_import
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "", line 850, in exec_module
File "", line 228, in _call_with_frames_removed
File "/home/eftekhar/models/scenic/scenic/projects/vivit/data/video_tfrecord_dataset.py", line 18, in
from scenic.dataset_lib import video_ops
File "/home/eftekhar/models/scenic/scenic/dataset_lib/video_ops.py", line 35, in
from official.vision.image_classification import augment
ModuleNotFoundError: No module named 'official'

I encoutered the same problem.

@parhameftekhar @Claire874
I ran into the same problem. So I did the following to solve the problem.

pip install tf-models-official
Modify the import -> from official.legacy.image_classification import augment

Please try it out.

@satojkovic Thanks solved the issue. However, a new one came up. I close this issue.

@satojkovic
were you able to run the code successfully? if so could you share your config file?

@parhameftekhar
Hi, Could you please share your config file? I have some problems with optimizer.


237 behavior node dummy client for send plan to motion planning

@yunlong-wang-cn
As we discussed, here is a dummy node with that you can publish the audio input/commands and the vision input. :+1:
You should just launch the new launch file and the new node separately.
I still have some bug that the behavior node dies, I try to solve that.
However, you can still give it a try and look whether it works already.
Was I correct that you want to simulate the vision and command inputs?
@scaredycode Do we "need" the other launch file "test_behavior.launch" or was it just for local / one-time-use testing? :)

I close this for now until a response from @yunlong-wang-cn

@Juphex which response do you means? I think I already give approvement of this PR.


In Enterprise Wallet, the main screen title doesn't match with highlighted side menu options

As shown in image below, on selecting Product as main screen, Batch is still highlighted in bold font on side menu bar. Both should be in sync ideally.

Iterations:
[1] https://github.com/webcardinal/webcardinal-core/commit/8f34f123313e56271ef141a4889cb33244fb61d4
[2] https://github.com/webcardinal/webcardinal-core/commit/a502351fb186f515f75fe12d2b00ab99f0a6a469
Modifcations were mainly made in webc-app-menu and webc-app-router components of WebCardinal.


WIP: Alby Boost Button

Describe the changes you have made in this PR
A first version of the boost button, enabled for Youtube.

BoostButton component written in react (styles are applied inline and won't affect injected pages)
Uses web components (but no shadow DOM)
Migrated 2 Youtube Batteries into one (we have to watch the DOM for changes)
New window.alby.lnurl() to retrieve invoices from LNURLs (migrated existing actions from keysendto webln/keysend to handle the different messages respectively)
Created new inject script which is limited to the top frame (otherwise the button e.g. would be displayed in every iframe)

Type of change (Remove other not matching type)

feat: New feature (non-breaking change which adds functionality)

Open tasks

[ ] Integrate inpage-components into build (or put into a separate repository & use npm?)

Checklist

[ ] My code follows the style guidelines of this project and performed a self-review of my own code
[ ] New and existing tests pass locally with my changes
[ ] I checked if I need to make corresponding changes to the documentation (and made those changes if needed)

Summary of the conversation with @reneaaron

web components with shadow dom (didn't work because i.e. YT is messing up the shadow dom usage)
external component without shadow dom (didn't work because of CSS build and possible CSS interference from the host-site)
button within extension with simple UI (hard to maintain and possible CSS interference from the host-site)
iframe (possible issue with payment-origin)


Gadt translation can notice some parameters

Closes #129
With this pull request I also start a big revamp of TypeIsGadt.ml to provide a much better interface. The key idea is to stop using Name.t list option in favor of InductiveVariable.t list, this way the datatypes constructors in TypeDefinition will have direct access to all the parameters and indexes, even if we are going to erase them later. Notice that in the future we will still need all this information stored somewhere in order to be able to fully translate Gadts.
It seems to me that TypeIsGadt should be a module to manage all the treatment of TypeVariables of inductives, with Name.t list option the clients of this module will still have to figure out how to treat this information.
gadt_shape is the same as check_if_not_gadt, but still using InductiveVariable.t option list because I didn't update how Type.non_phantom_typs works, and it still relies on option list. In the future I will simplify Typ.non_phantom_typs to take full advantage of the new interface that I am building in TypeIsGadt.ml.

Top for the PR! There are some errors with the tests, looking into it.

Don't worry I think I know what's the issue.

I'll close this pull request and open a new one with cleaner commits.

OK 👌


Parameter settings for increased speed over 1.0m/s

There is a hard-coded speed limit.

https://github.com/CMU-cabot/cabot/blob/47d333d8d8116dcf2ba4e8a1ac33977b2804ad49/cabot/src/safety/tf_speed_control_nodelet.cpp#L84

add max_speed_ parameter to lidar and people speed control (default=1.0)  - for robot and simulation
change nav2_params.yaml
change menu limit

https://github.com/CMU-cabot/cabot/blob/47d333d8d8116dcf2ba4e8a1ac33977b2804ad49/cabot_ui/menu/menu.yaml#L90-L91

URDF simulator limits

With those changes, still, the simulator does not speed up faster than around 1.08m/s.
Navigation2 controller may need to be optimized for faster speed.

The following parameters for Navigation2 need to be updated to increase the max velocity.
-      prune_distance: 2.7 # default: 1.7 # should be greater than max_vel_x * sim_time + some margin
+      prune_distance: 5.4 # default: 1.7 # should be greater than max_vel_x * sim_time + some margin
-      max_vel_x: 1.0
+      max_vel_x: 2.0
-      max_speed_xy: 1.0
+      max_speed_xy: 2.0


Using groups in keyvault access policies causes destroy error

If I use a group to control my keyvault access policy, e.g.
azuread_groups = {
  keyvault_level0_rw = {
     ...
  }
}

keyvaults = {
  level0 = {
    ...
  }
}

keyvault_access_policies = {
  level0 = {
    keyvault_level0_rw = {
      azuread_group_key  = "keyvault_level0_rw"
      secret_permissions = ["Set", "Get", "List", "Delete", "Purge", "Recover"]
    }
  }
}

Then when running terraform(rover) destroy, I get a permission error. This happens because the principal is removed from the AD group before terraform attempts to clean up the secrets in the vault.
It would appear a depends_on is needed around the secrets and azuread modules, I've yet to find one that works and doesn't cause a circular dependency.

Thanks for the issue @brk3 - let us know if any new developments. Adding @SergioMiyama in case is needed.

Closing as resolved


Minor: Convert Count's name to lowercase

Which issue does this PR close?

Part of #10695
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Thanks @jayzhan211 I personally prefer function names to be uppercase, its more readable imho, but what is more important all of it has to be the same case to avoid a mess. Are you planning to convert all others to lowercase as well?

I think the case is being converted as part of the migration to UDAF -- for example https://github.com/apache/datafusion/pull/10964

Thanks @jayzhan211 I personally prefer function names to be uppercase, its more readable imho, but what is more important all of it has to be the same case to avoid a mess. Are you planning to convert all others to lowercase as well?

We can introduce config setting to have upper case for display, but lowercase for internal use (we get the function or compare with it's name)

Thanks @alamb and @comphead


Add Support for TCP GNSS Sensor

I added a new nmea_tcpclient_driver, because Sensors like the Emlid Reach RS2 do not offer any UDP connection, but transmit NMEA-Sentences on Port 9001/tcp. The nmea_tcpclient_driver connects to it via TCP and forwards the NMEA-Sentences line by line to the driver.
PS: Sorry about the commit-Messages, got a little exhausted during debugging ;)

Thanks for the contribution. I'll review the PR soon!

Also, I'm sorry it took so long for me to review this. Maintaining this package is a side hobby for me, and it often gets away from me.

I'm sorry it took so long for me to review this. Maintaining this package is a side hobby for me, and it often gets away from me.

No problem, I appreciate that you use your free time for maintaining this. It has been really useful for me during a project for university.

I have some requests and suggestions. Thanks for the PR!

I now implemented those changes.

Reverted the change


fluid.ampfeature~ helpfile improvements

Seems like a super missed opportunity to have the threshold visualization be dynamic:

Also, live.scope~ > snapshot~ + multislider, particularly if you are chasing onsets which may be missed by the 10ms snapshot~ window.

There's likely something I'm missing, but I mainly just mean for the helpfiles. So for the current fluid.onsetfeature~ having a multislider tied to the threshold parameter of the corresponding fluid.onsetslice~.

In the case of ampslice and noveltyslice you have some relatively known boundaries for the features but onsetslice varies wildly from some @metric giving you something nice in the range 0.0 -> 1.0 while others go off into infinity. So the notion of a threshold changes a lot, which means dynamic GUI's and potentially lots of confusing things to take care of.


SSH questions

Please provide more questions from SSH on the registration page, according to the doc provided with security, hints for SSH where to target, and categories of questions to ask.

Improve the streamlit template for the creation of controls on our API such that when SSH gives us many questions we can programmatically, data-driven, add them by reading from a CSV file or something.


Revert "Using VC main view disposer as VC Disposer"

This reverts commit 3bb25ef17d57b623c40c03a7ed6b548922d4c151.

Nice PR!


Unable to write high-precision decimal values to Delta table using serde_json/JsonWriter

Environment
Delta-rs version:
0.15.0 (also tried 0.16.1)
Binding:
Rust
Environment:

Cloud provider: any
OS: any
Other: n/a

Bug
What happened:
Writes began failing when attempting to insert high-precision decimal values into a Delta table using the JsonWriter with a Vec<serde_json::Value>.  Discovered serde_json was deserializing these values as strings in scientific notation which could not be parsed into the Arrow DecimalType:
Generic DeltaTable error: Failed to convert into Arrow schema: Parser error: can't parse the string value 3.9178294781e-6 to decimal
Some digging uncovered the serde_json feature flag "arbitrary_precision" which retains the value in its full form stored in string format, however this too cannot be decoded to an Arrow DecimalType:
Generic DeltaTable error: Failed to convert into Arrow schema: Json error: whilst decoding field 'decimal_col': expected decimal got {"$serde_json::private::Number": "0.0000039178294781"}
What you expected to happen:
High-precision decimal values be written accurately and successfully to a Delta table
How to reproduce it:
Cargo.toml
[package]
name = "decimal_issue"
version = "0.1.0"
edition = "2021"

[dependencies]
deltalake = "0.15.0"
serde = "1"
serde_json = { version = "1", features = ["arbitrary_precision"] } # remove feature to see original behavior
tokio = "1.33.0"

src/main.rs
use deltalake::{operations::create::CreateBuilder, writer::{JsonWriter, DeltaWriter}};

#[tokio::main]
async fn main() {
    let data = serde_json::from_str::<Vec<serde_json::Value>>(r#"[{"decimal_col": 0.0000039178294781}]"#).unwrap();
    let table = CreateBuilder::new().with_location("memory://").with_column("decimal_col", deltalake::SchemaDataType::primitive("decimal(38,16)".to_string()), true, None).await.unwrap();
    let mut writer = JsonWriter::for_table(&table).unwrap();

    match writer.write(data).await {
        Ok(_) => {},
        Err(err) => {
            eprintln!("{}", err);
            std::process::exit(1);
        }
    }
}

cargo run
More details:
Lower precision decimals (5 or less) do not have this issue.
Using arrow_json to parse the value into a RecordBatch and then using RecordBatchWriter instead of serde_json with JsonWriter works, however the problem here is other Delta log interactions such as create_checkpoint use serde_json behind the scenes, so when the stats are read from the logs to be written to Parquet checkpoints the same issue occurs.

Update:
There are several concerns going on here.  First, there are shortcomings in arrow causing issues with arbitrary_precision and scientific notation.  I have opened two feature requests in the arrow-rs project to address these:

https://github.com/apache/arrow-rs/issues/5068
https://github.com/apache/arrow-rs/issues/5069

Second, delta-rs is using f64 as a stand-in for decimals, causing precision loss.  I know Rust does not have a native decimal type, but this seems like a big oversight.  For now I've added the BigDecimal crate to a fork of this library.  If this seems like the right direction for delta-rs broadly I'm happy to clean it up and submit a PR to this repo.

@ryanaston - of course we always happy about PRs.
In this case we may have the challenge, that we need to be true to the delta protocol, which at most supports precision / scale up to 38.


Migrated to Swift 3 syntax

Note: Swift 3 is incompatible with Swift 2.

@franklinyu to be honest, I just needed AEXML to work in my Swift 3 project 😬 I know that there's a compatibility issue. If I understand https://github.com/tadija/AEXML/pull/70#issuecomment-224823348 correctly, the plan is to migrate master to Swift 3 when it's officially released.  I'll keep the PR updated and hope we can just merge this in September.

Note: after this being merged, consider adding Continuous Integration. It is said that Travis CI also support Swift (With Xcode 8 beta).

So c03943b9d737661bdc938e1b20048bfbb435c4f8 is purely performed by the migration wizard? Seems basically fine.

Thanks very much for your efforts on this one. It's finally merged now!


iOS: TrackPlayer.getState does not match current state

Configuration
System:
OS: macOS Mojave 10.14.6
CPU: (8) x64 Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
Memory: 334.72 MB / 16.00 GB
Shell: 5.0.2 - /usr/local/bin/bash
Binaries:
Node: 13.3.0 - ~/.nvm/versions/node/v13.3.0/bin/node
Yarn: 1.21.1 - /usr/local/bin/yarn
npm: 6.13.4 - ~/.nvm/versions/node/v13.3.0/bin/npm
Watchman: 4.6.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
Android SDK:
API Levels: 23, 24, 25, 26
Build Tools: 19.1.0, 20.0.0, 21.1.2, 22.0.1, 23.0.1, 23.0.2, 23.0.3, 24.0.0, 24.0.1, 24.0.2, 24.0.3, 25.0.0, 26.0.2
System Images: android-24 | Intel x86 Atom, android-24 | Intel x86 Atom_64, android-24 | Google APIs ARM 64 v8a, android-24 | Google APIs ARM EABI v7a, android-24 | Google APIs Intel x86 Atom, android-24 | Google APIs Intel x86 Atom_64, android-25 | Android TV Intel x86 Atom, android-25 | Google APIs Intel x86 Atom, android-25 | Google APIs Intel x86 Atom_64
IDEs:
Android Studio: 3.0 AI-171.4443003
Xcode: 11.3/11C29 - /usr/bin/xcodebuild
npmPackages:
react: 16.9.0 => 16.9.0
react-native: 0.61.5 => 0.61.5
react-native-track-player: ^2.0.0-rc13
Issue
I've only tested this on iOS so it may be on Android too, but I don't know.
The state returned from TrackPlayer.getState does not match the actual state of the player
When the event "playback-state" is sent, the state does not change. Usually it starts on idle and then stays on paused no matter if the player is actually playing.
let state = await TrackPlayer.getState();
Meanwhile the code block below state.rawValue looks more accurate to what I hear but still off. Sometimes it sends paused when its actually playing
player.event.stateChange.addListener(self) { [weak self] state in
   self?.sendEvent(withName: "playback-state", body: ["state": state.rawValue])
 }

I did notice that I have a warning Sending `playback-state` with no listeners. That goes away if I use the hook, but my component is stateful so I can't use a hook. What do I need to do to sync TrackPlayer.getState(); to match the new state change?
The code below expects to play a track if no track is currently playing otherwise it should stop playing the current track and start playing the new track if the state is Playing
play = async (location: Feature) => {
    const {name, trackId} = location.properties;
    let track = tracks[trackId];
    if (track) {
      let state = await TrackPlayer.getState();
      if (state === State.Playing) {
        await TrackPlayer.add(track);
        await TrackPlayer.skip(trackId);
        await TrackPlayer.play();
      } else {
        await TrackPlayer.add(track);
        await TrackPlayer.play();
      }
    }
  };

To test the hook usePlaybackState I used the code below and added a <StateTest /> to my component.
const StateTest = () => {
  const playbackState = usePlaybackState();
  console.log('playbackState', playbackState);
  return null;
};

Here are some logs:
FIRST TEST
Play song w/ code below:
await TrackPlayer.add(track);
await TrackPlayer.play();

 LOG  playbackState idle
 LOG  playbackState loading
 LOG  playbackState ready
 LOG  playbackState buffering
 LOG  playbackState playing -> `Song is actually playing`
 LOG  playbackState paused -> `Song did actually stop playing`
TrackPlayer.getState -> also paused

Play song w/ code below:
await TrackPlayer.add(track);
await TrackPlayer.play();

 LOG  playbackState playing
 LOG  playbackState paused
 LOG  playbackState loading
 LOG  playbackState paused -> `Song is actually still playing... this is incorrect`
TrackPlayer.getState -> also paused

SECOND TEST

 LOG  playbackState idle
 LOG  TrackPlayer.getState idle

Play song w/ code below:
await TrackPlayer.add(track);
await TrackPlayer.play();

 LOG  playbackState loading
 LOG  playbackState ready
 LOG  playbackState buffering
 LOG  playbackState playing -> `Song is still playing`
 LOG  TrackPlayer.getState playing

While song is playing, play a new song w/ code below:
await TrackPlayer.add(track);
await TrackPlayer.skip(trackId);
await TrackPlayer.play();

 LOG  playbackState paused
 LOG  playbackState loading
 LOG  playbackState playing
 LOG  playbackState paused
 LOG  playbackState playing
 LOG  playbackState paused ->`Song did end`
 LOG  TrackPlayer.getState paused

Play song w/ code below:
await TrackPlayer.add(track);
await TrackPlayer.play();

 LOG  playbackState playing
 LOG  playbackState paused
 LOG  playbackState loading
 LOG  playbackState paused -> `Song is actually still playing...`
 LOG  TrackPlayer.getState paused

While song is playing (even though it says paused), play a new song w/ code below:
Play song w/ code below:
await TrackPlayer.add(track);
await TrackPlayer.play();

 LOG  playbackState paused -> `New song doesn't play but previous song continues`
 LOG  TrackPlayer.getState paused
 LOG  playbackState playing

 LOG  playbackState paused
 LOG  playbackState loading
 LOG  playbackState paused
 LOG  playbackState playing -> `Song that I expected to play while the other song was playing now plays`
 LOG  playbackState paused -> Song did end`

what is the solution?

Any solution?


The fixed menu styles are broken

It is possible to manually decide which addons to use using the defaults prop. The separator should be conditionally rendered so that  unnecessary separators are not introduced. Here is a screenshot from neetoSite.
@gaagul _a this is a priority _t issue.
bug _t

@AbhayVAshokan This will be taken up in #854


TextInput not visible when it's height greater than some value

Description
Text input becomes invincible when it's height is a bigger value (mostly >= window.hiegth ). Tested in Android.  It's working normal for lower values.
Code to reproduce the issue
import { AppRegistry } from 'react-native';
import { name as appName } from './app.json';
import React from 'react';
import {
    SafeAreaView,
    ScrollView,
    Dimensions,
    TextInput,
} from 'react-native';

const App = () => (
    <SafeAreaView>
        <ScrollView contentInsetAdjustmentBehavior="automatic">
            <TextInput
                style={{
                    backgroundColor: "red",
                    width: Dimensions.get('window').width,
                    height: 2 * Dimensions.get('window').height,
                }}
            />
        </ScrollView>
    </SafeAreaView>
);

AppRegistry.registerComponent(appName, () => App);

React Native version:
react": "16.11.0",
react-native": "0.62.2"

Steps To Reproduce

react-native init
update above code and run

Expected Results
TextInput should not hide when it reaches to some limit

Let me just add that this is not reproducible on all devices / versions. I cannot reproduce it on my android 10 device but can reproduce on android 8 device and emulator.
Here's a snack where you can observe the problem: https://snack.expo.io/@vonovak/rn-28701

the issue is present with multiline textinput too, where this indeed is a problem
the trouble seems to be in wrapping the textinput in ScrollView. Without ScrollView the issue is not there, so perhaps this is a ScrollView problem, not a TextInput one.
I can also reproduce this in RN tester, I reproduced this on master with bf5f3c6a79fba839225576da889d3b7f835a19f6

Still facing the same issue

I have tested on Android 5-10, only happens on Android 8 and it is easy to reproduce.
Please re-open this issue.

Let me just add that this is not reproducible on all devices / versions. I cannot reproduce it on my android 10 device but can reproduce on android 8 device and emulator.
Here's a snack where you can observe the problem: https://snack.expo.io/@vonovak/rn-28701

note that the textinput is still mounted, you can interact with it, just not see it
the issue is present with multiline textinput too (if you fill it up with enough text), where this indeed is a problem
the trouble seems to be in wrapping the textinput in ScrollView. Without ScrollView the issue is not there, so perhaps this is a ScrollView problem, not a TextInput one. However, if I just change TextInput to View, it works; so some interplay between TextInput and ScrollView is causing this
I can also reproduce this on master, I tried it with RNTester on bf5f3c6

@vonovak Would you please re-open this issue?

Can verify this issue is present on RN 61.2 when running on an android 8 device.
Setting TextInput's maxHeight to screen height or less seems to "fix" it

I've created a $100 bounty for this issue. Please submit a PR if you could help.
https://issuehunt.io/r/tarouboy/react-native/issues/1


Add contributors

@all-contributors please add @bariscan-mobven for content, eventOrganizing, ideas, translation.


find_sparkle_appcast

find_sparkle_appcast doesn't always find the appcast
Reported in https://github.com/caskroom/homebrew-cask/pull/39859
I noticed this occur with another Cask but I don't remember which one, I'll find out and update this post.

find_sparkle_appcast finds the SUFeedURL key in the Info.plist and uses that. I don’t know of a different way to get it.
The mentioned app does not have that key, nor do I see another URL on the plist.

Using strings 'Rest Time.app/Contents/MacOS/Rest Time' | grep '.xml' I can indeed find the appcast URL reported in the other issue, but looking around it there is no other useful information to say for sure that is an appcast (i.e. not scriptable).

I can't find anything in the sparkle docs about an alternative implementation method.
However it works, it probably not the "correct" way to use sparkle and I'd imagine it would vary between developers.
Closing.


update packages

:white_check_mark: Build TheIdServer 7.4.7-fix-dependencies1-0205 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/e4bb35750d by @invalid-email-address)


improv(logger): streamline Logger types

Summary
Changes

Please provide a summary of what's being changed

I started working on this PR to improve the work merged in #2736 and provide a better DX for customers using the feature.
The main goal was to have TypeScript suggest the possible values for log keys as well as accept arbitrary strings.
While working on this improvement, which I achieved (see screenshot below),  I quickly realized that the types for the Logger had become convoluted.

Most of the types were not documented, and especially the types around log keys and contents of a log had become confused.
In this PR I have refactored, reorganized, and documented all the types for the Logger class - as well as some methods - to ensure that each one of them provides useful info for customers and maintainers alike.
The internal logic of the Logger remains unchanged, however due to having added many new comment blocks, the diff seems abnormally larger than it really is.

Please add the issue number below, if no issue is present the PR might get blocked and not be reviewed

Issue number: closes #3053

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

I'll start reviewing it tomorrow morning and this will take longer to complete.
If @am29d approves it before then, please go ahead and merge.

No rush - thank you!


added link to article and ugly color to indicate so

Coverage increased (+0.08%) to 26.316% when pulling 614644bc2f406c4adbf862802d980534107bf1ea on articleLink into 909d65fb2342873f0bbf1d390a7437344aa83c75 on master.


[BEAM-1723] deduplication of UnboundedSource in Flink runner

Changes Unknown when pulling cec71028ff63c7e1b1565c013ae0e378608cb5f9 on aljoscha:cherry-pick-1723 into ** on apache:release-2.0.0**.

run flink validatesrunner

This has been merged, but automatic closing was stymied by the merge-time rebase. Please close at your leisure.

Merged.


Adds FAQ

Conversational explination of UOR in plain language.

A page is ready for you to load this content at content/docs/faq/_index.adoc


Android - Notification - User receives a notification with wrong attention sound

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.82-1
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Navigate to a conversation that is not the one with the secondary user
Secondary device - Send a message to the user in the main testing device
Verify the user receives a notification with the attention sound (https://www.dropbox.com/scl/fo/t3j9sstsml5yofsrf26od/AIUtqgtjunDRLz8ebrbJGmc/Attention?e=1&preview=Expensify_SB_SecretHandshake_UI_Attention_Mono_240109.mp3&rlkey=dpvqt7h5xjzdsafj4ent564ne&subfolder_nav_tracking=1&dl=0
)

Expected Result:
The user should receive a notification with the attention sound when receiving a message with the app open.
Actual Result:
User receives a notification with wrong attention sound and sometimes no notification sound received
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/43996225/96903737-90d4-4857-bb1b-9d00ff66d5cd

View all open jobs on GitHub

Thanks for reporting! We've determined here this is likely not a priority at this time so closing this out.


Expand scope of Azure Pipelines build

As per earlier comment, got rid of -DDJV_BUILD_TINY=TRUE for the Azure Pipelines builds to expand the scope of what gets build. Had to add the following fixes to get this to build:

ASWF container needs the nasm assembler for ffmpeg
recent changes to VBO class in lib/djvAV/OpenGLMesh.h hadn't been propagated to OpenGL ES2 build case (in lib/djvDesktopApp/EventSystem.cpp)
OpenGL ES 2 doesn't have some symbols used in lib/djvViewApp/ColorPickerWidget.cpp (changed introduced in da356ebcfd25817926d0f776f39ea52a389d763d) so commented out a couple of calls for now, most likely not the correct solution
in gcov.sh script, exclude the CMake...gcno artifact files from CMake

Hi, sorry for the delay and thanks again for the changes! It's great to get the rest of the code building with the CI.
Disabling the OpenGL code for ES2 is fine, support for ES2 is still experimental and there will be a number of features that won't work with it.
Darby
PS. Sorry, I forgot to ask last time, would you like to be added to the credits page for contributing to the project?
https://darbyjohnston.github.io/DJV/credits.html

Sure, happy to have my name there, although for now my contributions are pretty minimal, hopefully I can contribute more over time.


Set default for "fs.gs.working.dir" to "/"

The code comment on line 149 was already saying that "/" was the default value, but it was actually not effectively set. This avoids having to manually set the property.

Working directory should be initialized here if property is not set:
https://github.com/GoogleCloudPlatform/bigdata-interop/blob/eb8e5a9f7783cad9ad3a7efb94aa95e7833ffe6d/gcs/src/main/java/com/google/cloud/hadoop/fs/gcs/GoogleHadoopFileSystemBase.java#L1691
Maybe we should fix comment instead? What use case are you trying to fix with setting it to /?

I started from a fresh custom Hadoop install (i.e. not using Dataproc), and got this warning:
$ hadoop fs -ls gs://MY_BUCKET
Dec 17, 2018 4:22:13 AM com.google.cloud.hadoop.fs.gcs.GoogleHadoopFileSystemBase configure
WARNING: No working directory configured, using default: 'gs://BUCKET/'

The warning only went away after explicitly setting the fs.gs.working.dir property to / in core-site.xml. So it didn't seem to be automatically set for me.


Question regarding compilation of DISCON (.so) files

Hi all,
I have compiled OpenFAST on a Linux system and am slightly confused about the variation of the use of the DISCON controller files compared to FAST v8. With FAST V8, these files were compiled as part of the compilation of FAST and easily available. With OpenFAST, I cannot seem to find the files. In particular, I need the HywindOC3 shared library. The tests cannot run without the files in place, as specified in the ServoDyn.dat file. Are there any instructions for the compilation of these shared libraries? Or are they even needed in OpenFAST? Thank you in advance for your help, and my apologies if I am missing something obvious.
Kind regards,
Luke Cunningham

Hi @lucunnin ,
I've posted this comment on issue #224 and added it here as well for future reference.
The ".dll" suffix is simply for compatibility with the regression test input files. On linux, that library is actually a shared object, but its renamed after it is compiled. You can see this step in the CMake config for the DISCON libraries. Without this, we would have to maintain a set of input files for each system.
The controllers are included as targets in the cmake configuration. You can verify this by running "make help" and looking for "DISCON_OC3Hywind". If those don't show up, verify that the BUILD_TESTING cmake flag is on and that the contents of "openfast/reg_tests/r-test" exist (in other words, that the r-test git submodule is initialized and updated). If that looks ok, you should compile and install these drivers with the "make install" command. See the OpenFAST regression test documentation for more details.


[:punct:] expanded incorrectly

[:punct:] is changed to [!"#$%&\'()\\*+,-./:;<=>?@[\\]^_`{|}~] here and because it starts with [! it's changed to the negated version right after. I suppose the most simple fix would be changing order of the characters in the regex class.

want to do a pr?

actually this issue would belong on expand-brackets

actually this issue would belong on expand-brackets

Yeah, if we want to just swap the order, then yes. But technically speaking, it's an issue with micromatch, the pattern generated by expand-brackets is correct.

the pattern generated by expand-brackets is correct

Actually, I take that back. This isn't the only problem with that pattern. Will submit a PR at https://github.com/jonschlinkert/expand-brackets


Is userdata function 'js_newcconstructor' missing a 'js_Finalize' callback?

Hi!
Thanks for your great project!
It is really awesome!
In the userdata example, there is this function call:
js_newcconstructor(J, new_File, new_File, "File", 1);
For example, let's say the 'new_File' constructor function allocates memory with the malloc function.
If in the .js file I call:
var test = new File('abc'); //Creates the object and allocates some memory with malloc
test = null; //Dereference the object and the garbage collector can claim the memory

When the garbage collector claims the memory, how can I tell him to free the previously allocated memory in the constructor function too?
Maybe the 'js_newcconstructor' signature could be changed like this:
void js_newcconstructor(js_State *J,
	js_CFunction fun, js_CFunction con, js_Finalize finalize
	const char *name, int length);

So the garbage collector could call the 'finalize' function and this callback function could free the memory allocated by the constructor function.
Thanks!

Hi!
I'm closing this issue because I will first try using a 'finalize' function in 'js_newuserdata(J, TAG, stdin)'.
Thanks!


Rest of bindable properties for AutoCompleteView

Rest of remaining properties in https://github.com/enisn/UraniumUI/pull/152


Allow cpubars to run on FreeBSD

FreeBSD can run cpubars natively, albeit using its linprocfs, with minor
changes. These same changes also allow FreeBSD to run cpubars
compiled for Linux. (I tested this since writing the commit message.)
Two changes were needed:

Provide for the location of /proc to change. On FreeBSD, a
Linux-compatible procfs is mounted at /compat/linux/proc by convention.
Use sysconf to get the number of CPUs. I think this may change the
behavior of cpubars in the event that CPUs are hotplugged, but it is
more portable.

I've finally pushed a revision. I think 9d6120a addresses both comments. Rather than switching based on OS, both /proc and /compat/linux/proc are checked for usable pseudofiles. The check for the maximum number of CPU has been preserved, although doing this requires OS-specific code.

Thanks!


Listing Management - Notify admins when a listing is deleted

When an app is deleted admins should be notified or the fact it was deleted should be recorded on the "Recent Activity" tab.  Currently there is no record that an app has been deleted or that a deleted app ever existed.

@clarkjefcoat This is a duplicate of #87

As noted above, this is a dup of #87


Release creation

This PR is related to Release 2.7.2 for Lightning Core

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.6 out of 7 committers have signed the CLA.:white_check_mark: frank-weindel:white_check_mark: erikhaandrikman:white_check_mark: g-zachar:white_check_mark: itsjamie:white_check_mark: sandeep-vedam:white_check_mark: michielvandergeest:x: Ian SmithIan Smith seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Updating project.json versions and renaming WilsonEventSource to IdentityModelEventSource

This pull request is now managed with CodeReviewHub. Refresh.

Hi @tushargupta51, I'm your friendly neighborhood Microsoft Open Technologies, Inc. Pull Request Bot (You can call me MSOTBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft Open Technologies, Inc. and real humans are currently evaluating your PR.
TTYL, MSOTBOT;


Short solution needed: "Convert Numpy array to string" (python-numpy)

Please help us write most modern and shortest code solution for this issue:
Convert Numpy array to string (technology: python-numpy)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

Convert Numpy array to string


How to get Jongos MongoCollection with JongoNative

Using 1.4-early-20160606-0118.
Perhaps I've misunderstood but to use the latest MongoDB java driver we are supposed to use the new JongoNative.
But, it only returns the MongoCollection from MongoDB (com.mongodb.client.MongoCollection) unlike before (org.jongo.MongoCollection).
How do we go about getting org.jongo.MongoCollection from JongoNative? Or, are we required to use the MongoDB BSON filters now?

This because org.jongo.MongoCollection can only be used with deprecated DB instance from mongo java driver. JongoNative returns com.mongodb.client.MongoCollection.
You can still use Jongo as follow :
collection.findOne(jongo.query("{name : #}", "Abby"));
collection.insertOne(pojo);
...

You can find more examples here : https://github.com/bguerout/jongo/tree/master/src/test/java/org/jongo/use_native
Let me know what do you think about this new API.

Ahh okay, I was basing it off the examples on the website. Oops, this works fine :)

OK i'm really interested in your feedback about how it is easy to use JongoNative with com.mongodb.client.MongoCollection (jongo.query....)

I'm about to have a go at putting it into a project, so i'll let you know how I go!

Looks like it is just "find" not "findOne"
Like:
User user = collection.find(query).first();

I simplified it a bit by creating a small helper method in my base class (everything here is very abstract haha) - makes it a bit tidier:
	public Optional<T> get(String id) {
		T result = collection.find(q("{id : #}", id)).first();
		if (result == null) return Optional.empty();
		return Optional.of(result);
	}

	public Bson q(String query, Object... parameters) {
		return this.jongo().query(query, parameters);
	}


PostgreSQL - Update replica server failover documentation

The read replica service endpoint is never pointed at the primary PostgreSQL server. Update the documentation to reflect this and inform that in the event of standby node (replica) failure the read replica endpoint will be unavailble on a Business plan until a new replica is available.
SUP-845

Thank you for adding this information :-) A few editorial comments

Thanks for the feedback, I've made these changes, I think they look better =)


An error occurred (DeleteConflict) when calling the DeleteRole  operation: Cannot delete entity, must delete policies first.

The kf roles need to have their attached policies removed first. Initially on destroy, we delete all policies, but we agreed that if we wouldnt want to remove eg an AWS managed policy.
The code was then changed to check if the policy starts with "kf-", and to remove those. But the current policies we attach have not had their names changed to reflect that check. So the error is produced because the kf- roles have policy names like iam_blahblah

believe this was fixed, @rb201 ?


Why did khipu choose scala programming language

Scala is a general purpose programming language, thus it can be used to write blockchain applications too.


Website: nextgen sitemap

Task
Create an IA for the nextgen website.
Preliminary proposal, to be tweaked and refined
https://www.figma.com/file/Nkn8gqjyKVEbIiWcBRUsMm/Carbon-for-Dotcom%3A-Sitemap?node-id=0%3A1

Comments and stuff to keep in mind

From Linda: Terminology should align. Be careful of the word "Pattern" and what it means for all the design systems within IBM
Personalization team would benefit from having a place to document how one content type can be expressed by multiple components
Template should be build out much more extensively. Example template documentation (Solution template) https://ibm.invisionapp.com/d/#/console/15370992/319811795/preview

Current secrete pages that might need a place in the left nav

Expressive sunset
Web components tutorials
Beacon

No changes needed.  Site map is up to date.


Make docker error 2

I am having issues with make docker. I need help asap. please assistnce. I have search online and done all i could but cant find solution. error paste below. I am a beginner in this field.
gotools.mk:58: recipe for target 'gotool.counterfeiter' failed
make[1]: Leaving directory '/opt/gopath/src/github.com/hyperledger/fabric'
make[1]: *** [gotool.counterfeiter] Error 2
make: *** [/opt/gotools/bin/counterfeiter] Error 2
gotools.mk:63: recipe for target '/opt/gotools/bin/counterfeiter' failed
Makefile:253: recipe for target '.build/docker/gotools' failed
make: *** [.build/docker/gotools] Error 2

Please reserve github issue list for code issues.
For getting help please use Discord chat, see https://hyperledger-fabric.readthedocs.io/en/latest/CONTRIBUTING.html#communications-and-getting-help


How does pagespeed decide what memcached server to ask?

Lets say I have a few memcached servers, by what logic does pagespeed ask them for informaiton?

PageSpeed selects a memcached server from the configured pool of memcached servers based on a hash of the key involved for the memcached operation it wants to perform.
The hash algorithm can be found over here:
https://github.com/pagespeed/mod_pagespeed/blob/master/third_party/aprutil/apr_memcache2.c#L634
For the specifics of using that hash to pick a memcached server for the operation about to be performed, see https://github.com/pagespeed/mod_pagespeed/blob/master/third_party/aprutil/apr_memcache2.c#L177
Does that clear it up?

YES thank you!!!! Very much!!!!!


Boss gague bar is biggerr than ususal

Hi foglio,
The boss gauge is bigger than usual there is this black border surrounding the boss gauge bar covering up lot of things.

enable transparency in settings and restart TCC


Add in the documentation a table for the relationship of shape/loc/scale and the pdf equation

Add a table to the documentation to make this relationship clear to everybody.
It should be placed like in the screenshot and look like in the screenshot.

Implemented at two occasions, 1:

2:


Helix configuration

How do I add rust_with_rstml to Helix?
Here's my current Helix configuration:
[[language]]
name = "rust"
scope = "source.rust"
injection-regex = "rust"
grammar = "rust_with_rstml"
file-types = ["rs"]
roots = ["Cargo.toml", "Cargo.lock", "tailwind.config.js"]
language-servers = ["rust-analyzer", "tailwindcss-language-server"]
formatter = { command = "dprint", args = ["fmt", "--stdin", "rs"] }
auto-format = true

[[grammar]]
name = "rust_with_rstml"
source = { git = "https://github.com/rayliwell/tree-sitter-rstml", rev = "c0a807f052c80575e2b5e2c0e01a5c65baaaa08c", subpath = "rust_with_rstml" }

The new grammar works in Helix with regular Rust code.
However, the html inside of the view! macro looks the same as before.

I have fetched and built the grammar using hx --grammar fetch and hx --grammar build respectively.
I have confirmed it is using the built grammar rust_with_rstml.so by deleting rust.so.
Using helix -vvv --log debug.log does not give any errors or insight.
I can also add support if it is necessary.

The way tree-sitter is implemented in Neovim, Helix, Zed... is that queries have to be written (like so) to match highlighting and indentation rules to the grammars output.
The main problem with these queries is that all these editors implement their rules differently (Helix highlighting rules) and they're pretty annoying to write.
I might take a look at Helix at some point, but adding support isn't trivial.

This editor is actually quite interesting, give me some time to play with it and I might add support later on.

Thanks for the info :) Let me know if you need any help or decide against adding support. I know I could just use neovim, but Helix is my first non-IDE editor, so I am kind of used to it.


Accept splitting at the tail of dataset in split_dataset

Fixes #5890

Thank you for the fix!
jenkins, test this please

Jenkins CI test (for commit 268cf8d3a754fab3a25fe897fcc60a4c39946b89, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

LGTM!


[#4558] Enhance missing exception handling

Enhance missing exception handling

Codecov Report

Merging #6214 into master will increase coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #6214      +/-   ##
==========================================
+ Coverage   39.64%   39.67%   +0.02%
==========================================
  Files        3033     3033
  Lines       91229    91229
  Branches    11862    11862
==========================================
+ Hits        36172    36191      +19
+ Misses      52054    52034      -20
- Partials     3003     3004       +1

Impacted Files
Coverage Δ

...npoint/profiler/sender/grpc/ReconnectExecutor.java
0% <0%> (ø)
:arrow_up:

...pc/server/LoggingServerMessageListenerFactory.java
90% <0%> (-10%)
:arrow_down:

...cluster/zookeeper/ZookeeperClusterDataManager.java
52.9% <0%> (-1.94%)
:arrow_down:

...pinpoint/profiler/monitor/DeadlockMonitorTask.java
18.6% <0%> (-1.17%)
:arrow_down:

...m/navercorp/pinpoint/profiler/AgentInfoSender.java
84.15% <0%> (-1%)
:arrow_down:

...point/rpc/client/DefaultPinpointClientHandler.java
73.05% <0%> (+0.97%)
:arrow_up:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
80.28% <0%> (+1.05%)
:arrow_up:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
85.36% <0%> (+2.43%)
:arrow_up:

...m/navercorp/pinpoint/rpc/client/ConnectFuture.java
90% <0%> (+5%)
:arrow_up:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
66.42% <0%> (+6.42%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


Add example apps

Add apps for CPU temperature (no sensor required) and sensor block
Closes #61

@balena-ci I self-certify!


Add authentication for system_restore_post

This patch fixes the API spec to add authentication to the POST /system/restore endpoint.
Fixes: https://github.com/Nitrokey/nethsm-sdk-rs/issues/15

@dvzrv Can you please test if this fixes your issue?

Shouldn't the OpenAPI spec be fixed in the original nethsm repo instead?

@jans23 Yes, but this PR allows us to test the changes before it is fixed upstream.

Without this change I get:
error: nethsm::Error - NetHSM API error: Restoring backup failed: Deseralization error (no message in body) (status code 401)

With this change I get:
error: nethsm::Error - NetHSM API error: Restoring backup failed: Body cannot be empty. (status code 400)

I am running restore against an "operational" container.
While the authentication error appears gone, I don't really know what to do about the 400, which is documented as

Bad request - restore did not apply.

I am also getting a 400 when trying to restore an "unprovisioned" container from backup.

Um, backup_file appears to be entirely unused in system_restore_post?!

It looks like we missed the code path for multi-part requests when replacing reqwest with ureq in https://github.com/Nitrokey/nethsm-sdk-rs/pull/1.  Let’s handle that in a different issue: https://github.com/Nitrokey/nethsm-sdk-rs/issues/20.

FWIW: we implemented multipart after we changed to ureq, but  because we only used the Rust SDK for the PKCS#11 module, which doesn't handle restore or updates, we didn't care. 😉


Cannot reproduce the results of SSD-300 on WIDER_FACE

Hi @sovrasov,
Recently, I'm reproducing the results of SSD-300 on WIDER_FACE.
I use the provided config based on mmdetection and get Recall 36.9, Precision 0.023 and AP 27.3.
The results are worse than Recall 43.4, Precision 0.029 and AP 34.7 (shown in https://github.com/open-mmlab/mmdetection/pull/765#issuecomment-502579220).
Generally, the models provided by mmdetection are trained on 8 gpus. So, is the SSD-300 on WIDER_FACE also trained on 8 gpus? If so, maybe I will change the learning rate because I trained the model on 4 gpus.
Moreover, any other advice?
Thanks a lot.

The problem should be in the learning rate value.
As far as I remember this config is for one or two GPUs, so you have to increase the LR in 2 or 4 times.
That's my fault, I didn't provide the correct LR for 8 GPUs.

Thanks a lot. I will try it.

@sovrasov Maybe you could open a PR to fix that.

@sovrasov @hellock
I have tried some training settings. In these experiments, I only changed the #GPU and LR. When adopting lr=0.0015 and training the SSD-300 on 1 GPU, I can get the comparable performance as @sovrasov.

Nice! According to these results most likely I had used 2 gpus with lr=1e-3.
I'll create a PR with lr changed to 8*1.5e-3 for 8 GPUs.


Add common errors

Can we add TSG for common errors like below to explain what the cause might be and how to mitigate it?
From Clipboard

resultCode
resultType
errorDetails

RollingUpgradeBatchStartTimeNotWithinScheduledWindows
2
Rolling Upgrade failed because the current time is no longer within any of the Automatic OS Upgrade scheduled windows.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 9d11782b-537a-cf3d-6b9c-f789fdd2f16a
Version Independent ID: e4d466be-afd7-5e07-58d9-7de51ae80fab
Content: Maintenance control for OS image upgrades on Azure Virtual Machine Scale Sets using PowerShell - Azure Virtual Machines
Content Source: articles/virtual-machines/virtual-machine-scale-sets-maintenance-control-powershell.md
Service: virtual-machine-scale-sets
GitHub Login: @ju-shim
Microsoft Alias: jushiman

@ju-shim
Could you please review add comments on this, update as appropriate.

@ju-shim
Thank you for your update.

@qqnarwhal
Our Product team working on this, once its done we will update you. Now we are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
#Please-close


Add tests for emptying list, then adding elements

Add tests cases to ensure the linked list's state is properly reset when popping/shifting its last element.

@jcshih -- thanks for catching this (especially since that was my example.js ;) !!)
I think it could be written so the logic is a little more clear.  In hindsight I think the ternaries were not the best choice. What do you think about this, for example:
  shift() {
    if (!this.tail) {
      return undefined;
    }
    const value = this.tail.value;
    if (this.tail.next){
      this.tail = this.tail.next;
      this.tail.next = null
    } else {
      this.tail = null;
      this.head = null;
    }
    return value;
  }

Also, we usually like to see one commit per PR if possible.  If you need help squashing the commits please let me know, I'd be happy to assist.

@matthewmorgan Thanks for the tips. I agree that conditionals are more readable than ternaries in this case.
I went ahead and refactored and squashed all the changes. I also switched next/prev because it made more sense to me.

Thanks @jcshih !


Extreme Frame Drops while observing specific blocks (1.19.2 - 1.2.2)

Bug Description
Extreme frame drops when looking at trees up close, seems to happen when looking at blocks with multiple branch connections. Frame differences are from 500FPS+ to 1FPS.
To Reproduce
Just looking at one of the aforementioned multi-branched log block on a Dynamic Tree.
Screenshots
Looking at a log with multiple connections, frame rate of 1FPS shown in top left,

Looking at base log block with no connections (log size makes no difference), with 1000+FPS,

(tested several hundred blocks in the sky w/ day and weather cycles disabled and mob spawns also disabled)
Rendering Glitches
Embeddium, Oculus and various entity rendering optimizer are part of the pack, but after extensive testing, involving disabling all but the first two, it seems like none of the mods that might affect improve the issue by being removed.
Mod List
This is in a modpack of my own making, I've done my best to optimize it and avoid conflicts, but still find immense frame drops at what feels like random when moving about. Seems the source of these drops are somehow the trees.
I'm honestly hoping that it's another mod in this list, as that's something that I can resolve once identified. But I have tested disabling or configuring a large number of mods troubleshooting the issue with no change between each test. The problem doesn't seem to be made any better or worse by having shaders on or off as well.
My best guess is maybe some strange highlight box issue? I've tried leaf culling mods, removing entities, turning off falling leaves, setting tree growth speed to 0, with no effect on the drops seen above, so whatever it is, it's really hard to pin down.

Issue resolved with further troubleshooting on my end. It was unrelated to Dynamic Trees and was in fact something to do with how a mod called Whispering Spirits interacts with more complex blocks. Caused issues with Sophisticated Backpacks too.
Not sure how these blocks differ in code, my guess is that static non-entity blocks with more complex models for some reason freak it out. Either way, not a Dynamic Trees issue!


Add a default limit to how many commits back it will process

When operating on very large git repos with tons of commits it takes a really long time to process all of the commits into Commit objects before it even gets to the good part where it processes the bodies to get the CHANGELOG content.
Something like maybe default to a max of 100 commits from HEAD.

You might be able to use ancestory references for this, https://git-scm.com/book/en/v2/Git-Tools-Revision-Selection.
Specifically the ~ version.

This has already been resolved.


Update Tasks “skin-scrub”

Automatically generated by Netlify CMS

👷 Deploy Preview for amazing-stroopwafel-c43386 processing.

Name
Link

🔨 Latest commit
d21e8c290ef3b31750613b0625967b6954c3192d

🔍 Latest deploy log
https://app.netlify.com/sites/amazing-stroopwafel-c43386/deploys/63ed46e97b37d00008681871


option in fix_client to send few warmup orders before the main loop

the idea is to reduce influence of initial few messages which are typically have lots of latency

@divaykin You need to update the option string passed to getopt() for this to work. Please also update the help text for usage to include the new option.

Thanks @penberg should be fixed now


InteractionRequiredAuthError: AADSTS65001: The user or administrator has not consented to use the application

What type of issue is this?
Question
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Target SharePoint environment
SharePoint Online
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[X] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version: 124.0.6367.208
SPFx version: 1.18.2
Node.js version: 18.17.1

Issue description
I'm trying to connect to MSGraph via my SPFx Web Part, but I don't really have a clue what I'm doing and now I've hit an error, as shown below:
InteractionRequiredAuthError: AADSTS65001: The user or administrator has not consented to use the application with ID 'xxxx' named 'SharePoint Online Client Extensibility Web Application Principal'. Send an interactive authorization request for this user and resource.
I've no idea how to fix this - anything I've found has sent me to Azure, but when it says 'Click on x option' the option is invariably not there!
In my Web Part, I have the following (I've just copied what I think is relevant; I can add more if needed):
import { MSGraphClientV3 } from "@microsoft/sp-http";

export default class My_WebPart extends BaseClientSideWebPart<My_Props> {
  public render(): void {
    this.context.msGraphClientFactory
      .getClient("3")
      .then((client: MSGraphClientV3): void => {
        client
          .api("/me")
          .get((error, response: AnalyserNode, rawResponse?: any) => {
            console.log(error);
            console.log(response);
            console.log(rawResponse);
          });
      });
    const element: React.ReactElement<My_Props> =
      React.createElement(My_Component, {});

    ReactDom.render(element, this.domElement);
  }

When I run this, I get the error above.  I haven't done anything in terms of giving permissions etc because I don't know what I need to do, and haven't been able to find anything in the docs.  Any help would be appreciated.

Hi @SharonGilmore,
I guess that you haven't completed the two actions mentioned in the tutorial:
in package-solution.json file of your SPFx solution ->
https://learn.microsoft.com/en-us/sharepoint/dev/spfx/use-aad-tutorial#configure-the-api-permissions-requests
and in the SharePoint Admin site (point 5 of the section) ->
https://learn.microsoft.com/en-us/sharepoint/dev/spfx/use-aad-tutorial#configure-the-api-permissions-requests
You can also double-check the entire provided page, but I hope that these two actions will be enough. :)

Ah, thanks - I hadn't spotted the package-solution changes!


JSONP multisource support

Modify Highchart and Highstock classes to support multiple JSONP sources within a single chart in order to display multiple graph lines loaded at runtime via ajax / jsonp calls.  The implementation is based on the original python-highcharts design only adding in code for looping over multiple jsonp sources instead of assuming only a single source could be used.

Done.  I clearly was not paying close attention to your standard style when I added that variable.


[Bug]: Authorized User Should Be An Intermediate Option

Contact Details
<EMAIL_ADDRESS>Section Number
53.2.2 Explicit Basic Option
What is wrong
The Explicit Basic Option states that the agent accessing the data is one parameter that may be used under this option. Many trust domains today do not have such a provision; they allow access by way of being a member of the trust domain.
For this reason, implementations will want to be able to support the other aspects of Explicit Basic without concern for who has access.
Describe the solution you'd like
Make this an Intermediate option.
Relevant log output
No response
Priority
{"Medium"=>"Significant issue or clarification. Requires discussion, but should not lead to long debate."}
Code of Conduct

[X] I agree to follow the IHE Code of Conduct

make this an open issue.

created open issue PCF_19 (issue #19)


UI - Update SI from Percentage to Dropdown

*Issue:*https://github.com/bcgov/entity/issues/20215
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

/gcbrun

Temporary Url for review: https://bcregistry-btr-dev--pr-140-69ylwd9i.web.app

Codecov Report
Attention: Patch coverage is 76.97595% with 67 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (main@6cd19c5). Click here to learn what that means.

Files
Patch %
Lines

...in-app/utils/btr-bods/si-to-btr-bods-converters.ts
52.00%
19 Missing and 5 partials :warning:

...r-main-app/components/individual-person/AddNew.vue
64.86%
11 Missing and 2 partials :warning:

btr-web/btr-main-app/utils/validation.ts
62.85%
13 Missing :warning:

...r-main-app/utils/btr-bods/bods-to-si-converters.ts
89.47%
3 Missing and 3 partials :warning:

...s/individual-person/control/PercentageDropdown.vue
91.07%
0 Missing and 5 partials :warning:

btr-web/btr-main-app/interfaces/form-input-i.ts
0.00%
2 Missing :warning:

...tr-main-app/interfaces/significant-individual-i.ts
0.00%
2 Missing :warning:

btr-web/btr-main-app/enums/percentage-range-e.ts
85.71%
0 Missing and 1 partial :warning:

...r-main-app/services/file-significant-individual.ts
87.50%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #140   +/-   ##
=======================================
  Coverage        ?   68.51%
=======================================
  Files           ?       50
  Lines           ?     3402
  Branches        ?      239
=======================================
  Hits            ?     2331
  Misses          ?      841
  Partials        ?      230

Flag
Coverage Δ

btr-ui
68.51% <76.97%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

/gcbrun

Temporary Url for review: https://bcregistry-btr-dev--pr-140-69ylwd9i.web.app

/gcbrun

Temporary Url for review: https://bcregistry-btr-dev--pr-140-69ylwd9i.web.app

/gcbrun

Temporary Url for review: https://bcregistry-btr-dev--pr-140-69ylwd9i.web.app


Request: Installing aarch64 compatible software

Hi
I have been looking in wiki & web about how to install linux arm apps on ARM handhelds. I heard ArkOS and Retro Arena support that, but do not know where to start.
Would you be so kind and point to some guide or post, if they exist or maybe add some instructions in wiki?
I was planning to use my Anbernic 353V for some video work on the go, because handhelds are pocketable and have many useful buttons for quick editing. this is the app I use on desktop [https://github.com/mifi/lossless-cut/releases]
If this is not possible/or to complicated, I'll just have to buy a x86 handheld.
Thank you!

Sure.  See here for more info on how to enable developer mode in ArkOS to allow building apps right on the device.  I don't know if that app will work on these devices but you can also try it and see what happens.  Good luck and have fun!


Linear Damping formula seems to be missing mass scaling.

I noticed that when using Box2D, linear damping felt off. I checked the code and noticed that the ODE being solved is dv/dt + c * v = 0. However, I believe this should be m * dv/dt + c * v = 0, otherwise changing the mass of an object has unexpected results. I expect that when I increase the mass of an object, it will speed up slower, and slow down slower. However, this isn't what happens, unless I divide the linear damping by mass, which then gives the result I expected.
Here is the relevant code starting on line 209 of b2Island.cpp:
// Apply damping.
// ODE: dv/dt + c * v = 0
// Solution: v(t) = v0 * exp(-c * t)
// Time step: v(t + dt) = v0 * exp(-c * (t + dt)) = v0 * exp(-c * t) * exp(-c * dt) = v * exp(-c * dt)
// v2 = exp(-c * dt) * v1
// Pade approximation:
// v2 = v1 * 1 / (1 + c * dt)
v *= 1.0f / (1.0f + h * b->m_linearDamping);
w *= 1.0f / (1.0f + h * b->m_angularDamping);

Think of b->m_linearDamping as c/m. This is an arbitrary choice with none better than the other except that the units for b->m_linearDamping are simpler. This makes it somewhat easier to tune.

I disagree with this, what if your mass is not constant, and is changed at runtime. Even in the case where the mass is constant, if I want to tweak my parameters, increasing mass should should make my object harder to stop. I'm sure most people believe that this variable is k in the equation F=-kv as well, I don't think people should have to look at the code to figure out that it's not.

Your case makes perfect sense and is mathematically valid. Different users of Box2D have different needs and different points of view. In some cases it is not possible to please everyone.
Imagine this user: "I tried setting the damping parameter on various bodies so they all slow down at the same rate. However, I don't have a good sense for the units of the damping parameter, they are in units of kg * m / s. Can Box2D provide a function to tune the damping parameters for a set of bodies so they all slow down at the same rate, regardless of their mass?"
Or this user: "I have a body with that changes mass often and it has some linear damping. I need the body to slow down at the same rate regardless of the mass. Can you provide an option to change the damping automatically so that the damping strength remains the same regardless of the mass?"
That said, I agree that the units should be documented and I will do so.

https://github.com/erincatto/Box2D/commit/a03a41934e87b5515dcefccde8a2b8ddd136b130


[Question] yaml equivalent of docs

Question
Having good docs and a web interface is nice, but I want to do as much as I can with yaml and config. I've not found a way to set the backup target in the docs, but I'd really like to.
Questions:

how to set backup target in yaml
how to select the secret to use in yaml

Thanks!

kubectl get settings.longhorn.io backup-target -o yaml -n longhorn-system
apiVersion: longhorn.io/v1beta1
kind: Setting
metadata:
  creationTimestamp: "2021-03-05T20:39:06Z"
  generation: 1
  managedFields:
  - apiVersion: longhorn.io/v1beta1
    fieldsType: FieldsV1
    fieldsV1:
      f:value: {}
    manager: longhorn-manager
    operation: Update
    time: "2021-03-05T20:39:06Z"
  name: backup-target
  namespace: longhorn-system
  resourceVersion: "311580"
  selfLink: /apis/longhorn.io/v1beta1/namespaces/longhorn-system/settings/backup-target
  uid: 182af078-6500-451d-bebd-b5d94727eafd
value: ""

kubectl get settings.longhorn.io backup-target-credential-secret -o yaml -n longhorn-system
apiVersion: longhorn.io/v1beta1
kind: Setting
metadata:
  creationTimestamp: "2021-03-05T20:39:06Z"
  generation: 1
  managedFields:
  - apiVersion: longhorn.io/v1beta1
    fieldsType: FieldsV1
    fieldsV1:
      f:value: {}
    manager: longhorn-manager
    operation: Update
    time: "2021-03-05T20:39:06Z"
  name: backup-target-credential-secret
  namespace: longhorn-system
  resourceVersion: "311582"
  selfLink: /apis/longhorn.io/v1beta1/namespaces/longhorn-system/settings/backup-target-credential-secret
  uid: c9c282ad-4f51-4972-81ef-f8874f3c0ceb
value: ""

I stripped them down to the bare minimum, that should work for kubectl apply -f
apiVersion: longhorn.io/v1beta1
kind: Setting
metadata:
  name: backup-target
  namespace: longhorn-system
value: "" # backup target here
----
apiVersion: longhorn.io/v1beta1
kind: Setting
metadata:
  name: backup-target-credential-secret
  namespace: longhorn-system
value: "" # backup secret name here

@ukd1 You can give this a try and report back, I didn't test this but should work :)

@joshimoo magic, thanks. Is this something that could be put in the docs?

Btw, if you want to build automation script to control Longhorn, we have some docs at https://longhorn.io/docs/1.1.0/references/longhorn-client-python/

@ukd1 documentation pull request are very much welcome https://github.com/longhorn/website#contributing-to-docs
The reason we don't publish things like this is that it bypasses the backends validation layer, since we currently don't run a kubernetes validation webhook for the crds and instead expect the settings to be set via the backend api.


Embedded Videos show up as a black rectangle with no controls in Waterfox/Firefox 36 and above.

I updated both Youtube center(to 2.1.7) and Waterfox to 36.0.1 and youtube embedded videos don't work. So I had to disable the youtube center add on and the embedded videos work fine now. Can you please fix this bug? Thank You

2.1.7 is known to have bugs of this nature. Use the Developer Version linked on the wiki until a new stable version is released.


results are empty ? what is wrong no error is coming ?

I have configured database with doctrine 2 and trying this tutorial: http://docs.doctrine-project.org/projects/doctrine-orm/en/latest/reference/native-sql.html
I can access the data by this way:
$repo = $em->getRepository('users');
$users = $repo->findAll();

I can not access the data by this way:
<?php
use Doctrine\ORM\Query\ResultSetMapping;

$rsm = new ResultSetMapping();
// build rsm here

$query = $entityManager->createNativeQuery('SELECT id, name, discr FROM users WHERE name = ?', $rsm);
$query->setParameter(1, 'romanb');

$users = $query->getResult();

what is wrong ?

You have an empty ResultSetMapping there.

What should be in ResultSetMapping  ? It is same as in tutorial. @Ocramius

See // build rsm here. You'd need to map the entitoes and entity fields there.
I strongly suggest avoiding this API if you didn't get to explore further yourself, as it only gets more and more complex, so you need to be able to walk without any aid before attempting to run.


Add Page component

I had a WIP Page component that Corey was implementing in a few apps (and I believe I was as well) but it never got backported.
Find it, clean it up, add to both UI v4 and v5.
Make sure it checks off all the a11y stuff we need, e.g.

title updates
live regions for content changes
main content skip link targets (see PI Portal a11y manual review issues 1, 36, and 41)

Added to 5.0.0-beta.1


Update and Deploy with SVN

Hi,
I use Rockeeter with SVN but I recognized, that it works not correctly. When I run "rockeeter update" or "rocketeer deploy" it seems to work correctly, but on the server remain the same SVN revision as before.
I found the problem, It's not related with the deploy strategy, but in the SVN class  Rocketeer\Scm\Svn is a little mistake.
public function reset()
{
		$command = sprintf('status -q | grep -v \'^[~XI ]\' | awk \'{print $2;}\' | xargs %s revert', $this->binary);

		return $this->getCommand($command);
}

"%s revert" this is the mistake, correct should be "%s revert .". SVN revert need path as argument
example from console in some SVN working copy
➜  trunk  svn revert
svn: E205001: Try 'svn help revert' for more information
svn: E205001: Not enough arguments provided
➜  trunk  svn revert .
➜  trunk
Could you fix it?
Thank you.

I think that's been fixed in 2.0.6 no? In 32fadfd43290f9d4253048c5f2f469cae1f50e90 ? Or would that not be what you meant? I don't use SVN personally, the SVN class has been contributed by users so I'm not sure what works or doesn't work to be honest, nor do I have an SVN repository under hand to test with :/

Yes, you are right it was fixed in 2.0.6. Sorry I had 2.0.5. It can be closed. Thank you


Fix typo in README

The README refers to yaml files in a couple of places but these are toml files

Thanks, @stevewillcock, for spotting and fixing this!

Thanks, @stevewillcock, for spotting and fixing this!

No problem. Thanks for putting this repo out there. I've been putting together a rough event sourcing framework in rust also inspired by Akka Persistence and it's great to read through this code and see how you approached it - some really nice ideas in here that I hadn't considered at all.
I've updated the PR as the CI failed on some (new?) clippy lints and one compilation issue in eventsourced-nats

Do you have a link to your ES framework?

Do you have a link to your ES framework?

Not yet, I still need to put this into a separate repo on GitHub and make it public. I was looking for suitable names to use when I came across your eventsourced project :) I'll let you know as soon as it is up on GitHub though.


pcsclite: 2.2.3 -> 2.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for pcsclite is: Middleware to access a smart card using SCard API (PC/SC)
meta.homepage for pcsclite is: https://pcsclite.apdu.fr/
meta.changelog for pcsclite is: https://salsa.debian.org/rousseau/PCSC/-/blob/2.3.0/ChangeLog
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.3.0 with grep in /nix/store/v38qacy9i1cifkz1mqcxykbyrb536br2-pcsclite-2.3.0
found 2.3.0 in filename of file in /nix/store/v38qacy9i1cifkz1mqcxykbyrb536br2-pcsclite-2.3.0

Rebuild report (if merged into master) (click to expand)

26878 total rebuild path(s)

26877 package rebuild(s)

First fifty rebuilds by attrpath

CuboCore.coreaction
CuboCore.corearchiver
CuboCore.corefm
CuboCore.coregarage
CuboCore.corehunt
CuboCore.coreimage
CuboCore.coreinfo
CuboCore.corekeyboard
CuboCore.corepad
CuboCore.corepaint
CuboCore.corepdf
CuboCore.corepins
CuboCore.corerenamer
CuboCore.coreshot
CuboCore.corestats
CuboCore.corestuff
CuboCore.coreterminal
CuboCore.coretime
CuboCore.coretoppings
CuboCore.coreuniverse
CuboCore.libcprime
CuboCore.libcsys
DisnixWebService
LPCNet
MMA
OSCAR
OVMF
OVMF-cloud-hypervisor
OVMFFull
R
SDL2
SDL2_Pango
SDL2_gfx
SDL2_image
SDL2_image_2_0
SDL2_image_2_6
SDL2_mixer
SDL2_mixer_2_0
SDL2_net
SDL2_sound
SDL2_ttf
SDL_audiolib
SDL_compat
SDL_gpu
SDL_mixer
_1oom
_1password-gui
_1password-gui-beta
_20kly

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v38qacy9i1cifkz1mqcxykbyrb536br2-pcsclite-2.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A pcsclite https://github.com/r-ryantm/nixpkgs/archive/ce320a2af3db742326ec21f64c7186d75b21e7d3.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ce320a2af3db742326ec21f64c7186d75b21e7d3#pcsclite

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v38qacy9i1cifkz1mqcxykbyrb536br2-pcsclite-2.3.0
ls -la /nix/store/v38qacy9i1cifkz1mqcxykbyrb536br2-pcsclite-2.3.0/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @anthonyroussel for testing.

@anthonyroussel what's the state of this PR? I build pcsclite from this commit and things seem to work (pcsc_scan correctly identifies SmartCards).

@ofborg eval

https://salsa.debian.org/rousseau/PCSC/-/compare/2.2.3...2.3.0

Rebased pcsclite against staging branch (CI checks were failing)
Build and test pcsclite OK (tested with branch rebased against master locally, because my computer is not powerful enough to rebuild 500+ packages) on x86_64-linux and aarch64-darwin


lit-html dependency is not declared.

At least DistanceGuide.ts imports from lit-html/directives/style-map but lit-html is not declared as a dependency. This breaks yarn plug-n-play at least, but is generally bad for consistent behavior. Perhaps you intended a peer dependency?

Fixed in<EMAIL_ADDRESS>


Error logging in export attachment function

As an analyst
I want to see any errors created in the Export Attachments feature, if they are to occur, in a text file
So that I can ask the devs to fix the problems
Acceptance Criteria:

[x] Scenario: Error message structure
Errors detected - please send this file to<EMAIL_ADDRESS>[Todays timestamp]

These files were not included in the downloaded zip file due to errors:
[Error number] - [filename on S3] - [file name in db] - [filetype] - [size on S3] - [size according to database] - [CCBC-ID] - [upload date and time]  - [Error timestamp]
Error Details:
[error messages for developers to decipher]
[Error number] - [filename on S3] - [file name in db] - [filetype] - [size on S3] - [size according to database] - [CCBC-ID] - [upload date and time]  - [Error timestamp]
Error Details:
[error messages for developers to decipher]
[Error number] - [filename on S3] - [file name in db] - [filetype] - [size on S3] - [size according to database] - [CCBC-ID] - [upload date and time]  - [Error timestamp]
Error Details:
[error messages for developers to decipher]

[x] if no errors found then text file says 'Download successfull'
.
.
.

[x] Error code definition
The error code maps to an enumerator in our control system.

[x] Scenario: Capture the errors in the Export Attachments feature
Given there are one or more errors in the Export Attachments feature
When I try to export
Then each error captured will be saved
And these errors will be saved in the errors.txt with the logs

[x] Scenario: Add the error log file to the ZIP file
Given I try to Export Attachments on the Dashboard
When an error occurs in one or more files
Then an errors.txt file that includes a list of filenames and their respective errors description is added to the ZIP file.

Not included:

Ultra refinement
DB:

no updates in the database.

Middleware:

[x]  Map the error code in the JSON

Frontend:

no updates in the front end.

Integration/API:

no integration.

Tests:

[x] yes, ...

Documentation:

[x] Create the Error code definition.

Wireframes

The errors.txt I think will be moreso for the devs, so that if one exists it can be given to the developers/us to look into.

@marcellmueller @AntBush @VB-at-Bis @pdoc1111 please add a list of the errors that we do know we may get to this ticket so that I can add the desired wording

Putting a blocked label on this as we can't realistically start on it until #1043 is completed which will involve a refactor and moving it to AWS Lambda.

The ticket should be tested in test or prod environment because dev env does not have applications with attachments and generated Intake_1 ..json is empty.
Steps to reproduce after deployment:

delete Intake_1 .. json and zip files from S3;
run https://test.connectingcommunitiesbc.ca/api/analyst/admin-archive/1 in Postman with header x-api-key set to S3 secret key;
locate file Intake_1 .. json in S3 bucket, download it locally and verify that list of attachments contain file types and sizes;
locate file Intake_1 .. zip in S3 bucket, download it locally and verify that error.txt has single line with 'Download successful' text;
choose any file from the list, save it as backup copy, note it's size and file type, then create new file with different size and upload it manually to S3 with the same key (overwrite real attachment);
repeat steps 1 and 2;
locate file Intake_1 .. zip in S3 bucket, download it locally and verify that error.txt contains expected error regarding size mismatch;
upload original file to S3 with same key (restore attachment)
choose any file from the list, save it as backup copy, note it's size and file type, then create new file with different file type and upload it manually to S3 with the same key (overwrite real attachment);
repeat steps 1 and 2;
locate file Intake_1 .. zip in S3 bucket, download it locally and verify that error.txt contains expected error regarding file type mismatch;
repeat step 8;
choose any file from the list, locate object in S3 bucket and change tag av_status to dirty;
repeat steps 1 and 2;
locate file Intake_1 .. zip in S3 bucket, download it locally and verify that error.txt contains expected error regarding infected file;
repeat steps 8, 1 and 2 to ensure valid intake archive to be stored in S3;

@MarsRomer @MarsRomer-zz, we can provide a demo if you wish. Otherwise, let's close this card.

https://app.zenhub.com/files/476885245/f32723c7-391a-40f5-ab69-caecdb63bba9/download
Attached error.txt file.


Revert "net: zoap_server: Unify Makefile and add prj file"

This reverts commit 350d5d4d60eaf3e96524f761e8e3d98c67a2a07d.
This change break with many boards and leaves prj.conf unused.

The commit is missing signed-off-by line.
Why wasn't this issue seen earlier during the original commit checks?

Why wasn't this issue seen earlier during the original commit checks?

The testcase.ini was crafted to make it pass the limited checks we run in CI


doc: override table data css to wrap long sentences

Override the readthedocs css theme for td and th, changing the white-space value from nowrap to normal. Long descriptions will be wrapped instead of forcing the user to horizontally scroll, improving the readability of the API Reference page.

Thanks Francesco!


[multistage] clean up AggregateOperator and add test coverage

see description, this one is pretty straightforward :)

Codecov Report

Merging #9767 (340a124) into master (e3f2835) will decrease coverage by 43.19%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master    #9767       +/-   ##
=============================================
- Coverage     68.58%   25.39%   -43.20%
+ Complexity     4976       44     -4932
=============================================
  Files          1955     1943       -12
  Lines        104851   104520      -331
  Branches      15874    15836       -38
=============================================
- Hits          71917    26539    -45378
- Misses        27869    75283    +47414
+ Partials       5065     2698     -2367

Flag
Coverage Δ

integration1
25.39% <0.00%> (?)

integration2
?

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...inot/query/runtime/operator/AggregateOperator.java
0.00% <0.00%> (-85.72%)
:arrow_down:

...in/java/org/apache/pinot/spi/utils/BytesUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/spi/trace/BaseRecording.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/spi/trace/NoOpRecording.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/config/table/FSTType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/config/user/RoleType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/data/MetricFieldSpec.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/stream/StreamMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/common/tier/TierFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/pinot/spi/config/table/TableType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 1468 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Great demonstration of how java testing should work! I learn a lot of Mockito stuff from this PR.


A nest in itemization become canceled.

Hello. Thanks for good tool.
I wrote following comments using itemization in header file.
- vector2 : time parameters of filter
    - sampling period [s]
    - time constant [s]

When I run doxide build, I got following result in converted markdown file.
- vector2 : time parameters of filter
- sampling period [s]  <-- nest is deleted!
- time constant [s]      <-- nest is deleted!

Did I write something wrong?
If you know a solution, please let me know.
Thanks in advance.

Thanks for the report @harumo11. I'm looking into this.

@harumo11 What sort of documentation comments are you using here? e.g. /// or /** ... */ style?

I'm doing some work on the release pipeline, but this should be available in a proper release within a few days. In the meantime it is available if installing from source.

@harumo11 What sort of documentation comments are you using here? e.g. /// or /** ... */ style?

I used /** ... */ style comment.

I'm doing some work on the release pipeline, but this should be available in a proper release within a few days. In the meantime it is available if installing from source.

Thank you for your super quick response!!
I'll download the source code and try it out.

Thanks for the update @harumo11 ! And for everyone else, the fix is included in the latest v0.4.6, download available from the site.


Use requests-ecp to handle login

See https://github.com/duncanmmacleod/requests-ecp/

We'll use weblogin instead.


'ssl' is a little archaic. I have badhttps.com, do you want it?

Hey Lucas,
Like most of the (non-crap) part of the industry, I've avoided the TLS/SSL discussion (which is mainly by pedantic nerds) entirely by talking about https. https://certsimple.com now uses 'https' almost everywhere, Google Security Blog and Let's Encrypt do too. I suspect https is also more easier to understand as that's what the browser shows.
With that in mind, do you want badhttps.com? I registered it before anyone else could, just let me know and I'll transfer it.
Mike / CertSimple

Thanks for holding on to it, but I actually chose badssl.com on purpose:

It's shorter and catchier (2 letters shorter, half the syllables).
It doesn't duplicate the scheme in the domain name. https://badhttps.com doesn't look too bad, but I think the duplication of https can be unnecessarily confusing. This site is meant for testing UI edge cases, and the domain may show up in all sorts of screenshots. We often write test instructions for people who may not know much about what we're doing; I can easily imagine someone misdescribing/misunderstanding an expected behaviour because the domain contains https.
 ;-)

I usually support  using the term HTTPS instead of TLS/SSL when talking about transport security, but I very consciously chose not to do so here. I hope that's alright.

Not to mention that getting the certificates reissued would be both
expensive and time-consuming.  :)
On Jun 29, 2016, at 3:02 PM, Lucas Garron<EMAIL_ADDRESS>wrote:
Thanks for holding on to it, but I actually chose badssl.com on purpose:

It's shorter and catchier (2 letters shorter, half the syllables).
It doesn't duplicate the scheme in the domain name.
https://badhttps.com doesn't look too bad, but I think the duplication
of https can be unnecessarily confusing. This site is meant for testing
UI edge cases, and the domain may show up in all sorts of screenshots. We
often write test instructions for people who may not know much about what
we're doing; I can easily imagine someone misdescribing/misunderstanding an
expected behaviour because the domain contains https.

;-)

I usually support using the term HTTPS instead of TLS/SSL when talking
about transport security, but I very consciously chose not to do so here. I
hope that's alright.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/badssl.com/issues/172#issuecomment-229471296,
or mute the thread
https://github.com/notifications/unsubscribe/AAOEgF5Frsel2usiVSlih2CY5bLy1D69ks5qQs9OgaJpZM4JBPtn
.

would be both
expensive and time-consuming.  :)

Ah, yes. SHA-1 would also be against the Baseline Requirements. ;-)

lol excellent point @marumari I remember what it was like thr first time round :^)

Fair enough Lucas. Give me a shout when you're in London next and we'll get beer! 😀

Give me a shout when you're in London next and we'll get beer! 😀

Woah, you're in London?
I'll be there July 18-July 22, how's Wed 20 or Thu 21 for you?

Thu 21st it is! Do you want me to get an Infosec  beer posse together?
Get Outlook for iOS
On Thu, Jun 30, 2016 at 9:25 PM +0100, "Lucas Garron"<EMAIL_ADDRESS>wrote:
Give me a shout when you're in London next and we'll get beer! 😀
Woah, you're in London?
I'll be there July 18-July 22, how's Wed 20 or Thu 21 for you?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thu 21st it is! Do you want me to get an Infosec  beer posse together?

I don't drink alcohol, but I'm happy to join with a large glass of whatever refreshing tea-based beverage is in season. :-)
I'll be close to Victoria Station, but I haven't been to London this millennium yet, so I don't know my way around. Any suggestions for a time and place?


Make it clear in docs that we parse LLVM triples

Fixes https://github.com/bytecodealliance/target-lexicon/issues/111.

Thanks!

This is released in target-lexicon 0.13.0.


update dependancy to lodash 4

But it also dropped IE 6-8 support.

I would be happy to do this but I'm not sure if others rely on that browser
support. Maybe we could just update all lodash dependencies to the latest
3.x.x, but 4 may be a bit of a risk.
On 18 Feb 2016 07:49, "Chen-Heng Chang"<EMAIL_ADDRESS>wrote:

But it also dropped IE 6-8 support.
—
Reply to this email directly or view it on GitHub
https://github.com/Olical/react-faux-dom/issues/36.

I see three options here.

Upgrade lodash to v4, make the library only work in modern browsers (maybe bad)
Stick with v3 but update all to latest minor / patch
Replace everything with my own code that relies on ES5 features

I'm thinking option two or three, but using built ins probably has the same browser compatibility issues. Maybe people can just stick with the older version in that case.
react-faux-dom@3 could just be for modern browsers. If I'm going to break compatibility, I'd like to remove dependencies at the same time. The fewer the better!
I'm thinking option three with a major version bump would be best.

Yeah, the fewer the better!

I'll be releasing this in v2.4.0. No more lodash dependencies :)


[webapi] Rollback WebGL subcase changes

Impacted tests(approved): new 2, update 6, delete 0
Unit test platform: Crosswalk Project for Android 15.44.394.0
Unit test result summary: pass 1583, fail 12, block 2
https://crosswalk-project.org/jira/browse/XWALK-4888

Hi @jennycao as talked offline, I would like to accept this and #2752, because the sub-case enabling has huge impact on the current being testing cycles, though these changes may cause test results difference between weekly testing and web testing service.
CC @yugang @tinazhao

Yes, Thanks for the update:)


test: ensure the running container is stopped even when a test breaks

Use exit tap to ensure the running container sn-table-playwright is stopped even when a test breaks.

Good idea but this means even more duplicate code. The only thing that differs is the update flag. it should be passed as a flag/argument instead.

Good idea but this means even more duplicate code. The only thing that differs is the update flag. it should be passed as a flag/argument instead.

Right, sure, I will do the refactor in the next PR.

In that case, there will be only one bash script. What file name do you suggest?

In that case, there will be only one bash script. What file name do you suggest?

run-rendering-tests works, you are still running them when you update the snapshots

Right, sure, I will do the refactor in the next PR.

I think you should do it the other way around though. Make it into one file before adding code to both files


Unreleased version of EKS AMI for 1.20 in eu-central-1

In eu-central-1, there is an unpublished, undocumented version of the EKS 1.20 AMI that is being picked up by Managed Node groups and is present in when running
aws ssm get-parameter --name /aws/service/eks/optimized-ami/1.20/amazon-linux-2/recommended/image_id --region eu-central-1 --query "Parameter.Value"
"ami-097c83d9974db16f4"

The AMI is named  amazon-eks-node-1.20-v20220226
Can someone confirm if this is intentional or not?

Closing, as I seem to have stumbled across https://github.com/awslabs/amazon-eks-ami/releases/tag/v20220226 before the release was created on Github.


Debug control to draw on the screen

Playing around with CodeWorld debug controls, I'd kind of like to have a control to mark a point or line on the screen, which would stay in place as I zoom, pan, or press play.  For instance, I could mark the outline of the screen before zooming out.  Or I could mark the point at which something happens, and then play the animation or simulation further.

Another request here would be to see the x and y coordinates of points.  Not sure if that's the same feature, or something else.  (For example, the coords feature could just be a text label that shows the coordinates of the mouse pointer.)


Tmuxinator plugin improvements

Plugin find works with symlinked configuration folders (eg when using some kind of dotfiles management)
@phstc is it possible to check that it works well on OSX too?
Also completion now supports the shorthand syntax mux project_name

See my comment in #3540

As far as the zsh builtins go, never occured to me, but yes, this should be more portable, will change accordingly.

Concerning the $_configs in the cmds part, mux supports a shorthand syntax for start, where instead of mux start name you can just type mux name

Perfect. Okay then, you have my 👍
—
Marc

#2812 also tries to achieve this but still using the find command. Add Fix #2812 in the description to auto-close that one.

All set

Perfect, this will probably be merged later on this week. Thanks!

Thanks @agios !


Update driver.compose.json

Add "TUYATEC-v3uxbuxy" to temphumidsensor.
According to #122 and forum add to switch_2_gang "TS0002":

"_TZ3000_fisb3ajo"
"_TZ3000_bvrlqyj7"

According to forum add "_TZ3000_ke7pzj5d" to curtain_module.

Hi again @danielgajdos, not sure what is what in this PR.. you have 4 commits with a few changes. I'll put this one on hold for now. The other request has been added and I have quite some more devices added already so lets see if this is still needed after the upcoming update.

All devices requested in this PR are added to the latest release. Closing PR,


fix(android): shutdown leak

When the Player::dispose function is called, the mpv instance shutdown is induced via mpv_command_string with the command quit 0.
On android, quit 0 does not actually clean some resources (a thread is left active, consuming high CPU). mpv_destroy must be called for actual cleaning to happen.
When the player is shut down, a MPV_EVENT_SHUTDOWN event is emitted to the event loop, causing it to quit.
In this PR, we simply call mpv_destroy on Android devices.
We also wait for MPV_EVENT_SHUTDOWN to be emitted before finishing disposing.
I reproduced and tested it out by having an android app display a video and then unload it, over and over again.
It must be noted that after 100+ times, cpu usage becomes high again, and new videos stop loading in time. Maybe somewhere else is leaking? Perhaps a problem in mpv itself? It should be investigated.

Dealt as part of #189. A more reliable & safer approach; that for all platforms.


Just some Boto guidance

Hi,
Poking around in your code looking for useful Boto examples I noted you are explicitly deleting s3 buckets provisioned by a CloudFormation stack.
https://github.com/awslabs/data-pipeline-samples/blob/master/setup/stacker.py#L79
if r.resource_type == "AWS::S3::Bucket":
       if not s3:
           s3 = boto3.resource("s3")

           bucket = s3.Bucket(r.physical_resource_id)
              for key in bucket.objects.all():
                 key.delete()

Was wondering why you felt the need to explicitly delete s3 buckets that have been provisioned by CloudFormation. Are they not being handled by Stack.Delete()
Thanks
Terry

Got it figured. Did not realise delete would fail if S3 bucket had objects.


Android (12) not showing notification or going to call screen for incoming call

Issue Summary
Upon receiving an incoming call, no call notification is shown nor does the app show the incoming call screen UI to answer/reject. The phone does ring.
Steps to Reproduce

Ensure app is in background
Call the device
Observe no notification/incoming call UI (but incoming call ringing audio plays)

Any other relevant information. For example, why do you consider this a bug and what did you expect to happen instead?
Show incoming call notification and/or accept call UI
Android 12, Galaxy Note 10.

Can you Please push the new changes..?

Can you Please push the new changes..?

Hi @Ahmadgnoor - this is still in-development, specifically I'm testing new ConnectionService implementation. Expect new changes this week.
This addressed issues listed above.

#151 merged

See release 0.1.0

See release 0.1.0