Added type for task into 4 cat.

Added four categories for tasks ->Personal, Shopping, Wishlist, Work
Made a simple dropdown list for that.
Attaching the SS for reference. Kindly review it
Give feedback if necessary :)

resolve conflicts

resolve conflicts

Done

@Kritika30032002 Kindly accept the pr now :)

Resolve conflicts.

push to demo branch

create pr to demo branch

I have created a new pr please check #62 :)

create pr to demo branch

I have created a new pr please check #62 :)

its in main branch
it should be to demo branch


[SPARK-5598][MLLIB] model save/load for ALS

following #4233. @jkbradley

Test build #26907 has started for   PR 4422 at commit f487cb2.

This patch merges cleanly.

Test build #26907 has finished for   PR 4422 at commit f487cb2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26907/
Test PASSed.

I just had a few small comments.  One question which occurred to me: Does SQLContext have configuration info which would be relevant to saving/loading?  I'm wondering if users should be able to pass in a SQLContext to save/load.

This is the spark.mllib package. We don't expect users to use Spark SQL here.

Test build #26959 has started for   PR 4422 at commit 2622870.

This patch merges cleanly.

Test build #26964 has started for   PR 4422 at commit 14b7ea6.

This patch merges cleanly.

Having SaveLoadV1_0 be a Loader is weird since it loads metadata, and the metadata gives the version number.  It works now since there is only 1 version, but it will have to stop being a Loader in the future.

Other than the Loader issue, looks good.

Test build #26964 has finished for   PR 4422 at commit 14b7ea6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26964/
Test FAILed.

Test build #26959 has finished for   PR 4422 at commit 2622870.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
     "public class " + className + " implements java.io.Serializable
  case class RegisterExecutor(executorId: String, hostPort: String, cores: Int)
          throw new SparkException("Failed to load class to register with Kryo", e)
