[BUG] Linux black screen

Expected Behavior
Hello, does someone use Linux and as output have a window with a completely black screen?
I used this lib also on mac with no problem, I even tried starting from scratch creating a virtual machine but no luck.
Anyone have a clue?

Thank's.
Current Behaviour
Using chart debug I have a white browser page, nothing else.

Reproducible Example
if __name__ == '__main__':

    startDate = datetime.datetime(2006, 1, 1)
    endDate = datetime.datetime(2024, 5, 22)
    spy_data = yf.Ticker('SPY')
    spy_df = spy_data.history(start=startDate, end=endDate)

    spy_df.drop(['Volume', 'Dividends', 'Stock Splits','Capital Gains'],axis = 1,inplace = True)

    chart = Chart()
    chart.set(spy_df)
    chart.show(block=True)

Environment
- OS: Linux Mint / MX Linux
- Library: Last version and tried also on 1.0.21

Refer https://github.com/louisnw01/lightweight-charts-python/issues/334 this.

export WEBKIT_DISABLE_COMPOSITING_MODE="1"
Add this to working terminal and hit enter.


Class remap subscenario

[To merge after #245 and #246] A utility function to remap classes, this utility function should be applied by the user to make clear that a remapping is executed

Need doc web

Need doc web

done


#428 Add a Windows Events profile

Adds the long-awaited Windows Events profile. Kept it short and simple to start:

I thought we would have platform naming, e.g. Windows, rather than WinLog which is the caption in the code (vs. Windows Events which is the popup caption in the screen shot).  Also, I would avoid camel case as we don't use it anywhere.  Can we make the profile name Windows?

I thought we would have platform naming, e.g. Windows, rather than WinLog which is the caption in the code (vs. Windows Events which is the popup caption in the screen shot). Also, I would avoid camel case as we don't use it anywhere. Can we make the profile name Windows?

This profile really only adds support for Windows Event Log messages, it doesn't really provide any general Windows functionality. I think we discussed this on the call before you were able to join


issue while testing the model

Error :
from modules import paths, shared, modelloader, devices, script_callbacks, sd_vae, sd_disable_initialization, errors, hashes, sd_models_config
ImportError: cannot import name 'sd_disable_initialization' from 'modules' (unknown location)
Can you please help with this
Thank you in advance :)

solved
I deleted the folder and download it again
Thank you very much :)


Feature/meeting updates

Added speaker preview notes, retooled CDN/import structure for CSS includes.

closes #4


Reduction of timestamp granularity mask to limit orphan bias

Assuming we will have forking changes during one of the upcoming updates, might it make sense to reduce the 16s timestamp granularity mask.  Would this lessen the likelihood of a race condition and possibly alleviate some of the concerns of JD orphaning solo stakers?
EDIT: By "reduce" I am suggesting a mask such as 8s, 4s, etc.

I've seen a few solo miners complaining lately, so I've come up with a non-answer for you:

It makes sense to me that you'd want to bias the time in your block with mean time of all peers on the network by preference of gaining the widest acceptance possible. In the event that there is a huge majority, you'd want to bias it with their time. That's not to say people still shouldn't update their clocks every day, though.

A majority miner gains an inherent mining advantage due to the start time of scanning work. That advantage is compounded when the same miner solves the subsequent block quickly. Peers with poor/indirect connections can start the work second(s) later due to wallet size, latency, and hardware. This advantage seems fickle, but makes a majority miner the only hashrate for this stairstep-like period of the whole network stopping/starting work. If you've ever been the highest hashrate of a coin, you know what I mean.

There's also the issue of chain trust. The majority always has the most chain trust and cannot be beaten by any other block within the granularity window(I think?)


Replacing all ARGV references with Homebrew.args

