TapGesture

[0:] Element: Xamarin.Forms.TapGestureRecognizer is already a child of XFPINView.BoxTemplate. Remove Xamarin.Forms.TapGestureRecognizer from XFPINView.BoxTemplate before adding to XFPINView.BoxTemplate.

@angelru
I will look for this issue later (mostly in a week or two) when I get some free time. Meanwhile, would you please provide me some more details to reproduce this issue like:

Platform
Steps to reproduce
etc.


Add bind parameters to neuroglancer instantiations

This PR just adds a bind_address and bind_port parameter to all neuroglancer instantiation calls. The parameter defaults do not change prior behavior, and specifying bind parameters is useful for me to access neuroglancer via ssh tunnels to a cluster that does not normally expose any ports/addresses.

gtg! thanks @atc3


Allows filters and persistent document stores for Chroma

Fixes #340 and #341

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: illorca-verbi:x: nachollorcaYou have signed the CLA already but the status is still pending? Let us recheck it.


Resolve Drag and Drop error in Obsidian 1.5.3+

This change is to resolve the issue in Obsidian 1.5.3+ where drag and drop is no longer possible.
https://github.com/vslinko/obsidian-outliner/issues/506

See #530, it implements a more reliable version of this fix as well as isomorphic bugs in other parts of the code.


iPhone 6+ mobile safari landscape with mulitple tabs open and bootstrap modal hides immediately after open

I am facing problem in bootstrap modal with iPhone safari 6+ landscape mode. This bug occurs only in iPhone 6+ landscape mode with multiple tab open in safari browser.
Below are the step to generate the bug.

Goto http://getbootstrap.com/javascript/#modals-sizes on your iPhone 6+ in landscape mode - be sure you must open multiple tab
Scroll down the page and click on "Small Modal" button in Optional size section.
You can notice that modal is opening and closed immediately in flash and grey background remain as it is. You can't do any clicks now.
Now, try to hide the Nav-bar and tab bar manually by scrolling up the page. It will show the bootstrap modal as expected. But it's not working when it open with multiple tab open in browser.

Confirmed, with the proviso that the tab bar must be visible when you go to click the button.

You can't do any clicks now.

I could only repro that part some of the time.

Filed WebKit bug: https://bugs.webkit.org/show_bug.cgi?id=153056

And added the bug to our Wall: http://getbootstrap.com/browser-bugs/

Did anyone find any (JS-)workaround? I reported the same problem for jQuery Mobile: https://github.com/jquery/jquery-mobile/issues/8539
Interestingly the invisible content is still working (modal buttons) here.
To show the elements: Scroll alot or change tab back and forth

The team did not attempt to work around this, given how very hacky it would be, and how tricky it would be to detect the condition.
My general observation is that position: fixed; still tends to have buggy corner cases on mobile. Avoid it whenever you can.

I ran into the exact same problem and tried to "solve it" by forcing safari to repaint/recalculate the layout -- as the dialog becomes visible if you toggle between the open browser-tabs. I tried the now classical zoom (better support) and and a few other attributes but came up with this rather ugly workaround that seems to work for this situation:
.modal.in .modal-dialog{      transform:translate3d(0px,0px,0px); }
the css3 transform property will force the repositioning of the modal.


Add popular Graphql libraries from The Guild

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 250 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~10 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
No city for a crunchbase entry for the-guild-9777 at https://www.crunchbase.com/organization/the-guild-9777
No city for a crunchbase entry for the-guild-9777 at https://www.crunchbase.com/organization/the-guild-9777
No city for a crunchbase entry for the-guild-9777 at https://www.crunchbase.com/organization/the-guild-9777
No city for a crunchbase entry for the-guild-9777 at https://www.crunchbase.com/organization/the-guild-9777
No city for a crunchbase entry for the-guild-9777 at https://www.crunchbase.com/organization/the-guild-9777
No city for a crunchbase entry for the-guild-9777 at https://www.crunchbase.com/organization/the-guild-9777
No city for a crunchbase entry for the-guild-9777 at https://www.crunchbase.com/organization/the-guild-9777
No city for a crunchbase entry for the-guild-9777 at https://www.crunchbase.com/organization/the-guild-9777
No city for a crunchbase entry for the-guild-9777 at https://www.crunchbase.com/organization/the-guild-9777

Added a city to the Crun


Not available for Android KitKat

I'm using the DuckDuckGo search engine for a wihile in Firefox browser and now I'll try the browser on my smartphone but I can't download it. The reason is because my device is not compatable for DuckDuckGo. I'm using Android version 4.4.2 KitKat. Is it possible to make it compatable for my device?

When we kicked off this project, we decided that our minSDK would be 21 (Lollipop) based on how the Android marketshare was looking.
Targeting Lollipop has allowed us to build features faster and reduced time required to test and build compatibility workarounds for KitKat and before.
While I appreciate that does cut some people's devices out, I think we're likely to continue with this approach.


Need of mentioning package versions

Can you please mention version numbers for the following packages
Python
Tensorflow
GPflow

I'm also interested in the version numbers.
@patel-zeel Have you already found out with which versions it worked?


The sync status is Synced, but the block height is behind

By executing the command coda client status, it displays the sync status is Synced, but actually, the block height (4553) is behind the normal (4916).
The below is the screenshot.

coda client export-local-logs, the below is the log files.
2021-02-02_02-25-53.tar.gz
Pls. check it. Thanks.

Have the same problem couple of times and I see this issue on a regular basis in qa-task-force

This is a dup of #7555. Closing.


build dependencies did not run successfully because there is no compatible version of numpy

ERROR: Ignored the following versions that require a different python version: 1.25.0 Requires-Python >=3.9; 1.25.1 Requires-Python >=3.9; 1.25.2 Requires-Python >=3.9; 1.26.0 Requires-Python <3.13,>=3.9; 1.26.1 Requires-Python <3.13,>=3.9; 1.26.2 Requires-Python >=3.9; 1.26.3 Requires-Python >=3.9; 1.26.4 Requires-Python >=3.9; 2.0.0 Requires-Python >=3.9; 2.0.1 Requires-Python >=3.9; 2.0.2 Requires-Python >=3.9; 2.1.0 Requires-Python >=3.10; 2.1.0rc1 Requires-Python >=3.10; 2.1.1 Requires-Python >=3.10; 2.1.2 Requires-Python >=3.10
ERROR: Could not find a version that satisfies the requirement numpy<3.0.0,>=2.0.0 (from versions: 1.3.0, 1.4.1, 1.5.0, 1.5.1, 1.6.0, 1.6.1, 1.6.2, 1.7.0, 1.7.1, 1.7.2, 1.8.0, 1.8.1, 1.8.2, 1.9.0, 1.9.1, 1.9.2, 1.9.3, 1.10.0.post2, 1.10.1, 1.10.2, 1.10.4, 1.11.0, 1.11.1, 1.11.2, 1.11.3, 1.12.0, 1.12.1, 1.13.0, 1.13.1, 1.13.3, 1.14.0, 1.14.1, 1.14.2, 1.14.3, 1.14.4, 1.14.5, 1.14.6, 1.15.0, 1.15.1, 1.15.2, 1.15.3, 1.15.4, 1.16.0, 1.16.1, 1.16.2, 1.16.3, 1.16.4, 1.16.5, 1.16.6, 1.17.0, 1.17.1, 1.17.2, 1.17.3, 1.17.4, 1.17.5, 1.18.0, 1.18.1, 1.18.2, 1.18.3, 1.18.4, 1.18.5, 1.19.0, 1.19.1, 1.19.2, 1.19.3, 1.19.4, 1.19.5, 1.20.0, 1.20.1, 1.20.2, 1.20.3, 1.21.0, 1.21.1, 1.21.2, 1.21.3, 1.21.4, 1.21.5, 1.21.6, 1.22.0, 1.22.1, 1.22.2, 1.22.3, 1.22.4, 1.23.0, 1.23.1, 1.23.2, 1.23.3, 1.23.4, 1.23.5, 1.24.0, 1.24.1, 1.24.2, 1.24.3, 1.24.4)
ERROR: No matching distribution found for numpy<3.0.0,>=2.0.0

I tried different version of python3.8, none of them worked

may be you can try this https://github.com/salesforce/LAVIS/issues/641

Same error

same issue

I encountered the same issue. It seems that the package salesforce-lavis is supported on python>=3.9 now. If your pip is bounded with python 3.8, numpy version can not be satisfied.
So try this:
pip install numpy==2.0.1

If it fails, you should use the higher version of python. And then install salesforce-lavis:
pip install salesforce-lavis

I think it is helpful to you.


Add dot:true to glob for file matching

Include directories and files that begin with periods in file pattern matching.
Rather than filtering available files at the glob level, make use of the existing includes and excludes options.
One example where a site would need this is one that had an Android app config file. It is required that they are placed in a .well-known directory.

Hi @nickpisacane - any status on if/when this might make a release?

@dtesta @brianpaulrue  Sorry for the hold-up! @brianpaulrue  Thanks for your contribution!

Available in v0.0.14


Create_box

Andrew
Brian
Michael

Before doing an add/commit it is important to do a git status to make sure you aren't adding extraneous files. It appears you've added your virtualenv files as part of your commit.


Javascript hooks and refetch not working.

Hi everyone.
First of all, thanks for the great gem! :+1:
I am implementing sync with the Mailboxer gem (on Faye), in order to make user messages realtime. The basic functionality works. Message sent - message received, no problem.
Here is my setup:
ConversationsController
def reply
  @reply = current_user.reply_to_conversation(@conversation, params[:body])
  sync_new @reply, scope: @conversation
end

show.html.erb
<div class="messages">
  <% @conversation.receipts_for(current_user).each do |receipt| %>
    <%= sync partial: 'receipt', resource: receipt %>
  <% end %>
  <%= sync_new partial: 'receipt', resource: current_user.reply_to_conversation(@conversation, params[:body]), scope: @conversation %>
</div>

The partial is named "receipt", and it's placed in "sync/mailboxer/receipts" because otherwise (if I place it just in "sync/receipts"), I get a "partial not found" error.
As I said, the above works just fine for the basic functionality.
Issue No1
Refetch: true won't work. Simply adding refetch:true results in a 400 - "Bad request" from the server. The "_receipt.html.erb" partial is located in "sync/mailboxer/receipts/refetch".
Issue No2
JavaScript hooks. I want to do a few things after loading the message, such as scrolling the container up a bit to show the message, but the hooks do not respond. Here is the application.coffee, with a simple alert for the afterUpdate:
class Sync.Receipt extends Sync.View
  afterUpdate: -> alert "Hello!"

I tried changing "Receipt" to "MailboxerReceipt", but got nothing. I suspect this is happening because of the folder structure ("sync/mailboxer/receipts" instead of just "sync/receipts"), but correct me if I'm wrong.
Thanks again, and any help would be really appreciated!

