upgrade to node 18

This PR:

sets all the CI values for node to 18
updates the nix package to one that's compatible with node 18
changes install commands to ignore peer dependency errors (npm got stricter about that)

There must be something in the scheduler function lambda code that Node 18 dislikes that we may need to resolve. https://stackoverflow.com/questions/69692842/error-message-error0308010cdigital-envelope-routinesunsupported (the accepted answer is pretty bad! the next one offers a saner explanation).
Perhaps for the sake of getting this in, we can keep it at 16 in the github task-runner yaml for lambda build and open a separate issue to resolve this (this would be code found at backend/packages/Scheduler). This is a lower priority than other things, as this function is a nice-to-have that is rarely (never?) utilized.

There must be something in the scheduler function lambda code that Node 18 dislikes that we may need to resolve. https://stackoverflow.com/questions/69692842/error-message-error0308010cdigital-envelope-routinesunsupported (the accepted answer is pretty bad! the next one offers a saner explanation).
Perhaps for the sake of getting this in, we can keep it at 16 in the github task-runner yaml for lambda build and open a separate issue to resolve this (this would be code found at backend/packages/Scheduler). This is a lower priority than other things, as this function is a nice-to-have that is rarely (never?) utilized.

Ok, that makes sense. I suspect upgrading to webpack 5 might fix the problem, but that's not usually a trivial thing to do.

