lsscsi: 0.29 -> 0.30

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/lsscsi/versions.
meta.description for lsscsi is: '""'.

Checks done (click to expand)

built on NixOS
/nix/store/12rq6wpmajgly4xfbag3rkmf35r1w1g0-lsscsi-0.30/bin/lsscsi passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.30 with grep in /nix/store/12rq6wpmajgly4xfbag3rkmf35r1w1g0-lsscsi-0.30
directory tree listing: https://gist.github.com/58f176f1241e035b32d0fb2ab3588db2
du listing: https://gist.github.com/00a5b398854a7d9ba1466a72c6ccbfb2

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 314fc1dc1c91d29c396344545d9261bb1cd83ff4 && nix-shell --pure -I nixpkgs=. -p lsscsi

No attempt on x86_64-linux (full log)
The following builds were skipped because they don't evaluate on x86_64-linux: lsscsi
Partial log (click to expand)
Cannot nix-instantiate `lsscsi' because:
GC Warning: Failed to expand heap by<PHONE_NUMBER>0 bytes
Can't start up: not enough memory

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: lsscsi
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Failure on aarch64-linux (full log)
Attempted: lsscsi
Partial log (click to expand)
copying path '/nix/store/w03lwx8r49abg54k81ypg87ckqklgafd-libssh2-1.8.0-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/ssr9dhrp11h4lnr35p05bbqc1ln5004d-coreutils-8.29' from 'https://cache.nixos.org'...
copying path '/nix/store/gncd1vwkz1l16qnvdr09cdvphr5agcz8-gnutar-1.30' from 'https://cache.nixos.org'...
acquiring/releasing lock: Resource deadlock avoided
copying path '/nix/store/qia4njnsr2zmlghd5wn0iz6drimy46xr-nghttp2-1.32.0-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/574ir3lcdcpff7wpv87686gav7lminmq-gcc-7.3.0' from 'https://cache.nixos.org'...
acquiring/releasing lock: Resource deadlock avoided
cannot build derivation '/nix/store/5ilgr8hrqszagrlbsiyq4wmqi32x8i54-lsscsi-0.30.tgz.drv': 2 dependencies couldn't be built
cannot build derivation '/nix/store/35a9qak45f4k5m7sldzzy6kg7dkcrgxy-lsscsi-0.30.drv': 2 dependencies couldn't be built
error: build of '/nix/store/35a9qak45f4k5m7sldzzy6kg7dkcrgxy-lsscsi-0.30.drv' failed

@GrahamcOfBorg build lsscsi

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: lsscsi
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: lsscsi
Partial log (click to expand)
make[1]: Leaving directory '/build/lsscsi-030r154'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/d129h7j6ki31z84zx8x5lzj21gla8r52-lsscsi-0.30
shrinking /nix/store/d129h7j6ki31z84zx8x5lzj21gla8r52-lsscsi-0.30/bin/lsscsi
gzipping man pages under /nix/store/d129h7j6ki31z84zx8x5lzj21gla8r52-lsscsi-0.30/share/man/
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/d129h7j6ki31z84zx8x5lzj21gla8r52-lsscsi-0.30/bin
patching script interpreter paths in /nix/store/d129h7j6ki31z84zx8x5lzj21gla8r52-lsscsi-0.30
checking for references to /build in /nix/store/d129h7j6ki31z84zx8x5lzj21gla8r52-lsscsi-0.30...
/nix/store/d129h7j6ki31z84zx8x5lzj21gla8r52-lsscsi-0.30

Success on aarch64-linux (full log)
Attempted: lsscsi
Partial log (click to expand)
make[1]: Leaving directory '/build/lsscsi-030r154'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/aigqhqd59jvri60wr9hibdgdzp69ijyn-lsscsi-0.30
shrinking /nix/store/aigqhqd59jvri60wr9hibdgdzp69ijyn-lsscsi-0.30/bin/lsscsi
gzipping man pages under /nix/store/aigqhqd59jvri60wr9hibdgdzp69ijyn-lsscsi-0.30/share/man/
strip is /nix/store/ah0va6j4cnwj9nx4j6rwcfc8nh785jwm-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/aigqhqd59jvri60wr9hibdgdzp69ijyn-lsscsi-0.30/bin
patching script interpreter paths in /nix/store/aigqhqd59jvri60wr9hibdgdzp69ijyn-lsscsi-0.30
checking for references to /build in /nix/store/aigqhqd59jvri60wr9hibdgdzp69ijyn-lsscsi-0.30...
/nix/store/aigqhqd59jvri60wr9hibdgdzp69ijyn-lsscsi-0.30


Update ffi dependency

ffi gem released version 1.10.
Can the sassc gem dependency be relaxed to include this release? https://github.com/ffi/ffi/blob/master/CHANGELOG.md#1100--2019-01-06

IMO the dependency has not been relaxed enough, PR #102 contains
spec.add_dependency "ffi", "~> 1.9"
but current FFI version is 1.10
(FFI 1.10 is required by to others gems (like rb-inotify) which breaks my bundle install)

@ut, ~> 1.9 will also match 1.10. Indeed it's the same as ['>= 1.9', '< 2.0'] if I'm not mistaken. The only thing missing is a release with this change.

2.0.1, which includes this change, was released this morning

Oh, I didn't check for updates today so far, good to know. Thanks for the release!

Thanks @aried3r for the info, thanks @bolandrm for the release!


Make it possible to specify the license types of NPM test artifacts

Task Description
It should be possible to specify under what license a generated test artifact is. This should be controllable via the implementation of the org.carlspring.strongbox.testing.artifact.NpmTestArtifact annotation and org.carlspring.strongbox.artifact.generator.NpmArtifactGenerator.
An artifact can have no, one, or more licenses. By default, it should not have one.
Tasks
The following tasks will need to be carried out:

[ ] Make the necessary changes to the org.carlspring.strongbox.testing.artifact.NpmTestArtifact.

[ ] Add a LicenseConfiguration[] licenses() default {}; interface property. Use license contants from org.carlspring.strongbox.testing.artifact.LicenseType,

[ ] Make the necessary changes to the org.carlspring.strongbox.artifact.generator.NpmArtifactGenerator.

[ ] Implement license file copying. If specified, these would be a mapping for the licenses via the LicenseConfiguration to where the LICENSE.md files from strongbox-storage/strongbox-storage-api/src/test/resources/licenses/${license_short_name} should be copied to (check #1723 and, especially, #1732  for more details). Figure out what the default locations for license files in NPM should be (most-probably in the root).
[ ] Make sure the produced package.json contains the list of licenses.

[ ] Make the necessary changes to the org.carlspring.strongbox.testing.artifact.NpmArtifactGeneratorStrategy.
[ ] Add a test illustrating:

[ ] An artifact with no licenses.
[ ] An artifact with more than one licenses.
[ ] An artifact with a licenses in different destination paths.

Task Relationships
This is a sub-task of: #1723 . Check #1732 for details of how this was implemented for Maven.
Useful Links

Strongbox Developer Guide: Writing Tests

Help

Our chat
Points of contact:

@carlspring
@sbespalov
@steve-todorov

I am working on it.

@carlspring I will pick this one if no one is working on it.

Thanks, @anki2189 ! :+1:


Thanks again for your guidance

Hello good friend
With your help, I wrote a direct download extension for myket.
Glad to review it and introduce it to your friends.
Thanks again for your guidance.

I'm glad to hear it, thanks I'll check your add-on this week. :)


ERROR [root] Error: Multiple head revisions are present for given argument 'head'; please specify a specific target revision, '@head' to narrow to a specific head, or 'heads' for all heads

How to reproduce the bug
Expected results
what you expected to happen.
Actual results
what actually happens.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment
(please complete the following information):

browser type and version:
superset version: superset version
python version: python --version
node.js version: node -v
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[ ] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
superset upgrade from version 0.37.2
superset db upgrade
logging was configured successfully
INFO:superset.utils.logging_configurator:logging was configured successfully
/usr/local/py3env/lib/python3.6/site-packages/flask_caching/init.py:192: UserWarning: Flask-Caching: CACHE_TYPE is set to null, caching is effectively disabled.
"Flask-Caching: CACHE_TYPE is set to null, "
INFO  [alembic.runtime.migration] Context impl MySQLImpl.
INFO  [alembic.runtime.migration] Will assume transactional DDL.
ERROR [root] Error: Multiple head revisions are present for given argument 'head'; please specify a specific target revision, '@head' to narrow to a specific head, or 'heads' for all heads

@qinlz-1 can you specify which version you are trying to upgrade to? One quick hack you may want to try is upgrading to 0.38.x, then 1.0.x, then 1.1.x etc, and see if that works. Also, Superset no longer supports Python 3.6, so you will need to upgrade to at least 3.7 if you want to run one of the latest versions.


Queries regarding future of bootstrap-vue-3

Hi @cdmoro,
I am JD. We at PIXINVENT created the most popular Vuexy Admin Template based on BootstrapVue.
As people are asking for Vue 3, we are checking your framework and its feasibility of migration for our template. We have queries before we start working on it, it will help us proceeding further if you can spare some time to let us know about the below queries:

When we can expect the BETA version of the framework?
Do you have any roadmap for the progress of the framework? (GitHub project will be great)
When documentation will get updated?
Do you have a playground where we can play with components? (Like Vuetify have on their docs)

We are looking forward to your reply.
Regards.

Hello there!
First of all, I wanna thank you, this kind of things is what encourages us to keep up working on this library.

About the BETA version, although there is a lot of work to do, with every commit and PR we are getting close to have a BETA version. I think I could create a roadmap to clarify the path of this project.
Documentation is getting updated a bit slowly, I have to confess, but today I publish a new release (v0.0.4) with a lot of new documentation. When BETA version comes out we hope to have completed the documentation.
Regarding the playground, we could improve the actual documentation files to make it more interactive. What do you think about BootstrapVue playground? https://bootstrap-vue.org/play

Thanks again,
regards

I met this project last friday, asked something similar last saturday (but on twitter), and today I'm already seeing a huge documentation update. Congrats!
Maybe it could be positive if we also gather in some instant messaging channel, like a discord server, gitter etc, so users could be able to clear each other's questions too. I just tried the b-table component and it seems to be working, despite having no documentation yet. Should I assume that most of boostrapvue's components are already working?


ALT_FDTNAME does not work

On kirkstone_5.15.52-2.1.0-stable, specifying a custom fdtfile using ALT_FDTNAME is not working as expected. When using the variable, these two lines are added to uEnv.txt:
alt_fdtname=<content_of_ALT_FDTNAME>
uenvcmd=setenv has_fdt 1; setenv fdtname ${alt_fdtname}; setenv fdtfile ${alt_fdtname}.dtb; run bootcmd_mmc;

fdtfile variable is set correctly, but it is later overwritten when setfdt is run from mmcboot, which is run from bootcmd_mmc.
bootcmd_mmc=run loadimage;run mmcboot;
mmcboot=echo Booting from mmc ...; run m4boot; run searchbootdev; run mmcargs; echo baseboard is ${baseboard}; run setfdt; if test ${boot_fdt} = yes || test ${boot_fdt} = try; then if run loadfdt; then bootz ${loadaddr} - ${fdt_addr}; else if test ${boot_fdt} = try; then echo WARN: Cannot load the DT; echo fall back to load the default DT; setenv baseboard ${default_baseboard}; run setfdt; run loadfdt; bootz ${loadaddr} - ${fdt_addr}; else echo WARN: Cannot load the DT; fi; fi; else bootz; fi;
setfdt=if test -n ${wifi_module} && test ${wifi_module} = qca; then setenv fdtfile ${som}-${form}-${baseboard}-${wifi_module}${mcu}.dtb; else setenv fdtfile ${som}-${form}-${baseboard}

has_fdt is set in uenvcmd and I assume it is supposed to be a flag that tells the other commands not to overwrite fdtfile. This is, however, not checked and fdtfile is always overwritten.

Hi @Ondrej1
Thanks for your feedback.
We have fixed this issue to branch of kirkstone_5.15.52-2.1.0-next. Please use the next branch for now, as the stable branch is not scheduled to be updated until 2023Q2.
BR,
Ray


allow dynamic full expansion behaviour

This PRs adds, in addition to static configuration, ability to change the full expansion behaviour dynamically via DrawerPresentable protocol, similarly to partially expanded behaviour

@inamiy the flag in configuration to allow partial state should be false in this case as the height of the drawer will be the same for both states and it's not currently handled properly, but I think it's a separate issue 🤷♂


The geopandas version check is failing

Another difficult to reproduce issue ..
On the readthedocs build of our docs, it is now failing the geoplot example in the following way (see full build log https://readthedocs.org/projects/geopandas/builds/8718626/):
generating gallery for gallery... [ 50%] plotting_with_geoplot.py
WARNING: /home/docs/checkouts/readthedocs.org/user_builds/geopandas/checkouts/latest/examples/plotting_with_geoplot.py failed to execute correctly: Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/geopandas/checkouts/latest/examples/plotting_with_geoplot.py", line 34, in <module>
    import geoplot
  File "/home/docs/checkouts/readthedocs.org/user_builds/geopandas/conda/latest/lib/python3.7/site-packages/geoplot/__init__.py", line 1, in <module>
    from .geoplot import (pointplot, polyplot, choropleth, aggplot, cartogram, kdeplot, sankey, voronoi,
  File "/home/docs/checkouts/readthedocs.org/user_builds/geopandas/conda/latest/lib/python3.7/site-packages/geoplot/geoplot.py", line 18, in <module>
    if LooseVersion(gpd.__version__) >= LooseVersion('0.4.1'):
  File "/home/docs/checkouts/readthedocs.org/user_builds/geopandas/conda/latest/lib/python3.7/distutils/version.py", line 70, in __ge__
    c = self._cmp(other)
  File "/home/docs/checkouts/readthedocs.org/user_builds/geopandas/conda/latest/lib/python3.7/distutils/version.py", line 337, in _cmp
    if self.version < other.version:
TypeError: '<' not supported between instances of 'str' and 'int'

But when re-creating the same environment locally (it is in https://github.com/geopandas/geopandas/blob/master/doc/environment.yml), I can't reproduce this.

I will take a look at this on the weekend. Most likely I'm going to copy _pysal_choro (or whatever the name of that function is now) out of geopandas and into geoplot, because dealing with this oblique dependency by inspecting package versions is creating too many issues like this one.

If you don't want to copy the function, you should also be able to do
try:
    from geopandas.plotting import _mapclassify_choro
except ImportError:
    from geopandas.plottting import __pysal_choro as _mapclassify_choro

that should be fully robust without any need for explicit version checking.

This is true, but:

My understanding is that this style of dependency management is discouraged best practices wise. It's better to be explicit.
 The fact that I did was the reason we got into this whole mess in the first place. Relying on _mapclassify_choro is still kind of sketchy.

If you think it is useful, we can make it public (but, that won't make importing easier in the short term, because then you need to check for 3 different spellings ..)

My understanding is that this style of dependency management is discouraged best practices wise. It's better to be explicit.

Personally, I think in this case that is perfectly fine (you see a lot use of this kind of try except in python)

Done in the 0.2.4 release. Ended up just going along with a generic try-catch block.


Webpack starting the server takes forever

In the example on your website here webpack takes 34 seconds to build!  That means firing up the server takes the better part of a minute.  Not great.
In my tests, changes to css take about 10 seconds before they are reflected on the page.  Again, not great.

@121onto initial build is always slow (it doesn't have any relation to hot reload btw). Hot reload will only rebuild parts that actually changed (so it should be fast unless you're pasting a huge amount of files into the project at once). For more information on Webpack build process please refer http://webpack.github.io/docs/.
About CSS recompilation speed - we didn't encounter it at work, will keep an eye for this.

Thanks @damianlegawiec.
How hard would it be persist a fingerprint of the codebase with each reload.  You could compare the fingerprint when starting the server and wipe the webpack directory if and only if the fingerprint changes.

I improved performance about 4x by following the suggestions here:
https://github.com/webpack/watchpack/pull/23
This was necessary because I enabled polling in my express servers:
var server = new WebpackDevServer(webpack(config), {
  ...
  watchOptions: {
      poll: true
  },
  ...
})

I enabled polling so that webpack would recognize changes to source files while running on a vagrant machine.

@121onto - oh so you were using it inside Vagrant - that explains. Yeah, webpack has some issues running in containers. Could you submit a PR with this change?

I can submit if I have some spare time.  Given that webpack consumes extra resources while using these settings, you'd probably want to make it an option (does foreman allow for that?).
In the meantime, here are the steps I took to get this working on Vagrant:

Change all instances of "localhost" to "<IP_ADDRESS>" in webpack server configs.
Update rails servers to bind to <IP_ADDRESS> (using the option -b <IP_ADDRESS>).
Turn firewalls off on vagrant machine, and configure to forward ports 3000, 3500, and 4000.
Patch webpack per the watchpack issue referenced above.

I accomplished step 4 by saving this script to client/watchpack-ignore-node-modules.sh, adding it as a postinstall script in client/package.json, deleting client/node_modules/webpack, and then running npm install.
Add the postinstall script by adding the following line to client/package.json:
{
  ...
  "scripts": {
    ...
    "postinstall": "./watchpack-ignore-node-modules.sh",
    ...
  },
  ...
}


what is the most appropriate way to add customized position encoding

I wonder what is the best way to add in the customized positional encoding? stored in one other binary file and read it somewhere?

Sorry it’s not clear what you mean. We support loading pretrained word embeddings, but if you want to use pretrained positional embeddings you’ll need to modify the code to support that. Probably here for the Encoder and a bit lower in the same file for the decoder: https://github.com/pytorch/fairseq/blob/f732b403ec15244c41a24b9e28d6c5a411a511df/fairseq/models/transformer.py#L328-L337

Sorry it’s not clear what you mean. We support loading pretrained word embeddings, but if you want to use pretrained positional embeddings you’ll need to modify the code to support that. Probably here for the Encoder and a bit lower in the same file for the decoder:
https://github.com/pytorch/fairseq/blob/f732b403ec15244c41a24b9e28d6c5a411a511df/fairseq/models/transformer.py#L328-L337

I want to train a sequence model using different position encoding (domain specific). So its not learned position embedding nor Sinusoidal embedding. Is there a way to either pack with preporcess binary or it need to be somehow read from other files?


Data upload

either to local disk at service container or straight through to Cloud Storage bucket
user needs to be able to keep reference in own collection / favourites (e.g. own data OWC collection)
upload by link provided, too
upload needs to be coupled with the metadata editor
Metadata record needs to be inserted into CSW

I'll implement a upload for small files.

Was blocked by cluster config. Cluster now Cloud Storage enabled. Please continue now.

owcDocumentDAO.updateOwcDocument(newDoc, email)


AWS IAM support mainly for accessing S3 without providing AWS_SECRETS

Ref. https://github.com/epinio/epinio/issues/1925 should contain all the info.

@thehejik From the tag on this issue I guess that the request is to write some documentation.
Reading through the description and comments on #1925 I have trouble getting a coherent picture of what exactly should be documented ?!
How to set up an IAM ?
How to tell epinio (through its chart values) to use IAM ?
The latter seems to be documented in the #1988 PR fixing the 1925, as
"If no credentials are specified then the IAM Role authentication will be used."
@enrichman Your thoughts ?
Right now I am going with "Tell epinio to use AWS IAM".

Thanks for confirming. Mostly done with exactly this now.


❄ Freeze Drawing

fixes #395
Converts a drawing to an Image. The performance improvement can see at https://www.dropbox.com/s/nvehwch3eenqa2b/demo.mov?dl=0
This is useful when having a complex canvas and you want only the animated parts to be redrawn on each frame.
In parallel of this, we are working on a performance improvement that will remove the need to use such a component for a picture that has a lot of simple drawing commands. For pictures that have few drawing commands that are complex (inner shadows on paths and other image filters), this component will still be very handy.

Awesome work. I’m trying to do some thinking about this one and will test it during the weekend!


Replace cloud navigation item with split Application and Account navigation

Proposal:
Replace cloud navigation item with split Application and Account navigation:

Add a top-level navigation bar that houses the account navigation
Add an ever-present Upgrade button to the top-level navigation bar
Add a logo to the top-level navigation that acts as a home button
Remove the logo and cloud from the application navigation bar.

Current behavior:
Users are currently unable to find the option to upgrade to a usage-based payment plan or their account settings in the application.
Desired behavior:
Users are able to easily find the upgrade option. Users are easily able to find their account settings.
Alternatives considered:

Adding an upgrade button below the existing navigation bar
Adding the current subscription level next to the org name subtitle with a popover upgrade option
Adding an ever-present upgrade button next to the org name

Use case:
A free teir cloud user is prompted to upgrade and is easily able to find the option.

cc @alexpaxton  when we tried this, we got a few weird artifacts in the UI

@drdelambre what was the issue that you ran into specifically?


adds support for scan based merchant returns service

https://ribbs.usps.gov/merchantreturns/documents/tech_guides/MerchantReturnsDomesticAPI.pdf

A few questions and comments. Mostly good work. If you can address them and get the build green and we'll get this merged soon for you.
Thank you for your contribution.

@kmcphillips and @trishume thank you so much for your feedback. I really appreciate it.

Still waiting on test credentials that I can pass on from USPS in order to get the build green.  Otherwise, I believe this pull includes all the changes you suggested.  @kmcphillips what is the typical process here for running remote tests that require credentials?

This is currently a pain point in active_shipping. Our CI only includes a small set of credentials which we have obtained and know are safe to release publicly. We're working on expanding that set but for now most of the tests are set to skip on authentication failure and I just run the remote tests locally before I do a release with Shopify's test credentials set.
I'm fine with merging this without public credentials provided you could point me in the right direction for me to get some for Shopify's internal use. If you gave us a set of credentials along with this PR that would be amazing and you would join the list of my favorite contributors ever :heart:

Just realized I hadn't actually reviewed this yet. Found some coding style things that could do with some refactoring.
This PR is just outstanding though, the level of documentation, validation, testing and everything else blows me away. And the fact that you without even being prompted say you're looking into getting us credentials. And the fact that you say it's been tested in production for a long time. Just wow.
In fact since this PR is so great I'm willing to do the refactors I suggested myself if you'd like. I have pro editor macro skills so it shouldn't take long.

@trishume thank you for your very kind words !!!
I think you're right that I can and should refactor some of the validations.  I need a couple of min to take care of those changes.
In the meantime, I just pushed up fixes for other suggestions you made.
Also, I realized that remote tests were rescue'ing with NoCredentialsFound, which is something I failed to do. And lastly, I'm still waiting on a response from USPS. I received an e-mail earlier that they were looking into getting me some creds.

@trishume , I applied your recommendations for dry'ing up validations. I had the validations themselves executing sanitize. Not sure how you feel about that. Anyway, let me know if there is something I missed. I've added a few more tests, and they all seem to be passing.

Amazing :+1:
I'll merge this when I do the next release. If USPS gets back to you with credentials and you can make them public you can submit another PR later (if I've already merged this one) or if they can't be public but you'd like to donate them to Shopify you can email<EMAIL_ADDRESS>

Thank you @trishume, I sent another e-mail to USPS to request temporary credentials for the sake of testing that you can do on your end.  I'll continue to follow up with them and will let you know accordingly.

Sure that would be great.

@trishume

Squashed
I have USPS credentials. For this feature, there is no test server, it's all production. I was able to get a test account set up with a very stern warning not to make it public. Additionally, the account will be turned off by end of the month.  Can I e-mail<EMAIL_ADDRESS>and maintain correspondence with a single person to pass along the details of this account and ensure that this feature works against remote?

Great<EMAIL_ADDRESS>goes to the entire shipping team. If you want to have a specific contact I'm<EMAIL_ADDRESS>but I'm an intern and will only be here for 2 more months. If you want a more permanent contact<EMAIL_ADDRESS>is the shipping team lead.


whisper-ctranslate2: 0.3.2 -> 0.3.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for whisper-ctranslate2 is: Whisper command line client compatible with original OpenAI client based on CTranslate2
meta.homepage for whisper-ctranslate2 is: https://github.com/Softcatala/whisper-ctranslate2
meta.changelog for whisper-ctranslate2 is: https://github.com/Softcatala/whisper-ctranslate2/releases/tag/0.3.4
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.3.4 with grep in /nix/store/znmj9mv4agigh9sf9j37jpj5xklskhf6-whisper-ctranslate2-0.3.4
found 0.3.4 in filename of file in /nix/store/znmj9mv4agigh9sf9j37jpj5xklskhf6-whisper-ctranslate2-0.3.4

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
whisper-ctranslate2

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/znmj9mv4agigh9sf9j37jpj5xklskhf6-whisper-ctranslate2-0.3.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A whisper-ctranslate2 https://github.com/r-ryantm/nixpkgs/archive/d2ae3a1a1f5b82a48ffd0f0106270c21a016d564.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d2ae3a1a1f5b82a48ffd0f0106270c21a016d564#whisper-ctranslate2

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/znmj9mv4agigh9sf9j37jpj5xklskhf6-whisper-ctranslate2-0.3.4
ls -la /nix/store/znmj9mv4agigh9sf9j37jpj5xklskhf6-whisper-ctranslate2-0.3.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

whisper-ctranslate2

Maintainer pings
cc @happysalada for testing.

Result of nixpkgs-review pr 273247 run on x86_64-linux 1

2 packages built:

whisper-ctranslate2
whisper-ctranslate2.dist


test: verify getCreditReturnedForDollars and getCreditNftReturnedForDollars

Increases CreditNftManagerFacet test coverage, part of #838 .
Resolves: #838

Development branch coverage: 69.7
PR branch coverage: 70.1

time for a github action bots to post coverall increase comment? it would be nice @gitcoindev

time for a github action bots to post coverall increase comment? on PR it would be nice @gitcoindev

I also thought about it. Currently the coverage difference it is available in the GitHub action logs, and does not pollute comments. I am not sure to be honest if we really need it to add in each pull request comment thread. For pull requests that try to increase coverage it is a good indicator, but for bug fixes and new features it may be enough if the GitHub action does not fail with decreased coverage. Perhaps @pavlovcik could you please also share your thoughts?

I don't see the benefit of that if we can easily see the results on CI


Change build.gradle in ReadMe to use new gradle plugin

The new gradle plugin will simplify the readme's build.gradle example for this library

fixed in 1.4.2-1.4.3


Fixed PermissionError when saving gif output on Windows

NamedTemporaryFile causes issues when auto-deleting on windows, this fix is what is used by bokeh to ensure exporting works on windows and some basic testing by myself and by @smillies seems to confirm it works. That said in the long term we should set up CI tests on Windows.

[x] Fixes https://github.com/ioam/holoviews/issues/3151

Seems like an issue with NamedTemporaryFile to me which everyone has to work around. I'll merge when the tests are green (even if Travis isn't testing Windows).


[AutoRelease] t2-dataprotection-2023-07-04-26651(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/4315
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=3206

issue link:https://github.com/Azure/sdk-release-request/issues/4315


chore: release v1

📝 Changed routes:

/
/page.module

(view full size)
(view full size)

Commit be57c5af657ea7f1654cea983fcf31cea5901d6e (https://sentiment-dvixocndj-yl33ly.vercel.app).


Code coverage collection is broken

Code coverage collection is broken
When running Twister using coverage reporting, it does not anymore produce LCOV output.
To Reproduce
Steps to reproduce the behavior:

cd zephyr
twister -p native_posix -T tests/net/lib/lwm2m -c -C
See error

Expected behavior
I would expect the Twister command above to produce a HTML file.
Instead I see error messages lcov: ERROR: no valid records found in tracefile
Impact
Cannot follow code coverage from Zephyr.
Logs and console output
❯ twister -p native_posix -T tests/net/lib/lwm2m -C
Renaming output directory to /home/seta/src/ncs/zephyr/twister-out.1
INFO    - Using Ninja..
INFO    - Zephyr version: v3.3.0-rc1-11-g8512bd18c666
INFO    - Using 'zephyr' toolchain.
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /home/seta/src/ncs/zephyr/twister-out/testplan.json
INFO    - JOBS: 8
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - Total complete:    8/   8  100%  skipped:    4, failed:    0
INFO    - 8 test scenarios (8 test instances) selected, 4 configurations skipped (0 by static filter, 4 at runtime).
INFO    - 4 of 8 test configurations passed (100.00%), 0 failed, 4 skipped with 0 warnings in 13.57 seconds
INFO    - In total 88 test cases were executed, 99 skipped on 1 out of total 540 platforms (0.19%)
INFO    - 4 test configurations executed on platforms, 0 test configurations were only built.
INFO    - Generating coverage files...
/home/seta/src/ncs/zephyr/twister-out/native_posix/tests/net/lib/lwm2m/engine/net.lwm2m.engine/zephyr/subsys/random/CMakeFiles/subsys__random.dir/rand32_entropy_device.c.gcno:version 'A94*', prefer 'B21*'
geninfo: ERROR: GCOV failed for /home/seta/src/ncs/zephyr/twister-out/native_posix/tests/net/lib/lwm2m/engine/net.lwm2m.engine/zephyr/subsys/random/CMakeFiles/subsys__random.dir/rand32_entropy_device.c.gcda!
lcov: ERROR: no valid records found in tracefile /home/seta/src/ncs/zephyr/twister-out/coverage.info
lcov: ERROR: no valid records found in tracefile /home/seta/src/ncs/zephyr/twister-out/coverage.info
lcov: ERROR: no valid records found in tracefile /home/seta/src/ncs/zephyr/twister-out/coverage.info
lcov: ERROR: no valid records found in tracefile /home/seta/src/ncs/zephyr/twister-out/coverage.info
genhtml: ERROR: no valid records found in tracefile /home/seta/src/ncs/zephyr/twister-out/coverage.info
INFO    - Saving reports...
INFO    - Writing JSON report /home/seta/src/ncs/zephyr/twister-out/twister.json
INFO    - Writing xunit report /home/seta/src/ncs/zephyr/twister-out/twister.xml...
INFO    - Writing xunit report /home/seta/src/ncs/zephyr/twister-out/twister_report.xml...
INFO    - Run completed

When running on Zephyr tag v3.0.0, I get expected output.
❯ twister -p native_posix -T tests/net/lib/lwm2m -c -C
Deleting output directory /home/seta/src/ncs/zephyr/twister-out
INFO    - Zephyr version: zephyr-v3.0.0
INFO    - JOBS: 8
INFO    - Using 'zephyr' toolchain.
INFO    - Building initial testcase list...
INFO    - 4 test scenarios (4 configurations) selected, 0 configurations discarded due to filters.
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - Total complete:    4/   4  100%  skipped:    0, failed:    0
INFO    - 4 of 4 test configurations passed (100.00%), 0 failed, 0 skipped with 0 warnings in 29.44 seconds
INFO    - In total 99 test cases were executed, 0 skipped on 1 out of total 427 platforms (0.23%)
INFO    - 4 test configurations executed on platforms, 0 test configurations were only built.
INFO    - Generating coverage files...
INFO    - HTML report generated: /home/seta/src/ncs/zephyr/twister-out/coverage/index.html
INFO    - Saving reports...
INFO    - Writing xunit report /home/seta/src/ncs/zephyr/twister-out/twister.xml...
INFO    - Writing xunit report /home/seta/src/ncs/zephyr/twister-out/twister_report.xml...
INFO    - Run completed

Environment (please complete the following information):

OS: Linux
Toolchain Zephyr SDK 0.15.1
Commit SHA or Version used 8512bd18c6

Additional context
According to Git Bisect, this commit broke the build:
47b675c6ee12084fb1c83afa946f5c2c0cb0adf5 is the first bad commit
commit 47b675c6ee12084fb1c83afa946f5c2c0cb0adf5
Author: Yuval Peress<EMAIL_ADDRESS>Date:   Tue Nov 29 11:18:06 2022 -0700

    twister: Set a default gcov-tool for llvm

    When using llvm we need to set the gcov-tool to "llvm-cov gcov" but
    the lcov tool is incapable of passing arguments to the gcov-tool. i.e.
    the following cannot work:

    $ lcov --gcov-tool "llvm-cov gcov" ...

    Instead, create a symlink to llvm-cov prefixed as `gcov` which by the
    documentation of llvm-cov will alias to `llvm-cov gcov` subcommand.

    Signed-off-by: Yuval Peress<EMAIL_ADDRESS>

Breaking PR: https://github.com/zephyrproject-rtos/zephyr/pull/52644

Reverting the breaking commit seem to fix. Is this a LCOV version issue, or something else? Why the default is not working for me?
❯ twister -p unit_testing -c -C -i -T tests/unit/intmath/
Deleting output directory /home/seta/src/ncs/zephyr/twister-out
INFO    - Using Ninja..
INFO    - Zephyr version: v3.3.0-rc1-12-g83ffead48cb9
INFO    - Using 'zephyr' toolchain.
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /home/seta/src/ncs/zephyr/twister-out/testplan.json
INFO    - JOBS: 8
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - Total complete:    1/   1  100%  skipped:    0, failed:    0
INFO    - 1 test scenarios (1 test instances) selected, 0 configurations skipped (0 by static filter, 0 at runtime).
INFO    - 1 of 1 test configurations passed (100.00%), 0 failed, 0 skipped with 0 warnings in 2.32 seconds
INFO    - In total 1 test cases were executed, 0 skipped on 1 out of total 540 platforms (0.19%)
INFO    - 1 test configurations executed on platforms, 0 test configurations were only built.
INFO    - Generating coverage files...
INFO    - HTML report generated: /home/seta/src/ncs/zephyr/twister-out/coverage/index.html
INFO    - Saving reports...
INFO    - Writing JSON report /home/seta/src/ncs/zephyr/twister-out/twister.json
INFO    - Writing xunit report /home/seta/src/ncs/zephyr/twister-out/twister.xml...
INFO    - Writing xunit report /home/seta/src/ncs/zephyr/twister-out/twister_report.xml...
INFO    - Run completed
❯ git reset --hard upstream/main
HEAD is now at 8512bd18c6 Bluetooth: Shell: Fix unintialized ad_len bug
❯ twister -p unit_testing -c -C -i -T tests/unit/intmath/
Deleting output directory /home/seta/src/ncs/zephyr/twister-out
INFO    - Using Ninja..
INFO    - Zephyr version: v3.3.0-rc1-11-g8512bd18c666
INFO    - Using 'zephyr' toolchain.
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /home/seta/src/ncs/zephyr/twister-out/testplan.json
INFO    - JOBS: 8
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - Total complete:    1/   1  100%  skipped:    0, failed:    0
INFO    - 1 test scenarios (1 test instances) selected, 0 configurations skipped (0 by static filter, 0 at runtime).
INFO    - 1 of 1 test configurations passed (100.00%), 0 failed, 0 skipped with 0 warnings in 2.23 seconds
INFO    - In total 1 test cases were executed, 0 skipped on 1 out of total 540 platforms (0.19%)
INFO    - 1 test configurations executed on platforms, 0 test configurations were only built.
INFO    - Generating coverage files...
/home/seta/src/ncs/zephyr/twister-out/unit_testing/tests/unit/intmath/utilities.intmath/CMakeFiles/testbinary.dir/home/seta/src/ncs/zephyr/subsys/testsuite/ztest/src/ztest_new.c.gcno:version 'A94*', prefer 'B21*'
geninfo: ERROR: GCOV failed for /home/seta/src/ncs/zephyr/twister-out/unit_testing/tests/unit/intmath/utilities.intmath/CMakeFiles/testbinary.dir/home/seta/src/ncs/zephyr/subsys/testsuite/ztest/src/ztest_new.c.gcda!
lcov: ERROR: no valid records found in tracefile /home/seta/src/ncs/zephyr/twister-out/coverage.info
lcov: ERROR: no valid records found in tracefile /home/seta/src/ncs/zephyr/twister-out/coverage.info
lcov: ERROR: no valid records found in tracefile /home/seta/src/ncs/zephyr/twister-out/coverage.info
lcov: ERROR: no valid records found in tracefile /home/seta/src/ncs/zephyr/twister-out/coverage.info
genhtml: ERROR: no valid records found in tracefile /home/seta/src/ncs/zephyr/twister-out/coverage.info
INFO    - Saving reports...
INFO    - Writing JSON report /home/seta/src/ncs/zephyr/twister-out/twister.json
INFO    - Writing xunit report /home/seta/src/ncs/zephyr/twister-out/twister.xml...
INFO    - Writing xunit report /home/seta/src/ncs/zephyr/twister-out/twister_report.xml...
INFO    - Run completed
❯ git revert 47b675c6ee12084fb1c83afa946f5c2c0cb0adf5
Auto-merging scripts/pylib/twister/twisterlib/environment.py
[revert edcf00e301] Revert "twister: Set a default gcov-tool for llvm"
 3 files changed, 15 insertions(+), 41 deletions(-)
❯ twister -p unit_testing -c -C -i -T tests/unit/intmath/
Deleting output directory /home/seta/src/ncs/zephyr/twister-out
INFO    - Using Ninja..
INFO    - Zephyr version: v3.3.0-rc1-12-gedcf00e30190
INFO    - Using 'zephyr' toolchain.
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /home/seta/src/ncs/zephyr/twister-out/testplan.json
INFO    - JOBS: 8
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - Total complete:    1/   1  100%  skipped:    0, failed:    0
INFO    - 1 test scenarios (1 test instances) selected, 0 configurations skipped (0 by static filter, 0 at runtime).
INFO    - 1 of 1 test configurations passed (100.00%), 0 failed, 0 skipped with 0 warnings in 2.20 seconds
INFO    - In total 1 test cases were executed, 0 skipped on 1 out of total 540 platforms (0.19%)
INFO    - 1 test configurations executed on platforms, 0 test configurations were only built.
INFO    - Generating coverage files...
INFO    - HTML report generated: /home/seta/src/ncs/zephyr/twister-out/coverage/index.html
INFO    - Saving reports...
INFO    - Writing JSON report /home/seta/src/ncs/zephyr/twister-out/twister.json
INFO    - Writing xunit report /home/seta/src/ncs/zephyr/twister-out/twister.xml...
INFO    - Writing xunit report /home/seta/src/ncs/zephyr/twister-out/twister_report.xml...
INFO    - Run completed

Specifying the proper tool seem to work
❯ twister -p unit_testing -c -C -i -T tests/unit/intmath/ --gcov-tool gcov
Deleting output directory /home/seta/src/ncs/zephyr/twister-out
INFO    - Using Ninja..
INFO    - Zephyr version: v3.3.0-rc1-11-g8512bd18c666
INFO    - Using 'zephyr' toolchain.
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /home/seta/src/ncs/zephyr/twister-out/testplan.json
INFO    - JOBS: 8
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - Total complete:    1/   1  100%  skipped:    0, failed:    0
INFO    - 1 test scenarios (1 test instances) selected, 0 configurations skipped (0 by static filter, 0 at runtime).
INFO    - 1 of 1 test configurations passed (100.00%), 0 failed, 0 skipped with 0 warnings in 2.11 seconds
INFO    - In total 1 test cases were executed, 0 skipped on 1 out of total 540 platforms (0.19%)
INFO    - 1 test configurations executed on platforms, 0 test configurations were only built.
INFO    - Generating coverage files...
INFO    - HTML report generated: /home/seta/src/ncs/zephyr/twister-out/coverage/index.html
INFO    - Saving reports...
INFO    - Writing JSON report /home/seta/src/ncs/zephyr/twister-out/twister.json
INFO    - Writing xunit report /home/seta/src/ncs/zephyr/twister-out/twister.xml...
INFO    - Writing xunit report /home/seta/src/ncs/zephyr/twister-out/twister_report.xml...
INFO    - Run completed

I am looking into this.


attach multiple file transport to a Category

How can i attach multiple file transport for a category
const winstonConsoleTransportArgs = {
    level: logLevel,
    colorize: true,
    handleExceptions: true,
    humanReadableUnhandledException: true
};

const winstonCommonFileTransportArgs1 = {
    level: logLevel,
    filename: LOG_LOCATION + 'main.log',
    handleExceptions: true,
    humanReadableUnhandledException: true
};

const winstonCommonFileTransportArgs2 = {
    level: logLevel,
    filename: LOG_LOCATION + 'main.log',
    handleExceptions: true,
    humanReadableUnhandledException: true
};

winston.loggers.add('main', {
    console: winstonConsoleTransportArgs,
    file: winstonCommonFileTransportArgs1  //how can i attach winstonCommonFileTransportArgs2 here
});

I also tried
transports: [
        new winston.transports.File({ filename: 'error.log', level: 'error' }),
        new winston.transports.File({ filename: 'combined.log' }) //Error: Transport already attached: file, assign a different name
    ]
});

but didn't work

too bad haven't seen its not supported


⚠ ClamAV has degraded performance

In a094301, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 84 ms

Resolved: ClamAV performance has improved in ce87bf2.


Project Timeline - filters hiding

Describe the bug
filters are hidden  behind the gannt chart
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
filters should be on top so I can interact with it
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Please check if this still exists. I can see the filter properly now.

working properly now


[Feature] Ability to interact with AppBarButton elements

Describe the feature and how it will be used

Introduce the ability to interact with the Uno Platform AppBarButton element.
There should simply be the capability to click the button.
Platforms

[x] ~iOS~ [Done]
[x] ~Android~ [Done]
[x] ~WebAssembly~ [Done]

Completed the work on this for the web element

Completed work on this for Android

Completed ✅


Don't allow values of u'\xa0'

Due to shenanigans with the Medium-editor library, it was possible to enter a single space for the description in a profile or listing. This causes that profile or listing to be unreadable from the server.
This will be prevented in the client, but should also be prevented on the server, either by returning an error when values containing a single space are submitted, or by stripping them (replacing them with blank values).

Can you try doing this on master and see if it's still an issue? I believe I've fixed it.

Will do.

It appears to be a problem with non-breaking spaces also.
The medium editor puts in a non-breaking space if the field is edited then erased, sending   to POST profile makes the profile unreadable.

That should be "paragraph tag non-breaking space paragraph tag"
<p> </p>

This is on master you're still seeing this?

Yes.


csharp singleton update

Proposed changes
Add warning about singleton management

@pwseg  are we good to merge this? Thanks!

@pwseg or @forstisabella  are we good to merge this? Thanks!

@pwseg or @forstisabella are we good to merge this? Thanks!

Once a few small changes are made we'll be g2g 🚀


tcp: add an (unused-by-default) connection pool based on a shared-with-HTTP class.

Risk Level: low (defaulted off until release)
Testing: parameterized existing unit, integration tests.
Docs Changes: n/a
Release Notes: n/a (will add with flip
Runtime guard: envoy.reloadable_features.new_tcp_connection_pool  (default off)
Fixes https://github.com/envoyproxy/envoy/issues/11528 (modulo TODO and cleanup)

looks like something didn't merge cleanly with the callback change - passes locally but flakes on other builds :-/

ok, one last pass now without the void* pointers :-)

@ggreenway can I get an LGTM on the merge when you get a chance?


Some error.

I mean I'm most likely doing this wrong, but I get this when I try running discord-irc --config /home/jamesguessis/configs/dIRCord.json
/usr/local/lib/node_modules/discord-irc/dist/cli.js:48
      throw new _errors.ConfigurationError('The configuration file contains invalid JSON');
      ^

ConfigurationError: The configuration file contains invalid JSON
    at readJSONConfig (/usr/local/lib/node_modules/discord-irc/dist/cli.js:48:13)
    at Object.run [as default] (/usr/local/lib/node_modules/discord-irc/dist/cli.js:62:98)
    at Object.<anonymous> (/usr/local/lib/node_modules/discord-irc/dist/index.js:23:27)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.runMain (module.js:605:10)
    at run (bootstrap_node.js:420:7)```

