pcm-power doesn't support Skylake

When I run pcm-power on my Skylake CPU I get:
 Power Monitoring Utility
Number of physical cores: 4
Number of logical cores: 4
Number of online logical cores: 4
Threads (logical cores) per physical core: 1
Num sockets: 1
Physical cores per socket: 4
Core PMU (perfmon) version: 4
Number of core PMU generic (programmable) counters: 8
Width of generic (programmable) counters: 48 bits
Number of core PMU fixed counters: 3
Width of fixed counters: 48 bits
Nominal core frequency:<PHONE_NUMBER> Hz
Package thermal spec power: 45 Watt; Package minimum power: 0 Watt; Package maximum power: 0 Watt;
Unsupported processor model (94).
Cleaning up

pcm-power requires hardware performance monitoring units that exist only on mid-range and high-end server CPUs (i.e. Xeon E5, Xeon E7, Xeon SP). Your Skylake processor model does not have them.


Document how to trigger client events in README.md

This PR resolves #114

Adds documentation to the README detailing how to trigger client events.

Codecov Report

Merging #319 (00cf245) into master (c384473) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #319   +/-   ##
=======================================
  Coverage   89.91%   89.91%
=======================================
  Files          38       38
  Lines        3499     3499
=======================================
  Hits         3146     3146
  Misses        353      353

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 c384473...00cf245. Read the comment docs.


Fix fix

Codecov Report

Merging #181 into develop will decrease coverage by 2.94%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #181      +/-   ##
===========================================
- Coverage    41.77%   38.82%   -2.95%
===========================================
  Files           11       11
  Lines           79       85       +6
  Branches         7        8       +1
===========================================
  Hits            33       33
- Misses          45       50       +5
- Partials         1        2       +1

Impacted Files
Coverage Δ

src/models/TranslationKey/index.js
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fix Label Studio image annotation example

updated README instructions removing zenml example... CLI commands
updated README instructions to make the example useable / workable again
updated example + testing on AWS, Azure, GCP clouds to ensure it works
allowed a wider range for our LS integration

Note that the pipeline still uses old pipeline syntax, but functionally it's working again. Pending other decisions around what happens to our examples we can still convert the syntax at some later point.
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.
[x] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.

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)
[ ] Other (add details above)

@fa9r I just ported the changes over directly into this PR from #1628 since I've changed some of the behaviour in this PR etc.


Manage IPsec tunnels

Implement IPsec configuration via opn-cli from OPNsense API: https://docs.opnsense.org/development/api/core/ipsec.html

IPsec tunnel management edit uses the legacy code. So only list and show are implemented here: https://github.com/andeman/opn-cli/pull/22
The OPNsense Team ist working on the migration AFAIK.

Thanks for your efforts!
Do you know if there is a way for the OPNsense team to prioritize this
migration? maybe by sponsoring them?
On Thu, Nov 10, 2022 at 05:04 Andreas Stürz @.***>
wrote:

IPsec tunnel management edit uses the legacy code. So only list and show
are implemented here: #22 https://github.com/andeman/opn-cli/pull/22
The OPNsense Team ist working on the migration AFAIK.
—
Reply to this email directly, view it on GitHub
https://github.com/andeman/opn-cli/issues/21#issuecomment-1310115439,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGE4RBZISEF2VLGEWU2GEV3WHTJCRANCNFSM6AAAAAARWK4EAA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Juan Carrillo

Not exactly. I´ll disuss this and the other features with the OPNsense devs in the next days.
Keep you updated.


Learn The Difference Between str and repr

The entire library is built with str in mind, but I find that maybe repr should also be implemented, if for no other reason than to mirror str. Otherwise, the REPL just prints generic garbage, which I don't love.

[x] https://github.com/TheRenegadeCoder/SnakeMD/pull/143
[x] https://github.com/TheRenegadeCoder/SnakeMD/pull/144
[x] https://github.com/TheRenegadeCoder/SnakeMD/pull/145
[x] https://github.com/TheRenegadeCoder/SnakeMD/pull/146

Okay, I definitely want to implement repr along almost the same exact requirements as dataclass output. I like how it looks, and the format is much, much better than <snakemd.elements.Paragraph object at ...>

Going to have to do this in pieces:

[ ] https://github.com/TheRenegadeCoder/SnakeMD/pull/143

Very exciting feature that is fully implemented now. Now, a doc can be printed in its repr form and be completed reconstructed. I think that's pretty cool!


Search results not saving

Search results not saving. See: search by keyword > click on card > update item > save > back to list > (shows all results here, but would like to see previous search...should remember state)

fixed

fixed


Added query fee effective cut

Looks good. Do we have this syncing somewhere?

@davekaj Yes, https://thegraph.com/explorer/subgraph/juanmardefago/dev-subgraph

Amazing, okay I will merge this to mainnet-staging later today when I deploy the subgraph to the correct endpoint

has been merged into mainnet-staging


chore(remix-dev): always minify

some folks are running into issues in dev that are solved by minification.
also esbuild is faster when minifying
Signed-off-by: Logan McAnsh<EMAIL_ADDRESS>
Closes: #

[ ] Docs
[ ] Tests

Testing Strategy:

Won't this make debugging harder in dev mode? 🤔

Won't this make debugging harder in dev mode as we now don't have actual function names, ... anymore? 🤔

If it generates a source map it shouldn't be an issue

If it generates a source map it shouldn't be an issue

That's another option, but it isn't enabled by default if I'm correct.

Let's not do this. There should be no need to minify in development. If there are issues that are solved with minification, we should be able to address those separately.


Sync JVM flags with Trino defaults

As expected, older versions fail to run with:
Unrecognized VM option 'G1NumCollectionsKeepPinned=10000000'

I could add a condition on the image.tag, but when using custom-built images, it doesn't have to be a version number. It's not possible to specify a different app version when installing the chart, so we can't use this field.
Maybe we should consider removing the default JVM flags from the chart and rely on those in the image, unless explicitly set?

I tried preparing another PR that would not override the default jvm.config but it's tricky, because we include values from the .Values.coordinator.jvm section. Handling this would require complex logic and IMO it's not worth it.


Update toolchain-flow.svg to be clearer.

Change the "Open FPGA\nassembler" to instead be "(open source) (FPGA assembler)" (and "(closed) (FPGA assembler)").
Should make it harder to confuse with the "OpenFPGA" project.
Signed-off-by: Tim 'mithro' Ansell<EMAIL_ADDRESS>

Superseeded by #535.


UART0 RTS and CTS pins have been swapped

The UART0 RTS and CTS pins are swapped. On the real board, nrf52840 and VCOM1 on J-Link OB do not operate properly without these changes. The transmission works in only one direction, from nRF52 to J-Link. I guess these signals were originally confused on the schematic.  After the changes, UART and VCOM1 work properly.
Upstream PR: https://github.com/zephyrproject-rtos/zephyr/pull/51802

Clang check required a fix, which was merged recently. Please rebase the PR to get the workflow properly triggered. Sorry for an inconvenience.

Needs rebase

Already fixed in upstream project, closing.


Supported Device:

Roomba Model:
960
Did It Work
No

please re-create this issue for it to update the readme (sorry for any inconvenience)


jrc_fly-fsb-1

there are nuclei, just sneaky and tucked away

after 16 training slices, model is performing very poorly, try manual as there doesn't seem to be too many nuclei  @jonesa3

@yuriyzubov once you create the zarr I'll check out the nuclei for Alyson to see if they're all complete

@avweigel
/avetissiane/tiff_slices/jrc_fly-fsb-1/s4_amira_proofread.zarr

this is gtg


Implement "last processor" column.

Description of the feature, modification, idea or suggestion
We already have the option to enable "ideal processor" in the Threads section when viewing properties for a process, would it be possible to implement the column "last processor" ?

Proposed implementation details (optional)
No response

would it be possible to implement the column "last processor"

I think this information was removed long ago. At least I don't see anything in the symbols? @jxy-s

As far as I'm aware this is not exposed in any standard query capability. At least, not the specific KTHREAD.NextProcessor.
KTHREAD.NextProcessor which updated to the "last processor" the thread was scheduled to. Which will then later become the "next processor" if the "ideal processor" is unavailable the next time the thread gets scheduled. The latest Windows Internals book actually now calls out "three CPU numbers" but I believe that is only a conceptual distinction to convey the intention in following sections of the literature. I do not believe the "last processor" tracking has changed structurally (e.g. KTHREAD.NextProcessor) however there have been additions to the state machine which is choosing the processor for a given thread (mostly in support of ARM).


Move privacy acceptance below payment step

Description
This Task will adjust README file

:tada: This issue has been resolved in version 4.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Rename uffizzi preview to uffizzi compose

Rename the preview command to compose to distinguish it from cluster environments.
Other updates

Update the DESCRIPTION on the help page for uffizzi preview create to "Run $ uffizzi project compose --help for details."
Fix typo "usefull"  in FLAGS description - Should be "useful"

--set-labels=METADATA
Metadata of deployment that contains any information which can
be usefull for filtering deployments.

Fix FLAGS wording for uffizzi preview list help. Should be "Metadata to filter list of deployments."

@zipofar Update the help description to:

Manage Uffizzi compose environments (previews) and view logs

Done in https://github.com/UffizziCloud/uffizzi_controller/issues/92

[ ] command uffizzi compose list help doesn't work
[ ] Fix typo "usefull" in FLAGS description - Should be "useful"
[ ] Fix FLAGS wording for uffizzi preview list help. Should be "Metadata to filter list of deployments."

TESTED - OK


ipfs: consider ipld-prime and ipld-selectors