class LogisticRegression extends Estimator[LogisticRegressionModel] with LogisticRegressionParams
class DoubleParam(parent: Params, name: String, doc: String, defaultValue: Option[Double] = None)
class IntParam(parent: Params, name: String, doc: String, defaultValue: Option[Int] = None)
class FloatParam(parent: Params, name: String, doc: String, defaultValue: Option[Float] = None)
class LongParam(parent: Params, name: String, doc: String, defaultValue: Option[Long] = None)
class BooleanParam(parent: Params, name: String, doc: String, defaultValue: Option[Boolean] = None)
  class EMOptimizer(
case class GetField(child: Expression, fieldName: String) extends UnaryExpression
class DefaultSource extends RelationProvider
case class ParquetRelation2(path: String)(@transient val sqlContext: SQLContext)
protected class CaseInsensitiveMap(map: Map[String, String]) extends Map[String, String]
trait CreateableRelationProvider

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26959/
Test PASSed.

Test build #27007 has started for   PR 4422 at commit ba1ab4e.

This patch merges cleanly.

Test build #27007 has finished for   PR 4422 at commit ba1ab4e.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27007/
Test FAILed.

Test build #27011 has started for   PR 4422 at commit a059394.

This patch merges cleanly.

Test build #27011 has finished for   PR 4422 at commit a059394.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27011/
Test PASSed.

Merged into master and branch-1.3.


Question about input sample size

Why is the input size of the trained sample [511,511]
but the input size of the test sample is :
inp_height = new_height | 127
inp_width  = new_width  | 127
Is there any benefit to doing this?

Why is the input size of the trained sample [511,511]
but the input size of the test sample is :
inp_height = new_height | 127
inp_width = new_width | 127
Is there any benefit to doing this?

I'm also confused. Have you figured it out?


Update dependencies

This update runs swift build and swift test successfully.

Coverage decreased (-22.2%) to 77.778% when pulling d6e6417dbdae0b9c4d76f115e4487611c32928b4 on jkingoliver:master into efb457b25bc52309ff5a628fcdc64f2260439d1d on ibm-bluemix-mobile-services:master.

Travis would not pass without adding swift package update to the travis.yml. I'm not sure why that's required?
I'm also not sure why code coverage decreased so much with this PR. Let me know if you have any suggestions for updates.

I'm guessing that swift package update was required because the dependency versions from the existing pins didn't match up. Maybe from the SwiftyJSON version change. Either way, it's fine to leave that in.
As for the test coverage, it looks like Coveralls just now began to report the correct test coverages. Some defect in Coveralls must have been resolved, because clearly those unit tests do not cover 100% of the code.
@jkingoliver Can you open a different pull request to merge into the development branch? All outside PRs go to the development branch, which then gets merged into the master branch once we're ready to release a new version of this library.

Done. See PR #22


KDE

Greetings,
will Fly-Pie ever work with KDE or should I just forget about it?

Fly-Pie will never work on KDE. However, I am currently working on a platform independent successor project! You can follow and support the development on Ko-fi: https://ko-fi.com/schneegans
Here was the initial announcement: https://ko-fi.com/post/Introducing-Ken-Do-L3L7L0FQ2

Thank you for the quick reply


Added DNTProfiler.

Added DNTProfiler. It's an EF 6.x and NH 4.x profiler.

Hi, thank you for your contribution. What about project documentation?
@vladen Im not sure about this one, what do you think? Looks very interesting, but too young as well.

Young but useful. @VahidN, it would be worth to make more docs (maybe video guide as well to cut the learning curve). Thing seems to be very helpful for those, who work with EF, NH.


Completed task #1414

Pull Request Type
What kind of change does this PR introduce?
Solved an issue/task
What is the current behavior?
No code to add 2 numbers
Issue Number: [#1414]
What is the new behavior?
Implemented code to add numbers in C
Code Output:

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


add enabled property to item

Hi @nebukadhezer !
This looks like a good fix. Could you please fix the DCO issue by following the guide in the "Details" of failed test above?
It would also be great with a unit test for this addition. A super simple xml file with two clips which each have the enabled true/false or something would be awesome.

Hey @apetrynet I will add a simple test and also look for the other types that can carry the enabled property and see if we can propagate them too. But it might take some days not sure when I can fiddle it in.

Thanks! No rush :)

@apetrynet  I added a simple unittest, let me know if this should be more, also I used "each_clip", "find_clips" does not exist in my otio version 0.15.0

ahh and I used the track name info if there is one to name the tracks according to the info in the fcp xml.

In regards to the linter it asked for one line in the unittest I did not touch, I added it, but I wonder if that is all, as it exists right after the first error. Is this intentional so CI time is saved ?
  flake8 --show-source --statistics --ignore=E501,W503,W504
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    pythonLocation: /opt/hostedtoolcache/Python/3.7.17/x64
    LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.7.17/x64/lib
./tests/test_fcp7_xml_adapter.py:38:1: E30[2](https://github.com/OpenTimelineIO/otio-fcp-adapter/actions/runs/6795623326/job/18524280881#step:6:2) expected 2 blank lines, found 1
class TestFcp7XmlUtilities(unittest.TestCase, test_utils.OTIOAssertions):
^
1     E[3](https://github.com/OpenTimelineIO/otio-fcp-adapter/actions/runs/6795623326/job/18524280881#step:6:3)02 expected 2 blank lines, found 1
Error: Process completed with exit code 1.```


Strip out persistence

Changes Unknown when pulling 237a35ab6f551e304f78060d322d2ae9fccd6aec on strip-out-persistence into ** on master**.

Changes Unknown when pulling ccfa49bfea7c21d4e9fa447fd6484d9cb73b641d on strip-out-persistence into ** on master**.


Synchronous communication in EDA

Before clicking the "Create pull request" button:
Enter a meaningful title. We suggest using the keywords "New" or "Update" indicating the nature of changes.
Summary, scope, and intent of this PR
Sometimes, it's needed to have synchronous communications within EDA. I have added two ways  of implementing it.  I've made this PR following my chat with @ckittel
Links to any related work item(s) or supporting detail

[DELETE-OR-REPLACE-THIS-TEXT]
[DELETE-OR-REPLACE-THIS-TEXT]

AFTER YOUR PR HAS BEEN CREATED, expand this section for tips and additional instructions.
These are common guidelines for contributions across the repos managed by the Azure patterns & practices (PnP) content team. Some repositories may have additional specific requirements that are not listed here.
Guidance for all contributors

Topic
Guidance

Draft PR
If your PR will be a work-in-progress for more than a day or two, select the Convert to draft link in the upper right of the page (under Reviewers) to change it to a draft. For future reference, you can also do this using the Create pull request button drop-down during PR creation.

ms.date metadata
Don't update an article's "ms.date" metadata property unless you've done a full freshness review of the content. A full freshness review includes changes required to correct or improve the full technical accuracy of the article.Don't update "ms.date" if you're doing targeted changes to improve non-technical aspects of the article, such as editorial quality, art improvements, article template alignment, etc.If you've changed any "ms.date" properties for work that wasn't part of full review for freshness, please reset them to their previous value.

Placement and linking
If you're creating a new article or articles, include updates to the related TOC.yml file to propose where the article(s) should be placed. Also consider other places within the document set where it would be beneficial to cross-reference and link to your new article(s).

PR build
After you open your PR, and for each successive commit that you push to your branch, the publishing platform will run validation on the files in your pull request. A summary of the build results for each file will be inserted inline into your pull request, which includes any build suggestions/warnings/errors. PRs cannot be merged until all build errors and most warnings are resolved.

Publishing
Following a successful merge, most repos publish to the live site about once per (business) day.

Additional resources
learn.microsoft.com contributor guide

Additional guidance for private repos and internal contributors

Topic
Guidance

PR size
If your PR is more than ~5 lines of changes, or you'd like for the changes to go through editorial or larger review, open a contribution request at https://aka.ms/contribution and include a link to the PR in response #8. Once it's processed, you'll be notified of the next steps.

PR title prefix
Select the Edit button to the right of the PR title if you need to revise it. The following prefixes are reserved for specific contribution types:[Quality Check] - maintenance work related to content quality (edit passes, art improvements, template alignment)[LinkFix] - recurring/adhoc PRs to correct link URLs[Pipeline] - new/updated contributor success pipeline content[WIP] - a work-in-progress draft requiring several days/weeks

PR preview
Following successful build of your PR, publishable files will also include Preview URL links to staged previews of your new/updated articles. Be sure to review these for verification of your intended contributions, or to send to other internal contributors for review.

PR sign-off (public repo)
If an article you own is updated in a public repo PR, you are responsible for sign-off. You will be automatically notified via email. The PR will not be merged until you've had a chance to review and sign-off.

PR sign-off (private repo)
After you've completed your proposed changes, addressed build warnings, and completed all review work, you can begin the sign-off process for review and merge:If your PR is in draft mode, remove "[WIP]" from the title and select Ready for review button at the bottom of the PR.Enter "#sign-off" in a new comment. This comment indicates that you're confident the work meets or exceeds Microsoft's standards for publication, and will trigger the review process.Your PR may be selected for initial review by the PnP content team. Following PnP review, you may receive questions or requests for additional changes. You should have initial feedback from PnP review within a few business days. If you have an urgent request or need to contact the team, please mention @MicrosoftDocs/patterns-and-practices-team-pr-reviewers in your PR and someone will get back to you. After the Azure patterns & practices content review is complete, a PnP #sign-off will be added.Final review/merge is done by the PR review team. The PR team may also respond with feedback, categorized as "Blocking" (requires action from you), or "Non-blocking" (to be addressed in a future PR).

Additional resources
learn.microsoft.com internal contributor guideAuthoring templates: architecture-center-pr, well-architected-prTo contact the Azure patterns & practices team use e-mail, or @mention our GitHub team in your PR comments using: @MicrosoftDocs/patterns-and-practices-team-pr-reviewers

@ckittel thanks for raising those fair points. I tried to incorporate them. Let me know if there is anything else needed.

@ckittel  as always, it's pleasure to work with you

PnP #sign-off
#remove-label:"pnp-review-in-progress"
#remove-label:"do-not-merge"
#label:"ready-to-merge"

#hold-off

@ckittel as per our conversation, I removed the second implementation. I think it should be fine now but a final check won't hurt :)


Not Working

Game crashes on Startup. No idea why. No log was given. No idea what to do.

Log always exists. .minecraft/logs/latest.log

Log always exists. .minecraft/logs/latest.log

latest.log

You're using the 1.19.3 version on 1.19.2. Try 2.13 instead

You're using the 1.19.3 version on 1.19.2. Try 2.13 instead

Tried that earlier. Didn't work. None of the commands work except /seed d
ata and /seed gui. It also states that it's the 1.18 version in the /seed gui.

It also states that it's the 1.18 version in the /seed gui.

That's because the cracking is the same in both versions, it does not matter.
What other commands did you try? If you send an unfilled command it will return an error because you didn't provide any further input

Well /locate is the one I care the most about but apparently the command doesn't exist and I can't gather data without finding valid structures. Without that I apparently can't crack the seed.

It also states that it's the 1.18 version in the /seed gui.

That's because the cracking is the same in both versions, it does not matter.
What other commands did you try? If you send an unfilled command it will return an error because you didn't provide any further input

/locate is not a Seedcracker command. I assume you watched my video on it. I was simply demonstrating what the different structures look like by teleporting to them. You have to walk around and find the structures, like I showed in the second part of the video.

/locate is not a Seedcracker command. I assume you watched my video on it. I was simply demonstrating what the different structures look like by teleporting to them. You have to walk around and find the structures, like I showed in the second part of the video.

What's the command to outline the structures? Is it /seed render outlines ON ?

I just found a shipwreck and got no data bits from it so now I'm just confused.

Also. Anti Xray doesn't seem to work.

Anyone able to help with this??

help is in discord

help is in discord

I don't have the discord. You could've just helped me here but aight.

Anyone able to help? Please?

I give up on this.


Add channel reserve as a constraint during order matching/expression

Pull Request Checklist

[ ] LndServices minimum version has been updated if new lnd apis/fields are
used.

For the record, per 2022-11-08 meeting this feature will be removed from sprints until work could resume in 2023-Q1


Retries with exponential backoffs for JRPS calls

Used by Indexed and Protocol chain structs.
Cloning was necessary for some web3 and backoff crate function calls,
as they won't take their parameters as references.
Fixes #15.

I really like backoff trait! I think we might want to try encapsulate this logic in a struct that implements web3::Transport. This way, we'd have something like inner: Web3<JsonRpcExponentialBackoff<Http>> and we don't have to create ad-hoc methods. WDYT? Kinda like this:

/// A [`web3::Transport`] that mimics another [`web3::Transport`] using
/// exponential backoff to retry calls.
#[derive(Debug, Clone)]
pub struct JsonRpcExponentialBackoff<T> {
    transport: Arc<T>,
    strategy: RetryStrategy,
}

impl<T> web3::Transport for JsonRpcExponentialBackoff<T>
where
    T: web3::Transport + Send + Sync + 'static,
    T::Out: Send,
{
    type Out = futures::future::BoxFuture<'static, <T::Out as futures::Future>::Output>;

    fn prepare(
        &self,
        method: &str,
        params: Vec<jsonrpc_core::Value>,
    ) -> (web3::RequestId, jsonrpc_core::Call) {
        self.transport.prepare(method, params)
    }

    fn send(&self, id: web3::RequestId, request: jsonrpc_core::Call) -> Self::Out {
        let transport = self.transport.clone();
        let retry_strategy = self.retry_strategy.clone();

        Box::pin(async move {
            retry(retry_strategy, || async { transport.send(id, request.clone()) }).await
        })
    }
}

This is a


This page needs suppression guidance for things like Application_Start

The current advice in "When to suppress warnings" doesnt make sense. Its a warning about a method name. It can be whatever is legal and still be "safe".
It also doesnt explain what to do for the MSFT issued names like Application_Start that cause this CA rule to pop.  Clearly we want to suppress those instances, but really we dont want to be informed of them in the first place.
Do note that some of us use "Treat Warnings as Errors" to prevent compiler warning creep, and that suppressing these kind of things is required for production code to even compile.

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

ID: 06cee199-3ccc-7575-2c1d-296e2714a16e
Version Independent ID: 2be241bf-b98a-5088-1e1b-0e39b1384b9a
Content: CA1707: Identifiers should not contain underscores - Visual Studio
Content Source: docs/code-quality/ca1707.md
Product: visual-studio-windows
Technology: vs-ide-code-analysis
GitHub Login: @jillre
Microsoft Alias: jillfra

It appears that Application_Start needs to be renamed. Please use the Report a Problem tool to report the issue. Then your report gets listed in the in the Visual Studio Developer Community, which allows you to engage directly with the engineering team while they address and fix the problem.

Thanks @Mikejo5000, I'm pretty familiar with devcomm. This is a document problem though, because the direction on how to fix this for some MSFT required method names (like application_start) will break functionality in an asp.net website. For those well known methods the rule should be suppressed.
I can create a separate issue for the CA rule flagging this once I can find that repo again (any hints? I know I've been there but GH search is returning a few hundred results)

Sure, I believe that is https://github.com/dotnet/roslyn-analyzers/

Updating When to Suppress section for well-known methods.


docs.rs failed to build drink-0.9.0

Crate name
drink
Build failure link
https://docs.rs/crate/drink/0.9.0/builds/1121750
Additional details
Normal cargo doc works fine. My guess is that stdsimd feature was removed in the recent nightly that is used for building docs. Probably relevant errors reported recently: https://github.com/rust-lang/rust/issues/48556

This is an issue with ahash, see also https://github.com/tkaitchuck/aHash/issues/200
I'm happy to trigger a rebuild if it's fixed.

@pmikolajczyk41 since ahash was fixed, I triggered a rebuild, now we're seeing another build failure.
Can you check if there also happen locally for you?

@syphar again, locally works well, fails with stdsimd at ahash

oh, I was looking at the most recent version of library, i.e. 0.10.0, where the problem with ahash persists (https://docs.rs/crate/drink/0.10.0/builds/1122573); locally, when I run cargo doc (for version 0.10.0) it succeeds

I queued a rebuild, and we're now seeing another error, this time with pallet-contracts:
https://docs.rs/crate/drink/0.10.0/builds/1129453
Interestingly, pallet-contracts itself also had this error : https://docs.rs/crate/pallet-contracts/27.0.0/builds/1099632 , and there is a new version that doesn't have it: https://docs.rs/crate/pallet-contracts/28.0.0/builds
So it looks like upgrading pallet-contracts would be the next step to try


Market looks ok

when Market looks ok, bot is running...
this error comes out:
Traceback (most recent call last):
File "c:/Users/pc/Desktop/TradingTry/Binance Detect Moonings.py", line 597, in
orders, last_price, volume = buy()
File "c:/Users/pc/Desktop/TradingTry/Binance Detect Moonings.py", line 294, in buy
volume, last_price = convert_volume()
File "c:/Users/pc/Desktop/TradingTry/Binance Detect Moonings.py", line 255, in convert_volume
volatile_coins, number_of_coins, last_price = wait_for_price()
File "c:/Users/pc/Desktop/TradingTry/Binance Detect Moonings.py", line 139, in wait_for_price
if historical_prices[hsp_head]['BNB' + PAIR_WITH]['time'] > datetime.now() - timedelta(minutes=float(TIME_DIFFERENCE / RECHECK_INTERVAL)):
KeyError: 'BNBUSDT'
what's the solution ?

Have you found the answer?

You  must have  BNB symbol in files tickers & signalsample, after this ishould work!

Make sure you have BNB as one of the tickers in tickers.txt. It worked for me


[Feature Request] Autoupdating Stewfile

Hi, thanks for stew, its been really useful.
I sync my Stewfile between workstations: when I install or update a binary on one PC, I manually regenerate a new Stewfile to propagate the changes to the other.
Could we set a Stewfile to be updated automatically after every operation? Something like stew file ~/dotfiles/Stewfile. I could contribute a PR for this.

If I understood you correctly, then the file you're looking for is Stewfile.lock.json which exists in the stewPath. The Stewfile.lock.json is pretty much a human-readable db for Stew. Every time you install/update/uninstall a binary, the Stewfile.lock.json gets updated.
The Stewfile was always meant to be easy to create, easy to destroy, and human-writable. If you only wanted to install a subset of your currently installed binaries, then you could manually write out a Stewfile or autogenerate one and remove the lines you didn't want.
Currently, you can't install from the Stewfile.lock.json, but I think that you should be able to.
Desired new behavior:
stew install /path/to/Stewfile.lock.json

I would be happy to review a PR that adds this feature.

this sounds good, I'll work on it over the weekend.


edit-profile does not point to $profile

edit-profile wants to create profile.ps1 rather than edit microsoft.powershell_profile.ps1

same here

As stated in setup.ps1:

If you want to add any persistent components, please do so at [$profilePath\Profile.ps1] as there is an updater in the installed profile which uses the hash to update the profile and will lead to loss of changes

So it seems to be working as intended.

For more context, the profile provided in this repo is for all users. By using the Edit-Profile function, you create a separate profile for the current user, where new functions can be defined without interfering with the main profile.


Add filter by custom tag

Mostro send nip33 events to port 30078, currently we are sending events related with different data, those have a data_label custom tag, right now we have these possible values order, dispute and rating, in order to have a more efficient subscription we should add a filter rule to get only rating events
Here an example of an event with that custom tag
[
  "EVENT",
  "RAND",
  {
    "id": "44cad49530f45150dee36a8c3e5e11e94c8ba2f44ba9960013f0ec6f84e50c2d",
    "pubkey": "dbe0b1be7aafd3cfba92d7463edbd4e33b2969f61bd554d37ac56f032e13355a",
    "created_at":<PHONE_NUMBER>,
    "kind": 30078,
    "tags": [
      [
        "d",
        "npub1qqqxssz4k6swex94zdg5s4pqx3uqlhwsc2vdzvhjvzk33pcypkhqe9aeq2"
      ],
      [
        "total_reviews",
        "6"
      ],
      [
        "total_rating",
        "3.6"
      ],
      [
        "last_rating",
        "4"
      ],
      [
        "max_rate",
        "1"
      ],
      [
        "min_rate",
        "5"
      ],
      [
        "data_label",
        "rating"
      ]
    ],
    "content": "",
    "sig": "225f73bae421b4f19eb3797ea20b6549afc2e498ae13876e24e79e7b83e2d95bbf0bc278ae89023bdb2d333bec4e564002f3e6e316ff41a7864c4c5e6f48ff1c"
  }
]

We should implement that filter here:
https://github.com/MostroP2P/mostro/blob/main/src/app/rate_user.rs#L23-L26

this is implemented already


bug when using 'some' alias for the 'contains' method Collection

Laravel version: 5.7.13
PHP version: 7.2.12
Illuminate\Support\Collection

Laravel 5.7.13 introduced an alias (#26376), named some, for the contains method on the Illuminate\Support\Collection class.
Although contains is working , some is not.
For instance, this is the actual behavior using contains:
$collection = collect(['name' => 'Desk', 'price' => 100]);
$collection->contains('New York'); // false

However, when using some, the output isn't the same as contains:
$collection = collect(['name' => 'Desk', 'price' => 100]);
$collection->some('New York'); // true

Another example (same as from the Laravel documentation):
$collection = collect([1, 2, 3, 4, 5]);
$collection->contains(function ($value, $key) {
    return $value > 5;
});

// false

Now using some:
$collection = collect([1, 2, 3, 4, 5]);
$collection->some(function ($value, $key) {
    return $value > 5;
});

// PHP Warning:  explode() expects parameter 2 to be string, object given ...

I wrote a testSome() test using the same logic for the testContains() and it failed.
I believe the reason for such a odd behavior is because of the func_num_args() call used by the contains method.
Anyways, I'm going to submit a PR that tries to handle that situation.

Gonna close this as your PR was merged.


Update pgrx, add postgres 16

Note: Solves #1088

Would love to have this since Postgres just removed 16.rc1 from its FTP repo and we are breaking on pgrx init using 0.10.0.

@maparent I'll take us all the way to pgrx 0.11.0. This will require some handholding with our CI, since you can't really our Github actions locally (requires access to our APT repo and such).
https://github.com/postgresml/postgresml/pull/1099
Thank you for the PR, mine is largely inspired by yours.

Superseded by https://github.com/postgresml/postgresml/pull/1102


Implement interface methods for 2017.3

fixes #2152
In 2017.3 readExternal and writeExternal are deprecated regular interface methods (so they need to be implemented).
In 2018.1 they are made default methods so no impl in needed. This adds an empty impl to quite 2017.3.

why they decided to make the default instead of removing them is beyond me..


what happened to this repo?

How come all commits get squashed?

I'm wondering the same thing...

+1

How come all commits get squashed?

Can you provide more details ? This is not an active repo it seems


Hang when trying to connect to a device that is not present

Hi,
I am using the latest code from arduino-esp32.
When trying to connect to a device which is not available, the application hangs.
Is there a way to pass a timeout to the connect function or something like that?
Here is a capture of the logs:
.[0;32m[I][bleESP32.cpp:558] esp_ble_gap_cb(): Stop scan successfully.[0m
Connecting
.[0;32m[I][BLEDevice.cpp:600] addPeerDevice(): add conn_id: 0, GATT role: client.[0m
.[0;36m[D][FreeRTOS.cpp:158] take(): Semaphore taking: name: RegEvt (0x3ffe39d0), owner: <N/A> for connect.[0m
.[0;36m[D][FreeRTOS.cpp:167] take(): Semaphore taken:  name: RegEvt (0x3ffe39d0), owner: connect.[0m
.[0;36m[D][BLEDevice.cpp:154] gattClientEventHandler(): gattClientEventHandler [esp_gatt_if: 4] ... Unknown.[0m
.[0;36m[D][BLEClient.cpp:158] gattClientEventHandler(): gattClientEventHandler [esp_gatt_if: 4] ... Unknown.[0m
.[0;36m[D][FreeRTOS.cpp:158] take(): Semaphore taking: name: OpenEvt (0x3fff3e4c), owner: <N/A> for connect.[0m
.[0;36m[D][FreeRTOS.cpp:167] take(): Semaphore taken:  name: OpenEvt (0x3fff3e4c), owner: connect.[0m
.[0;36m[D][BLEDevice.cpp:154] gattClientEventHandler(): gattClientEventHandler [esp_gatt_if: 4] ... Unknown.[0m
.[0;36m[D][BLEClient.cpp:158] gattClientEventHandler(): gattClientEventHandler [esp_gatt_if: 4] ... Unknown.[0m
.[0;32m[I][BLEDevice.cpp:611] removePeerDevice(): remove: 0, GATT role client.[0m
.[0;36m[D][BLEDevice.cpp:154] gattClientEventHandler(): gattClientEventHandler [esp_gatt_if: 4] ... Unknown.[0m
.[0;36m[D][BLEDevice.cpp:154] gattClientEventHandler(): gattClientEventHandler [esp_gatt_if: 4] ... Unknown.[0m

Thanks!

I would suggest to change this line:
https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils/BLEClient.cpp#L127
and use this instead with some values that will satisfy your needs:
https://github.com/nkolban/esp32-snippets/blob/master/cpp_utils/FreeRTOS.cpp#L188

Hi,
I tried your suggestion and it does not work.  I think I must change something else
The connect semaphore is taken even if the connection has not been made, so bool FreeRTOS::Semaphore::take(uint32_t timeoutMs, std::string owner) never returns false.
Here is my code in case you want to have a loook:
bool FreeRTOS::Semaphore::take(uint32_t timeoutMs, std::string owner) {
	ESP_LOGV(LOG_TAG, "Semaphore taking: %s for %s", toString().c_str(), owner.c_str());
	bool rc = false;
	if (m_usePthreads) {
		assert(false);  // We apparently don't have a timed wait for pthreads.
	} else {
		rc = ::xSemaphoreTake(m_semaphore, timeoutMs / portTICK_PERIOD_MS) == pdTRUE;
	}
	m_owner = owner;
	if (rc) {
		ESP_LOGV(LOG_TAG, "Semaphore taken:  %s", toString().c_str());
	} else {
		ESP_LOGE(LOG_TAG, "Semaphore NOT taken:  %s", toString().c_str());
	}
	return rc;
} // Semaphore::take

bool BLEClient::connect(BLEAddress address, esp_ble_addr_type_t type) {
	ESP_LOGD(LOG_TAG, ">> connect(%s)", address.toString().c_str());

	m_peerAddress = address;

	// Perform the open connection request against the target BLE Server.
	bool semaphoreTaken = m_semaphoreOpenEvt.take(2000, "connect");
	if (!semaphoreTaken)
	{
		ESP_LOGE(LOG_TAG, "connect failed");
		return false;
	}
	esp_err_t errRc = ::esp_ble_gattc_open(
		m_gattc_if,
		*getPeerAddress().getNative(), // address
		type,          // Note: This was added on 2018-04-03 when the latest ESP-IDF was detected to have changed the signature.
		1                              // direct connection <-- maybe needs to be changed in case of direct indirect connection???
	);
	if (errRc != ESP_OK) {
		ESP_LOGE(LOG_TAG, "esp_ble_gattc_open: rc=%d %s", errRc, GeneralUtils::errorToString(errRc));
		return false;
	}

	uint32_t rc = m_semaphoreOpenEvt.wait("connect");   // Wait for the connection to complete.
	ESP_LOGD(LOG_TAG, "<< connect(), rc=%d", rc==ESP_GATT_OK);
	return rc == ESP_GATT_OK;
} // connect

Logs:
.[0;36m[D][BLEClient.cpp:120] connect(): >> connect(4c:65:a8:d4:13:78).[0m
.[0;37m[V][FreeRTOS.cpp:189] take(): Semaphore taking: name: OpenEvt (0x3ffdfd10), owner: <N/A> for connect.[0m
.[0;37m[V][FreeRTOS.cpp:198] take(): Semaphore taken:  name: OpenEvt (0x3ffdfd10), owner: connect.[0m
.[0;37m[V][FreeRTOS.cpp:69] wait(): >> wait: Semaphore waiting: name: OpenEvt (0x3ffdfd10), owner: connect for connect.[0m
.[0;37m[V][BLEUtils.cpp:952] gattClientEventTypeToString(): Unknown GATT Client event type: 41.[0m
.[0;36m[D][BLEDevice.cpp:150] gattClientEventHandler(): gattClientEventHandler [esp_gatt_if: 4] ... Unknown.[0m
.[0;37m[V][BLEUtils.cpp:952] gattClientEventTypeToString(): Unknown GATT Client event type: 41.[0m
.[0;37m[V][BLEUtils.cpp:1284] dumpGattClientEvent(): GATT Event: Unknown.[0m
.[0;37m[V][BLEUtils.cpp:952] gattClientEventTypeToString(): Unknown GATT Client event type: 41.[0m
.[0;36m[D][BLEClient.cpp:173] gattClientEventHandler(): gattClientEventHandler [esp_gatt_if: 4] ... Unknown.[0m
.[0;37m[V][BLEUtils.cpp:952] gattClientEventTypeToString(): Unknown GATT Client event type: 2.[0m
.[0;36m[D][BLEDevice.cpp:150] gattClientEventHandler(): gattClientEventHandler [esp_gatt_if: 4] ... Unknown.[0m
.[0;37m[V][BLEUtils.cpp:952] gattClientEventTypeToString(): Unknown GATT Client event type: 2.[0m
.[0;37m[V][BLEUtils.cpp:1284] dumpGattClientEvent(): GATT Event: Unknown.[0m
.[0;37m[V][BLEUtils.cpp:952] gattClientEventTypeToString(): Unknown GATT Client event type: 2.[0m
.[0;36m[D][BLEClient.cpp:173] gattClientEventHandler(): gattClientEventHandler [esp_gatt_if: 4] ... Unknown.[0m
.[0;37m[V][FreeRTOS.cpp:119] give(): Semaphore giving: name: OpenEvt (0x3ffdfd10), owner: connect.[0m
.[0;37m[V][FreeRTOS.cpp:85] wait(): << wait: Semaphore released: name: OpenEvt (0x3ffdfd10), owner: <N/A>.[0m
.[0;36m[D][BLEClient.cpp:143] connect(): << connect(), rc=0.[0m
.[0;36m[D][BLEClient.cpp:153] disconnect(): >> disconnect().[0m
.[0;33m[W][BLEClient.cpp:154] disconnect(): gattIf: 4, connId: 0.[0m
.[0;36m[D][BLEClient.cpp:160] disconnect(): << disconnect().[0m

Then its not problem with hang when connect.
.[0;36m[D][BLEClient.cpp:153] disconnect(): >> disconnect().[0m
.[0;33m[W][BLEClient.cpp:154] disconnect(): gattIf: 4, connId: 0.[0m
.[0;36m[D][BLEClient.cpp:160] disconnect(): << disconnect()

Maybe the same change should be applied somewhere else, but i dont know where.

Thanks anyway :)
I'll try to patch it or work around it. I will let you know if I make any progress.
If not , hopefully you get some time to look at it further on.


Clog Copy-Paste Error

Click to expand!
Issue Type
Bug
Source
source
Tensorflow Version
master
Custom Code
No
OS Platform and Distribution
Any
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
In workspace2.bzl on line 162 clog was copy and pasted from cpu-info (below it) so both import the same archive and clog is not included.
    tf_http_archive(
        name = "clog",
        strip_prefix = "cpuinfo-5e63739504f0f8e18e941bd63b2d6d42536c7d90",
        sha256 = "18eca9bc8d9c4ce5496d0d2be9f456d55cbbb5f0639a551ce9c8bac2e84d85fe",
        urls = tf_mirror_urls("https://github.com/pytorch/cpuinfo/archive/5e63739504f0f8e18e941bd63b2d6d42536c7d90.tar.gz"),
    )

    tf_http_archive(
        name = "cpuinfo",
        strip_prefix = "cpuinfo-5e63739504f0f8e18e941bd63b2d6d42536c7d90",
        sha256 = "18eca9bc8d9c4ce5496d0d2be9f456d55cbbb5f0639a551ce9c8bac2e84d85fe",
        urls = tf_mirror_urls("https://github.com/pytorch/cpuinfo/archive/5e63739504f0f8e18e941bd63b2d6d42536c7d90.tar.gz"),
    )

Standalone code to reproduce the issue
View master branch

Relevant log output
No response

Hi, I think the issue has been taken care with the fix https://github.com/tensorflow/tensorflow/commit/d90b6bc733db5a78ae28b815495391e6715d4ac2 which removed clog section and kept only cpuinfo as below.
https://github.com/tensorflow/tensorflow/blob/d90b6bc733db5a78ae28b815495391e6715d4ac2/tensorflow/workspace2.bzl#L162-L167.
Could you please close this issue, if you think your issue is addressed. Thanks!


Add word analogies task

This task replicates word analogies questions on tests often administered to human students.

@chiafullo This task needs approval for the workflow to run

I just approved the workflow.
On Tue, Jun 15, 2021, 07:42 Stella Biderman @.***>
wrote:

@chiafullo https://github.com/chiafullo This task needs approval for
the workflow to run
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/BIG-bench/pull/402#issuecomment-861559016, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADZW4AQRT5ZD2OIZ4YNU5DTS5RELANCNFSM455UDE2A
.

@p-patil We are now requesting that task authors please include an explicit section in the README.md file called "data source", listing exactly where you got your data from (or explicitly state that you made it up yourself). Many thanks!

I am one of the reviewers assigned to review this task.
Thank you for the submission! Overall, I am weakly positively disposed towards this task. However I have several questions that I need the authors to answer before I can feel comfortable recommending this task for inclusion.
Correctness

The README needs a "data source" section.
The task passes the necessary checks.

Formatting

The formatting of the task appears to be good.

Specificity

I don't have any particular concerns about specificity.

Thoroughness

The authors write "the task is limited to analogies which is only one type of desired knowlege. While every attempt has been made to cover various types of analogies there still might be some cases which might have been missed." Could they elaborate as to what this means?

Difficulty

The authors evaluate on GPT-1 and GPT-2, but note that GPT-3 might do better. It would be nice to see an evaluation using EleutherAI's GPT-Neo 2.7B or GPT-J if evaluating on GPT-3 is financially unviable.

Memorizing the Internet

The authors write "the word analogies themselves were scraped from the Internet" which leads me to worry that this can be solved merely by memorizing the internet. If the authors could elaborate that would be great.

Novelty

There appears to be some novelty in this task. It implements a common test, albeit with data that appears to be new.

Justification

The justification raises as many questions as it answers. I strongly urge the authors to edit the README to include more detail into their rationale and reasoning about designing this task.
The authors write "we aimed for a standard commensurate to college-level aptitude exams administered to human students." Were specific college aptitude exams used as a point of reference? College aptitude exams can massively vary by country. Were problems that were too hard removed, or only ones that were too easy? Why was a college aptitude test selected as the target difficulty?
The authors write "some of the words used are not very common and thus may not exist in the chosen language model's vocabulary, which will likely render the containing question invalid." Did the authors remove questions due to this criteria? How was the obscurity of a word measured or assessed? As phrased, there's nothing actionable here.

Size

There are 70 questions in Analogies 1.
There are 35 questions, the bare minimum allowed for consideration.
Could the authors explain how resource limitations effected the number of tasks? Were there far fewer "hard" (Analogies 2) problems than easy (Analogies 1) problems? Was the difference in size a deliberate choice?

Compute Resources

I see no reason to think running this task will require excessive amounts of compute.

Thank you for your feedback, @StellaAthena ! Sahib, the author of Analoges 2, and I have tried to address your comments by updating the relevant README.md files. Please note that this task was merged with another task #351 which is where Analogies 2 comes from, so this may account for any discrepencies between the two.
In summary:

The "Data source" section has been added to both Analogies 1 and Analogies 2.
We agree that additional evaluations would be nice to have. Unfortunately we have been running into some difficulty evaluating other models, but I am trying to get the results with GPT-Neo 2.7B by tomorrow.
As also mentioned in the "Data source" section, the questions in Analogies 1 were obtained from the book "501 Word Analogy Questions", which is available as a PDF on the internet. Unfortunately, I am not very familiar with the details of how the data source for GPT-3 was scraped, and whether this includes PDF files, but I believe the Analogies 2 task was not obtained from the internet and so a large score differential would indicate that the model is memorizing. I understand the concern here and am happy to replace some of the questions with ones not available on the Internet if you prefer!
Apologies for the vagueness in the "Design considerations" section. To clarify, these questions were designed for preparatory use for American aptitude tests, primarily the SAT or GRE. It is thus very likely that these exams were used as a point of reference when designing the questions. These questions were chosen from the larger set of questions to reflect the full range of difficulty available in the data source. Questions were not explicitly removed for being too easy or too difficult.
We targeted college-level aptitude tests primarily because word analogies in this context are well studied and have a solid basis in the psychometric community as a measure of general knowledge and verbal reasoning in adults fluent in the language. Our rationale was that it would be interesting and valuable to evaluate the performance of very large language models capable of generating human-quality natural language against these standards. In our estimation, the questions often require a nuanced and multi-faceted understanding of various semantic concepts, and thus could be a good proxy for measuring general verbal intelligence and world understanding.
With regards to the inclusion of obscure words, we did not remove any questions due to this criterion. This design consideration is based on the our intuitive observations that some of the questions use words rarely used in English vernacular, and was mentioned as a warning that this could distort results. If this phrasing is causing more confusion that it's worth, I'm happy to remove it!
Analogies 1 and Analogies 2 were initially independently created tasks that were later merged. No difference in difficulty or size was explicitly intended.

Happy to go into more detail on any of these points, or modify the README.md further!

Also, would like to mention I have updated the README to reflect the Thoroughness point better.

@chiafullo Reject. All questions come from "501 Word Analogy Questions," a book that is readily available online. This is a quintessential example of a task that can be solved by memorizing the internet.

@StellaAthena As the author of AnalogyTask2, I can tell you that none of the questions were picked from that book. I handpicked them from various textbooks none of which are available online. I even sampled a few of my questions to check if they were in the book you mentioned and none of them were there.

@StellaAthena As the author of AnalogyTask2, I can tell you that none of the questions were picked from that book. I handpicked them from various textbooks none of which are available online. I even sampled a few of my questions to check if they were in the book you mentioned and none of them were there.

I see, I misread @p-patil's comment.
@chiafullo I am not sure if this meets the requirements. It seems that this PR contains two tasks, one of which meets the requirements and one of which doesn't. I'll defer to you as to if it should be accepted as is, or if the Analogies 1 task should be deleted before acceptance.

@StellaAthena thank you for pointing this out.  I will push through for meta review and note your comment about AnalogyTask2. The meta reviewer will advise @p-patil  from there! Thank you again :)

Your task has been accepted in the initial review stage.  The next stage of this process is for us to assign your task a meta-reviewer for final review and merge.  An assigned meta-reviewer will follow-up by commenting on the PR should it need further revisions.

Hello, I'm the metareviewer for this task!
While we appreciate the effort you put into this task, we unfortunately can't accept it. Note that I took a look at both subtasks included in this PR, and there seems to be outstanding issues remaining with both (one needs to be deleted, and after that deletion, the remaining task is quite small).  Further, when spot-checking many of those remaining analogies, they seem to be confusing or possibly wrong.  Some examples even appeared in google searches, which means this task fails the "memorize the internet" criteria.
Best regards,
Daniel


Unify Entry::deriveAddress() methods

Description
Unify the 3 deriveAddress() methods in the coin Entry classes; internal refactor with no change exposed outside.
There are 3 variants of the deriveAddress() methods:

The original one, with p2pkh and hrp prefix parameters, used by most coins
Version with extra derivation parameter, used by Bitcoin. This became a separate variant not to touch all Entry classes.
Version with extra PrefixVariant parameters.

All these can be replaced by one version, with derivation and prefix parameters.
How to test
All tests.
Types of changes

Non-breaking refactor

Checklist

[x] Create pull request as draft initially, unless its complete.
[x] Add tests to cover changes as needed.
[x] Update documentation as needed.
[x] If there is a related Issue, mention it in the description.

If you're adding a new blockchain

[ ] I have read the guidelines for adding a new blockchain.

Nicely done!

Sonarcloud complains about code dupplicates

I realized I still have to update the code generation templates.


how to setup?

i get errors after installing this plugin. how do i found out the right port? do i have to install the vcontrold-plugin too?
ip adress of the homebridge or the heating system/vitoconnect?

Hi,
thanks for installing the plugin.
It uses the vcontrol daemon. You have to set up the IP address and port under which vcontrol is reachable.
This plugin is made for my individual configuration and setup with two heating circles.
In order to use it you would need to have the same configuration and commands in vcontrol as the plugin expects.
Best
Roman


There is no method for collision on vertex shader shapes or heightmaps

Godot version: 7f691e8bb9d7ffc97b018710ebe5ae3a3b5a81d1

OS/device including version:
Windows 10 Pro (64), Intel 7700k, GTX 1080 Gaming X MSI
Issue description:
When using vertex shaders for things such as terrain generation for example there does not seem to be any way to have a collision shape for it. This is what happens when you apply a static collision mesh to a plane affected by terrain code made in this tutorial: https://docs.godotengine.org/en/2.1/community/tutorials/3d/mesh_generation_with_heightmap_and_shaders.html
https://docs.godotengine.org/en/3.1/tutorials/shading/vertex_displacement_with_shaders.html?highlight=terrain

i found some related issues and PRs but none seem to be implemented, available, or conver all cases (for example people not using bullet, using your own terrain generation code instead of bullets height map implementation, or other vertex transforms that people may want to make collide-able)
https://github.com/godotengine/godot/issues/24548
https://github.com/godotengine/godot/pull/17806
https://github.com/godotengine/godot/issues/9046
Some potential options I can think of for allowing collision on these types of objects are:
Vertex Shaders on collision, Heightmaps for bullet, vertex "baking" (applying the vertex transformation currently happening permanently, having the collision generator check for vertex transforms, or allowing vertex shaders on physics shapes?
and of course the built in bullet functionality is still an option if anyone ever decides to add that.
Steps to reproduce:
follow https://docs.godotengine.org/en/2.1/community/tutorials/3d/mesh_generation_with_heightmap_and_shaders.html
or
https://docs.godotengine.org/en/3.1/tutorials/shading/vertex_displacement_with_shaders.html?highlight=terrain
click add trimesh static body to resulting terrain
If there is some buried way to make a collision mesh for this already I ask that the documentation on that be made more clear, but as I cannot find it being anywhere at all I doubt there is a feature for this at all. I would like to see at least some way for this to be achievable in godot as having no proper or standardized method of terrain generation that can actually have physics is kind of a road block.
Minimal reproduction project:
https://gitlab.com/atorresm/Godot-LowPolyTerrainDemo (not mine)

You are misunderstanding how shaders work. They are not applied until the data is sent to the GPU to be drawn to the screen. The data never comes back to the CPU. The mesh itself is never changed. Each frame a flat plane is being drawn and the shader deforms it based on the vertex shader. The key part is that this happens every frame and the values are thrown away every time. There is no way to bake the height values in. This is a limitation of shaders (not Godot), but it is also a reason why shaders are able to run so fast.
You can see in the issues you linked to, there is interest in exposing the heightfield abilities of bullet. This is as close as you can get to what you want to do. Making a static heightmap and having bullet use it for collisions and also apply it to your terrain.

that is why I mentioned "baking" as a fairly simple solution. The editor already displays the vertex shaders modified positions so for something like terrain where it is static if you generate a collision mesh in the editor "baking" the vertex shaders output at that time then you would actually be able to properly use terrain generation.

what about the option of native heightmap mesh instance or shape which you can pass a texture into?

That's exactly what the issue you linked is asking for https://github.com/godotengine/godot/issues/24548 and the PR you linked implements https://github.com/godotengine/godot/pull/17806 (but only from physicsserver)

What on earth... I just found this. https://github.com/Zylann/godot_heightmap_plugin/tree/master/addons/zylann.hterrain
If this guy can check the collision of shader terrain using only gdscript i'm sure the devs can do it in c++

He is the author of PR #17806. The technique he implemented in that PR is the one he uses in that plugin. Its using a heightmap. And the functionality is in the engine.

What on earth... I just found this. https://github.com/Zylann/godot_heightmap_plugin/tree/master/addons/zylann.hterrain
If this guy can check the collision of shader terrain using only gdscript i'm sure the devs can do it in c++

I'm pretty sure that implementation uses a shader for the mesh generation but still computes the mesh CPU-side for the collisions.

That's the case, I use shader displacement for rendering the terrain, but the heightmap is also copied in RAM for handling collisions and performing edition/height/texture query tasks. The PR I made basically exposes Bullet's heightmap shape to a minimum, which was just there waiting to be used. If not using Bullet, Godot Physics is lacking an implementation of that shape.
The PhysicsServer API is powerful but a bit harder to use, so https://github.com/godotengine/godot/pull/27156 has been recently proposed to add this API to the node/resource system.

I think this issue can be closed, it's not clear what the expected outcome is, but the feature requested is available via #17806 and #27156.
Anything beyond that should now go through the godot-proposals repository (though I'm sure there are already proposals relative to making terrains open there).


Added imvotenet image branch pretrain

Task linked: CU-5dqv4g ImVoteNet Faster R-CNN Pre-training (#217)


[CLI] - Key Server Issue

I don't know if this is going to be a long term issue, but just trying to do a fresh build of SIFT using the instructions on the CLI page and the key server listed there (pgp.mit.edu) is really tempremental.
Anyway, to cut a long story short, according to an article from March 2017, this seems to be a common occurrence (https://github.com/nginxinc/docker-nginx/issues/156) .
As per that article, may I suggest that you change the instructions to reference the pool of PGP servers rather than just the MIT one specifically. This would mean that installation instruction three would read:

Import the PGP Key - gpg --keyserver hkp://pool.sks-keyservers.net --recv-keys 22598A94

I understand that there may be a valid reason for solely referencing the MIT server, but for those who have also been experiencing this issue, I have tested this successfully and can now continue with the build.

If you are wanting to stick with a single server, maybe the Ubuntu one might be more stable, in which case:

Import the PGP Key - gpg --keyserver hkp://keyserver.ubuntu.com:11371 --recv-keys 22598A94

Kind regards
Gavin
Information on the different types of SKS pools
SKS pool server status page
P.S. Just for clarification, I have no affiliation with the SKS setup at all :-)

@SaintHeimFyrstr what type of network are you on? IPv4 or IPv6?

I'm on IPv4, and I was able to import the key from hkp://keyserver.ubuntu.com:11371, but not from hkp://pgp.mit.edu:80 and hkp://pool.sks-keyservers.net.
~$ date -u
Tue Mar 26 10:41:21 UTC 2019
~$ gpg --keyserver hkp://pgp.mit.edu:80 --recv-keys 22598A94
gpg: keyserver receive failed: No data
~$ gpg --keyserver hkp://pool.sks-keyservers.net --recv-keys 22598A94
gpg: keyserver receive failed: No data
~$ gpg --keyserver hkp://keyserver.ubuntu.com:11371 --recv-keys 22598A94
gpg: /home/<username>/.gnupg/trustdb.gpg: trustdb created
gpg: key 15B9AD7122598A94: public key "SANS Investigative Forensic Toolkit<EMAIL_ADDRESS>imported
gpg: Total number processed: 1
gpg:               imported: 1


Fix use_appearance_embedding for instant-ngp

@liruilong940607 can you take a glance at this?
The issue I was trying to resolve was that the use_appearance_embedding config field is being ignored, which led to noticing that the TCNNInstantNGPField class isn't being used anywhere.
Perhaps this was changed temporarily to the nerfacto field and then accidentally merged?
After changing the NGP model's field from the nerfacto field to the NGP field I can still get reasonable results on lego, as long as I pass --alpha-color black to the dataparser config. (also not sure if this is the intended behavior)
cc @kevin-thankyou-lin

( for context this is related to PR #1809 )

I remember when I was working on that PR, using the nerfacto field gives noticeable better result than ngp field on the poster scene (not entirely sure now bc I didn't keep the record of it).
I was not sure what's the reason, but considering those two are very similar and the nerfacto field has been actively maintained, I felt it's a reasonable choice to just use the nerfacto field.
If you confirm the poster scene has similar result then I think it's fine to switch back. Other choice i would propose, is to modify the config use_appearance_embedding to use_average_appearance_embedding and feed it to the nerfacto field. This would keep the ngp models to have the same behavior with the PR #1809

Ok I just tried it out; I can confirm that the poster scene looks worse after this PR with the default settings, but at least qualitatively the NGP and nerfacto fields look the same when I set use_appearance_embeddings=True for the NGP field. Maybe that was the cause of the discrepancy you were seeing @liruilong940607? Appearance embeddings seem important for the poster scene.
All of this does raise the question of: do we even need separate Nerfacto field and NGP field implementations? Currently the latter is completely unused, and I agree it would make sense to just modify the Nerfacto field instantiation to respect use_appearance_embedding. cc @tancik

The only reason I was keeping the NGP field there is that the nerfplayer was relying on that (they have a ngp variation that inherits from ngp field).
I think it makes sense to just get rid of the ngp field as the nerfacto field is a superset of that, with only minor differences on the default behavior of the appearance embedding (anything else?).

It would be nice to remove TCNNInstantNGPField if it isn't actually useful.

Sounds good, might take a few days but I can try to remove the separate NGP field implementation.
In that direction, we could also add occupancy grid support to the nerfacto model and drop the InstantNGPModel implementation too? We could still keep the instant-ngp subcommand if we want to call it that, it would just point to the same config class but with the proposal networks off by default.

Any updates on this @brentyi ?

Forgot about it, but just added to my to-do list...


[BUG]

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
it doesnt show the multi account launching thing

Bloxstrap Versions 2.6.0 and above will no longer have Multi-Instancing.

Bloxstrap Versions 2.6.0 and above will no longer have Multi-Instancing.

why's that?

why's that?

https://github.com/pizzaboxer/bloxstrap/wiki/Plans-to-remove-multi‐instance-launching-from-Bloxstrap


Migrate history from conda-forge

Continuing from the discussion near the tail end of issue ( https://github.com/conda-forge/conda-execute-feedstock/issues/17 ).
Would like to figure out how we preserve the history from the conda-forge fork of conda-build-all. In particular the history from issue/PR discussions should they need to be revisited. One option might be to migrate the issues/PRs to this repo somehow (though not sure of a tool to do this for PRs). Another option would be to migrate that repo over here and note that it is historic. Though if there are other options, would be interested to hear them.

@jakirkham - this got done, right? https://github.com/conda-forge/conda-build-all shows a deprecated README.

This is referring to the discussion under issues and PRs in the conda-forge repo. So this isn't done.


Independent cities

Resolves #3584
Description
The initial counties implementation didn't include the independent cities of Baltimore, St. Louis, Carson City, and a slew in Virginia.   This addresses that.
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
manual verification that Baltimore is in the county lists
On copy of production, checked one partner that has served areas.  Same before and after.

Done and done!


[soci] Fix dependencies in, intern, out

Fixes #34168.
Major revision of dependency control.
The "empty" backend is now optional.
i.e. neither core nor default features include any backend. (sqlite3 might be useful. empty isn't.)
The odbc backend now has a unixodbc dependency when not on windows.
Adds a test-port that actually links to some backends.
Tested in x64-linux with (preview of) vcpkg test-features soci.
CC @baziorek @JonLiu1993

@dg0yt I've checked with provided commands (https://stackoverflow.com/questions/77140854/how-to-install-soci-db-library-with-vcpkg-on-linux) inside docker container and it is working. APPROVE:)

@dg0yt, Thanks for your PR, when I tested the usage I get this error:
1> CMake generation started for configuration: 'x64-Debug'.
1> Command line: "C:\WINDOWS\system32\cmd.exe" /c "%SYSTEMROOT%\System32\chcp.com 65001 >NUL && "C:\PROGRAM FILES\MICROSOFT VISUAL STUDIO\2022\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Visual Studio 17 2022" -A x64  -DCMAKE_CONFIGURATION_TYPES:STRING="Debug" -DCMAKE_INSTALL_PREFIX:PATH="C:\Users\v-zhli17\source\repos\usageTest\out\install\x64-Debug" -DCMAKE_TOOLCHAIN_FILE=F:/Feature-test/soci/vcpkg/scripts/buildsystems/vcpkg.cmake "C:\Users\v-zhli17\source\repos\usageTest" 2>&1"
1> Working directory: C:\Users\v-zhli17\source\repos\usageTest\out\build\x64-Debug
1> [CMake] -- Selecting Windows SDK version 10.0.22621.0 to target Windows 10.0.19045.
1> [CMake] -- The C compiler identification is MSVC 19.36.32538.0
1> [CMake] -- The CXX compiler identification is MSVC 19.36.32538.0
1> [CMake] -- Detecting C compiler ABI info
1> [CMake] -- Detecting C compiler ABI info - done
1> [CMake] -- Check for working C compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.36.32532/bin/Hostx64/x64/cl.exe - skipped
1> [CMake] -- Detecting C compile features
1> [CMake] -- Detecting C compile features - done
1> [CMake] -- Detecting CXX compiler ABI info
1> [CMake] -- Detecting CXX compiler ABI info - done
1> [CMake] -- Check for working CXX compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.36.32532/bin/Hostx64/x64/cl.exe - skipped
1> [CMake] -- Detecting CXX compile features
1> [CMake] -- Detecting CXX compile features - done
1> [CMake] -- Found ZLIB: optimized;F:/Feature-test/soci/vcpkg/installed/x64-windows/lib/zlib.lib;debug;F:/Feature-test/soci/vcpkg/installed/x64-windows/debug/lib/zlibd.lib (found version "1.3.0")
1> [CMake] -- Found OpenSSL: optimized;F:/Feature-test/soci/vcpkg/installed/x64-windows/lib/libcrypto.lib;debug;F:/Feature-test/soci/vcpkg/installed/x64-windows/debug/lib/libcrypto.lib (found version "3.1.3")
1> [CMake] -- Performing Test CMAKE_HAVE_LIBC_PTHREAD
1> [CMake] -- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Failed
1> [CMake] -- Looking for pthread_create in pthreads
1> [CMake] -- Looking for pthread_create in pthreads - not found
1> [CMake] -- Looking for pthread_create in pthread
1> [CMake] -- Looking for pthread_create in pthread - not found
1> [CMake] -- Found Threads: TRUE
1> [CMake] -- Configuring done (10.3s)
1> [CMake] CMake Error at usageTest/CMakeLists.txt:9 (target_link_libraries):
1> [CMake]   Target "test" links to:
1> [CMake]
1> [CMake]     SOCI::soci_core_static
1> [CMake]
1> [CMake]   but the target was not found.  Possible reasons include:
1> [CMake]
1> [CMake]     * There is a typo in the target name.
1> [CMake]     * A find_package call is missing for an IMPORTED target.
1> [CMake]     * An ALIAS target is missing.
1> [CMake]
1> [CMake]
1> [CMake]
1> [CMake] -- Generating done (0.1s)
1> [CMake] CMake Generate step failed.  Build files cannot be regenerated correctly.
1> 'C:\WINDOWS\system32\cmd.exe' '/c "%SYSTEMROOT%\System32\chcp.com 65001 >NUL && "C:\PROGRAM FILES\MICROSOFT VISUAL STUDIO\2022\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Visual Studio 17 2022" -A x64  -DCMAKE_CONFIGURATION_TYPES:STRING="Debug" -DCMAKE_INSTALL_PREFIX:PATH="C:\Users\v-zhli17\source\repos\usageTest\out\install\x64-Debug" -DCMAKE_TOOLCHAIN_FILE=F:/Feature-test/soci/vcpkg/scripts/buildsystems/vcpkg.cmake "C:\Users\v-zhli17\source\repos\usageTest" 2>&1"' execution failed with error: ''C:\WINDOWS\system32\cmd.exe' '/c "%SYSTEMROOT%\System32\chcp.com 65001 >NUL && "C:\PROGRAM FILES\MICROSOFT VISUAL STUDIO\2022\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Visual Studio 17 2022" -A x64  -DCMAKE_CONFIGURATION_TYPES:STRING="Debug" -DCMAKE_INSTALL_PREFIX:PATH="C:\Users\v-zhli17\source\repos\usageTest\out\install\x64-Debug" -DCMAKE_TOOLCHAIN_FILE=F:/Feature-test/soci/vcpkg/scripts/buildsystems/vcpkg.cmake "C:\Users\v-zhli17\source\repos\usageTest" 2>&1"' returned with exit code: 1'.

Please take a look.

All features are tested successfully in the following triplet:

x86-windows
x64-windows
x64-windows-static

Tested usage successfully by soci[*]:x64-windows-static:
soci provides CMake targets:

  # this is heuristically generated, and may not be correct
  find_package(SOCI CONFIG REQUIRED)
  # note: 2 additional targets are not displayed.
  target_link_libraries(main PRIVATE SOCI::soci_core_static SOCI::soci_odbc_static SOCI::soci_empty_static SOCI::soci_mysql_static)

PR tested by user @baziorek https://github.com/microsoft/vcpkg/issues/34168#issuecomment-1752147650

It is tested by a test port...

Amazing work as usual, thanks @dg0yt!

Thanks for fixing its!
Author of corrections has priority (if he wants) to answer the question and earn some reputation:
https://stackoverflow.com/questions/77140854/how-to-install-soci-db-library-with-vcpkg-on-linux
if @dg0yt does not want - I'll answer the question:).

@baziorek I'm not active on SO.

@baziorek I'm not active on SO.

OK, I'll reply on SO.

@dg0yt Something got wrong after my approve, because when I checked that time - everything was working, but when I check after merging into master (current top commit: 51475feaccb720ab40e5a0e3f8da70a4510defc0):  There are strange suggestions made by vcpkg in newest version. When I use commands (inside docker container Ubuntu22:04):
./vcpkg/vcpkg install 'soci[sqlite3]'

the output is:
soci provides CMake targets:

    find_package(soci CONFIG REQUIRED)
    target_link_libraries(main PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_core>,SOCI::soci_core,SOCI::soci_core_static>)

    # Linking specific backends (enabled: sqlite3)
    target_link_libraries(main PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_<BACKEND>,SOCI::soci_<BACKEND>,SOCI::soci_<BACKEND>_static>)

when I use those commands in CMake:
./vcpkg/vcpkg integrate install
cmake . -DCMAKE_TOOLCHAIN_FILE=/home/vcpkg/scripts/buildsystems/vcpkg.cmake && make

I see:
Soci found:  and
-- Configuring done
-- Generating done
-- Build files have been written to: /home
[ 50%] Building CXX object CMakeFiles/Learning_VCPKG.dir/main.cpp.o
[100%] Linking CXX executable Learning_VCPKG
/usr/bin/ld: CMakeFiles/Learning_VCPKG.dir/main.cpp.o: warning: relocation against `_ZN4soci7sqlite3E' in read-only section `.text'
/usr/bin/ld: CMakeFiles/Learning_VCPKG.dir/main.cpp.o: in function `main':
main.cpp:(.text+0x61): undefined reference to `soci::sqlite3'
/usr/bin/ld: warning: creating DT_TEXTREL in a PIE
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/Learning_VCPKG.dir/build.make:99: Learning_VCPKG] Error 1
make[1]: *** [CMakeFiles/Makefile2:83: CMakeFiles/Learning_VCPKG.dir/all] Error 2
make: *** [Makefile:91: all] Error 2

CMakeLists.txt (with suggestion used):
cmake_minimum_required(VERSION 3.16)
project(Learning_VCPKG LANGUAGES CXX)

find_package(SOCI CONFIG REQUIRED)

if (SOCI_FOUND)
    message("Soci found: ${SOCI_INCLUDE_DIRS} and ${SOCI_LIBRARY_DIRS}")
endif ()

add_executable(${PROJECT_NAME} main.cpp)

# find_package(soci CONFIG REQUIRED)
target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_core>,SOCI::soci_core,SOCI::soci_core_static>)

and main.cpp one more time:
#include <exception>
#include <iostream>
#include <soci/soci.h>
#include <soci/sqlite3/soci-sqlite3.h>
#include <string>

using namespace soci;
using namespace std;

int main()
{
    try {
        soci::session sql(sqlite3, "test.db");

        sql << "CREATE TABLE IF NOT EXISTS Employees("
               "ID     INT PRIMARY KEY NOT NULL,"
               "name   TEXT            NOT NULL,"
               "Age   INT             NOT NULL,"
               "Address  TEXT,"
               "Salary REAL);";
        sql << "INSERT INTO Employees (ID, name, Age, Address, Salary) VALUES (1, 'Tadeusz', 24, 'Krakow', 2000.01 ); ";
        sql << "INSERT INTO Employees (ID, name, Age, Address, Salary) VALUES (2, 'Symeon', 25, 'Texas', 15000.00 ); ";
        sql << "INSERT INTO Employees (ID, name, Age, Address, Salary) VALUES (3, 'Rodion', 23, 'Norway', 20000.00 );";

        int count;
        sql << "select count(*) from Employees", into(count);

        cout << "We have " << count << " entries in the Employees table.\n";
    } catch (exception const& e) {
        cerr << "Error: " << e.what() << '\n';
    }
}

target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_core>,SOCI::soci_core,SOCI::soci_core_static>)