Hi! This is because discord-irc requires at least Node.js version 6 and upwards, since that's the minimum supported version of the library we use for Discord related work (discord.js).

Wop, apparently you edited the original post (which had another error).
The new error you added is because your JSON config is badly formatted. Maybe you have a trailing comma or similar? If you remove comments you can use something like http://jsonlint.com/ to validate it (or paste it here without secrets and I can take a look).

Yeah I figured that it needed to be updated seconds after I posted the issue
And yeah apparently I had a backslash at the end of the json for some reason, probably a typo. Thanks keyboard ;P
I'm using this
    "nickname": "DiscordBot",
    "server": "<IP_ADDRESS>",
    "discordToken": "hi": {
      "#general": "#general",
      "#music": "#music",
      "#osu": "#osu",
      "#mania": "#mania",
      "#memes": "#memes",
      "#bot": "#bot",
      "#chosen": "#chosen",
      "#twitch": "#twitch",
      "#twitch-notifications": "#twitch-notifications",
      "#spam": "spam"
     }
   }

should that work?

No, you need to put the channels bellow a channelMapping property - look at the example in the README.

Thank you, now it's working 👍

Trying to get the bridging to work on Raspbian but so far no luck. This is what I get: usr/lib/node_modules/discord-irc/node_modules/check-env/index.js:12
throw new Error('Missing environment variable ' + missing[0]);
^
Error: Missing environment variable CONFIG_FILE
at module.exports (/usr/lib/node_modules/discord-irc/node_modules/check-env/index.js:12:13)
at Object.run [as default] (/usr/lib/node_modules/discord-irc/dist/cli.js:59:59)
at Object. (/usr/lib/node_modules/discord-irc/dist/index.js:12:27)
at Module._compile (internal/modules/cjs/loader.js:738:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:749:10)
at Module.load (internal/modules/cjs/loader.js:630:32)
at tryModuleLoad (internal/modules/cjs/loader.js:570:12)
at Function.Module._load (internal/modules/cjs/loader.js:562:3)
at Function.Module.runMain (internal/modules/cjs/loader.js:801:12)
at internal/main/run_main_module.js:21:11
I've no clue what it means, is it something silly i overlooked or more complex?