Another interesting suggestion brought up by @Wondertan is to use ipld-prime.   Particularly, the IPLD selectors look extremely cool and relevant to what we are doing. E.g. they could be used to download a range of leafs that belong to a particular namespace at once (#221).

Note: If it turns out much simpler to use ipld-selectors instead of manually adding logic to traverse the NMT for celestiaorg/celestia-node#5, we should prioritize this issue. Let's keep in mind though that ipld-prime is a relatively young project and it might be better to wait until it matures and stabilizes a bit further.

implementation: https://github.com/ipld/go-ipld-prime
selectors: https://github.com/ipld/specs/blob/master/selectors/selectors.md
selectors package: https://github.com/ipld/go-ipld-prime/tree/master/traversal/selector
blog-post with examples: https://www.memoryandthought.me/go,/ipld/2021/01/23/go-ipld-prime-intro.html

Another reason why we should not rush with Graphsync is that it depends on IPLDv1 and it's currently under major refactoring. I would suggest not to touch this topic for several months and return to this only after devnet.

closing in favour of #614


String generation error in MissingParameter.format_message

When requiring a argument to be present with @click.argument('myargument', required=True), the output from Click is Error: Missing argument "myargument".None.
This is because the output string created with:
            param_type,
            param_hint and ' %s' % param_hint or '',
            msg and '.  ' or '.',
            msg,
        )```

 expects msg to be set to a value but Click does not set this value.

That issue is already fixed in master, https://github.com/mitsuhiko/click/pull/313


Servers are acting up

Everything was working fine the last time i made accounts, but now i cant make a single one using gmail or hotmail. i've tried different names and emails as well as without 2captcha and with 2captcha. Not sure what changed or how to fix it. any insight to the cause would be greatly appreciated. i tried discord help channel but everyone was on a different topic and i was ignored.

I facing this problem too

Pikaptcha is also acting up, from issues reported there, it seems either that the password field has changed and requires symbols now, or that the Gmail trick is fixed?

its both. i proposed a way around this
On Thu, Dec 15, 2016 at 6:09 AM, ivar111<EMAIL_ADDRESS>wrote:

Pikaptcha is also acting up, from issues reported there, it seems either
that the password field has changed and requires symbols now, or that the
Gmail trick is fixed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sebastienvercammen/ptc-acc-gen/issues/93#issuecomment-267302143,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXUgLCRLci5j5qGqUMnBogq3Jc0g_OQXks5rISAEgaJpZM4LNthD
.

"Servers are acting up" refers to a single thing: the Nintendo servers were acting up and returning an invalid response. Only patience (and Nintendo) can fix this.
The other issue was that Nintendo updated their password requirements. I've updated the code to reflect these changes.
The + e-mail trick has also been patched for Gmail (it now has a cooldown), so we now recommend using a different e-mail provider (e.g. hotmail). You can still forward account creation e-mails to Gmail afterwards and use our auto-verifier on Gmail. These edits are also in the updated readme.


Translate the Spanish lessons titles into Spanish

This needs to be done before the syncing of the readme file.

Which files specifically are being referred to?

https://github.com/rcallaby/Learn-Git/pull/650


.egg-info shouldn't be checked in

https://github.com/matrix-org/synapse-spamcheck-badlist/tree/main/synapse_spamcheck_badlist.egg-info are files which are generated during the build/installation. They shouldn't be checked in.

fixed by #14


First release

We should look at whether there is anything left to do before building/publishing the first release of the template

I just pinged bot to update this repo with latest workflows (https://github.com/asyncapi/java-template/pull/5), now update name of the template in package.json from mq-asyncapi-java-template to @asyncapi/java-template. Your PR that updates it should have a feat prefix, and release will be triggered automatically after merge, if you added all required semantic-release dependencies -> checkout template for generator templates project and it talks about semantic-release package and related plugins. You do not have them in the package.json but I see you have release property with release configuration.
I also created https://github.com/asyncapi/java-template/releases/tag/v0.0.1 for you already. Otherwise this template would land on NPM directly as 1.0.0...unless this is your plan 😅  so then just add ! to PR title

Release published!


migrate convert-to-signless pass from mhlo

as title, close https://github.com/openxla/stablehlo/issues/2356

I've sent a request to add you to the OpenXLA GH org so that future PRs don't require CI approval. If there are other names I should add please let me know!


Fadg - Forward AD native MFEM class with unit tests and example

This pull request consists of a simple automatic differentiation template class and several unit tests. It is the first of several pull-requests for adding new features to accommodate AD-related implementations in MFEM. The class is templated in fdual.hpp and the unit tests are implemented in tests/unit/linalg/test_fdual.cpp. The implementation is based on dual numbers.
Two examples - one serial ex71 and one parallel ex71p demonstrate the utilization of AD.
The native AD implementation can be replaced with external ones. In this case the user has to compile either ADEPT (compile it with flag --dissable-openmp) or just download FADBAD++.

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#1612
@bslazarov
@tzanio
@v-dobrev + @jandrej + @jamiebramwell
07/25/20
⌛due 03/19/21
⌛due 03/26/21

General point of view after looking at the PR for the first time.
We had an internal discussion to limit the use of templated methods. I understand the need for templated methods in this context, but this will put AD enabled builds back to square one. Just an observation.
The patch set duplicates most of the vector and matrix capabilities to introduce templated versions suitable for AD. I discourage the duplication of general functionality. This will frustrate users who use or want to try AD and want to use a feature that is available in a regular DenseMatrix but not in the templated version. Same goes for the vector and templated vector. It will also introduce a maintanance burden to keep up on both versions.

The idea is to use the same operations utilized in a standard MFEM dense vector/matrix implementation. If we change the template classes, i.e., we do not duplicate the functionality,  a user will have to implement one version for development/test proposes with AD, and later another one for production runs. In all cases, we will need a templated vector and matrix.  We can utilize an external library like Eigen - in this case; we will have one additional dependency, which is avoided with the current solution.

This work will be refactored in a new PR


New version: AdGuard.dnsproxy version 0.62.0

Pull request has been created with Komac v1.11.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-Executable-Error


Does not work in group chats?

Hi,
at first, thank you for sharing your node.js examples of an telegram bot. Receiving messages in normal chats is working and very cool. But what about when adding the bot to a group chat? I checked the /setprivacy (enabled or disabled) and tried to add some commands via /setcommands. But sending messages in the group the bot was added, I do not receive any "message" events. Do you have a hint?
regards
Sebastian

I have a bot named GaussBot have been running fluently for more than 12 hours since this module created and it works quite well in both normal chats and group chats regardless of the privacy mode.
Maybe you can try navigate to https://api.telegram.org/botTOKEN/getUpdates (replace TOKEN with your bot token) in the browser to see if the Telegram server returns correct results.

Oh. Thanks for the hint! Navigating to the url and seeing the response should solve my problem. The data is there, therefore I can search my problem.
Very helpful small project! Thanks a lot.


[Issue]: extract tools means tool calling？

Describe the issue
Assuming that I customize the tool function, must my model have the ability to tool calling? If my model does not have the tool_calling capability, how should I embed the tool into my custom agent?
Steps to reproduce
No response
Screenshots and logs
No response
Additional Information
No response

Currently for tool calling it requires LLM support. But you can use code generation capability of LLM. You would need to customize the system_message to include the definition of the function.

Thank you very much for your answer.
But my tools may contain interfaces for calling other microservices. I have an immature idea: refer to
‘‘‘self.register_reply([Agent, None], MathUserProxyAgent._generate_math_reply, position=2)’’’in  'MathUserProxyAgent'.
I define my own 'ToolSUserProxyAgent' and my own '_generate_tool_reply', and implement the method of calling other microservices in '_generate_tool_reply'.
I'm not sure if this is possible?Could you help me?

If the LLM at least support generating structured response i.e., a JSON object corresponding to the remote API service call, then you should be able to prompt it to generate that given a natural language task. This is for the Assistant Agent.
For the UserProxyAgent, you can register a new reply function like what you mentioned, and, in that function, you parse the JSON object and make the actual call to the remote API service, and then pass the response back to the Assistant Agent.
Then you call user_proxy_agent.initiate_chat(assistant_agent, "<your task description>") to start the conversation.
The hardest part in my opinion is the getting the system prompt right for the assistant agent since it is not an OpenAI model.  Depending on the model maybe you also need to give some examples as in-context learning.


compose: Fixes #20639 by preventing upload files tooltip from hiding …

Prevent upload files tooltip from hiding behind the left sidebar.
https://github.com/zulip/zulip/issues/20639
All tests ran successfully.

@timabbott Can you review this please ?

@alya Can you have a look at this ?

This looks okay to me, but it's probably worth asking if @amanagr thinks the same?

LGTM, there is a full stop missing in your commit message.

Commit name changed. @amanagr

Oh, right, could you change the commit message to be:
compose: Prevent upload files tooltip from hiding behind left sidebar.

Resolves the issue by appending tooltip to document body.

Fixes: #20639.

because we usually want that "Fixes: xyz" to be on its own line at the end of the commit.

All set.

Screenshot looks good to me!

@timabbott Can u merge this please ?

This looks good, merged, thanks @My-Name-Is-Nabil!


Bugfix/status plots

Metadata boxplots excluded for status (#59 addressed)
Relocate legegend
Change filenames, titles and partly the ylabel for statusplots

Normally a barplot (barplot_status.png) and a mapplot status (overview_status.png) is now produced. When triple collocation is chosen then a second barplot (barplot_status_tc.png) and a second mapplot (overview_status_tc.png) is produced with a title that indicates that triple collocation is included (see plots above). Therefore, there are a maximum of 4 status plots. If more status array exist then status arrays processed later just overwrite the earlier versions (which is fine because we said that all "2d - error arrays" are alike and all "triple collocation error arrays" are also the same.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 18 (61.11%)  changed or added relevant lines in 2 files are covered.
10 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 80.519%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/qa4sm_reader/plotter.py
8
15
53.33%

Files with Coverage Reduction
New Missed Lines
%

src/qa4sm_reader/plotter.py
10
81.27%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
1648

Relevant Lines:
1946

💛  - Coveralls

Upon talking to Wolfi, the output was changed, either overview_status.png or overview_status_tc.png (barplot_status.png or barplot_status_tc.png) are produced. Hence there should always be exactly 2 status plots in the results.

@pstradio could you review the changes?

Hi @daberer , in general we should avoid hardcoding or intricated forloops. but I know it is hard and if you think this is the best way then OK for me, as long as it's functional.
Please check my comments and afterwards you can merge it yourself.
There are some changes in the file names, I guess this means something needs to change in the qa4sm fron-end too? In that case we should make a small release and a PR in qa4sm too - better coordinate with monika

yes, addressed the changes and its also coordinated with Monika.


feat: Snos job

👷♂ In progress
Resolves: #11
Added

Implementation of the SnosJob.

Updated

toolchain to nightly so it matches the one used in the snos repository. It caused a lot of lint updates, hence all the files updated in this PR,
some package versions so they're compatible with snos.

Thanks @akhercha for this work! I notice there are a lot of formatting reviews but the CI still passes implying that our checks aren't strict enough in the CI. Do you've any specific recommendations?

A lot of those changes were needed since I updated the toolchain to nightly - some rules must have changed.
Also it must be that I like to order my imports as:
[Imports from the Rust standard library]
- 1 line
[Imports from installed packages]
- 1 line
[Imports from other sub crates]
- 1 line
[Imports from this crate]

I may have updated some imports in the code while I was working 🥷
It really doesn't matter, I just find it clearer since you can find stuff faster.

@apoorvsadana The tests aren't running currently in the CI. Also, a lot of them (from crates/orchestrator) are failing locally for me with the same error:
thread 'tests::jobs::verify_job_with_verified_status_works' panicked at crates/orchestrator/src/database/mongodb/mod.rs:39:76:
Failed to ping MongoDB deployment: Error { kind: ServerSelection { message: "Server selection timeout: No available servers. Topology: { Type: Unknown, Servers: [ { Address: localhost:27017, Type: Unknown, Error: Kind: I/O error: Connection refused (os error 111), labels: {} } ] }" }, labels: {}, wire_version: None, source: None }

Closed in favor of https://github.com/madara-alliance/madara-orchestrator/pull/122


is adaptive loss suitable for 3d shape reconstruction?

Hi, I'm wondering if the adaptive loss is suitable for 3d shape reconstruction, especially for implicit representation, e.g, predicting inside/outside values? Thanks

Hard to say, if it's a regression task where you don't know the noise model a-priori, it could be helpful. If you have a noise model then this probably isn't a good idea.


(MODULES-2175) Create vendored symlink with Ruby

Symlinks created the first time during a PowerShell operation don't
always allow the symlink to be used immediately by PowerShell DSC
during the same run.
Prefer instead to create the symlink natively in Ruby. This has
the advantage that PowerShell modules brought in by the symlink
should be ready to be used by the time DSC runs.
Prefer to create the link only once. If the symlink already
exists but points to the wrong location, or if a file/folder
exists where the symlink should be, ensure that the symlink is
recreated to the proper location.
When determining the path for the symlink, check the registry
without redirection to determine where the Program Files
directory is located. C:\Program Files\WindowsPowerShell\Modules
is the preferred location for Machine-level PowerShell Modules and
the location where other tools, e.g. PowerShell-Get, also put
modules during installation.

Everything passes for specs. I'm curious to know if there are actual issues in a run though. Going to test this.

@Iristyle Okay, this will pass specs, but it is still not good. Will not override an existing folder with the same name.

Still a WIP, but you can keep an eye on proposed changes at https://github.com/ferventcoder/puppetlabs-dsc/compare/ticket/master/MODULES-2175_symlinks...Iristyle:pr/49?expand=1

Ok @ferventcoder , finished moving all the specs over along with the vendors_dsc? feature - see latest at https://github.com/ferventcoder/puppetlabs-dsc/compare/ticket/master/MODULES-2175_symlinks...Iristyle:pr/49?expand=1
Interestingly, I made a minor refactor in the final commit - and it now causes an intermittent test failure.  I haven't had the time to track that down yet though - but if you have any ideas, that would be good.

@Iristyle specs are running, but everything is squashed down and ready.

AppVeyor died on S3 issue. Rekicking.


Using throttle method to improve performance

I use handsfreejs to train my custom CNN/RNN classifier on top of mediapipe models in the browser.
I would eventually like to run it on my Raspberry Pi 4 so I played around with throttle() method for performance improvement.
I use it to reduce the calls to model like so:
handsfree.model.hands.getData = handsfree.throttle(handsfree.model.hands.getData, 1000 / targetFPS)

but this doesn't reflect the FPS property so onFrame (data) is still getting called every frame but it just passes frame duplicates until model is run next time.
This feature would be proper if throttling would also limit respectively the number of the sampled frames, so that data that comes out of 'onFrame()' callback is always reflected correctly. Otherwise, this throttling method is only affecting the calls to the model, but maybe it should also handle frame duplicates in that case.

Hi thanks for starting this issue, this is an interesting case! Throttling models individually isn't easily possible right now (and to be honest I'm not sure what the best way to approach this would be yet) but you can throttle the actual loop. Here's what that would look like:
handsfree.loop = handsfree.throttle(handsfree.loop, 1000 / targetFPS)

Also this is a minor point, but I just learned that I got the documentation for throttle wrong. handsfree.throttle is now just an alias to lodash.throttle and can receive a few extra arguments: https://lodash.com/docs/4.17.15#throttle
Tasks

[ ] Fix handsfree.throttle documentation

'handsfree.loop = handsfree.throttle(handsfree.loop, 1000 / targetFPS)'
Throttling the main loop did the trick, There are no duplicates and sampling can be set via targetFPS safely.
Thanks @MIDIBlocks


🛠 [TASK] : Implement UI to setup campaign details

Summary
Implement UI to setup campaign details
Description
Child of #1002
As an admin user, I need to enter campaign details, so that my community can learn what my campaign is about.
GIVEN an admin user logged in with API key
WHEN create a new campaign OR select existing campaign
THEN allow admin user to edit preset campaign detail questions
Acceptance Criteria

If user selects "Setup Campaign Details" in left nav menu, then view jumps to component and selects it
Editable fields include campaign title and description
If user clicks "edit", put component into edit mode
If user clicks "cancel" while in edit mode, then go to view mode without saving changes
If user clicks "save" while in edit mode, then save changes to backend and return to view mode (refresh data?)

Integration

GIVEN admin key GET brand
GIVEN brand GET campaigns
GIVEN admin key UPDATE campaign

Later:

File upload for campaign featured image

Designs

Screens → Setup Campaign Details
Text sections component details & behaviour
Addtl context in design ticket; #1100

@nielskijf update figma file. edit view of component shows JSON input instead of simple text edit


Added password strength validation on signup page

Fixes #25

📋 Description
Turns into different colours depending on the password strength
🔨 Changes Made

The input changes to different color based on the password strength
weak password returns red
medium gives orange
strong password gives green
solves #25

🔨 Screen Shots OR Short videos of working

✅ Checklist
Before submitting the PR, please make sure you have completed the following:

[x] My code follows the style guidelines of this project.
[x] Read Contributing.md

🏷 Types of Changes
What type of changes does your code introduce? (Check all that apply)

[ ] Bug fix (non-breaking change which fixes an issue) 🐛
[x] New feature (non-breaking change which adds functionality) ✨
[x] UI enhancement (non-breaking change which enhances UI) 🎨
[ ] Documentation update 📚

🤝 Related Issues
Reminder:
Please ensure your pull request is made to the test branch, not main.
Thank you for contributing to Talk-Trove! We look forward to reviewing your PR and getting it merged!

@kartikayasija Please review and let me know if any changes required

Add a video plz


Projects slow to start when external DAC is plugged in

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Windows 10, Godot v2.1.4
Issue description:
When running a project via the play button / hitting F5 / etc, it takes between 10 and 20 seconds to start up. (The game window appears, then remains as an uninteractable white box for some time.) This seems universal, and not related to the specific project I'm trying to play. Once they do start, there don't seem to be any other problems, aside from occasional Audio Underflow errors.
But here's the wild part: when my external (USB) digital audio converter is disconnected, projects start up almost instantly!
It's difficult to say how to reproduce this precisely, since there's a lot of possible factors. I'm using the RtAudio-DirectSound driver. As for hardware, in no particular order:

Creative SoundCore3D
ODAC-revB USB DAC
GA-Z170X-Gaming 7 motherboard
GTX 1070

I'll try to provide more information if you can think of anything relevant!

CC @marcelofg55 for potential clues about the audio drivers, and @Faless @Geequlim in case this would be related to the debugger somehow.
If you are able to build from source, it would be interesting to see how it behaves in the current 2.1 branch, since it uses a new WASAPI audio driver instead of RtAudio, and has fixes regarding connecting/disconnecting devices: https://github.com/godotengine/godot/commits/2.1

Might be related to OS audio settings (surround, sample rate, and bit depth).

I ran into this problem as well (with the same DAC). It's not related to the audio drivers, but rather the input device setup (specifically the DirectInput EnumDevices call).
Full details are available at this stackoverflow answer I found when searching for a solution: https://stackoverflow.com/a/40449680
Essentially the (Massdrop) ODAC seems to be misbehaving and adds two devices under Human Interface Devices: USB Input Device and HID-compliant consumer control device. Disabling the USB Input Device fixes the problem, without impacting the regular function of the DAC. Check the Bus reported device description property of the device to get the right one.
I imagine that this USB Input Device is malformed, and so the EnumDevices function gets stuck iterating over it and waits for a timeout or something.
(As a bonus, this same bug also cropped up in Dark Souls, and prevented the game from ever reaching the main menu if the ODAC was enabled).

@opcon thanks a lot for the info. Sadly, this means there's nothing we can do to solve it.

Is this still reproducible in Godot 3.0.6 and/or 3.1-alpha?

I won't be in a position to test this until the end of the year sorry

Closing for now. If anyone can reproduce it in the current master branch (3.1 builds), please comment here with details and we'll reopen it.


The plugin epub_viewer uses a deprecated version of the Android embedding.

Crash / Error
The plugin epub_viewer uses a deprecated version of the Android embedding.
To avoid unexpected runtime failures, or future build failures, try to see if this
plugin supports the Android V2 embedding. Otherwise, consider removing it since a
future release of Flutter will remove these deprecated APIs.
If you are plugin author, take a look at the docs for migrating the plugin to the V2
embedding: https://flutter.dev/go/android-plugin-migration.
Steps to reproduce:
flutter pub get
Flutter version 2.5.0

any solution

please Try version 0.2.3


Check if XLog already initialized

Hi,
Can you add the possibility to check whether the XLog has already been initialized?
When I run a bunch of Robolectric tests the following exception is being thrown:
java.lang.RuntimeException: java.lang.IllegalStateException: XLog is already initialized, do not initialize again
I am initializing the XLog in my application subclass:
 override fun onCreate() {
        super.onCreate()

        XLog.init(if (BuildConfig.DEBUG) LogLevel.ALL else LogLevel.NONE)
 }

It seems that Robolectric creates the application multiple times but XLog for some reason stays initialized.
What do you think? Maybe the public getter of sIsInitialized would be enough to solve this issue?

@storix Maybe this exception is not necessary to be thrown out, anyway, can you use
static {
  XLog.init(...);
}

instead, to see if it works?
I will fix this issue in next release.

Here is the relevant Robolectric issue - link
There was a similar problem with another library called ARCA. They decided to change the exception to a simple warning.
Currently I'm using a static field isLoggerInitialized to fix this issue like this:
    if (!isLoggerInitialized) {
          XLog.init(if (BuildConfig.DEBUG) LogLevel.ALL else LogLevel.NONE)
          isLoggerInitialized = true
    }


Added union method

#2715  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


If i put wrong email it shows validated email

Is this bug report about an incorrectly validated email address?

[x] Yes
[ ] No

If yes, what is the email address?

[ ] Valid
[x] Invalid

If no, please describe the bug

Additional context

Hi @SabariM27, can you please share the email address that you expect to be invalid?


Update ros interface node

Resolves #199
Also updates some setup files

Now that the interface node is written, we need to test it. We can do this by sending in inputs and checking if the outputs match what we desire.
To run the ros interface node, we can:
roscore in one terminal to setup ROS
rosrun local_pathfinding ros_interface.py to run the interface node in another terminal
rostopic pub /sensors <TAB> to publish to the sensors topic, and see if the interface does the right thing
rostopic echo /GPS to see what the ros_interface node is publishing and see if it matches.
The better long term solution is to write a MOCK_sensors.py that does this process for us:
rosrun local_pathfinding MOCK_sensors.py (make sure to chmod +x MOCK_sensors.py first)

[ ] GPS sensors - use can, ais, or both? which takes priority?
[ ] Coordinates - always DDMM.MMMMMMM? Fix logic if not

#!/usr/bin/env python
import rospy
import json
import time
from utilities import MAUI_LATLON

from std_msgs.msg import Float64
import sailbot_msg.msg as msg
from geopy.distance import distance

# Constants for this class
PUBLISH_PERIOD_SECONDS = 10.0  # Can keep high to simulate real global pathfinding
NEW_GLOBAL_PATH_PERIOD_SECONDS = 3600.0 * 24
AVG_WAYPOINT_DISTANCE_KM = 30  # TODO: Set this to match global pathfinding

# Global variables for tracking boat position
desiredHeading = None

def desiredHeadingCallback(data):
    global desiredHeading
    desiredHeading = data.heading_degrees

def MOCK_sensors():
    global desiredHeading

    rospy.init_node('MOCK_sensors', anonymous=True)
    pub = rospy.Publisher("sensors", msg.Sensors, queue_size=4)
    rospy.Subscrbier('heading_degrees', msg.heading, desiredHeadingCallback)
    r = rospy.Rate(float(1) / PUBLISH_PERIOD_SECONDS)  # Hz

    # Publish periodically
    while not rospy.is_shutdown():
        sensors = msg.Sensors()
        sensors.wind = utils.globaltomeasured(speed, dir)
        pub.publish(sensors)
        r.sleep()
        lat+=1*speed*cos(desiredhheading)
        lon+=1 *speed*cos(desiredhheading)

if __name__ == '__main__':
    MOCK_sensors()

Rough draft of MOCK_sensors.pu


test1--8-main

Please pull this in!

 Quality gate: Failed
26 Issues

 24 Code issues
 2 Anti-patterns

Details

 Quality gate: Failed
26 Issues

 24 Code issues
 2 Anti-patterns

Details


Explore re-introducing the Build tab for character creation.

Just wanna point out that in Roll20's sheet author code of conduct, it says "Community sheets should not include character creation or advancement due to potential copyright restrictions."
If the Mythras sheet is indeed a community sheet and not an official sheet you're maintaining with explicit approval from the publisher, I suggest to look into these details before investing time into coding features that Roll20 might block by being cautious.
If you have approval from publisher or this is actually an official sheet, updating that info for the sheet would help out you and the publisher.


We don't have enough contributors!

 Issue by chaorace
Saturday Mar 18, 2017 at 21:41 GMT
Originally opened as https://github.com/CQUI-Org/cqui/issues/410

Love CQUI and already hang out around here all day anyway? Why not make it official! Check out the recruitment project page for more info

 Comment by gelodgreat
Sunday Mar 19, 2017 at 09:23 GMT

can i join? i'm knowledgeable in sql and xml editting for civ mods.

 Comment by chaorace
Monday Mar 20, 2017 at 14:31 GMT

@angelodgreat I'd be happy to bring you on board, once I've seen more examples of your work. Not just showing experience with SQL/XML, but also wielding the tools git/github offer proficiently. If you'd like to show what you've got, there's a specific SQL issue open right now you could try tackling: #329.
To get you started:

The desired implementation is that you can just copy the existing cqui_settings.sql, rename it to cqui_settingsoverride.sql, and just change the settings there.
When the override file is absent, we'd like to simply ignore it and use defaults
When the file exists, we only want to update the settings defined in default, as opposed to overriding them. This means old override files missing new settings will still use the default settings where it doesn't specify an override

Working theory for implementation:

Attempt loading the override file, which creates the settings table
If the settings table exists now, rename it to something else (this prevents the mostly identical default file from failing to load if we were able to create a table using the override file, since both file attempt to create a table of the same name)
Load the default settings file, which will create another settings table
Merge the two tables, overwriting values in the table the default settings file created where both define a setting

My working knowledge of SQL is only so-so, so it'd be great to have someone who really knows it to take a crack at this implementation

 Comment by chaorace
Monday Mar 20, 2017 at 14:44 GMT

@CQUI-Org/feature-team @CQUI-Org/issue-moderation
Welcome aboard guys! As org members, you have access to the Slack board and the waffle.io issue management tools. I suggest you become acquainted with both using the org project page here.
@CQUI-Org/language-team
It's particularly good to have you guys all in one place! As translators you guys really are appreciated, which I want to show by allowing you more access to the conversations the development team has, which is why I'm especially eager to have you join the Slack channel (see the same project page here for the join link). That way, when a UI can be changed to better accommodate new LOCs, you can just say it openly instead of worrying if it's important enough to create a tracked issue about. Seriously, we're all ears and we want to make things easier for you guys! That also extends to making it easier to translate without necessarily needing to send PRs yourselves... obviously the Google Doc thing flopped a bit and I'm hoping to facilitate discussion for replacing it with something better.

 Comment by KORAD255
Wednesday Mar 29, 2017 at 22:26 GMT

Hey, I would like to join your team. I am a .Net developer from Poland with 3 years of professional experience. I love games, I don't know LUA, however, I think I can manage to learn it relatively quickly. I think your mod is great but too various bugs, unfortunately, unplayable so to speed things up I decided that I will help (I hope).

 Comment by chaorace
Wednesday Mar 29, 2017 at 23:11 GMT

@KORAD255 Anyone can help develop CQUI by creating a fork and submitting PRs to the main project. Only trusted contributors who consistently demonstrate an ability to wield lua and git effectively are invited to the feature team. If you're still interested, I encourage you to tackle some of the bugs you're experiencing and to submit your fixes via PR

 Comment by AlexandreDecan
Thursday Mar 30, 2017 at 13:18 GMT

Maybe you should consider adding some issue labels (eg. "very easy", "easy", "medium", "hard" and "very hard") to illustrate the expected/considered difficulty to solve a specific issue. This way, people who are willing to contribute but are not familiar with CQUI can decide to focus on "(very) easy" issues at first.
This could be combined with a generalized use of priority labels (low/medium/high priority) or with well defined milestones.

 Comment by chaorace
Friday Mar 31, 2017 at 02:18 GMT

@AlexandreDecan I think that's a good idea, I'd be happy to induct you into the issue management team if you'd like to oversee the implementation of a system like this

 Comment by AlexandreDecan
Friday Mar 31, 2017 at 06:36 GMT

As a first step, just add a label "easy" and use it to identify which are the issues that can be easily (hence the name ;-) addressed by external contributors. This could be translation issues, documentation issues or even issues that only requires a very basic knowledge of LUA.

 Comment by chaorace
Saturday Apr 01, 2017 at 03:00 GMT

@AlexandreDecan I suppose it was worth a try! 🦊
@SpaceOgre In light of your recent SQL contribution and several prior additions, would you like to join the @CQUI-Org/feature-team feature team?

 Comment by SpaceOgre
Saturday Apr 01, 2017 at 05:14 GMT

@chaorace Could be fun! But just so you know my contributions will most likely come in irregular bursts, with work and actually playing games taking my time ;) but if that won't dissuade you then I'm happy to join.

 Comment by chaorace
Tuesday May 30, 2017 at 18:02 GMT

Hey @CQUI-Org/feature-team and @CQUI-Org/language-team
We're trying out a discord channel, depending on feedback, we may use this in the future instead of Slack. Click here to tell me what you think: https://discord.gg/YUXRR32


feature: Merge code from presto_drf into allianceutils

Docs for renderer & parsers need to be added to README otherwise looks good


NiflVeil: A minimalistic window minimizer for Hyprland (Request)

NiflVeil lets you temporarily hide windows in a special workspace and restore them via wofi menu or direct commands. Features waybar integration, window state persistence, and works with or without wofi. Perfect for decluttering workspaces while keeping windows easily accessible.

that's not a plugin

that's not a plugin

Ah! I see. I will have to find another way to share it. thank you for taking a look.


feature(github): Issue/PR templates

Suggestion
There could be Issue and Pull Request templates.
Why is this needed?
Makes issues and pull requests follow a standard which can be more informational than just description - IMO makes job of maintainer/s easier.

+1 from me. Cause this will make the Pull Request and issues cuter (I want a cute template) but any works.

+1 from me too.. Will make the devs work easier and they will be able to quickly find the major issues from the less important ones...

I think
ex)
issues
ㄴ bug
ㄴ  question
ㄴ enhancement
ㄴ other
PR
ㄴ bugfix
ㄴ enhancement
ㄴ  fix typo
ㄴ other

Link: https://docs.github.com/en/free-pro-team@latest/github/building-a-strong-community/about-issue-and-pull-request-templates


gyp: inherit CC/CXX for CC/CXX.host

Gyp defaults to gcc/g++ if CC.host/CXX.host is unset. This is not
suitable for environments that only uses the clang toolchain.
Since we already assume that the user will provide clang/clang++
through CC/CXX, lean against it (then drop to gcc/g++).
Refs: https://github.com/nodejs/node/pull/6173
/cc @jbergstroem

LGTM, lets hold off and hope that upstream merges first. I will merge in nodejs on Monday unless anything happens.

Merged as a follow-up to nodejs/node#6173 which was merged a few days ago.


added optional params to identify private channel from multi people chat

@gilbertw1 please review, thanks

Sorry, it took so long to finally review. I've merged all of your changes.
One small change that I made was to remove the ability to specify the optional 'id' parameter in the sendMessage method on the RTM Client. I didn't want to allow users to specify this parameter since the client needs to be able to guarantee uniqueness of the message ids generated on the connection. Instead I opted to return the newly generated message id from the sendMessage call, which can be used to verify the Reply message.
You can see the gist of the change here: https://github.com/gilbertw1/slack-scala-client/blob/master/src/main/scala/slack/rtm/SlackRtmClient.scala#L46
I've bumped the version to '0.1.7' and will be publishing to maven central shortly. Thanks again for contributing!


Update beam_search_decoder.py

#22172
probably not the neatest way to update my previous pull request...

@ebrevdo  Did you get a chance to look into this ?

@georgesterpu  Please sign the CLA in order to proceed looking into this PR.

@ebrevdo  -  I think we need to wait until the user signs a CLA to proceed.

Sure; I guess I thought George had written one.  Thanks!
On Mon, Nov 19, 2018 at 2:43 PM, harshini-gadige<EMAIL_ADDRESS>wrote:

@ebrevdo https://github.com/ebrevdo - I think we need to wait until the
user signs a CLA to proceed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/23460#issuecomment-440068936,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtimysoxIEEa5jk57WvOaiHm95qAwLdks5uwzQYgaJpZM4YMBxK
.

Hello,
I just signed the CLA with my university email, although gmail is listed as the primary one on GitHub, fingers crossed.

Hi. Is the problem still on my side ?

Hi. Is the problem still on my side ?

Thanks for signing CLA. We are looking into the PR.


Pass Auth-Token in the Header Request for the service Instance creation

Is this a BUG REPORT or FEATURE REQUEST?:

/kind bug

What happened:  For Instance creation, Auth Token is passed in the Request body.
What you expected to happen: For Instance creation,  Auth Token is supposed to pass in the Request Header
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Orchestration(release/branch) version:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

Must fix


Add lint rule against .tooltipped and introduce accessibility.yml config

What are you trying to accomplish?
I started this rule in dotcom, but then I realized that PVC might make more sense so this rule can be shared across many projects.
This PR:

Introduce a lint rule against .tooltipped in favor of alternatives, or the more recent tooltip.
Started a accessibility.yml which will allow consumers to easily pull in recommended accessibility linting configurations which we can count against the scorecard. This will specifically contain rules that count towards the accessibility scorecard.
I moved helpers out of base_linter.rb to a helpers module. BaseLinter seems to come with a lot of additional things this rule doesn't need, but I need to use these helpers.

I think consumers can now add the following in their .erb-lint.yml to automatically pull in rules:
inherit_gem:
  primer_view_components:
    - primer/view_components/linters/accessibility.yml

Integration
No
List the issues that this change affects.
Closes https://github.com/github/accessibility/issues/3704
Risk Assessment

[x] Low risk the change is small, highly observable, and easily rolled back.
[ ] Medium risk changes that are isolated, reduced in scope or could impact few users. The change will not impact library availability.
[ ] High risk changes are those that could impact customers and SLOs, low or no test coverage, low observability, or slow to rollback.

What approach did you choose and why?

I introduced a accessibility.yml config so consumers can easily configure recommended rules.
I namespaced the rule as Primer::Accessibility.

Anything you want to highlight for special attention from reviewers?
I namespaced the rule Primer::Accessibility:: to align with what we do in erblint-github.
The pro of this is that it can help to identify where a rule originates from. The con, is that the disable comment is kind of long (e.g. `erblint:disable Primer::Accessibility::TooltipHasMigration).
This also isn't consistent with the other rules living in PVC. What do you think? @primer/rails-reviewers?? 📣
I would also like to add docs for this rule but I'm not sure where it belongs, also given the doc migration work. What do you think? 📣
Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Added/updated previews (Lookbook)
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

