Prefect Blocks 405 Response

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request.
Please do not leave +1 or me too comments, they generate extra noise for issue followers and do not help prioritize the request.
If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Terraform Version
Terraform v1.9.8
on darwin_arm64
Affected Resource(s)

prefect_block

Terraform Configuration Files
terraform {
  required_providers {
    prefect = {
      source = "prefecthq/prefect"
    }
  }
}

provider "prefect" {
  endpoint    = "https://my-prefect-server.com/api"
}

resource "prefect_work_pool" "example" {
  name        = "my-work-pool"
  type        = "kubernetes"
  description = "example work pool"
  paused      = true
}

resource "prefect_block" "secret" {
  name      = "food"
  type_slug = "secret"

  data = jsonencode({
    "name"   = "mine"
    "target" = "prefect-dbt-profile"
  })
}

Output
prefect_block.secret: Creating...
╷
│ Error: Error during create Block Document
│
│   with prefect_block.secret,
│   on main.tf line 13, in resource "prefect_block" "secret":
│   13: resource "prefect_block" "secret" {
│
│ Could not create Block Document, unexpected error: status code 405 Method Not Allowed, error={"detail":"Method Not Allowed"}

Behavior
I am trying to create a prefect block on my self-hosted server. I am only getting a 405 response back though. I am able to create work pools, variables etc on the server using terraform. Blocks just return a 405. Are blocks a cloud-only resource? I have only taken example resources from the documentation.

Hi @quoimec, thanks for reaching out. I wasn't actually able to replicate the error - I took your configuration and applied it against both a Prefect 2 and Prefect 3 instance and it created the work pool and block.
Could you please share the following:

Version of Prefect you're using
Version of the Prefect Terraform provider you're using
Any relevant output fromTF_LOG=debug terraform apply (just make sure nothing secret is published here)

Thanks!

Hey Mitch,

The version of Prefect that I am running currently is 3.0.10. But yesterday as part of debugging, I downgraded to the latest v2 release. Same issue.
The provider version I am using is 2.3.1
I was having a look at this output yesterday too, there's not much here either unfortunately:

prefect_block.secret: Creating...
2024-10-22T06:20:50.507+1100 [INFO]  Starting apply for prefect_block.secret
2024-10-22T06:20:50.507+1100 [DEBUG] prefect_block.secret: applying the planned Create change
2024-10-22T06:20:50.808+1100 [ERROR] provider.terraform-provider-prefect_v2.3.1: Response contains error diagnostic: @caller=github.com/hashicorp/terraform-plugin-go@v0.23.0/tfprotov6/internal/diag/diagnostics.go:58 diagnostic_detail="Could not create Block Document, unexpected error: status code 405 Method Not Allowed, error={\"detail\":\"Method Not Allowed\"}" diagnostic_summary="Error during create Block Document" tf_provider_addr=registry.terraform.io/prefecthq/prefect tf_req_id=1f7dfeeb-c50d-08cc-c20a-4285765278e8 tf_rpc=ApplyResourceChange @module=sdk.proto diagnostic_severity=ERROR tf_proto_version=6.6 tf_resource_type=prefect_block timestamp="2024-10-22T06:20:50.806+1100"

Is there something I can change on the prefect server end to see more logs when the resource is attempting to be created?
Another piece of context - because self hosted has no auth, my server is behind an IP whitelist. I assume the block resource isn't doing something weird and doing something that sends a request from an IP that is not my own right?

Are you seeing anything in the debug output that shows which endpoint it's trying to hit? That'd help debug why we're getting that response.
This almost looks similar to https://github.com/PrefectHQ/prefect/issues/10193 🤔 Any chance you could check your state with terraform state show <resource> or try this with a fresh Terraform state?

Is there something I can change on the prefect server end to see more logs when the resource is attempting to be created?

You should be able to set PREFECT_LOGGING_SERVER_LEVEL, which is by default WARNING I believe. If you're deploying with the Helm chart, the value is loggingLevel.

Another piece of context - because self hosted has no auth, my server is behind an IP whitelist. I assume the block resource isn't doing something weird and doing something that sends a request from an IP that is not my own right?

I wouldn't suspect that to cause a problem, but if we run out of options maybe we can try reaching it directly (temporarily).

hey @quoimec - could you also share any configurations around your prefect-server installation (with any sensitive information redacted/removed)?  Are you using one of our prefect-helm charts - if so, could you share a values file?  If not, could you share any environment variables and relevant settings?

@mitchnielsen This is from a totally fresh terraform state, there are no existing resources on the server or in the state. I have also upped the logging in prefect to DEBUG but saw nothing in the logs during an attempted apply. Interestingly when I apply a work pool, I also don't see any logs regarding creation, just that it's coming online.
resource "prefect_work_pool" "example" {
  name         = "my-work-pool"
  type         = "kubernetes"
  paused       = false
}

00:22:41.521 | INFO    | prefect.server.api.validation - Cannot validate job variables for work pool my-work-pool because it does not have a base job template
00:22:41.556 | DEBUG   | prefect.server.events.messaging - Publishing event: prefect.work-pool.not-ready with id: e117623b-46c4-451b-abdb-117bdf875ce2 for resource: prefect.work-pool.d279f4bd-d16b-4b54-9d8f-f796c2c03153
00:22:41.560 | DEBUG   | prefect.server.events.services.event_persister - Received event: prefect.work-pool.not-ready with id: e117623b-46c4-451b-abdb-117bdf875ce2 for resource: prefect.work-pool.d279f4bd-d16b-4b54-9d8f-f796c2c03153

@parkedwards It's a fresh server with no historical data. The PG database has also been wiped multiple times during testing as I tested both v2 & v3 packages.
I am not using the helm chart, I am managing prefect as a dependency through a currently very light poetry install inside a docker container.
[tool.poetry.dependencies]
python = "^3.12"
prefect = "^3.0.10"
prefect-kubernetes = "^0.5.1"

The docker image is running in K8 as a deployment with the following environment variables:
PREFECT_API_URL                     = "https://my.domain/api"
PREFECT_UI_API_URL                  = "https://my.domain/api"
PREFECT_SERVER_API_HOST             = "<IP_ADDRESS>"
PREFECT_SERVER_API_PORT             = "80"
PREFECT_LOGGING_SERVER_LEVEL        = "DEBUG"
PREFECT_API_DATABASE_CONNECTION_URL = "postgresql+asyncpg://.../prefect"

The thing I find weird about this issue is that:

I can access the UI without issue
I can create blocks/work pools/variables via the UI without issue
I can create work pools/variables via Terraform without issue

It is only blocks via terraform that are failing in this way.

I was playing around with the API & UI and I think I may have found something relevant. I noticed that the UI sends the create block request to the api/block_documents/ endpoint which successfully created the block.
When I send the same payload (with a different name obvs) via python requests to that endpoint, it also successfully creates the block. However, when I change the endpoint to api/block_documents - dropping the trailing slash I get the same 405 error. Is there any way to see what endpoint terraform is actually hitting?
API docs list the trailing slash: https://docs.prefect.io/3.0/api-ref/rest-api/server/block-documents/create-block-document

Is there any way to see what endpoint terraform is actually hitting?

i think you're onto something, but im not sure it's the trailing slash specifically, since we always append one in a helper
where we construct the endpoint prefix:

https://github.com/PrefectHQ/terraform-provider-prefect/blob/main/internal/client/block_documents.go#L44

the helper we use to do ^:

https://github.com/PrefectHQ/terraform-provider-prefect/blob/main/internal/client/util.go#L39

thanks for sharing your server setup, let me noodle on this a bit more

ah - @quoimec im being silly. you're right - we're not appending the trailing slash here.  testing it locally, my server handles the redirect through a 307, but im wondering if your specific setup with the whitelist is preventing the redirect from being handled properly
INFO:     <IP_ADDRESS>:54928 - "POST /api/block_documents HTTP/1.1" 307 Temporary Redirect
INFO:     <IP_ADDRESS>:54928 - "POST /api/block_documents/ HTTP/1.1" 201 Created

i'll open up a PR to patch your use case, and we can go from there

hey @quoimec - give 2.3.2 (just released) a go, and see if that resolves your issue


feat: Add fetchCandid() function to @dfinity/agent

Description
This PR adds a function to abstract away the implementation details for retrieving the current Candid interface based on canister metadata and the hidden __get_candid_interface_tmp_hack method (for canisters without this metadata).
At the moment, several projects (notably the IC Dashboard and Candid UI) directly call the _tmp_hack method. This fetchCandid() function can be used as a replacement in both of these web applications.
Example usage:
import { fetchCandid, HttpAgent } from '@dfinity/agent';

const agent = new HttpAgent(...);

const candidSource = await fetchCandid(agent, 'ryjl3-tyaaa-aaaaa-aaaba-cai');

How Has This Been Tested?

Added a new test case: agent/src/fetch_candid.test.ts
Works as expected in a browser environment.

Checklist:

[x] My changes follow the guidelines in CONTRIBUTING.md.
[x] The title of this PR complies with Conventional Commits.
[x] I have edited the CHANGELOG accordingly.
[x] I have made corresponding changes to the documentation. (JSDoc)

I think this is more intuitive than CanisterStatus by default! One suggestion - since candid is public metadata, why not move agent to the second position and make it optional, using an anonymous HttpAgent by default?

Sounds good; updated to use the same anonymous HttpAgent logic from both Candid UI and Motoko Playground.
I also added HttpAgent::isLocal() to simplify the relatively common use case of fetching the root key for local environments. Let me know if there's a better way of approaching this from within the agent-js codebase.

We've avoided the concept of local or mainnet for the sake of handling testnets more naturally for NNS dapps, but I don't think I mind introducing this pattern now - it's useful for most use cases


WIP: report write-up updated

#105

@wyshi @lynnzhao92 @Chenyu-Renee please check this out. This is just a rough update on the report write up to include what we are doing. Please help me to correct any gramma issues or any other problems.

Coverage remained the same at 49.787% when pulling 5ea4958e9463c528630688c34d21ab21801e2a66 on harryh1993:contributing-issue#105 into b294e9dcf3d51b4081570a03989003d9a25faa9a on berkeley-stat222:master.

Harry, good job! The report is more clear now. But there are still some minor mistakes that I want to point out.
Line 14, some grammar problems and some exploration.
The statistical problems in this project are mainly naturally divided into two parts, the modeling part and the simluation of the model part. The modelling part consists of estimating the probability of events transition using the raw data of mouse's spatial and temporal data, and once we've got the model, we could use this model to do predictions and simulate a fake mouse day.
Line 16,
For the estimation, the statistical problems include:

Define the states of interest.
Estimate the transition probability matrix given the data we have
Expand the capacity of our model to take into account the effect of time in our estimation. We want our model to be time-dependent (non-homogeneous) and could also model some of the time series structure of the behaviors of the mice during one typical mouse day

line 24,
We need labels of states of interest with respect to the time intervals in a specific mouse day to extract the structure of the Markov Chain.
line 31,

Feeding: labeled by event F
Drinking: labeled by event W
Other active state behaviors: labeled by event O. This could possibly include all other movements in the AS state of a mouse besides drinking and eating.
Inactive State: labeled by event I.

line 42
"One of the key challenges"
line 65
Project Details:
Data Preprocessing: Hongfei
Modeling: Jianglong
Simlation: Chenyu
Evaluation of length of small time interval chosen: Weiyan, Lynn, Mingyung
Final Report writeup: Weiyan, Lynn, Mingyung
I think these make it more clear. But if you have any further questions, feel free to comment below.

Hello Harry,
line 9 can be paraphrased as below because the meaning is a little unclear:
The objective of Dynamics Analysis is to analyze the behaviors of different strains of mice by using Markov Chain model. The MC model can be used further to explore the discrepancy between different strains, and it even can be used to simulate a particular mouse day for a particular strain.

Thanks for those detailed comments! I will modify the markdown file right away

Coverage remained the same at 53.157% when pulling 80efcd4bbff46452d4a5ba962d101bd6188e84a5 on harryh1993:contributing-issue#105 into 987dbe31b1152d348f4282feb7aef8549bdef16d on berkeley-stat222:master.

@jarrodmillman Please merge and so we can continue to work on the final report based on this! THX!


FR: Support custom lint rules or scripts

Problem
The current lint rules are becoming more robust as developers work on them, but they currently lack a certain amount of flexibility. For example, for the title rule in yaml, I need to read the name of the parent folder as a prefix and add it to the title. Such rules are too personal to be integrated directly into the lint plugin.
This may be more common, and it's hard for developers to add options one by one in the settings.
Solution
This solution can be difficult or easy, depending on the developer, and I offer the following ideas for reference.
Probably the easiest：
Add a new lint rule and execute the script after the other rules have run. This part can even be further simplified with quickadd, which executes the quickadd command after the other lint rules are executed.
Probably the most difficult:
Similar to eslint, standardize, register and manage lint rules, load into existing lint rules via npm packages

Additional Context
How do you think this feature should be handled better? Looking forward to any response from you.

I would like to develop this idea to support the kind of rule that would convert a 'smart' quote to a 'straight' quote
“This phrase is inside smart quotes” and "this phrase is inside straight quotes"

This is definitely is something that I think could be added. It is something that would be hard to make sure it works and we do not get a lot of erroneous bug reports due to bad syntax and things of that nature. I do like the idea and think it definitely could be feasible.
I am guessing it would have to work kind of like custom scripts for Templater in that it lets you specify which folder to look in for the scripts it will use as custom rules. Defining when they run might be tricky as well, but it should be able to be handled. I am definitely not opposed to PRs around this while I work on bugs and adding warning mode. However this is a promising idea to help people out with adding some rules they want without having to add it for everyone if it is a more niche need.
How would this work for mobile?
Would we allow for an option to disable a script that is present in the directory?
How would we determine run order between the custom scripts? Would this be alphabetical, user defined, etc.?

How would this work for mobile?

Just like quickadd plugin, it can work for mobile.

Would we allow for an option to disable a script that is present in the directory?

This can also be handled like quickadd, which writes the options to data.json and then loads them in the settings.

How would we determine run order between the custom scripts? Would this be alphabetical, user defined, etc.?

Just put the custom script after the linter rule and run it, I guess it doesn't really need to be executed before the other rules, at least I do.
I think I can do it in a day if I use the existing quickadd function. There may be problems with debugging and testing, but it's enough for personal use.

If you create the changes to add the basic functionality in a draft PR, I likely can add the UTs and get it into the format that follows how rules are linted in the Linter.
Otherwise, it might take me a little while to get to this. But I do see the benefit of this.

I have submitted a draft PR：

This way I just need to write custom lint code in quickadd, register it as a command, and it will work with linter.

I am not sure on the part about adding it as an Obsidian command. But wriiting the logic to run it makes should be all that is needed yet. I am curious how the js scripts will be run (I am hoping they are functions that can run as using eval is not recommended). I am still taking a look at the PR, but it looks like there are several minor things that woukd need addressing.

install quickadd plugin
write a custom js file in the vault
go to quickadd setting page, create a macro and choose your js file
register this macro as a obsidian command in the quickadd settting page
register obsidian command in the linter page，so that linter can excute the command sometime

While that does seem like that makes running the custom lint rules easy, I don't think making the Linter use another plugin would be recommended here. I do like the concept of using a custom js file.

You can even format md files with python, rust, go and other programming languages with the help of shell command plugins. quickadd, the shell command only serves to register the operation of running the code, as a command, and we linter only use to call the command.

Using another plugin comes with a lot of extra considerations that needed to be taken into account like checking that the specified plugin is installed before enabling the Linter setting toggle for the custom lint rule. It also makes verifying that the custom lint rules are working a decent bit harder as UTs cannot verify that.

I am not sure that QuickAdd being used is something that is not feasible, but there several things that needed to be addressed before we can do that including how to verify that the rule is working correctly.

I probably can't help much then, quickadd does quite a bit of work. templater so far can't run scripts on the phone, but quickadd can, and it works well.

The ease of use of quickadd is reflected in the ui, dragging, suggester, automation, etc. Our linter only has a main.js, I did not even dare to import the ui library. for fear of spoiling the simplicity of the linter plugin.

like checking that the specified plugin is installed

The read is an obsidian command, it doesn't matter if you have quickadd installed or not, and it doesn't matter if you have verified the installation of the plugin.

It also makes verifying that the custom lint rules are working a decent bit harder as UTs cannot verify that.

Verification and testing can also be left entirely to the users themselves, after all, quickadd debugging is quite convenient.

I probably can't help much then, quickadd does quite a bit of work. templater so far can't run scripts on the phone, but quickadd can, and it works well.

It just removed that restriction, but it may not work for iOS.

like checking that the specified plugin is installed

The read is an obsidian command, it doesn't matter if you have quickadd installed or not, and it doesn't matter if you have verified the installation of the plugin.

It also makes verifying that the custom lint rules are working a decent bit harder as UTs cannot verify that.

Verification and testing can also be left entirely to the users themselves, after all, quickadd debugging is quite convenient.

Letting users be the QAs should be done with caution as this can easily create a poor quality result. In this case that may not be the case, but we need to be careful to make sure it works correctly and well.

To make sure I am not misunderstanding what is being said here, the following will be true about this feature:

It will use the specified Obsidian commands as custom rules
It will not require a dependency on another plugin (i.e. it only adds references to Obsidian or Linter specific logic)

If the above two things are true (please correct me if I am misunderstanding something), then I think this addittion s

Sorry I accidentally hit the wrong button.

It will not require a dependency on another plugin (i.e. it only adds references to Obsidian or Linter specific logic)

Well, I'm probably not much help, it's pretty hard for me to custom run the code and get it stable. But now I'm writing this should satisfy my needs, haha！
Thank you for your patience, you can choose to close this issue.

It will not require a dependency on another plugin (i.e. it only adds references to Obsidian or Linter specific logic)

Well, I'm probably not much help, it's pretty hard for me to custom run the code and get it stable. But now I'm writing this should satisfy my needs, haha！
Thank you for your patience, you can choose to close this issue.

I am more than happy to take it from wherever you would like to stop as this does seem like it would fill a need that several people have where they would like a rule added, but we have not gotten to it or it is too niche to justify adding for everyone. Please let me know if you would like me to work off of your current implimentation or if you plan to add some more changes yet.

My last step is to run all the commands of lintCommand array in the data.json file at the end of this function. That's good enough for me, it currently doesn't rely on any other plugins and just runs obsidian's commands.  There are some places I may have missed without reading through the code, and I'll leave it up to you to decide where you should run custom commands

To make sure I am not misunderstanding what is being said here, the following will be true about this feature:
It will use the specified Obsidian commands as custom rules
It will not require a dependency on another plugin (i.e. it only adds references to Obsidian or Linter specific logic)

I just understood what you meant, lol.

Gotcha. I would not add them at the end of that file. I would add it at the end of the try catch statement as that is where the linting logic resides. I may want to move that out to a testable function somewhere, but that would be down the road.

Or you consider more thoughtful, after you add the code I will learn to how to increase the reliability of the plugin.👍👍👍

Sounds good. I would like to refactor it and move it over today, but we will see how that goes.

@windily-cloud , could you provide an example usage of a quick add macro that would update a file? I am curious since I do not see how it is getting the current file info to update the correct information. Thanks for helping me visualize what is going on here.

It may be a little tricky, but it's easy once configured, and I'll elaborate on each step:
Step 1:

Install quickadd plugin
Create a folder in your vault and create a .js file like this:

Step 2:

Write the logic for modifying the current md file in the js file
I write a modify the yaml title key as example:

This file changes the title value in yaml to the file name
module.exports = async (params) => {
  const currentFile = app.workspace.activeLeaf.view.file
  const newTitle = currentFile.basename
  if(!currentFile){
    return
  }
  const frontmatter = app.metadataCache.getFileCache(currentFile)?.frontmatter
  if(!frontmatter){
    return
  }

  const {position: {start, end}} = frontmatter
  const fileContent = await app.vault.cachedRead(currentFile)
  const newFileContent = fileContent.split("\n").map((line, index)=>{
    if(index > end){
      return line
    }
    if(/^title: .*/.test(line)){
      return `title: ${newTitle}`
    }
    return line
  }).join("\n")
  //console.log(newFileContent)
  await this.app.vault.modify(currentFile, newFileContent);
}

Step 3:

Create a quickadd macro and config it like this
Regist this macro to obsidian command
Restart obsidian（Important）

Step 4:

linter config

If something is not clear, please point out (this part of quickadd's documentation is an issue), it took me a lot of effort to figure out how to use macro and how to use quickadd api...😭

That makes sense. It took me a minute to realize that this was essentially updating a file from outside the linter. I understand how this works now. I will do my best to add that to the documentation and the potential caveats that it creates with things like yaml-timestamp. I think I can point people in the rigbt direction on how to create custom scripts.


question about pred_bboxes.detach() in DynamicHead.forward()

In the forward method of DynamicHeadm, pred_bboxes are detached from graph before being used as input of next iteration, but proposal_features are kept. Why not just detach both or keep both at the same time?

Hi~
We tried detach both, keep both, detach one and keep another, current version is the setting with best result.

Thanks for your reply


Error: Abstract getter length accessed

Attach (recommended) or Link to PDF file here:
t.pdf
Configuration:

Web browser and its version:  Chrome 114.0.5735.106（Official Build）(arm64)
Operating system and its version: macOS 11.7.7 20G1345
PDF.js version: PDF.js: 3.8.49 [f2a29e858]
Is a browser extension: No

Notice that the issue also occurs on Safari 16.5.
Steps to reproduce the problem:

Go to https://mozilla.github.io/pdf.js/web/viewer.html
Open the PDF file.
Fonts turn to be sans-serif.
They should be serif.

What is the expected behavior? (add screenshot)

What went wrong? (add screenshot)

The console of Chrome says
PDF c6bc9a289220d30d4a8d30f3dbc266b9 [1.5 dvipdfmx (20210318) / TeX output 2023.06.09:1837] (PDF.js: 3.8.49 [f2a29e858])
util.js:380 Warning: Failed to compile font g_d3_f1
util.js:380 Warning: Error: Abstract getter `length` accessed

It seems that the length getter is called for FlateStream although the getter is not defined for FlateStream:

https://github.com/mozilla/pdf.js/blob/f2a29e858f0e1320c809190d1afa7b937a063be8/src/core/opentype_file_builder.js#LL85C1-L85C1

The max here:
https://github.com/mozilla/pdf.js/blob/f2a29e858f0e1320c809190d1afa7b937a063be8/src/core/cff_parser.js#L1393
is too high.

I'm not sure it's chrome specific: maybe it could happen on Firefox too with an other context.
I tried this in Chrome: [].push(...Array.from(new Array(120000).keys())) and it works well so I guess the number of items is not the only criteria, even [].push(...Array.from(new Array(120000).keys()).map(x => x.toString().repeat(1000000))) is fine.


Cant save rutorrent settings

After installation using this wonderful guide, everything work beside that I cant save settings like default path and sucha things in rutorrent GUI. The log gives no error.
Any idea?

this is an rtorrent setting, so changing in RuTorrent will only last as long as the session, before it reverts back. To change it permanently you need to edit the .rtorrent.rc file in the user home directory.
RuTorrent settings for example the theme should be saved for future sessions.
As far as I know there is no way to permanently change rtorrent settings in RuTorrent
Any additional info you will need to consult the RuTorrent project people.


Use a nice icon for the extension

Currently it just uses the default placeholder.
You already have a nice logo, use it there

thanks for reporting! should be fixed now.


Option for different date format presented to user and used as input.value (or match Chrome & Firefox user-format/value-format behaviour out of the box)

I'm having trouble implementing validation for a date field because this polyfill behaves differently to Chrome's and Firefox's date picker implementations.
[I'm British by the way, so I'm using dd/mm/yyyy as my human readable examples]
Chrome and Firefox will display the date to me in the format dd/mm/yyyy. But if you call .value on the input the result is in the format yyyy-mm-dd,
…makes sense for localisation (e.g. the British/U.S. date format differences).
Calling .value upon an input this polyfill is implemented on returns the date format as it is displayed to the user — as if the input was type="text".
I think because of this difference, the validation I'm using (Bootstrap4) balks at the date value.
For this polyfill, there is an option to choose the date format but it acts on both the user facing and value formats simultaneously.
jQuery UI's date picker has options for both altFormat and dateFormat which allow you to manage a user facing format and a value format. I don't agree with the naming of these options but they facilitate a flexible solution to this scenario.
If this is actually possible, apologies. Perhaps I can help write some documentation on how to implement it.

Ran into this same problem. The value of input needs to be yyyy-mm-dd no matter what is displayed to users to be considered a true polyfill.


--taints and not --taint

This caused me a lot of trouble today... Not a big fix, but can drive someone crazy

Yes these should do exactly the same thing and singular matches the other options better.  If it doesn't do the same thing, that should be addressed in the script, not the Ui.

@Typositoire It looks like the script accepts both
        "--taint" | "--taints"

Are there issues when using just --taint?

There might be another issue then. Documentation specify taints not taint. But when using taint nodes were not tainted upon registering.

@Typositoire Can you raise an issue over at https://github.com/rancher/rancher detailing the problems with --taint? Please reference this issue, and also feel free to tag me and I'll raise it internally.


Any chance to have it upstreamed?

Works great here, would be fantastic to have it part of flycheck upstream.

Hi @peterhoeg.
I'm not using Emacs too much these days (switched to Vim) but I've found 10 minutes to open PR to original Flycheck project. We will see if they accept it.
Kind Regards,
KM.
PR: https://github.com/flycheck/flycheck/pull/1640

Upstreamed successfully.
Thanks for your interest in my project,
KM.

Thanks!!


UNIDIRECTIONAL_SEQUENCE_LSTM run exception in xtensa hifi5 platform.

UNIDIRECTIONAL_SEQUENCE_LSTM run exception in xtensa hifi5 platform.
error msg: "Number of allocated temp buffers: %d. Checksum passing status: %d"
The possible cause is that dealloc is not called.
And the model in "tflite-micro/third_party/xtensa/examples/micro_speech_lstm/micro_speech_lstm.tflite" also run exception.
The model as follows.
AI_aec_LSTM-111--1MSE_Synthetic_cus_-12db_int8.zip
.

I'm uncertain how to replicate this issue. I do see that the micro_speech_lstm_test currently works @HEAD. Does this work for you as well?
rocky@rocky-ironman:~/tflite-micro$ xt-run tensorflow/lite/micro/tools/make/gen/xtensa_hifi4_default/bin/micro_speech_lstm_test
Testing TestInvoke
Ran successfully
1/1 tests passed

rocky@rocky-ironman:~/tflite-micro$

You could replicate this issue in hifi5. [xa_nnlib_hifi5_06_30.zip]
root@DESKTOP-P4I3L06:/mnt/d/eric/tensorflow-micro/daily-micro/tflite-micro# xt-run tensorflow/lite/micro/tools/make/gen/xtensa_hifi5_default/bin/micro_speech_lstm_test
Testing TestInvoke
All temp buffers must be freed before calling ResetTempAllocations()
WARNING Unhandled user exception: LoadProhibitedCause (0x00000000)
( Xtensa ISS ) WARNING run exited with status 'virtual breakpoint'
root@DESKTOP-P4I3L06:/mnt/d/eric/tensorflow-micro/daily-micro/tflite-micro#

@nyadla-sys can you take a look at this issue? This seems to be specific to the xtensa LSTM implementation.

@nyadla-sys have any progress on this issue？Please let me know if you have any questions. Thanks.


Python 3 version should be master

It's time to leave python 2 behind, so I suggest we either merge everything into a single master, or branch python2 off to its own branch for those who still live in The Old World and change the current python3 branch to master.

Sounds sensible. Keeping python2 around as a separate branch is probably a good idea; at least for a while longer.

Fixed


use showOpenFilePicker instead of (in addition to?) uploader file input

This is an experimental browser feature but well worth following. It somewhat allows us to open a file chooser in a specific directory
https://developer.mozilla.org/en-US/docs/Web/API/Window/showOpenFilePicker

Here's some interesting documentation https://developer.chrome.com/docs/capabilities/web-apis/file-system-access


Initial integrate our L0 Llama impl into ipex-llm

Description

1. Why the change?

2. User API changes
See pipeline llama.py example.
3. Summary of the change

4. How to test?

[ ] Unit test: Please manually trigger the PR Validation here by inputting the PR number (e.g., 1234). And paste your action link here once it has been successfully finished.
[x] Application test

PR validation is passed: https://github.com/intel-analytics/ipex-llm-workflow/actions/runs/11478332409


Add flag for isBundled

I think it would just be a matter of allowing on the commandline a parameter for whether the assessment type is bundled such as -IsBundled 1, then when you call the API to Start Scans you supply that value for isBundledAssessment.
Any logic that checks whether it really is bundled should be handled by the API. The default should be -IsBundled 0 in case the intent is to start a Static Premium without ever running a dynamic scan.

I have a pull request that will be coming for this, once I also have a fix for issue #6 (as this won't work until that's corrected as well)


popup is going back

my popup is going back of slider which i want it to be front

I had a similar problem and solved it by setting a higher z-index value on the first element inside the parent. Not ideal, but a work around.


Update installing-core-linux.rst

Updated references to the latest version and fixed a typo in the run command. Issue #44

Thanks! RC1 hasn't shipped yet. Love to take the change, but want to stick to stable releases.
Only beta 7 has shipped: https://github.com/aspnet/Home/releases. Beta 8 should ship next week.

Understood and thanks. Trying to get my feet wet with some low hanging fruit.  I'll resubmit when beta 8 comes out.

Sweet. Looking forward to it.  Appreciate the help.


[BACKUP] Added support for Customer Managed Keys for Recovery Services Vault

Description

The purpose of this PR was to introduce Customer Managed keys feature in Azure CLI. The following commandlets were created:

vault update – It is for setting the managed identities for Recovery service vaults. It has following parameters:

identity_type {userassigned/systemassigned/none}
identity_id {identityId}
remove_system_assigned
remove_user_assigned
resource_group
vault_name

vault encryption update – It is to update the encryption settings of Recovery service vault. It has following parameters:

encryption_key_id
identity_id
use_systemassigned_identity
infrastructure_encryption
resource_group
vault_name

vault encryption show – It is to show details of encryption properties of Recovery service vault. It has following parameters:

resource_group
vault_name

Restore-disk – Added disk-encryption-set-id parameter in restore_disks command for encrypting restored data with it.

Apart from the code for above features, the PR also contains the recordings of the required tests.
Testing Guide

Identity commands
Addition of Identities

az backup vault update --resource-group {rg} --vault-name {vault} --identity-type systemassigned
az backup vault update --resource-group {rg} --vault-name {vault} --identity-type userassigned --identity-id <list_of_identity_id[]>

Removal of Identities

az backup vault update --resource-group {rg} --vault-name {vault} --remove-system-assigned
az backup vault update --resource-group {rg} --vault-name {vault} --remove-user-assigned --identity-id <list_of_identity_id[]>

Clear all identities assigned

az backup vault update --resource-group {rg} --vault-name {vault} --identity-type none

Encryption commands
Set encryption

az backup vault encryption update --resource-group {rg} --vault-name {vault} --encryption-key-id {encryptionkeyId} [--identity-id {identityId}|--use-system-assigned] [--infrastructure-encryption {Enabled, Disabled}]

Get encryption

az backup vault encryption show --resource-group {rg} --vault-name {vault}

Restore command

az backup restore restore-disks --storage-account {storageaccount} --container-name {container_name} --item-name {item_name} --resource-group {rg} --rp-name {rp} --target-resource-group {target_rg} --vault-name {vault} --disk-encryption-set-id {des} --restore-to-staging-storage-account true

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

BACKUP


Update misleading certificate in docs

One of the issues we had is that we were trying to decrypt the token with Merchant Identity Certificate, but it failed because we were using the wrong keys.
If I understood correctly, Merchant Identity Certificate is only for web applications (SSL I guess), once we used the Payment Processing Certificate we could decode the token successfully.

If this changes are fine, I'll update the php script to use the new file names.

Thanks @gonzalovilaseca Feel free to take on some of #8 which ties into this if you'd like.

Will have a look, thxs.

@gonzalovilaseca hello sir, I am still having trouble with the certificates and would love some help. If you are available, could you please email me at<EMAIL_ADDRESS>

See 3bd53f1 Thanks @gonzalovilaseca


Creating skills in Skill Group via the copy skill button does not cause Go Live to be enabled

Create a Skill Group.
Create the first skill via Add Skill to Group
Create the second skill via copy button on skill created in step 1
Go Live button is not enabled

:+1:


Cannot use the turbo login command on a headless server

What version of Turborepo are you using?
1.0.24
What package manager are you using / does the bug impact?
Yarn v1
What operating system are you using?
Linux
Describe the Bug
When I run turbo login I get an error saying exec: "xdg-open": executable file not found in $PATH. I am on a headless system and therefore there is no way for a browser to automatically open.
Expected Behavior
The login to provide a hyperlink to login to Vercel's Turborepo instead of crashing
To Reproduce
Start a headless Linux server (I use Ubuntu 20.04), and run turbo login
bam
that should do it

Closing as Tom has detailed how to provide credentials from CI.


ID-826 convert user.js to typescript

Jira Ticket: https://broadworkbench.atlassian.net/browse/ID-826
Summary of changes:

Converted User.js -> User.ts
Added response types for API calls to Sam, Orch, Bond, and ECM

Why

Will allow modules that integrate with the Users functions to be typed which will allow for more readability and maintainability in the codebase.

Changes from #4283 also show up in the diff here. Would it be possible to either rebase this on dev once that PR merges or change the base branch of this PR to the other PR's branch?
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/changing-the-base-branch-of-a-pull-request

Yep, my plan was to rebase once the merge went through 👍


Support to remove disable over-zoom effect

Is there any way we could remove the over zoom effect (i.e. disabling the ability to zoom out more than the image dimensions)?

Not at the moment. The ZoomSpec#preventOverOrUnderZoom setting might sound like it controls this behavior, but it actually manages the rubber banding effect. I'm considering changing it to an enum to support this.

This is what I meant in my issue

Not at the moment. The ZoomSpec#preventOverOrUnderZoom setting might sound like it controls this behavior, but it actually manages the rubber banding effect.

Ah yes that's right, I tried passing a ZoomSpec to rememberZoomableState with preventOverOrUnderZoom parameter set to true, but that doesn't seem to help with this.

I'm considering changing it to an enum to support this.

Perfect sounds great!
Thanks a lot for developing and maintaining this great and super-helpful library!

@MHShetty What's your usecase btw?

It's mainly to just have the overzoom and under zoom effect disabled while showing images in the gallery of our camera app. Our previous XML code didn't have the fancy zoom effect so we are trying to have something similar in our new compose code.


Removed usage of sudo from test-raf.sh

Fixed the problem with usage of sudo in tests, which is bad practise.
Closes #179

@zaqbez39me still an error :(

@yegor256 Actually, I do not set the date anywhere in the updated version. However, the error occurs.

@zaqbez39me it's weird. I merged. Let's see how GHA behaves now.

@yegor256 I suppose GHA behaves like this because of caching


Fix QueryTrail walk failing on snake_case fields

Long story short, juniper turns snake_case names in a schema (e.g.
app_users) into mixedCase (e.g. appUsers). This is currently not
mirrored by the generated QueryTrail code, resulting in the field walk
methods always returning None even though the field was actually
requested.
The relevant piece of code in juniper: https://github.com/graphql-rust/juniper/blob/master/juniper_codegen/src/derive_object.rs#L48
As the codegen is already using heck, the solution is just to always
convert the field name to mixedCase like juniper.
(It's a little confusing, because hecks camel case is actually pascal
case, and the mixed case is what's traditionally called pascal case)

Sounds totally plausible however I spent a bit of time looking into this and was not able to reproduce the bug in a test. Here is what I tried a04dd643be7f3.

I'll try to get a minimal example that reproduces the issue up and running tomorrow.

So here's a test case that showcases the basic issue i encountered: https://github.com/kiljacken/jfs-snake-case-example
I'm having trouble getting it to compile with the fixed branch right now, but it should illustrate the issue as is.

Ah yeah now I'm following. I totally see the bug if you use snake_case in your schema. I actually thought that was against the GraphQL spec but turns out it isn't.
I'll merge this later or tomorrow.

Thanks for finding and fixing this!

I just released fix in 0.2.3


Add a page detailing a vanilla "Stackable Demo Cluster"

We currently do not give a lot of guidance on how beefy clusters should be that are used to deploy our demos.
One notable exception is the Lakehouse demo that has a disclaimer with a few details.
It can be very frustrating for users to spin up a cluster, deploy the demo and notice that nothing works due to memory limitations or similar things.
Can we maybe come up with one shared cluster definition like 5 nodes, 8 cores, 20 gig ram or something like that, which we describe on an extra page along with proposed instance types on the Hyperscalers, and then we just refer to that page from all demos.

Is this similar to what you were imagining? https://github.com/stackabletech/documentation/issues/247


Relation definition format change?

The User class like below
class User extends ActiveRecord implements IdentityInterface

And then in one of my model, I add below relation.
class ApprovalTree extends \kartik\tree\models\Tree
{
...
    public function getUser()
    {
        return $this->hasOne(User::className(), ['id' => 'user_id'])->one();
    }
...
}

Then I can use below code to get username
$userNode->getUser()->username

However, there will pop up issue if I remove "->One()" from relation code.
As I remember, the relation definition no need "->one()", right?  (Even according latest online guide)
Any change for the relation definition recently? Or it is right since my model derive from one extension/module.
Btw, I use v2.0.3 dev version.

You dont need ->one()

I think so too. However, my yii2 will show error if I remove ->one().

http://www.yiiframework.com/doc-2.0/guide-db-active-record.html#accessing-relational-data
If you define realtion as getUser() you may access it as $model->user.

moved the rather hidden note a bit to be more visible and added an example code:
https://github.com/yiisoft/yii2/commit/b4062ab246595a01c9e0de32b963ac04ca5951a6#diff-e9e5f516d52124c69f3ca4a0374943d3R767


RANDOM UNPREDICTIBLE CRASH

My modpack has quite a few mods but your mod seemed to be working just fine until I opened the Morph Menu GUI for a second time.
CRASH FILE
https://pastebin.com/B3hTrSsH

I'm working on it :+1:

I was able to identify the problem. This crash occurs when you press the key to favourite a morph whilst having the red X selected in the morph menu.
In version 1.18.2-1.0.2, this would simply write a message to stdout (meaning that a message would appear in the log), but this used System.out.println, which I had tried to get rid off in this new release in favour of using Log4J. However, in this particular case, instead of logging this, I simply instructed the mod to throw an error (because I was not aware that this edge case could actually happen).
Anyways, I have fixed this in commit dab5919a59594c50e02a3d01590c6247b65e4802. I will release a quickfix soon.
Thanks for your help :)

Awesome! thank you :)


TrueNas Scale -No Connection to server

Hello,
I'm running the server following the docker-compose instructions in this guide. The system is True Nas Scale. I've managed to install everything and get the server up and running (I see the Game Server Connected message). Trying to connect to the server through the local ip of the nas (<IP_ADDRESS>:2457) it doesn't appear neither through steam server browser nor on valheim connect screen, ends up in failed to connect (in-game) or not responding (from steam).
I can open the nas webpage, so port 80 is fine. Listening on port 2456 on my nas with  "tcpdump -i enp3s0 port '(2456 or 2457)' " I see the packets from my pc reaching the nas server.
(from steam server browser)
16:35:24.486222 IP PC.55538 > unknown305a3a817080.2457: UDP, length 25
(from valheim client)
16:37:19.638687 IP PC.58889 > unknown305a3a817080.2457: UDP, length 512
Any suggestions? I'm not sure what the problem is. Thanks in advance
valheim-log.txt

Docker PS File
DockerPS.txt

After a lot of trying and reading around, I've found what the problem was. For anyone using TrueNas Scale for hosting this. The way TrueNas uses docker is kinda different, it converts them to kubernetes or something, I'm not very familiar with this so I can't explain in detail.

If trying to install valheim through docker-compose app on the catalog list
-> in Global Pod Options (Advanced) menu before installing it, check the Expert-Pod Options and then the Host Networking. This makes your port accessable through lan.
--> I didn't get it to work outside of my router for some reason, the ports are forwarded correctly.

If installing valheim through "Launch Docker Image" option in Applications menu
you have two options:

either forward the 2456 ip to port > 9000, for example what I did was I made it to 9456. You do in the Port Forwarding option. Container Port ist 2456 and the Node Port 9456 (could also be 9001, 9002... your choice) and ofcourse set the protocol to UDP Protocol. Do the same for the 2457 Port.
OR in the Networking tab you set the Host Interface to your ethernet interface you're using. then in IPAM Type you choose either a static or dynamic ip, static makes more sense and set it to whatever you want, for example <IP_ADDRESS>. This IP will be only for your valheim server. But then the ports used will be the default 2456+2457
===> in both cases you have to set up the Container Environment Variables, atleast the server_pass and the server_public.

After all it worked on LAN. Portforwarding the ports on my router didn't give access to my friends, I don't know why, help here would be nice if anyone has any suggestions.
Also I have a problem, when restarting my nas, it isn't just one pod created for some reason, one is the old one, which I can't access anymore through shell it says failed state and a new pod which run the valheim docker image once again, instead of using the old files from pervious build. How do I make the valheim server files saved in permament storage? Couldn't figure it out.

I only run into this on container startup in TrueNas and it appears that it wont allow connections till after the backup process is completed.

Pod Is an ephemeral kubernetes resource that have files inerithed by his deployment. If you want to override a file or some other things you must pass them to pod mounting a configmap as volume, declaring it into deployment manifest so at every pod restart, you will find your wanted configuration.


Fix object has no attribute 'venID'

Fix Exception with missed 'venID' described in https://github.com/VOLTTRON/volttron-applications/issues/65

@rcalvert-cpi @craig8 Sorry for delay for such small pull request.
I've tested this change and it works


Netflix Website Clone

Project Title:- Netflix Website Clone
Project Description:-
This project is a clone of the Netflix website. It is built using HTML, CSS. The project includes the following features:
A homepage with a search bar and a call to action button to sign up for a free trial.
A section about Netflix Originals.
A section with frequently asked questions.
A footer with links to important pages on the website.
Doc Domain

[✅] Solidity , Html, Css , JavaScript , Web3.js / Ether.js
[] Solidity , ReactJs /NextJs , Web3.js /Ether.js
[] Solidity , Python3 , Web3.py
[] Other , feel free to mention below

Are you contributing under any open-source program ?
yes , SSOC'23

Hey! @Vikash-8090-Yadav @shubhisingh184
Please assign this task to me under ssoc'23.

@NupurHardiya assigned to you. Start working on it.


Dash - Terraform Provider does not show up in the extension

Dash

Author: @RSO

Description
The Providers do not show up when using the Dash search extension, it only seems to show Resources and Data Sources.
Steps To Reproduce

Download the Terraform docset
Search using Raycast for Azure Provider

Doing this same query for Azure Provider within Dash itself yields the below result:

The current behavior
The Provider will not show up in Raycast
The expected behavior
The Provider should show up in Raycast
Raycast version
Version: v1.29.0

Thanks for the report! I replicated the error and noticed that everything after the space got ignored in Dash, this lead me to realize that we didn't quote the query string when we passed it to the Dash shell script. This got fixed in ba34101!


error deploy app in azure

Repro steps:

Action: appService.Deploy
Error type: Error
Error Message: "janas-webapp" is connected to a GitHub repository. Push to GitHub repository to deploy.
Version: 0.14.0
OS: darwin
Product: Visual Studio Code
Product Version: 1.34.0
Language: en

Call Stack
Object.<anonymous> extension.bundle.js:262:3454929
module.exports.n extension.bundle.js:262:3453562
module.exports.n extension.bundle.js:262:3453339
module.exports.t.deploy.i.window.withProgress extension.bundle.js:262:3454790
c._withProgress extensionHostProcess.js:448:323
c.withProgress extensionHostProcess.js:448:77
Object.withProgress extensionHostProcess.js:715:635
Object.<anonymous> extension.bundle.js:262:3454633
o extension.bundle.js:262:3453395
process._tickCallback next_tick.js:68:7

Hi @gserra75 If your app is connected to a GitHub repo, you should make a push to that branch.  It will trigger a build on your web app and continuously integrate.


Rename symbol fails to consider all components of a method call

Input
→ View on sorbet.run
# typed: true
class A
  def foo; end
end
class B
  def foo; end
end

extend T::Sig
sig {params(x: T.any(A, B)).void}
def example(x)
  x.foo
#   ^ apply-rename: [A] newName: bar
end

Observed output
Making the indicated rename request in sorbet.run only applies the rename to the first component of the dispatch result.
Namely, it only renames A#foo → A#bar, and neglects the fact that the LSP query returned that the method call could have also been to B#foo.
Expected behavior

Either:

We should rename each method in the dispatch result, or
We should detect the case where there are multiple dispatch results, and abort (better than doing the wrong thing).

cc @soam-stripe

Thanks @jez, thanks for filing this! Yes, union types are currently not handled correctly, I'm planning to address this in upcoming changes.


Use proper class loading mechanism to find classes during interop validation

Description:
Refering to https://github.com/ballerina-platform/ballerina-lang/blob/v1.0.0/stdlib/jvm/src/main/java/org/ballerinalang/nativeimpl/jvm/interop/JInterop.java#L341
Here the class is loaded using the default class loader. However assume a scenario where a module like ballerina-http is using java interop, and:

Current ballerina distribution uses some java module-x v1.0.
module-x v1.0 has a Foo class with method bar().
User writes his own module by importing module-x v2.0, which has have a new method baz() in the Foo class, and it is used for java-interop purposes.
Now during the compilation of user's module, there are two classes with the same name Foo in the java class path. One is coming from the module-x v1.0 which is inside ballerina distribution and a another coming from the user's imported module-x v2.0.
The java-interop validation may fail depending on which class is picked by the class loader.
Hence need to have our own class loader, which will load the classes in a proper predefined manner.

Currently all standard library modules faces this issue when using interop. Because theres an older version of stdlib that gets packed inside the bootstrapped ballerina distribution. And the latest version is coming from the sources. Resulting two version of the same classes to be available at the same time during compilation.
Versions:
V.1.0.0

Fixed with #19601


New branch

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: torrid-fish:x: Kuan-Ting WuKuan-Ting Wu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Ensure goenv and go play together well

Description
This PR seeks to address several issues surrounding the go and goenv plugins.
The nature of goenv allows for a situation where the initial shell does not
point to a working go binary, and instead at the included shim script. The
result is that one must run reload after moving to a project directory with a
version file, however in doing so, PATH is updated, requiring they exit the
shell or risk lookup collisions and unexpected behavior.
This is solved by using preexec to check the version before changing
directories and restarting the shell if the version has changed. The exec
pattern is copied from _bash-it-restart, but is edited to support this specific
use case.
Additionally, tests have been updated and these are now being linted.
How Has This Been Tested?
manually and locally. opening up for discussion while I continue such testing.
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] If my change requires a change to the documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] If I have added a new file, I also added it to clean_files.txt and formatted it using lint_clean_files.sh.
[x] I have added tests to cover my changes, and all the new and existing tests pass.

