Definition/distinction between Pityriasis streptogenes/alba ?

MONDO:0024345 Pityriasis streptogenes has no definition and I cannot find any articles in Pubmed using this string. Likewise a google search doesn't return any specific matches on this specific term. There do seem to be multiple hits connecting pityriasis rosea and streptococcus infection. (including https://www.ncbi.nlm.nih.gov/books/NBK448091/ )
MONDO:0024349  Pityriasis alba also has no definition in Mondo, but I was able to find some clearer references to this condition.  i.e.  https://www.ncbi.nlm.nih.gov/books/NBK431061 .
UMLS has the two SNOMED CT codes on the same CUI (C0085657) , possible because another one of their sources "SNOMED International" term Pityriasis alba (D0-22163) includes 'pityriasis simplex' and 'pityriasis streptogenes' as synonyms.
Bottom line ask is this: can Mondo find any additional definitions or information for these two conditions to help determine if they are, indeed, distinct conditions with specific clinical presentations? Or bring this to some dermatology expert for review?

@katiermullen could you work on this one? thx!

I am not a dermatology expert. I agree there no pubmed articles with the string "Pityriasis streptogenes."
I would propose:

Definition for pityriasis rosea: A common acute, self-limited papulosquamous dermatosis. It is characterized by a herald patch followed by scaly oval patches on the trunk and proximal extremities in a "Christmas-tree" appearance. Source: PMID:32964824
Pityriasis streptogenes could be an obsoletion candidate with exclusion reason MONDO:excludeCausalAgents - previous strep infection is a suggested cause of pityriasis rosea, but that connection does not appear well established.
Definition for pityriasis alba: A prevalent and benign dermatological condition that predominantly affects children and adolescents. The name "pityriasis alba" derives from its appearance, where "pityriasis" denotes the fine scales and "alba" signifies the pale color (hypopigmentation). Source: https://www.ncbi.nlm.nih.gov/books/NBK431061/

to do: add MONDO:0024345 'pityriasis streptogenes' to obsoletion list with exclusion reason MONDO:excludeCausalAgents @sabrinatoro

I added a def to pityriasis alba

Checking in on what is left to do with this ticket:

"pityriasis rosea" (MONDO:0006601) already has a definition with reference NCIT:P378:

A mild, self-limited skin disorder that is most commonly seen in children and young adults. It is characterized by an initial large round spot on the chest, abdomen, or back, often referred to as a herald patch, that is usually followed within a week by a distinctive pattern of similar but smaller papules on the torso, arms, and legs. There may also be itching, especially when overheated.

Katie in a comment above proposed:
A common acute, self-limited papulosquamous dermatosis. It is characterized by a herald patch followed by scaly oval patches on the trunk and proximal extremities in a "Christmas-tree" appearance. Source: PMID:32964824

Question: Are any changes needed in the definition?

"Pityriasis streptogenes" (MONDO:0024345) will be obsoleted as indicated in comments above

@twhetzel I think that all that is left to do is obsolete "Pityriasis streptogenes" (MONDO:0024345). Since "pityriasis rosea" (MONDO:0006601) already has a definition with reference NCIT:P378, you can ignore the definition that I proposed.
Thank you!


Getting com.consol.citrus.exceptions.ActionTimeoutException

Problem statement :
com.consol.citrus.exceptions.ActionTimeoutException: Action timeout while receiving message from channel 'soapJaxwsServer.inbound'.
Exception came when timeout(600000ms) in mock server configuration is over, sometime it is working.
7 out of 10 attempt will shows the exception.
citrus-context.xml (mock server configuration)
<citrus-ws-server id="soapJaxwsServer" port="13080" auto-start="true" timeout="600000"
root-parent-context="true" resource-base="src/citrus/resource" interceptors="clientInterceptors"/>

citrus-ws-server.xml
<bean id="defaultWsEndpoint" class="com.consol.citrus.ws.server.WebServiceEndpoint">
     <property name="endpointAdapter" ref="emptyResponseEndpointAdapert"/>
</bean>
<citrus:empty-response-adapter id="emptyResponseEndpointAdapter" />

mytest.xml

When i got the exception, response is coming properly.
As per my understand, it is generating RACE condition somewhere in citrus-core code, might be it is in QueueChannel.
Could you please suggest how i can moved out from this issue.
(sorry i am unable to put the logs at this time)
Thanks,
kapil

The information given in your issue description is giving me headaches. You give an empty mytest.xml - why that? You give the citrus-ws-server.xml configuration but I do not see that soapJaxwsServer is referencing that configuration anywhere. Are you overwriting the defaultWsEndpoint bean?
You define a static empty response endpoint adapter in that configuration on the defaultWsEndpoint and at the same time you want to receive the request in your test? This is not working as the empty response adapter will always take the request first. Your test will always get ActionTimeoutException then.
Please clarify the configuration and test code so I can have a look at this, thanks!


fstl: 0.9.4 -> 0.10.0

Description of changes
https://github.com/fstl-app/fstl/releases/tag/v0.10.0
Things done
Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Unfortunately, i'm not able to build/test on darwin anymore.
It would be cool if someone could help with that!

Done in #264257.


add 2.10 release notes

Description
adding 2.10 release notes
Check List

[x] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

CI for PR labels is failing, can you fix it? @YANG-DB


How do I remove cydia or how do I get the sources back because when I try to rejailbreak it don't work or when I try to add the sources back I get this message .

How do I remove cydia or how do I get the sources back because when I try to rejailbreak it don't work or when I try to add the sources back I get this message .

You can see the picture here https://twitter.com/michael94700763 .

I fix it by going to More package sources to  get them back.

I have fixed it by watching this video https://www.youtube.com/watch?v=lsFGzCvM4dY .


Feat: Improve interoperability on server-to-server calls+

Summary
Code changes to address some feedback received after using Sunodo tech combined with the latest docker images of rollups-explorer and rollups-explorer-api.
PR reference:

https://github.com/sunodo/sunodo/pull/244

Changes:

Fix the double encoding on the send-raw input form. The for was correctly enforcing the input to be a hex value (e.g. 0x4020) but also re-encoding it as hex before sending it as a transaction argument.
Add optional environment variable to improve interoperability between servers.
A few DX improvements related to typescript autocompletion.

LGTM! I took the liberty to commit a few more changes:

Applied the same fix for the incorrect hex parsing to the EtherDepositForm
Covered this case with unit tests for both RawInputForm and EtherDepositForm.

I hope you don't mind @brunomenezes :)

No problem! you could have included the typos in that commit =). I did a commit fixup, and I will be merging since you approved. it is just a comment typo fix.


Configuration for Multiplier buttons

Bit of a refactor involved, but it all works.
Chosen Multiplier is highlighted in Blue, and there are settings for enabling/disabling any of the multipliers between 1x-10x.

This obsoletes #39


Update Games List.json

Added Call of Duty 4: Modern Warfare
Changed name to "Call of Duty: Advanced Warfare" from "COD: Advanced Warfare"
Changed name to "Call of Duty: Black Ops II" from "COD: Black Ops II"
Changed name to "Call of Duty: Black Ops III" from "COD: Black Ops III"

I'm gonna merge it, but just in case check if it shows the image, if it doesn't make a new pull request

It only shows this.

I figured, looks like the name is only Modern Warfare, is there someone that can check for you if it shows the same name on a console?

No, it shows Call of Duty 4
On Thu, Sep 29, 2022, 20:09 Pablo Reinoso @.***> wrote:

I figured, looks like the name is only Modern Warfare, is there someone
that can check for you if it shows the same name on a console?
—
Reply to this email directly, view it on GitHub
https://github.com/MrCoolAndroid/Xbox-Rich-Presence-Discord/pull/23#issuecomment-1262637893,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3K6R63UXEVR2XWPC72YJFLWAXLNFANCNFSM6AAAAAAQY5J3W4
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm gonna change it to "Call of Duty 4: Modern Warfare" and see what happens

Still the same, maybe changing the image to this link:
https://tinypic.host/images/2022/09/29/59b109c700b500daa9ef3a6769bc8c6f-1.png
would fix it?

I see you joined the Discord, wanna take it there?

Yeah sure


Add a dropdown menu to generate the account extended public/private key

I once felt confused when I saw both Account Extended Private Key and BIP32 Extended Private Key were showed to me, on the BIP44/49/84 tab.
Now I have learned that the BIP32 Extended Private Key is actually a child key of Account Extended Private Key.
I think this tab could be more intuitive, by adding a dropdown menu, which contains four options:

Account Extended Public/Private Key (for Wallet Import/Recovery)

BIP32 Extended Public/Private Key of Receiving Addresses only

BIP32 Extended Public/Private Key of Change Addresses only

Custom BIP32 Extended Public/Private Key

When the 1st option was selected, the External / Internal field would be hidden. The following Derived Addresses view should also show both receiving addresses and change addresses, with clear distinction. Maybe a reminder about actual basis of these derived addresses should also be showed to the user.
When the 2nd/3rd option was selected, the External / Internal field would be visible, but uneditable.  The following Derived Addresses view should explicitly show the address type (receiving or change). The user should also be warned not to import these extended keys into any wallet (because the wallets would tend to derive addresses from a child key of provided extended key).
When the 4th option was selected, the following Derived Addresses view should show some warning to remind the user of deviation from the standard.

Maybe a reminder about actual basis of these derived addresses should also be showed to the user.

The BIP44/49/84 tabs have the text
"The account extended keys can be used for importing to most BIP44 compatible wallets, such as mycelium or electrum."
and
"The BIP32 derivation path and extended keys are the basis for the derived addresses."
Is this not enough explanation?
I see your point that there are potentially many child extended keys that are useful to the user.
Dropdown option number 1 is currently shown as "Account Extended * Key"
Dropdown option number 2 is current shown as "BIP32 Extended * Key"
Dropdown option number 3 is current shown as "BIP32 Extended * Key" when the 'external / internal' field is changed to value 1.
Dropdown option number 4 is currently achieved using bip32 tab.
So I don't see much improvement in UX by having the dropdown. Option 3 changes field values when the dropdown changes which is sorta dangerous I feel, action-at-a-distance type stuff.
I'm open to further comments though.

"The account extended keys can be used for importing to most BIP44 compatible wallets, such as mycelium or electrum."
and
"The BIP32 derivation path and extended keys are the basis for the derived addresses."
Is this not enough explanation?