You are linking core only. In a static build, you have to link the backend you want.

target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_core>,SOCI::soci_core,SOCI::soci_core_static>)

You are linking core only. ~In a static build~ When directly accessing a particular backend, you have to link that backend.

When I change the CMakeLists.txt:
cmake_minimum_required(VERSION 3.16)
project(Learning_VCPKG LANGUAGES CXX)

find_package(SOCI CONFIG REQUIRED)

if (SOCI_FOUND)
    message("Soci found: ${SOCI_INCLUDE_DIRS} and ${SOCI_LIBRARY_DIRS}")
endif ()

add_executable(${PROJECT_NAME} main.cpp)

target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_core>,SOCI::soci_core,SOCI::soci_core_static>)
target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_<BACKEND>,SOCI::soci_<BACKEND>,SOCI::soci_<BACKEND>_static>)

I see compilation problem:
cmake . -DCMAKE_TOOLCHAIN_FILE=/home/vcpkg/scripts/buildsystems/vcpkg.cmake && make
Soci found:  and
-- Configuring done
CMake Error at CMakeLists.txt:14 (target_link_libraries):
  Error evaluating generator expression:

    $<TARGET_EXISTS:SOCI::soci_<BACKEND>

  $<TARGET_EXISTS:tgt> expression requires a non-empty valid target name.

