net: SocketCAN support and SocketCAN CAN-utils tools

Summary
Adds SocketCAN (AF_CAN) support to netlib require for https://github.com/apache/incubator-nuttx/pull/1238.
And adds SocketCAN cansend & candump utilities ported from https://github.com/linux-can/can-utils
Testing
Verified on rddrone-uavcans32k146 (S32K146) & rddrone-fmuk66 (Kinetis K66).

Since this change includes third party code, we will need to update the LICENSE file.  Perhaps we can do this after a merge.


Question: how to aggregate output across multiple branches or actions

There are several cases where multiple outputs need to be merged back into state data:

Parallel State
-- each branch might produce its own output (correct assumption?)... what does the union of all branch outputs look like? how do you (for example) author a single output filter for the parent parallel state that acts upon individual branch outputs?
-- completion type will likely factor in here, too (XOR only produces a single element, the others produce multiple, etc.)
-- Branch does not have a name, which makes it hard to construct a synthetic output for all branches that let's you later identify the output from any given branch

ForEach State
-- the spec defines OutputCollection which is fine, but suffers from the 'can't create a new node with JSONPath' problem (perhaps #216 is meant to address this?)

Operation State with multiple actions (or any other use of 'Actions' array)
-- action has results path, so we can merge individual action results back into state data (which is good!)

These seem to be slightly different takes on the same general pattern of "merging multiple results back into the main workflow state". I think it would be useful to either define a common expected solution for this pattern in the spec, or at least to clarify the differences and why they exist.
Glad to help with that, if we agree its a worthy cause.  :-)

@jplane thanks for submitting the issue!

parallel state:
a) yes merging here is highly dependent on completionType param
b) definitely let's add "name" property to branch definition

foreach state
a) jq will help alot with this. with jsonpath its assumed that the runtime has to create this. we also have pr for this: https://github.com/serverlessworkflow/specification/pull/249

actions
a) definitely can look into this further to make sure to iron out any kink

+100 on working on defining data filtering and merging better, and you helping on this :)

Fantastic... appreciate the quick reply. I'll pull something together in a draft PR over the next day or so.

I'm wondering if we can solve this by extending the data filter concept:

add parallel branch result data filter (one for each branch, defined on the branch entity)
remove ForEachState.OutputCollection and add ForEachState.IterationDataFilter (applied to each result of the iteration, to merge back to state data)
action input/results filter is fine as it is, I think

Thoughts? I'll submit a PR if we agree on the general direction.

+1 on per-branch data filter
the problem with doing the proposed filter, especially with jq is that it could alter/change/modify the inputCollection array which the state iterates over and thats a big problem (like modifying the array/list while iterating it in programming). you would imo have to specify that the IterationDataFilter can access the outputcollection object, but with jq i do not atm see a benefit of that as jq can do it alone

thoughts?

RE: problem of proposed IterationDataFilter modifying the InputCollection... agreed, but that problem still exists today anyway (Input/OutputCollection could resolve to the same collection, even with different jq expressions, etc.)
For me, the simplest solution has the fewest moving parts, and borrows from concepts that already exist elsewhere. DataFilter is one of the best parts of the existing spec, and appears in many places... which makes it familiar for workflow authors and promotes modular design/implementation for devs building runtimes (hopefully, anyway ;-) ).
I'm almost of the opinion a minimalist implementation like this would be ideal:

remove ForEachState.InputCollection
remove ForEachState.OutputCollection
remove ForEachState.IterationParam
make ForEachState.StateDataFilter required
implementation note... ForEachState.StateDataFilter.DataInputPath should produce a JSON array (runtime error if not) where each element is implicitly the input an iteration, no need for an extra "iterationParam" property in that input
implementation note... ForEachState.StateDataFilter.DataOutputPath should define where to insert/append output collection (jq is good for this, as already noted)

Again, I think this minimizes the moving parts and leverages concepts like data filters that already exist.

@jplane i have some questions about this, will try to explain and let me know what you think:

it moves the complexity from "inputCollection" to the filter and makes the foreach states data filter "special/different' than for other states -> having to produce the iteration array in the data filter may remove some other state input data that is needed for the actions that are executed in the foreachstate iterations. also the foreach states state data filter now  also has a special use now that does not apply to any other state data filters (in addition to it now being required which is not the case in other states).

unlike with a "loop" (repeat in subflow states for example) the foreach state is executed only once. therefore its state data filter (datainputpath and dataoutputpath) is be executed only once as well, when the state is started and before it ends.
for filtering of the results of each action execution in foreach state you should probably use the event data filter, and if user defines a workflowid instead, the filter then applied in that sub-workflow (basically that sub-workflows data output should be added to the outputcollection.

wdyt?

@jplane
Regarding ** but that problem still exists today anyway (Input/OutputCollection could resolve to the same collection, even with different jq expressions, etc.)** : good point, we really need to add in spec that input and output collections should not be the same (if its not already in the spec doc). runtime impls should assure that is not the case...or dont worry about it which will most likely result in runtime exceptions during execution (user error :) )

Okay, you raise some valid points about ForEach. So perhaps this:

add ParallelState.Branch.DataOutputFilter (as discussed above)
keep ForEachState.InputCollection and .OutputCollection
-- I suggest an implementation note that runtimes either a) detect and prevent overlap between InputCollection/OutputCollection (which might be hard/slow from a practical standpoint) or b) operate against a copy of InputCollection so that modifications via OutputCollection don't affect ForEach processing
eliminate ForEachState.IterationParam (its not clear to me why we need this at all vs. just pass the input collection element directly as input for each iteration)


Login Page Setup

How to make the Login And signup page work as i tried hosting it locally in my system but the login page gives error do i need add my .env variable in it to work. If so can u provide me with a Template of the .env file

There is no env file. The issue might be related to the fact that the backend api  has CORS enabled. so you can't access backend locally or with any domain other than netlify.

Bro how can I get access backend locally.. Please me bro

I am going to  make the backend repo public soon.


Customlocations fix

Also added 4 melon crate location :)

Let's not needlessly merge dev with this branch ever again.


Cannot read property 'avatar_url' of null

Getting this error and I'm unsure why.
TypeError: Cannot read property 'avatar_url' of null
at Object.createMessageCard (/home/runner/work/_actions/jdcargile/ms-teams-notification/v1.3/dist/index.js:2446:16)
at /home/runner/work/_actions/jdcargile/ms-teams-notification/v1.3/dist/index.js:3088:54
at Generator.next ()
at fulfilled (/home/runner/work/_actions/jdcargile/ms-teams-notification/v1.3/dist/index.js:3042:58)
at processTicksAndRejections (internal/process/task_queues.js:93:5)
Error: Cannot read property 'avatar_url' of null

sorry found this was already reported


fix(S2): clear isRequired context for CheckboxGroup in Form

Previously, CheckboxGroup would incorrectly sets all its children to isRequired when a parent Form is set to isRequired. This means all the checkboxes would need to be checked to be valid, however we need just one to be checked (matches S1 behavior.)
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:
In the Form story (with isRequired: true), select a checkbox. There should be no error message.
Next, unselect that checkbox. There should now be an error message.
🧢 Your Project:

Good candidate for a test?

@snowystinger Good idea, added!


Site Showcase Request

Please include:

Description
Primary Link
Screenshot

Do you forget to include the details in your showcase request?


chore(client): update python-dateutil to 2.4.0

See https://github.com/dateutil/dateutil/compare/e49a25913...2.4.0

Code LGTM.

code LGTM


docs: Add info on zulip-archive on for/working-groups-and-communities.

Testing plan:
N/A
GIFs or screenshots:
N/A

Merged, thanks @rht!  I think I'd rather spend focus on trying to merge the public archive code in this case rather than on linking to issues, especially in this article which we're likely to end up rewriting soon.


Language injections does not work with HEREDOC

Code highlighting is not working when used language injection on HEREDOC.
Double quoted text works fine.
#!/usr/bin/env bash

#language=php
/usr/bin/php << PHPCODE
<?php

\$variable = 'value';
echo \$variable;

PHPCODE

#language=php
phpcode="<?php \$variable = 'value'; echo \$variable;"
echo $phpcode | /usr/bin/php

Language injections: https://www.jetbrains.com/idea/help/using-language-injections.html

Which highlighting is missing for you?
Bash highlighting in the heredoc? Or PHP highlighting?

I'm missing php highlighting in a heredoc. Check out attached screenshot.

Please try
`
#language=php
/usr/bin/php << "PHPCODE"
<?php

$variable = 'value';
echo $variable;

PHPCODE
`

(Note the quotes around the heredoc start marker and the removed escapes). BashSupport currently only supports language injections into heredocs which do not evaluate variables (i.e. with quotes). Otherwise the rename variable refactoring would be broken because the injected language takes control of the snippet.
To fix this would need a lot of time which is out of scope atm. I hope that you can live with the quotes :)

Double quotes works! Thank you.

I have a related issue. I am not using the language injection, but I do need variable expansion inside the HEREDOC.
cat << _theUsage
line 1 of some text
with variable expansion for ${USER}
last line
_theUsage
The editor starts highlighting everything from "line 1" and never stops.
Since I need variable expansion, I cannot use the quotation mechanism.
Is this worthy of revisiting this ticket, creating a new ticket, or should I just expect this will never get fixed?

@Naruki Thanks for the report. This looks like a parsing problem and will be fixed for the next update. I've created #325 to track your issue.

I was able to get this working by disabling interpolation ('EOM') in the heredoc like this:
# language=BASH
cat > /usr/bin/a2disconf <<- 'EOM'
  #!bin/bash
  SOMEVAR="asdf"
  echo "${SOMEVAR}"
EOM


Add perPage param and disable page, "next" and "last" links accordingly

I.e. if next page is beyond the actual count of items.

I can also update the index.html accordingly if you would like to accept this PR.


make weavedns the name resolver for containers by default

Right now, containers started with weave run only use weavedns for name resolution when the --with-dns option has been specified. We did that because weavedns used to lack some key features required to act as a nameserver in a fair few cases. That has changed now, and hence that choice should be inverted, i.e. we should have a --without-dns option, though, arguably, only after #382.
NB: we'd want to avoid an ordering dependency here; i.e. it should be possible to start containers with dns pointing at weavedns even when weave/weavedns isn't running.

NB: we'd want to avoid an ordering dependency here; i.e. it should be possible to start containers with dns pointing at weavedns even when weave/weavedns isn't running.

One way to do that is to check whether it's running, and run it if not. Are there other ways?

One way to do that is to check whether it's running, and run it if not.