At least this was not enough explanation for me in the past. I think dumb users like me might get a headache while facing a flood of unrecognised awkward proper nouns :-)
Also, I think it would be more intuitive or educational that:

The following Derived Addresses view should also show both receiving addresses and change addresses, with clear distinction.

I see your point that there are potentially many child extended keys that are useful to the user.

So I don't see much improvement in UX by having the dropdown.

Actually I'm not sure whether you've got my point. Just like what you said:

Dropdown option number N is current shown as "XXXXXX"

That's exactly what I thought - the functionalities are already present, these functions already work well.
I just think it's not intuitive/educational enough, especially for a smatterer like me.
Also, the dropdown option number 2/3 should have some notice/warning, stating that they are generally not intended to be imported into a wallet, otherwise the user would see an empty wallet, or even worse, store funds on non-standard derivation paths, which are not visible to common wallets.

Option 3 changes field values when the dropdown changes which is sorta dangerous I feel, action-at-a-distance type stuff.

Sorry, but I haven't realized how this could be dangerous yet. I think it just works like the autofill feature of browsers - user chooses an option, then the textfields get filled. If you think the value change is not quite visible to the user, I think the dropdown menu entry text could be modified, like:

BIP32 Extended Public/Private Key of External Keychain (Derives Receiving Addresses)

BIP32 Extended Public/Private Key of Internal Keychain (Derives Change Addresses)

Oh, it's tricky that Bitcoin Core didn't use the internal keychain in the past - maybe some other wallets did the same? I don't know. Therefore, showing "receiving/change" would be misleading in this case.

Also, in the title I actually meant "reorganize" rather than "add".
Anyway, again, this tool already works perfectly for these purposes, I just think it might be even more user friendly.


Contribute this Maven plugin to protobuf project

Hi @os72 ,
I was searching for an "official" protobuf plugin for Maven, but I couldn't find any. You did a great job by implementing this version, it seems really useful, but unfortunately since it is not part of the official propobuf project, we can't use it. Did you think of contributing it to protobuf project?

"since it is not part of the official propobuf project, we can't use it"