Issue reference: https://github.com/Homebrew/brew/issues/5730

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? [Here's an example] (https://github.com/Homebrew/brew/blob/master/Library/Homebrew/test/PATH_spec.rb).
Updated all existing tests.

[x] Have you successfully run brew tests with your changes locally?

Questions:

It looks like the build checks are failing. Is there a way to reproduce that locally?
We did a global replace. We would have preferred to make smaller, incremental changes, but weren't sure if that was the best move and weren't sure how to run individual file tests. We are open to feedback.
Where ARGV filenames are still in existence, should they be changed to Homebrew_args or something similar?

@BenMusch is this what you were expecting?

Yep, this is basically what I was planning on doing. Thanks for picking up the issue!

Wow, thanks for this! Great work so far!

It looks like the build checks are failing. Is there a way to reproduce that locally?

Yup, run brew tests locally!

We did a global replace. We would have preferred to make smaller, incremental changes, but weren't sure if that was the best move and weren't sure how to run individual file tests. We are open to feedback.

I'd be happy with a global change but it's probably a little bit more involved than a global search and replace, unfortunately. Note from https://github.com/Homebrew/brew/issues/5730#issue-409874027:

formula_installer.rb and build.rb should be handled last as they read from and modify ARGV:
https://github.com/Homebrew/brew/blob/b4f73e61649fcfc5aaa779c311e2514619ce01e7/Library/Homebrew/formula_installer.rb#L678-L751

So I'd suggest excluding at least those files and brew.rb for now (that's why brew tests will fail immediately because no brew command` can run currently.
Shout at all if I/we can help at all and thanks again for the PR!

brew tests locally do pass for us so maybe we're doing something wrong?
after excluding build.rb, brew.rb and formula_installer.rb and running brew tests all tests passed and CI still failed.
is there a way to pass a single file to maybe brew test to check our work as we go so we don't have to run the full suite every time?

Thanks!

brew tests locally do pass for us so maybe we're doing something wrong?

How are you running it? Does brew --repo match the directory you're editing/pushing to GitHub?

is there a way to pass a single file to maybe brew test to check our work as we go so we don't have to run the full suite every time?

Yup: brew tests --only=foo will run only Library/Homebrew/test/foo_spec.rb. See brew test --help for more information.

I'd be happy with a global change but it's probably a little bit more involved than a global search and replace, unfortunately.

I agree with @MikeMcQuaid here. An incremental approach would be best in this case.
Global change would also work as long as you have a lot of patience to fix all the failing tests one by one.

@mofo37 Let us know if you'd rather open some smaller PRs instead as per-@GauthamGoli's suggestion as he may also be able to help a bit with this project.

@BenMusch did you ever get these tests to fail locally? https://github.com/Homebrew/brew/pull/5762#issuecomment-464422336

Sorry @mofo37, I don't remember enough about my work on this and don't really have the time to set it up again. Sorry and good luck!

@mofo37 How are you running brew tests? Does brew --repo match the directory you're editing/pushing to GitHub?

@MikeMcQuaid
brew --repo returns this: /usr/local/Homebrew
I'm not sure how to point it to the directory I'm editing/pushing to GitHub.
When I point to the folder I'm working in with brew --repo <file_path>, I get this error:
Error: Invalid tap name "<file_path>"
How do I get it to point to the correct directory?

@mofo37 I don't think this is the preferred way to do things, but the quick fix I used to develop on Homebrew was just to symlink Homebrew to my source code directory and work from there. Some clearer steps:

Commit and push all of the changes you have on your fork of homebrew
cd /usr/local/Homebrew
git remote set-url origin git@github.com:mofo37/brew.git
git remote add upstream git@github.com:Homebrew/brew.git
ln -s /usr/local/Homebrew {your preferred source code/documents directory}/brew

This sets things up so that:

The brew that you run via the brew command will point to your local version of homebrew
Editing the files from either the /usr/local/Homebrew or your source/docs directory will both modify the same source, since one is just a link to the other
Your git repo will have an upstream source pointing to @Homebrew's repo and an origin pointing to your own

Then, the only issues I ran into were brew auto-updating and changing the branches. to fix that, I set these variables in my ~/.bash_profile:
HOMEBREW_DEVELOPER=1
HOMEBREW_NO_AUTO_UPDATE=1

If you add those and then run source ~/.bash_profile, you should be good to go. Note, however, that this will make it your responsibility to manually get updates.
Again: I'm not sure if this is the preferred way of doing things, but it's what I managed to work make for myself when I worked on a few bugfixes

Sorry @mofo37 but I'm closing this out until you're able to figure out how to test the changes locally. Happy to help with that: just shout.


Enhanced MultiReader

Build an enhanced multireader based upon the channel multireader but
support ReadAt and some other convienence bits such as ReadByte and
peek as a lead in for a better implementation for unzip, gzip, and other
fo4rmats.
This commit adds:

BufferedMultiReader
BufferedMultiReader Tests
BufferedReader
BufferedReader Tests
Use of new BufferedMultiReader by Download package

Removal

Old Channel MultiReader
Old BufferedReader

I've started reviewing this, and so far I have no real idea what I'm looking at. The PR description says what's in the code, but it would be really helpful for it to say why. Why is this needed? What problem are we solving?


[Translation] Saviors (Spanish)

What kind of translation do you want to add (mark with an X, e.g. [X] Descent)?

[X ] Mansions of Madness
[ ] Descent

Scenario name or url
https://github.com/ChinaMajesty/Saviors

Translation File
Localization.Spanish.txt

@ChinaMajesty, the Spanish translation is finished.

Still waiting for...

Time goes by... so slowly... @ChinaMajesty

@jradavenport are you, by any chance, @ChinaMajesty?

Keep trying...
Hello, @ChinaMajesty, anybody at home?

3 weeks have passed, and nothing new, @ChinaMajesty?

Just a reminder, @ChinaMajesty, the package is here:
https://github.com/ChinaMajesty/Saviors/files/5448219/Saviors.zip

Another reminder, @ChinaMajesty

One more, do you read this, @ChinaMajesty?


fix(api-client): sidebar fixtures

this pr fixes:

sidebar width issue with long request name
address bar font size consistency
sidebar search background fade
delete modal request name length

Still pretty big, eh? Shouldn’t we limit this to three lines or so?

Still pretty big, eh? Shouldn’t we limit this to three lines or so?

decreased to 3 ✔

Nice!
Can we also limit Workspace name display length?

great catch


Escape single quotes

Single quotes in JSON input were not being escaped - caused error when saving Apex class.
Test data
{
"key": "I'm a value"
}

Thanks!


sphinx with synonyms

Hello,
I want to achieve the following behavior:
If I search for "os coxae" it should search for "os coxae", "ossae coxae" and "ossis
coxae". It should also sort the result in that way that results correspond to "os coxae"
stay on top. Additionally I use metaphone.
I tried to use wordforms with the following file
ossa > os
ossis > os
Unfortunately it only works in one direction. If I search for os for example it finds all
elements which contain os, ossa and ossis. But I want to have the opposite behavior which
doesn't work.
It would be great if anyone could help me finding the best solution.
Thanks in advance,
Johannes

Hi Johannes
I'm afraid I don't know the answer to this question (or #887) - and my brain's a little broken after running a conference over the last few days. I think a better place to ask is the Sphinx forum, given these aren't issues specific to Thinking Sphinx - and the people there will certainly have a deeper understanding of what the right approach could be.
So, I'm going to close this issue - but if you do find a solution, it'd be great to hear it :)

No problem Pat. This forum seems to me quite abandoned. That's why I was hoping you could help me out since you must have a really good understanding of sphinx through developing thinking-sphinx.

I certainly have a somewhat decent understanding of Sphinx, but I've not delved into metaphone, nor played much with wordforms. I think the Sphinx forum does receive some activity - the first few posts in the list are pinned to the top (hence why some are very old) but beyond that it seems there are posts/comments pretty much every day.


tests: subsys: zbus: do not use verbose logging

This change fixes a regression on qemu_riscv64_smp and qemu_riscv32_smp that arose because debug logging was enabled. My guess is that subtle races exist and that debug logging exacerbates the conditions for those races.
Fixes #63409
Testing Done:
twister -i -p qemu_riscv64_smp -p qemu_riscv32_smp -T tests/subsys/zbus/integration
...
INFO    - Using Ninja..
INFO    - Zephyr version: v3.5.0-rc1-32-gc74a7ef2f6b4
INFO    - Using 'zephyr' toolchain.
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /Users/cfriedt/zephyrproject/zephyr/twister-out/testplan.json
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - Total complete:    2/   2  100%  skipped:    0, failed:    0, error:    0
INFO    - 1 test scenarios (2 test instances) selected, 0 configurations skipped (0 by static filter, 0 at runtime).
INFO    - 2 of 2 test configurations passed (100.00%), 0 failed, 0 errored, 0 skipped with 0 warnings in 29.08 seconds
INFO    - In total 8 test cases were executed, 0 skipped on 2 out of total 626 platforms (0.32%)
INFO    - 2 test configurations executed on platforms, 0 test configurations were only built.
INFO    - Saving reports...
INFO    - Writing JSON report /Users/cfriedt/zephyrproject/zephyr/twister-out/twister.json
INFO    - Writing xunit report /Users/cfriedt/zephyrproject/zephyr/twister-out/twister.xml...
INFO    - Writing xunit report /Users/cfriedt/zephyrproject/zephyr/twister-out/twister_report.xml...
INFO    - Run completed

Not sure what the official process is with hotfixes and waiting for the maintainer’s approval, but this change seems trivial enough that I’ll go ahead and merge it. @rodrigopex hopefully that’s ok with you.


FIX-#2978: Define public API for internal frame classes

Signed-off-by: Igoshev, Yaroslav<EMAIL_ADDRESS>
What do these changes do?

 Let's merge this PR first and then #2993 after resolving conflicts there.


Update request: pipewire 0.3.84 → 1.0.0

Package name: pipewire
Latest released version: 1.0.0

Current version on the unstable channel: 0.3.84
Current version on the stable/release channel: 0.3.71
[X] Checked the nixpkgs pull requests

Notify maintainers
@Kranzes
@K900
https://gitlab.freedesktop.org/pipewire/pipewire/-/releases/1.0.0

Note for maintainers: Please tag this issue in your PR.
Priorities
Add a :+1: reaction to issues you find important.

Linking for posterity https://github.com/NixOS/nixpkgs/pull/270110

https://github.com/NixOS/nixpkgs/pull/270111


style: remove unused import

after using poetry import general_utilities is no longed needed

Codecov Report

Merging #51 (54acf26) into master (4654de7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #51   +/-   ##
=======================================
  Coverage   98.82%   98.82%
=======================================
  Files          12       12
  Lines         427      427
  Branches       80       80
=======================================
  Hits          422      422
  Partials        5        5

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


font-input: Use default lineHeight 1.2

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download https://raw.githubusercontent.com/anderscarling/homebrew-fonts/patch-1/Casks/font-input.rb is error-free.
[x] brew cask style --fix font-input.rb left no offenses.

http://input.fontbureau.com/download/ actually defaults to 1.2
Also added missing name sanza as required by brew cask audit

Thank you.


bad child specification, more than one child specification has the id: Telegram.Bot.ChatBot.Chat.Supervisor

hello, I'm back :)
I am getting the following error:
** (Mix) Could not start application my_app: MyApp.Application.start(:normal, []) returned an error: shutdown: failed to start child: Telegram.Poller
    ** (EXIT) bad child specification, more than one child specification has the id: Telegram.Bot.ChatBot.Chat.Supervisor.

my Application :
defmodule MyApp.Application do
  use Application

  alias MyApp.TelegramBot

  @tg_bot_token_bot1 Application.compile_env!(:zuo_shou, :tg_bot_token_bot1)
  @tg_bot_token_bot2 Application.compile_env!(:zuo_shou, :tg_bot_token_bot2)

  @impl true
  def start(_type, _args) do
    bots = [
      {TelegramBot.Bot1, bot_config(@tg_bot_token_bot1)},
      {TelegramBot.Bot2, bot_config(@tg_bot_token_bot2)}
    ]

    children = [{Telegram.Poller, bots: bots}]

    opts = [strategy: :one_for_one, name: MyApp.Supervisor]
    Supervisor.start_link(children, opts)
  end

  @spec bot_config(String.t()) :: Keyword.t()
  defp bot_config(token), do: [token: token, max_bot_concurrency: 1]
end

I've double checked and the two tokens are definitely different (IO.inspect({@tg_bot_token_bot1, @tg_bot_token_bot2})), so the issue is not the same as #153

Hello @samm81,
Fix available in release 1.1.1

amazing :clap: fastest I've ever seen an issue fixed. thank you!


[QUESTION] - do you have a script to generate the color variants?

I am asking because I want to generate darker variants and I have no idea how to achieve that and get perfect calculated values :page_facing_up:

No, I haven`t script for generate color variants. But if you find it in other place I will be grateful for the link.


Add a readme

Add a readme that explains how to use the project

https://github.com/lcockerham/dms_guild_sales/pull/8


'noImplicitReturns' doesn't report an error if any branch returns 'any'

The following correctly reports an error on foo under noImplicitReturns.
function foo(y: number, z: number) {
    if (y)
        return z;
}

However, removing the type annotations from y and z squelches the error.
function foo(y, z) {
    if (y)
        return z;
}

Discussed offline with @mhegazy and @billti - current behavior for .ts files is by design - we don't report noImplicitReturns error if inferred return type is any or void. It makes sense however to report it for .js files and we already have an issue to track this. Closing this one in favor of it.


Do a prelimary check for numbers in openssl prime command.

Fixes openssl#16241.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge


fix: well-known-event-providers.md SampleProfiler sampling rate

Summary
I've just noticed the docs say the SampleProfiler samples every 10 milliseconds but that is not true, see https://github.com/dotnet/runtime/issues/82939 for context

@vaind please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@dotnet-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@dotnet-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@dotnet-policy-service agree company="Microsoft"

Contributor License Agreement

@dotnet-policy-service agree


A small change that only affects dev, which uses browser alert instead of redirect when org data is not able to be fetched

#3319  👈
main

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

Join @nickgraz and the rest of your teammates on  Graphite


Is there documentation?

I can't find documentation (eg. on keybindings, how to use, customisation) on the Github page or Wiki. If it needs to be written, a note to that effect would be helpful on the main readme so new users don't go looking for it (if I use it enough I'd contribute myself; if you have a system in mind add that to the note).
I installed Smith with v1.3.0 with Cargo on Ubuntu 19.10.

As it is there is no documentation of the keybindings, though they are all in https://github.com/IGI-111/Smith/blob/master/src/command/mod.rs
I thought about adding a help page you could access with F1 with all of them neatly explained but I haven't gotten around to it so far.

Cheers. I'll take a look, use it some more, and contribute something if I can.


chore(main): release 1.4.3

:robot: I have created a release beep boop
1.4.3 (2023-09-18)
Bug Fixes

docs (1e74d25)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/aidanaden/canvas-sync/releases/tag/v1.4.3 :sunflower:


Make sure fake zocalo, used for developer testing, is tidied up well

Fixes #547
To test:

Start then kill fake zocalo
Confirm /home/$USER/.zocalo/ is cleaned
Start fake zocalo again (note will need a few seconds before stopping it) and confirm a new rabbit mq instance is created

Clean up works when exiting the terminal, but it doesn't seem to do anything when simply killing process with Ctrl+C

works for me

I'm gonna spend a bit of time debugging for @noemifrisina before merging but thanks for checking @dperl-dls

After some testing it looks like for some reason the trap command doesn't work for @noemifrisina, I will instead do the cleanup in the keyboard interrupt exception in the python code

@noemifrisina to look into the trap issue on their machine further. If you have no luck, I'll do the above


Filter fixes and tests

Change Comparables to Strings in ValueMatcher related code.
Break out RowBasedValueMatcherFactory, fix a variety of issues around nulls, and add tests.
Fix bound filters on long columns with non-numeric bounds, and add tests.
All Filter tests now exercise the CNF and post-filtering features.

👍

:+1:


Can't build from checkout

Looks like there is a class format error int he build somewhere.
This can likely be fixed by specifying the java version to build with.
This is my entire output:
cx2s-bpappin:libaddressinput bpappin$ gradle clean install
WARNING: Dependency org.json:json:20090211 is ignored for debug as it may be conflicting with the internal version provided by Android.
         In case of problem, please repackage with jarjar to change the class packages
WARNING: Dependency org.json:json:20090211 is ignored for release as it may be conflicting with the internal version provided by Android.
         In case of problem, please repackage with jarjar to change the class packages
WARNING: Dependency org.json:json:20090211 is ignored for debugUnitTest as it may be conflicting with the internal version provided by Android.
         In case of problem, please repackage with jarjar to change the class packages
WARNING: Dependency org.json:json:20090211 is ignored for releaseUnitTest as it may be conflicting with the internal version provided by Android.
         In case of problem, please repackage with jarjar to change the class packages
WARNING: Dependency org.json:json:20090211 is ignored for debugAndroidTest as it may be conflicting with the internal version provided by Android.
         In case of problem, please repackage with jarjar to change the class packages
:android:clean
:common:clean
:android:preBuild UP-TO-DATE
:android:preDebugAndroidTestBuild UP-TO-DATE
[... removed cruft for clarity ...]
:android:bundleDebug
:android:assembleDebug
:android:processDebugAndroidTestManifest
:android:compileDebugAndroidTestRenderscript
:android:generateDebugAndroidTestBuildConfig
:android:generateDebugAndroidTestAssets UP-TO-DATE
:android:mergeDebugAndroidTestAssets
:android:generateDebugAndroidTestResValues UP-TO-DATE
:android:generateDebugAndroidTestResources
:android:mergeDebugAndroidTestResources
:android:processDebugAndroidTestResources
:android:generateDebugAndroidTestSources
:android:compileDebugAndroidTestJava
:android:preDexDebugAndroidTest
UNEXPECTED TOP-LEVEL EXCEPTION:
com.android.dx.cf.iface.ParseException: bad class file magic (cafebabe) or version (0034.0000)
	at com.android.dx.cf.direct.DirectClassFile.parse0(DirectClassFile.java:472)
	at com.android.dx.cf.direct.DirectClassFile.parse(DirectClassFile.java:406)
	at com.android.dx.cf.direct.DirectClassFile.parseToInterfacesIfNecessary(DirectClassFile.java:388)
	at com.android.dx.cf.direct.DirectClassFile.getMagic(DirectClassFile.java:251)
	at com.android.dx.command.dexer.Main.processClass(Main.java:704)
	at com.android.dx.command.dexer.Main.processFileBytes(Main.java:673)
	at com.android.dx.command.dexer.Main.access$300(Main.java:83)
	at com.android.dx.command.dexer.Main$1.processFileBytes(Main.java:602)
	at com.android.dx.cf.direct.ClassPathOpener.processArchive(ClassPathOpener.java:284)
	at com.android.dx.cf.direct.ClassPathOpener.processOne(ClassPathOpener.java:166)
	at com.android.dx.cf.direct.ClassPathOpener.process(ClassPathOpener.java:144)
	at com.android.dx.command.dexer.Main.processOne(Main.java:632)
	at com.android.dx.command.dexer.Main.processAllFiles(Main.java:510)
	at com.android.dx.command.dexer.Main.runMonoDex(Main.java:280)
	at com.android.dx.command.dexer.Main.run(Main.java:246)
	at com.android.dx.command.dexer.Main.main(Main.java:215)
	at com.android.dx.command.Main.main(Main.java:106)
...while parsing com/google/i18n/addressinput/common/AddressData$1.class
1 error; aborting

:android:preDexDebugAndroidTest FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':android:preDexDebugAndroidTest'.
> com.android.ide.common.process.ProcessException: org.gradle.process.internal.ExecException: Process 'command '/Library/Java/JavaVirtualMachines/jdk1.8.0_60.jdk/Contents/Home/bin/java'' finished with non-zero exit value 1

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

Total time: 1.979 secs

To fix locally, i added:
common/
apply plugin: 'java'

sourceCompatibility = 1.7
targetCompatibility = 1.7

an in android/
android {
    ....
    compileOptions{
	    sourceCompatibility=JavaVersion.VERSION_1_7
	    targetCompatibility=JavaVersion.VERSION_1_7
	}
}

Does not repro with "gradle clean build", but repros with "gradle clean install". Will try your fix and send a PR if it works.

Only changes in common/ are required. Changes in android/ are not necessary. @bpappin Can you verify this in your build?

https://github.com/googlei18n/libaddressinput/pull/102

I can't, I'm not using the library as I could not getting it working easily and had to move on.
I intend to come back to it, but until then it will have to wait.

Thank you for the response, @bpappin


feat: use application graph to configure examples

This PR makes some first adjustments to the lines2words example, letting it use the JSON-based application graph instead of the YAML-based configuration files we are using so far.
The adjustment is not yet finished, as there are still some aspects that need to be discussed. I've added two TODOs for that so far that we can pick up later.

I think this PR is now ready for another review :) Besides lines2words, all the examples now mostly depend on the information from the application graph, demonstrating that we can use the DAG to set up an IceFlow pipeline with it.
As a next step, I would move most of the (currently duplicated) code for setting up consumers and producers into the main IceFlow class (or maybe a IceflowNode wrapper class?), which would now accept an application graph as its input. Based on that, I would add methods for registering handler functions for either the consumer or producer side of available edges, simplifying the setup of individual IceFlow nodes.
Looking forward to your feedback!


styleguide breaks bootstrap sortable

specifically the arrows that show the sort direction are no longer rendering

This issue has been hanging around for a while. Is it important / worth doing?

not mission critical but if you have time to investigate @louh

it's only for admin so it doesn't matter that much, which is why i've been putting off finding a solution. i'll take a look


fbv won't cross-compile...

 $ nix-build -A all -I nixpkgs\=channel:nixpkgs-unstable --argstr device asus-z00t
these derivations will be built:
  /nix/store/kyqkgiima57fczglk8a98vzmij46jg6x-fbv-aarch64-unknown-linux-gnu.drv
  /nix/store/g5y10hfw6jmwky4afzk9k02njn1j4hfh-extra-utils-asus-z00t-aarch64-unknown-linux-gnu.drv
  /nix/store/2vgz1vjfzfr7y6lcpil28nafkif4hgqc-stage1-aarch64-unknown-linux-gnu.drv
  /nix/store/kbmr35hqa1rhg347gi5yv201g80l7pxv-initrd-aarch64-unknown-linux-gnu.drv
  /nix/store/0ss3hdkxf9pc89yx87zljn5z2mqgambq-initrd-asus-z00t-aarch64-unknown-linux-gnu.drv
  /nix/store/pa79lwbzdg189ap37zp5zx9ydyhs3hk9-nixos-mobile_asus-z00t_boot.img-aarch64-unknown-linux-gnu.drv
building '/nix/store/kyqkgiima57fczglk8a98vzmij46jg6x-fbv-aarch64-unknown-linux-gnu.drv'...
unpacking sources
unpacking source archive /nix/store/b3pi8ygkpj4xzkd5s45gfwsj1043hjir-fbv-1.0b.tar.gz
source root is fbv-1.0b
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file fbv-1.0b/Make.conf
patching sources
updateAutotoolsGnuConfigScriptsPhase
configuring
configure flags: --prefix=/nix/store/biskykra79jgz034w58v5mpvh3kbil2b-fbv-aarch64-unknown-linux-gnu --build=x86_64-unknown-linux-gnu --host=aarch64-unknown-linux-gnu
./configure: line 57: getopt: not found
Usage: ./configure [options]
Options: [defaults in brackets after descriptions]
If a long option shows an argument as mandatory, then it is mandatory
for the equivalent short option also.  Similarly for optional arguments.
General:
  --help        print this message
  --libs=LIBS   additional libraries required (try -lX11 for ungif, -lz for PNG)
Directory and file names:
  --prefix=PREFIX       install files in PREFIX [/usr/local]
  --bindir=DIR          binary executable in DIR [PREFIX/lib]
  --infodir=DIR         info documentation in DIR [PREFIX/info]
  --mandir=DIR          man documentation in DIR [PREFIX/man]
Features and packages:
  --without-libungif    disable libungif support even if found
  --without-libjpeg     disable libjpeg support even if found
  --without-libpng      disable libpng support even if found
  --without-bmp         disable bmp support
builder for '/nix/store/kyqkgiima57fczglk8a98vzmij46jg6x-fbv-aarch64-unknown-linux-gnu.drv' failed with exit code 1
cannot build derivation '/nix/store/g5y10hfw6jmwky4afzk9k02njn1j4hfh-extra-utils-asus-z00t-aarch64-unknown-linux-gnu.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/2vgz1vjfzfr7y6lcpil28nafkif4hgqc-stage1-aarch64-unknown-linux-gnu.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/kbmr35hqa1rhg347gi5yv201g80l7pxv-initrd-aarch64-unknown-linux-gnu.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/0ss3hdkxf9pc89yx87zljn5z2mqgambq-initrd-asus-z00t-aarch64-unknown-linux-gnu.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/pa79lwbzdg189ap37zp5zx9ydyhs3hk9-nixos-mobile_asus-z00t_boot.img-aarch64-unknown-linux-gnu.drv': 1 dependencies couldn't be built
error: build of '/nix/store/pa79lwbzdg189ap37zp5zx9ydyhs3hk9-nixos-mobile_asus-z00t_boot.img-aarch64-unknown-linux-gnu.drv' failed

This is caused by its non-standard ./configure script.
Work will need to be done to figure out why it has issues.

Will use ply-image instead, which plays fine.


Only build linux musl and update CFS auth

PR Summary

Since musl build is statically linked, it will work on non-musl based Linux distros and have no dynamic dependencies.
Update release pipeline to only build musl
Update build.ps1 tar.gz packaging to rename linux package to just linux
Add suffix to CFS feed so it always authenticates
With CFS change, we can update cargo.toml and cargo.lock to just specify major.minor versions

PR Context

Validated test release


Add optional multipleOr tests for code & vaccine-code & intent

AU Core says SHOULD support multipleOr for code and vaccine-code parameters. Please add a multipleOr test to:

1.3.01 Server returns valid results for Observation search by patient + code (and equivalent in other Observation profiles)
1.3.03 Optional Server returns valid results for Observation search by code (and equivalent in other Observation profiles)
1.3.10 Optional Server returns valid results for Observation search by patient + code + date (and equivalent in other Observation profiles)
1.16.04 Optional Server returns valid results for Immunization search by vaccine-code
1.16.07 Optional Server returns valid results for Immunization search by patient + vaccine-code
1.17.06 Optional Server returns valid results for MedicationRequest search by intent
1.17.08 Server returns valid results for MedicationRequest search by patient + intent
1.17.09 Server returns valid results for MedicationRequest search by patient + intent + status
1.17.10 Server returns valid results for MedicationRequest search by patient + intent + authoredon
1.19.02 Optional Server returns valid results for Procedure search by code
1.19.06 Optional Server returns valid results for Procedure search by patient + code + date

[ ] 1.3.01 Server returns valid results for Observation search by patient + code (and equivalent in other Observation profiles)
[ ] 1.3.03 Optional Server returns valid results for Observation search by code (and equivalent in other Observation profiles)
[ ] 1.3.10 Optional Server returns valid results for Observation search by patient + code + date (and equivalent in other Observation profiles)
[ ] 1.16.04 Optional Server returns valid results for Immunization search by vaccine-code
[ ] 1.16.07 Optional Server returns valid results for Immunization search by patient + vaccine-code
[ ] 1.17.06 Optional Server returns valid results for MedicationRequest search by intent
[ ] 1.17.08 Server returns valid results for MedicationRequest search by patient + intent
[ ] 1.17.09 Server returns valid results for MedicationRequest search by patient + intent + status
[ ] 1.17.10 Server returns valid results for MedicationRequest search by patient + intent + authoredon
[ ] 1.19.02 Optional Server returns valid results for Procedure search by code
[ ] 1.19.06 Optional Server returns valid results for Procedure search by patient + code + date

The fix for this issue will be merged.
Now, multipleOr and multipleAnd requests are independent tests. It's simpler to read and understand.
https://github.com/hl7au/au-fhir-core-inferno/pull/75

@StephThaoOng Hello
This was fixed. Could you review it? If all good, please close the ticket

Server returns valid results for Observation search by patient + code and equivalent in other Observation profiles)

