Update addin-tutorial.md

Fixed typo, 'hml' to 'html'

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/add-ins/addin-tutorial.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@lindalu-MSFT  I'll let you wrap this one up since it's add-ins, but it's a great catch by @NishiokaTakeo. Thank you!


Enable Bicep recipe unit-test with fake registry server

Description
This is to enable bicep recipe unit test with the fake registry server.
Type of change

This pull request fixes a bug in Radius and has an approved issue (issue link required).

Fixes: #9827

@youngbupark - I think you linked the wrong issue, #9027 doesn't resolve to anything.

@youngbupark - I think you linked the wrong issue, #9027 doesn't resolve to anything.

I think it is related to the test being unskipped :) https://github.com/radius-project/radius/blob/main/pkg/recipes/driver/bicep_test.go#L379
https://github.com/radius-project/radius/issues/6490


xcode tag set on Ubuntu Travis CI jobs

The xcode tag is set on Ubuntu Travis CI jobs due to how the matrix is configured.  I ran into the same issue on my Armor PowerShell Module project last month while I was figuring this out.
Expected Behavior
The xcode tag should not be set for Ubuntu Trusty Travis CI jobs.
Current Behavior
The xcode tag is set for Ubuntu Trusty Travis CI jobs.  Example: https://travis-ci.org/psake/psake/builds/305600748
Possible Solution
Minor update to the .travis.yml file to be more explicit in the matrix.
Context
Job tagging can be improved to help prevent end user confusion.
Your Environment

Module version used: N/A
Operating System and PowerShell version: N/A

Fixed in #232. Thanks @tlindsay42!


Official releases

Hi,
when you bump the version up, could you also tag the commit, that will make sure github makes a release tarball from it and Debian will also detect the new version and notify me so to update the package.
Thanks!

I will do this for future releases.  (I've been adopting this for all of my repos as I make new releases, but I don't believe this package has had a release for a while.)
There's a Debian package?

Yes, there is, and I'm the culprit.
Thanks!


fix: readme section element bold font

Strongly Connected Components Algorithms title bold issue

Bold issue fix
Identation

Closing this as a duplicate of #110

@bobluppes Thanks for feedback, can you add hacktoberfest-accepted label on this PR ?


stress: refactor code a bit and fix some data races

Codecov Report

Merging #262 into master will not change coverage.
The diff coverage is n/a.

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


[transactions] Prevent NullPointerException due to Recycled object used in different threads

