Calculating estimated time remaining for Transformer

I'm attempting to calculate an estimated time remaining for transformer. Currently from the docs all I can see is getting the progress from 0-100%. Is there a way to get the video file that is currently being processed? That way I can calculate the estimated time remaining by calculating how long each video takes?

There is no available public API that can give this information at the moment. The best solution today would be to use the progress available and the total duration of the composition. The approach would be to estimate how much time it took to process 10% of the composition and estimate the time remaining.
We will add this feature request to our backlog.

Reopening to track as an enhancement.


Show email address in the email sent pages

See https://www.pivotaltracker.com/story/show/86103092
The email parameter was already available in both the registration code sent and email verification email sent pages

@rjkip

Well that was easy. :+1:


Add TrueGalaxyCash (TGCH) icon

Per my comment in https://github.com/atomiclabs/hyperdex/pull/521#discussion_r217946231:

Open an issue on https://github.com/atomiclabs/cryptocurrency-icons and we'll have a proper icon made for you.

We don't accept contributions as all logos should follow the exact same template.


allow grafana container to be named

For people like me, who monitor various clusters, I end up with
a lot of clusters named agraf-3000, agraf-3001, agraf-3002.
Weeks later, I have no idea which is which. I usually resort to
docker rename after the fact, but it doesn't cost us much to add
it as an option here.
Signed-off-by: Glauber Costa<EMAIL_ADDRESS>

Thanks @glommer


Allow VS using repository SDK to build for latest TFM

Summary
Manual backport of #4123 to release/7.x

Release Notes Entry

/backport to release/7.1

/backport to release/6.x

/backport to release/7.0


compat fix

Ant-cobertura multi module fix went in S92.
Support for Ant-cobertura single module went in S91.
This fix will handle S91-S92 compat
Testing done : manual testing with S91 and S92 bits.

looks fine


Support Kubernetes Devfile component

/kind feature
service operations should occur on odo push. we need to store the service information in the Component as a CR in the devfile.

In order for this to work, we first need to add support for kubernetes components.
Devfile can specify kubernetes component like this:
components:
  - name: my-mongo
    kubernetes:
      inlined: >
        apiVersion: v1
        kind: Pod
        metadata:
          name: nginx
          labels:
            name: nginx
        spec:
          containers:
          - name: nginx
            image: nginx
            ports:
            - containerPort: 80

or
components:
  - name: my-mongo
    kubernetes:
      inlined: >
        apiVersion: mongodb.com/v1
        kind: MongoDB
        metadata:
          name: my-replica-set
          namespace: tkral-test
        spec:
          credentials: my-credentials
          members: 3
          opsManager:
            configMapRef:
              name: my-project
          type: ReplicaSet
          version: 4.4.0-ent
          persistent: true
        status:
          phase: Running
          type: ReplicaSet

The idea is that the content of a kubernetes component can be any Kubernetes resource (core one, or custom resource).
odo push needs to recognize kubernetes components and apply them to the cluster.

In order for this to work, we first need to add support for kubernetes components.
odo push needs to recognize kubernetes components and apply them to the cluster.

I think we can take a small step first by adding the definition of the resource to the devfile. In the next step, we can ensure that odo push does something of it. I think there will be a lot of discussion for that. For example, I have asked some questions in https://github.com/openshift/odo/issues/4160#issuecomment-721552794. I also remember we were playing with the idea of breaking down odo push so that individual pieces are pushed: a component, the URLs, service, etc.

Devfile can specify kubernetes component like this:
The idea is that the content of a kubernetes component can be any Kubernetes resource (core one, or custom resource).

+1 to the examples. Makes sense.

I think we can take a small step first by adding the definition of the resource to the devfile. In the next step, we can ensure that odo push does something of it. I think there will be a lot of discussion for that. For example, I have asked some questions in #4160 (comment). I also remember we were playing with the idea of breaking down odo push so that individual pieces are pushed: a component, the URLs, service, etc.

I'm actually proposing doing it the other way around. The first step needs to be to ensure that odo push can correctly interpret kubernetes component.

I'm actually proposing doing it the other way around. The first step needs to be to ensure that odo push can correctly interpret kubernetes component.

My concern on doing that first is that we're just getting started with refactoring and a lot of things are expected to change. I don't intend to wait till refactoring is over but, at the moment, there's almost no clarity on how odo push code is going to change.

My concern on doing that first is that we're just getting started with refactoring and a lot of things are expected to change. I don't intend to wait till refactoring is over but, at the moment, there's almost no clarity on how odo push code is going to change.

We will first need to address changes in devfile/parser or in parser in odo as there is no support for kubernetes component.
Before we get into writing the actual odo push logic the refactoring will probably be done, or at least it will be clear how much changes will be needed there.

We're waiting on some movement on https://github.com/devfile/api/issues/216 for this to be able to move further. The code for this needs to go into library and not in odo.

@kadel @dharmit  - I'm noticing the recent updates like #4656 to support kubernetes-typed components focus on support operator-backed services.
I'm curious if a use case like Tomas's naked Mongo Pod described here:  https://github.com/openshift/odo/issues/4159#issuecomment-723113667
is still considered a valid use case.
On the one-hand, I can see how using an operator would be more powerful for something like a database.
But I'm interested in where we're headed with this case.    Is the inlined kubernetes component going to allow a devfile author to insert any yaml that will be kubectl apply(d)?   Or is odo always going to carefully parse and validate it against the resources in the cluster?
That is, even if this isn't the most useful way to create a Mongo DB I'm curious if the vision is to allow any yaml to be applied via an inline component?

@scottkurz I'll try to answer that but will rely on @kadel to validate my response.
At a high level, odo is currently not focussing on the users who would modify devfile by hands. That is, if someone's editing it manually and not interacting with it via odo, there's no guarantee that things would work.
Having said that, we do understand that for architects or Devfile authors like you, Andy and others, modifying Devfile by hands could be a "norm".
What odo does in the code is use the dynamic client to create the Kubernetes inlined component: https://github.com/openshift/odo/blob/c0be940e1df15ca1ebf410de962932b327d9d5a7/pkg/service/service.go#L856-L867
Theoretically, you could very well create a Pod or any other Kubernetes resources of your choice using this. We haven't tested it, however, because such use cases are currently not a priority for odo.
@kadel correct me if I am wrong.

At a high level, odo is currently not focussing on the users who would modify devfile by hands. That is, if someone's editing it manually and not interacting with it via odo, there's no guarantee that things would work.

Not really. Odo uses Devfile v2. And as such, it needs to be able to correctly interpret any valid devfile.

The thing that we are not focusing on and it will always be out of the scope for odo is providing a command to modify every aspect and field in devfile. Odo devfile editing capabilities will be always limited to just the most common use cases. Fot the rest usrs will have to edit devices directly.

Theoretically, you could very well create a Pod or any other Kubernetes resources of your choice using this. We haven't tested it, however, because such use cases are currently not a priority for odo.
@kadel correct me if I am wrong.

If that Devfile has Pod or any other k8s resource as a Kubernetes component odo needs to be able to interpret it.

OK.. I opened https://github.com/openshift/odo/issues/4685  to follow-up on this use case.
Let me just note again I'm not saying this is particularly important to the set of top Open Liberty use cases.   I just view it as a test case understanding the full devfile vision.

Original request in the issue description is implemented.
/close


Add dashboard banner on Grid page

Reorgnize Grid page structure. ResultSummary.jsx contains
TestBreadcrumb, Overview, PieChart, Checkboxes React component.
update getTotals API to take either id or combination of url,
buildName and buildNum
remove renderResults() and change it to a react component so it can be
used by mutiple places
update to use componentDidUpdate() as componentWillReceiveProps() is
deprecated
update 3rd party libs

Signed-off-by: lanxia<EMAIL_ADDRESS>


Remove eslint-plugin-import-order-autofix

… and replace it with import/order since eslint-plugin-import-order-autofix isn't maintained well.
The alternative doesn't have an auto fix option, but since all dependants already have the autofix applied, this shouldn't be a big deal.

There's also eslint-plugin-codebox/sort-imports. However it would be great if we'd simply stick to the built-in one.

import/order is now autofixable 🎉


Regarding usage

how to use this tool

It's under development. Hope, it will be complete by the this or next month.


Repicate To-DO List repository

Clone To-do-list repository

Approved ✔
Hi George,
Great job, Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

(Highlights) Good Points: :+1:

You documented your work professionally. ✔
You used the correct workflow.  ✔
All linters checks are ok. ✔

(Changes Required) Aspects to improve: ♻

N/A

[Optional] suggestions:

N/A

**Cheers and Happy coding!👏👏👏

Hello @ohyajk .
Thank you for the positive review. 👍
Happy coding!


"srcset" doesn't seem to be working

Here is my file structure.
component > subcomponent > Example.vue
component > subcomponent > assets > photo.jpg
component > subcomponent > assets ><EMAIL_ADDRESS>
Below is what I am trying to.

<img src="./assets/photo.jpg"<EMAIL_ADDRESS>2x">

<img src="/static/img/photo.jpg"<EMAIL_ADDRESS>2x">

<img src="/static/img/photo.jpg"<EMAIL_ADDRESS>2x">

Maybe html-loader support srcset, but vue-html-loader does not support it?
Looks like same problem here.
https://stackoverflow.com/questions/40733102/vue-webpack-project-img-cannot-load-static-src

you have to tell vue-html loader to parse these attributes.
How to do this is explained here, and you have to insert those options in your wenpack files after this line:
Not tested but should look something like this:
vue: {
  html: {
    query: {attrs: "img:src img:srcset"}
  }
  loaders: //....
}

I try to your recipe. Unfortunately it does not work.

<img src="./assets/photo.jpg" srcset="./assets/photo.jpg 1x<EMAIL_ADDRESS>2x">

<img src="/static/img/photo.jpg" srcset="./assets/photo.jpg 1x<EMAIL_ADDRESS>2x">
<img src="data:image/png;base64,..." srcset="./assets/photo.jpg 1x<EMAIL_ADDRESS>2x">

<img src="/static/img/photo.jpg"<EMAIL_ADDRESS>2x">

Also I found it related base64 encoding loader(probably url-loader?). But I am not sure srcset attirbute can parse base64 encoding with multiple image target such as 1x or 2x.

It seems that srcset specifically is not currently supported by HTML-loader. There was an attempt to add this as a feature, but it was reverted: webpack/html-loader#79
So I guess you have to do this manually, by importing the impages in the Javascript and using a templatze literal:
var photos = {
  1: require('./assets/photo.jpg'),
  2<EMAIL_ADDRESS>}

