Add namespace detection to aio make_logger

This will bring AB aio logging namespace in line with the existing code on the tx side https://github.com/crossbario/txaio/blob/master/txaio/tx.py#L242 so all loggers returned from make_logger() calls will be in the 'autobahn.*' hierarchy.
A user's app level code can then configure log events from AB like:
ab_logger = logging.getLogger('autobahn')
ab_logger.setLevel(logging.INFO)

independently of the logging namespace used by the app:
myapp_logger = logging.getLogger(__name__)
myapp_logger.setLevel(logging.DEBUG)

Currently make_logger() calls always return the root logger.

This looks plausible -- but obviously some impacts on the unit-tests (at least); is this because those are only monitoring the root logger or something?

no response from OP

Looks like there have been some commits changing how testing works since I submitted this PR - rebased on top of master and tests are running ok now.
This PR should fix this https://github.com/crossbario/autobahn-python/issues/952.

@oberstet can you reopen this please?  Not sure if you were confused about the timeline on this PR, but it didn't receive any attention for 3 months, then closed as I didn't reply to a question within 1 week - that's a little draconian!

@spr0cketeer uups, sorry! I was indeed confused;) now, my practical problem is: the "Reopen" button is deactive . mmh. let me see what happens when I first comment ..

so not sure: the button just is not there, means, I can carry over the changes manually, but that looses authorship - or @spr0cketeer you reopen from your side. sorry! let me know ..

@oberstet I rebased and force pushed so maybe that's why the Reopen button is disabled?
Following the example in this: https://gist.github.com/robertpainsi/2c42c15f1ce6dab03a0675348edd4e2c
I've now done step 3 and force pushed back to the commit when this PR was closed - is the Reopen button enabled now?  If not I'll just resubmit a new PR.

@spr0cketeer cool, whatever you did worked (I could reopen the PR;) thanks

Codecov Report

Merging #121 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #121      +/-   ##
==========================================
+ Coverage   95.76%   95.81%   +0.04%
==========================================
  Files           9        9
  Lines         708      716       +8
==========================================
+ Hits          678      686       +8
  Misses         30       30

Impacted Files
Coverage Δ

txaio/aio.py
99.2% <100%> (+0.02%)
:arrow_up:

Continue to review full report at Codecov.

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

@meejah could you have a look into this? I really can't say much .. quite cryptic;)

The code looks good to me. I haven't tested it yet and would like to at least know what effect this has on Autobahn logging messages -- I'm pretty sure the effect is good: instead of all logs going to the root logger they'll have a more-proper namespace.
Ideally we'd have an explicit test for the namespace, too. I wouldn't hold up merging for this though.
But, we should probably put that in the next Autobahn release-notes?

This change will allow component.start() users to control the output location and format of AB's logs as all AB loggers will be under the autobahn.* namespace.  AB does not call txaio's start_logging() using this method, so no output handlers or log format is set.
run([components]) and ApplicationRunner.run(ApplicationSession) both call start_logging(). This attaches a handler to the root logger with a log format string which doesn't include the (now namespaced) logger name, so they will see no difference to the output format of AB's log messages.
However it will not fix the problems inherent to start_logging()'s use of the root logger e.g https://github.com/crossbario/autobahn-python/issues/952 - this would require some changes to the way start_logging() works, with any changes requiring this PR be in place first.


viceroy: 0.10.2 -> 0.11.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/fastly/viceroy/releases.
meta.description for viceroy is: Viceroy provides local testing for developers working with Compute@Edge
meta.homepage for viceroy is: https://github.com/fastly/Viceroy
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.11.0 with grep in /nix/store/pnqkjwckkknxin59ms9hipq3lwhmx34g-viceroy-0.11.0
found 0.11.0 in filename of file in /nix/store/pnqkjwckkknxin59ms9hipq3lwhmx34g-viceroy-0.11.0

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

fastly
viceroy

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pnqkjwckkknxin59ms9hipq3lwhmx34g-viceroy-0.11.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 viceroy https://github.com/r-ryantm/nixpkgs/archive/d25d48272393c641cdd449a69bfa539c496e5c00.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d25d48272393c641cdd449a69bfa539c496e5c00#viceroy

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

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

fastly
viceroy

Maintainer pings
cc @ereslibre @shyim for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

Result of nixpkgs-review pr 336244 run on x86_64-linux 1

2 packages built:

fastly
viceroy

Result of nixpkgs-review pr 336244 run on aarch64-darwin 1

2 packages built:

fastly
viceroy


libmysofa: init at 0.6 and enable in ffmpeg-full

Motivation for this change
Using HRTFs with mpv.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build mpv

Success on x86_64-linux (full log)
Attempted: mpv
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/q5pcpkvkzl3p6fnvrz4r53jdsshmrnwi-mpv-0.29.0
shrinking /nix/store/q5pcpkvkzl3p6fnvrz4r53jdsshmrnwi-mpv-0.29.0/lib/libmpv.so.1.101.0
shrinking /nix/store/q5pcpkvkzl3p6fnvrz4r53jdsshmrnwi-mpv-0.29.0/bin/.mpv-wrapped
gzipping man pages under /nix/store/q5pcpkvkzl3p6fnvrz4r53jdsshmrnwi-mpv-0.29.0/share/man/
strip is /nix/store/a3nk8z2i7m7wa3jdckgv710n7j3yx4b5-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/q5pcpkvkzl3p6fnvrz4r53jdsshmrnwi-mpv-0.29.0/lib  /nix/store/q5pcpkvkzl3p6fnvrz4r53jdsshmrnwi-mpv-0.29.0/bin
patching script interpreter paths in /nix/store/q5pcpkvkzl3p6fnvrz4r53jdsshmrnwi-mpv-0.29.0
checking for references to /build in /nix/store/q5pcpkvkzl3p6fnvrz4r53jdsshmrnwi-mpv-0.29.0...
/nix/store/q5pcpkvkzl3p6fnvrz4r53jdsshmrnwi-mpv-0.29.0

Success on aarch64-linux (full log)
Attempted: mpv
Partial log (click to expand)
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/2yyw98rk6zplkgafq1dqsmhihzzyx0jw-mpv-0.29.0
shrinking /nix/store/2yyw98rk6zplkgafq1dqsmhihzzyx0jw-mpv-0.29.0/lib/libmpv.so.1.101.0
shrinking /nix/store/2yyw98rk6zplkgafq1dqsmhihzzyx0jw-mpv-0.29.0/bin/.mpv-wrapped
gzipping man pages under /nix/store/2yyw98rk6zplkgafq1dqsmhihzzyx0jw-mpv-0.29.0/share/man/
strip is /nix/store/a245zacjzf3qw0davhvlfarihcy2yyrc-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/2yyw98rk6zplkgafq1dqsmhihzzyx0jw-mpv-0.29.0/lib  /nix/store/2yyw98rk6zplkgafq1dqsmhihzzyx0jw-mpv-0.29.0/bin
patching script interpreter paths in /nix/store/2yyw98rk6zplkgafq1dqsmhihzzyx0jw-mpv-0.29.0
checking for references to /build in /nix/store/2yyw98rk6zplkgafq1dqsmhihzzyx0jw-mpv-0.29.0...
/nix/store/2yyw98rk6zplkgafq1dqsmhihzzyx0jw-mpv-0.29.0


operator: add part-of and managed-by labels

Type of chang

Enhancement / new feature

Description
Adding a labelSelector to the KubernetesDependent annotation means that when setting up the watches via fabric8 it can pass down the label selector. This means the API server won't send events to the operator for all Deployments/Services/Secrets, only those managed by the operator.
Part-of is just one of the recommended labels, it will allow users to use label selectors to locate the resources associated with a particular proxy CR
https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/
Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Write tests
[ ] Update documentation
[ ] Make sure all unit/integration tests pass
[ ] Make sure all Sonarcloud warnings are addressed or are justifiably ignored.
[ ] If applicable to the change, trigger the system test suite.  Make sure tests pass.
[ ] If applicable to the change, trigger the performance test suite. Ensure that any degradations to performance numbers are understood and justified.
[ ] Ensure the PR references relevant issue(s) so they are closed on merging.
[ ] For user facing changes, update CHANGELOG.md (remember to include changes affecting the API of the test artefacts too).

NOTE:  You must be a member of @kroxylicious/developers to trigger the system test and performance test suites.  If you are not part of this group, comment on the PR requesting a trigger, tagging @kroxylicious/developers.

I've switched things around because as you mentioned, by the docs kafka looks like a more appropriate part-of label. I think the KafkaProxy name is a better fit for the instance label.
So for a KafkaProxy named my-proxy-cluster you get:
    app.kubernetes.io/managed-by: "kroxylicious-operator"
    app.kubernetes.io/name: "kroxylicious-proxy"
    app.kubernetes.io/part-of: "kafka"
    app.kubernetes.io/instance: "my-proxy-cluster"
    app.kubernetes.io/component: "proxy"


The error about prepare_code_trace.py

Hi, @peikexin9. I just ran the code  'python micro_trace/prepare_code_trace.py' ,but I met this error "micro_trace/prepare_code_trace.py", line 67, in hex2str assert len(num) <= 8 AssertionError. First I ran the code 'python command/pretrain/prepare_json.py' to generate the data 'data-raw/funcbytes/', the ran the code 'python micro_trace/prepare_code_trace.py' to generate the data 'data-raw/functraces', but the error occured. Could you please help me. Thank you very much.

Sorry @RobinHan24, there might be some incompatibility issues between data preparation scripts as I am still working on this. Feel free to edit the script when you see bugs.


Mrtk v next input action rollback2

Rolling back the interface implementation to fix issues with profiles.

What's the benefit to removing the interface? Or is this a temporary rollback?

It broke the profiles by making them an interface. This fixes that issue.

It was a nice idea @keveleigh to possibly allow users to create their own custom actions, sadly it was a solution in need of a problem, hence the rollback.
We may reevaluate in the future but for the foreseeable future, there is no demand.

@keveleigh any update on this?

Thanks for the explanation @SimonDarksideJ!


[bug] 'planetoids' Attempting to enter the asteroid belt causes mapgen to crash

