Windows: ionic resources creates invalid paths (for Mac) in config.xml (slash vs. backslash)

When calling ionic resources on Windows from a terminal prompt all the images are correctly generated and config.xml is updated with the new image paths. But as soon as you want to build this on Mac, you get cp errors as Mac can't find these paths. Ultimately your app may be built with old images as splash and icon.
Reason for this is that when generating on Windows you get \ in the paths, on Mac you get /. As / seems to work on both platforms, ionic resources should probably replace \ by / on Windows.

+1
Side effect of this is that when you have two developers, one on Windows, one on Linux, every time one of them runs ionic resources, there's a big diff in config.xml like below due to slashes-to-backslashes conversion or the other way around
-    <icon src="resources/android/icon/drawable-ldpi-icon.png" density="ldpi"/>
-    <icon src="resources/android/icon/drawable-mdpi-icon.png" density="mdpi"/>
-    <icon src="resources/android/icon/drawable-hdpi-icon.png" density="hdpi"/>
-    <icon src="resources/android/icon/drawable-xhdpi-icon.png" density="xhdpi"/>
-    <icon src="resources/android/icon/drawable-xxhdpi-icon.png" density="xxhdpi"/>
-    <icon src="resources/android/icon/drawable-xxxhdpi-icon.png" density="xxxhdpi"/>
-    <splash src="resources/android/splash/drawable-port-ldpi-screen.png" density="port-ldpi"/>
-    <splash src="resources/android/splash/drawable-port-mdpi-screen.png" density="port-mdpi"/>
-    <splash src="resources/android/splash/drawable-port-hdpi-screen.png" density="port-hdpi"/>
-    <splash src="resources/android/splash/drawable-port-xhdpi-screen.png" density="port-xhdpi"/>
-    <splash src="resources/android/splash/drawable-port-xxhdpi-screen.png" density="port-xxhdpi"/>
-    <splash src="resources/android/splash/drawable-port-xxxhdpi-screen.png" density="port-xxxhdpi"/>
+    <icon src="resources\android\icon\drawable-ldpi-icon.png" density="ldpi"/>
+    <icon src="resources\android\icon\drawable-mdpi-icon.png" density="mdpi"/>
+    <icon src="resources\android\icon\drawable-hdpi-icon.png" density="hdpi"/>
+    <icon src="resources\android\icon\drawable-xhdpi-icon.png" density="xhdpi"/>
+    <icon src="resources\android\icon\drawable-xxhdpi-icon.png" density="xxhdpi"/>
+    <icon src="resources\android\icon\drawable-xxxhdpi-icon.png" density="xxxhdpi"/>
+    <splash src="resources\android\splash\drawable-port-ldpi-screen.png" density="port-ldpi"/>
+    <splash src="resources\android\splash\drawable-port-mdpi-screen.png" density="port-mdpi"/>
+    <splash src="resources\android\splash\drawable-port-hdpi-screen.png" density="port-hdpi"/>
+    <splash src="resources\android\splash\drawable-port-xhdpi-screen.png" density="port-xhdpi"/>
+    <splash src="resources\android\splash\drawable-port-xxhdpi-screen.png" density="port-xxhdpi"/>
+    <splash src="resources\android\splash\drawable-port-xxxhdpi-screen.png" density="port-xxxhdpi"/>

so either we have constant noise in the diffs or one of use has to do git reset --hard after the build :)

Hi, any ETA on this? The offered PR should work.

Hello all! I will be closing this issue as a duplicate of https://github.com/driftyco/ionic-cli/issues/913. Thanks for using Ionic!

@jgw96 I do not believe it is a duplicate of #913. This issue is about the resources being created but with a wrong path ('' instead of '/').

@jrobichaud is right. This is absolutely NOT a duplicate of #913. In fact, with the backslash, if you run "ionic state restore", it breaks even on windows.


add fallback for search

This PR implements fallback search, using grep by default.
It can be disabled in configuration like this
fallback_search = {
  command = false,
}

I guess there is a better way to call it, rather than "fallback search".

Sorry, but this doesn't make a lot of sense to me to add this as a config option.
Just do the logic you want and set the correct options for command etc before calling setup


Clockwork Handler is misnamed as Gold Tank Broker

Also the token settings include an image from the pf2e-ap178-punks-in-a-powderkeg module and I am not sure whether that's the system detecting I have the module installed (but not enabled), is intended, or is a typo


Reduce clicking required to label

Remove need to click 'OK' to save behavior labels

Is it possible to select a species, then a behavior, then draw a box?

Moved to LabelStudio for labeling


chore(main): release 1.3.0

:robot: I have created a release beep boop
1.3.0 (2024-08-21)
Features

adiciona cache no docker build (4042d90)
normaliza textos para melhor geração de audios (560186e)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.3.0
:sunflower:


[KYUUBI #6271] Upgrade kafka-clients from 3.5.1 to 3.5.2

:mag: Description
Issue References 🔗

This pull request fixes #6271
Describe Your Solution 🔧
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Types of changes :bookmark:

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests

Checklist 📝

[ ] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

Thanks, merged to master/1.9/1.8


Grid Creation Problem

Hello everyone,
I'm following these  tutorials and as I'm trying to insert a grid in my video I get this error:

Does anyone know what is the problem?

NVM guys, I found the error. Just replace:
label = Text(f"{coord_point}",font="Arial",stroke_width=0).scale(self.labels_scale)
with
label = TextMobject(f"{coord_point}",font="Arial",stroke_width=0).scale(self.labels_scale)
in line 72.

NVM guys, I found the error. Just replace:
label = Text(f"{coord_point}",font="Arial",stroke_width=0).scale(self.labels_scale)
with
label = TextMobject(f"{coord_point}",font="Arial",stroke_width=0).scale(self.labels_scale)
in line 72.


Opt_einsum contractors are slow for copy nodes

I mentioned this in the past however I haven't been able to solve it so I am opening this to keep it into account.
The issue is that the opt_einsum contractors treat Copy nodes as normal nodes and do not exploit net.contract_copy_node, which may be sub-optimal: For example, in the contraction "bca,cdb,dbf,afc", b and c appear more than two times and hence require Copy nodes. In this case tensornetwork scales worse than pure opt_einsum (\chi is the tensor size):

This is for the optimal contractor and numpy. Repeating the same for a contraction without copy nodes (eg. "bca,cdg,dbf,afg"), both libraries perform similarly.

I have done some work on fixing this, however my solution requires either extending net.contract_copy_node for copy nodes with dangling edges or doing a different einsum contraction for contract_between from scratch (which currently has some bugs). If someone has of a way to avoid this, feel free to jump in. For now I assume that this is not a priority until it is decided how the opt_einsum contractors will fit into the new (non-TensorNetwork) API.

Huh. I'm curious as to how opt_einsum deals with these mutliple indicies. The API we use through them only contracts 2 nodes at a time.

I think ultimately the solution to this is to just implement hyper edges. That is being tracked in #401


feat: WebForms CTA integration

WebForms Project Integration With CTA
Description

Add WebForms handling in CTA Code Replacer
Update W2B parameters to serve as a library for CTA

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Before digging into the code, i have a high level question/concern. The CTA.Rules.Update project should not contain any porting logic, and we now have two branches in there (WCF & WebForms). Why can't these be in the PortCore project? Let me know if we need to discuss this.

@marknfawaz I see what you mean. I still think it makes sense to modify CTA.Rules.Update, but a more sustainable approach might be to create WcfProjectRewriter and WebFormsRewriter classes, and use SolutionPort to call a factory to construct the appropriate project rewriter based on project type.

Before digging into the code, i have a high level question/concern. The CTA.Rules.Update project should not contain any porting logic, and we now have two branches in there (WCF & WebForms). Why can't these be in the PortCore project? Let me know if we need to discuss this.

@marknfawaz I see what you mean. I still think it makes sense to modify CTA.Rules.Update, but a more sustainable approach might be to create WcfProjectRewriter and WebFormsRewriter classes, and use SolutionPort to call a factory to construct the appropriate project rewriter based on project type.

@jonlouie "I still think it makes sense to modify CTA.Rules.Update" Not sure what u mean by that. As mentioned, the update project shouldn't be aware of how the actions are used, it should just run these actions. As for the Rewriters, yes, that should be our approach, but it should be similar to what we have in the PortCore project.


kendra_retriever_samples/requirement.txt needs update

Existing requirements.txt version refers to a wrong langchain version

langchain==0.0.219  >>> does not exist
+langchain==0.0.21
boto3>=1.26.159
openai
anthropic
streamlit

after updating langchain to 0.0.21, prompt suggest to use the pip 2020 feature resolver:
pip install -r requirements.txt --use-feature=2020-resolver
Resulted in :
Successfully uninstalled pydantic-2.0.3
Successfully installed pydantic-1.10.12

this error happened, when using cloud9.

Hi, I can confirm langchain==0.0.219 does exist and installs fine for me. I think the most likely cause of this is you need to upgrade your Python version? Per the link, v0.0.219 requires Python >=3.8.1, <4.0
I probably wouldn't recommend downloading to such an old version as 0.0.21 - the library has changed quite a bit since then.

Upgraded to langchain 0.0.263


Kitty won´t close window after exiting yazi if an orphaned mpv task is running

What system are you running Yazi on?
Linux Wayland
What terminal are you running Yazi in?
kitty 0.36.4
yazi --debug output
❯ yazi --debug

Yazi
    Version: 0.3.3 (da1b236 2024-10-11)
    Debug  : false
    OS     : linux-x86_64 (unix)

Ya
    Version: 0.3.3 (da1b236 2024-10-11)

Emulator
    Emulator.via_env: ("xterm-kitty", "")
    Emulator.via_csi: Ok(Kitty)
    Emulator.detect : Kitty

Adapter
    Adapter.matches: Kgp

Desktop
    XDG_SESSION_TYPE           : Some("wayland")
    WAYLAND_DISPLAY            : Some("wayland-1")
    DISPLAY                    : Some(":0")
    SWAYSOCK                   : None
    HYPRLAND_INSTANCE_SIGNATURE: Some("b65773bea9b912a41cfcbc789fb2e60a07e3d0c1_1728652043_1885625831")
    WAYFIRE_SOCKET             : None

SSH
    shared.in_ssh_connection: false

WSL
    WSL: false

Variables
    SHELL              : Some("/usr/bin/zsh")
    EDITOR             : Some("/usr/bin/nvim")
    VISUAL             : None
    YAZI_FILE_ONE      : None
    YAZI_CONFIG_HOME   : None

Text Opener
    default     : Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })
    block-create: Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })
    block-rename: Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })

Multiplexers
    TMUX               : false
    tmux version       : tmux 3.5a
    tmux build flags   : enable-sixel=Unknown
    ZELLIJ_SESSION_NAME: None
    Zellij version     : No such file or directory (os error 2)

Dependencies
    file             : 5.45
    ueberzugpp       : No such file or directory (os error 2)
    ffmpegthumbnailer: 2.2.3
    magick           : 7.1.1-39
    fzf              : 0.55.0
    fd               : 10.2.0
    rg               : 14.1.1
    chafa            : No such file or directory (os error 2)
    zoxide           : 0.9.6
    7z               : 17.05
    7zz              : No such file or directory (os error 2)
    jq               : 1.7.1

