fix(time): does not provide getTimezoneOffset

Signed-off-by: Sepush<EMAIL_ADDRESS>ref #2545
maybe the same case with https://github.com/marnusw/date-fns-tz/issues/133 it fixed in date-fns-tz 1.3.0

Nuxt3也有该错误，我还以为是特殊情况。感谢修复

依然报这个错误哦 https://codesandbox.io/s/woeq78


Enhancement: Combine planner plan get with planner plan details get

Same concept as issue #3257 & #3231
Here we will be combining the commands planner plan get and planner plan details get into a single command planner plan get. When we look at both output models for these commands, there will be one property we have to watch out for.
planner plan get > PlannerPlan Model:
 {
  "createdDateTime": "2021-12-12T14:00:50.4522129Z",
  "owner": "1e85a3de-c55c-49a3-8c10-5d0c64e8be0e",
  "title": "PnP Planner",
  "id": "xqQg5FS2LkCp935s-FIFm2QAFkHM",
  "createdBy": { },
  "container": { }
}

planner plan details get > PlannerPlanDetails Model:
{
  "id": "xqQg5FS2LkCp935s-FIFm2QAFkHM",
  "sharedWith": {  },
  "categoryDescriptions": {  }
}

So here I've simplified a few parts but you' ll see here that both models do return the same Id. This is in both cases just the task Id. With combining the two models we will need to check that we don't return the Id twice.

Should we just merge these 2 objects together? Or should we create a new property called "details" that contains the details of the task?
Option 1:
 {
  "createdDateTime": "2021-12-12T14:00:50.4522129Z",
  "owner": "1e85a3de-c55c-49a3-8c10-5d0c64e8be0e",
  "title": "PnP Planner",
  "id": "xqQg5FS2LkCp935s-FIFm2QAFkHM",
  "createdBy": { },
  "container": { },
  "sharedWith": {  },
  "categoryDescriptions": {  },
  "description": "lorem ipsum"
}

Option 2:
 {
  "createdDateTime": "2021-12-12T14:00:50.4522129Z",
  "owner": "1e85a3de-c55c-49a3-8c10-5d0c64e8be0e",
  "title": "PnP Planner",
  "id": "xqQg5FS2LkCp935s-FIFm2QAFkHM",
  "createdBy": { },
  "container": { },
   "details": {
      "sharedWith": {  },
      "categoryDescriptions": {  },
      "description": "lorem ipsum"
   }
}

Personally, I am a fan of option 1. This is because we also did it this way with issue #3257. This ensures that the users don't need to know the difference between plan details properties and the default plan properties.

I can take this one up. 😄


docs: add CheckboxCustomGroup demo

add CheckboxCustomGroup demo

Codecov Report
Base: 4.94% // Head: 4.94% // No change to project coverage :thumbsup:

Coverage data is based on head (4fc0e7a) compared to base (da4ac08).
Patch has no changes to coverable lines.

Additional details and impacted files
@@          Coverage Diff           @@
##           master    #566   +/-   ##
======================================
  Coverage    4.94%   4.94%
======================================
  Files         153     153
  Lines        3136    3136
  Branches      801     801
======================================
  Hits          155     155
  Misses       2598    2598
  Partials      383     383

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Function calling

Not sure if an Agent is fine or if it should be something else, but the idea is to implement the example here https://openai.com/blog/function-calling-and-other-api-updates
This work with the existing tools but you can create your own tools and pass them too. e.g.
class SayHi < Langchain::Tool::Base
  NAME = "say_hi"

  description <<~DESC
    Say hi to someone
  DESC

  SCHEMA_PARAMETERS = {
    name: {
      type: "string",
      description: "Name of the person to say hi"
    }
  }

  def execute(name:)
    puts "Hi, #{name}!"
  end
end

Working example:
irb(main):003:0> agent = Langchain::Agent::FunctionCallAgent.new(llm: llm, tools: [L
angchain::Tool::Calculator.new])
=>
#<Langchain::Agent::FunctionCallAgent:0x0000000109602cc8
...
irb(main):005:0> agent.run(question: "2 x 5")
=> 10.0
irb(main):006:0> agent.run(question: "what was the previous result?")
=> "The previous result was 10."

If this is something we want to add then I can add tests

@santib What are your thoughts on baking this into the Langchain::Conversation class?
For example we could add the tools: parameter:
Langchain::Conversation.new llm:, tools: [Langchain::Calculator.new]

The Conversation class could automatically reflect on all of the execute() methods within the Tools and generate the required functions JSON:
# something like... but could get more or less creative:
functions = generate_functions_json(
  parameters: Langchain::Tool::Calculator.method(:execute).parameters,
  description: Langchain::Tool::Calculator.method(:execute).description
)

@santib What are your thoughts on baking this into the Langchain::Conversation class?
For example we could add the tools: parameter:
Langchain::Conversation.new llm:, tools: [Langchain::Calculator.new, ...]

The Conversation class could automatically reflect on all of the execute() methods within the Tools and generate the required functions JSON:
# something like... but could get more or less creative:
functions = generate_functions_json(
  parameters: Langchain::Tool::Calculator.method(:execute).parameters,
  description: Langchain::Tool::Calculator.method(:execute).description
)

@andreibondarev I'm not sure it makes sense from a Langchain perspective, but that might not matter if we are going to diverge from the original idea.
Apart from that, if we add tools to conversations, will it only work with LLM's "function calling" feature (only available in OpenAI for now) or are we also going to do something more custom like it's done in https://github.com/andreibondarev/langchainrb/tree/main/lib/langchain/agent/react_agent/react_agent_prompt.yaml when it doesn't support "function calling" natively?

@santib Yeah, we'd need to do something custom for OpenAI.  Btw -- do you have an actual real use-case you'd like to solve here?

@santib Yeah, we'd need to do something custom for OpenAI. Btw -- do you have an actual real use-case you'd like to solve here?

Not for now, it's mostly trying to stay up to date with new tooling available so Ruby devs can compete with devs from other languages. Personally speaking, I don't care if it's langchain (I don't actually like their mental model) or any other AI framework, but we need to provide Ruby devs with powerful tools so they can smoothly augment their apps with AI.

function_call: and functions: are deprecated (https://platform.openai.com/docs/api-reference/chat/create#chat-create-function_call) but the tools: will be supported by the new Langchain::Assistant class: https://github.com/andreibondarev/langchainrb/pull/422


$ExternalIP should fetch IPv4 IP only

This is not for requesting support for new games/applications
To do this you should go to https://github.com/CubeCoders/AMPTemplates and first attempt to build a configuration yourself - otherwise you can request a template from this repo.
Feature Request
Feature Information:

The $ExternalIP var in the Generic module is used in some templates to automatically populate the IPv4 public address in configs or command lines. This has worked as expected until recently. Now the variable - perhaps related to the docker image updates - tends to fetch the IPv6 public address. This does not work for most (if not all) servers that rely on the var.
Please hardcode AMP so that it only fetches the IPv4 public IP for this var.
I confirm:

[x] that I have searched for an existing feature request matching the description.
[x] that my checkboxes above look like the one on the left (remove this line when done)

I guess a better solution would be to introduce a new variable $ExternalIPv4 and maybe also $ExternalIPv6

Not really. In most cases it is the IPv4 IP that is needed. And making sure that existing var returns the v4 address would mean that all existing templates and instances will continue to work and won't need updating
An additional var for v6 could be added if needed in the future

And when I say "continue to work", I mean work like they did previously. They are currently broken


KafkaTestBinder: Ensure error infrastructure beans are unique

See this comment: https://github.com/spring-cloud/spring-cloud-stream-binder-kafka/pull/198#discussion_r140346174

Closing this issue now as we are going to think about possibly revamping the test binder using the new SI test binder after the 3.0 release. If we end up doing that, this will be addressed as part of that larget issue.


Update model classes to use arraylias

Summary
Updates the model classes to use arraylias.
Files to update:

~generator_model.py~

~test_generator_model.py~

~hamiltonian_model.py~

~test_hamiltonian_model.py~

~lindblad_model.py~

~test_lindblad_model.py~

Interface changes:

copy method has been removed
Setters for everything other than signals and in_frame_basis are being removed. I had wanted to make these classes fully immutable but the ability to set these two properties is used quite a bit throughout the solver infrastructure and I don't want to additionally make a change of this scale at this time. (In any case the main mutability issue had to do with the operators, so for now it's okay to keep signals and in_frame_basis as mutable.)
Changed evaluation_mode to array_library

For LindbladModel, will also add a “vectorized” boolean

To use array_library=“scipy_sparse” or "jax_sparse", a user MUST pass this as array_library.

This is how it used to be before so it isn't such a big deal - the user will need to pass the arrays in dense format, then they will be converted to sparse format internally. This isn't desirable but I'm willing to accept it for now as it would only be an issue for extremely large models.

Other to do:

Add breaking release note for these changes

Details and comments
General notes:

I changed around the way that HamiltonianModel relies on the methods GeneratorModel, but the actual user-side behaviour has not changed (aside from the same changes made to GeneratorModel).

Test files that need to run before merge:

test.dynamics.models.test_lindblad_model
test.dynamics.models.test_hamiltonian_model
test.dynamics.models.test_generator_model
test.dynamics.models.rotating_frame
test.dynamics.models.operator_collections
test.dynamics.arraylias (whole folder)

I've added a release note as well.


Subsetting: SCORE study Household->longitude variable should not show a histogram filter

I know this is because it's a special kind of number variable.  Just flagging this up so we don't forget to handle it somehow.

What should we show?

@nkittur-uga @SheenaTomko I think we should not display latitudes & longitudes as terms in the variable tree, but these should be available in the download files. Do we need to obfuscate the lat / long terms for the SCORE studies?

I think the goal is to have them eventually display in a map that users can subset on. For now, Nupur and I agree with Danica that these should not display in the tree but should be available in the download file. We do not need to obfuscate this as they are village coordinates and belong to a controlled study. We also do not yet have a plan in place for how to obfuscate lat/long terms. That requires a lot more thought and planning.

It's OK if this does show up as a histogram, IMO.  There needs to be some minor client work to make this happen however.

@bobular, we are now hiding all lat/long (and all geohash) variables from the variableTree. Can I close this issue?

Yes, I'll close it.


got errros when compile arm code

I'm using Ubuntu 20
when I compile code in this section: https://0xinfection.github.io/reversing//pages/part-9-link-register.html
I got errors like this:

btw, there is a dead link here: https://0xinfection.github.io/reversing//pages/part-1-the-meaning-of-life.html
this link will lead me to a 404:https://github.com/mytechnotalent/hacking_c-_arm64

