fix(class): Don't reconcile when harbor-class changed

Hello,
When changing Harbor custom resource annotation goharbor.io/harbor-class from class-1 to class-2, operator managing class-1 still reconcile custom resource in same time than harbor-operator managing class-2, so operators are both modifying resources in same time breaking everything.
I don't see why operator should still reconcile resources not handled by its harbor class anymore.
Thomas

Thanks @MarcelMue, yeah it looks like cert-manager installation is broken, do you want me to take a look ?

Hello 👋
Are you sure that the following use-case will be handled successfully?

Existing harbor resource with class harbor-1.
Patch the harbor resource (in a single request):

Change the class to harbor-2
Update the core spec

A while ago it did not.
Other question in mind, what about sub-resources, is operator managing the class harbor-2 will be able to take ownership of resources created by the other operator?
IMO the first operator needs to be triggered so it can cleanup resources. If I remember well, the original plan was that the first operator get the patched resource at the beginning of the reconciliation loop, and delete the sub-resources (core, registry, ...) because it can not find the harbor resource for its own class.

Hello 👋
Are you sure that the following use-case will be handled successfully?

Existing harbor resource with class harbor-1.
Patch the harbor resource (in a single request):

Change the class to harbor-2
Update the core spec

A while ago it did not.
Other question in mind, what about sub-resources, is operator managing the class harbor-2 will be able to take ownership of resources created by the other operator?
IMO the first operator needs to be triggered so it can cleanup resources. If I remember well, the original plan was that the first operator get the patched resource at the beginning of the reconciliation loop, and delete the sub-resources (core, registry, ...) because it can not find the harbor resource for its own class.

Hello @holyhope,
Thank you for your feedback, I'll do more tests and come back to you :)
Thomas

Hello,
Sorry for the delay, been quite busy.
I made some tests and it looks like your use case is working. Do you remember why it didn't work in the past ?
About the question about subresources ownership, I only tested with subresources CRs directly so I can't answer about what happens if we change harbor class on Harbor CR. BUT so that new operator modify labels on its managed resources (deploy, secrets, cm...) I needed to modify the checksum function to take into account harbor class and not only spec (see my last commit).
Finally, concerning your last point I agree that we should have a way to clean resources managed by old operator not needed anymore by new operator but it seems it's not implemented like that for now because when changing harbor class, the handler only tries to reconcile resources, not delete them. Moreover even if it tried to delete its resources it may have conflicts because the new operator will try to create the resources with the same name.
So should we merge it it as it is even if it's not perfect it's still better that previous situation, or may we cancel it and let's see it later ?
FYI: We don't need to change Harbor class anymore, we took another solution for our use case so we don't need this feature to work on our side.
Thomas


feat(raft): introduce read index

Signed-off-by: noneback<EMAIL_ADDRESS>

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Draft. I only change the enforce-related code to see if it fits.

@noneback don't make WIP PR. First work in your local branch and then create the final PR

I need to spend some time for review this PR.

nodece

Thx for your patience！ You can review it at your convenience.

Well done, But I consider we still need to add tests for this PR. Before adding tests, we need to decouple the main function's parsing flags(the relevant issue #58 ), which makes integration testing more manageable. I will handle this issue later this week. If you are willing, you can make a decoupling of the parsing flags PR, then add tests for this PR.

Thanks for your contribution, most of the work is great, I left my comment.

Thx. I will fix them all in my spare time.

@noneback can you continue pushing this PR? Also resolve the conflict:

Sure. I will fix conflict and try to add some test

/cc @nodece @WenyXu


Fix spacing issues, Badge to spec

Update of #1132:
Fixes spacing issues,
Also think i may have made the badge fit the spec now:

Currently the left label is placed in the middle (5px either side) as x position is calculated as width/2 instead of (width/2)+1 to give 6 px on the left and 4px on the right.
on the textLength I originally didn't account for the total padding of 10px i for some reason thought it was 4px each side.
Heres some test:
Windows 10 - Chrome 61

This PR
Production

Windows 10 - Firefox 56

This PR
Production

Notes:

the Codetally badge shows up weird because for some reason it has an extra space at the beginning, should probably strip all outer spaces at some point in another PR
the gem badge (rank) also has extra width due to when the width is calculated it counts the ᵗʰ as being a standard th
no idea whats up with the social badge, not sure what the spec is supposed to be with those.

Thanks for picking this up. 😃
master (37bcddb2):

this PR (8c9699d):

Indeed, these are a bit different, though you're right that they better match the spec!
Will leave this open a bit in case anyone else would like to comment. Meanwhile would you mind merging master into this branch and resolving the conflicts, so I can merge through the UI?

Made a quick test page,
Will add some more badges to it soon,
https://shields.redsparr0w.com/1161/
Also just realised ?style=plastic does not support logos,
https://img.shields.io/github/issues/detail/s/badges/shields/979.svg?style=plastic&logo=github

Thanks. This also resolves #671 then!
Do you think you could articulate some of the invariants in the badge design so we could express them in code? e.g. width of the left side === the left text width + x, first y pixels are gray followed by a white pixel, the height is z, etc. I'd love to get some more tests written. There's no reason to hold up this on more tests. Though perhaps it would be easier to identify the invariants while they're fresh in your mind.
Also… I'm finding the long math expressions in the templates pretty opaque. I might ask for your help sometime to move some of the logic from the template into the code.

Do you think you could articulate some of the invariants in the badge design so we could express them in code? e.g. width of the left side === the left text width + x, first y pixels are gray followed by a white pixel, the height is z, etc. I'd love to get some more tests written. There's no reason to hold up this on more tests. Though perhaps it would be easier to identify the invariants while they're fresh in your mind.

Not quite sure what you mean sorry 😆

Also… I'm finding the long math expressions in the templates pretty opaque. I might ask for your help sometime to move some of the logic from the template into the code.

Yeah most of the template values could be calculated before,
But some of them are probably best left in the template in case more templates are added later that follow a different layout?
The main one i can see that would work is the textLength as that is what the badges width is calculated from originally anyway,
Another thing i've just noticed badge width is rounded up to the nearest even number which would change the look if an odd number was calculated for text length:
6pxLabel5px 4pxData7px

But some of them are probably best left in the template in case more templates are added later that follow a different layout?

I think we could structure the data along these lines:
{
   layout: {
      standard: {
         leftWidth: ...
      },
      social: {
         leftWidth: ...
      }
   }
   text: [...]
}

Alternatively the code could switch on the format, and provide only the data that template needs.
If we add another template we could add additional math logic along with it in the .js file. The more of this math logic is in code, the easier it should be to avoid duplicating it.

That would work,
Sounds like a good plan,
Would definitely make it easier to manage the look of the badges as well.

Not quite sure what you mean sorry 😆