Did you try the latest nightly build to see if the problem got fixed?
Yes, and I updated the debug information above (yazi --debug) to the nightly that I tried
Describe the bug
Kitty won´t close window after exiting yazi if an orphaned mpv task is running until mpv itself is closed.
When trying to kill kitty (hyprland killactive in my case) it tells you that yazi is still running.
Other terminals like alacritty, wezterm and foot just close the window and keep mpv running.
Minimal reproducer

kitty yazi
open a .mkv file with mpv (default config opens it with orphan=true
quit yazi with q
kitty shows an emtpy window (cursor but no prompt) until mpv is closed

Anything else?
Seeing that other terminals like alacritty, wezterm and foot don´t have this problem Kitty might be the culprit.

https://github.com/user-attachments/assets/ea28eee9-a0e3-41a6-ae4b-7353e0cc6d86
I press q in yazi and it doesn´t exit, but keeps the kitty window with the cursor open.
Also, it only seems to happen when I run kitty yazi. If I just execute yazi within kitty it seems to close the process just fine.

Hmmm that's weird, the yazi process has indeed gone.
Have you tried kitty -e yazi? Also, what's your mpv command? Adding --force-window, i.e. { run = 'mpv --force-window "$@"', orphan = true } does that work?
This looks like a kitty bug, if none of these works, I suggest opening an issue with kitty to ask for their support.

Sadly, kitty -e yazi behaves the same. This doesn´t come as a surprise since kitty doesn´t document the flag in their manpage.
Well, let's see what the kitty people say.

It looks like kitty has responded in https://github.com/kovidgoyal/kitty/issues/7966#issuecomment-2408284669, so let's just close this one since it's a kitty-specific issue.


MINOR: Gradle build optimizations

Applies various build optimizations to take full advantage of Gradle's incremental build feature, and Gradle's local build caching feature. The optimizations will help to reduce build times for developers when running builds locally.
Specific optimizations:

Enable the local build cache
The local build cache avoids re-running tasks whose inputs have not changed,
even if the build output has been cleaned or if the build is executed in a
different directory.

Instruct Gradle to ignore version files when doing up-to-date checks
This can save a lot of developer time when working locally and creating a lot of commits. In particular, tests won't be re-executed simply because the commit ID changes (unless one of the other test inputs changes).

Update createVersionFile tasks support incremental builds and build caching
The version files are no longer recreated unless one of their inputs changes (the commit ID or the version property). The tasks are also marked as cacheable (the cached version can be used even on a clean build if the inputs are the same).
These tasks don't take a lot of time to run, but optimizing them shows how other tasks could be optimized if needed.

Set the root project name in settings.gradle
The root project name is used to name the build as a whole. It is used in reports and other types of artifacts. If a root project name is not set, then the containing directory's name is used as the root project name.
It is a best-practice to explicitly set the root project name. Otherwise, the root project name could change if the directory the project belongs to is different from the desired name (for example, of someone checks out the repository into a directory named differently from the default).

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@ijuma it looks like you review a lot of these changes.

Full disclosure: I'm an engineer that works for Gradle (on Gradle Enterprise, and I found these optimizations while using the project to test out Gradle Enterprise features.
Also, FWIW, I was able to reduce overall test execution time by more than half by using Gradle Enterprise's support for Test Distribution. This might be something worth considering at some point in the future.

One last comment: these changes should be compatible with Gradle 7.

Thanks for the PR! Looks like the build failed:

[2021-04-06T20:21:12.201Z] * What went wrong:
[2021-04-06T20:21:12.201Z] A problem occurred evaluating settings 'Kafka_kafka-pr_PR-10490'.
[2021-04-06T20:21:12.201Z] > No signature of method: org.gradle.initialization.DefaultProjectDescriptor.name() is applicable for argument types: (String) values: [kafka]
[2021-04-06T20:21:12.201Z]   Possible solutions: any(), wait(), path(), dump(), setName(java.lang.String), any(groovy.lang.Closure)

Also, have you measured a performance improvement without Gradle Enterprise?

Thanks for the PR! Looks like the build failed:

Whoops! That was silly of me. Somehow I accidentally deleted an important equals sign when getting these commits ready for submission. Should be fixed now (tested locally this time).

Also, have you measured a performance improvement without Gradle Enterprise?

For locally executed builds, the build caching can really help avoid re-running tasks with inputs that haven't changed since the last time they were executed. This can save a lot of time when working locally.  You'll observe a noticeable performance gain when running clean builds. Additionally, tests aren't executed anymore if all you do is make a commit that changes something not directly related to the tests themselves, like build logic (although if you change the test tasks, then they'll of course re-run).
On the CI-server side, these changes have little to no impact at all, unfortunately.
This is already a pretty mature build and the optimizations in this PR are relatively minor. By far, the vast majority of the build time is taken up executing tests. In particular, the tests in the core module tend to take a long time to execute, even with parallel test execution enabled.

On the CI-server side, these changes have little to no impact at all, unfortunately.

The CI build improvements will come from using the Gradle Enteprise remote build cache on CI builds to re-use outputs from previous builds. There is no remote build cache configured we cannot see these improvement.

For locally executed builds, the build caching can really help avoid re-running tasks with inputs that haven't changed since the last time they were executed. This can save a lot of time when working locally. You'll observe a noticeable performance gain when running clean builds. Additionally, tests aren't executed anymore if all you do is make a commit that changes something not directly related to the tests themselves, like build logic (although if you change the test tasks, then they'll of course re-run).

Does the clean task delete the local caches or not? If so, how would one force that in case of a bug? It would be good to update the Readme.md with this info.

Does the clean task delete the local caches or not?

It does not delete the local build cache.

If not, how would one force cache deletion in case of a bug?

It's best to address bugs like those by finding the offending task and fixing it to properly declare it's inputs and outputs (or if the issue is with a third-party plugin, to file a bug report against the plugin).
But in a pinch, you can pass --rerun-tasks to instruct Gradle to skip the cache and up-to-date checks. If a cache entry has been corrupted, this will also tell Gradle to overwrite the cache entry with the new task output.
However, these types of issues are quite rare. It is also best to avoid training users to use --rerun-tasks (as it eliminates all of the incremental build and build cache benefits).  For these reasons, I'm somewhat hesitant to update the README.


[Feature]Can you provide multi-environment deployment capabilities?(能否提供多环境部署的功能)

When developing a normal project, there are several environments, such as: dev (development environment), test (test environment), staging (pre-release environment), prod (production environment)
When I have developed 10 components according to one requirement in dev, easyscheduler does not currently support upgrading to other environments. It is impossible to say that when I go online, I will redo the 10 new components in each environment again.
Isn't this similar to writing a java program, and then rewriting it in other environments without a git?
I hope to provide such a function, at least like azkaban can use git to manage files, div->test directly branch and merge, and then submit the zip package. I don't know if easyscheduler can provide such a function.

正常项目开发的时候，都是有好几个环境的，比如：dev（开发环境），test（测试环境），staging（预发布环境），prod（生产环境）
当我在dev已经根据一项需求开发了10个组件，easyscheduler目前不支持升级到其他环境，不可能说我上线的时候再单独把本次新增的10个组件在各个环境重做一遍吧？
这不就类似我写个java程序，然后不用git，直接一个一个类在其他环境重写一遍吗
希望能提供这样的功能，起码像azkaban是可以用git来管理文件，dev->test的时候直接分支合并，然后提交zip包就可以了。不知道easyscheduler能否提供这样的功能。

If the process definition has import and export functions, can you solve the problem you said?

如果工作流定义有导入和导出功能，是否就可以解决你说的这个问题么？

?

it's better have database backup and restore then new environment will be look like the origin
最好有数据库的备份和恢复，这样新搭建的环境直接可以原样恢复。

Very good idea， is there anybody who want to implement it and contribute to community?

please use English，English is needed, and you also can append Chinese description,  thx!

if anybody want to implement this feature , please reply or let me know!

@chongchongzi ， please review “process definition import and export functions” and check whether can support cross-cluster or not, thank you!

The workflow import and export function can support cross environment. As long as the project name, resource name and data source name are consistent, it is OK
（工作流导入导出功能可以支持跨环境，只要项目名称，资源名称，数据源名称一致可以了）

If the import and export cannot meet the current needs, when we need to develop again, please pay attention to keeping the environment equivalent.


Unexpected setting of boundary condition behavior

I wanted to see if anyone had insight into (what I think is) strange behavior when trying to set boundary conditions on my own. My use case is a bit non-standard, I am trying to add one matrix generated by a bilinear form (let's call this matrix A) to another matrix generated by a nonlinear form (lets call this matrix N).
The first implementation I tried was to assemble A with the boundary conditions applied, and then to assemble N and eliminate the rows and columns corresponding to the boundary conditions, while placing a 0 on the diagonals. The code I have to do that looks something like this.
 aVar->AddDomainIntegrator(new VectorDiffusionIntegrator());
 aVar->Assemble();
 aVar->EliminateEssentialBC(ess_bdr, x_bdr, rhs.GetBlock(0));
 aVar->Finalize();
 SparseMatrix &A(aVar->SpMat());

 NonlinearForm *Non(new NonlinearForm(Xh_space));
 Non->AddDomainIntegrator(new MyNonLinearForm());
 SparseMatrix *N = dynamic_cast<SparseMatrix*>(&Non->GetGradient(u));

 for (int i = 0; i < ess_tdof_list.Size(); i++)
        {
           Non->EliminateRowColDiag(ess_tdof_list[i], 0.0);
        }

*N += A;

However, this appears to be giving me the wrong the solution (although it is close to being correct). On the other hand when I add them both and then apply the boundary conditions I get the correct answer
  aVar->AddDomainIntegrator(new VectorDiffusionIntegrator());
  aVar->Assemble();
  aVar->Finalize();
  SparseMatrix &A(aVar->SpMat());

  NonlinearForm *Non(new NonlinearForm(Xh_space));
  Non->AddDomainIntegrator(new MyNonLinearForm());
  SparseMatrix *N = dynamic_cast<SparseMatrix*>(&Non->GetGradient(u));

  *N += A;

   for (int i = 0; i < ess_dof.Size(); i++)
          if (ess_dof[i] < 0)
          {
              N -> EliminateRowCol(i, x_bdr(i), rhs.GetBlock(0));
          }

Based on my reading of the documentation both of these approaches should have yielded the same answer. Am I missing something and it is obvious why they do not?

Hi @nnpp-schneier,
I will let @jandrej comment if this setup is the best way to accomplish your goals, but I can comment and hopefully clarify the handling of boundary conditions.
Borrowing from https://github.com/mfem/mfem/issues/1720#issuecomment-684174484, lets say A, N the solution x, and the rhs.GetBlock(0) vector b have the interior/boundary form
A = | A_I  A_IB |   N = | N_I  N_IB |   x = | x_I |   b = | b_I |
    | A_BI A_B  |       | N_BI N_B  |       | x_B |       | b_B |

Then in the first approach,  aVar->EliminateEssentialBC(ess_bdr, x_bdr, rhs.GetBlock(0)); transforms
A = | A_I  A_IB | -->  A = | A_I  0 |
    | A_BI A_B  |          |  0   I |

and
b = | b_I | --> b = | b_I - A_IB x_B |
    | b_B |         | x_B            |

while  Non->EliminateRowColDiag(ess_tdof_list[i], 0.0); transforms
N = | N_I  N_IB | -->  N = | N_I  0 |
    | N_BI N_B  |          |  0   0 |

Therefore the final matrix and vector are
| A_I + N_I  0 |   | b_I - A_IB x_B |
|     0      I | , | x_B            |

In the second approach, the call N -> EliminateRowCol(i, x_bdr(i), rhs.GetBlock(0)); produces final matrix and vector
| A_I + N_I  0 |   | b_I - (A_IB + N_IB) x_B |
|     0      I | , | x_B                     |

which has the additional N_IB x_B component in the interior portion of the right-hand side.
This is a small difference that only matters for nonzero essential boundary conditions.
Hope this helps,
Tzanio

I believe the question was answered, so I am closing this issue for now. Feel free to reopen if you have further questions.

I believe the question was answered, so I am closing this issue for now. Feel free to reopen if you have further questions.


Gintro fails with latest 1.9.1, and GtkFileChooser fails with GTK 4.10

Yes, we know, and it is not a real gintro issue this time :-)
For the Nim compiler, the issue is related to
import macros