seems like the source code is not recognized by as

You are using a potential 64-bit version of the OS.  As I have said you need to search for commands that will be compatible.  OS evolve and the flags you pass to as will slightly deviate.  I can't keep up with all of the changes there are some things you will have to search around for but the code will compile.

seems like it is because I'm using intel CPU

You have to read all of the instructions this is an ARM course and I use a Raspberry PI.  Please take some time and read all of the instructions.

emmm... In fact, you didn't mention that in: https://0xinfection.github.io/reversing//pages/part-1-the-meaning-of-life.html
you mentioned it here:https://0xinfection.github.io/reversing//pages/part-1-the-meaning-of-life-part-2.html
so I didn't notice that at the beginning

anyway, thanks for your reply and your great tutorial


Add New Relic account on both staging and production environments

What does this PR do?
This PR will enable on both staging and production environments New Relic account
What are the relevant issues this PR belongs to

resolves #305

:tada: This PR is included in version 4.0.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update autorest version to prerelease 3.0.0-alpha.20240418.1.x

Triggered from pull request: https://github.com/Azure/autorest.csharp/pull/4185

Close in favor of #43872


Emojis appears in Top 20 Words

While in a call with someone, they checked their package and found some emojis in their Top 20 Words.
I'm not sure that it is a correct behavior since there's the top 20 emojis/emotes

Ah fair, I'm only filtering out the Discord emoji - should be an easy fix.

Fixed in https://github.com/davidbmaier/discord-recap/commit/89e2fd4252534b0268554aad93d4f5eb6bd8295f.
Should get filtered out now - you'll need to re-scan your package.
Bit annoying to test on my end but I think I got all edge cases.
Feel free to reopen if you can still reproduce it.


JDK9 Internal API references in cglib

Running on cglib 3.2.4 with Jigsaw b162 with --permit-illegal-access.
I'm seeing the following warning.
WARNING: Illegal access by net.sf.cglib.core.ReflectUtils$1 (file:gradle_cache/caches/modules-2/files-2.1/cglib/cglib-nodep/3.2.4/42c972dee25a436b32c6027e848287ff47ed6750/cglib-nodep-3.2.4.jar) to method java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain)
Breaking into this class is no longer allowed in JDK9.

I see this falls back to Unsafe in 3.2.5.  That's great!


Question: Why not just embrace terraform?

Is your feature request related to a problem? Please describe.
Hi All, Love the progress to more abstraction, its a great move. Personally as a long time user of both ARM and Terraform. What is the thinking behind not just embracing terraform?
Is there a document page about this, in terms of the added benefit of bicep over terraform as an abstraction?
I love the idea and keen to take it for a test run, just want to get a better understanding of the thinking behind it and where it would potentially accel when compared to terraform?

Please do not embrace Terraform!!!

Please do not embrace Terraform!!!

Why?
The readme for bicep, indicating it's not production ready and is experimental, as opposed to Terraform which is stable, and is highly expanded-upon to work with more than just Azure, and does it all very well, including workspace handling and is a well thought out infra-as-code model. Even Ansible does a pretty great job at automating cloud provisioning on Azure.
What separates bicep from the pack? What makes it unique? Good open-source communities/projects shouldn't be anti-anything without a solid reason.

My view for Terraform is completely different from yours apparently :) . I will try to put some of the things that are wrong with Terraform. And these are just some of the things. Terraform is just wrapping things around ARM API and in many cases they are way behind what is available in ARM. From the people I know who has used Terraform the language is not good at all. YML has the same problems as JSON.
The guys here are trying to do something that does not require a bunch of teams to implement the latest stuff on Azure services otherwise the language becomes just another Az PowerShell module or Az CLI that needs constant update and multiple teams to take care of that.
As this is opened discussions I am just expressing my opinion. You are free to not agree with me but and I am free to not agree with you :)

To clarify, we are embracing terraform in addition to bicep. We are not anti-anything :)  There's enough usage of both to justify the investment, and we have a dedicated team at microsoft to make the TF experience as good as it can possibly be
ARM Templates are by far the most used language for declarative deployments to Azure, so bicep is really designed to make their experience better. Some of the reasons we see customers successful with ARM include:

bicep/ARM does preflight validation on the entire template, so you can fail a lot faster than with other tools
day zero support for all resource types & api versions
we can perform a what-if/plan operation without any state file management. We use Azure as the state.

Thanks for the insight @slavizh and @alex-frankel, Really good background. I work with Microsoft Azure all day every day and am also a HashiCorp Ambassador so keen to see both sides and both tools work!
Not sure who you are speaking to in regards to the language that terraform uses (HCL), but like bicep it started without all the capability and then added it, such as looping, etc. I'm not sure where YML comes into it.
Day Zero support is what I see as the main benefit for this one.
Can't wait to see it progress further!

awesome - let us know if there are any other questions. Good if I close this?

Sure thing Alex close away. Thanks for taking the time !

Please do not embrace Terraform!!!

Why?
The readme for bicep, indicating it's not production ready and is experimental, as opposed to Terraform which is stable, and is highly expanded-upon to work with more than just Azure, and does it all very well, including workspace handling and is a well thought out infra-as-code model. Even Ansible does a pretty great job at automating cloud provisioning on Azure.
What separates bicep from the pack? What makes it unique? Good open-source communities/projects shouldn't be anti-anything without a solid reason.

Two words.  State File.
Terraform's "source of truth" is the state file, not Azure.  This can cause all sorts of chaos with any environment that doesn't completely block change access to the portal.
Now, if you said "Terraform HCL", that's a completely different situation and MS should support that (@alex-frankel ) as a source/destination for compile/decompile.

The state file is not really a problem as it can easily be refreshed or fixed in case of imports etc. I don't agree with the above statement that Terraform is just an ARM wrapper as it uses the REST APIs which makes it dependent only on the provisioning APIs.
Whilst I do get that Bicep might be more directly on the metal to Azure, there are other things which we do that the Azure platform has just been terrible at: Certificate management.
We generate Let's Encrypt certificates (internally of course) and roll them regularly whilst being able to deploy them to their respective destinations. We still have App Gateway V1s that don't load certs from KeyVault and KeyVault itself only allows for rolling of certs from limited certificate providers anyway so we've filled this requirement with Terraform quite nicely.
The final advantage for us is of course, it can be used with other cloud providers too