Also having the same problem. Although beforeInsert and afterInsert are invoked properly (I can see my log outputs generated in the console). However, beforeUpdate, afterUpdate, beforeRemove, and afterRemove doesn't seem to get called (although I can see the page updates/removes partials properly, but the callbacks doesn't just get called)

I managed to work it now! :sweat_smile: Found out that my
$(->
  class Sync.CustomerIndexDiv extends Sync.View

      beforeInsert: ($el) ->
        $el.hide()
        @insert($el)
    # ...
    # ...
)

has the $(->) on it which translates Jquery's ready() function, as I put it there initially as I thought that the page needs to be loaded first before this javascript is invoked. But it looks like, conversely it creates the problem for me :smile: After removing the $(->) wrapper, beforeUpdate, afterUpdate, beforeRemove, and afterRemove are now being invoked properly. You can try checking your code if you have a $(document.ready(function() {}) or the above Jquery $(function(){}), and try removing them for these extendes View classes. Hopefully that will solve you problem too :smiley:

@jrpolidario Thank you so much for your help! Seems to work now. At least for the beforeInsert and afterInsert. :smile:
P.S: refetch: true still doesn't work, however. If anyone has any ideas on what could be messing it up, that would be awesome of you! It'd be great to be able to sync partials in the context they're served (e.g such as partials belonging to "current_user" ).
Thank you again!

It's my pleasure! :)
I just started using refetch: true now (I am not using mailboxer). It's working for me. I can verify this because in my partial sync file, I have <%= current_user.first_name %> which returns properly the name of the current user who's browsing  the page where the partial  file gets  rendered. I tried adding turbolinks (just in case if you're using it), but it still works for me. Maybe this is already mailboxer incompatibility. Best of luck! :)

Issue No1
@MindRave
I also have the same problem. Server log shows
Filter chain halted as :find_resource rendered or redirected
Completed 400 Bad Request in 28ms

It seem there is problem with refetchs_controllers#find_resource
 def find_resource
   @resource = RenderSync::RefetchModel.find_by_class_name_and_id(
     params[:resource_name],
     params[:resource_id]
   ) || render_bad_request
 end

 RenderSync::RefetchModel.find_by_class_name_and_id call the method RefetchModel#self.supported_classes to check something.
However, in the supported_classes , Dir["app/views/sync/*/refetch"] return empty array.
The correct path should contain app/view/sync/mailboxer/receipts/refetch.
I think  refetech:true may not work on model with namespace. Sorry for my poor English.


Falling state: player not able to move left and right

Player cannot use arrow keys to move player obj left or right when falling.

resolved with Wip collision with obstructions #6 merge


Help with building opencv-python with non-free modules in vs code

Expected behaviour
Expected opencv-contrib-python wheel package to be created in dist in visual studio code.
Actual behaviour
scikit-build could not get a working generator for your system. Aborting build.
Building windows wheels for Python 3.11 requires Microsoft Visual Studio 2022.
Get it with "Visual Studio 2017":
Steps to reproduce
#306 comment
Issue submission checklist

[x] This is not a generic OpenCV usage question (looking for help for coding, other usage questions, homework etc.)

[x] I have read the README of this repository and understand that this repository provides only an automated build toolchain for OpenCV Python packages (there is no actual OpenCV code here)

[x] The issue is related to the build scripts in this repository, to the pre-built binaries or is a feature request (such as "please enable this additional dependency")

[x] I'm using the latest version of opencv-python

Can non-free modules not be installed without using visual studio code?

Related https://github.com/opencv/opencv-python/pull/831

OpenCV uses scikit-build infrastructure for C++ code. It supports CMAKE_ARGS environment variable to provide custom CMake options.
See:

https://github.com/scikit-build/scikit-build/issues/479
https://github.com/scikit-build/scikit-build/issues/588

For example, if you want to build OpenCV-Python  package from sources for Linux and enable OpenVINO and GStreamer:
 export CMAKE_ARGS="-DWITH_OPENVINO=ON -DWITH_GSTREAMER=OFF"
python3 setup.py bdist_wheel


benchmark streaming speed with tar vs zip archives

do not merge
Hypothesis
packing data into a single zip archive could allow us not to care about splitting data into several tar archives for efficient streaming which is annoying (since data creators usually host the data in a single tar)
Data
I host it here
I checked three configurations:

All data in one zip archive, streaming only those files that exist in split metadata file (we can access them directrly with no need to iterate over full archive), see this func
All data in three splits, the standart way to make streaming efficient, see this func
All data in single tar, iterate over the full archive and take only files existing in split metadata file, see this func

Results

one zip

three tars

one tar

didn't check on the full data as it's time consuming but anyway it's pretty obvious that one-zip-way is not a good idea. here it's even worse than full iteration over tar containing all three splits (but that would depend on the case).

I'm closing the PR since we're not going to merge it


languages.xx.disabled not working with v0.112.0 and later

This works as expected with v0.111.3 and earlier:
[languages.es]
languageCode = 'es-ES'
weight = 3
disabled = true

With v0.112.0 and later, the site above is published (the disabled key is ignored) with this warning:

WARN  config: languages.es.disabled: custom params on the language top level is deprecated and will be removed in a future release. Put the value below [languages.es.params].

See https://github.com/gohugoio/hugo/issues/4963#issuecomment-406076190.
I don't think languages.xx.disabled was ever documented.

Workaround: disableLanguages = ["en']


Upgrade to catch2 v3.4.0, and switch benchmarks to use catch

Fixes issues #143 and #142

Oooo nice! It looks like the CI config needs to be updated (ubuntu-18.04 is no longer supported, as well as OSX something). I can do that, and you can rebase on top
Reviewing your changes...

Changes look great :)
My only comment is: Should we include catch2 as raw files, or use git submodules? raw files allows us to make edits if we need to, but IMO, we probably won't ever need to with catch2.
If we do keep it as raw files, can we include a txt file or something notating which version / commit you copied from? So when we upgrade in the future, we know which changelog entries to check.
Thoughts?
(@martty and @BrodyHiggerson for thoughts as well on git submodules or no)

Can you rebase on master and check that CI passes now?

Oh, one other pedantic request: You can delete mention of nonius from the README, and remove its licence from LICENSE-THIRD-PARTY

Unfortunately looks like catch isn't building properly on gcc 9 and the older versions of clang/gcc

Also additionally i was thinking about this from #140

We'd only have to bump the tests to c++14 (which they might even already require 🤷♂ ). Which I'm fine with. As you mentioned, 14 is almost 10 years old now

Is it really a good idea to not support testing on potentially "supported" c++11 compilers since there wouldn't be much validation happening there with testing.

With that said technically gcc 5 should be fine with C++14 looking at what the version supports so seems like potentially some issue with catch.

Looking over the log more
Check_And_Add_Flag(${TARGET} -Wno-aligned-new)
is also causing an error on gcc 5 from:
cmake\AddCompilerFlags.cmake
https://dev.azure.com/adastley/adastley/_build/results?buildId=179&view=logs&j=75da2154-1c41-5424-3a3b-cd670808b5e4&t=c9fb99c5-dcdd-5bdf-ac8d-6b4ca6067273&l=428
Which is the only additional error over gcc 9

Actually yeah another one of these is causing the other error:
Check_And_Add_Flag(${TARGET} -Wctor-dtor-privacy)

I'm also fine with dropping gcc 5 and clang 3.7 for CI purposes

That last commit probably needs a bit more work to set the cmake variable names correctly, but basically the jist of the issue can be found here:
https://gcc.gnu.org/wiki/FAQ#wnowarning
if the -Wno- form is used, the behavior is slightly different: no diagnostic is produced for -Wno-unknown-warning unless other diagnostics are being produced.
Basically this means that CHECK_CXX_COMPILER_FLAG isn't going to work properly with -Wno- flags so we have to instead test the reverse of the flag to determine if it's available.

@RichieSams Any thoughts on clang here? We might should consider increasing the version being used for testing since clearly it has some kind of issues with unique_ptr that the other compilers don't have.

Also i refreshed after i sent that and now i see your message, i think we should probably just increase the version of clang being tested on linux a bit rather than totally dropping it. And i got gcc 5 working anyways so it's fine.

sorry, clarification. Just meant dropping the lower versions of clang. IE, just using clang 12 and 16 like we do for OSX

Ok rebase again

Ok so everything should be good to merge now if you are happy with it.

Woohoo! Congratulations on your first contribution. And thank you very much :)

Of course now onto #141 :)


Feature request: Neovim compatibility

The cmatcher works perfectly in vim, but in neovim, it seems to revert back to the default matcher. Is it possible for the cmatcher to work with neovim?

AFAICT it's not a Neovim incompatibility—you just need to install Python plugin support for Neovim:
pip install neovim

(See the Neovim Python quickstart for details.) Talk about an easy fix!

I was seeing no results returned when using cmatcher, and installing the python plugin support worked for me.  You can see more with :help nvim-python.  I think this issue can be closed.

Can confirm - works in neovim great

Hey all, so I'm having an issue with Neovim.  The matcher works great in vanilla vim.
When I load ctrlp all the files appear to be there, however when I begin typing, a 0 shows up where the files usually are.  Simply a single 0.

When I load ctrlp all the files appear to be there, however when I begin typing, a 0 shows up where the files usually are. Simply a single 0.

Double that, I have the same issue.

@kossnocorp try doing ’pip install neovim’ - worked for me

@kossnocorp Oh, you also need to run install.sh from the bundle's dir.

@eyusupov heh, hey there! I did both with no luck.

Same with me. Tried that and had to revert to the built in marcher.

I'm getting the same issue. Had vim before and the ctrlp-cmatcher was working great. Then switched to neovim and it doesn't work! I also had to revert to the default matcher (which sucks in comparison).
When I run CheckHealth in neovim everything seems to be fine. It says I have both Python 2 and Python 3 linked, so I'm not sure what the issue is. Is there a way to uninstall the c-libraries?

Ah ok I realised my issue! So it turns out that I was compiling the wrong ctrlp-cmatcher. I needed to go into my ~/.config/nvim/bundle directory and find the ctrlp-cmatcher dir there.
Kind of silly I didn't realise this before 😛


the textviews of tabs are invisible

i dont know why it happened . i use this widget according to the introduce. who can help me?please

oh i have solved this problem, coz  i made a silly mistake by accident


I look forward to leveraging my extensive experience and knowledge to contribute to your team’s goals and achievements.

Hi, I'm Ethan Tan, a seasoned web full-stack and blockchain developer with over 8 years of experience in the industry. I specialize in building robust, scalable web applications and innovative blockchain solutions. Leveraging my expertise in frameworks like React.js, Vue.js, and Next.js, along with my proficiency in Solidity for smart contract development, I deliver high-quality, secure, and efficient solutions tailored to meet your specific needs.