@NoahGorny Okay, sorry about that. For real this time :)

I'd like to look into the composure issues before merging this. hold on to your butts


bug: look like to watch same address when assign to another variable from sub-slice.

What version of Go are you using (go version)?

$ go version
go version go1.19.3 darwin/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
intel MacOS Monterey version 12.6, yes I use go env.
go env Output
$ go env
GO111MODULE="off"
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/tomoyuki/Library/Caches/go-build"
GOENV="/Users/tomoyuki/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/tomoyuki/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/tomoyuki/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.19.3"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/wz/r55pw77x0_1dv15r4w3wtds00000gn/T/go-build1838909510=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I assign y slice variable from x sub-slice.
I try to append 30 to y variable. I expect to append y variable only. but append y and x.
perhaps, y and x is look like watch some address.
In this case, should be pass by value, but it's call by pointer.
package main

import "fmt"

func main() {
	x := []int{1, 2, 3, 4}
	y := x[:2]

	fmt.Println("x:", x) // x: [1 2 3 4]
	fmt.Println("y:", y) // y: [1 2]

	y = append(y, 30)
	fmt.Println("x:", x) // x: [1 2 30 4] ← I think x should be [1 2 3 4].
	fmt.Println("y:", y) // y: [1 2 30]
}

play : https://go.dev/play/p/HWwI5WCjDXm
What did you expect to see?
Of course, I know how to fix this bug, should use copy.
package main