CMake Error at CMakeLists.txt:14 (target_link_libraries):
  Error evaluating generator expression:

    $<TARGET_EXISTS:SOCI::soci_<BACKEND>

  $<TARGET_EXISTS:tgt> expression requires a non-empty valid target name.

CMake Error at CMakeLists.txt:14 (target_link_libraries):
  Error evaluating generator expression:

    $<TARGET_EXISTS:SOCI::soci_<BACKEND>

  $<TARGET_EXISTS:tgt> expression requires a non-empty valid target name.

-- Generating done
CMake Generate step failed.  Build files cannot be regenerated correctly.

So I tried to replace <BACKEND> with sqlite3, so CMake is now:
cmake_minimum_required(VERSION 3.16)
project(Learning_VCPKG LANGUAGES CXX)

find_package(SOCI CONFIG REQUIRED)

if (SOCI_FOUND)
    message("Soci found: ${SOCI_INCLUDE_DIRS} and ${SOCI_LIBRARY_DIRS}")
endif ()

add_executable(${PROJECT_NAME} main.cpp)

target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_core>,SOCI::soci_core,SOCI::soci_core_static>)
target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_sqlite3,SOCI::soci_sqlite3,SOCI::soci_sqlite3_static>)

And compilation also failures:
cmake . -DCMAKE_TOOLCHAIN_FILE=/home/vcpkg/scripts/buildsystems/vcpkg.cmake && make
Soci found:  and
-- Configuring done
CMake Error at CMakeLists.txt:13 (target_link_libraries):
  Error evaluating generator expression:

    $<TARGET_EXISTS:SOCI::soci_sqlite3,SOCI::soci_sqlite3,SOCI::soci_sqlite3_static>

  $<TARGET_EXISTS> expression requires exactly one parameter.