???
An internal policy allowing to only use 3rd party libraries with their own tooling, but not other 3rd party tooling? That sounds arbitrary :(.
I'm not this plugin's author, I'm only interested in the rationale behind this, I'd like to see what drives this request. If your case is common enough, perhaps the author might consider doing this. As it currently stands, though, it sounds like a silly policy of a company that should, honestly, be solved by changing the policy.


feat: move Use this template to top of ifram

Description

Moved Use this template from description to header.
Restructured template Header to its separate component.

PR fixes following issue(s)
Fixes #15946
Media
Type of change

New feature (non-breaking change which adds functionality)

Testing

How Has This Been Tested?

[x] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview env=release

/ok-to-test

@rahulbarwal QA Done


Testing two sources(brokers) and third invocation for an override on the first

Changes

:broom: Enhancing test coverage of a different scenario

/kind

Fixes #

Release Note

Docs

/lgtm


events

hello can i add the easter event to the server ? i used arkflag_ActiveEvents=Easter but it is not working i also added the forcerespawndinos so they will respawn but it is not working any guide to add the event or it is not possible to do this thanks

Two years later and this command still doesn't seem to work, the docs don't even mention it.

@BakaOppai are you saying that adding arkopt_ActiveEvent=Easter to your instance config doesn't add -ActiveEvent=Easter to the server command-line, or are you saying that adding -ActiveEvent=Easter to the server command-line doesn't activate the Easter event?
You can see what options the ARK server supports at https://ark.gamepedia.com/Server_Configuration#Command_line_arguments.  See also https://github.com/FezVrasta/ark-server-tools#instance-config-files for how to specify those options in the instance config.
Wherever the option is -optionname with no equal sign, use arkflag_optionname=true
Wherever the option is -optionname=value, use arkopt_optionname=value
Wherever the option is ?optionname with no equal sign, use ark_optionname=""
Wherever the option is ?optionname=value, use ark_optionname=value

It adds the command line when I check with ps and all but the event isn't working on the maps

Well after restarting again for no apparent reason it seems its working now, thx anyway.  Tho I have yet to see any easter colored dinos I have a lot of bunny dodos :P


Chore(ci): Test builds also on Node.js v20

I think the build workflow didn't start at all?

I think the build workflow didn't start at all?

I know, this is caused by the conditions set in that pipeline. I run this manually on my machine and it succeeded.


install<EMAIL_ADDRESS>up to supported version

Would fix this bug: https://github.com/storybookjs/storybook/issues/14311

This was done in another PR already, thanks for the contribution 👍


Fix some implicit memory aliasing in for loops

There were 3 findings (in test files) for implicit memory aliasing in for loops:
ecc/secp256k1/fr/element_test.go:375:10: G601: Implicit memory aliasing in for loop. (gosec)
                reduce(&s)
                       ^
ecc/secp256k1/fr/element_test.go:762:23: G601: Implicit memory aliasing in for loop. (gosec)
                                c.Add(&a.element, &r)
                                                  ^
ecc/secp256k1/fr/element_test.go:801:11: G601: Implicit memory aliasing in for loop. (gosec)
                                c.Add(&a, &b)
                                      ^

It should probably be noted that golangci-lint does not run on generated files.

It should probably be noted that golangci-lint does not run on generated files.
!!! really? 😮

Yes, I can't find a good document that states this, but here's the code:
// isGenerated reports whether the source file is generated code.
// Using a bit laxer rules than https://go.dev/s/generatedcode to
// match more generated code. See #48 and #72.
func isGeneratedFileByComment(doc string) bool {
	const (
		genCodeGenerated = "code generated"
		genDoNotEdit     = "do not edit"
		genAutoFile      = "autogenerated file" // easyjson
	)

If the file contains "do not edit", which the generated files do, it will skip it.

And sorry, I realized that I changed the generated files instead of the generator, and that there's probably more than three instances of this problem. It's going take a little while to do it right. Marking it as a draft in the mean time.

So, I believe this is fixed now. Alternatively, we could replace this:
for _, s := range testValues {
    s := s // prevent memory aliasing
    myFunc(&s)
}

with something like:
for i := range testValues {
    myFunc(&testValues[i])
}

I think I prefer the second option, but the first option requires fewer changes.


[Bug]: API GW is reporting CORS issues in EMIT-DEV

Checked for duplicates

Have you checked for duplicate issue tickets?

Yes
Describe the bug
When I deployed the unity-ui application to the emit-dev venue, the application calls the production instance of API GW and reports a CORS issue.
What did you expect?

A clear and concise description of what you expect to happen

I did not expect API GW to report a CORS error since the request is originating from a known unity domain (www.mdps.mcp.nasa.gov)
Reproducible steps

Open your browser's dev tools,
Navigate to https://www.mdps.mcp.nasa.gov:4443/emit/dev/dashboard (fyi, at the time of writing this ticket we were experiencing intermittent 504 issues in the emit-dev venue)
The UI will redirect to the cognito login so you can login (again, at the time of writing this ticket, we were experiencing 504 timeouts after login)
if you are successfully redirected back to the UI application after login, observe in the dev tools that a CORS issue is reported for the API (see screenshot)

What is your environment?
This occurred in the emit-dev venue

I tested the API using Postman and the response I received leads me to believe that the health check API may not yet be deployed to the EMIT DEV venue.


Update NbdevLookup to support import aliases and improve docstrings

This PR adds support to us NbdevLookup on import aliases such as import numpy as np. For example, you can now do the following:
import numpy as np
NbdevLookup(ns=globals()).linkify('this is an aliased import link `np.array`')

'this is an aliased import link [`np.array`](https://numpy.org/doc/stable/reference/generated/numpy.array.html#numpy.array)'

The way this works, is that an alias mapping is created in NbdevLookup:
self.aliases = {n:o.__name__ for n,o in (ns or {}).items() if isinstance(o, ModuleType)}

This maps the alias to its true name. We then use this in the get item function when necessary:
def __getitem__(self, s):
        if '.' in s:
            pre,post = s.split('.', 1)
            if pre in self.aliases: s = f"{self.aliases[pre]}.{post}"
        return self.syms.get(s, None)

However, it required removing lru_cache(None) as a class level decorator since the dictionary for the namespace used to create the aliases of the imports and their true symbol names is not hashable. A work around would be to not do it on the class level and just decorate the functions that need it, but wanted to open up the PR to start a discussion on this as a direction before investing more time into it.

@jph00

I added a commit that separates out the building of the lookup table from the init function of nbdev lookup class, as discussed with @jph00. Also added some test cases that evaluate whether or not the caching is actually working. I think it's ready for primetime!

Super great @ncoop57 ! :D


Integration Test - Release - 2019/05/15 13:44:06

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#backlog'

#pri2

#backlog

#assign PRMergerTest1

#assign NonAllowedUser

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

#reassign PRMergerTest2

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#reassign NonAllowedUser

NonAllowedUser is not a valid GitHub ID, or is not a collaborator on this repo.

#please-close


Docker start failure

I'm testing Vespa locally in a single Docker container. What I'm frequently experiencing is that Vespa, on restart, will not respond. curl requests simply close immediately with no body. The log shows multiple messages like this:
1571165255.993046	vespa-container	569/1	configproxy	configproxy.com.yahoo.vespa.config.proxy.ProxyServer	event	started/1 name=configproxy
1571165256.190677	vespa-container	569/1	configproxy	configproxy.com.yahoo.vespa.config.proxy.RpcConfigSourceClient	info	Could not connect to config source at tcp/vespa-container:19070
1571165256.202582	vespa-container	569/1	configproxy	configproxy.com.yahoo.vespa.config.proxy.RpcConfigSourceClient	info	Could not connect to any config source in set [tcp/vespa-container:19070], please make sure config server(s) are running.
1571165256.748182	vespa-container	569/25	configproxy	configproxy.com.yahoo.config.subscription.impl.JRTConfigRequester	info	Connection to tcp/vespa-container:19070 failed or timed out, clients will keep existing config, will keep trying.
1571165264.858030	vespa-container	569/25	configproxy	configproxy.com.yahoo.config.subscription.impl.JRTConfigRequester	warning	Request callback failed: APPLICATION_NOT_LOADED. Connection spec: tcp/vespa-container:19070, error message: Failed request (No application exists) from Connection { Socket[addr=/<IP_ADDRESS>,port=47834,localport=19070] }

And:
1571165709.808430	vespa-container	1335/41720	config-sentinel	config-sentinel.config.frt.frtconnection	warning	Connection to tcp/localhost:19090 failed or timed out
1571165709.808589	vespa-container	1335/41720	config-sentinel	config-sentinel.config.frt.frtconnection	warning	FRT Connection tcp/localhost:19090 suspended until 2019-10-15 18:55:39 GMT
1571165709.808669	vespa-container	1335/41720	config-sentinel	config-sentinel.config.frt.frtconfigagent	info	Error response or no response from config server (key: name=sentinel,namespace=cloud.config,configId=hosts/vespa-container) (errcode=103, validresponse:0), trying again in 8000 milliseconds

This earlier message seems pertinent:
1571196983.060447	vespa-container	607/24	configproxy	configproxy.com.yahoo.config.subscription.impl.JRTConfigRequester	warning	Request callback failed: APPLICATION_NOT_LOADED.
I've also noticed that even when if the system starts up eventually, HTTP requests will disconnect without a reply while the system is initializing, as opposed to, say, returning 503.
Here is a complete log.

It's unclear what you mean by restart? Restart of the docker daemon, restart of the docker container or restart vespa within the container?
See https://docs.vespa.ai/documentation/docker-containers-in-production.html especially the /opt/vespa/var instead of /opt/vespa/var

Yes, the services believes no app has been deployed  so nothing will start.


Edit README

Edit README
User Story
As an employer
I want to view a potential employee's deployed portfolio of work samples
So that I can obtain a quick summary of his portfolio in the README file

Acceptance Criteria
* Given the portfolio's github address,

* When I view the README for the repository
  Then I can see a professional README that provides information about the coding quiz.

A professional README file is attached.


Move settings to new lines

In Organization settings > Organization settings, we currently have some settings that are on the same line as their label. This is inconsistent with the general pattern.
Instead, the following setting should be moved to a new line beneath their label. The colon at the end of the label should be removed.

@zulipbot claim

@zulipbot claim

Hi @alya. Is this the intended appearance?

Yep, that looks right!

@zulipbot claim

Hi @HiddenOgre28 . I have an open PR for the same issue and am working on this already.

@Manan-Rathi Sorry. I saw it unassigned and didn't see the PRs.

Can I get this as a first contribution?

@zulipbot claim

@zulipbot claim

Hi @alya
Is this okay ?

#20729

@skpatro23 , yes, that screenshot looks fine.

@alya may I work on this issue, if abandoned?


[Bittrex] V3 API

If anyone has the time and knowledge, greatly appreciated!
https://bittrex.zendesk.com/hc/en-us/articles/360027160871-Changelist-04-19-2019
https://bittrex.github.io/api/v3

I hope to be able to look at this.  They've finally got the order history in there which is pleasing.  Client order ids too.

@npomfret How are you getting along with this?


Cross compiling with Hunter

I have a cmake crossplatform c++ project which uses Protobuf and has to support at least Linux, iOS and Android. Cross compiling protobuf is not the issue, but the project setup gets kind of messy when cross compiling libprotobuf for target (i.e iOS) and having protoc on host (i.e macOS) available to generate source files from .proto in cmake.
I had a custom setup with a hierarchy of ExternalProject's where the outer-most project uses default toolchain and creates protoc for host and the actual project for target with custom toolchains for iOS
and Android.
As you see, this is hard to maintain, so i am investigating polly's toolchains with hunter for dependencies (Protobuf).
I was able to adapt the hunter package of Protobuf to skip compiling protoc when cross compiling and import a user specified host-protoc so everything works fine. Only problem is, that i need to supply the instance of protoc which i would like to also have as a hunter dependency.
Do you guys have some idea, how i can manage hunter dependencies for different toolchains?
If there is some mechanic in hunter that i could extend for that purpose or if there is some other way to achieve this with hunter, please let me know.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

I haven't worked with protobuf but as far as I understand you need to have protobuf library and some protoc executable tool, right? I.e. you need some kind of mix:

Linux host + Android target: protoc tool compiled with Linux host toolchain + protobuf lib compiled with Android toolchain
OSX host + Android target: protoc tool compiled with OSX host toolchain + protobuf lib compiled with Android toolchain
OSX host + iOS target: protoc tool compiled with OSX host toolchain + protobuf lib compiled with iOS toolchain
OSX host + OSX target: protoc tool and protobuf lib compiled both with OSX host toolchain

Is my vision correct?

Thx for the fast response!
Your vision is totally right.
Also the hunter package already builds the library and the executable (obviously for the same target). I would imagine, that i need to switch toolchains in code, or supply a separate host toolchain for hunter_add_package...

Your vision is totally right

It is good :) Another good news is that it's important feature and I have a plan to implement it.
However the bad news is that it's quite tricky and I can't say in will be done in near future. This is something that is not designed in CMake itself: https://cgold.readthedocs.io/en/latest/overview/cmake-can-not.html#incomplete-functionality-coverage

supply a separate host toolchain for hunter_add_package

Kind of. You need to build protoc using host toolchain (I guess default will be OK) first, then feed path to host protoc to the target toolchain. I guess the next question will be how to connect them in automated fashion. You can try to run host toolchain from target CMakeLists.txt by
execute_process(
    COMMAND
    "${CMAKE_COMMAND}" -Hhost "-B${CMAKE_CURRENT_BINARY_DIR}/host-build"
    ...
)

This will trigger host build and install protoc. You can save path to current host-root directory in plain text in some file like "${CMAKE_CURRENT_BINARY_DIR}/host-build/path.to.protoc" and read this file in target toolchain.
Host toolchain (host/CMakeLists.txt):
hunter_add_package(Protobuf)
file(WRITE "${CMAKE_BINARY_DIR}/path.to.protoc" "${PROTOBUF_ROOT}/bin/protoc")

Target toolchain (top-level CMakeLists.txt):
execute_process(...)
file(READ "${CMAKE_CURRENT_BINARY_DIR}/host-build/path.to.protoc" host_protoc) # find host protoc
execute_process(COMMAND "${host_protoc}" ...) # use it

I haven't tried it myself, let me know how it will work if you wish to try.

You can try to run host toolchain from target CMakeLists.txt by
The idea is similar to what i did before hunter integration, but i used ExternalProject and got all kind of problems with toolchains partly passed through and order of dependency execution...

Your approach with execute_process could be the simple and more elegant solution. I will try this out and report back.
What would be the preferred way to make this kind of a more general solution (to play nice with expectations of other hunter users)?

Make building protoc optional for all
Assume protoc is only relevant for host (correct for Android and iOS but but maybe not for other cross compiling cases)
Have all Targets as protobuf:: and protobuf_host:: variants (which would be just alias when target==host)

How to figure out we want to cross compile at all?

Android toolchains sets CMAKE_CROSSCOMPILING
iOS toolchain only sets IOS
only make special cases for those two?

Anyways, i guess i will start to try this out and figure out the details if everything work. Thanks again for your input!

Just figured out that your approach is all client/user side and not in the package

Yep, I think package should not contain this stuff. It may looks solid in this case when you can say that "we are building executable hence it's not for target but for host", but it may be not true for other packages. Though I can't provide any examples. Another problem with this approach is about CMake itself, just like I say there is no support of two toolchain in one project, so it's simply not possible - we have to run build twice anyway.
However what project should provide is an option to do optimal build. E.g. if we have foo_lib for target library and foo_tool for host executable there should be option FOO_BUILD_LIB and FOO_BUILD_TOOL:

  option(FOO_BUILD_LIB "Build library" ON)
  option(FOO_BUILD_TOOL "Build tool" ON)

  if(FOO_BUILD_LIB)
    add_library(foo_lib ...)
  endif()

  if(FOO_BUILD_TOOL)
    add_executable(foo_tool ...)
  endif()

Next cases:

native development: by default both options are ON and we install everything to host root
while cross-compiling: set FOO_BUILD_LIB=OFF and use host toolchain to install tool
while cross-compiling: set FOO_BUILD_TOOL=OFF and use target toolchain to install library

I've openned feature request here: https://github.com/ruslo/hunter/issues/495

I know this isn't constructive feedback, but I've been keeping an eye on support for cross compiling as well. The hosts and targets we support at $DAYJOB are:
Hosts:

Windows x86
Linux x86

Targets:

Windows x86 (MSVC toolchain)
Linux x86 (GCC or Clang toolchain)
Android ARM (NDK-supplied toolchain)
Android x86/Intel (NDK-supplied toolchain)

This particular issue is about mixing both toolchains in one project, i.e. using project simultaneously for the target and host. For example if you're working on Linux and create package for Android and you want package Foo to install some stuff for Linux x86 and Android arm in one shot.
@rcdailey Is this your case?

Yes that's it.
On Sat, Aug 20, 2016, 2:56 PM Ruslan Baratov<EMAIL_ADDRESS>wrote:

This particular issue is about mixing both toolchains in one project,
i.e. using project simultaneously for the target and host. For example if
you're working on Linux and create package for Android and you want package
Foo to install some stuff for Linux x86 and Android arm in one shot.
@rcdailey https://github.com/rcdailey Is this your case?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ruslo/hunter/issues/494#issuecomment-241220406, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABr6dlYEke3sJ3jyFoG-SIkjn2Hu05Pyks5qh1vagaJpZM4JoW6m
.

Yes that's it

@rcdailey Can you provide details? Does it fall at any cases described here or it is something new?

Here is some detail on specifically what we do. We support a few different platforms & toolchain combinations:

Windows + MSVC
Windows + Android NDK (armeabi-7v / gcc 4.9)
Linux + GCC 4.9 x86
Linux + Android NDK (armeabi-7v / gcc 4.9)

Simply put, on a given platform we need to be able to use CMake "normally" (on Windows, use Visual Studio; on Linux, use selected make program + C/CXX env variables to define toolchain), and also specify the toolchain manually for cross compilation on Android using the NDK's prebuilt toolchains. We use a custom version of takanome's CMake toolchain for Android NDK to give CMake the information it needs to find and use the Android NDK toolchains.
For each combination of platform and toolchain, we generate 1 time in CMake. There is a separate CMake binary directory for each combination. In other words, we do not support swapping the toolchain after generation. You must clear the CMake cache and generate all over again.
As mentioned before, we have separate manual builds of third party libraries for every platform+toolchain combo. What I expect from hunter is the ability to automate builds for these third party libraries, but allow me to specify the toolchain to use when building these libraries. This specifically covers the Android NDK case, where without this functionality, hunter would have no idea what toolchain to use.
If it can somehow use the same configuration that CMake is using, that'd be great. But I haven't used hunter at all yet so I really have no idea how it works so far. This is my general feedback, though.
I hope that is helpful and clear enough. Let me know if you have other questions.

I hope that is helpful and clear enough

It is perfectly clear. I'm using the same Android toolchains for Hunter. I haven't used it on Windows host (usually it's Linux host/Android target, OSX host/Android target) but it should work I guess.
@rcdailey I'm still thinking that you misunderstood my question. This particular issue is about doing package management for both host and target toolchain simultaneously. For example if you have next CMakeLists.txt:
find_package(foo REQUIRED CONFIG)

With Hunter you can do:
hunter_add_package(foo)
find_package(foo REQUIRED CONFIG)

Then for your cases:

Start this code on Windows with "-G Visual Studio" and this code will install foo using Visual Studio (target and host toolchains matches)
Start this code on Windows with "-DCMAKE_TOOLCHAIN_FILE=android" and this code will install foo using Android toolchain (target Android, host Windows)
Start this code on Linux with "-G Unix Makefiles" and this code will install foo using Makefile (target and host toolchains matches)
Start this code on Linux with "-DCMAKE_TOOLCHAIN_FILE=android" and this code will install foo using Android toolchain (target Android, host Linux)

This issue about packages that need to be installed for both target and host, i.e. when you use this CMakeLists.txt it should do:

Start this code on Windows with "-DCMAKE_TOOLCHAIN_FILE=android" and this code will install foo using Android toolchain and foo using Visual Studio
Start this code on Linux with "-DCMAKE_TOOLCHAIN_FILE=android" and this code will install foo using Android toolchain and foo using Makefile

In this case there will be two root directories. We will take some components from foo installed on host and use them for building foo for target.
If you take a look at this pull request you may see that by hunter_add_package(Protobuf) we do install protoc tool for host platform, then protoc used in the CMakeLists.txt for helping to use protobuf library for the target platform.

You're right, it does sound like I'm misunderstanding. To make sure I get it now, what you're saying is that for each single CMake binary directory, you want to do a 2 pass with hunter: Build & install for host and then again for provided toolchain?
This is, in effect, the same as having 2 cmake binary directories, one generated for host and one generated against toolchain file. Is this correct?

you want to do a 2 pass with hunter: Build & install for host and then again for provided toolchain?

For this particular issue yes. Note that the files may differs. E.g. you want to build OpenCV, Boost and Protobuf for Android. You can use Android toolchain and Hunter packages:
hunter_add_package(OpenCV) # install Android library
hunter_add_package(Boost) # install Android library
hunter_add_package(Protobuf) # install Android library

but additionally you must have separate host CMakeLists.txt:
# host/CMakeLists.txt
hunter_add_package(Protobuf) # install 'protoc' tool for host

It should be executed before we use Protobuf and information about 'protoc' should be transmitted to target toolchain.

This is, in effect, the same as having 2 cmake binary directories, one generated for host and one generated against toolchain file. Is this correct?

Yes. But the question about how they will know about each other. To be accurate: target build need to know information about host root directory. That's the issue now.

It's good to hear that hunter already supports the scenario I need. My case is simple: I just need to build the libraries for target toolchain and link to them. Thanks for explaining @ruslo!

Closing inactive discussion.


Incorrect nodeKind for prov:wasRevisionOf in commons/entity/v0.1.0.json

The relevant section of https://github.com/INCF/neuroshapes/blob/793326dbb8e42fc56ca716fbe3c240080d49b3c8/modules/commons/src/main/resources/schemas/neurosciencegraph/commons/entity/v0.1.0.json is
        {
          "path": "prov:wasRevisionOf",
          "name": "Revision of",
          "description": "The entity for which this entity is a revision of.",
          "nodeKind": "xsd:dateTime",
          "maxCount": 1
        }

I don't think "nodeKind": "xsd:dateTime" is correct

Hi,
Thank you for reporting this. I made a fix


heterogeneous array

Is there a way to specify an array with items having different types? E.g. for data
['square' 1 1 4 4]
['circle' 1 1 3]

the schema should match either array of string 'square' + four numbers, or array of string 'circle' + three numbers. It should not match array of 'circle' + four numbers, or 'square' with three.
I haven't found a recursive Array declaration, only Array (match class only), Array.of (match homogeneous elements), and Array.like (exact match).

:+1: :+1: :+1:
it sounds like you're looking for typed tuples. i need something similar - union types. eg.
{
  foo: Array.of(String),
  bar: Array.of(Number),
  baz: Array.of(String.or(Number))
}


Browser refresh required for changes to appear

When I click "Run" (or type Ctrl+Enter) the preview disappears.  Data appears to be saved but I need to refresh the browser window in order to see changes.
$ google-chrome --version
Google Chrome 48.0.2564.103
$ cat /etc/lsb-release
DISTRIB_ID=ManjaroLinux
DISTRIB_RELEASE=16.06.1
DISTRIB_CODENAME=Daniella
DISTRIB_DESCRIPTION="Manjaro Linux"

Cool project!  I'd love to use it.

@wulftone have you tried using the Reset button: Ctrl/Cmd + Shift + Enter? Does that solve the problem?

Yes, that works, thank you!  I should have been more experimental.  Why have the two separate options?  What's the point, if I may ask?

Not entirely sure to be honest. I was looking at the source and it seems as if the play button tries to persist the state of every database except for @session, thereby only re-evaluating blocks tht use @session, however it overwrites @browser too. On the other hand, a reset re-evaulates the entire document.
Someone with more experience may be able to shed more light on this.

@Pranay: Close! The "run" button reruns your new code with the exact same
state you currently have (that is, every fact you've committed is still
just as you left it). The reset button on the other hand throws away your
entire state and starts over from scratch. In the event that you need to
reset your state, or get a new block to evaluate that does an unconditional
commit, then you'd want to use the reset button.
On Sun, Nov 6, 2016 at 2:25 PM Pranay Prakash<EMAIL_ADDRESS>wrote:

Not entirely sure to be honest. I was looking at the source and it seems
as if the play button tries to persist the state of every database except
for @session, thereby only re-evaluating blocks tht use @session, however
it overwrites @browser too. On the other hand, a reset re-evaulates the
entire document.
Someone with more experience may be able to shed more light on this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/witheve/Eve/issues/527#issuecomment-258703779, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AATKDn9CWrqbokMSrpgLH8gQZWxbMe6aks5q7imugaJpZM4Kqnvq
.

Had same issue, extremely confusing, I thought it some sort of a bug until googled it and found this tread.


[firebase_admob] Exposed Ad Banner width and height to flutter

Is your feature request related to a problem? Please describe.
There is no way to know the width and height of the smart ads in flutter and banner is always at the bottom of the screen. I need a way to add bottom margin of the content to avoid overlapping with the banner.
Describe the solution you'd like
Expose the width and height from the native code.
e.g.
int widthPixels = AdSize.SMART_BANNER.getWidthInPixels(this);
int heightPixels = AdSize.SMART_BANNER.getHeightInPixels(this);

Previous discussion: https://github.com/flutter/flutter/issues/19832#issuecomment-530817975

The firebase_admob plugin will be deprecated in April 2021 in favor of Google Mobile Ads SDK for Flutter.
Google Mobile Ads SDK for Flutter is a new Flutter plugin that supports more Ads formats than firebase_admob does. Google Mobile Ads SDK for Flutter currently supports loading and displaying banner, interstitial (full-screen), native ads, and rewarded video ads across AdMob and AdManager. It also supports displaying banner and native ads as Widgets as opposed to being overlayed over all app content.
We're now recommending projects currently using firebase_admob migrate to Google Mobile Ads SDK for Flutter following the instructions outlined here.
Issues with migrating or with using the new package should now be raised over on the Google Mobile Ads SDK for Flutter repository.


Efficient deserialization for Buffer

This implements ArrowDeserialize for Buffer as part of https://github.com/DataEngineeringLabs/arrow2-convert/issues/101
There are a few hoops to jump through because the BinaryArray<i32> normally exposes an iterator which is &[u8] slices, whereas we want to expose a Buffer primitive directly to take advantage of the Arc behavior of arrow Buffers.

Looks like the Buffer-oriented optimization introduced a bug. Will sort it out tomorrow.

Codecov Report

Merging #103 (3d6e7c0) into main (b2fd8f8) will decrease coverage by 0.31%.
The diff coverage is 73.91%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #103      +/-   ##
==========================================
- Coverage   94.63%   94.33%   -0.31%
==========================================
  Files           8        8
  Lines        1565     1588      +23
==========================================
+ Hits         1481     1498      +17
- Misses         84       90       +6

Impacted Files
Coverage Δ

arrow2_convert/src/field.rs
100.00% <ø> (ø)

arrow2_convert/src/deserialize.rs
96.36% <73.91%> (-3.64%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

It could be nice to cover Buffer<u8> in arrow2_convert/benches/bench.rs too

It could be nice to cover Buffer<u8> in arrow2_convert/benches/bench.rs too

Interestingly these benches show this work might not have actually been all that necessary. Vec<u8> looks like it has same constant-time behavior as Buffer<u8>.


[feature] Find file from Library stored in GD

Example command: !d termodinámica final --> Bot sends the latest final test for this course in PDF format.
A premium feature could be to specify the amount of tests: !d termodinámica final 2.
Chart flow:

Use stringify
Match the name with the subfolder names (XXX999) inside semester folders
Get the last modified file
Send it

added on 7940251f7ad49a78d9d78f3eecf217eb5c6c34d2


HistorySetSampling does not handle the HistorySet correctly

Issue Description
What did you expect to see happen?
For given HistorySet, the code should run successfully.
What did you see instead?
The code crashed:
Traceback (most recent call last):
  File "/Users/wangc/projects/wangc/raven/framework/Driver.py", line 281, in <module>
    raven()
  File "/Users/wangc/projects/wangc/raven/framework/Driver.py", line 234, in raven
    simulation.run()
  File "/Users/wangc/projects/wangc/raven/framework/Simulation.py", line 798, in run
    stepInstance.takeAstep(stepInputDict)
  File "/Users/wangc/projects/wangc/raven/framework/Steps.py", line 312, in takeAstep
    self._localTakeAstepRun(inDictionary)
  File "/Users/wangc/projects/wangc/raven/framework/Steps.py", line 466, in _localTakeAstepRun
    model.collectOutput(finishedJob,output)
  File "/Users/wangc/projects/wangc/raven/framework/Models/PostProcessor.py", line 240, in collectOutput
    self.interface.collectOutput(finishedjob,output)
  File "/Users/wangc/projects/wangc/raven/framework/PostProcessors/InterfacedPostProcessor.py", line 198, in collectOutput
    output.load(evaluation['data'], style='dict', dims=evaluation['dims'])
  File "/Users/wangc/projects/wangc/raven/framework/DataObjects/DataSet.py", line 435, in load
    self._fromDict(fileName,**kwargs)
  File "/Users/wangc/projects/wangc/raven/framework/DataObjects/DataSet.py", line 1257, in _fromDict
    self.asDataset()
  File "/Users/wangc/projects/wangc/raven/framework/DataObjects/DataSet.py", line 260, in asDataset
    return self._convertToXrDataset()
  File "/Users/wangc/projects/wangc/raven/framework/DataObjects/DataSet.py", line 1074, in _convertToXrDataset
    raise e
ValueError: setting an array element with a sequence.

Do you have a suggested fix for the development team?
Please attach the input file(s) that generate this error.  The simpler the input, the faster we can find the issue.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or improvement?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest stable branch? If yes, is there any issue tagged with stable (create if needed)?
[x] 5. If the issue is being closed without a merge request, has an explanation of why it is being closed been provided?

The issue does not cause bad results but a crash of the code... No email needed.
Closure checklist passed...


View Menu : UX Ideas

Currently we have quite number of views and view actions under Views menu and making it cumbersome. We should organize them better and effectively.

cc: @bpasero @gregvanl @stevencl

One suggestion is to move views under main section and move other actions like toggle to secondary section.

@stevencl @bgashler1 @chrisdias fyi we should make a decision. I do like the idea of moving the views and panel back to be top level and the other stuff into a sub menu.

I think Views can be used as a top leve menu.

Does not work for me if we also have a View menu.

UX meeting decision:


Scatter plot viz

A basic scatter plot viz is made but it needs more works, e.g.,:
a) add line plot
b) add density plot
: note that scatter, line, and density plots are handled at the same plot component,
c) typing (typescript): some are currently assigned as any
d) probably add plot controls: this may be handled in the other PR