export default {
  // ...component stuff
  computed: {
    photos() { return photos }
  }
}

<img src="/static/img/photo.jpg" v-bind:srcset="`${photos.1} 1x, ${photos.1} 2x`">

If your webpack config has a limit for the url-loader and you want to prevent inline data-urls, you may have tooverwrite the loader:
require('!!url-loader?limit=0!./assets/photo.jpg')

I made a PR. (https://github.com/vuejs/vue-loader/pull/953)

Would love for this PR to be merged...


chore: move autosuggest tests from Enzyme to RTL

Description
Move autosuggest tests from Enzyme to RTL
Deploy Preview
N/A
Merge Checklist

[x] Is there adequate test coverage for your changes?

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

So the current failing test is the error state test. It was written by having the simulated user enter the textbox and press escape. This worked before https://github.com/openedx/paragon/commit/4816baac16644eabb6362405627555d71ce77cd8#diff-96e13c1a773ec8d218ea5667d3dec6c8f6eeadcb00c81584f50d6c3d4d52135fL133, but now that doesn't trigger the error state.
It should probably be updated to simulate a click outside, as that will still trigger the error state (I verified on https://paragon-openedx.netlify.app/components/form/form-autosuggest/)

So the current failing test is the error state test. It was written by having the simulated user enter the textbox and press escape. This worked before 4816baa#diff-96e13c1a773ec8d218ea5667d3dec6c8f6eeadcb00c81584f50d6c3d4d52135fL133, but now that doesn't trigger the error state.
It should probably be updated to simulate a click outside, as that will still trigger the error state (I verified on https://paragon-openedx.netlify.app/components/form/form-autosuggest/)

Fix here: https://github.com/openedx/paragon/compare/ba5085a9a58e9a35f545070541decaba69c8130f..33fb35b955f6e7de8b872791fe5794616300cff9

@PKulkoRaccoonGang I'm sorry I was a bit hasty and didn't see these comments before hitting merge. We should be able to address them in a quick follow-up PR.

:tada: This PR is included in version 21.1.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 22.0.0-alpha.6 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Nested tables?

Is it possible to nest tables?  i.e. to pass in a BootstrapTable component as the content of the parent table's TableHeaderColumn's dataField?

HI, check following and let me know if you got any questions
https://github.com/AllenFang/react-bootstrap-table/blob/master/examples/js/expandRow/expandRow.js


Model key with name "attributes" not works

Hi MongoMapper team!
Please explain, attributes is a valid name for key or not?
If yes, could you please complain about that to stderr/stdout while you see user tries to create such attribute? We spending a lot of time founding why our app not working with such key name..
class Point
  include MongoMapper::Document
  key :name_id, ObjectId
  key :value, Float
  key :attributes, Hash, {}
...
end

Thank you!

Thanks for the bug report.
Yes - attributes is the method used in ActiveModel / ActiveRecord to get a hash from the object.  Usually #attributes= will take a hash and assign each key.  I definitely wouldn't use it!   :)
Will write a test + raise an error for this.  Thanks for reporting!


Add Windows and MacOS to the testing matrix

I suggest to add new workflows:

[ ] Running tests on Safari
[ ] Running tests on Chrome + Mac
[ ] Running tests on Chrome + Windows
[ ] Running tests on a Retina screen (window.devicePixelRatio = 2)

Why Chrome?
Because the other Browsers already tested on a Linux system and with testing Chrome on Windows & Mac we include the biggest part of the users.
@mourner @jonkoops is this possible and do we want this?

I remember trying this and running into some issues, although I don't quite remember what was blocking. This should be possible by restoring the matrix for operating systems that was removed under #8264.
Perhaps we also want to test on Windows?

Perhaps we also want to test on Windows?

Good point.
I added https://github.com/Leaflet/Leaflet/labels/hacktoberfest, maybe there are some devs with knowledge out there.

Sounds good to me :+1:

I am new to this repository, can i get a headsup like from where to begin?
As per the above discussion what i understood is to add tests for windows and macos in this file (.github/workflows/main.yml). Correct me if am wrong or missing something? @Falke-Design

We are runnning our tests with karma / mocha. For how to run tests look here.

add tests for windows and macos in this file (.github/workflows/main.yml).

Yes we need windows and macos and also Safari.
Have you experience with creating workflows?


Nixos installation issue in 22.05

Not using flake,
Latest nixos (22.05) has wayland version 1.20.0, I got an error saying I need at least 1.21.0 so I created a nixpkgs overlay Ty use it from unstable repos, then after hours of compiling I got another similar error for libdrm. I have those chacix lines in my configuration. I have decent specs in my laptop and it's been like 8h 😂. Any better way?

Upgrade to nixos-unstable, we don't support nixos-22.05


issue/pr migration from ophyd/v2

Issues/PRs relating to ophyd v2 should be migrated to this repository, perhaps by using a tool like Github Issue Mover.

Let's just close them and reopen here


Shayan/feat: added @deriv/eslint-config-deriv

changes: removed eslint packages and instead added the new version of @deriv/eslint-config-deriv package which uses latest version of eslint which is compatible with TS version 5

I'm closing this PR  in favor ofhttps://github.com/deriv-com/smarttrader/pull/13


ARROW-17228: [Python] dataset.write_data should use Scanner.projected_schema when passed a scanner with projected columns

Issue: https://issues.apache.org/jira/browse/ARROW-17228

@lidavidm thanks for approving. I don't think I have permission to merge this, so I'll leave it to you.


Is there a plan for CondenseNet?

"CondenseNet: An Efficient DenseNet using Learned Group Convolutions"   paper says it is more efficient than DenseNet.

Also looking forward someone could transfer the condensenet to keras.


Changes to the TracksActivity class

making changes to the TracksActivity class code by moving certain code from onCreate() to onResume() function.

@arp95 I'd suggest to make a single PR for similar changes.
Making multiple pull requests with similar fixes makes no sense.
Please take this into consideration from now onwards.

@the-dagger actually i saw somewhere you told someone to make different PR for different activities. thats why i was doing all this individually which was also taking up my time.
Will take this into account.

@arp95 All similar issues need to come under one PR. Issues that require different fixes will come under different PR's. Cheers!

Closing it. Please follow advise. Cheers.


add readme from https://code.google.com/archive/p/lightweight-java-profiler

Thanks for the PR, but this fork is no longer maintained.  I highly recommend https://github.com/jvm-profiling-tools/async-profiler as a much better replacement

OK thanks.


orientation data gets removed after compression

Current behavior

orientation data gets removed after compression, resulting in rotation of images on some cases, specifically Android.
test in a Samsung Phone when using camera taken images. my guess is photos taken by samsung camera all default to horizontal/landscape orientation and gets rotated 90 CW for portraits

Expected behavior
Platform

[X] Android
[X] iOS

React Native Version
0.66.7
React Native Compressor Version
1.5.2
Reproducible Steps And Demo

For reference, here are the images I used:

+1

Any movement on this one @numandev1?

@briangonzalez can you provide me any reproducible example because it is working well on my side

@jeemercado No, I don't have a Samsung device, without reproduce I am unable to fix it

@jeemercado After 3 to 4 weeks, I am going to revamp the compressor package, hopefully, I will fix this issue by revamping

🙏 sorry for a long ago I was not maintaining due to the huge office workload, hopefully, I am going to start fixing bugs on a weekend basis,
I will fix at least 2 issues on the weekend, for priority bugs you can join the discord channel https://discord.gg/dtYzk8sp where we can discuss which bug is most important

can you check 1.8.2 version if you got this issue again?

fixed in the latest version, please feel free to reopen this issue again if this issue happens again

@numandev1  Same issue when taking picture via react-native-vision-camera and compression is setting to manual (on auto it's normal). Image picker working good.
Testing on iPhone 7 iOS 15.8 and couple of android devices.
React-native 0.71.8
"react-native-image-picker": "^5.6.0",
"react-native-compressor": "^1.8.23",
"react-native-vision-camera": "^2.15.4",


The charm thing isnt updated

When you press C the charm thing shows I am on p2 and need around 600 charms. It still shows 440 total charms that were required in p1

Hello @Dasher-Minecraft , thanks for the information, I thought the charms values were fixed so I "hard-codded" it to save memory. It will be dynamically set on future updates !


Added a CentOS box that works

The old CentOS box was unavailable.

Sorry I totally missed this. (Had an overly aggressive mail filter.)
I have merged another PR that should fix this.
Sorry about the super long delay.


Support v16 of react and react-dom

React 16 is out.
Getting this:
warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^15.0.0".
warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^15.0.0".

Fixed in 1a7aae


How to implement without diffusers?

I'd like to add support for controlnet inpainting in ComfyUI - so that we can use it in the AI horde - but none of our pipelines are using diffusers. Any idea how this can be done with the RunwayML version of stable diffusion?

Yes, it should be possible if you're only talking about the change of code implementation.
This project is already running on the RunwayML version of SD ("runwayml/stable-diffusion-inpainting"), but needs a separate set of weights ("lllyasviel/sd-controlnet-canny") for the ControlNet component.
I am not familiar with ComfyUI, so I can't provide more detailed advice, but yes, as long as you have access to both model weights (inpainting and ControlNet) you should be able to rewrite everything without involvement of diffusers, but it will require quite a bit of manual work. This type of approach will not be provided in this repository but you are of course welcome to edit it in any way and publish as a separate project or make a PR.


Feature/movie source/evans

I have done the following in this branch

[ ] Created Movie class in a separate .rb file.
[ ] Created Source class with an association to the Item class (in a separate .rb file).
[ ] All Movie class properties visible in the diagram are defined and set up in the constructor method.
[ ] All Source class properties visible in the diagram are defined and set up in the constructor method.
[ ] Implemented methods:

[ ] add_item method in the Source class

[ ] takes an instance of the Item class as an input
[ ]  adds the input item to the collection of items
[ ]  adds self as a property of the item object (by using the correct setter from the item object)

[ ] can_be_archived?() in the Movie class

[ ] overrides the method from the parent class
[ ] returns true if parent's method returns true OR if silent equals true
[ ] otherwise, it return false

[ ] Added unit tests for all implemented methods.
[ ] The following options are available:

[ ] List all movies
[ ] List all sources (e.g. 'From a friend', 'Online shop')
[ ] Add a movie

[ ] All data is preserved by saving collections in .json files.
[ ] Created a schema.sql file with tables that will be analogical to the structure of the classes that you created:

[ ] movies table (add all properties and associations from the parent Item class as table columns)
[ ] sources table

I have update this part of the item.rb `def to_h
{
genre: @genre,
author: @author,
label: @label,
source: @source.name,
publish_date: @publish_date
}
end
private
def can_be_archived?
publish_date = Date.parse(@publish_date)
#date >= 10.... update/add >= in your code. this ensures we include the current year
(Date.today.year - publish_date.year) >= 10
end`


fix: release

Previous release failed https://github.com/defuse-protocol/defuse-sdk/actions/runs/12056031471 because commit message was not "correct"

:tada: This PR is included in version 2.14.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Smart folder possibilities within Sanity

While work is being carried out on the production environment could be good to look at smart folders. Particularly in the TOPIC CONTENT lane. Be good to have editorial control to set up/name the folders and allocate topic under the relevant one. This would follow the logic of the site structure.
FOLDER NAME Sections

Energy
Sustainability
etc

FOLDER NAME energy

Natural gas
fields and platforms
etc

How should we deal with this one. Is it captured in other stories? Nice to have?


Go to end of line, beggining of line shortcuts don't work in Jupyter Notebook

Bug: Notebook Editor, Interactive Window, Editor cells
Steps to cause the bug to occur
To reproduce :

Bind cursorLineEnd cursorLineStart to keys
Create jupyter cell in notebook (markdown or code)
press keybind, nothing happens

Actual behavior
Nothing happens
Expected behavior
Going to line start / end
Your Jupyter and/or Python environment
Please provide as much info as you readily know

Jupyter server running:                       Local
Extension version:                               2019.11.50794
VS Code version:                                1.41.0-insider
Setting python.jediEnabled:               true | false
Python and/or Anaconda version:     3.7.5
OS:                                                     Linux (xubuntu):
Virtual environment:                          any

Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer

Sorry but keybindings from within VS code are not currently honored in the notebook editor. Our notebook editor is really a hack that uses our own custom editors under the covers.
See this bug for other key bindings that don't work:
https://github.com/microsoft/vscode-python/issues/7968
Please up vote the other issue. Thanks.


Doesn't work without sprockets

I've just created a new rails app like this: rails new app_name --skip-sprockets
Then I ran the two commands:
./bin/bundle add tailwindcss-rails - worked fine
./bin/rails tailwindcss:install
This second command fails with:

NameError: uninitialized constant Tailwindcss::Engine::Sprockets

Since I'm planning on using Webpacker exclusively, I don't see the need for having to add Sprockets just to be able to use this gem. What do you think?

Confirmed. After I've uncommented require "sprockets/railtie" in application.rb, the installation succeeded.

Hi @vfonic.
Unfortunately, that error is unrelated to the Tailwind CSS Nova extension, but instead is related to the tailwindcss-rails gem. If you submit an issue there, hopefully someone should be able to help. The Tailwind CSS Nova extension in no way relies on Sprockets.
Good luck!
Thanks,
Jason

This gem is designed to be used exclusively with the asset pipeline. It has no relevance for a Webpacker setup. You can just install Tailwind as normal for that.

Perhaps that was true in the past, but it seems like the gem supports both Sprockets and Webpacker today.
As per README:

Tailwind CSS for Rails works with both the asset pipeline and Webpacker.

The install generator checks whether the app uses Sprockets or Webpacker:
https://github.com/rails/tailwindcss-rails/blob/main/lib/tasks/tailwindcss_tasks.rake#L3-L9
Please reopen. Thanks!

Lol, yes, forgot we added that. Not sure it’s a good idea to have dual purpose like this, though.

I understand. It's a tough call to make. I'd argue that, since the introduction of Webpacker, many gems have opted for the dual purpose/support. It's just the way things are now. I think having dual support makes more sense than having to maintain two separate gems that could have a lot of shared functionality (things like: purger, compressor, etc.)

Tailwind packed through Webpack has all those functions natively. This gem specifically implements those functions via Ruby for the asset pipeline. And then offers a slim, probably too-confusing wrapper on installing TW for Webpack.

@dhh I could do two things:

Fix this issue and continue to support webpack
Drop webpack support

I agree with your point that easiest way forward would be to just support asset pipeline and have one north star. Fix would include conditionals inside the engine.rb(or somekind of conditionals :))
require "tailwindcss/compressor" if defined?(Sprockets) # maybe also here

module Tailwindcss
  class Engine < ::Rails::Engine
    if defined?(Sprockets)
      ...
    end
  end
end

That being said I'm not gonna lie, spined-up numerous webpacker apps and this generator was helpful, but...  :)

Main issue to use TW with asset pipeline is inability to use @apply out of the box. Every TW production app that I've worked with used @apply to extract common components such as button. Using @apply is maybe anti-pattern if we look at the TW idea, but without it some kind of componentization(view_component,partials) would be "necessary"

I think we should remove the webpack installer. This heart of this gem is a sprockets-specific approach to using TW with the asset pipeline. Let's just go with that.
Re: @apply, yes, I agree, would be nice to have a a sprockets version of this.

This is "fixed" here https://github.com/rails/tailwindcss-rails/pull/64


Adding additional step

for some reason my setup needed quart app environment variable

That shouldn't be necessary because it's in the .env file already, I don't understand why you needed it.

I just realized that this was needed in my messed up enviroment and I didnt realize how messed up it was. Tried it on a clean ubuntu install and indeed it works. My apologies.


CSS for portal and foundations updates

This PR is simply to do some CSS/styling updates (or any updates to the theme that are backwards compatible) for its use in the Pythia Portal and the Pythia Foundations sites.

Merging to get updated release for changes to Portal/Foundations


Store calibration on camera optionally.

This PR adds a field in the request to decide whether to store the calibration on the camera or not.
Depends on https://github.com/fizyr/dr_ensenso_msgs/pull/1
@vsuryamurthy  can you review it? @hgaiser maybe you want to have a look ;)

Could you also expose this functionality in the Ensenso calibrator class ? https://github.com/fizyr/dr_ensenso_node/blob/master/src/ensenso_calibrator.cpp
And probably also in our private Ensenso calibration server.

LGTM! Do not forget to update the ensenso_calibration.yaml file!


Need Support for .Net Core 3.1

Hi I'd like to test this tool, but I can't it run under Visual Studio 2019 and my .Net Core 3.1 project.

Hey, just came across this today. could you try to get it working with core yet and LinqPad6?

Hi installed today the extension in Version 2.0. But don't work under VS 2019 and LinqPad 6 on an .net Core 3.1 Project. I will retry on an .net Framework 4.6 Project next days

On starting the extension ask me for LinqPad path. Where can I insert this information. Is there a config file?

Hi, unfortunately this extension doesn't support .NET core yet. I should update the project as I haven't done it in a year. Last time I tried debugger visualizers didn't work with .net core.

https://docs.microsoft.com/en-us/visualstudio/debugger/create-custom-visualizers-of-data?view=vs-2022#special-debugger-side-considerations-for-net-50


refactor: manually track dials, instead of using iroh::dialer

Description
iroh::dialer is going away, this removes the usage
Breaking Changes

Notes & open questions

Change checklist

[ ] Self-review.
[ ] Documentation updates following the style guide, if relevant.
[ ] Tests if relevant.
[ ] All breaking changes documented.

This no longer de-duplicates concurrent dials to the same remote node.

true, need to investigate more if this was actually needed

looks like it is needed..

inlined a version of the dialer instead


Replacing in xml string extracted from _element.xml

Hi, I have a .docx file which has few template tags(django, jinja) which I want to use.  Unfortunately when I see the xml I see between the tags some junk is often seen. I got a regex by which I can filter out the junk.
Can you see the junk between {{ }}? The tags were put simply like  {{SOMETHING}} or {{SOMETHING|to_upper}} in the docx but when the xml is seen it inserts garbage between the tags.
This nice package https://docxtpl.readthedocs.io/en/latest/   also fails to handle such cases and gives error which is my next task to use after cleaning the junk.
Can someone help in how can I make replacements in document._element.xml(which is a class) and then save a docx file with that xml?
**`document._element.xml = remove_junk(document._element.xml) #gives AttributeError  can't assign`**

  <w:t>{{</w:t>
            </w:r>
            <w:proofErr w:type="spellStart"/>
            <w:r>
              <w:rPr>
                <w:rFonts w:ascii="Times New Roman" w:hAnsi="Times New Roman" w:cs="Times New Roman"/>
                <w:sz w:val="15"/>
                <w:szCs w:val="15"/>
                <w:lang w:val="en-IN" w:eastAsia="en-IN"/>
              </w:rPr>
              <w:t>HOL|to_words</w:t>
            </w:r>
            <w:proofErr w:type="spellEnd"/>
            <w:r>
              <w:rPr>
                <w:rFonts w:ascii="Times New Roman" w:hAnsi="Times New Roman" w:cs="Times New Roman"/>
                <w:sz w:val="15"/>
                <w:szCs w:val="15"/>
                <w:lang w:val="en-IN" w:eastAsia="en-IN"/>
              </w:rPr>
              <w:t>}}</w:t>

whereas the actual  text that I put in word was   {{HOL|to_words}}. I don't know how these big chunks of junks get inserted in between.
Code

def remove_junk(test_str):
    regex = r"{{(?:(?!}}).)*?<w:t>([^<]+)<\/w:t>(?:(?!}}).)*?}}"
    subst = "{{\\1}}"

    # You can manually specify the number of replacements by changing the 4th argument
    result = re.sub(regex, subst, test_str, 0, re.DOTALL)

    return result

for i in sys.argv[1:]:
    filename = os.path.splitext(Path(i).name)[0]
    document = Document(i)
    document._element.xml = remove_junk(document._element.xml) #gives AttributeError  can't assign

    document.save(os.path.splitext(i)[0]+"_new.docx")

I just studied that renaming the word file's extension to .zip solves the problem.


TestGetPodByNode is flakey

This test is sporadically failing (10% of the time maybe).
Caused by: feat(controller): opt-in to sending pod node events as pod (#6377)

@josephmcgovern-wf could you please take a look if you have time?

This test is flakey.

I've commit a change to skip this test.

@tczhao still seeing failures.

@alexec @tczhao This issue was for TestGetPodByNode but was closed for a fix to TestPodExists. TestGetPodByNode is still skipped so this should be reopened.


Some entities enabled by default don't have Modified On filter condition

Was looking at some of the entities enabled by default - looks like not all of them would have "last 10 days" filter condition in the organization data download filter (for example: accounts, phones calls are using "status equals active" condition instead. Contacts do have that last 10 days condition, though). Might be more of a preview issue, but still.

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

ID: 47bcdcf0-46c7-2e85-2dc2-1e9afe7220c1
Version Independent ID: a315aeaf-2034-d310-88ee-77dd760e24b7
Content: Configure mobile offline synchronization in Dynamics 365 for phones and tablets
Content Source: ce/mobile-app/setup-mobile-offline-for-admin.md
Service: crm-online
GitHub Login: @mduelae
Microsoft Alias: mkaur

@ashlega Thank you for the feedback.
Since this is product feedback and not an issue with the docs, I will close the issue here and forward your feedback to the PM Team.
Let me know if you have further questions.
@Kavishi FYI
Thanks,
Mints


Confirmation answer ignored on graphcool delete

When issuing the graphcool delete command the project is deleted regardless of the user's input.

I checked and the project was actually deleted. The same thing happens if I don't type anything.
Version info: graphcool/0.6.0-beta.3 (linux-x64) node-v8.6.0
Edit: Typing y cancel the deletion

The same thing happens if I don't type anything.

The same thing happens if I don't type anything.

what do you mean with this?

If I press enter without typing a response the default behavior should be to cancel the operation (as the message says). Right now the project is deleted anyway.

Maybe the text should be updates to 'Are you sure you want to keep cj....?' or 'Do you not want to delete cj....?' or 'Do you confirm that you don't want to stop keeping access to cj...?' :smiling_imp: :rofl:

This is fixed now, the code simply confused y with n.


[libiconv] build error

Host Environment

OS: Windows 10
package: libiconv[core]:x64-windows-release -> 1.16#12
vcpkg-tool version: 2022-03-09-1affd32f93b299d5a907816c328ca3ededb73a7e
vcpkg-scripts version: 6f827f25f 2022-03-17 (18 hours ago)

To Reproduce
Steps to reproduce the behavior:
building darknet with the following command:
./build.ps1 -UseVCPKG -InstallDARKNETthroughVCPKG -EnableOPENCV -EnableCUDA -EnableCUDNN -EnableOPENCV_CUDA -InstallDARKNETdependenciesThroughVCPKGManifest
Failure logs

the out.log file content:

Makefile:74: update target 'all' due to: iconv_no_i18n.exe iconv.obj iconv.res
test ls -ld . | sed -e 's/^d\(.........\).*/\1/' = rwxrwxrwx || chmod 777 .
Makefile:113: update target 'install' due to: all force
if [ ! -d /C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/tools/libiconv/bin ] ; then /bin/sh .././../src/1.16-b0c41d3469.clean/build-aux/mkinstalldirs /C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/tools/libiconv/bin ; fi
mkdir -p -- /C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/tools/libiconv/bin
case "mingw32" in
hpux*) compile cl.exe -LC:/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib -LC:/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib/manual-link -nologo -W3 -utf-8 -MP -MD -O2 -Oi -Gy -Z7 if test -n ':'; then : /C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/tools/libiconv/bin; fi iconv.obj ../srclib/libicrt.a -L/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib -liconv  iconv.res if test -n '/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release'; then echo " -Wl,+b -Wl,/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib"; fi -o iconv.exe;;
*) /bin/sh ../libtool --mode=link compile cl.exe -LC:/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib -LC:/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib/manual-link -nologo -W3 -utf-8 -MP -MD -O2 -Oi -Gy -Z7 if test -n ':'; then : /C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/tools/libiconv/bin; fi iconv.obj ../srclib/libicrt.a /C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib/libiconv.la  iconv.res -o iconv.exe;;
esac
libtool: link: compile cl.exe -nologo -W3 -utf-8 -MP -MD -O2 -Oi -Gy -Z7 iconv.obj ../srclib/libicrt.a iconv.res -o .libs/iconv.exe  -LC:/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib -LC:/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib/manual-link /C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib/iconv.lib -L=C:/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib -L=C:/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib/manual-link
cl : Command line warning D9002 : ignoring unknown option '//?/C:/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib/iconv.lib'
cl : Command line warning D9002 : ignoring unknown option '//?/C:/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib/iconv.lib'
cl : Command line warning D9002 : ignoring unknown option '//?/C:/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib/iconv.lib'
cl : Command line warning D9024 : unrecognized source file type '../srclib/libicrt.a', object file assumed
cl : Command line warning D9024 : unrecognized source file type '../srclib/libicrt.a', object file assumed
cl : Command line warning D9024 : unrecognized source file type '../srclib/libicrt.a', object file assumed
Creating library .libs\iconv.lib and object .libs\iconv.exp
LINK : warning LNK4217: symbol 'program_name' defined in 'libicrt.a(progname.obj)' is imported by 'iconv.obj' in function 'main'
iconv.obj : error LNK2019: unresolved external symbol libiconv_open referenced in function main
iconv.obj : error LNK2019: unresolved external symbol libiconv referenced in function convert
iconv.obj : error LNK2019: unresolved external symbol libiconv_close referenced in function main
iconv.obj : error LNK2019: unresolved external symbol libiconvctl referenced in function convert
iconv.obj : error LNK2019: unresolved external symbol libiconvlist referenced in function main
iconv.obj : error LNK2019: unresolved external symbol iconv_canonicalize referenced in function main
iconv.obj : error LNK2019: unresolved external symbol __imp__libiconv_version referenced in function print_version
.libs\iconv.exe : fatal error LNK1120: 7 unresolved externals

the error.log content:

libtool: warning: remember to run 'libtool --finish /C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib'
libtool: warning: remember to run 'libtool --finish /C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib'
libtool: warning: library '/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release/C/Users/tsaiyenk/Documents/SnobBog/ML/TensorFlowTraining/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib/libiconv.la' was moved.
libtool: warning: library '/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release/C/Users/tsaiyenk/Documents/SnobBog/ML/TensorFlowTraining/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib/libiconv.la' was moved.
cl : Command line warning D9002 : ignoring unknown option '//?/C:/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg/packages/libiconv_x64-windows-release/C/Users/[my path]/Yolov3TinyTrain/darknet-master/vcpkg_installed/x64-windows-release/lib/iconv.lib'
cl : Command line warning D9024 : unrecognized source file type '../srclib/libicrt.a', object file assumed
make[1]: *** [Makefile:114: install] Error 2
make: *** [Makefile:54: install] Error 2

Additional context
I read through issue #15592 and my issue is the same as the second half of that thread. However, i didn't see a new issue open for that problem as suggested, so I open another issue for it. Any help will be appreciated.

What's the file system of your C disk, ReFS or NTFS?

Could you please provide me with your triplet file x64-windows-release and the build.ps1 file? Thank you!

Probably a path length issue, cf. https://github.com/microsoft/vcpkg/issues/25269#issuecomment-1158687687.

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.


Set thread title

This helps understand thread usage with commands like top -H and ps -e -T
Before this change the thread name shows up as thread.rb:70. After, it may show up as whatever title was given. The exact behavior will depend on the implementation of Ruby's native_set_another_thread_name.

It looks like the Thread.name= method was introduced in Ruby 2.3.0, hence the build failures. Any suggestions on how to get that to work? We are investigating what is causing a high number of threads in fluentd, which affects the memory usage in GKE.
FWIW, Ruby 2.3.8 is already EOL and Travis doesn't seem to be testing against 2.3.x. When are you planning to stop supporting 2.1.10?

Any suggestions on how to get that to work?

How about thread.name = title.to_s if thread.respond_to?(:name) ?

Any suggestions on how to get that to work?

How about thread.name = title.to_s if thread.respond_to?(:name) ?

Cool, that seems to have worked! Thanks, I'm all new to Ruby.

Thanks!


[BUG] Operatorhub is not working correctly

General information

OS: Linux / macOS / Windows:  Linux (Fedora 31)
Hypervisor: KVM / Hyper-V / hyperkit: KVM
Did you run crc setup before starting it (Yes/No)?  Yes

CRC version
crc version: 1.4.0+d5bb3a3
OpenShift version: 4.2.13 (embedded in binary)

CRC status
CRC VM:          Running
OpenShift:       Running (v4.2.13)
Disk Usage:      11.65GB of 48.3GB (Inside the CRC VM)
Cache Usage:     26.98GB
Cache Directory: /home/croberts/.crc/cache

CRC config
- cpus                                  : 6
- memory                                : 16384

Host Operating System
NAME=Fedora
VERSION="31 (Workstation Edition)"
ID=fedora
VERSION_ID=31
VERSION_CODENAME=""
PLATFORM_ID="platform:f31"
PRETTY_NAME="Fedora 31 (Workstation Edition)"
ANSI_COLOR="0;34"
LOGO=fedora-logo-icon
CPE_NAME="cpe:/o:fedoraproject:fedora:31"
HOME_URL="https://fedoraproject.org/"
DOCUMENTATION_URL="https://docs.fedoraproject.org/en-US/fedora/f31/system-administrators-guide/"
SUPPORT_URL="https://fedoraproject.org/wiki/Communicating_and_getting_help"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=31
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=31
PRIVACY_POLICY_URL="https://fedoraproject.org/wiki/Legal:PrivacyPolicy"
VARIANT="Workstation Edition"
VARIANT_ID=workstation

Steps to reproduce

crc setup
crc start
login as kubeadmin
oc get opsrc -n openshift-marketplace

Expected
Should see no errors
Actual
oc get opsrc -n openshift-marketplace
NAME                  TYPE          ENDPOINT              REGISTRY              DISPLAYNAME           PUBLISHER   STATUS        MESSAGE                                                                                AGE
certified-operators   appregistry   https://quay.io/cnr   certified-operators   Certified Operators   Red Hat     Configuring   Get https://quay.io/cnr/api/v1/packages?namespace=certified-operators: context deadline exceeded   31d
community-operators   appregistry   https://quay.io/cnr   community-operators   Community Operators   Red Hat     Configuring   Get https://quay.io/cnr/api/v1/packages?namespace=community-operators: context deadline exceeded   31d
redhat-operators      appregistry   https://quay.io/cnr   redhat-operators      Red Hat Operators     Red Hat     Configuring   Get https://quay.io/cnr/api/v1/packages?namespace=redhat-operators: context deadline exceeded      31d

Are you running behind a proxy? Could you try with 1.5.0 which was released a few days ago?

Not behind a proxy.

Where can I find the 1.5.0 package?  Not seeing it on the "releases" on github.

should be accessible from https://cloud.redhat.com/openshift/install/crc/installer-provisioned

Seems to work in 1.5.0.  Thanks for steering me in the right direction.

Can the issue be closed?

Sure


Entra Connect Health: updates for versions, ADDS, cmdlets, endpoints

Add multiple changes for Entra Connect Health:

Updated version history
Updated information for the ADDS agent
Updated PowerShell cmdlets due to branding change
Additional required endpoints for CRL verification

@billmath
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Manifest updates

Updates the config.xml file so Cordova creates an AndroidManafest.xml file that better reflects the developer contact information, and bumps the maximum supported Android version (9.0 (SDK v28)).

I don't think bumping the version is a requirement, it should work on 9.0 with a max SDK version of 26, we just wouldn't be able to use any of the fancy new android features added in SDK versions 27 or 28.  Including it in this PR because I was asked too, and there are no ill effects to the best of my testing.


Question about type

In this function from the tutorial, i note you set the type with a symbol :string
What are the other types available, i try :bool but it does not work.
I am not fluent with database so sorry if it is a silly question.
function up()
  create_table(:books) do
    [
      primary_key()
      column(:title, :string)
      column(:author, :string)
    ]
  end
end

I found an answer in the source file https://github.com/essenciary/SearchLight.jl/blob/master/src/database_adapters/SQLiteDatabaseAdapter.jl

Sorry it is a mistake it works now

@pnavaro Sorry, missed this. Yes, it's a way to map Julia-like types to native DB types so that the migrations can work with multiple dbs (instead of writing db specific code).


W-11613205-http-connector-1.8-kt

Writer's Quality Checklist
Before merging your PR, did you:

[ ] Run spell checker
[ ] Run link checker to check for broken xrefs
[ ] Check for orphan files
[ ] Perform a local build and do a final visual check of your content, including checking for:

Broken images
Dead links
Correct rendering of partials if they are used in your content
Formatting issues, such as:

Misnumbered ordered lists (steps) or incorrectly nested unordered lists
Messed up tables
Proper indentation
Correct header levels

[ ] Receive final review and signoff from:

Technical SME
Product Manager
Editor or peer reviewer
Reporter, if this content is in response to a reported issue (internal or external feedback)

[ ] If applicable, verify that the software actually got released

Original draft https://github.com/mulesoft/docs-connectors/pull/1895/files

Closing PR as unverified commits https://github.com/mulesoft/docs-connectors/pull/2061


chore: bdd-spring-1578757641 to 0.0.1

chore: Promote bdd-spring-1578757641 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Estimation of the project size

This is fix for https://github.com/yegor256/takes/issues/149.

@davvd PR is ready for review.

@ikhvostenkov Many thanks, I will find someone to review it before we merge

@pinaf this pull request is for you, please review

@davvd PR is ready for review.

@ikhvostenkov thank you, I will check it

@davvd original ticket has been closed so far. Shall we merge this PR or?

@rultor merge

@rultor merge

@pinaf Thanks for your request. @yegor256 Please confirm this.

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 12min)

