Global parameter file

This implements an idea for:

a global parameter file: this defines the different programs, but also global parameters that are common among all simulations, this could (and probably should) include working directory, etc. All simulation ensembles need this global parameter file. Stylistically maybe should supply path to this global parameter file as an argument or use argparse functions like other command line programs.

different parameter file formats. It is more secure to read files through json or yaml instead of executing text files that contain python code. Presently formats are specified with extension - but that is a security hole that should be closed probably eventually!

multiple parameter files for each simulation. The concept is to have one default set of parameters that is always up to date - then subsets of the parameters can be read 'on top' of that base set. This allows old parameter files to always be read on top of a newer model that contains lots more parameters. It also allows assembly of programs like lego, with little re-usable pieces that only specify certain parts of the simulations. This would make grabbing components from the method library easier as they are just added in as another ingredient.

This one is open for discussion to hone in the right approach.

Please take a look:
@KeeganShaw-GIS @tarcadius @AkaMrmuffin Soroush
You will need to install pyyaml with either requirements.txt or pipfile
A bit of a parameter re-org is warranted with this project - I have not done this - I think it is probably good idea to put things like number of iterations as a global parameter so one can re-use the program definitions and mix and match them.
I've also added a third hierarchy of so called 'method files', which are the same deal, definitions of specific methods that should be re-usable - pop them in whatever program you are making by adding them to the method_file key in program definitions.
I've also set things up to allow absolute paths - so you can have a separate folder of method and program definitions to yank canned definitions from.
I have not set up a way for program definitions to modify global parameters - this feels a bit weird, but I suppose it could be coded?
Please take a look and see if something like this solves the set of issues here.

Hi @tbarchyn  - this looks great, I'm super excited about it.
Some of how this works is over my head, but I get the general idea here I think.
Everything runs fine on my computer. The outputs are all wonky though (e.g., emissions = zero for regulatory program). Baseline seems fine?
My one general concern/comment about the structure is that the default parameter file is built around program names that are kind of made up (or could be considered examples). I guess I would point specifically to P_alt - really just some random aircraft program. There's nothing special about it and no reason really to make it a default. I wonder if we should just have default programs for each type of technology that are simple as they come? P_fixed, P_OGI, P_aircraft, etc. etc.? But then when you run LDAR-Sim would you have to name your program one of these things? What if you use multiple different types of technology?
This is the main thing I don't understand - otherwise seems like a great idea but I'll let @KeeganShaw-GIS  and Soroush (invite to repo pending) comment on the technical details.

I think you can name your programs anything you want? Same with methods? Any number of methods, any number of programs - I'm not sure if this locks you into anything. Just keep adding methods to your programs, or programs to your simulation.
The idea of a default set of parameters is kind of a different concept - more related to the idea of making a test simulation set that has a fixed seed and always produces a known result that can be used to test.

This is great! I like the ability to use the .yaml / text / json format. Hopefully we can add Excel files to the mix soon.

@tarcadius @KeeganShaw-GIS
Can I ask for help in identifying what parameters should be 'global parameters' to the simulation?
The following parameters were originally in ldar_sim_main.py, originally mined from the P_ref program, and are set to be global parameters, but there probably are more that should be globals as well, yet are in program definitions?
 'reference_program': 'P_ref',
 'simulation_programs': ['P_ref', 'P_base'],
 'wd': '..//inputs_template//',
 'output_directory': '..//outputs//',
 'n_processes':  None,
 'print_from_simulations': True,
 'n_simulations': 2,
 'spin_up': 0,
 'write_data':  True,

Ideally there would be nothing in a program definition that affects the actual simulation.
I'm thinking of bringing the following out to be globals:

weather
infrastructure
leak generation stuff? (lpr, etc.) (possibly leave in program but call a program something different?)
flags to make plots maybe?

Thoughts welcome here - I'm in favour of making this as conceptually clean as possible now to help new onboards to the model - but perhaps it makes sense to just leave as it is. Thanks.

Hey @tbarchyn sorry for the delay on this ... need to be better at checking Github (or my gmail).
Your list looks good. I would also include (however several of the below are slated to change):