a) added lineplot: note that it does not work at this stage because the backend respond incorrect data format starting with scatterplot, not lineplot. Discussed with Danielle and she confirmed the issue and would fix it
b) changed axes labels as selected X and Y variables' displayName
c) merging main: fixed error on scatter plot viz caused by merging


Header at the bottom

Is your feature request related to a problem? Please describe.
The problem is that chips card + tab header make a big header not ergonomic
Describe the solution you'd like
May be chips card can automaticaly and some code to do like this : https://github.com/UI-Lovelace-Minimalist/UI/issues/346

It's a theme feature, a close one is opened here: https://github.com/piitaya/lovelace-mushroom-themes/issues/1
I set won't fix 'cause it's a won't fix here.


Subtitles appear late when using HLS streaming

Bug
Open streaming video in some web/desktop player, not react-native-video - observe that subs are in sync
Open streaming video in app - observe that subtitles appear late (~2s delay)
Open regular video - observe that subs are in sync

Platform

Which player are you experiencing the problem on:

iOS
Android ExoPlayer
Android MediaPlayer
Windows UWP
Windows WPF

Environment info

React native info output:
 // paste it here

Library version: x.x.x
Steps To Reproduce

...
Expected behaviour

Reproducible sample code

Video sample
If possible, include a link to the video that has the problem that can be streamed or downloaded from.

