python_requires not written when only --min-py3-version is specified

See repro:
def test_guess_python_min_version(tmpdir):
    setup_cfg = tmpdir.join('setup.cfg')
    setup_cfg.write(
        '[metadata]\n'
        'name = pkg\n'
        'version = 1.0\n',
    )

    assert main((str(setup_cfg), '--min-py3-version=3.8'))

$ pytest -vvv -k test_guess_python_min_version tests/setup_cfg_fmt_test.py
========================================= test session starts ==========================================
platform darwin -- Python 3.9.15, pytest-6.2.4, py-1.10.0, pluggy-0.13.1 -- /Users/mxr/src/mxr/unkey/venv/bin/python
cachedir: .pytest_cache
rootdir: /Users/mxr/src/asottile/setup-cfg-fmt
collected 67 items / 66 deselected / 1 selected

tests/setup_cfg_fmt_test.py::test_guess_python_min_version FAILED                                [100%]

=============================================== FAILURES ===============================================
____________________________________ test_guess_python_min_version _____________________________________

tmpdir = local('/private/var/folders/mg/2_7nx7493qg0vn9jhdp951tw0000gp/T/pytest-of-mxr/pytest-10/test_guess_python_min_version0')

    def test_guess_python_min_version(tmpdir):
        setup_cfg = tmpdir.join('setup.cfg')
        setup_cfg.write(
            '[metadata]\n'
            'name = pkg\n'
            'version = 1.0\n',
        )

>       assert main((str(setup_cfg), '--min-py3-version=3.8'))
E       AssertionError: assert 0
E        +  where 0 = main(('/private/var/folders/mg/2_7nx7493qg0vn9jhdp951tw0000gp/T/pytest-of-mxr/pytest-10/test_guess_python_min_version0/setup.cfg', '--min-py3-version=3.8'))

tests/setup_cfg_fmt_test.py:667: AssertionError
======================================= short test summary info ========================================
FAILED tests/setup_cfg_fmt_test.py::test_guess_python_min_version - AssertionError: assert 0
=================================== 1 failed, 66 deselected in 0.17s ===================================

probably worth fixing -- I think this can be fixed with a None check after trying tox and classifiers and if an explicit minimum version is passed then use that (otherwise do nothing)

The behavior repros when --py3-min-version is not passed too and a default is used for min_py3_version. So should the fix change the parsing code to determine when absolutely nothing is passed in, or could the fix just set python_requires to whatever the minimum is after ingesting all sources (tox envlist, python_requires in setup.cfg, min_py3_version flag, etc)?

yeah probably should do it for the default too -- I guess there'd then be no way to avoid python_requires being set by setup-cfg-fmt -- but I think that's a good thing (everything should set it always)

Going to try to get a PR for this out this week

Working on this and the behavior I'm writing is "set python_requires to whatever the minimum is after ingesting all sources (tox envlist, python_requires in setup.cfg, min_py3_version flag, etc)" but I wanted to confirm if this is really the desired behavior. For example the test test_min_version_removes_classifiers would not longer rewrite python_requires, classifiers, etc to be >=3.4 because >=3.2 is already in python_requires. Is that what you want?
Perhaps the right behavior is: if min_py3_version is passed in, use that to determine the min, otherwise, try to guess the min based on all the other sources. That has the convenient behavior of being able to drop/support python versions in the setup.cfg metadata by simply changing the min_py3_version flag

yeah the latter probably makes more sense

So I realized from the test_min_py3_version_updates_python_requires test that min_py3_version should only override python_requires when py27 is not supported in python_requires, which makes the implementation more complicated. Wdyt about min_py3_version being the floor for python_requires even if py27 is listed? (Effectively changing it to min_py_version)

I'm fine killing python 2 support if it's easy -- actually probably about time

sg


github - automatically delete branch after merge

someone with admin access will need to do this
https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/managing-the-automatic-deletion-of-branches

Thought this was already enabled? Can do this here if there is no objections to enabling it
cc @InfamousVague

I think it's sometimes helpful to keep the branches around for a little while after merging because a subsequent merge could incorrectly handle merge conflicts. in that case it would be helpful to have the old branch for reference.

There's a grace period of a couple days where you can undo the delete. I think I find myself spending more time deleting branches than encountering the case above

that's fair. and the branch still exists locally until the dev deletes it, so there's a backup most of the time too.

this is just a setting on admin 😂 we have this enabled on core as well

also if you need the branch we can just restore it, we don't need to keep hoarding branches

darius activated the setting


[Task]: Playground integration test: Run an unchanged precompiled example

What needs to happen?
Playground integration test: Run an unchanged precompiled example
Issue Priority
Priority: 3 (nice-to-have improvement)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[X] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue


Issue in model output. Model's output changed to NoneType

I used the following code to generate the augmented paraphrased sentences. After the last model's output changed to NoneType. Although I am iterating over my question lists, it outputs None. I am not sure about it and could not find any issues.
import time
ts = time.time()
augmented_questions = []
for question in finance_list:
  para_phrases = parrot.augment(input_phrase=question.lower(), use_gpu=False)
  print(f"Length of para phrased by model are: {len(para_phrases)}")
  for aug in para_phrases:
    print(f"Printing aug in this: {aug}")
    augmented_questions.append(aug)
    print(f"Appended the {aug} with type {type(aug)}")

te = time.time()
print(len(augmented_questions))
print(f"time taken to augment " + str(len(questions)) + " is "+ str(te-ts) + " seconds ")

ChatGPT suggested to add the line of code for checking the NoneType in output (line 7 below). Anyone has information why I am getting this NoneType error from model output?
import time
ts = time.time()
augmented_questions = []
#19 thy without .lower()
for question in finance_list:
  para_phrases = parrot.augment(input_phrase=question.lower(), use_gpu=False)
  if para_phrases is not None:
    print(f"Length of para phrased by model are: {len(para_phrases)}")
    for aug in para_phrases:
      print(f"Printing aug in this: {aug}")
      augmented_questions.append(aug)
      print(f"Appended the {aug} with type {type(aug)}")

te = time.time()
print(len(augmented_questions))
print(f"time taken to augment " + str(len(questions)) + " is "+ str(te-ts) + " seconds ")

Same issue here.


[Tizen] Handle relaunching event

Problem
Can't relaunch the lighting app on the Tizen device.
Change overview
Check the current app state and omit initialization.
Testing
Call two times command below:
app_launcher --start=org.tizen.matter.example.lighting discriminator 43 wifi true

/rebase


removed symlink to doc.html, not supported on Windows

I ran a quick test with compiling on Windows WSL using a Yocto genereted SDK.
So far its working well, only problem I found is that the doc/index.html file is symlinked.
This is not supported on Windows, hence the repo gets flagged dirty.
I've checked on my linux system and the link is broken anyway so I guess it is not a problem to just remove that file?

It is a shortcut to documentation generated by doxygen. But if it makes problems, it can safely be deleted.


Add cleanup function to remove all the objects from a bucket after tests

Partially fixes : https://github.com/minio/mint/issues/165
and improves logging by adding args in few test cases.

Updated the PR with a simpler approach to cleanup, since previous approach failed if there are multiple instances of minio-go tests running in parallel against a single server, cleanup deleted buckets belonging to other test instances as well - for example when travis runs multiple minio-go tests against play server.
Please take a look @harshavardhana @poornas

Partially fixes : minio/mint#165
and improves logging by adding args in few test cases.

Is this true that it partially fixes? @nitisht

Is this true that it partially fixes?

Let me change it to fixes, initially there were stale buckets left by minio-js as well
[2017-09-29 17:11:02 PDT] 100KiB 41b17354-ff91-478d-9a46-353830d48767/datafile-100-kB
[2017-09-29 17:11:02 PDT] 100KiB 41b17354-ff91-478d-9a46-353830d48767/datafile-100-kB.buffer

but that is not seen anymore.


Unknow error after entering user code on smart life integration in HA 2024.2

The problem
I have a problem integrating the beta version of Smart life into 2024.2 version. After entering the user code I receive an "Unknown Error" and cannot get the QR code
Your Environment
Include as many relevant details about the environment you experienced the bug in

tuya-device-sharing-sdk Version used:
Device diagnostic file:

Expected Behavior
If you're describing a bug, tell us what should happen
If you're suggesting a change/improvement, tell us how it should work
Current Behavior
If describing a bug, tell us what happens instead of the expected behavior
If suggesting a change/improvement, explain the difference from current behavior
Possible Solution
Not obligatory, but suggest a fix/reason for the bug, or ideas how to implement the addition or change

Change home assistant  default theme

Same here. Tried default theme and home assistant theme without success

Ok, so, for me, I rollback to last HA release, I redownload last release of HA, and reconfigure Tuya with default theme. After that My tuya integration works.


Restore coverage content filters

Re-covers 3 lines which were un-covered by PR #273.
Delivers https://www.pivotaltracker.com/story/show/137998397
Should stop those annoying, incorrect Coveralls failures which are caused by these lines not being covered.  E.g., https://coveralls.io/builds/10544157

Coverage remained the same at 87.859% when pulling 08df63c28d4ec5e97e58a5dc7695ada51ec1514b on JoeCohen:restore-coverage-content_filters into 156a8daefdb54d5fe5a0f6d68f18be03f96620d3 on MushroomObserver:master.


How to use CaffeOnSpark

I want to use the caffenet to train my data, and I use it to train my data on Caffe before, but I don't know how to use this model in CaffeOnSpark. The wiki just show me how to train a DNN, so I want to know how I modify the spark submit order to use the caffenet model, or some other way to use it.

I modify the example of spark submit in the wiki like the code below to use caffenet for training,
spark-submit --master ${MASTER_URL}
--files ${CAFFE_ON_SPARK}/caffe-public/models/bvlc_reference_caffenet/solver.prototxt,${CAFFE_ON_SPARK}/caffe-public/models/bvlc_reference_caffenet/train_val.prototxt
--conf spark.cores.max=${TOTAL_CORES}
--conf spark.task.cpus=${CORES_PER_WORKER}
--conf spark.driver.extraLibraryPath="${LD_LIBRARY_PATH}"
--conf spark.executorEnv.LD_LIBRARY_PATH="${LD_LIBRARY_PATH}"
--class com.yahoo.ml.caffe.CaffeOnSpark
${CAFFE_ON_SPARK}/caffe-grid/target/caffe-grid-0.1-SNAPSHOT-jar-with-dependencies.jar
-train
-features accuracy,loss -label label
-conf solver.prototxt
-clusterSize ${SPARK_WORKER_INSTANCES}
-devices 1
-connection ethernet
-model file:${CAFFE_ON_SPARK}/data/myself/myself_caffenet.model
-output file:${CAFFE_ON_SPARK}/data/myself/myself_result
uh, the solver,prototxt & the train_val.prototxt in the ${CAFFE_ON_SPARK}/caffe-public/models/bvlc_reference_caffenet, and my data in the ${CAFFE_ON_SPARK}/data/myself,
I wonder know whether my change is right, and when I use input the above code, it show me the error below:
17/01/11 14:27:19 INFO spark.SparkContext: Running Spark version 1.5.1
17/01/11 14:27:19 WARN util.NativeCodeLoader: Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
17/01/11 14:27:19 WARN spark.SparkConf:
SPARK_WORKER_INSTANCES was detected (set to '1').
This is deprecated in Spark 1.0+.
Please instead use:

./spark-submit with --num-executors to specify the number of executors
Or set SPARK_EXECUTOR_INSTANCES
spark.executor.instances to configure the number of instances in the spark config.

17/01/11 14:27:19 INFO spark.SecurityManager: Changing view acls to: master
17/01/11 14:27:19 INFO spark.SecurityManager: Changing modify acls to: master
17/01/11 14:27:19 INFO spark.SecurityManager: SecurityManager: authentication disabled; ui acls disabled; users with view permissions: Set(master); users with modify permissions: Set(master)
17/01/11 14:27:20 INFO slf4j.Slf4jLogger: Slf4jLogger started
17/01/11 14:27:20 INFO Remoting: Starting remoting
17/01/11 14:27:20 INFO Remoting: Remoting started; listening on addresses :[akka.tcp://sparkDriver@<IP_ADDRESS>:46792]
17/01/11 14:27:20 INFO util.Utils: Successfully started service 'sparkDriver' on port 46792.
17/01/11 14:27:20 INFO spark.SparkEnv: Registering MapOutputTracker
17/01/11 14:27:20 INFO spark.SparkEnv: Registering BlockManagerMaster
17/01/11 14:27:20 INFO storage.DiskBlockManager: Created local directory at /home/master/Downloads/spark_sdk/spark-1.5.1/blockmgr-0a1f42d3-9d6e-441d-b223-0f7b60df7607
17/01/11 14:27:20 INFO storage.MemoryStore: MemoryStore started with capacity 530.3 MB
17/01/11 14:27:20 INFO spark.HttpFileServer: HTTP File server directory is /home/master/Downloads/spark_sdk/spark-1.5.1/spark-df60b6b8-cbc9-45f4-a001-004923b2196e/httpd-1aed9920-fae1-46b8-a4d1-66ffaba49548
17/01/11 14:27:20 INFO spark.HttpServer: Starting HTTP Server
17/01/11 14:27:20 INFO server.Server: jetty-8.y.z-SNAPSHOT
17/01/11 14:27:20 INFO server.AbstractConnector: Started SocketConnector@<IP_ADDRESS>:36673
17/01/11 14:27:20 INFO util.Utils: Successfully started service 'HTTP file server' on port 36673.
17/01/11 14:27:20 INFO spark.SparkEnv: Registering OutputCommitCoordinator
17/01/11 14:27:20 INFO server.Server: jetty-8.y.z-SNAPSHOT
17/01/11 14:27:20 INFO server.AbstractConnector: Started SelectChannelConnector@<IP_ADDRESS>:4040
17/01/11 14:27:20 INFO util.Utils: Successfully started service 'SparkUI' on port 4040.
17/01/11 14:27:20 INFO ui.SparkUI: Started SparkUI at http://<IP_ADDRESS>:4040
17/01/11 14:27:20 INFO spark.SparkContext: Added JAR file:/home/master/Downloads/spark_sdk/CaffeOnSpark/caffe-grid/target/caffe-grid-0.1-SNAPSHOT-jar-with-dependencies.jar at http://<IP_ADDRESS>:36673/jars/caffe-grid-0.1-SNAPSHOT-jar-with-dependencies.jar with timestamp<PHONE_NUMBER>832
17/01/11 14:27:20 INFO util.Utils: Copying /home/master/Downloads/spark_sdk/CaffeOnSpark/caffe-public/models/bvlc_reference_caffenet/solver.prototxt to /home/master/Downloads/spark_sdk/spark-1.5.1/spark-df60b6b8-cbc9-45f4-a001-004923b2196e/userFiles-c70014b3-28ad-4b30-919f-35815b830b2f/solver.prototxt
17/01/11 14:27:20 INFO spark.SparkContext: Added file file:/home/master/Downloads/spark_sdk/CaffeOnSpark/caffe-public/models/bvlc_reference_caffenet/solver.prototxt at http://<IP_ADDRESS>:36673/files/solver.prototxt with timestamp<PHONE_NUMBER>918
17/01/11 14:27:20 INFO util.Utils: Copying /home/master/Downloads/spark_sdk/CaffeOnSpark/caffe-public/models/bvlc_reference_caffenet/train_val.prototxt to /home/master/Downloads/spark_sdk/spark-1.5.1/spark-df60b6b8-cbc9-45f4-a001-004923b2196e/userFiles-c70014b3-28ad-4b30-919f-35815b830b2f/train_val.prototxt
17/01/11 14:27:20 INFO spark.SparkContext: Added file file:/home/master/Downloads/spark_sdk/CaffeOnSpark/caffe-public/models/bvlc_reference_caffenet/train_val.prototxt at http://<IP_ADDRESS>:36673/files/train_val.prototxt with timestamp<PHONE_NUMBER>924
17/01/11 14:27:20 WARN metrics.MetricsSystem: Using default name DAGScheduler for source because spark.app.id is not set.
17/01/11 14:27:21 INFO client.AppClient$ClientEndpoint: Connecting to master spark://master:7077...
17/01/11 14:27:21 INFO cluster.SparkDeploySchedulerBackend: Connected to Spark cluster with app ID app-20170111142721-0004
17/01/11 14:27:21 INFO client.AppClient$ClientEndpoint: Executor added: app-20170111142721-0004/0 on worker-20170111005531-<IP_ADDRESS>-50421 (<IP_ADDRESS>:50421) with 2 cores
17/01/11 14:27:21 INFO cluster.SparkDeploySchedulerBackend: Granted executor ID app-20170111142721-0004/0 on hostPort <IP_ADDRESS>:50421 with 2 cores, 1024.0 MB RAM
17/01/11 14:27:21 INFO client.AppClient$ClientEndpoint: Executor added: app-20170111142721-0004/1 on worker-20170111005518-<IP_ADDRESS>-41380 (<IP_ADDRESS>:41380) with 1 cores
17/01/11 14:27:21 INFO cluster.SparkDeploySchedulerBackend: Granted executor ID app-20170111142721-0004/1 on hostPort <IP_ADDRESS>:41380 with 1 cores, 1024.0 MB RAM
17/01/11 14:27:21 INFO client.AppClient$ClientEndpoint: Executor added: app-20170111142721-0004/2 on worker-20170111135519-<IP_ADDRESS>-38652 (<IP_ADDRESS>:38652) with 1 cores
17/01/11 14:27:21 INFO cluster.SparkDeploySchedulerBackend: Granted executor ID app-20170111142721-0004/2 on hostPort <IP_ADDRESS>:38652 with 1 cores, 1024.0 MB RAM
17/01/11 14:27:21 INFO client.AppClient$ClientEndpoint: Executor updated: app-20170111142721-0004/0 is now LOADING
17/01/11 14:27:21 INFO client.AppClient$ClientEndpoint: Executor updated: app-20170111142721-0004/2 is now LOADING
17/01/11 14:27:21 INFO client.AppClient$ClientEndpoint: Executor updated: app-20170111142721-0004/1 is now LOADING
17/01/11 14:27:21 INFO client.AppClient$ClientEndpoint: Executor updated: app-20170111142721-0004/0 is now RUNNING
17/01/11 14:27:21 INFO client.AppClient$ClientEndpoint: Executor updated: app-20170111142721-0004/1 is now RUNNING
17/01/11 14:27:21 INFO client.AppClient$ClientEndpoint: Executor updated: app-20170111142721-0004/2 is now RUNNING
17/01/11 14:27:21 INFO util.Utils: Successfully started service 'org.apache.spark.network.netty.NettyBlockTransferService' on port 44782.
17/01/11 14:27:21 INFO netty.NettyBlockTransferService: Server created on 44782
17/01/11 14:27:21 INFO storage.BlockManagerMaster: Trying to register BlockManager
17/01/11 14:27:21 INFO storage.BlockManagerMasterEndpoint: Registering block manager <IP_ADDRESS>:44782 with 530.3 MB RAM, BlockManagerId(driver, <IP_ADDRESS>, 44782)
17/01/11 14:27:21 INFO storage.BlockManagerMaster: Registered BlockManager
17/01/11 14:27:22 INFO cluster.SparkDeploySchedulerBackend: Registered executor: AkkaRpcEndpointRef(Actor[akka.tcp://sparkExecutor@<IP_ADDRESS>:38869/user/Executor#-249835689]) with ID 1
17/01/11 14:27:22 INFO storage.BlockManagerMasterEndpoint: Registering block manager <IP_ADDRESS>:37499 with 530.3 MB RAM, BlockManagerId(1, <IP_ADDRESS>, 37499)
17/01/11 14:27:23 INFO cluster.SparkDeploySchedulerBackend: Registered executor: AkkaRpcEndpointRef(Actor[akka.tcp://sparkExecutor@<IP_ADDRESS>:38675/user/Executor#1450319069]) with ID 2
17/01/11 14:27:23 INFO storage.BlockManagerMasterEndpoint: Registering block manager <IP_ADDRESS>:38538 with 530.3 MB RAM, BlockManagerId(2, <IP_ADDRESS>, 38538)
17/01/11 14:27:23 INFO cluster.SparkDeploySchedulerBackend: Registered executor: AkkaRpcEndpointRef(Actor[akka.tcp://sparkExecutor@<IP_ADDRESS>:39565/user/Executor#-1870584609]) with ID 0
17/01/11 14:27:23 INFO cluster.SparkDeploySchedulerBackend: SchedulerBackend is ready for scheduling beginning after reached minRegisteredResourcesRatio: 1.0
17/01/11 14:27:23 INFO storage.BlockManagerMasterEndpoint: Registering block manager <IP_ADDRESS>:59533 with 530.3 MB RAM, BlockManagerId(0, <IP_ADDRESS>, 59533)
Exception in thread "main" java.io.FileNotFoundException: solver.prototxt (No such file or directory)
at java.io.FileInputStream.open(Native Method)
at java.io.FileInputStream.(FileInputStream.java:146)
at java.io.FileInputStream.(FileInputStream.java:101)
at java.io.FileReader.(FileReader.java:58)
at com.yahoo.ml.jcaffe.Utils.GetSolverParam(Utils.java:14)
at com.yahoo.ml.caffe.Config.protoFile_$eq(Config.scala:64)
at com.yahoo.ml.caffe.Config.(Config.scala:366)
at com.yahoo.ml.caffe.CaffeOnSpark$.main(CaffeOnSpark.scala:34)
at com.yahoo.ml.caffe.CaffeOnSpark.main(CaffeOnSpark.scala)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at org.apache.spark.deploy.SparkSubmit$.org$apache$spark$deploy$SparkSubmit$$runMain(SparkSubmit.scala:672)
at org.apache.spark.deploy.SparkSubmit$.doRunMain$1(SparkSubmit.scala:180)
at org.apache.spark.deploy.SparkSubmit$.submit(SparkSubmit.scala:205)
at org.apache.spark.deploy.SparkSubmit$.main(SparkSubmit.scala:120)
at org.apache.spark.deploy.SparkSubmit.main(SparkSubmit.scala)
17/01/11 14:27:23 INFO spark.SparkContext: Invoking stop() from shutdown hook
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/metrics/json,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/stages/stage/kill,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/api,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/static,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/executors/threadDump/json,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/executors/threadDump,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/executors/json,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/executors,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/environment/json,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/environment,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/storage/rdd/json,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/storage/rdd,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/storage/json,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/storage,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/stages/pool/json,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/stages/pool,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/stages/stage/json,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/stages/stage,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/stages/json,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/stages,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/jobs/job/json,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/jobs/job,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/jobs/json,null}
17/01/11 14:27:23 INFO handler.ContextHandler: stopped o.s.j.s.ServletContextHandler{/jobs,null}
17/01/11 14:27:23 INFO ui.SparkUI: Stopped Spark web UI at http://<IP_ADDRESS>:4040
17/01/11 14:27:23 INFO scheduler.DAGScheduler: Stopping DAGScheduler
17/01/11 14:27:23 INFO cluster.SparkDeploySchedulerBackend: Shutting down all executors
17/01/11 14:27:23 INFO cluster.SparkDeploySchedulerBackend: Asking each executor to shut down
17/01/11 14:27:23 INFO spark.MapOutputTrackerMasterEndpoint: MapOutputTrackerMasterEndpoint stopped!
17/01/11 14:27:23 INFO storage.MemoryStore: MemoryStore cleared
17/01/11 14:27:23 INFO storage.BlockManager: BlockManager stopped
17/01/11 14:27:23 INFO storage.BlockManagerMaster: BlockManagerMaster stopped
17/01/11 14:27:23 INFO scheduler.OutputCommitCoordinator$OutputCommitCoordinatorEndpoint: OutputCommitCoordinator stopped!
17/01/11 14:27:23 INFO spark.SparkContext: Successfully stopped SparkContext
17/01/11 14:27:23 INFO util.ShutdownHookManager: Shutdown hook called
17/01/11 14:27:23 INFO util.ShutdownHookManager: Deleting directory /home/master/Downloads/spark_sdk/spark-1.5.1/spark-df60b6b8-cbc9-45f4-a001-004923b2196e
Can someone help me?

Were you able to run the examples in Wiki? You command appeared to be correct, but Spark was complaining about not able to find solver.prototxt.

Sure, I have run the example successfully. But I don't know why this command is error, and I have solver.prototxt in the path in my command.

Glad to know the examples worked for you. I don't know why your command failed.

Thank you for your help. I change my solver.prototxt and train_val.prototxt  to the ${CAFFE_ON_SPARK}/data/, so the spark submit is:
spark-submit --master ${MASTER_URL}
--files ${CAFFE_ON_SPARK}/data/solver.prototxt,${CAFFE_ON_SPARK}/data/train_val.prototxt
--conf spark.cores.max=${TOTAL_CORES}
--conf spark.task.cpus=${CORES_PER_WORKER}
--conf spark.driver.extraLibraryPath="${LD_LIBRARY_PATH}"
--conf spark.executorEnv.LD_LIBRARY_PATH="${LD_LIBRARY_PATH}"
--class com.yahoo.ml.caffe.CaffeOnSpark
${CAFFE_ON_SPARK}/caffe-grid/target/caffe-grid-0.1-SNAPSHOT-jar-with-dependencies.jar
-train
-features accuracy,loss -label label
-conf solver.prototxt
-clusterSize ${SPARK_WORKER_INSTANCES}
-devices 1
-connection ethernet
-model file:${CAFFE_ON_SPARK}/myself_caffenet.model
-output file:${CAFFE_ON_SPARK}/myself_result
However, the there is also a error:
17/01/11 20:49:34 ERROR caffe.DataSource$: source_class must be defined for input data layer:Data
Exception in thread "main" java.lang.NullPointerException
at com.yahoo.ml.caffe.CaffeOnSpark.train(CaffeOnSpark.scala:103)
at com.yahoo.ml.caffe.CaffeOnSpark$.main(CaffeOnSpark.scala:40)
at com.yahoo.ml.caffe.CaffeOnSpark.main(CaffeOnSpark.scala)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at org.apache.spark.deploy.SparkSubmit$.org$apache$spark$deploy$SparkSubmit$$runMain(SparkSubmit.scala:672)
at org.apache.spark.deploy.SparkSubmit$.doRunMain$1(SparkSubmit.scala:180)
at org.apache.spark.deploy.SparkSubmit$.submit(SparkSubmit.scala:205)
at org.apache.spark.deploy.SparkSubmit$.main(SparkSubmit.scala:120)
at org.apache.spark.deploy.SparkSubmit.main(SparkSubmit.scala)
17/01/11 20:49:34 INFO spark.SparkContext: Invoking stop() from shutdown hook
It make me sad, I don't know why it throw the NullPointerException.....

Should I use the caffenet_train_net.prototxt in the ${CAFFE_ON_SPARK}/data, instead of the train_val.prototxt in the ${CAFFE_ON_SPARK}/caffe_public/models/bvlc_reference_caffenet ? And whether should I change the mean in the caffenet_train_net.prototxt?

You did not define source_class?
Depending on your source data format, you need to tell CaffeOnSpark about it, e.g.
https://github.com/yahoo/CaffeOnSpark/blob/master/data/lenet_cos_train_test.prototxt#L10-L12

Hello, I'm a newer for CaffeOnSpark. I have a same question , How to use myself model to detect image. you have solve it? Could you give me some example? Thanks!


Benchmark cannot run in Android12 phone

connect issue https://github.com/flutter/flutter/issues/100760

The latest templates have this but no one updated this one yet.  Thanks!

@

cc @blasten


Valence Security solution

Change(s):

Created Talon Solution including connector and workbook in the correct path

Reason for Change(s):

Valence and Sentinel Integration

Testing Completed:

Verified ValenceAlerts_CL data worked correctly and displayed ingested data

Hello @shlomimatichin, Can you please provide me access to your branch. I've some required changes for this Solution in my local.

Please add workbook metadata to this file https://github.com/Azure/Azure-Sentinel/blob/master/Workbooks/WorkbooksMetadata.json

Add a custom table with name "ValenceAlert_CL" at this location https://github.com/Azure/Azure-Sentinel/tree/master/.script/tests/KqlvalidationsTests/CustomTables. You can refer any of the other table for reference.

Solution description is short. Please add a few more lines to it.

v-prasadboke

done.

Hello @shlomimatichin Workbook preview images are needed in White and Black background. Please add one with White background.

Hello @shlomimatichin Workbook preview images are needed in White and Black background. Please add one with White background.

easy peasy! whats next?

Hello @shlomimatichin, Can I get a dummy account to test the connection of data connector.

Hello @shlomimatichin can we schedule a call, I have some doubts about the content. It would be helpful if we just resolve them on the call.
Please let me know your availability so that I can plan for the same, I work in IST time zone.
You can ping me directly on teams and mail me at<EMAIL_ADDRESS>


Filtering prefixes based on IP version

Can I specify a filter for IPv4 announcements?

@mbUSC Thanks for using bgpkit!
Currently, filtering by IP version is  not directly supported with the library. You can always check prefixes' IP version during iteration.
Could you provide some more details on your use cases? It should be not too complex to add a new filter for IP version.

Hi @digizeph,
I am using bgpkit for a small research project.
Having a prefix version filter that can be specified with add_filter(...) would be handy for ease of use
P.S.
Thank you for your work on bgpkit! I have found it to be a very useful tool

I've added a new ip_version filter with values to be either ipv4 or ipv6. Please see PR #155 for more details.
This new feature is included in the v0.10.2 release. Try it out and see if it resolves this issue. Cheers!

Wow that was so fast!
It works great, thanks a lot!!!


Adjusting Edit Phrase # of Rows & Columns 🎛

Description of Work
The number of rows and columns were not matching designs, so there were much less phrases for each page when editing phrases on iPhone and iPad. The number of rows on iPhone were made a bit larger than the Figma designs in order to better fit wrapped text.
Before

Orientation
iPhone 13
iPad Pro 11"

Portrait

Landscape

After

Orientation
iPhone 13
iPad Pro 11"

Portrait

Landscape

Test Notes 4/27

Verified in Portrait and landscape on iPhone 8 iOS 14 - the trash icon is not overly large.
Verified in Portrait and landscape on iPhone 12 iOS 15
Verified in Portrait and landscape on iPad 8th gen iOS 14
Verified paginating on all devices.
Good to go from Test.


⚠ Spryr has degraded performance

In b8cbc88, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 4818 ms

Resolved: Spryr performance has improved in 614f71f after 8 minutes.


updated armor

#5
PES2UG21CS241
SPHOORTHI YANDAMURI
changed the no of armors from 3 to 4  added Silver armour and updated the price too

@sphoorthiY Update on Progress?

This issue has been closed since there was already a valid PR. See #13


STORM-2851: Fix ConcurrentModificationException in doSeekRetriablePar…

…titions
See https://issues.apache.org/jira/browse/STORM-2851.
The bug is caused by the call to retriableTopicPartitions.remove(tp) in doSeekRetriablePartitions. Rather than just rewriting it to use an iterator I thought it was better to get rid of the loop. It was meant to ensure that we didn't seek the consumer on partitions that would be paused in the current call to nextTuple. I've moved this filtering into PollingInfo, and have made poll() return it. This also allows us to only get earliestRetriableOffsets once for the call to nextTuple, rather than once in poll and once in doSeek.
The added test is not related to the fix, it just adds more verification that the maxUncommittedOffsets limit is enforced.

@HeartSaVioR Thanks, I agree about the map copy. I've made the change you suggested, and also very slightly expanded a test to check that we only seek on partitions that are both pollable and retriable when retrying tuples.
I'll squash before merging if it looks good to you.

@hmcl Thanks, made most of the changes you mentioned. I thought PollInfo was a little vague, how about PollablePartitionInfo?
Also the tests are failing for some reason, give me a minute.

Updated with renames. I went with should* because I think the is* convention gets a little awkward here.

+1. Can you please squash such that I can merge it in. Thanks.

Squashed. I'll merge it in a few minutes, and put up a 1.x version soon.

Postponing merge a bit until Travis works again.

@srdo do you know when you will get the chance to create the PR for 1.x? Not to rush, but I was hoping we could start converging towards getting 1.2.0 out. Thanks.

@hmcl I'll put up a 1.x version later today. It would be good to get 1.2.0 out soon, but my impression is that we're still waiting for metrics v2, as well as a few other storm-kafka-client fixes (https://github.com/apache/storm/pull/2454 and https://issues.apache.org/jira/browse/STORM-2844).

Thanks for the reviews.


[189] Allow jobseekers to hide their profile from organisations

Changes in this PR:
See the "Hide profile from schools or trusts" section of the jobseeker profile.

 Candidates can hide their profile from organisations

@elliotcm this all looks good to me 👍

I found one interesting scenario here - if a user selects "yes" on the first question, and then cancels it results in profile displaying "Yes" next to hide from organisations with no organisation listed. The same happens when the last organisation is removed from the list. It's a minor thing, not even sure if it is an issue, but I'd imagine it should change to "No" if the list is empty.

I found one interesting scenario here - if a user selects "yes" on the first question, and then cancels it results in profile displaying "Yes" next to hide from organisations with no organisation listed. The same happens when the last organisation is removed from the list. It's a minor thing, not even sure if it is an issue, but I'd imagine it should change to "No" if the list is empty.

@BroiSatse Yeah the behaviour of that page when there's no orgs set (it's the same if you delete them all) is a bit weird but that's how the prototype works.

The only question I have is if you checked i18n phrases with someone from content. Other than that super solid! 🚀

@ddippolito All the copy came straight from the prototype.


Set cursor to pointer when hover over the logo

Set cursor to pointer when hover over the logo

claimed by me

I agree that this is a nice feature for the project presentation page at https://remsfal.de/ and will be implemented with issue https://github.com/remsfal/remsfal.github.io/issues/32. However, for reasons of user-friendliness, this behavior should not be implemented in the frontend application.


GH-44290: [Java][Flight] Add ActionType description getter

GitHub Issue: #44290

Would it be possible to add a test case too? Something simple?

@vibhatha, I added a simple test, let me know if that is sufficient.


support-logging -- update variable/interface naming for consistency

dbClient -> persistenceClient; interfaces.Logger -> interfaces.Persistence
From https://github.com/edgexfoundry/edgex-go/pull/2204#discussion_r355002100
Fixes: https://github.com/edgexfoundry/edgex-go/issues/2211
Dependency on #2071

Thank you for the additional work in this refactor.

Your suggestion had merit.  I played with naming in my original PR, waffled a bit, and ended up leaving the dbClient naming intact.  Your suggestion was timely and provided a good naming substitute to key off of.  Thank you!

Codecov Report

Merging #2212 into master will decrease coverage by 0.17%.
The diff coverage is 12.87%.

@@            Coverage Diff             @@
##           master    #2212      +/-   ##
==========================================
- Coverage   39.33%   39.15%   -0.18%
==========================================
  Files         175      175
  Lines       12626    12684      +58
==========================================
  Hits         4967     4967
- Misses       7392     7450      +58
  Partials      267      267

Impacted Files
Coverage Δ

cmd/support-logging/main.go
0% <0%> (ø)
:arrow_up:

internal/support/logging/init.go
0% <0%> (ø)
:arrow_up:

internal/support/logging/router.go
36.32% <15.85%> (-9.27%)
:arrow_down:

Continue to review full report at Codecov.

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

Rebased.

Rebased.


Trying to run pytest using Austin in VS code.

This may be similar to https://github.com/P403n1x87/austin-vscode/issues/21 but I'm not sure. I read the documentation at the VS code marketplace here
Here's my situation:
I have python project that I'm trying to profile using Austin within VS code. I'm using a MacBook (macOS Big Sur). Here's my tasks.json
{
    "version": "2.0.0",
    "tasks": [
        {
            "type": "austin",
            "label": "Profile tests",
            "args": [
                "python",
                "-m",
                "pytest",
            ],
            "problemMatcher": []
        }
    ]
}

I'm just trying to run pytest over the project. I'm also running this within a conda virtual environment. I get the following error:
> Executing task: Profile tests <

Starting Profiler in xxx/yyy/zzz/project-name.
Running 'austin' with args '-i 100 --pipe  python -m pytest'.
austin process exited with code 127
The terminal process failed to launch (exit code: 127).

Terminal will be reused by tasks, press any key to close it.

Any guidance/suggestion would be great appreciated.

I think this might be due to the fact that austin is running without sudo. On MacOS the use of sudo is unfortunately mandatory (unless you have SIP disabled). One possible solution is to allow austin to run without password, by adding a rule in the sudoers file, and then actually running sudo, e.g.
        {
            "type": "austin",
            "label": "Profile tests",
            "command": ["sudo"].
            "args": [
                "python",
                "-m",
                "pytest",
            ]
        },

@hrshdhgd thanks for trying that. I think this is now the same as #23, which I haven't been able to figure out yet 🙁 .

Not a problem , please let me know when you figure it out. In the meanwhile, would you know which Austin VSCode extension version and Austin version combination has proven to work in unison in the past?

same error.

Could you direct me to an example please?

You can run the same command that VS Code is trying to run from a terminal, e.g.
sudo austin -i 100 -o /path/to/austin/output.austin python -m pytest

and then open the `/path/to/austin/output.austin' from VS Code by clicking the OPEN button in the Flame Graph panel (or using the Cmd + Shift + A shortcut). This is equivalent to running the task.

Thank you so much @P403n1x87 ! That works as expected!


Investigate if the macOS toggle is possible in system tray app build with Tauri

We're building a Desktop System Tray app for Ockam, see https://github.com/build-trust/ockam/issues/5228
Here are two ways we could design the UI for an Ockam node running in the background.

With a toggle, this is the more modern macOS approach:

A status menu item (like Docker is running) combined with menu items to start and stop.

We should investigate if the toggle is possible with Tauri/Tao/Muda
https://github.com/tauri-apps/tauri
https://github.com/tauri-apps/tao/tree/muda
https://github.com/tauri-apps/muda
I imagine that the rendering is delegated to macOS so makes sense for it to be possible somehow.
Run
You can run the current version of the app as follows:
cd implementations/typescript
pnpm install
pnpm build

cd ../../implementations/rust/ockam/ockam_app
cargo tauri dev

Source Code
The code for this is here:

https://github.com/build-trust/ockam/blob/develop/implementations/rust/ockam/ockam_app
https://github.com/build-trust/ockam/blob/develop/implementations/typescript/ockam/ockam_app

We love helping new contributors!
If you have questions or need help as you work on your first Ockam contribution, please leave a comment on this discussion or join us on discord. If you're looking for other issues to contribute to, checkout labels -  https://github.com/build-trust/ockam/labels/good first issue or https://github.com/build-trust/ockam/labels/help wanted

on it

@tusharxoxoxo Thanks! please let us know if you have any question!

@nazmulidris should we still keep this issue open as such or is it included in another piece of work?

I think we can close it, it's included in @davide-baldo's work with https://github.com/build-trust/ockam/issues/5774


update node_exporter dashboard

Add new panel on node_exporter dashboard,to display the buddyinfo. and add  Mlocked,Slab metrics in node_exporter dashboard.

Could you add a screenshot of what the panel would look like?


Avoid erroneous diagnostics in speed measuring

Fixes #20291

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Ping?

This looks good to me (mild preference for != 1 rather than == 0, but PR author has ultimate choice in these things) - I would love to see this approved and merged

Removing approval: otc review pending as @paulidale has approved

I presume tests aren't needed for this fix/

I presume tests aren't needed for this fix?

The test policy is here:
https://www.openssl.org/policies/technical/testing.html
In this case I'd say this exemption applies:

A test may be omitted where writing the test would result in disproportionately more effort than writing the code being tested.

In general we of course prefer tests for all PRs, but there are no existing tests for the "speed" app at all - so writing a test for this small fix would be very difficult.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged.
Should this have gone to 3.1 and 3.0 too?

I think this code was in master only (and definitely not in 3.0).

I've confirmed that the change in #15905, which introduced the problem, is not in 3.1


When should we apply hidden_z?

I notice that hidden_z is applied in almost every module in every layer, and I'm curious about whether it will result in issue like gradient vanishing and exploding? And, will it have a large influence on the magnitude of the last hidden state, as the same scale is repeatedly multiplied?

Given how hidden_z is modeled (with hard concrete distributions), it's essentially learning a discrete binary value of 1 (retain the dimension) or 0 (prune the dimension). As the values strictly fall in side [0, 1], I don't think it will cause gradient explosion issues. For the same reason, it should not have an influence on the magnitude of the last hidden state. Let me know if it is clear!

As the values strictly fall in side [0, 1]

Yes, the hidden state won't be too large, but will it be too small? Just plan to check it on the model without re-training.


Using Angular Material without @angular/http depandency

Bug, feature request, or proposal:
Bug
What is the expected behavior?
@angular/material must work without @angular/http depandency in package.json.
What is the current behavior?
Without @angular/http depandency I have an error: @angular/material/typings/icon/icon-registry.d.ts (10,22): Cannot find module '@angular/http': https://cl.ly/2a0a1e3q010v .
What are the steps to reproduce?
Run npn i and then ng build for project with package.json without @angular/http depandency.
What is the use-case or motivation for changing an existing behavior?
Convenience of work with Angular Material library.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular - 4.3.0
Material - 2.0.0-beta.8
OS - Windows 10
Is there anything else we should know?
In my project I use @angular/common/http module instead of @angular/http, so currently @angular/http module is necessary only for @angular/material and I want to get rid of @angular/http depandency in my package.json.

https://github.com/angular/material2/issues/5814

Duplicate of #5814


TypeScript missing TextArea$ChangeEvent

VS Code cannot find the type TextArea$ChangeEvent of sap/m/TextArea.
There is only a ChangeEvent type for sap/ui/webc/main/TextArea.
Used version: 1.120

sap.m.TextArea inherits the change event from class sap.m.InputBase. Therefore, the event type is named InputBase$ChangeEvent and exported from sap/m/InputBase.


Table row actions not working on custom livewire page

Package
filament/filament
Package Version
v3.0.68
Laravel Version
v10.26.2
Livewire Version
v3.0.5
PHP Version
8.1.19
Problem description
I wanted to add a row action to a table element on a custom livewire page. So I did this according to the documentation. But when I then clicked the row action in the table itself only a short loading icon appears and then nothing.
I did test with a ->url(route('someroute'))->openUrlInNewTab() that the action is actually being triggered, which it is.
After that I tried a bulk action with the exact same code which then worked perfectly. So I assume this is a bug in Filament.
Said action.
->actions([
                 \Filament\Tables\Actions\Action::make('DissociateTeamMember')
                    ->requiresConfirmation()
                    ->hidden(!$this->isTeamAdmin)
                    ->color('danger')
                    ->icon('heroicon-o-x-mark')
                    ->label('Teamlid verwijderen')
                    ->action(function () {
                        dd('test');
                    }),

            ])

Expected behavior
That the action modal triggered and the code in the ->action(function() { //code  }) runs.
Steps to reproduce

Create a livewire page
add filament table to page
add action to table ( Example action above )

Aditional info which might be useful,

I have multiple other standalone actions in the same page,
Yes I did include: x-filament-actions::modals/
the query for the table is:

->query($this->team->members()->getQuery())

Reproduction repository
https://github.com/biebthesecond/filamentActionBug
Relevant log output
No response

Have you tried using the recordAction instead?
public static function table(Table $table): Table
    {
        return $table
            ->columns(...)
            ->filters(...)
            ->actions(...)
            ->bulkActions(...)
            ->recordAction(Tables\Actions\ViewAction::class)
            ->recordUrl(null);
    }

I have same issue, pls kill the bug~

same issue

I think I'm having the same issue, but not a table action. Just an anonymous action on my livewire component. if I do mountUsing I can dd a value, but nothing happens in the action method itself.

I got the same issue with filament 3.0.87 version.

class ListTools extends Component implements HasTable, HasForms, HasActions
{
    use InteractsWithTable;
    use InteractsWithForms;
    use InteractsWithActions;
    public function table(Table $table): Table {
        return $table
            ->query(Tool::query())
            ->columns([
                Tables\Columns\TextColumn::make('name')
                    ->searchable(),
                Tables\Columns\TextColumn::make('product.name')
                    ->sortable(),
.....
.....
->actions([
                Tables\Actions\ViewAction::make()
            ])

As a workaround, I use recordUrl() from the $table instance instead of actions(), and sets it to redirect to another custom Form with all disabled fields. This fulfills my requirement to show the row info, though it is slightly inconvenient to create new Form, as it is not necessary if ViewAction does work in actions().
 public function table(Table $table): Table {

        return $table
            ->query(Tool::query())
            ->columns([
....
...
-> recordUrl(fn (Tool $tool): string => route('tool.view', ['id' => $tool -> id]),)

In new Livewire component for the read-only form:
class ViewTool extends Component implements HasForms
{
    use InteractsWithForms;

    public Tool $tool;
    public ?array $data = [];

    public function mount($id) {
        $this -> tool = Tool::findOrFail($id);
        $this -> form -> fill($this -> tool -> toArray());
    }

    public function form(Form $form): Form
    {
        return $form
            ->model($this -> tool)
            ->schema([
                Forms\Components\Fieldset::make('essentials')
                    -> label('Essentials')
                    -> schema([
                        Forms\Components\TextInput::make('name')
                            -> readOnly()
                            -> disabled(),
......
......

Routing as full page Livewire component:
Route::get('/tools/{id}', \App\Livewire\ViewTool::class) -> name('tool.view');

There are many comments in this thread, and I highly doubt that any of them are related to the original issue.
The original issue is that the author is trying to use a relationship query without defining it properly:
Instead of:
->query(
                $this->team?->members()->getQuery() ?? Team::query()
            )

Use:
->relationship(fn () => $this->team->members())

And it will work. Docs: https://filamentphp.com/docs/3.x/tables/adding-a-table-to-a-livewire-component#building-a-table-for-an-eloquent-relationship

Hello,
In case someone hits this issue, it seems the cause is when you do something like this in your table query
return  Costs::select([
'costs.id as id_costs',
for some reason that id_costs confuses things, and doesn't works even if you set getTableRecordKey to it


Update API call delete

not only user has to be deleted but also all its matches and games
Bug appears if trying to delete a user who has played a card

Best solution would be to use the correct Hibernate annotations, so that the mapped relation tuples also get deleted without violating referential integrity.


Fix typos

PR description
Fix typos in comments.

@Thabokani thanks for your contribution - we can merge this if you add signoff to your commits - follow the instructions here - to make the DCO check pass

@Thabokani thanks for your contribution - we can merge this if you add signoff to your commits - follow the instructions here - to make the DCO check pass

@macfarla thanks for your review. I have added signoff.

ok one more thing to fix - I don't have permissions to update your branch otherwise I'd do it for you - but the plugin-api hash  needs updating - instructions here - that's why the unit tests are failing https://app.circleci.com/pipelines/github/hyperledger/besu/26758/workflows/0246a9b0-4020-414d-a270-3eb6027d82ba/jobs/176846

Thanks for persevering @Thabokani - should merge shortly!


Added How to increase the columns available for calendar view formatting

Category

[ ] Content fix
[x] New article

I thought it would be clearer to put the product name in the title of the article and in the header image, so I have modified it. Sorry if this has already been checked😥

Thank you so much @LuiseFreese 🙏✨


flask-sqlachemy has a dependency to sqlalchemy

sqlalchemy v2.0.0 does not match 2.5.1 of flask-sqlachemy
fixes #1657

unfortunately there are more changes in setuptools etc. I am not able to do a build on CF currently

this discribes my current problem, I try to fixate that lib too
https://github.com/pypa/setuptools/issues/3772

I currently fixate setuptools too, and will add this here too, until that is main stream solved
https://github.com/conda/conda-build/issues/4733

We verified on CF that with that patch it is builded. this will need a 7.0.8 soon.
https://github.com/conda-forge/mss-feedstock/pull/141/files


Increase unit tests code coverage installable item

Increase unit tests code coverage installable item

Current coverage is 85.26% (diff: 100%)

Merging #475 into master will increase coverage by 1.76%

@@             master       #475   diff @@
==========================================
  Files            18         18
  Lines          2079       2178    +99
  Methods         374        374
  Messages          0          0
  Branches        181        181
==========================================
+ Hits           1736       1857   +121
+ Misses          343        321    -22
  Partials          0          0

Powered by Codecov. Last update 774f9f4...ca90cb8


optimize: console display the version number

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did

Ⅱ. Does this pull request fix one issue?

fixes #6240
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (7227cc1) 51.95% compared to head (f3188d6) 51.91%.

:exclamation: Current head f3188d6 differs from pull request most recent head 6b6d488. Consider uploading reports for the commit 6b6d488 to get more accurate results

Additional details and impacted files

@@             Coverage Diff              @@
##                2.x    #6313      +/-   ##
============================================
- Coverage     51.95%   51.91%   -0.05%
+ Complexity     5174     5167       -7
============================================
  Files           918      918
  Lines         32039    32039
  Branches       3866     3866
============================================
- Hits          16647    16633      -14
- Misses        13767    13783      +16
+ Partials       1625     1623       -2

see 11 files with indirect coverage changes


How to pass variables to a module in terratest

How do we pass values to a  module , I remember having terratest work without having to define a variables.tf file inside root-module. But now it fails if i don't define a variables.tf file in root-directory.
Am I missing something, or its mandatory that we should pass the variables.tf in root-directory?
My directory structure looks liks
tf_acm_module github repo

main.tf
variables.tf
data.tf
output.tf
test folder
- terraform directory (root-directory)
- backend.tf
- providers.tf
- main.tf ,
- output.tf
- terraform_acm_test.go

package test

import(
    "testing"
    "os"
    "github.com/gruntwork-io/terratest/modules/aws"
    "github.com/gruntwork-io/terratest/modules/logger"
    "github.com/gruntwork-io/terratest/modules/terraform"
    "github.com/stretchr/testify/assert"
)

func TestTerraformAwsNetworkExample(t *testing.T) {
    t.Parallel()

    logger.Log(t, "This will show up in stdout immediately")
    os.Setenv("TERRATEST_IAM_ROLE", "arn:aws:iam::xxxxx:role/role")
    awsRegion := "us-east-1"
    PrimaryDomain := "module.foo.com"
    SubAltDomNames := []string{"*.foo.com","bar.dev"}
    ValidMethod := "DNS"
    Tag := map[string]string{"owner":"xxx"}
    terraformOptions := &terraform.Options{
        TerraformDir: "terraform/",
        Vars:  map[string]interface{} {
            "primary_domain": PrimaryDomain,
            "subjective_alternative_names": SubAltDomNames,
            "validation_method": ValidMethod,
            "tags": Tag,
        },
        EnvVars: map[string]string{
            "AWS_DEFAULT_REGION": awsRegion,
        },
    }
    defer terraform.Destroy(t, terraformOptions)
    terraform.InitAndApply(t, terraformOptions)
    acmARN := terraform.Output(t, terraformOptions, "acm_arn")
    ttACMARN := aws.GetAcmCertificateArn(t,awsRegion,PrimaryDomain)
    assert.Equal(t,acmARN,ttACMARN)
}

Error:
TestTerraformAwsNetworkExample 2020-06-24T23:31:19-04:00 logger.go:66: Error: Value for undeclared variable
TestTerraformAwsNetworkExample 2020-06-24T23:31:19-04:00 logger.go:66:
TestTerraformAwsNetworkExample 2020-06-24T23:31:19-04:00 logger.go:66: A variable named "primary_domain" was assigned on the command line, but the
TestTerraformAwsNetworkExample 2020-06-24T23:31:19-04:00 logger.go:66: root module does not declare a variable of that name. To use this value, add a
TestTerraformAwsNetworkExample 2020-06-24T23:31:19-04:00 logger.go:66: "variable" block to the configuration.
TestTerraformAwsNetworkExample 2020-06-24T23:31:19-04:00 logger.go:66:

TestTerraformAwsNetworkExample 2020-06-24T23:31:19-04:00 logger.go:66:

Any variables you put in Vars within terraform.Options get passed to your Terraform module via -var arguments. As of Terraform 0.12, if you pass -var foo=bar to a module, then it must have a variable "foo" declared within it.


Create aquatica.thedev.me.json

Added NS Records to allow easy addition of new sub-sub-domains on the aquatica.thedev.me sub domain for future and current projects :)

Turned to a draft to change to namecheap freedns

Update: the domain now uses Namecheap FreeDNS Nameservers


PubSub.close throws error

This bug report is related to https://github.com/googleapis/nodejs-pubsub/pull/916.
Environment details

OS: Linux
Node.js version: v10.16.3
npm version:  6.9.0
@google-cloud/pubsub version: 1.7.0

Steps to reproduce
Here is a test written to test this newly introduced close method.
import { PubSub } from '@google-cloud/pubsub';

it('test pubsub close', async (): Promise<void> => {
  expect.assertions(1);
  const topicName = 'test-topic';

  const pubSub = new PubSub();

  const [topic] = await pubSub.topic(topicName).get({ autoCreate: true });

  await topic.publish(Buffer.from('test-message'));

  expect(await pubSub.topic(topicName).exists()).toStrictEqual([true]);

  await pubSub.close();
});

When running this simple test in Jest, I get the following error.
 FAIL  src/test.spec.ts (7.801s)
  ✕ test pubsub close (106ms)

  ● test pubsub close

    TypeError: gaxClient.close is not a function

      13 |   expect(await pubSub.topic(topicName).exists()).toStrictEqual([true]);
      14 |
    > 15 |   await pubSub.close();
         |                ^
      16 | });
      17 |

      at PubSub.closeAllClients_ (node_modules/@google-cloud/pubsub/src/pubsub.ts:983:31)
      at PubSub.close (node_modules/@google-cloud/pubsub/src/pubsub.ts:302:12)
      at PromiseCtor (node_modules/@google-cloud/promisify/build/src/index.js:69:28)
      at PubSub.wrapper (node_modules/@google-cloud/promisify/build/src/index.js:54:16)
      at Object.it (src/test.spec.ts:15:16)

// @feywind

@mad-it @stephenplusplus Thanks! I will take a look in a moment.

Well, this will teach me to not write a system test for a feature. :S The Pub/Sub client API objects are hand-written and not using a standard generated gax client, and this gets cast to a standard gax interface, which is really confusing. I'll work on getting this piped through tomorrow and add a system test based on the case above.
Thanks again for the report!

I'm not sure I like the issue auto-close. Anyway, this PR should fix the problem, can you try again?
https://github.com/googleapis/nodejs-pubsub/pull/941

@feywind Were you able to run it yourself? I just tried, made sure I have the latest version, but I still see the same error.

@feywind it looks like this issue has been resolved for the very specific test case provided, but the same issue occurs when a subscriber is created.
By adding await topic.subscription('subscriber-1').get({ autoCreate: true }); to the test case above we get the exact same error.
So the complete test case becomes:
import { PubSub } from '@google-cloud/pubsub';

it('test pubsub close', async (): Promise<void> => {
  expect.assertions(1);

  const topicName = 'test-topic';

  const pubSub = new PubSub();

  const [topic] = await pubSub.topic(topicName).get({ autoCreate: true });

  await topic.subscription('subscriber-1').get({ autoCreate: true });

  await topic.publish(Buffer.from('test-message'));

  expect(await pubSub.topic(topicName).exists()).toStrictEqual([true]);

  await pubSub.close();
});

Indeed. It looks like PublisherClient has the close method, but SubscriberClient does not.

Hmm, that's interesting - the subscriber version was already there before my work on the publisher, so I didn't spend much time with it. It's possible that there was some code rot in there from the underlying gapic generation, too. I'll take a look at that.

@merlinnot @mad-it
1.7.2 adds the close() method to the SubscriberClient also. Let me know if that works for you? I think that should be all of them, since there are only stubs for publisher and subscriber. There is another system-test that will validate that this one works going forward as well.
We're in the process of releasing 2.0.0 that will bring a lot of dependencies up to date, and also introduce TypeScript generated service stubs that have a close() method built in. I wanted to backport this to the existing 1.x branch though... it doesn't seem like a good idea to require a major version update for a bug fix. :)

Thanks for fixing it so quickly. It doesn't crash now, but it also doesn't seem to close gRPC connections. I'll try to create a minimal reproduction and report it in a separate issue, this one can be considered closed.

My bad, it works perfectly fine. Thank you! 🎉

Thanks for the quick fix!


Issues with react-bootstrap

I've created an example:
http://esnextb.in/?gist=61d8891f6bb8e6a5009b
I'm still having issues I was wondering if you could have a look. If you view the console, the action seems to always fire with true.

I'll close for now as the above seems to be a bug in exnextb.in , they're still not working in my project but seem to be in a text project ive set up so will investigate further..

Hey @sattaman, try doing this:
const store = compose(applyMiddleware(thunk, createLogger()))(createStore)(combineReducers({
-  fields: createModelReducer('myModel')
+  myModel: createModelReducer('myModel')
}));

The model name needs to match the actual key for the slice of state that the model represents.
Proof that that might fix your problem:

@sattaman Does this solve your issue?

Hi, yes that explains why I was having issues.
To clarify, at present I had named my modal reducers names like:
sms: createModelReducer("message.compose.sms")
Am I right in thinking it's only going to work if I instead use names such as
`composeSms: createModelReducer("composeSms")
Does the hierarchy of where the reducer is affect anything? For instance, if the reducer where my initial model was 'message.compose'?
`

I may have just spotted the answer in the other issues that was created, namely the nested state example.
http://davidkpiano.github.io/react-redux-form/#/recipe/model-in-nested-state?_k=t4nbbq

@sattaman Is the example link you gave here, a working example? If so, why do you use controls.text instead of controls.checkbox if you are using an Input of type checkbox? The docs for react-redux-form say that controls has a list of mappings, text is for text-based inputs and checkbox is for checkbox-based inputs. Does controls.text just happen to work for this instance or am I not understanding the docs correctly?

@nickzarate It seemed to work , I don't think you can define more than one createFieldClass for the same input type, and putting them as text seemed to work. I initially had issues with the state path not matching the model.


New connector ExtractTopicFromValueSchema

We would need to have same kind of functionality as the ExtractTopic have, but would want to take the schema name value and use that as a Topic name. I have already created PR:
https://github.com/aiven/transforms-for-apache-kafka-connect/pull/93
where I have implementation for review. Could you please check that out.

Should be addressed by this https://github.com/aiven/transforms-for-apache-kafka-connect/pull/93


Wrong indentation level of the quote block in the Dockerfile reference

File: engine/reference/builder.md
Refer the following screenshot, the quote block should be on the same level of the list node's content.

Thanks for sharing this issue. That particular content is maintained in an upstream repo. If the issue is still applicable, please help create an issue or a PR in the repo at https://github.com/docker/cli/tree/master/docs/reference


AzureMonitorWorkspacesOperations Class does not support to publicNetworkAccess configuration.

Is your feature request related to a problem? Please describe.
I am using "azure-mgmt-monitor==6.0.2" to create azure monitor workspace.
the following block would create it:
def create_azure_monitor_workspace(rdata: PMap) -> None:
        """
        Creates/updates Azure Monitor Workspace
        :Param data: pmap or v with the info
        """
        subscription_id = rdata['subscription_id']
        resource_group_name = rdata['resource_group_name']
        # parameters = rdata['parameters']
        #For clarity I declare below the value of parameters
        parameters = {
                'location': "westeurope",
                'tags': {
                    'env': 'test1'
                },
                'properties': {
                    "publicNetworkAccess": "Disabled"
                }
        }
        name = rdata['name']
        monitor_client = MonitorManagementClient(credential, subscription_id)
        amw = monitor_client.azure_monitor_workspaces.create(resource_group_name, name, parameters)

It creates the AMW without problem EXCEPT it ignores to leave the Public Network Access disabled. The parameter block is a type AzureMonitorWorkspaceResource or IO that accepts as parameters location and tags but not other specs such as the publicnetworkaccess.
As result using the Azure SDK it is NOT possible to set the desired public network access as it should.
I am open to workarounds, if anybody knows any!
Describe the solution you'd like
The block parametersas mentioned  above or similar should be able to set the publicNetworkaccess of the created AMW.
Describe alternatives you've considered
There is NONE if you pursue full automation.
Additional context
Add any other context or screenshots about the feature request here.

Thanks for the feedback, we’ll investigate asap.

Python sdk is generated from the swagger definition. Such feature is already defined in the stable api version and may be applied in python package in the next releases.

Hey @ChenxiJiang333 ,
I am using ** "azure-mgmt-monitor==6.0.2"** that was regenerated after stable api version.
As you can see I am passing a dict to set it up:
parameters = {
'location': "westeurope",
'tags': {
'env': 'test1'
},
'properties': {
"publicNetworkAccess": "Disabled"
}
}
which is compliant with the REST API spec. I believe that property is readonly or similar because the execution does not fail NOR updates the publicNetworkAccess property.

Hi, @romangallego.
I have tried the code and the execution didn't return any failure as well, but it doesn't mean that the parameters passed were all suitable.
Here's my log. You can see that invaild parameters would be abandoned in the serialization so the publicNetworkAccess property would not be included in the request body and was not sent at the beginning.
2023-09-05 13:04:37,445 - azure.core.pipeline.policies._universal - DEBUG - Request URL: 'https://management.azure.com/xxxxx'
Request method: 'PUT'
Request headers:
    'Content-Type': 'application/json'
    'Content-Length': '52'
    'Accept': 'application/json'
    'x-ms-client-request-id': 'b4b0fa37-4ba9-11ee-868e-a8b13b79b46f'
    'User-Agent': 'azsdk-python-azure-mgmt-monitor/6.0.2 Python/3.10.11 (Windows-10-10.0.22621-SP0)'
    'Authorization': xxxxx'
Request body:
{"tags": {"env": "test1"}, "location": "westeurope"}

Also, the source code of the latest package can be found here. It shows that when it comes to the operations about azure_monitor_workspaces, the current api used is still in preview version.

Thank you @ChenxiJiang333! I do appreciate your help!
I will open the issue to the azure-rest-api-specs


getTile for different projections

Hi guys,
I'm trying to display tiles from some geopackage, which has srs EPSG:31467 and it doesn't work.
I use your code example from the project description to get tiles. It works perfectly for rivers.gpkg from the example and also for other geopackages, which have srs EPSG:4326 or EPSG:3857.
What should I do to get another projections work? Could you please help me?

I believe that you are going to want to use this method: GeoPackageTileRetriever.getTileWithWgs84BoundsInProjection  That takes a latitude longitude bounding box, the zoom level you want, and a target projection ('EPSG:31467').
If you could provide me with the GeoPackage you are trying to use and the sample of code you are using I can take a look.


Update weaver-test-extra to 0.4.11

Updates io.github.dimitarg:weaver-test-extra from 0.3.9 to 0.4.11.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.github.dimitarg", artifactId = "weaver-test-extra" } ]

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

Superseded by #69.


Python 3

Any plans for python 3 support?

The dev fork works fine in Python 3:
https://github.com/threebytesfull/pcbmode/tree/develop

@JarrettR unable to get hello-solder working on fork, any suggestions for other example boards?

Yeah, that one is for an older version. Thelady  is a complicated one that works.
I like Binco, a lot less stuff to strip out as an example project.

I'm starting to work on the next version of PCBmodE at branch cinco-master. We'll drop support for Python2 and require Python 3.7+.

Please just use the highest python that inkscape is using. not sure if its going to be 3.7 or 3.5 ???

It's not explicitly stated... if it's using what's available on the system it'd launch which Python3 you're already using for PCBmodE. Right?
https://wiki.inkscape.org/wiki/index.php/Updating_your_Extension_for_1.0

They bundle a version with the install so they can control the version used by all of the extensions and includes TlInter as a UI. I think they have not decided for sure yet which version they will bundle with 1.0 but I'd have to check the dev email list.
in 0.48 it was a really old version of python. in 0.92 they finally updated it and for 1.0 they will log to Python 3.

Ah. OK. It would be great to know.

https://wiki.inkscape.org/wiki/index.php/Release_notes/1.0#For_extension_writers
3.7 I surmise


Scaffolding problem

Scaffold pages will not set namespace automatically.
Like scaffold students in Pages.Students folder, the namespace should be: namespace ContosoUniversity.Pages.Students
But in fact, it will be: namespace ContosoUniversity
Is this operate correctly? Or it's a new change to this function?

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

ID: d0026f69-7e5d-0d3e-5586-16ecc0656ff0
Version Independent ID: 3cb264da-2ff1-1256-572f-aa4ebca6e255
Content: Razor Pages with EF Core in ASP.NET Core - Read Related Data - 6 of 8
Content Source: aspnetcore/data/ef-rp/read-related-data.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Same here, tried several folders but everytime I end up with the wrong namespace. The Students and the Courses scalffolding process set the namespace to "ContosoUniversity".
I understand you must run the scaffolding over the "Pages\Courses" and "Pages\Students".
Any help would be great, thank you !!!

Had the exact same problem, seems it doesn't work when doing it the Visual Studio 2019 way.
In the package manager console I ran the following commands (not sure if the first 2 are necessary). I also navigated the PMC to the directory my *.csproj file is in, not sure that was require either.
dotnet tool install --global dotnet-aspnet-codegenerator
dotnet add package Microsoft.VisualStudio.Web.CodeGeneration.Design
From the VS Code instructions in this turtorial:
dotnet-aspnet-codegenerator razorpage -m Course -dc SchoolContext -udl -outDir Pages\Courses --referenceScriptLibraries
I hope this helps some future people when they get stuck here, and come looking for the solution in the feedback section.

I had the same problem.  I did not run scaffolding on the wrong folder either or the new pages would end up in the wrong folder.  What happens is for example on the original Pages/Students/Index page the page model is "Index".  Then in courses the scaffolding also names it index.  Now you have to ContosoUniverstiy.Index classes and the compiler complains.  This happens for all the scaffolded pages: Delete, Details, etc.  I just went through all the pages under the Courses folder and renamed the namespace from "ContosoUniversity.IndexModel" to "ContosoUniversity.Courses.IndexModel" for example on the index page.

I'm actually surprised VS 2019 and Core 3 does this mistake for Razor Pages.  Why not add the sub folder into the namespace as has been the convention for years and years.

Thank you for contacting us. Due to a lack of activity on this discussion issue we're closing it in an effort to keep our backlog manageable. If you believe there is a concern which hasn't been addressed, please file a new issue.


Unit tests for utils and small improvements

Added test/postagga/utils_test.cljc, which contains unit tests for the methods of utils
Fixed an edge-case for arg-max (on empty collection it threw an exception, now it returns nil)
Corrected some typos in the doc-strings
Added forward declaration of fast-forward-all-ors and fast-forward to make the code compile.

👍 Thank you SO MUCH !


Dependency issue when building with 6.0.3

Describe the bug
Our pipeline is currently failing to build the Android app due to what appears to be a broken dependency:
`> Could not resolve com.github.bumptech.glide:glide:4.7.1.
Required by:
project :app > project :react-native-fast-image
project :app > project :react-native-fast-image > com.github.bumptech.glide:okhttp3-integration:4.7.1

Could not resolve com.github.bumptech.glide:glide:4.7.1.
Could not get resource 'https://jitpack.io/com/github/bumptech/glide/glide/4.7.1/glide-4.7.1.jar'.
Could not HEAD 'https://jitpack.io/com/github/bumptech/glide/glide/4.7.1/glide-4.7.1.jar'. Received status code 522 from server: Origin Connection Time-out
Could not resolve com.github.bumptech.glide:annotations:4.7.1.
Required by:
project :app > project :react-native-fast-image
Could not resolve com.github.bumptech.glide:annotations:4.7.1.
Could not get resource 'https://jitpack.io/com/github/bumptech/glide/annotations/4.7.1/annotations-4.7.1.pom'.
Could not GET 'https://jitpack.io/com/github/bumptech/glide/annotations/4.7.1/annotations-4.7.1.pom'. Received status code 522 from server: Origin Connection Time-out`

Expected behavior
App should build successfully
Dependency versions

React Native version: 0.59.5
React version: 16.8.6
React Native Fast Image version: 6.0.3

Note: if these are not the latest versions of each I recommend updating as extra effort will not be taken to be backwards compatible, and updating might resolving your issue.
We are in the processing of undergoing the new react-native upgrade but we're not quite there yet. It's unclear if that would help this issue.

We managed to resolve this by completing the react native upgrade which allowed us to use a newer version of RNFI, but my guess would be that the problem remains. Thanks!

@jonassvalin to what version did you upgrade your react-native?
facing the same issue running RN-"0.59.10" and RNFI-"^6.1.1" and RNFI-"^7.0.2"

@jonassvalin  Please update glideVersion to  "4.10.0"

Also had this, navigated to https://www.jitpack.io/com/github/bumptech/glide/glide/4.7.1/glide-4.7.1.pom and got a failed build warning.
Incredibly, this solved it for me:
https://gitmemory.com/issue/jitpack/jitpack.io/3970/531199447


setCookie always sets an encoded value in authentication scripts

Even if we disable "sign" and "encrypt" as below
setCookie(context.response, "name1", "value1", {"expires" : "session",
                                            "path" : "/",
                                            "httpOnly" : false,
                                            "hostOnly" : false,
                                            "secure" : false,
                                            "encrypt" : false,
                                            "sign" : false});

The cookie sent to the browser has the following format
base64encode({"signature": null,"value": "value1"})
Suggestions

Update the doc [1] to describe the cookie format to avoid confusion when observing the cookies from a browser
Consider about setting the plain text or encrypted value of the cookie to the browser when "sign" is false.

[1] https://docs.wso2.com/display/IS570/Adaptive+Authentication+JS+API+Reference#AdaptiveAuthenticationJSAPIReference-setCookie(response,name,value,properties)

I have created https://github.com/wso2/product-is/issues/5758 to track the 2nd suggestion

In the documentation please add the following.

The cookie is only interpreted by Identity server.
Please do not make any assumption about the cookie value or format.
Please do not access the cookie with front end Javascript.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Stop logging expected GraphQL errors in test output

The current test set up intercepts GraphQL errors to print them as output. This is useful, as they warn about missing mock data and other errors. However, these messages are mixed up in 'expected' failures that we've set up in the tests themselves.
This PR switches to separate classes for our fake errors so we can exclude them from the console output.

 2.0: FE test/linting improvements from James


Create a Webpack loader

It would be great if RT.js was bundled up as a Webpack loader so that a user can integrate RT.js seamlessly into their project. This would require creating a new NPM package named rtjs-loader (or rtjs-webpack-loader) that could be added to a project which would pass source code through the transpiler before being bundled by Webpack.
I'd be happy to get a prototype / reference implementation set up. Please let me know if you have any thoughts on this!

I would love to see RT.js being integrated into the normal  Javascript-Developer workflow. However, I'm not on par with the current state of the art in Javascript Web Development as I come from the OS/Real-Time OS side of things. Therefore, I'm no super familiar webpack.


Code generated using new version of OpenAPI specification

Description
Code generated using new version of OpenAPI specification
Issues Resolved
Closes #636
Closes #474
Before reviewing this PR, please review PR #721. PR #721 contains only the changes made for the generator, without the generated code.
The failing tests have been addressed in https://github.com/opensearch-project/opensearch-api-specification/pull/246.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

One of the big changes here is that common query_params are repeated all over the place. I suppose we don't have a better idea to represent that? Maybe there can be a shared annotation like @common_query_params?


fix: dashboard get by id or slug access filter

SUMMARY
This PR re-introduces the defined dashboard access filter into Dashboard DAO get item method.
Currently viewing a list of dashboards and requesting a specific dashboard by its slug or id will allow for different access levels.
I believe that access filters at the DAO level need to be applied on all methods (that fetch or mutate a not yet fetched item) and offer a coherent centralised first layer of security checks
This PR also:

Changes HTTP response status codes for filter states, since a dashboard that a user does not have permission to access is automatically filtered at the DAO level, filter states for these dashboards will return 404 instead of 403 (Improved security)

Fixes a bug when using DASHBOARD_RBAC, currently on master, a Gamma user (for example) that has access to a dashboard using data access permissions can list the dashboard but can't render the dashboard itself (/api/v1/dashboard/<id> and /api/v1/dashboard/<id>/charts/ returns HTTP 403). This happens because only raise_for_dashboard_access is called and not data access permission is previously checked.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up


Task 393990 panicked

"Meilisearch ApiException: Http Status: 500 - Message: task 393990 panicked - Code: internal - Type: internal - Link: https://docs.meilisearch.com/errors#internal"
To Reproduce
Steps to reproduce the behavior:

Search "Wedding ceremony 結婚式"
Error 500 occurs

Meilisearch version:
v1.3 Meilisearch cloud

Hey @apozeus 👋
Thank you very much for your report. I will investigate this panic. I have a snapshot of a database with the same issue. I hope it will be enough to debug and fix that 🤞

Hey @apozeus again 👋
I am sorry, but do you accept that we use a copy of your Meilisearch data on the Cloud to debug? And if so, please communicate to me the URL of your instance here or via email. (you can find it on my GitHub profile).

Seeing a similar issue here. Is it related?
unaccepted status code found: 500 expected: [200], MeilisearchApiError Message: Inside `.queries[79]`: task 508094 panicked, Code: internal, Type: internal, Link: https://docs.meilisearch.com/errors#internal (path "POST /multi-search" with method "MultiSearch")

Hey @obrhoff 👋
Thank you for reporting this error. Do you use the self-hosted version of Meilisearch? And if so, can you give me the log shown when this error is triggered please?

I just gave it a try and found this in the logs. thread 'tokio-runtime-worker' panicked at 'The ranking rule fid did not sort its bucket exhaustively', milli/src/search/new/bucket_sort.rs:162:13

Fixed by #4013.


Improved header

Show summary & make it collapsible by double click

The code looks great, thanks a lot for looking into this! I'll give it a try this morning and hopefully we can merge it then.
:thumbsup:

ps: it seems to be built on top of #16 so let's merge the other one first.

Merged fix for #16 into this as well, so they are in line

Thanks again for your contributions
now that #16 is merged into master, are you able to rebase this branch onto master? That will get rid of the Merge branch 'master' into summary commit which is no longer needed

Done.

@pbihler sorry but it looks like your branch is in a bit of a weird state, I think it might have been merged, rather than rebased
can you try resetting you summary branch back one commit e.g. git reset HEAD~1 then doing a force push git push origin summary --force , that is assuming your fork has a remote name of origin

That looks better now :)

@pbihler looks great!! thank alot


Add partial upsert config and mergers (WIP)

Description

Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes

Documentation

Add more description and summary on the work done in this PR. Link to relevant section of the design doc.

Codecov Report

Merging #6899 (34b10b9) into master (0185482) will decrease coverage by 7.86%.
The diff coverage is 32.00%.

@@             Coverage Diff              @@
##             master    #6899      +/-   ##
============================================
- Coverage     73.23%   65.37%   -7.87%
  Complexity       12       12
============================================
  Files          1439     1455      +16
  Lines         71333    71935     +602
  Branches      10334    10433      +99
============================================
- Hits          52243    47029    -5214
- Misses        15578    21503    +5925
+ Partials       3512     3403     -109

Flag
Coverage Δ

integration
?

unittests
65.37% <32.00%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ata/manager/realtime/RealtimeTableDataManager.java
10.74% <0.00%> (-58.04%)
:arrow_down:

...not/segment/local/upsert/PartialUpsertHandler.java
0.00% <0.00%> (ø)

...inot/segment/local/upsert/merger/IgnoreMerger.java
0.00% <0.00%> (ø)

...t/segment/local/upsert/merger/IncrementMerger.java
0.00% <0.00%> (ø)

...t/segment/local/upsert/merger/OverwriteMerger.java
0.00% <0.00%> (ø)

...he/pinot/segment/local/utils/TableConfigUtils.java
75.20% <11.11%> (-6.67%)
:arrow_down:

...t/local/upsert/PartitionUpsertMetadataManager.java
84.41% <38.88%> (-10.83%)
:arrow_down:

...ent/local/realtime/impl/RealtimeSegmentConfig.java
89.43% <55.55%> (-9.70%)
:arrow_down:

...rg/apache/pinot/spi/config/table/UpsertConfig.java
78.26% <73.33%> (-9.24%)
:arrow_down:

...local/indexsegment/mutable/MutableSegmentImpl.java
66.21% <88.88%> (-12.65%)
:arrow_down:

... and 384 more

Continue to review full report at Codecov.

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


Restore content in the editor

Hello, thank you so much for creating a wonderful API for us to use. Our team has been planning to work on it. We are saving the content of the editor in a database, and then we try to retrieve them and put it back in the editor. We can't figure out how to put content inside of the editor back again from the database, as the data will be of HTML format and we also want to display codeblocks, but if we just copy and paste the content it will print stuff like
' xyz ' in the editor instead of just xyz
Could some one help me out in this one?

Paa it as a props to the editor

Pas it as a props to the editor

Thanks for the answer. Should I pass the entire HTML content as props and that will render it inside the text editor? Else could you elaborate how to do so? A small poc would be appreciated highly

// using it with redux form just pass the value back
import React, { LabelHTMLAttributes } from 'react'
import { WrappedFieldMetaProps, WrappedFieldInputProps } from 'redux-form'
import ReactQuill from 'react-quill'
import 'react-quill/dist/quill.snow.css'
import Label from './Label'
interface LabelProps extends LabelHTMLAttributes {
input: WrappedFieldInputProps
meta?: WrappedFieldMetaProps
required?: boolean
className?: string
name?: string
id?: string
label?: string
}
const TextArea = ({
input,
className,
label = '',
id,
required = true,
}: LabelProps) => {
return (

)
}
export default TextArea

You can use the value prop to pass in the initial HTML.
https://codepen.io/alexkrolick/pen/xgyOXQ?editors=0010
The onChange prop is how you can read the value and save it to the server when the user types something.
Also see the docs in the README about controlled and uncontrolled mode.


Augmenting help pages

I suggest adding a section to the current help page on resources to provide guidance on how resource types are intended to be used. Currently, the page simply defines each resource type.
This could be as simple as adding a few sentences at the end of the current page such as

Users are encouraged to use resource types that reflect the underlying data structure rather than creating a generic resource. Hydroshare supports automatic metadata creation for resources such as shape files and netcdf files and various apps permit users to preview the resource prior to downloading for declared resource types. These apps cannot access generic resources. If users create a generic resource, it is considered best practice to include a readme.txt file that describes the content of each file contained in the generic resource.

If a user wishes to combine multiple files, potentially of different resource types, we encourage users to use a Composite Resource type rather than a Generic Resource.

@Lizabrazil can you update docs or close if irrelevant or done?

@cuahsimarko can close


Cleanup PKI usage, make path mapping configurable

This PR cleans up the PKI and fixes some bugs. I was trying to set up a local test environment and for that I replicated the PKI using certin (or any other CLI to generate certs), but found that using my PKI, my virtual workspace could not communicate with the kcp shard. After some further digging (because this worked perfectly fine using the Helm chart), I found that the chart makes some weird decisions on how it uses the clean PKI it created. Meaning that the PKI structure makes total sense, but the CLI flags had some weirdness to them that I consider bugs.

The comments for the LE email addresses were moved to a location that is more visible to the admin installing the chart.
The letsencrypt-ca was removed. Since #38 the certificate wasn't even the LE CA anymore, it was CN = kcp.faros.sh   Not After: May 22 13:33:10 2023 GMT. The supposed LE CA was used as the -root-ca-file, which to me doesn't make sense as LE won't be signing certificates inside kcp. This is what the kcp-service-account-signing-ca is probably meant to be used for.
The kcp-selfsigned-issuer was removed, as the default issuer for the kcp-front-proxy cert was changed to be the kcp-server-issuer. There is no good reason in my mind why the front-proxy should not simply use the existing kcp CA.
The kcp-pki-ca issuer was renamed to kcp-pki to be more consistent with the other issuer names.
The external-logical-cluster-admin-kubeconfig used the wrong CA. The CA should be the one that issued the front-proxy's server, not the cert itself.
The logical-cluster-admin-kubeconfig used the wrong CA. The CA should be the one that issued kcp's server, not the CA of the client-auth.
The path mapping can now be configured via values.yaml, allowing users to integrate external virtual workspaces. To make PKI work for those, extraVolumes and extraVolumeMounts can be added to the kcp-front-proxy.
Fixed flags:

kcp-front-proxy:

--client-ca-file=/etc/kcp-front-proxy/client/tls/ca.crt is pointing to the CA that signed the kcp-client-ca, which is effectively the kcp PKI CA. I assume the intention was to actually specify the kcp-client-ca, so this was adjusted accordingly.
unnecessary mounts and volumes were removed; they can be added via values.yaml as needed.

kcp:

--client-ca-file=/etc/kcp-front-proxy/client/tls/ca.crt is the same issue as with the front-proxy.
--requestheader-client-ca-file=/etc/kcp/tls/requestheader-client/ca.crt is the same issue as with --client-ca-file: this accidentally used the CA of the CA.
--root-ca-file=/etc/kcp/tls/ca/root-ca.pem pointed to the seemingly-LE-but-not-really CA, which should not be the CA for the cluster, as all relevant certificates are created internally. The value was therefore changed to /etc/kcp/tls/service-account/ca.crt, to dynamically use whatever CA signed the kcp-service-account certificate (usually this is done internally, but to be flexible and make fewer assumptions, we do not hardcode the CA).
--shard-virtual-workspace-ca-file=/etc/kcp/tls/ca/root-ca.pem should not actually be relevant, as core virtual workspaces run in-process, but nonetheless the kcp cert is not signed by LE and so this flag was adjusted to /etc/kcp/tls/kcp-virtual-workspaces/ca.crt (the CA for a future, not yet existing, external kcp-virtual-workspace server).

With this PR applied, the intended structure of the PKI is like so:

(Apologies, this is a Google slide and not something easily maintainable.)

/approve
/lgtm
Can you trow this picture into hacks dir Readme file? Would like to preserve this for future me :D

/lgtm
/approve


Add Lee County School System

https://www.lee.k12.nc.us/lchs
https://www.lee.k12.nc.us/domain/1295

@DC-Martin Pull request merged. Thank you!


⚠ Bofrost.lu has degraded performance

In e4d412f, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1650 ms

Resolved: Bofrost.lu performance has improved in 007f70b after 6 minutes.


sample_pnc_topics.py: fix the argument check up and use classmethod decorator [#4764]

We should show the usage as user friendly style.
Use classmethod instead of staticmethod decorator since the later is most comfortable resides in helper class.

Well done. Thank you very much!
Please sign the CLA above so we can merge it.


Mostly remove Mixed property support

This leaves in just enough to be able to read from existing Mixed columns in migrations.
Also removes some old pre-Swift 2 cruft related to String/String? being registered oddly with the obj-c runtime.
Closes #3580.

So this will prevent accessing mixed properties but will keep it stored in the Realm. Should we instead be removing those columns during the migration?

They'll be removed by the normal migration functionality since it's now impossible to declare a model with a mixed property.

They'll be removed by the normal migration functionality since it's now impossible to declare a model with a mixed property.

Ha, yes that's obvious now... Looks good to me! 👍

👍


Timestamp property in Sync endpoints is marked as supported methods GET|POST|PUT whereas the endpoint only supports GET

Timestamp property in Sync endpoints is marked as supported methods GET|POST|PUT whereas the endpoint only supports GET
  "\/api\/v1\/{division}\/sync\/CRM\/Accounts": {
        "endpoint": "CRM\/Accounts",
        "documentation": "https:\/\/start.exactonline.nl\/docs\/HlpRestAPIResourcesDetails.aspx?name=SyncCRMAccounts",
        "scope": "Crm accounts",
        "uri": "\/api\/v1\/{division}\/sync\/CRM\/Accounts",
        "supportedMethods": {
            "get": true,
            "post": false,
            "put": false,
            "delete": false
        },
        "example": "\/api\/v1\/{division}\/sync\/CRM\/Accounts",
        "properties": [
            {
                "name": "Timestamp",
                "type": "Edm.Int64",
                "description": "Timestamp",
                "primaryKey": true,
                "supportedMethods": {
                    "get": true,
                    "post": true,
                    "put": true,
                    "delete": false
                },
                "hidden": false,
                "mandatory": false
            },

The same seems to happen for all properties in Cachflow.Banks The endpoint only supports GET (see https://start.exactonline.nl/docs/HlpRestAPIResourcesDetails.aspx?name=CashflowBanks) but multiple propoerties have other methods marked as well.


Handling of missing geomprops in GLSL backend

When loading the following document:
<?xml version="1.0"?>
<materialx version="1.38" colorspace="lin_rec709">
  <geompropvalue name="N_missing_geomprop" type="color3">
    <input name="geomprop" type="string" value="missing" />
    <input name="default" type="color3" value="0.0, 0.0, 1.0" />
  </geompropvalue>
  <UsdPreviewSurface name="SR_missing_geomprop" type="surfaceshader">
    <input name="diffuseColor" type="color3" nodename="N_missing_geomprop" />
  </UsdPreviewSurface>
  <surfacematerial name="MAT_missing_geomprop" type="material">
    <input name="surfaceshader" type="surfaceshader" nodename="SR_missing_geomprop" />
  </surfacematerial>
</materialx>

MaterialXView displays an error:
"Geometry buffer could not be retrieved for binding: i_geomprop_missing. Index 0".
I don't think this is correct behaviour. The <geompropvalue> node has a default input which's value should be used instead.
For comparison, HdStorm allows geomprops to be missing, but uses an (incorrect) default value of 0.0:
https://github.com/PixarAnimationStudios/USD/blob/857ffda41f4f1553fe1019ac7c7b4f08c233a7bb/pxr/imaging/hdSt/materialXShaderGen.cpp#L677-L689

@pablode Good catch, and your description of the intended behavior matches my understanding of the specification.  Here is the method in MaterialXRenderGlsl that generates this warning, and it seems logical to implement the correct fallback behavior there:
https://github.com/AcademySoftwareFoundation/MaterialX/blob/main/source/MaterialXRenderGlsl/GlslProgram.cpp#L273