Remotec ZXT-120 thermostat

It looks very much like the owner of this excellent project has abandoned contribution a few years ago now - a real shame :(
Regardless, in the hope of someone having some insight; I’ve got a bunch of z-way devices that are working fine in Homekit except for a Remotec ZXT-120 thermostat (heatpump controller).  It comes across in to Homekit just fine, displays the current temp, even allows me to adjust the temp to heat - but I can’t turn it on or off.  This on/off function works fine in z-way itself.
If there is anyone out there with any idea or knowledge on how to help, it would be great thing!  Thx.

Did you ever get this fully working? Having the same issue, my zxt-120 with v1.8 firmware works great in z-way and somewhat in the Homebridge web interface but not in Home on my iPhone

Did you ever get this fully working? Having the same issue, my zxt-120 with v1.8 firmware works great in z-way and somewhat in the Homebridge web interface but not in Home on my iPhone

I’m afraid not.  I use Siri and can turn my heatpump on using the ZXT-120 by saying “Hey Siri, heat the room to xx degrees”.  I can alter the temperature this way also.  But I need to use z-way to turn the heatpump off.


Option to set strict mode as default across codebase

Is your feature request related to a problem? Please describe.
Strict mode offers many benefits to ensuring stable code.  I would like to have strict mode enabled across our whole project so that developers don't need to remember to use strict mode everywhere
Describe the solution you'd like
This is a good question.  I'm not quite sure how I'd design this.  Maybe in our project we could export our own moment, and have something like
import * as baseMoment from 'moment'
function moment(...args) { baseMoment(args[0], args[1], true) }
export { moment };

Another option is that you all could do this in the moment repo, and with ES6 modules we can do
import {StrictMoment as moment} from 'moment';

Describe alternatives you've considered
Besides the above, I have considered writing an ESLint rule to enforce setting strict mode.
The advantage of doing it my first method is I could export strict mode in development and lazy mode in production.
Additional context
Of course, the best way would just be if moment moved to Strict mode by default like they've been saying they'd do soon for years.

This is a nice idea. I don't know how hard it will be to implement.

I've decided we should not implement this.
See https://momentjs.com/docs/#/-project-status/
Hopefully you can make that ESLint rule.

Sad to see Moment go.  Need to figure out how to get started on ESLint

Thank you for all your hard work


Switch irys node

Update the Irys mainnet nodes to:
https://up.arweave.net/
And see what happens when you test the webapp. Need to see how this will affect the app and its usecases.

Closing due to Irys resolving their issues with Arweave. The optimistic gateway and nodes will continue to run for the foreseeable future.


Added Ratelimiting

added a ratelimit to the backend server so that posts cant be sent at a rate higher than 15 posts per second.
[this is ajak4k's friend btw, hmu on zyugyzarc#0314]

Ratelimiting is needed for posts and comments, users cannot categories manually as of now ( non-admins), if you could add similar logic for posts and comments, the issue would be resolved


[BUG]Microsoft.Azure.ServiceBus.RunOperation has unused collection of exceptions

In Microsoft.Azure.ServiceBus.RunOperation there is defined a list of exceptions which is never used.
https://github.com/Azure/azure-sdk-for-net/blob/599133b04a9f62b776beda43413a450fa3884296/sdk/servicebus/Microsoft.Azure.ServiceBus/src/RetryPolicy.cs#L62

@jbrezina , Thanks for reporting this.
Yes, looks like it is indeed unused. Is this causing any issues right now?

If the exception list was not meant for anything I don't think it causes any problems. I was just looking for exceptions which are thrown in functions using this code and noticed it.

@jbrezina, got it. Thanks. Initially, the plan was to return all the exceptions that happened back to the customer. But then it doesn't make sense to pass the exception if ultimately retry did fix it. So instead, the exceptions are just logged right now. We will just have to clean up this variable. I'm not sure if when will we end up prioritizing on this task given the impact being too low. Feel free to raise a PR on this if needed. :)

@jbrezina, We will proceed to close this issue. There is a new SDK that is being developed and the changes on the older SDK (Microsoft.Azure.ServiceBus) are limited to bug fixes or features (on a case by case basis). Since this is not breaking any specific scenarios, cleaning it up is a feature change that we will defer.


Ext function form

Description
Changes proposed in this pull request:

Add function form
Add columns property to FormGroup
Add language property to CodeEditor
Handle min and max props in NumberInput
Handle enum translations
Add quick fix for default values for variables, to be done properly in #1835
Fix RuleInput bug ("Add all" would insert core-api-group instead of '').

I'll also add Env variables to the form.

/pony

/retest

/retest

/retest

/retest


Fix xiaomi vacuum tests

Breaking Change:
N/A

Description:
Fix xiaomi vacuum tests. They became more broken after #27773. I've also updated the two tests that were stale and reactivated them.
Related issue (if applicable):
fixes #27773
Checklist:

[ ] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

I'll go ahead and merge this to get dev branch back in order.


[cluster-test] Migrate debug_interface_log_tail to async api

Motivation
See issue 1863
Currently DebugPortLogThread spawns a dedicated thread for each node
this change is for switching to the async api and use the tokio runtime
instead ot std::threads
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Just ran the entire test suite and linters, not sure if any other test is needed.
Related PRs
None

@mrbuzz do you mind signing CLA? See cla bot comment above^

Overall this diff looks great, thank you! But we need CLA signed before we can merge this

@bors-libra r+

:pushpin: Commit a3fcd35 has been approved by andll

:hourglass: Testing commit a3fcd35b094063b49d1016ff87b830ac5f5db0af with merge 646408a5b48adafed5378c9e92034fa9250c239e...

:broken_heart: Test failed - checks-actions_land_blocking_test

Looks like it failed with this:
```:5thread 'ct-tokio' panicked at 'Cannot start a runtime from within a runtime. This happens because a function (like block_on) attempted to block the current thread while the thread is being used to drive asynchronous tasks.', /root/.cargo/registry/src/github.com-1ecc6299db9ec823/tokio-0.2.13/src/runtime/enter.rs:19:5
Looks like `client.get_events` on [line 62](https://github.com/libra/libra/pull/2844/files#diff-3a23b2bd8251c97ba64d0f32f2b5281aR62) calls `rt.block_on` inside, which does not work inside tokio worker. I suggest adding `NodeDebugClient::get_events_async` method and then call it from `DebugPortLogWorker`

:umbrella: The latest upstream changes (presumably cea66f4c223194dfcb1991151962086ef8f53770) made this pull request unmergeable. Please resolve the merge conflicts.

:lock: Merge conflict

:lock: Merge conflict

Looks like it failed with this:
:5thread 'ct-tokio' panicked at 'Cannot start a runtime from within a runtime. This happens because a function (like `block_on`) attempted to block the current thread while the thread is being used to drive asynchronous tasks.', /root/.cargo/registry/src/github.com-1ecc6299db9ec823/tokio-0.2.13/src/runtime/enter.rs:19:5

Looks like client.get_events on line 62 calls rt.block_on inside, which does not work inside tokio worker. I suggest adding NodeDebugClient::get_events_async method and then call it from DebugPortLogWorker

Is there a way for me to run this locally? It would be much easier if I could debug the cluster-test binary

We have --swarm flag that allows to run cluster test against test cluster.
To try this you first need to run cluster with cargo run -p libra-swarm, and then it will spit out command that allows to run limited version of cluster test against this swarm.
However, right now only flag that cluster test supports in this mode is --emit-tx, which does not use debug interface that you are working on.
One solution to this could be adding another method like tail_logs to BasicSwarmUtil(similar to ClusterTestRunner::tail_logs) and allowing to call it from --swarm. This would allow to test cases like this.
Would you want to take a look at doing this too?

. I suggest adding NodeDebugClient::get_events_async method and then call it from DebugPortLogWorker

Adding an async get_events method to NodeDebugClient is not enough to fix this, there are other places in NodeDebugClientwhere the inner runtime it's used. I think that a way to solve this is to create a version of  NodeDebugClient not relying on it's own runtime, a new struct called AsyncNodeDebugClient (or something along these lines). Otherwise we would get the same  Cannot start a runtime from within a runtime.
/// Implement default utility client for AsyncNodeDebugInterface
pub struct AsyncNodeDebugClient {
  addr: String,
  client: Option<NodeDebugInterfaceClient<tonic::transport::Channel>>,
}

impl AsyncNodeDebugClient {
  /// Create AsyncNodeDebugInterface from a valid socket address.
  pub fn new<A: AsRef<str>>(address: A, port: u16) -> Self {
      let addr = format!("http://{}:{}", address.as_ref(), port);

      Self {
          client: None,
          addr,
      }
  }

  async fn client(
      &mut self,
  ) -> Result<
      &mut NodeDebugInterfaceClient<tonic::transport::Channel>,
  > {
      if self.client.is_none() {
          self.client = Some(
            NodeDebugInterfaceClient::connect(self.addr.clone()).await?
          );
      }

      // client is guaranteed to be populated by the time we reach here
      Ok(self.client.as_mut().unwrap())
  }

  pub async fn get_node_metric<S: AsRef<str>>(&mut self, metric: S) -> Result<Option<i64>> {
      let metrics = self.get_node_metrics().await?;
      Ok(metrics.get(metric.as_ref()).cloned())
  }

  pub async fn get_node_metrics(&mut self) -> Result<HashMap<String, i64>> {
      let client = self.client().await?;
      let response = client.get_node_details(GetNodeDetailsRequest::default()).await
          .context("Unable to query Node metrics")?;

      response
          .into_inner()
          .stats
          .into_iter()
          .map(|(k, v)| match v.parse::<i64>() {
              Ok(v) => Ok((k, v)),
              Err(_) => Err(anyhow::format_err!(
                  "Failed to parse stat value to i64 {}: {}",
                  &k,
                  &v
              )),
          })
          .collect()
  }

  pub async fn get_events(&mut self) -> Result<GetEventsResponse> {
      let client = self.client().await?;
      let response = client.get_events(GetEventsRequest::default()).await
          .context("Unable to query Node events")?;
      Ok(response.into_inner())
  }
}

We have --swarm flag that allows to run cluster test against test cluster.
To try this you first need to run cluster with cargo run -p libra-swarm, and then it will spit out command that allows to run limited version of cluster test against this swarm (will be something like cargo run -p cluster-test --swarm ..more args..).
However, right now only flag that cluster test supports in this mode is --emit-tx, which does not use debug interface that you are working on.
One solution to this could be adding another method like tail_logs to BasicSwarmUtil(similar to ClusterTestRunner::tail_logs) and allowing to call it from --swarm. This would allow to test cases like this.
Would you want to take a look at doing this too?
Also note that for --tail_logs you would need different ports (config.debug_interface) instead of what is currently used in libra-swarm. Perhaps we can just spit out two command lines there, one for tx-emitter and one for debug interface test

I would be glad to work on this as well but I need more details on how this should work. ClusterTestRunner::tail_logs seems to be reading from a stream of ValidatorEvent events but it's not clear who generates the events.
fn tail_logs(self) {
        for log in self.logs.event_receiver {
            info!("{:?}", log);
        }
    }

It's not clear how  I should replicate this behaviour in BasicSwarmUtil

Thanks for looking into that!
Creating Async client makes sense.
Mirai error is probably spurious and can be ignored, however there are bunch of code style related failures:

:umbrella: The latest upstream changes (presumably 22584a4ac3eba0339ce07e5f38f5c40998cd4730) made this pull request unmergeable. Please resolve the merge conflicts.

Closing this out as its been sitting here for a looong time. Feel free to re-open.


It keeps on crashing when loading a custom icon

Describe your bug here.
every time I try and add a custom icon in the character editor it crashes
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no, it was a fresh copy of the engine that had a few other assets added

show logs


build.yml:  Workflow aligned with nuttx repo

Summary
build.yml: Limit the GitHub Runners
see https://github.com/apache/nuttx/pull/13412
Impact
none
Testing
CI

[Experimental Bot, please feedback here]
Yes, this PR appears to meet the NuttX requirements.
Here's why:

Clear Summary:  The summary clearly states the purpose of the change ("Limit the GitHub Runners") and links to the relevant NuttX pull request.
Minimal Impact: The PR states there is no impact beyond the CI build process itself.
Testing Addressed: While not providing specific logs, the PR confirms that testing was done within the CI environment, which is appropriate for a change focused on the build system.

Potential Improvements:

Expanded Testing: While CI testing is great, specifying the tested host and target combinations (even briefly) would strengthen the testing section.
Impact Details: Even though the impact is minimal, a sentence explaining how the GitHub Runners are being limited would be helpful for reviewers.


Feature: Allow access to viewed feature?

Feature Description
No idea if the REST API allows this but if possible it might be good to allow marking files as viewed like you can via the web GUI:

This is a good idea but it's some state we'd have to manage locally.
Looking at Gitlab's site, it seems like they're storing this data in the user's browser in Local Storage, at the following key:
`/{org}/{project_name}/-/merge_requests/{merge_request_id_number}-file-reviews`

And that object has some sort of data structure that retains information about the file.
I'm guessing they store somehow the hash of the file itself, and then the commit that you're "reviewing" at the time you press "view file" or something like that. We'd have to replicate this functionality locally, they don't have an API for it.

Hmm, OK that's far more complicated than I thought it would be. It's a pity it's not covered by the API. I'd consider this a very low priority feature request.

I'm going to close this for now, but we can reopen if we want to replicate a "viewed" functionality in the future.


US RD200 with FR:HA

my RD200 purchased 2022-01-30 in the US has serial/name starting with FR:HA.
It appears to be working fine, although I don't have any long term data yet as I just got it.

Thanks. So weird there are at least 3 US serial number prefixes now.


Add Pearl MAI-USDR sLP

MAI-USDR sLP pearl-mimatic-usdr

Also includes polygon:pearl:add command and other platform data for Pearl.

70k tvl


Fixes tar is nil on postExecutorSecondStageSteps.commitImageStep

It looks like /tmp/.s2i/image_metadata.json was never meant to be created during the assemble-runtime step, only during the assemble step, so tar was nil in that specific commitImageStep
Fixes #838

[test]

@openshift/sig-developer-experience ptal

lgtm

[merge]


Update text related to the recent Stand Panel token name change request

gripper-color-drag in light.json should be gripper-drag like what we have in the dark theme.
Motivation and context
Updated text.
Related issue

Types of changes

 [x] Patch (bug fixes, typos, mistakes; non-breaking change which fixes an issue)
 [ ] Minor (add a new token, changing a value, deprecating a token; non-breaking change which adds functionality)
 [ ] Major (deleting a token, changing token value type, renaming a token by deprecating the old one; fix or feature that would cause existing functionality to change)
Checklist:

 [x] I have signed the [Adobe Open Source CLA](https://opensource.adobe.com/cla.html).
 [x] I updated the token in all applicable sets. This applies if updating, adding, or deleting a token that has data across different sets (for example, if the value differs across color themes.)

TESTING spectrum2
READING spectrum-tokens: /spectrum-tokens/packages/tokens/src/
READING   tokens studio: /spectrum-tokens-studio-data/src/tokens-studio/spectrum2-colors/
READING   tokens studio: /spectrum-tokens-studio-data/src/tokens-studio/spectrum2-non-colors/

WARNING - STUDIO INCLUDES DEPRECATED TOKEN: overlay-opacity/dark [31d5b502-6266-4309-8f8a-3892e6e158da]
WARNING - STUDIO INCLUDES DEPRECATED TOKEN: overlay-opacity/light [26b9803c-577f-4a29-badd-dfc459e8b73e]

STUDIO SYNC SUMMARY
UPDATED METADATA - SPECTRUM UUID: spectrum2/component.Component.standard-panel.gripper-drag

SPECTRUM SYNC SUMMARY
MODIFIED TOKENS: (1) standard-panel-gripper-color-drag
DEPRECATED TOKENS: (1) standard-panel-gripper-color-drag/dark

TEST COMPLETE


Input source

This implements input source on TransactionRecordsById and tests it.
The test fails.

looks like this is an agglomeration of multiple changes
-mocks
-move some methods from transactions.rs into transaction_records_by_id.rs
-impl InputSource on transaction_records_by_id and test
should be easy enough to separate. thats my next goal


Implement ell1 to measure distance in Robustness evaluation

Description
There are several ways to measure distance for vectors.We aim to apply "L1"/"ell1" (Manhattan distance), defined as

The current evaluation notebook seems to use Euclidean Distance, pls change @idrees535
References
https://www.overleaf.com/project/667bebbdb70211dfcff39701

solved, L1 implementation is correct


refactor: adapting plugins to new Go SDK version

Co-authored-by: Leonardo Grasso<EMAIL_ADDRESS>Signed-off-by: Jason Dellaluce<EMAIL_ADDRESS>
What type of PR is this?
/kind cleanup
/kind design
Any specific area of the project related to this PR?
/area source-plugins
/area extractor-plugins
What this PR does / why we need it:
This alignes the json, dummy, and cloudtrail plugins to to the new Go SDK design and version. Some legacy code portions have been removed from each plugin, and cloudtrail has been broken down in three different source files to better separate the plugin framework methods from the business logic.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

/ok-to-test

/kind cleanup
/kind design
/check-dco

/area source-plugins
/area extractor-plugins

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jasondellaluce
To complete the pull request process, please assign mstemm after the PR has been reviewed.
You can assign the PR to them by writing /assign @mstemm in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, 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 [mstemm]

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

LGTM label has been added.  Git tree hash: b7dafb7a763547db74b2d4f60d2449c829de69f2


After upgrate to Net Core 2.0 Localization in View (Inject) is not Working

Hello,
After upgrate to Net Core 2.0 Localization in View is not Working, but In Controller works .
I tried to create a default project in Net Core 2.0 and is not working too .
ConfigureServices
-  services.AddLocalization(options => options.ResourcesPath = "Resources");

services.AddMvc()
            .AddViewLocalization(LanguageViewLocationExpanderFormat.Suffix)
            .AddDataAnnotationsLocalization();
In Folder Resource I have SharedResource.fr-FR.resx
Shared Resource Class :
namespace PL
{
    public class SharedResource
    {
    }
}
- @inject Microsoft.Extensions.Localization.IStringLocalizer<PL.SharedResource> SharedResource
-  @SharedLocalizer["Add"]
public IStringLocalizer _SharedResource;
 public HomeController(IStringLocalizer sharedResource)
        {
            CultureInfo uiCulture = new CultureInfo("fr-FR");
            CultureInfo.CurrentUICulture = uiCulture;
            CultureInfo.CurrentCulture = new CultureInfo("fr-FR");
            _SharedResource = sharedResource;
        }
But I I test with a ViewData from controller is working
 ViewData["Message"] = _SharedResource["Add"];
Thank's

@kichalla another investigation.

@lucianparvu I was unable to repro the problem at my end, but to be sure, can you try finding out what the culture is on the view?
For example, you can try printing them out in the view:

CurrentCulture: @System.Globalization.CultureInfo.CurrentCulture.ToString()

CurrentUICulture: @System.Globalization.CultureInfo.CurrentUICulture.ToString()

@SharedResource["Add"]

Based on your code, looks like you were following this article. I wasn't able to repro the issue, so closing it for now. If you can still repro it, could you share a small repro app
https://docs.microsoft.com/en-us/aspnet/core/fundamentals/localization


Remove explicit template instantiations

I know of no reason why explicit template instantiations should ever appear in a header file: code that includes the header file will already implicitly instantiate any templates that it uses. (See https://en.cppreference.com/w/cpp/language/class_template.) The effect of the instantiations is only to copy unnecessary object code into every translation unit that calls this library.
In any case, for trivial wrapper structs it's always better to use function inlining anyway; for performance, we don't actually want to be jumping through extern functions when dealing with these wrappers.

This was giving a lot of compilation warnings during unit tests, all along the lines of this:
In file included from /Users/pgoulart/projects/clarabel/Clarabel.cpp/include/Clarabel:6:
/Users/pgoulart/projects/clarabel/Clarabel.cpp/include/cpp/DefaultSettings.h:300:25: warning: 'clarabel_DefaultSettings_f64_default' has C-linkage specified, but returns incomplete type 'DefaultSettings<double>' which could be incompatible with C [-Wreturn-type-c-linkage]
DefaultSettings<double> clarabel_DefaultSettings_f64_default();

The issue appear to be that the template instantiations make the compiler accept things like this.
Happy to remerge this if we can resolve the warnings.

Thanks for the feedback!  I learned something new about C++ today.
When I have time to circle back to this, I'll work on developing a working patch, and open a new PR.


activity: Add date joined to /activity page.

A useful metric to add to the /activity page is the date a realm joined. It can go as the last column, and use a format like 2017-05-19 to simplify sorting.
The current columns are

The code for this will go in analytics/views.py, likely in the realm_summary_table function.
This is a good first project for someone who knows SQL.
Notes:
It's possible that to get to the /activity page in the dev environment, you have to log in as Iago.

Hi!
I would like to take up this issue.

@Riju19 Can you comment @zulipbot claim to take the issue up? Thanks!

@zulipbot claim

Hi! Since I’m new to this whole thing, it’ll take me a while to understand how it works. Just wanted to know that the date a realm joined is already mentioned somewhere, or do I need to code that part too.

@Riju19 I took a look at this issue earlier. Most databases automatically create some kind of 'created at' date field or attribute when a record is saved. In this case, there is an attribute on the realm records that is equivalent, 'date_created.' At least that is what I think is meant by 'date joined.' You'll have to edit a few files here to make this ability work and in order to check that it's working properly can create realms in the dev environment and see if they show up.

I see that no one is assigned this issue, but someone has already claimed it above. So can someone clear if this is up for grabs. Thank you

@Riju19 are you still working on this?
The data is already present in the realm.date_created field; we just need to display it.

Yes, I am !

It says that no one is assigned, can that be changed then because it might cause further confusion.

Can I take this issue ?

@kriti21 it seems like @Riju19 is still working on this issue.

Okay

My pull request just needs to be reviewed. In case it doesn’t work, you can take over.

@zulipbot claim

So in the /activity view, under the "Counts" tab there already exists a column called "Created" which uses "realm.date_created" field. @rishig can you please elaborate a little on this issue? Not quite sure what you're looking for.

Thanks for looking into this!
It looks like this issue was resolved by someone who didn't realize this issue existed. Sorry about that; let me know if you need help finding a different issue to work on!


Unable to install cmdlets. "Unable to read file...to calculate its hash."

Hello,
Running through the installation instructions...link

Open PowerShell as Admin
Run Install-Module -Name MicrosoftPowerBIMgmt
Accept PSGallery untrusted repository message with a Y
Cmdlets start to load then...
Error:

PackageManagement\Install-Package : Unable to read file C:\Users\me\AppData\Local\Temp\442391038\MicrosoftPowerBIM
gmt.Data\lib\netstandard2.0\Microsoft.PowerBI.Common.Api.dll to calculate its hash.
At C:\Program Files\WindowsPowerShell\Modules\PowerShellGet\<IP_ADDRESS>\PSModule.psm1:1809 char:21
+ ...          $null = PackageManagement\Install-Package @PSBoundParameters
+                      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (Microsoft.Power....InstallPackage:InstallPackage) [Install-Package],
   Exception
    + FullyQualifiedErrorId : UnableToReadFileToHash,Microsoft.PowerShell.Commands.TestFileCatalogCommand,Microsoft.Po
   werShell.PackageManagement.Cmdlets.InstallPackage

I have .NET 4.7.2 installed already...

How can I get past this?
Thank you

Here is what worked so far. Install-Module -Name MicrosoftPowerBIMgmt -Force -SkipPublisherCheck
Could use some insights on which flag made this work.

Was it the -Force or the -SkipPublisherCheck flag?
I imagine the latter, so, what is the security effects of skipping the publisher check?
How do we get the publisher check enabled?

Thank you

Hello.
Please check the cmdlet publisher sig/hash of the MicrosoftPowerBIMgmt in PSGallery. It appears it is incorrect. Would like to hear back on this issue.
Thank you

@ericthomas1 what was your powershell version?
We are following code signing process and will revisit it based on https://docs.microsoft.com/en-us/powershell/scripting/gallery/concepts/publishing-guidelines?view=powershell-7.1#sign-your-code

I'm not sure what version of PowerShell I was running 6-7 months ago when I reported this. At the moment, I'm running 5.1.18362.1171. I don't specifically remember updating, but not sure if any programs/plugins/extensions/etc. have updated PowerShell automatically/in the background since 8/2020.


[WIP][FEATURE] Heartbeat on query execution

Is your feature request related to a problem?
A clear and concise description of what the problem is, e.g. I'm always frustrated when [...]
What solution would you like?
A clear and concise description of what you want to happen.
What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.
Do you have any additional context?
Add any other context or screenshots about the feature request here.

[Catch All Triage - 1, 2, 3]
@noCharger Please provide more detail if you're working on this feature.


New empty key destroys the json structure

In practice, I also solve the configuration with MCJson. Therefore, due to new situations, unexpected errors also occur. If I add a key to an existing structure (asJson, LoadFromFile) (main key, without value, because it won't have any), it destroys the json, and next time it won't even be loaded.
It's possible that an empty key (category) shouldn't be added this way, but I don't think it should spoil the json structure.
procedure TForm1.Button1Click(Sender: TObject);
const
  fn = 'test.json';
var
  N: TMcJsonItem;
begin
  DeleteFile(fn);

  N := TMcJsonItem.Create;
  try
    N.AsJSON := '{"i": 123, "f": 123.456, "s": "abc", "b": true, "n": null}';

    //N.SaveToFile(fn);
    //N.LoadFromFile(fn);
    //Memo1.Lines.LoadFromFile(fn);

    N.Add('EmptyNewKey'); // <- this destroy json structure

    N.SaveToFile(fn);
    N.LoadFromFile(fn);
    Memo1.Lines.LoadFromFile(fn);

  except
    N.Free;
  end;
end;

Hi!
Yeap, it need a fix. I'm working on it.
Meanwhile, try:
N.Add('EmptyNewKey')->AsString = '';

Thanks for the tip! :)

But the result is the key, with empty value:

N.Add('EmptyNewKey')->AsString = '';
I think this works.

Works, but the result is different.
excepted: (o = EmptyNewKey)
{
	"o": {
		"k1": "v1",
		"k2": "v2"
	}
}

your example (invalid json):
{
	"o" = "" {
		"k1": "v1",
		"k2": "v2"
	}
}

Created with a subkey, it obviously works, and you also wrote that this is a bug that you are fixing, thank you!

I think we are diverging.
Empty key is different from empty value.
I didn't get your sequence.
A different example
Here N.AsJson:='{"o": {"k1":"v1", "k2":"v2"}}'; works fine.
Where in this JSON string do you want a empty value?
Doubt

Created with a subkey

How? Please, always make your example as clear as possible.
Again, different examples.

and you also wrote that this is a bug that you are fixing

The bug is as you shown and I agreed:
N.Add('EmptyNewKey');
To sum up
With all my respect, let me say something: I think you didn't get how a little piece of code published into Github with MIT license works.
If you see something wrong, this is not a Helpdesk platform, OK?
Try to collaborate, mainly spending some time with a good error description.
For example, your first post is what I expect. The second and third ones are confusing/misleading.

As I wrote, I use a translator to write text. You have written a very good library, and I think it will only improve if I flag errors and make comments. Since I also write programs, I know this is not a help desk, but as I wrote, your library will get better and better. If you fix a mistake after 5 years, I can't even say a word :)
I try to be as precise as possible, but it mainly depends on the translator, who I think does a good job anyway. Thank you for your patience!

English isn't a problem.
I was talking about the code examples.
My suggestion is to keep them clear as water and the issues more atomic.
Best.

Thank you, json isn't damaged now, but the result is not perfect, because I got this:
"Options": "",
Mean: Key with value (as you suggested earlier).
But I obviously only wanted to add a key alone (parent key), since if I gave it together with a value, there would have been no problem.
So I wanted from it:
JsonConfig.Add(cOptions);
to get this result in json (new parent key):
"Options":
or
"Options": {}
But in the end there is no problem, because if I add it together with a subkey and its value, I get the correct result. :
    JsonTmp := JsonConfig.Add(cOptions);
    JsonTmp.Add(cAutoBackup).AsBoolean := FAutoBackup;

"Options": {
    "AutoBackup": true
}

I tried to be as precise as possible. Thank you!

Hi!
With good examples like these, it is simpler to follow you along.
Important: "Options": and {"Options":} are invalid JSON objects.
If you want a object with empty value, different from the default jvtString, try this:
JsonConfig.Add(cOptions).ItemType := jitObject;

It will generate:
{
  "Options": {}
}

I've included a test about this into Test05() (next commit).
Best.

"jitObject" is the key, thank you!


Verify if context is an instance of FeatureContext

When using multiple contexts KernelAwareInitializer and more specifically the initializeContext method overrides the FeatureContext by another one hence making the rebootKernel method using the wrong context class.
This might not be the best way to fix this so if you have any idea I'll be glad to help.
Cheers.

will review. Hoping to get tests working on these PRs just need to get more permissions @phleobon

@alnutile Thanks for the reply 😃
I just want to point out that this solution makes the assumption that the user will make FeatureContext extend MinkContext (which is common) but if the user actually import MinkContext from within the behat.yml file it won't work.
One way around this would be to check if the context extends MinkContext and if yes use it in the KernelAwareInitializer but it would need to require "behat/mink-extension" in order to make this check possible.

is there any advantages to "...  import MinkContext from within the behat.yml file.."
Just curious?

As far as I know there isn't any advantages to doing it this way, this is just a matter of preference.
After having just review the PR I made, i realized that my fix is wrong even though it works.
I'll explain this tomorrow.


Add more Locate Junit test

Pull Request Test Coverage Report for Build 402

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.887%

Totals

Change from base Build 399:
0.0%

Covered Lines:
2223

Relevant Lines:
2650

💛  - Coveralls


Arnej/intermediate parser 2

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.
@bratseth and/or @baldersheim please review

For things that are immutable we usually skip the "get" (as in "getName") as there will never be a "set".


DatePicker on close opens mobile Keyboard

                                    <IconSettings iconPath="/assets/icons">
                                        <Datepicker
                                        labels={{
                                            label: 'Date of Birth'
                                        }}
                                        onChange={(event, data) => {
                                            console.log(data);
                                            console.log(data.date.getUTCFullYear());
                                            console.log(data.date.getUTCDate());
                                            console.log(data.date.getUTCMonth()); //Need to add 1 (0 indexed)
                                        }}
                                        relativeYearFrom={-70}
                                        relativeYearTo={-10}
                                        value={this.state.dateOfBirth}
                                        //menuPosition="overflowBoundaryElement"
                                        />
                                    </IconSettings>

When the date is selected in the datePicker, the virtual calendar goes away but the keyboard on your phone will open afterwards. This is an annoying behavior. Anyone know of a way to disable the keyboard onClose?

passing
input={
<Input
readOnly={true}
required={this.state.hasSkillsPassKey && true}
/>
}`
into the component is a hacky fix but it removes the styling from the component like the border around the date picker, requiring custom css. There really should be a better way to resolve this


Complex utility functions if imaginary part is 0

Are there some similar utility functions for the Complex number type as in the Guava DoubleMath utility class, if the imaginary part is 0.0?
isMathematicalInteger
https://github.com/google/guava/blob/45958948bea8f7aaf38c1a3dd200ce0263c0bcfa/android/guava/src/com/google/common/math/DoubleMath.java#L286
roundToInt:
https://github.com/google/guava/blob/45958948bea8f7aaf38c1a3dd200ce0263c0bcfa/android/guava/src/com/google/common/math/DoubleMath.java#L131

For questions, please use the new forum at https://forum.orekit.org/c/hipparchus-usage/.
There are not (yet) similar functions in Complex. We have added quite a large number of functions so Complex now implements the new CalculusFieldElement interface, for the sake of solving issue #67. The closest function to your second need would be round, but it is in the RealFieldElement interface, not in the CalculusFieldElement class. We could move it up as explained in the comment I wrote here.
We could implement the first method directly in Complex, without reference to any higher level interface.


[Expose SPIFFE ID] add logging when multiple IDs

Sorry for the delay. I was pulled in a number of directions by other projects :-)

