Create container image on push to master and on release

See: https://docs.github.com/en/actions/publishing-packages/publishing-docker-images
Publish a container image (with multiple tags that correspond to the release event, whether it's nightly or stable build) to Github Container Registry via Github Actions.

hey @aldy505 I'd like to help with this one!
Are you going to release tags for those events? Can you specify how would you like to trigger them?


contribute to this repo

Hi, thanks for your greatly insightful survey. I had summarized related papers two months ago which may not be as comprehensive as yours. Sincerely hope it will contribute to this nice repo.  For more info please visit NLP-ParadigmShiftingPapers.

Thanks for the valuable contribution! I've read your nice repo and found many papers that we missed in our paper. The papers that we missed will be included in the website and the next version of our paper, and of course, we will acknowledge your helpful contribution.

Thanks for the valuable contribution! I've read your nice repo and found many papers that we missed in our paper. The papers that we missed will be included in the website and the next version of our paper, and of course, we will acknowledge your helpful contribution.

it's my great honor to be able to contribute to this work.  I like this work very much.  If possible, I will continue to contribute to this work to make it more perfect.


[ci skip] adding user @bnavigator

Hi! This is the friendly automated conda-forge-webservice.
I've added user @bnavigator as instructed in #53.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #53

I'm not sure.  I just tried it again and it worked.  Maybe I was doing something wrong before (not sure what; it is literally just pressing a button...).


ENOTFOUND error after updating to 0.12.2

Environment
Docker
Version
0.12.2
Describe the problem
Hi, I just pulled the latest image but when I try to start the container I get the error:
Invariant: failed to start render worker Error: getaddrinfo ENOTFOUND mydomain
I'm using a Synology NAS hosted on mydomain.synology.me, I think the mydomain part of the error is referred to the URL above as I also tried to restart the container without a config file to make sure it's not a config issue.
Logs
Invariant: failed to start render worker Error: getaddrinfo ENOTFOUND mydomain
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:109:26) {
  errno: -3008,
  code: 'ENOTFOUND',
  syscall: 'getaddrinfo',
  hostname: 'mydomain'
}

Context
As reported above, I tried to restart the container without a config file but I'm still getting the same issue.
It shouldn't be an issue on the hosting configuration as it worked with the previous version before the update.
Please tick the boxes

[X] I confirm that I attached the proper logs
[X] I've read the docs
[X] I've checked for duplicate issues
[X] I've tried to debug myself

Might be related to https://github.com/vercel/next.js/issues/44384
will investigate.
@manuel-rw maybe the renovate PR to bump the node version introduced this ? We didn’t get it since we’re using the local version with a different node version. It seems that fetch is broken somehow ?

Hi, thanks for the issue.
This is sadly a known problem with the new version, that only applies to Synology and specific orchestrators.
I attached you a screenshot of the currently still ongoing discussion.

We'll close the thread on Discord and continue here.

Just for completeness, I'm starting Homarr with Portainer Stacks and the docker-compose from the docs so no watchtowers or orchestrators.
And I also tried to delete the Stack and start it with an empty config, in my case it wasn't enough to fix the issue.

Okay, thanks for letting us know. Is anyone here available for testing? I will create a potential fix and test image.

Reminder: While using development and test images, it is always highly recommended to back up your configuration before. We do not take any responsibility for data loss.

A test image with a potential fix is available at :test-enotfound-error.
Please let me know if it works!

