debugging python code with pipenv: ModuleNotFoundError: No module named X

Environment data

VS Code version: 1.24
Extension version (available under the Extensions sidebar): XXX
OS and version: MacOS Sierra 10.12.16
Python version (& distribution if applicable, e.g. Anaconda): Python 3.6
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): pipenv
Relevant/affected Python packages and their versions: Python package

Actual behavior
I try to use the python code debugger. I now get this error when I try to use the debugger, with pipenv
    import protos
ModuleNotFoundError: No module named 'protos'

My configurations are set in workspace settings:

{
    "python.pythonPath": "/Users/cathleenturner/.local/share/virtualenvs/services-3KSPTmbN/bin/python",
    "python.pydev.pythonExecutable":"/Users/cathleenturner/.local/share/virtualenvs/services-3KSPTmbN/bin/python",
    "python.venvPath": "/Users/cathleenturner/.local/share/virtualenvs"

}

Expected  behavior
After selecting my python interpreter to be the pipenv environment, the debugger should use this version. The import error shows that the environment is not in the python path.
Steps to reproduce:
mkdir code
cd clone
git clone <your python repo>
cd py-repo
pipenv install
pipenv install -d
pipenv shell
(services-3KSPTmbN) bash-3.2$ which python
/Users/cathleenturner/.local/share/virtualenvs/services-3KSPTmbN/bin/python

I put this dir in my settings file:
    "python.pythonPath": "/Users/cathleenturner/.local/share/virtualenvs/services-3KSPTmbN/bin/python"

I run debug file and get errors:
Output from consolse
cathleens-MacBook-Pro:rio cathleenturner$ cd /Users/cathleenturner/code/rio ; env "PYTHONIOENCODING=UTF-8" "PYTHONUNBUFFERED=1" /Users/cathleenturner/.local/share/virtualenvs/services-3KSPTmbN/bin/python /Users/cathleenturner/.vscode/extensions/ms-python.python-2018.5.0/pythonFiles/PythonTools/visualstudio_py_launcher.py /Users/cathleenturner/code/rio 57037 34806ad9-833a-4524-8cd6-18ca4aa74f14 RedirectOutput,RedirectOutput /Users/cathleenturner/code/rio/../my_py_file.py
Traceback (most recent call last):
  File "/Users/cathleenturner/code/rio/../my_py_file.py", line 6, in <module>
    import protos
ModuleNotFoundError: No module named 'protos'
cathleens-MacBook-Pro:rio cathleenturner$

I run the file in the terminal, and I am able to execute code.
The debugger worked fine before. Then several days ago it just stopped working. I have tried reinstall cscode and the python plugin.

Did you install the module? E.g. can you run import protos normally without the debugger with VS Code?

yes, the module is installed and set up correctly. From the terminal I can run the same script with no issues. :/

I noticed you have python.pydev.pythonExecutable in your configurations. Are you by chance using the PyDev extension? If so then can you try our extension with all other extensions turned off (i.e. just the "Python" extension and not "PyDev")?
Basically you're currently the only person on the plant reporting this issue so my gut thinks this is a configuration issue somehow.

I encountered the same problem.
My envs:

VS Code version: 1.25.1
Extension version (available under the Extensions sidebar): Python extension 2018.7.1
OS and version: MacOS 10.13.6
Python version (& distribution if applicable, e.g. Anaconda): Python 3.6
Type of virtual environment used: pipenv

My project structure:
frontend directory is a VueJS app
backend directory as a Python Flask app as a restful api

app directory is where the flask app lied

tests directory is where the tests files lied

the settings.json includes the current vietualenv:
"python.pythonPath": "${workspaceFolder}/backend/.venv/bin/python"
the launch.json includes the python debug current file:
{
      "name": "Python: Current File",
      "type": "python",
      "request": "launch",
      "stopOnEntry": false,
      "pythonPath": "${config:python.pythonPath}",
      "program": "${file}",
      "cwd": "${workspaceFolder}/backend"
 }

the  backend/tests/test_tasks.py file includes an import:
 from app.tasks import some_task
when I debug the test_tasks.py file , the vscode debug console always show:
ModuleNotFoundError, No module named 'app'
and the debug code was:
cd /Users/desmond/Repos/quatek_web_app_project/backend ; env "PYTHONIOENCODING=UTF-8" "PYTHONUNBUFFERED=1" /Users/desmond/Repos/quatek_web_app_project/backend/.venv/bin/python /Users/desmond/.vscode/extensions/ms-python.python-2018.7.1/pythonFiles/PythonTools/visualstudio_py_launcher.py /Users/desmond/Repos/quatek_web_app_project/backend 59784 34806ad9-833a-4524-8cd6-18ca4aa74f14 RedirectOutput,RedirectOutput /Users/desmond/Repos/quatek_web_app_project/backend/tests/test_tasks.py
but when I run the test_tasks.py directly, it works with no errors.
So I am wondering what's wrong with it?

@desmondddd
Please could you provide a link to your repo so we can replicate this at our end.

@DonJayamanne here it is:
https://code.aliyun.com/weiguda/quatek_web_app_project.git

Please create a file named .env in your project directory with the following value
PYTHONPATH=./backend

@desmondddd
Please could you upload your source here (zip file) or upload to GitHub.
I cannot register on Alibaba, as it is asking for credit card info, etc. it looks like a very complicated and lengthy process to register. Would be much simpler for you to upload a zip file here or just upload to GitHub (registering on GitHub is simpler).

pushed to GitHub: https://github.com/desmondddd/quatek_web_app_project

Solution 1:

Add .env file next to .gitignore file with the contents:

PYTHONPATH=./backend

Solution 2:

Add PYHTONPATH in your launch.json as follows:
{
"name": "Python: Current File",
"type": "python",
"request": "launch",
"stopOnEntry": false,
"pythonPath": "${config:python.pythonPath}",
"program": "${file}",
"cwd": "${workspaceFolder}/backend",
"
}

Solution 3:

Open the backend directory in a separate instance of VS Code.
This should work (no need of any custom PYTHONPATH settings)
This way you end up using the Pipfile.

Solution 4:
Make use of Multi-root workspaces
This way you end up using the Pipfile.
I've used it this way, and in my opinion this would be the best way.
No need of any custom PYTHONPATH settings.


feat(automate) semi-automate Docker update

I used these scripts to push the 1.1.2 update PR to this repo, and also to push the 1.1.2 update PR to official-images.

also, if it adds confidence for merging: this PR just adds the pair of scripts to the repo, it does not integrate them in any way (choosing to use them is still manual, and the scripts ask for confirmation before doing anything).


Toggling alt at the end of column selection focuses menubar

Use Alt+Mouse column selection
Release Alt without releasing the selection with mouse
Do not move the mouse at all
Press and release Alt again

fyi @alexandrudima , not for February


SidebarItem data cy

This attempts to fix two problems:

Ugly border in Firefox for SidebarItems with url-s.

Before

No data-cy for SidebarItems.

Before

After

A third problem that I could not fix was that clicking on the left or right edge of a SidebarItem with url does nothing since it's the padding of the parent <li> element.

updated test again to make CI pass. i think we should not have 4 commits here. 1 or 2 should be enough :)

Thanks, @pierrebeitz .

ERR! docker container or on a CI, you need to pass the '--ci' flag to prevent opening a

ERR! browser by default.

This is weird. Has anyone seen this before?

Thanks, @mperrotti .


discuss: no feature gating in trace_decoder

Hi Robin,
Could you help me understand your objection better here?
You mention that there are several other places which would need to be network-aware. Are they fundamentally different in a way that would break this approach?
Perhaps we'd need the code to be aware of different structs at different times (which proposed design cannot be).
As a note, I have no problems with a --network eth-mainnet | polygon-cdk style CLI argument - I think it would be much more elegant than having different binaries that are build with different features. And the relevant option simply gets plumbed through to the entrypoint.
Representing our constraints as data, using the more powerful language of e.g enums is I think much better than the blunt object of #[cfg(feature - ...)]. We're currently hacking on mutual exclusivity to a part of features that was not designed for, nor really supports it.
To be clear, the benefits to developers and users of trace decoder of this change are:

I don't need to build/run/test with different combinations of feature gates. You may recall fixing a bug where I failed to do this one before: https://github.com/0xPolygonZero/zero-bin/pull/84. This makes that kind of oversight impossible
My IDE knows all my code is live. This is impossible as it stands because our cargo features are mutually exclusive (this goes against cargo's feature guidelines btw - one of a few reasons those guidelines exist). So my feedback loop is either manual, or I have to switch between feature flags.

The main point is that following an if branch is significantly easier than following different #[cfg(feature = ...)] statements, and from my understanding, most of the code outside of evm_arith could make this change with very little product-level consequence (perf), but with a big devx and cognitive complexity upside.
I also think this is a decision with a low architectural risk today. Whether @hratoanina's thinking gives us different design requirements for example, the change is trivial to make to trace decoder if we take this approach.

You mention that there are several other places which would need to be network-aware. Are they fundamentally different in a way that would break this approach?

I haven't extensively investigated the approach to be honest, but it does feel weird to me to bloat API with arguments that end up being specific to a given network, that we need to expose through some enums for all variants. FWIW, withdrawals do not exist for non eth_mainnet, ger_data as well for non cdk_erigon. There are more things coming still. We're mixing things up here while I'd ideally advocate for unified, simpler entry points, and have all these specific logic be done internally.

I think it would be much more elegant than having different binaries that are build with different features.

The thing is, we will have different binaries anyway, because evm_arithmetization heavily relies on this, and we had a consensus on relying on feature-gating for this crate at least.

I don't need to build/run/test with different combinations of feature gates. You may recall fixing a bug where I failed to do this one before: https://github.com/0xPolygonZero/zero-bin/pull/84. This makes that kind of oversight impossible

To be fair, I think this is a non-issue. Our CI is currently pretty poor, and this issue you mentioned arised because we were not testing a feature we were supporting which is a big red flag. We shouldn't have any supported feature untested.

but with a big devx and cognitive complexity upside.

I would say this is a personal take here. I personally find your approach (taken as a whole, not just this limited example but applied to all the items I mentioned on slack), more confusing than internally handling them in a "silent" way. That being said, it's true that if your IDE greys out feature-specific code, this isn't ideal (although tweaking the default feature temporarily would alleviate the issue).

The main point is that following an if branch is significantly easier than following different #[cfg(feature = ...)] statements

@0xaatif What do you think of a middle ground, still explicitly highlighting the need for a specific network feature without clogging the API with multiple enums that end up being specific to only a small subset. Something like this:
Original

#[allow(unused_variables)]
/// Performs all the pre-txn execution rules of the targeted network.
fn do_pre_execution<StateTrieT: StateTrie + Clone>(
    block: &BlockMetadata,
    ger_data: Option<(H256, H256)>,
    storage: &mut BTreeMap<H256, StorageTrie>,
    trim_storage: &mut BTreeMap<ethereum_types::H160, BTreeSet<TrieKey>>,
    trim_state: &mut BTreeSet<TrieKey>,
    state_trie: &mut StateTrieT,
) -> anyhow::Result<()> {
    // Ethereum mainnet: EIP-4788
    #[cfg(feature = "eth_mainnet")]
    do_beacon_hook(
        block.block_timestamp,
        storage,
        trim_storage,
        block.parent_beacon_block_root,
        trim_state,
        state_trie,
    )?;

    #[cfg(feature = "cdk_erigon")]
    do_scalable_hook(
        block,
        ger_data,
        storage,
        trim_storage,
        trim_state,
        state_trie,
    )?;

    Ok(())
}

Proposed

/// Performs all the pre-txn execution rules of the targeted network.
fn do_pre_execution<StateTrieT: StateTrie + Clone>(
    block: &BlockMetadata,
    ger_data: Option<(H256, H256)>,
    storage: &mut BTreeMap<H256, StorageTrie>,
    trim_storage: &mut BTreeMap<ethereum_types::H160, BTreeSet<TrieKey>>,
    trim_state: &mut BTreeSet<TrieKey>,
    state_trie: &mut StateTrieT,
) -> anyhow::Result<()> {
    // Ethereum mainnet: EIP-4788
    if cfg!(feature = "eth_mainnet") {
        return do_beacon_hook(
            block.block_timestamp,
            storage,
            trim_storage,
            block.parent_beacon_block_root,
            trim_state,
            state_trie,
        );
    }

    if cfg!(feature = "cdk_erigon") {
        return do_scalable_hook(
            block,
            ger_data,
            storage,
            trim_storage,
            trim_state,
            state_trie,
        );
    };

    Ok(())
}

This should also solve your IDE problem.

Not really my area, FWIW no issues from the zero side.

Happy with the middle ground for now :) thanks robin!