@ypshenychka review this ticket please, for compliance with our quality rules

@rultor deploy pls

@rultor deploy pls

@davvd OK, I'll try to deploy now. You can check the progress here

@rultor deploy pls

@davvd Done! FYI, the full log is here (took me 7min)

@ikhvostenkov
According to our QA Rules:

Pull request description explains the solution proposed and contains a link to the original ticket it is related to.

Please provide appropriate description about what was done in scope of PR.

@pinaf
According to our QA Rules:

The code reviewer found at least three problems in the code.

No issues were found during code review. Try to find at least three problems while future reviews.
Please confirm that everything is clear and you have no questions regarding this approach.

@ypshenychka Yes it's clear. Usually there are no issues found when reviewing these estimates :(

@pinaf Thank you, I see. But as per comment in PR #764, we should try to find some issues in this kind of PRd as well.

@ypshenychka fixed description.

@ikhvostenkov Great! Thank you.

@davvd Quality is acceptable.

@davvd Quality is acceptable.

@ypshenychka thank you, let's try to make it "good" next time

@pinaf I added 10 mins to @ypshenychka (for QA review) in transaction 59135526... 17 mins added to your account (payment number AP-39G09874EG069254X), many thanks for your contribution! 54 hours and 27 mins spent here.... review comments (c=2) added as a bonus... +17 to your rating, your total score is +6961