Just tried it but unfortunately I have the same error in the logs :(

@Meierschlumpf do you maybe have an idea? I would like to hotfix this before 0.12.3

Updated the image, can you please re-check?
:test-enotfound-error

Same error with the latest image. I don't know if I can provide any more info to debug the issue but the logs really only show the error posted before.

Damn, weird. @ajnart do you have an idea?
@Havock94 can you tell me, if you run using the bridge or host network mode? A user just reported, that they solved this issue by using the bridge network mode...

@manuel-rw I was actually running the container in host mode, changing to bridge network solves the problem with both the latest and test-enotfound-error tags!
I just need to change internal IPs because localhost pings now don't work but it's usable now.

Ok nice, we narrowed down the problem. So probably all instances of Homarr don't work on host network mode, regardless what system they are running on?

I have the same problem with latest image on casaos with debian.And It works fine under v0.12.1. @manuel-rw

I have the same problem with latest image on casaos with debian.And It works fine under v0.12.1. @manuel-rw

so v12.2 and 12.3 aren't working for you ?

yes. I tried the v12.3 to v11.5. it has the eof problem on v12.3 and v12.2. It works fine on V12.1 and V11.5.

澤田綱吉
@.***

------------------ 原始邮件 ------------------
发件人: "Thomas @.>;
发送时间: 2023年6月11日(星期天) 中午11:07
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [ajnart/homarr] ENOTFOUND error after updating to 0.12.2 (Issue #973)
Reopened #973.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

_Replying to https://github.com/ajnart/homarr/issues/973#issuecomment-1585986542
yes. I tried the v12.3 to v11.5. it has the eof problem on v12.3 and v12.2. It works fine on V12.1 and V11.5.

Probably related to:

https://medium.com/swlh/node-js-error-message-getaddrinfo-enotfound-localhost-solution-9b2fa4f61a9c
https://github.com/mswjs/msw/issues/1388
https://github.com/vercel/next.js/issues/37300

I'll try to bump the version to nodejs 20.3 first. Because it could also be an issue with nodejs 20.0

Can somebody try the image :test-fix-enotfound-error?

I'll try to bump the version to nodejs 20.3 first. Because it could also be an issue with nodejs 20.0

Agree, we try upgrade then downgrade. I don’t think the node version matters that much because we use very little features of node we should aim for the most stable one

@ajnart were you able to downgrade?

:test-fix-enotfound-error

:test-fix-enotfound-error has the same error

Okay, thanks for confirming.
I'm sorry for the inconvenience and I hope that we can find a fix soon.
Things like this usually don't get tested by the automated tests - therefore it has been present for quite some time without us noticing it.

Same error message and log. In the same environment, access through a domain name is not possible.

ok so, downgrading to v16 wouldn't work because apparently this https://github.com/ajnart/homarr/pull/976 was added before we upgraded to v20

New updated downgrading NextJS got pushed to dev image. Can you switch to that and tell us if it works ?

 Same error message and log. In the same environment, access through a domain name is not possible.

I think you error is related to Dashdot more than anything, we'd have to fix it in a new issue

@ajnart can this issue be closed?

Yes I think so


Any plans to support Pull request builds in YAML syntax?

At this point I don't know if this is still a limitation of the platform or if the documentation is outdated.
Do we already support pull request builds in YAML syntax? If not, are there any plans or any place I can follow up on this? It really doesn't make sense to have part of your pipeline automated, part you have to use the designer. Also, the PR flow is pretty basic for any pipeline I guess.
Thanks

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

ID: f5673ec1-c7d1-e0ac-043a-8fc3d5b226a9
Version Independent ID: b4872831-51c0-6cf8-1c56-de20beb38a80
Content: Build pipeline triggers - Azure Pipelines & TFS
Content Source: docs/pipelines/build/triggers.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

Coming in the next 3 weeks. It'll be documented once the feature ships.

Shipped but not documented here or anywhere.

Shipped for a fraction of our customers, but not the majority. We roll features out in rings.

Is this rollde out to all customers yet? I have specified it in my yml-file but I can't seem to get it to work.

@nonameolsson it's available everywhere. One gotcha is that new pipelines still set the "override PR triggers" flag in the UI. You have to turn that off to explicitly opt into YAML-based syntax. I'll make that clearer in the docs.

Coming in the next 3 weeks. It'll be documented once the feature ships.

Any news?

pr triggers in YAML work for non-Azure Repos repos. For Azure Repos, you must configure branch policy.

pr triggers in YAML work for non-Azure Repos repos. For Azure Repos, you must configure branch policy.

So what exactly was "coming in the next 3 weeks" ?

PR triggers for GitHub repos.


Implement a mechanism for providing macro placeholders and substitutions

This is an issue to backfill an explanation for PR #2000, which I already created as a possible implementation.
The issue to be solved here is how to handle operations with run-all that might have side effects such as creating files. In my case, the specific issue to be solved is that I have a number of repositories in which several different terragrunt.hcl files are defined. Currently each terragrunt.hcl file spawns its own pipeline which must be executed individually. For reasons of auditing, the CI pipeline generates the plan in one stage and then that plan file is passed to the next stage to be applied. This is where the problem arises: if Terragrunt is executed with a command like terragrunt run-all plan -out=plan.cache, the plan file is written to some deeply nested directory inside .terragrunt-cache. This makes it annoying and difficult to pull out to be archived in an artifact, and one of Terragrunt/Terraform/go-getter also get confused by Gitlab creating these directory structures when it moves the artifact to the next stage and fail to check out the required modules again.
THEREFORE, I came up with the idea of implementing a sort of macro replacement mechanism where certain placeholders could be added as part of commands and then Terragrunt would replace those placeholders appropriately as it processes each terragrunt.hcl file. This allows for precise control of where the plan files get written to and where the apply phase looks for them. There are undoubtedly other use cases for this kind of behavior; I only implemented what was necessary to make my specific use case work, though I tried to implement the feature in a way that would make it easy to add additional placeholders and replacements.
Examples from the PR:
With the PR applied:
➜  tg-stuff ./tg-dirty run-all --terragrunt-substitute-macros plan -out=::TERRAGRUNT_DIR::/plan.cache
INFO[0000] The stack at /Users/erhudy/Desktop/tg-stuff will be processed in the following order for command plan:
Group 1
- Module /Users/erhudy/Desktop/tg-stuff/tg1
- Module /Users/erhudy/Desktop/tg-stuff/tg2

WARN[0000] No double-slash (//) found in source URL /Users/erhudy/Desktop/tg-module. Relative paths in downloaded Terraform code may not work.  prefix=[/Users/erhudy/Desktop/tg-stuff/tg2]
WARN[0000] No double-slash (//) found in source URL /Users/erhudy/Desktop/tg-module. Relative paths in downloaded Terraform code may not work.  prefix=[/Users/erhudy/Desktop/tg-stuff/tg1]

[...]
Saved the plan to: /Users/erhudy/Desktop/tg-stuff/tg1/plan.cache

To perform exactly these actions, run the following command to apply:
    terraform apply "/Users/erhudy/Desktop/tg-stuff/tg1/plan.cache"

[...]
Saved the plan to: /Users/erhudy/Desktop/tg-stuff/tg2/plan.cache

To perform exactly these actions, run the following command to apply:
    terraform apply "/Users/erhudy/Desktop/tg-stuff/tg2/plan.cache"

➜  tg-stuff find . -name plan.cache
./tg1/plan.cache
./tg2/plan.cache

Without the PR:
➜  tg-stuff terragrunt run-all plan -out=plan.cache
INFO[0000] The stack at /Users/erhudy/Desktop/tg-stuff will be processed in the following order for command plan:
Group 1
- Module /Users/erhudy/Desktop/tg-stuff/tg1
- Module /Users/erhudy/Desktop/tg-stuff/tg2

WARN[0000] No double-slash (//) found in source URL /Users/erhudy/Desktop/tg-module. Relative paths in downloaded Terraform code may not work.  prefix=[/Users/erhudy/Desktop/tg-stuff/tg1]
WARN[0000] No double-slash (//) found in source URL /Users/erhudy/Desktop/tg-module. Relative paths in downloaded Terraform code may not work.  prefix=[/Users/erhudy/Desktop/tg-stuff/tg2]

[...]
Saved the plan to: plan.cache

To perform exactly these actions, run the following command to apply:
    terraform apply "plan.cache"

[...]
Saved the plan to: plan.cache

To perform exactly these actions, run the following command to apply:
    terraform apply "plan.cache"

➜  tg-stuff find . -name plan.cache
./tg1/.terragrunt-cache/-AC1FyvAUkFhtEx9xGyhaHhLPi0/8xYrlZ6_2OfZv1qY-qqWFyVln1w/plan.cache
./tg2/.terragrunt-cache/4ld8tgyvym06idFVd31zqRpSdfQ/8xYrlZ6_2OfZv1qY-qqWFyVln1w/plan.cache

Hi,
I was wondering if the same result can be achieved using block terraform?
terraform {
  extra_arguments "plan" {
    commands = [
      "plan",
    ]
    arguments = [
      "-out=${get_terragrunt_dir()}/plan.out"
    ]
  }
}

@denis256 That does indeed work. I was looking for a way to do more or less exactly what the description here does, and your suggestion works great!
A nice bonus is that running terragrunt run-all apply plan.cache works too, as the plan.cache will resolve to the appropriate file when terraform is invoked.

Sorry, I never got a notification on your reply here. I think your suggestion should work from a technical standpoint, though I haven't tried it myself (at least not recently). I think I wanted to avoid using extra_arguments because those would have to be committed and then would apply equally to both CI/CD and running Terragrunt locally for dev purposes; the approach I took in the PR with the CLI flags was done to allow templating out the path to the plan cache in CI/CD only, but locally it wouldn't apply unless I also chose to run that same command. Maybe this distinction doesn't actually matter in practice, but at the time I originally wrote the code I felt like it did.

Now I remember - https://github.com/gruntwork-io/terragrunt/issues/454 was the original reason why, because extra_arguments isn't compatible with positional arguments. Using terragrunt run-all apply plan.cache does work, but I didn't prefer it because of the requirement to commit changes to VCS. Changing the extra_arguments block in all our terragrunt.hcl files will require committing changes across probably 100+ repositories and coordinating that with changes to the CI/CD pipelines they use, versus the changes I proposed which would require only modifying the Gitlab CI pipeline definitions.

I am also still finding that it does not react as expected when it runs in Gitlab. If I use the extra_arguments block and run it locally, it works as expected. When it runs in Gitlab, the plan.cache file still ends up deep inside the Terragrunt cache folder structure. This causes a separate issue that seems to be hitting some bug either in Terragrunt or go-getter where it mistakes the presence of the top-level Git repository to mean that it has already checked out the modules and then blows up elsewhere.
@denis256 I hope I can ask you to reconsider this proposal, because extra_arguments is not working as desired for me, whereas I have verified that the placeholder/macro stuff does work properly inside Gitlab.

I was able to resolve the initial extra_arguments issue, but a new issue has cropped up - if extra_arguments is specified in a Terragrunt file for plan, and then something runs terragrunt plan -out=blah, this seems to result in the -out parameter disappearing entirely. This means I can't just slip extra_arguments blocks in in certain places where I want to use run-all and then gradually roll it out everywhere else later, because the -out parameter disappearing from our current CI stuff is going to break all the pipelines that I haven't updated yet.


Align dependencies to match react-native 0.66

Description
Why
Current dependencies are not aligned with react-native 0.66
What
Bumped react, react-test-renderer and @types packages to match what 0.66 requires. Also removed metro-config and metro-react-native-babel-preset to avoid duplicates. react-native is already pulling them in via @react-native-community/cli
Screenshots
n/a

cc @chiaramooney, @NickGerleman

Pipeline now restored, pull in latest from main to confirm PR passes pipeline.


Fixes #325 (fixed inline message editing)

Fixes JSON parse exceptions while parsing the bool result from Telegram in the following methods: EditInlineMessageTextAsync, EditInlineMessageCaptionAsync and EditInlineMessageReplyMarkupAsync

included this in the commits for api 3.1
thx


Deprecated: Shanty_Mongo_Connection_Group::getWriteConnection(): The 'connected' property is deprecated in /library/Shanty/Mongo/Connection/Group.php on line 133

http://php.net/manual/en/class.mongoclient.php#mongoclient.props.connected
connected
This property will be set to TRUE if we have a open connection to the database, FALSE otherwise. If the connection is to a replica set, this property will only be TRUE if the driver has a connection to a node matching the current read preference. This property does not take authentication into account.
This property is deprecated since version 1.5.0.

in Shanty/Mongo/Connection/Group.php (line 133):
if ($write && !$write->connected) {
change to
if ($write && !($write->getConnections() != [])) {

@vvasystem , @mkommar I'm happy to accept a pull request to fix this but I really don't want to encourage people to use this library. It's got some critical data loss bugs that were never fixed. You shouldn't use this library either.

Thanks, @coen-hyde ! There were changes in the more recent version of the Mongo module in PHP that deprecates some properties. Nothing that can't be fixed, but I see your point.  I appreciate the heads up and it makes sense to move away. We've just been using it for several years so it's part of our software now. =)


GVM report not adding hostnames on defectdojo with CSV report

Hi im a new to the community and im sorry if this has been addressed before but i cant find any info on this situation.
I installed the version 1.7.1 of DefectDojo
Then i import some vulnerability scan reports from GVM 28.1 without issue
When i check the endpoints i can see the ip address but not the hostname
The data related to the endpoint seems to be complete except for the missing hostname, Is maybe a compatibility issue or im missing something with the import.
Thanks

Please upgrade to 1.14.0 and try again. There's a lot of work being on endpoints in https://github.com/DefectDojo/django-DefectDojo/pull/4188

Please upgrade to 1.14.0 and try again. There's a lot of work being on endpoints in #4188

Hi @valentijnscholten thanks for the reply I actually upgrade as you recommended but still have the same behavior ... Should a start clean?.... I keep the volumes on the upgrade and all the reports previously added to test.

I think it just needs to be fixed, I don't know the details, but 4188 looks like it will fix a lot of peculiarities around endpoints.

Thanks for the info I'll keep an 👀 for when merge the 4188 to dev branch and try again

Hi i tray the latest version and it indeed bring the host names please close


Preact X Browser support

Nice work on Preact X ❤! Are there any guidelines on what browser versions Preact X will support? is the intention to support IE9+ like previous Preact versions?

In the readme: https://github.com/developit/preact#preact
the browser support is shown

I believe as of the next release Preact core supports IE9+ once again (courtesy of #1395).

We should summon @StoneCypher for that. He may even get it to work in older versions of IE<9 :+1:


Auctions which includes OMG tokens don't start

Description:
When you submit an auction including OMG tokens, it never starts.
To reproduce:
1- Go to dx.staging.gnosisdev.com
2- Submit an auction with OMG tokens
3- Observe.
Expected result:
The auction starts and after some hours the user is able to claim their funds.
Actual result:
The auction doesn't start.
Device: PC
OS: Windows 10 - Google chrome v71.0.3578.98 with Metamask extension.
Net: Rinkeby

I believe this is related to the protocol and bots. @anxolin can you comment?

closing for now, please re-open after #467 2.2.0 release if still happening


prep-v0.1.6-release

Do we need a new release for this?

Closes #
Example

Checklist

[ ] This pull request references any related issue by including "Closes #<ISSUE_NUMBER>"

If no issue exists and your change is not a small fix, please create an issue first.

[ ] This pull request includes tests or only affects documentation.
[ ] Summarized PR's changes in CHANGELOG.md

I think so - the Cloud Run Jobs won't work correctly without it.


Support native StoreKey in FilesystemStore

Description
Please include a summary of the changes and the related issue. Please also
include relevant motivation and context.
Fixes # 1108
/claim #1108
Design Document
https://hackmd.io/@HPxqrI77Q02izyGstTHbdg/ByS50OhGJx
Type of change
Please delete options that aren't relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Passes all of current test suite
Checklist

[] Updated documentation if needed
[x] Tests added/amended
[x] bazel test //...  passes locally
[x] PR is contained in a single commit, using git amend see some docs

This change is

Unsure what is happening with the local runner on CI,  seems to be a Tekton issue

If the aim is to remove the string variant from StoreKey, I'm not really sure what the aim of the bounty was for - as far as I could understand the aim was to keep using the string key as much as possible? The previous code already handled the digest variant fully?

Regarding moves with the make temp digest, it's a small function that is used once, so if this was limiting would assume it would get inlined - can add inline explicitly, but prefer to trust the compiler unless I am very sure it's a win

-- commits line 18 at r3:
Previously, aaronmondal (Aaron Siddhartha Mondal) wrote…
nit: Remember to squash at the end and remove this. Commit title lgtm :+1:

Keeping it separate for now (in case any change needs to be reverted), but will squash before any merge

nativelink-store/src/filesystem_store.rs line 322 at r3 (raw file):
Previously, aaronmondal (Aaron Siddhartha Mondal) wrote…
Interesting choice! Since this is just 40 bytes I'd assume that we get less cache misses by moving here.

As mentioned in GH, small function that is called in one place, so would expect it to be inlined - if not, passing a relatively small copy type value by value as opposed to by reference is still probably preferable

nativelink-store/src/filesystem_store.rs line 152 at r5 (raw file):
Previously, allada (Nathan (Blaise) Bruer) wrote…
After thinking about this for a bit, I think we should put Str and Digest into different folders. I suggest:
StoreKey::Str -> {folder}/s/{str}
StoreKey::Digest -> {folder}/d/{digest}
We'll then need to make a reverse-compatibility layer that will read StoreKey::Digest from both {folder} and {folder}/d at load time, and move all the files to the new location (ie: migrate them).

I think this would be a breaking change as key_from_filename is in the public API, and would have no way of distinguishing which folder the filename was in? (Or need to be modified to take a Path / Into<Path> so it is clear it takes the path relative to the folder above the string and digest folders?) Though also doesn't seem used outside the crate except for testing?

Given this is to interact with an on disk cache and bar the array access most of these are in the ns region I'd have assumed latency on actually doing disk operations would be far higher (maybe not for a ram disk)? Access over array difference is bigger but difference between is still around 200 us so comparable magnitude to disk access?

nativelink-store/src/filesystem_store.rs line 484 at r7 (raw file):
Previously, allada (Nathan (Blaise) Bruer) wrote…
nit: can we pass in the folder here and remove filetype here.
The caller knows the FileType, so just store Digest and Str as different variables and zip the FileType during the for-loop.
You should also be able to support the reverse compatibility mode trivially with this change.

Changed - looking through all three locations for files now

nativelink-store/src/filesystem_store.rs line 538 at r7 (raw file):
Previously, allada (Nathan (Blaise) Bruer) wrote…
nit: Side note: I have been wanting to remove all the atime stuff, as it really doesn't provide any value and costs a lot to maintain, so don't worry too much about perfection here, even feel free to remove the tests that verify this.

Kept the atime values but only per folder - it didn't make the code much more difficult so felt better to keep it in given that

nativelink-store/src/filesystem_store.rs line 152 at r5 (raw file):
Previously, allada (Nathan (Blaise) Bruer) wrote…
Yeah, this logic is one-time startup cost, so doing additional allocations and such are fine. Whatever it takes to make the code more readable and easier to use.

Fully working now with the subfolders and reading the old cache

nativelink-store/src/filesystem_store.rs line 538 at r7 (raw file):
Previously, allada (Nathan (Blaise) Bruer) wrote…

nit: Side note: I have been wanting to remove all the atime stuff, as it really doesn't provide any value and costs a lot to maintain, so don't worry too much about perfection here, even feel free to remove the tests that verify this.

Kept the atime values but only per folder - it didn't make the code much more difficult so felt better to keep it in given that

I have removed this as the tests were working locally but failing in CI - corresponding test was oldest_entry_evicted_with_access_times_loaded_from_disk() that ensured that on startup only the newest file stayed in cache

If we're moving the files, it may be best to do it in the new_with_timeout_and_rename_fn so the appropriate rename_fn is used?

Can you identify?
Marcus Eagan
On Fri, Dec 13, 2024 at 10:17 Kirpal Grewal @.***>
wrote:

Additionally one of the commits that has gone into main appears to have
stopped cargo test -p nativelink-store compiling making it a lot harder
to work out why this is timing out
—
Reply to this email directly, view it on GitHub
https://github.com/TraceMachina/nativelink/pull/1489#issuecomment-2542004712,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR6TSD4N2LJ6KZBI7MTM7D2FMQE5AVCNFSM6AAAAABSHR5ODSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBSGAYDINZRGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Have a working (locally at least) version using the specified rename function to move old cache files from their previous location to under DigestFolder

assume this broke with dependency upgrade
I can reproduce with cargo. Working on finding a fix. In the meantime, the Bazel test still works.

assume this broke with dependency upgrade

I can reproduce with cargo. Working on finding a fix. In the meantime, the Bazel test still works.
bazel test ...

Thanks - managed to get a working version now too (the advantage of cargo test was so I could have had more fine grained control of which tests to run / no-capture to see debug output, compared to bazel test //...)


Menu entry swapper for bank all on shift click

Enable bank-all/secondary banking on shift click
Describe the solution you'd like
While holding shift and clicking a stack of items in the inventory it would deposit all off the stack into the bank. (swapping deposit 1 with deposit all)
This would help when managing lots of items quicker.

This has been implemented in #10631 and can be found under the 'Menu Entry Swapper' plugin settings.


hijack presents one option if 2 tasks are named the same

When creating a test pipeline I added two tasks with the same name in a single job. When I hijacked in I was presented with only one of the steps and I'm not sure how fly chose which one to use. The fix is likely to, not name 2 things the same, but seems like it's worth bringing up. Example yml:
      - task: some-task
        config:
          platform: linux
          run:
            path: sh
            args:
            - -c
            - echo "things"
      - task: some-task
        config:
          platform: linux
          run:
            path: sh
            args:
            - -c
            - echo "other-things"

The fly hijack helper only outputs a single task named some-task.

I think this is fixed now.


Generalize the tuning code in tools/tune/tune.c

Right now the code works as expected only if the platform is "Advanced Micro Devices, Inc." AND the platform has a GPU.
Generalize this tuning code to work for a given (platform, device) combination.

I assigned it to myself as I have begun some work on this already. I am trying to add the following parameters.
tune --list

This will list all the available platforms and devices available on the machine.
tune --platform pid --device id

This will for the specified platform and device. When an option is not provided, it will assume the id is 0.


Remove read_attrs and config_attrs from repr

The repr method returns a string that is to long and confusing, I have removed read_attrs and config_attrs from the string in order to reduce the complexity.

Thanks for the fast turnaround, @awalter-bnl!
For anyone following alone, here is some sample output from current master, for context. It makes exceptions way too verbose.
QuadEM(prefix='qem:', name='qm', read_attrs=[], configuration_attrs=['integration_time',
'averaging_time', 'current_names', 'current_offsets', 'current_offset_calcs', 'current_scales', 'current1',
'current1.configuration_names', 'current1.port_name', 'current1.asyn_pipeline_config',
'current1.blocking_callbacks', 'current1.enable', 'current1.nd_array_port', 'current1.plugin_type',
'current1.bgd_width', 'current1.centroid_threshold', 'current1.compute_centroid',
'current1.compute_histogram', 'current1.compute_profiles', 'current1.compute_statistics',
'current1.hist_max', 'current1.hist_min', 'current1.hist_size', 'current1.profile_cursor',
'current1.profile_size', 'current1.ts_control', 'current1.ts_num_points', 'current2',
'current2.configuration_names', 'current2.port_name', 'current2.asyn_pipeline_config',
'current2.blocking_callbacks', 'current2.enable', 'current2.nd_array_port', 'current2.plugin_type',
'current2.bgd_width', 'current2.centroid_threshold', 'current2.compute_centroid',
'current2.compute_histogram', 'current2.compute_profiles', 'current2.compute_statistics',
'current2.hist_max', 'current2.hist_min', 'current2.hist_size', 'current2.profile_cursor',
'current2.profile_size', 'current2.ts_control', 'current2.ts_num_points', 'current3',
'current3.configuration_names', 'current3.port_name', 'current3.asyn_pipeline_config',
'current3.blocking_callbacks', 'current3.enable', 'current3.nd_array_port', 'current3.plugin_type',
'current3.bgd_width', 'current3.centroid_threshold', 'current3.compute_centroid',
'current3.compute_histogram', 'current3.compute_profiles', 'current3.compute_statistics',
'current3.hist_max', 'current3.hist_min', 'current3.hist_size', 'current3.profile_cursor',
'current3.profile_size', 'current3.ts_control', 'current3.ts_num_points', 'current4',
'current4.configuration_names', 'current4.port_name', 'current4.asyn_pipeline_config',
'current4.blocking_callbacks', 'current4.enable', 'current4.nd_array_port', 'current4.plugin_type',
'current4.bgd_width', 'current4.centroid_threshold', 'current4.compute_centroid',
'current4.compute_histogram', 'current4.compute_profiles', 'current4.compute_statistics',
'current4.hist_max', 'current4.hist_min', 'current4.hist_size', 'current4.profile_cursor',
'current4.profile_size', 'current4.ts_control', 'current4.ts_num_points'])

After a little discussion on NSLS-II Slack, we arrived at the consensus that alternatives, like showing only a partial list in the repr is more potentially confusing than helpful. The most important information in a Device repr is the class, name, and parent.

I agree @danielballan that we need to think about this (which I didn't). perhaps we can separate out str and repr which are currently set to the same. and have one with the read/ config attrs and one without?

@awalter-bnl Not a bad idea in principle. I think a big wall of text for either print(...) or repr(...) could be surprising at times, so given the string of approvals here I'm inclined to merge. OK by you?

@danielballan Ok with me, I was just suggesting it as a way around a problem that people think is such a problem

I've always been a big fan of having objects be represented in a way that you could recreate them through their repr, and against things that break it. That said, I don't disagree that a 2000 character repr string is unwieldly and ugly.
My initial thought: if the user leaves the configuration/hints as in the class definition, then they could be removed from the repr without loss of information. It's when that list starts getting customized in different ways that this information is now important. Perhaps eventually something could be done with this down the line in a separate PR.

Thanks @klauer. I think that is worthwhile and feasible. I created an issue: https://github.com/NSLS-II/ophyd/issues/561
@awalter-bnl I suspect that Ken defended his eval-able reprs with a test, which is why all of the builds have failed twice in a row. If I'm right about that, this PR will need to update or remove that test. Can you go digging for it in the Travis logs?

All builds passed except the nightly. Thanks, @awalter-bnl!

Rebased to squash into one commit.


docs(contributors): Add nkbt to contributors list

What: Add myself to contributors list

Why: Contributed to docs

You need to add yourself as a contributor by following this guide. It will update the .all-contributorsrc file, which is used to generate the contributors list later on (otherwise you'll be removed next time the list gets updated).
Sorry you had to go through the process of manually adding yourself :s

Yup, what @jfmengels said

Sure, but honestly, who reads readmes? 😜
On Thu, 19 May 2016 at 23:31, Kent C. Dodds<EMAIL_ADDRESS>wrote:

Yup, what @jfmengels https://github.com/jfmengels said
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/kentcdodds/p-s/pull/11#issuecomment-220324657

Fixed.

LGTM


Linux support

This commit adds support for Linux, this was the only thing I had to change to get it up and running.

Nice! Thank you @torkelrogstad, all of us have Macs and totally got outta whack for multiple OS support. I am gonna be super picky with code style haha hope you don't mind. If you want I can make the changes myself, lemme know.

Better?

Yep! Nice, merging


Turn pinned sliver header temporarily to floating pinned sliver header when showOnScreen is called.

Description
Still need to update the docs. Running on frob to see how much of a breaking change this is.
With this change, a pinned non-floating header can temporarily turn itself into a pinned floating header when showOnScreen is called. This is to prevent the showOnScreen call from scrolling the content to the top, in an effort to reveal the contents of the header.
Also snap is allowed for pinned headers and will snap the header back to its normal state when it's currently in a "floating" state.
Example
void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      home: Scaffold(
        body: MyHomePage(),
      ),
    );
  }
}

class MyHomePage extends StatefulWidget {
  @override
  State<StatefulWidget> createState() => PageState();
}

class PageState extends State<MyHomePage> {
  final Key centerKey = const Key('center');
  @override
  Widget build(BuildContext context) {
    return CustomScrollView(
      // center: centerKey,
      // reverse: true,
      slivers: <Widget>[
        SliverAppBar(
          pinned: true,
          //floating: true,
          title: const Text('YAY'),
          expandedHeight: 500,
          showOnScreenConfiguration: PersistentHeaderShowOnScreenConfiguration.maxExpansion,
          flexibleSpace: FlexibleSpaceBar(title: ColoredBox(color: Colors.red, child: TextField(scrollPadding: EdgeInsets.only(top: 200)))),
        ),
         SliverAppBar(
          pinned: true,
          //floating: true,
          title: const Text('YAY'),
          expandedHeight: 500,
          showOnScreenConfiguration: PersistentHeaderShowOnScreenConfiguration.maxExpansion,
          snap: true,
          flexibleSpace: FlexibleSpaceBar(title: ColoredBox(color: Colors.red, child: TextField(scrollPadding: EdgeInsets.only(top: 200)))),
        ),
         SliverAppBar(
          pinned: true,
          //floating: true,
          title: const Text('YAY'),
          expandedHeight: 500,
          showOnScreenConfiguration: PersistentHeaderShowOnScreenConfiguration.maxExpansion,
          flexibleSpace: FlexibleSpaceBar(title: ColoredBox(color: Colors.red, child: TextField(scrollPadding: EdgeInsets.only(top: 200)))),
        ),
        ...List<Widget>.generate(20, (int i) {
            final Widget sliver = SliverToBoxAdapter(
              child: Container(
                color: Colors.red,
                height: 100.0,
                child: Text('Tile $i'),
              ),
            );
            return SliverPadding(
              key: i == 19 ? centerKey : null,
              padding: const EdgeInsets.only(top: 2.0, bottom: 100.0),
              sliver: sliver,
            );
        })
      ] ,
    );
  }
}

Related Issues

https://github.com/flutter/flutter/issues/64973

Tests
I added the following tests:
Replace this with a list of the tests that you added as part of this PR. A change in behavior with no test covering it
will likely get reverted accidentally sooner or later. PRs must include tests for all changed/updated/fixed behaviors. See Test Coverage.
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I signed the CLA.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] All existing and new tests are passing.
[ ] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[ ] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

cl/331074687

@chunhtai @HansMuller does this change make sense?


It will trigger modal's "hide.bs.modal" event when close datepicker

Expected behaviour
It will not trigger modal's "hide.bs.modal" event when close datepicker.
Actual behaviour
It will not trigger modal's "hide.bs.modal" event when close datepicker. But why ?
Datepicker version used
ex. 3.0.0
Example code
bug.zip

Sorry, found the fix
https://github.com/eternicode/bootstrap-datepicker/pull/962

It's same to the show.bs.modal


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 2963 ms

Resolved: Bofrost.at performance has improved in c05dec1.


inlet/outlet sensors not updating automatically

The inlet and outlet temperatures no longer update automatically in home assistant. The temperature stays the same for days. The temperature will update if i start the control-R 2.0 android app and click "refresh results" on the status page. For what it's worth the temperatures reported in the control-R 2.0 android app and in home assistant are consistent.
After installing a new control-R this bug was exposed, and I now have two of them associated with my rinnai account.  With my older and single control-r this wasn't an issue. It is possible having two control-R devices is related to this problem, or not.

It was checked (has been untouched for over a year), and unchecking, saving, checking and saving fixed it.
Perhaps the bug is “run maintenance interval checkbox does not propagate to newly added (second) control-r device when more than one control-r device exists”.  Just a guess.
From: explosivo22 @.>
Sent: Tuesday, December 6, 2022 5:19 AM
To: explosivo22/rinnaicontrolr-ha @.>
Cc: Andrew Radin @.>; Author @.>
Subject: Re: [explosivo22/rinnaicontrolr-ha] inlet/outlet sensors not updating automatically (Issue #45)
Can you check to see if the maintenance retrieval is checked under option when you click on configure for the integration?
If it is enabled, try toggling it off and saving and then toggling it back on and saving again. I had a bug #34https://github.com/explosivo22/rinnaicontrolr-ha/issues/34 that caused this setting to get messed up on occasion.
—
Reply to this email directly, view it on GitHubhttps://github.com/explosivo22/rinnaicontrolr-ha/issues/45#issuecomment-1339376118, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAGYM3WZB37MLB3NYF3N6YLWL44MXANCNFSM6AAAAAASUTPPE4.
You are receiving this because you authored the thread.Message ID: @.***>


BUG PHPUnit shiv breaks code inspection in all unit tests

This breaks all code auto-inspection in unit tests.
Related to https://github.com/silverstripe/silverstripe-framework/pull/6833.
Ideal fix would continue to address the original issue without the unfortunate side effects.

It's an annoyance, but much less important than other bugs in the beta milestone - removing milestone.

This file has been removed with the SapphireTest changes a while back; No longer an annoyance!


NOMRG Augustin de Saint Affrique

@AdeStAff cf msg above, PR is not complete

deleted .DS_Store file as requested

:+1:


Ask for login appropriate to the API selected

Running locally, after I log in (via Facebook),
on the /api page, I click "Twillio".
I fillout the number and a message "Hello".
Click Send. I get:
Connect
500 Authenticate
The log reports:
GET /api/twilio 200 189.969 ms - -
Authenticate
POST /api/twilio 500 364.286 ms - -

@hotwilson Do you have updated the env config variables for twilio?

I suspect the problem was resolved in the particular case since the requester hasn't got back here in a while.  Closing the issue.


More block explorer updates

Updated for RPC changes around quorum state calls in 4.0.2 (replaced batch quorum call with new quorum states call).
Adds checkpoint quorum display to quorum states page and to main index
Add display of a single obligations or checkpoint quorum, linked where appropriate
Cut off quorum display after 20 nodes with a "+ 7 more ↪" that links to the full obligations quorum details.
Adds list of last three checkpoints to the main page (and a link to a page with up to 50).

(This is live on https://tn.imaginary.stream)

This is going to need some additional changes for the RPC changes in dev/dev PRs.  Will come back to update once they are settled.


upgrade vue-loader

https://vue-loader.vuejs.org/migrating.html

implement this, too: https://github.com/GoogleChromeLabs/webpack-libs-optimizations


6 failing test cases for i686 platform during fedora 32/rawhide build

Problem description
During the build of pyproj for Fedora rawhide (32) for pyproj release 2.4.1 I switched on the new pytest test system.
The build went well for all supported platforms, but running the test suite failed on the i686 platform because of 6 failed test cases.
The log file for this build is available on:
https://koji.fedoraproject.org/koji/taskinfo?taskID=38858317
Details for failing tests start at line 631 in the build.log file.
Expected Output
all tests should pass
Environment Information
py.test-3 output
============================= test session starts ==============================
platform linux -- Python 3.8.0, pytest-4.6.6, py-1.8.0, pluggy-0.12.0
rootdir: /builddir/build/BUILD/pyproj-2.4.1
collected 325 items
...
proj version 6.2.0 (as packaged for fedora-rawhide)
Installation method
from source: pyproj-2.4.1.tar.gz as downloaded from pypi

for reference, attached please find a copy of this log file:
build.log.i686.gz

I see you are using Python 3.8. For some reason the i686 builds didn't work for the wheels on Python 3.8.

https://travis-ci.com/pyproj4/pyproj-wheels/jobs/251285905

From pyproj wheel logs:
**********************************************************************

File "/venv/lib/python3.8/site-packages/pyproj/transformer.py", line 720, in pyproj.transformer.transform

Failed example:

    '%9.3f %11.3f' % (x2,y2)

Expected:

    '569722.342 4268814.028'

Got:

    '      inf         inf'

Trying:

    '%8.3f %5.3f' % p2(x2,y2,inverse=True)

Expecting:

    ' -92.200 38.567'

**********************************************************************

File "/venv/lib/python3.8/site-packages/pyproj/transformer.py", line 722, in pyproj.transformer.transform

Failed example:

    '%8.3f %5.3f' % p2(x2,y2,inverse=True)

Expected:

    ' -92.200 38.567'

Got:

    '1000000000000000019884624838656.000<PHONE_NUMBER>000000019884624838656.000'
File "/venv/lib/python3.8/site-packages/pyproj/transformer.py", line 730, in pyproj.transformer.transform

Failed example:

    '%9.3f %9.3f %9.3f %11.3f %11.3f %11.3f' % xy

Expected:

    '567703.344 351730.944 728553.093 4298200.739 4353698.725 4292319.005'

Got:

    '432296.656 648269.056 271446.907 -15697729.147 -15642231.161 -15703610.881'
File "/venv/lib/python3.8/site-packages/pyproj/transformer.py", line 733, in pyproj.transformer.transform

Failed example:

    '%9.3f %9.3f %9.3f %11.3f %11.3f %11.3f' % xy

Expected:

    '567721.149 351747.558 728569.133 4297989.112 4353489.645 4292106.305'

Got:

    '      inf       inf       inf         inf         inf         inf'
File "/venv/lib/python3.8/site-packages/pyproj/transformer.py", line 737, in pyproj.transformer.transform

Failed example:

    '%8.3f %8.3f %8.3f %5.3f %5.3f %5.3f' % xy

Expected:

    ' -92.220  -94.720  -90.370 38.830 39.320 38.750'

Got:

    '1000000000000000019884624838656.000<PHONE_NUMBER>000000019884624838656.000<PHONE_NUMBER>000000019884624838656.000<PHONE_NUMBER>000000019884624838656.000<PHONE_NUMBER>000000019884624838656.000<PHONE_NUMBER>000000019884624838656.000'

From fedora:
**********************************************************************
File "/builddir/build/BUILDROOT/pyproj-2.4.1-1.fc32.i386/usr/lib/python3.8/site-packages/pyproj/transformer.py", line 718, in pyproj.transformer.transform
Failed example:
    '%9.3f %11.3f' % (x1,y1)
Expected:
    '569704.566 4269024.671'
Got:
    '430295.434 -15726905.215'
**********************************************************************
File "/builddir/build/BUILDROOT/pyproj-2.4.1-1.fc32.i386/usr/lib/python3.8/site-packages/pyproj/transformer.py", line 720, in pyproj.transformer.transform
Failed example:
    '%9.3f %11.3f' % (x2,y2)
Expected:
    '569722.342 4268814.028'
Got:
    '      inf         inf'
**********************************************************************
File "/builddir/build/BUILDROOT/pyproj-2.4.1-1.fc32.i386/usr/lib/python3.8/site-packages/pyproj/transformer.py", line 722, in pyproj.transformer.transform
Failed example:
    '%8.3f %5.3f' % p2(x2,y2,inverse=True)
Expected:
    ' -92.200 38.567'
Got:
    '1000000000000000019884624838656.000<PHONE_NUMBER>000000019884624838656.000'
**********************************************************************
File "/builddir/build/BUILDROOT/pyproj-2.4.1-1.fc32.i386/usr/lib/python3.8/site-packages/pyproj/transformer.py", line 730, in pyproj.transformer.transform
Failed example:
    '%9.3f %9.3f %9.3f %11.3f %11.3f %11.3f' % xy
Expected:
    '567703.344 351730.944 728553.093 4298200.739 4353698.725 4292319.005'
Got:
    '432296.656 648269.056 271446.907 -15697729.147 -15642231.161 -15703610.881'
**********************************************************************
File "/builddir/build/BUILDROOT/pyproj-2.4.1-1.fc32.i386/usr/lib/python3.8/site-packages/pyproj/transformer.py", line 733, in pyproj.transformer.transform
Failed example:
    '%9.3f %9.3f %9.3f %11.3f %11.3f %11.3f' % xy
Expected:
    '567721.149 351747.558 728569.133 4297989.112 4353489.645 4292106.305'
Got:
    '      inf       inf       inf         inf         inf         inf'
**********************************************************************
File "/builddir/build/BUILDROOT/pyproj-2.4.1-1.fc32.i386/usr/lib/python3.8/site-packages/pyproj/transformer.py", line 737, in pyproj.transformer.transform
Failed example:
    '%8.3f %8.3f %8.3f %5.3f %5.3f %5.3f' % xy
Expected:
    ' -92.220  -94.720  -90.370 38.830 39.320 38.750'
Got:
    '1000000000000000019884624838656.000<PHONE_NUMBER>000000019884624838656.000<PHONE_NUMBER>000000019884624838656.000<PHONE_NUMBER>000000019884624838656.000<PHONE_NUMBER>000000019884624838656.000<PHONE_NUMBER>000000019884624838656.000'

Looks like the same failures. Not sure why Python 3.8 on i686 does this. It works for Python 3.6 and Python 3.7 for i686 for the wheels.

Thanks for noticing the similarity.
I will try to dig deeper when I find the time.

Thanks for noticing the similarity.
I will try to dig deeper when I find the time.

Thanks, that would be great 👍


Installation issue, node version conflict

I'm getting this issue when installing.
npm WARN engine<EMAIL_ADDRESS>wanted: {"node":"0.10.*"} (current: {"node":"4.2.3","npm":"2.14.7"})
Killed

My specs:
# node -v
v4.2.3
# npm -v
2.14.7

svgicons2svgfont needs to be updated, pull request are welcome.

Updates on this or in the pull request?
Thanks!


cmd/vet: shadow false positive when using exported function

go tool vet --shadow returns a false positive when the variable being shadowed is declared from the return of an exported user function from another package. That's a mouthful, so here's some code:
./other/other.go:
package other

func Foo() (int, error) {
	return 0, nil
}

./shadow_test.go:
package shadow

import (
	"errors"
	"testing"

	"github.com/tamird/shadow/other"
)

func foo() (int, error) {
	return 0, nil
}

func TestVetShadow(t *testing.T) {
	// Local varibles: this passes.
	// a, err := "a", errors.New("Foo")

	// Function from same package: this passes.
	// a, err := foo()

	// Function from standard library: this passes.
	// r := &bytes.Buffer{}
	// a, err := r.Read(nil)

	// Function from different package: this triggers shadowing warning.
	a, err := other.Foo()

	if err != nil {

	}

	if _, err := other.Foo(); err != nil {
	}

	b, err := "b", errors.New("Foo")
	if err != nil {
	}

	_, _ = a, b
}

The comments in the code describe the problem. Use other.Foo() and you get a false positive; use any of the others (local function, local variables, exported method from the stdlib) and the warning goes away. Weird!
All the code is also available here: https://github.com/tamird/shadow
cc @mberhault

If you run go install github.com/tamird/shadow/other and then re-run vet, does the false positive still show up?

Yes.

Thanks.
/cc @robpike

The shadow code is marked experimental. It has too many false positives to be enabled by default, so this is not entirely unexpected, but don't expect a fix soon. The right way to detect shadowing without flow analysis is elusive.

@robpike flow analysis doesn't seem relevant here. @mberhault and I dug into this a bit and discovered that go/types (underlying vet) regards the return of other.Foo() to be of unknown type. We didn't get to the bottom of it, but there doesn't seem to be anything requiring more sophisticated tools in this example; this is an honest bug.


Spike: Explore ways to scale our GitHub data collection

Currently, we have been relying on fetching a known number of GitHub repositories from the Augur database for our graphical  analysis work. However, in the future we would like to apply our graphical techniques to a much wider (unknown) set of relevant repositories that exist in GitHub. For this, we will need to explore different ways to scale our data collection process such as leveraging GitHub APIs and their functionalities.

The proposed ways we have considered to extend our list of Github projects are:

Using the Github API to scan through OSS projects & filter by org level projects, forks, stars
Project dependencies using https://www.stackaid.us/ and https://thanks.dev/home
Get list of contributed projects from interesting user groups using the Github API

Explore https://docs.github.com/en/graphql to scan through & query repos on Github

A couple proposed filters that will hopefully allow new github repos to be queried in a small enough number to regularly analyze to identify "interesting" new repos to track via Aspen:

select repos with >= k forks, where k > 0 but exact number is experimentally determined. Generally k should be large enough to filter out at least 90% of repos but not larger than necessary. (90% is also a parameter to be determined experimentally)
since the goal is to identify interesting new projects, we can probably ignore any repo that is older than x months, where x is experimentally determined but probably something like 6 months.  Initial searches might go back a little farther until a regular search workflow is established.
possibly also select repos with a minimum of other activity (prs, issues, commits, etc).  However, number of forks may turn out to be a sufficient surrogate for activity.

Fyi - @JamesKunstle @cdolfi mentioned that Augur has been building on functionality to go from user id to list of projects to they have contributed to. This will be very helpful in expanding our initial list of projects that we include into Augur and scale our data collection.

Looked into https://docs.github.com/en/graphql and https://docs.sourcegraph.com/ for scaling our data collection.
Some observations based on my initial research:

GraphQL - has an Explorer UI to test and run queries. A nice way to learn and test your queries. I was able to stitch together a query to pull all public, non-forked repositories and the first n issues, PRs in each of the repositories. However, I was unable to fetch the contributor information as its restricted based on authentication tokens and need to look into if specific privileges are needed for this.

Sourcegraph - tool that leverages GitHub GraphQL to make it easy for developer to search for code across GitHub repositories. Although its primarily used for code search, we can modify our queries to filter for public GitHub repositories. Still exploring queries for fetching issue, PRs and contributor information.

One thing to keep in mind while using both the tools are the API request limitations. Although there are limits on the number of queries we can make per hour, we can still use it as a first pass for collecting our initial repo set.

@oindrillac @hemajv Should this be 'In Progress' at this stage, or should it be closed / backlogged?

This was a spike and looks complete. I think we can close this out.


Allow percent width and height of large image

In my project, some pictures are too small, and large picture shows smaller then small one (which covers container element 100% width). In this case I pass width: "100%", height: "100%" to large image props. It works, but I get warning:

index.js:1 Warning: Failed prop type: Invalid prop largeImage.width of type string supplied to EnlargedImage, expected number.

YES PLEASE!

YES PLEASE!


error in iOS example

TypeError: undefined is not a constructor (evaluating 'new _reactNativeSpruce.CorneredSort(60)')

This error is located at:
    in App (at renderApplication.js:33)

render
    App.js:8:36

The issue is here:
https://github.com/prscX/react-native-spruce/blob/7d4e556ab8e5cecc4f35ccd0c0998156a5ddd82b/SpruceExample/App.js#L8
steps
git clone git@github.com:prscX/react-native-spruce.git
cd react-native-spruce/SpruceExample
yarn
react-native run-ios

Got this warning when building the app:
The following commands produced analyzer issues:
	Analyze Base/RCTModuleMethod.mm normal x86_64
(1 command with analyzer issues)

Extra info:
$ react-native info

Environment:
  OS: macOS High Sierra 10.13.4
  Node: 8.11.1
  Yarn: 1.5.1
  npm: 5.8.0
  Watchman: 4.9.0
  Xcode: Xcode 9.3.1 Build version 9E501
  Android Studio: 3.1 AI-173.4720617

Packages: (wanted => installed)
  react: 16.3.1 => 16.3.1
  react-native: 0.55.4 => 0.55.4

Thanks @gianpaj for raising the issue and providing a detailed description.
Which platform are you trying to test (Android/iOS)?

Note: On iOS it is not currently supported. Please refer below issue for the same:

Issue: 101
Soon I am planning to work on Objective-C wrapper in order enable this library for iOS as well.
Thanks
</ Pranav >

got it. would be useful probably to update the readme file then:

React Native Spruce - iOS & Android


Can we got a permanent static download link for the latest build?

Is there anyway that you could offer a persistent download link such as the one from arcdps "https://www.deltaconnected.com/arcdps/x64/d3d9.dll" that will always have the latest build?
This is make the player's lives easier by making the download process a lot easier.
Currently, going to https://github.com/Krappa322/arcdps_healing_stats/releases/latest shows the 2.4 version, but the 2.5rc2  is actually the latest version.
And attempting to download the file directly using the path fails too: https://github.com/Krappa322/arcdps_healing_stats/releases/download/v2.5rc2/arcdps_healing_stats.dll
Please assist.
Thanks.

2.5rc2 is a pre-release, which is why it's not shown when looking at latest release. When deemed stable, it will be released and then show up as the latest release :)
You can also control this through the "auto update" option in-game, where you can choose whether you want to automatically download latest stable or latest pre-release

Where is the auto update option?
I don't see it.

SJ.

On Thu, Jun 30, 2022 at 11:43 AM Kappa322 @.***> wrote:

2.5rc2 is a pre-release, which is why it's not shown when looking at
latest release. When deemed stable, it will be released and then show up as
the latest release :)
You can also control this through the "auto update" option in-game, where
you can choose whether you want to automatically download latest stable or
latest pre-release
—
Reply to this email directly, view it on GitHub
https://github.com/Krappa322/arcdps_healing_stats/issues/23#issuecomment-1171378415,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHKJHF4YXEVZJGE76DU7B7TVRW6A5ANCNFSM52JPIJKA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Sorry, I mixed up my addons. I've yet to make one for this addon

Auto update is tracked in https://github.com/Krappa322/arcdps_healing_stats/issues/24
Static download link is not something I'm interested in pursuing right now

Auto updating has been added in the latest pre-release


[Storage] Use Pick/Omit feature in typescript interfaces

There is a lot of duplication w.r.t the attributes in the interfaces which can be simplified.
https://www.typescriptlang.org/docs/handbook/utility-types.html#picktk

We will consider using these for new interfaces if applicable. But will only modify existing interfaces if they are causing issue.
Could we close this or do you want to use this to track interfaces that you think we should change?

Since the library is stable, I don't see many returns for the investment here. Let's close!


Folder hierarchy delimiter

hi cory ,
first i would like to say thank you so much - great job man !!!
there is a request about the delimiter .
if i want to make a request from a bucket -  only folders or some different  hierarchy and not all the hierarchy i have to use a delimiter  "/".
i saw that you didn't implement this issue.
backblaze related that issue
https://www.backblaze.com/b2/docs/b2_list_file_names.html
is there any chance that you will add it soon.. very soon :)

When I originally wrote this I either missed that, or it wasn't there yet. I was unaware of such a thing. I have some time this week, I'll try to implement it then.

Perfect news man . Waiting for update.
On Dec 24, 2017 17:17, "Cory"<EMAIL_ADDRESS>wrote:

When I originally wrote this I either missed that, or it wasn't there yet.
I was unaware of such a thing. I have some time this week, I'll try to
implement it then.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/coryrwest/B2.NET/issues/8#issuecomment-353789603, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AhM3Rb17pA-88UcdeG7U_7PYAJzKtqFUks5tDmsmgaJpZM4RL8WC
.

Hi cory ,
do you have any update ??

Just released 0.4.5 with this new functionality to nuget.


Key error when setting units on AMI430

The AMI430 magnet power supply produces an error while setting the units (either field_units or ramp_rate_units). Below, an example is given for the field_units.
Steps to reproduce

Initialize power supply:
ix = AMI430("x2", address="<IP_ADDRESS>", port=7180)

Read units from power supply (example given for field_units):
ix.field_units()
This produces the following expected output:
'tesla'

Set unit (example given for field_units):
ix.field_units('kilogauss')
This does not work as expected

The error can also be reproduced by:
ix.field_units(ix.field_units())

Expected behaviour
It should silently set the field_unit to kilogauss (or tesla). Without any error.
Actual behaviour
It produces a key error:
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
<ipython-input-7-f9d194d73bef> in <module>
----> 1 ix.field_units('kilogauss')

c:\users\g-gre-gre058050\qcodes\qcodes\instrument\parameter.py in __call__(self, *args, **kwargs)
    354         else:
    355             if hasattr(self, 'set'):
--> 356                 self.set(*args, **kwargs)
    357             else:
    358                 raise NotImplementedError('no set cmd found in' +

c:\users\g-gre-gre058050\qcodes\qcodes\instrument\parameter.py in set_wrapper(value, **kwargs)
    557             except Exception as e:
    558                 e.args = e.args + ('setting {} to {}'.format(self, value),)
--> 559                 raise e
    560
    561         return set_wrapper

c:\users\g-gre-gre058050\qcodes\qcodes\instrument\parameter.py in set_wrapper(value, **kwargs)
    541                     t0 = time.perf_counter()
    542
--> 543                     set_function(raw_value, **kwargs)
    544                     self.raw_value = raw_value
    545                     self._save_val(val_step,

c:\users\g-gre-gre058050\qcodes\qcodes\utils\command.py in __call__(self, *args)
    175             raise TypeError(
    176                 'command takes exactly {} args'.format(self.arg_count))
--> 177         return self.exec_function(*args)

c:\users\g-gre-gre058050\qcodes\qcodes\instrument_drivers\american_magnetics\AMI430.py in <lambda>(units)
    154                            get_cmd='FIELD:UNITS?',
    155                            set_cmd=(lambda units:
--> 156                                     self._update_units(field_units=units)),
    157                            val_mapping={'kilogauss': 0,
    158                                         'tesla': 1})

c:\users\g-gre-gre058050\qcodes\qcodes\instrument_drivers\american_magnetics\AMI430.py in _update_units(self, ramp_rate_units, field_units)
    452         else:
    453             self.write("CONF:FIELD:UNITS {}".format(field_units))
--> 454             field_units = self.field_units.val_mapping[field_units]
    455
    456         # Map to shortened unit names

KeyError: (0, 'setting x2_field_units to kilogauss')

System
operating system
Win10
qcodes branch
master
qcodes commit
SHA-1: 134ddc1de0d5fddf49f65bb32d1257e5bd53bf06

Merge pull request #1688 from jenshnielsen/newenv

Update conda env to python 3.7

I think this is the issue resolved by https://github.com/QCoDeS/Qcodes/pull/1599 which is just waiting on a test on a real instrument.

Yes, it looks exactly like this. Sorry, I didn't find  #1599 before. I think one can then close this issue.

No worries. If you have a chance to do so it would be great if you can test the code from that pr and confirm that it resolves the issue

The code from #1599 fixes the above mentioned issue.

@simonzihlmann Thanks a lot. I will go ahead and merge :)


Support for nested attributes.

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
✔

Fixed issues
reference #17

The only reason this PR also breaks BC is that I have removed public setAttributes method from FormModelInterface and FormModel as it didn't have any reason to be there. The same thing could have been done with load. Even documentation for load was saying the same thing, but wasn't supporting that behavior. I have also added new public method to the interface and class setAttribute(string $name, $value) which encapsulated some of the logic from setAttributes that was removed and this new method is used in my implementation of nested attributes.

Also docs are missing, there is already issue #7 for adding overall documentation.

The current format for nested attribute is attribute.nestedAttribute. I was considering attribute[nestedAttribute] as that would allow easier usage in API context (JSON), but I haven't looked through whether that format will conflict with array attributes (if they are supported at all). So maybe in the future :)

I don't like how nested attribute labels and hints are got. But would prefer to refactor that as a separate PR with issue. There is some things that need discussion beforehand.

👍

@samdark Oops, you merged it :) the validation and errors are not working for nested attributes, I wrote to @terabytesoftw earlier about that and forgot to mention it here. I was finishing the work on it, to add another commit. Will open a new PR with added parts and some refactoring for the merged code.


Support for custom dns settings

What this PR does / why we need it:
Add support for custom DNS settings for the cert-manager pod
This allows HTTP / DNS challenges to be self checked in AWS setups in which both a public and private DNS zone are present for the same domains. Without this the pod will never see the challenge that is going to be presented to LE because they'll be added to public ingresses / DNS zones, failing with something along these lines:
I0427 13:23:33.027444       1 sync.go:241] Error preparing issuer for certificate frehm/testing: http-01 self check failed for domain "testing.dev.k8s.cafe

Which issue this PR fixes:
I remember seeing some comments / issues related to this but haven't had a chance to look around again.
Special notes for your reviewer:
This is my first ever contribution to a Helm chart. LMK if it should be done in a different way.
Add support for custom DNS settings for the cert-manager pod (Kubernetes 1.10+)

Hey @kragniz! Is this something you guys are really interested in? I can rebase if you are!
Thanks in advance.

Sorry for the delay getting back to you -
Thanks very much for the contribution, there's been quite a few requests for this!
This looks good to me - would you mind rebasing so I can add the lgtm labels?
You'll also need to bump the version number in the Helm chart, and afterward run ./hack/update-deploy-gen.sh to regenerate the static deployment manifest files.

/ok-to-test

@munnerz

Sorry for the delay getting back to you -

All good :)

This looks good to me - would you mind rebasing so I can add the lgtm labels?

Done!

You'll also need to bump the version number in the Helm chart, and afterward run ./hack/update-deploy-gen.sh to regenerate the static deployment manifest files.

Also done, I bumped the chart to 0.3.2, I was thinking about going 0.4.0 but looking at the chart's history I think you guys keep the chart version "more or less in sync" with the app version right? LMK if I should tag as 0.4.0!

Thanks! 😄
I'll make a PR to cherry pick this change into release-0.3 shortly 😄
/lgtm
/approve


MUSL wheels are broken

Hi!
Just tried to install pydantic-core on a python:3.10-alpine docker image. The installation works, but then the library is not usable:
/wrk # pip install pydantic-core
Collecting pydantic-core
  Downloading pydantic_core-0.7.1-cp310-cp310-musllinux_1_1_x86_64.whl (1.7 MB)
     |████████████████████████████████| 1.7 MB 8.3 MB/s
Collecting typing_extensions
  Downloading typing_extensions-4.4.0-py3-none-any.whl (26 kB)
Installing collected packages: typing-extensions, pydantic-core
Successfully installed pydantic-core-0.7.1 typing-extensions-4.4.0
WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
WARNING: You are using pip version 21.2.4; however, version 22.3.1 is available.
You should consider upgrading via the '/usr/local/bin/python -m pip install --upgrade pip' command.
/wrk # python
Python 3.10.0 (default, Oct  6 2021, 00:17:54) [GCC 10.3.1 20210424] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import pydantic_core
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.10/site-packages/pydantic_core/__init__.py", line 1, in <module>
    from ._pydantic_core import (
ImportError: Error relocating /usr/local/lib/python3.10/site-packages/pydantic_core/_pydantic_core.cpython-310-x86_64-linux-gnu.so: (null): initial-exec TLS resolves to dynamic definition in /usr/local/lib/python3.10/site-packages/pydantic_core/_pydantic_core.cpython-310-x86_64-linux-gnu.so

To be clear, I'm reporting this because granian suffers from the same issue and since I use the same build strategy I was curious :)
It is not clear to me (yet?) if is related to the Maturin action, Maturin itself, PyO3 or Cargo.

Cc @messense who might have an idea on this?

Might be related to mimalloc, I've seen similar issues with jemalloc in the past.

Perhaps we can try to enable the local_dynamic_tls feature of mimalloc for musl to see if it make a difference. See also https://github.com/purpleprotocol/mimalloc_rust/pull/38
https://github.com/purpleprotocol/mimalloc_rust/blob/bd030f8fe56ddfefcfc7de8a843d34eb0cfe7a92/Cargo.toml#L32

weird, I get a seg fault instead.
I've tried mimalloc/local_dynamic_tls in #363.

@gi0baro can you check #363 is working for you, it's working for me.
It would be nice if we could run tests or at least run
python -c 'import pydantic_core._pydantic_core; print(pydantic_core._pydantic_core.__version__)'
python -c 'import pydantic_core; print(pydantic_core.SchemaValidator({"type": "int"}).validate_python("42"))'

For each binary, or most binaries, but I can't see a way without a lot of faff.

For each binary, or most binaries, but I can't see a way without a lot of faff.

You can sort of do that, see https://github.com/messense/py-dissimilar/blob/main/.github/workflows/CI.yml (search for run-on-arch-action and docker-run-action)

:+1: trying that now.

@gi0baro can you check #363 is working for you, it's working for me.

Just checked, works for me as well 🚀


remove redbull and brown ale copyright

About the PR
I'm no drinks expert but I placed an energy drink and recognised the brand straight off so thought that oughta change.
Went through the icons and noticed the brown ale star is definitely a trademark (It's from my town so yeah).
Otherwise, I'm no drinks expert but went through most of them and couldn't see anything glaring.
Screenshots

We also have "monkey energy" right?


Added a way to find the ubuntu parent distribution

This was only tested on Sylvia and might not be portable. I do not know how to test it somewhere else.
Proposed changes
I added a bash command that should help people figure out which ubuntu distribution their mint is based upon
Related issues (optional)
Closes #8699

Deploy preview for docsdocker ready!
Built with commit c0839b815ac0fe6d05c261293f03d520ea4306f9
https://deploy-preview-8700--docsdocker.netlify.com


domain

@GerbenJavado @EdOverflow
Is there any workaround to append domain name before the relative url's like full url ?

You can pipe the output into sed.
$ python linkfinder.py -i https://example.com/example.js -o cli | sed 's#^\/#https://example.com/#g'

Hi @EdOverflow thank you for your reply. Here as per your solution the parent only the parent domain is appended to all the relative urls. But I have some scenario where I use -d flag and linkfinder found some more sub domains and their relative url's too, so how we append them ?
Example:
python linkfinder.py -i https://example.com -d -o cli
/test
/test1
/test2
https://new.example.com
/demo1
/demo2
So in the above scenario there is one more subdomain and their relative url's found so how we append https://new.example.com too.
Regards,
@jaikishantulswani

Hi @EdOverflow any workaround for this ?

Hi @EdOverflow any workaround for this ?

@jaikishantulswani, as far as I can tell this might be too difficult to solve as is because it is not always clear which host the JS file belongs to. In your example, there is no guarantee that /demo2 belongs to https://new.example.com, so you might end up with a lot of false-positives.
I am open to suggestions though. Maybe someone out there knows of a potential solution for this problem.


Questions

You have done a very excellent work in CBCT intensity correction. I have some question, and i will appreciate it if you can help me. Is the label obtained by pCT and vCT? Could you please show me the code how to this?

@huangwei9 , I have not myself used the scatternet program, yet.
However, according to the paper on it, another open-source application is used to generate the input: https://gitlab.com/agravgaard/cbctrecon
I'm currently the primary developer (Previously it was  YK. Park I think scatternet was tested with this old version) on this as the German group, who were collaborating on ScatterNet, doesn't appear to open-source any of their other work (?).
CbctRecon requires the raw CB projections (or at least the CB geometry and the reconstructed CBCT) and a pCT to generate the scattermaps.
I would be happy to answer any questions you might have on CbctRecon (just open an issue on gitlab, or send me a mail)


id / localkey

Hey I try for ours to get id and localkey for device and tried different methods!
i am still trying this: https://github.com/AMoo-Miki/homebridge-tuya-lan/wiki
As soon I try to start Tuya-lan-find I get this error:
pi@raspberrypi:~ $ tuya-lan-find --ip <IP_ADDRESS>
/usr/local/lib/node_modules/homebridge-tuya-lan/bin/cli.js:175
let {address, port} = proxy.httpServer.address();
^
TypeError: Cannot read property 'address' of undefined
at /usr/local/lib/node_modules/homebridge-tuya-lan/bin/cli.js:175:44
at /usr/local/lib/node_modules/homebridge-tuya-lan/node_modules/http-mitm-proxy/lib/proxy.js:62:14
at /usr/local/lib/node_modules/homebridge-tuya-lan/node_modules/http-mitm-proxy/lib/ca.js:130:14
at /usr/local/lib/node_modules/homebridge-tuya-lan/node_modules/http-mitm-proxy/node_modules/async/dist/async.js:3888:9
at /usr/local/lib/node_modules/homebridge-tuya-lan/node_modules/http-mitm-proxy/node_modules/async/dist/async.js:473:16
at iterateeCallback (/usr/local/lib/node_modules/homebridge-tuya-lan/node_modules/http-mitm-proxy/node_modules/async/dist/async.js:988:17)
at /usr/local/lib/node_modules/homebridge-tuya-lan/node_modules/http-mitm-proxy/node_modules/async/dist/async.js:969:16
at /usr/local/lib/node_modules/homebridge-tuya-lan/node_modules/http-mitm-proxy/node_modules/async/dist/async.js:3885:13
at /usr/local/lib/node_modules/homebridge-tuya-lan/node_modules/mkdirp/index.js:48:53
at FSReqCallback.oncomplete (fs.js:167:21)
can u help me? I already closed all lan connections except wlan0

There is a more updated version of this plugin called Homebridge-Tuya.

If that that may help, here's the method I used to get my switch's ids and keys
https://www.youtube.com/watch?v=oq0JL_wicKg


udp(bitwig-studio-deb): 4.1.2 -> 4.2

udp(bitwig-studio-deb): 4.1.2 -> 4.2

@Zahrun do you want to connect this pacscript with repology and thus use Pacup?

@Zahrun do you want to connect this pacscript with repology and thus use Pacup?

2 days have passed so I assume that's a no then.

@Henryws i386 error.


logo proposal

HI, I @nostra13 I am a graphics designer, I am a graphics designer. I will like to know if you want me to create a new logo for your project.

Hey, why not :)

Ok, I will go and make a sketch and work on logo as soon as possible.

Thanks :+1:

@nostra13 after making different sketch I came up with this, you can make suggestions on any of the design and color.
I will make other sizes based on the one you choose.

@nostra13 here is a brief break down of what inspire the logo

@mlc @nostra13 @talklittle are you there?

@davidmind  Thanks for your efforts and explanation of design moves but it seems a little boring comparing to current logo :)

My thoughts about logo with bubbles: maybe it will be cooler to use stars (from small to larger) around center-pic and try other colors.

Oops i was trying to make it simple, but I will work on the star as you suggested with other colors.

@nostra13 this is what it turn out to be after making use of star on the final layout of the icon
also I try to give it a much brighter and cool color.

I like it more, definitely :) It reminds me something... maybe Chealse FC logo :) Nevermind.
Can you make samples on white background?
I think we can remove black crescent to simplify it)
Also I would appreciate if you try different options: squared/rounded background, with/without white circles near stars, different bg colors (solid/gradient/maybe other shades of green).

@nostra13 this is what I came up with

I think I have my favorite) Which one do you like?

If I am to choose I will choose number 2 or 4, which of it is your favorite?

I like 6/7 with solid green bg, probably #00673a.
Some suggestions (maybe final) to this option:

bg should be a perfect circle :)
make stars closer to edge (maybe cut out outer circle part)
centered pic should be perfectly aligned in the center
increase resolution:
- green bg circle - 72x72dp (288x288 px)
- whole image with transparent bg - 108x108dp (432x432 px)