We don't do that for weave.

Are there other ways?

I want to preserve what is possible now; there is nothing stopping you from weave run --with-dnsing a container when neither weave or weavedns are running.

+1

it should be possible to start containers with dns pointing at weavedns even when weave/weavedns isn't running.

While that is possible currently, doing so by default strikes me as dangerous; seems way to easy to accidentally start containers pointing at a non-existing (and never-to-exist) weavedns server.
So, how about the default behaviour is to point at weavedns if it is running. And have --with-dns and --without-dns options to override that choice.
The proxy's behaviour should be changed to match.

Should this issue be solved after #382? If so, I guess #382 should have the same milestone...

Should this issue be solved after #382?

No.


Updates for ITKv5.2 code cleanups ... ROUND1

There are some build errors:
cd /Users/travis/build/InsightSoftwareConsortium/ITKSubdivisionQuadEdgeMeshFilter/_skbuild/macosx-10.6-intel-2.7/cmake-build/Wrapping/Modules/SubdivisionQuadEdgeMeshFilter && env SDKROOT=/Applications/Xcode-9.4.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk MACOSX_DEPLOYMENT_TARGET=10.9 /Users/travis/build/InsightSoftwareConsortium/ITKSubdivisionQuadEdgeMeshFilter/_skbuild/macosx-10.6-intel-2.7/cmake-build/Wrapping/Generators/CastXML/castxml/bin/castxml -o /Users/Kitware/Dashboards/ITK/ITKPythonPackage/scripts/../ITK-2.7-macosx_x86_64/Wrapping/itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilter.xml --castxml-gccxml --castxml-start _wrapping_ --castxml-cc-gnu "(" /Applications/Xcode-9.4.1.app/Contents/Developer/usr/bin/g++ -std=c++11 ")" -w -c @/Users/Kitware/Dashboards/ITK/ITKPythonPackage/scripts/../ITK-2.7-macosx_x86_64/Wrapping/SubdivisionQuadEdgeMeshFilter.castxml.inc /Users/Kitware/Dashboards/ITK/ITKPythonPackage/scripts/../ITK-2.7-macosx_x86_64/Wrapping/itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilter.cxx
/Users/Kitware/Dashboards/ITK/ITKPythonPackage/scripts/../ITK-2.7-macosx_x86_64/Wrapping/itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilter.cxx:27:18: error: too few template arguments for class template 'TriangleEdgeCellSubdivisionQuadEdgeMeshFilter'
    typedef itk::TriangleEdgeCellSubdivisionQuadEdgeMeshFilter< itk::QuadEdgeMesh< double,2 > > itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilterQEMD2;
                 ^
/Users/travis/build/InsightSoftwareConsortium/ITKSubdivisionQuadEdgeMeshFilter/include/itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilter.h:33:7: note: template is declared here
class TriangleEdgeCellSubdivisionQuadEdgeMeshFilter
      ^
/Users/Kitware/Dashboards/ITK/ITKPythonPackage/scripts/../ITK-2.7-macosx_x86_64/Wrapping/itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilter.cxx:28:18: error: too few template arguments for class template 'TriangleEdgeCellSubdivisionQuadEdgeMeshFilter'
    typedef itk::TriangleEdgeCellSubdivisionQuadEdgeMeshFilter< itk::QuadEdgeMesh< double,2 > >::Pointer itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilterQEMD2_Pointer;
                 ^
/Users/travis/build/InsightSoftwareConsortium/ITKSubdivisionQuadEdgeMeshFilter/include/itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilter.h:33:7: note: template is declared here
class TriangleEdgeCellSubdivisionQuadEdgeMeshFilter
      ^
/Users/Kitware/Dashboards/ITK/ITKPythonPackage/scripts/../ITK-2.7-macosx_x86_64/Wrapping/itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilter.cxx:30:18: error: too few template arguments for class template 'TriangleEdgeCellSubdivisionQuadEdgeMeshFilter'
    typedef itk::TriangleEdgeCellSubdivisionQuadEdgeMeshFilter< itk::QuadEdgeMesh< double,3 > > itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilterQEMD3;
                 ^
/Users/travis/build/InsightSoftwareConsortium/ITKSubdivisionQuadEdgeMeshFilter/include/itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilter.h:33:7: note: template is declared here
class TriangleEdgeCellSubdivisionQuadEdgeMeshFilter
      ^
/Users/Kitware/Dashboards/ITK/ITKPythonPackage/scripts/../ITK-2.7-macosx_x86_64/Wrapping/itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilter.cxx:31:18: error: too few template arguments for class template 'TriangleEdgeCellSubdivisionQuadEdgeMeshFilter'
    typedef itk::TriangleEdgeCellSubdivisionQuadEdgeMeshFilter< itk::QuadEdgeMesh< double,3 > >::Pointer itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilterQEMD3_Pointer;
                 ^
/Users/travis/build/InsightSoftwareConsortium/ITKSubdivisionQuadEdgeMeshFilter/include/itkTriangleEdgeCellSubdivisionQuadEdgeMeshFilter.h:33:7: note: template is declared here
class TriangleEdgeCellSubdivisionQuadEdgeMeshFilter
      ^
4 errors generated.

