HWP-405 Community page only display posts in that community

Description
This PR fixes an issue in CommunitiesPage.jsx where said page was displaying all posts, not just posts specific to that community.
This PR includes the following proposed change(s):

Modified PostsList.jsx to take in a posts array as props
Modified all instances of PostsList to pass in a posts array as props

Requires
Add 'Requires Attention' label if appropriate.

[ ] Requires an update to the .env file
[ ] Requires adding a file listed in the .gitignore
[ ] Requires an opinion or answer to a question, see comments.
[ ] Requires other attention, see below.

If there are any requirements for the developer to make after this PR is merged please list them here.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
For pull requests that are not just a simple fix, please check that the branch works on your local machine.

[x] Tested by Zach.
[x] Tested by Brandon.
[ ] Tested by Brady.

Does the change impact or break the Docker build?

[ ] Yes
[x] No

If Yes: Has Docker been updated accordingly?

[ ] Yes
[ ] No

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] New and existing unit tests pass locally with my changes
[ ] If attention is required by the developer such as updating the .env file, these requirements have been listed.

Tested.


[Prod] keycloak client to test in the interim

This action is NO LONGER NEEDED as we already have prod set up.


Allow adjusting padding spaces in the list markup

Actual Result
/* --- copy-selection-as-markdown debug information --- */ copy.js:54:14
/* --- INPUT ------------------------------------------ */ copy.js:55:14

Manage the life cycle of docker containers.

Supports check mode. Run with --check and --diff to view config difference and list of actions to be taken.
 copy.js:56:14
/* --- OUTPUT------------------------------------------ */ copy.js:57:14
-   Manage the life cycle of docker containers.
-   Supports check mode. Run with `--check` and `--diff` to view config difference and list of actions to be taken. copy.js:58:14
/* --- URL -------------------------------------------- */ copy.js:59:14
https://docs.ansible.com/ansible/latest/modules/docker_container_module.html#docker-container-module copy.js:60:14
/* ---------------------------------------------------- */

Expected Result
Allow customizing  the quantity of the padding spaces from three to one so the resulting output can be adjusted as the following:
- Manage the life cycle of docker containers.
- Supports check mode. Run with `--check` and `--diff` to view config difference and list of actions to be taken. copy.js:58:14

Thanks @Lin-Buo-Ren for the request. Such feature keeps popping up in my mind. Let me see how I can implement it.

Hey @Lin-Buo-Ren  I just released v0.17.0 which implements your request -- "Reduce padding spaces for list items." Grab the new from Add-on page at your Firefox setup. Open an issue if you find any problem. Thanks!


Add german translation

oh and it would be cool if in options.fastquit.renderSavingScreen the Text in would represent the german translation of the screen ingame ("Welt wird gespeichert")

Thanks for the correction, should be fine now.

will merge, thank you for the contribution! :)


TiledMap properties don't get html decoded