import "fmt"

func main() {
	x := []int{1, 2, 3, 4}
	y := make([]int, len(x[:2]))
	copy(y, x[:2])

	fmt.Println("x:", x) // x: [1 2 3 4]
	fmt.Println("y:", y) // y: [1 2]

	y = append(y, 30)
	fmt.Println("x:", x) // x: [1 2 3 4] ← no problem
	fmt.Println("y:", y) // y: [1 2 30]
}

play : https://go.dev/play/p/IayY-PdoM0l

That is how the language works.  Please see https://go.dev/blog/slices-intro.


Depend on serverengine <2.0.0

Fixes #1195
(Edited on github, so no real testing.)

Apparently the 0.14 branch was the wrong place to open this PR (which I wrongly assumed to equivalent to the 0.14.2 tag at HEAD).


Shorten method bodies

See googleapis/gax-python#104

We should ensure compatibility with googleapis/gax-python#108

cc: @garrettjonesgoogle

Sorry @geigerj. I ran out of time to review this, due to working on the release of the logging API. I skimmed over it and nothing jumps out as being a problem, but I didn't do a proper review.
Can @garrettjonesgoogle review this instead? Thanks.

No problem, I'll switch out the assignees.

Thanks, Jacob.

Just one missing comma that needs to be fixed - otherwise LGTM.