To clarify, I'm thinking something like this:
describe('The badge spec', function () {
  it('The badge is the expected height', ...
  it('The left side of the badge has the expected amount of left and right padding', ...
  it('The right side of the badge has the expected amount of left and right padding', ...
});

No particular urgency, just something I'd much rather get into code than test manually!

This change got deployed over the weekend. Yay!
I noticed the spacing on this social badge looks off.

Do you know what might be causing it?

Yeah it does look a little strange

Before
After

The possible problem could be something with this line it.widths[1]-=4; which is pushing the text closer together by 4px
Best guess is an old fix for some spacing issues?
If it's changed to be -=2 or completely removed and shrink the textLength it doesn't look too bad.
 - (master) without it.widths[1]-=4; line
 - (master) without it.widths[1]-=4; & textLength-10px (right label 5px padding)
 - (master) changed to it.widths[1]-=2; (right label 4px padding)
 - current live badge
 - commit merged just before this PR was merged

Yea, it's like the left side got extended and the right side got compressed.
It seems to me that when the computed length is correct, lengthAdjust="spacing" should have no effect. How about seeing what kind of width adjustment looks the same, with or without that?


Fixed bold

Docs Build status updates of commit 3c2ef5c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/resources/schema/manifest-schema.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Import from deckstats.net is not correctly pulling the correct card image for the Kaladash Inventions Sets

The set is just whatever set it happened to pick as far as I can see.

I've been unable to get the program, to pull set specific art from any website, always just the latest art.

@Aznlilly86 Can you tell me which URL or deck file you're trying to import exactly? Is it this one?
@LoardVader If the set is not specified by the deck, the art from the latest set the card was released in will be used. It depends if the website sets the set name or not for each cardwhen exporting a deck. Which website do you have an issue with?

By the way, if you're using the From text tab instead of From URL, you can use this method to pull specific art for a card.

I was using the deckstats.net site, here is an example deck: https://deckstats.net/decks/161156/1740667-fblthp-proteus-scepter-combo

i've been using "From URL" on scryfall

@Aznlilly86 @LoardVader Thanks, I see the issue now. It seems that deckstats.net uses some set names (the ones with an underscore, like MPS_KLD for Kaladesh Inventions) that are different from the names used by Scryfall (which expects MPS for Kaladesh Inventions).
I made a manual fix for these set names in 0f675771f351815537b6906e662a6a4e6637b517, which seems to be working:

It will be included in the next release, but in the meantime you can test it here (if you're on Windows).

The newly updated version works flawlessly for me so far.
On Sun, Sep 6, 2020 at 9:57 AM Alexis Jeandeau<EMAIL_ADDRESS>wrote:

@Aznlilly86 https://github.com/Aznlilly86 @LoardVader
https://github.com/LoardVader Thanks, I see the issue now. It seems
that deckstats.net uses some set names
https://deckstats.net/sets/?lng=en (the ones with an underscore, like
MPS_KLD for Kaladesh Inventions) that are different from the names used
by Scryfall https://scryfall.com/sets (which expects MPS for Kaladesh
Inventions).
I made a manual fix for these set names in 0f67577
https://github.com/jeandeaual/tts-deckconverter/commit/0f675771f351815537b6906e662a6a4e6637b517,
which seems to be working:
[image:<PHONE_NUMBER>2809_1]
https://user-images.githubusercontent.com/23304955/92327342-ddc8bb00-f093-11ea-85a7-276f62520331.jpg
It will be included in the next release, but in the meantime you can test
it here
https://github.com/jeandeaual/tts-deckconverter/suites/1151090358/artifacts/16592224
(if you're on Windows).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jeandeaual/tts-deckconverter/issues/5#issuecomment-687796502,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACSIQESARLKPC6W7EPY7Q53SEOIMDANCNFSM4QPHYLTA
.

I released this fix as part of v0.6, so I'm closing the issue.
Please feel free to re-open it if it doesn't work for you.


Expanding TLV lib

This PR seeks to make progress toward building out the TLV library (type-length-value) to be more robust and cover a wider range of generic use cases for TLV.

Moves PodSlice and associated types/functions into type-length-value, rather than tlv-account-resolution
Cuts unnecessary dependencies and trait impls in favor of spl-program-error

The PodSlice types I felt are extremely useful and might be useful to other TLV use cases - beyond just account resolution. It's my belief that they should be included in the standard library. Open to feedback!

I'm assuming that everything is just copy-pasted, so I didn't take a deep look. But it looks good! We might just want to move all of the pod types into their own library at some point...

Yes, copy-pasta'd !
Agreed a Pod lib might be useful/appropriate in the future but I vote waiting until we can identify more use cases ie. zero-copy stuff


fix: upgrade to version 3.0.697

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign jenkins-x-labs-bot
You can assign the PR to them by writing /assign @jenkins-x-labs-bot in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
f517e2ecb74c603dd4c5ee475b1e4127a1fd59c1
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
f517e2ecb74c603dd4c5ee475b1e4127a1fd59c1
link
/test vault

kube
f517e2ecb74c603dd4c5ee475b1e4127a1fd59c1
link
/test kube

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
f517e2ecb74c603dd4c5ee475b1e4127a1fd59c1
link
/test vault

kube
f517e2ecb74c603dd4c5ee475b1e4127a1fd59c1
link
/test kube

gsm
f517e2ecb74c603dd4c5ee475b1e4127a1fd59c1
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
f517e2ecb74c603dd4c5ee475b1e4127a1fd59c1
link
/test vault

kube
f517e2ecb74c603dd4c5ee475b1e4127a1fd59c1
link
/test kube

gsm
f517e2ecb74c603dd4c5ee475b1e4127a1fd59c1
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

kube
f517e2ecb74c603dd4c5ee475b1e4127a1fd59c1
link
/test kube

gsm
f517e2ecb74c603dd4c5ee475b1e4127a1fd59c1
link
/test gsm

vault
f517e2ecb74c603dd4c5ee475b1e4127a1fd59c1
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


sar_tas_protocol_server breaks after multiple uses

on the first time you enable it, it works fine but if you use it again the vscode extension won't be able to connect to it anymore unless you restart the game

Should be fixed by b56f32f

@rudkoLA can you confirm? (ci build)

yes works


Triton&Flashibi in examples/llm

This pr integrates the triton implementation of FlashAttn (with ALiBi) into examples/llm.
Using this variant, memory usage is the same

it runs slightly faster

and with Alibi convergence is faster (and we converge to a lower final loss)

Meta-comment: MosaicGPT is currently structured so that the Causal Attn Bias is a member of the layer. It should really be a member of the model and should then be passed to all layers in their fwd pass. FlashAttn does not take a Bias / Mask so it does not suffer the memory penalty of having multiple masks. The Torch and Triton implementations do therefore are affected by the issue. @abhi-mosaic maybe we should create an issue to track this. We could make it part of this PR, but its slightly outside the scope.
Note: I tried a symlink bert/src/flash_attn_triton.py -> llm/src/flash_attn_triton.py but the result was a copy pasta; I'm not sure git supports symlinks...

waiting for https://github.com/mosaicml/composer/pull/1854

cc @abhi-mosaic

refactor

Probably a good idea if / when we create a 'common' folder which has (handles) attn, flash attn, triton flash attn, alibi etc

@alextrott16

how much re-building of the attention mask happens on each forward call

The generic and alibi attn masks are built and part of the model (shape: 1 x 1 x 1 x SeqLen or if alibi is used shape: 1 x heads x 1 x SeqLen; broadcasted to shape: 1 x 1 x SeqLen x SeqLen internally in the kernel; causal=True takes care of the causal portion of the mask)
These masks are only extended to shape: BatchSize x 1 x 1 x SeqLen when key_padding_mask says some of the tokens are invalid (only happens at eval since durring training group_method: concat).
As a result, yes "re-building is minimal", I "re-use pre-computed masks" as much as possible.


[Event Request] Codeunit 5708 - OnAfterReopenTransferDoc

Add new integration event in Codeunit 5708 - see attached files for details
COD5708 upd.txt
COD5708 orig.txt

Closed because already added.


question: How to bypass URI

A clear and concise description of what you want to accomplish.

I want to bypass /api and some other paths from my policy
Before with caddy-auth I had this in my caddyfile; (worked fine)
qbit.website.com {
    import tls
    authorize {
        bypass uri prefix /api
        bypass uri prefix /command
        bypass uri prefix /query
        bypass uri prefix /login
        bypass uri prefix /sync
        set auth url https://auth.website.com
        allow roles authp/admin
		validate bearer header
		inject headers with claims
    }
    encode zstd gzip
    import headers
    reverse_proxy qbittorrent:8080
}

Now with caddy-security Im trying to achieve the same thing, so I looked here:
https://authp.github.io/docs/authorize/bypass
    security {
		authentication portal myportal {
	     crypto default token lifetime 86400
             cookie lifetime 172800
             cookie domain website.com
			 backend local /config/auth/local/users2.json local
             ui {
                 links {
                      "Flame" https://flame.website.com/ icon "las la-star"
                 }
             }
			 transform user {
                match email<EMAIL_ADDRESS>				action add role authp/admin
			}
        }
		authorization policy admins_policy {
            bypass uri prefix /api
            bypass uri prefix /command
            bypass uri prefix /query
            bypass uri prefix /login
            bypass uri prefix /sync
            allow roles authp/admin
            set auth url https://auth.website.com/
			acl rule {
				comment default deny
				match any
				deny log warn
			}
		}
    }

This however does not work, the reverse proxy stops working after auth is done (shows a blank page), also log spams with:
{"level":"error","ts":1643842456.2998343,"logger":"http.handlers.authentication","msg":"auth provider returned error","provider":"authorizer","error":"user authorization failed: src_ip=IP, src_conn_ip=IP, reason: user data not found"}
{"level":"error","ts":1643842194.8351867,"logger":"http.handlers.authentication","msg":"auth provider returned error","provider":"authorizer","error":"user authorization failed: src_ip=IP, src_conn_ip=IP, reason: no token found"}
Am I not understanding the bypass function? Or is it a better way to do this? Thanks

@abbekeff , please re-post your config after you ran formatting over it.
bin/caddy fmt -overwrite /path/to/Caddyfile

What is your current configuration for qbit.website.com?
Do you have order configured? (relevant explanation is https://github.com/greenpau/caddy-security/issues/34#issuecomment-1024293316)
order authenticate before respond
order authorize before basicauth

{
	http_port 8282
	https_port 4443
	order authenticate before respond
	order authorize before basicauth
	email "email"
	dynamic_dns {
		provider cloudflare API
		domains {
			website.com
		}
		check_interval 30m
		ip_source simple_http https://icanhazip.com
		ip_source simple_http https://api64.ipify.org
	}
	security {
		authentication portal myportal {
			crypto default token lifetime 86400
			cookie lifetime 172800
			cookie domain website.com
			backend local /config/auth/local/users2.json local
			ui {
				links {
					"Flame" https://flame.website.com/ icon "las la-star"
				}
			}
			transform user {
				match email "email"
				action add role authp/admin
			}
		}
		authorization policy admins_policy {
			bypass uri prefix /api
			bypass uri prefix /command
			bypass uri prefix /query
			bypass uri prefix /login
			bypass uri prefix /sync
			allow roles authp/admin
			set auth url https://auth.website.com/
			acl rule {
				comment default deny
				match any
				deny log warn
			}
		}
	}
}
(tls) {
	tls {
		issuer zerossl API{
			dns cloudflare API
			resolvers <IP_ADDRESS>
		}
	}
}
(headers) {
	header {
		Strict-Transport-Security "max-age=31536000; includeSubDomains; preload"
		X-Frame-Options "SAMEORIGIN"
		X-Content-Type-Options "nosniff"
		Referrer-Policy "strict-origin"
		Content-Security-Policy "upgrade-insecure-requests"
		X-Robots-Tag "noindex, nofollow, nosnippet, noarchive"
	}
}

auth.website.com {
	log {
		output file /config/logs/access.log
		level WARN
	}
	import tls
	import headers
	route {
		authenticate with myportal
	}
}

flame.website.com {
	import tls
	encode zstd gzip
	import headers
	route {
		authorize with admins_policy
		reverse_proxy flame:5005
	}
}

radarr.website.com {
	import tls
	encode zstd gzip
	import headers
	route {
		authorize with admins_policy
		reverse_proxy radarr:7878
	}
}

sonarr.website.com {
	import tls
	encode zstd gzip
	import headers
	route {
		authorize with admins_policy
		reverse_proxy sonarr:8989
	}
}

prowlarr.website.com {
	import tls
	encode zstd gzip
	import headers
	route {
		authorize with admins_policy
		reverse_proxy prowlarr:9696
	}
}

qbit.website.com {
	import tls
	encode zstd gzip
	import headers
	route {
		authorize with admins_policy
		reverse_proxy qbittorrent:8080
	}
}

jellyfin.website.com {
	import tls
	import headers
	reverse_proxy jellynightly:8096 {
		flush_interval -1
	}
}

@abbekeff , please enable debug in global config and try debugging. Please email me the logs.
Also, for troubleshooting, please configure static keys:
authentication portal myportal {
  crypto key sign-verify 39a7ed93-6341-4232-a4fe-c44d242eca7d
}

authorization policy admins_policy {
  crypto key verify 39a7ed93-6341-4232-a4fe-c44d242eca7d
}

@abbekeff , for troubleshooting, amend your ACL:
from:
			allow roles authp/admin
			acl rule {
				comment default deny
				match any
				deny log warn
			}

To:
      acl rule {
        match role authp/admin
        allow stop log debug
      }
      acl rule {
        comment default deny
        match any
        deny log warn
      }

Also, try without route:
flame.website.com {
	import tls
	encode zstd gzip
	import headers
        authorize with admins_policy
	reverse_proxy flame:5005
}

@greenpau
I tried all your suggestions, same issue still.
I have logs for you, I cant find your email. where do I send them? (I'm new to github)

<EMAIL_ADDRESS>

Release fix in v1.0.13.


scylla_conifgure.py: skip construct data volume on dev_instance_type

Since dev_instance_type does not have local SSD, we should skip running scylla_create_devices.
Fixes #551

jenkins test passed: https://jenkins.scylladb.com/view/master/job/scylla-master/job/releng-testing/job/next-machine-image/384/

@syuu1228 @yaronkaikov
there is a specific test for t3.micro
and it seems to be running just fine, on master and on 2024.2:
https://argus.scylladb.com/tests/scylla-cluster-tests/8885cca7-19dd-478b-88fd-4a0f3347ddf4
I would recommend checking out why it's not catching the issue encounter, and adapt the test to catch it.

I realized that extra disks may attached to the dev instance (at least SCT test does it), changed the code to handle it: https://github.com/scylladb/scylla-machine-image/compare/39213f94e52694846b218ff0ba6cf6c629bf60f9..25e13c4f217535ee9b60e7345aafb64137506c5a
And testing it now.

I realized that extra disks may attached to the dev instance (at least SCT test does it), changed the code to handle it: https://github.com/scylladb/scylla-machine-image/compare/39213f94e52694846b218ff0ba6cf6c629bf60f9..25e13c4f217535ee9b60e7345aafb64137506c5a
And testing it now.

Test passed.
https://jenkins.scylladb.com/view/master/job/scylla-master/job/releng-testing/job/next-machine-image/385/
https://jenkins.scylladb.com/view/master/job/scylla-master/job/releng-testing/job/artifacts/job/artifacts-ami-t3_micro-test/13/

Maybe there should be a test that shows scylla working with adding an extra disk  ?
As in the original failure in issue this PR  is fixing ?

@fruch IIUC the the artifact test we have in SCT, it must have an extra disk:
data_volume_disk_num: 1  # t3.micro doesn't include any local volumes (outside of root), so an additional disk is required for scylla

Which I assumed was a requirement for it to work.
@syuu1228
What is the current claimed problem? that it doesn't work without extra disk?


cat: /tmp/htpwd_ip_patch.json: No such file or directory

When we install jenkins role top of openshift where identity provider htpasswd is not installed, then we get this error :
TASK [install_jenkins : Get patch file] ***********************************************************************************************************************************************************************************************************
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": true, "cmd": ["cat", "/tmp/htpwd_ip_patch.json"], "delta": "0:00:00.004362", "end": "2018-05-07 08:10:05.887029", "msg": "non-zero return code", "rc": 1, "start": "2018-05-07 08:10:05.882667", "stderr": "cat: /tmp/htpwd_ip_patch.json: No such file or directory", "stderr_lines": ["cat: /tmp/htpwd_ip_patch.json: No such file or directory"], "stdout": "", "stdout_lines": []}

Additional error reported by jenkins role, if the Centos VM is running locally and identity provider htpasswd defined is
TASK [restart_master : Check and see if a custom service exists] **********************************************************************************************************************************************************************************
fatal: [<IP_ADDRESS>]: FAILED! => {"changed": true, "cmd": ["systemctl", "status", "dummy"], "delta": "0:00:00.009039", "end": "2018-05-07 08:37:13.825487", "msg": "non-zero return code", "rc": 4, "start": "2018-05-07 08:37:13.816448", "stderr": "Unit dummy.service could not be found.", "stderr_lines": ["Unit dummy.service could not be found."], "stdout": "", "stdout_lines": []}
...ignoring

That error is to be expected, that's why it's being ignored :)


Fix unrecognized selector error for 'taskIdentifier' and 'originalRequest' methods

I'd love to get other folks eyes on this before merging it, as I don't maintain an up-to-date DVR test suite anymore.
@soffes @hyperspacemark or other contributors, would you mind reviewing this as well?

Hi @nebiros , could we update this PR with a description of the problem, the expectation of performance, then how the fix tackles that?

@dmiluski sure, I update the description

@dmiluski @eliperkins ping


feat(backend): add GNOME login support

Pretty simple as just another form of GitLab login.
Not familiar with SQLAlchemy at all, so looks like quite a lot of repeated code to me, but as far as I can tell just a necessary evil since we need unique tables for these.
Frontend has no login logo in this PR, the vertical SVG here might be suitable, but has a unique aspect ratio compared to the others: https://foundation.gnome.org/logo-and-trademarks/

Looks good to me.


Integrate usage statistics

Based on the invenio-stats module
Configured on Zenodo to...

Support views/downloads tracking across individual/all record versions
Adhere to the COUNTER Code of Practice and Make Data Count Recommendations

Statistics are used for reporting in the OpenAIRE-NEXUS and BiCIKL projects
Statistics are sent directly to the OpenAIRE Usage Counts project
Views and downloads are pre-computed and index in ElasticSearch on a per-record basis

Prerequisites

Record versioning

Next steps

Module review and study of integration into InvenioRDM
a. Identify hook-in point, i.e. view/service layer, or event-based (it used to be signal-based on specific views)
b. Indexing of pre-computed statistics into records serialization (REST API)
c. Landing page display design
Data migration for historical data

Current

Future

@slint could you confirm we covered the whole scope? (I have moved migration to separate issue_

Statistics are used for reporting in the OpenAIRE-NEXUS and BiCIKL projects
Statistics are sent directly to the OpenAIRE Usage Counts project

does it mean that we need to implement any further integrations

Statistics are used for reporting in the OpenAIRE-NEXUS and BiCIKL projects
Once we migrate usage statistics data to ZenodoRDM we'll have to check if we can reproduce our reporting for these projects.

Statistics are sent directly to the OpenAIRE Usage Counts project

This is planned for MS8 from what I see, and has a separate task already.
Otherwise, scope-wise we're good. By finishing the sorting by views we should also be able to have the statistics data served for each record via the REST API as well.


apply cql files from directory

@h1alexbel thank you for reporting this. I'll assign someone to take care of it soon.

@h1alexbel this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2023-08-20T11:49:09.117604.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@h1alexbel thank you for resolving this ticket. I've just added it to your active invoice. You can always check all your invoices and more on the Contributor Dashboard.


team: another retry condition on chain append races

No, i think this case client A is trying to append a team deletion with key rotation, but it didn't get the team rotation via hidden chain, so it sent up the wrong PTK generation... I think...

I don't know why this didn't fail the total order check :/

Wait, it did fail the total ordering check:
2019-07-29 21:42:31.36244 api.go:883             : [D] - API call sig/multi error: cannot make a total ordering for ["aac0978484d167a670ee21ff93572c24",18,1] (error 1022) [tags:TEST=CjnMqNc5M2mv,API=qgZh4HDm653E]
2019-07-29 21:42:31.36244 service_helper.go:337  : [D] - AddMemberTx.Post -> cannot make a total ordering for ["aac0978484d167a670ee21ff93572c24",18,1] (error 1022) libkb.AppStatusError [tags:TEST=CjnMqNc5M2mv]

FSWfkweporiwer
This isn't the right fix.  I'll keep working on it

OK, @mlsteele, this is the right fix, pull the "ratchet" from the loaded team, and not from the global loader.  This'll catch the bug we saw at the total ordering step, not way later in the PTK gen step.


Add field renaming to the LSP

Following up on #4734
Implement instance variable (field) renaming on the LSP. We have symbols associated to fields, so the implementation follows the exact same pattern as the one for classes and methods.
The only thing that is different is checking whether we're renaming the attr_* or the instance variable itself (the first not requiring the @ symbol and the second requiring it). That is,
class Person
  attr_reader :age # when renamed, should never have `@` in front

  def initialize
    @age = T.let(1, Integer) # when renamed, should always have `@` in front
  end
end

Question:

I was not 100% sure if class variables are considered fields. Maybe we can add support for them as well in this PR?

Motivation
To further expand our rename support and improve the editor experience.
Test plan

See included automated tests.

I was not 100% sure if class variables are considered fields

They are modeled as static fields, and you can verify this with -p symbol-table:

But they might come back as FieldResponse's in LSP, I'm not sure. I'm fine punting on them if the changes in this PR don't work, but I would like to have the test either way.


Investigate issues with nested code blocks in wd.io markdown

Observed issues with this markdown: https://github.com/webdriverio/webdriverio/blob/main/CONTRIBUTING.md
Please make recommendations for follow-up actions.

TLDR; Categories of issues:

Miss representation of code blocks as markdown.
Possibly related issues with incorrect lang guesses.
Issues with unnesting blocks, most commonly incorrect retain of white spaces prefix.

There are a few issues here:

This block is reinterpreted in the extension as Markdown. I haven't found yet why because runme json returns "executable": "sh".
It is incorrectly rendered due to whitespaces before the closing backticks:```sh
    $ git clone git@github.com:<your-username>/webdriverio.git
    ```

This can be fixed in the parser. The opening backticks also have a prefix with spaces but it is removed.

This is a problem with nested blocks. The top-level block, a list, was kinda split correctly but the nested blocks' (a list item) source retained prefixing white spaces. And prefixing a paragraph with at least four spaces in Markdown tells it to interpret it as code (not a fenced code block but regular code block). This is to be fixed in the parser.

Original source:
- *Dependency Checks*
  We automatically check if every sub-package has all the dependencies from its `package.json` installed.
  You can manually trigger this check by calling:
  ```sh
  $ npm run test:depcheck
  ```

Again, the code block is interpreted as a markdown block. The new line after * [...]* is actually dismissed in any Markdown renderer so it's not an error.

Mixed results from the language detector.

Is this done @adambabik? I suppose with the parser/renderer rewrite we'll be starting over on testing for edge cases?

@sourishkrout yeah, we need to verify these edge cases visually again. After that I will rebuild acceptance tests for deserializer which are gone in my development branch as the output changed significantly.
I have a bunch of unit and integration tests and verified the most problematic files using runme fmt. There are minor visual differences which I started collecting in https://github.com/stateful/runme/issues/66. However, runme fmt is a part of equation as the complete flow works as follows: byte array --> AST --> Intermediate Representation (it uses the same functions as runme fmt) --> cells. We don't have enough verification of the IR --> cells tranformation yet. Problems described here are a combination of errors from the IR and IR --> cells.

I am closing this issue as done. The file renders correctly using https://github.com/stateful/vscode-runme/pull/81/commits/2169911c39f1a18af0592992f9840a44415dad9d. Using runme fmt yields a bunch of differences but most of them is covered in https://github.com/stateful/runme/issues/66.


draft: Task table optimizations

Description
Currently this draft show cases:

the addition of a parameter on the /tasks and /task/{taskId}/status endpoints respectively (the parameter name lite is a placeholder for now)
the addition of a new class to capture the state we'd like to communicate to the UI and any client that queries the endpoints with the new parameter
new postgres queries that retrieve the nested fields of the new JSONB columns rather that the whole payload

Things that are incomplete and thus make this PR not appropriate to merge just yet:

missing tests for postgres (limited manual testing has been performed)
logic around the creation of the new JSONB columns when upgrading from a Druid version that supports only BYTEA columns for task payloads.
further specialization of the liltering of tasks based on their status at the DB side (instead of the web server side at OverlordResource).

Key changed/added classes in this PR

MyFoo
OurBar
TheirBaz

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

We've gotten the improvements from upstream. Closing here


Unexpected results from compare operators.

After span's comparison functions have been made free functions in release 0.9.0, they now give incorrect results when comparing spans differing in constness:
char data[] = {'a', 'b'};
gsl::string_span a = gsl::as_span(data);
gsl::cstring_span b = gsl::as_span(data).last(1);

assert(a != b); // previously worked as expected, now converts to bool and compares equal!

Resolved by issue #33 and commit 53dfc40d70e04783538deebeef908587e4530a95:
Allow comparison of spans only differing in cv-ness
Note: The C++ standard does not allow to compare containers that only differ in cv-ness.
This also holds for M-GSL. However M-GSL does allow to compare string_span with cstring_span.
The comparation of string_span in GSL Lite (as it is now) is the same as that of span.
To continue allowing to compare string_span with cstring_span in GSL Lite, it is now possible to compare spans differing in cv-ness.
Perhaps a configuration flag that allows to switch between this behavior and the strict behavior is in order.


Break down authors per month chart to per org

This is the first example of drawing stacked charts.
It supports tracking multiple orgs, not just Starburst, but the members table is only populated manually for the starburst org. It also doesn't represent membership over time, so when someone joins a new org (starburst...) all their historical commits will count toward this org.
Membership for commit authors is tracked using:

when building the idents table, git commits are joined to GitHub commits that also include GitHub users for author and committer, so all possible logins for an ident are attached
GitHub commits are updated on a regular basis; git commits are fetched by cloning the repo on every run
org membership has been fetched once, manually, for a single org, and is not tracked over time

Maybe we should also annotate orgs in the top10 authors/mergers reports.


test HVM against a naive implementation of its rewrite rules

a good way to test if something is correct is to compare its outputs to a reference implementation which is easier to check for correctness manually.
this uses proptest, which HVM was already using, though was missing from the Cargo.toml, so i added it back, like in #200, except I also changed the tests so that they pass by taking into account the pre-compiled rules.
it heavily relies on #237 for directly manipulating Terms, instead of just evaluating strings. so it current contains it commits.
the naive implementation currently does not handle constructors, and consequently no user-generated rules, so proptest doesn't generate those. i skipped handling let expressions since the way it is used in HVM is basically syntactic sugar for application on a lambda, not sure why it isn't actually sugar.
i made sure to maintain linearity while normalizing terms and only undo it at the end, in case some of the rules depend on this property. didn't maintain globally unique names since all expressions are scoped in the AST, so there is no ambiguity we substituting.
this is a draft to get some general feedback on this approach, and to see if this is something you'd be interested in.

Yes, it makes a lot of sense to do this. I was also thinking that it makes sense to have a high-level/research variant of the HVM which is not optimized for performance but is instead optimized for

readability / making it easy to reason about its code (and formally verifying its correctness)
having as little unsafe code (or micro-optimizations that reduce readability) as possible
quicker prototyping of different new ideas/features, such as new node types / new rewrite rules or static analysis / compile-time inet transformation passes (which is where a lot of performance gains can come from, with a proper IR)

And then having a test suite (with proptests & manual/regression tests) in place to ensure that the hand-optimized version of the HVM has the same semantics as the high-level/research version.
Also fuzz testing the HVM to find terms on which it doesn't behave as expected, e.g. to catch things like https://github.com/HigherOrderCO/HVM/issues/232 or https://github.com/HigherOrderCO/HVM/issues/183.
Also it could make sense having a test suite that compares HVM's output to a lambda calculus evaluator and compare their costs to catch things like https://github.com/HigherOrderCO/HVM/issues/60.
We could also use randomly generated terms to find terms where the lambda-calculus-reduced result differs from the interaction-net-reduced result, to get a clearer picture of the set of terms for which they differ (for addressing https://github.com/HigherOrderCO/HVM/issues/44).


Add pinned vendor/ dependencies

Could you please add a short section or command line that can be used to update & maintain the vendored deps?

Here's a quick intro on how to set up govendor, the dependency management tool we're using. https://github.com/mesosphere/mesos_exporter/wiki/Dependency-management

Thanks! :)


rolling calculations issue

Hi all , Hi Kevin
I have run in to the below error while applying a window calc to one of the features implemented within the lib
it goes as
df['roll_entp_10']= df_test.ENTP_10.rolling(10)   which is a series , after explicitly turning in to a series still getting the same error

just for reference
pd.version  1.0.3

I have looked though in to the following as suggested
https://github.com/pandas-dev/pandas/issues/11704
is there any additional guidance you can provide on that issue ?
best regards

Hello @ashraf-01,
Pandas Version

I have looked though in to the following as suggested
pandas-dev/pandas#11704

You are using Pandas version 1.0.3. So, did you upgrade Pandas v 1.1.x per the suggestion?

Pandas Moving Window
Also usually when you use, df.rolling(window), one applies a function to the window such as sum(), mean(), et al as described here: Standard Moving Window Functions. Also type(df.rolling(window)) = pandas.core.window.rolling.Rolling and not Series.
Your Code

df['roll_entp_10']= df_test.ENTP_10.rolling(10) which is a series , after ...

I am also running Pandas 1.0.3 and I have no issue with using rolling(window), see the screenshot below. Are you unable to show the code that led up to and including df['roll_entp_10']= df_test.ENTP_10.rolling(10) to see if you introduced logic that would cause that not to work?

Best Regards,
Kevin

Hello @ashraf-01,
I will assume the prior response was sufficient and will close this issue in a few days if I do not hear back from you.
Best Regards,
KJ


Verify that clustermesh externalworkloads command still works under Helm mode

One subcommand which has not yet been tested under Helm mode, and appears to require no Helm-related changes is the clustermesh externalworkloads subcommand.
This subcommand is used to configure Cilium on a a machine which is not a member of a Kubernetes cluster, in order to participate in a clustermesh. It is not particularly easy to test this command interactively, as it requires the availability of some machine or virtual machine which is a not a member of a clustermesh but can connect to all instances of clustermesh-apiserver in the clustermesh.
The goal of this task is to set up such a machine and verify that it can still connect to the clustermesh after the mesh has been esablished using the new Helm-based implementations of enable and connect using the new methods for CA-cert generation and distribution.

One aspect to be addressed here concerns the fact that support for external workloads is not enabled by default (which provides a few advantages IMHO, including possible better performance: https://github.com/cilium/cilium/pull/25259). clustermesh externalworkloads should likely check for that and enable it accordingly.

PR is up here https://github.com/cilium/cilium-cli/pull/1640

completed in https://github.com/cilium/cilium-cli/pull/1640


Finetuning results reproduction

Description
Due to mxnet or gluonnlp updating, the finetuning results may be slightly different with the same command. Currently we reported the best result achieved by a certain version of mxnet and gluonnlp, maybe we can report an average value and provide the log and command(especially the seed) as reference? Or even we can provide the finetuned parameters?
References

list reference and related literature
list known implementations

For future experiment I suggest we report both best score and the average score with 5 random seeds


Upgrade codemirror-graphql to 0.8.3

Fixes #773

Merged! Thanks for taking the time to fix this!

Yes! Thanks @AGS-! Are you able to get a new version published, as well?

@AGS could we get a release for this? Would love to get rid of the log spam

@AGS- Can you publish a new version?

@AGS- Also looking for a new version 😄

Note that there are still sub-dependencies of codemirror-graphql that do not support graphql 14.
https://github.com/graphql/codemirror-graphql/issues/258
IMO, we need a new release #793 after the sub-dependencies have been updated.

resolved by https://github.com/graphql/graphql-language-service/pull/353
need to cut a new release

@austinnichols101 see https://github.com/graphql/codemirror-graphql/issues/258


“The Best Defence…” not activating on UpperCut

Created by: DpsDoomMain
“The Best Defence…” should be activating upon hitting any ability but for some reason uppercut is an exception .when you land an uppercut you get nothing in terms of health.

moved from ow1_emulator_backup#660

Created by: MaxwellJung
It's because uppercut is a completely workshop-made ability, so it doesn't trigger the logic to give Doomfist passive shields.


DynamoDB: UTF8 Strings on AttributeValue

Describe the feature
Instead of getting and setting AttributeValue.S (type string, UTF16), I would like another getter/setter that exposes the value as UTF8 (type ReadOnlyMemory or a wrapper around it).
Use Case
My application uses UTF8 strings internally, and it's annoying to encode/decode the values when interacting with DynamoDB.
Proposed Solution
A new config on the DynamoDB client instructs it to serialize and deserialize using AttributeValue.S_U8 instead of AttributeValue.S. This would be opt-in; the current behavior remains the default.
Other Information
No response
Acknowledgements

[X] I may be able to implement this feature request
[ ] This feature might incur a breaking change

AWS .NET SDK and/or Package version used
AWSSDK.DynamoDBv2 <IP_ADDRESS>
Targeted .NET Platform
.NET 6+
Operating System and version
N/A

Per Supported data types and naming rules in Amazon DynamoDB, Strings are Unicode with UTF-8 binary encoding.. Whereas the .NET string represents text as a sequence of UTF-16 code units.
Needs review with the team.

@zmj Could you please provide a scenario where your application would need to use UTF-8 strings? .NET string is UTF-16 by default.
Thanks,
Ashish

My application doesn't use the System.String type (other than to interact with the DynamoDB SDK). It deserializes from UTF-8 json input, storing ReadOnlyMemory<byte> and using that for business logic. Only the ASCII character set is valid input.
My goal with this proposed change is to pass those ReadOnlyMemory<byte> buffers containing UTF-8 text to the DynamoDB SDK, instead of converting to UTF-16 System.String to set AttributeValue.S.

Sorry but this isn't a feature that is practical for us to do in the SDK.
The Attribute.S property is generated from the service model using our .NET SDK generator. In order to do this we would have to  update the generator for all string properties to generate an auxiliary using ReadOnlyMemory<byte>. That would have a massive foot print across the SDK. Also given our JSON parser, which is not the modern System.Text.Json, the ReadOnlyMemory<byte> would end up still being converted to a string.


HealthChecks no longer exposed via JMX

was once working / implemented in #92 , however it is only available via the Admin Servlet .

It appears JMX was due for a total overhaul in #149, but that never happened. I need to get a sense of how many people are actively using JMX, because at this point supporting it would only take time away from the current major goals of the v4 release.

thanks, totally reasonable.

Tracking in #832


feature(): Getting started page

This PR is for the getting started page and an initial update profile page.
The issues with the linking APIs have been resolved by changing the workflow a bit. The fabric8-ui landing page now redirects to the getting started page instead of home. I also removed the login/authorize?link=true param from the login service URL to allow the getting started page to perform linking instead.
Note that we cannot not redirect to an external URL and provide an authorization header. Therefore, instead of using /login/link, I'm using /login/linksession and parsing the JWT token. The clientSession and sessionState are provided as query params.
That said, I'm now able to link GitHub and OpenShift together. I can also link GitHub and OpenShift separately.
The username can only be updated once, which is enforced by the user patch API.
The update profile page is non-functional -- layout only right now.
Success snapshot:

Individual linking snapshot:

Update Profile

There were the following issues with your Pull Request

Commit: b9bdd7a5dde3c01e7f7bba1449291e4b79d0458a
Invalid type. Valid types are feat, fix, docs, style, refactor, perf, test, chore, revert

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com

@dlabrecq please update the commit message from "feature(): ..." to "feat(login): ...".

To test, revoke the access given on GitHub's authorized applications page. Then, remove the GitHub and OpenShift links from keycloak:
https://sso.prod-preview.openshift.io/auth/realms/fabric8/account/identity
This will trigger the getting started page to display upon login.


Not working as expected in a new network

After successfully setup a connection, the script is not working properly in a new network. It should be able to switch mode as the diagram on README if the current network is not found, so I would be able to setup a new one when runs <IP_ADDRESS>?

the Join on line 80 takes an empty string as separator, this should be a newline.
f.write(''.join(lines))
should be
f.write('\n'.join(lines))

@Karijn https://github.com/tayfunulu/WiFiManager/blob/b86a900cf5d46601be1d850828038f6cfad8fecc/wifimgr.py#L78
There is the "\n" you were missing.


How to send and receive message per sender_id?

Rasa Core version:
Current version from master (0.9.0a6)
Python version:
3.6
Operating system (windows, osx, ...):
OSx
Issue:
I have trained the NLU and dialog model. Now I run them as a server using
$python -m rasa_core.server -d models/dialogue -u models/nlu/default/customernlu/ --debug -o out.log --cors *
And from another terminal, I use the below CURL command to get reply
$curl -XPOST localhost:5005/conversations/default/respond -d '{"query":"Hello"}'
Which works fine with below reply:
[{"recipient_id":"default","text":"Hello! How can I help?"}]
Now I need to send message per sender id - assuming many chat clients are accessing the server. So I used the sender_id parameter in the CURL command
curl -XPOST localhost:5005/conversations/default/respond -d '{"sender_id":"me","query":"Hello"}'
But the agent is still interpreting this message as a message from default sender_id.
[{"recipient_id":"default","text":"Hello! How can I help?"}]
What am I missing? My dialogue management model training code below:
def train_dialogue(domain_file = 'customer_domain.yml',
					model_path = './models/dialogue',
					training_data_file = './data/stories.md'):

	agent = Agent(domain_file, policies = [MemoizationPolicy(), KerasPolicy()])

	agent.train(
				training_data_file,
				epochs = 300,
				batch_size = 50,
				validation_split = 0.2)

	agent.persist(model_path)
	return agent

Content of domain file (if used & relevant):

In Rasa HTTP server, sender_id is a URL route parameter, rather than a POST body parameter. Your HTTP request should look like this if you want sender_id to be me:
curl -XPOST localhost:5005/conversations/me/respond -d '{"query":"Hello"}'

Please see how respond() is defined for Rasa HTTP server here.

Resolved with @LawnboyMax technique

Resolved with @LawnboyMax technique


Writing with schemas defined in different files gives UnknownType errors

I've been trying to make use of avro schemas in separate files, which is supposed to be possible: https://github.com/tebeka/fastavro/blob/b53954df0c6e5f466316792671c28839510d73e9/fastavro/_schema.pyx#L110-L116
Even using the checked in examples https://github.com/tebeka/fastavro/blob/master/tests/avro-files/Child.avsc and https://github.com/tebeka/fastavro/blob/master/tests/avro-files/Parent.avsc, I found that this is not working.
If one loads the Parent.avsc schema, you get:
# inside fastavro/tests/avro-files
schema = fastavro.schema.load_schema('Parent.avsc')
{'fields': [{'name': 'child', 'type': 'Child'}],
 'name': 'Parent',
 'type': 'record'}

Now, if you try to serialize records with this schema, you get UnknownType error
In [6]: with open('parent.avro', 'wb') as out:
   ...:     fastavro.writer(out, schema, [{'child': []}])
   ...:
---------------------------------------------------------------------------
UnknownType                               Traceback (most recent call last)
<ipython-input-6-071f2d2cfde3> in <module>()
      1 with open('parent.avro', 'wb') as out:
----> 2     fastavro.writer(out, schema, [{'child': []}])
      3
fastavro/_write.pyx in fastavro._write.writer()
fastavro/_write.pyx in fastavro._write.Writer.__init__()
fastavro/_write.pyx in fastavro._write.acquaint_schema()
fastavro/_schema.pyx in fastavro._schema.extract_named_schemas_into_repo()
fastavro/_schema.pyx in fastavro._schema.extract_named_schemas_into_repo()
UnknownType: Child

That seems strange, seeing as the "Child" reader is indeed properly "acquainted"
fastavro.read.READERS
{'Child': <cyfunction acquaint_schema.<locals>.<lambda>.<locals>.<lambda> at 0x103659830>,
 'Parent': <cyfunction acquaint_schema.<locals>.<lambda>.<locals>.<lambda> at 0x1036598e8>, ...}

Does the writer need to prepare the schema in a fully dereferenced form before being able to write? My understanding was that schemas embedded in avro files were fully self-contained, so perhaps this step is missing. I don't think serializing with such schemas is covered in the tests  https://github.com/tebeka/fastavro/blob/23b52f3376bb1777d55d9d239ed021ff558a191a/tests/test_fastavro.py#L219-L229

You can temporarily fix this by calling fastavro.acquaint_schema for Child schema manually.

Hmm.
I found this recipe https://github.com/tebeka/fastavro/issues/55

Thanks for the suggestion. Indeed, acquainting the writer with the Child schema does get us past one hurdle here, in that fastavro allows us to serialize some records:
In [1]: import fastavro
In [2]: records = [{"child": {}}, {"child": {}}]
In [3]: schema = fastavro.schema.load_schema('Parent.avsc')
In [4]: fastavro.acquaint_schema(fastavro.schema.load_schema('Child.avsc'))
In [5]: with open('parent.avro', 'wb') as out:
   ...:     fastavro.writer(out, schema, records)

all good so far, except now if we try to read the contents:
fastavro --schema parent.avro
Traceback (most recent call last):
  File "/Users/x/.local/share/virtualenvs/y/bin/fastavro", line 11, in <module>
    sys.exit(main())
  File "/Users/x/.local/share/virtualenvs/y/lib/python3.6/site-packages/fastavro/__main__.py", line 64, in main
    reader = avro.reader(fo)
  File "fastavro/_read.pyx", line 678, in fastavro._read.iter_avro.__init__
  File "fastavro/_schema.pyx", line 163, in fastavro._schema.acquaint_schema
  File "fastavro/_schema.pyx", line 100, in fastavro._schema.extract_named_schemas_into_repo
  File "fastavro/_schema.pyx", line 69, in fastavro._schema.extract_named_schemas_into_repo
fastavro._schema_common.UnknownType: Child

It seems to me, being only somewhat familiar with the code here, that although fastavro knows about all the types it needs for ser/deser after all schemas and sub-schemas are acquainted, it doesn't prepare a fully self-contained schema to write into the avro file.

@hajapy I believe https://github.com/tebeka/fastavro/pull/160 should resolve this. I fixed the test in tests/test_fastavro.py:test_compose_schemas so that it now checks for both the Child and Parent schemas in both the READERS and WRITERS.
With the changes I was able to run your example without an error:
In [1]: import fastavro

In [2]: from io import BytesIO

In [3]: schema = fastavro.schema.load_schema('tests/avro-files/Parent.avsc')

In [4]: s = BytesIO()
   ...: fastavro.writer(s, schema, [{'child': []}])
   ...: print(s.getvalue())
   ...:
b'Obj\x01\x04\x14avro.codec\x08null\x16avro.schema\x98\x01{"name":"Parent","type":"record","fields":[{"name":"child","type":"Child"}]}\x00\xf5\x1f\xa8O\xc9\x0bj\x08\xa3\x10\xcd!m\x0e\x1e\xca\x02\x00\xf5\x1f\xa8O\xc9\x0bj\x08\xa3\x10\xcd!m\x0e\x1e\xca'

@scottbelden thanks for looking into it. I think the acquainting the reader and writer when you load the schema is good, but I think there is still something missing.
You can see plainly that the embedded schema doesn't define Child, so anyone trying to read this avro file without the original writer schema will not be able to.
Can confirm even on your branch:
In [1]: with open('parent.avro', 'wb') as f:
   ...:     f.write(b'Obj\x01\x04\x14avro.codec\x08null\x16avro.schema\x98\x01{"name":"Parent","type":"record","fields":[{"name":"child","t
   ...: ype":"Child"}]}\x00\xf5\x1f\xa8O\xc9\x0bj\x08\xa3\x10\xcd!m\x0e\x1e\xca\x02\x00\xf5\x1f\xa8O\xc9\x0bj\x08\xa3\x10\xcd!m\x0e\x1e\xca
   ...: ')
   ...:

In [2]:
Do you really want to exit ([y]/n)?
(fastavro-qZB9hzSx) ➜  fastavro git:(6b9e222) ✗ fastavro parent.avro
Traceback (most recent call last):
  File "/Users/x/.local/share/virtualenvs/fastavro-qZB9hzSx/bin/fastavro", line 11, in <module>
    load_entry_point('fastavro', 'console_scripts', 'fastavro')()
  File "/Users/x/workspace/forks/fastavro/fastavro/__main__.py", line 64, in main
    reader = avro.reader(fo)
  File "fastavro/_read.pyx", line 689, in fastavro._read.iter_avro.__init__
    acquaint_schema(self.writer_schema)
  File "fastavro/_read.pyx", line 614, in fastavro._read.acquaint_schema
    extract_named_schemas_into_repo(
  File "fastavro/_schema.pyx", line 100, in fastavro._schema.extract_named_schemas_into_repo
    namespaced_name = extract_named_schemas_into_repo(
  File "fastavro/_schema.pyx", line 69, in fastavro._schema.extract_named_schemas_into_repo
    raise UnknownType(schema)
fastavro._schema_common.UnknownType: Child

I see. That's a bit trickier. Let me see what I can do.

This may be helpful. I realized that if I put Child.avsc and Parent.avsc into a single schema file, I get get everything working:
Combo.avsc
[
  {
    "type": "record",
    "name": "Child",
    "fields": []
  },
  {
    "type": "record",
    "name": "Parent",
    "fields": [
      {
        "name": "child",
        "type": "Child"
      }
    ]
  }
]

In [1]: import fastavro
In [2]: schema = fastavro.schema.load_schema('Combo.avsc')
In [3]: records = [{"child": {}}, {"child": {}}]
In [4]: with open('parent.avro', 'wb') as out:
   ...:     fastavro.writer(out, schema, records)
   ...:

Now, I see this:
(fastavro-qZB9hzSx) ➜  avro-files git:(6b9e222) ✗ fastavro --schema parent.avro
[
 {
  "type": "record",
  "name": "Child",
  "fields": []
 },
 {
  "type": "record",
  "name": "Parent",
  "fields": [
   {
    "name": "child",
    "type": "Child"
   }
  ]
 }
]
(fastavro-qZB9hzSx) ➜  avro-files git:(6b9e222) ✗ fastavro parent.avro
{"child": {}}
{"child": {}}

One additional note: the order of the schemas matters here. If you put Parent before Child, you still get the UnknownType error. So I think the solution is to combine the schemas into an array sorted by dependency order. Tricky, but should be solvable.

Good point. I had forgotten that schemas could be lists, not just dictionaries. That made things a lot easier. I updated by PR branch and added a test for the error when reading, so I think it should be good now.

Great stuff @scottbelden, your branch is working for me now and seems to handle this issue nicely. I like that using load_schema returns the schema as it will be used in the writer and that it isn't some behind-the scenes conversion that goes on.

Great. Will probably merge/release shortly.


DB Reset w/o reinstall

Hi, first, thanks for this great product, we are testing it and we didn't find any issues so far.
One thing that I want to know is if there is some way to reset the database without reinstalling the server.
Thanks for your help
F-

Hi, just stop Ambar, remove everything from db, es and rabbit folders, and start it again.

Hi, thanks for your response, I tried that at first, but after I restart the service, It shows an "oops" error and the search didn't work.

Are you sure that you removed everything and properly restarted Ambar? Did you stop it with 'docker-compose down'?


Default learning rate to None (default lr for the optimizer)

What does this PR do?

See discussion here: https://github.com/PyTorchLightning/lightning-flash/pull/1163#discussion_r806535708
Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes?
[x] Did you write any new necessary tests? [not needed for typos/docs]
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[x] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

just a note: not all optimizers do have a default (e.g. SGD doesn't).


TU - Bhaktapur Multiple Campus

Domain: bkmc.tu.edu.np

Tribhuvan University
Bhaktapur Multiple Campus
Institute of science and technology of Tribhuvan University: ( https://iost.tu.edu.np/ )
Constituent-Colleges of tuiost with course: ( https://iost.tu.edu.np/pages/constituent-colleges-277  )
The picture as a proof of bhaktapur multiple campus as a constituent collegees of TU-IOST where program such as CSIT (computer science and information technology) and BIT (bachelors in information technology) are included

@WhoIsAoH I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host bkmc.tu not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


is this compatible with nuxt v3 ?

hi
can we use this module with nuxt version 3 ?

hi
can we use this module with nuxt version 3 ?

Hey! Unfortunately not:(

you can use vue-recaptcha and create a global compenent using nuxt 3 plugins it works for me

hi can we use this module with nuxt version 3 ?

hi
can we use this module with nuxt version 3 ?

Hey! Unfortunately not:(

you can use vue-recaptcha and create a global compenent using nuxt 3 plugins it works for me

hi can we use this module with nuxt version 3 ?

You can create Nuxt Plugin
npm i --save vue-recaptcha

If this gave an error try to install with try to add --force flag
npm i --save vue-recaptcha --force

In plugins directory create file recaptcha.ts
Paste this code in recaptcha.ts
import {defineNuxtPlugin} from "#app";
import {VueRecaptcha} from "vue-recaptcha";

export default defineNuxtPlugin((nuxtApp)=> {
    nuxtApp.vueApp.component('VueRecaptcha', VueRecaptcha)
})

Inside your component
<template>
   ...
   <VueRecaptcha
       sitekey="YOUR_SITE_KEY"
       :loadRecaptchaScript="true"
       @verify="verifyRecaptcha"
       @expired="expiredRecaptcha"
   />
   ...
</template>

Here's the complete docs

I found this issue while Googling this problem. There are several attempts floating around to get reCaptcha working with Nuxt 3. This is the one I ended up using.
https://github.com/AurityLab/vue-recaptcha-v3/issues/609
You can see my implementation of that library in this pull request.
https://github.com/dan-bowen/nuxt-blog-starter/pull/8
That being said... I hope the community will be able to rally around an updated reCaptcha plugin now that Nuxt 3 is stable.

I found this issue while Googling this problem. There are several attempts floating around to get reCaptcha working with Nuxt 3. This is the one I ended up using.
https://github.com/AurityLab/vue-recaptcha-v3/issues/609
You can see my implementation of that library in this pull request.
https://github.com/dan-bowen/nuxt-blog-starter/pull/8
That being said... I hope the community will be able to rally around an updated reCaptcha plugin now that Nuxt 3 is stable.

I am working on that 😉

Hi, I'm the author of vue-recaptcha. vue-recaptcha v3 is in alpha and has natively Nuxt integration. I hope could get some feedback.

hi can we use this module with nuxt version 3 ?

You can create Nuxt Plugin but first install vue-recaptcha package/lib
npm i --save vue-recaptcha

If this gave an error try to install with try to add --force flag
npm i --save vue-recaptcha --force

In plugins directory create file recaptcha.ts
Paste this code in recaptcha.ts
import {defineNuxtPlugin} from "#app";
import {VueRecaptcha} from "vue-recaptcha";

export default defineNuxtPlugin((nuxtApp)=> {
    nuxtApp.vueApp.component('VueRecaptcha', VueRecaptcha)
})

Inside your component
<template>
   ...
   <VueRecaptcha
       sitekey="YOUR_SITE_KEY"
       :loadRecaptchaScript="true"
       @verify="verifyRecaptcha"
       @expired="expiredRecaptcha"
   />
   ...
</template>

Here's the complete docs

Hey @waleedtariq109 thank you! I wasn't using reCaptcha but was trying to get hCaptcha to work, and your code helped me understand the component registration part of the plugin was the missing link.
Here was my code for vue-hCaptcha in case it helps anyone else:
In your vue file
<vue-hcaptcha
sitekey="xxxx"
></vue-hcaptcha>

Then in plugins\vue-hcaptcha.js:
import { defineNuxtPlugin } from "#app";
import VueHcaptcha from '@hcaptcha/vue3-hcaptcha'

export default defineNuxtPlugin((nuxtApp) => {
nuxtApp.vueApp.component('vue-hcaptcha', VueHcaptcha)
})

Check out this comment


Cleans up uninitialized warning in LogMessage.cc

Warning:
 log_function(message.c_str(), empty_args);
                                  ^~~~~~~~~~
LogMessage.cc:74:23: note: initialize the variable 'empty_args' to silence this warning
    va_list empty_args;
                      ^
                       = nullptr

This was introduced in #7279


Use stdlib JSON instead of multi_json gem

Why is this an improvement?

http://www.mikeperham.com/2016/02/09/kill-your-dependencies/

So many gems declare a dependency on json, oj, multi_json, or yajl-ruby. There’s so many ossified layers of cruft around JSON processing that only one course of action makes sense: remove it all. JSON has been in the stdlib since 1.9

I'm sorry but one blog post is not a significant enough argument for me. It's not unreasonable to use YAJL over the json gem. multi_json is also a really unobtrusive dependency.

A Rubygem should never use anything but Net::HTTP internally!

Oh yeah, the hard and fast rules from opinionated Ruby developers who know better. thanks, I stop reading right there and close this PR. Thank you for your time.

@michaelklishin
i agree that Mike's post is very strongly (over-?)stated
but i would apply occam's razor here - why include a dependency if it's in the stdlib?

Because json gem is sufficient for many but not all.
Hutch has only 4 runtime dependencies: Bunny/March Hare, a Ruby HTTP API client, MultiJSON, ActiveSupport. It's not 39, and while the latter two could probably be eliminated with some effort and sacrifices, I find them to be fairly unobtrusive. To my knowledge the number of dependencies Hutch has has never been an issue for its users.


Improvements on drawing circuits.

Following some discussions with @joseignaciolatorre I would like to make some proposals on changes do the .draw() feature for ease of reading.
The line_wrap flag should be initialized at some value instead of being set to unlimited by default. We should decide a sensible value in order to not affect small and simple circuits, but at the same time if a large circuit needs to be drawn, have it split into chunks so that it can be properly interpreted.
Additionally, when splitting the circuits into different lines, it would be very useful to have the qubit labels repeated also on each new line in order to easily follow gates acting on a qubit on the following lines as well. The same argument can be made for having labels on the right side of the circuit as well.
If it does not crowd the screen too much, having extra qubit labels helps a lot when analyzing large circuits.

@igres26 thanks for the suggestion. Any guess for the line wrap default value or algorithm?

An algorithm might be best, but things depend too much on the display where the circuit is drawn. I think something around 70 (between 50 and 100) might be a good starting point.


MoPub integration

As the title suggests.

Codecov Report

Merging #5 into master will not change coverage.

@@           Coverage Diff           @@
##           master       #5   +/-   ##
=======================================
  Coverage   72.01%   72.01%
=======================================
  Files          16       16
  Lines         411      411
  Branches       62       62
=======================================
  Hits          296      296
  Misses        103      103
  Partials       12       12

Impacted Files
Coverage Δ

...a/mobi/pocketmedia/nativeadslib/core/NativeAd.java
100% <ø> (ø)
:white_check_mark:

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 2e8fc61...57a23c8. Read the comment docs.


logging_helpers: TypeError: can only concatenate str (not "bytes") to str

I have a large pyproject.toml on a commercial project with git+ deps which is failing, and importantly the logging is also failing.  This issue is only about the logging - I'll need to do more analysis to find the root of the other problems.
fwiw, the git hash mentioned refers to https://github.com/viper-development/django-rest-auth/commit/29a24402ad448f1d0d3f0992664d34a5621e0c7a
I doubt that repo is the problem - I assume it could happen to any git+ repo and maybe only if there are network issues of some sort.
> dephell deps convert --from-path pyproject.toml --from-format poetry --to-format piplock --to-path requirements.txt
WARNING cannot find tool.dephell section in the config (path=pyproject.toml)
⠋ layers: 1, mutations: 0--- Logging error ---
Traceback (most recent call last):
  File "/usr/lib64/python3.8/logging/__init__.py", line 1081, in emit
    msg = self.format(record)
  File "/usr/lib64/python3.8/logging/__init__.py", line 925, in format
    return fmt.format(record)
  File "/usr/lib/python3.8/site-packages/dephell/logging_helpers.py", line 100, in format
    record.msg = Fore.WHITE + record.msg + Fore.RESET
TypeError: can only concatenate str (not "bytes") to str
Call stack:
  File "/usr/bin/dephell", line 11, in <module>
    load_entry_point('dephell==0.8.3', 'console_scripts', 'dephell')()
  File "/usr/lib/python3.8/site-packages/dephell/cli.py", line 84, in entrypoint
    exit(main(argv[1:]))
  File "/usr/lib/python3.8/site-packages/dephell/cli.py", line 66, in main
    result = handler()
  File "/usr/lib/python3.8/site-packages/dephell/commands/deps_convert.py", line 59, in __call__
    resolved = resolver.resolve(silent=self.config['silent'])
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 90, in resolve
    resolved = self._resolve(debug=debug, silent=silent, level=level, spinner=spinner)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 119, in _resolve
    no_conflicts = self._apply_deps(deps, debug=debug)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 208, in _apply_deps
    conflict = self.apply(dep)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 53, in apply
    if not other_dep.compat:
  File "/usr/lib/python3.8/site-packages/dephell/models/dependency.py", line 154, in compat
    for group in self.groups:
  File "/usr/lib/python3.8/site-packages/dephell/models/groups.py", line 173, in __iter__
    if len(self.releases) <= i:
  File "/usr/lib/python3.8/site-packages/dephell/cached_property.py", line 18, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/usr/lib/python3.8/site-packages/dephell/models/groups.py", line 37, in releases
    releases = self.dep.repo.get_releases(self.dep)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 88, in get_releases
    version=self.metaversion,
  File "/usr/lib/python3.8/site-packages/dephell/cached_property.py", line 18, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 58, in metaversion
    return self.get_nearest_version(self.link.rev)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 120, in get_nearest_version
    result = self._call('describe', '--contains', ref)[0]
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 141, in _call
    logger.error(result.stderr)
Message: b"fatal: cannot describe '29a24402ad448f1d0d3f0992664d34a5621e0c7a'\n"
Arguments: ()
⠴ layers: 1, mutations: 0--- Logging error ---
Traceback (most recent call last):
  File "/usr/lib64/python3.8/logging/__init__.py", line 1081, in emit
    msg = self.format(record)
  File "/usr/lib64/python3.8/logging/__init__.py", line 925, in format
    return fmt.format(record)
  File "/usr/lib/python3.8/site-packages/dephell/logging_helpers.py", line 100, in format
    record.msg = Fore.WHITE + record.msg + Fore.RESET
TypeError: can only concatenate str (not "bytes") to str
Call stack:
  File "/usr/bin/dephell", line 11, in <module>
    load_entry_point('dephell==0.8.3', 'console_scripts', 'dephell')()
  File "/usr/lib/python3.8/site-packages/dephell/cli.py", line 84, in entrypoint
    exit(main(argv[1:]))
  File "/usr/lib/python3.8/site-packages/dephell/cli.py", line 66, in main
    result = handler()
  File "/usr/lib/python3.8/site-packages/dephell/commands/deps_convert.py", line 59, in __call__
    resolved = resolver.resolve(silent=self.config['silent'])
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 90, in resolve
    resolved = self._resolve(debug=debug, silent=silent, level=level, spinner=spinner)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 119, in _resolve
    no_conflicts = self._apply_deps(deps, debug=debug)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 208, in _apply_deps
    conflict = self.apply(dep)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 53, in apply
    if not other_dep.compat:
  File "/usr/lib/python3.8/site-packages/dephell/models/dependency.py", line 154, in compat
    for group in self.groups:
  File "/usr/lib/python3.8/site-packages/dephell/models/groups.py", line 173, in __iter__
    if len(self.releases) <= i:
  File "/usr/lib/python3.8/site-packages/dephell/cached_property.py", line 18, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/usr/lib/python3.8/site-packages/dephell/models/groups.py", line 37, in releases
    releases = self.dep.repo.get_releases(self.dep)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 88, in get_releases
    version=self.metaversion,
  File "/usr/lib/python3.8/site-packages/dephell/cached_property.py", line 18, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 58, in metaversion
    return self.get_nearest_version(self.link.rev)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 120, in get_nearest_version
    result = self._call('describe', '--contains', ref)[0]
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 141, in _call
    logger.error(result.stderr)
Message: b"fatal: cannot describe 'ddcda12bd76a0c69bbbadc5d2826394288f3af64'\n"
Arguments: ()
⠙ layers: 1, mutations: 0--- Logging error ---
Traceback (most recent call last):
  File "/usr/lib64/python3.8/logging/__init__.py", line 1081, in emit
    msg = self.format(record)
  File "/usr/lib64/python3.8/logging/__init__.py", line 925, in format
    return fmt.format(record)
  File "/usr/lib/python3.8/site-packages/dephell/logging_helpers.py", line 100, in format
    record.msg = Fore.WHITE + record.msg + Fore.RESET
TypeError: can only concatenate str (not "bytes") to str
Call stack:
  File "/usr/bin/dephell", line 11, in <module>
    load_entry_point('dephell==0.8.3', 'console_scripts', 'dephell')()
  File "/usr/lib/python3.8/site-packages/dephell/cli.py", line 84, in entrypoint
    exit(main(argv[1:]))
  File "/usr/lib/python3.8/site-packages/dephell/cli.py", line 66, in main
    result = handler()
  File "/usr/lib/python3.8/site-packages/dephell/commands/deps_convert.py", line 59, in __call__
    resolved = resolver.resolve(silent=self.config['silent'])
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 90, in resolve
    resolved = self._resolve(debug=debug, silent=silent, level=level, spinner=spinner)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 119, in _resolve
    no_conflicts = self._apply_deps(deps, debug=debug)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 208, in _apply_deps
    conflict = self.apply(dep)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 53, in apply
    if not other_dep.compat:
  File "/usr/lib/python3.8/site-packages/dephell/models/dependency.py", line 154, in compat
    for group in self.groups:
  File "/usr/lib/python3.8/site-packages/dephell/models/groups.py", line 173, in __iter__
    if len(self.releases) <= i:
  File "/usr/lib/python3.8/site-packages/dephell/cached_property.py", line 18, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/usr/lib/python3.8/site-packages/dephell/models/groups.py", line 37, in releases
    releases = self.dep.repo.get_releases(self.dep)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 76, in get_releases
    for tag, time in reversed(self.tags.items()):
  File "/usr/lib/python3.8/site-packages/dephell/cached_property.py", line 18, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 40, in tags
    self._setup()
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 182, in _setup
    self._call(
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 141, in _call
    logger.error(result.stderr)
Message: b"Cloning into 'django-oscar'...\nerror: RPC failed; curl 18 transfer closed with outstanding read data remaining\nfatal: the remote end hung up unexpectedly\nfatal: early EOF\nfatal: index-pack failed\n"
Arguments: ()
--- Logging error ---
Traceback (most recent call last):
  File "/usr/lib64/python3.8/logging/__init__.py", line 1081, in emit
    msg = self.format(record)
  File "/usr/lib64/python3.8/logging/__init__.py", line 925, in format
    return fmt.format(record)
  File "/usr/lib/python3.8/site-packages/dephell/logging_helpers.py", line 100, in format
    record.msg = Fore.WHITE + record.msg + Fore.RESET
TypeError: can only concatenate str (not "bytes") to str
Call stack:
  File "/usr/bin/dephell", line 11, in <module>
    load_entry_point('dephell==0.8.3', 'console_scripts', 'dephell')()
  File "/usr/lib/python3.8/site-packages/dephell/cli.py", line 84, in entrypoint
    exit(main(argv[1:]))
  File "/usr/lib/python3.8/site-packages/dephell/cli.py", line 66, in main
    result = handler()
  File "/usr/lib/python3.8/site-packages/dephell/commands/deps_convert.py", line 59, in __call__
    resolved = resolver.resolve(silent=self.config['silent'])
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 90, in resolve
    resolved = self._resolve(debug=debug, silent=silent, level=level, spinner=spinner)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 119, in _resolve
    no_conflicts = self._apply_deps(deps, debug=debug)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 208, in _apply_deps
    conflict = self.apply(dep)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 53, in apply
    if not other_dep.compat:
  File "/usr/lib/python3.8/site-packages/dephell/models/dependency.py", line 154, in compat
    for group in self.groups:
  File "/usr/lib/python3.8/site-packages/dephell/models/groups.py", line 173, in __iter__
    if len(self.releases) <= i:
  File "/usr/lib/python3.8/site-packages/dephell/cached_property.py", line 18, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/usr/lib/python3.8/site-packages/dephell/models/groups.py", line 37, in releases
    releases = self.dep.repo.get_releases(self.dep)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 76, in get_releases
    for tag, time in reversed(self.tags.items()):
  File "/usr/lib/python3.8/site-packages/dephell/cached_property.py", line 18, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 40, in tags
    self._setup()
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 188, in _setup
    self._call('checkout', self.link.rev)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 141, in _call
    logger.error(result.stderr)
Message: b'fatal: not a git repository (or any parent up to mount point /)\nStopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).\n'
Arguments: ()
--- Logging error ---
Traceback (most recent call last):
  File "/usr/lib64/python3.8/logging/__init__.py", line 1081, in emit
    msg = self.format(record)
  File "/usr/lib64/python3.8/logging/__init__.py", line 925, in format
    return fmt.format(record)
  File "/usr/lib/python3.8/site-packages/dephell/logging_helpers.py", line 100, in format
    record.msg = Fore.WHITE + record.msg + Fore.RESET
TypeError: can only concatenate str (not "bytes") to str
Call stack:
  File "/usr/bin/dephell", line 11, in <module>
    load_entry_point('dephell==0.8.3', 'console_scripts', 'dephell')()
  File "/usr/lib/python3.8/site-packages/dephell/cli.py", line 84, in entrypoint
    exit(main(argv[1:]))
  File "/usr/lib/python3.8/site-packages/dephell/cli.py", line 66, in main
    result = handler()
  File "/usr/lib/python3.8/site-packages/dephell/commands/deps_convert.py", line 59, in __call__
    resolved = resolver.resolve(silent=self.config['silent'])
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 90, in resolve
    resolved = self._resolve(debug=debug, silent=silent, level=level, spinner=spinner)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 119, in _resolve
    no_conflicts = self._apply_deps(deps, debug=debug)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 208, in _apply_deps
    conflict = self.apply(dep)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 53, in apply
    if not other_dep.compat:
  File "/usr/lib/python3.8/site-packages/dephell/models/dependency.py", line 154, in compat
    for group in self.groups:
  File "/usr/lib/python3.8/site-packages/dephell/models/groups.py", line 173, in __iter__
    if len(self.releases) <= i:
  File "/usr/lib/python3.8/site-packages/dephell/cached_property.py", line 18, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/usr/lib/python3.8/site-packages/dephell/models/groups.py", line 37, in releases
    releases = self.dep.repo.get_releases(self.dep)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 76, in get_releases
    for tag, time in reversed(self.tags.items()):
  File "/usr/lib/python3.8/site-packages/dephell/cached_property.py", line 18, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 41, in tags
    tags = self._call('tag')
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 141, in _call
    logger.error(result.stderr)
Message: b'fatal: not a git repository (or any parent up to mount point /)\nStopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).\n'
Arguments: ()
--- Logging error ---
Traceback (most recent call last):
  File "/usr/lib64/python3.8/logging/__init__.py", line 1081, in emit
    msg = self.format(record)
  File "/usr/lib64/python3.8/logging/__init__.py", line 925, in format
    return fmt.format(record)
  File "/usr/lib/python3.8/site-packages/dephell/logging_helpers.py", line 100, in format
    record.msg = Fore.WHITE + record.msg + Fore.RESET
TypeError: can only concatenate str (not "bytes") to str
Call stack:
  File "/usr/bin/dephell", line 11, in <module>
    load_entry_point('dephell==0.8.3', 'console_scripts', 'dephell')()
  File "/usr/lib/python3.8/site-packages/dephell/cli.py", line 84, in entrypoint
    exit(main(argv[1:]))
  File "/usr/lib/python3.8/site-packages/dephell/cli.py", line 66, in main
    result = handler()
  File "/usr/lib/python3.8/site-packages/dephell/commands/deps_convert.py", line 59, in __call__
    resolved = resolver.resolve(silent=self.config['silent'])
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 90, in resolve
    resolved = self._resolve(debug=debug, silent=silent, level=level, spinner=spinner)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 119, in _resolve
    no_conflicts = self._apply_deps(deps, debug=debug)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 208, in _apply_deps
    conflict = self.apply(dep)
  File "/usr/lib/python3.8/site-packages/dephell/controllers/_resolver.py", line 53, in apply
    if not other_dep.compat:
  File "/usr/lib/python3.8/site-packages/dephell/models/dependency.py", line 154, in compat
    for group in self.groups:
  File "/usr/lib/python3.8/site-packages/dephell/models/groups.py", line 173, in __iter__
    if len(self.releases) <= i:
  File "/usr/lib/python3.8/site-packages/dephell/cached_property.py", line 18, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/usr/lib/python3.8/site-packages/dephell/models/groups.py", line 37, in releases
    releases = self.dep.repo.get_releases(self.dep)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 76, in get_releases
    for tag, time in reversed(self.tags.items()):
  File "/usr/lib/python3.8/site-packages/dephell/cached_property.py", line 18, in __get__
    value = obj.__dict__[self.func.__name__] = self.func(obj)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 42, in tags
    result = [(tag, self._get_rev_time(tag)) for tag in tags]
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 42, in <listcomp>
    result = [(tag, self._get_rev_time(tag)) for tag in tags]
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 145, in _get_rev_time
    data = self._call('show', '-s', r'--format="%cI"', rev)
  File "/usr/lib/python3.8/site-packages/dephell/repositories/_git/git.py", line 141, in _call
    logger.error(result.stderr)
Message: b'fatal: not a git repository (or any parent up to mount point /)\nStopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).\n'
Arguments: ()
ERROR ValueError: time data '' does not match format '%Y-%m-%dT%H:%M:%S%z'

--level DEBUG didn't show anything significant
WARNING cannot find tool.dephell section in the config (path=pyproject.toml)
DEBUG load dependencies... (format=poetry, path=pyproject.toml)
DEBUG resolving...
⠋ layers: 1, mutations: 0--- Logging error ---
...

Need more debugging.. ;-)


Tool 'test' failed with Command *** returned non-zero exit status 1, whereas running the single test returns 0

When I run evaluate.py I get errors for each tool runned

whereas if the single test is runned everything looks fine.

Is there a way to get the output of each run from evaluate.py?

Hi you should not run your interpreter using the bin/evaluate.py tool, instead use the bin/test.py tool.
I hope this fix the problem, otherwise please use a lot of -vs so that we can get as much debugging as possible.

If it did not fix the problem fell free to reopen the issue! :)


Update command

Wouldn't running the "vs_installer.exe update --passive --installpath '<pathToVS>'" option be preferable since it updates both in one line?

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

ID: eedaccf1-bbb7-9a7c-21be-64217359acb5
Version Independent ID: da785eb8-7187-0831-0b3c-1b422c6f2a15
Content: Command-line parameter examples for Visual Studio installation - Visual Studio
Content Source: docs/install/command-line-parameter-examples.md
Product: visual-studio-dev15
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@JamesMBristow2 Thanks for your feedback; we appreciate it. And yes, while you could use one line, we chose to use multiple lines in our examples so that the individual command lines can be more easily differentiated between. But thanks for including an alternate example via this inquiry mechanism.

Can you update the command reference?  Nowhere on here does it mention that "--update" is a valid option or that it updates the Updater.  - https://docs.microsoft.com/en-us/visualstudio/install/use-command-line-parameters-to-install-visual-studio?view=vs-2019 or https://docs.microsoft.com/en-us/visualstudio/install/use-command-line-parameters-to-install-visual-studio?view=vs-2017
vs_enterprise.exe --update --quiet --wait
vs_enterprise.exe update --wait --passive --norestart --installPath "C:\installPathVS"


sickgear: 0.25.35 -> 0.25.40, patch & optional dependencies

Description of changes
Sickgear update (would supercede #183847). Also added a patch to override the python version check in sickgear, as per this comment - a lot of times the version packaged in nixpkgs is too new for sickgear - so the patch just warns that it's unsupported instead of quitting.
I think this would also close #169182 (although I'm still not sure if it's better to bundle libarchive, or maybe include unrar as an optional dependency since it's unfree).
Changelog for the update can be found here
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Please squash commits

Ok done, also updated to 0.25.40

Result of nixpkgs-review pr 187983 run on x86_64-linux 1

1 package built:

sickgear

Thank you!

Coolio, thanks


Could not find com.facebook.react:react-native-webrtc:1.69.2-jitsi-2062090 dependency

Hello,
I am facing an issue regarding com.facebook.react:react-native-webrtc:1.69.2-jitsi-2062090 (check snippet below). Seems like it is removed from jCenter so I am unable to build application with Matrix SDK since it cannot fetch the dependency anymore. This is a big issue at the moment since basically I am stuck with development and CI also won't create builds and migration to Matrix2 is currently difficult to make since there is no guide or similarities between Matrix1 and Matrix2. Is there any other place where Matrix SDK and all of its dependencies are hosted and can be used even though it is deprecated? Or is there any other solution besides switching to Matrix2?
Any help would be appreciated!
Thank you
Mario
> Could not resolve all artifacts for configuration ':app:debugRuntimeClasspath'.
   > Could not find com.facebook.react:react-native-webrtc:1.69.2-jitsi-2062090.
     Searched in the following locations:
       - https://github.com/vector-im/jitsi_libre_maven/raw/master/releases/com/facebook/react/react-native-webrtc/1.69.2-jitsi-2062090/react-native-webrtc-1.69.2-jitsi-2062090.pom
       - https://dl.google.com/dl/android/maven2/com/facebook/react/react-native-webrtc/1.69.2-jitsi-2062090/react-native-webrtc-1.69.2-jitsi-2062090.pom
       - https://jcenter.bintray.com/com/facebook/react/react-native-webrtc/1.69.2-jitsi-2062090/react-native-webrtc-1.69.2-jitsi-2062090.pom
       - https://repo.maven.apache.org/maven2/com/facebook/react/react-native-webrtc/1.69.2-jitsi-2062090/react-native-webrtc-1.69.2-jitsi-2062090.pom
     Required by:
         project :app > com.github.matrix-org:matrix-android-sdk:v0.9.32 > com.github.matrix-org.matrix-android-sdk:matrix-sdk:v0.9.32

It was probably a temporary issue. You closed it because it's fixed now?

It was probably a temporary issue. You closed it because it's fixed now?

@bmarty I excluded jitsi from MatrixSDK and submitted path to https://github.com/vector-im/jitsi_libre_maven/tree/main/releases/com/facebook/react. Luckily that repo still exists! :) Thanks for the help anyway