Fix tests that fail between cycles

Context
This fixes tests that would fail during the end of cycle period. The specific dates that have been tested are:

18 September (right after apply deadline)
26 September (right after reject by default)
26 October (once the new cycle has started)

Changes proposed in this pull request

Specify the point in the cycle where the test is relevant (eg, if we are testing submitting an application form, we must be mid-cycle)
Removing the :continuous_application trait from application forms and application choices. Now that we are in cycle 2024, all application are part of the continuous application process. This trait breaks tests when the next cycle (2025) begins because those applications are now in the previous cycle(2024).
Remove specific dates where they are no longer relevant
Add specific dates to a few tests where relevant.
Add methods in the CycleTimetable that can extrapolate the when find and apply will reopen, even if the next cycle is not defined.

Guidance to review
If you think any of the failures are genuine -- as in, is there code that needs to be fixed, rather than the test.
Things to check

[x] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[x] API release notes have been updated if necessary
[x] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[x] Required environment variables have been updated added to the Azure KeyVault
[x] Inform data insights team due to database changes
[NA] Make sure all information from the Trello card is in here
[x] Rebased main
[x] Cleaned commit history
[x] Tested by running locally
[NA] Add PR link to Trello card

 Fix tests that will fail during end of cycle, and when new cycle starts


Add changes for Sherlock feature

Added a backend that uses a test database to evaluate the recall, precision etc. of text searches

@tidickerson as discussed, please abandon this PR and we'll find a different home for your contribution. thanks.


Gitea as Package Registry Reverse Proxy

Feature Description
As a team with plenty of fetches from external servers like dockerhub, composer or pip, especially now with the CI/CD pipelines, I would suggest adding an optional proxy for these package registries.
The huge advantage of this would be, that the build + delivery speed inside a company would improve, but also less international internet traffic would occur, if certain servers are e.x. only in one country given.
A busy CI/CD pipeline could also try to prevent an external registry call and map it to itself to fetch the cached resource. But this should be a check and not checked by default imo 🤔
I could imagine that a server administrator could either whitelist a couple of packages to do this, but also by giving it a limited amount of disk space, and he caches everything himself from registry xyz.
The urls could look like this:
http://my-domain/package-cache/<registryname>/<organization>/<package>

http://my-domain/package-cache/dockerhub/gitea/gitea

http://my-domain/package-cache/composer/phpmailer/phpmailer

http://my-domain/package-cache/ghcr/linuxserver/wikijs

What do you think? Could that be a feature for 1.20 or 1.21? 🤔
Screenshots
No response

Duplicate of #21223


Semantic highlighting does not show for already opened editors

tldr:

VS Code was just opened. Newly opened files are highlighted immediately.
rust-analyzer version: unreleased (d89c189)

I haven't seen any semantic highlighting problems like this one in a long time.

Agreed, I'm not sure if it was a VSCode change or a change in the RA plug-in itself, but I haven't seen it recently either (and used to see this daily).


Don't sync default files in pull-from-upstream

In propose-downstream, we by default sync the specfile and Packit config, this should not be done in pull-from-upstream that doesn't contain these files.

RELEASE NOTES BEGIN
N/A
RELEASE NOTES END

recheck


Algo Bubble Sort in C programming

Need to make a readme file which should have the
Definition
Explanation in simple terms
Code related explanation.

can you please assign me this issue

go ahead. @sowmyasri-Z

you haven't assigned me this .can you please check?

you haven't assigned me this .can you please check?

done

@Abishethvarman can you please assign me this issue as i have already started working on it

@sakshipatil4403 start working on it. No progress from @sowmyasri-Z


Avoid using WinHttpQueryDataAvailable, and use WinHttpReadData directly for better throughput.

Performance optimization for download, as part of https://github.com/Azure/azure-sdk-for-cpp/issues/1201, especially for large payloads.
~Will share performance numbers soon, but local testing showed promise.~

Avg of 6   (ops/s)
5
10 KiB
10 MiB
1 GiB (8)
1 GiB (32)

Before
173.00
179.67
209.00
1.15
3.67

After
168.83
176.17
228.33
1.78
5.51

Ratio
0.98
0.98
1.09
1.55
1.50

I verified on the same VM as the reported, in release, and the numbers are similar.
We get a pretty big perf boost for large payloads, now track2 winhttp is 20%+ faster for large payloads :)
Investigation for the ~10% drop in small payloads is pending, but so far, that might just be difference between libcurl/winhttp, so don't have great ideas for fixing low hanging fruit there.


running terraform plan after applying aws-auth configmaps on eks cluster runs into error

I have issues
when applying aws-auth-config map for the first time , it works fine.
later when I do run terraform apply or terraform plan I get an error saying the couldn't connect to cluster(localhost) to obtain configmaps
workaround : I had remove resource aws-auth-config map from the terraform state to either run plan or apply.
providers information