Repeatable.  Just position your player at approx 5950 and fly/move upwards.  The server will crash when trying to run mapgen for Y >= 6000.  It should be noted that the moon mapgen works fine.
2022-01-15 05:28:01: ERROR[Main]: ServerError: AsyncErr: Lua: finishGenRuntime error from mod 'planetoids' in callback environment_OnGenerated(): .../bin/../worlds/world/worldmods/
planetoids/mapgen.lua:41: attempt to call field 'is_occupied' (a nil value)
2022-01-15 05:28:01: ERROR[Main]: stack traceback:
2022-01-15 05:28:01: ERROR[Main]:       .../bin/../worlds/world/worldmods/planetoids/mapgen.lua:41: in function '?'
2022-01-15 05:28:01: ERROR[Main]:       /home/container/bin/../builtin/game/register.lua:449: in function </home/container/bin/../builtin/game/register.lua:433>

Also crashes when descending from Mars towards top surface of asteroids at Y=10,000.  Part of an asteroid did mapgen (I saw ore and ice), but it then quickly crashed.

So far, EdenLost is no longer crashing with the bugfix provided by @BuckarooBanzay


Subset of Freebase

Hi,
In the paper, you mentioned you extracted a subset of the Freebase. Is the subset attached in the google driver? Thank you.

yes it is. Please see the data folder, it is under fbwq_full and fbwq_half


Summarize agent_scratchpad when it exceeds n tokens

Feature request
Similarly to memory=ConversationSummaryBufferMemory(llm=llm, max_token_limit=n) passed in initialize_agent, there should be a possibility to pass ConversationSummaryBufferMemory like-object which would summarize the intermediate_steps in the agent if the agent_scratchpad created from the intermediate_steps  exceeds n tokens
Motivation
Agents can run out of the context window when solving a complex problem with tools.
Your contribution
I can't commit to anything for now.

Did you solve this? I am looking to solve the same issue when the agent loops and continues to pull extra documents from the vector store. I think what needs to happen is the scratchpad almost needs to be run through a document compressor? Thoughts?

Hi! I have not developed a solution. For now, I just moved on to other things.
I saw that there is a intermediate_steps variable in /langchain/agents/agent.py:1029 which is than changed to the text agent_scratchpad. IMHO the solution is either to compress intermediate_steps or the resulting agent_scratchpad. TBH I hoped for any input from dosu bot.
I'd suggest that anyone who is interested in this feature should "up vote" the issue - maybe we will be able to gain some momentum. :)

I had the same thought, I will try and work on compressing the agent_scratchpad inside the loop and let you know if I have any success.

@dosu-bot thank you for answer. However, my understanding of the problem is a little bit different. I would like to summarize agent_scratchpad created from intermediate_steps within one prompt from the user, so that complex problem can be solved.
Your solution adds content of the agent_scratchpad to the memory of the entire conversation, this not a desired behavior. Could you adjust your reponse?

I have to be the one to ask dosu bot :)
@dosu-bot thank you for answer. However, my understanding of the problem is a little bit different. I would like to summarize agent_scratchpad created from intermediate_steps within one prompt from the user, so that complex problem can be solved.
Your solution adds content of the agent_scratchpad to the memory of the entire conversation, this not a desired behavior. Could you adjust your reponse?

At least we know where to start looking now, I wouldn't be using 'sumy' but an extra call to the existing given LLM. I haven't had time to progress this personally.

Did someone achieve this? I am trying to retreive mails but as soon as I request more than two the agent stops because the token length is exceeded. If I could summarize the scratchpad this wouldn't happen. I tried to adapt the toolkit I used and add an additional chain for summarization after the retreive step of on tool but fail to add the chain as an intermediate step so summarizing the scratchpad would work as well I think.

This should definitely not be closed. There is still no way to easily achieve this in the libaray and is a major blocker

Agreed. Major blocker.

@mdziezyc  did you find a proper solution to this issue?


Feature Request: add possibility to style td and th

Would it be possible to give td and th elements a part="metatable-td" or part="metatable-th" attribute so we can have control over their styling via ::part? I wanted to change some things at the level of td and th, like for example setting overflow: hidden for long URLs (to avoid the scrollbar) and also centering the values (they seem a bit skewed towards the top, not centered, but this may be caused by something else).
This could also apply to other intermediate elements, but I didn't have a use case for it. Is there a technical constraint here somewhere? I tried doing this in developer tools and it worked
Awesome plugin btw, it removes the need for some dataview weird syntax, and makes obsidian closer to some notion functionalities I was used to :)

Hi, sounds like a fairly easy thing to bring in for both keys and values. As long as you intend to affect only th and td you'll have control over their presentation.
The technical limitations (by design) of ::part is that you can only affect the element with the part, so you can't influence its descendants. In the case of say having a  you wouldn't be able to redefine properties of e.g. the child .  Well, more generally you can't use composed CSS rules.
The top skew is also bothering me, but I haven't had time to debug why it's happening. Any ideas? 😄

The technical limitations (by design) of ::part is that you can only affect the element with the part, so you can't influence its descendants. In the case of say having a  you wouldn't be able to redefine properties of e.g. the child . Well, more generally you can't use composed CSS rules.

I removed the underline from the rest of my regular links, but it doesn't seem like too big of an incovenience.

The top skew is also bothering me, but I haven't had time to debug why it's happening. Any ideas?

Playing around in Developer Tools I found that using display: flex and align-items: flex-end on td and th actually centered the content. I hope this helps (I didn't do much more research on it)

Version 0.10.2 adds both ::part(key) and ::part(value)

Ah, I think you are onto something, thanks :)

Version 0.10.2 adds both ::part(key) and ::part(value)
Awesome! Thanks :)

Ah, I think you are onto something, thanks :)
No problem :)

I've been playing around some more in Developer Tools and the flex-end workaround only changes some of the content,  not every row, which is weird.

I found a small problem with the part attribute in some cases. I have an entry in some of my notes' frontmatter that looks like this:
Days:
  Friday: "9:45 → 11:15"
  Tuesday: "9:45 → 11:15"

The plugin creates an embedded table. In this case, the inner keys and values get the part attribute, i.e., Friday, "9:45 → 11:15", Tuesday, and "9:45 → 11:15" get the attribute, but the th Days and the td that contains the embedded table do not get it.

Mmm, what seems to work well is using display: grid and changing the fold icon positioning to use grid columns. I'll release this one and see whether it works well across OSes.

Version 0.10.3 fixes the missing key/value parts, aims to fix the slight vertical skew in keys and, you convinced me, adds parts for link, external-link and internal-link in case you want to remove underlining.


chain.json

Please use a descriptive PR title

Hello @vinjan23, please wait for #5452 to be merged and work on that


Add Inpad v0.1.0

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[x] Named the cask according to the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.
[x] Checked there are no open pull requests for the same cask.
[x] Checked that the cask was not already refused in closed issues.

Ping @vitorgalvao RE https://github.com/caskroom/homebrew-cask/issues/28220.

Ping @vitorgalvao RE https://github.com/caskroom/homebrew-cask/issues/28220.

Consensus so far was 10 stars, so lets merge it.

Thanks for approving! Have a nice day. 😄


ArgoCD and Azure Private Repo

I've been trying to install and configure ArgoCD to use an Azure private repo.
I can't see any examples for Azure in the documentation so I have used the URL used when I clone the repo.
here is the configuration used
  argocd_applications =
    addons = {
      path                = "argocd/poc"
      repo_url            = "tfs-abcdef@vs-ssh.visualstudio.com:v3/tfs-abcdef/Apps.DevOps/ckc"
      ssh_key_secret_name = "2624-ckc-ado-ssh"
      add_on_application  = false
      insecure            = true
    }

The repo secret looks as expected so the parameters are being processed by terraform correctly.
I've checked the ssh key by cloning the repo locally but ArgoCD returns a connection timed out error.
I've tried prefixing it with ssh:// and also using git@ in the url.  But still no luck
Could someone suggest what I'm doing wrong

@SteveL1 : I'm using the following repo_url with no problems:
git@ssh.dev.azure.com:v3/organization/project/repository


Button currently looks a little small, especially in Chinese.

This is the proper size for english. Not sure if there should be a conversion of font sizes when using chinese.

@lweitzman No I haven't seen this. We should increase it by 2px if possible

just a clarification @tonycgil  thats a 2px increase for chinese only, right?

Raise issue to Clarity team. Let's wait for their response and solution.

@lweitzman yes. I got that number by doing simple photoshop resize of the font. We can wait to see what Clarity says per @steven-zou reply.

Outdated, close our issue now.


HWKBTM-14 Change integration tests to be based on Hawkular Accounts d…

…istribution instead of BTM distribution

:+1:


limit detrainment to 1/dt

Purpose

Missed in #1931, detrainment should be limited too.
To-do

Content

[ ] I have read and checked the items on the review checklist.

bors r+


Check that the court addresses are populating correctly

Courts for Lowell, MA addresses seem to be working fine for me.

Cannot reproduce and NLA has not had this issue more than once a while ago.


cpal not building properly on macOS

Describe the bug
I'm having trouble compiling my project that works on Windows and Linux perfectly fine for macOS. I recently got myself a Mac system and am using the exact same code base as on the 2 other platforms that I had before. There seems to be some kind of issue with coreaudio not being compiled properly through either flutter_rust_bridge or cargokit. I'm not sure where to post this issue so I just went here because you can probably help me out with that.
Steps to reproduce
This is quite hard to reproduce by hand, so I just created a GitHub repo you can clone to test it: https://github.com/Unbreathable/microphone_test.
Just running flutter run -v there reproduces the issue hopefully.
If you want to do it on your own machine without downloading my code, here is a simple tutorial on how to reproduce it step by step.

Create a new project with frb on macOS (you should be familiar with this xd).
Add cpal to the project by typing cd rust && cargo add cpal
Write some code to interact with cpal.

use std::{thread, time::Duration};

use cpal::{traits::{DeviceTrait, HostTrait, StreamTrait}, StreamConfig};