@nostra13 have a quick look before I make it into transparent background

"make stars closer to edge" I meant like this

I made two different option for you to choose from
1

2

You make stars further from bg edge, not closer.

I just understand what you are saying, until you mention the background edge

Too close :) I showed the example earlier https://github.com/nostra13/Android-Universal-Image-Loader/issues/1340#issuecomment-484920037

What about now

I like it :+1:
Last step you can do - to make it perfect. If you wish:

make every star at the same distance from bg edge (now it's different)

rotate whole constellation clockwise a little bit to make beginning star and ending star aligned to center-pic
OR
rotate whole constellation counterclock-wise a little bit to make the biggest star in the vertical center

@nostra13 sorry I was a little bit busy this days, but I just check through your last comment I have made an ammendment. Check it out

I can see noticable disproportional gaps between stars (e.g. between 2nd and 3rd stars, between 6th and 7th stars, 11th and 12th). Gaps between stars should evenly decrease from the smallest to the biggest one.

Oops I must have missed that up, while trying to adjust the star. I just made an amendment though the space may look unequal, using smart guide everything is equal except you want to inspect yourself through the editable file.

Well, now I don't need a ruler to see that constellation isn't centered in the circle :)

At this point i think i give up i will have to close the issue, an element like star with a non proportional size is not easy to control. The bigger the size, the beginner the space it occupied. I think you can continue from where i stop. It seems you are more knowledgable than i am.