LGTM


How to install an app from the Windows Store via the command line?

[Enter feedback here]
This article is about sideloading apps. While it's cool and all, it's not very helpful for apps on the store.
I just want to make a script for my first time setup so I can have a bunch of apps easily accessible without having to click all over the place and wait huge delays just to install apps.
So, how do we install apps from the store via command line?

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

ID: e9757ffa-ccdb-e9f3-3f74-6f147dd04066
Version Independent ID: 15b5e687-f04d-d893-81fe-8fc86b63fe2d
Content: Packaging UWP apps - MSIX
Content Source: msix-src/package/packaging-uwp-apps.md
Product: msix
GitHub Login: @dianmsft
Microsoft Alias: diahar

Hello, the ability to install apps from the store via the command line is currently not supported.

Hello, the ability to install apps from the store via the command line is currently not supported.

Any plans for future?

@tesar-tech Its an item in our backlog.  We do not have a specific date.

Is there a publicly available issue tracker with this item so I can subscribe to it?


Example OData Query fails in Power BI

When running the example on how to expand Pipeline information getting a Power BI error:
DataSource.Error: OData: Request failed: The remote server returned an error: (400) Bad Request. (VS403483: The query specified in the URI is not valid: VS403489: The Analytics Service doesn't support key or property navigation like WorkItems(Id) or WorkItem(Id)/AssignedTo. If you getting that error in PowerBI, please, rewrite your query to avoid incorrect folding that causes N+1 problem..) Details:     DataSourceKind=OData     DataSourcePath=https://analytics.dev.azure.com/{organizationName}/{Project Name}/_odata/v3.0-preview/PipelineRuns(334448)/Pipeline     Url=https://analytics.dev.azure.com/{organizationName}/{Project Name}/_odata/v3.0-preview/PipelineRuns(334448)/Pipeline
Based on some research it might be an issue with how the OData is structured.
I get data back running:
https://analytics.dev.azure.com/{organizationName}/{Project Name}/_odata/v3.0-preview/PipelineRuns?%20
the error occurs when expanding the PipelineRuns Object.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a08cdff9-2c5f-1df2-87ba-f2c4606effc3
Version Independent ID: eaab2889-466c-de7c-def6-6e3b13026b4e
Content: Pipeline outcome summary for all pipelines sample report - Azure DevOps
Content Source: docs/report/powerbi/sample-pipelines-allpipelines.md
Product: devops
Technology: devops-analytics
GitHub Login: @KathrynEE
Microsoft Alias: kaghai

Hi @JFolberth - This could be a product issue. To make sure it gets in front of the right people, please submit your bug here.
I'm going to keep this issue open for now, since you came across this running an example. We may need to update it or work with the Power BI team.
Thanks -

@TimShererWithAquent I have submitted https://developercommunity.visualstudio.com/t/Power-BI-OData-connection-to-Azure-DevOp/1479839
I always am hesitant as I never get any responses from Developer Community and honestly what I do get back is months after the fact so appreciate any assistance you can provide for visibility.

Hey @TimShererWithAquent am I able to get any help on this item? Developer Community looks to be a bit of a dead end as they've been unresponsive.
Thanks

@JFolberth - On this board, we maintain the docs, not support the product. There's a few resources for support if the dev community hasn't panned out:

Azure DevOps Services
Azure DevOps Support Bot
Azure DevOps on Stack Overflow

Extremely sorry for the late response @JFolberth .
You can get the details for a Pipeline Run using the following:
https://analytics.dev.azure.com/{organizationName}/{projectName}/_odata/v3.0-preview/PipelineRuns?$apply=filter(PipelineRunId eq {pipelineRunId})
This is just normal odata syntax. If you wish to see the schema for PipelineRun, or any other entity you can ping
https://analytics.dev.azure.com/{organizationName}/{projectName}/_odata/v3.0-preview/$metadata
PipelineUrl is not an attribute we support for PipelineRun.
Regarding the data being returned when you pinged:
https://analytics.dev.azure.com/{organizationName}/{Project Name}/_odata/v3.0-preview/PipelineRuns?%20
It is likely sending back all the data (for many runs) in pages of 10000

@JFolberth - Query given in above solution is supported odata format.
So please use this URL:
https://analytics.dev.azure.com/{organizationName}/{projectName}/_odata/v3.0-preview/PipelineRuns?$apply=filter(PipelineRunId eq {pipelineRunId})

@JFolberth - Article has been updated with the correct syntax. Closing this issue.


Added exercise set 10.

This will add exercise 10.
Binder: https://mybinder.org/v2/gh/andersle/chemometrics/add_exercise_10

LGTM, merging


conf continue-on-error: true in application.yaml absent in java compreface-api cause microservice crashing on deployment

On admin java project, you added in application.yaml this properties  " continue-on-error: true", and that's great because we have a specific databse configuration (come from zalando) and they have in pga_hba.conf one line with reject for non ssl host, as the admin api have thos conf it is working well, but as the api java application does not have this configuration it make crashing the API app.
Could you harmonize the application.yaml for both of API and add in the second one "continue-on-error: true" ?
It could be great
thank's for your great work

Hello there! If the property does not break anything, we will add it in the next release.

Are you sure the problem is in the missing "continue-on-error" property?
Remove the "continue-on-error" property for the Admin application, and then run the application. If an exception occurs, please share it with me.

Hello,
yes we try with this parameter and without, if you take zalando docker postgres for example admin is working but not api
the cause is because there is in the priority list trustssl wich make craching applications not using https...

Hello there, could you please send me a link to that docker image?

https://github.com/zalando/postgres-operator

 - hostssl all             all                all                md5
  - host    all             all                all                md5

for example with teh parameter finally we go on last line in conf, but if there is not this parameter api or admin crash on hostssl line because there is no ssl between spring boot and Database

Thanks, I will check, but for now, could you please try to add the following to the DB URL: sslMode=require.
An example: jdbc:postgresql://localhost:5432/mydb?sslMode=require

we put thos on api continue-on-error
but we can try, you think it will fix it ? because there is no ssl on both server right ?

I'm not sure if it fixes your problem, but at least you should try.

Could you please send me stacktraces from admin and api after an error occurred?

it is complicated as the code is in production and all initialization was really hard to make (and we overrided the elm) and i dont find the exception in our history :/


SplitButton doesn't open when pressing the "Space" key or the "Enter" key

Describe the bug

The SplitButton doesn't open when pressing these keys. This is an accessibility issue, as there's no other way to open it, other than by clicking on it.
Steps to reproduce the bug

Steps to reproduce the behavior:

Create a SplitButton
Tab to focus it.
Press space or enter.
The splitbutton doesn't open.

Expected behavior

Expected:
4. The splitbutton should open, just like a DropDownButton
Screenshots

Version Info

NuGet package version:
Microsoft.UI.Xaml.2.6.0-prerelease.210315002

This is by design, the space and enter key invoke the primary button the chevron is invokable via Alt+Down


[Feature-13429][Remote Logging] Add support for writing task logs to Google Cloud Storage

Purpose of the pull request

close: #13429
A sub-task of #13017

Brief change log

Add support for writing task logs to Google Cloud Storage
Fix the log path bug since the log full path has changed in #13673

Verify this pull request
manually test and UT

run a shell task

check log file on GCS

Codecov Report

Merging #13777 (661d3db) into dev (69e7449) will decrease coverage by 0.17%.
The diff coverage is 7.24%.

:exclamation: Current head 661d3db differs from pull request most recent head edbcc17. Consider uploading reports for the commit edbcc17 to get more accurate results

@@             Coverage Diff              @@
##                dev   #13777      +/-   ##
============================================
- Coverage     39.02%   38.85%   -0.17%
+ Complexity     4450     4424      -26
============================================
  Files          1133     1135       +2
  Lines         42353    42345       -8
  Branches       4789     4777      -12
============================================
- Hits          16528    16453      -75
- Misses        23999    24067      +68
+ Partials       1826     1825       -1

Impacted Files
Coverage Δ

...e/dolphinscheduler/common/constants/Constants.java
75.00% <ø> (ø)

...heduler/common/log/remote/GcsRemoteLogHandler.java
0.00% <0.00%> (ø)

...ler/common/log/remote/RemoteLogHandlerFactory.java
0.00% <0.00%> (ø)

...apache/dolphinscheduler/common/utils/LogUtils.java
25.00% <25.00%> (ø)

...hinscheduler/common/log/remote/RemoteLogUtils.java
25.00% <100.00%> (+4.31%)
:arrow_up:

... and 54 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

 Hi @rickchengx please check the fail CI case.

Hi, @caishunfeng , thanks a lot for your comment. I've update the pom

Hi, @caishunfeng , could you please help review this again?


[Feat] Stroke color for Slider value indicator

Consider a scenario where the background color and indicator's background color are the same.
Adding a stroke color to the value indicator would be a valuable for the following reasons:

Visual Clarity: It would allow developers to make the value indicator stand out more against the background, making it easier for users to notice.
Customization: It would provide more flexibility in customizing the appearance of the sliding widget, allowing developers to match the design requirements of their apps.
Accessibility: Improved visual distinction can enhance the accessibility of the sliding widget for users with various needs.

List which issues are fixed by this PR. You must list at least one issue.
#135984
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] All existing and new tests are passing.