I'm afraid this project has nothing to do with web or blockchain. Thank you for your interest.


News: inlay hints has been supported

https://github.com/neovim/neovim/issues/18086

Nevermind, just found #77.


Provide example for running tests from command line

You mention that you provide mocha and jasmine adapters, but I can't figure out how to run jasmine or mocha tests with testacular and display results on command line.

Links are broken again :(


Enable multi in/out for rnn variant layers

For now only 1 input/output is supported for rnn, lstm, gru layer.

:octocat: cibot: Thank you for posting issue #1901. The person in charge will reply soon.


Option to show first volume cover instead of last by default

Describe your suggested feature
Currently the cover shown for any given series is the latest one. For many series, particularly romance series, the final volume cover may contain significant story spoilers (often a given couple shown being married).
To remedy this, an option to instead display the first volume's cover while browsing would prevent such spoilers from happening.
I'm not suggesting this be the default mode, but the option would be greatly appreciated.
I searched and could not find this option either in the app or mentioned in the issues. I'd be thrilled if I missed something.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.16.5.
[X] I will fill out all of the requested information in this form.

Good to know! Thank you!


Allow not to use vTaskDelete()

Currently, if INCLUDE_vTaskDelete is set to 0, the compilation will fail.
rust-lld: error: undefined symbol: freertos_rs_delete_task
>>> referenced by task.rs:167 (src\task.rs:167)

I don't like the approach of modifying the internals of the function - if the user was trying to not include it, but the function is still available from the Rust side, it could result in some weird behavior for them. I'd much rather gate it with a feature flag on the Rust side since that's the approach we already took elsewhere.
Right now it looks like freertos_rs_delete_task() is only used when spawning a task so maybe that's easy to do. We have had issues in the past where the linker still complains, and I'm not sure why - would be nice to learn more.

Add a feature in Cargo.toml, is this appropriate?

Yes you've got the idea, I would prefer the logic to not be negative here (cfg(not(not_delete_task)) -> cfg(delete_task)). This also means the feature is needed in the default list too.

Yes you've got the idea, I would prefer the logic to not be negative here (cfg(not(not_delete_task)) -> cfg(delete_task)). This also means the feature is needed in the default list too.

Modified

Thanks!


Add a noconsole option for apply-patches

This adds an option to the apply-patches script that suppresses the writing of console.log to the patched Elm files.
Fixes #3

I tried it locally and it seems to work. The changes to the network fetches were an effort to improve the error handling when you reach the API limit for GitHub (which I did when testing)

Thanks!


User "Can" when is guest

What steps will reproduce the problem?
When user is guest
Yii::$app->getUser()->can('addDiary', ['permission' => function () {
    if (!Yii::$app->getUser()->getIsGuest()) {
        return Yii::$app->getUser()->getIdentity()->getPermission()->one();
    }
    return null;
}])

What is the expected result?
Return 'false' and execute rule.
What do you get instead?
Return 'false' but doesn't execute rule.
Additional info

Q
A

Yii version
2.0.13

PHP version
7.1

Operating system
win10

Why should it execute the rule if the result is false?

If we have to make some more additional checks, we need to do this in every "can", otherway we can do this in the rule. But, maybe i'm wrong...

If we know for sure that can() failed before checking the rule then checking the rules does not make sense.

It's make sense for me.


Question: Resource limits

is there any possibility of adding settings for resource consumption similar to plv8's "memory_limit"?

Do you have a particular resource in mind? v8 can do so without impeding program usage because it can stop when it hits that allocation limit and run a GC cycle in order to reclaim memory, tanking time efficiency in order to obtain space efficiency. But a Rust program that runs up against an allocation limit like that is simply going to die. So it's possible, and I can think of several ways of doing so, but is that actually what you want to see happen?

Rust program that runs up against an allocation limit like that is simply going to die. So it's possible, and I can think of several ways of doing so, but is that actually what you want to see happen?

yes, that's what I was thinking. fail and throw a resource error of some kind
the goals would be to fail fast on under-provisioned hardware and prevent the system from becoming sluggish/unresponsive

If it's a global hardware resource limit then the Rust procedure will simply hit it and die because Rust programs largely do not handle the allocation failure case (because it is not meaningfully something they can handle). When it dies, the Rust procedure will begin unwinding and destroy all its memory usage, and then announce its death to the PL/Rust language handler, which will shrug and carry on, doing something else. When the Rust procedure ends, all its memory should be freed, either way. So I do not see how that is not "failing fast", basically.
Adding a tighter resource limit is possible but allows much more "thrash" in resource usage, as the system keeps executing PL/Rust procedures over and over that all keep failing. That seems counterproductive? If you are noticing memory leaks across transactions, do let us know, of course, I would want to fix such, as there should be very minimal ongoing resource usage after the initial startup of the PL/Rust language handler. But not zero. The language handler needs a few data structures to keep track of the functions it has compiled, loaded, and executed.

There might come a time when we can teach plrust to use Postgres as the rust GlobalAllocator -- that would mean each individual plrust function, not necessarily the extension itself.  Then we'd be able to have some kind of global "PL/Rust Memory Context".  Perhaps per-function or globally on the backend maybe.  Just thinking out loud here.
I think we're a little away from that as we need to sort out some lifetime tracking issues within pgrx.  The lifetime issues are things we're actively working on now, however.
I dunno if @workingjubilee is thinking of some other kind of "counting" allocator or something that could just panic at some hard limit.  I think a lot of this kind of thing is complicated by the fact each plrust function has no knowledge of its surroundings.

all its memory should be freed, either way. So I do not see how that is not "failing fast", basically.

yep that makes sense. I disabled swap and see the behavior your describing
but with swap enabled, if memory use spills onto swap during compilation on low power machines they (and postgres) become unresponsive. that's the main reason I'm interested in setting a threshold

that's the main reason I'm interested in setting a threshold

Maybe this is more about ulimit or control groups?

Yes, this is a problem that would affect the entire Postgres instance if the concern is about swapping disk, so solving it at the PL/Rust level does not seem appropriate for that, unless one is okay with a Postgres user's normal SQL queries triggering "slow the system unto unresponsiveness" behavior but not the PL/Rust procedures.

@olirice IIUC the failure is on compiling the function, not during the execution of the plrust function itself, is that right? So we'd be applying the resource constraints on cargo invocations, using cgroups or otherwise.
The failure during function execution, where the function panics and unwinds, looks like expected behavior to me and not cause for concern in this case.


fix(build): allow dynamic import treeshaking when injecting preload

Description
fix #14145
refs https://github.com/rollup/rollup/pull/4952
After this PR, vite can treeshaken following dynamic imports when injecting __vitePreload:
 const { foo } = await import('./treeshaken/treeshaken.js')
  const { bar, default: tree } = await import('./treeshaken/treeshaken.js')
  const baz1 = (await import('./treeshaken/treeshaken.js')).baz1
  const baz2 = (await import('./treeshaken/treeshaken.js')).baz2.log
  const baz3 = (await import('./treeshaken/treeshaken.js')).baz3?.log
  const baz4 = await import('./treeshaken/treeshaken.js').then(
    ({ baz4 }) => baz4,
  )
  const baz5 = await import('./treeshaken/treeshaken.js').then(function ({
    baz5,
  }) {
    return baz5
  })

Additional context

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[ ] Read the Contributing Guidelines.
[ ] Read the Pull Request Guidelines and follow the PR Title Convention.
[ ] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

IIUC this PR transforms dynamic imports with __vitePreload to still preserve the const { ... } = await import('...') syntax temporarily so it still works? It's a bit hard to see how it transforms without more comments or snapshots 😬
But I think the regex and implementation is fairly impressive. I'd lean to a more robust alternative still, but it's also harder. Ideally if __vitePreload is only injected in the render phase, but that makes the preload code hard to be chunked. So perhaps this is good enough for now.

IIUC this PR transforms dynamic imports with __vitePreload to still preserve the const { ... } = await import('...') syntax temporarily so it still works?

Yeah, that's what this pr mainly does. The regexp contains three sub-regexps related to three dynamic import formats, e.g:

const {foo} = await import('foo')
(await import('foo')).foo
import('foo').then(({foo})=>{})

After injecting __vitePreload, they will be transformed to something like:

const {foo} = __vitePreload(async () => {const {foo} = await import('foo'); return foo}, ...)
__vitePreload(async () => {const __vite_temp__ = await import('foo')).foo; return __vite_temp__;}, ...)
__vitePreload(async () => {const __vite_temp__ = await import('foo')).foo; return {foo: __vite_temp__};}, ...)

/ecosystem-ci run


feat: Enabled Linux ARM64 pre-compiled builds

See https://github.com/near/near-cli-rs/pull/397 for more details
@race-of-sloths

@frol Thank you for your contribution! Your pull request is now a part of the Race of Sloths!

Current status: waiting for scoring
We're waiting for maintainer to score this pull request with @race-of-sloths score [0,1,2,3,5,8,13] command. Alternatively, autoscoring [1,2] will be applied for this pull request
What is the Race of Sloths

Race of Sloths is a friendly competition where you can participate in challenges and compete with other open-source contributors within your normal workflow
For contributors:

Tag @race-of-sloths inside your pull requests
Wait for the maintainer to review and score your pull request
Check out your position in the Leaderboard
Keep weekly and monthly streaks to reach higher positions
Boast your contributions with a dynamic picture of your Profile

For maintainers:

Score pull requests that participate in the Race of Sloths
Engage contributors with fair scoring and fast responses so they keep their streaks
Promote the Race to the point where the Race starts promoting you
Grow the community of your contributors

Feel free to check our website for additional details!

Bot commands

For contributors

Include a PR: @race-of-sloths include to enter the Race with your PR

For maintainers:

Invite contributor @race-of-sloths invite to invite the contributor to participate in a race or include it, if it's already a runner.
Assign points: @race-of-sloths score [1/2/3/5/8/13] to award points based on your assessment.
Reject this PR: @race-of-sloths exclude to send this PR back to the drawing board.
Exclude repo: @race-of-sloths pause to stop bot activity in this repo until @race-of-sloths unpause command is called

🥁  Score it!
@frol, please ask your peers to score the PR using @race-of-sloths score [1/2/3/5/8/13].
If no scoring is provided within 24 hours, this PR will be scored as 1 🦥


cfn_resource.py: fix rescheduling

the update() method will not work with None, and we do return from this
function anyway, so catch it and call update only, if something is
there.

Also since this is a public package the HISTORY.rst should have a new note

Closed, as the problem was not raising NoResponse in Lambda handler.


Slow APPLY operations when filtering with labels

With a bucketized dataset of:

a list of 1600 GTS named A with only one point inside, having two labels ("host" and "disk").
another list of GTS, same conditions, but named B.
Every combination of host + disk in labels makes the gts unique (in A and B).

I want to add each unique combination of gts A to B:
[ $a $b [ 'host' 'disk' ] op.add ] APPLY
It works, but it's very slow on a recent warp:
On a old warp 1.2.22-88, it took less than 1000ms.
On a recent warp 2.0.2-162, the APPLY alone takes more than 4500ms every time.
I tested to divide each combination by myself like this:
$a [ 'host' 'disk' ] PARTITION 'pa' STORE
$b [ 'host' 'disk' ] PARTITION
[] SWAP
<%
   'series' STORE
   'labels' STORE
   [
      $series
      $pa $labels GET
      []
      op.add
   ] APPLY
   APPEND
%>
FOREACH

... and it took ~100ms.
I am a bit worried here about the low performance with the APPLY function. I am searching about what could be responsible for it.
What would be the best way to perform this operation?
PS: All time measurements has been done in the WarpScript with NOW time subtractions around the APPLY.

Can you provide an example dataset?
Also can you try applying PARTITION instead of APPLY, there is a great chance that most of the time is spent in the partitioning phase.

Code with dataset:
1600 'n' STORE

[] 'a' STORE
1 $n
<%
    'i' STORE
    NEWGTS
    {
        'host' 'host-' $i TOSTRING +
        'disk' 'disk-' $i TOSTRING +
    }
    RELABEL
    'gts A' RENAME
    1 NaN NaN NaN $i ADDVALUE
    $a SWAP +! DROP
%>
FOR
$a 'name-list-a' RENAME 'a' STORE

[] 'b' STORE
1 $n
<%
    'i' STORE
    NEWGTS
    {
        'host' 'host-' $i TOSTRING +
        'disk' 'disk-' $i TOSTRING +
    }
    RELABEL
    1 NaN NaN NaN $i ADDVALUE
    $b SWAP +! DROP
%>
FOR
$b 'name-list-b' RENAME 'b' STORE

NOW
[ $a $b [ 'host' 'disk' ] op.add ] APPLY DROP
NOW SWAP -
HUMANDURATION 'Time with APPLY: ' SWAP + 'time_apply' STORE

NOW
$a [ 'host' 'disk' ] PARTITION 'pa' STORE
$b [ 'host' 'disk' ] PARTITION
[] SWAP
<%
    'series' STORE
    'labels' STORE
    [
       $series
       $pa $labels GET
       []
       op.add
    ] APPLY
    APPEND
%>
FOREACH DROP
NOW SWAP -
HUMANDURATION 'Time with PARTITION: ' SWAP + 'time_partition' STORE

NOW
[ $a $b APPEND [ 'host' 'disk' ] reducer.sum ] REDUCE DROP
NOW SWAP -
HUMANDURATION 'Time with REDUCE: ' SWAP + 'time_reduce' STORE

$time_apply
$time_partition
$time_reduce

Results:
0: "Time with REDUCE: 0.018644s"
1: "Time with PARTITION: 0.024088s"
2: "Time with APPLY: 2.773569s"

Can you try with PR #480 please?

It's better, but there is still a difference in comparison with the "PARTITION way".
With 1600 to the for loop:
0:"Time with REDUCE: 0.028881s"
1:"Time with PARTITION: 0.047194s"
2:"Time with APPLY: 0.405302s"
With 10000:
0:"Time with REDUCE: 0.127254s"
1:"Time with PARTITION: 0.230172s"
2:"Time with APPLY: 12.623684s"

This is normal, APPLY does way more things since it has to deal with the ordering of the parameters among others. It will never be on par with the performance of REDUCE. For cases when you want to do a REDUCE, you should not consider APPLY as it adds a lot of overhead

That being said, the sort was at the wrong place, you can check the latest #480, it should perform better though still slower than REDUCE

Yeah, this is what I want :)
Results with 10000:
0:"Time with REDUCE: 0.236395s"
1:"Time with PARTITION: 0.405102s"
2:"Time with APPLY: 0.416306s"
As you said, reducer is still faster.
To replace a op.add I can use reducer.sum (cf above).
What can I use to replace a op.div ?
I think it's possible to use reducer.product with the opposite of B, but what is the best way to get the opposite of B ?

Thanks for the head up, this is a great catch whose solution leads to improved performance for both APPLY and FILTER


Using taky with nginx

Hello,
I've put taky on a Raspberry Pi 400 that's behind a router and already running a few webpages using nginx. I tried to set up nginx to put through the necessary ports for taky, but taky refuses to start afterwards because nginx is already listening to those ports. Has anyone used a similar setup? I have forwarded the ports to the RPI in the router, but that didn't seem to be enough.

Hi @jkimsis!
You can reconfigure the bind port for the CoT server in the taky.conf file, though I do not believe that the default port conflicts with NGINX and you thus I believe your issue is elsewhere in your configuration.
May I recommend switching to Docker and using nginx-proxy for automatic configuration as this would resolve a lot of your issues via automatic configuration.
Ultimately this is not an issue with taky and appears to a be a configuration issue in your specific deployment,

@Matthew-Beckett -- Thanks for helping out the users!
@jkimsis Just because this is a more advanced usage / config issue, I'd like to close the issue and keep this related to the development. Or, if you think there could be some documentation to support this better, I'd welcome assistance maintaining that!
Closing for now.


Reset after second message of MQTT

Hello.
I'm using MQTT to send a news from Node Red to wledPixel but after the second message sent to the panel it resets.
It doesn't matter how log I wait to send the second or the third message.
Any idea?
Thank you

Yes, I also had the same problems and the developer solved them by increasing the maximum length of the message to 100 characters, I think, so, check your length.

Thank you very much martusi61. Now it works!!!

Happy to help you!


Multiline logs split when rotated by Kubelet (containerd)

Describe the bug
We are running Fluentd in a Kubernetes environment using the containerd runtime and having logs rotated by the Kubelet when they reach 10MB in size and retaining the last four plus the current log. When the log rotation happens when a large multi-line log is being written and therefore gets split across the old and the new log file, the log is picked up by Fluentd as many single-line logs rather than a multiline log.
To Reproduce

Run Kubelet with the containerd runtime
Configure Kubelet to rotate logs with the following two parameters:

containerLogMaxFiles: 5
containerLogMaxSize: 10Mi

When the log file is close to the 10MB limit, start writing a large multi-line log

The first line should begin with the tag [SKY_LOG_START] as per the configuration below

Ensure that the multi-line log spans both the old and the new rotated logs files

The multi-line log should end with [SKY_LOG_END] as per the configuration below

Observe in the output log file that the multi-line log you wrote has been captured as many single-line log entries and not one multi-line log

Expected behavior
A multi-line log that get split across two files when our logs get rotated should still be picked up as a single log entry by Fluentd.
Your Environment
- Fluentd version: v1.14.3
- TD Agent version: N/A
- Operating system: Red Hat Enterprise Linux 8.7
- Kernel version: 4.18.0-425.10.1.el8_7.x86_64
- Kubernetes version: 1.21.13
- containerd version: v1.6.4

Your Configuration
<system>
  log_level info
</system>

#enable monitoring endpoint for ready and liveness probes
<source>
    @type monitor_agent
    bind <IP_ADDRESS>
    port 24220