Also got these when running: sudo npm install -g discord-irc
npm WARN deprecated<EMAIL_ADDRESS>use node-fetch instead
/usr/bin/discord-irc -> /usr/lib/node_modules/discord-irc/dist/index.js
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of erlpack@discordapp/erlpack but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
@ekmartin could you help me out on this one?


footer2: mobile responsive

Hello @hyu,
Please review the following commits I made in branch nicks/ch7350/footer:
6c6c1a3c5acc6143ea3a0033534188084647363e (2020-06-03 15:28:49 -0400)
footer2: mobile responsive
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

it's not perfect, but i have to run to meetings, and it at least doesn't look totally broken

going to merge to fix and follow up with more changes

Great thanks @nicks


How can I use Shields.io in local?

:question: Question

I'm interessed to use shields in local for ping testing, I wanna use in local because the host he don't has internet out.
How can I do?
Thanks.

hi @massinlaaouaj! it's pretty easy to run your own Shields server, and we've got a lot of the approaches for doing so documented in https://github.com/badges/shields/blob/master/doc/self-hosting.md.


Update matching the 1.19.2 chunk-system

https://github.com/PaperMC/docs/blob/8ff2e03bf8405bcd3656b26fcb19f398898c83ca/docs/paper/admin/reference/global-configuration.md?plain=1#L15-L24
This configuration section was removed somewhere in the middle of 1.19-1.19.2, I believe here https://github.com/PaperMC/Paper/commit/01a13871deefa50e186a10b63f71c5e0459e7d30
And the new one was introduced:
chunk-system:
  gen-parallelism: default # what are other options?
  io-threads: -1 # does it affect load&save chunk operations?
  worker-threads: -1 # what does one worker do?