Maybe I'm imagining it, but on android it seems that now it is more in sync after https://github.com/react-native-video/react-native-video/pull/2317

Seems like we needed to add some metadata about timestamps, vtt was not enough


Refactoring Fluid Mesh Rebuild Time

So, this will be something I work on after the release of Alpha 1.0. But it is just something that I noticed after I increased the size of the "Dream Space" world and tried removing fluid source blocks.
 The reason being is that the fluid mesh is totally rebuilt every time a change is made. The chunk template that describes the resulting mesh data is cached and then used to rebuild the entire mesh.
This is so the fluid mesh can be one big mesh and allow for the fluid shader effects to properly work.
So, here are my ideas going forward with this:

See if there is a way not to have to combine the fluid mesh together for the shader effects to work (though I doubt this would work I could be very wrong though and will need to do more research)
Create a way to parallel process the chunk templates and build the resulting fluid mesh.

How would I go about the second option?

First, after each chunk mesh is generated cache all the sizes of the resulting arrays.
Second make sure all chunk templates are synced in the multiple fluid builder threads.
When it comes time to build the mesh build the new one first, then create an array with a Shared Array Buffer that would be big enough to store the needed mesh data [positions, indices, colors...).
The world thread would send to each fluid builder what chunk to rebuild and the index and length to start adding to the shared mesh data array. The size of the array would only be calculated after rebuilding the new fluid meshes.
Once complete it would just send the data over to the render thread like normal.

Either way, I will need to do so more research and thinking before I start on this. I just wanted to get this as an issue to let people know that I know it is problem and that I am working on solving it.
Also, another big problem to consider when dealing with the fluid mesh. How will the engine handle building the fluid mesh when on a server and giving that data to a client?
Maybe the mesh is built per-client connected rather than one just per world. But that is something I will have to deal with later.

Currently, the only thing that I can find related to not combing the meshes directly is this:
Seamless, endless, and performant custom water using skinned meshes and PBR

I asked a question about this on the  Babylon.Js forum about this:
Link To Question
I guess I will see if anyone has an answer before I dive into my other ideas.

So, I did get an answer and with that, I was actually able to do what I wanted to do.
I can actually totally get rid of the Fluid Builder thread!
The problem was that I was using the mesh space position now the world space position for the noise function to produce the waves.
I am pretty happy and will include this update in the Alpha 1.0 release.
I will close this issue once I make the engine no longer use the Fluid Builder thread.

All right I got rid of the fluid builder in the latest commit.
I updated all the examples with the new code
So, I am closing this out.


Allow rerunning startReplaceSegment with the same segmentsTo if the revious protocol fails in the middle (lineage entry in IN_PROGRESS status).

This pr enhance current startReplaceSegment api to cover the following case:
Initial state:
lineageEntry1: { segmentsFrom: [], segmentsTo: [s1, s2], status: IN_PROGRESS}

Restart the protocol with the same segmentsTo will go through and revert lineageEntry1:
lineageEntry1: { segmentsFrom: [], segmentsTo: [s1, s2], status: REVERTED}
lineageEntry2: { segmentsFrom: [], segmentsTo: [s1, s2], status: IN_PROGRESS}

Note:
The segment deletion in startReplaceSegment api is async, the client side should wait till the clean up is finished before uploading the new segments.

Codecov Report

Merging #8639 (763e104) into master (9873ed9) will decrease coverage by 3.00%.
The diff coverage is 85.09%.

@@             Coverage Diff              @@
##             master    #8639      +/-   ##
============================================
- Coverage     69.46%   66.45%   -3.01%
+ Complexity     4411     4328      -83
============================================
  Files          1669     1261     -408
  Lines         87387    63837   -23550
  Branches      13092     9919    -3173
============================================
- Hits          60700    42423   -18277
+ Misses        22450    18404    -4046
+ Partials       4237     3010    -1227

Flag
Coverage Δ

integration1
?

unittests1
66.45% <85.09%> (+0.05%)
:arrow_up:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ion/function/FirstWithTimeAggregationFunction.java
49.27% <49.27%> (ø)

...gregation/function/AggregationFunctionFactory.java
80.91% <60.00%> (-2.71%)
:arrow_down:

...n/FirstDoubleValueWithTimeAggregationFunction.java
100.00% <100.00%> (ø)

...on/FirstFloatValueWithTimeAggregationFunction.java
100.00% <100.00%> (ø)

...tion/FirstIntValueWithTimeAggregationFunction.java
100.00% <100.00%> (ø)

...ion/FirstLongValueWithTimeAggregationFunction.java
100.00% <100.00%> (ø)

...n/FirstStringValueWithTimeAggregationFunction.java
100.00% <100.00%> (ø)

...che/pinot/segment/spi/AggregationFunctionType.java
89.41% <100.00%> (+0.12%)
:arrow_up:

...va/org/apache/pinot/core/routing/RoutingTable.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/org/apache/pinot/common/config/NettyConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 616 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 b317246...763e104. Read the comment docs.


[Vim sqlite-lua] Insert of sqlite binary is not a string

The sqlite-lua override is incorrect and doesn't work because the substitution does not place the path as a string in the file causing an error when loading the sqlite/defs.lua file. This is the line that I have in my store: path = vim.g.sqlite_clib_path or /nix/store/g2i5j8i0bwfih7hg2wbc15k2969bckn3-sqlite-3.46.0/lib/libsqlite3.so.
I think it's simply a missing wrap of the ${lib.escapeShellArg libsqlite} value.
https://github.com/NixOS/nixpkgs/blob/86e78d3d2084ff87688da662cf78c2af085d8e73/pkgs/applications/editors/vim/plugins/overrides.nix#L2018
@khaneliman

Thanks for reporting. I'll take a look real quick.

Does the PR resolve this for you?

@khaneliman let me check! But it looks like it should work.

Looks like I came across this being broken during a nixvim tests, too... just forgot to come back to it. https://github.com/nix-community/nixvim/blob/a20fbbc4b9665ec215e7bea061a1d64f6fd652ce/tests/test-sources/plugins/by-name/yanky/default.nix#L63-L65 Looks like it fixes that test when I point at the PR branch. Thank you very much for identifying the issue!

Yeah seems my build now works properly :) Thanks for the quick response!


Added GOPATH/bin to PATH in install.md

Aded GOPATH/bin to PATH in install.md
Signed-off-by: TomSweeneyRedHat<EMAIL_ADDRESS>Addreses #1407.   When doing a make, make clean, make, an error would arise concerning easyjson.  Adding the $GOPATH/bin to the $PATH environment variable clears this up and per @mheon should have been in the docs for a while now anyway.