Issue details
Common xml characters (e.g. <>&") get encoded (e.g. &) inside a .tmx file.
Libgdx doesn't convert those values back to their original values and keeps those instead.
Reproduction steps/code

Create a new tilemap in tiled.
Create a new tileset.
Create a custom property on a tile in the tileset with key hi& and value Hello&<>"

<properties>
    <property name="hi&" value="Hello&<>""/>
</properties>

Parse that propery in libgdx:

for (int x = 0; x < layer.getWidth(); x++)
{
    for (int y = 0; y < layer.getHeight(); y++)
    {
        TiledMapTileLayer.Cell cell = layer.getCell(x, y);
        if (cell == null)
            continue;

        String prop = cell.getTile().getProperties().get("hi&", null, String.class); // never triggers
        if (prop != null)
        {
            System.out.println(prop);
            return;
        }

        prop = cell.getTile().getProperties().get("hi&", null, String.class); // triggers
        if (prop != null)
        {
            System.out.println(prop); // Prints Hello&<>"
            return;
        }
    }
}

Expected
The key hi& should exist, and it's value should be Hello&<>"
Actually
The key hi& exists and it's value is Hello&<>"
Version of LibGDX and/or relevant dependencies
1.9.14
Please select the affected platforms

[x] Android
[x] iOS
[x] HTML/GWT
[x] Windows
[x] Linux
[x] MacOS

Seems to be a bug in the xml parser.

i think it's a duplicate of #4831 or at least realted.

If you need to wrangle the .rl files used to generate XmlReader.java, you can get a working version of Ragel here https://github.com/sci-bots/ragel/releases/tag/ragel-6.10 . Some @Null annotations have been added to the current Java code, so you'd probably need to track where those were added manually to the generated file. There may be other changes, I don't know.

i'm closing since it's a duplicate.


Migrated Newtonsoft.Json to System.Text.Json

In Net Core 3.0 the new System.Text.Json types were introduced.
According to
https://docs.microsoft.com/en-us/dotnet/core/whats-new/dotnet-core-3-0#fast-built-in-json-support
it should have some speed and allocation advantages
Most migrations are pretty straight forward and should create no problems, the only change I wasn't 100 % sure about is in LoggingService

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Join my Discord Guild so we can chat about this real quick. https://discord.gg/H8bcFr2
I just have a few questions about this, and I'll likely merge it if all my questions are answered!


HIL(QOL): Add missing timeouts to various tests

I'd prefer investing into https://github.com/probe-rs/embedded-test/issues/3 instead.

I guess https://github.com/probe-rs/embedded-test/issues/3 is the best way to go (is no timeout at all ever a good idea ....?)
So, what should happen with this PR? Does it solve a real problem we have for currently? (Then we might want to get it in until we have a global timeout)

(is no timeout at all ever a good idea ....?)

The default is 30 seconds right now, there is no such thing as no timeout in embedded-test.

(is no timeout at all ever a good idea ....?)

The default is 30 seconds right now, there is no such thing as no timeout in embedded-test.

Oh good - I never had the patience to wait 30 secs  .... I don't see where the code sets 30 secs if None but I didn't try hard to find it

My bad it's 60

My bad it's 60

Ah thanks .... the runner does it on its own .... no wonder I wasn't able to find it in embedded-test itself 😆

PR pending: https://github.com/probe-rs/embedded-test/pull/44

PR pending: https://github.com/probe-rs/embedded-test/pull/44

Thank you! I will mark this as a draft and when it gets merged, I can update our HIL.

Thank you! I will mark this as a draft and when it gets merged, I can update our HIL.

You can use Daniel's branch with git and rev in hil-test for now


[flutter_plugin_tools] Add a bulk version update command

We sometimes make code changes across the repository, for example when adding a new analysis option. When we make production code changes we want to do releases, but it's currently very tedious to update the version across all the changed packages, and add identical entries to all of their changelogs.
We should make a tool command that takes a string and for every target package:

Does a bugfix bump (since anything cross-package should be minor, not a new feature)
Updates the changelog to have a version bump with that string as the change description.

This needs to include handling NEXT if it's there.

Done in https://github.com/flutter/plugins/pull/5643


User Impersonation Problem

Describe the bug
I want to run Powershell scripts from asp.net core, that is hosted as Windows Service. With Windows Authentication
I did it almost, but if i run "whoami" i get the UserName who run the service not the authenticated User.
To Reproduce
see uploaded File
WindowsAuthenticationProblem.zip
deploy it, and start it as Windows Service
call http://localhost:8888
Actually behavior
If i run it now, i get:
Without	'WindowsIdentity.RunImpersonated'	With	Current Authenticated User	{User: ##The Authenticated User##}
Without	'WindowsIdentity.RunImpersonated'	With	Current Windows Identity	{User: NT AUTHORITY \ SYSTEM}
Without	'WindowsIdentity.RunImpersonated'	With	PowerShell command whoami	{User: NT AUTHORITY \ SYSTEM}
With	'WindowsIdentity.RunImpersonated'	With	Current Authenticated User	{User: ##The Authenticated User##}
With	'WindowsIdentity.RunImpersonated'	With	Current Windows Identity	{User: ##The Authenticated User##}
With	'WindowsIdentity.RunImpersonated'	With	PowerShell command whoami	{User: NT AUTHORITY \ SYSTEM} ####Here, the authenticated username should be printed###
Expected behavior
The auhtenticated UserName
"Sorry for my bad English."

Hi. It looks like this is a question about how to use ASP.NET Core. While we do our best to look through all the issues filed here, to get a faster response we suggest posting your questions to StackOverflow using the asp.net-core-mvc tag.


DX: make integration tests matching entries in FixerFactoryTest

Do not allow file names with the swapped order.
There are 203 files with names having the order matching the priority and 11 files (updated in this PR) having the order swapped.

Coverage remained the same at 91.849% when pulling a2ecc1ed52386195719665e2b8c1b65ad068ecae on kubawerlos:dx_integration_tests_file_names into c40975866e753a724377af50de9148a36be9179e on FriendsOfPHP:2.18.

Thank you @kubawerlos.


Replaced vulnerable functions and outdated dependencies

Potential vulnerability risks were detected in your dependencies and used functions.
Some vulnerabilities have been replaced by safe alternatives.
Vulnerable Functions
setup.py:26:2: exec

Reason: Could execute vulnerable and untrusted code.
Severity: warning

note_sequence_io.py:39:25: hashlib.sha1

Reason: Attacks can find collisions in the full version of SHA-1.
Replacement: hashlib.sha512()
Severity: critical

lib_hparams.py:168:18: yaml.load

Reason: Untrusted input can result in arbitrary code execution.
Replacement: yaml.safe_load(___0)
Severity: critical

Vulnerable Dependencies
Some versions of dependencies used in the project might pose security threads. Please make sure to inform users to use safe versions.

Dependency
Vulnerable Versions
Reason

requests
<2.3.0
requests before 2.3.0 exposes Authorization or Proxy-Authorization headers on redirect.  Fix CVE-2014-1829 and CVE-2014-1830 respectively

requests
<2.6.0
requests 2.6.0 fixes handling of cookies on redirect. Previously a cookie without a host value set would use the hostname for the redirected URL exposing requests users to session fixation attacks and potentially cookie stealing.

requests
>=2.1,<=2.5.3
The resolve_redirects function in sessions.py in requests 2.1.0 through 2.5.3 allows remote attackers to conduct session fixation attacks via a cookie without a host value in a redirect.

setuptools
<0.9.5
setuptools 0.9.5 fixes a security vulnerability in SSL certificate validation.

setuptools
<1.3
setuptools before 1.3 has a security vulnerability in SSL match_hostname check as reported in Python 17997.

setuptools
<3.0
setuptools 3.0 avoids the potential security vulnerabilities presented by use of tar archives in ez_setup.py. It also leverages the security features added to ZipFile.extract in Python 2.7.4.

urllib
>0,<0
urllib is a package affected by pytosquatting: http://www.nbu.gov.sk/skcsirt-sa-20170909-pypi/

Source: Safety
Test Report
No tests found or tests could not be executed

This tool was developed as part of a Software Engineering course. The intention is to make project maintainers aware of potential vulnerabilities. If you have feedback then please reply to this pull-request. Thank you!

I signed it!


Codify the methods and their arguments and result types

This crate could provide a minimal encoding of the methods and notifications in the protocol.
pub trait Request {
    type Param;
    type Result;
    // Has to be a method as associated constants are unstable
    fn method() -> &'static str;
}

pub trait Notification {
    type Param;
    fn method() -> &'static str;
}

https://github.com/Microsoft/language-server-protocol/blob/master/protocol.md#showmessage-request
pub struct ShowMessageRequest;
impl Request for ShowMessageRequest {
    type Param = ShowMessageRequestParams;
    type Result = MessageActionItem;
    fn method() -> &'static str { "window/showMessageRequest" }
}

https://github.com/Microsoft/language-server-protocol/blob/master/protocol.md#showmessage-notification
pub struct ShowMessageNotification;
impl Notification for ShowMessageNotification {
    type Param = ShowMessageParams;
    fn method() -> &'static str { "window/showMessage" }
}

I probably won't implement this myself right now as I already have the methods I need working.
cc @nrc Would this be useful for rls?

That'd be great! However it'd be very convenient to have same kind of structs for requests/notifications as there are for params wrt (de)serialization. It'd make the communication a lot more straightforward and easier to use. While I think exposing this as traits is a great idea, how would that mesh together with the struct (de)serialization?

I'll try and have a go at it in spare time and see how it goes.

@Xanewok Definitely add a pair of macros to avoid some of the repeition as well decl_request decl_notification or something.

While I think exposing this as traits is a great idea, how would that mesh together with the struct (de)serialization?

What do you mean?

That'd be great! However it'd be very convenient to have same kind of structs for requests/notifications as there are for params wrt (de)serialization.

The structs set as Param and Result are still the same types as they always are. This just gives a way to abstract over requests and notifications.
I was thinking of adding something like this for my own implementation of a language server but this issue is intentionally made to be unaware of how it is actually used.
struct RequestImpl<Func, Tag : Request> {
     handler: Func,
    _marker: PhantomData<Tag>
}

impl RequestImpl<Func, Tag>
where Func: FnMut(Tag::Param) -> Result<Tag::Result, Error>
    Tag: Request
{
     fn handle(...) { ... }
}

There is also https://github.com/RustDT/RustLSP, which I think does something similar to this.

I think the general idea here is good. It would be great to be more disciplined about how the RLS deals with messages and such in the protocol. If we could type check more stuff, that would be even better. I think it is kind of complex to get a good design here. It would be worth experimenting with a language server (like the RLS) to make sure any design here is ergonomic to use and efficient (i.e., doesn't have tonnes of virtual dispatch). Currently the RLS uses a pretty complex system of macros, and I don't think it is very readable, although it is concise.

However it'd be very convenient to have same kind of structs for requests/notifications as there are for params wrt (de)serialization.

Do you mean that each impl of say Request should be implemented on the parameters of the method? That is fine where it is possible, but there are some methods which do not take a unique parameter https://github.com/Microsoft/language-server-protocol/blob/master/protocol.md#completion-request https://github.com/Microsoft/language-server-protocol/blob/master/protocol.md#goto-definition-request.

There is also https://github.com/RustDT/RustLSP, which I think does something similar to this.

RustLSP has a big trait specifying each method https://github.com/RustDT/RustLSP/blob/e39ed06b14d711edfa912ce5e24e73ca0921f848/src/lsp.rs#L117-L152. Personally I feel like this trait does a bit to much. It assumes all methods can be implemented and only offers one way of implementing the RPC.
The proposed way to do it here should (as much as possible) be agnostic to how RPC are implemented.

I think it is kind of complex to get a good design here.

Yep, which is why I think we should only link method name to its parameters and return type. How method calls actually are implemented should be implemented outside this crate.

There is also https://github.com/RustDT/RustLSP, which I think does something similar to this.

Yes, unfortunately I wasn't able to update it to version 3.0 of LSP (started a full-time job in the meanwhile, I don't think I will be able to work on it anymore)

RustLSP has a big trait specifying each method https://github.com/RustDT/RustLSP/blob/e39ed06b14d711edfa912ce5e24e73ca0921f848/src/lsp.rs#L117-L152. Personally I feel like this trait does a bit to much. It assumes all methods can be implemented and only offers one way of implementing the RPC.

If an LS can't effectively process the request method, it can just implement the trait function by completing the response with an error saying the request is not supported, so I don't see this as an issue. (see for example: https://github.com/RustDT/MockLS/blob/master/src/mock_ls.rs)
Also, what do you mean "only offers one way of implementing the RPC"? You can have different trait implementations... I guess you can't control how the Params object gets deserialized though. That might be a minor performance limitation?

If an LS can't effectively process the request method, it can just implement the trait function by completing the response with an error saying the request is not supported, so I don't see this as an issue. (see for example: https://github.com/RustDT/MockLS/blob/master/src/mock_ls.rs)

Sure, but by not using a big trait we don't even have to add those dummy methods. I just wanted this as bare bones as possible to avoid encoding any sort of assumption about how requests are handled.

Also, what do you mean "only offers one way of implementing the RPC"? You can have different trait implementations... I guess you can't control how the Params object gets deserialized though. That might be a minor performance limitation?

There isn't an easy way to handle requests with https://github.com/alexcrichton/futures-rs for instance. I don't doubt it is possible, but bending that abstraction to accommodate futures is less ideal that just having a direction abstraction on top futures (and if one does not want to use futures then another abstraction should work equally well).

I think it's more important to provide efficient and ergonomic data structures and related type logic here wrt protocol, while leaving more freedom for the consumer how exactly message/error handling will be implemented using the types in this crate.
I tried to extend @Marwes's solution a bit: https://gist.github.com/Xanewok/5b062485a98c1312481e1790772ae477
Thoughts/feedback on this? This is only a general direction in which it could go.
With this I'd say it's quite ergonomic and separates handling logic. If we're going with static dispatch approach, we could later introduce a union type for message types (like RLS does), which would probably allow for straightforward deserialization.

@Xanewok
fn id(&self) -> &ls_types::NumberOrString;
fn params(&self) -> &Self::Param;

These methods conceptually exists for all requests and are also the same for any json rpc protocol so they aren't unique to the language server. For instance, I use the core crate from https://github.com/paritytech/jsonrpc to receive forward requests to the correct callback which means I don't need to bother with the object containing the id field at all.
Tweaking your example a bit https://gist.github.com/Marwes/93a5d9530226e5404493000d653db979 to remove a bit of boilerplate.

With this I'd say it's quite ergonomic and separates handling logic. If we're going with static dispatch approach, we could later introduce a union type for message types (like RLS does), which would probably allow for straightforward deserialization.

Is there any benefit to introducing an enum as an intermediate stage?

Oh, I see it now, I'm sorry. Wasn't aware of the core crate you mentioned and I thought the end goal would be to implement all the JSON-RPC types LSP uses here...
Looking at what this crate provides, the only thing that would be to add here is to specify which exacatly methods/notifications are supported via the JSON-RPC and LSP (ShowMessage, Completion etc...) and for those, binding params type to the result type? (So traits like https://paritytech.github.io/jsonrpc/jsonrpc_core/trait.RpcMethodSimple.html, but parametrized with a similar Trait you showed previously with associated Params and Result types?)

Has to be a method as associated constants are unstable

I would just use an assoc const - stable is overrated

but there are some methods which do not take a unique parameter

Could we just use unions or enums for those?

Oh, I see it now, I'm sorry. Wasn't aware of the core crate you mentioned and I thought the end goal would be to implement all the JSON-RPC types LSP uses here...

:) I could have been clearer. But yeah, I think the JSON-RPC part can be implemented separately as there both other protocols using JSON-RPC and multiple approaches of handling requests.

(So traits like https://paritytech.github.io/jsonrpc/jsonrpc_core/trait.RpcMethodSimple.html, but parametrized with a similar Trait you showed previously with associated Params and Result types?)

I guess that is what I really meant, though as you can see, even that crate has two traits defining an RPC method https://paritytech.github.io/jsonrpc/jsonrpc_core/trait.RpcMethodSync.html (Future vs Result) and I thought it best to leave that choice out of this library.
I do have a trait like that in gluon's language server https://github.com/gluon-lang/gluon_language-server/blob/23a3eb23e5434bc78567804bfc6ea54d7b761aac/src/rpc.rs#L29-L37. But each method I define using it has the method name, parameter and return type defined in an ad-hoc way which I'd rather avoid. I know that my initial proposal will work for what I had in mind at least but perhaps there is a more general way of doing it.
I am not against defining something larger than my initial proposal, I just figured it would be difficult to agree on a design which works everwhere.

I would just use an assoc const - stable is overrated

Maybe both, and hide the associated constant behind a feature? If a macro is used to define these things there shouldn't be much extra work.
Footnote
I want to turn this https://github.com/gluon-lang/gluon_language-server/blob/23a3eb23e5434bc78567804bfc6ea54d7b761aac/src/rpc.rs#L75-L118 into something like.
impl<P, T> RpcMethodSimple for ServerCommand<T>
where
    T: Request,
    P: FnMut(T::Param) -> BoxFuture<T::Result, Error>
{
    fn call(&self, param: Params) -> BoxFuture<Value, Error> {
        let value = match param {
            Params::Map(map) => Value::Object(map),
            Params::Array(arr) => Value::Array(arr),
            Params::None => Value::Null,
        };
        let err = match from_value(value.clone()) {
            Ok(value) => {
                return (self.handler)(value)
            }
            Err(err) => err,
        };
        let data = self.0.invalid_params();
        futures::failed(Error {
            code: ErrorCode::InvalidParams,
            message: format!("Invalid params: {}", err),
            data: data.as_ref()
                .map(|v| to_value(v).expect("error data could not be serialized")),
        }).boxed()
    }
}

fn register_command<T>(command: T) where T: Request + RpcMethodSimple {
    // extract the method name with T::method() to register a handler
}

fn main() {
     let command = ServerCommand::<CompletionRequest, >::new(
        // param is now inferred to be `<CompletionRequest as Request>::Param`
        // and the result type to be `<CompletionRequest as Request>::Result`
        |param| ...
    );

     register_command(command);
}


feat(v2): add a banner that links to latest version of documentation

Motivation
Add a badge that links older versions of documentation to the latest version of the documentation. This PR closes #2432.
Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan

After visiting the netlify preview site, go to any older version of the documentation . Check that there is a badge that links to the latest version

Click on the link to the latest version. There should be no badge next to the version number, since we are now at the latest version

The automated tests has been updated accordingly.

Hey @teikjun
I have this other PR with some changes to metadata.ts, just wanted to mention that so that we can avoid too much conflicts :)
https://github.com/facebook/docusaurus/pull/2905

Docs plugin: createDocsBaseMetadata(version) allows to pass data to the DocPage component, which is the layout component for a whole doc version.
I think we should add data here with a permalink to the latest docs homepage (not necessarily /docs, see routeBasePath)
But this is likely to conflict with my existing PR. Can you wait before merging this?

Thanks a lot for the reviews! We'll make a POC following @lex111's solution, and wait for #2905 before merging.

The changes are:

Changing the small badge to a banner.
Replacing isOld with latestPermalink. latestPermalink will always link to the latest docs homepage.

Considerations
Regarding point 2, we tried linking to the same doc in the latest version. But we couldn't find a good way to handle pages that have been renamed/deleted.
From our understanding, this requires looking up the list of currently available pages (so that we can link to homepage if the page does not exist). Hence, it would not be efficient.

Summary of changes:

Linked to first doc of sidebar instead of doc homepage
Changed banner color to warning orange

Hey,
I'd be fine if you fix the homeDocId problem to merge this PR.

Note, in the future it's probably needed we'll actually need to link to the same doc of the latest version, with a fallback to homeDocId, then firstDoc (of latest version)
This kind of behavior keeps being asked (yesterday here: https://github.com/facebook/docusaurus/issues/2923)
I think we'd need to provide some algo like
function getAllLinks(sidebarItems: DocsSidebarItem[]): SidebarItemLink[] {
 return flatten(sidebarItems.map(item => {
   if (sidebarItem.type === 'category') {
     return getAllLinks(sidebarItem.items);
   } else {
     return [sidebarItem.href];
   }
 }));
}

function getLatestDocsPermalink(links: SidebarItemLink[],preferedPaths: string[]): string | undefined {
 const existingPreferedPath = preferedPaths.find(preferedPath => {
   return links.some(link => link.href === preferedPath);
 })
 if ( existingPreferedPath ) {
   return existingPreferedPath;
 }
 else {
      return links[0]?.href;
 }
}

const docsLatestPermalink = getLatestDocsPermalink(latestSidebarData, [
  currentDocPath,
  homeDocIdPath
]);

This is really pseudo code, maybe you'll find it useful

Didn't try, but this could work fine:
      const latestHomeDoc = options.homePageId ? content.docsMetadata[options.homePageId] : undefined;
      let latestDocPermalink: string
      if ( latestHomeDoc ) {
        latestDocPermalink = latestHomeDoc.permalink;
      }
      else {
        latestDocPermalink = getHrefFromSideBar(
          content.docsSidebars[`version-${versioning.latestVersion}/docs`],
        );
      }

Actually, there's a case where this fails, if the homeDocId selected does not exist in latest docs version.
As the "home" concept is currently not versioned, it is possible that the user selects a home that exists in next (ie /docs/next path will be created), but does not exist in current docs version (ie /docs will not be created)
Test with:
          homePageId: 'doesNotExist',

I have a fix, I will push changes

LGTM, merging as #2905 is not reviewed yet

Released in https://github.com/facebook/docusaurus/releases/tag/v2.0.0-alpha.58


Fixed Responsiveness for Hero and Conversation

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fixes for Hero and Conversation component.

Why was this change needed? (You can also link to an open issue here)
The UI was buggy when the sidebar closes for mobile screens.
https://github.com/arc53/DocsGPT/issues/1192

Other information:

Hi @ManishMadan2882

Yes, I think the Height issue is fixed on the official website,
About the tagline I think it looks better in mobile also, if done correctly.
Lastly, When sidebar closes this is how it looks, which i think is a little off.

Ultimately, Its your repo your rules. 😊
Official:

From my branch:

Hi @Vaibhav91one
Agree with the 3rd point, but we need to make sure that it doesn't affect the desktop version.
It looks like a issue in Tablet screens, also the tagline will be good to have in tab.
Thanks!

Hi @Vaibhav91one Agree with the 3rd point, but we need to make sure that it doesn't affect the desktop version. It looks like a issue in Tablet screens, also the tagline will be good to have in tab. Thanks!

HI @ManishMadan2882, So, the changes that will be done are done in previous PR in hero and conversation components. I will submit a new PR with the updated branch. Works?

Absolutely @Vaibhav91one .

Hey @Vaibhav91one , there  is also a chance we would request you to fix as it aligns with the query bar issue.

Currently when we close the side bar and ask a query the query bar and the generated text arent aligned as shown in the picture.
So we need to fix this and make sure they are aligned.

Hey @Vaibhav91one , there is also a chance we would request you to fix as it aligns with the query bar issue.

Currently when we close the side bar and ask a query the query bar and the generated text arent aligned as shown in the picture. So we need to fix this and make sure they are aligned.

@shelar1423 , You mean the Answer and the ouput right?

I think it looks like after sidebar is closed, the input box is not really centered, and that makes it mismatched with the output. Basically all things need to be aligned well here

Please find the below PR for the changes.
Pull Request


Error when resizing PackedByteArray bigger than int32 limit

Godot version
4.0.3
System information
Windows 11 64-Bit
Issue description
The PackedByteArray can't be bigger than<PHONE_NUMBER>. Following this issue it should be fixed in Godot 4.0 RC 1, but running the code below in Godot 4.0.3 results in this error.
E 0:00:00:0834   Node.gd:9 @ _ready(): Condition "p_size < 0" is true. Returning: ERR_INVALID_PARAMETER
  <C++-Quelle>   ./core/templates/cowdata.h:262 @ resize()
  <Stacktrace>   Node.gd:9 @ _ready()

Steps to reproduce
Run this code:
	var INT32_MAX =<PHONE_NUMBER> # 2^31 - 1
	var byte_arr = PackedByteArray()
	byte_arr.resize(INT32_MAX) # OK
	byte_arr.resize(INT32_MAX + 1) # error

Minimal reproduction project
PackedByteArrayError.zip

Related to https://github.com/godotengine/godot/issues/62585, https://github.com/godotengine/godot/issues/45296 and https://github.com/godotengine/godot/issues/54679.
Making CowData use 64-bit unsigned integers is welcome, but it's a significant undertaking as it requires replacing the type in all places where relevant.


Update design system team page

Add Sara and remove Stephen

You can preview this change here:
Built with commit ed115348d0adc40e102a3d752bbfcb3ebcba9ec0
https://deploy-preview-1525--govuk-design-system-preview.netlify.app


Improve BinaryFieldMapper.CustomBinaryDocValuesField.bytes()

Falling out of #6884 this issue is to explore improving (and benchmarking) the BinaryFieldMapper.CustomBinaryDocValuesField.bytes() method to prevent sorting and deduping on every call. Options include switch from an ArrayList<byte[]> to a TreeSet<byte[]> or using sort/dedup after every add.

@navneet1v can I assign this one to you so you can own this exploration either here or upstream in Lucene?

@navneet1v can I assign this one to you so you can own this exploration either here or upstream in Lucene? No rush... and I'd look at this as a follow on to just getting MultiValue in Lucene. I don't expect any performance improvements whether it's a TreeSet or something else to be terribly impactful but every little bit helps.

Yes sure.

Excellent!! Thank you!

I chatted with @navneet1v.  I will be picking up this issue.

I chatted with @navneet1v. I will be picking up this issue.

Thanks @kkmr for picking this up.


release cached transaction for BASE transaction type in proxy

BASE transaction should not hold connections in transaction

Pull Request Test Coverage Report for Build 2233

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 58.147%

Totals

Change from base Build 1168:
0.0%

Covered Lines:
11890

Relevant Lines:
20448

💛  - Coveralls


Add Raw JSON Reader (~2.5x faster)

Which issue does this PR close?

Closes #3441
Rationale for this change

This adds a new JSON reader that reads directly into arrow arrays, this leads to non-trivial performance improvements vs the current serde_json::Value approach, whilst also I think making the logic for handling nested schema easier to follow.
What changes are included in this PR?

Are there any user-facing changes?

My basic plan is to do something similar to https://github.com/simdjson/simdjson/blob/master/doc/tape.md . There is a Rust implementation of simd-json but it is a bit heavy-weight for our needs, and is a fairly substantial dependency, so I'd like to try doing something simpler :sweat_smile:
Not sure when exactly I'll get around to doing this, perhaps in a week's time

@tustvold I saw this coming by https://github.com/PSeitz/serde_json_borrow we might take some inspiration from there or potentially use that crate?

Cool, happy to learn about the results / do a review!

It needs some more cleanup, and I've not really spent any time trying to optimize it, but getting a more respectable 2.5x performance improvement with the new approach, albeit for a fair amount of additional code complexity...
large_bench_primitive (basic)
                        time:   [1.5475 ms 1.5488 ms 1.5501 ms]

large_bench_primitive (raw)
                        time:   [660.08 µs 660.37 µs 660.67 µs]

I am happy that this is now ready for review, whilst it is a fair amount of complexity, the 2.5x performance improvement I think justifies it.
Furthermore, for async workloads this will be even more pronounced, as it avoids having to perform a pre-parse to delimit newlines. Data can be directly streamed from object storage, and fed into RawDecoder as it arrives.

Writing some integration tests, comparing RawReader with Reader that is interesting is that Reader will:

Interpret non-string payloads as strings, e.g. {"string": false}\n{"string": "foo"} can be parsed into a StringArray
Convert scalars into lists, e.g. {"list": 2}\n {"list": [2, 1]} can be parsed into a ListArray

I'm not sure if this is a behaviour we wish to replicate, I at least found it very surprising
Thoughts @nevi-me @alamb ?

I broke something, fixing it :smile:

This is exactly what I intend to do 👍 As the current one exposes serde_json::Value it will be a breaking change, so I want to do it slowly, but I agree that maintain two readers is not a good idea long-term

It is probably good to file a ticket with this overall plan to make it clearer -- I can do so if you would like


Set up a documentation site

This will probably be in gitbooks or something

https://github.com/mso-net/lucee-aws-docs
http://mso.gitbooks.io/lucee-aws/content/


Context distance when training models of 3-views version

Dear devs,
Thanks a lot for your contribution. I would like to kindly ask how the min and max context distances are set when training the 3-view version of the model. Thank you very much for your help!

Two context views are chosen normally, and a third is then chosen randomly between them. See here:
https://github.com/dcharatan/pixelsplat/blob/5d5735eaeea59b67eb9527192bcde3aa20da0342/src/dataset/view_sampler/view_sampler_bounded.py#L109-L121


It looks like Stripe.js was loaded more than one time. Please only load it once per page.

When using in a SPA every time you navigate to the page with the component it loads stripe.js

Can you simply put an id on the script tag and look for the script before appending it?

P.S. if I can get to it today I will try to submit a PR

Hello @MohawkMan let me know if you come up with a solution. Otherwise I'll add this to my to do list. Thank you!


Feature Request: PixiJS v8 Support

With the new massive rewrite of PixiJS 8, it seems a good amount of work will need to be done to support the new paradigm. Is this project still maintained, and if so, what would it take for v8 support? I might be interested in collaborating on this if there is interest.

I have too much other stuff going on to do that I'm afraid. Also, I don't really see the need of PixiJS 8 I general, why would I go with that instead of v7? (At least in the areas I'm working with). I guess it's to support WebGPU, but not sure if there is anything else?

Mostly because v8 is a complete rewrite  and they likely won't do any further major work on v7, so it will eventually code rot. There are a lot of optimizations in the new code and API. WebGPU is interesting because it offers some more possibilities. I also work with @pixi/react and that's being moved with a complete rewrite from v7, so I'll eventually have to make the transition myself. I get being busy though. I'm still working through a bunch of upgrades myself, but I was curious if there was interest here.

Yes, I will definitely use v8 later. But right now v8 doesn’t really offer any new features (as far as I know) compared to v7. I’ll stick with v7 for a while more for the production apps. WebGPU works in all major browsers nowadays?

I'm not entirely certain on the browser support front, but one thing I am looking into is React Native's WebGPU rendering layer, which would allow me to stop needing to use a WebView with RN once I get it working.

PIXI v8 is a lot faster than v7 (and not only with WebGPU, also with the WebGL renderer). I guess this is the main reason why one would want to use it over v7. Especially when doing 3D, which is presumably more resource intensive than 2D in general, a faster renderer seems like a good argument to upgrade.


Better Hyperparameter Search

Is your feature request related to a problem? Please describe.
Currently the number of components used to determine book embedding is 200. Since it seemed to work and the search takes a lot of time for us locally, We fixed the hyperparameter to that.
Describe the solution you'd like
Does a better hyperaparameter exist? Can we do a Randomized Search or a Grid Search to determine that, maybe even tune other parameters? Even after finding a better one with a lesser loss, does it produce a better output when the similarity metric is used?
Additional context
Refer the Embedding Notebook in The Notebook & Files Section for code. You can either Integrate Surprise with ScikitLearn Grid Search or use another  package to determine embedding and do they perform well

Can I work on this issue ?

can you assign this issue to me?

@micksharma as you are assigned #35
@subhangi2731 you can go ahead with this issue.

@subhangi2731 Could you open a draft PR with your present progress, so we know you're going in the right direction

sure I will @Praful932 I am working on this but I had a doubt where can I reach U?

Hi @subhangi2731 you can either reach out on GSSOC Discord/start a new discussion here

@subhangi2731 any updates?

Hello I was out of station I was not able to communicate and reach out to u can u please reassign me I will send the pr by today

@subhangi2731 any updates?

@Praful932 I would like to work on this issue under GSSOC'21

@Aditi0102 Go ahead :rocket: . check Starter Guide it will help.

@Aditi0102 any updates?

@maneprajakta I'll complete in a day or two.

@Aditi0102 You can open a WIP PR(does not need to be complete), so we can track the progress :)

@Aditi0102 ?

hey @Praful932
Can I work on this issue?

Sure go ahead

Sure go ahead

Thank you

@Kavs-123 any updates?

hey @maneprajakta
I am not well, therefore,
I won't be able to work on this issue!
I am extremely sorry for bothering you!

@Kavs-123 No issues.
Take care :)

Closing as not a priority right now


[Bug] Custom css not loading from save in detached window

Describe the Bug
The custom css window shows up empty (doesnt load the saved css).
To Reproduce

Save any custom css
Set the Custom CSS Editor Location to Detached Window
Open the Custom CSS editor

Expected Behavior
It should load.
Screenshots

Discord Version
Stable 81171 (2a43a4e)
BD 1.0.0 (stable)
Host 0.0.14
Injector 0.6.0
Linux 64-Bit (5.10.0-5-Amd64)

Enable live update or save your changes before detaching.

Fixed in this commit https://github.com/rauenzi/BetterDiscordApp/commit/a2238060607a98720b761672f603bbff4a40f99b
It will go live in the next update


deprecate DSA campaigns

Docs Build status updates of commit 1b48213:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Advertising/bingads-13/bulk-service/account-filter-link-ad-extension.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/ad-group-dynamic-search-ad-target.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/ad-group-filter-link-ad-extension.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/ad-group-negative-dynamic-search-ad-target.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/bulk-file-schema.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/campaign-filter-link-ad-extension.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/campaign-negative-dynamic-search-ad-target.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/campaign.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/dynamic-search-ad-label.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/dynamic-search-ad.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/filter-link-ad-extension.md
:white_check_mark:Succeeded

Advertising/bingads-13/campaign-management-service/campaign.md
:white_check_mark:Succeeded

Advertising/bingads-13/campaign-management-service/dynamicsearchad.md
:white_check_mark:Succeeded

Advertising/bingads-13/campaign-management-service/dynamicsearchadssetting.md
:white_check_mark:Succeeded

Advertising/bingads-13/campaign-management-service/filterlinkadextension.md
:white_check_mark:Succeeded

Advertising/bingads-13/campaign-management-service/googleimportoption.md
:white_check_mark:Succeeded

Advertising/bingads-13/campaign-management-service/webpage.md
:white_check_mark:Succeeded

Advertising/bingads-13/customer-management-service/getaccountpilotfeatures.md
:white_check_mark:Succeeded

Advertising/bingads-13/customer-management-service/getcustomerpilotfeatures.md
:white_check_mark:Succeeded

Advertising/bingads-13/guides/mixed-campaigns.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 1b48213:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Advertising/bingads-13/bulk-service/account-filter-link-ad-extension.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/ad-group-dynamic-search-ad-target.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/ad-group-filter-link-ad-extension.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/ad-group-negative-dynamic-search-ad-target.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/bulk-file-schema.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/campaign-filter-link-ad-extension.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/campaign-negative-dynamic-search-ad-target.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/campaign.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/dynamic-search-ad-label.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/dynamic-search-ad.md
:white_check_mark:Succeeded

Advertising/bingads-13/bulk-service/filter-link-ad-extension.md
:white_check_mark:Succeeded

Advertising/bingads-13/campaign-management-service/campaign.md
:white_check_mark:Succeeded

Advertising/bingads-13/campaign-management-service/dynamicsearchad.md
:white_check_mark:Succeeded

Advertising/bingads-13/campaign-management-service/dynamicsearchadssetting.md
:white_check_mark:Succeeded

Advertising/bingads-13/campaign-management-service/filterlinkadextension.md
:white_check_mark:Succeeded

Advertising/bingads-13/campaign-management-service/googleimportoption.md
:white_check_mark:Succeeded

Advertising/bingads-13/campaign-management-service/webpage.md
:white_check_mark:Succeeded

Advertising/bingads-13/customer-management-service/getaccountpilotfeatures.md
:white_check_mark:Succeeded

Advertising/bingads-13/customer-management-service/getcustomerpilotfeatures.md
:white_check_mark:Succeeded

Advertising/bingads-13/guides/mixed-campaigns.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


About Us page: Due Week 5 completion (module 8.1)

●	Static HTML/CSS content related to the company, Proviso.

Completed prototype for About Us page

It's a great team work. Thank you.


Where can I get        talk-llama-wav2lip.bat or talk-llama-wav2lip-ru.bat or talk-llama-just-audio.bat?

in Releases section. https://github.com/Mozer/talk-llama-fast/releases/tag/0.1.3
download talk-llama-fast-v0.1.3.zip

Thanks!


Support DEM format

See https://github.com/migurski/DEM-Tools

I guess this is now supported, right @psi29a ?

Jup, DEM isn't a format... just means digital elevation map which covers everythign from bmp to ter files. I think we got this covered, as covered in the link above with GTiff :)


Feature request: don't specify attn_flash. Select when possible

Rather than manually specifying attn_flash, why not allow the code to adaptively figure out if possible in the forward() method?
As far as i can tell you can use it when there is no fancy relative positional bias and not doing an ONNX export.
I feel like it would be a case of doing something like:
if not onnx_export and not t5_rel_bias and not ...:
     return F.scaled_dot_product_attention(...)
else:
    return some_vanilla_sdpa(...)

I don't know if there is way to introspect if we're doing an ONNX export. If not, we could simply add a parameter to the forward() method like onnx_exporting

Or at least, it would be nice if you could specify attn_flash in forward() not in the constructor of the transformer.


fix bug to deny to add orgnization as a member of an orgnization or team

Will fixes #1789

LGTM

LGTM


*.bmp

I've added support for BMP files, in different bit depths too. There's a few bugs, but it mostly works.


Added verbiage that rules are system-level

Added verbiage that these rules are not simply form-level, but system-level. Also added that changes to fields do not get added to the history.

@ChristopherMank Thank you for your contribution, I have assigned this PR over to the article author for review. Thank you.


AnimatedCursor jumps back and forth infront of the MotionController

Describe the bug
Just pointing into nowhere results in the Cursor switching position like crazy, bewteen linerenderer tip and somewhere along that line.
To reproduce
Steps to reproduce the behavior:
Just watch your UWP Motion Controller cursor
Expected behavior
Pointer is stable at the linerenderer tip
Your Setup (please complete the following information)

UNITY Version 2018.3.13f1
MRTK Version v2.0

Target Platform (please complete the following information)

WMR immersive

The issue was that there was no proper case for distances longer than the raysteps total length. And in case of an exact same lenght, floating point precision coused the raysteps length to sometimes be shorter than the distance.


Improve Radical properties via Roslyn

e.g. by doing:
class Person : Entity
{
   [RadicalProperty]
   public String Name
   {
      get;
      set;
   }
}

that at compile time is automatically translated to:
class Person : Entity
{
   [RadicalProperty]
   public String Name
   {
      get { return this.GetPropertyValue(() => this.Name); }
      set { this.SetPropertyValue(() => this.Name, value); }
   }
}

+1 Radical Properties are the most common feature used... so if we "cut" the number of needed LOC is good also for the readability of the code, and moving from a not Radical VM will be more easy.

👍 but how it works? you should install an extension?

Roslyn stuff can be shipped in a nuget package, Compiler will pick it up.

Roslyn stuff can be shipped in a nuget package, Compiler will pick it up.

great then! 👍 👍


Configure scripts per test in Spring boot 3.1 test containers configuration

I prepared data scripts per test in Spring Boot 3.1 and Postgres. How can i configure
script with @BeforeAll annotation in new spring boot style? As described this chapter "Using Spring Boot 3.1.0 @ServiceConnection" for new spring boot test containers configuration.
Old way:
var containerDelegate = new JdbcDatabaseDelegate(postgreSQLContainer, "");
var scripts = new PathMatchingResourcePatternResolver().getResources(directory);
var scriptDir = new PathMatchingResourcePatternResolver().getResource("db");
for (Resource script : scripts) {
    ScriptUtils.runInitScript(containerDelegate,
        Paths.get("db", scriptDir.getFile().toPath().relativize(script.getFile().toPath()).toString()).toString());
}

How is new way?

If you have separate SQL script for tests then you can use @Sql("script_name.sql") annotation on test method that will be executed by Spring.

Not working with spring-boot-starter flyway. Flyway applied migrations:
logback: 11:55:37.595 [Test worker] INFO  o.f.core.internal.command.DbMigrate - Successfully applied 10 migrations
I created simple insert script, and next run with  @Sql("simple.sql") on the method/class.
I get communicate from pg container : "relation not exists" . When run debug mode, tables are created. (Run without @Sql)

Can you create a issue reproducer so that I can take a look at it?

My bad. Resolved it. But i have another problem.  Open new issue.


changed hard-coded reply to event emit

Previously you had no choice but to let the bot attempt to reply to a users message when it matched a command that was disabled on a given server.
This functionality could be rather annoying when you wanted a bot to remain quite and ignore requests for disabled commands. The only way to intercept this would be to build an inhibitor
With this change you can use the client#commandBlocked event to write custom logic to handle instances where the command is disabled.

This change seems to be against the whole purpose of having a command handler at all. The lib is supposed to handle things like this, so the user doesn't have to. This change brings on the reverse of that, forcing the user to handle this themselves. That's not how the handler should work. If everything was changed to be like this, it would defeat the entire purpose of having the handler in the first place, to reduce the amount of things the user has to handle.

Also, the bot just ignoring valid input is always a big no-no in my book, as it's awful for UX.


[issue-8] Add spring-data-redis support

Fixes #8
@malafeev please take a look

Pull Request Test Coverage Report for Build 46

329 of 357 (92.16%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+3.0%) to 4.085%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

opentracing-redis-spring/src/main/java/io/opentracing/contrib/redis/spring/connection/TracingRedisConnectionFactory.java
9
10
90.0%

opentracing-redis-spring/src/main/java/io/opentracing/contrib/redis/spring/connection/TracingRedisClusterConnection.java
54
55
98.18%

opentracing-redis-spring/src/main/java/io/opentracing/contrib/redis/spring/connection/RedisTracingUtils.java
13
17
76.47%

opentracing-redis-spring/src/main/java/io/opentracing/contrib/redis/spring/connection/TracingRedisConnection.java
253
275
92.0%

Totals

Change from base Build 45:
3.0%

Covered Lines:
443

Relevant Lines:
10844

💛  - Coveralls

thanks @ddcprg

java-redis-client version 0.0.7 with spring support is released.
It will be available in maven central during 24 hours.


Show partial of previous / next image

Enhancement request to add ability to see the next or previous image in the slide show and able to determine how much is shown.

Thanks for the suggestion @jaylaw81 . I'll look into this and see what can be done

You can achieve this using the following styles
.custom-slideshow {
  width: 40rem;
  margin-left: auto;
  margin-right: auto;
  overflow: hidden;
}
.custom-slideshow .react-slideshow-wrapper {
  width: 30rem;
  margin-left: auto;
  margin-right: auto;
  overflow: visible;
}

Just call your Slideshow component with custom-slideshow class
I have 4 slides with different background color


Error when trying to open .unity3d file

I attached an example of 1 of the unity3d files as .txt, because you cant upload unity3d
4064719251_2067814599.txt
Game name: Honkai Impact 3rd
Engine version: -
Platform: PC
Error message: Invalid version format
Stack trace:    at uTinyRipper.Version.Parse(String version)
at uTinyRipper.BundleFiles.BundleHeader.Read(EndianReader reader)
at uTinyRipper.BundleFileScheme.ReadHeader(Stream stream)
at uTinyRipper.BundleFileScheme.ReadScheme(Stream stream)
at uTinyRipper.GameCollection.ReadScheme(SmartStream stream, String filePath, String fileName)
at uTinyRipper.GameCollection.LoadScheme(String filePath, String fileName)
at uTinyRipper.GameStructure.ProcessPlatformStructure(GameStructureProcessor processor, PlatformGameStructure structure)
at uTinyRipper.GameStructure.Load(List1 pathes, LayoutInfo layinfo)    at uTinyRipper.GameStructure.Load(IEnumerable1 pathes, LayoutInfo layinfo)
at uTinyRipperGUI.MainWindow.LoadFiles(Object data)

Ether devs modified bundle format or they just try to protect their game. Anyway, such files aren't supported by ripper


added support for bountiful fares, architect's pallete, ad astra, and 1.20.1 quark

Better-Leaves-7.2-1.19+.zip
Test file if needed

Thank you very very much :)


Add support for simple enum values

In Go you can assign constants but when you have many they tend to:

Pollute the global namespace with many values
Can become confusing if you have multiple "groups" of constants flattened on the namespace (think status codes + common headers)
Requiring compile-time validation of values through a repetitive, non-performant operation

Objective:
package main

import (
	"fmt"
)

type MyType string

const (
	A MyType = "A"
	B MyType = "B"
	C MyType = "C"
)

// foo can accept any string however ideally you'd want to check the value it's passed
// matches an item within the the constants group you consider it belonging to
func foo(bar MyType) {
	fmt.Println(bar)
}

func main() {
	foo(A)
	foo("D")
}

A common way around this is to use a runtime validation strategy, however it's still a bit cumbersome and you tend to have a lot of package namespace pollution (imagine if you wanted to collect a list of common HTTP header names, header values and status codes - or all of the country codes in the iso spec, or all the currency codes in the iso spec. The package namespace would be nothing but constants).
type MyType int

const (
	A MyType = iota
	B
	C
)

func GetMyType(v MyType) (string, error) {
	switch v {
	case 0:
		return "A", nil
	case 1:
		return "B", nil
	case 2:
		return "C", nil
	default:
		return "", fmt.Errorf("Not found")
	}
}

func foo(bar MyType) {
	fmt.Println(GetMyType(bar))
}

func main() {
	foo(A)
	 foo("D")
}

You can also group properties in Go using duck-typed structs - however they are heap allocated and still require runtime validation.
var TooType = struct {
  A string
  B string
  C string
}{"A",  "B", "C"}

If Go could introduce a method to group constants (simple values) and allow for value types which can be validated at compile time, that would improve the performance and management of Go applications that require the use of grouped constants that require validation.
Note: I don't know how to include it without adding a new keyword, that's the magic of the brilliant people behind Go, so I will just suggest it using my own pseudo-syntax
const StatusCode = enum {
  OK int = 200
  NoContent = 201
  NotFound = 404
}

func foo(f StatusCode) {}
func bar(f int) {}

func main() {
  fmt.Println(StatusCode.OK) // "200"
  foo(StatusCode.OK) // valid
  foo(200) // Invalid, must come from group
  bar(StatusCode.OK) // valid
}

It almost sounds like you are proposing a change to the Go programming language here, which unfortunately not the correct location to do so.
This repo is a client library to the GitHub v3 API using the Go programming language.
If you wish to propose changes to the Go programming language, I suggest you start here:
https://github.com/golang/proposal
Closing issue.


CMake build Tensorflow C++ on Windows 10 Error : Test COMPILER_OPT_ARCH_NATIVE_SUPPORTED - failed

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary): source
TensorFlow version: the latest version
Python version: 3.6
Bazel version (if compiling from source): I am using CMake
GCC/Compiler version (if compiling from source): gcc 6.30
CUDA/cuDNN version: Installing only the CPU version
I am also using Microsoft Visual Studio Community 2017 Version 15.9.3, btw

Describe the problem
I am following the "Step by step Windows Build" from https://github.com/tensorflow/tensorflow/tree/master/tensorflow/contrib/cmake. However, at step 3, I can't seem to pass Performing Test COMPILER_OPT_ARCH_NATIVE_SUPPORTED, which resulting failed building process.
Provide the exact sequence of commands / steps that you executed before running into the problem
PS C:\Users\bw\tensorflow\tensorflow\contrib\cmake\build> cmake .. -A x64 -Thost=x64 -DCMAKE_BUILD_TYPE=Release `
>> -DSWIG_EXECUTABLE='C:\Program Files\swigwin-3.0.12\swig.exe' `
>> -DPYTHON_EXECUTABLE='C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python36_64\python.exe' `
>> -DPYTHON_LIBRARIES='C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python36_64\libs\python36.lib'
-- Building for: Visual Studio 15 2017
CMake Warning at CMakeLists.txt:9 (message):
  Your current cmake generator is set to use 32 bit toolset architecture.
  This may cause "compiler out of heap space" errors when building.  Consider
  using the flag -Thost=x64 when running cmake.

-- The C compiler identification is MSVC 19.16.27024.1
-- The CXX compiler identification is MSVC 19.16.27024.1
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.16.27023/bin/Hostx64/x64/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.16.27023/bin/Hostx64/x64/cl.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.16.27023/bin/Hostx64/x64/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.16.27023/bin/Hostx64/x64/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Performing Test COMPILER_OPT_ARCH_NATIVE_SUPPORTED
-- Performing Test COMPILER_OPT_ARCH_NATIVE_SUPPORTED - Failed
-- Performing Test MSVC_OPENMP_SUPPORT
-- Performing Test MSVC_OPENMP_SUPPORT - Success
-- Found PythonInterp: C:/Program Files (x86)/Microsoft Visual Studio/Shared/Python36_64/python.exe (found version "3.6.7")
-- Found PythonLibs: optimized;C:/Program Files (x86)/Microsoft Visual Studio/Shared/Python36_64/libs/python36.lib;debug;C:/Program Files (x86)/Microsoft Visual Studio/Shared/Python36_64/libs/python36_d.lib (found version "3.6.7")
-- Found SWIG: C:/Program Files/swigwin-3.0.12/swig.exe (found version "3.0.12")
CMake Error at tf_python.cmake:811 (string):
  string sub-command REPLACE requires at least four arguments.
Call Stack (most recent call first):
  CMakeLists.txt:583 (include)

CMake Error at tf_python.cmake:812 (string):
  string sub-command REPLACE requires at least four arguments.
Call Stack (most recent call first):
  CMakeLists.txt:583 (include)

CMake Error at tf_python.cmake:813 (string):
  string sub-command REPLACE requires at least four arguments.
Call Stack (most recent call first):
  CMakeLists.txt:583 (include)

-- Configuring incomplete, errors occurred!
See also "C:/Users/bw/tensorflow/tensorflow/contrib/cmake/build/CMakeFiles/CMakeOutput.log".
See also "C:/Users/bw/tensorflow/tensorflow/contrib/cmake/build/CMakeFiles/CMakeError.log".

Any other info / logs
CMakeOutput.log: https://www.dropbox.com/s/7fweyunxdbmxa1k/CMakeOutput.log?dl=0
CMakeError.log: https://www.dropbox.com/s/tucx0tl6346kdpd/CMakeError.log?dl=0

Can you please switch to following known good configurations and build again:
Microsoft  Visual Studio 2015
Python 3.5

Thanks for your reply @ymodak ! I have indeed tried building it with vs 2015 and Python 3.5, and got same error:

cmake .. -G 'Visual Studio 14 2015 Win64' -Thost=x64 -DCMAKE_BUILD_TYPE=Release -DSWIG_EXECUTABLE='C:\Program Files\swigwin-3.
0.12\swig.exe'  -DPYTHON_EXECUTABLE='C:\Users\user_1\AppData\Local\Programs\Python\Python35\python.exe'  -DPYTHON_LIBRARIES='C:\Users\user_1\AppData\Local\Programs\Python\Python35\libs\py
thon35.lib'
CMake Warning at CMakeLists.txt:9 (message):
Your current cmake generator is set to use 32 bit toolset architecture.
This may cause "compiler out of heap space" errors when building.  Consider
using the flag -Thost=x64 when running cmake.
-- Selecting Windows SDK version  to target Windows 10.0.17134.
-- The C compiler identification is MSVC 19.0.24234.1
-- The CXX compiler identification is MSVC 19.0.24234.1
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64/cl.exe -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Performing Test COMPILER_OPT_ARCH_NATIVE_SUPPORTED
-- Performing Test COMPILER_OPT_ARCH_NATIVE_SUPPORTED - Failed
-- Performing Test MSVC_OPENMP_SUPPORT
-- Performing Test MSVC_OPENMP_SUPPORT - Success
-- Found PythonInterp: C:/Users/user_1/AppData/Local/Programs/Python/Python35/python.exe (found version "3.5.4")
-- Found PythonLibs: C:/Users/user_1/AppData/Local/Programs/Python/Python35/libs/python35.lib (found version "3.5.4")
-- Found SWIG: C:/Program Files/swigwin-3.0.12/swig.exe (found version "3.0.12")
CMake Error at tf_python.cmake:811 (string):
string sub-command REPLACE requires at least four arguments.
Call Stack (most recent call first):
CMakeLists.txt:583 (include)
CMake Error at tf_python.cmake:812 (string):
string sub-command REPLACE requires at least four arguments.
Call Stack (most recent call first):
CMakeLists.txt:583 (include)
CMake Error at tf_python.cmake:813 (string):
string sub-command REPLACE requires at least four arguments.
Call Stack (most recent call first):
CMakeLists.txt:583 (include)
-- Configuring incomplete, errors occurred!
See also "C:/Users/user_1/tensorflow/tensorflow/contrib/cmake/build/CMakeFiles/CMakeOutput.log".
See also "C:/Users/user_1/tensorflow/tensorflow/contrib/cmake/build/CMakeFiles/CMakeError.log".

(The CMake build is no longer supported by the core TensorFlow team, so I'm marking this issue as "community support".)

@bwang482 We are checking to see if you still need help on this issue? We recommend that you upgrade to 2.7 which is latest stable version of TF . Also please have a look on the link and let us know if it helps? Thanks!


New aws_string type for immutable strings which can be used as keys and values for hash table

-Reorganized byte buffer, byte cursor, split, and string tests
-Updated wording of split functions so as not to refer to strings
-Change hash table to use different destroy functions for key and value
-Helper functions for string use in hash table
-Unit tests for new string and hash table functionalities
Issue #, if available:
Description of changes:
Implementation of hash table was not usable with binary strings because if they are not null terminated we have no way to determine the length once we retrieve the pointer. The byte buffer implementation was not suitable for this either, because the struct itself is generally declared on the stack and only its data bytes are allocated.
This implementation of a string type solves these problems and also includes added functionality and tests in the hash table code to allow for hash tables with strings as keys and/or values.
There is also a significant amount of cleanup and reorganization of the test code, as our tests for byte buffers, cursors, and split functions had become scattered and the file names were not always accurate.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I made a very minor update to the AWS_STATIC_STRING_FROM_LITERAL macro, but otherwise this PR is still unchanged from before. Unfortunately, I mishandled my git branches, so the commits that I had originally all squashed together on this got added to the git history making it looks much hairier than it is.


Can we make Ruby DSL config format deprecated?

Is your feature request related to a problem? Please describe.*
https://github.com/fluent/fluent-plugin-prometheus/issues/149
Describe the solution you'd like
I'd like to make ruby DSL config format deprecated because we don't have enough resources to maintain and I couldn't find the ruby DSL format in our document.
Describe alternatives you've considered
leave it as it
Additional context

closed via #2958


sync-from-downstream: construct a PR correctly when origin is a fork

I know I know, I'm crazy.
Fixes #200
I can put these changes into a different PR:

api: make upstream & distgit properties

refactor and simplify api.sync_from_downstream

default no_pr to False, not None
try/finally did not do anything
sync_upstream was only used by sync_from_downstream
update the messages a bit

It works: https://github.com/packit-service/ogr/pull/46

@TomasTomecek f763221
recipe: set metadata in the playbook
Signed-off-by: Tomas Tomecek<EMAIL_ADDRESS>
It woul be nice to have this in the sepparate PR, but leave it here for now...;-)

Another one bites the dust! https://github.com/TomasTomecek/pretty-git-prompt/pull/45

ehm those test errors -_- is github down? we're getting 500 when downloading an archive from github

werking on it

ogr 0.2.0 is now on PyPI

Let's merge!


Added bg.json translation

Hey, I've added Bulgarian translation - bg.json

@TheDjRider Thank you!
P.S. Would be great if you could give a star to the repo :) Thanks


Spyder frequently crashing on MacBook Pro with Touch Bar

Description
I have a late 2016 MacBook Pro and Spyder frequently crashes. I get the following error message in terminal after it has crashed. It has happened when I'm running both Python2.7 or Python 3. I thought the problem had gone away when I migrated to Python 3 but has now reappeared.
I didn't have this problem on a MacBook Air running Python 2.7.
Messages in the Terminal below:
/Users/karim/miniconda2/envs/riskpy36/bin/pythonw: line 3:  2901 Segmentation fault: 11  /Users/karim/miniconda2/envs/riskpy36/python.app/Contents/MacOS/python "$@"
What steps will reproduce the problem?
No particular steps, I have found no pattern in the crashes.
What is the expected output? What do you see instead?
Continued operation of Spyder.
Please provide any additional information below
Thanks for all your work, despite the crashes Spyder is a great tool. Let me know if you need any further information.
Version and main components

Spyder Version: 3.2.3
Python Version: 3.6.2
Qt Versions: 5.6.2, PyQt5 5.6 on Darwin

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.20.3 (OK)
numpy >=1.7      :  1.13.1 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
rope >=0.9.4     :  0.9.4-1 (OK)
jedi >=0.9.0     :  0.10.2 (OK)
psutil >=0.3     :  5.2.2 (OK)
nbconvert >=4.0  :  5.2.1 (OK)
sympy >=0.7.3    :  None (NOK)
cython >=0.21    :  None (NOK)
qtconsole >=4.2.0:  4.3.1 (OK)
IPython >=4.0    :  6.1.0 (OK)
pylint >=0.25    :  1.7.2 (OK)

It appears to be related to launching of iPython consoles within Spyder. On starting Spyder, in the Terminal I get the following:
[warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0
When I launch more iPython consoles, I get a few more of these error messages. When the same error appears 11 times, Spyder crashes and I get the following:
[warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0 /Users/karim/miniconda2/envs/riskpy36/bin/pythonw: line 3:  4622 Segmentation fault: 11  /Users/karim/miniconda2/envs/riskpy36/python.app/Contents/MacOS/python "$@" (riskpy36) AKs-MacBook-Pro:Model Karim$ [1004/160259:ERROR:mach_broker_mac.mm(52)] bootstrap_look_up: Unknown service name (1102) [1004/160259:ERROR:browser_io_surface_manager_mac.cc(47)] bootstrap_look_up: Unknown service name (1102) [warn] kq_init: detected broken kqueue; not using.: Undefined error: 0

Thanks for reporting. This could be fixed when Anacoda updates Qt, the graphical toolkit used by Spyder.
Unfortunately, in the mean time we can only recommend you to run
spyder --reset

in a terminal and try again.

Hi Carlos,
Thanks for responding. If I installed Python directly and then Spyder (i.e. not use Anaconda) would this issue go away?

Probably. Please try it and let us know if that fixes the problem for you.
You could use Homebrew to do it.

@dscience7, we're tracking this problem in
https://github.com/ContinuumIO/anaconda-issues/issues/6554
Anaconda Inc. is in the process of updating Qt, and that should fix this problem.


Can't connect to SQL Server Database with Integrated Authentication / Kerberos

I tried to connect to a SQL Server Database using kerberos authentication from a Linux host.
Connection string I use is : Server=XX.XX.XX.XX,1433;Database=YYYYYY;trustedConnection=true;trustServerCertificate=true
(host name resolution didn't work)
returned error is : http://localhost:4177: Login failed for user ''.
It seems that the trustedConnection parameter is completly ignored.
Is integrated authentication on SQL Server Database supported by the gateway ?

Hi @frederic-luce
Thanks for raising this.
The authentication part is handled by the connector, can you try to run the connect.test.ts and see how it goes?
If you can find a working setup (with the second test), I can make the change in the connector.
(as I don't have an SQL Server with kerberos, I can't test on my side)
Or we can setup a meeting to debug this together if you want.
In any way, you will have to checkout the repo and run the JS test on your machine.


Unexpected behavior when running tape command

Using the provided test runner (tape command), I pass it a glob which gives me the following found files:
[ 'lib/config/test/env-test.js',
  'lib/config/test/string-test.js',
  'lib/models/test/club-test.js',
  'lib/models/test/member-test.js',
  'lib/models/test/notice-test.js',
  'lib/models/test/template-test.js',
  'lib/models/test/types/email-test.js',
  'lib/models/test/types/phone-test.js',
  'lib/models/test/types/zip-test.js',
  'lib/models/test/user-test.js' ]

However, as it processes the files I get the following invalid TAP output (note: TAP header appears multiple times, numbering starts over).
TAP version 13
# config?env
ok 1 valid json configs
# Date#toGolfHitsString
ok 2 converts < 10 month > 10 day
ok 3 converts > 10 month < 10 day
TAP version 13
# setup
# Club.schema
ok 1 chair person is required
# Club#setLogo

It seems to "start over" when it switches from the config folder to the models folder but I don't know why.  Seems like it shouldn't matter.

#97 was closed in favor of https://npmjs.com/cached-tape.


PHP Fatal error:  Allowed memory size exhausted

PHP Fatal error:  Allowed memory size of 134217728 bytes exhausted (tried to allocate 20480 bytes) in /var/www/html/vendor/halaxa/json-machine/src/Parser.php on line 177
Json-machine version is 0.3.2
I am trying to read file having size of 27mb.

Hello. Please take a look at #3 (you probably using it wrong). If that's not your case, please provide a malfunctioning code sample and data. Thanks.

Thanks for your quick response @halaxa.
As per #3, it is true that I'm trying to read the entire json file.
What I need is to read the key and it's respective value from root level.
Because the subtree results is a key at root level it is giving Fatal error to me.
The subtree results is consists of an array with almost 60K records.
I can read the subtree results using json pointer, that works great!
Is there is a way I can skip the key results from the root level and read all other keys and its values. Fetch the value for specific keys like  root/last-update-time, root/resultCount from the entire json file.
Here is the structure of the json file:

{
"last-update-time":"2019-04-09 01:00:00.0",
"timezone":"PDT",
"results":[
{  },
{  },
{  },
],
"resultCount":60638
}

There is no feature to skip a key (results in your case). But I see why that could be useful. If you have control over the json structure, I'd suggest to put resultCount key before the results key and thus make the whole structure make more sense. Putting count at the end makes little sense to me, because I could have them counted by the time I reach the resultCount key. If you don't have control over the json structure and the order of the keys does not change across requests, I'd suggest to read first two keys and then stop. Then use json pointer and read the results key. You can manually count the results which means you don't need to get to resultCount.
JsonMachine is intended to be simple iterator over huge json structure. If one needs more complexity, I'd recommend https://github.com/salsify/jsonstreamingparser. It's more complicated to set up, but more low level and may better fit your needs.
Dou you think JsonMachine should have support skipping "big" keys? How would the function signature look like? Thanks.

Sorry, I don't have control on the json structure since it is coming from api response.
Since at first go I have to read the metadata from the json response. There is an another script, that reads the results json object and stored it in database table.
Since you mentioned

JsonMachine is intended to be simple iterator over huge json structure.

so I thought it should work start from the root level and not specific from subtree inside json response.
Definitely it will be helpful if we could only read the required keys from the json object. At this moment I haven't thought of how we can skip the large keys, but I will try it in spare time.
Thanks again.

Why it doest not work

so I thought it should work start from the root level and not specific from subtree inside json response.

If you wanted to start in document root and have control over depth of the document, you'd have to do tree traversing. Travevesing tree in client code always adds some complexity, that's why philosophy of Json machine avoids it. Thanks to that the initial example in readme is so simple. The main goal was just to make a library you throw some huge list on and iterate it via foreach. In most of the cases one just needs to process big list of items. That's what Json machine is for.
Let's imagine
It does not mean some inventive implementation could not take place. I have an idea for some time now. What if the value you are given in foreach wouldn't be parsed structure but rather lazy reference. It would not parse and load anything into memory until you for example access an index on it. Therefore you could iterate on the root level and just continue if you want to skip a big key. This implementation would be mostly backwards compatible with current solution and more flexible then adding another parameter to iterator. Not sure when I get to it though.

Will be addressed in #36

Long time no update from me. Currently one can utilize json pointer or recursive parsing waiting to mrege in #36 to achieve the result you want.

Recursive parsing is now finished, merged, and released.


[Question] Auto width View with <UITextView> inside

Hi there, just wondering how to do an auto-width View with <UITextView> inside?
For example in the code below, the one with <UITextView> will take up the whole width, while the one with RN <Text> will render as inline text
import { Text, View } from "react-native";
import { UITextView } from "react-native-uitextview";

export default function HomeScreen() {
  return (
    <View
      style={{
        flex: 1,
        alignItems: "center",
        justifyContent: "center",
        gap: 8,
      }}
    >
      <View style={{ backgroundColor: "blue" }}>
        <UITextView selectable uiTextView style={{ color: "white" }}>
          UITextView - Hello World
        </UITextView>
      </View>

      <View style={{ backgroundColor: "blue" }}>
        <Text style={{ color: "white" }}>Text - Hello World</Text>
      </View>
    </View>
  );
}

I'm using
"expo": "~51.0.28",
"react": "18.2.0",
"react-dom": "18.2.0",
"react-native": "0.74.5",
"react-native-uitextview": "^1.4.0"

Awesome library btw, appreciate the great work 🙏

+1

+1

I created the following component, assuming that children will not change after the initial render.
This component ensures that the width of <UITextView> matches the dimensions of a placeholder <Text> element measured during the initial render. In my case, this approach worked without any issues.
import { type ComponentProps, memo, useState } from "react";
import { type LayoutChangeEvent, Text } from "react-native";
import { UITextView } from "react-native-uitextview";

const SelectableText = memo(({ children, style, ...props }: ComponentProps<typeof UITextView>) => {
  const [measuredWidth, setMeasuredWidth] = useState<number | null>(null);

  const handleLayout = (e: LayoutChangeEvent) => {
    const { width } = e.nativeEvent.layout;
    setMeasuredWidth(width);
  };

  if (measuredWidth === null) {
    return (
      <Text {...props} style={[style, { opacity: 0 }]} onLayout={handleLayout} selectable>
        {children}
      </Text>
    );
  }

  return (
    <UITextView {...props} style={[style, { width: measuredWidth, maxWidth: measuredWidth }]} selectable uiTextView>
      {children}
    </UITextView>
  );
});

export default SelectableText;

To improve the user experience, adding opacity: 0 to <Text> until <UITextView> is rendered might be helpful. This prevents users from briefly seeing the placeholder content.


cmd/go: go build follows symlinked directories, but go list does not

$ cat ~/foo/foo.go
package main

func main() {}
$ ln -s ~/foo $GOPATH/foo
$ go build foo # works
$ go list ... | grep foo # shows nothing
$

Should the Go tool support symlinks? Right now go build does, but go list does not.

It's not build vs list. It's wildcards vs not.
If you say go build , it resolves dir to its real value before continuing.
If you say go list ... it lists all known canonical paths.
I don't believe your example. I think it would show $GOPATH/src/foo as foo.
To make your example work you'd need to give the true $GOPATH dir a different name.
Symlinks cause all manner of weird behavior. It's best not to use them at all.
No intention to fix this, since I don't see what there is to fix.

Closing. The underlying bug is that x/tools/go/loader behaves differently than go build when given a package name. I thought I had distilled this down into an issue in the Go tool, but I suspect I was wrong. Leaving the original issue for @adonovan.


Implement search quote feature

Description
There is already  a button available as Search Quote but it is not functional. When the button is clicked a search field should appear from which the user should be able to search quotes on the basis of quote phrase, author or tags.

Expectation
Search feature should be implemented.

i will work on this.

i will work on this.

Great! I have assigned you this task 🥳

@Sandesh-Ghimire, if possible could you help improve the suggestions list design.


fix: isolate binding EventEmitter

Which problem is this PR solving?
Binding an EventEmitter a second time via another instances of ContextManager shouldn't have side effects to the first ContextManager.
Short description of the changes
Use an unique symbol per ContextManager instance to isolate them.

@dyladan You added the enhancement label whereas I used fix: in commit message/PR headline.
I think we should agree on if this is a feature or a fix.


Load all keys from ssh agent if no specific AuthMethods is passed

This should solve the problem I mention in #161

The failed tests are failing for main too at least for me :thinking:

They were passing in the previous PR 🤔

https://github.com/k0sproject/rig/pull/163 tests seem to pass.

#163

I found where the problem was in my machine. I was missing one dependency.
I have fixed the problem that broke the test and wrote a test to reproduce my issue.
Unfortunately, my solution does not fix the issue. :disappointed:
I'll continue debugging.
The test suit is not the easiest thing to run, unfortunately.

The linting errors are from main.

The linting errors are from main.

So it worked :rocket: :partying_face:


fix: raise error in sqllab when using reserved column name

SUMMARY
Currently SQL Lab accepts the column name __timestamp, which causes problems when pressing the "Explore" button. This usually happens when clicking the "Run in SQL Lab" button on a timeseries chart. This adds a check for column names starting with double underscores.
BEFORE

AFTER

TEST PLAN
Local testing + new tests
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

It is a good fix to improve the error message but CHARTs generate the query with reserved words so if the workflow is
Chart -> SQL Lab : you get the reserved word and needs to be changed if you want to Explore the data again.
If in CHARTs we don't generate reserved words, that should resolve the issue to the root.

Codecov Report

Merging #9859 into master will increase coverage by 1.39%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #9859      +/-   ##
==========================================
+ Coverage   64.61%   66.01%   +1.39%
==========================================
  Files         536      588      +52
  Lines       29405    30610    +1205
  Branches     2879     3238     +359
==========================================
+ Hits        19001    20206    +1205
+ Misses      10225    10217       -8
- Partials      179      187       +8

Flag
Coverage Δ

#cypress
?

#javascript
59.30% <100.00%> (?)

#python
70.79% <ø> (-0.48%)
:arrow_down:

Impacted Files
Coverage Δ

...end/src/SqlLab/components/ExploreResultsButton.jsx
74.07% <100.00%> (+69.07%)
:arrow_up:

superset-frontend/src/SqlLab/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/SqlLab/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/setup/setupColors.js
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/setup/setupPlugins.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/chart/ChartContainer.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 319 more

Continue to review full report at Codecov.

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

@eugeniamz the thing here is that the query as such is not really usable for re-exploration, as it has time grains, filters etc burned into the query. My interpretation is that this feature is used for throwaway analysis purposes only. Perhaps we should also add a comment above the query to that effect to explain that the query is only a reflection of a certain chart, and should usually not be used as a basis for a new chart?


Backend takes over 10 seconds to shutdown

This is because of incorrect CMD format in Dockerfile.
See https://github.com/heusalagroup/fi.hg.core/issues/78
Fix here: https://github.com/heusalagroup/create-backend/blob/main/templates/Dockerfile

Fixed in version v1.0.10


Updated translate script to use corrected attr

<string corrected="true" id="blah">Blah Blah</string>

Will not be updated

https://developer.garmin.com/connect-iq/core-topics/resources/ Just wanted to check what you thought of using the translatable=false attribute? It would be more in keeping with the existing schema.

That attribute specifies whether the string should be translated by ConnectIQ so if we have translatable="true", it may try to translate them itself. This attribute also will prevent translation per string not per language so it will not work for corrections.

A quick test as follows:
<strings>
    <string id="AppName">HomeAssistant</string>
    <string id="MenuItemOn" corrected="true">On</string>

ERROR: venu: A:\Philip\Programming\MonkeyC\HomeAssistant\resources\strings\strings.xml:17: Unable to validate resource file: cvc-complex-type.3.2.2: Attribute 'corrected' is not allowed to appear in element 'string'.
ERROR: A string resource matching the provided app name can't be found. You must provide a valid string resource for the app name.

So we can't use custom attributes in the XML unless we amend the validation...

We will need to find another way to embed this data. Second XML file?

<?xml version="1.0" encoding="utf-8"?>
<!--

   Distributed under MIT Licence
     See https://github.com/house-of-abbey/GarminHomeAssistant/blob/main/LICENSE.

   GarminHomeAssistant is a Garmin IQ application written in Monkey C and routinely
   tested on a Venu 2 device. The source code is provided at:
              https://github.com/house-of-abbey/GarminHomeAssistant.

   P A Abbey & J D Abbey, 14 November 2023

-->

<!--
  Corrections for the French language
  Corrections pour la langue française
-->

<corrections>
  <string id="MenuItemOn">Allumée</string>
</corrections>


ADD: Read me

create work for coming with the readme. can you sync with #9 He has created the readme already. Just update it.


create a copy of DEFAULT_HEADERS for compliance hook

Is your feature request related to a problem? Please describe.
In the refresh_token function of oauthlib/oauth_client.py, the headers are set to DEFAULT_HEADERS, and then refresh_token_refresh compliance hook is called. If compliance hook modifies the headers pointing to DEFAULT_HEADERS, it would taint global DEFAULT_HEADERS and all consequent calls would be using altered version of headers unintentionally
        if headers is None:
            headers = DEFAULT_HEADERS

        for hook in self.compliance_hook['refresh_token_request']:
            url, headers, body = hook(url, headers, body)

Describe the solution you'd like
headers should be set to a copy
headers = DEFAULT_HEADERS.copy()

Describe alternatives you've considered
Compliance hook implementation should be extra careful to not update headers/always create a copy to be returned
Additional context
Add any other context or screenshots about the feature request here.

Thanks for your feedback. I've fixed it in master code.


Adds keys and values iterators

This PR
Adds keys and values iterator helpers to the MapReadRef per this comment.

This change is

This looks great once again, thank you!

 And thanks for all your great reviews!
While writing these assertions I was thinking, "It might be convenient to implement Eq on Values if the inner value implements Eq". But I can't tell if it's just convenient for tests. Is that something you might want?
Otherwise, looking through the indexmap tests there are tests for map.values_mut(), map1 == map2, and then some stuff based on sort order that I'm guessing we don't care about because we don't care about order. Would you prefer that kind of functionality?
Above and beyond all that, are there any other things you've been looking to revisit/clean up that I could tackle? This comment looks like an obvious place to start. I could think on that if it's valuable. I'm not sure I'd be able to brain out something at this level, but I'd also be interested in trying 🤷
Or if there are any other projects you have or know about that need some more chores like this done (or easy features or whatever), I'm game!

It's been a pleasure!
I think PartialEq + Eq on Values is probably mostly useful for tests, but it seems reasonable enough to me! One thing that'll be tricky is to check that the cardinality is right if one thing is Short (but has duplicates) and the other thing is a Long.
I think the particular code comment you linked to is unlikely to matter much in practice, since it only every runs once for a given evmap. In general, a good strategy is to look for things that are missing in the coverage map and add tests for those. In terms of other issues, it would be awesome to add loom tests (https://github.com/jonhoo/rust-evmap/issues/17), which will also require some thinking to get right.
As far as other projects go, here are a couple:

griddle has some chore-like stuff in https://github.com/jonhoo/griddle/issues/4 (RawEntry in particular)
inferno has some test coverage bits in https://github.com/jonhoo/inferno/issues/63
flurry needs more tests here https://github.com/jonhoo/flurry/issues/20


Document how to manually set up a Meltano Droplet on Digital Ocean

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/927
Originally created by @dmor on 2019-08-27 19:21:08

I am doing this for getting our one-click app listed in the marketplace, but it might be worthwhile to also tell Advanced users how they can do this manually (if they want to).
At the very least, this will document how we set up our app for contributors.

[x]  @dmor In the next image setup, try using the exact same update
[x]  @dmor also try a version where we install all our dependencies directly on the box (no need for virtual environment)
[x]  @dmor check to see what comes on the Ubuntu box by default
[x]  @dmor check for Postgres

View 15 previous comments from the original issue on GitLab


Generator: Update SDK /services/dns

Generated from GitHub run<PHONE_NUMBER>0

There is a newer PR with same changes. https://github.com/stackitcloud/stackit-sdk-go/pull/991


Always assume special urgency for contributor PR pings

Fixes

Fixes #4327 by @zackkrida
Description

This PR modifies the PR reminder review DAG to use a special urgency level, contributor (3 business days), for any pull request that isn't opened by one of the WordPress/openverse-maintainers users (or a few other bot accounts). I modified the DAG workflow to use the TaskFlow API syntax, and updated tests accordingly as well.
3 days was chosen for this level based on the linked issue and this documentation: https://docs.openverse.org/general/contribution/good_first_and_help_wanted_issues.html#timing-and-process
Testing Instructions

The new tests should pass and make sense. We don't have any PRs open right now that qualify for this new ping criteria, but if you have a copy of the GitHub PAT we use, you can test that the DAG still runs as expected with the new dependency.
Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[x] I ran the DAG documentation generator (just catalog/generate-docs for catalog
PRs) or the media properties generator (just catalog/generate-docs media-props
for the catalog or just api/generate-docs for the API) where applicable.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Nice change! Surprised at how many lines had to be changed for this.

Most of the lines were around machinery related to actually getting the maintainer members from GitHub - not something we had done before! 😄


License information

Steps

Scan all image tarballs for license updates
Update the NOTICE file.
Place following disclaimer text at the end of repository description in DockerHub.

License
This software is provided under Apache 2.0 License. View the NOTICE file for the third party software contained in this image.

Done - https://hub.docker.com/r/servicecatalog/oscm-deployer/


"Semlac" are villages in "Semlac" common, "ARAD" county

Steps 1
Acces to the link https://peviitor-ro.github.io/ui.orase/
Expected result
The page opens

Steps 2
Write "Semlac"
Expected result
In the drop down menu you find "Semlac, ARAD(Semlac)"
Test Data
Semlac


Added a small project starting script

Basically, sets up a small directory structure for most projects, gives you a gitignore file, index template, main application file.

I saw a interesting project that using Paster to generate flask site. How about add an official Paster template of flask?
https://bitbucket.org/danjac/flask-kickstart/overview

But, that shouldn't be added into mainline.

For the first comment, I'll modify the code to use it, thanks.
And, for the second, you can do --git to execute the git code, I guess.

I'm not sure if I understand it fully, but, I don't think its related...

This is more something for a separate project (maybe a flask extension?) than flask itself. But maybe I will change my mind later on, but certainly not now :-)

http://github.com/thomaslee/gener does this too