Looks like the failed Kokoro builds are unrelated. Merging now.


when i am try go get github.com/veandco/go-sdl2/sdl

or compile sdl2 test,
i am received this error
src/github.com/veandco/go-sdl2/sdl/hints.go:86:50: could not determine kind of name for C.SDL_HINT_ANDROID_SEPARATE_MOUSE_AND_TOUCH
i am use osx mojave.
latest sdl2
and latest golang.
please help.

Hi @sadek69,
That issue should have been fixed in the master branch. Could you try running go get -v -u github.com/veandco/go-sdl2/sdl?

Hi @veeableful thank you who you try to help but after this command i am received this error
Aborting
package github.com/veandco/go-sdl2/sdl: exit status 1
he try to start update but exit with this mistake....
maybe you have more idea what me do?

Hi @sadek69, is that the full output? If you run the command I posted, I believe it should at least show:
github.com/veandco/go-sdl2 (download)

exes-Mac-mini:go _exe$ go get -v -u github.com/veandco/go-sdl2/sdl
github.com/veandco/go-sdl2 (download)
cd /Users/_exe/go/src/github.com/veandco/go-sdl2; git pull --ff-only
error: The following untracked working tree files would be overwritten by merge:
.go-sdl2-libs/include/SDL2/SDL.h
.go-sdl2-libs/include/SDL2/SDL2_framerate.h
.go-sdl2-libs/include/SDL2/SDL2_gfxPrimitives.h
.go-sdl2-libs/include/SDL2/SDL2_imageFilter.h
.go-sdl2-libs/include/SDL2/SDL2_rotozoom.h
.go-sdl2-libs/include/SDL2/SDL_assert.h
.go-sdl2-libs/include/SDL2/SDL_atomic.h
.go-sdl2-libs/include/SDL2/SDL_audio.h
.go-sdl2-libs/include/SDL2/SDL_bits.h
.go-sdl2-libs/include/SDL2/SDL_blendmode.h
.go-sdl2-libs/include/SDL2/SDL_clipboard.h
.go-sdl2-libs/include/SDL2/SDL_config.h
.go-sdl2-libs/include/SDL2/SDL_config_android_arm.h
.go-sdl2-libs/include/SDL2/SDL_config_darwin_amd64.h
.go-sdl2-libs/include/SDL2/SDL_config_linux_386.h
.go-sdl2-libs/include/SDL2/SDL_config_linux_amd64.h
.go-sdl2-libs/include/SDL2/SDL_config_linux_arm.h
.go-sdl2-libs/include/SDL2/SDL_config_windows_386.h
.go-sdl2-libs/include/SDL2/SDL_config_windows_amd64.h
.go-sdl2-libs/include/SDL2/SDL_cpuinfo.h
.go-sdl2-libs/include/SDL2/SDL_egl.h
.go-sdl2-libs/include/SDL2/SDL_endian.h
.go-sdl2-libs/include/SDL2/SDL_error.h
.go-sdl2-libs/include/SDL2/SDL_events.h
.go-sdl2-libs/include/SDL2/SDL_filesystem.h
.go-sdl2-libs/include/SDL2/SDL_gamecontroller.h
.go-sdl2-libs/include/SDL2/SDL_gesture.h
.go-sdl2-libs/include/SDL2/SDL_haptic.h
.go-sdl2-libs/include/SDL2/SDL_hints.h
.go-sdl2-libs/include/SDL2/SDL_image.h
.go-sdl2-libs/include/SDL2/SDL_joystick.h
.go-sdl2-libs/include/SDL2/SDL_keyboard.h
.go-sdl2-libs/include/SDL2/SDL_keycode.h
.go-sdl2-libs/include/SDL2/SDL_loadso.h
.go-sdl2-libs/include/SDL2/SDL_log.h
.go-sdl2-libs/include/SDL2/SDL_main.h
.go-sdl2-libs/include/SDL2/SDL_messagebox.h
.go-sdl2-libs/include/SDL2/SDL_mixer.h
.go-sdl2-libs/include/SDL2/SDL_mouse.h
.go-sdl2-libs/include/SDL2/SDL_mutex.h
.go-sdl2-libs/include/SDL2/SDL_name.h
.go-sdl2-libs/include/SDL2/SDL_opengl.h
.go-sdl2-libs/include/SDL2/SDL_opengl_glext.h
.go-sdl2-libs/include/SDL2/SDL_opengles.h
.go-sdl2-libs/include/SDL2/SDL_opengles2.h
.go-sdl2-libs/include/SDL2/SDL_opengles2_gl2.h
.go-sdl2-libs/include/SDL2/SDL_opengles2_gl2ext.h
.go-sdl2-libs/include/SDL2/SDL_opengles2_gl2platform.h
.go-sdl2-libs/include/SDL2/SDL_opengles2_khrplatform.h
.go-sdl2-libs/include/SDL2/SDL_pixels.h
.go-sdl2-libs/include/SDL2/SDL_platform.h
.go-sdl2-libs/include/SDL2/SDL_power.h
.go-sdl2-libs/include/SDL2/SDL_quit.h
.go-sdl2-libs/include/SDL2/SDL_rect.h
.go-sdl2-libs/include/SDL2/SDL_render.h
.go-sdl2-libs/include/SDL2/SDL_revision.h
.go-sdl2-libs/include/SDL2/SDL_rwops.h
.go-sdl2-libs/include/SDL2/SDL_scancode.h
.go-sdl2-libs/include/SDL2/SDL_shape.h
.go-sdl2-libs/include/SDL2/SDL_stdinc.h
.go-sdl2-libs/include/SDL2/SDL_surface.h
.go-sdl2-libs/include/SDL2/SDL_system.h
.go-sdl2-libs/include/SDL2/SDL_syswm.h
.go-sdl2-libs/include/SDL2/SDL_test.h
.go-sdl2-libs/include/SDL2/SDL_test_assert.h
.go-sdl2-libs/include/SDL2/SDL_test_common.h
.go-sdl2-libs/include/SDL2/SDL_test_compare.h
.go-sdl2-libs/include/SDL2/SDL_test_crc32.h
.go-sdl2-libs/include/SDL2/SDL_test_font.h
.go-sdl2-libs/include/SDL2/SDL_test_fuzzer.h
.go-sdl2-libs/include/SDL2/SDL_test_harness.h
.go-sdl2-libs/include/SDL2/SDL_test_images.h
.go-sdl2-libs/include/SDL2/SDL_test_log.h
.go-sdl2-libs/include/SDL2/SDL_test_md5.h
.go-sdl2-libs/include/SDL2/SDL_test_memory.h
.go-sdl2-libs/include/SDL2/SDL_test_random.h
.go-sdl2-libs/include/SDL2/SDL_thread.h
.go-sdl2-libs/include/SDL2/SDL_timer.h
.go-sdl2-libs/include/SDL2/SDL_touch.h
.go-sdl2-libs/include/SDL2/SDL_ttf.h
.go-sdl2-libs/include/SDL2/SDL_types.h
.go-sdl2-libs/include/SDL2/SDL_version.h
.go-sdl2-libs/include/SDL2/SDL_video.h
.go-sdl2-libs/include/SDL2/SDL_vulkan.h
.go-sdl2-libs/include/SDL2/begin_code.h
.go-sdl2-libs/include/SDL2/close_code.h
.go-sdl2-libs/libSDL2_android_arm.a
.go-sdl2-libs/libSDL2_darwin_amd64.a
.go-sdl2-libs/libSDL2_gfx_android_arm.a
.go-sdl2-libs/libSDL2_gfx_darwin_amd64.a
.go-sdl2-libs/libSDL2_gfx_linux_386.a
.go-sdl2-libs/libSDL2_gfx_linux_amd64.a
.go-sdl2-libs/libSDL2_gfx_linux_arm.a
.go-sdl2
Aborting
package github.com/veandco/go-sdl2/sdl: exit status 1
this is full output!

