Add support to sort_param and drop_param

In examples we typically pass -many style association as a list:
%{"name" => "john doe", "addresses" => [
  %{"street" => "somewhere", "country" => "brazil", "id" => 1},
  %{"street" => "elsewhere", "country" => "poland"},
]}

However, it is also common to pass the addresses as a map, where each
key is an integer representing its position:
%{"name" => "john doe", "addresses" => %{
  0 => %{"street" => "somewhere", "country" => "brazil", "id" => 1},
  1 => %{"street" => "elsewhere", "country" => "poland"}
}}

Using indexes becomes specially useful with two supporting options:
:sort_param (embeds only) and :drop_param. These options tell
the indexes should be reordered or deleted from the data. For example,
if you did:
cast_embed(changeset, :addresses,
  sort_param: :addresses_sort,
  drop_param: :addresses_drop)

You can now submit this:
%{"name" => "john doe", "addresses" => %{...}, "addresses_drop" => [0]}

And now the entry with index 0 will be dropped from the params before casting.
Note this requires setting the relevant :on_replace option on your
associations/embeds definition.
Similar, for sorting, you could do:
%{"name" => "john doe", "addresses" => %{...}, "addresses_sort" => [1, 0]}

And that will internally sort the elements so 1 comes before 0. Note that
any index not present in "addresses_sort" will come before any of the
sorted indexes. If an index is not found, an empty entry is added in its
place.
These parameters can be powerful in certain UIs as it allows you to decouple
the sorting and replacement of the data from its representation.

Would it be a good idea to have an option to raise on missing sort indexes? Just in case someone is making a mistake instead of intentionally trying to add blanks.

@greg-rychlewski good question. We are currently using missing entries as a mechanism to create new entries. We could add an option to either raise, create, or discard them, but there is the question of which one should be the default.
Discard them is certainly bad. There is an argument for raising at the cost of making it more bureaucratic. Creating a new one will be empty and most likely lead to validation errors, but can be surprising.
Another option is to raise but have a special value to indicate you want to create new entries, such as "new". However, because the indexes can be anything, there is a chance of conflict at least in theory, so we would need to make this opt-in (which perhaps is fine), something like: sort_param: "addresses_sort", add_empty_entries_on: "new". We likely need a better name though.
Thoughts?

I like the one option that is either raise/discard/create, like you said. The special value to me seems like a bit too much work for not much gain. And introduces some new ways to make mistakes.
I would personally default to raise but if a large % of the people who would use this would mostly want to create I can see the case for that. I am not too much into UI work so I'm mostly concerned about introducing mistakes. It might be that no one using this for UI wants to raise. For sure I wouldn't discard it by default, 100% agree.

I like this idea of "declarative" editing for one-to-many relationships. I'm not sure I like the asymmetry between using *_drop for dropping elements, but using _sort for creating elements.
Symmetrical would be using just _sort and any missing indexes are dropped and any additional indexes have an record created. Not sure if that's a sound idea.
The other symmetrical approach would be _drop and _add, but that's less flexible in terms of where new records should be sorted into the existing ones.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:

@LostKobrakai the rationale for being distinct is that you may want to expose different capabilities. Some UIs may only allow sorting. Others only deletions. And so forth.

@josevalim That's a totally fine approach hence my two alternate suggestions. My concern was more about how sorting and creation are bundled into one capability, while deletion is separate. I don't know how to do improve on that front though without the downsides mentioned before.

@LostKobrakai perhaps it is a matter of documentation but adding is not bundled in sorting. You can "append" by simply submitting something like parent[child][new][_]. What addition on sort_param allow is arbitrary additions between any children but you need sorting to be able to that anyway. :D


get_gradients() is empty

When extracting the gradient values in the call method of the GradCam class:
self.extractor.get_gradients()[-1].cpu().data.numpy()

the get_cradients() call, returns an empty list, and therefore the [-1] produces an IndexError exception.
The only different thing that I am doing is to use a custom ResNet18 model to work with grayscale images that do not follow the standard (224, 224) size. Has anyone faced this problem before?

Mea culpa. I loaded by error a smaller ResNet model and therefore, the target layer 2 did not exist and I was getting an empty list when trying to extract the corresponding gradients.


Investigate MP3 Compression

Closing for now, as CDN will deal with compression on way out. This is not a priority at this time.


Need better description for "Failed to create a milestone" scenarios

Testing #3764
Try to create an existing closed milestone such as "Backlog"
The creation will fail and there is a notification saying "Failed to create a milestone".
GitHub shows a "Title has already been taken" error message for this case

Interestingly, the title too long case does append the explanation to the notification text

Validation error messages are wrongly parsed and can lead to not displaying them at all.


[v1] Container-related checks

Add E003 (misplaced .row), and remove E004 (nestable containers) since that works now.
Status

Code: Assumed to be done
Tests: Assumed to be done
Wiki: To be done

Oops, some of the tests use the old column classes, now I am in a quandary, fixing this would make this PR depended on #420.

Oops, some of the tests use the old column classes, now I am in a quandary, fixing this would make this PR depended on #420.

Rebased it now with the new column classes.

/cc @XhmikosR @Johann-S


The PHPReflectionUtilities library's API has changed. Refactor source code and tests for the MockClassInstance to accommodate this change.

The PHPReflectionUtilities library's API has changed.
A \Darling\PHPReflectionUtilities\classes\utilities\Reflection's __construct() method now expects an instance of a \Darling\PHPTextTypes\classes\strings\ClassString instead of an instance of a \ReflectionClass.
Refactor source code and tests for the MockClassInstance to accommodate this change.

resolved in 3194d60c8f714af8a4ca6edf2283dddcf91f6683

resolved in 3194d60c8f714af8a4ca6edf2283dddcf91f6683