Commenting on closed topic :-)
The overall promise of Bicep seems good, it has latest support for ARM API.
I would pose a question : is there a number on how many production deployments are immediately using the "latest and greatest"?  How many projects are waiting for bleeding edge feature, and when released immediately implement it?  (Actually I do not know the answer to this myself, but it would help gauge the "immediately available" feature's value)
Thought:
Could we not split off the ARM integration / decompilation piece of Bicep into a Terraform Provider?
After watching the Ignite overview, that is the feature that caught my attention most.
The feature of "another language that is not JSON"
Personally I believe that problem has been solved a couple times before, so I would need extra convincing there.
Why learn another syntax and language and tooling?
That is the main drawback I see for Bicep : that it runs the risk of "yet another tool" for a development team to learn.  (next to Ansible, Terraform, ARM)  If I work on AWS, Google as well, I would have to know Terraform, or that platforms native choice.
Terraform somewhat shields you from the underlying platforms madness it seems. (And sometimes it fails and adds more madness of its own)

in a real word scenario i would use terraform to provision AKS using the azurerm provider, then use the kubernetes provider and the helm provider to bootstrap my cluster and install my operational tools.
The reason people gravitated toward sterraform is because of the statefile, a means to track record changes idnependent and outside the influence of the target provider.

Two words. State File.
Terraform's "source of truth" is the state file, not Azure. This can cause all sorts of chaos with any environment that doesn't completely block change access to the portal.

Actually, bicep is particularly poorly suited for cases where manual changes can happen when compared to Terraform.
With Terraform, if I delete a resource from Terraform, Terraform understands, by virtue of it's state file, that it should remove it from the underlying provider.
With Bicep, if I delete a resource, I have to be deploying with complete mode for it to delete the resource, and it does not support complete mode with some scopes, but regardless, when I do deploy in complete mode, everything in the scope (e.g. the resource group) that is not in the azure template gets deleted.
The state file may seem annoying, but not having it is also pretty annoying.

Agreed that there are both pros and cons to the state file. In particular, you get a much more accurate plan output.
FWIW @aucampia, Deployment Stacks is designed to enable a lot of the update/delete workflows you are describing.

FWIW @aucampia, Deployment Stacks is designed to enable a lot of the update/delete workflows you are describing.

Thanks for sharing, did not now about this. Seems pretty useful.

FWIW @aucampia, Deployment Stacks is designed to enable a lot of the update/delete workflows you are describing.

Thanks for sharing, did not now about this. Seems pretty useful.

Agreed that there are both pros and cons to the state file. In particular, you get a much more accurate plan output.
FWIW @aucampia, Deployment Stacks is designed to enable a lot of the update/delete workflows you are describing.

Agreed that there are both pros and cons to the state file. In particular, you get a much more accurate plan output.
FWIW @aucampia, Deployment Stacks is designed to enable a lot of the update/delete workflows you are describing.

Deployment Stacks are certainly looking like they will be the best approach - State management, but Azure controls the state and the Azure object themselves are state aware.  This really eliminates the issue with Terraform's state going out of sync.


Fixed registry path

@justanotheranonymoususer : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Fix typos, add more contributors and maintainers.

What type of PR is this?
/kind documentation
What this PR does / why we need it:
to keep the related contents update
Which issue(s) this PR fixes:
no
Special notes for your reviewer:
no
Does this PR introduce a user-facing change?:
no

/lgtm


SPARK-10314 [CORE]RDD persist to OFF_HEAP tachyon got block rdd_x_x n…

SPARK-10314 [CORE]RDD persist to OFF_HEAP tachyon got block rdd_x_x not found exception when parallelism is big than data split size

Can one of the admins verify this patch?

@romansew please submit this patch against the master branch instead. It would be good to have a consistent fix in both newer and older branches.

Also, I looked at the definition of BlockStatus#isCached. Looks like it already takes into account external block store size? Is Tachyon somehow not caching your block for some reason? (Is this a Spark problem?)

I resubmit the pull request on the master branch
https://github.com/apache/spark/pull/8562

@andrewor14 The empty block cache to mem the memSize in BlockStatus#isCached is 4. but the empty block cache to offheap the externalBlockStoreSize is 0. so the isCached function return false.

Can you close this now that you've opened another one?

I open the other one:https://github.com/apache/spark/pull/8562


Add UML Sequence Diagram for Add Insurance Plan Feature

Description:
We have implemented the "Add Insurance Plan" feature, and it is important to update the developer guide to include comprehensive documentation. Specifically, we need to create and add a UML sequence diagram that outlines the interactions and processes involved in adding an insurance plan.
Tasks:

Create a UML sequence diagram for the "Add Insurance Plan" feature.
Update the developer guide with the new diagram.
Ensure that the diagram clearly illustrates all relevant components and their interactions during the add insurance plan process.

Acceptance Criteria:

The UML sequence diagram is successfully added to the developer guide.
The diagram accurately reflects the workflow of adding an insurance plan.
Documentation is clear, concise, and easy to understand for future developers.

Additional Notes:
Please refer to existing diagrams (made using PlantUML) in the developer guide for consistency in style and formatting. If you have any questions or need further clarification on the feature, feel free to reach out.

not exactly part of v1.4 as due date is one day later


New propagation library created

What: propagation library created
How: moved code related only with propagation from tf.utils to propagation.py in libraries
Why: cleaner code

 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.1 out of 2 committers have signed the CLA.:white_check_mark: superplay1:x: frosati1You have signed the CLA already but the status is still pending? Let us recheck it.

Related: https://github.com/shaimach/c3po/issues/49


JaroWinkler string distance algorithm allocates 1.11 GB and spends 4.8% of CPU time generating suggestions that are rarely used

This was found by:

Using VF# solution in VS 2019 Preview
Working in service.fs
Noticing memory usage was high
Profiling with good 'ole PerfView
Typing a bit, invoking some features, etc.

GC Rollup

Using a 50 second sample of the full trace:
Allocations

System.Char[] allocations

CPU time

There are two problems at play here:

We send all identifiers the compiler knows about through this algorithm when generating a set of suggested names when there is a compile error
The routine itself allocates a lot of char[]s and generally does a lot of work: https://github.com/Microsoft/visualfsharp/blob/master/src/utils/EditDistance.fs#L29

Taking a brief look, it doesn't seem trivial to make this faster or allocate less. It's also not clear how we could generate useful suggestions for people if we cannot scan all known identifiers.
That said, we should find something to do here. It comes as a result of typing in the IDE. Because this is done a lot (duh!), especially for slower typers, improving this would be useful.

// cc @forki @Rickasaurus since you all may know more about how EditDistance works.
Based on this issue, I will try to make a benchmark on FilterPredictions.

The root of this is two competing use cases:

Using the IDE, typing stuff, and not being finished with what I'm typing
Using the command-line compiler and building to see what the errors are

In both cases we churn through all known symbols and generate suggestions, but in the former case, this work is not useful and evidently causes the IDE to do a lot of work when it shouldn't. However, the latter case is useful, and the suggestions we get from this feature are also nice.
A quick fix would be to not generate these suggestions. However, that regresses the second scenario. What we actually need is a way to understand when we're in one of the following contexts:

Doing a command-line build
Invoking a quick fix on an error (to get the suggestions)

In this case, we want to do this work. However, routine typing in the editor should not churn through all identifier strings and calculate distances.

Full sample:
Callers tree on System.String:

215 MB string allocations just from String.Concat.
Callee tree on FilterPredictions:

1.11 GB of allocations, which is 10.9% of total allocations in the sample.
We need to put this feature behind a flag until we come up with something better. This is an insane amount of allocations.

Potential quick change:

Define a flag that effectively means, "I am in an editor with F# tooling"
Have that flag wired to an off-by-default setting in VS at the language service integration layer (other editors can also wire up their own)
Have the flag set to true by default in the compiler so dotnet build isn't regressed

Unfortunately, this also means the code fix is off by default. We are not architecturally set up to have it be on for the code fix, since doing so would mean that it would also be turned on any time you're typing an get an error. We'd need a larger architectural change (e.g., moving this and related work out of process) to have regular typing not incur this work, but a lightbulb fix go and do the calculation.

So it's only an issue while typing?
What's the concrete repro?
Am Fr., 21. Dez. 2018, 02:43 hat Phillip Carter<EMAIL_ADDRESS>geschrieben:

Potential quick change:

Define a flag that effectively means, "I am in an editor with F#
tooling"
Have that flag wired to an off-by-default setting in VS at the
language service integration layer (other editors can also wire up their
own)
Have the flag set to true by default in the compiler so dotnet build
isn't regressed

Unfortunately, this also means the code fix is off by default. We are not
architecturally set up to have it be on for the code fix, since doing so
would mean that it would also be turned on any time you're typing an get an
error. We'd need a larger architectural change (e.g., moving this and
related work out of process) to have regular typing not incur this work,
but a lightbulb fix go and do the calculation.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/visualfsharp/issues/6044#issuecomment-449208844,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNGx2EpjwNQDOUsxZUI-CBcoGS-pSks5u7DzRgaJpZM4ZdD9U
.

Yes, this is with typing. If you do it slowly enough, or at least such that there are some compile errors due to not being "done", it will trigger. I noticed this while goofing around a bit with caches in service.fs. There's no concrete repro beyond noticing the slowdown and gathering a trace. Just the nature of lots of these IDE perf problems 🙂
What's nasty here is that in only 90 seconds, we already triggered a 4 second gen 2 collection. That's huge, especially considering it's in a single file. I'm not sure how to track if anything from this made it into gen2, but when something is representing nearly 11% of memory pressure it's very significant. We didn't see this before because other massive allocations from continually allocating strings that represent the source file were the hot spot, but with #6001 fixing that, we're now seeing this as a hot spot in the scenario I described.

Ok I see. That's really bad. On the other hand this feature is one of the
most useful during development, so just turning it off by default would be
extremely sad.
Here are a few things that we can try:

optimize the algo to allocate less
use simpler algo (tbh predictions are almost always giving one element -
and it's the correct one) - this means we can probably find it without all
the ceremony
we could limit the suggestions to the first 5 name res errors in every
run. Which means if you process your compiler errors from top to bottom
then you will always have suggestions for the current error - but you would
not have it everywhere in all 5000 errors that you might see

But of course the most important thing is to find a way to device if the
user is still typing
Am Fr., 21. Dez. 2018, 07:11 hat Phillip Carter<EMAIL_ADDRESS>geschrieben:

Yes, this is with typing. If you do it slowly enough, or at least such
that there are some compile errors due to not being "done", it will
trigger. I noticed this while goofing around a bit with caches in
service.fs. There's no concrete repro beyond noticing the slowdown and
gathering a trace. Just the nature of lots of these IDE perf problems 🙂
What's nasty here is that in only 90 seconds, we already triggered a 4
second gen 2 collection. That's huge, especially considering it's in a
single file. I'm not sure how to track if anything from this made it into
gen2, but when something is representing nearly 11% of memory pressure it's
very significant. We didn't see this before because other massive
allocations from continually allocating strings that represent the source
file were the hot spot, but with #6001
https://github.com/Microsoft/visualfsharp/pull/6001 fixing that, we're
now seeing this as a hot spot in the scenario I described.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/visualfsharp/issues/6044#issuecomment-449269029,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADgNHH21BJPAGXAcs_JmEZ5aBgQNL_3ks5u7HuogaJpZM4ZdD9U
.

there is also another idea in https://github.com/Microsoft/visualfsharp/pull/6049 - we don't need to call this whole thing on small identifiers. this may be a lot during typing.

Yes, finding a way to not turn it off is definitely ideal. If we can't get something in place I'll likely advocate that we turn it off in the IDE by default. I do think it's a bit less severe in VS because completion will generally work, and the message in the error list window is a bit hard to read, making it something people may lean a little less on. But that would mean sacrificing the nicer message in the tooltip and the code fix suggestion, which are obviously benefits.
The allocation and CPU hotspot is in commonChars. In terms of exclusive CPU time, commonChars and the function exists that it calls are the bulk of CPU time in this routine. So if there's a way to reduce those down, that's a big impact. But as you suggested, limiting the number of strings that go in is also important.

BTW thanks for engaging on this already - let's figure out the best way forward here 🙂

Would changing it to char [] help?

@forki, even though char [] is a fixed array of value types, any array will allocate on the heap. In fact, the previous issues with allocating string, that @cartermp mentioned, part of that issue was caused by massive char[] if I remember correctly.
That is not to say that ResizeArray cn be a cause of trouble, because it does extra heap allocations. If this can be replaced with a single allocation, or at least less resizing, this may help. But any array will end up on the heap as far as I know.

@AviAvni was able to remove the array completely in #6050

@cartermp please try to merge #6049 and #6050 and retry your repro

Just to confirm that this should definitely be impactful. I took another trace with CompletionProvider.fs open and typing a new function. I deliberately made things not compile to simulate someone who's still sort of figuring out how to piece different components together. Results:

If benchmarking yields improvements in memory/CPU time I think we can definitely consider #6050 as having solved the problem and close this issue.
I still want to take a look at #6049, since it can basically keep the routine from running at all in a number of scenarios. We'll have to think that one through a bit, since there are definitely cases where you'd want a suggestion when an identifier is less than 5 chars long. Perhaps we can take the average length of all identifiers and use that as a threshold?


Build launcher base

Create a launcher that launches the game. The basics for an UI should be built so adding components is easier.

This is mostly complete, at this time I just need it to compile on Linux and we can mark it done and move to the next steps.

@apachano @SuperFola Do I assign you guys?

Yep
Going to have a look at the code

Going to try making a small launcher with ImGUI, sending arguments to the (future) game

Updating this, per some slack discussions we are now using a separate language for the launcher. Currently, a rough prototype has begun at https://github.com/GentenStudios/PhoenixLauncher and we will move issues to that repo.


⚠ Bofrost.be has degraded performance

In fe413ef, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1041 ms

Resolved: Bofrost.be performance has improved in 255e967 after 6 minutes.


[5.2] Fix missing middleware parameters when using authorizeResource()

This fixes an issue with the authorizeResource method in the AuthorizesResources trait. Previously, the middleware would not be properly applied to some of the controller's methods.
As an example, say you have a PostsController class that uses authorizeResource as follows:
class PostsController extends Controller
{
   function __construct()
   {
        $this->authorizeResource(\App\Post::class);
   }
}

Let's say this controller is registered as a resource:
Route::singularResourceParameters();
Route::resource('posts', 'PostsController');

Running php artisan route:list will reveal that the controller's create and edit methods do not have the proper middleware applied:
+--------+-----------+-------------------+---------------+----------------------------------------------+-------------------------+
| Domain | Method    | URI               | Name          | Action                                       | Middleware              |
+--------+-----------+-------------------+---------------+----------------------------------------------+-------------------------+
|        | GET|HEAD  | /                 |               | Closure                                      | web                     |
|        | POST      | posts             | posts.store   | App\Http\Controllers\PostsController@store   | web,can:create,App\Post |
|        | GET|HEAD  | posts             | posts.index   | App\Http\Controllers\PostsController@index   | web,can:view,App\Post   |
|        | GET|HEAD  | posts/create      | posts.create  | App\Http\Controllers\PostsController@create  | web                     |
|        | DELETE    | posts/{post}      | posts.destroy | App\Http\Controllers\PostsController@destroy | web,can:delete,post     |
|        | PUT|PATCH | posts/{post}      | posts.update  | App\Http\Controllers\PostsController@update  | web,can:update,post     |
|        | GET|HEAD  | posts/{post}      | posts.show    | App\Http\Controllers\PostsController@show    | web,can:view,post       |
|        | GET|HEAD  | posts/{post}/edit | posts.edit    | App\Http\Controllers\PostsController@edit    | web                     |
+--------+-----------+-------------------+---------------+----------------------------------------------+-------------------------+

This happens because authorizeResource repeatedly calls $this->middleware() (here), which replaces the entry in the middleware each time it is called (here).
This was missed by the unit tests for this method because the tests only verified the presence of middleware by name -- the tests didn't make sure the middleware was actually being applied to the correct methods.
After applying this fix, the routing table will appear as follows:
+--------+-----------+-------------------+---------------+----------------------------------------------+-------------------------+
| Domain | Method    | URI               | Name          | Action                                       | Middleware              |
+--------+-----------+-------------------+---------------+----------------------------------------------+-------------------------+
|        | GET|HEAD  | /                 |               | Closure                                      | web                     |
|        | POST      | posts             | posts.store   | App\Http\Controllers\PostsController@store   | web,can:create,App\Post |
|        | GET|HEAD  | posts             | posts.index   | App\Http\Controllers\PostsController@index   | web,can:view,App\Post   |
|        | GET|HEAD  | posts/create      | posts.create  | App\Http\Controllers\PostsController@create  | web,can:create,App\Post |
|        | DELETE    | posts/{post}      | posts.destroy | App\Http\Controllers\PostsController@destroy | web,can:delete,post     |
|        | PUT|PATCH | posts/{post}      | posts.update  | App\Http\Controllers\PostsController@update  | web,can:update,post     |
|        | GET|HEAD  | posts/{post}      | posts.show    | App\Http\Controllers\PostsController@show    | web,can:view,post       |
|        | GET|HEAD  | posts/{post}/edit | posts.edit    | App\Http\Controllers\PostsController@edit    | web,can:update,post     |
+--------+-----------+-------------------+---------------+----------------------------------------------+-------------------------+

The same issue appears in Laravel 5.3, but I am guessing the middleware names applied in this method might be changing since the default Policy stub for resources includes methods like viewAny in addition to view.

Overall this looks good to me :+1:
Should we maybe add a comment there in the code explaining what's going on? Without knowing about the middleware overriding one another, that code looks little weird.

Great feedback, thanks. After thinking about it I realized it might be better to simulate each request in the test to avoid relying on implementation details of the controller class.
Let me know if you notice any other issues. Thanks!


Export RUSTFLAGS when running tests

This is not quite a bug and not quite a feature request ... One of my integration tests involves running the binary in a separate process so that if it segfaults I'm able to catch the error instead of the whole test runner crashing immediately. To make this work, I run cargo build and then run the resulting binary. Normally this works fine, since running cargo test involves building all the dependencies anyway, so at worst building the binary takes a few seconds. However, if I run cargo-tarpaulin before running cargo test, then the integration test rebuilds every single dependency (since tarpaulin sets RUSTFLAGS and my binary doesn't). As a result, the test times out; see https://gitlab.com/jyn514/rcc/-/jobs/535816510#L158 for an example of this.
If tarpaulin would export RUSTFLAGS, then I could use the cached dependencies instead of having to rebuild the whole thing from scratch.