This looks good to me 👍 Have you tested that it works without choking in dotcom?

I'll give it a try! And also fix the tests, which are breaking for some reason :(

Have you tested that it works without choking in dotcom?

@camertron I started a test branch using the vendor script and pointing to this commit in dotcom. I updated the .erb-lint.yml config and got this rule running, and tried inserting inline disables. 💪
The one unexpected thing was the path which I had to set to get this config to work: Related commit. I did not expect to need to prefix the path to the config with lib. I didn't need to do that with erblint-github. I wonder why? It's not a blocker though.

Since this was last reviewed, I made these updates @primer/rails-reviewers:

Updated logic to include class as suggested in https://github.com/primer/view_components/pull/2086#discussion_r1237139807
Fixed breaking tests by prefixing :: to some of the Primer helper calls.
Enabled this config for this project https://github.com/primer/view_components/pull/2086/commits/aa180043aa5f789edcaad234c07180e5e0acd29b and added tests to make sure the config pulls in rules as expected.

Your input is appreciated here:

https://github.com/primer/view_components/pull/2086#discussion_r1238663883
https://github.com/primer/view_components/pull/2086#issuecomment-1602853655

Thank you so much 🙏.


Search Pattern

Description
Describe the pattern
Context
Describe scenarios where the pattern is most applicable
Extra information
Add extra info
### Tasks
- [ ] https://github.com/jpmorganchase/salt-ds/issues/2917