Hi @sadek69, could you delete the whole directory like so:
rm -rf /Users/_exe/go/src/github.com/veandco/go-sdl2

and run go get -u -v github.com/veandco/go-sdl2/sdl again?

thank you.now all work.


fix(tendermint-rpc): add earliest block to sync info

This should fix issue #1448. Those fields are returned from /status endpoint but I haven't tested this code. I'm open to any suggestions how I can test it easily.

Thew CI runs Tendermint version 0.34 and 0.37.
Looking at the code you linked there it seems like the fields may not be set if the metadata about the earliest block is missing

So all 4 fields are probably optional and some sort of unset/default/zero value. Could you just dump status.syncInfo and check what you actually get from the node?


Add a GeoIP processor.

Provide a new processor which can enrich Data Prepper events with location information using a provided IP address.
The minimal configuration is to provide a source_key with the JSON Pointer key path.
processor:
  - geoip:
      source_key: "peer/ip"

Additionally, this plugin should be able to use either a MaxMind GeoIP Lite2 database or the GeoIP2 Commercial Licensing database. The Data Prepper author must provide information for configuring the commercial license.
The pipeline author can also specify an optional target_key property to specify where the location fields are written. By default, this will be the root of the event.
Example 1 - Minimal Configuration
processor:
  - geoip:
      source_key: "peer/ip"