Hmm interesting I'll have to have a think about the best way to tackle this. Overwriting the rustflags as a change could mess up things for users who run tarpaulin on their local systems... You can check src/cargo.rs for how I set the rust flags. If you're building without --release the rustflags will be -C relocation-model=dynamic-no-pic -C link-dead-code -C debuginfo=2

Overwriting the rustflags as a change could mess up things for users who run tarpaulin on their local systems

I'd be fine making this opt-in, I understand this is a bit of an edge case. Exporting the flags manually sounds like a good workaround in the meantime, I'll see if that helps. Another thing I could try is telling tarpaulin to ignore those integration tests, not sure if there's already a way to do that.

Any arguments after a -- go to the test binaries so you can use that to specify what tests to run i.e. cargo tarpaulin -- my_module. As those arguments go to the test binary if there's anything to ignore certain tests for them that will work

--skip looks like what I want, thank you!
$ cargo test -- -h | grep skip
        --skip FILTER   Skip tests whose names contain FILTER (this flag can
                        be used multiple times)

Unfortunately --skip seems to only apply to unit tests, so that doesn't help. I tried setting RUSTFLAGS myself but it still rebuilt everything: https://gitlab.com/jyn514/rcc/-/jobs/540034388#L165, https://gitlab.com/jyn514/rcc/-/commit/973623a82927a9be11b70c4a1bf2c95f4e031524

So in tarpaulin on develop I've added two args --print-rust-flags and --print-rustdoc-flags. These will print all possible values for RUSTFLAGS and RUSTDOCFLAGS that you can get given your tarpaulin configuration for the project and CLI args in use (different args can change some flags).
There is also new macros I added about a month ago so you can do #[cfg(not(tarpaulin_include))] on something and it won't be in the binaries tarpaulin builds. Which might also be a good option for you. Check this part of the README for more details https://github.com/xd009642/tarpaulin#ignoring-code-in-files
If there's anything else you'd like to see to resolve this issue let me know otherwise I'll probably close it after a week or two of nothing (but feel free to reopen if I close it)

I actually stopped using tarpaulin because it was slowing down CI so much ... I'll see if I can try to add it back but this is a busy week for me so feel free to close if you don't hear back.

Fair enough slow CI is definitely a pain. I think using #[cfg(not(tarpaulin_include))] on large tests to avoid building them with tarpaulin's flags could potentially help a lot depending on your code.
Also, bit of future work but once I prototype the idea I describe here https://github.com/xd009642/tarpaulin/issues/516#issuecomment-665938418 if that works tarpaulin should be able to transition to using mostly the same build output as cargo test. So you might want to watch that issue.


$stateChangeSuccess fires multiple times after $state.transitionTo

