Dll load failed error

ImportError: Traceback (most recent call last):
File "C:\Users\parkar\Anaconda3\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\parkar\Anaconda3\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\parkar\Anaconda3\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "C:\Users\parkar\Anaconda3\lib\imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "C:\Users\parkar\Anaconda3\lib\imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: DLL load failed: A dynamic link library (DLL) initialization routine failed.
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

I used colab but in the project which i made is using GUI(tkinter library) so for that colab is not useful

@Div123chauhan,
You might be facing this issue because of the following reasons

You are running 32-bit Python or 32-bit OS
You have not installed the Microsoft Visual C++ Redistributable package
Your CPU does not support AVX instructions.

Please take a look at the system requirements and check if you have the correct dependencies installed.
Also, check these similar duplicate issues: #36167 #36138. Thanks!

Help Getting errors while importing TF
ImportError                               Traceback (most recent call last)
~\anaconda3\lib\site-packages\tensorflow\python\pywrap_tensorflow.py in
63   try:
---> 64     from tensorflow.python._pywrap_tensorflow_internal import *
65   # This try catch logic is because there is no bazel equivalent for py_extension.
ImportError: DLL load failed while importing _pywrap_tensorflow_internal: The specified module could not be found.
During handling of the above exception, another exception occurred:
ImportError                               Traceback (most recent call last)
 in