as well as
C:\Python35-x64\python.exe : CMake Warning in wrapping/CMakeLists.txt:
At C:\projects\itksubdivisionquadedgemeshfilter\windows-download-cache-and-build-module-wheels.ps1:18 char:1
+ C:\Python35-x64\python.exe C:\P\IPP\scripts\windows_build_module_whee ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (CMake Warning i...CMakeLists.txt::String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

  The object file directory
    C:/projects/itksubdivisionquadedgemeshfilter/_skbuild/win-amd64-3.5/cmake-build/Wrapping/Modules/SubdivisionQuadEdgeMeshFilter/CMakeFiles/SubdivisionQuadEdgeMeshFilterPython.dir/
  has 178 characters.  The maximum full path to an object file is 250
  characters (see CMAKE_OBJECT_PATH_MAX).  Object file
    itkModifiedButterflyTriangleCellSubdivisionQuadEdgeMeshFilterPython.cpp.obj
  cannot be safely placed under this directory.  The build may not work
  correctly.

which is a side-effect of build infrastructure and long names in this module.


I Can't find rncryptor.h

what shoud i do , I can't find this file " rncryptor.h"

This project is in Swift. Why do you believe you need a .h file?

@rnapier Because the docs specifically say to use RNCryptor.h.


tests: increase zookeeper discovery retry count in CI

Zookeeper tests seems a bit flaky since we added more travis instances or since the changes in node management. Because we have a variable timer now to connect to the Agents.
This increases the retry count to match the ones of other tests to give a bit more window to discover the nodes. Let's try with 20 for now.
Signed-off-by: Alexandre Beslic<EMAIL_ADDRESS>

LGTM

LGTM

Let's wait for Jenkins to trigger first before merging

Job: Swarm-PRs (engine master) [FAILED](https://jenkins.dockerproject.org/job/Swarm-PRs (engine master)/1923/console):

---
- docker network rm
ok 161 mesos - docker network disconnect connect
ok 162 mesos - docker ps
ok 163 mesos - docker run with wrong user
ok 164 mesos - docker run
ok 165 mesos - docker run short lived
ok 166 mesos - docker run no resources
ok 167 mesos - docker run with long pull
ok 168 mesos - docker stats
ok 169 mesos - docker-compose ps
ok 170 mesos - docker-compose scale
ok 171 mesos - docker-compose up
ok 172 mesos - zookeeper
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[Swarm-PRs (engine master)] $ /bin/sh -xe /tmp/hudson6374749054725701484.sh
+ set +x

---

Job: Swarm-PRs (engine 1.9) [FAILED](https://jenkins.dockerproject.org/job/Swarm-PRs (engine 1.9)/87/console):

---
- docker network rm
ok 161 mesos - docker network disconnect connect
ok 162 mesos - docker ps
ok 163 mesos - docker run with wrong user
ok 164 mesos - docker run
ok 165 mesos - docker run short lived
ok 166 mesos - docker run no resources
ok 167 mesos - docker run with long pull
ok 168 mesos - docker stats
ok 169 mesos - docker-compose ps
ok 170 mesos - docker-compose scale
ok 171 mesos - docker-compose up
ok 172 mesos - zookeeper
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[Swarm-PRs (engine 1.9)] $ /bin/sh -xe /tmp/hudson3761714935622952711.sh
+ set +x

---

not ok 146 refresh loop detects failure failed in conjunction to not ok 52 zk discovery: node removal. Do not merge. This needs more investigation.

Job: Swarm-PRs (engine 1.9) [FAILED](https://jenkins.dockerproject.org/job/Swarm-PRs (engine 1.9)/89/console):

---
078cc3a3edf37bd88fd356a6086c5450fca66706c
# Stopping e4758dacb4f277421e8ce13ca7cc833ed61832821b19d029182b974f04ece15d
# Stopping 7b65e6621cbd935c1cce83323fe6a241783d19dc49f175334894151f791c8b74
# Stopping 83a147b472f80e1923bc5a5efadd0962b7907fc879218832bb065d34404a027e
# Stopping 36953bf736d11c0df5f8c706fc7fb6244c9692ab0b4fb94847c885586a250705
ok 170 mesos - docker-compose scale
ok 171 mesos - docker-compose up
ok 172 mesos - zookeeper
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[Swarm-PRs (engine 1.9)] $ /bin/sh -xe /tmp/hudson6692225103887005559.sh
+ set +x

---

Job: Swarm-PRs (engine master) [FAILED](https://jenkins.dockerproject.org/job/Swarm-PRs (engine master)/1944/console):

---
- docker network rm
ok 163 mesos - docker network disconnect connect
ok 164 mesos - docker ps
ok 165 mesos - docker run with wrong user
ok 166 mesos - docker run
ok 167 mesos - docker run short lived
ok 168 mesos - docker run no resources
ok 169 mesos - docker run with long pull
ok 170 mesos - docker stats
ok 171 mesos - docker-compose ps
ok 172 mesos - docker-compose scale
ok 173 mesos - docker-compose up
ok 174 mesos - zookeeper
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[Swarm-PRs (engine master)] $ /bin/sh -xe /tmp/hudson2524926071229860459.sh
+ set +x

---


This wasn't possible in the past because we didn't get my collection artwork id, but we have it now.
https://github.com/artsy/eigen/assets/11945712/99685735-a198-46db-8230-84197e04683a
Unfortunately, the experience is not ideal currently because we have issues with images processing. This is why I suggest keeping this PR as a draft because it will worse the UX

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

go to my collection artwork post-submission - mounir

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (go to my collection artwork post-submission - mounir)

  Generated by :no_entry_sign: dangerJS against afc7a4ffffb2affaace7a80e527624f052ca77d0

@olerichter00 I agree- I asked in #dev-help if there is a way to create a new Sidekiq queue for these photos and maybe then we won't have to deal with this. I am waiting for an answer then I will adjust my approach here


Resolve custom editor with canonical resource

Close #146732
In textModelResolverService will return a canonical URI which leads to different URI in extension host.

@mjbvz Could you take a look at this PR? Thanks!


Plugin 'iissite' could not be found. Press enter to exit.

hi
i have downloded version <IP_ADDRESS>06
when i try to use each of plugin Arguments like "iissite" ,  "iisbinding"  and ... the following error occurs and say plugin could not be found:
Plugin 'iissite' could not be found. Press enter to exit.

1.9.3 is a pretty old version, there are a lot of command line options that have changed in the past year. I'd recommend getting the latest version.

i have the lastet release 1.9.10  but when i type letsencrypt.exe --version it returns : 'Let's Encrypt Simple Windows Client <IP_ADDRESS>06'
but when i see file properties of  letsencrypt.exe file the version is <IP_ADDRESS>
and my problem exist about command lines and none of plugin Arguments could not be found
what is the problem?

You're definitely not running the executable that you downloaded and are checking the file properties of. Perhaps you have an older version of the tool in your system path.

i know what i am running very well!
i never downloaded any old version previously that i can run!
i have downloaded the lastest version a few days ago and cd to currect folder and run that! and before that time i never downloaded letsencrypt!

I'm with @WouterTinus on this. Something is fishy with your download:

If you didn't download another version then someone else did. I'm going to close this because we can't help any further if you don't want to accept that you're not running the right .exe..
A tip might be to rename/delete the file that you downloaded and then see if the "letsencrypt" command still works on your system. Then through the task manager you should be able to figure out where the image of the old version is located.

Tip: on windows you can use the 'where' command to find where an exe in your path is coming from. So if you type 'where letsencrypt' you may find that it's not where you expected it to be (perhaps in system32 etc)

to expand on that pro tip: if you're in a powershell shell, you have to use where.exe since where is an alias for the where-object cmdlet


feat: use github app for VHD automation tasks (#5379)

What type of PR is this?
/kind chore
What this PR does / why we need it:
Cherry picks two commits from dev branch so we can build new Linux and Windows VHDs from master
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

looks like I forgot to do this, thanks!


Index finding performance

I have been looking at faster ways to perform index searching for a non-continuous cell ID indexes (as in the case of DCCRG with AMR). Basically 3 ways now:
a = [findfirst(==(id), cellid) for id in ids] # currently using in v0.8.5
b = findfirst.(.==(ids), (cellid,)) # found in discourse
c = indexin(ids, cellid) # Base.array method

The 3rd one is pretty interesting: it first convert the array into a dictionary and then perform the search. Even though it uses more memory, it is faster especially for larger sizes:
using BenchmarkTools

function myindexin(a, b::AbstractArray)
   inds = keys(b)
   bdict = Dict{eltype(b),eltype(inds)}()
   for (val, ind) in zip(b, inds)
       get!(bdict, val, ind)
   end
   [ get(bdict, i, nothing) for i in a ]
end

N = 100
cellid = collect(1:N)
ids = rand(cellid, N ÷ 10)

a = [findfirst(==(id), cellid) for id in ids]
b = findfirst.(.==(ids), (cellid,))
c = indexin(ids, cellid)
d = myindexin(ids, cellid)

println(a==b==c==d)

@btime [findfirst(==(id), $cellid) for id in $ids]

@btime findfirst.(.==($ids), ($cellid,))

@btime indexin($ids, $cellid)

@btime myindexin($ids, $cellid)

println("")

# N = 100
  361.950 ns (2 allocations: 192 bytes)
  1.131 μs (7 allocations: 448 bytes)
  2.007 μs (11 allocations: 6.75 KiB) # ~0.15
  1.985 μs (12 allocations: 6.77 KiB)
# N = 1000
  24.979 μs (2 allocations: 928 bytes)
  26.191 μs (7 allocations: 1.16 KiB)
  20.650 μs (19 allocations: 93.30 KiB) # ~1
  21.007 μs (20 allocations: 93.22 KiB)
# N = 10000
  2.452 ms (2 allocations: 7.97 KiB)
  2.541 ms (7 allocations: 8.22 KiB)
  218.157 μs (25 allocations: 373.55 KiB) # 10x faster
  219.673 μs (26 allocations: 372.58 KiB)
# N = 100000
  248.996 ms (3 allocations: 78.23 KiB)
  260.640 ms (8 allocations: 78.48 KiB)
  2.783 ms (38 allocations: 5.76 MiB) # 100x faster
  2.771 ms (39 allocations: 5.75 MiB)

There are 2 places where this is been used:

value extraction from a selection of cells;
VTK conversion.


Implementation of get_policy (IAM) to the moto library

Hello all,
I am in progress of implement get_policy (IAM) to moto library.
When I execute a test with some python script that use "get_policy", I get the following error message:
TypeError: get_policy() got an unexpected keyword argument 'policyarn'
Here is the changes I applied, after I cloned the moto library to my local environment:
..tests/test_iam.py:
@mock_iam
def test_get_policy():
conn = boto3.client('iam', region_name='us-east-1')
conn.create_policy(
PolicyName="TestGetPolicy",
PolicyDocument='{"some":"policy"}')
version = conn.create_policy_version(
PolicyArn="arn:aws:iam::aws:policy/TestGetPolicy",
PolicyDocument='{"some":"policy"}')
with assert_raises(ClientError):
conn.get_policy(
PolicyArn="arn:aws:iam::aws:policy/TestGetPolicy",
retrieved = conn.get_policy(
PolicyArn="arn:aws:iam::aws:policy/TestGetPolicy",
VersionId=version.get('PolicyVersion').get('VersionId'))
retrieved.get('Policy').get('Document').should.equal({'some':'policy'})
..moto/iam/responses.py:
def get_policy(self):
    policyarn = self._get_param('PolicyArn')
    policyarn = iam_backend.get_policy(
        policyarn = policyarn)
    template = self.response_template(GET_POLICY_TEMPLATE)
    return template.render(policyarn = policyarn)

Regarding "..moto/iam/models.py", it looks like get_policy entries were already implemented (it is the same file as the current one on moto library).
Any help / comment would be appreciated,
Thanks,

@josenrihernand Thanks for working on this!
The problem you're running into is that the current implementation of get_policy in the IAM Moto backend gets the policy by name: https://github.com/spulec/moto/blob/master/moto/iam/models.py#L471
You'll want to extend that to get it by name or by arn (or create a new get_policy_by_arn method, if that feels better). Each policy has a .arn propery on it so you can list the policies and find the one with a matching arn and return it.

If you push your work on a pull request and mark the PR as 'WIP' (work in progress) then we'll be able to run your actual failing tests locally to help debug

Hi @JackDanger,
Thanks a lot to take a moment, and review it !
I'll summarize the advance so far about this issue:
The most recent adjustment I have applied on the code, follows:
Template GET_POLICY_TEMPLATE on ../moto/iam/responses.py:
GET_POLICY_TEMPLATE = """

{{ policyarn.document }}
{{ policyarn.default_version_id }}
2017-10-04T15:45:35Z

20f7279f-99ee-11e1-a4c3-27EXAMPLE804

"""
get_policy function on ../moto/iam/responses.py:
def get_policy(self):
    policyarn = self._get_param('PolicyArn')
    policyarn = iam_backend.get_policy(
        policyarn)
    template = self.response_template(GET_POLICY_TEMPLATE)
    return template.render(policyarn = policyarn)

test_get_policy function on ../tests/test_iam.py
@mock_iam
def test_get_policy():
conn = boto3.client('iam', region_name='us-east-1')
conn.create_policy(
PolicyName="TestGetPolicy",
PolicyDocument='{"some":"policy"}')
version = conn.create_policy_version(
PolicyArn="arn:aws:iam::aws:policy/TestGetPolicy",
PolicyDocument='{"some":"policy"}')
with assert_raises(ClientError):
conn.get_policy(
PolicyArn="arn:aws:iam::aws:policy/TestGetPolicy",
retrieved = conn.get_policy(
PolicyArn="arn:aws:iam::aws:policy/TestGetPolicy",
VersionId=version.get('PolicyVersion').get('VersionId'))
retrieved.get('Policy').get('Document').should.equal({'some':'policy'})
After these adjustments, when I execute my unittest & mock, the following code works (it did not work before, because the get_policy was not implemented. In this code, "list_attached_group_policies" was also recently implemented):
groupPolicies = iam_conn_client.list_attached_group_policies(
GroupName = groups["GroupName"]
for attachedPolicy in groupPolicies["AttachedPolicies"]:
currentPolicy = iam_conn_client.get_policy(
PolicyArn = attachedPolicy["PolicyArn"]
);
print("Attached Policy: ", attachedPolicy["PolicyArn"]);   --> it works.
--> ('Attached Policy: ', 'arn:aws:iam::aws:policy/TestDBPolicy')
print("Current Policy: ", currentPolicy);	--> it also works.
--> ('Current Policy: ', {u'Policy': {u'CreateDate': datetime.datetime(2017, 10, 4, 15, 45, 35, tzinfo=tzutc()), u'DefaultVersionId': ''}, 'ResponseMetadata': {'RetryAttempts': 0, 'HTTPStatusCode': 200, 'RequestId': '20f7279f-99ee-11e1-a4c3-27EXAMPLE804', 'HTTPHeaders': {u'Content-Type': u'text/plain', u'server': u'amazon.com'}}})
However, the next lines of my script fail, with the following error message:
groupPolicies = iam_conn_client.list_attached_group_policies(
GroupName = groups["GroupName"]
policy_version = iam_conn_client.get_policy_version(
PolicyArn = attachedPolicy["PolicyArn"],
VersionId = currentPolicy["Policy"]["DefaultVersionId"]
);
An error occurred (NoSuchEntity) when calling the GetPolicyVersion operation: Policy version not found
I don't know if these error is related to the code I added recently, to implement get_policy, or not.
It looks like the root cause, is because "DefaultVersionId" return no value (please check the output for "Current Policy" above).
I'll spend some time trying to fix it, if not, maybe I'll push a pull request with the mark "WIP"...
However, if by the time you read this input, you have any comments / suggestion, it'd be appreciated...
Again, thanks a lot !

Hi @JackDanger
What is the procedure for PR ?
Should i create a new branch on moto repo ? or should i push the moto repo on my own repo ?
thanks a lot!

@josenrihernand You'll want to fork the repo and push a branch to your fork and then create a pull request from that branch. Only a few folks have permissions to push branches to Moto but anybody can create a pull request from a fork. Thanks!

ok JackDanger, I got it.
thanks !

Hi @JackDanger ,
I just create a PR from my fork, and also I describe a help request:   WIP Iam get policy #1240
I really appreciate your help (or from someone else), in order to implement get_policy and, also, something about a error messages i am getting with get_policy_version.
thanks a lot !

Closed with https://github.com/spulec/moto/pull/1759


simulate click "expected to have been called one time, but it was called two times"

Current behavior
I have a click handler on li's inside a ul:
import React, { Component } from "react";

class ItemList extends Component {
  constructor(props) {
    super(props);

    this.buildItems = this.buildItems.bind(this);
    this.delete = this.delete.bind(this);
  }

  buildItems(item) {
    return (

 this.delete(item.key)} key={item.key}>
        {item.text}

    );
  }

  delete(key) {
    this.props.delete(key);
  }

  render() {
    let todoItems = this.props.items.map(this.buildItems);

    return (

{todoItems}

    );
  }
}

export default ItemList;

test:
it("calls delete with correct key when item clicked", () => {
    const mockPropDelete = jest.fn();
    const deleteSpy = jest.spyOn(ItemList.prototype, "delete");
    let todoItems = [
      { key: 1234, text: "fold laundry" },
      { key: 4321, text: "mow lawn" }
    ];
    const wrapper = shallow(
      <ItemList items={todoItems} delete={mockPropDelete} />
    );
    const item = wrapper.find("li").at(1); //get item at index 1 since find 'li' will return more than 1 result

    item.simulate("click");

    expect(deleteSpy).toHaveBeenCalledTimes(1);
    expect(deleteSpy).toHaveBeenCalledWith(4321);
    expect(mockPropDelete).toHaveBeenCalledTimes(1);
  });

and I get the error "Expected mock function to have been called one time, but it was called two times."
Expected behavior
That my spy should only be called once. I feel like I'm missing something that I haven't been able to find searching around and I suspect this has to do with there being multiple li's.

Works for me. No errors.
Your ItemList component refactored:
`import React from 'react';
const ItemList = ({ items, deleteTodoItem }) => (

      {items.map(({ text, key}) => (

          {text}

      ))}

);
export default ItemList;`
Don't use delete as a method name. It's a reserved word in JasaScript.
Your code has 3 handlers to delete todo item:

one that you pass as prop to ItemList component;
one more redundant handler that you create in your component;
and one more that you pass to onClick property as arrow function with no args.

You can pass parameters using data attribute and access it in your event handler e.target.dataset.
As for me, class method that generates JSX is redundant. This approach makes your code overly complicated and difficult to test.

First off, thank you for your response and questions...I'm doing by best to teach myself React, Jest and Enzyme in my spare time outside of work so your questions are forcing me to look at things through a different lens which is absolutely appreciated! After reading through your questions it is clear that this component I wrote is overly complicated. My initial thought process was to pass in a function in this component's parent as a prop which would mutate the state. I created a "local" function within this component to handle the click and in turn call the function passed in as a prop...but obviously that is totally unnecessary and as you said, makes the code overly complicated. After refactoring the code to your example you provided and tweaking the tests it bit it is now working for me. So I guess my over-engineered solution was leading to side effects. Thank you again for helping me simplify this component and more importantly, helping me think through how to simplify this code because clearly, there was a better way.

One other question for you @voliev is there a way for me to access e.target.dataset within my JSX? Something along the lines of

This doesn't work, but what I'm trying to solve is a way to pass just the item's key to the deleteTodoItem  function as an argument

@bkrainock pass a function - onClick={(e) => { whatever(); }}

@ljharb thank you! Looks like I needed to mock my event with a target and dataset object, but it looks like I'm up and running and all my tests are passing. Thanks for pointing me in the right direction!

Huh that's weird...my tests pass, but when I run it in the browser there is no onClick on my li's. Am I missing something???

import React from "react";

const ItemList = ({ items, deleteTodoItem }) => (

      {items.map(({ text, key }) => (

 {
            deleteTodoItem(event.target.dataset);
          }}
          key={key}
          data-key={key}
        >
          {text}

      ))}

);

export default ItemList;

@bkrainock, if you set arrow function as a handler, it doesn't make sense to pass the event object to deleteTodoItem unless you really need it. Just onClick={() => deleteTodoItem(key)} is enough in your case.
In this example onClick={deleteTodoItem(event.target.dataset)} you are calling deleteTodoItem first and then passing it's return value as onClick handler.

@voliev thank you again for your assistance, you truly have been a life saver today! I was trying to figure out why my filter wasn't working in my HOC when deleteTodoItem was being called with the key. Just using a plain arrow function that calls deleteTodoItem(key) works as expected, and I was able to get my tests all passing. Thank you for your help it's much appreciated!


Fix box score when team has bye week

Should fix #50

Codecov Report

Merging #66 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #66      +/-   ##
==========================================
+ Coverage   91.37%   91.43%   +0.06%
==========================================
  Files          12       12
  Lines         545      549       +4
==========================================
+ Hits          498      502       +4
  Misses         47       47

Impacted Files
Coverage Δ

ff_espn_api/box_score.py
100% <100%> (ø)
:arrow_up:

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 1867aae...487817a. Read the comment docs.


Added Curve in Animation NodeTransition and StateMachine

Co-authored-by: jeronimo-schreyer<EMAIL_ADDRESS>Fixes https://github.com/godotengine/godot-proposals/issues/1402
Supersedes #63369
I can't get @jeronimo-schreyer's response to the review in #63369. However, I believe this is an important addition so I send this PR on his behalf in order to get it implemented into Godot 4 in time.
In the video example below, the difference may appear small, but the actual feel of the control is much better with Ease enabled.
Enable ease:
https://user-images.githubusercontent.com/61938263/182272494-a0a0e6cf-5d85-4b0c-8303-962b8df33d99.mp4
Disable ease:
https://user-images.githubusercontent.com/61938263/182272487-dcd9122f-8114-441e-a606-324e6c40ab1d.mp4

Hey @TokageItLab, really sorry I couldn't get my hands on this up until now. I understand the hurry. Happy that this is merged already. Thanks

Is not using a Curve a bit too much for this? Would it not be better use regular easing functions?

@reduz I guess is for finer control. Is a performance cost of using a curve?

Personally, I think easing is sufficient. But I also think it is necessary to implement the same amount of easing functions as for tweening.

I think it will allow users to fully customize state transitions instead of more static easing functions.
We discussed this feature a bit with @punto- around this video
https://youtu.be/LNidsMesxSE
so, I think the best use case for this feature is IK animation and this video kinda shows what this type of customization can achieve.
Its not perfect because you need to define your own easing function instead (a bit hard for Bounce and Elastic maybe). But that, I think, is something more related to the Curve. Like, should we make tools for the Curve to have presets similar to the ones available in Tween?

Like, should we make tools for the Curve to have presets similar to the ones available in Tween?

Presets are already available by right-clicking the curve in the inspector:

Well, for sure given what I saw in the video above and the presence of the Ease preset, a custom curve would make sense.
I would insist that this enhancement makes good sense because when we changed the blending algorithm previously, blending outside of the 0 to 1 range is now done correctly.
@reduz So I would like to push for this PR, how about it?

Yeah I guess what it adds is nice and it does not really hurt in either case.

Thanks!


Misc doc changes

List of changes:

Set readme as main html page
Add additional docs to html page
Fix "documentation reference file" warnings
Use common source url
Enable source reference
Fix code examples indentation
Add license year
Add module intro section
Badges and more badges!

@kianmeng Apologies for the long delay here, but if you're willing to rebase this on master and fix the conflicts, I'd be willing to accept it! :pray:

@danielberkompas No worries. I've updated as requested.

Thank you! :star:


feature request: press  escape  to stop

would be nice if pressing ESC would stop generating,
instead of having to click the button, which defocuses the input, and then having to click the input field again.
thanks!

Added to dev!


Preview doesn't show anything

I can't see anything like a level preview in the browser. Tried both Opera and Chrome, nothing.
But it does produce some kind of JSON file in the mesh directory if I click "Build navigation mesh"

Here's the server response I get when clicking "Load Level"
{"scene":{"metadata":{"version":4.5,"type":"Object","generator":"Object3D.toJSON"},"geometries":[{"uuid":"2fdf332c-2195-45ad-8ec6-4ea670c0c233","type":"BufferGeometry","userData":{"mergedUserData":[{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{}]},"data":{"attributes":{"position":{"itemSize":3,"type":"Float32Array","array":[-448,0,-64,-448,128,-64,-448,0,192,-448,128,192,-32,0,-448,-32,128,-448,-192,0,-448,-192,128,-448,224,0,112,224,128,112,224,0,32,224,128,32,128,0,112,128,128,112,-32,0,96,-32,128,96,-32,0,192,-32,128,192,0,0,32,0,128,32,0,0,96,0,128,96,-32,0,128,-32,128,128,0,0,128,0,128,128,0,0,192,0,128,192,128,0,192,128,128,192,-192,0,-64,-192,128,-64],"normalized":false}},"index":{"type":"Uint16Array","array":[0,1,2,2,1,3,4,5,6,6,5,7,8,9,10,10,9,11,12,13,8,8,13,9,14,15,4,4,15,5,2,3,16,16,3,17,18,19,20,20,19,21,22,23,24,24,23,25,10,11,18,18,11,19,16,17,22,22,17,23,24,25,26,26,25,27,26,27,28,28,27,29,28,29,12,12,29,13,30,31,0,0,31,1,6,7,30,30,7,31,20,21,14,14,21,15]}}},{"uuid":"43046b48-a0b9-495d-94ce-a25f9f3eff52","type":"BufferGeometry","userData":{"mergedUserData":[{},{},{}]},"data":{"attributes":{"position":{"itemSize":3,"type":"Float32Array","array":[-192,0,-64,-448,0,-64,-448,0,192,-32,0,192,-32,0,128,-32,0,96,-32,0,-448,-192,0,-448,0,0,128,0,0,96,0,0,192,128,0,192,128,0,112,224,0,112,224,0,32,0,0,32],"normalized":false}},"index":{"type":"Uint16Array","array":[0,1,2,2,3,4,5,6,7,0,2,4,4,5,7,7,0,4,5,4,8,8,9,5,8,10,11,12,13,14,14,15,9,9,8,11,12,14,9,9,11,12]}}}],"materials":[{"uuid":"bcdb9497-dbc8-4902-951a-e9fea4c8b5ae","type":"MeshBasicMaterial","color":1114146,"reflectivity":1,"refractionRatio":0.98,"depthFunc":3,"depthTest":true,"depthWrite":true,"colorWrite":true,"stencilWrite":false,"stencilWriteMask":255,"stencilFunc":519,"stencilRef":0,"stencilFuncMask":255,"stencilFail":7680,"stencilZFail":7680,"stencilZPass":7680},{"uuid":"e730b06b-ed55-4d2b-a30f-a7b1c3b44af8","type":"MeshBasicMaterial","color":1114146,"reflectivity":1,"refractionRatio":0.98,"side":2,"depthFunc":3,"depthTest":true,"depthWrite":true,"colorWrite":true,"stencilWrite":false,"stencilWriteMask":255,"stencilFunc":519,"stencilRef":0,"stencilFuncMask":255,"stencilFail":7680,"stencilZFail":7680,"stencilZPass":7680}],"object":{"uuid":"06b63b17-9fb3-45d8-a32b-971ef067d43e","type":"Scene","layers":1,"matrix":[1,0,0,0,0,1,0,0,0,0,1,0,0,0,0,1],"children":[{"uuid":"e35aed51-2e1d-42f8-abdc-3affb355b6dd","type":"Object3D","layers":1,"matrix":[0.015625,0,0,0,0,0.015625,0,0,0,0,0.015625,0,0,0,0,1],"children":[{"uuid":"44d9bb3d-4be9-4132-b5ab-f2d4d80a608a","type":"Object3D","name":"walls.preview","layers":1,"matrix":[1,0,0,0,0,1,0,0,0,0,1,0,0,0,0,1],"children":[{"uuid":"d5b35c7f-2cd7-48da-b9c2-da99c6769820","type":"Mesh","name":"walls.preview","layers":1,"matrix":[1,0,0,0,0,1,0,0,0,0,1,0,0,0,0,1],"geometry":"2fdf332c-2195-45ad-8ec6-4ea670c0c233","material":"bcdb9497-dbc8-4902-951a-e9fea4c8b5ae"}]},{"uuid":"8783a2ce-1490-4796-9878-4534a590abf1","type":"Mesh","name":"floors","layers":1,"matrix":[1,0,0,0,0,1,0,0,0,0,1,0,0,0,0,1],"geometry":"43046b48-a0b9-495d-94ce-a25f9f3eff52","material":"e730b06b-ed55-4d2b-a30f-a7b1c3b44af8"},{"uuid":"94fbce0f-ad53-4e28-936d-6ced157287ce","type":"Object3D","name":"nav.nodes","layers":1,"matrix":[1,0,0,0,0,1,0,0,0,0,1,0,0,0,0,1]}]}]}},"config":{"triangulation_algorithms":["libtess","earcut","delaunay","smallest"],"triangulation":"libtess","options":{"cellSize":0.25,"cellHeight":0.1,"agentHeight":1,"agentRadius":0.5,"agentMaxClimb":0.30000000000000004,"agentMaxSlope":40,"regionMinSize":12,"regionMergeSize":32,"edgeMaxLen":16,"edgeMaxError":2.5},"merge_distance":1,"solo":true}}

and the browser console shows:

server messages when clicking load level:

and the map I tested it on:
MAP01.zip

hmm, the path names need to be full file paths, can you confirm that's the case?

I configured full paths and restarted the server, behaves exactly as before. Is there any other kind of information or test I can try?
I also tried that on Firefox, Edge, Chrome, Opera... still nothing.

I'll dig into this more when I am at my computer.  I'll take a closer look at this later, the scene object that you posted, however, does not appear to contain any geometry data in it. This tells me that the problem is not in the browser, but that's something with the server or the parser is messing up.

@MrRaveYard I was not able to replicate the issue. it works just fine for me. Did you click the 'load level' button at the top? Also, you can use WASD controls to fly around. try looking around and flying around to see if your level is visible anywhere.

closing this because I am unable to replicate the issue.

@disasteroftheuniverse  Ah, thanks. It was the WASD to move around...


could not be converted to string

Hi guy's, I'm trying to simply call this function.
activity()->log('Look mum, I logged something');
but i have this error

ErrorException (E_RECOVERABLE_ERROR)
Object of class Illuminate\Auth\GenericUser could not be converted to string

can anyone help me?

Hey,
could you provide a full stack trace? Without we can't know where this error is thrown.
But if I have to guess I would say that the default user resolver used for the causer has an error.

you can find full stack trace here
Crash.txt
Thanks for answering

Hey,
if I got your stack trace right the exception is thrown in laravel core logic. The user provider generates an instance of \Illuminate\Auth\GenericUser which is wrongly passed into Illuminate\Database\QueryException:formatMessage() and there it's treated like string but doesn't contain a __toString() method.
Because there are no method parameters listed you should try to debug the part from #62 to #68 in your stack trace.

@Gummibeer
i figured out my mistake and fixed the problem by changing provider for

config -> auth.php

'providers' => [         'user' => [             'driver' => 'eloquent',             'model' => \App\Models\User::class,         ], //        'user' => [ //            'driver' => 'database', //            'table' => 'user', //            'model' => \App\Models\User::class, //        ],     ],
ty for answering.


fix: make sure that a MeasurableSpace instance uses Discreteness

The discreteness assumptions were intended, but not used: they are now.
Zulip

Thanks!
bors r+


Incorporate Adobe Spectrum Web Components into the Extension

The Markdown Preview currently uses hand-coded CSS for styling.  Now that we include more complex elements like support for tabs, it makes sense to add support for the Adobe Spectrum Design System.
Spectrum provides several paths to integration, but the one that makes sense for this project seems to be the Spectrum Web Components.

Added a preview script to the package configuration. Now loads  Adobe Spectrum CSS for interactive Tabs and Themes.
    "markdown.previewScripts": [
      "./dist/preview/index.bundle.js"
    ],


🐛 BUG: Language server not starting properly in non-VSCode editors if prettier or the prettier plugin are missing

Describe the Bug
Hey everyone,
this is Christian from the WebStorm team at JetBrains and we have been working on integrating the Astro LS for our most recent version but noticed an unexpected issue close to release that we'd like to raise with you in order to discuss whether you would consider fixing it on your end before we build some unconventional workarounds.
During the startup of the language server it attempts to load prettier and the prettier plugin for the purposes of initializing the Volar services:
https://github.com/withastro/language-tools/blob/6fd9314b22b12a19ec104f19ce4ad5e3856a2ece/packages/language-server/src/languageServerPlugin.ts#L98-L107
However, if no instance of prettier was found then the upstream volar-service-prettier will throw an error and prevent the Astro LS from starting at all. While it is true that this is fixed once the user installs those two peer dependencies (prettier and the plugin) explicitly into their project, this is not necessary when using VSCode. The readme does mention this, but I had the assumption this formatting is optional and not required.
Seemingly, the VSCode Astro plugin is silently (and unconventionally) installing the LS into the node_modules together with prettier and the plugin and thus the LS is able to pick up those dependencies. I came to this conclusion after creating a fresh Astro project that caused the LS to fail in WebStorm and noticed that opening it once in VSCode and then again in WebStorm causes no more issues until I noticed that the node_modules folder has been changed.
I would like to propose to make the detection and inclusion of the prettier service more robust and optional, i.e. making the following function call be handled more gracefully or the implementation check whether the dependencies exist in the first place before attempting to initialize the Volar service. Calling the create function from volar-service-prettier with no instance causes the failure to start.
https://github.com/withastro/language-tools/blob/6fd9314b22b12a19ec104f19ce4ad5e3856a2ece/packages/language-server/src/languageServerPlugin.ts#L55
I'm open for discussions and willing to contribute the necessary changes to handle this use-case more gracefully than before if there are no objections.
Steps to Reproduce

npm init astro
Open the project outside of VSCode in any editor/IDE that supports the language server, but hasn't prettier or prettier-plugin-astro installed
The LS is not starting at all

After a quick conversation with @Princesseuh we noticed that this issue has been fixed last week as part of upstream changes that were done in https://github.com/volarjs/services/pull/84 and then pulled in via https://github.com/withastro/language-tools/pull/823. The problems are no longer occurring in newer versions of the LS.


fix worklist template to be compatible with @ui5/cli v2

fix for #882

Thx @devinea . Hopefully this can be reverted shortly.

@IainSAP Thanks. Added revert to the tasks here https://github.com/SAP/open-ux-tools/issues/904

Thanks all


deleted npminstall.ts for simplicity

@t-jakurl,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


The veil.... IS.... TORN!!!

About the PR
Makes the Nar'sie spawn animation a bit more fancy and more like the one on ParadiseStation 13.
Why / Balance
It looks nicer, has more depth to it than the previous one.
Technical details
Attribution changes, a couple of sound file changes and additions, and the main code changes for narsie.yml
Media
https://github.com/user-attachments/assets/418d2f57-03d4-404e-bddb-a3ebaca14de4

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

dw the rest of the checks passed last time


[CI][GHActions] Introduce GitHub Actions Iroha1

Description of the Change
Partially clozes #948
Benefits
Checks with GitHub Actions

@BulatSaif Could you please take a look ones more and approve this PR

@BulatSaif Could you please take a look ones more and approve this PR

I will check it tomorrow morning, meanwhile, can you fix failed builds, I see it has problems cleaning VM.

I reviewed PR,  big work done!
Did we achieved goals?

It is faster than Jenkins ci.
It is easy to maintain by developers.

I couldn't check the first one because the build is failing currently.

As I see the level of complexity remains the same (my subjective opinion), I believe that more developers know YAML than groovy, but in your code, you used advanced YAML (Anchors and Aliases) which negates all the simplicity of YAML. You can reduce the complexity by splitting action into multiple files and adding more documentation on how to edit and test GH Actions. But if it satisfies the developers I am fine with what.

I will approve pr when my previous comments are  applied:

can you fix failed builds, I see it has problems cleaning VM.
please comment line 213 and 223 in https://github.com/hyperledger/iroha/blob/main/Jenkinsfile to disable yellow status

@BulatSaif

Yes it is faster ~= 22min

1.a. Build currently is falling due to problem in runner configuration, I notified @ravenolf

At the moment the only developer works on CI side is @kuvaldini, others are aware how to work the that workflow but not deeply involved

/build ubuntu release ursa

/build abracadabra

no reaction from CI

/build abracadabra

/build normal ubuntu release

/build ubuntu gcc10 release debug
/build ubuntu clang

/build ubuntu clang release

/build ubuntu clang release

/build ubuntu clang release

/build ubuntu clang release

/build ubuntu clang release

/build ubuntu clang release

/build ubuntu clang release

/build ubuntu clang release


Error: Stream yields empty buffer

node: v7.10.0
npm: v5.0.0
❯ smartcrop --width 100 --height 100 avatar.jpg
Error: Stream yields empty buffer
    at Socket.<anonymous> (/usr/local/lib/node_modules/smartcrop-cli/node_modules/gm/lib/command.js:57:17)
    at emitNone (events.js:91:20)
    at Socket.emit (events.js:188:7)
    at endReadableNT (_stream_readable.js:975:12)
    at _combinedTickCallback (internal/process/next_tick.js:80:11)
    at process._tickCallback (internal/process/next_tick.js:104:9)

Hello Alex,
I can't reproduce this issue. Can you please provide me the image in question and the version of image magick that you are using (convert --version)?

I found the issue. Running convert --version gave me an error since my globally set version of Ruby didn't have it. Looks like my last upgrade was what caused the trouble.
Here's my $PATH:
.git/safe/../../bin
/Users/alexbaldwin/.rbenv/shims
/Users/alexbaldwin/.bin
/usr/local/sbin
/usr/local/bin
/usr/bin
/bin
/usr/sbin
/sbin
/opt/X11/bin
/usr/local/MacGPG2/bin
/Users/alexbaldwin/.fzf/bin

Any ideas how I can force /usr/local/bin to take priority over RVM?

Hello Alex,
I'm not sure what rbenv has got to do with smartcrop. But the priority of the paths in $PATH is defined by the order so just change that and you should be good to go.
Cheers,
Jonas

Windows 7 x64, Node 20.2.0
$ smartcrop --width 100 --height 100 out.jpg out-t.jpg
Error: Stream yields empty buffer
    at Socket.<anonymous> (C:\APPS\08-Разработка\Node\node_modules\smartcrop-cli\node_modules\gm\lib\command.js:56:21)
    at Socket.emit (node:events:523:35)
    at endReadableNT (node:internal/streams/readable:1367:12)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)

@sergeevabc what magick version are you running?

@jwagner, the latest one. Point is that Windows has its own convert utility, which has a higher priority, so the command magick or magick convert is used instead.
$ where convert.exe
C:\Windows\System32\convert.exe
C:\Apps\Media\IM\convert.exe

Could you check

var gm = require('gm').subClass({ imageMagick: '7+' });

and
var gm = require('gm').subClass({
  appPath: String.raw`C:\Apps\Media\IM\magick.exe`
});

did not bring relief, the error is the same
$ smartcrop --width 100 --height 100 in.png out.png
Error: Stream yields empty buffer
    at Socket.<anonymous> (C:\Node\node_modules\smartcrop-cli\node_modules\gm\lib\command.js:56:21)
    at Socket.emit (node:events:523:35)
    at endReadableNT (node:internal/streams/readable:1367:12)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)

Does the GM package work for you (outside of smartcrop-cli)?

I installed the package and tried a basic command, but there was no output at all.
$ npm install gm -g

$ node -e "const gm = require('gm').subClass({appPath: String.raw`C:\im\magick.exe`}); gm('c:\im\in.png').identify()"

At least the following one still works as expected
$ node -e "console.log('Hello, world!')"
Hello, world!

To get some output you need to log something:
.identify(function (err, data) {
  console.log(err, data)
});

'c:\im\in.png'  likely also won't work, because the \i acts as escape sequence. Using String.raw`content`  prevents that.

Some good news, Jonas.
var gm = require('gm').subClass({
  appPath: String.raw`C:\im\magick.exe`
});

gm(String.raw`c:\in.png`).identify(function (err, data) {
  console.log(err, data)
});

threw an error as follows
Error: Could not execute GraphicsMagick/ImageMagick: C:\im\magick.exegm "identify" "-ping" "-verbose" "c:\in.png" this most likely means the gm/convert binaries can't be found
    at ChildProcess.<anonymous> (C:\Node\node_modules\gm\lib\command.js:249:12)
    at ChildProcess.emit (node:events:511:28)
    at cp.emit (C:\Node\node_modules\gm\node_modules\cross-spawn\lib\enoent.js:36:37)
    at ChildProcess._handle.onexit (node:internal/child_process:293:12) {}

However, this one finally worked.
var gm = require('gm').subClass({
  imageMagick: '7+'
});

gm(String.raw`c:\in.png`).identify(function (err, data) {
  console.log(err, data)
});

Alas, I don't know what to do next, because we've already tried to put imageMagick: '7+' in smartcrop-cli.js


[PT Run] Add option to quickly jump between Virtual Desktops

Description of the new feature / enhancement
Implement a feature in PowerToys Run that allows users to jump to a specific Virtual Desktop directly by entering a query like "Virtual Desktop 1" or "Virtual Desktop NameOfVirtualDesktop".
This functionality will streamline the process of navigating among various virtual desktops, enhancing user efficiency and workflow management. By just typing a simple command in PowerToys Run, users can swiftly transition to the desired virtual desktop, making multitasking and organization more effortless and intuitive. This feature embodies a user-centric approach to workspace management, fostering a more productive and user-friendly environment.
Scenario when this would be used?
Scenario: Multiple Project Management
Suppose a user is managing multiple projects and has organized each project within its own virtual desktop to maintain a clear and focused workspace. They are currently working on Project A in "Virtual Desktop 1" but need to quickly reference some information from Project B located in "Virtual Desktop 2".
Instead of manually navigating through the virtual desktop interface, with the proposed feature, the user can swiftly transition to "Virtual Desktop 2" by activating PowerToys Run (using the Alt+Space shortcut), typing "Virtual Desktop 2", and hitting enter. This action immediately takes them to "Virtual Desktop 2" where Project B is organized, allowing for a quick reference.
After obtaining the necessary information, the user can similarly use PowerToys Run to type "Virtual Desktop 1" and return to Project A with minimal disruption in their workflow, thus saving time and maintaining their focus on the task at hand. This feature enhances the user experience by providing a quick, intuitive, and efficient method to manage and navigate multiple projects across different virtual desktops.
Supporting information
No response

/dup #23572


Allow HTTP Agent options to be configured for uplink

Type: feature
This allows HTTP Agent options to be configured for an uplink. This gives users control over settings like maxSockets (controls the maximum number of sockets that can be open/created for a host), which is important in certain environments. See #1329 for one example.
Any options set via a new agent_otions uplink configuration option are passed directly through to request, which combines them with other options to create an HTTP Agent (or reuse an existing agent if one with the same configuration already exists in the pool). See request.js#L698
Example configuration with agent_options:
uplinks:
  npmjs:	  npmjs:
    url: https://registry.npmjs.org/
    max_fails: 40
    maxage: 30m
    timeout: 60s
    agent_options:   /* customizes HTTP agent options for connections between Verdaccio and the upink */
      keepAlive: true
      maxSockets: 40
      maxFreeSockets: 10

Description:
See #1329

I'm fine with not having a fixed contract for "agent_options", you have my 👍

@willsmythe be aware I updated the base to the branch 4.1.x, since this, a feature and should be included in the next minor release.

@juanpicado - cool. Let me retest off this new base. Assuming this gets merged in the next few days, will it make the next minor release? And when is the next minor release (approximately)?
@ianschmitz - FYI

@wiggisser I usually set dates here https://github.com/verdaccio/verdaccio/milestones for each milestone (that might depend on available resources), the rule is simple, whatever is ready for release, goes in, otherwise, wait for the next minor release (1 per month).

@juanpicado - thanks. Btw, I didn't see any docs/guidance about updating @verdaccio/types. My main question: what branch should I target a PR against and what is the general process for consuming an updated version in verdaccio/verdaccio?

4.1.x still uses https://github.com/verdaccio/verdaccio/blob/4.1.x/package.json#L55 5.0.2 so https://github.com/verdaccio/flow-types/tree/5.x branch.
Type repo is really hard to handle, so many features ongoing, but 5.x is safe. The guide for typings still needs to be done but depends of https://github.com/verdaccio/verdaccio/issues/1166 , I guess soon I'll put an order in that repo :-)

Hey @juanpicado. Any progress or thoughts on this? We're hoping to bring create-react-app up to date with Verdaccio but i think this is the last remaining piece.
Thanks!

Actually I was waiting @willsmythe to resolve the conflict. But I'll do it myself in a new PR.

@ianschmitz you should be able to test wit<EMAIL_ADDRESS> Let me know if you need something else. 👍


Provide a wrapper for hydra's ConfigStore

Intended usage:
sr = SchemaRegistration()
sr.register(Config, path="experiment_schema")
sr.register(TrainerConf, path="trainer/trainer_schema")

with sr.new_group("schema/data", target_path="data") as group:
    group.add_option(CelebaDataConf, name="celeba")
    group.add_option(WaterbirdsDataConf, name="waterbirds")

Not completely sure about / vs ..

Maybe this is better:
sr = SchemaRegistration()
sr.register(Config, name="experiment_schema", path="/")
sr.register(TrainerConf, name="trainer_schema", path="/trainer")

with sr.new_group("schema/data", target_path="data") as group:
    group.add_option(CelebaDataConf, name="celeba")
    group.add_option(WaterbirdsDataConf, name="waterbirds")


Make the Social Icons Responsive at the Footer

I would like to work on this issue can you assigned this to me @SauravMukherjee44


fix build monitor configuration for weekly builds

https://github.com/dotnet/dnceng/issues/1368
This makes it so weekly builds get tagged for First responders to fix. It also removes the FRops sections, as we no longer use that label.

OOF nevermind, my fork is super out of date. I'll fix this up and reopen with a cleaner PR


chromaprint: add darwin compatibility

Motivation for this change
Software relying on chromaprint cannot currently be built with nix on macOS.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@ehmry thanks for the review and the explanations. I've fixed the issues.


cross-cli-browser apps with TUI+Xterm.js

Per conversations with Jay in Slack:
It would be cool to be able to develop a CLI app in CLJS and be able to wrap the app such that the same exact app could be used in the browser.
For instance, suppose a nano-like text editor was implemented in this CLJS/TUI system for command line usage. Using something like Xterm.js it should be possible to wrap said nano-like app and have a working version of the app in browser.
This could be especially interesting for building tools for the CLJS ecosystem that can span both CLJ/CLJS and self-hosted/CLJS ecosystems.

Found a bit of research from someone attempting to use blessed with xterm through websockets https://github.com/embark-framework/neo-blessed/issues/17. Not sure how relevant it is though.


Swift Compiler Errors in Esewa Flutter SDK during iOS Build

I encountered multiple Swift compiler errors while building my Flutter app for iOS with the Esewa Flutter SDK. The errors include the following:
ValidationType is not a member type of class EsewaSDK.EsewaSDK.
Reachability is not a member type of class EsewaSDK.EsewaSDK.
FormTextField is not a member type of class EsewaSDK.EsewaSDK.
TextFieldValidation is not a member type of class EsewaSDK.EsewaSDK.
These errors are occurring in the generated iOS module files. Below are the details:
Flutter Version: 3.24.5
Esewa Flutter SDK Version: '>=3.0.6 <4.0.0'
Xcode Version: 16.1
Swift Version: 6.0.2 (swiftlang-<IP_ADDRESS>.2 clang-16<IP_ADDRESS>)
Platform: iOS
Device: iOS 18.0 Simulator
Steps to Reproduce:
Add esewa_flutter_sdk to pubspec.yaml.
Run flutter pub get and build the project using flutter build ios.
Errors appear during the build process in Xcode.
Expected Behavior:
The app should compile and build successfully without any Swift compiler errors.
Actual Behavior:
The build fails with the above-mentioned Swift compiler errors in the EsewaSDK module files.
Possible Solutions:
Ensure compatibility with Swift 6.0.2.
Verify that the necessary type definitions are included in the EsewaSDK.

@rabindra-vrit As the package is discontinued, and there are no plans to release any new updates. You might have to look for alternatives for the package


added dataset builder for AlterTableCommand

Signed-off-by: tomasznazarewicz<EMAIL_ADDRESS>
Problem
in spark 3.2 the AlterTable class functionality was extended and moved to AlterTableCommand. There was no dataset builder for this class in our integration.
Closes: #616
Solution
AlterTableCommandDatasetBuilder class was created to obtain data from AlterTableCommand

Codecov Report

Merging #927 (8b1d8db) into main (fee53ee) will decrease coverage by 17.38%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #927       +/-   ##
=============================================
- Coverage     84.32%   66.94%   -17.39%
  Complexity       56       56
=============================================
  Files            73       17       -56
  Lines          3088      239     -2849
  Branches         20       20
=============================================
- Hits           2604      160     -2444
+ Misses          456       51      -405
  Partials         28       28

Impacted Files
Coverage Δ

...eage/common/provider/great_expectations/results.py

client/python/openlineage/client/transport/http.py

.../openlineage/airflow/extractors/mysql_extractor.py

integration/common/openlineage/common/models.py

...enlineage/airflow/extractors/postgres_extractor.py

...age/common/provider/great_expectations/__init__.py

...egration/common/openlineage/common/sql/__init__.py

integration/common/openlineage/common/utils.py

integration/dagster/openlineage/dagster/adapter.py

integration/airflow/openlineage/airflow/utils.py

... and 46 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


[System.Environment]::SetEnvironmentVariable does not set registry key value type correctly

Steps to reproduce
In powershell,  Execute
[System.Environment]::SetEnvironmentVariable('JAVA_HOME', '%JAVA_HOME_x64%', 'Machine');
check  value in as visible in HKLM:\SYSTEM\CurrentControlSet\Control\Session Manager\Environment\JAVA_HOME
it is of type REG_SZ, it should instead be of type REG_EXPAND_SZ.

Expected behavior
if %somevar% is present in value string it should instead be of type REG_EXPAND_SZ  else it should be REG_SZ

Actual behavior
it is always of type REG_SZ.

Environment data
PowerShell 7.2.0-preview.6

You call .Net method. It is .Net issue. See https://github.com/dotnet/runtime/issues/1442


unused resources cost optimization policy and set

Overview/Summary
Optimize cost by detecting unused but chargeable resources. Leverage this Policy definition as a cost control to reveal the following resource types that are driving cost:

Microsoft.Compute/disks
microsoft.network/publicIpAddresses
Microsoft.Web/serverfarms

This PR fixes/adds/changes/removes
3 policy definitions, 1 policySet
Breaking Changes
n/a
Testing Evidence
Once assigned policy compliance is shown as per policyRule
Testing URLs
n/a

Please also replace the curly brackets on the placeholders {}

Azure Public
n/a
Azure US Gov (Fairfax)
n/a
As part of this Pull Request I have

[x] Checked for duplicate Pull Requests
[ ] Associated it with relevant issues, for tracking and closure.
[ ] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[x] Ensured contribution guidance is followed.
[ ] Updated relevant and associated documentation.
[ ] Updated the "What's New?" wiki page (located: /docs/wiki/whats-new.md)

@JulianHayward can you resolve the merge conflict please, normally just good to accept the changes and the build process, rebuilds the file anyway

Thanks @JulianHayward Julian Hayward FTE for the contribution here. Firstly, I think this is a great idea.
Looping in @paulgrimley Paul Grimley (HE/HIM) FTE @Springstone Sacha Narinx FTE for awareness in relation to policy-refresh work.
I'm keen to merge this but I think we should make this a default assignment to the intermediate root management group by default for all customers, it does nothing but good and its audit only so they dont need to be able to turn it off.
Can you therefore @JulianHayward Julian Hayward FTE make the policy assignment file and also update the eslzArm.json file to make the assignment using the policy assignment file to the intermediate root management group?
Also can you update the ALZ policies wiki page (https://aka.ms/alz/policies) with the assignment detials and also the wiki Whats New page and add it to the TBD section for now 👍

100% agree, this needs to be enabled by default. Great idea and contribution.

kudos goes to @dolevshor  - the policy definitions are based on the queries that are provided at https://github.com/dolevshor/azure-orphan-resources (slightly modified to only audit those that generate cost). That repo should be tracked.

@Springstone kudos goes to @dolevshor - the policy definitions are based on the queries that are provided at https://github.com/dolevshor/azure-orphan-resources (slightly modified to only audit those that generate cost). That repo should be tracked.

Dolev is a good friend and colleague, I know him well, mentino my name :)

@JulianHayward could you please resolve merge conflicts:
git checkout policy-refresh
git pull upstream policy-refresh
git checkout
git merge policy-refresh
Resolve + commit + push

@jtracey93 worked with @JulianHayward to merge, kindly review.


Cannot convert CSV to NPY

I am unable to run cvs_to_numpy.py in python 3.6.3 as it throws an error:
Traceback (most recent call last):
  File "cvs_to_numpy.py", line 7, in <module>
    cascade_classifier = cv2.CascadeClassifier(CASC_PATH)
NameError: name 'CASC_PATH' is not defined

SOLVED!
Simply place constants.py in the data directory!

OpenCV Error: Assertion failed (!empty()) in cv::CascadeClassifier::detectMultiScale, file C:\bld\opencv_1496444098483\work\opencv-3.2.0\modules\objdetect\src\cascadedetect.cpp, line 1681
Traceback (most recent call last):
File "D:/Masters/Research/facial expression detection using neural nets/emotion-recognition-neural-networks-master/data/cvs_to_numpy.py", line 72, in
image = data_to_image(row['pixels'])
File "D:/Masters/Research/facial expression detection using neural nets/emotion-recognition-neural-networks-master/data/cvs_to_numpy.py", line 60, in data_to_image
data_image = format_image(data_image)
File "D:/Masters/Research/facial expression detection using neural nets/emotion-recognition-neural-networks-master/data/cvs_to_numpy.py", line 23, in format_image
minNeighbors = 5
cv2.error: C:\bld\opencv_1496444098483\work\opencv-3.2.0\modules\objdetect\src\cascadedetect.cpp:1681: error: (-215) !empty() in function cv::CascadeClassifier::detectMultiScale
Got these errors.. Can someone help me??


Spill head projection to remaining sequence

I propose to add an extension that takes one or more elements at the head of a sequence and spills a projection to remaining elements of the sequence.
Sometimes you have a sequence where the initial element(s) contains information about the processing of the rest of the sequence. A typical example is a table (think CSV) where the table is composed of rows; where the first row is a header and the remaining the data rows. Processing such a sequence should only generate a projection of the data rows.
The signature would be as follows:
public static IEnumerable<R>
    SpillSpan<T, M, A, H, R>(
        this IEnumerable<T> source,
        Func<T, (bool, M)> chooser,
        A empty,
        Func<M, A> seeder,
        Func<A, M, A> accumulator,
        Func<A, H> headerSelector,
        Func<H, T, R> resultSelector)

The chooser identifies header elements. Data rows commence as soon as it returns (false, _) for a T. The header elements are accumulated via accumulator and the seeder is used to seed the accumulation with the initial header element. The empty value is used for the headless case. The headerSelector function is used to create a single projection out of the accumulated header elements and which is subsequently paired with or spilled to remaining elements.
I propose to add overloads for simpler cases.
SpillSpan should never throw an exception. If the user wants to ban the headless case, he/she can throw in headerSelector upon receiving the empty value.
Example
const string csv = @"
    a,c,b
    1,3,2
    4,6,5
    7,9,8";

data result =
    from rows in new[]
    {
        from line in Regex.Split(csv.Trim(), @"\r?\n")
        select line.Split(',').Select(f => f.Trim()).ToArray()
    }
    from row in
        rows.Index()
            .SpillSpan(
                r => r.Key == 0,
                null,
                r => r.Value,
                (r, _) => r,
                h => MoreEnumerable.Return(h.Index().ToDictionary(e => e.Value, e => e.Key)) // name => index
                                   .SelectMany(_ => new[] { "a", "b", "c" }, (m, n) => m[n]) // lookup index of name
                                   .ToArray(),
                (bs, r) => bs.Select(i => int.Parse(r.Value[i], CultureInfo.InvariantCulture))
                             .Fold((a, b, c) => new { A = a, B = b, C = c }))
    select row;

foreach (var row in data)
    Console.WriteLine(row);

Output:
{ A = 1, B = 2, C = 3 }
{ A = 4, B = 5, C = 6 }
{ A = 7, B = 8, C = 9 }

Based on work so far in PR #753, I feel it's best to rename the extension to SpillHead. The simplest overload will spill the first element of the sequence to the second and beyond so in the case of one element representing the head, span seems confusing. However, head works well whether you have a single element or several making up the “head” of the sequence.

Is it essentially this, but working in constant space?
public static IEnumerable<R>
    SpillSpan<T, P, R>(
        this IEnumerable<T> source,
        Func<T, bool> predicate,
        Func<T, P> prefixMapping,
        Func<P, T, R> resultSelector)
{
    var (prefix, rest) = source.Span(predicate);
    var mappedPrefix = prefixMapping(prefix);
    return rest.Select(e => resultSelector(mappedPrefix, e));
}

// applied to a predicate `predicate` and an enumerable `source`, returns a tuple where first element is longest prefix (possibly empty) of `source` of elements that satisfy `predicate` and second element is the remainder of the enumerable
// >>> span (< 3) [1,2,3,4,1,2,3,4]
// ([1,2],[3,4,1,2,3,4])
public static (IReadOnlyCollection<T> Prefix, IReadOnlyCollection<T> Rest) Span<T>(this IEnumerable<T> source, Func<T, bool> predicate);

The A empty+Func<M, A> seeder+Func<A, M, A> accumulator+Func<A, H> headerSelector logic can be made external and encapsulated into Func<T, P> prefixMapping. chooser handles both predication and mapping, so the mapping part can be moved into prefixMapping too.

Not quite like span/Parition, because here the proposal is to consume in a streaming fashion. So while the head/header is collected and projected to help process the remainder, the remainder of the sequence (assuming they are the data rows) is streamed. If you have billions of rows, this operator will lazily only consume what's needed. It can be combined with other streaming operators to avoid committing the entire source to memory.


Fix Pandas 0.2x breaking Google BigQuery change

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[X] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1179

Description

[X] Here are some details about my PR, including screenshots of any UI changes:
Added pandas-gbq to setup.py and change the import

Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Only import path has changed, so I didn't add unit-tests

Commits

[X] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #2279 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2279      +/-   ##
==========================================
+ Coverage   68.86%   68.87%   +<.01%
==========================================
  Files         142      142
  Lines       10969    10969
==========================================
+ Hits         7554     7555       +1
+ Misses       3415     3414       -1

Impacted Files
Coverage Δ

airflow/models.py
87.01% <0%> (+0.04%)
:arrow_up:

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 4284e64...bcd31cf. Read the comment docs.

To keep packaging a bit simpler, can you include the pandas_gbq as part of gcp_api, rather than its own subpackage?

Can you make sure the CI passes, too? Might require a rebase.

@criccomini I did a rebase, so the CI should pass again.

@criccomini I did a rebase, so the CI should pass again.

@criccomini @NielsZeilemaker the travis' builds on master are current failing with errors in BigQuery/Panda's can you guys have look?


Inconsistent behavior

The generated GraphQL interface can change depending on the order in which operations are processed and this order is influenced by a number of factors including the node implementation and the structure of the OAS.
We need to make sure the behavior is consistent as it can affect users and tests. Perhaps we need to sort the OAS before processing?

Yes, sorting the operations sounds like a good idea.

Fixed in #138


Add systemtest mk file to include changes to enable testing on cygwin

Resolves: https://github.com/AdoptOpenJDK/openjdk-systemtest/issues/199
Signed-off-by: Mesbah Alam<EMAIL_ADDRESS>

Tested in this build: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/475/console

fyi @AdamBrousseau  after this PR, we should be able to turn on wins system test builds.


Add track_times=False to Model.save_weights('.h5')

System information

TensorFlow version: 2.4.1
Are you willing to contribute it: Yes

Describe the feature and the current behavior/state.
Currently, when calling keras.models.Model.save_weights() in hdf5 format, the weights are saved using h5py.Group.create_dataset with the argument track_times set to the default value True. The effect of this is following: even when the weights are exactly the same, the resulting saved files will not be the same (see for example this stack overflow question). This is a problem when one has a replicable model pipeline and uses some version control system (like DVC).
The proposed feature is adding h5_track_times argument to the Model.save_weights() function and pass it down to h5py.
Will this change the current api? How?
Yes, the Model.save_weights() would get an additional optional h5_track_times argument.
Who will benefit with this feature?
Everyone who needs to have replicable model pipeline and needs to version the model weights in hdf5 format.

Original issue: https://github.com/tensorflow/tensorflow/issues/51405

@fchollet and @k-w-w, I think we should actually not putting the timestep into the h5 and its checksum at all. Only the weight itself should be saved, and the metadata shouldn't affect the artifacts.

Hello, Thank you for reporting an issue.
We're currently in the process of migrating the new Keras 3 code base from keras-team/keras-core to keras-team/keras.
Consequently, This issue may not be relevant to the Keras 3 code base . After the migration is successfully completed, feel free to reopen this Issue at keras-team/keras if you believe it remains relevant to the Keras 3 code base. If instead this Issue is a bug or security issue in legacy tf.keras, you can instead report a new issue at keras-team/tf-keras, which hosts the TensorFlow-only, legacy version of Keras.
To know more about Keras 3, please read https://keras.io/keras_core/announcement/


matrix does not support settings

not sure if something changed with how GH actions works, but settings: no longer works with matrix. Replacing settings: with include: effectively does the same thing, but any of the  ${{ matrix.settings.platform }} must be changed to ${{ matrix.platform }} (removing settings)

In what way does it not work? Do you have a source (like an announcement) or any kind of error? I still use it just fine in my workflows.
That said, i misunderstood the docs about include and looking at your workflow runs i think i'd prefer it either way 🤔

Hey Fabian! Glad to see you here! I got errors when copying + pasting the examples into my workflow.yml. Here's an example:

I would've loved to have some sort of announcement, but I can't seem to find anything. Going back into the docs, since they break out the sections into individual .mds, there isn't an easy way to go back in time to see when settings: fell out of favour.

I went all the way back to 2022, and settings: isn't mentioned at all in the matrix docs? Where is settings: documented? I can't find it even when searching across the whole repo.

Oh right, the editor always complained about this, but only the editor (and idk vscode plugins sometimes) but in the run it actually works.
Technically the error isn't even correct because include is not a primitive yml value either x)

Also to be clear, settings is not a special word, it could be configs or whatever too.

Hmm, correct. However, since include: appears to be treated as a special case that collapses the available choices into the matrix, I figure it would be a good idea to change things out to include: anyway since it also gets rid of the error and is functionally identical? Pretty interesting that the run works anyway, got me quite confused because it doesn't seem like it should work matrix-wise at all according to the docs :joy:

I figure it would be a good idea to change things out to include: anyway since it also gets rid of the error and is functionally identical?

already agreed to that in my first message:

That said, i misunderstood the docs about include and looking at your workflow runs i think i'd prefer it either way 🤔

got me quite confused because it doesn't seem like it should work matrix-wise at all according to the docs 😂

To be fair, i think those docs are a bit, let's say, suboptimal overall. And we've even used the settings object before the docs existed in that capacity lol.
Anyway, i don't have time to do this anytime soon (especially since i consider this low prio), so i'd appreciate if you'd open a PR for the readme and example files. No problem if you don't have time either :)

no prob! will get a PR done within the week :)


Allow non enterprise users to deploy individual apps

This appears to be a pretty standard feature that is missing.

Thanks for the feedback -- I'll pass along to the product owners.


Invalid prop: type check failed for prop "backdrop". Expected Boolean, got String.

OS, browser, uiv version
Windows 10 64 bits / UIV: 0.11.10
Which component
Modal
Steps to reproduce the problem
<modal v-model="editing" size="lg" backdrop="false"></modal>
Expected behavior
Backdrop should be considered as boolean false
Actual behavior
Invalid prop: type check failed for prop "backdrop". Expected Boolean, got String.

I'm using UIV registering it all at once and then using it in my components directly in the markup as you can see in the "Step to reproduce the problem section".
Like this...
// Import and register all components once
import * as uiv from 'uiv'
Vue.use(uiv)

I've just realized that using dynamic binding using :backdrop="false" does pass an actual boolean.

It worth mention this in the docs, I think


fix formatting

What does this PR do?
fixes #2088

LGTM, thanks!

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


fix: typo in translation in Textarea Class

bug

fixed
from 'min minLength' to 'min length'
from 'max maxlength' to 'max length'

Thank you
can you fix the conflicts in the file: composer.lock, just run composer update.


docker compose and build update for orthanc-keycloak-recipe

Recipe updates due to errors when building with latest versions. All fixed made based on current experience on Oct 2022
a) Node version fixed to version 16
b) Keycloak load from source forced to be enabled (default is false now)
Changes made based on issues found when trying to setup from recipe (thus no link to issues)
No User Experience of main product affected, just docker build and compose
PR Checklist

[ X] Brief description of changes
[ ] Links to any relevant issues
[ ] Required status checks are passing
[ ] User cases if changes impact the user's experience
[ ] @mention a maintainer to request a review

We've fixed this in the master branch. I'm gonna close it. If the issue still exists, feel free to reopen it.


Revert "Create 0000-01-02-UrvashiBhatt-eaton.md"

Reverts UrvashiBhatt-eaton/github-slideshow#3

Wrong Pull Reqest


Hide scroll bars on mobile

Description

Scrollbars take up substantial screen space on mobile, and they should be hidden.

This is an issue in the section table as well as the calendar pane.
I vaguely remember this issue being solved some time ago, but migrating MUI versions might've messed with that.

Hi, I would be happy to work on this issue. Could you please assign it to me?


failing test not showing in jasmine-node when using the --coffee switch

I have the following basic test for a web service:
var request = require('http'),
    url     = 'http://localhost:1337/';

describe('webservice', function() {

  it('should respond to /ping', function(done) {
    request.get(url + 'ping', function(res) {
      expect(res.statusCode).toBe(200);
      res.on('data', function(body) {
        var json = JSON.parse(body);
        expect(json.message).toBe("Hi, I'm alive");
        expect(json.date).toBeDefined();
        done();
      });
    });

  });

});

which seems to work ok:
$ jasmine-node --verbose tests/ping/

webservice
    should respond to /ping

Finished in 0.032 seconds
1 test, 3 assertions, 0 failures

But if I make one test fail (change the expected json message, for example) I get the following error:
$ jasmine-node --verbose --coffee tests/ping/
StreamWrap: Aborting due to unwrap failure at ../src/stream_wrap.cc:125
Aborted (core dumped)

Any idea what could I be doing wrong?
(more over, any mistake I have in my js, the test just silently fails, with no clue on waht happened, the js errors just dissapears)
update: the error appears only if the failing test is inside the request.get callback...
ps: also askes at SO in case anybody wants to earn a couple of reputation points: http://stackoverflow.com/questions/15469980/failing-test-not-showing-in-jasmine-node
ps2: I couldn't find a google group for this project...

I'm getting the same issue years later with Node v5.10.1.