Tauto performances

Made some major improvements in the performance and coverage or tauto. I left comments about how much run time decreased which I can remove once the reviewers are satisfied with the PR

I rolled my own congruence closure structure and it turned out to be faster than the one in core. I'm not sure why. I'm thinking it might be because the implementation in core is more general

done


Proposal: simulate_site_dynamics function that automatically converts to/from site basis

An awkward aspect of the current API (e.g., as discussed in #36) is that using simulate_dynamics requires knowing how to represent the state vector for the particular dynamical model you are using. Depending on the choice of dynamics, these can differ greatly (e.g., for ZOFE or even just site vs exciton basis).
This is sometimes a useful thing to be able to do. However, in many cases, usage of qspectra would be cleaner if input and output of simulate dynamics was represented in a standard way (e.g., in the site basis for the Hamiltonian). This is exactly what I propose simulate_site_dynamics should do.
To implement this cleanly, we should implement methods that do the canonical conversion on each DynamicalModel. Then simulate_site_dynamics can be a thin wrapper that looks something like:
def simulate_reduced_dynamics(dynamical_model, initial_state, *args, **kwargs):
    init_state_prime = dynamical_model.liouville_to_state_vector(initial_state)
    t, states_prime = simulate_dynamics(dynamical_model, init_state_prime, *args, **kwargs)
    states = dynamical_model.state_to_liouville_vector(states_prime)
    return t, states

CC @donghyun208

This was fixed by the HEOM PR. simulate_dynamics now always simulates the density matrix in the site basis.


Adding registry authentication configs/mirrors throws error on console and adds broken values in YAML

Setup

Rancher version: 2.7-head & 2.9-head

Describe the bug

When adding registry authentication configs on cluster's Edit Config page an error gets
To Reproduce

Start an RKE2 cluster creation process with any provider
Go to Registries Tab and enable the checkbox
Add a registry host
Click on "Show Advanced" and add a registry authentication
Save it and come back to the page
Screenshots

https://github.com/rancher/dashboard/assets/135728925/21f4d084-26ae-468a-b2b0-0930fafaf491

cc @gaktive

/backport v2.8.next1

For an e2e, one could use https://github.com/rancher/dashboard/pull/10898 as a basis to build the test

Coverage looks good to me. Tests on generated YAML is enough as unit testing.


[WIP] fix: disable autobreadcrumbs

We disable the autoBreadcrumbs option in Raven such that no
information is sent implicitly.
See: https://github.com/resin-io-modules/resin-corvus/issues/18

Looking great! Can we add, probably in another PR, a function to manually log context, though, so we don't lose what we're currently printing?

Ah, I think logDebug does that already, so we basically just need to move to corvus.logDebug instead of console.log in most places at Etcher.

What do you mean by log context?
Also the reason this is marked WIP right now anyway is because I haven't tested it yet, but in accordance with the docs this should work. Do you know of a way to dry-run Raven, or otherwise see what data is actually sent to the servers?

What do you mean by log context?

Sorry, that's a bit of Sentry parlance. Sentry calls any line that you console.log the "context" of an error.

Also the reason this is marked WIP right now anyway is because I haven't tested it yet, but in accordance with the docs this should work. Do you know of a way to dry-run Raven, or otherwise see what data is actually sent to the servers?

We have test Sentry applications that you can use. You can basically set the tokens of those testing accounts, and either:

Set the Corvus token as an environment variable so the Makefile catches it (https://github.com/resin-io/etcher/blob/master/Makefile#L142), and package Etcher with a production release type, so they get injected
Hot patch corvus to bypass the asar check (https://github.com/resin-io-modules/resin-corvus/blob/master/should-fake.js#L23), so events are emitted even from outside an asar, and manually insert the tokens in the package.json

@Shou what's the state of this – does this need renewed attention?


chopsticks.tunnel.RemoteException in basic usage on Mac OS X

Executing in  Macosx box with cpython 2.7:
#!/usr/bin/env python
from chopsticks.tunnel import Tunnel
tun = Tunnel('localhost')
import time
print('Time on %s:' % tun.host, tun.call(time.time))

Results to
Traceback (most recent call last):
  File "./py0.py", line 11, in <module>
    print('Time on %s:' % tun.host, tun.call(time.time))
  File "/Users/username/example/.chopsticks/lib/python2.7/site-packages/chopsticks/tunnel.py", line 179, in call
    raise RemoteException(ret.msg)
chopsticks.tunnel.RemoteException: Unexpected EOF on stream

And the same if not executing from a virtualenv
Traceback (most recent call last):
  File "./py0.py", line 11, in <module>
    print('Time on %s:' % tun.host, tun.call(time.time))
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/chopsticks/tunnel.py", line 179, in call
    raise RemoteException(ret.msg)
chopsticks.tunnel.RemoteException: Unexpected EOF on stream

Same scripts works when executed against a linux box.
('Time on localhost:',<PHONE_NUMBER>.645448)

If you type 'ssh localhost python' at a bash prompt do you get a Python
shell?
I appreciate that the ability to identify and report the errors encountered
here could be improved.
On Sun, 16 Oct 2016, 14:18 Petros Makris<EMAIL_ADDRESS>wrote:

Executing
#!/usr/bin/env python
from chopsticks.tunnel import Tunnel
tun = Tunnel('localhost')
import time
print('Time on %s:' % tun.host, tun.call(time.time))
Results to
Traceback (most recent call last):
File "./py0.py", line 11, in
print('Time on %s:' % tun.host, tun.call(time.time))
File "/Users/username/example/.chopsticks/lib/python2.7/site-packages/chopsticks/tunnel.py", line 179, in call
raise RemoteException(ret.msg)
chopsticks.tunnel.RemoteException: Unexpected EOF on stream
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lordmauve/chopsticks/issues/13, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAlrCMAhdyDJ7c4_NH0ONHzpoDVjN7yMks5q0iQ1gaJpZM4KX-BW
.

Yes I do:
~ > ssh localhost python
print 'hello world'[Ctrl+D]
hello world
~ >

Closing this issue until ability to identify and report errors encountered is improved.

This is a genuine issue; I'd rather keep it open.

Did you get the error message line
[localhost] bash: /usr/bin/python2: No such file or directory

? If this is the problem, it's relatively easy to fix; just add (before connection):
Tunnel.python2 = 'python'

This seems to be the situation on an out-of-the-box Mac OS X Sierra. I believe this is a duplicate of #14.
The solution is documented here: http://chopsticks.readthedocs.io/en/latest/howto.html#how-to-customise-interpreter-paths


Feature/ohhttp stubs carthage

moving from a submodule to Carthage using Cartfile**.private**
https://github.com/Carthage/Carthage/blob/master/Documentation/Artifacts.md#cartfileprivate

too many conflicts with the recent changes, will redo it later

Apologies for pushing to master directly. Will finish with the Travis CI integration (almost there, only some flaky tests pending) and then move to pull requests from tomorrow onwards.

No problem :-)


Mock two consecutive calls with different responses without explicit access to the calling context.

I have a function that makes multiple fetch calls with the same request, and expects to receive different responses.
It would appear logical that you can use .mock(), and then .once() (or vice versa, or multiple .once() calls) in order to achieve this, but what happens instead is the following error is thrown:
fetch-mock: Adding route with same name or matcher as existing route. See `overwriteRoutes` option.

, which makes the once functionality almost entirely useless.
The doc says:

If the route has already been called repeat times, the call to fetch() will fall through to be handled by any other routes defined

Which would imply that one URL can be handled by multiple mocks, but, apparently, it seems like that is not the case.
Code example:
fetchMock
      .once(
        `${apiHost}/endpoint`,
        () =>
          new Response(
            JSON.stringify({
              code: 401,
              message: 'Unauthorized',
            }),
            {
              status: 401,
            },
          ),      )
      .mock(
        `${apiHost}/endpoint`,
        () =>
          new Response(
            JSON.stringify({
              code: 200,
              message: 'ok',
            }),
            {
              status: 200,
            },
          ),
      );

Found the solution: apparently, if two handlers match the same URL, they are given the same name by default, and the name is supposed to be unique. The solution is to explicitly pass a unique name:
fetchMock
      .once(
        `${apiHost}/endpoint`,
        () =>
          new Response(
            JSON.stringify({
              code: 401,
              message: 'Unauthorized',
            }),
            {
              status: 401,
            },
          ),
        { name: 'unauthorised' },
      )
      .mock(
        `${apiHost}/endpoint`,
        () =>
          new Response(
            JSON.stringify({
              code: 200,
              message: 'ok',
            }),
            {
              status: 200,
            },
          ),
        { name: 'success' },
      );

Hi - you'll be glad to hear this behaviour will be improved in the new version of fetch-mock I'm working on. matchers will no longer be coerced into pseudo-names, and only routes that are explicitly named the same thing will cause a clash like this.

@wheresrhys thanks, looking forward to integrate


flat issue

I'm having an issue using the flat feature.  If I use spectrum with the default settings, it works fine.  But I'd like to use it with flat: true to have the selector always open.  When I go to flat, it doesn't allow me to move the selector to choose a different color.
One thing I have noticed is that if I pop the firebug panel open or slide it up or down, it will start kick in and working fine, but otherwise it won't work until I do that once.
any ideas what is wrong with it?  if it helps, I've put the element inside of a jquery-ui accordion.  Like I said though, default implementation works fine, just doesn't allow a change when I use the flat setting.

Interesting that it works once you open firebug! Any way you can send a link to a jsfiddle that reproduces the problem? http://jsfiddle.net
On Dec 20, 2011, at 3:24 PM<EMAIL_ADDRESS>wrote:

I'm having an issue using the flat feature.  If I use spectrum with the default settings, it works fine.  But I'd like to use it with flat: true to have the selector always open.  When I go to flat, it doesn't allow me to move the selector to choose a different color.
One thing I have noticed is that if I pop the firebug panel open or slide it up or down, it will start kick in and working fine, but otherwise it won't work until I do that once.
any ideas what is wrong with it?  if it helps, I've put the element inside of a jquery-ui accordion.  Like I said though, default implementation works fine, just doesn't allow a change when I use the flat setting.

Reply to this email directly or view it on GitHub:
https://github.com/bgrins/spectrum/issues/6

Yep, that worked as well.
Thanks!


Issue compiling examples/shapes.rs

Upon attempting to compile the example/shapes.rs file I receive the following message from the compiler:
error[E0277]: the trait bound `mint::Quaternion<f32>: std::convert::From<cgmath::Quaternion<f32>>` is not satisfied
  --> src\main.rs:60:18
   |
60 |             mbox.set_orientation(q);
   |                  ^^^^^^^^^^^^^^^ the trait `std::convert::From<cgmath::Quaternion<f32>>` is not implemented for `mint::Quaternion<f32>`
   |
   = help: the following implementations were found:
             <mint::Quaternion<T> as std::convert::From<[T; 4]>>
   = note: required because of the requirements on the impl of `std::convert::Into<mint::Quaternion<f32>>` for `cgmath::Quaternion<f32>`

error[E0277]: the trait bound `mint::Quaternion<f32>: std::convert::From<cgmath::Quaternion<f32>>` is not satisfied
  --> src\main.rs:61:18
   |
61 |             mcyl.set_orientation(q);
   |                  ^^^^^^^^^^^^^^^ the trait `std::convert::From<cgmath::Quaternion<f32>>` is not implemented for `mint::Quaternion<f32>`
   |
   = help: the following implementations were found:
             <mint::Quaternion<T> as std::convert::From<[T; 4]>>
   = note: required because of the requirements on the impl of `std::convert::Into<mint::Quaternion<f32>>` for `cgmath::Quaternion<f32>`

error[E0277]: the trait bound `mint::Quaternion<f32>: std::convert::From<cgmath::Quaternion<f32>>` is not satisfied
  --> src\main.rs:62:21
   |
62 |             msphere.set_orientation(q);
   |                     ^^^^^^^^^^^^^^^ the trait `std::convert::From<cgmath::Quaternion<f32>>` is not implemented for `mint::Quaternion<f32>`
   |
   = help: the following implementations were found:
             <mint::Quaternion<T> as std::convert::From<[T; 4]>>
   = note: required because of the requirements on the impl of `std::convert::Into<mint::Quaternion<f32>>` for `cgmath::Quaternion<f32>`

error[E0277]: the trait bound `mint::Quaternion<f32>: std::convert::From<cgmath::Quaternion<f32>>` is not satisfied
  --> src\main.rs:63:19
   |
63 |             mline.set_orientation(q);
   |                   ^^^^^^^^^^^^^^^ the trait `std::convert::From<cgmath::Quaternion<f32>>` is not implemented for `mint::Quaternion<f32>`
   |
   = help: the following implementations were found:
             <mint::Quaternion<T> as std::convert::From<[T; 4]>>
   = note: required because of the requirements on the impl of `std::convert::Into<mint::Quaternion<f32>>` for `cgmath::Quaternion<f32>`

error: aborting due to 4 previous errors

You aren't building the shape.rs (which is btw tested on CI to build). Your errors come from src\main.rs, which I assume you created and copied the content from shape.rs. Take a closer look at the dependencies used, version of cgmath in particular.

You are absolutely correct! That is what I did, and once I changed the dependency version on cgmath everything functioned as expected. Apologies for opening an issue on this. I had a feeling it was something on my end (as this was the first time I've done anything with rust). Greatly appreciate the assistance, and love the idea of this crate! Would there happen to be another location for asking questions and providing feedback such as a community forum?

@whitwhoa great!
We talk on gitter. Let me know if this doesn't work for you, and we'll think of other means ;)


bump hamcrest to 0.1.1 - license clarification

Hi,
I wonder if hamcrest could easily be bumped to 0.1.1 ?
Currently, cargo use hamcrest-0.1.0 (dev-dependency). This particular version has some licenses issues:

cartes.io reports it as MIT
but published 0.1.0 crate doesn't contains any license information

Some months ago, hamcrest has officially switched to Apache2/MIT (see https://github.com/carllerche/hamcrest-rust/issues/22), and some days ago a 0.1.1 has been published with updated license (see https://github.com/carllerche/hamcrest-rust/issues/40).
Updating to 0.1.1 would permit a more easy distribution of prerequistes for cargo.
As example, the current status for OpenBSD is the following (from http://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/devel/cargo/Makefile?rev=1.4&content-type=text/x-cvsweb-markup):
# cargo is dual licensed MIT/Apache-2.0
# third-parties (detail below):
#  - MIT
#  - Unlicense/MIT
#  - MIT/Apache-2.0
#  - MPL-2.0
#  - one with no indication of licence (hamcrest)
#
# hamcrest is used only for the testsuite, and isn't builded with cargo.
# so the package doesn't contain it.
#
PERMIT_PACKAGE_CDROM = Yes
PERMIT_DISTFILES_FTP = No as hamcrest has no license information

we are unable to distribute source archives for cargo due to hamcrest dependency (cargo and dependencies are considere as a whole). Hopefully, binary-package is possible as hamcrest isn't builded with cargo (only used for testing).
Having cargo to depends to properly licensied dependencies would simplify the whole.
Thanks.

https://github.com/rust-lang/cargo/pull/3174 is updating some dependencies, but not this one. It shouldn't be difficult.

Sure! Want to send a PR to update?


feat: Multicall3 Implementation

Motivation
#328
Currently Alloy is missing multicall functionality. This PR aims to provide an initial implementation which can be iterated on and improved in the future.
It provides support for Multicall 1/2/3 and all of its associated read-only methods.
Solution
The logic is heavily inspired by the logic in ethers-rs.
A Multicall instance can be instantiated via Multicall::new(provider: Provider, Option<Address>).
Calls can then be added via add_call which simply appends to the the internal calls vector without returning anything, or via a builder pattern by chaining with_call.
The Multicall query is then sent by awaiting .call().
When adding calls, the functions expect a DynCallBuilder, which it then extracts the calldata, target, and decoder from. The returned output is:

If there is only 1 return value: A DynSolValue of the corresponding type.
More than 1 return value: A DynSolValue::Tuple containing the corresponding DynSolValue for each return value.

Originally I tried to make this work with a SolCallBuilder, but couldn't wrap my head around how to do this nicely due to SolCallBuilder instances being different types for each function. I wanted to avoid boxing and/or requiring a bunch of type assertions from the end user.
Due to the usage of dynamic/json sol types, this means that the multicall feature does require the alloy-sol-types/json feature, and so this is enabled whenever the multicall feature flag in the contracts crate is enabled.
PR Checklist

[x] Added Tests
[x] Added Documentation
[ ] Breaking changes

Closing as stale, as we have not reviewed in a timely manner. Thanks for your contribution. We may revisit this work in the future


chore: cloudform update, typescript version lock

Description of changes:

update to cloudform 4.1
lock typescript version to ~3.6.4 to block 3.7.2 update (cloudform needs a fix https://github.com/bright/cloudform/pull/36)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #2816 into master will increase coverage by 2.43%.
The diff coverage is 100%.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0fd0dd5...3d969b6. Read the comment docs.


[Feature] Configurable editor key-bindings

Is your feature request related to a problem? Please describe.
It's possible but very hard to override the default keybindings for panning, selecting, etc.
Describe the solution you'd like
I would like to have the possibility to change the default keybindings.
Additional context
WolvenKit

Fixed in #47


[LLVMCPU] Add LLVMCPU support for winograd.filter_transform op

This PR adds a tiling configuration for the winograd.filter_transform op in the LLVMCPU winograd pipeline.

This PR is based on https://github.com/openxla/iree/pull/17104. Please only review the last commit.


Problem with textures in filters since 4.2.1

Hello,
I have a filter that takes a texture as a uniform and uses it as a sampler2D in the fragment shader. This texture gets updated every frame.
The filter was working nicely in v4.1.1, since v4.2.1 the texture appears all black or as a mosaic of random colors. If I use a static texture which doesn't get updated then it works as expected.
What can be causing this? I've seen 4.2.1 introduced some changes in texture cacheing, should I tell the filter to update the texture on every frame or something similar?
Thanks.

Hi there! Can you try version 4.2.2?
If the issue still persists,  fiddle / example of the issue would be ace!
thanks!

Hello, sure.
This jfiddle is using PIXI v4.1.1: https://jsfiddle.net/nu2k4txr/7/
This is the same code using v4.2.2: https://jsfiddle.net/nu2k4txr/8/
It is late so I hope I'm not making any mistakes.. As you can see the texture is not visible with the latest version.
Thanks.

Oh, and if you comment the texture.update() on the 4.2.2 version and rerun the fiddle, it works as expected.

I have a similar problem in 4.2.2; I have a series of MovieClips (now called AnimatedSprite) which get reused (and their textures set) which no longer show anything.
There's a filter on top of them for clicking; maybe they're breaking it.

Hello! This is fixed in the latest version, https://jsfiddle.net/sjp2f43g/
Cheers!


Jsonld Export with new bioschemas types

This PR updates the jsonld/ro-crate export such that the output and contexts now use the new bioschemas types LabProcess and LabProtocol. Also, some structural changes were made to the jsonld output. The json decoders were changed accordingly, so that they can now read typical ISA-JSON or the new RO-Crate format. Each data type therefore has a fromJsonString and a fromJsonldString function.
The tests were adapted accordingly. However, they are not yet finished. The InvestigationLD tests need to be updated, since the current expected json strings are still using the old structure and terms. Furthermore, a new OutputMatchesInputLDtoLD test was added in the ProcessLD test list that reads the new jsonld string, writes it again and then checks for equality. Such tests should be added for all types (the existing ones read normal ISA-JSON and then write jsonld). @HLWeil pleas add/update these tests before merging.
Another part that needs revision are the readers for Material/Sample/Source. Currently, the type is not determined correctly when reading jsonld. The test ProcessInputLD - Sample - ReaderSuccessSimple is deactivated therefore.

Closing in favor of #330


dig returns two interfaces when querying zeronsd server - any way to restrict to zerotier interface?

I use a real DNS domain on all my hosts, using multiple split-horizons. On cloud instances in public and private subnets, the real domain is CNAME aliased to the public and private and domains.
This setup has been working fine for me for years, and I would prefer to keep it this way.
However, my zeronsd server, which is running on a cloud instance is giving out two IP addresses when I use dig to query it - one from the zerotier interface and another from the main interface used on its private network, as it also uses the same domain.
I know this is how DNS works, but is there a way to make it only give the address of the zerotier interface?

Sorry, I missed the following in the README.md:

Linux users are strongly encouraged to use systemd-networkd along with systemd-resolved to get per-interface resolvers that you can isolate to the domain you want to use. If you'd like to try something that can assist with getting you going quickly, check out the zerotier-systemd-manager repository.


Enable website publishing in CI

https://github.com/scalacenter/scalac-profiling/actions/runs/7441157666/job/20242939961 was automatically triggered when I merged this, but it seems to have failed for reasons I don't understand. @danicheg any insight?

@SethTisue Thank you!
In the logs, I see this
Cloning into 'scalac-profiling-gh-pages'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.

Have you enabled the Allow write access in the settings as specified in docs?

I did use an empty passphrase.
You mean GIT_DEPLOY_KEY, not GITHUB_DEPLOY_KEY. It appears correct to me at https://github.com/scalacenter/scalac-profiling/settings/secrets/actions

Have you enabled the Allow write access in the settings as specified in docs?

Yes, at https://github.com/scalacenter/scalac-profiling/settings/keys I see it says "Read/write"
Maybe the clone is failing because the gh-pages branch doesn't exist yet? Maybe you need to create it first?

Maybe the clone is failing because the gh-pages branch doesn't exist yet?

You mean GIT_DEPLOY_KEY, not GITHUB_DEPLOY_KEY. It appears correct to me at https://github.com/scalacenter/scalac-profiling/settings/secrets/actions

I cannot visit that page. But just to confirm, you've set up the GIT_DEPLOY_KEY env var, right? If so it might be the reason, because in CI, I expected this one ${{ secrets.GITHUB_DEPLOY_KEY }}

Yes, that must be the problem. I used GIT_DEPLOY_KEY because that's what https://scalameta.org/mdoc/docs/docusaurus.html#deploy-key suggested.
Hopefully https://github.com/scalacenter/scalac-profiling/pull/77 will fix it


How to mock GET request with Regex of URL ?

Hi,
The official method of $httpBackend.when support Regexp as url parameter.
How does ngDescribe permit mocking url via RegExp?

Hmm, need to think about that one.

We mock http using an object, which can only have string keys. Maybe we could introduce another syntax using an array of pairs
http: {
  get: {
    '/some/url': 42
  }
}

would be same as
http: {
  get: [
    '/some/url', 42,
    /full\/regex\/, [500, 'server error']
  ]
}

This syntax could work, but won't be very intuitive I think.
As a workaround, I'd use the native $httpbackend for Regex matching :
ngDescribe({
    name: 'sample $httpBackend',
    modules: ['module.name', ],
    inject: ['$httpBackend', ],
    tests: function (deps) {
        beforeEach(function () {
            deps.$httpBackend.whenGET(/.+GetContacts.*/).respond(function () {
                return [200, [{ "Id": 12, "Name":"Toto"}]];
            });
        });
    }
});

Yup, that is the way. I think you can even combine http with this syntax
http: {
  get': {
    '/some/url', 42
  }
},
tests: function (deps) {
  beforeEach(function () {
    deps.$httpBackend.when ...
  })
}


⚠ Processors has degraded performance

In 15e996d, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 114 ms

Resolved: Processors performance has improved in b0971de.


Adding in the ability to specify --junit and have the build produce junit xml output

Let me know what you think about this. The branch is pulled off of 2.1.0.rc.2 because there are 2 failing tests on master.
When I first got into the code there was a lot of magic going on. I could not tell what methods were being called on the "formatters" on purpose and what on accident.
With the way I implemented it I added a generic method_missing method to the fanout_formatter at first but after running the bwoken tests on a device I started getting errors that didn't make sense to me.
I then changed it so that the fanout formatter only receives messages that it is currently forwarding on, and the formatter can decide whether it wants to implement the method or not.
It would be very simple to do this for XUnit output as well if someone still wants that done.

Apparently when testing this I edited a file to have a require and forgot to commit it. I also noticed after the fact that the junit formatter does not need to produce output for the build running.
I'm closing this pull request and creating a new one that contains those changes.


Xamarin Forms Quickstart won't compile

Hi,
I followed your Xamarin Forms Quickstart Guide and tried to compile it.
However, Ammy is unable to compile the source due to a NullReferenceException being thrown.

1>------ Erstellen gestartet: Projekt: AmmyXamarinApp.Android, Konfiguration: Debug Any CPU ------
1>(1,1,1,1): error : Ammy compilation error: System.NullReferenceException: Der Objektverweis wurde nicht auf eine Objektinstanz festgelegt.
1>(1,1,1,1): error :    bei DotNet.TopConstructedTypeSymbol.EvalProperties(DependentPropertyEvalContext context)
1>(1,1,1,1): error :    bei Ammy.Backend.ReflectionBackend.CreateConstructedTypeSymbol(Type type)
1>(1,1,1,1): error :    bei Ammy.Backend.ReflectionBackend.GetTypeSymbol(Type type)
1>(1,1,1,1): error :    bei Ammy.Backend.ReflectionBackend.InitBaseTypeSet()
1>(1,1,1,1): error :    bei Ammy.Backend.ReflectionBackend.LoadExternalSymbols(IEnumerable`1 libs, String libPath, ICompilerMessages compilerMessages, DependentPropertyEvalContext context)
1>(1,1,1,1): error :    bei Ammy.Language.Start.RefreshReferences(Project project)
1>(1,1,1,1): error :    bei Ammy.Language.Start.RefreshReferences(CancellationToken _cancellationToken, Project project)
1>(1,1,1,1): error :    bei Ammy.Build.AmmyProject.RefreshReferences()
1>(1,1,1,1): error :    bei Ammy.Build.AmmyCompiler.Compile(AmmyProject project, Boolean generateMetaFile, Boolean needBamlGeneration, Boolean needUpdate)

I installed VS 2017 with Xamarin, created a new project, installed Ammy.XamarinForms, added a new page to my shared project (AmmyXamarinApp) and pasted in the sample code from your guide.
Is it an issue on my part or is it an Ammy related issue?

This is definitely an Ammy issue. I will take a look as soon as I get a chance.
Meanwhile, can you, please try using Ammy with PCL project? I'm not sure Ammy was ever meant to work with Shared project configuration.
Thanks!

Hi,
Thanks for the fast response.
I have created a PCL project and the syntax highlighting works now.
However, when trying to compile the sample, I get the compilation error that InitializeComponent( ) is not valid in the current context, that is moving it from App's constructor to MainPage's one.
When trying to compile the project with leaving the InitializeComponent call inside of App's constructor, I get the same compilation error from above, with the difference being that the PCL project itself compiles but the Android project doesn't.

2>------ Erstellen gestartet: Projekt: AmmyXamarinPCL.Android, Konfiguration: Debug Any CPU ------
2>    Processing: obj\Debug\res\layout\tabbar.xml
2>    Processing: obj\Debug\res\layout\toolbar.xml
2>    Processing: obj\Debug\res\values\styles.xml
2>(1,1,1,1): error : Ammy compilation error: System.NullReferenceException: Der Objektverweis wurde nicht auf eine Objektinstanz festgelegt.
2>(1,1,1,1): error :    bei DotNet.TopConstructedTypeSymbol.EvalProperties(DependentPropertyEvalContext context)
2>(1,1,1,1): error :    bei Ammy.Backend.ReflectionBackend.CreateConstructedTypeSymbol(Type type)
2>(1,1,1,1): error :    bei Ammy.Backend.ReflectionBackend.GetTypeSymbol(Type type)
2>(1,1,1,1): error :    bei Ammy.Backend.ReflectionBackend.InitBaseTypeSet()
2>(1,1,1,1): error :    bei Ammy.Backend.ReflectionBackend.LoadExternalSymbols(IEnumerable`1 libs, String libPath, ICompilerMessages compilerMessages, DependentPropertyEvalContext context)
2>(1,1,1,1): error :    bei Ammy.Language.Start.RefreshReferences(Project project)
2>(1,1,1,1): error :    bei Ammy.Language.Start.RefreshReferences(CancellationToken _cancellationToken, Project project)
2>(1,1,1,1): error :    bei Ammy.Build.AmmyProject.RefreshReferences()
2>(1,1,1,1): error :    bei Ammy.Build.AmmyCompiler.Compile(AmmyProject project, Boolean generateMetaFile, Boolean needBamlGeneration, Boolean needUpdate)
2>c:\users\prakti\documents\visual studio 2017\Projects\AmmyXamarinPCL\packages\Ammy.1.2.74\build\Ammy.targets(16,5): error : Ammy compilation failed

You only need to install Ammy.XamarinForms to a PCL. So try uninstalling it from an Android project.

Hi,
Thanks for another quick reply.
I'm new to Xamarin and C# in general, sorry for any mistakes on my end.
I have removed Ammy.XamarinForms from the Android and the IOS project while keeping it for the PCL one.
Now I get a different error:

1>c:\users\prakti\documents\visual studio 2017\Projects\AmmyXamarinPCL\packages\Ammy.1.2.74\build\internal.cs(4,16,4,28): error CS0246: The type- or namespacename "AmmySidekick" not found (is a using directive or assembly reference missing?)

Where can I find that to reference it in my project?

No problem at all.
It seems that Ammy package is referenced somewhere where Ammy.XamarinForms is not. I would guess that you have uninstalled Ammy.XamarinForms, but didn't uninstall Ammy.

Hi,
Thanks for your response.
You were correct, Ammy was still installed.
Thanks for your help resolving my issue.

@ionoy Does this mean Ammy can't be used at all with a Xamarin shared project? I'm trying to convert my Xamarin shared project to use Ammy, but I'm also getting the Ammy compilation error: System.NullReferenceException: Object reference not set to an instance of an object. error.

NullReferenceException can mean a lot of things, I'm not sure what it means in your case. Did you install Ammy.XamarinForms package? In a Shared project configuration, you should install it into the platform project (Android).

@ionoy  I did. I've noticed that, while it works fine with WPF, with Xamarin Form the syntax highlighting won't work on .ammy files and the files won't build on save either, which leave me with an empty .g.xaml file. I definitely have the extension installed as well.

Can you send me a sample project that I can run on my machine?
Thanks!

Hey @JohnLouderback
Sorry for answering so late. I've looked at the project and found a few issues.
First of all, I don't see Ammy.XamarinForms package installed in the Android project.
Secondly, it seems, that you've added Page1.ammy manually. Although there is not much difference, you should still add it with the Add New dialog.
Third, is that this is a Shared project configuration and Ammy wasn't properly tested with one.
Unfortunately, I can't build this project at all right now. MSBuild gives me this weird error: error MSB4166: Child node "2" exited prematurely. Shutting down. I'm not sure how to fix it, and I don't want to reinstall the Visual Studio at the moment.

@ionoy  Does Ammy not work with iOS projects? I believe that's where I installed the Ammy.XamarinForms package


SAM-2900: Show scoring option to student if they have been allowed retakes

https://jira.sakaiproject.org/browse/SAM-2900

This looks good to me. Of note, we've been running this code in production for about a month.


Where can I get the input and execution's files for openFAST 2.5.0 ? Nothing available on github for this version

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

Describe the solution you'd like

Describe alternatives you've considered

Additional context

https://github.com/OpenFAST/openfast/releases/tag/v2.5.0
https://github.com/OpenFAST/r-test/tree/v2.5.0


Binding key Q to the running the query in explore view

Coverage decreased (-0.2%) to 81.167% when pulling ac628aa216accd25072304891a21980dcea37d1f on mistercrunch:run_q into 4191b75966bdfec22a4dcb04eb62f6884765d55d on airbnb:master.

Coverage decreased (-0.2%) to 80.917% when pulling 77a5c90a254fa3c8ab15cfb7b2c5d7869f09e9e4 on mistercrunch:run_q into a016d181d7f084fbad6b9907b250a981a895b773 on airbnb:master.

Coverage decreased (-0.2%) to 80.923% when pulling 2504576acb1e75b490428119692aeb721ebf7624 on mistercrunch:run_q into 1a952a496102b28cf60d9b31d53ded28337e8971 on airbnb:master.


Push new minor release of crossbeam-queue

Would it be possible to publish a new minor version of crossbeam-queue on crates.io? The 33e4c64a change cleaned up the license files. crossbeam-utils, crossbeam-channel and crossbeam-epoch have been released with this change but not crossbeam-queue. Thanks.

Published crossbeam-queue 0.3.2

Great, thanks!


unable to download resource-agents

Hi,
I'm following the documentation for the NFS creation but I'm stuck at "sudo zypper in resource-agents" because my SLES is unable to find the package...
I'm running SLES 12-sp4 (BYOS)
Linux version 4.12.14-95.54-default (geeko@buildhost) (gcc version 4.8.5 (SUSE Linux) ) #1 SMP Thu Jun 4 12:49:28 UTC 2020 (892ef1f)
Thank you!

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

ID: a20453e6-5273-2119-69f7-85b5108a8f7d
Version Independent ID: 4ac93e53-6969-4c8d-5b5c-0db47a8976ce
Content: Setting up Pacemaker on SLES in Azure
Content Source: articles/virtual-machines/workloads/sap/high-availability-guide-suse-pacemaker.md
Service: virtual-machines-windows
GitHub Login: @rdeltcheva
Microsoft Alias: radeltch

@Heloiseboanerges Thank you for your query, our team will look into it and get back to you at the earliest.

@rdeltcheva Can you please add your comments.

Sharing with you the output message

I've found the error, I was using the bare SLES. I've changed the image of the VMs to use SLES 12 SP4 for SAP Applications and the agents are there.


[move] Loc::file is a move_symbol_pool::Symbol

Use the newly added move_symbol_pool module from #8782 to represent file names in Move locations. This allows us to eliminate the use of "leak string" functions, and re-enable libc leak detection in LeakSanitizer.

LEC dependency change summary:
target packages:
  A symbol-pool 0.1.0 (workspace, path 'language/symbol-pool')
    * features: default

release binaries dependency change summary:
target packages:
  A symbol-pool 0.1.0 (workspace, path 'language/symbol-pool')
    * features: default

LEC dependency change summary:
target packages:
  A symbol-pool 0.1.0 (workspace, path 'language/symbol-pool')
    * features: default

LEC dependency change summary:
target packages:
  A symbol-pool 0.1.0 (workspace, path 'language/symbol-pool')
    * features: default

release binaries dependency change summary:
target packages:
  A symbol-pool 0.1.0 (workspace, path 'language/symbol-pool')
    * features: default

LEC dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

release binaries dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

LEC dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

release binaries dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

release binaries dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

LEC dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

LEC dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

release binaries dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

LEC dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

release binaries dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

LEC dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

release binaries dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

LEC dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

release binaries dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

Phew, sorry this ended up taking a little while @tnowacki, but I think this and the underlying #8782 are ready for review (and re-review) ♻

release binaries dependency change summary:
target packages:
  A move-symbol-pool 0.1.0 (workspace, path 'language/move-symbol-pool')
    * features: default

/land

I wonder if this may have broken source maps, since Loc::file is now serialized as an address value of type NonZeroU64. When deserialized in a separate process, that address will no longer be valid. I believe I will need to implement custom serde serialization/deserialization for Symbol. I'm working on a pull request for that now. (Also I'll try to add some tests that would catch something like this!)


Add dockerfile for releasing clients

Signed-off-by: Sayali Gaikawad<EMAIL_ADDRESS>Description
This image will be used in standard clients release pipeline. Incremental changes will be made as and when required to add new packges.
Issues Resolved
Part of https://github.com/opensearch-project/opensearch-build/issues/1606
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #2667 (0d84dd1) into main (0bbeeb2) will increase coverage by 5.75%.
The diff coverage is n/a.

:exclamation: Current head 0d84dd1 differs from pull request most recent head 901acb4. Consider uploading reports for the commit 901acb4 to get more accurate results

@@             Coverage Diff             @@
##             main     #2667      +/-   ##
===========================================
+ Coverage   94.24%   100.00%   +5.75%
===========================================
  Files         162         6     -156
  Lines        4311       105    -4206
  Branches       19        19
===========================================
- Hits         4063       105    -3958
+ Misses        248         0     -248

Impacted Files
Coverage Δ

src/ci_workflow/ci_args.py

src/assemble_workflow/bundle_rpm.py

src/run_ci.py

src/ci_workflow/ci_check_package.py

...orkflow/ci_check_gradle_dependencies_opensearch.py

...w/perf_test/perf_test_runner_opensearch_plugins.py

src/build_workflow/build_artifact_check.py

src/build_workflow/build_artifact_checks.py

...orkflow/integ_test/integ_test_runner_opensearch.py

...ts_workflow/component_opensearch_dashboards_min.py

... and 146 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

My problem with this file is, it is very similar to the normal opensearch centos7 build image.
What are the additional changes that clients need?
I would even suggest a alpine3 image if they have limited differences or just use the same build image.

As stated in the PR description changes will be incremental. Starting with what we have but there are 9 clients and other components that will need different version of nodejs, different java versions, etc that will be added in this file. Also changes in opensearch image should be affected to others and vice versa hence keep if different.

As stated in the PR description changes will be incremental. Starting with what we have but there are 9 clients and other components that may need different version of nodejs, different java versions, etc that will be added in this file. Also changes in opensearch image should be affected to others and vice versa hence keep if different. I'll start working on npm client next that will give me some idea what all changes will go in here. Also OpenSearch ci image is more for build and this one is more for release. I'll add / remove things going forward.

Sure, please create an image in staging hub with this naming:
ci-runner-centos7-clients-v1 and ci-runner-centos7-clients-v1.1.
Next time if there is any update just create ci-runner-centos7-clients-v1.2 and copy v1.2 to v1.
We will merge this PR once the image is online.
Thanks.

Thanks will be naming it release-centos7-clients-v1 since ci-runner is already the name of the repository. No need to state that again. The final image with repo will say:
opensearchstaging/ci-runner:release-centos7-clients-v1
opensearchstaging/ci-runner:release-centos7-clients-v1.1


Feature Request: Show extremas (min, max)

Would it be possible to add an option to the main show option to show the extremas, mix/max values, below the legend? Similiar to what you can do int the mini-graph card:

I don't think I could fit that into this card. How would that look with multiple entities?
On mini-graph-card it only applies to the first entity. I don't want to give specific treatment to the first entity vs the others.

Can't use the extramas of all entities, and then add an in_extrema option to include/exclude some of them? Just as you do with in_chart and in_header?

Maybe, I'll think about it

Just a 2c from me.
I'm using mini-graph mostly for showing temp and humidity on the same graph. While I can show max/min values for one of them, you can imagine I'm missing an option to show the same for the second one.
I admit it's impossible to show those data for all values in general, considering there might be more entities  (maybe in form table somewhere next to graphs but for sure not incorporated in graphs layout). But somehow I'm missing such the feature.
Well.. At the end I used to not having it anymore ;)

Maybe min and max could be indicated by an annotated line at top/bottom?

At a slight tangent is it possible to expose the min / max values so they can be used as parameters in the transform function?

At a slight tangent is it possible to expose the min / max values so they can be used as parameters in the transform function?

No, because transform is applied way before we can know what the min and max is.

Actually sort of already possible e.g
'''

entity: sensor.ude_zb_temperature
curve: smooth
color: blue
name: Min
type: line
group_by:
duration: 24h
func: min
stroke_width: 1
'''

Would be nicer if the line could be dashed and the value displayed inside the graph instead of in the legend though

So I've been experimenting with some stuff. What do you think about showing min-max like this?

I think the top one looks amazing!

I prefer the top one as well. With logic so if multiple series they won't overwrite too?

With logic so if multiple series they won't overwrite too?

Not sure that is handled by ApexCharts, I'd have to investigate, but I have little hope.

Of the two I like the one on top best.
I still think a footer would be be useful.
Perhaps by adding a new footer option (similar to what you have in the entities card) supporting basically the same options as the header option, but it shows up below the graph. And adding an in_footer option to each serie in which you can specify what should be shown in the footer. Maybe something like:
in_footer:
show: true/false/max/min/avg/"func" - not sure what true means
title: "some text"
time: true/false                                     - probably only for max/min or other single point values connected to a specific time
Not sure if it is doable though.

I prefer the top one as well.

I guess at some point I could add a non-ApexCharts legend with those features (min, max, avg, last) like grafana would do:

But it's going to take time and is going to be compute heavy :)

So I've released a beta with extremas support like so:

I'll keep this issue opened to work later on the more advanced legend feature.

After running a few graphs with extremas turned on I am starting to like it (though my original feature request still stands :-).
Would it be possible to add the time (and perhaps the unit) to the label, so that you don't have to hover over the point to see the time (hovering over the label does not do anything)? Or is that too much work?

So I initially experimented with adding the time also, but the label doesn't support to be multi-line/I didn't find a way to make it multi-line so it gives you massive labels.

So I found a way to do something like that. It takes quite some space though, so...

I have not yet made my mind up what I think about multiline labels. It depends on what the graph are used to show I guess.
Another Idea. ;-)
Would it be possible to use strftime options to specify the time and/or the whole label? Maybe something like:
extremas: "Value: %#\n%H:%M:%S"
Where %# indicates the value (I don't think it is used by strftime) and "\n" for splitting the line in two. Using this method it would also be possible to add arbitrary strings to the label.


In Project details page the contract file uploaded is not alligned

Describe the task
The Project details page contains the contract file uploaded during the creation of the project. Currently, the project is displayed as available in the screenshot

Steps to create a project with a contract

Go to CRM
Click on Projects, then click on Add new project
Fill the form and select status as Active, upload a pdf file in the Upload Contract field, then click on the Save button
Click on the project created in step 3

Expected behavior
The alignment of the contract file should be as shown in the screenshot below

References
Specify any reference material / links

This issue PR is merged. Closing it.


Iconcolor doesn't change for custom light themedata.

final lightTheme = ThemeData.light().copyWith(

 brightness: Brightness.light,
    scaffoldBackgroundColor: Color(0xfff5f5f5),
    iconTheme: IconThemeData(
      color: Colors.blueAccent.shade100, // Doesn't change the icon color when light theme is activated.
    ),
    ),

final darkTheme = ThemeData.dark().copyWith(

 brightness: Brightness.dark,
    iconTheme: IconThemeData(
      color: Colors.green, // Icon color changes when dark theme is activated.
    ),
    ),

Hi @Sh00ny
This seems to be working fine on the latest stable channel, when I switch from dark to light and light to dark, icon color changes on Android

minimal code sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      theme: ThemeData.light().copyWith(
        iconTheme: IconThemeData(
          color: Colors.red
              .shade700, // Doesn't change the icon color when light theme is activated.
        ),
      ),
      darkTheme:ThemeData.dark().copyWith(
        iconTheme: IconThemeData(
          color: Colors.blue
              .shade700, // Doesn't change the icon color when light theme is activated.
        ),
      ),
      home: Home(),
    );
  }
}

