Very high memory usage in some cases

Relevant versions

postgres_exporter version: 0.5.1
os: CentOS 7

Issue Description
I've been running this exporter on many Postgres instances and I've had cases where it eventually reached 4GB+ of mem and almost 100% CPU usage of 6 cores.  In this specific case, the postgres_exporter is running on a Postgres host which does have very high query volumes at time, and some queries can be large.  I'm just trying to determine if this is something that can normally occur, or if it's a potential memory leak?
If it's a valid behavior, then I'd like to understand in better details the exact specifics to the high mem and CPU usage?  Thanks for your help.

Sounds good!

Any news? Looks like we face with the same issue: we remove "cache_seconds" but every 30 seconds memory usage increased

also noticing a slow memory leak, after a few weeks of running the exporter, it slowly chews up all the available ram

I can confirm that incorrect connection settings caused memory leak.
I've been using latest docker image "wrouesnel/postgres_exporter"

I can confirm that incorrect connection settings caused memory leak.
I've been using latest docker image "wrouesnel/postgres_exporter"


[bilibili] how to download a item in a list

Checklist

[ how to download:  https://www.bilibili.com/video/BV1V54y1B7K3?p=17] I'm asking a question
[ ] I've looked through the README and FAQ for similar questions
[ ] I've searched the bugtracker for similar questions including closed ones

Question

WRITE QUESTION HERE

Here is clickable URL:
https://www.bilibili.com/video/BV1V54y1B7K3?p=17

--playlist-items ... RTFM

... This is probably a duplicate of https://github.com/ytdl-org/youtube-dl/issues/31051 ...
Indeed,
https://www.bilibili.com/video/BV1V54y1B7K3
is a playlist consisting of 18 items (parts), but when that URI is fed to youtube-dl, it is not treated as such:
youtube-dl -v -F "https://www.bilibili.com/video/BV1V54y1B7K3" =>

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-F', 'https://www.bilibili.com/video/BV1V54y1B7K3']
[debug] Encodings: locale cp1253, fs mbcs, out cp737, pref cp1253
[debug] youtube-dl version 2<IP_ADDRESS>19
** This build is unofficial daily builds, provided for ease of use.
** Please do not ask for any support.
[debug] Python version 3.4.4 (CPython) - Windows-Vista-6.0.6003-SP2
[debug] exe versions: ffmpeg 4.4.1, ffprobe 4.4.1, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[BiliBili] 1V54y1B7K3: Downloading webpage
[BiliBili] 1V54y1B7K3: Downloading video info page
[info] Available formats for 1V54y1B7K3:
format code  extension  resolution note
0            flv        unknown    230.13MiB

and even when issuing
youtube-dl -v -F "https://www.bilibili.com/video/BV1V54y1B7K3?p=17"

or (as suggested)
youtube-dl -v -F "https://www.bilibili.com/video/BV1V54y1B7K3" --playlist-items 17

the outcome is identical:
[debug] Proxy map: {}
[BiliBili] 1V54y1B7K3: Downloading webpage
[BiliBili] 1V54y1B7K3: Downloading video info page
[info] Available formats for 1V54y1B7K3:
format code  extension  resolution note
0            flv        unknown    230.13MiB

According to testimony in #31051, I expect the 230MiB file to correspond to the 1st item of that playlist (while OP wants item/part 17); out of curiosity, I DLed that file and it's a 1080p30 encode, with a duration of ca. 1h31m.
Contrast all the above with how (latest) yt-dlp handles those URIs:
yt-dlp "https://www.bilibili.com/video/BV1V54y1B7K3" --flat-playlist =>

BiliBili] 1V54y1B7K3: Grabbing original ID via API
BiliBili] 840983192: Downloading webpage
BiliBili] 840983192: Extracting videos in anthology
BiliBili] Downloading anthology 840983192 - add --no-playlist to just download video
download] Downloading playlist: ?????????? ?????? ?? ?????(???)
BiliBili] Playlist ?????????? ?????? ?? ?????(???): Downloading 18 videos of 18

download] Downloading video 1 of 18
download] Downloading video 2 of 18
download] Downloading video 3 of 18
download] Downloading video 4 of 18
download] Downloading video 5 of 18
download] Downloading video 6 of 18
download] Downloading video 7 of 18
download] Downloading video 8 of 18
download] Downloading video 9 of 18
download] Downloading video 10 of 18
download] Downloading video 11 of 18
download] Downloading video 12 of 18
download] Downloading video 13 of 18
download] Downloading video 14 of 18
download] Downloading video 15 of 18
download] Downloading video 16 of 18
download] Downloading video 17 of 18
download] Downloading video 18 of 18
download] Finished downloading playlist: ?????????? ?????? ?? ?????(???)

and
yt-dlp -v -F "https://www.bilibili.com/video/BV1V54y1B7K3?p=17" =>

[BiliBili] 1V54y1B7K3: Grabbing original ID via API
[BiliBili] 840983192: Downloading webpage
[BiliBili] 840983192: Extracting videos in anthology
[BiliBili] 840983192: Downloading tags
[info] Available formats for 840983192_part17:
ID EXT RESOLUTION |   FILESIZE  TBR PROTO | VCODEC         VBR ACODEC      ABR
------------------------------------------------------------------------------
0  m4a audio only | ~ 46.22MiB  67k https | audio only         mp4a.40.2   67k
1  m4a audio only | ~ 87.59MiB 127k https | audio only         mp4a.40.2  127k
2  m4a audio only | ~ 87.59MiB 127k https | audio only         mp4a.40.2  127k
3  mp4 640x360    | ~ 33.21MiB  48k https | av01.0.01M.08  48k video only
4  mp4 640x360    | ~ 36.15MiB  52k https | avc1.64001E    52k video only
5  mp4 852x480    | ~ 42.70MiB  62k https | av01.0.04M.08  62k video only
6  mp4 852x480    | ~ 49.83MiB  72k https | avc1.64001F    72k video only
7  mp4 1280x720   | ~ 67.65MiB  98k https | av01.0.05M.08  98k video only
8  mp4 1280x720   | ~ 82.54MiB 120k https | avc1.640028   120k video only
9  mp4 1920x1080  | ~109.79MiB 159k https | av01.0.08M.08 159k video only
10 mp4 1920x1080  | ~126.64MiB 184k https | avc1.640032   184k video only

or
yt-dlp -F "https://www.bilibili.com/video/BV1V54y1B7K3" -I 17 =>

[BiliBili] 1V54y1B7K3: Grabbing original ID via API
[BiliBili] 840983192: Downloading webpage
[BiliBili] 840983192: Extracting videos in anthology
[BiliBili] Downloading anthology 840983192 - add --no-playlist to just download
video
[download] Downloading playlist: ?????????? ?????? ?? ?????(???)
[BiliBili] Playlist ?????????? ?????? ?? ?????(???): Downloading 1 videos
[download] Downloading video 1 of 1
[BiliBili] 1V54y1B7K3: Grabbing original ID via API
[BiliBili] 840983192: Downloading webpage
[BiliBili] 840983192: Extracting videos in anthology
[BiliBili] 840983192: Downloading tags
[info] Available formats for 840983192_part17:
ID EXT RESOLUTION |   FILESIZE  TBR PROTO | VCODEC         VBR ACODEC      ABR
------------------------------------------------------------------------------
0  m4a audio only | ~ 46.22MiB  67k https | audio only         mp4a.40.2   67k
1  m4a audio only | ~ 87.59MiB 127k https | audio only         mp4a.40.2  127k
2  m4a audio only | ~ 87.59MiB 127k https | audio only         mp4a.40.2  127k
3  mp4 640x360    | ~ 33.21MiB  48k https | av01.0.01M.08  48k video only
4  mp4 640x360    | ~ 36.15MiB  52k https | avc1.64001E    52k video only
5  mp4 852x480    | ~ 42.70MiB  62k https | av01.0.04M.08  62k video only
6  mp4 852x480    | ~ 49.83MiB  72k https | avc1.64001F    72k video only
7  mp4 1280x720   | ~ 67.65MiB  98k https | av01.0.05M.08  98k video only
8  mp4 1280x720   | ~ 82.54MiB 120k https | avc1.640028   120k video only
9  mp4 1920x1080  | ~109.79MiB 159k https | av01.0.08M.08 159k video only
10 mp4 1920x1080  | ~126.64MiB 184k https | avc1.640032   184k video only
[download] Finished downloading playlist: ?????????? ?????? ?? ?????(???)

I used -f 10+2 to fetch item/part 17 (1080p30 encode again, ca. 1h34m) and it's different to the video file yt-dl fetched...
So, unless I'm missing something obvious (I know it has happened before 😜 ), the answer to OP's question would be (i.e. if I were to supply it) that it's impossible to download referenced video via youtube-dl in its current state, because its bilibiliIE is not up to that task (requires an update) ...

yt-dlp works out that the page with ID 1V54y1B7K3 is actually 840983192; but that page also gets just the single 230MB flv.
Apparently a yt-dlp back-port would be good.

Duplicate of #30151.

Duplicate of https://github.com/ytdl-org/youtube-dl/issues/30151.

Actually, as I wrote:

duplicate of https://github.com/ytdl-org/youtube-dl/issues/31051

😜


EuroStack

Part 4 could benefit from references to growing work in Europe on EuroStack.

 Gov4Git notices
On Friday, 06-Sep-24 18:14:06 UTC by Gov4Git dev
Notice j46eyi
Started managing this issue, using the Plural Management Protocol v1,  as Gov4Git concern 1005 with initial priority score of 0.000000.
This project is managed by Gov4Git, a decentralized governance system for collaborative git projects.
To participate in governance, install the Gov4Git desktop app.
Notice nrfq32
This issue's priority score is now 0.000000.
The cost of priority is 0.000000.
The projected bounty is now 0.000000.
Notice xc6rb6
The set of eligible proposals claiming this issue is empty.

 Gov4Git notices
On Tuesday, 10-Sep-24 14:38:13 UTC by Gov4Git dev
Notice zqod7t
This issue's priority score is now 10000.000000.
The cost of priority is 10000.000000.
The projected bounty is now 10000.000000.
Notice v4ilvm
The set of eligible proposals claiming this issue is empty.


Simplify revert

Simplify the implementation of Wiki#revert_page by abstracting low-level grit calls to the grit adapter (gollum/grit_adapter#14). I've manually tested that this branch with the corresponding branch of grit_adapter behaves the same as master, but would appreciate if someone else could verify this (since the revert tests don't work with grit).

Just pushed grit-adapter 2.0 and made sure this branch uses that dependency. That should fix the build.


Trying alternative simulation order.

I notice a bug in the current simulation loop when choosing no_agents_t as stop condition.
Now the order is:

simulation status callback
factory of new agents
bid phase
on_* and trade callbacks
ask phase
check agent::stop
check stop condition