kubernetes version 1.11
aws version 2.69
``terraformversion12.24`

I'm submitting a...

[x ] bug report
[ ] feature request
[ ] support request - read the FAQ first!
[ ] kudos, thank you, warm fuzzy

What is the current behavior?
when applying aws-auth-config map for the first time , it works fine.
later when I do run terraform apply or terraform plan I get an error saying the couldn't connect to local host to obtain configmaps
workaround : I had remove resource aws-auth-config map from the terraform state to either run plan or apply.
If this is a bug, how to reproduce? Please include a code sample if relevant.

create cluster and have to apply aws-auth-configmaps ..
run terraform plan or terraform apply again

What's the expected behavior?
terraform plan should run as is
Are you able to fix this problem and submit a PR? Link here if you have already.
Environment details

Affected module version: tweaked module and created one for own .
OS:
Terraform version: 12.24

Any other relevant info

What is your terraform configuration? Have you defined the provider "kubernetes" block as in the Usage Example?

Yes, I have defined the provider kubernetes
Here is the provider block using it as a example
provider "kubernetes" {
  host                   = module.eks-cluster.endpoint
  cluster_ca_certificate = base64decode(module.eks-cluster.kubeconfig-certificate-authority-data)
  token                  = data.aws_eks_cluster_auth.eks-cluster.token
  load_config_file       = false
  version                = "~> 1.11"
}

In my module,
I have defined
resource "kubernetes_config_map" "cluster-auth-config" {
  count      = var.manage_aws_auth ? 1 : 0
  depends_on = [null_resource.wait_for_cluster[0]]
  metadata {
    name      = "aws-auth"
    namespace = "kube-system"
  }
  data = {
    mapRoles = yamlencode(
      distinct(concat(
        local.configmap-roles,
        var.configMap-roles,
      ))
    )
    mapUsers = yamlencode(var.configMap-users)
  }
}

here is the terraform configuration
locals {
  cluster-main = {
    name                = "my-test-cluster"
    target-environments = ["sandbox"]
  }
}

data "aws_eks_cluster_auth" "eks-cluster" {
  name            = local.cluster-main["name"]
  depends_on = [module.eks-cluster]
}

provider "kubernetes" {
  host                   = module.eks-cluster.endpoint
  cluster_ca_certificate = base64decode(module.eks-cluster.kubeconfig-certificate-authority-data)
  token                  = data.aws_eks_cluster_auth.eks-cluster.token
  load_config_file       = false
  version                = "~> 1.11"
}

module "eks-cluster-sg" {
  source              = "../../modules/terraform-aws-earnin-security-group"
  description         = "eks cluster security group"
  owner               = "Myteam"
  environment         = var.environment
  target-environments = ["sandbox"]
  name                = "${local.cluster-main["name"]}-sg"
  sg-rules = [
    {
      type              = "ingress"
      description       = "ingress to allow port 0 - 65535 in TCP"
      from-port         = 0
      to-port           = 65535
      protocol          = "tcp"
      cidr-blocks       = []
      self              = true
      security-group-id = "none"
  }]
  vpc-id = "vpc-xxxxxxxxxx"
}

data "template_file" "ec2-key-file" {
  template = file("./ec2-key-pairs/my-public-key.pub")
}

resource "aws_key_pair" "eks-cluster-key" {
  key_name   = "${var.environment}-${var.region}-${local.cluster-main["name"]}-eks-cluster-key"
  public_key = data.template_file.ec2-key-file.rendered
}

module "eks-cluster" {
  source                      = "../../modules/terraform-aws-earnin-eks-cluster/"
  owner                       = "myteam"
  environment                 = var.environment
  target-environments         = ["sandbox"]
  name                        = local.cluster-main["name"]
  kubernetes-version          = var.eks-cluster-version
  vpc-endpoint-private-access = var.cluster-vpc-endpoint-private-access
  vpc-endpoint-public-access  = var.cluster-vpc-endpoint-public-access
  vpc-public-access-cidrs     = var.cluster-vpc-public-access-cidrs
  vpc-subnet-ids              = ["subnet-xxxxxxxxx123456", "subnet-xxxxxxxxx1234567", "subnet-xxxxxxxxx12345677"]
  vpc-sg-ids                  = [module.eks-cluster-sg.id]
  enabled-cluster-log-types   = var.cluster-enabled-log-types

  manage_aws_auth = true

  aws-configMap-roles = [
       module.eks-cluster-node-role.arn,
       module.eks-cluster-node-group-1.role-arn,
       module.eks-cluster-admin-role.arn,
       module.eks-cluster-viewer-role.arn,
       module.eks-cluster-app-admin-role.arn,
       module.eks-cluster-app-deployer-role.arn,
  ]

  configMap-users = [
    {
      userarn  = "arn:aws:iam::123456789:user/user2F.user2L"
      username = "user2F.user2L"
      groups   = ["system:masters"]
    },
    {
      userarn  = "arn:aws:iam::123456789:user/user1F.user1L"
      username = "user1F.user1L"
      groups   = ["system:masters"]
    },
  ]
}

resource "null_resource" "wait_for_cluster" {
  count = var.manage_aws_auth ? 1 : 0

  provisioner "local-exec" {
    command     = var.wait_for_cluster_cmd
    interpreter = var.wait_for_cluster_interpreter
    environment = {
      ENDPOINT = module.eks-cluster.endpoint
    }
  }
  depends_on = [
    module.eks-cluster,
    module.eks-cluster-sg,
  ]
}

resource "local_file" "kubeconfig" {
  content  = module.eks-cluster.kubeconfig
  filename = "${local.cluster-main["name"]}.yaml"
}

module "eks-cluster-node-group-1" {
  source                           = "../../modules/terraform-aws-earnin-eks-node-group/"
  cluster-name                     = module.eks-cluster.name
  environment                      = var.environment
  target-environments              = ["sandbox"]
  node-group-name                  = "${module.eks-cluster.name}-nodegroup-1"
  owner                            = "cloudops"
  instance-types                   = ["c5.xlarge"]
  scaling-desired-size             = 3
  scaling-max-size                 = 5
  scaling-min-size                 = 3
  subnet-ids                       = ["subnet-xxxxxxxxx123456", "subnet-xxxxxxxxx1234567", "subnet-xxxxxxxxx12345677"]
  remote-access-ssh-key-name       = aws_key_pair.eks-cluster-key.key_name
  remote-access-security-group-ids = [module.eks-cluster-sg.id]

  ng-depends-on = [module.eks-cluster]

}

Please can you refer to the basic example and this doc to see how to manage the aws-auth configmap and if needed how to manage kubernetes resources during the same apply.

@barryib  My cluster does get created and it applies aws-auth config too ( only when it is for first time ) but on the next terraform run ( either apply or plan ) ...  it errors out...

Please share your error output and debug.

I have the same issue, the error that appears is:
Error: Unauthorized

Could the problem be that the token is only valid for a limited amount of time, but is saved in the state and as a result when reused every time it is not valid anymore?

Could the problem be that the token is only valid for a limited amount of time, but is saved in the state, and as a result when reused every time it is not valid anymore?

@fliphess I found a workaround. Can you separate the process of AWS_AUTH creation from EKS cluster creation? though there is a little caveat, you would have manually delete AWS_AUTH configmap first and then let terraform create it.

I've managed to fix it for myself: my issue was not related.
We used read only credentials when running the terraform apply, but this was unsufficient as the kubernetes provider needed some more permissions to acquire a token.


Attempting to get Unit Tests running on Jenkins

This is my attempt to get unit tests running for Dragon under GtiHub Actions, just like they are for ARMI.

This will close #16


DynamoDB spike

This PR is only here for the purposes getting the build working on travis.

Yeah it's become quite a feature-full spike.
There's still some gap for this to be considered production-ready:

setting created_at and updated_at timestamps
adding tests for RBS fallback
consider allowing data storage to be set on a per-table basis (or at least allow us to still deploy with the submission table in Dynamo and the other three still in RDS)
Investigate the extra 40-60ms latency that seems to be exhibited on each request when dynamodb is enabled

Closing for now as spike is on hold.


Add .dockerignore file

Adds .dockerignore file so the build context reduces massively.
Please review @justb4 ... TIA!

Yes, good idea, especially ignoring node_modules, speeds up build. Good to go!

Thanks for the review @justb4.


Retain CI logs on test failure

The GitHub CI now retains the event as an artifact log whenever a test fails.
Useful for debugging the run that failed with mircat.

This is a quite trivial PR, so I'd say the reviews I requested are optional. I'll merge it on Monday afternoon in any case, unless anybody objects.


Update runc/libcontainer dependency to v0.0.7

For a list of changes, see https://github.com/opencontainers/runc/releases/tag/v0.0.7
Signed-off-by: Mrunal Patel<EMAIL_ADDRESS>

Thanks, LGTM

I opened https://github.com/docker/docker/pull/19751 to not bump the dep to master for 1.10

@crosbymichael Yep, I noticed the other PR. When can we do a full update?

@mrunalp i would say after the release would be a good time to bump it

@crosbymichael Sounds good.

dammit needs a rebase wahhh

@jfrazelle @crosbymichael @icecrime I've carried this in #20187.

Moving forward with #20187.


will open finetune code ?

I am guessing you are asking "will the finetune code be open-sourced?".
For this question, you can finetune this model with huggingface Trainer, just like other hf models.


General improvements to libpeer

So this is a batch of changes that should probably really be 3 pull requests; I made them a single one both out of convenience (esp. if it's approved) and since it's a RFC anyway.

Third-party shell script build is migrated to CMake (so that it no longer depends on make being the default CMake tool, among others); this is probably the most controversial change, so consider this pull request an RFC.
Make libpeer build successfully in Windows. Did not test in POSIXy systems (mostly due to laziness), though I expect any problems are going to be minor.

As an aside: There was a bug in the sample, fixed it.

Get the library working in Firefox, also enables trickle ICE.

There's a bug in CMakeLists.txt that I spotted. Should be easily fixed; I'll make a new PR soon.

Took a bit longer than expected, and I noticed another small issue. Fixed all of that now.

Hello, because this PR failed to compile under linux, it cannot be merged. Sorry for that


feat: add rxjs operators poll and whenDocumentVisible

Add two RxJs operators that works well together but still can be used independently.
poll can be used to poll an API at a fixed interval with an optional initial delay.
whenDocumentVisible is the RxJs counterpart of the injectDocumentVisibility and can be used to pause and resume a RxJs stream based on the document visibility state.
Related to #405

PR not found

What do you mean?

@all-contributors please add @FabienDehopre for code

https://github.com/all-contributors please add @FabienDehopre for code

@FabienDehopre looks like the test failed. Can you take a look?

@all-contributors please add @FabienDehopre for code
(third time the charm?)

@all-contributors please add @FabienDehopre for code
(fourth time 🥲)

I've fixed the failing tests.


Step one of global user management

Step one of making Chorus more enterprise friendly is to make SSO easier.  We're doing this in Chorus by supporting oAuth, with  Keycloak as the "demo" system.   This PR integrates Quepid with Keycloak, so you don't need to create an account in Quepid, instead you register with Keycloak.
We'll expand this in the future to SMUI and Solr.   See the related PR #47 for some earlier work that didn't quite mature!

I guess this should be marked as WIP instead of PR so that it doesn't get merged ;-)

I flipped on the Draft tag, thanks @renekrie ....     This is is ready to go as soon as Solr 8.11 gets released...   Or, if Querqy worked with Lucene 9....   hint hint ;-)    I am pretty stoked to have the user authentication story being simplified...

Oh, and i have no idea why the smoke test won't run...    something pending on something...   May need to rethink how we do smoketesting...

lots of progress, down to I can't get JSONP with Basic Auth to work...    I think we need to make it publice (and or swap to cors in Solr!!)

Okay, now with the latest Solr, keycloak, and Quepid, this all seems to be working!!!!


refactor: reduce duplication of flagd provider service

This PR

Use generics to reduce duplication of flagd provider service.

Related Issues

Notes

Tested e2e using flagd's integration tests.
Follow-up Tasks

How to test

@thisthat this is an equivalent to https://github.com/open-feature/java-sdk-contrib/pull/234 if you're interested.

It would be nice to reduce the duplication in these funcs too but it doesn't seem trivial without using generic structures (breaking the provider contract). If anyone has ideas, feedback would be welcomed :)

lgtm, very understandable IMHO. I hadn't seen type constraints done before. Thanks!


Menu popup items become inacessible in certain scrolling positions

Steps to Reproduce

Go to any feed, click on the Ellipsis button on any post
Depending on the height of the button in relation to the screen, it fails to show on its entirety
I have to click out, adjust scroll and try again to be able to access the item I want.

Attachments
Menu cut on top

Menu cut on bottom

Video of me trying to delete a post
https://github.com/user-attachments/assets/989dab6e-f5bd-42d9-9bea-6442f8126298
What platform(s) does this occur on?
Web (Desktop)
Device Info
Windows 11 home; Mozilla Firefox 130.0.2 (64-bits)
What version of the app are you using?
Build version: 1.92.0; Bundle info: 3b44ea2 (prod); Bundle date: 24101017; Platform: web
Additional Information
Screen size: 1366x768

I think this might be or is part of issue #5681.


Failures on [cfe] Implement new async return rules

There are new test failures on [cfe] Implement new async return rules.
The tests
co19/LanguageFeatures/nnbd/future_flattening_A01_t12 RuntimeError (expected Pass)
co19/LanguageFeatures/nnbd/future_flattening_A01_t40 RuntimeError (expected Pass)

are failing on configurations
dartdevk-weak-linux-release-chrome

This seems to be a co19 failure: https://github.com/dart-lang/co19/issues/785


Change method name getNetworkRightDownCorner

I would change the method name
getNetworkRightDownCorner
to
getNetworkBottomRightCorner
which is more consistent with other graphics tools.

updated. visualizeSBML line 910. details see (https://github.com/SunnyXu/SBMLDiagrams/commit/281f86d40d78e4d22df94d5e6a8acefa843e9906)


[Addition]: Self witnessing - Farming - Location spoofing cluster in Malaysia [5 HS]

Hotspot b58 Addresses
112iCp1THmsTYekQLqXhSSEksEXsF63gfzUdCLc6kHoLMxPD4RQU
11Ku4DwjFn1AooePvvVsAHXwxGbYtQ4ziP83EvDk6nxwNgXz5Jd
11rtjkrPdSiK2Mg5kuq4FAerVsRhRk6XJ7HYAaC6FvnhUhXWVJd
11XRBfcRWWTeUZWuebwkPnNbDZjtNWckhBJ7RbLevcA9RVqWm5k
11d8A8EY4yfJVXJaqCZsbTmEr8ZsRP4GnRzky2S3qFBt3aF4Uzx
Discord Handle
may1s#8232
Reason(s)
Very suspicious location changes, possible location spoofing+farming.
I think the gamer has too many hs and cheats with all of them. I will leave a reference to this topic after detailed research.

It's a scam helium device. This is a complete scam.


Sample code

Hi, can we see the sample code for the adaptive cards in this page? The samples and templates in adaptivecards.io/samples/ do not contain the group membership approval.

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

ID: fc039d3b-b1e6-79ae-3b83-6e90d7d8af94
Version Independent ID: f72d624f-36b9-bab1-9c14-e9fc1e81082b
Content: What are actionable messages in Office 365? - Outlook Developer
Content Source: docs/actionable-messages/index.md
Product: outlook
Technology: o365-connectors
GitHub Login: @jasonjoh
Microsoft Alias: jasonjoh

Hi @sthi-1005 Apologies for the late reply. Apparently no one is monitoring this repo! I stumbled across your issue and and wanted to tell you about the Adaptive Cards developer community on https://pnp.github.io/. You'll probably have better luck there. I'll keep this open as a doc request.

The designer at https://amdesigner.azurewebsites.net/ has the Group Membership template. It's listed as the "Multiple Request Approval" sample.


Duplicate access logs are recorded when pulling image with VIC

As described in https://github.com/vmware/vic/issues/5950, VIC pulls image manifest twice for every image, this causes two records in the access log.

We don't have good solution for this issue with reasonable effort, until VIC behaves same way as vanilla docker.

Added as a Known Issue in https://github.com/vmware/harbor/releases/tag/v1.2.0#knownissues

The work for VIC Engine is tracked by https://github.com/vmware/vic/issues/5963


Links not rendering within a table

I couldn't get the hyperlinks to work within the right-hand column of the table. The HTML is simply displayed.
This is a screenshot from Wikifreedia, but it's the same on Wikistr.
Nostr-Event-Register

Nevermind, you can close this one. Anytime there is asciidoc markup in a column, you can just put "a" in the column description (at the top of the table) and it'll render correctly.
Took me hours to figure it out.

There is at least one other bug with these tables, they're being rendered without a <table> HTML element so they look all broken.

Maybe that's why the HTML within the tables is messed up.

Should be working on wikistr.com now.

Seems to be fine, now. Thanks!


Allow cell template to be retrieved from $templateCache

Hi,
Just a feature request.
It would be great if the TemplateService tried the $templateCache when requesting the template. This is something ng-grid handled and would be handy as we use grunt-angular-templates for caching views.

is there anything except a performance benefit? i'm slow to use any Angular feature that is not of great benefit, as I plan on also supporting Angular 2 and other frameworks - currently the grid can exists outside any framework - so limiting exposure to Angular is good for the grid.

Its purely for performance. Its a fair standpoint you make regarding angular features. Our grunt strips .html file extension when naming templates in the cache. I guess we could alter the grunt process, or alternatively allowing the override of the Template Service implementation might be nice.  We currently use ng-grid and its not performing too well for us so are trialling swapping it out.

what about not using templates? i only put that in to ease peoples journey away from ng-grid. however, in my applications using Angular Grid, I simply don't use them. your grid will work much faster if you don't use templates. I'm going to close this request if you don't mind, I don't think I'll be making a code change :)

With Angular 1.4 and ag-grid 1.14.1 I have to monkey patch TemplateService:
someModule.run([
    '$window',
    '$templateRequest',
    '$log',
    function($window, $templateRequest, $log) {
      $window.awk.grid.TemplateService = TemplateService;

      function TemplateService() {
        this.templateCache = {};
        this.requestPromises = {};
      }

      TemplateService.prototype.init = function($scope) {
        this.$scope = $scope;
      };

      // returns the template if it is loaded, or null if it is not loaded
      // but will call the callback when it is loaded
      TemplateService.prototype.getTemplate = function(url, callback) {
        var templateFromCache = this.templateCache[url];

        if (templateFromCache) {
          return templateFromCache;
        }

        var promise = this.requestPromises[url];
        var self = this;

        if (!promise) {
          promise = this.requestPromises[url] = $templateRequest(url, false).then(function(tmpl) {
            self.templateCache[url] = tmpl;
          }).catch(function(err) {
            $log.warn('Unable to get template error ' + err.status + ' - ' + url);
          }).finally(function() {
            self.requestPromises[url] = undefined;
          });
        }

        promise.then(function() {
          callback();
        });

        // caller needs to wait for template to load, so return null
        return null;
      };

    }
  ]);

It will be nice to have an option to properly overwrite it.


database dump optimize

Please describe the feature you would like to see implemented.
It might be good, to add the following to the beginning to improve performance:
    SET autocommit=0;
    SET unique_checks=0;
    SET foreign_key_checks=0;

See: https://dev.mysql.com/doc/refman/8.0/en/optimizing-innodb-bulk-data-loading.html

I don't know about autocommit. Maybe this blows up the mysql server on bigger data sets, as it never writes it to disk and then everything at once. maybe memory fills up completely or something like that 🤔

I would keep it is as it is right now 🤔


npx create-react-app tutorial | NOT WORKING | First command

I started a tutorial but It stopped at first command ... what should I do? The bar is not moving forward.

I will pull the code and try it first.
------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2021年2月17日(星期三) 晚上9:34
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS>主题: [facebook/create-react-app] npx create-react-app tutorial | NOT WORKING | First command (#10568)
I started a tutorial but It stopped at first command ... what should I do? The bar is not moving forward.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

All codes was formatted in master branch!
------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2021年2月17日(星期三) 晚上9:34
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS>主题: [facebook/create-react-app] npx create-react-app tutorial | NOT WORKING | First command (#10568)
I started a tutorial but It stopped at first command ... what should I do? The bar is not moving forward.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

It should work! Try it again.

It should work! Try it again.

it halts again and again... 😅... I've even reinstalled NODE.

Try running the following commands in order.

npm cache clean --force

npm install create-react-app -g

npx create-react-app your-app-name

Try running the following commands in order.

npm cache clean --force

npm install create-react-app -g

npx create-react-app your-app-name

Thank you @RohankumarM , It works!


Data Science Updates per PR 628

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. Your PR title can be also used here.
Types of changes
What types of changes does your content introduce to this project?
Put an x in the boxes that apply (without "`")