[#137] FreeBSD support

Now that containerd supports FreeBSD, we can port nerdctl, too!
:rocket:. For full-fledged functionality, we will need to port

buildkit. Status: POC is ready, need to upstream fixes to
dependencies.
runtime + containerd shim. runj/knast should work just fine.
CNI bridge plugin for networking.

This change fixes FreeBSD compilation errors by renaming linux files
to unix where necessary, or otherwise introducing FreeBSD versions for
the necessary files.
Signed-off-by: Artem Khramov<EMAIL_ADDRESS>

Thanks for working on this.
Could you update make artifacts to cross-compile GOOS=freebsd GOARCH=amd64 binary on GitHub Actions?
https://github.com/containerd/nerdctl/blob/95dd996155be515ef1f8a69cd3e19d9f27118880/Makefile#L83-L84
Also please add docs/freebsd.md to clarify the installation steps, usage, and the known limitations.
Eventually we should have CI, but it can be another PR if it is difficult.
We can use Cirrus CI to run FreeBSD virtual machines. https://cirrus-ci.org/guide/FreeBSD/

Probably we need to update https://github.com/containerd/nerdctl/blob/95dd996155be515ef1f8a69cd3e19d9f27118880/cmd/nerdctl/run_runtime.go#L44 too, if we want to support wtf.sbk.runj.v1

cc @samuelkarp

@AkihiroSuda thanks for the thorough review.
I'm going to focus on resolving the crosscompilation issue.
Also I would like to hear your input on doc -- I wasn't sure what to put there.

9s
Run vagrant up
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Box 'freebsd/FreeBSD-13.0-STABLE' could not be found. Attempting to find and install...
    default: Box Provider: virtualbox
    default: Box Version: >= 0
==> default: Loading metadata for box 'freebsd/FreeBSD-13.0-STABLE'
    default: URL: https://vagrantcloud.com/freebsd/FreeBSD-13.0-STABLE
==> default: Adding box 'freebsd/FreeBSD-13.0-STABLE' (v2021.09.16) for provider: virtualbox
    default: Downloading: https://vagrantcloud.com/freebsd/boxes/FreeBSD-13.0-STABLE/versions/2021.09.16/providers/virtualbox.box

Progress: 0% (Rate: 0*/s, Estimated time remaining: --:--:--)
Progress: 100% (Rate: 635/s, Estimated time remaining: --:--:--)
An error occurred while downloading the remote file. The error
message, if any, is reproduced below. Please fix this error and try
again.

vagrant up seems extremely flaky.
Could you mirror the image to somewhere more stable, or could you use Cirrus instead of Vagrant?

@AkihiroSuda indeed it's flaky. I'm sorry for the notifications mess I caused. I mitigated the problem using the cache action.
Let's test that, otherwise I'll set up a Cirrus pipeline.

CI seems flaky.
Can be fixed later, though
time="2021-09-21T14:19:05Z" level=fatal msg="failed to create shim: failed to mount rootfs component &{nullfs /var/lib/containerd/io.containerd.snapshotter.v1.native/snapshots/3 [rw]}: mount [[-o rw -t nullfs /var/lib/containerd/io.containerd.snapshotter.v1.native/snapshots/3 /run/containerd/io.containerd.runtime.v2.task/default/70886b2a11483b463360ab4f419294075698a64754c02bc1dda57fc642e60ba9/rootfs]] failed: \"mount_nullfs: /run/containerd/io.containerd.runtime.v2.task/default/70886b2a11483b463360ab4f419294075698a64754c02bc1dda57fc642e60ba9/rootfs: Resource deadlock avoided\\n\": exit status 1: unknown"

Hi Team,
Any update on the FreeBSD support on ContainerD/nerdctl?

@leandroscardua
There is a known issue https://github.com/containerd/nerdctl/issues/868 and WIP PR to resolve it: https://github.com/moby/sys/pull/114 .
Hopefully nerdctl on FreeBSD can work smoothly when the issue is resolved.

Thank you for the information @gmshake


Remove style and styleModule from package.json

Follow-up to #14
See https://github.com/telamonian/theme-darcula/pull/33 and https://github.com/jupyterlab/jupyterlab/pull/10381 for more info.

Looks good on the Binder for this branch:
https://mybinder.org/v2/gh/jtpio/jupyterlab_miami_nights/style-package-json?urlpath=lab
https://user-images.githubusercontent.com/591645/121717506-8acbda80-cae1-11eb-859e-a60601d1a852.mp4


Requesting support for DISTINCTCOUNT on BYTES typed columns

Currently DISTINCTCOUNT on a column with the BYTES type does not work.  Our usage of BYTES columns is to represent fields that were originally in bytes form and were ByteBuffer before being serialized into a Pinot column. Would you suggest to convert these to a hex-encoded string? The error returned is:
org.apache.pinot.client.PinotClientException: Query had processing exceptions: [{"errorCode":200,"message":"QueryExecutionError:
java.lang.IllegalStateException: Illegal data type for DISTINCT_COUNT aggregation function: BYTES
  at org.apache.pinot.core.query.aggregation.function.DistinctCountAggregationFunction.aggregate(DistinctCountAggregationFunction.java:91)
  at org.apache.pinot.core.query.aggregation.DefaultAggregationExecutor.aggregate(DefaultAggregationExecutor.java:94)
  at org.apache.pinot.core.operator.query.AggregationOperator.getNextBlock(AggregationOperator.java:65)
  at org.apache.pinot.core.operator.query.AggregationOperator.getNextBlock(AggregationOperator.java:35)
  at org.apache.pinot.core.operator.BaseOperator.nextBlock(BaseOperator.java:49)
  at

cc @kishoreg

It's very straight forward to add BYTES support for DISTINCTCOUNT. Will submit a PR soon.

Cool. Thanks @Jackie-Jiang

#5701

Thanks @Jackie-Jiang


error loading file - "Not a diagram file" when I tried to open it, file corrupted after sudden shut-down

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

You must agree to search and the code of conduct. You must fill in this entire template. If you delete part/all or miss parts out your issue will be closed.
If you are technical, you should reporting bugs along the lines of https://marker.io/blog/how-to-write-bug-report. If you are not technical, we will make allowances, please try to make an effort to understand the process.
Describe the bug
Error upon loading drawio file - "Not a diagram file" when I tried to open it, file corrupted after sudden shut-down
To Reproduce
Steps to reproduce the behavior:

Go to 'Diagrams.net'
Click on 'open existing diagram' from local device
Scroll down to '....'
See error "Not a diagram file"

Expected behavior
Should open existing diagram
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 24.5.2

Desktop (please complete the following information):

OS: Windows 10
Browser [firefox]
Browser Version [126.0.1]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

I tested the problem in incognito/private mode with all browser extensions switched off, write "yes" below:
Additional context
Diagram file corrupted upon sudden shut down, opening the source , there are only 'NULL's inside, still it is not empty, file size 3.4 MB.

Is there a .bkp file in the same folder?

Not at all, checked ..Unfortunately windows file history is disabled by group policy.  Sent from Mail for Windows From: David BensonSent: Friday, June 7, 2024 17:47To: jgraph/drawioCc: djt-djt; AuthorSubject: Re: [jgraph/drawio] error loading file - "Not a diagram file" when I tried to open it, file corrupted after sudden shut-down (Issue #4462) Is there a .bkp file in the same folder?—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
ok, thanks ..
if there is no chance to repair , will close the case.

Unfortunately, not. Sorry for the inconvenience.


metatables: don't handle by default. add API to toggle it. Fix #21.

Coverage remained the same at 100.0% when pulling 324b7fbd59407121a36a8319887b882e1524276e on adriweb:toggle_metatable into 66042a5b09186ac2fbd8337479c66b6d2e3adcc1 on gvx:master.


fix condition preventing multiple default device-classes

When there are multiple volume groups and device-classes and more than 1 node then "!=1" condition is triggering false error on secondary and next nodes.

I can't understand your case. Could you tell me more details? Example configurations could help me.

I am using LVMCluster on 3-node Openshift. I have 3-volume groups: each vg for each node.
config below:
apiVersion: lvm.topolvm.io/v1alpha1
kind: LVMCluster
metadata:
finalizers:
- lvmcluster.topolvm.io
name: lvmcluster
namespace: openshift-storage
spec:
storage:
deviceClasses:
- default: true
deviceSelector:
paths:
- /dev/disk/by-id/scsi-3300xxxxabcde
fstype: xfs
name: vg1
nodeSelector:
nodeSelectorTerms:
- matchExpressions:
- key: topology.topolvm.io/node
operator: In
values:
- n1.openshift.fqdn
thinPoolConfig:
name: thin-pool-1
overprovisionRatio: 10
sizePercent: 90
- default: false
deviceSelector:
paths:
- /dev/disk/by-id/ata-TOSHIBA_MQ0abcdef
fstype: xfs
name: vg2
nodeSelector:
nodeSelectorTerms:
- matchExpressions:
- key: topology.topolvm.io/node
operator: In
values:
- n2.openshift.fqdn
thinPoolConfig:
name: thin-pool-1
overprovisionRatio: 10
sizePercent: 90
- default: false
deviceSelector:
paths:
- /dev/disk/by-id/ata-TOSHIBA_MQ0xxxyaT
fstype: xfs
name: vg3
nodeSelector:
nodeSelectorTerms:
- matchExpressions:
- key: topology.topolvm.io/node
operator: In
values:
- n3.openshift.fqdn
thinPoolConfig:
name: thin-pool-1
overprovisionRatio: 10
sizePercent: 90
When cluster is built - on nodes n2 & n3 topolvm-node pods are failing with lvmd  "should have only one default device-class" error.
When I change lvmd.yaml configs on those nodes setting their volumegroups "default"  to "true" - then topolvm-node pods are starting correctly.
Since then I can provision PVs on nodes n2 & n3 when I specify appropriate storage-class/device-class pointing to volume groups vg2 & vg3.
regards
Przemek Dmochowski

Hi @elproko
The default device-class is used when no specific device-classes are specified in the StorageClasses. As such, we cannot permit multiple default device-classes. Consequently, we are unable to accept your PR.
By the way, I presume you are using the lvm-operator, which is not a part of the TopoLVM project. Could you kindly report your issue directly to their project? Thank you.

Hi @elproko dev from the lvm-operator here, please close this issue and move it to OCPBUGS under the LVMS component.
That being said: For LVMCluster with 3 deviceClasses on 3 nodes you currently need to specify the paths as optionalPaths. Also one of the nodes has to act as default provisioner.
For more information see the docs:

To control or restrict the volume group to your preferred devices, you can manually specify the local paths of the devices in the deviceSelector section of the LVMCluster YAML. The paths section refers to devices the LVMCluster adds, which means those paths must exist. The optionalPaths section refers to devices the LVMCluster might add. You must specify at least one of paths or optionalPaths when specifying the deviceSelector section. If you specify paths, it is not mandatory to specify optionalPaths. If you specify optionalPaths, it is not mandatory to specify paths but at least one optional path must be present on the node. If you do not specify any paths, it will add all unused devices on the node.

That being said, multi-node LVMS on OpenShift is not supported in 4.14 or below officially.
@toshipp we should really consider how we deal with us sharing the same API group as you guys. I don't know when that was decided but it regularly leads to you having to deal with support requests from LVMS, which is not nice.  Please feel free to close the issue.

@jakobmoellerdev
Thank you for your comment.

@toshipp I think we have misunderstanding here.
My goal is: to use topolvm on more than single node with only single default device-class.
My previous explanation was only to show why lvmd is complaining with error when in reality there was no default class on the specific node. Setting "default" on nodes n2 & n3 was only a workaround against wrong constructed condition.
Again - currently error is triggering in 2 cases:

when there are more defauld device classes than one.
when there is no default device class on specific node.

By this PR I wanted to cancel second triggering case.
Kindly please consider this case again.

Hi @elproko The default device-class is used when no specific device-classes are specified in the StorageClasses. As such, we cannot permit multiple default device-classes. Consequently, we are unable to accept your PR.
By the way, I presume you are using the lvm-operator, which is not a part of the TopoLVM project. Could you kindly report your issue directly to their project? Thank you.

I created bug as You suggested - thank You.
Again - to clarify: this PR was not to use multiple default device-classes but to fix wrong condition which triggers false-positive when there is no default device-class on specific node.

Hi @elproko please find my answer on how you can solve your original issue (for lvm operator) in https://github.com/openshift/lvm-operator/pull/483#issuecomment-1806846303
That being said, I do understand your concern, and I think the misunderstanding is that lvmd.conf should be aligned on all nodes. If you do not do that (and configure lvmd differently on multiple nodes) you will have issues like above. From what I understand you attempted to configure lvmd.conf on multiple nodes and only one of the nodes contained a default deviceClass while the other node didn't. This is not intended. Instead lvmd should be deployed with a single config that is the same on every node.
@toshipp I think the use case of @elproko is valid. Should we think about allowing differing lvmd.conf files for individual nodes? Then the validation change that is suggested actually makes sense.

@elproko
I misunderstood your case.

My goal is: to use topolvm on more than single node with only single default device-class.

Did you mean that your issue is that you want to enable the default device-class on only a single node?
If so, this breaks our assumption that the default device-class is always used, which may cause another issue.
For example, this code returns a pointer to the default device class structure. When your change is applied, this will return nil, then the process will crash due to the nil dereference.
https://github.com/topolvm/topolvm/blob/main/lvmd/device_class_manager.go#L180
I will ask our team if we can accept relaxing the current limit.

@elproko I misunderstood your case.

My goal is: to use topolvm on more than single node with only single default device-class.

[...]
I will ask our team if we can accept relaxing the current limit.

Thank you very much for reconsidering and going deeper into this.

When your change is applied, this will return nil, then the process will crash due to the nil dereference.

After all it seems I had luck with my dirty hack :)
I am glad because I hope it can be changed with best results.
regards
Przemek

Hi @elproko
I discussed this with our team and concluded that allowing no default device classes was reasonable.
To merge this PR, we will solve two major problems I found.

The DeviceClass function should return ErrNotFound because the callers dereference the returned pointer even if it is nil.
The checkFunc should be changed not to depend on the default device class. This RPC call is intended to check the connection is open, it is better to add a ping RPC and use it instead, IMO.

Hi @elproko I discussed this with our team and concluded that allowing no default device classes was reasonable. To merge this PR, we will solve two major problems I found.

The DeviceClass function should return ErrNotFound because the callers dereference the returned pointer even if it is nil.
The checkFunc should be changed not to depend on the default device class. This RPC call is intended to check the connection is open, it is better to add a ping RPC and use it instead, IMO.

Please check and update the PR.

Sorry for delayed answer.
I agree with you about those two code places to fix, but I am not able to do that myself. I hope you could do that.

@elproko
Ok. I'll take over this PR. It might take some time due to our resources. Please be patient.


How to select a tree in double tree?

Hi there,
From #47
<x-settings.nav-link href="{{ route('bookings.stats.by.pet') }}">Speaks</x-settings.nav-link>
____________________________^

The _^ is the cursor position.
Using:
local current_node = ts_utils.get_node_at_cursor()
print(func:type())

prints attribute_value in:

Question: how to use the blade tree instead of html one?

I am not sure why why that is not working out, it seems to be working for me in Helix. The only issue I have is the Tag Name #62 , which I am yet to tackle, I have some ideas on how to fix it though ;)

The html is an injected language---whatever api you are using should allow you to get the uninjected vs injected tree

@calebdw indeed, ignore_injected_langs did the trick


[Question] Stepper Motor Smooth Controller for More Than One Channel  (IDFGH-8670)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
I modified the rmt step_motor example for more than one channel.
By repeating the following sequence,
step_motor_create_rmt
step_motor_init
step_motor_step
step_motor_deinit
4 motors were able to be driven one by one.
However, move than one motor cannot be driven simultaneously:
for (int i = 0; i < 4; i ++) {
step_motor_create_rmt, RMT_CHANNEL_0 + i, gpiopin+i,
step_motor_init(motor[i])
}
...
wait for some commands
for (int i = 0; i < 4; i ++) {
step_motor_step(motor[i])
}
...
for (int i = 0; i < 4; i ++) {
step_motor_deinit(motor[i])
}
...
I found inside step_motor_create_rmt function,
rmt_register_tx_end_callback
may damage other rmt channels, cause the problem.
Any idea?

Hi @homerliu1 I read the callback function in the example, seems all function calls are operating on per-motor's RMT channel. I guess it's because when multiple channels issue the interrupt almost at the same time, and in the interrupt handler, we always process the RMT channels in the same order (from channel 0->n).
BTW, can you try the example from the master branch or release/v5.0? this example has been cleaned up and uses the new RMT API, maybe the new example can offer the ability to drive multiple-step motors.

Hi morris, I tried the master branch example according to your suggestion.
xTaskCreate was used to generate 4 tasks, each one represents for one rmt motor channel.
It is working. Thank you for your support!


Fixing bugs in regular expressions and code logic (DEV)

Closes https://cccs.atlassian.net/browse/AL-1245

Codecov Report

Merging #269 (69df790) into dev (da73b0e) will decrease coverage by 0.21%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev     #269      +/-   ##
==========================================
- Coverage   67.03%   66.81%   -0.22%
==========================================
  Files         119      119
  Lines       10419    10362      -57
==========================================
- Hits         6984     6923      -61
- Misses       3435     3439       +4

Impacted Files
Coverage Δ

assemblyline/common/constants.py
87.50% <ø> (ø)

assemblyline/common/identify.py
90.74% <100.00%> (+2.46%)
:arrow_up:

assemblyline/common/backupmanager.py
69.70% <0.00%> (-5.81%)
:arrow_down:

assemblyline/filestore/transport/sftp.py
26.60% <0.00%> (-5.02%)
:arrow_down:

assemblyline/filestore/transport/ftp.py
18.47% <0.00%> (-3.96%)
:arrow_down:

assemblyline/datasource/alert.py
88.46% <0.00%> (-3.85%)
:arrow_down:

assemblyline/common/heuristics.py
82.81% <0.00%> (-1.57%)
:arrow_down:

assemblyline/datastore/__init__.py
62.60% <0.00%> (-1.47%)
:arrow_down:

assemblyline/datastore/helper.py
31.26% <0.00%> (-1.13%)
:arrow_down:

assemblyline/datasource/common.py
70.37% <0.00%> (-1.06%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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


Empty state components don't scale well for small widgets

I love the idea of the empty state panels. I switched some of our components to the new one that also displays an example for the annotation to add. That works well on fullscreen pages and maybe on bigger widgets. But on our small widgets that looks a bit odd :laughing:

Btw, there is a seconds row of widgets that the user can't see on the screen without scrolling right now.
I would expect the component to somehow fit into smaller widgets.
Any suggestions how that empty state component can be improved to better fit into small widgets?
Not directly related, but maybe we can make the example content a bit smaller by skipping fields that aren't important. I would also like if I could specify an example value for the annotation. Maybe we should also make that component a bit more entity kind independent (right now everything is targeted at components).

@katz95 wanna chime in?

@stefanalund @katz95 Me and @samiramkr can take a look at this shortly.

Here are some thoughts by @freben https://github.com/backstage/backstage/pull/3479#issuecomment-734920731

This seems to relate to #3217 as well. I've experimented with not using the default widget cards directly on the overview page. When it detects the plugin isn't applicable it displays a custom message instead which has a link to the full tab which has the full empty state component. This works around the scaling issue. But ideally, there was some built in support for both handling different views and/or customizing the empty state in an easier way.

Still something I would love to see. If we have a suggestion for a different design / user flow, I would be happy to implement it.

I would love to see this as well. A few thoughts:

It's a bit odd the empty state is not displayed inside a card. That means they stick out from the rest of the widgets.
The yaml empty state takes up a lot of space and doesn't scale well. Perhaps a help button with a popup or expandable section could be used instead. Either permanently, at certain size, or via a prop configuration.
Ideally the empty state can be customized or overridden as needed #3217

I'll reiterate here that I think that the root problem is that these empty states are just too large / noisy to begin with. I would prefer if they were very small and un-dramatic. Like a small text saying No <thing> found. [more info] or something like that, and you can click on the more info link to know what it's about and how to change that fact.

@freben I think I design a less noisy yet really efficient empty state for the page & also make it responsive for all screen/widget size

I would love to get the issue assigned to me @freben @backstage-service

@Sonofigma, sure, thank you! 👍

Okay, will be update the issue soon with a solution


Dat Web API v2

Closes https://github.com/beakerbrowser/beaker/issues/157

Methods implemented and tests are passing. Merging.


Allow auth server bind address to be changed

adds convar ns_server_bind_address (which defaults to <IP_ADDRESS>), allowing the server host to choose which local ip the server binds itself to.
Likely allows fixes for some issues with setups using multiple network adapters.

Shouldn't this ideally synchronize to the bind address of the game's sockets? (I think it's controllable with ip cvar or -ip command line option, would need to look into whether the cvar is synchronized from the option and at which stage)

https://developer.valvesoftware.com/wiki/Command_Line_Options, -ip is the option

On the valve site it says

-ip
Does the same as the ipname Cvar

So would it make sense to use the ipname Cvar to change the auth server bind address?


Normalized output for B.D’Orbigny should not add a space after D'

"authorship": {
    "verbatim": "B.D’Orbigny",
    "normalized": "B. D' Orbigny",
    "authors": [
      "B. D' Orbigny"
    ],
    "originalAuth": {
      "authors": [
        "B. D' Orbigny"
      ]
    }
  },

As a related issue, I will fix Acteonina d'Orbigny 1849

Cannot reproduce, html entities are already taken care of


[Port] Bump to langpack versions (#17910)

An issue was identified with the langpack versions not being updated, without this change the generated langpacks will not be useable at release
This PR fixes #17909

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 41.717%

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
27295

Relevant Lines:
59373

💛  - Coveralls


PBS video says: "Unable to extract video ID; please report this issue on https://yt-dl.org/bug"

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.06.04. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.06.04

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones This link is broken, it lists recent issues instead of giving a way to search for issues. The current URL seems to be to just use the issues page in the format https://github.com/rg3/youtube-dl/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+search+term...
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
kyan@elegiac ~/ytdled-2018june09 nohup compiz --replace openbox $ youtube-dl -i --geo-bypass --write-description --write-info-json --write-annotations --write-thumbnail --write-all-thumbnails --write-sub --write-auto-sub --all-subs -k --verbose 'https://player.pbs.org/partnerplayer/tOz9tM5ljOXQqIIWke53UA==/'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-i', u'--geo-bypass', u'--write-description', u'--write-info-json', u'--write-annotations', u'--write-thumbnail', u'--write-all-thumbnails', u'--write-sub', u'--write-auto-sub', u'--all-subs', u'-k', u'--verbose', u'https://player.pbs.org/partnerplayer/tOz9tM5ljOXQqIIWke53UA==/']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.06.04
[debug] Python version 2.7.14 (CPython) -<EMAIL_ADDRESS>[debug] exe versions: ffmpeg 3.4.2, ffprobe 3.4.2, rtmpdump 2.4
[debug] Proxy map: {}
[debug] Using fake IP <IP_ADDRESS> (US) as X-Forwarded-For.
[pbs] Downloading JSON metadata
[pbs] tOz9tM5ljOXQqIIWke53UA==: Downloading player page
ERROR: Unable to extract video ID; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/lib64/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 792, in extract_info
    ie_result = ie.extract(url)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/common.py", line 500, in extract
    ie_result = self._real_extract(url)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/pbs.py", line 493, in _real_extract
    video_id, display_id, upload_date, description = self._extract_webpage(url)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/pbs.py", line 485, in _extract_webpage
    r'<div\s+id="video_([0-9]+)"', player_page, 'video ID')
  File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/common.py", line 880, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract video ID; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
Downloading this item fails with 'Unable to extract video ID; please report this issue on https://yt-dl.org/bug'. I'm using a version from git earlier today, rather than tagged release, for what it's worth. Using Gentoo GNU/Linux. Here is the video link for test in Web browser: https://player.pbs.org/partnerplayer/tOz9tM5ljOXQqIIWke53UA==/

Sweet, tested and it works now! Thank you @Urgau and @dstftw ! :)


Unexpected input(s) 'registry-url'

I saw there registry-url in here
but when I try with a workflow like this:
jobs:
  publish-npm:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
         uses: actions/checkout@v3
         with:
           fetch-depth: 0

      - name: Setup environment with volta
         uses: volta-cli/action@v1
         with:
           registry-url: https://registry.npmjs.org/

      - name: Publish Package
         run: npm publish --access public
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

I get this warning:
Warning: Unexpected input(s) 'registry-url', valid inputs are ['volta-version', 'node-version', 'npm-version', 'yarn-version']
Run volta-cli/action@v1
downloading<EMAIL_ADDRESS>
then I get this error:
npm ERR! code E404
npm ERR! 404 Not Found - PUT https://registry.npmjs.org/@logustra%2fvountdown - Not found
npm ERR! 404
npm ERR! 404  '<EMAIL_ADDRESS>is not in the npm registry.

The issue was that those changes weren't published just yet, if you update to v2 or higher you should be good to go...


The project page shows an old version number, 1.2.6, rather than the current version, 1.5.2

The top left corner of the project page shows 1.2.6, but this repository has 1.5.2.

Updated this thanks @michaellenaghan ! 👍


Is this a build pack issue

Hi,
When using the latest java buildpack, I am having a strange behaviour.
My stream is the following:
trigger --fixed-delay=10 | uaa-get-token: httpclient --logging.level.org=DEBUG --url-expression='''https://xx.yyy.io/oauth/token''' --headers-expression={'Authorization':'Basic dGltZXNlcmllcy1jbGllbnQ6dtrgegegrtgreM=','Content-Type':'application/x-www-form-urlencoded'} --http-method=POST --body="client_id=gsgre_client&grant_type=client_credentials" | log
I can create the stream but when I am depoying, I have some strange message:
2017-05-29T14:05:17.01+0200 [STG/0] ERR /bin/bash: line 7: ${vcap.application.instance_index}: bad substitution
2017-05-29T14:05:17.01+0200 [STG/0] ERR /bin/bash: line 8: {"spring.cloud.dataflow.stream.app.label":"uaa-get-token","spring.cloud.stream.metrics.properties":"spring.application.name,spring.application.index,spring.cloud.application.*,spring.cloud.dataflow.*","logging.level.org":"DEBUG","spring.cloud.dataflow.stream.name":"STREAM_1","httpclient.headers-expression":"{'Authorization':'Basic dGltZXNlcmllcy1jfdsdsfafwQDU1d29yZF8xMjM=','Content-Type':'application/x-www-form-urlencoded'}","httpclient.http-method":"POST","spring.metrics.export.triggers.application.includes":"integration**","spring.cloud.stream.metrics.key":"STREAM_1.uaa-get-token.${spring.cloud.application.guid}","spring.cloud.stream.bindings.input.group":"STREAM_1","spring.cloud.stream.bindings.output.producer.requiredGroups":"STREAM_1","httpclient.url-expression":"'https://xx.yyy.io/oauth/token'","httpclient.body":"client_id=timeseries-client&grant_type=client_credentials","spring.cloud.stream.bindings.output.destination":"STREAM_1.uaa-get-token","spring.cloud.dataflow.stream.app.type":"processor","spring.cloud.stream.bindings.input.destination":"STREAM_1.trigger"}: bad substitution
2017-05-29T14:05:17.01+0200 [STG/0] ERR /bin/bash: line 10: ${vcap.application.name}:${vcap.application.instance_index}: bad substitution
2017-05-29T14:05:17.15+0200 [STG/0] ERR Cloning into '/tmp/buildpacks/java-buildpack'...

The app is starting anyway, then I have :
2017-05-29T14:05:51.25+0200 [App/0] OUT 2017-05-29 12:05:51.255  INFO 29 --- [           main] o.apache.catalina.core.StandardService   : Stopping service Tomcat
2017-05-29T14:05:51.31+0200 [App/0] OUT 2017-05-29 12:05:51.318  INFO 29 --- [           main] utoConfigurationReportLoggingInitializer :
2017-05-29T14:05:51.31+0200 [App/0] OUT Error starting ApplicationContext.
Thanks in advance
Regards

Hi,
It doesn't seem like a build pack issue. Looks like the url-expression you pass is probably not set correctly.

@ilayaperumalg thanks for the quick reply, I am using the exact same stream expression in local with docker and it's working.

@BLaurent do you have any WARN or ERROR messages in the log about properties configuration (especially from PropertiesConfigurationFactory)?

@ilayaperumalg
Here is the WARN and ERR I have
2017-05-29T16:27:56.19+0200 [App/0] OUT 2017-05-29 14:27:56.197  WARN 29 --- [           main] c.c.c.ConfigServicePropertySourceLocator : Could not locate PropertySource: I/O error on GET request for "http://localhost:8888/dataflow-server-STREAM_1-uaa-get-token/cloud": Connection refused (Connection refused); nested exception is java.net.ConnectException: Connection refused (Connection refused)
...
2017-05-29T16:27:57.17+0200 [App/0] OUT 2017-05-29 14:27:57.170  WARN 29 --- [           main] o.s.a.r.c.CachingConnectionFactory       : ***
...
2017-05-29T16:27:59.07+0200 [App/0] OUT 2017-05-29 14:27:59.075  WARN 29 --- [           main] ationConfigEmbeddedWebApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.springframework.cloud.stream.app.httpclient.processor.HttpclientProcessorConfiguration$HttpClientProcessor': Unsatisfied dependency expressed through field 'properties'; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'httpclient-org.springframework.cloud.stream.app.httpclient.processor.HttpclientProcessorProperties': Could not bind properties to HttpclientProcessorProperties (prefix=httpclient, ignoreInvalidFields=false, ignoreUnknownFields=true, ignoreNestedProperties=false); nested exception is org.springframework.validation.BindException: org.springframework.boot.bind.RelaxedDataBinder$RelaxedBeanPropertyBindingResult: 1 errors
2017-05-29T16:27:59.07+0200 [App/0] OUT Field error in object 'httpclient' on field 'exactlyOneUrl': rejected value [false]; codes [AssertTrue.httpclient.exactlyOneUrl,AssertTrue.exactlyOneUrl,AssertTrue.boolean,AssertTrue]; arguments [org.springframework.context.support.DefaultMessageSourceResolvable: codes [httpclient.exactlyOneUrl,exactlyOneUrl]; arguments []; default message [exactlyOneUrl]]; default message [Exactly one of 'url' or 'urlExpression' is required]
2017-05-29T16:27:59.13+0200 [App/0] OUT Error starting ApplicationContext.

I am using :
java-offline-buildpack       16         true      false    java-buildpack-offline-v3.9.zip
cf api
api version:    2.62.0

Thanks! Since this is an app specific problem (not JBP or CF-server) and given there's already a similar thread at spring-cloud-stream-app-starters/httpclient#3, I will close this and share the findings there instead.


glfwGetX11Display error

Hi, I have installed glfw3.2.1. When I  make, there are errors:
glfw-occt/GlfwOcctWindow.cpp:58:66: error: conversion from ‘Display* {aka _XDisplay*}’ to ‘const TCollection_AsciiString’ is ambiguous
myDisplay = new Aspect_DisplayConnection (glfwGetX11Display());
Can someone help me with that. Thanks a lot.

I find this works, just delete glfwGetX11Display() as follows:
(I am using ubuntu18.04)
#if !defined(_WIN32) && !defined(__APPLE__)
  /* myDisplay = new Aspect_DisplayConnection (glfwGetX11Display()); */
  myDisplay = new Aspect_DisplayConnection();
#endif

Hi, I have installed glfw3.2.1. When I make, there are errors:
glfw-occt/GlfwOcctWindow.cpp:58:66: error: conversion from ‘Display* {aka _XDisplay*}’ to ‘const TCollection_AsciiString’ is ambiguous
myDisplay = new Aspect_DisplayConnection (glfwGetX11Display());
Can someone help me with that. Thanks a lot.

Which OCCT version do you use for building?

Hi, I have installed glfw3.2.1. When I make, there are errors:
glfw-occt/GlfwOcctWindow.cpp:58:66: error: conversion from ‘Display* {aka _XDisplay*}’ to ‘const TCollection_AsciiString’ is ambiguous
myDisplay = new Aspect_DisplayConnection (glfwGetX11Display());
Can someone help me with that. Thanks a lot.

Which OCCT version do you use for building?, Aspect_DisplayConnection (glfwGetX11Display()) working on OCCT7.4

Thanks. I was using occt7.3


isPathInside is reversed?

The docs say the first parameter is containingPath and the second parameter is path, so I assumed that you pass the directory as the first parameter and something inside that directory as the second.
Nope. They are completely reversed.

You should absolutely swap the names or change them to something else.

Yeah, that's the part I want to carefully gauge. I want to see what circumstances even led to such a disparity.


Callback when the dialog has been dismissed

As Ordinary Dialog does, DialogPlus seems like to have the listener that notifies the user when DialogPlus dismisses. mind if I push a pull request for this issue and review for my code? thanks.

@doyonghoon Of course you can, It will be good to have it.

@orhanobut thanks for quick reply. I just tried to push a pull request for this issue, I encountered an error. here is the message.
fatal: unable to access 'https://github.com/orhanobut/dialogplus.git/': The requested URL returned error: 403
To solve this problem, i think that you may add me as a contributor or something in this project.

To contribute, you need to fork the project and then you can push it to the your origin. then you can create a pull request from that point.

aha, got it. thanks.

solved issue.


Fix 'event.keyIdentifier' depreciation in favor of 'event.key'

There was a warning for a change due in October, this should fix it
https://www.chromestatus.com/features/5316065118650368

just realized I have quite a few PR pending, I'll try to wait for you guys to process these before adding more lol


Encrypt backups?

How hard would it be to add the ability to encrypt backups? I am thinking that we could just override the "command" for the database provider and add  | gpg ... or whatever, but it would be nice if it was "supported" directly.
Tommy

That's a good idea. And you're right, you need to override command (and filename) if you want to do that now.
Some notes to myself:

Use gpg's own compression with --compress-algo (ZIP by default)
Use gpg --yes --batch --passphrase=[passphrase] for non-interactive processing

Alternatively, if you use gpg to encrypt with a "public key" you would not need a password at all, and as an added bonus, you would be able to ensure that no one could retrieve/read the backups unless they had the private key (pass phrase for the private key), where as the "shared key" model you have "noted" above would have the "key" in a config file on the server.
~tommy

Yes, it's a trade-off between convenience and security - public key is definitely better for security but it adds operational complexity, so I'd opt for non-interactive / SSH-less workflow. Maybe it's possible to support both, though.


[Question] Rancher HA cluster load balancing

I have created a Kubernetes cluster on 3 Ubuntu 16.04 machines using RKE as described in the high availability guide here (https://rancher.com/docs/rancher/v2.x/en/installation/ha/)
I have a front end load balancer on a machine (e.g. my.host.com) routing to the Kubernetes cluster on my three nodes. This routes all traffic on port 443 to the cluster on port 443 of each cluster node. NGinx then routed the request to Rancher which is registered with the ingress for any traffic with the hostname provided on install (i.e. my.host.com).
My issue however is that I want to run a Traefik ingress controller in my Kubernetes cluster on a specific node port (e.g. 30000) and then I want my front end load balancer to send all traffic to Traefik and not NGinx and route to my services running in the cluster registering with the Ingress at different urls.
Some examples of my ideal routing would be:

my.host.com/example1 = load balancer -> traefik -> example1Service
my.host.com/example2 = load balancer -> traefik -> example2Service
my.host.com/rancher = load balancer -> traefik/nginx -> Rancher UI

My questions:

The HA rancher documentation uses the Rancher helm chart to deploy Rancher. Is there a way to change its ingress configuration to not use the hostname but instead a registered route e.g. /rancher?

If this is not possible, what other solutions can I do to be able to route to both my services and the rancher UI from a single load balancer?

The cluster with 3 nodes created by RKE for Rancher is meant for running Rancher only. If you want to run your workloads, please create a new cluster from within Rancher (https://rancher.com/docs/rancher/v2.x/en/cluster-provisioning/#cluster-creation-options). In this newly created cluster you can then either disable the built-in Ingress controller (https://rancher.com/docs/rke/v0.1.x/en/config-options/add-ons/ingress-controllers/#disabling-the-default-ingress-controller via https://rancher.com/docs/rancher/v2.x/en/cluster-provisioning/rke-clusters/options/#config-file) and launch your own, or configure the built-in controller to point to your controller or basically however you want to configure it.

Oh wow... So if I run a single instance of Rancher using the docker run command and install some catalog apps, If i lost the machine with Rancher on but create a new rancher container on a new machine. Will the new Rancher know about my catalog apps installed or was my data on the original machine and therefore lost?
Is there a way to deploy Rancher to the same cluster as my workloads?

Keeping state with single install is described here: https://rancher.com/docs/rancher/v2.x/en/installation/single-node/#persistent-data. You will need to move the data manually to another host (or have some other sync mechanism). Single install doesn't give you a cluster to run your workloads, you still need to create a cluster so that won't help you. (except maybe https://rancher.com/docs/rancher/v2.x/en/installation/single-node/#running-rancher-rancher-and-rancher-rancher-agent-on-the-same-node)
You can run workloads in the cluster for Rancher but it's not recommended/supported, as your workloads can take down Rancher, and there are some other technical challenges that arise you decide to do so.

Thanks for the quick response.
Can I use the RKE cluster for my workloads without putting Rancher on it? I like the setup scenario of creating a cluster using this tool.

RKE was made to easily setup Kubernetes clusters, if you want to use just that, that's fine.


Created a CldPitcure Component as suggested in isssue description

Created a CldPitcure Component as suggested in isssue description, Fixes #160

quick thought on implementation here that I'd love to get both of your thoughts on
a lot of patterns I'm seeing in different frameworks and with Nuxt itself is not to generate  tags from a top level prop, rather allow someone to define those as they wish from within the component, just as you would with native HTML
For instance:

Nuxt Picture appears to be a drop-in replacement - it doesn't offer a prop for sources, and I assume the expectation is to pass your own source? https://image.nuxt.com/usage/nuxt-picture also worth skimming this https://github.com/nuxt/image/issues/309
Next.js exposes helper functions instead of components, i think components can be a bit cleaner, but worth pointing out, as they still use: https://nextjs.org/blog/next-13-5#nextimage-improvements
Unpic is actually not creating a Picture tag and only creates Source tags: https://dev.to/ascorbic/level-up-your-web-images-with-unpic-picture-tags-4jh1

so with that in mind, and not from a sense of "everyone else is doing it", but I'm wondering if creating an API that's closer to HTML standards, but also seems to be the pattern Nuxt is following as well (correct me if i'm wrong, im skimming docs/google here)
perhaps something like what was in the original issue would make more sense?
<CldPicture>
    <CldSource
        srcset={[{ src: 'cc0-images/surfer', width: 240, height: 200 }]}
        media="(min-width: 800px)"
    />
    <CldImg width="298" height="332" src="cc0-images/painted-hand" alt="" />
</CldPicture>

Your comment @colbyfayock is really reasonable.
But at this point I wonder what the user will actually get when he will need to use CldPicture and CldSource that works almost the same as normal html tags.
This is a code that must be maintained while it does not provide any significant value as it can be 1:1 replaced with natvie html.
Dont get me wrong. We can try with the implementation of it as it was innthe original description but it could not be as useful as CldImage, CldVideoPlayer, CldOgImage, etc

ill defer to your judgement but i would expect the biggest reason is to remain closer to HTML standards for the picture API. it also might give additional flexibility in how they compose the source? i'm not sure


Environment
Docker
Version
?
Describe the problem
so im not sure were to put this i have never used github before so please forgive me. but i'm having a installer issue trying to install homarr in my lab it keeps saying that im missing the line /data in my compose file and it gives me the porper compose file and i run it and it still gives me this error. im running docker desktop on windows 10 with portainer installed i even have a 2nd instance of docker on ubuntu and i still face the same issue and have zero idea what to do
Logs
No response
Context
No response
Please tick the boxes

[X] I confirm that I attached the proper logs
[X] I've read the docs
[X] I've checked for duplicate issues
[X] I've tried to debug myself

the following above is what it keeps telling me

Can you share what your docker compose looks like?
(remove things that you don't want to share if there's anything of the sort)

Hey @aspieboy it should work and you should be able to just click on continue. This page is shown to all users and was used to describe the changes users had to do when we released v14 with authentication. So it just work without any changes to your docker-compose in portainer.

Related to #1766

Closing this issue. This page should no longer appear on new installations. Fixed in 0.14.3. Please reopen if you still have questions or problems.


Citation information

In GitLab by @wtbarnes on Sep 16, 2020, 21:25
Now that aiapy is on Zenodo, we need to add some information in the docs (and the README) about how to cite aiapy. We should also include a CITATION.RST, similar to the one that sunpy uses. Also like sunpy, aiapy.__citation__ should return the BibTeX info for the citation.

In GitLab by @PaulJWright on Sep 22, 2020, 15:16
I can do this in the coming days.

In GitLab by @wtbarnes on Sep 22, 2020, 15:57
Great! We will probably have to update this once !79 is resolved and the JOSS paper is published, but I see no issue with just proceeding with the Zenodo information for now as people may want to cite aiapy in the interim.

In GitLab by @wtbarnes on Sep 28, 2020, 10:15
Closed by !82


HttpSessionListener#sessionDestoyed isn't invoked (Tomcat 8)

Description
I have two nodes:

node 1 -> N1
node 2 -> N2

Expected behaviour
sessionDestroyed event must be handled by both nodes
Actual behaviour
sessionDestroyed event handled only by N1
Steps to reproduce or test case
WHEN client log in N1
THEN N1 and N2 receive sessionCreated event
AND service invalidate session (on N1)
THEN sessionDestroyed event handle only N1
Redis version
Redis server v=3.2.6 sha=00000000:0 malloc=jemalloc-4.0.3 bits=64 build=6135068212e7fc88
Redisson version
3.10.7
Redisson configuration
redisson.conf
{
	"singleServerConfig": {
		"idleConnectionTimeout": 10000,
		"pingTimeout": 1000,
		"connectTimeout": 10000,
		"timeout": 3000,
		"retryAttempts": 3,
		"retryInterval": 1500,
		"password": null,
		"subscriptionsPerConnection": 5,
		"clientName": null,
		"address": "redis://some-ip",
		"subscriptionConnectionMinimumIdleSize": 1,
		"subscriptionConnectionPoolSize": 50,
		"connectionMinimumIdleSize": 32,
		"connectionPoolSize": 64,
		"database": 0,
		"dnsMonitoringInterval": 5000
	},
	"threads": 0,
	"nettyThreads": 0,
	"codec": {
		"class": "org.redisson.codec.SerializationCodec"
	},
	"transportMode": "NIO"
}

context.xml
<Manager className="org.redisson.tomcat.RedissonSessionManager"
          configPath="${catalina.base}/conf/redisson.conf" readMode="MEMORY" updateMode="DEFAULT" broadcastSessionEvents="true"/>

Fixed! Thanks for report

@mrniko thanks, we wait for the final version for the check.

@mrniko hi! We tried to use 3.11.0 version and creation and destroying events got both nodes. But when we tried to invalidate session then the session wasn't removed from redis.

And it still available by session id

I looked at your code and as I understood session will be removed only after 1 minute.
I think this is to make the session accessible to other nodes, right?

@sibmaks
That's correct. This would allow you to have an access to it in HttpSessionListener#sessionDestoyed listener

@mrniko I rechecked it. And the session wasn't removed from redis after 5 minutes.

The session was removed only after an hour. I think it's too long

I found that's the problem. After session invalidate we request the new session:
session.invalidate();
newSession = request.getSession();

But we got the old session

@mrniko
I think that this looks like a bug. Because if I invalidate session on the node then I expect get new a new session. What do you think?

I confirm this behaviour. When I invalidate the session, with setup of tomcat session manager same as above, I got the same session, with same attributes. I think, the map with values is populated again from Redis.

@sibmaks
I confirm this issue.

@sibmaks
Here is updated version containing fix of the issue. Could you try it and give your feedback?
redisson-tomcat-8-3.11.1-SNAPSHOT.jar.zip

@mrniko
I tried it and it is not working. Same situation. If i set it without broadcastSessionEvents, it is ok. If set to true then it is not working as expected.
I call:
a) session.getAttribue("abcd") => Object@123 - values - {a: 1, b: 2}
b) session.invalidate();
c) session.getAttributeNames() -> size : 0
d) session.getAttribue("abcd") => Object@456 - values - {a: 1, b: 2} but now, it should return null according to my opinion

@JiriFrankCZ
Did you wait for 60 seconds since session invalidation? 60 is enought time so every HttpSessionListener#sessionDestoyed listener could pickup session and handle it

@mrniko
This is single node run. One tomcat one redis. The sequence of scenario above is executed directly one after other, no wait. There is usecase that, I have something in session, I invalidate it but I have it still in session when i call getAttribute

@mrniko
This is single node run. One tomcat one redis. The sequence of scenario above is executed directly one after other, no wait. There is usecase that, I have something in session, I invalidate it but I have it still in session when i call getAttribute

@JiriFrankCZ
Thanks for your feedback!
Here is yet another updated version:
redisson-tomcat-8-3.11.1-SNAPSHOT.jar.zip

On simple setup looks good! :)

@JiriFrankCZ
Thanks for checking! Fixed


docs: fix missing next auth url

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

clean work


Apply graphql-tag/loader on graphql/gql files

We need to apply this loader rather than use the loader alias workaround
because the loader relies on the ability to recursively load more graphql
files when importing fragments. This still works with the gql macro approach
and that can remain the recommended approach.

!merge


[RFC]: Add @stdlib/random/strided/rayleigh

Description
This RFC proposes adding support for filling a strided array with pseudorandom numbers drawn from a Rayleigh distribution. This is the strided interface analog of @stdlib/random/base/rayleigh.
Package: @stdlib/random/strided/rayleigh
Alias: rayleigh
Related Issues
None.
Questions
No.
Other

A package with a similar implementation/API design is @stdlib/random/strided/exponential.

Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

can you assign me this work?

I am a newbie and honestly I did not get my task. what am I supposed to do?


Pricing tab on a new item creation /app/item

Objection💬:

Users must be able to set standard item price (price list) when creating a new item.
For sale price, users need to save item creation first before setting the sale price.

--
Check list✔:

Hide the "On Sale" option box when creating a new item.
Hide "Step1 - 2" when creating a new item.
Add text below price field. This text will display when creating a new item only.

It should look like⤵

1. Hide this after saving.

2. This message shown after saving new item creation.
-> happened this only if I added price in pricing tab.

3. This feature doesn't work on a new item creation
-> need to press 'Save' button to create item price.

Nabeel and Umer said it possible to create pricing rule on a new item creation.
So, Users can set normal price and sale price when crating a new item’.
@Muzmal  After you fixed issue above (item price). Can you move to sale price on a new item creation please? If you struggle, please contact @deviljutt. 🙏

New issue: Faced warning message after changing item name.🚨

How to reproduce an issue

Go to any item and double click at item name.

Change item name

You will see warning message

Can create Item price & Sale price together with creating a new item.
Can change item name without any warning.


draft diagram for our solution

According to Colorado, most people will read the paper; they will glance at the first figure in the paper. We need to come up with some explainer diagrams and share with the group for feedback.
Based on the discussion in the team meeting (03/13 8:00 am), first we need to come up with a rough sketch of the current architecture.  No need to focus on graphic creation.

Draft to be confirmed

Good illustrations in https://generallyintelligent.ai/understanding-self-supervised-contrastive-learning.html

Is it called "squeeze layer" for both? In one convolution layer, we are making it 3 from 2 channels.

Main visualizations for the paper: https://docs.google.com/presentation/d/17CQJykk605wl_nkwAuWmFnTjzKO689GbVS2H2yma9eQ/edit#slide=id.gc554c97402_0_19

called squeeze layer is because both are cross channel pool and after that it's an output of 3 so a 3x3 also applied could called it expansion layer. im not sure if we need to explicit 3x3 in the diagram after 1x1 conv, or just call the conv layer as dimension filter. can double check with colorado.

Main visualizations for the paper: https://docs.google.com/presentation/d/17CQJykk605wl_nkwAuWmFnTjzKO689GbVS2H2yma9eQ/edit#slide=id.gc554c97402_0_19

can we make the input image multiple layers? 2 and 10?

yea that's a good idea, like showing the 2 vs 10 channels

Updated the graphics with channel information.


Update UTransport interface specification

Changed registerListener method signature to support
matching on message source and sink addresses by means
of UUri patterns.
This change allows UTransport to be used by higher level
APIs for implementing all message exchange patterns
supported by uProtocol.
Fixes #73

@stevenhartley @PLeVasseur I have pushed changes and have amended your comments ...

Question/comment. Can/should we also align the error messages returned as well? Good that we're aligning the UCodes, but would ask if it's worth it to align the UStatus messages as well between implementations. Benefit would be consistency, but not sure if it's too much micromanagement.

Question/comment. Can/should we also align the error messages returned as well? Good that we're aligning the UCodes, but would ask if it's worth it to align the UStatus messages as well between implementations. Benefit would be consistency, but not sure if it's too much micromanagement.

FMPOV relying on error message format/content is an antipattern that leads to code that is hard to maintain and will produce hard to find bugs during runtime. So, no, I do not think that we should standardize error messages ...

@stevenhartley @PLeVasseur I have pushed changes as discussed in the call.
Would you mind taking another look?
@tamarafischer I have tried to add some wording to the expected outcome of the send() method. WDYT?

Would you mind taking another look?

Changes to the section we discussed on the call look good to me!

@stevenhartley I have accepted your proposed changes but in order to merge the PR, GitHub still requires your explicit approval ...


Enhanced 2D Shapes Calculator and Color Code Calculator

Fixes Issue🛠

Closes #458
Description👨💻

The 2D Shapes and color code calculator have now got a new, improved and more visually attractive UI. The headings have animations, there are stylish buttons which have shadow effect on hovering and also have an outline border when they are in focus. The font family, font size, and font colors have also been changed accordingly.
Type of change📄

[x] New feature (non-breaking change which adds functionality)

How this has been tested✅

Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[X] I have added demonstration in the form of GIF/video file
[X] I am an Open Source Contributor

Screenshots/GIF📷
https://github.com/Rakesh9100/CalcDiverse/assets/114403339/d358b330-ba42-4c82-9da9-6d96ae924b8b
https://github.com/Rakesh9100/CalcDiverse/assets/114403339/22cce3f5-aa88-486c-8adb-d883cdc2db97

Pls delete your images from folder assets/images and keep those background images in the respective Calculator folder and then link and use it @Hansawani

Okay I will do the requested changes asap.
On Wed, 31 Jan, 2024, 19:44 Rakesh Roshan, @.***> wrote:

Pls delete your images from folder assets/images and keep those
background images in the respective Calculator folder and then link and use
it @Hansawani https://github.com/Hansawani
—
Reply to this email directly, view it on GitHub
https://github.com/Rakesh9100/CalcDiverse/pull/503#issuecomment-1919187548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3I2QCY2ARI2DTAC7CXX6YDYRJGVHAVCNFSM6AAAAABCN56E22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJZGE4DONJUHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Please check now. I have done the changes that you requested.
On Wed, 31 Jan, 2024, 19:44 Rakesh Roshan, @.***> wrote:

Pls delete your images from folder assets/images and keep those
background images in the respective Calculator folder and then link and use
it @Hansawani https://github.com/Hansawani
—
Reply to this email directly, view it on GitHub
https://github.com/Rakesh9100/CalcDiverse/pull/503#issuecomment-1919187548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3I2QCY2ARI2DTAC7CXX6YDYRJGVHAVCNFSM6AAAAABCN56E22VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJZGE4DONJUHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Nice UI, Merged!! Congrats on getting your PR merged to this repo under SWOC S4 🎉🎉 @Hansawani

Please update the leaderboard🙏🏻😄
My name is Hansawani Saini.
On Thu, 1 Feb, 2024, 00:41 Rakesh Roshan, @.***> wrote:

Merged #503 https://github.com/Rakesh9100/CalcDiverse/pull/503 into
main.
—
Reply to this email directly, view it on GitHub
https://github.com/Rakesh9100/CalcDiverse/pull/503#event-11660201991,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3I2QC2EJE6OTEH5OMXZWVTYRKJPPAVCNFSM6AAAAABCN56E22VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJRGY3DAMRQGE4TSMI
.
You are receiving this because you were assigned.Message ID:
@.***>


notification index.js  is missing

notification index.js  is missing

Please fix it

have you fix it

yes, check attached file
index.js.zip

Sir when image is send it will be displayed but
when many message is send when I scroll up few message will b disappeared
and some times image will not be visible can you please help me
On Tue, Nov 12, 2019, 2:47 PM said Erraoudy<EMAIL_ADDRESS>wrote:

yes, check attached file
index.js.zip
https://github.com/TheHasnatBD/uMe/files/3835280/index.js.zip
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TheHasnatBD/uMe/issues/3?email_source=notifications&email_token=ANXNRH67E4BWGCRVRWUWRPDQTJYD5A5CNFSM4GWKBMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDZSGVI#issuecomment-552805205,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANXNRH3MAN77OPQL7S7NAGLQTJYD5ANCNFSM4GWKBMAQ
.

yes, check attached file
index.js.zip

Where attach it please help me

yes, check attached file
index.js.zip

Sir when image is send it will be displayed but
when many message is send when I scroll up few message will b disappeared
and some times image will not be visible can you please help me its compulsory

Thanks


[Bug]  The request text in the log is always none

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.
[X] 3. Please note that if the bug-related issue you submitted lacks corresponding environment info and a minimal reproducible demo, it will be challenging for us to reproduce and resolve the issue, reducing the likelihood of receiving feedback.
[X] 4. If the issue you raised is not a bug but a question, please raise a discussion at https://github.com/sgl-project/sglang/discussions/new/choose Otherwise, it will be closed.
[X] 5. Please use English, otherwise it will be closed.

Describe the bug
I checked the docker logs and tried to find the request text in the logs, but the logs showed text=None, but input_ids was returned. I want it to display the request text directly. What parameters should I add when starting it?
docker Logs：
in=GenerateReqInput(text=None, input_ids=[151644, 8948, 198, 2610, 525, 264, 10950, 17847, 13, 151645, 198, 151644, 872, 198, 108386, 151645, 198, 151644, 77091, 198], image_data=None, sampling_params={'temperature': 0.0, 'max_new_tokens': None, 'min_new_tokens': 0, 'stop': [], 'stop_token_ids': [], 'top_p': 1.0, 'presence_penalty': 0.0, 'frequency_penalty': 0.0, 'repetition_penalty': 1.0, 'regex': None, 'n': 1}, rid='640143780ce94c81a56689020f8a5b09', return_logprob=False, logprob_start_len=-1, top_logprobs_num=0, return_text_in_logprobs=True, stream=False, modalities=[], is_single=True, lora_path=None), out={'text': '你好，有什么我可以帮助你的吗？', 'meta_info': {'prompt_tokens': 20, 'completion_tokens': 9, 'completion_tokens_wo_jump_forward': 9, 'finish_reason': {'type': 'stop', 'matched': 151645}, 'id': '640143780ce94c81a56689020f8a5b09'}, 'index': 0}
INFO:     <IP_ADDRESS>:59398 - "POST /v1/chat/completions HTTP/1.1" 200 OK
Reproduction
docker run：
docker run -itd --name n72 --runtime nvidia --gpus '"device=0,1,2,6"'
-p 1090:30000
-v /mnt/data1/home/fusion_large:/32k
--ipc=host
lmsysorg/sglang:latest
python3 -m sglang.launch_server --model-path /32k --host <IP_ADDRESS> --port 30000  --dtype bfloat16 --tensor-parallel-size 4   --served-model-name cosmic-32k --log-requests
Environment
Image：
sglang v0.3.0
OS：
cat /etc/os-release
NAME="TencentOS Server"
VERSION="2.4"
ID="tencentos"
ID_LIKE="rhel fedora centos tlinux"
VERSION_ID="2.4"
PRETTY_NAME="TencentOS Server 2.4"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:tencentos:tencentos:2"
HOME_URL="https://cloud.tencent.com/product/ts"

Now the openai api applies the chat template and tokenizes it at the same time to avoid some double BOS issues.
https://github.com/sgl-project/sglang/blob/23cc66f7b65f885969d4608fd4964e0ba98fb7f5/python/sglang/srt/openai_api/adapter.py#L868
So the server does not see the input text. If you want this feature, can you help us support it? You can add some logging in the openai api server to print the raw input if a flag is set.


fix: dont show Controller component when first frame is not ready

fix: don't show Controller component when the first frame is not ready
Description
If the user executes seek action when the fist frame is not ready, will throw some error.
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

Codecov Report

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

@@          Coverage Diff           @@
##           master     #74   +/-   ##
======================================
  Coverage    51.5%   51.5%
======================================
  Files         123     123
  Lines        1722    1722
======================================
  Hits          887     887
  Misses        835     835

Impacted Files
Coverage Δ

packages/griffith/src/components/Player/Player.js
0% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Syft finds no apks for some images with apks

Please provide a set of steps on how to reproduce the issue
syft -q registry:cgr.dev/chainguard/sdk@sha256:871b75ddd7b91a29ea0ed8695e271f59056c21af6a2f937f9224e81d6030dbec | grep 'apk\s'

What happened:
No apk type packages are found by Syft for this image.
What you expected to happen:
Since the image's /lib/apk/db/installed does contain several package entries, Syft should have reported those packages.
Anything else we need to know?:
I did a small amount of debugging. It seems like Syft does find the installed db and starts to find packages, but then hits a scanner error (ErrTooLong) and so it returns a nil slice of packages.
Package entries in the apk db can be pretty large. I think Syft's current parsing approach isn't able to handle these larger entries, because the entry's byte count exceeds the scanner's buffer size.
This may be solvable by increasing the size of the buffer, but it may be better to try an approach that doesn't require the scanner to hold an entire package entry in the buffer at once. I haven't thought through this idea, but would be happy to chat through it on this issue if that helps!

We definitely should log issues instead of completely failing to return any packages when we hit an error like this. Also -- as you say, for this specific one, we could probably bump a buffer size somewhere.


Flutter tool does not handle failure of file or directory deletion well

Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter build bundle --suppress-analytics --target E:\flutter_project\flutter_palyday\lib\main.dart --filesystem-scheme org-dartlang-root --track-widget-creation --compilation-trace-file compilation.txt --depfile E:\flutter_project\flutter_palyday\build\app\intermediates\flutter\debug/snapshot_blob.bin.d --asset-dir E:\flutter_project\flutter_palyday\build\app\intermediates\flutter\debug/flutter_assets --debug
exception
FileSystemException: FileSystemException: Exists failed, path = 'E:\flutter_project\flutter_palyday\build\app\intermediates\flutter\debug/flutter_assets' (OS Error: 拒绝访问。
, errno = 5)
#0      _Directory.existsSync (dart:io/directory_impl.dart:99:7)
#1      _Directory.createSync (dart:io/directory_impl.dart:131:11)
#2      _LocalDirectory&LocalFileSystemEntity&ForwardingDirectory.createSync (package:file/src/forwarding/forwarding_directory.dart:22:16)
#3      writeBundle (package:flutter_tools/src/bundle.dart:230:13)
<asynchronous suspension>
#4      assemble (package:flutter_tools/src/bundle.dart:220:9)
<asynchronous suspension>
#5      build (package:flutter_tools/src/bundle.dart:153:9)
<asynchronous suspension>
#6      BuildBundleCommand.runCommand (package:flutter_tools/src/commands/build_bundle.dart:75:11)
<asynchronous suspension>
#7      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:549:18)
<asynchronous suspension>
#8      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:486:33)
<asynchronous suspension>
#9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#10     _rootRun (dart:async/zone.dart:1124:13)
#11     _CustomZone.run (dart:async/zone.dart:1021:19)
#12     _runZoned (dart:async/zone.dart:1516:10)
#13     runZoned (dart:async/zone.dart:1463:12)
#14     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#15     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:477:20)
#16     CommandRunner.runCommand (package:args/command_runner.dart:196:27)
<asynchronous suspension>
#17     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:396:21)
<asynchronous suspension>
#18     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#19     _rootRun (dart:async/zone.dart:1124:13)
#20     _CustomZone.run (dart:async/zone.dart:1021:19)
#21     _runZoned (dart:async/zone.dart:1516:10)
#22     runZoned (dart:async/zone.dart:1463:12)
#23     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#24     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:356:19)
<asynchronous suspension>
#25     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:111:29)
#26     new Future.sync (dart:async/future.dart:224:31)
#27     CommandRunner.run (package:args/command_runner.dart:111:11)
#28     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:242:18)
#29     run.<anonymous closure> (package:flutter_tools/runner.dart:60:20)
<asynchronous suspension>
#30     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#31     _rootRun (dart:async/zone.dart:1124:13)
#32     _CustomZone.run (dart:async/zone.dart:1021:19)
#33     _runZoned (dart:async/zone.dart:1516:10)
#34     runZoned (dart:async/zone.dart:1463:12)
#35     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#36     runInContext (package:flutter_tools/src/context_runner.dart:51:24)
<asynchronous suspension>
#37     run (package:flutter_tools/runner.dart:51:10)
#38     main (package:flutter_tools/executable.dart:59:9)
<asynchronous suspension>
#39     main (file:///C:/Users/Administrator/flutter/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#40     _startIsolate.<anonymous closure> (dart:isolate/runtime/lib/isolate_patch.dart:298:32)
#41     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/lib/isolate_patch.dart:171:12)

flutter doctor
[✓] Flutter (Channel beta, v1.3.8, on Microsoft Windows [Version 6.1.7601], locale zh-CN)
    • Flutter version 1.3.8 at C:\Users\Administrator\flutter
    • Framework revision e5b1ed7a7f (2 weeks ago), 2019-03-06 14:23:37 -0800
    • Engine revision f4951df193
    • Dart version 2.2.1 (build 2.2.1-dev.0.0 571ea80e11)

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at D:/tool/android-sdk-studio/android-sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = D:/tool/android-sdk-studio/android-sdk
    • Java binary at: E:\studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Android Studio (version 3.3)
    • Android Studio at E:\studio
    • Flutter plugin version 33.4.1
    • Dart plugin version 182.5215
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[!] IntelliJ IDEA Ultimate Edition (version 2018.3)
    • IntelliJ at D:\tool\IntelliJ IDEA 2018.3
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins

[✓] Connected device (1 available)
    • OPPO R11 Plus • <IP_ADDRESS>:21503 • android-x86 • Android 5.1.1 (API 22)

! Doctor found issues in 2 categories.

Duplicate of https://github.com/flutter/flutter/issues/23519.


Adjust speed of animation

Is there a way to adjust the speed of the animation? I ask in order to use it as a loading panel while loading some data. That data takes between 200 and 500ms to load and will be loaded before the animation finishes, which means the animation slows the page down.
I tried using .finish(true), but that has the consequence of only showing a short flicker if the loading time is too short.

The animations are all defined in the CSS, so it should be relatively easy to write some rules in your stylesheet to override them. For example, this rule defines the speed that the inner element fade in, etc.

Let me know if you have any issues with that, but tweaking the CSS should give you what you need!


Backport #794 to v0.1

:clipboard: #794
Overview
This backports our #794 Sentry environment changes to the v0.1 branch. We already had Sentry on this branch so this should only need the configurations backported.
Changes

Adds current Sentry configurations.

Testing
N/A

Closing this PR due to staleness and move to v0.2.


Frameworks Don't Track All Hyper-Parameters

Pytorch-Lightning modules are supposed to be self-contained units that manage all hyper-parameters?
The frameworks currently take in two arguments that break this requirement.

make_optimiser_fn
make_model_fn

One possible solution might be to stop extending pytorch-lightning modulein base framework and rather leave this as an implementation detail for the researcher.

See: #19 for proposed changes

Fixed for optimiser in new development versions.
See #19 for changes

Model params are still not tracked by default


Json import does not work

I'm having problems when importing this json:
{"version":"0.1.1","name":"scum2","faction":"scum","pilots":[{"name":"serissu","ship":"m3ainterceptor","upgrades":{"title":["heavyscykinterceptor"],"torpedo":["flechettetorpedoes"]}},{"name":"bobafett","ship":"firespray31","upgrades":{"title":["slavei"],"torpedo":["protontorpedoes"]}},{"name":"kathscarlet","ship":"firespray31"}],"description":"","points":105,"vendor":{"xwsd":{"builder":"X-Wing Squads Designer","builder-link":"https://play.google.com/store/apps/details?id=es.dmb.xws"}}}
Ships and pilots are correctly created but no upgrade is loaded.
Greetings!
David

I think this works now (probably just forgot to close it).  Reopen if it's still an issue.


popup is not working in chrome browser thought its working fine in firefox and IE

Does this happen to anymore else?
I tested with http://kelp404.github.io/angular-form-builder/ and it is the same

Sometimes it happens to me too... And it lasts until a full reboot, even if I close Google Chrome it doesn't work after opening again.
As a workaround, I'm trying to place a button to manually open the popup just in case. If anyone knows how to do it, I would be glad to know!

I think I have found the bug. Google Chrome fires both mousemouve AND mouseclick when clicking something. When this happens, the popover doesnt open because it thinks we are trying to move a field.

I have solved this bug by storing the mouseX and Y, comparing it every time the mouse move around.
Add the following variables after "this.mouseMoved = false;":
this.mx=0;
this.my=0;
And modify the mousemove handler as below:
$(document).on('mousemove', function(e) {
if(e.pageX == _this.mx && e.pageY == _this.my)
return;
_this.mx = e.pageX;
_this.my = e.pageY;
var func, key, _ref;
_this.mouseMoved = true;
_ref = _this.hooks.move;
for (key in _ref) {
func = _ref[key];
func(e);
}
});

thanks pesootti. Your solution works!

Thanks Pessoti. It works.

+1

Thanks @Pessotti. It really works.


DatePicker component

I'm submitting a ...

[ ] feature request
DatePicker component, any plan to add a datepicker component?

I will need to have DatePicker Component in my project (sometimes) soon, after I finish it I will be glad to integrate it.

+1

+1, duplicates #837


Figure out a way to keep the session alive

I've been doing an experimental test to see if I'm able to keep the session alive by simply making requests to the GetConsumptionData API endpoint. Unfortunately it looks like that  alone is not sufficient to keep the session alive.
Last night I refreshed the session at 20.45 in the evening and scheduled a data fetch to take place every 2 hours starting at 22:00.
20.45: OK (session established)
22:00: OK
00:00: OK
02:00: OK
04:00: OK
06:00: HTTP 401, session expired

Hi,
I investigated the Datahub web UI cookie behavior by making repetitive requests to GetConsumptionData API by changing the date range in the UI.
It looks like Datahub updates two cookies in every response: TS012fcd69 and XSRF-TOKEN. The next request obviously includes these updated values from the previous response.
I'll make a new experiment so that these two cookie values are updated in the session.txt after each response and make a new "endurance test" next night. Let's see how that works...

Unfortunately my experiments do not look promising. Updating the two cookies mentioned above (which Fingrid updates with every response) does not help to keep the session alive. The authentication token expires consistently after 8 hours based on my experiments and I'm quite sure this invalidation happens on the server side.
This means that for my use case this is a big bummer because my idea was to authenticate once and then use the session for scheduled consumption data retrievals.
I'm planning to contact Fingrid customer service about this. They already have the API endpoints. The only thing that is missing is that users could generate a proper API token which would not expire. Let's see how they respond to this.
Cheers,
Markus


Fix bug in doctran document transformers

The latest doctran library no longer supports asynchrony.
The doctran library has removed support for asynchrony during this modification.
https://github.com/psychic-api/doctran/commit/22ec68d1fefaf3a141083489452ed0f82cb03a88

@baskaryan

cc @jasonwcfan

@liaokongVFX thanks for the fix, the intent behind making the library synchronous is to give people flexibility on how they want to run it.

@baskaryan  Hi,baskaryan. Can we merge it now?

sorry for delay, fixed in https://github.com/langchain-ai/langchain/pull/15264


Re-integrate content from #22 due to revert in #55.

Fix #62
What type of PR is this?

correction

What this PR does / why we need it:
Re-integrate content from #22 due to revert in #55.
Which issue(s) this PR fixes:
<!-- Automatically closes linked issue when PR is merged.

Fixes #62

#### Special notes for reviewers:

#### Changelog input

```
Re-integrate content from #22 due to revert in #55.
```

#### Additional documentation

As this is technical correction for PR https://github.com/camaraproject/Commonalities/pull/22 already approved, it can be merged without further discussion.


Hardcoded DATASTORE location in lxdone

Hello,
In a preexisting installation of OpenNebula 5.4, after changing the value of DATASTORE_LOCATION in oned.conf, the default /var/lib/one/datastores/ are hardcoded in the lxd addon, hence the value in onde.conf is never changed.
Relevant files are:
root@lxd_host:/var/lib/one# grep -rnw . -e '/var/lib/one/datastores/'
./remotes/vmm/lxd/lxd_common.py:135:        disk = "/var/lib/one/datastores/" + DS_ID +
./remotes/vmm/lxd/deploy.py:111:        DS_LOCATION = '/var/lib/one/datastores/' + DS_ID + '/' + VM_ID + '/'
Thanks!

We'll apply the patch in the 5.4-5 branch.

These are the conflicting lines of code
https://github.com/OpenNebula/addon-lxdone/blob/7bc9b165e97e3a37488ba9169a2bb3ed186460ca/src/remotes/vmm/lxd/deploy.py#L103
and
https://github.com/OpenNebula/addon-lxdone/blob/7bc9b165e97e3a37488ba9169a2bb3ed186460ca/src/remotes/vmm/lxd/lxd_common.py#L153


Update sql-database-elastic-scale-federation-migration.md

Adding information about error message page.

Hi @SilviaDoomra, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

Please remove the "en-us" from the link, because locale indicators are not allowed. Thanks.


goreleaser: use default template

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 52.32%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
203

Relevant Lines:
388

💛  - Coveralls


Update index.js

In this pull request, we have added the following functionalities

[ ] Add and Remove notes
[ ] Edit notes
[ ] Update Id
[ ] Update everything to local storage

Thank You.

Hello @DammyShittu , code reviewer.
My bundle file is causing linters error. How do I fix it sir.


Too high window sync time

Hey Benny,
I got a issue with my window sync time. I profile SDLSwapBuffers which calls SDL_GL_SwapWindow(window) and it needs a lot of time. On our Forward-Renderer needs it nearly the same amount of time to sync as to render, at the moment about 32ms. On my Deferred-Renderer is the render time nearly 3ms but the sync time about 60 ms. Do you have any idea whats going on?
Thanks in advance and sorry for my bad english.

I have the same problem, is there any way to fix it? It happens on nvidia(?) mobile gpus (laptops) as far I know. I've tested it on several computers. It happens only on my laptops.

try to set this in the code
_declspec(dllexport) DWORD NvOptimusEnablement = 0x00000001;
maybe you run on intel graphics, don't know if that would be a reason, but OpenGL in my own framework missed this one line and was painfully slow, but got speed eventually when I set enablement on

Thanks, it helped me a lot.

neni zac :)

This helps, thanks a lot.
Just for completion, there is an AMD equivalent:
_declspec(dllexport) int AmdPowerXpressRequestHighPerformance  = 1;

@zeyorama oh cool, thanks, I didn't knew there is AMD counterpar of it :)

@zeyorama thanks for the AMD equivalent!

Found it on
http://stackoverflow.com/questions/17458803/amd-equivalent-to-nvoptimusenablement


Enhancement: Add space between words of App name.

Fixes #70
You may need to clear derived data after building with this one.

Thank you!


x/website: 404 Error

after click download go file show me 404 error
url: https://golang.org/dl/

Also, where are you trying to connect from?

Now checked website and everything is ok.


Allow page margins to be set in the document

Allow the page margins to be set in the document using the pdf-page-margin attribute. This is particularly useful for quickly creating a slideshow of images that are set to stretch the width of the page. In this case, page margins interfere with the layout.

For now, we won't support the inner and outer margin settings.


Add VisionGeneration task

Description
This PR adds a VisionGeneration task to work with image-tex-to-text tasks (chatting with images).
with Pipeline(name="vision_generation_pipeline") as pipeline:
    loader = LoadDataFromDicts(
        data=[
            {
                "instruction": "What’s in this image?",
                "image": "https://upload.wikimedia.org/wikipedia/commons/thumb/d/dd/Gfp-wisconsin-madison-the-nature-boardwalk.jpg/2560px-Gfp-wisconsin-madison-the-nature-boardwalk.jpg"
            }
        ],
    )

    vision = VisionGeneration(
        name="vision_gen",
        llm=InferenceEndpointsLLM(
            model_id="meta-llama/Llama-3.2-11B-Vision-Instruct",
        ),
        image_type="url"
    )

    loader >> vision

It has been tested with InferenceEndpointsLLM and OpenAILLM. There are differences across providers, so we need more tests to ensure the remaining are working, specifically, vLLM internally calls to llm.chat instead of llm.generate, so we need to update it for the vision case

@plaguss Just an API level observation, but does it make sense for VisionGeneration to be task? because it means that it cannot be combined with other tasks. Like for example, you could not do SelfInstruct on an image of a document.
Also VisionGeneration needs logic and behaviour that is generally in the llms module, like processing messages.

Here are some use cases that would require vision language models.

caption generation
video caption generation
DocVQA on pdf (SelfInstruct)
Judge (ultrafeedback ++) on image generation


Add 1T1R support to gateware/software.

Will allow reducing PCIe bandwidth (and higher samplerates in 1T1R PCIe Gen2 X1) and sample processing/copy in SoapySDR.

Support was already partially in place and https://github.com/enjoy-digital/litex_m2sdr/commit/fa8ea4646c0609ffefb52a555449e5c90a0fde4d make it functionnal at build time when _1T1R_MODE is defined. This a first step before making it dynamically configurable. This indeed half PCIe bandwidth and allow higher samplerates when mounted in the baseboard.


rollups.sh reports missing file

Running rollups.sh results in this message:
cat: ../plugins/grouping/js/Container.js: No such file or directory
Container.js doesn't exist, but Container.js.temp - is that intentional or something that slipped through?

I cannot find rollups.sh in the repository and it's not mentioned in the user guide.


Feature - Multiple REPL with Remote REPL support

feature allows the possibility to connect to a REPL instance running on a remote machine over a TCP socket.  this allows one to connect to moleculer services running on remote machines over telnet or socat
the implementation uses the approach described in https://nodejs.org/api/repl.html#starting-multiple-repl-instances-against-a-single-running-instance
TCP socket port is configurable with the tcpPort REPL opts or with the replTcpPort broker configuration.
is simultaneous with PR https://github.com/moleculerjs/moleculer/pull/1153

I don't think it's a good idea to allow accessing the whole Moleculer project via a TCP socket. If you expose the port, anybody can join your cluster without any authentication and can make wrong things as well.
If you expose a port, it can be the port of the transporter and you can join from your computed with moleculer-cli as moleculer connect nats://some-server:4222.

I don't think it's a good idea to allow accessing the whole Moleculer project via a TCP socket. If you expose the port, anybody can join your cluster without any authentication and can make wrong things as well.

That's a valid concern.
However, IMHO since moleculer-repl is added under devDependencies, its never packed during the docker image build or when the moleculerjs package is built with NODE_ENV=production. Furthermore, the broker.repl() is only called when the "--repl" option is passed when to the moleculer-runner. Thus, the TCP socket is never opened during production by error.

If you expose a port, it can be the port of the transporter and you can join from your computed with moleculer-cli as moleculer connect nats://some-server:4222.
Thanks for the tip 👍🏾


VSCode chart.js extension

Just wanted to drop you a note about vscode-chartjs extension I thought devs using your library and vscode would enjoy:
https://github.com/RandomFractals/vscode-chartjs
will be loosely based on a similar vscode extension I created for Vega devs:
https://marketplace.visualstudio.com/items?itemName=RandomFractalsInc.vscode-vega-viewer
Let me know if you'd be interested in this collabo.

see your Slack general & dev channels for more info & some questions I have about your lib so far...

here is the gist of it:

most likely will publish it today || tomorrow ...

MVP is now in store: https://marketplace.visualstudio.com/items?itemName=RandomFractalsInc.vscode-chartjs

Feel free to close this issue if you are not interested in this collabo.
Otherwise, please log your feedback in my repo :)
https://github.com/RandomFractals/vscode-chartjs
cheers!

adding this vscode ext to your popular extensions page would be nice: https://github.com/chartjs/awesome
Thanks in advance!

@RandomFractals Your plugin looks great! I think you could make a PR in https://github.com/chartjs/awesome

thx! will do! also, I need to find time to add more samples.