timesteps
start_year (though I want to change this and timesteps to "start_date" and "end_date"
UTC_offset @KeeganShaw-GIS is this a permanent param? What happens if we are simulating across multiple time zones?
weather_file
infrastructure
leak_file
count_file
vent_file
time_offsite
...

I'm going through the list and I feel like everything should be global. If it's global it can still be changed on a program level if the user chooses to right? So why not make everything that isn't method-specific global?
Maybe I'm not understanding and we need to chat through this!

To do for Tom:

Add globals from Thomas to global section of parameter file.
Allow programs to assign globals somehow (this probably needs a warning so people know what they are doing). Preference is to default to globally assigned globals, if those are missing the globals are mined from a program file. The situation where program files can re-assign previously specified globals feels dangerous because folks might want a library of programs, and it is not good to have things like timesteps in those files inadvertently modifying the simulation number timesteps you specified. Or perhaps allow with a warning?
Build translation from older parameter files to create new, compliant parameter files. This would actually simplify things as there is a specific translation function.
Documentation - explain everything in detail.
Deal with method assignment - it is not consistent with programs right now and should probably be consistent.

@tbarchyn In the meeting we discussed the idea of an Input Mapper where all inputs are handled, converted to variables used within LDAR-sim regardless of input type.  This allows us to not change LDAR-Sim code because input format changes, so we can maintain reverse compatibility and some options . I am going to create an issue for building this, and we should move the discussion there.

Closing - refer to newer, better, faster, stronger, more awesome iteration of the same initiative in #77


Block Machine: Handle case where number of rows is just enough

Before this PR, we used to panic if the number of rows in a block machine was just enough to fit the needed number of blocks. This PR fixes it.

Test fails


nextcloud: Argument #2 ($key) cannot be empty in file...

Describe the bug
This is another bug in Nextcloud I have hit on the server. I can't access the "Security" tab in the admin settings and I am not able to use my Nextcloud as an OAuth provider anymore.
The log looks something like this:
{
    "reqId": "********************",
    "level": 3,
    "time": "2023-09-05T22:09:46+00:00",
    "remoteAddr": "*************************************",
    "user": "*****",
    "app": "index",
    "method": "GET",
    "url": "/settings/admin/security",
    "message": "{\"Exception\":\"Exception\",\"Message\":\"hash_hkdf(): Argument #2 ($key) cannot be empty in file /nix/store/7lihj0wc690gzd7nlvhazp5xqz981jbb-nextcloud-27.0.2/lib/private/Security/Crypto.php line 160\", <... A LOT more lines here, I am a little afraid to leak any sensitive information>}",
    "userAgent": "Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/116.0",
    "version": "<IP_ADDRESS>"
}

There is an upstream issue I think is related to this problem: #34012. There is a proposal for a patch, but no verification or update on implementation of the patch in the upstream project.
I am not sure if there is anything that can be done in nixpkgs right now. If that is the case this is probably more of a tracking issue until upstream is fixed.
Steps To Reproduce
I am not sure whether or how this is reproducible.
Expected behavior
I should be able to access the "Security" settings in my Nextcloud.
Notify maintainers
@schneefux @bachp @globin @Ma27
Metadata
 - system: `"aarch64-linux"`
 - host os: `Linux 6.1.47, NixOS, 23.11 (Tapir), 23.11.20230822.a2eca34`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.0`
 - nixpkgs: `/etc/channels/nixpkgs`

Hi this issue still persists here and upstream (nextcloud/server#34012). Sadly there does not seem to be an intent to fix this upstream, and I am not quite sure how to fix this in NixOS myself...

And you're nto affected by the following case? https://github.com/nextcloud/server/issues/34012#issuecomment-1807093454
Regarding the patch: since it seems completely untested (from what the author says) and isn't even a proper patch, but the method with the patch applied, I don't think that we should apply this to nixpkgs.
You however can like this:
nextcloud.overrideAttrs ({ patches ? [], ... }: {
  patches = patches ++ [ ./your-fix.patch ];
})

And you're nto affected by the following case? nextcloud/server#34012 (comment)

I though I was, but changing it didn't fix the issue. I would be fine with replacing it, discarding the old secret store altogether, but I can't figure out how to do this.

You however can like this:

Thanks! I'll try it and report back...

So the thing is, this is clearly an upstream issue and with all the information I got so far, this has nothing to do with how Nextcloud is packaged here. That being said, I don't see anything actionable here which is why I'd like to close this if you agree (and I'm not missing something).
If you need help with e.g. applying the patch to your system, let me know!

I have not tried applying the patch (in part, because I don't want to mess up my installation even worse that it is now), but somehow this problem also occurs when upgrading, preventing me from updating to the latest 24.05 version. I don't think there is anything that can be done about it, just wanted to let you guys know.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/rant-on-nextcloud/54016/7


Measuring latency hyper async (based on tokio-rs)

Can you provide an example of how to use this to measure latency for hyper (0.0.11 async - based on tokio-rs) ?

Hi @arpitsr
Here's an example for a Hyper client.
First, we have to initialize tokio, hyper, and tacho. Metrics (in this case, latency) should be created during initialization if possible.
let core = tokio_core::Core::new().unwrap();
let (metrics, reporter) = tacho::new();
let latency = metrics.prefixed("client").timer_ms("latency_ms");
let client = hyper::Client::new(&core.handle());

Then, wrap the things you want to measure with latency.time to produce a Timed future that records latency.
latency.time(client.get(uri)).map(|rsp| {
  ...
})

To integrate this with a hyper server, you would just wrap the response future with latency.time.


I cannot connect with codewars.

i am using google chrome, and when i click in the site my browser crashes, how could i fix this?

I have the same problem!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Same problem...

same problem
i try firefox and IE as well - same result


fix: reduce cls caused by nav

Description
Currently, all header height styles are located in block/header/header.css, which is lazy-loaded. This generates significant CLS when the loaded content in main is shoved down to accommodate the newly loaded stylesheets impacting header and nav. Proposing to move header nav styles impacting height into styles.css so circumvent this issue. Also proposing the removal of duplicative styles.

Before vs. After

Changed

header nav and main height set in styles.css, rather than in the lazy-loaded header block

Removed

duplicative css

Test URLs

Before (Changes from main): main--merative2--hlxsites.hlx.page
After (Changes from this PR): nav-cls--merative2--hlxsites.hlx.page
Before (Changes from main): main--merative2--hlxsites.hlx.page/clinical-development
After (Changes from this PR): nav-cls--merative2--hlxsites.hlx.page/clinical-development
Before (Changes from main): main--merative2--hlxsites.hlx.page/blog
After (Changes from this PR): nav-cls--merative2--hlxsites.hlx.page/blog

Looks good to me. Thanks @fkakatie !
@helms-charity please take a look and merge when you have a moment.


Implement keyed messages

This PR should implement keyed messaging on the PulsarTemplate (#26).
A detailed description follows.

Thank you for the feedback, I added the type to the message builder and updated the docs to reflect the changes. If it looks good to you I'll squash and rebase so you can merge :)

Looks good @alpreu . I went ahead and squash/rebase/merge. Thanks again for contribution!


Successfully creating a debian package.

Note: the project folder must be renamed to passphrase-identity.
Go with:
make dist
mv passphrase-identity-1.0.0.tar.gz ../passphrase-identity_1.0.0.orig.tar.gz
dpkg-buildpackage [-us -uc]

debian package at https://mentors.debian.net/package/passphrase-identity.

LGTM.


Bugfix serbian translation placeholder

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

| Fixed issues  | comma-separated list of tickets # fixed by the PR, if any

Thanks!


Update styling document type meta fields

Few styling changes to make for the meta fields content app for the document type:

See if we can have the "Enable seo settings" take less space. Should check if Umbraco has anything like this in place.
See if we can improve the UI for the "Open settings" button. Possibly moving all the sidepanel logic to the main panel. Just have to make sure that the main panel doesn't get too cluttered with information for the user.
See if we can use markup for the field descriptions. Might help the developer/user with setting up the fields. Target group is developers, but it would be nice to provide some extra information anyway.

@patrickdemooij9 I've been hacking around with a potential alternative.
Of course, it's still a work-in-progress, but let me know what you think of this...
The default state, a <umb-box-header> with a <umb-toggle> and label...

When enabled, the toggle's label is hidden (I did have it saying "Disable SEO settings?" but that felt odd), and the tabs are displayed in the <umb-box-header> title area.

@leekelleher
I think it does look very good. I think aligning it on the right is the right call here and I don't think there will ever be that many tabs that it'll cause an issue here.
It does feel a bit odd in the way that all configuration actions for document types are on the left while this one is on the right, but I wouldn't worry about it that much. I think it is definitely an improvement on the current implementation.
There also isn't really any way Umbraco currently does this. There is the "Enable List view", but that of course doesn't have tabs so it can be used a bit easier here.

Coming back to it a day later, the "Enable SEO settings?" being right aligned still isn't feeling right to me.
I'm happy to send over a pull-request, if you want the code - but I can't say that I'm completely happy with my progress at the moment.

We can just put it on the "possible solutions" pile for now. Will see if I can also come up with some solutions later today so we have some more solutions to compare with

I tried out a few things:
1:

2:

3:

4:

I think buttons might even be nicer here than a random checkbox, but still not quite sure. I do wonder if tabs are the way to go or if I should have different boxes for each setting. Just worried it will be a lot of content on one page then.

I'm liking number 3, the button does make a difference to it being a toggle.
I thought the same about swapping tabs for boxes/fieldsets going down the page. I don't think it'd feel too long for someone configuring the doctype.


RTL problem when too many tabs available in api 17 and 18

hi
please open sample app in emulator for api 17 or 18 in rtl direction! all the tabs go off screen! :((

This sample has been deprecated/archived (check README for more information on newer samples related to this technology).
As recommended by GitHub, we are closing all issues and pull requests.


KAFKA-4082: Upgrade to Gradle 3.0

Also upgrade scoverage (required for compatibility) and remove usage of
useAnt which doesn't exist in Gradle 3.0
It turns out that one cannot even run gradle to download the project Gradle version if useAnt is used in the build. This is unfortunate (the SBT launcher has much saner behaviour).
Release notes: https://docs.gradle.org/3.0/release-notes

Review by @ewencp

+1 LGTM. I tested on a few versions of Gradle 2.x to make sure it was backwards compatible too.
Removing useAnt = false wont change any behavior since it was change to false by default since 2.12. (release notes). Builds before 2.12 still work but result in ant output.
As a side note, what are your thoughts on removing the checked in gradlew and gradle.bat? I know it has come up in the past but still hangs around. Downloading the wrapper replaces these anyway and they cant be used until the wrapper is downloaded. It seams they are confusing to have checked in, result in arbitrary changes, and at worst could cause some gradle compatibility problem at some point.

It turns out that one cannot even run gradle to download the project Gradle version if useAnt is used in the build.

Yeah this is an unfortunate gradle behavior that has bit me before. Incompatible or broken scripts prevent the wrapper task from downloading a compatible version. I am not sure the best way to fix it, or if its worth fixing.

Thanks for the review @granthenke.
To be honest, I am not sure why we have checked in gradlew. Do you know? I think someone submitted a PR for removing those files, but I haven't reviewed it since I am missing the background for why things are as they are.

@ijuma I can look back for the history. I think what happend is all of gradle was checked in, then based on Apache policy the executables were removed. The scripts were left behind assuming they were useful. But because the gradle wrapper task needs to be called to download the executables anyway they really aren't.

Interestingly, the build passed, but the test results were not found for Jenkins considered it failed. A potential change in behaviour with regards to tests.

Perhaps its the unique directory change: https://docs.gradle.org/3.0/release-notes#unique-default-test-result-and-report-directories-for-test-tasks

@ijuma This LGTM assuming we sort out the Jenkins integration issue.

+1 as well after jenkins integration fix.

The build passed after I made a slight change to the Jenkins job configuration:
https://builds.apache.org/job/kafka-trunk-git-pr-jdk7/5226/testReport/
The change was from **/build/test-results/*.xml to **/build/test-results/**.xml. If people are happy with this, I'll change all trunk jobs and merge this PR.

@ijuma Makes sense to me.

Merged to trunk. For the 0.10.0 branch, we may want to remove the scalaCompileOptions.useAnt = false line only.

@ijuma Sounds like a good idea since I upgraded Gradle to test this and can no longer build the 0.10.0 branch. Oops. I think you need the scoverage change too.


Outdated for new data structure?

Hi everyone!
I was trying to experience Neurosynth at first hand, but I'm afraid that the current version might not work with the updated filename conventions? There is no https://github.com/neurosynth/neurosynth-data/blob/master/current_data.tar.gz file anymore yet, in the dataset module tries to download it (neurosynth/neurosynth/base/dataset.py: line 38).
I've manually downloaded data files from https://github.com/neurosynth/neurosynth-data but there is no file ending in dataset.txt. I could only imagine this might have been replaced with *coordinates.tsv.gz but then I believe the documentation (and the download module) should be updated.
If I'm missing something, please help this poor noobie! 😢

Hello,
As noted in the README:
Note: this package is no longer actively maintained; most of its functionality has been integrated into the much more expansive NiMARE package, which we recommend using instead.

I would highly recommend trying out NiMARE for meta-analysis instead

Oh I missed that. Thank you!

On Sep 23, 2021, at 20:23, Alejandro de la Vega @.***> wrote:
Hello,
As noted in the README:
Note: this package is no longer actively maintained; most of its functionality has been integrated into the much more expansive NiMARE package, which we recommend using instead.
I would highly recommend trying out NiMARE for meta-analysis instead
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/neurosynth/neurosynth/issues/101#issuecomment-926050960, or unsubscribe https://github.com/notifications/unsubscribe-auth/APUZ6NCKFUT4ZWPYLQ3NZX3UDNWCJANCNFSM5ENY7ZPQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


HADOOP-17039. Change scope of InternalDirOfViewFs and InodeTree to make ViewFileSystem extendable outside common package

The commit changes the scope to public for following classes:

InodeTree
InternalDirOfViewFs
NflyFSystem
ChRootedFileSystem

This change makes sure the ViewFileSystem class can be extended outside of hadoop-common package and that class can initialize the mount table (fsState) in its own way

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 22s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
22m 14s
trunk passed

+1 :green_heart:
compile
18m 20s
trunk passed

+1 :green_heart:
checkstyle
0m 47s
trunk passed

+1 :green_heart:
mvnsite
1m 22s
trunk passed

+1 :green_heart:
shadedclient
17m 46s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 54s
trunk passed

+0 :ok:
spotbugs
2m 12s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
2m  9s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 49s
the patch passed

+1 :green_heart:
compile
17m 20s
the patch passed

+1 :green_heart:
javac
17m 20s
the patch passed

-0 :warning:
checkstyle
0m 47s
hadoop-common-project/hadoop-common: The patch generated 17 new + 68 unchanged - 13 fixed = 85 total (was 81)

+1 :green_heart:
mvnsite
1m 20s
the patch passed

-1 :x:
whitespace
0m  0s
The patch has 2 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

+1 :green_heart:
shadedclient
15m 25s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 54s
the patch passed

+1 :green_heart:
findbugs
2m 15s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
9m 31s
hadoop-common in the patch passed.

+1 :green_heart:
asflicense
0m 45s
The patch does not generate ASF License warnings.

115m  6s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-2017/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2017

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux a0f391f55ab2 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / d60496e6c66

Default Java
Private Build-1.8.0_252-8u252-b09-1~18.04-b09

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-2017/1/artifact/out/diff-checkstyle-hadoop-common-project_hadoop-common.txt

whitespace
https://builds.apache.org/job/hadoop-multibranch/job/PR-2017/1/artifact/out/whitespace-eol.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-2017/1/testReport/

Max. process+thread count
1699 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-2017/1/console

versions
git=2.17.1 maven=3.6.0 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Authenticating with a personal access token to ghcr.io

Hello! Thanks for
I can Authenticating with a personal access token to ghcr.io
 cat ~/TOKEN.txt | docker login https://docker.pkg.github.com -u USERNAME --password-stdin

But how Authenticating with a personal access token to ghcr.io by  jib ?
gradle jib \
    -Djib.to.image=myregistry/myimage:latest \
    -Djib.to.auth.username=$USERNAME \
    -Djib.to.auth.password=$PASSWORD

gradle jib \
    -Djib.to.image=myregistry/myimage:latest \
    -Djib.to.auth.username=$USERNAME \
    -Djib.to.auth.password=${{ secrets.GITHUB_TOKEN }}

If you can do docker login as you've shown before running gradle jib, it should work. Jib will pick up the login credentials from docker login. If you decide to use jib.to.auth, then the username and the password will be exactly those you use with docker login. No surprise there. But please do take a look at this Stack Overflow answer first. I also suggest the GitHub Actions and ghcr.io docs to know what would be the best way to auth inside GitHub Actions.
Lastly, you said ghcr.io but you're specifying docker.pkg.github.com. They are different.

Comparing the Docker registry and the Container registry:

The Container registry (https://ghcr.io) offers improved performance for containers and will supercede the Docker registry in the future.
You can migrate your Docker images and workflows from the Docker registry (docker.pkg.github.com) to the Container registry (https://ghcr.io) for granular permissions, container storage optimization, and more. For more information, see "Migrating to the Container registry from the Docker registry."

I'm closing the issue, but feel free to ask questions further.


AC load includes the GridMeter value as Output

Hi Manuel. I have installed your multiplus emulator. I have latest venus OS installed on my RP3 and an energy meter connected via USB to venus OS directly. Set up as Gridmeter on all 3 phases:

The grid values show correctly.
However when I select any phase on the emulator config I noticed that the AC loads on the selected phase gets way too high. seems that the grid amount is added on top again.
here the setting with just L1:

In the log I see that the energy meter is recognized as well by the emulator:

and it shows under the emulator the power as AC input but also AC output. But there is no output. and exactly that output is added I guess to the AC loads.

I just updated the driver. installed it again. set it to three phases and also marked inverter output in use to show the critical load. here it is visible that the AC output is shown up and that output (which is actually the grid meter) is added to ac loads. hiding again the output in use does not change anything on the ac loads, still adding up the grid meter amount:

what do I miss?

Sounds like a bug in the code, since also others describe a similar behaviour: https://github.com/mr-manuel/venus-os_dbus-mqtt-pv/issues/32
Currently I'm very busy and have no time to look at it. Will take a few weeks/months.

@mr-manuel I am reengineering and debugging your code. Very nicely written!! And wow, complex requirements. I found out so far that AC out is only for critical loads. maybe with my seput also AC in should only be what goes to battery.
I will let you know if I find something. Thanks for the link above!

AC out is only for critical loads

That's true. Maybe this is the problem and the output should always be 0, but it's only for some cases.

@mr-manuel I got it working, I think. In my set-up the AC In should only reflect the power to and from the BMS. Venus OS is then calculating automatically the power from inverter and energy meter. I also set the BMS value (hardcoded) only to that phase where the BMS is connected to. And I set all AC Out to none.
Strange is that the Inveter shows in VRM as off and does not reflect charging/discharging (remote control shows on) and I have not seen yet the charged Wh amount, discharge however shows up. Code looks fine. however I am running the latest Venus OS Beta. Will install latest official release and further test.

@mr-manuel  got it working with official venus OS, it shows charging and discharging on the multiplus icon in VRM.
only thing not yet working is the capturing of the battery kWh for charge and discharge. It worked yesterday for 1h. have to find out what I changed on the code.
In general: for me it makes sense that only the battery should be considered for AC in (and having AC out 0 as no critical load should be simulated), because MultiPlus whole purpose is to get and provide power out of the battery only.
I will keep this thread open, until I have solved all issues and will drop you my changes (well indicating what I changed).

Yes, you are right. The battery should be considered only for AC In.
For the capturing of the kWh of the battery, the battery itself needs to fill the energy forward field.

yes, I found that part of the code.
/Energy/InverterToAcOut
...
/Energy/OutToInverter

and I like your approach of aggregating it and based on my testing should also be just fine. Will debug later :) need to work now.

From my tests I did a while back ago, the MultiPlus emulator was not able to provide the kWh from the battery to the VRM portal. You need the battery device/service to do that. How did you connect the battery?

@mr-manuel  works like a charm

somehow in my set-up the battery was connected but did not provide any current, therefore I had that division by 0 and the program never submitted any Wh. Now it works.
I changed the following area:

Excluded AC out (on my code just for one phase and just the power, when I excluded it all, VRM did not like that)
Only set AC in to battery power, charge, ...
Defaulted my battery voltage to 24 (to prevent division by 0, I tried some kind of if statement, to default it when current = 0 but Python is not my friend... seems we don't like each other :-))

just the charging is not showing up. very strange. I see the charging value building up:
2024-07-04 11:28:59.686483500 INFO:root:--> data_watt_hours(): {"time_creation":<PHONE_NUMBER>, "dc": {"charging": 827, "discharging": 0}, "count": 1}
2024-07-04 11:28:59.712955500 INFO:root:--> charge/discharge: {"dc": {"charging": 0.418, "discharging": 1.484}}

and the code submitting it to:
self._dbusservice["/Energy/OutToInverter"] = (
            json_data["dc"]["charging"]
            if "dc" in json_data and "charging" in json_data["dc"]
            else 0
        )

but nothing in VRM. Will need to check if that is the correct path.

@mr-manuel I got it working!!!!

finally. don't ask me why, but you have to set the charging amount to the following path:
self._dbusservice["/Energy/AcIn2ToInverter"] = (
            json_data["dc"]["charging"]
            if "dc" in json_data and "charging" in json_data["dc"]
            else 0
        )

around line 744 in dbus-multiplus-emulator.py.
I will close this item here.

Can you add your script here as attachment so I can compare it?

dbus-multiplus-emulator20240704.2.txt
yes of course. here we go. but I have not changed it for all phases and on the ACOut I tricked it by asking if "skip" is in the selected phases ;-)


add Request.CloneWithBody and Request.SetBody

These methods wrap provided io.Readers that aren't already also closers in our own nopCloser type, which we already do in NewRequest.  This extends our ability to send Content-Length with certain in-memory types to other uses of requests.
We note in the documentation now that using SetBody is preferred over setting the Body field directly.)
Fixes #104

I wonder if we should also have a SetBody method and prefer that to assigning to .Body directly for the same reason?

I prefer your suggestion of exposing the type, with a clear doc string explaining how it enables the content-length header. That leaves the author with the most flexibility.

I pushed a replacement commit that adds SetBody.  You could make the argument that we no longer need CloneWithBody now.


Describe Image None? Image

Description
When I run the chain with the provided url to the image source it gives error did you mean https://None?

Steps to Reproduce the Bug
Run Chain with 1 Step for Describe Image I used raw JPG file to a website source  you can test with other image types.
Expected Behavior
Supposed to Give you a description of the image but instead it shows error.
Operating System

[X] Linux
[ ] Microsoft Windows
[ ] Apple MacOS
[ ] Android
[ ] iOS
[ ] Other

Python Version

[ ] Python <= 3.9
[X] Python 3.10
[ ] Python 3.11

Environment Type - Connection

[X] Local - You run AGiXT in your home network
[ ] Remote - You access AGiXT through the internet

Runtime environment

[ ] Using docker compose
[ ] Using poetry
[ ] Custom setup (please describe above!)

Acknowledgements

[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of AGiXT.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

Confirmed resolved.


[feature] Update project search fields on user full name change

Purpose
Closes #1237
Originally, if a user changes his name and searches for his public projects, his public projects would be under his previous name, instead of the latest one.
Cause
Changing and saving the full name does not trigger a update on search on his projects, so the search fields that yield to the project remains unchanged.
Changes
Imported elasticsearch.helper. Now changing the full name will trigger the bulk api, so that the contributor list of the projects/components that this person is a contributor in will be updated in time.
Side effects
I haven't changed any existing code, so to my knowledge, nothing.

Superseded by #2956.


do you have a plan about supporting grpc?

I heard about the support plan for http 2 of backend.
I have a lot of interest in grpc.
Do you have a plan  support for proxy of grpc?
I expect a grpc filter including server streaming.
Can you tell me about this?
I believe it is a great plan.

Yeah, it's on the roadmap but can't give you a timeframe.

Yeah it's available with 2.x architecture. There's no need or plan for 3.x.


ASP.NET Core 3.1 support

Full disclosure I'm a bit naive on the subject, but wonder what is gained by retargeting to netcoreapp3.1?  My understanding is that the current netstandard2.0 target makes this library consumable across a wider range of framework impls and versions?
Unfortunately we've backed ourselves into a corner where it will take some significant effort to upgrade to 3.1 in our current project.  Is it possible to maintain a separate branch supporting netstandard2.0...or is this a case for multi-targeting?  Happy to assist in the effort if anyone can provide some insight on best approach here.

ASP.NET Core targets netcoreapp3.1 now, and the AssignmentGradeServices and NamesRoleProvisioningService have dependencies on Microsoft.AspNetCore packages. I briefly investigated splitting those components out into separate packages, which seemed like it would work, but was a more invasive change. Mutitageting is another option if 2.x support needs to be maintained; a couple #ifdefs would cover that.
Are you still using 2.2 even though it’s EOL? I had a couple projects I had to decided to either move up to 3.1 or back down to 2.1 to stay on a framework that would still get security updates. Fortunately they were simple, so I was able to upgrade them. It’s a bummer the 2.2 to 3.x upgrade path is difficult in some cases.

Are you still using 2.2 even though it’s EOL?

Unfortunately.  A significant portion of our dependencies target full framework so we've been facing the same decision.  We'll likely end up moving back to 2.1 for security updates and then begin the refactoring in order to move forward.

Are you still using 2.2 even though it’s EOL?

Unfortunately. A significant portion of our dependencies target full framework so we've been facing the same decision. We'll likely end up moving back to 2.1 for security updates and then begin the refactoring in order to move forward.

I'll update this PR to multi-target 2.1 and 3.1.

@srijken, can you (or another committer) review this please?

@andyfmiller, can you review this please?

@andyfmiller, can you review this please?

What's the process for getting this merged?

What's the process for getting this merged?

Just thought I'd check in and see if there are plans to have the 3.1 changes merged into master.  ??

Sorry about the delay in the merge!

Thanks!


clarify current locations of PT and SS tables

This is an update to the last PR updating the schema page, but adds reference to the current publication location for Paris Traceroute and Sidestream tables. @georgiamoon could you please review?

This change is

Any reason to not just directly link these to the right views?


大佬有java版本调用吗

this is an android project, but you can translate it to java

no way,this project depends on the native Android library. I recomend you use the Skia library.

额，不都是中国人吗？我还以为老外看懂了中文。。。


Windows 10 Setup asking for driver - Synology DSM 7.2.1 DS918+

Operating system
Synology DSM 7.2.1 DS918+
Description
I am running docker windows 10 pro from Container Manager -> Projects as docker compose.  It looks like everything is loading until get to setup screen, system is asking for driver!? am I missing something from the yaml file? Thanks.
System is runing on Synology DSM 7.2.1 DS918+
KVM is present (Virtual Machine App was installed)
Docker compose
version: "3"
services:
windows:
image: dockurr/windows
container_name: windows
environment:
VERSION: "win10"
RAM_SIZE: "8G"
CPU_CORES: "3"
DISK_SIZE: "60G"
devices:
- /dev/kvm
cap_add:
- NET_ADMIN
ports:
- 8006:8006
- 3389:3389/tcp
- 3389:3389/udp
stop_grace
devices:
- /dev/kvm
cap_add:
- NET_ADMIN
ports:
- 8006:8006
- 3389:3389/tcp
- 3389:3389/udp
stop_grace_period: 2m
volumes:
- ./data:/storage
network_mode: bridge
Docker log

date | stream | content
-- | -- | --
2024/06/07 05:47:30 | stdout | [=3h❯ Windows started succesfully, visit http://localhost:8006/ to view the screen...
2024/06/07 05:47:20 | stdout | BdsDxe: starting Boot0001 "UEFI QEMU QEMU CD-ROM " from PciRoot(0x0)/Pci(0x5,0x0)/Scsi(0x0,0x0)
2024/06/07 05:47:20 | stdout | BdsDxe: loading Boot0001 "UEFI QEMU QEMU CD-ROM " from PciRoot(0x0)/Pci(0x5,0x0)/Scsi(0x0,0x0)
2024/06/07 05:47:20 | stdout | [=3h[=3h[=3hBdsDxe: failed to load Boot0002 "UEFI QEMU QEMU HARDDISK " from PciRoot(0x0)/Pci(0xA,0x0)/Scsi(0x0,0x0): Not Found
2024/06/07 05:47:19 | stdout | ❯ Booting Windows...
2024/06/07 05:47:19 | stderr | ❯ Warning: your configured RAM_SIZE of 8 GB is very close to the 8 GB of memory available, please consider a lower value.

Screenshots (optional)

Just a guess, Synology offers an iso image for windows guest systems and it contains drivers which might be what is required in your case.
Though, I do not know how to connect it as a CD-ROM within docur.


Fix incorrect type and default for expanded option

Coverage remained the same at 100.0% when pulling 28b12757c3f456fd669b4d28adabd91df3b51657 on akshayp:patch-1 into ac10128667b70ade6b9adb5fb676595d3dab9ff6 on glennjones:master.

Thanks for correction. Manually merged to avoid conflict with previous PR


Swipe to delete without SwipeAction

Can I implement simple "swipe to delete" without adding any SwipeAction(or mb with clear background and title)? I just want to move cell at some point and delete without actions view appearing.

@ikrachko func tableView(_ tableView: UITableView, editActionsForRowAt indexPath: IndexPath, for orientation: SwipeActionsOrientation)
this worked for me for swipe to delete only


resolve method ambiguity

ERROR:
  +(A::SciMLOperators.AbstractSciMLOperator, ::NullOperator{N}) where N in SciMLOperators at /Us
ers/vp/.julia/dev/SciMLOperators/src/basic.jl:165
  +(A::SciMLOperators.AddedOperator, B::SciMLOperators.AbstractSciMLOperator) in SciMLOperators
at /Users/vp/.julia/dev/SciMLOperators/src/basic.jl:315
Possible fix, define
  +(::SciMLOperators.AddedOperator, ::NullOperator{N}) where N

tests passing on my machine


Make rawdashboardbyuid method public

Would like rawDashboardByUID method public so it can be called outside the current package. This can help read the dashboard data for various other purposes.

Since this method exposes an object coming from the sdk package, I'm not too keen on making it public.
grabana is meant to be a dashboard builder, it doesn't try to expose a complete Go client.
If that's what you need, https://github.com/grafana/grafana-openapi-client-go might be a good fit :)


Support multiple checksum for certain filesystems

This is for MMENG-3056.
I tried to make this pr but found this dramatically increase the complexity of storage code. Mainly because the checksum can be a file OR a field in db entry. This cause big trouble when we want to use it, because we have to check both. Many methods are impacted, e.g, openInputStream, exist, etc.
Because the checksums were to be stored in one filed, it has to go through json serialize/deserialize when write/read. This also introduce more complexity and performance concerns.
I am afraid this could cause unnecessary issues, with no big gains. I want to drop this pr. wdyt?

If there are more complexities than benefits gained, I agree to drop the change. So does this mean we will still keep the file checksum way, right?

Yeah, agree, if this way does not work well, let's figure out a better approach later if possible.

We still use checksum for deduplicate, via the default sha256. By abandoning this pr, we don't add others like md5,sha1,etc. This was planned to reduce the physical checksum file entries in path DB. But looks it brings more trouble.


rankSample is tricky to use

Not sure if I'm using it wrong, but I had to do some gymnastics to use rankSample in the new categorical sampler I just wrote for categorical nodes.  Advice or discussion on modifications to rankSample() welcome:
    run = function() {
        currentValue <- model[[target]]
        for(i in 1:maxval) {
            model[[target]] <<- i
            probs[i] <<- exp(calculate(model, calcNodes))
        }
     ## can't use model[[target]] directly in rankSample as it expects int NimArr for 3rd arg.
        newValue <- integer(1)
   ## for second arg, rank sample wants an int&; I need these gymnastics instead of simply doing rankSample(probs, 1, newValue)
        size <- integer(1,1)
        rankSample(probs, size[1], newValue)
        model[[target]] <<- newValue[1]
        calculate(model, calcNodes)

rankSample was originally written for a specific internal purpose and so not generally tested.  I don't think there's much (any?) point in taking a scalar as a reference instead of a value.  We should be able to make it an int argument in C++.

I believe this was fixed by #492 (@paciorek feel free to reopen if I'm incorrect)


machine code in dot_toolchain_support.h

Output of 'strings libarm_compute.so | grep arm_compute_version':
Platform:
Operating System:
hi, what is the machine code of "sdot v10.2s, %[v1].8b, v0.8b     \n"  in dot_toolchain_support.h
this dot_toolchain_support.h only  support "sdot v10.4s, %[v1].16b, v0.4b[0]     \n"

Problem description:

Hi @chenjiaoAngel
The macro _DECLARE_UDOT in
https://github.com/ARM-software/ComputeLibrary/blob/master/src/core/NEON/kernels/arm_gemm/kernels/a64_gemm_u8_12x8/dot_toolchain_support.h
is just a workaround to generate the opcode for the dot product instruction when the toolchain does not support it. If you compile for arch=armv8.2-a and your compiler does not know about the sdot instruction, ACL will use this macro generate the machine code for sdot.
Quoting the reference manual about the dot product instruction.

ARMv8.2-DotProd provides SIMD instructions that perform the dot product of the four 8-bit subelements of the 32-bit elements of one vector with the four 8-bit subelements of a second vector. It provides two forms of the instructions, each with signed and unsigned versions

For more details please take a look at the reference manual: https://developer.arm.com/docs/ddi0487/latest/arm-architecture-reference-manual-armv8-for-armv8-a-architecture-profile?_ga=2.40959778.1213988665.1560783811-833326948.1485870095&_gac=1.217587044.1560429485.Cj0KCQjw6IfoBRCiARIsAF6q06sJzDPBIsg_G9PpEWQT19iGrbzEn55n27jUd8RRa5MY1c_dcvlc1j4aAgZIEALw_wcB

Thanks very much


[Fix] Turn off auto update in offline mode

Pull Request Checklist
This PR ensures that all auto-update settings (WHISPER_MODEL_AUTO_UPDATE, RAG_EMBEDDING_MODEL_AUTO_UPDATE, RAG_RERANKING_MODEL_AUTO_UPDATE) are automatically disabled (False) when OFFLINE_MODE is enabled.
Changes:

Modified logic for auto-update flags to respect OFFLINE_MODE.
Auto-update settings now only evaluate their respective environment variables when OFFLINE_MODE is False.

Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
https://github.com/open-webui/open-webui/discussions/7742
Before submitting, make sure you've checked the following:

[x] Target branch: Please verify that the pull request targets the dev branch.
[x] Description: Provide a concise description of the changes made in this pull request.
[x] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[x] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
[x] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
[x] Testing: Have you written and run sufficient tests for validating the changes?
[x] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring adherence to the project's coding standards?
[x] Prefix: To cleary categorize this pull request, prefix the pull request title, using one of the following:

BREAKING CHANGE: Significant changes that may affect compatibility
build: Changes that affect the build system or external dependencies
ci: Changes to our continuous integration processes or workflows
chore: Refactor, cleanup, or other non-functional code changes
docs: Documentation update or addition
feat: Introduces a new feature or enhancement to the codebase
fix: Bug fix or error correction
i18n: Internationalization or localization changes
perf: Performance improvement
refactor: Code restructuring for better maintainability, readability, or scalability
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)
test: Adding missing tests or correcting existing tests
WIP: Work in progress, a temporary label for incomplete or ongoing work

Changelog Entry
[Unreleased]
Added
Logic to disable all auto-update settings (WHISPER_MODEL_AUTO_UPDATE, RAG_EMBEDDING_MODEL_AUTO_UPDATE, RAG_RERANKING_MODEL_AUTO_UPDATE) when OFFLINE_MODE is enabled.
Changed
Auto-update flags now depend on OFFLINE_MODE to determine whether updates should be disabled.
Description

[Concisely describe the changes made in this pull request, including any relevant motivation and impact (e.g., fixing a bug, adding a feature, or improving performance)]

Added

Logic to disable all auto-update settings (WHISPER_MODEL_AUTO_UPDATE, RAG_EMBEDDING_MODEL_AUTO_UPDATE, RAG_RERANKING_MODEL_AUTO_UPDATE) when OFFLINE_MODE is enabled.

Changed

Auto-update flags now depend on OFFLINE_MODE to determine whether updates should be disabled.

Deprecated

None

Removed

None

Fixed

[List any fixes, corrections, or bug fixes]

Security

[List any new or updated security-related changes, including vulnerability fixes]

Breaking Changes

BREAKING CHANGE: [List any breaking changes affecting compatibility or functionality]

Additional Information

[Insert any additional context, notes, or explanations for the changes]

[Reference any related issues, commits, or other relevant information]

Screenshots or Videos

[Attach any relevant screenshots or videos demonstrating the changes]

LGTM, Thanks!


build failures./usr/bin/ld: cannot find -lc++  recipe for target 'cmTC_b4e2d'   failed

clang 3.8.0
ubuntu 16.04
I followed the instruction
polly.py --toolchain libcxx --install --reconfig --verbose
But get error
/usr/bin/ld: cannot find -lc++
clang: error: linker command failed with exit code 1 (use -v to see
invocation)
CMakeFiles/cmTC_8bae2.dir/build.make:97: recipe for target 'cmTC_8bae2'
failed
make[1]: *** [cmTC_8bae2] Error 1

I just noticed this issue.  This seems like a configuration error.  I'll add the same toolchain to travis (Trusty image) and see how that works.

You are most likely missing libc++, try:
apt-get install clang-3.8 libc++-dev libc++abi-dev

With this configuration I notice a build error in cereal:
/home/dhirvonen/.hunter/_Base/75c7e58/6a87f3a/09f3179/Install/include/cereal/details/polymorphic_impl.hpp:235:61: error: chosen constructor is explicit in copy-initialization
          auto & derivedMap = baseMap.insert( lb, {baseKey, {}} )->second;

referenced here: https://github.com/USCiLab/cereal/issues/339
I'm not sure if it is an option, but you could try the tested TOOLCHAIN=gcc-4-8-pic-hid-sections from the current travis setup for now.

This has been fixed in the latest cereal develop branch.  I've tested this locally in xgboostcpp and it works fine.  I will add the required patches for a libcxx toolchain build to the travis configuration.
For the trusty image this requires:
$ apt-get clang-3.8 libc++-dev libc++abi-dev
$ sudo update-alternatives --install /usr/bin/clang++ clang++ /usr/bin/clang++-3.8 380
$ sudo update-alternatives --install /usr/bin/clang++ clang++ /usr/bin/clang++-3.8 380

I've updated travis to test libcxx with clang-3.8 in this pr and it seems to be working fine:
https://github.com/elucideye/xgboostcpp/pull/10
This is using a newer cereal via GIT_SUBMODULE.  You can use this one temporarily if needed, but I will update the PR using standard hunter packages once the change has been accepted upstream and will resubmit the PR and merge to master then.  Thanks for pointing out the problem before I ran into it.  I'll leave the issue open until then.

An ubuntu clang 3.8 + libc++ toolchain test has been added to travis in  #14. This PR also replaces the hunter xgoost GIT_SUBMODULE with a standard hunter package, which makes the project cleaner.  Please try the latest version.


Rgb features only?

Hi,
Thanks for your great job!
I wonder if the flow features can be extracted  from videos by you code?

Hello,
Thank you for using the framework. Unfortunately no, in its current state it does not support Flow features. You can of course try by tuning the model parameters like input number of channels and dataset code. But you would have to train your own model.

@tridivb
Thanks for your reply!
But I got another problem now, the code seems working but no output comes, displaying:
**Loading Video List ...
Done
1 videos to be processed...

Processing /home/work/slowfast_feature_extractor/video/0a244c71991f386b161bcd787c7da607...**
So, does it really work?

Hi,
Thanks for bringing this to my attention. Please follow the new issue #10. I am closing this as the original question has been addressed. To report bugs, please follow the bug report template next time.


feat: use docker-ce

Description
Uses docker-ce rather Ubuntu
In my test create did improved from 2m3s to 1m38s
Related Issue(s)

Fixes #5

Wonderful - thank you very much


docs(README.md): notice regarding the project status

Signed-off-by: Leonardo Grasso<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

The changes look okay of course, but it looks like I didn't set up the build job properly. I'll work with @leogr on fixing up the test-infra side of things.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, mstemm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr,mstemm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Cannot Configure ROS2 Galactic from Docker Build Script

I was following the instructions to download ROS2 Galactic with using the flags --with-pytorch and --with-slam, however I run into an error after (I assume) it tries to build the slam container.
Successfully tagged ros:galactic-pytorch-l4t-r32.6.1

Building container ros:galactic-slam-l4t-r32.6.1
BASE_IMAGE=ros:galactic-pytorch-l4t-r32.6.1

Building ros:galactic-slam-l4t-r32.6.1 container...

and this is the error I get at the very end:
-- Configuring incomplete, errors occurred!
See also "/tmp/pangolin/build/CMakeFiles/CMakeOutput.log".
See also "/tmp/pangolin/build/CMakeFiles/CMakeError.log".
The command '/bin/bash -c git clone https://github.com/stevenlovegrove/Pangolin /tmp/pangolin &&     cd /tmp/pangolin &&     mkdir build &&     cd build &&     cmake ../ &&     make -j$(nproc) &&     make install' returned a non-zero code: 1

It seems that errors occur after it tries to clone this repository.
Not really sure how to go about fixing this, as I am not very experienced with Docker.

Update:
I added this new code to the Dockerfile.ros.slam code before Pangolin was being cloned:
RUN apt-get update && \
    apt-get install -y --no-install-recommends \
    pkg-config \
    && rm -rf /var/lib/apt/lists/* \
    && apt-get clean

and I added libxkbcommon-dev package right after libsuitesparse-dev \. This code block is located at line 49 on the Dockerfile currently on the main branch.
RUN apt-get update && \
    apt-get install -y --no-install-recommends \
            ffmpeg \
            libglew-dev \
		  libboost-all-dev \
		  libboost-system-dev \
		  libcanberra-gtk-module \
            libsuitesparse-dev \
    && rm -rf /var/lib/apt/lists/* \
    && apt-get clean

I did not run into the same error anymore (yay!), however I did run into a different issue:
[ 43%] Generating xdg-shell-client-protocol.h
Could not open input file: No such file or directory
CMakeFiles/pango_windowing.dir/build.make:81: recipe for target 'xdg-shell-client-protocol.h' failed
make[2]: *** [xdg-shell-client-protocol.h] Error 1
make[1]: *** [CMakeFiles/pango_windowing.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
CMakeFiles/Makefile2:812: recipe for target 'CMakeFiles/pango_windowing.dir/all' failed
[ 44%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/pango.cpp.o
[ 44%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/pango_video_output.cpp.o
[ 45%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/debayer.cpp.o
[ 46%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/shift.cpp.o
[ 46%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/transform.cpp.o
[ 47%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/unpack.cpp.o
[ 48%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/pack.cpp.o
[ 48%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/join.cpp.o
[ 49%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/merge.cpp.o
[ 49%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/json.cpp.o
[ 50%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/thread.cpp.o
[ 51%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/shared_memory.cpp.o
[ 51%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/v4l.cpp.o
[ 52%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/ffmpeg.cpp.o
[ 52%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/ffmpeg_convert.cpp.o
[ 53%] Building CXX object CMakeFiles/pango_video.dir/components/pango_video/src/drivers/ffmpeg_output.cpp.o
[ 54%] Linking CXX shared library libpango_video.so
[ 54%] Built target pango_video
Makefile:135: recipe for target 'all' failed
make: *** [all] Error 2

It seems as if there is still an issue when it comes to cloning Pangolin.
Not sure how to move forward.


Fix memcached 1.5.4 compatibility.

memcached 1.5.4 changed order of evaluation of command line parameters
[1]. As a result, the test suite failed with errors such as:
$ ruby -Ilib:test -e 'Dir.glob('\''./test/test_*.rb'\'').sort.each{ |x| require x }'
Testing with Rails 5.1.4
Using standard socket IO (ruby 2.5.0p0 (2017-12-25 revision 61468) [x86_64-linux])
Run options: --seed 35324
# Running:
Found memcached 1.5.4 in PATH
.........................................................Cannot set item size limit higher than 1/2 of memory max.
Cannot set item size limit higher than 1/2 of memory max.
F
Failure:
Dalli::using a live server::in low memory conditions#test_0001_handle error response correctly [/builddir/build/BUILD/dalli-2.7.6/usr/share/gems/gems/dalli-2.7.6/test/test_dalli.rb:740]:
unexpected failure on iteration 0
/usr/share/gems/gems/railties-5.1.4/lib/rails/test_unit/reporter.rb:70:in `method': undefined method `test_0001_handle error response correctly' for class `Minitest::Result' (NameError)

This commit adjust the memcached parameters to make the test suite pass.
[1] https://github.com/memcached/memcached/commit/95246f7947e9d95969d04db0898a93ef66235fb0

@voxik It turns out that this isn't backwards compatible, so we need a more sophisticated approach.  In theory we could query the memcached server in the test suite for its version, and pass the appropriate set of command line arguments.  That may be a viable approach.

@brandonsturgeon No.  I'd love to see a PR that is generally compatible, but no one has submitted one.


build(ci): readable config

v13.5.2 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


Using pyGPCCA under the hood

[ ] Additional function parameters, change functionality or change defaults?
[ ] New estimator in cr.tl.estimators?
[ ] New kernel in cr.tl.kernels?
[ ] New gene trend model in cr.ul.models?
[ ] New plotting function in cr.pl?
[x] Other?

What will the process look like of switching to pyGPCCA under the hood? When can we initiate the move?

What will the process look like of switching to pyGPCCA under the hood? When can we initiate the move?

I can start now, but merge only after pyGPCCA is on PyPI - we also need to discuss write release notes:

mention circular embedding
mention pyGPCCA

What will the process look like of switching to pyGPCCA under the hood? When can we initiate the move?

I can start now, but merge only after pyGPCCA is on PyPI - we also need to discuss write release notes:

mention circular embedding
mention pyGPCCA

Oh, now that you mention it, I just checked our release notes here and we don't seem to have any for 1.1 - what happened there?

Oh, now that you mention it, I just checked our release notes here and we don't seem to have any for 1.1 - what happened there?

closed via #472

Very nice!


Error : MC is not defined

this appears as a distude error and will need @kabirsingh2004 and crew to fix this error

show ss of terminal error

Nothing is outputted to console

This looks like the new error (nothing change) happens then Posting a YT Link in its own channel

This is what happens when you type ("Among us") in to its music channel

This is output from /play command with link

This is output from /play with song name instead of a link

This is the console the whole time of running all of these commands

Found a Error...
PlayingError: FFmpeg/avconv not found!
Id: <YT ID>
Name: <Song Name>
------------------------------------------------------------------------------------------
Found a Error...
PlayingError: Cannot destructure property 'duration' of 'queue.songs[0]' as it is undefined.

Close this @kabirsingh2004 it appears to be fixed?

You need to get FFmpeg


Debouncing Server Requests

I have included Tokenize in my non-commercial web site. It's really a great tool and saved me much time in creating my own suggest / multi-select from scratch. One item which I included in my Tokenize implementation is a debounce mechanic, whereby an ajax server request is not sent for every 'keyup' event, which would cause an unnecessary load on the server. I would suggest implementing a similar mechanic in your code for server-based requests. Here is the segment of code where I included debounce in my implementation. However, in a more general implementation, it may make sense to only debounce if 'search' is using ajax rather than pulling from a select element.
        case KEYS.BACKSPACE:
                    if(this.searchInput.val()){
                        this.debounce(this.search, 300);
                    } else {
                        this.dropdownHide();
                    }
                    break;
                default:
                    if(this.searchInput.val()){
                        this.debounce(this.search, 300);
                    }
                    break;
            }
        }

    },

	  debounce: function(func, wait){
			var $this = this;
			var later = function(){
				clearTimeout(timeout);
				func.call($this);
			};
			if(timeout)
				clearTimeout(timeout);
			timeout = setTimeout(later, wait);
    },

Hi Phant0mass, this is a good idea I will implement it for the next version, I will just modify it a little bit to use debounce function only if ajax is used...


ignore resistances in temperature system

About the PR

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

tweak: made temperature ignore resistances
fix: fixed shields making blocking sound when user is damaged by temperature

What about lizards receiving +50% cold damage?
Is it ignored now? Is there any reason for cold resist to exist?

What about lizards receiving +50% cold damage? Is it ignored now? Is there any reason for cold resist to exist?

it still works the same
i think ice anomaly does cold damage


Reference PermissionsAndroid in Geolocation doc

Motivation (required)
Current Geolocation API docs do not mention using PermissionAndroid when working with API 23+. This updates the docs to reflect that requirement.
Test Plan (required)
This a documentation change, so no testing is required.

While this is just a docs change, I'd like to see a test plan where the website is built and you've confirmed that the newly added documentation is displayed correctly. The change looks innocent and I'm confident it will work just fine, but minor doc changes like these have bit us in the past.

@hramos I updated the requested changes, built site, and validated that everything compiles, and looks good. Please see attached screenshot below. Please let me know if you require anything further.


fix typo in JsonDeSerializer.TYPE_MAPPINGS

fixed a small typo in docs in 'JsonDeSerializer.TYPE_MAPPINGS'
Thanks for the docs!

@system205 ,
Thank you for contribution; looking forward for more!
Will be merged when PR build is green.


Refactoring : Remove TableClientFactory

@kbeaugrand, We cannot remove TableClientFactory, it is used by seeders.
We have to maintain TableClientFactory until we no longer support data migration from azure storage account table to database.

The Table Client Factory was used during migration from v2 to v3.
Since we are now in v4. We can achieve that?

Blocked by #1617


Added copy to clipboard

Issue: #1097

Great work thanks!
@all-contributors please add @Yashs911 for code


[5.5] Normalize to lowercase the dictionary key in the eloquent relations mapping multiple records while eager loading records

The dictionary keys are case sensitive and therefore if the foreign keys are strings and differ in both tables, then the records are not mapped properly and loaded.

I am closing this pull request because it lacks sufficient explanation, tests, or both. It is difficult for us to merge pull requests without these things because the change may introduce breaking changes to the framework.
Feel free to re-submit your change with a thorough explanation of the feature and tests - integration tests are preferred over unit tests. Please include it's benefit to end users; the reasons it does not break any existing features; how it makes building web applications easier, etc.
Thanks!


[1216] Add content status

Context
The course enrichment status is shown to publishers in the course table
Changes proposed in this pull request
Add content_status to Course API V2.
Guidance to review
This does a query per-course, which means N+1 queries on the courses table. We may need to consider performance going forward.

Looking good, now why has travis gone on holiday...

@heidar I've addressed all your comments. Ready for re-review.
@misaka I backed out your association additions to CourseEnrichment because they were causing the specs to fail (and anyway aren't needed as part of this change).


Extend the builder to support using the DOCKER_HOST environment variable

Is your feature request related to a problem? Please describe.
We have begun looking into test containers in our integration and performance testing environments, and for this reason we are collecting some common extensions in a central project shared among our testing projects. Whenever possible, we would prefer such generic extensions to just be a part of this upstream repository.
One such thing is the ability to automatically honour the DOCKER_HOST environment variable. We use this in every test project, since our CI/CD environment exposes the Docker daemon via TCP, while our local development environments do not.
Describe the solution you'd like
We would like if handling the DOCKER_HOST environment variable was something that was baked into the dotnet-testcontainers project, since we think it is a good candidate for an upstream contribution from our internal extension library.
To avoid introducing a breaking change, we considered if it would be best to introduce an additional builder method such as .WithDockerEndpointFromEnvironment() or perhaps .WithDockerHost().
One could also consider supporting a parameterless call to .WithDockerEndpoint() and fall back to using the environment variable if no explicit endpoint is set - although that could be confusing behaviour I guess. I would like some other thoughts on this.
Describe alternatives you've considered
Currently we just have a small chunk of code in an internally shared project that does the magic for us:
var dockerHost = Environment.GetEnvironmentVariable("DOCKER_HOST");
if (dockerHost != null)
{
  builder = builder.WithDockerEndpoint(dockerHost);
}

This is perfectly fine, but we figured it would be a good candidate for something in the upstream repository.
Additional context
I see there was some discussion about the DOCKER_HOST environment variable in #370, but nothing has been contributed so far from what I can tell. Using this environment variable alone is also out of scope for #370, so I guess a separate pull request would be in order.

Thank you for the clarification. I was not sure what to expect from the state of #370.
What you are saying, is that there will be an "authentication" implementation that simply respects the DOCKER_HOST environment variable (as in, it is planned / underway), or that it could be implemented in that way?
If there is a standard implementation that uses the DOCKER_HOST environment variable to identify and talk to the Docker daemon, that certainly satisfies our requirement.

What you are saying, is that there will be an "authentication" implementation that simply respects the DOCKER_HOST environment variable

Yes, there will be an implementation that detects the env variables (even TLS, etc.) and  configures the builder automatic. Then, the above suggest code is not necessary anymore. I hope I can bring in myself a bit more and push this feature. I know many ask for this feature. It will still be possible to override the automatic applied configuration.

@HofmeisterAn In that case, you can go ahead and close this issue if you wish, or leave it open for tracking. I am really happy to at least hear that this support is coming!

I prepared most parts: https://github.com/HofmeisterAn/dotnet-testcontainers/issues/370#issuecomment-1146842098. It shouldn't be much work anymore to set the endpoint automatic from the environment variable (DOCKER_HOST).

DOCKER_HOST is now automatically considered by 1.6.0-beta.2258.


:sparkles: speedup surface temperature

Speed up geometry clipping by running rough clip_box before the actual clip operation. I googled this recommendation by accident on some random forum. Loading gzip still takes a lot of memory (peak is ~10gb), but the actual processing is much faster now, and the whole step takes ~8min on the staging server. I visually checked the output and it looks the same.
@veronikasamborska1994 I am leaving the decision on whether to merge this or the snapshot version to you. Both are valid approaches.

hey @Marigold! cool you found this! let's go with your solution! I also just checked the output and it looks basically the same except I think we lose Aruba (Neth.) and Saint Lucia data when we do it this way but tbh I am not sure whether we should be doing this clipping for such small countries anyway.


diffOnly option

diffOnly option allow you to keep only what changed from your last update to your new update.
Signed-off-by: Joao Manoel Pampanini Filho<EMAIL_ADDRESS>

Hi! Thanks for this great pull request! :)

Has some things to change like:

You keep the code comments
The test name for the diffOnly.js

Sorry about the code comments ...
I am very bad at naming things , what you think is a good name for the test?


Version String for Extended Stable

In multiple places in this document, it says "The version string on edge://settings/help will indicate that you are running a different channel."  How exactly does it indicate this?
My edge://policy says TargetChannel has a value of "extended", and edge://settings/help version string is "Version 94.0.992.50 (Official build) (64-bit)".  This is the same version string that computers without this policy have.

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

ID: 42602884-85fa-b00c-d742-84f097f53334
Version Independent ID: 137d9161-f81a-3b3f-7a0a-519f8455839c
Content: Microsoft Edge channel overview
Content Source: edgeenterprise/microsoft-edge-channels.md
Product: microsoft-edge
GitHub Login: @RyanHechtMSFT
Microsoft Alias: srugh

Hi Jason,
The version string will change the next time Microsoft Edge takes an update. This is because the channel switch to Extended Stable doesnt happen until the target channel has an update available that is greater than the version of the current channel (so because you were already on 94.0.992.50, the latest version available, there hasn't yet been an update that will force the channel switch!)
Hope this helps!
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: JasonNVSD @.>
Sent: Friday, October 15, 2021 1:54:05 PM
To: MicrosoftDocs/Edge-Enterprise @.>
Cc: Ryan Hecht @.>; Mention @.>
Subject: [MicrosoftDocs/Edge-Enterprise] Version String for Extended Stable (#243)
In multiple places in this document, it says "The version string on edge://settings/help will indicate that you are running a different channel." How exactly does it indicate this?
My edge://policy says TargetChannel has a value of "extended", and edge://settings/help version string is "Version 94.0.992.50 (Official build) (64-bit)". This is the same version string that computers without this policy have.

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

ID: 42602884-85fa-b00c-d742-84f097f53334
Version Independent ID: 137d9161-f81a-3b3f-7a0a-519f8455839c
Content: Microsoft Edge channel overviewhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-ca%2Fdeployedge%2Fmicrosoft-edge-channels%23feedback&data=04|01|Ryan.Hecht%40microsoft.com|09b6f2a2ef8f4c961f4708d99004c858|72f988bf86f141af91ab2d7cd011db47|0|0|637699172515163818|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=m32msW%2FUezpnl7U3NcPP00TjjWGGEEg%2BcjprgUoY680%3D&reserved=0
Content Source: edgeenterprise/microsoft-edge-channels.mdhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FEdge-Enterprise%2Fblob%2Fpublic%2Fedgeenterprise%2Fmicrosoft-edge-channels.md&data=04|01|Ryan.Hecht%40microsoft.com|09b6f2a2ef8f4c961f4708d99004c858|72f988bf86f141af91ab2d7cd011db47|0|0|637699172515163818|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=anCpLp5yawnf11ZzbUF3PrfDMQur%2BU7ND1UZTl0pzAo%3D&reserved=0
Product: microsoft-edge
GitHub Login: @RyanHechtMSFThttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FRyanHechtMSFT&data=04|01|Ryan.Hecht%40microsoft.com|09b6f2a2ef8f4c961f4708d99004c858|72f988bf86f141af91ab2d7cd011db47|0|0|637699172515173775|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=vzmCMm3UexCyTZovKu2C56q%2FAASb41DlPwhxQx4EheI%3D&reserved=0
Microsoft Alias: srugh

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FEdge-Enterprise%2Fissues%2F243&data=04|01|Ryan.Hecht%40microsoft.com|09b6f2a2ef8f4c961f4708d99004c858|72f988bf86f141af91ab2d7cd011db47|0|0|637699172515173775|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=2j1D8LLcgNCeu3M%2FeSP99Jp0YbH6j7aRGj%2Behjtf9D4%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAST5IVBJP4ZDHYDIJUSR333UHBTD3ANCNFSM5GCO5ELA&data=04|01|Ryan.Hecht%40microsoft.com|09b6f2a2ef8f4c961f4708d99004c858|72f988bf86f141af91ab2d7cd011db47|0|0|637699172515183732|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=aDnNGSi7kjNcnsZoZuInS6nk7z4RSdAnfED9LjHVzBM%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|Ryan.Hecht%40microsoft.com|09b6f2a2ef8f4c961f4708d99004c858|72f988bf86f141af91ab2d7cd011db47|0|0|637699172515183732|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=WnLgojb7qHqU962mdzYgzueiB8osnd1XFwwA6Mj6gwU%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|Ryan.Hecht%40microsoft.com|09b6f2a2ef8f4c961f4708d99004c858|72f988bf86f141af91ab2d7cd011db47|0|0|637699172515183732|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=1VwAX4QDqYqlw%2FfGchMRg5CUZvpFXsrDgH3cOlxiLwo%3D&reserved=0.

Yes, here's an example


Pranav paper review

finished range: 1001-1100

Please resolve the conflict

Please resolve merge conflict again, sorry


[NTR] border associated macrophage

Please check that the term does not already exist by using the ontology search tool OLS:
https://www.ebi.ac.uk/ols4/ontologies/cl
Preferred term label
border associated macrophage
Synonyms (add reference(s), please)
CNS border-associated macrophage (PMC10217436)
BAM (PMC10217436)
CNS-associated macrophages (PMC10217436)
CAM (PMC10217436)
Definition (free text, with reference(s), please. PubMed ID format is PMID:XXXXXX)
A type of macrophage located in the non-parenchymal regions of the nervous system including the choroids plexus, meninges and pervascular spaces of the brain.
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC10217436/ ( also check refs in intro in this paper)
https://pubmed.ncbi.nlm.nih.gov/29426702/
Parent cell type term (check the hierarchy here https://www.ebi.ac.uk/ols4/ontologies/cl)
macrophage
Anatomical structure where the cell type is found (check Uberon for anatomical structures: https://www.ebi.ac.uk/ols4/ontologies/uberon)
Your ORCID
0000-0002-7073-9172
Additional notes or concerns

Duplicate with #2285 - some different content in each version of request - needs amalgamating.


There is no method for Extension.Configuration for asp.net core

[Enter feedback here]
I can't access this class Microsoft.Extensions.Configuration
I've added Microsoft.Extension.Configuration over by Nuget Package Manager but still i'm seeing "The Name Configuration doesnt current context" even I added using Microsoft.Extension.Configuration.

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

ID: de9db02d-7310-1141-6aeb-539c02326a36
Version Independent ID: 90062036-aa01-3f73-dac9-b4a092245721
Content: Connection Strings - EF Core
Content Source: entity-framework/core/miscellaneous/connection-strings.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @rowanmiller
Microsoft Alias: avickers

i should use DI


give space ninja a microbomb implant

About the PR
gives the ninja a microbomb instead of death acidifier
Why / Balance
solves #352
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl:

tweak: ninja has a microbomb inplant instead of a death acidifier

real

this doesnt seem to have worked, just killed a ninja and they had an acidifier for some reason


Warnings of "data file translations/en.yaml not found" although file exists

Hi,
I've got a Debian stable host running pandoc <IP_ADDRESS>, pypandoc 1.10, Python 3.11.2 and I do have an issue when running https://github.com/novoid/lazyblorg/ in a virtualenv in case that this matters.
The issue consists of many warning messages, "polluting" the output console with those snippets:
WARNING:pypandoc:Could not load translations for en-US
  data file translations/en.yaml not found
WARNING:pypandoc:The term Abstract has no translation defined.

One for each pypandoc call, which it uses often.
I already "disabled" logging messages via logging.getLogger('pypandoc').addHandler(logging.NullHandler()) as described in the README. This removed some other output but not the one above.
I tried to symlink the translations dir according to hints from https://github.com/jgm/pandoc/issues/6885
mkdir ~/.pandoc/
cd ~/.pandoc/
ln -s /usr/share/pandoc/data/translations .
ls ~/.pandoc/translations/en.yaml   ## shows correct file

That didn't help.
I could not identify any relevant PATH parameter in convert_text() from https://github.com/JessicaTegner/pypandoc/blob/master/pypandoc/__init__.py which would allow me to set the data/translations path.
Furthermore, I did not find any pandoc option I would have been able to use with the extra_args of convert_text().
Somehow, (py)pandoc is not able to locate the existing translation file /usr/share/pandoc/data/translations/en.yaml.
Not that this file is needed in my case but I think it would solve my issue when (py)pandoc would be able to access that file.
Any idea what I could do to fix this?

Hmm. Interesting one.
The log output you provided, suggest that it's "pandoc" that can't find the translation file, however I have never seen this before.
I know running pypandoc in a venv shouldn't effect this. My only suggestions would be to either try to set the data dir argument when using convert_*, check the pandoc documentation, or asking JGM about it as this seems more like a pandoc issue more than a (py)pandoc issue

I may not be that familiar with calling pandoc in the shell. What would be a command line to reproduce the issue without the whole Python layer?
Simple "pandoc input.org -o output.pdf" calls are working without that warning message, for example.

Identical issue, likewise on Debian stable running pandoc <IP_ADDRESS>, pypandoc 1.7.4, and python 3.11.2.
Resolved (for my purposes) by upgrading pandoc and pypandoc from Debian testing to pandoc 3.1.3, pypandoc 1.12.


Updates versions notably Spring Boot, MariaDB and ErrorProne

ErrorProne caught a small bug in cassandra3. MariaDB bump caught a
missing dialect.

gonna merge and see if this helps with aws aurora


[nrf fromtree] net: config: Fix IPv6 setup when DAD is unused and...

Cherry picked from https://github.com/zephyrproject-rtos/zephyr/commit/6edf5bde11680fff5708caadd3bd317381334991 .
It didn't make it on the "upmerge" time. It help fixes part of https://projecttools.nordicsemi.no/jira/browse/KRKNWK-6681 which was reported based on https://devzone.nordicsemi.com/support-private/support/251619 .

The test results does not seem to be parsed properly, there were no sanitycheck failures in the logs, merging.


Bring more config options into YAML (such as the Brimcap root)

A community user had been working off the Brim wiki Remote Workspaces (v0.25.0+) article, employing the technique of using Brimcap to locally index the pcaps that corresponded to logs that had been loaded into a remote Workspace, since this makes the Packets button work in the app. They then asked this question in a message in a public Slack thread (edited slightly for clarity):

Is it possible to configure the "brimcap-root" path for brimcap when it is being used by the Brim app. If this was possible I presume it wouldn't be necessary for me to index the pcaps twice, since the brimcap-root directory on the zed server side is being shared over NFS and would be accessible to my local brim workstation...

There's a couple ways I can see that we could go about this. Since this represents a configurable setting, one approach that came to mind is Brimcap's Config YAML, since the Brim app already has the ability to point to a custom YAML config via a setting in Preferences (see the Brimcap wiki article for Custom Brimcap Config). However, at the moment, only brimcap analyze uses the config YAML for things like pointing to analyzers, shaper configs, etc. In discussing this with @mattnibs, he agreed that we could expand the YAML to cover the other config options used by other commands such as what's currently only specified via -root for  index and search.
While discussing this, @mattnibs had a couple additional thoughts of what we could cover at the same time, and these sound like fine ideas to me also.

We could have an environment variable that points to your config location
I'd also like a config command that prints out the current config. This would be helpful just to see what the default config looks like.

I will follow this issue with keen interest!

I've verified this functionality is working in Brimcap release tagged v1.0.4, though the Brim app is not able to fully make use of it yet. See https://github.com/brimdata/brim/issues/1821.
Using this YAML config based on what's shown in the Custom Brimcap Config wiki article:
$ cat zeek-suricata.yml
root: /home/phil/other-root
analyzers:
  - cmd: /usr/local/bin/zeek-wrapper.sh
    name: zeek
  - cmd: /usr/local/bin/suricata-wrapper.sh
    name: suricata
    globs: ["eve.json"]
    shaper: |
      type port=uint16;
      type alert = {
        timestamp: time,
        event_type: bstring,
        src_ip: ip,
        src_port: port,
        dest_ip: ip,
        dest_port: port,
        vlan: [uint16],
        proto: bstring,
        app_proto: bstring,
        alert: {
          severity: uint16,
          signature: bstring,
          category: bstring,
          action: bstring,
          signature_id: uint64,
          gid: uint64,
          rev: uint64,
          metadata: {
            signature_severity: [bstring],
            former_category: [bstring],
            attack_target: [bstring],
            deployment: [bstring],
            affected_product: [bstring],
            created_at: [bstring],
            performance_impact: [bstring],
            updated_at: [bstring],
            malware_family: [bstring],
            tag: [bstring]
          }
        },
        flow_id: uint64,
        pcap_cnt: uint64,
        tx_id: uint64,
        icmp_code: uint64,
        icmp_type: uint64,
        tunnel: {
          src_ip: ip,
          src_port: port,
          dest_ip: ip,
          dest_port: port,
          proto: bstring,
          depth: uint64
        },
        community_id: bstring
      }
      filter event_type=="alert" | put this := shape(alert) | rename ts := timestamp

Invoking it in the index phase, we see the setting is used, as the root directory is populated with the index entry.
$ brimcap index -config zeek-suricata.yml -r sample.pcap

$ ls -l /home/phil/other-root
total 416
-rw------- 1 phil phil 424864 Sep  8 18:57 idx-aG5tLflou0WVcgKcj8VWAd30gui9d3SitZcqUahrKO4.json

Likewise during the search phase:
$ brimcap search -config zeek-suricata.yml -dst.ip <IP_ADDRESS> -dst.port 20669 -duration 2us -proto tcp -src.ip <IP_ADDRESS> -src.port 55811 -ts 2018-03-23T20:00:14.328598Z -w /tmp/mypcap.pcap

$ ls -l /tmp/mypcap.pcap
-rw-rw-r-- 1 phil phil 176 Sep  8 19:00 /tmp/mypcap.pcap

$ tcpdump -r /tmp/mypcap.pcap
reading from file /tmp/mypcap.pcap, link-type EN10MB (Ethernet)
13:00:14.328598 IP <IP_ADDRESS>.55811 > <IP_ADDRESS>.20669: Flags [S], seq<PHONE_NUMBER>, win 1024, length 0
13:00:14.328600 IP <IP_ADDRESS>.55811 > <IP_ADDRESS>.20669: Flags [S], seq<PHONE_NUMBER>, win 1024, length 0

That extracted flow isn't much to look at, but it shows it worked. 😉
Keep an eye on https://github.com/brimdata/brim/issues/1821 for app-side updates.
Thanks @mattnibs!


[convert] write_to_h5 not keeping the order of groups

I think there is a bug in write_to_h5 because the order of groups is not kept, even when track_order is passed to h5py.
import tempfile
import h5py
from silx.io.commonh5 import Group
from silx.io.convert import write_to_h5
#: create a group and write to file
g = Group("")
g['z'] = 10
g['a'] = 5
g['b'] = 6
g['c'] = -1
ft = tempfile.mktemp(suffix='.hdf5')
write_to_h5(g, ft, create_dataset_args=dict(track_order=True))
#: read back the file
h5 = h5py.File(ft, mode='r')
#: check the order of groups in the initial object and the read-back one
g.keys()
# Out: odict_keys(['z', 'a', 'b', 'c'])
h5.keys()
# Out: <KeysViewHDF5 ['a', 'b', 'c', 'z']>

I have tried a couple of things in this branch : https://github.com/maurov/silx/tree/hdf5writer_track_order
Unfortunately, it still does not work...
Any help is welcome.

I forgot to add that track_order should be added also here:
https://github.com/silx-kit/silx/blob/e55044960d203eacf81033802cef060814339714/silx/io/commonh5.py#L909

Hi. I will try to check your problem.
But i think it would be better for the writer to check that g was using track_order (which is then the case by default for our commons.Group), instead of forcing a configuration for the full tree. But a public h5py API have to exists to check the track order state.

Hi again. Your problem is tricky cause i guess the root group pre-exists. Maybe it is also possible to create a file with track_order option.
Then if you use:
g['foo/z'] = 10
g['foo/a'] = 5
g['foo/b'] = 6
g['foo/c'] = -1

and
h5["foo"].keys()

your will see that you have already fixed the problem.
I guess your main use case is not relative to tracking order on the root?

I check h5py. A File is a Group.

You could use track_order at the file creating, it should track the order for the root.
A note about track_order in create_dataset_args. As it is used for the dataset factory, this is used to track the creation of the attributes. That's maybe not what you want.

As reminder
def get_track_order(group):
    v = group.id.get_create_plist().get_link_creation_order()
    is_tracked = (v & h5p.CRT_ORDER_TRACKED) == h5p.CRT_ORDER_TRACKED
    is_indexed = (v & h5p.CRT_ORDER_INDEXED) == h5p.CRT_ORDER_INDEXED
    if is_tracked and is_indexed:
        return True
    elif not is_tracked and not is_indexed:
        return False
    else:
        # In between, unsupported by h5py hi-level API
        return None

filename = tempfile.mktemp(suffix='.h5')
with h5py.File(filename, mode='w') as h5:
    a = h5.create_group("a", track_order=True)
    track_order = get_track_order(a)
    print(track_order)
    b = h5.create_group("b", track_order=False)
    track_order = get_track_order(b)
    print(track_order)

@vallsv thanks for the comments. Well, I would like to keep track_order=True also in the root group. Unfortunately, even with h5py, I do not understand why when reading back the file, the order of the root group is lost:
import tempfile
import h5py
ft = tempfile.mktemp(suffix='.hdf5')
h5 = h5py.File(ft, mode='w', track_order=True)
h5.create_group('z')
h5.create_group('a')
h5.keys()
# Out: <KeysViewHDF5 ['z', 'a']>
h5.close()
h5 = h5py.File(ft, mode='r', track_order=True)
h5.keys()
# Out: <KeysViewHDF5 ['a', 'z']>

By the way, the order is correctly kept in subgroups created with track_order=True. Maybe I am doing something fundamentally wrong with the root group...
PS: what is  h5p in  get_track_order definition?

@maurov
I guess your comment is worth opening an issue at the h5py repository.

Yes it looks like a bug on there side. BTW, you can remove the track_order on the read part.
Sorry, h5p come from h5py
from h5py import  h5p

No feedback from h5py, just closing the issue as this is not related to silx anymore.


Modeladmin needs fixing for non-superusers

flake8 currently failing on https://github.com/torchbox/wagtail/blob/master/wagtail/contrib/modeladmin/helpers.py#L182 due to the missing import of UserPagePermissionsProxy, which suggests that that particular code path needs fixing / testing...

Yup, I'm seeing that too with my latest PR commits in #2463.

:+1:

Fixed by #2479. It was tested by the way, it must've been a bad rebase. sorry about that.


Search on example repo does not seem to work in production instance

Describe the bug
No results in any query

I think the repo was somehow outdated. I re-crawled it just now and the search seems to work again.


chore: updating/adding README, CONTRIBUTING and code of conduct #1

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Richard Browning seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


fix wrong choose for module

webpack default options for resolve.mainFields is module.In everyday habits we ignore node_modules in babel plugins.so if we ues es6 it will have some mistakes in the browser which is lower version

Everything is correct, it should bundle module by default


Use Set to deduplicate any extra versions

JIRA Ticket: https://fedora-repository.atlassian.net/browse/FCREPO-3655
What does this Pull Request do?
Fixes the application/link-format output when there are more than one version in a second. The RDF formats seem to automatically deduplicate, but we manually generate this format.
How should this be tested?
This is most obvious with an automated script because you need to generate more than one version in a single second.

Once you have generated multiple versions in a single second, then GET the Timemap (/fcr:versions) as text/turtle and see only one version per second.
Get the Timemap with Accept: application/link-format and see duplicate versions.
Pull in this PR and do the second GET again and see only one per second

Interested parties
@fcrepo/committers

I accidentally merged this into 6.2.0-RC .  I will push this to main right now.


Transform declaration

I'm under the impression that there is a significant performance cost when global variables aren't defined as constants. I think I read a section about this in the Tom Kwong book. @BrianGun do you know any more about this, or does it not really matter? E.g. https://github.com/microsoft/OpticSim.jl/blob/6253fba6928ee37494410e3581d358a50229fb3e/src/Geometry/Transform.jl#L180

This should be okay because it is an alias for a type declaration. You would use it like so:
a = Transform{Float64}
where a is a variable.

I understand the usage, but I'm concerned because the compiler can't guarantee that Transform won't be re-assigned to something (anything) else during runtime. Since it's not a const, the compiler can't treat it differently just because it's assigned to a type declaration here.
Even if it doesn't affect performance, we should change this to a const for good code style.

The compiler treats these type declarations as const. Just tested this: you get an error if you try to redefine Transform{T} to something other than its original definition.
So it is not necessary to mark them as const.

Ah that's good to know
https://docs.julialang.org/en/v1/manual/types/#Type-Aliases

That's good to know, I didn't realise they were treated differently. So it comes down to code style preferences. I think JuliaLang tends towards const - having an explicit declaration is a good idea imo.
https://docs.julialang.org/en/v1/manual/types/#Type-Aliases
Also, I forgot to mention, my reason for noticing this is that Documenter.jl wasn't building the docstring for Transform. This is because it only processes types, functions and consts.
So I guess there's no performance benefit, but it's still worth changing.

@alfredclwong  did you make the change to const Transform? Can we close this issue?

I'll start work on a PR soon to address various doc build issues and I'll group this in with it.


Add the option to disable linking against the main crate

Hi,
I have a suggestion for an improvement: At the moment, the archives produced by a build script using gcc-rs are unconditionally linked to the crate that is being built. But there are certain use cases where that is not desirable, for example when you only want to link the built artifact for tests. This is presently only possible by dropping down to constructing the compiler command line manually, which is not really desirable for portability reasons.
So I propose adding a link() option that you can add to the configuration in order to disable linking. The default would still be true, of course. Please have a look and tell me what you think.
Thanks,
Niels

Sounds like a good idea to me, thanks! Could you also guard the print for rustc-link-search and the C++ standard library below on this flag?
I also think that link may not be the best name as it generally implies that the linker itself is being invoked, perhaps something like cargo_metadata?

Could you also guard the print for rustc-link-search and the C++ standard library below on this flag?

Actually there's small convenience upside in keeping the println statement for the search path, but that might be trumped by consistency concerns (either you emit metadata for cargo or you don't), and it's not that difficult to get the reference to the OUT_DIR and add that yourself.

I also think that link may not be the best name as it generally implies that the linker itself is being invoked, perhaps something like cargo_metadata?

Yep, that's a good idea.

Yeah these are basically all pure convenience (e.g. doing the prints for you) so I'd prefer to be consistent and not print any metadata if this option is specified, and as you mentioned you can always print the OUT_DIR as a search path in the script afterwards.

I'd prefer to be consistent and not print any metadata if this option is specified

Agreed. That's what the updated PR does now.

Thanks!