[ ] Proofreading (fix spelling, grammar, punctuation, accuracy) Proofreading
[x] New content (new section, new readme, etc.)
[ ] Modifications (modifying existing content)
[ ] Other (please describe)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. Don't hesitate to ask if you're unsure about any of them. We're here to help! This simply reminds us of what we will look for before merging your code.
Note: If you submit code make sure to test it

[ ] Tested any code that was added/modified in this PR
[x] Added external links
[x] Added internal links
[ ] Added images
[ ] Added videos
[ ] Added code snippets

Further comments
Took original content by niv-png and requested updates from https://github.com/shhossain/computer_science/pull/628
Adding your name to the contributor's list
Comment "@all-contributors please add @your_username for content" to add your name to the contributor's list.
You can change the "content" with any of this
Reviewers:

[ ] @username

Please ensure that you are not adding any sensitive information like API keys, secrets, passwords, tokens, usernames, or other credentials. Refrain from adding your name, email, or other personal information.

@all-contributors add @WalterMarch for bug

@all-contributors add @niv-png for doc


Use if let operation instead of explicit nil check

Use if let operation instead of an explicit nil check since the scope of the variable s should be limited.

Please verify that the change compiles and passes tests: utils/build-script -RT.

You are right about the invalid conditional binding. The output is as follows, please close this PR.