CMake Error at CMakeLists.txt:13 (target_link_libraries):
  Error evaluating generator expression:

    $<TARGET_EXISTS:SOCI::soci_sqlite3,SOCI::soci_sqlite3,SOCI::soci_sqlite3_static>

  $<TARGET_EXISTS> expression requires exactly one parameter.

CMake Error at CMakeLists.txt:13 (target_link_libraries):
  Error evaluating generator expression:

    $<TARGET_EXISTS:SOCI::soci_sqlite3,SOCI::soci_sqlite3,SOCI::soci_sqlite3_static>

  $<TARGET_EXISTS> expression requires exactly one parameter.

-- Generating done
CMake Generate step failed.  Build files cannot be regenerated correctly.

-target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_sqlite3,SOCI::soci_sqlite3,SOCI::soci_sqlite3_static>)
+target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_sqlite3>,SOCI::soci_sqlite3,SOCI::soci_sqlite3_static>)

-target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_sqlite3,SOCI::soci_sqlite3,SOCI::soci_sqlite3_static>)
+target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_sqlite3>,SOCI::soci_sqlite3,SOCI::soci_sqlite3_static>)

Thanks, when I correct as You suggested CMakeLists.txt:
cmake_minimum_required(VERSION 3.16)
project(Learning_VCPKG LANGUAGES CXX)