</source>
<source>
    @type tail
    enable_stat_watcher false
    pos_file /var/lib/containerd/fluentd/fluentd-containers.pos
    tag containers.*
    read_from_head true
    # Read from /var/log/containers/ as filenames have details used by the
    # kubernetes_metadata plugin.
    path /var/log/containers/*.log
    refresh_interval 1s
    <parse>
        @type regexp
        keep_time_key true
        expression /^(?<time>[^ ]+) (?<stream>stdout|stderr) (?<logtag>[^ ]*) (?<log>.*)$/
    </parse>
</source>

<filter containers.**>
    @type kubernetes_metadata
    verify_ssl false
</filter>

<filter containers.**>
    @type concat
    key log
    separator "\n"
    stream_identity_key kubernetes.pod_name
    multiline_start_regexp /\[SKY_LOG_START\]/
    multiline_end_regexp /\[SKY_LOG_END\]/
    flush_interval 5
    timeout_label @NORMAL
</filter>

<filter **>
    @type record_transformer
    <record>
        location slu
        environment sandbox
    </record>
</filter>

<match **>
    @type relabel
    @label @NORMAL
</match>

<label @NORMAL>
    <match **>
        @type file
        @id output
        delimiter SPACE
        output_tag false
        output_time false
        path /var/lib/containerd/fluentd/containers

        #Rotate the output file every hour
        time_slice_format %Y%m%d%H
        append true
        flush_interval 0s
        flush_at_shutdown true
        #Force the event time as now
        localtime true
    </match>
</label>

## Prometheus Input Plugin Configuration

# Prometheus input plugin that exports metrics
<source>
    @type prometheus
</source>

# Prometheus input plugin that collects metrics from MonitorAgent
<source>
    @type prometheus_monitor
    <labels>
        host ${hostname}
    </labels>
</source>

# Prometheus input plugin that collects metrics for output plugin
<source>
    @type prometheus_output_monitor
    <labels>
        host ${hostname}
    </labels>
</source>

Your Error Log
N/A

Additional context
When the Kubelet does the log rotation it swaps the current log and the most recently rotated log's inodes around as follows:
393234 -rw-r-----. 1 root root 9.7M Mar 10 10:33 0.log
393223 -rw-r-----. 1 root root  11M Mar 10 10:32 0.log.20230310-103201

# Rotation 1
393223 -rw-r-----. 1 root root 4.6M Mar 10 10:33 0.log
393234 -rw-r-----. 1 root root  11M Mar 10 10:33 0.log.20230310-103312

# Rotation 2
393234 -rw-r-----. 1 root root 6.7M Mar 10 10:44 0.log
393223 -rw-r-----. 1 root root  12M Mar 10 10:43 0.log.20230310-104354

Are there any updates on this? Keen to get this resolved.


grid column not working

Hi,
I'm using ng-OfficeUIFabric to create a custom SharePoint 2013 form. For some odd reason, I can only get one "uif-" directive to render within a column. This should be pretty straightforward since it's just a grid system, but I'm completely stuck. I can use multiple fields in a column with bootstrap, but with ng-OfficeUIFabric I'm stuck with one.
Expected Behavior:
I should be able to have multiple fields/directives within a column.
Actual Behavior:
Only one field/directive gets rendered in a column.
The fields in line 36 and 41 don't get rendered. Nothing gets rendered after the first directives (lines 35 & 40), even if I type plain text.

I'm really looking forward to developing custom SharePoint forms with ng-OfficeUIFabric and appreciate any help with this issue.
Thanks

Don't use self-closing tags for e.g. uif-textfield. Please try that :)

Thanks @rolandoldengarm !  Removing the self-closing tags worked.

This project actually schooled me on that... self-closing tags aren't typically legit. There are a few in HTML that is always shown that way, but Angular exposed that thinking as bad...


Fixed some grammar errors in the std.experimental.logger help text

LGTM thank you

Auto-merge toggled on


Aes mal inject

This PR updates the prediction to actively use idle as an indicator for when registers can be written and when writes should be ignored.
it also adds a knob to enable Writes to a random register when DUT is busy.
this should bring the functional coverage above the required V2 threshold.

Sorry, I don't understand this test. The idle output is used by the clkmgr to determine if the aes clock can be turned off. The AES functional spec says writes can be pipelined with ongoing AES processing. So I don't understand why it uses idle as a gate for writes.

@matutem I think you need to read the specification.
Idle alone is not enough for clock-manager to decide if the clock can be turned off.
as indicated by this quote many situations require IDLE before continuing.

Every output register must be read at least once for the AES unit to continue. In contrast, the initial key, and control register can only be updated if the AES unit is idle, which eases design verification (DV). Similarly, the initialization vector (IV) register can only be updated by the processor if the AES unit is idle. If the AES unit is busy and running in CBC or CTR mode, the AES unit itself updates the IV register.

you are correct writes can be pipelined - and if you look at the added code you will find that the write register is NOT gated by IDLE.


Fix handling of signed API URLs

I encountered the following error when I deployed the airseeker-v2:
{"cause":{"address":"<IP_ADDRESS>","code":"ECONNREFUSED","errno":-111,"port":80,"syscall":"connect"},"code":"ECONNREFUSED","dataFetcherCoordinatorId":"1708953860436","level":"warn","message":"connect ECONNREFUSED <IP_ADDRESS>:80","ms":"+0ms","name":"Error","stack":"Error: connect ECONNREFUSED <IP_ADDRESS>:80\n    at AxiosError.from (/app/node_modules/.pnpm/axios@1.6.5/node_modules/axios/dist/node/axios.cjs:837:14)\n    at RedirectableRequest.handleRequestError (/app/node_modules/.pnpm/axios@1.6.5/node_modules/axios/dist/node/axios.cjs:3090:25)\n    at RedirectableRequest.emit (node:events:529:35)\n    at RedirectableRequest.emit (node:domain:489:12)\n    at eventHandlers.<computed> (/app/node_modules/.pnpm/follow-redirects@1.15.4_debug@4.3.4/node_modules/follow-redirects/index.js:37:24)\n    at ClientRequest.emit (node:events:517:28)\n    at ClientRequest.emit (node:domain:489:12)\n    at Socket.socketErrorListener (node:_http_client:501:9)\n    at Socket.emit (node:events:517:28)\n    at Socket.emit (node:domain:489:12)","timestamp":"2024-02-26T13:24:20.441Z"}

After investigating it, I found out that when activeDataFeed function returns empty string for signedApiUrls (it's when signedApiUrls isn't set in the AirseekerRegistry), airseeker uses empty string as base URL and create a complete signed API URL (including airnode address) to make request, see here. It creates URLs like /airnodeAddress and make request for them.
Besides that, as I understand from here, airseeker expects to see complete signed API URLs for each airnode in the config file like following:
{
"signedApiUrls": [
"https://signed-api.stage.nodary.io/public/0x5791Fb78D4e37A9D0f0003199D1AE1A8C04C8d89",
"https://signed-api.stage.nodary.io/public/0xc52EeA00154B4fF1EbbF8Ba39FDe37F1AC3B9Fd4",
"https://signed-api.stage.nodary.io/public/0x8676eA8B6Ebe5b8FBbc25FF55192bADf39D7D61b"]
}

This setup complicates airseeker deployments because each time a new partner provider (airnode) is added, all airseekers must be redeployed. We only want to re-deploy airseekers when we add/remove chains.
My suggestion here:

Behave signed-api URLs in configuration same with the ones on the contracts. So if we expect to read URL without airnode address in the contract, we should also expect to read URL without airnode address in the configuration file
Detect empty URLs coming from the contract and don't include them into URLs to be called

airseeker uses empty string as base URL and create a complete signed API URL (including airnode address) to make request

Mhh, are you sure? We are decoding the result here so I'd expect it to be an empty array.

Besides that, as I understand from here, airseeker expects to see complete signed API URLs for each airnode in the config

Yeah, you are right. This is a bug - it should be the base URL and we should append the Airnode address as needed. However, there is related issue about the auth tokens. Each (full URL) Signed API can have different auth token which we won't have unless we specify it in Airseeker (and always redeploy). The only workaround I see is to change the authentication scheme (e.g. signature based by the Airseeker wallet which would also be compatible with Auctioneer).

Mhh, are you sure? We are decoding the result here so I'd expect it to be an empty array.

Yes, it's array with size of beacons like = ['', '', '', '', '',..., '']

Yeah, you are right. This is a bug - it should be the base URL and we should append the Airnode address as needed. However, there is related issue about the auth tokens. Each (full URL) Signed API can have different auth token which we won't have unless we specify it in Airseeker (and always redeploy). The only workaround I see is to change the authentication scheme (e.g. signature based by the Airseeker wallet which would also be compatible with Auctioneer).

Shouldn't airseeker always utilize public endpoint of signed api?

Yes, it's array with size of beacons like = ['', '', '', '', '',..., '']

Oh, I see 🤦 . I was too thoughtless when I looked at the function. The e2e tests I used always populate the array, so I didn't notice. Anyway, completely agree with filtering out empty URLs.

Shouldn't airseeker always utilize public endpoint of signed api?

Actually, I am not sure. I remember discussing this a while ago, but forgot on the outcome. If only public endpoint is used then we don't care about the auth token issue.

Behave signed-api URLs in configuration same with the ones on the contracts. So if we expect to read URL without airnode address in the contract, we should also expect to read URL without airnode address in the configuration file

Provided the config scheme is good as is, then I agree with this as well.

You can disregard my point about auth tokens. Airseeker will always use the public endpoint.


feat: faster and simpler DNS checks, better ip-address determination

shrinks and streamlines the DNS-checking code to use 'execnet' which allows to execute python functions remotely.

DNS-checking is much faster as the ssh-connection is re-used, and there are much fewer roundtrips.

Instead of parsing the output of the "ip" command this PR uses a standard python "socket" pattern to determine IPV4 and IPV6 addresses.  This is likely less dependent on the target enviroment, so could also work better with non-debian systems.

DNS checking now gives progress indicators (use -v|--verbose to see commands executed remotely)

Can we remove checking PTR records completely? See #336 Then there is no need to determine IP address which is not reliable anyway even with the new method: the host may not know its external IP address, this address may change in the future etc.

we could probably remove the ip-address detection but
it's useful to at least check that the IP addresses A/AAAA records have reverse PTR records.
I'd suggest to do this in a separate PR though.


error cannot read properties of indenfined (reading chat)

please help me, I encountered this kind of error, I am using node.js and using wapi.js file?
please help mee :"((

?

#1191


Customizing #show

Hi,
I would have very basic changes on a #show. I could recode the whole view, but is it possible to cal activeadmin internals for some of the blocks? Like 'Comments'? Or use activeadmin internals to show only some of the attributes (in MyModel block)?
Thanks a lot

Greg, not sure that you've given us everything we need here. Would be great if it were documented how to use the standard attributes_table but customizing our own list of attributes (so we can control what's show and in what order) but using your helpers.


builder error: 'cimgui.h' not found

When I am trying to run the program vig/examples/mainig/mainig.v then I am getting the error message "builder error: 'cimgui.h' not found" although I have installed imgui as well as cimgui. The file cimgui.h is in the same folder in which mainig.v is saved. So what could be error? How to specify the path of the header file cimgui.h while running program of https://github.com/nsauzede/vig? Please help me so that I can start using this package.

Yes I am able to install v properly. Even when I am running command v install nsauzede.vsdl2 as per instruction given on the page then it is also installing properly and I am able to run the example with command v run ~/.vmodules/nsauzede/vsdl2/examples/tvintris/tvintris.v. Similarly when I executed the command v install nsauzede.vig then it also worked. However when I am trying to run the command v run ~/.vmodules/nsauzede/vig/examples/mainig/mainig.v then I am getting following error message:
builder error: 'cimgui.h' not found
I think something is wrong with cimgui installation. Can you please give me a hint?

Thanks for reply. When I am running the command make mrproper all then I am getting following error:
cc -std=gnu99 -w -I ./thirdparty/stdatomic/nix -o v1.exe ./vc/v.c -lm -lpthread ./v1.exe -no-parallel -o v2.exe cmd/v cmd/v/v.v:6:1: builder error: bad module definition: cmd/v/v.v imports module "help" but /home/ccpplinux/Nextcloud/VLang/vig/examples/mainig/sdl2test/v/cmd/v/help/help.v is defined as module examples.mainig.sdl2test.v.cmd.v.help 4 | module main 5 | 6 | import help | ~~~~~~~~~~~ 7 | import os 8 | import term make[1]: *** [GNUmakefile:94: all] Error 1 make[1]: Leaving directory '/home/ccpplinux/Nextcloud/VLang/vig/examples/mainig/sdl2test/v' /home/ccpplinux/Nextcloud/VLang/vig/examples/mainig/sdl2test touch nsauzede/vsdl2/v.mod ./v/v install nsauzede.vsdl2 make: ./v/v: No such file or directory make: *** [Makefile:150: /home/ccpplinux/.vmodules/nsauzede/vsdl2/v.mod] Error 127

I think I now understand what happens : you seem to have cloned sdl2test within vig, which is itself a V module.
Don't do that. Because that seems to interfere with V compiler modules handling (unrelated to my actual vig project)
Please rather do this:
$ cd /home/ccpplinux/Nextcloud/VLang
$ git clone https://github.com/nsauzede/sdl2test.git
$ cd sdl2test ; make mrproper all ; ./mainig_v.exe

=> This is to ensure that my current vig project is working within sdl2test (which is the primarily dev setup), somewhere else than in my environment.
Then, I could try and figure out what's wrong with other use cases of vig.
Thanks.

Thanks. Now it is working. I am able to run and view the output of the program mainig_v.exe. Thanks a lot.

Great.
Next, I have to investigate more why vig doesn't work when used standalone..

I have been able to get closer, but it appears that the V compiler has indeed changed and will not compile vig.v. See my issue on V's GitHub. I changed the header paths to the full path. I also switched #include <SDL.h> to  #include <SDL/SDL.h> which fixed the import issues.

i should note that I am also on Manjaro Linux too because it's the best ; ).

Hello all.
I pushed some commit to try and fix the issue
Please test & report

                                                                                Forgot to mention that you should git clone vig and try as below:

$ mkdir -p ~/.vmodules/nsauzede
$ cd ~/.vmodules/nsauzede
$ git clone https://github.com/nsauzede/vig
$ cd vig
$ make
$ v run examples/mainig/mainig.v

Note that separate nsauzede/vsdl2 should first be installed in .vmodules, too..

Sorry to burst your bubble, but I am still having issues.
(Use v -cg to print the entire error message)
builder error:
C error. This should never happen.
This is a compiler bug, please report it using v bug file.v.
https://github.com/vlang/v/issues/new/choose
You can also use #help on Discord: https://discord.gg/vlang
`
Thanks!
(sdl2 is installed)

Wow, that did not format correctly 😂, sorry!

@Jah-On can you please perform the following and put back the result ?
$ pwd
$ ls -l

Just to see where you run your command from and whether you have obtained the proper files..

@nsauzede
pwd:
/home/jah-on/.vmodules/nsauzede/vig
ls -l:
total 2424 -rw-r--r-- 1 jah-on jah-on    1072 Nov 13 23:59 LICENSE -rw-r--r-- 1 jah-on jah-on    1691 Nov 13 23:59 Makefile -rw-r--r-- 1 jah-on jah-on     820 Nov 13 23:59 README.md drwxr-xr-x 1 jah-on jah-on     280 Nov 13 23:59 cimgui -rw-r--r-- 1 jah-on jah-on  169990 Nov 13 23:59 cimgui.h -rwxr-xr-x 1 jah-on jah-on 1660576 Nov 13 23:59 cimgui.so drwxr-xr-x 1 jah-on jah-on      12 Nov 13 23:59 examples -rw-r--r-- 1 jah-on jah-on    3443 Nov 13 23:59 imgui_impl_opengl3.h -rw-r--r-- 1 jah-on jah-on   30736 Nov 13 23:59 imgui_impl_opengl3.o -rw-r--r-- 1 jah-on jah-on    1677 Nov 13 23:59 imgui_impl_sdl.h -rw-r--r-- 1 jah-on jah-on   15456 Nov 13 23:59 imgui_impl_sdl.o -rw-r--r-- 1 jah-on jah-on  563286 Nov 13 23:59 libcimgui.a -rw-r--r-- 1 jah-on jah-on      58 Nov 13 23:59 v.mod -rw-r--r-- 1 jah-on jah-on    4909 Nov 13 23:59 vig.v

hmm
now this is strange;
I've got the same kind of results on my Ubuntu 21.10...
I'm sorry but I'm afraid I have no clues...

hmm
now this is strange;
I've got the same kind of results on my Ubuntu 21.10...
I'm sorry but I'm afraid I have no clues...

I have an issue on the V repository for cimgui. You should post your findings there and maybe they can tweak the compiler.

Solved my issue
I had to change
cmake .. && make to cmake .. && cmake --build . --target all


Need a way to customize the route

Right now the default route is always used for Quay. Which makes for a rather "funny" URL. For a quay-enterprise installation of name 'quay' it is quay-quay-quay-enterprise.apps.cluster-shared.shared.example.opentlc.com.
It would be good to just specify the route name as "quay.apps.cluster-shared.shared.example.opentlc.com".
So instead of service-project (which is quay-quay - quay-enterprise) just 'quay' would be enough.
Add a field to the CR to change this.

This turns out to be a docs bug. spec.quay.routeHost can be set to the full URL including wildcard domain.

@wkulhanek docs describing this option will be added as part of the next milestone


about the organization of Cars196

hello, I want to reproduce your result on Cars196, but when i download the dataset from the official website, the organization of the dataset doesn't like the one you provide in the readme. Could you please provide a hint or the code of how to handle these data. Thanks :)

That seems weird that it's not the case, but you can go to  the successor of this repo, https://github.com/Confusezius/Revisiting_Deep_Metric_Learning_PyTorch, which has Dropbox links to download each dataset in exactly the layout required. Hope that helps :).


on day view, popup only show on the side invisible

by using allowedPlacements: ['top', 'bottom', 'right'],


Merge remote-tracking branch 'ak-upstream/2.4' into 2.4

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

The merge was clean. The Jenkins job fails downstream with secretregistry compilation error. The PR to fix that is here: https://github.com/confluentinc/secret-registry/pull/25

So downstream build failed with expected error:
[INFO:2020-02-13 01:46:21,171]: [INFO] -------------------------------------------------------------
[INFO:2020-02-13 01:46:21,171]: [ERROR] COMPILATION ERROR :
[INFO:2020-02-13 01:46:21,172]: [INFO] -------------------------------------------------------------
[INFO:2020-02-13 01:46:21,172]: [ERROR] /home/****/workspace/test-cp-downstream-builds/projects/secret-registry/core/src/main/java/io/confluent/kafka/secretregistry/masterelector/kafka/KafkaGroupMasterElector.java:[129,20] method bootstrap in class org.apache.kafka.clients.Metadata cannot be applied to given types;
[INFO:2020-02-13 01:46:21,172]: required: java.util.List<java.net.InetSocketAddress>
[INFO:2020-02-13 01:46:21,172]: found: java.util.List<java.net.InetSocketAddress>,long
[INFO:2020-02-13 01:46:21,172]: reason: actual and formal argument lists differ in length
[INFO:2020-02-13 01:46:21,172]: [INFO] 1 error
[INFO:2020-02-13 01:46:21,172]: [INFO] -------------------------------------------------------------

