PeakGroup name parsing and compound validation

FEATURE DESCRIPTION
Feature Inspiration
PeakGroup names are generally the name of the associated compound, but sometimes are actually the names of multiple compounds.
Feature Description
A method for parsing these "combined names" should be part of the PeakGroup class to keep that logic together and allow reuse. Compounds that associated with the PeakGroup should have names that match those parsed from the "combined name" and be checked by validation.
Both Compound and PeakGroup have a formula attribute, which in principle should match. There should be some validation to ensure that the PeakGroup.formula matches each of the Compound.formula values for associated Compounds.
Design suggestion

A PeakGroup class method that returns a iterable of individual compound names based on the provided string.
Validation that names and formulas between PeakGroup  and associated Compound records match.

Alternatives Considered
none
Comment
Some overlap with #68

ISSUE OWNER SECTION
Assumptions
none
Requirements

User provided data files will be required to follow the format used by this parsing function for PeakGroup names.
Edit the following comment once we've settled on the error behavior for non-mathcing / empty PeakGroup formulas:

https://github.com/Princeton-LSI-ResearchComputing/tracebase/blob/68469624485631fe8624d83a7090c2cccfe7b2c7/DataRepo/utils.py#L485-L488

There should be some validation to ensure that the PeakGroup.formula matches each of the Compound.formula values for associated Compounds.

Limitations
none
Affected/Changed Components

PeakGroup class

DESIGN
GUI Change description
Code Change Description (Pseudocode optional)
Tests

Test that an empty formula in an insert into the PeakGroup table fails
Test that a matching formula succeeds

This is essentially done.


release: 0.4.0

Description
The hey-amplify Discord bot has moved to a dedicated bot package, removing the serverless implementation in favor of a containerized bot client.
closes https://github.com/josefaidt/amplify-discord-bots/issues/42
closes https://github.com/josefaidt/amplify-discord-bots/issues/24
closes https://github.com/josefaidt/amplify-discord-bots/issues/26
To-do

[x] create single cdk app for bot and frontend
[x] create cdk stack for bot
[x] create cdk stack for frontend
[ ] rework deployment process -- github actions?
[ ] update deployment diagram
[ ] update architecture diagram
[x] ensure CONTRIBUTING.md is updated to reflect new deployment and contribution workflow

merging prior to full release in favor of improving libraries in parallel with documentation updates


How to get the button view in the recycleview ?

Hi,

The button is child of recycleview,
Text and id of button are same.
So how could I get the button view with position 1 in recycleview ?

Thanks.
brs.

If the button is Holder item of your recycler view then you can use this:
onView(withId(R.id.recycler_view_id)).perform(RecyclerViewActions.actionOnItemAtPosition(1, click()))


[Question]: how can I avoid my nice markdown tables in a README.rst file to trigger (ERROR/3) (bash) syntax error near unexpected token `|'

To Dos

[X] I already searched with my favorite search engine "How to do X with rstcheck".

[X] I already read and followed the README and didn't find an answer.

[X] I have checked the issues and think that this is not a duplicate.

[X] I added a very descriptive title to this issue.

Description
I have tried adding very nice looking markdown tables that use pipe symbols and other un-expected character sequences and it of course triggers rstcheck errors (ERROR/3) (bash) syntax error near unexpected token `|'
What is the best way to exclude such a markdown table stanza from being flagged in the pre commit hook?
I have no control over the git repo I am submitting my "pretty" PRs to. But the poor maintainers  trying to merge my PRs do not like my PRs.
What can I do?
Sample lines submitted to a README.rst that triggers this is:
Setup venv

linux & macos
windows *

python3 -m venv .venv --prompt idemenv
python3 -m venv .venv --prompt idemenv

source .venv/bin/activate
. .venv\Scripts\activate

pip install -e .
pip install -e .

Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools":
https://visualstudio.microsoft.com/visual-cpp-build-tools/

Operating System
Linux
Operating System Details
No response
Python Version
3.8.12
rstcheck Version
v5.0.0

One such PR can be found at https://github.com/eitrtechnologies/idemenv/pull/6

You can't use Markdown in reStructuredText. See here how tables work in rST:
https://docutils.sourceforge.io/docs/ref/rst/restructuredtext.html#tables

With @polyzen 's comment I guess this issue is resolved. If you have further questions or problems feel free to reopen the issue.


department-level enable/disable does not work

UPDATE: On further investigation, department-level enable/disable just doesn't work.
The docs say this should work works:
All cops are then disabled by default. Only cops appearing in user configuration files are enabled. Enabled: true does not have to be set for cops in user configuration. They will be enabled anyway. It is also possible to enable entire departments by adding for example

Style:
  Enabled: true

But it turns out this doesn't work at all. This is because Style is seen as an "invalid" config in Config::Validator.validate() and thus ignored. :/

Original ticket (for context)
This is probably not a rubocop bug, but I dunno where else to ask.
In the cookstyle project, we just sit ontop of the awesomeness that is Rubocop, and define a bunch of our own Cops and departments.
However, departments defined by our own cops don't seem to be enable-able the way, say Style is. So I can do this:
AllCops:
  DisabledByDefault: true

Style:
  Enabled: true

But what I can't do is (Chef/Correctness here is a department, you can see code for cops in that department here).
AllCops:
  DisabledByDefault: true

Chef/Correctness:
  Enabled: true

or even:
AllCops:
  DisabledByDefault: true

Chef:
  Enabled: true

What's really strange is that in robocop I don't see any special setup for the Style or Layout etc modules, so I would think this would "just work"... but it doesn't seem to. When I we try we get:
Warning: Chef/Correctness does not support Enabled parameter.

Supported parameters are:

  - StyleGuideBaseURL

If I do just Chef, then I get:
Error: unrecognized cop Chef found in .cookstyle.yml

I'm sure is some minor dumb thing we're doing or not doing that's preventing this from working like the rubocop-built-in departments.
Any help would be greatly appreciated.

OK I dug through the code, and realized that the code does not work that way:
Style:
  Enabled: true

is actually just entirely ignored. But the docs say it works:
All cops are then disabled by default. Only cops appearing in user configuration files are enabled. Enabled: true does not have to be set for cops in user configuration. They will be enabled anyway. It is also possible to enable entire departments by adding for example

Style:
  Enabled: true

This is because Style is seen as an "invalid" config in Config::Validator.validate() and thus ignored. :/

I don't understand. Invalid configuration should make RuboCop stop and report the error, not be ignored. Do you get any error or warning message?

I wrote the above because you said that department level enable/disable doesn't work at all. And that sounds strange to me, because I know that it basically works. :slightly_smiling_face:
About the original ticket, I can see that Chef/Correctness does not support Enabled parameter is reported because there needs to be an Enabled: true in the default configuration of the department. Otherwise it can't be overridden.
Also, there's no concept of department hierarchy implemented in RuboCop, so Chef doesn't exist as a department and it cannot be disabled.
I hope that helps.