macro m1(s: string): untyped =
  var ProcID {.compileTime, global.}: int
  inc(ProcID)
  echo s, ProcID

proc main =
  echo "calling macro m1"
  m1("Macro argument")
  m1("Macro argument")
  m1("Macro argument")

nim c m.nim
Hint: used config file '/home/salewski/Nim/config/nim.cfg' [Conf]
Hint: used config file '/home/salewski/Nim/config/config.nims' [Conf]
Hint: used config file '/home/salewski/.config/nim/nim.cfg' [Conf]
.......................................................................
.
Macro argument1
Macro argument1
Macro argument1
CC: m.nim

Desired output would be

Macro argument1
Macro argument2
Macro argument3

We don't know if they did it by intent. {.compileTime, global.} is used in gintro for the mconnect macro and at a few other locations. Using module global variables may fix it, but that is ugly and may lead to name conflicts for a lot of compile-time macro variables. Using a compiler version 1.9.1 with git hash from beginning of 2023 seems to fix it for now, we will see how Nim 2.0 may behave. Maybe Nim 1.6.12 still works?
The other issue seems to be related to latest GTK 4.10. The example from https://ssalewski.de/gtkprogramming.html#_filechooserdialog does not work any more, we get a lot of
(filechooserdialog:11122): Gtk-CRITICAL **: 21:59:32.837: Error building template for list item: .:0:0 Invalid object type 'GtkFileChooserCell'

Converting that Nim example back to plain C leads to a working program. So debugging took us again a full day, with the funny result that compiling with
nim c --passL="-lgtk-4" filechooserdialog.nim

makes it working. Was not easy to guess, but the gcc compile command
gcc -Wall t.c -o t `pkg-config --cflags --libs gtk4`

with
pkg-config --cflags --libs gtk4
-I/usr/include/gtk-4.0 -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/lib64/libffi/include -I/usr/include/harfbuzz -I/usr/include/freetype2 -I/usr/include/libmount -I/usr/include/blkid -I/usr/include/fribidi -I/usr/include/cairo -I/usr/include/libpng16 -I/usr/include/pixman-1 -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/graphene-1.0 -I/usr/lib64/graphene-1.0/include -mfpmath=sse -msse -msse2 -pthread -lgtk-4 -lpangocairo-1.0 -lpango-1.0 -lharfbuzz -lgdk_pixbuf-2.0 -lcairo-gobject -lcairo -lgraphene-1.0 -lgio-2.0 -lgobject-2.0 -lglib-2.0

gave a hint. We still have to investigate the actual reason, maybe some resources have to be included by the linker.

Progress: https://github.com/nim-lang/Nim/pull/21502

Yes, it is working already since some days, due to your fast patch and due to released GTK 4.11. But I will let this issue open for a few weeks, in case someone should have trouble with GTK4.10 still. Thanks.


When writing unit tests with expressions that always evaluate to true, an NPE is thrown

Description:
When adding unit tests to master with expressions that always evaluate to true, an NPE is thrown. Need to investigate the cause for this.
function func() returns int {
    return 1;
}

public function test() {
    boolean b = func is function () returns int;
    panic error(string `${b}`);
}

The above example throws an NPE at https://github.com/ballerina-platform/ballerina-lang/blob/master/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/AnnotationUtils.java#L112
public function test() {
    int a = 12;
    boolean b = a is int;
    panic error(string `${b}`);
}

The above example throws an NPE at https://github.com/ballerina-platform/ballerina-lang/blob/master/bvm/ballerina-runtime/src/main/java/io/ballerina/runtime/internal/TypeChecker.java#L2524
Describe your task(s)
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Probably same as this https://github.com/ballerina-platform/ballerina-lang/issues/29887


[enhancement] --cppcheckargs flag was missing

Analyzer config options can be forwarded to the analyzers by --saargs and --tidyargs flags. This was missing for Cppcheck, so --cppcheckargs flag has now been introduced.
Fixes #3936

@bruntib please rebase this PR and then we can merge.


ipython5

I just upgraded to ipython5 and now when I send a large code fragment, the code is no longer sent properly.  e.g. for any code fragment:
def something():
    print('')

only this is sent:
In [1]: %cpaste -q

and the code does not appear.  I guess something has changed so that the newline insertion is not sent to start the paste, and so the text does not appear properly.

I have started a rewrite of SendTextPlus https://github.com/randy3k/SendREPL
It should works with ipython 5, unfortunately, it only supports terminal and iterm now.


Getting "Authenication required" on first push to cluster

Getting this when pushing the first app to new cluster.
$ git push flynn master
remote: Authentication required
fatal: Authentication failed for 'https://<cluster domain>/pigeons-curse-harty.git/'

It works if I try to push again.
Note: I have not downloaded the CA-file if that makes any difference.

What git version are you using? Please post the output of a failing push with GIT_TRACE=1 set.

$ GIT_TRACE=1 git push flynn master
13:46:11.181848 git.c:346               trace: built-in: git 'push' 'flynn' 'master'
13:46:11.183214 run-command.c:351       trace: run_command: 'git-remote-https' 'flynn' 'https://git.<cluster domain>/zebras-invite-gore.git'
13:46:12.746628 run-command.c:351       trace: run_command: 'git credential-osxkeychain get'
13:46:12.747378 run-command.c:199       trace: exec: '/bin/sh' '-c' 'git credential-osxkeychain get' 'git credential-osxkeychain get'
13:46:12.750910 git.c:555               trace: exec: 'git-credential-osxkeychain' 'get'
13:46:12.751460 run-command.c:351       trace: run_command: 'git-credential-osxkeychain' 'get'
13:46:14.273043 run-command.c:351       trace: run_command: 'git credential-osxkeychain erase'
13:46:14.273848 run-command.c:199       trace: exec: '/bin/sh' '-c' 'git credential-osxkeychain erase' 'git credential-osxkeychain erase'
13:46:14.276977 git.c:555               trace: exec: 'git-credential-osxkeychain' 'erase'
13:46:14.277588 run-command.c:351       trace: run_command: 'git-credential-osxkeychain' 'erase'
13:46:14.307501 run-command.c:351       trace: run_command: '/usr/local/bin/flynn git-credentials erase'
13:46:14.308195 run-command.c:199       trace: exec: '/bin/sh' '-c' '/usr/local/bin/flynn git-credentials erase' '/usr/local/bin/flynn git-credentials erase'
remote: Authentication required
fatal: Authentication failed for 'https://git.<cluster domain>/zebras-invite-gore.git/'

kwando at alice in ~/<project path> on master

$ git --version
git version 2.3.2 (Apple Git-55)

I have successfully replicated the issue with the stock git on OS X:

Install the git-credential-osxkeychain helper.
Enable the helper with git config --global credential.helper osxkeychain
Bootstrap and configure a development cluster and do a git push.
The push is successful. During the push, no credential is available from osxkeychain and is correctly read from our flynn credential helper and stored into the keychain by the osxkeychain helper.
Re-bootstrap a cluster at the same URL, creating a new, different password and do a git push.
The old password is read from the keychain by osxkeychain and the result is a 401 error.
The old credential is erased from the helpers.
The subsequent push succeeds as the old credential is no long available from osxkeychain and the process repeats from step 4.

I have been unable to replicate this issue with git v2.5.1 from Homebrew.
Since this issue is specific to those who are using stock git, have a third-party credential helper installed, and have re-bootstrapped with the same URL and a different key I don't think we need to do anything about this since it resolves itself after another push. If anyone finds a config flag or response code that solves this issue, I'd be happy to accept a PR. It doesn't seem like any more effort is warranted to investigate non-workaround solutions to this.

I've got the following. This is a cluster my team created and I grabbed the info from ~/.flynnrc. Anything I was missing there? Thanks!
$ GIT_TRACE=1 git push flynn master
23:44:38.348416 git.c:350               trace: built-in: git 'push' 'flynn' 'master'
23:44:38.358447 run-command.c:336       trace: run_command: 'git-remote-https' 'flynn' 'https://git.qgg4.flynnhub.com/catch.git'
23:44:38.916757 run-command.c:336       trace: run_command: 'git credential-osxkeychain get'
23:44:38.921633 run-command.c:195       trace: exec: '/bin/sh' '-c' 'git credential-osxkeychain get' 'git credential-osxkeychain get'
23:44:38.944360 git.c:563               trace: exec: 'git-credential-osxkeychain' 'get'
23:44:38.945144 run-command.c:336       trace: run_command: 'git-credential-osxkeychain' 'get'

@alfredqy Did you run flynn cluster add using the details or copy the .flynnrc file over?

I copied the rc file over. What are the proper steps to set up multiple
people to have access to a cluster?
On Fri, Feb 3, 2017 at 6:22 PM Jonathan Rudenberg<EMAIL_ADDRESS>wrote:

@alfredqy https://github.com/alfredqy Did you run flynn cluster add
using the details or copy the .flynnrc file over?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flynn/flynn/issues/1820#issuecomment-277390189, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFOHR-d1GjM-OaBP_E_bSTPljJ-XYfSNks5rY7bEgaJpZM4F3z4E
.

You should run flynn cluster add with the details from the file. It will handle setting up git credentials and the CA certificate.


enum/ record/ string types don't work anymore in standard dialect (doc question)

the following code does not work anymore:
fn main() {
    let content = include_str!("./main.star");
    let ast = AstModule::parse(
        "main.star",
        content.to_owned(),
        &Dialect {
            enable_types: DialectTypes::Enable,
            enable_def: true,
            ..Dialect::Standard
        },
    )
    .unwrap();
    let globals = GlobalsBuilder::new().build();
    let module = Module::new();
    let mut eval = Evaluator::new(&module);
    eval.enable_static_typechecking(true);
    eval.eval_module(ast, &globals).unwrap();
}

rt = enum("one", "two")
 --> main.star:1:6
  |
1 | rt = enum("one", "two")
  |      ^^^^
  |

and also
def hello(input : str) -> str:
     return input + "world"

 --> main.star:1:19
  |