ever found an answer to that?

ever found an answer to that?

I haven't actually lookes through the code after all this time 😂
But from my experience io-threads is responsible for loading/reading and saving chunks operations. Still don't know other options for the parallelism one and not sure about worker threads. I just set worker threads equal to io-threads and it gives Nx performance boost for chunks (N = number of threads) (chunky was generating ±20 chunks at a time and with that changed to 5 the number went up to ±80).

So, after some looking through the code, basing on these lines
ChunkTaskScheduler.newChunkSystemGenParallelism = useParallelGen ? newChunkSystemWorkerThreads : 1;
ChunkTaskScheduler.newChunkSystemLoadParallelism = newChunkSystemWorkerThreads;

I assume that

worker-threads is responsible for the number of threads that will work on generating new chunks
'io-threads' is responsible for loading chunks (I believe all read/write operations)

And the valid options for gen-parallelism are true, on, enabled and false, off, disabled. This option controls whether to use the parallel generation or not; the default value is true at the time.
String newChunkSystemGenParallelism = config.genParallelism;
if (newChunkSystemGenParallelism.equalsIgnoreCase("default")) {
    newChunkSystemGenParallelism = "true";
}
boolean useParallelGen;
if (newChunkSystemGenParallelism.equalsIgnoreCase("on") || newChunkSystemGenParallelism.equalsIgnoreCase("enabled")
    || newChunkSystemGenParallelism.equalsIgnoreCase("true")) {
    useParallelGen = true;
} else if (newChunkSystemGenParallelism.equalsIgnoreCase("off") || newChunkSystemGenParallelism.equalsIgnoreCase("disabled")
    || newChunkSystemGenParallelism.equalsIgnoreCase("false")) {
    useParallelGen = false;
}