The previous logo was almost perfect (except gaps between stars). And last logo has some unexpected changes like whole constellation was moved to the left. And I can't ignore that because it can be seen with the naked eye. You as a designer should see it as well, don't you?
If you can share editable file please do it.

I will share the editable file later, the bigger the resolution. The bigger the space between the star. The more i keep editing, more problem keep surfacing that is because i tend to move the whole thing from the initial postioning.

Here is a link to an editable file
https://drive.google.com/folderview?id=1zlLRj6zVWOcQgRVTtGBhpcZjC-LvqQN8


Using workspace's typescript version should look in parent node_modules

When asking VSCode to use workspace's version of Typescript I get this error:

The path [workspace dir]/node_modules/typescript/lib doesn't point to a valid tsserver install.
Falling back to bundled TypeScript version.

This is because I work in a monorepo and I open in VSCode only the package I'm working on. As a result typescript is installed in parent directory and VSCode does not find it.
I think VSCode should use the standard node module resolution and walk up the directory tree to find Typescript installation.

VSCode Version: 1.31.1
OS Version: macOS 10.14.3

Steps to Reproduce:

mkdir testproject && cd testproject && npm init -y && npm i typescript
mkdir nested && code nested
In vscode Run command Typescript: select typescript version
VsCode error:

The path [workspace dir]/node_modules/typescript/lib doesn't point to a valid tsserver install.
Falling back to bundled TypeScript version.