SS 3.2.1 - PDO does not escape query string

$titles = array( "Test1", "Test 2", "Test's 3" ); foreach( $titles as $title ) Debug::dump( Object::get()->where( " Title = '{$title}' " ) );
The code above with fail on Test's 3 because PDO does not automatically escape the string passed

Why would it? You're passing a string to a function, how does it know which apostrophies to escape and which not to?
You need to use the parameterized queries for this.
Object::get->filter('Title', $title);


Automated Package Upgrade

:tada: This PR is included in version 2.2.110 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update Issue - 21232876

default description
URL tested: http://ci.bsstag.com/welcome
Open URL on Browserstack

Property
Value

Browser
Chrome 77.0

Operating System
Windows 10

Resolution
1024x612

Screenshot Attached
Screenshot URL

Click here to reproduce the issue on Browserstack

Updated Screenshot Attached


Could not parse version: line is '// TypeScript Version: 2.6'

I'm trying to lint the React types in DefinitelyTyped, but I get:
❯ npm run lint react

><EMAIL_ADDRESS>lint /Users/OliverJAsh/Development/DefinitelyTyped
> dtslint types "react"

Error: Could not parse version: line is '// TypeScript Version: 2.6'
    at Object.parseTypeScriptVersionLine (/Users/OliverJAsh/Development/DefinitelyTyped/node_modules/definitelytyped-header-parser/index.js:180:15)
    at getTypeScriptVersion (/Users/OliverJAsh/Development/DefinitelyTyped/node_modules/dtslint/bin/index.js:90:44)
    at /Users/OliverJAsh/Development/DefinitelyTyped/node_modules/dtslint/bin/index.js:66:28
    at Generator.next (<anonymous>)
    at fulfilled (/Users/OliverJAsh/Development/DefinitelyTyped/node_modules/dtslint/bin/index.js:5:58)
    at <anonymous>