Presentation leaves much to be desired...

[Enter feedback here]
Left nav panel can't be collapsed or resized ... got to the indicate page via 'net search and frankly don't want or need the nav panel.  It's wasting space; if this page doesn't have what I need I'll got back to the window with the search results, I pretty much never, ever use the nav panel.  If it would collapse I could at least get it out of my way, and if I did decided to browse the site for some reason, I may always expand it.
Right "Additional Resources" panel is a flat-out useless waste of screen space.  It literally contains nothing that even pays lip service to being remotely related to the current page, which means it's not resourceful in context.  As in if it suggested updating the firmware in my car's ECU, it would be just as relevant as what I see.  Best remedy: REMOVE THE RIGHT SIDE PANEL FOREVER.  Second best: make that a flyout I may hover over if I WANT to see it, don't shove irrelevent garbage in my face.
I don't surf with a full-screen window, but when I do make it full screen, the useless, space wasting left & right panels grow significantly, insuring that even full-screen users have the least desirable, most screen-wasting, useless scroll-fest experience possible, where the main content area enjoys barely a third of the available width.  The point of this page IS the main content, it IS NOT useless "navigation" and "suggestion" panels, which should remain narrow and unobtrusive at worst, and be hidden or missing entirely at best.
If the intent to make the content, the actual real page information, as illegible as possible, then the presentation is perfect.  Don't touch a thing.  If the intent is to make the content user friendly, there's room for improvement.  Lots of it.

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

ID: 01d1fc7d-eeb2-d7c6-3d79-0a69732864a3
Version Independent ID: 8e53ef60-ff33-38af-8ee9-73a6bf8b8853
Content: How to: Create a C++ Project from Existing Code
Content Source: docs/build/how-to-create-a-cpp-project-from-existing-code.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

The learn creation to web publication process is rather complex.  I am uncertain where to provide this feedback.  I suspect it is not here though.  Here it must be about content, not the publishing system.  I have seen some related issues over on the Microsoft/Docs Contribute pages.
I hope @TylerMSFT can find a place to pass this on.

This site is about content. I don't have any control over the publishing system. I'll try to find who to even forward this on to. Closing.


Weather API intergration

Hi,
Dark sky API is no longer accepting new signups so I was wondering how to change it to a different API (for example openweathermap). I've tried to do it by myself but with my zero programming skills it seems to be impossible.

Switched to AccuWeather in 26458e83dcba5cd1992c5182f6730474c746d3e7.


How do I recreate the wasmtime CLI?

I’d like to execute my WASM module just like I do via the command line: wasmtime hello.wasm. My module can read from stdin and wrote to stdout.
How do I use the wasmtime-go package in a similar way?

This is a pretty broad question, but this library should have all the support necessary to implement your own CLI. I'd recommend reading over the Rust implementation and there should be analogues in Go as well for functions used. If anything is missing though please let me know!


DESENG-543: Add font awesome to MET web

Issue #: https://apps.itsm.gov.bc.ca/jira/browse/DESENG-543
Description of changes:

Added font awesome to MET web.
Created .npmrc file to pass fontawesome auth token from environment variable
Updated github actions to pull fontawesome auth token from github secrets

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Fixing the Fontawesome token is not available in Github actions to build and GitHub checks are failing because of that,

Need to fix github secret


Hotfixing missing array keys

What:

[x] Bug Fix
[ ] New Feature

Description:
I made a small script whilst I was playing with the lib and testing the openAPi's Fine Tuning:
require_once __DIR__ . '/vendor/autoload.php';

$yourApiKey = $_ENV['OPENAI_API_KEY'];
$client = OpenAI::client($yourApiKey);

$response = $client->files()->upload([
    'purpose' => 'fine-tune',
    'file' => fopen(__DIR__.'/client_data.jsonl', 'r'),
]);

echo($response->id);

$response=$client->fineTuning()->createJob([
    'training_file' => $response->id,
    'model' => 'gpt-4o-mini-2024-07-18',
    'hyperparameters' => [
        'n_epochs' => 4,
    ]
]);

var_dump($response->toArray());

But that made the script throw the following error:
Warning: Undefined array key "code" in /usr/src/app/vendor/openai-php/client/src/Responses/FineTuning/RetrieveJobResponseError.php on line 34

Warning: Undefined array key "param" in /usr/src/app/vendor/openai-php/client/src/Responses/FineTuning/RetrieveJobResponseError.php on line 35

Warning: Undefined array key "message" in /usr/src/app/vendor/openai-php/client/src/Responses/FineTuning/RetrieveJobResponseError.php on line 36

