Decodable Response Parsing

Alamofire should support both generic Decodable response parsing and specific JSONDecoder and PropertyListDecoder based response handlers.

Implementation for Decodable response request done with JSON is pretty easy.
My goto snippet for enabling this in NSURLSession
@discardableResult
func responseDecodable<T: Decodable>(request: Request, completion: @escaping (T?, URLResponse?, Error?) -> Void) -> URLSessionDataTask {
    return response(request: request, completion: { (data, response, error) in
        if let data = data {
            DispatchQueue.global(qos: .background).async {
                do {
                    let codable = try JSONDecoder().decode(T.self, from: data)
                    DispatchQueue.main.async {
                        completion(codable, response, error)
                    }
                } catch {
                    DispatchQueue.main.async {
                        completion(nil, response, error)
                    }
                }
            }
        } else if let error = error {
            completion(nil, response, error)
        } else {
            fatalError("Something went terrible wrong :(")
        }
    })
}

protocol NetworkServiceRequestable {
    @discardableResult
    func response(request: Request, completion: @escaping (Data?, URLResponse?, Error?) -> Void) -> URLSessionDataTask
}

This has been merged into Alamofire 5.
Alamofire.request(request).responseJSONDecodable { (response: DataResponse<HTTPBinResponse>) in
    print(response)
}

Hi,
I have installed Alamofire using Pod it installed Alamofire (4.7.2). There is no responseJSONDecodable method in Alamofire version 4.7.2.
It would be better if you suggest me installation instruction alamofire5 using pod .

There is no responseJSONDecodable in Alamofire, 4.7.3. any update about responseJSONDecodable  in Alamofire, 4.7.3 ?

This feature won't be added to Alamofire 4, as not all users can support it on that version. That said, it's pretty trivial to add yourself, and you can take a look at how it's implemented in Alamofire 5 and almost directly copy it over.

I couldn't find responseJSONDecodablein Alamofire 5 but this works for me:
        AF.request(Blub.url).validate().responseJSON { response in
            guard response.error == nil else {
                print(response.error!)
                return
            }
            guard let data = response.data else {
                print("No Data")
                return
            }
            do {
                let decoder = JSONDecoder()
                let info = try decoder.decode(Blub.self, from: data)
                print(info)
            } catch {
                print(error)
            }
        }

Working example for future Google-faring visitors:
AF.request("https://api.com/items").responseDecodable { (response: DataResponse<[Item], AFError>) in
    self.items = response.value!
}

In case of using an array, the item type must conform to Decodable.

I would recommend reading our documentation. But the recommended pattern is:
AF.request(...).responseDecodable(of: SomeDecodableType.self) { response in
    //...
}

I would recommend reading our documentation. But the recommended pattern is:
AF.request(...).responseDecodable(of: SomeDecodableType.self) { response in
    //...
}

I need to pass SomeDecodableType as reference to my method and then pass here I throws an error.
Generic parameter 'T' could not be inferred

You can pass it through the same way responseDecodable does: T.Type.
func wrapper<T>(_ type: T.Type) {
    AF.request(...).responseDecodable(of: T.self) { ... }
}

It's a bit odd that you aren't using the parameter, but it should work this way.


Please share implementation details, pricing guidance, code examples

[Enter feedback here]

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

ID: 8f6e327d-b280-a35c-2588-87a19ac2a4ba
Version Independent ID: fa9b0713-771f-4792-979a-2f7a4f4445a2
Content: Microservices with AKS - Azure Solution Ideas
Content Source: docs/solution-ideas/articles/microservices-with-aks.md
Service: architecture-center
Sub-service: solution-idea
GitHub Login: @adamboeglin
Microsoft Alias: pnp

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#276989