Does this issue occur when all extensions are disabled?: Yes

This is an intentional behavior. We used to resolve typescript this way but it turns out people would have really old TS versions installed in their user directory, and these would unexpectedly get picked up.
Two options:

Configure tsdk in your user settings using an absolute path
Open the repo as a multiroot workspace. Put the root as the first folder in the workspace and the sub folder you want to work on below it. This should let you load the typescript version from the root.

I can confirm that Open the repo as a multiroot workspace. Put the root as the first folder in the workspace and the sub folder you want to work on below it. This should let you load the typescript version from the root. works!
thx @mjbvz


Update FakeIP addresses and Ports

IP addresses and Ports to be updated according to FakeIpList:

ServiceList
CONFIGfile
OAS

IP_1 and TCP_1 to be applied.
Release numbers to be regarded!

was basically already done.
Just found a mistake inside the oas, where there was a TAR 1-0-0 example with 2-0-0 ip/port.


Microsoft.Xaml.Behaviors.WinUI.Managed should support 17763

As discussed, when Project Reunion became the Windows App SDK the min version went down from 18362 to 17763 (the LTS version). This wasn't picked up by XAML Behaviors (among other dependencies).
We'd like to update the Windows Community Toolkit to support 17763 for WinUI 3 as well, but XAML Behaviors is one of our dependencies.
Thanks!
Also, would be good to stay on 1.0.x for an update as there's a blocking issue with 1.1 and Windows Server 2022 we use for our pipeline, see: https://github.com/microsoft/WindowsAppSDK/issues/2829