Still missing multipleOR tests on code for this combined search

Server returns valid results for Observation search by patient + code + date (and equivalent in other Observation profiles)

Still missing multipleOR tests on code for this combined search
Still missing multipleAND tests on date for this combined search

Server returns valid results for Immunization search by patient + vaccine-code

Still missing multipleOR tests on vaccine-code for this combined search

Server returns valid results for MedicationRequest search by patient + intent

Still missing multipleOR tests on intent for this combined search

Server returns valid results for MedicationRequest search by patient + intent + status

Still missing multipleOR tests on intent and status for this combined search

Server returns valid results for MedicationRequest search by patient + intent + authoredon

Still missing multipleAND tests on authoredon (with date comparators) for this combined search

Server returns valid results for Procedure search by patient + code + date

Still missing multipleOR tests on code for this combined search

@StephThaoOng Steph,
All tests for "multipleOr" and "multipleAnd" were implemented separately from the combined search.

@StephThaoOng Steph,
All tests for "multipleOr" and "multipleAnd" were implemented separately from the combined search.
Initially, the ticket was created asking to add multiple "Or" tests to existing tests. However, when we started development, we decided not to change the working logic of the combined search parameters and made it separately with new tests.

Thanks @projkov - ack that tests for "multipleOr" and "multipleAnd" are implemented separately from the combined search for now. We'll discuss with the team (next Sprint meeting) whether this is acceptable in the longer term.