swift/stdlib/public/core/Collection.swift:587:8: error: initializer for conditional binding must have Optional type, not 'UnsafeMutablePointer<Self.Element>'
if let s = self._baseAddressIfContiguous {
^
[513/645] Generating UnicodeExtendedGraphemeClusters.cpp ...UnicodeExtendedGraphemeClusters.cpp.gyb with ptr size = 8
ninja: build stopped: subcommand failed.
./utils/build-script: command terminated with a non-zero exit status 1, aborting

@hvy Thanks for checking!


Can't login onto WebUi

hi there,
I' might be missing something,
Just installed a fresh install of PCFDev, and deployed the CF Webui onto it,
But I can't login, and i'm having a hard time to understant which user I should use to login.
is there a default user?
Should I be using a CF user, that I have to create specially for cf-webui?
Could you please try to help me in the right direction?
Regards,
Michael

Hello,
cf webui is just a front end, it relays api call to your PCF cloudfoundry API. just have to set up the api url, and use your end user accounts (same as cf login etc ...)

Thanks Poblin,
In fact when trying to use my CF user account,
I'm facing this issue,
My app respond with a 502 bad gateway response when posting to  ***.pcfdev.io/oauth
And my server logs indicates :
2016-07-05T09:37:33.72+0000 [APP/0]      ERR 2016/07/05 09:37:33 [error] 33#0: *64 login.local.pcfdev.io could not be resolved (110: Operation timed out), client: <IP_ADDRESS>, server: , request: "POST /oauth HTTP/1.1", host: "console-cf-webui-snappier-nipa.local.pcfdev.io", referrer: "https://console-cf-webui-snappier-nipa.local.pcfdev.io/"
it seems i'm missing something else but can't figure out what,
Thx for the help,
Regards

If your PCF deployment is in private network, maybe u shoud set the NGINX_RESOLVER variable to the ip holding your deployment dns wildcard (as indicated in README, <IP_ADDRESS> is google dns)

Thanx again poblin,
That's exactly what I was missing,
In fact, my PCF deployment is running on a VM on my main machine, needed to update the NGINX_RESOLVER that wasn't reachable otherwise, just struggled to find the right address.


It is crash on Mac M1, do you have the same issue? Any idea? Thanks

Hi, sorry for the late reply! Did you make any progress with this? What does the crash say?

Cannot form weak reference to instance (0x1018c0000) of class QLRemoteAccessoryViewController. It is possible that this object was over-released, or is in the process of deallocation.

Hello @chengkunblog are you using it in Catalyst app or in the "iPad apps on Mac"?


fix group button when this is tab

close #1622
url test: hostlocal + documentacao/exemplos/painel2/resume

url test: hostlocal + documentacao/componentes/botoes/

I don't like this solution too much because it's private the user to use others colors at the same context of the ls-btn. But I think that's the solution.

:+1:


Zahara/og3 63 cleanup yaml

Fixes for KafkaJS compatibility, M1 issues with jre-alpine image + merge fixes


Update kms-utils to 0.1.169 in scala-steward-dependencies

About this PR
📦 Updates uk.gov.nationalarchives:kms-utils from 0.1.168 to 0.1.169
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "uk.gov.nationalarchives", artifactId = "kms-utils" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "uk.gov.nationalarchives", artifactId = "kms-utils" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #461.


Cannot change password within the app on Android

The Issue is pretty simple, on the desktop app in the Preferences section, there is an option to change the password, but that option is not available on the Android app in the Preferences section.

Hi Andrew
Thank you for reporting this.
At present, this is due to a known issue with the wallet. Due to the possibility of data corruption, we have disabled the "Change Pin" button for the time being, with a view to ensuring that people's wallets aren't affected negatively. We plan to address this bug and reintroduce the button after making 100% certain that the potential for data corruption has been fully addressed.
Please refer to #13


Limit the set of people who can access cloud.gov IaaS resources

In order to minimize the risk of cloud.gov infrastructure being tampered with at the AWS level, only members of the cloud.gov team should be able to access cloud.gov AWS resources.
Acceptance Criteria

[ ] Only people in the cloud.gov ops team are able to access cloud.gov resources in AWS

(This refers to the fact that other people at 18F outside the team retain some access as we've never detangled cloud.gov from non-cloud.gov. That access needs to be restricted, whether via negative IAM, or totally separate AWS accounts.)

Since cloud.gov has it's own account in GovCloud, do we need to do anything else to close this out?

Nope, I think that demoing #25 (redeploy in GovCloud) basically knocks this one straight to "completed", because we're doing this:

That access needs to be restricted, whether via negative IAM, or totally separate AWS accounts.

BUT, please don't close this issue until that's done... Our Readiness Assessment Report refers specifically to this one as where to track the state of the remediation.

I set the estimate on this one to "none" since it's zero effort based on #25 completing and should not be factoring into our burndown.

I updated the ACs on this one to reflect our intention to not only resolve the existing situation, but have a test to prevent it from recurring in future. I also raised the estimate from a 0 to a 1 to reflect that it's no longer a no-op resolved via the redeployment to GovCloud.

Test exists, but no alerts yet. Steve wants to add some additional conditions to test for.

(Feel free to finish this @sharms, but I'm disconnecting it from the epic about FedRAMP remediations since we're in a clean state right now.)


 If you're familiar with git, you could enter the flutter repo and run git reset --hard v1.3.12

@jonahwilliams Thanks, I executed the command in flutter (master):
$ git reset --hard v1.3.12
HEAD is now at a1bee54fd Roll engine 5b09204e3037..e75f6de01fee (2 commits) (#29537)

However, this leaves me with a different issue:
Downloading Dart SDK from Flutter engine e75f6de01fee738eb15f24efb0ec549587009f1a...
Unknown operating system. Cannot install Dart SDK.

Nervermind, only happened in Git Bash.

This now happened to me when upgrading from 1.3.13 to 1.3.14.  When retrying I see:
Failed to stash local changes: ProcessException: Process "git" exited abnormally:
fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
  Command: git stash push -m flutter-upgrade-from-v1.3.14

Then I did git reset --hard v1.3.13 (hint given above), but this didn't help, as doing a new upgrade again results in:
===== CRASH =====
si_signo=Segmentation fault(11), si_code=1, si_addr=0x67
version=2.2.1-dev.1.1.flutter-None (Thu Mar 14 15:39:18 2019 +0000) on "linux_x64"
thread=12579, isolate=flutter_tools.snapshot:main()(0x18c0aa0)
  [0x00007fdf0ccc4fa4] Unknown symbol
  [0x00007fdf0ccc4fa4] Unknown symbol
-- End of DumpStackTrace
/home/jos/flutter1/flutter/bin/flutter: line 177: 12128 Aborted                 (core dumped) "$DART" $FLUTTER_TOOL_ARGS "$SNAPSHOT_PATH" "$@"

Can this issue be reopened or can someone explain what's going on here?

After getting this issue i re-ran flutter upgrade and it worked for me

UPDATE: I now installed the most recent Studio/SDK on another, freshly installed system (Linux Fedora 29 with all updates), everything worked with Flutter 1.4.7 (I could generate working apps), but when I upgrade to 1.4.8 dart crashes in exactly the same way:
Building flutter tool...

===== CRASH =====
si_signo=Segmentation fault(11), si_code=128, si_addr=(nil)
version=2.2.1-dev.2.0.flutter-None (Thu Mar 28 00:48:30 2019 +0000) on "linux_x64"
thread=27922, isolate=flutter_tools.snapshot:main()(0x1e48da0)
  [0x00007f99110e5d60] Unknown symbol
  [0x00007f99110e5d60] Unknown symbol
-- End of DumpStackTrace
/home/jos/DEVEL/flutter/bin/flutter: line 177: 27543 Aborted                 (core dumped) "$DART" $FLUTTER_TOOL_ARGS "$SNAPSHOT_PATH" "$@"

When doing flutter upgrade again it starts saying Already up to date. and then continues with Upgrading engine....
Any suggestions?

@josvos Please open another issue for that. This is not related to the problem in the original question.

@creativecreatorormaybenot I will open a new issue when I have collected some more info. Upgrading from 1.4.8 to 1.4.9 on system #2 did work without a crash, upgrading from 1.4.7 to 1.4.9 on system #1 still crashes...

$ flutter upgrade
git status failed: ProcessException: Process "git" exited abnormally:
error: bad signature
fatal: index file corrupt
Command: git status -s
i got this error while upgarding the flutter  any help

@shashi2602
I ran into the same issue after updating Linux, I managed to fix it with...
get init
flutter upgrade

ProcessException: Process exited abnormally:
fatal: unable to access 'https://github.com/flutter/flutter.git/': Could not resolve host: github.com
what to do..? please help me


Update min width of role and name column.

Set minimum width for role column in users list and name column, deactivated users list and bots list such that role value always fits in one line and there's more space for the email.
Fixes: #23218
Screenshots and screen captures
Russian before max-width the name column

After

English

Tooling tips: https://zulip.readthedocs.io/en/latest/tutorials/screenshot-and-gif-software.html
Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Thanks for working on this, @thaiscodafond !
Please fix the failing tests and post a comment requesting a review.

Hi @alya !
Could you review my pull request ?
Thanks in advance !

Thanks!
I think this is an improvement, but things can still get pretty weird, e.g.:

Maybe we should hide some columns ("Last active" is a good candidate on the Users tab) in narrow screens? If we decide to do this, I'd be fine with making it a follow-up.
@sahil839 It would be great to your review and your thoughts.

While testing this, I also noticed that it's possible for the sorting triangle on "Name" in these panels to get partially hidden. I was able to replicate this without this PR as well, so we can fix it separately if it's not easy to do so.

I could work on hiding the last active column if that's what you want, but for the triangle, I would have to look more into it.

Adding max-width property to name element leads to name column being short and the user Id column being long with much of the space wasted since user ID values in most of the cases would be about 4-5 digits. I am not sure why max-width property is added, but I guess it is to keep the User word in heading in one line in Russian.

To be honest, these kind of issues are hard to solve with these many columns (and with long values possible for names and emails) and to handle multiple languages and multiple screen sizes. Probably we can try to remove one or two columns permanently and possibly allow opening the user profile popover by clicking on name to provide the admin easy access to more information of the user that admin is trying to edit/deactivate.

I just kicked off a related thread here about changing User ID -> ID to reduce the challenge a bit.

Actually, I guess that would help quite a lot in Russian!

Yes I wanted to put max width on the ID column but couldn't because of the Russian.
Do you want me to change it and send it here ?

I think we can go with changing the column name to just "ID", with a small chance that we'd have to undo the change if concerns come in on the discussion thread.

I changed User ID to ID but it still gets messed up when you compress it too much

@sahil839 Did you have technical concerns about hiding "Last active" in narrow windows?

I cannot remember any, we can hide it.

HI ! Here's the result of my changes, tell me if there's anything more to add or to change.
https://user-images.githubusercontent.com/101876673/205937939-9cf5ede1-fbf9-47b1-b127-c4fc868ffba4.mov


Thinkpad E550 - service fails to load despite kernel parameters adjusted

Hello everyone,
I'm very interested in activating this script on my system as I experience CPU throttling from time to time. I have a ThinkPad E550. Tried on  Kernels (latest Linux-lts / Linux 5.10.11.arch1-1).
lsm=capability,yama msr.allow_writes=on
Service fails to start, getting the following error:
` lenovo_fix.service - Stop Intel throttling
Loaded: loaded (/etc/systemd/system/lenovo_fix.service; enabled; vendor preset: disabled)
Active: failed (Result: exit-code) since Fri 2021-01-29 16:03:31 CET; 2min 16s ago
Process: 436 ExecStart=/opt/lenovo_fix/venv/bin/python3 /opt/lenovo_fix/lenovo_fix.py (code=exited, status=1/FAILURE)
Main PID: 436 (code=exited, status=1/FAILURE)
Jan 29 16:03:31 haibar python3[436]:     set_hwp(config.getboolean('AC', 'HWP_Mode', fallback=False))
Jan 29 16:03:31 haibar python3[436]:   File "/opt/lenovo_fix/lenovo_fix.py", line 554, in set_hwp
Jan 29 16:03:31 haibar python3[436]:     cur_val = readmsr('IA32_HWP_REQUEST', cpu=0)
Jan 29 16:03:31 haibar python3[436]:   File "/opt/lenovo_fix/lenovo_fix.py", line 193, in readmsr
Jan 29 16:03:31 haibar python3[436]:     raise e
Jan 29 16:03:31 haibar python3[436]:   File "/opt/lenovo_fix/lenovo_fix.py", line 181, in readmsr
Jan 29 16:03:31 haibar python3[436]:     val = struct.unpack('Q', os.read(f, 8))[0]
Jan 29 16:03:31 haibar python3[436]: OSError: [Errno 5] Input/output error
Jan 29 16:03:31 haibar systemd[1]: lenovo_fix.service: Main process exited, code=exited, status=1/FAILURE
Jan 29 16:03:31 haibar systemd[1]: lenovo_fix.service: Failed with result 'exit-code'.
`
Is it this Model not supported or the recent kernels managed to completely block how this script interferes with it's behavior?
Thanks

It's failing while reading the MSR. Can you manually test with rdmsr?

It's failing while reading the MSR. Can you manually test with rdmsr?

I will if I get the CPU to throttle again. After Kernel update (5.10-11) it hadn't throttled even after stressing the system to the maximum.
On a side note, how do you use the rdmsr exactly for this purpose?

I will if I get the CPU to throttle again. After Kernel update (5.10-11) it hadn't throttled even after stressing the system to the maximum.
On a side note, how do you use the rdmsr exactly for this purpose?

Jan 29 16:03:31 haibar python3[436]: cur_val = readmsr('IA32_HWP_REQUEST', cpu=0)

In this particular case throttled is failing in the IA32_HWP_REQUEST MSR, so:
rdmsr -a 0x774

@erpalma thank you.
Just for now, without thermal throttling active, when I'm trying
rdmsr -a 0x774
This is an output:
rdmsr: CPU 0 cannot read MSR 0x00000774
And this is an output from status check:
lenovo_fix$ systemctl status lenovo_fix.service
lenovo_fix.service - Stop Intel throttling
Loaded: loaded (/etc/systemd/system/lenovo_fix.service; enabled; vendor preset: enabled)
Active: failed (Result: exit-code) since Mon 2021-03-01 17:02:09 CET; 24h ago
Process: 1148 ExecStart=/opt/lenovo_fix/venv/bin/python3 /opt/lenovo_fix/lenovo_fix.py (code=exited, status=1/FAILURE)
Main PID: 1148 (code=exited, status=1/FAILURE)

Mar 01 17:02:09 T480-SA python3[1148]:     set_hwp(config.getboolean('AC', 'HWP_Mode', fallback=False))
Mar 01 17:02:09 T480-SA python3[1148]:   File "/opt/lenovo_fix/lenovo_fix.py", line 554, in set_hwp
Mar 01 17:02:09 T480-SA python3[1148]:     cur_val = readmsr('IA32_HWP_REQUEST', cpu=0)
Mar 01 17:02:09 T480-SA python3[1148]:   File "/opt/lenovo_fix/lenovo_fix.py", line 193, in readmsr
Mar 01 17:02:09 T480-SA python3[1148]:     raise e
Mar 01 17:02:09 T480-SA python3[1148]:   File "/opt/lenovo_fix/lenovo_fix.py", line 181, in readmsr
Mar 01 17:02:09 T480-SA python3[1148]:     val = struct.unpack('Q', os.read(f, 8))[0]
Mar 01 17:02:09 T480-SA python3[1148]: OSError: [Errno 5] Input/output error
Mar 01 17:02:09 T480-SA systemd[1]: lenovo_fix.service: Main process exited, code=exited, status=1/FAILURE
Mar 01 17:02:09 T480-SA systemd[1]: lenovo_fix.service: Failed with result 'exit-code'.

Do you have any idea why? I can try to assistance you with troubleshooting. If there is any other test I can perform to provide you with more details. Please let me know.

And that's it, I run in to thermal problem again and the output of rdmsr command is exactly the same.

You can't read that (and possibly other) MSR. It can be due to a bios lock, kernel problem or whatever. Try with a older kernel, if this is not solving your issue I don't think there is much you can do.

Can you test if any of these MSRs can be read?
MSR_DICT = {
    'MSR_PLATFORM_INFO': 0xCE,
    'MSR_OC_MAILBOX': 0x150,
    'IA32_PERF_STATUS': 0x198,
    'IA32_THERM_STATUS': 0x19C,
    'MSR_TEMPERATURE_TARGET': 0x1A2,
    'MSR_POWER_CTL': 0x1FC,
    'MSR_RAPL_POWER_UNIT': 0x606,
    'MSR_PKG_POWER_LIMIT': 0x610,
    'MSR_INTEL_PKG_ENERGY_STATUS': 0x611,
    'MSR_DRAM_ENERGY_STATUS': 0x619,
    'MSR_PP1_ENERGY_STATUS': 0x641,
    'MSR_CONFIG_TDP_CONTROL': 0x64B,
    'IA32_HWP_REQUEST': 0x774,
}

Sure. Just one more remark. I've upgrade my kernel to 5.8.0-45.
rdmsr -a 0xCE
4043df1011400 4043df1011400 4043df1011400 4043df1011400 4043df1011400 4043df1011400 4043df1011400 4043df1011400
rdmsr -a 0x150
18563 18563 18563 18563 18563 18563 18563 18563
rdmsr -a 0x198
178b00000f00 17b300000f00 17b300000f00 17dc00000f00 17a200000f00 177900000f00 17cb00000f00 17a200000f00
rdmsr -a 0x19C
883b2802 883b2802 883a2802 883a2802 883b2802 883b2802 883a2802 883a2802
rdmsr -a 0x1A2
1e640000 1e640000 1e640000 1e640000 1e640000 1e640000 1e640000 1e640000
rdmsr -a 0x1FC
24005f 24005f 24005f 24005f 24005f 24005f 24005f 24005f
rdmsr -a 0x606
a0e03 a0e03 a0e03 a0e03 a0e03 a0e03 a0e03 a0e03
rdmsr -a 0x610
4280e800dd80c8 4280e800dd80c8 4280e800dd80c8 4280e800dd80c8 4280e800dd80c8 4280e800dd80c8 4280e800dd80c8 4280e800dd80c8
rdmsr -a 0x611
2a38429 2a38429 2a38429 2a38429 2a38429 2a38429 2a3849e 2a3849e
rdmsr -a 0x619
7af721 7af721 7af721 7af721 7af721 7af721 7af721 7af721
rdmsr -a 0x641
623c8 623c8 623c8 623c8 623c8 623c8 623c8 623c8
rdmsr -a 0x648
12 12 12 12 12 12 12 12
rdmsr -a 0x774
rdmsr: CPU 0 cannot read MSR 0x00000774

Ok I've pushed a small fix for this issue. Now if you disable (comment) HWP_Mode in config the 0x774 MSR should no more be read/write. I hope that everything else should still work.


Support char type

Add support for the Rust char type.
Closes #103

Looks great, just a small Swift nit and it's good to merge

@Lucretiel already fixed ;).

I'm not seeing it on the PR; does it need to be pushed?

I'm not seeing it on the PR; does it need to be pushed?

@Lucretiel You mean the change requested by @CerulanLumina? I amended the change just after they noticed it. This is the current version that shows up when I check the changes in this PR: https://github.com/1Password/typeshare/pull/128/files#diff-e3f812823b9b43de87bbd7c54ab61d3e102c387d74245c65cd8a0e3bbca2193e, is there some other thing I missed?

Whoops, it was I who didn't didn't press Push on my review and just let it sit as a draft

@Lucretiel done!

Looks good! Just need the commit to pass signing then we'll merge!

There was some weird mixup with my GitHub account configuration, reuploaded my GPG key and it seems to work. Thanks!


HTML/CSS changes to make comparison table header sticky

PR adds stickiness to the header of comparison table.

@deobald Hi, I've tried to assign you as a reviewer but for some reason it doesn't work hence this comment stands as a call.

Apologies! I was traveling and then this got lost in my email.


Update Composer dependencies (2023-04-26-00-09)

Visual regression test passed!
View the visual regression test report


hive: Unify parsing of string slices

Hive uses pflag and viper to parse configuration flags from multiple sources. If a flag is set via command-line then the pflag parser is invoked to get to the destination type as defined in the FlagSet ("flags.StringSlice" 1), however if the flag comes from enviroment or config-map, then the parsing was done by a mapstructure hook 2.
This is all well and good as long as these two ways of parsing into say []string are aligned with each other. And they were. Unfortunately these were not aligned with the pre-Hive way of parsing which used viper.GetStringSlice and similar methods. Specifically viper.GetStringSlice is implemented (3) via cast.ToStringSlice, which uses strings.Fields that splits by whitespace instead of by commas.
So to summarize the different ways a StringSlice can be parsed:

1: flags.StringSlice: parses with csv.Reader (split by comma)
2: stringToSliceHookFunc: splits by comma
3: viper.GetStringSlice: splits by whitespaces

So while arguably the first two are more consistent, we can't just flip from splitting by spaces to splitting by commas as that creates a huge foot-gun when fields are moved from option.Config to individual hive.Config structs.
To solve this we allow splitting both ways by using two mapstructure hooks that process the values before they're pushed to the config struct:

stringToSliceHookFunc now splits first by commas and if it doesn't produce multiple values it will split by commas. This handles the values going to []string and coming from "flags.String", config file, config map and the environment.

fixupStringSliceHookFunc takes []string coming from flags.StringSlice that was split by comma and resplits it if splitting by comma resulted in a single value.

With this, we have unified the parsing of []string across all the config input methods:

"foo,bar,baz" => []string{"foo", "bar", "baz"}
"foo bar baz" => []string{"foo", "bar", "baz"}
"foo bar,baz" => []string{"foo bar", "baz"}

Fixes: #29210
Fixes: b407ffce15 ("hive: Reimplement on top of dig")
Unify parsing of StringSlice flags and allow splitting by commas (preferably) or by spaces. This fixes parsing of 'prometheus.metrics'.

Marking with needs-backport/1.14 as pkg/metrics was modularized in v1.14 and thus split  '--metrics' by comma resulting in prometheus.metrics helm value to be incorrectly set.

/test

It's not clear to me why we need both stringToSliceHookFunc and fixupStringSliceHookFunc. Doesn't mapstructure.StringToSliceHookFunc(","), already split by comma? There's some overlap between both functions introduced here.

stringToSliceHookFunc is getting the input as string (e.g. from environment or configmap), fixupStringSliceHookFunc  is getting the input from pflag's StringSlice where it's already of type []string and split by comma and we're post-processing it to split the same way as we do when it's coming from environment variables or configmap.

Also, isn't is possible to wrap mapstructure.StringToSliceHookFunc() and make one take precedence over the other?

Ah now that you said it, yes I think we can just have StringToSliceHookFunc split by commas and then have fixupStringSliceHookFunc do the rest. I'll try it out.

A quick overview of all the kinds of expected inputs would be nice to have as a comment. 👍

Doesn't the commit description list them? ("So to summarize the different ways a StringSlice can be parsed:").

/test


Add option to directly delete artifacts from build results

Describe your changes

Added deleteArtifact callback to CompareArtifactModal
Modified ArtifactCard call sites in CompareArtifactModal to pass deleteArtifact to onDelete
Renamed ExcludeButton to ArtInclusionButton as this button is now a checkbox indicating whether to include the specified artifact in optimization and moved it onto its own line to avoid squishing the character selection box any further with the addition of the delete button
Added new translation keys includeArtifactTip and includeArtifactButton and removed excludeArtifactTip, exclude, and include under excludeArt in page_character_optimize.json

Issue or discord link

Resolves #849

Testing/validation

CompareArtifactModal with these changes (as well as the Exclude Artifacts button showing the artifact is excluded)

Modal after deleting an artifact as well as the deletion being reflected in the build itself

Confirming that the artifact has indeed been removed from the database

Checklist before requesting a review (leave this PR as draft if any part of this list is not done.)

[x] I have commented my code in hard-to understand areas.
[x] I have made corresponding changes to README or wiki.
[x] For front-end changes, I have updated the corresponding English translations.
[x] I have run yarn run mini-ci locally to validate format and lint.
[x] If I have added a new library or app, I have updated the deployment scripts to ignore changes as needed

There are a couple ways I could go about that, either make the box turn red indicating the artifact is currently excluded or make it turn green indicating the artifact is currently included (I'm leaning towards this one since it would match the style of the other two exclude buttons).
What should I do about translations though? "Exclude/Include this artifact from optimization" doesn't currently exist in the translation entries. Or should I make do with the excluded/included text for now and let the tooltip clarify the rest?

https://github.com/frzyc/genshin-optimizer/blob/c4356cedb2e32807eed121954fa9534b1a8f0df9/libs/gi/localization/assets/locales/en/page_character_optimize.json#L36
you can add a new key here for 'includeArtifactTip' and a new key for the button text too. if the old keys aren't used anywhere else, we should delete them as well

I've made the necessary translation changes in page_character_optimize.json for the new tooltip and button text and removed the old ones, don't think anything else uses them (but I'll check this again to be extra sure). Testing/validation images have been updated again.

Thanks for your work!


I can't log in To My Business account !

I just downloaded version 1.8.2 but I can not log in, opens a second screen(Pop up) blank .
Could anyone help me ?

write your business name  just
like this:
btitlearning

@viniciosds
write your business name  just
like this:
btitlearning

@viniciosds
escreva o nome da sua empresa
assim:
btitlearning

@ devameer
Thank you very much, it worked !

ya it really work. you have to just type following syntax
companyname-> login using access token


fix: remove invalid indentations and empty line break, so that the partial can be used in nested shortcode

Alternative to #32.

You're welcome, I drafted a new release, I also added you as a collaborator on 30f968b65c67214c8937c76012455f8f4c515547 commit, thanks for your feedback.


location of the header file

hello.
I wonder where can I find the header files.
#include <drivers/fpga.h>
#include <sys/printk.h>
Thank you.

Hi,
These files are part of the Zephyr RTOS tree itself, please follow the README file for more details about how to build it.
Please let us know if you have any extras questions!


Check server connection on settings page

Maybe you can add a simple "check" button on the page to inform the user if everything is entered correctly?

That would be nice I guess. I'm not actively working on this project though. Feel free to do it yourself.


[KYLIN-4106] fix Illegal partition for SelfDefineSortableKey when Extract Fact Table Distinct Columns

do % opration before abs ，more detail：https://issues.apache.org/jira/browse/KYLIN-4106

didn't connect with jira


chore: use const MAX_STATUS_LENGTH instead of 512

What this PR does 📖
Uses the const MAX_STATUS_LENGTH instead of 512. It was already being used everywhere else in warp-ipfs, but not in this case. Also, 512 is value of MAX_STATUS_LENGTH
Which issue(s) this PR fixes 🔨
Special notes for reviewers 🗒
Additional comments 🎤

Can you correct the errors before merging?

Can you correct the errors before merging?
EDIT: Looks like its a function that isnt used. Might be best to remove that instead?

Somehow I left the constant out from the use statement, my mistake. But yeah, if it isn't used by anything in Warp or Uplink, then makes sense to remove it.

Can you correct the errors before merging?
EDIT: Looks like its a function that isnt used. Might be best to remove that instead?

Done.


Add go report badge

Our go report here looks pretty good, so we should get that badge in ASAP.

Hi, I'm new to the repo, but I can submit a pull request if you would like me to. I found this repository through the "good first issue" label.
Also, the Go report card did catch some errors

Yeah... I've been trying really hard to not think about the fact that I misspelled "propagate" three times. Whoops.
Issue assigned, we're happy to have you! If you would like to fix those spelling and assignment errors, that would be greatly appreciated. It looks like they're all either in tests or non-exported code, so it shouldn't be a breaking-the-api issue to do so.

Happens, lol.
@jakenichols2719 is it ok if I submit badge and fixes as 1 PR?

That would be great, thank you! ☺

Closed by #68, thank you @ravsii!


Unused imports

Hi @zxcpoiu, great job on the voip push notifications!
I have a small question. While I was inspecting your code I noticed that there are unused imports:
#import <React/RCTConvert.h>
#import <React/RCTUtils>
I am new to Objective-C so I cannot be 100% sure about this. So if they are really unneeded, should I fire a PR for that or you will just adjust that in next release?

I was refer to some boilerplate somewhere, if you removed the two import and everything are build and work as usual, it's welcome to fire a PR then :+1:


Feature Request - Option to Show .git/hooks when workspaceContains:.git/hooks/"any file that has no extension"

Feature Request
Option to show .git/hooks when workspaceContains:.git/hooks/"any file that has no extension"
Usage
Would be handy to edit git hooks

Closest glob pattern I could find

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


MAIN: Enable support for Sphinx>=4

This PR enables support for sphinx>=4

@chrisjsewell the current commit to this should setup a test environment through tox that includes sphinx4 but it doesn't seem to work for me. It keeps installing sphinx=3.5.4. Any ideas?
I widened the sphinx version requirement in setup.cfg and then added a sphinx4 deps section in the tox.ini

myst-parser 0.14 does not support sphinx v4:https://github.com/executablebooks/MyST-Parser/blob/master/CHANGELOG.md#0150---2021-06-13

myst-parser 0.14 does not support sphinx v4:https://github.com/executablebooks/MyST-Parser/blob/master/CHANGELOG.md#0150---2021-06-13

Ah of course. Thanks @chrisjsewell. I'll pin to myst-parser=0.15. Thank you!

Pin myst-parser~=0.15.1 👍

Note: when upgrading jupyter-book with this new version, it will be going from myst-parser v0.13 -> v0.15.
There are a number of additions to configurations to consider: https://github.com/executablebooks/MyST-Parser/blob/master/CHANGELOG.md#0150---2021-06-13

Adding tasklist to the default myst_enable_extensions will close https://github.com/executablebooks/jupyter-book/issues/1133
setting myst_dmath_double_inline=True will close https://github.com/executablebooks/jupyter-book/issues/1181

perhaps that could also be set to true by default here (currently set to False, to be overly cautious about being back-compatible)

the myst_url_schemes can be removed from the default config, since it is now set already
sphinx 4 will close https://github.com/executablebooks/jupyter-book/issues/963, but at the same time user who were previously setting mathjax_config should be warned that they now need to set mathjax2_config/mathjax3_config https://www.sphinx-doc.org/en/master/usage/extensions/math.html#confval-mathjax3_config
users should be warned about the removed deprecations: https://github.com/executablebooks/MyST-Parser/blob/master/CHANGELOG.md#remove-v013-deprecations-

Great points from @chrisjsewell:

[ ] test fixtures need to track docutils https://github.com/executablebooks/MyST-Parser/blob/be4838c35d374a51427a92a2736b29a9abc018d4/setup.cfg#L74
[ ] test fixtures need to track sphinx https://github.com/executablebooks/MyST-Parser/blob/be4838c35d374a51427a92a2736b29a9abc018d4/tests/test_sphinx/test_sphinx_builds.py#L101

So myst-parser==0.15.1 seems responsible for the changes in fixture outputs. Maybe this is due to the docutils==0.17 rather than a sphinx3 vs sphinx4 update. @chrisjsewell from your work on myst-parser tracking sphinx versions is still going to be needed going forward right?
If you look at diff for: tests/test_parser/test_complex_outputs.sphinx3.xml
those outputs were built with sphinx3 but the toxenvironment moved up tomyst-parser=0.15.1anddocutils~=0.16`

Closing this as work is now contained in #356


Duplicate 'Xunit.TestFramework' attribute

Note: New issues raised, where it is clear the submitter has not read the issue template, are likely to be closed with "please read the issue template". Please don't take offense at this. It is simply a time management decision. If someone raises an issue, and can't be bothered to spend the time to read the issue template, then the project maintainers should not be expected to spend the time to read the submitted issue. Often too much time is spent going back and forth in issue comments asking for information that is outlined in the issue template.
Preamble
When using recent versions of Verify and Verify.Xunit, both assemblies include the
[assembly: VerifyXunit.UsesVerifyAttribute()]
output to the [].Attributes.cs
and gives this error when compiling the test project
Duplicate 'Xunit.TestFramework' attribute

Current stable versions:

Microsoft Visual Studio Professional 2022
Version 17.8.7
VisualStudio.17.Release/17.8.7+34601.278
Microsoft .NET Framework
Version 4.8.09032
Installed Version: Professional
ASP.NET and Web Tools   17.8.358.6298
ASP.NET and Web Tools
C# Tools   4.8.0-7.23572.1+7b75981cf3bd520b86ec4ed00ec156c8bc48e4eb
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.
Microsoft Azure Hive Query Language Service   2.6.5000.0
Language service for Hive query
Microsoft Azure Stream Analytics Language Service   2.6.5000.0
Language service for Azure Stream Analytics
Microsoft Azure Tools for Visual Studio   2.9
Support for Azure Cloud Services projects
Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines
NuGet Package Manager   6.8.1
NuGet Package Manager in Visual Studio. For more information about NuGet, visit https://docs.nuget.org/
PowerShell Pro Tools for Visual Studio   1.0
A set of tools for developing and debugging PowerShell scripts and modules in Visual Studio.
Razor (ASP.NET Core)   <IP_ADDRESS>5201+d135dd8d2ec1c2fbdee220e8656b308694e17a4b
Provides languages services for ASP.NET Core Razor.
Any code or stack traces must be properly formatted with GitHub markdown.
Describe the bug
Because of output the assembly reference from both packages, the project gives an error:
Duplicate 'Xunit.TestFramework' attribute
https://github.com/search?q=repo%3AVerifyTests%2FVerify+WriteCodeFragment&type=code
If you updated src/Verify.Xunit/buildTransitive/Verify.Xunit.props
to only add the output if the assembly is not already referenced, or remove it altogether, it will resolve the issue.
A clear and concise description of what you expected to happen.
Update the library to eliminate the bug.
Add any other context about the problem here.
Minimal Repro
Create test project on framework 4.7.2
Add reference to

[VerifyXunit.UsesVerify]
[TestClass]
public partial class ApiGatewayKeyHelperTests : VerifyXunit.VerifyBase, IClassFixture<ApiGatewayKeyHelperTestsContext>, IDisposable
{
ApiGatewayKeyHelperTestsContext _context;
public ApiGatewayKeyHelperTests() : base() { _context = new ApiGatewayKeyHelperTestsContext(); }
private ApiGatewayKeyHelperTests(ApiGatewayKeyHelperTestsContext context) //: base()
{
    _context = context;
}
[Theory]
[MemberData(nameof(GetData))]
public Task UseFileNameWithParam(string arg)
{
    //VerifyXunit.Verifier.Verify()
    return Verify(arg)
    .UseFileName($"{arg}.UseFileNameWithParam");
}
public static IEnumerable<object[]> GetData()
{
    var temp = new List<object[]>();
    temp.Add(new object[]
    {
        "Blue"
    });
    return temp;
}
}
public partial class ApiGatewayKeyHelperTestsContext : IDisposable
{
}

Submit a PR that fixes the bug
https://github.com/VerifyTests/Verify/pull/1250
https://github.com/search?q=repo%3AVerifyTests%2FVerify+WriteCodeFragment&type=code

you should no longer need [VerifyXunit.UsesVerify] on any test class

VerifyBase already has [TestClass]. so u dont need it on any inherited classes

if you are still having issues please follow the issue template

Upload this repro as part of the issue, as a public GitHub repository or a downloadable zip.

VerifyBase already has [TestClass]. so u dont need it on any inherited classes

Even when removed the error occurs. I appreciate you relooking into this issue.

@mlinderor can you share a repro as a zip or a github repository

https://github.com/mlinderor/VerifyDemoXunitAttr

can u try 26.0.1

That eliminated the error and allows me to run my unit test. Thanks.


sonarr: support additional param for wanted

Proposed change

Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (which adds functionality)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[ ] The code change is tested and works locally.
[ ] Local tests pass.
[ ] There is no commented out code in this PR.
[ ] The code has been formatted (make lint)
[x] Tests have been added to verify that the new code works.

@tkdrob one more here. question have you tried Sonarr v2 api or focused on v3? I'm just working out forcing users to upgrade or if there's some way to mostly work with v2 api (no api version in path)

@tkdrob one more here. question have you tried Sonarr v2 api or focused on v3? I'm just working out forcing users to upgrade or if there's some way to mostly work with v2 api (no api version in path)

No. I have not focused on v2. How different is it?

I know the some endpoints had things like automatic series or episode file inclusion which are now parameters in v3.
Honestly, Sonarr devs announced v2 EOL about a year ago so I'll likely just rip the bandaid now.

@tkdrob any chance to merge and release this weekend? Need these changes for my sonarr PR

Thanks!


DOCS: 4.x Scheduling

Problem Description
We need to update docs for 4.x

[x] update scheduling.adoc for SE
[x] update scheduling.adoc for MP

Guidelines

[x] Make sure any fixes from SE 3.x are forward ported to 4.x #7515
[x] Make sure any fixes from MP 3.x are forward ported to 4.x
[ ] Initial focus is on correctness
[x] Record migration impact in this issue

Migration Impact
Summarize incompatibilities between Helidon 3 and Helidon 4.
Summarize actions customers must take to migrate code
OK to reference external specs or documents (for example MicroProfile specs)

No changes or migration is needed


Make Vagrant find the embedded "bsdtar" on 64-bit MSYS2

The 64-bit version of MSYS2, which ships with 64-bit Git for Windows 2.x,
returns "MINGW64_NT-6.3" for "uname -s", which does not match "MINGW32*".
Make it match by removing the bitness from the case string.
Fixes mitchellh/vagrant#5871.

I see this has been addressed in a slightly different way in d747df2fb7e7a9ba89bec4bd828f976aa3d81053. However, that fixes vagrant_substrate only. I'll update this PR to also fix vagrant_installer in the same way.

Hi @sschuberth
Thank you for the PR. This has actually been fixed on master.

Sorry. I didn't read your second comment :smile:

Thanks!

Actually (at least on Win10 with ConEmu) it returns "MSYS_NT"... https://github.com/mitchellh/vagrant-installers/pull/59

@Hades32 Well, that depends on the type of MSYS2 shell environment you're using. Granted, by default the MSYS2 installer starts the MSYS2 Shell, but there's a MinGW-w64 Win32 Shell (and MinGW-w64 Win64 Shell) as well. The MSYS2-based Git for Windows 2.x uses the MinGW shell for its Git Bash, so in that environment checking for MINGW32* and MINGW64* is sufficient.


Sites, specimens, and timeseriesresults should not be anchors

These tables are just extension tables and shouldn't have any sort of anchor.  They shouldn't be referred to on their own.  So in the column definitions block, the results should be the *ResultID##, not the *TimeSeriesResultID##.  I don't think it makes much practical difference for the YAML itself, but it makes it a bit off of ODM2.

This is probably true of Sites because the Site anchors are not referenced by anything else.  I think I added them for consistency.  I was thinking - just give everything an anchor.  The same may be true of Specimens.  I would be OK with either convention:

Give every entity instance an anchor; OR
Only give things anchors if they are referenced somewhere else.

The anchors for TimeSeriesResults are actually used.  TimeSeriesResults are not the same as Results and need to have separate anchors.  The Result anchors are referenced by TimeSeriesResults, and the TimeSeriesResultAnchors are referenced by the column definitions in the data block.

In your example you did use the TimeSeriesResults anchors, but they really should not be used.  You should refer directly all the way back to the Results anchor.  That's what would happen in ODM2 because the TimeSeriesResults table doesn't actually have any primary key that can be referred to. I think for consistency with ODM2 we should have anchors be equivalent with primary keys.  So tables that have their own primary keys have anchors and those that don't, don't.  The only exception is if the table's primary key is "BridgeID," those tables also should not have anchors.

The TimeSeriesResults table does have a primary key in ODM2 (ResultID).  It's primary key shares the same name as the primary key of the Results table because TimeSeriesResults is a specialization of Results.  But, since there are different attributes in both tables and they are separate tables in ODM2, they need to appear separately in the YAML file with separate anchors.  Stephanie and Jacob have made it clear that this is needed (or at lease makes it WAY easier) to parse the files into the ODM2 API objects.  You can't use the same anchor to reference two different objects in YAML.

Ok.  I think I'm confused about the structure in ODM2.  I thought the "ResultID" acting as a primary key in the TimeSeriesResults table was actually itself a foreign key and identical to the "ResultID" in the result table.
Either way, though, because there aren't really primary keys in the YAML, if adding an anchor to the TimeSeriesResults table makes parsing easier, we can do it.

The relationship between the Results entity and the TimeSeriesResults entity is effectively one-to-one.  So, the ResultID in TimeSeriesResults acts as both a primary key and a foreign key.
Yes - add the anchor to TimeSeriesResults in the YODA files.

I did do this some ways back.  Just closing issue to get it out of the way.


Failed reading config: Invalid db.major_version: 15.

Bug report
I try to use github actions to reflect the information in the remote project, but it fails.
Describe the bug

Failed reading config: Invalid db.major_version: 15.

To Reproduce

git push
my check github action
See error

Expected behavior
Successfully validate github action
Screenshots
error

db version

my reference
https://github.com/supabase/supabase-action-example
https://www.youtube.com/watch?v=rOLyOsBR1Uc

I had the same issue.
I've just updated the version of the github action to latest:
      - uses: supabase/setup-cli@v1
        with:
          version: latest

Had to upgrade supabase cli with brew upgrade works like a charm afterwards ;)

Thank you all, I have successfully upgraded the supabase cli as described above.
commit : fix: supabase/setup-cli@v1 version: latest

Issue closed as resolved.


Brainstorm .clang-format proposal

IMPORTANT: This is NOT ready to merge, this may NEVER be merged. It aims to serve as subject for discussion if and how clang-format may become useful for us.
Description
Following ~90~ 100 character column limit, I decided to try clang-format and start development of annotated .clang-format configuration file that may be useful for Boost.GIL
Please, consider this as a proposal to brainstorm, try and refine .clang-format for Boost.GIL.
Goals:

Provide .clang-format with reasonable defaults for Boost.GIL as a basic formatting guideline and helper.
Support edit - select (class, function) - format workflow  with consistent settings across editors.
Help for manual formatting: run clang-format first, then  adjust formatting in those (few) palces where necessary.
Use clang-format from its latest release.

No goals:

No automatic reformat on file save or pre-commit is proposed.
No bulk reformat is proposed.
No CI build running clang-format verifying code style.

Environment

clang-forma 5.0 or later

Can you add a couple of problematic files which would benefit? Just so I can see a difference.
Also, are these tools good enough for complicated templated code?

@chhenning No, I can't because I didn't have time to provide you guys with comprehensive picture that you can just have a look at.
I just

prepared .clang-format with annotations so it is easier to read and modify without constantly looking at https://clang.llvm.org/docs/ClangFormatStyleOptions.html
I tried to filter out options not available in clang-format 5.0

ATM, this PR is broken, see Recent develop vs master shuffles broke existing pull requests

I have just asked the StackOverflow for advise on desire formatting using clang-format
Can clang-format split template parameters list in multiple lines?

I answered a StackOverflow question on spacing and breaking trailing return type with clang-format which is also related to desired formatting discussed here.

23702f9879e81bb3b08971eb153b199ea56bb0af adds good-enough .clang-format to example/clang-format/
This is an example which offers good enough configuration and
may be useful when writing code for GIL, but is not required though.
After using it myself for fair bit of time now, it proved useful.
This is not a complete configuration!


made the methods skipped in the json/multipart/urlencoded middlewares optional

Added a new options parameter for each; `skip_methods'. it, by default, is set
to ["GET", "HEAD"]. This is in line with the previous behaviour. All existing
tests have been confirmed to pass still. As far as I can see, there are no BC
breaks.

What's the use case? Have you measured the performance hit?

The use case is sending a JSON body (containing a complex query) with a GET request. I actually haven't tested the performance - is there a preferred benchmark setup I should use to do so?

we check the content-type each time anyway, let's just remove the method checks and add some tests for GET

That would work even better.


Obuilder_spec.Docker.dockerfile_of_spec: obtaining a Dockerfile.t

Hi, I'd like to use obuilder_spec to describe an image and be able to locally build it using Current_docker.
For now it's complicated because Current_docker.Default.build expects a ('Contents of Dockerfile.t), but Obuilder_spec.Docker.dockerfile_of_spec returns the dockerfile as a string. So I actually have to write down the content to a file using Current_fs.save and then use 'File of Fpath.t in the current_docker build.
Do you think it would be possible to have the conversion from an obuilder spec to a Dockerfile.t to simplify the process ?

If ocaml-dockerfile added a Dockerfile.of_string function you could just map it using that. Otherwise, I don't think obuilder can do anything about it. Note that OBuilder doesn't use ocaml-dockerfile itself (since https://github.com/ocurrent/obuilder/pull/51).

Assuming I have followed through ocaml-ci's code correctly (!!), this works in ocaml-ci because the Dockerfile is physically written and the build is then carried out by a direct call to Current_docker.Raw.Cmd.docker.
The suggestion this morning was that the ?dockerfile argument should also have an option for a raw dockerfile - i.e. add  `Raw of string or some such to the ?dockerfile argument in S.DOCKER.build.

Ah, yes! OCurrent should probably just stop using the Dockerfile.t type completely. The only function OCurrent ever calls from that module is Dockerfile.string_of_t!


Extraneous preview window title highlights

Info

Operating System: macOS Ventura 13.4
Shell: Fish
Terminal: Kitty
nvim --version: NVIM v0.10.0-dev-569+gfa0a25dcb
fzf --version: 0.42.0 (brew)

require('fzf-lua').setup({
  winopts = {
    preview = {
      scrollbar = false,
    },
  },
})

Description
Title highlights are not cleared when winopts.preview.scrollbar = false, resulting in highlight artifacts that erroneously apply to the preview window border instead, as demonstrated in the following screenshot:

Tentative fix in https://github.com/ibhagwan/fzf-lua/pull/797.

https://github.com/ibhagwan/fzf-lua/commit/7aff4a3a089c65c6e60646945d8a5c8116078061
@0xcharly  - should solve your problem, lmk if otherwise.

@0xcharly if you don’t mind me asking, what settings you used to get the look and feel in your screenshot?
I believe @RayZ0rr was after something similar in #760 :)

I've replied on #760 directly. Let me know if you want me to make it a profile.

I've replied on #760 directly. Let me know if you want me to make it a profile.

It does look great, would love to accept a PR with your profile, I’m just not sure how to craft this without the theme overrides, maybe with a custom winopts.border using “fat” characters?


Trace Objective-C method calls

Ability to trace some classes would be great part of ALPHA. We can integrate Xtrace library to bring this functionality for ALPHA.

One more implementation example:
https://github.com/DavidGoldman/InspectiveC


Bump circe to 0.14.1

No rush I’m just keeping my libraries up to date. It’s the version caliban (a graphql client) is using.


QoL: OoT Speed up Owl Flying Cutscenes

What the title says. PR for the Owl cutscenes in OOTR: https://github.com/TestRunnerSRL/OoT-Randomizer/pull/677 (possibility they've been updated further since then, not 100% sure).

Added


chore: publish type information

For others to depend on this library and get type support with mypy this
file is needed.
Fixes #5 .

Looks good, will this file automatically be included/packaged in the distributed library?

AFAIK, yes. Took me some time to figure it out last time, so I didn't check in detail now.
As long as you have include_package_data = True (which you do) and this file you should be good.

I'm sorry. I forgot this link: https://www.python.org/dev/peps/pep-0561/


Variables declared inside false conditionals result in error

If a variable is declared inside a conditional and the user attempts to access it inside the same block, an error will occur saying it does not exist. This is correct behaviour in regards to the fact that the variable is not created as a result of the condition being false (it is created if its true, however), though the attempted access should not occur, thus resulting in the error. The below examples demonstrate this:
var = 0
if 2 < 2
{
    loop i from 0 to 10
        var += i
}
var
Error: variable 'i' does not exist

var = 0
if 2 < 2
{
    x = 1
    var = x
}
var
Error: variable 'x' does not exist

Its worth noting that these code snippets work correctly if the conditions are true.

Refer to issue #60 for further details on issues caused by this fix.


Incorrect Shell Completion Docs for oh-my-zsh users

(Sorry for deleting parts of the issue template but it doesn’t seem designed for reporting documentation issues.)

[x] ran brew update and can still reproduce the problem?
[ ] ran brew doctor, fixed all issues and can still reproduce the problem?
[x] ran brew config and brew doctor and included their output with your issue?

What you were trying to do (and why)
I was trying to enable shell completions for the Github CLI which I’ve installed via Homebrew. I added the Homebrew completion snippet to .zshrc. after the call to oh-my-zsh.sh and opened a new terminal but when I typed ‘gh’ and pressed tab I got file-path completion options instead of Github CLI completion options.
My solution was to add the following to .zshrc after source $ZSH/oh-my-zsh.sh:
if type brew &>/dev/null; then
  FPATH=$(brew --prefix)/share/zsh/site-functions:$FPATH

  autoload -Uz compinit
    rm -f ~/.zcompdump; compinit
fi

Assuming this is not an issue specific to my machine the docs should be updated as follows:

Replace the snippet with the one I provided.
Specify that the snippet should be included after calling oh-my-zsh.sh rather than before.

I also reported a related issue on the Github CLI because they link to the Homebrew Shell Completion docs.
Output of brew config and brew doctor commands

➜  ~ brew config
HOMEBREW_VERSION: 2.2.16
ORIGIN: https://github.com/Homebrew/brew
HEAD: c0f5a52d85d3606e7600162a9d38936ab4b46f4f
Last commit: 6 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: ed660bb2cebacacfc8d7f3569d9de1c049ea7792
Core tap last commit: 24 minutes ago
HOMEBREW_PREFIX: /usr/local/homebrew
HOMEBREW_REPOSITORY: /usr/local/homebrew
HOMEBREW_CELLAR: /usr/local/homebrew/Cellar
HOMEBREW_EDITOR: code
HOMEBREW_MAKE_JOBS: 4
CPU: quad-core 64-bit skylake
Homebrew Ruby: 2.6.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
Clang: 11.0 build 1103
Git: 2.26.2 => /usr/local/homebrew/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 10.15.4-x86_64
CLT: N/A
Xcode: 11.4.1
➜  ~ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: You have unlinked kegs in your Cellar.
Leaving kegs unlinked can lead to build-trouble and cause brews that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
  python
  gettext

Warning: Your Homebrew's prefix is not /usr/local.
Some of Homebrew's bottles (binary packages) can only be used with the default
prefix (/usr/local).
You will encounter build failures with some formulae.
Please create pull requests instead of asking for help on Homebrew's GitHub,
Discourse, Twitter or IRC. You are responsible for resolving any issues you
experience while you are running this unsupported configuration.

Warning: Homebrew's sbin was not found in your PATH but you have installed
formulae that put executables in /usr/local/homebrew/sbin.
Consider setting the PATH for example like so:
  echo 'export PATH="/usr/local/homebrew/sbin:$PATH"' >> ~/.zshrc

Warning: Some installed formulae are missing dependencies.
You should `brew install` the missing dependencies:
  brew install node

This looks like it's pretty specific to your machine (or at least: your scripts). Could you submit a PR to edit the docs? It'll be easier to discuss that. Thanks!


Bug: Sign up link to create OpenSauced account is not working

Describe the bug
The sign-up link to get started with OpenSauced is not working.

Gives this;

I tried like three different browsers, still not working.
Besides, I can't log in to my OpenSauced account.
Steps to reproduce

Got to open source intro course
Click on how to contribute to the Open Source section. Better still, follow this link:
https://intro.opensauced.pizza/#/05-how-to-contribute-to-open-source?id=discovering-open-source-with-opensauced
Under discovering open source with OpenSauced
Click on the link that states sign up for an account.

Browsers
Chrome, Firefox, Edge
Additional context (Is this in dev or production?)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

Hi, @BekahHW  and @adiati98.
Please let me know if I am the only one having an issue with my OpenSauced account.

I wasn't able to reproduce on my end. Were you able to try without www. Perhaps was an mis fire or cache thing.

I'm also not able to reproduce it on my end. I typed it into my browser and clicked the link. What browser are you using? Can you try a different one?

Hey @Lymah123,
I can't reproduce it because I have no problem opening the OpenSauced app as shown in the screen recording below.
Can you try https://app.opensauced.pizza ?
Also, this issue is supposed to be opened in the intro repo and not here at docs. 😊
https://github.com/open-sauced/docs/assets/45172775/d129dc61-77a2-464b-9c52-6f94fe667247


[r/ci] Debug intermittent bad-function-call R CI failures

Example:
https://github.com/single-cell-data/TileDB-SOMA/actions/runs/8011378811/job/21884475539#step:16:114
NEEDRESTART-VER: 3.5
NEEDRESTART-KCUR: 6.2.0-1019-azure
NEEDRESTART-KEXP: 6.2.0-1019-azure
NEEDRESTART-KSTA: 1
Error: Failure in `/tmp/RtmpPTUVxA/R_LIBS1c914feff32e/tiledbsoma/tiledbsoma-tests/testthat.Rout.fail`
ng

⠏ |          0 | OrderedAndFactor
⠏ |         10 | OrderedAndFactor
✔ |         11 | OrderedAndFactor

⠏ |          0 | PlatformConfig
✔ |         26 | PlatformConfig

⠏ |          0 | ScalarMap
⠼ |         55 | ScalarMap
✔ |         70 | ScalarMap

⠏ |          0 | SCEOutgest
✔ |      3   0 | SCEOutgest

⠏ |          0 | SeuratIngest                                                   terminate called after throwing an instance of 'std::bad_function_call'
  what():  bad_function_call
Aborted (core dumped)
Execution halted

Will reopen upon reoccurrence


CASMINST-5591: Elaborate on CFS image customization session procedure

Description
The NCN image customization procedure is somewhat vague about the step where the CFS session is run. The command is given for how to run it, but no mention is made of how to monitor it while it runs, how to know when it is done, how to know if it was successful, and how to troubleshoot problems if they occur.
This PR modifies several documents to try and clarify that process. Other minor linting and reorganization is done around the image customization procedure.
This PR is against the CASMINST-5590-1.3 branch because it builds on top of the work done there. Once that PR merges, I can rebase this one to be against the release branch. I just wanted to get this out for review in the meantime.
Checklist Before Merging

[X] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[X] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[X] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport backport/2798-to-main


listening on hard-coded ports in tests considered harmful

I already have a service running on :8080. The test suite fails because it tries to listen on that port. Don't do that.

Quick fix here. I've updated 8080 to 9090. @substack how would you handle this? I could use node-portfinder, but it seems like overkill.

My tests use random ports >10000 but portfinder sounds even better.


[Debugger] Add expression evaluation support for binary and grouped expressions

Purpose
This PR introduces initial implementation for jBallerina debugger expression evaluation with the support for following expression/sub-expression types.

Arithmetic expressions

x + y
x - y
x * y
x / y

Relational expressions

x < y
x > y
x <= y
x >= y

Approach
The proposed implementation approach can be divided into 4 phases as shown below.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[x] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #23963   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

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 6ffea87...9a2016f. Read the comment docs.


command line support

Something like: pocketsaw <sub-module.json> <dependencies.file> {<first-deps-source>|<second-deps-source>} <pocketsaw-dependency-graph.html>.
In order to support a workflow similar to Workflow for using Pocketsaw in a project there must also be a way to skip the check for illegal code dependencies (e.g. --ignore-illegal-code-dependencies).
Also meaningfull error codes will make integration into some scripts easier.

Fixed in f99fc7672f5ddfcc1bca9412f82b17d68f23b3a9, was a typo in the description.


Add a cargo fmt GitHub action

I think that this would just need to run cargo fmt --all --check in CI.


dart2js: combine assignments (shorter code)

[user feedback]
I have this repeating pattern in dart2js output:
W.ew7.$isa=true
W.Koa.$isrg=true
W.Koa.$isa=true
W.ipe.$isrg=true
...
some hundred lines...
Couldn't they be combined to:
W.ew7.$isa=W.Koa.$isrg=W.Koa.$isa=W.ipe.$isrg=true
That would save around 5kb in my situation.
////////////////////////////////////////////////////////////////////////////////////
Editor: 0.7.5_r27776 (2013-09-23)
OS: Windows 8 - amd64 (6.2)
JVM: 1.7.0_21
projects: 1
open dart files: 1
auto-run pub: true
localhost resolves to: <IP_ADDRESS>
mem max/total/free: 889 / 447 / 70 MB
thread count: 28
index: 421861 relationships in 79695 keys in 678 sources
SDK installed: true
Dartium installed: true

Obsolete. We no longer generate this code, rather we use properties.


Support cnpj on transparent checkout and payment request

In order to support Companies as a sender, this PR intends to support CNPJ document on transparent checkout and payment request.

@antoniofilho Thank you for your PR! But could you send it to dev branch, I'm really sorry about it, I will create the CONTRIBUTING.md informing people about this.
I think also it could be good if you rebase its three commits into one.
Again: thank you :)

no problems @dmitryrck and tks by instructions.

@antoniofilho I just created the PR#217, take a look and let me know if it need anything that we can turn better :)


Ensure KeplerPRF integrates to the requested flux in the presence of focus parameters

This PR is an attempt to fix #219.  It makes the integral of KeplerPRF significantly more accurate when focus parameters are passed, though the integral of the PRF still only approximates the requested flux to within ~5%. (This may be due to errors introduced by the interpolation step?)

Those changes make sense to me. As a result, the model gradients have to be changed as well.

I can't figure out how to fix the gradient =/

@barentsen I can do that soonish!

@barentsen Travis seems happy, lucky me :D

Yassss!!! :fireworks:   @mirca is the best :+1: :+1:

@barentsen actually, could you add unit tests for the gradients for different values of scale_col, scale_row?

Codecov Report

Merging #220 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #220      +/-   ##
==========================================
+ Coverage   88.22%   88.25%   +0.02%
==========================================
  Files          24       24
  Lines        3084     3091       +7
==========================================
+ Hits         2721     2728       +7
  Misses        363      363

Impacted Files
Coverage Δ

lightkurve/prf/tests/test_prfmodel.py
100% <100%> (ø)
:arrow_up:

lightkurve/prf/prfmodel.py
97.24% <100%> (+0.02%)
:arrow_up:

lightkurve/targetpixelfile.py
73.07% <0%> (ø)
:arrow_up:

lightkurve/utils.py
100% <0%> (ø)
:arrow_up:

lightkurve/tests/test_targetpixelfile.py
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@mirca Fixed it!  (Thanks, I finally figured out how parameterize works. So simple!)
It's still failing tho =(   I think interpolation errors are at play.

The gradients look fine for me, I think the test
assert np.max(np.abs(prf_grad[param_order[param_to_test]] - diff_prf)
                / (1. + np.abs(diff_prf))) < 1e-5

is actually a bit too rigorous. We should probably use np.mean rather than np.max and increase the tolerance a bit.

@mirca @barentsen this is quite old but it seems from the comments that you guys came to a conclusion that this should be merged. What should we do with this?


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/eventing-writers knative-sandbox/serving-writers
/assign knative-sandbox/eventing-writers knative-sandbox/serving-writers
Produced by: knative-sandbox/knobots/actions/update-deps

#415 should fix the downstream failures