(If this PR fixes a github issue, please add Fixes #<xyz>.)
(or if this PR is one task of a github issue, please add Master Issue: #<xyz> to link to the master issue.)
Motivation
Prevent NullPointerException due to Recycled object used in different threads
Modifications
Add a volatile for the variable
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation
Check the box below.
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[ ] no-need-doc
(Please explain why)

[ ] doc
(If this PR contains doc changes)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Yiming Zang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Remove references to /irods/hydroshare_vault and IRODS_CACHE_ROOT

Description of the bug
The backing storage for the mount /irods/hydroshare_vault was removed from service. References to it must be removed.
Steps to reproduce the bug:

./hsctl start
See error

Expected behavior
Expected a clean Hydroshare startup
Additional information
Starting hydroshare ... error

ERROR: for hydroshare  Cannot start service hydroshare: driver failed programming external connectivity on endpoint hydroshare (ac9ded313cc51ce3a99e23496a01d747aa3378876069da3b2cae54872e3e3ee1): Bind for <IP_ADDRESS>:15001 failed: port is already allocated
No containers to start

https://github.com/search?q=repo%3Ahydroshare%2Fhydroshare %2Firods%2Fhydroshare&type=code
and:
https://github.com/search?q=repo%3Ahydroshare%2Fhydroshare IRODS_CACHE_ROOT&type=code

./beta/deploy/hydroshare-config.yaml:IRODS_CACHE_ROOT: /irods/hydroshare_vault/nginx_tmp/beta/cache/
./beta/deploy/hydroshare-ssl-nginx.conf.template:        alias IRODS_CACHE_ROOT/;
./beta/deploy/hydroshare-config.yaml.test:IRODS_CACHE_ROOT: /irods/hydroshare_vault/nginx_tmp/beta/cache/
./beta/deploy/hydroshare-ssl-nginx.conf.template_bkp110822:        alias IRODS_CACHE_ROOT/;
./beta/hs_proxy/config/hydroshare-config.sh:IRODS_CACHE_ROOT=/irods/hydroshare_vault/nginx_tmp/beta/cache/
./beta/hs_proxy/config/hydroshare-config.sh.test:IRODS_CACHE_ROOT=/irods/hydroshare_vault/nginx_tmp/beta/cache/
./beta/nginx/config/hydroshare-config.sh:IRODS_CACHE_ROOT=/irods/hydroshare_vault/nginx_tmp/beta/cache/
./beta/nginx/config/hydroshare-config.sh.test:IRODS_CACHE_ROOT=/irods/hydroshare_vault/nginx_tmp/beta/cache/
./prod/deploy/hydroshare-config.yaml:IRODS_CACHE_ROOT: /irods/hydroshare_vault/nginx_tmp/www/cache/
./prod/deploy/hydroshare-ssl-nginx.conf.template:        alias IRODS_CACHE_ROOT/;
./prod/deploy/hydroshare-ssl-nginx.conf.template.bk:        alias IRODS_CACHE_ROOT/;
./prod/deploy/hydroshare-ssl-nginx.conf.template.bak141:        alias IRODS_CACHE_ROOT/;
./prod/deploy/hydroshare-ssl-nginx.conf.template.bak112222:        alias IRODS_CACHE_ROOT/;
./prod/deploy/hydroshare-config.yaml~:IRODS_CACHE_ROOT: /irods/hydroshare_vault/nginx_tmp/www/cache/
./prod/deploy/hydroshare-config.yaml_111422:IRODS_CACHE_ROOT: /irods/hydroshare_vault/nginx_tmp/www/cache/
./prod/deploy/hydroshare-ssl-nginx.conf.template.bak010422:        alias IRODS_CACHE_ROOT/;

Also need to remove IRODS_CACHE_URI and irods-cache references.
https://github.com/search?q=repo%3Ahydroshare%2Fhydroshare IRODS_CACHE_URI&type=code
and
https://github.com/search?q=repo%3Ahydroshare%2Fhydroshare irods-cache&type=code

./beta/deploy/hydroshare-config.yaml:IRODS_CACHE_URI: irods-cache
./beta/deploy/hydroshare-ssl-nginx.conf.template:    location /IRODS_CACHE_URI/ {
./beta/hs_proxy/config/hydroshare-config.sh:IRODS_CACHE_URI=irods-cache
./beta/nginx/config/hydroshare-config.sh:IRODS_CACHE_URI=irods-cache
./prod/deploy/hydroshare-config.yaml:IRODS_CACHE_URI: irods-cache
./prod/deploy/hydroshare-ssl-nginx.conf.template:    location /IRODS_CACHE_URI/ {

released as 2.10.3


Remove -dev.infinity from SDK constraint

This is actually the reason I started the last PR – ha!

Coverage remained the same at 82.743% when pulling 6ad1905820476dd76e04876acf00a5e593916e0a on kevmoo-patch-1 into 5788c58f8ebed90c6c77e4ede45f6e442fea44cf on master.


Initial loading screen isn't pretty

The first screen that users see when loading zulip isn't pretty:

We should make a few changes here:

The content should be centered on the page, horizontally and vertically
We should include the zulip circle logo (it's only 767 bytes, so we're not really risking slowing things down much)
If the user has prefers-color-scheme: dark, we should make the page background dark (It appears that currently we make the text the color that is correct for the dark colorscheme, but don't change the background?)
De-emphasize the "If this message does not go away, please wait a couple seconds and reload the page." text, by moving it to the bottom of the page vertically.
Consider changing the text to something like "Zulip is loading..." instead of just "Loading..."
Consider adding a spinner in front of the "Zulip is loading..." or "Loading..." text

My impression is that Zulip loads quickly enough for most people that we don't need to go out and make a whole fancy animated splash screen that shows the tip of the day or whatever, but I do think some simple visual improvements would go a long way :)

Actually, giving this another thought, this logo might be better. Probably the sort of thing where trying out both and posting screenshots to #design would be a good choice.

I agree this would be really nice to fix.  I agree a logo with a loading indicator is basically all we need.

There's something to kick this off. I also like the other suggestions above which could be next step:

* De-emphasize the "If this message does not go away, please wait a couple seconds and reload the page." text, by moving it to the bottom of the page vertically.

I could give that a go, if the above change is acceptable.
But the rest, might be outside of my skillset:

* Consider adding a spinner in front of the "Zulip is loading..." or "Loading..." text

* If the user has `prefers-color-scheme: dark`, we should make the page background dark (It appears that currently we make the text the color that is correct for the dark colorscheme, but don't change the background?)

This was completed, with everything except that we've kept the "Loading..." text.  I'm not sure it makes sense to add a loading indicator, since in most cases the page loads so fast that you wouldn't see the animation happen, at least if one counts loading time starting from when this page loads (which is after the server does all the work it'll do on the backend).  Maybe worth a #design discussion about any next steps.

Yeah, I don't think it needs animation.


Export SignalOptions

Summary
Make this available will make it easier to create custom signals that pass options to createSignal. There is currently no way to access this as a generic with NodeNext module resolution.

Thank you

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.08%) to 88.607%

Files with Coverage Reduction
New Missed Lines
%

packages/solid/store/src/modifiers.ts
1
92.81%

Totals

Change from base Build<PHONE_NUMBER>:
0.08%

Covered Lines:
1288

Relevant Lines:
1378

💛  - Coveralls


[worldclock] Use vector font (feature request)

Affected hardware version
Bangle 2
Your firmware version
2v14
The bug
Use a vector font instead of the bitmap font in the worldclock app, as the current font is very pixelated (especially the large numbers).

You could check out hworldclock if you like. I use the vector font for a couple of items.


Mark job FAILED instead of KILLED

Coverage remained the same at 93.912% when pulling 0c68530b4b640d16644a4477b955d50fba91f4e0 on nvhoang:an-genie-error into c478d9a8942e6df9938921ec24812f9b46b27402 on Netflix:master.


GearClient.add should raise an exception when the chosen external port is in use.

GearClient.add should raise a custom exception if the chosen external port is already in use.
See:

https://github.com/ClusterHQ/flocker/pull/149#discussion_r14247046 and
https://github.com/ClusterHQ/flocker/commit/cce686392ceffd0292720d2307428db26909c283
https://github.com/ClusterHQ/flocker/issues/158

We are moving our development planning to JIRA. This issue is now being tracked at https://clusterhq.atlassian.net/browse/FLOC-157. You are welcome to file additional issues in GitHub if that's easier for you.


@file:KotlinOpts("--module-path") @file:KotlinOpts("--add-modules java.sql") doesn't seem to work w/ JDK11

Trying to use this because when I run w/ JDK11, GSon can't find java.sql.Time.
I get this error:
error: unsupported argument: --add-modules
Is there any way to add this module if >JDK8 is detected?

I think to pass an argument directly the JVM you need to prefix it with -J. See kotlin -help

Duplicate of https://github.com/holgerbrandl/kscript/issues/163


TransportBroker NPE

Description
TransportBroker.shutDownRouterServiceMessenger hitting NPE, likely due to routerServiceMessenger being set to null before we close routerServiceMessageEmitter. This NPE was hit by accident, testing custom RS logic with an incorrect app. The code referenced with this NPE has not been released yet and is only on the RC.
E  FATAL EXCEPTION: RouterServiceMessageEmitter
                                      Process: com.sdl.hellosdlandroid, PID: 17777
                                      java.lang.NullPointerException: Attempt to invoke virtual method 'android.os.IBinder android.os.Messenger.getBinder()' on a null object reference
                                      at com.smartdevicelink.transport.TransportBroker.sendMessageToRouterService(TransportBroker.java:153)
                                      at com.smartdevicelink.transport.TransportBroker$1$1.onMessageToSend(TransportBroker.java:116)
                                      at com.smartdevicelink.transport.RouterServiceMessageEmitter.run(RouterServiceMessageEmitter.java:32)

Closed with #1882


Addition of type="button" to directives

uifDialog, uifPeoplePicker, uifFacePile all use <button> elements, but do not set the type attribute to button.
The issue can be overcome with uifButton by adding the attribute to the markup, this is right as we are leaving the behavior to the consumer. However, for the directives above, we do not want a postback to occur - so must add the attribute.
Expected Behavior
Clicking buttons on one of the above directives should not issue a postback where the directives are within a <form> tag
Actual Behavior
Postback occurs due to the lack of type attribute
@andrewconnell  - Raising this as an issue formally as having thought about it, I believe the correct thing to do is add type="button to any directives where we need to control the behavior of the buttons e.g. clicking the 'add' button in FacePile.

n.b. uifPanel, uifCallout and uifSearchBox already use type="button"

OK... let's make sure we get more eyes on this. I think I'm in agreement with you, but implementing this could have some significant impacts so just want more eyes on it. What say you @ngOfficeUIFabric/ngofficeuifabric-members ?

The only significant impact would be that uifPeoplePicker etc. would be finally useable in a form. Please fix this issue.

But in the case of that one directive, couldn’t that directive be updated to add type=button to the button it generates?

Personally it would help me if it got added just for the uifPeoplePicker directive. Still it the type=button should be added to those kinds of buttons in other directives as otherwise it causes troubles with how form submission works in Angular.

Wait @newq... <uif-button> isn't even used in the <uif-peoplePicker>... it's only used in <uif-callout> & <uif-messageBanner> directives.
We're crossing the streams in this issue... let's stay focused on just adding type="button" to directives that use the <button> tag.
So let's start this over... any reason why we shouldn't add this to those directives mentioned in @tobiaswest83 's original post for this issue?

Hmm, @andrewconnell - my list was created by looking for directives which use <button>, not <uif-button>. In both cases a consumer of the library would get issues with using these directives in a <form>
The list of directives with some kind of <button> are now below, the fix in both cases is the same...

uifDialog (uses <button>)
uifPeoplePicker (uses <button>)
uifFacePile (uses <button>)
uifCallout (uses <uif-button>)
uifMessageBanner (uses <uif-button>)

Love the Ghostbusters reference though :laughing:

@andrewconnell Okay, I'm sorry. I might be confused on what exactly this issue is supposed to be discussing. Adding type=button to button or to uif-button. For now I'm just strongly in favor of adding it to button elements used in directives like the people picker.

Thinking about this more... we've got a few different things to think about... I'm going to cross the streams again because I think it's all part of the same discussion now. Let's start with an agreement that something needs to be addressed... we just need to figure out how.

FWIW, I looked at Angular Material... but it doesn't apply to us. They don't use element directives, they use attribute (CSS class) directives for their buttons... so the developer is always creating a <button> and including type=button themselves.

Should uif-button Always Render type=button?
Right now it doesn't, but there are plenty of cases where it should. The challenge is if we change this default, it would force it on everyone. However, we should add some sort of failsafe where you can exclude the type attribute from being added... something like <uif-button uif-isTypeExcluded=true></> which would default to false.
Other ideas for the attribute name:

isTypeExcluded (default=false)
typeAttrExcluded (default=false)
isButtonTypeExcluded (default=false)
isButtonTypeAttrExcluded (default=false)
includeButtonTypeAttr (default=true)
includeTypeAttr (default=true)

I vote to do this... also vote for isButtonTypeAttrExcluded... just hate how long it is
Should Directives with Buttons use uif-button?
Right now there's a mix... as @tobiaswest83 said in his comment above, two directives use uif-button and three use button. If we updated all of them to use uif-button & did the suggestion above (default rendered buttons to include type=button, it would fix the issue.
However, what about cases where you don't want that type included? You'd have to add the same new attribute used on the button (hence when picking the attribute name above, I think button should be in the attribute to be clear on directives that inherit from it, what it will be doing.
I vote for this.
Thought?

In my opinion uifButton could have a type attribute that's patched through to the actual button that is created. Then directives like the uifPeoplePicker could just use a uifButton where type=button is set in the directives template and uifButton would behave like a button would be expected to behave.

Thoughts here @andrewconnell
Should Directives with Buttons use uif-button?
I don't think we should only be using uif-button. Taking uif-panel as an example, the markup for the close button is quite divorced from what uif-button renders and would break stylings...plunk to follow laterif need be.
Should uif-button Always Render type=button?
No, it shouldn't and actually for the reasons above (that using button and uif-button are both valid) i don't think we should make any change to uif-button. Consumers of uif-button should be able to add type="button" without any extra legwork (as they do already), and all of the directives that use uif-button within ngOfficeUIFabric should have type="button" added to them as a matter of course, and I believe this would not have any impact on consumers of the library, since we don't want forms to submit on any of the actions in directives listed above.

If we want to keep flexibility what if uifButton would by default render type="button". In cases you wanted to use a different value all you would have to do is to specify your own value explicitly, so:
<uif-button />  --> <button type="button">

<uif-button type="submit"> --> <button type="submit">

The only scenario this wouldn't support, is if you wanted to avoid rendering the type attribute altogether but that's error prone as the result depends pretty much on the surrounding markup - exactly what led to this discussion 😄

yo @waldekmastykarz, not sure we're ready to close this one. We're all in agreement that we shouldn't add type="button" to uifButton, this is more about the inclusion of that attribute in directives which use uifButton

yeah, closed it by accident. my bad 😊

@tobiaswest83 if, as you said, all directives that use uif-button should also include type="button" why not include it ourselves as a safe default if no type is explicitly specified and simplify the markup?

@waldekmastykarz, that approach would work and gets my vote. We'd still need to fix non uifButtons but there are only a handful

Resolution based on this discussion - no changes to uif-button, but anywhere it's used within directives, it should include a type="button". So... we need to add issues for each directive that needs this change. Closing this issue as work should continue on those...

Resolution: this now


Add: Binary Prefix Divisible By 5

Codecov Report

Merging #466 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #466   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files         158    159    +1
  Lines        1574   1580    +6
=====================================
+ Hits         1574   1580    +6

Impacted Files
Coverage Δ

...fix-divisible-by-5/binary_prefix_divisible_by_5.go
100% <100%> (ø)

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 5d05326...efda7a6. Read the comment docs.


Shader Graph fails on display UV

The UV dont repeat in this view.
It's show wrong in editor, but when I compile that run fine!
And sometimes that I reimport image its work fine again, until I not add a new node.

i think this depends solely on the image resource texture repeat flag, can you make a small test scene for me to debug?

https://github.com/marcosbitetti/Godot_Shader_Test_Project
In this scene all runing fine. But when i try to edit shader the issue begin

wow you found a really old bug, glad I could fix it


Syntax highlighting does not use git.commentchar for highlighting comments

Currently, syntax highlighting assumes # is the comment character used in Git Commit messages. The comment character used in Git Commit messages can be configured using core.commentChar in .gitconfig. That functionality is commonly used to change the comment character to something else such that an issue tracker number can be used at the beginning of the commit message such as: #123: Some changes.
If possible, the syntax highlighting would read the configuration to determine the comment character instead of assuming # and highlight comments based on the configured character.

Hey, that's really interesting !
I don't know if it's possible to do that with tree-sitter but I will do some research.
I don't have much time right now but I'll work on it :sweat_smile:

That's all I can and would ask for 😄 and thank you for any time you can spend looking at this.
I'll try to do some digging myself. Neovim, treesitter, etc. are somewhat new to me so don't hold your breath though 😄

I'm probably only telling you what you already know but, just in case this helps at all... https://tree-sitter.github.io/tree-sitter/creating-parsers#external-scanners
It says...

Many languages have some tokens whose structure is impossible or inconvenient to describe with a regular expression

...and...

..in order to add custom logic for recognizing certain tokens.

So, I'm hoping those statements mean that this could be used to in some way read the commentchar config and then detect the comment token without affecting the rest of the parser too heavily.


Multiple paths For One Server

I found the nginx tmpl you supply couldnt handle muilti location in one server.
the new tmpl solved the problem however it use VIRTUAL_PATH env
hope to fix it later

I'm not going to merge this PR in its current form. I think it's important that the upstream nginx.tmpl start using define blocks for common chunks of configuration, so I don't like that those were removed. Besides that, it makes it difficult for me to review what you've actually changed here.

In addition, since @jwilder doesn't want to support this functionality by extending the syntax of the VIRTUAL_HOST variable instead of using a separate VIRTUAL_PATH variable (as discussed in https://github.com/jwilder/nginx-proxy/pull/254), I'm not keen to spend much time on this branch.
If I do get any time to work on supporting multiple paths in the nginx-proxy image, that time will be spent creating a usable library from docker-gen so that we can use a custom nginx-proxy binary written in Go that can streamline a lot of the nasty template logic in nginx.tmpl. See my (not working) generator branch of docker-gen for where I left off with that work.

actually the tpl is comes from another authoer, because it really works, so i think maybe its helpful to the branch, just the branch.
nice
i use traefik for a time, which combine path with host with mux,  the regexp follows the host confuse me a lot, which doesnt support ^ \b .etc, the tpl in muiltipath branch has the same problem, so is there any question combing the path & host?
anyway im look forward the new version of nginx-proxy, its amazing.


Failed to upload images

Hi,
i'm getting the following warnings and errors while importing carddav for every contact with an image.
PHP Warning:  Invalid argument supplied for foreach() in /opt/carddav2fb/carddav2fb.php on line 724
 FTP-Upload 'First_Last_902ed.jpg'...PHP Warning:  in_array() expects parameter 2 to be array, null given in /opt/carddav2fb/carddav2fb.php on line 706
PHP Warning:  ftp_put() expects parameter 1 to be resource, boolean given in /opt/carddav2fb/carddav2fb.php on line 708
 retrying... PHP Warning:  ftp_login() expects parameter 1 to be resource, boolean given in /opt/carddav2fb/carddav2fb.php on line 713
PHP Warning:  ftp_pasv() expects parameter 1 to be resource, boolean given in /opt/carddav2fb/carddav2fb.php on line 714
PHP Warning:  ftp_put() expects parameter 1 to be resource, boolean given in /opt/carddav2fb/carddav2fb.php on line 715
 ERROR: while uploading file First_Last_902ed.jpg

my system specs are:
debian jessie, php 5.6.20, owncloud 9.0.1, fritzOS 6.30
is there an error or is this my mistake?
Greets ManOki

Which version of carddav2fb are you using? the latest version in Git?

yes, latest version from git

Please update your git working copy and see if my latest modification actually fixes the problem.

Nope, didn't help
 FTP-Upload 'First_Last_e0d45.jpg'...PHP Warning:  ftp_put() expects parameter 1 to be resource, boolean given in /opt/carddav2fb/carddav2fb.php on line 711
 retrying... PHP Warning:  ftp_login() expects parameter 1 to be resource, boolean given in /opt/carddav2fb/carddav2fb.php on line 716
PHP Warning:  ftp_pasv() expects parameter 1 to be resource, boolean given in /opt/carddav2fb/carddav2fb.php on line 717
PHP Warning:  ftp_put() expects parameter 1 to be resource, boolean given in /opt/carddav2fb/carddav2fb.php on line 718
 ERROR: while uploading file First_Last_e0d45.jpg

Please try again. it seems your FTP connection to your fritz.box doesn't seem to work.

you missed some semikolons after your print commands, i added them locally. here is the result
Upload data to FRITZ!Box @ fritz.box
 WARNING: Secure connection to FTP-server 'fritz.box' failed, retrying without SSL. ERROR: couldn't connect to FTP server 'fritz.box'. Uploading Phonebook XML to fritz.box
  FRITZ!Box returned message: 'FRITZ!Box Telefonbuch wiederhergestellt'

So obviously your FRITZ!Box does either not support FTP connections or you have disabled them.

Please read here for more information about how to setup your FRITZ!Box for FTP connections (in case your FRITZ!Box model allows FTP connections at all):
https://github.com/jens-maus/carddav2fb#configuration

hm, that seems correct .. i can set the permission without having the opportunity. i didnt checked that twice, sorry.
is there any option to disable image upload at all?
thanks for your help!

Nope. You have to leave with the error message for the time being.


Automatically draw on paste of data

I think the 'Clear' button is no longer needed (pressing draw seems to clear the existing drawing out anyway) and I'm wondering if we even need 'Draw'.
If #18 is addressed, could we have a redraw trigger by any valid change to the BODS JSON in the input area?

This now happens.


Print current version of CLI

Currently, the CLI's --version option gives back unknown, needs to be fixed.

fixed in PR #6


The locations of the components must be clear

The website has been updated. Header, content, and footer have been added.

Please edit the description.

This problem is a problem that was opened for testing purposes.


Request: Add On/Off Toggle

Please add a toggle to be able to turn on/off individual auras. It would be great to be able build auras then enable or disable based on circumstances or need. Thanks!

I presume you are looking for something more binary than changing the "Visible to" property?
I'll see about adding a show/hide toggle to the configuration page.
In the meantime, you could use the new API to make custom macros

Ah, I will also give the visible to option a go. Yes, mainly looking for an
on/off binary option.
On Mon, Nov 25, 2024 at 10:49 Anthony Edmonds @.***>
wrote:

I presume you are looking for something more binary than changing the
"Visible to" property?
I'll see about adding a show/hide toggle to the configuration page.
In the meantime, you could use the new API to make custom macros
—
Reply to this email directly, view it on GitHub
https://github.com/AnthonyEdmonds/token-aura-ring/issues/23#issuecomment-2498535239,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLLL4GJKMIH3HFEZOPKLOD2CNIK7AVCNFSM6AAAAABR2M44L2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJYGUZTKMRTHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Toggle button is now available in 2.3.0


Type casting with arrays does not work

Description and Reproduction Steps
args := ['abc.txt', 'foo.md', 'bar.md']
md_files := []string(args.filter(fun (a string) bool {
	return a.ends_with(".md")
}))

for file in []string(md_files) {
}

Current Behavior
first snippet:
error: expression evaluated but not used

second snippet:
error: unexpected lpar, expecting lcur

Expected Behavior
No errors.
Possible Solution and Additional Information
It's planned to unify the casting syntax to var as Type.
Thus fixing this issue before is not advisable.
Bait version
Bait 0.0.2-dev
Environment details (OS name and version, Node.js version, etc.)
Arch Linux
Node 19.9.0

Type casting on arrays should actually be forbidden except for matching type aliases:
type Arr := []i32

fun x(a Arr){
	println(typeof(a))
	println(a)
}

fun main() {
	n := [1,2] as Arr
	x(n)
	x([1,2] as Arr)
}


Unable to get NTFS file owner

I run diskover in Docker. Because the diskover container is linux-base, I'm not able to get the NTFS file owner when I scan my NetApp CIFS shares. I tried different ways but in vain.
I tried:

Docker volume
Mount using CIFS with Kerberos
Mount using CIFS with Windows a/c

Is there any way to get NTFS owner when we scan CIFS shares?

@raychankit did you ever get this to work?  Following the somewhat sparse instructions in the wiki and linked documentation did not lead to a happy place.


Add  for valid markup

Syntax highlighting & prettier didn't work until I added the closing tag.
The resulting marking is what prettier auto-formatted to on save.

Thanks for catching! The file had moved so I pulled this change into fc3f535 myself.


Generate general sitemap.xml for projects

This PR makes Read the Docs to generate a general (non specific per project) sitemap.xml served at the root of the project /sitemap.xml based on discussions from #557
I think it would be good to split this in two phases:

Generate a general sitemap.xml for all the project without option to customize it (this PR as is)
Check if the project is already generating a sitemap.xml and instead of generating a general one, generate a specific one for this project using sitemapindex

Example with a toy project locally,
$ http http://fastcgi-for-net.dev.readthedocs.io:8000/sitemap.xml
HTTP/1.0 200 OK
Content-Language: en
Content-Length: 1713
Content-Type: application/xml
Date: Wed, 16 Jan 2019 21:15:44 GMT
Server: WSGIServer/0.2 CPython/3.6.6
Vary: Accept-Language, Cookie

<?xml version="1.0" encoding="UTF-8"?>
<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9"
        xmlns:xhtml="http://www.w3.org/1999/xhtml">

  <url>
    <loc>http://fastcgi-for-net.dev.readthedocs.io:8000/bn/latest/</loc>

    <xhtml:link
        rel="alternate"
        hreflang="en"
        href="http://fastcgi-for-net.dev.readthedocs.io:8000/en/latest/"/>

    <xhtml:link
        rel="alternate"
        hreflang="bn"
        href="http://fastcgi-for-net.dev.readthedocs.io:8000/bn/latest/"/>

    <lastmod>2019-01-16T21:13:53.325602+00:00</lastmod>
    <changefreq>daily</changefreq>
    <priority>1</priority>
  </url>

  <url>
    <loc>http://fastcgi-for-net.dev.readthedocs.io:8000/bn/stable/</loc>

    <xhtml:link
        rel="alternate"
        hreflang="en"
        href="http://fastcgi-for-net.dev.readthedocs.io:8000/en/stable/"/>

    <xhtml:link
        rel="alternate"
        hreflang="bn"
        href="http://fastcgi-for-net.dev.readthedocs.io:8000/bn/stable/"/>

    <lastmod>2019-01-16T10:54:16.792524+00:00</lastmod>
    <changefreq>weekly</changefreq>
    <priority>0.9</priority>
  </url>

  <url>
    <loc>http://fastcgi-for-net.dev.readthedocs.io:8000/bn/make-docs-dir-a-list/</loc>

    <xhtml:link
        rel="alternate"
        hreflang="en"
        href="http://fastcgi-for-net.dev.readthedocs.io:8000/en/make-docs-dir-a-list/"/>

    <xhtml:link
        rel="alternate"
        hreflang="bn"
        href="http://fastcgi-for-net.dev.readthedocs.io:8000/bn/make-docs-dir-a-list/"/>

    <lastmod>2019-01-16T11:03:10.324661+00:00</lastmod>
    <changefreq>monthly</changefreq>
    <priority>0.8</priority>
  </url>

</urlset>
$

I added small docs for sitemap to at least communicate that we are doing this automatically, without deep too much into its implementation. Once sitemap index is implemented, we can use this page to extend for that feature.

I was looking through the changes and it doesn't seem the sitemapindex is added to the robots.txt file, which would likely make it easier for the search engines to find.
If you think that is worth implementing here is an example of what I am talking about: https://github.com/jdillard/sphinx-sitemap#getting-the-most-out-of-the-sitemap

doesn't seem the sitemapindex is added to the robots.txt file, which would likely make it easier for the search engines to find.

I think this is a good addition and should be easy to add it to the default robots.txt returned by Read the Docs at https://github.com/rtfd/readthedocs.org/blob/799480827f20f04ded7239a6307853c721de39fa/readthedocs/core/views/serve.py#L312

I just pushed the changes suggested on feedback. I will merge this PR once tests pass.

Mmm... It seems that I can't merge because as I sent new changes I need a new approval now:

At least 1 approving review is required by reviewers with write access.


Update Scorecard for changecontrol

https://bestpractices.coreinfrastructure.org/en/projects/6008#changecontrol
We have a few misses around releasing, We seemingly have some of them already so may just need to connect the dots.
Every merged PR creates a unique snapshot tag for the build.

It is SUGGESTED that the [Semantic Versioning (SemVer)](https://semver.org/) or [Calendar Versioning (CalVer)](https://calver.org/) version numbering format be used for releases. It is SUGGESTED that those who use CalVer include a micro level value. [version_semver]

It is SUGGESTED that projects identify each release within their version control system. For example, it is SUGGESTED that those using git identify each release using git tags. [version_tags]

Release Notes

The project MUST provide, in each release, release notes that are a human-readable summary of major changes in that release to help users determine if they should upgrade and what the upgrade impact will be. The release notes MUST NOT be the raw output of a version control log (e.g., the "git log" command results are not release notes). Projects whose results are not intended for reuse in multiple locations (such as the software for a single website or service) AND employ continuous delivery MAY select "N/A". (URL required) [release_notes]
// No release notes file found.

The release notes MUST identify every publicly known run-time vulnerability fixed in this release that already had a CVE assignment or similar when the release was created. This criterion may be marked as not applicable (N/A) if users typically cannot practically update the software themselves (e.g., as is often true for kernel updates). This criterion applies only to the project results, not to its dependencies. If there are no release notes or there have been no publicly known vulnerabilities, choose N/A. [release_notes_vulns]

Done: https://bestpractices.coreinfrastructure.org/en/projects/6008


Add refreshControl on FlatList

Hello,
I needed to customize my pull to refresh loading spinner. I could not find a refreshControl prop in the FlatList (only in ScrollView). Unless I missed something, here enters my PR to add it !
RN Documentation: https://facebook.github.io/react-native/docs/flatlist#onrefresh

@abenoit thanks! Great work!


Propaganda site sometimes serves Jitsi

Sometimes accessing propaganda.pydis.wtf forwards to Jitsi. I don't really know
why, but I assume something is wrong with the nginx configuration.
The secret service is informed.

That did it, thanks !


Koalix website is down?

When visiting it from my machine:

Thank you for bringing up this point.
We will take care of this point as soon as possible.

Reduced the issue to major because I added a "Under construction" information instead of showing a very ugly django debug message.


Unable to run devtron

Im probably missing something obvious, but I'm unable to get devtron to run in the console.
I've just fired up meteor-desktop for the first time following the quick start guide.
When I enter require('devtron').install() into the console I get:
Uncaught Error: Cannot find module 'devtron'(…)
I can see the devtron module under the desktop-build\node_modules directory. Is there anywhere else I should be looking?
Thanks

I ran into this same problem. It's not documented, but I had to run Desktop.devtron.install() from the developer console for the electron app before I could get it to work.

That is weird.
I have removed the activation of it intentionally here: https://github.com/wojtkowiak/meteor-desktop/commit/9f5c335d15b4a01dcbbe77daf42315d9364b3bdb
because electron-debug was doing it anyway: https://github.com/sindresorhus/electron-debug#activates-devtron. But it seems that for some reason it sometimes fails...
I will add it back in 0.3.0 that will be released in few days - stay tuned.

0.3.0 released - let me know if this is now fixed.

Yes this is now fixed - thanks!


Implement RubyFeatures.distincts_kw_args_from_positional_hash?

Extracted from: https://github.com/rspec/rspec-mocks/pull/1461
cc @pirj

Please don't hesitate to rubocop:disable Metrics/ModuleLength for this class. Once we release RSpec 4, we should be back down under 100 lines.

Please don't hesitate to rubocop:disable Metrics/ModuleLength for this class.

Done.


Photino window is completely blank when the entrypoint (i.e. Main) method is defined as an async method that returns a 'Task' object

When the 'Program.Main' method (i.e. the entrypoint method) is defined as an async method that returns a Task object, the opened Photino window is completely blank (i.e. completely white) with no HTML being rendered to it despite waiting for a long time.

Another user seems to have found a way to get the "async Main scenario" working. See https://github.com/tryphotino/photino.NET/issues/52#issuecomment-2074325176
Please let us know if it works for you as well

Hi, thanks for the response and for the linked workaround, however, I have already found and made use of a different method to get the said scenario working, the said method was found in this StackOverflow answer: https://stackoverflow.com/a/9212343 and seems to be working for my current use case.


Explosions still drop blocks with canBreakBlocks off

I've been attempting to manually handle rebuilding explosion damage by disabling the explosion's ability to break blocks. Unfortunately this still results in blocks being dropped. While I can manually clean these blocks up, I don't think they should be present after block drops are disabled.

What this comes down to is that in MixinExplosion we cancel the explosionA method which does the break. And explosionB which handles the sound and block drops is ignored. Either this should also be ignored with canBreakBlocks off or we need more options for controlling an explosion. I'm willing to contribute either fix.

Re-test with stable-5 and reply back if still an issue. We'll re-open in the event that it is.


Upgrade dYdX perpetual

Upgrade dYdX perpetual to gold standard. HIP: https://snapshot.org/#/hbot-ip.eth/proposal/0x9f6a13e17917c688bac237fb311d652a787b46843598ec46050b36ae3780624d

HBOT bounty: 250,000


Overview UI

Overview should contain the account information rather than the recent transactions. QR code on overview for easy access, as well as QR code on each account detailed view (could be a small icon that enlarges when clicked).
For example:

Yes I also think this is a good idea. Do you confirm that @ruannelloyd ?

I close this one as the planned redesign for the v1.2.0 will include a final decision about that.


use node cluster

Tested locally, opened 3 connections for 3 workspaces and saw them round-robin'ed across 3 separate node processes.

Just double checking, a workspace stays on one node process right due to the persistent connection between lsp-proxy and the LSP server?

Yep!
On Wed, Oct 12, 2016 3:01 AM, Keegan Carruthers-Smith<EMAIL_ADDRESS>wrote:
Just double checking, a workspace stays on one node process right due to the
persistent connection between lsp-proxy and the LSP server?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.
John Rothfels
The future sooner @ ✱Sourcegraph
(916) 990 4297


[docs][base-ui] React Hook Form guide/tutorial

Continues https://github.com/mui/material-ui/issues/36697
While researching another issue about Base's FormControl, I made a sandbox using React Hook Form and made a guide out of it – RHF is the leader in that space right now.
The topic of the guide pivoted to "how to integrate with RHF", which I think may have broader appeal than the previous "how to apply CSS to make this specific design". There is more focus on how to use different features of the components than how to apply styling to them.

Material Autocomplete + RHF: https://codesandbox.io/s/material-ui-autocomplete-rhf-hry7yp?file=/src/App.tsx

Following the decision to ensure that hooks are given (at least) equal attention as unstyled components, I'll experiment with the format of each section to see if usage of both hooks and components can be demonstrated


When can't connect to BitTorrent DHT, other methods aren't working either.

Repro

share a directory
open a new terminal and download the directory
they can't find eachother.
turn off wifi
re-share, re-download
they find eachother.

Output during step 3 seems like its trying to reach the bittorrent dht and failing. But it should still be trying to find the local dat using another mthod, right? This person also wasn't able to successfully use other programs that relied on the bittorrent dht, like utorrent and transmission.
dat doctor output
https://gist.github.com/baruchlab/5a101bc8d47c2263fae79809c85e4892

found by @baruchlab

I think I've run into cases where dats intermittently won't find each other.

@karissa that doesn't look like doctor output (looks like DEBUG=*). you didn't happen to try a dat --doctor test?

Just noting that it actually connects to the DHT fine based on the DEBUG output (no need to debug that) but it probably cannot hole punch any dat connections.

@mafintosh i'll see this guy tonight again i think and i'll try to get more output

@karissa if you could run p2p-test on his network that would be amazing (run it with karissa-test)

hello hello
I ran p2p-test on the same network we were testing dat on on Wednesday... here are the results
Network tested. Thank you for participating.
{ time: '2016-09-23T16:15:19.557Z',
description: '[karissa-test]',
holePunchable: true,
pings:
[ { port: 50571, host: '<IP_ADDRESS>' },
{ port: 50571, host: '<IP_ADDRESS>' } ] }
everything looks fine

I am sharing the directory now. ping me on patchwork for the link, see if you can get it. Baruch

hi Karissa I updated dat and am sharing now for the next 15 mins

@baruchlab what's the hash? :)

oops didn't see yr message.  6aab4cecd5f8a35ee29f8baa9b419579bed8e5b6e8b6a420f43fdf7d41be97d2
starting dat up again now

@baruchlab we weren't able to connect. Is this still the same network as before? And you upgraded the dat installation?

no I am in a train in Canada no with wifi completely different network.  I won't be back on the original network until the week of Oct. 18

results of p2p-test
{ time: '2016-10-06T21:15:46.349Z',
description: 'karissa-test2',
holePunchable: true,
pings:
[ { port: 53646, host: '<IP_ADDRESS>' },
{ port: 53646, host: '<IP_ADDRESS>' } ] }

version is 11.4.0

okay, thanks. Someone was able to connect and get data (but @mafintosh and I weren't, so it may be our network here). We will be interested when you get back to the other network if it still works. Thanks!

do you want me to try again on other random networks, or only the first one I originally tried?

@baruchlab feel free to run p2p test on other networks, that will give us any data we need to help improve dat across networks.

I think this is fixed


Export container

This exports the container struct to fix #14. I'm also adding golang 1.10 to the travis test suite.
@byxorna @qx-xp @roymarantz @aeriff

Hey, could this be merged? We are also interested in having this :smile_cat:


Split menus into primary & sub parts.  all pages use the same template.

Modified the Rakefile which got it working for me, including translations.
Works with Make also.

Let's do some of this here: https://github.com/bitcoincashorg/bitcoincash.org/pull/26

Next round of updates. This should be right to go.
Added the new pages into the menu system.
Set the new pages up to follow the new header + submenu layout.

Some of the menu links don't work
The workgroups being at the same level as the dev portal is odd
If you goto workgroups you get "developers" : http://<IP_ADDRESS>:4000/workgroups.html
If you goto spec, you get "workgroups" and specifications: http://<IP_ADDRESS>:4000/specs.html

These should be fixed before merging.


Adding Qos::ExactlyOnce transmission support

This PR adds support for the QoS::ExactlyOnce transmission mechanism.
This PR builds on #90
TODO:

[x] Determine an integration test for exactly once transmission

     Running tests\exactly_once_transmission.rs (target\debug\deps\exactly_once_transmission-aaf28aea1c417482.exe)

running 1 test
[2022-08-19T17:08:43Z INFO  minimq::mqtt_client] Sending CONNECT
[2022-08-19T17:08:43Z TRACE minimq::network_manager] Wrote: [10, 17, 0, 4, 4d, 51, 54, 54, 5, 2, 0, 3c, a, 11, ff, ff, ff, ff, 27, 0, 0, 1, 0, 0, 0]
[2022-08-19T17:08:43Z TRACE minimq::network_manager] Read: [20, 44, 0, 0, 41, 11, 0, 0, 7, 8, 21, 0, a, 12, 0, 33, 68, 6d, 71, 5f, 45, 64, 35, 55, 36, 5f, 31, 37, 34, 31, 31, 31, 39, 32, 5f, 65, 30, 32, 64, 30, 36, 30, 39, 38, 33, 32, 30, 63, 36, 31, 35, 36, 63, 39, 34, 31, 66, 31, 34, 38, 36, 37, 36, 35, 31, 33, 32, 22, 0, 5]
[2022-08-19T17:08:43Z DEBUG minimq::mqtt_client] Received 70 bytes
[2022-08-19T17:08:43Z DEBUG minimq::mqtt_client] Processed 70 bytes
[2022-08-19T17:08:43Z INFO  minimq::mqtt_client] Received ConnAck(ConnAck { session_present: false, reason_code: 0, properties: [SessionExpiryInterval(1800), ReceiveMaximum(10), AssignedClientIdentifier("hmq_Ed5U6_17411192_e02d06098320c6156c941f1486765132"), TopicAliasMaximum(5)] })
[2022-08-19T17:08:43Z INFO  minimq::mqtt_client] Ignoring property: SessionExpiryInterval(1800)
[2022-08-19T17:08:43Z INFO  minimq::mqtt_client] Ignoring property: ReceiveMaximum(10)
[2022-08-19T17:08:43Z INFO  minimq::mqtt_client] Ignoring property: TopicAliasMaximum(5)
[2022-08-19T17:08:43Z DEBUG minimq::de::packet_reader] Popping packet of 70 bytes
[2022-08-19T17:08:43Z DEBUG minimq::mqtt_client] Publishing to `data`: [80, 105, 110, 103] Props: []
[2022-08-19T17:08:43Z TRACE minimq::network_manager] Wrote: [34, d, 0, 4, 64, 61, 74, 61, 0, 1, 0, 50, 69, 6e, 67]
[2022-08-19T17:08:43Z INFO  exactly_once_transmission] Publishing message
[2022-08-19T17:08:43Z TRACE minimq::network_manager] Read: [50, 2, 0, 1]
[2022-08-19T17:08:43Z DEBUG minimq::mqtt_client] Received 4 bytes
[2022-08-19T17:08:43Z DEBUG minimq::mqtt_client] Processed 4 bytes
[2022-08-19T17:08:43Z INFO  minimq::mqtt_client] Received PubRec(PubRec { packet_id: 1, reason_code: 0, properties: [] })
[2022-08-19T17:08:43Z INFO  minimq::mqtt_client] Sending PubRel(1)
[2022-08-19T17:08:43Z TRACE minimq::network_manager] Wrote: [62, 2, 0, 1]
[2022-08-19T17:08:43Z DEBUG minimq::de::packet_reader] Popping packet of 4 bytes
[2022-08-19T17:08:43Z TRACE minimq::network_manager] Read: [70, 2, 0, 1]
[2022-08-19T17:08:43Z DEBUG minimq::mqtt_client] Received 4 bytes
[2022-08-19T17:08:43Z DEBUG minimq::mqtt_client] Processed 4 bytes
[2022-08-19T17:08:43Z INFO  minimq::mqtt_client] Received PubComp(PubComp { packet_id: 1, reason_code: 0, properties: [] })
[2022-08-19T17:08:43Z DEBUG minimq::de::packet_reader] Popping packet of 4 bytes
[2022-08-19T17:08:43Z INFO  exactly_once_transmission] Transmission complete

As can be seen in the logs, the whole handshake process (PUB, PUBREC, PUBREL, PUBCOMP) is completed as expected.


README.md: link to new wiki

This commit updates the the link from the former, unofficial nixos wiki page to the new https://wiki.nixos.org
ref: NixOS/foundation#113

Thank you!


fast-json-stable-stringify

copied from https://github.com/vega/lyra/pull/447
Currently, Webpack fails at correctly compiling Vega-Lite's reference to fast-json-stable-stringify which is an ES6 import statement of a CommonJS module.
Pending a more thorough investigation/overhaul of our build system, we will need to dig into node_modules/vega-lite/src/util.ts and replace the import with an old-style CommonJS require:
const stableStringify = require('fast-json-stable-stringify');

Resolved with fd7c0ad7. I'd recommend rm -rF node_modules and then re-running yarn after your next git pull.


Workaround nested TabNavigation swipe issue

Please provide enough information so that others can review your pull request:
Explain the motivation for making this change. What existing problem does the pull request solve?
Workaround nesting TabNavigation swipe bug for Android.
https://github.com/react-native-community/react-native-tab-view/issues/297
Test plan (required)
Demonstrate the code is solid. Example: The exact commands you ran and their output, screenshots / videos if the pull request changes UI.
Make sure you test on both platforms if your change affects both platforms.
The code must pass tests and shouldn't add more Flow errors.
Code formatting
Look around. Match the style of the rest of the codebase.

Looks like CI is failing because of Flow issues:
><EMAIL_ADDRESS>flow-check /root/react-navigation
> flow check --show-all-errors

[2017-10-21 18:28:42] File /root/react-navigation/node_modules/art/parsers/svg/text.js is malformed
Error: src/views/TabView/TabView.js:205
205:       props.initialLayout = { width: 1, height: 0 };
                 ^^^^^^^^^^^^^ property `initialLayout`. Property not found in
205:       props.initialLayout = { width: 1, height: 0 };
           ^^^^^ object literal

Found 1 error
npm info lifecycle<EMAIL_ADDRESS>Failed to exec flow-check script
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>flow-check: `flow check --show-all-errors`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>flow-check script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2017-10-21T18_28_52_596Z-debug.log
error Command failed with exit code 2.
Exited with code 1

I think it should be easily fixable by addding flow types, this comment should help you out.

@kelset Addressed flow issues.

I'm not liking this PR very much - the problems I see are following:
1  we're mutating the props, which is not a good practice, see the docs
2  what if somebody is passing their custom initialLayout and they're running on android? We just replace their custom layout with { width: 1, height: 0 }, no questions asked.(!)
3 @ssssssssssss why is the initial layout specified only for android? The issue you are mentioning talks about iOS only.

I see your points @vonovak, I guess we can revert if you prefer.
That said, here's why it was done for Android (I guess) and probably based on this too.
Again, no probs in reverting and changing for a better PR.

@vonovak @kelset #2843 might help?

@vonovak It's an Android only issue.
@Minishlink 's approach should be more elegant.
Thanks for all your insights.

this was fixed with another PR


Flash back crash at startup

log:'Can't resolve assembly for name StoryboardWithReferenceAssembly'

hey @xiubojin, StoryboardWithReferenceAssembly - is only included into Typhoon's unit tests target, and shouldn't be mentioned in main target or apps. Can you share more details about the problem? I'll close the issue for now as outdated and will reopen if more details provided.


cosmetic improvements + link  to data repo

https://github.com/AIRCentre/JuliaEO/issues/36#issuecomment-1461749731 @EmanuelCastanho

ready for merge this one?


Fix optional starttls for inbound checks

For some reason I already passed "starttls" as an argument, but then never used.. Thank you!


Extend custom fields to Position, Organization, Location

Position, Organization and Location will now have custom fields defined. System admins can update anet-dictionary.yml file to choose what kind of fields they would like to add to these types.
Added customFields columns to positions, organizations and locations table. The development anet-dictionary.yml file has these three new custom fields same as person.customFields, system admins should update these according to their requirements.
Closes #3367
User changes

Users will see additional fields from custom fields in these pages/forms.

Super User changes

Super users will see additional fields ( custom fields)  that needs to be filled when creating Position, Location, Organization

Admin (System admin) changes

dictionary.fields.position.customFields, dictionary.fields.organization.customFields, dictionary.fields.location.customFields can be filled according to requirements.

System admin changes

[X] anet.yml or anet-dictionary.yml needs change
[X] db needs migration
[ ] documentation has changed
[ ] graphql schema has changed

Checklist

[X] Described the user behavior in PR body
[X] Referenced/updated all related issues
[X] commits follow a repo#issue: Title title format and these 7 rules
[X] commits have a clean history, otherwise PR may be squash-merged
[ ] Added and/or updated unit tests
[ ] Added and/or updated e2e tests
[ ] Added and/or updated data migrations
[ ] Updated documentation
[X] Resolved all build errors and warnings
[ ] Opened debt issues for anything not resolved here

I can save the customFields to the database, but I can't seem to query customFields for what is saved, could use some help :smile:

I can save the customFields to the database, but I can't seem to query customFields for what is saved, could use some help :smile:


[BUG] - Trace ParentId is not restored from Diagnostic-Id message property

Library name and version
Azure.Messaging.ServiceBus 7.10.0
Describe the bug
I'm investigating W3C context propagation over ServiceBus. It looks like message sent correctly with valid Diagnostic-Id is not correlated correcly during the message read handling. For both servicebusreceiver.receive and servicebusprocessor.processmessage operations the ParentId property is null. Because of that, these operations are reported as separated traces instead of one trace, started on the web api request.

I tried to set ParentId manually with lines like:
    private async Task ProcessorOnProcessMessageAsync(ProcessMessageEventArgs arg)
    {
        var bodyAsString = arg.Message.Body.ToString();
        var diagnosticsId = arg.Message.ApplicationProperties["Diagnostic-Id"].ToString();
        using var activity = new Activity("ServiceBusProcessor.processing");
        activity.SetParentId(diagnosticsId);
        activity.Start();

        [...]
        activity.Stop();
    }

but it moves only user-code to the correct trace. Both servicebusreceiver.receive and servicebusprocessor.processmessage operations are received outside of the flow

Expected behavior
ServiceBus reader (both processor and receiver) should use TraceId from Diagnostic-Id as a ParentTraceId
Actual behavior
servicebusreceiver.receive and servicebusprocessor.processmessage don't read ParentTraceId
Reproduction Steps
Available at https://github.com/lukasz-pyrzyk/opentelemetry-demo, API and webjob project, together with HTTP request to start the flow.
Environment
No response

Hi @lukasz-pyrzyk.   Thank you for reaching out and we regret that you're experiencing difficulties.   The decision to not use the ParentId is intentional, as the library uses links to connect the individual messages activities with the parent, as per the OpenTelemetry messaging spec.
Further discussion can be found in #29907.
I'm going to mark this as addressed, but please feel free to unresolve if you'd like to continue the discussion.

//cc: @lmolkova, for any additional thoughts.

Hi @jsquire and thank you for the quick reply.
How Links functionality corresponds to the https://learn.microsoft.com/en-us/azure/service-bus-messaging/service-bus-end-to-end-tracing?tabs=net-standard-sdk-2#trace-message-processing ? That code sample gave me the idea to link spans with ParentTraceId, but now it looks like it is not recommended way, because it is done automatically with Links, so I think i got confused with it.
After reading the sentence If you use ProcessMessageAsync of ServiceBusProcessor (message handler pattern) to process messages, the message processing is also instrumented. All Service Bus calls done by your service are automatically tracked and correlated with other telemetry items. I thought that SDK has a bug, because ParentId is not populated automatically.
Is there a chance that the article comes from the times of Microsoft.Azure.Servicebus, where this approach was used?

Is there a chance that the article comes from the times of Microsoft.Azure.Servicebus, where this approach was used

I don't know the origin of the article nor who maintains it, but the code samples appear to have been updated for the current generation and I notice that our telemetry expert is listed as a contributor.    @lmolkova:  Would you be so kind as to offer your thoughts?

@lukasz-pyrzyk thanks for reaching out!
We do correlation for messaging scenarios using links, i.e. processor span/activity does not become a child of producer activity, but we add a link to it.
We also fixed some issues related to receive and complete/defer/etc in the last 7.11.0-beta.1 - they also use links.
The reason for it is that we don't really know what Activity is current at the moment we receive/settle/process spans. I.e. we want them to be children of your current context. By using links we don't override it, but the downside is that not all backends support links.
As a result, if your backend supports links, you'd see something like this:

I don't think we can change this behavior for receive and settle calls, but I believe we can change it for process, where there is no chance for user context to appear before message processing starts.
I believe we can do it in non-breaking way, i.e. keep the link for now and also use the same context as a parent. I saw similar concerns from users in Java and we're starting to set parent instead of a link for processor spans.

Hi @lmolkova and thanks for the quick reply!
I've updated to 7.11.0-beta.1 and unfortunately, on my backend I don't see a link, tried Zipkin and Jaeger (I read somewhere that Jaeger supports it, but unfortunately without a luck for me).
I think adding this behavior for process functionality would be a great improvement. It would simplify the many investigations and answer many questions that come from lack of that. Personally, I thought about this as  a bug, because in previously mentioned article setting parentTraceId is done manually. In my PoC from two years ago, I've implemented flow tracing for servicebus manually to achieve this behavior. Now, with official support for the Messaging, I expected to have it out of the box. If it is possible without doing a breaking change, I would vote for it!

Thanks for trying @lukasz-pyrzyk

Jaeger supports it, but unfortunately without a luck for me

They're called 'references' in Jaeger and if you find a process span there and expand it, you should see a list of references, they will point to the message. It only works in one direction, so finding the process span from the message is not possible in Jaeger.

Ok, I got it. I found FOLLOWS_FROM in the references section for ServiceBusReceiver.Complete and ServiceBusProcessor.ProcessMessage actions. I wasn't able to find it on the UI, maybe because link is to the trace from a different application (and it seems like Jaeger filters traces by specific service).
Thank you for great the explanation @lmolkova . I think we can close the issue. I hope that connecting parentTraceId to the Process span will be implemented in the future :)


Allowing passing tabsequence strategy

When working with Ionic apps that heavily use Shadow DOM, it looks like we'll need to be able to pass strategy: 'strict' to this plugin.

I could try submitting a PR. A new option strategy?
cy.get('body').tab({ strategy: 'strict' });

For my app, I think I'd then use overwrite to always set this as my default.
import 'cypress-plugin-tab';

Cypress.commands.overwrite('tab', (originalFn, options) => {
  // allow overriding `strategy` per query, but default to 'strict' if undefined
  return originalFn({ strategy: 'strict', ...options });
});

Thoughts?

I've opened a draft PR with a proposal 🙏

Going to close for now since this didn't really address the underlying issue.


Remove sorted edges

addressing #90

Pull Request Test Coverage Report for Build 1087

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 98.832%

Totals

Change from base Build 1086:
0.6%

Covered Lines:
1354

Relevant Lines:
1370

💛  - Coveralls

Pull Request Test Coverage Report for Build 1087

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 98.832%

Totals

Change from base Build 1086:
0.6%

Covered Lines:
1354

Relevant Lines:
1370

💛  - Coveralls

Pull Request Test Coverage Report for Build 1087

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 98.832%

Totals

Change from base Build 1086:
0.6%

Covered Lines:
1354

Relevant Lines:
1370

💛  - Coveralls

Pull Request Test Coverage Report for Build 1087

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 98.832%

Totals

Change from base Build 1086:
0.6%

Covered Lines:
1354

Relevant Lines:
1370

💛  - Coveralls


How does skywalking integrate with elk?Can I just install Kibana?

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion

Question

What do you want to know?

I want to search log with elk.I had install skywalking and elasticsearch.Can I just install Kibana?

Requirement or improvement

Please describe about your requirements or improvement suggestions.

Use Kibana to search who's log? SkyWalking or Logstash?

      Use Kibana to search who's log? SkyWalking or Logstash?

SkyWalking . I want more convenient query logs from SkyWalking . So I want to use Kibana to query logs from SkyWalking.

SkyWalking provided querys based on GraphQL. Try that.

Can skywalking-ui show the searching page ? Can you provide me with the related material ?
发送自 Windows 10 版邮件应用
发件人: 吴晟 Wu Sheng
发送时间: 2018年10月23日 11:57
收件人: apache/incubator-skywalking
抄送: wangnanyang; Author
主题: Re: [apache/incubator-skywalking] How does skywalking integrate withelk?Can I just install Kibana? (#1796)
SkyWalking provided querys based on GraphQL. Try that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

What do you mean seaching page? I am not following. GraphQL query protocols are here, https://github.com/apache/incubator-skywalking/tree/5.x/apm-protocol/apm-ui-protocol/src/main/resources/ui-graphql


Upgrade to WP 6.7

Dev instance

Shows a more blurry image in the egg on the homepage, as described here.


Orekiil

In #319, Lukes mentions "a hang out spot (Orekiil)". Can someone confirm?  Where exactly is it?

My mother-in-law says it means the final day or days, apparently used commonly by Palauan Catholics in mourning. Related to the custom called "novena" in English.

we already have orekiolel n.poss.3s. both words seem to be related.

I think my original question was from the song Orekil (which is how it should be spelled).  It is n.poss.3s meaning "it's end." The Kerresel dictionary defines it as "orekil a rekil" where rekil comes from merek.  Yoichi Rengiil thought that this was a place name in the Rock Islands on the island of Ngerechol.
On Tuesday, October 18, 2022 at 03:52:08 PM PDT, ChrisPerrette @.***> wrote:
My mother-in-law says it means the final day or days, apparently used commonly by Palauan Catholics in mourning. Related to the custom called "novena" in English.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I looked up #319 and I only referred to it as a place because that's what
it sounds like in the sentence from the song... possibly rendezvous at
Orekiil... Double "i"
Rekil, orekiolel, orekil... These are different and relate to completion or
conclusion, as in "merek" (mla mo merek) pointed out in this chain.
J
On Tue, Oct 18, 2022, 4:58 PM JimGeselbracht @.***>
wrote:

I think my original question was from the song Orekil (which is how it
should be spelled).  It is n.poss.3s meaning "it's end." The Kerresel
dictionary defines it as "orekil a rekil" where rekil comes from merek.
Yoichi Rengiil thought that this was a place name in the Rock Islands on
the island of Ngerechol.
On Tuesday, October 18, 2022 at 03:52:08 PM PDT, ChrisPerrette
@.***> wrote:
My mother-in-law says it means the final day or days, apparently used
commonly by Palauan Catholics in mourning. Related to the custom called
"novena" in English.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/tekinged/missing/issues/349#issuecomment-1283104323,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXYSPXX44ARICHZTFTYFDPTWD4TPTANCNFSM6AAAAAARIFQY5E
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Orekil/orekiolel: final, finale
orekii: a place not sure and also, past for merereek. Lak monherodech e ko orekii a mechiuaiu.
Ngera merereek er a rebai?

But wouldn't that be double "r" to show the strong sound?
Orreek
Orrekii
Like the sound of "orrekim"
On Tue, Oct 18, 2022, 6:16 PM Gnalu @.***> wrote:

Orekil/orekiolel: final, finale
orekii: a place not sure and also, past for merereek. Lak monherodech e ko
orekii a mechiuaiu.
Ngera merereek er a rebai?
—
Reply to this email directly, view it on GitHub
https://github.com/tekinged/missing/issues/349#issuecomment-1283165905,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXYSPXRIY5Z7XXRD2X3WXRTWD44UXANCNFSM6AAAAAARIFQY5E
.
 :-)


Puppeteer version upgrade?

Hello, I just migrated to this from react-snap because I thought it's better maintaned.
It works fine on my Windows machine but I believe the no longer supported puppeteer version is breaking my build on Linux.
Are there any plans to upgrade? We are at 10.4.0, the latest is at 19.2.2.

Hi @gergelyszabo94, thanks for mentioning it, i'm going to ship out new version soon, it will contain deps updates.


Update info for Orlin Vasilev

Signed-off-by: OrlinVasilev<EMAIL_ADDRESS>

I don't understand your pronoun entry.  This might be helpful.

@cjyabraham believe me or not I read that as nickname ... sorry it's fixed now!


CFE parser doesn't accept late var, but the grammar requires it

@danrubel
One of the errors produced by co19_2/LanguageFeatures/nnbd/syntax_A04_t01 on front-end-linux-release is:
tests/co19_2/src/LanguageFeatures/nnbd/syntax_A04_t01.dart:26:15: Error: Members can't be declared to be both 'var' and 'late'.
Try removing one of the keywords.
  static late var staticVar3 = "staticVar3";
              ^^^

Full log (contains other errors that are bugs in the test itself): https://dart-ci.appspot.com/log/front-end-linux-release-x64/fasta-linux/6370/co19_2/LanguageFeatures/nnbd/syntax_A04_t01
But the grammar requires 'var' or 'final' or a type after 'late'.

Relevant part of the grammar:
| late? final (type)? (initializedIdentifierList)
| late? (varOrType) (initializedIdentifierList)


Convolutional deep neural network module

libprotobuf required to sucessfuly buid this module

Hi!
Generally you shouldn't fail whole OpenCV build if some requirements are missing for module (at first it is needed for other users who uses opencv_contrib). In this case you should simply skip module build with optional message.
Use something like this:
https://github.com/Itseez/opencv/blob/48c9c24da6d8d4c432b674eae8808ee717c422c4/modules/cudev/CMakeLists.txt#L1-3
BTW, This patch is rather big, about 5.5Mb, but I believe it is not very critical. Most part is the .caffemodel file.
@vpisarev We can think about installation of the libprotobuf on a some builder to test compilation of this patch (I prefer Linux).

@vpisarev We will have the same problem of the libprotobuf and probably glog gflag dependency on the other DNN project. Then we will put caffe dependency with minimal cmake flag activated in a submodule if you think that will be buildable on buildbot servers.

/cc @mtamburrano @Wangyida

Hi,
libprotobuf have been installed on Linux slave.
BTW, about big .caffemodel file. I believe, that this size is not very critical (I usually saw files about hundreds of megabytes). Also it is very useful to have good completed example and test with data (if there are no licence issues).

@alalek I think that 300MB data file of AlexNet network in this repo it's not good idea (opencv_contrib.zip takes 56MB now).
Caffe original repo has special .py script int repo to download trained .caffemodel files.
Maybe add this script and download data using it and CMake add_custom_target()?
So, there is no need to add tests at present time, therefore I delete metioned data file of small network.

Right, I mean gtsrb_iter_36000.caffemodel file.

Amalgamated protobuf is proposed to include inside the module. Not external protobuf dependency is planned.

@bhack You can try this simple header to avoid glog depedendecy.

@ludv1x We are trying to modularize caffe to let it build with minimal dependencies. See https://github.com/BVLC/caffe/pull/2619

@Nerei, is it ready for integration?

@vpisarev Any urgency here? The code still has external protobuf dependency. Unsure if cmake is forrect with respect to double pass done by opencv scripts.

@Nerei, ah, it's ok then, no urgency, I've just asked

@ludv1x , @vpisarev ,

Cannot see in header the license and authors, from where exactly are these imports ? (I suspect its BVLC's caffe).
Will be opencv_contrib / itseez able to maintain such a big chunk of complex thing ?
If its a fork from caffe, is there any ack or discuss with caffe maintainers ? Who's merit is this here ?

Header for sampe too

@ludv1x, could you please also fix the error reported by the doc builder?
http://pullrequest.opencv.org/#/summary/contrib

:+1:
Buildbot failures are not related this this module

@ludv1x Please recreate PR with "dnn" branch name (same as the extra PR branch name). Also please squash/replace commits into one (102 commits are too much and there are many useless/nonworking intermediate changes)

If you have some problems (there are many "merge" commits from master, squashing will not go well) I prepared a sample squashed commit: https://github.com/Itseez/opencv_contrib/commit/c94fdefcfff9cc5b8300c49052f0c1eee8896f0b
You can get it via:
git fetch https://github.com/Itseez/opencv_contrib c94fdefcfff9cc5b8300c49052f0c1eee8896f0b:dnn
git commit --amend # edit commit message
git push -f ...

@ludv1x The branch should match with opencv_extra brach so that buildbot  could test them jointly.

This PR moves to https://github.com/Itseez/opencv_contrib/pull/362


openrtm_aist: 1.1.2-3 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository openrtm_aist to 1.1.2-3:

upstream repository: http://svn.openrtm.org/OpenRTM-aist/tags/RELEASE_1_1_2/OpenRTM-aist/
release repository: https://github.com/tork-a/openrtm_aist-release.git
distro file: melodic/distribution.yaml
bloom version: 0.6.6
previous version for package: 1.1.2-2

this should fix http://build.ros.org/job/Mbin_uB64__openrtm_aist__ubuntu_bionic_amd64__binary/2/consoleFull


any methods to modify users?

Hi there,
Are there any methods in this api to edit users in an ActiveDirectory?

No. The ldapjs does provide the necessary functionality via standard LDAP calls. Pull requests are welcomed.


Only create the sg if network_mode == awsvpc

what

Only create the sg if network_mode == awsvpc

why

Looks like the security group is used only if var.network_mode = "awsvpc"

references

https://github.com/cloudposse/terraform-aws-ecs-alb-service-task/issues/102

/test all

/test all


Still working?

does it work fine for you?

any update @Solana0x ?

its working FIne please use the webshare website to buy proxy instead of Proxies.fo !

which package in webshare? datacentre?

You can buy the below one i have buyed also one plan can be used for multiple Nodepay gmail accounts i am running on 20 accounts and earning ~ 25-30M per day from all accounts combined my code uses least bandwidth as compared to any other code on the Github !! IF you wana earn more you can buy 2500 Proxy also but buy datacenter ones !!
https://www.webshare.io/?referral_code=gppl5h10bwn5


Add SCaleb-O's file

created a branch, created a file and made a commit, and opened a pull request

Moving on

Moving on

Next...

Next...


Question about the GKR implementation

I'm reading your GKR implementation. I'm wondering which technique you are using in the code.
It looks like the implmentation combines multiple techniques, it will be helpful if you can list all related papers.

solve here: https://github.com/ConsenSys/gnark-crypto/pull/243#issuecomment-1416407868


Seems no dependency on gin-swagger (cyclic dependency ?)

Describe the bug
It looks that current swag does not have dependency on gin-swagger though go.mod has dependency on<EMAIL_ADDRESS>Due to this dependency, performing go mod tidy in project using echo-swagger fails with following error.
        github.com/swaggo/echo-swagger tested by
        github.com/swaggo/echo-swagger.test imports
        github.com/swaggo/gin-swagger/example/docs: module github.com/swaggo/gin-swagger@latest found (v1.2.0), but does not contain package github.com/swaggo/gin-swagger/example/docs

To Reproduce
Steps to reproduce the behavior:

Create a 'go moduled' project depends on  github.com/labstack/echo/v4 and github.com/swaggo/echo-swagger.
Perform go mod tidy in the project

Expected behavior
go mod tidy completes w/o error.
Screenshots
n/a
Your swag version
1.6.3
Your go version
1.13.4

@gasuketsu  GO is right, there is no docs package in swaggo/gin-swagger/example for version 1.2.0 as swagg/go.mod is describing.

#800 along with swaggo/echo-swagger#33 should fix this issue

Closing: fixed.


Incorrect Selection of Variants for Changing Prices in Configurable Product

Describe the bug
When using the quick price change function for configurable products in the Bagisto admin panel, the selection via the drop-down menu does not respect the specified parameters. Variants that do not match the set parameters are also selected, which leads to price changes even for unwanted products.
To Reproduce
Steps to reproduce the behavior:
Access the Bagisto Administrator Panel.
Navigate to "Catalog" > "Products" and select a configurable product.
Click on "Edit Product".
Scroll down to the "Edit Prices" section.
Use the drop-down menu to select specific variants of the configurable product (for example, Black color, TOP grade, 64GB memory).
Note that other variants not matching the selected parameters are also included in the selection.
Any price changes applied will also affect these unwanted variations.
Expected behavior
The drop-down menu should only select variants that match the parameters chosen by the user. Price changes should only apply to correctly selected variants.
Screenshots

Additional context
Bagisto version: 2.0 (Master with latest fixes)
Environment: Administrator Panel
Localization: Italian (it)

@matts93
When we are seleting any opotion in the Select Varient, Let Suppose in your case you have selected
Colore-> New
Grado -> ECCELLENTE
Memoria -> 64 GB
that means it will check and verify that which all varients are having these options, if any of the option selected  like 64GB. ECCELLENTE etc, then all the varients which are having these information will appear in mass price updating section.
Hence this is not a issue.
i hope that you are clear with the concern, You may check below video for the same.
Video
https://github.com/bagisto/bagisto/assets/92721837/460f62eb-bb1b-4fc5-8267-432ac11c92fb
Looking Ahead for your reply.
Thank You. Really appreciated for your contribution.

If we select those variants, the products that contain only those variants and not one of these should be selected
https://github.com/bagisto/bagisto/assets/22118523/09ee0edc-8060-4318-a4a2-9dbdc22adb36


Optionally pass an OpenAI::Client to the Langchain::LLM::OpenAI initializer

This allows the user to pass an instantiated OpenAI::Client to the Langchain::LLM::OpenAI initializer.
This is related to this comment.

@josh-lauer Thank you for the PR. Sorry for the delay!
Thoughts on changing this to attr_accessor :client and then setting llm.client = OpenAI::Client.new(...) directly?
That would allow this type of customization on other Langchain::LLM::* classes.

@josh-lauer Thank you for the PR. Sorry for the delay!
Thoughts on changing this to attr_accessor :client and then setting llm.client = OpenAI::Client.new(...) directly?
That would allow this type of customization on other Langchain::LLM::* classes.

I thought about that, though it would mean one would need to instantiate a Langchain::LLM::OpenAI, passing it the token and having it build a client that the user would then immediately clobber. It felt clunky to me at the time. However, maybe that's ok for a rare use case like mine. Maybe it's better to not have the OpenAI LLM class not be an exception to the pattern, taking an extra argument. It's your call. If you want we can make this a one-line change.
As long as we have a way of injecting an instrumented client via an interface rather than having to use instance_variable_set or similar tricks, i'm happy.

@josh-lauer Thank you for the PR. Sorry for the delay!
Thoughts on changing this to attr_accessor :client and then setting llm.client = OpenAI::Client.new(...) directly?
That would allow this type of customization on other Langchain::LLM::* classes.

I thought about that, though it would mean one would need to instantiate a Langchain::LLM::OpenAI, passing it the token and having it build a client that the user would then immediately clobber. It felt clunky to me at the time. However, maybe that's ok for a rare use case like mine. Maybe it's better to not have the OpenAI LLM class not be an exception to the pattern, taking an extra argument. It's your call. If you want we can make this a one-line change.
As long as we have a way of injecting an instrumented client via an interface rather than having to use instance_variable_set or similar tricks, i'm happy.

Yeah, let's add attr_accessor :client if that's okay.

@josh-lauer I went ahead and added a quick fix per our conversation: https://github.com/patterns-ai-core/langchainrb/pull/78. Would you please confirm that it fixes your usecase?

@josh-lauer I went ahead and added a quick fix per our conversation: #78. Would you please confirm that it fixes your usecase?

Changing that to an attr_accessor would address my issue.  I linked the wrong PR previously. Correct one: https://github.com/patterns-ai-core/langchainrb/pull/788.

@josh-lauer Closing this PR since I merged https://github.com/patterns-ai-core/langchainrb/pull/788. Thank you! Please let me know if you have any other issues!

@josh-lauer Closing this PR since I merged #788. Thank you! Please let me know if you have any other issues!

Works for me. Thanks for taking care of this!


[BUG] Colors aren't returned to how they looked before in a tty

Describe the bug
When quitting a program in a tty, the colors (foreground and background) don't reset and the terminal looks weird. It only happens in a tty
To Reproduce (for my application)

Compile and run wiki-tui (should also happen with any other cursive program) in a tty
Quit the program
Look at your terminal

Expected behavior
The colors get reset and the terminal looks like before.
Screenshots
Screenshot from the original bug report:

Environment

Operating system used: Arch Linux (new/minimal install to test the AUR package)
Backend used: crossterm
Cursive version (from crates.io, from git, ...): crates.io (v0.17)

Additional context
The original bug report in the wiki-tui repo can be found here: https://github.com/Builditluc/wiki-tui/issues/74
I think this bug has to do with cursive because the whole color resetting thing gets handled by cursive itself.
EDIT: The readme states that custom colors are not supported in tty mode. Does this have something to do with the colors not resetting properly? If yes, how can I detect if we are in a tty so the program doesn't use custom colors?

Hi, and thanks for the report!
I suppose the native console does not support alternative screen? In any case I added a commit to explicitly reset the colors when leaving the application; does this fix the issue for you?

The issues seems to be fixed on local testing. Fix was released as part of cursive-core 0.3.6.
I'll close the issue for now, feel free to re-open if needed.


[docs] Broken link in Setting Up Your Editor - Formatting Code Automatically

Formatting Code Automatically page to see it in action  a Broken link.
But this PR seems to fix the link.
Are you forgetting to build the documentation?

We only publish the documentation when we do a release, since the docs contain new information that is only applicable to that release.


Config file doesn't have the right path on non-Windows platforms

You'll notice the vanilla_hammers directory is empty and the actual config file generated with the name vanilla_hammers\vanilla_hammers.json. Looks like you've hardcoded the \ in the path, instead of using the File/Path methods that handle non-Windows platforms properly.

Should be fixed in 1.1.0. Thanks for the report!


stat /bin/prometheus-config-reloader: no such file orr directory: unknown

k8sveriosn：1.20.1

kube-prometheus:0.8
kube-prometheus-release-0.8
我在安装的时候

发现promethues的资源对象提示缺少文件，但是我的镜像来自镜像仓库pull的
报错情况如下：

希望能够得到帮助，我需要提供什么

Container image "<IP_ADDRESS>:1603/prometheus/prometheuis:v2.26.0" already present on machine
Created container prometheus
Started container prometheus
Container image "<IP_ADDRESS>:1603/prometheus/prometheuis-operator:v0.47.0" already present on machine
Created container config-reloader
Error: failed to start container "config-reloader": Error response from daemon: failed to create shim: OCI rcuntime create
failed: container linux.go:380:
starting container process caused: exec: "/bin/prometheus-config-reloader": stat/bin/prometheus-config-reloader: no such file or directsory: unknown

I encountered this problem when upgrading, and I don't know how to solve it

0.13 same

my k8s version is 1.28
my kube-prometheus version is 0.13

I can't find any logs about alertmanager

describe pod is:

Any progress on this issue, I can see the reloader image has the bin/prometheus-config-reloader file but when it is deployed in a pod, it cannot find it.


[ALLUXIO-1600] Improve javadoc in webapp

Hello!
I just provided a tentative fix for Alluxio-1600. Please let me know if it works.
Thanks
Yanqin

alluxio-bot, check this please.

alluxio-bot, check this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15866/
Test PASSed.

alluxio-bot, check this please.
https://tachyon.atlassian.net/browse/ALLUXIO-1600
Add description for possible IOException that can be thrown from WebInterfaceBrowseLogsServlet's methods, including displayLocalFile and doGet.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15867/
Test PASSed.

Hi @riversand963 , thank you very much for contributing to Alluxio!
The change itself looks great and Jenkins is happy.
You probably want to use our latest Alluxio JIRA instead of the deprecated Tachyon JIRA. Note that https://alluxio.atlassian.net/browse/ALLUXIO-1600 is closed, can you close this PR and open another one for a separate new contributor task?
Thanks, and looking forward to your contribution to Alluxio!

Hi Chaomin,
Sure, already done.
Thanks
Yanqin

From: Chaomin Yu<EMAIL_ADDRESS>Sent: Monday, June 26, 2017 4:27:01 PM
To: Alluxio/alluxio
Cc: Yanqin Jin; Mention
Subject: Re: [Alluxio/alluxio] [ALLUXIO-1600] Improve javadoc in webapp (#5696)
Hi @riversand963https://github.com/riversand963 , thank you very much for contributing to Alluxio!
The change itself looks great and Jenkins is happy.
You probably want to use our latest Alluxio JIRAhttps://alluxio.atlassian.net/ instead of the deprecated Tachyon JIRAhttps://tachyon.atlassian.net/. Note that https://alluxio.atlassian.net/browse/ALLUXIO-1600 is closed, can you close this PR and open another one for a separate new contributor task?
Thanks, and looking forward to your contribution to Alluxio!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Alluxio/alluxio/pull/5696#issuecomment-311209568, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATKe737tinNize08yehJ-xZcc9Elj2Dtks5sID5FgaJpZM4OF5wg.


Added $CANINE_DOCKER_ARGS for streaming

Streams will now create a temporary directory in /tmp and the fifo will be created there. The localizer also maintains a $CANINE_DOCKER_ARGS variable for each job which is used to ensure dockers can still work regardless of the job setup.
Any scripts using docker should expand $CANINE_DOCKER_ARGS as part of their command
Blocks #55 (That PR should use CANINE_DOCKER_ARGS after this merges)
Closes #58

Coverage increased (+0.08%) to 59.631% when pulling 016b6f572c5b391c08613dc981faba7aa22de550 on canine-docker-args into b21d474e9d7e14a0fbbf658bb882e12b8706eefc on master.


💥  (git-cz) semver: init|breaking [build]

💽  (git-cz) fix branch-type
♻ (semantic) breaking change formatting
BREAKING CHANGE: 💥 emoji after :

:tada: This PR is included in version 13.0.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 7.0.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Add 100% scale variant to GitHub comments icon workflow

Building on @jguddas excellent work in #946, could we also include a 100% scale (24x24px) preview?

         uses: tj-actions/changed-files@v35
         with:
           files: icons/*.svg
+      - name: Generate preview
+        id: generate-preview
+        run: |
+          delimiter="$(openssl rand -hex 8)"
+          echo "body<<$delimiter" >> $GITHUB_OUTPUT
+          for file in ${{ steps.changed-files.outputs.all_changed_files }}; do
+            echo "![$file](https://raw.githubusercontent.com/${{ github.event.pull_request.head.repo.full_name }}/${{ github.event.pull_request.head.ref }}/icons/$file) "
+          done | tr '\n' ' ' >> $GITHUB_OUTPUT
+          echo >> $GITHUB_OUTPUT
+          echo "$delimiter" >> $GITHUB_OUTPUT
       - name: Generate comment
         id: generate-comment
         run: |
@@ -50,6 +60,7 @@ jobs:
           issue-number: ${{ github.event.pull_request.number }}
           body: |
             ### Added or changed icons
+            ${{ steps.generate-preview.outputs.body }}
             <details>
             <summary>Icon X-rays</summary>
             ${{ steps.generate-comment.outputs.body }}

Result (not tested)
Added or changed icons

Icon X-rays

We could also add a preview with 3px and 1px stroke width.

I wonder if we could run similar code on  issue_comments, so that we can simply paste SVG markup, and have it render in the same way…?
Something like:
on:
  issue_comment:
    types: [created]

https://github.com/lucide-icons/lucide/pull/1180

Fixed by #1180


⚠ ClamAV has degraded performance

In c14864e, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 66 ms

Resolved: ClamAV performance has improved in 2cd0452.


Translate Tests

We want to translate the tests from repy SafePOSIX to here. This test directory has the tests we want to translate.
There's probably more than we need here, at this point we should probably just translate the unit tests that being with "ut_lind_"

ut_lind_fs_broken_close.py
ut_lind_fs_chmod.py
ut_lind_fs_dir_chdir.py
ut_lind_fs_dir_mode.py
ut_lind_fs_dir_multiple.py
ut_lind_fs_dup.py
ut_lind_fs_dup2.py
ut_lind_fs_fcntl.py
ut_lind_fs_fdflags.py
ut_lind_fs_file_link_unlink.py
ut_lind_fs_file_lseek_past_end.py
ut_lind_fs_file_read_write.py
ut_lind_fs_fstat_complex.py
ut_lind_fs_getdents.py
ut_lind_fs_getuid.py
ut_lind_fs_load_fs.py
ut_lind_fs_mknod.py
ut_lind_fs_multiple_open.py
ut_lind_fs_persistance_setup.py
ut_lind_fs_persistance_test.py
ut_lind_fs_rmdir.py
ut_lind_fs_simple.py
ut_lind_fs_stat_file_complex.py
ut_lind_fs_stat_file_mode.py
ut_lind_fs_statfs.py
ut_lind_fs_truncate.py
ut_lind_net_bind.py
ut_lind_net_bind_multiple.py
ut_lind_net_bind_on_zero.py
ut_lind_net_connect_basic_udp.py
ut_lind_net_getpeername.py
ut_lind_net_getsockname.py
ut_lind_net_listen.py
ut_lind_net_poll.py
ut_lind_net_recvfrom.py
ut_lind_net_select.py
ut_lind_net_shutdown.py
ut_lind_net_socket.py
ut_lind_net_socketoptions.py
ut_lind_net_socketpair.py
ut_lind_net_udp_bad_bind.py
ut_lind_net_udp_connect.py
ut_lind_net_udp_simple.py
ut_seattlelibtests_teststruct.mix

Now that we've completed most of the calls lets make sure all of these are translated, and try to add more corner cases.

All of the file system tests are done! Will work on net tests and then add some more corner cases.

Let's try to mark our names here before we work on each test so that we don't do repetitive work! @TristanB22

Ok @kliu513 ! The branch that I am working on right now is net_tests. Not totally working yet, though! I also had to comment out the fs tests because they weren't allowing the net tests to run properly. I imagine that it is a concurrence issue.

@TristanB22 something I'd like to see soon is if we could make the tests such that we don't have to delete the entire filesystem every time we test it. Can you work on that?

@rennergade yes, I can try to get that done for you. Do you mean like removing all of the metadata files in between “cargo test”s or lindrustinit() and lindrustfinalize()’s?

@TristanB22 @kliu513 how close are we to having these finished. I think this is the last step before we merge networking and have RustPosix "finished".

Not yet, I am having issues with the server that I am trying to fix. Almost done.

Done with networking merge!


Nitrokey 3C NFC FIDO authentication failure

Security key (FIDO) authentication fails (always fails) on when using NFC interface. Authentication works fine with USB C interface.
Hardware: Nitrokey 3C
Firmware: v1.5.0-test.20230704
Device: Pixel 6 Android 14
Browser: Firefox, Chrome, Brave
Steps:

Go to google.com and select login.
Provide user and password. Select Next. This is always successful.
Prompted for security key. This step is unreliable and often takes quite some time for device to be recognized. From the Nitrokey forums, this seems to be a know issue.
When device is finally recognized, site prompts to continue to hold Nitrokey next to device.
Screen displays "There was a problem. Try using your security key again or try another way to verify it's you." This always occurs.

A YubiKey 5C NFC works fine using the NFC interface. It is also immediately recognized by my phone.

I cannot edit the initial report so some more details here.

This failures happens on all sites I've tried.
When device is recognized, screen displays a) continue to hole security key next to device; b) you're all set, you can remove the security device c) authentication failure. try again or another way.

Is there any update on this?
I am having the same issue across different devices. I tested

iPhone (different generations)
LineageOS (Fairphone 3)
Other Android phones

The firmware versions tested were:

v1.7.0
v.1.7.2
v1.7.2-test.20240813

Browsers:

Firefox
Safari
Chrome
Brave

The phones can usually read out the data of the NFC tag, but FIDO authentication using the NFC interface won't work.


x/tools/gopls: support semantic highlighting

Version 3.16 of LSP will have support for semantic tokens. We should add support for this in gopls. Syntax highlighting is currently done by the better-go-syntax repo.

This has now been implemented and is available by an opt-in semanticTokens configuration.


impossible to use 96 errors

Hello, I am trying to use your plugin to monetize, my app html5 cordova apache. I tried to create a test project, following your guide, but when I import the project in Eclipse, I returns 96 errors. can you help me figure out what is wrong.

thanks a lot.
PS sorry for my english

Check issue #169, I guess you did not have google libs installed properly.

I did not understand what to do, I downloaded all the extras, such as a guide.

The error means the google play service jar is not found, so must be something wrong with the path of sdk or configuration.
Can you build with Cordova CLI?
cordova build android

If the launch command gives me this:
C:\Users\Tore\test1>cordova build android
Running command: cmd "/s /c "C:\Users\Tore\test1\platforms\android\cordova\build
.bat""
[Error: ANDROID_HOME is not set and "android" command not in your PATH. You must
fulfill at least one of these conditions.]
ERROR building one of the platforms: Error: cmd: Command failed with exit code 2
You may not have the required environment or OS to build this project
Error: cmd: Command failed with exit code 2
at ChildProcess.whenDone (C:\Users\Tore\AppData\Roaming\npm\node_modules\cor
dova\node_modules\cordova-lib\src\cordova\superspawn.js:134:23)
at ChildProcess.emit (events.js:110:17)
at maybeClose (child_process.js:1015:16)
at Process.ChildProcess._handle.onexit (child_process.js:1087:5)
C:\Users\Tore\test1>

Error: ANDROID_HOME is not set and "android" command not in your PATH. You must
fulfill at least one of these conditions.
Quite clear: You did not configure the Android SDK env properly. Please check the android docs and set the env variable ANDROID_HOME to your android SDK folder.

I solved the problem of ANDROID_HOME, it was not set the environment variable. BUT once amount eclipse gives me the same problem in the first place my 96 errors

What happen if you use Cordova CLI to build?
Remember to start a new command line to get the valid ANDROID_HOME first.

Use version 5.1.1 Cordova CLI
The error appears only when imported into eclipse. I attach video of the steps executed
https://youtu.be/q6kfsDWDEok

If you can build with Cordova CLI, then the plugin should work.
As to eclipse, there must be some configuration missing.

I uninstalled and reinstalled sdk and eclipse. I added the environment variables.
Cordova with CLI starts. with eclipse gives me this error when I import the project

and after these other two.

I do not know what to do, I could try with Android study?

As to the errors above, clean up your workspace folder, the try again. When import the project, import the folder /path/to/your/project/platforms/android only.
The errors are very basic, find a senior developer or teacher around you and ask for help step by step, will be more efficient.

Hello, I was able to fix all the errors listed above. But now the app is closed when you start going to crash. I saw that other users have this problem:
issue #203
Can you help me?

A good issue report should include more info, like crash log, etc. Or else I don't know what happens.

As the eclipse error is resolved, I'd like to close this issue.
For same crash problem, you can discuss in that issue #203.