@StephThaoOng Steph,
All tests for "multipleOr" and "multipleAnd" were implemented separately from the combined search.
Initially, the ticket was created asking to add multiple "Or" tests to existing tests. However, when we started development, we decided not to change the working logic of the combined search parameters and made it separately with new tests.

Thanks @projkov - ack that tests for "multipleOr" and "multipleAnd" are implemented separately from the combined search for now. We'll discuss with the team (next Sprint meeting) whether this is acceptable in the longer term.

24/6 Infra&Tooling meeting - agreed that for AU Core capability testing, it is reasonable to implement tests for "multipleOr" search and "multipleAnd" search separately.


Can We Use Storage Feature?

Issue with current documentation:
No response
Idea or request for content:
Hi,
Am i able to send chats to openAI and fetch them from storage with this package? So basically i wanna create history part for old chats and achieve them.

Currently, we only have support for storing the history in-memory:
https://langchaindart.dev/#/modules/memory/memory
But, it should be possible to do a custom implementation to persist the history to disk as @Nana-Kwame-bot mentions. #171 does that with Firestore.
Chat history is an area where we still need to improve the framework.


Support for crosswordr.com?

https://crosswordr.com/

Looks reasonably feasible. The site itself is heavily-compiled ReactJS so we probably can't pull the content from the local page, but
we could replicate the network call the page is making to https://api.crosswordr.com/graphql in a way that would hopefully be durable.

Added an initial cut of this in version 1.3.19.
Scraping subscription-only doesn't work, and doesn't seem to have an easy path forward to supporting. But I think both of the currently-supported subscriptions are also distributed directly as downloadable files, so this doesn't seem like a huge deal.
There are probably also some quirks to be worked out as I don't know how their data representation handles certain cases that I wasn't able to find in a few example puzzles. These can be addressed as they are discovered.


Add documentation for step methods returning tuples

Hi, thanks for creating TickSpec. It's a great library!
I had to look through the source code before I found that step methods returning tuples would allow me to pass multiple values between step methods.
This PR adds some info to the README to mention return tuples.

In general I'd venture a record is a much cleaner way to manage this type of contract. i.e. Explicit is better than implicit; your code does not end up littered with fst and snd and adding a third element to a tuple is not going to land you with some very confusing type errors.
IMO the readme should probably mention it can be a record and recommend that -
🤔 Perhaps one of the examples should probably be made to illustrate this if it does not already (perhaps it can illustrate that tuples are types too and work, but recommend records as an alterative in the comment).

Sorry, had completely forgotten about that feature / aspect of tuples - so never mind me!
Wrt docs - a wiki might indeed be nice and/or you might want to blog or do something simple that works like gists yourself.
However I'd say putting even relatively verbose examples toward the bottom of the readme is ultimately going to be the most useful in terms of conveying the capabilities of a system or project to someone browsing at a glance - example: https://github.com/App-vNext/Polly
The bottom line is that any documentation and examples contribution would be fantastic - whatever works for you as a good way to get a baseline in place and let others take it and run is good, so please don't view my subjective preferences as a constraint on how you do it.


Schema validation failing when optional field is missing

We have noticed an issue when upgrading to v19.0.0 (this also occurs in v20.0.0). We have the following type in our GraphQL Schema
type Contributors {
          directors: [PersonName!]
          actors: [PersonName!]
          narrators: [PersonName!]
          writers: [PersonName!]
        }

PersonName is described as follows
type PersonName {
          givenName: String!
          familyName: String!
        }

Contributors are referenced in the type Foo like so :
 type Foo {
          contributors: Contributors!
        }

Within the Foo type,  Contributors is a required field i.e Contributors! however, within the the Contributors type all of it's fields are optional values and therefore according to GraphQL some fields may completely be omitted, so based on this understanding when asking for the following Contributors data via a query returning Foo, like so:
  contributors {
        directors{ givenName familyName }
        actors { givenName familyName }
        narrators { givenName familyName  }
        writers { givenName familyName }
      }

The expectation is that any of the following responses would be deemed valid due to the fields within Contributors being set to optional
"contributors": {}
"contributors": {"actors": []}
"contributors":{
     "directors": []
     "actors": []
     "narrators": []
     "writers": []
    }
"contributors":{
     "directors": ["givenName": "john",
              "familyName": "smith"]
     "actors": null
     "narrators": null
     "writers": ["givenName": "jane",
              "familyName": "smith"]
    }

The data corresponding to these types are stored in a column as a JSON blob in our database, see example below. We have noticed that after upgrading to grackle v19.0.0 , and running tests with this data the mapping now fails against the GraphQL Schema, due to narrators not being present:
          "actors": [],
          "writers": [
            {
              "givenName": "john",
              "familyName": "smith"
            }

          ],
          "directors": [
            {
              "givenName": "jane",
              "familyName": "smith"
            }
          ]
        }

It is our understanding of GraphQL, that the JSON from the DB should map correctly to the GraphQL response below:
contributors:{
     directors: [{
             "givenName": "jane",
              "familyName": "smith"
            }],
     actors: [],
     narrators: null,
     writers: [{
              "givenName": "john",
              "familyName": "smith"
            }]
    }

However this doesn't work as expected, and grackle complains that narrators isn't present please see :
 java.lang.AssertionError: Obtained {"errors":[{"message":"No field 'narrators' for type Contributors"}],"data":null} data
Expected: a JSON object
     got: null
 ;
Unexpected: errors

The messaging indicates that we need to specify a narrators field in our JSON object to either point to null or empty array for this to work. Backfilling the data like so causes the test to pass. However, we are not sure if this is desired behaviour as the fields within Contributors are Optional and therefore if the field doesn't exist in the JSON then the query should not fail, but the field should instead be omitted from the response i.e as described:
contributors:{
     directors: [{
              "givenName": "jane",
              "familyName": "smith"
            }],
     actors: [],
     narrators: null,
     writers: [{
              "givenName": "john",
              "familyName": "smith"
            }]
    }

We didn't have this issue in v18.0.1, it started occuring in v19.0.0 even when upgrading to v20.0.0 the issue still persists. Please can this be looked into?

You're right, this is a change in behaviour in 0.19.0 and it is intentional that you now have to provide an explicitly null or empty array value in those cases.
The previous behaviour where the missing field was omitted in the GraphQL response was violating the GraphQL spec, ie. it's not the case that "if the field doesn't exist in the JSON then the query should not fail, but the field should instead be omitted from the response" ... if a field is requested in the query, then it must be present in the response, as a null value or an empty array if that's permitted by the schema.
In principle it would be possible for the CirceMapping to synthesize these nulls or empty arrays, but the general consensus was that this had the potential to interfere with the mechanism for delegating to other mappings, and could also mask errors in the applications data representation, so on balance it was felt that providing null/empty fields should be done explicitly by the application.

@milessabin Thanks for clarifying, will close this issue and look to make changes on our side to make this possible


HTML page through Android or IOs app does not run javascript

Hi
We've developped a Team message extension so as to create tickets into a database.
To do so, we have a first page that does authenticate using Azure token then access our "auth" endpoint to ensure the current user is allowed in our database.
In the PC version, everything is ok !
In tactile devices, the external page is not closed automatically, we are obliged to provide end-user with a custom HTML page to inform that he can close the page manually...
Will you have some idea to make our authentication process with tactile apps as great as the PC version ?

@MLEGIsilog - Thanks for reporting your issue.
We will try it from our end and get back to you.

Hello @MLEGIsilog - Have you implemented the Authentication via following below documentation for your Messaging extension or any other custom auth?
https://learn.microsoft.com/en-us/microsoftteams/platform/messaging-extensions/how-to/add-authentication
Could you please share more information.