I will merge this PR as soon as I get LGTM to ce-kafka PR: https://github.com/confluentinc/ce-kafka/pull/1203 (which allows me to quickly pull this pr in ce-kafka, followed by secretregistry pr)


Added partial support of const expr evaluation on enums.

It does not yet support payloads.

@swift-ci please test tensorflow

Thanks for the review! I'll land this PR as is, and work on the next PR to address the various suggested improvements along with getting payload to work.

SGTM thanks!


returns type must defined in message.proto

May I ask why the object in the file cannot be referenced in the return value of go-zero rpc, which can be compiled successfully with the original command, why set this restriction in goctl
Generate code using goctl:

Generate code using protoc:

goctl's interception of external references to proto weakens proto's capabilities, which I don't think should be done.

Pls see https://go-zero.dev/docs/tutorials/proto/faq#2-为什么使用-goctl-生成-grpc-代码时-proto-不支持使用包外-proto-和-service

Pls see https://go-zero.dev/docs/tutorials/proto/faq#2-为什么使用-goctl-生成-grpc-代码时-proto-不支持使用包外-proto-和-service


buttonBarItemSpec fatal error: unexpectedly found nil while unwrapping an Optional

I'm having the same issue , how did you solve this ?


Future exception was never retrieved future: <Future finished exception=ConnectionClosedError(None, None, None)> websockets.exceptions.ConnectionClosedError: no close frame received or sent

No handling for ConnectionClosedError. DisconnectEvent not triggered.
...
Future exception was never retrieved
future: <Future finished exception=ConnectionClosedError(None, None, None)>
websockets.exceptions.ConnectionClosedError: no close frame received or sent
Future exception was never retrieved
future: <Future finished exception=ConnectionClosedError(None, None, None)>
websockets.exceptions.ConnectionClosedError: no close frame received or sent
Future exception was never retrieved
future: <Future finished exception=ConnectionClosedError(None, None, None)>
websockets.exceptions.ConnectionClosedError: no close frame received or sent
Future exception was never retrieved
future: <Future finished exception=ConnectionClosedError(None, None, None)>
websockets.exceptions.ConnectionClosedError: no close frame received or sent
Future exception was never retrieved
future: <Future finished exception=ConnectionClosedError(None, None, None)>
websockets.exceptions.ConnectionClosedError: no close frame received or sent
Future exception was never retrieved
future: <Future finished exception=ConnectionClosedError(None, None, None)>
websockets.exceptions.ConnectionClosedError: no close frame received or sent
Future exception was never retrieved
future: <Future finished exception=ConnectionClosedError(None, None, None)>
websockets.exceptions.ConnectionClosedError: no close frame received or sent
Future exception was never retrieved
future: <Future finished exception=ConnectionClosedError(None, None, None)>
websockets.exceptions.ConnectionClosedError: no close frame received or sent

Package Version
My version of TikTokLive is 6.0.8.
Operating System
MacOS

same issue here

Fixed in the latest version. Re-open if occurs again.


[HxGrid] Add sticky header and scrollbar

Is it possible in the HxGrid to fix the header at the top when you set a fixed height to scroll as in the following image?

Basically I would like what you have demonstrated here but applied to the table tboby
https://havit.blazor.eu/components/HxGrid#InfiniteScroll
Happy Holidays

@leonardopugliese I have added examples to the TestApp project with sticky header and first column pivot in https://github.com/havit/Havit.Blazor/commit/f90c5352de9959148b62b57545038661011c14a7
If it proves to be a stable across the browsers, we will add it to the docs in the future.

Btw. @alexax578 and @Arnnes you could be interested in this. The solution proposed here is way more elegant than the one we have in 176.BT2 projects.

It seems to work correctly as desired. The mode WITHOUT InfiniteScroll but with pagination also works well (I wanted this mode too).

Closing for now. The first demo of sticky header will be on HxListLayout that wraps HxGrid in the next release.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 992 ms

Resolved: Bofrost.at performance has improved in b7fed07.


FlatList does not accept (number) zero or empty string items

When using plain numbers or strings as items (in the "data" list property), rendering FlatList results in an error when encountering number 0 or string ''.  The error is "No item for index X" where X is the index of the 0 or '' in the list.  Examples where the error is "No item for index 2" are data={[-2, -1, 0, 1, 2]} and data={['a', 'b', '']}.

I have encountered this situation
Now the solution is to turn Number to String
<Text>{`${count}`}</Text>

Dupe of #12905. Try the workaround above.

@Lizhooh's solution didn't work for me as I had a data source of empty URIs for Images, and received the 'No item for index X'
How I fixed it was for my images to just pass in a string 'blank' and check like so.
source={(imgSource != 'blank') ? {uri: imgSource} : undefined}


hide_sidebar and landscape not working

I checked this issue today while testing on a tablet in landscape mode, and the side menu doesn't disappear, it only works in portrait mode. I also tested it on my smartphone, and the problem persists.

Hi @joaopedros2,
Please, follow the template to open an issue and give all the information that is required to debug an issue further.
Regards

Hi @joaopedros2,
Please, update your issue providing all the details required in the template so we can try to debug it further.
Regards

Hi @joaopedros2,
It seems that you already solved your issue or it doesn‘t interest you anymore.
In any case, closing this issue.
Regards


Add server side rendering?

We use SSR pretty religiously. wondering if this will eventually support SSR - realize the complexity it would add to this project