The thing is, imagine if the factory would still generate new agents in the next iteration, however all agents in the current iteration have stopped. Since we are checking the stop condition at the end, the simulation would stop and the factore is never called. Thus, simulation is terminated before the user expects it to terminate.
I propose the following order:

factory
simulation status callback
stop condition
bid phase
on_* and trade callbacks
ask phase
check agent::stop

Consequence: now when the factory provides more agents, the simulation do not stop early. However, if one chooses time_threshold as an stop condition, some agents are created but never run. But user can learn the last run iteration by checking the last call of simulation_callback.

I am not totally satisfied with this order, though...
@JorgeLuizFranco @vitorcurtis-ita any ideas?

Discussing with @vitorcurtis-ita , we realized that it does make sense to simulation_status_callback be called in the case of t_end + 1. The ideia is that the user might want to check the book status after the simulation has finished (at iteration t_end). It is a little bit useless the last call of factory in the case of time_threshold stop condition, but since the factory knows t, the user can easily avoid unnecessary creations.

@JorgeLuizFranco any thoughts?

@verri , from what I understood in this solution some agents are going to be created and not run. I think this is expected since this means that the t_chosen was not sufficient for the number of agents generated. Thus, simulation_callback is how the user is going to know which agents were run and which not, right? This is a good solution, but from what I read in the code, the user is going to be responsible for creating the simulation_callback to do this, probably it would be good if we had a default function that does this if the user does not want to waste time in creating its own.

Sorry guys, I have not read the code yet. So my contributions will be minimal.
@JorgeLuizFranco, I guess this line prevents simulation_callback from being called if it's not defined: https://github.com/drone-comp/uatlib/blob/30a8650d48478e1669c15eff0891283a451bac84/include/uat/simulation.hpp#L203
Is that what you mean?

@JorgeLuizFranco we already provide a way for the user to avoid generating useless agents. Just use a generator function that does not generate anything agents after t_end.

@JorgeLuizFranco we already provide a way for the user to avoid generating useless agents. Just use a generator function that does not generate anything agents after t_end.
But, answering your question, if the user wants to write a simpler factory and deal with the agents in the simulation callback, it would work as well.

Just notice a nice symmetry in this proposal.
Both
#include <uat/simulation.hpp>
#include <iostream>

int main() {
  uat::simulate<std::monostate>({
    .stop_criterion = uat::stop_criterion::time_threshold_t{3},
    .simulation_callback = [](uat::uint_t t, const auto&, auto) {
      std::cout << t << '\n';
    },
  });
}

and
#include <uat/simulation.hpp>
#include <iostream>

struct do_nothing : uat::agent<std::monostate> {
  auto stop(uat::uint_t, int) -> bool override { return true; }
};

int main() {
  uat::simulate<std::monostate>({
    .factory = [](uat::uint_t t, int) -> std::vector<uat::any_agent> {
      if (t <= 3) {
        std::vector<uat::any_agent> agents;
        agents.emplace_back(do_nothing{});
        return agents;
      }
      return {};
    },
    .stop_criterion = uat::stop_criterion::no_agents_t{},
    .simulation_callback = [](uat::uint_t t, const auto& status, auto) {
      std::cout << t << ' ' << status.active_count() << '\n';
    },
  });
}

call simulation_callback five times, for t = 0, 1, 2, 3, and 4.

Even nicer version of the second example:
#include <uat/simulation.hpp>
#include <iostream>

struct do_nothing : uat::agent<std::monostate> {
  auto stop(uat::uint_t, int) -> bool override { return true; }
};

int main() {
  uat::simulate<std::monostate>({
    .factory = [](uat::uint_t t, int) -> std::vector<do_nothing> {
      if (t > 3)
        return {};
      return {do_nothing{},};
    },
    .stop_criterion = uat::stop_criterion::no_agents_t{},
    .simulation_callback = [](uat::uint_t t, const auto& status, auto) {
      std::cout << t << ' ' << status.active_count() << '\n';
    },
  });
}

with the modifications I have just pushed.

Symmetry is such a beautiful thing.

@JorgeLuizFranco take a look at the last commit. One of the reasons I chose _t for the factory type is that it has the potential to become more complex over time even keeping compatibility.


Enable price list in details kebab while grouped by project, cluster, & node

User Story
As user, I expect to see the price list option in the details kebab, while grouped by cluster. However, it would be a nice addition if the price list were also available when grouped by project and node. Since the cluster name is available, it should be possible to enable the price list option in the details kebab.
Impacts
Ex. UI, Docs
Role
User
Assumptions

Details kebab is available via https://github.com/project-koku/koku-ui/issues/627
Price list option will be disabled while grouped by "Tags"

UI Details
Acceptance Criteria

[ ] Each item in the table will continue to show a Kebab menu
[ ] Kebab menu will always have a "View price list" option
[ ] View price list is disabled when grouped by Tags

During our last conversation, it was my impression that we needed the cluster name, which grouped by tags does not provide. Thus, we suggested to disable the price list while grouped by tag.
While reviewing the price list, I noticed the code does not depend on the cluster associated with the group by. I now see that we're obtaining UUIDs from a request to the providers API.
That said, it would not matter if the user grouped by cluster, tag, etc. I believe the kebab could show the price list enabled, regardless of what group by has been applied.
@nlcwong Do we really want to disable when grouped by tag?

After speaking with @nlcwong, I believe we should only enable when group by cluster. Disable when group by is anything else.
I'm thinking that we also need to pass the cluster to the price list modal. I believe the intention of having a kebab in each table row is to show the price lists associated with individual clusters.
The demo works because we only have one provider, but there will be others.

Thank you @dlabrecq for pointing this out.  What I also want to point out is that for the OpenShift details, its possible that a selected cluster might not have a price list attached to it (because it is an OpenShift cluster whose infrastructure is on AWS), would it be technical possible that we have the ability to know if there is a price list associated with a cluster so that when they click on the kebab, it would disable the view price list link if the cluster did not have a price list associated with it?  Actually, is there a flag of some sort to know if that cluster is on AWS?

Currently, the rates API request is generated only when the user selects the price list option in the kebab. If there is no price list for the associated cluster, a "No rates were found" message is displayed in the modal.
Note that the rates API request is intentionally lazily loaded for better page loading performance. Similar to expanding a row, we want to avoid making extra requests that may go unused.
We could change this so that the request is made when the user loads the details page. Thus, the price list option in the kebab can be disabled when there's nothing associated with an individual cluster.
That said, would we be making an extra request for something that is rarely used? We're currently making up to 6 API data requests when the details page is rendered. Don't feel strongly against adding another, but something to consider.

After speaking with @nlcwong, it sounds like users may want an empty state to explain why there is no price list (and how to create one) for a given cluster? Currently, when the price list option is disabled, users have no explanation as to why.
In the Ocp details page, we also have clusters on AWS. However, the clusters on AWS are not known to the UI. If the APIs could provide that information, we could disable the price list option (or hide it), when a cluster is on AWS. Until then, I suggest we continue to lazily load, but show an empty state in the price list modal.

Sounds reasonable but I will open a new issue for that as it will be easy for me to manage.

Opened ticket #797 to address empty state in the modal.


Scenario Outlines missing replacement placeholders

This section is missing placeholders;

Should have had <Field> and <Value> placeholders;

Generated file has text for placeholders, is conflicting with part of the Markdown syntax.
>
> **Then** the <Field> column returns the value "<Value>"
>
> **And** I have 1 row returned
>
> Examples:
>
> | Field | Value |
> | --- | --- |

Github's render;

Then the  column returns the value ""
And I have 1 row returned
Examples:

Field
Value

Need to change Markdown output to
> **Then** the \<Field\> column returns the value "\<Value\>"

Then the <Field> column returns the value "<Value>"
And I have 1 row returned
Examples:

Field
Value

The same issue applies to table values


Re-enable CSV tests

GH actions was failing these tests so they were disabled

Codecov Report

Merging #75 (e5d453e) into main (7db244f) will not change coverage.
The diff coverage is n/a.

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

@@           Coverage Diff           @@
##             main      #75   +/-   ##
=======================================
  Coverage   80.04%   80.04%
=======================================
  Files         104      104
  Lines        3618     3618
=======================================
  Hits         2896     2896
  Misses        722      722

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 7db244f...fdc8f0d. Read the comment docs.


adds the barebones for the organization model file

adding in the basic bare minimum for the organization table, things like id and name. Will get refined later.

I closed this one since the other pull request has them both in there as well


Alert for Maintenance Window

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
New Notification
🔖 Feature description
Can we add where I can send notifications to Discord or email when I enable Maintenance Windows to let everyone know whats going on.
✔ Solution
Would like to let people know when I enable a Maintenance Window.
❓ Alternatives
None really.
📝 Additional Context
No response

Is this a duplicate of #2722?
If no, please edit your issue to make it more distinct.
If yes, please close this issue, as duplicate issues just make issue-management harder. ^^
Also maybe duplicates depending on meaning: #3022 or #3170


Git Conflict Issues

hello! the git conflict issues seem to have made their way into the file content. you can see what i'm talking about by looking here: https://github.com/search?q=repo%3AWizqdev%2FOpenX+<<<<<<<&type=code

Hey! Thanks for the report.
as said in readme its yet underdevelopment
soon there will be a relase with the fixes and updates
hope it helps


Bucket diagnostics

Purpose

[x] Adds a Diagnostics module to ClimaLand/src, using ClimaDiagnostics.jl
[x] Make default diagnostic variables for the Bucket model
[x] Use these defaults diagnostic variables to write a netcdf output file in Bucket experiments
[x] Add documentation with all bucket variables, see https://clima.github.io/ClimaAtmos.jl/dev/available_diagnostics/
[x] Write unit tests for ClimaLand.Diagnostics functions
[x] Write methods to compute diagnostic variables with bucket in a script, and in another script, the add_diagnostics dispatching of the bucket model (later on, we will have more methods for other models, e.g., soilcanopy, that will dispatch as well, e.g., both have air temperature)
[x] Write more documentation on how to use ClimaLand.Diagnostics (ClimaLand.CLD) - users and developpers docs
[x] Add comments to Bucket diagnostic variables
[x] Better names for Bucket diagnostic variables
[ ] More links in the docs
[x] Tests
[x] Modify a Bucket experiment by using ClimaLand.Diagnostics
[x] Add SoilCanopyModel diagnostics

@Sbozzolo
Do you know why ClimaLandSimulations CI fail?
The error is
Run julia --project= -e 'using Pkg; Pkg.develop(path="$(pwd())"); Pkg.develop(path="$(pwd())/lib/ClimaLandSimulations")'
   Resolving package versions...