find_package(SOCI CONFIG REQUIRED)

if (SOCI_FOUND)
    message("Soci found: ${SOCI_INCLUDE_DIRS} and ${SOCI_LIBRARY_DIRS}")
endif ()

add_executable(${PROJECT_NAME} main.cpp)

target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_core>,SOCI::soci_core,SOCI::soci_core_static>)
target_link_libraries(${PROJECT_NAME} PRIVATE $<IF:$<TARGET_EXISTS:SOCI::soci_sqlite3>,SOCI::soci_sqlite3,SOCI::soci_sqlite3_static>)

everything works:
cmake . -DCMAKE_TOOLCHAIN_FILE=/home/vcpkg/scripts/buildsystems/vcpkg.cmake && make
Soci found:  and
-- Configuring done
-- Generating done
-- Build files have been written to: /home
Consolidate compiler generated dependencies of target Learning_VCPKG
[ 50%] Linking CXX executable Learning_VCPKG
[100%] Built target Learning_VCPKG
root@agh-legion:/home# cmake . -DCMAKE_TOOLCHAIN_FILE=/home/vcpkg/scripts/buildsystems/vcpkg.cmake && make && ./Learning_VCPKG
Soci found:  and
-- Configuring done
-- Generating done
-- Build files have been written to: /home
Consolidate compiler generated dependencies of target Learning_VCPKG
[100%] Built target Learning_VCPKG
We have 3 entries in the Employees table.