@838
Thanks for contributing!
Overall this looks great. One should've one test that checks the Slider using RectangularSliderValueIndicatorShape with the new valueIndicatorStrokeColor value indicator is actually rendered as expected.

@TahaTesser
Thanks for the feedback 😊.
The test is now added.

@838
Looks like the changes are failing some tests in the slider_theme_test.dart class

@TahaTesser
Should I also add a test to verify that 'overlappingShapeStrokeColor' behaves correctly when a valueIndicatorStrokeColor is provided for RangeSlider?

Yes, in the same test class

Goldens are approved!


Install git 2.11.0 instead of 2.1.4

This is the latest "comfortable" version we can get (specifically provided as a jessie backport). Low risk of regressions and such.

In quick testing, I found that 2.11.0 is new enough that it mitigates the problem with we are seeing with very slow fetches of repos with many files onto the site volume.


Click on merchants can kick players

Version
Sponge: 8.0.0
SpongeForge: 1.16.5-36.2.5-8.1.0-RC1352-universal
The bug
When a player clicks on a merchant, an error can be thrown and the player kicked off the server.
This bug typically happens when clicking on a TCG Trader from Pixelmon mod.
This bug was reported by #3871 and #3828.
Description
When the packet ClientboundMerchantOffersPacket is sent, the itemstacks and its NBTs are written into the byte buf. Sponge manipulates the NBTs to do server-side translation and for this, it needs to deserialize the name of the item and its lore.
The lines of the lore of the itemstacks sold by the TCG traders are a simple string and not a JSON in a string, so the deserialization throws an JsonParseException which kicks the player.
The fix
The fix is to support simple strings as lore lines. The are several ways to do it, and the solution of this PR is to use a  TextComponent if the deserialization fails.
An another solution would be to use fromJsonLenient instead of fromJson but I don't know if this can cause "corrupted" merchant because of JSON deserializable with SpongeForge but not with Minecraft.