ERROR: Unsatisfiable requirements detected for package ClimaDiagnostics [1ecacbb8]:
 ClimaDiagnostics [1ecacbb8] log:
 ├─possible versions are: 0.0.1-0.0.5 or uninstalled
 └─restricted to versions 0.2 by ClimaLand [08f[4](https://github.com/CliMA/ClimaLand.jl/actions/runs/9321596680/job/25660930914?pr=628#step:5:5)d4ce] — no versions left
   └─ClimaLand [08f4d4ce] log:
     ├─possible versions are: 0.12.2 or uninstalled
     └─ClimaLand [08f4d4ce] is fixed to version 0.12.2

But if I run
julia --project= -e 'using Pkg; Pkg.develop(path="$(pwd())"); Pkg.develop(path="$(pwd())/lib/ClimaLandSimulations")'

locally, it works just fine

Maybe you have to add ClimaDiagnostics to the Project.toml for ClimaLandSimulations?

@kmdeck Could you just look at src/Diagnostics/define_diagnostics.jl and check if the names, units, and comments look good? No need to review the full PR!
Thank you!!

@Sbozzolo

I moved the experiment to test (there is nothing new in experiment now). The idea is having a global bucket model in test that then calls some Diagnostic functions and test them. To do that, I had to add SciMLBase and ClimaTimeSteppers to the test Project.toml, I am not sure if this is something we are trying to avoid.
I added comments and changed some names to src/Diagnostics/define_diagnostics.jl
I added your macro to generate error functions
Docs: available_diagnostics works well
Docs: I added more content to developers and users documentation page, but there is still more to add (and add more links)
All CI checks passed

Let's discuss the Bucket experiment a bit, we want to use an existing experiment, but it would be good if variables vary in space at least... I am not sure it is the case with global_bucket_function.jl (our heatmap plots are 1 color), maybe we should use another. We were using global_bucket_temporalmap.jl because of this, which has albedo varying in space (read from a file), but it defines the bucket model inside a function that I commented out before... and we want to not change the previous code but add diagnostics to it. Anyway, let's discuss.

Let's not forget about the issue on @generate_error_functions


Made Scope optional for both request and response (#124)

So as per spec. the scope is optional. Introducing and Option[Scope] is an API breaking change, however, it keeps it clean.

Thanks for the contribution and sorry for the delay!
I'll this PR tomorrow

Merged and published in https://github.com/ocadotechnology/sttp-oauth2/releases/tag/v0.14.0-RC1 :tada:


Update dataspace by zone

Technical Ticket
Replace all reference to Dataspace in the Dataverse Portal applicaiton by Zone.

Tested on desktop => Success


Last seen level of encountered Pokemon now tracked

This adds support for tracking and displaying the last level seen for an encountered Pokemon. This helps match move data tracked from previous encounters with the newly encountered, likely higher level Pokemon. To make space for displaying this feature, the old "HP: ?/?" useless text has been removed when viewing the enemy Pokemon.
When encountering a Pokemon for the first time, a blank line is shown:

Future encounters will show the level of the previously encountered Pokemon:

A few notes here:

This is a change to the Tracker Data object. It should be compatible and upgrade safe, as it doesn't change existing data. It adds a new data property to be tracked. Code that uses this property will check if it's nil just in-case.
Noting the last seen level of a Pokémon doesn't care if it was a wild Pokémon or one that belonged to a Trainer. This is intentional and by design.


Support for traits defined on base classes

Hi,
thanks for the great plugin. One feature I am missing is support for traits defined on abstract base classes of tests. These are currently not picked up by Resharper (at least with XUnit 1.9.2, which I am still using because of performance problems with v2).
Actually, I even have a fix for this problem. Sorry for not making a pull request, but I didn't have a chance to test it properly with all Resharper-XUnit version combinations.
Anyway, the only change needed to make it work (at least for Resharper 9 / xunit v1) is replacing MetadataTypeInfoAdapter.GetCustomAttributes method with this implementation:
public IEnumerable GetCustomAttributes(Type attributeType)
{
var current = metadataTypeInfo;
while (current != null)
{
foreach (var attribute in (from attribute in current.CustomAttributes
where attributeType.IsAssignableFrom(attribute.UsedConstructor.DeclaringType)
select attribute.AsAttributeInfo()))
{
yield return attribute;
}
current = (current.Base == null) ? null : current.Base.Type;
}
}

Thanks for the code snippet! Traits on base classes are currently working in 9.2 and 10.0


Should depend on pytest

Fresh install:
ERROR:colcon.colcon_core.entry_point:Exception loading extension 'colcon_core.python_testing.pytest': No module named '_pytest'
Traceback (most recent call last):
  File "/Users/mikepurvis/colcon_env/lib/python3.6/site-packages/colcon_core/entry_point.py", line 86, in load_entry_points
    extension_type = load_entry_point(entry_point)
  File "/Users/mikepurvis/colcon_env/lib/python3.6/site-packages/colcon_core/entry_point.py", line 128, in load_entry_point
    return entry_point.load()
  File "/Users/mikepurvis/colcon_env/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2322, in load
    return self.resolve()
  File "/Users/mikepurvis/colcon_env/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2328, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/Users/mikepurvis/colcon_env/lib/python3.6/site-packages/colcon_core/task/python/test/pytest.py", line 7, in <module>
    from _pytest.main import EXIT_NOTESTSCOLLECTED
ModuleNotFoundError: No module named '_pytest'

usage: colcon [-h] [--log-level LOG_LEVEL]
              {build,extension-points,extensions,info,list,metadata,test,test-result}
              ...

Error: No verb provided

Thanks for the report. The problem has already been addressed in colcon/colcon-core#5 and will be part of the next release.

Thanks @mikepurvis for reporting this.
This has been addressed in https://github.com/colcon/colcon-core/pull/5 and will be part of the next release.

Ah, so I see. I took a quick look at recent commits but missed the fix. 👍

Is the latest release usable, or is it best to try it from source? I'm running into some other random things which I won't bother trying to ticket if everything is still in flux anyway.

Since the last release several things have changes. On ci.ros2.org we use the latest state from the default branch. That should be working. A new release will hopefully be around next week.


SyntaxError: Unexpected token '.'

Hello,
Not sure if it's an issue with the code or my environment, but when I run any of the vodafone-station-cli command I get this error: SyntaxError: Unexpected token '.'
I've installed it via npm and haven't seen any errors at the time of the installation (vodafone-station-cli/1.2.8 linux-arm node-v12.22.12)
It runs on a RPI 3B+, latest raspian release: Linux 6.1.21-v7+ #1642 SMP armv7l GNU/Linux
The target is Arris TG3442VF, Firmware: AR<IP_ADDRESS>_060822_7244.SIP.10.X1

I just ran it on my Pi4 and didn't see it when running the discover command. How did you install/run it?

install: sudo npm install vodafone-station-cli
run: vodafone-station-cli discover (only help works) / running with sudo - same error.

Do you mind trying it with a more recent version of node (if you have access to it)?

@cifrosul I've just tried it with different node versions from 12 up and can confirm, that it doesn't work for node versions <16. Thank you for bringing this to my attention! I will update the documentation and package.json accordingly.
Here's a record of my tests:
nyxor@nyx:~ $ nvm install 12
Downloading and installing node v12.22.12...
Downloading https://nodejs.org/dist/v12.22.12/node-v12.22.12-linux-arm64.tar.xz...
########################################################################################################################################### 100.0%
Computing checksum with sha256sum
Checksums matched!
manpath: can't set the locale; make sure $LC_* and $LANG are correct
Now using node v12.22.12 (npm v6.14.16)
Creating default alias: default -> 12 (-> v12.22.12)
nyxor@nyx:~ $ nvm use 12
manpath: can't set the locale; make sure $LC_* and $LANG are correct
Now using node v12.22.12 (npm v6.14.16)
nyxor@nyx:~ $ npx vodafone-station-cli discover
Unexpected token '.'
nyxor@nyx:~ $ nvm use 14
N/A: version "v14" is not yet installed.

You need to run `nvm install 14` to install and use it.
nyxor@nyx:~ $ nvm install 14
Downloading and installing node v14.21.3...
Downloading https://nodejs.org/dist/v14.21.3/node-v14.21.3-linux-arm64.tar.xz...
########################################################################################################################################### 100.0%
Computing checksum with sha256sum
Checksums matched!
manpath: can't set the locale; make sure $LC_* and $LANG are correct
Now using node v14.21.3 (npm v6.14.18)
nyxor@nyx:~ $ npx vodafone-station-cli discover
Possibly found modem under the following IP: <IP_ADDRESS>
Something went wrong. TypeError: Promise.any is not a function
    at ModemDiscovery.discover (/home/nyxor/.npm/_npx/311409/lib/node_modules/vodafone-station-cli/lib/modem/discovery.js:63:46)
    at Discover.discoverModem (/home/nyxor/.npm/_npx/311409/lib/node_modules/vodafone-station-cli/lib/commands/discover.js:16:49)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async Discover.run (/home/nyxor/.npm/_npx/311409/lib/node_modules/vodafone-station-cli/lib/commands/discover.js:24:9)
    at async Discover._run (/home/nyxor/.npm/_npx/311409/lib/node_modules/vodafone-station-cli/node_modules/@oclif/core/lib/command.js:80:22)
    at async Config.runCommand (/home/nyxor/.npm/_npx/311409/lib/node_modules/vodafone-station-cli/node_modules/@oclif/core/lib/config/config.js:301:25)
nyxor@nyx:~ $ npx vodafone-station-cli discover
Could not find a router/modem under the known addresses.
Something went wrong. Error
    at discoverModemIp (/home/nyxor/.npm/_npx/311465/lib/node_modules/vodafone-station-cli/lib/modem/discovery.js:17:15)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async Discover.discoverModem (/home/nyxor/.npm/_npx/311465/lib/node_modules/vodafone-station-cli/lib/commands/discover.js:13:29)
    at async Discover.run (/home/nyxor/.npm/_npx/311465/lib/node_modules/vodafone-station-cli/lib/commands/discover.js:24:9)
    at async Discover._run (/home/nyxor/.npm/_npx/311465/lib/node_modules/vodafone-station-cli/node_modules/@oclif/core/lib/command.js:80:22)
    at async Config.runCommand (/home/nyxor/.npm/_npx/311465/lib/node_modules/vodafone-station-cli/node_modules/@oclif/core/lib/config/config.js:301:25)
nyxor@nyx:~ $ nvm install 16
Downloading and installing node v16.20.0...
Downloading https://nodejs.org/dist/v16.20.0/node-v16.20.0-linux-arm64.tar.xz...
########################################################################################################################################### 100.0%
Computing checksum with sha256sum
Checksums matched!
manpath: can't set the locale; make sure $LC_* and $LANG are correct
Now using node v16.20.0 (npm v8.19.4)
nyxor@nyx:~ $ nvm use 16
manpath: can't set the locale; make sure $LC_* and $LANG are correct
Now using node v16.20.0 (npm v8.19.4)
nyxor@nyx:~ $ npx vodafone-station-cli discover
Possibly found modem under the following IP: <IP_ADDRESS>
Discovered modem: {"deviceType":"Arris","firmwareVersion":"<IP_ADDRESS>.14.EURO.PC20","ipAddress":"<IP_ADDRESS>"}

Thank you. I've managed to update to node 19 and indeed it works.


Add missing license logic to template

/hold
I want to update this according to https://github.com/knative/serving/pull/7152

/lgtm
/approve

/hold cancel


Old Cheat Sheet for Creating Collections

Under "Creating Collections (Sets or Groups of Files) in ScholarSphere" on the help page, there is a link to an out-of-date resource for creating collections.

Check google analytics to see if this is being used before updating

@cam156 No views that I can see since 2012.


FastAdapter.getPosition implementation question

About this issue
FastAdapter.getPosition loops through adapters and accumulates item count until the target is found. My question is, why to use position = adapter.adapterItemCount instead of +=? Using = remembers the last adapter item count rather than all item count before. Is this a bug, or is my understanding wrong?
Details

[5.3.5] Used library version
[4.1.3] Used tooling / Android Studio version

Checklist

[√] Searched for similar issues
[√] Checked out the sample application
[√] Read the README
[√] Checked out the CHANGELOG
[√] Checked out the MIGRATION GUIDE

This looks indeed like a bug. Thanks for pointing this out. Will have a look


[FEAT] : Publish the extension to the Open VSX Registry

Would you be able to publish this extension to the Open VSX Registry (https://open-vsx.org/) so it's available to VS Code Compatible Editors, such as VSCodium?

Hi @mhkb,
Sorry for the late reply, just reopened this repo.
I'll do it after the update i'm working on to upgrade this extension to PHP 8.x new features.
Regards.
H.


Make AsyncMqttClient.h optional like USETFT

please add download reference to AsyncMqttClient.h to documentation and make the usage of Mqtt configurable.
see https://github.com/NoNeedForAName123/Esp-radio/blob/master/Esp_radio.ino how it worked for me.

pls. see https://github.com/NoNeedForAName123/Esp-radio/blob/master/Esp_radio.ino how it worked for me.

If you mean "upload", thats already documented.  You can switch off Mqtt by removing the "mqtt"-lines in the radio.ini file.  I documented the last feature in the new pdf-file.

Sorry, I mean in the ESP-radio.pdf in the chapter "Arduino IDE installation for Esp-radio" there is the download location for the AsyncMqttClient missing. So even with switching off mqtt in the ini file compilation fails:
`/Esp-radio/Esp-radio.ino:113:29: fatal error: AsyncMqttClient.h: No such file or directory
#include <AsyncMqttClient.h>
https://github.com/marvinroger/async-mqtt-client
^
compilation terminated.`

Fixed.


Nonexistent hyperlink

[Enter feedback here]

As the Teams admin, you control which features guests can (and can't) use in Teams. Guests in Teams are covered by the same compliance and auditing protection as the rest of Microsoft 365, and can be managed within Azure AD. Guest access is subject to Azure AD and Microsoft 365 or Office 365 service limits.
Hyperlink for bold section above doesn't work.

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

ID: 2492e210-08f9-a476-97ec-008926def961
Version Independent ID: adcb0dd9-4a3c-b665-6e34-ce3276c89b4d
Content: Guest access in Microsoft Teams - Microsoft Teams
Content Source: Teams/guest-access.md
Service: msteams
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@madnata  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@madnata thank you for pointing this out. Would this link suffice - https://docs.microsoft.com/microsoft-365/solutions/collaborate-as-team?view=o365-worldwide

Yes I think this section in particular of the first article linked to works
best:
https://docs.microsoft.com/en-us/microsoft-365/solutions/collaborate-as-team?view=o365-worldwide#teams-guest-access-settings.
Thanks!
On Thu, 19 Nov 2020, 11:44 pm Lindsay<EMAIL_ADDRESS>wrote:

@madnata https://github.com/madnata thank you for pointing this out.
Would this link suffice -
https://docs.microsoft.com/microsoft-365/solutions/collaborate-as-team?view=o365-worldwide
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/6223#issuecomment-730351745,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABF72FVFSYUZSXHSUF6IK3TSQUHLHANCNFSM4TYACYUQ
.

@madnata 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.


Go to definition or Find references not working

Using latest preview Release Date: March 7, 2017 (build 26304.00).
Hitting F12 will work to find definition of functions in imported or inherited modules, but won't find variable definitions. If in same module, it works fine. Get a 'Cannot go to definition' error.
The same issue for Ctrl-K,R Find all references.

So I can F12 to a variable definition in another module just fine, though trying to rename it crashed VS (trace below). FAR also found all the references I expected, though some of the lines showed as blank in the results window.
I was working in our Flask web project template. Perhaps there's something about your project layout that's causing us trouble? Can you share your folder/file structure and show the ones where variables should be resolved properly?
Application: devenv.exe
Framework Version: v4.0.30319
Description: The process was terminated due to an unhandled exception.
Exception Info: System.ArgumentOutOfRangeException
   at System.Globalization.CompareInfo.Compare(System.String, Int32, Int32, System.String, Int32, Int32, System.Globalization.CompareOptions)
   at System.String.Compare(System.String, Int32, System.String, Int32, Int32)
   at Microsoft.PythonTools.Refactoring.LocationPreviewItem..ctor(Microsoft.PythonTools.Intellisense.VsProjectAnalyzer, Microsoft.PythonTools.Refactoring.FilePreviewItem, Microsoft.PythonTools.Intellisense.AnalysisLocation, Microsoft.PythonTools.Analysis.VariableType)
   at Microsoft.PythonTools.Refactoring.PreviewChangesEngine.CreatePreviewItems()
   at Microsoft.PythonTools.Refactoring.PreviewChangesEngine..ctor(System.IServiceProvider, Microsoft.PythonTools.Refactoring.IRenameVariableInput, System.String, Microsoft.PythonTools.Refactoring.RenameVariableRequest, System.String, System.String, Microsoft.PythonTools.Intellisense.VsProjectAnalyzer, System.Collections.Generic.IEnumerable`1<Microsoft.PythonTools.Intellisense.AnalysisVariable>)
   at Microsoft.PythonTools.Refactoring.VariableRenamer+<RenameVariable>d__4.MoveNext()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(System.Threading.Tasks.Task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(System.Threading.Tasks.Task)
   at Microsoft.PythonTools.Infrastructure.TaskExtensions+<DoNotWait>d__0.MoveNext()
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore+<>c.<ThrowAsync>b__6_0(System.Object)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(System.Delegate, System.Object, Int32)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(System.Object, System.Delegate, System.Object, Int32, System.Delegate)
   at System.Windows.Threading.DispatcherOperation.InvokeImpl()
   at System.Windows.Threading.DispatcherOperation.InvokeInSecurityContext(System.Object)
   at MS.Internal.CulturePreservingExecutionContext.CallbackWrapper(System.Object)
   at System.Threading.ExecutionContext.RunInternal(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
   at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
   at System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object)
   at MS.Internal.CulturePreservingExecutionContext.Run(MS.Internal.CulturePreservingExecutionContext, System.Threading.ContextCallback, System.Object)
   at System.Windows.Threading.DispatcherOperation.Invoke()
   at System.Windows.Threading.Dispatcher.ProcessQueue()
   at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr, Int32, IntPtr, IntPtr, Boolean ByRef)
   at MS.Win32.HwndWrapper.WndProc(IntPtr, Int32, IntPtr, IntPtr, Boolean ByRef)
   at MS.Win32.HwndSubclass.DispatcherCallbackOperation(System.Object)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(System.Delegate, System.Object, Int32)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(System.Object, System.Delegate, System.Object, Int32, System.Delegate)
   at System.Windows.Threading.Dispatcher.LegacyInvokeImpl(System.Windows.Threading.DispatcherPriority, System.TimeSpan, System.Delegate, System.Object, Int32)
   at MS.Win32.HwndSubclass.SubclassWndProc(IntPtr, Int32, IntPtr, IntPtr)

Interesting...It seems to work once if I click F12 on self.var1 in module1.py, if I click on self.func it seems to always work.   I have two very small test files.
PythonApplication1.py
class Parent(object):
    def __init__(self):
        self.var1  = 'test'
        self.var2  = 'data'

    def func1(self, s):
        print(s)

module1.py
from PythonApplication1 import Parent

class Child(Parent):

    def __init__(self):
        super().__init__()

    def test(self):
        self.func1(self.var1);

if __name__ == '__main__':
    c = Child()
    c.test()

Forgot to attach image...

Ah, I see. Attribute definition is a different code path in our analyzer (also different from method definitions), so it doesn't necessarily have to match top-level variable definition behavior. We'll take a look.

Clearing the milestone - the crash was urgent to fix, but the problem of attributes not associating properly between classes and subclasses can be deferred while we focus on higher priorities.
The cause of this issue is that ClassScope.AssignVariable does not check whether the variable has already been assigned in a superclass, so we end up with multiple distinct variables for the name instead of sharing it.

I often use the F12/GoToDefinition, to jump to function definitions - this works in VS2015 but has never worked for me in VS2012.  Tried VS2017 vs VS2015 on 2 Different machines, running the same project from SVN.  I don't get an error message in VS2017 for this, it just doesn't do anything.

@lcorrigan You may be hitting a different issue where if you've selected the whole word (and the cursor is right at the end) then we don't recognize it. This has been fixed for our next release. Putting the cursor in the middle of the name should let you go to definition.
If you're selecting in the middle of the name and it's still not working, and the name is not an inherited attribute, you've got a different problem from this one, so please create a new issue.

@zooba I see what you mean about the cursor vs the whole word - but only when I'm lucky to get that far.  I'm seeing a mixed bag of intellisense problems/responsiveness even after a re-install and cleaning / renewing the completion database.  I see many tickets for intellisense so I won't muddy the water here. Thanks.

I'm completely unable to Go To Definition or Find All References with VS 15.5.2 and PTVS v3.0.17320.1
Not variables, top-level methods, or class instance properties and methods.
Also, the drop-down list boxes at the top of the editor which used to contain the list of methods for quick navigation are also gone.
I get no error message or message box, simply nothing happens.
This used to work for me but I'm not sure if in the previous update or the one before it.

I was able to get things working again the usual 'uninstall everything you can and start over' method

I uninstalled Visual Studio (2015 and 2017 in my case). I recommend using Revo Uninstaller, to remove the remnants that VS doesn't remove.
Did the same to remove my IronPython and my always troublesome Anaconda.
Delete all folders it installs or saves anything in, including the PTVS plugin folder on in %APPDATA%
I ran Glary Utiltities, which found another slew of things that were not cleaned up even by Revo.
From there I did this:
Reboot and run VS2017 installer with Python support + .NET support an no other python-related anything.
Installed IronPython
Installed Miniconda (less bloat than full Anaconda) as a separate package.
Installed my few favorites for CPython like Matplotlib, pandas, numpy, pythonnet etc.
== Intellisense back to normal / go-to-whatever is working again. Even pandas lookup is fast again.

I think my 2 years of installing/uninstalling different VS's, PTVS's, Anaconda's created for a typically windows registry soup that makes the python intellisense database creator go haywire. My CPU had been running like mad every time I opened VS.
I also wonder if having CPython in the same large project as IronPython had anything to do with it.  Seems like things also degraded fast over time dating back to when I made that decision.


Best practice for Recovery Services Vault region location

I couldn't find this in the docs anywhere, but thought it might be good to add a note explaining best practice for region location of the vault.  You don't want the vault in the source region, presumably, so is target region the best practice?

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

ID: c714cc40-47f5-fb3c-51e0-f504c38ec937
Version Independent ID: 15da2499-61eb-ec11-53be-7d006cdd4b38
Content: Set up disaster recovery for Azure VMs to a secondary Azure region with Azure Site Recovery (Preview)
Content Source: articles/site-recovery/azure-to-azure-tutorial-enable-replication.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

@kpraxor Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Thanks for your feedback. We have no best practice other than saying it shouldn't be in the source region. No advantage to recommending target region.
We can close this. #please-close


Can't build on CentOS 7

OK, so I'm probably missing something trivial but I'm fighting this for 3 days now so I'd like to ask for help. I once managed to build tsduck and Dektec with no issue and then I failed with DVB tuners. So I tried again, installing new CentOS, upgraded kernel from 3.10 to 4.18 and was able to install DVB tuner but building tsduck just didn't work. So again I made a new CentOS install and I wanted to start with TSDuck, then Dektec and then upgrade to kernel 4.18 and install DVB tuner(s). But I'm getting stuck on step 1 - tsduck.
I installed CentOS 7 Minimal from a freshly downloaded ISO.
Then I installed yum group install "Development Tools".
Then I cloned tsduck from git.
Then I installed dependencies.
Then I ran make but it fails with these errors:
[root@localhost tsduck]# make
  [CXX] tsApplicationRecordingDescriptor.cpp
tsApplicationRecordingDescriptor.cpp: In member function 'virtual void ts::ApplicationRecordingDescriptor::deserialize(const ts::Descriptor&, const ts::DVBCharset* ':
tsApplicationRecordingDescriptor.cpp:135:52: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
         _is_valid = size >= 1 && size >= data[0] + 2;
                                                    ^
tsApplicationRecordingDescriptor.cpp:146:61: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
     _is_valid = _is_valid && size >= 1 && size >= 1 + data[0];
                                                             ^
tsApplicationRecordingDescriptor.cpp:155:61: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
     _is_valid = _is_valid && size >= 1 && size >= 1 + data[0];
                                                             ^
tsApplicationRecordingDescriptor.cpp: In static member function 'static void ts::ApplicationRecordingDescriptor::DisplayDescriptor(ts::TablesDisplay&, ts::DID, const uint8_t*, size_t, int, ts::TID, ts::PDS)':
tsApplicationRecordingDescriptor.cpp:197:52: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
             valid = size >= 1 && size >= data[0] + 2;
                                                    ^
tsApplicationRecordingDescriptor.cpp:210:53: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
     valid = valid && size >= 1 && size >= 1 + data[0];
                                                     ^
tsApplicationRecordingDescriptor.cpp:222:53: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare]
     valid = valid && size >= 1 && size >= 1 + data[0];
                                                     ^
cc1plus: all warnings being treated as errors
make[2]: *** [release-x86_64/tsApplicationRecordingDescriptor.o] Error 1
[root@localhost tsduck]#

gcc version is gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-28)
I tried some other versions (5, 7) but that didn't help much. So before messing up the system even more - anyone knows what's tsduck missing in my system right now? Why isn't it built?

I was able to build after removing -Werror from Makefile.common.
Not sure why I was able to compile in other CentOS installations and not in this Centos Minimal ISO without removing the flag but at least it worked out in the end. So if noone knows what's the reason for all this, we can close this issue.

Hi @rezna11,
This certainly not normal. TSDuck is meant to build on the latest version of CentOS since this is the only "production-certified" system for many companies.
My development policy is very strict to chase potential bugs in advance. So, I always activate the most paranoid set of warnings for the each compiler and turn them into errors. I never let warnings of any kind in the code.
Currently, I use three compilers: clang (LLVM) on macOS, gcc on Linux and MSVC on Windows. They have different marginal warning checks and can spot different issues. I can see that your warnings occur on a source file I added last night (so, you must have cloned the latest repository snapshot). As of today, it has been built on macOS only (which is my main development platform). It is possible that additional warnings can be triggered with another compiler. A file is fully certified only when it has been successfully tested on all platforms. I will check. Please retry after the next commit.
Sorry for the inconvenience. This can happen with fresh code snapshots.

Not sure why I was able to compile in other CentOS installations and not in this Centos Minimal ISO without removing the flag but at least it worked out in the end.

It worked because you cloned the repository before last night, before I added a new file that was tested on macOS only.

Great, it all makes sense. Thanks for looking into this.
By the way, CentOS, after updating kernel to 4.18, is so far the best linux I've ever tried... in just 3 days I was able to make it work with 4 different DVB tuners (DVB Sky S960, TBS 5922, 5925, Technotrend TT-4100) at the same time, together with TSDuck and Dektec (I tested DTA-111 and DTA-140).  I was never able to do this on Debian or Ubuntu. I'm not sure if upgrade to 4.18 was absolutely necessary but on 3.10, I could not build TBS drivers.

CentOS + kernel 4.18 ?
You should try Fedora, this is RedHat with bleeding edge techno and pre-built integration (no need to mess with kernel versions).


Nextjs@12 SSR throws unhandledRejection: TypeError: f is not a function

When creating a NextJS React app using<EMAIL_ADDRESS>with SSR the application throws unhandledRejection: TypeError: f is not a function after compiling when importing flagsmith and initializing it.

I'm seeing this as well in next@11:
freshline-marketplace  | TypeError: f is not a function
freshline-marketplace  |     at /app/node_modules/.pnpm/flagsmith@1.7.3/node_modules/flagsmith/isomorphic.js:1875:20
freshline-marketplace  |     at runMicrotasks (<anonymous>)
freshline-marketplace  |     at processTicksAndRejections (internal/process/task_queues.js:95:5)

this is where the error is originating from for me: https://github.com/Flagsmith/flagsmith-js-client/blob/167b883d1457612d08e4ad7046b3d80338bf7924/flagsmith-core.js#L279
this is because onError is undefined, and other uses guard with onError && onError(...)
the fix for now is to provide an onError function to flagsmith.init(...), e.g.:
await flagsmit.init({
  environmentID: "...",
  onError: (error) => {
    console.error(error);
  },
});

Setting onError resolved it for me 👍


podman run with pod and uidmap: mount mqueue not permitted

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
This is a follow-up issue to containers/crun#396, which was closed and marked as solved.
I came to the above mentioned issue after facing the very same problem it describes, in my case on an Ubuntu 18.04 host with runc runtime:
# The host is Ubuntu 18.04 on a physical laptop
matteo@matteo-laptop:~$ cat /etc/issue
Ubuntu 18.04.4 LTS

# Kernel is the latest available through HWE
matteo@matteo-laptop:~$ uname -r
5.3.0-61-generic

# I am trying to map container user 1000 to host user 3001 (this fails)
matteo@matteo-laptop:~$ sudo podman run \
    --rm \
    --uidmap 1000:3001:1 \
    "docker.io/library/ubuntu:18.04" whoami
Error: error creating libpod runtime: Container ID 0 cannot be mapped to a host ID

# Once the system users are mapped, too, it works
matteo@matteo-laptop:~$ sudo podman run \
    --rm \
    --uidmap 0:1000:1000 \
    --uidmap 1000:3001:1 \
    "docker.io/library/ubuntu:18.04" whoami
root

# However, trying to join a pod seems not to work (as per #396)
matteo@matteo-laptop:~$ sudo podman pod create \
    --name test_pod
matteo@matteo-laptop:~$ sudo podman run \
    --rm \
    --pod test_pod \
    --uidmap 0:1000:1000 \
    --uidmap 1000:3001:1 \
    "docker.io/library/ubuntu:18.04" whoami
Error: container_linux.go:349: starting container process caused "process_linux.go:449: container init caused \"rootfs_linux.go:58: mounting \\\"sysfs\\\" to rootfs \\\"...\\\" at \\\"/sys\\\" caused \\\"operation not permitted\\\"\"": OCI runtime permission denied error

Having found the solution in containers/crun#396, I installed crun from master (65e35c5).
Switching to the newly installed crun runtime and retrying the last command gives a different error:
# Repeating the previous command with crun
matteo@matteo-laptop:~$ sudo podman pod create \
    --runtime crun \
    --name test_pod2
matteo@matteo-laptop:~$ sudo podman run \
    --rm \
    --runtime crun \
    --pod test_pod2 \
    --uidmap 0:1000:1000 \
    --uidmap 1000:3001:1 \
    "docker.io/library/ubuntu:18.04" whoami
Error: mount `mqueue` to '/dev/mqueue': Operation not permitted: OCI runtime permission denied error

# Making the container privileged does not help
matteo@matteo-laptop:~$ sudo podman run \
    --rm \
    --privileged \
    --runtime crun \
    --pod test_pod2 \
    --uidmap 0:1000:1000 \
    --uidmap 1000:3001:1 \
    "docker.io/library/ubuntu:18.04" whoami
Error: mount `mqueue` to '/dev/mqueue': Operation not permitted: OCI runtime permission denied error

These sources might be related (although they deal with docker and runc):

moby/moby#20798
moby/moby#36674
redhat bugzilla 1401537

Steps to reproduce the issue:
See description.
Describe the results you received:
Various permission errors.
Describe the results you expected:
The commands should succeed without errors.
Additional information you deem important (e.g. issue happens only occasionally):
None.

We do not presently support using --uidmap with pods. We're looking into enabling it at the pod level, but when we do, it's likely going to be a flag to give a configuration for the whole pod - I don't think it is possible to sanely support containers with varying mappings inside the same pod.
Besides that, though, is the fact that we just allowed this to happen without error, and happily handed off a container configuration that would never work to the OCI runtime, which handed back a not particularly relevant error. We should catch this earlier and error at the container creation stage with an error that actually makes sense.

@rhatdan Good issue for an intern, I think

Thank you for the prompt response, @mheon. Indeed I was led to think that solving containers/crun#396 was meant to enable this feature. Seems like I will have to wait for further development, and come up with a different solution for my use case.

Let's let @QiWang19 Look at this one.

This is a kernel-level restriction - you can't join namespaces not owned by your user namespace, so we can't join the uidmapped containers to the pod-level network namespace (which existed before the new container and its user namespace). I believe that the work @giuseppe did in crun is to enable the entire pod to share a single user namespace (which Podman still does not have support for, but could add at this point).

crun 0.14 will permit doing it but it requires Linux 5.3+: https://github.com/containers/crun/pull/400

@mheon IIUC one would set the mapping on the pod, and the various containers would not be allowed to specify a different mapping. That would probably be enough in my case.
@giuseppe containers/crun#400 is the PR that solved the containers/crun#396 (mentioned in the OP). My kernel is 5.3.0-61 so the test seems to cover that: IIUC only the proper option in podman pod create is missing (as @mheon says).

@giuseppe Wow - the kernel is allowing that now?
Also, I only see proc/IPC listed - what about net? We're still broken if pods can't share network.

@giuseppe Wow - the kernel is allowing that now?

I am using a dirty trick of creating the mount before creating the user namespace and then I mount it once I am in the user+mount namespace :-)

latest crun version has this feature, so closing the issue.


Add buildah,kaniko,ko buildstrategy

Signed-off-by: wanjunlei<EMAIL_ADDRESS>Add buildah, kaniko, ko build strategy, user can build apps with Dockerfile using buildah or kaniko, and build go application with ko.

@wanjunlei Can we add some guides? it's a big enhancement

@wanjunlei Can we add some guides? it's a big enhancement

I will add some examples later.


(next-urql) - getStaticProps & getServerProps

Summary
Relates to: https://github.com/FormidableLabs/urql/issues/754
We need to implement a switch between getInitialProps, getStaticProps and getServerProps I've been browsing some examples but have not yet succeeded in finding a reasonable approach. This is a first step towards some common guidelines.
Set of changes

Optionally bind getInitialProps to withUrql

Next up: consolidate with https://github.com/FormidableLabs/urql/pull/816

I'm still not sure how to use that feature so I can start using urql client in getServerSideProps method. Can you provide some example?


Nans in generation data when no new data is available

Pull Request
There are two cases when generation data needs to be filled with Nans when the consumer runs:

[x] When the API returns no value for a given asset (DONE)
[ ] When the API returns a value that hasn't been updated since the last run (WIP)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration

[ ] Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

[ ] Yes

Checklist:

[ ] My code follows OCF's coding style guidelines
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

Moving work to india forecast app instead


What is the best way to disable a hand?

Describe the problem
I would like to disable one hand (the visualization joints/mesh + all interactions) dynamically in HL2. Is there existing way to achieve this?
Describe the solution you'd like
An API to the enable/disable certain hand type (left/right)

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visithttps://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


Field "firstAppearedInBlock" is not defined by type Asset_order_by.

https://github.com/input-output-hk/cardano-graphql/blob/55524476ca625756f45734ce4dbf261910d33278/packages/api-cardano-db-hasura/src/HasuraClient.ts#L299
{
    "errors": [
        {
            "message": "Field \"firstAppearedInBlock\" is not defined by type Asset_order_by.",
            "locations": [
                {
                    "line": 5,
                    "column": 29
                }
            ],
            "extensions": {
                "code": "GRAPHQL_VALIDATION_FAILED",
                "exception": {
                    "stacktrace": [
                        "GraphQLError: Field \"firstAppearedInBlock\" is not defined by type Asset_order_by.",
                        "    at ObjectField (/app/node_modules/graphql/validation/rules/ValuesOfCorrectType.js:103:29)",
                        "    at Object.enter (/app/node_modules/graphql/language/visitor.js:324:29)",
                        "    at Object.enter (/app/node_modules/graphql/language/visitor.js:375:25)",
                        "    at visit (/app/node_modules/graphql/language/visitor.js:242:26)",
                        "    at Object.validate (/app/node_modules/graphql/validation/validate.js:73:24)",
                        "    at validate (/app/node_modules/apollo-server-core/dist/requestPipeline.js:233:34)",
                        "    at Object.<anonymous> (/app/node_modules/apollo-server-core/dist/requestPipeline.js:119:42)",
                        "    at Generator.next (<anonymous>)",
                        "    at fulfilled (/app/node_modules/apollo-server-core/dist/requestPipeline.js:5:58)",
                        "    at runMicrotasks (<anonymous>)"
                    ]
                }
            }
        },
        {
            "message": "Cannot query field \"firstAppearedInBlock\" on type \"Asset\".",
            "locations": [
                {
                    "line": 7,
                    "column": 17
                }
            ],
            "extensions": {
                "code": "GRAPHQL_VALIDATION_FAILED",
                "exception": {
                    "stacktrace": [
                        "GraphQLError: Cannot query field \"firstAppearedInBlock\" on type \"Asset\".",
                        "    at Object.Field (/app/node_modules/graphql/validation/rules/FieldsOnCorrectType.js:53:31)",
                        "    at Object.enter (/app/node_modules/graphql/language/visitor.js:324:29)",
                        "    at Object.enter (/app/node_modules/graphql/language/visitor.js:375:25)",
                        "    at visit (/app/node_modules/graphql/language/visitor.js:242:26)",
                        "    at Object.validate (/app/node_modules/graphql/validation/validate.js:73:24)",
                        "    at validate (/app/node_modules/apollo-server-core/dist/requestPipeline.js:233:34)",
                        "    at Object.<anonymous> (/app/node_modules/apollo-server-core/dist/requestPipeline.js:119:42)",
                        "    at Generator.next (<anonymous>)",
                        "    at fulfilled (/app/node_modules/apollo-server-core/dist/requestPipeline.js:5:58)",
                        "    at runMicrotasks (<anonymous>)"
                    ]
                }
            }
        }
    ]
}

Seems to prevent cardano-graphql from booting, so might require a hotfix.
https://github.com/input-output-hk/cardano-graphql/blob/55524476ca625756f45734ce4dbf261910d33278/packages/api-cardano-db-hasura/schema.graphql#L388

I can't reproduce this. Are you using 7.0.0?


PLz mak3 default R00t Margin 0

Hi there,
Long term fan here.
Just wondering if you could make the default root margin 0 rather than 200px x 200px. Wouldn't that make more sense?
Yours,
TRIBE

Sorry, @neefrehman – where is this feature? I've been waiting for ages!


Yarn install React webpart for SP2016 Onprem fails

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
Install and run a React webpart for 'SharePoint 2016 onwards, including SharePoint Online' with Yarn v1.5.1 using Yeoman v2.0.1 with  generator @ microsoft/sharepoint v1.4.1 on windows .
Observed Behavior
After yarn install; webpart fails to run with gulp serve.
Steps to Reproduce
yo @ microsoft/sharepoint --skip-install
Select baseline package: SharePoint 2016 onwards, including SharePoint Online
Select framework: React
After scaffold is complete: yarn install
After install is complete: gulp serve
Build fails with errors, see attached file...
Attached file:
Yarn Spfx React onprem issue.txt

Does the same issue repro if you use npm 4?

It works when using npm.
And by the way, when using Yarn:
Creating a Online/latest webpart using React works.
Creating a SharePoint 2016 onwards, including SharePoint Online webpart using No JS framework works.
Creating a SharePoint 2016 onwards, including SharePoint Online webpart using Knockout works.
Creating a SharePoint 2016 onwards, including SharePoint Online webpart using React fails!

Technically we do not support Yarn usage and have not verified that process completely. There could be ways to get this work as it's being used by some community members, but we can't provide direct support on solving this issue.


Setting locale failed

I am having trouble with my dokku installation. Dokku can't seem to find the locale. It's set correctly when logging in via bash though, any ideas whats missing?
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
	LANGUAGE = (unset),
	LC_ALL = (unset),
	LC_CTYPE = "UTF-8",
	LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").

How did you install dokku and how/from where are you connecting it to?

I spun up a brand new Ubuntu 14.04 machine on AWS and ran the install script:
wget https://raw.githubusercontent.com/progrium/dokku/v0.4.1/bootstrap.sh
sudo DOKKU_TAG=v0.4.1 bash bootstrap.sh
then simply pushed my repo (node.js) to the server...

I just reran this on a new server and saw the following lines during installation:
    ubuntu@ip-172-31-xx-xx:~$ sudo DOKKU_TAG=v0.4.1 bash bootstrap.sh
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
	LANGUAGE = (unset),
	LC_ALL = (unset),
	LC_CTYPE = "UTF-8",
	LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
locale: Cannot set LC_CTYPE to default locale: No such file or directory
locale: Cannot set LC_ALL to default locale: No such file or directory

Maybe just add this to bootstrap.sh to ensure it's set?
export LANG=en_US.UTF-8
export LANGUAGE=$LANG

Thanks Michael, I tried that, doesn't help, though...
Now i think the problem is that the locale is not set inside the docker virtual environment... Because when I login via ssh to my aws instance:
Ben@Bens-MacBook-Pro ~/d/vagrant> vagrant ssh box1
ubuntu@box1:~$ echo $LANG
en_US.UTF-8

When I push the nodejs-example-app edited with this line:
app.get('/', function(request, response) {
response.send('$LANG: "' + process.env.LANG + '"')
})
It outputs nothing...! Any ideas how to set it?

Interesting. Can you try deploying this to heroku and see if it returns a value there?

Both are the same:
https://murmuring-everglades-1807.herokuapp.com
http://test.box1.colibriapps.com
I still see the LOCALE issue in the deploy log though :/ - Any other ideas?!
Full Log: https://gist.github.com/benmarten/fd2bba8c7a516db68459

Ok. Just wanted to verify we weren't doing something different than Heroku.
You can set these variables in your app using the config:set command.
root@dokku:~/dokku# dokku config:help
    config (<app>|--global), Display all global or app-specific config vars
    config:get (<app>|--global) KEY, Display a global or app-specific config value
    config:set (<app>|--global) KEY1=VALUE1 [KEY2=VALUE2 ...], Set one or more config vars
    config:unset (<app>|--global) KEY1 [KEY2 ...], Unset one or more config vars

Tried that, but also no luck...
I also tried adding a Dockerfile with that content:
Pull base image.
FROM ubuntu:14.04
Set the locale
RUN locale-gen en_US.UTF-8
ENV LANG en_US.UTF-8
ENV LANGUAGE en_US:en
ENV LC_ALL en_US.UTF-8

but the push is always rejected:
remote: Error response from daemon: Cannot kill container 7ee13107a5c7cdd0147c1b98a7a0a48c536721add19a7e0a8e04c070c9994ad6: Container 7ee13107a5c7cdd0147c1b98a7a0a48c536721add19a7e0a8e04c070c9994ad6 is not running
remote: Error: failed to kill containers: [7ee13107a5c7cdd0147c1b98a7a0a48c536721add19a7e0a8e04c070c9994ad6]
To dokku@box1.colibriapps.com:test
 ! [remote rejected] master -> master (pre-receive hook declined)
error: failed to push some refs to 'dokku@box1.colibriapps.com:test'

This might be also related to setting the timezone?!

Now I'm not clear on what we're solving here. The original question seemed to indicate an issue with the bootstrap.sh installation method.
To further troubleshoot, please include the information outlined here

Ok, so this turned out to be a linux issue. Found that out by running: ubuntu@box1:~$ locale
These answers fixed it for me:
http://askubuntu.com/a/229512
http://askubuntu.com/a/162394
Thanks for your help ;)

What exactly did you end up doing?

I ssh'ed into my instance and ran: locale which showed the same errors as mentioned in my first post.
To solve it, i put these lines into /etc/default/locale
LC_ALL=en_US.UTF-8
LANG=en_US.UTF-8

Then I ran:
sudo locale-gen en_US en_US.UTF-8
sudo dpkg-reconfigure locales

Note: You should always set server time to UTC, otherwise you're going to fall into a world of hurt when trying to query against data inserted from different timezones :)

Yeah I was apparently confused as your question was in regards to setting locale environment variables on your host and not in your app container.


presence of organism by "bacteriology" terms

This should resolve #387.
There are three EUPATH IDs removed (EUPATH_0010630, EUPATH_0015441, EUPATH_0015442), because they belong in the raw data template/module, and the open pull request for that module adds the terms with those IDs back to the ontology.

OK, this branch currently fixes the "bacteriology" terms. There are some older 'presence of ... by microscopy assay' terms that need to be resolved.

Some terms under 'presence of organism' in specimen' are unchanged; I am addressing them here: https://github.com/VEuPathDB-ontology/VEuPathDB-ontology/issues/414


Nilpointer during ocm download component

What happened:
Hi,
I tried to download a component version with the command
ocm download component eu.gcr.io/gardener-project/landscaper/examples//github.com/gardener/landscaper-examples/helm-deployer/helm-1:v0.1.0 -O ./archive

It failed with the following nilpointer error:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x320d6a4]

goroutine 1 [running]:
github.com/open-component-model/ocm/pkg/contexts/ocm/transfer.copyVersion({0x0, 0x0}, {0x4142e90, 0xc00062b700}, {0xc00036a220, 0x1, 0x1}, {0x4163a88, 0xc000c01800}, {0x4163920, ...}, ...)
	/Users/d032864/hub/git/ocm/ocm/pkg/contexts/ocm/transfer/transfer.go:289 +0x1084
github.com/open-component-model/ocm/pkg/contexts/ocm/transfer.CopyVersion({0x0, 0x0}, {0x4142e90, 0xc00062b700}, {0xc00036a220, 0x1, 0x1}, {0x4163a88?, 0xc000c01800?}, {0x4163920, ...}, ...)
	/Users/d032864/hub/git/ocm/ocm/pkg/contexts/ocm/transfer/transfer.go:218 +0xf6
github.com/open-component-model/ocm/cmds/ocm/commands/ocmcmds/components/download.(*action).Save(0xc00075ae80, 0xc000033340, {0x7ff7bfeff5e6, 0xc})
	/Users/d032864/hub/git/ocm/ocm/cmds/ocm/commands/ocmcmds/components/download/cmd.go:147 +0x4cd
github.com/open-component-model/ocm/cmds/ocm/commands/ocmcmds/components/download.(*action).Out(0xc00075ae80)
	/Users/d032864/hub/git/ocm/ocm/cmds/ocm/commands/ocmcmds/components/download/cmd.go:113 +0x190
github.com/open-component-model/ocm/cmds/ocm/pkg/utils.HandleOutput({0x411ad90, 0xc00075ae80}, {0x411c9a0, 0xc000a37860}, {0xc0004f5890?, 0x1, 0x0?})
	/Users/d032864/hub/git/ocm/ocm/cmds/ocm/pkg/utils/handling.go:102 +0x2a6
github.com/open-component-model/ocm/cmds/ocm/commands/ocmcmds/components/download.(*Command).Run(0xc0007b79c0)
	/Users/d032864/hub/git/ocm/ocm/cmds/ocm/commands/ocmcmds/components/download/cmd.go:83 +0x328
github.com/open-component-model/ocm/cmds/ocm/pkg/utils.SetupCommand.func1(0xc0001bf600?, {0xc000d84570, 0x1, 0x3})
	/Users/d032864/hub/git/ocm/ocm/cmds/ocm/pkg/utils/command.go:194 +0x12e
github.com/spf13/cobra.(*Command).execute(0xc000c4a600, {0xc0001121b0, 0x3, 0x3})
	/Users/d032864/go/pkg/mod/github.com/open-component-model/cobra@v0.0.0-20230329075350-b1fd876abfb9/command.go:943 +0x87c
github.com/spf13/cobra.(*Command).ExecuteC(0xc0007dcf00)
	/Users/d032864/go/pkg/mod/github.com/open-component-model/cobra@v0.0.0-20230329075350-b1fd876abfb9/command.go:1071 +0x3a5
github.com/spf13/cobra.(*Command).Execute(0x4155350?)
	/Users/d032864/go/pkg/mod/github.com/open-component-model/cobra@v0.0.0-20230329075350-b1fd876abfb9/command.go:995 +0x13
main.main()
	/Users/d032864/hub/git/ocm/ocm/cmds/ocm/main.go:21 +0xdc

Actually, the component descriptor was downloaded, but the blobs directory was empty.
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Just try to run the command. The component version still exists.
Anything else we need to know:
Here you can find the component constructor that I used to create the component version:
component-constructor.yaml
Environment:
OCM version (latest commit of the main branch):
{"Major":"0", "Minor":"9", "Patch":"0", "PreRelease":"dev", "Meta":"09e73686933678cc5ec9335d8b6e8c10fc3a7021", "GitVersion":"0.9.0-dev+09e73686933678cc5ec9335d8b6e8c10fc3a7021", "GitCommit":"09e73686933678cc5ec9335d8b6e8c10fc3a7021", "GitTreeState":"dirty", "BuildDate":"2024-03-22T07:38:21+0000", "GoVersion":"go1.21.5", "Compiler":"gc", "Platform":"darwin/amd64"}

@mandelsoft @hilmarf Regression?

@robertgraeff Which version of OCM are you using?


Settings screen: Alignment issue.

Alignment does not look correct

It is according to the XD design screen.


Storacha / Akave integration

Integrate on a separate branch with Storacha / Akave for file-storage example... an initial idea is with NFTs?
Implement basic NFT contract containing URI field
Allow user to upload large files onto Storacha / Akave, using the output as URI
Testing complete workflow in calibration.
Proper ABI/artifact generation to be used in frontend.

for storacha and akave (need to find out more here but they are putting a bounty for bangkok, so definitely something available), I see them as pluggable options for storage onramps within the starter kit. will let you decide on use case

We can have a different PR for lighthouse integration.
Lighthouse supports natively encryption which is kind of cool !

For storacha, does this require a credit card on testnet still?

For storacha, does this require a credit card on testnet still?

Yes Storacha is not depending on a blockchain.
Users/devs should include their credit card but they can have first GBs for free

@xBalbinus Created the Storacha-nfts branch here https://github.com/FIL-Builders/fil-frame/tree/storacha-nfts
If this satisfies the issue let's close it!!!


UI idea for month and year selection

We can have a switch or a selection kind of a thing on the left hand side on the current date view. Where the user can select the view he/she wants something like the Samsung calendar(Attaching screenshots for the same). The date / month / year selection will become extremely easy and the UX is gonna be brilliant.
Let me know if this works for you. I really love you library though

@fitut nice calendar.
For year and month selection, the UI will be OK with the style as follows:

We could change the title into year. User can swipe to select years.
And change the date selection into months.

Hi, this is my customized with added year. Year's range can be set by yourself.

Hi, this is my customized with added year. Year's range can be set by yourself.

plz share code brooo


Strict Mode, Cross-Platform Compatibility, and JSDoc Documentation

Include strict: true, as it is considered a best practice. This modification will necessitate some updates in the code but will substantially reduce the occurrence of bugs.
Initiate a discussion on how the library should behave on Windows versus macOS concerning file_path. Additionally, provide a fix for unit tests (as they currently fail on Windows due to path library differences)."
Add Jsdoc documentation for important functions in the library

I will not do strict Mode for now as it requires a lot of changes....


Molecule replace latest jinja2 2.10 to <IP_ADDRESS> buggy version which causes template issue in ansible with namespace

Issue Type

Bug report

Molecule and Ansible details
ansible --version
ansible <IP_ADDRESS>
config file = /etc/ansible/ansible.cfg
configured module search path = [u'/root/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
ansible python module location = /usr/local/lib/python2.7/dist-packages/ansible
executable location = /usr/bin/ansible
python version = 2.7.14 (default, Sep 23 2017, 22:06:14) [GCC 7.2.0]
molecule --version
molecule, version 2.10.1
Molecule installation method (one of):

pip

Ansible installation method (one of):

pip

Desired Behavior
do not replace jinja2 2.2.10 to <IP_ADDRESS>
Actual Behaviour (Bug report only)
root@OED-EX-DIG-L:~# pip install molecule --upgrade
Requirement already up-to-date: molecule in /usr/local/lib/python2.7/dist-packages
Requirement already up-to-date: pbr==3.0.1 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: python-gilt==1.1.0 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: colorama==0.3.7 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: yamllint==1.11.0 in /usr/local/lib/python2.7/dist-packages (from molecule)
Collecting Jinja2==2.9.6 (from molecule)
Using cached Jinja2-2.9.6-py2.py3-none-any.whl
Requirement already up-to-date: pexpect==4.2.1 in /usr/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: click==6.7 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: tree-format==0.1.2 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: cookiecutter==1.5.1 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: tabulate==0.7.7 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: PyYAML==3.12 in /usr/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: ansible-lint==3.4.19 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: testinfra==1.7.1 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: flake8==3.3.0 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: click-completion==0.2.1 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: psutil==5.2.2; sys_platform != "win32" and sys_platform != "cygwin" in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: sh==1.12.14 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: anyconfig==0.9.1 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: marshmallow==2.13.5 in /usr/local/lib/python2.7/dist-packages (from molecule)
Requirement already up-to-date: fasteners in /usr/local/lib/python2.7/dist-packages (from python-gilt==1.1.0->molecule)
Requirement already up-to-date: git-url-parse in /usr/local/lib/python2.7/dist-packages (from python-gilt==1.1.0->molecule)
Requirement already up-to-date: pathspec>=0.5.3 in /usr/local/lib/python2.7/dist-packages (from yamllint==1.11.0->molecule)
Requirement already up-to-date: MarkupSafe>=0.23 in /usr/lib/python2.7/dist-packages (from Jinja2==2.9.6->molecule)
Requirement already up-to-date: whichcraft>=0.4.0 in /usr/local/lib/python2.7/dist-packages (from cookiecutter==1.5.1->molecule)
Requirement already up-to-date: binaryornot>=0.2.0 in /usr/local/lib/python2.7/dist-packages (from cookiecutter==1.5.1->molecule)
Requirement already up-to-date: future>=0.15.2 in /usr/local/lib/python2.7/dist-packages (from cookiecutter==1.5.1->molecule)
Requirement already up-to-date: poyo>=0.1.0 in /usr/local/lib/python2.7/dist-packages (from cookiecutter==1.5.1->molecule)
Requirement already up-to-date: jinja2-time>=0.1.0 in /usr/local/lib/python2.7/dist-packages (from cookiecutter==1.5.1->molecule)
Requirement already up-to-date: ansible in /usr/local/lib/python2.7/dist-packages (from ansible-lint==3.4.19->molecule)
Requirement already up-to-date: six in /usr/local/lib/python2.7/dist-packages (from ansible-lint==3.4.19->molecule)
Requirement already up-to-date: pytest!=3.0.2 in /usr/local/lib/python2.7/dist-packages (from testinfra==1.7.1->molecule)
Requirement already up-to-date: enum34; python_version < "3.4" in /usr/lib/python2.7/dist-packages (from flake8==3.3.0->molecule)
Requirement already up-to-date: configparser; python_version < "3.2" in /usr/local/lib/python2.7/dist-packages (from flake8==3.3.0->molecule)
Requirement already up-to-date: pyflakes<1.6.0,>=1.5.0 in /usr/local/lib/python2.7/dist-packages (from flake8==3.3.0->molecule)
Requirement already up-to-date: mccabe<0.7.0,>=0.6.0 in /usr/local/lib/python2.7/dist-packages (from flake8==3.3.0->molecule)
Requirement already up-to-date: pycodestyle<2.4.0,>=2.0.0 in /usr/local/lib/python2.7/dist-packages (from flake8==3.3.0->molecule)
Requirement already up-to-date: monotonic>=0.1 in /usr/local/lib/python2.7/dist-packages (from fasteners->python-gilt==1.1.0->molecule)
Requirement already up-to-date: chardet>=3.0.2 in /usr/local/lib/python2.7/dist-packages (from binaryornot>=0.2.0->cookiecutter==1.5.1->molecule)
Requirement already up-to-date: arrow in /usr/local/lib/python2.7/dist-packages (from jinja2-time>=0.1.0->cookiecutter==1.5.1->molecule)
Requirement already up-to-date: cryptography in /usr/local/lib/python2.7/dist-packages (from ansible->ansible-lint==3.4.19->molecule)
Requirement already up-to-date: setuptools in /usr/local/lib/python2.7/dist-packages (from ansible->ansible-lint==3.4.19->molecule)
Requirement already up-to-date: paramiko in /usr/local/lib/python2.7/dist-packages (from ansible->ansible-lint==3.4.19->molecule)
Requirement already up-to-date: attrs>=17.2.0 in /usr/local/lib/python2.7/dist-packages (from pytest!=3.0.2->testinfra==1.7.1->molecule)
Requirement already up-to-date: py>=1.5.0 in /usr/local/lib/python2.7/dist-packages (from pytest!=3.0.2->testinfra==1.7.1->molecule)
Requirement already up-to-date: funcsigs; python_version < "3.0" in /usr/local/lib/python2.7/dist-packages (from pytest!=3.0.2->testinfra==1.7.1->molecule)
Requirement already up-to-date: pluggy<0.7,>=0.5 in /usr/local/lib/python2.7/dist-packages (from pytest!=3.0.2->testinfra==1.7.1->molecule)
Requirement already up-to-date: backports.functools-lru-cache>=1.2.1; python_version == "2.7" in /usr/local/lib/python2.7/dist-packages (from arrow->jinja2-time>=0.1.0->cookiecutter==1.5.1->molecule)
Requirement already up-to-date: python-dateutil in /usr/local/lib/python2.7/dist-packages (from arrow->jinja2-time>=0.1.0->cookiecutter==1.5.1->molecule)
Requirement already up-to-date: cffi>=1.7; platform_python_implementation != "PyPy" in /usr/local/lib/python2.7/dist-packages (from cryptography->ansible->ansible-lint==3.4.19->molecule)
Requirement already up-to-date: idna>=2.1 in /usr/local/lib/python2.7/dist-packages (from cryptography->ansible->ansible-lint==3.4.19->molecule)
Requirement already up-to-date: asn1crypto>=0.21.0 in /usr/local/lib/python2.7/dist-packages (from cryptography->ansible->ansible-lint==3.4.19->molecule)
Requirement already up-to-date: ipaddress; python_version < "3" in /usr/local/lib/python2.7/dist-packages (from cryptography->ansible->ansible-lint==3.4.19->molecule)
Requirement already up-to-date: pynacl>=1.0.1 in /usr/local/lib/python2.7/dist-packages (from paramiko->ansible->ansible-lint==3.4.19->molecule)
Requirement already up-to-date: pyasn1>=0.1.7 in /usr/local/lib/python2.7/dist-packages (from paramiko->ansible->ansible-lint==3.4.19->molecule)
Requirement already up-to-date: bcrypt>=3.1.3 in /usr/local/lib/python2.7/dist-packages (from paramiko->ansible->ansible-lint==3.4.19->molecule)
Requirement already up-to-date: pycparser in /usr/local/lib/python2.7/dist-packages (from cffi>=1.7; platform_python_implementation != "PyPy"->cryptography->ansible->ansible-lint==3.4.19->molecule)
Installing collected packages: Jinja2
Found existing installation: Jinja2 2.10
Uninstalling Jinja2-2.10:
Successfully uninstalled Jinja2-2.10
Successfully installed Jinja2-2.9.6
Current workaround solution
after pip install molecule --upgrade you have to run pip install ansible --upgrade to fix the jinja2 version

Do you have the Ansible bug where this version of jinja2 is an issue?

I can reproduce it any time and provide that info if it helps. Should i paste an example here?

Playbook:

name: jinja2 test
hosts: all
gather_facts: no
tasks:

name: create template
local_action:
module: template
src: "{{ playbook_dir }}/test.j2"
dest: "{{ playbook_dir }}/test.template"
run_once: yes

template (test2.j2):
{% set ns = namespace(_something = true) %}
anything
You can see the different result


Symbol at cursor functionality for type inits is incomplete

It seems the functionality of the symbol at cursor API for type inits is not entirely correct. If the cursor is at an explicitly stated type reference, it should return the symbol for the init method. The current behaviour doesn't consider the init method at all. Even if the init method is considered, there's the case where there may not be a user-defined init method.

As per the latests discussions, we can always return the type-descriptor symbol.
User of the symbol can decide what information to use - Either can directly use the type information or can use init function's information if available.

Checked this. Currently this doesn't seem to work properly. Will address this when introducing class symbols


Monetary Amount added as a primitive type

Monetary Amount added as a primitive type, right now Monetary Amount ends up opening all the internal classes on UI, making it virtually impossible to use.

Well not if the PR breaks all the tests!  See if you can address the tests and we can look.

Its major minor version, are you guys on Java 7, seems to be depend on java 8

Yes Java 7. Next version Java 8. It can be done with class names though

Will give you a java 7 fix

Build has passed now, after adding java-money-bp

I need to pull it back, will contribute again, this may lead to certain violations.

I can check in this fix/feature in from my company portal, are you ready to accept this PR or you guys have different plans to implement this?

I'm confused :)  Would be happy to add the support when builds pass and your company gives you permission to contribute, assuming it follows the framework style and doesn't break things.


Mark exception as unused

If compiled without RTTI e is unused and issues a warning. Casting to void marks it as intentionally unused.
Should we create eg. an "unused()" for improved documentation (cast-to-void might not be known to everyone)?

Coverage remained the same at 99.336% when pulling f20a956bd2141b707cb3a4457ac6db30f1830d80 on offa:norrtti_unused_excpetion into 8d2e6fa39f83fcb7acb69c660c2360f422058f34 on cpputest:master.

There's this but it only works in the parameter list and only for a couple compilers.


Allow URL params in $url() RPC

What is the feature you are proposing?
I love the $url() feature.
It just that I don't find the returned mergePath very useful.
Can it use the replaceUrlParam too if the url has input params ?
I'm thinking about some like this :
const router = new Hono()
  .get(
    "/foo/:id/bar",
    zValidator("param", z.object({ id: z.string() })),
    function ({ text, req }) {
	  const { id } = req.valid("params");
	  // [...]
      return text("OK");
    },
  );

// [...]

app_hc.foo[":id"].bar.$url({ param: { id: "1" } });
// >> http://localhost:3000/foo/1/bar

Hi @douglasduteil
Sorry for the delayed response, and thank you for the suggestion. This is cool!
I've implemented this feature in PR #1885. If you can, could review or see it?

Closed with #1885


[portaudio]: fix Linux build

Wrap some Win specific stuff in an if (WIN32)
Fixes #4526

Hi @tzneal
Thanks for contributing to vcpkg!
Thanks to your PR, portaudio now builds succesfully in Linux and MacOS.
A quick note, when modifying a portfile please update the corresponding CONTROL version.


Unexpected behaviour of open-choice 'other' option when using tabs

Describe the bug
Within a Questionnaire organised by tabs, and with a Questionnaire item of type open-choice, with either repeats = true or false, the free text entry box for 'other' is populated by the value of one of the choice options after first navigating to a different tab and going back to the tab with the open choice question.
To Reproduce
This is evident in the Aboriginal Torres Strait Islander Health Check questionnaire found on https://www.smartforms.io/standalone

open the Aboriginal Torres Strait Islander Health Check form

go to the tab 'Ear health and hearing'

under otoscopy findings for either left ear or right ear, check one or more of the choices, leaving 'other, please specify' empty and unchecked; eg:

go to another tab, like 'mood' or 'gambling' etc

go back to the tab 'Ear health and hearing'

notice that the check box 'other' is now checked and that the entry box is prepopulated with one of the selections made in step 3; eg:

Expected behavior
When navigating back to the open-choice question from another tab, its state regarding previously answered options, should be retained. And that the 'other' option should not be checked/prepopulated

Hey Sean, I wonder if you could have another look at this one please. The behaviour described in the OP is fixed (thanks) however there is a new behaviour with the 'other' box that is unexpected.
The new bit is (also found on the CSIRO Aboriginal Torres Strait Islander Health Check questionnaire found on https://www.smartforms.io/standalone):

open the Aboriginal Torres Strait Islander Health Check form

go to the tab 'Ear health and hearing'

under otoscopy findings for either left ear or right ear, check 'other, please specify' and add some text

switch to another tab then go back to 'Ear health and hearing', noting that the previously entered text alongside 'other' is gone and 'other' is unchecked

I can create a new ticket if you prefer

Thanks for catching this! I'll get it resolved by the end of the week.

Hi Rob, fixed it. Can you check if it's behaving as expected on your end?

Perfect, thanks Sean


typo

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

Thanks for the contrib. Please take care of the CLA and bump the chart version and we should be GTG. 👍