Input Event:
"peer" : {
  "ip" : "<IP_ADDRESS>"
  "host" : "example.org"
}
"status" : "success"

Output Event:
"peer" : {
  "ip" : "<IP_ADDRESS>"
  "host" : "example.org"
}
"status" : "success"
"country" : "United States"
"city_name" : "Seattle"
"latitude" : 47.64097
"longitude" : 122.25894
"zip_code" : "98115"

Example 2 - Target Key
processor:
  - geoip:
      source_key: "peer/ip"
      target_key: "location"

Input Event:
"peer" : {
  "ip" : "<IP_ADDRESS>"
  "host" : "example.org"
}
"status" : "success"

Output Event:
"peer" : {
  "ip" : "<IP_ADDRESS>"
  "host" : "example.org"
}
"location" : {
  "status" : "success"
  "country" : "United States"
  "city_name" : "Seattle"
  "latitude" : "47.64097"
  "longitude" : "122.25894"
  "zip_code" : "98115"
}

Is your feature request related to a problem? Please describe.
Pipeline users want to add geographical location details based on IP address to enrich data for analytical purposes.
Describe the solution you'd like
Create GeoIP plugin which will enrich traces with a geo location field, derived from the IP address. This will give customers a more insightful value that will allow them to visualize where traces are coming originating from.
Plugin should be able to use Geo data from MaxMind database / Amazon location Service / User provided path of Geodata.
source:
    - geoip:
        source_key: "peer/ip"
        target_key: "location_info"
        database_path: "path/to/database.mmdb
        geoip_attributes: ["location", "city_name", "country_name"]