1 | def hello(input : str) -> str:
  |                   ^^^
  |

and
 --> main.star:1:1
  |
1 | print(hello("world"))
  | ^^^^^
  |

I saw a commit which changed str.type to str but could not find a way to enable it, probably I'm missing some basic config option?

GlobalsBuilder::new().build()

creates empty globals. I'm not sure this ever worked.

P.S. Next time when reporting bugs, please include full error. I think message (which is omitted) states that symbol is not resolved.

thanks for the quick response, GlobalsBuilder::extended_by worked

I think message (which is omitted) states that symbol is not resolved.

somehow I only looked at the dialect type and thought that its settings would enable all the things because enable_types: DialectTypes::Enable seemed to me like it would include e.g. starlark::environment::LibraryExtension::RecordType but on second though thats of course not true cause it can be enabled with only custom types as well so these two settings are somewhat orthogonal.
on the other hand starlark::environment::LibraryExtension::Typing seems like it is closely related to Dialect.enable_types so maybe there is still an opportunity to unify both/ combine them somewhat

The idea is that LibraryExtension is changing your standard library, whereas enable_types is changing your parser. You can still write some types without the LibraryExtension (e.g. str), it's just things like typing.Any that doesn't make a whole lot of sense without the other. I've created an internal patch to make each reference the other, which should be in the repo later today.


Why I don't see venv paths?

This happens when i open a python file after any other?
This happens even when search options are enabled

Hmm I hope its not related to the last commit where we made the Telegram viewer refresh itself in a different way...
I will see if I can reproduce this. You just open two python files after eachother? Do you run VenvSelect after each one?

first I just open some nvim configuration file, then I open the python file through the telescope

Hmm I hope its not related to the last commit where we made the Telegram viewer refresh itself in a different way...
I will see if I can reproduce this. You just open two python files after eachother? Do you run VenvSelect after each one?

first I just open some nvim configuration file, then I open the python file through the telescope

Whats the output of the ":pwd" command if you run it in neovim?

Whats the output of the ":pwd" command if you run it in neovim when you dont get results?

hehe

Yeah I think this is whats happening - its still looking in your pwd directory from the first file, goes up 2 parent folders, and finds nothing. When you open the second file, the pwd directory hasnt changed.
But i think my intention was that it would look at the currently opened file, and go 2 parent folders up from there, and search downwards.
I will have a look and see if this is a bug. I havent noticed this but thats because i run another plugin to set the pwd to the project folder. But that shouldnt be required.
Will look at this tonight! In the mean time, you can open neovim from your project folder or just change directory inside neovim, but thats annoying of course. :)

Yeah I think this is whats happening - its still looking in your pwd directory from the first file, goes up 2 parent folders, and finds nothing. When you open the second file, the pwd directory hasnt changed.
But i think my intention was that it would look at the currently opened file, and go 2 parent folders up from there, and search downwards.
I will have a look and see if this is a bug. I havent noticed this but thats because i run another plugin to set the pwd to the project folder. But that shouldnt be required.
Will look at this tonight! In the mean time, you can open neovim from your project folder or just change directory inside neovim, but thats annoying of course. :)

Okay, I will be waiting! :)
But, why you don't use this?

You can see the exact commands its running if you are curious - just set enable_debug_output to true in the VenvSelect setup function, and you get messages (type :messages in nvim) showing the fd search parameters etc.

I looked at this a bit and I cant reproduce the bug here. I open neovim, and I navigate to some file and run VenvSelect, and it correctly searches for venvs in the parent directory 2 steps up (and all folders below it).
If I then change directory to another location with "cd" inside neovim, open a file there, and runs VenvSelect there, it again correctly does the same - searches for venvs 2 steps up and all folders below it.
I cant reproduce what you get - you open a second file and there is no VenvSelect showing up? Can you send me the debug printout when you do these things? They should give some clues.

If you find the time to send me some debug output, I would love it, specially since i cant seem to reproduce this.. :/

Im closing this but if you have the time later to send some debug output and some description of what you do, feel free to reopen. :)

I have also found that with the latest commits I cannot load a single environment. Result from enabling debug output when simply running :VenvSelect in a folder that has a single folder with an environment named venv.
I did a bisect, that this is the first bad commit: https://github.com/linux-cultist/venv-selector.nvim/commit/af5a452ee89dc979ef6ee5704451a72580ac3847
Logs:
VenvSelect:
{
  anaconda_base_path = vim.NIL,
  anaconda_envs_path = "/home/micah/.conda",
  auto_refresh = false,
  cache_dir = "/home/micah/.cache/venv-selector/",
  cache_file = "/home/micah/.cache/venv-selector/venvs.json",
  changed_venv_hooks = { <function 1>, <function 2>, <function 3> },
  dap_enabled = false,
  enable_debug_output = true,
  hatch_path = "~/.local/share/hatch/env/virtual",
  name = { "env", ".env", "venv", ".venv" },
  notify_user_on_activate = false,
  parents = 0,
  pipenv_path = "~/.local/share/virtualenvs",
  poetry_path = "~/.cache/pypoetry/virtualenvs",
  pyenv_path = "~/.pyenv/versions",
  search = true,
  search_venv_managers = true,
  search_workspace = true,
  venvwrapper_path = "/home/micah/.virtualenvs"
}
VenvSelect: Setting fd_binary_name to 'fd' since it was found on system.
VenvSelect: Telescope path: /home/micah/welp
VenvSelect: Looking for parent venvs in '/home/micah/welp' using the following parameters:
VenvSelect:
{ "--absolute-path", "--color", "never", "-E", vim.NIL, "-E", "/home/micah/.conda", "-E", "/proc", "-HItd", "(^env$|^.env$|^venv$|^.venv$)", "/home/micah/welp" }
E5108: Error executing lua: .../nvim/lazy/venv-selector.nvim/lua/venv-selector/venv.lua:188: attempt to index local 'selected_venv' (a nil value)
stack traceback:
	.../nvim/lazy/venv-selector.nvim/lua/venv-selector/venv.lua:188: in function 'activate_venv'
	...azy/venv-selector.nvim/lua/venv-selector/mytelescope.lua:157: in function 'key_func'
	...hare/nvim/lazy/telescope.nvim/lua/telescope/mappings.lua:257: in function <...hare/nvim/lazy/telescope.nvim/lua/telescope/mappings.lua:256>
E5108: Error executing lua: .../nvim/lazy/venv-selector.nvim/lua/venv-selector/venv.lua:188: attempt to index local 'selected_venv' (a nil value)
stack traceback:
	.../nvim/lazy/venv-selector.nvim/lua/venv-selector/venv.lua:188: in function 'activate_venv'
	...azy/venv-selector.nvim/lua/venv-selector/mytelescope.lua:157: in function 'key_func'
	...hare/nvim/lazy/telescope.nvim/lua/telescope/mappings.lua:257: in function <...hare/nvim/lazy/telescope.nvim/lua/telescope/mappings.lua:256>
VenvSelect: No workspace folders found
VenvSelect: Found no workspaces to search for venvs.
VenvSelect: Found no venv manager directories to search for venvs.
VenvSelect: There are 0 results to show:

That commit changed how Conda works a little bit and VenvSelect now requires two environment variables for Conda environments - one for the base environment and one for the others venvs.
I can see that you get Anaconda base path = vim.nil in the debug output there also.
But it shouldn't error out like this no matter what config you have. Let me try and replicate this tonight. I will just create a folder, add a venv inside and then open vim when standing inside that folder. That's what you are doing right?

Yeah that's what I'm doing! Also I do not use conda at all so that makes sense that environment variables wouldn't exist on my machine

Ok thats good.
I will see if i can figure out what the errors are from. They sound like they would occur if you dont pick something in the telescope picker (because there are no results) and you press enter? Or do they always show up even if you dont press enter?

I made a small change which i think may fix that error message you get, and instead just print: "No virtual env selected in telescope."

What happens if there actually are virtual environments and it's not displaying them?

Ah actually, the changes you made fixed the issue it appears and my virtual environments are correctly displaying now even without conda installed!

Here are updated logs. It looks like currently if you do not have anaconda set up and base paths then it just doesn't work with anything else.
VenvSelect:
{
  anaconda_base_path = "",
  anaconda_envs_path = "/home/micah/.conda/envs",
  auto_refresh = false,
  cache_dir = "/home/micah/.cache/venv-selector/",
  cache_file = "/home/micah/.cache/venv-selector/venvs.json",
  changed_venv_hooks = { <function 1>, <function 2>, <function 3> },
  dap_enabled = false,
  enable_debug_output = true,
  hatch_path = "~/.local/share/hatch/env/virtual",
  name = { "env", ".env", "venv", ".venv" },
  notify_user_on_activate = false,
  parents = 0,
  pipenv_path = "~/.local/share/virtualenvs",
  poetry_path = "~/.cache/pypoetry/virtualenvs",
  pyenv_path = "~/.pyenv/versions",
  search = true,
  search_venv_managers = true,
  search_workspace = true,
  venvwrapper_path = "~/.virtualenvs"
}
VenvSelect: Setting fd_binary_name to 'fd' since it was found on system.
VenvSelect: Telescope path: /home/micah/welp
VenvSelect: Looking for parent venvs in '/home/micah/welp' using the following parameters:
VenvSelect:
{ "--absolute-path", "--color", "never", "-E", "", "-E", "/home/micah/.conda/envs", "-E", "/proc", "-HItd", "(^env$|^.env$|^venv$|^.venv$)", "/home/micah/welp" }
VenvSelect: No virtual env selected in telescope.
VenvSelect: No workspace folders found
VenvSelect: Found no workspaces to search for venvs.
VenvSelect: Found no venv manager directories to search for venvs.
VenvSelect: There are 0 results to show:
Error executing vim.schedule lua callback: ...azy/venv-selector.nvim/lua/venv-selector/mytelescope.lua:79: attempt to index local 'picker' (a nil value)
stack traceback:
	...azy/venv-selector.nvim/lua/venv-selector/mytelescope.lua:79: in function 'show_results'
	.../nvim/lazy/venv-selector.nvim/lua/venv-selector/venv.lua:59: in function 'find_other_venvs'
	.../nvim/lazy/venv-selector.nvim/lua/venv-selector/venv.lua:154: in function ''
	vim/_editor.lua: in function <vim/_editor.lua:0>

For reference, here is ls output:
13:36:57 ~/welp 1m 27s
➤ ls
venv

It has to do with -E being added in there with an empty string when there is no conda base path provided
13:38:30 ~/welp
➤ fd --absolute-path --color never -E /home/micah/.conda/envs -E /proc -HItd "(^env$|^.env$|^venv$|^.venv$)" .
/home/micah/welp/venv/

13:38:35 ~/welp
➤ fd --absolute-path --color never -E "" -E /home/micah/.conda/envs -E /proc -HItd "(^env$|^.env$|^venv$|^.venv$)" .

Ok great, will fix tomorrow :)

Made a quick attempt at fixing this tonight. See if the latest commit fixes the issue for you.
Will continue tomorrow otherwise with this. Dont like errors. :)