Added "needs more info" because this needs verification about its scope

Likely to be shared internally through our business pattern index as part of a stakeholder contribution during Q4.
No work planned in public repo for Q4, at least. Keeping ticket but moving to Q1.
Can see us using this ourselves as part of our site/documentation strategy.
@bhoppers2008

PUI mentioned this yesterday, where the use case is a search bar in app header. We suggested combo box. However, a few users already reached out around not wanting the chevron come with combo box for this use case.


Journald logging omits accurate CODE_FILE, CODE_LINE, and CODE_FUNC fields

Version: -feature
CUPS.org User: twaugh.redhat
In https://cups.org/strfiles.php/3406/0001-Support-for-logging-to-the-systemd-journal-STR-4474.patch I had some code to use the pre-processor to log accurate CODE_FILE, CODE_LINE, and CODE_FUNC fields when using systemd's journald.
More specifically, the changes to scheduler/conf.h are what I mean.
cupsdLogJob() etc are changed to macros.

At this point I can't see implementing this; today we have a bunch of different logging subsystems, all wanting to take full control, whereas with cupsd we want to offer both the traditional file-based logging as well as syslog/journald/asl/os_log.


Circular dependency error with S3 event in Lambda function

I'm getting the circular dependecy error below. I've tried every example possible I've found on the web and always get the same error. Please, advise.