How does the mod add these lores? Looking at the Vanilla's code, it also uses Component.Serializer.fromJson and if that fails it just removes the whole lore?


Fails to report errors with custom errorformat

Running reviewdog with jsonlint: ./node_modules/.bin/jsonlint -q -c $i | ./reviewdog -efm="%f: line %l, col %c, %m" -name="json lint" -reporter=github-check -fail-on-error
2020/07/12 06:03:26 [json lint] reported: https://github.com/mrchief/test-super-linter/runs/862206024 (conclusion=success)
2020/07/12 06:03:27 [json lint] reported: https://github.com/mrchief/test-super-linter/runs/862206064 (conclusion=success)
2020/07/12 06:03:28 [json lint] reported: https://github.com/mrchief/test-super-linter/runs/862206093 (conclusion=success)
2020/07/12 06:03:29 [json lint] reported: https://github.com/mrchief/test-super-linter/runs/862206105 (conclusion=success)
test/bad.json: line 1, col 1, found: 'INVALID' - expected: 'STRING', '}'.
2020/07/12 06:03:30 [json lint] reported: https://github.com/mrchief/test-super-linter/runs/862206112 (conclusion=success)
test/bad2.json: line 1, col 6, found: 'INVALID' - expected: 'STRING', 'NUMBER', 'NULL', 'TRUE', 'FALSE', '{', '['.
2020/07/12 06:03:31 [json lint] reported: https://github.com/mrchief/test-super-linter/runs/862206124 (conclusion=success)