ah couple notes, the terraform scripts (and any AWS-related configs in the repo) are no longer what SRE wants to use. These scripts are in purgatory, they should be removed from the open-source project (see https://carnegielearning.slack.com/archives/C051CUABJKA/p1680032439060529) for some conversation and apparently a task i never followed up on. These were only really used to launch new resources, and managed in AWS thereafter. So, no updates there will have any effect.
Also, the lambda build in github workflows should get set to 16 there, as the error is github-job-runner trying to build it using 18. Although setting dockerfiles to 18 will get us cooking for frontend / backend, I forgot that the lambdas we have will need updates in AWS manually for the actual runtimes, no docker there.

https://github.com/CarnegieLearningWeb/UpGrade/issues/982

ah couple notes, the terraform scripts (and any AWS-related configs in the repo) are no longer what SRE wants to use. These scripts are in purgatory, they should be removed from the open-source project (see https://carnegielearning.slack.com/archives/C051CUABJKA/p1680032439060529) for some conversation and apparently a task i never followed up on. These were only really used to launch new resources, and managed in AWS thereafter. So, no updates there will have any effect.
Also, the lambda build in github workflows should get set to 16 there, as the error is github-job-runner trying to build it using 18. Although setting dockerfiles to 18 will get us cooking for frontend / backend, I forgot that the lambdas we have will need updates in AWS manually for the actual runtimes, no docker there.

I'm not seeing the error anymore. But if the AWS console (or cli) needs to be involved, I still don't have access.


Core: Target JDK8

We have to decide when to drop JDK7 support that has reached EOL.
As Gatling is a load test tool where performance is a must, and modern JVM always bring performance improvements (such as G1), we already recommend using JDK8 anyway.
Some of our dependencies have already switched or plan to:

Typesafe config (as of 1.3.0, already released): https://github.com/typesafehub/config/tree/v1.3.0#binary-releases
Scala 2.12: http://scala-lang.org/news/2.12-roadmap

Note: Scala 2.11.7 will introduce the ability to use JDK8's LambdaMetaFactory to generate lambdas : https://github.com/scala/scala/pull/4463
This could bring (possibly huge) performance and memory usage improvements.

Dependencies available in Scala 2.12 (current is 2.12.0-M2):

[]   scala-swing
[]   scala-xml
[]   scala-parser-combinators
[]   akka-actor
[]   fastring
[]   scopt
[]   scala-logging
[]   jsonpath
[]   boopickle
[]   redisclient
[]   scalatest
[]   scalacheck


Support unconfined apparmor

We should support AppArmor unconfined after
https://github.com/kubernetes/kubernetes/pull/52395 is merged.

kubernetes/kubernetes#52395 has been merged.
@miaoyq @yanxuean Are you interested in this?

@Random-Liu  I'll fix this.


Proposal: Boilerplate frontend component

Description

Proposal for sending to  you. A refernce  for boilerplate template  but that needs to customised for kaia i built it for shardeum

Bounty track

Contribute an Open Source Developer Tool

this bounty paid?

you both can hunt for this bounty. But don't copy one another.
there are a lot of boilerplate ideas. Think beyond smart contract, add in some frontend as well.
@anudeep11k can you elaborate on your boilerplate, like the features you want to include.
@Faizalhidayat you can create a new boilerplate proposal for yourself.

My proposing boilerplate consists of
Components
NFTBalances
KRC20 token Balances
Krc20 token Transfers
NFTTransfers
Transactions

these are frontend components? @zxstim just approve that already senpai 🙇

Approved to proceed. Revert back with links.

Can you also edit the title of this Issue to be more descriptive on your boilerplate? Like frontend component boilerplate or something.

Yeah. Sure .
Im currently working on it.

Can i submit the frontend which i have done for Erc20 supoorted networks ?.
But to work on klaytn it needs a provider and moralis substitute since moralis doesnt support klaytn.

what did you use moralis for?

For getting transactions for getting Nft balances and token balances

hello, here is the link for boilerplate repo please check and revert back : https://github.com/anudeep11k/ethereum-boilerplate.git


Don't warn about funcs that want to use a specific type

For example:
func WriteToSomeStruct(s SomeStruct) {
    b := make([]byte, 10)
    s.Write(b)
}

We shouldn't be recommending io.Writer here.

Just to clarify, that is if the type name appears in the function name - doesn't have to be at the end. Case insensitive.

Even if the function name doesn't include the struct name, but takes the argument by value, using an interface should not be recommended as that will cause an allocation.

So you're saying this is correct?
func Foo(s SomeStruct) {
    s.Close()
}

(ignoring the fact that this is closing a copy of the struct)

Yes.

Just because of the overhead of encapsulating the struct into an interface? I don't see why passing by value or by reference makes any difference in that aspect.
If you were to say that encapsulating as an interface is like passing by reference, then I do see why passing by value is very different and shouldn't trigger a warning. In which case I'll change the example from SomeStruct to *SomeStruct.

And open a separate issue to avoid those warnings, of course.

Yes, just because of the overhead. Encapsulating in an interface doesn't imply passing by reference (it depends if the value or the pointer implements the interface), but it results in an allocation either way.
I don't have a good source for this, but that's how it is.

Sure, it results in an allocation because that's how interfaces work in Go. But if we're to drop all of those, then this tool becomes rather useless because it would only suggest changes from interfaces to narrower interfaces. A huge part of the improvements this tool suggests (and which are valid) rely on struct types which sholud be interfaces.
I still don't see why you don't care about the overhead if the structs were passed by reference before.

Because if they're passed by reference then they're already on the heap,
and wrapping them in an interface won't allocate.
On Mar 8, 2016 10:38, "Daniel Martí"<EMAIL_ADDRESS>wrote:

Sure, it results in an allocation because that's how interfaces work in
Go. But if we're to drop all of those, then this tool becomes rather
useless because it would only suggest changes from interfaces to narrower
interfaces. A huge part of the improvements this tool suggests (and which
are valid) rely on struct types which sholud be interfaces.
I still don't see why you don't care about the overhead if the structs
were passed by reference before.
—
Reply to this email directly or view it on GitHub
https://github.com/mvdan/interfacer/issues/17#issuecomment-193827300.

Ah, I get what you mean now. I thought if you passed a local struct variable as an interface type it would just use the pointer to that local variable. If what you say is true, then it probably makes sense to do this change.

@tamird I just opened #19 for the passing by value thing. If you have any suggestions, they'll be welcome :)
I also can't seem to find any reference to this copying-to-heap overhead when passing by reference. Would be useful to find it. Although if you're sure about it, I'll trust you since I myself had no idea.

Perhaps I don't fully understand, but even passing by reference causes this warning, e.g.:

// Produces: foo.go:77:22: fh can be io.Writer
func createCSVWriter(fh *os.File) *csv.Writer {
	outputWriter := csv.NewWriter(fh)
	outputWriter.Write([]string{
		"...",
	})

	outputWriter.Flush()

	return outputWriter
}

So why is this not recommended.. ?

@Dynom I don't understand what you mean. That warning is expected and AFAIK is correct. This issue is about the warning not being shown e.g. if the func name was createCSVWriterFromFile.

Ah apologies, I figured the warning is incorrect.
Can you explain why my example is incorrect?

NewWriter() takes an io.Writer. All you're doing with fh is passing it to NewWriter(). Hence it can be an io.Writer.

So if I understand it correctly, the only reason Interfacer complains is because the function doesn't actually do anything specifically with os.File. I suppose that makes sense, although I don't think I necessarily agree with it.
I understand the simplification. However I expect the function to work with file handles, not writers in general, even though it might not reflect the current behaviour per se.

This is why I'm opening these issues, to detect when the developer really means *os.File and doesn't want io.Writer. What this issue suggests is for you to mention File in the function name somewhere.
What else would you do? I need blacklisting rules like these. Otherwise, interfacer would do nothing at all.

Makes total sense. I'll just play along for now see how I feel about it.
Thanks for your time and efforts! It's greatly appreciated and I did help me out a few times already :+1:


failing fs access in web validator

fixes #544 (a regression caused by merge conflict resolution here).
we now use typeof window === 'undefined' (isNode vs fs) as a check for node vs web environment

CircleCI now generates web version of the validator for each PR (see in artifacts). You can grab that link and ask the person reporting the issue if this solves it for them.

@chrisfilo that's rad!


InspectorControls

Looks like InspectorControls is now under wp.editor . Not sure when that changed.
wp.editor.InspectorControls

Breaks:
const { InspectorControls } = wp.blocks;

Thanks, we are pinned to a version back for this client now, I can add a fix for this later today

Sorry about not getting to the other ones, been low priority for me :(

No worries.

Looks like this applies to RichText as well wp.editor.RichText

https://github.com/crossfield/gutenblock/releases/tag/1.0.0
Should be all set, I am continuing to test on our site

Nice thanks for turning that around :) my blocks work again now.


Update ScaleBarSample

Updating the format spacing of this sample.

@tschie Can you look this over? Need change the spacing format to be like the other samples so that it will display properly on dev site.


BUG:  Example checker has problems suppressing type information

The example-checker.ipynb has problems supressing type information.  Instead of displaying 0.0, it will display np.float64(0.0).  Part of the issue was to format arrays properly, but in doing so we now get all kinds of extra type information being displayed.
It would be nice if the example checker could automate supressing type information, while still maintaining a nice clean format for arrays.  One option might be array2string, which I tried with little success when first creating the checker. It needs more work.  For now, you have to remove the extra type information, which is a bit better than trying to properly format output and smashing the space bar.

The new example checker found in the tools/review/ folder has tried to address this a bit. It correctly handles multiline output and arrays (even if on one line), but there could be some instances where a single value returned ends up showing repr() instead of print() value. With time, it should work the way we want it.

[ ] I'll mark this complete when I don't see type information showing up in examples for a week.

I saw some today, so this still has more work.


added multipart specs, finalize to ZMQ::Message, refactored send and …

…receive message to use  LibZMQ.msg_recv and  LibZMQ.msg_send as older are deprecated according 0mq docs

Very nice cleanup! Thanks again!


Will the results be published regardless of outcome?

This is a major concern for me,  I'd hope that the results are publicly published regardless of if they appear as the researches expected.

Yes! This is an open data project and we'll be publishing the data under an open source license for the open source community and researchers to use. We may hold back some data if there are privacy concerns, but otherwise we plan to release the full data set with accompanying docs on sampling methodology etc.

@franniez can i ask when do you plan to publish the data ? and if you will notify survey participants ?
I just wantto make sure i won't miss it !

@piercus Since the survey is totally anonymous, we don't have a way of notifying participants, but you can sign up for updates at opensourcesurvey.org. I think we'll be able to release the data in May, but since we're still in data collection it's hard to predict.


Change the default download for customized templates

This is a user request. When a user downloads a completed DMP from a customized version of the DCC template, the box that reads: "supplementary section not requested by funding orgs" should default to be checked.
I think this makes sense to implement, as if the org has added additional questions to their customized template these questions should appear in the downloaded version.
I realize this is explanation may be confusing, so I can walk you though it to demonstrate.

This has been updated. You can test by changing your Org to UCOP and then creating a plan using the DCC default template (which UCOP has customized)


[BugFix] Fix 'ErrorMsg' attached to the wrong operator

push_chunk returns error, so the error must be attached to the next_op rather than curr_op
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[x] 3.0
[x] 2.5

[FE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[BE Incremental Coverage Report]
:heart_eyes: pass : 1 / 1 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
be/src/exec/pipeline/pipeline_driver.cpp
1
1
100.00%
[]

@Mergifyio backport branch-3.2 branch-3.1 branch-3.0 branch-2.5


Composes trusses to sharing GPUs

Is your feature request related to a problem? Please describe.
After some exploration, it seems like we can definitely composes various trusses into one image so that can share a GPU. This has many advantages for server costs, but also for local development when I want to work with multiple trusses, but my machine only happens to have one GPU.
This is issue is to open up a discussion on what could be the goals for bringing this into truss, both in the near term and long term.

https://github.com/basetenlabs/truss/tree/main/examples/gpu-sharing


Enums: keep original value option

Describe the bug
There isn't a way to keep the literal enum values from the oas spec.
The external api may require the enum to be exactly the same as what is defined.
Regardless of the options the values always seem to be sanitized to uppercase + underscore.
Proposal

Map the enum values to valid values
Store the mapping in state somehow
Reverse the mapping of the values before the request is made

OR
I think it would be much more useful to add a fetch override option to the configuration -
so the consumer can implement more complex solutions / fix assumptions made by the library

More easily map custom resolvers
Add delegated resolvers
Linking is currently near impossible with message envelopes
Many more

No idea how complicated this would be within the library, I'm trying to avoid attempting it
Looking for thoughts on this
To Reproduce
Example
TextFilter:
      description: Text filter
      required:
        - key
        - operator
        - value
      properties:
        key:
          type: string
          description: Key on an object
          enum:
            - id
            - name
            - assetType
        operator:
          type: string
          enum:
            - Equals
            - Does not equal
            - Contains
            - Does not contain

Expected behavior
I need the value posted to the external api to be "Does not equal" rather than "DOES_NOT_EQUAL"

@PseudoSky Thanks for the issue and sorry for the late response. Could you verify that, indeed, the sanitzation/desanitization is not working? OtG is supposed to sanitize enum values to ALL_CAPS (to reflect GraphQL conventions), and desanitize back to the original values when the API call is made. I just want to make sure that this is not a misunderstanding.
If indeed, desanitization is functioning incorrectly, then this could potentially related to #274 where I discuss our naive approach to sanitization/desanitization mapping. However, if that is the case, I would assume that we would still see some kind of desanitization going on, just to an incorrect value.
However, I do agree, there probably should be an option that allows you to control whether the values are sanitized or not. I believe there are some strict limitations on the characters you can use in enum values but besides that, we can try to preserve as much of the original value as possible.

More easily map custom resolvers

We do offer custom resolvers via the customResolvers option. You can find that option here. However, the enum values will be sanitized. If I recall correctly, we also expose the sanitize() and desanitize() functions.

Add delegated resolvers

To be honest, I am not very familiar with schema delegation but it seems to me that there isn't a clean, straightforward way to do this programmatically without some external configuration and at that point, customResolvers is probably the best way forward.

additionally this is undesirable behavior for some. for example the GeoJSON standard specifies that enum names must be in titlecase, eg type: "Point". I want my generated GraphQL schema to permit the query and input of valid GeoJSON types that must have this casing regardless of the GQL norms.

This should be resolved via #365


Linux UFW firewall rules

Is your feature request related to a problem? Please describe.
I want to allow localsend in UFW firewall. What ports should I allow?
Describe the solution you'd like
UFW rule file for localsend application.
Describe alternatives you've considered
No response
Additional context
No response

Hello @wizard1989
Here is what I found to assist you going forward.
To allow the "localsend" application through the UFW firewall, you'll need to allow the specific ports that "localsend" uses. If you're unsure which ports "localsend" utilizes, you can typically find this information in the documentation or configuration files of the application. Once you've identified the ports, you can create UFW rules accordingly.
Assuming "localsend" uses TCP and UDP protocols on port 12345, here's how you can create UFW rules for it:

Allow TCP traffic on port 12345:
sudo ufw allow 12345/tcp

Allow UDP traffic on port 12345:
sudo ufw allow 12345/udp

If "localsend" uses additional ports or protocols, you'll need to add rules for those as well following the same syntax.
After adding these rules, make sure to enable UFW to apply the changes:
sudo ufw enable

Here's how you would create a UFW rule file specifically for "localsend" application, assuming it uses TCP and UDP on port 12345:
# localsend UFW rule file
# Allow TCP traffic on port 12345
-A ufw-user-input -p tcp --dport 12345 -j ACCEPT

# Allow UDP traffic on port 12345
-A ufw-user-input -p udp --dport 12345 -j ACCEPT

Save this content to a file, for example, localsend.ufw, and then you can apply these rules using the ufw command:
sudo ufw --file localsend.ufw enable

This will enable the UFW rules specifically for the "localsend" application. Make sure to adjust the port numbers and protocols according to the requirements of your specific application.

Thank you!


Additional labels in the helm charts

Please check the FAQ documentation before raising an issue

Describe the bug (required)
Below is snippet of line 26 in nebula-operator/charts/nebula-cluster/templates/nebula-cluster.yaml
  labels: {{ toYaml .Values.nebula.graphd.podLabels | nindent 6 }}

Can someone please take one more look at this?
IMHO, it should be something like following!
  labels: {{ toYaml .Values.nebula.storaged.labels.additionalProperties | nindent 6 }}

Your Environments (required)

OS: uname -a
Commit id (e.g. a3ffc7d8)

How To Reproduce(required)
Steps to reproduce the behavior:

Step 1
Step 2
Step 3

Expected behavior

Additional context

@porscheme Is there an error running here?

We have noticed that the issue you created hasn’t been updated for nearly a month, so we have to close it for now. If you have any new updates, you are welcome to reopen this issue anytime.


Created doc for Show automatic jobs statement and updated relevant docs

Closes #4585
Still working on the SQL diagram but wanted to open the PR to get the content reviewed.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/90d7e783379eb8478ee224025ec72f65c3c4c7c9/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/4dd5200436ca1f3e94cac68ff28e84efe5f8decd/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/bfa2bd4675c6968adcb2ff2dc03e1ea6bbcd156d/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/da3bb422fea956813410ce9fbda64d6b8d777bdb/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/73e189de40520198cf475f06b32129017fe263a7/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/ff1ef5029b78a4359c556ba44de433caa944051c/

Discussed offline. We're going to merge these pages.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/57daa08707668210c03d73050d44ce73b785c245/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/3a3f188fb8139c529e5de3d4583e3b1890381940/

@rytaft @jseldess Merged the docs as discussed. PTAL

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/c7edb6f00d81dca4627a324bde349cc514b29a59/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d793f88d34b34eb1d08be08db953f7f443dc1c37/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/33fe906783106ba0cead8c81f88dfb7865272400/

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/960d67bbb73c81ed6b65b3af6880726b93fe26a3/
Edited pages:

v19.1/admin-ui-jobs-page.md
v19.1/cancel-job.md
v19.1/pause-job.md
v19.1/show-jobs.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/35852d203d732ecf3d93b5865aa7d4d7fdfcd1ef/
Edited pages:

v19.1/admin-ui-jobs-page.md
v19.1/cancel-job.md
v19.1/pause-job.md
v19.1/show-jobs.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/aad56c16022dcbd2bea39969328eeccab2df3da1/
Edited pages:

v19.1/admin-ui-jobs-page.md
v19.1/cancel-job.md
v19.1/pause-job.md
v19.1/show-jobs.md


error ns-process-data images

Describe the bug
Running COLMAP bundle adjustment... (This may take a while)──────────────────────────────────────────────  💀 💀 💀 ERROR 💀 💀 💀  ───────────────────────────────────────────────
Error running command: colmap mapper --database_path /workspace/nerfdata/068/colmap/database.db --image_path
/workspace/nerfdata/068/images --output_path /workspace/nerfdata/068/colmap/sparse --Mapper.ba_global_function_tolerance
1e-6
────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
ERROR: failed to create sparse model

I stumbled upon this error, too. I'm on a MacBook Air M2 AARCH64, PyTorch with mps support for silicon. I tried installing COLMAP by downloading a precompiled release, which runs, but I couldn't make it work for python scripts. So I installed COLMAP via brew and it seems to kinda work. Now I got exactly this error, too.

I have the same error. Any solutions found so far?

I have this error too :( any solutions?

same problem

I am also getting this error, I'm just not getting the "ERROR: failed to create sparse model" message. Have you tried running this with the --verbose flag to see extended logs? I personally don't get any extra information from this though...

Did you fix the error by using more video data?

I used different videos for different scenes, so I got this error. Once I changed one video like shooting arround the same scene, the error is fixed.

Okay, I am currently just using a single video but maybe I was moving the camera too fast, I guess I’ll try with another video

Not sure if this is helpful but for me, CUDA was out of memory. I was able to get feature matching and sparse point cloud by closing other memory-consuming scripts.

 How to deal with it?
nerfstudio) root@autodl-container-8b8a4382f2-a9434246:/#  ns-process-data images --data /root/autodl-tmp/mtree3-305/images --output-dir /nerfstudio/data1/mtree3
[16:13:59] 🎉 Done copying images with prefix 'frame_'.                                        process_data_utils.py:340
[16:15:38] 🎉 Done extracting COLMAP features.                                                       colmap_utils.py:137
[16:23:40] 🎉 Done matching COLMAP features.                                                         colmap_utils.py:151
[16:39:49] 🎉 Done COLMAP bundle adjustment.                                                         colmap_utils.py:173
🎉 Done refining intrinsics.                                                              colmap_utils.py:184
einsum(): the number of subscripts in the equation (2) does not match the number of       process_data.py:526
dimensions (1) for operand 1 and no ellipsis was given

I encountered the same error, at first I thought it was because of the data or that colmap didn't extract features. But after several experiments I found it was because that I didn't set up visible cuda devices. For me using instructions like export CUDA_VISIBLE_DEVICES=0 works. You could try debugging to see if the feature is correctly extracted using colmap feature_extractor --database_path processed_data_path/database.db --image_path processed_data_path (change the corresponding processed_data_path to your directory). If no matches were found, try colmap exhaustive_matcher --database_path processed_data_path/colmap/database.db to further locate the problem and it is where I found that colmap couldn't find the proper devices.

 This can appear if the number of images are too few <10

same issue! have you fixed it?


Unable to generate bot response. Details: Error: 400: => Invalid request: The request is not valid, HTTP status: 404

Describe the bug
A clear and concise description of what the bug is.i have followed the read me file and changed it to use OpenAI in the appsettings.json the key is set but when i launch using yarn start in the copilot when i write in generates that error
To Reproduce
Steps to reproduce the behavior:

Go to semantic-kernel/samples/apps/copilot-chat-app/webapi
write dotnet user-secrets set "AIService:Key" "MY_AZUREOPENAI_OR_OPENAI_KEY"
continue with the read me steps when it launches write something
See error

Expected behavior
it is supposed to generate  a response
Desktop

OS: windows 11
IDE: Visual Studio, VS Code
NuGet Package Version  0.1.0

@IhebGhazala - are you still experiencing this issue? If so, could you confirm the information I requested above?

Hello, we have the same issue with our deployment.
We are using AzureOpenAI. We have deployed a Static web app for the front-end and using an azure web app for the API.
Bug description:
When we type any prompt to CopilotChat, we receive an HTTP 400 error. On the web application, we get the following error;
Unable to generate bot response. Details: Error: 400: Bad Request => Invalid request: The request is not valid, HTTP status: 404
I confirmed the payload seems to be valid.
appsettings.json
  "AIService": {     "Type": "AzureOpenAI",     "Endpoint": "myendpoint.openai.azure.com/", // ignored when AIService is "OpenAI"     "Key": "mykey",     "Models": {       "Completion": "gpt-35-turbo", // For OpenAI, change to 'gpt-3.5-turbo' (with a period).       "Embedding": "text-embedding-ada-002",       "Planner": "gpt-35-turbo" // For OpenAI, change to 'gpt-3.5-turbo' (with a period).     }   },
Do you have any idea what could be the problem?

@martinoez My first thought is to try prefixing the endpoint with https://. My next thought is to ensure that those models are all indeed available from your endpoint. I'll keep digging into what else it could be.

Hi Gina, thanks for getting back to me. The https:// prefix was already there, I just removed it when I posted the comment.
I confirmed on Azure OpenAI Studio, the models gpt-35-turbo and text-embedding-ada-002 are successfully deployed.

Hello Gina,
I am also facing the same issue (local deployment for Azure OpenAI). Required models are deployed and is showing a green check mark.

Error:
fail: Microsoft.SemanticKernel.IKernel[0]
Invalid request: The request is not valid, HTTP status: 404: Microsoft.SemanticKernel.AI.AIException: Invalid request: The request is not valid, HTTP status: 404
---> Azure.RequestFailedException: The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.
Status: 404 (Not Found)
ErrorCode: DeploymentNotFound
  Content:
  {"error":{"code":"DeploymentNotFound", "message":"The API deployment for this resource does not exist. If

you created the deployment within the last 5 minutes, please wait a moment and try again."}}
  Headers:
  OpenAI-Processing-Ms: REDACTED
  x-ms-client-request-id: b21ed67a-460f-4b99-82e8-2e6f1863b148
  apim-request-id: REDACTED
  Strict-Transport-Security: REDACTED
  X-Content-Type-Options: REDACTED
  x-ms-region: REDACTED
  Date: Tue, 11 Jul 2023 11:52:58 GMT
  Content-Length: 198
  Content-Type: application/json

     at Azure.Core.HttpPipelineExtensions.ProcessMessageAsync(HttpPipeline pipeline, HttpMessage message, RequestContext requestContext, CancellationToken cancellationToken)
     at Azure.AI.OpenAI.OpenAIClient.GetChatCompletionsAsync(String deploymentOrModelName, ChatCompletionsOptions chatCompletionsOptions, CancellationToken cancellationToken)
     at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.RunRequestAsync[T](Func`1 request)         --- End of inner exception stack trace ---
     at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.RunRequestAsync[T](Func`1 request)         at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.InternalGetChatResultsAsync(ChatHistory chat, ChatRequestSettings chatSettings, CancellationToken cancellationToken)
     at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.InternalGetChatResultsAsTextAsync(String text, CompleteRequestSettings textSettings, CancellationToken cancellationToken)
     at Microsoft.SemanticKernel.SkillDefinition.SKFunction.<>c__DisplayClass24_0.<<FromSemanticConfig>g__LocalFunc|1>d.MoveNext()

fail: Microsoft.SemanticKernel.IKernel[0]
Function call fail during pipeline step 0: ChatSkill.Chat. Error: Invalid request: The request is not valid, HTTP status: 404
info: Microsoft.AspNetCore.Hosting.Diagnostics[2]
Request finished HTTP/2 POST https://localhost:40443/chat application/json 300 - 400 - text/plain;+charset=utf-8 1282.8842ms

Hello Gina,
I am also facing the same issue (local deployment for Azure OpenAI). Required models are deployed and is showing a green check mark.

Error: fail: Microsoft.SemanticKernel.IKernel[0] Invalid request: The request is not valid, HTTP status: 404: Microsoft.SemanticKernel.AI.AIException: Invalid request: The request is not valid, HTTP status: 404 ---> Azure.RequestFailedException: The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again. Status: 404 (Not Found) ErrorCode: DeploymentNotFound
  Content:
  {"error":{"code":"DeploymentNotFound", "message":"The API deployment for this resource does not exist. If

you created the deployment within the last 5 minutes, please wait a moment and try again."}}
  Headers:
  OpenAI-Processing-Ms: REDACTED
  x-ms-client-request-id: b21ed67a-460f-4b99-82e8-2e6f1863b148
  apim-request-id: REDACTED
  Strict-Transport-Security: REDACTED
  X-Content-Type-Options: REDACTED
  x-ms-region: REDACTED
  Date: Tue, 11 Jul 2023 11:52:58 GMT
  Content-Length: 198
  Content-Type: application/json

     at Azure.Core.HttpPipelineExtensions.ProcessMessageAsync(HttpPipeline pipeline, HttpMessage message, RequestContext requestContext, CancellationToken cancellationToken)
     at Azure.AI.OpenAI.OpenAIClient.GetChatCompletionsAsync(String deploymentOrModelName, ChatCompletionsOptions chatCompletionsOptions, CancellationToken cancellationToken)
     at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.RunRequestAsync[T](Func`1 request)         --- End of inner exception stack trace ---
     at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.RunRequestAsync[T](Func`1 request)         at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.InternalGetChatResultsAsync(ChatHistory chat, ChatRequestSettings chatSettings, CancellationToken cancellationToken)
     at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.InternalGetChatResultsAsTextAsync(String text, CompleteRequestSettings textSettings, CancellationToken cancellationToken)
     at Microsoft.SemanticKernel.SkillDefinition.SKFunction.<>c__DisplayClass24_0.<<FromSemanticConfig>g__LocalFunc|1>d.MoveNext()

fail: Microsoft.SemanticKernel.IKernel[0] Function call fail during pipeline step 0: ChatSkill.Chat. Error: Invalid request: The request is not valid, HTTP status: 404 info: Microsoft.AspNetCore.Hosting.Diagnostics[2] Request finished HTTP/2 POST https://localhost:40443/chat application/json 300 - 400 - text/plain;+charset=utf-8 1282.8842ms

Just an FYI - Application Client ID from AD and the Azure OpenAI endpoint are from two different Azure subscriptions.

Thanks for reporting this issue and finding the solution!
Yes, for OpenAI you will need to specify the model name, whereas if you use Azure OpenAI you will need to specify the deployment name. I will update the ReadMe to reflect this.

Addressed by https://github.com/microsoft/semantic-kernel/pull/1956.


Battery level information incorrect

Hello,
In Home Assistant, my battery shows as 84% full, but when I connect to my lock with the Nuki app, it shows 20%. I have already deleted the MQTT lock and cleared all entries in my MQTT database using MQTT Explorer, but it still shows the battery as 84% full.
Nuki Hub: firmware 9.01
Nuki Firmware: 3.10.7

Please check using an MQTT client like MQTT explorer what value you receive for battery level after executing a lock action (nuki/lock/battery/level).

Hi. I had the same problem until I noticed the path has changed and the old value was there from a previous build:
Old: Old: mqtt.0.nuki.battery.level
New: mqtt.0.nuki.lock.battery.level

What does it mean? Do you must change that in the next release? I have already cleared all previous entries with mqtt explorer.

I'm using iobroker and had a UI element tied to that path. It was still showing the old value which was not updated anymore. For home assistant no idea, I don't use it. I guess auto discovery should point to the correct path. Maybe try clearing all data under under /nuki in MQTT and restart Nuki Hub. Maybe @iranl has an idea if the home assistant integration can cause some issue.

I have a suspicion: when the battery has the status “charging,” the hub might receive an incorrect value from the lock. Could that be possible?

Definitely not a NukiHub issue based on the information provided.
Could be the NukiBLE library interpreting the raw data from the device incorrectly (because of recent firmware changes by Nuki?)
But seeing as firmware 3.10.7 is not really new I would be amazed if this is a general issue with this firmware that no one else has reported this earlier.
To debug this issue you would have to provide a USB/serial log generated with a debug build of NukiHub (switch to a debug build by browsing to http://NUKIHUBIP/otadebug and selecting a release/beta/master build)

I have now replaced my batteries, and the battery level is showing 100%. The MQTT path should be correct. I believe the issue occurs when the battery status is 'charged,' as it seems to send an incorrect level to the hub.

How would the battery status be "charging" when not using a Power pack?
Or do you mean by "I have now replaced my batteries" that you have replaced your power pack?

I don't know, i don't use a powerpack, but the app had shows me "charging"

Definitely sounds like a Nuki Lock 3 firmware bug in that case as it should never be in "charging" state when not using a Power Pack.
Closing as not NH related.

@isibizi I've created a debug build:
https://github.com/technyon/nuki_hub/actions/runs/11952124998
After flashing, you'll get an additional node next to "level" called "debug". This node will publish the raw "critical battery state" information (it's a bit of an odd name). This byte contains the battery level, and two more flags. From the API:
Bit 0	Smart Lock Battery State Critical
Bit 1	Charging Flag
Bit 2-7	Smart Lock Battery in %
It's published as decimal, so it has to be converted to binary manually. Please post the value here together with the reading in your app (level, charging, battery critical).

I think the problem is, the zip file contains only the esp32 firmware, i have a c3

Hi. You're right there must have been some problem with the build job. I rebuilt the branch, this one has all assets:
https://github.com/technyon/nuki_hub/actions/runs/11971198737

Hello Jan-Ole,
I replaced my batteries and reinserted the old ones. In the app, I changed the automatic detection setting to manual and selected alkaline. Currently, the battery level appears to be correct. I can flash my hub with the debug firmware, but I don’t think debugging is possible with the current data. What do you think?

Hi,
if the bug doesn't occur anymore, there's no point in testing.

Okay, I’ll keep an eye on it. If the error occurs again, I’ll post here. In the app, I no longer see the “charging” status; it currently shows the status “critical.”

Critical means your batteries are almost empty, is that correct?

Yes, right! My old batteries had 18%, i think everything under 20% have the state „critical“ I replaced it with new ones again.


system error

hi. When i install fourseven:scss and go meteor command, i receive an system error:
Assertion failed!
Programm:...odules\node-sass\vendor\win32-ia32-11\binding.node
File:....\src\libsass\src\json.cpp
Expression: utf8_validate(str)
Please help.

Can you retry with the latest version?

Closing due to inactivity

Getting the same error. Maybe caused by trying to run meteor inside Windows 10 in Virtualbox.


extract events from a new dataset using your code?

Hi,
Can you provide guideline of how to extract events from a new dataset using your code?

Hi,
Any update on this? :)


Doesn't work with the latest ARK

ERROR: Unable to rename C:/Program Files (x86)/Steam/steamapps/common/ARK/ShooterGame/Saved/SavedArksLocal/LocalProfile.local.arkprofile. Check your files/permissions and try again.
LocalProfile.local.arkprofile doesn't exist anywhere.

I'll take a look at this tonight so the version should be the same, but in the future you should really include the ARK version number instead of saying latest as the latest version changes often.


Problem regarding this package on npmjs.com

https://www.npmjs.com/package/nodobjc (currently version 2.1.0)
vs
https://www.npmjs.com/package/NodObjC (currently version 1.0.0)
and the latter comes up when searching for "nodobjc"

The package name with uppercase letter (NodObjC) is deprecated and will no longer be updated. If you try to npm install NodObjC, you will be greeted with a deprecation warning. The lowercase nodobjc is now preferred.


Android: adding swift suffix to icu generated libs to resolve conflict

Android: adding swift suffix to icu generated libs to resolve conflit with Android's internal icu libs

https://bugs.swift.org/browse/SR-5672

Resolves SR-5672.

I missed the original reasoning here. Why not use Android's ICU libraries? Are they not accessible, or not guaranteed to be available? Also, is this shipping ICU with or without symbol renaming? Either way, there will likely be symbol clashes when both are loaded into memory.

LibswiftCore uses symboles which doesn't exist in Android icu libs, but they exist in our generated icu libs. My PR makes sure that we add suffix to libs name only and not to internal symbols

Was that due to a mismatch in symbol renaming configuration, or does Android suppress ICU symbols? I'm asking because Linux may want to make a similar choice of trade offs.
Either way, this PR LGTM.

@swift-ci please smoke test

If I rename the ICU symbols, i'll get linking error, as libswiftCore call the original symbol names without my suffix. Android doesn't suppress ICU symbols.

Then why not link against Android's?

To be clear, shipping your own ICU is a totally valid approach and probably the right one for Android (and maybe Linux too). I'm just wondering if this is an intentional decision or an accidental result of misunderstanding ICU version pinning through symbol renaming. Either way, out of scope for this PR.

Android uses a different version ICU libs which doesn't have all the symbols that libswiftCore uses.


support register services through glob expressions.

Create a support to register services automatically, avoiding the annoying task to register each service, as previously describe in #21, or in #9.
The approach would be:
  Server.loadServices(app, 'lib/controllers/*');

or
  Server.loadServices(app, ['lib/controllers/*', '!**/exclude*']);
  // includes all controllers, excluding that one which name starts with 'exclude'

or
  Server.loadServices(app, 'controllers/*', `${__dirname}/..`]);
  // Inform a folder as origin for the patterns

@thiagobustamante Could you explain why you named method "loadServices"? So we have controllers right now. What are other types of "services" exists?

I just used 'services' because we already have a method called Server.buildServices() that receives the annotated classes directly... I thought it would be more orthogonal.
Server.buildServices(app, MyRestService, MyRestService2);
// or
Server.loadServices(app, 'controllers/*');

But I think we can have  an alias... I understand that a lot of people should prefer 'controller'... What do you think if we create this alias?
Server.loadControllers(app, 'controllers/*');

Yeah, "loadControllers" is good and it comes from MVC pattern. And another point of view is - this project depends on "express" framework i think we could use the same naming convention as it described in official documentation. I see that word 'HANDLER' is what describes the thing we annotate with @Path decorator. We also can use "loadHandlers" or "loadRestHandlers" too. But anyway i think you decision is more preferable. And i understand why you choosed "loadServices" because those classes are "serve" REST requests. No problemo. Just wanted to know the truth :call_me_hand:


Update azure-vm-get-sas-uri.md

Changed date in header.

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

Changed date in header. #sign-off

@v-jrushing: I'm sorry - only the author of this article, @iqshahmicrosoft, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

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

We'll go ahead and merge this. Please make future updates in the private repo. (You can make quick edits from the live site in the private repo. Let me know if you want a link to the internal contributor guide, which describes how to do it. Thanks.)


fix: normalize url in ensureServingAccess

Description
Fix https://github.com/vitejs/vite/issues/3346
What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

Looking forward to this being released!


'media-control-notransition' class works wrong

Starting with commit 41b2897d2169e8c799871977d9d045abc05379cd, the MediaControl setSeekPercentage method does it wrong removing the 'media-control-notransion' class.
This negatively affects the experience of dragging the seek-bar scrubber.
Adding this class in the rendering method is also useless.

Hi, @sapolio. Can you please open this issue on the clappr-plugins repo? This helps a lot with the bug control between the projects inside the Clappr org.


Use Harfbuzz for shaping

This adds proper text shaping using Harfbuzz. I've tested with several complex Indian scripts and it its output is comparable to other software as long as an appropriate font is used. It works quite well with the FreeType code path. Windows is yet untested. I still need to figure out how to handle the dependency there.

Looks like the Harfbuzz library available via EDM (version 2.6.4) does not have GDI support. So no support on Windows for now. Guess I'll chase that down in a future PR...

Ready to move on with this one...


Develop

認可の参考実装を追加。

@sugaryo Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


What granular read permission is required to be able to see the Monitor logs in Azure function on the azure portal

[Enter feedback here]

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

ID: 74d4c770-e59d-63bc-3710-7ed2c8867a75
Version Independent ID: a770673c-f4bc-ad56-8154-04dfb5f52ad4
Content: Stream execution logs in Azure Functions
Content Source: articles/azure-functions/streaming-logs.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@rohitiscancerian Thank you for your feedback! We will review and update as appropriate.

@rohitiscancerian It looks like you are added as the reader permission only to the function app. In case if you want to read the monitoring data you need to add the Monitoring Reader permission on your application insights. For more details about the different built-in RBAC roles you can always refer to this document.
I was able to reproduce the issue if I only assigned reader permission on my function app. Once I added the Monitoring Reader permission to my configured application insights the Monitor option was enabled on my function app.
MicrosoftDocs Github issue is mainly related to the document issue. I would recommend you to create a thread on the forums - Microsoft Q&A or Stack Overflow for any how to/troubleshooting issue.
Feel free to get back to me if you have any queries or concerns.

@rohitiscancerian Thanks for your response. Feel free to reach out to us in Microsoft GitHub for any document related issue.
For any troubleshooting/how to question you can always reach out to us in Microsoft Q&A or Stack Overflow


a problem about dataset

Hello,
I noticed that your article mentions the collection of datasets with 1143, 1055, and 963 molecules in MED1, NPM1, and HP1α droplets, respectively, and that you provided the data file probe_screen-data.xlsx. However, it seems that the article does not specify how these datasets are divided, nor does it mention which data are held out. I have reviewed the data file, but the molecule counts for each droplet do not seem to match those mentioned.
Therefore, I would like to understand your method for dividing the datasets and how to derive the training, validation, and test datasets for the deep learning models described in the readme.md from this data file. Your assistance in clarifying these points would be greatly appreciated.
Thank you!

Hi,
Thanks for reaching out!
As mentioned in the methods, the molecules were assigned to the training set (80%), validation set (10%) or test set (10%) using a scaffold split. Specifically, the code in this README provides the exact parameters used to split the data and define which ones are held out.
Since not all molecules could be processed through RDKit and Chemprop, 1143, 1055, and 963 refers to those that could be processed and were used for developing the model.


Issue with importing nested components/sub components

From App.vue, components can be imported like in the example files:
import MyComponent from './components/MyComponent'
However, importing a component from MyComponent, or any other imported component (a child component/nested component/subcomponent) needs to imported like this:
import MyNestedComponent from '../components/MyNestedComponent'
Now I suspected that the import file path from MyComponent may have been relative from MyComponent.vue, but that turns out not to be the case, as this does not work:
import MyComponent from 'MyNestedComponent'
I'm not sure if this is a bug, but I couldn't find documentation on this.

Nevermind, this was me not understanding the relative file path syntax of ./
./ means relative to the current file. Knowing this, I can see how this totally makes sense.


Automatic codebase backup

Summary
I request an important, and useful, feature, which would allow Multihack users to quickly restore files lost by the synchronizer (which happens quite commonly, to be honest): automatic backup creation.
Description
This feature would be capable of automatically creating a backup of a project's changes (or the whole project itself) everytime a change is made to a file. This backup would likely be a compressed tarball, with additional metadata (to describe the project and room name, for identification), and would be saved in a Multihack-specific folder (such as %APPDATA%\Multihack on Windows, or ~/.multihack on Unix).
If the backups are deltas, one can simply use an initial state tar, load it to a temporary folder, and apply the deltas to the folder, one by one, until we reach the desired backup state.
This would make the plugin more productive (and usable), since the loss of a file would be not but a small nuisance, instead of a worrisome event.
Versions

OS: Windows 7 64-bit.
Brackets: Release 1.13, build 1.13.0-17696 (release 49d29a8bc). Built on Thursday, January 14th 2018, 08:13:54 GMT+01:00.

The CRDT data model used for sync is in the process of being completely replaced (the sync errors are mostly due to the buggy model we're using now.
Traditional deltas applied to a tarball wouldn't converge, but the idea of "snapshot" backups is sound. I'll keep this in mind for the next iteration.

You're welcome to try and patch this version of Multihack, but I'm more interested in solving the issues in the underlying CRDT and editor bindings so we don't lose data or get desyncs at all.
This was a first attempt at realtime collab, so there's some glaring design flaws in my approach here (I knew very little about CRDTs or even the Brackets extension API when this was written). Everything is in the process of a ground-up redesign.


Health check not waiting for selenium hub to be up

First of all thanks for this service.
disclaimer: I am very new to webdriverio and javascript.
Issue:
docker service doesn't wait for hub to be up when using "zalenium"
Details
I am using zalenium docker images to start hub/node, and using health property for docker-service. Since zalenium take sometime to start hub and register node , docker service is not waiting for it to be up
reason for this behaviour is,  health end point doesnot wait for the service to return 200OK which is kind of make sense as well after reading fetch documentation , but right now I am stuck with this problem.
Fix:
Ideal fix: would be to take a call back in health options and wait for it to be complete.
Quick fix: would be to implement checkStatus function in health , similar to this
If you agree this is issue and should be fixed, I can take first stab at fixing this.

@kapilag Option healthCheck is designed to be configurable for cases like that. If you feel that Zelenium takes longer to start simply adjust the delay like so:
healthCheck: {
    url: 'http://localhost:4444',
    maxRetries: 3,
    inspectInterval: 1000,
    startDelay: 2000
}

This setup should cover scenario even for Zelenium.
Another thing you want to pay attention to is the way test is written. For example if test is not waiting for a browser to open a url it will run through test before it is ready. Make sure you return promises and or wait for await to complete.

Yes I have used the same solution mentioned by you as of now , but start
delay was something I thought we can get rid off.
Also zalenium gives a status URL, which return status of hub/nod in
response body, Which is the correct way of checking whether hub/node is up
Similar functionality can be provided by different services like selenoid
or other and call back function should be ideal solution? What do you think?
On Sat, 24 Aug, 2019, 8:25 PM Simon Tsvilik<EMAIL_ADDRESS>wrote:

@kapilag https://github.com/kapilag Option healthCheck is designed to
be configurable for cases like that. If you feel that Zelenium takes longer
to start simply adjust the delay like so:
healthCheck: {
url: 'http://localhost:4444',
maxRetries: 3,
inspectInterval: 1000,
startDelay: 2000
}
This setup should cover scenario even for Zelenium.
Another thing you want to pay attention to is the way test is written. For
example if test is not waiting for a browser to open a url it will run
through test before it is ready. Make sure you return promises and or wait
for await to complete.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stsvilik/wdio-docker-service/issues/57?email_source=notifications&email_token=AA7ARBRTGP5SUJZJJBDQ6GDQGFDX5A5CNFSM4IPFTJB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5CBWEA#issuecomment-524557072,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA7ARBT5JKKD3DNRYQK4TGDQGFDX5ANCNFSM4IPFTJBQ
.

@kapilag Are you suggesting to wait on Docker events for built-in health check monitoring before starting a test? As an alternative to polling for service start using fetch?

@stsvilik :not really docker events, if you go to zalenium page over here , there is section having status url which is : http://localhost:4444/wd/hub/status
Response for this url would:
      "status": 0,
      "value": {
        "ready": true,
        "message": "Hub has capacity",
        "build": {
          "revision": "6e95a6684b",
          "time": "2017-12-01T19:05:32.194Z",
          "version": "3.8.1"
        },
        "os": {
          "arch": "amd64",
          "name": "Linux",
          "version": "4.9.49-moby"
        },
        "java": {
          "version": "1.8.0_151"
        }
      }
    }

So ideally user of docker service would like to wait for "value.ready=true" in case of zalenium, now this can be different for different service.
If docker-service provides call backs function which get executed untill condition is true or < timeout, then it is upto user to implement such logic and wait for value.ready=true.

@kapilag I think a custom healthecheck function is a doable option, however I'm not sure when I would be able to look into this.

Are we still interested in this feature?
If I understand correctly, we'd want to have healthCheck be a function that returns a Promise that resolves when the service is ready, and rejects when it times out. Based on the example response from Zalenium above and reworking the polling from https://github.com/webdriverio-community/wdio-docker-service/blob/37e7ba717b47aa407f6e8f8d69987c40701aed66/src/utils/docker.js#L140.
I believe that'd be something like:
/**
 * Resolves when Zalenium indicates the service is ready (`value.ready` is `true`).
 * Rejects when the service is not ready before `timeout` milliseconds.
 */
const healthCheck = async ({ interval = 500, timeout = 10_000 }) => {
  const start = Date.now();

  for (let elapsed = 0; elapsed < timeout; elapsed = Date.now() - start) {
    const response = await fetch('http://localhost:4444/wd/hub/status');
    const body = await response.json();
    if (body?.value?.ready === true) {
      return;
    }
    await setTimeoutPromise(interval);
  }

  throw new Error(`Timed out waiting for Zalenium after ${timeout.toLocaleString()} ms`);
};


Versioning of Binable.Of_binable

Most versioned types use deriving bin_io. A few use include Binable.Of_Binable (M) (...), where the serialization depends on existing code in M. If the serialization of M changes, then the serialization of the containing module changes.
Therefore, we should

check that M is a specific versioned module
detect changes where Of_binable is applied

Specifically, we need to detect changes to the version of M, and the to_binable function.

This was completed by #4534. Closing.


The html built by the scraper has invalid css

If you can even get the scraper to work the resulting html output is barely human readable. If fixing this is not feasible then outputting the scraped data as json might be better. The blur is post processing to keep confidentiality of the slack's contents.

ah okay sure but the resulting html file is still poorly formatted and not really human readable.


4.0.0 - Customization set issue

SPT Version
4.0
Project Type
Server
Expected Result
(logs might not be that helpful, confirmed in VC whats wrong)
When setting what upper and lower to use, this would be set and used.
Actual Result
When setting it atm, the request gives the type as "suite", in the code CustomizationController setClothing it uses that as a Key, which is now incorrect,
when getting an Upper, we need to set the Body and Head.
when getting a Lower, we need to set the Feet only.
Steps To Reproduce

make an account, (this should happen to all types)
buy an available outfit, (i brought the instructor set as Bear)
Equip said outfits, (this will be fine till you recieve a response with the profile in again)
You can run a raid to force this, or restart.

To unfuck the profile, go PMC.Customization and remove "suite" key and value.
Server Log
server-2024-12-30.log
BepinEx Log
LogOutput.log
Client Log
2024.12.30_15-52-34_<IP_ADDRESS>.34501 errors.log
Player Profile
6772ad66000363deb1fb02ec.json
Screenshots
No response

Can confirm fixed with some testing


Footer not resizing

My footer is not resizing when using the following

Unfortunately this library doesn't currently support <ion-textarea> components inside a <ion-header> or <ion-footer>. PRs are welcome to fix this particular issue.

Seems to be working ok for me in this setup.
<ion-footer>
  <ion-toolbar>
  	<ion-item class="message-input">
    	<ion-textarea fz-elastic></ion-textarea>
	</ion-item>
   </ion-toolbar>
<ion-footer>

Thanks for the confirmation @adamkearsley. Closing as this appears to work as expected.


Revert "[version] Update to the latest test package"

Reverts flutter/flutter#50818
This broke the flutter_test_performance test:
2020-02-14T22:01:50.575468: RunningProcesses{pid: 21486, commandLine: /home/flutter/.cocoon/flutter/bin/cache/dart-sdk/bin/dart --enable-vm-service=20000 --no-pause-isolates-on-exit --disable-service-auth-codes bin/tasks/flutter_test_performance.dart --cloud-auth-token=ya29.c.Ko8BvQe4XuOPxxv1697CnvRjUCrMA6adSphoMn7ilXU-bH87KrT8hqRleAgmDxngrYSvn2dMmLQAKvd0L47dRwzF2gZ7l1R7OJ7l2iVn0Zjt06JwfsrXgXxYgqojoWy0v34nLtqe-44ilZ_tOL-oyzW9DssY1ojxa9BOUrBizYdAM_vOqS7o1csjI3R2WG5cPhc, creationDate: 2020-02-14 22:01:49.000}
RunningProcesses{pid: 30800, commandLine: dart bin/agent.dart ci -c config.yaml, creationDate: 2020-01-13 16:38:59.000}
2020-02-14T22:01:50.596949:
Executing: /home/flutter/.cocoon/flutter/bin/flutter test flutter_test/trivial_widget_test.dart in /home/flutter/.cocoon/flutter/dev/automated_tests
2020-02-14T22:01:51.068236: test stdout (TestStep.starting): Shuffling test order with --test-randomize-ordering-seed=0
2020-02-14T22:01:51.154935: test stdout (TestStep.starting):
2020-02-14T22:01:52.151464: test stdout (TestStep.testWritesFirstCarriageReturn): 00:00 +0: loading /home/flutter/.cocoon/flutter/dev/automated_tests/flutter_test/trivial_widget_test.dart
2020-02-14T22:01:53.149852: test stdout (TestStep.testLoading): 00:01 +0: loading /home/flutter/.cocoon/flutter/dev/automated_tests/flutter_test/trivial_widget_test.dart
2020-02-14T22:01:54.149891: test stdout (TestStep.testLoading): 00:02 +0: loading /home/flutter/.cocoon/flutter/dev/automated_tests/flutter_test/trivial_widget_test.dart
2020-02-14T22:01:55.149956: test stdout (TestStep.testLoading): 00:03 +0: loading /home/flutter/.cocoon/flutter/dev/automated_tests/flutter_test/trivial_widget_test.dart
2020-02-14T22:01:56.149811: test stdout (TestStep.testLoading): 00:04 +0: loading /home/flutter/.cocoon/flutter/dev/automated_tests/flutter_test/trivial_widget_test.dart
2020-02-14T22:01:57.149867: test stdout (TestStep.testLoading): 00:05 +0: loading /home/flutter/.cocoon/flutter/dev/automated_tests/flutter_test/trivial_widget_test.dart
2020-02-14T22:01:57.245430: test stdout (TestStep.testLoading): 00:06 +0: loading /home/flutter/.cocoon/flutter/dev/automated_tests/flutter_test/trivial_widget_test.dart
2020-02-14T22:01:57.359343: test stdout (TestStep.testLoading): 00:06 +0: A trivial widget test
2020-02-14T22:01:57.376776: test stdout (TestStep.testRunning): 00:06 +1: A trivial widget test
test stdout (TestStep.testRunning): 00:06 +1: All tests passed!
2020-02-14T22:01:57.447013: "/home/flutter/.cocoon/flutter/bin/flutter" exit code: 0
2020-02-14T22:01:57.456721: Task failed: Exception: flutter test rendered unexpected output (1 bad lines)2020-02-14T22:01:57.456799:
2020-02-14T22:01:57.457019:
Stack trace:
2020-02-14T22:01:57.474783: bin/tasks/flutter_test_performance.dart 94:5               runTest
===== asynchronous gap ===========================
dart:async                                                 _asyncErrorWrapperHelper
package:flutter_devicelab/framework/framework.dart         _TaskRunner._performTask.<fn>
package:stack_trace                                        Chain.capture
package:flutter_devicelab/framework/framework.dart 154:11  _TaskRunner._performTask
package:flutter_devicelab/framework/framework.dart 88:41   _TaskRunner.run

TBR @christopherfujino

LGTM.


The useLocation().query object values should return string or Array

Qwik Version
qwik 0.11.0, qwik city 0.0.112
Operating System (or Browser)
chrome
Node Version (if applicable)
16.14.2
Which component is affected?
Qwik City
Expected Behaviour
The url example.com?brand=amd&brand=intel&category=cpu contains two "brand" query params and one "category" query param.
useLocation().query should return { brand: ['amd', 'intel'], category: 'cpu' }
Actual Behaviour
useLocation().query returns { brand: 'intel', category: 'cpu' }, only returning the last "brand" query param.
Additional Information
demo https://stackblitz.com/edit/qwik-starter-7bcp3c?a=1&file=src%2Froutes%2Findex.tsx

The query object is now a URLSearchParams as of https://github.com/BuilderIO/qwik/pull/2385
Thanks for opening the issue!


Localize TestPlatform.TranslationLayer nuget package

Hi @Faizan2304, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Faizan Ahmad). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

This change is for RTM+.
Tagging @KevinH-MS @tannergooding for any comments w.r.t usage in LUT repo.


Does it support react-native-reanimated?

It seems that the babel transformation for 'worklet' functions is almost similar to what reanimated plugin transforms so will they clash if we use both plugins together? And will react-native-reanimated will also be supported while using this?
Also does it support multiple background threads for running JS functions or create only one?

it is working i verified , closing the issue

it is working i verified , closing the issue


add empty dataset userInteractionEnabled delegate support

Hi,
I recently add the empty dataset to my collection view, and my collection view has a header to show some informations, buttons and so on. When the empty dataset show, I don't want it cover my collection view header, so I use offset, but the empty dataset view is actually covered the whole collection view. And the userInteractionEnabled for empty dataset is true, so any interaction occurred in collection view header would be prevent, so I have to disable the userInteraction in empty dataset view, but there is no such view exported or delegate to use, so I added this delegate. I'd like to have some your options about this situation. Thank you very much!
Yours, Hao

@dzenbot YES! Firstly, I used emptyDataSetShouldAllowTouch: delegate, but nothing happened, after I reviewed source code, I found it only disables the button interaction. Maybe the name emptyDataSetShouldAllowTouch: is more suit for disabling the whole view. I would make this change, and re-push. Thanks.


GLES2 not working on some Android devices

Godot version:
3.1 -beta1
OS/device including version:
Xiaomi Redmi 6 - Android Oreo
Issue description:
When playing a GLES2 exported game on android there is nothing displayed. Just a grey screen, no render although if there is sound it plays it.
Steps to reproduce:

Export the minimal repro project to android using the GLES2 renderer.
Play it

Minimal reproduction project:
GLES2.zip

I'm finding Godot 3.1 beta 4 working well on most Android devices I've tried but experienced exactly the same issue with a Nexus 7 2012 (CyanogenMod 12.1) as qonnop. Splash screen shows, then a blank screen. The scene was just a Node2D and a sprite with the Godot Logo.

I tested on my two phones, and can confirm the issue on one of them.

Not working: Samsung S3, GPU ARM Mali-400 MP4, running LineageOS 14.1
Working: Xiaomi Pocophone F1, GPU Qualcomm Adreno 630, running MIUI <IP_ADDRESS> / Android 8.1.0

Relevant adb logcat output when running above demo on my Samsung Galaxy S3 (note that I've removed the default environment and made sure to import ETC to rule out potential issues with those):
02-25 14:13:48.450 30239 30256 D libEGL  : loaded /system/lib/egl/libEGL_mali.so
02-25 14:13:48.485 30239 30256 D libEGL  : loaded /system/lib/egl/libGLESv1_CM_mali.so
02-25 14:13:48.845 30239 30256 D libEGL  : loaded /system/lib/egl/libGLESv2_mali.so
02-25 14:13:48.868 30239 30259 I godot   : Android modules: Nothing to load, aborting
02-25 14:13:48.933 30239 30260 E         : Device driver API match
02-25 14:13:48.933 30239 30260 E         : Device driver API version: 29
02-25 14:13:48.933 30239 30260 E         : User space API version: 29
02-25 14:13:48.933 30239 30260 E         : mali: REVISION=Linux-r3p2-01rel3 BUILD_DATE=Tue Jul 22 19:59:34 KST 2014
02-25 14:13:48.937 30239 30260 I OpenGLRenderer: Initialized EGL, version 1.4
02-25 14:13:48.937 30239 30260 D OpenGLRenderer: Swap behavior 1
02-25 14:13:48.940 30239 30259 W GodotView: creating OpenGL ES 2.0 context :
02-25 14:13:49.018 30239 30259 I godot   : OpenGL ES 2.0 Renderer: Mali-400 MP
02-25 14:13:49.903  2362  2396 W WindowManager: Window freeze timeout expired.
02-25 14:13:49.904  2362  2396 W WindowManager: Force clearing orientation change: Window{4a7fef5 u0 Starting test.com}
02-25 14:13:49.904  2362  2396 W WindowManager: Force clearing orientation change: Window{78be32e u0 test.com/org.godotengine.godot.Godot}
02-25 14:13:49.904  2362  2396 W WindowManager: Force clearing orientation change: Window{1196e03 u0 com.android.systemui.ImageWallpaper}
02-25 14:13:50.154  2362  2396 E WindowManager: Performed 6 layouts in a row. Skipping
02-25 14:13:50.263  2362  4431 E WindowManager: Performed 6 layouts in a row. Skipping
02-25 14:13:50.274  2362  2396 W WindowManager: App freeze timeout expired.
02-25 14:13:50.296  2362  2396 I WindowManager: Screen frozen for +2s393ms due to Window{4a7fef5 u0 Starting test.com}
02-25 14:13:50.438 30239 30259 E godot   : **WARNING**: Could not create framebuffer!!
02-25 14:13:50.439 30239 30259 E godot   :    At: drivers/gles2/rasterizer_storage_gles2.cpp:4259:_render_target_allocate() - Could not create framebuffer!!

Here's a report from the OpenGL ES Caps Viewer app with my phone's supported OpenGL ES 2.0 extensions: http://opengles.gpuinfo.org/displayreport.php?id=3407

I guess the issue is that my phone doesn't support https://www.khronos.org/registry/OpenGL/extensions/OES/OES_required_internalformat.txt, while we hardcode an enum value from this extension in https://github.com/godotengine/godot/blob/31656301b2cfe3710a1bfe8d636eb8206d31dd7b/drivers/gles2/rasterizer_storage_gles2.cpp#L57
For WebGL we use a different branch, so on Android we should likely check for the extension before trying to force its use. https://github.com/godotengine/godot/blob/31656301b2cfe3710a1bfe8d636eb8206d31dd7b/drivers/gles2/rasterizer_storage_gles2.cpp#L4521

I'll test locally to confirm this hypothesis.

Confirmed, using GL_DEPTH_COMPONENT or GL_DEPTH_COMPONENT16 on Android fixes the issue for me.
@reduz is actually working on a big changeset around similar issues, so I'll wait for his changes to try again and maybe make a PR.

I tested again with e47b4bdc, which includes a32b26df which removed the problematic _DEPTH_COMPONENT24_OES above.
It works fine on Xiaomi Pocophone F1, but still fails on Samsung Galaxy S3, so it didn't fix this bug.
I'll have to dig into it again and find how to fix it in the new code.

Ah nevermind, a32b26d only removed one instance of _DEPTH_COMPONENT24_OES in https://github.com/godotengine/godot/commit/a32b26dfa26f2a039bf9c84b90d10666bcf785c9#diff-edd2de5c7988aafce34e98cc791e6274L526
There's still another one at framebuffer creation.

Fixed by 01a3dd3.

Google Nexus 7 is still not working with this fix. It seems to be a problem with the shadow buffer in this case ("Directional shadow framebuffer status invalid") for NVIDIA Tegra 3:
D/libEGL  (20580): loaded /system/lib/egl/libEGL_tegra.so
D/libEGL  (20580): loaded /system/lib/egl/libGLESv1_CM_tegra.so
D/libEGL  (20580): loaded /system/lib/egl/libGLESv2_tegra.so
I/OpenGLRenderer(20580): Initialized EGL, version 1.4
D/OpenGLRenderer(20580): Enabling debug mode 0
V/WindowManager(  510): Adding window Window{2c2d725f u0 SurfaceView} at 4 of 12 (before Window{31e53080 u0 com.example.gles2/org.godotengine.godot.Godot})
W/GodotView(20580): Trying ConfigChooser fallback
W/GodotView(20580): creating OpenGL ES 2.0 context :
I/godot   (20580): OpenGL ES 2.0 Renderer: NVIDIA Tegra 3
E/godot   (20580): **ERROR**: Directional shadow framebuffer status invalid
E/godot   (20580):    At: drivers\gles2\rasterizer_scene_gles2.cpp:3332:initialize() - Directional shadow framebuffer status invalid
I/ActivityManager(  510): Displayed com.example.gles2/org.godotengine.godot.Godot: +1s803ms
E/godot   (20580): **WARNING**: Could not create framebuffer!!
E/godot   (20580):    At: drivers\gles2\rasterizer_storage_gles2.cpp:4335:_render_target_allocate() - Could not create framebuffer!!

Seems to be related to #25568... Is it worth opening a new issue?

Nope, just note in the linked issue that you have this problem.

@qonnop I would open a new issue, your issue and #25568 seem different. (In your case, it's on mobile and prevents the game from running, while on #25568 it's on desktop and the game runs fine even with that error.)

Fixed by 01a3dd3.

For the reference, I've just tried it on my Samsung Galaxy S3 and I confirm that the MRP from this issue works fine now.

Reopening, 057d3e8 broke it again.

From adb logcat:
02-28 10:28:24.633 19422 19445 W GodotView: creating OpenGL ES 2.0 context :
02-28 10:28:24.750 19422 19445 I godot   : OpenGL ES 2.0 Renderer: Mali-400 MP
02-28 10:28:24.957 19422 19445 E godot   : **ERROR**: Directional shadow framebuffer status invalid
02-28 10:28:24.957 19422 19445 E godot   :    At: drivers/gles2/rasterizer_scene_gles2.cpp:3330:initialize() - Directional shadow framebuffer status invalid
02-28 10:28:26.755 19422 19445 E godot   : **WARNING**: Could not create framebuffer!!
02-28 10:28:26.757 19422 19445 E godot   :    At: drivers/gles2/rasterizer_storage_gles2.cpp:4335:_render_target_allocate() - Could not create framebuffer!!

Re-fixed by 8799f69.


Scrolling improvements

Add RepeatButton
ScrollBar has {line,page}/{up/down} buttons
Slider track buttons
Fixed a bug with mouse wheel coordinates

Fixes #822

At the moment it scrolls by a fixed amount.

Actually, it already scrolls by percentage to the viewport size (see here. I just made the default small change 5% and the large 20% :) You're right it should be a whole page.

Also: there are some failing tests.

@grokys I force pushed an update to my branch to clean up unnecessary changes (and rebased onto master).

btw it's intended that scrollbar keys are focusable?

@Sorien no, we should fix that

Was #822 fixed by this?

Yep. Closed


DB-44 Menu pointer and menu entries are done.

@Gragas88 @MrChampion91

DB-44 Start menu pointer and entries.

DB-44 Start menu pointer and entries.


HM-2986-expired-link

I suggest we close this pull request and use https://github.com/AusDTO/dto-digitalmarketplace-supplier-frontend/pull/329 so that the commit history is cleaner.


go/runtime/client: Recheck failed blocks and implement max tx age

Fixes: https://github.com/oasisprotocol/oasis-core/issues/3412

runtime client remembers failed roothash blocks and rechecks them on every new received block
additionally transactions that do not succeed in 1500 blocks are considered failed and client drops them

should this be configurable?
as a future improvement should the client also support SubmitTxNoWait operation (similar to the one in consensus layer)?
todo: would be nice to test this somehow not sure how really - maybe create a test client with very low expiry (1 block), and make sure that all transactions fail?

should this be configurable?

Yeah probably.

as a future improvement should the client also support SubmitTxNoWait operation (similar to the one in consensus layer)?

Good idea, open an issue.


Determine how options should be listed on items

When selecting a single item, options should be available, but how to construct them is currently unknown. We'll need to figure this out.

It was determined that this was due to selList not being properly - the children id and name within values must be in their own parent node together. (This was resolved in 8551811cfdc3f681e20d7cb417eda92b64965a0e.)
For example, to have two options, one would need to do the following with our current setup:
"selList": {
    "values": {
        "option": [
            RepeatedElement(
                {
                    "id": 1,
                    "name": "First Option",
                }
            ),
            RepeatedElement(
                {
                    "id": 2,
                    "name": "Second Option",
                }
            ),
        ]
    }
},

The channel will query with action type webApi_basket_list to denote adding an item. The value of selList will be only be set as a query parameter upon the second call.


Failure policy for Jobs

Feature Description

One-line feature description (can be used as a release note):  Failure policy for Jobs
Primary contact (assignee): @soltysh
Responsible SIGs: @kubernetes/sig-apps-misc
Design proposal link (community repo): https://github.com/kubernetes/community/pull/583
Reviewer(s) - (for LGTM) recommend having 2+ reviewers (at least one from code-area OWNERS file) agreed to review. Reviewers from multiple companies preferred:
Approver (likely from SIG/area to which feature belongs): @erictune
Feature target (which target equals to which milestone):

Alpha release target (1.7)
Beta release target (1.8)

Belated issue in the features repo but we have agreed on working on this in sig/apps for 1.7. @soltysh is OOO today so I am filling this out for him.
https://github.com/kubernetes/community/pull/583

This won't make it for 1.7, I'm removing the milestone. I'll be reworking this issue to be a single point of truth about all job-related enhancements and its current state.

@soltysh @kargakis any plans to make it for 1.8?

Currently being worked on by @clamoriniere1A (https://github.com/kubernetes/kubernetes/pull/48075)

/cc

@clamoriniere1A @soltysh any updates for 1.8? Is this feature still on track for the release?

@idvoretskyi #48075 and #51153 that implement this feature was merged 3days ago.

@kow3ns there are several reasons this went straight to v1 and not beta or alike. First of all this can be seen more like a bigger bug fix (see https://github.com/kubernetes/kubernetes/issues/27997) rather than a feature. Secondly, lots of users where pointing us that this is the most desirable extension of a job (see https://github.com/kubernetes/kubernetes/issues/30243, https://github.com/kubernetes/kubernetes/issues/43512, https://github.com/socialwifi/kubepy/issues/18, https://twitter.com/b0rk/status/902627552195436547, as well as from talking to users during KubeCons), frequently mistaken with the restartPolicy on a pod template.
Finally, I was hoping to change this feature issue to a general job feature describing the future extensions of a job scrambled all over the k8s issues, didn't get to it yet though :/ I think I'll just open a new one to track it properly.
I hope that clears your doubts.

@idvoretskyi yes it's already merged. I'll go ahead and close this issue in that case.

@soltysh Can you please add the k8s.io Docs PR for this feature for 1.8 release here: https://docs.google.com/spreadsheets/d/1AFksRDgAt6BGA3OjRNIiO3IyKmA-GU7CXaxbihy48ns/edit#gid=0

@soltysh I'm not saying it shouldn't go in as is. It's just listed as alpha, but it will be enabled by default on the GA surface. So for the purposes of communication/documentation I don't see how we can call it an alpha feature.

@soltysh I'll reopen the issue, we need it to keep open to track it.
Thanks :)

@kow3ns hmm... I see what you mean, now. Not sure how to answer it, though.


travis: add Go tip for testing

It would be good to test etcd with the latest Go branch,
so that we can find bugs either in Go or etcd in advance.
Reference:
https://docs.travis-ci.com/user/languages/go

LGTM


FEATURE - Additional writer for manifest

As a developer,
I want to write changes to the manifest,
so that a subsequent update of the UI (Fiori Tools: Page Editor) shows the right content
Description
open-ux-tools already provides the FPM writer for writing extensions.
But the given scenario is not part of the Flexible Programming Model, thus a new template as part of the FPM writer might be misleading.
The scenario in detail is a switch of a Fiori List report from a single table to multiple views.
Most urgent for OData V4, but a later extension to the equivalent V2 scenario (quickVariantSelectionX) should be kept in mind.
In the given scenario, several steps need to take place:

creation of SelectionPresentationVariants for the views
registration of the views in manifest
transfer of settings from the original table to the view (table settings, custom columns, custom actions)

The manifest updates could also take place by writing in to the config file of sap/ux-specification, but this works only incompletely, as long as the JSON schema is not updated from the new versions of the OData annotations as well as the updated manifest.
Technical Design
The idea is to follow the design of the FPM writer, and to profit as much as possible from the types and functions that are already provided there. This means using the mechanisms of mem-fs and mem-fs-editor here as well.
As we do not talk about Flexible Programming Model in the given scenario, we should rather introduce the new function as part of the fiori-elements-writer: here we already can find a similar module https://github.com/SAP/open-ux-tools/blob/main/packages/fiori-elements-writer/templates/v4/lrop/extend/webapp/manifest.json.
The new function shall also not be a general file writer that can write anything into the manifest, but be restricted to the special case (aka template).
The exposure takes place in form of functions and parameter types, similar to the index.ts of the fpm-writer.
Acceptance Criteria
Given I have the new module and function (generateViews) available
when I import it and use the new function
then the manifest update will take place as soon as the consumer calls the commit function of mem-fs.
In the following, at the Fiori Tools, the file watcher will trigger further activities, like the update of the JSON schema and config of sap/ux-specification.
Notes
Tasks

[ ] Create the new package manifest-writer
[ ] Define the new template
[ ] Expose reusable functions and types from the FPM writer
[ ] Define the new API function generateViews at manifest-writer
[ ] Unit Tests added
[ ] Test Cases defined

The requirement is not valid anymore, I could solve my problem by other means.


docs: add test overview for unit and integration testing

i'll add my content to this beginning of next week

commit is already in #68
close this pr
continue with this #68 cause just two pages


add GenericTraitCatalogImp

Codecov Report

Merging #379 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #379      +/-   ##
==========================================
- Coverage   54.48%   54.48%   -0.01%
==========================================
  Files         341      341
  Lines       58417    58417
==========================================
- Hits        31829    31826       -3
- Misses      26588    26591       +3

Impacted Files
Coverage Δ

src/lib/profiles/security/WeaveCert.cpp
88.5% <0%> (-0.46%)
:arrow_down:

src/test-apps/TestCASE.cpp
95.89% <0%> (-0.23%)
:arrow_down:

Continue to review full report at Codecov.

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


Show Frameworks and Qt version for Plasma. Closes #2013

Description
Added KDE Framworks and Qt version for Plasma as requested in #2013
Features

Issues
TODO

Thank you for your contribution!
This PR is merged into hyfetch since this repo (dylanaraps/neofetch) seems no longer maintained.
HyFetch is a fork of neofetch with LGBTQ pride flags, but the repo also maintains an updated version of the original neofetch, addressing many pull requests that are not merged in the original repo.
Read the "Running Updated Original Neofetch" section for more info!

Works!


Channel override in landing mode

Hi together,
in the wiki they say that you should open an issue before trying to use channel override.
Our usecase:
We are a students group developing a drone that should pick objects up. To center on the objects we're setting the vehicle-velocity setting in GUIDED-Mode with dronekit. However, the guided-mode doesn't seem to be suitable for landing, as the motors don't turn off automaticly (we tried it once and the copter flipped and crashed, but that was like 6 months ago).
That's why we need to use the landing-mode at some point (the copter should center all the way till right before the motors disarm).  However, the landing mode doesn't support the send vehicle-velocity settings, the only option nowadays is to use the channel-override method.
Maybe is there an option we missed to check if the drone has already landed, so that we can use guided-mode? (like in the fc-software). As now I don't see a better way as to use channel-override (specification for landing mode )

On Sat, 28 Dec 2019, deepdigger wrote:

We are a students group developing a drone that should pick objects up. To center on the objects we're setting the vehicle-velocity setting in GUIDED-Mode
with dronekit. However, the guided-mode doesn't seem to be suitable for landing, as the motors don't turn off automaticly (we tried it once and the copter
flipped and crashed, but that was like 6 months ago).
That's why we need to use the landing-mode at some point (the copter should center all the way till right before the motors disarm). However, the landing
mode doesn't support the send vehicle-velocity settings, the only option nowadays is to use the channel-override method.
Maybe is there an option we missed to check if the drone has already landed, so that we can use guided-mode? (like in the fc-software). As now I don't see
a better way as to use channel-override (specification for landing mode )

Perhaps you could look at using precision landing?
Here's ArduPilot's autotest:
https://github.com/ardupilot/ardupilot/blob/master/Tools/autotest/arducopter.py#L3729
(additionally, you should probably specify which flightstack you're using;
PX4Firmware or ArduPilot are typical :-)  )

Hi Peter, thanks for your answer :).
Sorry, I missed saying our flightstack. We're using the Pixhawk cube with Ardupilot.
I saw the precision landing but thought it would only work with an IR-sensor (from ardupilot documentation ).
However, I didn't knew of the other way by sending mavlink messages and specifing the landing target. :)
So thanks for giving us the hint!

I got it to work:). Important thing is to have the height in the message (like specified here and to enable this feature in the settings.
Thanks!


Cannot read property 'statusText' of undefined

Window 10 system

What version are you using?

We no longer have any references to statusText so this must be from v1.1.0:
https://github.com/Nutlope/aicommits/blob/68fc8ad736e38a9497e94236800d32c9264576d7/src/utils.ts#L98
Please try the latest release: https://github.com/Nutlope/aicommits/releases/tag/v1.2.0

The bug does not seem to have disappeared in version 1.2 either.

Hopefully this fixes it: https://github.com/Nutlope/aicommits/pull/106
I also added this: https://github.com/Nutlope/aicommits/pull/124 which should improve debugging

Closing this for now but let me know if it's still an issue and we can re-open it.


Adjustment to iphone onboarding flow

current

change to

Currently looks like:

I assume this change still needs to be applied.

haha oops, lets close this one and stick with the update one I posted


Bot wont go online.

Please tick all applicable boxes.

[ ] I am using Python 3.5 or higher (run python --version on the command line)
[x] I have followed the official guides to install the bot for my system
[x] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[ ] The latest master version (1.9.8)
[ ] The latest review version

What type of issue are you creating?

[x] Bug
[ ] Feature request
[ ] Question

Description of issue
I tried all I could do to get the bot online. Nothing will work. I went through as many tuts as I could find and nothing worked to get the bot online. I went through every step in every tut and when it came to getting the bot online it still didn't work. I spent 3-5 hours trying to find a solution to the problem but I haven't found anything. I'm close to giving up and seeing if someone else can script the bot for me. Im still new to this whole developing discord bots of my own thingy but I legit tried everything I could do for a straight 3-5 hours and nothing seems to work.
Steps to reproduce

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.
musicbot.log

from comment above AutoJoinChannel

Allows the bot to automatically join servers on startup. To use this, add the IDs
of the voice channels you would like the bot to join on startup, seperated by a
space. Each server can have one channel. If this option and AutoSummon are
enabled, this option will take priority.

You put no after AutoJoinChannel, which is not what it expected. To not automatically join voice channels you simply leave it blank

from comment above AutoJoinChannel

Allows the bot to automatically join servers on startup. To use this, add the IDs
of the voice channels you would like the bot to join on startup, seperated by a
space. Each server can have one channel. If this option and AutoSummon are
enabled, this option will take priority.

You put no after AutoJoinChannel, which is not what it expected. To not automatically join voice channels you simply leave it blank

thank you so much. It worked!


Don't pass though the game mode (closes #41)

Removed the game mode argument from respawn()
Moved set the backup weapon to spawnstate

+1!

I'm not sure if it works as expected, because i cannot test it on the current master (no bomberman files, no working console, not able to select the bomberman mode)... still have to wait until a_teammate merged the newpackaging and new data repository.


restore with hasManyThrough throws ambiguous 'deleted_at'

Laravel Version: 5.8.35
PHP Version: 7.2.1
Database Driver & Version: MySQL 5.7

Description:
"Column 'deleted_at' in field list is ambiguous" is thrown when trying to restore model with relationship on hasManyThrough.
Steps To Reproduce:
Model:
class Category extends Model {
 public function articles() {
        return $this->hasManyThrough(Article::class, Section::class);
    }
}

class Section extends Model {
 public function articles() {
    	return $this->hasMany(Article::class)->orderBy('sort_order', 'asc');
    }
}

Controller:
public function restore(){
        $cat = \App\Models\Category::find(19);
        $cat->articles()->withTrashed()->restore(); // This is where error occurs
}

Error Messaage:
Illuminate \ Database \ QueryException (23000)
SQLSTATE[23000]: Integrity constraint violation: 1052 Column 'deleted_at' in field list is ambiguous (SQL: update articles inner join sections on sections.id = articles.section_id set deleted_at = ?, articles.updated_at = 2019-12-11 06:16:48 where sections.deleted_at is null and sections.category_id = 19)
As far as I understand we need to mention table name when using joins and deleted_at does not have table name like article.deleted_at but as it seems in query it only has deleted_at = ?

Hey there,
Unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? We'll help you out and re-open this issue if so.
Thanks!

Good news then @driesvints because this happens on Laravel 6 as well.

This isn't possible atm afaik. Maybe https://github.com/laravel/framework/pull/31070 will solve this?

It's not a huge problem but it exists. It can easily be resolved by prepending pivot table names to the columns in the queries.


⚠ ClamAV has degraded performance

In d472c56, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 572 ms

Resolved: ClamAV performance has improved in a499c35.


Fix: Use correct data/extras type in tracer

:scroll: Description

:bulb: Motivation and Context

:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[ ] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps

Codecov Report

Merging #693 (17abd97) into main (1f32c29) will increase coverage by 5.50%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #693      +/-   ##
==========================================
+ Coverage   90.65%   96.15%   +5.50%
==========================================
  Files          97        2      -95
  Lines        3145       52    -3093
==========================================
- Hits         2851       50    -2801
+ Misses        294        2     -292

Impacted Files
Coverage Δ

dart/lib/src/sentry_tracer.dart

dart/lib/src/protocol/sentry_device.dart

dart/lib/src/protocol/sentry_transaction.dart

dart/lib/src/protocol/max_request_body_size.dart

dart/lib/src/throwable_mechanism.dart

dart/lib/src/sentry_traces_sampler.dart

dart/lib/src/platform/platform.dart

dart/lib/src/protocol/sentry_trace_header.dart

...lib/src/invalid_sentry_trace_header_exception.dart

dart/lib/src/protocol/mechanism.dart

... and 85 more

Continue to review full report at Codecov.

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


The examples "Hello World" and "Hello Button" don't working | Windows 11

'>ruby test.rb'
<internal:C:/Ruby32-x64/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:136:in 'require': cannot load such file -- wxruby_core (LoadError)
        from <internal:C:/Ruby32-x64/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:136:in 'require'
        from C:/Users/User/.local/share/gem/ruby/3.2.0/gems/wxruby3-0.9.8/lib/wx/core.rb:9:in '<top (required)>'
        from <internal:C:/Ruby32-x64/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:136:in 'require'
        from <internal:C:/Ruby32-x64/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:136:in 'require'
        from C:/Users/User/.local/share/gem/ruby/3.2.0/gems/wxruby3-0.9.8/lib/wx.rb:9:in '<top (required)>'
        from <internal:C:/Ruby32-x64/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:141:in 'require'
        from <internal:C:/Ruby32-x64/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:141:in 'rescue in require'
        from <internal:C:/Ruby32-x64/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:135:in 'require'
        from test.rb:1:in '<main>'
<internal:C:/Ruby32-x64/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:136:in 'require': cannot load such file -- wx (LoadError)
        from <internal:C:/Ruby32-x64/lib/ruby/site_ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:136:in 'require'
        from test.rb:1:in '<main>'

Alright, it works. I forgot to type wxruby setup. Sorry for the inconvenience.


HTML

HTML
User Story
AS AN employer
I WANT to view a potential employee's deployed portfolio of work samples
SO THAT I can review samples of their work and assess whether they're a good candidate for an open position
Acceptance Criteria
GIVEN I need to sample a potential employee's previous work
WHEN I load their portfolio
THEN I am presented with the developer's name, a recent photo or avatar, and links to sections about them, their work, and how to contact them
WHEN I click one of the links in the navigation
THEN the UI scrolls to the corresponding section
WHEN I click on the link to the section about their work
THEN the UI scrolls to a section with titled images of the developer's applications
WHEN I am presented with the developer's first application
THEN that application's image should be larger in size than the others
WHEN I click on the images of the applications
THEN I am taken to that deployed application

t

added html but did not add any class yet left comments

finished
GIVEN I need to sample a potential employee's previous work
WHEN I load their portfolio
THEN I am presented with the developer's name, a recent photo or avatar, and links to sections about them, their work, and how to contact them
WHEN I click one of the links in the navigation
THEN the UI scrolls to the corresponding section
remaining
WHEN I click on the link to the section about their work
THEN the UI scrolls to a section with titled images of the developer's applications
WHEN I am presented with the developer's first application
THEN that application's image should be larger in size than the others
WHEN I click on the images of the applications
THEN I am taken to that deployed application