But IMO suggestion from VCPKG needs modification, when I compare diff between suggestion and working version:

First occurrence of text <BACKEND should be replaced with sqlite3, but another two occurrences to replace are: <BACKEND> (extra >). IMO it should be the same text to replace in all three places.
Of course it would be much easier if suggestion will be just copy&paste without replacing <BACKEND>  keyword, but maybe it is impossible:)

I see, we lost a >. I want to avoid <...> for BACKEND, but I couldn't come up with an alternative fast enough. I avoided adding usage in the beginning because it wouldn't be trivial, but the uninformed heuristics is helpful either.


Added Members API

I need the members API to sync against external source - so I added that.

Thanks for the PR. Currently really busy with another project but I will merge it asap


[chore] Add a blog to Korean category blog section.

Add a blog name of "Evans Library"

Modified: blogs.json

Hi @devKobe24. Is this a Swift blog? I am relying entirely on Google Translate to check it, but I can't see any reference to Swift at all in the pages I checked. Could you give an example of some relevant content? It doesn't all have to be Swift-related, but just something.
Also, there is an additional unnecessary comma in the diff which is upsetting the validation.
Thanks!

HI @daveverwer, upon a detailed examination of this blog, it appears it is not solely a Swift blog. There are many other articles mixed in. Next time, I will ensure to understand the blog's precise intention before sending a PR.

It's fine if something is not ONLY a Swift blog, but I couldn't see any Swift at all there. If there is even a small amount then it qualifies.

Upon further review, it was found that the author of that blog is a JavaScript engineer and has not posted anything about Swift, hence it was removed. Additionally, Kobe's blog has been updated. Since all the Swift-related posts have disappeared from Kobe's blog, it has been removed as well for the time being


CONTRIBUTION GUIDE

I'll love to develop a contribution guide to guide the members to make efficient contribution

I think that has been, kindly check the Readme.md file


Library version info idea

I have a branch that adds a printdepversions that prints (if knowable) the header and binary versions for each of our external library dependencies. I don't think it's a good idea to merge it as-is though: it may be fragile to ensure that the #include always matches the file where the library is actually used, lest you get different results while having more than one version installed. Also including that many headers in one file is likely to result in macro conflicts and stuff.
So I have a new idea: store the version info in ROM cvars, e.g. version.sdl.header for the SDL header version and version.sdl.binary for the binary version. Then the cvar for a given library can be populated from the file where it is primarily used (even in gamelogic). Also this is more flexible w.r.t. stuff like alGetString(AL_VERSION) which actually requires a valid context to work, so it is not always available. To list all the versions you can do /listcvars version. or there can be a convenience command. What do you think?