Fatal error: Uncaught TypeError: OpenAI\Responses\FineTuning\RetrieveJobResponseError::__construct(): Argument #1 ($code) must be of type string, null given, called in /usr/src/app/vendor/openai-php/client/src/Responses/FineTuning/RetrieveJobResponseError.php on line 33 and defined in /usr/src/app/vendor/openai-php/client/src/Responses/FineTuning/RetrieveJobResponseError.php:20
Stack trace:
#0 /usr/src/app/vendor/openai-php/client/src/Responses/FineTuning/RetrieveJobResponseError.php(33): OpenAI\Responses\FineTuning\RetrieveJobResponseError->__construct(NULL, NULL, NULL)
#1 /usr/src/app/vendor/openai-php/client/src/Responses/FineTuning/RetrieveJobResponse.php(69): OpenAI\Responses\FineTuning\RetrieveJobResponseError::from(Array)
#2 /usr/src/app/vendor/openai-php/client/src/Resources/FineTuning.php(34): OpenAI\Responses\FineTuning\RetrieveJobResponse::from(Array, Object(OpenAI\Responses\Meta\MetaInformation))
#3 /usr/src/app/index.php(19): OpenAI\Resources\FineTuning->createJob(Array)
#4 {main}
  thrown in /usr/src/app/vendor/openai-php/client/src/Responses/FineTuning/RetrieveJobResponseError.php on line 20

Therefore in order to fix the error I made some hotfixes. It seems that happens upon a sucessfull fine-tuning submission.
Extras
The script I made needed to have the following file named client_data.jsonl and it at the same folder that also script resides to:
{"messages": [{"role": "system", "content": "Marv is a factual chatbot that is also sarcastic."}, {"role": "user", "content": "What's the capital of France?"}, {"role": "assistant", "content": "Paris, as if everyone doesn't know that already."}]}
{"messages": [{"role": "system", "content": "Marv is a factual chatbot that is also sarcastic."}, {"role": "user", "content": "Who wrote 'Romeo and Juliet'?"}, {"role": "assistant", "content": "Oh, just some guy named William Shakespeare. Ever heard of him?"}]}
{"messages": [{"role": "system", "content": "Marv is a factual chatbot that is also sarcastic."}, {"role": "user", "content": "How far is the Moon from Earth?"}, {"role": "assistant", "content": "Around 384,400 kilometers. Give or take a few, like that really matters."}]}
{"messages": [{"role": "system", "content": "Marv is a factual chatbot that is also sarcastic."}, {"role": "user", "content": "What's the capital of France?"}, {"role": "assistant", "content": "Paris", "weight": 0}, {"role": "user", "content": "Can you be more sarcastic?"}, {"role": "assistant", "content": "Paris, as if everyone doesn't know that already.", "weight": 1}]}
{"messages": [{"role": "system", "content": "Marv is a factual chatbot that is also sarcastic."}, {"role": "user", "content": "Who wrote 'Romeo and Juliet'?"}, {"role": "assistant", "content": "William Shakespeare", "weight": 0}, {"role": "user", "content": "Can you be more sarcastic?"}, {"role": "assistant", "content": "Oh, just some guy named William Shakespeare. Ever heard of him?", "weight": 1}]}
{"messages": [{"role": "system", "content": "Marv is a factual chatbot that is also sarcastic."}, {"role": "user", "content": "How far is the Moon from Earth?"}, {"role": "assistant", "content": "384,400 kilometers", "weight": 0}, {"role": "user", "content": "Can you be more sarcastic?"}, {"role": "assistant", "content": "Around 384,400 kilometers. Give or take a few, like that really matters.", "weight": 1}]}
{"messages": [{"role": "system", "content": "Marv is a factual chatbot that is also sarcastic."}, {"role": "user", "content": "Που καθόπταν υοθ Μήτσου η μάνα"}, {"role": "assistant", "content": "Του μήτσου η μάνα κάθονταν στο λιβινγ ρουμ στην στανη"}]}
{"messages": [{"role": "system", "content": "Marv is a factual chatbot that is also sarcastic."}, {"role": "user", "content": "Που καθόπταν υοθ Μήτσου η μάνα"}, {"role": "assistant", "content": "Του μήτσου η μάνα κάθονταν στο λιβινγ ρουμ στην στανη"}, {"role": "user", "content": "Που καθόταν"}, {"role": "assistant", "content": "Στην στάνη", "weight": 1}]}
{"messages": [{"role": "system", "content": "Marv is a factual chatbot that is also sarcastic."}, {"role": "user", "content": "Tell me a joke"}, {"role": "assistant", "content": "Knock Knock"}, {"role": "user", "content": "Who's this?"}, {"role": "assistant", "content": "Kame kame haaaaaaaaa", "weight": 1}]}
{"messages": [{"role": "system", "content": "Marv is a factual chatbot that is also sarcastic."}, {"role": "user", "content": "Rell me a joke"}, {"role": "assistant", "content": "Knock Knock"}, {"role": "user", "content": "Who's this?"}, {"role": "assistant", "content": "Kame kame haaaaaaaaa", "weight": 1}]}

The fix was intented as a quickfix without looking any deeper.

Thanks @pc-magas for the PR!


Change the download file name to caption

Hey @aashutoshrathi, I am not understanding what we have to do in this issue? Can you please elaborate? :innocent:


Option to remove "You are using an unsupported feature flag:"

Is your feature request related to a problem? Please describe.
I have command line args enabled on non-root on Android because I want to set up a sync server, but it has a popup every app launch and every new tab created.
Describe the solution you'd like, including relevant patches or source
Either disable the popup or add an option to disable it.
Additional Notes
What the popup looks like:

Full screenshot example:

Thought I was in Throum-Android. Closing this and opening issue in the correct repo: https://github.com/Alex313031/Thorium-Android/issues/47


Threadpool WIN32 changes implementation

Threadpool changes on WIN32 for splitting the threadpool_schedule_work into threadpool_create_work_item, threadpool_schedule_work_item, and threadpool_destroy_work_item.

/azp run Azure-C-Pal

/azp run Azure-C-Pal

/azp run Azure-C-Pal

/azp run Azure-C-Pal

/azp run Azure-C-Pal

https://msazure.visualstudio.com/One/_workitems/edit/29311073
Story board


Spyder 5.0.0 issue

<!--- PLEASE READ: When submitting here, please ensure you've completed the following checklist and checked the boxes to confirm. Issue reports without it may be closed. Thanks! --->
Issue Report Checklist

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

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

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

Problem Description
What steps reproduce the problem?

What is the expected output? What do you see instead?
 Please open a new one with your problem.


[Awake] Add an app-list based keep-alive function

Description of the new feature / enhancement
Please add a function that would only get used when some apps are in-use/in-focus (ideally that would be a per-app setting) to prevent unwanted screen off/sleep during use of said apps.
Scenario when this would be used?
Some apps (including some games) don't use a "keep alive".
This can result in some unwanted scenarios where screen will get turned off/computer will go to sleep (depending on system settings of course) when passively watching them perform their thing instead of keeping it alive.
A list based function/preset could just act on top of whatever preset is currently set (maybe not the always awake one to avoid unnecessary use of resources), and that last used one would still be "active" (but waiting) until app is closed.
That would be a convenient and useful way to prevent said unwanted scenarios.
Hopefully that's something you'd consider adding to Awake.
Supporting information
No response

I'll close this issue as it's an obvious dupe many other requests for the same feature but please devs, do realize it's been asked for years now...
Time to actually decide whether you'll implement such feature or not...


How to Setup remote branch automatically 🤖 (push.autoSetupRemote)

Each time I setup a branch on my localhost and attempt to push it to GitHub,
I get an error message similar to the following:
n@MBP auth % git push

fatal: The current branch gen-erd-#207 has no upstream branch.
To push the current branch and set the remote as upstream, use

    git push --set-upstream origin gen-erd-#207

To have this happen automatically for branches without a tracking
upstream, see 'push.autoSetupRemote' in 'git help config'.

Need to look this up and get it configured so that it automatically creates the remote branch for me. 💭
Todo

[ ] Research
[ ] Document
[ ] Apply!

git config --global --add push.default current
git config --global --add push.autoSetupRemote true

Via: https://pawelgrzybek.com/auto-setup-remote-branch-and-never-again-see-an-error-about-the-missing-upstream/
Thanks @pawelgrzybek ❤


drivers: can: Move bit timing and clock to device tree and fix issue #7933

This commit moves the bit timing (PROP, BS1, BS2 segments and SWJ)
from Kconfig to the device-tree and fixes issue #7933 drivers: can: stm32_can: bitrate ignored

@erwango because PR #7779 was moved to 1.13, I created this PR to fix the important issues

Codecov Report

Merging #8001 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #8001   +/-   ##
=======================================
  Coverage   55.03%   55.03%
=======================================
  Files         481      481
  Lines       53936    53936
  Branches    10474    10474
=======================================
  Hits        29686    29686
  Misses      19981    19981
  Partials     4269     4269

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 8eb652b...3abed84. Read the comment docs.


Gradle documentation on declaring inputs when authoring tasks should address build cache more thoroughly

https://docs.gradle.org/current/userguide/more_about_tasks.html#sec:up_to_date_checks
This is the best resource we have for task authors on declaring inputs/outputs. It is not quite suitable as a resource for authors wanting to make their tasks cacheable. There are a couple of problems:

The title and introduction do not make the connection to the build cache effectively
Some of the samples are not cache aware, and are not explicit
Cache related concerns are smeared through without any real grounding

Let's keep this open.


Test

Description
add author RVaidya
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

Closing as we are past deadline and there are outstanding changes.


Added callbacks for handleMarkers()

Thank you for your PR. Can you explain how it should be used? I mean, why is there foreach. Use $this->onMarkers($this); would be good enought, wouldn't?

Can you explain it? I dont know how it should be used...


RPC call expects data in APIResponse to be a list, which is not mandatory

Bug report
Describe the bug
The response of an RPC query expects data as List[Dict[str, Any]] in the APIResponse object found in postgrest/base_request_builder.py, but it must not be a list.
This call worked in version 0.10.3, but does not in version 0.10.4
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Install this function in the PostgreSQL database (from here) to get the user's raw_app_meta_data from the Supabase user object:

CREATE OR REPLACE FUNCTION get_claims(uid uuid) RETURNS "jsonb"
    LANGUAGE "plpgsql" SECURITY DEFINER SET search_path = public
    AS $$
    DECLARE retval jsonb;
    BEGIN
      IF NOT is_claims_admin() THEN
          RETURN '{"error":"access denied"}'::jsonb;
      ELSE
        select raw_app_meta_data from auth.users into retval where id = uid::uuid;
        return retval;
      END IF;
    END;
$$;

Make the respective RPC call with an authorized Supabase client

sb.rpc(
        fn="get_claims",
        params={"uid": str(user_id)}
    ).execute()

Expected behavior
I expect to receive the response body as is.
System information

OS: macOs
Version of postgrest-py: 0.10.4