Failed to create the changeset: Waiter ChangeSetCreateComplete failed: Waiter encountered a terminal failure state Status: FAILED. Reason: Circular dependency between resources: [Bucket, ProcessFileFunctionFileUploadedEventPermission, ProcessFileFunction]

AWSTemplateFormatVersion: '2010-09-09'
Transform: AWS::Serverless-2016-10-31
Description: >
  Csv Integration

Parameters:
  OutputBucketName:
    Type: String

Globals:
  Function:
    Handler: app.handler
    Runtime: nodejs8.10
    Timeout: 15

Resources:
  Bucket:
    Type: AWS::S3::Bucket
    Properties:
      BucketName: !Ref OutputBucketName

  ProcessFileFunction:
    Type: AWS::Serverless::Function
    DependsOn: Bucket
    Properties:
      CodeUri: ProcessFile/
      Policies:
        - AWSLambdaExecute
        - AmazonS3ReadOnlyAccess
      Events:
        FileUploadedEvent:
          Type: S3
          Properties:
            Bucket: !Ref Bucket
            Events: s3:ObjectCreated:*

Solution: DO NOT USE DependsOn

I'm having that issue even without DependsOn.

The solution: git your S3 bucket a BucketName.

The general solution is to add a BucketName parameter to your S3 bucket resource.

I've tried that, still getting circular dependency issues, either when setting up Events in the Lambda function or NotificationConfiguration on the S3 bucket.


⚠ Spryr has degraded performance

In db5755d, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 4977 ms

Resolved: Spryr performance has improved in 022d9c3 after 10 minutes.


[Feature][Addon] Add KubeFlow and Volcano addons

Hi guys. As we know Kubeflow is a one-stop solution has many components and its installation is painful. I want to add some ComponentDefinition such as PyTorchJob or TFJob. I think it'll make us run AI workloads more Lightly and flexibly.

Great!

/assign

Hi, we already have KubeFlow addon in model-training. You can use this definition to train your model with different frameworks: https://github.com/kubevela/catalog/blob/master/addons/model-training/definitions/training-comp-def.yaml#L220

Hi, we already have KubeFlow addon in model-training. You can use this definition to train your model with different frameworks: https://github.com/kubevela/catalog/blob/master/addons/model-training/definitions/training-comp-def.yaml#L220

Oh, thanks. That's great. I'll look into it.

You can also checkout our machine learning blog here: https://kubevela.io/blog/2022/03/02/kubevela-with-machine-learning


Clarify docs for visual

I've added some clarifications for how keyword arguments are passed (it was not obvious to me until I read the source)
I've indicated that any @recipe created plot function can be used with visual, since this was also not obvious to me until I read the source / read the response in #321
By the way, thanks for creating this awesome package!

Thanks for the nice words and the PR! Yes, the clarification is very useful, because it is one of the design goals that custom recipes work out of the box with AlgebraOfGraphics.


Added alastair

Alastair is a cooking service written in elixir. It can be used to plan the food for events

Still need to have some time to look into this, probably will have so tomorrow.

I might be stupid, but how can I get to it? It is not in the menu for me :stuck_out_tongue:

Additionally: It is not enabled in oms.sh, and I think it is not being checked to be running by Travis, could you change this?

Okay disregard the not getting to it part for now: The git repo of alastair is empty for me.
I checked out to this branch and did git submodule update, and it succesfully added alastair to the submodule file, however the folder is empty. Additionally when in the folder git status says it is up to date.
Any ideas? How did you do it?

This seems to be the issue:
i3anaan@i3anaan-Legion:~/projects/oms-docker/alastair$ git remote -v
origin	https://github.com/AEGEE/oms-docker.git (fetch)
origin	https://github.com/AEGEE/oms-docker.git (push)

Okay I need to look into this better, might have time for that later today. Might go for a full folder reset. Maybe someone else can see if they run into the same issue? @serge1peshcoff ?

I just tried a git clone --recursive --branch add-alastair https://github.com/AEGEE/oms-docker.git and the folder wasn't empty...

maybe try this one? https://stackoverflow.com/questions/1030169/easy-way-to-pull-latest-of-all-git-submodules