This is after a fresh install of Node modules.

Works for me with a fresh install -- are you sure you don't have a package-lock holding you back? Note the update would not be of dtslint but of https://github.com/Microsoft/definitelytyped-header-parser .

Ah, I hadn't cleared my yarn.lock. Apologies!


Small README.md Improvments

Just some small fixes, trying to stop #178 from happening again, and adding a link to the telegram setup.

Cheers!


Upgrade ajv dependency and fix failing tests.

Summary
Upgraded ajv dependency to newest version. Fixed failing tests. https://github.com/feathers-plus/feathers-hooks-common/issues/372
Problem

outdated
There's still problem with using newest Ajv and validateSchema hook in TypeScript, that I tried to fix, but couldn't figure how.
Version 6.0.0 of Ajv removed Thenable interface and replaced it by PromiseLike
Diff
Using validate schema with instance or constructor of ajv results in one of these errors:
Error:(63, 28) TS2345: Argument of type '{ (options?: Options): Ajv; new (options?: Options): Ajv; ValidationError: typeof ValidationError; MissingRefError: typeof MissingRefError; $dataMetaSchema: object; }' is not assignable to parameter of type 'AjvOrNewable'.
  Type '{ (options?: Options): Ajv; new (options?: Options): Ajv; ValidationError: typeof ValidationError; MissingRefError: typeof MissingRefError; $dataMetaSchema: object; }' is not assignable to type 'new (options?: Options) => Ajv'.

or
Error:(19, 10) TS2345: Argument of type '{ (options?: Options): Ajv; new (options?: Options): Ajv; ValidationError: typeof ValidationError; MissingRefError: typeof MissingRefError; $dataMetaSchema: object; }' is not assignable to parameter of type 'AjvOrNewable'.
  Type '{ (options?: Options): Ajv; new (options?: Options): Ajv; ValidationError: typeof ValidationError; MissingRefError: typeof MissingRefError; $dataMetaSchema: object; }' is not assignable to type 'new (options?: Options) => Ajv'.
    Types of parameters 'options' and 'options' are incompatible.
      Type 'import("/node_modules/feathers-hooks-common/node_modules/ajv/lib/ajv").Options' is not assignable to type 'import("/node_modules/ajv/lib/ajv").Options'.
        Types of property 'loadSchema' are incompatible.
          Type '(uri: string, cb?: (err: Error, schema: object) => void) => Thenable<boolean | object>' is not assignable to type '(uri: string, cb?: (err: Error, schema: object) => void) => PromiseLike<boolean | object>'.
            Type 'Thenable<boolean | object>' is not assignable to type 'PromiseLike<boolean | object>'.
              Types of property 'then' are incompatible.
                Type '<U>(onFulfilled?: (value: boolean | object) => U | Thenable<U>, onRejected?: (error: any) => U | Thenable<U>) => Thenable<U>' is not assignable to type '<TResult1 = boolean | object, TResult2 = never>(onfulfilled?: (value: boolean | object) => TResult1 | PromiseLike<TResult1>, onrejected?: (reason: any) => TResult2 | PromiseLike<TResult2>) => PromiseLike<...>'.
                  Types of parameters 'onFulfilled' and 'onfulfilled' are incompatible.
                    Type 'TResult1 | PromiseLike<TResult1>' is not assignable to type 'TResult2 | Thenable<TResult2>'.
                      Type 'TResult1' is not assignable to type 'TResult2 | Thenable<TResult2>'.
                        Type 'TResult1' is not assignable to type 'Thenable<TResult2>'.

Seems like the problem should be fixed after this pull request as well. feathers-hooks-common types were referenced from its own, older ajv dependency, whereas my application from newer version in my node_modules

Thank you!


Update README for VanillaFixes 1.5

I added VanillaMultiMonitorFix.dll to the default list in VanillaFixes 1.5, so no need to edit the file manually anymore.

I see, thanks for including it in there by default!


Short solution needed: "How to set image quality on save" (python-pillow)

Please help us write most modern and shortest code solution for this issue:
How to set image quality on save (technology: python-pillow)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to set JPG quality on save


slowThreshold  should depends on the timeout config

The problem
As for now ,  the slowThreshold is default to :
const (
	limitBodyBytes       = 1024
	defaultSlowThreshold = time.Millisecond * 500
)

var slowThreshold = syncx.ForAtomicDuration(defaultSlowThreshold)

However the timeout set is 3000ms,  and there are lots of logs in production.
My solution
In engine.go:
func (ng *engine) getLogHandler() func(http.Handler) http.Handler {
	if ng.conf.Verbose {
		return handler.DetailedLogHandler
	}

	return handler.LogHandler
}

Maybe we can add a  line of additional code 👍
func (ng *engine) getLogHandler() func(http.Handler) http.Handler {

        //  SetSlowThreshold here
        handler.SetSlowThreshold(time.Millisecond * time.Duration(ng.conf.Timeout))
	if ng.conf.Verbose {
		return handler.DetailedLogHandler
	}

	return handler.LogHandler
}