Hello,
We used MSAL for the authent using this doc : https://learn.microsoft.com/en-us/azure/communication-services/quickstarts/manage-teams-identity?pivots=programming-language-javascript
It seems that js script is run using default browser for desktop versions but it is not the same for tactile devices : is it a feature that could be implemented soon ?
Regards
De : ChetanSharma-MSFT @.>
Envoyé : mardi 8 novembre 2022 07:52
À : MicrosoftDocs/msteams-docs @.>
Cc : Michaël LEGER @.>; Mention @.>
Objet : Re: [MicrosoftDocs/msteams-docs] HTML page through Android or IOs app does not run javascript (Issue #7411)
Hello @MLEGIsiloghttps://github.com/MLEGIsilog - Have you implemented the Authentication via following below documentation for your Messaging extension or any other custom auth?
https://learn.microsoft.com/en-us/microsoftteams/platform/messaging-extensions/how-to/add-authentication
Could you please share more information.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/msteams-docs/issues/7411#issuecomment-1306716229, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A3BLTKLSRHJDQEFGHUJY543WHH2AXANCNFSM6AAAAAARXLHBG4.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@MLEGIsilog -We will confirm with the engineering team and let you know once we get any update.

Teams Clients use different browsers to run our apps - In Windows Desktop Client we use the Electron webview (which is based on Chromium), in iOS we use the WebKit webview, etc. This issue is causes due to you doesn't  implemented authentication correctly for your app
Please follow the document- https://learn.microsoft.com/en-us/azure/communication-services/quickstarts/manage-teams-identity?pivots=programming-language-javascript

Actually, this is the correct link that should be followed when implementing authentication for your Teams App:
https://learn.microsoft.com/en-us/microsoftteams/platform/concepts/authentication/authentication

@MLEGIsilog - Could you please confirm if your issue has resolved with above suggestion or still looking for any help?

@MLEGIsilog -
#6091-We are checking the status of the fix release with the engineering team. We will let you know once we have any updates on it.
#6142-This is by design a difference between chat (react) and channel (angular)


[CRASH] Game crashes when joining world

I launched my game with a minimal set of just 37 mods (Quilt 1.20.1 v0.24.0) and put Cyanide into that modpack, and when I try to join a world, it instantly crashes with the stack trace below:
Description: Exception in server tick loop

java.lang.NullPointerException: Exception in server tick loop
	at java.base/java.util.Objects.requireNonNull(Unknown Source)
	at org.quiltmc.qsl.worldgen.biome.impl.TheEndBiomeData.getAddedBiomes(TheEndBiomeData.java:101)
	at org.quiltmc.qsl.worldgen.biome.impl.TheEndBiomeData$Overrides.<init>(TheEndBiomeData.java:132)
	at org.quiltmc.qsl.worldgen.biome.impl.TheEndBiomeData.createOverrides(TheEndBiomeData.java:97)
	at com.google.common.base.Suppliers$NonSerializableMemoizingSupplier.get(Suppliers.java:183)
	at net.minecraft.class_2169.method_28443(class_2169.java:590)
	at net.minecraft.class_7869.method_46706(class_7869.java:79)
	at net.minecraft.class_7869.method_46699(class_7869.java:66)
	at java.base/java.util.stream.ReferencePipeline$2$1.accept(Unknown Source)
	at java.base/java.util.AbstractList$RandomAccessSpliterator.forEachRemaining(Unknown Source)
	at java.base/java.util.stream.AbstractPipeline.copyInto(Unknown Source)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(Unknown Source)
	at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(Unknown Source)
	at java.base/java.util.stream.AbstractPipeline.evaluate(Unknown Source)
	at java.base/java.util.stream.ReferencePipeline.collect(Unknown Source)
	at net.minecraft.class_7869.method_46702(class_7869.java:67)
	at net.minecraft.class_2794.method_46696(class_2794.java:108)
	at net.minecraft.class_3898.<init>(class_3898.java:190)
	at net.minecraft.class_3215.<init>(class_3215.java:87)
	at net.minecraft.class_3218.<init>(class_3218.java:231)
	at net.minecraft.server.MinecraftServer.method_3786(MinecraftServer.java:399)
	at net.minecraft.server.MinecraftServer.method_3735(MinecraftServer.java:324)
	at net.minecraft.class_1132.method_3823(class_1132.java:69)
	at net.minecraft.server.MinecraftServer.method_29741(MinecraftServer.java:646)
	at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer.java:265)
	at java.base/java.lang.Thread.run(Unknown Source)

Modlist:

Client side noteblocks
Boat break fix
Chunksending Mod (Not available on Modrinth)
Cloth Config API
Concurrent chunk management engine
Cyanide
Dashloader
Debugify
Enhanced block entities
Entity culling
Fast anim
Fast chest (Not available on Modrinth)
Fast quit
Faster random
FerriteCore
Idle tweaks
ImmediatelyFast
Krypton
LanguageReload
LazyDFU
Lithium
Logical zoom
MemoryLeakFix
Methane
Mod menu
Modernfix
Moreculling
Nvidium
QFAPI
Reese's sodium options
Server pinger fixer
Shared resources
Smoothchunk mod (Not available on Modrinth)
Sodium
Sodium Extra
ThreadTweak
Cupboard (Not available on Modrinth)

Detailed mod list with mod versions, filenames, and library mods not located in the regular mods folder can be found here: https://pastebin.com/1AB6z36S (The text formatting is messed up, sorry.)

I created a profile (Quilt 1.20.1 v0.24.0) where I have only Cyanide installed to the profile, and my game did not crash when I joined a world.

Cyanide is not tested against, not developed for, and does not support quilt. It is not marked on any platform as supporting quilt.


How to get the picture in Figure 5  Red lines denote the ground truth.

How to get the picture in Figure 5  Red lines denote the ground truth.?

First get the predicted probability as fchollet/keras#41 . Then use matlab "bwperim", you can get the perimeter of objects in binary image.

thanks  for your help!


typecast from bool to string doesn't seem to be correct

In my tests, boolean will typecast to "True", but documentation says "true".

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

ID: 77c58a78-a567-e99a-9eb7-62dddd1b90b6
Version Independent ID: 680a79bc-11de-39fc-43e3-e07dc762db18
Content: Expressions - Azure Pipelines
Content Source: docs/pipelines/process/expressions.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

@horihel Thank you for noticing this! I'm updating the article now and the changes will be live by the end of the day.


Cleanup DEFS file for generating infer genrules

Only generate one extra genrule for running infer. Remove all other java library rules currently being generated
Generate infer genrule only if the java_library has srcs, otherwise there is nothing to analyze
Use SRCDIR to avoid making a copy of the target sources as buck will just symlink them instead

Will make another change for the ability to add .inferconfig and .inferversion etc to the srcs as that will cause the genrule to invalidate when the infer version or configuration changes


Bug: cannot add more py files and non-python files to the package read by the pdt

Hi @achille-martin
When I am trying to deploy an application created with more than one .py file, the .apk isn't running on Android.
For the example you can see the followig structure:

And of course, the config.ptd file:

I think that the files are not collecting correct by the building tool, can you try by yourself and come with a feedback?

There is something tricky around python packages and Android filesystem that I have not fully understood yet.
There is work in progress in the background to get this to function fine, but focus is at the moment on this issue (with a single file).
The workaround (for now) is to create all extra python package files from a unique python script.
Cf. this script for example.
Not ideal, but I'll get back to this issue.
Thanks for your contribution.

The same problem exists with resources like images, icons, etc. I think this is also related to the Android file system.
I will also do some research and provide you with any information I have.

No worries, there might be something we have missed.
Interestingly enough, I was looking at the issue I mentioned and I managed to access resources from within the built "pyqt5 android" package.
There is an example here (the whole external_python_project_2 folder actually) if you are curious. It is using importlib.resources to access resources from within the package and it seems to be the python standard to do so. Note that only importlib.resources.read_text() is properly working. I still need to investigate how to get yaml to read a file within the package, but there is potential there!

I have created a very basic package that can be read by the pdt -> Link.
At the moment, it is only composed of a config file and a python module, but it works on Android through importlib.resources. Note that it also works on Linux.
You can give it a try and get the inspiration for your own package.

Servus @achille-martin
I'm trying to load images into an application based on your externalpy example project.
You can find the changes I made to the pyqt5_app_with_yaml.py file below:
        # Initialise Layout
        self.layout = QHBoxLayout()
        self.layout.setSpacing(10)
        self.layout.setContentsMargins(10, 10, 10, 10)
        self.setLayout(self.layout)

        # Initialise image
        img_label = QLabel(self)
        pixmap = QtGui.QPixmap()
        self.img_path = None

        try:
            logger.debug(f"""MainWindow::__init__ - Getting file path and file itself img.png""")
            with resources.path(validate_pkg_format('externalpy_pkg.resources', logger), 'img.png') as path:
                self.img_path = realpath(path)
                logger.debug(f"""MainWindow::__init__ - Obtained file path for img.png: {self.img_path}""")
            logger.debug(f"MainWindow::__init__ - Obtained file path: {self.img_path}")
        except Exception as e:
            logger.warn(f"""MainWindow::__init__ - Exception caught for img.png: {e}""")

        if not isfile(self.img_path):
            logger.debug(f"""MainWindow::__init__ - Cannot import img.png file from `{self.img_path}` Because file does not exist""")
            img_label.setText('No image founded.')
        else:
            pixmap.load(self.img_path)
            pixmap = pixmap.scaledToWidth(100)
            img_label.setPixmap(pixmap)
            logger.debug(f"""MainWindow::__init__ - Imported img.png file from `{self.img_path}`.""")
        self.layout.addWidget(img_label)

So, I expected it to work, but unfortunately it didn't.

Android application logs are given below:
[2024-05-21 11:38:39,116] [DEBUG] - MainWindow::__init__ - Getting file path and file itself img.png
[2024-05-21 11:38:39,117] [DEBUG] - validation_tools::validate_pkg_format - Received request to validate: externalpy_pkg.resources
[2024-05-21 11:38:39,117] [DEBUG] - validation_tools::validate_pkg_format - Generated joined package name: externalpy_pkg/resources
[2024-05-21 11:38:39,118] [DEBUG] - validation_tools::validate_pkg_format - Found joined package name
[2024-05-21 11:38:39,118] [DEBUG] - validation_tools::validate_pkg_format - Returning package name: externalpy_pkg/resources
[2024-05-21 11:38:39,157] [DEBUG] - MainWindow::__init__ - Obtained file path for img.png: /data/user/0/org.qtproject.example.ExternalPyApp/files/tmpw_qg4e34img.png
[2024-05-21 11:38:39,160] [DEBUG] - MainWindow::__init__ - Obtained file path: /data/user/0/org.qtproject.example.ExternalPyApp/files/tmpw_qg4e34img.png
[2024-05-21 11:38:39,160] [DEBUG] - MainWindow::__init__ - Cannot import img.png file from `/data/user/0/org.qtproject.example.ExternalPyApp/files/tmpw_qg4e34img.png` Because file does not exist Returning empty file

The resource directory looks like this:

I have no idea what's wrong