GeoIP attributes should have many optional attributes like ip, city_name, country_name, continent_code, country_iso_code, postal_code, region_name, region_code, timezone, location, latitude, longitude . Default should be all values included. Location attribute refers to latitude and longitude.
Design Considerations: Any Geo data - consumer needs to keep updating the latest data in the plugin periodically
Additional context
Resources :
MaxMind data: https://dev.maxmind.com/geoip/geolite2-free-geolocation-data?lang=en
Amazon Location Services: https://aws.amazon.com/location/, https://docs.aws.amazon.com/location/latest/developerguide/search-place-index-geocoding.html
Java Implementation : https://maxmind.github.io/GeoIP2-java/

@ashoktelukuntla , I'd like to propose that we decouple the service configuration from the pipeline.
In data-prepper-config.yaml
plugins:
  geoip:
    location_service:
      maxmind:
        database_path: /usr/share/local/maxmind

The pipelines need only have:
pipeline1:
  processor:
    - geoip:
         source_key: "peer/ip"
pipeline2:
  processor:
    - geoip:
         source_key: "attributes/ip_address"

The work being done for #2588 can help support this type of configuration

Hi team, this feature would also be tremendously useful for security analytics usecases. Can you share any timelines for this feature? Thanks


Add a small note explaining that the keybinds are based on the QWERTY layout

I think this is fine for now; in the future we might want to add alt text or something similar by the header to take up less vertical space.


Pattern not found

Pattern not found
I'm using OmniSharp on Neovim for the first time and I can't seem to get Omni-completion working.
init.vim (or .vimrc for vim)
""" Plugins

call plug#begin(stdpath('data') . '/plugged')

Plug 'vim-airline/vim-airline'
Plug 'vim-airline/vim-airline-themes'
Plug 'OmniSharp/omnisharp-vim'
Plug 'nickspoons/vim-sharpenup'
Plug 'scrooloose/nerdtree'

call plug#end()

""" Plugin configurations

" Enable powerline symbols
let g:airline_powerline_fonts = 1