close this Plz


python311Packages.google-cloud-bigquery: 3.11.0 -> 3.11.1

Changelog: https://github.com/googleapis/python-bigquery/blob/v3.11.1/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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

[ ] Fits CONTRIBUTING.md.

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

4 packages built:

python310Packages.google-cloud-bigquery
python310Packages.google-cloud-bigquery.dist
python311Packages.google-cloud-bigquery
python311Packages.google-cloud-bigquery.dist


Update echoip, switch to buildGoModule

Motivation for this change
Upstream update
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Any update on this?

So sorry for to have let this sit for so long @rvolosatovs.
How about just emulate the output structure from buildGoPackage and have
outputs = [ "bin" "out" ];

and call it a day.
Or the other alternative mentioned that would break configs.

Thanks @rvolosatovs. My apologies also on the long stretch of communication that happened here.


Add codec wrapper modifier

Requires

Supports

I don't have enough knowledge about chain reader and its concepts to review it from the functional perspective. High level looks good to me, how would the config for the ccip case (BigInt) look like?

The config for MsgsBetweenSeqNums would have to add this
&codec.WrapperModifierConfig{
											Fields: map[string]string{
												"FeeToken":       "Int",
												"FeeTokenAmount": "Int",
											},
										},


Amend SE-0364 to make the wording stronger