@mrRaduga,
with resources.path(validate_pkg_format('externalpy_pkg.resources', logger), 'img.png') as path: is a ContextManager more info here.
Therefore, the file is only accessible within the with ... block. Once you exit the block, the resource is "destroyed" (not accessible anymore).
So I would suggest that you move your condition checks if not isfile(self.img_path): into the with ... block.
Basically, under this line: logger.debug(f"""MainWindow::__init__ - Obtained file path for img.png: {self.img_path}""").
Let me know if that does the trick!


Invalid toastr message on cancelling upload

Description: Invalid Toastr message when user cancel an image upload
Video Reference : https://vimeo.com/810787301/59dfe50101

@gaagul _a This error is provided by the neetocommons' axios interceptor. Let's provide a showError: false flag for the upload API and construct the error on our side.


fix cache missing

package_lookup_context.CurrentPackagePath generated by unpack_build_context is always empty.
So reflect.DeepEqual(g_daemon.context, context) is always false, it cause drop cache everytime.

That is true, sorry about that.

That is true, sorry about that.


Download issue

HA core version
2024.7.3
Intergration version
xxxxx
Device type and model
download
Used App
HACS
The description of problem
Download failed - Got status code 404 when trying to download https://github.com/georgezhao2010/midea_ac_lan/releases/download/v0.3.22/mieda_ac_lan.zip
The logs
No response

I also got this same error when I tried to download via the UI in Home Assistant Yellow.
2024-07-30 21:54:33.266 ERROR (MainThread) [custom_components.hacs] Download failed - Got status code 404 when trying to download https://github.com/georgezhao2010/midea_ac_lan/releases/download/v0.3.22/mieda_ac_lan.zip

Apparently it’s abandoned. But an earlier issue says someone else took up the repository.

Apparently it’s abandoned. But an earlier issue says someone else took up the repository.

Do you have a link for the new repository?

This is what I installed and it worked for me: https://github.com/nbogojevic/homeassistant-midea-air-appliances-lan

mieda or midea ac lan?

@AGreatSound  @RAiDX0r @nstephenh @themayna
as current repo is not active more than 6 months.
we have forked it and consider to maintain it in futuer.
https://github.com/wuwentao/midea_ac_lan
PR is welcome
Thanks


Fix for #1280 - Don't drop SyntaxError context for longStackTraces

SyntaxErrors in node.js are different from other Errors in JS - the first 3 lines of the stack specify the file, lineno, and code snippet where the error occurred.  Previously, longStackTraces were stripping this information, which made identifying the fix problematic.  This diff keeps this info in the stack.

I'm +1 on this change.

Also 👍

Sigh... futzing around with this to get a branch I can make the necessary changes needed to npm install from my github repo and have made a clusterfuck of things.  Closing and will re-request shortly.


Enable assertions

For TP Week 10 requirements.

Codecov Report

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

@@            Coverage Diff            @@
##             master     #151   +/-   ##
=========================================
  Coverage     80.16%   80.16%
  Complexity      894      894
=========================================
  Files           134      134
  Lines          2581     2581
  Branches        282      282
=========================================
  Hits           2069     2069
  Misses          439      439
  Partials         73       73

Continue to review full report at Codecov.

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


Extract observed plant totals updates helper function

The code that updates planting site observation stats is going to use the same
update logic that's used when completing plot observations. Extract a helper
function to make it easier to call from two places.
No change in behavior or functionality here.