class Home extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Material App Bar'),
      ),
      body: Center(
        child: Icon(Icons.flight)
      ),

    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.0, on Microsoft Windows [Version 10.0.19043.985], locale en-US)
    • Flutter version 2.2.0 at C:\Users\Taha\Code\flutter_stable
    • Framework revision b22742018b (9 days ago), 2021-05-14 19:12:57 -0700
    • Engine revision a9d88a4d18
    • Dart version 2.13.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\Taha\Code\SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_SDK_ROOT = C:\Users\Taha\Code\SDK
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java.exe
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.9.6)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.9.31313.79
    • Windows 10 SDK version 10.0.19041.0

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[✓] VS Code (version 1.56.2)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.22.0

[✓] Connected device (5 available)
    • Redmi K20 Pro (mobile)  • def0ad20      • android-arm64  • Android 10 (API 29)
    • sdk gphone x86 (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
    • Windows (desktop)       • windows       • windows-x64    • Microsoft Windows [Version 10.0.19043.985]
    • Chrome (web)            • chrome        • web-javascript • Google Chrome 90.0.4430.212
    • Edge (web)              • edge          • web-javascript • Microsoft Edge 90.0.818.66

! Doctor found issues in 1 category.

Can you please provide your flutter doctor -v and device details

`shoony@Shoonys-MacBook-Air brief9_pro % flutter doctor -v
[✓] Flutter (Channel stable, 2.0.4, on macOS 11.3.1 20E241 darwin-arm, locale en-US)
• Flutter version 2.0.4 at /Users/shoony/Developer/flutter
• Framework revision b1395592de (8 weeks ago), 2021-04-01 14:25:01 -0700
• Engine revision 2dce47073a
• Dart version 2.12.2
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
• Android SDK at /Users/shoony/Library/Android/sdk
• Platform android-30, build-tools 30.0.3
• Java binary at: /Applications/Android
Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
✗ Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/macos#android-setup for more
details.
[✓] Xcode - develop for iOS and macOS
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.5, Build version 12E262
• CocoaPods version 1.10.1
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 4.2)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
[✓] VS Code (version 1.53.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.19.0
[✓] Connected device (2 available)
• iPhone 12 Pro Max (mobile) • A36AE527-A2F7-4397-B693-77C6C944F11D • ios
• com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
• Chrome (web)               • chrome                               •
web-javascript • Google Chrome 90.0.4430.212
! Doctor found issues in 1 category.
`

shoony@Shoonys-MacBook-Air brief9_pro % flutter doctor -v
[✓] Flutter (Channel stable, 2.0.4, on macOS 11.3.1 20E241 darwin-arm, locale en-US)
    • Flutter version 2.0.4 at /Users/shoony/Developer/flutter
    • Framework revision b1395592de (8 weeks ago), 2021-04-01 14:25:01 -0700
    • Engine revision 2dce47073a
    • Dart version 2.12.2

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/shoony/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more
      details.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.53.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.19.0

[✓] Connected device (2 available)
    • iPhone 12 Pro Max (mobile) • A36AE527-A2F7-4397-B693-77C6C944F11D • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • Chrome (web)               • chrome                               •
      web-javascript • Google Chrome 90.0.4430.212

! Doctor found issues in 1 category.

Hi @Sh00ny
Can you please upgrade to the latest  stable and try again
Execute the following commands to upgrade
flutter channel stable
flutter upgrade --force
flutter doctor -v


ERROR: [TV4] Failed to parse JSON  https://www.tv4play.se/

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.11.11 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Sweden
Provide a description that is worded well enough to be understood
Trying to download an episode from www.tv4play.se gives ERROR Failed to parse JSON
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-vU', '--cookies-from-browser', 'chrome', 'https://www.tv4play.se/program/bingolotto/del-15-s%C3%A4song-42/13767129']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.11.11 [8b64402] (win_x86_exe)
[debug] Python 3.7.9 (CPython AMD64 32bit) - Windows-10-10.0.22000-SP0 (OpenSSL 1.1.1g  21 Apr 2020)
[debug] exe versions: ffmpeg 4.2.1, ffprobe 4.2.1
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.09.24, mutagen-1.46.0, sqlite3-2.6.0, websockets-10.4
[Cookies] Extracting cookies from chrome
[debug] Extracting cookies from: "C:\Users\Anders Ripa\AppData\Local\Google\Chrome\User Data\Default\Network\Cookies"
[debug] Found local state file at "C:\Users\Anders Ripa\AppData\Local\Google\Chrome\User Data\Local State"
[Cookies] Extracted 2994 cookies from chrome
[debug] cookie version breakdown: {'v10': 2994, 'other': 0, 'unencrypted': 0}
[debug] Proxy map: {}
[debug] Loaded 1723 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.11.11, Current version: 2022.11.11
yt-dlp is up to date (2022.11.11)
[debug] Using fake IP <IP_ADDRESS> (SE) as X-Forwarded-For
[debug] [TV4] Extracting URL: https://www.tv4play.se/program/bingolotto/del-15-s%C3%A4song-42/13767129
[TV4] 13767129: Downloading video info JSON
ERROR: [TV4] 13767129: 13767129: Failed to parse JSON (caused by JSONDecodeError("Expecting value in '': line 1 column 1 (char 0)")); please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
  File "yt_dlp\extractor\common.py", line 674, in extract
  File "yt_dlp\extractor\tv4.py", line 81, in _real_extract
  File "yt_dlp\extractor\common.py", line 1034, in download_content
  File "yt_dlp\extractor\common.py", line 1004, in download_handle
  File "yt_dlp\extractor\common.py", line 994, in parse
  File "yt_dlp\extractor\common.py", line 981, in _parse_json
  File "yt_dlp\extractor\common.py", line 964, in __print_error

  File "yt_dlp\utils.py", line 607, in decode
  File "json\decoder.py", line 337, in decode
  File "json\decoder.py", line 355, in raw_decode
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "yt_dlp\extractor\common.py", line 979, in _parse_json
  File "json\__init__.py", line 361, in loads
  File "yt_dlp\utils.py", line 610, in decode
json.decoder.JSONDecodeError: Expecting value in '': line 1 column 1 (char 0)

See #5535.


Add padding to last chunk instead of ignoring it

Problem
If the duration of the last chunk is lower than the desired duration, the chunk is ignored.
Idea
Pad the last (incomplete) chunk with zeros. This mainly affects FileAudioSource and AudioFileReader. It is important that num_chunks() (that estimates the number of chunks without actually doing the chunking) also counts this incomplete chunk.

Implemented in PR #55


Terraform examples to show people how to set up "best practices" infrastructure, how to use Rancher Terraform provider

I think we should include Terraform files that would basically show examples of how you could set up infrastructure that matches what is recommended in the best practices guide.
We could put this in the best practices guide and show different examples for different major cloud providers.
This could help people troubleshoot problems, especially if they use Terraform, because then they could figure out what might be wrong by comparing their Terraform vs. Rancher's examples.
It would be good to also include illustrative examples of how to use the Rancher Terraform provider.

Might be able to leverage some of the Terraform developed for quickstart - https://github.com/rancher/quickstart

relevant links:
https://github.com/rancher/onboarding-content/tree/master/phase_I/automate_ha_deployment
https://github.com/rancher/quickstart/pull/54


subsys/storage/flash_map: DT independent flash area access API

subsys/storage/flash_map: DT independent flash area access API
This commit provides DT independent flash area access API. This API
is supposed to provide compile time generated flash area access and
information, the same way the DT API did, but that is easier to replace
and using labels instead of identifiers
Enums are generated as identifiers.
Signed-off-by: Dominik Ermel<EMAIL_ADDRESS>Note: This changes are placed on top of changes to gen_defines and util.h that are drafted here:
https://github.com/zephyrproject-rtos/zephyr/pull/24514
https://github.com/zephyrproject-rtos/zephyr/pull/24847
https://github.com/zephyrproject-rtos/zephyr/pull/24848
~~https://github.com/zephyrproject-rtos/zephyr/pull/24503~~

Please move the macro definitions into <storage/flash_map.h> unless there's a good reason not to.
I remember there was some discussion about two APIs serving the same general purpose, with one of them preferred over the other. If <storage/flash_map.h> is not the preferred one then more thought is required.

@mbolivar-nordic Wasn't the case that the flash_map API should be deprecated?

The work has been abandoned.


Accept Input Source Maps

The typescript compiler produces sources maps mapping from the javascript it produces to the typescript sources it compiled.  This is great, but doesn't go far enough if there are tools which generate the sources that are consumed by the typescript compiler.
For example, Tsickle preprocesses typescript sources before passing them to the typescript compiler.  As things stand we've had to implement a hack to get our source maps to work.  We use a compiler host which performs the tsickle transforms as the typescript compiler reads files, saves the resulting source maps and then compose them with the source maps produced by the typescript compiler as they're emitted.
Instead the typescript compiler could accept source maps as input alongside sources, and compose the input source maps with the source maps produced by the compiler.
The ideal api would be that the typescript compiler would look for either a. an inline source map as a comment in the source file or b. a file in the same directory with the name {source file name}.ts.map, which shouldn't require any crazy file resolution.

For Tsickle, i would say the better way is to implement it as a transformation on the compiler. this saves you the double pass in the compiler. https://github.com/Microsoft/TypeScript/pull/13940 exposes the APIs needed to achieve this.

Tsickle may not be the best example.  The actual motivating reason for this issue is the Angular compiler is going to produce source maps in typescript it compiles from templates (html).

Another example is the html to tsx a templating tool that I use (yet to be released). Without sourcemaps debugging is rather difficult.

This is also something I would be interested in - I'm working on a TypeScript project that involves writing a small DSL which I would like to transpile to TypeScript code. With input source maps I could presumably just write a source map like you'd do for any other transpiler, and then TSC could use it as input so that the final JS could be mapped nicely back to the original DSL code.
While I could always just make the DSL compile directly to JS too, I would really like to do it with TS instead so that the code can be type checked for possible scripting errors.

I was looking for some tool that can bundle any kind of multiple source files into a single file, transpile new JavaScript to something suitable for today's browsers and then minify all that. uglify-es can't bundle or transpile. rollup can't transpile or minify. babel can't do source maps. tsc can't bundle or do source maps either. I'm about to give up on JavaScript. Or I might revert to some stupid text-replace bundling (which is really just some full-file include) and pass that to uglify, but I'd need to create a source map for that myself then. Not production-ready.

@ygoe There is a tool for concat multiple source files
https://www.npmjs.com/package/tool-cat-with-sourcemap
#!/usr/bin/env node

const {SourceNode, SourceMapConsumer, SourceMapGenerator} = require('source-map')
const fs=require('fs')
const assert=require('assert').strict
const args=require('yargs')
      .usage('Usage: $0 -s [output/sourcemap] -o [output] <file> ...')
      .help('h').alias('h', 'help')
      .example('$0 -s sourcemap.json -o c.ts a.ts b.ts','concat a.ts and b.ts, write c.ts and sourcemap.json')
      .demandOption(['s','o'])
      .argv
const files=args._
assert(files.length>0)

const sn = new SourceNode(
    1,
    1,
    "global",
    files.map(file=>new SourceNode(1,1,file,[fs.readFileSync(file,'utf-8')])))

const x=sn.toStringWithSourceMap({file: ''})
fs.writeFileSync(args.s, x.map.toString())
fs.writeFileSync(args.o, x.code)

As mentioned in #34608, it would be useful if tsc, when accepting input sourcemaps, not only generated output sourcemaps based on them, but provided error reporting that used the sourcemaps as well.

Here's another use case:
We use tsc to compile our TS code to js, which we then deploy to our production servers. But before actually sending it to the user we run a second pass (on the production server, not the build server) on the compiled and minified code to inject translations into the files.
Since we're already heavy users of TS transforms it would be handy to run this process with TS. But for that to be viable we'd need TS to accept the source maps generated during build as input.

Hi guys:
2 weeks ago I was facing the same problem and trying to find the solutions. The search brought me to this topic, and then I realized after so many years the problem is still not well solved.
So I decided to solve it myself. Now I finished this minify-ts.
github link
It can minify the ts file, and merge the source map files generated in several steps.
The minifying is slow because it relies on the TypeScript Language Service find-reference feature.
But the merging implementation is quite efficient, it can batch process all the source-map files in a folder, climb up through every source-map chain to the top to collect information, and use the merged result to overwrite the tail one.
Welcome to have a try and report bugs. Hope this solves your problem.

@foreverflying While I prefer your approach to minification, I have to mention there is also tsickle used by Google for the same purpose.

Running into a use case that would be well served by this today when converting YouTube's HTML template files to TypeScript at build time

I've discovered that input source maps were also requested in https://github.com/Microsoft/TypeScript/issues/26843
I wonder if there are more...
The tsickle use case no longer matters to us at Google, but there are others and more to come.
We would like to generate TypeScript files, but we need to have source maps that refer back to the original source from which they were generated.
Surely we are not the only people interested in this. TypeScript is mature enough now that surely lots of people would like to automatically generate it.
We would likely be able to provide some engineering effort.
Would the TypeScript team be willing to collaborate on a design and implementation?

Me too.
I'm writing a term-rewrite (hygienic macros) system for typescript.  During rewriting/expanding a file, the plan is to keep track of the source of each node in the AST whether it originated from the same file or came from another file. The end AST (representing  typescript code with no macros) can have a mix of sources.  My initial thought was to produce the .ts file with sourcemaps and let tsc/tsserver/whatever deal with the .ts files either for type checking or bundling or whatever they wanna do with it.
Without input sourcemap supprt, I will have to:

integrate very deeply with the typescript compiler for the purpose of producing meaningful error messages and sourcemap files (both of which I have to map back to the user's initial code).  This is wasteful and fragile.
produce 4 files .js/.js.map/.d.ts/.d.ts.map instead of just a .ts file with sourcemap. This means that I went from being a term-rewrite for typescript to being a full compiler. This is really out of scope of rewrite-ts.

So no, can't do that. For now, I will keep my project's scope and produce .ts files only, hoping that this issue is resolved in time.
Lots of projects today compile TO typescript and more will come in the future. Supporting input sourcemaps in the typescript compiler will help all of these projects.
Thanks for all the good work.

https://github.com/azizghuloum/rewrite-ts-visualized/ (repo)
https://azizghuloum.github.io/rewrite-ts-visualized/ (playground)

FWIW if you're already generating .ts and .ts.map files, you can combine TypeScript's .js.map output files and your provided .ts.map files with https://www.npmjs.com/package/@ampproject/remapping; this package is exactly what Babel uses to implement its own input source map functionality.
See also:

https://github.com/babel/babel/blob/e66517c1765cf6fb2c39ea0525b1a5d2141ab421/packages/babel-core/src/transformation/file/generate.ts#L65
https://github.com/babel/babel/blob/e66517c1765cf6fb2c39ea0525b1a5d2141ab421/packages/babel-core/src/transformation/file/merge-map.ts#L2

@jakebailey right, but that's assuming the end destination is a js file on disk and that I want to take care of the entire pipeline.  I honestly have no idea what people would do with the ts file.  I want to generate it and get out of the way.
If I'm generating .ts file with embedded sourcemaps, it might be ready for use by most tools today (say, deno, bun, vite, etc.) that would strip out the types and bundle/stream/run the code and probably respect the embedded sourcemaps.  In the future, browsers would also load the ts files with their sourcemaps and do the right thing.  The microsoft typescript compiler is not the only consumer of ts files today.
Also, if I'm generating the .ts file with embedded sourcemaps, and the file goes to tsc or tsserver, any diagnostics produced should be in terms of the original source locations, not the generated source code, just like how runtime stack traces go back to the original source, not the generated ones.

What many tools need to do is being able to map a location (e.g. from an error message) to the original document position.
This can be achieved outside of TypeScript by mapping the location via the sourcemap to the original location.
However, this is something many different tools need to manage by themselves, and it is kind of a hassle. The complicating matter is that it could be hard for each tool-developer to identify all places where TypeScript are exposing source code locations, and to remember to map those locations. (Or even worse, if the location would end up in text form in some message, the consumer would have to parse the message format)
I would classify this as a convenience feature, but I can imagine that most tools generating typescript code would in some way need this functionality (DSLs, TS-generating tools, etc.).
I need it for two tools I am maintaining.


feat(map): cluster&rotate bus

I published new version of generated client: https://www.npmjs.com/package/open-bus-stride-client v0.1.3
it should contain any new models and apis.

@ReallyLiri thaks for the review! Great comments, I'll fix it tomorrow in the workshop 🥳


testing.md created

@caglarcercinli  we can use fake api for mock data ie faker, etc

#7  basic tutorial added for testify

@caglarcercinli we can use fake api for mock data ie faker, etc

during the unit testing I assume?

@caglarcercinli we can use fake api for mock data ie faker, etc

during the unit testing I assume?

for testing or when we need mock records during development, i do like working a data which look like actual data


Make LazyListState.visibleItemsCount public

Hi, is it possible for you guys to change LazyListState.visibleItemsCount from internal to public? It's a very useful API that we have to currently workaround by calculating it ourselves, in a very convoluted way, as we have no reach to the internals.
https://github.com/JetBrains/androidx/blob/bc4f98ccbba27171f1bc419f2fb5b920401bc781/compose/foundation/foundation/src/commonMain/kotlin/androidx/compose/foundation/lazy/LazyListState.kt#L125-L128

@jimgoog what do you think about it?

Or am I meant to use this API instead? 🤔 https://github.com/JetBrains/androidx/blob/bc4f98ccbba27171f1bc419f2fb5b920401bc781/compose/foundation/foundation/src/commonMain/kotlin/androidx/compose/foundation/lazy/LazyListLayoutInfo.kt

I could be wrong but wouldn't it be possible to calculate the value from the lazy list state by just doing state.layoutInfo.visibleItemsInfo.size? I would assume the visible items info amount would match the corresponding value you're hoping to get, but I could be wrong.

Yes, that's what my question above was implying. I just found out about state.layoutInfo, which returns LazyLayoutListInfo. I didn't know about it when I was posting this issue.
So I assume state.layoutInfo.visibleItemsInfo.size is the way to go, right? I'll close this issue now.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Add icon for dark themes

https://stackoverflow.com/questions/58880234/toggle-chrome-extension-icon-based-on-light-or-dark-mode-browser


UI: notification for address->clipboard copy status

I clicked it... did it copy to clipboard? Did it fail? Have to paste to find out...

fixed in https://github.com/ethereumproject/emerald-js-ui/pull/9


issue when query a single CSV file using SQL on-demand  in Azure Synapse Analytics

Dear Sir:
I followed the doc https://docs.microsoft.com/en-us/azure/synapse-analytics/sql/query-single-csv-file?toc=/azure/synapse-analytics/toc.json&bc=/azure/synapse-analytics/breadcrumb/toc.json
to test the feature. however  I encountered this error
Failed to execute query. Error: Content of directory on path 'https://sqlondemandstorage.blob.core.windows.net/csv/population/*.csv' cannot be listed.
and I tried other part for parquet files ,the same error returned, I wonder any privilege missing?
Document Details
https://docs.microsoft.com/en-us/azure/synapse-analytics/sql/query-single-csv-file?toc=/azure/synapse-analytics/toc.json&bc=/azure/synapse-analytics/breadcrumb/toc.json
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c9e9fbf8-bd1e-d52c-2057-ef9ab326b2d6
Version Independent ID: 77d21a9a-2ce3-4e21-584b-0c1f5d62a318
Content: Using SQL on-demand (preview) - Azure Synapse Analytics
Content Source: articles/synapse-analytics/quickstart-sql-on-demand.md
Service: synapse-analytics
Sub-service: ****
GitHub Login: @azaricstefan
Microsoft Alias: v-stazar

@jianyeliu Thanks for bringing this to our notice. We are investigating into the issue and will update you shortly.

Hi @jianyeliu ,
Please check the First Time Setup link in the Prerequisites section at the start of the article.
This section in First Time Setup explains it in the note:

In order to successfully run How To's in this section you have to use SAS token.
To start using SAS tokens you have to drop the UserIdentity which is explained in the following article.
SQL on-demand by default always uses AAD pass-through.

Thanks for your reminder. It works 😊
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Stefan Azarić<EMAIL_ADDRESS>Sent: Monday, May 4, 2020 10:05:36 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: jianyeliu<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] issue when query a single CSV file using SQL on-demand in Azure Synapse Analytics (#53951)
Hi @jianyeliuhttps://github.com/jianyeliu ,
Please check the First Time Setup link in the Prerequisites section at the start of the article.
This sectionhttps://docs.microsoft.com/en-us/azure/synapse-analytics/sql/query-data-storage#create-credentials in First Time Setup explains it in the note:
In order to successfully run How To's in this section you have to use SAS token.
To start using SAS tokens you have to drop the UserIdentity which is explained in the following article.
SQL on-demand by default always uses AAD pass-through.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/53951#issuecomment-623484494, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AI6VQKGI2SQ6LGJQ74P7CRTRP3DTBANCNFSM4MYLR6RA.

Good to hear that! 😄
@ChiragMishra-MSFT can you close this issue?


Enhancement Add 'Copied!' confirmation pop up. useRef deleted as prop (#2)

useRef no longer passed as prop. Changed css files popup-window.

Hey @cod500 thank you for opening a new PR after PR #16!


Add Current Thoughts Scale

http://www.yorku.ca/rokada/psyctest/statese.pdf

 More Catalogue Questionnaires


I am not able to run Django Debug Toolbar.

Hello there.
I tried to enable the django toolbar by following the documentation and setting ENABLE_DEBUG_TOOLBAR  to True. But when I visit /graphql/ I am not able to view it.
Thanks for your support.

We no longer support django-debug-toolbar so I’ll close this issue 🙌


Use a different navigator widget

Hey, is there a way to use a different bottom navigation bar with customscaffold? I want to use bottomnavybar instead of the default one

nvm figured it out


fix: don't focus logs windows unintentionally

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#63
#62
#61  👈
#60
#59
#58
#57
#56
#55
#54
#53
#52
#51
#50
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#64
#63
#62
#61  👈
#60
#59
#58
#57
#56
#55
#54
#53
#52
#51
#50
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


"bad set of key/value pairs for associative array" when completing options with zsh

STR:
# setup poetry completion with zsh
$ poetry add --<tab>
_poetry_8070accb5c835cab_complete:16: bad set of key/value pairs for associative array
_poetry_8070accb5c835cab_complete:16: bad set of key/value pairs for associative array
_poetry_8070accb5c835cab_complete:16: bad set of key/value pairs for associative array

Completing commands (e.g poetry <tab>) works just fine. I can reproduce with another project using cleo as well.

This appears to be happening because opts is already defined globally for me (unsure where). Adding it to the local statement fixes the problem for me.


Add links to asb videos from upskilling hack to the hack guide

Type of PR

[x] Documentation changes
[ ] Code changes
[ ] Test changes
[ ] CI-CD changes
[ ] GitHub Template changes

Purpose of PR
Add links to the ASB overview and components sessions from the One CSE Week upskilling hack.
Does this introduce a breaking change

[ ] YES
[x] NO

Validation

[ ] Unit tests updated and ran successfully
[x] Update documentation or issue referenced above

Issues Closed or Referenced

Closes #https://github.com/retaildevcrews/asb-hack/issues/299

Should this be added to the asb-hack docs as well? https://github.com/retaildevcrews/asb-hack/blob/main/docs/HackGuide.md

We could PR both repos or sync once content is done for Spark. I'm OK either way.
@AAkindele @joheec


A very minor bug when switching between UI color scheme

The menu entry Window -> UI Prefers Color Scheme is not updated for the new UI color scheme. It will remain in the old UI color scheme until you close and open ecode again. For example, if the previous UI color scheme is Dark, if you switch to Light, then both Window and UI Prefers Color Scheme are invisible. I guess it's because the text's color is not updated, but the background color is updated.

Oh, yeah, I've seen it for theme change but only fixed it for that, OK, now it's fixed for color scheme also. The fix is just hack, since there's a more complex issue while re-applying a sheet to the UI node tree, issue is that those node element change state during the re-apply of the style and "miss" to apply some of the new changes, so the current "fix" forces to keep the same state during this change. It's good enough for now.


Collection defaultAccessConditions

Once resource policies endpoint PR is merged the collection endpoint will not expose the defaultAccessConditions link.
This link is used in submission's upload section here to retrieve the collection's defaultAccessConditions.
the lack of the link will break the upload section's functionality.
This issue is to decide how to proceed to fix the issue. I suggest to add a temporary patch to fix the blocked issue. afterwards we can discuss on how to resolve definitely, essentially there are two ways:

implement on angular side the resourcepolicies service to allow to use the new search method
/api/authz/resourcepolicies/search/resource?uuid=<:collection-uuid>&action=DEFAULT_BITSTREAM_READ. This solution requires changes to the rest contract, because DEFAULT_BITSTREAM_READ must be made visible to those who can submit in that collection

permanently remove functionality

@atarix83 : I'd lean slightly towards the first option.  It seems reasonable to me that someone with submit permissions in a Collection should also be able to view that Collection's DEFAULT_BITSTREAM_READ settings.
That said, I don't know whether @artlowel or others have any concerns or objections to this approach.  If we find it to be too difficult for some reason, I'm OK with removing the functionality as needed.


Document that to use Hero widgets, inside TabPage PageStackNavigator navigator must be wrapped in a HeroControllerScope

This information is currently only found in change log for version 0.9. New adopters and future users are unlikely to find it there.

Good call, thanks - added to README and FAQs page on the wiki.


get / set pixels function

I just had a look at https://github.com/pimoroni/unicorn-hat/blob/master/python/UnicornHat/unicornhat.py and I wonder if the set_pixels and get_pixels functions need to be updated to use the get_shape function?
Or is it expected that the PHAT will always simply ignore the "bottom half" of the 8*8 block?
At the very least, the docstring for get_pixels ought to be updated to say "8x8x3" ;-)

(And I also guess some of the demos might need updating too?)

I have updated most of the example to be pHAT/HAT compatible using autodetect and get_shape.
(I have just made a pull request of my code, waiting for that you can look at https://github.com/dglaude/unicorn-hat )
Not sure about the change you want on set_pixels and get_pixels, I did not touch that part.

The revised version of this PR has been merged now, and I can see where the change might come in handy. Both get_pixels and set_pixels have hard-coded values of 8 and 8 for width/height, which is incorrect for a pHAT:
def get_pixel(x, y):
    """Get the RGB value of a single pixel"""
    index = get_index_from_xy(x, y)
    if index is not None:
        pixel = ws2812.getPixelColorRGB(index)
        return int(pixel.r), int(pixel.g), int(pixel.b)

def set_pixels(pixels):
    for x in range(8):
        for y in range(8):
            r, g, b = pixels[y][x]
            set_pixel(x, y, r, g, b)

I think I've fixed this in: cf9ba6f8feaf25038895d625bd50b681c1b47c0f
Plus corrected the fact that set_all does not in fact work like a pixel shader, added shade_all which does, and made set_all just call shade_all with a lambda to pull out the right colours from the 2d array.


Electricity should not be exported when domestic and foreign electricity prices are both 0

In our discussion on Basecamp about the flex-modelling improvements, an edge case was described where both the domestic and the foreign electricity price are 0 euro/MWh. In this case we would expect no electricity to be exported, because the neighbouring country is not likely to want any excess electricity when it has a price of 0 euro/MWh. This led to the addition of a new node attribute, max_consumption_price:

If the max. consumption price is set to zero, the technology will not receive energy from always-on producers. This is to avoid exporting when both the domestic and foreign electricity prices are zero. We'll need to ensure that all other flex technologies have a non-zero price.

However, when I run the model locally on the price-sensitive-flex branches, there are occurrences of electricity being exported while both the merit_order_price_curve and the interconnector_1_marginal_cost_curve are 0.
It also visible in the following charts (scenario is II3050 National governance, with the power-to-gas installed capacity set to 0 MW). In a number of hours at the beginning of the year there clearly is some export, even though both the hourly electricity price and the price of interconnector 1 are 0:

@antw can you reproduce this issue or am I doing something wrong? Did I miss some commits, am I on the wrong branch or is this yet to be implemented?

This should now be fixed on the ETEngine price-sensitive-flex branch.


Compile UICore on Linux

UICore has a dependency on Ishiko/FileTypes which copes with file type associations on Windows only. I need to port that to Linux/Ubuntu.


add danish lang

Checklist
Please put "X" in the below checkboxes that apply:

[x] If committing a change of production code, I have tested it in different browsers (Chrome, Firefox, Safari).
[x] If committing a new feature, I have first submitted an issue (Please note: you are free to open PRs for non-issued features, but opening an issue increases your chance of a successful PR).
[x] If committing a new feature, I have also written the appropriate tests for it.
[x] I have tried to build the feature in a way, that it does not cause any breaking changes for others (unless
agreed upon in an issue).

This PR solves the following problem**.
Add danish languge

Thanks - sorry for the delay.

No worries! Thanks for adding the translations :)


Reimplement prop via path

As discussed in #2059 we are unsatisfied that prop and path have
different behaviour when passing undefined as the object.
We could not yet agree which behaviour we want to have but at least
we want to have the same behaviour. To achieve that, we now
reimplement prop with path.
So if we ever define the behaviour of path then prop
automatically also has the same behaviour.
This also closes #1941.

Just realised that while I reimplemented prop via path, I did not write a test to ensure that somebody else does break that consistency again unintentionally. 😬  So I added a test for that.

While I don't think it should hold up the PR, I think this is a great opportunity to add some property based tests, if only because we have a very clear property we can test for:
forall string s, a -> prop(s, a) is equivalent to path([s], a)

@Bradcomp, I agree both that we should start looking at where to use property-based testing, and that this shouldn't hold up the PR.

Thank you very much, @datenreisender!


Add missing TCPRouteList

This PR:

Adds missing TCPRouteList to scheme registration

The Informer in #33 requires a lister and a watcher. The lister returned TCPRouteList, which was not recognized, and the informer failed to sync.
Fixes #33

This is great! Thank you @dtomcej
As a follow up, we'll probably need to figure out what the testing strategy here should be.


Login Failed,

I've tried to use pins create method and I got following error:
AuthException in ProviderLoginCheckWrapper.php line 54: You must log in before.
Yes I have used login function before calling this method and I get return true (as I assume its successfully logged in).
Currently I am using XAMPP

Hi, I've checked now, everything seems to be working with the latest version, can you show you code?

$bot = PinterestBot::create();
$boards = $bot->boards->forUser('USER GOES HERE');
<EMAIL_ADDRESS>'passwordGoesHere');
$bot->pins->create($book->img_url, $boards[0]['id'], $desc);

I've used it with laravel.

Very strange. I've just checked it in Laravel.
As a quick solution you may comment 34 line in /Api/Providers/ProviderLoginCheckWrapper.php and try. I'll continue research for bugs.

yes, I did try that, but it doesn't make pin. Let me retry again and update, I've missed something.

So I dig deeper in login method, here is the response I get at line 109 in Pinners.php
array:5 [▼
"request_identifier" => "224645968114"
"resource_data_cache" => []
"resource" => array:2 [▼
"name" => "UserSessionResource"
"options" => array:3 [▼
"username_or_email" => "EMAIL HERE"
"bookmarks" => array:1 [▼
0 => "-end-"
]
"password" => "PASSWORD HERE"
]
]
"client_context" => array:39 [▼
"origin" => "https://nl.pinterest.com"
"unauth_id" => "89e7c9d2b449458cb3f2ed1bacfa280a"
"is_retina" => false
"user_agent_platform" => "other"
"sterling_admin_mode_view_as" => null
"is_authenticated" => true
"allow_image_prioritization" => true
"browser_locale" => "en-US"
"deep_link_action" => null
"triggerable_experiments" => array:201 [▶]
"visible_url" => "/login/"
"is_tablet_agent" => false
"utm_campaign" => null
"browser_type" => 4
"deep_link" => ""
"site_type" => 0
"from_open_in_app_click" => null
"invite_code" => null
"use_react" => true
"locale" => "en-US"
"utm_medium" => null
"giftwrap_holdout" => false
"app_version" => "d6936eb"
"site_const" => array:5 [▶]
"real_ip" => "<IP_ADDRESS>"
"user_agent_is_ios_9_2_or_above" => false
"active_experiments" => array:2 [▼
"related_pins_closeup" => "enabled"
"render_using_react_3" => "enabled"
]
"user" => array:39 [▼
"is_employee" => false
"username" => "USERNAME HERE"
"last_name" => ""
"domain_verified" => false
"nags" => []
"image_medium_url" => "https://s-media-cache-ak0.pinimg.com/avatars/bookler4288_1461317847_75.jpg"
"has_password" => true
"full_name" => "FULL NAME HERE"
"image_small_url" => "https://s-media-cache-ak0.pinimg.com/avatars/bookler4288_1461317847_30.jpg"
"gplus_url" => null
"id" => "202310345669271373"
"is_write_banned" => false
"first_name" => "FULL NAME HERE"
"domain_url" => "URL HERE"
"has_seen_notifications" => false
"personalize_from_offsite_browsing" => true
"facebook_timeline_enabled" => false
"personalize_nux_from_offsite_browsing" => false
"connected_to_rakuten" => false
"facebook_id" => ""
"connected_to_google" => false
"is_partner" => false
"type" => "user"
"email" => "EMAIL HERE"
"image_large_url" => "https://s-media-cache-ak0.pinimg.com/avatars/bookler4288_1461317847_140.jpg"
"connected_to_yahoo" => false
"verified_identity" => []
"twitter_url" => null
"facebook_url" => null
"twitter_publish_enabled" => false
"ads_customize_from_conversion" => true
"facebook_publish_stream_enabled" => false
"connected_to_microsoft" => false
"gender" => "female"
"created_at" => "Fri, 22 Apr 2016 09:35:40 +0000"
"login_state" => 1
"website_url" => "WEBSITE LINK HERE"
"country" => "PK"
"extension_access_token" => null
]
"deep_link_default" => ""
"http_referrer" => "https://nl.pinterest.com/"
"is_mobile_agent" => false
"is_internal_ip" => false
"language" => "en"
"is_sterling_on_steroids" => false
"referrer" => null
"user_agent_can_use_native_app" => false
"browser_name" => "Firefox"
"country" => "PK"
"error_template_path" => "templates/error.nunjucks"
]
"resource_response" => array:2 [▼
"data" => array:1 [▼
"token_type" => "bearer"
]
"error" => null
]
]

Yes, logged in successfully, let's debug. Next on line 115 place var_dump($this->request->isLoggedIn());
And in ProviderLoginCheckWrapper on line 53 var_dump($methodRequiresLogin && !$isLoggedIn);

Line 115:
Request {#172 ▼
#userAgent: "Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Firefox/31.0"
#http: CurlAdapter {#178 ▼
-curl: Unknown resource @243
}
#loggedIn: null
#cookieJar: "pinterest_cookie"
#options: array:11 [▼
10018 => "Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Firefox/31.0"
19913 => 1
64 => 0
52 => 1
10102 => "gzip,deflate"
10023 => array:11 [▼
0 => "Accept: application/json, text/javascript, /; q=0.01"
1 => "Accept-Language: en-US,en;q=0.5"
2 => "DNT: 1"
3 => "Host: nl.pinterest.com"
4 => "Content-Type: application/x-www-form-urlencoded; charset=UTF-8"
5 => "X-Pinterest-AppState: active"
6 => "X-NEW-APP: 1"
7 => "X-APP-VERSION: 04cf8cc"
8 => "X-Requested-With: XMLHttpRequest"
9 => "X-CSRFToken: 1234"
10 => "Cookie: csrftoken=1234;"
]
10016 => "https://nl.pinterest.com/"
10031 => "pinterest_cookie"
10082 => "pinterest_cookie"
47 => 1
10015 => "source_url=%2Flogin%2F&data=%7B%22options%22%3A%7B%22username_or_email%22%3A%22social%40bookler.xyz%22%2C%22password%22%3A%22windows%22%7D%2C%22context%22%3A%7B%7D%7D"
]
+csrfToken: null
#requestHeaders: array:9 [▼
0 => "Accept: application/json, text/javascript, /; q=0.01"
1 => "Accept-Language: en-US,en;q=0.5"
2 => "DNT: 1"
3 => "Host: nl.pinterest.com"
4 => "Content-Type: application/x-www-form-urlencoded; charset=UTF-8"
5 => "X-Pinterest-AppState: active"
6 => "X-NEW-APP: 1"
7 => "X-APP-VERSION: 04cf8cc"
8 => "X-Requested-With: XMLHttpRequest"
]
}