No errors reported

Errorformat playground tells me my format is correct.
I checked with latest as well as nightly build but no luck.
Same issue happens with markdownlint
Run npx -q markdownlint-cli -c .rules/.markdownlint.json -p .rules/.markdownlintignore test/bad.md test/bad2.md \
  npx -q markdownlint-cli -c .rules/.markdownlint.json -p .rules/.markdownlintignore test/bad.md test/bad2.md \
    | ./reviewdog -efm="%f:%l:%c %m" -efm="%f:%l %m" -name="markdown lint" -reporter=github-check -fail-on-error
  shell: /bin/bash -e {0}
  env:
    REVIEWDOG_GITHUB_API_TOKEN: ***
test/bad.md:1 MD041/first-line-heading/first-line-h1 First line in file should be a top level heading [Context: "## Introduction"]
test/bad.md:6 MD001/heading-increment/header-increment Heading levels should only increment by one level at a time [Expected: h3; Actual: h4]
test/bad.md:6:1 MD019/no-multiple-space-atx Multiple spaces after hash on atx style heading [Context: "####  Instructions"]
test/bad.md:6:18 MD047/single-trailing-newline Files should end with a single newline character
test/bad2.md:1 MD041/first-line-heading/first-line-h1 First line in file should be a top level heading [Context: "## Introduction"]
test/bad2.md:6 MD001/heading-increment/header-increment Heading levels should only increment by one level at a time [Expected: h3; Actual: h4]
test/bad2.md:6:1 MD019/no-multiple-space-atx Multiple spaces after hash on atx style heading [Context: "####  Instructions"]
test/bad2.md:6:18 MD047/single-trailing-newline Files should end with a single newline character
2020/07/12 06:03:36 [markdown lint] reported: https://github.com/mrchief/test-super-linter/runs/862206621 (conclusion=success)

Errorformat playground

I had similar issue with docker lint but that got fixed with -fail-on-error
./hadolint $i | ./reviewdog -efm="%f:%l:%c %m" -name="docker lint" -reporter=github-check -fail-on-error

I'm using pylama and yamllint - both with custom formats and they seem to work just fine, even without -fail-on-error. So I'm not sure what's going on.

Here's my Github Actions script if that helps:
      - name: Install Reviewdog
        run: wget -nv -O - https://raw.githubusercontent.com/reviewdog/nightly/master/install.sh | sh -s -- -b .

      - name: Lint JSON Files
        if: always() && endsWith(steps.group_files.outputs.jsonFiles, '.json"]')
        run: |
          yarn add jsonlint
          jsonFiles=${{ join(fromJSON(steps.group_files.outputs.jsonFiles), ',') }}
          for i in ${jsonFiles//,/ }
          do
            ./node_modules/.bin/jsonlint -q -c $i | ./reviewdog -efm="%f: line %l, col %c, %m" -name="json lint" -reporter=github-check -fail-on-error
          done

      - name: Lint Markdown Files
        if: always() && endsWith(steps.group_files.outputs.markdowns, '.md"]')
        run: |
          npx -q markdownlint-cli -c .rules/.markdownlint.json -p .rules/.markdownlintignore ${{ join(fromJSON(steps.group_files.outputs.markdowns), ' ') }} \
            | ./reviewdog -efm="%f:%l:%c %m" -efm="%f:%l %m" -name="markdown lint" -reporter=github-check -fail-on-error

I'm guessing jsonlint and yamllint output results to stderr instead of stdout, so the results are not piped to reviwedog. Can you confirm?


2FA for login

Feature Request 2FA for login in UserS checkbox

We already offer this via our external authentication extension paired with oauth.
Rgds,
Neil Cresswell
On 21/12/2019, at 7:51 PM, Zip Code<EMAIL_ADDRESS>wrote:

Feature Request 2FA for login in UserS checkbox
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_3475-3Femail-5Fsource-3Dnotifications-26email-5Ftoken-3DAFQ2XFOSEGOHV3UDMUQTDCLQZW4GLA5CNFSM4J6FYKJ2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ICDPH2A&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=l0umyVe6ABw0vQDlS4nMVAQ3uEnHxQmtt0yOklbqZYs&s=60IdTcY6FcEV_3apcxJqRPeOQx1KoxAnEafH-0gpMLI&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AFQ2XFMK5UOQP5SJH2DKMF3QZW4GLANCNFSM4J6FYKJQ&d=DwMCaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=l0umyVe6ABw0vQDlS4nMVAQ3uEnHxQmtt0yOklbqZYs&s=L1DyOq3wdkv4Ta74bvoEofonbkJ-nZC8hKaWBOmVm9c&e=.

thanks took me a few days to find this as it was not in the documentation.

Closing, will continue discussion on PR opened against portainer-docs here


No climate entity created for Hombli TRV

Describe the bug
I just updated to the latest release 2024.3.2. I noticed the climate entities no longer were provided for my Hombli TRVs. The secondary entities are working as expected.
If I remove the device and re-add it, only 9 entities are created, not including the climate entity.
Please help!
To Reproduce
Get and configure a Hombli TRV. Add it to HA. Note there's no 'climate' entity for it
Expected behavior
The primary climate  entity should be present.
If the bug involves a device, then please include device diagnostics from
Settings / Devices & Services / Tuya Local (your device) / "1 device"
Preexisting device:
tuya_local-9c75467585768332fe17c087a58b3dde-Workshop Thermostat-f07a583d837948c81d04072bded23fc5.json.txt
Newly added device:
tuya_local-3f261895b21e0e331f6b1132f2d92061-Bathroom Radiator-dd77c25fe08cf5e7306dcb389e7e4331.json.txt

The log may show some reason why HA has decided the climate entity is unavailable.  From the returned data, it seems like it should be all OK.
        {
          "disabled": false,
          "disabled_by": null,
          "entity_category": null,
          "device_class": null,
          "original_device_class": null,
          "icon": null,
          "unit_of_measurement": null,
          "state": {
            "entity_id": "climate.workshop_thermostat",
>>>            "state": "unavailable",
            "attributes": {
              "restored": true,
              "hvac_modes": [
                "off",
                "heat"
              ],
              "min_temp": 5.0,
              "max_temp": 30.0,
              "target_temp_step": 0.5,
              "preset_modes": [
                "away",
                "home"
              ],
              "supported_features": 17
            },
            "last_changed": "2024-04-24T21:00:39.078013+00:00",
            "last_updated": "2024-04-24T21:00:39.078013+00:00"
          }

I've updated to the latest HA docker image (seems I was using an older/non official version I started using when it was running on rasberry pi). That seems to have solved it!

If you are not installing through HACS, then the minimum HA version would not be enforced, and I guess that may have been the cause (probably due to the translation key that was added, which required HA 2024.2 or later)


Update Export-ActivityExplorerData.md

Goal is to provide close to a full script instead of single step which seems to be creating confusion currently

@chrisda  for your review


GACall field phaseset unset

currently using pysam tool to extract elements for GACall. (HtslibVariantSet)

Can we get some more details on this please @shajoezhu?

The pysam VCF/BCF code does not yet expose phase information.  As I
described in another thread, I am still considering how best to proceed:
https://github.com/ga4gh/server/issues/231#issuecomment-82643656
On Thu, Mar 19, 2015 at 8:33 AM, Jerome Kelleher<EMAIL_ADDRESS>wrote:

Can we get some more details on this please @shajoezhu
https://github.com/shajoezhu?
—
Reply to this email directly or view it on GitHub
https://github.com/ga4gh/server/issues/265#issuecomment-83610189.

We should fill this in ourselves in the mean time, so we have something reasonable. It's a bug if we're exporting a phaseset that's definitely wrong.

Can you point me to the GA4GH API spec for how phase is communicated?
On Fri, Mar 20, 2015 at 12:41 PM, Jerome Kelleher<EMAIL_ADDRESS>wrote:

We should fill this in ourselves in the mean time, so we have something
reasonable. It's a bug if we're exporting a phaseset that's definitely
wrong.
—
Reply to this email directly or view it on GitHub
https://github.com/ga4gh/server/issues/265#issuecomment-84064675.

I've never really understood what this means, but this is what we're going by:
http://ga4gh.org/documentation/api/v0.5.1/ga4gh_api.html#/schema/org.ga4gh.GACall

Fixed in #285


Consider adopting ruff

As a replacement for flake8:
https://github.com/charliermarsh/ruff
My personal experience has been great with it

awesome, thanks for the recommendation, I'll run some tests to see if we can adopt it without many changes!


[Docs] Integrations: Don't display "Components" section if no components are available

Current State
An empty Components section is displayed:

Example page - https://docs.meshery.io/extensibility/integrations/artifact-hub
This logic isn't working - https://github.com/meshery/meshery/blob/master/docs/_layouts/integration.html#L11
Desired State
If an integration doesn't have any components, don't show the component section.

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@vishalvivekm bug?

could i give a shot?


Easier spacing of HTML tags in sentences

Is there a way to get around using lots of ' to make sure HTML tags in sentences are spaced correctly?
This code:
strong Head's up!
'
' This is
strong a preview
'
' of the
'
strong= @landing_page.name
'
' landing page

Renders correctly:
Head's up! This is a preview of the  Foobar landing page
 (note the spacing before the  tags and after the  tags)
However, I would like to do this:
strong Head's up!
' This is
strong a preview
' of the
strong= @landing_page.name
' landing page

But that renders as
Head's up!This is a previewof the Foobarlanding page
 (the next word comes right after the  tag, without a space in between)

Hi! You can use strong<> to ensure spaces around tag
https://github.com/slim-template/slim#trailing-and-leading-whitespace--

Oh, nice, I never knew that existed. Thanks!


Update youtube_dl_gui.po Japanese

How about adding Japanese to the subtitle download?

It is possible to increase the translation rate (%) in https://www.transifex.com/yt-dlg/yt-dlg/language/ja_JP/ ?

Thanks for the translations 🙏🏾


make a new dev branch from the new layout branch to hold the converted project.

branch from old master to have a backup, then force newest branch over the master branch. work from there.


Enhance is_uuid validation and descriptions

UUID validation has been enhanced to support multiple versions (1,3,4,5) instead of just the previous UUID4 format. This update is reflected in code, comments, and README documentation. Updated test cases reflect the newly extended UUID validation.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.007%) to 94.418%

Totals

Change from base Build<PHONE_NUMBER>:
-0.007%

Covered Lines:
795

Relevant Lines:
842

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.007%) to 94.418%