No error messages but also not fixed sadly. It seems the first time I run it it doesn't work and then magically works the second time:
First run:
VenvSelect:
{
  anaconda_base_path = "",
  anaconda_envs_path = "/home/micah/.conda/envs",
  auto_refresh = false,
  cache_dir = "/home/micah/.cache/venv-selector/",
  cache_file = "/home/micah/.cache/venv-selector/venvs.json",
  changed_venv_hooks = { <function 1>, <function 2>, <function 3> },
  dap_enabled = false,
  enable_debug_output = true,
  hatch_path = "~/.local/share/hatch/env/virtual",
  name = { "env", ".env", "venv", ".venv" },
  notify_user_on_activate = false,
  parents = 0,
  pipenv_path = "~/.local/share/virtualenvs",
  poetry_path = "~/.cache/pypoetry/virtualenvs",
  pyenv_path = "~/.pyenv/versions",
  search = true,
  search_venv_managers = true,
  search_workspace = true,
  venvwrapper_path = "~/.virtualenvs"
}
VenvSelect: Setting fd_binary_name to 'fd' since it was found on system.
VenvSelect: Telescope path: /home/micah/welp
VenvSelect: Looking for parent venvs in '/home/micah/welp' using the following parameters:
VenvSelect:
{ "--absolute-path", "--color", "never", "-E", "/proc", "-HItd", "(^env$|^.env$|^venv$|^.venv$)", "/home/micah/welp" }
VenvSelect: No virtual env selected in telescope.

Second run:
VenvSelect:
{
  anaconda_base_path = "",
  anaconda_envs_path = "/home/micah/.conda/envs",
  auto_refresh = false,
  cache_dir = "/home/micah/.cache/venv-selector/",
  cache_file = "/home/micah/.cache/venv-selector/venvs.json",
  changed_venv_hooks = { <function 1>, <function 2>, <function 3> },
  dap_enabled = false,
  enable_debug_output = true,
  hatch_path = "~/.local/share/hatch/env/virtual",
  name = { "env", ".env", "venv", ".venv" },
  notify_user_on_activate = false,
  parents = 0,
  pipenv_path = "~/.local/share/virtualenvs",
  poetry_path = "~/.cache/pypoetry/virtualenvs",
  pyenv_path = "~/.pyenv/versions",
  search = true,
  search_venv_managers = true,
  search_workspace = true,
  venvwrapper_path = "~/.virtualenvs"
}
VenvSelect: Setting fd_binary_name to 'fd' since it was found on system.
VenvSelect: Telescope path: /home/micah/welp
VenvSelect: Looking for parent venvs in '/home/micah/welp' using the following parameters:
VenvSelect:
{ "--absolute-path", "--color", "never", "-E", "/proc", "-HItd", "(^env$|^.env$|^venv$|^.venv$)", "/home/micah/welp" }
VenvSelect: No virtual env selected in telescope.
VenvSelect: Found venv in parent search: /home/micah/welp/venv/
VenvSelect: No workspace folders found
VenvSelect: Found no workspaces to search for venvs.
VenvSelect: Found no venv manager directories to search for venvs.
VenvSelect: There are 1 results to show:

Ok, then I will take a proper look tomorrow at this. Thank you for the detailed debug info. :)

Ive been trying to understand why this is happening.
Do you use LazyVim by any chance? I do right now, and it uses a vim.notify plugin to show nice notifications. For me, this causes VenvSelector to close the telegram picker the first time its run (when VenvSelect is run with debug_output_enabled = true), but not on future runs.
But I still get results. My log shows the same results every time I run VenvSelect, including the first time.
I cant reproduce this still. Can you try and see if you get this bug also when debug_output_enabled = false ?

Ah yes it does acutally work as intended without the debugging output! My venv is showing up correctly even without conda things setup or installed. Thanks!


searching max_iter in LinearSVC?

I'm a bit surprised that you search over max_iter in LinearSVC https://github.com/rhiever/sklearn-benchmarks/blob/master/model_code/LinearSVC.py#L12 What is the purpose of that? Checking if few iterations are enough? And where did you get these values from? I'd be surprised if this ever stopped early.
Also, I see you are trying out hinge and squared hinge, but you are not searching over L1 vs L2 penalty.

A lot of these parameters came from that DefaultGridSearch issue that we were discussing months ago. They had chosen a bunch of parameters and ranges there, and everyone seemed happy with them, so I figured that was a good starting point. :-)
If you have suggestions for specific parameters/parameter ranges for each model, I can queue those up for the benchmark pretty easily.


Usage of Pre-Trained Models

Hello,
Apologies for my ignorance, but how can I use the pre-trained models to caption a video? I see instructions for extracting features, training, and evaluating the model, but I can't figure out how to generate captions.
Thank you!

Thanks for your comments. We have added a demo notebook (demo.ipynb) to extract three-levels of hierarchical captions from any video using our pretrained models.

Cool, thanks so much! Must have missed that, sorry about that!


Cannot select shipping method in Apple Pay payment sheet

Describe the bug
When using Apple Pay from the shopping cart, I cannot select a shipping method, even when Magento provides multiple shipping methods. The first shipping method provided by Magento is automatically chosen.
To Reproduce
Steps to reproduce the behavior:

As a guest user, add an item to the cart
Go to the shopping cart
Click the Apple Pay button
In the payment sheet, select a shipping address that will return multiple shipping methods from Magento

Expected behavior

I can select which shipping method I want to use.
If I've already selected a shipping method in Magento, the one I selected should be selected in the Apple Pay payment sheet, not simply the first one.

Actual behavior

No shipping methods are displayed, and first shipping method is chosen.
I cannot change shipping method from the payment sheet.
If I customer selects a shipping method in Magento, then initiate an Apple Pay session from the cart; the selected shipping method is changed to the first one returned by Magento.

Screenshots

Desktop (please complete the following information):

OS: MacOs
Browser Safari

Smartphone (please complete the following information):

Device: iPhone
OS: iOS
Browser: Safari

Adyen Versions

adyen/adyen-magento2-expresscheckout: 2.1.0
adyen/module-payment: 9.7.2

Additional context
The ApplePaySession's completeShippingContactSelection should be called with all available shipping methods, which will cause them to show up as options on the payment sheet, according to the Apple Documentation

When I step through button.js and get to the point where the shipping methods are retrieved from Magento, I can see multiple shipping methods returned, but they seem to be being discarded.
I can see pretty clearly here where Adyen's Express Payment javascript is selecting the first available method rather than checking to see what shipping method has been selected in Magento:

Additional Information:
I've confirmed that when I downgrade to 2.0.0, I can see shipping methods, so this appears to be a regression in the latest version.

Hello @pignion,
Thank you for reporting this issue in details. As I see, @maxparkergene has already created a pull request for this bug. We will review and merge the fix soon.
Best Regards,
Can


Update createBasicXMLRequest

If provided serialized XML, use it as is. So if I create XML in code I can use it without encoding in grequests.

Thanks for the PR. Can you please write a test to validate this functionality?
Levi

Yes, I will create new PR

LGTM

Thanks!


Support for ingress resource in meshsync

Signed-off-by<EMAIL_ADDRESS>Description
This PR fixes #217
Notes for Reviewers
1:Updated the default_config.go to support ingress resource
Signed commits

[x] Yes, I signed my commits.

Excellent, @Vadiraj-Puranik 👍

@Vadiraj-Puranik please run gofmt -w -s -l . on future commits.

@Vadiraj-Puranik please run gofmt -w -s -l . on future commits.

@leecalcote  sure will keep this in mind


Amount is not getting reset to '0' when user selects other token from the dropdown on enter amount page.

Frequency: 100%
Repro on build version: Android Internal build V:1.25.0 ,iOS TestFlight build V:1.25.0
Repro on device : OnePlus 7t(11.0), Google Pixel 2XL(11.0), iPhone 12(14.7.1), iPhone 13mini(15.1.1)
Pre-condition:
1] User must have installed the application
2] User must have completed on boarding flow
3] User must have multiple token balance in the account
Repro Steps:
1] Tap on send button from homepage and select a contact / enter address / scan QR code of another user
2] It redirects to enter amount page
3] Enter a valid amount in the amount field
4] Tap on dropdown from RHS corner of the page and select other token
5] Observe the amount field
Bug : The amount is not getting reset to 0 on selcting other token from dropdown.
Expected Behavior: If user has entered '1' in amount field and selects other token then it should change to '0'
Observation :

For Single token there is back button to change amount to 0 on long press and also there is no dropdown to select other token

Attachments : reset.mp4

@ValoraQA this will be fixed in 1.26.x by https://github.com/valora-inc/wallet/pull/1925.

Hey @MuckT We have verified the above issue on latest Android Internal release build V:1.27.0 and latest iOS testflight release build V:1.27.0 and observed that amount is getting reset to '0' when user selects other token from the dropdown on enter amount page. Hence we are closing this issue.
Attachment: Reset.mp4
Devices:  iPhone 13 mini(15.1.1), Redmi Note 7 Pro(10.0), iPhone 6+(12.5.5)
Thanks..!!


Require a 'properties' field in package config.json.

Makes sure the bug fixed in #48 can't happen again.

cc @swartzrock


update innoxel to 0.4.2

Updates the innoxel adapter from 0.4.1 to 0.4.2 in stable.
The update contains adapter checker feedback and dependency updates.

RE-CHECK!

E124 should not be raied if nogit is set to true.
The issue is still open: https://github.com/ioBroker/ioBroker.repochecker/issues/270


Add Michka to Linuxbrew maintainers

[x] Add to Slack
[x] Add to the Slack maintainers channel
[x] Introduce to existing members
[x] Announce at Linuxbrew Updates
[x] Add to the GitHub team
[x] Add to the Bintray organization
[x] Add to Google Analytics if desired
[ ] Add name to Who Are You? in README.md

https://github.com/Linuxbrew/brew/issues/1#issuecomment-309870740

Should we also add "Add to maintainer list on brew README" to the checklist?

Yes, definitely. Good catch.
@iMichka Would you please open a PR to add yourself to https://github.com/Linuxbrew/brew/blob/master/README.md#who-are-you ?

A heads up that the CI for Linuxbrew/brew has been failing one test on macOS recently, so you can expect that failure. :100: points if you submit a PR to fix it. 😉

I'll add myself on Thursday; not there today :)

PR opened here: https://github.com/Linuxbrew/brew/pull/413


fix: no fallback when no gaia profile found

Try out this version of the Hiro Wallet - download extension builds.

TIL you need to return await if you want to hit the catch  block

Merging as urgent