OK, so department is always everything up until the last slash, as opposed to everything up to the first slash.
So if you have a rule A/B/C/D you can disable A/B/C (if it has a Enabled default) or A/B/C/D, but never A/B or A, is that correct?
I can throw a PR at the chefstyle folks to add a default to Chef/*... how hard would it be to extend rubocop to understand department hierarchies?
Thanks!

OK, so department is always everything up until the last slash, as opposed to everything up to the first slash.

Yes.

So if you have a rule A/B/C/D you can disable A/B/C (if it has a Enabled default) or A/B/C/D, but never A/B or A, is that correct?

Yes.

I can throw a PR at the chefstyle folks to add a default to Chef/*...

Great!

how hard would it be to extend rubocop to understand department hierarchies?

It's one of those things that feel like it should be fairly easy, but you can't say for sure until you've actually done it. I can try to implement it. Let's keep this issue open for a while.

Hey @jonas054 since you closed this, should I open up a separate enhancement for hierarchical departments?

Yes, feel free to do so. I'll take a look at the problem. Please tag me in the new issue and reference this issue in it.


CLI for adding changesets at release time, based on merges?

Affected Packages
the CLI
Problem
As a maintainer, working with other maintainers, not all PRs with meaningful changes have changesets added.
Trying to figure out what changed for a release, is really hard with a manual change-tracking process.
Proposed solution
it would be a life saver (saving tooons of time), to have a tool walk me through the the merged PRs and prompt about creating a changeset, using the commits (or PR description with the GitHub integration), as the default changeset entry.
Related
These would also be useful as well:

https://github.com/changesets/changesets/issues/1059

I made it here: https://github.com/NullVoxPopuli/changeset-recover


Enhancement: Handle https listener creation in Network construct

Codecov Report

Merging #54 (19e0c0f) into release (80d2621) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             release      #54      +/-   ##
=============================================
+ Coverage      99.17%   99.26%   +0.08%
- Complexity       161      162       +1
=============================================
  Files              9        9
  Lines            968      949      -19
  Branches          26       25       -1
=============================================
- Hits             960      942      -18
+ Partials           8        7       -1

Flag
Coverage Δ

unittests
99.26% <100.00%> (+0.08%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/main/java/org/wcdevs/blog/cdk/DomainStack.java
100.00% <100.00%> (ø)

src/main/java/org/wcdevs/blog/cdk/Network.java
99.53% <100.00%> (+0.53%)
:arrow_up:

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 adb27b2...19e0c0f. Read the comment docs.


Add "-Snapshot" or a custom parameter to instruments command

I would like to pass some custom parameters to the application so it knows its running for snapshot. I've seen a few questions before about it and if you prepend the arguments to the end of the instruments command then it should work. Right now, i just check [[NSProcessInfo processInfo] arguments] for "-AppleLanguages" but I would prefer to check for a custom argument or just "-Snapshot".
Thanks

You mean something like this?
https://github.com/KrauseFx/snapshot#custom-args-for-the-build-command
In your code, you can use an #ifdef to check if it's snapshot.

Ah, perfect. I was thinking arguments for the instruments command but build/preprocessor is just fine.
Thanks

Is there any way I could improve the project documentation to make this clearer?

I don't think so, its very clear but I just had stuck in my head that I wanted it to pass it as a argument to the instruments cmd as this would be the same as arguments passed from a scheme.


User Story: Unpin opinion

As a verified user, I can unpin an opinion I previously created, so that it no longer appears at the top of my list of opinions.
Acceptance Criteria:

There is an option to unpin an opinion
Once unpinned, an opinion no longer appears at the top of a users' list of opinions

Tasks:

[ ] Add a unpin option to the opinion view
[ ] When a users' list of opinions is viewed, unpinned opinions do not appear at the top

To be actioned in conjunction with #13


chore: fix auth flaky test

Lancé une 15aine de fois -> Toujours l'erreur :
does reset password:
    TypeError: Cannot read properties of undefined (reading 'KeyboardEvent')
     at Keyboard.fireSimulatedEvent (http://localhost:3000/__cypress/runner/cypress_runner.js:116728:28)
     at Keyboard.simulatedKeydown (http://localhost:3000/__cypress/runner/cypress_runner.js:116851:58)
     at Keyboard.typeSimulatedKey (http://localhost:3000/__cypress/runner/cypress_runner.js:116889:10)
     at <unknown> (http://localhost:3000/__cypress/runner/cypress_runner.js:116585:14)
     at tryCatcher (http://localhost:3000/__cypress/runner/cypress_runner.js:1807:23)
     at Promise.attempt.Promise.try (http://localhost:3000/__cypress/runner/cypress_runner.js:4315:29)
     at <unknown> (http://localhost:3000/__cypress/runner/cypress_runner.js:116594:70)
     at tryCatcher (http://localhost:3000/__cypress/runner/cypress_runner.js:1807:23)
     at Object.gotValue (http://localhost:3000/__cypress/runner/cypress_runner.js:6476:18)
     at Object.gotAccum (http://localhost:3000/__cypress/runner/cypress_runner.js:6465:25)
     at Object.tryCatcher (http://localhost:3000/__cypress/runner/cypress_runner.js:1807:23)
     at Promise._settlePromiseFromHandler (http://localhost:3000/__cypress/runner/cypress_runner.js:1519:31)
     at Promise._settlePromise (http://localhost:3000/__cypress/runner/cypress_runner.js:1576:18)
     at Promise._settlePromiseCtx (http://localhost:3000/__cypress/runner/cypress_runner.js:1613:10)
     at _drainQueueStep (http://localhost:3000/__cypress/runner/cypress_runner.js:2411:12)
     at _drainQueue (http://localhost:3000/__cypress/runner/cypress_runner.js:2400:9)
     at Async._drainQueues (http://localhost:3000/__cypress/runner/cypress_runner.js:2416:5)
     at Async.drainQueues (http://localhost:3000/__cypress/runner/cypress_runner.js:2286:14)


why sqlruuner.selectOne method return map key need upperCase

when i use sqlruuner class to run my custom sql, but the method selectOne return hashmap, and the map key is convert to uppercase, then the key can't not match  column name in database?

To fix some migratins issue, it seems.
As stated in the issue template, we use GitHub Issues only to track bugs and feature requests.
Please post your question to the mailing list.
Thank you!


Constructor is included in the list of references for a contract

Given this code:
contract Foo {
  constructor () {}
}

contract Bar {
  function f() public {
    Foo foo = new Foo();
  }
}

If you find all the references to Foo, the constructor of the contract will be listed as one.
In a somewhat equivalent typescript example:
class Foo {
  constructor() {}
}

const foo: Foo = new Foo();

The constructor is not included.

I suspect this is a side-effect of our use-def overlays on the ast. We don't differentiate between the type of relationship when looking up references, we show all of them.
I suspect this one could be fixed through a targeted patch, avoiding the complete restructure of our ast use-def chains.


Provide some concrete examples of the usage and results of different %libyt [Command] for [Interactive Python Prompt].(https://github.com/calab-ntu/libyt/blob/main/doc/InteractivePythonPrompt.md#interactive-python-prompt)

Ah, I accidentally clicked it.... :sweat:


Consider shorthanding appservice.botIntent.underlyingClient to appservice.botClient

The client holds a vast number of useful things that we are using in the bridge, but it's a little ugly to have to access the client indirectly. I realise it might slightly uglyify the shape of your API, but would be useful nontheless.

Concern is more that people would be bypassing ensureRegistered and less API shape (I don't mind throwing around MatrixClient objects). Nothing documentation can't solve though.


[files] do something about the "different" art files

There's some art files that we have separated per level such as ecovalve-ag, but when I checked what the differences were it was just some gibberish data (always less than 16 bytes) at the end that I am pretty sure is unused. It would be nice to verify this so that we can merge them all together in the file lists to simplify them a bit, the loss of the unusable garbage data isn't really a big deal.

Just posting here to remind myself.
FILES:

orb-cache-top-ag - this is just the "top" of the orb cache, the interactive portion of it. The rest is baked into the background as TIE usually. The difference between some files is in the padding section (15 bytes) of the prejoint string, which is the last thing in the object data. My theory is its garbage data from NDI's tools. This data does not appear in decompiled output. I wonder if we can find this unused data in other random other object files where it is used? It might also be a slight clue as for the DGO build order. Anyway, all of these files should yield the exact same output in the game so it's fine to use any of them. Found in: BEA, CIT, JUN, MIS, SNO, SUN, VI1, VI2
ecovalve-ag - I'm pretty sure this is the valve that blocks eco vents. Also, it appears in too many DGOs. I think it might have been automatically included with the levels that use eco vents. Which might have been an accident considering GAME.CGO already has it as part of the art package. Same situation as orb-cache-top-ag.

Ok I'm not gonna list the rest of them, it turns out ALL of our "duplicate" art files have this thing, so we're free to remove all duplicates.


Adds pagination to Blog List page

Set it so each page shows 8 blogs. I first made a numbered pagination, (1,2,3...) but then after looking around at how other folks handle it I felt that select is easier to use so I switched to that. Could customize the select options more.
Fixes #140

Thanks for taking care of a11ty 🙏
Small rendering glitch on Safari:

Good catch, should be fixed now!

Nice 😄
I just noticed the render of the focus state looks a bit shrinky 😊

compared to


Fix invalid SafeAreaProvider import

<If necessary, assign reviewers that know the area or changes well. Feel free to tag any additional reviewers you see fit.>
Details

Fixed Issues

Fixes GH_LINK
Tests

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

@TomatoToaster Sorry, auto-import feature of my IDE let me down (which is its fault), and I haven't properly tested for regressions on web and desktop (which is my fault). So it broke your web/desktop builds.
Let me double check everything again

Closing due to reverting of the original PR

Hey no worries, I should have requested that change, I forgot I fixed that locally! I put up a fix for that PR and the following comments.


no_superfluous_phpdoc_tags needs an option to allow params that aren't on the signature

In Symfony, it's quite typical to find signatures like:
public function foo($a, $b/*, $c = null*)
we do this when we want to add an argument in a BC-compatible way.
when we do this, we also add an @param ... $c in the docblock to be explicit.
no_superfluous_phpdoc_tags currently removes this @param, but we don't want it to do that.

Duplicate of #3933.


Add desktop entry and svg logo

🎉 New feature
Part of https://github.com/ignition-tooling/release-tools/issues/598
Summary
Include a desktop entry for Ignition Gazebo in upstream sources: the PR is composed by two separate files, a desktop file conformant with version 1.0 of the standard. The other file is the Ignition logo in svg that illustrate the desktop entry.
A PR in the package metadata install the files in the corresponding directories.
Test it
New packages 3.13.0~pre1 are in the prerelease repository. They can be installed and open your desktop menu, search for Ignition and, depending on the desktop manager, will appear something like:

Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Oh I should have mentioned that I tried the pre-release and it worked for me 👍🏽 👍🏽

Tested new changes in:
3.13.0~pre3-1~focal is the prerelease repository for testing.

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-releases-2022-04-27-fortress-citadel/1389/1


Input port not connected in testharness

In PR #265 the default for enable_debug is set for false, but this is only for the testharness. The cluster wrapper template still has the default set to true, which means there still is a debug_req_i port.

Ahhh ayayay yes you're right! I made that wrong. Wait hehe.

Close as finished


update ddnet to v10.1.1

Thank you for the contribution. It needed some fixes, so they were made in 630ef3da9ba0b6ddc2ec270fd25a3359a8577ea8. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


File does not exist error with mounts

While trying to make use of the new mount features, I can consistently reproduce the "file does not exist" errors.
Steps to reproduce:

git clone https://github.com/twbs/bootstrap.git -b master-xmr-hugo-2
npm i
npm run docs-serve
Modify any file from the dist folder like dist/css/bootstrap.css

Result:
Change of Static files detected, rebuilding site.
2019-08-05 10:38:34.775 +0300
Syncing \css\bootstrap.css to C:\Users\xmr\Desktop\bootstrap\
ERROR 2019/08/05 10:38:34 file does not exist
ERROR 2019/08/05 10:38:34 file does not exist

This is with Hugo 0.56.3 on Windows 10 64-bit, but it seems to happen on macOS too.
If I modify any non-static file and Hugo rebuilds, then the error goes away.

Just reproduced on Linux Mint 19.2 so it must not be related to the operating system.
xmr@xmr-VirtualBox:~/Desktop/bootstrap$ npm run docs-serve

><EMAIL_ADDRESS>docs-serve /home/xmr/Desktop/bootstrap
> hugo server --port 9001 --disableFastRender

                   | EN
+------------------+-----+
  Pages            | 105
  Paginator pages  |   0
  Non-page files   |  20
  Static files     | 125
  Processed images |   0
  Aliases          |  16
  Sitemaps         |   1
  Cleaned          |   0

Total in 395 ms
Watching for changes in /home/xmr/Desktop/bootstrap/{dist,site}
Watching for config changes in /home/xmr/Desktop/bootstrap/config.yml
Environment: "development"
Serving pages from memory
Web Server is available at http://localhost:9001/ (bind address <IP_ADDRESS>)
Press Ctrl+C to stop

Change of Static files detected, rebuilding site.
2019-08-05 14:52:26.810 +0300
Syncing /css/bootstrap.css to /
ERROR 2019/08/05 14:52:26 file does not exist

Just reproduced on Linux Mint 19.2 so it must not be related to the operating system.

I was just about to say that it's probably a "Windows slash issue" ... I will have a look at it.

OK, I see what's happening. I will ponder a little on what the simplest fix is.

Thanks! It's the only thing blocking us to use this feature on Bootstrap :)
And then I'll try to get rid of the version number from our source folders.
On Thu, Aug 8, 2019, 22:27 Bjørn Erik Pedersen<EMAIL_ADDRESS>wrote:

OK, I see what's happening. I will ponder a little on what the simplest
fix is.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gohugoio/hugo/issues/6165?email_source=notifications&email_token=AACVLNKNO3AGCAHCG6TPJOTQDRXQXA5CNFSM4IJILBTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD34U5TI#issuecomment-519655117,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACVLNN2JTWKD7H23BYUM7LQDRXQXANCNFSM4IJILBTA
.

@bep: I just tried v0.57.0 and I still get the same error on Windows.
C:\Users\xmr\Desktop\bootstrap>npm run docs-serve

><EMAIL_ADDRESS>docs-serve C:\Users\xmr\Desktop\bootstrap
> hugo server --port 9001 --disableFastRender

Building sites …
                   | EN
+------------------+-----+
  Pages            | 105
  Paginator pages  |   0
  Non-page files   |  20
  Static files     | 125
  Processed images |   0
  Aliases          |  16
  Sitemaps         |   1
  Cleaned          |   0

Total in 1246 ms
Watching for changes in C:\Users\xmr\Desktop\bootstrap\{dist,site}
Watching for config changes in C:\Users\xmr\Desktop\bootstrap\config.yml
Environment: "development"
Serving pages from memory
Web Server is available at http://localhost:9001/ (bind address <IP_ADDRESS>)
Press Ctrl+C to stop

Change of Static files detected, rebuilding site.
2019-08-14 11:40:33.799 +0300
Syncing \css\bootstrap.css to C:\Users\xmr\Desktop\bootstrap\
ERROR 2019/08/14 11:40:33 file does not exist
ERROR 2019/08/14 11:40:33 file does not exist

I will try on Linux later.

@bep: confirmed, still happens on Linux Mint too:
xmr@xmr-VirtualBox:~/Desktop/bootstrap$ npm run docs-serve

><EMAIL_ADDRESS>docs-serve /home/xmr/Desktop/bootstrap
> hugo server --port 9001 --disableFastRender

                   | EN
+------------------+-----+
  Pages            | 105
  Paginator pages  |   0
  Non-page files   |  20
  Static files     | 125
  Processed images |   0
  Aliases          |  16
  Sitemaps         |   1
  Cleaned          |   0

Total in 418 ms
Watching for changes in /home/xmr/Desktop/bootstrap/{dist,site}
Watching for config changes in /home/xmr/Desktop/bootstrap/config.yml
Environment: "development"
Serving pages from memory
Web Server is available at http://localhost:9001/ (bind address <IP_ADDRESS>)
Press Ctrl+C to stop

Change of Static files detected, rebuilding site.
2019-08-14 13:41:05.912 +0300
Syncing /css/bootstrap.css to /
ERROR 2019/08/14 13:41:05 file does not exist

@XhmikosR a closed issue is a... closed issue. If you think this still is an issue => open a new issue. But before doing so, please confirm that you are on the correct version. I have confirmed this fixed on MacOS and I would be surprised if Linux should behave differently.

I just didn't want to make a new issue about the same thing, but I'll create a new one now. The issue is definitely there for me.


bump libc 0.2.164 rustix 0.38.41

Waiting on fix in https://github.com/nix-rust/nix/issues/2540 then update this PR with newer nix
+ cross check --locked --lib --bins --target x86_64-sun-solaris
...
   Compiling phf_macros v0.11.2
error[E0425]: cannot find value `O_DIRECT` in crate `libc`
   --> /cargo/registry/src/index.crates.io-6f17d22bba15001f/nix-0.29.0/src/fcntl.rs:113:9
    |
113 |         O_DIRECT;
    |         ^^^^^^^^ not found in `libc`

error[E0425]: cannot find value `SIGINFO` in crate `libc`
    --> /cargo/registry/src/index.crates.io-6f17d22bba15001f/nix-0.29.0/src/sys/signal.rs:119:9
     |
119  |         SIGINFO,
     |         ^^^^^^^
     |
    ::: /cargo/registry/src/index.crates.io-6f17d22bba15001f/libc-0.2.164/src/unix/solarish/mod.rs:1285:1
     |
1285 | pub const SIGINT: ::c_int = 2;
     | ------------------------- similarly named constant `SIGINT` defined here
     |
help: a constant with a similar name exists
     |
119  |         SIGINT,
     |         ~~~~~~
help: consider importing this unit variant
     |
6    + use crate::sys::signal::Signal::SIGINFO;
     |

error[E0531]: cannot find unit struct, unit variant or constant `SIGINFO` in crate `libc`
    --> /cargo/registry/src/index.crates.io-6f17d22bba15001f/nix-0.29.0/src/sys/signal.rs:119:9
     |
119  |         SIGINFO,
     |         ^^^^^^^
     |
    ::: /cargo/registry/src/index.crates.io-6f17d22bba15001f/libc-0.2.164/src/unix/solarish/mod.rs:1285:1
     |
1285 | pub const SIGINT: ::c_int = 2;
     | ------------------------- similarly named constant `SIGINT` defined here
     |
help: a constant with a similar name exists
     |
119  |         SIGINT,
     |         ~~~~~~
help: consider importing this unit variant
     |
6    + use crate::sys::signal::Signal::SIGINFO;
     |

error[E0081]: discriminant value `8` assigned more than once
   --> /cargo/registry/src/index.crates.io-6f17d22bba15001f/nix-0.29.0/src/macros.rs:131:9
    |
131 | /         $v enum $BitFlags {
132 | |             $($entries)*
133 | |         }
    | |_________^
...
255 |                   $entry = libc::$entry,
    |                            ------------
    |                            |
    |                            `8` assigned here
    |                            `8` assigned here
    |
   ::: /cargo/registry/src/index.crates.io-6f17d22bba15001f/nix-0.29.0/src/sys/signal.rs:28:1
    |
28  | / libc_enum! {
29  | |     /// Types of operating system signals
30  | |     // Currently there is only one definition of c_int in libc, as well as only one
31  | |     // type for signal constants.
...   |
121 | |     impl TryFrom<i32>
122 | | }
    | |_- in this macro invocation
    |
    = note: this error originates in the macro `libc_enum` (in Nightly builds, run with -Z macro-backtrace for more info)


Added react-native-track-player

An audio player package dedicated to music/podcast apps.

Nice one :) Thanks @Guichaguri


Make commands be case-insensitive.

!HanGMan and !Hangman will now both work.
(https://user-images.githubusercontent.com/37850395/38173315-9e406ee0-35bc-11e8-8f92-b7837616c4d1.JPG)

Don't forget about the CommandPolicy, which should also have a comparison in it.

I tested commands for case sensitivity and my changes did fix it for me (see linked screenshot in my pull request) - simply by adding .ToUpperInvariant() in the CommandHandler.cs  and without changing the command policy.
I haven't figured out how/where would case sensitivity occur if I left the CommandPolicy unchanged? Just trying to understand how I could test/reproduce a change regarding the CommandPolicy. Again, for me the commands all seem to be case insensitive already.

Yep. Yours gets all uses of it perfectly! The added commands like the !Magnus command can only be removed using that casing doing !RemoveCommand maGNus will fail (I think).

I tested that but it works for me. I can add a command like !testcommand and then remove it saying !RemoveCommand teSTcommand.
If I'm not mistaken the debugger showed that every command will eventually be handled through the CommandReceivedHandler method in CommandHandler.cs  where I made my change? Sorry, I'm really trying to wrap my head around this.


Add signup and login pages

The changes I made are :

Add signup page with 2 pages to authenticate
Add login page with 3 pages to authenticate
Add options to login and signup like google, facebook ,github ,etc.

This solves the issue #1  which was raised by me and was assigned to me under hacktoberfest and Gssoc 2024
Things kept in mind :
-I have reviewed my code before submission.
-I have read and followed the project's Contribution Guidelines.
-I have tested the code thoroughly.
-I have attached related issue numbers, screenshots, or videos (if applicable).
-I  have added comments to my code where necessary for clarity.
-I have checked the existing code base and my changes do not introduce any breaking changes.
-My PR addresses only one issue; I have created separate PRs for other issues.
-I have ensured that the PR does not include unrelated files.
-I am mindful of the project's coding standards and best practices.
I am attaching some screenshots about changes I made :

@manya1632 Please resolve the merge conflicts

Thank You !! Best wishes .


can't install msixbundle. 0x800B010A

but zip was available so i used that. however the certificate issue should be fixed.

Unfortunately, until we were able to have our own certificate, or able to publish to Microsoft Store, this is the way you'll have to deal with.


How to send large size streamed data through the gateway

Description:
Could not find any information of the $subject or the possible timeout error in the docs.
Following is an answer suggested to a customer.
You have an option to send large size streamed data through the gateway.
Please follow the following steps.

Navigate to <APIM_HOME>/repository/conf/axis2/axis2.xml file.
In the Message Builders section uncomment –

<messageBuilder contentType="multipart/form-data"
class="org.wso2.carbon.relay.BinaryRelayBuilder"/>

In the Message Formatter section uncomment –

<messageFormatter contentType="multipart/form-data"
class="org.wso2.carbon.relay.ExpandingMessageFormatter"/>

After doing the above configurations, if you got the timeout error, please do the following configurations to increase the API Manager timeouts.
Increasing global timeout.
Navigate to <APIM_HOME>/repository/conf/synapse.properties file and increase the synapse.global_timeout_interval.
Affected Product Version:
APIM

@dilinisg  Which APIM version does this refer to?

We will not be fixing this as this issue is based on an older version of the product


odd error when attempting to swap through a univ3 pool on a local fork

got the following when trying to swap a uniswap v3 pool
✗ node_modules/.bin/hardhat trace --hash 0xb89c8ee689b95663128571e426b1608ded1263dcc72a18047e3a055f179a08cc --rpc http://localhost:8545
You are forking from block 14361020, which has less than 31 confirmations, and will affect Hardhat Network's performance.
Please use block number 14360994 or wait for the block to get 26 more confirmations.
Switched mainnet fork to block 14361020

node_modules/hardhat/src/internal/core/providers/http.ts:78
    const error = new ProviderError("HttpProviderError", -1);
                  ^
ProviderError: Errors encountered in param 1: Storage slot argument must have a length of 66 ("0x" + 32 bytes), but '0x0' has a length of 3
    at HttpProvider.request (node_modules/hardhat/src/internal/core/providers/http.ts:78:19)
    at JsonRpcClient._send (node_modules/hardhat/src/internal/hardhat-network/jsonrpc/client.ts:346:39)
    at JsonRpcClient._perform (node_modules/hardhat/src/internal/hardhat-network/jsonrpc/client.ts:276:34)
    at JsonRpcClient.getStorageAt (node_modules/hardhat/src/internal/hardhat-network/jsonrpc/client.ts:56:17)
    at ForkStateManager.getContractStorage (node_modules/hardhat/src/internal/hardhat-network/provider/fork/ForkStateManager.ts:206:51)
    at VMDebugTracer._getContractStorage (node_modules/hardhat/src/internal/hardhat-network/stack-traces/vm-debug-tracer.ts:469:34)
    at VMDebugTracer._stepToStructLog (node_modules/hardhat/src/internal/hardhat-network/stack-traces/vm-debug-tracer.ts:312:39)
    at VMDebugTracer._stepHandler (node_modules/hardhat/src/internal/hardhat-network/stack-traces/vm-debug-tracer.ts:141:34)
    at node_modules/async-eventemitter/lib/AsyncEventEmitter.js:58:8
    at node_modules/async/internal/withoutIndex.js:9:16

This may be related to https://github.com/ethers-io/ethers.js/issues/2982
Can you try using the latest version of ethers?

i did. thanks @zemse. unfortunately the same issue is occuring. i think it might be related to https://github.com/NomicFoundation/hardhat/issues/2672 can't say for sure though

Hey @3commascapital, are you still facing this problem?

will check when next able


Move Telegraf migration into bootstrap script

High-level description
Move the shell script that performs the Telegraf migration into the bootstrap.py code.
During this, I realised that Path.mkdir mode is affected by the umask, so changed code in some other places to ensure that the correct permissions are used.
Also removed vestigial references to Windows.
Corresponding DC/OS tickets (required)

D2IQ-69295 permission denied in /run/dcos/telegraf/dcos_statsd/containers

Related tickets (optional)

COPS-6232

@mesosphere-mergebot changelog-not-required to be back-ported

@mesosphere-mergebot request-review @janisz @wavesoft @jkoelker


maxDefrost status "working" but not sending command

Hi all,
First of all, thank you for the awesome work.
I have a problem with command maxDefrost. It always shows status "working" and won´t send the command to the car. Can´t do a debug as there is no debug message since its stuck in "working".
Anyone else have this problem or can help me solve this problem.
Thanks

double entry


DSP Resource Tracker

Hello.
This issue is not related to the current project and I created it to contact you about a mod for the Dyson Sphere Program: DSP Resource Tracker. As I have not found any other way to do it.
I have been using your mod for a long time with pleasure. But then I had a break in the game (due to computer problems). And when I came back to the game I was disappointed to find out that neither mod does not work (the site is not available). Is there any chance that the mod will return? And if not, is there any way to get the source code to deploy it on my server and continue to support it?

+1 would also like it

also willing to buy you couple of cofees for it ;-)


Allow mutliple arguments for [] function

Currently [] supports only single arguments. However, in case of ruby objects this function can accept multiple arguments. Support this in Rubex also.

https://github.com/SciRuby/rubex/commit/ac2c8292c8785bff3d44eb286016f963e3bf3a8a


Kcolor minimizer

This K-coloring evaluation functions takes the approach of actually applying the randomly generated array of colors to the nodes in a graph.  It will return a fitness that minimizes the amount of miscolored nodes.
The current K-coloring Evaluation function does not actually color any nodes and should possibly be removed.

@jlm429
No conversation, no discussion here, just "close"?  The current k-coloring problem flawed, it does not aim to minimize the amount of adjacent nodes of the same color.  This code achieves it correctly.

I've had similar experiences with other open source projects before (ahem Apache).   I've actually had tickets closed and code stolen with no response! Argh.  I think sometimes it's easier to let trusted people implement changes at the expense of being more inclusive and democratic.  My initial issue was that including a csv folder instead of generating it on the fly as needed seemed kludgy.  My apologies; I should have made that clear.  After reviewing, I think it's ok as it is.  I will reopen and merge!

Hi @jlm429
Thanks for the re-look and taking the merge.   Good point on the csv files, those contain the edge connections for larger node graphs that are used in the example KColoringMinimizeTest.  In the example KColoringMinimizeTest, I have comments that give the URLs to where each of the test node graphs originated I.e. the example graph from MLrose, or the Peterson graph. Yes, agree the csv files were a dilemma for me, but I didn't like the way abalone had external data files sitting there in the same directory as the test code so opted for the separate directory.  I felt it was hard for users to draw the connection to solving K-coloring on those well-known graph configurations, thus the csv files containing the graphs helps get past that hump.  (Some of the graphs you will see in CS7280 network science)
Additionally, I would like to float the idea of removal of the original MaxKColorFitnessFunction and its testing example.   This implementation leads for users/students in the wrong direction actually giving an illogical fitness result that is backwards, leading them to believe higher values of K equate to a more difficult problem.  Where, in reality, lower values of K are always going to be more difficult.  Would you be receptive to another PR for removal of the original MaxKColorFitnessFunction?

That's fine with me.  Maybe the new one should be moved over to the same directory as the other opt problem tests then (/src/opt/test instead of /src/shared/test).

Hi John @jlm429 , I just got back around to look into the remaining cleanup of K-coloring.  Sorry I dropped off during the end of the holiday and into the new year.  But, noticed you already did half of the cleanup by moving to to /src/opt/test Commit 770059ce82.  And you added a Jython wrapper as well!  Commit 8afed80dc.  THANKS!
I still think it is worth while to do a little more cleanup, I would like to submit a PR for this:

Removal of erroneous MaxKColorFitnessFunction
Readme-kcoloring.md - to give usage notes on ABAGAIL K-coloring as a whole.

If you are still on-board with the idea, I can submit in a new PR?
Steve

Sure!


change TOR hidden_service_electrs port to 50001

What
Set TOR hidden_service_electrs to port 50001 directly without going through the SSL-proxied 50002.
Why
To improve compatibility with other wallets.
Incompatible wallets fail to create TOR+SSL connections.
This is the case with Trezor Suite. Comments on telegram claim that Blue wallet also has this issue.
All(?) wallets support TOR+TCP connections.
How
Change the port of the electrs TOR hidden service from 50002 to 50001
Scope

[ ] significant change to core configuration
[ ] independent bonus guide
[x] simple bug fix

Test & maintenance
Tested on

Electrum Desktop Wallet
Sparrow Wallet
Trezor Suite

This does not change anything in "Firewall & reverse proxy" nginx config, which also listens to 50001, proxying to 50002. So, there's conflict.
I could test this with Blue Wallet, it didn't work with RaspiBolt when I tested last time.

This does not change anything in "Firewall & reverse proxy" nginx config, which also listens to 50001, proxying to 50002. So, there's conflict.

I'm not sure I understand the conflicting part.
With this change, we would provide electrs service in two ways:

LAN: access to electrs service ssl-reverse-proxied via nginx on port 50002 (internal port 50001 -> ssl 50002)
WAN: access to electrs service via Tor on port 50001 (internal port 50001 -> onion:50001)

nginx is upstreaming port 50001 to 50002, Tor on same port 50001.
We also do this with bonus parts like e.g. Thunderhub:

LAN: thunderhub internal port 3010 -> (SSL/nginx) -> 4002
WAN; thunderhub internal port 3010 -> Tor hidden service -> onion:80

Right, it only affects onion service. Ok, will test this.

..
V iv bnnbgBvjbjnnbb
Nb jbnb nn bh jnnn
Gesendet von Proton Mail für Mobilgeräte
hhb
nnb b
-------- Original-Nachricht --------
Am 14. Mai 2023, 15:32, schrieb ⚡2FakTor⚡ < @.***>
@twofaktor commented on this pull request.
In guide/bitcoin/electrum-server.md:

@@ -284,7 +284,7 @@ Note that the remotejb devbnnbnnice needs to have Tor installed as well. ############### This section is just for location-hidden services ### HiddenServiceDir /var/lib/tor/hidden_service_electrs/ HiddenServiceVersion 3 - HiddenServicePort 50002 <IP_ADDRESS>:50002
It's possible to use both to keep compatibility with some wallets that require an SSL connection (it only supports SSL connection) e.g Bluewallet / Trezor suite and others
We could have only one onion address and only switch ports/SSL enable, on the wallets
Tested on MiniBolt: https://minibolt.info/guide/bitcoin/electrum-server.html#remote-access-over-tor
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.￼Message ID: @.>
[ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "[https://github.com](https://github.com/raspibolt/raspibolt/pull/1279#pullrequestreview-1425551409", "url": "[https://github.com](https://github.com/raspibolt/raspibolt/pull/1279#pullrequestreview-1425551409", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { @.***": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

.. V iv bnnbgBvjbjnnbb Nb jbnb nn bh jnnn Gesendet von Proton Mail für Mobilgeräte hhb nnb b -------- Original-Nachricht -------- Am 14. Mai 2023, 15:32, schrieb ⚡2FakTor⚡ < @.> @twofaktor commented on this pull request. In guide/bitcoin/electrum-server.md:
@@ -284,7 +284,7 @@ Note that the remotejb devbnnbnnice needs to have Tor installed as well. ############### This section is just for location-hidden services ### HiddenServiceDir /var/lib/tor/hidden_service_electrs/ HiddenServiceVersion 3 - HiddenServicePort 50002 <IP_ADDRESS>:50002
It's possible to use both to keep compatibility with some wallets that require an SSL connection (it only supports SSL connection) e.g Bluewallet / Trezor suite and others We could have only one onion address and only switch ports/SSL enable, on the wallets Tested on MiniBolt: https://minibolt.info/guide/bitcoin/electrum-server.html#remote-access-over-tor — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.￼Message ID: @.> [ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "[[https://github.com](<a](https://github.com%5D(#1279 (review)", "url": "[[https://github.com](<a](https://github.com%5D(#1279 (review)", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { @.": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

😮

open_mouth

Not sure how this happend, please disregard and delete.

Was that a mistake to '[Merge branch 'raspibolt:master' into the-jade-rabbitt-patch-1]' ? Not sure how to undo that ...

Was that a mistake to '[Merge branch 'raspibolt:master' into the-jade-rabbitt-patch-1]' ? Not sure how to undo that ...

All good. this catches up with base branch changes but usually isn't required.


Vertical split between windows should consistently separate status lines.

The vertical split between windows seems to separate status lines inconsistently. Below are some examples, using VIM 8.0.1608 on macOS 10.13.3, compiled with all default build options. (This may be the intended behavior, but that seems unlikely to me... If so, please help me understand the current behavior.)
Example 1: Left/Right Window Status Lines are Different
Start vim, and do a vertical split: vim -c "vsp". The result looks like this:

(Here the status lines have been set up to show text at the extreme left and right sides.) Note the cell at the intersection of the vertical split and the status lines. Where should it belong? Is it part of the vertical split? Or part of the left or right status line? The current behavior makes it appear to belong to the left window's status line (i.e. it matches the left window's status line color, and it lacks the vertical split char). This behavior makes text at the right side of the status lines appear inconsistent between the left and right windows, i.e. the left status line contains an extra (unusable cell) which is not present in the right status line.
Example 2: Top/Bottom are Different when Vertical Split is First
Now start vim, first do a vertical split, and then split each window horizontally, and go to the lower-left window: vim -c "vsp | sp | wincmd l | sp | wincmd h | wincmd j". The result looks like this:

Note how the bottom vertical split/status line intersection acts like Example 1 above, but the top two status line are actually separated by the vertical split. This shows a second inconsistency beyond that described in Example 1: the top and bottom act differently.
Example 3: Top/Bottom are Similar when Horizontal Split is First
Now start vim, but this time first do a horizontal split, and then split each window vertically: vim -c "sp | vsp | wincmd j | vsp". The result looks like this:

Note how, in this case, both the bottom and top pairs of status lines act like Example 1. So this is yet another inconsistency: the result depends on the order of vertical/horizontal splitting.
Further Examples
Try splitting into more windows (e.g. >= 3 vertical splits) and/or try various combinations of vertical and horizontal splits to demonstrate the current behavior.
Suggested Solution
In all cases, it would seem best for the vertical split to separate the status lines (i.e. the cell at the intersection should belong to the vertical split).

This is intentional.  The difference is needed to show what will happen if you try to drag a status line.
If the vertical separator continues, then the left and right status lines are dragged independently.

Thanks for the explanation!


fix(RELEASE-1304): invalid result value

It turns out Tekton only supports passing task results as pipeline results, so we need to pass the PLR name to the task first and then emit it from the task.

I propose promoting this to staging once merged. The only other commit we would be fast promoting would be the jq default bug fix, and if we don't, this pipeline will go to prod broken eventually

I propose promoting this to staging once merged. The only other commit we would be fast promoting would be the jq default bug fix, and if we don't, this pipeline will go to prod broken eventually

Agreed, exactly what I was gonna propose :)

I will merge this once task tests are finished. e2e does not test internal resources. The test actually confirms that this should work in the pipeline as well - it passes the exact same context variable.


Random Double Battle: Update level scales based on item/ability

This will prevent Pokemon like Azumarill from being too powerful due to
their low base stats with an ability that boosts it to incredible levels.
Open for discussion: Do we need to scale for these items/abilities:

Sheer Force
-ate abilities
Expert Belt

Please discuss a lot before merging!

@TheImmortal @Joimer

Sheer force, -ate: yes. Other permanent boost abilities that make a Pokémon good should be taken into account too.
Items in general: Better not, besides Soul Dew (which isn't used). In fact, can we get rid of Life Orb check and just leave in Eviolite check (as it is damn powerful)?

What about Thick Club?

Thick Club and Light Ball make useless Pokémon slightly less useless, I don't think they need a level nerf since they aren't exactly OP to begin with.

And how should Sheer Force be scaled? Should it do its stat calculation by factoring in counter['sheerforce']/damagingMoves.length or just the naive boost? Or maybe some percentage in between?

If we account for sheer force, it shouldn't be much. Any mon with sheer force will have a boosted attack regardless, so we don't need to count them. Naive boost will do.

Then what of -ate?

Same drill.

Added most boosting abilities (Hustle, Mega Launcher, Protean, Iron Fist...) and removed all item checks other than Eviolite.
How should I deal with Multiscale? Pokemon with it effectively have higher HP than usual, but I'm not sure how I'd scale it.

Multiscale is broken by a first hit, priority, double target, etc, dragonite already has decent level without nerf I think.

Do you think that BSR's usefulness could apply to Doubles, too? http://www.smogon.com/forums/threads/stats-ratings.39556/
If it were, then maybe we could just plug the modified stats into X-Act's formula?

I don't know if we would be adding too much code for level balance then tbh

That seems like something more tailored towards EV generation than this, I think. Randdubs uses its own EV generator instead of going 84-flat, after all.

Yeah, I don't want to bother with that level of calculations. We'd need to build a lookup table to get speed factors and run a bunch of other calculations that aren't worth it for the situation.

Actually, after thinking about it, I don't think you need to add every single ability/item. The major ones that double stats with no side effect (Pure Power / Fur Coat) should be included but everything else shouldn't imo.
Not sure what @Joimer would say about that.

Indeed, we only shall take into account the abilities which directly enhace Pokémon with no drawbacks. I'd still lower somewhat some -ate users, because, for instance, Sylveon's curren't level can lead to problems with Hyper Voice spam.

I "nerfed" the BST boost Guts/Hustle give to 1.2 instead of 1.5, but would you rather I remove it entirely?

Are you really going to add every ability? Speed Boost improves a Pokemon significantly, for example. I suggest remove all these and just include broken ones like Huge Power and Parental Bond.

Question: What levels are we getting right now on the important mons with the current state of this balance code? @ascriptmaster

Azumarill: 87 -> 78
Diggersby tho?: 87 -> 77
Medicham: 88 -> 78
Mega Med: 78 -> 70
Mega Maw: 81 -> 70
Sylveon: 77 -> 72
Aurorus: 77 -> 73

Mega Kang/Alt/Mence/Garde/Pinsir all stay at level 70.
I'll calc the rest later (lots of Sheer Force mons) but what do you think of these?

I removed Iron First/Hustle/Guts ability check.
In other news, Chansey's level goes from 84 to 78 under these changes.

TI wanted me to remove Adapatability, Tough Claws and -ate Abilities, so I did. Sylveon being Level 77 should be fine even with Hyper Voice spam.

I also think you should remove Sheer Force. Finally, Protean should maybe get half the boost of Parental Bond.

I removed Sheer Force and put Protean in its old place (so Protean is treated as if it gives a 1.3x boost)
I also bumped Huge Power and Pure Power's level scales to its actual effective base stat value. Fur Coat / Eviolite still have reduced-ish base stat increases (+13 instead of +18)

Hmm, how about slightly lowering Eviolite (increasing level) because it loses out on the item benefit that everything else has, so for example it would be the same level as another Pokemon with the same defenses after Eviolite (assuming other stats are equal...) but the other Pokemon has the benefit of an item.

Actually, just remove the added number. And the same for the abilities because of the same reason that Pokemon benefit from their ability, etc...

The added constant numbers are removed.


linksdk: improve log

[Detail]
improve log with ulog system
[Verified Cases]
Build Pass: <py_engine_demo>
Test Pass:  <py_engine_demo>

感谢提交 Pull Requests！
Thanks for submitting a pull request.

Security Check : 0d8417cde98afea040be5426b0fcdfa21f9e5d26

auto build success,number:234

autotest fail,number:1738


Custom atmospheric lighting

Hi.
It would be great if we could set the lighting color of the atmosphere (which is apparently plain white). To be clear, I am talking about the lighting which can be set using lightingFadeInDistance and lightingFadeOutDistance. The subtle rim halo around the globe could be made better with customization.
There is option to change the hue/brightness/saturation of the atmosphere, but that affects the whole globe and makes imagery weird when tweaked (photo below). We just need to change the lighting color.

Increasing saturation in sky atmosphere sandcastle example makes the imagery yellowish.

The effect we're looking for is somewhat similar to Google Earth.

@AliFlux I think this is possible with the newer atmosphere shader parameters.

OK to close as complete?


Deprecated NdElement data type still being used

I've come across this behavior which I believe is a bug:
>>> hmap= hv.HoloMap({1.0:hv.Table(np.array([[0, 0], [ 1, 0]]), kdims=['ROI'], vdims=['mean'])},
           kdims=['Time'], vdims=[])
>>> type(hmap.table().data)
holoviews.core.element.NdElement

Surely this should be using the DictColumns format or maybe the DFColumns format (if pandas is available?)
Edit: It is now clear the issue begins in the table() method call.

Fixed in https://github.com/ioam/holoviews/commit/ebca1fc0dea50cd6e917ea4f59b2efa9a5405ab6. Closing.


I get an error when building after following the install steps

Hi, I'm pretty new to iOS and developing with cocoapod, so I apologise in advance if this is a rookie mistake.
I get a build error after following installation step 5 (Add libRNMixpanel.a to your project's Build Phases).
This is the the error:
Undefined symbols for architecture armv7:
  "_OBJC_CLASS_$_Mixpanel", referenced from:
     objc-class-ref in libRNMixpanel.a(RNMixpanel.o)

(I get the same error when building for simulator, but with x86_64)
I'm pretty sure I'm doing step 5 correctly, but I am wondering if I'm not following step 6 correctly?
I go to the RNMixpanel.xcodeproj in my Project/Libraries folder, then edit the Header Search Paths like this -
$(inherited)
/Applications/Xcode.app/Contents/Developer/Toolchains/........
$(SRCROOT)/../../React
$(SRCROOT)/../../node_modules/react-native/React
$(SRCROOT)/../react-native/React

All recursive except inherited.
I also think it could be another issue with the installation of the cocoapod having to be for a specific version of ios, or not having copied some files across. Do you have any idea why I would be getting this problem?
Thanks

How does your Podfile look like?

How does you Podfile look like? Seems like an issue with the Pod itself, as RNMixpanel is loading and looking for Mixpanel, but can't find it.

It just has the one line in it, that mixpanel themselves tell you to use here -
pod 'Mixpanel'
Since posting I've found a solution, on stackoverflow -
Go to your target Build Settings -> Other linker flags -> double click . Add $(inherited) to a new line.
It was apparently because I was getting errors on pod install that looked like this -
[!] The `Corona [Debug]` target overrides the `OTHER_LDFLAGS` build setting defined in `Pods/Target Support Files/Pods/Pods.debug.xcconfig'.
This can lead to problems with the CocoaPods installation

Hopefully this will help anyone else who comes across this issue.

Glad you could fix it, next time it would be best to use the command pod init first, which should create a Podfile looking like this
# Uncomment this line to define a global platform for your project
# platform :ios, '7.0'

source 'https://github.com/CocoaPods/Specs.git'

target 'YourProject' do

end

target 'YourProject' do

end

to which you can add pod 'Mixpanel' like so:
# Uncomment this line to define a global platform for your project
# platform :ios, '7.0'

source 'https://github.com/CocoaPods/Specs.git'

target 'YourProject' do
    pod 'Mixpanel'
end

target 'YourProject' do

end

So does that mean that I don't have to use the xcworkspace that is generated? It would add it to the project itself?
Also, would I specify the Project just as 'YourProject', or would it be 'YourProject.xcodeproj'?
Thanks very much for the help.

Still need to use the workspace, but you just need to input the pod init command and it will take care of generating workspace and Podfile.
I would suggest to look at this https://cocoapods.org/

Thanks again.

No worries! Let me know if that works better :)


Make sure the CVAT backend and all necessary services

My actions before raising this issue

[ ] Read/searched the docs
[ ] Searched past issues

Steps to Reproduce (for bugs)

Expected Behaviour

Current Behaviour

Possible Solution

Context

Your Environment

Git hash commit (git log -1):
Docker version docker version (e.g. Docker 17.0.05):
Are you using Docker Swarm or Kubernetes?
Operating System and version (e.g. Linux, Windows, MacOS):
Code example or link to GitHub repo or gist to reproduce problem:
Other diagnostic information / logs:
Logs from `cvat` container

Please provide logs from cvat_server and cvat_db containers

Duplicate of #5922


TestNewMetricsHandler fails race tests

See #3206
This test fails race and it has been disabled for race tests.
@girishranganathan you are the author of this, please asses the cause of failure.

cc @douglas-reid


Linkcheck and prepare alpha

Adds linkcheck to make sure links work

note conf.py includes some ignore links that linkcheck gives false negatives for

Removes blank pages (commits reference issue to add them back later)
Adds missing content to pages
closes #1958

Needs https://github.com/RadeonOpenCompute/rocm-docs-core/pull/120 to add new words for spelling linting


Support for proxying to path in a (virtual) folder

Imagine you want to have the proxy done in such a way that
example1.com/my/method is proxied to example2.com/some/folder/my/method. The current implementation does not support this case as the resource path is always taken from the original request.
This could be achieved by having additional parameter in ProxyOptions and adding this parameter's value to the proxied request URL.

This issue was moved to aspnet/Home#2497


Yoda seems to have disappeared? I'm getting a 404 when I try to use it

tagging @jens-markussen - hopefully you can fix it or let me know what's happened.

Hi. Yoda is moving, and the move was perhaps done a bit sudden. Sorry about that.
Now avaialble at https://plan4hub.github.io/yoda/
I will be updating further links, descriptions, etc.

Ah, thanks for the prompt response!

Welcome

Do I need to redo the setup, or can I just use the new URL?

Looks like I do


bump to 0.0.3-SNAPSHOT

Coverage remained the same at 71.413% when pulling dd36121ff0a0c69e077873bc0a9dfc755975e9f0 on akmorrow13:master into de5c0c7b43896b256d95698d714774f8985da044 on bigdatagenomics:master.

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


Is xdemangle.hpp redundant ?

I see that xdemangle.hpp provides the demangle function.
https://github.com/compiler-research/xeus-cpp/blob/e9df705c9f6a4e6b7f7f0b7655726ca4dc294ce7/src/xdemangle.hpp#L41-L42
This should technically be used in xinspect in the implementation of find_type (as done for xeus-cling)
But we are currently using find_type_slow (the function name suggests this might not be the best approach ?!)
https://github.com/compiler-research/xeus-cpp/blob/e9df705c9f6a4e6b7f7f0b7655726ca4dc294ce7/src/xinspect.hpp#L82-L99
But if we are not trying to implement a "find_type_fast" and are comfortable with what we have, let's get rid of xdemangle.hpp ?

Yes, it can go away.


Improved Workflow Build Release

Questions
Answers

Description?
Improved Workflow Build Release

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
Relative to PrestaShop/PrestaShop#26767

How to test?
N/A

This change is

@PierreRambaud what more do you want ? I've followed @Progi1984 guidelines

Thank you @jf-viguier


just some hackthon branch i will delete later

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-kit@0.2.47-alpha.704f61.0@codecademy/gamut-labs@13.2.10-alpha.704f61.0@codecademy/gamut@29.7.1-alpha.704f61.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://606637246887e500dc2ff128--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.2.47-alpha.5cccda.0@codecademy/gamut-labs@13.2.10-alpha.5cccda.0@codecademy/gamut@29.7.1-alpha.5cccda.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60663b8f681d03008077b8c7--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.2.47-alpha.9832f4.0@codecademy/gamut-labs@13.2.10-alpha.9832f4.0@codecademy/gamut@29.7.1-alpha.9832f4.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60664711ee4daa12420fdb94--gamut-preview.netlify.appDeploy Logs


fix: update NodeFeatureDiscovery to work with the latest OCP

@e0ne i think we should link to red hat NFD installation guide, keep only the changes we need in the pci section
image can change and cause issues if we specify and users will just copy paste it

@e0ne i think we should link to red hat NFD installation guide, keep only the changes we need in the pci section
image can change and cause issues if we specify and users will just copy paste it

I updated an image as mentioned in a NFD guide and added a link to official documentaion


Test run hangs when ran from VSTS given incorrect runsettings format

Description
When runsettings contains incorrect attribute (For e.g. name instead of friendlyName in dataCollectorRunSettings or LoggerRunSettings) and test is ran via VSTS, then VSTS hangs.
Steps to reproduce
In runsettings add LoggerRunSettings and add trx logger. Instead of mentioning friendlyName="trx", write name="trx" (which is incorrect)
Run test dll from vsts.
Test run hangs.
Expected behavior
Test Run should not hang
Actual behavior
Test run hangs.

This issue is happening because SettingsException, TestPlatformeException, InvalidOperationException are eaten up by TestRequestManager and not forwarded to design mode client. Thus design mode doesn't know about the failure and keeps on waiting for test discovery/excution to complete.


Parse Failed. Unexpected String

Parse failed at Rakuten_FAQ_Bot.dialog: SyntaxError: Unexpected string in JSON at position 6893

SyntaxError: Unexpected string in JSON at position 6893
at JSON.parse ()
at ConfigurationResourceExporer.loadType (/Users/ar-s.ganapathysubram/Rakuten_FAQ_Bot/Rakuten_FAQ_Bot/node_modules/botbuilder-dialogs-declarative/lib/resources/resourceExplorer.js:201:29)
at new CoreBot (/Users/ar-s.ganapathysubram/Rakuten_FAQ_Bot/Rakuten_FAQ_Bot/node_modules/botbuilder-dialogs-adaptive-runtime/lib/coreBot.js:22:45)
at /Users/ar-s.ganapathysubram/Rakuten_FAQ_Bot/Rakuten_FAQ_Bot/node_modules/botbuilder-dialogs-adaptive-runtime/lib/index.js:211:16
at /Users/ar-s.ganapathysubram/Rakuten_FAQ_Bot/Rakuten_FAQ_Bot/node_modules/botbuilder-dialogs-adaptive-runtime-core/lib/serviceCollection.js:129:67
at Array.reduce ()
at /Users/ar-s.ganapathysubram/Rakuten_FAQ_Bot/Rakuten_FAQ_Bot/node_modules/botbuilder-dialogs-adaptive-runtime-core/lib/serviceCollection.js:129:40
at Array.reduce ()
at ServiceCollection.buildNodes (/Users/ar-s.ganapathysubram/Rakuten_FAQ_Bot/Rakuten_FAQ_Bot/node_modules/botbuilder-dialogs-adaptive-runtime-core/lib/serviceCollection.js:114:32)
at ServiceCollection.makeInstances (/Users/ar-s.ganapathysubram/Rakuten_FAQ_Bot/Rakuten_FAQ_Bot/node_modules/botbuilder-dialogs-adaptive-runtime-core/lib/serviceCollection.js:171:21)

Hi @ganogs,
In order to investigate this, I need you to provide a lot more information.

What platform are you on?
What version of Composer?
What language is this bot made in?
How are you hosting the bot?
How are you testing the bot?
What is the bot doing in the dialog where this error occurs?
Where is the offending JSON coming from?
What objects does this JSON describe?
Can you provide an example?

The error message suggests a syntax error in some JSON. Have you attempted to debug the bot and check the specified JSON for errors?

Closing this issue due to inactivity. If you still require assistance, please comment and we can reopen the issue.


Why function and not arrows

Due to your question, we can only use "function"s here and the reason comes from those "this" keywords we're using inside the function.
This article may help you about why you get some errors when you change this file's "function"s to "arrow function"s:
https://dmitripavlutin.com/when-not-to-use-arrow-functions-in-javascript/
app/models/user.js:
// ** generating a hash **
userSchema.methods.generateHash = function(password) {
    return bcrypt.hashSync(password, bcrypt.genSaltSync(8), null);
};

// ** checking if password is valid **
userSchema.methods.validPassword = function(password) {
    return bcrypt.compareSync(password, this.local.password);
};

How can I ever thank you enough ¯_(ツ)_/¯


Multiple Subscription Identifiers in PUBLISH message

The section about Subscription Identifier in PUBLISH message of the the MQTT v5 specification says:

Multiple Subscription Identifiers will be included if the publication is the result of a match to more than one subscription, in this case their order is not significant.

I see that only 1 identifier is read in MqttV500PacketDecoder:
private static MqttBasePacket DecodePublishPacket(byte header, IMqttPacketBodyReader body)
{
  //...
  packet.Properties.SubscriptionIdentifiers.Add(propertiesReader.ReadSubscriptionIdentifier());
}

Is this an issue or I'm misundertanding the specs?

Hi, in my opinion this is correct. The subscription ID is sometimes added multiple times. So that the while loop above ensures that the IDs are all added to the same list.

Thanks for the quick reply!
I understand, sorry, I haven't read the code carefully enough.
My doubt originated from this use case:

subscribe the same topic twice, with different Subscription Id
publish on the same topic
the message received contains only one Subscription Id, not both

I'll further investigate, it may be a broker issue...
I'm using Mosquitto 1.6.0 btw

I made some tests and I can confirm that the MQTTnet client processes the Subscription Id correctly, so I'm closing the issue. Sorry for the waste of time.
Moreover, I found on the MQTT spec why the behaviour described in my previous comment is correct:

If a Server receives a SUBSCRIBE packet containing a Topic Filter that is identical to a Non‑shared Subscription’s Topic Filter for the current Session, then it MUST replace that existing Subscription with a new Subscription


Stats theme is not matching with the home page theme.

Description
Can fix the Stats theme with dark mode and use duo-gradient to enhance UI.
Screenshots

Assignee: Please assign this issue to me.
Additional information
No response
What browser are you seeing the problem on?
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

please pull the updated code to your local. That one is old. Although someone messed up it already so assigning this to you.
This is current one

Make sure that it should go with current theme of the webpage.

which one should I change:
1.

or 2.

the difference is just in the amount of white bg

@Yash636261 Please assign me with this task.

which one should I change: 1.  or 2.  the difference is just in the amount of white bg

first one looks good to me, go for it.


Addition of mixed resende scheme

Resende scheme that utilizes both primer sets for base scheme amplicon 2

Worked on my test set, will merge in and if anyone wants names changed or moved around we can try that out and mess with it

Worked on my test set, will merge in and if anyone wants names changed or moved around we can try that out and mess with it


Allow serializer to override default name generator

This would allow EVMConnect to behave the same as EthConnect, naming outputs output/output1/output2 - but it will need a separate serializer for inputs and outputs.

Codecov Report

Merging #14 (14ec4c0) into main (4390cf5) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #14   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           32        32
  Lines         2479      2485    +6
=========================================
+ Hits          2479      2485    +6

Impacted Files
Coverage Δ

pkg/abi/outputserialization.go
100.00% <100.00%> (ø)

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 4390cf5...14ec4c0. Read the comment docs.


"Get In Touch With Us" Section Design Improvement

Describe your changes
I have mainly changed form's shades, background, borders, padding, margin, etc. I have mixed the properties of the CSS to make its look-and-feel even better.
Screenshots - If Any (Optional)

Issue ticket number and link - If Any
https://github.com/amupedia2021/Project-Amupedia/issues/70
Checklist before requesting a review
[✓ ] I have performed a self-review of my code.
[✓ ] Starred the repository.
[  ] Added My Name to the Contributor's List.
[ ✓] Followed the repository's Contributing Guidelines.
[ ✓] I ran the app and tested it locally to verify that it works as expected.
Additional Information (Optional)
If you want any design changes, you can specify them. I would love to consider your requirements. Also, if you like the work, I want to work on other design modules of website to create consistent design.

Hello @iamRabia-N,
Could you please tell me how did you set up the project locally.?
and did you install any dependencies.?
could you please help me out as I am facing merge conflicts with package.json and yarn.json.

Hi @SoundaryaKoutharapu!
In order to avoid errors:

Reinstall node.js (in case if you have already installed).
Checkout if npm installation is fine by checking its version in vs code terminal.
Install yarn by command "yarn install"
Then run "yarn dev".
In my case, these steps help me setup balanced and correct repo local environment for me.

For project setup and any issue... You can ask on the channel, please. so we can resolve it. Thanks for replying btw @iamRabia-N

Thanks for the contribution we will provide feedback. soon!

Yeah sure @anmode, from next time onwards I'll ask in channel.

Hi @SoundaryaKoutharapu! In order to avoid errors:

Reinstall node.js (in case if you have already installed).
Checkout if npm installation is fine by checking its version in vs code terminal.
Install yarn by command "yarn install"
Then run "yarn dev".
In my case, these steps help me setup balanced and correct repo local environment for me.

Hello @iamRabia-N ,
These steps have worked for me!
Thanks much.

@iamRabia-N it is optimized for mobile?

@iamRabia-N it is optimized for mobile?

Yes.
Take a look at this screenshot.

great!!


b2BlockAllocator is unsafe

I posted on this issue in the forums:
http://box2d.org/forum/viewtopic.php?f=4&t=10159
b2BlockAllocator uses static arrays for its allocation space.  While this is okay, it also references these static variables in instance methods other than the constructor, which is not okay (and violates C++ coding conventions).  As a result, there is a rare bug where the memory space for a b2World becomes corrupted when it passes across multiple compilation units.
The fix is easy and is outlined in the post on the forums.

The good ole static initialization order fiasco raises its head once again. Thanks for pointing this static code out!
Looks like this may be the only place that's used. I just did a quick check for that in the library code however.
Having non-constant class static member variables that have to get run-time initialized also looks thread unsafe. Some undesired scenarios that I could imagine arising:

The table gets initialized more than once (okay no big deal);
the one-byte sized array values get torn in by multiple threads in the midst of read modify write cycles on architectures that don't have unique memory locations for every byte;
a thread sees s_blockSizeLookupInitialized as set before the array values have actually been all written to memory but goes merrily on its way assuming the table value it just looked up is correct.

There may be lots of other thread safety problems (posts seem to suggest never using Box2D with multiple threads) but if we can get rid of this one at the same time, that seems like an added bonus.
If nothing else, I dislike having to have the additional startup time overhead this initial assignment adds when it is unnecessary. Looking at the code, would a more attractive fix instead be to do the following?

move the static members from the class scope (the b2BlockAllocator.h header file) and into the top of the file scope of the b2BlockAllocator.cpp file;
remove the s_blockSizeLookupInitialized variable;
make the s_blockSizes variable constant (or constexpr if using C++11 or newer);
make the s_blockSizeLookup variable constant (or constexpr if using C++11 or newer); and,
write out and do the full 641 element array assignment directly to the s_blockSizeLookup array.

This way initialization should be done completely at compile time and the tables are constant so no runtime startup cost nor runtime modifications get done to the two remaining variables. No doubt some template meta-programming could be used instead for writing all the data out. I cringe at the obfuscation that TMP often seems to add though and would prefer the 641 entry values all written out.

I think 1 is important if you remain static.  The reason this hit me was (on Windows), Cocos2d links Box2d and puts it in a DLL, while the game is a separate layer above the DLL that also links against Box2d.  Because the static is in the header, this results in two different static instances of the same variable.  I found the error by watching the static pointer for the array change over the method call boundary between the game and the DLL.
I had not caught the thread issues, but you are completely right.  It seems to me that your only options are to hard code the static variable or treat everything as instance variables and eat the memory overhead.

Thanks for the report. Maybe I should be more clear that Box2D is not designed to be used in a DLL or to be thread safe.

I updated the FAQ: https://github.com/erincatto/Box2D/wiki/FAQ

My apology for re-opening the thread. However, I am facing the similar issue. Additionally, tried to create the static library in Xcode but failed. I know I am a noob, but can you please help me for this?

I spent some more time looking at this. This is not a static initialization order fiasco. There is no problem with the order of operations in the block allocator. This is certainly a shared library (DLL) problem with static data.
Another potential issue with shared library usage is memory allocation. You have to be sure that memory allocated in the library is freed in the library.
Another issue is thread safety. Box2D was not designed for thread safety, so there is no point in making one small aspect of Box2D thread safe.

Putting aside whether or not these issues are real, I’d rather functionality that could be handled at compile time got done then instead of at run time.
I don’t know what’s happened to Erin’s Box2D master branch but I believe that the change I prefer (the patch) can be seen at: Comparing erincatto:master...louis-langholtz:fix/block-allocator-static-initialization · erincatto/Box2D https://github.com/erincatto/Box2D/compare/master...louis-langholtz:fix/block-allocator-static-initialization (which should be a comparison between what I last believed the master branch of Box2D to be with the patch applied).
If you’d like to use the patch, great. I won’t object if you don’t though.
Thanks.

The cost of filling out that array is not important unless you are allocating and destroying thousands of b2BlockAllocators per second. I don't think this is a reasonable use case.
If I change the layout of the block allocator, someone will have to be very careful not to mess up that explicitly filled array. In my opinion, this is brittle code.
Thanks for the input. I think this is an important issue to be aware of. I'd be more inclined to remove all static data. However, there may be use cases where this wastes a lot of memory.


multi-gpu train command

hi boss:
how to train on multi-gpu in single machine. how to spell train command.

This will be explained after the release of training details.


Testing styled components with snapshots

We are currently using styled components with Jest snapshot testing. Works pretty nicely except for if I add a new styled component to one of my react components it causes the css class names and then therefore the snapshots to change in multiple unrelated components.
The diffs in git are now all css classname changes in completely otherwise untouched parts of the codebase which is making the diffs both really large and also confusing.
So I am assuming by adding a new styled component some other place this affects how all the classnames are generated, throughout the project?
It this something I am doing wrong? Or is this by design?
I'm using styled-components v2

There's a separate project that makes this whole experience with Jest a lot nicer: https://github.com/styled-components/jest-styled-components
:+1:

Thanks, I did check out the project, and it looks like it makes the snapshot error messages nicer looking, which is great.
However the original issue still exists. I add a new styled component in one react component, unrelated components (not using the component I changed in any way) have their classnames updated so a new snapshot gets generated for multiple components, which makes the diff really big and misleading.
I'll try and create a reduced example to illustrate the issue.
If the class names are always being regenerated then I can look at another way of testing that doesn't use snapshots, however it is a nice feature.
Thanks for the quick response and all the work on this great library. Other than this small issue, I'm finding it awesome to use!

This should be circumventable with our babel-plugin with the ssr option. It generates stable componentIds. I'm not sure if that applies to the jest plugin though

Ok just to document my findings here:  I just tried this out with both the babel-plugin ssr option and using jest-styled-components: expect(tree).toMatchStyledComponentsSnapshot() and looks like I'm still getting unrelated css class names being updated.

cc @MicheleBertoli

Hello @jameslnewell, It'd be awesome if you could provide a minimal example to repro the issue.
I'll be happy to help and improve the package.

Lets move this conversation over to this issue please :) https://github.com/styled-components/jest-styled-components/issues/29


allow null values in csv

This pr allows null values in csv:

if the field is of type string, then an empty field is considered null (,,), anything other is turned into a string (i.e , , is a single whitespace string)
if the field is of type number, when the trimmed field is empty, we consider the value null (i.e ,,, , , are both null numbers) otherwise we try to parse the number.

Thanks, @MarinPostma!


Configurable altitude range

Added a min and max altitude range in config
Added alt variance to calls to set_position in API

I did not change test files, or add a set_position wrapper as I initially intended. This does not also currently look up alt for each long/lat. It is purely random and is a very basic start. At the least, they can't query their data for alt == 0 and ban us all in one quick sweep.
Big note, I am still not sure what unit altitude needs to be sent as. For the time being, I put a default 0.75 to 2.5.

Thanks a lot for the PR ! let me merge it.

@solderzzc Fantastic! Thank you for reviewing and merging so quickly.

just gitpull it, start bot and alt still zero ? (lat e long edited):
2016-08-19 10:05:06,953 [PokemonGoBot] [INFO] [x] Coordinates found in passed in location, not geocoding.
2016-08-19 10:05:06,954 [PokemonGoBot] [INFO] [location_found] Location found: xx.495530, xx.603577 (xx.49553, xx.603577, 0.0)
2016-08-19 10:05:06,954 [PokemonGoBot] [INFO] [position_update] Now at (xx.49553, xx.603577, 0)
2016-08-19 10:05:06,954 [PokemonGoBot] [INFO] [login_started] Login procedure started.

@MZorzy Because your saved position last had an alt of 0. It is possible that my edits didn't save alt to account locations cache. I'll take a gander.

@mjmadsen I noticed a sudden elevation on each step:
"alt": 16.070621059253707 to "alt":  82.59619883853987.
There's a sudden 66 units (meters? feet?) of elevation since we use random alt between min and max altitude. I think the bot will be detected. It's like the user jumps very high when playing. Your thoughts?

@supergithubo It's meters. Our initial thought was at least putting something out so we're not sticking out like a sore thumb with an alt of 0 all the time. I suppose we could generate the alt randomly somewhere else and then just rely on noise. I'd prefer if we could fetch an alt from our long/lat and just use noise as we move around.
You could look up alt for your starting coords, then change your min/max based off that number.

I'm just a bit cautious on how they will detect the bot from elevating in very high altitude then suddenly goes down to low altitude on each step. I guess I'll just set min/max with a very short range in between so the bot elevates smoothly. Thanks @mjmadsen


PR references are too spammy

dogger/dogger.io

Should be a direct reference to the issue when it's the same repo, but an indirect reference if it's a different repo.


Persist pending payment amount

Resolves #599
Builds upon #621

@dleutenegger could you please rebase it on top of master, currently there are some conflicts.

Will wait for @dangeross to finish his review before merging.


[Playground Bug]: Need to up flutter version to deploy front-end successfully

What happened?
The current version of flutter is not suitable for deployment. An error occurred while building the docker image
Issue Priority
Priority: 0
Issue Component
Component: beam-playground

.take-issue


[Sprint 22/23 / PI-143 ] [Feature] add config service

Jira Link:
https://ichiro-its.atlassian.net/jira/software/projects/PI/boards/2/backlog?selectedIssue=PI-143
Description
I have added two Srv named SaveConfig and GetConfig to atama_interfaces for reading and writing config on Atama using Atama_app.
Describe problems, if any, clearly and concisely.
Describe any changes that have been made in this pull request.
Type of Change

[ ] Bugfix
[ ] Enhancement
[x] New feature
[ ] Breaking change (fix or feature that would cause the existing functionality to not work as expected)

How Has This Been Tested?

[ ] New unit tests added.
[x] Manual tested.

Checklist:

[x] Using Branch Name Convention

feature/JIRA-ID-SHORT-DESCRIPTION if has a JIRA ticket
enhancement/SHORT-DESCRIPTION if has/has no JIRA ticket and contain enhancement
hotfix/SHORT-DESCRIPTION if the change doesn't need to be tested (urgent)

[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have made the documentation for the corresponding changes.

Please chacnge the ubuntu-latest in workflow to ubuntu-20.04


High CPU usage on startup when using --file option

Hi,
I'm using a file with songs, that ashuffle should use (generated with mpc). The file has ~17.000 songs. I'm starting ashuffle like this: ashuffle --queue_buffer 2 --file /tmp/ashuffle-songlist.txt
After ashuffle has started the CPU usage of mpd goes up to 100 % (one core fully loaded) for several minutes (measured 8 minutes on last try). Time increases with longer song lists.
CPU is a i5-4570 @ 3.20GHz.
During this time no songs are added to the queue.
There probably is some communication between ashuffle and mpd about the song list. Why and can it be deactivated?

The --nocheck option fixes this. But actually the description implies that this option is only useful, when also --exclude is given:
When reading URIs from a file, don't check to ensure that the URIs match the given exclude rules. This option is most helpful when shuffling songs with -f, that aren't in the MPD library.

When ashuffle is started with the -f option, it still tries to apply the exclusion rules (set with --exclude). This works by querying MPD for each song in the song list to verify that it is both A) a valid song in MPD, and B) it does not match any of the exclusion rules. I think this is what you're running into. With a large song-list, ashuffle will synchronously execute these queries, and it is probably swamping MPD.
--nocheck fixes the issue because it disables this checking. I didn't mention it in the docs because it didn't occur to me that it could affect performance 😀. There is a simple fix to make this "edge" a little less sharp: We can skip this processing if no exclusion rules are set. Basically implicitly enable "no check". Unfortunately this is a bit of an API break, since right now ashuffle will detect if you supply invalid URIs to -f, and implicitly enabling nocheck would disable that behavior. I'm going to be cutting a v2 release soon, so I'll consider including this break then. Until then, I advise using --nocheck.
Longer term, we should still handle large libraries even if they set exclusion rules. I chatted about this with @huctica the other day, and he had the idea of lazily processing exclusion rules, so we can amortize their evaluation (i.e. we can process a small number of songs quickly, queue them, then check the rules against the remaining songs in the background). This seems like a good approach to me.
I'm going to leave this bug open to track:

Implying --nocheck when there are no exclusion rules in v2.
Benchmarking (and tracking) startup performance with large libraries.
Lazily evaluating rules for URIs given with -f.

Both the startup time issue, and the benchmarking should be resolved with PR #47. I will follow-up with tracking the lazy rule evaluation (which looks to be somewhat complex in its own right) in issue #48.


[BUG] Folders with a lot of notes are not being displayed entirely

Describe the Bug
In the navigation bar, if a folder has a lot of notes it just stops displaying them.
To Reproduce
Steps to reproduce the behavior:

Go to 'File Explorer'
Click on 'a folder with probably 100+ notes'
Scroll down until it stops displaying

Is the Problem related to Obsidian 0.13+ / Live Preview Mode?
No. It also appears with Legacy Mode turned on.
Relevant Screenshots

Checklist

[ x] I have updated to the latest version of the theme (Obsidian Settings → Appearance → Themes - Manage → Update).
[ x] I checked that the issue does not occur when using the default theme.
[ x] In case I am missing an UI element, I have checked that it cannot be re-enabled with the Style Settings plugin.

Came here to say the same. Also, scrolling now gets wonky and skips to different areas. The latter is really apparent in ipados when Obsidian is in slide-over mode.

@cashew90 Did this bug only occur after an update of Shimmering Focus it was this problem there before?
@daxmonson Could you post a screen recording of your issue?

Hard to tell... I only discovered it by chance because I don't often open folders to scroll through them. Most of the time I use Quick Switcher. So I really can't tell, sorry. For the moment, I switched to the Minimal theme where everything works as intended.

Hmm, I have a suspicion. Could you turn off the file explorer icons in the settings and then try again?

Sure, I tried with the file explorer setting icons off and it still won’t render files names after a certain point. Tried to count exactly how many down, but the file explorer kind of skips around and loses place after a while. Will get a loom or similar recording up, will probably have to be over the holiday when I get a minute.

Same for me.

Okay, has been fixed thanks to @SlVrb (reference)


A way to get template as []byte?

By now i can only see a way to retrive generated template as a string (or write it directly to io.Writer). It would be nice to also have a possibility to retrive it as []byte, without overhead of []byte-string-[]byte conversion.

Just use bytes.Buffer as io.Writer:
var buf bytes.Buffer

// Pass buf into the corresponding Write* function for the template function
WriteFoobar(&buf)

// Obtain byte slice
b := buf.Bytes()

UseByteSlice(b)

// Now the buffer may be re-used - just call buf.Reset() on it


[Feature Request]: Combo-box  - option to remove the clear icon

The problem
Into the combo-box (https://web-components.carbondesignsystem.com/?path=/docs/components-combo-box--default), i would like an option to remove the clear icon, to force the user to select always an option.
Like into the dropbox ( but dropbox does not allow to search , so user need to scroll into the list).
The solution
I used this to test :
document.querySelector('#my-combo-box').shadowRoot.querySelector('#selection-button').style.display = "none";
But the shadowRoot need to be initialized ( and we don't have control on that ) so i could not use the document.querySelector('#my-combo-box').shadowRoot.querySelector('#selection-button').style.display = "none";   as i am not sure that the shadowRoot will be available.
Examples
No response
Application/PAL
No response
Business priority
High Priority = pressing release
Available extra resources
No response
Code of Conduct

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

@cedricpelle wouldn't making the combo box required achieve what you are looking for?