Understanding index for shards

Thanks for building zarrita! We've been using zarrita to write sharded V3 data as a test for new parsing capabilities for V3 data in zarr-js (cc @freeman-lab). However, we're having some difficulties understanding the structure of the index in the output from zarrita.
As an example, here is a sharded dataset:
import zarrita
import numpy as np

store = zarrita.LocalStore('testdata')

data = np.arange(0, 128 * 128, dtype='int32').reshape((128, 128))

shard_shape = (64,64)
chunk_shape=(32,32)

a = await zarrita.Array.create_async(
    store / 'zarrita-sharding',
    shape=data.shape,
    dtype='int32',
    chunk_shape=shard_shape,
    chunk_key_encoding=('v2', '.'),
    codecs=[
        zarrita.codecs.sharding_codec(
            chunk_shape=chunk_shape,
            codecs=[zarrita.codecs.blosc_codec(typesize=data.dtype.itemsize)]
        ),
    ],
)

a[:, :] = data

Based on https://zarr.dev/zeps/draft/ZEP0002.html#binary-shard-format, we'd expect the index as the last 64 bytes at the end of the file. However, reading those data gives unexpected results. Are you able to offer guidance for interpreting the shard index?
import os
from functools import reduce
shard = 'testdata/zarrita-sharding/0.0'
nbytes = os.path.getsize(shard)
nchunks_per_shard = int(reduce(lambda x, y: x*y, list(shard_shape)) / reduce(lambda x, y: x*y, list(chunk_shape)))
header_bytes=int(nchunks_per_shard)*16
remainder = nbytes - header_bytes

chunks = {}
with open(shard, 'rb') as f:
    _ = f.read(remainder)
    for ind in range(nchunks_per_shard):
        chunks[ind] = {}
        chunks[ind]['offset'] = int.from_bytes(f.read(8), byteorder='little')
        chunks[ind]['nbytes'] = int.from_bytes(f.read(8), byteorder='little')

print(chunks)

{0: {'offset':<PHONE_NUMBER>784, 'nbytes':<PHONE_NUMBER>784}, 1: {'offset':<PHONE_NUMBER>344, 'nbytes':<PHONE_NUMBER>0128}, 2: {'offset':<PHONE_NUMBER>144, 'nbytes':<PHONE_NUMBER>4272}, 3: {'offset':<PHONE_NUMBER>256, 'nbytes':<PHONE_NUMBER>1116578816}}

I've copied the equivalent code using Zarr-Python below in case it's helpful for understand the issue.
import os
from functools import reduce

os.environ["ZARR_V3_EXPERIMENTAL_API"] = "1"
os.environ["ZARR_V3_SHARDING"] = "1"

import numpy as np
import zarr
from zarr._storage.v3 import DirectoryStoreV3
from zarr._storage.v3_storage_transformers import ShardingStorageTransformer

shape=(128,128)
data = np.arange(0, 128 * 128, dtype='int32').reshape(shape)
chunk_shape=(32,32)
chunks_per_shard=(2,2)

path = "testdata/zarr-python"
sharded_store = DirectoryStoreV3(path)
sharding_transformer = ShardingStorageTransformer("indexed", chunks_per_shard=chunks_per_shard)

arr_sh = zarr.create(
    shape,
    chunks=chunk_shape,
    dtype=data.dtype,
    store=sharded_store,
    storage_transformers=[sharding_transformer],
)

arr_sh[:] = data

shard = "testdata/zarr-python/data/root/c1/1"
nbytes = os.path.getsize(shard)
header_bytes=reduce(lambda x, y: x*y, list(chunks_per_shard))*16
remainder = nbytes - header_bytes
nchunks_per_shard = reduce(lambda x, y: x*y, list(chunks_per_shard))

chunks = {}
with open(shard, 'rb') as f:
    _ = f.read(remainder)
    for ind in range(nchunks_per_shard):
        chunks[ind] = {}
        chunks[ind]['offset'] = int.from_bytes(f.read(8), byteorder='little')
        chunks[ind]['nbytes'] = int.from_bytes(f.read(8), byteorder='little')

print(chunks)

{0: {'offset': 1068, 'nbytes': 356}, 1: {'offset': 356, 'nbytes': 356}, 2: {'offset': 712, 'nbytes': 356}, 3: {'offset': 0, 'nbytes': 356}}

Hi @maxrjones,
a recent addition to the ZEP2 draft was to add a checksum to the end of the index: https://github.com/zarr-developers/zarr-specs/pull/237. Currently, that is a crc32c checksum. So, your index would be 64+4 bytes long. Does this resolve your issue?

I realize, we haven't updated the ZEP but only the spec. I'll male a PR to sync them up.

a recent addition to the ZEP2 draft was to add a checksum to the end of the index: https://github.com/zarr-developers/zarr-specs/pull/237. Currently, that is a crc32c checksum. So, your index would be 64+4 bytes long. Does this resolve your issue?

This makes sense, thank you!


AWS support

This Epic will track the work for Steeltoe components built to simplify the development process on AWS.

The core Steeltoe team does not have capacity to work on this effort for the foreseeable future. If this effort is important to you (yes you, the reader of this comment) or you can add some context or ideas around what it would look like, please open a new issue to restart the conversation.


Feature - change quote/brackets - with matching

I think it would be cool if by changing opening/closing bracket, matching bracket would be changed too e. g.:
test[x]  -  to:  test(x)
Could work on "" and other characters (maybe user defined).

Thank you for your suggestion.
I am wondering if it makes sense to add parenthesis manipulation functionality to dial.nvim now that there are many excellent plugins that specialize in parenthesis manipulation.
For example, vim-sandwich is my favorite plugin for these purposes. It's so versatile and can serve most purposes.
Is there anything that can only be accomplished with dial.nvim?