This removes "considered" and expands a bit on why same-package conformances will be non-retroactive, once duplicate conformances are able to become link errors.

I can amend further if we don't think the duplicate symbol will be detectable at link time, and will instead be detected at load-time.

In the following package structure, where B and C are dylibs and the dependency from B to C is internal/@_implementationOnly, overlapping conformances in A and C can only be detected at load time I think:
graph TD;
    A-->B;
    B-->C;


Improve logging

Fixes #52 by including the most recent commit hash in the runscan.sh script terminal output, as well as logging the browser type and version in the crawler.py file.
Might circle back and reduce the amount of redundant output in both areas, but for now this is the MVP to fix the aforementioned issue.

Ah, this makes sense! Thanks for the clarification @ghostwords

You should rebase the branch on master as I recently refactored how the summary section/crawl parameters output is printed. (It's now in a single place in the Crawler constructor, instead of duplicated in two versions of crawl().)

@ghostwords will do. good call on reducing that down to a single constructor btw

Example logging output with these changes:

here's example output from the most recent commit, for when/if the pb checkout isnt a git directory


Support op-accept_all API call

Expose op-accept_all call via api in order to provide ability for newly discovered machines

@dpoluyanov thank you for your contribution!


Make sort sensitivity opt-in and configurable

In a recent PR (#35), the sorting of strings was adjusted to be case-insensitive. This is not an option for us, so I made the functionality optional with the parameter --sort-sensitivity. If not provided, the old behavior is re-applied.
The unit tests for the postprocessor so they have been updated and expanded

Thanks for the feedback. I pushed some changes to my initial PR with the proposed additions.

Thank you!

This feature was just published in v9.2.0 👍


⚠ Bofrost.de has degraded performance

In 8ddf4b4, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 799 ms

Resolved: Bofrost.de performance has improved in ef83b19 after 6 minutes.