[!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

#2027
#2026
#2025
#2024
#2023
#2022  👈
#2021
main

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

Join @sgrimm and the rest of your teammates on  Graphite

Merge activity

Jun 3, 7:50 PM EDT: Graphite rebased this pull request as part of a merge.


Use dict to store data for {nan,}arg{min,max} in certain cases

Certain array types don't support structed arrays (e.g., CuPy). In such cases, we may need to resort to a different data type. Details have been presented in https://github.com/dask/dask/issues/7997 and this is an attempt at solving that problem.
With this change, data stored during {nan,}arg{min,max} uses a dict instead of a structured array, provided that the array type fails during structured array creation. An additional change to _concatenate2 is required to handle the case where an attempt is made to concatenate dicts.

Thank you @pentschev for submitting a PR.  We'll need to rebase/merge with main since https://github.com/dask/dask/pull/8013 reorganizes the tests.
@pentschev can you spend a little time measuring if there is any negative performance impact for the NumPy/CPU case with the PR.  i would be surprised if there is any but confirming would be could.
@rabernat, if you have a moment, I'm curious if you've ever run into issues with structured arrays when using xarray and dask.  If not, apologies for the ping.
Lastly, while this PR generally looks ok to me we should find a third party to review before merging in.  @mrocklin if you have time that would be appreciated, however, we can also find additional parties to review

I don't see any noticeable difference in performance, see plot below:

The above computes da.argmin on 3D arrays of same length in all dimensions (e.g., (10, 10, 10) .. (1000, 1000, 1000)) and chunk sizes as half the dimension length in each dimensions, totalling 8 chunks. Each line represents whether is a "Dask main branch" or "PR 8014" run, and the choice of axis ([None, 0, 1, 2]). Probably not the best way to visualize but gives an idea that time is about the same when comparing both original Dask vs the change in this PR.
The particular setup used above is not based on any pre-existing benchmark but just something I quickly thought of, so if anyone has a specific performance measurement in mind of what they would like to see, please let me know.

Thanks @pentschev for taking the time to benchmark the changes. There is a planned released for this Friday dask/community#176 -- would you be ok to wait until Monday to merge in or do you think this is critical to getting into tomorrow ?

I think it's fine to wait until Monday. Both this and https://github.com/dask/dask/pull/8021 will be required for the sgkit case, so no rush to merge this while the other PR is pending.

@tomwhite you might be interested in this PR as it has come it in trying to accelerate some cupy-sgkit workloads. If you have comment that would be extremely appreciated

Thanks for the heads up @quasiben. Is there a sgkit workload that you had in mind? We use np.unique (with return_inverse, return_counts, etc) in some places, so the changes here should be useful for that.
cc @eric-czech

@tomwhite the particular workflow is regenie, specifically this PR is necessary for https://github.com/pystatgen/sgkit/blob/5d58d09f22f023c866e6b252662f84f83302eb61/sgkit/stats/regenie.py#L417 .
Also the np.unique case is used in https://github.com/pystatgen/sgkit/blob/5d58d09f22f023c866e6b252662f84f83302eb61/sgkit/stats/regenie.py#L471, although neither return_inverse nor return_counts are used there. Thus, I also have a PR to treat that one particular case in https://github.com/dask/dask/pull/8021. In the future we will hopefully deal with np.unique in a better way (see https://github.com/dask/dask/issues/2851), but this discussion is better left for https://github.com/dask/dask/pull/8021.
Also just to let you know, I have already local changes to support Dask+CuPy for regenie (requires both Dask PRs I mentioned above), we're only working with NVIDIA's legal team to get approval to contribute to sgkit, so hopefully next week I'll open a PR for that.

If folks don't have any objection I'll plan to merge in later this afternoon.

Thanks again @pentschev


Set the latest version as latest release

Hi @fpfcarvalho,
Thanks for all the commits. Do you think you can set the latest commit (now v3.2.1) as the latest release?
I believe when I'm loading https://cdn.fluidplayer.com/v3/current/fluidplayer.min.js I get v3.0.4 instead of the latest one.
Thanks

Bump

@Tony-MF Now the CDN is always updated to the last release, can I close this?


Instead of writing text from source file use text property to write t…

…ext of synthesized node
Fixes #4364

:+1:


Crashes with newest version 1.04

It seems I forgot to add the standalone flag when compiling the binaries. I've updated the binaries on the release, it should be fixed now.


Build Yoga.dll with Visual Studio 2017

Report

[X] I have searched existing issues and this is not a duplicate

Issues and Steps to Reproduce
Unable to build Yoga.dll using buck (to be fair I think I don't know how to use buck to build Yoga)
So... I decided to open the Facebook.Yoga.Desktop.sln solution. I then selected the "Release" target and "x86" platform before "Regenerate" on the Native/yoga project.
Linking failed due missing files (YGFloatOptional.cpp), I removed that file reference from "Source Files". Then compilation succeeded but build failed due to missing symbols (YGValueZero, YGValueUndefined and YGValueAuto and the log methods)
I had to add YGValue.h and YGValue.cpp and log.h and log.cpp to the project then regenerate and... yipee ! compilation and linking succeeded.
Now I'm asking whether to port Yoga code in Delphi (Object Pascal) or to wrap that Yoga DLL...

It's been like this since 1.8 (or when the rewrite in C++ happened).
Doubt the tests on Windows are running at all so use the compiled DLL on your own risk.

The C# bindings have not had a dedicated maintainer. We are removing them from the repo as part of cleanup for the next OSS release of Yoga.


I'm new to VS  and  mapDriver is hard to  build ,  can you give me the mapDriver.exe? thx  very much

I build the usemode app success but mapDriver was failed

How to build your usermode success?


Error loading nbextensions with pip install

Using the pip install with Jupyter version 4.x gives me this error
Error loading server extension nbextensions
    Traceback (most recent call last):
      File "/Users/tvu/anaconda/lib/python2.7/site-packages/notebook/notebookapp.py", line 1005, in init_server_extensions
        mod = importlib.import_module(modulename)
      File "/Users/tvu/anaconda/lib/python2.7/importlib/__init__.py", line 37, in import_module
        __import__(name)
    ImportError: No module named nbextensions

The pip install doesn't seem to work well for everyone, a number of people have had problems with it. You may find it easiest to run the python setup.py install from a cloned repo/downloaded archive. Alternatively, see below.
The error from the server log you quote indicates that:

the jupyter_notebook_config.json config file was correctly modified to load the nbextensions server extension
the jupyter_notebook_config.py file was not correctly modified to edit sys.path to allow the extension to be imported.

You can try to find the relevant jupyter_notebook_config.py config file using the following python snippet:
import os
from jupyter_core.paths import jupyter_config_path

for p in jupyter_config_path():
    conf_path = os.path.join(p, 'jupyter_notebook_config.py')
    if os.path.exists(conf_path):
        print(conf_path)

Hi @jcb91,
By cloning the repository again, and running this:
pip install --upgrade https://github.com/ipython-contrib/IPython-notebook-extensions/archive/master.zip --user

Everything seems to work fine now. I will close this issue then.
Thanks,
Tuan


[Triggerbot] Not shoting on Dust2 from T-Spawn to middle Double Door.

Hello Everyone,
It happens that the Triggerbot doesn't shot the Enemy standing in Doubledoor clear visible and the Triggerbot doesn't fire. Can someone explain this? And sometimes it shots sometimes not.
Thanks for declaration! I am totally confused.

It works fine on mine. I have ran stable and forked and edited versions of Osiris and all systems work. Maybe it's your config, you could try changing the shot delay. It could also be if the fov and crosshair plaecment isn't on the person ie(too high or low whatever) that it doesn't work then. I'm not sure. Feel free to send me a vid of what exactly happens and I will try help as best as I can


Failed to pull image : Error response from daemon: toomanyrequests:

Failed to pull image "aerokube/selenoid:1.10.7": Error response from daemon: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
Solutions we have tried so far.

We have purchased a Pro account and use Docker Login command  before pulling aerokube/selenoid image.
As selenoid directly talks to Docker API how can we make selenoid use those credentials while pulling the images from Docker Hub.

We hosted all the images in our own Docker registry and tried to pull the images from that registry. But selenoid by default pulling the images from Docker HUB - How to change this behavior ?

Using Docker

Your Docker API version is 1.40

Downloading Selenoid...

Fetching tags for image aerokube/selenoid
registry.ping url=https://index.docker.io/v2/
registry.tags url=https://index.docker.io/v2/aerokube/selenoid/tags/list repository=aerokube/selenoid

@ananthraj007 Selenoid is not pulling images automatically.

Fetching tags for image aerokube/selenoid
registry.ping url=https://index.docker.io/v2/
registry.tags url=https://index.docker.io/v2/aerokube/selenoid/tags/list repository=aerokube/selenoid
Pulling image aerokube/selenoid:1.10.7
x Failed to pull image "aerokube/selenoid:1.10.7": Error response from daemon: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit

@ananthraj007 please use CM as a binary for your platform. In that case CM will read your local authentication settings from ~/.docker/. When running the same as container, config files don't get mounted to container.

As you can see we are mounting the config file to container using -v /root/.docker:/root/.docker/

@ananthraj007 this was never tested.

@ananthraj007

Using Selenoid is container is OK and is recommended. Using CM is container is too complicated and error-prone according to our experience. You can even start Selenoid without using CM tool at all (just with regular docker run command).

CM binary is reading this information here: https://github.com/aerokube/cm/blob/8d351c8fc84937913d5059d797d422d496d2c723/selenoid/docker.go#L150-L170 You just need to provide your registry with --registry flag.

Thanks @vania-pooh  --registry worked. At least now we have a workaround to the issue.

@ananthraj007 this is not a workaround. This is how it expected to work.

@vania-pooh  It is work around for actual issue "Failed to pull image : Error response from daemon: toomanyrequests:"
Expected behavior : cm should be able to authenticate to Docker Hub! --> any other flag to pass credentials to cm ?

@ananthraj007 as I said these credentials are also read from Docker config file, so should just work when you do docker login before using CM on the same machine.

@vania-pooh  I as mentioned in the first comment we have tried using Docker Login command and share to Docker config to cm container but it is not working.

Same issue, docker login doesn't affect cm to use credentials

@Tarasovych it depends on the command you are using to run CM. Usually Docker credentials are stored under ~/.docker/config.json, so CM will read everything from this file.

@vania-pooh what if they're stored under /root/.docker/config.json?

I've built cm from source, added some debug to https://github.com/aerokube/cm/blob/1.8.2/selenoid/docker.go#L200, and I see that configFile is being read correctly. For some reason I still get rate limit error

In my case host in config.json is https://index.docker.io/v1/. I'm getting index.docker.io in registryHost, so configFile.AuthConfigs[registryHost] is empty

Having this issue as well. Running from root account, so docker credentials are at /root/.docker/config.json and it seems docker login doesn't affect cm. We're using docker hub registry, so --registry shouldn't be needed and regular pulls do work of course.
Is there any solution for this or some kind of workaround?


helpers: Add missing properties for create guild channel

Closes: #2513
Upstream: https://github.com/discord/discord-api-docs/commit/c2f74223c6f00d48a678bcd75892ff43ea5a2ddc

Codecov Report

Merging #2581 (ab570a8) into main (788bd85) will decrease coverage by 0.08%.
The diff coverage is 11.76%.

@@            Coverage Diff             @@
##             main    #2581      +/-   ##
==========================================
- Coverage   49.31%   49.23%   -0.09%
==========================================
  Files         384      384
  Lines        9950     9967      +17
  Branches      460      460
==========================================
  Hits         4907     4907
- Misses       5033     5050      +17
  Partials       10       10

Flag
Coverage Δ

*Carryforward flag

integration
48.50% <50.00%> (ø)

Carriedforward from 788bd85

unit
24.13% <0.00%> (-0.05%)
:arrow_down:

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files
Coverage Δ

helpers/channels/createChannel.ts
54.00% <11.76%> (-27.82%)
:arrow_down:

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


Activate if workspace have Python-related files

close: #4765
For #

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[x] Appropriate comments and documentation strings in the code.
[x] Has sufficient logging.
[x] Has telemetry for enhancements.
[x] Unit tests & system/integration tests are added/updated.
[x] Test plan is updated as appropriate.
[x] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

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

Codecov Report

Merging #10856 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10856      +/-   ##
==========================================
- Coverage   60.69%   60.67%   -0.02%
==========================================
  Files         580      580
  Lines       31512    31512
  Branches     4480     4480
==========================================
- Hits        19125    19119       -6
- Misses      11419    11421       +2
- Partials      968      972       +4

Impacted Files
Coverage Δ

src/datascience-ui/react-common/arePathsSame.ts
75.00% <0.00%> (-12.50%)
:arrow_down:

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
:arrow_down:

src/client/common/installer/productInstaller.ts
92.82% <0.00%> (ø)

Continue to review full report at Codecov.

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

There's so much stuff in Python community 😆

@lengthmin Thank you for the updates. We might want to wrap this in an experiment. See discussion on #4765.

@brettcannon it seem that this feature has already merged into main branch
should I close this pr?

Sorry, I forgot we put that change in already.
Yeah, unfortunately we have to balance triggering the extension with the fact that a lot of people only use Python occasionally and thus don't want to have it launch just because they run Python once every two months or something. Thanks for trying, though.


Only use version we know work as dependency

Fix #19
For people having this issue in the meantime, you can install from my git fork directly: npm -g install SBoudrias/generator-mean-seed#patch-1

Hello sir, can you please approve or provide a fix for this issue? I'm using your generator and this is a showstopper.

@awaemmanuel Just to make sure, you can install from my git fork directly: npm -g install SBoudrias/generator-mean-seed#patch-1

Also, any chance pinging @lukemadera directly helps? I can see he's been active the last few days on Github.

@SBoudrias just merged. Sorry, been swamped!


Suggestion: preserve both static and instance sides of class types when class is used in value position

Following up on #9358 and #6543. It hasn't stopped bothering me that generic identity functions don't exactly return their incoming types when it comes to classes.
declare function decorate<T>(Class: T): T;

class Foo {
	static bar = 42;
	baz () {}
}

//full type information seems present...
const DecoratedFoo = decorate(Foo);
DecoratedFoo.bar === 42; //ok
new DecoratedFoo().baz(); //ok

//but the instance side has been dropped from the DecoratedFoo type
let fooConstructor: typeof Foo; //ok
let foo: Foo; //ok
let decoratedFooConstructor: typeof DecoratedFoo; //ok
let decoratedFoo: DecoratedFoo; //error: cannot find name 'DecoratedFoo'

Would it be possible for decorate(Foo)'s return type to be class Foo, or something equivalent, instead of typeof Foo? It's an identity function; I'm trying to tell TypeScript that whatever I'm doing to the class, if anything, is invisible to the caller. The workaround in #6543 is a bit unfortunate in that it places the burden of the aliasing/merging on each and every caller instead of in the function that's doing the work.
Edit: Simpler repro, looks like it has nothing to do with identity functions and everything to do with a class  being assigned to a value, whether via local alias or passed as argument. Would just like a class to stay a class everywhere it goes :)
class Foo { }
const Bar = Foo;
let bar: Bar; //error: cannot find name 'Bar'

let decoratedFoo: DecoratedFoo; //error: cannot find name 'DecoratedFoo'

Because DecoratedFoo is just a value. you need to obtain it's type to use it in type position.

Because DecoratedFoo is just a value. you need to obtain it's type to use it in type position.
Right, that's the current limitation. You can't get to or preserve the instance type of a class once it enters a function as an argument; it's a bit of a black hole. I'd like to see that limitation lifted somehow.

@jwbay I see, and I've since read the linked issues. It's not clear what the best way to do this would be because implicitly introducing a name into the type space is problematic. Mixing the static and instance types into a single parameter type is kind of complicated because classes represent multiple types.
I recently trying to write a better declaration for dojo/lang/mixin and had a very frustrating experience. Because the function takes a single argument which is an object literal which represents a class was static and instance members using member names as conventions for what should go where. I realize that is the slightly off topic.

by the definition of the identity function the result is always of the type of its argument, if so then you don't need to call it to get the type, because you got the type already before you called it
problem is that it is not and identity function, it's a transformation from T to another U, but then why is it called identity?

@jwbay I don't see why one would expect
const DecoratedFoo = decorate(Foo);

to create a new manifest type. It doesn't make sense.
@aleksey-bykov I'm not entirely sure what you mean but I think you're saying the identity operation is lossy when applied to a class. Classes have a weird asymmetry.

@aleksey-bykov I'm very likely using the term 'identity function' incorrectly, sorry for the confusion. What I was trying to say, very poorly, was this

I'm trying to tell TypeScript that whatever I'm doing to the class, if anything, is invisible to the caller

@aluanhaddad I would disagree. If Foo has a type meaning attached to it, then it seems to me that applying X => X to it should carry the type to the result.
It might come down to the class keyword being painfully special since it creates both a value and a type.

@jwbay binding to a variable binds a value and a type but the variable does not itself represent the type in the same way that it represents the value. We say that the variable refers to a value of a type but it does not exist in type space as an identifier and it does not represent the type itself.
Part of the confusion is indeed the special nature of the class keyword. It actually declares a value and two types. The type having the name of the class is the type of the instances which result from instantiating it, while the type of the class itself is the type of the implicit variable to which it is bound.
I think it also comes from the notion that in most other languages, the object of the new operator is a type but in JavaScript it is a value.

@aluanhaddad Right, I suppose what I'm asking for is essentially for types to follow values around into new bindings. I feel like the behavior of the simpler repro in the OP is especially counter-intuitive (to anyone like myself who's not familiar with the intricacies of the type system :sweat_smile:). It's a local alias, so I would expect to be able to use Bar in exactly the same way I use Foo.
(Obviously the fix for the small repro is to add type Bar = Foo, but that breaks down a bit at scale as noted right above it)