----> 1 import tensorflow as tf
~\anaconda3\lib\site-packages\tensorflow_init_.py in
39 import sys as _sys
40
---> 41 from tensorflow.python.tools import module_util as _module_util
42 from tensorflow.python.util.lazy_loader import LazyLoader as _LazyLoader
43
~\anaconda3\lib\site-packages\tensorflow\python_init_.py in
38 # pylint: disable=wildcard-import,g-bad-import-order,g-import-not-at-top
39
---> 40 from tensorflow.python.eager import context
41
42 # pylint: enable=wildcard-import
~\anaconda3\lib\site-packages\tensorflow\python\eager\context.py in
33 from tensorflow.core.protobuf import config_pb2
34 from tensorflow.core.protobuf import rewriter_config_pb2
---> 35 from tensorflow.python import pywrap_tfe
36 from tensorflow.python import tf2
37 from tensorflow.python.client import pywrap_tf_session
~\anaconda3\lib\site-packages\tensorflow\python\pywrap_tfe.py in
26
27 # pylint: disable=invalid-import-order,g-bad-import-order, wildcard-import, unused-import
---> 28 from tensorflow.python import pywrap_tensorflow
29 from tensorflow.python._pywrap_tfe import *
~\anaconda3\lib\site-packages\tensorflow\python\pywrap_tensorflow.py in
81 for some common reasons and solutions.  Include the entire stack trace
82 above this error message when asking for help.""" % traceback.format_exc()
---> 83   raise ImportError(msg)
84
85 # pylint: enable=wildcard-import,g-import-not-at-top,unused-import,line-too-long
ImportError: Traceback (most recent call last):
File "C:\Users\Gaganu\anaconda3\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 64, in
from tensorflow.python._pywrap_tensorflow_internal import *
ImportError: DLL load failed while importing _pywrap_tensorflow_internal: The specified module could not be found.
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

Help Getting errors while importing TF

@Nayanasalur,
Could you please submit a new issue from this link and fill in the template, so that we can track the issue there. Thanks!

I solve my issue using the making new environment in anaconda and then downloding all the libraries like keras, keras --gpu, etc.
and also installing Microsoft Visual C++ Redistributable package. Now, my tensorflow is working without any error.


Adding LICENSE file

README says "Unless otherwise specified, source code are licensed under MIT".

Thank you!


Bug: overlay of chatbot button on links

Describe the bug
The opensauced chat modal button is displaying over some links which makes it unable for the users to click and see on mobile view

Steps to reproduce

Go to opensauced.pizza
Click on get started after authentication
Scroll down see chatbot button
See error

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

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

Contributing Docs

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

This bug is quite the tricky one... A possible fix would be to move the bot icon a bit upward to sit in between the Dev and Status link.
Are you up for it @dev-phantom ?

This bug is quite the tricky one... A possible fix would be to move the bot icon a bit upward to sit in between the Dev and Status link.
Are you up for it @dev-phantom ?

My PC can't run the repo locally 😔

My PC can't run the repo locally

You're still having that issue :exploding_head: What was it again? :sweat_smile:

My PC can't run the repo locally

You're still having that issue 🤯 What was it again? 😅

Can't really remember
But the repo is large and my PC spec is quite low

Please can I get assigned to this issue?

Please can I get assigned to this issue?

Feel free to take on this issue
@takanome-dev will assign you to it shortly

Please can I get assigned to this issue?

Feel free to take on this issue @takanome-dev will assign you to it shortly

Thanks

In the future, you can use .take to assign yourself 🍕

@OpherM  for the future could you check this little doc

I'm still able to reproduce this issue 🤔
@takanome-dev can you reopen this?


Option -save is not supported

Is your feature request related to a problem? Please describe.
The current scala runner (based on Scala 2.13) does support a -save option.
The experimental scala runner (based on Scala CLI) does not recognize that option:
$ scala -save test.scala
Unrecognized argument: -save

To list all available options, run
  /home/julien/.local/share/coursier/bin/.scala.aux --help

Describe the solution you'd like
For the sake of compatibility with the current runner, it would be great to recognize the -save option. As far as I understand, Scala CLI now always behaves like the current scala -save, so the option is not necessary anymore. Therefore, it could simply print a warning like “Unnecessary option: -save. This option is always enabled.”

As far as I understand, Scala CLI now always behaves like the current scala -save, so the option is not necessary anymore.

Actually, it seems that part isn't entirely true.
There doesn't seem to be any official docs on the -save (and tied -nosave) options for the old scala command beyond what's in the runner code itself.
For context, here are the relevant code bits:

Scala 2: https://github.com/scala/scala/blob/f8d3c4d57854f08ae261b65d87005de004e8b2c8/src/compiler/scala/tools/nsc/GenericRunnerSettings.scala#L60
Scala 3: https://github.com/lampepfl/dotty/blob/main/compiler/src/dotty/tools/MainGenericRunner.scala#L142

Looking at the actual behaviour:
▶ tree -a
# .
# └── smth.scala
#
# 0 directories, 1 file
▶ scala -save smth.scala
# Hello
▶ tree -a
# .
# ├── smth.jar
# └── smth.scala
#
# 0 directories, 2 files
▶ scala smth.jar
# Hello

It seems that the -save option forces a compiled jar to be saved in the cwd, while -nosave overrides it and prevents the jar being saved (for whatever reason).
Now, this looks like a way to combine the run and package --lib subcommands of Scala CLI.
We could theoretically support it by running the 2 subcommands one after another (that'd be a quick bandaid) or (preferably) combine both effects in a single build.

Therefore, it could simply print a warning like “Unnecessary option: -save. This option is always enabled.”

In this context, that wouldn't be true. We could always just not support it if it's not actually needed and print a warning like Deprecated option '-save' ignored., so that it doesn't fail the build.
@julienrf thoughts? Is this option necessary?

I think the fact that it saves the compiled code as a .jar is an “implementation detail”. But maybe not. Maybe some people are relying on the produced jar. In that case, I think a warning saying that this option is ignored but the compilation artifacts are saved for reuse would be enough.

We could even suggest running a subsequent package --lib in the warning, too, in case someone actually needs that jar... That'd reuse whatever is in .scala-build/, so it wouldn't be much slower beyond the necessity of running another command.
I guess that if it's a niche use case, maybe that'd be enough.


Phase 3 Funding | Starkman | Cosmology

Still a draft.
Signed-off-by: nstarman<EMAIL_ADDRESS>

Because the estimated total of all the proposals is a bit higher than the amount available for allocation, we suggest that you (@nstarman) provide a budget range - i.e., a "minimum for this project to be viable" and "maximum you could use effectively for this project".

Thanks @jdswinbank for this update. I'm excited to receive this funding and grateful for Astropy's support of the core maintainers.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 432 ms

Resolved: ClamAV performance has improved in 6a5bcc2.


Function Average has a bug

Expression expression = new Expression("AVERAGE(data)");
List list = List.of(2, 4, 6);
Double result = expression.with("data", list).evaluate().getNumberValue().doubleValue();
System.out.println(result);

The AVERAGE function doesn't support arrays as input parameter(s).
I'll mark this as an enhancement.

I can implement this enhancement.


Batch ensembler orchestration

This PR introduces the orchestration for building an OCI image for the ensembling job on Kubernetes and creating a spark application on kubernetes using the spark-on-k8s-operator. State of the Spark Application/Image building is also synchronised to the Turing database.
However, local development is not working at the moment for batch ensembling. Will work on it in a separate PR.
How the whole orchestration flow works:

Create an ensembler using the POST /projects/{project_id}/ensemblers.
Create an ensembling job using POST /projects/{project_id}/jobs, this should return a 202 Accepted response, which will be executed async in step 3.
A Kaniko Job is sent to the Kubernetes cluster, building a Spark ensembler with this ensembler Dockerfile: https://github.com/ashwinath/turing/blob/batch-ensembler-orchestration/engines/batch-ensembler/app.Dockerfile with the base image: https://github.com/ashwinath/turing/blob/batch-ensembler-orchestration/engines/batch-ensembler/Dockerfile
This newly built spark ensembler needs to be sent to spark-on-k8s-operatorm, there is requirement for the Kubernetes cluster to have the spark-on-k8s-operator set up. This ensembling job creates the service accounts, secrets and configmaps required for the spark ensembler built in step 3, then it will submit the SparkApplication resource to Kubernetes. This is part of the run loop in the ensembling job (https://github.com/ashwinath/turing/blob/batch-ensembler-orchestration/api/turing/batch/runner/ensembling.go#L63).
Because the Spark application requires some time to pull the image, run the driver + executors, there is an update loop put in to update the status of the Spark application (https://github.com/ashwinath/turing/blob/batch-ensembler-orchestration/api/turing/batch/runner/ensembling.go#L64). This loop simply polls the status of the SparkApplication and updates the Turing database. The status can be queried using the GET /projects/{project_id}/jobs or GET /projects/{project_id}/jobs/{id} endpoint. Also, if the application crashes inbetween, i.e. during the image building step, it will reset the job to pending and the batch runner will pick up the job again for reprocessing. In summary: poll for running -> check SparkApplication status, poll for building + some configurable timeout -> check if imagebuilding has hanged. More details in the comments here: https://github.com/ashwinath/turing/blob/batch-ensembler-orchestration/api/turing/batch/runner/ensembling.go#L111

This PR adds steps 3-5.

@ashwinath could I ask for a couple of things to help give me more background for reviewing this MR?

Could you add more details to the PR description summarizing the implementation changes? Eg: Which APIs does this PR cover (I assume it is one or more APIs from this list), what steps are carried out internally, etc.
Are there any notes or changes to the infra that are required for running this (eg: the PR description says using the spark-on-k8s-operator - I don't believe this is covered in our README). I understand that the local dev env changes and e2e tests may be WIP or planned to be added in a future MR.

@ashwinath could I ask for a couple of things to help give me more background for reviewing this MR?
* Could you add more details to the PR description summarizing the implementation changes? Eg: Which APIs does this PR cover (I assume it is one or more APIs from [this](https://github.com/gojek/turing/blob/main/api/openapi/jobs.yaml) list), what steps are carried out internally, etc.

I have added more details more details on the whole batch ensembling process in the PR description.

* Are there any notes or changes to the infra that are required for running this (eg: the PR description says `using the spark-on-k8s-operator` - I don't believe this is covered in our README). I understand that the local dev env changes and e2e tests may be WIP or planned to be added in a future MR.

Ok, I will add in the infra details in the readme for this MR.

Have updated the readme @krithika369

Hopefully I have not missed out any comments/suggestions. I would require a re-review as I have addressed the concerns and rebased on main.


Improve data table to fix multiple issues

Several issues did come up in the past days and months of which a lot are caused by the use of the wxDataListCtrl.
I looked around and found this demo https://github.com/wxWidgets/Phoenix/blob/master/demo/DVC_DataViewModel.py which I think is the way to go!

Groupable columns for reference and value
Sortable
Use of dataobjects which hopefully eleiminate the need to populate the list completely on every change.

I still need to look into it a bit more to make sure it solves all or at least most of the issues.

This has the potential to solve #139

I'm interested to know if instead of bringing the functions of the spreadsheet to data list if we can bring the features of jlc tools to the spreadsheet. I don't know what direction kicad devs are headed for v7 though just a thought

Done :-)


Run command should not trigger auto save when use "files.autoSave": "onFocusChange"

Some time I want to run some editor edit commands, but this will trigger auto save and code actions on save.
I can't make sure this is a bug or feature request.
A real example:
https://github.com/microsoft/vscode/assets/41773861/852275cf-6936-4bf6-82b2-6d39754fce5e
In above video, I want to transform a link to markdown link with title. But when I run command Url Title: Run, VSCode will first trigger auto save because the focus is in command panel input now which is a corespond behavior of setting: "files.autoSave": "onFocusChange".
However, code actions on auto save trigger prettier format, prettier add the brackets which need not.
In my opinion, run command is an special editor edit operation, should not trigger auto save.
@mjbvz

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

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

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


(feat) Subjects with built-in distinct behavior

Feature Request
I suppose this is a common use case, that's why I'm proposing to have this one in RxJS rather than implementing it manually.
Often we want a Subject \ BehaviorSubject \ ReplaySubject that emits only when actually a new value passed to the next() method is different from a previous value.
Obviously,  we could use the distinct() operator when subscribe()-ing, but it would be incorrect when this responsibility should be on the Subject itself.
Second solution: in the code that emits values we could check if the value is different before emitting. But that should be done in every such place, not to mention that it could be implemented easily only with BehaviorSubject (using the value property \ getValue() method), but using those is geberaly discouraged.
So the cleanest solution seems to have a custom Subject with this behavior built-in.
It would be nice if those Subjects (DistinctSubject \ DistinctBehaviorSubject \ DistinctReplaySubject) were a part of RxJS.

@cartant nice, thank you! This is exactly my intention: to propose it and see if the feature  is going to be popular enough throughout the community :)


Error: In OnFrame:Modules/build.lua.... (i think files are missing)

Not really sure what is going on. I tried re-installing... can't read any pastabin or code.

Get this version: https://github.com/PathOfBuildingCommunity/PathOfBuilding/releases
and it will work again.


Farmaciatei - "Inactive" status; Graph without active data;  No data in Local Storage

ENVIRONMENT: production
URL: https://scrapers.peviitor.ro
Browser: Chrome
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:
1.Open the URL in the browser
2.Search for "Farmaciatei" Company
3.Run scraper
4.Check the "Status" section
5.Check Graph
6. Check Local Storage
EXPECTED RESULTS:
The scraper runs correctly and the message "Active" is displayed in the "Status" section
The Graph displays the data on the cover of the page
Local Storage display all data
ACTUAL RESULTS:
The "Inactive" message is displayed in the "Status" section
The graphic on the cover of the page does not returns the data
The scraper does not return data in Local Storage

This scraper was removed


Evaluate yet another review of JuMP Parameters

at JuMP dev the new syntax for parameters was shown and exists since JuMP 1.14. We should refactor some of the parameter handling in PSI that could benefit from better syntaxes.
cc. @odow

Yeah using @variable(model, p in Parameter(1.0)) and set_parameter_value: https://jump.dev/JuMP.jl/stable/api/JuMP/#JuMP.set_parameter_value
should mean that we automatically add the MOI.EqualTo constraints if you use Gurobi and we automatically use parameters if you use POI or Ipopt.


Support for CalVer

For some projects (esp. continuous release internal projects) it would be quite useful to use bump-my-version to generate versions in YYYY-0M-0D-0h-0m and other similar calendar-based formats.  See https://calver.org
We do CI/CD with multiple releases each day but tools like Sentry can use releases to understand when bugs were first created and when they were resolved.  See https://docs.sentry.io/product/releases
We could do this with major.minor.patch  but we sense that we would churn through a lot of patch IDs and would constantly be mapping between version numbers and release dates.  CalVer is more self-documenting for internal DevOps purposes.

My goal was to support this. Now that you bring it up, I can see some difficulties since the interface calls to bump a specific part.
What kind of interface would you propose to make this easier? The underlying mechanisms will support CalVer already, so it is really just a matter of interface.


spring cloud kubernetes configuration watcher does not support CMs from multiple namespaces

Describe the bug
Using springcloud/spring-cloud-kubernetes-configuration-watcher:3.0.1-SNAPSHOT
The documentation is very unclear about using the configuration watcher.
Especially about supporting multiple namespaces.
The closest option I thought that should work is spring.cloud.kubernetes.reload.namespaces but this one is said as
This functionality has been deprecated in the 2020.0 release. Please see the [Spring Cloud Kubernetes Configuration Watcher](https://docs.spring.io/spring-cloud-kubernetes/docs/3.0.1-SNAPSHOT/reference/html/#spring-cloud-kubernetes-configuration-watcher) controller for an alternative way to achieve the same functionality.
So actually I don't think I should be be using it, and use another parameter but no idea which one.
anyways, let's describe the bug with spring.cloud.kubernetes.reload.namespaces.
When you provide a list of namespaces, only the first one in the list is taken into account.
Sample
SENARIO 1: I specify this list: namespace1,namespace2
I am using this deployment  to deploy the watcher in the spring NS, and so it watches configmaps from 2 NS: namespace1 and namespace2
apiVersion: v1
kind: List
items:
  - apiVersion: apps/v1
    kind: Deployment
    metadata:
      name: configuration-watcher-deployment
      namespace: spring
    spec:
      selector:
        matchLabels:
          app: configuration-watcher
      template:
        metadata:
          labels:
            app: configuration-watcher
        spec:
          serviceAccount: configuration-watcher-service
          containers:
            - name: configuration-watcher-container
              image: springcloud/spring-cloud-kubernetes-configuration-watcher:3.0.1-SNAPSHOT
              imagePullPolicy: IfNotPresent
              readinessProbe:
                httpGet:
                  port: 8888
                  path: /actuator/health/readiness
              livenessProbe:
                httpGet:
                  port: 8888
                  path: /actuator/health/liveness
              ports:
                - containerPort: 8888
              env:
                - name: logging.level.org.springframework.cloud.kubernetes
                  value: "TRACE"
                - name: spring.cloud.kubernetes.discovery.all-namespaces
                  value: "true"
                - name: spring.cloud.kubernetes.reload.enabled
                  value: "true"
                - name: spring.cloud.kubernetes.reload.namespaces
                  value: "namespace1,namespace2"

here is the 2 config maps:
apiVersion: v1
kind: ConfigMap
metadata:
  name: cm1
  namespace: namespace1
  labels:
    spring.cloud.kubernetes.config: "true"
data:
  application.properties: |-
    bean.message=Hello World from NS1!
---
apiVersion: v1
kind: ConfigMap
metadata:
  name: cm2
  namespace: namespace2
  labels:
    spring.cloud.kubernetes.config: "true"
data:
  application.properties: |-
    bean.message=Hello World from NS2!

and how to create the SA and CRB
---
apiVersion: v1
kind: List
items:
  - apiVersion: v1
    kind: ServiceAccount
    metadata:
      labels:
        app: configuration-watcher
      name: configuration-watcher-service
      namespace: spring

  - apiVersion: rbac.authorization.k8s.io/v1
    kind: ClusterRoleBinding
    metadata:
      creationTimestamp: null
      name: admin-a
    roleRef:
      apiGroup: rbac.authorization.k8s.io
      kind: ClusterRole
      name: cluster-admin
    subjects:
      - kind: ServiceAccount
        name: configuration-watcher-service
        namespace: spring

and the 3 NS
apiVersion: v1
kind: Namespace
metadata:
  name: spring
---
apiVersion: v1
kind: Namespace
metadata:
  name: namespace1
---
apiVersion: v1
kind: Namespace
metadata:
  name: namespace2

out of the logs I can see it recognizes my 2 NS :
023-01-13T13:12:24.523Z DEBUG 1 --- [           main] o.s.c.k.c.c.KubernetesClientConfigUtils  : informer namespaces : [ra-demo-app, namespace1]
2023-01-13T13:12:24.528Z  INFO 1 --- [           main] sClientEventBasedConfigMapChangeDetector : Kubernetes event-based configMap change detector activated
2023-01-13T13:12:24.530Z DEBUG 1 --- [           main] sClientEventBasedConfigMapChangeDetector : added configmap informer for namespace : namespace1
2023-01-13T13:12:24.533Z DEBUG 1 --- [           main] sClientEventBasedConfigMapChangeDetector : added configmap informer for namespace : namespace2
2023-01-13T13:12:24.543Z  INFO 1 --- [           main] o.s.c.k.client.KubernetesClientUtils     : Created API client in the cluster.
2023-01-13T13:12:24.544Z DEBUG 1 --- [           main] o.s.c.k.c.c.KubernetesClientConfigUtils  : informer namespaces : [namespace1, namespace2]
2023-01-13T13:12:24.548Z  INFO 1 --- [           main] tesClientEventBasedSecretsChangeDetector : Kubernetes event-based secrets change detector activated
2023-01-13T13:12:24.552Z DEBUG 1 --- [           main] tesClientEventBasedSecretsChangeDetector : added secret informer for namespace : namespace1
2023-01-13T13:12:24.556Z DEBUG 1 --- [           main] tesClientEventBasedSecretsChangeDetector : added secret informer for namespace : namespace2

I can see out of the logs it detects the CM named cm1 from Namespace1:
2023-01-13T13:12:24.623Z DEBUG 1 --- [ool-14-thread-1] sClientEventBasedConfigMapChangeDetector : ConfigMap cm1 was added.
but not cm2 from Namespace2!
I can confirm a manual change of cm1 is detected but not in cm2
SENARIO 2: I specify this list: namespace2,namespace1
basically NS2 is now first.
I do this by updating the above deployment with this part:
                - name: spring.cloud.kubernetes.reload.namespaces
                  value: "namespace2,namespace1"

The logs confirm that both NS have been detected, and with a different order:
2023-01-13T13:32:37.489Z DEBUG 1 --- [           main] o.s.c.k.c.c.KubernetesClientConfigUtils  : informer namespaces : [namespace2, namespace1]
2023-01-13T13:32:37.492Z  INFO 1 --- [           main] sClientEventBasedConfigMapChangeDetector : Kubernetes event-based configMap change detector activated
2023-01-13T13:32:37.493Z DEBUG 1 --- [           main] sClientEventBasedConfigMapChangeDetector : added configmap informer for namespace : namespace2
2023-01-13T13:32:37.496Z DEBUG 1 --- [           main] sClientEventBasedConfigMapChangeDetector : added configmap informer for namespace : namespace1

but this time it shows cm2 from the log and NOT cm1:
2023-01-13T13:32:37.564Z DEBUG 1 --- [ool-14-thread-1] sClientEventBasedConfigMapChangeDetector : ConfigMap cm2 was added.
I can confirm a manual change of cm2 is detected but not in cm1
SUMMARY:
It sounds like only the first from the list is really watched.
The configuration-watcher-service SA has the ClusterRole.
so I don't think it is a permission issue, and we can see that anyways depending on the order, each CM from any of the 2 NS individually work. so it has access to those NS.
and my CM1 and CM2 have the expected label:
spring.cloud.kubernetes.config: true
Note that I also tried with enable-reload-filtering: true  and adding this label to the CMs spring.cloud.kubernetes.config.informer.enabled: true but I have the same behavior
any idea?

Can you try switching the environment properties to
- name: spring.cloud.kubernetes.reload.namespaces[0]
  value: "namespace1",namespace2"
- name: spring.cloud.kubernetes.reload.namespaces[1]
  value: "namespace2"

I tried this earlier this morning as I was not sure about how to provide a list, but got this error when applying the deployment:
`The Deployment "configuration-watcher-deployment" is invalid:

spec.template.spec.containers[0].env[3].name: Invalid value: "spring.cloud.kubernetes.reload.namespaces[0]": a valid environment variable name must consist of alphabetic characters,
digits, '_', '-', or '.', and must not start with a digit (e.g. 'my.env-name',  or 'MY_ENV.NAME',  or 'MyEnvName1', regex used for validation is '[-._a-zA-Z][-._a-zA-Z0-9]*')
spec.template.spec.containers[0].env[4].name: Invalid value: "spring.cloud.kubernetes.reload.namespaces[1]": a valid environment variable name must consist of alphabetic characters,
digits, '_', '-', or '.', and must not start with a digit (e.g. 'my.env-name',  or 'MY_ENV.NAME',  or 'MyEnvName1', regex used for validation is '[-._a-zA-Z][-._a-zA-Z0-9]*')
`

Sorry lets try this instead
- name: SPRING_CLOUD_KUBERNETES_RELOAD_NAMESPACES_0
  value: "namespace1"
- name: SPRING_CLOUD_KUBERNETES_RELOAD_NAMESPACES_1
  value: "namespace2"

thanks for your help.
I get the same behavior unfortunately, where only the 2 informers are created, one for each.
but only one CM is added, inked to the first NS in the list :-(
here is my new config
              env:
                - name: logging.level.org.springframework.cloud.kubernetes
                  value: "TRACE"
                - name: spring.cloud.kubernetes.discovery.all-namespaces
                  value: "true"
                - name: spring.cloud.kubernetes.reload.enabled
                  value: "true"
                - name: SPRING_CLOUD_KUBERNETES_RELOAD_NAMESPACES_0
                  value: "namespace2"
                - name: SPRING_CLOUD_KUBERNETES_RELOAD_NAMESPACES_1
                  value: "namespace1"

Initially I only provided the deployment.yaml in the issue description, I just updated the description with everything else you need to reproduce i.e. the configmaps.yaml , the sa.yaml, and ns.yaml
and I use this command to check which NS has been added (allways 2):
kubectl logs -l app=configuration-watcher --tail=-1 -n spring | grep "added configmap"
and this one to check which CM have been added (only 1 instead of 1, the first in the list of NS):
kubectl logs -l app=configuration-watcher --tail=-1 -n spring | grep "was added"

Sorry for being late to the party, I am on vacation with somehow limited internet access.

The documentation is here and it does specify how you must set the namespaces. Can you please detail on how you would may be like it to be? This is a new functionality we added with 3.x.x releases and any hints or improvements on it would be lovely (if any).

The reload functionality even without configuration-watcher (where you need to deploy a separate pod in your cluster) is not going anywhere yet. Yes, it is deprecated, but as a matter of fact it was improved in 3.x.x releases, so it is safe to still use it. I see no issues or planned something (unless Ryan corrects me) to actually drop this.

"one string, command separated" is not understood by config watcher, it just outputs namespace1, namespace2 as a single String. I don't understand how you see in logs :|

added configmap informer for namespace : namespace1
added configmap informer for namespace : namespace2

there are some things that immediately jump into my eyes where we can improve in some parts, and I will work on them, so thx for raising this. For example you should have never seen these logs:

ClientEventBasedSecretsChangeDetector : added secret informer for namespace : namespace1

Secrets (unlike configmaps) must be disabled by default and you need to opt-in to monitor them. We do this for plain source reading from secrets, but we don't here. I will fix this.
Also, we need to improve on the logging on which namespaces are being tracked, otherwise this gets confusing indeed.

All in all, I need a reproducible example here, a github repo with some clear steps on what is going on. Until you provide that, I will try to write an integration test to see if I can "catch" this, may be this is indeed a bug. At this point, I can't be sure.

thanks @wind57
you said
"one string, comma separated" is not understood by config watcher, it just outputs namespace1, namespace2" as a single String.
well my string was:
namespace1,namespace
without a space between, but it shows:
namespace1, namespace
do you see the "space" between? so it did some transformation, it did explode by "comma" which is the separator.
it is clear in the other part of the logs
added configmap informer for namespace : namespace1
added configmap informer for namespace : namespace2

but anyways, as per ryan suggestion I did test the following:
- name: SPRING_CLOUD_KUBERNETES_RELOAD_NAMESPACES_0
value: "namespace2"
- name: SPRING_CLOUD_KUBERNETES_RELOAD_NAMESPACES_1
value: "namespace1"
I assume this is the correct way to setup the parameter but it has the exact same behavior :-(
you said :
All in all, I need a reproducible example here, a github repo with some clear steps on what is going on. Until you provide that, I will try to write an integration test to see if I can "catch" this, may be this is indeed a bug. At this point, I can't be sure.
well, I put everything in my description, so anyone can reproduce by doing a copy/paste in a yam file. and the steps.
The only thing I didn't show are the app, but I don't think they are  needed to describe and understand the issue, out of the logs we should see the 2 configmaps .
I should see:
sClientEventBasedConfigMapChangeDetector : ConfigMap cm1 was added.
sClientEventBasedConfigMapChangeDetector : ConfigMap cm2 was added.

but I only see
sClientEventBasedConfigMapChangeDetector : ConfigMap cm1 was added.
I assume it is enough to identify the issue (we don't even need to update the CM, and we don't even need to see the 'refresh' request yet), if not I can add also a fake app to illustrate (my current app is real so I can't share for now).
the thing is there is various example on the web, but there is no based on multiple NS

fair enough, I will try to reproduce this with an integration test. I don't have any ETA, but as soon as I am ready, I will update this post.

thanks you so much :-)
I spent 3 full days on this, I literally read ALL the web about some keywords like "spring cloud kubernetes configuration watcher",  spring.cloud.kubernetes.reload.namespaces.
I cloned all examples I found out of github, but actually no-one is even talking about multiple NS.
my brain can't progress anymore on this, so I will wait for your feedback :-)

no worries, next time just come here directly or try stackoverflow, I am trying to be active there too on spring-cloud-kubernetes tag, at times.

@webrod thanks for the addition info we will look into this!

I put the files on github for your convenience:
https://github.com/webrod/spring-cloud-watcher

I am still on vacation for one week, so my screen time is somehow limited. As soon as I return, this will be the top priority for me too look at. FYI

@ryanjbaxter this is a bug, I can reproduce via an integration test. can you add the proper labels please? I will of course fix it

@wind57 got it!
Did you try it against 2021.0.5?  Is it just in 2022.0.0?

This is a new feature that we added in main branch only (selective namespaces), so I guess only 2022.0.0, right?

@webrod can you update the title of the issue please? it should include that secrets (not only configmaps) have the same issue. thank you.

I took care of it


Performance overhead of always adding the offsets object

This proposal implies allocating a new object as part of the result for all existing RegExp calls, even though existing call sites don't use this object. I worry that this allocation will have some performance overhead. Another way would be to introduce a new method somewhere to get this information.
Any thoughts from implementers on whether the potential performance cost here is acceptable? cc @schuay @petermarshall @hashseed

The problem with a method is that by the time we've received the match we would have to then recompute the match again. Another option would be to add a flag to indicate whether this information is needed.

If you have an internal slot with the information you have the same overhead, its just stored in a different way. I don't think I want to have to call execWithOffsets, but I could see /a*(z)/o or something indicating an intent to capture offsets.

If you have an internal slot with the information you have the same overhead, its just stored in a different way.

This is a really concrete claim. I'd like to hear from people maintaining engines on this.

I don't think I want to have to call execWithOffsets

Could you explain why not? I think we might be able to come up with a nicer name.

For one, I'd like to be able to get offsets from String.prototype.match as well. A flag seems a good way to trigger that behavior, if we can't do that by default.
As far as storage size, yes it's feasible for a host to have some kind of internal packed information format, that saves some space, but hosts often already do that for arrays anyways.

I think putting things on the match object JS the best choice - it covers exec, match, replace (assuming it could be passed like groups is), and matchAll.
A flag is the most backwards compatible behavior imo, but a memoizing getter seems like a potential optimization, depending on what implementors feel is most efficient.

In V8, we use a similar format (a list of [start_index,end_index] tuples) to pass the result of an execution from the irregexp-jitted code to the RegExp.prototype.exec builtin. After constructing the result object, this index list is thrown away.
A few thoughts on this:

Keeping the index list is additional memory overhead for every result object.
Creating the result.offsets array won't be free but probably acceptable (this should be verified though).
Creating result.offsets.groups could be more expensive (it's a new JS object with named properties).
The flag would only make sense for exec, @@matchall, and possibly @@replace (but please don't make @@replace more unwieldy than it already is). But not for test, @@split, @@search, and @@match. IMHO we shouldn't have a flag that is useless for the majority of regexp functions.

My gut feeling here is ambiguous. I see how it could be useful in some situations, but do we really want to slow down / use more memory for every regexp user? A dedicated function may be the nicer option.

cc @mathiasbynens

Both RegExp.prototype.exec and String.prototype.match currently only take a single argument, could we consider a second argument to supply options to opt in to this behavior? Per @schuay's comment (and I expect most implementations do something similar), this information already exists, so there would be little cost in terms of processing time.
The only major cost would be in terms of memory, which could be made available via an opt-in mechanism (be it either a flag, argument, or overload). The overhead of offsets.groups would only be incurred when both requesting offsets and using capturing groups.
If holding onto the offsets is an option, we could consider @ljharb's suggestion of using a memoizing getter to fill this in on demand.
Also, as far as this being a "niche" use, there are a number of projects (including VS Code) that are forced to rely on packages like the NodeJS bindings for Oniguruma, which are highly complex and require native bindings and compilation. Alternatives like the "regexp-measure" package I linked in the explainer are slow and expensive as they must re-parse the regexp and wrap every non-capturing segment in a capturing group to produce these offsets.

Making it an option is something I could get on board with. The performance hit would mostly result from having to add object properties.

Niche or not, I think that the use case for this feature is clear, especially after reading this comment (maybe mention this in the README?).
I like the memoized getter idea, as it doesn’t hurt performance for the common case.

Why are options OK but a different-named method is not?

@littledan:

Why are options OK but a different-named method is not?

The only difference between the result of exec and the result of an execWithOffsets (or any other name) would be the inclusion of the offsets property. It seems a shame to clutter RegExp.prototype and String.prototype with additional methods whose functionality is almost nearly identical.
I'm not strictly opposed to a different-named method, but it would be far from my first choice.
@mathiasbynens:

(maybe mention this in the README?)

In https://github.com/rbuckton/proposal-regexp-match-offsets#motivations I offer a specific example. VS Code uses Oniguruma for TextMate grammar parsing because ECMAScript's RegExp lacks the necessary features. The missing feature this addresses is the inability to determine the offset into a match for a capturing group.

If we were to add a new method, an option would be to just return a list of [start, end] indices...

@schuay that's an interesting suggestion, as it avoids slicing the string for captures you don't need, but unfortunately puts that overhead on the user.
If we took that approach, I could imagine the resulting being an array of start/end tuples, with an optional groups object similar to exec. It wouldn't need the index property that exec adds, since index would effectively be match[0][0]. Unmatched capture groups would probably be either null (to match the behavior of exec for the whole match) or undefined, but I'd prefer to see the result array be packed rather than sparse.
You could even effectively reconstruct the exec result from that result fairly easily:
const exec2Result = regexp.exec2(text); // ignore method name for the time being...
const execResult = exec2Result.map(pair => pair ? text.slice(pair[0], pair[1]) : "");
execResult.index = exec2Result[0][0];
execResult.input = text;

In https://github.com/rbuckton/proposal-regexp-match-offsets#motivations I offer a specific example. VS Code uses Oniguruma for TextMate grammar parsing because ECMAScript's RegExp lacks the necessary features. The missing feature this addresses is the inability to determine the offset into a match for a capturing group.

My suggestion was to mention this specific example, i.e. VS Code using Oniguruma for [reasons], in the README. Just pointing to Oniguruma is not as useful IMHO as it doesn’t detail any actual use case.

In https://github.com/rbuckton/proposal-regexp-match-offsets#motivations I offer a specific example. VS Code uses Oniguruma for TextMate grammar parsing because ECMAScript's RegExp lacks the necessary features. The missing feature this addresses is the inability to determine the offset into a match for a capturing group.

My suggestion was to mention this specific example, i.e. VS Code using Oniguruma for [reasons], in the README. You already explain the hypothetical use case, but pointing to VS Code actually wanting this helps make the argument.


Add a tab to the dashbaosd with the profile itself (not the editor)

When editing your proofile, its good to be able to see the edited pofile. Plus. generally useful.  And the profile mmenu ""go to profile" only works under some conditions.

A small but useful change to the menu seems to work when  tested


Identify URL and Structure of Page That lists all quests

In order to start building a list of all of the quests in Runescape, we first need to identify the site, and the structure of said site that holds this data.
AC

A new python file is started specifically for scraping the quest list
This file has a function that returns the HTML object of the quest list.

Beyond This Task
Once this task is done, we will then be able to review the quest list structure, and take any information from it that is needed.

PR/Merge #7 Resolved this issue successfully.


Update region tags for Cloud Run

Description
Fixes #
Note: It's a good idea to open an issue first for discussion.
Checklist

[ ] Tests pass
[ ] Lint pass: bundle exec rubocop
[ ] Please merge this PR for me once it is approved.

Unrelated tests are failing


weight decay feature

Hi,
Is it possible to implement weight decay to control the over-fitting problem although these is dropout.

How would you see this working, as a decay per epoch?  Would it be a relative decay or absolute?

I am pretty sure pylearn2 has weight decay as a feature - it works layer-wise, similiar to dropout. I think this is very important feature because the only way you can regularise now is dropout - and l2 would be nice to have.

Maybe an additional decay parameter for the Layer objects? Are per-layer learning rate decay common at all? Never used them.


File loading error

#14 seems to break the track loading
a simple
mtrack_list = mdb.load_all_multitracks()

for track in mtrack_list:
    print track

now results in:
(.env)$ py test.py
Traceback (most recent call last):
  File "medleydb/utils.py", line 62, in load_multitracks
    yield M.MultiTrack(multitrack)
  File "medleydb/multitrack.py", line 75, in __init__
    self.title = _path_basedir(mtrack_path).split('_')[1]
IndexError: list index out of range

I think it's time for some continuous build pipelines eh?
Looking into this...

This is fixed now in v1.1


Resolve bug: task run interactive doesn't work

ubuntu@ip-172-31-24-161:~/test/temp$ datmo task run --ports 8888:8888 --interactive
Running a new task
An exception occurred: 'controller.task.create.arg'

closed with pr #85


getEnv  import  error!

Bug description
Test fail because of getEnv  import  error in dzcode.io\frontend\src\common\utils\index.spec.ts:4
Environment
OS: Windows 10
node version:  v13.12.0
Expected behavior
Test pass.
Error outputs
Test suite failed to run
Jest encountered an unexpected token

This usually means that you are trying to import a file which Jest cannot parse, e.g. it's not plain JavaScript.

By default, if Jest sees a Babel config, it will use that to transform your files, ignoring "node_modules".

Here's what you can do:
 • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
 • If you need a custom transformation specify a "transform" option in your config.
 • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

You'll find more details and examples of these config options in the docs:
https://jestjs.io/docs/en/configuration.html

Details:

dzcode.io\frontend\src\common\utils\index.spec.ts:4
import { getEnv } from '.';
^^^^^^

SyntaxError: Cannot use import statement outside a module

  at Runtime.createScriptFromCode (node_modules/jest-runtime/build/index.js:1295:14)`

Hi @mohsenuss91 thank you for reporting this bug, can you please provide more context on this,

OS? e.g. Window 7
Node version? node -v
did u run yarn on the root?
yarn lint:check passes on your machine?

i'm unable to reproduce this bug on my machine.

Hi @mohsenuss91 thank you for reporting this bug, can you please provide more context on this,

OS? e.g. Window 7
Node version? node -v
did u run yarn on the root?
yarn lint:check passes on your machine?

i'm unable to reproduce this bug on my machine.

Bug description updated with OS and node version.

I'm unable to reproduce the bug on my windows 10 machine with node 14, all tests are passing.
Can you please retry running these commands on the root folder of the project:

yarn
yarn lint:check
yarn test

I'm unable to reproduce the bug on my windows 10 machine with node 14, all tests are passing.
Can you please retry running these commands on the root folder of the project:

yarn
yarn lint:check
yarn test

Tests pass successfully when I use:
yarn test
or
npm tun test
I'm having these errors with VSCODE debugger!

Thank you for clarifying @mohsenuss91 , debugging is not yet supported, if you want this feature please open a Feature Request issue for it and we will address it in the next milestone, additionally, you can implement this feature yourself in PR if you'd like to.
Since test are passing, I'm closing this issue.


Add option to remove deprecated fields from documentation

Describe the problem
We have quite a few of deprecated fields in our API schema and GraphiQL does a good thing with these: it hides them by default and moves them into a separate section of the docs.
Describe the proposed solution
It would be great if magidoc could either:

completely remove the input fields
move them to a separate section

Importance
Nice to have

Hi @leonardehrenfried. Sorry for the late reply.  You're right, that is a good proposition. I'm curious, how would you organize that ? There are many things that could be deprecated.

a type
a query/mutation/subscription
a field inside a type or a query

In the current page layout, I'm not sure there is a good way to show them in a separated section. They are already crossed out so I guess it separates them a bit from the others.
Removing them completely is a good option however, I would not have any problem adding that option.
That being said, I am quite busy in the recent days. It might take a little while until I get time to implement this. Contributions are always appreciated if you get some time to look at it and if this is urgent.

Personally I would totally be happy if there was a way to completely hide them.
It's doesn't stop me from using this tool though, it's definitely in the nice-to-have category.


paged results yield Object instead of array

:warning: Introduces Breaking Changes :warning:
Currently fetching paged results yields an array with extra fields slapped on. For example, fetching the list of commits for a repo would yield an array with extra functions (like nextPage() and lastPage()) on it.
Instead, this makes paged results return an object with the following structure:
{
  items: [ ... ],           // the results
  nextPage: function(),     // the following are included only when GitHub provides them
  lastPage: function(),
  firstPage: function(),
  prevPage: function()
}

Motivation

Slapping extra functions onto an array, while cool is not very discoverable and it makes serializing the object (ie to add to sessionStorage for caching) much more difficult
When disableHypermedia is set then there is no way to get the link headers

TODO

[x] Always convert an Array returned by GitHub to {items: [...]} even when there are no link headers

+1 I'd support this change, despite the backwards incompatibility.

:+1:

I'm still thinking a bit on how this would work with .fetchAll and ES6 iterators but I think this is close to being mergeable.

Work on this is folded into #81:

.fetch() yields the following format: {items: [...], next_page_url: } (same as /search/* results)
.fetchAll() yields an array since there is no need to have a place for next/previous URLs

Hope that helps!


error on duplicate schema

Hi there. Is it possible to write a rule that would error if locally defined @client schema is a duplicate of schema from a remote endpoint? This would be very useful for projects in which client schema is written temporarily on the front-end and then implemented on the back-end later. It would be great to have the linter remind us to remove the local schema once it exists on the server

Hi, do you mean @client directives?

I guess to simplify, what I mean is if we have schema defined in two places, say:
parserOptions: {
  schema:[
    'schema-1.gql',
    'schema-2.gql'
  ]
}

if schema-1 contains a type A and schema-2 also contains a type A, then we have duplicated schema. This breaks some tools like Apollo's VSCode extension. It would be great if we could be alerted of this by the linter to make sure those tools are always working. Is that possible?


Make _get_version() more reliable

Add a limit to the SQL query to avoid "unread result found" in case the DB has 2 entries (which is not normal).
Also remove the unnecessary lock as we just perform a simple read.

Coverage decreased (-0.01%) to 95.87% when pulling 513d68fb428db749a9556a7f9a2dd6107749743d on get_version_reliability into 6cc3d9ad1b689e73bbfce6c66caa6667dac9650f on master.

Coverage decreased (-0.01%) to 95.87% when pulling 513d68fb428db749a9556a7f9a2dd6107749743d on get_version_reliability into 6cc3d9ad1b689e73bbfce6c66caa6667dac9650f on master.


wrong error display

When token is expired and i click on the input field, error should be hidden

Mine


Updated EDA test functions

Hey all, I have updated the test functions for the EDA. Please feel free to review and merge the branch.

The EDA test functions look good. Good coverage of the grid plot layout. Attached the test run results for reference.


Change section title "public health data"

Please change the section title "Public Health Data" to "Health registry data" and keep only the RUT link in this section. In Swedish, the title should be "Registerdata".

Is there a plan to have other pages there in the future? Otherwise I recommend removing the page hierarchy for RUT. That is, make  public_health_data/_index.md the RUT page and remove rut.md. Otherwise we are adding additional clicks for no reason.

We don't know yet, but it's likely to come other data from health registers and researchers coming from healthcare side - not sure if "register data" is perfect, and for now we only have RUT but there may be other things like biobank information and similar.
"Public health data" is quite particularly the situation in the country with numbers of people infected or hospitalized and we will never really have that data. I want to make sure people don't expect the portal to be the major frontend or dashboard like the FHM or Johns Hopkins' sites - focus is on research data really. "Health data" is also complicated because it's not apparent how we will ever really be able to serve it. So I am a bit uncertain how to handle it, and not sure what to. expect when it comes to future content.

Ok - how about we just make it RUT then, solves the question of the generic title? We can always revert this in the future if we want more pages.

Is this what we want?

Yeah exactly - will now work as a direct link instead of a dropdown (and no empty page when linked to from the top navigation).
Not sure if a different name would be appropriate is the only thing, I'm not sure how many people will know what Register utiliser tool really means (I don't).

@johanrung Any suggestions for title for the section? And also: who will fill it with text?

I've asked VR for content. If I don't hear from them by noon tomorrow, we'll decide something to do as a placeholder. Per suggested "Public Health Statistics" which I think is a better term and I think even with input from VR, the title could be "Public Health Statistics".

Any updates?


是否可以在picker

Is it possible to select a color by clicking?

Not at the moment. Feel free to open a PR with this additional gesture :)


add default impl for AsyncTokenCred

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-core


Feat: Keep response tab last position

It would be interesting to persist the position of the answer tab in the last location
When we use the client as in the image example, the position of the tab is always reset when a request is made
PS: I know about workbench.editor.openSideBySideDirection but it's not ideal as it changes the whole behavior of the editor

@andremacola You got the wrong extension. Your screenshots use [vscode.rest-client}(https://github.com/Huachao/vscode-restclient).  But I wish you good luck that the bug will be fixed there.
I think my extension already does it correctly.

@AnWeber Thks for response. Yep, the screenshot is from rest-client but the behavior is equal on httpyack.
Try to put the http tab at the same side of the json results tab. Like this:
https://github.com/AnWeber/vscode-httpyac/assets/3408809/14743c8a-10e8-47ce-8fa7-60dbe2b3b36e

Now it should work better. Now the editor always opens in the same place as the previous preview, as desired. However, it is unfortunately possible due to the lack of interface that I restore the arrangement after closing the document.

That's nice. Works flawlessly


Allow async arrow function to accept yield as the parameter

Fix #1747

Codecov Report

Merging #1754 into master will not impact coverage.

@@          Coverage Diff           @@
##           master   #1754   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files           1       1
  Lines        4031    4032    +1
  Branches      711     712    +1
======================================
+ Hits         4031    4032    +1

Impacted Files
Coverage Δ

dist/esprima.js
100% <ø> (ø)
:white_check_mark:

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 311bca0...315079a. Read the comment docs.


fix: disable emptyOutDir in non-development modes

Description 📖
This pull request solves the issue discussed in https://github.com/ElMassimo/vite_ruby/discussions/186
Staging mode was inconsistent with production mode
Background 📜
This was happening because the code was checking for mode === 'production', but we need the same settings in all non-dev environments (not sure about CI but I guess?).
The Fix 🔨
Replaced previous isProduction with !isDevelopment, also added a test but my test game is pretty bad, feel free to rework it

Hi Antoine! @antoinematyja
Looks great, thanks for your contribution!

Released in<EMAIL_ADDRESS>thanks again!


update bitcoin-tx to output witness data

In https://github.com/bitcoin/bitcoin/commit/7c4bf779e8b74e474551982a24f5acc265293abd , jl2012 updated getrawtransaction() to return the witness data for transaction inputs spending P2WPK and P2WSH.
bitcoin-tx doesn't call into this function and instead calls TxToUniv() in core_write.cpp, so bitcoin-tx calls don't return witness data.
I think the proper fix is to have getrawtransaction() call into TxToUniv() so we don't have duplicate code. For now, I'm just adding code to TxToUniv() to return witness data.

I've addressed @sdaftuar's code nit and also added the witness hash to the TxToUniv() output.
Suhas, @TheBlueMatt , do you have any further comments before I open a PR against https://github.com/bitcoin/bitcoin/ ? I'm planning to open this PR and also https://github.com/HackerResidency/bitcoin/pull/4 to deduplicate the TxToUniv/TxToJSON code.

utACK, though you might just want to do them together upstream, to make it more clear what you're doing.

Thanks. All squashed. I'll open a PR to bitcoin/bitcoin.

I've opened a PR to bitcoin/bitcoin. Closing this PR.


drivers: led: Add lp5569 led controller driver

Add a minimal driver and sample for the ti lp5569 led controller. The driver supports multiple instances. Commands on|off|set_brightness are supported.
In addition this PR adds the possibility for the driver to use device power management with or without e.g. using a power domain for power gating the led controller device.

@simonguinot thanks for your review! I added all your suggestions.

Hi @Mani-Sadhasivam. It is good for me. Please have a look.

And let's add more eyes in order to get one more review.


Uncontrolled: onChange fires when initial value is set

I'm using the uncontrolled component and loading the initial contents asynchronously. When it's arrived I make a call to setState which triggers a rerender with the new initial value prop. Unfortunately, that triggers the onChange function. So is there a reliable way to discriminate between an change coming from a changed value prop, or from the editor itself?
Using onChange = function (editor, diffData, value) { ..., I get the following for diffData:
{ from: {line: 0, ch: 0},
  origin: undefined
  ...
}

Is it a good way to just check whether origin is undefined? Or are there other cases when that's the case?
This may be related: https://github.com/JedWatson/react-codemirror/issues/5#issuecomment-162303642
Thanks for a great tool!

@mb21 thanks for opening this up. I haven't yet ran into this issue where the initial value in onChange is causing a problem. Trying to think of a workaround, would you be able to set a boolean through the editorDidMount prop? Perhaps set something such as init = false => editorDidMount () => init = true, and check of a if init [...] inside onChange This way you can ignore any erroneous  event firings until you set init = true, whether through the lifecycle mount or on the success of your ajax call.

Thanks for getting back to this. But I've since switched to the Controlled component, so this isn't an issue for me anymore... feel free to close this.

@mb21 will do, sorry for the terribly late reply, I'm glad you've been able to make forward progress 👍


Remove unused defaults module

I think this was done to be more explicit about the actual default value. I don't think it was a bad idea, but also not really necesary.


Include interrupt for version format mismatch

If there is a version mismatch:

Interrupt user with warning
Prompt for confirmation to continue (Y/N)
Register and continue feature creation

Implementing in BENS-0022

Implemented in BENS-0022


Metadata contracts should pass through missing non-POCO context

V3.NET SDK is ignoring the missing non-POCO context, resulting in unexpected behaviors as the service contract evolves.
For metata serialization V3 SDK is using JSON.net, JsonExtensionDataAttribute might be handy to address this gap.

@sourabh1007 can you please take a look into it?


docker-compose - should be persisted

motivation - to avoid downloading the latest version when docker-compose file is missing for the engine.
solution -
when user upgrade its engine we should download and persist the docker-compose in both the tmp directory and ide persisted storage

The docker compose file is bundled with the plugin.
it is unpacked to a persistence folder on the user's machine and reused without unpacking again or downloading.
see  org.digma.intellij.plugin.docker.ComposeFileProvider
when upgrading the local engine the plugin will download the latest compose file and replace the existing file.
if the file is deleted somehow the plugin will unpack the bundled compose file again when its necessary.


get_rotation()[2] means left rotation of head is not working properly as compared to right side head rotation.

The get_rotation()[2] does not change properly and is always in between 0.0 to 0.15.
Compared to that  get_rotation()[1] right side rotation is between 0.1 to 0.6 .

get_rotation() provides Euler's angle; get_rotation()[2]  and get_rotation()[1] are rotation around different axis


Updated build process checks for referenced libraries

Summary
Some users have reported build issues with cleveldb and with the libwasmvm (both of which are shared libraries).  The build process should be modified to require specific libraries with verified hashes to lock in these dependency versions.
Problem Definition
As the following linker report shows, shared libraries (in this case leveldb and libwasm are not locked against specific versions.  This leads to issues because the implementation can vary widely across the revisions (or in the case of libwasm there is no restriction at all).
otool -L ./build/provenanced
./build/provenanced:
        /usr/local/opt/leveldb/lib/libleveldb.1.dylib (compatibility version 1.0.0, current version 1.23.0)
        /System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 1675.129.0)
        /System/Library/Frameworks/Security.framework/Versions/A/Security (compatibility version 1.0.0, current version 59306.101.1)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1281.100.1)
        @rpath/libwasmvm.dylib (compatibility version 0.0.0, current version 0.0.0)
        /System/Library/Frameworks/IOKit.framework/Versions/A/IOKit (compatibility version 1.0.0, current version 275.0.0)

Proposal
Require that all shared libraries target explicit compatibility versions.  When the library does not provide version information (such as libwasmvm that reports 0.0.0 require that an explicit file hash matches the expected build version.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

We can update this process to remove cleveldb support with 1.14.x it looks like...


Always query devices by name so hardware with duplicate IDs can be found

Feeling pretty good about this. We're having lots of trouble with by-id, especialy on arm systems. systemd doesnt appear to create it until a device is plugged in for instance on my novena (arm) I get a crash on listing ports until something is plugged in.
Need to get more platforms testing to see if theres anything were not thinking of before the switch

@abrinsmead what did you develop and test on so far?


Robot console plugin system

I recently made a code contribution to the Robot Framework, which I think will improve usability and flexibility.
I have extended the existing "ConsoleOutput" feature in the Robot Framework to further extend its functionality. This modification allows more easily to select console output types that are determined with the --console argument.
The code that I added discover and load entry points based on the provided group and name. If multiple entries are found, it issues a warning and proceeds with the first discovery. This eliminates the need for manual updates when new/custom output types are added.
I made these changes to the Robot Framework because they provide practical solutions to streamline the selection of console output types. This improvement would help us hard in developing the project we are currently working on. The project is intended to allow users to easily track their test status, but we would also like to be able to easily call the project as a --console option. The modification also has a contribution in the project for other developers using the Robot Framework, provided others also want to pass their own values to the --console argument.

This sounds like a really useful enhancement, but it needs some more planning.
First of all, we have lot of other APIs that would benefit from a better plugin API and we should plan the plugin system accordingly. We don't need to add all possible plugin APIs now, but the design must be generic. The plugin API would also need documentation and tests.
Secondly, the logging API currently used by console outputs isn't public. We have also lately been changing it as part of logging related changes needed by listener API enhancements. Once it gets stable, making it public shouldn't be an issue. It will basically be the same API as the enhanced listener API v3.

I just realized I hadn't understood the PR properly. I though the ideas was to be able to just use robot tests.robot and a registered console output would be used. It seems that the idea is to be able to register plugins, but they need to nevertheless be enabled like --console myplugin. That's otherwise a nice approach, but it would be inconsistent with how, for example, listeners and model modifiers are registered. With them you can either use a name of the module (needs to be importable) like ´--listener exampleor an absolute or relative path to a module like--listener example.py`. I believe we should use the same approach also with console outputs. The existing tooling we have for importing modules ought to make that pretty easy.
The above said, a plugin API would be really nice. I believe it should make it possible to automatically enable things a custom console output so that separate command line option wouldn't be needed. That's a lot bigger task than making the console output customizable, though.

(For reference: The Author is a student employee of ours (@Qteal), I'm the Product Owner of the project he's working on.)
So the preference would be to change this mechanism to be in line with the existing listener style arguments (via imports) for now. Would the PR then be acceptable or would the lack of public API still be a problem?
An alternative is that we try to integrate our console improvement project with RF itself, but that will be a harder sell internally, plus you'll end up with the maintenance bourdon :smile:.
What is the best channel to engage in a discussions about a plugin API? Because we would love to have a one for everything from listeners to libraries, even if it takes until RF8.0 :smile:.
Our reasoning for wanting the API is that the current system forces either a lot of unstructured packages (with lots of top level files) or exposing a lot of technical details to users who don't want or need to know them (package structures etc). Also a lot of very long command line arguments and the inevitable associated pile of scripts to cut down on those.


remove motoko mechs details fixes

Removed css fixes on Entrepot because we're moving towards fixing css directly on canister.

added - iceleb final (video) NFT - details and thumbnail fix


Exception thrown if WebHostBuilder config's application name does not correspond to an assembly name

Steps to repro:

Configure WebHostBuilder with .UseSetting(WebHostDefaults.ApplicationKey, "Something")
Ensure Something.dll does not exist
Call .AddMvc() on corresponding IServiceCollection

Outcome:
An unhandled exception of type 'System.IO.FileNotFoundException' occurred in System.Private.CoreLib.ni.dll

Additional information: Could not load file or assembly 'Something, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified.

The docs - https://docs.asp.net/en/latest/fundamentals/hosting.html - don't mention that Application Name is used for loading assemblies.
A bug in the docs or a bug in MVC?

It's not a bug, it's by design. Sure we can update the docs. What were youa actually trying to do?

My guess would be the Application Name would be useful for different modules sitting on ASP.NET to know the name of the application for error messages, logging, etc.  At least, that's my use case - I'm looking to use IHostingEnvironment.ApplicationName for alert emails etc from the app framework I'm building.
To be honest there's already a "Startup Assembly" key - that kind of sticks out as something that specifies an assembly, would it be better to call this "Application Assembly" if that's what it's used for?
(I just found https://docs.asp.net/projects/api/en/latest/autoapi/Microsoft/AspNetCore/Hosting/IHostingEnvironment/ which explains it, but it seems a bit of a hack - like the host would set this to the startup assembly name /by default/ so that you have an Application Name to work with if you don't override... then MVC happens to pick up this string which defaults to an assembly name later on?)

My guess would be the Application Name would be useful for different modules sitting on ASP.NET to know the name of the application for error messages, logging, etc. At least, that's my use case - I'm looking to use IHostingEnvironment.ApplicationName for alert emails etc from the app framework I'm building.

Seems fine. Just don't set it. It's possible we should have made it get only. Ah well.

To be honest there's already a "Startup Assembly" key - that kind of sticks out as something that specifies an assembly, would it be better to call this "Application Assembly" if that's what it's used for?

That's used in very narrow scenarios where you want to change the Startup assembly. It doesn't flow out of Hosting either.

@danroth27 if you don't mind me saying so, updating the docs might still be a good idea?  I've copied an excerpt from https://docs.asp.net/en/latest/fundamentals/hosting.html below, which really does make it seem like ApplicationName can be used ad-hoc, doesn't depend on an assembly with that name existing, and won't throw exceptions?
Configuring a Host
The WebHostBuilder provides methods for setting most of the available configuration values for the host, which can also be set directly using UseSetting and associated key. For example, to specify the application name:
new WebHostBuilder()
    .UseSetting("applicationName", "MyApp")

Host Configuration Values
Application Name string
Key: applicationName. This configuration setting specifies the value that will be returned from IHostingEnvironment.ApplicationName.

@kierenj Yup, I totally agree: https://github.com/aspnet/Docs/issues/1870

I have also tried this but it was giving me same reflection error.
As it always going to throw an exception why can't we make changed to docs and even if possible not allow the user to set this property in first place??

@Vedant-Koditkar Setting the property is still useful for cases where you want to specify the assembly that the dependency context should be loaded from.


ci: github actions

:tada: This PR is included in version 1.0.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Cvx parsing

Updated the Github Action to pull new CVX file from CDC and update CVX terminology on the first of every month.

Workflow - I don't see where the commit is happening in cvx-workflow, how does it commit the changes to the branch? Also would option to run manually using workflow_dispatch.
Script - this seems to be just writing a diff file. Is the expectation that the ValueSet would be updated manually from this? Any reason the value set can't also be updated in the script? It seems like you could parse the json value set for CVX directly in retrieve_old_cvx and use that as the original for the diff, rather than writing a separate file.

This has been merged manually.


liveReload doesn't render changes when using local module replacements, despite acknowledging file change

What version of Hugo are you using (hugo version)?

$ hugo version
v0.111.3

Does this issue reproduce with the latest release?
Yes

Issue Description
Not sure if this bug presents for ALL instances of local module replacements, or if I've encountered some kind of edge-case due to the specific module I'm importing, but what's happening is:
have a (admittedly large) external content module, mounted with hugo modules like:
module:
  replacements: "github.com/user/repo/some/subpath -> ../.external/subpath"
  imports:
    - path: github.com/user/repo/some/subpath
      mounts:
        - source: "a.md"
          target: "content/docs/a-prime.md"
        # ... more mounts

When a.md is modified in ../.external/subpath,
-- the hugo server stdout acknowledges that the file was written,
-- and the --debug output also acknowledges the file was changed and re-rendered,
-- and the browser window flashes a refresh
but the content remains the same.
The same behavior is observed with --disableFastRender and --renderToDisk, and the file in public/docs/a-prime.html is not up-to-date.
This behavior is observed on both M1 and Intel macOS machines, as well as a x86_64 linux machine.
This behavior is also observed when using the HUGO_MODULE_REPLACEMENTS env var, and with the replacement target being outside the site directory.
Fully stopping and restarting the hugo server allows the re-render to persist.
Other flags tried: --ignoreCache --gc --renderToDisk --disableFastRender --noChmod --poll 1000ms (in several combinations)
(Repro can be provided upon request, I would just have to anonymize some things).

Not sure if this bug presents for ALL instances of local module replacements

It's not. I suspect it's a symptom of your mount setup confusing the "what changed" detection. The original idea behind the file mounts was to mount file folders. Files works, too, but you may have found a limitation. I have a reimplementation of the parts involved here in the pipeline which may or may not fix your problem, but it will certainly improve the situation.

I have a reimplementation of the parts involved here in the pipeline which may or may not fix your problem

just curious as a budding contributor, do you have a branch that I can look at? have felt the pains of a couple fsnotify/file-watching related bugs/panics, and just curious to understand the flow better

I am having the same issue. Only my css changes get updated, any changes in my templates only get updated when I restart hugo serve. The live reload does acknowledge change but it does not get rendered.

Also I have notice, while changes to the templates aren't updated in live reload, changes to content md files does update the site.

I'm finding that Hugo simply doesn't detect changed files reliably.  I'm using:
hugo v0.111.3-5d4eb5154e1fed125ca8e9b5a0315c4180dab192+extended linux/amd64 BuildDate=2023-03-12T11:40:50Z VendorInfo=gohugoio

on Debian 11, running hugo serve (and adding --disableFastRender seems to have no effect on this problem).
When I, e.g. edit a CSS file and save it, sometimes Hugo detects the change, printing messages like:
Change detected, rebuilding site.
2023-05-15 23:45:15.098 -0500
Total in 183 ms

And then the browser gets refreshed automatically.
But other times, editing the same files is simply not noticed by Hugo, and I have to restart the hugo serve command for the changes to take effect.  I can't find any rhyme nor reason to why sometimes Hugo notices the changes and sometimes it doesn't.

The browser does refresh but the template code remains unchanged. I am using webpack with browser sync. In order to get an update from the template, I have to restart it or change a md file. Changing md doesn't always work but currently is my only way to avoid a full rebuild.

I've noticed that using the --poll option seems to help detect changes that are otherwise missed.  Also, the file I'm changing is a few subdirectories deeper than the directory which Hugo says it's watching for changes.  Maybe the filesystem watcher isn't catching changes in subdirectories (or not reliably), while the polling method is?

I am experiencing same here on newly updated v0.115.0
Changes to the config files do not update in the browser at localhost

@freerbeing Does using the --poll option make a difference?

@alphapapa --poll doesn't make a difference
nor does
--disableFastRender
I also notice that:
Once server is started I can make changes to content files which update in the browser as they should
But as soon as I make a change in config folder then all updating stops
Additionally, sometimes server/localhost stops serving any page at all, does not respond to any click/request
The page tries to load but does not
Also cured by a server restart
Server stop can also be slow - does not immediately respond to Ctrl+C
I wonder is this is related?

I'm using:
hugo v0.117.0-b2f0696cad918fb61420a6aff173eb36662b406e+extended darwin/arm64 BuildDate=2023-08-07T12:49:48Z VendorInfo=brew

We have the same problem in our project. Live reload does not work when changing a component from a module. But if the code is changed in the module and another change in layouts then both changes will be visible. I found a hotfix. If you make a symlink from the module to the layouts directory, it will work.
cd layouts
ln -s ../component-library/ _tmp
cd ../
hugo server

./layouts/_tmp -> ../component-library/
In this case, a module change will be detected as a change in layouts and the page will be refreshed.

liveReload doesn't render changes when using local module replacements, despite acknowledging file change

I am also unable to reproduce this problem with v0.111.3 or later. I have tested multiple variations of the described mounts setup... no problems.
All of the other comments in this thread are describing other problems which are already covered in other issues.


Memory Issue with Log Scale Minimums set to Powers of 1000

Using ChartJS 2.8.
Conditions for Bug:

ChartJS Bubble Plot
In IE 11 (bug is not present in chrome)
Axis uses log scale
Minimum of axis is a (positive integer) power of 1000

When those conditions are met, the memory usage of chartjs explodes to over 1.5GB of memory.
If the minimum of the axis is 999 or 1001, the bug does not appear.
I replicated the bug with the scale minimum set to 1,000 and 1,000,000.  Other powers of ten do not cause the issue.
The bug happens when calling the chart constructor.  It is almost as if there is an infinite loop because the function never returns.

Do you have a running sample? Preferably in JSFiddle, CodePen or similar.

Closing due to lack of test case


'Description' and 'User Field' fields update

Need to devise a good way to stuff all info about sandboxes in those.

Decided to drop User Field.. Have to scroll for it.


Cannot update the AllowedVmSizesInLab policy through UI after login with the custom role

Hello,
I have followed the documentation to create the custom role and assign it to a user.
After I logged in using the user with the custom role, I saw that the AllowedVmSizesInLab policy UI is still disabled for this user. I am not be able to update the policy.
How can I verify if the permission is granted to the user or not?
Thanks,
Amber

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

ID: b77a2644-42a1-3aaa-afeb-872d5242c78f
Version Independent ID: 94761f79-0d77-0c1e-eb2a-d02fd8008bfd
Content: Grant user permissions to specific lab policies
Content Source: articles/lab-services/devtest-lab-grant-user-permissions-to-specific-lab-policies.md
Service: lab-services
GitHub Login: @spelluru
Microsoft Alias: spelluru

@jyzhang56 - Apologize for the delay. I was able to reproduce the issue and opened a bug with the product team. I will update this GitHub issue as I hear from them. Thanks.

##reassign:juliako

Closed per @j-martens

Thanks for sharing your feedback. We apologize for our delayed response. Due to the sheer volume of feedback in our repo, we have decided to prioritize the newest issues and are automatically closing our oldest issues with an Issues bot. If you believe your feedback is still actionable, then either respond directly in this issue or open a new documentation issue and we’ll review. #please-close


[bug] Using bindings as a depency fails to compile.

ver: ff8853fde770fac31ea071892242e12cd386a3f1
After adding as a dependency
piltover_bindings = { package = "bindings", git = "https://github.com/keep-starknet-strange/piltover.git", rev = "ff8853fde770fac31ea071892242e12cd386a3f1" }
The whole program failes to compile.
3 | / abigen!(
4 | |     AppChain,
5 | |     "../target/dev/piltover_appchain.contract_class.json",
6 | |     output_path("src/bindings.rs")
7 | | );
  | |_^

How to reproduce:
Add a dependency to any program
Proposed solution
Include compiled json in this repository

Should be solved in #32 with cainome being bumped. Don't hesitate to re-open if necessary.


Add 1PR logo to the navbar and header

Interesting idea.  I personally like the octicon pull request logo we are using however.

@sabatesduran - I'm with @nrobinson2000 on the navbar logo, but I think an in-page logo might be cool :octocat:


Add ownerrefs to machines owned by CPMS

Add ownerrefs to machines owned by CPMS. From function TODO:
TODO: Iterate over the MachineInfos, for each Machine, check the owner references for an owner reference matching
that of the current CPMS (it should be the controller so should be easy to find).
If required, look up the GVK using the rest mapper from the GVR, then uses metav1.PartialObjectMetadata to Patch
the owner references. This should mean we can update the metadata of any type given we know the GVR and existing
ObjectMeta.

Once you fixed the nit, please squash the fixups into the feature commit to make the history clean

/approve
/lgtm


hi, does it work inside a background service?

this looks like tricky, am not sure if it can detect apps goes back from background and using inside backgounrd service.
What I want to do, is judge inside backgounrd service, if in background then send notificatio when new messages comes in, otherwise not notify since user already on UI.
Could lfutter-fgbg able to do this?
Why i need this is bacause  i can not found a good way to control backgroudn services with flutters's widgetBindgObserver

I'm unsure. Is this a native background service? Sample code? There could be APIs in notification system to do this, because this is how most apps behave.
A workaround will be your Flutter app storing the last flutter_fgbg event somewhere your background service can access. Service can just check the value to figure out app's current state. But there might be an issue when user swipe closes the app from recent apps or app crashes during the use. That will have to be checked.


Reset to Default Button

Description of the new feature / enhancement
When pressed each utility is reset to a default state and when clicked in General all utilities are reset to default
Scenario when this would be used?
It would be a quick fix if you are seeing some strange behaviour in a utility and are unsure why it is occurring.
Supporting information
No response

https://github.com/microsoft/PowerToys/issues/3822

/dup https://github.com/microsoft/PowerToys/issues/3822


Correct mistakes in the password policy variable.

Now tested by creating a Terraform plan from a repository that uses the avm module.

Adding checks here ->  https://github.com/schubergphilis/terraform-aws-mcaf-landing-zone/pull/50


Allow lookup of GVK from client-go types

This is useful when constructing clients from type. WE don't do this
yet, but splitting out to avoid larger PR.

wanna to know what's this for, right now we donot call this function for istio API at all

I have another PR that is going to call this but I do not want to bloat the
PR
On Tue, Jun 6, 2023 at 6:56 PM Zhonghu Xu @.***> wrote:

wanna to know what's this for, right now we donot call this function for
istio API at all
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/45308#issuecomment-1579741711, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXMWGISSNXYPBQUWA6TXJ7NV7ANCNFSM6AAAAAAY4ROWCE
.
You are receiving this because you authored the thread.Message ID:
@.***>


#432 - Linked Accounts: login cancellation not handled - Implemented

https://github.com/Microsoft/EmbeddedSocial-iOS-SDK/issues/432 implemented

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Vadim-Bulavin sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Feat: Lets Update menu make them using Cards #320

Feat: Lets Update menu make them using Cards #320
updated Menu-Page Now Showing Cards
Fixes # 320
Type of change
Please mark with an X the type that applies:

[ ] Bug fix (non-breaking change which fixes an issue)
[ y] New feature (non-breaking change which adds functionality)
[ ] Refactor code
[ ] Documentation update
[ ] Other (mentioned in the issue number)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce them. Please also list any relevant details for your test configuration.
Test A: Describe here
Test B: Describe here (if required)
Screenshots and Videos
Please provide screenshots and videos of the changes you made.
https://github.com/VanshKing30/FoodiesWeb/assets/101548649/422a044b-dd95-4910-a1d1-e643608b058e
Checklist
Please mark with an X the items that apply:

[ y] My code follows the style guidelines of this project
[y ] I have performed a self-review of my code
[y ] I have commented my code, particularly in hard-to-understand areas
[y ] I have made corresponding changes to the documentation
[ y] My changes generate no new warnings
[ y] I have added tests that prove my fix is effective or that my feature works
[ y] New and existing unit tests pass locally with my changes
[ y] Any dependent changes have been merged and published in downstream modules

hey @hustlerZzZ Can you please review this pr


Explicit python version dependency

It's related to https://github.com/tensorflow/tensorflow/issues/31983
Using static  python version dependency description is better at least for poetry,
like backports.weakref or enum34 has one.

Sorry, wrong user and i will recreate another one.

I opened #32758


Bugs: Actions not working, can't assign user, settings not updating

Hi,
So I got it installed fine once I actually followed the instructions. :)
Here's a few bugs I've found:

I can't assign myself to a task, even if I'm the only user created. It stays unassigned
Settings don't update automatically - for instance, I edit a board then click save, the modal just sits there. I then have to close it - on occasion, I have to refresh the settings to see the changes as well
When I create a new task, I have to refresh the page to see it
I can't create an action to change the color of a task upon setting the category

It looks like it's got something to do with the js in the api folder - I've set it all to 777, so I'm not sure what's going on. I'm getting 500 (internal server error) in the inspector, both on angular.js as well as jquery.
Thanks.

I have same things , i run chmod 0777 api/taskboard.db after the first run (ie after creating the database)
Cheers.

I'd need more information to figure out what's going on here. There is no JS in the API directory, so I'm not sure what you meant by that. The only thing you should need to make 0777 is the api directory and everything should work from there.

Closing due to lack of response. Please let me know if this needs to be reopened.


Detail view layout

Fix Css for Detail Page Layout. It comprises of following components

Detail View Header
Total Links Component
Assignees, Creator, Iterations Wrapper
Labels Dropdown

Issue:
https://github.com/openshiftio/openshift.io/issues/963
https://github.com/openshiftio/openshift.io/issues/964 (Part of it has been resolved)
https://openshift.io/openshiftio/openshiftio/plan/detail/1551
https://openshift.io/openshiftio/openshiftio/plan/detail/1547

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-2 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-1 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-3 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-4 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-5 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-7 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-8 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-9 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-10 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@vikram-raj I didn't get this issue. Can you please tell how to reproduce this issue

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-11 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil comment author avatar issue is still there.

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-12 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-13 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-14 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

Not sure why but I cannot open the preview. Can anyone else open it?

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-15 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-16 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil In the board view user avatar shrink.

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-17 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-18 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil The issue https://github.com/openshiftio/openshift.io/issues/963 asks for larger description and comments fields, but the field sizes appear to be the same as that in preview mode.

@SMahil The issues mentioned here https://github.com/openshiftio/openshift.io/issues/964 are not fixed. I cannot enter text into description or comment fields without using the Pen icon. Also, saving my changes via Keyboard is not working.

Escaping edits using keyboard is not working @SMahil

@SMahil The issue https://openshift.io/openshiftio/openshiftio/plan/detail/1547 still exists.

@rgarg1 As per UX Suggestion I make description fields and comments area consistent rather than making larger.
afa escaping edits using keyboard concerned, I didn't add new functionality enhancement
For entering text into description or comment fields without using the Pen icon, that should markdown component feature which is in patternfly-ng

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-19 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil Agreed! The original defects which are not addressed in this PRs need to be updated accordingly so that the issues are not lost. Can you pls update the defects referenced?

@SMahil The issue https://openshift.io/openshiftio/openshiftio/plan/detail/1547 needs a fix.
Also, the comments and description fields are overflowing and need a fix.

@rgarg1 I fixed #1547. Also for overflown content, I have added temporary fix. This has to be fixed in markdown component. Please verify the changes

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-20 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil Both issues are fixed. Great work. Approving!

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-21 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-22 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io

@SMahil fabric8/fabric8-ui:SNAPSHOT-PR-2177-23 is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-2177-fabric8-planner.badger.fabric8.io


dynamic pick list state tracking

Hi, I'm not sure whether I'm missing something here.
In my code, I keep a list of items (DeviceInfoModel) which I need to mutate at runtime. I'd like to represent these as a pick list. All the existing examples (pick_list and game_of_life) use statically typed enumerations as pick list backing, so those do not represent my usecase.
My code looks like this (simplified):
use iced::{executor, pick_list, Application, Column, Command, Element, PickList, Row, Settings};

fn main() {
    Eyece::run(Settings::default())
}

#[derive(Default)]
struct Eyece {
    devices: Vec<DeviceInfoModel>,
    device_list: pick_list::State<DeviceInfoModel>,
    device_selection: DeviceInfoModel,
}

#[derive(Debug, Clone)]
enum Message {
    DeviceSelected(DeviceInfoModel),
}

impl Application for Eyece {
    type Executor = executor::Default;
    type Message = Message;
    type Flags = ();

    fn new(_flags: ()) -> (Self, Command<Message>) {
        // initial device enumeration
        let mut devices = Vec::new();
        devices.push(DeviceInfoModel { index: 0 });
        devices.push(DeviceInfoModel { index: 1 });

        (
            Eyece {
                devices,
                ..Default::default()
            },
            Command::none(),
        )
    }

    fn title(&self) -> String {
        String::from("Eyece")
    }

    fn update(&mut self, _message: Message) -> Command<Message> {
        Command::none()
    }

    fn view(&mut self) -> Element<Message> {
        const SPACING: u16 = 10;
        const PADDING: u16 = 10;

        let config = Row::new().spacing(SPACING).push(PickList::new(
            &mut self.device_list,
            &self.devices,
            Some(self.device_selection),
            Message::DeviceSelected,
        ));

        Column::new().padding(PADDING).push(config).into()
    }
}

#[derive(Debug, Default, Clone, Copy, PartialEq, Eq)]
struct DeviceInfoModel {
    pub index: u32,
    //pub name: String,
}

impl std::fmt::Display for DeviceInfoModel {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        write!(f, "{}", self.index)
    }
}

If you click on an item, the view does not "update" correctly, the first item is still selected.

fn update(&mut self, _message: Message) -> Command<Message> {
       Command::none()
}

Do you handle the message in the actual update function?

Not sure how I missed that, sorry for the noise.