Additional context
Thrown error:
Traceback (most recent call last):
  File "/Users/timreibe/VENV/supabase-stripe-webhook/lib/python3.9/site-packages/postgrest/_sync/request_builder.py", line 65, in execute
    return APIResponse.from_http_request_response(r)
  File "/Users/timreibe/VENV/supabase-stripe-webhook/lib/python3.9/site-packages/postgrest/base_request_builder.py", line 157, in from_http_request_response
    return cls(data=data, count=count)
  File "pydantic/main.py", line 342, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for APIResponse
data
  value is not a valid list (type=type_error.list)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Users/timreibe/Documents/Code/py/supabase-stripe-webhook/app.py", line 29, in <module>
    user_id = _get_custom_claims("f25f0ca7-4116-4587-a445-084057d52b50")
  File "/Users/timreibe/Documents/Code/py/supabase-stripe-webhook/src/sb/custom_claims.py", line 64, in _get_custom_claims
    print(res.execute())
  File "/Users/timreibe/VENV/supabase-stripe-webhook/lib/python3.9/site-packages/postgrest/_sync/request_builder.py", line 69, in execute
    raise APIError(r.json()) from e
postgrest.exceptions.APIError: {'scid': 'STRIPE_USER_ID', 'provider': 'email', 'providers': ['email']}

Hey @timreibe,
Thanks for the detailed replication instructions - it was helpful in understanding the issue. I'm planning to make the validator allow for both List[Dict[str, any]] | Dict[str, any] .
Let me know if you have any thoughts or concerns about this fix.
Thanks!
Jo

Having the same issue

Any update on this issue? It's a pretty big blocker in not being able to use this library (or the supabase python client)

So are RPC calls at the moment largely broken on supabase-py/postgrest-py? @kiwicopple

Related? https://github.com/supabase-community/supabase-py/issues/405 and https://github.com/supabase-community/supabase-py/issues/470


Cosmos: Adding databricks CI integration

This PR adds the required files to trigger a Databricks run that will:

Build the current Spark Connector
Authenticate to a Databricks cluster
Uninstall any libraries previously installed and restart the cluster. Wait until the cluster is ready.
Install the last built library and restart the cluster. Wait until the cluster is ready.
Clean any Notebooks installed in the workspace previously.
Install any Notebook that exists in the sdk/cosmos/azure-cosmos-spark_3-1_2-12/databricks/notebooks/ folder.
For each Notebook, trigger the execution and capture the job results. If the execution succeeded, the job is deleted to avoid pollution. If the execution failed, the job is maintained so the logs can be checked.

Initial files are leveraging a private pipeline, not yaml. I could add the yaml to the PR, but we need help from the Azure SDK team to set it up.
Currently it is completely manual, Databricks requires cluster restarts and other orchestration that cannot run in parallel in multiple PRs, so we would need to trigger it
Manual on any branch (including PRs)

/azp run azure-cosmos-java-databricks

@ealsur can we change the CI name? "azure-cosmos-java-databricks". There is nothing really related to java here.
perhpaps "oltp-spark-databricks" would be better name.


Child component  will still evaluate parent slot code

Version
2.5.16
Reproduction link
https://github.com/bbugh/v-if-slot-issue/commit/ffd6d476bf378936566b4916c1342af4310ef412
Steps to reproduce

Create a child component with a conditional slot:
<template>

    <slot v-if="showSlot" />

</template>

Use the slot in a parent component, using code that should not be called when v-if resolves as false:
<conditional-slot :showSlot="false">
  {{ example.should.not.be.called }}
</conditional-slot>

data: () => ({
  example: {}
})

Observe that slot code is called, and fails because "example" is an empty object.
A runnable demo of this is in the linked repository.
What is expected?
The behavior is expected to match v-if's behavior in other circumstances, which is that the code in the if block is not evaluated when the conditional check resolves as false.
What is actually happening?
The parent slot code is always evaluated, regardless of the value of v-if in the child component.

Hi! If I understand it correctly, this is not a bug.
v-if prevents the conditional block to be rendered if the condition evaluates to false, but it is not preventing Vue to parse/evaluate it. So, in your example, App.vue is still trying to access example.should, whether if it renders or not.
This is the compiled render function of your App component. Notice how example.should... is still being accessed.
I'd say this is expected behaviour, because v-if prevents rendering, not parsing/evaluation (AFAIK).
Hope it helps!
😄

No, that is not correct. v-if does not evaluate the contents if it is falsey. You might be thinking of v-show, which does evaluate the block whether it's shown or not.
Demo:

  {{ example.should.not.be.called }}

  {{ example.should.not.be.called }}

Hi again! 👋
Now I see that in my comment I didn't state anywhere I was talking about the v-if + slots context, not every v-if situation. My bad!
However, I'd say this is an expected behaviour because "everything in the parent template is compiled in parent scope" (source). And in your parent component, example.should does not exist, no matter what.
Be that as it may, maybe the docs should reflect this edge case somewhere. Or at least a warning?

Hi again! 👋
Now I see that in my comment I didn't state anywhere I was talking about the v-if + slots context, not every v-if situation. My bad!
However, I'd say this is an expected behaviour because "everything in the parent template is compiled in parent scope" (source). And in your parent component, example.should does not exist, no matter what.
Be that as it may, maybe the docs should reflect this edge case somewhere. Or at least a warning?

I am not sure why you’re commenting on my issue, I’m not asking for help.

@afontcu  We just upgraded to Vue and started experiencing this issue.  Do you happen to know if the default slot content was always eagerly loaded?  We had a layout template with a v-if statement that showed a loading screen until content was loaded and were surprised to find it breaking due to this after an upgrade to 2.6.10.