not necessary to set this line of code:
handler.SetSlowThreshold(time.Millisecond * time.Duration(ng.conf.Timeout))

it's timeout, will be logged.

not necessary to set this line of code:
handler.SetSlowThreshold(time.Millisecond * time.Duration(ng.conf.Timeout))

it's timeout, will be logged.

Ok. I can add handler.SetSlowThreshold(my custom slow call time ) in main file.


Hotfix v1.12.1 related to separate theme installation has broken "install theme" link

Current behavior
You get a 404 error when trying to access instruction for theme installation
Expected behavior
Access the information for theme installation
Steps to reproduce the issue

switch to branch "Hotfix v1.12.1"
navigate to /docs/guide/installation/linux-mac.md
scroll down to last paragraph "Install the vue-storefront"
click link "install theme"

You get a 404 error when trying to access instruction for theme installation

@kkdg can you take a look?

To me the link is still broken, here it is the URL missing:
https://github.com/DivanteLtd/vue-storefront/blob/hotfix/v1.12.1/guide/installation/theme.html


Error during sending logs on some error

During sending logs on some issue the client terminated and I was asked to create this bug here
entered message:
I cannot accept a user who follows me. The UI freezes and sometimes throws an error. I accepted this user as follower befor
e and am not sure why the status was changed back?
More feedback (or press <Enter> when done):

error message:
▶ ERROR Error reading logs: Journalctl exited with an error: exit status 1

my log id: df0a2873aaf1a56eb4fa831c
Note:
I recently started using firejail [1] and am wondering whether this issue might be related due to some required directory not being visible :thinking:
[1] https://github.com/netblue30/firejail

Update
After re-runnign firecfg (firejail config utility) the user state is now chaged to the correct one, but sending logs seems to be still broken due an permission issue Journalctl not having permissions to access the log files I think.


shouldComponentUpdate is skipped on change route, every component re-rendered

[x] I have searched the issues of this repository and believe that this is not a duplicate.

This is related to #88, but not quite the same. In this case, we are routing between the same page (/user.js), not different pages. Something weird is happening with the normal react lifecycle hooks for ALL page child components (all the way down the tree) that makes it impossible to prevent something from re-rendering.
Essentially, this means that on every route change (even if you are routing between the same page) every single component on the page is forced to re-render, and there is no way to stop it.
Expected Behavior
shouldComponentUpdate should be triggered as normal on the page component, and its child components.
Current Behavior
shouldComponentUpdate is skipped on the page component, and its child components.
Steps to Reproduce
I have created an example that reproduces the issue in an isolated environment. It includes instructions for running, and seeing what's going on. It also includes a link to an animated gif for you lazy folks out there :).
https://github.com/stoplightio/nextjs-router-bug
Context
We have complicated inner components and not being able to prevent them from re-rendering is a problem.

Hi @marbemac, sounds like this is the same bug I've reported a few months ago:
https://github.com/zeit/next.js/issues/2114

Ah didn't see that one, thanks @coluccini - closing this and commenting over there.


make docs-api-local fails for the utilities docs

When running make docs-api-local I get the following error when browsing the docs for the utilities
Error importing module aws_lambda_powertools.utilities:

Traceback (most recent call last):
  File "/Volumes/data/projects/aws-lambda-powertools-python/.venv/lib/python3.8/site-packages/pdoc/cli.py", line 234, in do_GET
    out = self.html()
  File "/Volumes/data/projects/aws-lambda-powertools-python/.venv/lib/python3.8/site-packages/pdoc/cli.py", line 266, in html
    return pdoc.html(self.import_path_from_req_url,
  File "/Volumes/data/projects/aws-lambda-powertools-python/.venv/lib/python3.8/site-packages/pdoc/__init__.py", line 173, in html
    mod = Module(import_module(module_name, reload=reload),
  File "/Volumes/data/projects/aws-lambda-powertools-python/.venv/lib/python3.8/site-packages/pdoc/__init__.py", line 234, in import_module
    importlib.reload(mod)
  File<EMAIL_ADDRESS>line 169, in reload
    _bootstrap._exec(spec, module)
  File "<frozen importlib._bootstrap>", line 604, in _exec
  File "<frozen importlib._bootstrap_external>", line 843, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/Volumes/data/projects/aws-lambda-powertools-python/aws_lambda_powertools/utilities/parser/models/apigw.py", line 46, in <module>
    class APIGatewayEventRequestContext(BaseModel):
  File "/Volumes/data/projects/aws-lambda-powertools-python/aws_lambda_powertools/utilities/parser/models/apigw.py", line 72, in APIGatewayEventRequestContext
    def check_message_id(cls, values):
  File "/Volumes/data/projects/aws-lambda-powertools-python/.venv/lib/python3.8/site-packages/pydantic/class_validators.py", line 119, in root_validator
    f_cls = _prepare_validator(_func, allow_reuse)
  File "/Volumes/data/projects/aws-lambda-powertools-python/.venv/lib/python3.8/site-packages/pydantic/class_validators.py", line 144, in _prepare_validator
    raise ConfigError(f'duplicate validator function "{ref}"; if this is intended, set `allow_reuse=True`')
pydantic.errors.ConfigError: duplicate validator function "aws_lambda_powertools.utilities.parser.models.apigw.APIGatewayEventRequestContext.check_message_id"; if this is intended, set `allow_reuse=True`

What were you trying to accomplish?
Local testing of the API Docs
Expected Behavior
No errors
Current Behavior
Errors
Possible Solution
Add allow_reuse=True as suggested. (Or some other better fix @risenberg-cyberark ?)
Steps to Reproduce (for bugs)

run make dev docs-api-local
visit http://localhost:8080/aws_lambda_powertools/utilities/

Environment

Powertools version used: 1.19.0
Packaging format (Layers, PyPi): Git
AWS Lambda function runtime: Python 3.8.11 locally
Debugging logs

@heitorlessa  @risenberg-cyberark I am not sure why this suddenly started, and I am sure my fix it not right. But it does at least render the docs

@heitorlessa i need some help confirming the fix for this.

I need time to dig into that — I tried running the API Docs locally on
develop branch and it worked for me. I’ll try creating a fresh virtual env
to see if it’s a dependency issue perhaps.
That error typically suggests we did something wrong in the code which is
why I’m puzzled
On Thu, 19 Aug 2021 at 06:11, Michael Brewer @.***>
wrote:

@heitorlessa https://github.com/heitorlessa i need some help confirming
the fix for this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-lambda-powertools-python/issues/611#issuecomment-901594663,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZPQBH63FMPKP7HECILITTT5SAAZANCNFSM5CGOKDBQ
.

@heitorlessa  - if it is not happening it might be my setup running Python 3.9.6 on MacArm. :P


RHIDP-2376 Document which plugins are recommended for Prod Readiness

@kim-tsao WRT you comment about closing this PR, do we not now need to add the Production Readiness note after the dynamic plugins table in the Cnfiguring Plugins guide?

@kim-tsao WRT your comment ^^^ about closing this PR, do we not now need to add the Production Readiness note after the dynamic plugins table in the Configuring Plugins guide?

@Gerry-Forde Please remove the note in the dynamic plugins table as well.  In an effort to fix the wording, I decided that the content in the Audit Logging and RBAC sections in the "New Features" section of the RN will suffice for now.

@kim-tsao WRT your comment ^^^ about closing this PR, do we not now need to add the Production Readiness note after the dynamic plugins table in the Configuring Plugins guide?

@Gerry-Forde I may have responded in haste, seeing how this PR contains much more content than the Prod Readiness note.  Please remove any mention of Prod Readiness for plugins as I stated in my previous comment


run test.py error:(Pdb)

When I run test.py, it doesn't work.
In the terminal displays as follows,and test.py is running status.

/home/FasterRCNN/faster-rcnn.pytorch/test_net.py(251)()
-> for i in range(num_images):
(Pdb)

If I stop test.py, the terminal displays:"Process finished with exit code 137 (interrupted by signal 9: SIGKILL)".
Thanks for your help!

This is just a debugger. Remove/comment pdb.set_trace() or breakpoint() line.


ForeignKey relationships aren't being credited, no errors thrown.

What version of Go are you using (go version)?
1.12.9
Which database and its version are you using?
Mysql 8.0.13.
I've tried every approach to a foreign key relation in the docs and none of them work. The tables get created without errors but no foreign key relationships get created.
I am wanting a "has one" relationship.
type Volumes struct {
	RowId            int64  `gorm:"type:bigint(20) unsigned auto_increment;primary_key"`
	Id                    string  `gorm:"type:varchar(40);unique;not null"`
	VirtualMachineID VirtualMachines `gorm:"foreignkey:VMID"`
	CreatedAt        time.Time
	DeletedAt        time.Time
	UpdatedAt      time.Time
}

type VirtualMachines struct {
	RowId       int64  `gorm:"type:bigint(20) unsigned auto_increment;primary_key"`
	Id          string `gorm:"type:varchar(40);unique;not null"`
	VMID        string `gorm:"type:varchar(40)"`
	CreatedAt   time.Time
	DeletedAt   time.Time
	UpdatedAt   time.Time
}

The "foreignkey" field specifies in which field the primary key of the referenced row is being stored in the same struct:
type Volumes struct {
	VirtualMachine VirtualMachines `gorm:"foreignkey:VirtualMachineID"`
	VirtualMachineID string
}

Also, it doesn't create a foreign key constraint in the database at all, that must unfortunately be done manually during setup:
mydb.Model(&Volumes{}).AddForeignKey("virtual_machine_id", "virtual_machines(vmid)", "cascade", "cascade")

IMO, this is documented pretty poorly, and foreign keys should be created automatically in the database, but it seems to work if you figured that out. :)

So you can only create a foreign key agains the primary key of the referenced table?

That actually works, and is called an Association ForeignKey, you just have to use gorm:"association_foreignkey:Id" - also see http://gorm.io/docs/belongs_to.html#Association-ForeignKey.
The problem is that everything you described exists in Gorm, somewhere, somehow. It's IMO just that the tag names are not exactly intuitive, and the documentation is all over the place, and just not exhaustive enough for what it can do. The best thing is to just read & understand it completely, and to check the code for the rest. :neutral_face:

I think I figured out my model. It gets created in mysql correctly. (I manually create my foreign keys). I need to right code to query some data and see if it's working as intended. Probably gonna wait till Monday and salvage my weekend.


terminal opens to root of multiroot project of file initially opened

VSCode Version: Code 1.18.1 (929bacba01ef658b873545e26034d1a8067445e9, 2017-11-16T18:32:36.026Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author (truncated)
Version

better-toml
bun
0.3.1

tslint
eg2
1.0.24

debugger-for-chrome
msj
3.5.0

peep
nwa
0.0.5

vscode-docker
Pet
0.0.22

perforce
sle
3.0.0

autoimport
ste
1.5.3

Seemingly random start location when first encountered this.   Figured out that the cwd is the root of the  the current file that was opened when the terminal was first opened - but it remembers that cwd even when closing the terminal (hide or X) and opening while editing a different file in a different root)
To make it even more confusing, Vscode also remembers between executions - if the file that it is re-opened to is not actually in the workspace (ie, the user settings editor), it will use the previous run's cwd.
setting "" in settings terminal.integrated.cwd results in the same behavior
setting the name of the root desired to be the cwd actually causes opening the terminal to hang (ie "terminal.integrated.cwd": "files" for a root called files)
workaround is to specify full path to desired cwd in settings.  That isn't a good solution, since it cannot be set in the workspace file, since local project location might not be in same path for all developers
preferred feature could be something along the lines of:
new setting: "terminal.integrated.preferredRoot": ""
new setting: "terminal.integrated.newRoot": "ask" | "usePreferred" | "useFile" - for pressing the '+' to ask vs using the above setting, vs using the current file's root.
could also just make the first root the 'preferredRoot'

Steps to Reproduce:

open multi root workspace
open terminal
open file in a root other than the terminal's current root / cwd
close vscode.  open vscode
note terminal's cwd is path to root of 3's file
open file in a root other than the terminal's current root / cwd
close / open vscode
note cwd is path to root of 6's file
open user settings
close / open vscode
note cwd is path to root of 6's file

set user setting: "terminal.integrated.cwd" to one of the names of the root paths
close/open vscode, open terminal if it wasn't already
note broken terminal window

Reproduces without extensions: Yes

I think you're opening the terminal using ctrl+` which actually toggles the panel and focuses the terminal, creating a terminal in the "default" root in the process. The default root here is defined as the first root in the explorer list.
I suggest you use ctrl+shift+` instead which will allow you to select the root:

I think you may have missed the finer details of the bug report.   It would be fine / expected for it to always create the initial terminal in the default / first root.
Actual behavior is some remembered root, or a frozen console, in certain situations.
Specifically, the "... root of the the current file that was opened when the terminal was first opened"

One of the things we didn't want to emphasize was the concept of a "default root", preferring the "last active root" instead. We wanted to move in this direction because there is no UI affordance that indicates the active/default root and it's not clear that it would be the first. For the terminal we only opt for the first root when no files have been opened in the workspace yet, this is the case because we want the terminal panel to open instantly when toggled.
As for the frozen/broken terminal window, I couldn't seem to reproduce this. Does is happen every time you do these steps?

set user setting: "terminal.integrated.cwd" to one of the names of the root paths
close/open vscode, open terminal if it wasn't already
note broken terminal window


Can't publish module to local repository since v2.1.3

Steps to reproduce
Register local repository:
$repo = @{
Name='Test'
SourceLocation='D:\Test'
PublishLocation='D:\Test'
InstallationPolicy='trusted'
}
Register-PSRepository @repo

Try to publish any module to local repository:
Publish-Module -Repository Test -Verbose -Name Pester

Expected behavior
Module is published successfully on PowerShellGet v2.1.2

Actual behavior
Publish-Module hangs indefinitely on PowerShellGet v2.1.3

Environment data

> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      5.1.14393.2879
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.2879
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

> Get-Module -ListAvailable PowerShellGet,PackageManagement

    Directory: C:\Program Files\WindowsPowerShell\Modules

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Script     1.3.2      PackageManagement                   {Find-Package, Get-Package, Get-PackageProvider, Get-PackageSource...}
Script     2.1.2      PowerShellGet                       {Find-Command, Find-DSCResource, Find-Module, Find-RoleCapability...}

> Get-PackageProvider -ListAvailable

Name                     Version          DynamicOptions
----                     -------          --------------
msi                      <IP_ADDRESS>          AdditionalArguments
msu                      <IP_ADDRESS>
nuget                    <IP_ADDRESS>
NuGet                    <IP_ADDRESS>          Destination, ExcludeVersion, Scope, SkipDependencies, Headers, FilterOnTag, Contains, AllowPrereleaseVer... PowerShellGet            <IP_ADDRESS>          PackageManagementProvider, Type, Scope, AllowClobber, SkipPublisherCheck, InstallUpdate, NoPathUpdate, A...
Programs                 <IP_ADDRESS>          IncludeWindowsInstaller, IncludeSystemComponent

That's quite odd. If you explicitly run the previous build (import-module powershellget -requiredversion ...) does the issue reproduce?

Yep, it does reproduce. I did also check it on second machine with the same result.

OK, it's weird. I've tried to publish other popular module - Plaster. And it works with both builds. So I did little test and reinstalled latest Pester module from PSGallery (4.8.1) but it didn't help and I can't publish it with v2.1.3.

Thanks! I can reproduce the issue. Nuget.exe is not terminating when we run it to generate the package. So far I'm not sure why, the same command-line completes just fine when run interactively.

The issue is, when building some packages nuget writes enough data to standard out to fill the buffer, then blocks. But the calling script doesn't read anything from the output until it nuget has exited. It only affects some packages because different packages produce different amounts of output from nuget/dotnet.

2.1.3 appears to be installed in Azure Pipelines hosted agents, so I am running into this there. Maybe you can drop a word to whoever maintains that that this fix could be welcome?

@edyoung nice finding. How to work around it?
Having my module publishing stuck at "VERBOSE: Performing the operation "Publish-Module" on target ....", found this thread and how scratching my head.

The issue is, when building some packages nuget writes enough data to standard out to fill the buffer, then blocks. But the calling script doesn't read anything from the output until it nuget has exited. It only affects some packages because different packages produce different amounts of output from nuget/dotnet.

In my case, I have sub-folders for my customer PowerShell module.
NuGet stuck, indeed. The dotnet.exe process stuck in the process explorer. Once killed, here is what we see in the verbose output.
Trying to find a way to trick nuget/dotnet. Placing everything into /tools sub-folder didn't work.
Now I wonder how come other
ERROR: Failed to generate the compressed file for module 'dotnet cli failed to pack M
icrosoft (R) Build Engine version 16.4.0+e901037fe for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.

  Restore completed in 275.36 ms for C:\Users\user\AppData\Local\Temp\f44
2cac3-564d-4afe-b84d-6cdc6a36d902\Temp.csproj.
  Temp -> C:\Users\user\AppData\Local\Temp\f442cac3-564d-4afe-b84d-6cdc6a
36d902\bin\Debug\netcoreapp2.0\NotUsed.dll
  Successfully created package 'C:\Users\user\AppData\Local\Temp\84405364
2\InvokeMyModule\InvokeMyModule.0.2.2002.1129.nupkg'.
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tas
ks.Pack.targets(198,5): warning NU5110: The script file 'InvokeMyModule.ps1' is outside
the 'tools' folder and hence will not be executed during installation of this package
. Move it into the 'tools' folder. [C:\Users\user\AppData\Local\Temp\f442
cac3-564d-4afe-b84d-6cdc6a36d902\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tas
ks.Pack.targets(198,5): warning NU5111: The script file 'InvokeMyModule.ps1' is not reco
gnized by NuGet and hence will not be executed during installation of this package. R
ename it to install.ps1, uninstall.ps1 or init.ps1 and place it directly under 'tools
'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-4afe-b84d-6cdc6a36d902
\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tas
ks.Pack.targets(198,5): warning NU5111: The script file 'tools\deploy.ps1' is not rec
ognized by NuGet and hence will not be executed during installation of this package.
Rename it to install.ps1, uninstall.ps1 or init.ps1 and place it directly under 'tool
s'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-4afe-b84d-6cdc6a36d90
2\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tas
ks.Pack.targets(198,5): warning NU5111: The script file 'tools\components\CloudCompon
ent.ps1' is not recognized by NuGet and hence will not be executed during installatio
n of this package. Rename it to install.ps1, uninstall.ps1 or init.ps1 and place it d
irectly under 'tools'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-4a
fe-b84d-6cdc6a36d902\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tas
ks.Pack.targets(198,5): warning NU5111: The script file 'tools\core\BaseClass.ps1' is
 not recognized by NuGet and hence will not be executed during installation of this p
ackage. Rename it to install.ps1, uninstall.ps1 or init.ps1 and place it directly und
er 'tools'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-4afe-b84d-6cd
c6a36d902\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tas
ks.Pack.targets(198,5): warning NU5111: The script file 'tools\core\CliUtils.ps1' is
not recognized by NuGet and hence will not be executed during installation of this pa
ckage. Rename it to install.ps1, uninstall.ps1 or init.ps1 and place it directly unde
r 'tools'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-4afe-b84d-6cdc
6a36d902\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tas
ks.Pack.targets(198,5): warning NU5111: The script file 'tools\core\DeploymentContext
.ps1' is not recognized by NuGet and hence will not be executed during installation o
f this package. Rename it to install.ps1, uninstall.ps1 or init.ps1 and place it dire
ctly under 'tools'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-4afe-
b84d-6cdc6a36d902\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Tas
ks.Pack.targets(198,5): warning NU5111: The script file 'tools\core\DeploymentStage.p
s1' is not recognized by NuGet and hence will not be executed during installation of
this package. Rename it to install.ps1, uninstall.ps1 or init.ps1 and place it direct
ly under 'tools'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-4afe-b8
4d-6cdc6a36d902\Temp.csproj]
'.
At C:\program files\powershell\6\Modules\PowerShellGet\PSModule.psm1:10810 char:17
+ ...             Publish-PSArtifactUtility @PublishPSArtifactUtility_Param ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
At C:\Users\user\source\repos\my-module\platform-MyModule-cli\.build-sha
red.ps1:157 char:1
+ task BuildModule {
+ ~~~~~~~~~~~~~~~~~~
At C:\Users\user\source\repos\my-module\platform-MyModule-cli\.build-sha
red.ps1:285 char:1
+ task DefaultBuild UnitTest,
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
At C:\Users\user\source\repos\my-module\platform-MyModule-cli\.build-sha
red.ps1:294 char:1
+ task . DefaultBuild
+ ~~~~~~~~~~~~~~~~~~~
Build FAILED. 8 tasks, 1 errors, 0 warnings 00:00:33.0177652
pwsh : Publish-PSArtifactUtility : Failed to generate the compressed file for
module 'dotnet cli failed to pack Microsoft (R) Build Engine version
16.4.0+e901037fe for .NET Core
At line:6 char:1
+ pwsh -c invoke-build
+ ~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (Publish-PSArtif...e for .NET Core:Stri
   ng) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

Copyright (C) Microsoft Corporation. All rights reserved.
  Restore completed in 275.36 ms for C:\Users\user\AppData\Local\Temp\f4
42cac3-564d-4afe-b84d-6cdc6a36d902\Temp.csproj.
  Temp -> C:\Users\user\AppData\Local\Temp\f442cac3-564d-4afe-b84d-6cdc6
a36d902\bin\Debug\netcoreapp2.0\NotUsed.dll
  Successfully created package 'C:\Users\user\AppData\Local\Temp\8440536
42\InvokeMyModule\InvokeMyModule.0.2.2002.1129.nupkg'.
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Ta
sks.Pack.targets(198,5): warning NU5110: The script file 'InvokeMyModule.ps1' is
outside the 'tools' folder and hence will not be executed during installation of
this package.
 Move it into the 'tools' folder. [C:\Users\user\AppData\Local\Temp\f442
cac3-564d-4afe-b84d-6cdc6a36d902\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Ta
sks.Pack.targets(198,5): warning NU5111: The script file 'InvokeMyModule.ps1' is not
recognized by NuGet and hence will not be executed during installation of this
package. Re
name it to install.ps1, uninstall.ps1 or init.ps1 and place it directly under
'tools'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-4afe-b84d-6cdc6
a36d902\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Ta
sks.Pack.targets(198,5): warning NU5111: The script file 'tools\deploy.ps1' is not
recognized by NuGet and hence will not be executed during installation of this
package. R
ename it to install.ps1, uninstall.ps1 or init.ps1 and place it directly under
'tools'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-4afe-b84d-6cdc6
a36d902\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Ta
sks.Pack.targets(198,5): warning NU5111: The script file
'tools\components\CloudComponent.ps1' is not recognized by NuGet and hence will not
be executed during installation
 of this package. Rename it to install.ps1, uninstall.ps1 or init.ps1 and place it
directly under 'tools'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-
4afe-b84d-6cdc6a36d902\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Ta
sks.Pack.targets(198,5): warning NU5111: The script file 'tools\core\BaseClass.ps1'
is not recognized by NuGet and hence will not be executed during installation of
this pa
ckage. Rename it to install.ps1, uninstall.ps1 or init.ps1 and place it directly
under 'tools'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-4afe-b84d
-6cdc6a36d902\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Ta
sks.Pack.targets(198,5): warning NU5111: The script file 'tools\core\CliUtils.ps1'
is not recognized by NuGet and hence will not be executed during installation of
this pac
kage. Rename it to install.ps1, uninstall.ps1 or init.ps1 and place it directly
under 'tools'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-4afe-b84d
-6cdc6a36d902\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Ta
sks.Pack.targets(198,5): warning NU5111: The script file
'tools\core\DeploymentContext.ps1' is not recognized by NuGet and hence will not be
executed during installation of
 this package. Rename it to install.ps1, uninstall.ps1 or init.ps1 and place it
directly under 'tools'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-
4afe-b84d-6cdc6a36d902\Temp.csproj]
C:\Program Files\dotnet\sdk\3.1.101\Sdks\NuGet.Build.Tasks.Pack\build\NuGet.Build.Ta
sks.Pack.targets(198,5): warning NU5111: The script file
'tools\core\DeploymentStage.ps1' is not recognized by NuGet and hence will not be
executed during installation of t
his package. Rename it to install.ps1, uninstall.ps1 or init.ps1 and place it
directly under 'tools'. [C:\Users\user\AppData\Local\Temp\f442cac3-564d-
4afe-b84d-6cdc6a36d902\Temp.csproj]
'.
At C:\program files\powershell\6\Modules\PowerShellGet\PSModule.psm1:10810 char:17
+ ...             Publish-PSArtifactUtility @PublishPSArtifactUtility_Param ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : InvalidOperation: (:) [Write-Error], WriteErrorException
+ FullyQualifiedErrorId : FailedToCreateCompressedModule,Publish-PSArtifactUtility

@edyoung I might be wrong but maybe we might need to add something like for the PR https://github.com/PowerShell/PowerShellGet/pull/479 while generating $CsprojContent?
NU5110;NU5111
https://github.com/PowerShell/PowerShellGet/pull/479/files#diff-8a5643a86665a53bd2054068d6f33fe5R60
Feels that we might need to suppress output warnings by dotnet pack for some warning. These might be exactly the warning which go to the output and locking dotnet.exe.

Same issue under linux container, azuresdk/azure-powershell-core with dotnet-sdk 3.1


forc test not recognizing tests in contracts

From @bitzoic

Create a new toolchain fuelup toolchain new v0.46
Install Forc fuelup component add<EMAIL_ADDRESS>Install Fuel-core fuelup component add<EMAIL_ADDRESS>Create a new folder mkdir forc_test
Init a new project forc init
Add a test
Run forc test

contract;

abi MyContract {
    fn test_function() -> bool;
}

impl MyContract for Contract {
    fn test_function() -> bool {
        true
    }
}

#[test]
fn my_test() {
    assert(false);
}

This SHOULD fail. Yet I get that no tests were run

Seems like compiler is no longer preserving test entries after d2dbacbce27e2906e11825d23865a4752b79bdd0f5006cd2a7415ecbdec192b3 only for contracts.

My guess is the following:
While compiling contracts forc-test compiles without tests fist, maybe cache is used while compiling it with tests. Since the file does not change the cache is going to be hit always for tests included version although we want to compile it with tests.
We can check if current path is pointing to a contract and check tests included parameter, if that is the case we can omit the cache.

This is happening because module cache is only using paths as key. But same source file (same path) compiled with and without tests yields different AST and thus needs different entries in the module cache.


Rename nnx.compat to nnx.bridge

To avoid any future confusion of compat vs compact.

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

Project coverage is 0.00%. Comparing base (31adb00) to head (3085fc9).
Report is 78 commits behind head on main.

Files
Patch %
Lines

flax/nnx/__init__.py
0.00%
3 Missing :warning:

Additional details and impacted files
@@          Coverage Diff           @@
##            main   #4066    +/-   ##
======================================
  Coverage   0.00%   0.00%
======================================
  Files        106     107     +1
  Lines      13582   13825   +243
======================================
- Misses     13582   13825   +243

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


fix/certificate

fixed an issue on macOS when downloading vdjdb data where ssl certificate was not available -- hence changed default certificate to unverified on macOS

I'm not sure if this is such a good idea, as this will disable certificate validation for the entire python session; and the certificate validation is there for a reason.
Also it doesn't seem to be a Mac problem per se, as it can be downloaded without issues on the MacOS CI.


Should we only allow DRep delegations to existing DReps?

We do this for stake pools, so it would be consistent.

Relevant ticket on the ledger implementation side: https://github.com/IntersectMBO/cardano-ledger/issues/4598


Add Eduardo Diaz to contributors list

My first Pull request :)

@edjroz, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?

@rohanjossey it was a great tutorial to get me started, already tried helping another project in my country. I've been trying to find ways to start contributing for some time so it was great for me, thanks, hope to keep using what i learned.
I believe a more advanced optional tutorial could be useful for places in which contributing could be more conplex.

@edjroz, I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://firstcontributions.herokuapp.com


Dragged and resized elements don't return to their original position correctly when using jQuery in the project.

This is caused by:
https://github.com/mattlewis92/angular-bootstrap-calendar/blob/master/src/directives/mwlDraggable.js#L111
https://github.com/mattlewis92/angular-bootstrap-calendar/blob/master/src/directives/mwlResizable.js#L109
To blank/remove a css property it needs to be assigned to the empty string not to null.
From http://api.jquery.com/css/:

Setting the value of a style property to an empty string — e.g. $( "#mydiv" ).css( "color", "" ) — removes that property from an element if it has already been directly applied, whether in the HTML style attribute, through jQuery's .css() method, or through direct DOM manipulation of the style property.

I will submit a pull request to fix the issue.

Thanks for finding this. I was struggling with the same issue, but couldn't find the solution.

Great, another undocumented difference between jqlite and jquery. Thanks for the fix :)


time_remaining

I’ve just installed this integration, and can see sensors for my 2-drawer Fisher & Paykel dishwasher.
For some reason the “time_remaining” attribute for both the upper and lower drawers is set to 4:03, despite the fact that the machine has been idle for a few hours (the last cycle completed about 10 hours ago). A restart of HA made no difference.
I then ran a wash cycle, and the time remaining counted down from 4:00 as expected, but when the cycle ended the time remaining reset itself to 4:03.
I then power cycled the dishwasher, and once again the time remaining for both drawers was reset to 4:03

Might be something with the API, the integration is generally just pulling information from the smarthq servers.  Not sure if it can really be changed, but perhaps if you explore the raw values in the sdk debugging, you might find something.  Closing for now, but feel free to open an issue on the sdk repo if you see something promising.


add dockerfile

add docker file for easy use.

Coverage decreased (-0.5%) to 4.974% when pulling b958c66ef825530d175cc61468ec4e4e83bc78c4 on songbinliu:docker into 7a6f5c0d8761cf4a99a879b3ae8db2551172c5c2 on turbonomic:master.

Coverage increased (+2.009%) to 7.461% when pulling 18750b4c671d61725ead7d5e791d1d29f36351ec on songbinliu:docker into 7a6f5c0d8761cf4a99a879b3ae8db2551172c5c2 on turbonomic:master.

Coverage increased (+2.009%) to 7.461% when pulling 269e0039b513bd04dfef92a4851a07498b289fab on songbinliu:docker into 7a6f5c0d8761cf4a99a879b3ae8db2551172c5c2 on turbonomic:master.


Screen protection over "recent apps" and "screenshot"

Purchased foods are datas that can describing a person habits and its health status. For example I cannot eat dairy products. I don't want reveiling some informations about my lifestyle or stomach issues to other prying apps or to advertising Ai.
Some apps give the ability (optional) to switch ON/OFF the "screenshot" and "recent apps" protection feature by other prying apps.
Basically a privacy feature.
Here an implementation and some examples.

Hey @lorenzovngl, I want to work on this feature.

Enjoy 😊


QueryPicker and form need Sources and Collection information

now in the QueryPicker. Just have to set up the fieldArray and 'chips' for the arrays

fetch the collection info if we have collection items in the selected query.
-for demo sample queries
-for demo keyword searches
-for demo new keywords BUG

fixed - now loads for new keywords


Preparing small patch release (Big Endian fix).

What does this PR do?

Fixes # (issue) or description of the problem this PR solves.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.