Totals

Change from base Build<PHONE_NUMBER>:
-0.007%

Covered Lines:
795

Relevant Lines:
842

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.007%) to 94.418%

Totals

Change from base Build<PHONE_NUMBER>:
-0.007%

Covered Lines:
795

Relevant Lines:
842

💛  - Coveralls


Verify signatures on pull

Adds /etc/atomic/policy.json, and a command-line option to override (skopeo --policy some_path copy …), and enforces this policy in skopeo copy.
make install now installs default-policy.json to /etc/atomic/policy.json; the policy is set to insecureAcceptAnything for all images by default. (This may not be desirable long-term, but it is a reasonable transition strategy for now I think. The policy always uses the most specific matching scope, so it is easy to e.g. pin a key and require signatures for a specific repository namespace.)
Note that with merging this, ./skopeo will abort if the policy in /etc/atomic does not exist. This should not affect real deployments, but may be noticeable with local checkouts. A bit regrettable, but we do need to fail closed if a policy can’t be read.  (Use ./skopeo --policy default-policy.json … in a checkout.)
This depends on, and vendors in, unmerged https://github.com/containers/image/pull/49 and https://github.com/containers/image/pull/50 , I will revendor as necessary.
@lsm5 , note that adding the config file will affect packaging.
@rhatdan , any comments on the path /etc/atomic/policy.json?

 (5) Container image signing and verification with a single key

On Wed, Aug 10, 2016 at 04:03:54PM -0700, Miloslav Trmač wrote:

@lsm5 , note that adding the config file will affect packaging.

sure thing, thanks for the headsup
Lokesh
Freenode: lsm5
GPG: 0xC7C3A0DD

I like /etc/atomic/policy.json
Unsure about failing hard if not /etc/atomic exists though - I guess I have no strong opinion, perhaps just tell something useful to the user suffices
LGTM though, reviewed #50 as well

Unsure about failing hard if not /etc/atomic exists though - I guess I have no strong opinion, perhaps just tell something useful to the user suffices

$ ./skopeo copy dir:a dir:b
FATA[0000] Error loading verification policy: open /etc/atomic/policy.json: no such file or directory

Unsure about failing hard if not /etc/atomic exists though - I guess I have no strong opinion, perhaps just tell something useful to the user suffices
$ ./skopeo copy dir:a dir:b
FATA[0000] Error loading verification policy: open /etc/atomic/policy.json: no such file or directory

yup, saying I'm fine with that :)

Should the path be vendor neutral?  Or is this a path handed to container/images?

after meeting: /etc/container seems ok

The path comes from containers/image, so that various tools all use the same file.
To be precise, the path would be /etc/container/policy.json? (I can’t find any package owning /etc/container in F24, so I’m not sure where to look for other users of that directory and possible problems.)

Revendored to use /etc/containers/policy.json (pending re-review of https://github.com/containers/image/pull/49 ), and simplified getPolicyContext a little.

Just a nit, LGTM otherwise
Feel free to merge containers/image bit, re vendor here and merge here

Thanks for the review!
I have decided to drop the dependency on https://github.com/containers/image/pull/50 ; the cost is that the link in skopeo.1 will be pointing at a non-existent URL for a time, the benefit is that this makes https://github.com/containers/image/pull/50 a branch with no relationships and greatly simplifies the structure of outstanding branches.


Optimize ConsumeMessageOrderlyService enumerated types of comparison

Optimized performance

Which Issue(s) This PR Fixes

Optimized performance：
The equals of enumeration types can be replaced by ==, which improves performance and is safer at compile and run time.
see:

https://docs.oracle.com/javase/specs/jls/se9/html/jls-8.html#jls-8.9
https://stackoverflow.com/questions/1750435/comparing-java-enum-members-or-equals

Brief Description

How Did You Test This Change?
Just compare the pressure measurements with the previous ones。
Space formatting has been added

Codecov Report

Merging #7471 (2334e99) into develop (0a6ae46) will increase coverage by 0.33%.
Report is 21 commits behind head on develop.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop    #7471      +/-   ##
=============================================
+ Coverage      42.62%   42.96%   +0.33%
- Complexity      9462     9660     +198
=============================================
  Files           1150     1161      +11
  Lines          82750    83926    +1176
  Branches       10779    10899     +120
=============================================
+ Hits           35269    36055     +786
- Misses         43053    43389     +336
- Partials        4428     4482      +54

Files
Coverage Δ

...nt/impl/consumer/ConsumeMessageOrderlyService.java
49.46% <0.00%> (ø)

... and 93 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


PutObject throws error from AWS SDK

When trying to PutObject, I'm getting the following message from AWS SDK:
A header you provided implies functionality that is not implemented
Possibly related to https://github.com/aws/aws-sdk-go/issues/122
Any ideas?

@drgomesp

If you are using minio server and awk-sdk-go, please provide minio version, setup type you use and awk-sdk-go version.

If you are using minio-go with amazon s3, please use TraceOn() and provide the output.

@balamurugana thanks for the reply.
I'm using the latter. What should I pass onto TraceOn?

@drgomesp for minio-go tracing, refer http://docs.minio.io/docs/golang-client-api-reference#TraceOn

This is what I'm getting:
---------START-HTTP---------
PUT /mongodb/00010110.json HTTP/1.1
Host: pf-mongodb-backup.s3-ap-southeast-1.amazonaws.com
User-Agent: Minio (darwin; amd64) minio-go/2.0.3
Content-Length: 0
Authorization: #####
Content-Md5: 1B2M2Y8AsgTpgAmY7PhCfg==
Content-Type: application/octet-stream
Expect: 100-continue
X-Amz-Content-Sha256: UNSIGNED-PAYLOAD
X-Amz-Date: 20170330T094611Z
Accept-Encoding: gzip

HTTP/1.1 501 Not Implemented
Connection: close
Transfer-Encoding: chunked
Content-Type: application/xml
Date: Thu, 30 Mar 2017 09:46:11 GMT
Server: AmazonS3
X-Amz-Id-2: Tt7eks5WnICUgCHv96Gc5vNtoxjjhNwjnIEq2U5umrLKnXOlFhXGmB9cGWI2u1QA+AnNwwywkCA=
X-Amz-Request-Id: A568F217B5E2F1A2

14d
<?xml version="1.0" encoding="UTF-8"?>
<Error>NotImplemented<Message>A header you provided implies functionality that is not implemented</Message>Transfer-Encoding<RequestId>A568F217B5E2F1A2</RequestId><HostId>Tt7eks5WnICUgCHv96Gc5vNtoxjjhNwjnIEq2U5umrLKnXOlFhXGmB9cGWI2u1QA+AnNwwywkCA=</HostId></Error>
0
---------END-HTTP---------

@drgomesp, looks like the bug is fixed in the last master, can you test with it ?

Expect: 100-continue
HTTP/1.1 501 Not Implemented

This is because of the Expect header new S3 endpoints don't yet support this.

@drgomesp can you apply this PR locally and test out for us?  https://github.com/minio/minio-go/pull/640
Thanks.

Apparently the issue happened because the file was empty. It was a problem on my application level, but the error message was not very helpful.
Can we try adding a more helpful feedback?

We cannot server sent the error. We can't know what to say at this point in time. Because the error sent by server is arbitrary. So as a client we can't make sense of anything that went wrong.

@drgomesp, were you able to test with minio-go master ?

@drgomesp  I will go ahead and close this issue. Please do reopen if you think we have not addressed this correctly.

@deekoder no problem. As I said before, I think we need a proper message when throwing this error. Otherwise, it works fine.


termite: add patch to fix handling nix urls

(Builds on #32860 since patch is slightly different on v13 vs v12)
Motivation for this change
Nix-provided terminal shouldn't break on URL patterns commonly used in *.nix files!
This PR includes a patch addressing this.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Anecdotally: I've been using termite patched in this manner for months (tracking git, more or less) and I haven't experienced any problems.

Can you mention the pull request in the comment next to the patch?

Good catch, added mention of the PR in the nix expression and patch file.

@GrahamcOfBorg build termite

Note that v13 also adds support for fancy hyperlinks, see: https://gist.github.com/egmontkob/eb114294efbcd5adb1944c9f3cb5feda
@jtojnar check your config, maybe? I thought the default was clickable_url = true but might be worth setting explicitly to see if that helps.
Mouse-over should underline and then be clickable (using browser config, probably falls back to $BROWSER or somethiing?)

Okay, creating ~/.config/termite/config with the following content worked:
[options]
clickable_url = true

Thanks.


Feature request: auto queue next research if queue is empty

If the queue is empty, add the first unresearched item to the queue.
Implemented  either as a setting, button (similar to the requeue infinite technologies button), or on by default (since there is a pause button players can still decide to not have any research running.

I don't really like the idea of this, it's automating away all decisions.


Automatically pad colorbar and axes labels

It would be nice to generate automatic paddings for colorbar and axes labels

This has been partially fixed by #17. I haven't done the ylabel though


Update elixir

Update Elixir version and also use the mix formarter to format the code.

Sorry, these changes have already been made


Enable IRSA for cert-manager

Story
-As a cluster admin, I want cert-manager to use IRSA (IAM Roles for Service Accounts) instead of Kiam in order to improve the cluster's security.
Towards epic.
Background
cert-manager is using Kiam even though it has support for IRSA.
Links

Customer request;
AWS policy used by the customer.

The AWS policy used by the customer is available here.


DF Curved Walls: Nodes are obnoxiously big

Module
DF Curvy Walls v3.2.1
Describe the issue
The tool nodes (and the font size) are obnoxiously big. See screenshot: https://i.imgur.com/EMaoEBK.png
To Reproduce
Steps to reproduce the behavior:

Use one of the module's tools.
Have a hard time positioning things accurately due to the visual size of the tool handles.

Expected behavior
Tool handle nodes be as big as the wall nodes. And the font size to be around half the size
Screenshots (Optional)

I can maybe look at adding the ability to scale the handles and text size to a Small/Medium/Large.