BeamSearchSampler failing with mx.numpy input

Description
I follow the instructions to generate sequences with Beam Search and it works correctly when the input is an mx.nd object. However, I get an error message when the input is an mx.numpy object.
With the original line, it works correctly:
inputs = mx.nd.full(shape=(1,), ctx=ctx, val=bos_ids[-1])

With this other line, it makes the sampler fail:
inputs = mx.numpy.full((1,), bos_ids[-1], ctx=ctx)

Error Message
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-18-711d82cb4e91> in <module>()
----> 1 generate_sequences(beam_sampler, inputs, begin_states, 5)

<ipython-input-17-c4c52c7fd9b6> in generate_sequences(sampler, inputs, begin_states, num_print_outcomes)
      1 def generate_sequences(sampler, inputs, begin_states, num_print_outcomes):
      2
----> 3     samples, scores, valid_lengths = sampler(inputs, begin_states)
      4     samples = samples[0].asnumpy()
      5     scores = scores[0].asnumpy()

~/anaconda3/envs/mxnet_p36/lib/python3.6/site-packages/gluonnlp/model/sequence_sampler.py in __call__(self, inputs, states)
    525                                       state_info=state_info)
    526         step_input = _expand_to_beam_size(inputs, beam_size=beam_size,
--> 527                                           batch_size=batch_size).astype(np.int32)
    528         # All beams are initialized to alive
    529         # Generated samples are initialized to be the inputs

~/anaconda3/envs/mxnet_p36/lib/python3.6/site-packages/gluonnlp/model/sequence_sampler.py in _expand_to_beam_size(data, beam_size, batch_size, state_info)
    199         new_shape[batch_axis] = batch_size * beam_size
    200         new_shape = tuple(new_shape)
--> 201         return data.expand_dims(batch_axis+1)\
    202                    .broadcast_axes(axis=batch_axis+1, size=beam_size)\
    203                    .reshape(new_shape)