LGTM

LGTM

@TomSweeneyRedHat Fix your commit message and we can get this merged.

@rhatdan Thought I'd fix that up prior, tried again with a new push.

@rh-atomic-bot r+


Fix verify Kubespray e2e in cluster reset step

What type of PR is this?
/kind failing-test

What this PR does / why we need it:
Fix verify Kubespray e2e in cluster reset step
This is a continuation of the previous one https://github.com/kubean-io/kubean/pull/1094
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm

/lgtm


Nginx installed in wrong directory path name

Cookbook version
chef_nginx (4.0.2)
Chef-client version
12.16.42
Platform Details
Test-Kitchen w/ Docker using ubuntu:14.04
Scenario:
Install Nginx from source with PageSpeed module
Steps to Reproduce:
run cookbook as such:
node.override['nginx']['source']['version'] = '1.11.5'
node.override['nginx']['source']['modules'] = %w(chef_nginx::pagespeed_module)
node.override['nginx']['source']['checksum'] = '223f8a2345a75f891098cf26ccdf208b293350388f51ce69083674c9432db6f6'
#node.override['nginx']['source']['prefix'] = "/opt/nginx-#{node['nginx']['source']['version']}"
node.override['nginx']['source']['sbin_path'] = "#{node['nginx']['source']['prefix']}/sbin/nginx"
node.override['nginx']['source']['url'] = "http://nginx.org/download/nginx-#{node['nginx']['source']['version']}.tar.gz"

include_recipe 'chef_nginx::source'

Expected Result:
Nginx v 1.11.5 installed with pagespeed module in /opt/nginx-1.11.5/
Actual Result:
Nginx 1.11.5 installed with pagespeed in /opt/nginx-1.10.1

@drahamim I believe you missed one final derived attribute
default['nginx']['source']['default_configure_flags'] = %W(
  --prefix=#{node['nginx']['source']['prefix']}
  --conf-path=#{node['nginx']['dir']}/nginx.conf
  --sbin-path=#{node['nginx']['source']['sbin_path']}
)

That's what is being passed into the configure && make && make install block.

@Sauraus is right here. Derived attributes are pretty awful and this requires setting both attributes. If we could do it all over we'd do it differently, but this was all before my time.


Error after installing & setting up

After installing and following the setup guide carefully i still get ERROR below :: What am i not doing right ??
Datatable is not a valid SSR component. You may need to review your build config to ensure that dependencies are compiled, rather than imported as pre-compiled modules

There is no SSR support at this time.
It must be installed in DevDependancies.
npm i -D svelte-simple-datatables
Then it will be compiled with the app
https://github.com/sveltejs/sapper-template#using-external-components


as soon as I try to do anything

Traceback (most recent call last):
File "palworld_server_toolkit\editor.py", line 1863, in delete_player
File "palworld_server_toolkit\editor.py", line 4572, in DeletePlayer
File "palworld_server_toolkit\editor.py", line 2666, in GetPlayerGvas
File "palworld_save_tools\palsav.py", line 33, in decompress_sav_to_gvas
Exception: incorrect compressed length: 2503

but in the same saves made an hour before it worked


Define a default environment by Processing Organization

As an end user, I sometimes need to define and use default parameters available by processing organization.
For example, when pricing the collateral value of a GC Repo, it would be very useful to use a QuoteSet that would have been defined at PO level.
A solution for that can be to define a new "ProcessingOrganizationDefaults" object, having its own GUI, so defaults values can be defined at PO level. A first parameter on this object could be a default Pricing Parameters Set  (that can be optionally set ).

This ticket is being implemented, few notes:

it will be subject to the segregation rule by PO
a first version will contain a QuoteSet (and not a Pricing Parameters Set as only a QuoteSet is needed for collateral pricing)
it will use the module mechanism to have different parts, depending on the product type(s) configured

Tested OK.

Note : A Processing Org Defaults "Collateral Management" module has been created, it is only available when the Security module is installed.
This module contains a Quote Set from where the securities's prices can be searched when calculating exposures.


Logic operator's execute order is wrong

Example:
a || b || c

In Ruby if b is not a false value, than the expression returns.
But in Goby it'll execute like (a || (b || c)). This is because we treat the operators as method instead of special keyword.

at least operators works in goby ++
> 1 && 0 || 1
 => 0  WTF RUBY???
> 1 and 0 or 1
 => 0  WTF RUBY???


Example of using ngReact with a third-party React component

I'm trying to use ngReact to implement the Airbnb react-dates calendar, and so far my only successes have been getting the component to render on the page via reactDirective, and then being able to set initial prop values and receiving events from the component. Beyond that, it's difficult, for me at least, to understand how to trigger the component to render with the new state, since all of the examples I can find involve components that are 100% dictated in the Angular app. An example of using ngReact with a third-party, npm-linked component would be very helpful.

@3dg4r your that that would be good with an example that better illustrated how to  use a third-party component.
Any help with that would be great.

@3dg4r Hey, I've just implemented airbnb's react-date using ngReact.
Was relatively fine, but I did feel like I was creating an abomination 😥 .
About your question, we need to differ between two cases:

Changes within the React component itself
Changes within the containing Angular app

When we're using this component (the date picker) we need to understand that it has its own state (it's not a stateless component), therefor whenever we perform a change of the component's state (i.e. change date) the date picker will render itself without "notifying" angular (you can check it with ease, add a console.log to the head of the render function) . This is the first case.
The second case is a change originated in the containing app.
Assuming we have some object in the controller ($scope.myObj = {a:1}) that we pass as props to our component <react-component name="myComp" props="$ctrl.myObj"> angular should update every DOM element that uses $ctrl.myObj.
However, this doesn't work for react-component (at-least not for me) so an easy solution was to wrap the component with an ng-if, and whenever there's an update toggle the boolean value for the if.
 For me it's not an issue as I've got a loading screen covering my whole view on each update thus there's no "blink" of the react component.

Have you implemented a third party reusable component with ngReact? if so can you share it with me

Do you have any example of using 3rd party react components published in npm ???? i too need it urgently kindly share it please

i want to use that 3rd party component with ngReact in angular 1.6.x


Scrollbar fix

Before:

After (only after hovering the mouse over):

@elopez take a look, merge if it's ok

@elopez I am not sure if it is the best way to achieve this (https://github.com/trailofbits/testing-handbook/pull/31/commits/b9cba8201e4fe8f0e290b298deca7a6d92a868a1), but you can see Firefox/Safari/Chrome:


Create a new pull request by comparing changes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.03%) to 94.799%

Files with Coverage Reduction
New Missed Lines
%

js/src/dom/event-handler.js
1
92.27%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
1980

Relevant Lines:
2041

💛  - Coveralls


Create rule DDF00018: The study protocol version should match the ScheduledTimeline version (aka the schedule of activities).

Create rule defined for #20

@nickdedonder I created this PBI for this non-executable rule that we discussed last week.  This rule definition has been created in the rule editor and has been flagged as "Not Executable". I have therefore marked this PBI as Done and added a new "not executable" label so that we can find it (and similar non-executable rules) in the future.  I assigned the Epic as "CORE-DDF: Create executable DDF rules" even though it is not executable - I hope this is OK.


Chained Geocoder doesn't indicate which provider was used

It would be potentially useful to indicate in some way which provider successfully returned data.  If the library doesn't construct a proper address for example while reverse geocoding, that info can at least be used to help parse the returned string.

Based on the argumentation, is this still desired, since we now have structured address, so no need to parse the string with flat representation of the address?

I think this would still be useful and I'm sure the information is available but closing for now since there is a solution for my original concern.


Update tokens structure

Current tokens structure have few limitations. For example links (and all other elements using typography.color) are not automatically inheriting color from app. There's few questions I like to discuss

Simplest thing could be done is moving tokens from typography.color to color, so app colors applying without any user action. But there's a chance that user might want to have separate colors for his typography. For example, we has blue as his color.primary but he wants lightblue palette to be applied on typography.
What about purely typography tokens, like typography.fontSize, typography.letterSpacing, typography.fontWeight, typography.lead? Currently they also scoped to typography. Which means that if user installs Typography, possibly could have 2 separate typography tokens, because he could have fontSize, letterSpacing, fontWeight, lead already. Does it make sense to enforce Typography tokens if user decide to install package?
I would rename lead to lineHeight

I've started PR here, but then those questions appeared - https://github.com/nuxt-themes/typography/pull/19

What you think @Tahul @Atinux

One solution would be to inherit typography.color.primary from color.primary so the user can customise the typography color if needed.
What do you think @bdrtsky  ?

Great suggestion, I've pushed it https://github.com/nuxt-themes/typography/pull/19/commits/863c5da5bec38f8e12ae066e49e7f4178294dfff


update LoadAwareScheduling proposal

Signed-off-by: Tao Li<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
Update resource usage estimation in the scoring algorithm and LoadAwareSchedulingArgs
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews

Codecov Report

Merging #155 (2909cdf) into main (889465a) will decrease coverage by 0.05%.
The diff coverage is n/a.

:exclamation: Current head 2909cdf differs from pull request most recent head baaf64b. Consider uploading reports for the commit baaf64b to get more accurate results

@@            Coverage Diff             @@
##             main     #155      +/-   ##
==========================================
- Coverage   56.90%   56.85%   -0.06%
==========================================
  Files          86       86
  Lines        7882     7882
==========================================
- Hits         4485     4481       -4
- Misses       3004     3006       +2
- Partials      393      395       +2

Flag
Coverage Δ