I see in your branch you seem to be relying on defines. Does that still work at runtime or if the user replaces the linked so with another one, it would report the build time version?

That said, I think it's a good idea for debugging purposes.


Problem in preStoreUpdate when include async function

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[x] Support request
[ ] Other... Please describe:

Current behavior

When include async function in preStoreUpdate AuthGuard return false after refresh browser.
I would like refresh token after get him from storage, preStoreUpdate nice place for that, but when return fetch Promise with result AuthGuard return false, how to fix it?
sorry for google translate eng.
It will be nice if you correct me!
Expected behavior

Minimal reproduction of the problem with instructions
main.ts
preStoreUpdate(storeName: string, state: any) {
    if (storeName === 'auth' && !!state.token) {
      return fetch(`${window.location.protocol + environment.ApiHost}/refresh-token`, {
        method: 'POST',
        cache: 'no-cache',
        headers: {
          Authorization: `Bearer ${state.token}`
        },
        redirect: 'error',
        referrer: 'no-referrer'
      }).then(response => {
        const { token, exp_at }: any = response.json();
        return {
          ...state,
          token,
          exp_at,
        };
      }).catch(err => {
        return { token: null, exp_at: Date.now() };
      });
    }
    return state;
  },

auth.guard.ts
import { selectPersistStateInit } from '@datorama/akita';

@Injectable({
  providedIn: 'root'
})
export class AuthGuard {
  constructor(private router: Router, private authQuery: AuthQuery) {
  }

  canActivate(): Observable<boolean> {
    return combineLatest([
      this.authQuery.isLoggedIn$,
      selectPersistStateInit(),
    ]).pipe(
      map(([isAuth]) => { // <--- return false
        if(isAuth) {
          return true;
        }
        this.router.navigateByUrl('login');
        return false;
      }),
      take(1)
    );
  }
}

What is the motivation / use case for changing the behavior?

Environment

Angular version: 9.0.0

Browser:
- [x] Chrome (desktop) version 80
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [x] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: XX
- Platform:

Others:

Hooks doesn't support async operations.

Which storage do you use?

Which storage do you use?

Right now i use localStorage, but in future i planned use indexedDB

You shouldn't do this call in the preStorageUpdate. Do it in your AuthGuard

thanks, I'll try.


182

Resolves #182

Current coverage is 91.63%

Merging #190 into master will decrease coverage by -0.75% as of 99ecfa0

@@            master    #190   diff @@
======================================
  Files           12      26    +14
  Stmts          512     765   +253
  Branches       107     156    +49
  Methods
======================================
+ Hit            473     701   +228
- Partial         22      32    +10
- Missed          17      32    +15

Review entire Coverage Diff as of 99ecfa0

Powered by Codecov. Updated on successful CI builds.


Add back ports so docker-compose up succeeds

If you don't add jessie back ports mongodb-org-shell will not install due to the fact its looking for libssl1.0 which isn't installable.

All right will do I’ll try to get this done over the weekend

I have made the changes you requested.

Hi, I added trailing slashes like you wanted.

Merged. Thanks for the PR, @zitterbewegung!


The email format validation message is not displayed properly.

Description:
The email format validation message is not displayed properly.
-Env:
Crafter Exigo Blueprint
Device: MacBook Pro
Google Chrome Version 58.0.3029.110 (64-bit)
REPRODUCIBILITY LEVEL:
Always
STEPS:
Steps

Open the Crafter studio with the Exigo site.
Go to the “Contact us” section.
Fill Out the fields with the next text.
Your name: Juan
Phone: 123456
Your Email Address: juan.com
You Message: Test
Click on the “Send Message” button

CURRENT RESULT:
The email format validation message is not displayed properly.
https://www.screencast.com/t/PKnbnwNkn
EXPECTED RESULT:
The email format validation message should be displayed properly.

Fixed

Bug verified and closed as fixed in build 8380508.


Scrollbar is over header and not just body

mantine-react-table version
1.3.4
react & react-dom versions
v18.2.0
Describe the bug and the steps to reproduce it
Unless I'm missing something, there does not seem to be a way to make the scrollbar go over just the body and not the header.

Minimal, Reproducible Example - (Optional, but Recommended)
You can even see this happening at the table in https://www.mantine-react-table.com/docs/guides/virtualization#enable-row-virtualization-demo
Screenshots or Videos (Optional)
No response
Do you intend to try to help solve this bug with your own PR?
No, because I do not know how
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Hi, this is intended behavior, HTML tables are not really meant to have scrolling bodies but on the table as a whole. So the workaround is using position sticky on the header so it stays at the bottom while the rest of the table scrolls.


Added bundles as a separate plugin on parxer

Will leave here to not forget but DO NOT MERGE YET.
Depends on https://github.com/tes/parxer/pull/5/files and revert of https://github.com/tes/compoxure/commit/3b4dc7c904d5d1dedeedc61751dd0b1d445a4281

I've tested with linked latest parxer code and tests are back to passing so keep calm and code on.... at midnight....

Haha - I'm not sure why I put the bundles in the same plugin as the url one - so it's good work to do.

We are back to green. This is good to merge now

Manually merged.


Suggestion: Player Reports / Claims

So this may require some explanation, which I'll try to do to the best of my ability.
Player report ability will be critical so that players can report those breaking the rules to the admin team.
Another component to this is admin claiming and clearing of reports. One thing that is very much lacking is the ability to view any active reports, and claiming a report identifying that you are handling the admin sit.
Example: Player Bob reports Randy for RDM. There's a pending report yet to be handled by a staff member. Prior to a staff member teleporting to handle the sit, that staff member claims, whether via a button or automatically on teleport. Then that report turns into an "admin sit/in progress" in which no other staff is able to teleport to that incident.
Another phase of this would be an escalation during a sit, where say a "Moderator" lacks the permissions or cannot help a user that they showed up to a sit for, and needs to escalate to an "Admin" which allows a member of that role to teleport to the sit, thus bypassing the previous preventative.
Once done, any staff members involved in a said incident would write a brief note about what happened, and then close out the report. We require all of our staff here to file incident reports on all bans, for example.
This will go a long way, especially on bigger servers to preventing eight staff members from showing up to one incident, as then eight staff members are using an admin function, thus causing a drain of system resources to handle each of those requests. Something we struggle with as a bigger server that none of our admin or management panels is able to address.

Additionally from a management perspective, a log somewhere of who goes to sits, and how many that user does, would also be extremely helpful. Something that an individual of a specific permission level can pull up individual staff members and see similar how we have hours played how many sits they've completed. Perhaps even a potential situation for an API straight off to Discord to a channel id we specify that accumulates said log for all sits handled, and any completed notes for sits completed.

One thing to note about this function, there should be a mandatory cooldown that restricts a player from spamming a /report or /calladmin command. This doesn't entirely need to be a setting I don't think, but definitely should be put into place. I'd say probably a minute is a safe amount of time (cooldown) for an individual player to use the above commands.
Additional functionality, such as /report and under suggestion, shows something like /report {id} {reason} so that they know to report the id of a player, and the reason for the report.


WIP Adds a Length-Prefixed Wire Protocol

It currently still uses the BinaryFormatter to do the serialization, but it adds the length prefix.

Apparently when you force push it closes your PR?


uniform action behavior

actions and objects behave differently.
e.g., I can pick up food from the blender by F, but not from the cutboard, there it gets cut.
should be different actions to pick/place objects or to use a tool for example. currently the blender can't be actively used for example.

A new action EXECUTE_ACTION has been introduced. It allows an agent to execute the action of an ActionObject when facing in its direction, while standing in front of it. (Conditions of the ActionObject apply). At the same time the INTERACT_PRIMARY action does not execute the action of ActionObjects anymore.
The new action binding is the key 'e'


[RFC] introduce top level gb package

This issue proposed augmenting gb to produce, on demand a package, nominally "gb", which contains information about the build environment, vcs tags, etc
Background
There is a clear need to provide build info in Go programs. Currently this is done at link time with the -X flag to the linker.
I have thought to offer this for gb, with a set of default values which would always be applied during linking. That is, always on.
The response was luke warm. Especially as this would not solve the problem when gb itself was built with the go tool.
Agreeing on the names of build values would be provlematic. See the recent gb version discussion for background. #212 #447
Proposal
At compile time, gb itself will synthesise a package "gb" (just like import "C" or "unsafe") which will contain information about the build context and project.
Other packages can import this package (importing actually triggers the package to be synthesised) and inspect it's values, print then out,  whatever.
Pros

no on disk state
compile time constants vs link time variables
easily unit tested.

Cons

gb specific feature, would not compile with Go tool, but gb projects are not go gettable
gb package would end up a dumping ground of informational values
gb vendor information unavailable, would probably require a meta plugin proposal and more complicated arrangement.

Thoughts ?

I think this is brilliant, and would use this in every project.


Adds quiet success option

Adds an option to suppress the terminal output (standard out and success banner) when Flow reports no errors.

Thank you @farneman for this great PR!


Float / decimal / double values converted in comma-separated ones (instead of point-separated ones)

I've noticed that depending on the locale, these values are converted from float (PHP) to a string (may be a simple cast or whatever) relying on the default PHP behaviour. This means that if the current locale translates the decimal separator in a comma instead of a dot, this will translate in some troubles database-side (value = "1,23" instead of "1.23" means, at least in PHP, "1.00").
The easiest way is to use sprintf with %F as formatting string, so that the locale is being ignored.

I've done a workaround using setlocale(LC_NUMERIC, 'en_US') - but as previously written, it's a workaround. In my particular case it's not a problem, but some software may need to format decimals just the way the locale is set.


[REVIEW] My GitHub Profile

GitHub Profile Link
https://github.com/abhilipsasahoo03

Why linking on your GitHub when we are already on your GitHub profile? otherwise I really like your profile 🚀

Why linking on your GitHub when we are already on your GitHub profile? otherwise I really like your profile 🚀

Actually I thought of linking all of my socials on my profile, but now I realise linking GitHub is unnecessary. So, I've made the appropriate changes. Thanks!