default dummy text

Hello,
when setting the locals to en-gb, and then clicking "insert url, ordered list etc.." a text is inserted in non english.
can this be changed?

Try the new version 1.0.4 again.
Default dummy text is english when set the locals to except for zh-cn and zh-cht.

Hello,
when setting the locals to en-gb, and then clicking "insert url, ordered list etc.." a text is inserted in non english.
can this be changed?

Try the new version >=1.0.4 again.


Backport: Restricting access to fork functioanlity to users with Code access

Backport for v1.2 of #2534

Trusted LGTM

LGTM

LGTM

@lunny @appleboy CI test fail is not related to this changes

Force-merging :trollface:


doc: Update PubSub - Azure Event Hub for clarity

Added the following changes to make it more easy to utilize the docs for PubSub - Azure Event Hub

Make the example easily copy-pastable (now it's not working when copy pasted due to several lines causing Dapr to fail on start-up with error initializing pub sub pubsub.azure.eventhubs/v1: json: invalid use of ,string struct tag, trying to unmarshal "" into int32)

Signed-off-by: Xavier Geerinck<EMAIL_ADDRESS>Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Read the contribution guide
[x] Commands include options for Linux, MacOS, and Windows within codetabs
[x] New file and folder names are globally unique
[x] Page references use shortcodes instead of markdown or URL links
[x] Images use HTML style and have alternative text
[x] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description

Issue reference

@greenie-msft added them to the top

Branch was outdated and diverged too much so had to open a new PR under #2772 since DCO didn't want to pass, apologies


VM affinity not working

We're using the foreman-xen plugin (which uses this under the covers) for The Foreman to build VMs on multiple Xen clusters.  It seems the affinity is not working correctly and always builds on the master node.  No matter what slave the affinity is set to it seems to be ignored.  Have you experienced this problem?
When using the Java API years ago we had the same issue so I don't think it's specific to your code, just hoping you've come across this and had an answer.

Hey @hanej . I never had this problem so far.
Just looked at the code and could not find anything that could replicate this behaviour. Perhaps it is XEN related?
Thanks!

Thank you for checking.  I posted this question on the XenServer discussion forums too.  Hopefully we figure it out.
Very nice library though!  It works well.

Thank you @hanej . Thats awesome to have feedback. Could you post the link of the forum here plz?
Thanks!

The link is
http://discussions.citrix.com/topic/373231-vm-affinity-not-working/

@hanej this weekend playing with xenserver 6.5 I got the same thing as you mentioned
as Tobias mentioned on the forum, you need to set the affinity for the vm being created, so this will work:
template.class
# => Fog::Compute::XenServer::Models::Server
template.affinity = conn.hosts[0]
template.save
# true

hope this may help you


Development fit producing incorrect result with quarterly data

The ldfs fit for quarterly data are incorrect.
Simple short example below. With n_period = 1 the ldfs should be the same as the last diagonal/row of link_ratios, regardless of average='simple' or 'volume'.  This appears to only work on the first quarter. Note that simple and volume (and regression) produce different results even with n_periods=1.
data = cl.load_dataset('quarterly')
data=data['paid']
print(data.link_ratio)
print(cl.Development(n_periods=1, average='simple').fit(data).ldf_)
print(cl.Development(n_periods=1, average='volume').fit(data).ldf_)

Example is easier to see if you run in a notebook without using print().

Issue in determining weights for asymmetric triangles, i.e. origin_grain != development_grain
Development._assign_n_periods_weight_int needs to be written to acount for this.
    def _assign_n_periods_weight_int(self, X, n_periods):
        ''' Zeros out weights depending on number of periods desired
            Only works for type(n_periods) == int
        '''
        if n_periods < 1 or n_periods >= X.shape[-2] - 1:
            return X.values*0+1
        else:
            # This section produces incorrect results for asymmetric triangles
            flip_nan = np.nan_to_num(X.values*0+1)
            k, v, o, d = flip_nan.shape
            w = np.concatenate((1-flip_nan[..., -(o-n_periods-1):, :],
                                np.ones((k, v, n_periods+1, d))), 2)*flip_nan
            return w*X.expand_dims(X.nan_triangle())

6551a3f only appears to fix when n_periods = 1.
The above example produces the same results for n_periods = 1-4 for the first couple ldfs.


:book: Instructions for deploying KS v0.21 using K3D

Summary
KS install instructions tested on ubuntu 22.04 using K3D

/ok-to-test

@eaepstein You need to sign of your commits (see https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits).
You can git amend  old commit to fix it


python3Packages.chia-rs: fix build

Description of changes
broken after #218614

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

1 package failed to build:

chia-dev-tools

3 packages built:

chia
python310Packages.chia-rs
python311Packages.chia-rs

Oh wait, chia-rs has breaking changes that break Chia and cdv. We should revert it until Chia 1.7.1 is released.

#221544


Check for plot_wireframe_radec wraparound issue at RA=0

Could have similar issue to #323 if observation has some data with e.g. RA=359, and some with RA=1.

[x] Add appropriate warnings to documentation (as in #325).
[x] Add (optional?) fix to plot_wireframe_radec to change meridian between 180 and 0. Can use ra, dec functional transform being added in #324 to implement this, then probably add a flag to manually enable this.
[x] Fix lines all the way across plots when e.g. trying to plot a limb that has parts on both sides.

body = planetmapper.Body('saturn', '2020-01-01', observer='mimas')

has this issue

Previous behaviour:

New default behaviour:

New bahaviour with body.plot_wireframe_radec(use_shifted_meridian=True):


As a user I want to easily upload files into my pod

Added to readme

Opening this as part of moving all user stories into issues again.


Cannot modify data of list entry. Was this meant to be this way?

There are functions to get the value of a ListEntry, but there are no functions to write to the data of a ListEntry.
The underlying struct is defined in the list.c file so it is not possible to access it directly.
I am willing to implement the feature if needed.

Good point. I think there should probably be a way to set the data like you say (and for slist too).
I think it was an oversight on my part when I first wrote it; I usually think of linked lists as an immutable data structure, or at least one that should be used in an immutable way. But this is C and not Lisp after all.
There should probably be similar functions for setting the 'next' pointer (unless necessary, not for 'prev' though).

Im not sure if you shoukd be able to modify the next pointer. At least for the double linked list, since it would not be clear what to do with the prev pointer of the new next entry. If we do not set the prev option it would render a corrupt double linked list since it the path backwards ends up in other entries than in the path forward. If we set prev we also imply that we get rid of a piece of list which might not be intended by the user.
For a single linked list we would have the problem that you can easily remove a sublist from the list.
The set data mehtods will be committed.


Feature Request: Dynamic Defaults

Hey, I was wondering if this would be something you'd be interested to have integrated:
import { schema, types } from 'papr';

const userSchema = schema({
  active: types.boolean(),
  age: types.number(),
  firstName: types.string({ required: true }),
  lastName: types.string({ required: true }),
  abTestGroup: types.enum(["A", "B"]),
}, {
  defaults: {
    abTestGroup: () => Math.random > 0.5 ? "A" : "B"
  }
});

Depending on your answer I would try to implement it.

Can you please elaborate with a different real world example for this feature?
I worry a bit about the performance impact this will have. Feel free to try it on a fork and report back some benchmarks.

@avaly An example would be the default Date.now function. We use it in mongoose when you want a field to have the current date at the moment of creation. As I understand, this is not possible in Papr.

@BahaaZidan papr supports two default timestamp attributes: createdAt and updatedAt. These attributes are pre-populated with the timestamp of the insert/update operation of the document. You can enable these attributes with the timestamp option in the schema() definition.
Unless there's a real-world use case of having dynamic defaults, I don't think this feature will get added in papr.


Fix Windows login issue

Windows seems to have a very low limit for the length of command arguments when spawning, so our recent currency additions made it go over that limit. A quick fix for now is to exclude the electrumServers property as it's not used by marketmaker. A proper longer-term fix would be for marketmaker to let us dynamically enable currencies with this info instead of having to supply it on startup.
Fixes #519
Fixes #525

Great catch @sindresorhus, apologies, I should have tested the beta release on Windows before releasing.


 I can remove this commit if you prefer not to have these files formatted by prettier

Thanks @tatchi! I'd been meaning to fix it 😁


openshot-qt does not start: ModuleNotFoundError: No module named 'openshot'

Describe the bug
A clear and concise description of what the bug is.
Steps To Reproduce
Steps to reproduce the behavior:

nix-shell -p openshot-qt
openshot-qt
ModuleNotFoundError: No module named 'openshot'

Expected behavior
openshot should start
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
$ openshot-qt
Loaded modules from: /nix/store/gpfhhd1d5sbxgx7j6162f6k920rjbff3-openshot-qt-3.2.1/lib/python3.11/site-packages/openshot_qt
INFO sentry: No sentry_sdk module detected (error reporting is disabled)
INFO app: ------------------------------------------------
INFO app:             Sun Sep 29 17:51:16 2024
INFO app:               Starting new session
ERROR app: OpenShotApp::Import Error
Traceback (most recent call last):
  File "/nix/store/gpfhhd1d5sbxgx7j6162f6k920rjbff3-openshot-qt-3.2.1/lib/python3.11/site-packages/openshot_qt/classes/app.py", line 100, in __init__
    from classes import settings, project_data, updates, sentry
  File "/nix/store/gpfhhd1d5sbxgx7j6162f6k920rjbff3-openshot-qt-3.2.1/lib/python3.11/site-packages/openshot_qt/classes/project_data.py", line 48, in <module>
    import openshot
ModuleNotFoundError: No module named 'openshot'
Traceback (most recent call last):
  File "/nix/store/gpfhhd1d5sbxgx7j6162f6k920rjbff3-openshot-qt-3.2.1/lib/python3.11/site-packages/openshot_qt/launch.py", line 202, in main
    app = OpenShotApp(argv)
          ^^^^^^^^^^^^^^^^^
  File "/nix/store/gpfhhd1d5sbxgx7j6162f6k920rjbff3-openshot-qt-3.2.1/lib/python3.11/site-packages/openshot_qt/classes/app.py", line 100, in __init__
    from classes import settings, project_data, updates, sentry
  File "/nix/store/gpfhhd1d5sbxgx7j6162f6k920rjbff3-openshot-qt-3.2.1/lib/python3.11/site-packages/openshot_qt/classes/project_data.py", line 48, in <module>
    import openshot
ModuleNotFoundError: No module named 'openshot'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/nix/store/gpfhhd1d5sbxgx7j6162f6k920rjbff3-openshot-qt-3.2.1/bin/.openshot-qt-wrapped", line 9, in <module>
    sys.exit(main())
             ^^^^^^
  File "/nix/store/gpfhhd1d5sbxgx7j6162f6k920rjbff3-openshot-qt-3.2.1/lib/python3.11/site-packages/openshot_qt/launch.py", line 204, in main
    app.show_errors()
    ^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'show_errors'
INFO app:             OpenShot's session ended
INFO app:             Sun Sep 29 17:51:16 2024
INFO app: ================================================

Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.52, NixOS, 24.11 (Vicuna), 24.11.20240926.1925c60`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.7`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/store/fpivx4sjcp2vk4rp9nhliln5cwcp3kc6-source`

Add a :+1: reaction to issues you find important.

Did you find an older version that works?

I haven't tried. I'm fairly new to nixos, what would be a quick way to bisect versions?

I used this script before to do a date based bisect:
startdate=$1
enddate=$2

# date format is 'YYYY-MM-DD'
c1=$(git log --before $startdate --format='format:%H' | head -n1)
c2=$(git log --after $enddate --format='format:%H' | head -n1)

echo $c1
echo $c2

git bisect start
git bisect bad $c2
git bisect good $c1
git bisect run nix run .#openshot-qt

for the bad commit that should work fine but not sure whether you need a slightly different command line option for the good so that it doesn't hang when it works. I don't have time, compute and disk space to do it :\
Also, I don't really understand where the openshot library comes from. Is it from libopenshot or is it within the package?
Maybe @AndersonTorres knows better since he is tagged as maintainer.

@raas . I found a good version in latest stable:
nix run github:nixos/nixpkgs/nixos-24.05#openshot-qt << this one works for me. Does it work for you?

@MMesch the version from stable does indeed work, thank you!

I started bisecting it but ran into a staging merge.
Known good commit: fa4b81d8e9ad1f7b35c9f2cdb41318e3487eaf9a
Known bad commit: 149a15141ad00c1b525ac02dacdb526b62c53e50
Diff: https://github.com/nixos/nixpkgs/compare/fa4b81d8e9ad1f7b35c9f2cdb41318e3487eaf9a...149a15141ad00c1b525ac02dacdb526b62c53e50
cc @AndersonTorres as the package maintainer


[Rebalancing] Edit Controller, Edit Depositories Weights Bps

PR Combo:

uxd-client: https://github.com/UXDProtocol/uxd-client/pull/41
uxd-program: https://github.com/UXDProtocol/uxd-program/pull/248

Program branch was merged:
https://github.com/UXDProtocol/uxd-program/pull/248


Fix executor crashed

Fix executor crashed
Description
Fix panic usage in is_proof_ok() executor module.
Applicable Issues
#302

Cooperate on the fix/executor-tolerant-staled.


Issue pulling Flogo container from Docker

Hi I just installed the Docker Toolbox for Windows 7 successfully. But when I try to run following command I get an error. Any ideas what I could be doing wrong?
   docker run -it -p 3303:3303 flogo/flogo-docker:latest eula-accept

I'm getting this error:
$ docker run -it -p 3303:3303 flogo/flogo-docker eula-accept
Unable to find image 'flogo/flogo-docker:latest' locally
C:\Program Files\Docker Toolbox\docker.exe: Error response from daemon: Get http
s://registry-1.docker.io/v2/: dial tcp: lookup registry-1.docker.io on <IP_ADDRESS>:
53: no such host.
See 'C:\Program Files\Docker Toolbox\docker.exe run --help'.

The command is correct. This looks like a network problem, docker is unable to connect to its registry.
You might want to check your DNS, firewall or proxy/vpn configuration.

Hi @rickdantz are you still having issues?

Hi I was able to resolve this issue. Now I have other issues but I'll address them in another issue. Thanks for checking.


Inquiry: average number of ships

I was wondering if you could assist with confirming or disproving a data point in my pace of play findings.  Can you calculate the average number of ships being taken?  I am wondering if it is around 4?  If it is not possible it is ok, just confirming a curiosity.

@patrickpatrickxwing It's not something I can calculate off hand with MetaWing, but you can hit listfortress.com/api/v1/tournaments API and listfortress.com/api/v1/tournaments/{id} API to pull the information and do a calculation like that of the data.


This policy does not appear when using GDAP

This Voicemail policy tab does not appear in Teams Admin Center (TAC)  when using GDAP. Roles combination tried without success: Teams administrator, Skype for Business administrator, User administrator, Billing administrator
It does appear as when using in tenant user with same admin roles.
Please advise GDAP roles required to work or add that it does not currently work via GDAP.

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

ID: 85634d87-7729-6449-9663-556097e75504
Version Independent ID: b75446bd-07af-a7f2-d346-fe7634634548
Content: Manage Voicemail Policies - Microsoft Teams
Content Source: Teams/manage-voicemail-policies.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

Hi @SvenBell, thank you for your feedback.
You can find the information in a specific GDAP article:
https://learn.microsoft.com/partner-center/gdap-supported-workloads#teams-admin-center

Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


add green h2 to exporter, include methanol in price for liquids from h2

Before asking for a review for this PR make sure to complete the following checklist:

[x] Workflow with target rule ariadne_all completes without errors
[x] The logic of export_ariadne_variables has been adapted to the changes
[ ] One or several figures that validate the changes in the PR have been posted as a comment
[ ] A brief description of the changes has been added to Changelog.md
[x] The latest main has been merged into the PR
[x] The config has a new prefix of the format YYYYMMDDdescriptive_title

Green H2 prices (€/MWh):
2020
Green H2 : nan
lmps: 30.46898349450408
capex+opex of all H2 gens: 31.515582282103953
2025
Green H2 :<PHONE_NUMBER>084224
lmps: 76.57157149991603
capex+opex of all H2 gens: 79.01682498877406
2030
Green H2 :<PHONE_NUMBER>4725655
lmps: 71.24381648923318
capex+opex of all H2 gens: 86.62507239689464
2035
Green H2 : 88.36131525126635
lmps: 89.55960318024617
capex+opex of all H2 gens: 92.59703231371033
2040
Green H2 : 86.26398721793382
lmps: 82.33084793130935
capex+opex of all H2 gens: 90.433637862454
2045
Green H2 : 89.62484826219891
lmps: 75.0459948243917
capex+opex of all H2 gens: 91.36552760251482
new values for "Price|Secondary Energy|Liquids|Hydrogen"
2020
FT only: 57.45082283888327
with meoh : 52.611853423277
2025
FT only:<PHONE_NUMBER>627392
with meoh :<PHONE_NUMBER>0900955
2030
FT only: 97.26908985931344
with meoh : 75.80026942266635
2035
FT only: 45.21916423885248
with meoh : 45.27604799167047
2040
FT only: 46.07182014764879
with meoh : 46.06785618373825
2045
FT only:<PHONE_NUMBER>5859833
with meoh :<PHONE_NUMBER>6745718


Add a feature in Tabular Explainer to deal with missing values?

Hi Lime team,
When using a tree-based model (Xgbost or LightGBM), missing values can be automatically binned and processed. However, the Tabular Explainer doesn't allow the existence of missing values and will throw an exception: Input contains NaN, infinity or a value too large for dtype('float64').
Filling any missing values with  fillna() in such case will cause incorrect explanations. Is there any way to deal with that?
Thank you!

Uh, good question. I don't know what to do with missing values, maybe we could support them and sample them just like we do any other values if the feature is categorical. For numerical features though, I don't know. We could maybe assume the frequency of missing values and sample MISSING at that rate, and do what we're doing otherwise.
Anyway, this is not something I can see myself working on soon. If you are interested, we'll take a pull request : )


Docs - Add a small note for using release container version

Description
Minor doc change to highlight sb CLI version is independent of the sb container version.

/azp run


Implement retry logic for intermittent errors in communication with Communication Services.

Description
We already know of multiple potential faults that can occur when sending an email

connection to server is down
429 (?)

We need to define a strategy for how to handle the various errors.
Do we set a status code to retry after a given time frame?
Do we stop attempting as the error indicates it will never work?
etc.
useful doc: https://learn.microsoft.com/en-us/azure/communication-services/concepts/email/sender-reputation-managed-suppression-list
In scope

Identify all known error cases and define an action plan for each case.
Create user stories to handle cases if required or enrich already existing issues

Constraints

Analysis

We have agreed on a data flow in the error situations.
An additional topic for system errors or "system updates" will be created.
Related microservices can use this topic to alert to instabilities or downtime in their system
causing us to for instance pause the request for sending emails.
Tasks

[x] Implement changes in Notifications email
[x] Implement repository logic for setting flag to hinder retrieval of email notifications
[x] Implement repository logic for halting retrieval of email notifications if flag is set
[x] Implement changes in Notifications consumer to process system messages
[x] Add new email status Failed_SupressedRecipient
[x] Implement logic for picking up shipments that are sending in a longer period of time
[x] QA
[x] Test

Test cases

[x] Regression test happy path, email is sent and status logged successfully
[x] Deploy temp code for email Notification publishing a timeout service update to kafka and validate that app flag in db is set and that sending of email is paused for the duration of time

Regression tests suceessful. New feature tested ok in at22


Allow assigning shipping zones to channels

Add model relation between Channel and ShippingZon. Assign to existing channels, shipping zones, based on existing ShippingMethodChannelListing.
Update ChannelCreate mutation - allow defining shipping zones
Update ChannelUpdate mutation - allow adding and removing shipping zones
Extend Channel type with ShippingZones

Linked task SALEOR-2549

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-2549 Allow assigning shipping zones to channels


Updates to last_login should not update modified_by to the system user

from activity stream
{
    "count": 11,
    "next": null,
    "previous": null,
    "results": [
        {
            "id": 11,
            "url": "",
            "related": {
                "created_by": "/api/v1/users/2/"
            },
            "summary_fields": {
                "created_by": {
                    "id": 2,
                    "username": "admin",
                    "first_name": "",
                    "last_name": ""
                },
                "content_object": {
                    "id": 2,
                    "username": "admin",
                    "first_name": "",
                    "last_name": ""
                }
            },
            "created": "2024-04-08T14:48:22.190325Z",
            "created_by": 2,
            "operation": "update",
            "changes": {
                "added_fields": {},
                "changed_fields": {
                    "last_login": [
                        null,
                        "2024-04-08 14:48:22.178879+00:00"
                    ],
                    "modified_by": [
                        "_system",
                        "admin"
                    ],
                    "modified_by_id": [
                        1,
                        2
                    ]
                },
                "removed_fields": {}
            },
            "content_type": 17,
            "object_id": "2",
            "related_content_type": null,
            "related_object_id": null,
            "content_type_model": "user",
            "related_content_type_model": null
        },

                    "modified_by": [
                        "_system",
                        "admin"
                    ],

It's updating it from the system user to the logged in user, which is technically accurate, but arguably this change shouldn't store an activity stream update

I'm arguing the modified_by point here, not the activity stream point. This is why I put it in the app:lib label.
User story:

joe is not logged in
admin user changes the email address of user joe
user joe logs in and views their own user details

The user details of joe should alert them to the fact that something changed via an edit by admin. With this, it won't.


⚠ T:Riza Api has degraded performance

In 59a6009, T:Riza Api (https://infra.triza.dev/api) experienced degraded performance:

HTTP code: 200
Response time: 3639 ms

Resolved: T:Riza Api performance has improved in 41a5c60.


Please add a license file to this repo

Thank you for making this project available! It's very helpful for quickly and interactively debugging Graphviz files.
I have a minor request about making the project license more explicit in the GitHub repo.
The README.md says:

Licence
BSD

but since there's no explicit LICENSE file in this repo, GitHub does not automatically mark it as BSD.
Also, there are multiple types of the BSD license, e.g.,

2-clause BSD
3-clause BSD

Could you please select the specific license and place a full copy of the license text into a LICENSE file in the repo? Thank you!

Hi @mbrukman,
Thank you for point this out. I have supplied the license file. It's BSD-3.

For future reference, this was fixed in PR #6. Looks great, thank you!


[Bug]: issue: after running Appsmith client locally(dev.appsmith.com) it's showing that "Your connection is not private", how to resolve it please help?

Is there an existing issue for this?

[X] I have searched the existing issues

Description
after running Appsmith client locally(dev.appsmith.com) it's showing that "Your connection is not private", how to resolve it please help?

Steps To Reproduce
I followed each step in WSL:
https://github.com/appsmithorg/appsmith/blob/release/contributions/ClientSetup.md
Public Sample App
No response
Environment
Production
Severity
High (Blocker to building or releasing)
Issue video log
No response
Version
Self hosted v1.41.0

hey @deepakk0109  just type "thisisunsafe" while you are on this page it will take you to the dev.appsmith.com
I do not have the correct explaination but what i noticed is, it is because of the chrome and the system security issue which led to this issue of not opening a local https server

After typing "thisisunsafe"
I got "502 Bad Gateway", what could be the possible reason?
I followed these steps for running the application: (https://github.com/appsmithorg/appsmith/blob/release/contributions/ClientSetup.md)

@ALOK9442

Hey @deepakk0109 make sure you run the command  "sudo ./start-https.sh https://release.app.appsmith.com" before starting the server each as it connects it to the backend services and then start the frontend

Hey @ALOK9442 it works, and I am using local server(http://localhost:8080/) and getting following error on (dev.appsmith.com)  while going for application building(editing page), how to resolve it?
Error: RTS server error while processing request: Error fetching latest DSL version

hey @deepakk0109 I never did the setup of server side locally, so could not help with it but this issue never arises for me as I was using the backend service from their web only


Convert MCA (Anvil) to TNT fails on backup region files

This isn't a major issue, but I found a issue while converting an Anvil world it threw an error
Unexpected exception thrown: java.lang.NumberFormatException: For input string: "mca"
I found that it was trying to read r.0.0.mca.2710657221739100675.backup
This line here reads the last two items in the array... which from the backup file is ["mca", "2710657221739100675"]
My suggestion to fix it would be
Replace val args = it.nameWithoutExtension.split(".").takeLast(2) with

Regex - Something along the lines of val args = Regex("(-?\\d+).(-?\\d+)").find(it.nameWithoutExtension)?.groupValues?.drop(0)
val args = it.nameWithoutExtension.split(".").slice(1..2)

PR your suggestion maybe

PR your suggestion maybe

I was going to but as there are many different ways to resolve this issue I couldn’t figure out which was the best, so I thought the repo owner may have more knowledge and is able to choose the correct result.

Turns out this was fixed in 723cb52!


FEATURE: allow users to send messages in #practice-room-chat.

Adds a permissions override allowing SEND_MESSAGES in PR chat for each user that
joins a registered PR. Removes that override once the user is no longer in a PR
(does not deny SEND_MESSAGES, so that the default behaviour is customizable by staff).

deleted the wrong branch -_-


⚠ Bofrost.be has degraded performance

In 3cffa87, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1508 ms

Resolved: Bofrost.be performance has improved in 7f97313.


RealmSwift with Xcode Version 7.1 beta 3 (7B85)

I have tried using Realm build 0.96.1 with XCode 7.1 Beta 3 and get an error when trying to import RealmSwift.
The error is: "MasterViewController.swift:12:8: Module file was created by an older version of the compiler; rebuild 'RealmSwift' and try again: RealmSwift.framework/Modules/RealmSwift.swiftmodule/arm64.swiftmodule"
I have tried downloading latest source and using "build.sh build" but I still get the same message when including the build frameworks from build\ios\swift-2.0*.framework.
Any ideas when the latest swift frameworks will be released?

The latest release (0.96.1) compiles with Xcode 7.1 Beta 3, but you'll have to build it with that version of Xcode for it to be compatible with Swift 2.1.
Looks like you've been using versions of Realm Swift compiled with Xcode 7.0.x. You can specify which Swift version to build for by specifying the REALM_SWIFT_VERSION=2.1 environment variable when running build.sh build, or you could use CocoaPods, which would enforce that Realm Swift is built with the same version of Xcode as your own project.

I'm getting the same issue with the new release of Xcode 7.1. I build through Carthage though. @jpsim any suggestions?

After upgrading to the released version of XCode 7.1 and building per jpsim's instructions, all is working again. Thanks!

@jpsim - Can you provide some more details of how to reproduce your Swift 2.1 issue resolution?

Never mind. Got it. Similar to Swift 1.2 installation, but with 2.1 as the variable: https://realm.io/docs/swift/latest/#installation-swift-12

@codywinton i updated to xcode 7.1(swift 2.1) this morning and now i am stuck with this issue on my current project (my first realm project) and most of the stuff said above is beyond my current knowledge, (i know how to add the realm frameworks to xcode and really not much else outside of swift/xcode/objc). can you you advise me or point me to a version thats already built for swift 2.1?

@hardinm78 -

Clone the repo or download as a Zip file and open: https://github.com/realm/realm-cocoa
Open up a terminal window (mac) or some comparable alternative.
On mac, you'd change to the directory of the repo via. cd ~/Path/To/The/Repo command in the terminal window. cd stands for change directory.

Got me most of the way there. I was using the strip bash script. That threw up an error so I removed it. Now I am getting dyld library not loaded. Reason: image not found.

Nevermind: remembered how to solve that. Added back to Embedded Binaries. Thanks!

Hi, I get following error when try building RealmSwift with Xcode 7.1:
** BUILD FAILED **

The following build commands failed:
    CompileC build/DerivedData/Realm/Build/Intermediates/Realm.build/Release-
    iphoneos/iOS.build/Objects-normal/armv7/RLMConstants.o Realm/RLMConstants.m normal armv7
    objective-c com.apple.compilers.llvm.clang.1_0.compiler
    (1 failure)

I did set REALM_SWIFT_VERSION=2.1, also tried "sh build.sh build", "sh build.sh ios-swift","sh build.sh ios-static" and "sh build.sh ios-dynamic", but everything failed.

We’re also working to make a new release of Realm compiled with Xcode 7.1 which should make these errors go away. Sorry for the delay!

@moogle19 - Did you try Cleaning your project and deleting the DerivedData?

@codywinton Yes, even tried to do clone the repo again and also used different versions and tested the tg-xcode-7.1 branch. No success. :(

@moogle19 - Then I think you've gone beyond my diagnostic ideas.

@timanglade Any ETA on that release? For those of us using carthage we're pretty much blocked until it's released :cry:

@jakecraige You could always install Xcode7 thill new release.

@Dudi00 Good idea. Thanks!

@jakecraige yea, no problem, glad that I could help :)

@jakecraige Working on it at the moment! @bdash and @tgoyne are trying to get something out the door asap but the CI upgrade was more complicated than expected due to vacations and other complications. Keep an eye on this issue, our blog or Twitter!

thanks @codywinton

Is there any chance that Realm is not compatible with 7.1? Thanks guys!

Realm 0.96.2 was released today and is compatible with Xcode 7.1 and Swift 2.1.

Building with Realm 0.96.2 Carthage fixes the issue, @bdash @jakecraige

Had the same issue and now it's working again! Thanks guys

Awesome! Thanks guys!


ENH: implement partial_fit for SciKeras models

What does this PR implement?
It implements and tests partial_fit for scikeras.BaseWrapper models. It also exposes a warm_start parameter in BaseWrapper.fit.
Reference issues/PRs
This will be useful for https://github.com/dask/dask-ml/issues/696.

This is super cool! Definitely something I was thinking about. I will take a look later tonight.

Do you think it would be necessary to support serialization + partial fits (i..e partial fit -> serialize -> partial fit, etc.)?

I'd like to see that because it'd be really useful in Dask-ML's model selection. I'll implement a test for that.

@stsievert mind giving me write access to your fork so that I can try and work out the test failures? Thanks!

Codecov Report

Merging #17 into master will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
+ Coverage   97.41%   97.50%   +0.08%
==========================================
  Files           1        2       +1
  Lines         310      320      +10
==========================================
+ Hits          302      312      +10
  Misses          8        8

Impacted Files
Coverage Δ

scikeras/wrappers.py
97.48% <100.00%> (+0.06%)
:arrow_up:

scikeras/__init__.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

Everything looks good. Ready to merge @stsievert ?

Looks good to me. Thanks @adriangb!

Give me a sec – looks like I committed an extra file.

I'd appreciate if you could "squash and merge". That'll keep the git history clean – I accidentally committed test-output.xml (not to mention my commit messages).

Got it. Thanks again for this PR! Please let me know if there's anything I can do to facilitate contributions.

Please let me know if there's anything I can do to facilitate contributions.

To be honest, I'm not sure how you could have better facilitated this contribution:

Google wasn't involved, so I didn't have to sign anything.
This is a small third party package, so development can happen pretty smoothly (no lengthy reviews, not many breaking tests, etc).
You even debugged for me!

ping me if you need a release from this package for dask/dask-ml#696.

Let's wait for https://github.com/dask/dask-ml/pull/699, which depends on some other issues.

Could you make a new release and push to PyPI? I'm testing on CI now, which runs pip install scikeras.

Do you need that asap, or within the next couple of days? Could you install
from source until then? Just so I can try to collect 1-2 more improvements
before releasing.
On Wed, Jul 15, 2020, 6:06 PM Scott Sievert<EMAIL_ADDRESS>wrote:

Could you make a new release and push to PyPI? I'm testing on CI now,
which runs pip install scikeras.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/adriangb/scikeras/pull/17#issuecomment-659059714, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AANMPP2GST3Y36EVO72Q3QDR3YZA7ANCNFSM4OYTLZOQ
.

I don't need it immediately – I'm installing from the master branch now in Dask-ML's CI pipeline. It would be nice to have in a couple days; I think https://github.com/dask/dask-ml/pull/699 will be ready to merge then.

Got it. I'll try to match up the timelines.
On Wed, Jul 15, 2020, 6:33 PM Scott Sievert<EMAIL_ADDRESS>wrote:

I don't need it immediately – I'm installing from the master branch now in
Dask-ML's CI pipeline. It would be nice to have in a couple days; I think
dask/dask-ml#699 https://github.com/dask/dask-ml/pull/699 will be ready
to merge then.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/adriangb/scikeras/pull/17#issuecomment-659068363, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AANMPP46J7WFPD27FGZTXC3R3Y4FDANCNFSM4OYTLZOQ
.


php: configure error - your system does not support systemd

Describe the bug
Error build php packages.
To Reproduce
Steps to reproduce the behavior:
Run nix-build -I nixpkgs=channel:nixos-unstable-small -A php72.unwrapped
Notify maintainers
@NixOS/php
Build error occurs after this PR #89388
Build log:
checking for select... yes
checking for apparmor... no
checking for sd_notify in -lsystemd-daemon... no
checking systemd/sd-daemon.h usability... yes
checking systemd/sd-daemon.h presence... yes
checking for systemd/sd-daemon.h... yes
configure: error: Your system does not support systemd.
builder for '/nix/store/k3h2hj6vhn93r2v9slbj3nd3fyzsv7ny-php-7.2.29.drv' failed with exit code 1
error: build of '/nix/store/k3h2hj6vhn93r2v9slbj3nd3fyzsv7ny-php-7.2.29.drv' failed

After this PR #87705 error build php.
cc @Ericson2314

https://github.com/NixOS/nixpkgs/pull/90249 fixes it


Schema Compare + Apply results in unexpected folder structure for non-schema-contained objects

For Schema/ObjectType organization:
Objects that don't belong to a schema are created at <objectName>/<objectType>/objectName.sql, when they should be created at just <objectType>/objectName.sql

Fixed in DacFx


[improve][client] Add partitioned-stats api to avoid partition abstraction

Motivation
As discussed in https://github.com/apache/pulsar/pull/18212, that PR introduced support to get partitioned-topic stats with a partitioned-topic stats interface. However, the Producer and Consumer still return ProducerStats and ConsumerStats. So, it's better to introduce API to get partitioned-stats in both ProducerStats and ConsumerStats so, user can call the API without casting to MultiTopicConsumerStats or PartitionedTopicProducerStats.
Modifications

Add default getPartitionStats() API to producer/cosnumer stats which will not impact existing implementation.
If producer/consumer is partitioned then API will return collection of partitioned-topic stats.

Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

Codecov Report

Merging #18308 (60077f5) into master (0866c3a) will decrease coverage by 13.05%.
The diff coverage is 16.66%.

@@              Coverage Diff              @@
##             master   #18308       +/-   ##
=============================================
- Coverage     38.97%   25.91%   -13.06%
+ Complexity     8311     3331     -4980
=============================================
  Files           683      374      -309
  Lines         67325    41433    -25892
  Branches       7217     4234     -2983
=============================================
- Hits          26239    10739    -15500
+ Misses        38079    28964     -9115
+ Partials       3007     1730     -1277

Flag
Coverage Δ

unittests
25.91% <16.66%> (-13.06%)
:arrow_down:

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

Impacted Files
Coverage Δ

.../main/java/org/apache/pulsar/PulsarStandalone.java
0.00% <ø> (ø)

.../pulsar/broker/service/AbstractBaseDispatcher.java
34.14% <0.00%> (-18.01%)
:arrow_down:

...rg/apache/pulsar/broker/service/BrokerService.java
44.46% <0.00%> (-13.46%)
:arrow_down:

...pulsar/broker/service/PulsarCommandSenderImpl.java
46.59% <ø> (-27.77%)
:arrow_down:

...apache/pulsar/broker/service/TopicListService.java
10.65% <0.00%> (-44.27%)
:arrow_down:

...sar/broker/service/persistent/PersistentTopic.java
41.66% <0.00%> (-15.32%)
:arrow_down:

...ransaction/buffer/impl/InMemTransactionBuffer.java
0.00% <ø> (ø)

...nsaction/buffer/impl/TransactionBufferDisable.java
52.63% <ø> (ø)

.../metadata/v2/TransactionBufferSnapshotIndexes.java
0.00% <0.00%> (ø)

...a/v2/TransactionBufferSnapshotIndexesMetadata.java
0.00% <0.00%> (ø)

... and 475 more


chore: release 0.31.0

:robot: I have created a release *beep* *boop*
0.31.0 (2019-08-15)
Features

debug: capture stack trace in errors rather than message (#718) (bfed3f1)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/nodejs-pubsub/releases/tag/v0.31.0

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


Lineage API depth 0 throws Not Found error

Describe the Issue
If you specify  bw=0, fw=0 it returns Table Not found.
The table actually exists but it does not have any edges.
To Reproduce
http://localhost:8080/api/v1/lineage/table/LOREM?bw=0&fw=0

Expected behavior
Return only one table node

@sambekar15 I would like to this up for the OSD.

@sambekar15 I would like to this up for the OSD.

Sure join this slack channel - #opentrack-superglue


don't use proxyfix if not using wsgi

The ProxyFix does not work with flask's direct HTTP server, as it assumes data in the wsgi.*-environment keys. This breaks setups not using uwsgi, but plain ProxyPass, such as the sipa_debug setup in the development.yml docker-compose config.

What is Flask's server assuming exactly?

To be more specific, the stacktrace is:
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1997, in __call__
    return self.wsgi_app(environ, start_response)
  File "/usr/local/lib/python3.6/site-packages/werkzeug/contrib/fixers.py", line 152, in __call__
    return self.app(environ, start_response)
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1985, in wsgi_app
    response = self.handle_exception(e)
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1540, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
    raise value
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1512, in handle_user_exception
    return self.handle_http_exception(e)
  File "/usr/local/lib/python3.6/site-packages/flask/app.py", line 1471, in handle_http_exception
    return handler(e)
  File "/home/sipa/sipa/sipa/blueprints/generic.py", line 63, in error_handler_redirection
    message=message
  File "/usr/local/lib/python3.6/site-packages/flask/templating.py", line 134, in render_template
    context, ctx.app)
  File "/usr/local/lib/python3.6/site-packages/flask/templating.py", line 116, in _render
    rv = template.render(context)
  File "/usr/local/lib/python3.6/site-packages/jinja2/asyncsupport.py", line 76, in render
    return original_render(self, *args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/jinja2/environment.py", line 1008, in render
    return self.environment.handle_exception(exc_info, True)
  File "/usr/local/lib/python3.6/site-packages/jinja2/environment.py", line 780, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python3.6/site-packages/jinja2/_compat.py", line 37, in reraise
    raise value.with_traceback(tb)
  File "/home/sipa/sipa/sipa/templates/error.html", line 2, in top-level template code
    {% set page_title = _("Fehler {} - {}").format(errorcode, message) %}
  File "/home/sipa/sipa/sipa/templates/base.html", line 160, in top-level template code
    <a href="{{ url_self(locale=locale.language) }}">
  File "/home/sipa/sipa/sipa/utils/__init__.py", line 100, in url_self
    raise RuntimeError("No endpoint available.")
RuntimeError: No endpoint available.

Where the nginx config is as given in example/nginx.conf
http {
    server {
        listen 80;
        server_name localhost;

        proxy_set_header Host $http_host;
        proxy_set_header X-Forwarded-Proto $scheme;
        proxy_set_header X-Forwarded-For $remote_addr;
        proxy_redirect off;
        # […]
        location = /sipa_debug { rewrite ^ /sipa_debug/; }
        location /sipa_debug/ {
            proxy_pass http://sipa_debug:5000;
            proxy_set_header Host $host;
            proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
            proxy_set_header X-Scheme $scheme;
            proxy_set_header X-Script-Name /sipa_debug;
        }

    }
}

The fix of #355 in 9e3ec85c15b8ca109fe3056fba7065d8c04ac2e9 addresses the exception you reported.
Could you check, if changes to ProxyFix are still required?

Currently, the url root is not recognized correctly, leaving requests not mapping correctly. I'm really confused about which part of the stack is actually responsible for parsing SCRIPT_ROOT (or SCRIPT_NAME? Is this even standardized?)

Ah, found it: https://github.com/pallets/werkzeug/blob/1c70c3e1855d32a285843c0851f52e20160f6656/werkzeug/wrappers.py#L604
As it appears, request.environ does not contain SCRIPT_NAME at all (when trying to access localhost/sipa_debug/news)

This has been effectively resolved by 2668b07719497225ac7b893f553916142671f0fe


Bevy game crashed on ios if swipe from screen bottom up

I've met a reproducible crash on bevy ios game. After some trouble shooting, it seems the root cause is, window is not created inside UIApplicationMain. For more info, refer to https://github.com/rust-windowing/winit/issues/1705#issuecomment-1236363196, madsmtm mentioned that the window should created in StartEvent(Init) handler.
In bevy_winit code, window is created early, any idea why? Is it possible to move this code to StartEvent(Init)?
https://github.com/bevyengine/bevy/blob/76ae6f4c6e77866104229659a211045748cfb4bf/crates/bevy_winit/src/lib.rs#L58

In bevy_winit code, window is created early, any idea why?

See the comment above that line:

Note that we create a window here "early" because WASM/WebGL requires the window to exist prior to initializing the renderer.

@bjorn3 yea, I saw it. But not sure is it a hard requirement, any other way to solve the WebGL requirement?

I think it is a hard requirement for WASM/WebGL as the render context is created by calling a method on the <canvas> element of the winit "window".

@bjorn3 Good to know, if this is purely for wasm/webgl, then we can condition compile just for that platform?

I just tried comment that code out, crash gone. 🎉 🎉

You could change the line above from #[cfg(not(target_os = "android"))] to #[cfg(not(any(target_os = "android", target_os = "ios")))].

Related issue: https://github.com/bevyengine/bevy/issues/3185


Mounting SAMBA/NFS share using CIFS  and Kerberos authentication giving mount error(95): Operation not supported

Windows Build Number
Microsoft Windows [Version 10.0.19042.985]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.4.72
Distro Version
Ubuntu 20.04
Other Software
Docker Desktop 3.3.3 (64133)
Docker version 20.10.6
Repro Steps
WSL2 - ubuntu 20.04 (from microsoft store)
mapping NFS/samba using cifs:
Mounting using cifs and username/ password  works:
mount -t cifs -o username=somedomainuser@somefulldomainname //someserverfuldomainname/somesharename /somelinuxfolder
Listing shares from the same server using kerberos ticket (generated by kinit just before)  works:
smbclient -k -L  //someserverfuldomainname/somesharename
Mounting using cifs and  sec=krb5 does not work,
mount -t cifs -o sec=krb5,vers=3.0 /someserverfuldomainname/somesharename /somelinuxfolder
-> it fails with error message mount error(95): Operation not supported:
and dmesg not listing any error.
Exactly same beahviour as in the WSL2 UBUNTU distro also in the container (from ubuntu 20.04 image) running in the Docker Desktop (set to use WSL2) start via compose file that has set:
cap_add:       - ALL     privileged: true
Expected Behavior

If the smbclient can list and also connect to the nfs/smb share using kerberos authentication, why the cifs can not ?

Why the mount -t cifs  can connect only using username /password  and using the kerberos ?

What is the  recommended / supported SECURE way how to map NFS/SAMBA shares from the WSL2 linux distros or Docker desktop containers based on linux images - any documentation ? Is there currently any ?

Actual Behavior
root@WEB-DT-027:/home/pavel# mount -t cifs -o sec=krb5,vers=1.0 //FQDNhost/sharedFolder  /somelinuxmountpoint
mount error(95): Operation not supported
Refer to the mount.cifs(8) manual page (e.g. man mount.cifs) and kernel log messages (dmesg)
root@WEB-DT-027:/home/pavel# dmesg
[640512.077244] WSL2: Performing memory compaction.
[640515.629016] CIFS: Attempting to mount //devdockerhg01.od.rferl.org/docker-recording
[640573.129658] WSL2: Performing memory compaction.
[640634.184655] WSL2: Performing memory compaction.
[640695.261290] WSL2: Performing memory compaction.
root@WEB-DT-027:/home/pavel#
Diagnostic Logs
No response

If the smbclient can list and also connect to the nfs/smb share using kerberos authentication, why the cifs can not ?

Because they are different implementations of the protocol(s). One is implemented in userspace. And the other is implemented in kernelspace.

cap_add: - ALL privileged: true

It isn't operation not permitted. It is operation not supported. At the end of the day that "operation not supported" is by-design linux-kernel. It is unlikely you have a WSL2 networking issue or you wouldn't have made it this far.

any documentation ? Is there currently any ?

Not in the sense you want, that I am aware of. SOP is to try variations of vers= and sec= until one works, which will differ depending on the nature of //someserverfuldomainname. Try -o sec=krb5,vers=3.02 on a lark.
It might be that WSL2 lacks CONFIG_RPCSEC_GSS_KRB5, but that is a WAG (for my part) without any actual supporting evidence. That particular kconfig is =m on a Real Linux box, which at least counts as a WSL diverge. This submission could in principle be discussed its way into a WSL actionable given fail repro steps where //someserverfuldomainname is //<IP_ADDRESS>.

Hello,
Many thanks for your response and throughout explanation.
When forcing vers=3.02   ...  still getting the mount error(95): Operation not supported.
But perhaps the  WSL2 lacking CONFIG_RPCSEC_GSS_KRB5  that you pointed out  might not be  a WAG , but rather the root cause ?
See the issue (an even the pull request) here:
Please support Kerberos protection for NFS by enabling CONFIG_RPCSEC_GSS_KRB5 #6572
If you can confirm / reject that would be helpful  - at least to know that this is current status quo.
Without having the option to use Kerberos to map SMB/NFS the only option that we have for mounting the windows smb shares  seems to be using the username / password  from the linux containers running on WSL2 enabled Docker desktop on windows 10  workstations.   :-(
I guess here is another description of the very same issue:
https://stackoverflow.com/questions/65525400/mounting-cifs-share-using-kerberos-fails-with-operation-not-supported
Many thanks again


integration

How can i simply integrate 0-form over the whole space ?
For instance in fenics i can use the command : assemble(u*dx) where u is gridfunction.
Janusz

I don't think we have a direct function to get the integral of u, but you can create a simple LinearForm (using the space of u) that has a DomainLFIntegrator with Coefficient 1. Then assemble the linear form and dot it with u to get the integral.

To add to @vladotomov's answer -- it depends what other objects do you have built already in your application.
For example, if you have the mass matrix M, you can compute the integral as 1^T M u, where 1 is a vector of ones.
The LinearForm approach above is also a good one, especially if you need the integral of several fields.
If you just need the integral and don't have any other pieces available, the most efficient computation will be to use something similar to ComputeLpNorm in fem/coefficient.hpp with a GridFunctionCoefficient for u, see https://github.com/mfem/mfem/blob/master/fem/coefficient.cpp#L584-L613.
Hope this helps,
Tzanio


KEP for kube-controller-manager -> cloud-controller-manager migration plan

We need a KEP outlining how we intend to migrate existing clusters from using the kube-controller-manager to the cloud-controller-manager for the cloud provider specific parts of Kubernetes.
At KubeCON NA 2018, we discussed grouping the existing cloud controllers under 1 leader election that is shared by the kube-controller-manager and the cloud-controller-manager. For single node control planes this is not needed, but for HA control planes we need a mechanism to ensure that not more than 1 kube-controller-manager or cloud-controller-manager is running the set of cloud controllers in a cluster.

/assign @mcrute
@mcrute is working on the initial design for this.

cc @cheftako

Here's a first draft. There's plenty more to be done but getting this out there for discussion.

Thanks @mcrute!

Thanks @mcrute https://github.com/mcrute! I would like us to also discuss
as part of this how we do a better job of running Controllers in HA
environments. Currently we do not utilize HA well as part of this. If we
could get rid of the kill process when leader election is lost, then we
could get much better utilization in HA. The problem has been that
Controllers tend to kick of goroutines (and similar asynchronous
processing). The problem is that Controller actions may not be idempotent.
So we end up with mutations from something other than the main controller
thread which did not get shut down (or at least shut down in a timely
manner). One thought for this could be to attach an election token (or
similar) to mutations. If the mutator is no longer leader, then the write
is refused and the mutator is notified that they are no longer the leader
(and should stop). While I believe is more than we need for the KCM->CCM
migration, I would like us to consider it as where we are going. It would
be good for us to make sure we are generally heading in that direction.
On Fri, Mar 1, 2019 at 10:41 AM Andrew Sy Kim<EMAIL_ADDRESS>wrote:

Thanks @mcrute https://github.com/mcrute!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/cloud-provider/issues/11#issuecomment-468767347,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA53A-drkWbYBQ3TMM_J5azU7dY8Qhoyks5vSXRtgaJpZM4bD0lq
.

/milestone v1.15
/priority critical-urgent

/assign

This is going to slip into the next release since we couldn't get the KEP reviewed in time for the KEP deadline. Further discussions happening for this in https://github.com/kubernetes/enhancements/pull/979 & https://github.com/kubernetes/kubernetes/pull/77878, hoping to have an implementable KEP in time for v1.16.

/remove-lifecycle stale

/assign @yastij

/remove-lifecycle stale

/lifecycle frozen


some pkgs not on CRAN

One package for my lesson tomorrow may not have binaries up by time of workshop start, and one definitely won't.  Should I put instructions in the lesson for getting from github, or just do that before we start separately?

Instructions in the lesson would be nice, but not necessary.  We can walk folks through that during, especially since we have as many instructors as we do participants!

Okay, i'll add install instructions


without using UserDestructor object, UserName is not getting included in the log

Log.setLogger(new LoggerConfiguration()
.setMinimumLevel(LogEventLevel.Verbose)
{Exception}"))
.writeTo(rollingFile("test-{Date}.log"))
.createLogger());
//I did not use UserDestructor

 User user = new User();
 	user.setUserName(System.getProperty("user.name"));
 	Log.forContext(JavaConsole.class).warning("Hello {name} from {@user}", "World", user);
 	Log.information("Hello {name} from {@user}", "World", user);

Output

2017-09-02 00:24:43.170 SGT [Information] Hello "World" from User { userId: 0 }
userName is not getting included in the log eventhough it has a public accessor method.

Just having a poke around in the code, it looks like only fields are currently accounted for:
https://github.com/80dB/serilogj/blob/master/src/serilogj/parameters/PropertyValueConverter.java#L209
It seems like it would be natural to include gettable properties by default - @80dB what do you think?

Strange, I would have expected the serializer to pickup the "property" too. The boon library is nice, but the lack of documentation is a little bit disturbing.
According to the documentation:
Gets a list of fields merges with properties if field is not found.
https://github.com/advantageous/boon/blob/master/reflekt/src/main/java/io/advantageous/boon/core/reflection/Reflection.java#L304
So it should actually work.
Unlike C#, Java doesn't really seem to have properties, just "emulating" properties using set/get methods.  I'm a little bit surprised it's not working on userName. It seems to do the job just fine in our production environment.

if field is not found

Perhaps the issue is that there is a public field, so property reflection is skipped? @ritchieroi how does that theory stack up against reality? :-)

@ritchieroi I think the issue is fixed now, using another reflection method it now returns both the userId and userName and no private/protected fields (tested this too), so should be exactly what we expect.

Works great.   Thanks guys.


Automatic reporting of errors?  How?

Just started using this and the documentation is confusing.
I assume it would automatically work, but it seems I have to put Sentry.captureException in every catch()?  All over my code?
I see in the docs it says "Sentry's Flutter SDK enables automatic reporting of errors, messages, and exceptions"
Is it automatic or not?  If I remove the Sentry.captureException, nothing happens, so it doesn't seem automatic to me.
I saw this in another issue.  Do I need runZonedGuarded for it to work?
FlutterError.onError = (FlutterErrorDetails details,) async {
if (!isProduction) {
FlutterError.dumpErrorToConsole(details);
return;
}
// turn to Zone
Zone.current.handleUncaughtError(details.exception, details.stack);
};
// Handle dart error
runZonedGuarded<Future>(() async {
SentryFlutter.init((options) => options.dsn = 'xxx',
() {
runApp(MyApp());
},
);
}, (Object error,  StackTrace stackTrace) async {
if (!isProduction) {
print(error);
print(stackTrace);
return;
}
// Only upload error on release mode
Sentry.captureException(
error,
stackTrace: stackTrace,
);
});

Sentry automatically captures uncaught exceptions.
Exceptions you catch urself aren't usually needed to be reported because you're handling themself and thus aren't really bad. Though there's nothing stopping from reporting them, too. If you're catching them yourself you need to use Sentry.captureException.
You don't need to add a runZoneGuarded. SentryFlutter does this for you if you're using the appRunner callback like in your example.
Also SentryFlutter automatically listens to FlutterError.onError, so you don't need that to do it yourself.
A minimal example with automatic capturing for runZoneGuarded and FlutterError.onError looks like this:
import 'package:flutter/widgets.dart';
import 'package:sentry_flutter/sentry_flutter.dart';

Future<void> main() async {
  await SentryFlutter.init(
    (options) {
      options.dsn =<EMAIL_ADDRESS>    },
    // Init your App.
    appRunner: () => runApp(MyApp()),
  );
}

In order to disable reporting to Sentry you can set the DSN to an empty string.
I hope that clears up your questions.

Also, https://docs.sentry.io/platforms/flutter/usage/#tips-for-catching-errors mention about not needing runZonedGuarded nor FlutterError.onError


Guide: Downloading History (Old Messages)

hello im trying to download all the videos from a specific channel even ones sent in the past and im not really sure how to combine the two snippets of code you provided. i tried following the old messaged tutorial you wrote but i kept getting confused.
this is what my settings file looks like without the information filled in:
{
"credentials": {
"email": "REPLACE_WITH_YOUR_EMAIL",
"password": "REPLACE_WITH_YOUR_PASSWORD"
},
"scanOwnMessages": true,
"presenceEnabled": false,
"channels": [
{
"channel": "DISCORD_CHANNEL_ID_TO_DOWNLOAD_FROM",
"destination": "FOLDER_LOCATION_TO_DOWNLOAD_TO",
"allowCommands": false,
"errorMessages": false,
"reactWhenDownloaded": false
}
]
}
but im not sure how to place this snippet of code you provided into it to download all the other messages;
ddg history<PHONE_NUMBER>11000 --since=2020-01-02

That last portion is a command to be used in one of the channels registered to the "adminChannels" section of the settings. Follow settings examples in the readme to see how to structure it.
Once a channel is setup properly in adminChannels, send that command in that channel. (i.e. ddg history 000000000 --before=2020-04-05)

so if i dont have permission to type in the channel i want the videos from i cant download previous videos?

The admin channel is for executing commands for channels like that, where you can't directly use the bot.
Set up the channel you want to download from in the "channels" section, follow the examples.
Set up a channel of yours in a personal/dummy server in "adminChannels" for executing commands.
Let's say the Discord ID for the channel you want to download from is 5. You would go into the personal channel you setup for "adminChannels" and send a message saying ddg history 5 --after=2020-05-06
This bot is controlled via Discord messages, not by the programs console.

thanks so much man just got it working there, love your project


ngMock breaks during clean-up with mocked $rootScope

#13433 introduced $rootScope clean-up after each test. The implementation assumes that $rootScope will always have a $destroy() method, which might not be true for a mocked/decorated $rootScope, resulting in an error.
Also discussed in https://github.com/angular/angular.js/issues/13985#issuecomment-186409320.

I think if you are mocking essential services then you need to provide their API surface. I would go as far as recommending against mocking things such as $injector, $rootScope etc.

Ok, I see how this can be confusing if you inject this in a service but you don't actually use any of the functions of the rootScope. So I'm okay with fixing this.


How do we display our issues on our digital land site?

We maintain and work on our issues in Github but we also want to be able to display issues on own digital land site. Particularly the relevant issues on each of the project pages.
What do we display on the site? And what tech do we need to use to do this?

I started using [octokit/rest.js] to fetch the right issues for a project.
There is a WIP PR with an example. It prints the developer-contribution issue titles to the console.


DRAFT: Enable large zoom levels

What does this PR do?
Builds on the work done on the data fetching methods to allow larger portions of the file to be viewed than was previously possible.
The principal here is that zoom levels now represent what portion of the file the user is asking to view. We calculate the fftSkip value required to meet that need. The starting point is always 0, don't skip and currently we increment the zoom levels by 10% each step until 100% i.e. view the whole recording.
This PR is currently draft for the following reasons:

Are linear 10% increments useful? Not much visual difference between 50% and 100% tbh. Maybe a curve?
If you drag the window down to the bottom of the recording and set an extreme zoom we try to display past the end of the file. Should we move the window back up the recording or some other strategy (prevent the zoom?!?!)
There's a small gap at the bottom of the image when setting zoom to 100%. Need to look into that (or set the yellow background to black in PROD builds so you don't see it?)

All Submissions:

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you successfully ran tests with your changes locally? Including container validation.
[ ] Have you lint your code locally prior to submission?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] Have you labelled this PR correctly? E.G. major, milestone, breaking, breaking-change, minor, feature, enhancement, patch, fix, chore, refactor etc

Changes to Core Features:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?

Are linear 10% increments useful? Not much visual difference between 50% and 100% tbh. Maybe a curve?
Last chat I had with @777arc the suggested ranges for the fft was [1, 2, 5, 10, 20, 50, 100, 1000, 10k, 100k]

If you drag the window down to the bottom of the recording and set an extreme zoom we try to display past the end of the file. Should we move the window back up the recording or some other strategy (prevent the zoom?!?!)

I think we should cap the max level to the zoom to something that display 100% of the record in the screen.

the colour it si due to incomplete FFTS meaning that somethig. that has a partial fft in the end of the file.


Add currency setting to Registry Configuration

Add currency setting to  Registry Configuration ...
so the Financial Information would be in repspective currency
for example registry is based in Japan so all financial info in Japanese yen...
and for example if registrar is from UK they could select to Display their currency... aka add menu simmilar to Language Menu...

I'll add the ability to choose the currency per registry soon and change it based on that, but for the registrar individually to select currency it will require currency exchange which unless you really need it for a project now, I'll postpone to v1.1

Latest commit fixed this one in general.
Regarding registrars choosing also their own currency and currency exchange, I'll add to the todo list for 1.1


[Snyk] Security upgrade org.springframework.boot:spring-boot-starter-validation from 2.5.5 to 3.0.10

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

651/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 6.6
Arbitrary Code Execution SNYK-JAVA-ORGYAML-3152153
org.springframework.boot:spring-boot-starter-validation:  2.5.5 -> 3.0.10
Yes
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Arbitrary Code Execution

Addressed in development as part of in-process measure eval.


Owasp Zap Wrapper shall handle simple form based login

Current Situation
Currently the Owasp Zap Wrapper can execute unauthenticated scans and scans using HTTP basic authentication.
Wanted
We want to able to scan websites with form based login pages.
Solution
Implement a solution, that can be extended later to handle more complex login mechanisms (with and without multi-factor authentication). The implementation uses the sechub webscan config and follows the steps to log into the target website.
For this issue multi-factor authentication is out-of-scope.

This will be done via templates.


Move all standard plugins to pack/dist

Is your feature request about something that is currently impossible or hard to do? Please describe the problem.
Vim is shipped along with several plugins. Some of them sit under normal "packages tree" (i.e. pack/dist/opt) while others are installed directly into $VIMRUNTIME (pre-v8 installation).
Describe the solution you'd like
Move all such plugins (plus relevant files from doc/ or autoload/ etc.) into pack/dist/start. Also, at least few of them could be made optional. In particular, I think that getscript, logipat and vimball are unmaintained and not used widely. So they are good candidates to be moved to pack/dist/opt instead.
Additional context
Another questionable plugin is TOhtml. Albeit being useful, it's a bit specific and not needed by many of us. But I can imagine that some people would object if it's made optional.
Yet another story is about rrhelper. It provides (rarely used and undocumented) callback for --remote-wait switch. Perhaps, it could be refactored/redesigned and moved into main runtime tree, as it doesn't really look like a normal plugin. But, for now, it's easier just to move it into pack/dist/start as everything else.
As for "breaking changes", I can only think of one thing: scripts from $VIMRUNTIME/plugin are loaded before any user plugins, while $VIMRUNTIME/pack/dist/start right after that. But I don't see any problem with that.

scripts from $VIMRUNTIME/plugin are loaded before any user plugins, while $VIMRUNTIME/pack/dist/start right after that. But I don't see any problem with that.

In the sense that these built-in plug-ins don't make any assumptions about settings set to Vim defaults ? They shouldn't In any event, that could be solved while moving each of them. They shouldn't since the user's vimrc is always loaded first.
However according to :h 'rtp', say for Unix,
					Unix:  "$HOME/.vim or
						$XDG_CONFIG_HOME/vim,
						$VIM/vimfiles,
						$VIMRUNTIME,
						$VIM/vimfiles/after,
						$HOME/.vim/after"

$VIMRUNTIME is loaded after the user's files, while according to :h packages
When Vim starts up, after processing your .vimrc, it scans all directories in
'packpath' for plugins under the "pack/*/start" directory.  First all those
directories are added to 'runtimepath'.  Then all the plugins are loaded.
See |packload-two-steps| for how these two steps can be useful.

those packages are loaded before $VIMRUNTIME?

I believe you as :scriptnames is the more reliable source; just commenting how load order was intended according to documentation and could possibly be clarified. At least to me it read the way inferred above?

@Konfekt The docs might be misleading here. There was this relevant question on SO few years ago.


How to use global styles with this module

Hi, In my app we are using styled components, fontawsome and some global styles. fontawsome and global styles are being attached to head. Is there any way I can attach them inside shadow root ?
For global css imports I tried configuring with style-loader insert option but it requires shadow root to be available before hand.
Can the styleSheets prop on root be helpful in this case ? How to use this prop ?

any update ?

Hi @kkorada
I had the same issue and I solved it with:
const sheet = new CSSStyleSheet();
sheet.addRule("*", `box-sizing: border-box`);
const styleSheets = [sheet];

export function Root() {
  return (
    <root.div styleSheets={styleSheets}>
      <App />
    </root.div>
  );
}

I don't know if it's the best way to handle it - if someone finds a more elegant way I will be happy to hear

Also, one could use a polyfill for CSSStyleSheet
https://github.com/calebdwilliams/construct-style-sheets


"Core" module dependency is in both private and public dependency list

In {ModuleName}.Build.cs, "Core" module dependency is defined in both PublicDependencyModuleNames and PrivateDependencyModuleNames. It should be enough to only list it in PublicDependencyModuleNames.

Fixed starting 5.0


[Contribute]

I can resolve mail<EMAIL_ADDRESS>password is diug cuqe rmwv **** and up to date all file, you can waiting me update the next day

what do you mean?

what do you mean?

I can resolve issue for file your can meet


Cartographer grpc crashes when requesting submaps

I've tried to evaluate multi-trajectory SLAM via gRPC whereas one robot served as 'cloud server' as an uplink for another one.
Opening RVIZ and the map display runs cartographer-rviz submaps plugin, which lets the grpc_client send a proto to request a Submap from the server. This works on the robot which does NOT act as a cloud server but crashes (without further stack trace) on the robot with cloud server.
Currently, I'm scanning the code base and trying to understand more details but up to the point where it touches the pose_graph it looks alright for me.
Potential differences when running a grpc_server as uplink "master" (aka cloud server) to the 'slave' servers are:

Slaves do run a LocalTrajectoryUploader
The master grpc_server does not have range scan data in its submaps as this gets filtered out by the slaves before streaming submaps to the master

Other issues I could imagine are resource conflicts when accessing internal datastructures or missing lockings because of threading?
 I cant find that in the code but I might have simply overlooked it. Thank you very much

Hey, i can't answer you question but i am struggling with the same currently.
I am hosting a server on one machine and register two robots/trajectories on it. I believe, that there is simply no feedback from the global optimization on the server back to the robots.
I found this video from roscon 2018 where @MichaelGrupp introduced the cloud computing and if I understood them correctly they feedback the remote result by exporting the pbstream on the server using the write_state service, streaming it to the robots manually and then localize in it. It sounds like this is done in intervals and not really automated by cartographer.

@Tristan9497 I dived into the grpc code and there is indeed no feedback of global optimization. The transport isn't hard to do, in fact it seems all necessary handler and protobuf messages are in place (for submaps, posegraph) and you have the corresponding methods to get all  of these using the posegraph class. However, there are no methods to alternate the states of the posegraph nodes - as far as I did see last week.  Now, as the pose graph is running in a seperate thread and I dont know how all the datastructures are linked I stepped back for the moment to just insert new methods to directly manipulate the internals.  If you are interested we could collaborate on this issue
There is another fork https://github.com/shreyasgokhale/cartographer_ros which allows you to send another grpc server instance the pbstream file. However, it contains as far as I can tell ALL internal data and everytime you do that you would have to relocate yourself in existing trajectories. The robots would not be really in the same coordinate system and transmission time grows with the size of the recorded map

Hey @uahic, sure working on this together sounds nice.
Your Idea sounds indeed very demanding for the network, although i am convinced it would work really well only thing i am concerned is the localizing step.
I was thinking it might be possible to approximate/calculate the current position of the robots by comparing their submaps, meaning the ones of the local slam and then the global optimized ones. We have very easy access to those, since they are on the ROS network anyway.  In the end its just a little less data then your recommendation and of course will grow over time too.
Using the first optimized submap location of both trajectories we could fairly easily determine the starting positions of the trajectories to each other.
Then we could jump to the latest optimized submap compare it to the first one and get the position of it in the "real world". Which will get us already very close depending on submap size.
The remaining part of the trajectory is a not yet finished submap which would be the trasformation between the latest not optimized submap and the robot. This should be fairly easy to get since this is just the pose of local slam compared to the latest non optimized submap position.
Adding all of that up should allow to publish map->odom transforms that are as good as possible without constantly localizing.
It sounds a little hacky though
Let me know if that makes any sense to you

@Tristan9497 I was more referring to implementing the missing functions of the posegraph class to (simply?) change the pose of an already existing node.
A client sends only finished submaps of its trajectories to the 'master' (uplink server) and thus the uplink server. After optimization steps the poses of multiple nodes of each trajectory has changed and should communicated back (that part is easy). On client side 'only' the modification methods are missing.
Method 1)
The user has to set global optimization to disabled for the clients (that is possible already) and gets the posegraph constantly modified (reflected) by that downlink stream. So basically the work here would be to understand the posegraph and optimizer classes and allow for modifications via new API methods
Method 2)
In presence of an uplink server we swap the posegraph class instantiation on the client all together with the posegraph_stub  class. This might cause a larger network load as you get send data from all trajectories back and right now I dont know how much work is left in that stub class

Oh i see now, what you are trying and think this is definetely the right way to go. To be honest i think i need to spend some time with the cartographer pose_graph stuff to actually know what is going on in there. I think this would be an amazing feature to have.
If i have time on the weekend i will investigate a little.

@Tristan9497 @uahic I am currently facing the same problem (not the crash, but the sync between master and slave), have you started to work on this issue? I am willing to contribute and work with you on this, however, I am not using ROS just cartographer in a standalone manner.

您好！我已收到你的邮件！谢谢！          ——曾君

@adiego73 Hi Diego, not yet. The reason is that currently there is no demand in my institution (it may come back soon though!) which doesnt allow me to work on this issue at least during official work times;  Still, I'd consider working on it at a very slow pace from my side. ROS doesnt matter so much for this issue (its basically just an additional wrapper package) and as far as I can see the methods to exchange data with GRPC do already exist for virtually all interesting datastructures.
I studied the RFCs of the former Cartographer developer group and they mentioned the desire to implement all of this but postponed it with some comments about that they have to decide how to implement it which sounds likes: its not as trivial as it looks like. I cant judge (yet) the internals of the classes which asynchronously run the optimization loops but it all seems really to boil down to "how to insert data on the fly without messing up the optimization". I cant remember all the details from before christmas when the knowledge was 'fresh' but I think studying this classes (posegraph optimization or something like that or just posegraph?) is necessary;
The first step may be to draw some rough diagrams of the overall architecture and interaction between classes; I'm right now very busy but I will come  back at this issue

Great, I will start looking at those classes first to understand well how all this works Thanks!

@adiego73 Hey sorry, i was not able to work on this and did not get any further since i was too busy with the project i was working on at that time.


@Andne observed that it looks like a pathing issue, and that appveyor_ci_build.ps1 needs updating. I'm doing that and testing now, will open a PR that fixes it soon.

PR here: https://github.com/cpputest/cpputest/pull/795


Clip cursor width when soft-wrap is on and cursor is at the end of a line

Description of the Change
This pull request prevents the parent tile from disabling sub-pixel anti-aliasing by clipping the cursor width to be fully contained within the editor when soft-wrap is on and the cursor is at the end of a line. For some reason, adding overflow: hidden to the cursor container element doesn't solve the issue, so we're adding this workaround instead.
Alternate Designs
Unclear.
Verification Process

Enable soft-wrap
Keep writing until reaching the end of the line (without wrapping to the next line)
Notice how sub pixel anti-aliasing doesn't break anymore

Before
After

Applicable Issues
Fixes #16889
/cc: @Ben3eeE @50Wliu can you confirm this pull request fixes it? Thanks! 🙇

@as-cii I can still reproduce this issue when the cursor is on the very edge of the window.
Before:
Subpixel AA is lost when the cursor is near the edge of the window

After:
I can move the cursor to this position but when adding another a subpixel AA is lost

Possibly unrelated but I also noticed that changing tabs when the cursor is in this position causes the editor to scroll horizontally:

@Ben3eeE thanks for the feedback! 💯 We should pair on this sometime, as I can't reproduce the problem anymore on my machine unfortunately.

Sounds good, thanks for the feedback @Ben3eeE!

:white_check_mark: Opened https://github.com/atom/atom/issues/16996
:white_check_mark:  Reopened https://github.com/atom/atom/issues/16889 and moved it to the backlog