""" Settings

" This is needed and idk why
filetype plugin on

" This is cursed, but I'll use it anyways
set mouse=a

" Enable line numbers
set number

" Disable menu bar in gvim
set guioptions=

" Enable 256 colours
set t_Co=256

" Enable syntax highlighting
syntax on

" Set the color scheme
colorscheme gruvbox

after/init.vim (I don't know the Vim equivalent)
AirlineTheme gruvbox

.NET info
dotnet --version: 5.0.203
dotnet --list-sdks: 5.0.203 [C:\Program Files\dotnet\sdk]
msbuild --version: <IP_ADDRESS>03
omnisharp msbuild --version: MSBuild completely screws up here, but OmniSharp uses the one from Visual Studio, so it is ok.
OmniSharp log
NVIM v0.4.4

OmniSharp server started.
    Path: C:\Users\CENSORED\AppData\Local\omnisharp-vim\omnisharp-roslyn\OmniSharp.exe
    Target: C:\Users\CENSORED\Programming\CS\HelloWorld
    PID: 55084

[info]: OmniSharp.Stdio.Host
        Starting OmniSharp on Windows 6.2.9200.0 (x64)
[info]: OmniSharp.Services.DotNetCliService
        DotNetPath set to dotnet
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Located 2 MSBuild instance(s)
            1: Visual Studio Community 2019 16.9.31313.79 - "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\MSBuild\Current\Bin"
            2: StandAlone 16.11.0 - "C:\Users\CENSORED\AppData\Local\omnisharp-vim\omnisharp-roslyn\.msbuild\Current\Bin"
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Registered MSBuild instance: Visual Studio Community 2019 16.9.31313.79 - "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\MSBuild\Current\Bin"
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpFormattingWorkspaceOptionsProvider, Order: 0
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.Completion.CompletionOptionsProvider, Order: 0
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.RenameWorkspaceOptionsProvider, Order: 100
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.ImplementTypeWorkspaceOptionsProvider, Order: 110
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.BlockStructureWorkspaceOptionsProvider, Order: 140
[info]: OmniSharp.Cake.CakeProjectSystem
        Detecting Cake files in 'C:\Users\CENSORED\Programming\CS\HelloWorld'.
[info]: OmniSharp.Cake.CakeProjectSystem
        Could not find any Cake files
[info]: OmniSharp.MSBuild.ProjectSystem
        No solution files found in 'C:\Users\CENSORED\Programming\CS\HelloWorld'
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for 'C:\Users\CENSORED\Programming\CS\HelloWorld\HelloWorld.csproj'
[info]: OmniSharp.Script.ScriptProjectSystem
        Detecting CSX files in 'C:\Users\CENSORED\Programming\CS\HelloWorld'.
[info]: OmniSharp.Script.ScriptProjectSystem
        Could not find any CSX files
[info]: OmniSharp.WorkspaceInitializer
        Configuration finished.
[info]: OmniSharp.Stdio.Host
        Omnisharp server running using Stdio at location 'C:\Users\CENSORED\Programming\CS\HelloWorld' on host -1.
[info]: OmniSharp.MSBuild.ProjectManager
        Loading project: C:\Users\CENSORED\Programming\CS\HelloWorld\HelloWorld.csproj
[info]: OmniSharp.MSBuild.ProjectManager
        Successfully loaded project file 'C:\Users\CENSORED\Programming\CS\HelloWorld\HelloWorld.csproj'.
All projects loaded
[info]: OmniSharp.MSBuild.ProjectManager
        Adding project 'C:\Users\CENSORED\Programming\CS\HelloWorld\HelloWorld.csproj'
[info]: OmniSharp.MSBuild.ProjectManager
        Update project: HelloWorld
Replaying on-load requests

Extra info

I'm using OmniSharp through Plugged.
The project targets .NET 5.0
There is no solution, just a project.
:version returns NVIM v0.4.4.
I've already looked at other issues and they didn't help.

OmniSharp-vim uses vim omni-completion. Trigger it from insert mode by typing <C-x><C-o>.
If you want completions to be displayed automatically as you type, you'll need an auto-completion plugin, as described in the README. The example config uses asyncomplete.
To verify that OmniSharp-vim is working as expected, Try some things like :OmniSharpGotoDefinition, :OmniSharpFindUsages, :OmniSharpGlobalCodeCheck.

I don't know if you want feedback on your init.vim, but the line "This is needed and idk why" triggered me :smile:. As you are using vim-plug, you don't need line 21 filetype plugin on, because vim-plug does this for you. And I expect the line you have in after/init.vim could go directly in your init.vim, after you set colorscheme gruvbox

I've been using <C-x><C-o> and that didn't work. The thing that surprised me was that OmniSharp commands didn't do anything and didn't throw any errors as far as I know. Also, I'm quite new to Vim, so I didn't know what filetype plugin on did.

Ohhhh ... neovim 0.4.4 on Windows. That doesn't work 😅
There was a neovim bug, which I reported, and was fixed. See https://github.com/OmniSharp/omnisharp-vim/issues/499 for details.
So your options are to download a nightly build of neovim, or use vim. Sorry I didn't put the pieces together immediately.

Norton hates Neovim 5.0 (just like many other things) (32 bit and 64 bit), I don't know how to get the missing dependencies to build it with MinGW and I don't know how to get Plug working with Vim. I'll just wait for 5.0 to release unless there are workarounds. I'll leave this open for now.

I don't think there's any reason to keep the issue open: it's a duplicate of #499 and is also mentioned in the README, under Asynchronous server interactions over Studio.
Getting vim-plug to work with vim or gvim is very simple, it's described in the project README. But if you'd prefer to wait for neovim 0.5 to be released, that's up to you. The only other workaround to get OmniSharp-vim working on Windows with neovim is to use the HTTP version of the server instead of the stdio version. However, I wouldn't recommend this, using the HTTP server means you need python, all server requests are synchronous, and many features are not available, such as popups and full semantic highlighting.

I managed to get it working with Vim. You can easily tell why I want to wait for Neovim 0.5 instead:

My guess is that the omnicompletion pop-up was added in by Neovim, so I'll just wait then.

You can easily tell why I want to wait for Neovim 0.5 instead

No? What's the problem? I can't see much in that small screenshot. Is the completion menu not being displayed? That's peculiar. What version of vim is this, and is it terminal or gvim?

My guess is that the omnicompletion pop-up was added in by Neovim.

No, omnicompletion and the completion menu has been part of vim for many years, long before neovim was forked.

No, omnicompletion and the completion menu has been part of vim for many years, long before neovim was forked

Sorry, I made a mistake. I set my completeopt setting to noinsert, which, apparently, is required by muComplete. I guess NeoVim automatically adds in the menu option. I guess I'll use Vim for now until NeoVim 0.5 comes out or I find a nightly build of 0.5 that Norton doesn't mind.

The default completeopt for both vim and neovim is completeopt=menu,preview


feature: create roi selection

Beginnings of ROI selection features

Closes #87.

@DaniBodor @ajonkman This feature is now ready for review. It has become vastly more complicated than I first intended due to two factors:

there should be a way to split a row or column between two regions, if otherwise regions would consist of a different number of rows/columns;
it was pointed out that splitting rows/columns should be set individually, so you can e.g. split the left/right without splitting a column, but then split ventral/dorsal with splitting rows;
in some cases, you want to ignore rows/columns that entirely consists of NaN (not a number) values, while in other you don't want to do that;
ignoring NaN values should also be set for rows and columns individually.

These features have now been built in.
I'm not entirely sure this API will work best, yet. I think it might be better to save the matrices to the object itself when calling find_grid(), and have a function to do the actual splitting of data into regions as well. But that concern can be addressed later. Maybe this API will just work.

TODO

[ ] Write tests where ignore_nan_columns and/or ignore_nan_rows is/are False.


code widget contains many blank lines

The size of the widget used to display compilable code does not always fit the code it contains, sometimes it has a huge number of blank lines, see e.g. the third widget on the page:
https://learn.adacore.com/labs/intro-to-ada/chapters/imperative_language.html

This is a side effect of PR #333 (https://github.com/AdaCore/learn/pull/333).
The main.adb tab contains more code than the other tabs, which sets the minimum tab length to be the size of the main.adb tab code length. This is intended behavior.

ok, looks weird but we can live with it!

I agree, it does look weird. We will keep this issue open as an enhancement to fix the UI problem created by the UX fix on #333


Can't build with Carthage

I tried building with Carthage, but error occurred.
xcodebuild: error: Scheme PiedPiper is not currently configured for the build action.

How can I build?

Hi @rizumita, did you try to build Carlos after checking out the repository, or did you try to build Carlos as a dependency part of your Cartfile?

Hi @vittoriom
I'm using Carlos in my project.
I wrote following Cartfile.
github "ReactiveCocoa/ReactiveCocoa"
github "WeltN24/Carlos"

The Cartfile looks fine, can you please try to remove the Carlos folder from Carthage/Checkouts and run carthage update Carlos --platform iOS?
With the latest version of Carlos (0.8), I extracted PiedPiper to its own repository and maybe it's better to run a clean update.
Please let me know if this solves the issue

@vittoriom
I became to be able to build after deleting Carthage folder and DerivedData folder.
Thank you for your advise!

No problem, sorry for the inconvenience and I'm glad everything is sorted out!


Added a line of text

Line 15.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Looks good @Hawkings296, congratulations on your first contribution to freeCodeCamp!


Captured images are dark and green

The preview seems in the live feed on a Studio project seems to show the correct lighting and white balance for an image. However, when I capture the image, it appears dark and green. Not sure where the brightness and white balance values are getting changed.

Issues reproduced and fixed. The fix will be published with next ESP32 firmware update.

Should be fixed after merging latest PR to our monorepo master. Fix will be in production within next few days. Feel free to re-open if issue persists.

I encounter the exact same problem.

I followed and installed the latest firmware linked in the official Edge Impulse guide.

The development board does not come with the right firmware yet. To update the firmware:

Download the latest Edge Impulse firmware, and unzip the file.
Open the flash script for your operating system (flash_windows.bat, flash_mac.command or flash_linux.sh) to flash the firmware.
Wait until flashing is complete.

The link in the docs was not updated. I updated it now, check again please.


Translate a Docker Compose File to Kubernetes Resources - Command Typo / Change

From this page: https://kubernetes.io/docs/tasks/configure-pod-container/translate-compose-kubernetes/
Small change requested: I notice that after I run
kompose convert

I don't get a frontend-service.yaml but rather a frontend-tcp-service.yaml.
So the line:
kubectl apply -f frontend-service.yaml,redis-master-service.yaml,redis-slave-service.yaml,frontend-deployment.yaml,redis-master-deployment.yaml,redis-slave-deployment.yaml

Should be changed to:
kubectl apply -f frontend-tcp-service.yaml,redis-master-service.yaml,redis-slave-service.yaml,frontend-deployment.yaml,redis-master-deployment.yaml,redis-slave-deployment.yaml

I'm on macOS, not sure if that has an impact.

Actually this then follows to further commands. It should instead of:
minikube service frontend

Should be:
minikube service frontend-tcp

Really, the culprit or source of the bug would be to find where the -tcp is coming from. It's not present anywhere in the original docker-compose.yml file, but kompose convert puts it in there for some reason...

Hah, even after running the proper service command, there is apparently a recent regression with port mapping as described here: https://github.com/kubernetes/minikube/issues/13736#issuecomment-1063470210
I found this comment useful as a fix: https://github.com/kubernetes/minikube/issues/14097#issuecomment-1117622322
In summary, you just forward the port manually by issuing:
kubectl port-forward service/frontend-tcp 8080:80

/assign

Is this a pr to work on ? @princefishthrower


[FINOS_PROJECT_BLUEPRINT] - Fixes to apply

NOTE! This issue was automatically generated by a FINOS GitHub Action.
We have found some repository configurations that should be changed in order to comply with FINOS Governance and bylaws, see the details below.
List of fixes:

[ ] no-badge - README.md file is missing the FINOS badge; check the README.md template and make sure that it embeds one of SVG FINOS badges.
[ ] has-user - One or more user collaborators were found in this GitHub repository. FINOS Governance only allows GitHub users to be added via Teams. Please remove it, therefore it must be removed.
[ ] no-code-conduct - CODE_OF_CONDUCT.md file is missing; check the CODE_OF_CONDUCT.md template.
[ ] no-whitesource - WhiteSource configuration was not found; make sure that dependencies are scanned against security vulnerabilities. Read more on the WhiteSource Wiki page.

This is a message generated by the FINOS project compliance scanning action.
For any question, do not hesitate to contact @maoo and @mcleo-d or email<EMAIL_ADDRESS>Thank you!

dup of #961


How to stretch Panel 100% height

I am trying to figure out what is the best/recommended way of stretching a panel to 100% height of a page. I have a panel that is stretched vertically to 100% percent but this only works until the window is resized because the height of the content is only updated once. If I set the panel's height to 100%, the Panel will properly calculate the height of the content and explicitly set it. Unfortunately once you resize the window, the height of the content is not updated.
What is your recommendation for stretching a panel to 100%? Several ways come to mind like hooking up to the resize event and manually updating this height of the panel which will force update of the height of the content. Another possible way is to just set the height to auto any rely of pure CSS. What do you think is the best way?
One problem that I see for solving the problem with CSS is that the renderer always adds explicit height and width styles to the html which won't allow me to override those properties easily.
oRm.addStyle("width", oControl.getWidth()); oRm.addStyle("height", oControl.getHeight());

Since there is no response from the author for more than 4 weeks I'm closing the ticket.


NPC sheets can activate their items despite their actor being in a compendium

This is not the case for character-type actors.

Bit more nuanced than that: character actors' Feature items can also be rolled.


Fixed CsvToDataTable issue to remove the outer quotes from the first field of each row

When importing a csv file through the CsvToDataTable option it would leave the double outer quotes around the first field of each row.  A fix was added to remove the double outer quotes when it is breaking apart each string of the line.  Any inner double quotes are left alone.

HI there, we can use this approach to solve the problem with quotes because we are changing the core of the library (FieldBase.cs)
Can you create an Issue with the problem se we can review it ?
Thanks

Hi,
Firstly: thanks for FileHelpers, it's a very helpful library!
Secondly: I'm currently experiencing the issue mentionned in this request:
I'm using the code below to load a csv file in a datatable:
CsvOptions options = new CsvOptions("Dummy", ',', numberOfColumns, 0);             return FileHelpers.CsvEngine.CsvToDataTable(file, options);
It works fine except for file where fields are double quoted:
"field1","field2","field3"
Once in datatable, the quotes are gone for field2 and field3 but not for field1.
Is there a fix for this issue?
I'm using FileHelpers <IP_ADDRESS>
Best regards.


Manually set mime types

Hi,
I realize I am probably not using arangodb as intended but it makes sense for my use case.
I have a foxx application that serves an angularjs app which intern calls back to the foxx app itself. This application also needs to serve a lot of different types of files.
I have to do this because my entire application must be able to run on a standalone system, and I don't want to install a second web server just to serve the files.
I don't expect you to have a complete mime.types list for your web server (though I am surprised you don't just use the /etc/mime.types file). I just need to find a way to map some extensions to content types.
A couple of the extensions I tried and returned as text are *.JPG (but not *.jpg) and *.mov
Any help you could offer would be great.
Thanks

You probably already saw our cookbook for handling binary data? https://docs.arangodb.com/cookbook/FoxxBinaryFileUploads.html - It demonstrates howto return binary data and set arbitrary headers (content-type)
Next to looking up the data in a pre-seeded mime database another good trick is to trust the content type header when uploading the file from a browser.
I will extend this example with an arangosh script to feed /etc/mime.types into a collection for lookup from FOXX in the next days - stay tuned.

You can use the mime-types module from npm (just npm-install it in your Foxx service folder as if it were a node app): https://www.npmjs.com/package/mime-types
Alternatively if you want to send files from Foxx you can also use res.sendFile(filename), which will guess the content type the same way as files specified in the manifest.

I've added https://github.com/arangodb/arangodb/blob/devel/scripts/read_mimetypes.js to read /etc/mime.types into a collection; However one problem persists: extension->mimetype is not uniq.

@dothebart Do we use that for anything at the moment? Also, do we use arangodb.guessContentType for anything other than Foxx at the moment? Because we're going to use mime-types from npm in most of FOTF to maintain consistency throughout npm modules, so if we only use it for Foxx we could phase it out entirely in the future (mime-types brings a full mime type list along).

@pluma Its brand new. arangodb.guessContentType could probably also be used; However, if you want new mimetypes and don't want to alter the code base the mimetype collection could be the better way.

I am not using a foxx controller to deliver the files. I am just browsing to them directly. I have
"files": {     "/html": {       "path": "html/"     }
in my manifest.json and all the files I use are manually loaded into html/uploads (not entirely correct I know but I am dealing with a proof of concept and the client doesn't care how the files are entered).
I was more wondering if there was a way to configure the web server arangodb is using to detect additional mime types.

@geekygnr ArangoDB currently uses a hardcoded mime type list to guess the mime type based on extension when serving entire folders or not specifying the mime type:
https://github.com/arangodb/arangodb/blob/2.8/js/common/modules/org/arangodb/mimetypes.js#L14
FOTF (the upcoming major version of Foxx) will use mime-db (via mime-types), which pretty much ships the entire kitchen sink of mime type mappings (via Apache and nginx): https://github.com/jshttp/mime-db/tree/master/src This also includes the mime type for "mov".
Currently there is no way to make arangodb recognize additional mime types. I'm not sure how valuable tapping into this for the files manifest field is as this usage seems like an edge case. You probably don't want to serve file uploads from ArangoDB directly, at least not in production and automatic detection with a more complete mime type list should be good enough for your specific use case.
I noticed there is no way to specify the mime type of an individual file in the manifest, so it would probably be useful to be able to specify the type explicitly (via a type field which I thought already existed). I've added a task for that to the FOTF PR.

Thank you @pluma my situation is definitely an edge case. I will have to look into creating a controller for those files to deliver the mime types.

Fixed via FOTF.


Fix android-4.19

Looks like Sami pushed https://android-review.googlesource.com/c/kernel/common/+/1606628 so we can probably hold off on this for now.

android-4.19 should pass now that https://android-review.googlesource.com/c/kernel/common/+/1606628 is merged.


Add GitHub Action CI

Builds should be checked by GitHub Actions CI to ensure that the development branch is always in a working state.

Fixed in #29


Running code after site deployment

I have a deployment script that creates a database with a couple of tables and I was hoping I could run some php code after the deployment to populate those tables. Is there a way to sync to forge's deployment?

You could utilize Forge's default deployment script for this. Put the file on your server and call it in the script (php /path/to/your/file.php, or any other way 😉).

The only issue with that is that the file will run everytime a deployment is made. I don't want to be adding users everytime I make a code change and deploy it.

We're planning a post-deployment webhook solution, but no ETA yet :)

Thanks. Closing ticket


optimization proposal: append to body in batches

This is a proposal to optimize DOM access, make the bootstrap runtime webpack code, faster.
When we  append invidual chunks to document head, in bootsrap part, here, instead doing it one at a time, we should do it in batch, as DOM access is a slow process and should be limited. I am specifically looking at optimising the prefetch and the preload DOM appends.
So, instead of document.head.appendChild(script);, we can use
fragment = document.createDocumentFragment();
fragment.appendChild(script); //do for all chunks/scripts
document.head.appendChild(fragment); //and then finally add that fragment to DOM

I'm pretty sure this is not a bottleneck, in worst case we are adding like 10 script tags.


nixos/keymapp: init

Description of changes
Sets up the necessary udev rules to flash keyboards from Keymapp. udev rules pulled from here.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 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.

Enabled the module and tested major functionality of keymapp: connecting to the keyboard, the heatmap, and flashing.

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

I should have looked closer... closing since you can just use hardware.keyboard.zsa.enable.


Describe measurement documents in the spec

Going to leave this open & not merge for a while. This is an early idea we're working with & may have internal/beta implementation support for soon, but aren't going to consider a stable part of the spec yet.

Closing. We are indeed using this schema privately but have not seen any demand for this to be more widely available & have no plans to support it more widely any time soon.

Duplicate of #


Typo in the README.md file

androidMainImplementation("pro.respawn.flowmvi:compose:$flowmvi")

Should be:
androidMainImplementation("pro.respawn.flowmvi:android-compose:$flowmvi")

Oops, I will fix that. Thanks for pointing that out.


Update hab to 0.50.3-20171201233208

After making all changes to the cask:

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

Additionally, if updating a cask:

[ ] sha256 changed but version stayed the same (what is this?).
I’m providing public confirmation below.

@RoboticCheese Your cask-repair is outdated, you should brew upgrade cask-repair


[Style] Make AQI Calculator responsive

What's the style idea?
In mobile view the contents are displayed outside the box. Also there is an extra white space in the bottom of the screen. I would like to improve the styling of both of them.

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Go ahead @m3kvt


Retain client_address on Postgres connection when using Private IP

Question
We're currently rewriting a number of our Golang services to use cloudsqlconn and overall things are working great.
However one thing that we've spotted is that there's no client_address reported on the connections within Postgres, even if we're using the instance private IP to connect.
However if we connect using psql from the same pod using the same details then the client_address is recorded.
So is there a specific reason why the client_address isn't being reported when connecting via cloudsqlconn using the private IP?
Code
No response
Additional Details
No response

That's an artifact of how the Proxy path works -- it goes from the Go Connector to the Proxy server to the database server. The connection between the Proxy server and the database server is over a local unix socket. So when the database sees the connection, it's from localhost. The team is aware of the limitation, but probably won't be fixing it anytime soon.
Out of curiosity are you using Auto IAM AuthN with the Go Connector?

That's an artifact of how the Proxy path works -- it goes from the Go Connector to the Proxy server to the database server. The connection between the Proxy server and the database server is over a local unix socket. So when the database sees the connection, it's from localhost. The team is aware of the limitation, but probably won't be fixing it anytime soon.
Out of curiosity are you using Auto IAM AuthN with the Go Connector?

Thanks for the quick reply @enocom .
Yeh, so we're using Auto IAM AuthN with GKE service accounts and Workload Identity.
All working great apart from the lack of client_addr.
Am I right in assuming that if we enable the private-ip setting that Go Connector is talking to the proxy server on the private ip? As you mentioned that the proxy is collocated with the database and uses a local socket.

FWIW, it's possible to connect directly and still use Auto IAM AuthN (with a little know-how). In that scenario, you'd get the correct IP in the client logs.
Also, a caveat: if you've enabled "Allow only SSL connections" on your instance, you'll have to configure client certificates on the direct path. Doable, but a bit of a nuisance.

FWIW, it's possible to connect directly and still use Auto IAM AuthN (with a little know-how). In that scenario, you'd get the correct IP in the client logs.

Ooh, interesting... Are you able to provide an example?
I could see a working scenario using a manually generated sql access token, but that would require coding a refresh wouldn't it?

If you're using database/sql, you could write a driver like this:
import (
	"context"
	"database/sql"
	"database/sql/driver"

	"github.com/jackc/pgx/v4"
	"github.com/jackc/pgx/v4/stdlib"
	"golang.org/x/oauth2"
	"golang.org/x/oauth2/google"
)

func init() {
	sql.Register("gcp-postgres", &pgDriver{})
}

type pgDriver struct{}

func getAuthToken() (*oauth2.Token, error) {
	ts, err := google.DefaultTokenSource(context.Background())
	if err != nil {
		return nil, err
	}
	return ts.Token()
}

func (p *pgDriver) Open(name string) (driver.Conn, error) {
	config, err := pgx.ParseConfig(name)
	if err != nil {
		return nil, err
	}
	tok, err := getAuthToken()
	if err != nil {
		return nil, err
	}
	config.Password = tok.AccessToken

	dbURI := stdlib.RegisterConnConfig(config)
	if err != nil {
		return nil, err
	}
	return stdlib.GetDefaultDriver().Open(dbURI)
}

And then usage looks like:
package main

import (
	"database/sql"
	"fmt"
	"time"

	_ "enocom.dev/driver/postgres"
)

func main() {
	db, err := sql.Open("gcp-postgres", "host=localhost dbname=postgres<EMAIL_ADDRESS>sslmode=require")
	if err != nil {
		panic(err)
	}
	row := db.QueryRow("SELECT NOW()")
	var t time.Time
	if err := row.Scan(&t); err != nil {
		panic(err)
	}
	fmt.Println(t)
}

If you were using pgx, it would look like this:
import (
        "context"
        "fmt"
        "time"

        "github.com/jackc/pgx/v5"
        "github.com/jackc/pgx/v5/pgxpool"
        "golang.org/x/oauth2"
        "golang.org/x/oauth2/google"
)

func getAuthToken() (*oauth2.Token, error) {
        ts, err := google.DefaultTokenSource(context.Background())
        if err != nil {
                return nil, err
        }
        return ts.Token()
}

func main() {
        config, err := pgxpool.ParseConfig(
                "host=<INSTANCE_IP> user=postgres password=empty sslmode=require",
        )
        if err != nil {
                /* ... */
        }
        // This function is called before every connection
        config.BeforeConnect = func(ctx context.Context, c *pgx.ConnConfig) error {
                tok, err := getAuthToken()
                if err != nil {
                        /* ... */
                }
                c.Password = tok.AccessToken
                return nil
        }

        pool, err := pgxpool.NewWithConfig(context.Background(), config)
        if err != nil {
                /* ... */
        }

        conn, err := pool.Acquire(context.Background())
        if err != nil {
                /* ... */
        }
        defer conn.Release()

        row := conn.QueryRow(context.Background(), "SELECT NOW()")
        var t time.Time
        if err := row.Scan(&t); err != nil {
                /* ... */
        }

        fmt.Println(t)
}

Looks like Bun (the rewrite of go-pg) makes this easy at leaset.

Marking this as a feature request. The change would be on the backend, but might be feasible.


Make default platform android on non-mac platforms for start command

Directly following on from https://github.com/isleofcode/corber/issues/581 which did so for the build command.

I think in this case the issue was with serve vs start?
Start is generally smart enough to look at all of your platforms and list, whereas we had missed updating serve.
I'm just merging a PR that resolves this issue for serve - please let me know if I missed something obvious. And thanks for opening the separate issues!

Apparently not, this is what I get:
 $ npx corber start
Corber Starting
spawn /usr/bin/instruments ENOENT
events.js:183
      throw er; // Unhandled 'error' event
      ^

Error: spawn /usr/bin/xcrun ENOENT
    at Process.ChildProcess._handle.onexit (internal/child_process.js:190:19)
    at onErrorNT (internal/child_process.js:362:16)
    at _combinedTickCallback (internal/process/next_tick.js:139:11)
    at process._tickCallback (internal/process/next_tick.js:181:9)

Or perhaps this is it "checking" what to do?


Check appimaged

Check appimaged, it has similar functionality. Regarding the sandbox file, I think we should make something like this part of the AppImage specification, but define rights such as

May use the network
May read $HOME
May write $HOME/.local
May write all of $HOME

etc... which we could then ask the user to permit (similar to what Android does).

I've gone off the idea of using a daemon to be honest as won't it only have specific directories that can be scanned? This launcher can launch AppImages using the MIME type association, so AppImages can be located anywhere and it does not need to be run as a daemon constantly.
This is exactly what I had in mind with the sandbox file, but just quickly threw something in there as a proof of concept. Is there an issue open in the AppImageSpec for this kind of functionality.

This launcher can launch AppImages using the MIME type association, so AppImages can be located anywhere and it does not need to be run as a daemon constantly.

True, but it also means that they won't show up in the menus before you have executed them at least once. Which may be an annoyance, especially on Live systems that get wiped on each reboot.

Is there an issue open in the AppImageSpec for this kind of functionality.

No but please open one so that we can brainstorm.

I have opened an issue.
You make good points as to why this solution might not work for certain people, but there are also reasons as to why a daemon might not be suitable in certain scenarios, which is why this exists. appimaged will obviously the best solution, but this serves as a solution for people that might not want to (or can not) run a daemon on their machines.

Could appimaged also use the MIME type association. So for AppImage's that aren't stored in a scanned location, they could still be launched without having to manually make it executable?

Yes. Should be relative straightforward I suppose.


[Major Issue] Crash with newest TechReborn

TechReborn had another update recently that your mod, is not liking it on launch.  It was working before I updated techreborn, and did work then.
crash-2021-09-21_14.56.58-client.txt

sorry, I thought I downloaded the correct one, but seems as though I might have download by mistake another one... not sure which one though...
yep was the wrong one.... Now I feel dumb.


pull request for e2e test at 12/03/2016 16:56:03

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Switch Edge back to y for variable font support

I'm not entirely sure why this was listed as having issue #4 as we have support for truetype-variations, woff-variations, woff-variations and opentype-variations. Additionally, we support the font-weight and font-stretch properties and their aliasing to their respective axes within the font.

That said, Chrome does have the issues in #4, as well as not supporting local() variable font references. At any rate my thinking is this must have been a mistake or I'm missing something. Thanks for taking the time to review @Fyrd

@gregwhitworth Thanks for confirming! That note was based on this article which suggests Edge 17 has the same shortcomings as the other browsers.


RC1

This bumps the version to the first Release Candidate. If no new bugs get filed, I'll send this version live as v2 in April 2017.

Coverage remained the same at 89.683% when pulling d067d33dbd879dedbcb027465c58aee5d76a0f56 on scott-w:master into b65301267921bd019625a3ecf37637dd3f6970af on jeromegn:master.


Update default SIMPLESAMLPHP_VERSION to 1.14.11

Sorry for the long response time. I updated to SimpleSAMLphp 1.14.15. I hope the intermediate versions are not needed.


easier way to split the datasets

Hello,
I rewrite the code to subset the full dataset and split the dataset. I think the logic in my code is easier to understand.
I hope it is helpful.

can you help me in understanding this code?


Show pathway 'name' on view pane

The view pane should reproduce the search result 'name' or 'title' that was selected in order to provide an explicit link between the two.

This is slightly complicated problem for a few reasons. When, I go to a pathway URL, lets say this one: http://localhost:8000/#/pathway?uri=http%3A%2F%2Fpathwaycommons.org%2Fpc2%2FPathway_1d99625aa7f8f4a3f3189e8b237522c3, I make the call to get the SBGNML, which does not contain the name, therefore I do not automatically have the information I need to display the name.
There are two ways I can think of to get the name. One is to link the pathway to the search result, which I am reluctant to do because there are a load of drawbacks to doing this, including no longer being able to open a pathway by URL has I have done above. The second is to attach the name to the URL of the pathway as well as the URI, which can get messy in the long run.
I think I will need to think on this one further, and maybe talk to Igor.

This is a client-side concern as the browser has the SearchHit. Is there no way for ReactJS to pass this info to another component? I'm thinking this is a must-have.

No there definitely is, I just feel its a bit iffy code-hygeine wise and not something I particularly want to do. However if its the only option (nothing can be done server side) then it can easily be implemented.

On Jan 17, 2017, at 11:14 PM, Jeffrey<EMAIL_ADDRESS>wrote:
This is a client-side concern as the browser has the SearchHit. Is there no way for ReactJS to pass this info to another component? I'm thinking this is a must-have.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/mj3cheun/pathways-search/issues/8#issuecomment-273378417, or mute the thread https://github.com/notifications/unsubscribe-auth/ANaTemqN-I9rJPemCaMqqQOGGXkjaXYDks5rTZGMgaJpZM4Ll1GW.

What about passing in a 'state' key/value to the Link?

Fixed in commit bd4ef4122dcee01b19a21aa45139f5dd1aa1d9f7. Added name to Pathway page.
Passing the 'state' key/value to the link is essentially the same as linking the pathway to the search result, and will result in the drawbacks described above, mainly not being able to pass the URL of the pathway as a standalone link. I have chosen to instead place all required information in the URL itself. Please see commit for implementation details. Originally, I didn't want to do this because it is further cluttering the URL but I still feel this is the best possible solution.


CSS Polyfill conflicts with browsers that have support enabled

Trying out https://codepen.io/bramus/pen/popYBRz/19bf451ebc93a14378b4d439124a0d61 in stock Chrome works like a charm.
Trying the same demo out in a browser with support (i.e. Chrome with the --enable-experimental-web-platform-features flag or Firefox with layout.css.scroll-linked-animations.enabled set to true) doesn't do anything.
I would have expected the polyfill to only do its thing when @scroll-timeline is not available.
The easy fix would be to wrap the call to initCSSPolyfill() in a if (!CSS.supports("animation-timeline: works")) { … } block.

Did a quick workaround in #51, but perhaps this isn’t the way to go.
Digging deeper into the demo when viewed in a non-iframed/standalone window (i.e. without the CodePen chrome), I find this error in the console:
proxy-animation.js:571 Uncaught DOMException: Failed to set the 'currentTime' property on 'Animation': Invalid currentTime. Setting currentTime using absolute time values is not supported for progress based animations.
    at e.F (https://flackr.github.io/scroll-timeline/dist/scroll-timeline.js:1:12240)
    at f (https://flackr.github.io/scroll-timeline/dist/scroll-timeline.js:1:4171)
    at y (https://flackr.github.io/scroll-timeline/dist/scroll-timeline.js:1:5509)
    at W (https://flackr.github.io/scroll-timeline/dist/scroll-timeline.js:1:11806)
    at n.play (https://flackr.github.io/scroll-timeline/dist/scroll-timeline.js:1:14354)
    at https://flackr.github.io/scroll-timeline/dist/scroll-timeline.js:1:31487
    at Array.forEach (<anonymous>)
    at https://flackr.github.io/scroll-timeline/dist/scroll-timeline.js:1:30354

Note also issue #4. I think we should skip both the CSS and JS polyfills when native support exists.


Where is "messages"?

Issue in ./help/channels/import-and-author-html-email-content.md
This video shows going to "messages" in the left nav, but I don't see this in the AJO interface. Did this move?

Yes, it was removed. The video still needs to be updated.


Add a way to include Squad Leader on the character sheet

Perhaps add a checkbox option in the upgrades area?

Coding wise, I don't think this is that complicated.
It's mostly a UX problem of where to put the checkbox that doesn't clutter the interface. i will have a think

The obvious answer which only just occured to me was to treat this as any other "free" starting upgrade, which lets it be equipped at any point for no cost.
This is done in #181


ADD Onboarding and Uninstalled page  Chrome extension Keyboard shortcuts

Is your feature request related to a problem? Please describe.
ADD Onboarding and Uninstalled page  Chrome extension Keyboard shortcuts
Describe the solution you'd like
in this i made a keyboard shortcut for opening extension popup.
Describe alternatives you've considered

Opening Onboarding page when extension installed.
Adding Uninstall website URL.
Messaging between Content scripts, Background page & Popup.
Keyboard Shortcuts

Additional context
Add any other context or screenshots about the feature request here.
What problem is this feature trying to solve?
How do we know when the feature is complete?

Kindly update otherwise I will close this @kartikmehta18


Using connection.escape adds single quotes around my value?

If I do the following:
console.log(value);

I get the expected output of:  tmp/myValue
But the following code:
console.log(connection.escape(value));

It outputs 'tmp/myValue'. Notice the single quotes that are now around the value.
Is there a reason for this or is it a bug?

This is great, although it does cause problems when trying to do things like:
console.log(`SELECT * FROM book where name LIKE '%${connection.escape(searchName)}%'`)
// SELECT * FROM book where name LIKE '%'SEARCH_NAME'%'

// You get those extra quotes inside the LIKE ...

@tarwin See https://github.com/mysqljs/mysql/issues/1386 about escaping Values for LIKE.
The example should be in the Docs. There will be someone taking care of that soon(tm), I guess.

@kai-koch I realized I could simply do:
let searchName = 'SEARCH_NAME'
let searchLike = `%${searchName}%`
console.log(`SELECT * FROM book where name LIKE ${connection.escape(searchLike)}`)
// SELECT * FROM book where name LIKE '%SEARCH_NAME%'


dataset error

Model Builder Version: In which Model Builder version was the bug encountered?
Visual Studion Version: In which Visual Studio version was the bug encountered?
Bug description
A clear and concise description of what the bug is.
Steps to Reproduce

List the minimal steps required to reproduce the bug.
Thanks for reporting!

Expected Experience
A description of what you expected to happen. If applicable, add screenshots or "Machine Learning" Output Logs to help explain what you expected.
Actual Experience
A description of what actually happens. If applicable, add screenshots or "Machine Learning" Output Logs to help explain what actually happened.
Additional Context
Add any other context about the problem here, such as unusual system configurations, network restrictions, etc.

@SandhyaKourla Sorry you've run into problems using Model Builder. Could you provide more details on the error you experienced? If possible, please attach the dataset that has problems. Thanks!

@SandhyaKourla Friendly ping, any more details you can provide on this error?


Cart does not move with player in a seat.

Not a lot I can do about this, I could add a magnetic upgrade or something. That will come in 1.11


Start implementing auto reconnect on SignalR TypeScript client

Partially addresses #5282
Fixes #8709 (added by @anurse :trollface:)

@JameNK @mikaelm12 @BrennanConroy @anurse This is ready for review.
I only plan to add two more tests similar to AutomaticReconnect.test.ts's "does not reset after hub handshake failure" test that tests the integration between HubConnection and HttpConnection.

A test to verify the onreconnected connectionId parameter appropriately provides the new connection id.
A test to verify that the client can reconnect again after it successfully and fully reconnects once.

I'm also considering adding a test to verify the reconnect delay timeout gets cleared when the connection stops, but making the delay long enough to be reliably but quick enough not to slow down our tests might be tricky.

@BrennanConroy @mikaelm12 ping


A demo PR to test fix in #7077

The relevant fix that this PR was demoing has been landed. Thus, I'm closing this PR.


python311Packages.lightning-utilities: 0.11.2 -> 0.11.3

Description of changes
Changelog: https://github.com/Lightning-AI/utilities/releases/tag/v0.11.3

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] 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.11 Release Notes (or backporting 23.11 and 24.05 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 322628 run on x86_64-linux 1

14 packages marked as broken and skipped:

piper-train
piper-train.dist
python311Packages.elegy
python311Packages.elegy.dist
python311Packages.treex
python311Packages.treex.dist
python312Packages.lightning-utilities
python312Packages.lightning-utilities.dist
python312Packages.pytorch-lightning
python312Packages.pytorch-lightning.dist
python312Packages.rising
python312Packages.rising.dist
python312Packages.torchmetrics
python312Packages.torchmetrics.dist

12 packages built:

python311Packages.lightning-utilities
python311Packages.lightning-utilities.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.rising
python311Packages.rising.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist
whisper-ctranslate2
whisper-ctranslate2.dist

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

8 packages marked as broken and skipped:

python312Packages.lightning-utilities
python312Packages.lightning-utilities.dist
python312Packages.pytorch-lightning
python312Packages.pytorch-lightning.dist
python312Packages.rising
python312Packages.rising.dist
python312Packages.torchmetrics
python312Packages.torchmetrics.dist

12 packages built:

python311Packages.lightning-utilities
python311Packages.lightning-utilities.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.rising
python311Packages.rising.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist
whisper-ctranslate2
whisper-ctranslate2.dist

Result of nixpkgs-review pr 322628 run on x86_64-darwin 1

8 packages marked as broken and skipped:

python312Packages.lightning-utilities
python312Packages.lightning-utilities.dist
python312Packages.pytorch-lightning
python312Packages.pytorch-lightning.dist
python312Packages.rising
python312Packages.rising.dist
python312Packages.torchmetrics
python312Packages.torchmetrics.dist

8 packages built:

python311Packages.lightning-utilities
python311Packages.lightning-utilities.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.rising
python311Packages.rising.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist

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

10 packages marked as broken and skipped:

piper-train
piper-train.dist
python312Packages.lightning-utilities
python312Packages.lightning-utilities.dist
python312Packages.pytorch-lightning
python312Packages.pytorch-lightning.dist
python312Packages.rising
python312Packages.rising.dist
python312Packages.torchmetrics
python312Packages.torchmetrics.dist

12 packages built:

python311Packages.lightning-utilities
python311Packages.lightning-utilities.dist
python311Packages.pyannote-audio
python311Packages.pyannote-audio.dist
python311Packages.pytorch-lightning
python311Packages.pytorch-lightning.dist
python311Packages.rising
python311Packages.rising.dist
python311Packages.torchmetrics
python311Packages.torchmetrics.dist
whisper-ctranslate2
whisper-ctranslate2.dist