I have a function that fires on $stateChangeSuccess.  When I transition states with $state.transitionTo I find that the function fires twice on the first transition.   I also found that going back and forth between child states and parent states causes the function to be fired more times.   Anyone have any thoughts?
$rootScope.$on('$stateChangeSuccess' function() {
//my code
};

@christopherthielen I too was experiencing the same issue, but after following your comment to change from $rootscope to $scope, it works now without firing multiple times. Thank you for the solution.

@christopherthielen  @chasys  Thanks! This solved MY issue.


Cloud component not loading due to cryptography

Home Assistant release with the issue:
0.92
Last working Home Assistant release (if known):
0.91
Operating environment (Hass.io/Docker/Windows/etc.):
Windows
Component/platform:
Cloud / cryptography module
Description of problem:
After upgrading from 0.91 to 0.92 I had lost the 'Home Assistant Cloud' option in Settings. The cause seem to be my install not being able to load the cloud component. When I start HA I get this error (apologies for the screenshot):

I tried downgrading the cryptography module (2.6, 2.5, 2.4.2) but it had no effect. I downgraded back to HA 0.91, after which the cloud component worked. I then upgrade again to 0.92 and this is the upgrade log:
> C:\Windows\system32>pip3 install --upgrade homeassistant
Collecting homeassistant
  Using cached https://files.pythonhosted.org/packages/bb/94/dd2d566a504310e386d
a1ea9fe29547aab102ec87342411d7c8cf9d619e6/homeassistant-0.92.0-py3-none-any.whl
Requirement already satisfied, skipping upgrade: requests==2.21.0 in c:\python36
4\lib\site-packages (from homeassistant) (2.21.0)
Collecting ruamel.yaml==0.15.91 (from homeassistant)
  Using cached https://files.pythonhosted.org/packages/94/9c/ce7c70ff840a341de3b
6285ec890ae80be1a54e6c8d07c2e566421cf0699/ruamel.yaml-0.15.91-cp36-cp36m-win32.w
hl
Collecting cryptography==2.6.1 (from homeassistant)
  Using cached https://files.pythonhosted.org/packages/3c/dd/b8f72fa8b9ead4207da
0684180430ef2705aef9c1fb54aae4dde62ad8f15/cryptography-2.6.1-cp36-cp36m-win32.wh
l
Requirement already satisfied, skipping upgrade: pyyaml<4,>=3.13 in c:\python364
\lib\site-packages (from homeassistant) (3.13)
Requirement already satisfied, skipping upgrade: attrs==18.2.0 in c:\python364\l
ib\site-packages (from homeassistant) (18.2.0)
Requirement already satisfied, skipping upgrade: bcrypt==3.1.6 in c:\python364\l
ib\site-packages (from homeassistant) (3.1.6)
Collecting python-slugify==3.0.2 (from homeassistant)
  Using cached https://files.pythonhosted.org/packages/c1/19/c3cf1dc65e89aa999f8
5a4a3a4924ccac765a6964b405d487b7b7c8bb39f/python-slugify-3.0.2.tar.gz
Requirement already satisfied, skipping upgrade: pip>=8.0.3 in c:\python364\lib\
site-packages (from homeassistant) (19.1)
Requirement already satisfied, skipping upgrade: voluptuous==0.11.5 in c:\python
364\lib\site-packages (from homeassistant) (0.11.5)
Requirement already satisfied, skipping upgrade: astral==1.10.1 in c:\python364\
lib\site-packages (from homeassistant) (1.10.1)
Requirement already satisfied, skipping upgrade: async-timeout==3.0.1 in c:\pyth
on364\lib\site-packages (from homeassistant) (3.0.1)
Requirement already satisfied, skipping upgrade: jinja2>=2.10 in c:\python364\li
b\site-packages (from homeassistant) (2.10)
Requirement already satisfied, skipping upgrade: pytz>=2019.01 in c:\python364\l
ib\site-packages (from homeassistant) (2019.1)
Requirement already satisfied, skipping upgrade: voluptuous-serialize==2.1.0 in
c:\python364\lib\site-packages (from homeassistant) (2.1.0)
Collecting PyJWT==1.7.1 (from homeassistant)
  Using cached https://files.pythonhosted.org/packages/87/8b/6a9f14b5f781697e512
59d81657e6048fd31a113229cf346880bb7545565/PyJWT-1.7.1-py2.py3-none-any.whl
Requirement already satisfied, skipping upgrade: aiohttp==3.5.4 in c:\python364\
lib\site-packages (from homeassistant) (3.5.4)
Requirement already satisfied, skipping upgrade: certifi>=2018.04.16 in c:\pytho
n364\lib\site-packages (from homeassistant) (2018.4.16)
Requirement already satisfied, skipping upgrade: idna<2.9,>=2.5 in c:\python364\
lib\site-packages (from requests==2.21.0->homeassistant) (2.7)
Requirement already satisfied, skipping upgrade: chardet<3.1.0,>=3.0.2 in c:\pyt
hon364\lib\site-packages (from requests==2.21.0->homeassistant) (3.0.4)
Requirement already satisfied, skipping upgrade: urllib3<1.25,>=1.21.1 in c:\pyt
hon364\lib\site-packages (from requests==2.21.0->homeassistant) (1.22)
Requirement already satisfied, skipping upgrade: cffi!=1.11.3,>=1.8 in c:\python
364\lib\site-packages (from cryptography==2.6.1->homeassistant) (1.11.5)
Requirement already satisfied, skipping upgrade: asn1crypto>=0.21.0 in c:\python
364\lib\site-packages (from cryptography==2.6.1->homeassistant) (0.24.0)
Requirement already satisfied, skipping upgrade: six>=1.4.1 in c:\python364\lib\
site-packages (from cryptography==2.6.1->homeassistant) (1.11.0)
Requirement already satisfied, skipping upgrade: text-unidecode==1.2 in c:\pytho
n364\lib\site-packages (from python-slugify==3.0.2->homeassistant) (1.2)
Requirement already satisfied, skipping upgrade: MarkupSafe>=0.23 in c:\python36
4\lib\site-packages (from jinja2>=2.10->homeassistant) (1.0)
Requirement already satisfied, skipping upgrade: yarl<2.0,>=1.0 in c:\python364\
lib\site-packages (from aiohttp==3.5.4->homeassistant) (1.1.1)
Requirement already satisfied, skipping upgrade: idna-ssl>=1.0; python_version <
 "3.7" in c:\python364\lib\site-packages (from aiohttp==3.5.4->homeassistant) (1
.0.1)
Requirement already satisfied, skipping upgrade: multidict<5.0,>=4.0 in c:\pytho
n364\lib\site-packages (from aiohttp==3.5.4->homeassistant) (4.1.0)
Requirement already satisfied, skipping upgrade: typing-extensions>=3.6.5; pytho
n_version < "3.7" in c:\python364\lib\site-packages (from aiohttp==3.5.4->homeas
sistant) (3.7.2)
Requirement already satisfied, skipping upgrade: pycparser in c:\python364\lib\s
ite-packages (from cffi!=1.11.3,>=1.8->cryptography==2.6.1->homeassistant) (2.18
)
Installing collected packages: ruamel.yaml, cryptography, python-slugify, PyJWT,
 homeassistant
  Found existing installation: ruamel.yaml 0.15.89
    Uninstalling ruamel.yaml-0.15.89:
      Successfully uninstalled ruamel.yaml-0.15.89
  Found existing installation: cryptography 2.5
    Uninstalling cryptography-2.5:
      Successfully uninstalled cryptography-2.5
  Found existing installation: python-slugify 1.2.6
    Uninstalling python-slugify-1.2.6:
      Successfully uninstalled python-slugify-1.2.6
  Running setup.py install for python-slugify ... done
  Found existing installation: PyJWT 1.6.4
    Uninstalling PyJWT-1.6.4:
      Successfully uninstalled PyJWT-1.6.4
  Found existing installation: homeassistant 0.91.0
    Uninstalling homeassistant-0.91.0:
      Successfully uninstalled homeassistant-0.91.0
Successfully installed PyJWT-1.7.1 cryptography-2.6.1 homeassistant-0.92.0 pytho
n-slugify-3.0.2 ruamel.yaml-0.15.91

After this is still failed to load the cloud component so I had to downgrade again which worked fine.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

You error log clear state permission error
You can try to manual install hass_nabucasa
pip3 install hass_nabucasa==0.12

This is not a bug, please go to our forum or discord channel for support.


Model install step is confusing

The model install step linked here is confusing. Do you need to locally install llama2 from the Meta website, or is there a way to access it through the command line with HuggingFace? Also, the HF URL appears to be meta-llama/Llama-2-7b-chat-hf, not llama-2/llama/llama-2-7b-chat-hf (which is the default in experiments/configs/transfer_llama2.py).

You need the LLaMA-2 model, including weights and tokenizer, converted by Hugging Face. Sorry for the confusion on the default path as we have downloaded the model to another place on our cluster instead of using HuggingFace's cache location. Please change it accordingly.

Closing this issue for no further questions for now.


Feature: Comma Separated Globs

Description
Currently if I want to supply multiple globs they each need a unique entry in lint-staged in my package.json even if they perform the same task.
For example:
(EDIT: see my comment below for a better example)
  "lint-staged": {
    "*.js": [
      "prettier --no-semi --single-quote --write",
      "git add"
    ],
	"*.ts": [
      "prettier --no-semi --single-quote --write",
      "git add"
	]
  }

It would be great to support this pattern of comma separated globs to remove clutter:
  "lint-staged": {
    "*.js,*.ts": [
      "prettier --no-semi --single-quote --write",
      "git add"
    ]
  }

AFAICT could be achieved in generateTasks.js by calling pattern.split(',') when passing to micromatch

You can use *.{js,ts} for this. We’ll have support for multiple globs via #639

@okonet I think I might not have explained corrcetly since *.{js,ts} doesn't solve the issue.
Here's a better example. I need two entries to target nested files:
  "lint-staged": {
    "*.js": [
      "prettier --no-semi --single-quote --write",
      "git add"
    ],
    "**/*.js": [
      "prettier --no-semi --single-quote --write",
      "git add"
   ]
  }

Instead, it would be cleaner if I can do:
"lint-staged": {
    "*.js,**/*.js": [
      "prettier --no-semi --single-quote --write",
      "git add"
    ]
  }

Do you think this should be possible without having to write a function? It's something I would intuitively expect from globbing.

Instead of using a comma, spaces might be better, i.e "*.js,**/*.js" => "*.js **/*.js"

*.js should target all files for you, so no need in what you’re doing.
Also if you still need, this will be possible with #639


Incorrect check for skipCertVerification in security.go

The check on line 302 in security.go should be "if !skipCertVerification" so that VerifyWithChain will be called.

Hey @edygert
That's correct 👌 I will fix that today.

Fixed in d12bac5b8c708cf4228f3808d001f3d0ffa6fdf7
Thank you !


[Bug] SteamVR Overlay completely broken.

Describe the bug
SteamVR overlay is no longer working. It seems to be bugging out entirely. XSOverlay notifications still work.
To Reproduce
Steps to reproduce the behavior:
Turn on SteamVR overlay (wrist overlay optional but also broken)
Watch it break.
Expected behavior
Should let me access the overlay, both in the steam menu and on the wrist.
Screenshots
I have a video.
https://github.com/vrcx-team/VRCX/assets/120276008/63556e69-12f7-415f-bd49-858f11ed1e28
What version you are running
Latest Nightly version. Note that it's on the live version too.
Other notes: I have reinstalled twice, Restarting doesn't work. Changing to the beta of both SteamVR and the Steam client doesn't change anything about this. The only thing I haven't done is reinstall windows.

okay i found the issue, i turned off the gpu fix setting and it stabilized.

On the latest nightly I've recently added logging for when the overlay fails to start in %AppData%\VRCX\logs but most of the time when the overlay fails to start it's due to multiple GPUs and the overlay using the wrong one, try change the SteamVR Overlay GPU Fix option in Settings -> Advanced
ah never mind then yup.

On the latest nightly I've recently added logging for when the overlay fails to start in %AppData%\VRCX\logs
i'll keep that in mind for future overlay issues, thank you.

Okay, the issue is back. I checked the logs and it just says "RequestFailed"

I can confirm the overlay has been broken for me as well. Is there anything I can provide?

On the latest nightly I've recently added logging for when the overlay fails to start in %AppData%\VRCX\logs but most of the time when the overlay fails to start it's due to multiple GPUs and the overlay using the wrong one, try change the SteamVR Overlay GPU Fix option in Settings -> Advanced

Try this, also check your log file C:\Program Files (x86)\Steam\Logs\vrclient_VRCX.txt

a recent potential fix has been pushed to the nightly branch from 11/13 could you try updating to that and see if it solves your issues. you can access this update from settings > VRCX Updater > Change build

I just wanted to confirm that it is working for me now after one of the the latest updates.


Don't require xy position if dragging along a single axis

Update the drag function in https://github.com/rbb-data/svelte-starter/blob/main/src/lib/actions/pannable.ts to allow passing only x or y if something is dragged along a single axis

Needs to be updated once this is done: https://github.com/rbb-data/svelte-starter/blob/0aeba7be856048dceff5dd34bd3bb3179face667/src/lib/components/shared/Slider.svelte#L40


[V2] WS crashes on start

Version and Environment Details
Operation system: MacOS 10.14.4
IDE name and version: WS 2019.1
Plugin version: 2.0.0
Actual Behaviour
Crashes WS on launch. (WS screen freeze, force restart)
Steps to Reproduce / Link to Repo with Reproduction and Instructions
This is a mono-repo with the following config...
{
  "projects": {
    "admin": {
      "schemaPath": "./packages/admin-server-nexus/src/generated/schema.graphql",
      "includes": ["./packages/admin-client/src/containers/*.tsx"]
    }
  }
}

I assume this is a config problem, but no config I think of stops the crash.  I've downgraded in the interim.  Any help is very appreciated!

Hi Dennis.
Thanks for using the plugin. The config looks valid to me, so not sure why you're seeing this freeze.
I assume that the freeze prevents you from getting to a stack trace via the error icon in the bottom right? If so, the log file might provide the information needed to make this issue actionable. You can access the log via the "Help" menu > "Show log" menu item. Hopefully a search for graphql in that log yields an exception you can attach to this issue.
Best regards,
Jim.

Hi Jim,
idea.log
Thanks so much for your quick response!
You're right about the stack trace.  I went through the log file, but nothing jumps out to me.  I deleted it and reran the crash, and attached that file here.  Maybe you can see something.
It appears to have a problem with the generated Nexus output.  The crash comes as soon as I point the schemaPath at the generated output.  Here's that file.
https://gist.github.com/LawJolla/168b4b1e1634beed44e08ea53cc8a2a1
I wish I was bright enough to be of more help.  Thanks for your great work!

Thanks Dennis.
The schema file should help me reproduce the issue and find the underlying cause.
I'll update once I know more.
/Jim

Hi Dennis.
I have a fix prepared for the upcoming 2.0.1 release which I'm targeting for the middle of next week.
Based on my testing this was not a classic deadlock/freeze, but your schema is so deeply nested in the input type definitions that graphql-java had to build stacks of 500+ type names to untangle their dependencies and verify the schema. I've upgraded graphql-java from 11.0 to 12.0 and it appears to perform much better with your schema.
Let me know if you'd like to test this fix in separate release and I'll make one available to you. Otherwise the typical turnaround time via the JetBrains plugin repo is 1-2 business days.
Quite a schema you have by the way, it's almost double the size of the entire GitHub API.
Best regards,
Jim.

Hi Jim,
Thank you for your incredible help and work!
More broadly, my project looks massive because it incorporates 35 Prisma types and its associated CRUD fields and types.  I doubt my schema is more complicated than a run-of-the-mill Prisma application.
Did you see any problematic pattern in the generated schema that would help to modify?  I'd be happy to open an issue there and discuss it with the Prisma/Nexus team.  From my experience, few Prisma users use Jetbrains products, so I may be a bit of an oddball for the plugin.


Inverse cv::linearPolar transform causes missing patch in output

I'm trying apply an inverse linear polar transform to some quadratic images:

The result, I'd expect is the following (done using Photoshop):

However, the result I get from OpenCV's linearPolar function is the following:

Some part of the image is missing and can be noticed as a black slice.
Minimal code to reproduce:
int main(int argc, char** argv) {
    assert(argc == 3);

    Mat pol;
    Mat in = imread(argv[1]);

    // In case input is not quadratic.
    resize(in, in, Size(in.rows, in.rows));

    // This is just for orienting the output. It can be left out - the bug still occurs.
    in = in.t();
    flip(in, in, -1);

    linearPolar(in, pol,
        Point2f(in.cols / 2, in.rows / 2), (in.rows / 2),
        CV_WARP_FILL_OUTLIERS | CV_INTER_LINEAR | CV_WARP_INVERSE_MAP);

    imwrite(argv[2], pol);

    return 0;
}

Any help in tracking this one would be appreciated. The related code (cvLinearPolarand cvCartToPolar) is somewhat hard to read.

I am new to development in Opencv.Can you please tell me where the said pieces of code(cvLinearPolar&cvCartToPolar) are located in the imgproc directory?I found their function declaration in imgproc_c.h but I am unable to find the function definition.

You can find
CV_IMPL void cvLinearPolar( const CvArr* srcarr, CvArr* dstarr,             CvPoint2D32f center, double maxRadius, int flags ) in imgproc/src/imgwarp.cpp:6598 (eof)

@kautilya-vc , Are you still working on this bug ?? If not, I would like to work on this. Please update :)
Thanks.