Ok @i3anaan I added it to tests and auto-activated it, so I think it should be mergeable now. However the frontend of alastair partially depends on https://github.com/AEGEE/oms-neo-core/pull/61, maybe we should merge that one first?

Got it working on a clean install, that is a good sign. (I'd still rather have a solution to prevent having to clean-install, I will look into that later)
Before merging, however, I would like to see the logging level changed. It is currently outputting debug level messages, which I find unnecessary. I tried changing this myself, but that didn't seem to work. Since I do not know elixir, I'd like to ask you to fix the log level.

Found the command to fix the empty-folder issue when switching to Alastair on an existing installation:
git submodule update --init

(Called in oms-docker folder)
So that issue is fixed, remaining is the log level.


Disable clusteroperator for package server

Disable clusteroperator for package server while we fix https://bugzilla.redhat.com/show_bug.cgi?id=1723818.

This is likely to fail upgrade because it leaves a stranded operator version.
I would recommend reverting the change that broke the upgrade rather than completely disabling the operator (which is likely to break others who depend on it for testing)

(If this doesn’t fail upgrade tests that’s probably a bug in the upgrade tests...)

This is likely to fail upgrade because it leaves a stranded operator version.
I would recommend reverting the change that broke the upgrade rather than completely disabling the operator (which is likely to break others who depend on it for testing)

@smarterclayton We have recently introduced this clusteroperator resource to track the deployment progress of packageserver component within olm. I don't believe this clusteroperator resource would exist in a 4.1 cluster.
No other test is depending on this newly introduced clusteroperator object yet, I believe.
I can definitely revert the PR that that introduced this clusteroperator object as well if that is desired. But I think it would have the same effect.

/test unit

Upgrade tests run against both 4.1 and recent builds of 4.2, so the e2e-aws-upgrade job should be failing because you removed an operator it knows about

/retest

/lgtm

/retest

/hold

/hold cancel

/hold

/refresh

Keeping this staged just in case, but #925 should have fixed the underlying bug, making this unnecessary.

Fixed the underlying issue


Go Live

This project uses semver, please check the scope of this pr:

[x] #patch# - backwards-compatible bug fix
[ ] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

CHANGELOG

Initialize addon and wire up to CI so it publishes to npm.

:+1:


iOS cordova.file.syncedDataDirectory writable/readable but files aren't synced between devices

Bug Report
Hi Cordova team,
I'm not sure if it's a bug or if I misunderstood the documentation about cordova-plugin-file so any help/advices would be very appreciated (I'm targeting/working on an iOS webapp).
Problem
I'de like to read/write file in a synced folder automatically synced with iCloud. The goal is to be able of writing the file from device1 and reading the same file from device2 (and vice-versa). Files wouldn't have to be accessible from the user, only from the app (private files such as user preferences, user infos, etc).
What is expected to happen?
From the doc, I understood that files saved into the cordova.file.syncedDataDirectory folder would be automatically synced. The value of the syncedDataDirectory key returns: file:///var/mobile/Containers/Data/Application/01234-56789/Library/Cloud/
What does actually happen?
I can write/read file from this folder from the same device, everything works as expected.
However, files are not synced at all between devices.
In a first time, I just like to get a confirmation from developers of this plugin about the goal of this syncedDataDirectory?
What do you mean by "Holds app-specific files that should be synced (e.g. to iCloud)"? Do you confirm that files written on device1 could be read from device2?
Information

Of course all iOS devices on which I did test are connected to the same iCloud account, iCloud is enabled in device settings, iCloud Drive is enabled (don't know if it has anything to do with), iCloud for my app is enabled too.
Command or Code

I have added to my config.xml (from the doc but I'm not sure it would be necessary in my case):
<preference name="iosExtraFilesystems" value="library,library-nosync,documents,documents-nosync,cache,bundle,root" />
<preference name="iosPersistentFileLocation" value="Library" />

I also tried to add iCloud Capabilities from Xcode (from TARGET > SIGNING & CAPABILITIES).
I have created a new iCloud container iCloud.com.company.container-name.
I have checked the box iCloud Documents in my iCloud Capabilities.
I have upgraded my app build version number (such as explained in the Apple Doc about NSUbiquitousContainers).
I also tried to add these new key/value in my info.plist:
<key>NSUbiquitousContainers</key>
<dict>
<key>iCloud.com.company.container-name</key>
    <dict>
        <key>NSUbiquitousContainerIsDocumentScopePublic</key>
        <true/>
        <key>NSUbiquitousContainerName</key>
        <string>TEST NEW CONTAINER</string>
        <key>NSUbiquitousContainerSupportedFolderLevels</key>
        <string>ANY</string>
    </dict>
</dict>

Environment, Platform, Device
• cordova -v: 11.0.0
• platform ios: 6.1.1
• cordova-plugin-file: 7.0.0
• Devices tested: iPhone7 Plus, iPhone 12 Pro, iPad Air Pro, iPad Air
Thanks for your help.

Did you find any solution? @Faksprod

@JamesIde: Unfortunately no (if you find how to make it works your feedback would be very much appreciated). I had to find an other way for saving/syncing file. I finally used cordova-plugin-save-dialog which allow a user to save a file in the iOS File folder, then the user can choose the Cloud folder. Not the best experience for the user but it works... Good luck!

I believe syncedDataDirectory is working as intended here.
The directly is synced with iCloud, which means users that backups their device can restore the app and the files for that app install. It doesn't mean that file will be synced across different devices. If it did, it's also the wrong way to do it (see last paragraph).
When you install an iOS app, it will have an install ID, which is something that is randomly generated. These ids may not match across devices, even for the same app. This part I'm not really sure on, but I think syncedDataDirectory effectively links against an install id, rather than a bundle id, making the files synced to iCloud only accessible by the app install that put the files there. Synced directories in this context effectively means the files will be saved as part of itunes backups.
Syncing files across devices I believe requires the use of other APIs that isn't really a File System based to manage documents on iCloud, and to resolve potential version conflicts, should remote have a version that local doesn't have, and local also has an update that remote doesn't have. This is something not exposed by the file plugin.

Ok it seems I misunderstood the cordova-plugin-file documentation 😅
Thanks for great clarifications @breautek


refactor: correct comments

Description
Placeholder PR to correct comments and add missing ones.
Type of change

[x] This change requires a documentation update

How has this been tested?
N/A
How has this been benchmarked?
N/A
Checklist:

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I did not modify files generated from templates
[x] golangci-lint does not output errors locally
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

should we merge @yelhousni ?


Fix broken link in install-and-run.md

Fixed broken link to configure_earlgrey_pods.rb in install-and-run.md.

LGTM

I signed it!

Thanks for the fix!


remove let(:source) from shared_contexts.rb

Poking around in the tests, I noticed that we were parsing this source
for every test, even in cases where we never use it, which is most of
them. This removes the default let(:source) and stops automatically
calling begin_investigation on the cop. Instead it calls it in places
where it is necessary.
I benchmarked it locally, and while not a huge win, it consistently cut
about 2-3 seconds off my build times, from ~52 seconds to ~50 seconds.

Before submitting the PR make sure the following are checked:

[X] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[X] Feature branch is up-to-date with master (if not - rebase it).
[X] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[X] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[X] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

I think that's a nice improvement. Thanks!


[spark] Compaction add parallelize parallelism to avoid small partitions

Purpose

Linked issue:  https://github.com/apache/paimon/issues/4157

Tests

API and Format

Documentation

can you have time to cc @Zouxxyy

unaware-bucket table also not support parallelism，should also support it?

+1


OSM Japan Slack Invite is no longer valid

When making a commit to OSM via iD among other social network sites/services, it presents an invalid Slack invite for OSM Japan that has expired:
https://join.slack.com/t/osm-japan/shared_invite/zt-d0my5ek2-SRxGCIsPPIyaOWkJOZ4EMg

This should be changed to a permanent Slack invite

iD is pulling this data from https://github.com/osmlab/osm-community-index and it should be fixed there

iD is pulling this data from https://github.com/osmlab/osm-community-index and it should be fixed there

Yep, someone can open an issue or PR there.


[ic-select-with-multi] Code example mistake

Summary of the bug
The top code demo for IcSelectWithMulti doesn't contain an options array which breaks the example
Copy the options from examples below to fix the issue
📸 Screenshots or code
Link to code example

@gd2910 can you assign this issue to me? I would like to work on it.

Done @otutukingsley 👍

@gd2910 @GCHQ-Developer-112 which line am I adding the options array to? I assume we are to make this change in the code.mdx file which is in the src/content/structured/components/multi-select directory.

Hi @otutukingsley , you're adding the options array to line #108, above the return statement.

@gd2910 I just created the PR for this issue, can you take a look when you have the time? Thank you!

@gd2910 I just created a new PR for this issue, can you take a look when you have the time? Thank you!


Example in NestedScrollView documentation is overcomplicated and misleading

I was using the NestedScrollView widget in my flutter application and was following the example present in the widget documentation here.
Following the example, I wrapped my SliverAppBar in a SliverOverlapAbsorber and used a SliverOverlapInjector in my inner scrollview to align it with the bottom of the SliverAppBar
Even though this has the desired effect of aligning the scrollview with the app bar, it has the following issues:

The GlowingOverscrollIndicator is hidden by the sliver app bar when you are scrolling upwards
The RefreshIndicator is not visible if the inner scroll view is wrapped in one.

The fix to all of these issues, to my surprise, was simply to completely remove the overlap absorber and injectors. The scrollview was appropriately aligned and the overscroll indicators and refresh indicators are visible.
I think the documentation here is misleading as it seems to suggest that these absorbers and injectors are necessary in order to use the NestedScrollView correctly, and it does not indicate the exact scenario when these special widgets are required (if they are required at all anymore).
Maybe the documentation should be updated to contain a simpler example that targets the simplest most common use case (that is, a scrolling app bar shared across multiple tabs / pages).

Was about to open the same issue.
In general when it comes to Slivers I find not only the docs confusing but the APIs hard to work with.
Seems like its one of those use-cases that don't quite fit with the everything is a widget motto and all that we have are hacky work-arounds.


Attestation Readme issues

Section link:

Reason：No create_thing method .
Suggestion:  Delete the whole part about Create the thing.

Section link:

Suggestion:
We should keep the items listed in sample summary  match to the headlines.
Update Get the thing to Get an attestation policy for a specified attestation type.
Update List the things to Set an attestation policy for a specified attestation type.

Section link:

Reason：No list_things  method.
Suggestion: Delete the code  things = client.list_things()
@jongio for notification.

Funny you should mention that - I just went through building a decent one for Java, let me see what I can do about transferring it.

Funny you should mention that - I just went through building a decent one for Java, let me see what I can do about transferring it.

Removed the label - I've been working on other issues, getting back to this one soonish.


Update QA button strings

See https://github.com/openstreetmap/iD/pull/7095#issuecomment-570066602 for motivation.

"Remove" better reflects that the issue is being permanently hidden for everybody. With "False Positive" also helping convey what's being flagged to the QA service.
"Issue" is a little more suitable than "Error" because it has a slightly different meaning which can apply to a wider variety of things (e.g. "Missing Geometry" in ImproveOSM isn't necessarily an error, but could be described as an issue).
Just removed "Error" from the KeepRight title as this is what's done in iD v3 anyway and works fine in the title spot.
Removed the detail_title key since it isn't used anywhere (made me ponder whether localised string coverage should be tested).

The idea to add information buttons with more detailed explanations was also floated in Slack. Could be investigated in a further PR.
I'm somewhat unfamiliar with Transifex, will these strings be prompted for translation updates in other languages because they've changed?

@SilentSpike Looks like an improvement. "False Positive" is a bit academic but "Not an Issue" isn't quite right and I can't think of anything better. Mostly advanced mappers will see it anyway so it's fine.

I'm somewhat unfamiliar with Transifex, will these strings be prompted for translation updates in other languages because they've changed?

Yep. Transifex pulls everything in core.yaml and anything that changed requires re-translation. However, if the key is the same then the prior translations are shown as quick options, so it's not as bad as starting from scratch.


Feature/scroll watch aside

Merged PR #28
Added scroll-aware marker on Aside
Fix for Main overflowing when the main element had too wide code samples. The current fix prevent heading links visibility.

Scrolling no longer works when you click on the links on the right. However, it works fine when you click on the title.
https://github.com/gsoft-inc/wl-hopper/assets/2379339/4c60224b-67a8-4642-8e7c-7ad1ec8f6450

The colors that should be below the header do not appear in safari.

The colors that should be below the header do not appear in safari.

good catch, transform: translate3d(0, 0, 0); fixes the issue.


Remove onByDefault autofill pixel parameter

Task/Issue URL: https://app.asana.com/0/488551667048375/1206348970623342/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

develop

#4137   👈

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CDRussell and the rest of your teammates on  Graphite


esp-idf-svc Not compiling from template project

Bug description
Not compiling esp-idf-svc when using the generated template from here

Would you like to work on a fix? [y/n]

To Reproduce
Steps to reproduce the behavior:

Install prerequisates
Download template
Attempt to build

Expected behavior
Build the templates example code
Screenshots

Environment

OS: macOs 15
How did you install the environment: [e.g. espup 0.3.2]

Additional context
Compiling esp-idf-svc v0.49.1
error[E0308]: mismatched types
   --> /Users/user/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/esp-idf-svc-0.49.1/src/tls.rs:214:36
    |
214 |                 rcfg.alpn_protos = bufs.alpn_protos.as_mut_ptr();
    |                 ----------------   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected `*mut *const u8`, found `*mut *const i8`
    |                 |
    |                 expected due to the type of this binding
    |
    = note: expected raw pointer `*mut *const u8`
               found raw pointer `*mut *const i8`

error[E0308]: mismatched types
     --> /Users/user/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/esp-idf-svc-0.49.1/src/tls.rs:653:25
      |
652   |                     sys::esp_tls_conn_new_async(
      |                     --------------------------- arguments to this function are incorrect
653   |                         host.as_bytes().as_ptr() as *const i8,
      |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected `*const u8`, found `*const i8`
      |
      = note: expected raw pointer `*const u8`
                 found raw pointer `*const i8`
note: function defined here
     --> /Users/user/Documents/Projects/test-esp32c3-app/target/riscv32imc-esp-espidf/debug/build/esp-idf-sys-eeec4fa03cb86953/out/bindings.rs:40961:12
      |
40961 |     pub fn esp_tls_conn_new_async(
      |            ^^^^^^^^^^^^^^^^^^^^^^

error[E0308]: mismatched types
     --> /Users/user/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/esp-idf-svc-0.49.1/src/tls.rs:661:25
      |
660   |                     sys::esp_tls_conn_new_sync(
      |                     -------------------------- arguments to this function are incorrect
661   |                         host.as_bytes().as_ptr() as *const i8,
      |                         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected `*const u8`, found `*const i8`
      |
      = note: expected raw pointer `*const u8`
                 found raw pointer `*const i8`
note: function defined here
     --> /Users/user/Documents/Projects/test-esp32c3-app/target/riscv32imc-esp-espidf/debug/build/esp-idf-sys-eeec4fa03cb86953/out/bindings.rs:40943:12
      |
40943 |     pub fn esp_tls_conn_new_sync(
      |            ^^^^^^^^^^^^^^^^^^^^^

error[E0308]: mismatched types
   --> /Users/user/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/esp-idf-svc-0.49.1/src/private/cstr.rs:168:8
    |
163 |         cstrs[i] = cstr.as_ptr();
    |         ----- here the type of `cstrs` is inferred to be `[*const u8; N]`
...
168 |     Ok(cstrs)
    |     -- ^^^^^ expected `[*const i8; N]`, found `[*const u8; N]`
    |     |
    |     arguments to this enum variant are incorrect
    |
    = note: expected array `[*const i8; N]`
               found array `[*const u8; N]`
help: the type constructed contains `[*const u8; N]` due to the type of the argument passed
   --> /Users/user/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/esp-idf-svc-0.49.1/src/private/cstr.rs:168:5
    |
168 |     Ok(cstrs)
    |     ^^^-----^
    |        |
    |        this argument influences the type of `Ok`
note: tuple variant defined here
   --> /Users/user/.rustup/toolchains/nightly-aarch64-apple-darwin/lib/rustlib/src/rust/library/core/src/result.rs:532:5
    |
532 |     Ok(#[stable(feature = "rust1", since = "1.0.0")] T),
    |     ^^

For more information about this error, try `rustc --explain E0308`.
error: could not compile `esp-idf-svc` (lib) due to 4 previous errors

Hi! The errros are already fixed on git, adding:
[patch.crates-io]
esp-idf-svc = { git = "https://github.com/esp-rs/esp-idf-svc" }
esp-idf-hal = { git = "https://github.com/esp-rs/esp-idf-hal" }

to your Cargo.toml should fix the issues

Hi! The errros are already fixed on git, adding:
[patch.crates-io]
esp-idf-svc = { git = "https://github.com/esp-rs/esp-idf-svc" }
esp-idf-hal = { git = "https://github.com/esp-rs/esp-idf-hal" }

to your Cargo.toml should fix the issues

Actually you need all of:
[patch.crates-io]
esp-idf-svc = { git = "https://github.com/esp-rs/esp-idf-svc" }
esp-idf-hal = { git = "https://github.com/esp-rs/esp-idf-hal" }
esp-idf-sys = { git = "https://github.com/esp-rs/esp-idf-sys" }
embuild = { git = "https://github.com/esp-rs/embuild" }

Alternatively, you can downgrade your nightly to a one from ~ 2 weeks ago, and use that.
Or, just use the esp toolchain.
NOTE: We'll have a new release very soon now (as in a week if not a few days).

Yes that has worked, thanks guys for the fast replies


Add proxy functionality to AsyncWebCrawler and AsyncPlaywrightCrawler

This PR adds proxy functionality to the AsyncWebCrawler and AsyncPlaywrightCrawlerStrategy classes.
Linked Issue: #116

Modified AsyncWebCrawler to accept a proxy parameter.
Updated AsyncPlaywrightCrawlerStrategy to handle proxy settings when launching the browser.
Added usage example in the documentation.

This allows users to specify a proxy server for crawling, which can be useful for accessing content behind firewalls.

Are we going to take this in?

@neelthepatel8 Thx for the suggestion, however, AsyncWebCrawler already supports the proxy parameter. Or perhaps I am missing something, right now in the constructor of AsyncWebCrawler we set the proxy in line 51 self.proxy = kwargs.get("proxy"), and within the start function this code snippet set up the proxy:
if self.proxy:
      proxy_settings = ProxySettings(server=self.proxy)
      browser_args["proxy"] = proxy_settings

Am I missing anything?
@Barrierml already we have the proxy support, just simply pass the proxy when you are creating an instance from AsyncWebCrawler class.

@unclecode My bad on re-setting the proxy in the AsyncWebCrawlerStrategy class, must have missed it. What I noticed though is that even though the AsyncWebCrawler can take in a proxy but it never uses it. I think the line 51 you mentioned is in AsyncWebCrawlerStrategy. So is it the case that you have to pass in the proxy only through the strategy?

@neelthepatel8 Not a problem at all, let's review and please help to make sure this works. Look at here async_webcrawler.py:

In this image line 28 I am passing kwarg to AsyncPlaywrightCrawlerStrategy, so technically, in the line 51 of async_crawler_strategy.py the passed proxy should be in the right place. Please cross check my answer, thx

That makes sense, I didn't notice that my bad!
We can close this PR then.

Yes, I miss understood another error to be a proxy error. My working proxy just times out when using it so I presumed it was not being used correctly.
[LOG] 🌤  Warming up the AsyncWebCrawler
[LOG] 🌞 AsyncWebCrawler is ready to crawl
[LOG] 🕸 Crawling https://www.trychroma.com/engineering/serverless using AsyncPlaywrightCrawlerStrategy...
[ERROR] 🚫 arun(): Failed to crawl https://www.trychroma.com/engineering/serverless, error: [ERROR] 🚫 crawl(): Failed to crawl https://www.trychroma.com/engineering/serverless: Page.goto: Timeout 60000ms exceeded.
Call log:
navigating to "https://www.trychroma.com/engineering/serverless", waiting until "domcontentloaded"

2024-10-23 08:35:44 - app.helpers.web - INFO - Successfully extracted content

No issues without the proxy and its the same proxy used for multiple other things. Will just keep a pin in it and explore it later if people are successfully using the proxy feature. Thanks!

@kylesf @neelthepatel8 Just for the reference this is the way U can use the proxy:
async def proxy():
    async with AsyncWebCrawler(headless = True, proxy_config = {
            "server": "http://ADDRESS",
            "username": "USR",
            "password": "PWD"
        }) as crawler:
        url = "https://www.nbcnews.com/business"
        result = await crawler.arun(
            url=url,
            bypass_cache=True,
            word_count_threshold = 10,
        )

    print("Done")

This is assuming you need username and password, if not do not pass it.


OpenShift group info support

Controller now will get associated group information from login-in OpenShift user.
The OpenShift API is [apis/user.openshift.io/v1/users/~], details please refer to
https://docs.openshift.com/container-platform/4.6/rest_api/user_and_group_apis/user-user-openshift-io-v1.html
It then fetch corresponding roles for each group role and combine them.

This is for NVSHAS-5977 -
support clusterrolebindings or rolebindings with group info in IDP for Openshift


Unable to Reset Zoom and Center to default Zoom and default Center using setZoom and setCenter methods

I want to set map zoom and center to default zoom and default center by using methods like setZoom and setCenter when someone zoom in or changed the center.
But these two methods are not available in react-google-maps package. How can i solve this issue ??

Not sure if I get what you mean, but If you dont want to change map zoom and center, you could just set zoom and center (not defaultZoom and defaultCenter) in GoogleMaps.

@evonsdesigns is right, and that decision is silly to me.
The suggested solution is to manage zoom in state and pass the zoom prop to your GoogleMap component, but for me it seems to only work the first time. So if like for me, it doesn't work for you, then it seems you gotta also listen for when zoom changed and update the props with that value as well. Something like this:
onZoomChanged = () => {
    this.setState({zoom: this.map.getZoom()});
}

render() {
    <GoogleMap
        ref={map => this.map = map}
        zoom={this.state.zoom}
        onZoomChanged={this.onZoomChanged}
    >
    </GoogleMap>


Update to go 1.4

Please bump the version

This was resolved recently by pulling my PR #2.

Thanks


New Skybrud.Social v1+ support

Hey Anders, Firstly high five for your awesome work on these packages they've been extremely useful. I was just wondering if there were any plans to use the new Skybrud.Social v1+ project with this. I assume this would need to wait until the supporting services are out of beta? Cheers, Tom

Hi Tom and thanks for the feedback.
Yes, this package will support the new Skybrud.Social packages. I have thought about creating a beta release of a new version of this package as well, but haven't gotten around to it yet. But perhaps I should give that some focus ;)


Callout default background

Thanks for this wonderful plugin. I have an issue on the behaviour of callout. Without using grid, the background of callout is white, so that it looks not pretty on white background. (I use the simple theme.) I wonder if I miss something or it is expected.

Yeah i've noticed that too, will fix it in an upcoming version

will be fixed with the upcoming version


Ember upgrade to v3.28 and others

Basically trying to upgrade packages that depend on an older version of embroider/macros which causes problems in frontend-gelinkt-notuleren. However, this turned into a complete Ember upgrade (using ember-cli-update).
Might be best to separate the embroider/macros fixes into their own separate branch.
There are still issues with this branch, which I could not figure out:

Invalid .eslintrc.js config, even though the part that it complains about is just like any other project.
Incorrect tests/dummy/app/router.js types.

There is one more error that I havn't been able to fix. This requires more TypeScript knowledge. Hopefully @abeforgit can take a look at least.
Upon building via ember serve the error presented is:
tests/dummy/app/router.ts:5:3 - error TS2416: Property 'location' in type 'Router' is not assignable to the same property in base type 'Router'.
  Type 'string' is not assignable to type '"none" | "auto" | "history" | "hash"'.

5   location = config.locationType;
    ~~~~~~~~

The project continues to be built, and runs correctly otherwise.


add bitpos command

Added bitpos command, please review.
resolves #105
Thank you,
Grigoriy

Coverage increased (+0.02%) to 90.648% when pulling 7fe7e8f6e0400596b6777733c89ea3aede4961a2 on gchudnov:bitpos into faf59ff67962e68db2292993eb88c88ae1671a7f on etaty:master.

Coverage increased (+0.02%) to 90.648% when pulling f8d7e775f34adcdb6f867f17f678ddf764e79b71 on gchudnov:bitpos into faf59ff67962e68db2292993eb88c88ae1671a7f on etaty:master.

thanks


Try out node-fetch instead of got; test [*****]

The saga continues
Refs https://github.com/badges/shields/issues/4655
OK, so lets start off with a summary of where we are.:
We migrated the majority of our HTTP traffic to got. There's definitely some noticable increase in memory usage compared to request, although we're now running with a bit more headroom. Before doubling down on got and moving the rest of our traffic, lets try out node-fetch and see if we get better performance with that first. Interesting side note is that right now our resource usage seems to be weirdly low. Maybe everyone is on summer holidays? Once reviewed we might want to hold off deploying this until our usage looks a bit more typical.
I think having got this far with node-fetch, if the resource usage seems about the same I think I'm in favour of got because the higher-level abstraction is just a bit nicer to work with, but if it looks like node-fetch improves our memory footprint I think its a fine solution to move forward with.
For the moment I'm really just interested in seeing what happens to performance with node-fetch vs got so

I'm going to leave influx-metrics.js using got for the moment. Its not on the 'critical path'. Once we've decided on a way forward, we can standardize one one client but while we work it out we're going to have a random mashup of some stuff using node-fetch, some stuff still using request and influx-metrics.js using got.
There are some rough edges we might want to sand down if we commit to this approach. More on this inline.

@calebcartwright are you able to double-check this works ok through a proxy when you get a chance?
There will be a few failing service tests on the build, but they are all unrelated. Its https://github.com/badges/shields/pull/6913 , https://github.com/badges/shields/issues/6912 and the usual suspects for flaky/transient failures.

Warnings

:warning:
This PR modified service code for security-headers but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for opm but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for mozilla-observatory but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @chris48s!

  Generated by :no_entry_sign: dangerJS against fe5d480a08e4c43754c1b652037f5a4efc0c9f1f

@calebcartwright are you able to double-check this works ok through a proxy when you get a chance?

Yup, will do

Will test this in my env this weekend

Working fine behind a proxy :+1:

Right. All seems to be in order. I'll plan to deploy this tomorrow evening


ClassNotFoundException when create transformer.Transformer

Hi,
I'm having the following Issue:
"java.lang.ClassNotFoundException: com.myapp.domain.model.transformer.Transformer"
It occurs when Transformer.java try get instance of the generated "transformer.Transformer" class
I have two modules: domain and data.
I want to map one object from data module to another object in domain module.
I added the dependencies in data module (transformer library and compiler).
What's wrong here?

+1


NEST-486: (Preview branch) Renaming EmailQuotaIndex columns and dropping index

NEST-486

OK then. I don't think this will be needed in the end; after the merge of the other PR dev can be merged to orchard-core-preview-17696.

Sure we can do that, it was needed only for the checks. Thanks.

Merged the latest dev to the preview branch after merging https://github.com/Lombiq/Hosting-Tenants/pull/95 so this is not needed.


New profile photo fails to apply on new Hubzilla-channel

I can access the menu for setting a new profile photo, photos are uploaded and stored in the profile folder in the channel cloud, but the image cannot be applied.

Same here.

Sorry. Forgot "chmod -R 777 store" :-/

Right, I deleted the channel - also in the database - and started over. A new big profile Image is applied on the channel page - , e.g. https://zot-mor.rhcloud.com/photo/profile/l/7 but not the little one included in the navbar or in posts, e.g. https://zot-mor.rhcloud.com/photo/profile/m/3 . How to fix?

Updating the profile image URL in column xchan_photo_l, xchan_photo_m and xchan_photo_s in table xchan at ZotMor seems to do the trick there. Now to see if it propagates to clones.

There are a number of issues with that xchan, compounded by the fact that you've got a number of hublocs and the primary is dead. If you want zot-mor to be the primary, please visit /locs on that site and make it so. This will probably fix several of the xchan issues. Delete any locations that aren't valid. Once you've done that, go back and edit the xchan_photo_date and make it later than the date that's there currently. Within about 24-48 hours the photo should be propagated.

Right, I have done all steps except the date bit now. The OpenShift-server seems to have some issue and does not respond well at the moment.
I was hoping to have fixed my OpenShift-Hubzilla-deploy script first before making ZotMor my primary - see initial mostly working version at https://github.com/HaakonME/openshift-hubzilla-deploy/blob/master/openshift-hubzilla-deploy .
The store directory, .htconfig.php and .htaccess needs to be handled correctly upon each git push, ie. they must be in the persistent $OPENSHIFT_DATA_DIR to survive, and then symlinked in the new current repo.
The store seems fine now, but I got some "cannot stat" error when it got to the .htconfig.php , so either I am creating the symlinks wrong, or they cannot be created this way, or perhaps should be moved into the step before deploy and put in a script called build. The documentation is very vague about what should be done in which step.
I think we can close this one. Thank you!


Supports playwright json format report

We already provide a "playwright" plugin and support JUnit reporter report.
However, the JUnit reporter produces the final result of the test even if the test is retried. But JSON reporter includes all results when the test're executed multiple times.
So updated to support JSON reporter reports and add the --json option.

Thank you!


How to comeback to a normal person (Im stucking at becoming a vampire)

Hi can you tell me how to come back to normal person. There is no instruction for that?


update directive priority and replace transcluded element content if it exists

If an element included both an i18ng directive and another directive that used a transcluded template, the transcluded template was getting overwritten by the translation. Apparently, transcluded templates are rendered before directives are executed, so even with a higher priority, i18ng was still overwriting the html of the transclusion. The fix here adds the priority AND THEN looks for a [ng-transclude] element. It will apply the translation to the transclude element, or to the base element if one is not found.

I'm assuming this isn't needed if it's been this long.. we must have found another way.


add support for rst files to view-markdown-source

Related to #4244, this adds support for another common markup format that is rendered by GitHub, reStructuredText.
Should this feature be renamed to view-markup-source since it would now support more formats than just markdown? I'll let someone else decide.
Test URLs

https://github.com/github/markup/blob/46908504f5/test/markups/README.rst

Screenshot

I think it can be renamed to view-source, since it could apply to any rendered file, but let's wait for a more comprehensive search of compatible files before we make changes.


Follow upstream instructions to include system pybind11

The previous way doesn't work with Arch Linux's pybind11, but the official recommended way does.
Ref: https://pybind11.readthedocs.io/en/latest/cmake/index.html

As author of commit 6cdd438556c176d9f13f092796663e8745bc76e7, I have no objections.
Both solutions seem to work for me.