can't read AIN0, AIN1, ... analog inputs using HAL

Hi,
I'm trying to get some readings in with MAX32650 of AIN0, AIN1, etc. external channels.
No matter what I provide on AIN0, AIN1, pins of the FTHR board, I always get the raw value 1 as reading.
However, if I perform ADC conversions of internal channels I get meaningful values.
The seqence of calls is:
MXC_ADC_Init();
MXC_ADC_StartConversion(ch); // ch is MXC_ADC_CH_0, MXC_ADC_CH_1 etc.
MXC_ADC_GetData(&reading)

Here's the readings that I get converted to Volts using reading * 1.22V / 1024
ain0 = 0.001191 Volts
ain0/5 = 0.001191 Volts
ain1 = 0.001191 Volts
ain1/5 = 0.001191 Volts
vcore = 1.122305 Volts
vdda = 1.218809 Volts
vddb/4 = 0.795859 Volts
vddio/4 = 0.459883 Volts
vddioh/4 = 0.847090 Volts

First values are ALWAYS 1 * 1.22 / 1024 = 0x001191.
The rest of the values seem correct.

nevermind, the FTHR board routes AIN0, AIN1 etc. pins to an external I2C ADC, not to the embedded ADC of the microcontroller...

S

@kevin-gillespie any idea ?

Found the reason...
Board_Init() was doing nothing in my firmware and so by comparison with your example, it wasn't initializing the MXC_SYS_PERIPH_CLOCK_GPIO2 (GPIO2.3 and GPIO2.4).
https://github.com/Analog-Devices-MSDK/msdk/blob/main/Libraries/Boards/MAX78000/FTHR_RevA/Source/board.c#L85

Thanks @buha, can we close this?


Incremental mTLS adoption

Allows mutually authenticated TLS to be used without breaking existing services.

Ideally, this can be done without downtime for the services
Must not break the service for clients outside the Istio mesh
Must not break services which already use TLS (workload, not proxy originated or terminated)

WIP Design Doc

@diemtvu can you link the relevant PRs to this issue for tracking?

https://github.com/istio/api/pull/229
https://github.com/istio/old_pilot_repo/pull/1443


Complete basic MongoDB tutorial

https://www.tutorialspoint.com/mongodb/index.htm
From the start to the "Indexing" (including) should be enough.

Done


[BUG] MGS2 & MGS3 not being detected

I'm unsure if I'm supposed to post the issue here or for Ludusavi, since I don't know where the problem lies.
'Metal Gear Solid 2: Sons of Liberty - Master Collection Version' and 'Metal Gear Solid 3: Snake Eater - Master Collection Version' aren't being detected in Ludusavi.
I've checked the manifest.yaml and PCGamingWiki and the entries look correct
PCgamingWiki entries
Metal Gear Solid 2: Sons of Liberty - Master Collection Version
<path-to-game>\mgs2_savedata_win\<user-id>\

Metal Gear Solid 3: Snake Eater - Master Collection Version
<path-to-game>\mgs3_savedata_win\<user-id>\

manifest.yaml entries
'Metal Gear Solid 2: Sons of Liberty - Master Collection Version':
  files:
    <base>/mgs2_savedata_win/<storeUserId>:
      tags:
        - save
      when:
        - os: windows
    <base>/mgs2_savedata_win/<storeUserId>/launcher:
      tags:
        - config
      when:
        - os: windows
  steam:
    id: 2131640
'Metal Gear Solid 3: Snake Eater - Master Collection Version':
  files:
    <base>/mgs3_savedata_win/<storeUserId>:
      tags:
        - save
      when:
        - os: windows
    <base>/mgs3_savedata_win/<storeUserId>/launcher:
      tags:
        - config
      when:
        - os: windows
  steam:
    id: 2131650

For reference the save locations on my system
Metal Gear Solid 2: Sons of Liberty - Master Collection Version
D:\Steam\steamapps\common\MGS2\mgs2_savedata_win\76561197960485586
Metal Gear Solid 3: Snake Eater - Master Collection Version
D:\Steam\steamapps\common\MGS3\mgs3_savedata_win\76561197960485586

Thanks for reporting this! This is the correct repo in this case, but if you're not sure, then either is fine :)
The manifest was missing the installDir field needed to find the MGS2 part of the path. I've refreshed the Steam data for these games and updated the manifest just now. Once Ludusavi downloads the new manifest, it should start working.
Right now, there's an issue where the Steam metadata can become outdated. I'll look into a solution to make sure that it's refreshed more proactively.

Just tested it, and it looks like it's working now.


Look at AWS autoscaling

Pros:
-Stats collection as part of cloud watch
-We can create custom cloud watch metrics, and auto scaling uses cloud watch metrics as 'triggers' for scaling up or down, but I haven't seen an explicit answer whether custom metrics can be used as triggers. If we can, we could potentially publish task queue lengths as 'triggers'
-Preexisting metrics for resource utilization
-spot market support
-existing instances can be added to auto scaling group
Cons:
-Only one instance type per auto scaling group.
-Triggers might be too simplistic. They are in the form of "If (metric), scale up/down x percent/instances". An example of a problem this causes: there are separate metrics for CPU and memory, so if we were using 100% CPU but our memory is below the set trigger point, the memory trigger would scale the cluster down.

Good summary.


fix(stepper): Fix visual flake in focusing stepper header

When the step is changed through a button click sometimes the focus of the new
step header is attributed by focus monitor to 'mouse' instead of 'program'. This
results in a visual difference in the step header (focus background is missing).
Instead, when focusing the step header focus it via 'program' explicitly through
the focus monitor.

@mmalerba could you take a look at this PR?

