[SPARK-12184][Python] Make python api doc for pivot consistant with scala doc

In SPARK-11946 the API for pivot was changed a bit and got updated doc, the doc changes were not made for the python api though. This PR updates the python doc to be consistent.

Test build #47282 has started for PR 10176 at commit a77bdb3.

Test build #47282 has finished for PR 10176 at commit a77bdb3.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *  public abstract static class PrefixComputer\n

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47282/
Test PASSed.

Merged build finished. Test PASSed.

@rxin or @yhuai can we get this doc change merged for 1.6?

Thanks. Merging to master and branch 1.6.


Using holder with  element doesn't work ?

Hi, not sure if it me doing something wrong or trying to do something that's not possible.
Using holder with just <img .. > works great. But if i want to do something like below to select the img based on orientation, or use webp with jpeg has the fallback then holder.js doesn't seem to work. Is it possible to have holder.js working in these situations ?
<picture>
    <source media="orientation:landscape" srcset="horizontal-image.jpg" />
    <source media="orientation:portrait" srcset="vertical-image.jpg" />
    <img src="image.jpg" alt="an image" />
</picture>

many thanks

there's no built-in support yet, but this is a good suggestion. thanks for bringing it up

could you post a jsfiddle that shows Holder not working?
i don't think there's built-in support, but it's definitely something that should work.

Same problem for me...
Fiddle

If i add text on each srcset, i get the default value... strange :/
Fiddle

will look at this, maybe worth putting it into v3 along with other general improvements. this may change the general processing logic too much to be a cheap addition.


Can't load feature extractors in TF 2.5

Hello,
First, thanks for allowing us to work with Vision Transformers with tensorflow. I have an issue when trying to load one of your feature extractor using tensorflow==2.5.0 and tensorflow-hub==0.12.0:
import tensorflow_hub as hub
hub.KerasLayer("https://tfhub.dev/sayakpaul/vit_s16_fe/1", trainable=True)

2021-09-22 12:28:04 absl WARNING Importing a function (__inference_<lambda>_173777) with ops with custom gradients. Will likely fail if a gradient is requested. (function_deserialization.py:500)
2021-09-22 12:28:04 absl WARNING Importing a function (__inference_<lambda>_169866) with ops with custom gradients. Will likely fail if a gradient is requested. (function_deserialization.py:500)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-17-7922087f3731> in <module>
----> 1 hub.KerasLayer("https://tfhub.dev/sayakpaul/vit_s16_fe/1", trainable=True)

~/vision/env/lib/python3.8/site-packages/tensorflow_hub/keras_layer.py in __init__(self, handle, trainable, arguments, _sentinel, tags, signature, signature_outputs_as_dict, output_key, output_shape, load_options, **kwargs)
    151
    152     self._load_options = load_options
--> 153     self._func = load_module(handle, tags, self._load_options)
    154     self._has_training_argument = func_has_training_argument(self._func)
    155     self._is_hub_module_v1 = getattr(self._func, "_is_hub_module_v1", False)

~/vision/env/lib/python3.8/site-packages/tensorflow_hub/keras_layer.py in load_module(handle, tags, load_options)
    447       except ImportError:  # Expected before TF2.4.
    448         set_load_options = load_options
--> 449     return module_v2.load(handle, tags=tags, options=set_load_options)
    450
    451

~/vision/env/lib/python3.8/site-packages/tensorflow_hub/module_v2.py in load(handle, tags, options)
    104         module_path, tags=tags, options=options)
    105   else:
--> 106     obj = tf.compat.v1.saved_model.load_v2(module_path, tags=tags)
    107   obj._is_hub_module_v1 = is_hub_module_v1  # pylint: disable=protected-access
    108   return obj