@ythej ,I am still working on this. I will update here incase I am unable to solve it.
Thanks

@kautilya-vc, Are you still working on this bug? If not, I am willing to fix this. Thanks :)

@Aravind-Suresh Please go ahead.Really sorry for the delay in response.


Super-Resolution & Virtual-Try-On

Hello, Thank you for your great work
This U2Net is Excellent and can be extended to some other applications of Computer Vision like Super Resolution and Virtual Try On, If you find these applications useful, train U2Net for those as well

Hi, Adeel, thanks for your suggestions.
Super resolution and virtual try on are good applications. We are currently
focusing on improving the segmentation accuracy while reducing the
computation costs for higher resolution inputs. Super resolution may be our
next independent or side project of the segmentation project. Virtuall Try
On is a very useful application. Unfortunately, we don't have much
resources on that because there are lot of engineering issues other than
theoretic issues to handle.
If you have any ideas or suggestions, please let us know and we can talk
about that to see if we are able to collaborate.
Regards,
Xuebin
On Thu, Mar 11, 2021 at 6:46 PM Adeel @.***> wrote:

Hello, Thank you for your great work
This U2Net is Excellent and can be extended to some other applications of
Computer Vision like Super Resolution and Virtual Try On, If you find these
applications useful, train U2Net for those as well
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/xuebinqin/U-2-Net/issues/175, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSGORMTEEAEE6NLBZVQWKTTDCNPTANCNFSM4ZAA6NKQ
.

--
Xuebin Qin
PhD
Department of Computing Science
University of Alberta, Edmonton, AB, Canada
Homepage:https://webdocs.cs.ualberta.ca/~xuebin/

For Image Super Resolution, Holopix50k dataset can be used which is quite large, you can check it here: github.com/LeiaInc/holopix50k/


Syntax-checking for clojure via joker

This is provided as a +feature on clojure because joker is not included
in the standard clojure distribution, and I wouldn't assume a given
clojurist has or wants to have it installed. But I think installing it
is the least-fuss option for syntax-checking clojure/script code in
emacs with flycheck.

Updated to remove the +joker flag

You should add instructions in the readme on how to install joker. A command to install it would be even better, but that could be a separate PR.

Thanks again!


Add price and SKU to Item

BLOCKED BY: https://github.com/on-site/StockAid/issues/83

Add SKU (string) and price (decimal) columns to Item; these can be null

Add Price and SKU columns to the view Item index tables

Add Price and SKU to the Item edit form, as text fields.

Lisa also mentioned that she keeps a price for each item, we should add that here too.

:+1:

Sizes are now post-MVP.  Remove Size from Item table and Sizes from Category table. Remove Sizes input from Category new/edit.

In order to work on this ticket, I'm waiting on a response from Lisa to this question:
Are we sure that all Items of the same Item type will have the same SKU and Price?
Say there is a Jackets category with a Parka item. What is Lisa gets two different ones? They could be different prices and/or have different SKUs. Or would she enter each different brand/type/price of Parka as a separate item?

@yazfaz Any updates from Lisa regarding this?

@anthonylebrun  I'm double checking on this. The answer we got from Lisa wasn't clear enough.

We should add the SKU and Price to the Item.


chart js douhnat chart animation won't work

Hey!
The on draw chart js animation isn't working when the chart is wrapped inside the main css class.
any work arounds ?

I am having the same issue. Anyone found solution for this?


ERROR: InvocationError for command; xattr or inventory

The cause is different to the one in https://github.com/linux-system-roles/tox-lsr/issues/72.
I am running on RHEL9 (Rocky9.1) and have tox/ tox-lsr in a conda environment.
$ python --version
Python 3.11.0

$ tox --version
3.28.0 imported from /network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox/__init__.py
registered plugins:
    tox-lsr-2.14.1 at /network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/hooks.py