pub fn record() {
    thread::spawn(move || {
        // Get a cpal host
        let host = cpal::default_host(); // Current host on computer

        // Get input device (using new API)
        let device = host
            .default_input_device()
            .expect("no input device available"); // Current device

        // Create a stream config
        let default_config = device
            .default_input_config()
            .expect("no stream config found");

        let sample_rate: u32 = default_config.sample_rate().0;
        let _work_channels = 1; // Stereo doesn't work at the moment (will fix in the future or never)
        let mic_channels = default_config.channels();
        let config: StreamConfig = StreamConfig {
            channels: mic_channels,
            sample_rate: cpal::SampleRate(sample_rate),
            buffer_size: cpal::BufferSize::Fixed(4096),
        };

        // Create a stream
        let stream = match device.build_input_stream(
            &config.into(),
            move |data: &[f32], _: &_| {

                let mut max = 0.0;
                for sample in data.iter() {
                    if *sample > max {
                        max = *sample;
                    }
                }

                println!("Max: {}", max);
            },
            move |err| {
                println!("an error occurred on stream: {}", err);
            },
            None,
        ) {
            Ok(stream) => stream,
            Err(_) => {
                return;
            }
        };

        // Play the stream
        stream.play().unwrap();

        loop {
            thread::sleep(Duration::from_millis(100));
        }
    });
}

Try running the project again with flutter run -v (-v for verbose mode)

Logs
[        ] ld: symbol(s) not found for architecture arm64
[        ] clang: error: linker command failed with exit code 1 (use -v to see invocation)
[        ] note: Run script build phase 'Run Script' will be run during every build because the option to run the script phase "Based on dependency
analysis" is unchecked. (in target 'Runner' from project 'Runner')
[   +1 ms] ** BUILD FAILED **
[        ] The following build commands failed:
[        ]      Ld
/Users/julian/Documents/microphone/build/macos/Build/Products/Debug/rust_lib_microphone/rust_lib_microphone.framework/Versions/A/rust_lib_microphone normal
(in target 'rust_lib_microphone' from project 'Pods')
[        ] (1 failure)
[  +11 ms] Building macOS application... (completed in 2,237ms)
[        ] Error: Build process failed
[        ] "flutter run" took 7,968ms.
[   +6 ms]
           #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
           #1      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:815:9)
           <asynchronous suspension>
           #2      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1389:27)
           <asynchronous suspension>
           #3      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:19)
           <asynchronous suspension>
           #4      CommandRunner.runCommand (package:args/command_runner.dart:212:13)
           <asynchronous suspension>
           #5      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:355:9)
           <asynchronous suspension>
           #6      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:19)
           <asynchronous suspension>
           #7      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:295:5)
           <asynchronous suspension>
           #8      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:119:9)
           <asynchronous suspension>
           #9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:19)
           <asynchronous suspension>
           #10     main (package:flutter_tools/executable.dart:90:3)
           <asynchronous suspension>

Expected behavior
No response
Generated binding code
No response
OS
MacOS
Version of flutter_rust_bridge_codegen
2.0.0-dev.28
Flutter info
No response
Version of clang++
No response
Additional context
No response

Hi, could you please firstly run flutter run --verbose and paste the logs, usually there are some useful information

Possibly related: https://github.com/fzyzcjy/flutter_rust_bridge/issues/1690

Here is a log from the build process I uploaded to github gist: https://gist.github.com/Unbreathable/e59eaeff016cce3a95d9c0e26d356864.
After trying out the stuff from both of these issues, nothing worked for me.

So it seems that these symbols are not found:

[ +172 ms] Undefined symbols for architecture arm64:
[        ]   "_AudioObjectAddPropertyListener", referenced from:
[        ]       cpal::host::coreaudio::macos::property_listener::AudioObjectPropertyListener::new::h1235b9b10713a8d0 in librust_lib_microphone.a[arm64][46](rust_lib_microphone.3vtwaaftg5z9b48p.rcgu.o)
[        ]       cpal::host::coreaudio::macos::property_listener::AudioObjectPropertyListener::new::h2663444e4a7e3c55 in
librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       coreaudio::audio_unit::macos_helpers::RateListener::register::hf9d47954241a229f in librust_lib_microphone.a[arm64][199](coreaudio-0681dda8d128d987.coreaudio.208191b5c637fb5d-cgu.4.rcgu.o)
[        ]       coreaudio::audio_unit::macos_helpers::AliveListener::register::h44560ef786813afd in
librust_lib_microphone.a[arm64][199](coreaudio-0681dda8d128d987.coreaudio.208191b5c637fb5d-cgu.4.rcgu.o)
[        ]   "_AudioObjectGetPropertyData", referenced from:
[        ]       cpal::host::coreaudio::macos::Device::name::h3267e1a302e7fc99 in librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       cpal::host::coreaudio::macos::Device::name::h3267e1a302e7fc99 in librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       cpal::host::coreaudio::macos::Device::supported_configs::h8b1b8bd9f5e9f0e7 in librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       cpal::host::coreaudio::macos::Device::supported_configs::h8b1b8bd9f5e9f0e7 in librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       cpal::host::coreaudio::macos::Device::default_config::h745e25f92bd3e42e in librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       cpal::host::coreaudio::macos::set_sample_rate::hd82cfb9489818fba in librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       cpal::host::coreaudio::macos::set_sample_rate::hd82cfb9489818fba in librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       ...
[        ]   "_AudioObjectGetPropertyDataSize", referenced from:
[        ]       cpal::host::coreaudio::macos::Device::supported_configs::h8b1b8bd9f5e9f0e7 in librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       cpal::host::coreaudio::macos::Device::supported_configs::h8b1b8bd9f5e9f0e7 in librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       cpal::host::coreaudio::macos::set_sample_rate::hd82cfb9489818fba in librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       cpal::host::coreaudio::macos::enumerate::audio_devices::h6acff2c91b4ac468 in librust_lib_microphone.a[arm64][187](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.1.rcgu.o)
[        ]       coreaudio::audio_unit::macos_helpers::get_audio_device_ids_for_scope::h16645aa23d38445d in
librust_lib_microphone.a[arm64][199](coreaudio-0681dda8d128d987.coreaudio.208191b5c637fb5d-cgu.4.rcgu.o)
[        ]       coreaudio::audio_unit::macos_helpers::get_audio_device_supports_scope::h68b0af41548f0e42 in
librust_lib_microphone.a[arm64][199](coreaudio-0681dda8d128d987.coreaudio.208191b5c637fb5d-cgu.4.rcgu.o)
[        ]       coreaudio::audio_unit::macos_helpers::set_device_sample_rate::h09afa22c961f5af1 in librust_lib_microphone.a[arm64][199](coreaudio-0681dda8d128d987.coreaudio.208191b5c637fb5d-cgu.4.rcgu.o)
[        ]       ...
[        ]   "_AudioObjectRemovePropertyListener", referenced from:
[        ]       cpal::host::coreaudio::macos::property_listener::AudioObjectPropertyListener::remove_inner::h69a405e2141fd53a in
librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       coreaudio::audio_unit::macos_helpers::RateListener::unregister::hdb2276f85a20c291 in
librust_lib_microphone.a[arm64][199](coreaudio-0681dda8d128d987.coreaudio.208191b5c637fb5d-cgu.4.rcgu.o)
[        ]       coreaudio::audio_unit::macos_helpers::AliveListener::unregister::haaf6f952c9a737e7 in
librust_lib_microphone.a[arm64][199](coreaudio-0681dda8d128d987.coreaudio.208191b5c637fb5d-cgu.4.rcgu.o)
[        ]   "_AudioObjectSetPropertyData", referenced from:
[        ]       cpal::host::coreaudio::macos::set_sample_rate::hd82cfb9489818fba in librust_lib_microphone.a[arm64][186](cpal-4fe1478ffcebe343.cpal.b5ed2ec6efb8e2b1-cgu.0.rcgu.o)
[        ]       coreaudio::audio_unit::macos_helpers::set_device_sample_rate::h09afa22c961f5af1 in librust_lib_microphone.a[arm64][199](coreaudio-0681dda8d128d987.coreaudio.208191b5c637fb5d-cgu.4.rcgu.o)
[        ]       coreaudio::audio_unit::macos_helpers::set_device_physical_stream_format::hd28242475451425c in
librust_lib_microphone.a[arm64][199](coreaudio-0681dda8d128d987.coreaudio.208191b5c637fb5d-cgu.4.rcgu.o)
[        ]       coreaudio::audio_unit::macos_helpers::toggle_hog_mode::ha537b210c62c04b3 in librust_lib_microphone.a[arm64][199](coreaudio-0681dda8d128d987.coreaudio.208191b5c637fb5d-cgu.4.rcgu.o)
[        ] ld: symbol(s) not found for architecture arm64
[        ] clang: error: linker command failed with exit code 1 (use -v to see invocation)

What about asking in cpal, I guess it is like a general question "how to compile cpal on mac"

Alright, after some more trouble-shooting I figured out that I just had to link a few Apple frameworks statically. Sorry for wasting your time again. I truly appreciate everything you do for this community and your rust bridge helps me in so many ways. Thank you.
Now for everyone using cpal, here's how I fixed my problem:

Go into rust_builder/macos and then go into your .podspec file.
Somewhere near the end there should be an option called `OTHER_LDFLAGS. Add -lc++ -framework AudioToolbox -framework AudioUnit -framework IOKit -framework CoreAudio -framework OpenAL'`` to the end of that option, so it statically links everything cpal needs to operate. The line should then look something like this:

'OTHER_LDFLAGS' => '-force_load ${BUILT_PRODUCTS_DIR}/librust_lib.a -lc++ -framework AudioToolbox -framework AudioUnit -framework IOKit -framework CoreAudio -framework OpenAL',

@fzyzcjy I was also thinking we could put this onto the trouble-shooting page for flutter rust bridge. Just in case someone else stumbles upon the same issue. I've also seen some other people in the issues you linked describing the exact same issue, but not listing their solution and just saying roughly what to do (which is how I came up with this solution).
Again thanks for your time!

Alright, was about to do it. Anyway, thanks for doing it for me.


WIP: Workshop integration

Changes so far to integrate with the workshop controllers. Seeing a strange Porch error, I am going to simplify a little, I think there is some concurrency issue happening.

FYI commit-by-commit review may be easier to follow


28.0.0 Beta 1

:no_entry:  UnexpectedValueException: The files of the app "files_rightclick" were not correctly replaced before running the update

The one failing test is as expected before first major.


Add PS4 Controller

(I'm running linux, Fedora Silverblue 41).
I was working on gamepad guesser library based on GUIDs for love2d game framework. But found out that my DualShock 4 GUID is not in this database.
I used Steam's gamepad configuration window and got this string:
03008fe54c050000cc09000000016800,PS4 Controller,crc:e58f,platform:Linux,a:b0,b:b1,x:b2,y:b3,dpleft:h0.8,dpright:h0.2,dpup:h0.1,dpdown:h0.4,leftx:a0,lefty:a1,leftstick:b7,rightx:a2,righty:a3,rightstick:b8,leftshoulder:b9,lefttrigger:a4,rightshoulder:b10,righttrigger:a5,back:b4,start:b6,guide:b5,touchpad:b11,steam:2,
Then I adapted it to other PS4 strings and turned into PR.

6800 tail indicates the device is being handled by SDL's HID drivers and does not need a mapping per #478


Make PipPackage compatible with pip 10.x

Recently-released 10.x versions of pip use a different output format by default for pip list (columns). Here we are switching to specify that format explicitly, making the code compatible with both 9.x and 10.x versions of pip.

@philpep I updated the patch based on your feedback. Please take another look when you have a chance, thanks!

Merged with some changes to use --format=json for pip >= 9.
Thanks!

@philpep thank you!


docs: fix typo in README

s/bolean/boolean/

I'm sorry, a single letter change does not warrant a PR. However, I feel like lots of things in the readme are too complicated, so If you think you could simplify the readme overall, that would be appreciated.

Thanks for this project!
As a technical reviewer (for books), I tend not to look the other way even for something trivial like a misprint.
I'm just reporting the problems as I see them, and fixing those that can be fixed easily - which is easy for me to do online on GitHub. I wish I could be of more assistance, but nothing else stands out - or else I would have added it to the PR.


As an ECVM, I want to be able to see zoom+lat/lng in the url, so I can share and bookmark links to specific locations

Description
We should add zoom/lat/lng to the URL to make it easier for people to show specific locations on the map
Solution
Mapbox has a built-in solution for this which is super simple! (see here)
Describe alternatives you've considered

Without a hash you don't know your current zoom level, and can't share links

Tasks

[ ] Enable the hash feature in mapbox

Definition of "Done"

[ ] The URL displays zoom/lat/lng coordinates going forward

Change is available over here:
https://d2zjid6n5ja2pt.cloudfront.net/justice40-tool/27d947/en/cejst/?flags=mb

Looks good!


Branch 0.1.3

Pull Request Test Coverage Report for Build 39

90 of 90 (100.0%) changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 32:
0.0%

Covered Lines:
140

Relevant Lines:
140

💛  - Coveralls

Pull Request Test Coverage Report for Build 39

90 of 90 (100.0%) changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 32:
0.0%

Covered Lines:
140

Relevant Lines:
140

💛  - Coveralls


Add pg: ops.Store implementation

Codecov Report

Merging #114 into master will decrease coverage by 0.21%.
The diff coverage is 97.67%.

@@            Coverage Diff             @@
##           master     #114      +/-   ##
==========================================
- Coverage   99.79%   99.58%   -0.22%
==========================================
  Files          40       32       -8
  Lines        1989     1451     -538
==========================================
- Hits         1985     1445     -540
- Misses          2        3       +1
- Partials        2        3       +1

Flag
Coverage Δ

#dot_changes
100% <ø> (ø)
:arrow_up:

#dot_ops
98.97% <97.67%> (-1.03%)
:arrow_down:

#dot_ops_pg
97.67% <97.67%> (?)

#dot_refs
100% <ø> (ø)
:arrow_up:

#dot_streams
100% <ø> (ø)
:arrow_up:

#dot_streams_text
100% <ø> (ø)
:arrow_up:

#dot_test_seqtest
100% <ø> (ø)
:arrow_up:

#dot_x_fold
100% <ø> (ø)
:arrow_up:

#dot_x_idgen
60% <ø> (ø)
:arrow_up:

#dot_x_nw
98.88% <ø> (ø)
:arrow_up:

#dot_x_rle
?

#dot_x_rt
?

#dot_x_types
?

#dot_x_undo
?

Impacted Files
Coverage Δ

ops/pg/pg.go
97.67% <97.67%> (ø)

x/types/array.go

x/types/map.go

x/types/string.go

x/rle/rle.go

x/types/counter.go

x/undo/stream.go

x/rt/formatted.go

x/rt/run.go

x/undo/stack.go

... 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 7da9997...1c32d6b. Read the comment docs.


Incorrect response for admin.poolz.finance/lock-token-whitelists

Strapi collection LockTokenWhitelists has row #52 with type value Envelope

But response of admin.poolz.finance/lock-token-whitelists for #52 has incorrect value

There is fire wall with Cash, take up to 20 min after data change


I meet a awkward issue when I custom the text so I add those two code following your example:

       app:stl_customTabTextLayoutId="@layout/custom_tab_like_a_medium"
       app:stl_customTabTextViewId="@id/custom_text"

and comes with a error :Error:(191, 42) No resource found that matches the given name (at 'stl_customTabTextViewId' with value '@id/custom_text').   seek help @ogaclejapan

add + in id:
app:stl_customTabTextViewId="+@id/custom_text"

@liauli its true but i think you meant "@+id/custom_text"

oh yes, thank you for your correction


Fill username when accessing a team directly but login is required

Overview
Addresses https://github.com/FordLabs/retroquest/issues/18
Testing Instructions
Attempt to access a team/{ team-name } route

Fantastic work David, this really makes the app that much more polished.


Not work at all

After installed.
ctrl+shift+p > Indent XML
Nothing happen

Can you please provide console output (Ctrl + ~)?

Feel free to re-open when you can provide any details.


Add UpdateColumns which allows selective updates

Replaces #124
Fixes #92

Wrote this on a train whilst having forgotten that #124 existed. It's pretty much the same idea.

Rebased onto current master branch, to have a Travis config that works.

Thanks for the addition.


Resizing console window causes status bar to stay off screen

The only way to see the full status bar again is to resize the console back to the size you originally resized it to.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/26106269-resizing-console-window-causes-status-bar-to-stay-off-screen?utm_campaign=plugin&utm_content=tracker%2F6855523&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F6855523&utm_medium=issues&utm_source=github).

It seems to happen for any console that hasn't used up the initial whitespace at the top of the console.


Includes improved testing and bug fixes related to nested parameterized roles.

This is the first pull request I've ever made on github, so please let me know if it is incomplete in some way.
A team I work with uses parameterized roles almost exclusively, with very little conventional inheritance. I've been interested in visualizing this for some time, and your module was the first thing I've seen that could even come close. Even so, there appears to be an extra layer of anonymous roles when you are working with an object of 'MooseX::Role::Parameterized::Meta::Role::Parameterized'. The roles it consumes are accessible via ->get_roles, but its user-friendly name is accessible via ->genitor->name. ->genitor->get_roles() is always empty, which meant that the current implementation had a maximum depth of one (in obtaining the name it could no longer 'go back' to obtain the roles).
My changes effectively 'spam' duplicate edges that always use the name the user is interested in. Your existing code filters this down so that only one unique edge is ever added. In the process, all the interesting relationships get edges.

Can you provide some code that exercises this bug so I can see how your PR changes the output?

Sure. I'm using your module with a library my employer would like to keep
proprietary, but I'll mock up some test modules and put the test in a .t
file as soon as I have the opportunity.
On Mon, Aug 17, 2015 at 2:22 PM, Dave Rolsky<EMAIL_ADDRESS>wrote:

Can you provide some code that exercises this bug so I can see how your PR
changes the output?
—
Reply to this email directly or view it on GitHub
https://github.com/houseabsolute/Meta-Grapher-Moose/pull/1#issuecomment-131916632
.

I've added a test file which you can run against previous commits to see the difference in the edges discovered. I've also just now noticed the Travis CI failures, which I will also try to rectify.

Unfortunately, I haven't been able replicate the Travis CI failures to know what the underlying issues are that I need to fix. I've run perltidy against the files in question using the perltidyrc file in the repository; but since 112cb5c, this has not produced any new changes. In addition, and I suspect this is due to my lack of experience with it, I can't seem to get tidyall to work right -- I get these errors when I try to run 'tidyall -a':
carthanc: ~/Meta-Grapher-Moose $ tidyall -a
[checked] bin/graph-meta.pl
You are trying to redirect STDERR to a reference of type SCALAR
It can only be redirected to a file
Please check value of -stderr in call to perltidy
at /usr/local/share/perl/5.18.2/Code/TidyAll/Plugin/PerlTidy.pm line 21.
[checked] lib/Meta/Grapher/Moose.pm
You are trying to redirect STDERR to a reference of type SCALAR
It can only be redirected to a file
Please check value of -stderr in call to perltidy
at /usr/local/share/perl/5.18.2/Code/TidyAll/Plugin/PerlTidy.pm line 21.
[checked] t/nested_parameterized_roles.t
You are trying to redirect STDERR to a reference of type SCALAR
It can only be redirected to a file
Please check value of -stderr in call to perltidy
at /usr/local/share/perl/5.18.2/Code/TidyAll/Plugin/PerlTidy.pm line 21.
carthanc: ~/Meta-Grapher-Moose $
Do you mind giving me a push in the right direction?

You probably have an older version of Perl::Tidy installed. There's a regression in the latest version (which Travis pulls in) that tidies things a little differently from the past.

Thanks for being patient! It looks like the tests are all passing. What's next?

I'd like to improve the test I wrote. Do you have any suggestions?
One of the thoughts I had was building the packages programmatically at runtime and using UNIVERSAL::require to import them. That way the tests could be configuration driven rather than relying on so much boiler plate. I'd have to make a "syntax" for the config and write a subroutine for converting it into a compilable string. Maybe something like this:
    my $config = {
        ClassA => {
             ClassB => { RoleA => { RoleB => { RoleC => 1 } } },
             ClassC => { ParamRoleX => { ParamRoleY => { ParamRoleZ => 1 } } },
           }
        }
    };
    ...
    my $package = convert_to_package_string ( 'ClassA', 'ClassB', 'ClassC' );
    # "{ package ClassA; use Moose; extends 'ClassB', 'ClassC'; 1; }"
    $package->use; # via UNIVERSAL::require
    ...
    my $package = convert_to_package_string ( 'ClassB', 'RoleA' );
    # "{ package ClassB; use Moose::Role; with 'RoleA'; 1; }"
    $package->use; # via UNIVERSAL::require
    ...
    ok ( expect_edge( $graph, 'ClassA', 'ClassB' ), "Got the expected edge between ClassA and ClassB." );
    ok ( expect_edge( $graph, 'ClassA', 'ClassC' ), "Got the expected edge between ClassA and ClassC." );
    ok ( expect_edge( $graph, 'ClassB', 'RoleA' ), "Got the expected edge between ClassB and RoleA." );
    ...

Would this be useful direction to go? I've needed something like this module for a couple years now, and I'd really like to help out, if I can.

This has gotten kind of complicated, and my testing revealed bugs in my bugfixes, so I rolled back to v0.03 and added just my test file. I guess I should have started with a test-driven approach. In any case, we'll do it now.
The structure I chose to test is below. If you can think of some other complexity that would be good to add, we just need to extend the data structure.
          'ClassA' => {
                        'ClassC' => {
                                      'ParamRoleW' => {
                                                        'ParamRoleY' => {
                                                                          'ParamRoleZ' => 1
                                                                        },
                                                        'ParamRoleX' => 1
                                                      },
                                      'ClassD' => 1
                                    },
                        'ClassB' => {
                                      'RoleA' => {
                                                   'RoleB' => {
                                                                'RoleC' => 1,
                                                                'RoleD' => 1
                                                              }
                                                 }
                                    }
                      }
        };

The subroutines in the test file convert this into actual Perl packages, compile them during runtime, and then pass 'ClassA' to Meta::Grapher::Moose. Here is the image produced:

You can see how nested 'vanila' roles are visualized, but the parameterized roles are truncated at a depth of one.

Here are the results of the changes I made through yesterday (v0.04):

You can see that the parameterized roles are built out correctly. I didn't notice the self-edges until today and I'll fix them in the next commit. They are the link between the two roles (one is anonymous) that make up a single parameterized role.

Here is the result of removing the self-to-self edges as required to pass the more robust testing which did not expect them (v0.05):

I think this concludes everything I wanted to do with this merge request. Is there anything else that would help?

Sorry I've been slow to look at this. I'll try to get to it over this coming weekend.

For illustrative purposes:

This would be the graph if it was to show all the roles involved. You can see how the parameterized roles each have an anonymous role. This might be interesting to some users as a "verbose" mode, but I didn't take the time to code it that way or make any commits: I just temporarily commented out sections of the code and created the graph.
Also, the graph keeps flipping because I'm building the test modules at runtime from (randomly ordered) hashes. I could add sorting so that it converts the test data structure into code consistently each time.

I think you need to upload higher-res versions (or SVG is even better). I can't actually read the images.

Sorry about that, I didn't realize they were coming in so low on resolution. Is there a way to upload svg to github?

Hmm, doesn't look like GH lets you attach SVG, unfortunately. You could post them elsewhere and include a link.

http://carthanc.koding.io/meta-grapher-moose-test-v0.03.svg
http://carthanc.koding.io/meta-grapher-moose-test-v0.04.svg
http://carthanc.koding.io/meta-grapher-moose-test-v0.05.svg
http://carthanc.koding.io/meta-grapher-moose-test-verbose.svg
These will only work while I'm online. In the meantime, I'll try to get better jpgs.

Ok I've updated all the images above with readable images. So far I've collected:

[ ] Remove change to version
[ ] Add sorting to test to produce consistent a perfectly consistent test module hierarchy.

Anything else that would help?

I think I'm just going to redo this a bit to better fit my coding style. I could go point through point through your PR, but mostly the changes I want are to make the code more like how I'd write it.
What you've written is perfectly fine as-is, but it'll be easier for me to maintain this way. I should have a new version that incorporates these changes out later today.
Thanks for working on this!

Looking forward to seeing how you'll do it! This PR has been a straight up learning curve for me. Any suggestions on how I could approach this more effectively in the future?

Your PR was mostly fine. The big differences in what I'm working on are:

Mostly using the Moose meta API to generate classes instead of using eval (except for MXRP which makes this more difficult than its worth).
Changing the data structure that defines the set of packages to test.
Making it easier to write a whole bunch of test cases using this same test code without worrying about package name conflicts.
Various style nits that I care about and no one else is likely to notice ;)

Understood. If anything comes up on this module that you'd like to delegate, or if I can help in any way, please let me know. Thanks for putting this out there! I will be excited to see how it develops over time.

I fixed the underlying bug and added tests in b502a76c


fixed navbar issue in about us page, licensing page and also implemented a hamburger menu at width 600px

fixed issue #1011 and also there was a small issue in the licensing page fixed that too, I have started the implementation of hamburger menu and would be now visible at screen width of 600px(although not done completely).

Please resolve conflicts @supremeashu

It has conflicts @supremeashu

what was the conflict can you pls explain


Electron fails to start on Windows.

Even if the build issue due to go-ipfs-dep on Windows is fixed, a new issue appears.
When running npm start:
><EMAIL_ADDRESS>start C:\Users\matin\orbit
> npm run electron

><EMAIL_ADDRESS>electron C:\Users\matin\orbit
> electron .

fs.js:634
  return binding.open(pathModule._makeLong(path), stringToFlags(flags), mode);
                 ^

Error: ENOENT: no such file or directory, open 'C:\Users\matin\orbit\node_modules\electron-prebuilt\path.txt'
    at Error (native)
    at Object.fs.openSync (fs.js:634:18)
    at Object.fs.readFileSync (fs.js:502:33)
    at Object.<anonymous> (C:\Users\matin\orbit\node_modules\electron-prebuilt\index.js:4:42)
    at Module._compile (module.js:541:32)
    at Object.Module._extensions..js (module.js:550:10)
    at Module.load (module.js:458:32)
    at tryModuleLoad (module.js:417:12)
    at Function.Module._load (module.js:409:3)
    at Module.require (module.js:468:17)

npm ERR! Windows_NT 10.0.14936
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "run" "electron"
npm ERR! node v6.2.2
npm ERR! npm  v3.9.5
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>electron: `electron .`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>electron script 'electron .'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the orbit package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     electron .
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs orbit
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls orbit
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\matin\orbit\npm-debug.log

npm ERR! Windows_NT 10.0.14936
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "start"
npm ERR! node v6.2.2
npm ERR! npm  v3.9.5
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `npm run electron`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'npm run electron'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the orbit package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run electron
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs orbit
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls orbit
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\matin\orbit\npm-debug.log

There is nothing interesting in npm-debug.log.

@mateon1 thank you very much for reporting the issue! I'll work to pull in the latest version of Electron that you referenced. Can you confirm that that 1.4.3 fixes the issue for you?
We haven't done any work on Windows yet, so anything may come across is uncharted territory. Would you be interested in trying to fix those issues? Regardless, much appreciate the bug report.

@haadcode Yes, I tested the install updating Electron to 1.4.3, and everything seemed to install correctly (after npm link-ing packages depending on go-ipfs-dep with the Windows fix).
Now, my Windows machine is a Frankenstein's monster with GNU tools available on the path (like rm, which some post-install scripts seem to use), and properly interpreting .sh scripts (which are also used as post install scripts), so it working on my machine doesn't imply it working on any Windows machine.
There is still a lot of work to do for Windows, but this gets us significantly closer.

Yes, I tested the install updating Electron to 1.4.3, and everything seemed to install correctly

Great to hear! I'll make sure to pull the new version in as soon as I get the time to do it.
Re. those shell scripts, ideally we shouldn't use any shell scripts in the development environment (due to the reason you laid out above). The scripts were created to quickly get all the different custom versions of ipfsd-ctl, js-ipfs-api, go-ipfs-dep etc. to work together due to version conflicts with pretty much in all of them (eg. ipfs-api got pulled in 5(!) times, each from different package, each different version 😢). I expect that we can completely remove the scripts as soon as the work on go-ipfs-dep and js-ipfs-api is done to support a) custom version of go-ipfs binary and b) pubsub support in *-ipfs-api. @mateon1 what do you think, does that make sense?
cc @diasdavid @dignifiedquire see above the comment about upcoming  work on the ipfs packages. I can make sure go-ipfs-dep works, but would need some help on publishing custom versions (like, I want to be able to require<EMAIL_ADDRESS>

@mateon1 pushed Electron 1.4.3 dep to master.

Adding this issue to a milestone for the next quarter. The goal is to make Orbit work on Windows too.


Update test

Testing first marketplace article change for work item ISSUE 24531

@tschwebke : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@tschwebke do you still need this PR open? If not, can you please close? Thank you

@ktoliver can you please close this PR?


Error: "Analyze PDF form : Operation returned an invalid status code 'NotFound'"

This code initially worked, so I expanded the training documents to 30, and am now getting this error:
"Analyze PDF form : Operation returned an invalid status code 'NotFound'"
Each document is in the same directory as the program file. Each is called with these two lines of code:
private const string pdfFormFile1 = @"TrainingDoc1.pdf";
await AnalyzePdfForm(formClient, modelId, pdfFormFile1);
Interestingly, if I try to use the path for the file, it will say that it can't be found, which I would expect. When I call them like this, I get the error. It seems to me that the documents are being called correctly, but something else isn't working...
The rest of the code is unchanged from above.
Any ideas?

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

ID: 23d8b75f-c509-401f-2a66-4f7d1dab8f93
Version Independent ID: 94094caa-ebb4-6752-0ff7-04331a2fa31b
Content: Quickstart: Form Recognizer client library for .NET - Azure Cognitive Services
Content Source: articles/cognitive-services/form-recognizer/quickstarts/dotnet-sdk.md
Service: cognitive-services
Sub-service: forms-recognizer
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@davenull311 are you saying the code only works when you add one document?

@GiftA-MSFT, not exactly. I'd originally edited the code to accept 5 of my training docs. I ran 4-5 runs and it gave reasonable results. So I built out the code to include 30 documents, simply copying/pasting each document into the code appropriately. That's when it didn't work.
I considered that the SAS may need to be updated, but I've done that and it still leaves me with this same message. Is there another time-sensitive key/setting that I'm missing that needs to be updated for each batch of runs?

@davenull311 thanks for the followup. From your post it is hard to understand if you are getting this error on train or Analyze. If you are referring to analyze, it seems you are sending a blog storage to Analyze, however, analyze accepts a single file as input and can be called in parallel on several files to support batch but that API is file by file.

@davenull311 will proceed to close this thread, but comment down below if you have additional concerns and we can reopen the issue. Thanks.


Hacking-fileRename _vendors-content/general/user.md to Security🔒

Mai is file ko hacking ke liye use karna chahunga

This app is very interesting


File Explorer Breaks on 21H1

the title bar disappears and sometimes it shows the win10 one

IT IS NOT FIXABLE a solution to this would be to exclude explorer.exe and install secure UX theme and use Aero7 Basic theme then you will get the Basicthemer2 title bars for all other programs but the the aero 7 title bars for windows explorer and control panel


Use ConcurrentDictionary For Improving Get Enum Name

@Mahdigln  Please reply to the policy bot above to validate the license check

@microsoft-github-policy-service agree


Fix typo in README

🚨 Proposed changes
Just a small typo fix.
⚙ Types of changes
What types of changes does your code introduce? Put an x in the boxes that apply

[ ] New feature (non-breaking change which adds functionality)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation Update (if none of the other choices apply)
[ ] Refactor

Sorry I've missed a thing in the review :pray:
We're using convetional-commit to lint our commits, can you please rewrite the message so we can merge it? 😄
You can use the following commands:
git commit --amend -m "docs(README.md): fix typo"
git push origin patch-1 -f

@simonecorsi I'm not sure why it took me over a month to do this 😅 Just got lost in the backlog, sorry.
I fixed the commit message, sorry for not reading the guidelines properly before :)

@simonecorsi I'm not sure why it took me over a month to do this 😅 Just got lost in the backlog, sorry.
I fixed the commit message, sorry for not reading the guidelines properly before :)

Don't worry! It happens to me all the time! 😄
Thanks for fixing it!

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fixes #85 The NeutralRole class was defined in role.ttl (which is cor…

…rect). However, in victim.ttl it was referenced with an improper namespace.

Looks good


Shutdown the log

Hi, i'm changing my language project, coming from Javascript NodeJS to Python, so i'm little lost for this little thing.
I'm pretty new in Python ^^
I have a message that i would like to delete, i don't want use multiprocessing 'cause i'm using it for another thing in my project.
[i] Set 'df.ta.mp = True' to enable multiprocessing. This computer has 12 cores. Default: False

Someone know how i can shut down the log of pandas-ta ?

Hello @Morgiver,
Welcome to Python and thanks for trying Pandas TA!
To remove the following message:
[i] Set 'df.ta.mp = True' to enable multiprocessing. This computer has 12 cores. Default: False
Edit /path/to/pandas_ta/core.py and remove the following lines in the strategy() method:
            if random() < 0.1:
                print(f"[i] Set 'df.ta.mp = True' to enable multiprocessing. This computer has {cpus} cores. Default: False")

Hope that helps!
Drop a :star: if you find the library useful.
Thanks,
KJ

hi ! Thank you, exactly what i need :)
It could be interresting to have a parameter to set the verbose on or off !

@Morgiver,
Thanks for the prompt reply. It only shows 10% of the time it runs so it's not too invasive and just a helpful hint.
Thanks for the suggestion. Maybe I'll make a hints property instead since it really doesn't fall into the verbose category.
How come you are switching to Python from JS?
Regards,
KJ

@twopirllc
I'm working on a Trading Bot.
NodeJS is really slow and the multithreading is not easy to handle. I take one big week to approach Python and change the langage on my project. It was more easy than i expect.
Python have many really good package, i'm demanding my self why i don't take Python in the first place sooner.
Now i don't even look at JS, Python is my new girlfriend :p
I have a lot of work to handle Python properly but it's a pleasure to discover it !
Your package is really usefull, i thank you for your great work ! :)
Regards,
Morgiver

@Morgiver,
Thank you! It's come along way from a private repo to Open Sourcing as well as learning Python 3 and the other plethora of frameworks and libraries in the Fintech spectrum.
A couple of years ago I built a MVP Discord Bot. It would download data, do TA, make chart and send it back to the channel. Now I need a compatible back testing library and then build a better Bot again soon.

Now i don't even look at JS, Python is my new girlfriend :p

😆
Regards,
KJ


The http parser is not async, why does it hook into the async_hook machinery?

The http parser does quite a bit of additional JS/C++ transitions by being observable via async_hooks. In fact, a wasm build is faster than C++ by avoiding that.
(this is just a hunch, but the benefits should be there).

Can you reference some files you saw?
I did a little exploration on http.Agent and it looks like it exists to fix issues like this https://github.com/nodejs/node/issues/34401
Maybe after https://github.com/nodejs/node/pull/48528 lands we can explore whether it's possible to remove it.

This is regarding the http server implementation. http.Agent is for the client, is already pretty much inofficially deprecated in favor of undici.

Someone at some point decided they wanted to track http parsing as a resource as an http request would otherwise just look like a plain TCP socket. There's no new "resource" object created for an http request, it just upgrades a TCP socket to an HTTP connection. Because of how async_hooks is tied to the concept of a single id mapping to a single resource object the resource object of the TCP socket can't be reused for the HTTP socket events without breaking things. The choice was either don't differentiate HTTP connections in async_hooks or find some other object to be the "resource" of the http request. For whatever reason it was decided that making the http parser the resource was an appropriate solution. That decision then also led to discovering the http.Agent pooling problem on the client request side, but the use of the http parser as the resource object for an HTTP requests happens for both incoming and outgoing HTTP.
This is one of the countless reasons I want us making more purpose-focused APIs so we can mark the entire async_hooks API as deprecated or internal. Unfortunately removing this behaviour will probably break users, which is why I'm pushing so hard on getting AsyncLocalStorage off async_hooks so we can get the biggest use case off of it and not have to care so much anymore about breaking things.

It's all about tracking individual async "operations". For a long time async hooks was the thing to track this. By introducing AsyncLocalStore this was moved a bit towards using the async resource object instead keeping a separate map.
Still init hook is needed for propagation.
The current PR from @Qard moves parts of this propagation work into v8 as there was helpful functionality added.
There is work ongoing to move even more into the JS language (see https://github.com/tc39/proposal-async-context).
But still it's usercode which has to somehow "mark" individual operations (HTTP Requests) on shared resources (Socket). This is currently done via AsyncResource/AsyncLocalStore.run(),AsyncLocalStore.snapshot(),...
Besides HTTP there are quite a lot other locations inside node core which require special handling because v8 is not aware of it (fs, timers,...see async_wrap for details).

Unfortunately removing this behaviour will probably break users

@Qard Why? All code in http parser is synchronous, so there is literally no reason for it. The only fix that might be required is adding an AsyncResource somewhere.

It's all about tracking individual async "operations".

@Flarna the http parser is not async at all. It's a purely synchronous library.

The fact that the http parser has this behavior implemented in C++ (vs JS) requires more back-and-forth across the C++/JS boundary, slowing everything down. Undici got a performance speed-up by dropping the native http parser and moving to the same code running inside WASM (wasm is slower than native code).

It's not needed for async tracking. It is needed for differentiating raw TCP connections from HTTP connections.


chore: set stream.replicas=1 when in non-cluster mode

When jetstream is not in cluster mode, automatically sets the stream/KV replica to 1. @joelcomp1
	if r.isbSvc.Spec.JetStream.GetReplicas() < 3 {
		// Replica can not > 1 with non-cluster mode
		v.Set("otbucket.replicas", 1)
		v.Set("procbucket.replicas", 1)
		v.Set("stream.replicas", 1)
	}

So is that KV store configuration different then the one here: https://github.com/numaproj/numaflow/blob/86df4a84083613cf3912f35ada400434ce1b200c/pkg/isbsvc/jetstream_service.go#L100
must be?

So is that KV store configuration different then the one here:
https://github.com/numaproj/numaflow/blob/86df4a84083613cf3912f35ada400434ce1b200c/pkg/isbsvc/jetstream_service.go#L100
must be?

We can use the stream.replicas for sideinput KV as well for now, can always revisit later one.


Add a test to confirm that all arrays can be empty or null

We need to ensure that we support arrays that are:

[ ]  empty & present
[ ] null & present
[ ] not present

Without unintended side effects (like crashing).

For absolute schedule, need to ensure that any of the above can never be interpreted to:
    "absolute": [
        { "unix_time": 0, "indexes": null},
    ]

Just to be a little technical, and array is "[...]", and empty/null array is "[]"
Anything like {....} is a map.
An it would be a syntax violation to have "Foo": "Another_Thing":
In other words "not present" would mean the whole map is not present, i.e. "absolute": [...] is completely missing.  Which means we probably need to have a JSON validation routine (I think seshat had something similar)

Following are the testcases Wes provided that need to be added:

"index": null == "index": []  ---->  block nothing
"index"  missing -----------------> error
"weekly": null == "weekly":[]  == no weekly schedule

OK, but can'r have {"index":} That would/must cause an error on the sever side.

Also, all of these MUST have consequences as far as the "scheduler" goes, errors such as missing "indexs" should not crash the code, but what is the "scheduler" supposed to do? Assume indexes did not change" Or just throw away the new schedule?

Yes, the payload needs to be valid msgpack, so { "index": } without either a null or empty array is invalid.

How about just {} ? Is it valid?

Nop. "{}" is invalid msg pack.


Support Flatpak

Would the BitPay team be interested in supporting Flatpak widely used in RedHat OS family? Should be similar to how Snap works.

And Gentoo, and Arch, and, openSuse, and so on...
I'm baffled why snaps were chosen as the default when basically only Ubuntu distros use them


Prevent instances of CastorClient overwriting each other's tokens

CastorClient.headers is defined as a class attribute rather than an instance attribute.
This is a problem because authorization updates self.headers to set the bearer token, which means that if a user were to
create a second instance of CastorClient, it would overwrite/redefine the token used by the first client.
By changing headers from a class attribute to an instance attribute, the problem is avoided.
Note: I split this up into two commits, so it's possible to see the test fail when checking out just the first commit. The test case is a bit of a special-case for functionality that is unlikely to regress, so consideration could be given to removing it. However, I made it to serve as an example/scaffolding for another upcoming change.

@zoni, first of all, thanks for your contributions. Quick question: aren't all the commits of this PR not also covered in #77?

Added in #79


Fix tarball build and add travis test for tarball build

@hannesmuehleisen

The tarball test still runs within a valid git clone which git will find when traversing upwards


Add support for ingesting Mlab NDT data from bigquery

Currently the table used to populate the performance related API endpoints was done as a one-off sketchy script.
We should ideally do this as a daily pipeline batch operation which takes the metrics from bigquery using a query like:
#legacySQL
SELECT client_asn_number,
       client_asn_name,
       client_country_code,
       SUM(rtt_sum) / SUM(rtt_count) AS rtt_avg,
       AVG(packet_retransmit_rate) AS retransmit_avg,
       nth(51, quantiles(download_speed_mbps, 101)) AS download_speed_mbps_median,
       nth(51, quantiles(upload_speed_mbps, 101)) AS upload_speed_mbps_median,
       COUNT(*) AS count
FROM [mlab-oti.data_viz.all_ip_by_hour]
WHERE LENGTH(client_asn_name) > 0
 AND LENGTH(client_asn_number) > 0
 AND local_test_date >= '2018-03-01 00:00:00'
 AND local_test_date < '2018-04-01 00:00:00'
GROUP BY client_asn_number,
client_asn_name,
client_country_code
ORDER BY client_country_code;

Other useful links:
https://github.com/m-lab/mlab-vis-pipeline/tree/master/dataflow/data/bigtable/queries
https://data-api-dot-mlab-sandbox.appspot.com/locations/euit/clients/AS30722/metrics?startdate=2016-03&enddate=2019-05&format=json&download=1&timebin=month
https://github.com/m-lab/mlab-vis-pipeline/blob/beam-api-upgrade/dataflow/data/bigquery/queries/base_uploads_ip_by_day.sql & https://github.com/m-lab/mlab-vis-pipeline/blob/beam-api-upgrade/dataflow/data/bigquery/queries/base_uploads_ip_by_day.sql

This is related to: https://github.com/ooni/pipeline/issues/78


bug fixes for rotation and FWCPOS, orbit/subarray seettings and all v…

This PR address four separate bugs:

wfssconfig: FWCPOS should a scalar, but config code reads in as a 1-element vector.  So need [0].  Also, the rotation is applied in reverse, so a negative sign is misplaced on the rotation matrix
wfss: orbit/subarray are not getting set correctly.  Change all to WCS only transformations.  This is because things fail for images that are far apart.
region: many changes had to happen. It was incorrectly assumed that segmentation regions were "singles" (meaning that all of the pixels associated with an ID would be connected), but this is not true in practice where some bits might be related to the same ID but not connected.  This fixes that error.  This amounts to needing a loop over regions, where before things just indexing the 0th element.
single: the default values for some keywords were not getting set properly.  This fixes that.

I see nothing objectionable on a quick skim, but I'm going to see if I can figure out why CI isn't running (as of two months ago, it seems) before approving/merging.

FYI I re-enabled the CI and pushed codestyle fixes to see it run.


A couple of ways to improve the script

I just tried this and it seems to download the full resolution video with yt-dlp. I think it would be better if it just downloaded the audio, as it would save data and time.
Also, it would be nice if it would convert the video to mp3 while the next video is downloading. It would speed things up a lot (especially for longer videos), but it might be tricky to do.
Thanks for reading

It never downloaded the video itself, as "-x" flag or now "bestaudio" format downloads audio-only .webm. Then the conversion to .mp3 is made. You can try opening .webm using media player - it won't show any video.
As per speeding things up, it seems like the best possible option in yt-dlp for now is "-N" or its JSON analog "concurrent_fragment_downloads", which speeds up downloads, but sadly does nothing with postprocessing etc.


Support scanning k3s images with grype

from @dlorenc
Today, Grype can't determine which k3s version is used in k3s itself because it's built with the file. Here's what a scan looks like:
NAME                   INSTALLED  FIXED-IN  TYPE       VULNERABILITY        SEVERITY
github.com/k3s-io/k3s  (devel)    1.24.17   go-module  GHSA-m4hf-6vgr-75r2  High

It would be nice if the K3s binaries could be properly scanned by this tool to determine the version.

Validated on Version:
-$  k3s version v1.28.2+k3s-3d25e9f6 (3d25e9f6)

Environment Details
Infrastructure
Cloud EC2 instance
Node(s) CPU architecture, OS, and Version:
PRETTY_NAME="Ubuntu 22.04.1 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
Cluster Configuration:
1 node servers
Steps to validate the fix

Install k3s with commit
Install grype https://github.com/anchore/grype#installation
Run grype on the binary

Reproduction Issue:

k3s version v1.28.2+k3s-9597ea11 (9597ea11)

grype /usr/local/bin/k3s

 ✔ Vulnerability DB                [no update available]
 ✔ Indexed file system                                                                                /usr/local/bin
 ✔ Cataloged packages              [33 packages]
 ✔ Scanned for vulnerabilities     [1 vulnerability matches]
   ├── by severity: 0 critical, 1 high, 0 medium, 0 low, 0 negligible
   └── by status:   1 fixed, 0 not-fixed, 0 ignored
[0000]  WARN some package(s) are missing CPEs. This may result in missing vulnerabilities. You may autogenerate these using: --add-cpes-if-none
NAME                   INSTALLED  FIXED-IN  TYPE       VULNERABILITY        SEVERITY
github.com/k3s-io/k3s  (devel)    1.24.17   go-module  GHSA-m4hf-6vgr-75r2  High

Validation Results:

k3s version v1.28.2+k3s-3d25e9f6 (3d25e9f6)

ubuntu@ip-172-31-25-240:~$ grype /usr/local/bin/k3s
 ✔ Vulnerability DB                [no update available]
 ✔ Indexed file system                                                                                /usr/local/bin
 ✔ Cataloged packages              [33 packages]
 ✔ Scanned for vulnerabilities     [4 vulnerability matches]
   ├── by severity: 0 critical, 1 high, 3 medium, 0 low, 0 negligible
   └── by status:   4 fixed, 0 not-fixed, 0 ignored
[0000]  WARN some package(s) are missing CPEs. This may result in missing vulnerabilities. You may autogenerate these using: --add-cpes-if-none
NAME                   INSTALLED  FIXED-IN  TYPE       VULNERABILITY        SEVERITY
github.com/k3s-io/k3s  (devel)    1.24.17   go-module  GHSA-m4hf-6vgr-75r2  High
golang.org/x/net       v0.8.0     0.17.0    go-module  GHSA-qppj-fm5r-hxr3  Medium
golang.org/x/net       v0.8.0     0.17.0    go-module  GHSA-4374-p667-p6c8  Medium
golang.org/x/net       v0.8.0     0.13.0    go-module  GHSA-2wrh-6pvc-2jm9  Medium

as per see on the comment validation above it still not showing the version and showing  (delve) as per discussion with @brandond  we are leaving this open for this release.

@fmoral2 On the newest commits I am no longer seeing any problems
curl -sfL https://get.k3s.io | INSTALL_K3S_COMMIT=19fd7e38f674bddaa4571373d767c48bc52867f0 sh -

root@server-0:/home/vagrant# grype /usr/local/bin/k3s
 ✔ Vulnerability DB                [updated]
 ✔ Indexed file system                                                                                /usr/local/bin
 ✔ Cataloged packages              [32 packages]
 ✔ Scanned for vulnerabilities     [0 vulnerability matches]
   ├── by severity: 0 critical, 0 high, 0 medium, 0 low, 0 negligible
   └── by status:   0 fixed, 0 not-fixed, 0 ignored
[0000]  WARN some package(s) are missing CPEs. This may result in missing vulnerabilities. You may autogenerate these using: --add-cpes-if-none
No vulnerabilities found
root@server-0:/home/vagrant#

@dereknola I think the issue is that when there are vulns, it shows the installed version as (devel) instead of the actual k3s release version.

Can we update the milestone for this issue and related backports as it seems its not fixed for this release?
CC @brandond @carolid @rancher-max

$ k3s -v
k3s version v1.29.1-rc2+k3s1 (d8907ce6)
go version go1.21.6

$ grype /usr/local/bin/k3s --add-cpes-if-none
 ✔ Vulnerability DB                [no update available]
 ✔ Indexed file system                                                                                /usr/local/bin
 ✔ Cataloged contents                                                                                d8ed07a164b13b247b4ab1d5ad19f65e57b4058078258b284ab6196eee7c85c0
   └── ✔ Packages                        [32 packages]
 ✔ Scanned for vulnerabilities     [0 vulnerability matches]
   ├── by severity: 0 critical, 0 high, 0 medium, 0 low, 0 negligible
   └── by status:   0 fixed, 0 not-fixed, 0 ignored
No vulnerabilities found

$ k3s -v
k3s version v1.28.6-rc2+k3s1 (39a00015)
go version go1.20.13

$ grype /usr/local/bin/k3s --add-cpes-if-none
 ✔ Vulnerability DB                [no update available]
 ✔ Indexed file system                                                                                /usr/local/bin
 ✔ Cataloged contents                                                                                74c0a49904f4e6a801b866d93f5f949428f8f2ae667d1ae07f8004d04f2f6c5b
   └── ✔ Packages                        [32 packages]
 ✔ Scanned for vulnerabilities     [0 vulnerability matches]
   ├── by severity: 0 critical, 0 high, 0 medium, 0 low, 0 negligible
   └── by status:   0 fixed, 0 not-fixed, 0 ignored
No vulnerabilities found

Modified comments above to reflect requested changes.


[beta][UI] Change image size on following challenges

link to challenges:

http://beta.freecodecamp.com/en/challenges/basic-html-and-html5/add-images-to-your-website
http://beta.freecodecamp.com/en/challenges/basic-html-and-html5/link-to-external-pages-with-anchor-elements
http://beta.freecodecamp.com/en/challenges/basic-html-and-html5/link-to-internal-sections-of-a-page-with-anchor-elements

these pages should include img{width:100%;}

@tommygebru - thanks for going through these challenges is such detail! So, we don't want to show any CSS in the seed code for this section, since it comes before campers have learned it. Unfortunately, the output ends up looking a little rough...
@freeCodeCamp/moderators - the cat image that's used here is a bit bigger than the phone's width. Should we look into making it a touch smaller (behind the scenes), so it fits better, or is it okay as is? 🐱 📸

Yeah - the list of challenges include any with the CatPhotoApp. I would suggest only changing the image itself or if its easier we can include the image width be 100%
The website looks good.
I hope that we can change 2 things before release:

wider instructions panel (easier to read lengthy posts)
a single notification system (absolving the modal toast)

maybe brighter text and green color for the Night Mode UI

I would add that we might want to move the preview down below the code input so it can become wider (a switch to toggle between phone and landscape iPad view might be nice, for example).
This way the directions and code could be wider.
We could also potentially remove the fixed height from these elements and let users scroll up and down as necessary.
There is a lot of room for experimentation here imho.


feat: support influxdb

Thanks to the team for this great testing utility :pray:
This PR brings the possibility to spawn a Docker instance of InfluxDB (timeseries-oriented database) for integration testing. This PR supports both 1.x and 2.x versions of InfluxDB, which rely on different query languages and require the use of different Python client libraries. Therefore:

there is a root InfluxDbContainer class in testcontainers/influxdb.py for the common mechanisms (health check, connection url, etc.)
there are 2 separate classes, one for each InfluxDB major version: InfluxDb1Container class in testcontainers/influxdb1/__init__.py, InfluxDb2Container in testcontainers/influxdb2/__init__.py that are meant to be used by the testcontainers users. Each one has its own .get_client() method involving the dedicated Python client library

Unit and integration tests (meant to be examples for the testcontainers end-users) are provided in the PR.
I declared the 2 Python client libraries as dependencies in setup.py so that the continuous integration of testcontainers can be launched. I wonder whether it implies that both clients will be installed by the testcontainers end-users although they are likely to need only one: either the 1.x client or the 2.x client. Installing the 2 but using only 1 works, but it slows down the installation and test processes. I am open to advice from the maintainers for that :pray:

in the meantime, I created the https://github.com/Purecontrol/testcontainers-python-influxdb to publish the https://pypi.org/project/testcontainers-python-influxdb/ package.

please review:
https://github.com/testcontainers/testcontainers-python/compare/main...alexanderankin:testcontainers-python:feat/support-influxdb
fixed a test here:

https://github.com/testcontainers/testcontainers-python/commit/2e219285e6db2afd6c4aaae9c44bdccb7043f06b
https://github.com/testcontainers/testcontainers-python/compare/main...alexanderankin:testcontainers-python:feat/support-influxdb_adjust

hello @alexanderankin
Thank you for the feedback on this topic :pray:
I am sorry, I am not sure to understand what you want me to do :disappointed: most of the links you sent me are links to create pull requests on your github account. Don't you want to create them first and then ask me to review them?
Or would you like me to "cherry-pick" the changes on the PR I opened?

Sorry I should have clarified - can you confirm that I did not make any
unintentional changes while rebasing? I would force push but I think it's
impolite without first asking
On Fri, Mar 8, 2024, 3:51 AM Luc Sorel-Giffo @.***>
wrote:

hello @alexanderankin https://github.com/alexanderankin
Thank you for the feedback on this topic 🙏
I am sorry, I am not sure to understand what you want me to do 😞 most of
the links you sent me are links to create pull requests on your github
account. Don't you want to create them first and then ask me to review them?
Or would you like me to "cherry-pick" the changes on the PR I opened?
—
Reply to this email directly, view it on GitHub
https://github.com/testcontainers/testcontainers-python/pull/413#issuecomment-1985292912,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACECGJC4X4O4SGHUZCUGDQLYXF3ZPAVCNFSM6AAAAABB4WT432VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBVGI4TEOJRGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

If it's okay with you I will replace your 9 original commits with the seven commits which result from rebasing onto the new structure (two of them edited files which no longer exist, such as setup.py)

thank you for clarifying and asking.

If it's okay with you I will replace your 9 original commits with the seven commits which result from rebasing onto the new structure (two of them edited files which no longer exist, such as setup.py)

Yes, that is fine :+1:

there is just one change that should be done: those lines (https://github.com/testcontainers/testcontainers-python/compare/main...alexanderankin:testcontainers-python:feat/support-influxdb_adjust#diff-d05626efd4466a0f9ddf3973c13071ea2e534dc69a295e68cc9fc3e5a5541e0fR115-R145) should be removed because they force the main container class to import the dependencies for InfluxDB version 1.x and 2.x, whereas users may use either one or the other, but it is not a good idea to force them to install the dependencies of both versions.
Instead, both InfluxDb1Container and InfluxDb2Container classes are independent and have their own get_client() method.
Once you have force-pushed, your changes, I can add a commit to remove those unnecessary lines and imports, if you like?

Yes feel free to yourself as well as I will be at work. Since the commits
are pushed to GitHub public repo you have access to everything
On Fri, Mar 8, 2024, 4:17 AM Luc Sorel-Giffo @.***>
wrote:

there is just one change that should be done: those lines (
main...alexanderankin:testcontainers-python:feat/support-influxdb_adjust
#diff-d05626efd4466a0f9ddf3973c13071ea2e534dc69a295e68cc9fc3e5a5541e0fR115-R145
https://github.com/testcontainers/testcontainers-python/compare/main...alexanderankin:testcontainers-python:feat/support-influxdb_adjust#diff-d05626efd4466a0f9ddf3973c13071ea2e534dc69a295e68cc9fc3e5a5541e0fR115-R145)
should be removed because they force the main container class to import the
dependencies for InfluxDB version 1.x and 2.x, whereas users may use either
one or the other, but it is not a good idea to force them to install the
dependencies of both versions.
Instead, both InfluxDb1Container
https://github.com/testcontainers/testcontainers-python/compare/main...alexanderankin:testcontainers-python:feat/support-influxdb_adjust#diff-bd64360629a8b2ac551283a6d10500059bfb36e17340e484bc1d8fd0322c81ccR47-R63
and InfluxDb2Container
https://github.com/testcontainers/testcontainers-python/compare/main...alexanderankin:testcontainers-python:feat/support-influxdb_adjust#diff-e5cc1b8aa3516d4eff82f2f95976f6aeeb926b75a12a7d8aea92a2a043a287aaR82-R102
classes are independent and have their own get_client() method.
Once you have force-pushed, your changes, I can add a commit to remove
those unnecessary lines and imports, if you like?
—
Reply to this email directly, view it on GitHub
https://github.com/testcontainers/testcontainers-python/pull/413#issuecomment-1985331538,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACECGJENSR7XT7VIWJBLY53YXF6Y3AVCNFSM6AAAAABB4WT432VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBVGMZTCNJTHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

actually im not sure where the influxdb/__init__.py came from - maybe this is an artifact of a bad rebase - seems more cleanup is necessary as well

figured it out, fixing

fixed

feel free to layer your changes on this i rebased to address the part about the client_v1 and v2 so its in your repo now where you can work with it normally. feel free to run the "pre-commit run -a" to test linting locally as well

thank you for fixing the client_v1 and _v2.
I don't have anything to add, but I can easily fix the linting issues if you like.

I see that testcontainers-python now uses pre-commit to run black for formatting and ruff for linting. I would suggest to use ruff-format as a drop-in replacement for black, but maybe in another issue?

I pushed the commit related to the pre-commit hooks and the lint hooks now passes :+1:
I am just wondering how to launch the influxdb unit tests locally. I believe the steps are:

install the optional dependencies for influxdb only
run the tests for influxdb only
I am not sure how to do that, now that the codebase has been moved to the modules folder

poetry install -E influxdb seems to do the job for the dependencies

Poetry install -E arangodb
Poetry shell
Pytest modules/arangodb
Poetry run pytest/modules/arangodb
On Fri, Mar 8, 2024, 6:49 AM Luc Sorel-Giffo @.***>
wrote:

I pushed the commit related to the pre-commit hooks and the lint hooks now
passes 👍
I am just wondering how to launch the influxdb unit tests locally. I
believe the steps are:

install the optional dependencies for influxdb only
run the tests for influxdb only
I am not sure how to do that, now that the codebase has been moved to
the modules folder

—
Reply to this email directly, view it on GitHub
https://github.com/testcontainers/testcontainers-python/pull/413#issuecomment-1985557652,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACECGJDM2M2AFRUO65VB4TLYXGQUVAVCNFSM6AAAAABB4WT432VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBVGU2TONRVGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

poetry run pytest modules/influxdb worked indeed :pray:
I pushed a fix on the unit test. I the pipeline succeeds, I believe we're good to approve and merge the PR :smiley:

I would suggest to add the documentation about running the automated tests of a given module either in the root README.md or in modules/README.md. What do you think?

Ok lgtm then

Ok lgtm then

I think a maintainer needs to trigger the CI workflow to validate all the changes (see https://github.com/testcontainers/testcontainers-python/actions) and the PR. Moreover, I cannot approve the PR myself.

all the checks have passed successfully :smiley:

I resolved a conflict on the poetry.lock lockfile. The CI workflows need to be relaunched

all checks have passed :tada:  thank you :pray:
Is there an action that I should do for the merge?


[release-4.15] OCPBUGS-27656: bump go-git/v5 to 5.11.0 (#1175)

Upstream-repository: operator-registry
Upstream-commit: 146320b6da253cde65c18461796e19840fc04f0a
Manual cherry-pick of commit 57bbee9d2b3633b0df7d542ad44cbfa876fdc1b9 from https://github.com/openshift/operator-framework-olm/pull/650

do we want to try to do olm at the same time on this?

/retest

/jira refresh

/retest

/retest

/retest

do we want to try to do olm at the same time on this?

@kevinrizza PR now has fix for OLM too, thanks @ankitathomas for the commit.

/lgtm
/label backport-risk-assessed

/retest

retest

/retest

/test e2e-gcp-olm

/test e2e-gcp-olm

/test e2e-gcp-olm


[RtsBridge] Add new bridge for Radio Télévision Suisse

New bridge RSS for show and section available on https://www.rts.ch/play/tv.
The code is mainly based on ArteBridge.

Can you add an example value?
Like this: 'exampleValue' => 'GitHub' in the config.

Sure, done!

gj!