unittests
56.85% <ø> (-0.06%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...slo-controller/nodemetric/nodemetric_controller.go
44.68% <0.00%> (-8.52%)
:arrow_down:

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 889465a...baaf64b. Read the comment docs.

/lgtm
/approve


Not handling unknown message type 0x77

I'm writing a script that finds ballasts on a DALI network using INITIALISE / RANDOMISE / COMPARE / WITHDRAW / TERMINATE etc. as a test case for a library I'm writing.  See https://github.com/sde1000/python-dali - the script in question is "find-ballasts.py".
When the COMPARE command causes several ballasts to reply YES (0xff) simultaneously, daliserver logs the message "Not handling unknown message type 0x77" and then a second later "ERROR Error sending DALI message: Receive timeout".  It returns status code 255 over the network.
If I hack my script to interpret status code 255 as "YES" then the script runs as expected.
I believe message type 0x77 is the hardware indicating "response present but garbled".  It would be useful to handle this and return some appropriate status code over the network.

Back when we reverse engineered the protocol, we were not sure what the 0x77 messages meant exactly. They were not relevant for basic operation anyway, so we did not pursue them further.
It might be a good idea to contact the manufacturer and ask for technical documentation. Tridonic offers some for their RS-232 based DALI interface, but not for the USB version. I don't think they would mind, as they still only offer software for Microsoft Windows.

I believe Tridonic are resellers of these devices; the original manufacturer appears to be Lunatone.
Contacting them can't hurt, but I strongly suspect I will be ignored.  I haven't found a single manufacturer of DALI kit so far that gives out anything beyond a bare minimum of technical information on their products.

Further experimenting with the Tridonic software shows that the 0x77 messages are indeed "Framing error in response", which is to be expected when you issue a yes/no query to more than one device at once.  These could be treated as a simple "YES" response, or the network protocol could be extended to be able to express "Framing error".

Thanks for investigating. Did you find out anything else about the message content?
Or alternatively, can you send me sample packets?
I will add a "Invalid packet received" message type to the protocol, then.

I've figured out some more information about message type 0x77.  It means something along the lines of "bus status".  It can be received either unsolicited (packet type 0x11) or as a response to a transmission (packet type 0x12).
Here are some examples:
    166 1494.055413    host        1.10.2<PHONE_NUMBER>00ff9000 Query	FF90	Bcast	QUERY STATUS
    168 1494.073283    1.10.1      host        12730000ff90033c04 ... transmission report
    170 1494.083287    1.10.1      host<PHONE_NUMBER>03003e04   Error	 		framing error (received)

     19 34.395455      1.10.1      host<PHONE_NUMBER>05ffff00 "switch to DSI mode"
     21 34.887455      1.10.1      host<PHONE_NUMBER>02ffff00 "bus short" (I turned the bus power off)

      9 5.045834       1.10.1      host<PHONE_NUMBER>06006100 "switch to DALI mode"
     11 5.137824       1.10.1      host<PHONE_NUMBER>0404b100 "bus voltage OK"
     13 127.210069     1.10.1      host<PHONE_NUMBER>05ffff00 "switch to DSI mode"
     15 127.701023     1.10.1      host<PHONE_NUMBER>02ffff00 "bus short" (I turned the bus power off again)
     17 135.325249     host        1.10.2<PHONE_NUMBER>00ff0000 IAP	FF00	Bcast	OFF
     19 151.927529     host        1.10.2<PHONE_NUMBER>00ff0000 IAP	FF00	Bcast	OFF
     21 158.951875     host        1.10.2<PHONE_NUMBER>00ff2c55 DTR0=0x55 followed by SET SYSTEM FAILURE LEVEL (DTR0) (twice)
                                              about 12s later I turned the bus power back on
     23 181.991277     1.10.1      host<PHONE_NUMBER>06006100 "switch to DALI mode"
     25 182.018264     1.10.1      host        12730000ff0000e456 ... transmission report for first OFF command
     27 182.028265     1.10.1      host<PHONE_NUMBER>00008a56 ... no response to first OFF command
     29 182.045273     1.10.1      host        12730000ff00009257 ... transmission report for second OFF command
     31 182.055257     1.10.1      host<PHONE_NUMBER>00008a57 ... no response to second OFF command
     33 182.071276     1.10.1      host        12730000a355009258 ... transmission report for DTR0=0x55
     35 182.097240     1.10.1      host        12730000ff2c009158 ... transmission report for first SET SYSTEM FAILURE LEVEL
     37 182.124240     1.10.1      host        12730000ff2c009158 ... transmission report for second SET SYSTEM FAILURE LEVEL
     39 182.127232     1.10.1      host<PHONE_NUMBER>04003858 "bus voltage OK" - note it still has the 0x58 serial number

So, if received byte 1 contains 0x77 then received byte 5 contains a status code:

0x02: bus short / no bus power
0x03: received framing error (eg. overlapping transmissions by devices)
0x04: bus voltage OK
0x05: "switch to DSI mode"
0x06: "switch to DALI mode"

Note that in the second half of this example, I issued the two "OFF" commands and the "SET SYSTEM FAILURE LEVEL" command while the bus power was off.  The DALI USB device queued them, and returned the transmission reports (byte 1==0x73) all in one go after bus power was restored, followed by the "bus voltage OK" message with the serial number of the SET SYSTEM FAILURE LEVEL command.


composite unique constraint error while updating the changeset

Precheck

Do not use the issues tracker for help or support requests (try Elixir Forum, Stack Overflow, IRC or mailing lists, etc).
For proposing a new feature, please start a discussion on elixir-ecto.
For bugs, do a quick search and make sure the bug has not yet been reported.
Finally, be nice and have fun!

Environment

Elixir version (elixir -v):
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.):
Ecto version (mix deps):
Database adapter and version (mix deps):
Operating system:

Current behavior
I have a schema two_fa_details where answer and question_id are the fields and both are unique together.. Now when I am trying to insert data into it first it gets inserted but updating it next time isn't working.. It says constraint error.
I have a function set_two_factor_details written for updating table.. The function works fine for inserting the data very firsat time..but when iam updating it...its not working..i have a PUT API for this function. this is my migration file for schema two_fa_details
def change do
    create table(:two_fa_details) do
      add :answer, :string
      add :userprofile_id, references(:user_profile, on_delete: :nothing)
      add :question_id, references(:questions, on_delete: :nothing)

      timestamps()
    end

    create index(:two_fa_details, [:userprofile_id])
    create index(:two_fa_details, [:question_id])

    create unique_index(:two_fa_details, [:userprofile_id, :question_id], name: :user_twofa_detail)
  end

here is a snippet of code
def set_twofactor_details(client_id, twofa_records) do
    user = Repo.get_by(UserProfile, client_id: client_id)
    twofa_records = Enum.map(twofa_records, &get_twofa_record_map/1)

    Enum.map(twofa_records, fn twofa_record ->
      Ecto.build_assoc(user, :two_fa_details)
      |> TwoFaDetails.changeset(twofa_record)
    end)
    |> Enum.zip(0..Enum.count(twofa_records))
    |> Enum.reduce(Ecto.Multi.new(), fn {record, id}, acc ->
      Ecto.Multi.insert_or_update(acc, String.to_atom("twfa_record_#{id}"), record)
    end)|>IO.inspect()
    |> Ecto.Multi.update(
      :update_user,
      Ecto.Changeset.change(user, two_factor_authentication: true, force_reset_twofa: false)
    )
    |> Repo.transaction()|>IO.inspect()
    |> case do
      {:ok, _} ->
        {:ok, :updated}

      {:error, _, changeset, _} ->
        error_string = get_first_changeset_error(changeset)
        Logger.error("Error while updating TWOFA: #{error_string}")
        {:error, 41001, error_string}
    end
  end

Expected behavior
the output should be basically updating the table and returning two fa details updated message. but in the logs its showing constraint error.please help me with this..Iam new to elixir.
result should be
{:ok, updated}
Getting the below error
 #Ecto.Changeset<
   action: :insert,
   changes: %{answer: "a", question_id: 1, userprofile_id: 1},
   errors: [
     unique_user_twofa_record: {"has already been taken",
      [constraint: :unique, constraint_name: "user_twofa_detail"]}
   ],
   data: #Accreditor.TwoFaDetailsApi.TwoFaDetails<>,
   valid?: false
 >, %{}}
[error] Error while updating TWOFA: `unique_user_twofa_record` has already been taken

@josevalim

Hi @radhikapatil25. We reserve the issues tracker for issues only. For questions, we recommend using stack overflow and/or the elixir forum.


Update README.md

Purpose

azure-application-insights no longer valid slug. replaced with current slug value.
Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[X] Other... Please describe: metadata

@paolosalvatori could you please review and merge this PR. It updates a recently deprecated taxonomy value.

Done, thanks @mtrilbybassett


Add filter to publications

Filter by project category at the very least, or potentially venue.

Some considerations:
• I'd need to hide the years when filtered
• Metadata to filter on: awards, projects, HCI, SE, ComputingEd, conference, journal, chapter
• Multiple filters would be key
• Highlight what matches


Media URLs are incorrect in CSS

Describe the bug
I am serving my app under a path such as https://my-site.com/somePath. When running a build I set the variable PUBLIC_URL=somePath, and I have also set <base url="%PUBLIC_URL%" /> in index.html . This correctly updates the output files with the correct paths, except for URLs in the CSS, for example:
https://my-site.com/somePath/static/css/somePath/static/media/some-font.woff2

As you can see the CSS URL is incorrect and is not referencing the media directory relative to the CSS file.
Did you try recovering your dependencies?

Yes, and likely not applicable.
Which terms did you search for in User Guide?

Webpack, media, static, build
Environment

Environment Info:
current version of create-react-app: 5.0.1
running from REDACTED/.npm/_npx/c67e74de0542c87c/node_modules/create-react-app
System:
OS: macOS 13.5.1
CPU: (10) arm64 Apple M1 Max
Binaries:
Node: 18.12.0 - ~/.nvm/versions/node/v18.12.0/bin/node
Yarn: 1.22.19 - /opt/homebrew/bin/yarn
npm: 8.19.2 - ~/.nvm/versions/node/v18.12.0/bin/npm
Browsers:
Chrome: 117.0.5938.149
Edge: Not Found
Safari: 16.6
npmPackages:
react: ^17.0.2 => 17.0.2
react-dom: ^17.0.2 => 17.0.2
react-scripts: 5.0.1 => 5.0.1
npmGlobalPackages:
create-react-app: Not Found
Steps to reproduce

(Write your steps here:)

Expected behavior
Media file URLs in CSS should be relative to the CSS file and not incorrect as shown above.
(Write what you thought would happen.)
Actual behavior
Media paths are incorrect.
(Write what happened. Please add screenshots!)
Reproducible demo

(Paste the link to an example project and exact instructions to reproduce the issue.)

@boonware  can i work on this issue

@Chetax Apologies for (very!) late response. I am not the right person to ask, probably need to ask a maintainer of this project.


Can't retrieve tile properties

