Issue 150 additions

add missing count functions to the C api

@kdheepak can you help @HimanshuJain17 do the review and merge?

I assume that in general any modifications to the signatures of the C interface will require regeneration of the swig generated interfaces?  Is there any way we can spit out a warning if that is needed from Travis or something like that?

I don't mind having one of the builds have USE_SWIG=false  (probably the gcc-6 build,  but I would prefer to not add a new build to do it, and just keep the 3 current builds and have one of them have that option

Are you opposed to having 3 current builds with the USE_SWIG=false as an separate optional build similar to the OSX builds? Because I think over time as we get more configurations I'm not sure if we can keep to just three.

I am expecting we will have to expand it in the future,  that is why I want to be really sure we need it before we add any.  My understanding was the swig code generation is somewhat independent of the compiler used, which is primary difference in the 3 current current builds.  And having USE_SWIG=false on any one will test any potential issues with mismatches in the repo file and the library, therefore it is unclear to me the advantage of separate builds with just that option changed.

Okay, I see your point I've made the change.

For the USE_SWIG option, will that apply to any of the interface build options  (BUILD_PYTHON, BUILD_MATLAB, BUILD_JAVA,  BUILD_OCTAVE in the future?)

Now that I am looking through your code changes it looks like it is just an option for travis.  Might it make sense to have that as an option in CMAKE, so you can turn off the use of SWIG for any of the of the library builds with a single option, or turn the swig generation on.  Might even make sense to be default off.

If I understand correctly, we'd just have to rename GENERATE_PYTHON to USE_SWIG.

Although, if we did that, if someone wanted to build the MATLAB extension without SWIG ( since it uses a different version of SWIG ) and the Python one with SWIG, they'd have to do two different builds and installs.

I would say in a majority of user cases they shouldn't need to regenerate the code through SWIG, as they are just downloading the library and building it.  If I understand correctly the only time you need to regenerate the SWIG is when the public signatures on the C shared library has changed.  As developers that may be reasonably often but even this case it will likely need to be done for all interfaces otherwise there will be a mismatch.
I am just not seeing a use case where you would want to use swig for one component and not others except in active testing of one interface, in which case requiring two builds doesn't seem like that big of deal.

I don't think I realized this PR would cause such a big discussion

it probably makes sense to have USE_SWIG=false on one of the mac builds as well

@kdheepak are you going to add the updated swig generated files to the Repo?  I don't really have a matlab swig version set up.

I'll do this, but won't be able to get to it till Monday.

This PR now has the latest Python and MATLAB files.


flaky failure solved

The flaky test failure is resolved with my fixes. I run the test 10,000 times, and every time the test passes.
Description
try {
Thread.sleep(100);
} catch (InterruptedException e) {
System.out.println("Exception");
}
Related Issue
https://github.com/TooTallNate/Java-WebSocket/issues/1296
Motivation and Context
This removes the test failures.
How Has This Been Tested?
With my solution, I run the test 10,000 times with the following command, and the test has never failed.
mvn test -Dtest=org.java_websocket.issues.Issue580Test#runNoCloseBlockingTestScenario[7]
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

We cannot simple sleep in the main code


Leading Zeros Normalization

WIP: This PR Adresses https://github.com/RalphTro/epcis-event-hash-generator/issues/112

Now that the test is not failing any more, there is actually no need to merge it


fix: fix authwit package

https://github.com/AztecProtocol/aztec-packages/pull/7180 broke aztec-nr as it's renamed a function without updating all references to it

thanks

Benchmark results
Metrics with a significant change:

avm_simulation_time_ms (Token:mint_public): 637 (+843%)
avm_simulation_time_ms (Token:assert_minter_and_mint): 60.8 (-73%)
app_circuit_proving_time_in_ms (Token:transfer_public (avm)): 3,657 (+26%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Proof generation
Each column represents the number of threads used in proof generation.

Metric
1 threads
4 threads
16 threads
32 threads
64 threads

proof_construction_time_sha256_30_ms
11,407
3,058
1,441 (+4%)
1,469 (-1%)
1,620 (+5%)

proof_construction_time_sha256_100_ms
43,765
11,796 (+1%)
5,462 (+1%)
5,428 (+1%)
5,385 (+1%)

proof_construction_time_poseidon_hash_ms
78.0
34.0
34.0
58.0
87.0 (-2%)

proof_construction_time_poseidon_hash_30_ms
1,516
414
199
222 (-1%)
268 (+2%)

proof_construction_time_poseidon_hash_100_ms
5,753
1,563 (+1%)
718
764 (+1%)
794 (-1%)

L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
4 txs
8 txs
16 txs

l1_rollup_calldata_size_in_bytes
1,412
1,412
1,412

l1_rollup_calldata_gas
9,476
9,460
9,464

l1_rollup_execution_gas
610,297
610,281
610,285

l2_block_processing_time_in_ms
748 (-2%)
1,414 (+1%)
2,682 (-1%)

l2_block_building_time_in_ms
25,230
50,377 (-1%)
99,151

l2_block_rollup_simulation_time_in_ms
25,230
50,377 (-1%)
99,150

l2_block_public_tx_process_time_in_ms
21,676
46,633 (-1%)
95,464

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 8 txs.

Metric
3 blocks
5 blocks

node_history_sync_time_in_ms
7,044 (+1%)
9,878 (+1%)

node_database_size_in_bytes
12,181,584
16,121,936

pxe_database_size_in_bytes
16,254
26,813

Circuits stats
Stats on running time and I/O sizes collected for every kernel circuit run across all benchmarks.

Function
time_ms
bytecode_size_in_bytes

GasToken:_increase_public_balance
67.6
13,873

GasToken:set_portal
16.9 (-1%)
3,495

Token:constructor
95.3 (-2%)
24,207

FPC:constructor
64.8 (-3%)
13,893

GasToken:mint_public
53.2 (-1%)
10,241

Token:mint_public
:warning: 637 (+843%)
19,216

Token:assert_minter_and_mint
:warning: 60.8 (-73%)
13,034

AuthRegistry:set_authorized
31.9 (-1%)
7,869

FPC:prepare_fee
146 (-21%)
15,187

Token:transfer_public
35.0 (-7%)
31,425

FPC:pay_refund
142 (+21%)
20,080

Benchmarking:increment_balance
2,662 (-1%)
15,465

Token:_increase_public_balance
56.5
15,089

FPC:pay_refund_with_shielded_rebate
154 (+5%)
21,167

Public DB Access
Time to access various public DBs.

Function
time_ms

get-nullifier-index
0.147 (-1%)

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
256 leaves
512 leaves
1024 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.4
16.9
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.7
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.603
0.518
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
48.8
75.6 (-1%)
131 (-1%)
244
469

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
95.9
159
287
543
1,055

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.498
0.464 (-1%)
0.450 (-1%)
0.442
0.438

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
60.1
112
182 (-1%)
352
693 (-1%)

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
109
207
355
691
1,363

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.508
0.501
0.481 (-1%)
0.476
0.474 (-1%)

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
72.9
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
133
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
0.519
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
85,672
670,983

Transaction size based on fee payment method
| Metric |  |
| - |  |


Build artifacts in CI

as was discussed here let's add artifacts to CI for quick reference.
We can include executor, genesis, configs, and other binaries (iroha, irohad, kagami, etc)

Does this include a suggestion to stop tracking all that generative artifacts in git?

Does this include a suggestion to stop tracking all that generative artifacts in git?

Executor tracking will be stopped


add typehints=dict(a=int, b=float, c=string) and fields=['a', 'b', 'c'] to discover

These would help when discovering when particular types are known (typehints) as well as when column names are not available (fields)

something like
discover(resource('/path/to/file.csv'), fields=list('abc'))
discover(resource('/path/to/file.csv'), typehints=dict(d='int32', e='float64', f="string[32, 'A']"), fields=list('def'))


Generated Ids & Jongo migration: generated id is not set after an insert

Hi,
in Jongo 1.0, when using an String id field with "@Id" annotation, the String representation of an objectid would be used as id. That was great.
With Jongo 1.2, I have 2 issues:

in some cases, the generated id is not set
can't get any combination of "@MongoId" or "@MongoObjectId" to generate such a string represnetation of an object id without creating a nested $oid structure in mongo, which then creates issues when doing looking on with the _id parameter

more specifically, from the 6 examples listed here: http://jongo.org/#mapping, here is what I get following an insert
Friend1

pojo.id => 0
mongodb: "_id": 0

Friend2

pojo.id => 0
mongodb: "_id": 0

Friend3

pojo.id => null
mongodb: { "_id": { "$oid": "5612886bb388aeb4a8cb6961" } }

Friend4

pojo.id => "5612886a6b7cc42640791746"
mongodb: { "_id": { "$oid": "5612886a6b7cc42640791746" } }

Friend5

pojo.id => "5612886a6b7cc42640791747"
mongodb: { "_id": { "$oid": "5612886a6b7cc42640791747" } }

Friend6

pojo.id => "5612886b6b7cc42640791748"
mongodb: { "_id": { "$oid": "5612886b6b7cc42640791748" } }

and in particular, with jongo-1.0 and Friend3, I used to get
Friend3 (jongo-1.0)

pojo.id => "5612886bb388aeb4a8cb6961"
mongodb: { "_id": "5612886bb388aeb4a8cb6961" }

Any clue what I am doing wrong, how to get back the generated id in all cases, and how to use the simpler string representation?
Cheers,
Yann.

@yduponchel you may want to take another look at this issue with the current master.  Several fixes for id fields just went in.

Hello,
You can test the fixes @ctrimble are talking about in this early version.
This version must be considered as a SNAPSHOT of future 1.3 release
<dependencies>
  <dependency>
    <groupId>org.jongo</groupId>
    <artifactId>jongo</artifactId>
    <version>1.3-early-20151113-0140</version>
  </dependency>
</dependencies>
...
<repositories>
    <repository>
       <id>cloudbees-jongo-early-release</id>
        <url>http://repository-jongo.forge.cloudbees.com/release</url>
        <releases>
            <enabled>true</enabled>
        </releases>
        <snapshots>
            <enabled>false</enabled>
        </snapshots>
    </repository>
</repositories>

Please reopen if bug still occurs


Preliminaries to integrate scriptor

This PR is related to https://github.com/viur-framework/viur-core/pull/664

Let's wait for https://github.com/viur-framework/viur-core/pull/664 and the support of the  scriptor in the viur-cli to be complete.


Cannot read properties of undefined (reading 'translations')

Hey guys! I cloned the project and installed the dependencies and got this error. No changes were to the code. Something I thought was interesting was my first time installing the project it worked fine. After receiving this error and multiple times of cloning, installing, nothing will change. I had even thought it was the fact I upgraded to astro 4.12.3, but after just starting over I'm stuck.
TypeError: Cannot read properties of undefined (reading 'translations')
    at Module.t (/mnt/d/Developement/Phlex Design/Client Projects/Advanced-Astro-v4-i18n-main/i18n:astro:124:14)
    at eval (eval at instantiateModule (file:///mnt/d/Developement/Phlex%20Design/Client%20Projects/Advanced-Astro-v4-i18n-main/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:53441:24), <anonymous>:18:39)
    at 404 (eval at instantiateModule (file:///mnt/d/Developement/Phlex%20Design/Client%20Projects/Advanced-Astro-v4-i18n-main/node_modules/vite/dist/node/chunks/dep-CzJTQ5q7.js:53441:24), <anonymous>:19:12)
    at callComponentAsTemplateResultOrResponse (file:///mnt/d/Developement/Phlex%20Design/Client%20Projects/Advanced-Astro-v4-i18n-main/node_modules/astro/dist/runtime/server/render/astro/render.js:90:31)
    at renderToAsyncIterable (file:///mnt/d/Developement/Phlex%20Design/Client%20Projects/Advanced-Astro-v4-i18n-main/node_modules/astro/dist/runtime/server/render/astro/render.js:132:32)
    at renderPage (file:///mnt/d/Developement/Phlex%20Design/Client%20Projects/Advanced-Astro-v4-i18n-main/node_modules/astro/dist/runtime/server/render/page.js:31:30)
    at lastNext (file:///mnt/d/Developement/Phlex%20Design/Client%20Projects/Advanced-Astro-v4-i18n-main/node_modules/astro/dist/core/render-context.js:135:31)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async callMiddleware (file:///mnt/d/Developement/Phlex%20Design/Client%20Projects/Advanced-Astro-v4-i18n-main/node_modules/astro/dist/core/middleware/callMiddleware.js:21:10)
    at async RenderContext.render (file:///mnt/d/Developement/Phlex%20Design/Client%20Projects/Advanced-Astro-v4-i18n-main/node_modules/astro/dist/core/render-context.js:163:22)

I also noticed this:

Hopefully that helps, it was just something I noticed. Thank you!

Hi @NHLopez,
Thanks for the report.
Could you please elaborate:

is there a specific page the error happens?
OS
node version

Thanks.

Hey @BuckyBuck135,
Not sure what happened but it works now? I just cloned it fresh and now its fine.
Before. it would give that error whenever the page transitions. For example, if I do npm start the homepage showed up, when I clicked about, I would see the about page slightly then it would throw the above error so I thought it might be the transitions but when I looked on astros website it was fine.
Either way it works now so thank you so much. Just incase it happens again would you know anything I could do?

Ok great!
Without being able to reproduce the bug, it's difficult to say where it was coming from. let's hope it doesn't crawl back up again! :)

Hey @BuckyBuck135 !
I made a 4 minute video of trying to catch the bug. I ended up doing a video from a fresh clone to installing its dependencies to using the site and it crashing. Usually it happens when I try to click links to pages as well but it was fine this time around. I tried to show as much detail as possible. I hope this is helpful in some sort of way since I can't give more information.
Either way have a great day! :)
https://github.com/user-attachments/assets/19873b9d-f75a-4222-a718-d99c76fadf43

Hi @netmaster2000,
Thanks for your report. I've reopened the issue.
I'm currently away for family reasons, so I won't have time to dig into this just now. In the meantime, if you do have some free time to run some debugging tests to try and pinpoint which page / component causes this issue, that would be a tremendous help.

After some quick tests, I looks like the issue does not impact the production site, which is good. I'll try to tackle this when I have some free time.
npm run build + npm run preview shows you a copy of the deployed site on localhost.

Hi guys,
Thank you for your patience. I'm back at the keyboard after a family break.
The error seems to be coming from the custom 404 page. Short term: deleting this 404 page gets rid of the error.
Now, I'm working towards finding a more sustainable solution.

Hi everyone,
Since Astro patched the issue upstream with this PR, I reinstated the custom translated 404 route in the kit.
Make sure you upgrade Astro to at least v4.15.9, place 404.astro in the routes/ folder and the matching translation in locales/, as demonstrated in the kit.


Tabular: Removed predictor_legacy.py and tabular_prediction.py

Issue #, if available:
Description of changes:

Removed predictor_legacy.py and tabular_prediction.py
Added predictor_legacy.py code to predictor.py

This PR is purely a file move / delete PR. No logic changes have occurred.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-956-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-956/1/index.html


Font Resource Generation

Added struct generation for ttf/otf fonts. Also made the String struct to not appear in the generated file, if it contains no actual strings. Hope you don't mind the author addition? I can remove it if so.

Ah! Forgot tests!

Done!

Added @tonyxiao's filter commit. And realised I didn't include any exclusion filtering, so refactored some of the code for reusing, and added the same thing to mine.

Regarding your latest changes: Nice! Even when I'm a big fan of applying the Boy Scout Rule (as "Always leave the campground cleaner than you found it." ⚜) :+1:, we should get that PR now ready to be merged before it starts to diverge further.
Two remaining things:

Bumping the version doesn't belong in a feature PR. That's part of the release process and should happen, either directly on master or (in the best case) on a dedicated branch, when releasing the next gem version over RubyGems.
I see the authors in the gemspec also as major contributors and maintainers of the project, who take responsibility in case of issues, review PRs and made significant and lasting contributions to the code base. So for now, I think it's too early to add you. But from what I saw, I thought you could be perhaps interested in taking more responsibility in the project. If you want to do that, your support would be very welcome. Once you reach that point, where we both feel confident that you will stick around the project, I'd be more than happy to add you to the gemspec / you can add yourself. ;) Feel free to contact me via Twitter DM or on my Git author mail, if you prefer to give me feedback on that point 1-to-1.

Agreed to both points. Squashed all commits into a single one.
P.S. sorry to @tonyxiao, his commit got caught in the crossfire, and unfortunately all his credit got squashed also.

no big deal at all, happy to see this all getting merged in :)

I noticed that I had made some wrong assumptions about the way font files are used in Cocoa/CocoaTouch. The correct way would be to parse the ttf/otf header and retrieve the name (or names) contained in it. These are the names that are referenced when using fontWithName:. I'm thinking of using ttfunk for atleast working with ttf fonts. Looks like we won't be merging this pull request too soon.

@carlossless but don't you have to add them to the plist to use them anyway? wouldn't it be much more KISS when you just parse the plist?

@stefreak probably, though the plist contains only the filenames for font's that should be use. The filenames are not the same thing as the actual font name, which you use in fontWithName:. So my previous problem remains.
@mrackwitz would implementing something like opentype.js's font parser just for parsing fonts in xcres, make sense? I'm not sure If I'm willing to implement a whole separate package, and maintain it.
Also need to find the time to get this done...


Fix complex kernel matrix entries for FidelityStatevectorKernel

Summary
Takes the real part of the kernel array when enforcing a PSD matrix.
Closes #605.
Details and comments

where kernel entries could potentially have nonzero complex components due to truncation and rounding errors when enforcing a PSD matrix.

As the psd method is used by other kernels would it be applicable for the fix to be in the method itself, rather than external here for it's one use here and only applies to this specific kernel?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 7 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 90.053%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
2544

Relevant Lines:
2825

💛  - Coveralls


[FEATURE] Show time on chat

Description
I think we should also show the time of chat. I mean when the message was sent.
Screenshots
No response
Additional information
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Correct! you can go ahead!


start of the obervation type module

[ ] Test gapfill methods on other ModelObstypes
[ ] test gapfill methods on wind-derived ModeObstypes
[ ] Add them to the gapfill tests


Add deprecated v1 methods to BaseModel

I'll note that I have not copied tests over from v1 for the newly-added methods. Let me know if you have opinions about if/how we do that.

conflicts and comments, please update.


Glitched tooltips

Hello,
Still using / testing v<IP_ADDRESS>, everything works very well. :+1:
I noticed a weird animation / glitch on tooltips, under Windows :

Thx! Will try to find out what is going on.


Installer

I think I have covered the stuffs.
Let me know if I missed something.
Thanks!

Will try to address your comments when I am back.
Moving out for a while.

Thanks again! :+1:


1.2.0 release

@daniel-j-h has a bunch of nice enhancements for the library :pray: I've collected them onto a 1.2.0 milestone and will start bugging @jacquestardie for some time to work on a release. End of March too late for you, Daniel?

We've got 2 PRs to sort out and then we'll have a 1.2 pre-release on Monday, April 12.

Pushed 1.2a1 tag.

1.2a1 is on PyPI now: https://pypi.org/project/mercantile/1.2a1/.

I don't have anything else to add to the API. Anybody else?
If there are no objections, let's have a b1 Wednesday and 1.2.0 on the 19th.

Wunderbar! :tada:

1.2a1 checks out with a couple of my major work projects and we don't have any new features to be added. I'll tag and release a 1.2b1.

And we're done. Thank you, everyone!

https://pypi.org/project/mercantile/1.2.0/
https://mercantile.readthedocs.io/en/stable/changelog.html#id1


Add Evolve (#82)

Short Description:
Tries to evolve everything that it catches. If it fails (if it doesn't have enough candy), it checks if it should transfer the Pokémon.
Should add some settings to let user customize what to evolve.
Would be nice if there's a way to find out remaining candies.
Fixes:

Inability to evolve a Pokémon

Great, this is finally coming, is my README.me GOOD? or you figure it out all by yourself?
Have you already join our slack, would you mind drop me a PM.


[tests] Enable playwright tests on linux (#4838)

Playwright dependencies have been installed on the helix image.
Issue: https://github.com/dotnet/aspire/issues/4623 .
Fixed by resolution of https://github.com/dotnet/dnceng/issues/3122 .
Microsoft Reviewers: Open in CodeFlow

Looks like we need to wait for the new VMs to roll out.

/azp run

[ ] Next: confirm that the packages are actually installed on the agent. If they are, then are we missing something else which is causing the tests to complain about missing dependencies?


FATAL: Listen error: unable to monitor directories for changes. Visit https://github.com/guard/listen/wiki/Increasing-the-amount-of-inotify-watchers for info on how to fix this.

FATAL: Listen error: unable to monitor directories for changes. Visit https://github.com/guard/listen/wiki/Increasing-the-amount-of-inotify-watchers for info on how to fix this.

If you are not interested in the technical details and only want to get Listen to work:

If you are running Debian, RedHat, or another similar Linux distribution, run the following in a terminal:
echo fs.inotify.max_user_watches=524288 | sudo tee -a /etc/sysctl.conf && sudo sysctl -p

I executed that command.
fs.inotify.max_user_watches=524288
fs.inotify.max_user_watches = 524288

...OK?
close


new: add some tests to the Falco-driver-loader

This PR adds some e2e tests for the falco driver loader.
We just need to update the falco CI job in the following way:
 sudo ./build/falco-driver-loader.test

Moreover we need to add the toolchain to the test job, so clang, gcc, dkms

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

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


Keybindings not working after 0.3.1 release

After updating to 0.3.1 through homebrew, none of the keybindings work anymore. For example, alt-f for fullscreen now just types a ƒ character. The service is running and accessibility is approved. Has this happened to others?

Fixed on reboot

I'm having the same issue but reboot isn't fixing it :-\ also I installed with logging and it's not even outputing logs, or giving me any stdout or stderr


Fix floodPublish

Motivation

We use "msg.from" in "_publish" method, however it's only available in "StrictSign" signature policy (lodestar/eth2 uses "StrictNoSign") https://github.com/libp2p/js-libp2p-interfaces/blob/libp2p-interfaces%404.0.4/packages/interfaces/src/pubsub/index.js#L637

Description

Use "receivedFrom" which is always available https://github.com/libp2p/js-libp2p-interfaces/blob/libp2p-interfaces%404.0.4/packages/interfaces/src/pubsub/index.js#L687

Codecov Report

Merging #198 (20661ed) into master (db4a666) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #198   +/-   ##
=======================================
  Coverage   73.17%   73.17%
=======================================
  Files           2        2
  Lines        1909     1909
  Branches      110      110
=======================================
  Hits         1397     1397
  Misses        512      512

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 db4a666...20661ed. Read the comment docs.


Angular 10 support

Hello!
I was trying to update from Angular9 to Angular10 and got this error:
Using package manager: 'npm'
Collecting installed dependencies...
Found 57 dependencies.
Fetching dependency metadata from registry...
                  Package "ngx-tour-core" has an incompatible peer dependency to "@angular/core" (requires ">=7.0.0 <9.0.0" (extended), would install "10.0.0").
                  Package "ngx-tour-core" has an incompatible peer dependency to "@angular/common" (requires ">=7.0.0 <9.0.0" (extended), would install "10.0.0").
                  Package "ngx-tour-md-menu" has an incompatible peer dependency to "@angular/animations" (requires ">=7.0.0 <9.0.0" (extended), would install "10.0.0").
                  Package "ngx-tour-core" has an incompatible peer dependency to "@angular/router" (requires ">=7.0.0 <9.0.0" (extended), would install "10.0.0").
× Migration failed: Incompatible peer dependencies found.
Peer dependency warnings when installing dependencies means that those dependencies might not work correctly together.
You can use the '--force' option to ignore incompatible peer dependencies and instead address these warnings later.

Any plans to upgrade to Angular10?

I join the request

Any news? Is this repository still maintained?

Due to the lack of responses I finally managed to "solve" this issue with the --force flag. Everything is working as expected.

@fergardi you should leave this open until Angular 10 support is officially added to the project.

+1

+1

+1

@isaacplmann any idea when 4.2.0 will be published?

We are on Angular11 already, and no news...

I forked this library re-published under another name, I've not completed the automation of publishing (i order to avoid this scenarios) but for anyone interested you can use @ngx-tour/core
https://www.npmjs.com/settings/ngx-tour/packages

I have created a fork and I have published a new version under a new name: ngx-ui-tour. It's compatible with Angular 11. Test it out and let me know if it works as intended.


Fluentbit 0.13.1 crashes frequently

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind feature

What happened:
fluentbit version used by tarmak crashes frequently:
Fluent-Bit v0.13.1
Copyright (C) Treasure Data

#0 0x7fb614850e2f in ???() at ???:0
#1 0x558c51945ae2 in msgpack_sbuffer_write() at lib/msgpack-2.1.3/include/msgpack/sbuffer.h:84
#2 0x558c51b57155 in msgpack_pack_ext_body() at lib/msgpack-2.1.3/include/msgpack/pack_template.h:890
#3 0x558c51b57155 in msgpack_pack_object() at lib/msgpack-2.1.3/src/objectc.c:72
#4 0x558c519464c4 in pack_map_content() at plugins/filter_kubernetes/kubernetes.c:321
#5 0x558c51946d12 in cb_kube_filter() at plugins/filter_kubernetes/kubernetes.c:491
#6 0x558c518e69aa in flb_filter_do() at src/flb_filter.c:86
#7 0x558c518e43fc in flb_input_dbuf_write_end() at include/fluent-bit/flb_input.h:642
#8 0x558c518e5f5c in flb_input_dyntag_append_raw() at src/flb_input.c:894
#9 0x558c5190a98d in process_content() at plugins/in_tail/tail_file.c:290
#10 0x558c5190b7d8 in flb_tail_file_chunk() at plugins/in_tail/tail_file.c:651
#11 0x558c519091c7 in in_tail_collect_static() at plugins/in_tail/tail.c:129
#12 0x558c518e649b in flb_input_collector_fd() at src/flb_input.c:995
#13 0x558c518ed1de in flb_engine_handle_event() at src/flb_engine.c:296
#14 0x558c518ed1de in flb_engine_start() at src/flb_engine.c:515
#15 0x558c5188d4c6 in main() at src/fluent-bit.c:824
#16 0x7fb6147482e0 in ???() at ???:0
#17 0x558c5188b949 in ???() at ???:0
#18 0xffffffffffffffff in ???() at ???:0

people are seeing this one a lot on fluent/fluent-bit:0.13.1 (what we are running)
https://github.com/fluent/fluent-bit/issues/584
What you expected to happen:
Fluentbit keeps on running
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version):
Cloud provider or hardware configuration**:
Install tools:
Others:

/assign


DKIM > Your message is not signed

[X] Documentation

I need know how sign with DKIM the outgoing mails
I get -1 on mail-tester.com >> Your message is not signed with DKIM

[X] Always

Docker information
docker info
Containers: 7
 Running: 7
 Paused: 0
 Stopped: 0
Images: 10
Server Version: 18.09.1
Storage Driver: overlay2
 Backing Filesystem: extfs
 Supports d_type: true
 Native Overlay Diff: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
 Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
Swarm: inactive
Runtimes: runc runsc
Default Runtime: runc
Init Binary: docker-init
containerd version:<PHONE_NUMBER>f7fe2f4e74d43e2fc7ccd237edcbce
runc version: 96ec2177ae841256168fcf76954f7177af9446eb
init version: fec3683
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.15.11-mainline-rev1
Operating System: Ubuntu 18.04.1 LTS
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 1.95GiB
Name: scw-02
ID: XIU2:DK73:7VIS:Y3PG:2NTA:L5C7:EEFI:OKNH:GHZW:FNJ4:DY7A:GYYL
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Labels:
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false
Product License: Community Engin

docker images hardware/mailserver --digests --filter "dangling=false"
hardware/mailserver   1.1-stable          sha256:e451a7a01d7ab69af45575822a2cfce7d14473129f566fc66527cbb202b91063   b90667289394        11 days ago         380MB

Description
Briefly describe the problem you are having in a few lines.
I gent 9/10 score  mail-tester.com
thanks in advance,

Do you have the email when you sent your message to mail-tester.com?  docker logs mailserver or docker logs mailserver | less for searching.

This issue has been marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.


Update dnspolicy reconciler to use gateway status for placed cluster details

What
Update the dnspolicy reconciler to use the gateway status to build up the list of dns.ClusterGateway instead of having to query ocm APIs to retrieve the placed clusters.
With the introduction of the status aggregation of the hub Gateways as proposed here, we will now have enough information on the Gateway resource itself to build up the dns.MultiClusterGatewayTarget that we need to pass to SetEndpoints
This should disconnect the dnspolicy entirely from ocm (apart from the cluster event handler) and fix a current issue where we are only dealing with IPAddress type addresses.
Requires:

https://github.com/Kuadrant/multicluster-gateway-controller/issues/113

This issue is stale because it has been open for 30 days with no activity.

@mikenairn @makslion how do we feel about this making end of sprint?

should be completed.


Specify S3 store region in VRG input

When using an S3 store like velero, the concept of region doesn't apply and default's to us-east-1.  However, if we need to support a cloud based S3 store, yes, we may need take region as an input, among other TBD parameters.
Originally posted by @vdeenadh in https://github.com/RamenDR/ramen/pull/33#discussion_r605384004

These are technically closed, and latest work to move these out of DRPC is being done in #117


Rework mapping

Make the mapping of handlers to events work and easier to use.

Testing farm results and Copr events are handled as other events.
Mapping support multiple triggers for one job and get's a correct config part.

Fixes: #465

TODO:

[x] More tests (more combinations of triggers and configs).
[x] Check the commenting.
[x] Add docs for the decorators.
[ ] Remove internal job type (e.g. copr_end) from the JobType enum in packit. (After this PR, it can contain only the config-relevant values.)

The commenting part is still WIP -- I need to check/fix all the tests since I've changed the handling of the comment events. (Now, they should be handled in the same way as other events.)

recheck


Does Pygbe run in Centos 7?

Does Pygbe run in Centos 7?

We haven't tested it, so I wouldn't be able to say yes, but it runs on Ubuntu so I think there is a big chance that it'll run.


[PhpConfigTransformer] Failing test for lazy on service

Can't fix it right now. But this is the failing test that shows lazy is ignored on service.

What are you missing to finish this?

I moved onto a new project and finished my migration. I didn't want to leave the issues I discovered unreported. I can imagine that you want to close the PR, it's fine.  Thanks for the amazing converter.

Thanks for fast reply and slim test :+1:
Fortunatlly, it was similar to the other PR you've sent so I managed to fixed it my short work break :)
Merged in https://github.com/symplify/symplify/pull/3411/files

Amazing! 👏


Can not get feed, returns 404 not found.

I tried the test MVC app and authenticated successfully, but could not get the user feed. It always returns
Response status code does not indicate success: 404 (NOT FOUND).
Could you please help on this?

The Instagram API is changing. New clients doesn't support some endpoints:https://www.instagram.com/developer/changelog/

Deprecation of /users/self/feed


[Issue #739 fix] Keyboard overlap fix on Android

Fixes #739.
The issue targets the apps that run in fullscreen (like Sugarizer) where resizing of the UI is disallowed. I researched about this issue and found a Cordova plugin which detects the opening/closing of virtual keyboard on fullscreen and provides a workaround for the resizing issue.
Plugin: https://github.com/ionic-team/cordova-plugin-ionic-keyboard
This plugin should be added to the APK builder (make_android.sh file):
cordova plugin add<EMAIL_ADDRESS>
I have provided the config needed for resizing and also removed the padding on login screens in this PR.
Results
Login:

Markdown:

Get Things Done:

Writer:

Added the required plugin to the sugarizer-apkbuilder repository in llaske/sugarizer-apkbuilder#2. Please review @llaske.

Very smart fix! Thanks a lot.
Two things however:

Plugins should be added in the config.xml file
Unfortunately it don't work on iOS, so I've limited to Android

Thank you @llaske! I'm glad you liked the fix. I have added the the plugin to config.xml as a commit to this branch. I'm not sure if you can merge changes from this branch again so if need be, I'll add another PR to add the plugin to config.xml.
I'll research more on this and see what I can find to fix the iOS version as well. Happy to make Sugarizer better on all platforms.

No need to do a new branch. I've fixed config.xml before merging your branch. See here.
Regarding iOS, the plugin should work but it doesn't. I've tested all KeyboardResize and KeyboardResizeMode options (specific iOS options mentioned in the documentation) without success.
BTW it's not a blocking issue on iOS because iOS allow the user to scroll the screen by default. So the keyboard doesn't prevent the user to see the whole screen.

I see, that's good. Would still look for a method to completely solve it on iOS as well. Thanks!

@dhruvmisra I'm packaging the next Sugarizer version and I've got a weird behavior on the Gridpaint activity on Android (screen is flashing when starting to draw).
I suspect it could be related to this fix. Are you able to test it on your side on a real Android device?

It seems to be working fine on my Android device.

Could this be a device specific issue?

Hmmm. Thanks for testing. Did you test on the latest dev branch and with Sugarizer APK builder?
I'm going to do more testing on my side.

Yes I tested it with the latest dev branch. I'll try it on another device too.

I've tested on another tablet and it works.
I'm working to find the reason of this issue but it's not related to this fix: removing the cordova plugin don't fix the issue.
Thanks for your help.

Issue opened here https://github.com/llaske/sugarizer/issues/847


How to start a transaction

I use db in torndb_ Db. begin() starts the transaction. How to do it in ezmysql?
What should I do when I set autocommit to True in the connect constructor and some logic needs to manually start a transaction?

Sorry, it does not support transaction currently.
Welcome to make a PR if you want to add the support.


⚠ Bofrost.at has degraded performance

In e2f7ab1, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 3467 ms

Resolved: Bofrost.at performance has improved in 1773884.


filter_state: add shared pointer accessor

Signed-off-by: Kuat Yessenov<EMAIL_ADDRESS>Commit Message: Allow access to mutable filter state objects via shared pointer. This allows copying objects to different streams (e.g. from downstream to upstream filter state, or from downstream to another internal downstream filter state). Read-only objects are locked to the stream to avoid mutation via another stream.
Risk Level: low
Testing: unit
Docs Changes: none
Release Notes: none

/retest

According to https://github.com/envoyproxy/envoy/issues/14320
we should not add throw exception at worker thread and eventually remove the existing one.
What about starting to use StatusOr here?

/assign @KBaichoo

StatusOr sounds fine to me. The error here is not runtime error, it's a programming error, in case someone treats filter state as mutable accidentally.

Thanks @KBaichoo . I agree that we should stay consistent and then, separately, revisit exception use. IMHO, we should have used leaned on C++ type system to enforce const-ness instead of doing runtime checks.

/retest

/retest

/retest


simplify inventory

Codecov Report

Merging #654 into master will decrease coverage by 11.39%.
The diff coverage is 39.53%.

@@            Coverage Diff             @@
##           master     #654      +/-   ##
==========================================
- Coverage   88.02%   76.62%   -11.4%
==========================================
  Files          47       47
  Lines        3374     3359      -15
==========================================
- Hits         2970     2574     -396
- Misses        404      785     +381

Impacted Files
Coverage Δ

vent/menus/inventory.py
10.86% <0%> (-81.99%)
:arrow_down:

vent/api/actions.py
85.92% <100%> (-9.39%)
:arrow_down:

vent/menus/tools.py
54.71% <80%> (-10.53%)
:arrow_down:

vent/menus/add.py
20.63% <0%> (-79.37%)
:arrow_down:

vent/menus/logs.py
21.73% <0%> (-69.57%)
:arrow_down:

vent/menus/editor.py
19.44% <0%> (-69.45%)
:arrow_down:

tests/test_menu.py
35.71% <0%> (-64.29%)
:arrow_down:

vent/menus/add_options.py
27.27% <0%> (-60%)
:arrow_down:

vent/menus/inventory_forms.py
42.85% <0%> (-57.15%)
:arrow_down:

vent/menus/services.py
31.81% <0%> (-40.91%)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

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


Get version programmatically in integration tests

I'm using ASP.NET Core 5. I've set up my version format as "'v'VVV".
In an integration test, I'm using the WebApplicationFactory. To test an endpoint, I'll use:
var route = $"/api/v${version}/${controller}/${action}";
var response = await client.GetAsync(route);

I'd prefer not to hardcode version.
Is it possible to get it from the test server? Maybe I can get the api versioning service from the container and get the formatter version from there? Unsure how though.

Wow that is so much more info than I expected. Thank you! :smiley:
I'm going to wrap all that into something that only runs during testing. I think I'll create a test-only controller that exposes all that detail. That will keep it clean in my production code.
Last question, if you don't mind. During the integration test I do have the ApiVersion used by the server, and I know that it has format "'v'VVV". But I don't know how to format that. So I thought to search the test server's IServiceCollection for the library's service, and hoped that if I found it I'd be able to use it to format the version to "v0" or "v0.1.0" or whatever. I'm not sure what service this library registers so didn't know what to look for. Is that possible?
If not, I'll stick to the way you described. BTW it would make an excellent addition to the docs, you've explained it so clearly!

No problem.
To be clear v is never part of an ApiVersion. ApiVersion does support IFormattable with custom formatting. The format 'v'VVV is meant for formatting the display version, which is most commonly used in OpenAPI documents. If you're using a numeric format, the values 1.0 and 1 are semantically equivalent. API Versioning, however, will not omit the 0 if originally unspecified. Typically, that only matters for display purposes. v1.0 and v1 are exactly the same. Similarly, [ApiVersion("1.0")] and [ApiVersion("1")] are also the same. If you really, really want to get and use the format, you can get it from ApiExplorerOptions.GroupNameFormat by injecting IOptions<ApiExplorerOptions>.
I got the impression that the client and server were separated in your test suite, but maybe they are together. If they're together, you can also use the IApiVersionDescriptionProvider service (part of the API Explorer extensions) to list all API versions. The collection of API versions reported is flat for the entire application. If your APIs are not symmetrical in versions, then this might not work for because not every reported version will be available for all APIs. This service is mostly used to determine the versions required to produce OpenAPI documents.
Honestly, if you do have references to the server implementation, then the best option is to go through the API Explorer. That is how all OpenAPI documents are generated. By going through that path, you don't need to worry about formatting URLs, API versions, etc. The API Versioning extensions for the API Explorer will automatically collate APIs by their version. If you combine that with the IApiVersionDescriptionProvider, you can automate most everything (which was also a considered use case in the original design 😉 ).
Something like:
// get all these from ID
IOptions<ApiExplorerOptions> options;
IApiVersionDescriptionProvider versionProvider;
IApiDescriptionGroupCollectionProvider groupProvider;

// create a lookup
var groups = groupProvider.ApiDescriptionGroups
                          .Items
                          .ToDictionary(group => group.GroupName, group => group.Items);

// process each group (e.g. [formatted] API version)
foreach (var description in versionProvider.ApiVersionDescriptions)
{
    var apis = groups[description.GroupName];

    foreach (var api in apis)
    {
        var version = api.GetApiVersion();   // versioning extension; if needed
        var deprecated = api.IsDeprecated(); // versioning extension; if needed
        var url = api.RelativePath;

        // TODO: use info
    }
}

There's a lot of different ways this could be used. I haven't done it myself, but there is probably some way to match this up with IHttpClientFactory so you can use it in a test base class. Something like:
public class ApiServer : IDisposable
{
    private readonly Lazy<TestServer> server;
    private bool disposed;

    public ApiServer() => server = new Lazy<TestServer>(CreateServer);

    public IServiceProvider Services => Server.Services;

    protected TestServer Server => server.Value;

    public HttpClient CreateClient(string name)
    {
        var factory = Server.Services.GetRequiredService<IHttpClientFactory>();
        var client = factory.Create(name);

        // TODO: the 'magic' here would be extending/replacing the default
        // IHttpClientFactory to apply 'http://localhost/api/v{version}'.
        // this might need more parameters or a 'magic string' such as
        // 'Values-V1', which is probably 'ok' for testing purposes
        client.BaseAddress ??= new Uri("http://localhost");

        return client;
    }

    public HttpMessageHandler CreateHandler() => Server.CreateHandler();

    public void Dispose()
    {
        Dispose(true);
        GC.SuppressFinalize(this);
    }

    protected virtual void Dispose(bool disposing)
    {
        if (disposed)
        {
            return;
        }

        disposed = true;

        if (!disposing)
        {
            return;
        }

        if (server.IsValueCreated)
        {
            server.Value.Dispose();
        }
    }

    protected virtual TestServer CreateServer()
    {
        var builder = WebHost.CreateDefaultBuilder()
                             .ConfigureServices(OnConfigureServices)
                             .UseStartup<Startup>(); // your real or alt startup

        // use TestServer from Microsoft.AspNetCore.TestHost
        return new TestServer(builder);
    }

    private void OnConfigureServices(IServiceCollection services)
    {
        // TODO: configure your services here
    }
}

[CollectionDefinition(nameof(ApiTestCollection))]
public class ApiTestCollection : ICollectionFixture<ApiServer>
{
    // NOTE: this class intentionally has no implementation; Xunit will use this signature
    // to create and manage all tests that use the fixture as a single collection
}

[Collection(nameof(ApiTestCollection))]
public abstract class ApiTest
{
    private readonly string name;

    protected ApiTest(ApiServer server, string name)
    {
        Server = server;
        this.name = name;
    }

    protected ApiServer Server { get; }

    protected HttpClient Client => Server.CreateClient(this.name);
}

// adding API tests are now easy to light up
public class ValuesV1Test : ApiTest
{
    [Fact]
    public Task get_should_return_ok()
    {
        // arrange
        const string url = "values";

        // act
        var response = await Client.GetAsync(url);

        // assert
        response.StatusCode.Should().Be(HttpStatusCode.OK);
    }

    public ValuesTest(ApiServer server) : base(server, "Values-v1") { }
}

There is potentially a Chicken & Egg dilemma here if you try to use the API Explorer metadata to configure the IHttpClientFactory because they both require DI configurations, but you don't have the metadata (yet) to configure the IHttpClientFactory. A way around that would be to create separate IServiceProvider to get the metadata and then configure IHttpClientFactory (e.g. services.AddHttpClient("Values-V1")).  This can be achieved with services.CreateProvider() after the minimum set of required services are registered in the test suite. It's gonna hurt the test start up time because it will create and go through some level of duplicate DI twice, but it's only for testing and you only have to do it once per entire run. I think that's a reasonable compromise if you want this type of automation.
That's not an entire solution, but it should lay the ground work for creating fairly generic integration tests. Spinning up a server, even the in-memory TestServer, has a lot of overhead. You want to reuse it as much as possible. API Versioning uses this approach for its acceptance tests (e.g. the closest thing to integration tests). The time went from ~12 mins to 90 seconds by reusing the same server instance via test fixtures.
Long, overdrawn answer, but hopefully you (and others) may find that useful.

Once again very enlightening... Thank you!
I realize that if I do it the way I described I'll run into problems once my server supports multiple API versions. But I need to get to that point first! :smile:  When I do I'm going to do it the "proper" way as you've described, and expose the version information as an endpoint of its own. The code you supplied is basically a complete solution - I've used it and it works like a charm!
But for anyone like me who just need to get up and running on "v0" and throw down some integration tests, getting the "current api version" is possible using your second bit of code:
var currentVersion = _webApplicationFactory
  .Services
  .GetRequiredService<IApiVersionDescriptionProvider>()
  .ApiVersionDescriptions
  .Single()          // when you add more versions, this forces test to fail - at that point, redesign!
  .GroupName;        // I used format "'v'VVV" so this gives "v0"

var route = $"/api/{currentVersion}/{controllerName}/{actionName}";
var response = await client.GetAsync(route);

Thank you so much for explaining the process. I now have a both an unblocking solution, and a robust solution for when I add the next API version. :+1:


TYPO3 creates files as root on the OS X level

Software nesting: OS X -> Virtualbox -> vagrant-development -> docker -> docker container -> TYPO3
It's a wonder that the docker container can be mounted onto ./code/ on OS X at all. Now when TYPO3/PHP creates files or directories, they are created as root on OS X. To remove them I need to call sudo.
In the ideal world they would be created as my OS X user.

It's an issue of the vagrant box not of the boilerplate.
Hopefully the next vagrant box commits will fix this issue.
But the container is running with UID 1000 (default) so files should not be created as root.. strage

Very strange! I start a process and the process writes files with root privileges? Can't be true.
Can't? Well I opened one door: ENABLING NFS SYNCED FOLDERS for the vagrant startup.
/etc/sudoers:
%sudo ALL=(root) NOPASSWD: VAGRANT_EXPORTS_ADD, VAGRANT_NFSD_CHECK, VAGRANT_NFSD_START, VAGRANT_NFSD_APPLY, VAGRANT_EXPORTS_REMOVE

Something in this approach looks fatal to me. Does it explain this strangeness?

The sudoers entry only allows Vagrant to add/remove entries into /etc/exports for NFS shares.
The Vagrant VM is using a file share with an exported NFS share to read and write files from the host. Normally if root (uid=0) is creating files the original file is created as root (uid=0). Maybe it's fixed with latest version of Vagrant VM.
Please check again.
BTW: php-fpm is running as www-data (uid=1000) as default so all files should have at least uid 1000 if files are created by php.

This is fixed meanwhile.


NetworkTransform reimplemented

uMMORPG test: https://i.gyazo.com/ef438552bc59eff8e399f5c4458b69aa.mp4
works fine in asset store project's local player authority examples too

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fixes #4162.
Fixes #986.
Fixes #946.
Fixes #1356.
Part of #1367.
Fixes #1750.
Fixes #4534.
Fixes #1849.
Fixes #1957.

I was just triaging through old docs bugs and collected a bunch of low hanging fixes in a branch.


Session tracking

Adds session tracking support.
Notes:

Relies on SessionServiceProvider, which defaults to file based but will be overridden by whatever sessionServiceProvider a user specifies if they choose to do so.
Not entirely happy about the lock function, feel there may be some issues there, I don't like relying on while loops, although I couldn't find a native Silex lock solution.

This is now updated to use at least bugsnag-php 3.11, where session tracking was released.

This is on hold for now and will require review and redevelopment in the future

No longer supporting enhancements on Silex due to EOL.


Add brmesolve notebooks

I added notebooks for using the Bloch-Redfield solver with QuTiP 4 and 5. I split up the original notebook for introduction into one notebook explaining the basic usage of brmesolve() using a two-level system. The original brmesolve notebook includes a strong coupling Jaynes-Cumming model and a comparison to mesolve. And there are two notebooks on using time-dependence in brmesolve. One was already there and I only updated it a little bit to make it compliant with the current setup.

Other than asking for a couple of extra comment tweaks, I think this is good to merge now.


Include Python dependencies in installation steps

This PR update the installation steps in README.md to ensure required Python dependencies (i.e. pyyaml) are installed.
Note this is consistent with the installation steps used in CI
https://github.com/numfocus/jobs-board/blob/d38c5b3f60b5e20f08c7edddcd14a4d4959c2f85/.github/workflows/gh-pages.yml#L26-L30

Thanks, James!


confirm glance store disparity between glance hosts before alerting

Best I can deduce, will need 2x glance nodes and a sensu dashboard to try to test this

Hi Neal, what's happening is we get sensu alerts like "Unknown file f3aafa30-ad24-4ab6-8cb4-ab4d995cd785 found in /var/lib/glance/images" for example, on a 3.8+GB image still uploading or syncing on a particular node.  After the upload completes, the alert goes away.  The upload/sync typically completes within 5-10 minutes of the alert firing.
A few things can cause this: 1) The image might exist on the second controller and is just waiting to be auto-synced to the first controller.  This alert will fire until the sync occurs, and will then be resolved on its own.  2) The image might exist remotely and this check is unnecessary.
To fix this, you'll want to modify https://github.com/blueboxgroup/ursula-monitoring/blob/master/sensu/plugins/check-glance-store.py , and then push the fix out to the clusters and possibly restart sensu-client or other sensu services when doing so.

BTW - Occasionally, when an upload is cancelled before it finishes or is otherwise corrupted, we will find a tiny aborted image in /var/lib/glance/images, which will produce the same alert.  We have to manually move the image to something like /root/badglanceimages.  Would rather the on-call person not get woken up at 3AM to take care of this.  Thanks!

Ok it's going to take me some time to get a full stack + sensu dashboard going but I'll let you know asap.

Hi Neal, you can use the bbg-staging-01 cluster.  You can use the production sensu dashboard and just silence the alerts for bbg-staging-01 while you're working on it.
As an alternative - it may be easier - you can also put a sensu suppression window to suppress check-glance-store alerts outside of 8AM-5PM Pacific.

Ok thanks I'll see if anyone is using it

Might be better to use bbg-support-01. staging-01 is getting wiped and rebuilt as a 2.0 (Kilo) cluster.

The existing check-glance-store is analogous to a nagios check, it runs constantly and does real time checks.  It does not track time series like we want for the "eventually correct" new glance monitor.  I'm going to look into the glance-sla monitor we have to see if that tracks time series.  This will run on the sensu server itself and not the agent.
ulyssesbbg, do we currently alert on the sla monitors, or do we want a new server side glance monitor?

Neal, couldn't we change the threshold of how many times this check fails
before it becomes "critical"?

jlk

On Tue, Nov 17, 2015 at 9:48 AM, Neal Brown<EMAIL_ADDRESS>wrote:

The existing check-glance-store is analogous to a nagios check, it runs
constantly and does real time checks. It does not track time series like we
want for the "eventually correct" new glance monitor. I'm going to look
into the glance-sla monitor we have to see if that tracks time series. This
will run on the sensu server itself and not the agent.
ulyssesbbg, do we currently alert on the sla monitors, or do we want a new
server side glance monitor?
—
Reply to this email directly or view it on GitHub
https://github.com/blueboxgroup/ursula-monitoring/issues/43#issuecomment-157450769
.

Doh, apologies for the delay I needlessly set up my own rabbitmq/sensu because I thought I would need to do the time series : (  PR incoming.


Getting only first host that matches filter

doesn't seem to work for me 😕

I'm testing by replacing the last task with a debug one showing which hosts matched:
- hosts: collected_hosts
  name: foo
  tasks:
    - debug: msg="System {{ inventory_hostname }} has uuid {{ ansible_product_uuid }}"


Running spotdl web just searches Spotify for the song named "web"

System OS
Linux
Python Version
3.10 (CPython)
Install Source
pip / PyPi
Install version / commit hash
v3.9.6
Expected Behavior vs Actual Behavior
Expected: Running spotdl web should start the web interface.
Actual: Running spotdl web results in "Searching Spotify for song named "web"..."
Steps to reproduce - Ensure to include actual links!

Run pip install spotdl
Restart Terminal
Run spotdl web

Traceback
Searching Spotify for song named "web"...
Found YouTube URL for "070 Shake - Web" :
https://www.youtube.com/watch?v=6Vp3ucu2GUw

Other details
No response

Duplicate of #1562


Shortcut guide v2 - WinUI3 body

SVG support in D2D is lacking:

we need to use paths to render text
we cannot detect position of the rendered elements

Those make working with SVG a chore, especially placing dynamic elements. We should switch to software rendered content.

i would like better understanding on this.   SVG solves a lot of crispness and sizing issues.

There are a couple of issues with using SVG:
One is that Direct2D SVG support does not include <text> tags. We need to convert those to paths like in this 0.svg. This makes editing the labels hard.
Secondly, while the SVG is static, there are dynamic elements. We blink the key icons when the corresponding key is pressed. This requires that the key element in the SVG has a specific id - not a hard thing to do but one that requires some manual checking.
Secondly, we are scaling and transposing the SVG to fit various screen sizes and orientations, and there is no way to get the final location of the elements on the screen. This leads to some hackish code that finds the correct placement for WinKey + arrows tooltips.
On the other hand, rendering everything is ourselves is not hard, we already have some of the text elements dynamic. It would also allow us to have the list configurable (and also easy to localize). Since that would be a major rewrite, we wanted to combine that with switching to the Composition APIs.

I think we should come up with a game plan for v2 as this stuff should be localized and people want configuration ability here.

Moving away from SVG makes localization possible as for any other part of PowerToys that uses strings from resources.
For the Shortcut Guide localization there are extra steps since different localized versions have different keys, so it's not just a question of localizing the text, it will require to create maps to match the correct keys.

The keys could be stored with their descriptions in the resource file - e.g. [D] Display and hide the desktop would render the D as a key and the rest as a label.

We may consider it, but I'm not sure it's a best way to maintain the resource files, since each locale would have it's own values that are not strictly translations.

xref #890  for look/feel.

Note, this is so stuff is generated via a JSON file and create the UX, not stuff is user configurable.  Configuration is future work

closing this against #890


Add a reference to the Microsoft.XmlSerializer.Generator package in the MyApp project

Following up on my previous post I found the line dotnet add package Microsoft.XmlSerializer.Generator -v 1.0.0 to cause the problem.
According to 7 Note further down the page
7 Note

[dotnet run](https://learn.microsoft.com/en-us/dotnet/core/tools/dotnet-run) calls [dotnet build](https://learn.microsoft.com/en-us/dotnet/core/tools/dotnet-build) to ensure that the build targets have been built, and then calls dotnet <assembly.dll> to run the target application.

dotnet run invokes dotnet build. So I ventured to issue dotnet build directly, which gave me advice to download a missing framework:
To install missing framework, download:
  https://aka.ms/dotnet-core-applaunch?framework=Microsoft.NETCore.App&framework_version=2.0.0&arch=x64&rid=win10-x64

Trying that, I get told, that framework 2 is out of service and deprecated.  Downloading and installing the suggested replacement (v4) leaves me with...
To cut a long story short, I finally used version 7, as that seems to be the current one. Changing the line in question to:
PackageReference Include="Microsoft.XmlSerializer.Generator" Version="7"
Issuing dotnet run the umpteenth time now leaves me with:
C:\Users\MulMic\Documents\Personen\MulMic\Software\MyApp>dotnet run
C:\Users\MulMic\.nuget\packages\microsoft.xmlserializer.generator\7.0.0\build\Microsoft.XmlSerializer.Generator.
targets(52,5): warning : SGEN: Failed to generate the serializer for MyApp.dll. Please follow the instructions at https
://go.microsoft.com/fwlink/?linkid=858594 and try again. [C:\Users\MulMic\Documents\Personen\MulMic\Soft
ware\MyApp\MyApp.csproj]
Hello, World!

Checking the code in Program.cs once again, I pay attention to the top comment and visit
https://learn.microsoft.com/de-de/dotnet/core/tutorials/top-level-templates
While this page tells me that the new and the old template style are equvalent, I copy the old example to Program.cs replacing what I had so far, modifying it again in accordance with the article.
Running dotnet run once more I now get:
C:\Users\MulMic\Documents\Personen\MulMic\Software\MyApp>dotnet run
Hello World!

Which is probably what was to be demonstrated in the first place.
I suggest the article to be subjected to a general overhaul.

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

ID: cb8026b7-96b0-eab2-e6ec-93d9c61e95e6
Version Independent ID: 1ed6e1bf-7228-608d-b3fa-fa517e55386e
Content: Microsoft XML Serializer Generator - .NET
Content Source: docs/core/additional-tools/xml-serializer-generator.md
Product: dotnet-fundamentals
GitHub Login: @HongGit
Microsoft Alias: dotnetcontent

Thank you for the detailed issue @MulMic!


Add duration metrics for each test case

Issue #, if available:
Description of changes: Observing each e2e test case's execution time.
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

H


Menu Question

I have a question about the menu structure. It is possible to ignore the pages "Privacy" and "Terms and Conditions," from the menu structure. I want to put links in the footer to trim down the menu structure, so I need to retain the pages. I am new to GatsbyJS, and am having some trouble figuring out how these menu's are generated. I can tell from src/components/menu/menu.js how four menu's are specified but not the others. Help would be appreciated. Love the starter.

Never mind I figure it out after I sent the message. I changed src/components/menu/menu.jsfrom:
this.items = [
      { to: "/", label: "Home", icon: FaHome },
      { to: "/category/", label: "Categories", icon: FaTag },
      { to: "/search/", label: "Search", icon: FaSearch },
      ... pages,
      { to: "/contact/", label: "Contact", icon: FaEnvelope }
    ];

to:
this.items = [
      { to: "/", label: "Home", icon: FaHome },
      { to: "/category/", label: "Categories", icon: FaTag },
      { to: "/search/", label: "Search", icon: FaSearch },
      { to: "/about/", label: "About", icon: FaSearch },
      { to: "/contact/", label: "Contact", icon: FaEnvelope }
    ];

I hard-coded the About page because I wanted to keep that one, and the Terms and Privacy pages I can use at links on the Footer.


empty topic list

ISSUE TEMPLATE ver. 0.4.0

Which TurtleBot3 platform do you use?

[ ] Burger
[ ] Waffle
[x ] Waffle Pi

Which ROS is working with TurtleBot3?

[ ] ROS 1 Kinetic Kame
[ ] ROS 1 Melodic Morenia
[ ] ROS 1 Noetic Ninjemys
[ ] ROS 2 Dashing Diademata
[ ] ROS 2 Eloquent Elusor
[ ] ROS 2 Foxy Fitzroy
[ x] ROS 2 Humble Hawksbill
[ ] etc (Please specify your ROS Version here)

Which SBC(Single Board Computer) is working on TurtleBot3?

[ ] Intel Joule 570x
[ ] Raspberry Pi 3B+
[ x] Raspberry Pi 4
[ ] etc (Please specify your SBC here)

Which OS you installed on SBC?

[ ] Raspbian distributed by ROBOTIS
[ ] Ubuntu MATE (16.04/18.04/20.04)
[ ] Ubuntu preinstalled server (18.04/20.04)
[ x] Ubuntu 22.04
[ ] etc (Please specify your OS here)

Which OS you installed on Remote PC?

[ ] Ubuntu 16.04 LTS (Xenial Xerus)
[ ] Ubuntu 18.04 LTS (Bionic Beaver)
[ ] Ubuntu 20.04 LTS (Focal Fossa)
[ x] Ubuntu 22.04
[ ] Windows 10
[ ] MAC OS X (Specify version)
[ ] etc (Please specify your OS here)

Specify the software and firmware version(Can be found from Bringup messages)

Software version: [x.x.x]
Firmware version: [x.x.x]

Specify the commands or instructions to reproduce the issue.

ros2 launch turtlebot3_bringup robot.launch.py
ros2 topic list

Copy and Paste the error messages on terminal.

/parameter_events
/rosout

Please describe the issue in detail.

Sounds stupid, but I have no Idea how to end the bringup of the Turtlebot3. After ending it with control-c it says that the process has finished clearly. When I use the command ros2 topic list, I get only parameter_events and rosout, instead of the 13 results I should get. I think this is also the reason why I cant control my Turtlebot with the keyboard by using the teleop command. Am I doing sth wrong with the bringup? I've used the official guide from Robotis.

Hello. @chris9807
I think you are running teleop with the bring up launch turned off.
You need to run teleop in the other terminal with the bring up launch running.
Thanks.

@chris9807 parameter_events and rosout are topics that are always active in the ROS2 runtime environment by default.  This issue will now be closed.


Prevent invalid voting

Previously, an account could call vote() on a dispute that had closed. I added a require block on the vote function to revert the vote() call if the account is voting on a dispute that has closed.

I removed those edits to the Governance contract


Skip post actions/cache if cache exists

Is it possible to skip resaving the cached files if the cache already exists? I used the https://github.com/actions/cache/pull/37 example to save docker images for one of our Github Action. The execution time for the run action step improves from 1 minute to 8 seconds.. But resaving the cache takes 30-50 seconds which slows down the workflow speed more then it improves :-(
Example github action exection time:

GA without caching: +- 1:12m

  Set up job1s
  Run actions/checkout@v1 3s
  Docker login 5s
  Actions checkout 1s
  Run action 1m 2s
  Complete job

GA with caching: +- 1:23m
Current steps and total time

  Run actions/checkout@v1 1s
  Docker login 2s
  Run actions/cache@v1 9s
  Load cached Docker layers 23s
  Actions checkout 1s
  Run action 8s
  Build image 0s # Skipped, because cache already exists
  Post actions/cache @v1 39s
  Complete job```

Hi @pascallosss, the cache action should skip the post action if an exact match is found on the primary key.  If the cache matched on one of the prefixes, we will still save the cache using the primary key, since a prefix match indicates there are likely changes.  Do you see a message like Cache hit occurred on the primary key ${primaryKey}, not saving cache. in the post action?  Thanks!

Saving and loading the docker tarball will likely be slower unless your layers are really expensive to generate.  In your example you have a very trivial docker container and you have added the overhead of exporting the layers to a tar uploading/downloading and importing those layers.
As with all incremental execution strategies you are not always going to see improvement.


Log says I should report this here

[Using Thaumcraft-1.12.2-6.1.BETA22]
Now let's look at what this log says:
[18:10:58] [Server thread/WARN] [FML]: Thaumcraft loaded a new chunk [-2, 15] in dimension 0 (overworld) while populating chunk [-1, 16], causing cascading worldgen lag.
[18:10:58] [Server thread/WARN] [FML]: Please report this to the mod's issue tracker. This log can be disabled in the Forge config.
[18:10:58] [Server thread/WARN] [FML]: Thaumcraft loaded a new chunk [-2, 14] in dimension 0 (overworld) while populating chunk [-2, 15], causing cascading worldgen lag.
[18:10:58] [Server thread/WARN] [FML]: Please report this to the mod's issue tracker. This log can be disabled in the Forge config.
[18:10:58] [Server thread/WARN] [FML]: Thaumcraft loaded a new chunk [-3, 14] in dimension 0 (overworld) while populating chunk [-2, 14], causing cascading worldgen lag.
[18:10:58] [Server thread/WARN] [FML]: Please report this to the mod's issue tracker. This log can be disabled in the Forge config.
[18:10:59] [Server thread/DEBUG] [FML]: no fruit available for biome: thaumcraft:magical_forest with rarity < 36.
[18:10:59] [Server thread/WARN] [FML]: Thaumcraft loaded a new chunk [-2, 13] in dimension 0 (overworld) while populating chunk [-2, 14], causing cascading worldgen lag.
[18:10:59] [Server thread/WARN] [FML]: Please report this to the mod's issue tracker. This log can be disabled in the Forge config.
[18:10:59] [Server thread/DEBUG] [FML]: no fruit available for biome: minecraft:plains with rarity < 91.
[18:10:59] [Server thread/WARN] [FML]: Thaumcraft loaded a new chunk [-3, 13] in dimension 0 (overworld) while populating chunk [-2, 14], causing cascading worldgen lag.
[18:10:59] [Server thread/WARN] [FML]: Please report this to the mod's issue tracker. This log can be disabled in the Forge config.
It seems that there exists lag.

I have the exact same issue and log.
Once I installed thaumcraft on my private 30+ modpack, my fps just went rock bottom. I have 2 fps with thaumcraft enabled and this spams my log. Once I disable thaumcraft, I am back to 60fps (Vsync). I think there is something wrong with worldgen/chunk generation.

You are using a very old version of Thaumcraft.  Your logs says TC Beta 22...the current is Beta 26.
Update your version and try again.

I have the same problem but with beta 26.
Mod list:


JSONDecodeError session request

I get this error sometimes

Hi there, thanks for posting this issue. I have myself noticed that this sometimes occurs. However, the problem seems to be on YouTube's side. In fact, it seems to be happening on almost every YouTube video I have tested recently.
As an example, http://youtube.com/live_chat/get_live_chat?continuation=0ofMyANhGlhDaWtxSndvWVZVTnJWRU5PZFZFeWJVZG1Well0VTNCSWNHRjZaVjluRWd0Q1dFZzBZWHAzVjNKa1VSb1Q2cWpkdVFFTkNndENXRWcwWVhwM1YzSmtVU0FCMAGCAQIIAQ%3D%3D&pbj=1&hidden=false returns the exact same text:
)]}'
{"reload":"now"}

Nevertheless, I am looking into a way to fix it. I might have to use another endpoint.

Hi there, thanks for posting this issue. I have myself noticed that this sometimes occurs. However, the problem seems to be on YouTube's side. In fact, it seems to be happening on almost every YouTube video I have tested recently.
As an example, http://youtube.com/live_chat/get_live_chat?continuation=0ofMyANhGlhDaWtxSndvWVZVTnJWRU5PZFZFeWJVZG1Well0VTNCSWNHRjZaVjluRWd0Q1dFZzBZWHAzVjNKa1VSb1Q2cWpkdVFFTkNndENXRWcwWVhwM1YzSmtVU0FCMAGCAQIIAQ%3D%3D&pbj=1&hidden=false returns the exact same text:
)]}'
{"reload":"now"}

Nevertheless, I am looking into a way to fix it. I might have to use another endpoint.

An update to those waiting:
I have found the problem and I will fix it as soon as possible.
In short, YouTube removed support for an endpoint which this tool (and many others) used to get chat. They removed https://www.youtube.com/live_chat_replay/get_live_chat_replay and are now using https://www.youtube.com/youtubei/v1/live_chat/get_live_chat_replay and https://www.youtube.com/youtubei/v1/live_chat/get_live_chat.

An update to those waiting:
I have found the problem and I will fix it as soon as possible.
In short, YouTube removed support for an endpoint which this tool (and many others) used to get chat. They removed https://www.youtube.com/live_chat_replay/get_live_chat_replay and are now using https://www.youtube.com/youtubei/v1/live_chat/get_live_chat_replay and https://www.youtube.com/youtubei/v1/live_chat/get_live_chat.

Alright - I've been working a lot recently to get an updated/fixed version out for you all to use and test... and I think the package branch is ready for use. I still need to add a lot of documentation, but this will be much more useful than the current version in master (which doesn't work currently due to YouTube API endpoint changes).
This version is much more advanced than the original version. Some of the options are slightly different from version 1, but the help command (chat_replay_downloader -h) should assist you in the meantime while I write better documentation.
You can install it using pip install chat-replay-downloader (see https://pypi.org/project/chat-replay-downloader/ for more info).
Please let me know your thoughts and feel free to give suggestions! You can always post an issue or even create a pull request.

Alright - I've been working a lot recently to get an updated/fixed version out for you all to use and test... and I think the package branch is ready for use. I still need to add a lot of documentation, but this will be much more useful than the current version in master (which doesn't work currently due to YouTube API endpoint changes).
This version is much more advanced than the original version. Some of the options are slightly different from version 1, but the help command (chat_replay_downloader -h) should assist you in the meantime while I write better documentation.
You can install it using pip install chat-replay-downloader (see https://pypi.org/project/chat-replay-downloader/ for more info).
Please let me know your thoughts and feel free to give suggestions! You can always post an issue or even create a pull request.


#DC correct spelling

only some spelling error (at a few locations, I'm not sure if BE or AE should be used)

Thanks, @andife for catching those misspellings.


add code enhancement on class NestedSetBehaviorObjectBuilderModifier

Instead of calculating $objectClassName, $queryClassName and $useScope on every method that uses those variables, it calcultes it at the __construct and setBuilder and use the stored values on various locations.
It still passing all tests on my machine and code GPA is slightly improved. https://codeclimate.com/github/dmelo/Propel2/compare/code-enhancement#ratings

I'm really not in favour of this change due to the fact that is only a cosmetic change with a huge diff. Refactoring all those huge builder is PITA and hard to track when you hunt down some changes that broke something. We're working already on the data-mapper branch to clean this mess up.

I understand the concern about big changes. But bear in mind that those changes make the code 51 lines lighter. It does it by taking away a block of code that was repeated on multiple places. Consider the benefits of DRY. Also, it makes fewer calls to "get.*" methods, by using class attributes instead. So the code tends to be more efficient as well.


Fix for slice out of boundary on compression

Compression might reach a state where compressed data size is actually
bigger that original data. For those cases, we need to send original
data instead.
The fix make sure sensor won't send data bigger than original data.
Receiver will know if it needs to uncompressed via an extra header flag check.

Closed in favor of https://github.com/deepfence/PacketStreamer/pull/18


[BUG] Similar to #313 UI is messed up settings also don't open

Describe the bug
Updated to the latest version and the UI is broken and settings won't open.
Odd thing is the old UI is still showing up and everything is functioning fine serverside it's clientside that is broken.
Things I've done:
Uninstall/reinstall simple calendar.
Delete the simple calendar folder from the server and unpack the .zip manually
stop/restart the server between attempts
Expected behavior
Everything to be hunky dory.
Screenshots

Foundry Information

Foundry Version: 9.296
Game System: DnD5e
Simple Calendar Version: 2.0.30
Simple Calendar Configuration: [optional, can help in debugging custom calendars]

Browser (please complete the following information):

Browser: chrome 104.0.5112.81
Just tried in edge 104.0.1293.54 and old UI shows up and functions fine
Firefox is affected as well

Additional context
The only errors that come up are in the screenshot when I try to open settings from the foundry configure settings menu

This is an odd issue, I can't think of any reason why it would fail to load properly.
It almost seems like a caching issue but you did restart the foundry server and tried different browsers so that should have taken care of any lingering caches.
Have you tried with just Simple Calendar enabled? Just to quickly rule out other modules.
Also have you tried a different world? Maybe the issue is localized to a specific world for some reason?

Figured it out after having slept on it. All of my website traffic is routed through cloudflare so it was trying to use the cached stuff which is why it worked serverside and why visits from a new browser worked but re-visiting was affected.


[Self Host] Mystreamers command crashing bot

/root/M8-Bot/commands/Streaming/myStreamers.js:35
                        var fileCount = files.length
                                              ^

TypeError: Cannot read property 'length' of undefined
    at fs.readdir (/root/M8-Bot/commands/Streaming/myStreamers.js:35:26)
    at FSReqWrap.args [as oncomplete] (fs.js:140:20)

I get this whenever I run the command, and then the bot crashes.

files is undefined. Perhaps you are missing streamers in the streamers folders? The command was meant to be run once streamers are added.

I tried to add a streamer and I got
 Uncaught Promise Error:
[2019-07-08 13:30:43] TypeError: Cannot read property 'includes' of undefined
[2019-07-08 13:30:43]     at fetch.then.then.then.mixerInfo (/root/M8-Bot/commands/Streaming/addMixer.js:185:31)
[2019-07-08 13:30:43]     at process._tickCallback (internal/process/next_tick.js:68:7)


Add partial docker tags

name: Docker meta
        id: meta
        uses: docker/metadata-action@v3
        with:
          images: ghcr.io/peterbourgon/fastly-exporter
          tags: |
            type=ref,event=branch
            type=ref,event=pr
            type=semver,pattern={{version}}
            type=semver,pattern={{major}}.{{minor}}
            type=semver,pattern={{major}}

for 7, 7.0, 7.0.0 tags once we move out of alpha/beta/etc

https://github.com/marketplace/actions/docker-metadata-action#semver
https://github.com/docker/metadata-action#typesemver

I don't totally understand the request, here. Can you say a bit more, or maybe make a strawman PR?

Sure, so right now the tag options for your docker images is as follows
{"name":"peterbourgon/fastly-exporter","tags":["v7.0.0-alpha.1","v7.0.0-alpha.2","v7.0.0-beta.1","latest"]}

To make this clearer, let's consider the arbitrary version v7.1.2.
With these additions, in addition to the specific version tag, fastly-exporter docker releases would ALSO be tagged as follows
# Per type=semver,pattern={{version}}
7.1.2
# Per type=semver,pattern={{major}}.{{minor}}
7.1
# Per type=semver,pattern={{major}}
7

Which allows folks to also pull

ghcr.io/peterbourgon/fastly-exporter:7 if they want the latest version of major release 7.x.x
ghcr.io/peterbourgon/fastly-exporter:7.1 if they want the latest version of minor release 7.1.x
ghcr.io/peterbourgon/fastly-exporter:7.1.2 if they want specific patch version 7.1.2

Sounds good, but I don't understand what "these additions" means :) Is it a new stage in the Action?

aaah, gotcha!  essentially, this slots into https://github.com/peterbourgon/fastly-exporter/blob/main/.github/workflows/ci.yaml#L75,L78 where you're adding the tags: section after images:, in the with: block.
so just adding more bits to the pre-exiting uses: docker/metadata-action@v3 step.
      - uses: docker/metadata-action@v3
        id: meta
        with:
          images: ghcr.io/peterbourgon/fastly-exporter

to
      - uses: docker/metadata-action@v3
        id: meta
        with:
          images: ghcr.io/peterbourgon/fastly-exporter
          tags: |
            type=ref,event=branch
            type=ref,event=pr
            type=semver,pattern={{version}}
            type=semver,pattern={{major}}.{{minor}}
            type=semver,pattern={{major}}


How to use antiforgery in SLB without session keeping?

It seems that the tokens are stored in local machine, how to make it store in Redis or a distributed environment?

Antiforgery or CSRF protection is a mitigation for attacks where a user's credentials are persistent on the local machine, such as in a cookie. Nothing on the server side today stores the antiforgery tokens, they are encrypted data that can be read later by the server.
What is it that you're really trying to do?

@alexsandro-xpt the tokens will definitely work across machines in a cluster if the machines are configured correctly for a web farm. Here's a doc on this topic: https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/web-farm?view=aspnetcore-2.1
In particular, the "Data Protection" section is what is relevant to anti forgery tokens.


MLHR-1961 enhanced existing csv parser to take in schema for validations

enhanced existing csv parser to take in schema for validations
@gauravgopi123 could you please review.

incorporated review comments

Incorporated review comments

Hi, Can we merge this if there aren't any more review comments ?

Hello, Can we merge this if there aren't any more review comments ?

I don't see the test case for testing autometric counts being reset every begin window

Earlier had added test for just auto-metric verification. Added another test case for testing autometric counts being reset every begin window

Comments related to test case have been incorporated.

Updated tests for autometric verication and squashed commits

Please squash commits and rebase it..

Done.

Closing the PR as it is merged.


GURPS 2.9.5

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

My Modifier on Chat Window Skill Rolls. A new row was added to the chat window to show what number you entered in the pop-up dialog box.
New Sheet Option. Display Roll Modifier Summary in Chat Window. If checked (by default), the roll chat window will show a summary of modifiers, if any. This option is available on the campaign level and character sheet level.
Chat Window Roll Results. Added some formatting, including highlighting the Malfunction warning.
Grimoire tab. Added collapsible notes area.
Traits notes, removed some unnecessary text from default critical fail/success notes.
Repeating Melee and Ranged Tables. Increased the width of the Reference field and updated tooltip.
Hit Location & Armor Table. Added a new column to show the final to hit penalty, it will total the modifier for the hit location and the Size Modifier. This should avoid weird syncing issues when changing hit location and size modifiers. Based on feedback provided by underjack.
Note: When the sheet is updated, it will attempt to remove the size modifier from the hit location modifier. However, this may not work, you may have to manually adjust the hit location modifier.
Added new Help Section. This is the first iteration and will be expanded and revised over time.

Character Sheet Info Roll20 Internal Use only.


Step-by-step Blogging Guide

This issue has been raised before (https://github.com/picocms/Pico/issues/204), but I don't feel that it has been resolved.
I've tried to follow the description in the documentation, but as someone with close to zero knowledge about PHP and Twig, it really isn't as simple as it is claimed.
I would suggest that one provides a fully implemented but bare bones blogging example in the documentation. It is much easier to tinker with something that actually works.

Our current blogging example is the most bare bone example I can think of. In fact, step 4 is no longer necessary since Pico 2.0. So, the only remaining step which requires Twig is step 3 - and this step already includes a fully working Twig snippet that just needs to be copied.
Thus I unfortunately can't really understand where any difficulties araise. Since there was some trouble using the "Clean Blog" theme before about this particular step, the only simplification I can think of is to no longer recommend creating a distinct Twig template for blog posts (blog-post.twig). Anyway, I'm not sure whether this is really an improvement; if a user missed this step it would have worked anyways, and users will likely need a distinct Twig template for blog posts later. I've updated the blogging example accordingly (removed step 2 and 4, splitted step 3 in two steps), see cbed65c.
Anyway, if you want to create your own theme, you must learn Twig. There's no way around this, if you want to create something own, you'll have to learn how to do it. If you don't want to learn Twig you can use a third-party theme. There are quite a lot themes with blogging support out there (see http://picocms.org/themes/ and https://github.com/picocms/Pico/wiki/Pico-Themes). They often follow pretty different strategies to deal with blogging, there are many ways to do it.

Thank you for your reply, that did the trick.
I'm trying out the clutter theme by the way, and perhaps my stumbling blocks will show where it might be better explained for a newbie.
First it was step 4 where the left side menu got mangled up, skipping that step fixed that.
Then in step 3 what should be in the blog.twig file to begin with, this turned out to be just the same as in index.twig and blog-post.twig.
And then where to insert the suggested code in blog.twig, and this should go right after this:
<main class="main-content">
    <article class="article">
        ...
        {{ content }}
    </article>

Thank you again, and for the updated documentation. And yes I will definitely find some time to learn Twig!


Lançando versao 1.4.20

Coverage remained the same at 97.721% when pulling 4474ed1082a9051d4bb5e4f39886ae8e0e030472 on dev into 2bc3231677883b5dfd0fc38e25eac3d288e83f85 on master.


Docker v2 separate build and push task issue

We are using AZure DevOps Docker v2 tasks for building and pushing our docker images. We are pushing them to Azure Container registry.
First we used buildAndpush command, but because we need some build arguments (not supported by buildAndpush) we needed to split it.
Here is the excerpt from build yaml:
   $(imageTag) is set above as example `2019-10_dev_63cf6d0`
    ....
    - task: Docker@2
      displayName: DockerBuild
      inputs:
        command: build
        Dockerfile: src/WebApi/Dockerfile.prod
        buildContext: src
        tags: $(imageTag)
        arguments: '--build-arg PAT=$(AzureDevOpsPAT)'
    - task: Docker@2
      displayName: DockerPush
      inputs:
        containerRegistry: AzureContainerRegistryDev
        repository: eservice
        command: push
        tags: $(imageTag)

The build task completes successfully. The problem is when push command is running.
We get:
Starting: DockerPush
==============================================================================
Task         : Docker
Description  : Build or push Docker images, login or logout, or run a Docker command
Version      : 2.157.0
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/build/docker
==============================================================================
/usr/bin/docker push ***/***service:`2019-10_dev_63cf6d0`
The push refers to repository [***/***service]
An image does not exist locally with the tag: ***/***service
##[error]An image does not exist locally with the tag: ***/***service
##[error]The process '/usr/bin/docker' failed with exit code 1
Finishing: DockerPush

It looks like we are missing docker tag command? I checked the log of the buildAndPush task and indeed after it builds the image it tags it and then push it:
Successfully built 0b7e240d0373
Successfully tagged ***/***service:2019-10_dev_7e568d6
/usr/bin/docker push ***/***service:2019-10_dev_7e568d6

I've looked at the source code of the task and it's strange, no evidence of docker tag command. Any ideas?

Ok, I've figure it out. The buildAndPush calls tag in build script, and mine didn't so I changed the docker build task to:
- task: Docker@2
      displayName: DockerBuild
      inputs:
        command: build
        Dockerfile: src/WebApi/Dockerfile.prod
        buildContext: src
        tags: $(imageTag)
        arguments: '--build-arg PAT=$(AzureDevOpsPAT) -t $(containerRegistry)/$(imageRepository):$(imageTag)'

And now I am a happy camper :D

What worked for me was to put the registry connection service in the Build task.

What worked for me was to put the registry connection service in the Build task.

Can you elaborate?

@SwyserDev This:
                - task: Docker@2
                  displayName: Build docker image
                  inputs:
                      command: build
                      repository: $(imageRepository)
                      dockerfile: $(dockerfilePath)
                      containerRegistry: $(dockerRegistryServiceConnection) # <--- add this
                      tags: |
                          $(tag)

What worked for me was to put the registry connection service in the Build task.

Make sure you also have the "repository" specified. (I.e. all 3 are needed: repo., reg. & tag)

@chinloyal @morfj @josmare
I'm fairly new to docker and I'd like to understand why the build command would require to know the container registry?
I'd like to build the image once and then push it to three separate registries, but if you are all suggesting I need to specify the registry during the build command then it seems there is some dependency to the registry during the build and therefore I would need to build the image three times, one for each registry that I intend to push to. That just seems really odd to be a requirement. Can anyone familiar with this confirm that the container registry is required for the build command and if so provide an explanation as to why? Or is this just a quirk of the Azure pipeline Docker task to always provide a registry connection but it doesn't actually do anything with it when the build command is used?
Is it possible to run the build command once followed by three push commands? Can anyone share how I may accomplish that?


Exceeds token limit

So, I tried the tool on https://github.com/illacloud/illa-builder and the result is: "File tree and README combined exceeds token limit (50,000). Current size: 78518 tokens"

I am currently hosting the site for free so I'm trying to reduce costs, sorry! I have set a max limit of 50k tokens for the input, so if the repository is too large, it will reject. If you want the diagram, you can run it locally! The steps are in the README
I also plan on letting people bypass this by inputting their own API key (never stored of course)

Great, thank you :-) Maybe you could just add this info to the error.

will do.


Question: unsupport type 5120BFloat16FloatFloatFloatFloat

When I set torch_dtype=bfloat16, it occurs it don't support 5120BFloat16FloatFloatFloatFloat. What does this mean?  The readme said it supports Datatype fp16 and bf16 (bf16 requires Ampere, Ada, or Hopper GPUs).
<class 'RuntimeError'> FWD: Unsupported hidden_size or types: 5120BFloat16FloatFloatFloatFloat
> /home/xx/miniconda3/envs/lla/lib/python3.9/site-packages/flash_attn/ops/layer_norm.py(33)_dropout_add_layer_norm_forward()
-> zmat, xmat, dmask, mu, rsigma = dropout_layer_norm.dropout_add_ln_fwd(

bf16 input should work. The error message means that if weight_dtype=bf16, input_dtype=fp32, residual_dtype=fp32, output_dtype=fp32, then it's not supported.
So you'd need to cast input to bf16 if you want to use weight_dtype=bf16.
Personally I keep weights in fp32 and inputs in bf16.


Layout inconsistency in if/else expressions

Once upon a time I wrote
check :=
   if 2+2 is 7 then 'yes'
   else 'no'

and got 'no' and all was happy. Then I realized I had more to check, and changed it to
checks :=
   for j of [0..7]
      if 2+2 is j then 'yes'
      else 'no'

and Civet generated code that is a JavaScript syntax error, namely
const checks =
   (()=>{const results=[];for (let i = 0; i <= 5; ++i) {
      const j = i;
      if (2+2 === j) results.push('yes');;
      else results.push('no')
   }return results})()

(that double semicolon seems to be the killer; if I paste into a JavaScript playground and remove one, all seems well).
Removing then and putting 'yes' on its own line restores the functionality, but still there seems to be an inconsistency here.

Thanks for the report, definitely a bug!


4.1.0

@cjbrooks12 Any way to support you or buy you a coffee? Been using it in all my projects.

@cjbrooks12 Any way to support you or buy you a coffee? Been using it in all my projects. Would be nice if I could sponsor here on Github.

It's so encouraging to hear that you find this project so useful, and I'm truly humbled that you would like to support me and my development work on it. Thank you, your generosity means so much to me!
I've been trying to get set up with GitHub Sponsors but have had some technical difficulties, though I do have a page on OpenCollective that I initially started for one of my older projects, Orchid. I'm in the process of updating the Collective to reflect all of my OSS projects, including Ballast, so you can donate there.
https://opencollective.com/orchidssg