I updated the public API file to take into account the "api change" (not sure if this counts as a change since it's already inherited as a public method from the parent).
Let me know if this wasn't the right fix for the test


Ansible request for logging when the playbooks are executed

Please put the name of the software product (and affected platforms if relevant) in the title of this issue

[x] Missing install
[ ] Bug in ansible playbook
[ ] Request for new playbook addition

Details: I had a go at this once before and didn't quite manage to make it work, so it needs revisiting. It would be useful to record the last time that the ansible playbooks were run on a particular machine but logging it to e.g. /var/log/ansible.log A couple of lines in the log like:
202106031057 : Ansible playbook started
202106031203 : Ansible playbook completed

Would allow us to know

When the playbooks were last run
Whether they were successful (based on the presence or absence of the completed message)
This should be tagged as adoptopenjdk
Bonus points if we can include the SHA of the infrastructure repository used during the deployment and any list of explicit included/excluded tags.

I have found that by default ansible logs to the remote host's syslog. If I run a playbook onto a remote host:
hkhel-mac:playbooks hkhel$ ansible-playbook -u root -i hosts AdoptOpenJDK_Unix_Playbook/main.yml -t "debug" -v

On the remote host's  /var/log/syslog file I find
Mar  9 14:54:01 test-ibmcloud-ubuntu1604-x64-1 systemd[1]: Started Session 131518 of user root.
Mar  9 14:54:04 test-ibmcloud-ubuntu1604-x64-1 ansible-ansible.legacy.setup: Invoked with filter=[] gather_subset=['all'] fact_path=/etc/ansible/facts.d gather_timeout=10

You can tell ansible to save logs on the controller node by placing this variable in the ansible.cfg file:
log_path=/path/to/log

Which saves the ansible-playbook stdout to a file. Ideally I would want these same logs to be saved to a file on the remote host

Closing #2709 to cover all issues in here:
At the moment we have some provisional code that should do the first of these, but it is not working properly and does not create the log files. We should try and get a fix for that as a first pass, then look at the others.
Ideally this should include:

Start time
Completion times (if it finishes - the absence of a finish time should indicate failure of some sort)
Ideally which SHA of the infrastructure repository was used for the deployment.

See also https://ci.adoptopenjdk.net/view/Tooling/job/VagrantPlaybookCheck/1518/OS=Ubuntu2004,label=vagrant/consoleFull for a verbose debug run with some extra options

@Haroon-Khel Is this complete on all platforms now?

Windows: https://github.com/adoptium/infrastructure/pull/2848
UNIX: https://github.com/adoptium/infrastructure/pull/2746
AIX: https://github.com/adoptium/infrastructure/pull/2918

This is complete once https://github.com/adoptium/infrastructure/pull/2918 gets merged


Add Apple Silicon build and Universal build for macOS

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 42.353%

Totals

Change from base Build<PHONE_NUMBER>:
0.003%

Covered Lines:
29390

Relevant Lines:
64565

💛  - Coveralls


run test for multi-backend data access scenario

I suggest a test using cdse aggregator, where we combine data from cdse and terrascope backend.
It's also fine for me to add cdse to openEO platform dev aggregator, but that would come down to the same thing in the end.

This script works on CDSE-dev, with collections on terrascope and cdse:
https://gist.github.com/soxofaan/495deef94fc0b61654e2bf196e228a4f#file-issue115-crossbackend-execution-ipynb


updated latest changes

Updated as request with exception to adding images

Anvesh - did you pull down my latest changes before doing a this pull request? You should always incorporate the latest master changes before doing a pull requests. There are merge conflicts cited here


[EnumRequest] Table 7000002 - Field 40 - "Document type"

Hello,
we need to be able to extend the  options of field 40 - "Document type" in table 7000002 - "Cartera Doc.".
Could you please change it into an enum?
(It also'll affect to the same field in tables 7000003 - "Posted Cartera Doc." and 7000004 - "Closed Cartera Doc.")
Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: .


tf.estimator.train_and_evaluate does not show anything when running multi-worker example in TensorFlow 2.0 Alpha

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): 16.04
TensorFlow installed from (source or binary):
TensorFlow version (use command below): TF 2.0 Alpha CPU only
Python version: 2.7

I have run this example in TF 2.0, however the estimator train and evaluate does not show anything as provided in this link: https://www.tensorflow.org/alpha/tutorials/distribute/multi_worker
from __future__ import absolute_import, division, print_function

import tensorflow_datasets as tfds
import tensorflow as tf

import os, json

BUFFER_SIZE = 10000
BATCH_SIZE = 64

def input_fn(mode, input_context=None):
  datasets, ds_info = tfds.load(name='mnist',
                                with_info=True,
                                as_supervised=True)
  mnist_dataset = (datasets['train'] if mode == tf.estimator.ModeKeys.TRAIN else
                   datasets['test'])

  def scale(image, label):
    image = tf.cast(image, tf.float32)
    image /= 255
    return image, label

  if input_context:
    mnist_dataset = mnist_dataset.apply(tf.data.experimental.filter_for_shard(
        input_context.num_input_pipelines, input_context.input_pipeline_id))
  return mnist_dataset.map(scale).shuffle(BUFFER_SIZE).batch(BATCH_SIZE)

NUM_WORKERS = 1
IP_ADDRS = ['localhost']
PORTS = [12345]

os.environ['TF_CONFIG'] = json.dumps({
    'cluster': {
        'worker': ['%s:%d' % (IP_ADDRS[w], PORTS[w]) for w in range(NUM_WORKERS)]
    },
    'task': {'type': 'worker', 'index': 0}
})

LEARNING_RATE = 1e-4
def model_fn(features, labels, mode):
  model = tf.keras.Sequential([
      tf.keras.layers.Conv2D(32, 3, activation='relu', input_shape=(28, 28, 1)),
      tf.keras.layers.MaxPooling2D(),
      tf.keras.layers.Flatten(),
      tf.keras.layers.Dense(64, activation='relu'),
      tf.keras.layers.Dense(10, activation='softmax')
  ])
  logits = model(features, training=False)

  if mode == tf.estimator.ModeKeys.PREDICT:
    predictions = {'logits': logits}
    return tf.estimator.EstimatorSpec(labels=labels, predictions=predictions)

  optimizer = tf.compat.v1.train.GradientDescentOptimizer(
      learning_rate=LEARNING_RATE)
  loss = tf.keras.losses.SparseCategoricalCrossentropy(
      from_logits=True)(labels, logits)
  if mode == tf.estimator.ModeKeys.EVAL:
    return tf.estimator.EstimatorSpec(mode, loss=loss)

  return tf.estimator.EstimatorSpec(
      mode=mode,
      loss=loss,
      train_op=optimizer.minimize(
          loss, tf.compat.v1.train.get_or_create_global_step()))

strategy = tf.distribute.experimental.MultiWorkerMirroredStrategy()

config = tf.estimator.RunConfig(train_distribute=strategy, eval_distribute=strategy)

classifier = tf.estimator.Estimator(
    model_fn=model_fn, model_dir='/tmp/multiworker', config=config)
tf.estimator.train_and_evaluate(
    classifier,
    train_spec=tf.estimator.TrainSpec(input_fn=input_fn),
    eval_spec=tf.estimator.EvalSpec(input_fn=input_fn)
)

The result in the terminal is like this:
2019-04-06 19:26:08.807447: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 AVX512F FMA
2019-04-06 19:26:08.825158: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2019-04-06 19:26:08.825963: I tensorflow/compiler/xla/service/service.cc:162] XLA service 0x5944140 executing computations on platform Host. Devices:
2019-04-06 19:26:08.825997: I tensorflow/compiler/xla/service/service.cc:169]   StreamExecutor device (0): <undefined>, <undefined>
WARNING: Logging before flag parsing goes to stderr.
W0406 19:26:08.827748<PHONE_NUMBER>4000 cross_device_ops.py:1106] Not all devices in `tf.distribute.Strategy` are visible to TensorFlow.
2019-04-06 19:26:08.830148: I tensorflow/core/distributed_runtime/rpc/grpc_channel.cc:250] Initialize GrpcChannelCache for job worker -> {0 -> localhost:12345}
2019-04-06 19:26:08.831611: I tensorflow/core/distributed_runtime/rpc/grpc_server_lib.cc:359] Started server with target: grpc://localhost:12345
W0406 19:26:08.841363<PHONE_NUMBER>4000 cross_device_ops.py:1106] Not all devices in `tf.distribute.Strategy` are visible to TensorFlow.
W0406 19:26:08.843209<PHONE_NUMBER>4000 cross_device_ops.py:1106] Not all devices in `tf.distribute.Strategy` are visible to TensorFlow.
W0406 19:26:09.397347<PHONE_NUMBER>4000 monitored_session.py:344] Collective ops may deadlock with `save_checkpoints_secs` please use `save_checkpoint_steps` instead. Clearing `save_checkpoint_secs` and setting `save_checkpoint_steps` to 1000 now.
2019-04-06 19:26:09.526794: I tensorflow/core/distributed_runtime/master_session.cc:1194] Start master session fdddc32e0a2bc48c with config: device_filters: "/job:worker/task:0" device_filters: "/job:worker/task:0" device_filters: "/job:worker/task:0" allow_soft_placement: true graph_options { rewrite_options { meta_optimizer_iterations: ONE scoped_allocator_optimization: ON scoped_allocator_opts { enable_op: "CollectiveReduce" enable_op: "CollectiveReduce" enable_op: "CollectiveReduce" } } } experimental { collective_group_leader: "/job:worker/replica:0/task:0" }
2019-04-06 19:26:10.319007: W ./tensorflow/core/framework/model.h:202] Encountered a stop event that was not preceded by a start event.
2019-04-06 19:26:23.553348: W tensorflow/core/common_runtime/eager/context.cc:195] Unable to destroy server_ object, so releasing instead. Servers don't support clean shutdown.

Is this bug? or how to solve this problem?

It seems that when I enable: tf.compat.v1.logging.set_verbosity(tf.compat.v1.logging.INFO). This error happens
I0407 00:24:10.743343<PHONE_NUMBER>6048 monitored_session.py:241] Graph was finalized.
I0407 00:24:10.745394<PHONE_NUMBER>3616 monitored_session.py:241] Graph was finalized.

INFO:tensorflow:An error was raised while a session was being created. This may be due to a preemption of a connected worker or parameter server. A new session will be created. This error may also occur due to a gRPC failure caused by high memory or network bandwidth usage in the parameter servers. If this error occurs repeatedly, try increasing the number of parameter servers assigned to the job. Error: OS Error
INFO:tensorflow:Graph was finalized.

I solve this problem. It seems that we must use 'localhost', cannot use 'host' only

That's great. For now we are closing the issue. If you are stuck anywhere, we will open the same again. Thanks!

I changed my "localhost" to "ip" and I try other methods, however, it doesn't work.
@ymsaputra can you help me? thx!


Use the index of an event in its containing array as the id for each …

Inputting the same midi event (e.g. pressing the same key) into the "EndpointPickers" Example app (macOS & iOS) results in the following warning being posted to the console:
ForEach<ReversedCollection<Array>, MIDIEvent, Text>: the ID noteOff(36 (C1)), midi1(64)/midi2(32768), chan: 0x0, group: 0x0) occurs multiple times within the collection, this will give undefined results!x
Issue is that identical MIDIEvent's have the same id. In proposed fix we use the event's index in its containing array as the unique ID.

Yes, was a known issue in the example projects that log events to the UI and not the console.
Index seems like a fine solution for the meantime. Conforming MIDIEvent to Identifiable seems like more overhead than it's worth, even just as a stub in the example project. Really, MIDI events are just communication messages that should result in some application action as a response, apart from simply logging the events.
In truth, it's more for demonstration on how to integrate an asynchronous service like receiving MIDI events into SwiftUI views. The two worlds are not terribly friendly to each other.

Yeah, I don't think making MIDIEvent conform to Identifiable would be the right solution for this problem. I expect identical MIDIEvent's to have the same ID which would means this problem would not be solved with such a solution.
A "full" solution might be to capture the time a given MIDIEvent was received (in high precision) and encapsulate that time along with the received MIDI event in some type of a "MIDITimeStampedEvent" struct but I think that's way overkill given this is a simple example app.
The use of indices, while a little inefficient, gets rid of the console spew.
P.S. MIDIKIt rocks! Nicely done. Thank you for creating this.

Great points, and agreed on all. The goal is just to make some demo apps here of course, but to not lead anyone astray by demonstrating anything counter to best practises.
MIDIEvent's identity should of course follow its Equatable/Hashable result. So forming a list would mean we need an additional piece of information as a unique identifier - like array index or UUID. Keep in mind that it's possible for multiple events to arrive in the same packet and bear the same time stamp so that wouldn't be suitable for uniquing in a collection.
MIDIEvent doesn't (and shouldn't) carry a timestamp because it's an event definition. Timing is defined differently in MIDI I/O than it is in Standard MIDI File encoding, for example. Also, for I/O you already receive the time stamp in the receiver block of the input or input connection and it can be carried forward as you see fit.

P.S. MIDIKIt rocks! Nicely done. Thank you for creating this.

Thanks very much! I built it to be the most complete, correct, documented, stable, and versatile library. Especially with MIDI 2.0 it felt like the right time to build something from the ground up that fully considers 1.0 and 2.0 and doesn't just retrofit 2.0 in. I plan on continuing its development as time allows to fill in gaps and extend functionality.


socketify.AppOptions defaults are not working as expected

Describe the bug
When trying to run examples/https.py I had to specify all AppOptions defaults.
To Reproduce
Follow #85 and discover that AppOptions require all defaults to be passed explicitly. Namely, without the following arguments:
        dh_params_file_name=None,
        ca_file_name=None,
        ssl_ciphers=None,

running examples/https.py will throw exceptions like this one:
Traceback (most recent call last):
...
  File "<string>", line 10, in __init__
  File "/usr/local/lib/python3.9/site-packages/socketify/socketify.py", line 3355, in __post_init__
    raise RuntimeError("dh_params_file_name must be a str if specified")
RuntimeError: dh_params_file_name must be a str if specified

Expected behavior
No error should expected when one of the optional parameters missing.
Additional context
AppOptions attributes should probably be defined with None defaults:
class AppOptions:
    key_file_name: str = None
    cert_file_name: str = None
    ...

instead of a tuple (perhaps it's a typo):
class AppOptions:
    key_file_name: str = (None,)
    cert_file_name: str = (None,)
    ...

Yeah some refactor break it, will push the fix today thanks for reporting


DistributedSession with MemoryDistributedCache is unreliable

If DistributedSession uses MemoryDistributedCache, it is unreliable because MemoryDistributedCache uses MemoryCache without CacheItemPriority.NeverRemove option. Then MemoryCache flushes live session not respecting TTL (session idle time interval).
To fix this, both Session and Caching must be modified to set and allow priority when storing the cached item.

All caches are subject to cache purges, not just MemoryDistributedCache. Session is not designed as a reliable store, that's why it's built on cache.

I am unable to find HTTP session reliability definition :-)
I agree that in case of unauthenticated users, DDoS attack would be very easy if session cannot be expired before idle timeout.
But imagine authenticated user, who needs some time to fill complex form. If authentication is based on session which prematurely expires, user runs into login dialog losing the filled form data.
It would be great if session could be configured to be reliable or not. At least MemoryCache allows it.

