Player is not getting damage

Player character is not being damaged by any of the cases:

Vehicle Explosion
Zombie Bite
I'm getting error in console, but it's not included in the Rocket.log file. This is the gif of how it looks like in console GIF and this is it's part which I copied:

NullReferenceException: Object reference not set to an instance of an object - Rocket.Unturned.RocketUnturnedHost.OnPlayerDamaged (SDG.Unturned.Player uPlayer, SDG.Unturned.EDeathCause& cause, SDG.Unturned.ELimb& limb, Steamworks.CSteamID& killerId, UnityEngine.Vector3& direction, System.Single& damage, System.Single& times, System.Boolean& canDamage) (at <970f00d6a2c841f2a29c108bde8ba95e>:0)
SDG.Unturned.DamageTool.damage (SDG.Unturned.Player player, SDG.Unturned.EDeathCause cause, SDG.Unturned.ELimb limb, Steamworks.CSteamID killer, UnityEngine.Vector3 direction, System.Single damage, System.Single times, SDG.Unturned.EPlayerKill& kill, System.Boolean applyGlobalArmorMultiplier, System.Boolean trackKill, SDG.Unturned.ERagdollEffect ragdollEffect) (at <2de13a8df2bb4bd79a2f1071f29293c3>:0)
SDG.Unturned.Zombie.tick () (at <2de13a8df2bb4bd79a2f1071f29293c3>:0)
SDG.Unturned.ZombieManager.Update () (at <2de13a8df2bb4bd79a2f1071f29293c3>:0)

Fixed

Regression of #1093

How can i fix that problem??


Training match the paper performance

HI,
I am trying to follow your instruction to match the result of the paper usubg NYU dataset. But the mIOU and RMSE are still can not be the same. They get stable after 300 iterations and stop at 25% and 0.7. Is there anything I missed or lost in the instructions? Thanks!
Best
Kuo

HI,
I am trying to follow your instruction to match the result of the paper usubg NYU dataset. But the mIOU and RMSE are still can not be the same. They get stable after 300 iterations and stop at 25% and 0.7. Is there anything I missed or lost in the instructions? Thanks!
Best
Kuo

Have you achieved the multi-taskes by these codes?

HI,
I am trying to follow your instruction to match the result of the paper usubg NYU dataset. But the mIOU and RMSE are still can not be the same. They get stable after 300 iterations and stop at 25% and 0.7. Is there anything I missed or lost in the instructions? Thanks!
Best
Kuo

Have you achieved the multi-taskes by these codes?

To run multi-tasks is no doubt.
However, I can't fully reproduce the training procedure mentioned in the paper to get the claimed performance.
Interesting thing is that the provided model can reach the claimed performance of the pape.

Me too, I also trained the example/multitask/train.py,  aka mobilenetv2+MTLWRefineNet without any change.
Finally, I get the mIOU=0.27, RMSE=0.72 with 1000epoch.
It didn't  achieve the paper-claimed performance.
Anyone knows why?

See the following script for multi-task training with MobileNet-v2 and Multi-Task Light-Weight RefineNet: https://github.com/DrSleep/DenseTorch/blob/dev/examples/scripts/segm-depth-normals-mbv2-rflw.sh
Without changes, it should reach ~37% mean iou, 0.60 RMSE and 24 mean angular error. To achieve higher numbers, as described in the paper, you have to use the Raw NYUD dataset with knowledge distillation on missing segmentation labels -- this is not provided by this repository.


Ticket #34417 - Remove unnecessary recreation of foreign key constrains while dropping index.

Ticket #34417

Looks good to me - just need to get CI passing.

@avesda Do you have time to keep working on this?

Closing due to inactivity.


Completely remove TFM as a dependency and use reflections instead

Simplify removing color codes by using ChatColor#stripColor
Have an option to use sqlite or mysql, by default it will be sqlite
Remove TFM from the pom.xml and use reflections to access TFM methods to prevent a circular dependency

Codacy detected an issue:
Message: The utility class name 'Reflections' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Constants)'
Currently on:

Commit:  11b3e19
File:    src/main/java/me/totalfreedom/tfguilds/util/Reflections.java
LineNum: 6

Regarding this one, I think that's probably fine to ignore, it's a rule I think we turned off in the end on the main TFM Repo, but the first comment is one we need to resolve.


Fixed the bogus dev-dependencies pull in on regular build

Fixes #1796

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Thanks! Could you also add a test for this?

This patch breaks test_cargo_test::example_dev_dep.
My own opinion is that the example code shouldn't pull in the dev deps, as having example binaries being cross-compiled is perfectly valid use case while I can hardly imagine the need to use any additional code in examples that are not covered by main crate deps. Any advice on why that test exists in a way it exists?