Thanks for asking—probably not.
From the list of limitations, this is likely the least likely feature to be added.
It adds a ton of configuration headaches, and most beginners don’t need it.
(I do realize some projects need it—you might have better luck with alternative projects or starter kits.)

Hi all, I can highly recommend Razzle along with my Razzle Material UI Styled Example for those who need Server Side Rendering right now and the easy way.


Toolbar icons disappear

[x] I have verified the issue exists on the latest version
[x] I am able to reproduce it

Version used: 1.8.0
Android version: 6.0.1
I use "TapTargetSequence" with 2 "TapTarget.forBounds", in this 2 i have the same icon of my toolbar.
When the TapTarget finish or is canceled my toolbar icons disappear
Code:
`    private void firstTime() {
    Display display = getActivity().getWindowManager().getDefaultDisplay();
    Point size = new Point();
    display.getSize(size);
    Rect rectInfo = new Rect((size.x - 100), 0, (size.x - 50), (size.y / 6));
    Rect rectUpdate = new Rect((size.x - 200), 0, (size.x - 150), (size.y / 6));

    new TapTargetSequence(getActivity())
            .targets(
                    TapTarget.forBounds(
                            rectInfo, "Legenda", "Identifica as cores dos materiais")
                            .outerCircleColor(R.color.azul)
                            .targetCircleColor(R.color.branco)
                            .titleTextSize(30)
                            .descriptionTextSize(15)
                            .textColor(R.color.preto)
                            .textTypeface(Typeface.SANS_SERIF)
                            .drawShadow(true)
                            .cancelable(false)
                            .tintTarget(false)
                            .transparentTarget(false)
                            .icon(getResources().getDrawable(R.drawable.ic_help_outline_black_24dp))
                            .targetRadius(60).id(1),

                    TapTarget.forBounds(
                            rectUpdate, "Recarregar", "Refazer Validação de Materiais")
                            .outerCircleColor(R.color.azul)
                            .targetCircleColor(R.color.branco)
                            .titleTextSize(30)
                            .descriptionTextSize(15)
                            .textColor(R.color.preto)
                            .textTypeface(Typeface.SANS_SERIF)
                            .drawShadow(true)
                            .cancelable(false)
                            .tintTarget(false)
                            .transparentTarget(false)
                            .icon(getResources().getDrawable(R.drawable.reload))
                            .targetRadius(60).id(2)
            ).start();
}`

Screen after "TapTargetSequence" appear:

Screen without "TapTargetSequence" appear:

What wrong did i do?

How are you adding those buttons to your toolbar? Dump the layout hierarchy, are they still there?

The buttons still there, you can touch and interacts with they, but the image disappear.
The buttons are in a xml that i inflate in "onCreateOptionsMenu" method.

I tried replicating your code, however I am not able to replicate the issue...
Did you try using TapTarget.forToolbarMenuItem(...) ?
If not, can you provide a layout + code that reproduces the issue?

May i send you an email with the requested?
I prefer send you an email than write here

Sure you can email me via my github username at Gmail

Closing, please open another issue if you have more information.

I have the same problem.
I create the menu via xml.

I "was" using:
icon(...)
transparentTarget(false)
Changed it to:
NO icon(...) and transparentTarget(true) -> and now it's working.
I think it's the transparentTarget(false)

@xiphirx Could you have a look here? Thanks.


Readme is not up to date

I found readme is outdated for spree version 3.x

@akshatpaul any specific issues?

@damianlegawiec Getting too many errors to set up with Spree 3.x and rails 5. But closed the issue coz stack overflow would be better place for that.
Apologies for raising it here.


Allow Unattended Extracts to Specify File and Config Names

Pull Request (PR) description
Adds the -FileName parameter to the Export-O365Configuration cmdlet.
This Pull Request (PR) fixes the following issues

Fixes #216

This change is

Codecov Report

Merging #229 into Dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              Dev     #229   +/-   ##
=======================================
  Coverage   89.38%   89.38%
=======================================
  Files          55       55
  Lines        4976     4976
  Branches        8        8
=======================================
  Hits         4448     4448
  Misses        520      520
  Partials        8        8

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 2b15eae...a774769. Read the comment docs.


No eye paramaters?

Am I missing something, or are there no entries for SRanipal's eyetracking blendshape/setup? this tool is really nice to work with, so it would be nice if I could use this tool to generate both face and eyetracking so everything is kept consistent. Either way, really enjoying the tool. Thank you!
EDIT: Closing this as I saw Eyetracking is in your todo. I also Just had a look at VRCFTGenerator_Values.cs and I might try to add some basic ones for now for my own use. I could submit a PR, but I don't know if I'll be tackling the more complex issues you're looking for

Yeah, to get this written down: I don't personally have an eye tracking setup, so it's a) harder for me to test the results and b) harder for me to justify the development effort. Many of the VRCFT eye parameters have special handling which I'd have to write logic for, so it's just work I haven't been able to justify spending time on yet.
Native eye tracking is also coming to VRChat, so that'll alleviate some of the effort. I might be able to justify it at that point, since it won't require animating the eye bones.


Limit Testing

Please comment any bugs you find and how you plan to fix them/if you want help having them fixed
Try to actively write code that works if unexpected values are entered, note in commits if limits have been tested.

Small work has been done to only accept valid inputs for wavefunction form, see commit 551cda0


link/unlink not showing

Using nuxt, I'm not seeing the link/unlink option in the inline toolbar

Any ideas on what I'm doing wrong?  I'm using the example nuxt setup

Thank you for your issue. I'll try to this.


Dice Rolling Simulator Using Python

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I would like to do this project

Please assign this to me.

HEY ! @Ishaan28malik  I have done this kind of project earlier so i would be able to do this statement's code easily so please assign me to this issue .

I have Raised a PR Please accept @Ishaan28malik

Hello, please assign this to me.

Please assign me this issue under hacktoberfest 2023.


Decompiler can produce incorrect syntax with double negations

Describe the bug
If an operand is negated twice, the decompiler can display this as --var, which is incorrect.
To Reproduce
eg the following MIPS code:
      80044dcc C7 A2 00 7C  lwc1      f2,0x7c(sp)
      80044dd0 46 00 04 87  neg.S     f18,f0
      80044dd4 44 06 A0 00  mfc1      a2,f20
      80044dd8 46 00 92 07  neg.S     f8,f18
      80044ddc 46 00 10 87  neg.S     f2,f2
      80044de0 44 05 40 00  mfc1      a1,f8
      80044de4 44 07 10 00  mfc1      a3,f2
      80044de8 27 AE 00 DC  addiu     t6,sp,0xdc
      80044dec 27 AF 00 D8  addiu     t7,sp,0xd8
      80044df0 27 B9 00 D4  addiu     t9,sp,0xd4
      80044df4 E7 A0 00 78  swc1      f0,0x78(sp)
      80044df8 AF B9 00 18  sw        t9,0x18(sp)
      80044dfc AF AF 00 14  sw        t7,0x14(sp)
      80044e00 AF AE 00 10  sw        t6,0x10(sp)
      80044e04 E7 A2 00 48  swc1      f2,0x48(sp)
      80044e08 0C 00 53 B4  jal       FUN_80014ed0                      void FUN_80014ed0(float *
                                                                       param_1, float param_2, float
                                                                       param_3, float param_4)
      80044e0c 27 A4 00 84  _addiu    a0,sp,0x84

produces the line: FUN_80014ed0(afStack92,--fStack104,fVar13,fVar11);
--fStack104, while syntactically valid, does not have the intended effect.
Expected behavior
Ideally, the double-negation should be omitted entirely (ie fStack104). If not, parentheses should be added (-(-fStack104)) to ensure the code is correct.
Environment (please complete the following information):

OS: Artix x64
Java Version: 15.0.2
Ghidra Version: 9.2.2

Someone should commit this and this. These two must fix the bug using parentheses as it now prints -(-in_f0) for me.


Line#category shouldn't return None by default

I think that this is maybe intended to be overridden on a per-app basis, but it's a bit confusing to try to get the Category for a Line and see None come back.
https://github.com/django-oscar/django-oscar/blob/master/src/oscar/apps/order/abstract_models.py#L545-L551

The comment says it's used by Google analytics, so I wonder if has to be a Google product category: https://support.google.com/merchants/answer/160081?hl=en
Otherwise, here's a basic implementation:
@property
def category(self):
    if self.product:
        return self.product.get_categories()[0]

We've solved this in another way internally, but I felt like this may be an issue in the core codebase and wanted to contribute back to see what everyone else thinks.

I've just removed this property completely as it isn't doing anything.


Remove unnecessary @show

This PR removes unnecessary @show in solve() for SimpleNewtonRaphson, which was probably introduced during debugging.  Under certain conditions, this @show prints out x and fx, contaminating stdout.

Thanks!


Issues about channel (version?) property in rust-toolchain

Environment

proot-distro arch linux in termux
CPU structure: aarch64

How to recreate

Use the template on https://github.com/emilk/eframe_template (master, 96e4908b1c7ba2da6c0527589e5f6e3060f93ee9)
Clone the repo
Use cargo command, then the following message:

error: the 'cargo' binary, normally provided by the 'cargo' component, is not applicable to the '1.76.0-aarch64-unknown-linux-gnu' toolchain

Stack backtrace:
   0: <unknown>
   1: <unknown>
   2: <unknown>
   3: <unknown>
   4: <unknown>
   5: <unknown>
   6: <unknown>
   7: <unknown>
   8: __libc_start_main
   9: <unknown>

How to solve
Just edit file rust-toolchain :
[toolchain]
-channel = "1.76.0"
+channel = "1.76"

Then cargo can run successfully.
Description
Although it's just a minor difference from before and after the fix, the problem can cause the whole project not working.
I don't know if this problem only happens on my device, so I'm not sure if this counts as a bug or not.
Could someone provide further information?

Thanks!


🐞[Bug]:  Fixing Complete CSS of Delivery page

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug

fixing CSS in mobile view
Expected behavior
making website responsive
Add ScreenShots
No response
What browsers are you seeing the problem on?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@varshith257 please assign me

@varshith257 bro please assign me issue thanku


Add NixOS support

Changes

we have shown how we install on nix-darwin now we want to show the same process for NixOS

Things to do