You could still be DDoS'd by authenticated users, think Black Friday.
Is this question theoretical or have you actually had trouble with this on your site? In theory a MemoryCache purge should only happen when your site is under significant memory pressure.

Actually, I have the trouble which I described above. It is single page application and user have to fill more pages using wizard which takes some time. Authentication is based on session. Even the session idle time is set to 30 minutes, user is sometimes asked for reauthentication after 5 minutes, but there seems to be no memory pressure :-(

Interesting. We should investigate the cache purge trigger: https://github.com/aspnet/Caching/issues/221

We'll investigate the issue in Caching.


Create coverage reports with PRs

We have functionality to create coverage reports in the Makefile already.
I would like to see them added to our PRs so we can make sure new code is tested.

Our pre-commit GitHub Action will run a test against unit test coverage and fail if the threshold is not met. Closing.


Safari support

Will be safari support added in near future?

Safari works.
They use HLS, and Mp4.
but to get Autoplay on iPhone then it requires that you to add the playsinline parameter to your video tag.

https://webkit.org/blog/7734/auto-play-policy-changes-for-macos/

The player module adds the playsinline attribute by default, see https://github.com/matvp91/indigo-player/blob/master/src/player/HTML5Player/HTML5Player.ts for reference.
As for autoplay restrictions: if autoplay is enabled in the config, the player will attempt to autoplay and if it fails, it will be handled gracefully (& show a play button instead).

The player module adds the playsinline attribute by default, see https://github.com/matvp91/indigo-player/blob/master/src/player/HTML5Player/HTML5Player.ts for reference.
As for autoplay restrictions: if autoplay is enabled in the config, the player will attempt to autoplay and if it fails, it will be handled gracefully (& show a play button instead).

I know this issue is closed, but is this behaviour still the same?
I am expecting with autoPlay: false to get a play button displayed, but after firing .init(), it autoplays.


Updates golangci_lint error formats for lll

The long line linter doesn't include a column in the error output:
$ golangci-lint run --disable-all --enable=lll --out-format=line-number --print-issued-lines=false main.go
main.go:216: line is 124 characters (lll)
main.go:217: line is 124 characters (lll)
main.go:220: line is 124 characters (lll)
main.go:235: line is 122 characters (lll)
main.go:237: line is 134 characters (lll)
main.go:238: line is 128 characters (lll)
main.go:269: line is 122 characters (lll)
main.go:420: line is 148 characters (lll)

This commit adds an errorformat that matches lint output missing column information.

Yes and no, all the columns are marked as 0 in the JSON output.
$  golangci-lint run --disable-all --enable=lll main.go --out-format=json --print-issued-lines=false | jq
{
  "Issues": [
    {
      "FromLinter": "lll",
      "Text": "line is 124 characters",
      "Pos": {
        "Filename": "main.go",
        "Offset": 0,
        "Line": 216,
        "Column": 0
      },
      "SourceLines": [
        "[redacted]"
      ],
      "Replacement": null
    },
    ...
  ],
}

I'm not terribly familiar with vim's errorformat parsing, but another potential issue with JSON output is that it's all single-line. And this should probably be filled upstream, but golangci_lint doesn't seem to respect the --print-issued-lines flag when the output format is JSON.

Thanks for the info.
Handling JSON requires extra processing (cannot use "errorformat" directly with it (usually)).
So it's fine like that for now - especially given there's no extra benefit.

Thanks!
(I've missed that this affects a particular linter only, and that columns are still there otherwise.  All fine!)


Refactor node's public API to facilitate moving state into parallel tasks

The top-down design dictated by traits like SuzukaFullNode where the "god object" is borrowed immutably as &self by async methods does not work well with background tasks that ideally need to work on moved, rather than shared, bits of state. You end up needing Arc<Mutex<_>> for anything mutable even if conceptually it's only used in one task.
Consider refactoring to an API that will have two phases:

Initializing the node through a non-async constructor method or builder API.
The async method to run the node consuming the state, something like pub async fn run(self) -> Result<(), NodeError>.

Originally posted by @mzabaluev in https://github.com/movementlabsxyz/movement/pull/58#discussion_r1601170509

I agree with you, and this pattern of using Arc<Mutex<_>> to manage concurrent access can lead to at best contention and at worth deadlock when more and more functionalities are added.
From my experience, what works well is a sort of supervisor loop that start task and manage data sharing using actor model or event driven pattern.
The idea is to dissociate the way you access the data (you access them by moving, cloning, locking depending on the context) from how you process them.

@mzabaluev

An alternative approach is to construct two objects: a handle for invoking public methods on, and a future to drive the background implementation.

The advantages of this are presumably:

Signatures for public methods can remain & self because receiver-based tasks are being managed elsewhere.
Overall separation of concerns.

The disadvantages are:

Readability.
API disunity.

Additionally, any state in the background implementation should really only be accessed through the public methods. So, we are kind of disintegrating something that should be integrated.
For me, ideally, these concerns are pushed down from the top-level implementation. I certainly don't care if we're doing the above somewhere. But, to the extent we can wrap things up in a nice bow, I still encourage us to do so.
Ultimately, though, for me the ideal is still to have an API that manages this complexity and looks something like this w.r.t. background tasks: https://github.com/a16z/helios?tab=readme-ov-file#using-helios-as-a-library
@musitdev

The idea is to dissociate the way you access the data (you access them by moving, cloning, locking depending on the context) from how you process them.

I also don't mind a bit of an overuse of synchronization primitives in early development .But, I agree, there should be better patterns we can leverage here.  @mzabaluev and I have talked about this being an actor pattern already, though an impure one. Can you sketch out how you would modify the client to implement a purer actor pattern. Diagrams help.

Appropriately, our own unit tests demonstrate the shortcomings of the current approach: when you have a god object to serve external requests and run the service tasks, it needs to be cloned and it needs to have interior mutability.

I agree with your comment and the event (pubsub) / actor model for the API. The thing I would like to add is we should avoid forcing everything to be async. For example, some actor task like Tx execution should be executed using thread. Future can be a way to manage both, but in fact task are async (network related) or sync (processing task) not both. So when a task is defined, we should have the possibility to define it async or sync depending on it is CPU or Network bound.
Async function has the disadvantage to propagate and force everything to be async. This lead to complex issue where CPU bounded task are run with spawn_blocking everywhere, and we end with a thousand thread running when the load increase with no way to manage them.
If we refactor the code, we should provide an efficient sync execution part and all CPU task shouldn't be async.

where CPU bounded task are run with spawn_blocking everywhere, and we end with a thousand thread running when the load increase with no way to manage them.

Unless you also multiply runtime instances, all blocking tasks should be scheduled on the same work-stealing thread pool (which may present its own problems). But I agree that we should not make actors async unless there is a benefit to it. Tokio's channels support blocking operations and therefore can be used for message-passing between synchronous and asynchronous domains.

Yes, using channel is a pattern to mix async/sync.
If we change the archi, one thing that we should do is to integrate our logic in automate state machine so ease the dev and evolution. The pattern is the state of the machine change when receiving event and emit a message on change.
Some logic in node dev can be very complex and hard to implement because they involve a lot of separate components or process. For example, the block processing logic. Currently, it is spread in separate part: read_blocks_from_da manage the Da availability, the execution and the commitment sending but not the error that can occurs. The commiment logic (accepted, rejected) is not implemented but won't be in read_blocks_from_da. So to manage the block logic (Da, execution, commitment process and errors that can occurs) the process is going to be implemented in several places where it can be hard to link together all the logic.
What I propose is to define all the block logic in one place that receive event from the other process (Da, Execution, storage, Settlement, Error) and manage all state transition. These state transition will generate task in the other module (Execution,  storage, Settlement, ..). This way the complex part, the block logic, is done in one place and the other module are more like actor that process block logic messages and return event after the processing.

A relevant guide: Actors with Tokio.

@mzabaluev Can this be close in favor of #340 and follow-ons?


Cannot login

Describe the bug
When trying to login the CLI asks for a token, but you cannot paste or type at the prompt
To Reproduce
npx supabase login
then try to paste and nothing happens
pressing enter suggests that you have not entered in a token
Expected behavior
I would expect to log me in
Screenshots
not really
Desktop (please complete the following information):

OS: windows 11
Browser (if applicable) [e.g. chrome, safari]
Version of CLI: 1.75.2
Version of supabase-js: 2.26.0
Version of Node.js: v18.15.0

Additional context
Add any other context about the problem here.

Closing as duplicate. You may find the suggestions in these issues useful

https://github.com/supabase/cli/issues/1000#issuecomment-1590711838
https://github.com/supabase/cli/issues/1054#issuecomment-1536393966


Don't use sleep in update loop

Depends on https://github.com/ros-controls/ros2_control/pull/568

now depends on #263

now depends on #263

It is more  #263 replaced this one!


Get rid of virtual environments for python

I seems like virtual envs are redundant given that we will be containerizing our application in docker. So we could get rid of it and have requirements.txt file instead? or something like that. @codingbug671 What do you think?

Sure ! go ahead


Will I be able to consume the GitHub access_token?

I will need the GitHub access_token for my app, how can I access that if possible?
https://developer.github.com/apps/building-oauth-apps/authorizing-oauth-apps/#2-users-are-redirected-back-to-your-site-by-github

This wrapper doesn’t provide it because it doesn’t necessarily make sense. My advice would be to consume github oauth directly if you need this.
However, if for some reason you need to use cognito, you can find a short discussion on #31 that may help.

Thanks! I’ll try that out.

You’re welcome.
You’re not the first to ask about this, so it might make sense to add a pass-through for the token as an optional feature (even though it doesn’t make sense for most use cases, it would still be nice to enable it where possible).

Hey @TimothyJones following up on this!
I added the access_token as a custom attribute it seems like my Cognito User object doesn't return it.

Actually, I didn't give it time to propagate. It shows up now


PYCC does not have _dynfunc.Closure

The "python to nopython" call wrapper uses _dynfunc.Closure and the Enviornment object for certain operations: e.g. returning array, storing constant python objects, etc..  However, the PYCC does not emit the _dynfunc.Closure. When these features are used, we may get a segfault.  #1263 inserted a subtarget information to bypass the usage Closure as a temporary fix.

This is for the "extension module" mode of pycc? I'm not extremely surprised :)
In my opinion, the question is still what we want to do with pycc. The "extension module" mode competes with Cython, and Cython is orders to magnitude better than pycc currently. We would need to devote a lot of time to it if we wanted to reverse that situation.

Yes, this is for extension module.  I am also not surprised and agree that we will need to make a lot of effort to make PYCC competitive with Cython.  This issue should remind us that we should not forget its usecases and should not implement something that is very difficult to be AOT compiled.

Well, my opinion is that we should leave the "AOT extension modules" use case to Cython.

This is now fixed :)


VG-4547 - protractor test field editing

Can one of the admins verify this patch?

Current coverage is 79.50% (diff: 100%)

Merging #47 into master will not change coverage

@@             master        #47   diff @@
==========================================
  Files           111        111
  Lines          7225       7225
  Methods        1551       1551
  Messages          0          0
  Branches       1051       1051
==========================================
  Hits           5744       5744
  Misses         1481       1481
  Partials          0          0

Powered by Codecov. Last update 6e18dd8...f39c288


URGENT: Cluster amplitude incorrectly reported

Clusters that based on the Time v. Amplitude graph are very large (~70uVpp) have calculated amplitude much smaller (8.7uVpp) in the unit position graph and when the values are exported. Makes it hard to trust any of the values. See snippet, cluster 84 is a good example of this. I'll send you the files.

Fixed.


Split Code Option Added

Split Code Option Added. This is to allow automatic multiple split payments

Hi, I also came across the need for this. What is still required to get it in?

Hi @DevDianDankie, some unnecessary file changes need to be reverted

Hi @ashinzekene for now I pulled in the module directly into my project and updated it there.
But just thought to point it out.

Thanks for the help @DevDianDankie and sorry for the late response, could you please fix the merge conflicts and let's get this in?

#85
Fixed in this PR @onyilo @DevDianDankie

Fixed in another PR


Account Switching

Hi,
is there any way to quickly switch from one account to another (like in the ui of https://cloud.ibm.com) ?
The idea is to avoid the re-login step, overwriting the session, etc ... We are forced to use the login with the passcode on the browser so the login process is manual and painful.
If I try to go back to a previous session, i need a backup of the full 'HOME/.bluemix' directory or only to the 'HOME/.bluemix/config.json' file ?
Thx in advance

Hi @frichard35 - you can switch accounts by using the target command: ibmcloud target -c $account_id after you logged in to the CLI. To view which accounts the current login session has access to, you can run the ibmcloud account list command. Hope this answers your question.

Pretty obvious... I don't know why i didn't see this option.
Thx for your help.


Scrollbar LMB click and hold scrolling

Pull Request Description
Implements https://www.pivotaltracker.com/story/show/183390702
Important Notes
This PR adds browser timers (setTimeout and setInterval) support to the FRP core. Those then are used to implement the "hold to scroll repeatedly" functionality.
There was a camera-releated bug in the scroll-view debug scene, due to hardcoding made specifically for component browser. This is now fixed, and instead component browser sets the scroll-view camera manually. This is due to change when camera handling in layers will be refactored, and by then no explicit camera setting will be necessary.
There are also some minor extra changes or docs improvements in areas I was going through when looking at FRP implementation.
Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[ ] Unit tests have been written where possible.
[x] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run ide build and ./run ide watch.

I tried writing a fake browser time support in our mock bindings in order to test the timers, but it turns out to be a quite large change on its own due to an assumption that all mocks are ZSTs (there is a transmute that expects that). We would need to preserve the callback in Function object to be able to call it later, which violates that assumption. I think it warrants a separate task.


Failed to download in China

The intellicode extensions failed to work in mainland China due to network issues. Other MS products works fine. And this is a persistent issue.

Can you please provide the logs that you see in the IntelliCode output window when the model download fails?
Also, can you try the troubleshooting steps at https://docs.microsoft.com/en-us/visualstudio/intellicode/intellicode-visual-studio-code#no-starred-suggestions, including making sure the network proxy support is enabled?

@CavalloneChen - are you able to reproduce and share logs?

Now tracking in 102.


clear resharper error

nice

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


Pwning OWASP Juice Shop documentation needs updating

In a meetup we held yesterday for new and intermediate developers, we had attendees follow the challenge guide hints and solutions listed here: https://bkimminich.gitbooks.io/pwning-owasp-juice-shop/content/appendix/solutions.html
Unfortunately, attendees were unable to locate the main.js file in their Chrome Dev Tools as noted in the solutions.

We had deployed v7.5.1 to Heroku and our attendees saw this file hierarchy under "Sources" instead.

Should we have some language that directs people to update their version? Or that the documentation only covers a specific version of JuiceShop.
Love what you folks are doing with JuiceShop. Keep up the good work!

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Hi @jaybobo! The eBook's latest version is for the 8.x release of the Juice Shop, which is based on Angular, while the previous major version 7.x was still AngularJS. That's when the client-side JS file name changed from juice-shop.min.js into main.js.


Publish Package for Scala 2.12

Hi,
I was looking into this project while I was researching a way to integrate couchbase as a backend for akka-persistence snapshot or journal plugins. Our project is currently built in Scala 2.12.3, and depends on akka 2.5.6. Do you have any plans in the future to publish this library on newer versions of scala or akka?
Thank you!

Hi,
I can look to upgrade it, I don't expect it to cause any problems. That said, there are some issues with Couchbase,mainly when reading your own writes. I would recommend using something like Cassandra, which is supported by Typesafe. Couchbase 4 supports something like materialized views, which keeps the view always up to date, while this version depends on Couchbase 3, which does not have this feature. The library is used in a high-load production environment, without any issues, so in practice things are not too bad. It really depends on your usage requirements.
Regards,
Andre
—
ANDRE KAMPERT
SOFTWARE CRAFTSMAN
MOB: +31 6 4826 0837
<EMAIL_ADDRESS><EMAIL_ADDRESS>http://www.product-foundry.com/
PRODUCT FOUNDRY
PLANTAGE MIDDENLAAN 62
1018 DH AMSTERDAM
THE NETHERLANDS
WWW.PRODUCT-FOUNDRY.COM http://www.product-foundry.com/

On 1 Nov 2017, at 06:54, Hyun Min Choi<EMAIL_ADDRESS>wrote:
Hi,
I was looking into this project while I was researching a way to integrate couchbase as a backend for akka-persistence snapshot or journal plugins. Our project is currently built in Scala 2.12.3, and depends on akka 2.5.6. Do you have any plans in the future to publish this library on newer versions of scala or akka?
Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/Product-Foundry/akka-persistence-couchbase/issues/21, or mute the thread https://github.com/notifications/unsubscribe-auth/ABVRnzf4o8f-CnbXF1CuMqobl5dndvteks5syAd4gaJpZM4QNyAG.

Hi, Andre. Thanks for the response.
Do you have any experience on how Couchbase's query consistency might cause any troubles? Our team is considering using Couchbase as a persistence backend in a system that utilizes akka-persistence and cluster sharding together. My guess is that some snapshots or journals may be lost during the recovery process when persistent actors are rebalanced to a new shard. :(
Best,
Hyun Min

The problem with Couchbase 3 occurs because of the eventually consistent views. All data is written as a key value pair, with no support for quering. In order to implement the journal and Persistence Queries, some views are created. There is a slight delay however in the views being updated. Take a scenario for example where a persistent actor writes data to the journal, crashes and then recovers before the view is updated. In this scenario, the last write is missed, although it will eventually be available in the view. The in memory representation of the persistent actor is now out of sync with the stored data, which can cause all kinds of problems. In some cases, this may be okay, but it should be a concern.
If you are using Couchbase 4, I'd prefer the query API, as all updates will be included directly. Because of other commitments, I personally don't have time to make these changes at the moment.
Hope this helps.

I am planning to use Couchbase 4 in the production, perhaps I can help with the changes that are required to support query API.

Wonderful, feel free to create a pull request

@andrekampert - Sure, any pointers on where/what should I start first?

Unfortunately I am currently not in a position to help out, but the discussion above outlines the issues with the current approach.


API: hasStamp

Need to know if a BlElement have a stamp.
Actually: self allStamps isEmpty.

a test would be cool here


Create elasticsearch_mem_config.md

Wrote a quick tutorial on how someone can download elasticsearch from source on a debian machine and alter memory configurations to lower the heaviness of elasticsearch on local machine.

Thanks @InquestGeronimo for your contribution.
At this time we cannot support other than basics for elasticsearch. For mac and windows users, we include already a link to official installation homepages.  For other os environment we could include some links to the main installation page


tests: mem_pool: Remove useless specifier on function declaration

'extern' seems useless there, removing it. This was flagging as an error
on some board.
Signed-off-by: Tomasz Bursztyka<EMAIL_ADDRESS>

This was flagging as an error on some board.

this patch is harmless, but I'm having trouble conceiving on how this would generate an error. "extern" preceding a function prototype in a header is valid C, no?

@andrewboie it surprised me as well, on one of my PR CI complained about this test failing on lpcxpresso55s69_ns board.


Enabled the initial version of web UI using Flask for Issue #52

Addressed #52 partially with initial version that covers IQR
Added web app functionality using Flask
Added some test cases
Tested the basic functionality using UI

When there are no outliers

Where there are outliers


Feedback on 2022.11.0-dev

Hey,
Some feedback on 2022.11.0-dev after 24 hours or so. Everything is looking good, lights seem at least as responsive as before and nothing has been marked unavailable.
I have some log messages, but this is quite a lot less than I would normally expect to see.
Logger: custom_components.lifx
Source: custom_components/lifx/util.py:199
Integration: LIFX
First occurred: November 23, 2022 at 11:52:40 PM (6 occurrences)
Last logged: 7:14:27 AM
No response from LIFX bulb Dining Table Pendant 1 after 8.2568 secs
No response from LIFX bulb Dining Table Pendant 1 after 8.2744 secs
No response from LIFX bulb Dining Table Pendant 1 after 8.2552 secs
No response from LIFX bulb Living Room 14 after 8.2561 secs
No response from LIFX bulb Dining Table Pendant 1 after 8.2561 secs
None of these resulted in the bulb going unavailable.
Cheers,

That's good feedback, thanks! Let me know if any of them ever become unresponsive after a message like that for any noticeable period of time.


Implement Async EmrContainerOperator

Implement async versions for the following operator

[ ] EmrContainerOperator

Acceptance Criteria:

[ ] Unit Tests coverage in the PR (90% Code Coverage -- We will need to add CodeCov separately to measure code cov) with all of them passing
[ ] Example DAG using the async Operator that can be used to run Integration tests that are parametrized via Environment variables. Example - https://github.com/apache/airflow/blob/8a03a505e1df0f9de276038c5509135ac569a667/airflow/providers/google/cloud/example_dags/example_bigquery_to_gcs.py#L33-L35
[ ] Add proper docstrings for each of the methods and functions including Example DAG on how it should be used (populate
[ ] Exception Handling in case of errors
[ ] Improve the OSS Docs to make sure it covers the following:

[ ] Has an example DAG for the sync version
[ ] How to add a connection via Environment Variable & explain each of the fields. Example - https://airflow.apache.org/docs/apache-airflow-providers-postgres/stable/connections/postgres.html
[ ] How to use Guide for the Operator - example: https://airflow.apache.org/docs/apache-airflow-providers-postgres/stable/operators/postgres_operator_howto_guide.html

PR
https://github.com/astronomer/astronomer-providers/pull/234


Need clarification of "detection status"

Regarding Detection Status messages:
Using the word "Blocked" is misleading as it seems to by itself indicate we kept the malware from running.  But that is until you learn about "Prevented".  I think this article needs a chart that breaks these out like this (better wording of course):
Blocked - Process was running, we stopped it and blocked it from running again.
Prevented - File wasn't able to run because we prevented it.
Detected - We detected the process it could still be running (EDR in non-block mode)
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1814eead-2dcd-c8d6-96dd-b4ce2590fb08
Version Independent ID: df05098f-6d60-1b9d-c674-a964c5ac8d03
Content: Behavioral blocking and containment - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-atp/behavioral-blocking-containment.md
Product: w10
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

Thank you @jmmowrer for the feedback! We'll look into this.

Phrasing suggestion:

Blocked: A running process detected. This process has now been stopped and also blocked from running again.
Prevented: A detected file has been prevented from running.
Detected: A process has been detected as a security threat. The process might still be running (EDR in non-block mode).

(Alternate phrasing may be just as useful, feel free to pick and choose.)

@jmmowrer @illfated In my opinion, these 3 options "Blocked, prevented, detected" do not cover the full spectrum of actions that Defender for Endpoint does.
An automated EDR investigation can yield 3 verdicts: Malicious, Suspicious, or No threats found
and can results in one or more of the following remediation actions:

Quarantine a file
Remove a registry key
Kill a process
Stop a service
Disable a driver
Remove a scheduled task

More details here.
In my opinion, it's a matter of semantics in which of the 3 categories (Blocked,Prevented,Detected) you wish to put the above remediation actions (one or more of them).
@e0i Could we ask the author to check if an article update is needed in this case?

Good & fair point. Thank you for weighing in with your views. I agree. There are several important semantic choices to make.

@e0i : Did you close this ticket based on any recent changes or other work being pushed recently by any of the MS Docs team members?


Error encountered generating attestation

Hello!
I am trying to verify a device with another device, but as soon as I scan the QR code I get a red screen with the following error message:
Error encountered generating attestation:

invalid verified boot key fingerprint:
C51........................................78F

Any ideas?
Thanks!

The auditee is a Google Pixel 3 running Android 13 (DivestOS).

Thanks!
Just for the record:
I also tried to do it the other way around, use the Google Pixel 3 to audit the other phone, a Fairphone 5, but when I try to click "Auditee (device being verified) I get a small notification at the bottom saying:
Device is not one of the supported models.

The FP5 is currently running Android 14 - LineageOS.


[QIP]: Issues for Quality improvement plan

Issue/Feature Description:
The issue is an umbrella issue for Quality improvement plan :
Following are the identified issue

[ ] Split ci_check.sh script into multiple jobs (can run parallel)

maintain switch for conditional build for different jobs

release
respository
container

[ ] Reliability of all deployed services

Restart over reboot/crash

If deployed as process : use service (systemd)
if docker container : use restart always policy

[ ] Deployment of common services
Add strategy for common services (redis, kafka) deployment across different module.

Support installer support with docker for delfin module

Why this issue to fixed / feature is needed(give scenarios or use cases):
The issue handles quality improvement for the project
How to reproduce, in case of a bug:
Respective issues are explained in detail with tracking issues.
Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)

@joseph-v

@AmitRoushan I have created individual issues for tracking the following

Reliability of all deployed services

Restart over reboot/crash

* If deployed as process : use service (systemd)

Issue: #452

* if docker container : use restart always policy

Issue: #453


Helm Releases nav node

Description
Changes proposed in this pull request:

Introduce Helm Releases nav node, display only non-superseded ones
Fix Spinner in command palette, also add missing resources

Related issue(s)
Closes #852

/test pre-busola-smoke-test-docker

/retest

/test pre-busola-integration-test-k3s


Merge i184 to i188

require edits made in i184 in i188

merging i184 and 188 into a new issue.

ignore, outdated.


(ios) Apple's "advanced data protection" + seed backups

Apple recently added end-to-end encrypted data for iCloud via Advanced Data Protection:

When I first heard about this, I was under the impression that ALL iCloud data would be encrypted E2E (except mail, calendar & contacts). However, this is false. When enabled, all of Apple's own apps get E2E protection, but 3rd party apps ... nope! They have to opt-in themselves:

Advanced Data Protection also automatically protects CloudKit fields that third-party developers choose to mark as encrypted, and all CloudKit assets.

CloudKit Record fields must be explicitly declared as “encrypted” in the container’s schema to be protected, and reading and writing encrypted fields requires the use of dedicated APIs.

So with this PR, we are opting in.
Ramifications
Advanced data protection is not enabled by default. It's opt-in, and it's a relatively hidden setting. To enable it requires the user to setup either a "recovery key" or "recovery contact".
The recovery key option is similar to backing up a bitcoin seed. The user is given a 28-character hexadecimal string, and are told they're responsible for saving it.
The recovery contact option allows them to choose 1 or more trusted contacts:

A recovery contact can generate a code from their Apple device to help you get your data back.

The user can pick multiple options (e.g. recovery key plus multiple recovery contacts).
But throughout the setup process, the user is regularly reminded about data loss.

So if the user loses access to their Apple account, and they lose all their recovery options, then they've basically lost all their iCloud data (photos, files, notes, etc).
Apple can help the user regain access to their account, but only the non-encrypted data would be available. Such as email, calendar, contacts ... and unencrypted CloudKit records.
In my opinion, if a user explicitly enables "advanced data protection", they would probably assume that an iCloud backup of their recovery phrase (via Phoenix, or any other wallet) would be E2E encrypted. They might be wrong. But they would be excused for believing so, because most people do.
In fact, if you watch any tutorial on setting up Advanced Data Protection, most people say something like "all your iCloud data is E2E encrypted, except mail, calendar & contacts." (example) So the confusion is common.
For this reason (and others) I think the responsible choice is to opt-in to E2E encryption.
Technical notes
The record we're now backing up in the cloud looks like this:
{
  "name": "Optional backup name user can enter",
  "language": "en",
  "encrypted": {
    "mnemonics": "indoor fox shiver ..."
  }
}

In other words, in it's current state, this PR encrypts just the recovery phrase, and not the other metadata. Requesting feedback about this.

Fixes issue #342

but if we know from within Phoenix that this advanced option was enabled in iOS

There's a catch. Apple does not tell us whether or not "advanced data protection" is enabled. So Phoenix has no way of knowing. :slightly_frowning_face:
And also, if the user enables this option, Apple makes it really easy to disable (which simply uploads the private keys to Apple's servers). So if it's enabled today, it could theoretically be disabled tomorrow.

Apple does not tell us whether or not "advanced data protection" is enabled

Ok then we just need to add a short disclaimer about "Advanced data protection" in Settings > Recovery Phrase > iCloud backup, below the Your recovery passphrase will be stored... message, and I think we should be good.

Here's what I came up with:

So the user has to agree to 1 of the legal sections. And we use strikethrough to explain the implications of "advanced data protection".
The (i) button points the user to Apple's official page

Updated design:


Add mobile_robot files

mobile robot not complete yet

Please do not commit backup files and files with conflict marks.


fixed logging format, removed unnecessary quotation

relevant stuff for systemd's journal:
https://gist.github.com/GalaxyGorilla/added61676fc9620ae97

:+1:

I ran into some trouble here with the dependency resolution of hex. See https://github.com/elixir-lang/elixir/issues/3872. It seems the lager package has a flaw in the dependency specification. That's also the reason for the failing tests.

#80 fixed the goldrush dependency issue.


Modify pages to use the bp.openssf.org site, if we decide to switch to it

If we like the bp.openssf.org site & think it's ready, we should modify various pages to use it (e.g., link to it).

prefer to avoid an acronym, especially where overloaded and not common.  is an alternative possible?  e.g. best.openssf.org ?

We could do that.
That's not really the point of this issue, though. I suggest creating new issue so we can discuss naming as its own topic.
Until that new issue is created, let me quickly comment. Domain names are often best short so it's easy to se what it is. The WG discussion today accepted "bp.openssf.org" so that's what I set up. That said, it would be easy to change, and now is the time to change it (if we change).

will do -- I couldn't attend today, so was missing context.  thanks

The generated pages are now working at best.openssf.org, with https: (TLS) enabled. Also: if you prefer, you can omit the trailing ".html". You can try it out here:

https://best.openssf.org/Concise-Guide-for-Evaluating-Open-Source-Software
https://best.openssf.org/Concise-Guide-for-Developing-More-Secure-Software

These pages are using the draft OpenSSF font recommendations (those can be changed if desired).

I would like to modify https://openssf.org/resources/guides/ to use these new URLs. We can always improve the generated pages' looks, but they already look better than what we're doing now, and if we point people to the "right" URL then people will start using the new URLs.
Agree or disagree?

This is done.


Add Professional Institution of the city of Gaia "Escola Profissional de Gaia"

Escola Profissional de Gaia
The official URL: https://epg.pt/
The Page URL of the Programming Course: https://epg.pt/cursos-ensino-secundario/informatica-multimedia/curso-programador-informatica

@malicegoncalves47 lib/domains/epg.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@malicegoncalves47 First stage of verification passed, official domain is epg.pt as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.


Add a few extra Rope methods

Removed the call to the obselete Rope::LoadTextures method in World::AddRope in favour of

@crosire
Do you want UnloadTextures declared?
(BTW, do we need to mark them as obsolete? It's maybe useful if you want to edit existing ropes or need to load them for whatever reason? Although it's not possible yet ofc)

"Rope::LoadTextures" was going to be removed, so I don't see a pratical reason to ad "Rope::UnloadTextures"? @JohnnyCrazy The textures have to be in memory already if there are existing ropes you may want to modify ...

Removed Unload Textures now


CommandNotFoundError: Your shell has not been properly configured to use 'conda activate'.

I'm using hatch-conda to create envirnoment and run script as expect, but fail to enter shell.
Config is list as below
[tool.hatch.envs.default]
dependencies = [
  "coverage[toml]>=6.5",
  "pytest",
]
type="conda"
python="3.8"

Error is list as below
CommandNotFoundError: Your shell has not been properly configured to use 'conda activate'.
To initialize your shell, run

    $ conda init <SHELL_NAME>

Currently supported shells are:
  - bash
  - fish
  - tcsh
  - xonsh
  - zsh
  - powershell

See 'conda init --help' for more information and options.

IMPORTANT: You may need to close and restart your shell after running 'conda init'.

My OS is MacOS 12.6.4, Shell is Zsh with oh-my-zsh, conda is miniconda3

Same bug, seems like be fixed in master branch

Release 0.3.2 is now published with the fix on main.


Flake8 rule 'E999' reported an invalid point

Flake8 reported an invalid point for the rule E999, with the messge SyntaxError: EOF while scanning triple-quoted string literal.

Debug information:
Atom version: 1.15.0
Flake8 version: 3.0.4 (pycodestyle: 2.0.0, pyflakes: 1.2.3, mccabe: 0.5.2) CPython 2.7.12+ on Linux

Please update your flake8 version (pip install -U flake8). This particular error should be fixed in flake8>=3.2.1.

Thank you very much.
2017-03-23 18:50 GMT-03:00 Landon Abney<EMAIL_ADDRESS>
Please update your flake8 version (pip install -U flake8). This
particular error should be fixed in flake8>=3.2.1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/AtomLinter/linter-flake8/issues/375#issuecomment-288871536,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AP4MRawVEm3OzYpCLPPr_oktBxY9n3F5ks5rouk-gaJpZM4MnQdj
.


System time separator is used when Octo DeployAt argument is converted to string

What You Are Seeing?
System time separator '.' is used instead of ':' what is expected in the --deployat argument for Octo CreateRelease and Octo DeployRelease
What is Expected?
That the time separator ':' should always be used regardless of system settings.
What version of Cake are you using?
0.17.0-*
Are you running on a 32 or 64 bit system?
64-bit
What environment are you running on?  Windows? Linux? Mac?
Windows
Are you running on a CI Server?  If so, which one?
Nope
How Did You Get This To Happen? (Steps to Reproduce)
Switch to Finnish regional settings (In Windows 10, default Finnish regional settings will set time separator as '.') or switch Time Separator to '.' and run \build.ps1 and watch unit tests fail.
Output Log
Cake.Common.Tests.Unit.Tools.OctopusDeploy.OctoDeployReleaseTests+DeploymentArgumentBuilder.Should_Add_DeployAt_To_Arguments_If_Specified [FAIL]
      Assert.Equal() Failure
                                        (pos 151)
      Expected: ···loyat="2010-06-15 00:01"
      Actual:   ···loyat="2010-06-15 00.01"
                                        (pos 151)
      Stack Trace:
        src\Cake.Common.Tests\Unit\Tools\OctopusDeploy\OctoDeployReleaseTests.cs(408,0): at Cake.Common.Tests.Unit.Tools.OctopusDeploy.OctoDeployReleaseTests.DeploymentArgumentBuilder.Should_Add_DeployAt_To_Arguments_If_Specified()

Fix by switching:
Builder.AppendSwitchQuoted("--deployat", "=", _settings.DeployAt.Value.ToString("yyyy-MM-dd HH:mm"));

to
Builder.AppendSwitchQuoted("--deployat", "=", _settings.DeployAt.Value.ToString("yyyy-MM-dd HH:mm", CultureInfo.InvariantCulture));

@mholo65 this sounds like a very sensible change to me.  Are you in a position to create a PR for this?  If not, I can make the change.

I'm doing it now!


ValueError: Trying to use model to hydrogenate, but no model provided.

Getting this error when trying to do a simple unconstained generation
ValueError: Trying to use model to hydrogenate, but no model provided.
Please let me know how to resolve this issue.

Hi, can you give me more info on how you're trying to do the generation? I.e. are you hosting a local copy of ZnDraw and doing it interactively, or are you using the CLI? If you're using the CLI, could you please post your full bash command?


Add option for movie recording to start only on motion playback (not continuous)

This is a duplicate, so closing.


yarn build: ERR_FILE_NOT_FOUND due to loading from filesystem root

After running yarn build, the extension builds fine and seems to place everything where it should but for all my fonts and SVG files that have to be loaded in via CSS, I get the following error in the browser console:
index.html:52 GET file:///C:/assets/WebFontOrSVG.ext net::ERR_FILE_NOT_FOUND
This occurs for all assets that are linked inside of CSS files. The filepath for these assets becomes prefixed with the root of the filesystem, so the C:/ drive for Windows machines and / for MacOS.
This doesn't occur if I run yarn dev and load the extension in the development mode.
The reason for this is that in the final compiled CSS file inside of dist/js, all paths to external files are prefixed with '/assets/' when they should be prefixed with './'
Running a search and replace to replace all occurrences of '/assets/' with './' inside of the final merged and minified CSS file acts as a workaround for this issue.

Thanks for finding this. Resolved in https://github.com/hyperbrew/vite-cep-plugin/commit/862a166e392c0a9ead1e3bc65e1d0f61126bebfa


fix: light text color in about page, initial events page

Current dependencies on/for this PR:

main

PR #84   👈

This stack of pull requests is managed by Graphite.


Incorrect error when using destructuring multiple times in params

The following ES6 code will wrongly produce an error when parsed by esprima:
(function({A}, B, {C}) {'use strict';})
The error is:
Error: Line 1: Strict mode function may not have duplicate parameter names

An obscure bug indeed! I really appreciate the excellent reduced test case.

In fact, also (function([], []) {'use strict';}) and (function({}, {}) {'use strict';}).


Support Kind other than Deployment as a workload in k8s Pipeline Sync

What would you like to be added:
Support StatefulSet, CronJob, ...etc, as a workload in Kubernetes PipelineSync.
A document states that PipeCD supports Deployment as the only supported workload in the Kubernetes platform.
https://pipecd.dev/docs-v0.47.x/user-guide/configuration-reference/#kubernetesworkload
I tried to specify StatefulSet as a workload, and its deployment was planned as PipelineSync. So, StatefulSet seems supported.
So, the document seems wrong, or I missed some unsupported features.
Conversely, when I tried to specify CronJob as a workload, its deployment was planned as QuickSync. So, CronJob seems not supported.
This comes from the line below. CronJob's container spec is spec.jobTemplate.spec.template, not spec.template.
https://github.com/pipe-cd/pipecd/blob/b20163b70902d4f0f6698ff85f8ac427d61359f8/pkg/app/piped/planner/kubernetes/kubernetes.go#L260
Why is this needed:
We want to use WAIT_APPROVAL stage with these workloads.

Thanks @Warashi ! As mentioned in the docs, we want to support ReplicationController, DaemonSet, and StatefulSet in our plan. There is currently no CronJob on the list. Could you confirm the behavior of the current piped with the above planned Kind? If it all worked, then we can rename this issue as support CronJob as PipeCD managed workload
ref: https://pipecd.dev/docs-v0.47.x/user-guide/configuration-reference/#kubernetesworkload

I checked the behavior of ReplicationController, DaemonSet, and StatefulSet.
When I changed the image tags of these kinds of manifests, the sync strategy became PipelineSync.
However, I only checked the WAIT_APPROVAL stage and K8S_PRIMARY_ROLLOUT stage with example manifests in Kubernetes' official documents.
In perfect, I should check the behaviors of all stages, but I think it's enough to determine if PipelineSync is supported.


Incorrect text displayed back to users

Issue moved from MicrosoftDocs/feedback#2853

Please respond to @mniem.

From @mniem on Thursday, June 11, 2020 9:09:40 PM
Issue is on this  page.
Gets or sets <code>true to enable client affinity; false to stop sending session affinity cookies, which route client requests in the same session to the same instance. Default is true.

This seems to come from this line in the swagger: https://github.com/Azure/azure-rest-api-specs/blob/master/specification/web/resource-manager/Microsoft.Web/stable/2020-09-01/CommonDefinitions.json#L1812
And the generated code seems to not handle the 'code' tags very well.

This seems to come from this line in the swagger: https://github.com/Azure/azure-rest-api-specs/blob/master/specification/web/resource-manager/Microsoft.Web/stable/2020-09-01/CommonDefinitions.json#L1812
And the generated code seems to not handle the 'code' tags very well.

@rloutlaw How should the true/false values be formatted in the JSON?

@rloutlaw How should the true/false values be formatted in the JSON?

@AlexGhiondea weird, that file was copied directly from the previous version 2020-06-01, which also has the  tags

@AlexGhiondea weird, that file was copied directly from the previous version 2020-06-01, which also has the  tags

@calvinsID yeah ... I don't think that ever showed up correctly 🙁.

@calvinsID yeah ... I don't think that ever showed up correctly 🙁.

@gewarren Apologies for the late reply. We will update this thread once we have more details.
@calvinsID Could you please update this github thread once you get a chance ? Awaiting your reply.

Let me find the code owners in the backend repo that generates this, and tag them


Can not find the Point-wise Mesh Euclidean Distance measurement code

Hi, thanks for your good work, but we cannot find the protocols to measure the performance, especially the code of Point-wise Mesh Euclidean Distance for supervised and unsupervised (only list)
Could you provide one?

Hi,
we align the meshes first by letting all the points minus their own bbox centers and then use MSE loss to calculate the error of GT and result.

Hope this code helps you.

Hope this helps you.

Got it! Thank you so much for your help!

Hi, however, when I implemented your code to reproduce the results, the results I got are away worse than yours. The supervised and unsupervised results are similar, (0.2608 and 0.2526 with 72 subjects for each), could you figure out what is the problem with my code?
Here is the model I used:
from model import NPT
net_G=NPT()
net_G.cuda()
net_G.load_state_dict(torch.load('original_169.model'))
Here is the implemented code:

I tried different models, max_pooling and original model and I also trained them myself, but the results are more or less the same (around 0.25 which is far away from yours 0.00011), thank for you help!

Hi,
Thank you for your question.
I think there is a problem you missed. If you shuffle the identiy mesh, then the gt should also be shuffled in the same way, cause the output vertices is at the same order as the identity mesh. The output may looks like GT visually but has huge error in your code.
发件人<EMAIL_ADDRESS>发送时间: 2020年10月20日 16:43
收件人<EMAIL_ADDRESS>抄送: Jiashun<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [jiashunwang/Neural-Pose-Transfer] Can not find the Point-wise Mesh Euclidean Distance measurement code (#9)
Hi, however, when I implemented your code to reproduce the results, the results I got are away worse than yours. The supervised and unsupervised results are similar, (0.2608 and 0.2526 with 72 subjects for each), could you figure out what is the problem with my code?
Here is the model I used:
from model import NPT
net_G=NPT()
net_G.cuda()
net_G.load_state_dict(torch.load('original_169.model'))
Here is the implemented code:
[Screenshot from 2020-10-20 10-39-32]https://user-images.githubusercontent.com/17347668/96562201-c995f180-12c0-11eb-834e-564b424e8c8c.png
I tried different models, max_pooling and original model and I also trained them myself, but the results are more or less the same (around 0.25 which is far away from yours 0.00011), thank for you help!
―
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/jiashunwang/Neural-Pose-Transfer/issues/9#issuecomment-712693533, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMI6IKQ4GHNXSAQ7KAPVGXDSLVETRANCNFSM4SVF4LJQ.

Hi, Thank you for your question. I think there is a problem you missed. If you shuffle the identiy mesh, then the gt should also be shuffled in the same way, cause the output vertices is at the same order as the identity mesh. The output may looks like GT visually but has huge error in your code.

Thank you so much for your fast reply!  Exactly as you said, now I can make it close to your results.
WIth your pretrained max_pooling model, the supervised and unsupervised results are 0.000723 and 0.001367.
WIth your pretrained original model, the supervised and unsupervised results are 0.03597 and 0.03577079897669046.
It seems like there is still a problem of using original model. I only changed these:
from
from model_max_pool import NPT
net_G=NPT()
net_G.cuda()
net_G.load_state_dict(torch.load('maxpool.model'))

to
from model import NPT
net_G=NPT()
net_G.cuda()
net_G.load_state_dict(torch.load('original.model'))

Hi
I think the result we provide is in the situation which identity and pose meshes are in different orders. If they are the at same, I think for original_model, it may fail. So if you want to test original_model, I recommend you shuffle the mesh order first and then check the number.

Hi
I think the result we provide is in the situation which identity and pose meshes are in different orders. If they are at the same, I think for original_model, it may fail. So if you want to test original_model, I recommend you shuffle the mesh order first and then check the number. It is convenient if you just change the pose. But it also works if you shuffle all of them, but be careful for the identity and GT should be same.

Thank you so much for your help, problem solved! It's really a pleasure to discourse with you. Do you have WeChat, may you add me as friend? Mine: cc930315 :)


Match web's iframe behavior to native's WebView

Native's Webview follows this behavior: If height is not set (either explicitly or via expanded), its height will stretch to fit its content's height. We want web's iframe to follow this same direction.
Please test and confirm these 3 behaviors:

set explicit height. Scrollbar (if necessary) is within the iframe
use expanded with no height. Scrollbar within the iframe
no height, no expanded. Height should be as large as content. Scrollbar will be outside of the iframe.

okay @vusters

stale closing


Remove (Ruby) sass dependency and switch to sassc for Bourbon 6.0.0?

Hello. 👋
I was upgrading my dependencies on my project today and noticed that I'm still installing sass gem despite I already have sassc. After further inspection, it seems like that Bourbon specifies sass as a runtime dependency.
Given that Ruby sass is now deprecated and they suggested every one to move to sassc, I think it's best for Bourbon to stop specifying sass as a runtime dependency.
I think there're two ways we could approach this issue:

Switch to specify sassc as a runtime dependency.
Remove sass from a runtime dependency, and raise an error if neither sass nor sassc is found.

Please let me know what you preferred. I'd be happy to submit a patch as well.
Thank you.

Hi @sikachu!
Thanks for raising this issue…
I’m leaning towards removing the runtime dependency, because that more closely mimics how we handle Sass on the npm side (we do not list Sass as a dependency in package.json).
Would love a PR, if you’re up for that!

Sounds good. I'll submit the PR next week.

Thanks @sikachu! This has been merged as https://github.com/thoughtbot/bourbon/commit/9bedac008832cb779ded2f9298f12a5ebae9e88a.


Implement custom languages for meta schema

Exceptional PR for custom extensions and language names in meta.json

Merged without an approval.


add ANDROID_VERSION as ARG with 30 as default

As the build is failing due to invalid sysimage name, my solution could be to add a variable as ARG with 30 as default (is the more recent stable). That way should work.
Make me know if you have any thoughts about it or another idea.
I will be happy to discuss it.

LGTM 👍
I haven't noticed system-images;android-R;google_apis;x86 was replaced by system-images;android-R;google_apis;x86_64 recently (You can check it with sdkmanager --list | grep "system-images;android-R")


Predicate in repeat doesn't filter as expected

I have a repeat with a select question. I would like to make it so that each select option can only be selected once across repeat instances. I use the following predicate (XLSForm):
name = current()/. or not(selected(join(" ", /select-no-duplicates/repeat/option), name))
All repeat instances still show all options.
The first part of the predicate would be clearer as name = current() but Validate fails on that. I haven't been able to try the form with just current() and perhaps that would work in Enketo. As far as I know, current() and current()/. should be equivalent.

Ah, thank you!! I'm guessing this is caused by the workaround for this. I wish we could remove that workaround already. Enketo injects repeat positions in those absolute paths to repeat nodes (/select-no-duplicates/repeat/option), to 'correct' them.
I seems impossible to make a distinction between a reference to ${option} within the same instance (in an old XForm that use an absolute path) and a case like yours where you want a node-set of all ${option}s in all repeat instances. Does Collect have a clever way of doing that?

[ ] verify repeat-position-injection is the cause of this

I guess the repeat-injections could exclude references to its own path, but the bug would still exist for references to other repeat node.

I seems impossible to make a distinction between a reference to ${option} within the same repeat instance (in an old XForm that uses an absolute path) and a case like yours where you want a node-set of all ${option}s in all repeat instances.

I'll have to look at the JavaRosa code again but I believe it's based on usage. That is, in "true" XPath, /repeat/node refers to the node in the first repeat when a single node is requested and to all nodes from all repeats if a nodeset is requested. In JavaRosa, /repeat/node refers to the node in the current repeat when a single node is requested and to all repeats if a nodeset is requested.

Ah thanks, so this is done inside the evaluator in JavaRosa and in Enketo it is done before the evaluator comes into play.

Ahh, right, you don't have control over that kind of evaluator behavior, of course.
Is there a way we can gracefully switch clients over to the true intended behavior  (first node) when an absolute path is used to represent a single node? I can't really think of a way to detect user intent so it feels like we just have to wait until old forms are unlikely to be around to switch things over.

As a side note, current() and current()/. should both work there, right?

Oh gee, I didn't quite realize the wizardry you have going on in this form. I actually think I concluded in the past that it was impossible to do what you have found a way to do (at least in Collect). 👍

current() and current()/. should both work there, right?

I think you are right. Of course, it appears that Enketo doesn't support that yet.. I think this is bug number 4 or 5 you demonstrated with your 1-line form... 😩 It would be nice if you could occasionally share a form that demonstrated how well Enketo works :wink: (If such forms exists of course).

Is there a way we can gracefully switch clients over to the true intended behavior (first node) when an absolute path is used to represent a single node? I can't really think of a way to detect user intent so it feels like we just have to wait until old forms are unlikely to be around to switch things over.

Yes, I also think we'd have to wait. It might be worth putting some 'future' mode in Enketo's engine that anticipates this (but that probably can only be used in custom mini-ecosystems).

I actually think I concluded in the past that it was impossible to do what you have found a way to do

Don't worry, @yanokwa also said it was impossible. 😂

I think this is bug number 4 or 5 you demonstrated with your 1-line form... 😩 It would be nice if you could occasionally share a form that demonstrated how well Enketo works 😉

😅 It works great, I promise. I only get called in to write impossible forms.
I really wish we had a better way of sharing out these kinds of recipes. I have so many that I think are really cool. I'll be sure to point out others that run into Enketo trouble when I see it.

I only get called in to write impossible forms.

Haha. Well, you're really good at it.
I can sigh a little bit of a relief, because the easy workaround by bringing the repeat node ref outside the repeat, is working after all (I thought it wasn't but made a mistake in the calculation expression).  (live: https://odk.enke.to/preview?form=https://xlsform.opendatakit.org/downloads/5625gt__/select-no-duplicates-workaround.xml)

I really wish we had a better way of sharing out these kinds of recipes.

Yes, that would be a great thing to publish somewhere for those that wish to explore the nooks and crannies of XPath.

Haha. Well, you're really good at it.

😻

Yes, that would be a great thing to publish somewhere for those that wish to explore the nooks and crannies of XPath.

Any ideas on how to make them usable? I put most of my test forms at https://drive.google.com/drive/u/1/folders/0B3xDc_jQAXPwUjdyQ3JPWWRSbzg but without any kind of indexing scheme or metadata it's not particularly useful to others.

tests (manually add odk:generated-by attribute):

[ ] localhost:8005/?xform=repeatCount.xml
[ ] a version of the above form

@lognaturel Maybe the (future) website could have some advanced XPath tutorial section that lists these forms with a description.

changed to odk:xforms-version. We can just do a presence test or >= "1.0.0"
https://github.com/XLSForm/pyxform/pull/430

This requires copying model attributes in enketo-transformer unfortunately


Uploading code to microsoft repository master branch

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


fix: update required ibm provider version to < 1.7

/run pipeline

/run pipeline

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


Fix redundant percentages

https://github.com/upptime/upptime/blob/994e47412317f10d3ebc248d77a1007abbf95c71/history/summary.json

CLA Assistant Lite bot: Thanks for your submission! We ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by adding a comment below using this text:

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request

I have read the CLA Document and I hereby sign the CLA

This will be released shortly.

:tada: This PR is included in version 1.5.3 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Vector Store Retrieval Toolkit

fixes https://github.com/hwchase17/langchain/issues/1448
Adds a toolkit to interact with a vector DB to store and retrieve memories. I guess similar to https://github.com/openai/chatgpt-retrieval-plugin/tree/main/examples/memory?

Seems like it sometimes doesn't interpret the answer correctly, despite retrieving it correctly 🤔

Looks like this has been addressed in https://github.com/hwchase17/langchain/pull/2804


stage1: update coreos image to 1097.0.0

This is needed for a recent enough version of libseccomp (2.3.0), with support for
new syscalls (eg. getrandom).

Nice catch. LGTM on green.
Luca Bruno<EMAIL_ADDRESS>schrieb am Do., 7. Juli 2016, 19:36:

This is needed for a recent enough version of libseccomp (2.3.0), with
support for
new syscalls (eg. getrandom).
You can view, comment on, or merge this pull request online at:
https://github.com/coreos/rkt/pull/2884
Commit Summary

stage1: update coreos image to 1097.0.0

File Changes

M stage1/usr_from_coreos/coreos-common.mk
https://github.com/coreos/rkt/pull/2884/files#diff-0 (2)

Patch Links:

https://github.com/coreos/rkt/pull/2884.patch
https://github.com/coreos/rkt/pull/2884.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/coreos/rkt/pull/2884, or mute the thread
https://github.com/notifications/unsubscribe/ACewNzis9fSGQDSuxyreWiZE-nUV6OA2ks5qTTkmgaJpZM4JHUjp
.

It would be a good idea to re-push it as soon as jenkins is back to see if it introduces regressions.

Semaphore failed with
Files listed in /home/runner/rkt/builds/build-rkt-kvm/build-rkt-1.10.0+git/tmp/usr_from_kvm/cbu/manifest.txt are missing from /home/runner/rkt/builds/build-rkt-kvm/build-rkt-1.10.0+git/tmp/coreos-common/usr.squashfs:
lib64/libgcrypt.so.20.0.3
lib64/libseccomp.so.2.1.1

@s-urbaniak yes, I also need to double check arm64 now as they have different versions and we don't have any CI to cover us on that :(. I guess it will slip to tomorrow.

@lucab For now it should be enough that the travis arm64 build is ok.  That will detect if versions of existing dependencies have changed.
I don't think it would be too too difficult to setup a CI loop running rkt in qemu for arm64, but I guess it would take some work to get the actual rkt tests to support arm64.

For now it should be enough that the travis arm64 build is ok. That will detect if versions of existing dependencies have changed.

I'm not sure I've correctly understood this. My concern is that travis only builds [host, fly] flavors, so I don't see how it can spot changed libraries in coreos. For a real example, with this alpha update on aarch64 there is a libgcrypt.so.11.8.3 to libgcrypt.so.20.0.5 bump which I just spotted manually, and travis is not saying anything about that.

PTAL. It should be fine now on both amd64/arm64.

PTAL. It should be fine now on both amd64/arm64.

How did you check arm64?

How did you check arm64?

squashfs inspection and ldd on a private box. Not the whole testsuite, if you were asking about that.

squashfs inspection and ldd on a private box. Not the whole testsuite, if you were asking about that.

Yep, thanks!

LGTM on green.

@lucab perhaps I'm missing something dumb but shouldn't the libseccomp shared objects be there for arm too?

@jonboulle yes it should, but it looks like there is no libseccomp.so in current coreos release (not in the squashfs root). This also explains why libseccomp is currently not on the install list for arm64 stage1-coreos.


fix: Compact property not functional for menu widget

Description
This PR fixes the issue where compact property does not work for menu widget and menu type in table widget.
The CSS for compact property were getting overridden by some popover CSS
Fixes #13583
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manually tested

Checklist:

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

/ok-to-test sha=78b0e9e9

Tested this PR and working as expected on both Menu widget and Menu type on Table widget

/ok-to-test sha=78b0e9e9


Global job options that apply to all printers

On my system, I need to ensure that fitplot is always set to true across all printers, so that QubesOS trusted PDFs print correctly.  However, I was only able to figure out how to set it on a per-printer basis.

First, "fitplot" is the old option name. "print-scaling" is the current option (and provides more control).
As for global default options, there is currently no way to do this. The best you can do is to set this for every printer on your system.

Are global defaults planned?

@DemiMarie No.


test link to local file and future posts

Trying to include new blogposts

Hi @jaglima -- to create a new blog post, please create a new directory under content/report/osre23/nyu/noworkflow/ using a name consisting of the current date and the your username, e.g. 20230723-jesselima. The actual blog post is going to be in a new file called index.md. For efficiency you can just copy 20230524-jesselima/index.md and update it with new title, date, and content. Pretty much all the same as the first blog post, same with the picture.


adding cloudPKS to greeting

:star: PR built and available in a preview environment xgao3-webcast-demo-pr-2 here


NPE on player login

Doesn't happen on every player, but some players trigger a NPE from Quests when logging in:

[19:44:58 WARN]: java.lang.NullPointerException
[19:44:58 WARN]:        at me.blackvein.quests.Quester.getObjectives(Quester.java:530)
[19:44:58 WARN]:        at me.blackvein.quests.Quester.getObjectivesReal(Quester.java:379)
[19:44:58 WARN]:        at me.blackvein.quests.Quester.updateJournal(Quester.java:268)
[19:44:58 WARN]:        at me.blackvein.quests.Quester$4.put(Quester.java:193)
[19:44:58 WARN]:        at me.blackvein.quests.Quester$4.put(Quester.java:186)
[19:44:58 WARN]:        at me.blackvein.quests.Quester.hardDataPut(Quester.java:2565)
[19:44:58 WARN]:        at me.blackvein.quests.Quester.addEmptiesFor(Quester.java:1453)
[19:44:58 WARN]:        at me.blackvein.quests.Quester.loadData(Quester.java:1950)
[19:44:58 WARN]:        at me.blackvein.quests.PlayerListener.onPlayerJoin(PlayerListener.java:645)
[19:44:58 WARN]:        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
[19:44:58 WARN]:        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
[19:44:58 WARN]:        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
[19:44:58 WARN]:        at java.lang.reflect.Method.invoke(Method.java:498)
[19:44:58 WARN]:        at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:302)
[19:44:58 WARN]:        at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
[19:44:58 WARN]:        at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:499)
[19:44:58 WARN]:        at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:484)
[19:44:58 WARN]:        at net.minecraft.server.v1_12_R1.PlayerList.onPlayerJoin(PlayerList.java:346)
[19:44:58 WARN]:        at net.minecraft.server.v1_12_R1.PlayerList.a(PlayerList.java:166)
[19:44:58 WARN]:        at net.minecraft.server.v1_12_R1.LoginListener.b(LoginListener.java:159)
[19:44:58 WARN]:        at net.minecraft.server.v1_12_R1.LoginListener.e(LoginListener.java:57)
[19:44:58 WARN]:        at net.minecraft.server.v1_12_R1.NetworkManager.a(NetworkManager.java:233)
[19:44:58 WARN]:        at net.minecraft.server.v1_12_R1.ServerConnection.c(ServerConnection.java:140)
[19:44:58 WARN]:        at net.minecraft.server.v1_12_R1.MinecraftServer.D(MinecraftServer.java:842)
[19:44:58 WARN]:        at net.minecraft.server.v1_12_R1.DedicatedServer.D(DedicatedServer.java:405)
[19:44:58 WARN]:        at net.minecraft.server.v1_12_R1.MinecraftServer.C(MinecraftServer.java:678)
[19:44:58 WARN]:        at net.minecraft.server.v1_12_R1.MinecraftServer.run(MinecraftServer.java:576)
[19:44:58 WARN]:        at java.lang.Thread.run(Thread.java:748)

The error appears to reference code which is for Deliver Item quests. Have you noticed whether any of these players have accepted a particular delivery quest (you can check this in player data under /Quests/data/{player's UUID}). It'd be immensely helpful to see which of your quests is causing this.

Here's my Quests data folder:
http://loveyland.net/Downloads/QuestData.tar.gz
Let me know if you'd rather have me go through them, I just thought you might find the problem faster.  :)

Thanks, but without a quests.yml to compare quest names against, there's nothing much I can do with that.

Oops, sorry about that.  Try the link again, it should be the entire Quests directory now.

Sorry I wasn't able to get to finishing this issue. Chances are it has been resolved in recent builds of Quests. If you find that not to be the case, post the error generated by a recent version of Quests and I'll re-open the issue.


update packages

:white_check_mark: Build TheIdServer 2.3.1-fix-dependencies1-0031 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/5f8610ee48 by @)


Every effect "mix" parameter for consistency?

We need to figure out a common way to turn off effects.
Perhaps each effect's uniform should carry a "mix" component, which allows controlling how much of the input image is used vs. the post processed image.
That fixes the problem but also opens new doors for the user.

It seems that Bevy is moving toward (or already has?) two textures now, one for writing and one that can be read.
This is intended for post processing.
If we figure out more precisely how that works, we can probably use it for mixing.


Add slide prefetching on hover and on next / prev

@mikesherov @agaripian thank you so much for your time to review and help me through the challenging bits on this one! I've either fixed or answered all of the concerns you posted above. Please look over it one more time.

👍

good stuff here!!

👍

Great work!


Cannot upload dataset to datamart

Describe the bug
I defined a project in v2.4.0 and then upgraded to v2.4.0-Datamart. When I click on "Load to Datamart", I get the following error:

The dataset has an id, name, description and URL.
This error also appears in previous versions (tried v2.4.0 and v2.3.10-Datamart)
To Reproduce
Steps to reproduce the behavior:

Open a project
Click on 'Load to Datamart'
See error

Expected behavior
I would expect T2WML to register the dataset in Datamart.
Screenshots

Desktop (please complete the following information):

OS: Windows 10 Pro
Version: v2.4.0-Datamart

@dgarijo this happens when you delete a dataset after uploading an annotated spreadsheet to T2WML and before Loading to Datamart.
Could you confirm this? We should add a check before Loading to Datamart, so that it creates the dataset if it can

Closing this issue. There are errors in the tutorials when uploading to datamart, but regular datasets seem to work


task_wdt: Fix build warning on 64-bit platforms

When building for 64-bit platform (tested on native_posix_64), there was a cast warning for different sizes.
A void * is used to store a simple ID, fixed by using uintptr_t instead.

uintptr_t doesn't look right for this purpose, as the channel ID is not a pointer. Why not use size_t everywhere?

Not sure I agree, the documentation states:

Integer type capable of holding a value converted from a void pointer and then be converted back to that type with a value that compares equal to the original pointer.

Which is what is done here. I think this is a portable way without the need to use sizeof calls.