~/vision/env/lib/python3.8/site-packages/tensorflow/python/saved_model/load.py in load(export_dir, tags, options)
    867     ValueError: If `tags` don't match a MetaGraph in the SavedModel.
    868   """
--> 869   return load_internal(export_dir, tags, options)["root"]
    870
    871

~/vision/env/lib/python3.8/site-packages/tensorflow/python/saved_model/load.py in load_internal(export_dir, tags, options, loader_cls, filters)
    903     with ops.init_scope():
    904       try:
--> 905         loader = loader_cls(object_graph_proto, saved_model_proto, export_dir,
    906                             ckpt_options, filters)
    907       except errors.NotFoundError as err:

~/vision/env/lib/python3.8/site-packages/tensorflow/python/saved_model/load.py in __init__(self, object_graph_proto, saved_model_proto, export_dir, ckpt_options, filters)
    160       self._concrete_functions[name] = _WrapperFunction(concrete_function)
    161
--> 162     self._load_all()
    163     self._restore_checkpoint()
    164

~/vision/env/lib/python3.8/site-packages/tensorflow/python/saved_model/load.py in _load_all(self)
    256   def _load_all(self):
    257     """Loads all nodes and functions from the SavedModel and their edges."""
--> 258     self._load_nodes()
    259     self._load_edges()
    260     # TODO(b/124045874): There are limitations with functions whose captures

~/vision/env/lib/python3.8/site-packages/tensorflow/python/saved_model/load.py in _load_nodes(self)
    434         # interface.
    435         continue
--> 436       node, setter = self._recreate(proto, node_id)
    437       nodes[node_id] = node
    438       node_setters[node_id] = setter

~/vision/env/lib/python3.8/site-packages/tensorflow/python/saved_model/load.py in _recreate(self, proto, node_id)
    552     kind = proto.WhichOneof("kind")
    553     if kind not in factory:
--> 554       raise ValueError("Unknown SavedObject type: %r" % kind)
    555     return factory[kind]()
    556

ValueError: Unknown SavedObject type: None

I have the same error when setting trainable=False, but it works fine when using the whole classification model (for insrance https://tfhub.dev/sayakpaul/vit_s16_classification/1)
This error doesn't happen with TF2.6, but it's currently not possible for my team to upgrade our TF version from 2.5 to 2.6. Do you think there is a workaroung to make this work in TF2.5 ?
Thanks

Unfortunately, it has something to do with allowing the gradients of the operations contained in the TF graph. So, it's a requirement to use TensorFlow 2.6 or upper. I will make a note of it in the README.

Ok, thanks for the quick answer


[WebToolsE2E][docs] When running the "Tutorial: Use .NET Aspire messaging integrations in ASP.NET Core" document using .NET CLI, there is an error.

Describe the issue or suggestion
Issue description:
In "Tutorial: Use .NET Aspire messaging integrations in ASP.NET Core" doc, we need to make some updates to the instructions.
This issue only occurs when running the document using the .NET CLI development environment. There is no issue when using VS.
Issue:
Complete all the steps in the document using the .NET CLI development environment, the result obtained when Run and test the app locally is an error, which is different from the result described in the document.

Hosting failed to start
2024-11-11T09:15:01       System.InvalidOperationException: Cannot read keys when either application does not have a console or when console input has been redirected. Try Console.Read.
2024-11-11T09:15:01          at System.ConsolePal.ReadKey(Boolean intercept)
2024-11-11T09:15:01          at AspireSample.WorkerService.Worker.ExecuteAsync(CancellationToken stoppingToken) in C:\Users\v-reinawang\AspireSample\AspireSample.WorkerService\Worker.cs:line 31
2024-11-11T09:15:01          at Microsoft.Extensions.Hosting.Internal.Host.b__14_1(IHostedService service, CancellationToken token)
2024-11-11T09:15:01          at Microsoft.Extensions.Hosting.Internal.Host.ForeachService[T](IEnumerable1 services, CancellationToken token, Boolean concurrent, Boolean abortOnFirstException, List1 exceptions, Func`3 operation)

According to the error message, in Worker.cs file, Change "Console.ReadKey();" to "Console.Read();", but the expected result cannot be obtained. Display 'Stoped receiving messages' in console logs.

Expected:
After completing step 4 of the Run and test the app locally procedure, the corresponding information appears in the Console Log.

Close this issue and use #2045 to track this bug.


TypeError

im trying to run a test and it throws this error

im new to python so i dont know what the reason is
have tried to call the function without the return variable and its the same, so its probably not part of the issue

What test are you trying to run?

test_schedule.py
it has a line "from tests.details import lectioUsername, lectioPassword, schoolId" that refers to a file "details"
this is not in the folders
so i have just hardcoded the username, pwd, schoolid
i could not get "pip install LectioScraper" to work, it responds "ERROR: Could not find a version that satisfies the requirement LectioScraper (from versions: none)
ERROR: No matching distribution found for LectioScraper"
im on windows 10 but have also tried on the Ubuntu subsystem with same result
so i have just downloaded the files from git as a zip and unpacked them


Fix checker initialization crash with esModuleInterop and global merges resolving to ImportEquals aliases

Fixes #47299

@typescript-bot cherry-pick this to release-4.5


add window function for analyze

row_number()over(partition by order by )
...
Lag/lead/first_value....
Sent with GitHawk

In OmniSci 4.7, preliminary support for window functions was added. Because of the usefulness of this functionality, we will continue to improve it in future releases.

OmniSciDB now supports Window functions. Empty Window function partitions are currently not supported; that is, all Window functions must include a PARTITION BY key in the OVER clause. Also, OmniSci recommends that you use Window functions over aggregate query outputs at this time (instead of raw data). Future releases aim to improve scale and performance and remove these restrictions; see the documentation for details on what is supported.

https://www.omnisci.com/docs/latest/7_0_release.html#4-7-0-released-june-13-2019


Style : when option list is open

Is your feature request related to a problem? Please describe.
Style object when option list is open.
Describe the solution you'd like
activeStyle:{
borderColor:'green'
}
Describe alternatives you've considered
use of onSelectOpened and onSelectClosed to toggle the color in local state

@imdkbj, Thanks for a suggestion! All of the currently available styling properties you can find here: https://mobilereality.github.io/react-native-select-pro/docs/api/select#styles. I'm moving this thread to discussions for now.


Fix for AWS Lambda reserved environment variables

Regarding AWS doc (https://docs.aws.amazon.com/lambda/latest/dg/current-supported-versions.html#lambda-environment-variables) 'LANG', 'LD_LIBRARY_PATH', 'NODE_PATH', 'PATH' and 'PYTHONPATH' are not reserved and can be overrode.

Thanks!


Revisit parts of the spec to benefit from compatibility

Summary
I think some parts spec still deviate from existing implementations in the Cosmos-SDK. In some cases this might inevitable (e.g. changes to the staking logic to make it efficiently fraud provable) in other cases the changes might be purely cosmetic, small improvements, or, simply do not account for the existing implementation. In some cases these little changes break compatibility to existing Cosmos infrastructure where it is not really required. These should be resisted.
This issue is about revisiting the spec from this perspective, to list all these places and to tack if any changes need to be made.
Details
This should (eventually) contain an exhaustive list of changes:

[ ] SDK Txs have two kind of nonces: account number and account sequence (spec only has nonce); which already led to  including yet another Nonce field in implementation following the spec: https://github.com/celestiaorg/lazyledger-app/blob/0e60861ff7f5548d52391f5aa57014e2554bbbff/x/lazyledgerapp/types/tx.pb.go#L36 (cc @evan-forbes)
[ ] verify compatibility of all Tx types with SDK Txs (so they can be signed by a ledger without substantially modifying the software); same with the encoding
[ ] ...

Action Items

[ ] Complete above list
[ ] Change all uncontroversial places in the spec (if necessary at all)
[ ] discuss the others + their impact

Refs
https://github.com/celestiaorg/lazyledger-app/issues/33

Using app


Improve CMake to make oneMKL easier to use

Summary
I have faced issues when trying to use OneMKL as a dependency using CMake. From my testing currently users of oneMKL need to manually set the right includes or libraries to link in CMake to have oneMKL as a dependency. I would like to discuss if we can improve this.
Version
Latest oneMKL commit, AFAIK this is not a recent issue.
Environment
I am using the oneAPI 2023.1.0 package and using the modules files to load mkl and compiler.
CMake 3.26.3.
Steps to reproduce
First approach
I have tried to use OneMKL in 2 ways. First using FetchContent as shown with the following CMakeLists.txt:
cmake_minimum_required(VERSION 3.13)
project(sample)

include(FetchContent)
set(BUILD_FUNCTIONAL_TESTS OFF)
set(BUILD_EXAMPLES OFF)
FetchContent_Declare(
    open_onemkl
    GIT_REPOSITORY https://github.com/oneapi-src/oneMKL.git
    GIT_TAG dc22279396def65b1af1742a45dda7afcf7c0a22
)
FetchContent_MakeAvailable(open_onemkl)

add_executable(test test.cpp)
target_link_libraries(test PRIVATE onemkl)

and the simplest test.cpp:
int main() { return 0; }

The CMake command cmake -Bbuild . is failing with:
CMake Error at build/_deps/open_onemkl-src/src/blas/CMakeLists.txt:43 (target_link_libraries):
  Target "onemkl_blas" links to:

    ONEMKL::SYCL::SYCL

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

and the same errors for all the other domains and backends. I think FetchContent is quite common in modern CMake. I have had more success with ExternalProject but this method has limitations and requires the user to manually set the include directory and link to the libraries as far as I know. Do you think it would be valuable to support this? I'm not sure what changes are needed to fix this so help would be appreciated.
Second approach
Another common approach is to use find_package(oneMKL) like so:
cmake_minimum_required(VERSION 3.13)
project(sample)

find_package(oneMKL REQUIRED)
add_executable(test gemm_usm.cpp)
target_link_libraries(test PRIVATE MKL::onemkl)

Here I am copying some of the existing examples from gemm_usm.cpp (as well as example_helper.hpp).
As discussed in https://github.com/oneapi-src/oneMKL/issues/269 there are issues to set up the compiler (see the attached cmake_install_patch.txt).
I am building with:
cd /path/to/oneMKL
cmake -Bbuild -GNinja -DENABLE_MKLCPU_THREAD_TBB=OFF -DBUILD_FUNCTIONAL_TESTS=OFF -DBUILD_EXAMPLES=OFF -DCMAKE_INSTALL_PREFIX=/path/to/oneMKL/install .
cd /path/to/sample
cmake -Bbuild -GNinja -DCMAKE_CXX_COMPILER=/opt/intel/oneapi/compiler/latest/linux/bin/icpx -DoneMKL_ROOT=/path/to/oneMKL/install .
ninja -Cbuild

The build is failing to link with:
/opt/intel/oneapi/compiler/latest/linux/bin/icpx  -fsycl CMakeFiles/test.dir/gemm_usm.cpp.o -o test  -Wl,-rpath,/path/to/oneMKL/install/lib  /path/to/oneMKL/install/lib/libonemkl.so.0  /opt/intel/oneapi/compiler/latest/linux/lib/libsycl.so && :
/usr/bin/ld: /tmp/icpx-c0cdbb/gemm_usm-1d68df.o: in function `run_gemm_example(sycl::_V1::device const&)':
gemm_usm-b1c519.cpp:(.text+0x111c): undefined reference to `oneapi::mkl::blas::column_major::gemm(sycl::_V1::queue&, oneapi::mkl::transpose, oneapi::mkl::transpose, long, long, long, float, float const*, long, float const*, long, float, float*, long, oneapi::mkl::blas::compute_mode, std::vector<sycl::_V1::event, std::allocator<sycl::_V1::event> > const&)'
icpx: error: linker command failed with exit code 1 (use -v to see invocation)

I would expect that target_link_libraries add all the required dependencies for a minimal example like this to work. I can see the CMake of the example has more logic but I couldn't find a nice solution to fix this. Am I missing something here? It would be nice to be able to use oneMKL with as little code from the user as possible.
That's not the only issue with this approach. For instance if I try to build a DFT example it fails to use the expected headers:
FAILED: CMakeFiles/test.dir/real_fwd_usm.cpp.o
/opt/intel/oneapi/compiler/latest/linux/bin/icpx  -isystem /path/to/oneMKL/install/include -fsycl -MD -MT CMakeFiles/test.dir/real_fwd_usm.cpp.o -MF CMakeFiles/test.dir/real_fwd_usm.cpp.o.d -o CMakeFiles/test.dir/real_fwd_usm.cpp.o -c /path/to/samples/onemkl_sample2/real_fwd_usm.cpp
/path/to/samples/onemkl_sample2/real_fwd_usm.cpp:65:52: error: no member named 'INPLACE' in 'oneapi::mkl::dft::config_value'
                   oneapi::mkl::dft::config_value::INPLACE);
                   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^

CMake includes the installed oneMKL headers as a system include which ends up as a lower priority than the installed MKL headers so /opt/intel/oneapi/mkl/2023.1.0/include are used instead. I couldn't find what is causing this behavior in oneMKL CMake but I think the expected behavior is to include /path/to/oneMKL/install/include as a user include with -I instead of -isystem.
Please let me know if you think there is value in fixing these issues. If so I can help for some of them but I don't fully understand what are the changes needed.

There has been some CMake improvements but the 2 main issues remain:

FetchContent runs into the same issue with ONEMKL::SYCL::SYCL target not found.
The interface headers conflict with the closed-source MKL headers with find_package. I think the interface headers should not be included as system headers.

Thank you @Rbiessy , we will take a look.

Hello @Rbiessy
I think the first problem with FetchContent goes away if you specify the compiler like:
cmake -Bbuild . -DCMAKE_CXX_COMPILER=icpx -DCMAKE_C_COMPILER=icx
At least in my local runs, if I don't specify compiler, cmake uses /usr/bin/c++ by default and everything starts crashing from the beginning (like from the first steps "unrecognized command line option '-fsycl'". If I set the compiler, everything is fine and I don't have  ONEMKL::SYCL::SYCL target not found problem.

Hello @akabalov you're right, thanks for that. I'm usually not setting the compiler since oneMKL can detect which one is used automatically. I suppose it makes sense to let the users choose how to handle this.
I can see another issue when trying to build the sample with FetchContent, the generated header file config.hpp is not found
/home/romain/prog/samples/onemkl_sample1/build/_deps/open_onemkl-src/include/oneapi/mkl/detail/backends_table.hpp:32:10: fatal error: 'oneapi/mkl/detail/config.hpp' file not found
   32 | #include "oneapi/mkl/detail/config.hpp"
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

This file ends up in build/_deps/open_onemkl-build/bin/oneapi/mkl/detail/config.hpp. It would be very useful if the user doesn't need to manually add an include path in the CMake. I think it would be possible to add the right include path to the onemkl target, what do you think?

Hi, this issue is not directly related to my issue. However, the title is broad enough where it loosely fits.
For projects with multiple subprojects with different MKL usages, I found its difficult to use the provided MKLConfig.cmake since it doesn't contain all the targets. I understand its parameterized, but I suspect the variables all get cached so trying to specify multiple configurations seems nearly impossible.  Also the target MKL::mkl_sycl doesn't seem to link mkl_sycl.so.
Anyways, if its more proper to create a new issue for this. I can do that.

FetchContent

Yes, I got a fix for this problem. I'll open a PR soon once I complete the validation.

Hi, this issue is not directly related to my issue. However, the title is broad enough where it loosely fits.
For projects with multiple subprojects with different MKL usages, I found its difficult to use the provided MKLConfig.cmake since it doesn't contain all the targets. I understand its parameterized, but I suspect the variables all get cached so trying to specify multiple configurations seems nearly impossible. Also the target MKL::mkl_sycl doesn't seem to link mkl_sycl.so.
Anyways, if its more proper to create a new issue for this. I can do that.

Hello Matthew!
I would suggest to open a new issue for this problem. Thank you a lot in advance!

There has been some CMake improvements but the 2 main issues remain:

FetchContent runs into the same issue with ONEMKL::SYCL::SYCL target not found.
The interface headers conflict with the closed-source MKL headers with find_package. I think the interface headers should not be included as system headers.

As for the second issue, after some investigation, I found this page https://gitlab.kitware.com/cmake/cmake/-/issues/22033
We do use "INTERFACE_INCLUDE_DIRECTORIES" in oneMKL opensource in our oneMKLTargets.cmake (you can find it in your installation folder here lib/cmake/oneMKL/oneMKLTargets.cmake)
set_target_properties(MKL::onemkl PROPERTIES   INTERFACE_INCLUDE_DIRECTORIES "${_IMPORT_PREFIX}/include"
To make your example workable, just add after target_link_libraries
set_target_properties(test PROPERTIES NO_SYSTEM_FROM_IMPORTED true)

Hi @Rbiessy
I think I addressed all questions from this issue. Do you have any objections to close it?

Thank you for this @akabalov. I can confirm that my basic examples are working fine now so I am closing it.


Problem with press

Hi,
I printed my Homebrew and I have a problem with white empty spaces that appear in the page. I choose "Letter" option in print page, and in the print Preview I don't see white space and any problems. But when I print it's a disaster.
Can you help me?
Ty

Which size paper does your printer actually use? Is it A4?
On Sat, Nov 10, 2018, 8:30 AM Marcobertini6<EMAIL_ADDRESS>wrote:

Hi,
I printed my Homebrew and I have a problem with white empty spaces that
appear in the page. I choose "Letter" option in print page, and in the
print Preview I don't see white space and any problems. But when I print
it's a disaster.
Can you help me?
Ty
[image: prova]
https://user-images.githubusercontent.com/44923954/48301898-095cd780-e4f5-11e8-8983-a70e9adfcf07.jpg
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stolksdorf/homebrewery/issues/798, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFmzBsDBJSHRr0pNTvcxWYKj_oFEyCnaks5uttTfgaJpZM4YX_GQ
.

No Letter! (I add the photo)

Yes, so you are telling your printer to use Letter, but what size is the actual sheet of paper? I see you don't have English as your default language, so I assume the paper in your printer is actually A4, which is longer than Letter. Even if you tell the printer to print Letter size, the physical paper in your printer is probably A4.
You can change the format of your document to be taller and thinner, which will better match the A4 sheet size if you click the "Print" tab and then "A4 Pagesize". It will add a small bit of code to your homebrew so it fits on the different size paper.

I didn't see it! Ty very  much!
yes the language is italian =)
Ok, I see if I insert the code all pages change resolution. Now I must adapte all the pictures but it is enough? Now it will work?
Ty very much for the help, very helpful!!!!!

Glad I could help. I'm going to go ahead and close this. If you have more questions you can open this back up.

Ty, It's better but there is a little space white in bottom of the page (0.2-0.3cm), in which way I can avoid this?


NuGet package

If this is production ready would you mind submitting a NuGet package?

Good idea. Will do at some time

@JornWildt Is there anything we can help you with to get it published on nuget.org?

Sorry, was busy with other stuff. Its been years since I build this last time, so it will be interesting to see if it works.
Please try https://www.nuget.org/packages/Macaroons.Net/1.0.0 and let me know.

I tried to upgrade to Libsodium .10 (instead of .6) but then some of the tests started to fail signatures. I'm not going to try to fix that.

@JornWildt We are unable to use 4.7.2 in our project (actually we work with 4.6.1). Is it possible to make a package for an older .NET Framework version (or make it a multi target project)? The yesterdays version worked fine, btw =)

Please try again. Now targeting .NET 4.6.

@JornWildt Yes, now I can reference it in the project and nuget restores it. But I found another issue
0>/home/user/Macaroons.Net/packages/Baseclass.Contrib.Nuget.Output.2.1.0/build/net40/Baseclass.Contrib.Nuget.Output.targets(73,5): Error MSB4175 : The task factory "CodeTaskFactory" could not be loaded from the assembly "/usr/lib/mono/msbuild/Current/bin/Microsoft.Build.Tasks.vCurrent.dll". Invalid Image
0>------- Finished building project: Macaroons.Net. Succeeded: False. Errors: 1. Warnings: 0

As I can see, upgrading Baseclass.Contrib.Nuget.Output to v2.4.3 solves this problem when running Mono 6.8. This is pretty useful library and I don't know alternative Macaroons impls on C#, especially for Linux. Thank you!

Lets try again: https://www.nuget.org/packages/Macaroons.Net/1.0.2

Can I close this issue?

@JornWildt Hi! Thanks for rebuilding it. Looks like now I faced with https://github.com/baseclass/Contrib.Nuget/issues/57. Want to check if I can do anything with this.
Btw, I fixed tests on Linux: https://github.com/JornWildt/Macaroons.Net/pull/6.

@JornWildt Is there a reason why Baseclass.Contrib.Nuget.Output is not a development dependency? Looks like setting developmentDependency=true will exclude this dependency from .nupkg and probably everything will be okay.
And can you sign it to generate a PublicKeyToken?

And can you sign it to generate a PublicKeyToken?

Do you mean code signing or strong naming or NuGet signing? I cannot do code signing since the "Libsodium-net" package itself is unsigned.
Besides that, try again. V1.03 is with your pull requests.


Remove APIAdapter

Does the API adapter make sense at all?
Shouldn't we call the URLSession with extensions to call endpoint directly and pass some configuration?

Closed by #47.


Output schema reference by default

It might be nice to output the schema reference so that you get nice completions by default:
# yaml-language-server: $schema=https://raw.githubusercontent.com/prefix-dev/recipe-format/main/schema.json

Also, awesome tool! I think this will be super useful. First time giving it a go and it seems to work great! :heart:

Apologies for not responding to this sooner. I need to figure out why I'm not getting notified on new issues to this repo.
I'm not sure if I completely understand this request. Are you talking about completions for YAML for an IDE? I'm not sure what else you could be referring too.

Apologies for not responding to this sooner. I need to figure out why I'm not getting notified on new issues to this repo.
I'm not sure if I completely understand this request. Are you talking about completions for YAML for an IDE? I'm not sure what else you could be referring too.

Not the author of the issue but I think this feature isn't necessary anymore. The schema is nowadays on the schema store and every file called "recipe.yaml" will get completions and error messages in IDEs

Apologies for not responding to this sooner. I need to figure out why I'm not getting notified on new issues to this repo.
I'm not sure if I completely understand this request. Are you talking about completions for YAML for an IDE? I'm not sure what else you could be referring too.

Not the author of the issue but I think this feature isn't necessary anymore. The schema is nowadays on the schema store and every file called "recipe.yaml" will get completions and error messages in IDEs

True - completions do work by default now so this isn't needed anymore.


CoDi package issue with \mathbb{C} in object names

I am currently using the CoDi package to create commutative diagrams in LaTeX and I encountered an issue that I hope you could help me with.
Here is the minimal example to reproduce the problem:
\begin{codi}
    \obj { A_{\mathbb C} & B \\ };
    \mor A_{\mathbb C} f:-> B;
\end{codi}

Hi @I0amLK! Thanks for using CoDi and sorry for the delay.
The mechanism that automatically names nodes drops the special \ character, so you should drop it when referring to the nodes.
Here's a working example:
\documentclass[margin=.5cm]{standalone}
\usepackage{commutative-diagrams}
\usepackage{amsfonts}

\begin{document}

\begin{codi}[prompter]
    \obj { A_{\mathbb C} & B \\ };
    \mor A_{mathbb C} f:-> B;
\end{codi}

\end{document}

The prompter option decorate nodes with their names, so it's useful when you're unsure about automatically generated names:


[REVM-13] Test with Aaron

From SyncLinear.com | REVM-13

Hi how are you

Another

sander ping

I have sync enabled now


modify message type get value

Hello, while I was looking at this project with interest, I modified the MessageType within the service method so that the item that retrieves the value through getValue can be returned directly from the AbstractMessage class, which is an abstract method.
Additionally, in the switch-case statement of convertToMessage, in addition to the value declared as the existing MessageType, I changed the part that was repeatedly declared as a string to use the value within the MessageType.

I'm not super convinced on the change from getMessageType().getValue() to getMessageTypeValue()  but I suppose it is a bit more readable so will merge it.  The catch on using the string instead of the type is a good one!
Thanks

merged as 5ff2c3bb925df7a919cb3ab890a2e4de56e57569


Better event handling + new helpdesk event added

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Aleksandr seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Support records, variants, enums, unions, and flags in the component model

I'm splitting this issue out of https://github.com/bytecodealliance/wasmtime/issues/4185 to write up some thoughts on how this can be done. Specifically today the current Wasmtime support for the component model has mappings for many component model types to Rust native types but not all of them. For example integers, strings, lists, tuples, etc, are all mapped directly to Rust types. Basically if the component model types equivalent in Rust is in the Rust standard library that's already implemented. What that leaves to implement, however, is Rust-defined mappings for component model types that are "structural" like records.
This issue is intended to document the current thinking of how we're going to expose this. The general idea is that we'll create a proc-macro crate, probably named something like wasmtime-component-macro, which is an internal dependency of the wasmtime crate. The various macros would then get reexported at the wasmtime::component::* namespace.
Currently the bindings for host types are navigated through three traits: ComponentValue, Lift, and Lower. We'll want a custom derive for all three of these traits. Deriving Lift and Lower require a ComponentValue derive as well, but users should be able to pick one of Lift and Lower without the other one.
record
Records in the component model correspond to structs in Rust. The rough shape of this will be:
use wasmtime::component::{ComponentValue, Lift, Lower};

#[derive(ComponentValue, Lift, Lower)]
#[component(record)]
struct Foo {
    #[component(name = "foo-bar-baz")]
    a: i32,
    b: u32,
}

To typecheck correctly the record type must list fields in the same order as the fields listed in the Rust code for now. Field reordering may be implemented at a later date but for now we'll do strict matching. Fields must have both matching names and matching types.
The #[component(record)] here may seem redundant but it's somewhat required below for variants/enums.
The #[component(name = "...")] is intended to rename the field from the component model's perspective. The type-checking will test against the name specified.
Using this derive on a tuple or empty struct will result in a compile-time error.
variant
Variants roughly correspond to Rust enums:
use wasmtime::component::{ComponentValue, Lift, Lower};

#[derive(ComponentValue, Lift, Lower)]
#[component(variant)]
enum Foo {
    #[component(name = "foo-bar-baz")]
    A(u32),
    B,
}

Typechecking, like records, will check cases in-order and all cases must match in both name and payload. A missing payload in Rust is automatically interpreted as the unit payload in the component model.
Variants with named fields (B { bar: u32 }) will be disallowed. Variants with multiple payloads (B(u32, u32)) will also be disallowed.
Note that #[component(variant)] here distinguishes it from...
enum
use wasmtime::component::{ComponentValue, Lift, Lower};

#[derive(ComponentValue, Lift, Lower)]
#[component(enum)]
enum Foo {
    #[component(name = "foo-bar-baz")]
    A,
    B,
}

Typechecking is similar to variants where the number/names of cases must all match.
Variants with any payload are disallowed in this derive mode.
union
This will, perhaps surprisingly, still map to an enum in Rust since this is still a tagged union, not a literal C union:
use wasmtime::component::{ComponentValue, Lift, Lower};

#[derive(ComponentValue, Lift, Lower)]
#[component(union)]
enum Foo {
    A(u32),
    B(f32),
}

The number of cases and the types of each case must match a union definition to correctly typecheck. Union cases don't have names so renaming here isn't needed.
A payload on each enum case in Rust is required, and like with variant it's required to be a tuple-variant with only one element. All other forms of payloads are disallowed. Note that the names in Rust are just informative in Rust, it doesn't affect the ABI or type-checking
flags
These will be a bit "funkier" than the above since there's not something obvious to attach a #[derive] to:
wasmtime::component::flags! {
    #[derive(Lift, Lower)]
    flags Foo {
        #[component(name = "...")]
        const A;
        const B;
        const C;
    }
}

The general idea here is to roughly take inspiration from the bitflags crate in terms of what the generated code does. Ideally this should have a convenient Debug implementation along with various constants to OR-together and such in Rust. The exact syntax here is up for debate, this is just a strawman.
Implementation Details
One caveat is that the ComponentValue/Lift/Lower traits mention internal types in the wasmtime crate which aren't intended to be part of the public API. To solve this the macro will reference items in a path such as:
wasmtime::component::__internal::the_name

The __internal module will be #[doc(hidden)] and will only exist to reexport dependencies needed by the proc-macro. This crate may end up having a bland pub use wasmtime_environ or individual items, whatever works best.
The actual generated trait impls will probably look very similar to the implementations that exist for tuples, and Result<T, E> already present in typed.rs
Alternatives
One alternative to the above is to have #[derive(ComponentRecord)] instead of #[derive(ComponentValue)] #[component(record)] or something like that. While historically some discussions have leaned in this direction with the introduction of Lift and Lower traits I personally feel that the balance is now slightly in the other direction where it would be nice if we can keep derive targeted at the specific traits and then configuration for the derive happens afterwards.

cc @dicej as I believe you were possibly interested in taking a stab at this

Thanks, @alexcrichton -- this is extremely helpful.
BTW, you refer to ComponentValue here and in some of the comments in typed.rs, but the trait is actually named ComponentType, correct?
Anyway, yes, I'll take a stab at this and post questions here as they arise.

Whoops sorry yes it's ComponentType (can't keep track of my own historical list of changes)

Support for record types has been merged.  I'm working on variants now and will move on to the others after that.

Hm I'm not sure I understand, those both look pretty reasonable to me and like they should work. Could you clarify which part you're thinking probably needs an error?

I guess I assumed duplicate types (e.g. two f32s) in a union type with no way to distinguish between them would be a problem.  If not, then I guess (union float32 float32) is equivalent to (union float32) and both are equally valid?

Ah ok, my impression is that (union float32 float32) is valid in the component model and because it's a tagged discriminant you can distinguish between the two cases, albeit it does seem a bit silly.

Okay -- I guess I got thrown off by the fact that the cases aren't named.  Good point about them having separate discriminants, i.e. you can distinguish them based on the order in which they appear.

Would it make sense to have a custom syntax for unions, like you suggested for flags?  With flags, only the names need to be specified.  With unions, only the types need to be specified.  Seems kind of awkward to make the programmer choose names that will be ignored anyway.

Arguing with myself: the programmer will still use those names in Rust code, so they're not useless.  Nevermind my suggestion above.

In general I'm happy to leave it to your discretion. My assumption was that a union in the component model is represented in Rust with an enum one way or another so I figured "may as well let the input have the names" but I don't really mind one way or another.

I found it helpful to think of union cases as still being "named". There's a bit of text in the Type Definitions section of the component model explainer that describes union and enum as syntactic sugar for variants:
                (enum <name>+) ↦ (variant (case <name> unit)+)
            (union <valtype>+) ↦ (variant (case "𝒊" <valtype>)+) for 𝒊=0,1,...

So the case names for unions are "0", "1", ...
I've been tempted to suggest that we should expect a union type for any Rust enum where the cases are named _0, _1, ... But that might be too much magic.

Thanks for the feedback, @alexcrichton and @jameysharp .  Let's stick with the original plan; being able to specify meaningful names can be nice for use in Rust code, even if they don't get carried over into the component interface.

This is excellent work, @dicej! :+1:

How should we handle enums with explicit discriminants?  For example:
#[derive(ComponentValue, Lift, Lower)]
#[component(enum)]
enum Foo {
    #[component(name = "foo-bar-baz")]
    A = 42,
    B,
}

Should we ignore them?  Throw an error?
Similarly, should we pay any attention to e.g. #[repr(u16)] annotations on the target enum?

I think #[repr(u16)] can be ignored. As for discriminants I think those can be ignored as well as only the name matters for the wasm component model. They might be used for interacting with native code through a C abi for example.

I think #[repr(u16)] can be ignored. As for discriminants I think those can be ignored as well as only the name matters for the wasm component model. They might be used for interacting with native code through a C abi for example.

Agreed.  I think the guiding principle here is: just because a type is annotated with #[derive(ComponentValue)] doesn't mean that the only purpose of the type is for interop with Wasm components.  It could be used for C interop, among other things.

My assumption had been that setting an explicit discriminant would be an error. But, I guess we already keep the Canonical ABI used on the wasm side entirely independent of whatever ABI happens to be used on the Rust side. So yes, you've convinced me that neither explicit discriminants nor #[repr] matter for this purpose.

How does wit-bindgen fit into this? I'm currently trying to create a wasmtime::component::Component from a wasm component with the following export interface file:
record pci-device {
    vendor-id: u16,
    device-id: u16,
}

init: func(dev: pci-device)

but have ran into this unimplemented!() statement.
Is the idea that I would have a corresponding struct definition in my Rust i.e.:
#[derive(ComponentValue, Lift, Lower)]
#[component(record)]
struct PciDevice {
    #[component(name = "vendor-id")]
    vendor_id: u16,
    #[component(name = "device-id")]
    device_id: u16,
}

That I would use when interacting with the module (e.g. TypedFunc type parameters)?
Also, as a side note, if someone could give me some pointers on implementing "component type export" to remove the unimplemented!() statement, I'd be happy to give it a shot.

Currently wit-bindgen isn't updated to work with the component model support in Wasmtime, but yes as you've seen I'm imagining that's what wit-bindgen will one day generate.
Can you open a separate issue for the unimplemented!() block you're hitting? Type imports/exports I didn't think were even generated by wit-bindgen so that you're hitting that block at all may be a bug elsewhere that I think would be good to investigate.

#4414 should be the last PR for this issue :crossed_fingers:

I believe this can be closed now that #4414 has been merged.


Error Payment with Stripe

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
2.1.2
Issue Description
Used Codenteq – Stripe Payment Gateway with testing card.
After paid an error
Class "Webkul\Sales\Transformers\OrderResource" not found
Preconditions
Install, use, etc
Steps to reproduce
Install, use, etc
Expected Result
Class "Webkul\Sales\Transformers\OrderResource" not found
Actual Result
Callback to Shop
Screenshots

Additional context
No response

@ahmetarsiv thank you for reply!
But I don't have the folder /Transformers and file OrderResource

Hello @plyaskin,
Thank you for your message.
The folders and files you're looking for should indeed be located in the specified directory: Webkul/Sales/src.
If you don't see the /Transformers folder and OrderResource file, it might be due to one of the following reasons:

Version Difference: Ensure that you are using the same version of Bagisto as the one in the repository. There might be differences in the directory structure between versions.

Incomplete Clone/Download: If you cloned or downloaded the repository, ensure that the process was completed successfully. Sometimes, network issues or interruptions can cause incomplete downloads.

Custom Modifications: If you or your team have made custom modifications to the Bagisto installation, those folders and files might have been moved or deleted.

To resolve this, you can:

Check the Version: Verify that your version matches the one in the repository.
Re-clone the Repository: Try re-cloning the repository to ensure you have all the necessary files.
Manually Add Files: If the files are missing, you can manually add them from the repository.

If you continue to experience issues, please provide more details about your installation and setup so I can assist you further.
Best regards,
Ahmet

I did it yesterday and the installation was finished successfully.

Hello @plyaskin,
Thank you for your response.
Since your installation finished successfully and you haven't changed any files except the .env file, I recommend reviewing the Bagisto repository again to ensure all files are present: Bagisto.
If everything appears correct in the repository and you still face issues, it might be worth re-cloning the repository and repeating the installation process to rule out any possible errors during the initial clone.
If there are no further issues or questions, I will proceed to close this issue. However, if you encounter any other problems, feel free to reopen the issue or create a new one.
Best regards,
Ahmet

@ahmetarsiv I have loaded the folder with files and restarted the payment twice.

Have you fixed this ?


feat: Algebra.Group.Defs add some #align

Also changing Vsub to VSub.
There are two lemmas in mathlib3 aligned to zpow_ofNat, both zpow_coe_nat and zpow_of_nat, I'm not sure if this is a problem.
I didn't add #align to every declaration, only the ones where I thought there might be a problem.

bors merge


next.js API routes failing on Netlify

Bug description
Hello guys, awesome work with the library! I have enjoyed working with Prisma a lot!
Regarding the error that I have, it's basically that Prisma is not getting along with Netlify, I'm creating a next.js app and I get this error whenever I try to use the next.js API routes:
errorMessage: "Error: Query engine binary for current platform "rhel-openssl-1.0.x" could not be found.↵This probably happens, because you built Prisma Client on a different platform.↵(Prisma Client looked in "/query-engine-rhel-openssl-1.0.x")↵↵Searched Locations:↵↵  /.prisma/client↵  /opt/build/repo/node_modules/@prisma/client↵  /↵  /var/task/node_modules/.prisma/client↵  /var/task↵  /var/task/node_modules/.prisma/client↵↵You already added the platform "native" to the "generator" block↵in the "schema.prisma" file as described in https://pris.ly/d/client-generator,↵but something went wrong. That's suboptimal.↵↵Please create an issue at https://github.com/prisma/prisma-client-js/issues/new"
errorType: "Runtime.UnhandledPromiseRejection"

I already look for solutions in Google, but I wasn't able to find much besides adding: target: "experimental-serverless-trace" to my next.config which didn't help. I found an already closed bug: https://github.com/prisma/prisma/issues/4011 which mentions that 2.11 fixed this issue, but I'm working with "@prisma/client": "^2.17.0", and it's still happening, I don't know if it's related or if the error is different.
How to reproduce
Create a next.js app, using yarn and installing prisma 2.17 and the following generator:
generator client {   provider        = "prisma-client-js"   previewFeatures = ["createMany"]   binaryTargets   = ["native"] }
Implement any kind of API route and try to deploy to Netlify, deployment is going to be "successful", but when you try to use the actual API it's going to throw that error.
Expected behavior
prisma to work fine on Netlify, it's working good in local.
Prisma information
I'm using postgres with the generator mentioned above.
Environment & setup
Only with Netlify, I'm using yarn and next.js 10.0.6 in case that matters! Additionaly, I added npx prisma generate to the deployment process to see if anything changed, but it's not working either, I do was able to see some errors by setting DEBUG to *:
8:55:09 AM: 2021-02-26T13:55:09.989Z prisma:tryLoadEnv Environment variables not found at null
8:55:09 AM: 2021-02-26T13:55:09.991Z prisma:tryLoadEnv Environment variables not found at undefined
8:55:09 AM: 2021-02-26T13:55:09.991Z prisma:tryLoadEnv No Environment variables loaded
8:55:10 AM: Prisma schema loaded from prisma/schema.prisma
8:55:10 AM: { prismaClientDir: '/opt/build/repo/node_modules/@prisma/client' }
8:55:10 AM: __dirname /opt/build/repo/node_modules/@prisma/client/generator-build
8:55:10 AM: /opt/build/repo/node_modules/@prisma/client/generator-build
8:55:10 AM: 2021-02-26T13:55:10.866Z prisma:GeneratorProcess 2021-02-26T13:55:10.866Z prisma:loadEnv Error: Cannot find module '/opt/build/repo/node_modules/.prisma/package.json'
8:55:10 AM: 2021-02-26T13:55:10.866Z prisma:GeneratorProcess Require stack:
8:55:10 AM: 2021-02-26T13:55:10.866Z prisma:GeneratorProcess -<EMAIL_ADDRESS>8:55:10 AM: 2021-02-26T13:55:10.867Z prisma:GeneratorProcess     at Function.Module._resolveFilename (internal/modules/cjs/loader.js:966:15)
8:55:10 AM: 2021-02-26T13:55:10.867Z prisma:GeneratorProcess     at Function.Module._load (internal/modules/cjs/loader.js:842:27)
8:55:10 AM: 2021-02-26T13:55:10.867Z prisma:GeneratorProcess     at Module.require (internal/modules/cjs/loader.js:1026:19)
8:55:10 AM: 2021-02-26T13:55:10.867Z prisma:GeneratorProcess     at require (internal/modules/cjs/helpers.js:72:18)
8:55:10 AM: 2021-02-26T13:55:10.867Z prisma:GeneratorProcess     at /opt/build/repo/node_modules/@prisma/client/generator-build/index.js:72570:24
8:55:10 AM: 2021-02-26T13:55:10.867Z prisma:GeneratorProcess     at runMatcher (/opt/build/repo/node_modules/@prisma/client/generator-build/index.js:72287:25)
8:55:10 AM: 2021-02-26T13:55:10.867Z prisma:GeneratorProcess     at AsyncFunction.module2.exports.sync (/opt/build/repo/node_modules/@prisma/client/generator-build/index.js:72294:25)
8:55:10 AM: 2021-02-26T13:55:10.867Z prisma:GeneratorProcess     at getProjectRootEnvPath (/opt/build/repo/node_modules/@prisma/client/generator-build/index.js:72566:43)
8:55:10 AM: 2021-02-26T13:55:10.867Z prisma:GeneratorProcess     at Object.getEnvPaths2 (/opt/build/repo/node_modules/@prisma/client/generator-build/index.js:72545:31)
8:55:10 AM: 2021-02-26T13:55:10.867Z prisma:GeneratorProcess     at TSClient.toJS (/opt/build/repo/node_modules/@prisma/client/generator-build/index.js:78157:34)
8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess 2021-02-26T13:55:10.866Z prisma:loadEnv Error: Cannot find module '/opt/build/repo/node_modules/package.json'
8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess Require stack:
8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess -<EMAIL_ADDRESS>8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess     at Function.Module._resolveFilename (internal/modules/cjs/loader.js:966:15)
8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess     at Function.Module._load (internal/modules/cjs/loader.js:842:27)
8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess     at Module.require (internal/modules/cjs/loader.js:1026:19)
8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess     at require (internal/modules/cjs/helpers.js:72:18)
8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess     at /opt/build/repo/node_modules/@prisma/client/generator-build/index.js:72570:24
8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess     at runMatcher (/opt/build/repo/node_modules/@prisma/client/generator-build/index.js:72287:25)
8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess     at AsyncFunction.module2.exports.sync (/opt/build/repo/node_modules/@prisma/client/generator-build/index.js:72294:25)
8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess     at getProjectRootEnvPath (/opt/build/repo/node_modules/@prisma/client/generator-build/index.js:72566:43)
8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess     at Object.getEnvPaths2 (/opt/build/repo/node_modules/@prisma/client/generator-build/index.js:72545:31)
8:55:10 AM: 2021-02-26T13:55:10.868Z prisma:GeneratorProcess     at TSClient.toJS (/opt/build/repo/node_modules/@prisma/client/generator-build/index.js:78157:34)
8:55:11 AM: 2021-02-26T13:55:11.391Z prisma:tryLoadEnv Environment variables not found at null
8:55:11 AM: 2021-02-26T13:55:11.392Z prisma:tryLoadEnv Environment variables not found at undefined
8:55:11 AM: 2021-02-26T13:55:11.392Z prisma:tryLoadEnv No Environment variables loaded
8:55:11 AM: ✔ Generated Prisma Client (2.17.0) to ./node_modules/@prisma/client in 294ms
8:55:11 AM: You can now start using Prisma Client in your code. Reference: https://pris.ly/d/client
8:55:11 AM: ```
import { PrismaClient } from '@prisma/client'
const prisma = new PrismaClient()

I wasn't able to find anything related to that error with the build/index.js
I would appreciate any help that could be provided with this error! :)

Hey @NicolasBonet
Try this:
generator client {
provider = "prisma-client-js"
 previewFeatures = ["createMany"]
binaryTargets = ["native","rhel-openssl-1.0.x"]
}

You can also checkout this e2e test: https://github.com/prisma/e2e-tests/tree/dev/platforms/netlify-cli

Hello @pantharshit00,
Thank you very much, at the end, I just deployed to Vercel and everything worked without further configuration, so at least for me it's no longer a problem.

Okay, closing this for now.


small typo in documentation - download page

just fixed a small typo

Thanks for the fixes @aconstantin!


Issues reading in data from ACS_NSQIP_PUF_05_06_vr1 and acs_nsqip_puf19

Hello!
Running the nsqip() function with a directory of txt files does not read in ACS_NSQIP_PUF_05_06_vr1 for some reason (i.e. starts with the next file in the directory). Also, when the function gets to cleaning acs_nsqip_puf19, it stops and throws an error (copied below):
Error in if (unique(df[["pufyear"]]) > 2010) { :
missing value where TRUE/FALSE needed
Not sure where and why exactly this is occurring in the code.
Thank you for your help!

Have you tried re-running this since my last master branch update? I have made some significant changes. Sorry for the long response time here, I was away from personal projects for a while due to work.

Hi,
So to be clear, the steps you are taking are:

Download master branch
Run nsqip() on a directory

Can you describe the directory structure and which files are in it?

Hi Dylan,
Thanks for getting back to me on this.
Yes, those are the steps that I took. I downloaded the latest version of
the package via the line of code you suggested on your Github page:
devtools::install_github("dylanrussellmd/nsqipr")
And I ran nsqip() on a directory titled "nsqip_test" that contained just
the txt file for 2005-2006. I have attached a zip folder with the file
folder created after the function had finished running. As you can see,
there are no long table csv files for the CPT codes, and the actual Excel
sheet has completely missing entries for all fields except the year (which
based on your code is filled in based on the name of the file).
Let me know what you think the issue could be here. Really appreciate your
help on this.
Regards,
Rushmin
nsqip_test.zip
https://drive.google.com/file/d/1uQkyNLDArdV071LVcggf9Y3kl8CSYoac/view?usp=drive_web
On Mon, Jul 4, 2022 at 10:30 AM Dylan Russell @.***>
wrote:

Hi,
So to be clear, the steps you are taking are:

Download master branch
Run nsqip() on a directory

Can you describe the directory structure and which files are in it?
—
Reply to this email directly, view it on GitHub
https://github.com/dylanrussellmd/nsqipr/issues/15#issuecomment-1173886416,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APCVOVRTUMTSDC5RABXWC7DVSLYORANCNFSM46LHJOIA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Rushmin Khazanchi (he/him/his)
M.D. Candidate, Feinberg School of Medicine
B.A. Biological Sciences, Northwestern '21
(412) 538-8934 | @.*


What type of update are you requesting?
A new version of an existing package
Current Package Identifier
JetBrains.DataGrip.EarlyAccess
Package Version
232.7295.8
Please describe the changes you would like to see
There is new release (232.7295.8) available
Download Page Url: https://www.jetbrains.com/datagrip/nextversion/

This issue can be closed as "Completed". 😄

https://github.com/microsoft/winget-pkgs/pull/120655
https://github.com/microsoft/winget-pkgs/pull/120654
https://github.com/microsoft/winget-pkgs/pull/120653
https://github.com/microsoft/winget-pkgs/pull/120652


Changed requirements for using Enzyme in README.md

The existing tutorial triggered errors and warnings.
Starting with React 15.5 the requirements for Enzyme changed. The up to date installation instructions can be seen here: https://github.com/airbnb/enzyme#installation
I have opted to leave the instructions for React <15.5 as a note in parentheses and not write both bash commands because users following this tutorial are instructed to first install Create React App and then use it to generate a new React app. In other words, most readers will have the latest React version.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Shift + [numeric keypad Enter] runs code as it was before cell edit

There are two Enter buttons on my keyboard. The one that is on the main pad behaves as expected, but the one that is at the numeric keypad sometimes causes old code being run.
Steps:

create new blank notebook;
write:
i = 9
print(i)
press Shift+Enter
edit cell, replace 9 with 3
press Shift+Enter (the one at numeric keypad)

Expected output: 3
Actual output: 9

This does not happen, when using the other Enter key. This happens only if Shift+Enter is used right after edit (does not happen, if notebook is saved between edit and run or if you move cursor to another cell and back between edit and run).

Shift + numeric key pad enter doesn't submit for me (it enters a new line)? What OS are you running on?
And if you go back and use the regular enter, does it work again?
Thanks.

Windows 10.
Both Enters together with Shift for me runs the code in the cell.
"And if you go back and use the regular enter, does it work again?" <- yes. If i go back and use the "bad" Enter, it also works correctly.
The only time I see the unexpected behavior is when I run the cell with "bad" Enter right after editing the cell.

@ilze02  What version of the python extension are you using?
@rchiodo

Python extension:
Version: 2019.11.50794
VSCode:
Version: 1.41.0 (user setup)
Commit: 9579eda04fdb3a9bba2750f15193e5fafe16b959
Date: 2019-12-11T18:37:42.077Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.17763

@greazer, more info needed?

Thanks for the information, please could you try upsetting to the latest version of the extension?

Thanks for the feedback! However, we don't have plans on adding this functionality at this time. VS Code itself is investigating making notebooks a native experience in the editor. This particular problem will be solved by that effort.


Bump Version to 0.9.2-SNAPSHOT

After we did a lot at the hacktoberfest, let's bump the Version.

Jup, I agree! Later today sounds good. 👍

0.9.1 it not released yet, no need to bump


Make provisioning network optional

https://github.com/stolostron/backlog/issues/22473

/hold pending approval

/unhold


Automate invex management

Blocked on #462 as an extension to that task
Make Helpmebot manage the IRC invex list on suspend/approve. We'd also need to factor in extensions for changes of invex mask, and make this available to tool admins only, and extend the HMB notification interface into a commanding interface.
This is probably much more of a Helpmebot task than an ACC one.

I don't think this is particularly feasible in a post-multiproject world. It's also going to be a pain generally on the Helpmebot side, so I'm not really interested in implementing this at this time.


Demo of town

What changes do you make in this PR?

Please describe why you create this PR

Checklist

[x] I have merged the latest main branch into current branch.
[x] I have run bash scripts/format.sh before merging.
Please use "squash and merge" mode.

Tests is verified in local machine, merge


suggested search terms - autofill

A user suggested we can try to autofill a user's search with suggestions as they are typing. This would probably require a special lighthouse endpoint and some thought into the design.

YouTube also shows recent searches, which is quite useful sometimes.


Unable to set buyerDetails and NotifyURL

Hi ,
I am not able to set notifyurl . When invoice is created i got a response which contains invoice url but address is not shown so i was not even able to set address . I tried following
$invoice_arr = [ 'amount' => '10.00',
            'currency' => 'USD',
            'transactionId' => '1',
            'description' => 'Product description',
            'notifyUrl' => 'https://www.kodeinfo.com/notify',
            'notifyEmail' =><EMAIL_ADDRESS>            'returnUrl' => 'https://www.kodeinfo.com/return',
            "buyerName"=>"Imran","buyerAddress1"=>"Rein bazar",
            "buyerAddress2"=>"Rein bazar","buyerCity"=>"Hyderabad",
            "buyerState"=>"AP","buyerZip"=>"500023",
<EMAIL_ADDRESS>            "buyerPhone"=>"mobileno"];

and also
        $card = new \Omnipay\Common\CreditCard();
        $card->setName('Imran');
        $card->setAddress1('Rein Bazar');
        $card->setAddress2('');
        $card->setCity('Hyderabad');
        $card->setState('Telangana');
        $card->setPostcode('500023');
        $card->setCountry('India');
        $card->setEmail('someemail@gmail.com');
        $card->setPhone('mobileno');

$invoice_arr = [ 'amount' => '10.00',
            'currency' => 'USD',
            'transactionId' => '1',
            'description' => 'Product description',
            'notifyUrl' => 'https://www.kodeinfo.com/notify',
            'notifyEmail' =><EMAIL_ADDRESS>            'returnUrl' => 'https://www.kodeinfo.com/return',
            'card'=>$card];

unable to set fullnotifications and not getting any email when invoice expires
Thanks for your package

I had a similar issue. My development server was locked with HTTP Basic Authentication so BitPay was unable to send the notification request - even when I added username:password to notifyUrl!
There are also certain other requirements to make it work.
See IPN Troubleshooting.

Hi there,
I also need to set the fullNotifications, and i can't seem to find a workaround. Can someone help me with this?
Thanks for your work!
Bert


planning: tune the parameter and refine the calculation of grid search

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