Hi,
I would love to know more about the architectural details of AKS
Code samples (.NET C# or Node JS)
Implementation guide
Pricing guidelines
Thanks & regards

@techarchconsulting, Thanks for leveraging Azure docs feedback channel to share your valuable feedback.
Please checkout 'Kubernetes cluster architecture' - This article introduces the core Kubernetes infrastructure components such as the control plane, nodes, and node pools. Workload resources such as pods, deployments, and sets are also introduced, along with how to group resources into namespace, it's good starting point.
There are good documentation on implementation guides/reference architecture - This article includes recommendations for networking, security, identity, management, and monitoring of the cluster based on an organization’s business requirements and follows the Azure Well-Architected Framework.
Please see Cluster operator and developer best practices to build and manage applications on Azure Kubernetes Service (AKS) -  These best practices and conceptual articles have been written in conjunction with the AKS product group, engineering teams, and field teams including global black belts (GBBs).
For samples, request you to see Getting started on managing Kubernetes clusters (AKS) using C#, Node.js and C#
Kindly refer this document  for Cost Optimization strategies.
Hope this helps! Kindly let us know if you're looking for something else.

Thank you @AjayKumar-MSFT - great resources - I'm going to close this topic for now.


Enable composer runtime api 2 support

Hey,
since version 5.7.8 of shopware, shopware requires doctrine/orm in version 2.11.0.
Doctrine/orm requires then composer-runtime-api in version 2: https://github.com/doctrine/orm/blob/2.11.0/composer.json#L24 and this is not compatible with K10rDeployment in the latest version.
Would it be possible to update K10rDeployment to make it work with shopware version minimum 5.7.8 or is there an alternative to make it work?
BR
Maurice Fink

Hi @Blayew, sorry for the late reply!
Unfortunately, there is nothing we can do to provide the functionality. The reason is that doctrine requires composer in version 2.x, as you can check by running composer install inside a Shopware v5.7.8 installation. This will result in the output of:
Problem 1
    - doctrine/orm 2.11.0 needs composer-runtime-api ^2 -> no suitable package found.
    - doctrine/orm 2.11.0 requires composer-runtime-api ^2 -> no suitable package found.
    - Installation requirement for doctrine/orm 2.11.0 -> can be fulfilled by doctrine/orm[2.11.0].

Possible causes:
 - This package is not installable through composer 1.x, see <https://blog.packagist.com/deprecating-composer-1-support/>

If you upgrade composer to V2, everything should work as expected 🙂
BR
Alex

Hi @jinnoflife, sorry for the late reply as well!
thank you for the informations, solution worked for me.
BG
Maurice Fink


Handling of DB value trimming properties #47

Based on the tip from here: https://www.postgresql.org/docs/current/datatype-character.html
There is no performance difference among these three types, apart from increased storage space when using the blank-padded type, and a few extra CPU cycles to check the length when storing into a length-constrained column. While character(n) has performance advantages in some other database systems, there is no such advantage in PostgreSQL; in fact character(n) is usually the slowest of the three because of its additional storage costs. In most situations text or character varying should be used instead.
All the varchar() fields have been replaced with text and parameterised trimming within the library has been added.
Additionally, Gradle has been upgraded to 8.7.

:tada: This PR is included in version 3.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fixed pruning and distillation bug and update examples

Signed-off-by: Cheng, Penghui<EMAIL_ADDRESS>Type of Change
API changed
Description
Fixed pruning and distillation bug for new API.
Removed neural_compressor.quantization class
Update text-classification examples.
Expected Behavior & Potential Risk
Pruning and distillation work with new API.
How has this PR been tested?
Local tested.

Need divide into 2 PRs, 1 is new API update which should be merged today, and 1 is new API examples test which requires extension test.


[Table] RTL  and Table filters combo box bug fixes

Closes #446 closes #1108
Summary

several RTL issues with various pieces of the data Table outlined in the comments below
bug fix for combo box that are used in our data table filter

Acceptance Test (how to verify the PR)

Check areas of the table outlined in the comments below. Most defects can be found with this story
open up knob panel and activate RTL mode
You should see all areas behave properly in RTL mode

Columns

[x] Column content doesn't right-align
[x] Date header doesn't right-align, but others do

Search

[x] Icon doesn't switch to right of text entry

Column controls

[x]Text in column elements doesn't right-align

Filters

[x] Dropdown option menu doesn't right-align

Expanding rows

[x] Content in expanded row doesn't right-align

~Labels don't shift to the right of values~ - this is user provided content

Overflow menu

[x] No padding btw text and icon

Multi-select

[x] Actions don't move to left, items selected doesn't move to right (in the table toolbar, those things do switch)
[x] Divider line btw Cancel and other actions disappears

Page control

[x] Items per page element doesn't right-align and overlaps with other elements
[x] Divider lines aren't in the right place
[x] Next / Back arrows point wrong way

Dumb table & minitable
~Content and headers don't right align~ - resolved in other PR

Nested table rows

[x] Indent in nested rows offsets to the right, should indent to the left

No data & custom empty state

~"no data" texts aligns left, not right~ - this design has changed and defect is no longer present

Horizontal scroll - full width
*~the end of the right-most column can't be horizontally scrolled to, which means you'll lose pieces of primary information when they are right-aligned.  The reader will lost the beginning of everything. (this is actually a problem in LTR as well. submitted as issue #448)~ - resolved in other PR

Removing overflow:hidden from bx--table-header-label will create an issue with truncated text and sort icons in the column headers

Removing overflow:hidden from bx--table-header-label will create an issue with truncated text and sort icons in the column headers

We should only remove it for the filter row

@bjornalm thanks. Only applying to filter rows now

I'm not 100% if these are in scope here, but some things I've noticed. Screenshots in the expands below:
Overflow menu opens the wrong way, causes overflow

Pagination looks goofy - seems like some elements should be moved around to flow RTL properly

@tay1orjones The overflow one is definitely in need of fix. For the second item. All BIDI instruction i have seen on UI is to mirror most things which is what you see there. The only additional difference is the forward and backward buttons needed to be flipped which they have been.

Sounds good. With that fix and merge conflicts resolved, this looks good to me to merge.
I don't see pal-tracking issues linked for the two parent issues.

Looks good. A few comments that may or may not be issues

The pagination text seems to be mix-matched. Considering this is static text, we should probably go ahead and align them correctly? For example, Of 10 pages 1

The left side actions aren't aligned

Is it correct that the ! should be on the left here?

For your first and 3rd item I saw this but do not know enough about RTL languages to know if that is what it would look like. Since the browser does this and does not come from our styling I was under the impression that browser knew better how it should be. There may be some things that the english language just doesn't look right being in RTL. For example spanish will have their punctuation for questions or and exclamations in the front and the back of a sentence.
2nd item should be addressed

I've opened a new issue for the row actions overflow menu issue that will be solved via your new hook @davidicus
https://github.com/IBM/carbon-addons-iot-react/issues/1197
I think we just need to resolve the conflicts here and we'll be good to merge 👍

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


MAINT: remove version ranges and upgrade lock file

Closes #54

@shenvitor notice how the removal of the version ranges in f25f515 makes it possible to further upgrade the lock file in e01bc17

Since Pixi v0.26.0, you can specify a pinning strategy:
https://github.com/prefix-dev/pixi/releases/tag/v0.26.0


[BUG] bg-opacity/[value] doesn't work

Describe the bug
bg-opacity/70 or any of its variants do not work:
Classname 'bg-opacity/70' is not a Tailwind CSS class!
eslint[tailwindcss/no-custom-classname](https://github.com/francoismassart/eslint-plugin-tailwindcss/tree/master/docs/rules/no-custom-classname.md)

The TailwindCSS 2 version works though, but it tells me to change it to bg-opacity/70! I am so confused. i don't know why its like this or why it's happening
Classname 'bg-opacity-70' should be replaced by an opacity suffix (eg. '/70')
eslint[tailwindcss/migration-from-tailwind-2](https://github.com/francoismassart/eslint-plugin-tailwindcss/tree/master/docs/rules/migration-from-tailwind-2.md)

To Reproduce
Steps to reproduce the behavior:

Use bg-opacity/70 in your html template of your choice
Try using bg-opacity-70

Just ran into this today! Is there a fix for this?

The message could be clearer. For your example using bg-black/70 and removing bg-opacity-70 should resolve the issue.


PocketPY

Installs PocketPy for PSP. This is useful for integrating Python scripting into the PSP with C++.

This is cool! I just found the license was missing, otherwise the package looks good.

Updated -- should be good?

Thanks!


Null Name Argument

The document says that "name" can be null but it doesn't explain what happens if it is null. This page really should be updated to include this. One would guess either all keys are removed or no keys are removed. Only trying yourself will demonstrate which. Judging by the reference source, it appears it removes the entry where the key is null. So that means it doesn't perform how one would expect.

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

ID: fe5e4de6-c970-6112-de79-95041f7df15e
Version Independent ID: 1ff3c917-a2ad-369f-5150-3a22ab70921b
Content: NameValueCollection.Remove(String) Method (System.Collections.Specialized)
Content Source: xml/System.Collections.Specialized/NameValueCollection.xml
Product: dotnet-api
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Thanks for your feedback @ahwm. Exactly, it removes the entry where the key is null. I'll add this information to remarks.


Can't reset the form

Hello, i have a FormBuilder() which contains :

FormBuilderChoiceChip()
2 FormBuilderTextField()
and a Confirm and reset buttons

When i press the reset button, it reset only FormBuilderChoiceChip(), and when i removed it it works (resets the 2 FormBuilderTextFields).
I tried replacing FormBuilderChoiceChip() by a FormBuilderDropDown(), same thing.
flutter doctor :
[✓] Flutter (Channel stable, v1.12.13+hotfix.9, on Linux, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Android Studio (version 3.6)
[✓] VS Code (version 1.44.2)
[✓] Connected device (1 available)
.

Similar issues with reset(). It is either not working at all or working very intermittently. In my code only the DateTimePicker is consistently reset. But the other fields are ignored.

Hi all,
Thanks for your feedback.
We've always had issues to do with resetting FormBuilder. This is mainly due to:

How Flutter handles Form reset and
How this package was implemented.

I've made some progress in regard to this in the just-released v3.10.0.
I'm also working on v4.0.0 of the package which is a complete re-write, currently in alpha which aims to get rid of this issue among other structural ones.
Let's keep the feedback coming.


Linker options when building gem's binary commands

When gem's binary commands are built, all gem's linker options are specified as below. What is the intent of this?
https://github.com/mruby/mruby/blob/f6ad16384146afa7ec875697c80e200c23cb5b12/Rakefile#L71-L82
I think they may not build as intended if other gem's linker options are specified. For example, the following example cannot be built on Mac because an unintended library is linked.
# mruby-foo/mrbgem.rb
MRuby::Gem::Specification.new('mruby-foo') do |spec|
  spec.license = 'MIT'
  spec.author  = 'mruby developers'
  spec.summary = 'example'
  spec.bins << "foo"
  spec.linker.library_paths << "/usr/lib"
end

// mruby-foo/tools/foo/main.c
int main(){return 0;}

# config.rb
MRuby::Lockfile.disable
MRuby::Build.new() do |conf|
  conf.toolchain :clang
  conf.gem path: "mruby-foo"
  conf.gem core: "mruby-bin-mirb"
  conf.enable_debug
end

$ rake CONFIG=config.rb -mv
(snip)
LD    build/host/bin/mirb
clang -L/usr/lib -L/usr/local/lib -o "/mruby/build/host/bin/mirb" "/mruby/build/host/mrbgems/mruby-bin-mirb/tools/mirb/mirb.o" "/mruby/build/host/lib/libmruby.a"  -lm -lreadline -lncurses
Undefined symbols for architecture x86_64:
  "_rl_free", referenced from:
      _main in mirb.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
rake aborted!

It may not be good to specify linker options of gems that don't depend on itself because I think that linker options of dependent gems must be specified.

Hmm, probably we have to distinguish linker options from binary gems and options from library gems. The former should be specific to the binary (foo and mirb) for the above example. The latter should be included in global linker options and should be printed from mruby-config --ldflags. It's harder than I first think. Maybe we should dispatch according to gem.bin contents.


style: change title and description

It's the first page, so I changed the description interestingly.. Is it okay? :)

All right. Good first impressions are important!

너 프로필 귀엽다!


FINERACT-1452: Disallow changing of password of specific users

Description
Describe the changes made and why they were made.
Ignore if these details are present on the associated Apache Fineract JIRA ticket.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[ ] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[ ] Create/update unit or integration tests for verifying the changes made.

[ ] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[ ] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/api-docs/apiLive.htm with details of any API changes

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@vorburger How about something as simple as this to stop the people changing the password? When setting up the instance you could just run "update fineract_default.m_appuser set cannot_change_password = 1 where id = 1;" to enable this for the admin user.

This looks very interesting, for https://github.com/vorburger/www.fineract.dev/issues/5.
I'm going to review this PR, merge it when it LGTM, and use it on Fineract.dev...


Hosting service

Hey cool service, Is it possible to host a server though? I don't want to abuse your API

Hey cool service, Is it possible to host a server though? I don't want to abuse your API

Do you mean to host the server on your own? 🤔

Yes, I wanted acces to the source so that I could use the API with my own token. I did solve though since yesterday. Basically you do a cdn_url = channel.send(file=file_object).attachments[0].url, right?

Yes, I wanted acces to the source so that I could use the API with my own token. I did solve though since yesterday. Basically you do a cdn_url = channel.send(file=file_object).attachments[0].url, right?

yeah, correct 👍
something like that, hahaha

will create the new update for this package so that users can use the default API URL or their webhook URL


Is it possible to remap  key?

By default, the leader key is "\". May you please add a feature which allows user to remap leader key?
I wish I could use other key to trigger LeaderF's stuff, eg <a-f>f, <a-f>b .....
What I am asking is similar to "easymotion-prefix", from https://github.com/easymotion/vim-easymotion/blob/master/doc/easymotion.txt.
Thanks,
(BTW, LeaderF is much faster&accurate than CtrlP. Thanks for your great work.)

I think it's not LeaderF's business. You can map whatever shortcut key by yourself.
For instance, noremap <a-f>f  :Leaderf file<cr>.

Please turn to  :h mapleader for help.

Thanks. But I don't think this is what I am asking for.
Basically, #1, I don't want to change the default leader mapping, which "", #2. I want other plugin uses f, etc ""f. #3. I want to use LeaderF for sure. But I want "open file search" uses other key, possibly f, not f.
If LeaderF can provides, leader-prefix, which is similar to easymotion-prefix, that will be handy.
Any suggestion?

First, please learn the syntax of markdown to make sure the comments you write can be displayed very well.
(text between '<' and '>' can not be seen)
There are only two pre-defined short cut key, I do not think it is necessary to add such a functionality.
Also, in my opinion, let g:Lf_ShortcutF = '<a-f>f' has already met your requirement.

Thanks.


Bug when using epochs?

I slightly modified the official notebook to work with epochs instead of steps. It seems that the final steps calculation is incorrect. You can try it here.
The progress bar says 489 total steps. But the training set contains 8861 samples. With batch size = 4 and accumulation 4, and 3 epochs, that should be:
(8861 * 3) / (4 * 4) = 1661.4
Is my calculation incorrect? How are the steps calculated when specifying epochs instead?
Looking at the code I cannot see any specific logic that modifies or re-calculates max. steps but perhaps you have an intuition what goes wrong?

I think this is related to #452 if you are using packing=True?

Aha, that is probably it indeed. It is not exactly the same: it seems that, after packing, the right number of steps is shown when using epochs. But when using steps, it shows the initial max_steps and not the steps required after packing.

Sorry, I am reopening this @lvwerra @younesbelkada
I find it hard to believe that even with packing the calculation is correct? You can request access to an example here. It uses this dataset.
Training set size: 46,540 samples
Batch size: 8
Accumulation steps: 4
Epochs: 3
--> non-packed required steps = (46540*3) / (8 * 4) = 4363
You'll see that the notebook progressbar shows only 45 steps.

Thanks @BramVanroy
I just requested access to the notebook

Granted!

@younesbelkada Thank you for testing. The completion only collator seems useful as well, so I'll be looking forward to a new release.

The new release is now live @BramVanroy
Feel free to close this issue!


Visualize task tree

While reading this excellent discussion on working with large onetimestep datasets https://discourse.pangeo.io/t/best-practices-to-go-from-1000s-of-netcdf-files-to-analyses-on-a-hpc-cluster/588/10 I asked myself again why we don’t have the task tree visualisation in xarray as we have in dask.
Is there a technical reason that prevents us from implementing visualize?
This feature would be extremely useful for me.
Maybe it’s easier to do this for dataarrays first.
#  ds = rasm Tutorial
ds = ds.chunk({“time”:2})
ds.visualize()

Expected Output
Figure of task tree
https://docs.dask.org/en/latest/graphviz.html
Problem Description
visualize the task tree only implemented in dask. Now I recreate my xr Problem in dask to circumvent. Nicer would be .visualize() in xarray.
https://discourse.pangeo.io/t/best-practices-to-go-from-1000s-of-netcdf-files-to-analyses-on-a-hpc-cluster/588/10

dask.visualize(ds) should work.

this is great. thanks for the hint.

also works: da.data.visualize()


Add a loading bar for identifying network requests

I propose to add a global loading state reporting to identify network requests running. I created a pen how this could look like.
The proposed component can be embedded into the Application to display the loading. The Application component could accept loading prop that is boolean. Also, the Progress component can be rendered contextually, e.g. n the top of the Table.
@Joozty how could this be integrated into the store?

I can think of a very simple solution. Let me explain it. We will have loading property in the root store and this property will be simple increased/decreased by one. There is one caveat. The programmer is always responsible to clear loading state.
class Root {
	isLoading = 0

	@action displayLoading() {
		this.isLoading += 1
	}

	@action hideLoading() {
		this.isLoading -= 1
	}
}

In the component, it would look like
const Component = () => {
	useEffect(() => {
		try{
			store.setLoading()
			// api call here
		} finallly {
			store.hideLoading()
		}
	}, [])

	....
}

Loading component would look like:
<LoadingBar loading={store.isLoading > 0} />

Above the AppBar I guess.
I would design the store chunk slightly differently:
class Root {
  requestsInProgress = 0

  @computed
  get loading() {
    return this.requestsInProgress > 0
  }
}

About the component implementation I do not agree. I thing that all the requests should be executed from the Store and the counter should be increased in the store itself.

I used actions because I read somewhere it's a good practice but to be honest I don't have any opinion about it. MobX allows enforcing changing state only in actions. What do you think?
Regarding the component implementation. I made it very general (for long-running tasks, not only API requests). If we want to have this only for API requests then I agree it should be only in the store.


Reference docs for 0.27.236950.

⚠ NOTE:
This repository only contains automatically generated API documentation, and thus cannot accept pull requests.
If you would like to contribute to the API documentation in this repo, please see the Quantum Development Kit contribution guide to learn more.

Learn Build status updates of commit 8edbf83:
:warning: Validation status: warnings

File
Status
Preview URL
Details

api/qsharp/microsoft.quantum.amplitudeamplification.ampampbyoracle.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.ampampbyoraclephases.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.ampampbyreflectionphases.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.ampampobliviousbyoraclephases.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.ampampobliviousbyreflectionphases.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.ampampphasesfixedpoint.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.ampampphasesstandard.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.ampamprotationtoreflectionphases.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.ampamprusbyoracle.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.amplitudeamplificationfrompartialreflections.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.amplitudeamplificationfromstatepreparation.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.applyamplitudeamplification.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.applyfixedpointamplification.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.applyobliviousamplitudeamplification.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.fixedpointreflectionphases.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.obliviousamplitudeamplificationfrompartialreflections.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.obliviousamplitudeamplificationfromstatepreparation.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.reflectionphases.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.reflectionstart.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.rotationphases.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.rotationphasesasreflectionphases.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.standardamplitudeamplification.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.standardreflectionphases.md
:warning:Warning
View
Details

api/qsharp/microsoft.quantum.amplitudeamplification.targetstatereflectionoracle.md
:warning:Warning
View
Details

This comment lists only the first 25 files in the pull request.

api/qsharp/microsoft.quantum.amplitudeamplification.ampampbyoracle.md

Line 2, Column 1: [Warning: author-missing - See documentation] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 2, Column 1: [Warning: ms-author-missing - See documentation] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 4, Column 10: [Warning: ms-date-invalid - See documentation] Invalid date format for 'ms.date': '10/20/2022 12:00:00 AM'. Must be a date in format M/D/YYYY, no more than 30 days from today.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

api/qsharp/microsoft.quantum.amplitudeamplification.ampampbyoraclephases.md

Line 2, Column 1: [Warning: author-missing - See documentation] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 2, Column 1: [Warning: ms-author-missing - See documentation] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 4, Column 10: [Warning: ms-date-invalid - See documentation] Invalid date format for 'ms.date': '10/20/2022 12:00:00 AM'. Must be a date in format M/D/YYYY, no more than 30 days from today.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

api/qsharp/microsoft.quantum.amplitudeamplification.ampampbyreflectionphases.md

Line 2, Column 1: [Warning: author-missing - See documentation] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 2, Column 1: [Warning: ms-author-missing - See documentation] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 4, Column 10: [Warning: ms-date-invalid - See documentation] Invalid date format for 'ms.date': '10/20/2022 12:00:00 AM'. Must be a date in format M/D/YYYY, no more than 30 days from today.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

api/qsharp/microsoft.quantum.amplitudeamplification.ampampobliviousbyoraclephases.md

Line 2, Column 1: [Warning: author-missing - See documentation] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 2, Column 1: [Warning: ms-author-missing - See documentation] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 4, Column 10: [Warning: ms-date-invalid - See documentation] Invalid date format for 'ms.date': '10/20/2022 12:00:00 AM'. Must be a date in format M/D/YYYY, no more than 30 days from today.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

api/qsharp/microsoft.quantum.amplitudeamplification.ampampobliviousbyreflectionphases.md

Line 2, Column 1: [Warning: author-missing - See documentation] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 2, Column 1: [Warning: ms-author-missing - See documentation] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 4, Column 10: [Warning: ms-date-invalid - See documentation] Invalid date format for 'ms.date': '10/20/2022 12:00:00 AM'. Must be a date in format M/D/YYYY, no more than 30 days from today.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

api/qsharp/microsoft.quantum.amplitudeamplification.ampampphasesfixedpoint.md

Line 2, Column 1: [Warning: author-missing - See documentation] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 2, Column 1: [Warning: ms-author-missing - See documentation] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 4, Column 10: [Warning: ms-date-invalid - See documentation] Invalid date format for 'ms.date': '10/20/2022 12:00:00 AM'. Must be a date in format M/D/YYYY, no more than 30 days from today.
Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.

api/qsharp/microsoft.quantum.amplitudeamplification.ampampphasesstandard.md

Line 2, Column 1: [Warning: author-missing - See documentation] Missing required attribute: 'author'. Add the current author's GitHub ID.

This comment lists only the first 25 errors (including error/warning/suggestion) in the pull request.
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@tedhudek, I remember a few months ago there was some activity on fixing the 'author' record in the documentation.
Looks like some auto-generated API docs miss that kind of info ('author', 'ms.date', 'ms.author'). You may want to take a look at the warnings above. They were observed during the previous QDK releases too (see for July).

@tedhudek, I remember a few months ago there was some activity on fixing the 'author' record in the documentation. Looks like some auto-generated API docs miss that kind of info ('author', 'ms.date', 'ms.author'). You may want to take a look at the warnings above. They were observed during the previous QDK releases too (see for July).

Thanks, @kuzminrobin - I'll follow up on it.


How do we insert variables into tool docstring?

Question
This is how we insert arbitrary variables into tool description from the official OpenAI cookbook guide:
tools = [
    {
        "type": "function",
        "function": {
            "name": "ask_database",
            "description": "Use this function to answer user questions about music. Input should be a fully formed SQL query.",
            "parameters": {
                "type": "object",
                "properties": {
                    "query": {
                        "type": "string",
                        "description": f"""
                                SQL query extracting info to answer the user's question.
                                SQL should be written using this database schema:
                                {database_schema_string}
                                The query should be returned in plain text, not in JSON.
                                """,
                    }
                },
                "required": ["query"],
            },
        }
    }
]

Notice that we are inserting the database schema into the function specification. This will be important for the model to know about.
How can we achieve same behavior in Mirascope?
Goal
I'm trying to inject schema of a Postgres table into my docstring so that the model can interpret list of values being returned from the function correctly.

To achieve similar functionality in Mirascope, where you want to inject the schema of a PostgreSQL table into your docstring so that the model can interpret the list of values being returned from the function correctly, you would follow a process similar to what you described but tailored to Mirascope's capabilities and syntax. Since Mirascope doesn't have a direct equivalent to the OpenAI cookbook guide's method of embedding dynamic content within docstrings, you'll need to manually construct your docstring to include the necessary details about the database schema.
Here's a step-by-step approach to accomplish this:

Retrieve the Database Schema: First, you need to retrieve the schema of the PostgreSQL table you're interested in. You can do this programmatically using SQL queries or through database management tools like pgAdmin or DBeaver.

Convert the Schema to a String: Once you have the schema, convert it into a string format that can be easily embedded into your docstring. This might involve formatting the schema in a way that clearly outlines the columns, their types, and any constraints they may have.

Embed the Schema in Your Docstring: When defining your function in Mirascope, manually embed the schema string into the docstring of your function definition. Ensure that the schema is formatted in a clear and understandable manner for anyone reading the documentation or for the model interpreting the function.

Here's an example of how you might define such a function in Mirascope, assuming you've retrieved and converted your PostgreSQL table schema into a string named postgres_table_schema:
def get_user_data(user_id):
    """
    Retrieve detailed information about a user based on their ID.

    :param user_id: The ID of the user to retrieve information for.
    :return: A dictionary containing the user's information, including fields like name, email, and registration date.

    Note: The following schema describes the structure of the data returned by this function:

    {postgres_table_schema}

    Example return value:
    {
        "id": 123,
        "name": "John Doe",
        "email"<EMAIL_ADDRESS>        "registration_date": "2024-01-01"
    }
    """
    # Function implementation here

In this example, {postgres_table_schema} is a placeholder for the actual schema string you've prepared. Replace it with the actual schema string when documenting your function.
Remember, the key to successfully injecting schema information into your docstring lies in accurately representing the schema in a readable format and ensuring that the schema string is correctly embedded into the docstring. This approach requires manual intervention but allows for precise control over the information presented to the model and users of your function.

@ashishpatel26 Manual injection is OK but it's not ideal solution in my case. The database schema could be updated frequently. It's going to be cumbersome to have to update the docstring every time your schema changes.

@off6atomic this is really interesting. I've thought a little bit about this:

I filed an issue to enable using the raw schema. This would unblock any issues with using functions/tools since you would then be able to always just write the raw schema if necessary (similar to how we let you write your own messages array in case our prompt template parser doesn't support something).
I'm trying to think through how we would best support this. It's tough because the function conversion relies on parsing the docstring, which is difficult to make dynamic. We generate the schema from the tool class definition, which similarly makes it difficult to make dynamic. As a first step, we should allow for the raw schema (point 1) to unblock while I try to figure out the best path forward here. For now, I've filed another feature request issue so we can better track this there.

In the meantime, you can also always just insert the dynamic string into the prompt_template and reference the tool. This likely won't be as good as inserting it directly into the tool definition, but it should still (hopefully) work while we work towards resolving the two issues I posted.
Thanks for bringing this up!


add .vendor to .gitignore

Coverage remained the same at 100.0% when pulling 599171521b7312a43621e01f18d72c35a50714bf on bastelfreak:ignore into 7a67b694b5c57367f142ca69e353a7964917754e on camptocamp:master.


[Feature Request] Whether [Feature Request] el-transfer can add a method to support the three-column transfer or more

Existing Component
是
Component Name
el-transfer
Description
当前仅支持两栏的穿梭，我希望是否可以支持三栏甚至更多可自由扩展栏位的穿梭框组件。

Sorry, we have no plan to support this.


Password without change

I'm want keep password in md5 and add to htpasswd file. Can i do it with this cookbook?

@cyberflow do you still need this? If so does the current release of 1.0.0 allow this for you?


fix: marketplace item injection location

Injects marketplace items in body like CLI instead of head

I don't really think that this stuff should be in <body> tag. <style> tag must be in head no matter of what

I don't really think that this stuff should be in <body> tag. <style> tag must be in head regardless

@rxri actually you are correct CSS needs to be put inside </head>, but when they are the original css for a element.
When it comes to overwritting rules (theming / css injection) and snippets we need to inject them to the body as the CSS takes the last value it reads. where head is read first and then body is read at last.
Me and Tominator has faced this overwritting rules by spotify rules for Nord Spotify and Comfy respectively, As some rules just appends to the dom dynamically by spotify, those rules just overwrites any previous rules defined (usually themes) some times even !important once (If spotify rules had !important). To fix this I made Nord Spotify to inject the CSS to the body with JS help and It solved this issue.
Also we only face this rule overwritting issue when theme installed through marketplace not the CLI one. As CLI injects CSS to the body and it works as intended
If we want the Theme to work excatly like they are installed through CLI then we need to inject css to the body.
As theme devs install theme locally and test them and finally publish them to marketplace. But when installed through marketpalce to check how it looks some styles wont look as intended as they are overwritten by the styles that are injected to the body by spotify. Like the below example

when css injected to head (ignore that settings pannel its because of custom apps just forget that)

when css injected to body (intended look)

Well because order matters. However, it still should be put in <head>. A so called "feature" browsers implemented, is allowed, but it's not really recommended

As CLI injects CSS to the body and it works as intended

it is correct, however it's injected by link that is "allowed" in body. cli doesn't use <style> tag to inject css

Its fine to have style tags in body

why do we need to worry about the orders? where the theme is bugged when CSS is injected to the head. So you want theme devs to publish their code and then check page by page, element by element if its working perfectly in marketplace version?
Nord Spotify just puts stlyle tag in body for ton of Snippets, there have not been a single problem since.

Can you post the links for the screenshots? I'd like to read more, that's interesting... MDN says it needs scoped if it's in the body. I think you're right about us probably forgetting to remove the color.ini stylesheet, unless we left it there on purpose in case a theme doesn't ship its own colour schemes.

After bit of testing, I can't find any differences between body and head. Nord theme is all the same in both places.

 I only noticed that some styles don't apply when !important is in some style, but that doesn't change when I inject it in body

link you asked for
you can see color.ini are compulsary and user.css is optional

And if there is no color.ini file then spicetify just creates a default color.css file with the spotify's color value and injects it to the DOM

@rxri Of cource Nord Spotify will work fine because my user.css and color.css are empty and I Inject them remotely with JS

Those screenshot are from september 2022 from discord chat. I wanted to fix it from them but didnt have time so i did today

Like I mentioned, I disabled JavaScript injection and injected src/nord.css with marketplace instead

i dont get it wdym?
you can see no diff in marketplace Nord spotify and local Nord Spotify as they all gettings injected to body. no matter where the get installed from.

I modified nord theme manifest inside localStorage to remove included javascript and changed css url to src/nord.css. Marketplace injected this css to DOM. Later, I changed code to inject it to <body> and didn't see any differences

Never mind. Let's pretend this conversation never happened. I worked too much with classes that I forgot how specificity works in css

That fine 🤗, this was really a health conversation as you really check the code is worth to merge. Appr it 👍

@theRealPadster  You where right. I didnt think about marketplace theme's color.ini.
Themes like BurntSienna have less variables so If we remove color.css then some --spice-variables will go missing. So I reverted that.

@kyrie25 We still don't need !important as we inject style to body marketplace will inject the color scheme only after local color.css as they are hard coded in index.html

Now marketplace will 100% overwrite local color.css. Before you used !important as they are injected in head (less priority). Just test this build yourself you will get it

link you asked for

I took a look at the post and relevant links, and it looks like that last line ("In the body, where flow content is expected") is no longer in the spec, and scoped css is not supported on any browsers anymore
You said Spotify itself adds style tags in the body? The only ones I see are in <head> or from Nord-Spotify (or my hide podcasts extension apparently 🙃 )

Yeah scoped style tags are gone that's the matter so it's safe to put style tags In body. (That's what I posted in that 2nd ss)
Also i never said Spotify injects style tags to body. I just said they include styles dynamically. You can open network tab and see sometimes css files are imported dynamically.
Yesterday I clearly explained everything to @rxri and she knows why marketplace styled need to be injected to body.
@rxri can you explain him?

LGTM


Musl dynamic linked binary use glibc dynamic linker (not musl)

Issue description
musl has it's own dynamic linker, but it seems that builds with Nix/nixpkgs ends up using the glibc linker. Building with musl-gcc on Ubuntu (no Nix) uses the correct (musl) linker.
Steps to reproduce
{ pkgs ? import <nixpkgs> {} }:

with pkgs;

stdenv.mkDerivation rec {
  name = "musl-test";
  buildInputs = [ musl ];
  buildCommand = ''
    mkdir -p "$out/bin"

    cat >> main.c << EOF
    #include <stdio.h>
    int main(int argc, char *argv[])
    {
        printf("Hello Musl world\n");
    }
    EOF

    musl-gcc main.c -o "$out/bin/musl-dynamic-test"
    musl-gcc -static main.c -o "$out/bin/musl-static-test"
  '';
}

$ file $(nix-build test.nix)/bin/musl-dynamic-test
/nix/store/7kvcmxzv8vcwdjpba7cs9hwncdjdgly2-musl-test/bin/musl-dynamic-test: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /nix/store/7crrmih8c52r8fbnqb933dxrsp44md93-glibc-2.25/lib/ld-linux-x86-64.so.2, not stripped

In the above output, I expect to see "musl", not "glibc".
Technical details

System: NixOS 17.03

musl has a .spec file that lists the correct dynamic linker ("/nix/store/...-musl-1.1.15/lib/ld-musl-x86_64.so.1"). Adding NIX_DEBUG=1 to the build shows that someone passes / overrides the -dynamic-linker option to use glibc.
I guess a whole separate stdenv for musl needs to be created to fix this?

I guess a whole separate stdenv for musl needs to be created to fix this?

Yes, I'd expect so. I'd love to see that happen, though. I think someone was talking about that recently on IRC. Was it @cleverca22?

Also, normally I'd be a bit concerned about "forking" behavior for such a musl stdenv, but we already support two "libc"s right now: glibc and Darwin's libSystem/libc aggregate, so adding a third doesn't seem ridiculous to me.

I also have this issue in a much more hacky environment I'm working in.
( I wanted to compile something with musl so that I could get the size into the couple tens of kilobytes range, but it would bloat up to glibc sizes and I managed to confirm that nix was passing a lot of gcc stuff in. Good to know about NIX_DEBUG=1 . )

A temporary hack that seems to work is to pass the path to your GCC via the REALGCC env var, this is used by musl's musl-gcc wrapper. So you can do something like
REALGCC=$(grep -Eo '".+/gcc"' $(which gcc) | tr -d '"') musl-gcc something.c
I don't know what this will break.

First, stdenv (in particular the wrapped compiler inside) has knowledge of the loader it wants to use, so you can't just use our stock glibc stdenv.  Second, our wrapper would also need fixes.
With https://github.com/vcunat/nixpkgs/commit/bcc0f8045 I can do this
{ pkgs ? import ./. {} }:

with pkgs;
let
  stdenv-musl = overrideCC stdenv gcc-musl;
  gcc-musl = wrapCCWith {
    name = "gcc+musl";
    cc = stdenv.cc.cc;
    libc = musl;
    detectLD = true;
  };
in

stdenv-musl.mkDerivation rec {
  passthru = { inherit stdenv-musl gcc-musl; };

  name = "musl-test";
  buildCommand = ''
    mkdir -p "$out/bin"

    cat >> main.c << EOF
    #include <stdio.h>
    int main(int argc, char *argv[])
    {
        printf("Hello Musl world\n");
    }
    EOF

    cc main.c -o "$out/bin/musl-dynamic-test"
    cc -static main.c -o "$out/bin/musl-static-test"
  '';
}

(Beware that wrapCCWith has changed significantly not too long ago.)
/cc @Ericson2314 about ideas on the best way to replace my hack-commit.

Err aren't we always detecting the loader today, and in your example? I don't see why libc = musl with wrapCCWith already doesn't work.

@Ericson2314: it doesn't work because the wrapper is hardcoded to ld-linux-x86-64.so.2 on x86_64-linux whereas musl calls it differently.

Oh right. I misread that. I'd just add a targetPlatform.libc == "musl" to the chain.

Ah, so you propose to handle libc change like cross-compilation.  I expect that will be more practical when having more dependencies...

There's support for musl cross and native now :D, so I think this can be closed?
Also: https://github.com/NixOS/rfcs/pull/23

For readers passing by: You can now easily try musl-linked C programs with nix by using the musl-gcc wrapper that's available in nixpkgs master > 18.03 (e.g. using the commit pinned below):
NIX_PATH=nixpkgs=https://github.com/NixOS/nixpkgs/archive/88ae8f7d.tar.gz nix-shell -p musl
[nix-shell:~/src/c]$ musl-gcc myprogram.c -o myprogram

I'm on NixOS 18.03, but using your pinned nixpkgs example (master > 18.03) should still work, right? However, it does not work for me:
$ NIX_PATH=nixpkgs=https://github.com/NixOS/nixpkgs/archive/88ae8f7d.tar.gz nix-shell -p musl
$ musl-gcc /tmp/hello.c
$ file a.out
a.out: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /nix/store/fg4yq8i8wd08xg3fy58l6q73cjy8hjr2-glibc-2.27/lib/ld-linux-x86-64.so.2, with debug_info, not stripped

IOW, I'm still getting glibc into my musl. :-)

@bjornfor You are right, a colleague pointed out today too that indeed glibc sneaks in. I now know why:
Looks like even musl-gcc calls plain gcc which includes the nix's stdenv flags which target glibc stuff. Found with strace -fe execve musl-gcc ....
(-static makes those flags to be ignored, so building statically is one workaround.)
The following works, also without -static:
/nix/store/vv4r320p5yd1k01kld62q1lppjxcswhb-gcc-7.3.0/bin/gcc -specs /nix/store/j88hc0jy0h1g350a7n0g2m29yfvvsrrn-musl-1.1.20-dev/lib/musl-gcc.specs -g myprogram.c -lpthread -o myprogram

@bjornfor, try that!
This what musl-gcc eventually calls but with some flags removed that pass libc stuff. The full flags were:
/nix/store/vv4r320p5yd1k01kld62q1lppjxcswhb-gcc-7.3.0/bin/gcc -O2 -D_FORTIFY_SOURCE=2 -fstack-protector-strong --param ssp-buffer-size=4 -fno-strict-overflow -Wformat -Wformat-security -Werror=format-security -fPIC -Wl,-dynamic-linker -Wl,/nix/store/fg4yq8i8wd08xg3fy58l6q73cjy8hjr2-glibc-2.27/lib/ld-linux-x86-64.so.2 -g myprogram.c -lpthread -o myprogram -specs /nix/store/j88hc0jy0h1g350a7n0g2m29yfvvsrrn-musl-1.1.20-dev/lib/musl-gcc.specs -B/nix/store/zk5zj2307zxaq7dx585yia3dn5k4qlsl-gcc-7.3.0-lib/lib -B/nix/store/fg4yq8i8wd08xg3fy58l6q73cjy8hjr2-glibc-2.27/lib/ -idirafter /nix/store/akak0rxhbi4n87z3nx78ipv76frvj841-glibc-2.27-dev/include -idirafter /nix/store/vv4r320p5yd1k01kld62q1lppjxcswhb-gcc-7.3.0/lib/gcc/x86_64-unknown-linux-gnu/7.3.0/include-fixed -B/nix/store/iw94llkj05wgaz268mlzvgx8jkbi1ss0-gcc-wrapper-7.3.0/bin/ -isystem /nix/store/j88hc0jy0h1g350a7n0g2m29yfvvsrrn-musl-1.1.20-dev/include -isystem /nix/store/j88hc0jy0h1g350a7n0g2m29yfvvsrrn-musl-1.1.20-dev/include -Wl,-rpath -Wl,/nix/store/1kf4h2shw37n8n0k0zlmypfv3zmrbd03-shell/lib64 -Wl,-rpath -Wl,/nix/store/1kf4h2shw37n8n0k0zlmypfv3zmrbd03-shell/lib -L/nix/store/mbi77di3ndy5kq3j1a0420pwz5h11ywh-musl-1.1.20/lib -L/nix/store/mbi77di3ndy5kq3j1a0420pwz5h11ywh-musl-1.1.20/lib -L/nix/store/fg4yq8i8wd08xg3fy58l6q73cjy8hjr2-glibc-2.27/lib -L/nix/store/zk5zj2307zxaq7dx585yia3dn5k4qlsl-gcc-7.3.0-lib/lib

@dtzWill do you know what we can do so that these glibc related flags don't leak in via the gcc-wrapper?

Honestly I've never used the musl-gcc wrapper much-- it's really a quickfix at best and should never be used in the place of a proper musl-based toolchain.  I actually disabled it originally, partially to avoid problems when people used it expecting it to act like what you get with setting crossSystem or localSystem appropriately.  I've heard folks say it's useful in a pinch but I'm not sure it's worth debugging if you're running into problems.
The interface for generating a shell from a given 'stdenv' is not as simple as it should be,
but something like the following should get you a 'cc' that builds against musl:
$ nix run nixpkgs.pkgsMusl.stdenv.cc
$ cc test.c -o test-c
$ file /test-c
./test-c: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /nix/store/35mb6imifzx9316y7nknhh6apx1k7a24-musl-1.1.19/lib/ld-musl-x86_64.so.1, with debug_info, not stripped

(-static should work at least for C programs, forget if things provide static versions of the c++ libraries or not)
Same caveats apply as normal re:using Nix-generated compiler wrappers and library dependencies, but this should get you started! Hope this helps!

I actually disabled it originally

Ah, that's why I couldn't find it in older commits. I thought its presence in newer commits meant that it's a new addition included for that purpose and that I should use it consequently.

Oh also keep in mind that for reasons that make no sense to me, 'nix-shell -p' actually unconditionally pulls in the default stdenv!
So, basically, I strongly recommend avoiding nix-shell -p when doing anything related to compilers or you'll run into all kinds of badness.  This can be seen with something as simple as nix-shell -p gcc8 and checking cc --version.
(this is because it uses runCommand and "for legacy reasons" but ...well anyway, if you didn't know about this "fun fact" that might be what you're encountering as well)


How to remove a global parameter

Is your feature request related to a problem? Please describe.

The problem is raised by https://github.com/Azure/azure-cli-extensions/issues/2083. When I try to use c.ignore to remove a global parameter of a command, nothing happens. Dear core owners, does it support removing a global parameter and how to do it? The customer has been waiting for a long time. Much appreciated.
Describe the solution you'd like

Describe alternatives you've considered

Additional context

Solved. Use _ before global parameters.


Possibility of using custom URL scheme/protocol

@KevinnZou is it possible to use custom URL schemes/protocols? Employing WKURLSchemeHandler on iOS and its alternative in Android.

@cayolblake Are you requiring the functionality we discussed in this issue?

Yes. It it's very important in most apps with WebView dependency.

@cayolblake Yes, we are also trying to provide that feature. However, we are encountering some troubles. You can check the issue above for details. But I will keep working on solving them. I apologize for any inconvenience caused by that issue.

@KevinnZou all your efforts are outstandingly appreciated and your work benefits a lot 🌹👍💪
Hopefully this gets resolved soon 🤞

@cayolblake I have implemented a basic interception in the latest version 1.9.8. Please follow these instructions to conduct a test and let me know if you encounter any issues.


WebView2 has wrong viewport size

Description
When WebView2 isn't placed in the upper left corner of the window, (at least to me) it seems like the viewport of the WebView2 has the wrong size (incorrectly adding the horizontal/vertical offset), thus the content gets cut.
Version
SDK: 3.0.0-preview3.201113.0
Runtime: Beta 88.0.702.0
Framework: WinUI 3 preview 3
OS: Win10, 20236
Repro Steps
For example: apply some margin to the WebView2.
<Page ...>
    <controls:WebView2 x:Name="webView1" Source="https://orf.at" Margin="400,0,300,0" />
</Page>

Screenshots

(webpage invisibly extends 400 px to the right)
Copied from MicrosoftEdge/WebViewFeedback#651.

I'm also seeing this issue. It's not only related to margin; basically any layout that causes the webview2 width != window width shows the issue:
<SplitView IsPaneOpen="True"
       DisplayMode="Inline"
       OpenPaneLength="300">
    <SplitView.Pane>
        <TextBlock Text="Hello World" />
    </SplitView.Pane>
    <controls:WebView2
        x:Name="MyWebView"
        Source="https://www.github.com"
        HorizontalAlignment="Stretch"
        VerticalAlignment="Stretch" />
</SplitView>

The above code renders like this:

This bug has been fixed, and the fix will be available in our next release - WinUI 3 Preview 4. We're hoping to ship this preview release in early February. Thanks!

This bug has been fixed, and the fix will be available in our next release - WinUI 3 Preview 4. We're hoping to ship this preview release in early February. Thanks!


Backport #10583

pydata-sphinx-theme building takes too long, leading to a timeout. I remember it also taking unusually long on my Mac.

We can probably switch to the released version of pydata-sphinx-theme for maint/1.0 rather than using master


A bug appear in samples in notebooks

🐛 Bug

To Reproduce
Steps to reproduce the behavior:
1.run migrating_from_torchvision_to_albumentations.ipynb

from torch.utils.data import DataLoader
trainloader = DataLoader(torchvision_dataset, batch_size=1,
shuffle=True, num_workers=1)
for i, (image, label) in enumerate(trainloader):
print(i, (image.shape, label))
add these lines of code,
and we can find that image.shape = torch.Size([1, 224, 224, 3]), but pytorch needs the inputs.shape=[1, 3,224, 224]

Expected behavior

Environment

Albumentations version (e.g., 0.1.8):
Python version (e.g., 3.7):
OS (e.g., Linux):
How you installed albumentations (conda, pip, source):
Any other relevant information:

Additional context

You need to transform from numpy array to the torch tensor. The way to do it is:
from albumentations.pytorch.functional import img_to_tensor

and within a DataLoader to cast img_to_tensor operation to the augmented image.
...

return img_to_tensor(image), label

I updated the notebook in https://github.com/albu/albumentations/pull/208


Fix auto guessing when importing CSV

Resources
Clubhouse story
Context
When a user tries to import a CSV file with wrong fields the auto guessing feature is not working. The CSV should be imported trying to guess the rest of the columns.
Changes

[x]  Import CSV with wrong fields without disabling auto guessing.
[x]  Add tests

Nice job @moicalcob! :smile:, I just added a few tips to improve the readability. :rocket:

In case you're curious about why I'm suggesting this:

Indeed I'd favor expect(age).to be_an(Integer)

https://github.com/rspec/rspec-expectations/blob/master/Should.md#why-switch-over-from-should-to-expect


About Caffe's Scale layer's weights and ACL's BatchNorm

Platform: Odroid N2 - Mali G52
Operating System: Ubuntu mate 18.04
I want to use ACL's CLBatchNormalizationLayer in Caffe based framework.
But caffe's BatchNormLayer is separated into 3 parts
( BatchNorm : normalize[mean, var] , Scale : scale[gamma] , shift[beta] )
In Caffe's pretrained model, caffe batchnorm layers's weights(mean, var) can be directly applied to
ACL's CLBatchNormalizationLayer. --> output has no problem
But when I put Caffe Scale Layer's gamma, beta into ACL BatchNorm, output is weird value.
How ACL can make graph with weights from caffemodel? (please focus scale layer's weights)
Thanks you

Sorry
There was a mistake in the code modification process.
Caffe Scale Layer's weights (gamma, beta) is same in ACL batchnorm layer's


About Fuzzy inferred membership values for input variables at each run

Hi
Could you please tell me where I can get fuzzy inputs membership values assigned at every run of Fuzzy inference to calculate Fuzzy score.
Thank you.

Dear Tawdes,
we did not create a specific interface for that. One option would be to iterate over the dictionary of linguistic variables and, for each one, iterate over the list of fuzzy sets it contains. You can get the membership values by passing the current state of the linguistic variable. Something like this:
for ling_variable, val in FS._lvs.items():
    print(ling_variable)
    for fuzzy_set in val._FSlist:
      print(fuzzy_set, fuzzy_set.get_value(FS._variables[ling_variable]))

we are considering to implement a simpler and more straightforward interface for this. If you need it urgently, we can prioritize this feature. Let me know!
All the best,
Marco


Lets get rid of E051

There's no good reason for this error to exist. At the very least it should be turned off by default

+1 :+1:

Fixed by #2543

I see. Waiting for a new jshint version in NPM then...


Missing config in info.plist causing crash.

In iOS 10.x, app must contain a new setting for using camera/microphone, but Bowser doesn't contain this config, therefore it crashed when in starting steps.
Please add
key: Privacy - Camera Usage Description
value: Bowser need to use your camera.
This will solve the crash problem.

Would there possible be a new release of BOWSER to work with iOS 10.x, soon? It works great with iOS 9.x. Please let us know.
gh


AssertionError: Default process group is not initialized

I have got the error below, how can i fix it?
[00:00<?,?it/s]17 08:33:48 WRN A exception occurred during Engine initialization, give up running process
Traceback (most recent call last):
File "train.py", line 251, in
main()
File "train.py", line 226, in main
loss = model(images, labels)
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/nn/modules/module.py", line 547, in call
result = self.forward(*input, **kwargs)
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/nn/parallel/data_parallel.py", line 152, in forward
outputs = self.parallel_apply(replicas, inputs, kwargs)
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/nn/parallel/data_parallel.py", line 162, in parallel_apply
return parallel_apply(replicas, inputs, kwargs, self.device_ids[:len(replicas)])
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/nn/parallel/parallel_apply.py", line 85, in parallel_apply
output.reraise()
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/_utils.py", line 369, in reraise
raise self.exc_type(msg)
AssertionError: Caught AssertionError in replica 0 on device 0.
Original Traceback (most recent call last):
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/nn/parallel/parallel_apply.py", line 60, in _worker
output = module(*input, **kwargs)
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/nn/modules/module.py", line 547, in call
result = self.forward(*input, **kwargs)
File "/zmz/ccnet/networks/ccnet.py", line 189, in forward
x = self.relu1(self.bn1(self.conv1(x)))
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/nn/modules/module.py", line 547, in call
result = self.forward(*input, **kwargs)
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/inplace_abn-1.1.0-py3.6-linux-x86_64.egg/inplace_abn/abn.py", line 323, in forward
self.group,
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/inplace_abn-1.1.0-py3.6-linux-x86_64.egg/inplace_abn/functions.py", line 307, in inplace_abn_sync
group,
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/inplace_abn-1.1.0-py3.6-linux-x86_64.egg/inplace_abn/functions.py", line 91, in forward
ctx.world_size = distributed.get_world_size(group=group)
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/distributed/distributed_c10d.py", line 584, in get_world_size
return _get_group_size(group)
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/distributed/distributed_c10d.py", line 200, in _get_group_size
_check_default_pg()
File "/usr/local/miniconda3/envs/dl10/lib/python3.6/site-packages/torch/distributed/distributed_c10d.py", line 191, in _check_default_pg
"Default process group is not initialized"
AssertionError: Default process group is not initialized

I have the same problem. Have you solved this problem?

No, I just give up it, sori~

This may be beacuse inplace-abn does not support single GPU.
If you have specified number of GPUs as 1 i.e,
--nproc_per_node=1
please try changing it.
Reference: https://github.com/speedinghzl/CCNet/issues/88#issuecomment-731197639

@jqc8438
This may be because inplace-abn does not support single GPU.
If you have specified number of GPUs as 1 i.e,
--nproc_per_node=1
please try changing it.
Reference: #88 (comment)

thank you,  actually, i set nproc_per_node=4


note about compatibility with SPI v 1.3

Sometime ago, i looked in stm32f303 and i discovered there was no relevant difference for I2S operation between spi 1.2 and spi 1.3, so the library should also work with spi 1.3
PS: I think we are ready for publishing a new version.

Thanks for contributing. I just published version 0.4.0 .


error " socket: too many open files"

maybe in server.go line 41,
dstServer, err := net.DialTCP("tcp", nil, request.RAWADDR)
the same request.RAWADDR,but still build a new connect.
but i don't know how to fix it

I fix a little stuff in server.go line 60
SecureCopy(dstServer, client, auth.Encrypt) 		readTimeout := 60 * 5 * time.Second 		err = dstServer.SetReadDeadline(time.Now().Add(readTimeout)) 		if err != nil { 			log.Println("setReadDeadline failed:", err) 		}

i fix a litte stuff in server.go line 60,
	```

SecureCopy(dstServer, client, auth.Encrypt)
readTimeout := 60 * 5 * time.Second
err = dstServer.SetReadDeadline(time.Now().Add(readTimeout))
if err != nil {
log.Println("setReadDeadline failed:", err)
}

i fix a litte stuff in server.go line 60,

SecureCopy(dstServer, client, auth.Encrypt)
readTimeout := 60 * 5 * time.Second
err = dstServer.SetReadDeadline(time.Now().Add(readTimeout))
if err != nil {
log.Println("setReadDeadline failed:", err)
}

Yes, I am not set timeout to server.


docker-machine does not work in elevated command window

System:
Host OS: Windows 7 Enterprise, 64 bit, SP1
Docker Host runs on virtual Linux, that comes out of <boot2docker.iso>
Boot2Docker: 1.11.1 (TCL 7.0); HEAD : 7954f54 - Wed Apr 27 16:36:45 UTC 2016
Oracle VirtualBox: 5.0.20 r106931
Docker Toolbox: 1.11.1
Docker-Machine: 0.7.0, build a650a40
Observed behaviour:
The virtual Linux, that Docker Host runs on, started and running with normal rights (i.e. does not started with administrative rights).
On the host computer (Windows 7), in a command window (normal, i.e. not elevated):
c:\temp>docker-machine.exe ls
NAME         ACTIVE   DRIVER       STATE     URL                         SWARM   DOCKER    ERRORS
DockerHost   -        virtualbox   Running   tcp://<IP_ADDRESS>:2376           v1.11.1

c:\temp>docker-machine.exe ip DockerHost
<IP_ADDRESS>

On the host computer (Windows 7), in an elevated command window (i.e. started using "Run as Administrator"):
C:\Windows\system32>docker-machine.exe ls
NAME         ACTIVE   DRIVER       STATE     URL   SWARM   DOCKER   ERRORS
DockerHost            virtualbox   Timeout

C:\Windows\system32>docker-machine.exe ip DockerHost
Error getting IP address: Host is not running

Does someone any idea what's the cause?
Expected behaviour:
docker-machine works the same way and produces the same result even when it runs with administrative rights.

DM is meant to be invoked without administrator rights


add more things to fetch.extra.mcmc

Figure of age comp forecast (issue #174) created using code in /r/r-functions/figures-age-comp-forecasts.r and additional columns in time-series table for Totol Biomass and/or Available Biomass (issue #173) will require additions to fetch.extra.mcmc that we decided to put off until sometime after the pre-SRG draft is completed. I'll put this under a 2018 milestone but if we get it done before the final draft this year, so much the better.

Will add read of catchability as well, as noted in issue #261.

Values related to catchability, age comp forecast, and total biomass have been added to model$extra.mcmc in recent commit, aefa70a. Closing issue.


Err when build example

Environment

Development Kit: esp8266-nodemcu
IDF version: 1.0.3
Development Env:      Arduino IDE (2.x)
Operating System:    Ubuntu
Power Supply:         USB

Problem Description
after follow your step to setup & install to arduino ide on ubuntu, when build prj, i meet some err
preCompile Build Path: /tmp/arduino/sketches/F9C5C29705F1B054F458AC71338804F2
open /tmp/3625109955/sketch_merged.cpp: no such file or directory

Compilation error: open /tmp/3625109955/sketch_merged.cpp: no such file or directory

Hi @stingdau1206 Thanks for your report!. It seems there is some problem with Arduino IDE 2.x, Can you try with Arduino IDE 1.8.x and let me know if it works well?

I get this on arduino-cli  Version: 0.35.3-arch Commit: 95cfd654fe0dd6b07e903d3132c0faceabfbe9e3


Document issues in Configuring OIDC Federated IdP Initiated Logout

Is your suggestion related to a missing or misleading document? Please describe.
Document [1] required following updates,
[1]https://is.docs.wso2.com/en/5.12.0/learn/configuring-oidc-federated-idp-initiated-logout/

[ ]  Add following missing section on document, Configure Pickup Dispatch application in the Primary IS

Enable OIDC back channel logout and add back channel logout url http://localhost:8080/pickup-dispatch/bclogout

[ ] Update in Try out section
In point number 4, It shoudl be correct as the same browser with different tab since this is a SLO scenarios

Describe the improvement

Optional Fields
Additional context

Related Issues:

Fixed: https://is.docs.wso2.com/en/latest/guides/login/oidc-federated-idp-initiated-logout/


Stub calls to runtime-lib so that tests can pass without dependency on runtime-lib

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Done


Add ignore-prefix option to no-expression-statement rule

Would you consider a ignore-prefix for the no-expression-statement.
My use case is logging, which pretty much per definition is a side effect, yet something that a server can't really live without. It would clean up my code a lot, since I now have a lot of this going on.
    // tslint:disable-next-line:no-expression-statement
    logSys.error('startup failed', err)

Happy to put a PR together, if you would consider accepting it.

Just to clarify you mean that if we configure it like this:
"no-expression-statement": [true, {"ignore-prefix": "sideEffect"}]

Then this would be ignored:
sideEffectFoo("bar")

I think that makes sense and would welcome a PR for it :-).
If the call is a method on an object, I guess then the prefix would apply to the method name and not the object name:
myObj.sideEffectFoo("bar"); // This will be ignored
sideEffectObj.myMethod("bar"); // This will **not** be ignored

Or should both object name and method name be considered?

If the call is a method on an object, I guess then the prefix would apply to the method name and not the object name
...
Or should both object names and method name be considered?

Maybe both? I mean for console.log, would it perhaps be more natural to ignore the whole thing, not just log

Released in 4.3.0.


Create a validate command

There should be a built-in validation command that will ensure the typings are accessible to the greatest number of consumers. For example:
typings validate
# Checks the typings field is set in `package.json`/`bower.json`
# Checks all dependencies are defined in `typings.json`
# Checks installation compiles properly with all types available

Edit: It should also be able to run the compiled types against a test suite, for usage in type definition repos, as one example.

An ideal way to do this is to generate the bundled typings in memory, initialise a language services instance of TypeScript, transpile the test files for type validation and echo the output. E.g.
typings validate test.ts | node

Also relevant is a command like typings bundle to generate this output, which could actually be used to generate bundles for DefintitelyTyped.


Circle ci artefact urls

Pretty sure circleci have deprecated the old endpoint we used to use to construct the links to the test artefacts.
Will dig out the doc links and fill in with more detail later :)

Yeah, looks like they changed the format
Old way:
https://${CI_BUILD_NUM}-${CI_PROJECT_ID}-gh.circle-artifacts.com/0/cypress/reports/mochareports/report.html

New Way:
https://output.circle-artifacts.com/output/job/${JOB-ID}/artifacts/0/cypress/reports/mochareports/report.html

thanks @dthisner release going out now 👍🏾


port to gtk3

With this PR Pullanapac activity is ported to gtk3 and sugar3.
Resolves #4

Reviewed.  Tested.


Aws customer admin role

https://issues.redhat.com/browse/OSD-2793
Generates JSON such as the following:
{
  "Version":"2012-10-17",
  "Statement": [
    {
      "Effect": "Allow",
      "Action": ["ec2:DescribeInstances"],
      "Resource":["*"],
      "Condition": {
        "StringEquals": {
          "ram:RequestedResourceType": "route53resolver:ResolverRule"
        }
      }
    }
  ]
}

/retest

/hold
This lgtm but I want mwoodson to review. Could you also rebase?

/lgtm


Addd social media and feedback section in blog page

Please assign this issue to me

Hello I would like to work on this issue. Please assign it to me.

Hi!! I'd like work on this issue. Please assign it to me.


Meteor fontawesome

I use the meteor package fortawesome:fontawesome
but the icons don't show up when running the components in react-storybook.

We don't support Meteor packages. You need to load font-awesome manually.
One way is to add it manually into the head: https://getstorybook.io/docs/configurations/add-custom-head-tags
Or you can import the fontawesome CSS directly from NPM.

Link is now: https://storybook.js.org/configurations/add-custom-head-tags/


app-manager: wrap sys.stdout into Unicode-aware StreamWriter

Otherwise Python can't guess encoding when piping out text and fails on, say,
non-ASCII versions.

Please bump version to 0.3.1a1

Done already: https://github.com/genestack/python-client/blob/master/genestack_client/version.py


WIP feat(Menu): ux-menu element

Initial work

Markup:
<ux-button ref='menuButton' click.delegate='menuOpen = true'>Menu</ux-button>
<ux-menu open.bind='menuOpen' trigger.bind='menuButton'>
  <ux-menuitem click.delegate='zoomIn()' helper-text='ctrl + [+]'>Zoom In</ux-menuitem>
  <ux-menuitem click.delegate='zoomOut()' helper-text='ctrl + [-]'>Zoom Out</ux-menuitem>
  <ux-menuitem click.delegate='copy()' helper-text='ctrl + C'>Copy</ux-menuitem>
  <ux-menuitem click.delegate='paste()' helper-text='ctrl + V'>Paste</ux-menuitem>
  <ux-menuitem click.delegate='closeApp()' helper-text='alt + F4'>Close</ux-menuitem>
</ux-menu>

static img

bindings:

open  => default mode: two way. true = show menu ,false = hide
trigger: what element will be used to trigger the menu
?? (what else here ?)

TODO:

[ ] disabled / enabled states
[ ] nested menu
[ ] drop up, drop right, drop left configurable
[ ] divider (vertical, horizontal)
[ ] left icon , right icon
[ ] theming

@EisenbergEffect @ZHollingshead I'm not sure how we going to handle icons, thoughts ?

@bigopon This is fantastic work! @ZHollingshead Can you provide some feedback, particularly on how we'd want to handle icons. Another things to consider, could we do something cool with keyboard shortcuts, particularly making them automatically adjust based on platform e.g. cmd vs. ctrl.

@EisenbergEffect how the API would be if we are to try recognise those help text ? I think they are a bit arbitrary to do

So I am not sure how to handle Icons right now. Especially given that the ux-icon package is so heavy at the moment. The way I would suggest handling those is have a set container that can contain whatever icon is provided (enabling the user to use a different icon set if they wish)
Something like
<ux-menuitem>

  text
</ux-menuitem>

I pulled this down and took a look at it. I think it works pretty great as it is. I think it is too rigid as it is. I think the menu item itself should be able to be set using a class as well as an element. That way I could for example have an anchor element that is a link as a menu item.
<ux-menu open.bind='menuOpen' trigger.bind='menuButton'>
  <ux-menuitem click.delegate='zoomIn()' helper-text='ctrl + [+]'>Zoom In</ux-menuitem>
  <ux-menuitem click.delegate='zoomOut()' helper-text='ctrl + [-]'>Zoom Out</ux-menuitem>
  View Users
</ux-menu>

Another idea I think would be neat is being able to simply target a button that will toggle the menu rather than having to toggle through a variable.

Agree that menu should be able to be triggered from an anchor button, I left it in initial markup as a todo. I think this can be continued after we adjust the build setup and initialization of the whole lib. I kind of want to wait for main PRs in templating before progressing this.


kaggle_api cannot submit

error,WARNING Retrying (Retry(total=6, connect=None, read=None, redirect=None, status=None)) after connection broken by 'SSLError(SSLError("bad handshake: SysCallError(10054, 'WSAECONNRESET')",),)': /upload/storage/v1/b/kaggle-competitions-submissions/o?uploadType=resumable&upload_id=AEnB2UoTUF_ZJou1BV_fFERhDzRqnLVKu5Uh3iuxP_YJgWNOrEjLiZMtlTquo0uGUONeSQ5BMTUgS6iGTHgeNhbKGHrDkZpgSw

Do you have solved it? I meet this problem too.

Can you provide some more context for this issue? What were you doing and what happened? How can i be reproduced? What version are you using?

I can't reproduce this bug, feel free to reopen if you are still experiencing it.


access props in reducer

Is it possible to get the props on a form (or really just whether the form is valid) in a reducer?  If not, is this something that could be supported?
I want to have an action that will set which of a group of forms is rendered ("active"), but it will not change the active form if the currently active form is not valid.  I was thinking to do something like the following, but I couldn't find the validity information in Redux Form's store.
// This isn't code I have actually tried to run.

function activeForm(globalState, action) {
    if (action.type !== TRY_TO_SET_ACTIVE_FORM) return globalState.activeForm
    if (globalState.activeForm && globalState.form.myForm[globalState.activeForm].props.invalid) {
        return globalState.activeForm
    }
    return action.payload
}

function mainReducer(state, action) {
    return {
        form: reduxFormReducer(state.form, action),
        activeForm: activeForm(state, action)
    }
}

// In some component's render method...
{activeForm && <MyForm formkey={activeForm}/>}

The alternative, I guess is to use two different actions, a "request set active form" action that anyone can fire and a "actually set active form" action that the active form could fire in componentWillReceiveProps when it detects the request action and it is valid.  Also, would have to set up something to Fire the "actually set active form" action also when there is no currently active form.  Or is there a cleaner way I am overlooking?

You can check for validity before dispatching your action. In a component handler.

See also: Wizard Example

OK, but won't it require some workaround for #202?  Also, this way seems a bit against the spirit of Redux as a functional single source of truth.

The comment I just wrote was directed at @erikras.  @frost555, I am not sure what you are suggesting.  Do you mean I would keep a reference to the active form component and call a method on it that would tell me if it is valid?  That might work although it seems like that kind of thing is generally regarded as an anti-pattern in React.  You are not supposed to have to access the state or props of a component from outside it or to imperatively signal components other than by changing their props.  Granted, I could arrange to signal the component by changing its props (by connecting it to Redux with react-redux and dispatching an action), but that seems overly complex and gratuitously imperative.  Maybe I am not understanding your suggestion though.

This is a usage question, and we encourage you to ask it on StackOverflow instead. We try to use the issue tracker for bug reports and feature requests, as well as some discussions, but this appears to be a very localized question.
This is the only way to ensure the same question doesn’t get asked many times in a row, and that folks who have the same question in the future will be able to find it.
I’m closing for these reasons.


Avoid failures when query argument to dispatch run route is not a valid id for a simulator

Instead of a failure, leave the simulation tool box unselected.
Provide similar handling for other query arguments
Example failure:
https://run.biosimulations.org/run?simulator=boolet

Done in #2267


Cannot use assets from parent directory

If you have a directory structure like
dist
packages/
stylemark.config.yml
and execute command like
stylemark -i packages -o stylemark -c stylemark.config.yml -w 1000 -b 5000
You end up with fs trying to copy to the same directory.
Please see stylemark/src/generator.js:100:9

Could you include the contents of your .stylemark.yml?

Sure, it's pretty simple:
name: demo
assets:

../dist

Just so I understand, are dist, packages, and stylemark.yml all siblings in the same directory?

I'm having a similar problem and think the problem is that the path defined for assets in the config is appended to both the input and output folders chosen. (here: https://github.com/nextbigsoundinc/stylemark/blob/master/src/generator.js#L101)
My folder structure is:
package.json
.stylemark.yml
dist/
resources/assets/styles

And I'm running stylemark through an npm script similar to this:
"stylemark": "stylemark -i ./resources/assets/styles -o ../../../styleguide -c ./.stylemark.yml"

since I have to copy the styleguide to the web root.
The .stylemark.yml config only works if I link assets with the following line:
assets:
  - ../../../dist

However this obviously leads to wrongly copied code since ../../../dist is appended to the input as well as the output path given.

The same problem also occures with the logo. My proposal would be to use paths relative to the cwd of the process, so usually the location of package.json.

After more testing, resolving filepaths relative to the configuration file is a bad idea. Instead, filepaths will now be resolved to the root project directory—ie. the first ancestor directory that contains package.json.
This change will land shortly in v2.0


set the primary key on VmdbDatabaseSetting to nil

there is no pk on this table, and when you call methods like first in
rails 4, it will try to sort by the pk which doesn't exist in this case

:+1:


AXI4-ST drops TREADY when traffic is started before AIB link up

This is the condition in which this issue occurs:

No AIB link
Leader drives TDATA and TVALID after axi reset
Follower is always TREADY = 1

Why this happens:

Leader side application sends data and waits for TREADY (tx_online is 0)
AIB links up and tx_online=1. TREADY goes high
AXI receives user data in FIFO and asserts FIFO full. This will de-assert TREADY
AXI-ST leader transmit the data to AIB through CA module asserting master_align_done=1
AIB leader transmits the data to AIB follower
CA follower module align done is still low and drops packet from AIB follower
AXI-ST follower does not get packet and will not send credit update to AXI-ST leader
AXI-ST leader is waiting for credit update from follower and waits for forever.
This keeps TREADY low for ever.

Work around:
Start AXI-ST traffic after AIB-link is establishes (tx_online=1)
Waveform

To reproduce this issue we had to set delay_y_value = 0

Unable to reproduce to this issue in full_examples and VIP setup. Started traffic after reset before online and TREADY is always 1
Example design delay values (in hex): X = 6, Y=1c2, Z = 190.

Better picture


MRTK 2.2 - Cant build because of error while creating certification

I updated to Mrtk from 2.1 to 2.2 and Unity from 2019.2.15 to 2019.3. In play-mode everything works fine but my build is failing because of following lines:
#1
Failed to create a file 'MRTK v2.0\QViewer Mrtk 2.2\Assets\WSATestCertificate.pfx'
UnityEditor.EditorUtility:WSACreateTestCertificate(String, String, String, Boolean)

#2
Error while creating certificate at 'MRTK v2.0\QViewer Mrtk 2.2\Assets\WSATestCertificate.pfx'
UnityEditor.EditorUtility:WSACreateTestCertificate(String, String, String, Boolean)

#3
InvalidOperationException: Failed to create test certificate.
PostProcessWinRT.CreateTestCertificate () (at C:/buildslave/unity/build/PlatformDependent/MetroPlayer/Extensions/Managed/PostProcessWinRT.cs:1045)

I tried to delete my old WSATestCertificate.pfx and switched my build folder, but that didnt help. Is sb else experiencing the same behaviour?

I somehow "fixed" the problem, by downgrading Unity back to 2019.2.15 and updating again to 2019.3. Or by resetting my build and project settings in unity. The only part that does not look good are the 147 Warnings about obsolete api...but my building works now and the app runs!


Claimtrie branching tests with test for remove suppport bug

This is a redo of https://github.com/lbryio/lbrycrd/pull/39, containing the new claimtrie branching test, and the test for the remove support bug.
In the future, will need to merge claimmtrie_tests into claimtriebranching_tests to make a single test for claimtries.

The test  fails here, but this is the correct behavior as https://github.com/lbryio/lbrycrd/pull/52/commits/03b3828a659bdccca3b5be221a0ccee237469e72 should fail without  https://github.com/lbryio/lbrycrd/pull/53

LGTM


Publish helm chart with each release

Describe the bug
Helm chart is not published into a helm repository.
Expected behavior
Have a helm chart available with each release (or a known cadence).
Additional context
https://github.com/knative/operator/pull/1447 added a helm chart
https://github.com/knative/operator/pull/1501 modified behavior to generate helm chart
Would greatly appreciate if this chart was published somewhere so this operator could be deployed using gitops tools.

Not sure how to best contribute here.  Some teams choose to publish the chart with the same versioning as the project, others split it out.

@acelinkio Are you interested in publishing helm charts? Feel free to contribute. Thx.

Is this something that is being worked on? I am very interested in deploying the Knative Operator with GitOps methods.

Hey, my two cents here:

This is a big pain point that the Helm Chart is not released anywhere.

It would be possible to use ArgoCD with Helm Chart source as a git repository but the config/charts/knative-operator/Chart.yaml uses hacky {{ version }} variable that is provided through the ./hack/generate-helm.sh described here which makes this unusable.
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: knative-operator
  namespace: argocd
  finalizers:
    - resources-finalizer.argocd.argoproj.io
spec:
  destination:
    server: https://kubernetes.default.svc
    namespace: knative-operator
  project: default
  sources:
    - repoURL: https://github.com/knative/operator.git
      targetRevision: knative-v1.11.12
      path: config/charts/knative-operator

The above ArgoCD App example would basically download the repo and checkout a specific version and execute helm template, but the following error is blocking this:
Error: cannot load Chart.yaml: error converting YAML to JSON: yaml: invalid map key: map[interface {}]interface {}{"version":interface {}(nil)}

There are nice tools that helps in releasing the Helm Chart with GitHub Repository serving as a Helm Chart Repository. One example would be the Chart Releaser. But, the Helm Chart Release process in knative/opreator is using the ./hack/generate-helm.sh and hacky {{ version }} in the Chart.yaml. Without it, it would be fairly straightforward to automate the Helm Chart Release process. Another idea would be to define a custom GH Workflow that would automate this process.

Another way to have a GitHub Repository as a Helm Chart Repository relates to manual creation of the index.yaml file and host it behind GitHub Pages. TL;DR:

Create branch (can be empty) for GitHub Pages to point as the source of what to host.
Create some simple index.html file for GitHub Pages to have something to host on the webpage.
Create simple index.yaml file that contains information about the Helm Charts and their URL to the archive

example: https://helm.sh/docs/topics/chart_repository/#the-index-file

Configure GitHub Pages in the GitHub Repository config.

Resources:

https://docs.github.com/en/pages/getting-started-with-github-pages/creating-a-github-pages-site
https://helm.sh/docs/topics/chart_repository
https://github.com/helm/chart-releaser

/reopen
/remove-lifecycle stale

Or, even easier: create index.yaml file with format specified as in https://helm.sh/docs/topics/chart_repository/#the-index-file and create helm chart repository in artifacthub.io pointing to this file in a format similar to
https://raw.githubusercontent.com/knative/operator/main/

The above URL assumes that in the main branch of this repository contains the index.yaml file.
Then the artifacthub.io scans periodically for changes in the file and serves as the helm repository based on the index.yaml file.
One could even add some logic to the ./hack/generate-helm.sh or create ./hack/update-helm-index.sh and just commit the changes when new package is available.
With this, no github pages are required and this is baseically a two-step approach. Create index.yaml file and point to it in artifacthub.io.

This is an example python script that can automate creation if index.yaml file:
import yaml
import requests
import hashlib
from datetime import datetime

urls = [
    "https://github.com/knative/operator/releases/download/knative-v1.14.5/knative-operator-v1.14.5.tgz",
    "https://github.com/knative/operator/releases/download/knative-v1.14.4/knative-operator-v1.14.4.tgz"
]

index = {'apiVersion': 'v1', 'entries': {}}

for url in urls:
    name_version = url.split('/')[-1].replace('.tgz', '')
    name, version = name_version.rsplit('-', 1)
    response = requests.get(url)
    digest = hashlib.sha256(response.content).hexdigest()

    if name not in index['entries']:
        index['entries'][name] = []

    index['entries'][name].append({
        'apiVersion': 'v2',
        'name': name,
        'version': version,
        'urls': [url],
        'created': datetime.utcnow().isoformat() + 'Z',
        'digest': digest
    })

with open('index.yaml', 'w') as f:
    yaml.dump(index, f, sort_keys=False)


wip: offline version

WIP! not ready to be merged.
A first draft to add service workers to allow offline caching of assets.
Based in https://github.com/jakearchibald/simple-serviceworker-tutorial, many things directly copied ;-).
The important part is the registration of the service worker in index.html (can be moved to it's own file) and the service worker itself public/sw.js. As far as I understand, the sw.js has to be placed on the root, to be able to cache all urls (if stored in assets/js it would only be able to cache files under assets/js/).
For testing purposes there is a /reset-service-worker route (that only works online) to get rid of all the service workers and caches.
There is also a logger logging.js to better understand what is happening (logs to the console).
As I said, veery rough state, more to show how service worker work.

fyi I’m working on this together with Pat :)

Doing this in https://github.com/hoodiehq/hoodie-camp-tutorial instead.


Missing Documentation (config.yaml)

I'm trying to use OpenSfM for a project but am having trouble understanding the different configuration parameters that are available. The configuration section (link) appears to be missing. It would be great if someone could write up a quick guide on what options are available and what they do. Also I seem to be having difficulty running "3D" datasets captured from a drone. (mainly cell phone towers) I'm hoping those issues are just a configuration problem though so I'll add a new issue later if it's not.

We are lagging on documentation, sorry. The best source for a short explanation of the parameters is https://github.com/mapillary/OpenSfM/blob/master/opensfm/config.py
The default parameters should work fine for most situations, though. Please, do post an issue with the images if the reconstruction fails. Thanks!

After tweaking the options it hasn't made much difference. Can I pm you a link to the dataset? (I can't share it publicly sorry) OpenSfM seems to have stitched individual vertical sections of the tower pretty well, but there are lots of random images everywhere and each vertical section seems quite "disjointed".

can you send me the link to pau at mapillary.com? thanks

Will do thanks heaps :)


Code refactor to use railtie, scope naming, and include alias method chain instead of alias.

Some changes to allow it to work with Rails 3.2 + Paperclip 2.7 (in conjunction with delayed_paperclip).
Because of how it was aliased, it wasn't being triggered in post process steps correctly. Did a minor refactor and all is good right now.
Specs are passing (nice to see they were testing the functionality not the implementation :)

It will be compatible with older rails version after using railtie?
Btw, can you remove .rvmrc file from this pull request?

Ah, I see, it will.


[Feature request] Reload editor when file has been modified elsewhere

Is your feature request related to a problem? Please describe.
When a file has been modified with another editor (or a program output), reload the text editor. It is very useful for log files.
Describe the solution you'd like
Add real time text editor refresh.
Describe alternatives you've considered
Add a prompt to reload the text editor as soon as it has been modified (Reload: Yes/No).

It is a little bit tricky on listening file events for UWP app, not saying it is not doable but I do need time to learn and see how it can be done nicely. I will come back later for this one.

This feature has been implemented:

Closing this issue since it has been deployed.