[x] make distro dynamic so user only needs to provide archetype (ie. user provides "x86_64-linux" and it can generate systems for both darwinConfigurations and nixosConfigurations

`#!/bin/bash
architecture=$(uname -m)
case $architecture in
x86_64)
echo "64-bit"
;;
i86)
echo "32-bit"
;;
arm)
echo "ARM"
;;
*)
echo "Unknown architecture: $architecture"
;;
esac
read -p "Enter your username: " username
echo "Your username is: $username"
`

packages.${system}.user =  pkgs.writeShellScript "changeUser" ''
read -p "Enter your username: " username
echo "Your username is: $username"
sed ....username
'';
packages.${system}.default = pkgs.writeShelScriptBin "install" ''
 ${self.packages.${system}.user} && \
 uname thing ..
''

something along this line.

You could also write a function which accept some args like system
      mkDarwinConfig = system: inputs.darwin.lib.darwinSystem {
        # system: supports only 1 system
        inherit system;
        # modules: allows for reusable code
        modules = [
        {
          services.nix-daemon.enable = true;
          users.users.${username}.home = "/Users/${username}";
        }
        system-config

          inputs.home-manager.darwinModules.home-manager
          {
            # add home-manager settings here
            home-manager.useGlobalPkgs = true;
            home-manager.useUserPackages = true;
            home-manager.users." ${username}" = home-manager-config;
          }
          # add more nix modules here
        ];
      };

We wanted to try and make the system value be as easy to setup as possible without making it too "magical". Before you'd need both the username and system set before you could switch to a profile. Now you set the username only in flake.nix and then use the desired profile for your platform such as below:
darwin-rebuild check --flake ".#darwin-x86_64"
nixos-rebuild check --flake ".#nixos-x86_64"


Remove Text component dependency from List component

We want our components to be easy to copy and paste into the user's codebase and work with as few dependencies as possible. Right now the List component has a dependency on another component from our library Text.
It would be nice to remove that dependency, so we don't import anything from @mailingui/components in the List component, but keep the functionality and design the same as it is now. Afaik, List is the only component that is dependent on another component from our library so IMO it would be nice for all of our components to be independent of each other.
https://github.com/webscopeio/mailingui-web/blob/5e11969637a7194401a87f042bce644c7ad60802/src/mailingui/components/list/List.tsx#L11

Here's to get a bit familiar with the library. @DanielPuchala target this after the other two issues assigned or if time allows.


clear out downgrade lease on failed team op

got a repro for the original bug without this fix:

	member_test.go:696: permission is slated to be deleted (error 289)
	test_logger.go:54: TEST FAILED: TestOnlyOwnerLeaveThenUpgradeFriend

the fix is to "cancel" the downgrade lease come rain or sine, so that the next operation won't fail with the lease outstanding

This won't pass CI until the new keybase/keybase bubbles up.


OS.shell_open does not build newlines correctly

Godot version
3.4.stable
System information
Linux
Issue description
I use shell_open to open an email. The body for the email is inline in the code:
var _ignore = OS.shell_open("mailto:?body=First line\r\nSecond line")

I would expect this to make a newline on all platforms. However, when i compile to HTML5 and load the website in android or on MacOS, the two lines are together without separator.
Steps to reproduce
Compile an HTML5 application that has the line
var _ignore = OS.shell_open("mailto:?body=First line\r\nSecond line")

on a linux machine, then load it in a webbrowser.
Minimal reproduction project
No response

correction: i realized newlines are always lost when loading the game in the browser.
however, i found the workaround that %0A%0A ends up generating one newline. if i just to %0A, i get no newline. when starting the game from the IDE, the %0A does not show up in the email, so seems to work fine. so something seems wrong here, but at least there is a workaround.


Pressing Enter while changing title of timer entry deletes title entirely

This is a corollary to #2922. That issue is for running timers. This issue is for existing entries.
💻 Environment

Platform: macOS
OS Version: 10.14.4 (18E226)
Toggl Version: 7.4.373
🐞 Actual behavior

Click timer entry
In edit window, change title
While title input is still focused, press Enter.
Edit window closes, title gets deleted

💯 Expected behavior
Edit window closes, title is committed
📦 Additional info
You can see the number of (no description) timers increment from 5 to 6 in this:

Can confirm that this was fixed alongside #2922


style: re-enable sha256 checks for bottle blocks

Follow up PR to #10450

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?
[ ] Have you successfully run brew man locally and committed any changes?

Review period will end on 2021-01-29 at 18:35:09 UTC.

Review period will end on 2021-01-29 at 18:35:09 UTC.

Tests failing; I guess this needs #10452

Tests failing; I guess this needs #10452


ref(*): change RC name to be router inside the deis namespace

Required for https://github.com/deis/charts/pull/252

Apart from one non-blocking nit, LGTM.

I think this cannot pass CI until https://github.com/deis/charts/pull/252 passes-- and unfortunately, vice versa.

@helgi im guessing we aren't doing this anymore?

Punting it to later - going to see if it is worth updating env vars in all repos to function with the chart updates

@helgi, should this be closed, especially now that any remaining RC's are vestigial?

@helgi is this still in progress or should we close it?

Closing - doesn't seem feasible to pull off anytime soon


Problem running Dockerized protect / where to find SSE-C key for downloading reference data?

I'm having a trouble running protect, and figuring out what exactly is failing, as the output is 4655 lines long.. I'm most probably just missing some parameters, but I'm not sure which ones exactly.
Starting from line 1143 I start getting errors related to Amazon S3 authentication:
0091ac8c0f62 2021-06-16 13:00:23,448 MainThread WARNING toil.leader: H/2/jobmrdMHX    RuntimeError: s3am failed with (boto.exception.NoAuthHandlerFound: No handler was ready to authenticate. 1 handlers were checked. ['HmacAuthV1Handler'] Check your credentials) while downloading (S3://protect-data/hg38_references/gencode.v25.pc_transcripts.fa.tar.gz) 0091ac8c0f62 2021-06-16 13:00:23,448 MainThread WARNING toil.leader: H/2/jobmrdMHX    ERROR:toil.worker:Exiting the worker because of a failed job on host 0091ac8c0f62
I'm assuming I should use the --sse-key and --sse-key-is-master parameters to set up the SSE-C key in order to download the reference genome files, but I have trouble finding where could I find the actual key files. Also, I'm not sure if this is the main issue, or are the other issues as well.
Any help is highly appreciated!
Here is the command I'm using to run protect
docker run -v /var/run/docker.sock:/var/run/docker.sock -v /data:/data/ quay.io/ucsc_cgl/protect:2.5.6-1.13.0 --sample-name sample_name --tumor-dna /data/dna/tumor_dna_sample.merged_1.fq.gz --tumor-dna2 /data/dna/tumor_dna_sample.merged_2.fq.gz --normal-dna  /data/dna/normal_dna_sample.merged_1.fq.gz --normal-dna2  /data/dna/normal_dna_sample.merged_2.fq.gz --tumor-rna /data/rna/tumor_rna_sample_1.fq.gz --tumor-rna2 /data/rna/tumor_rna_sample_2.fq.gz --reference-build hg38 --tumor-type PRAD --work-mount /data/protect 2> error_log_2021-06-16.txt
Attached is the full output log
error_log_2021-06-16.txt

@erichweiler Unfortunately I don't know where the bucket is. But @hbeale assures me that the real problem here is that you need to have AWS account credentials available, and you need to use requester-pays mode to download, which is behind the --request-payer flag in aws. I'm not sure if s3am needs any special flag for that; it might accept needing to pay for downloads by default, if you give it AWS credentials to use.


The sample test cases provided in the docs are not isolated

test_comedic_timing fails unless it runs after test_dramatic_moment.
$ pytest -k test_comedic_timing

# Sample output:
>       assert_that(value, self.resolution, reason)
E       AssertionError:
E       Expected: 'laughter'
E            but: was 'tense'

Ideally, connect_to_audience() should be mocked on every test case; something similar to:
@patch('pollster.connect_to_audience')
def test_comedic_timing(pollsterMock, Cameron: AnActor, Polly: AnActor) -> None:
    pollsterMock.poll_mood.top_mood = 'laughter'
    ...

Whoa, nice catch, and thanks for the issue! I'll be able to address this in the small docs update i'm going to do, unless someone else gets to it first. :D

I think it should be fixed now, see this change.
I don't think we mention the conftest.py file in the docs, so i think this is the only place that needed to be updated. Let me know if we missed a spot @ramoscav-nokia, and thanks again for submitting this issue!

Seems like this is probably fixed. If you disagree, please reopen this!

Thanks for taking the time addressing this issue, @perrygoy.
I'd like to suggest a slightly different approach where mocks are defined within the tests so that it is closer to a real world scenario.
This following example is closer to the ideal solution I requested when first opening the issue and highlights how one could  configure the mock per test case; this way the pytest hook in conftest.py is no longer needed.
# features/test_mood.py
...
from pollster import laughter_packet, tense_packet, connect_to_audience
from functools import wraps

def mock_packet(packet):
    def decorator_mock_packet(func):
        @wraps(func)
        def wrapper_func(*args, **kwargs):
            connect_to_audience().poll_mood.side_effect = [packet]
            func(*args, **kwargs)
        return wrapper_func
    return decorator_mock_packet

@mock_packet(tense_packet)
def test_dramatic_moment(Cameron: AnActor, Polly: AnActor) -> None:
    """We can use the camera to create dramatic tension."""
    Cameron.has_ordered_cleanup_tasks(StopRecording())
    ...

@mock_packet(laughter_packet)
def test_comedic_timing(Cameron: AnActor, Polly: AnActor) -> None:
    """We can use the camera to make funny moments."""
    Cameron.has_independent_cleanup_tasks(StopRecording())
    ...

When I first opened this issue I was unfamiliar with how mocking works in python, but thanks to your fix I was able to figure out how to decorate these tests and provide the mocks directly.

That looks pretty slick, i like it!
However, would you be open to writing your own pull request to address those changes? I'd love for your name to be attached to your suggested fix. Plus it's super cool.
If you're not comfortable with that, i can do it for you instead!

Of course @perrygoy, it'll be a pleasure. I'll be using this account for the PR.

@rcavaz actually, on second thought, the goal for the sample test there is to show how the test would look using a custom library we invented. Those mocks would not be necessary in a "real world" scenario. The weird backstage magic  in the pollster library is actually OK in my book to make the actual test file look like it's supposed to.
I do really like the decorator-style mocks, though, so i forgot the goal of that example suite. Does the backstage magic not meet your needs?

There are real world scenarios where I may want to write screenpy tests with mocks.

In the context of unit tests, mocks would not be necessary as long as the system under test can interact with such libraries within the same process, but for any out-of-process dependencies, mocks would still be a reasonable and desirable approach.
Another possible scenario is representing a library or system that doesn't exist yet with a mock, and slowly replace the mock through iterations.

For those with limited understanding of mocks in python or who are new to the Screenplay pattern will greatly benefit from such examples and should be able to adopt them faster.

Hm... i think i'm OK with it as long as there's a nice docstring on the decorator explaining that it's needed to make our fake library work. :D

Thanks again for the discussion and perspective, @ramoscav-nokia! I'm going to close this issue in favor of the work in https://github.com/ScreenPyHQ/screenpy_examples/issues/1.


Update faqs.yaml

Minor typo - "dasbhoard"

@fmigacz : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks for this typo fix @fmigacz.