That's a test for how examples are specc'd which is that examples have access to dev-dependencies (e.g. it's a legit failure)

After discussing over irc I now understand better how exactly it is legit. I will try to dig deeper to make a fix for original issue in #1796 without breaking the example use case.

This seem to be a much bigger issue then two LoC change, I'm closing this PR continuing my investigation in #1796.


Add migration to Nightly wallet

feat: add nightly-wallet

fix: change to the way nightly extension opens

fix: improving singletons

This PR is no longer relevant as wallet team implementation will be handled from wallet-selector repo
(Nightly is already on the list of mainnet ready list)


Improve Edge Router connection life-cycle

some ideas:

keep failed edge router connections to avoid high connection attempt churn
expose edge routers status to the app

fixed in #191


VS Code will not start when installing from Marketplace from Chrome

MacOS
stable 1.16.0
Force Quit Stable
In Chrome, browse to the marketplace
Click on "Install" on any extension
Result: The stable icon just bounces, never starts
If I use Safari, Code will start correctly.
In this gif you can see the code icon bounce but it never starts (sorry its clipped):

I know it sounds like a joke but... did you try restarting your machine?

heh heh heh... I restarted and it works.


Newly released intl-tel-input v14 removed jQuery dependency!

Please update package dependency to new version!

Closed as maintainer upgraded intl-tel-input.


Support something akin to getInitialProps on App component

Ideally, similar to Next.js, each route could define a function that returns a Promise, which will defer prerendering until it resolves. This allows components to fetch their initial data (eg from a CMS) before prerender happens.
I'd love to open a PR supporting this. I went through the code to see how I would implement it, and I have a couple of questions.
Currently, an ssr function is defined in the html-webpack-plugin config. The problem with the ssr happening here, as part of the webpack plugin's template step, is that it must be synchronous. Is there any good reason it's defined here? It looks like the ssr function does take parameters, but they aren't used in any meaningful way.
Instead, I'd like to do the "SSR" before html-webpack-plugin runs so that we can asynchronously render the Preact app, and then just store the SSR result as a string in the html-webpack-plugin config, which the webpack plugin can then spit into its template as a variable.
And then we could do the SSR somewhere else, but I'm not sure where would be best. I'm thinking where the list of prerender urls are defined.
Does my suggestion sound like the correct way to implement this, and if so, do you have any pointers about which part of the prerender lifecycle in which to implement the async SSR step?

In the long run, it would be best to have getInitialProps on each component rendered by the Router, but that would require preact-cli to be able to get the component matching the current route. I've heard that this functionality is in the works (?), but that sounds like a later step anyway. We can get basic getInitialProps on the App component with async prerendering, that's most of the work done.

I'd be up for moving ssr out of the template. String rendering is still synchronous though, if we wanted to support something like getInitialProps I think we'd need to switch to something like preact-render-to-vdom and then have preact-cli inject code into the router to invoke and re-render in response to getInitialProps().

Sounds way more complicated. Couldn't we technically just do it on the top-level component only (since we already have an instance of that) and let the application handle the orchestration?

This would be awesome. Did this hit a stand-still?


Cutout material with outline fills in full mesh on other side

I don't know how to explain, but whenever I use an outline with the material, it fills in the rest of the mesh from the other side. This issue does not occur with Poi Toon

If you set the same texture as the main texture for the outline, or set another texture with transparency, it will become transparent.


VK1621

Hi, nice work with the teardown!
I would like to add some info, I don't think its worth it to make a PR on this.
You are mentioning the chip as a VINKA VK15218B MCU, I did some research on this and I found out it is probably a VK1621S LCD driver checkout datasheet here.
Think this could be useful for more reverse engineering!

I've just put a ESP32 inside (running esphome with bluetooth proxy) connected to the test pads next to the connector of the sensor and get the reading direct from the SEN54 sensor (all readings not just PM2,5 Hum and temp) display still works as well :)

@johanneswilkens thanks a lot! I will add it as soon as I get back to my laptop.
@Maart3nL wow, that sounds great! Would be nice if you could share the parser code (with PR or just the code snippet here)

I used the default from the sen54 sensor from esphome
https://esphome.io/components/sensor/sen5x.html

Ah, ok, thanks. Should have googled the library for this before 😁

Very cool, @oleksiikutuzov. Do you have a pinout anywhere showing how you connected the esp32?

Does anyone have a guide for this?

I've just put a ESP32 inside (running esphome with bluetooth proxy) connected to the test pads next to the connector of the sensor and get the reading direct from the SEN54 sensor (all readings not just PM2,5 Hum and temp) display still works as well :)

Hello, would you be able to put together a guide for doing this? I know nothing about ESP yet, but exposing the PM10 would be a big deal for me. Ideally on zigbee not bluetooth.

Shame it doesn't ventilate the inside.. could have added mh-z19 co2 to the esp32 aswell...

Instead of Winsen MH-Z19 I would choose Sensirion SCD4x family (photo-acustic sensor instead of NDIR). Smaller footprint and I2C.


Can't apply changes

LOG: onCreate, launching app...
LOG: Preparing UI
LOG: Device supports Overlay-Manager-Service
LOG: There is no overlay installed, refreshing preferences
LOG: Getting default values from device
LOG: Creating User Interface according to default values
LOG: Starting overlay process
LOG: isCreatable is true
LOG: Saving user input values to res file
LOG: Everything is ready, lets compile it
LOG: OK, compiling...
LOG: Overlay apk file is not found, failed

Device: Moto G4 plus
Root: Magisk
ROM: Custom (Cardinal AOSP)
Android Version: 8.0.0

same issue here.
It says that "Overlay apk file is not found, failed"
I tried for OMS and PRO.
LOG: onCreate, launching app...
LOG: Preparing UI
LOG: Device supports Runtime-Resource-Overlay
LOG: Device is not nexus device
LOG: There is no overlay apk installed in vendor directory, refreshing preferences
LOG: Getting default values from device
LOG: Creating User Interface according to default values
LOG: Starting overlay process
LOG: isCreatable is true
LOG: Saving user input values to res file
LOG: Everything is ready, lets compile it
LOG: OK, compiling...
LOG: Overlay apk file is not found, failed


Harmonize fields for better logging

This helps when debugging issues and is good for a cleaner code base in general

bors r+

bors retry

bors failed on payouts::quanto::api_tests::payout_intervals_have_no_gaps which will be solved by https://github.com/itchysats/itchysats/pull/2650


eslint.autoFixOnSave not working

VSCode Version: Code - Insiders 1.8.0-insider (7499a9f3c565f4fbe5205e30c8664e5a5a11ae94, 2016-12-06T16:10:17.195Z)
OS Version: Darwin x64 15.6.0

Steps to Reproduce:

testing #16497
follow instructions in last bullet of #16497

enable auto fix on save using the following workspace setting "eslint.autoFixOnSave": true (note as with format on save this is only available if "files.autoSave" is off or on focus lost.). Ensure that fixes are applied on save and that the fixes are applied at the correct positions.

use "files.autoSave": "off"
open .js file with too many semicolon such as

let rudi = 'rudi';

function rudi(valdor) {
    return rudi;;;;
}

const bill = 'bill';

bill = rudi;;;;;

-> no corrections are applied on save

Works for me:

Note that you need to set "eslint.autoFixOnSave": true

Removing the bug tag since it is not reproducible


Udeler Chrome Extension

Please add your extension to Chrome Web Store. Without adding that extension to Chrome Web Store, we can't use that. Read more about it here.

Same here unfortunately. Authenticator extension not working as it's permanently disabled and can't be enabled. Login via e-mail doesn't work and trying to login with google just complains that the browser may not be secure.

Same here unfortunately. Authenticator extension not working as it's permanently disabled and can't be enabled. Login via e-mail doesn't work and trying to login with google just complains that the browser may not be secure.


NfcManager.start doesn't properly remove the iOS event listener.

I created an NFC Screen & I've decided to initialize the NFC when I navigate to it. In my componentDidMount function I have the code shown below.
componentDidMount = () => {
    NfcManager.start({
      onSessionClosedIOS: () => {
          console.log('session closed');
          this._stopDetection();
          this.props.navigation.goBack();
       },
    });
};

This works just fine the first time around. I'll navigate to the screen, then on cancel it'll take me back, then I'll navigate to the screen again & when I click cancel I get the following error message.

I took a look at the source code & found that the issue is that Events.SessionClosed event listener is being removed properly. So I've submitted a pull request #8 with a fix for this. :-)

Hi, thanks for spotting the issue!
The PR is modified a little and already merged.


Dev kobeya 754: Fixed Broken UI Skeleton File

[x] Unit Testing
Nothing to visually inspect.

Live edit allows code changes to reflect instantly without restarting the spring app. I will figure out a place to document it.

Looking at the most recent version of master on our UI it seems as though these changes have already been applied.


fix: Change identifier used for the new runtime

Description
Change the identifier of the new runtime from csharp-azurewebapp-v2 to the more descriptive and accurate adaptive-runtime-dotnet-webapp
Also includes backwards compatibility for the old name so that people who have created bots in the pre-release period will still work.
Task Item
fixes #6338
Component side is here:
https://github.com/microsoft/botframework-components/pull/721

@benbrown is the name set? I recall talking about a rename for R12

@benbrown is the name set? I recall talking about a rename for R12

This is the change we discussed.


Add support for collision layers and masks in CSG shapes

Code mostly copied from PhysicsBody. Not having this hampers the usability of CSG shapes, so it might make sense to have it for 3.1. Not sure if the "use_collision" flag should be removed as a result of this.

@reduz mentions it should be fine to merge, could you rebase to fix the conflicts?

I pushed a rebased commit myself, will merge once CI tests are OK.

Thanks!


Random DG bug fixes

Here are the things that I have fixed:

Unbold-ed the subheaders for the use cases

Codecov Report

Merging #278 (e9e0cc2) into master (bf8099d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #278   +/-   ##
=========================================
  Coverage     72.78%   72.78%
  Complexity      754      754
=========================================
  Files           105      105
  Lines          2436     2436
  Branches        325      325
=========================================
  Hits           1773     1773
  Misses          537      537
  Partials        126      126

Continue to review full report at Codecov.

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


Fix bugs in rerunning jobs

Fix #28761
Fix #27884
Fix #28093
Changes
Rerun all jobs
When rerun all jobs, status of the jobs with needs will be set to blocked instead of waiting. Therefore, these jobs will not run until the required jobs are completed.
Rerun a single job
When a single job is rerun, its dependents should also be rerun, just like GitHub does (https://github.com/go-gitea/gitea/issues/28761#issuecomment-2008620820). In this case, only the specified job will be set to waiting, its dependents will be set to blocked to wait the job.
Show warning if every job has needs
If every job in a workflow has needs, all jobs will be blocked and no job can be run. So I add a warning message.

It's better to have some tests.

It's better to have some tests.

Yes. I will add some tests and more explanation later

Looks like when rerunning a job, it will return an error.

Caused by a missing ctx.JSON. Fixed in 5021f55b1663e63ce0fda2247f15ddf72f52e220


Unsafe ERC20 Operation(s)

ERC20 operations can be unsafe due to different implementations and vulnerabilities in the standard.
It is therefore recommended to always either use OpenZeppelin's SafeERC20 library or at least to wrap each operation in a require statement.
To circumvent ERC20's approve functions race-condition vulnerability use OpenZeppelin's SafeERC20 library's safe{Increase|Decrease}Allowance functions.
In case the vulnerability is of no danger for your implementation, provide enough documentation explaining the reasonings.
../talentlayer-id-contracts/contracts/TalentLayerArbitrator.sol::127 => payable(msg.sender).transfer(dispute.fee); // Avoid blocking.
../talentlayer-id-contracts/contracts/TalentLayerEscrow.sol::488 => IERC20(proposal.rateToken).transferFrom(sender, address(this), transactionAmount),
../talentlayer-id-contracts/contracts/TalentLayerEscrow.sol::981 => IERC20(_tokenAddress).transfer(_recipient, _amount);

https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/contracts/token/ERC20/IERC20.sol#L43


Pushing a New Version to the app store

Proper name #85 "Embroidermodder Mobile"?

Menu: Open File. Works. Loads recent, but if you select a proper embroidery file, it loads.

Menu: Statistics. Fails. Loads dialog, but displays nothing.

Menu: Share. Works. Displays list of formats to exports.

.cvs should be csv.
displays disabled unavailable formats. These are not included but might be distractive.

Menu: Load File. Fails. Loads dialog, but displays nothing.

This is thumbnail viewer.
"Load File" and "Open File" are synonymous names for different function.

Drawer: works.

This displays giant, and seemingly unhelpful recycler view of "stitch x, y" for thousands of stitches.

Drawer:
Displays

Metadata
Threads
Stitches.

The problem with the threads is if it had no threads to start, it doesn't force them to add as random. Loading something like a .dst file gives no thread colors, so it doesn't have them. Loading something like a .jef or .pes it has a color and can be changed accordingly with that view.

Fixed Statistics.
Upgraded thumbnail viewer to newer version that I made for TE. Can certainly be made significantly better but this is better than what was there, and actually works.

Fixed csv spelling error. Fixed menu name for "Load File" -> "Folder".
Changed app name to simply "Embroidermodder"

Autofixed the colors on load, tweaked the fragment close order.

@JoshVarga Okay. That should be it for the stuff that needed updating or seemed wrong. Not adding more features but I hooked all the parts up. So any tweaks you wanna make are good and I shouldn't end up second guessing them or changing something.
There's a couple strings that aren't translated. But, I don't speak those languages so I can't fix them. And a couple moved not-deleted files (but those will be dropped anyway so it's not like that cleaning is actually helpful).
I'm also good with just sending that version.

Did this hit a snag or something?

@JoshVarga you did something again. This is still kinda neglected. I've updated Embroidepy/pyembroidery and am planning on a similar update to EmbroideryIO. Which does some stuff like Jef trims and DST trims and PES stops. And am still curious as to your position on including it as a library vs. directly imbedding the the project.

I think it would be best to move forward with using EmbroideryIO as a library, so that we don't need to make updates in multiple spots. Getting out of sync gets really nasty. I imagine if there are any significant updates that I need to make you would be open to pull requests. If not we can always embed it again on this project.

The version here should be what I figured was solid enough for release last year..
The only thing I've done recently for pyembroidery is add a merge pattern method, a bunch of python specific dunder methods, and adding in trim and stop interpolations into jef, dst and pec/pes. And likely will have to port some of those changes into EmbroideryIO. I'm also mulling around adding in the color-file formats. So .EDR and .COL and .INI and then I could use the same STOP interpolation routine I specifically used for PES/PEC for things like .DST. If it loads the EDR and has a double color to the same color, it would make that a stop in the loaded DST rather than color change. The same could hold true for things like DST with an extended header, as that DST modification does include give it threads and those could duplicate. --- But these are all pretty rare edge conditions now.
I'll toss in an invite for you on EmbroidePy. The packaging for EmbroideryIO android and jvm is just done through jitpack so it only requires updating the master branch and adding a release version. Which is pretty nice. I use it in a different private JVM package as well, and as part of TouchEmbroidery. If everything else failed it could just be forked to Embroidermodder and the jitpack bit would work the same but just reference Embroidermodder rather than EmbroidePy.

A version was pushed.


Unnecessarily expect an explicit return

Following gives this function should explicitly return a value warning when it shouldn't.
const SOME_CONST = ...;

function f() returns boolean? {
    int i = 0;
    while true {
        if i == SOME_CONST {
            return false;
        }
    }
}

@manuranga this is the spec issue https://github.com/ballerina-platform/ballerina-spec/issues/973 for this change. With this change, you need to have a return stmt at the end of the function to avoid the warning.

@hasithaa The end of the function is not reachable (because the while condition is singleton true and there is no break), so this is a bug.

Ahh, My bad, I missed it.

but shouldn't the while block have a return when not satisfying the i == SOME_CONST condition?

@lasincl It's not an error to have a non-terminating loop.

Fixed in https://github.com/ballerina-platform/ballerina-lang/pull/33729


Duplicate key (bpfExtToServiceConnmark) in CRD causes kustomize to fail

Expected Behavior
Running kustomize should work.
Current Behavior
Running kustomize fails with error:
Error: map[string]interface {}(nil): yaml: unmarshal errors:
  line 61: mapping key "bpfExtToServiceConnmark" already defined at line 55

Possible Solution
There's a duplicate key (bpfExtToServiceConnmark) in the CustomResourceDefinition properties.
Steps to Reproduce (for bugs)

create file kustomization.yaml:

apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
resources:
  - https://docs.projectcalico.org/v3.20/manifests/calico.yaml

kustomize build system/network/calico/

Context
Kustomization is currently broken. The published YAML is likely invalid, and Kube could be swallowing the errors.

Hi @DanWard , thanks for bringing this to our attention. This should be fixed with https://github.com/projectcalico/calico/pull/4805. I just checked the v3.20 manifest and kustomize should work again for it as well. Let us know if you find any other issues.


plaits model 8 has a chance of not initializing properly

narrowing down on #1589
a [plaits~ -model 8 440 0 0 0.7] can be frozen in a state where it outputs -1 dc. it happens at random, sometimes when duplicating that object. or if you have many of them in a patch, some will have that issue when opening that patch. tested on linux and windows
test patch: plait-debug-9-8.pd.zip
(added -trigger to keep it quiet, but the issue does happen with or without trigger mode)

caught in on camera here, just to give you an idea of how random it can be, sometimes you're lucky, other times, it can take a while :p
https://user-images.githubusercontent.com/86204514/229537299-dcc094f4-2e27-4388-8aa9-1bd6ed21add8.mp4
.
and here's how it is when opening the patch
https://user-images.githubusercontent.com/86204514/229538531-d0744d19-15d6-46bf-a6cb-6909ff657643.mp4


Change floating point precision "on the fly"

small precision at the start of optim, higher precision later

quite complex, but possible.


Integrate feedback with on prem products

Integrate react feedback with on prem products.

OpenShift PR: https://github.com/openshift/console/pull/12611

OpenShift PR merged, PR created to backport user feedback to previous version: https://github.com/openshift/console/pull/12637

OpenShift has decided not to backport user feedback so we can close this issue.


General QOL improvements

Todo:
Add ThroughputMonitor
Add measure_flops
Add option to overwrite in inference (so we can e.g. continue from broken runs)
Remove additional large keys in checkpoints (new_sizes and new_spacings for now)

@Sllambias for flop counting you can see https://github.com/asbjrnmunk/agave/blob/main/src/count_flops.py :)

reviews of course conditioned on passing automatic test and the manual tests i've written above


carla can support the millimeter wave radar model?

the radar model support Ultrasonic radar ,can it support the mmw radar?

@DSantosO could you have a look at this please?

stale

How supports ultrasonic?


MP2 Audio support

Hi, is it possible to support MP2 audio? I'm willing to provide a source with MP2 audio for devs to test (but unfortunately I'm cannot post it in public).

Ya I think the Media Source Extensions doesn't support MP2 codec

Is it possible that we play this through some kind of magic, e.g. use wasm to software-ly decode it?

Yes, it's possible by using WebAssembly and WebAudio.

It seems that now mpegts.js supports the audio format with the MP3 update. Thank you for your hard work!


fixed bug with org.socialsignin.spring.data.dynamodb.repository.support....

...SimpleDynamoDBCrudRepository#findAll(java.lang.Iterable)

Thank you for raising this issue and for the fix regarding the correct handling of overridden table names.  This should now be resolved in the latest snapshot


Illegal character in path

I am new to using conductor so apologies if I am missing something obvious.
I have a work flow defined with two http tasks.
Task 1: Returns a Jason Object
Task 2 : Uses one of the Output fields  of the JSON Object of the first task  as an Input Parameter .
So in essence
[
    {
      "name": "tsk_1",
      "taskReferenceName": "tsk_1",
      "type": "HTTP",
      "startDelay": 0,
      "inputParameters": {
        "http_request": { "uri":"http://mysite/Method1/${workflow.input.clientId}",
          "method": "GET"
        }
      }
    },
    {
      "name": "tsk_2",
      "taskReferenceName": "tsk_2",
      "type": "HTTP",
      "startDelay": 0,
      "inputParameters": {
        "http_request": {
          "uri": "http://mysite/Method2/${workflow.input.clientId}/${tsk_1.output..Field1}",
           "method": "GET"
        }
      }
    }
  ]

tsk_1 works and executes correctly  , but for  tsk_2 the input gets parsed as :

   "http_request": {
      "uri": "http://mysite/Method2/1/[1,2,3,4]",
   "method": "GET"
   }
}

And get the Output as
{
   "response": "Illegal character in path at index :26 http://mysite/Method2/1/[1,2,3,4]
}

I am guessing the issue is because of the “[” and “]” brackets.
Posting “http://mysite/Method2/1/[1,2,3,4]” directly on the browser works as expected returning the expected JSON Output .
Not sure what I am missing here ?

What is the clientId passed as when starting the workflow?


docs: update api documentation for v1.2.

maybe we should really be fixing this by fixing it (#369)

This has caused me some pain as well though I don't have a great fix in mind. Agreed lowering the threshold might solve most issues.


Show when there's a blocking fast path operation

Motivation

Most fast path operations are fast, but some are quite slow. As a
heuristic, choose to report when the fast path is about to run on 100+
files. We may want to adjust this up or down, depending on how much it
causes flickering.
Test plan

[ ] @jez Write tests for this

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_M3fyqvHOlwVKHm
→ https://go/builds/bui_M3fyPUybNdgTwl


Cleanup and function references

Just tiny clean up and simplifications by function references and specialized assertions

No, it isn't

Thanks @mrsoto We'll check the fix and merge it if everything look ok

@amit2103  I'll merge the PR; let's add the update you mentioned afterwards


Validation errors and warnings related to SVG files

Create a page with this Markdown:

example.md
> [!ABSTRACT]
> This is an abstract.

> [!CAUTION]
> This is a caution.

> [!CODE]
> This is code.

> [!CONCLUSION]
> This is a conclusion.

> [!DANGER]
> This is a danger.

> [!ERROR]
> This is an error.

> [!EXAMPLE]
> This is an example.

> [!EXPERIMENT]
> This is an experiment.

> [!GOAL]
> This is a goal.

> [!IDEA]
> This is an idea.

> [!IMPORTANT]
> This is important.

> [!MEMO]
> This is a memo.

> [!NOTE]
> This is a note.

> [!NOTIFY]
> This is a notification.

> [!QUESTION]
> This is a question.

> [!QUOTE]
> This is a quote.

> [!SUCCESS]
> This is a success.

> [!TASK]
> This is a task.

> [!TIP]
> This is a tip.

> [!WARNING]
> This is a warning.

Then build the site and validate the published HTML file:
https://validator.w3.org/#validate_by_input
There are 40 errors and warnings such as:

I'll look into these SVG validation issues and fix them as soon as possible

I think import/export though Illustrator should clean these up. Let me know if you want to me to give it a try.

I think import/export though Illustrator should clean these up. Let me know if you want to me to give it a try.

Yes, Thanks. That would be very helpful!


Create GameObject subclasses for objects 0 ("Endlist") and 1 (Player data). Update Factory to acomodate that.

Don't forget to set these to not be movable.


Remove Very Good Cli Splash Screen

how to remove gray very good cli Splash Screen

Hello there,
You have to be a little more specific about what this refers to. Is this on a template created with the create command? On a specific platform?

I have used very good cli
To create my project after I created it
A gray splash screen appears with a logo on it
I removed the logo, but I could not remove the gray color
I used the native splash library, and now I have 2 splash screens, and the The problem got bigger

https://user-images.githubusercontent.com/27555246/226846882-665822cf-831c-423e-abea-c77260357690.mov
You can look here
A gray screen appears
Then the original screen of the application appears in white
Then it goes to flutter

Can you check if that happens on release mode?

the release need to do signing configuration
i try stop it but the APK Not Installed

Thanks @osamamohammed98 for opening the issue 💙 !
I would love to get this to reproduce in my personal computer (locally). Do you mind specifying the steps I need to do to get this issue showing up?
It would also help us if you include the output of your flutter doctor -v, the output of very_good --veresion and the android version you're device is on.

I just executed the command to create a Flutter application
Then I removed the image that appears in the splash screen
Then I tried to remove the gray color that appeared, but I couldn't
Note that a gray screen appears because the theme of my phone is dark
Then I added the Splash Native creation library, so it appeared to me
A gray screen, then the original screen of the Android application, then it moves to flutter framework

% flutter doctor -v
[✓] Flutter (Channel stable, 3.7.3, on macOS 13.2.1 22D68 darwin-arm64 (Rosetta), locale en-PS)
• Flutter version 3.7.3 on channel stable at /Users/osama/flutter_sdk/flutter_37
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision<PHONE_NUMBER> (6 weeks ago), 2023-02-08 15:46:04 -0800
• Engine revision 248290d6d5
• Dart version 2.19.2
• DevTools version 2.20.1
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
• Android SDK at /Users/osama/Library/Android/sdk
• Platform android-33, build-tools 33.0.1
• ANDROID_SDK_ROOT = /Users/osama/Library/Android/sdk
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 14C18
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2022.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
[✓] VS Code (version 1.76.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension can be installed from:
🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[✓] Connected device (3 available)
• Redmi Note 8 (mobile) • 115a432b • android-arm64  • Android 11 (API 30)
• macOS (desktop)       • macos    • darwin-arm64   • macOS 13.2.1 22D68 darwin-arm64 (Rosetta)
• Chrome (web)          • chrome   • web-javascript • Google Chrome 111.0.5563.110
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!

very_good --version
0.12.2

Facing a similar issue, also flavours icon [from flutter_launcher_icons] not working , any fixes yet?

linking #481

linking #481

I try it and it's not successful
I'm solving it by create flutter project without very good
And add flavor manual
And make all features manual
And it's working now


Some messages do not get delivered

Hello,
thank you for this software, it really suits my needs. However, I was doing some tests today and found out that not all the pushed messages get delivered to the subscribed devices. I usually run the server on an Amazon EC2 instance but the problem appears on a fresh virtual machine as well.
Here's a screenshot: https://i.imgur.com/6hEkIal.png
The top right terminal (client.py) should show numbers from 0 to 9 but only shows 6 and 8.
If I send all 10 numbers and then subscribe to the service everything works fine, but if sending and receiving at the same time, some messages get lost. The SQL 'message' table is empty as well.
Maybe I should mention that I run the server from the console by typing ./application.py (and it shows all the 10 POST requests). Am I missing some steps? Do I need to install pushjet-server-broker? Or maybe it's a bug in my code?
Thanks,
Francesco

I think this is a race condition. I'll write a quick fix for it

The SQL 'message' table is empty as well.
Messages are deleted after delivery to all subscribers.

The issue was that the TIMESTAMP field in MySQL does not store miliseconds and thus the system often shought that you already read a message when the timestamps overlapped.
It has been fixed using 5075634b94c11c6c82e1a51f9c3503cca682bc4f...67097bf668b88e788a0b0bffc41c9bb9a62398a2

Great! Thanks for the very quick fix. I hope I can contribute to the software someday :)


Support for PHP8

PHP8 has just been released. According to composer.json, this repo requires "php": "^7.1". Any plans to make it work with PHP8?

guzzle 6 does not work with php 8, so this would not help (they used the >7.1 version constraint, but that was a mistake).
for php 8, you should use guzzle 7 (which implements psr-18 natively) or if you need the httplug async interface, use guzzle7-adapter with guzzle 7.


RuntimeError: Pin memory thread exited unexpectedly while fine tuning with 100 samples in dataset

Hi,
I wanted to fine tune the Meta model on my 100 sample dataset until 60 k steps but it failed with memory issue after 9k steps. The error trace is the following :
Steps:              9912   0%|                                                                   | 0/2 [00:00<?, ?it/s]Exception in thread Thread-5 (_pin_memory_loop): Traceback (most recent call last):   File "/usr/lib/python3.10/threading.py", line 1016, in _bootstrap_inner     self.run()   File "/usr/lib/python3.10/threading.py", line 953, in run     self._target(*self._args, **self._kwargs)   File "/home/caraduf/ToucanTTS/toucanenv/lib/python3.10/site-packages/torch/utils/data/_utils/pin_memory.py", line 49, in _pin_memory_loop     do_one_step()   File "/home/caraduf/ToucanTTS/toucanenv/lib/python3.10/site-packages/torch/utils/data/_utils/pin_memory.py", line 26, in do_one_step     r = in_queue.get(timeout=MP_STATUS_CHECK_INTERVAL)   File "/usr/lib/python3.10/multiprocessing/queues.py", line 122, in get     return _ForkingPickler.loads(res)   File "/home/caraduf/ToucanTTS/toucanenv/lib/python3.10/site-packages/torch/multiprocessing/reductions.py", line 322, in rebuild_storage_filename     storage = torch.UntypedStorage._new_shared_filename_cpu(manager, handle, size) RuntimeError: unable to mmap 25664 bytes from file </torch_3135439_4056562685_15644>: Cannot allocate memory (12)   0%|                                                                   | 0/2 [00:05<?, ?it/s] Traceback (most recent call last):   File "/home/caraduf/ToucanTTS/IMS-Toucan/run_training_pipeline.py", line 71, in <module>     pipeline_dict[args.pipeline](gpu_id=args.gpu_id,   File "/home/caraduf/ToucanTTS/IMS-Toucan/TrainingInterfaces/TrainingPipelines/finetuning_LS.py", line 87, in run     train_loop(net=model,   File "/home/caraduf/ToucanTTS/IMS-Toucan/TrainingInterfaces/Text_to_Spectrogram/PortaSpeech/portaspeech_train_loop_arbiter.py", line 53, in train_loop     mono_language_loop(net=net,   File "/home/caraduf/ToucanTTS/IMS-Toucan/TrainingInterfaces/Text_to_Spectrogram/PortaSpeech/portaspeech_train_loop.py", line 115, in train_loop     for batch in tqdm(train_loader):   File "/home/caraduf/ToucanTTS/toucanenv/lib/python3.10/site-packages/tqdm/std.py", line 1195, in __iter__     for obj in iterable:   File "/home/caraduf/ToucanTTS/toucanenv/lib/python3.10/site-packages/torch/utils/data/dataloader.py", line 628, in __next__     data = self._next_data()   File "/home/caraduf/ToucanTTS/toucanenv/lib/python3.10/site-packages/torch/utils/data/dataloader.py", line 1316, in _next_data     idx, data = self._get_data()   File "/home/caraduf/ToucanTTS/toucanenv/lib/python3.10/site-packages/torch/utils/data/dataloader.py", line 1277, in _get_data     raise RuntimeError('Pin memory thread exited unexpectedly') RuntimeError: Pin memory thread exited unexpectedly
I don't know which memory is concerned by RuntimeError: unable to mmap 25664 bytes from file  whether it is RAM, VRAM, or swap.
However according to WandB VRAM was used at about 33%, System RAM was 75% before the training aborted. So swap may be the cause (even though it is not reported in WandB) ?
What do you think should I increase swap (it is actually 2GB) although someone with the same error did it without success ?
Any help appreciated,

Even after increasing the swap to 16GB the same error appears and the training stops after 110 minutes (roughly 4k steps).
I am now trying again with 4 workers in the Dataloader for the training loop. I'll keep you posted.

Hmm, I cannot really think of any reason something would utilize the disk so much other than writing a checkpoint. So it is  weird that this issue occured so late in training. Does the issue still persist, or did you find a solution in the meantime? The pin memory is a special region of the memory, maybe this has a hard limit and turning pin_memory off in the dataloader could fix this.


Updating dependencies

Description
This PR intends to create an initial setup for omgidl project. What is included:

Renaming the project to point to lichtblick projects and references
Change version on package.json
Update needed dependencies

Installed new dependencies to fix some errors (espree and acorn), fixed lint errors and updated the tests so they could run again with the new libraries versions


ApplicationLoadBalancer adds unexpected ingress rules to security group

Describe the bug
I'm trying to create an ApplicationLoadBalancer with predefined security group rules to allow traffic only from specific IP addresses.
To Reproduce
    const securityGroup = new ec2.SecurityGroup(this, `Rule`, {
      vpc,
    });
    securityGroup.addIngressRule(new ec2.CidrIPv4('<IP_ADDRESS>/27'), new ec2.TcpPort(443), 'IP address');
    const loadBalancer = new elbv2.ApplicationLoadBalancer(this, `LoadBalancer`, {
      vpc,
      internetFacing: true,
      securityGroup,
    });

The generated code seems to incorrectly allow traffic from any IP address for some reason:
        - CidrIp: <IP_ADDRESS>/0
          Description: Allow from anyone on port 443
          FromPort: 443
          IpProtocol: tcp
          ToPort: 443

Expected behavior
I would expect the traffic to be allowed only from the specified addresses.
Version:

OS: macOS
Programming Language: Typescript
CDK Version: 0.31.0

Can you try adding open: false in the listener props?

@jogold thanks for noticing! I wasn't aware of this property as it wasn't explicitly required.
    const listener = loadBalancer
      .addListener(`HttpsListener`, {
        port: 443,
        certificateArns: [certificate.certificateArn],
        open: false, // This will do the trick
      });


removed code syntax extra quotes

There was extra code syntax quotes.

I have included bootstrap.sh script that should take care of generating token for the worker nodes.


Error deploying a legacy subgraph

Hello,
Upon attempting to deploy a legacy subgraph using my access token, I get the following error:
✖ Failed to deploy to Graph node https://api.thegraph.com/deploy/: Invalid account name or access token
Commands used:
graph auth --product hosted-service <MY_ACCESS_TOKEN_HERE>
graph deploy --product hosted-service oikos1/exchanger --ipfs https://api.thegraph.com/ipfs/ --node https://api.thegraph.com/deploy/ --debug
The commands used were working before. Any idea what is causing the issue?
Thanks in advance

Interesting thanks for reporting @oikos1  - does it work if you pass --access-token <TOKEN_HERE> when deploying?

Interesting thanks for reporting @oikos1 - does it work if you pass --access-token <TOKEN_HERE> when deploying?

My pleasure. Yes, it works by passing the parameter directly with -access-token, thank you!

did you fix this problem?

did you fix this problem?

It worked for me with the --access-token flag. Not sure if the bug is fixed though. Closing this as you guys are aware of the issue and a solution was provided. Thank you!


Grpc testing responders

Needed for some tests I'm working on
Makes overriding servicer functions a bit more flexible:

override_default (now set_responder) now allows setting an async responder function with the same signature as the original method, instead of a blocking one
errors in the mock servicer methods are now caught and logged properly, regardless how/when they are added

@prbot approve


Regulation of regulated release

Dear all,
while I was adding new terms about neurotransmitter
secretion GO:0007269, I have noticed that there could
be a problem with the definition of "regulation of x
secretion".
The definition of secretion GO:0046903 is "The
regulated release of a substance by a cell or group of
cells"
and so the regulation terms are describe "the
regulation of a regulated process" (it seems to me as a
regulation of a regulation).
What do you think?
Do you think it might be necessary to change the
definition of "x secretion"?
Thanks
Erika
Reported by: arike
Original Ticket: "geneontology/ontology-requests/3275":https://sourceforge.net/p/geneontology/ontology-requests/3275

thanks www.movied.org


Support for postgres data type json?

Is there any way I can display the value of a column with datatype json in the edit view?
Currently I am just getting the string representation of the object, which is [object Object].
settings.json:
            {
                "name": "custom_data",
                "verbose": "Data",
                "control": {
                    "text": true
                },
                "type": "json",
                "allowNull": true,
                "defaultValue": null,
                "listview": {
                    "show": true
                },
                "editview": {
                    "show": true
                }
            }

Hey, I was facing with the same issue and I tweaked the library a bit  :D In the end, I did this: https://github.com/yusufyildirim/express-admin/commit/daf359f09bc2cd7dcc5e06bac8ec70c59271f29b
This allows you to use "text" control but converts JSON objects to the string so you can see it in edit view. Propably, we need to add extra json view or something like that to edit our json easily but in my case, I don't need it for know because my json datas are simple arrays.
If you want help, please comment so maybe I can help you to improve this.

Yes, excellent tip. I was thinking of doing exactly the same, but was hoping that I wouldn’t have to fork the repo and that I was just missing something :)

On 19 Oct 2018, at 19.09, Yusuf YILDIRIM<EMAIL_ADDRESS>wrote:
Hey, I was facing with the same issue and I tweaked the library a bit :D In the end, I did this: yusufyildirim@daf359f
This allows you to use "text" control but converts JSON objects to the string so you can see it in edit view. Propably, we need to add extra json view or something like that to edit our json easily but in my case, I don't need it for know because my json datas are simple arrays.
If you want help, please comment so maybe I can help you to improve this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I found a bug today. Sometimes, this function triggers for the same data and this behavior causes a problem. Assume you have a json array like ["A", "B", "C"], if you convert it to string with JSON.stringfy method, everything is allright but if you do the same thing again to that string, you found yourself with a data like "[\"A\", \"B\", \"C\"]". To prevent this, I changed my code a bit. Here is my new approach: https://github.com/yusufyildirim/express-admin/commit/b6980f4f5ec0fe4810bc812d6736d31bea518427

👍🏻

On 20 Oct 2018, at 21.46, Yusuf YILDIRIM<EMAIL_ADDRESS>wrote:
I found a bug today. Sometimes, this function triggers for the same data and this behavior causes a problem. Assume you have a json array like ["A", "B", "C"], if you convert it to string with JSON.stringfy method, everything is allright but if you do the same thing again to that string, you found yourself with a data like "["A", "B", "C"]". To prevent this, I changed my code a bit. Here is my new approach: yusufyildirim@b6980f4
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Merge pull request #4 from pkprajapati7402/pr-most

Merge pull request #3 from pkprajapati7402/main

Good


Best way to access DB outside requests

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from typing import Generator
from sqlalchemy import create_engine
from sqlalchemy.orm import sessionmaker
from app.core.config import settings

engine = create_engine(settings.SQLALCHEMY_DATABASE_URI, pool_pre_ping=True, max_overflow=-1, future=True)
SessionLocal = sessionmaker(autocommit=False, autoflush=False, bind=engine)

def get_db() -> Generator:
    try:
        db = SessionLocal()
        yield db
    finally:
        db.close()

# in my celery task

@celery_app.task()
def some_task():
    db = next(get_db()) # the connection remains after the task is finished
    db.query(.....)
    return 'success'

Description
I used the next(get_db()) method to use the db session outside the request because the Depends cannot be used here as I found out from #1693 but when more tasks get executed the db no longer accepts connection as it had 62 connections already.
I have looked around and found 2 ways people recommend for using the db session
1- contextmanager  from Stackoverflow question
@celery_app.task()
def some_task():
    with contextmanager(get_db)() as db:
        db.query(.....)
        return 'success'

2- another example from SQLAlchemy docs  is
@celery_app.task()
def some_task():
    with SessionLocal() as db:
        with db.begin():
            db.query(.....)
            return 'success'

Both methods significantly reduced the number of connections and close session properly after task while maintaining similar queries/sec performance in a small test I made.
Which of these is recommended as best practice and what are the pros and cons of each method?
how can I optimize the create_engine arguments for better connection and less problems with increased queries?
Operating System
Linux, Windows
Operating System Details
No response
FastAPI Version
0.73.0
Python Version
3.8
Additional Context
No response

I just ran into the same problem as you, and your question put me on the right track to fix it!
Reading through the fastapi "Dependencies with yield" documentation it seems that the preferred way is with contextmanagers, as that is what is used internally in fastapi. I hope I was able to help you with this!

I just ran into the same problem as you, and your question put me on the right track to fix it! Reading through the fastapi "Dependencies with yield" documentation it seems that the preferred way is with contextmanagers, as that is what is used internally in fastapi. I hope I was able to help you with this!

Hi! Can you give example of your code?

database.py
databaseURL = os.getenv("DATABASE_URL")
engine = create_engine(databaseURL, pool_pre_ping=True, pool_size=32, max_overflow=64)
SessionLocal = sessionmaker(autocommit=False, autoflush=False, bind=engine)
Base = declarative_base()

@contextmanager
def SessionManager():
    db = SessionLocal()
    try:
        yield db
    except:
        # if we fail somehow rollback the connection
        warnings.warn("We somehow failed in a DB operation and auto-rollbacking...")
        db.rollback()
        raise
    finally:
        db.close()

somewhere_where_you_need_a_session.py
import database

with database.SessionManager() as db:
    # your code using the database connection goes here

Thanks @dgtlntv for your solution, you saved my day! I was searching around but still unable to find a solution for db connection that is independent of FastAPI Depends. Once again, thanks a lot


Updates project object version to support BuildIndependentTargetsInParallel setting

Hi, I've been trying to implement the XcodeGen to a project and the generated project would always end up with a warning about the new "Parallelize build for command-line builds", after some investigation I've discovered that the attribute key that sets this value is BuildIndependentTargetsInParallel, and without updating the object version of the .pbxproj file this setting is completely ignored by Xcode.
This also would fix a open issue #1362
Discussion
I believe that this setting should be default from now on since that's the default setting when creating a new App from Xcode.
This PR doesn't set this value as default but allows it to be overwritten and interpreted by Xcode.
Setting "Parallelize build for command-line builds"
To enable the Parallelize build for command-line builds configuration add this to the project.yml file
attributes:
  BuildIndependentTargetsInParallel: YES

The project configuration section should have the "Parallelize build for command-line builds" checked ✅

Environment

XcodeGen Version: 2.35.0
Xcode Version: 14.3.1

Thank you @yonaskolb, the CHANGELOG.md has been updated as requested.


cli tool

Add a cli tool similar to the ftp unix command. This would allow to have a ftp client on NodeOS.

That should be very easy to do as another module. It's out of the scope for the jsftp library.


github authentication added

Related Issue
github authentication added
Closes: #201
Describe the changes you've made
I have added github authentication through firebase
Give a clear description what modifications you have made
Type of change
What sort of change have you made:

[x ] Bug fix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)
[ x] Code style update (formatting, local variables)
[ x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ x] This change requires a documentation update

Mention any unusual behaviour of your code (Write NA if not)
NA
Checklist:

[x ] My code follows the guidelines of this project.
[x ] I have performed a self-review of my own code.
[x ] I have commented my code, particularly whereever it was hard to understand.
[x ] My changes generate no new warnings.
[x ] Any dependent changes have been merged and published in downstream modules.

Screenshots
Add Screenshots to show your changes
Additional Info (optional)
Any additional information you want to give

@avinashkranjan @Utkarsh1999 Sir  Please review my PR  and you have not enabled github auth . So please enable it .

@avinashkranjan @Utkarsh1999 Please review my PR .

@Utkarsh1999 can i also work on facebook auth . I have done the coding part and it is working fine .So can i make the PR for that ?
I have also asked for issue ?

Yes you, but please create an issue for that.

@Utkarsh1999 issue is already created but it is assigned to someone  else .

@Utkarsh1999 issue is already created but it is assigned to someone else.

I assigned that to you, go ahead with Facebook auth.

@Utkarsh1999  @avinashkranjan  have you disabled the facebook auth ?


gnunet: 0.20.0 -> 0.21.1

Description of changes
https://git.gnunet.org/gnunet.git/tag/?h=v0.21.1
Closes #294145.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 299075 run on aarch64-linux 1

2 packages failed to build:

taler-exchange
taler-merchant

2 packages built:

gnunet
gnunet-gtk

Result of nixpkgs-review pr 299075 run on aarch64-linux 1

4 packages built:

gnunet
gnunet-gtk
taler-exchange
taler-merchant


Provide a template for jooq-codegen.xml

sbt build doesn't work from a clean checkout (even after running initial database population), complaining of the missing jooq-codegen.xml file. I'm guessing this wasn't provided in the repo as-is because it can contain login credentials. However, there's other potential configuration here that I'm not sure how to guess (Is the default Generator used, or ScalaGenerator? What is the target packageName and directory?).
It would be helpful to have a template version of the file, perhaps leaving the db credentials blank but having everything else present for demonstration purposes. Better yet, environment variable substitution in this file (e.g., <password>${PGPASSWORD}</password> could be used so that the credentials could be provided in the sys.env running sbt.

hello bpow,
I have copied our internal jooq-codegen.xml to jooq-codegen-template.xml. The codegen.xml contains some non-trivial Postgres Date to Joda Datetime conversion settings.


-h Docker Flag Shorthand Deprecated

Flag shorthand -h in Docker as mentioned on this page has been deprecated.
Docker rename [container_name] [new_container_name] is now more relevant.

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

ID: b152444b-dfb8-5cbe-cf45-9830bd085a68
Version Independent ID: 31719660-d4d0-7f63-efcc-846c822a63c2
Content: Docker: Install containers for SQL Server on Linux - SQL Server
Content Source: docs/linux/quickstart-install-connect-docker.md
Product: sql
Technology: linux
GitHub Login: @amvin87
Microsoft Alias: amitkh

@fennar01 -- thank you for your feedback.
@amvin87 -- please look into this issue.

Thank you @fennar01  I thought the -h shorthand for --help was deprecated as documented here https://docs.docker.com/engine/deprecated/#-h-shorthand-for---help, but the context for which it is used in our documentation is for --hostname. but I agree to avoid confusion we should update our documentation and replace -h with --hostname.
@WilliamAntonRohm  could you please help update our documentation and replace -h with --hostname in the command examples in the documentation.

@amvin87 -- thank you for clarifying.  I will make this update.
#reassign @WilliamAntonRohm

Thank you for your feedback.  I have updated the documentation and it should be live by tomorrow.


feat(rules): no-extra-parens

When my .eslintrc file is this:
{
  "extends": "amex"
}

The following code does not trigger any warnings / errors:
      if (!(((((child.data)))))) {
        return child.children.map((c) => c.data);
      }

However including that extra rule directly via
{
  "extends": "amex",
  "rules": {
    "no-extra-parens": "error"
  }
}

makes the error appear:

@PatNeedham i recently merged another pull request which created a conflict on this, could you please resolve the conflict

Thank you @PatNeedham  🙌


Instances status out of sync with hypervisor VM's

ISSUE TYPE

Bug Report

COMPONENT NAME

Cloudstack Manager

CLOUDSTACK VERSION

Cloudstack <IP_ADDRESS>, VMWare 6.5 as hypervisor.

CONFIGURATION

1 Cloudstack management server, 1 dedicated DB server with MariaDB 5.5, 3 node VMWare VSphere 6.5 cluster. Cloudstack advanced networking setup.
OS / ENVIRONMENT

Management server installed on CentOS 7.
SUMMARY

Often it happens that Instances status goes out of sync with VMWare virtual machines status.
I startup one instance, it boots up correctly and goes to the "Running" state. After some time (hours, days, months, there is not a defined amount of time) it appears in the "Stopped" status on the manager web interface (in the "instances" section) without any user interaction, but if I look in the VSphere interface the virtual machine is still running fine and I can connect to its console.
This happens with randomly on instances, both with Windows and Linux VMs.
VSphere cluster has DRS enabled, I have tried to turn DRS off but this behavior doesn't change.
STEPS TO REPRODUCE

Setup many instances, turn them on and wait. After an undefined amount of time one or more instances could appears stopped in Cloudstack management web interface but they are still powered on and running for VMWare VSphere.

EXPECTED RESULTS

ACTUAL RESULTS

Hi Andrea,
You mention that you had turned DRS off, had you still performed migrations on the effected VMs via vCenter, rather than via CloudStack?
thanks

Hi Paul,
I had not performed any migrations via vCenter, every operation was made via CloudStack.
Thanks!

@AndreaGHG there are some improvements with respect to status processing in the newer versions. Can you check if your issue is still happening in newer versions? (btw The fixes pertain to DRS out of bounds migrations so I wonder what could happen with DRS turned off)

@AndreaGHG as 4.15 is out and you didn't reply I'm assuming this can be closed, please re-open if i'm in err

@AndreaGHG as 4.15 is out and you didn't reply I'm assuming this can be closed, please re-open if i'm in err


Locol SIG Monthly: Wed, Aug 21 @ 10am PDT | 1pm EDT | 5pm UTC

We will be holding our next SIG (Special Interest Group) monthly on Wednesday, August 21 - feel free to add agenda items you'd like to discuss in the notes doc: https://docs.google.com/document/d/1Mol6epw0sbj5FqS6akD-X5KJkGF0MwsOZQ5t1ehOGiM/edit
Agenda so far:

Demo by Yiannis Psaras (@yiannisbot) of Data Hop, which uses Bluetooth low energy and WiFi direct together to enable file sharing between devices irrespective of network
Insights from DWeb Camp
ADD MORE!

Zoom: https://protocol.zoom.us/j/884541689
Everyone is welcome: @creationix @Jorropo @onggunhao @Gozala @raulk @terichadbourne @whyrusleeping @djdv @jedahan @jimpick @nicopace @raulk @codynhat @yiannisbot @momack2 @JustMaier @vrortvedt @autonome @parkan and others.

Just a reminder that our next Local & Offline Collaboration SIG (Special Interest Group) call takes place this Wednesday, August 21 at 10am Pacific / 1pm Eastern / 5pm UTC. Hope to see you there!
Agenda so far:

Demo by Yiannis Psaras (@yiannisbot) of Data Hop, which uses Bluetooth low energy and WiFi direct together to enable file sharing between devices irrespective of network
Insights from DWeb Camp
Upcoming Offline Camp Oregon (Sept 27-30)
ADD MORE in our agenda/notes doc: https://docs.google.com/document/d/1Mol6epw0sbj5FqS6akD-X5KJkGF0MwsOZQ5t1ehOGiM/edit

Zoom: https://protocol.zoom.us/j/884541689
Everyone is welcome: @creationix @Jorropo @onggunhao @Gozala @raulk @terichadbourne @whyrusleeping @djdv @jedahan @jimpick @nicopace @raulk @codynhat @yiannisbot @momack2 @JustMaier @vrortvedt @autonome @parkan and others.

Hi all,
I would also like to share with you all a project that I am supporting, with the Nam Community in Namibia, and indigenous collectives in Brazil, India and New Zealand for oral story sharing that uses DWeb technologies... hope we have time for that! See you all in a bit,

Thanks to all who attended today's call, and especially to @yiannisbot for telling us about the Data Hop project and to @nicopace and Nicola Bidwell for sharing their experiences create audio social networks for the San people of Namibia. It was also great to hear some reflections on DWeb camp from @parkan @momack2 & @nicopace.
If you missed today's call, you can watch the video or check out our meeting notes.

Hope to see you at next month's call on Wednesday, September 18. Watch the new event issue for updates, and please let me know if you'd like to share or demo your project!


White behind font??

This is a vestige of the default Notepad++ theme from which npp-material-theme is made.
In the default theme, in all the languages, the words have a white background.
This has been changed for the languages that our theme has been optimized for, say HTML, CSS, XML, SQL, etc. For other languages, changes need to be made.

fixed by https://github.com/Codextor/npp-material-theme/commit/04f0dfacd849d1c6eacd89229df93c240acd213a


deploy: split up deploy.sh (#314)

This project is licensed under Apache License v2.0 for any software, and Solderpad Hardware License v2.1 for any hardware - do you agree that your contributions to this project will be under these licenses, too? Yes.
Were any of these contributions also part of work you did for an employer or a client? No.
Does this work include, or is it based on, any third-party work which you did not create? No.

the changes were tested on a headless raspberry pi 4b!


Implement resuming a connection from a previously serialized state

New feature to export dtls state of an already running dtls connection in order to be able to resume it later.
Use case:
Serialize and persist to an external data source the dtls state in order to resume it from a different running instance of the application, avoiding performing a new handshake.

Hi, this looks like a cool feature. For documentation purposes I'm wondering if this is based of a spec or some other implementation we can refer to?
Some housekeeping in order to make the CI happy:

Please add yourself to the Contributing section of the root readme file.
Please change your commit messages to follow the following rules:
https://github.com/pions/dtls/blob/ce8ae7c846ee3feae922e26a3a2dbcd5ae98b158/.github/lint-commit-message.sh#L9-L16

Hi, this looks like a cool feature. For documentation purposes I'm wondering if this is based of a spec or some other implementation we can refer to?

I don't have other implementation references on hand, but I'll say OpenSSL and others already have mechanism to do this.  There a some use cases that would benefit from this implementation:

accept dtls connection and then transfer to another process
when the transport protocol is not UDP, but wrapped inside another protocol

I've already discussed it here https://github.com/golang/go/issues/25351#issuecomment-415357066 for the crypto/tls package  and even tried to "hack it" here: https://github.com/igolaizola/resume-tls

This is really cool @igolaizola I would love to see this land. You should also post about it! I think a lot of people could build cool things with this.
What do you think of embedding State in Conn that way we don't have to do as much copying back and forth?
I would also add a timeout to your tests, maybe it is causing things to deadlock? Do you have failures locally.

This is really cool @igolaizola I would love to see this land. You should also post about it! I think a lot of people could build cool things with this.

Thanks! I really think that is an interesting feature. Hopefully something similar will be added to crypto/tls.

What do you think of embedding State in Conn that way we don't have to do as much copying back and forth? The only thing that might be weird is the atomics :/ but I just worry things will diverge when have multiple structures, and will cause weird bugs in the future.

I'll take a look to it. I've coded it this ways in order to be less intrusive, but as you say in the future extra work must be done to have my code in sync.

I would also add a timeout to your tests, maybe it is causing things to deadlock? Do you have failures locally.

I am dealing with some data race issues and I might added some deadlocks while I was trying to fix them.

@Sean-Der PTAL. I've added changes to embed the State within the Conn. Also changed the serialization part to implement encoding.BinaryMarshaler and encoding.BinaryUnmarshaler
go test -v -race ./... seems to be working now locally but still failing on CI

Can you move srtpProtectionProfile and remoteCertificate into state, otherwise after resumption users will get broken values.
Nice it looks like you are green in CI! Just a few nits from me and I am happy to merge :) Would you mind squashing into a single commit, and tagging with the github issue (just helps in the future with debugging/clean history)
thank you so much for all the work, this is a really great feature.

Can you move srtpProtectionProfile and remoteCertificate into state, otherwise after resumption users will get broken values.

Done!

Nice it looks like you are green in CI! Just a few nits from me and I am happy to merge :) Would you mind squashing into a single commit, and tagging with the github issue (just helps in the future with debugging/clean history)

Done! Also improved a little bit the serialization in order to add a clone method.

thank you so much for all the work, this is a really great feature.

Thanks to you for dealing with this project. Right now dtls is a missing feature in the standard crypto library.

Hey @igolaizola
I added you to the Pions organization, and you can now merge your PR :)
Thank you so much for working on this, and if you have the time I would love your continued involvement working on DTLS! I think it would be a great thing for the community to have a good implementation, so hoping we can continue growing it until we get to that point!

Thank you so much for working on this, and if you have the time I would love your continued involvement working on DTLS! I think it would be a great thing for the community to have a good implementation, so hoping we can continue growing it until we get to that point!

I'll try to help and improve the package 👍


Added bigbluebutton-exporter as Systemd service

Added lock against installing bigbluebutton-exporter as Docker and as Systemd service at the same time
Changed reload monitoring handler name
Can variable bbb_monitoring_recordings_from_disk rename in bbb_monitoring_all_in_one_recordings_from_disk?

Looks OK so far.
Buy Why?
The bbb exporter doc states:
Two installation methods are supported:

    Docker (recommended)
    Systemd (not recommended)

Is there a valid reason why to add the systemd style version for this when the bbb exporter doc its not recommended?

They recommend Docker because it is easier to update, reproduce builds and comes with all dependencies pro-installed. All this advantages are general Docker advantages and have not really something specially to do with the BigBlueButton Exporter.
We do not want to use Docker for the BigBlueButton Exporter.

The docker setup this role is currently doing is quite a lot of overhead. You do not always need the grafana that is installed with it. I've done the original docker monitoring setup in this role and did only implement docker due to time constraints. I personally would be hugely in favor of this being merged because having something not in docker would make it easier to firewall it off to be able to access it from e.G. a wireguard tunnel.
Regarding your question about the renaming:
I would vote to keep it that way since this option is also available in the stand-alone version of the exporter with the RECORDINGS_METRICS_READ_FROM_DISK Variable. Since this is the same feature, I would keep it at that and just use it in both cases.

@wilkis3 @e1mo

You do not always need the grafana that is installed with it.

Yes I had the same problem and now there is the option to disable grafana / prometheus and to expose to public with htpasswd.
Also @Davka is working on another feature where you can limit which IPs are allowed to access the data.
See #111
https://github.com/ebbba-org/ansible-role-bigbluebutton/blob/bbb/2.3/defaults/main.yml#L81-L90
Thanks for explaining the use-cases.

The docker setup this role is currently doing is quite a lot of overhead. You do not always need the grafana that is installed with it. I've done the original docker monitoring setup in this role and did only implement docker due to time constraints. I personally would be hugely in favor of this being merged because having something not in docker would make it easier to firewall it off to be able to access it from e.G. a wireguard tunnel.
Regarding your question about the renaming:
I would vote to keep it that way since this option is also available in the stand-alone version of the exporter with the RECORDINGS_METRICS_READ_FROM_DISK Variable. Since this is the same feature, I would keep it at that and just use it in both cases.

I submitted a PR in which you can explicitly only install the BBB exporter. This can then be used to connect an external Grafana + Prometeus.

@e1mo and @wilkis3 can you check against @Davka PR if this would you as well?

@BrutalBirdie @Davka 's merge request #111 looks nice and it is a good extension. It does not solve the problem to work not with Docker in general. Maybe it solves @e1mo 's problem.

@BrutalBirdie @Davka 's merge request #111 looks nice and it is a good extension. It does not solve the problem to work not with Docker in general. Maybe it solves @e1mo 's problem.

I still don't understand why you don't want to run that in Docker. Docker is here for LibreOffice anyway.

@e1mo and @wilkis3 can you check against @Davka PR if this would you as well?

No. Well, if I only need the exporter this is perfectly fine. But why not?

Having it running as a native service provides (me) more flexibility. Systemds deep integration with the system allows me to employ more fine-grained controls about the behavior of a service. All of the logs are stored in a central place making debugging easier.
The in #111 proposed ip restrictions do not fit my need. All requests must be made via https due to the http -> https redirect. But in the case of any internal VPN we are dealing with private IPs. If we would do https with those, we would send a custom http Host in the request  (because nginx only serves this content over the normal domain).

@BrutalBirdie @Davka 's merge request #111 looks nice and it is a good extension. It does not solve the problem to work not with Docker in general. Maybe it solves @e1mo 's problem.

I still don't understand why you don't want to run that in Docker. Docker is here for LibreOffice anyway.

I usually like to take a step back and ask myself "is docker (or any other helper for that matter) really necessary and is the value I'm getting out of it in a proper relation to the energy invested?". In some cases (e.g. LibreOffice, Greenlight, the all in one Monitoring Stack because its entirety would be vastly out of scope) docker is a fine solution. But in other cases, I would not judge docker as being required to do the job. This is one of these cases. This PR is quite straight forward, we do not need to take a significant detour to install the service. Get the sources, the dependencies + configuration and run the service.
And not using docker makes a couple of things easier in my opinion. Coming back to my mentioned use-case (only being able to access the exporter from a wireguard interface). Firewalling a natively running process is less complex than in a docker setup. (Back reference to the nginx ip range issues)
To wrap it up, for me it comes down to have things native if its feasable due to more opportunities / less overhead. This PR is quite straight forward and should be very low in / avoid of maintenance needs and does not add a huge overhead to this role from my point of view. I would propose going on with this PR and not going to deep in the docker discussion (I tried to write this comment as neutral as I could and not fuel any "docker is{, not} useless and massive overhead" discussions. I hope this I achieved this, if not, I'm sorry. That was not my intention). I think both can co-exist happily, so can the people who implement it.

Sorry for my delay, due to school I probably will only find time for a review not before tomorrow or end of the week.

Hi @wilkis3, can you merge bbb/2.3 once again and remove the rest of bbb_baseurl from tasks/main.yml?
I will merge this PR afterwards.

Hi @crosscodr, I merged bbb/2.3 again. I doesn't found any existing bbb_baseurl things.

Hi @crosscodr, I merged bbb/2.3 again. I doesn't found any existing bbb_baseurl things.

Hi @wilkis3, they are in line 13 and 55 of tasks/main.yml

Hi @crosscodr, I merged bbb/2.3 again. I doesn't found any existing bbb_baseurl things.

Hi @wilkis3, they are in line 13 and 55 of tasks/main.yml

bbb_baseurl was found and removed!


Autocomplete: Wrong name when using custom subcommand names

Hi! When using custom names for commands the autocomplete does not work correctly. It outputs the name of the function (cmd), not the name assigned in add_command (my_custom_name):

@click.group()
def group():
    pass

@click.command()
def cmd():
    pass

group.add_command(cmd,    'my_custom_name')

I was able to fix the issue by overriding add_command in the following way:
    def add_command(self, cmd, name=None):
        cmd.name = name
        return super().add_command(cmd, name)

However I don't know if this raises other issues or if this works in all cases.

Still an issue in the new autocomplete system (https://github.com/pallets/click/pull/1622).

Passing the name in the decorator works.

This was fixed as part of #1622, specifically 00d5dcc.


Interactive mode

Checklist

[X] I've looked through the issues and pull requests for similar request

Describe your request
I would like to be able to run the containers on interactive mode.
Describe why this would be a good inclusion for cross
This could be useful to troubleshot build problems. For instance, right now git describe is not working on some of my targets.
I can of course do that by hand, but getting a terminal into the containers could be interesting. Something like cross interactive --target foo.

This wouldn't be in cross due the need for complete API compatibility (so any CLI flags should be processed identical by cargo and cross), but could be used for cross-util. The downside is the implementation is a bit non-trivial, since we'd:

Need to process all the package and config data
Parse the command line options
If using remote, do everything like that
Add an interactive mode to add the -i flag
Change it to use bash or sh for interactive mode (likely would require sh since with custom containers we can't guarantee the presence of bash).

All of this is doable, just most of it isn't currently exposed to cross-util without some refactoring.

I guess potentially we could have a cross-util cargo subcommands or similar which would process the command-line flags but also allow custom cross extensions.

Won't https://github.com/cross-rs/cross/pull/1280 allow you to now do this? I'm using it to debug and run cargo-deb.

You'd be correct @kingosticks, thanks! We also have #716 which is a bit more specific but not solved with #1280


Long file names in quick open

Issue Type: Feature Request
I work in a deeply nested folder structure so that using quick open is a bit of a hassle because it cuts of the path name right before I can make out in what folder is the file I was trying to open.
The files are generally named the same but are in different folders.
Would it be maybe possible to have the quick open floating on top of the editor so it can have a variable width for example?
VS Code version: Code - Insiders 1.27.0-insider (7b9afc4196bda60b0facdf62cfc02815509b23d6, 2018-09-03T09:07:33.052Z)
OS version: Darwin x64 17.7.0

@bm-stschneider you want the quick open widget to be wider, is that it?

or have it floating in the middle of the screen or something. I don't know how static the position is currently.


COU: Courier receives the shipment

Courier should create a new record in its own database and mount the shipmentDSU in it's own wallet.

done in #232


conda search fails due to memory usage on small memory systems

seamm_installer version: 2022.6.30
Python version: 3.9
Operating System: Linux

Description
Installer fails with errors like this
seamm-installer install mopac-step
Finding all the packages that make up SEAMM. This may take several minutes.
.............WARNING:seamm_installer.conda:expected output from conda search --json --override-channels -c local -c conda-forge seamm, got
Traceback (most recent call last):
  File "/root/miniconda3/envs/seamm/lib/python3.9/site-packages/seamm_installer/conda.py", line 423, in search
    output = json.loads(stdout)
  File "/root/miniconda3/envs/seamm/lib/python3.9/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/root/miniconda3/envs/seamm/lib/python3.9/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/root/miniconda3/envs/seamm/lib/python3.9/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

When run manually on machine with 2 GB memory, conda search fails with exit status of 137.

This is not easy to fix, and machines with such small memory are rare (cloud machines maybe more common! if so pay for a slightly larger machine!!!!).
Will not fix.