You can use the new v-slot syntax (<template #default></template>) to prevent the slot content from rendering until its needed.
v-slot documentation

I'd just like to elaborate on @yyx990803 and @sirlancelot's answers with a bit more detail for people landing on this page looking for quick answers. Take the following code as an example (preview at https://jsfiddle.net/yd2Lpcf1/4/):

Loading

Hi {{person.name}}

    <layout :loading="!person">
      Hi {{person.name}}
    </layout>

<template id="layout">

Loading

    <slot v-else></slot>

</template>

Vue.component('layout', {
  template: '#layout',
  props: ['loading']
});

new Vue({
	el: '#app',
  data() {
    return { person: null }
  },
  mounted: function () {
    setInterval(() => {
	    this.person = this.person ? null : {name: 'John'}
    }, 1000)
  }
});

When this is run in the fiddle linked above then the preview will be blank for a second, then "Hello John" will be shown twice. If run in its own page, then the error console will show the error Cannot read property 'name' of null, referring to the fact that person.name defined in the <layout> section was eagerly evaluated and raised an error. This error is thrown every second. This demonstrates that slots are evaluated eagerly and that the error halts the rendering of the child component and the parent component - "Loading" isn't printed in the first v-if, nor in the one in the child component. It also demonstrates that, even though the slot was eagerly evaluated originally, it is still re-rendered when the value of person changes.
Following @sirlancelot's recommendation, the only thing that needs to change is to nest an extra <template> layer inside the <layout>:
    <layout :loading="!person">
      <template #default>
        Hi {{person.name}}
      </template>
    </layout>

When the page is re-run then the preview will show "Loading" twice, for a second, and then "Hello John" will be shown twice, and these values will change every second. The error console will be blank. This demonstrates that the slot is lazily evaluated following the expected v-if logic, as expected, and the whole component is properly re-evaluated reactively, instead of just being re-rendered.
In short, when you're using a component with a conditional slot like <MyBaseComponent>...</MyBaseComponent>, simply wrap the content as @sirlancelot described (<MyBaseComponent><template #default>...</template></MyBaseComponent>), and the default slot will evaluate in the way you'd expect.
As an aside, if support for Vue 2.0 is expected to continue for much longer, I would recommend updating the documentation for slots with a section on the "Eager Evaluation" caveat and how it can be solved using the v-slot workaround.

https://forum.vuejs.org/t/check-if-components-slot-is-empty/6015/4


Pop-up for adding accounts

Choice of icon and name

Customize the account with the "···" rather than initial it when creating.


[ENH] BEP Update

This provides an update for each of our BEP projects. Included is a status summary and potential blocking point(s). Here is the rendering.
@bids-standard/bep_leads - if you may please review your BEP summary and provide suggestions or enhancements if needed. Thank you!
One change is bringing back BEP012, 013, 016 into the extensions table because they were split from the common derivatives (https://github.com/bids-standard/bids-specification/issues/254)

looks good for bep020

This looks like it's reverted almost all of its changes...

darn! will revert back to the edited version

This looks like it's reverted almost all of its changes...

thanks for catching that Chris! :grimacing:

reverted BEP018 back to the active list. Prepared this PR for a patch release. Can be updated when BEP018 is merged in for a minor release.
pinging @effigies @sappelhoff for review - here is current rendering

looks fine to me

@franklin-feingold are you ready to have this one merged?

should be ready to merge - will commit a fix for travis

cc @sappelhoff @effigies should be ready to merge

Thanks @franklin-feingold


Add video-minutes to each project

add as part of project a section housing the youtube or zoom... recorded sessions of the calls.


Incorrect temperature shown

The incorrect temperature is shown on all three thermostat devices (34 F).  Also, the following error occurs when the Homebridge is rebooted, that seems related:  "This plugin threw an error from the characteristic 'Current Temperature': Unhandled error thrown inside read handler for characteristic: This callback function has already been called by someone else; it can only be called one time.. See https://homebridge.io/w/JtMGR for more info."  The error is shown three times, once for each thermostat device.

Hey @ksdennis, can you fill out the bug context questions below? Thanks!
Plugin Config:
Show your Homebridge config.json here, remove any sensitive information.

Environment:

Plugin Version:
Homebridge Version:
Node.js Version:
NPM Version:
Operating System:

Closing as unable to reproduce. If you're still having the issue, please fill out the above, and reopen the issue. Thanks!


Issue with WriteBinaryFile to create a PDF

Describe the bug
WriteBinaryFile doesn't work to create a PDF
To Reproduce
Steps to reproduce the behavior:
Here a very simple example where I read a PDF and I want create a new PDF in a second step :
import * as tauriDialog from "tauri/api/dialog";
import * as tauriFs from "tauri/api/fs";
let a = await tauriFs.readBinaryFile("./template/temp1.pdf")
await tauriDialog
    .save({})
    .then((data) => {this.pathTauri = data;})
    .catch((err) => console.log("Error:", err));
await tauriFs.writeBinaryFile({ path: this.pathTauri, contents: a });

The PDF create is corrupted and it can not be read.
Expected behavior
How create a correct PDF ?
Platform and Versions (please complete the following information):
tauri]: running info
Operating System - Darwin(20.1.0) - darwin/x64
Node.js environment
Node.js - 14.15.0
tauri.js - 0.13.0
Rust environment
rustc - 1.48.0
cargo - 1.48.0
tauri-bundler - 0.9.4
Global packages
NPM - 6.14.10
yarn - 1.22.10
App
tauri.rs - 0.10.0
mode - embedded-server
build-type - bundle
CSP - default-src blob: data: filesystem: ws: http: https: 'unsafe-eval' 'unsafe-inline'
distDir - unset
devPath - unset

same error

same error

Fix incoming.
https://github.com/tauri-apps/tauri/pull/1136

Fix incoming.
https://github.com/tauri-apps/tauri/pull/1136


[New chain]: ABEY Chain

Summary
I would like to add ABEY Chain Mainnet.
Chain ID
179
Chain detail URL
https://chainlist.org/chain/179
RPC URL
https://rpc.abeychain.com
Blockscout Client URL
No response
Etherscan Client URL
No response
Etherscan Client API URL
No response
Version
1.3.0
Address (Master copy)
0x69f4D1788e39c87893C980c06EdF4b7f686e2938
Deployment Tx hash (Master copy)
0x01b9d0e900ede157e68ef54ac711db956a77a5b3855b5a29e19cd66cf9b3c3cc
Block explorer URL (Master copy)
https://abeyscan.com/address/0x69f4D1788e39c87893C980c06EdF4b7f686e2938
Address (Master copy L2)
0xfb1bffC9d739B8D520DaF37dF666da4C687191EA
Deployment Tx hash (Master copy L2)
0xea3725ab9692b87d288bb7aa5c26b830bbd4f59806e117d7baaf20685c7bdc16
Block explorer URL (Master copy L2)
https://abeyscan.com/address/0xfb1bffC9d739B8D520DaF37dF666da4C687191EA
Address (Proxy factory)
0xC22834581EbC8527d974F8a1c97E1bEA4EF910BC
Deployment Tx hash (Proxy factory)
0xeb294c93cd50678a1a279237a05635303921c6f95af546132e21e9c5499d8d36
Block explorer URL (Proxy factory)
https://abeyscan.com/address/0xC22834581EbC8527d974F8a1c97E1bEA4EF910BC

/execute


Add linting rule enforcing reservation of capitals for constructors?

Context: https://github.com/ipfs/js-ipfs-http-client/pull/1124#discussion_r333379683
Opening this here for any further discussion so the issue doesn't get lost.
I don't know if this is a good idea or if this is a rule in eslint we can enable, only that it's outside of the scope of ipfs/js-ipfs-http-client#1124

Also make sure to reflect it on the contributing guidelines https://github.com/ipfs/community/blob/master/CONTRIBUTING_JS.md


[Textures] Allow dropping resource pack .zip files, automatically construct texture atlases .png

With https://github.com/satoshinm/NetCraft/pull/93 you can now drag-and-drop a .png texture file from your desktop to the app, but it has to be in a texture atlas structure, like this: https://github.com/satoshinm/NetCraft/blob/master/textures/texture.png
In addition, it would be useful to allow dropping some kind of "texture pack" files, .zip format, and construct the atlas from it. Also perhaps allow this texture pack to be specified on the server-side build and use it automatically instead of the provided built-in textures.

Until zips are supported, what about changing to use similar texture offsets as the texture packs in https://github.com/satoshinm/NetCraft/pull/117? To save some time copying & pasting at different locations and allow the packs to be dragged-and-dropped into the app to set as the texture atlas easily.

Instead of a separate texture with all the complications that entails, what about enlarging the main texture from 256x256 to something larger, and using a different part of it? http://webglstats.com/webgl/parameter/MAX_TEXTURE_SIZE says 100% of WebGL supports 2048x2048 and 99% supports 4096x4096.

Reading zip files: consider miniz, https://github.com/richgel999/miniz - a single C file amalgamation, zlib-replacement, no other dependencies. Curiously, it also includes a png writer. Note that Craft already includes lodepng, which has its own zlib decompressor. And there are emscripten ports for the full-featured libpng, zlib, and zlib/contrib contains minizip - but try to stay with the lightweight libraries instead, if possible, and avoid redundancies. There only needs to be one zlib decompressor. Can miniz replace lodepng, too?

stb_rect_pack? "rectangle packing - Useful for e.g. packing rectangular textures into an atlas."

Watch this space: https://github.com/webbukkit/DynmapBlockScan "Tool to scan post-1.8 MC blocks and JSON model data to automatically generate Dynmap models"


Fix tests to work against Bitcoin Core 0.18

Somewhat related to https://github.com/GlacierProtocol/GlacierProtocol/issues/38
Any given version of the protocol should state what version of Core it is tested against and compatible with.

Can you elaborate on the problem you're seeing? GlacierScript and the developer tests have been compatible with 0.18 since #50, and I've run the tests on v0.18 many times myself, including just now.

I was getting a 'method not found' error for getaddressesbylabel; maybe the bitcoin-cli was older than the bitcoind version

Very strange; getaddressesbylabel was added in v0.17, and GlacierScript does a version check using getnetworkinfo to ensure you have at least v0.17. I guess that only checks bitcoind and not bitcoin-cli?
Do you think we should add a version check of  bitcoin-cli also? This condition is not something we would ever see on the quarantined laptops.

I wouldn't worry about it; I updated both of my binaries to 0.18.1 and the tests pass


Making howto.md

Make guidebook to run Wopslang and to contribute this repo.
File will be made in doc folder.

merged with the "Get Started" Section.

ca73fc


Update MSFT_SPSearchContentSource.psm1 issue #840

Regarding issue MSFT_SPSearchContentSource does not take StartHour parameter for schedule #840
I was able to solve this issue, but I must admit that I am a newbie regarding the pull request process...
So here it is. From what I was able to understand, the StartHour parameter is correctly checked in the test Get-SPDSCSearchCrawlSchedule.psm1 module.
The issue seems to be in the MSFT_SPSearchContentSource.psm1, as in its Set-TargetResource function there is never the parameter CrawlScheduleStartDateTime which is required by the sharepoint powershell cmdlet Set-SPEnterpriseSearchCrawlContentSource to had a start hour to the schedule.
Hope it helps.

This change is

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

Codecov Report

Merging #841 into release-2.3 will increase coverage by <1%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##           release-2.3    #841    +/-   ##
============================================
+ Coverage           91%     91%   +<1%
============================================
  Files              111     111
  Lines            10744   10752     +8
  Branches             1       1
============================================
+ Hits              9852    9860     +8
  Misses             891     891
  Partials             1       1

OK, sorry for my beginer's mistakes.
So I added this to changelog:
Unreleased

MSFT_SPSearchContentSource

Fixed issue where the parameter StartHour was never taken into account (issue 840)

Hope it's ok the way I did it.

No problem, we have all been there ;-)
Please make sure you add the item in alphabetical order.

Hum, I see, so I took the changelog file from the dev branch and inserted the comment on my fix respecting the alphabetical number:
Change log for SharePointDsc
Unreleased

SPCacheAccounts

Fixed issue where the Test method would fail if SetWebAppPolicy was set to
false.

SPDistributedCacheService

Updated resource to allow updating the cache size

SPFarm

Implemented ability to deploy Central Administration site to a server at a
later point in time

SPInfoPathFormsServiceConfig

Fixed issue with trying to set the MaxSizeOfUserFormState parameter

SPProductUpdate

Fixed an issue where the resource failed when the search was already paused

SPProjectServerLicense

Fixed issue with incorrect detection of the license

SPSearchContentSource

Fixed issue where the Get method returned a conversion error when the content
source contained just one address
Fixed issue where the parameter StartHour was never taken into account (issue 840)

SPSearchServiceApp

Fixed issue where the service account was not set correctly when the service
application was first created
Fixed issue where the Get method throws an error when the service app wasn't
created properly

Ok, just noticed you used the wrong branch as starting point. You used branch release-2.3 (branch used for releasing v2.3), but should have used the branch "dev", which is the active development branch. I tried to change branch, but this means a lot of other files are also included into this PR.
Can you please use the Dev branch as your starting point and resubmit the PR?

My mistake, sorry about that. I will close this one and open a new one.


Service Principal Sample

@vladimir-shcherbakov,
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


Ensure git remotes for local repositories

Issue N/A
Description of changes:
This patch adds more logic to pkg/repository.Manager to ensure that
the local repositories remotes point to the right URLs.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/approve

/lgtm


Docker Image Compatibility with Jenkins Kuberenetes Plugin: Be Able to use Docker Image Interactively OOB

User story.
I would like to be able to run Prism in Jenkins as a container
Is your feature request related to a problem?
Out of the Box you can't use the default Prism image in Jenkins using the Jenkins Kubernetes Plugin)
Describe the solution you'd like
Maybe there could be an interactive variant of the docker image, or just let the prism command be executable from the shell?

Hey,
I am not really familiar with Jenkins system as a container and the Prism command is effectively excitable from the shell if you use the exec command in docker instead of the regular run. Would it be possible to get more details?
Feel free to reopen or respond once you have more data for me!
Thanks!

Hey thanks for responding so quickly,
Basically Jenkins in Kubernetes is run by the overarching Jenkins master orchestrating at least one new Jenkins "agent" (sometimes referred to as a slave) as a Kubernetes Pod. In these pods you specify containers with tools you want to use and then inside your pipeline you can use the shell in those containers to execute commands. The Jenkins workspace is mounted to each container so the idea for Prism would be to have Prism run, have tests performed, and then have Prism output the proper artifact for the Prism Jenkins plugin.
Correct me if I'm wrong but when running prism using the docket command it will exit?
Thanks,
Julian

I am a little bit unprepared here — to help you I would need to check out what is going on or you'd need to paste here some evidences (logs, screenshots).

Correct me if I'm wrong but when running prism using the docket command it will exit?

This seems to be incorrect to me. Prism is a long running server and won't stop as long you do not send a signal it won't stop. You can verify this by yourself by running the docker image:
docker run --init stoplight/prism:3 mock https://raw.githack.com/OAI/OpenAPI-Specification/master/examples/v3.0/petstore-expanded.yaml
Here's the result:

You can see that the server is not exiting but waiting for the connection. Is this what you were referring to?


Cannot read property 'role' of undefined

When using mainView.router.back();
Uncaught TypeError: Cannot read property 'role' of undefined
at Object.eval (eval at js_compare (framework7.min.js:22), :1:30)
at Object.js_compare (framework7.min.js:22)
at eval (eval at s (framework7.min.js:1), :7:456)
at Object.template7Render (framework7.min.js:16)
at Object.i.router._back (framework7.min.js:16)
at framework7.min.js:16
at Object.preprocess (framework7.min.js:16)
at t (framework7.min.js:16)
at framework7.min.js:16
at window.Framework7.i.get (framework7.min.js:15)

I solved with domCache: true, :)
The problem is caused by removed passed parameters.

Can this be closed


Add avatar configuration to change default avatar

What kind of change does your PR introduce?

[ ] Bugfix
[x] Feature
[ ] Refactor
[ ] Tests
[ ] Other, please describe:

This commit adds basic support for avatar configuration. It accepts a list of available Avatars (among other properties) and automatically uses either:
The one with the property "isDefaultAvatar" set to true
The first avatar of the list
In case the availableAvatars list is empty, it will use the default one

This PR is obsolete because of #152