@jwbay that does make sense laundry clay, but I can't see it making sense syntactically without a new kind of declaration. Something along the lines of
class Bar = decorate(Foo);

But there are conceptual problems with this beyond the simple introduction of an additional declarator, and I'm not convinced the use cases are sufficiently broad given all of the available workarounds. That said, something like this would be reasonable.

I recently was wondering about this:
class A {}

var B = A;

class C extends B {} // OK

var c:C; // OK
var b:B; // Error: cannot find name 'B'

Does this make any sense?

@benjamin-hg
yes once you did this:
var B = A;

you lost type-side of A and left alone with a value-side of it only

@aleksey-bykov
But successfully using C extends B  doesn't make B look like it has lost its type-side.
Let's extend the example:
class A {
	propOfA:string;
}

var B = A;

class C extends B { // OK
	propOfC:string;
}

var c:C; // OK
c.propOfC; // OK
c.propOfA; // OK! The type information must have been transferred through variable B
var b:B; // Error: cannot find name 'B'

Maybe I don't understand the concept of type-side and value-side, but I didn't find so much on it on the internet so far. Can you point me to a resource where this is explained?

@benjamin-hg I might be wrong myself, this is how I understand this problem:
TypeScript has 2 lookups for a given scope:

for values: name <-> value and its type
for types: name <-> type only

Classes in TypeScript a hybrids of a value-side and a type-side so they are registered in both lookups.
When you do var B = A; it gets gets saved to a values-lookup (because of var), but not into types-lookup. Note that it is saved with its type information.
Now, later when you declare
var b: B;

we only check the type-lookup and we don't find an entry for B there
Now, when you do class C extends B, since B is a class (type + value) we go to the values lookup and this is where we find B and everything works.
Not saying it is right, just it is what it is.

in order to pull the type information from a value-lookup you use the typeofoperator:
var b: typeof B;

@aleksey-bykov that is a good explanation. I think what confuses people is that the value side of a class is used in places where most other languages expect a type. Some examples are with new, with instanceof, and with extends. When the object of an extends clause is a class it looks like a type is being extended when in fact it is a value. That the clause can be part of a class declaration which introduces a new type is incidental. I think a good way to illustrate this is with anonymous class expressions.

What got clear to me at least (and it took me some time):
var B = A;

is the same as
var B: typeof A;

(Assuming A is the name of a class)

it's not the same:
var B = A; // transfers the type and the value of A to a value B, registring B in values

var B: typeof A // declares an uninitialized value B using only the type of A, also registring B in values

difference is that, although the following compiles, it won't run:
var B: typeof A;
class C extends B {}; // <-- unitialized value is used

whereas this one will:
var B = A;
class C extends B {}

@aleksey-bykov
Right, that was not very precise. From an execution point of view, the statements are different. But I'm still pretty sure that form a typing point of view (which I had) b looks the same for the compiler in both cases.

(Tooltip for 'B' in the TypeScript Playground)

@benjamin-hg that is not correct
var B = A;

creates a mutable reference to the value A, which has the inferred type typeof A.
var B: typeof A;

creates a mutable reference explicitly of the type typeof A with the value undefined.
Consider
const B : typeof A; // error

Looks like the request here is a way to access the return type of a function and/or a constructor. If so, then this should be covered by the request in https://github.com/Microsoft/TypeScript/issues/6606

@mhegazy I don't think it quite fits. The linked proposal would indeed let you get at the instance type with something like typeof new A(), but how would that then be joined with the static side type? I don't think there's a way to "reconstruct" what the class keyword gives you (value + static type + instance type) from its disparate pieces.

@jwbay weren't you already able to get the static side with typeof A?

 That is how a lot of the built ins and Dom classes are defined anyways.

@mhegazy in the original example this fragment
//full type information seems present...
const DecoratedFoo = decorate(Foo);
DecoratedFoo.bar === 42; //ok
new DecoratedFoo().baz(); //ok

//but the instance side has been dropped from the DecoratedFoo type
let fooConstructor: typeof Foo; //ok
let foo: Foo; //ok
let decoratedFooConstructor: typeof DecoratedFoo; //ok
let decoratedFoo: DecoratedFoo; //error: cannot find name 'DecoratedFoo'

The expectation was that the declaration,
const DecoratedFoo = decorate(Foo);
would introduce a new type, namely DecoratedFoo into the type space. Basically the expectation was that a class stored in a variable of a different name with introduce that variable's name itself into the type space as the instance type of invoking the variable with new.

The only assignment Target that I believe has this property is a module export. We have type assignments, and value level assignments, namespace level assignments, but no assignment target that combines all of their meanings outside of an export. Basically this is what it would allow for:
var C = class {
    static staticMember;

    instanceMember;
};
var i :C;
if (blah) i = new C();

That is 'typeof new DecoratedFoo()'

Variables are not valid type names. Values are not types.

That was what I said in comment https://github.com/Microsoft/TypeScript/issues/9359#issuecomment-228604574 but that is what @jwbay is expecting.

@mhegazy But a class name is a valid type name, and once a class passed somewhere as a value, it stops being a type. That's the stumbling block.
On a second look the enhanced typeof operator would be a suitable workaround for my use case, so I'll close.


Improvement: split svg.html

It is bad decision to have one big file for all icons.
Let's imagine: I want to change only one icon, but I have to override full file. Ugly!

Idea: split svg.html into svg/name.html
So if a developer wants to change some icon, he doesn't override the full svg.html, just a concrete small file.

Hi @vanodevium,
For this, I would've to disagree with you.
svg.html partial acts as a single source of truth for all SVGs used in the theme; basically a glorified dictionary, so that if one decides to change, extend or modify the SVGs, it would be one file, as opposed to n number of files and if a component needs an image, asks "Yo! svg.html, what's the content for SVG for this name?"
This makes lives a whole lot easier for me, anybody using this theme (which includes you too :wink:) and anyone coming from original Hermit theme.
However, I am open to suggestions as always.

Hi @1bl4z3r
Okay, maybe if you consider this file as the only source of truth, it makes sense.
But personally, I don't see a problem with leaving the if-else conditions in this file, and moving the svg content to micro includes.
Let me show the problem with a live example.

I changed the colors of the theme as a whole:
// Colors
$theme: #0074d9;
$text: hsl(204, 28%, 93%);
$theme-highlight: $theme; // cursor selection
$theme-background: #1a569e; // background
$theme-background-dark: #1c3a67; // footer background
$theme-dark: #152034; // dark color for base code and scroll
$typewriter: $theme;

But they did NOT override the icon colors, because the icon colors are sealed in this one file.
So I had to override the whole file to change only 2 values (scroll-to-top circle and arrow)

Yep yep yep.
My bad, I'd pooped that thing (scroll-to-top circle and arrow) into the theme. I didn't take in account for the color changes.
I am fixing that as I speak

Hi @vanodevium again,
This is fixed.
You can extend _colors.scss and edit following according to your choice
// Scroll to Top Default colors

$stt-stroke:#CCC;
$stt-circle:#3b3e48;
$stt-arrow:#018574;

Let me know if you find any issues with this.

Closing this issue.
Re-open if required


Cannot be installed with silverstripe CMS 5

silvershop cannot be installed with silverstripe cms 5. Composer file specifies 4

See PR #789

4.x-dev now will get you a stable SS5 version.


New rules

These commits will form the basis of ansible-lint v3 and include three new default rules

All tasks should have a name
Packages do not use state=latest
Commands check whether stuff should/will change using changes/removes/changed_when

Let me know if any of these are show stoppers, or if you have feedback
Note that v3 will also include #134, #135 and #136 but they'll probably get into v2.5.x or similar
Notifying common contributors ansible-lint for heads up
@pabelanger
@mschuchard
@MiLk
@siddharthist
@dmsimard
@shaleh
@msabramo

If you want to make a rule throw an error when state=latest is used, you might want to throw an error when state=present is used without any version specified, because your cluster might end up with different versions of software if you don't provision all the nodes at the same time.
You might want to be able to use state=latest for things like openssl which should always be in the latest version. So that you don't need to release a new version of your role every time they fix a major issue.

@MiLk, you're quite right about state=present, the only difficulty will be checking that the packages have a version, particularly with with_items, but I'll give it a go.
With openssl, I guess that's a good time to use skip_ansible_latest tag.

We do have some roles where the specific version of a minor software dependency isn't that important, and look a little like
In particular with tasks that are of the form:
- yum: state=present name={{ item }}
  with_items:
  - application-{{ application_version }}
  - minor-dependency{{ "-" ~ minor_dependency_version if minor_dependency_version is defined else "" }}

I am not at all sure how we'd check the former has a version, let alone the latter!

Sure. Let's say it's ok and you can use skip_ansible_lint for openssl.

There are now some breaking changes in 3.0.0 as a result of #107
If you have your own custom modules and use matchtask, the keys for the task['action'] have changed to avoid clashes.

#118 seems to have already made it into HEAD.

These changes have been merged and are in the released version 3.


Possible to select dates that aren't actual dates in the Date picker

By using the arrow keys, and not the actual date picker element, it is possible to select dates that don’t actually exist, such as Feb. 31.
Reported by: Tim Bursey
Acceptance Critera:
It should not be possible to select dates that don’t actually exist.

Recommend teams use the GoADatePicker component instead


fix: discord does not receive args

Description

Summary generated by Reviewpad on 24 Aug 23 22:12 UTC
This pull request fixes an issue where the Discord command was not receiving any arguments. The fix includes checking the length of the arguments and returning help if there are any arguments provided.

/run


ci: support explicit-member-accessibility

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring
[ ] Code style update
[x] Build or CI related changes
[ ] Documentation content changes

@waterplea @nsbarsukov @vladimirpotekhin please, review me)


Guestbook

How to categorize this PR?
/area testing
/kind test
/priority normal
What this PR does / why we need it:
Azure guestbook tests are flaky, see #2423
This PR increases the timeout to wait for the guestbook app to serve its URLs from 10min to 20min and the timeout to wait for the namespace to be deleted (at the end of the test) from 5min to 10min.
It adds additional dumping logic to dump the nginx ingress controller logs at the end of a failed test so that we have a better chance to analyze the cause of the flakes.
Which issue(s) this PR fixes:
Maybe improves #2423, at least helps to better understand its potential causes.

/assign @ialidzhikov

Addressed comments, PTAL