53 returns false, which cause exception of auth

I see, i think it is because of rights. When logged in request is made, it saves cookies on disk to parse and to get token from it. In Request.php on 185 line. I think script has no rights to create files, so no token has been parsed.
It's my fault, I should raise an exception here. Please check rights on your directory.

Script saves cookies in it's root path in pinterest_cookie file.

are you talking about following function? Its doesn't seems to called, i can't debug it here. and I am using Windows I don't think any issue with permission?
public static function createQuery(array $data = [], $sourceUrl = '/', $bookmarks = [])
{
    $request = self::createRequestData($data, $sourceUrl, $bookmarks);

    return UrlHelper::buildRequestString($request);
}

No, Api\Request.php line 185:
public function setLoggedIn()
    {
        $this->csrfToken = CsrfHelper::getTokenFromFile($this->cookieJar);   // <--- parsing token from file
        if (!empty($this->csrfToken)) {
            $this->loggedIn = true;
        }

        return $this;
    }

I think the problem is here. No token has been parsed.

well this code intersect with in line no. 158.

public function setLoggedIn()
{
    $this->csrfToken = CsrfHelper::getTokenFromFile($this->cookieJar);
    if (!empty($this->csrfToken)) {
        $this->loggedIn = true;
    }

    return $this;
}

Yes, here. I think $this->csrfToken is empty in your case.

yes it does.

try to change directory permissions

can you tell me when your code store file I might debug it out!

If you are using Laravel, it will be in public folder

hmm that's strange I've provided right too all users still unable to do login.

however thanks for help, since its seems issue on my side, I must have miss something. I will close this bug and debug whole thing in morning.
Have a good day!
Good Night.

Ok, good luck to you! May be try manually create this file empty in laravels public folder.

@capripio  one more solution. I think it will solve problems with permissions. In file Api/Request.php in constructor change cookieJar for curl:
 /**
     * @param HttpInterface $http
     * @param string|null   $userAgent
     */
    public function __construct(HttpInterface $http, $userAgent = null)
    {
        $this->http = $http;
        if ($userAgent !== null) {
            $this->userAgent = $userAgent;
        }
        // $this->cookieJar = self::COOKIE_NAME; <--- before
        $this->cookieJar = tempnam(sys_get_temp_dir(), self::COOKIE_NAME); // <-- after
    }

ok let me try it now!

it's worked! I guess I need to learn about 2 new functions :+1:

Oh, that's great!)

Thanks! finally I can sleep now :D you know what I mean! Good Night and Thanks again!

Good night) I've published a new release of library with this fix. Update tomorrow, and there will be no need to change library source.