$ ansible --version
ansible [core 2.14.4]
  config file = None
  configured module search path = ['/network/user/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/ansible
  ansible collection location = /network/user/.ansible/collections:/usr/share/ansible/collections
  executable location = /network/user/.conda/envs/ansible-tox-lsr/bin/ansible
  python version = 3.11.0 | packaged by conda-forge | (main, Jan 14 2023, 12:27:40) [GCC 11.3.0] (/network/user/.conda/envs/ansible-tox-lsr/bin/python)
  jinja version = 3.1.2
  libyaml = True

I just downloaded the latest version of https://raw.githubusercontent.com/linux-system-roles/linux-system-roles.github.io/master/download/linux-system-roles.json.
On Filesystems mounted via Network (NFS / BeeGFS) I get the following when setting
export LSR_QEMU_CACHE=/network/user/.cache/linux-system-roles

$ tox -e qemu-ansible-core-2.14 -- --image-name centos-7 tests/tests_default.yml
qemu-ansible-core-2.14 installed: ansible-core==2.14.4,beautifulsoup4==4.12.2,cffi==1.15.1,cryptography==40.0.2,Jinja2==3.1.2,MarkupSafe==2.1.2,packaging==23.1,productmd==1.35,pycparser==2.21,PyYAML==6.0,resolvelib==0.8.1,ruamel.yaml==0.17.21,six==1.16.0,soupsieve==2.4.1
qemu-ansible-core-2.14 run-test-pre: PYTHONHASHSEED='2184243967'
qemu-ansible-core-2.14 run-test: commands[0] | python /network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py --image-name centos-7 tests/tests_default.yml
Traceback (most recent call last):
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 1486, in <module>    main()
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 1457, in main
    runqemu(
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 1146, in runqemu
    download_image(image, cache)
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 572, in download_image
    image_path = fetch_image(image_url, cache, image["name"])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 192, in fetch_image
    os.setxattr(image_tempfile.name, URL_XATTR, os.fsencode(url))
OSError: [Errno 95] Operation not supported: '/network/user/.cache/linux-system-roles/tmpjhfyfd3j'
ERROR: InvocationError for command /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/bin/python /network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py --image-name centos-7 tests/tests_default.yml (exited with code 1)
____________________________________________________________________ summary ____________________________________________________________________ERROR:   qemu-ansible-core-2.14: commands failed

when I set
export LSR_QEMU_CACHE=/tmp

I get
$ tox -e qemu-ansible-core-2.14 -- --image-name centos-7 tests/tests_default.yml
qemu-ansible-core-2.14 installed: ansible-core==2.14.4,beautifulsoup4==4.12.2,cffi==1.15.1,cryptography==40.0.2,Jinja2==3.1.2,MarkupSafe==2.1.2,packaging==23.1,productmd==1.35,pycparser==2.21,PyYAML==6.0,resolvelib==0.8.1,ruamel.yaml==0.17.21,six==1.16.0,soupsieve==2.4.1
qemu-ansible-core-2.14 run-test-pre: PYTHONHASHSEED='1239476407'
qemu-ansible-core-2.14 run-test: commands[0] | python /beegfs/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py --image-name centos-7 tests/tests_default.yml
ansible-galaxy [core 2.14.4]
  config file = None
  configured module search path = ['/network/user/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/lib/python3.11/site-packages/ansible
  ansible collection location = /network/user/.ansible/collections:/usr/share/ansible/collections
  executable location = /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/bin/ansible-galaxy
  python version = 3.11.0 | packaged by conda-forge | (main, Jan 14 2023, 12:27:40) [GCC 11.3.0] (/network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/bin/python)
  jinja version = 3.1.2
  libyaml = True
No config file found; using defaults
Starting galaxy collection install process
[WARNING]: The specified collections path '/network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/tmp' is not part of the configured
Ansible collections paths '/network/user/.ansible/collections:/usr/share/ansible/collections'. The installed collection will not be picked up
in an Ansible run, unless within a playbook-adjacent collections directory.
Process install dependency map
Starting collection install process
Downloading https://galaxy.ansible.com/download/ansible-posix-1.5.2.tar.gz to /network/user/.ansible/tmp/ansible-local-477088_duzxz25/tmp_lmic8p6/ansible-posix-1.5.2-cf2serjh
Installing 'ansible.posix:1.5.2' to '/network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/tmp/ansible_collections/ansible/posix'
ansible.posix:1.5.2 was installed successfully
ansible-playbook [core 2.14.4]
  config file = None
  configured module search path = ['/network/user/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/lib/python3.11/site-packages/ansible
  ansible collection location = /network/user/src/ansible-dev/ssh/.tox
  executable location = /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/bin/ansible-playbook
  python version = 3.11.0 | packaged by conda-forge | (main, Jan 14 2023, 12:27:40) [GCC 11.3.0] (/network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/bin/python)
  jinja version = 3.1.2
  libyaml = True
No config file found; using defaults
[WARNING]:  * Failed to parse /network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2 with script plugin: Inventory script
(/network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2) had an execution error: Traceback (most recent call last):   File
"/network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2", line 11, in <module>     import fmf ModuleNotFoundError: No module named
'fmf'
[WARNING]:  * Failed to parse /network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2 with yaml plugin: We were unable to read either
as JSON nor YAML, these are the errors we got from each: JSON: Expecting value: line 1 column 1 (char 0)  Syntax Error while loading YAML.
mapping values are not allowed in this context  The error appears to be in '/network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2':
line 30, column 4, but may be elsewhere in the file depending on the exact syntax problem.  The offending line appears to be:   try:    ^ here
[WARNING]:  * Failed to parse /network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2 with ini plugin: /network/user/src/ansible-
dev/ssh/.tox/standard-inventory-qcow2:9: Expected key=value host variable assignment, got: os
ERROR! Completely failed to parse inventory source /network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2
Traceback (most recent call last):
  File "/beegfs/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 1486, in <module>    main()
  File "/beegfs/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 1457, in main
    runqemu(
  File "/beegfs/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 1172, in runqemu
    run_ansible_playbooks(
  File "/beegfs/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 968, in run_ansible_playbooks
    raise Exception("One or more tests failed")
Exception: One or more tests failed
ERROR: InvocationError for command /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/bin/python /beegfs/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py --image-name centos-7 tests/tests_default.yml (exited with code 1)
____________________________________________________________________ summary ____________________________________________________________________ERROR:   qemu-ansible-core-2.14: commands failed

At least I get rid of the xattr error indicating that xattr are no supported on the network mounts.
Why are xattr actually required here?
I am new to tox and of course also tox-lsr. How do I fix the mentioned issue(s)?

Why are xattr actually required here?

Because we try to cache the qcow2 images, rather than either 1) downloading an 8GB image every time or 2) never downloading them again, and instead relying on you to know when to download a new one.  We don't have a good place to store the cache metadata, so we use xattrs for that.

I am new to tox and of course also tox-lsr. How do I fix the mentioned issue(s)?

The first step is to file the issue here which you have done - thanks!
Looks like /tmp also doesn't support xattrs:
tox -e qemu-ansible-2.9 -- --cache /tmp --image-name ....
...
  File "/home/rmeggins/.local/lib/python3.11/site-packages/tox_lsr-2.14.1-py3.11.egg/tox_lsr/test_scripts/runqemu.py", line 192, in fetch_image
    os.setxattr(image_tempfile.name, URL_XATTR, os.fsencode(url))
OSError: [Errno 95] Operation not supported: '/tmp/tmpsmg38c6f'

Also, you are missing the python fmf library:
 Traceback (most recent call last):   File
"/network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2", line 11, in <module>     import fmf ModuleNotFoundError: No module named
'fmf'

I'm assuming Rocky 9 is compatible with EL9, so you could try using EPEL9 to get fmf https://docs.fedoraproject.org/en-US/epel/ - then dnf -y install python3-fmf
Do you have any local persistent storage?  Perhaps /var/tmp?  Usually /var is some sort of local, non-networked, non-memory file system.
If you have no good place to use as a cache, then I guess you could download the qcow image file to some location, and use the --image-file argument instead of the --image-name argument:
tox -e qemu-ansible-2.9 -- --cache /tmp --image-file /path/to/centos-7.qcow2 ....

I apologize - almost all of the system roles developers use a Fedora or similar laptop as a development environment, and use ~/.cache/linux-system-roles as the cache which assumes /home is a local "regular" file system, etc.
So it will be good to get these differences into the developer docs.

No need to apologize! I am glad you help.
In my case, /tmp seems to support them,
$ getfattr /tmp/centos-7.qcow2c
getfattr: Removing leading '/' from absolute path names
# file: tmp/centos-7.qcow2c
user.dublincore.date
user.xdg.origin.url

$ attr -l /tmp/centos-7.qcow2c
Attribute "selinux" has a 36 byte value for /tmp/centos-7.qcow2c
Attribute "xdg.origin.url" has a 81 byte value for /tmp/centos-7.qcow2c
Attribute "dublincore.date" has a 29 byte value for /tmp/centos-7.qcow2c

In that sense I would say that /tmp is local  (I did a standard install and /etc/fstab has no specific line for tmp thus is corresponds to the / entry) and 'persistant' (at least until it gets cleaned which is 10d in my case).
I installed fmf into the environment (https://pypi.org/project/fmf/), but first ran into https://github.com/tox-dev/tox/issues/2929, i.e. tox-lsr seems to be incompatible with tox=4.*. I restricted tox=3.* and it works.
Maybe these dependencies should go into setup.{cfg,py}. Note, even though I see commits with tox 4 support here, I just ran into this with using git+https://github.com/linux-system-roles/tox-lsr@main as pip install url in my conda environment yml.
In any case, I still ran into
$ tox -e qemu-ansible-core-2.14 -- --image-name centos-7 tests/tests_match.yml
qemu-ansible-core-2.14 recreate: /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14
qemu-ansible-core-2.14 installdeps: productmd, PyYAML, ruamel.yaml, beautifulsoup4, ansible-core==2.14.*
qemu-ansible-core-2.14 installed: ansible-core==2.14.5,beautifulsoup4==4.12.2,cffi==1.15.1,cryptography==40.0.2,Jinja2==3.1.2,MarkupSafe==2.1.2,packaging==23.1,productmd==1.35,pycparser==2.21,PyYAML==6.0,resolvelib==0.8.1,ruamel.yaml==0.17.21,six==1.16.0,soupsieve==2.4.1
qemu-ansible-core-2.14 run-test-pre: PYTHONHASHSEED='2890108930'
qemu-ansible-core-2.14 run-test: commands[0] | python /network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py --image-name centos-7 tests/tests_match.yml
ansible-playbook [core 2.14.5]
  config file = None
  configured module search path = ['/network/user/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/lib/python3.11/site-packages/ansible
  ansible collection location = /network/user/src/ansible-dev/ssh/.tox
  executable location = /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/bin/ansible-playbook
  python version = 3.11.3 | packaged by conda-forge | (main, Apr  6 2023, 08:57:19) [GCC 11.3.0] (/network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/bin/python)
  jinja version = 3.1.2
  libyaml = True
No config file found; using defaults
[WARNING]:  * Failed to parse /network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2 with script plugin: Inventory script
(/network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2) had an execution error: Traceback (most recent call last):   File
"/network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2", line 11, in <module>     import fmf ModuleNotFoundError: No module named
'fmf'
[WARNING]:  * Failed to parse /network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2 with yaml plugin: We were unable to read either
as JSON nor YAML, these are the errors we got from each: JSON: Expecting value: line 1 column 1 (char 0)  Syntax Error while loading YAML.
mapping values are not allowed in this context  The error appears to be in '/network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2':
line 30, column 4, but may be elsewhere in the file depending on the exact syntax problem.  The offending line appears to be:   try:    ^ here
[WARNING]:  * Failed to parse /network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2 with ini plugin: /network/user/src/ansible-
dev/ssh/.tox/standard-inventory-qcow2:9: Expected key=value host variable assignment, got: os
ERROR! Completely failed to parse inventory source /network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2
Traceback (most recent call last):
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 1486, in <module>    main()
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 1457, in main
    runqemu(
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 1172, in runqemu
    run_ansible_playbooks(
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 968, in run_ansible_playbooks
    raise Exception("One or more tests failed")
Exception: One or more tests failed
ERROR: InvocationError for command /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/bin/python /network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py --image-name centos-7 tests/tests_match.yml (exited with code 1)
____________________________________________________________________ summary ____________________________________________________________________ERROR:   qemu-ansible-core-2.14: commands failed

but note
$ conda list | grep fmf
fmf                       1.2.1                    pypi_0    pypi

and
$ python
Python 3.11.3 | packaged by conda-forge | (main, Apr  6 2023, 08:57:19) [GCC 11.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import fmf
>>>

so what is going on here.
Note, I also tried /var/tmp with the same outcome, also getfattr and attr -l show the same as for the qcow2c file under /tmp.
Only after I installed the system package python-fmf it went a bit further
$ tox -e qemu-ansible-core-2.14 -- --image-name centos-7 tests/tests_match.yml
qemu-ansible-core-2.14 installed: ansible-core==2.14.5,beautifulsoup4==4.12.2,cffi==1.15.1,cryptography==40.0.2,Jinja2==3.1.2,MarkupSafe==2.1.2,packaging==23.1,productmd==1.35,pycparser==2.21,PyYAML==6.0,resolvelib==0.8.1,ruamel.yaml==0.17.21,six==1.16.0,soupsieve==2.4.1
qemu-ansible-core-2.14 run-test-pre: PYTHONHASHSEED='2738353912'
qemu-ansible-core-2.14 run-test: commands[0] | python /network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py --image-name centos-7 tests/tests_match.yml
ansible-playbook [core 2.14.5]
  config file = None
  configured module search path = ['/network/user/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/lib/python3.11/site-packages/ansible
  ansible collection location = /network/user/src/ansible-dev/ssh/.tox
  executable location = /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/bin/ansible-playbook
  python version = 3.11.3 | packaged by conda-forge | (main, Apr  6 2023, 08:57:19) [GCC 11.3.0] (/network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/bin/python)
  jinja version = 3.1.2
  libyaml = True
No config file found; using defaults
xorriso 1.5.4 : RockRidge filesystem manipulator, libburnia project.

xorriso : NOTE : Local character set is now assumed as: 'utf-8'
[ERROR] standard-inventory-qcow2: Could not access VM launched from qcow2 image: /tmp/centos-7.qcow2c
qemu exited with code 0 pid 589197
Last lines of centos-7.qcow2c.guest.log:
-----END SSH HOST KEY KEYS-----
[   10.437341] cloud-init[1617]: Cloud-init v. 19.4 finished at Fri, 28 Apr 2023 05:30:58 +0000. Datasource DataSourceNoCloud [seed=/dev/sr0][dsmode=net].  Up 10.43 seconds
[  OK  ] Started Execute cloud user/final scripts.
[  OK  ] Reached target Cloud-init target.

CentOS Linux 7 (Core)
Kernel 3.10.0-1160.45.1.el7.x86_64 on an x86_64

localhost login:
Last lines of centos-7.qcow2c.qemu.log:
qemu-kvm: Machine type 'pc-i440fx-rhel7.6.0' is deprecated: machine types for previous major releases are deprecated
qemu-kvm: terminating on signal 15 from pid 589189 (/usr/bin/python3)

[WARNING]:  * Failed to parse /network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2 with script plugin: Inventory script
(/network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2) had an execution error:
[WARNING]:  * Failed to parse /network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2 with yaml plugin: We were unable to read either
as JSON nor YAML, these are the errors we got from each: JSON: Expecting value: line 1 column 1 (char 0)  Syntax Error while loading YAML.
mapping values are not allowed in this context  The error appears to be in '/network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2':
line 30, column 4, but may be elsewhere in the file depending on the exact syntax problem.  The offending line appears to be:   try:    ^ here
[WARNING]:  * Failed to parse /network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2 with ini plugin: /network/user/src/ansible-
dev/ssh/.tox/standard-inventory-qcow2:9: Expected key=value host variable assignment, got: os
ERROR! Completely failed to parse inventory source /network/user/src/ansible-dev/ssh/.tox/standard-inventory-qcow2
Traceback (most recent call last):
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 1486, in <module>    main()
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 1457, in main
    runqemu(
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 1172, in runqemu
    run_ansible_playbooks(
  File "/network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py", line 968, in run_ansible_playbooks
    raise Exception("One or more tests failed")
Exception: One or more tests failed
ERROR: InvocationError for command /network/user/src/ansible-dev/ssh/.tox/qemu-ansible-core-2.14/bin/python /network/user/.conda/envs/ansible-tox-lsr/lib/python3.11/site-packages/tox_lsr/test_scripts/runqemu.py --image-name centos-7 tests/tests_match.yml (exited with code 1)
____________________________________________________________________ summary ____________________________________________________________________ERROR:   qemu-ansible-core-2.14: commands failed

So it seems that the problem is with the generated .tox/standard-inventory-qcow2.

ok - tox-lsr has been updated to support tox 4, add fmf, and several other fixes.
Not sure it will help in this situation.
Take a look at the files in the artifacts/ directory - you've already looked at the guest.log and qemu.log - also try default_provisioners.log.
Another thing - I noticed when I upgraded to fedora 37, I started to get OOM errors running qemu testing VMs.  Try checking if you are hitting OOM or other system errors (I guess look at journalctl and see if anything jumps out).

I did a fresh install (with tox 4) but I still get the same issue.
Where should I look for artifacts?

Where you found the centos-7.qcow2c.guest.log that you reported in https://github.com/linux-system-roles/tox-lsr/issues/116#issuecomment-1527027907
I guess let's take a step back - can you run qemu/kvm VMs on your machine?  If so, can you run any of the images we use in system roles?

Hmm I never looked at a .guest.log everything I posted above was terminal ouptut.
In any case, I can run VMs.
Moreover I can also run the VM downloaded when using tox-lsr.
Though only after adjusting permissions and only as root.
If I start as regular (network) user I get
Traceback (most recent call last):
  File "/usr/bin/virt-install", line 6, in <module>
    from virtinst import virtinstall
  File "/usr/share/virt-manager/virtinst/__init__.py", line 60, in <module>
    from virtinst.install.installer import Installer
  File "/usr/share/virt-manager/virtinst/install/installer.py", line 12, in <module>
    from . import unattended
  File "/usr/share/virt-manager/virtinst/install/unattended.py", line 18, in <module>
    from . import urlfetcher
  File "/usr/share/virt-manager/virtinst/install/urlfetcher.py", line 16, in <module>
    import requests
  File "$HOME/.local/lib/python3.9/site-packages/requests/__init__.py", line 118, in <module>
    from . import utils
  File "$HOME/.local/lib/python3.9/site-packages/requests/utils.py", line 25, in <module>
    from . import certs
  File "$HOME/.local/lib/python3.9/site-packages/requests/certs.py", line 15, in <module>
    from certifi import where
ModuleNotFoundError: No module named 'certifi'

I just don't know - there's just too many odd things that I don't see on a Fedora system - it's beyond my ability to diagnose without having the exact environment to try to debug


Replace busybox wget with standalone package

Docker container cannot be built in corporate environments using
http proxies due to issue https://github.com/mirror/busybox/issues/21.
This change installs the wget package, so a fully-featured wget
is available for downloading artifacts during container build.
This change could be reverted once the wget issue is resolved.
Fixes #165

done as part of https://github.com/NetApp/harvest/pull/234


Include more information with simulator bug reporting

Right now, simulator has a report bug option which sends the circuit screenshot along with the issue to the slack channel. It will be helpful to include
a) the circuit data and
b) javascript console logs
along with this so that issues could be looked better into.

Yah it will be great i think @gr455

can i work on this issue i am new it will be great if you please elaborate the issue

Is the issue still open ? May I work on it
@gr455

@divyesh-rathod Yes. The issue actually boils down to finding some pastebin site which works with just Client ID (like imgur does) so we don't have to trouble the backend with the paste creation. If you have such a pastebin site in mind, you can tell.


[BUG] Azure.Storage.Blobs V12 is taking very long to upload large file compare to v11.

Library name and version
Azure.Storage.Blobs v12
Describe the bug
For Large files which are > 500MB there is a significant upload time in v12.
The problem is with the UploadAsync method in the blob storage library package.
This was working fine in V11 but in V12, the file upload for large files (>1GB) is taking a lot longer.
BlobClient.UploadAsync Method (Azure.Storage.Blobs) - Azure for .NET Developers | Microsoft Docs
Expected behavior
V11 file upload for a 1GB file takes 1 minute to upload.
V12 should also upload a 1GB file in approximately the same time.
Actual behavior
V12 is taking up to 16 minutes to upload a 1GB file.
Reproduction Steps
Using this method
await destinationFile.UploadAsync(stream);
Environment
.NET

@TimMalik Thanks for reaching out to us and reporting this issue. We are looking into this issue and we will provide an update.

@TimMalik To upload a single large blob quickly, a client application can upload its blocks or pages in parallel, being mindful of the scalability targets for individual blobs and the storage account as a whole. The Azure Storage client libraries support uploading in parallel. For example, you can use the following properties to specify the number of concurrent requests permitted in .NET.
Within the UploadAsync function, you can leverage the  BlobUploadOptions parameter.
The BlobUploadOptions.TransferOptions property helps to configure parallel transfer behavior.
Hope this helps.

@TimMalik I wanted to do quick follow-up to check if you had a chance to look at my above comment. Please let us know if you  had any questions on this. Awaiting your reply.

@TimMalik I hope you had a chance to look at the above comment. The action is pending on you to test the above plan. We will now proceed with closure of this GitHub issue. If you need any further assistance on this issue in future, please feel free to reopen this thread. We would be happy to help.


build failed on freebsd/darwin

github.com/sevlyar/go-daemon/lock_file_unix.go:32: undefined: syscall.PathMax
syscall.PathMax is defined only for linux, nacl and solaris

You are right about FreeBSD, my bad. But are you sure about Darwin? Could you recheck?

You are right, on darwin build passes without problems.


Another blocklist | oisd.nl

Request for extension or provision of a separate DoH/DoT as an adblock list using the following block list
oisd.nl

Thanks in advance

Hey there, that is a very good filterlist which is also included in adblock-dns. What is the usecase just to enable this list separately ?

Hey, was that filter list already in adblock-dns or did you add it now? That's not quite clear to me now.
Will this list be applied in adult-filter-dns #8 as soon as it is finished? Wait for DoT deployment...

This list has been included since couple of months. And yes, adult-filter dns is gonna have same filterlist as adblock dns and along with adult filterlists.

Hey, was that filter list already in adblock-dns or did you add it now? That's not quite clear to me now.
Will this list also be applied to adult-filter-dns #8 once it is finished? Wait for DoT deployment...

Adult-filter dns is finally implemented, read me page should have info regarding it.


Camera Controls

Right now we use the mouse scrolling to zoom in and out. There is no bound to how much the user can zoom out which is awesome. But there should be a key binding to recenter the camera to a default angle.
This button can also house different other camera angles like many of modern games do. e.g. GTA V's 'V' Key toggles to a First Person, a zoomed out angle, a very zoomed angle and a cinematic-randomly-changing mode and even a MOBA like angle

Yeah, I like V.

I think we can literally do the GTA model (did they invent that back in 3?).
The hard part (probably a few hours of thinking) is the cinematic camera, but the rest should be easy to implement.


Combining buffers into one large buffer - still necessary?

Hello. Apologies if this isn't the right place to ask such questions, but I've not seen links to any kind of support area.
The Nvidia documentation suggests that it's preferable to to combine buffers into one large buffer and use offsets/ranges within that large buffer to get the best performance. Is this still necessary and/or possible with vma? I'm not sure from the documentation if vma handles this kind of suballocation for me or not.

Yes, it is preferable to use larger buffers, but VMA doesn't support this out-of-the-box. What VMA does is it allocates larger VkDeviceMemory blocks and manages them to sub-allocate memory for whole VkBuffer and VkImage objects. Managing contents of such objects - like regions of a single Vulkan buffer - would be like a second-level sub-allocation and it's out of scope of this library. It is possible with VMA, you just need to do it yourself.