The following should return the properties for a tile:
propertiesHashMap = _map.propertiesForGID( layer.tileGIDAt( point ) );
It seems that the poperties for a tileset's GIDs do not match up with the map's GID's.  The tile's GID's are numbered 1 through tileset size, so if there are 16 tiles, then it has GIDs 1 through 16.  The map's GID's are generated in some other manner. Example is 637534208.
So when I get the tile by GID at a point on the layer it returns 637534208.  When I use this GID to get the tile properties for that tile I get an error or null.

I am noticing the same issue.
When using CCTMXLayer.tileGIDAt(CGPoint pos) no matter what index I pass into the tiles.get(int index) method I get back a tile GID of 1677216.
When I inspect the contents of the 'tiles' object I see that it's populated with my 2500 tiles and their appropriate GID. In my case I have two tiles on my map, tile GID 1 and tile GID 2. Why then does tiles.get(int index) always return back 1677216 as my GID?


Replace Vector syntax {} to Any[] in README

From julia v0.5, Vector syntax {} is unusable.
Same meaning is Any[] which is usable after v0.5.

Thank you!


[automate-3030] Enhance edits pending bar to show failure

:nut_and_bolt: Description: What code changed, and why?
When a project update fails, we want to make that much more apparent to the user.
This replaces the standard "edits pending" message with an error message (and coloring) that makes it evident to the user.
The illustration shows the two possible states of the pending edits bar, showing both wording and color change, along with the subtle color change of the button itself when hovering or focusing it.

:chains: Related Resources
NA
:+1: Definition of Done
Here is the contract as defined by unit tests. Most of these were already in place. The new tests are after the ----- divider.
 update-start-confirmation modal...
   when it emits a cancellation, hides the modal
   when it emits a confirmation, hides the modal
   when it emits a confirmation, the projectService starts the rule updates
 pending edits bar...
   is hidden if no project has changes
   is visible if some project has changes
   is visible if some project has changes... UNLESS progress bar is active
   is visible if rules are not being applied
   is visible if update fails
   is visible if update is cancelled
   is visible if update is cancelled but update is still running
----
   displays edits pending message before update is applied
   displays edits pending message if an update is cancelled
   displays failed message if an update fails
   displays failed message if both an update fails and was cancelled

:athletic_shoe: How to Build and Test the Change
In hab studio: rebuild components/authz-service
Remove the last commit to expose a testing harness:
git revert -n 3df50d5

Then rebuild automate-ui per your preferred method.
Create a project and a rule in the browser.
Navigate to the projects page to expose the edits pending bar in its nominal condition (due to another issue, the edits pending bar will not start appearing until you visit that particular page).
Now just watch: every 5 seconds a random failure--or not--will occur. You should see the blue (normal) or pink (failure) bars both appear over time.
That shows the "normal" edits pending state--before an update is applied,
and the result of a failed update. To see the result of a cancelled update, first you have to reset the code--pull down a fresh copy of the branch so it no longer has that revert mentioned above. Then just proceed normally: start an update and cancel it. You will see the normal edits pending bar.
:white_check_mark: Checklist

[x] I have read the CONTRIBUTING document.
[x] Tests added/updated?
[ ] Docs added/updated?
[x] All commits have been signed-off for the Developer Certification of Origin.

@susanev Displaying the failure banner on cancel-project-update was not a bug in this PR; it just exposed that bug that we had not known about in the service. I adjusted the authz-service to correct that.


deps/yaracpp: update reference to yaracpp version that uses xcrun.

Fix #439.

@s3rvac will you please merge this if CIs pass?

Sure.


Create user-facing documentation for DAG Factory

At the moment, all the DAG Factory documentation is a README.
We should collaborate with Astronomer's doc team and @cmarteepants to decide what content to cover and how to represent it.
Once this is done, we should write the documents using Markdown or Rest, based on what is agreed upon.

Initiated an internal discussion: https://astronomer.slack.com/archives/C015V2JFKT5/p1733904505509939


Godot 4.1 uses higher CPU, and has lower performance in-editor.

Godot version
v4.1-stable
System information
Linux Mint 21.1 Cinnamon (Cinnamon Version 5.6.8, Kernel 5.15.0-76-generic) - Godot v4.1.stable - Vulcan (Forward+) - NVIDIA GeForce GTX 1050   &   AMD FX-6300 Six-Core Processor x 3
Issue description
Godot v4.1 seems to run more sluggishly on this computer, which I admit isn't exactly high end, but v4.0.3 ran significantly faster.
Here is my CPU performance in both :

"Active Use" here is me creating a control in the scene, and then just doing random stuff in the UI, such as hiding and unhiding catagories in the inspector or selecting and unselecting the node.
This is my computer with nothing but a single Nemo window and System monitor - same as what i had open for Godot.

Steps to reproduce
The project I used for testing was a completely empty project, with an unsaved scene and a single control, where i just messed around with hiding and showing and unselecting. Essentially just did menial tasks that did nothing to test it.
However, I notice this no matter what project it is and what it contains.
Note : This may be somewhat specific to my hardware - or at least hardware with similar performance. I would try it on my PC with higher spec, but I currently cannot.
Minimal reproduction project
N/A

Can you try the same steps on 4.1 dev/beta/RCs? You can download them here.

I think squareverylong mentioned on Discord that the issue is reproducible with beta1

yes, i tried beta-1 with the same result.

I don't remember, did you try the dev builds too?

My project is running at 86 fps in 4.0.3 and at 58 fps 4.1. The performance loss is very noticeable. I'll might make a separate issue with a demo project for anyone to test

I don't remember, did you try the earlier dev builds too?
I have not, no. I will do that soon and get back to you.

My project is running at 86 fps in 4.0.3 and at 58 fps in 4.1. The performance loss is very noticeable. I'll might make a separate issue with a demo project for anyone to test

For reference, this turned out to be an unrelated issue: https://github.com/godotengine/godot/issues/79222

I had a similar problem when upgrading my project from 4.0.3 to 4.1.1. I have a large number of labels that are constantly being displayed and hidden every second. My cpu is not good is i5-7600, the occupancy rate of about 15% went up to 26%.

I'm no longer sure if my initial testing was accurate at all, so I will try again next time i am free ( and able ) to do so.


[Bug] Requesting changes to 'Check connection for source connector.' (test-source-connection)

Describe the bug
No schema is presented for a 200:

An example is provided based on a schema but for a success.
What is expected for an error?

Closing as error types are now also available in the spec


Add toggles for critical insights and unread insights in the issues tab

Figma: https://www.figma.com/file/MxA95ZHK5NqGX4ptFQljRJ/⭐-Design-System-3.0?node-id=6055%3A95093&mode=dev


Content of merged cells

I am merging cells with the same content where I want the following table

to merge to

Based on this (from the documentation):

When two or more cells are merged, any existing content is concatenated and placed in the resulting merged cell. Content from each original cell is separated from that in the prior original cell by a paragraph mark. An original cell having no content is skipped in the contatenation process.
Merging four cells with content 'a', 'b', '', and 'd' respectively results in a merged cell having text 'a\nb\nd'.

I thought the following code should produce a merged cell with only one paragraph, however it looks like there are two paragraphs in the merged cell.
from docx import Document
doc = Document('fuits.docx')
tab = doc.tables[0]
c1 = tab.cell(1,0)
c2 = tab.cell(2,0)

c2.text = ''
cm = c1.merge(c2)
cm.paragraphs

Result:
[<docx.text.paragraph.Paragraph at 0x112cb9330>,
 <docx.text.paragraph.Paragraph at 0x112cba6b0>]

Is this expected?

Sounds plausible. What is the specific before and after text of the cells? The "\n" in the docs would be in cell.text. If you're looking in cell.paragraphs each "\n" would start a new paragraph.

If you merged the top version you would receive the bottom version but with "Apple\nApple". The merging algorithm doesn't have anything to do with deduplicating text. You'll have to take care of that yourself.

Thanks for taking a look at this @scanny. From the documentation it said "An original cell having no content is skipped in the contatenation process" so I thought by doing c2.text = '' the whole paragraph will be skipped.

This is the code that controls that behavior:
https://github.com/python-openxml/python-docx/blob/master/src/docx/oxml/table.py#L616-L630
I expect what's happening is that a paragraph with a run that contains the empty string is considered distinct from a paragraph with no runs. The latter being what you get if the cell was empty from the start.
If you wanted to hack something in then this might produce the result you're looking for:
# -- instead of `cell.text = ""`... --
tc = cell._tc
tc.clear_content()
p = tc.add_p()

This is what the Cell.text setter is doing but it goes a little further:
https://github.com/python-openxml/python-docx/blob/master/src/docx/table.py#L273-L284


APP-472: C and C++ app descriptions updated.

@kaa-leeroy test this please

@kaa-leeroy test this please

The PR has been recently rebased onto master to resolve conflicts as #546


In RadioButton the OuterEllipse.Fill shouldn't be bound to the RadioButton's background

in RaioButton.xaml the OuterEllipse.Fill and CheckOuterEllipse.Fill is bound to the RadioButton.Background. This is not 'correct' in the sense that it looks like disabled when the RadioButton.Background is set to Transparent.
For example: V2.0 is on the left, V1.6.5 on the right:

MahApps.Metro version [ v2.0]

Maybe this is a good point to implement the CheckBoxHelper for this Control as well.

Same happens with CheckBox.

This is just a styling issue; differentiate the general background from the radio 'activation area' background with another name.
Now if the CheckBox has the same problem, use the same naming convention for each case.
For example RadioButton.ActivationArea.Background and CheckBox.ActivationArea.Background respectively.


First Lens.. Mouse over TODO's

Description
Added the ability to mouse over and react to TODO's

@sestinj is the auto linter removing the last comma?

@rootedbox Yes most likely Prettier is doing that? But Peter Zaback added "trailingComma": "all" to the .prettierrc in his recent PR, which will help standardize

My thoughts after pulling and playing around with this a bit...I like that it's not a codelens because in a way that makes it less invasive. The one downside is that this feels like a bit of a hidden feature that would be extremely rare to discover. More importantly though, I am not sure how frequently this will be able to accomplish a TODO, which in practice is often a multi-file edit, or something difficult enough to accomplish that somebody decided to postpone the work. That is mostly feedback on the code insertion feature. As for chat, I can see more success.
Also: I'm curious what the limits of this hover view are. If there's any way to display something like a button, an image, or just larger text, this might be an extremely valuable UI tool