Uncaught Exception: Attempting to call a function in a renderer window that has been closed or released. Function provided

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.0.1
Time: Wed Mar 23 2016 23:13:27 GMT-0600 (Mountain Daylight Time)
Error:
Attempting to call a function in a renderer window that has been closed or released. Function provid
Error: Attempting to call a function in a renderer window that has been closed or released. Function provided here: Object.<anonymous> (C:\Users\{User}\AppData\Local\GPMDP_3\app-3.0.1\resources\app\build\inject\GPMInject\interface\mouseButtonNavigation.js:8:8.
    at BrowserWindow.callIntoRenderer (C:\Users\{User}\AppData\Local\GPMDP_3\app-3.0.1\resources\atom.asar\browser\lib\rpc-server.js:201:19)
    at emitTwo (events.js:92:20)
    at BrowserWindow.emit (events.js:172:7)

This issue was created automatically inside the uncaughtException handler
This popped up a couple of times, once when I was maximizing the window, and a second time when I was fiddling with the volume buttons on my keyboard. If it helps at all, I was watching the particles visualizer the second time it happened.
After some testing, it happens every time the little volume box pops up, though not when GPMDP isn't selected.
Running Windows 10 Pro x64.
Edit: Grammar

Duplicate, not sure why the unicorn didn't auto close this
BAD UNICORN :laughing:


Resolved #151: Add command line options to specify the max width of summary view.

Add new command line option for tui-viewer to specify the max width of summary view.

max-width: default to 80.

Or maybe using the following module to determine the width of the current terminal. https://pypi.python.org/pypi/backports.shutil_get_terminal_size

Coverage decreased (-0.02%) to 71.883% when pulling 2c365172895d51d028b512e0deb4c37a3d1691df on ui-width-#151 into e692dcc2c3b5e11bfefbfa6ee5755dcb88ce6f30 on master.

How is mitmproxy work on this?

Using get_terminal_size to get the width when tui is open.

Coverage decreased (-0.1%) to 71.791% when pulling 101c1954aeb4cb0423e98c9394996794b9ef1920 on ui-width-#151 into e692dcc2c3b5e11bfefbfa6ee5755dcb88ce6f30 on master.

LGTM :)


Operator Redesign

Part of #279.
Closes #281.

/hold

Looks good to me once comments are addressed

/unhold

/lgtm

this /lgtm as well

/lgtm


global: remove socket limit - fixes #522

#522
request has a defaults option where we can mimic the Node 0.12.0 behavior of not limiting open sockets to 5 at once.
@ryanseys - you're kindking of benching - do you have any tips how I might test that this is working as intended?

I can't think of any way to test that's easy, other than writing a unit test to make sure the proper defaults are set on the request object. I feel we should just trust the module to "do the right thing". The tests in this PR aren't all passing either so if you could take a look at that too.

Aye.

Fixed and added tests. No more 666! :tada:
Also, please see my typo correction from the initial post. That was probably weird.

Just datastore and storage get these defaults? Or am I reading something wrong here? Should it be added to every API?

Most importantly, it's added to common/util, which is where non-fancy API requests all go through (the makeAuthorizedRequest flow).
Everywhere you see require('request') in our library code, we're using the defaults.

Perfect! Definitely glossed over that. :gem:


firetv => repo?

Is it so far?

Yes, you can add the adapter to the repository.

Cam you Please to weite some more words about this skill. Even I do not understand, what can I do with that :)
What is adb?  And where can I get it ?

Hi, I have extended the readme. What do you think?

Documentation is much better. Thanks :)


Debugging tf.Keras Models with TFDBG get TypeError: Fetch argument None has invalid type <class 'NoneType'>

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution ( Linux Ubuntu 18.04):
TensorFlow installed from (source or binary):  binary
TensorFlow version (use command below): tensorflow-gpu  1.14.0
Python version: python 3.7.1
CUDA/cuDNN version: CUDA 10.1  cuDNN 7.5
GPU model and memory: GTX 2060

Describe the current behavior
I want to use debugging tf.keras model, but got an error message like:
Traceback (most recent call last):
  File "./debug_keras.py", line 82, in <module>
    vae.add_loss(loss_fn(x_in, x_out, μ, σ))
  File "/home/zqh/miniconda3/lib/python3.7/site-packages/tensorflow/python/keras/engine/base_layer.py", line 917, in add_loss
    new_layers = base_layer_utils.create_keras_history(symbolic_loss)
  File "/home/zqh/miniconda3/lib/python3.7/site-packages/tensorflow/python/keras/engine/base_layer_utils.py", line 200, in create_keras_history
    _, created_layers = _create_keras_history_helper(tensors, set(), [])
  File "/home/zqh/miniconda3/lib/python3.7/site-packages/tensorflow/python/keras/engine/base_layer_utils.py", line 244, in _create_keras_history_helper
    constants[i] = backend.function([], op_input)([])
  File "/home/zqh/miniconda3/lib/python3.7/site-packages/tensorflow/python/keras/backend.py", line 3292, in __call__
    run_metadata=self.run_metadata)
  File "/home/zqh/miniconda3/lib/python3.7/site-packages/tensorflow/python/debug/wrappers/framework.py", line 628, in wrapped_runner
    callable_runner_args=feed_values)
  File "/home/zqh/miniconda3/lib/python3.7/site-packages/tensorflow/python/debug/wrappers/framework.py", line 569, in run
    run_metadata=run_metadata)
  File "/home/zqh/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py", line 950, in run
    run_metadata_ptr)
  File "/home/zqh/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py", line 1158, in _run
    self._graph, fetches, feed_dict_tensor, feed_handles=feed_handles)
  File "/home/zqh/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py", line 474, in __init__
    self._fetch_mapper = _FetchMapper.for_fetch(fetches)
  File "/home/zqh/miniconda3/lib/python3.7/site-packages/tensorflow/python/client/session.py", line 261, in for_fetch
    type(fetch)))
TypeError: Fetch argument None has invalid type <class 'NoneType'>

Describe the expected behavior
It should be work...
Code to reproduce the issue
run this python script, then type in: run -t 10 will get an error message...
import tensorflow.python as tf
from tensorflow.python import keras as k
from tensorflow.python.keras import layers as kl
from tensorflow.python.keras import activations as ka
import matplotlib.pyplot as plt
import numpy as np
from scipy.stats import norm
from scipy.special import expit
from tensorflow.python import debug as tfdebug

config = tf.ConfigProto()
config.gpu_options.allow_growth = True

k.backend.set_session(tfdebug.LocalCLIDebugWrapperSession(tf.Session(config=config)))
# k.backend.set_session(tf.Session(config=config))

(x_train, y_train), (x_test, y_test) = k.datasets.fashion_mnist.load_data()
x_train = np.expand_dims(x_train, -1) / 255.
x_test = np.expand_dims(x_test, -1) / 255.

image_size = 28
input_shape = (image_size, image_size, 1)
batch_size = 100
kernel_size = 3
filters = 16
latent_dim = 2  # 隐变量取2维只是为了方便后面画图
epochs = 30
tf.set_random_seed(9102)

def encoder_fn(inputs, filters):
    x = inputs
    for i in range(2):
        filters *= 2
        x = kl.Conv2D(filters=filters, kernel_size=kernel_size, activation='relu', strides=2, padding='same')(x)
    x = kl.Flatten()(x)
    x = kl.Dense(32, activation='relu')(x)
    μ = kl.Dense(latent_dim)(x)
    σ = kl.Dense(latent_dim)(x)
    return μ, σ

def sampling(args):
    """ 重参数技巧 """
    μ, σ = args
    ε = tf.random_normal(shape=tf.shape(μ))
    return μ + tf.exp(σ / 2) * ε

def decoder_fn(z, filters):
    x = kl.Dense(7 * 7 * 32, activation='relu')(z)
    x = kl.Reshape((7, 7, 32))(x)
    for i in range(2):
        x = kl.Conv2DTranspose(filters=filters, kernel_size=kernel_size, activation='relu', strides=2, padding='same')(x)
        filters //= 2
    x = kl.Conv2DTranspose(1, kernel_size, activation=None, padding='same')(x)
    return x

def loss_fn(inputs, outputs, μ, σ):
    # 这里求和的时候先按每个样本求和，再按样本求平均
    xent_loss = tf.reduce_sum(tf.nn.sigmoid_cross_entropy_with_logits(labels=x_in, logits=x_out), axis=[1, 2, 3])
    # xent_loss = tf.reduce_sum(k.backend.binary_crossentropy(x_in, x_out), axis=[1, 2, 3])
    kl_loss = - 0.5 * tf.reduce_sum(1 + σ - tf.square(μ) - tf.exp(σ), axis=-1)
    vae_loss = tf.reduce_mean(xent_loss + kl_loss)
    return vae_loss

x_in = k.Input(shape=(image_size, image_size, 1))
μ, σ = encoder_fn(x_in, filters)
z = kl.Lambda(sampling, output_shape=(latent_dim,))([μ, σ])

latent_inputs = k.Input(shape=(latent_dim,), dtype=tf.float32)
outputs = decoder_fn(latent_inputs, filters)
decoder = k.Model(latent_inputs, outputs)
x_out = decoder(z)

encoder = k.Model(x_in, μ)

vae = k.Model(x_in, x_out)
vae.add_loss(loss_fn(x_in, x_out, μ, σ))
vae.compile(k.optimizers.Nadam(0.001))
vae.fit(x=x_train, batch_size=batch_size, epochs=epochs, shuffle=True, validation_data=(x_test, None))

Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Was able to reproduce the issue with Tensorflow 1.14.0.

@zhen8838, were you ever able to find a solution to this? I am running into the same issue!

@zhen8838, were you ever able to find a solution to this? I am running into the same issue!

This is a bug. You should try tensorflow 1.15 or tensorflow 2.0.  I'm not sure which version of the bug was fixed yet.

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.


Measurable Categories: Allow admin to create a new category

Description
Need a feature to create taxonomies through the UI

[ ] Should have the ability to bulk load the taxonomy for newly created taxonomies

Resourcing
We would like to add this request to the Waltz team's feature backlog

Created a new integration test to cover this scenario.
Also discovered that one of the old custom assert methods (assertChangeLogContainsAtLeastOneMatchingOperation) was not working correctly - this caused a few (minor) knock on changes / fixes.


JavaScript object expected

Using the latest version of bugsnag-js we get a stream of these errors only from IE 11.

@jbrooksuk thanks for the report. I wasn't able to reproduce this with the latest notifier, so to help us diagnose, do you mind providing:

an example of a stacktrace you are seeing with this error (feel free to email it to us at<EMAIL_ADDRESS>if you'd rather not paste that here)
the version you upgraded from

Closing due to lack of activity, but feel free to reopen if you have any more information. Cheers!

@eanakashima We are seeing this as well with the latest notifier version 3.3.2. I have reported this to<EMAIL_ADDRESS>with a link to some of the stack traces we are seeing.

I see this was also reported as #105 previously. Mentioning that here to create a cross-link.


Rewrites MNIST example in Linen API

The example is added to the directory linen_examples. Here's a side-by-side diff: https://github.com/google/flax/compare/master..marcvanzee:mnist

Cool! I'll take a look.
(looks like there's a failing test?)

Codecov Report

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

@@           Coverage Diff           @@
##           master     #434   +/-   ##
=======================================
  Coverage   73.64%   73.64%
=======================================
  Files          54       54
  Lines        3977     3977
=======================================
  Hits         2929     2929
  Misses       1048     1048

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 294575a...67eb8e1. Read the comment docs.

Tests are passing now.

LGTM!


stop crossScalaVersions cross publishing to 2.12, keep only 2.13

With pekko / play upgrade, the play-guice has support for Scala 3 and 2.13, thus kineticpulse publish for 2.13 only.

@realstraw new commit pushed to handle cross build for Scala 2.13 and 3.


De-duplicate "Owner" and "Maintainer" roles

Currently on pypi-legacy, it's possible for a user to have both the "Owner" and "Maintainer" roles.
However, an "Owner" can do everything a "Maintainer" can do, so it doesn't make sense for a user to be able to have both roles.
In the short term, we decided to add a hack to hide any "duplicate" roles in #2705, and remove the ability to add multiple roles for the same user in Warehouse.
Once pypi-legacy is shut down, we can run a migration which removes any of these "duplicate" roles., and remove this hack.

I think I'm unclear -- what can an Owner do that a Maintainer can't do? I am probably doing to open a documentation issue to put that info somewhere users can get at. Thanks.

@brainwane The following information box is included on the 'Collaboration' page where a user has the ability to change these roles:

I think that should probably suffice in terms of documentation, but it probably wouldn't hurt to duplicate it elsewhere.

Actually, this should probably get revised, because Owners can do other things too now (like delete files, releases, and the entire project).


yf.Ticker(ticker).info Error

yf.Ticker(ticker).info TypeError
- Info about your system:

yfinance version
0.2.3

operating system
Windows 10

- Simple code that reproduces your problem
import yfinance as yf
ticker = 'AAPL'
stock_info = yf.Ticker(ticker).info
- The error message
TypeError: string indices must be integers, not 'str'

Duplicate of #1291

I didn't see the duplicate issue.  Thanks for all your help @ValueRaider.


Fixing issue #7222

This pr fixes issue #7222

@biplobice that wouldnt work. We need a new instance of User object - of course the user object is registered if we get it by userid. It doesnt tell us if the person running the script is logged in
Actually isLoggedIn should not be depreciated by isRegistered() as they are two very different things..
IsRegistered() tells me if the current user object is registered, isLoggedIn() tells me if a new user object is registered

+1

Please review this PR before the bot close it


setup.py should not import the project, as this breaks installs via pip.

Given a requirements file, requirements.txt:
scrapy
scrapy-random-useragent

and the command:
pip install -r requirements.txt

results in the error:
ImportError: No module named scrapy

The installation of the requirements fails, as scrapy-random-useragent is relying on scrapy at a point where it has not been installed yet. This is because the setup.py gets read by pip, which in turn imports the project (import random_useragent) which in turn tries to import modules from scrapy, which is not yet installed, hence the ImportError.
This needs to be solved by removing the import in setup.py and any reliance on the data it imports.
A lot of projects just copy and paste the data, and bump both version numbers on doing a new release. However, [reading this thread](ImportError: No module named scrapy) seems like another way to handle the situation and keep things DRY.
If you wish, I can submit a PR which follows the path you wish to take the fix this - I'd just like your OK before I invest the time to do so.
Thanks,
Darian

For users looking for a fix in the meantime, the stop gap solution is to:
pip install scrapy==<your.version> && pip install -r requirements.txt

@cnu ping?

@djm It would be great if you could do a PR for this.

Amazing, I will get on that tomorrow. Thanks.

Closing in favour of PR #3


Add cleaning of bracketed text

Adds cleaning of bracketed text such as [Music] to the substudy clean command. This text is found on YouTube video subtitles such as the start of this video. Let me know if you want me to remove the test case I added to substudy/src/clean.rs or if there is a better place to put it. Thanks for making this project!

Thank you for the PR! This is an interesting idea, and I need to think it over a bit.


[Bug] too many "send fragment timeout. backend id" error

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
trunk
What's Wrong?
In some high concurrency scenarios, a large number of "send fragment timeout. backend id xxx" errors may appear in the fe.log. And all subsequent requests keep reporting this error and cannot be recovered.
What You Expected?
This is because in high load scenarios, the execution thread pool on the BE side is full, and new requests will enter the waiting queue of the thread pool.
However, the timeout period of the plan fragment request sent by FE to BE is only 5 seconds. The waiting time for requests entering the waiting queue may be very long, resulting in a large number of rpc errors send fragment timeout.
And subsequent requests will continue to enter the waiting queue, causing all subsequent requests to time out.
How to Reproduce?
Run a complex queries with high concurrency.
Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

version 0.15。
modified remote_fragment_exec_timeout_ms=600000, this problem still exists


Detect Java network issues when running flutter doctor --android-licenses

We've had reports of users needing to set the Java network proxy before being able to accept the Android licenses.  We should be able to write a simple Java executable that probes for basic network connectivity:
import java.net.InetAddress;
import java.net.InetSocketAddress;
import java.nio.channels.SocketChannel;

public class NetworkProber {
  private static final int HTTP_PORT = 80;

  public static void main(String[] args) {
    try {
      InetAddress google = InetAddress.getByName("www.google.com");
      InetSocketAddress address = new InetSocketAddress(google, HTTP_PORT);
      SocketChannel channel = SocketChannel.open(address);
      System.out.println("Network OK");
      closeChannelSilently(channel);
      System.exit(0);
    } catch (Throwable t) {
      System.err.println(t.getMessage());
      System.err.println("Network unreachable");
      System.exit(1);
    }
  }

  private static void closeChannelSilently(SocketChannel channel) {
    try {
      channel.close();
    } catch (Exception e) {
    }
  }
}

We could prebuild that tool into a JAR file in the engine repo and upload that with our other prebuilt artifacts, then use it in flutter_tools to ensure that we'll be able to reach the network when accepting Android licenses here:
https://github.com/flutter/flutter/blob/72926bdff72e7915136c653c8448107ba9252e36/packages/flutter_tools/lib/src/android/android_workflow.dart#L167
To do this, we can directly invoke Java, as discovered from the Android SDK here:
https://github.com/flutter/flutter/blob/72926bdff72e7915136c653c8448107ba9252e36/packages/flutter_tools/lib/src/android/android_sdk.dart#L447

I'm guessing that this is still relevant.


Feature host

The 'Host' button GUI is implemented and there are only a few logical things to fix inside there. The scope of feature-host is done, and other configurations to the file HostScreen.kv should be done in another branch.

Looks good, I,ll merge it at once.

It looks good, I'll merge it at once.


Include comparison to Preact

Can you add a comparison to Preact?
That's what my team is currently using and it's pretty fast and lightweight so we would need to see VanJS compared to Preact before switching
https://preactjs.com/

According to bundlephobia. Preact is 11.2kB minified (vs. VanJS 1.7kB) and 4.5kB minified + gzipped (vs. VanJS 0.9kB).


"debug.autoExpandLazyVariables": false replaced with the default

Testing #222238
There does not seem to be a need to keep the setting in the JSON, it can be removed because "auto" (with which the value is replaced) is the new default.

Also- should false be translated to off? That was my interpretation.

Not sure I understand. Can you state steps to repro, your expectation vs what actually happend?

I believe this is what Christof meant:

set "debug.autoExpandLazyVariables" to false.
reload the window
notice that it now says "debug.autoExpandLazyVariables":auto
Since this is the default value, we expected the line to disappear completely, since the absence of  "debug.autoExpandLazyVariables" already implies  "debug.autoExpandLazyVariables":auto

Thanks, are you in screen reader mode when that is the case?

yup! but I'm not sure if it happens regardless. Isn't it just the process of auto-migrating the setting to the new one?

In screen reader mode, if you have it set to false, and reload, you should see it set to off.
In screen reader mode, if you have it set to true and reload, you should not see it added because it's auto, the default.
In non-screen reader mode, if you have it set to false and reload, you should not see it added, because it's auto, the default.
In non-screen reader mode, if you have it set to true and reload, you should see it set to on.

I see some bugs with this, thanks for the issue.
To verify:

With or without screen reader mode, if you have it set to false and reload, you should see it set to off.
With or without screen reader mode, if you have it set to true and reload, you should see it set to on.

Thanks for the clear verification steps, @meganrogge!
Also made much easier with @sandy081's profiles editor 🎉


Add Visual Studio SmartTag support

Visual Studio SmartTag support for using directive for missing namespace would be great.

Looks like it was only on an extension method it didn't work.

What do you mean by smart tags?

I believe that MS calls this a SmartTag.

I've always called them / heard them called namespace imports, which are also required for extension methods.

Okay.
I can't get them working on the Critical extension to NLogs Exception method.

Are you seeing an error? Do you have the Exceptionless.NLog namespace imported?

It works fine if i manually type in the namespace. I would bust like it to work with the "SmartTag", but I don't think extension methods can do that.
Sent from my Windows Phone

From: Blake<EMAIL_ADDRESS>Sent: 4/9/2015 4:06 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: Re: [Exceptionless] Add Visual Studio SmartTag support (#91)
Are you seeing an error? Do you have the Exceptionless.NLog namespace imported?

Reply to this email directly or view it on GitHub:
https://github.com/exceptionless/Exceptionless/issues/91#issuecomment-91242591

Resharper will automatically ask you to import the namespace for extension methods.


styled the new contests page

What's this PR do? (related issue if exists)
it styles the new contests page
Any context you want to provide on the implementation?
Screenshots? (necessary if an UI/UX change)

@NikhilSharmaWe Thanks for working on this!
It would be better if you attach some screenshots, in the PR itself.

@NikhilSharmaWe Thanks for working on this!
It would be better if you attach some screenshots, in the PR itself.


Unable to delete advanced prices due to media bar covering actions button

Description

When trying to delete some advanced prices from a test product, I noticed that there doesn't appear to be any padding/margins to account for the absolutely positioned media bar.
This unfortunately prevents me from actually being able to delete them.
Additional context

Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know.
Thank you again for your valuable feedback.


do a performance test

For 700 mb file:
Nim takes ~20 seconds to upload and download file
Python takes ~109 seconds to upload and download file
For 16 mb file:
Nim takes ~0.3 seconds to upload and download file
Python takes ~2.2 seconds to upload and download file


STEALTH - Wallet Data Format

This issue is designed to document the data the wallet will need to maintain for stealth operations
 {
    "labeled_keys" : [[LABEL,KEY],...],
    "blind_receipts" : [ {
           fc::time_point                  date;
           public_key_type             from_key;
           string                              from_label;
           public_key_type             to_key;
           string                              to_label;
           asset                              amount;
           string                              memo;
           authority                         control_authority;
           stealth_confirmation::memo_data data;
           bool                                used = false;
           stealth_confirmation       conf;
    },...]
}

The format of stealth_confirmation is defined here:
https://github.com/cryptonomex/graphene/blob/master/libraries/chain/include/graphene/chain/protocol/confidential.hpp#L97-L121
The JavaScript API that needs to be implemented should mirror the C++ API found here:
https://github.com/cryptonomex/graphene/blob/master/libraries/wallet/include/graphene/wallet/wallet.hpp#L727-L799
This should be implemented by creating a ConfidentialWallet class in Javascript
   let wallet = new Wallet( ... ) /// create load current wallet
   let cwallet = new ConfidentialWallet( wallet ) /// uses current wallet to store that data defined above
   cwallet.FUNCTION_CALL(...) /// should match the C++ API linked above

To properly implement the cwallet API calls will require making some calls to advanced crypto methods
that are not currently implemented in Javascript.  These methods are defined in c++ here:
https://github.com/cryptonomex/fc/blob/master/src/crypto/elliptic_secp256k1.cpp#L405-L552
For starters we will assume these calls exist, and simply relay them to a C++ node to evaluate and return the result. Then we can replace the RPC calls with Javascript implementations. After we have proven the rest of the system works.

I think this should be referred to in this issue:
https://github.com/cryptonomex/graphene/issues/500


#2671

https://github.com/pygments/pygments/issues/2671

JavaScript: before vs after

TypeScript: before vs after

Semicolon is added to the end of line, which is different from the issue's sample code. Should I delete it when testing?

Semicolon is not important in JS, I believe. But yes, you should delete it since we need the test code to be the same as the code in the original issue

JavaScript: before vs after

TypeScript: before vs after

@ZuxingGit Hi, is the fix for the issue complete?

@ZuxingGit Hi, is the fix for the issue complete?
I will test it using Ishaan's method today. #3

So, I guess it passed all

In case you misunderstand how the command in Ishaan's method works. It doesn't check correctness of the output and, instead, it overwrite the old output with the one resulted from the current code (mind the --update-goldens option). Using Ishaan's method, you can check whether the sample code is highlighted correctly by looking at what is returned in the area in question.

If there's no problem with the output, I guess it's ok to create PR for this issue.


set :col-num=“6” is not working

Software version (please complete the following information):

Browser [Edge]
Vue Version [2.6.12]
vue-grid-layout Version: [e.g. 2.3.12]

Describe the bug
I already set :col-num=“6” , but the result displayed by the browser is still 12
Use the "examples/06-responsive.html"
To Reproduce
<grid-layout v-if="show"
                         :layout.sync="layout"
                         :col-num="6"
                         :row-height="30"
                         :is-draggable="draggable"
                         :is-resizable="resizable"
                         :vertical-compact="true"
                         :use-css-transforms="true"
                         :responsive="responsive"
           >

is caused by this parameter  :responsive="true"   ,when i set it "false",  :col-num=“6” is worked !

I want parameter :responsive="true", do you have the solution

I want parameter :responsive="true", do you have the solution

nope

I found a solution, you need change :cols = '{ lg: 6, md: 6, sm: 6, xs: 6, xxs: 6 }', or breakpoints


Add set -e to install.sh

(currently fails silently if maven acts up)

Created a new tag v3.1.1 that should now use the correct version of the KafSaxParser 3.1.1
I assume this is the same problems as with the EventCoreference module. I have to go through all the modules and fix the version tags.


cannot tmt-reboot from an independent session

While running a multi-host test I had a need to issue a reboot command through ssh session (an "operator" rebooting a worker). However, tmt-reboot didn't work properly in this case and no reboot was made as the tmt-reboot script relies on various environment variables in order to store the right data and these environment variables are not available in a custom ssh session.
This is what I see in terminal for the tmt process, variables like TMT_REBOOT_REQUEST are probably needed for proper functioning.
root        1333    1317  0 15:27 pts/1    00:00:00 bash -c export SERVERS='<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>'; export TMT_ROLE_OPERATOR=<IP_ADDRESS>; export TMT_ROLE_VERIFIER=<IP_ADDRESS>; export TMT_ROLE_REGISTRAR=<IP_ADDRESS>; export TMT_ROLE_AGENT=<IP_ADDRESS>; export TMT_GUEST_ROLE=AGENT; export TMT_GUEST_HOSTNAME=<IP_ADDRESS>; export TMT_PLAN_DATA=/var/tmp/tmt/run-057/plans/documentation/data; export TMT_TREE=/var/tmp/tmt/run-057/plans/documentation/tree; export TMT_TEST_NAME=/documentation/ensuring-system-integrity-with-keylime; export TMT_TEST_DATA=/var/tmp/tmt/run-057/plans/documentation/execute/data/guest/agent/documentation/ensuring-system-integrity-with-keylime-1/data; export TMT_TEST_METADATA=/var/tmp/tmt/run-057/plans/documentation/execute/data/guest/agent/documentation/ensuring-system-integrity-with-keylime-1/metadata.yaml; export TMT_REBOOT_REQUEST=/var/tmp/tmt/run-057/plans/documentation/execute/data/guest/agent/documentation/ensuring-system-integrity-with-keylime-1/data/reboot-request; export TMT_REBOOT_COUNT=0; export REBOOTCOUNT=0; export RSTRNT_REBOOTCOUNT=0; export BEAKERLIB_DIR=/var/tmp/tmt/run-057/plans/documentation/execute/data/guest/agent/documentation/ensuring-system-integrity-with-keylime-1; export BEAKERLIB_COMMAND_SUBMIT_LOG='bash /usr/local/bin/tmt-file-submit'; cd /var/tmp/tmt/run-057/plans/documentation/discover/ensuring-system-integrity-with-keylime/tests/documentation/ensuring-system-integrity-with-keylime; set -eo pipefail; ./tmt-test-wrapper.sh </dev/null |& ca

Just a note: The naive 'replace TMT_REBOOT_REQUEST' in tmt-reboot script with real path and make it call 'reboot' didn't help. What I did:
tmt-reboot command got changed into (just the end of it has changed)
[default-1 (agent)]                    out: printf "$json_fmt" "$command" "$timeout" > "/var/tmp/tmt/FOO/plan/execute/data/guest/default-1/test-1/data/reboot-request"
[default-1 (agent)]                    out: reboot
[default-1 (agent)]                    out: kill $PPID

However when it was executed:
default-0 (operator)]                     Connection to <IP_ADDRESS> closed by remote host.
[default-1 (agent)]            Command returned '255'.
        Append to file '/var/tmp/tmt/FOO/plan/execute/data/guest/default-1/test-1/output.txt'.
[default-1 (agent)]            Copy '/var/tmp/tmt/FOO/plan/execute/data/guest/default-1/test-1' from the guest to '/'.
[default-1 (agent)]            Run command: rsync -s -R -r -z --links --safe-links --protect-args --exclude '/var/tmp/tmt/FOO/plan/execute/data/guest/default-1/test-1/backup*' -e 'ssh -oForwardX11=no -oStrictHostKeyChecking=no -oUserKnownHostsFile=/dev/null -oServerAliveInterval=60 -oServerAliveCountMax=5 -oIdentitiesOnly=yes -p22 -i /var/tmp/tmt/FOO/plan/provision/default-1/id_ecdsa -S/run/user/17062/tmt/tmp2709csg5' root@<IP_ADDRESS>:/var/tmp/tmt/FOO/plan/execute/data/guest/default-1/test-1 /
[default-1 (agent)]            err: Warning: Permanently added '<IP_ADDRESS>' (ED25519) to the list of known hosts.
[default-1 (agent)]            err: Connection closed by <IP_ADDRESS> port 22
[default-1 (agent)]            err: rsync: connection unexpectedly closed (0 bytes received so far) [Receiver]
[default-1 (agent)]            err: rsync error: unexplained error (code 255) at io.c(231) [Receiver=3.2.7]
[default-1 (agent)]            Command returned '255'.
[default-1 (agent)]            Ensure that rsync is installed on the guest.
.... more attempts to install rsync
[default-1 (agent)]            fail: Failed to pull workdir from the guest. This usually means that login as 'root' to the guest does not work

execute step failed

Problem is that tmt-reboot has to create the file and exit the process tmt is running on that guest (as test). After that tmt needs to pull data (so guest has to be up and sshd has to be running) and tmt only then check whether TMT_REBOOT_REQUEST file exists. If it does, tmt reboots the guest.
Currently we have not channel to communicate with tmt process on the host :/

Another use case: Calling 'tmt-reboot' happens in child process (e.g. during pytest testcase).

@lukaszachy @kkaarreell so is there some advised solution?
that pidfile / envvar file is something we could have.
As this is blocking reboots for the new reserve workflow ... maybe I would look at it rather sooner

Is tmt running the actual test on a test system directly or using a wrapper script? I though there is some wrapper which could store that PID too.

Is tmt running the actual test on a test system directly or using a wrapper script? I though there is some wrapper which could store that PID too.

Yes, there's a wrapper script, https://github.com/teemtee/tmt/blob/main/tmt/steps/execute/internal.py#L31. I'm trying to update the wraper and tmt-reboot to play safely together.

Can you use these values as defaults so that users dont have to define them? Actually, why would I use different location?

Can you use these values as defaults so that users dont have to define them?

Good point, I will do that. There will be rough edges like this one, it’s still a draft. Point out more issues, please.

Actually, why would I use different location?

I don’t know, and I don’t want to care :) I for one am not using /var/tmp/tmt at all, therefore I want to keep these configurable.

Can you use these values as defaults so that users dont have to define them?

Done. Also improved logging, and fixed error handling.
There's a test for this functionality, but TBH, I'm not sure we're close to providing the needed fix. I'm worried about loose ends here and corner cases there. Testing & reviews are needed & appreciated.


Update documentation for the data package

Description of Changes
I saw the issue about getting the scaladocs out (#43) and I thought I'd update the docs a bit first. Mostly added single-sentence descriptions and some examples of use.
Might take a look at and update the other packages as well, should I continue in this PR or open a new PR per package?
Includes

[ ] Code changes
[ ] Tests
[X] Documentation

@matejklemen this is super useful! There's no need for separate PRs 🙂.

And don't worry about failed circleci jobs. This is a result of secrets not being copied to forked projects 👍.

@matejklemen let me know when this is not WIP anymore and I will review it then.

@inejc Right, I think this is ready for a review. I didn't manage to finish the docs for all packages, but I think we should try merging this before I continue (or somebody else does) writing docs in another PR.
Mostly in hopes that we keep the documentation style consistent, but also because it's getting a bit tedious to do rebasing :smile:

@matejklemen I completely agree with merging this, I will give it a quick look this week 👍.


Collapsing all nested Accordion's before displaying

Hi
Is it possible to collapse all nested accordions in an Accordion before the top-level accordion is displayed ?  Or to put is another way - is it possible to display a nested accordion will all the nested accordion's collapsed (aka a 'collapse-all') ?  If this is possible - how is done please ?

Thanks
Dave

One more thing - I'm using version 1.12.
Thanks Again
Dave

Yes, the setting is closeNested
$('.ui.accordion')
  .accordion({
    closeNested: true
  })
;

Yes, the setting is closeNested
$('.ui.accordion')
  .accordion({
    closeNested: true
  })
;

How to apply this setting? I am coding in ReactJs.


[Feature request] A function for querying transactions like the python driver provides

According to the documentation for js-bigchaindb-driver it apparently only offers functions for querying assets and metadata. I am speaking of searchAssets() and searchMetadata(). But unlike the python driver it does not support querying for transactions. I urgently need such a function in order to check that an asset (the transaction containing the asset) is owned by a certain public key.

@daniel-pan-moos Maybe the functions you are looking for are documented here.
You have getTransaction to retrieve a transaction by its id and listOutputs to retrieve transactions spent or unspent for a given publicKey.


Design improvement manual update page.

After the update to Bulma the design of this page didn't looked to well. This PR makes sure it looks a bit
better again :).
Fixes: https://github.com/intercity/intercity-next/issues/110

@brambokdam Visually looks good!

@brambokdam Can we use the same approach as with the SSH key? (Adding an SSH key that is). There is a copy button in the top.
Also for the visual look, so it has the same style?

Super seeded by: https://github.com/intercity/intercity-next/pull/121 (Fixed a jquery issue)