The surround plugins are not bad - except, if I do not want to override s key and I replace it to s - then replacing [] with () would be:
sr[(
Not bad but it takes 5 keys.
With dial it would only take ctrl+a.   Plus it would not break anything since, dial does nothing on ([" '  chars right now.  So it is like new extra feature - that does not have any cons IMO, but it would be cool to have.

Hi @monaqa the new bracket and quote feature works great.  It is cool that cursor does not have to be directly on bracket but can be inside of it, and brakcet cycling will work.
I noticed one issue thoug if we have quote inside brackets, like so:

Ten dial will ignore, that cursor is inside quote, and will always change brackets.  Thus in obove case it is impossible to change quote.

@JoseConseco Could I see your configurations?

Ok Here is its
use {  -- increse decrease numbers but also dates, true to false etc
		"monaqa/dial.nvim",
		config = function()
			local augend = require("dial.augend")
			require("dial.config").augends:register_group{
				-- default augends used when no group name is specified
				default = {
					augend.integer.alias.decimal,   -- nonnegative decimal number (0, 1, 2, 3, ...)
					augend.integer.alias.hex,       -- nonnegative hex number  (0x01, 0x1a1f, etc.)
					augend.date.alias["%Y/%m/%d"],  -- date (2022/02/19, etc.)
					augend.constant.alias.bool,
					augend.paren.alias.brackets,
					augend.paren.alias.quote,
					augend.constant.new{
				      elements = {"True", "False"},
				      word = true, -- if false, "sand" is incremented into "sor", "doctor" into "doctand", etc.
				      cyclic = true,  -- "or" is incremented into "and".
				    },
					augend.constant.new{
				      elements = {"==", "!="},
				      word = false, -- if false, "sand" is incremented into "sor", "doctor" into "doctand", etc.
				      cyclic = true,  -- "or" is incremented into "and".
				    },
				},

				-- augends used when group with name `mygroup` is specified
				mygroup = {
					augend.integer.alias.decimal,
					augend.constant.alias.bool,    -- boolean value (true <-> false)
					augend.date.alias["%m/%d/%Y"], -- date (02/19/2022, etc.)
				}
			}
			vim.api.nvim_set_keymap("n", "<C-a>", require("dial.map").inc_normal(), {noremap = true})
			vim.api.nvim_set_keymap("n", "<C-x>", require("dial.map").dec_normal(), {noremap = true})
			vim.api.nvim_set_keymap("v", "<C-a>", require("dial.map").inc_visual(), {noremap = true})
			vim.api.nvim_set_keymap("v", "<C-x>", require("dial.map").dec_visual(), {noremap = true})
			vim.api.nvim_set_keymap("v", "g<C-a>", require("dial.map").inc_gvisual(), {noremap = true})
			vim.api.nvim_set_keymap("v", "g<C-x>", require("dial.map").dec_gvisual(), {noremap = true})
		end,
	}

@JoseConseco  Thank you!
It seems an intended behavior.
dial.nvim allows multiple augend rules to be assigned to a single group, and then it is not obvious which rule to use when <C-a> / <C-x> is actually pressed.
Therefore, the order of precedence is determined based on the range of actual augends. If there are multiple augends under the cursor, the one with the wider range is given higher priority.
In fact, this specification does work well in most cases. For instance, when the cursor is at 3 in 2022/03/12, the date 2022/03/12 can be incremented or decremented in preference to the number 03.
In your configuration, both augend.paren.alias.brackets and augend.paren.alias.quote are in one group. These ranges are as follows:
  bl_options = {"REGISTER", "UNDO"}
               ~~~~~~~~~~~~~~~~~~~~     <- range of augend 'brackets'
                ~~~~~~~~~~              <- range of augend 'quote'

Because 'brackets' has a larger range than 'quote', 'brackets' always takes precedence over 'quote'.
A possible workaround is to create a group dedicated to augend.paren.alias.brackets and augend.paren.alias.quote, respectively.
require("dial.config").augends:register_group{
  default = {
    ...
  },
  quote = {
    augend.paren.alias.quote,
  },
  bra = {
    augend.paren.alias.brackets,
  },
}

In this way, it is possible to "switch only quotes" by typing "quote<CR><C-a>.

Ok, I did not know that outer scope is more important than inner one (it kind of feels counter-intuitive, but makes sense in date example). I thought I will be able to use same function/mapping for all augend types.  Using different group is ok, but then it takes more typing than vim sandwich (yep I know I can create new mapping - but its not as elegant ).
Is there any chance for you implementing special case behavior - for brackets, and quote - where inner scope  will be more important than outer, without breaking the plugin?

There are currently no plans to change or extend the behavior of the prioritization algorithm. This is because we rarely have trouble with numerical increments or decrements, and we cannot think of a simpler algorithm.
Simple values such as numbers or dates are the main focus of dial.nvim. In contrast, complex objects such as parentheses are not the primary focus. It is not desirable to complicate the overall structure and configuration just for the sake of parentheses manipulation.
To deal with it well within the current framework of dial.nvim, you can define an augend that combines both brackets and quote:
  default = {
    ...,
    augend.paren.new {
        patterns = { {"(", ")"}, {"[", "]"}, {"{", "}"}, {"'", "'"}, {'"', '"'} },
        nested = true,
        cyclic = false,
    },
  }

If you want more stable and versatile functionality, I still recommend a parentheses-specific plugin.

Ok, thanks. Above solution is best workaround. I will close the issue.

@monaqa I think I found perfect workaround after all:
augend.paren.new {
	patterns = { {"(", ")"}, {"[", "]"}, {"{", "}"}, {"(", ")"}, {"'", "'"}, {'"', '"'}, {"'", "'"} },
		nested = true,
		cyclic = false,
	},

Adding (,) pair twice - forces dial to go cyclic - (, [, {, ( and so on without reaching quotes. Similar trick can be used for ' and ".


Windows 32 bit: No android APPS are running (error:This app is incompatible with your device.check to make sure other android apps work for you from the chrome web store)

Hello vladikoff.
First of  all thanks for creating great software.But I am unable to use it.I dont know whether its my mistake or any system mistake.
I have used the archon custom runtime 1.2-x86_32 as shown below   :

and then tried to launch the whats app which is converted using twerk(to be compatible to run with archon)

and got the following result :

I have also tried other sample apps given in your page :
Got the same results stating as below in the image

I have also tried to include the Python file in the ARchon file and run it again(as given in  : https://github.com/vladikoff/chromeos-apk/issues/38) but it doesnt solve the problem.

I have also tried to install apps through ARC Welder (downloaded from the store) but it gives me a whole set of new problem.The extension doesnt even starts as shown below :

Some time it starts but doesnt shows me the option to include apk file and convert it.(sorry I couldnt get the snap).

Of the above ways i have tried,I couldnt get any app running.
Can you please help me run Apps on my chrome. I specially need whatsapp running.(please)
My sys config ::

And in the other versions of Archon custom runtime,when i try to run the apps I simply get a window with android symbol displayed and it stays as it is till I cancel it.

I really despise the fact that there isn't a response.

Bump


Add Oculus Platform SDK bindings

https://developer.oculus.com/documentation/platform/latest/concepts/developer-intro/

This seems very out of scope for lwjgl.

The Oculus Platform is designed to provide you with the infrastructure you need, so you can focus on creating a great game or experience. The Oculus Platform currently provides the following features:
Friend Management—to enable social functionality and increase engagement, this enables players to add friends and find friends that are currently online.
Rooms—your app can create rooms where players gather or you can enable players to create them.
Achievements—you can create trophies, badges, awards, medals, or challenges in your app and Oculus will manage it.
Leaderboards—if your app or experience is competitive, you can create leaderboards to track the top players.
Matchmaking—if players can't find a match, you can help them find available games. The platform supports both standard and skill-based pools.
In-App Purchases—enables you to monetize your games and experiences.
Peer-to-Peer Networking—enables players to communicate directly.

I agree with @huhlig about making this a subproject. I think that if LWJGL starts including a lot of bindings then #100 should become a much higher priority.

I wouldn't mind having libraries like this though #100 should really be finished first and maybe with it add a set of libraries that are less used and have less work out into them and are not part of the main jar but can be added if they are wanted by the user.


Problem adding influxdata repository to ubuntu apt sources

I followed the instructions on https://docs.influxdata.com/telegraf/v1.14/introduction/installation/ to install telegraf on Ubuntu.
My Ubuntu version:
$ cat /etc/os-release
NAME="Ubuntu"
VERSION="20.04 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04 LTS"
VERSION_ID="20.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal

When running
$ echo "deb https://repos.influxdata.com/${DISTRIB_ID,,} ${DISTRIB_CODENAME} stable" | sudo tee /etc/apt/sources.list.d/influxdb.list

as per the instructions I get a  bad substitution error and
echo ${DISTRIB_ID}

gives Ubuntu with a upper-case U.
Only when adding the repo with
$  echo "deb https://repos.influxdata.com/ubuntu focal stable" | sudo tee /etc/apt/sources.list.d/influxdb.list

by "hand" did
$ sudo apt-get update && sudo apt-get install telegraf

run without errors.
I see how this is a minor thing but as a not so experienced Linux user this took me a bit to solve.
Possibly related issue (where I have been told to open a new one) https://github.com/influxdata/telegraf/issues/7422

Thanks, @mwThinks! We'll look into this—

I was able to run the provided Ubuntu installation instructions successfully on Ubuntu 18.04. We definitely want to make sure as many Ubuntu versions as possible are supported, though!
@mwThinks: could you confirm the output of
$ echo ${DISTRIB_ID,,}

on your machine? The commas use Bash parameter expansion to lowercase the distribution name.

Sorry for the late response. So I tested it again. All of the commands below just give me an empty line
$ echo ${DISTRIB_ID,,}
$ echo ${DISTRIB_ID}
$ echo ${DISTRIB_CODENAME}

Some additional information, I am using a server version of Ubuntu 20.04 https://ubuntu.com/download/server.

The current install instructions for Ubuntu use the /etc/lsb-release, which has somewhat fallen out of favor in recent years. I suggest we update the instruction to make use of /etc/os-release.  Here are the /etc/os-release contents for our other supported Ubuntu versions:
NAME="Ubuntu"
VERSION="14.04.5 LTS, Trusty Tahr"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 14.04.5 LTS"
VERSION_ID="14.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"

NAME="Ubuntu"
VERSION="16.04.6 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.6 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial

NAME="Ubuntu"
VERSION="18.04 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

Closing this. Let us know if problems persist...


Validation requirements document

Create a document that details the validation requirements for the submissions.
In order to ensure a coherent and consistent data corpus, submission validation is required for metadata, files, and the experimental graph.
This is a companion document to the wrangling requirements document.
Validation Requirements

This is a supporting document for prototype issue #81.

LGTM!


Default type parameters

For some functions, like prompt, it makes sense to default to a particular type if one cannot be determined. For prompt, this would be Text — that way beginners don't have to know how type annotations work to start using prompt.
The syntax for default type parameters would be as follows:
prompt :: (A : Text) where (Read A) => Text -> A

We can also get rid of terminating type variables and have crash and the like default to ().

Implemented in 6507c99fa43d13810acf907e36b4f0171169d219!


Edge case where arrow-parens incorrectly complains and fixer produces invalid output

Bug Report

TSLint version: 4.4.2
TypeScript version: 2.1.5
Running TSLint via: CLI

TypeScript code being linted
angular
  .module('app')
  .filter('myFilter', function () {
    return (items): Items[] => {
      // ...

      return items;
    };
  });

with tslint.json configuration:
{
  "extends": "tslint:recommended",
  "rules": {
    "arrow-parens": [true, "ban-single-arg-parens"]
  }
}

Actual behavior
(items) is shown as invalid, and the fixer outputs the following invalid code:
angular
  .module('app')
  .filter('myFilter', function () {
    return items; : Items[]; => {
      // ...

      return items;
    };
  });

Expected behavior
This should not error or produce bad output, as the parens are necessary here

That should be fixed in a newer version of tslint. Testing this locally with v5.2.0 works correct.


Report: The population of people, people living in cities, and people not living in cities in each country

A report containing the following columns
The name of the continent/region/country.
The total population of the continent/region/country.
The total population of the continent/region/country living in cities (including a %).
The total population of the continent/region/country not living in cities (including a %).

provide a Report on The population of people, people living in cities, and people not living in cities in each country


Upgrading dependencies causes smokey to fail

I've forked this project. My fork can be found here.
I've tried to limit myself to upgrading the Scala 3 version to the LTS version, and upgrading the core dependencies to their latest versions supporting Scala 3. In terms of source code changes, this has been completely trivial.
However, when I stand everything up and run sbt smoke, the test fails due to a websocket connection timing out, and several of the expected messages not being received.
Doing the same with the original code passes using the same JVM on the same host.
Looking at the evolving dependencies, my biggest concern is the changes to the http4s JDK client used for websocket support. It appears to have undergone significant revision from M6 to M7, revolving around new cats-effect capabilities. However, neither reverting to M6 (the last version in which simple returns a Resource) and M3 (the version used in the original code) solves the problem. Nevertheless, the finicky-looking nature of the websocket client with respect to state and lifecycles continues to make me nervous.
Any suggestions would be greatly appreciated.

Hi @paul-snively 👋🏽
I'll have a look, that looks a lot like a bug in http4s. I had pending upgrading all dependencies anyway #289

I download the patch for #289 and applied it to my fork of your main branch, and sure enough, the smoke tests now pass.
Do you have some information about what the bug was, and where/how it was resolved?
Thanks so much!

@paul-snively I haven't upgraded http4s yet, so that's more likely where the bug resides. I'll have to investigate further...

@paul-snively looks like #290 fixed it. Let me know if that works for you.

Yep, patch applies cleanly on top of #289 applied to main, and sbt smoke passes.
Thanks so much!


Adding EVENT on a trigger

Please use this Pull request !
Hi, my change will allow EVENT trigger type -
On https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/glue_trigger please if you check the type; there is also type EVENT – Valid values are CONDITIONAL, EVENT, ON_DEMAND, and SCHEDULED.
what

This change will allow trigget to be kicked by the event bridge

why
*I need an EventBridge kicked trigger to kick the workflow when S3 folder is changed..
I can use normal https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/glue_trigger, but I would like to use cloudposse one ;)
references

https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/glue_trigger#type

/test all

/test all


Use dB as default for Bode plot?

Roberto Bucher suggests:

"bode" and "bode_plot": in my opinion the flag dB=True should be the
default choice (but this is only my opinion...)

This is obviously a matter of choice, but I agree that dB is the most commonly used convention.  Karl Astrom and I chose not to use dB in our book since it often wasn't part of a students vocabulary in areas outside of electrical engineering (eg, biology or economics).  The function use_matlab_defaults will set the default to use dB for Bode plots, if that helps.
There is stub code (currently commented out) in control/matlab.py that will call use_matlab_defaults and so using the matlab module would give plots compatible with matlab.

This has been fixed as of commit ebf83643d67126102caea9c0fd6940f6a4c8cfac as part of PR #62.  If you are using the MATLAB subpackage, the default units for Bode plots are in dB.


Added CN Locator endpoint

Added CN locator endpoint
Moved ServerRegion from SpatialGDKEditorSettings to SpatialGDKSettings

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.

Description
Locator endpoints can now follow the Server Region setting.
Release note
Updated.
Tests
Change the Region Settings in SpatialOS GDK for Unreal > Runtime Settings to CN and see if it connects to the CN environment.
Documentation
N/A

@joshuahuburn @jayprobable

I have created https://github.com/spatialos/UnrealGDK/pull/1652 and https://github.com/spatialos/UnrealGDK/pull/1653 to target preview and master, respectively, as I assume this needs to go into all of them. Let me know if that's the case.

Closing in favor of https://github.com/spatialos/UnrealGDK/pull/1678


What would a generic train operation look like?

It might be worth investigating a way to have a generic train operation. Not sure whether it should be there. But a generic train operation can be easily added for keras library. If to be added where should be added and what route should we take?

It sounds like there are a few questions here so I am going to try to answer each directly (let me know if I miss anything!):
Should a generic training operation be added?
I think it would be really valuable to add. A generic training operation would make it much easier for people to get started quickly.
Where should it be added?
Good question. Ideally, we would have some publicly available libraries that are able to be imported from within DeepForge (similar to importing extensions). WebGME has a mechanism for importing libraries but I do not know of any mechanism for adding these libraries to a registry to make them discoverable. This would be a good discussion to start with the WebGME community as it may be of interest to WebGME itself. If not, we can add this registry as a router in DeepForge. There are other considerations that would need to be discussed like private vs public libraries (ie, sharing between a single user's projects).
What route should we take?
There are two separate fronts to work on: creating the operations to include in the library and the library infrastructure to aid in discovery and distribution.
The first front is going to be the focus of the summer intern (not just with a training operation but it would be good to include). I would recommend starting with the existing TrainValidate operation from the redshift example then removing the code specific to the given example such as:
        X = dataset['X']
        y = dataset['y']
        y_cats = self.to_categorical(y)  # This is the most concerning
        # ... generic training code

    def to_categorical(self, y, max_y=0.4, num_possible_classes=32):
        one_step = max_y / num_possible_classes
        y_cats = []
        for values in y:
            y_cats.append(int(values[0] / one_step))
        return y_cats

and parameterizing the operation by more of the generic parameters such as optimizer, loss and metrics. These would probably be best suited as an enum on the operation. This will also require some modifications of things like the generated plots (which should certainly exist). Finally, the rest of the examples should be updated to use this standard Train operation. This would also serve as a nice demonstration of the general nature of the operation.
On the second front, we should first figure out if it should be added in DeepForge or contributed back to WebGME. It seems pretty generic so I suspect it would be a good candidate for contributing back. We will need to make sure we address a few questions/considerations:

public vs private/personal libraries
library dependencies. Libraries may require specific DeepForge extensions to be available (such as DeepForge-Keras) whereas others may be generic and may not require any extensions.
categories/labels for libraries. This may not be necessary initially but it would be useful in the future to be able to sort by things like "image processing", "astronomy", etc.

This would actually be better to implement as an interactive editor (#1808) as this will let the user get feedback immediately, use a more sophisticated UI (especially for customizing optimizers, etc) and will still generate a pipeline at the end of the day.
Closing this in favor of #1816.


Add reset button to user input

In some pages like GapsPage and TimelinePage, I think it will be a better user experience to add a reset button. When the user would like to change the selected bus number he/she will click this button and the inputs will be removed/change to the default input. What do you think about this? @ShayAdler @NoamGaash

I think it's a good idea - but what the default value should be? empty state isn't very useful

About the date, the default value will be today. Why is empty state not so useful? 🤔 I am not sure about the default values for the rest of the inputs.

I think we shouldn't be opinionated regarding default values for fields like "operator" (מפעיל)

I think so as well. In my opinion, we can reset the values to empty state , and only for the date reset it to today. However,  you said that empty state is not useful. I am not sure what we should do. What do you think?

Let's think of the problem, before jumping into solutions..
What's the use case for such a button? How does it help to the final user experience?
Did you suggested this idea because you considered a use case of a user that wants to change the operator they are searching, and feels frustration due to the need to delete the content of the input element when focused?

Yes, the user needs not only to change the operator but also the line number and date from the last search, so I think it is easier to remove those inputs and select/write new ones from blank.

Hi, I noticed something, in some input fields we have a "clear" X button in the input field , for example in "OperatorSelector" "RouteSelector" like in the image

Instead of making a separate button to reset values we can add this functionality in/beside the "LineNumberSelector" and "MinuteSelector" input fields

What do you think? @NoamGaash

I think it's a much better idea, thanks!


Introduce SentenceTransformer Reranker

Description
This PR introduces support for document reranking, specifically leveraging SentenceTransformer cross-encoders. The addition aims to provide a lightweight and optimized approach to reranking, enhancing the model's response quality and speed.
Motivation
The integration of a reranking feature addresses the need for more relevant and accurate responses by pre-filtering documents before answer generation. The choice of SentenceTransformer's cross-encoder as the reranker is motivated by its efficiency and effectiveness in identifying the most relevant documents, compared to traditional LLM-based reranking methods.
Changes Made

Added an optional SentenceTransformerRerank node_postprocessor to ChatService, which facilitates the reranking process using the SentenceTransformer cross-encoder.
Updated settings to include rerank-specific configurations, allowing users to enable and customize reranking according to their specific needs. This includes parameters such as model and top_n, where the latter controls the selection process of documents for final response generation.
Documentation updates to guide users through enabling reranking, installing necessary dependencies (poetry install --extras rerank-sentence-transformers), and configuring rerank settings effectively for optimal performance.

The reranking feature is disabled by default to accommodate the additional dependencies and the need for users to adjust configurations based on their unique use cases.
Future Considerations
Looking ahead, there's potential to further refine the reranking functionality by:

Creating a dedicated reranker component: This would facilitate the incorporation of different reranker types, enhancing flexibility and customization for users.
Expanding reranker support: Exploring and integrating additional reranker methods beyond the SentenceTransformer cross-encoder could provide users with more options to tailor the reranking process to their specific requirements.

Does llamacpp-python need to be rebuilt for gpu? I would assume so. I've offloaded all of my gpu stuff, so I might have to update my pipeline if so @machatschek

Does llamacpp-python need to be rebuilt for gpu? I would assume so. I've offloaded all of my gpu stuff, so I might have to update my pipeline if so @machatschek

The SentenceTransformer reranker does not rely on llamacpp-python. If you want to run the reranker on GPU, you would need to install GPU-enabled PyTorch version. SentenceTransfromer will then use GPU by default if one is available. If you want to overwrite this behaviour, you can set the device parameter in SentenceTransformerRerank to "cpu".


Workspace - WS can't be deleted after Expensify cards was disabled

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v9.0.48-0
Reproducible in staging?: Y
Reproducible in production?: Y
Issue reported by: Applause Internal Team
Action Performed:
Precondition: new private domain account. Workspace with added expensify card (physical or virtual).

Go to Workspace > Expensify cards
Click on added card > Deactivate card
Go to Workspace > More features
Disable Expensify cards
Delete workspace

Expected Result:
Workspace deleted.
Actual Result:
An error occur during deletion. It says that my ws has expensify card when actually it hasn't.
Error Message: Your company has enabled Expensify Cards. Please reach out to [concierge<EMAIL_ADDRESS>to have the cards removed. Until then, the Workspace cannot be deleted.
Workaround:
Unknown
Platforms:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/81077181-f12a-4500-be68-f768e6ae6953

View all open jobs on GitHub

We think that this bug might be related to #wave-collect - Release 2

@joekaufmanexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Not overdue. Will triage tomorrow

Discussing here.

We would like the modal to looks like this

@mountiny the more I think about it, I kinda wonder if this one is worth prioritizing right now. Deleting a workspace is in general such a rare action to take. Given this works now the same way it does in OldDot (which is totally fine), I wonder if it's worth just leaving as is for now until we experience a problem from it?
As this isn't really a bug now, and I wonder if it's worth investing resources in changing this design before really anyone is using this feature.

@joekaufmanexpensify I think its still just good to knock it out now, I think its a rare case, but when it happens its annoying, especially given its much more easier now to run into this case than then we required verified domain

But I think we can make it weekly and maybe @nkuoch would be up for helping with this one

Fair enough, sounds good. In addition to the deleting workspace case though, there's one other case we should align on. According to the design doc, the user should not be able to toggle off Expensify Card in More features when the workspace is provisioned at all. But in the above video, toggling this off is possible after terminating the workspace's only assigned card.
I'm thinking we intentionally allowed this when the workspace is provisioned (but there are no active cards). Is that right, @mountiny?

PRs in review!

Seems like toggling off doesn't actually do anything backend side? https://github.com/Expensify/Web-Expensify/pull/28072/files#diff-fd75745654fb72974da44d75c3835ee837a6e644a05a6ffaae48556924242946R877
cc @mountiny do you know if an issue currently exists for that?

I am not sure how that PR is related. This is the method used to toggle the expensify card feature https://github.com/Expensify/Web-Expensify/blob/fe926b0bc18fb7496fbc31280f092c5d5240ba7c/lib/PolicyAPI.php#L3395-L3411 and it changes the flag, but I dont think it does much more right now

I'm thinking we intentionally allowed this when the workspace is provisioned (but there are no active cards). Is that right, @mountiny?

And should we deprovision it when toggling it off when there is no active card? cc @mountiny ?

And should we deprovision it when toggling it off when there is no active card? cc @mountiny ?

Asked here

Still ongoing

Same

PR still in review

@nkuoch I see PR above was deployed. Are we good here now?

Asked for a retest here


Azure pipeline check

SourceCode/2024/Basics-April2024/anil/Task26 - Copy/project.json
Code Review for Main.xaml
The workflow looks good overall, but there are some improvements that can be made.

Log Message: It's great that you're using Log Messages! However, none of them are actually being used in the workflow. Please make sure to utilize these messages during development and after release.
Message Box display name: This is defined multiple times (5 instances). To avoid confusion, please give each activity a unique name that clearly indicates its purpose.

Activity State Machine
The default name for this activity should be more descriptive. Give it a title that explains what the state machine does.
Activities with Default Names
Many activities have default names. Please rename them to something more meaningful, such as:
+ Activity Assign: "Set Variable"
+ Activity Message Box: "Display Alert"
+ Activity Input Dialog: "Get User Input"
These suggestions should help make your code easier to understand and maintain.
Score: 4/5
The score is affected by the need for more descriptive activity names and the unused Log Messages.

SourceCode/2024/Basics-April2024/anil/Task26 - Copy/project.json
Here's my review:
Main.xaml

Log Message not used in workflow: Consider using log messages to help with debugging and understanding the flow path of your workflow.
Many duplicate message box display names: Please rename them to be more descriptive and unique.

Naming Conventions

Names of activities should give a clear indication of how they are being used. Default names can be too generic. Try to avoid duplicates.
For example, Activity Message Box could become ShowErrorMessageBox or something similar.

Code Review Score: 3/5
I'd like to see more descriptive names for the activities and message boxes. Additionally, consider using log messages to help with debugging and understanding the flow path of your workflow.


Create code object tab as module global with indices

This currently looks to only save a tiny amount of space so may not actually be worth it. Therefore I'm marking it as draft.
The one useful thing it does do is to restore some error origin information that was lost from the code object generation. However we could cherry-pick it anyway.
It also might make it easier to pack the structure better and so save some actual space.

We shouldn't be using prebuilt tuples for the argument names. They are really only used once and should not take up any space in the module state.

We shouldn't be using prebuilt tuples for the argument names. They are really only used once and should not take up any space in the module state.

Not convinced this is necessarily always right.
(self, ) is likely pretty common. (self, env), (self, code), and (self, node) will occur loads of times in the Cython code-base.
However - given many are only used once - I actually wonder if we should be reverting the whole "code object table" change? It made a pretty minor size difference anyway, and it makes it harder to be flexible about things like "mixture of prebuilt and non-prebuild tuples". (We should keep the cached code objects stored in an array on the module state rather than many individual constants though)

I didn't suggest to stop deduplicating these tuples, just that they should not end up in the module state. We don't need a permanent reference to them after the module init phase. And for the few that we still happen to need later, it's probably fine to use a copy.
But yes, of it doesn't pay off, there's no use in throwing more complexity at them.


M1 UX: Spike for C&I team to ramp up on existing Secrets Provider UT, IT, and E2E infrastructure

Is your feature request related to a problem? Please describe.
As the C&I team starts to do development on the M1 Push-to-File feature, we need someone to ramp up on what the current available test infrastructure is available for testing Secrets provider. The goal is to have an idea as to how we can quickly add tests for the M1 Push-to-file feature.
We might want to time-box this to limit the scope of time spent.
Describe the solution you would like

Any gaps in UT in the area of SP container configuration, e.g. tests for defaults and invalid values
Are there any Golang mock examples for retrieving secrets from Conjur
What Docker-compose based test infrastructure is available?
What E2E, holistic testing is available, and for what platforms (e.g. KinD, GKE, OpenShift)? (for future reference)
A clear and concise description of what the desired end result(s) would be.
Report results to team, perhaps a quick Gist.

Describe alternatives you have considered
Additional context

Closing, since this is already covered by Issue #306


Spyder terminal keeps running, even if Spyder is shutdown.

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
First of all, thanks for including the spyder-terminal in the standalone installers now as well!
However, it seems the underlying server process is not being shut down after Spyder quits. After a few Spyder restarts this leads to multiple python processes listening on different ports at <IP_ADDRESS> all serving the OS Web Terminal.
This is not only a minor nuisance, but I think can lead to a security incident.
If someone exposes a port(range) to the web while the OS Web Terminal is still running on that port (without any indication, unless you check the ports with lsof), I think someone might expose a terminal to the WWW, which is undesirable.
What steps reproduce the problem?

In a terminal run sudo lsof -i -P | grep LISTEN | grep python to see al listening python processes.
Open Spyder
Run sudo lsof -i -P | grep LISTEN | grep python to see al listening python processes.
Close Spyder
Run sudo lsof -i -P | grep LISTEN | grep python to see al listening python processes.

What is the expected output? What do you see instead?
Expected output
The list of processes retrieved in step 5 should match the list from step 1. I.e. all python processes started by Spyder should be shut down after closing Spyder.
Actual output
A new python process is found in step 5, which was also visible in step 3. If you visit the address that python process listens on (<IP_ADDRESS>:<some_port>) you will get the OS Web Terminal, a live terminal into your system.


Integration tests should be run on Glassfish 4

Integration tests should be run on Glassfish 4

Not enough time to work on this.


🛠 [TASK] : Recovered Account Details

Part of #821

Needs backend integration

Closing this in favour of #1000


Update prototype to run captive on online ingestion mode.

Just passing the config allow us to start using captive on online mode. It seems to work fine but it's very slow.

LGTM! Let's create captive-core-online branch from captive-core where we can push online related commits. This way we'll be able to merge offline commits to master and release it as an experimental feature.

@bartekn perfect, I created the captive-core-online-branch.


feat: supports simultaneous upload and download rate limiting

About this change - What it does
The functionality, which originally only limited uploads, has been changed to a global rate limit.
Resolves:
The purpose of this change is twofold:
When your remote storage service is located in a public cloud, such as AWS S3, and your server is on-premises, pulling data from the remote can easily saturate the local bandwidth, affecting the normal operation of other services.
Generally, AWS EC2's free disk throughput limit is 128MB. When uploading, downloading, and reading from local cache, without a global rate limiter to prevent competition between these operations, disk I/O can become uncontrollable, leading to issues with the producer's write operations.

Hi @jeqo，How can I quickly format my code to follow proper coding standards?


[HUDI-4052] Move HoodieMetricsCloudWatchConfig to hudi-client-common

What is the purpose of the pull request
Currently, HoodieMetricsCloudWatchConfig is part of hudi-aws module, while HoodieMetricConfig is part of hudi-client-common. The builder of HoodieMetricConfig uses HoodieMetricsCloudWatchConfig. When we want to use hudi write client in other engines (presto/trino), we need to include hudi-aws module because HoodieMetricsConfig initialization falls in that path. But, this bloats the presto/trino bundle, which we want to keep as light as possible.
This PR decouples the two. Let all metric configs be in client-common (e.g. prometheus and datadog configs are already there). This way we can exclude hudi-aws from the bundles.
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Closing it in favor of #5666


GPU info

How do I get it to read my actual GPU card instead of intel's integrated GPU?

Can you please send me a photo to see how it's showing ? I'm working on a update then I fix it. I do not have integrated so I didn't realise this =)

Sure. Here you go.

thanks, I will be sending new version soon.

@medowz I've updated, please tell me if it's working properly. web integrations has a cache, to see the new panel, just add this to my url : https://brunoandradebr.github.io/nzxt/?cache=1

@medowz I've updated, please tell me if it's working properly

It's working flawlessly for me. Thank you for the update.

nice!! this fix has broken for another user so I come up with a more
universal solution. Could you please tell me if it's still working and
sorry for theme reset... I will work on a better update process. To check
if it was updated, check if beside DUAL INFO name has a v1.0.2

Current version is 1.0.3
Still working great. Glad it worked out for the other user too after all.
Yes, the theme did reset indeed, but no worries man.
Again, thank you.


chore: typing b1

SSIA

Codecov Report
Attention: Patch coverage is 0% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 6.40%. Comparing base (4f18938) to head (bdb5728).
Report is 1 commits behind head on main.

Files
Patch %
Lines

midealocal/devices/b1/message.py
0.00%
7 Missing :warning:

midealocal/devices/b1/__init__.py
0.00%
6 Missing :warning:

Additional details and impacted files
@@           Coverage Diff            @@
##            main     #47      +/-   ##
========================================
- Coverage   6.41%   6.40%   -0.01%
========================================
  Files         77      77
  Lines       6581    6584       +3
========================================
  Hits         422     422
- Misses      6159    6162       +3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Pose / posture detection.

Do I have the right form? Am I bending correctly / holding sports equipment directly? Classification: good vs. bad form. Or good vs. multiple kinds of bad form (e.g. too tight / too loose, too open / too closed).

e.g. back posture (slouching vs. sitting upright). wrist posture while typing or playing piano (wrists straight vs. bent upwards). fencing posture (keeping arm straight ahead).


fix(azure): fix route name

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
The route name generated by nitro for azure was not correct due to a typo. Instead of erasing the suffix .html only the first 5 characters were used.

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Thanks!


Adding Support for Closed Generics

I also made some improvements to how enumerables are determined.  Additionally, I did some cleanup around strings being used to help reduce allocations.  I should have probably put that in a separate PR, but I do not see how to do so.  Maybe/hopefully you do.  Please do not hesitate to ask me any questions around any of my modifications.

Thank you for your work. Your change looks great. I must review it and try, but I haven't access to my computer.

Sure thing.  It is a holiday weekend after all.  I am curious though that you have appveyor installed.  Does that mean it has produced a nuget package available on a feed somewhere with my changes?

Boo, I don't see a way to link it, but I believe I fixed #10 with my latest push.

I've added information about contributors to readme.

Looks great... thank you!

I've had to restore orginal behaviors for function IsSatisfiedBy because your solution has worked wrong for custom serialization. I've added test for this situations. You can check it.
I know that current solution is not the best options but it should work for functions: ObjectReferences, Migrations, CustomSerializer and Encrypt.
If we think about inheritance of class we have situations:

Migrations - Currently, all configuration in hierarchy should run code from base migration class. In your solution only first matching config was used, e.g from base class.
CustomSerializer - Currently, all classes in the hierarchy should have configuration with own method to serialization. In your solution only first matching config was used, e.g from base class.
Encrypt - Currently, all classes in the hierarchy should have configuration with setting this option for property.
ObjectReference - Currently, all classes in the hierarchy should have configuration with setting this option for property.

I think it should be change like this:

Migrations - always all migration from hierarchy are running.
CustomSerializer - I don't have idea. I think that current work is ok.
Encrypt - Should be enough to set in the base class. But what would happen if the subclass would be a different configuration?
ObjectReference - Should be enought to set in the base class. But what would happen if the subclass would be a different configuration?

But this behaviours are contradictory. I think it should be redesigned.

Oh noes.  I will take a look into it and see if I can provide a recommendation.  FWIW, while it does make sense to have configuration-per-class, that does take a lot of work, which is why I opted for a more general (subclasses) approach.

OK I did look into this.  One consideration is to make a new configuration class specifically for Custom Serialization:
    public class CustomExtendedXmlSerializerConfig<T> : ExtendedXmlSerializerConfig<T>
    {
        public CustomExtendedXmlSerializerConfig() : base(type => type == typeof(T)) {}
    }

    public class ExtendedXmlSerializerConfig<T> : IMigrationConfiguration<T>, IObjectReferenceConfiguration<T>,
        IExtendedXmlSerializerConfig
    {
        private static readonly Func<Type, bool> Specification = typeof(T).IsAssignableFrom;

        public ExtendedXmlSerializerConfig() : this(Specification) {}

        public ExtendedXmlSerializerConfig(Func<Type, bool> specification)
        {
...

If Custom Serialization is causing the problems then you can use it as such.  However, this requires getting everyone to use it (breaking change?).
Otherwise maybe some further design around the extensions could help.  I have some ideas around this I hope to show soon.


Fix CI

Fix macOS CI compilation errors

No clue what's going on. Asked for help at https://github.com/actions/cache/issues/948

Seems like a path issue. It's fixed now, and also took the opportunity to improve on other systems (godot-videodecoder was changed as well)


Better enforce GenericFunctionType TypeVariableType invariant

Enforce that we don't have any type variables present in either the result or parameter types. To ensure the constraint system doesn't violate this invariant, refactor getTypeOfMemberReference slightly to avoid construction of a GenericFunctionType as a means of opening the generic parameters of the context for a VarDecl.

@swift-ci please test

@swift-ci please test source compatibility

@swift-ci please test macOS

@swift-ci please test source compatibility

Source compat failures unrelated


Added backwards compatibility for calling Clob#length() on NVarchar columns

Also includes a fix for image columns #1201.

Attaching compiled jre8 jar
mssql-jdbc-8.2.0-SNAPSHOT.jre8.zip


Separate TypeArguments into separate TypeParameters (binding) and TypeArguments (use) classes.

In CL 168101, we save the default arguments (i.e., results of instantiating to bounds) calculated by the CFE in TypeParameter objects. However, these default arguments are only needed when the instantiator of a TypeArguments object containing only TypeParameters has not provided any explicit type arguments, and won't be used when instantiating individual TypeParameters.
Instead, we should separate out binding and use occurrences of TypeArguments into two classes, TypeParameters and TypeArguments. The former will replace the use of TypeArguments to store the type parameters of classes and functionss, and it can store a vector that represents the entire vector instantiated to bounds without adding overhead to all TypeArguments objects. We can also move any methods on TypeArguments that only make sense for binding positions to the new TypeParameters class.

/cc @mkustermann @crelier

@sstrickl Now that my CL introducing a proper FunctionType has landed, I wanted to tackle this refactoring, unless you feel strongly about doing it yourself, of course.
This would also resolve the proper canonicalization of function type literals, for which I recently committed a temporary workaround.

@sstrickl Now that my CL introducing a proper FunctionType has landed, I wanted to tackle this refactoring, unless you feel strongly about doing it yourself, of course.
This would also resolve the proper canonicalization of function type literals, for which I recently committed a temporary workaround.

No worries, I have other things to focus on at the moment, go for it! :)
(Yaaaaaaay, separating binding and use!)

No worries, I have other things to focus on at the moment, go for it! :)
(Yaaaaaaay, separating binding and use!)

Great! Thanks. I am on it.

Great! Thanks. I am on it.


Cache cascading calc in fillTriggeredElements

Cache computation that finds generic references to children of a triggerable target. Makes use of templates when available, and shortcuts if the target is an attribute. Doesn't cache if the target reference points to a heterogeneous ref like /data/a/b/d found here: https://github.com/dimagi/javarosa/blob/master/core/src/test/resources/test_nested_preds_with_rel_refs.xml#L20-L30 . This would require an update to how templates are processed that I am weary of doing.
Evaluation is twice as fast when swiping forward past the first question of ICDS's Child Due List form in the due list test app.

This is ready to review again, though I'd like to do another pass to add comments, rename some methods, and make sure the tests are concise and functional

@phillipm This looks like a totally reasonable approach and a good optimization. I think it's good for cleanup/final push


Do not import image/png

This is a same problem as https://github.com/hajimehoshi/ebiten/issues/500

https://github.com/hajimehoshi/go-mplusbitmap/commit/3a30162dab421a31c3eec34f6409359a7b3cb3b8


why symbol name?

I am the beginner of mxnet. Now I am trying to create shared weights layers. My questions are:

for shared weights convolution layers, do they have to use different names?
why do we need name attribute? If I don't explicitly declare the name, what will happen.
Thank you!

http://blog.csdn.net/qq_25491201/article/details/51277817
maybe helpful to you

Solved by personal communication.


x/playground: https://play.golang.org was unavailable

What version of Go are you using (go version)?

$ go version

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?

What did you expect to see?
playground website is down.
What did you see instead?

Thanks for reporting. I've restarted the instances that were unhealthy, and it's back up now. We'll need to investigate further to prevent this from happening again.
/cc @toothrot

Thank you for quick action :) It is working now


Support DISTINCT for SetOperator

closes #688
This supports following  for GenericDialect ~BigQuery and MySql dialects~.

[x] UNION DISTINCT
[x] INTERSECT DISTINCT
[x] EXCEPT DISTINCT

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

31 of 36 (86.11%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 85.945%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/parser.rs
15
20
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
10927

Relevant Lines:
12714

💛  - Coveralls

okay, sorry for notification.. I didn't know I can set one reviewer.

sorry for the notification.. I didn't know I can set one reviewer only.

No worries -- without these notifications it takes me a while to find PRs to review!

Thanks again @unvalley !


Adjust css for dark mode

Description
The react components do not render properly if the browser or the OS has dark mode themes enabled.
Steps to reproduce
No response
Rucio WebUI Version
No response
Additional Information
No response

https://tailwindcss.com/docs/dark-mode


Release version 1.1.1

This is a component issue for release 1.1.1.
Coming from release issue 1.1.1, release version 1.1.1. Please follow the following checklist.
How to use this component issue

This Component Issue
This component issue captures the state of the OpenSearch release, on component/plugin level, its assignee is responsible for driving the release of the component.  Please contact them or @mention them on this issue for help.
Release Steps
There are several steps to the release process, components that are behind present risk to the release.  Component owners resolve tasks on this ticket to communicate with the overall release owner.
Steps have completion dates for coordinating efforts between the components of a release; components can start as soon as they are ready far in advance of a future release.
You can find all the corresponding dates of each step in the release issue above.
What should I do if my plugin isn't making any changes?
If including changes in this release, increment the version on 1.1 branch to 1.1.1 for Min/Core, and <IP_ADDRESS> for components. Otherwise, keep the version number unchanged for both.

You can find all the date in above issue
Preparation

[ ] Assign this issue to a release owner.
[ ] All the tasks in this issue have been reviewed by the release owner.
[ ] Create, update, triage and label all features and issues targeted for this release with v1.1.1.

CI/CD

[ ] All code changes for 1.1.1 are complete.
[ ] Ensure working and passing CI.

Pre-Release

[ ] Confirm that all changes for 1.1.1 have been merged.
[ ] Complete integration and sanity tests, and update results in the comment, example.
[ ] Find/fix bugs using latest tarball and docker image provided in meta issue.
[ ] Completed release candidate testing build #TBD.
[ ] All intermittent test failures have issues filed.

Release

[ ] Complete documentation.
[ ] Gather, review and publish release notes.
[ ] Verify all issued labeled for this release are closed or labelled for the next release.

Post Release

[ ] Create a release tag.
[ ] Suggest improvements to this template.
[ ] Conduct a retrospect, and publish its results.

Obsolete, closing.


Fix display of first steop of the cart on the narrower desktop view.

Fixed in https://github.com/EuroPython/epcon/pull/969


{Release} Hotfix: TS Supports Deleting a Single Version

Description:
Customer reported template specs (ts) issue where attempting to delete a single version of the template spec deleted the template spec and all the versions. This was the result of the delete_template_spec function calling the template_spec_delete operation instead of the template_spec_versions delete operation.
Note: #16251 should be closed.
Testing Guide:
az ts delete -g resource_group --name Issue37 -v 1.0
az ts list -g resource_group --name Issue37 -> should return the parent template spec and versions (excluding version 1.0)

resource

@detienne20 May I ask what is the expected release time for this PR? Is this sprint acceptable (2020-12-29)?
If you need a more urgent hotfix, please upgrade CLI version to a new hotfix version~


Question: Resolve container ip address of a stack service via dns query by using the NAME

I would like to resolve the containers of a service via their name:
The stack:
# docker stack ps ingress
ID            NAME           IMAGE           NODE          DESIRED STATE  CURRENT STATE           ERROR  PORTS
g5hz1ds18qur  ingress_lb.1   traefik:latest  aaaaa  Running        Running 30 minutes ago
r6mb8crnx9n1  ingress_web.1  nginx:latest    bbbbb  Running        Running 30 minutes ago
mlyy0r91o5i8  ingress_web.2  nginx:latest    ccccc  Running        Running 30 minutes ago
6ayloxworc2z  ingress_web.3  nginx:latest    ddddd  Running        Running 30 minutes ago

The service "ingress_web":
# docker service ps ingress_web
ID            NAME           IMAGE         NODE          DESIRED STATE  CURRENT STATE           ERROR  PORTS
r6mb8crnx9n1  ingress_web.1  nginx:latest  bbbbb  Running        Running 31 minutes ago
mlyy0r91o5i8  ingress_web.2  nginx:latest  ccccc  Running        Running 31 minutes ago
6ayloxworc2z  ingress_web.3  nginx:latest  ddddd  Running        Running 31 minutes ago

I know that this works from inside a container on the same overlay without problems:
> ingress_web
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53

Non-authoritative answer:
Name:	ingress_web
Address: <IP_ADDRESS>
> tasks.ingress_web
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53

Non-authoritative answer:
Name:	tasks.ingress_web
Address: <IP_ADDRESS>
Name:	tasks.ingress_web
Address: <IP_ADDRESS>
Name:	tasks.ingress_web
Address: <IP_ADDRESS>

I would like to:
> ingress_web.1
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53

** server can't find ingress_web.1: NXDOMAIN
>

Am I missing something? I would like to use the dns names of the service containers directly and not the VIP.
Thanks!

@kleinsasserm this issue is related to Docker Swarm Mode, which is a different project. Check README here for the distinction: https://github.com/docker/swarm.
Feel free to open this issue at https://github.com/moby/moby.


Support for FreeBSD guests ?

Description
There was some question in Slack, about supporting also FreeBSD guests.
I guess it would be "freema" or something, and not just "lima", but it is doable...
There are qcow2 images for all architectures, but it does involve adding OS.
I was just doing some experiments, so thought I would raise the question first.

Example:
os: "freebsd"

images:
- location: https://download.freebsd.org/ftp/snapshots/VM-IMAGES/14.0-CURRENT/amd64/Latest/FreeBSD-14.0-CURRENT-amd64.qcow2.xz
  arch: "x86_64"
- location: https://download.freebsd.org/ftp/snapshots/VM-IMAGES/14.0-CURRENT/aarch64/Latest/FreeBSD-14.0-CURRENT-arm64-aarch64.qcow2.xz
  arch: "aarch64"

video:
  display: "default"

It is supposed to have support for OCI runtime:
https://github.com/samuelkarp/runj

Hi! I was the original requestor on Slack. My need is to run a Puppet module acceptance tests on a FreeBSD. Lima is really attractive here because it's headless and scriptable. People are using VirtualBox (with or w/o vagrant) to do this usually. But after I got the M1-based Macbook I lost the ability to do this. In my case there is no real urgency but it'd be really nice to have. I'll try it on a coming week and will share the experience here.

The support for linux containers (on freebsd) seems to have merged as well, in 1.7.

Short list of portability issues:

There is no /run, only /var/run (added variable)
There is no /bin/bash, only /usr/local/bin/bash. (added variable)

And of course: s/linux/freebsd/
The LocalPorts support for guestagent can be implemented using sockstat.
Note: the output of the sockstat command is subtly different, between the OS.
The sshfs  options for "fusefs" does not have allow_other, unfortunately:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=215797
Not sure if -o slave works, but there is some -o passive flag instead ?
Also the kernel module is not loaded by default, needs /boot/loader.conf
But otherwise, it seems to be working:
   sudo mkdir -p /mnt/lima-cidata
   sudo mount -t cd9660 /dev/cd0 /mnt/lima-cidata
   sudo cp /mnt/lima-cidata/meta-data /var/run/lima-ssh-ready
   sudo install -m 755 /mnt/lima-cidata/lima-guestagent /usr/local/bin/lima-guestagent
   sudo -b lima-guestagent daemon
   sudo cp /mnt/lima-cidata/meta-data /var/run/lima-boot-done

Created the user manually, for now.
$ limactl shell freebsd uname
To see the output from when your computer started, run dmesg(8).  If it has
been replaced with other messages, look at /var/run/dmesg.boot.
		-- Francisco Reyes<EMAIL_ADDRESS>FreeBSD

Probably want to disable those "tips"...
# $FreeBSD$
#
# .login - csh login script, read by login shell, after `.cshrc' at login.
#
# See also csh(1), environ(7).
#

# Query terminal size; useful for serial lines.
if ( -x /usr/bin/resizewin ) /usr/bin/resizewin -z

# Display a random cookie on each login.
if ( -x /usr/bin/fortune ) /usr/bin/fortune freebsd-tips

Asked this some time ago: https://github.com/lima-vm/lima/issues/1096

Thanks for the reminder. I used QEMU 4 ☺

The support is quite rough, but nerdctl and friends (containerd and runj) are available in the default pkg.
It does require setting up a ZFS disK and pool, but this can be done using the limactl disk feature (/dev/vtbd1)
[anders@freebsd ~]$ sudo nerdctl version
WARN[0000] unable to determine buildctl version: exec: "buildctl": executable file not found in $PATH
WARN[0000] unable to determine runc version: exec: "runc": executable file not found in $PATH
Client:
 Version:	v1.2.1
 OS/Arch:	freebsd/amd64
 Git commit:	<unknown>
 buildctl:
  Version:

Server:
 containerd:
  Version:	1.7.0
  GitCommit:	unknown
 runc:
  Version:
[anders@freebsd ~]$ runj --version
runj version v0.0.1-dev ()
go: go1.20.3
[anders@freebsd ~]$ sudo nerdctl info
Client:
 Namespace:	default
 Debug Mode:	false

Server:
 Server Version: 1.7.0
 Storage Driver: native
 Logging Driver: json-file
 Cgroup Driver:
 Cgroup Version:
 Plugins:
  Log: fluentd journald json-file syslog
  Storage: zfs native
 Security Options:
 Kernel Version: 14.0-CURRENT
 Operating System: FreeBSD 14.0-CURRENT
 OSType: freebsd
 Architecture: amd64
 CPUs: 0
 Total Memory: 0B
 Name: freebsd
 ID: ebcf5f96-01a3-4f40-a063-e3f64163c948

You can run native (freebsd) containers, but so far I have not been able to run linux containers (issues mounting the fake /proc ) even though linux binaries (like limactl) do work on the VM (after enabling linux service)

https://www.freebsd.org/status/report-2023-01-2023-03/#_freebsd_as_a_tier_1_cloud_init_platform

Here is an example of a freebsd container image:
public.ecr.aws/samuelkarp/freebsd:13.1-RELEASE

Instructions for adding additional users:
https://docs.freebsd.org/en/books/handbook/basics/#users-synopsis
Instructions for adding containerd packages:
https://docs.freebsd.org/en/books/handbook/ports/
Instructions for setting up a ZFS disk pool:
https://docs.freebsd.org/en/books/handbook/zfs/

Similar work for Podman:

https://github.com/containers/podman/pull/19939

Hi @afbjorklund, are there any updates on this? I think this would be a great way to tinker with FreeBSD.

I don't have any further updates, but could do a rebase of the existing branch (#1509) if it helps anyone?
I guess 14.0 is out now, but I don't know if runj is more mature...  Or if it is going to be in containerd 2.0

I'm pretty excited about this idea but... what do you think about adding support for netbsd as well?

Not sure there are any official cloud images, would need to use e.g. https://bsd-cloud-image.org/
What would a NetBSD image be used for? For FreeBSD, there was the concept of testing runj etc

I realize there may not be an OCI-compatible runtime native/exclusive to NetBSD and that, therefore, it may fall outside the purview of this project...
I am interested, however, in NetBSD's veriexec functionality as it could relate to running containerized apps.  Similarly, a lock-down mechanism like securelevel.
I don't know.. just a thought.

Just found some time to try FreeBSD in lima finally. My current goal is to be able to ssh into the VM in plain mode.
Tried BASIC-CLOUDINIT with UFS from here: https://download.freebsd.org/releases/VM-IMAGES/14.2-RELEASE/aarch64/Latest/.
It stuck on waiting for ssh. Console shows login prompt but no input from keyboard was accepted. Though, I was able to login via serial console. Network was good, ssh port was forwarded. But user was not created. Later I found there is no cloud-init at all in this image 🤔 Will check other images later.


front-page.php vs. home.php

Wordpress settings differentiate between these two template files in such a way that I am not taking advantage of the system. Change these files to confirm to wordpress community standards.

the idea is to make it so that if you set a page as the blog feed, have that page look more like a typical blog with headline and content. Make the "more content" link at the end extend the content so that someone doesn't have to leave the page. Or, if javascript disabled, then have it link to the actual post.

have implemented base logic to make it so that static home page shows as it should and applied generic blog archive template with the tiles of 4 per row to the home page when showing latest posts or on blog feed. need to design a template that fits the intended design.


Crash in production build with Vite

The app was running pretty good at local machine until it was built and previewed. My project is bulid with Vite. The console throw out this error:
Uncaught TypeError: z0.jsxDEV is not a function

Seems to be working fine for me in Vite: https://stackblitz.com/edit/vitejs-vite-zcme9k?file=src%2FApp.tsx
Are you able to create a way for me to reproduce the problem?

@EvHaus thanks for the great package, but I also got an error Uncaught TypeError: z0.jsxDEV is not a function only in a build mode. It's good if you run it locally without building.
My dependencies:
"dependencies": {
"@react-pdf/renderer": "^3.1.12",
"@reduxjs/toolkit": "^1.9.2",
"@testing-library/jest-dom": "^5.16.5",
"@testing-library/react": "^13.4.0",
"@testing-library/user-event": "^13.5.0",
"react": "^18.2.0",
"react-beautiful-dnd": "^13.1.1",
"react-dom": "^18.2.0",
"react-pdf-charts": "^0.0.5",
"react-scripts": "5.0.1",
"recharts": "^2.7.2",
},
so basically when you run npm run react-scripts build and then serve the app you'll get this error, but if you run npm run react-scripts start it works great. I'd appreciate any help with that :)

Thanks! I think I found the issue. I just published v0.2.1 that should hopefully fix the problem. Let me know if it's still happening.


Fix release docs action and LIS User Guide formatting

Description
Hey LIS team! Hope you're all doing well. I checked in on the repo and saw that the GitHub Action that I set up to compile the user guides and attach them to releases was failing. These changes fix that CI job (and a minor issue in the lis.confic.adoc formatting).

Hi @bmcandr
Nice to hear from you.  Hope things are well with you also.
Thanks for checking in and for submitting a fix to this issue.
I will let @jvgeiger confirm and merge this fix.  Thanks again!
David

Hello @bmcandr,
Yes, it is good to hear from you.  I am doing well.  I hope that you are, too.
Thanks for the patch.  This looks good.
Jim


Inserting only new triples by BrightstarDB

Hi,
I'm new in BrightstarDB. How can I insert just new triples into the embeded database by the RDF Client API or something else?
Thanks for your advice
Best regards
Kamal

Through the API you basically have two options for adding triples:

Create a directory called import in the same directory that contains the database directory; add an RDF file in one of the supported syntaxes to that directory and then start an import job. See http://brightstardb.readthedocs.org/en/latest/RDF_Client_API/#data-imports

Add the triples in a transactional update - the syntax for your triples to add is simple NTriples/NQuads syntax. See http://brightstardb.readthedocs.org/en/latest/RDF_Client_API/#transactional-update

If you have the triples you want to create in memory then (2) is probably the easiest way to go. If you want a nicer syntax for creating the set of triples to add then I would recommend that you look through the dotNetRDF API as it has lots of good APIs for creating a graph of RDF triples which you can then serialize to a C# StringWriter in NQuads format - as BrightstarDB uses dotNetRDF, you already have the necessary assemblies available to do this. Take a look at https://bitbucket.org/dotnetrdf/dotnetrdf/wiki/User Guide for a good introduction to using dotNetRDF.
Cheers
Kal

Hi dear Kal,
Thank you so much for your respond but my problem is that the triples are recording more than one time when I use ExecuteTransaction more time with the same triples. I expect that BrightstarDB doesn't duplicate the triples and prevent to save duplicate triples but it doesn't.
I'm sorry for my english and hope you get my point.
Thanks for your advice
Kind regards
Kamal

Hi Kamal,
Your English is great :)
If you are seeing duplicated triples in BrightstarDB then that is definitely a bug - however it really should not happen with truly duplicated quads - the same (subject, predicate, object, graph) cannot be inserted twice into the indexes so in theory there should be no way for a duplicate quad to be inserted. A duplicate triple could be inserted, but only if the triples are in different graphs.
However you should note that BrightstarDB will not treat an attempt to add a duplicate quad as an error - it will simply continue, ignoring the duplicates - so you won't get any feedback that duplicates have been ignored (this is by design - it should always be possible to import partially or completely duplicated data without having to deal with lots of messages / errors from BrightstarDB).
So, I think what you are seeing is just that BrightstarDB doesn't treat an attempt to upload duplicate triples/quads as an error - if so, then this is "by design" and not a bug. However, if when you read the triples back from BrightstarDB (e.g. in a SPARQL query or an export) you get duplicates of the same quad (including the graph part), then we have a problem :-)
If you have a simple way to reproduce a condition where BrightstarDB produces duplicate quads, can you please share your code here? You can upload it as an attachment, or if you have it in a public GitHub repo, just let me know the URL for that repo.
Thanks,
Kal

Hi dear Kal,
It's my pleasure to talk to you :)
When I run the following code several times, the size of Data.bs, Transactions.bs and Master.bs files are increasing.
var client = BrightstarService.GetClient("type=embedded;storesdirectory=D:\Kamal;storename=Semantic");
IUriNode rdfType = schema.CreateUriNode(new Uri(NamespaceMapper.RDF + "type"));
INode seeAlsoSchema = schema.CreateUriNode(new Uri(NamespaceMapper.RDFS + "seeAlso"));
IUriNode dbPediaProperty = schema.CreateUriNode("wt:dbPedia");
string storeName="Semantic";
var data = new StringBuilder();
data.AppendLine("<" + dbPediaProperty + "> <" + rdfType + "> <" + seeAlsoSchema + "> .");
client.ExecuteTransaction(storeName, null, null, data.ToString());
but now I run the folowing paragraph instead of the last sentence, the size of Transactions.bs file just increases in the next runs.
if (client.DoesStoreExist(storeName))
{
var transactionData = new UpdateTransactionData
{
NonexistencePreconditions = data.ToString(),
InsertData = data.ToString()
};
client.ExecuteTransaction(storeName, transactionData);
        }

Thanks for your advice
Your sincerely
Kamal

Hi Kamal,
The data files will all increase in size, because even if nothing is added in a transaction, a page is created to record the entry points for that transaction in the database. The transactions.bs file is simply a record of what transactions have been submitted to the store (it is not a part of the database - it is there as a log / way to recover a store in the event of data corruption).
If you try exporting from the store (e.g. using Polaris) you should see that there are no duplicates in your data.
Cheers
Kal

Hi Kal,
It's great :)
Thank you so much.
Kind regards
Kamal


dnd4e system support

Can't let the most "video game" edition of dnd go without a fishing mini game now can we?

Thanks for the contribution!


Format Markdown

Issue #317 proposes the introduction of built documentation using MkDocs.
There are currently a few bugs in the rendered HTML generated by MkDocs. However, these can all be fixed with whitespace-only changes to the Markdown documents:

Empty lines between paragraphs and lists
Empty lines between paragraphs and tables
Consistent list indentation
Remove leading whitespace from headings and paragraphs

The above changes are compatible with the GitHub flavoured Markdown specification meaning rendering of Markdown on GitHub.com will not be affected.
Examples
As examples, two screenshots of the MkDocs output follow.
1. Section Project Team in README.md:

2. Section 3.2 Aluminum Rods: Drilling in mechanical/differential_pivot/README.md:

I have prepared #363 which has all Markdown files formatted with the tool markdownlint. It is currently a draft as it affects all Markdown files.
Having the same formatting access all files will set a precedent for future edits by introducing consistent whitespace formatting across all Markdown files. It won't be necessary for contributors to use markdownlint.
VSCode extension: DavidAnson.vscode-markdownlint
VSCode extension code: https://github.com/DavidAnson/vscode-markdownlint
Command-line tool: https://github.com/DavidAnson/markdownlint


Integrate PDS into SecHub

Situation
At the moment a product delegation server is not correct accessed by SecHub server.
Wanted
All configured (per profile+config) PDS instances

must be executed in job
must have parameters inside as defined in config
must be used for webscan, infrascan, codescan as being defined in config
(we need the possiblity to define type at executor configuration and evaluate this!)

Additional

We have dedicated product identifiers for PDS representing the  scan type: e.g. PDS_CODESCAN, PDS_WEBSCAN etc. so it is clear which kind of scan it is.
PDS executor(s) will be executable multiple times (for ONE config). Other executors will be executable onle once, even when having profiles containing multiple configurations for same executor (we will do a "first wins" approach + log warning)
Support credentails plain and also with "env:" approach were environment entries are used.

Albert Tregnaghi<EMAIL_ADDRESS>Daimler TSS GmbH, imprint

Normally we do our integration tests with mocked adapters only. For PDS integration tests in Scenario5 there should be a real communication and no mock operation.
To have possiblity to test this, we change the existing mock adapter to be used by a DelegatingMockablePDSAdapterV1 which uses normally the real one , but when job parameter do contain key/value pair is set to "mocked=true" use mocked variant

With last commit PDS and SecHub do now work "hand in hand" - at least for the PDS Integration test server and SecHub integration test server.
We still need #386 and PDS instances providing sarif results to have a common interchange format. Any other format must be also implemented in Sereco which is still a little bit cumbersome, so the aim for the future should be to always use SARIF when #386 has been done.
What's currently still missing/todo inside this issue?
The pds integration bash script called by PDS should evaluate given data and give feedback: So we will write just some additional integration tests and improve the PDS bash script and test the parameter interchange as well.
There will be more to implement in PDS (e.g. #434 and much more like auto cancelation by sechub wenn sechub job has been canceled etc.) but this will be done in dedicated issues.

There is a little bug inside Developer admin ui: TestAPI calls do use super admin credentials from tests - shall use given ones.


Filtering messages with regex.

Some content just never wants to be seen by the user.
An sqlite file will be used so filters can be managed.
A popup window should also be used so the user can read/write filters.
This keybind shouldn't be too easy to reach since filters can contain quite abrasive inputs. If another person is watching the user's screen, said keybind wouldn't be desirable to hit accidentally.

Another change. The regex items won't be stored in a database, but instead a text file, if it exists.
Due to #119 being too difficult to implement at this time with so many features being implemented, I have decided that a simple text file would be better at this time.

Another feature: If the filter is disabled, a button can be pressed to use the filters to purge all matches.


Unable to import related model in index

Software
Version

PHP
7.4

Elasticsearch
7.9

Laravel
8

Describe the bug
Hello, I'm unable to populate elasticsearch with a related model. In my example I'm indexing courses, every courses can have 1 or many keywords (belongstomany relation) all fields marked as searchableArray are indexed except the relation which should be a nested array.
See the stackoverflow discussion please. I'm not the only one to have this problem, I don't know if it's a bug or a not well documented feature ?
https://stackoverflow.com/a/66018497/5794757
Thank you very much.

Hey, I've answered on Medium, but I can see that you've already found a solution 🙂


Detect inactive people in Bugzilla

Multiple issues require detecting inactive users to perform an action or produce a report, e.g.:

#190
#266
#1231
#1179

The question is, how we should proceed in detecting inactive users?
@calixteman suggested using the diff in the people.json file to know who left (works for Mozilla employees only).
In assignee_no_login.py#L72, the assignee_last_login column is used to determine inactive assignees. However, based on a comment by Kohei, it seems that the assignee_last_login column does not give the desired signal.

https://bugzilla.mozilla.org/show_bug.cgi?id=1527841#c3:
Kohei Yoshino - comment 3
I’ve realized that the assignee_last_login column is useless. It’s literally the last login date of the user, not the last seen date in spite of the internal field name called last_seen_date , which is perhaps a legacy of the days when people had logged into Bugzilla everyday at 9am and shut down their workstation at 5pm. Now people keep logged in, so the date can be quite old.
We need the new assignee_last_seen column based on another internal field called last_activity_ts, which is currently a part of the UserProfile extension. I’ll file a separate bug for that.
The same applies to needinfo. What we need here is not needinfo_last_login but needinfo_last_seen, right?

Bugzilla still periodically requests to re-login, so if we set a high enough threshold we should be fine (indeed, so far the assignee_no_login didn't find any inactive user that was actually active).
We can then switch to a better method later.

A third way we can use is the Bugzilla User REST API (https://bugzilla.readthedocs.io/en/latest/api/core/v1/user.html), checking if an account is disabled.
A disabled account can be considered an inactive account.

#1342 is implementing most of this, except for the identification of people who left Mozilla through the usage of people.json (see also #1354 which is related).

#1342 is implementing most of this, except for the identification of people who left Mozilla through the usage of people.json (see also #1354 which is related).

@suhaibmujahid there is still this left to do after #1342 is fixed, do you want to file a new issue or reopen this one?

@marco-c the  people.json part is already covered with #1354. But I will fill a new issue to breakdown #1354.

I filed #1371 to tack the people.json part.


Improve how types are managed in message_store.set_message_booleans

See #27408 for details -- we didn't end up completing the message_store.js TypeScript migration through that PR, but it had some ideas for how to make the type-safety in that module a lot nicer.

@zulipbot claim

@zulipbot abandon

@zulipbot claim

@timabott I have read all the discussion and now have a basic understanding of what all changes might be required, just wanted to confirm is it just the send_message_booleans function that requires a typing improvement, or are there other functions as well, I should take a look over.

@zulipbot claim

@zulipbot abandon

This has been addressed in

#30227
#30289


Strip invalid spaces from Manufacturer slug generation

https://github.com/minitriga/Netbox-Device-Type-Library-Import/blob/1f31c11edadccbdfa984e2cb170fe05f2142f3cf/nb-dt-import.py#L55
Slug can't have spaces, but some directories and the Manufacturer Name does have spaces, strip them out.  Fixes problems importing devicetype-library for "Palo Alto"
data['manufacturer']['slug'] = manufacturer.lower().replace(" ", "")

Slug can't have char '&' and the process importing fail. To fix problem importing devicetype for "Rohde & Schwarz":
data['manufacturer']['slug'] = manufacturer.lower().replace(" ", "").replace("_&","")

In the PR, I implemented a clean solution in the original slugFormat() function which solves the problem with special chars


README.md better explanation of Parsing Zone Files

Add imports
change print() cause of python3
>>> import blockstack_zones
>>> import json
>>> zone_file_object = parse_zone_file(zone_file)
>>> print(json.dumps(zone_file_object, indent=4, sort_keys=True))

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

This would be awesome to get merged, I ran into the import issue on py3!
@jcnelson mind taking a look?

Hey @philipcristiano, @taich001,
Thank you for sending this PR!  Unfortunately, we have no plans to move to Python 3, and are in the process of removing all of the Python modules in this organization (we're moving to Typescript and Rust).  However, please feel free to maintain a Python 3 fork if you find it useful :+1:


Add a script to automate FSX resource creation

Description of your changes:

[x]  Add a script to automate FSX resource creation

Testing: -
Tested the change manually at every step to ensure it is doing what is expected and also ran the suite multiple times locally.
Link to testing logs - https://gist.github.com/mbaijal/1fec71cd7f4d421ccf007f074b110c28
Checklist:

[ ] Unit tests pass:
Make sure you have installed kustomize == 3.2.1

make generate-changed-only
make test

Looks good, script is working with revisions made ?

Running ...

Looks good, script is working with revisions made ?

Test is running, will update.

Looks good, script is working with revisions made ?

Test is running, will update.

Test completed successfully.


Update build_primer_pairs() to accept Iterable[Oligo] or Sequence[Oligo]

I have a use case where I have subclassed Oligo in order to store QC attributes (e.g. hairpin delta G and off-target hit counts) and filter individual primers before assembling them into pairs.
Because list is invariant, the current implementation does not support building pairs from subclasses of Oligo.
Note that the implementation prior to #75 supported this use case because the arguments were defined as Iterable[Oligo].

https://mypy.readthedocs.io/en/stable/common_issues.html#variance


Filtering filetypes with statistic-irrelevant content

In the current state, Ghost logs every filetype and shows that in the table. This includes files like package.json or localization, that users may or may not find useful.
Suggestion: adding some kind of filteroption for the table to exclude filetypes or folders

Proposed way to handle this, copied out of issue #22
use "somewhat similar to the git ignore
eg for i18n"


added square of a number in java, updated README.md and Index.md

Why:
Closes https://github.com/codinasion/codinasion/issues/323
What's being changed:

Added Square Class
Updated README to Include JAVA Code
Updated Index.md with my github username under contributors list

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Hey @SpadeQ22
Thanks for contributing to this open source project 😃
This PR will be reviewed and merged shortly.

Preview : https://codinasion--pr33-dev-mef9n8f9.web.app/programme/square-of-number/
(expires Wed, 16 Mar 2022 06:35:30 GMT)

LIVE : https://codinasion.web.app/programme/square-of-number/


V2 docs improvements

Still WIP, but I'm trying to organize sections better and tweak the wording a bit. It's based on the community feedback in general that the mutation hook is gonna have the biggest impact.

📝 Changed routes:

/blog/swr-v2.en-US
/docs/meta.en-US

(view full size)
(view full size)

/docs/mutation.en-US
/docs/mutation.es-ES

(view full size)
(view full size)

/docs/mutation.ja
/docs/mutation.ko

(view full size)
(view full size)

And 3 other routes:

/docs/mutation.pt-BR
/docs/mutation.ru
/docs/mutation.zh-CN

Commit c59442684982bd7274edd1a03f5e6d2945d7df29 (https://swr-site-nsptqjhb7.vercel.sh).


Can't fill up bucket

Description (Required)
When install DynaTech on server, normal bucket can't fill up water.
Steps to reproduce the Issue (Required)
https://user-images.githubusercontent.com/26039249/215283946-1c16c0c2-4929-4bdf-9880-6e5b481cfa09.mp4
Expected behavior (Required)
Bucket can fill up water.
Server Log / Error Report
Environment (Required)

The latest commit should fix this! Please confirm :)

Yeah, it's fixed. Thanks!


Make scripts consistent

Ross, the travis/ changes weren't applying cleanly here so I pulled in everything else separate with https://github.com/jay0lee/GAM/commit/2d0396da21095a1bb0ea7cfd215b1e723ffe932b and will go ahead and close this out.


Why does the test Block Stacking Demo fail to run after installation?

Whether I ran it directly in Pycharm or typed: python tutorials/block_stacking_demo.py on the terminal, the result would look like this

Does this project not support Windows?
https://discuss.pytorch.org/t/cant-pickle-local-object-dataloader-init-locals-lambda/31857/33

At the moment, we do not support Windows as we have no one who develops on Windows who could test. In terms of your issue, you should be able to change the 1 in line 5 to 0 like so: env = env_factory.createEnvs(0, 'block_stacking', env_config) . This will run everything in the main thread and not use any multiprocessing. If you need to run multiple simulators in parallel it looks like you would have to look into using Pathos and dill instead of the generic multiprocessing library. Unfortunately, we need to be able to pickle lambdas (which windows can't do) in order to package the configurations for each simulator in the mutli-simulator setting.

Thank you for answering my question, I will consider using Linux to run later
------------------ 原始邮件 ------------------
发件人:                                                                                "ColinKohler/BulletArm"                                                                                @.>;
发送时间: 2022年9月1日(星期四) 晚上11:02
@.>;
<EMAIL_ADDRESS>主题: Re: [ColinKohler/BulletArm] Why does the test Block Stacking Demo fail to run after installation? (Issue #4)
At the moment, we do not support Windows as we have no one who develops on Windows who could test. In terms of your issue, you should be able to change the 1 in line 5 to 0 like so: env = env_factory.createEnvs(0, 'block_stacking', env_config) . This will run everything in the main thread and not use any multiprocessing. If you need to run multiple simulators in parallel it looks like you would have to look into using Pathos and dill instead of the generic multiprocessing library. Unfortunately, we need to be able to pickle lambdas (which windows can't do) in order to package the configurations for each simulator in the mutli-simulator setting.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


404 at: /engine/getstarted/step_one/#docker-for-windows/

URL: /engine/getstarted/step_one/#docker-for-windows/

Thank you for your feedback. It appears that the issue at https://docs.docker.com/engine/getstarted/step_one/#docker-for-windows has been fixed. Please let me know if you're still having issues with this topic.

Thank you.


freebsd sched_setaffinity signature had been fixed upstream.

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit a9f83e86f359ae1ea62fdf74776675b22d3f4833 has been approved by Amanieu

:hourglass: Testing commit a9f83e86f359ae1ea62fdf74776675b22d3f4833 with merge e81e0685bc41b70944b0c169eff81daff15d6229...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing e81e0685bc41b70944b0c169eff81daff15d6229 to master...


Corrected Fortran

There were missing blank spaces at the end of the keyword strings.
A few keywords were missing

Thanks for everything! You are very reactive!


Update the analytics events for relevancy

Fixes

Fixes #5005 by @zackkrida
Fixes #5107 by @obulat
Description

This PR updates the analytics events so that we can analyze results for relevancy for their search queries.
From the single result page events (COPY_ATTRIBUTION, GET_MEDIA, RIGHT_CLICK), and for selecting the result (SELECT_SEARCH_RESULT), we can now see the search query and the position of the current result in the list of results.
Some of the event data were refactored to simplify the code.
Testing Instructions

Run the app using ov just frontend/run dev
Run a search and select a search result. You should see SELECT_SEARCH_RESULT event with query and position logged in the console.
If you click on "Copy button", or right-click on the image, you should see the new properties (position and query) for the analytics events in the console.
Checklist

[ ] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (ov just catalog/generate-docs for catalog
PRs) or the media properties generator (ov just catalog/generate-docs media-props
for the catalog or ov just api/generate-docs for the API) where applicable.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@obulat, I'll put this in draft while the tests are resolved.


Redirection not happening for Http Put request

I have a PUT request that simply redirects (308 Permanent Redirect) to another PUT request. In Karate I have the following code
Given path '/rest/v1/samples'
And request some-payload
When method put
Then status 200

The assertion always fails as the redirection is not happening and so getting the status as 308. I have not modified the default value of followRedirects.

@azarfaiz follow this process please: https://github.com/intuit/karate/wiki/How-to-Submit-an-Issue