~/anaconda3/envs/mxnet_p36/lib/python3.6/site-packages/mxnet/numpy/multiarray.py in expand_dims(self, *args, **kwargs)
   1435         this array as data.
   1436         """
-> 1437         raise AttributeError('mxnet.numpy.ndarray object has no attribute expand_dims')
   1438
   1439     def tile(self, *args, **kwargs):

AttributeError: mxnet.numpy.ndarray object has no attribute expand_dims

To Reproduce
Use the code for the sequence generation using Beam Search in this link:
https://gluon-nlp.mxnet.io/examples/sequence_sampling/sequence_sampling.html
And substitute the line
inputs = mx.nd.full(shape=(1,), ctx=ctx, val=bos_ids[-1])

with
inputs = mx.numpy.full((1,), bos_ids[-1], ctx=ctx)

What have you tried to solve it?

I tried to convert the mx.numpy input to mx.nd with .as_nd_ndarray(). It gets passed that error, but then Gluon Blocks requires all outputs to be mx.numpy and it fails in an old reshape method (mx.numpy should not have a named shape argument):

~/anaconda3/envs/mxnet_p36/lib/python3.6/site-packages/gluonnlp/model/sequence_sampler.py in hybrid_forward(self, F, samples, valid_length, outputs, scores, beam_alive_mask, states)
    418         beam_size = self._beam_size
    419         # outputs: (batch_size, beam_size, vocab_size)
--> 420         outputs = outputs.reshape(shape=(-4, -1, beam_size, 0))
    421         if self._top_k:
    422             ranks = outputs.argsort(is_ascend=False, dtype='int32')

Environment
We recommend using our script for collecting the diagnositc information. Run the following command and paste the outputs below:
curl --retry 10 -s https://raw.githubusercontent.com/dmlc/gluon-nlp/master/tools/diagnose.py | python

paste outputs here
----------Python Info----------
Version      : 3.6.5
Compiler     : GCC 7.2.0
Build        : ('default', 'Apr 29 2018 16:14:56')
Arch         : ('64bit', '')
------------Pip Info-----------
Version      : 10.0.1
Directory    : /home/ec2-user/anaconda3/envs/mxnet_p36/lib/python3.6/site-packages/pip
----------MXNet Info-----------
Version      : 1.6.0
Directory    : /home/ec2-user/anaconda3/envs/mxnet_p36/lib/python3.6/site-packages/mxnet
Num GPUs     : 1
Commit Hash   : 6eec9da55c5096079355d1f1a5fa58dcf35d6752
----------System Info----------
Platform     : Linux-4.14.171-105.231.amzn1.x86_64-x86_64-with-glibc2.9
system       : Linux
node         : ip-172-16-83-169
release      : 4.14.171-105.231.amzn1.x86_64
version      : #1 SMP Thu Feb 27 23:49:15 UTC 2020
----------Hardware Info----------
machine      : x86_64
processor    : x86_64
Architecture:          x86_64
CPU op-mode(s):        32-bit, 64-bit
Byte Order:            Little Endian
CPU(s):                4
On-line CPU(s) list:   0-3
Thread(s) per core:    2
Core(s) per socket:    2
Socket(s):             1
NUMA node(s):          1
Vendor ID:             GenuineIntel
CPU family:            6
Model:                 79
Model name:            Intel(R) Xeon(R) CPU E5-2686 v4 @ 2.30GHz
Stepping:              1
CPU MHz:               2701.008
CPU max MHz:           3000.0000
CPU min MHz:           1200.0000
BogoMIPS:              4600.14
Hypervisor vendor:     Xen
Virtualization type:   full
L1d cache:             32K
L1i cache:             32K
L2 cache:              256K
L3 cache:              46080K
NUMA node0 CPU(s):     0-3
Flags:                 fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx pdpe1gb rdtscp lm constant_tsc rep_good nopl xtopology nonstop_tsc cpuid aperfmperf pni pclmulqdq ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand hypervisor lahf_lm abm 3dnowprefetch cpuid_fault invpcid_single pti fsgsbase bmi1 hle avx2 smep bmi2 erms invpcid rtm rdseed adx xsaveopt
----------Network Test----------
Setting timeout: 10
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0022 sec, LOAD: 0.4192 sec.
Timing for GluonNLP GitHub: https://github.com/dmlc/gluon-nlp, DNS: 0.0005 sec, LOAD: 0.3731 sec.
Timing for GluonNLP: http://gluon-nlp.mxnet.io, DNS: 0.0927 sec, LOAD: 0.0981 sec.
Timing for D2L: http://d2l.ai, DNS: 0.0303 sec, LOAD: 0.0695 sec.
Timing for D2L (zh-cn): http://zh.d2l.ai, DNS: 0.0307 sec, LOAD: 0.1809 sec.
Timing for FashionMNIST: https://repo.mxnet.io/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.0259 sec, LOAD: 0.3750 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0034 sec, LOAD: 0.0990 sec.
Error open Conda: https://repo.continuum.io/pkgs/free/, HTTP Error 403: Forbidden, DNS finished in 0.003132343292236328 sec.

We'll have a new release of gluonnlp to support the numpy interface.
For now just convert the array to nd before calling the beamsearchsample
cc @sxjscience

For the numpy version, you may try to use the version that is initiated in https://github.com/dmlc/gluon-nlp/pull/1225


add education_api attribute for 'hidden' workflows

Add an education_api attribute to workflows (default is false) to avoid serializing a project's education api only workflows.
TODO

[x] Determine if migration will be a manual one - I am leaning towards yes to target just Education API workflows
[x] Allow this attribute to be set via the education API for new workflows.
[ ] Update education API to set this attribute to true for new lab workflows

Review checklist

[ ] First, the most important one: is this PR small enough that you can actually review it? Feel free to just reject a branch if the changes are hard to review due to the length of the diff.
[ ] If there are any migrations, will they the previous version of the app work correctly after they've been run (e.g. the don't remove columns still known about by ActiveRecord).
[ ] If anything changed with regards to the public API, are those changes also documented in the apiary.apib file?
[ ] Are all the changes covered by tests? Think about any possible edge cases that might be left untested.

The two assumptions being made here are that we'll never want to make a PB-created wf into an assignment

I made it read only after the initial create to raise this issue and avoid workflows 'disappearing'. We can add in the update paths if the need arises.

and that we never want to serialize an edu wf without it being explicitly requested. Those things both make sense to me, so lgtm.

After discussions with @srallen the classrooms use case find the workflow by id so no need to find the workflow from the project.


CAMEL-11633 - Upgrade client mqtt paho base lib

lib upgrade https://issues.apache.org/jira/browse/CAMEL-11633

Did you run Karaf integration test too?

umm... i run only mvn clean test before PR
How to run Karaf intetegration test?

in tests/camel-itest-karaf
mvn clean test -Dtest=CamelPahoTest

and before under platform/karaf/feature run mvn clean install

FIRST COMAMND OUTPUT:
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
SECOND COMMAND OUTPUT:
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 7.961 s
[INFO] Finished at: 2017-08-04T16:55:17+02:00
[INFO] Final Memory: 33M/360M

Thanks the PR has been merged. Can you close this?


SUBMARINE-265. Change hadoop-submarine to submarine in the repo

What is this PR for?
The repo have changed name to apache/submarine
we should update the name in README and code
What type of PR is it?
[Bug Fix]
Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-265
How should this be tested?
https://travis-ci.org/pingsutw/hadoop-submarine/builds/604238338?utm_medium=notification&utm_source=github_status
Screenshots (if appropriate)
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

No more comments will to be merge.


Feat: Store logged in user info in async store

Currently, users have to log in every time they open the app. This proposal suggests storing the logged-in user's information in async-storage so that the next time, we can retrieve the user info from the storage instead of initiating the Google login process again.

i want to work on this issue :smiley:

i want to work on this issue !

Every time we release a new version for download, a download update button shows up in the login page. The enhancement should take care of this and possibly load the login page every time there is an update for manual install.


BUG: false positives in B030

# reprod.py
black_list = (TypeError, ValueError)
grey_list = (ImportError, FileNotFoundError)

try:
    pass
except (*grey_list, *black_list):
    pass

$ flake8 reprod.py --select B030 # no error reported !
$ ruff reprod.py --select B030 --isolated
reprod.py:6:9: B030 `except` handlers should only be exception classes or tuples of exception classes
reprod.py:6:21: B030 `except` handlers should only be exception classes or tuples of exception classes
Found 2 errors.

ruff --version -> ruff 0.0.260
Note that this edge case used to be problematic in flake8-bugbear too (see https://github.com/PyCQA/flake8-bugbear/issues/153)

Thank you!

Wow that was quick. Thanks @charliermarsh !


Add installer creation to spack itself

e.g. run spack command to create installer

Resolved by https://github.com/spack/spack/pull/21171

Resolved by https://github.com/spack/spack/pull/21171

The latest version includes extra files in the scripts dir (e.g. CMakeLists, .in files, etc.).

Scripts dir was an easy fix, pushed to #21171


Add option to refresh youtube playlist in context menu

Right now, I have to choose the option of copying the playlist URL, deleting the existing playlist, and adding it one more time.

Duplicate - #16


false positives: Dropwizard's dependencies

Look at the following for Python false positives. See this thread regarding the issue.
<dependency>
    <groupId>io.dropwizard</groupId>
    <artifactId>dropwizard-core</artifactId>
    <version>0.9.0-rc1</version>
</dependency>
<dependency>
    <groupId>net.sourceforge.argparse4j</groupId>
    <artifactId>argparse4j</artifactId>
    <version>0.6.0</version>
</dependency>

Using 1.2.12-SNAPSHOT no python false positives were identified. However, other false positives have been resolved.


I cannot translate table title with gettext in dynamic table

Hi,
Tell me what if I have a ng-table-dynamic
ng-table-dynamic="table.tableParams with settingTable"
$scope.settingTable = [
{ title: " 'foo' | translate", field: "id", filter: { id : "text" }, show: true, sortable: "id" },
{ title: " 'bar' | translate", field: "installationAt" },
...
];
How me in this case to translate?
in continuation #695
Thank.

I'm just trying to bring on codepen  my pain
Here I show its work title, that field adminEmail working, but in this case it works one-off.
The "installationAt"  title: "'Installation' | translate" does not work. Despite the fact that if I write without ng-table-dynamic and replace it with write ng-table, data-title"'Installation' | translate" it working correctly.
I would like to first option on the codepen working. It's real?


Split genome API into three individual endpoints

Currently the  response of the genome API endpoint contains 3 combined outputs used to populate Gene, Evidence and Map View. It would be better to have separate endpoints for each of the views. It would enable us to call them individually for specific tasks that only need e.g. the ranked genes; and also in parallel to improve the loading time of the UI. Naming suggestion for the three endpoints could be:

[ ] get_ranked_genes for Gene View data
[ ] get_linked_terms for Evidence View data
[ ] get_chromosome_map for Map View data

Names to be reviewed, maybe search-xxx is clearer than get-xxx. Or, we might even end up with search/genes, search/entities, search/chromosome (reflecting the fact it's a variant of the same service).
Moreover, dash is more common for API names than underscore (ref).
Please, let's not start working new tasks before we finalise them and decide to actually tackle them.

Ok let's agree on the naming and structure first. With the new KnetMiner design in mind we could have three search modes to start with:

search/genome[input: keyword, taxid]
search/genelist[input: genelist, keyword, taxid]
search/region[input: region, keyword, taxid]

Currently these modes return the full content for Gene, Map and Evidence View. Do we want to enable users/applications to request for example just the Gene View (gene-knet-score) for reasons described above? If yes how?

search/genome
{     "geneTable": "ONDEX-ID\tACCESSION\tGENE_NAME\tCHRO\tSTART\tTAXID\tSCORE\tUSER\tQTL\tEVIDENCE\n6641999\tAT1G21970\tLEC1\t1\t7727577\t3702\t4.48\tno\t\tTrait__1__seed dormancy\n6652494\tAT1G61275\tU12\t1\t22603121\t3702\t4.32\tno\t\tTrait__1__seed maturation\n6643265\tAT1G80840\tWRKY40\t1\t30383561\t3702\t4.32\tno\t\tTrait__1__seed maturation\n6652499\tAT4G14110\tCOP9\t4\t8132880\t3702\t4.32\tno\t\tTrait__1__seed maturation\n6649574\tAT2G28056\tMIR172A\t2\t11942914\t3702\t4.03\tno\t\tTrait__1__seed weight\n6650736\tAT1G66750\tCAK4\t1\t24894523\t3702\t2.51\tno\t\tPublication__1__PMID:21908688\n6649417\tAT1G21190\tLSM3A\t1\t7419664\t3702\t1.91\tno\t\tPhenotype__1__GR21 cold\n",     "evidenceTable": null,     "geneCount": 7,     "docSize": 22,     "totalDocSize": 98,     "gviewer": null }
@KeywanHP  @marco-brandizi  are we expecting the genome response like this?

The above response has the geneTable (gene-view). Did you set evidenceTable (evidence-view) and gviewer (map-view) to "null"? All three search modes (genome, genelist and region) would produce a similar response as the current genome API but I think it would be nice if clients can request to get any of the views or combinations of views, and the output of each view would ideally be a an easy to parse plain JSON (not tabular or xml).
@marco-brandizi can advice on a good design pattern for refactoring the genome API beast. :)


ColorMatrixFilter properties won't change dynamically except on flash target

Describe the bug
Once applied to a DisplayObject instance, changing ColorMatrixFilter won't do anything after a frame passed (even when reassigning displayObject.filters = [myFilter] )
To Reproduce
Steps to reproduce the behavior:

Click on "apply grayscale"
Nothing happens visually

Expected behavior
the image on the right should turn to grayscale (it does on flash target)
OpenFL Targets
I have tested the issue on Neko, Windows and html5 targets (here is an html5 demo of it : https://matse.skwatt.com/haxe/debug/ColorMatrixFilter/ )
Additional context
the sample file uses FeathersUI as it was easier for me, if that's a problem I can create another sample file
ColorMatrixFilter_test.zip

https://github.com/openfl/openfl/commit/5b295db7003f38c77c152943b1a041b343f85beb Should resolve this issue.


feat(ingress): Check that ingress addon is enabled when using minikube

What does this PR do?
Check that ingress addon is enabled when using minikube
What issues does this PR fix or reference?
Lot of ingresses resources are created and then not working
Is it tested? How?
Make install and deploy an example like theia next

/test v5-devworkspaces-operator-e2e


ORC-702: Support big ORC files in Windows

Use 64-bit versions of fstat to support files bigger than 2GB.
Jira Issue: https://issues.apache.org/jira/browse/ORC-702

Hey @spektom is the issue Windows specific? Seems like the Cpp reader would face similar issues on a Linux env without the change right?

Quoting fstat man page:
"On  modern  64-bit  systems, life is simpler: there is a single stat() system call and the kernel deals with a stat structure that contains fields of a sufficient size."
So, apparently this isn't an issue in Linux.

Welcome to the Apache ORC community, @spektom .
You are added to the Apache ORC contributor group and ORC-702 is assigned to you.
Could you make a PR to branch-1.6, too?

Thanks @dongjoon-hyun !
Here's PR for branch-1.6: https://github.com/apache/orc/pull/585


Time unit should allow "hours" as well as "hour"

In GitLab by @jakebeal on Sep 9, 2019, 10:24

In GitLab by @jakebeal on Sep 9, 2019, 10:24
changed title from Time unit should allow to Time unit should allow{+ "hours" as well as "hour"+}

In GitLab by @nwalczak on Sep 16, 2019, 12:06
This got fixed in #110.

In GitLab by @nwalczak on Sep 16, 2019, 12:06
closed

In GitLab by @nwalczak on Sep 16, 2019, 12:06
assigned to @nwalczak


Remove Play12TestInvocation class

Play12TestInvocation class extending Invoker.DirectInvocation class in Play! 1.2.x+ is binary compatible with PlayTestInvocation class extending Invoker.DirectInvocation class in Play! 1.1.x.
I didn't find it before because Invoker.Invocation abstract class in Play! 1.2.x+ contains additional abstract method InvocationContext getInvocationContext(). This method doesn't exist in Play! 1.1.x.
Because Invoker.DirectInvocation (extending Invoker.Invocation) implements it, there is no difference between Play! 1.1.x and 1.2.x+ versions of this class from derived classes point of view (PlayTestInvocation and Play12TestInvocation).
Because Play12TestInvocation is the only class in play12-extensions module, whole module can be removed.
PlayJUnit4Provider code initializing invocation can be simplified.

Fixed. New 1.0.0-beta8-SNAPSHOT snapshot deployed. Maven site updated.


YouTubePlayerView doesn't go Full Screen

When i press the full screen button it doesn't go full screen.
I have also tried using addFullScreenListener and it goes into onYouTubePlayerEnterFullScreen, and also youtubeView.isFullScreen() returns true. So everything seems to say that yes the view is in full screen but it's not
I have also tried removing the margins and the bottom constraint but nothing
<com.pierfrancescosoffritti.androidyoutubeplayer.core.player.views.YouTubePlayerView                     android:id="@+id/youtubeView"                     android:layout_width="match_parent"                     android:layout_height="wrap_content"                     android:layout_marginStart="@dimen/margin_medium"                     android:layout_marginEnd="@dimen/margin_medium"                     android:elevation="20dp"                     app:layout_constraintBottom_toBottomOf="@+id/backdrop_image"                     app:layout_constraintEnd_toEndOf="@+id/backdrop_image"                     app:layout_constraintStart_toStartOf="@+id/backdrop_image"                     app:layout_constraintTop_toTopOf="@+id/backdrop_image"                     app:showYouTubeButton="true" />

Did you find a solution for that? I have the same issue.

Sorry for my late response, but yea i found it. Basically it seems that we have this issue when we are putting the YoutubeView inside a ConstraintLayout. I have created a fragment with a FrameLayout and the YoutubeView inside and everything works well.
Also if you check the code of the YoutubeView, when we press the full screen button the FullScreenHelper is forcing the layoutparams of the youtubeview to MATCH_PARENT. Maybe with the contraintlayout it does not allow it because of the contraints

Fragment:
import android.os.Bundle
import android.view.LayoutInflater
import android.view.View
import android.view.ViewGroup
import androidx.fragment.app.Fragment
import at.bzdevelopment.trailbook.R
import com.pierfrancescosoffritti.androidyoutubeplayer.core.player.YouTubePlayer
import com.pierfrancescosoffritti.androidyoutubeplayer.core.player.listeners.AbstractYouTubePlayerListener
import com.pierfrancescosoffritti.androidyoutubeplayer.core.player.views.YouTubePlayerView

class YoutubeFragment(val videoId: String) : Fragment() {
    private var youTubePlayerView: YouTubePlayerView? = null
    override fun onCreateView(inflater: LayoutInflater, container: ViewGroup?, savedInstanceState: Bundle?): View {
        val view: View = inflater.inflate(R.layout.fragment_youtube, container, false)
        youTubePlayerView = view.findViewById(R.id.youtube_player_view)
        initYouTubePlayerView()
        return view
    }

    private fun initYouTubePlayerView() {
        lifecycle.addObserver(youTubePlayerView!!)
        youTubePlayerView!!.addYouTubePlayerListener(object : AbstractYouTubePlayerListener() {
            override fun onReady(youTubePlayer: YouTubePlayer) {
                youTubePlayer.cueVideo(videoId, 0f)
            }
        })
    }
}

In Adapter:
val newFragment: Fragment = YoutubeFragment(variation.youtube)
val ft: FragmentTransaction = activity.supportFragmentManager.beginTransaction()
ft.add(R.id.youtube_player_fragment, newFragment).commit()

Still nothing happens if i click on the fullscreen button


Token refresh

Codecov Report

Merging #67 (c6697b0) into main (e96f868) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #67   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines          310       333   +23
  Branches        10        11    +1
=========================================
+ Hits           310       333   +23

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/client/RESTClient.ts
100.00% <100.00%> (ø)

src/lightstreamer/LightstreamerAPI.ts
100.00% <100.00%> (ø)

src/login/LoginAPI.ts
100.00% <100.00%> (ø)


How to remove the quantization  at model startup

How to remove the quantization     at model startup
(base) summer@summer:/data/ollama$ ollama show  reader-lm:0.5b
  Model
    architecture        qwen2
    parameters          494.03M
    context length      256000
    embedding length    896
    quantization        Q4_0

  Parameters
    stop    "<|im_start|>"
    stop    "<|im_end|>"

  License
    Creative Commons Attribution-NonCommercial 4.0 International
    By exercising the Licensed Rights (defined below), You accept and agree to be bound by the terms
      and conditions of this Creative Commons Attribution-NonCommercial 4.0 International Public License,
      including the Acceptable Use Addendum ("Public License"). To the extent this Public License may be
      interpreted  as a contract, You are granted the Licensed Rights in  consideration of Your acceptance
      of these terms and conditions, and the Licensor grants You such rights in consideration of benefits
      the Licensor receives from making the Licensed Material available under these terms and conditions.

ollama pull reader-lm:0.5b-fp16

th


generic-lens not building on OSX

Issue description
When trying to build any Haskell package that relies on generic-lens-<IP_ADDRESS> on OSX, it will fail building with the following error:
npacking sources
unpacking source archive /nix/store/4k9racj9716fznfq6a0169zkz2dy7i76-generic-lens-<IP_ADDRESS>.tar.gz
source root is generic-lens-<IP_ADDRESS>
patching sources
applying patch /nix/store/ppfialm5iifwbg6d6jpmf164qdlwpxc0-d9af1ec22785d6c21e928beb88fc3885c6f05bed.patch
patching file examples/doctest.hs
Reversed (or previously applied) patch detected!  Assume -R? [n]
Apply anyway? [n]
Skipping patch.
1 out of 1 hunk ignored -- saving rejects to file examples/doctest.hs.rej
builder for '/nix/store/h1fr5y17ffbzd3dv0f0799zmz3scnqmc-generic-lens-<IP_ADDRESS>.drv' failed with exit code 1
error: build of '/nix/store/h1fr5y17ffbzd3dv0f0799zmz3scnqmc-generic-lens-<IP_ADDRESS>.drv' failed

Steps to reproduce
Run nix-shell -p haskellPackages.generic-lens
Technical details
 - system: `"x86_64-darwin"`
 - host os: `Darwin 18.2.0, macOS 10.14.3`
 - multi-user?: `no`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.2`
 - channels(nick): `"nixpkgs-19.03pre167327.11cf7d6e1ff"`
 - nixpkgs: `/Users/nick/.nix-defexpr/channels/nixpkgs`

Apparently, if I override my package set with hint = self.callHackage "generic-lens" "<IP_ADDRESS>" {} it builds properly. What is causing the library to be patched?

This change https://github.com/NixOS/nixpkgs/commit/af11b2538dd120de4d33e74611edf0994ee57a37#diff-8f392bd0e4fd42a66d9e0cd4901ec287 should have removed the failing patch.


The "Style Guide" link does not work.

Clicking on the links to the "style guide", in the explanation for some of the rules, takes you to an URL that requires an username and password.

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

ID: e4cb6345-f9b7-5018-355c-6125088171f9
Version Independent ID: c7764a10-88ca-8146-6b45-b27e12423ae0
Content: CodeCop Analyzer - Business Central
Content Source: dev-itpro/developer/analyzers/codecop.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Hi, thanks for your feedback, this has now been fixed.


Newest Version of Alfred Won't Login

I am trying d > login and also trying to get it to open default browser to login and it will not do so.  Is there another way to authenticate to google drive.

There is no d > login — try d anything, does that not bring up an authentication page in your default browser?
If you wish to login using another browser, copy/paste the URL.

I rebooted and seems that after typing d anything it finally worked and asked me to login.


Validation fail in case of withdraw

It's not the first time, someone has posted this issue. I never had such problem when I setup a server. According to the log trace that you have shared, it looks like you have removed banks from banks.yml but database has the details of previously added banks which aren't exist in configuration file anymore. Try to fix this issue according to the above mentioned scenario.

Okay, I have further input on this issue. You have made a simple mistake in code. Satoshi is a coin not fiat currency so withdraw isn't bankable but coinable. Please change it accordingly. It will be fixed. Kindly, update, if this issue fixed otherwise, I would like to further assist you.


List browserify as a peer dependency instead of a normal dependency

This lets users specify their own browserify version.
This is is a backwards incompatible change and requires npm version major.
Also see #158 for a discussion

Any update this? Something to clarify or add?
Thanks

The reason I did not merge your changes is that I need to play around a bit more with peerDependencies and npm@3.
Two remaining issues:

People will regularily install karma-browserify, forgetting about the browserify dependency (installed it globally or installed incompatible version...).
We need to indicate that stuff properly via error messages if we give the user full control over the browserify.
Watchify is currently still a normal dependency. In order to work properly it should be a peerDependency too, right?

Watchify is currently still a normal dependency. In order to work properly it should be a peerDependency too, right?

Looking at the source for watchify I can see that:

It only executes require('browserify') from the bin and test files. karma-browserify invokes watchify using require('watchify'), which does not cause watchify to load browserify on its own.
watchify lists browserify as a dependency, which means that if nothing else depends on browserify, the version that watchify specifies will be used (this is only the case in npm 3, however it will pester the user with warnings every time he uses npm install). However if our parent (i.e. the user) depends on browserify, that version will be used.

So it will work properly, however the user might unknowingly start using the browserify version that watchify specifies.

People will regularily install karma-browserify, forgetting about the browserify dependency (installed it globally or installed incompatible version...). We need to indicate that stuff properly via error messages if we give the user full control over the browserify.

npm will warn the user if peer dependencies are unmet, it will look something like this:
<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY browserify@>=10 <=12

or:
npm WARN EPEERINVALID<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

It will keep pestering the user about his when he uses certain npm commands (install, update, ls, etc)

I'll probably just catch browserify and watchify require errors and report them to the user as errors.
Stuff in the npm log can easily be overlooked by users, especially because unmet peerDependencies are only warnings.
I'll look into the issue this week and hopefully merge it.

Pushed changes to the 5-0 branch. Could you check if it works for you?

Thanks, I took some time to make sure it works properly for me. I hit an unrelated issue which caused required me to find some free time to debug it.
Anyway, this is working great. I can also verify that watchify is doing its thing.
Also, browserify just released version 13, so maybe you could update the version range to include that before merging? (version 13 is working properly for me)

Ah, the unrelated issue I was hitting was caused by https://github.com/nikku/karma-browserify/commit/1e26c59579b4a3f58957f755a4912d1c02a0d85d#diff-4cd539e8c14d9e235231f6fe41be90eeR310 which is part of that 5-0 branch. I left a comment there

Thanks! I will investigate the browserify version bump and cut a release right after. Thanks for contributing :sunny:.

...and follow your suggestion regarding window :-).

Released as v5.0.0.

Moving from dependency to devDependency created me an issue. Consider the following package dependency tree:
root
├── mod-z
│   ├──<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>└── mod-a
    └──<EMAIL_ADDRESS>
If I install this with npm@3 the folder tree will look like this:
root
├── mod-a
├── mod-z
│   └──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
So if I try to run the installed karma-browserify from the mod-z it will run the<EMAIL_ADDRESS>and not the<EMAIL_ADDRESS>This was not an issue with the previous version because browserify was an explicit dependency.

You can always require a compatible version of browserify directly (i.e. in mod-z):
npm install<EMAIL_ADDRESS>karma-browserify@latest

You can always require a compatible version of browserify directly (i.e. in mod-z):

Exactly what I am doing :wink:  But IMO its kind awkward a package failing just because npm@3 is not deterministic.

You can always require a compatible version of browserify directly (i.e. in mod-z):

In the example<EMAIL_ADDRESS>is the correct version. The clash is only happening because NPM uses alphabetic ordering so mod-a is installed first and<EMAIL_ADDRESS>is placed on root.
A possible solution is to install the correct version on the root:
npm install<EMAIL_ADDRESS>

Anyhow, there is workarounds that handle that situation. :wink:


Added release NO-OP code for Hosts + added missing navigation bar setup

@krzysztofzablocki  What do you think about this approach to NO-OP release code for hosts?

Dzięki!


Let's Go Pikachu / Eevee

All of the following moves/Pokemon (placed in a list here): https://pokepast.es/cb70d2a0c6e49716 new to Let's Go Pikachu / Eevee don't pop up on the site.
Technically, Pika Papow and Veevee Volley could be added to said list (as they are also new to the series), but they aren't usable online/in Showdown, so you could just ignore them altogether, or add them for the sake of completion.
Thank you!!
For artwork purposes:
Meltan - https://i.redd.it/gouylvlezdo11.jpg
Melmetal - https://i.redd.it/bwwgrujnu4u11.png
Pikachu-Starter - https://cdn.bulbagarden.net/upload/b/b8/025Pikachu_LG.png
Eevee-Starter - https://i.pinimg.com/originals/2f/49/13/2f4913a0174ff40ddee9b2e9f65ac2fb.png
Let's Go isn't compatible with the Dream World, so unfortunately these lack official Dream World art - but these should do nicely!

Thank you for bringing this to my attention! I was holding out on the Pokemon due to the lack of Dream World art, but I hadn't even considered the new moves. I'll look into this later today.

Thank you for bringing this to my attention! I was holding out on the Pokemon due to the lack of Dream World art, but I hadn't even considered the new moves. I'll look into this later today.

Cool, thanks!


Update README to include Installation Guide

It would be great if you added an installation  guide to have this up and running @yash3001 @test-gamma

Yes i will be adding the guide in the future. Currently i have exams so not possible till 30th July.

okay, I understand.

but I've a quick question, I just clone the repo as
git clone --depth 1 https://github.com/yash3001/qtile.git

should I cd into qtile and execute autostart.sh file or I how should do it in the meantime?

autostart.sh will run by itself when qtile is started


🐛 [firestore-send-email] Sending attachments from a stream raises error

Step 1: Are you in the right place?
Issues filed here should be about bugs for a specific extension in this repository.
If you have a general question, need help debugging, or fall into some
other category use one of these other channels:

For general technical questions, post a question on StackOverflow
with the firebase tag.
For general Firebase discussion, use the firebase-talk
google group.
To file a bug against the Firebase Extensions platform, or for an issue affecting multiple extensions, please reach out to
Firebase support directly.

Step 2: Describe your configuration

Extension name: firestore-send-email
Extension version: 0.1.27
Configuration values (redact info where appropriate):

SMTP connection URI: smtp://app@redacted.com:redacted@smtp.redacted.com:587
Email documents collection: mail
Default FROM address: Redacted<EMAIL_ADDRESS>Firestore TTL type: never
Firestore TTL value: 1

Step 3: Describe the problem
Sending attachments from a stream raises an error:
Error: Value for argument "data" is not a valid Firestore document. Couldn't serialize object of type "ReadStream" (found in field "message.attachments.0.content"). Firestore doesn't support JavaScript objects with custom prototypes (i.e. objects that were created via the "new" operator).
I followed the documentation on https://nodemailer.com/message/attachments/ and added as an attachment:
{  // stream as an attachment
    filename: 'text4.txt',
     content: fs.createReadStream('file.txt')
},

Steps to reproduce:
Code:
admin.firestore().collection('mail').add({
    to: recipient,
    message: {
        subject: subject,
        text: 'Please find the data you exported in attach...',
        attachments: [{
            filename: filename,
            encoding: 'base64',
            content: fs.createReadStream(process.env.EXPORT_FOLDER + filename, { encoding: 'base64' })
        }],
    }
});

I tried without specifying the encoding, and with encoding 'binary', which both raise the same error.
Expected result
A new document in the mail collection
Actual result
The console output shows:
Error: Value for argument "data" is not a valid Firestore document. Couldn't serialize object of type "ReadStream" (found in field "message.attachments.`0`.content"). Firestore doesn't support JavaScript objects with custom prototypes (i.e. objects that were created via the "new" operator).

I checked whether the stream itself does not raise an error using:
try {
        fs.createReadStream(process.env.EXPORT_FOLDER + filename);
} catch (error) {
       console.log(`Error reading file ${process.env.EXPORT_FOLDER + filename}: ` + error);
}

The problem here is that Firestore cannot store a ReadStream, there is not a firestore data type for that. The way to support this would probably be a feature where we introduce some custom field in firestore to reference a storage object, and then the extension creates a ReadStream during its execution (i think).
Currently this behaviour is not supported, so I will close this and reopen as a feature request.


cleanup: remove lib/token-helper

Coverage increased (+1.15%) to 98.03% when pulling 6c80b3851ca615f6eef352d524a54e3d0098749a on mikesherov:tokenHelper into 2268c75cbfb07c78bf86ad965454f226aaa15f72 on jscs-dev:master.

/cc @mdevils @markelog @mrjoelkemp Everything in token-helper is superceded by the simpler methods in js-file. It has never been part of a documented public API, and only plugin developers who were specifically scrounging around and looking for it would be effected.
Can we remove this for 1.11 or must we wait for 2.0?

Whatever we decide, we should document it in our versioning guide, and we should also document the api of JsFile, etc.

98.03%

Wow!

Can we remove this for 1.11 or must we wait for 2.0?

Wasn't documented, i think it's safe to remove in minor.
Perhaps we should start documenting js-file and cli-config modules, one of the steps would be to solve https://github.com/jscs-dev/node-jscs/issues/747

Ok. I'm going to land this then and agree we should start thinking how to solve. #747


Moving route53domains (1) resource to v1beta1 version

Moving 1 resources in the route53domains group:

[ ] aws_route53domains_registered_domain

Failed with error:
Now it fails with error:  message: 'apply failed: error reading Route 53 Domains Domain (example.com): couldn''t
  find resource: '

yaml_output.txt
registereddomain.yaml.txt


The evidence file and annotation file

Hello,
I cannot find the evidence file and the annotation file (which contains column Run, Channel, Condition, BioReplicate, Mixture) from the git. I have noticed that the 'tables.pdf' mentioned the evidence file. Could you guide me the location of those files?
Thanks

Hi,
The evidence file was not uploaded to the repository since this file is not used for the analysis and is really large. However, I can look for it and upload it if you really need it.
Best,
Stephanie
From: BioLin<EMAIL_ADDRESS>Sent: Monday, November 09, 2020 10:30 AM
To: ByrumLab/ProteoViz<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [ByrumLab/ProteoViz] The evidence file and annotation file (#3)
Hello,
I cannot find the evidence file and the annotation file (which contains column Run, Channel, Condition, BioReplicate, Mixture) from the git. I have noticed that the 'tables.pdf' mentioned the evidence file. Could you guide me the location of those files?
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_ByrumLab_ProteoViz_issues_3&d=DwMCaQ&c=27AKQ-AFTMvLXtgZ7shZqsfSXu-Fwzpqk4BoASshREk&r=BpgEZuAfzn2Gtc1balIAAw&m=X8-_kWjDMOnmt0SWRPHsE4aUFkpQ3ot4BZWlOKf60Ug&s=p2gqOS6gvUogybjw4OnDlBlYda123c6EAJDNuOmlFww&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ANJMK5XO6GMZIYBMOKICVQ3SPAKJ5ANCNFSM4TPSGBWA&d=DwMCaQ&c=27AKQ-AFTMvLXtgZ7shZqsfSXu-Fwzpqk4BoASshREk&r=BpgEZuAfzn2Gtc1balIAAw&m=X8-_kWjDMOnmt0SWRPHsE4aUFkpQ3ot4BZWlOKf60Ug&s=zKuLn2bKtH3vGYvyoK82SA-m8FVSQpL1OvnBW0f-ntM&e=.

Confidentiality Notice: This e-mail message, including any attachments, is for the sole use of the intended recipient(s) and may contain confidential and privileged information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply e-mail and destroy all copies of the original message.

Thank you for the prompt reply. I am working on comparing different models and need to use the evidence file for MSStatTMT package. If it is too large for github, can you send it to my email<EMAIL_ADDRESS>Best


Enhancement: Remove duplicate ID properties in JSON output for spo listitem commands

Raised from https://github.com/pnp/cli-microsoft365/discussions/1762

The issue is specific to PowerShell implementation of ConvertFrom-Json rather than the CLI, see the below output from my tests.

ConvertFrom-Json appears to be unable to handle properties with the same name but have different casing, adding the -AsHashtable switch works around the issue in PowerShell Core, however I do know that -AsHashtable is not present in PowerShell 5 which is bundled with the Windows OS, so users will have to use the Replace workaround as you have already stated, which is less than ideal.
We should consider removing duplicate properties from our output in this case.

When ConvertFrom-Json is used to parse the JSON output from the spo listitem commands it can fail as it cannot determine that Id and ID properties are different as it does not take casing into account.
For PowerShell 5 users, the workaround is to use replace to change the Id property name before parsing
$spolItems = (o365 spo listitem list --webUrl "https://tenant-name.sharepoint.com/sites/site-name" --title "list-title" --fields "Id,Title,Modified" -o json).ToString().Replace("ID", "_ID") | ConvertFrom-Json

For PowerShell 7 users, the workaround is to use the -AsHashtable switch on ConvertFrom-Json (note that -AsHashtable is not present in PowerShell 5.
$spolItems = o365 spo listitem list --webUrl "https://tenant-name.sharepoint.com/sites/site-name" --title "list-title" --fields "Id,Title,Modified" -o json | ConvertFrom-Json -AsHashtable

This is less than ideal, so we should look to remove the extra ID property from the spo listitem list and spo listitem get commands.

It's weird that there are two IDs that we get back from the API and it makes sense to merge them. Let's keep in mind, that it's a breaking change so not something we should implement immediately.


Grouped Table Collapse Grouped Value

Issue Type (delete the irrelevant ones)

[x] Enhancement Request
[x] Question

Specs
What version are you using?
v2.13.3
What browser?
any
Expected Behavior
What did you expect to happen?
I thought I could collapse the table grouping
Actual Behavior
What actually happened?
It did not happed. Because this component hasn't grouped values collapse option.
Example:
...
:groupOptions="{
enabled: true,
collapseGroupedColumn: true // like this...
}"
...
How can i do?

hey @eklnc, this is currently not possible with vgt. Will chalk this up as an enhancement for now.

I was looking into this feature also, but not collapsible columns but collapsible rows instead, so that the spanned header rows appear collapsed by default and if I need to look at the children I could do so by clicking and expanding the row I'm interested in.


Memory Leak in "Human 3D Pose Estimation" https://github.com/openvinotoolkit/openvino_notebooks/blob/latest/notebooks/3D-pose-estimation-webcam/3D-pose-estimation.ipynb

Describe the bug
Program ran easily as expected with very pretty graphics.  After 30 minutes, memory usage exploded from 352 MB to 897 MB and continues to grow.  Will eventually cause the computer to crash (happened earlier).
Expected behavior
I expected the program to run within a reasonable amount of memory without growing out of control and crashing the system.
Screenshots
If applicable, add screenshots to help explain your problem.

Installation instructions (Please mark the checkbox)
[ X ] I followed the installation guide at https://github.com/openvinotoolkit/openvino_notebooks#-installation-guide to install the notebooks.
https://github.com/openvinotoolkit/openvino_notebooks/blob/latest/notebooks/3D-pose-estimation-webcam/3D-pose-estimation.ipynb.  Running on Webcam mode so changed "use_popup=True" and "use_webcam=True".
** Environment information **
Windows 11, not an IT build. Fully updated.  Python 3.11.9.  openvino version 2023.3.0-13775
Additional context
Add any other context about the problem here.
(openvino_env) C:\Users\Intel\openvino_notebooks>python check_install.py
System information:
Python executable: C:\Users\Intel\openvino_env\Scripts\python.exe
Pip version: 24.2
OpenVINO source: C:\Users\Intel\openvino_env\Lib\site-packages\openvino
OpenVINO IE version: 2024.4.0-16579-c3152d32c9c-releases/2024/4
OpenVINO environment activated: OK
Jupyter kernel installed for openvino_env: NOT OK
Python version: 3.11 OK
OpenVINO pip package installed: OK
OpenVINO import succeeds: OK
OpenVINO development tools installed: OK
OpenVINO not installed globally: OK
No broken requirements: OK
The Python version in openvino_env does not match the openvino_env Jupyter kernel.
This may not be an issue. If you experience issues, please follow the instructions
in the README to reinstall the kernel.
The README.md file is located in the openvino_notebooks directory
and at https://github.com/openvinotoolkit/openvino_notebooks

I initially just took the code and copy/pasted it into my customer demo without Jupyter anything.  I ran it overnight for a stability test and the memory overflowed and crashed the system.  To fill out the bug report, I went ahead and installed Jupyter something-or-other just so I could say I followed the instructions on the bug report.  Task Manager indicated that the memory was noticeably growing in the Jupyter environment too.  It would take many hours to crash, but it’s still a memory leak.
I lost my reservation on that computer and won’t be able to get onto it until tomorrow.  I’ll see what flavor of “Jupyter” is running.
Cheers,
Scott
From: markusbr @.>
Sent: Thursday, September 26, 2024 2:12 PM
To: openvinotoolkit/openvino_notebooks @.>
Cc: Thomas, Scott A @.>; Author @.>
Subject: Re: [openvinotoolkit/openvino_notebooks] Memory Leak in "Human 3D Pose Estimation" https://github.com/openvinotoolkit/openvino_notebooks/blob/latest/notebooks/3D-pose-estimation-webcam/3D-pose-estimation.ipynb (Issue #2420)
Do you use "Jupyter Notebook", or, as recommended within the notebook, use "JupyterLab"?
(I haven't used "JupyterLab", yet, I think, but just to make sure we all use the same fo reproduction)
—
Reply to this email directly, view it on GitHubhttps://github.com/openvinotoolkit/openvino_notebooks/issues/2420#issuecomment-2377945845, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AM4KMYEU3LTBOMR3VXW6WPLZYR2AXAVCNFSM6AAAAABO5Z4KROVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZXHE2DKOBUGU.
You are receiving this because you authored the thread.Message ID: @.***>

Hi @TheArbitraryConstant I didn't see any memory leaking when running in JupyterLab. Below are my observations:
use_popup=True" and "use_webcam=True".

use_popup=False" and "use_webcam=True".

use_popup=False" and "use_webcam=False".

It is recommended to use JupyterLab.

Sorry, I got busy.  If the leak is fixed on this model, I’ll just switch to this.  I’ll be sure and whine to you if it persists, but I don’t expect it to if it works for you.
Cheers,
Scott
From: Noor Aznie Syaarriehaah Baharuddin @.>
Sent: Monday, September 30, 2024 4:50 PM
To: openvinotoolkit/openvino_notebooks @.>
Cc: Thomas, Scott A @.>; Mention @.>
Subject: Re: [openvinotoolkit/openvino_notebooks] Memory Leak in "Human 3D Pose Estimation" https://github.com/openvinotoolkit/openvino_notebooks/blob/latest/notebooks/3D-pose-estimation-webcam/3D-pose-estimation.ipynb (Issue #2420)
Hi @TheArbitraryConstanthttps://github.com/TheArbitraryConstant I didn't see any memory leaking when running in JupyterLab. Below are my observations:
use_popup=True" and "use_webcam=True".
true.JPG (view on web)https://github.com/user-attachments/assets/fa93c6f4-ef92-4b1c-8fde-7642633fc7d7
use_popup=False" and "use_webcam=True".
Capture.JPG (view on web)https://github.com/user-attachments/assets/38530bdd-3a00-4bab-af9f-3cb0e11405f5
use_popup=False" and "use_webcam=False".
using-video.JPG (view on web)https://github.com/user-attachments/assets/c7851fd9-c819-412f-8443-f123023eb5f9
It is recommended to use JupyterLab.
—
Reply to this email directly, view it on GitHubhttps://github.com/openvinotoolkit/openvino_notebooks/issues/2420#issuecomment-2384495896, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AM4KMYDQPESRDCU56ZAVIQLZZHPSNAVCNFSM6AAAAABO5Z4KROVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBUGQ4TKOBZGY.
You are receiving this because you were mentioned.Message ID: @.***>


Improve license criticality shown in Project Statistics popup

Currently we have no clear strategy of license criticality identification while deduplication for the project statistics popup, see function getLicenseCriticality in the commit https://github.com/opossum-tool/OpossumUI/commit/e150967cb16b7195623e4bd50fd57b9d2715c231
The goal is to find a strategy, which will correctly highlight licenses and be clear for the user.

Improved in #2233


[Bug] REGRESSION - new "multiselect" causes silent issue on "select" grids

Steps to reproduce the bug
CC @robertjones1
With any prior https://github.com/bosskmk/pluto_grid/issues/615 version (5.3.0 and lower), any grids using "select" mode were able to indeed perform a "multiSelect" using hotkeys (shift and Ctrl). In a recent upgrade to the latest 5.4.5 (from 5.0.3) we found that we lost the ability to do that, and then the investigation pointed to 426cb8974af52ef36fbea7761d4d492ff4324fb8 as the root cause.
I've tried the new "multiSelect" mode but this is not exactly the same because selection occurs WITHOUT any keypress, so I think this is a regression since 5.3.1 changes.
So, the obvious workaround is stay at 5.3.0:
pluto_grid: 5.3.0

Expected results
Without any code change, "select" mode still let users to select multiple rows by hand using hotkeys.
Or a new "multiSelectWithKeyboard" mode is added (in order to differentiate with current "multiSelect" with only clicks/taps.
Actual results
"select" mode only allows to select one row at a time.
Code sample
n/a
Execution Environment
Flutter version
Flutter 3.3.9
PlutoGrid version
PlutoGrid version is 5.4.5
OS
MacOS Ventura

With this kind of changes you should take in consideration regression issues.
One alternative could be deprecate "select" mode and add newer, specialized modes, in order to alert all devs on the new changes. I know, I know, we must review every changelog 😅 but sometimes you simply "bump-version-and-go" :)
Thanks for your effort.

I agree with you.
I should have named it well when adding multiSelect.
But since I already have the current behavior of selectWithOneTap, I guess I'll have to replace multiSelect with multiSelectWithOneTap to be consistent.
I'll think about it some more and update.
thank you

@bosskmk Please keep open this one

@henry2man
yes i removed the stale tag.

@bosskmk Can we add some tag that prevents Github actions bot closing this ?

@bosskmk This is still closed but not fixed ❤🩹

@bosskmk This issue now is a blocker for us. We cannot properly update to Flutter 3.10.3 because newer versions depends on intl: 0.18.0 (https://github.com/bosskmk/pluto_grid/issues/836).
With current 7.0.2 the multiselect/select issue is still present.
We can apply a workaround (force intl to 0.18.0 in dependency_overrides) but this is dirty.

@bosskmk Are there any plans to consider this issue again? As I said this is a regression, so we cannot update to modern versions of the package.


test: pkg/cli EnvSettings.Namespace()

In pkg/cli EnvSettings.Namespace() does not have good unit test coverage.
It would be great to write a small test for this function.
We should mock s.config.ToRawKubeConfigLoader()... and return an error to test this
The default string should also be a constant.

ref #1489


[#11][Feature] Change trailing commas to 'always'

#11
From Compass: RFC: JavaScript / TypeScript trailing commas for the last item #765

What happened

Changed the trailing commas rule from never to always (as well as its comment)
Increment the package version from 2.1.1 to 2.2.0 (new feature, but not breaking changes as auto-fix is working well).
Improve some wording and update license date (README.md + LICENSE.md)

Insight
N/A
Proof Of Work
The Files changed seems more explicit ;-)

@malparty Once fully approved, please take care of merging the PRs, preparing the release PR, and pushing the new version to NPM. Credentials should be in 1Password.

@malparty Once fully approved, please take care of merging the PRs, preparing the release PR, and pushing the new version to NPM. Credentials should be in 1Password.

@olivierobert merged, release tag added (I follow the last release format, without the v prefix) and npm package published ;-)


macOS doesn't refresh window contents during resizes

glfwWaitEvents / glfwPollEvents block during resizing gestures, and the window contents don't get refreshed until the mouse button is released.
Windows has a similar issue that was fixed by patching https://github.com/glfw/glfw/pull/1426, which uses Win32 fibers to keep pumping the main loop during resizes (and do it in the main thread).
Another approach is to render in a separate thread, but that complicates a lot threading and state synchronization for JS API calls.

Resizing the window on Macs with Retina displays also shows instability about the size and/or position of the main canvas texture during resizes. This doesn't happen on other platforms nor on non-Retina Macs.

The "angle" branch solves many of these issues, after patching ANGLE to use CVDisplayLink for syncing with monitor refreshes.
The remaining issues are some red borders that appears during some resizes, slow resizes when close to the monitor large on 5K retinas, and unstable position of the content during resizes with vsync turned off.


UniProt API Update

This pull request addresses Issue #24 and implements protein mass mapping consistent with the new UniProt REST API.

Thank you (again :-) very much for this bug fix :D I saw your issue #24, but just (again xD) didn't find the time to change it. Since several bug fixes found their way into AutoPACMEN since the very old last release, I'll also try to start-up the very old release routine so that it should be directly integrated in the PyPI module.


Custom icon doesn't manage correctly

Describe the bug
I created a technology based on a custom icon. It works correctly when I created manually the technology.zip
But on new version, when the package is created automatically by the CI/CD, the icon was not present and I got an error.
Expected behavior
Modify the CI/CD process to include the icon files inside the package generated.

Trello card: [#221] Custom icon doesn't manage correctly


Draggabilly + packery and display attribute

I'm using packery and draggabilly to make a layout with draggable items. Sometimes I need to hide and show some of the items, the thing is, when items get shown after being hidden they act inconsistently (they bounce when dragged, the is-dragging state stays forever). Is there some way of achieving normal behavior of grid items after playing with their display attribute? Thanks.

I'm sorry to see you're having trouble with Packery. Could you provide a reduced test case? See Submitting Issues in the contributing guidelines.


App reports bug

What happened?
A bug happened!
Steps to reproduce

step one...
step two...

What OS are you seeing the problem on?
No response
Relevant log output
failed to open model

Caused by:
    Failed to create a new whisper context.

Location:
    core\src\transcribe.rs:39:65
App Version: vibe 3.0.0
Commit Hash: 371af90
Arch: x86_64
Platform: windows
Kernel Version: 10.0.19045
OS: windows
OS Version: 10.0.19045
Cuda Version: n/a
Models: ggml-distil-large-v3.bin, ggml-ivrit-v2-d4.bin, ggml-large-v3.bin, ggml-medium.bin
Default Model: "C:\\Users\\xxx\\AppData\\Local\\github.com.thewh1teagle.vibe\\ggml-ivrit-v2-d4.bin"
Cargo features: vulkan

{"avx":{"enabled":true,"support":true},"avx2":{"enabled":true,"support":true},"f16c":{"enabled":true,"support":true},"fma":{"enabled":true,"support":true}}

<details>
<summary>logs</summary>

whisper_model_load: failed to allocate memory for the model
whisper_init_with_params_no_state: failed to load model

</details>

You don't have enough memory. use the default model that comes with vibe (the new one)
https://huggingface.co/ggerganov/whisper.cpp/resolve/main/ggml-medium-q8_0.bin


audioprocess event is not fired on seeking

The documentation states that audioprocess fires on seeking, however this is not the case when using WebAudio
This issue occurs with the default backend setting. When setting the backend to MediaElement the audioprocess gets fired on seekTo, however it doesn't actually do the seeking.
The seek event is fired correctly.
Here's my code for testing. Notice the different console log messages when (un)commenting the backend option when clicking the button. EDIT: Here's a codepen: http://codepen.io/anon/pen/RrxaKM
This is on Chrome 46.0.2490.86 and Firefox 42.0 for Linux.
<button id="seek-btn">Seek</button>

yep i have the some issue with that i had to name my function and call it with the two events one ofter another
wavesurfer.on('seek',showProgress);
wavesurfer.on('audioprocess', showProgress);

I'm labelling this v1 because the v2 docs don't mention that audioprocess should be called on seekTo :chicken:  and IMHO I think it's good we have two events for this (even if it does mean more code).
(I know I'm contradicting the me from two years ago, but I now think audioprocess should be reserved for when the audio is actually playing)
I've tested this with both backends (in v2) and they seem to be consistent:

seekTo fires seek
playback fires audioprocess

stale issue, closing.


Unable to find where I can generate a device password

Hello, I can't find where I can generate a password device from my Spotify Account page 😅

I used my account password ans it seems to work... Maybe you should modify the doc 😅

https://bfy.tw/RqTE


AFL beginner - Program 'guff' not found or not executable

Hi,
I am a beginner of AFL, I have been following the instructions on "Fuzz testing with afl-fuzz" (https://spin.atomicobject.com/2015/08/23/fuzz-testing-american-fuzzy-lop/ ).
I have compiled guff and placed it on my Desktop.
I was wondering why 'guff' is detected as not found or not executable, and how to fix this?
Note that I am running AFL on Ubuntu 16.04.7
The terminal instruction and respond are as follow.
~/Desktop$ afl-fuzz -i AFL/afl/in -o AFL/afl/out guff -x -lyafl-fuzz 1.96b by<EMAIL_ADDRESS>[+] You have 8 CPU cores and 1 runnable tasks (utilization: 12%).
[+] Try parallel jobs - see /usr/share/doc/afl/parallel_fuzzing.txt.
[] Checking core_pattern...
[] Checking CPU scaling governor...
[] Setting up output directories...
[+] Output directory exists but deemed OK to reuse.
[] Deleting old session data...
[+] Output dir cleanup successful.
[] Scanning 'AFL/afl/in'...
[+] No auto-generated dictionary tokens to reuse.
[] Creating hard links for all input files...
[*] Validating target binary...
[-] PROGRAM ABORT : Program 'guff' not found or not executable
Location : check_binary(), afl-fuzz.c:6548
Many Thanks.

Hi @CK213, I'm not sure about this, but my guess is that you need to specify the executable as ./guff if it is just in your desktop directory, and not on your path. Generally, your current directory isn't on your path for security reasons.

Hi @ianruh, thanks for the fast response, and yea I have tried to specify the executable as ./guff and ~/guff but both execution return the same type of error.
PROGRAM ABORT : Program './guff' not found or not executable

Hi @ianruh, I have figured out my problem. I thought I needed to put the entire guff file inside AFL then run it, but I only have to put the program "guff" in.


Best way of building source code when other runtimes exist in the project

Wondering what exactly is the best way to use the plugin, and maintain the same file directory of "src/", but where not all functions are in the node runtime?
My project has a mix of lambda that are written in node, while others are written in python. Is there a better alternative than just building them as separate projects?

You need to build the Lambda as separate projects. One of the early decisions was to optimize the experience for Node developers even if that meant not supporting other runtimes.


Carstream Block!!!!

The new version of Android auto is now v3. Carstream is completely blocked!

Agreed. It is definitely blocked a new way. Repackaging the app no longer works. It still shows under audio playback but not under rightmost icon any longer. Tested with 1.8.3 and followed repackaging instructions which worked great before Android Auto be.

Downgrade AA version from 3.0 to 2.9

Well..
I nave never updated to 3.0 and it is 2.9 but since yesterday morning it doesn't work anymore. It keeps saying " Sorry, No more videos  available". So, Imply thet  3.0 can't be the problemi! Maybe Karum can say much more :)

Magically it has started to work again just after having written the comment over here. The only thing I did was to go the internal SD , listen to an mp3 and get back to the carstream home. And voilà...everything started working fine.
P.S: anyway, it is 2.9 ...i don't have auto update and checked it out it was 2.9. Do you all confirm that 3.0 isn't working?

Somebody claims it works but not proved. https://github.com/thekirankumar/carstream-android-auto/issues/109

@Scutley..... it worked great ... THANK YOU SO MUCH was really bummed when it stopped.

Hi, sorry for my bad english, the following happens to me. Uninstall AA 3.0, I go back to version 2.9.574923
suggested and so and all restart and I am in my car with carstream blocked and the next screen attached

Same problem here, reverting to 2.9 fixes the problem.

this situation occurred reverting to version 2.9.It is not the solution

Not in my case, and as I understand, not in the others.
With 3.0 doesn't work, with 2.9 it worked before, and keeps working after reverting from 3.0.

Yes, with AA2.9 works. Unistall AA3.0 and revert to AA2.9. Once you revert to AA2.9, do not start it in your mobile, otherwise it will upgrade again to AA3.0.
Go to PlayStore and block automatic updates. Assuming you have already installed fixed Carstream, then go in your car and just plug your mobile with AA2.9 installed. Once AA starts in your car, start Carstream and it should work fine.

AA older version is forcing auto update when I open the app. Is anybody else facing this issue?

It only forces it on versions < 2.9

Yes you are right it's working now. I had downloaded 2.8 just to be safe :)

I think google team is watching us)))

ook this. I revert to 2.9 .I have blocked the update the playstore reinstalled carstream and there is no solution. Surely the developer will solve the issue in a short time
https://user-images.githubusercontent.com/36753645/36578049-ade12fd8-1839-11e8-84e8-66f30b2df49c.png

You're looking in the wrong spot. I have this same behavior. There is that blank audio player in the music section of android auto, but what you're looking for is the part of android auto that is on the car dashboard in the very bottom right of the navigation bar. In my Audi, it'll usually just have the option "return to audi" but with carstream it'll have the carstream app listed there as well. At this point you can use the app, but it does not show if the phone is not plugged in to the car's screen.

I worked to go back to version 2.9 of android auto, thank you very much catshow2010

Me funciona perfectamente con 2.9

Hi I have downloaded new version of carstream and used app cloner also but  it is not working. Pls help

See issue #116

Reverting to 2.9 works with s6 (AA3) but not with A3 16 (AA 3). The last doesn't show carstream under "return to sync 3"...

Iv done it all and still can't get it to work. Down grade to aa2.9. Tried the apk editor trick. Tried the plug into windows and download from the computer trick. Nothing works.

You need to uninstall AA. Then install AA2.9. Then immediately go in to Google play and in the listing for AA hit the three dots in the upper right and disable auto update for AA. Then install CS1.8.3.
I just did all of these steps two days ago and it worked fine.

What's cs18.3?
On Thu, Apr 5, 2018, 5:49 PM Eric Smith<EMAIL_ADDRESS>wrote:

You need to uninstall AA. Then install AA2.9. Then immediately go in to
Google play and in the listing for AA hit the three dots in the upper right
and disable auto update for AA. Then install CS1.8.3.
I just did all of these steps two days ago and it worked fine.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/thekirankumar/carstream-android-auto/issues/112#issuecomment-379099509,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkWXytLl9jugAsuxznpF9Io615G5l85Aks5tlp9rgaJpZM4SN74U
.

Sorry. CS1.8.3 as in CarStream 1.8.3

So I figured out what's going on. I have the carstream 1.8 downloaded but
when I do the Android auto 2.9 download when I open it it forces me to do
the update now I've already gone into Google play settings and turned off
auto updates. When I download it it tells me it's version 2.9 when I plug
it into the cord into the vehicle and try to start it, it says I must
update Android auto to the latest software version in order to use it. I'm
using a Samsung Note 8.
On Thu, Apr 5, 2018, 6:49 PM Eric Smith<EMAIL_ADDRESS>wrote:

Sorry. CS1.8.3 as in CarStream 1.8.3
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/thekirankumar/carstream-android-auto/issues/112#issuecomment-379109402,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkWXyqb3oOJCm2yvF486Yrqj0elsWkm3ks5tlq2OgaJpZM4SN74U
.

It would not allow me to email everything all in one e-mail I'm sorry to
send you all the different emails with the attachments. As you can see all
apps are running diversion they should and as you can see it does not work
On Thu, Apr 5, 2018, 5:49 PM Eric Smith<EMAIL_ADDRESS>wrote:

You need to uninstall AA. Then install AA2.9. Then immediately go in to
Google play and in the listing for AA hit the three dots in the upper right
and disable auto update for AA. Then install CS1.8.3.
I just did all of these steps two days ago and it worked fine.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/thekirankumar/carstream-android-auto/issues/112#issuecomment-379099509,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkWXytLl9jugAsuxznpF9Io615G5l85Aks5tlp9rgaJpZM4SN74U
.

I just wondering when the new version of CarStream will out upload for us? V1.8.4

Im a network engineer and part time android dev. This does NOT work on my 2018 Chevy. Looks like the Google Play services update has blocked it. If you set your Android auto view to developer mode when you go to the last icon on your dashboard you can see all of the apps that it shows as compatible but carplay is not listed it only shows up under the headphones icon which is used for Google Play Music

This worked for me (root required):
https://forum.xda-developers.com/android/help/delete-disable-downgrade-prevent-update-t3091832

Ho Guys sale problem even downloading the apk 2.9 To be able to access the developer mode of the Android auto app, as in v3. Is not accessible.
I got now promt please update your app,but I cannot cancel so you are stuck.
I also deactivate auto update Google apps in app store...
Is there any code we can build to avoid this issue or create a version where the dev.mode is activated
My final aim is to be able to mirror my mobile to the screen of Mercedes Benz screen that is not touch screen

@AndreDylan you should root your phone and use AA Phenotype Patcher

@shmykelsa it is not working. The process assumed the AA had the dev mode locked. The issue here is that the new version does not have at all the dev mode...it is not there.
Plus even downloading  2.9 version and deactivating in Google market the auto update..it still block the app to proceed installation asking you..update the up, the cancel buttom is unresponsive
Is there any discussion on the mirroring of your phone on the screen of your no touch screen of a car..in my case a Mercedes

Developer options Is available in every version of Android Auto.
You simply can't use Android Auto 2.9 because newer Google Play Services block you from doing so. If you want to try it "the hard way" you should downgrade to something like Google Play Services 14 and Android Auto 2.9 and totally disconnect from the internet. If your phone has a recent software, chances are you won't be able to downgrade to Google Play Services 14.

Hi again, can you show me how to view the dev mode. When I download the last version the dev.mode does not exist anywhere
The ten times click on about it is not working and it is not executable it seems all the instruction our there are for older versions of AA.
I cannot attach the pic here otherwise I would show you that when you click about and the three dots there are only two option available no dev mode

Aww I finally find  how to do it...you need to tap in the top bar not the about as in the older versions thanks


Fix support for find-uses for definitions

Running haskell-mode-find-uses on the type or data constructor for Hughes only highlights the new type, pressing <TAB> and the like doesn't go anywhere.
Am I doing something wrong?

Dupe of #3, closing.


Class constructor does not infer return type in some cases

// @flow
class Foo {
    constructor() {}
}
function foo(x: Foo | boolean): Foo {
    return x instanceof Foo ? x : new Foo();
}

flow coverage shows that new Foo() expression is not covered. Running flow coverage --debug test.js produces:
...
6:34,6:43: (false)
...

This looks like a bug. cc @mroch @gabelevi

Somehow new Foo becomes any in the case. This works fine:
function foo(x: Foo | boolean): Foo {
  return typeof x === 'boolean' ? new Foo() : x;
}

@vkurchatkin here's a real-world example I reduced the case from:
'use strict'; // @flow

class LngLat {
    lng: number;
    lat: number;

    constructor(lng: number, lat: number) {
        this.lng = +lng;
        this.lat = +lat;
    }

    static convert(input: LngLat | [number, number]): LngLat {
        if (input instanceof LngLat) {
            return input;
        }
        if (Array.isArray(input) && input.length === 2) {
            return new LngLat(input[0], input[1]);
        }
        throw new Error("Nooooo!");
    }
}

I don't know how to make it fully covered.

What I see that we need 3 conditions to reproduce this bug:

a class should have a constructor defined;
variable should have union type, where one of the branches is the class;
the variable should be check in any way (not necessarily with instanceof );

Simple repro:
class Foo {
  constructor() {} // 1
}

function test(foo: number | Foo) { // 2
  if (foo) { // 3
  }
}

(new Foo(): number); // Foo is infected, probably in the whole file

@vkurchatkin perfect test case, thanks for taking the time to come up with it!

One small addition: Foo is only infected after foo check, but scope doesn't matter.

Strangely, this fixes the issue:
class Foo {
  constructor():void {}
}

Looks like I'm having this without (probably) the union operator at https://github.com/jamesgorman2/graphql-type-repository/commit/5b6ca89b4b91b5220a4c3c0c48853849cc9b41b1
With voids I have 97% coverage
with-voids.zip
without it's 87%
without-voids.zip
I haven't actually run up which classes suffer from this and when comparing the attached output from flow-coverage-report should help.


Closing curly brace after perl HereDoc breaks folding (and matchit %)

Description of issue
When there is a curly brace after the HereDoc (at the same line), then the vim folding gets confused and can't detect the end of the folding correctly, leading to sibling subs being in nested folds. Confused? Look at the screenshot and the sample code.
Screenshot of the issue:

Sample code:
#!/usr/bin/env perl

use strict;
use warnings;

sub foo {
    ...
    my $foobar = do { << "    EOF";
        This works.
        Because the closing brace is at another line.
    EOF
    };
}

sub foob {  # <- matchit % breaks here
    ...
    my $foobar = do { << "    EOF" };
        This breaks both folding and the % from matchit.
        Because the closing brace is right after the HEREDOC declaration.

        Note that % from matchit plugin is broken, but builtin % works:
        :nnoremap <F11> %

        Objects also work fine, see :help objects

        In summary:
        * Built-in vim features (% and objects) work fine.
        * vim-perl folding is broken.
        * matchit % is broken.
    EOF
    ...
}  # <- matchit % matches wrong brace here

# <- The folding from the previous sub continues on, but it shouldn't.

sub bar {  # <- This sub gets folded INSIDE the previous sub. That's wrong.
    ...
    my $foobar = << "    EOF";
        This works.
        Because the closing brace is at another line.
    EOF
}

sub backtonormal {
    ...
}

1;

vimrc
Minimal .vimrc that reproduces this issue:
set nocompatible
set foldcolumn=4
syn on
let perl_fold = 1

Output of vim --version
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Apr 10 2018 21:31:58)
Included patches: 1-1453
Modified by<EMAIL_ADDRESS>Compiled by<EMAIL_ADDRESS>Huge version with GTK3 GUI.  Features included (+) or not (-):
+acl               +farsi             +mouse_sgr         -tag_any_white
+arabic            +file_in_path      -mouse_sysmouse    +tcl
+autocmd           +find_in_path      +mouse_urxvt       +termguicolors
-autoservername    +float             +mouse_xterm       +terminal
+balloon_eval      +folding           +multi_byte        +terminfo
+balloon_eval_term -footer            +multi_lang        +termresponse
+browse            +fork()            -mzscheme          +textobjects
++builtin_terms    +gettext           +netbeans_intg     +timers
+byte_offset       -hangul_input      +num64             +title
+channel           +iconv             +packages          +toolbar
+cindent           +insert_expand     +path_extra        +user_commands
+clientserver      +job               +perl              +vertsplit
+clipboard         +jumplist          +persistent_undo   +virtualedit
+cmdline_compl     +keymap            +postscript        +visual
+cmdline_hist      +lambda            +printer           +visualextra
+cmdline_info      +langmap           +profile           +viminfo
+comments          +libcall           -python            +vreplace
+conceal           +linebreak         +python3           +wildignore
+cryptv            +lispindent        +quickfix          +wildmenu
+cscope            +listcmds          +reltime           +windows
+cursorbind        +localmap          +rightleft         +writebackup
+cursorshape       +lua               -ruby              +X11
+dialog_con_gui    +menu              +scrollbind        -xfontset
+diff              +mksession         +signs             +xim
+digraphs          +modify_fname      +smartindent       +xpm
+dnd               +mouse             +startuptime       +xsmp_interact
-ebcdic            +mouseshape        +statusline        +xterm_clipboard
+emacs_tags        +mouse_dec         -sun_workshop      -xterm_save
+eval              +mouse_gpm         +syntax
+ex_extra          -mouse_jsbterm     +tag_binary
+extra_search      +mouse_netterm     +tag_old_static
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
  system gvimrc file: "$VIM/gvimrc"
    user gvimrc file: "$HOME/.gvimrc"
2nd user gvimrc file: "~/.vim/gvimrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
    system menu file: "$VIMRUNTIME/menu.vim"
  fall-back for $VIM: "/usr/share/vim"
Compilation: gcc -c -I. -Iproto -DHAVE_CONFIG_H -DFEAT_GUI_GTK  -pthread -I/usr/include/gtk-3.0 -I/usr/include/at-spi2-atk/2.0 -I/usr/include/at-spi-2.0 -I/usr/include/dbus-1.0 -I/usr/lib/x86_64-linux-gnu/dbus-1.0/include -I/usr/include/gtk-3.0 -I/usr/include/gio-unix-2.0/ -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/pango-1.0 -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pixman-1 -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/libpng16 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -Wdate-time  -g -O2 -fdebug-prefix-map=/build/vim-NQEcoP/vim-8.0.1453=. -fstack-protector-strong -Wformat -Werror=format-security -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: gcc   -Wl,-E  -Wl,-Bsymbolic-functions -Wl,-z,relro -Wl,-z,now -Wl,--as-needed -o vim   -lgtk-3 -lgdk-3 -lpangocairo-1.0 -lpango-1.0 -latk-1.0 -lcairo-gobject -lcairo -lgdk_pixbuf-2.0 -lgio-2.0 -lgobject-2.0 -lglib-2.0 -lSM -lICE -lXpm -lXt -lX11 -lXdmcp -lSM -lICE  -lm -ltinfo -lnsl  -lselinux  -lacl -lattr -lgpm -ldl  -L/usr/lib -llua5.2 -Wl,-E  -fstack-protector-strong -L/usr/local/lib  -L/usr/lib/x86_64-linux-gnu/perl/5.26/CORE -lperl -ldl -lm -lpthread -lcrypt  -L/usr/lib/python3.6/config-3.6m-x86_64-linux-gnu -lpython3.6m -lpthread -ldl -lutil -lm -L/usr/lib/x86_64-linux-gnu -ltcl8.6 -ldl -lz -lpthread -lm

Misc. comments
I have read the guidelines.
I've reproduced it with both a checkout from the master branch and from the dev branch. (By the way, the master branch is quite behind.)
By looking at recent commits, user @zdm might be interested in this bug, as he has been working on similar stuff lately.

Thank you @denilsonsa, for the phenomenally thorough report!  I'll investigate in the near future.

@denilsonsa Thanks again for the report, and sorry I haven't gotten around to this yet - my wife and I were expecting a baby girl in September, and she decided to arrive a month early.  As a result, my time for vim-perl (and anything else for that matter) has become greatly diminished.  I'll get to this when I have a chance!

This issue is pretty complex.
Requires to rewrite rules for HEREDOC highlighting completely.
This may take 1-2 days, but I am also busy right now.

@zdm Thanks for the input!
I think the first step would be to add a test for this defect to the repository.


Is there a way to delay the initialization of sdk?

Recently in China, all apps need to pass the checking of the privacy policy, otherwise they will not be allowed to publish the app to the market. One of the core points of this checking is that in the case where the user does not have explicit consent (when the application is opened for the first time, a pop-up window is required to inform the user), the application is not allowed to obtain any device-related personal user information.

Well,  you can initialize it any time you want. Sentry recommends to do it on app startup, but nothing bad will happen if you start it later.
And since this postpone will be done only on the first run this is definitely not a problem IMO.

what @brustolin said, the only downside is that you won't be able to capture crashes before the SDK has been initialized.
you can also instead filter events out https://docs.sentry.io/platforms/flutter/configuration/filtering/ instead of delaying the init. of the SDK

any time

hello ，how any time to initialize sentry？


This commit implements the position interface of the robot using the sam...

...e joint impedance control strategy. That is, the robot behaves as a spring with stiffness K which is fixed for now.
This commit differentiates between joint positions controlled by effort (requires PID tunning) and controlled by means of joint stiffness (make the most of the impedance capabilities of the arm).
The stiffness value is set in the initialization of the joint_stiffness_command member here. I'm already using this in the multi-robot-test branch with MoveIt, and it is much more smooth.
For the moment, the stiffness value is hardcoded, but I plan to create a position controller for that, whence the _command, such that one can eventually control stiffness trajectories, and synchronize them with position trajectories.
I'm creating the pull request just to announce it, I will merge it in a couple of days.

Guys, I'm merging this.


Can't edit tag with unusual tag name

Bug Report
Expected behaviour
Tag editor should be able to edit tags named 1.0.3.dev--2 for a tool
Actual behaviour
"Save Changes" is disabled because 1.0.3.dev--2 does not match the regular expression /^[a-zA-Z0-9]+([-_.][a-zA-Z0-9]+)*$/i
Additionally, there is no message in the form to notify the regular expression is invalid.
Steps to reproduce the behaviour

Create a tool
Create a tag with the name 1.0.3.dev--2
Try to edit the tag and save changes

Environment (Browser or OS and Dockstore version)
DockstoreUI - 1.2.2
DockstoreApi - 1.2.1

Workaround:
use a tag name that matches the regular expression
Long-term: we should probably get a hotfix for this

@agduncan94  I think this might be a duplicate since you're already on the case

I am able to edit the tag.
But I notice if you click Add Tag and save, then click Add Tag again, the CWL and WDL test files are duplicated when the dialog starts up.

Seems like we might not be clearing the form fully after saving.

This seems resolved in staging.dockstore.org


Feature requests for enhanced nuxt generate options

In my current work around I am having a static website after running yarn build && yarn generate
in my <template> i have:
<nuxt-img class="gallery-item-image" style="background: url('/loader.svg')" :src="child.image.replace('/uploads/', '/')" image-style="small" :alt="child.title" />
in my <script> is:
created() {
      if (this.$store.state.menu) {
        for (let i = 0; i < this.$store.state.menu.length; ++i) {
          let menuParent = this.$store.state.menu[i].children
          for (let n = 0; n < menuParent.length; ++n) {

            if (menuParent[n].attributes.cover) {
              let attrs = menuParent[n]
              let img = menuParent[n].attributes.cover
              let cat = menuParent[n].attributes.gallery.gallery_cat
              let filename = img.substring(img.lastIndexOf('/') + 1);
              let menuThumb;

              console.log(filename)

              if (process.browser && !process.static) {
                menuThumb = '/' + filename + '?style=thumbnail'
              } else {
                menuThumb = '/image-styles/' + filename.substring(0, filename.lastIndexOf(".")) + "--thumbnail" + filename.substring(filename.lastIndexOf("."));
              }

              this.coverCats.indexOf(cat) > -1 ? null : this.coverCats.push(cat)

              this.coverItems.push({
                id: menuThumb.id,
                image: menuThumb,
                data: attrs
              })

              console.log(menuThumb)
            }
          }
        }
      }
    }

This an above a not well-working hack, how you can get it working for nuxt generate by changing around if (process.browser && !process.static) and run build and commit it to git.
... but it´s not 💯 nice ;)
So, my question is, is there any options to trigger the generation of images on nuxt build  or nuxt generate?
maybe by adding a post request to the URL?
or sth in store/index.js at nuxtServerInit?
as it looks now I have to visit the page to trigger generation.
btw thank you for the super docs and clear git repo :)
and maybe if I understand better the generate trigger I could try to help with this implementation.
Cheers

Hi @dev7ch
It looks like your workaround is doing some heavy lifting towards the image generation. Ideally, we'd get this working with nuxt-image-loader-module natively so you don't have to handle both the node server versus generate versions.
At the moment, I can only see part of your implementation so I can make a few guesses.
It's possible that the <nuxt-img> component is only picking up on the initial render and doesn't detect when the state changes.
For the node server (i.e. npm run build && npm run start) the images are not optimistically loaded ahead of time, which is why you'll see the serverMiddleware intercepts image requests on the server, detects whether the image exists at the path in your imagesBaseDir and whether there is work to do to process the image with graphicsmagick (i.e. if there isn't a cached version on disk) and then finally serve the image directly. This is done in the imageLoaderHandler function in index.js.
For npm run generate, we need to know ahead of time which images to process. When nuxt generate runs there is a global array set up on process.$imageLoaderRegistry. Then each time the <nuxt-img> is rendered with the render lifecycle hook the src is added (in the query format) to this 'registry'. At the end of nuxt's generate process, this module registers an event to fire on generate:done which takes all images in the registry and performs all the necessary image generations and places them in dist/image-styles/. Also the <nuxt-img> knows to point the resulting src to the statically generated file relative to the currently viewed page.
That's it, in a nutshell, however, I appreciate the two paths for static and dynamic generation are quite different, but necessarily so, I think.
In summary, I would be great if we can work together so I can first fully understand what's tripping up in your case especially as there are no tests yet for anything more complex than referencing images in the <nuxt-img> component. What would be great is to get a working repo up and running which replicates the bug you're experiencing with the minimal amount of implementation details. Then we can use this as a test case for this module alongside the existing ones.
Another thought I had is that it may not be possible with nuxt generate to always know ahead of time (during the component's render) which images will be loaded. In this case, I could add a feature where you could define a glob pattern in your nuxt config file to always process certain (or all) images for particular image styles. Just a thought, although this might be nice as a catch-all. I'm not sure if this is needed right now and it would be good to get to the bottom of the underlying issue first.
Would you be able to share more of the code which represents the problem? Are you working on a public repo or is this a private project? Feel free to email me: barry [dot] fisher [at] reallifedigital.com
Thanks for your input,
Barry

Thank you @barryrld  for the detailed answer.
Yes, the code is public and I very appreciate your open attitude.
https://github.com/dev7ch/eloisa-florido-navarro.com/tree/nuxt-img-loader
You can see a demo here: https://festive-hypatia-797f57.netlify.com/
I ve set up a bit "broken use case",  where locally and renderd all works fine.
But finally, i got it almost working.
Please see the Mainnav "Arbeiten" the submenu is a working example in the Sub Menu
In Line #66 is where <nuxt-img> is implemted.
The first 4 thumbnails are your module, the other ones a kind of hybrid with another module.
Is this good for you as a replica dev repo to work together?
Or should we start with a clean one?

Just to summarize your input up according to the usecas in the replica:
Status Quo

[x] ´´ component is only picking up on the initial render (static only?)
[x] ´process.$imageLoaderRegistry´defines whats is beeing generated (on generate:done)
[x] Then each time the  is rendered with the render lifecycle hook the src (universal or dev process?)

Did I understand and check the boxes above correctly?
.. the "?" in brackets mean I am not sure.
Workarounds/Requirements/Dreams:

[  ] Find all images and trigger local resizeCrop programmatically (catch all)
[x] Replace image source url on process.static
[x] Find a solution the keep images outisde from Repo
[  ] Fetching source images from outside to local and static Server (e.g media server, cdn)
[  ] Placeholder or Poster Property for component (and maybe fallback)
[  ] Rerender and update images if resizeCrop settings changed
[  ] Provide and carry Image Metadata (height, width, etc), store it somewhere
[  ] Option to enable pre-generation of all images for certain styles

Complex  Challenges

[  ] Create a file system map of all images on yarn build dynamic Img metadata
[  ] Allow style to be "null" or overwritten by v-bind and update path accordingly
[  ] Implement use of IntersectionObserver to handle big images and lazyLoading

... this came to my mind while dreaming about a perfect img loader, plz see this a brainstorming, not a tasklist :)

@barryrld  wrote: ... I could add a feature where you could define a glob pattern in your nuxt config file to always process certain (or all) images for particular image styles.

Yes, yes, yes 👍

... a catch-all. I'm not sure if this is needed right now and it would be good to get to the bottom of the underlying issue first.

That's was also my way of thinking, like "Doing the needed request on next build to all images"
I think it could be worth to consider adding this as an option for each style in nuxt.config.js
I hope this helps you and brings you fun.

Hi @dev7ch,
Thanks for the detailed list. There's a lot to think about here, so I'll curate this list into sub-feature requests at some time later.
For now, I wanted to let you know about a new feature in the latest release v1.2.0.
There is now the ability to force generate certain images style given a glob pattern for those instances where you need to dynamically load images asyncronously. There's more details in the updated docs:
https://github.com/reallifedigital/nuxt-image-loader-module#do-you-need-to-lazy-load-images-when-running-nuxt-generate
I hope this allow you to make so further progress in your use case for now. I haven't had a chance to review your project repo just yet. Before I do, I wonder if you'd be able to try out the latest version of this module and try the new option in nuxt.config.js to see if this solves anything for you.
Thanks,
Barry

@barryrld thanks a lot for adding the very useful forceGenerateImages  in nuxt.config.js

@barryrld sorry for my late response, i am on holidays but tested it quickly and it works fine so far. Do you want me to close this issue or keep it open for the future enhancement?
btw thanks for adding the credentials in the readme file :)

@dev7ch No problem!
I'll close this issue for now seen as the forceGenerateImages likely solve a lot of issues. I'd still like to revisit the possibility of pulling in external images in the future, however, this isn't a high priority for me at the moment.
Thanks, Barry


Support customizeRootView from RCTRootViewFactory

Summary:
The new customizeRootView does not have the feature parity as createRootViewWithBridge where reusing RCTRootViewFactory to create a root view, it does not call customizeRootView. This PR moves the customizeRootView support from RCTAppDelegate into RCTRootViewFactory and improves the customizeRootView support.
Changelog:
[IOS] [CHANGED] - Support customizeRootView from RCTRootViewFactory
Test Plan:
Add customizeRootView to packages/rn-tester/RNTester/AppDelegate.mm and test whether RNTester has blue background color in both new arch and old arch mode.
- (void)customizeRootView:(RCTRootView *)rootView
{
  rootView.backgroundColor = [UIColor blueColor];
}

Thanks for the PR, but I don't think we need this, @Kudo. We moved the customizeRootView method after the rootView is created, so that it is invoked independently from whether method is used to create the view.
Perhaps I'm missing something, but why we need to bring this customization into the RootView Factory?

@cipolleschi thanks for feedback. this is to resolve the issue when enable new architecture with react-native-bootsplash and expo-updates. there is a workaround patch in react-conf-app but i think it would be good to have the feature in RCTRootViewFactory, so that developers reuse RCTRootViewFactory can fully support customizeRootView.
we could apply the patch inside expo-modules-core if you think RCTRootViewFactory is not the right place.
more details: expo-updates will delay the timing to initialize react instance and create an empty root view rather than the RCTRootView. we need to show splash screen twice. one before empty root view and the other after we use RCTRootViewFactory to create the formal RCTRootView. react-native-bootsplash uses customizeRootView to setup and show splash screen twice. that's why we need  customizeRootView to be supported from RCTRootViewFactory (or our derived EXReactRootViewFactory)

@cipolleschi @Kudo Does a 0.74.x version with this fix is planned?

yes, it was already picked https://github.com/reactwg/react-native-releases/issues/312, would be available from next version


Prepare for Java 8 release

These are changes required to backport to Java 8 runtime. Java 8 releases will be kept in a separate branch, and will be released with slightly different artifact name.

please update branch protection rules


WebApp:azure-cli 2.24.0 az webapp config appsettings set Settings file with KeyVault references sets 'False' as value

Calling 'az webapp config appsettings set' at cli version 2.24.0, with a json setting file, containing key vault references sets 'False' instead of the reference value.
For the this file example...
[
  {
    "slotSetting": false,
    "value": "@Microsoft.KeyVault(SecretUri=https://KeyVaultName.vault.azure.net/secrets/SecretName)",
    "name": "TestSetting1"
  },
  {
    "slotSetting": false,
    "value": "TestValue2",
    "name": "TestSetting2"
  }
]

az webapp config appsettings set --name  --resource-group  --settings @JsonFileName.json
The call using 2.24.0 returns the following result:
[
 {
    "name": "TestSetting1",
    "slotSetting": false,
    "value": "False"
  },
  {
    "name": "TestSetting2",
    "slotSetting": false,
    "value": "TestValue2"
  }
]

Note the KeyVault reference is set to "False" as a string value, and not the expected reference string.
These same values are reflected within the app service configuration ui.
The call using 2.23.0 returns the following result:
[
 {
    "name": "TestSetting1",
    "slotSetting": false,
    "value": "@Microsoft.KeyVault(SecretUri=https://KeyVaultName.vault.azure.net/secrets/SecretName)"
  },
  {
    "name": "TestSetting2",
    "slotSetting": false,
    "value": "TestValue2"
  }
]

Works as expected in 2.23.0, which at the time of this writing was still available in Azure Cloud Shell, but using the cli in Azure Dev Ops pipelines has already been updated to 2.24.0 breaking for all pipelines requiring it.
Use of an actual key vault is not required for testing.

route to service team

Duplicate of #18266


FLEX-4472 ~ Uses variable exception instead of osgpException

SonarQube analysis reported 2 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AdHocManagementService.java#L76: Method has 7 parameters, which is greater than 6 authorized.
 AdHocManagementService.java#L134: Method has 7 parameters, which is greater than 6 authorized.

All is well.