ipV6 isInSubnet

Hey,
I'm currently working with this library to restrict some IP directly or with subnets.
IsInSubnet works great with IPv4 But i'm having an issue with IPV6
const subnet = new Address6("5800:10C3:E3C3:F1AA:48E3:D923:D494:1/127");
const client = new Address6("5800:10C3:E3C3:F1AA:48E3:D923:D494:0001");
return client.isInSubnet(subnet);

This code returns false whereas v6decode clearly says the ip is in the subnet.
Here is a dump of the objects
subnet
Address6 {
  groups: 8,
  v4: false,
  subnet: '/127',
  subnetMask: 127,
  zone: '',
  address: '5800:10C3:E3C3:F1AA:48E3:D923:D494:1/127',
  parsedSubnet: '127',
  addressMinusSuffix: '5800:10C3:E3C3:F1AA:48E3:D923:D494:1',
  elidedGroups: 0,
  valid: true,
  parsedAddress: [ '5800', '10c3', 'e3c3', 'f1aa', '48e3', 'd923', 'd494', '1' ] }

ip
Address6 {
  groups: 8,
  v4: false,
  subnet: '/128',
  subnetMask: 128,
  zone: '',
  address: '5801:10C3:E3C3:F1AA:48E3:D923:D494:1',
  addressMinusSuffix: '5801:10C3:E3C3:F1AA:48E3:D923:D494:1',
  elidedGroups: 0,
  valid: true,
  parsedAddress: [ '5801', '10c3', 'e3c3', 'f1aa', '48e3', 'd923', 'd494', '1' ] }

It works fine with ipv4 :  <IP_ADDRESS>/28 includes <IP_ADDRESS> but not <IP_ADDRESS>
Is this a bug?
Cheers,
Clément

I've tested this with the exact addresses you mentioned and it works, so if there was a bug it's been fixed since this issue was opened:
> (new ip.Address6("5800:10C3:E3C3:F1AA:48E3:D923:D494:0001")).isInSubnet(new ip.Address6("5800:10C3:E3C3:F1AA:48E3:D923:D494:1/127"))
true


The website's homepage is displayed well when zooming out the window

Description:  ensure that the website maintains functionality, readability, and usability when the user zooms out using the browser's zoom functionality or the scrolling button from the mouse.
Precondition : the website is up and running

Step 1 : Open the website in a web browser and ensure the browser is at its default zoom level (100%).
Expected results: Everyhing is set up.

Step 3 : At each zoom level, verify that the layout of the website does not break or leave excessive whitespace.
Expected result :  The layout is well displayed.

Step 4 : At each zoom level ensure that all text remains readable without becoming too small.
Expected result : the text remained readable and in an appropriate size.

Step 5: At each zoom level verify that buttons, links, and other interactive elements remain accessible and functional.
Expected results : Each element is functional and accessible.

Step 6 : At each zoom level verify that elements remains responsive and should resize appropriately without overlapping or becoming misaligned.
Expected result : Every element is well displayed.


fix: do not create Cluster Roles for non cluster config ArgoCD instance

What type of PR is this?

/kind bug

What does this PR do / why we need it:
This PR fixes the permissions associated with multiple service accounts.

Removes the cluster-wide permissions provided to a non-cluster config ArgoCD instance.

Cluster-wide permissions are only provided to ArgoCd instances for cluster config management.

Removes existing Cluster Role/RoleBinding for non cluster config ArgoCD instances.

Added vendor to .gitignore

fixed existing fmt issues

Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes #?
How to test changes / Special notes to the reviewer:

deploy the latest Operator
Make sure that no Cluster Roles and Role Bindings are created for ArgoCD application-controller and argocd-server for a non cluster config ArgoCD instance.
make sure the Cluster Role and Role Binding is still created for ArgoCd instances (application-controller) deployed for Cluster Config Management.

Upgrade Test

Make sure when the operator is upgraded, all the existing Cluster Roles and Role Bindings for non cluster config ArgoCd instances are deleted, unless they were created explicitly.

is it possible to release this fix soon?

is it possible to release this fix soon?

Hey @snorwin, we are waiting for a couple of more fixes to go in and then release a new version!!


Radius value used for evaluation in range search

Hi there,
There is a default value for radius here https://github.com/harsha-simhadri/big-ann-benchmarks/blob/e29a4fa45526c36fe04c65704dca558a519f897e/dataset_preparation/make_groundtruth.py#L241.
I am wondering will this default value 96237 be used for the final evaluation in the benchmark?

Yes, and thanks for the pointer too.


Keyword plugin not detecting "password" keyword?

I'm submitting a ...

[X ] bug report
[ ] feature request

What is the current behavior?
"detect-secrets scan" misses files in current directory or subdirectories containing "password" keyword

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
in git repository, create a file ATEST.cs with just two lines:
Get Connection String
String oraConnStr = "Data Source=sis2:1521/dprod.sis2.domain.org;User ID=svcacct;Password=svcacctpass;";

What is the expected behavior?
detect-secrets scan will identify this file as containing secrets according to keyword plugin

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

detect-secrets Version: 1.4.0
Python Version: 3.11.3
OS Version: Windows 10.0.09045.2965
File type (if applicable): text file

Other information
I've tried running detect-secrets via pre-commit hook or straight from command line. When I tried on real-world files, the Base64HighEntropyString did have some false-positive results, but none of the files containing the keyword "Password" are being detected.

It won't detect password unless it's in complete quotes

The tool fails to detect passwords unless they are enclosed in quotes. For example:

password=hjsadni221 is not detected,
but password="ansdna" is detected.

Hi @geekNero, @howard-adam,
I recently encountered an issue similar to the one described above. When a password is not enclosed in quotes, it is not detected as a secret. Here is an example:
PASSWORD=aijewga@#!%^ahgfdndbks211
This password is not detected, as shown in the following images:

However, when I add quotes around the password:
PASSWORD="aijewga@#!%^ahgfdndbks211"
The tool successfully detects it as a secret:

I have reviewed the relevant keyword_test.py test file from the Yelp/detect-secrets repository. I noticed that some test cases do not cover scenarios where the password is not enclosed in quotes:

Additionally, after looking into the source code of keyword.py, I found that the implementations were based on some heuristic approaches. While the design makes sense in some contexts, I am wondering if we could improve this behavior. Will it be more helpful for password detection or bring up more false positives?
Could you share your thoughts on this? I am interested in resolving this problem. Thanks!


working on Queuing_for_concurrent_requests_REST

Hey @lucylililiwang you need to open a PR linking to the issue by adding Closes #issuenumber in the description of your PR. Also, the Queuing issue has been labelled someday meaning it can't be fully implemented at the moment. We need to flesh out the REST api first. Kindly choose a more suitable issue to work on.


Docker Resources are Libraries Resources/Providers instead of Custom Resources 12.5-style

I'm not sure what the going forward plan is and why docker isn't necessarily following the docs provided by chef. So I figured I'd ask.
According to this link the docker cookbook isn't following the recommended way of writing resources in a cookbook. Is there a plan to move toward the more recommended way of creating custom resources? or are there technical hurdles that need to be overcome to move to that pattern of creating resources?

"Recommended" is relative. I recommend doing it this way.
Using the resources/ directory will magically name your class for you based on the file name.
Magic is bad.
This is more explicit. It also gives you something to search for in you text editor.
Also, I hope to one day include docker_resource into core Chef.... doing it this way, you can just move the file instead of having to go back in and un-magical-ify it.
-s

This patter is also used in mysql, httpd, etcd, and other cookbooks under chef-cookbooks.

@someara
So I guess the argument for whether to use one way or the other comes down to your proficiency with Ruby (and a bit of preference)? The libraries resources method is more explicit for folks who know a bit of Ruby and how it organizes things. If you aren't as familiar with Ruby you should use the resources 12.5-style of custom resources?

Again, these are 12.5 style Custom Resources. They just don't use the
magic directory.
They were among the first ever written. This cookbook drove a lot of
features in the new DSL... it was developed in tandem leading up to 12.5.
The major indicator is the use of :property instead of :attribute
Because they're in libraries/ you can see that they subclass Chef::Resource
instead of Chef::Resource::LWRPBase.
Docker ends up needing a lot of helper libraries to keep the reader sane
and resource patterns visible.
Using the magic directory, someone would need to jump back and forth
between two directories while trying to follow code instead of just two
files.
You can't put the helpers in the magic directory because the magic would be
applied inappropriately.
For the record, I've never bought the whole "Some people aren't as familiar
with Ruby" argument as an excuse to indulge in magic. If anything it's an
argument against it. This code is being used for operations, and will very
often be read for the first time in an emergency.
Being explicit is better for those not familiar with Ruby.
-s
On Tue, Jan 24, 2017 at 1:33 AM, David Brown<EMAIL_ADDRESS>wrote:

@someara https://github.com/someara
So I guess the argument for whether to use one way or the other comes down
to your proficiency with Ruby (and a bit of preference)? The libraries
resources method is more explicit for folks who know a bit of Ruby and how
it organizes things. If you aren't as familiar with Ruby you should use the
resources 12.5-style of custom resources?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chef-cookbooks/docker/issues/805#issuecomment-274721672,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAENzLNuyNCncrzCzGMQaNYf3c8zfNm_ks5rVZtFgaJpZM4Lrs45
.


ci(release): fix mispelling error

:tada: This PR is included in version 2.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix legacy support being initialized on modern servers

Sets the hook's API version to 1.13 so modern servers don't have to initialize legacy support. This has no effect on legacy servers as this property was only introduced in 1.13.

Merged, update will be released soon


[17.1.5] Configuration restore lists sections that no longer exist

E.g. Proxy Arp, Wake On Lan,...
I think some of these are now plugins.
There should probably be some sort of way for a plugin to say which sections it adds. And only list them if the plugin is installed.

Sure, this spot needs a similar addition like the XMLRPC sync plugin code that advertises possible config sections for the backup/restore.

timeout due to inactivity


Provide a reply for unauthorized commands

Currently, when using authorization groups, commands are silently ignored if the user is not in the corresponding authorized groups.
It would be useful to provide the ability to reply back to the user telling that the command can't be executed since he's not authorized, otherwise is not always clear if that's the case of there's a problem with the regex, or the bot is misbehaving, etc ...
Suggestions for this could be:

Configuraiton option with a string to display in these situations
Authorization hook to be called when this situation happens, providing the command executed, the user who did it, etc..

This is possible with the :route_authorization_failed event: https://docs.lita.io/plugin-authoring/events/
More discussion in #59.


[Grid] breakpoints on layout/grid containers

Hi all,
I have been using the alpha version of this framework for the past two weeks, and I'm having a blast.
I have one fundamental question, not sure if this is a feature request or not...
Please note that the original post on stackoverflow was done before Layout became Grid.
Description
So far I have used the Layout's container and item with success to rearrange items in a container according to different breakpoints.
But I would like to also be able to make the container responsive, meaning giving different values to 'align', 'direction', and 'justify, according to the same breakpoints.
<Layout
    container
>
<Layout item md={2} sm={12}>
    <Layout
        container
        align={"stretch"}
        direction={"row"}
        justify={"center"}
    // more Layout items here
    </Layout>
</Layout>
<Layout item md={10} sm={12}>
    <Layout
        container
        direction={"column"}
        justify={"space-around"}
        align={"flex-start"}
    >
    // more Layout items here
    </Layout>
</Layout>
</Layout>

In this example, the Layout items will resize properly according to the breakpoints md and sm, but there are no such rules for containers (for example I would like align to be 'flex-start' when the breakpoint is md, and 'center' when the breakpoint is sm.
so my question is : the 'breakpoints' (xs, sm, md, lg, xl) can be used to modify how the items are distributed on a line. So is it possible to use the same breakpoints to modify how containers distribute their items?
Thanks a lot.
Images & references
I have posted this question on stackoverflow:
http://stackoverflow.com/questions/43794780/material-ui-layout-breakpoints-on-containers

But I would like to also be able to make the container responsive, meaning giving different values to 'align', 'direction', and 'justify, according to the same breakpoints.

That's a good question. We anticipated that need when implementing the Grid but haven't implemented it.
We could add xsAlign, mdAlign, etc. properties. But that's going to be a lot of new properties, hence a large number of new CSS in the DOM. I think that we will need an elegant way to split the stylesheets.

Thanks a lot for this quick answer.
Do you have any recommendation on a workaround?
I was thinking about using window.matchMedia and its event listener to dynamically change the container properties as the viewport is resized.
Not sure if there is a more elegant way!

You can try to solve the issue with JS (withWidth) or CSS (media queries). Pick the solution that fit best your needs.


The flops may not decrease?

Thank you for providing this implementation!
As one earlier issue mentioned, I found that you computed the whole convolution and then did interpolation with the mask.
I wonder that if the x here in Convolution 1 is original? That is, you have sampled x before?

Thanks for your attention!
In fact, even in test mode, the implementation does not involve real-time acceleration. We just make sure that only sampling points are used in the test. As you can see in the function self.reconstruct_feature, we only use feature * mask to reconstruct the complete feature map, which means that we don't actually need to calculate the complete convolution 1.
https://github.com/zdaxie/SpatiallyAdaptiveInference-Detection/blob/master/mmdet/models/backbones/sparse_resnet.py#L190


No hardcode left in hecate.core.base

The main goal of this step is to rid of all the hardcoded parts in core.base module. In the course of action, related parts in other modules will be rewritten as well, expanding framework functionality.
TODO list:

[x] Implement simple dtype calculation algorithm using declared properties
[x] Make _elementwise_kernel args dynamic / using dtype
[x] Rewrite CUDA parts involving state getting in order to make them dynamic
[x] Make CellularAutomaton.buffers dynamic
[x] Implement color effects module
[x] Make fade_in, fade_out and smooth_factor dynamic
[x] Make _translate_code to actually run functions and get same result
[x] Make defines dynamic
[x] Make index_to_coord dynamic
[x] Make pack_state dynamic
[x] Implement Render / RenderPlain class
[x] Make render params and its functions (zoom, move, etc.) to be generated by Render class

Framework is fully dynamic now. We were not only got rid off all hardcoded parts, but actually complete all features we planned to include in the first release. And even more, the framework already has a support for multidimensional models and multiple properties per CA state.
At the next iteration, we will focus on first release preparations. Probably, some bugfixes will also follow.


Reorder bind and on: for select (fixes #461)

This should fix #461.

This will fix the issue for select.
Input maps to multiple underlying components, and it still won't work for those.
I'd apply the fix at every point possible in the file (bare in mind there are sub-components referenced in Input that will need the fix applying also i.e FormCheck).
Maybe add a test too?

I have created the PR.

Closing in favor of #473


Improve cast errors

@mccanne observed that the error from a failed time cast (Could not find format for "NotATime") isn't great and suggested giving a cast-specific error or wrapping the underlying error instead.

This has been fixed.  Reasonable Zed errors are now embedded when casts fail.


Fix blinking cursor on iOS

to: @majapw @ljharb
Removes the blinking cursor appearing in DateInput on iOS by adding user-select: none; (recommended as a workaround in another project). This PR should resolve the "Blinking cursor in mobile" issue.
Here's a gif of the blinking cursor and "Done" bottom bar:

LGTM after @ljharb's comment is addressed. Thanks for this, @moonboots!

Cool. Can you squash the commits into one before we merge tho?

Coverage remained the same at 86.501% when pulling 844ce6d285c175a1a753d61fc8957507951c9512 on moonboots:touch-input-fix into 57dcbaff11f3b3173b3700fbcdc11486ae748896 on airbnb:master.

Coverage remained the same at 86.501% when pulling 844ce6d285c175a1a753d61fc8957507951c9512 on moonboots:touch-input-fix into 57dcbaff11f3b3173b3700fbcdc11486ae748896 on airbnb:master.

Coverage remained the same at 86.501% when pulling 844ce6d285c175a1a753d61fc8957507951c9512 on moonboots:touch-input-fix into 57dcbaff11f3b3173b3700fbcdc11486ae748896 on airbnb:master.


Could you please tell me how to correct this error？Thank you in advance!!!

~/projects/condlanenet/conditional-lane-detection$ python tools/condlanenet/culane/test_culane.py
Traceback (most recent call last):
File "tools/condlanenet/culane/test_culane.py", line 21, in
from mmdet.models.detectors.condlanenet import CondLanePostProcessor
File "/home/yjx/projects/condlanenet/conditional-lane-detection/mmdet/models/init.py", line 1, in
from .backbones import *  # noqa: F401,F403
File "/home/yjx/projects/condlanenet/conditional-lane-detection/mmdet/models/backbones/init.py", line 1, in
from .hrnet import HRNet
File "/home/yjx/projects/condlanenet/conditional-lane-detection/mmdet/models/backbones/hrnet.py", line 7, in
from mmdet.utils import get_root_logger
ImportError: cannot import name 'get_root_logger' from 'mmdet.utils' (/home/yjx/projects/condlanenet/conditional-lane-detection/mmdet/utils/init.py)

And this....
###############################################
Traceback (most recent call last):
File "tools/condlanenet/culane/test_culane.py", line 20, in
from mmdet.utils.general_utils import mkdir
ModuleNotFoundError: No module named 'mmdet.utils.general_utils'

Anyone with the same puzzle is welcome......

The installation of mmdetection should be based on this repo since we have added some files
. The "Install.md" has been updated to avoid misunderstanding.
You can reinstall this framework as following:
git clone https://github.com/aliyun/conditional-lane-detection.git
cd conditional-lane-detection
pip install -r requirements/build.txt
python setup.py develop

python ./tools/condlanenet/culane/test_culane.py
/home/yjx/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/torch/_tensor.py:575: UserWarning: floor_divide is deprecated, and will be removed in a future version of pytorch. It currently rounds toward 0 (like the 'trunc' function NOT 'floor'). This results in incorrect rounding for negative values.
To keep the current behavior, use torch.div(a, b, rounding_mode='trunc'), or for actual floor division, use torch.div(a, b, rounding_mode='floor'). (Triggered internally at  /opt/conda/conda-bld/pytorch_1623448224956/work/aten/src/ATen/native/BinaryOps.cpp:467.)
return torch.floor_divide(self, other)
The model and loaded state dict do not match exactly
unexpected key in source state_dict: bbox_head.reg_branch.0.conv.weight, bbox_head.reg_branch.0.bn.weight, bbox_head.reg_branch.0.bn.bias, bbox_head.reg_branch.0.bn.running_mean, bbox_head.reg_branch.0.bn.running_var, bbox_head.reg_branch.0.bn.num_batches_tracked, bbox_head.reg_branch.1.conv.weight, bbox_head.reg_branch.1.bn.weight, bbox_head.reg_branch.1.bn.bias, bbox_head.reg_branch.1.bn.running_mean, bbox_head.reg_branch.1.bn.running_var, bbox_head.reg_branch.1.bn.num_batches_tracked, bbox_head.reg_branch.2.conv.weight, bbox_head.reg_branch.2.bn.weight, bbox_head.reg_branch.2.bn.bias, bbox_head.reg_branch.2.bn.running_mean, bbox_head.reg_branch.2.bn.running_var, bbox_head.reg_branch.2.bn.num_batches_tracked
completed: 0, elapsed: 0sSegmentation fault (core dumped)

@hustllz emmmm, why does this error pop up? Do you have any idea?

Solved. Just need to rewrite ~ as the full path in config.py.

Solved. Just need to rewrite ~ as the full path in config.py.

Hello, I also encountered the same problem. Could you please tell me in detail how you solved it? thank you very much

@HaoYongqiang1999
Which problem did you encounter? I will try my best to help you!!!

@HaoYongqiang1999
Which problem did you encounter?

Thanks to your previous comment,problem has been solved. Thank you!

Can you tell me How did you change it?

Can you tell me How did you change it?

Could you please say chinese? I am a little confused about what you have asked. And I solved two problems in this round of discussion. Which one do you want to tackle? I will try my best to help you, in the way the author did to me.

Can you tell me How did you change it?

Could you please say chinese? I am a little confused about what you have asked. And I solved two problems in this round of discussion. Which one do you want to tackle? I will try my best to help you, in the way the author did to me.
您好，我想请问一下您上面出现的问题（core dumped）是怎么解决的？

Can you tell me How did you change it?

Could you please say chinese? I am a little confused about what you have asked. And I solved two problems in this round of discussion. Which one do you want to tackle? I will try my best to help you, in the way the author did to me.

在您评论的帮助下，我的问题已经解决了，我现在没什么问题了。感谢！

Can you tell me How did you change it?

Could you please say chinese? I am a little confused about what you have asked. And I solved two problems in this round of discussion. Which one do you want to tackle? I will try my best to help you, in the way the author did to me.
您好，我想请问一下您上面出现的问题（core dumped）是怎么解决的？

我是把"/conditional-lane-detection/configs/condlanenet/culane/culane_large_test.py"中的第六行改成了我自己的数据集路径，就没问题了。

Can you tell me How did you change it?

Could you please say chinese? I am a little confused about what you have asked. And I solved two problems in this round of discussion. Which one do you want to tackle? I will try my best to help you, in the way the author did to me.
您好，我想请问一下您上面出现的问题（core dumped）是怎么解决的？

我是把"/conditional-lane-detection/configs/condlanenet/culane/culane_large_test.py"中的第六行改成了我自己的数据集路径，就没问题了。

谢谢

Getting Error:
/content/conditional-lane-detection
No CUDA runtime is found, using CUDA_HOME='/usr/local/cuda'
Traceback (most recent call last):
File "tools/condlanenet/tusimple/test_tusimple.py", line 14, in
import tools
ModuleNotFoundError: No module named 'tools'
Any solution??


Unable to install latest version of reviewdog v0.18.0 in my gitlab pipelien

go install github.com/reviewdog/reviewdog/cmd/reviewdog@latest
go: downloading github.com/reviewdog/reviewdog v0.18.0
go install: github.com/reviewdog/reviewdog/cmd/reviewdog@latest (in github.com/reviewdog/reviewdog@v0.18.0): go.mod:3: invalid go version '1.21.7': must match format 1.23
I checked in go.mod its written like this - 1.21.7
Can you plz correct this go mod version thing  ?

please tell me your version of Go.

1.21

What is the result when you run go version?
I suspect that you are using Go 1.20 or earlier.

its 1.21

Hi I'm facing issue as well, any solution?

I have same error... any update?


Raid Page ⛑

Raid Page ⛑
We should make a "raid page". The concept of this in my brain is inspired by Spotify's "Car Mode". For those unfamiliar, when you are driving a car, Spotify reduces all the icons to only the core ones you really care about. I purpose we do something similar for tarkov.dev.
I would like to see a "raid page" that is a page you visit during a raid. So you can very quickly tab in and out of game to lookup items and see the most important features about them without all the detailed info. Buttons should be big and icons should be too!

Neat idea! I just use the home page for this currently since it provides a search box, automatically updating results, and sell to trader and sell to flea prices (include per-slot values).

@GrantBirki  can you imagine a or a couple of scenarios where you would want the "in-raid mode" ?
What different scenarios and what information do you need to access quickly when in-raid?

Great idea, with a little bit of brainstorming we could build something nice.
I'd say it would be good to have a list of "hot items", to highlight the ones we're looking for.
And a map, preferably interactiv, like the mapgenie or tarkov.help ones. It could use a lib such as leaflet, or even embed an existing one.


Generated anchor IDs for all of the headings in the Client Developer's Guide

In order to support cross references in the format:
xref:client-dev-ADONET_Integration-ADONet-Integration[]

We need to have anchor IDs defined for every heading in a guide. This commit modifies the source files for the Client Dev guide so that every heading gets an associated anchor ID. This makes it possible to cross-reference any heading in the guide using the xref:anchor-id[] syntax.

Thanks very much for the speedy merge, @shawkins !


fix: skip abstract models in migrations

fixes: #126

@SafaAlfulaij thanks! I'm getting a few upgrades in place for pgtrigger before I merge this change. I'm going to write a failing test case for this in a separate PR. This fix should go out with the new release today


Merge branch 'astropy' into 'master' - [merged]

Merges cherry-pick-08d222a2 -> 0.6
astropy devdeps
See merge request LMSAL_HUB/aia_hub/aiapy!154
(cherry picked from commit 08d222a227e4942b6ea03fea3408d7a5299c77eb)
2a775fdb astropy devdeps

mentioned in commit d977c7ef834c19eceb0d644c054ced721770b6c7

approved this merge request


qiime.plugin utility function for making system calls

This will allow plugins to make system calls in the same way, which will facilitate error handling. I started on this in gregcaporaso/q2-dada2#1.

Just a note, we may want to allow passing in a stream for verbose output so
that control over the fd receiving the output can be managed higher in the
stack
On Tue, Sep 27, 2016 at 9:42 AM, Greg Caporaso<EMAIL_ADDRESS>wrote:

This will allow plugins to make system calls in the same way, which will
facilitate error handling. I started on this in gregcaporaso/q2-dada2#1
https://github.com/gregcaporaso/q2-dada2/pull/1.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/qiime2/qiime2/issues/153, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAc8srkFovjnKMs8auADm4ravmI9-wyUks5quUdlgaJpZM4KH5Gk
.

We're going to close this one open now as the different plugins require some different handling, so it's not clear that we're ready for a general purpose function. We might revisit in the future, once we have a few more plugins that are making system calls.


Add an example

Overview

Add example for code-based usage.

Coverage decreased (-1.0%) to 50.424% when pulling 8c14536f12300141561603b0f9a97e0a1942ca34 on example into 9a7942937550d84187c6c777abdd21be08141bd2 on master.

Coverage decreased (-1.2%) to 50.212% when pulling 8c14536f12300141561603b0f9a97e0a1942ca34 on example into 9a7942937550d84187c6c777abdd21be08141bd2 on master.

Coverage decreased (-1.3%) to 49.185% when pulling c02542bb2e978c9ef740bbdc5159c48b8b6d98f3 on example into c9305ad29c7e6c832afe829689a6ff0cbc898103 on master.

Coverage decreased (-1.3%) to 49.185% when pulling 9bf5506177f6f151e51c37960a8ea30431bb9421 on example into c9305ad29c7e6c832afe829689a6ff0cbc898103 on master.

Coverage decreased (-1.3%) to 49.185% when pulling 37fbf538d9885d01b8be2bb1e059731da9bab472 on example into c9305ad29c7e6c832afe829689a6ff0cbc898103 on master.


Hover message hides under other element

Hi guys! I use monaco editor in my web and the hover message usually hides under other element. Do you know how to fix this? Thank you!

I have the same issue and found this one while researching how to fix it.
The issue for me is that the DOM container for the hover is not high enough in the DOM tree, Or that auto positioning of the hover is not working and I can't figure out how to configure that.

Looks like fixedOverflowWidgets: true could help here. See https://github.com/microsoft/monaco-editor/issues/2485#issuecomment-842908682


SendMail doesn't work with SSL and doesn't handle errors

https://github.com/2sic/app-mobius-forms/blob/0520f71e7f0271d9f0c25c4ec92def479e52e935/live/api/Parts/SendMail.cs#L12-L36

SendMail.Send() always returns true and doesn't handle STMP-Errors. Instead the result should check wheter DotNetNuke.Services.Mail.Mail.SendMail() returned an empty string (success) but otherwise return false (handle the error).
It has smtpEnableSSL-parameter hardcoded to false (last parameter in DotNetNuke.Services.Mail.Mail.SendMail()). Instead this should be retrieved from DotNetNuke.Entities.Host.Host.EnableSMTPSSL.

great input - could you contribute? 😊


Fix twitchy scrolling on touchpads

Previously, any and all scroll events were interpreted as reason to switch
workspaces. This resulted in twitchy behaviour, where the scrolling was
practically unusable.
Now, we pool all scroll values, and only scroll if the value is larger than the
new config option "smooth-scrolling-threshold". If this option is not set, the
behaviour is unchanged.

Also, there is inconsistency between the discrete and continuous case. In the discrete case horizontal scroll is also considered, in the continuous case this is ignored. Do we want to change this behaviour? How should this behave on "diagonal" scrolling?

The actual scrolling behaviour is garbage, feel free to redo it completely :)
Also there is also a handleScroll in ALabel.cpp if you can also apply your changes here :)

@Alexays Should the threshold be configurable for every individual module, or is it possible to add some global option?

@Synthetica9 configurable for each module, soon I will make a kind of global configuration that the modules will inherit

@Alexays and the diagonal case?

What do you think is better?
Imo there is no need of the diagonal case.

I see a few options:

Only use the principle axis,
Add the axes,
Ignore "diagonal" scrolls, and only use events that are within a few degrees of either horizontal or vertical.

1 and 2 seem to work pretty well. I haven't programmed 3 yet.

(This implementation is 1. Horizontal scrolling doesn't seem to make sense for ALabel)

@Alexays What's currently holding this up?

I've been busy, i'm currently reviewing it :)

Seems quite good!
Just i think it's better to rename smooth-scrolling-threshold to scrolling-threshold, right?

Just i think it's better to rename smooth-scrolling-threshold to scrolling-threshold, right?

Maybe? It isn't relevant to all scrolling, only to the continuous case

Just i think it's better to rename smooth-scrolling-threshold to scrolling-threshold, right?

Maybe? It isn't relevant to all scrolling, only to the continuous case

Yeah, right, even if I think the smooth case is used in almost all cases, we'll keep smooth-scrolling-threshold
Thanks!


Git History Graph: enable jumping to associated PR

Type: Feature Request
In the screenshot below I'd like to be able to jump to the associated PR on the GH site, perhaps using the context menu.

I'm not sure if this would need to be implemented in the bundled Git extension or is more appropriately handled by the GitHub Pull Requests extension (pinging @alexr00)
VS Code version: Code - Insiders 1.92.0-insider (97d1b933ada0ca69bea107f7a89e029991277ccc, 2024-07-22T05:03:41.362Z)
OS version: Windows_NT x64 10.0.22631
Modes:

Cool idea. @lszomoru is there a menu that GHPR could contribute to for this?

There is, but I think that we should chat as I think you would need more information in order to decide when to show the action.

It would be particularly useful when the commit message doesn't give any indication what the change does. Some of the VS Code team commonly use such messages:

Ideally would should be able to link out to both GitHub issues as well as pull requests.

A plea to @sandy081 to use more informative PR descriptions:

Definitely! It would be helpful to show the associated issue in the hover.

Closing as duplicate of https://github.com/microsoft/vscode/issues/226427


System.Runtime.InteropServices.COMException at save

Ran into this error today, while trying to save changes after adding a single resource key.

Edit: It seems to have saved the changes, despite the error.

This is not critical, you can ignore this. I will avoid this in the next version.
Is this reproducible, or just happend once?

Happened twice. Stopped after a restart. Was as you said not critical so didn't use too much time testing.


Improving Mian.py Bugs

I'm improving main.py

Hey @Nouman-Usman!
I have assigned you this issue.

Done


inja templating engine support for ninja

inja is similar to jinja and using a templating engine can help us reducing a lot of repetitive work in ninja file.

Please used proper std:: namespacing on new or modified code.
You should pretend that the "using namespace std;" lines in some of the cpp files are not actually there.

I have to admit, i don't see how this is beneficial. It appears to be the case, based on your example, that this inja templating preprocesses the entire ninja file prior to running the dependency graph.
Why should that be built into ninja itself, instead of just run as a pre-processing step on the .ninja file after it's generated?
Ninja is almost always used as a machine-generated intermediate representation, e.g. CMake -> ninja,
So since CMake, just to use it as an example, already knows how to generate everything based on it's own internal rules, what does this templating syntax provide to cmake that it can't do itself?


Click event before animation is done, causes closing bug

If the user clicks on that noty notification, right before the animation of that noty is done, the click event for that noty is broken and cannot be closed by click event anymore, so it stays on the screen 4ever.
Thanks for Your attention.

I've found that is caused only with using animate.css, otherwise it's okay.

Yes it is only with animation.css I also notice. I have had to rid my code of using animation.css due to this but would love to be able to utilize it


Update docs for DefaultAzureCredential regarding AZURE_CLIENT_ID / managedIdentityClientId relationship

Add documentation to DefaultAzureCredential indicating that setting the AZURE_CLIENT_ID environment variable is equivalent to setting the managedIdentityClientId.

Closing this because it's already documented on the class (feel free to reopen if you meant to have it documented somewhere else):
https://github.com/Azure/azure-sdk-for-python/blob/89e7f4ba4459f44d18e216d3eb47a71d65307d35/sdk/identity/azure-identity/azure/identity/_credentials/default.py#L65-L66

Closing this because it's already documented on the class (feel free to reopen if you meant to have it documented somewhere else):
https://github.com/Azure/azure-sdk-for-python/blob/89e7f4ba4459f44d18e216d3eb47a71d65307d35/sdk/identity/azure-identity/azure/identity/_credentials/default.py#L65-L66

The other languages have a section about Managed Identities here that explains how to configure a custom user created managed identity client ID. Python doesn't have this (not sure if that is intentional), but it would go there if so.
The reason for the issue is that .NET had a customer question about this behavior which is not documented.

The other languages have a section about Managed Identities here that explains how to configure a custom user created managed identity client ID. Python doesn't have this (not sure if that is intentional), but it would go there if so.
The reason for the issue is that .NET had a customer question about this behavior which is not documented.


Reporting / Alert scheduling doesn't properly convert timezones

Bug description
Operating System: Ubuntu 22.04
Version: 4.0.2
Create an alert that will trigger.
How to reproduce the bug

Create an alert that will trigger.
Schedule the alert to run soon with a timezone other than UTC.
Verify that alert doesn't run when it should.

Screenshots/recordings
No response
Superset version
4.0.2
Python version
3.10
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[X] I have searched Superset docs and Slack and didn't find a solution to my problem.
[X] I have searched the GitHub issue tracker and didn't find a similar bug report.
[X] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

Fixed by https://github.com/apache/superset/pull/29798


Title

Would you update this for 1.20.12? It crashes when I do /script debug connect localhost 8080

@Lapis256


[203] Reduce cache invalidation time to 1+1

Instead of creating a new CDN with that config for the client we should implement configurable cache-control headers in both the SSR server and the Cloud server.

Site pageviews: (0.78)
Extra group sites: (0)
Extra group pageviews: (0)
Development size: (20)
Blocking: no frontity without it (x10)
Environment: integration is starting (x6)
Reusability potential: marginal (x2)
Case Study: yes (x5)
PRIORITY: [203.39]

#notify

:tada: This issue has been resolved in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Dell U2720 has not supported

Dell U2720 connect to Mac via type-c

Not working for my MBP 16" i9 neither

Just wanted to chime in and report the same issue for Dell U3818DW connected to an M1 MBA via USB-C cable. I understand M1's aren't currently supported but just wanted to report for posterity's sake

Ahhh, reading Issue #208 I tried quite a long to get this working. Just to come across this issue now - M1 is not yet supported 😅 So I'm having the same issue, Dell U2720Q, connected via USB-C to an M1 MBA.

I'm experiencing the same issue for Dell 2419H connected to a 2020 M1 MB Air via an HDMI cable. Looking forward to M1 support sometime soon.

Closing this because M1, see #323
If you're not using M1, see #208 and our troubleshooting wiki.


Convert to auto-plugin (closes #14)

Changes made for auto-plugin. I also added the sbt-jhm-tester subdirectory: just cd into it, open sbt and test sbt-jhm, e.g. by executing run.
Additionally I (hopefully) improved/simplified some things.

Thanks!


[MonthPicker] add shouldDisableMonth to MonthPicker

[X] I have followed (at least) the PR section of the contributing guide.

Took the liberty to add this small functionality, as I needed it in my project. I wonder if there was a rationale behind having shouldDisableYear and not shouldDisableMonth.
Use case: I want users to select editions of a monthly magazine, yet the magazine isn't systematically published every month.
The shouldDisableMonth has exactly the same functionality as shouldDisableYear, it disables the month only on the "month" view. It is still possible to access dates of that month using the "day" view and using arrows, so it may have to be combined with shouldDisableDate for 100% disabling.

Note, the pickers are moving inside the https://github.com/mui/mui-x repository. Please re-open the PR there, once they are established. cc @flaviendelangle


neodev instead of lua-dev

The config below:
local settings = {
    servers = {
        sumneko_lua = require('lua-dev').setup(),
    },
}
require('lsp-setup').setup(settings)

Then open .lua file, prompt message:
You're using the old way of setting up neodev (previously lua-dev). Please check the docs at https://github.com/folke/neodev.nvim#-setup

-- IMPORTANT: make sure to setup neodev BEFORE lspconfig
require("neodev").setup({
  -- add any options here, or leave empty to use the default settings
})

require('lsp-setup').setup({
  servers = {
    sumneko_lua = {
      settings = {
        Lua = {
          completion = {
            callSnippet = "Replace",
          },
          -- format = {
          --   enable = true,
          -- }
        }
      }
    },
  }
})

How is it configured in lsp-setup.nvim?

are you asking me @cppcoffee?  I just shared what worked for me. I took the setup from here.

Thank you @sourabhxyz , I using lsp-setup.nvim and not sure how the new neodev sumneko_lua should be configured on lsp-setup.nvim.
If you simply replace lua-dev with neodev, you get the same prompt, like below:
local settings = {
    servers = {
        sumneko_lua = require('neodev').setup(),
    },
}
require('lsp-setup').setup(settings)

@cppcoffee I am also using lsp-setup and my comment shows how I configured it.

Hi @sourabhxyz , using https://github.com/junnplus/lsp-setup.nvim/issues/41#issuecomment-1288394661 , it has been solved.
close issue.


How to run this on replit?

How to run this on replit?

help me pls!

Create a new Bash Repl,
create a replit.nix file and paste this:
{ pkgs }: {
    deps = [
        pkgs.bashInteractive
        pkgs.nodejs-16_x
    ];
}

This will install nodejs 16.
Then install prismarine-web-client with
npm i prismarine-web-client

(in the Terminal).
Now paste this into main.sh:
npx prismarine-web-client

Then just click the Run-Button.

thx :D

and where i can find proxy for this?

is any mods for this which is working on replit?

What do you mean by mods?

What do you mean by mods?

addons which add more things

no

There aren’t any add-ons available for Prismarine Web Client. Also, for longer discussion questions, I’d highly recommend going into our Discord.

Looks like this question is solved; closing


Support for extra long measurements

Currently, there are several limitations:

ring buffer from pyTrigger generates an array of zeros for the whole measurement duration
periodic saving loads ALL data, appends new data and saves data
Core class does not know if one of the acquisition threads received an error and has been stopped

How to solve the limitations:
1)

ring buffer should only contain a few samples, just enough that trigger events can be captured
pyTrigger should have an additional array to which all data is appended after the ring buffer is full OR
maybe data could be appended to the array right away.
The array with all measured data, should be emptied with periodic saving

periodic saving (and save_measurements() method) should save data as a file format, with which a minimal amount of data is loaded and data efficiently appended. GPT4 suggested library "h5py"

in case some of the threads are unexpectedly stopped (they receive an error), stop the whole acquisition.

1.) Solved.
2.) Data format will not be changed unless needed.
3.) TODO


macOS release ?

Thank you for this useful tool.
Please provide macOS amd64 and arm64 release binaries.

unfortunately, I don't have macOS.
You can easily

install rust with: rustup.rs
clone the repository with the git command: git clone https://github.com/graymind75/shams
move to the folder: cd shams
execute build command: cargo build --release
after build process done, go to target/release folder and the binary is there: shams

Thank you for explanations.


chore(main): release 1.2.6

:robot: I have created a release beep boop
1.2.6 (2023-11-05)
Bug Fixes

deps: bump dependencies (d132df6)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/JoshAlexis/utils-abstractions/releases/tag/v1.2.6 :sunflower:


test: running checks without any change

Let's see what checks fail.

Where is OwlBot Postprocessor? (Maybe it hasn't adopted branch change?)

OwlBot worked:
Processing monorepo
In monorepo but no owl-bot-staging. Formatting changes in the last commit

For "Kokoro" failure, @lqiu96 is going to look into this. It seems related to GraalVM tests.

"Kokoro - Test: Integration" failed:
[INFO] Results:
[INFO]
[ERROR] Failures:
[ERROR]   ITSystemTest.detectWebEntitiesGcsTest:495 expected to contain: The Palace Of Fine Arts
but was            : [Ancient Rome, Triumphal arch, Ancient history, Ancient Roman architecture, History, Palace of Fine Arts, Historic site, Architecture, Façade, Arch, Tourism, Hematocrit]
[ERROR]   ITSystemTest.detectWebEntitiesIncludeGeoResultsGcsTest:557 expected to contain: The Palace Of Fine Arts
but was            : [Ancient Rome, Triumphal arch, Ancient history, Ancient Roman architecture, Palace of Fine Arts, History, Historic site, Architecture, Façade, Arch, Tourism, Hematocrit]

Retrying.

"ci / windows" failed:
2022-08-26T14:55:47.5907470Z [INFO] Google Serverless VPC Access BOM 2.3.3-SNAPSHOT .... SUCCESS [  0.016 s]
2022-08-26T14:55:47.5907842Z [INFO] Google Cloud webrisk BOM 2.2.3-SNAPSHOT ............ SUCCESS [  0.000 s]
2022-08-26T14:55:47.5908239Z [INFO] Google Cloud websecurityscanner BOM 2.2.4-SNAPSHOT . SUCCESS [  0.000 s]
2022-08-26T14:55:47.5908634Z [INFO] Google Cloud Workflow Executions BOM 2.3.3-SNAPSHOT  SUCCESS [  0.015 s]
2022-08-26T14:55:47.5909012Z [INFO] Google Cloud Workflows BOM 2.3.3-SNAPSHOT .......... SUCCESS [  0.000 s]
2022-08-26T14:55:47.5909416Z [INFO] ------------------------------------------------------------------------
2022-08-26T14:55:47.5909621Z [INFO] BUILD SUCCESS
2022-08-26T14:55:47.5910574Z [INFO] ------------------------------------------------------------------------
2022-08-26T14:55:47.5910865Z [INFO] Total time:  18:33 min
2022-08-26T14:55:47.5911220Z [INFO] Finished at: 2022-08-26T14:55:47Z
2022-08-26T14:55:47.5911623Z [INFO] ------------------------------------------------------------------------
2022-08-26T14:55:48.1699729Z basename: missing operand
2022-08-26T14:55:48.1715759Z Try 'basename --help' for more information.
2022-08-26T14:55:49.1312267Z ##[error]Process completed with exit code 1.
2022-08-26T14:55:49.2491917Z Post job cleanup.
2022-08-26T14:55:49.8460616Z Post job cleanup.

"Kokoro - Test: Integration" failed:
[INFO] Results:
[INFO]
[ERROR] Failures:
[ERROR]   ITSystemTest.detectWebEntitiesGcsTest:495 expected to contain: The Palace Of Fine Arts
but was            : [Ancient Rome, Triumphal arch, Ancient history, Ancient Roman architecture, History, Palace of Fine Arts, Historic site, Architecture, Façade, Arch, Tourism, Hematocrit]
[ERROR]   ITSystemTest.detectWebEntitiesIncludeGeoResultsGcsTest:557 expected to contain: The Palace Of Fine Arts
but was            : [Ancient Rome, Triumphal arch, Ancient history, Ancient Roman architecture, Palace of Fine Arts, History, Historic site, Architecture, Façade, Arch, Tourism, Hematocrit]

Retrying.

@suztomo Just saw this. Seems to be related to Java-Vision. Alice and I have had issues with this repo for the past two weeks. We've emailed and pinged the people but it seems that they haven't gotten back to us about fixing the ITs. Going to take a look to see if we can just @IgnoreTest. It's also blocking quite a few renovate-bot and owlbot PRs in the the repo

"Palace Of Fine Arts" is included in the list. "The Palace Of Fine Arts" is not.
[ERROR]   ITSystemTest.detectWebEntitiesGcsTest:495 expected to contain: The Palace Of Fine Arts
but was            : [Ancient Rome, Triumphal arch, Ancient history, Ancient Roman architecture, History, Palace of Fine Arts, Historic site, Architecture, Façade, Arch, Tourism, Hematocrit]

@lqiu96 Given recent discussion, I think we should try to understand what is not working.
Cont of this PR: https://github.com/googleapis/google-cloud-java/pull/8229

@suztomo Sounds good. Will take a look in a bit to see if we can fix it


[JS] [KeyVault] Secrets and Keys as standalone packages

Separate KeyVault Secrets and Keys into standalone packages.
The following instructions are based to the latest feedback:

[x] Make a PR with the Keys package, called keyvault-keys, located at /sdk/keyvault/keyvault-keys.
[x] Make a PR with the Secrets package, called keyvault-secrets, located at /sdk/keyvault/keyvault-secrets. Don't reuse the shared code (it should be duplicated).
[x] Make sure CI works properly for each package.
[x] Make sure each package follows the guidelines for the TypeScript packages.
[x] Confirm that autorest is generating the new files in the folders of the new packages.
[ ] Remove the current folder at /sdk/keyvault/keyvault.

Confirmed with @daschult that the keyvault-keys and keyvault-secrets are fine as they are (no autorest needed).


text.is-alphabetic implemented on eo

https://github.com/objectionary/eo-strings/issues/48

@MikhailLipanin @includealex please review.

@EugeneDar sure

@levBagryansky please, check again.

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 3min)


Blog post: Streamline your mainframe development with Zowe Explorer & Zowe CLI (a series): Part 3 — REXX

If you’ve read the other articles, then you already have a pretty good idea what this recipe will be about. If not, here is the gist:
Author: Alex Dumitru
@RASakach and @Joe-Winchester would you please review and provide any feedback to Alex?

I've categorized this as {Core} and added trademarks, updated the display title, and made a few minor grammar corrections.


My chat history got deleted

Hey guys how are you?
So I found no chat history from past projects in my chat bar. Where can I find them?
Best

@andresroi #39 has the fix


Short solution needed: "How to truncate a table" (clickhouse)

Please help us write most modern and shortest code solution for this issue:
How to truncate a table (technology: clickhouse)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to truncate a table


build wheels on newer image for py3.12 support

cibuildwheel has dropped support for (launching from, separate to building) 3.7 and we need a newer one to pull in a docker image with py3.12. For now this keeps building 3.7 wheels

Why not go for at least Python 3.11?

good point. initially thought this might mean a new lower bound for wheels built but since it doesn't we may as well bump further. doing a test run with 3.12 and will update here if all ok

thanks! is it possible to upload the missing py3.12 wheels for the currently released version?


Cannot publish when using single file output via TypeScriptOutFile

Copying file Scripts/compiledFromTS.js to obj\Release\Package\PackageTmp\Scrippts\compiledFromTS.js failed. Could not find file 'Scripts/compiledFromTS.js'.
This is because the individual js files are not generated with single file output. The single output is also not published.
Probably related to https://github.com/Microsoft/TypeScript/issues/3178

what version of TS are you using? if you are using 1.7.4, is it the same issue as https://github.com/Microsoft/TypeScript/issues/5894?

I'm using 1.7.6 with VS 2015. I can verify that ItemGroup's Content Include is using relative paths, not absolute like in https://github.com/Microsoft/TypeScript/issues/5894.

@mykohsu can you share your repro?

Unfortunately the repro is internal only so may be difficult to reproduce. Weirdly, this is now working - I suspect there was conflict with a gulp script.
However, is it a known issue that the single file output is not published unless that target file is also added to the project? Is this the desired behavior?

This may be related to the following:
Can you try the following in the TypeScript targets file for VS 2015 (C:\Program Files (x86)\MSBuild\Microsoft\VisualStudio\v14.0\TypeScript), can you change the following line:
<TypeScriptBuildConfigurations Condition="'$(TypeScriptOutFile)' != ''">$(TypeScriptBuildConfigurations) --out "$(TypeScriptOutFile)"</TypeScriptBuildConfigurations>
to
<TypeScriptBuildConfigurations Condition="'$(TypeScriptOutFile)' != ''">$(TypeScriptBuildConfigurations) --outFile "$(TypeScriptOutFile)"</TypeScriptBuildConfigurations>
I changed --out to --outFile

@mykohsu any updates on this issue? did the workaround help?

I have the same issue where I am using typescript 1.8.6 with a tsconfig file specifying an outFile. This workaround didn't help.
What I did in the end is to add a build condition around the ItemGroup that contains all of my TypeScriptCompile elements to limit it to Debug mode.  My publish settings are for a Release configuration and my ts outFile is included as part of the project so it gets copied on build.
Not ideal but it got me going.

    ... etc

Closing this as it's unclear we can actually do anything here. We fixed the issue where specifying the outfile option in the tsconfig.json file would not work with publishing.

@paulvanbrenk is there an update that will fix the TypeScript targets file without manually editing it?

@paulvanbrenk so, what's the solution.

@jamesfancy for which question? The original bug is fixed in 2.0.

@paulvanbrenk Thank you. I am waiting for 2.0 release.

With 1.8.6, this will work if you use the project properties instead of using a tsconfig.json. It's not ideal but it works on a build server without modifying targets.


[doc] Overview section

Very nice. I think the file ur5.urdf should also be included within pinocchio (I would say in the folder models), especially if the example is meant to be automatically built and tested.

@gabrielebndn we had romeo's urdf included within pinocchio before, and I suggested to remove it in #424.
It doesn't change anything for CI automation, as it can be installed with all the other dependencies.

I like all changes. My comments follow. I've already mentioned some to Nicolas

I'm ok with not including ur5.urdf within pinocchio, but then a better link should be provided to the file. First of all, I was forced to dig inside the repo until I found the one file I was looking for, which is not nice. Most importantly, I could not find ur5.urdf. Instead, I found ur5.urdf.xacro, which I needed to convert to urdf with ROS. I do not think we should expect the user to that, nor should pinocchio testing routines depend on ROS

concerning the examples inverse-kinematics: the python example employs jointJacobian, while the one in C++ employs computeJointJacobians followed by getJointJacobian. Nicolas told me he could not make the two examples to be identical because of problems with deprecated methods. So I dug deep into the code, and I found some critical points.
First of all, the easy part: in  o use jointJacobian in C++, the correct procedure is the following:
se3::Data::Matrix6x J(6,model.nv); J.setZero(); se3::jointJacobian(model,data,q,JOINT_ID,J);
while the following procedure
se3::Data::Matrix6x J = se3::jointJacobian(model,data,q,JOINT_ID);
is deprecated.
Then there is a bigger problem: if you try to use the above line in Nicolas' example, it won't work. You also need to call forwardKinematics (before or after, it does not make a difference). You do not need it in python. Why? Because the python method jointJacobian DOES NOT call its corresponding C++ function. Instead, it internally calls computeJointJacobians followed by getJointJacobian, and computeJointJacobians internally does the equivalent of forwardKinematics, which C++ jointJacobian does not. In Nicolas' example, forwardKinematics is needed later in order to correctly retrieve data.oMi[i]. I opened an issue on the point, you can check #545

@nmansard ok

I think we have discussed enough about this. My suggestion:

Rebase topic/doc-v2 to bring it on par with the latest progress on devel
Address all comments made by me and @jcarpent
For the Jacobians in the inverse kinematics examples: let's freeze the discussion on the API. Here we have four options, listed from the ones I deem most to least desirable:

switch to computeJointJacobian + getJointJacobian in python
keep the two examples as they are, with a slight inconsistency between the two. In the text, just present the two methods as two alternative ways of achieving the same result.
use jointJacobian in C++, but add a call to forwardKinematics() in C++. Explain why this is needed.
use jointJacobian in C++, add a call to forwardKinematics() in C++, and also add a (useless) call to forwardKinematics() in python (notice that this would still not fully eliminate all inconsistencies).

merge in devel and close.
I would happily do the job myself, but the changes are in Nicolas' repo

Update: I will think about it by doing a PR on nicolas' repo.

Ping. I've done my part.You can find the changes in Nicolas repo, PR#4.

@gabrielebndn I marked some of the request changes of @jcarpent as resolved, but I am not sure of the other ones. Could you double check then merge?

Ok, so I marked as solved most of Justin's comments, except for 6:

4 in the code of i-inverse-kinematics.cpp . I preferred not to follow them. I think we can agree these are secondary and we should not focus on this piece of code
1 in overview-simple.py . I preferred not to follow it
1 About the supported systems. I mostly followed Justin's advice, mentioning Ubuntu 18.04 and Mac OS X, but I did not explicitly mention other systems. Probably I should have. What else do we support? Fedora, BSD and ... ? Maybe Nicolas can add a sentence when he makes his last modifications

Justin, if you agree with things as they are now, you can mark your unresolved changes as resolved yourself.

It seems fine for me now.


radosgw / RGW need configurable options like ssl_certificate

Hi,
I want to have in my ceph.conf config like this:
rgw frontends = "civetweb port=443s ssl_certificate=/etc/ceph/cog.pem"
There isn't currently a good way to add the ssl_certificate line (you can abuse some of the existing config parameters); could we have a radosgw_further_options variable or somesuch, please?
[I'm using your stable-2.1 branch]
Thanks.

This will be available in the next stable release. We should release stable-2.2 in a couple of weeks. Thanks for your patience.


🐛 Bug Report: Large zip breaking stream endpoint

📜 Description
Stream endpoint doesn't provide answer when embedded file in zip archive is long.
👟 Reproduction steps

Upload a zip file
Try chatting
docs_tester.zip

👍 Expected behavior
DocsGPT should provide an answer.
👎 Actual Behavior with Screenshots
No answer, stream endpoint breaks.

💻 Operating system
MacOS
What browsers are you seeing the problem on?
Chrome
🤖 What development environment are you experiencing this bug on?
Docker
🔒 Did you set the correct environment variables in the right path? List the environment variable names (not values please!)
No response
📃 Provide any additional context for the Bug.
No response
📖 Relevant log output
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🔗 Are you willing to submit PR?
None
🧑⚖ Code of Conduct

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

Looks like it happens because the file is not being chunked properly or at all when answering, thus resulting current context token overload

Hi, I would like to work on this issue.

When you upload a big zip file what answer should it provide?

Just shouldn't break. Basically make sure that it doesn't error out.
Try running it with the file attached.

I am interested to work on this issue.

I was playing around with the zip file and couple of different files, I found that its not an issue related to chunking of code, there is some issue with RstParser class, I did update the file extensions to text file, for that case, it was working fine.

Currently checking the Rstparser class to figure out the changes required.

The issue is with the implementation of rst parser, in each file, it looks for a header and a text below it, but for the zip file we are testing on, it is just a single file with no header available, hence it is not being chunked. This header and text breakdown thing also seems to be an issue for markdown parser. The file should be chunked based on tokens or bytes and this tuple implementation also need to be updated.

Yeah seems like thats the issue, lets add another token size handler to it maybe?

Can I direclty use nltk tokenizer or should write manual token counting and all? I will update the dependency list

Hey, I have updated the code and created a PR, can you review it and approve, I am new to open source contributions and do not know how it works, after making a PR. Open to feedbacks.

@dartpain Can you review my changes and provide feedback, and approve if implementation seems correct.


fix: skip UUID annotation webhook for clusters with nil topology

What problem does this PR solve?:
We only want this webhook to run on Clusters that are created with cluster topology. This also fixes an issue when an existing Cluster that was created before deploying CAREN and this webhook is moved to the bootstrap cluster.
Currently it fails the move with
  2024-08-14 11:14:55 ERR     err="unable to pivot to the to-cluster: error setting Cluster.Spec.Paused=true: action failed after 10 attempts: error patching Cluster default/e2e-preprovisioned-upgrade-573225644: admission webhook "cluster-validator.caren.nutanix.com" denied the request: missing cluster UUID annotation caren.nutanix.com/cluster-uuid"

Which issue(s) this PR fixes:
Fixes #
How Has This Been Tested?:

Special notes for your reviewer:

I'm also now wondering if we want the validate to fail when the annotation is missing. Its possible that an existing Cluster that was created using another ClusterClass that does not use CAREN and then when CAREN gets installed, it prevents those existing Clusters from being updated because of a missing annotation that won't even be used.
Should this webhook be smarter and only validate and default Clusters that are referencing a CC that is using CAREN?

I'm also now wondering if we want the validate to fail when the annotation is missing. Its possible that an existing Cluster that was created using another ClusterClass that does not use CAREN and then when CAREN gets installed, it prevents those existing Clusters from being updated because of a missing annotation that won't even be used.

I've updated the webhook to handle this use case by adding the annotation on update to clusters that use topology but do not have a UUID annotation already set. This wil handle clusters that are migrated to CAREN as well.

Should this webhook be smarter and only validate and default Clusters that are referencing a CC that is using CAREN?

This is pretty hard to figure out tbh so I don't think so, for now at least. Adding the annotation doesn't harm anything.
Ultimately I feel that this stable UUID should be part of CAPI core as an official annotation and will open an issue there to see if we can get traction.


Extend docs for diagnostics

Added the list of diagnostic sources
Described the OpenTracing with more details to manage expectations

Jezz, it's not properly rebased. Hold on a sec.


Autodetect XML-like languages script tag content language

This fixes #695.
@isagalaev, what are you think about this change?

The problem with this is that we introduce detection errors where there used to be none. On the other hand, non-javascript scripting becomes more common, so I think it might be a net positive. Let's merge it like this and see how it flies.


Organize code vertically by feature

Generally, it would make the file structure cleaner and the app would be easier to understand and navigate if files were grouped based on feature and/or functionality.
First off, your src/client/example folder contains an operations.ts file, so it would make sense to move this folder (and all other feature folders) to the top of src, since they contain both client and server code.
As another example, your components folder currently looks like this:
└── components
    ├── landing            # All Landing Page related Page, component, and helper files
    ├── ui
    └── some-other-files.tsx

I would do the same thing as with example and move landing a level up so that it is within src, and move the ui folder to client. If there are any components within the ui folder that are specific to certain features or functionality, you can move those to their respective feature folders, e.g. some-feature/components. There may be some overlap here and, for example, there may be reusable components that could potentially live in more than one folder. That's fine. Just choose the one that seems to make the most sense.
Also, you have many framer-motion related files strewn about. Because the template is so animation focused, I would suggest that these could all be consolidated into a folder called e.g. motion :
└── src
    └── motion
        ├── config
        ├── components
        └── other-motion-files.tsx

I would also move src/components/ui/motion.tsx here and rename it to something more descriptive like transitionsPresets.ts. Same with src/types/motion.ts. Since there is only one type, just delete that file and add the type to the top of the file where it would make the most sense to live, like transitionsPresets.ts perhaps.
These are just a few examples I could find. I'm sure there is more that could benefit from reorganization. You will probably know best but let me know if you have any questions or need any help here. We can also have a quick call on discord to clarify anything if you need.

@vincanger thanks for the guidance on these, it's looking way cleaner.

Looks great! @wardbox


Reorganized to satisfy Github 1000-file-per-dir limit.

Fixes Issue #19

I'm not sure this is necessary, since this only affects the web portal, see https://github.com/iconic/open-iconic/issues/19#issuecomment-191294716

Effectively as it stands not all 233 unique images can be viewed. Perhaps if they were renamed to move 2x, 3x, etc to the front of the filename, that would fix it. In any case necessary and optimal are not always one in the same.

The view is only blocked when viewing Github through the web interface. You still get full access to all of the files after running git clone. Is it vital that we can see each of the files through the web interface? That doesn't seem like a high priority to me...


How can I make it link initially to desc?

link_to_sorted "Email", :email
generates
sort=email_asc
How can I make it to be desc by default?

For some reason it uses the Sorted::SQLQuery parser so you can just do:
link_to_sorted "Email", 'email DESC'

PR welcome to change it to something more rational.
See also:

https://github.com/mynameisrufus/sorted-actionview/blob/master/lib/sorted/actionview/builder.rb#L10
https://github.com/mynameisrufus/sorted_app/blob/master/app/views/users/index.html.erb


Try QR code without URI

Some websites (Kraken) don’t have an URI prefix for their QR code.
The app could try to read if it a valid address if there is no prefix.

Definitely


Generation mode that changes hyperlinks from / to /index.html where applicable.

Usage Scenario: book stitching, using a hugo site as the place where the book content is drafted.
When stuff is served by Apache (etc), a GET of /foo/ is understood to mean a GET of /foo/index.html . All is good in this world.
When I'm handing a table-of-contents page to Calibre's CLI tool to stitch into a single PDF, the tool chokes.
../ is a link to a directory. Ignoring.

I can hack it via sed but would prefer to not hack. Thus can we have an option --expand-links-to-index-files.
I'll raise a similar request for an option in Calibre.

how about "uglyURLs = true"?

You can use uglyURLs, in any case, we are not doing anything more in this area.

--uglURLs does not do what I am asking for. I've just tested it. Behavior is as expected after reading the documentation for --uglURLs.

in any case, we are not doing anything more in this area.

Can you link to a policy document that gives more information please?


Purchase order

Can we rely on the list of items in the response being returned in any particular order? For instance, will the most recent purchases always be returned first?

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

ID: 3b95f043-9483-170e-3aef-fa675bfc0fb7
Version Independent ID: 819f2b1e-a55b-7b05-bf54-111eed1a9916
Content: Query for products - UWP app developer
Content Source: windows-apps-src/monetize/query-for-products.md
Service: unspecified
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

According to the engineering team that owns this service, no order is guaranteed in the response data. To determine the more recent purchases, you can use the acquiredDate field.


Cherry pick Remove HostBuilderContext parameter from AddBotRuntimeConfiguration() #5439  Merged

Replaced HostBuilderContext parameter in ConfigurationBuilderExtensions.AddBotRuntimeConfiguration() extension method by explicit EnvironmentName (optional) in order to support azure functions and dotnet core apps. (#5439)

@EricDahlvang @gabog @chrimc62 the CI is failing due to delta in the schema test files:

Failed Microsoft.Bot.Builder.Dialogs.Declarative.Tests.SchemaMergeTests.TestDialogResourcesAreValidForSchema(resourceId: "SwitchLanguageTurnActivity.test.dialog", resourceName: "D:\\a\\1\\s\\tests\\Microsoft.Bot.Builder.Dialogs."...) [1 ms]
  Error Message:
   System.AggregateException : One or more errors occurred. (tests.schema has changed when running tests on the build server.) (The following constructor parameters did not have matching fixture data: SchemaTestsFixture schemaTestsFixture)
---- System.InvalidOperationException : tests.schema has changed when running tests on the build server.
---- The following constructor parameters did not have matching fixture data: SchemaTestsFixture schemaTestsFixture

Hi @stevengum, I thought this passed so I merged, ping me tomorrow if there are still issues.

@gabog no problem, seems like we'll just have to keep an eye out for transient failures and find the root cause. The failures due to the tests.schema looked transient as it only occurred in one CI configuration, not all four.

OK @stevengum, from what I learned with another PR that failed, test.schema gets updated when you run the tests locally ( TestDialogResourcesAreValidForSchema run bf to update this) and if you forget to update test.schema as part of your PR it seems to fail for the Release build.
I am not sure why it fails for release and not for debug and why that failure was impacting this PR that doesn't update schema so yes, let's keep an eye on it.


decision tree has 2 redundant entries

2 and 4 is the same

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

ID: e6bba86f-f473-368c-e517-c02e4635119c
Version Independent ID: 66be4c19-ccf8-a0e2-31b1-95a9d677ba12
Content: What is Azure Data Explorer?
Content Source: data-explorer/data-explorer-overview.md
Service: data-explorer
GitHub Login: @orspod
Microsoft Alias: orspodek

Thanks for the feedback and bringing this to our notice. At this time we are reviewing the feedback and will update the document as appropriate.

Thanks @Holger1968 for noticing that. Have opened WI to track this fix. https://mseng.visualstudio.com/TechnicalContent/_workitems/edit/1896675
#please-close


Reads to be renamed

Dear Balázs,
I am interesting in building the mtDNA of a parasitic nematode species. It seems the software you’ve built is particularly well suited for that and I have tried to use it.
However I have an error message thrown at me : “entries in the read file must be renamed”. I cannot really understand why and cannot find out an appropriate format. Sampled read 1 looks like: @HS23_6953:2:1305:1339:52188/1
Do you have any hints?
Cheers,
G

Dear G,
Thank you for reporting this error.
If you run into this problem that means that you are using two read files for the run. When GRAbB gets two read files, then it assumes that they are paired. The way GRAbB handles read pairs is by read id names. I built in a test, so GRAbB checks whether it can find the proper pairs.
So in your example, the read id is @HS23_6953:2:1305:1339:52188/1 for the forward read, which means the reverse is probably @HS23_6953:2:1305:1339:52188/2. The problem is that it will look for the whole @HS23_6953:2:1305:1339:52188/1 string inside the reverse file and fail. The problem is that there is no space between @HS23_6953:2:1305:1339:52188 and /1.
There is a helper program, rename_fastq, which can rename the read files for you. Using the renamed read files should solve the problem.
Thanks for using GRAbB!
Cheers,
Balázs


Will the problem be solved?

the new game just added and It caused the process of getting the key to be a problem. Will it solve the problem?

Duplicate: https://github.com/masterking32/MasterHamsterKombatBot/issues/236


@types/lodash should be a dev dependency or a peer dependency

Hey! First of all I want to tank you for the awesome Mocking Frameworks that you made.
In package.json @types/lodash is referenced as a direct dependency and its loaded automatically in the ts-mockito depended projects.
It wouldn't be a problem if everyone would use lodash but that's not the case. At the compile the variable _ enters in conflict with other libraries that define the same variable name.
Resulting in a nasty, not so easy to debug :

Error "Duplicate identifier '_'"

As the users of this framework use only the compiled .js with the typing definitions the @types/lodash is not required into dependencies.

Thx, for reporting this. I will check it and fix it soon.

Should be fixed in https://github.com/NagRock/ts-mockito/releases/tag/v1.1.5


EPD unable to communicate beyond docker container

Issue Description
Running EPD's Deployment docker images allows building the ROS2 package but when deployed, I am unable to feed sensor_msgs::msg::Image via /virtual_camera/image_raw.
In other words, active ROS2 topics within the docker container are uninteractable.
I will be using this issue to track and capture as much details to resolve this swiftly.

Minimal Working Docker ROS2 Example
To prove that ROS2 works with docker, I have created the following minimal example to prove that ROS2 nodes instantiated within Docker containers are interactable.
Environment

Ubuntu 20.04 Focal Fossa
ROS2 Foxy Fitzroy

# Open a new terminal
docker run -it --rm osrf/ros:foxy-desktop ros2 run demo_nodes_cpp talker

# Open a new terminal
docker run -it --rm osrf/ros:foxy-desktop ros2 run demo_nodes_cpp listener

Verify
By running the commands above, this minimal example proves that ROS2 nodes are interacting between seperate docker containers.
Conclusion
Therefore, there is a way to interact with the input ROS2 topic for EPD when running with its own docker container.
Author's Comment
Running rqt_graph outside the docker containers for this minimal example reveals that it does not show the ROS2 nodes within the docker containers themselves.

References

https://answers.ros.org/question/370595/ros2-foxy-nodes-cant-communicate-through-docker-container-border/?answer=371655#post-id-371655

https://github.com/eclipse-cyclonedds/cyclonedds/issues/677

https://answers.ros.org/question/296828/ros2-connectivity-across-docker-containers-via-host-driver/?answer=298320#post-id-298320

Communicate between container and host on Ubuntu 20.04
docker run -it --rm --net host osrf/ros:foxy-desktop ros2 run demo_nodes_cpp talker

Objective: Read /chatter outside of container in host.

Setting the latest EPD v0.3.1 build on a Freshly Installed Native Ubuntu 20.04 sees that Dockerized EPD able to have Container <-> Host ROS2 interactions.
However, more work will be done to only allow sudo docker command runs in order to properly mimic functioning example in Freshly Installed Native Ubuntu 20.04.


Add Python 3.12 support

Before submitting
Please complete the following checklist when submitting a PR:

[ ] All new functions and code must be clearly commented and documented.

[ ] Ensure that code is properly formatted by running make format.
The latest version of black and clang-format-14 are used in CI/CD to check formatting.

[ ] All new features must include a unit test.
Integration and frontend tests should be added to frontend/test,
Quantum dialect and MLIR tests should be added to mlir/test, and
Runtime tests should be added to runtime/tests.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.

Context:
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues:

Blocked by https://github.com/PennyLaneAI/catalyst/pull/401

Duplicated PR https://github.com/PennyLaneAI/catalyst/pull/425! :) will not work on the other one and we can close it! But ensure that you consider all the suggested changes from the other PR into this one.


Minor sub-title change for repo

We can remove the wording about 'early access preview' :-)
"ArcGIS Runtime SDK for Android Early Access Preview Samples for Android Studio"

@andygup Yep, we are out of EAP :-)


fix typo

Current coverage is 91.87% (diff: 100%)

Merging #2525 into master will not change coverage

@@             master      #2525   diff @@
==========================================
  Files            96         96
  Lines         10867      10867
  Methods        1342       1342
  Messages          0          0
  Branches       1756       1756
==========================================
  Hits           9984       9984
  Misses          883        883
  Partials          0          0

Powered by Codecov. Last update 2f1ee21...462c036


[Datepicker] Ability to get the active date in the dateClass callback

Using the dateClass callback I want to set a custom class on the dates that are from the previous month or the next month. I can compare the date argument against my current month and that works fine as long as the pickers view is the current month. But as soon as I use the next/previous pager buttons and I'm looking at a different month that no longer works.
It would be useful to be passed the active date as another parameter in the dateClass callback. I'd be happy with the 1st day of the currently active month or something like that.
What I'm ultimately trying to do is hide the buttons for those dates completely with visibility:hidden so they aren't showing but they keep their size. That way the height of the calendar doesn't change when paging through the months.

I'm coloring days depending on data from the server, so maybe that helps you.
My component is watching month changes:
mounted() {
  // plugin doesn't expose month change event
  this.$watch(() => {
    return this.$refs.datePicker.currentMonth;
  },
  (val) => {
    this.onMonthChange(val);
  },
  );
},

and builds a new base date:
const month = val;
const year = this.$refs.datePicker.currentYear;
...

Sry but I don't understand: you always get the activeDay in your v-model, why do you want it from dateClass callback again?

@wxsms activeDay is probably not the right thing to call it. What would be useful is to know what month is currently showing. When you use the left/right arrow buttons to change to a different month. I no longer know what month is showing to compare the date sent to dateClass against.
I want to add a custom class to dates that are not in the same month as what is showing in the picker. The solution @asiamov posted seems like a good way to handle it but it'd be nice to have either a month change event or the current month/year passed in to the dateClass callback


Document new include_labels option

What does this PR do?
Document new include_labels openmetrics option.
Motivation
Additional Notes
Also bumps the minimum base package in each check that uses the new option.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

/azp run PR All


Function support?

Not sure if gemini in itself even has function support, but it would be nice as I'd like to use this for my Home Assistant pipeline

The SDK now supports function calls, but it requires developers with specific needs to contribute pull requests.


Improved custom boost-python detection on Windows

The custom boost-python detection code in externalpackages.cmake was failing on Windows when the boost libraries used versioned filenames (the default.) This patch build the actual name of the boost-python library from the boost-system one (which is already required by oiio) so that find_library has better chances of succeeding.
It also renames a variable which was interfering with the built-in FindBoost module.

So sorry for the long delay.  I will go ahead and merge this in the coming days if nobody objects.

I squashed this to a single commit and merged it into the trunk.


error TS2307: Cannot find module 'platform'.

Maybe is very simple but i cant make this work, i put this code in the app.module.ts
import * as platform from "platform";
declare var GMSServices: any;
but always give me the error
[19-07-10 16:40:58.282] (CLI) ERROR in src/app/app.module.ts(12,27): error TS2307: Cannot find module 'platform'.
so what i need to install? or where? i already tried many times, regards

Hi Cloud, I'm facing the same issue. Did you solve it?
I think that it's due to an outdated behavior to detect which platform the app is running on.
you can replace it with:
import * as platform from "tns-core-modules/platform";
if (platform.isIOS) {  // <--- Here you set the API KEY only when running on ios
GMSServices.provideAPIKey(globals.GOOGLE_API_KEY);
}

Hi Cloud, I'm facing the same issue. Did you solve it?
I think that it's due to an outdated behavior to detect which platform the app is running on.
you can replace it with:
import * as platform from "tns-core-modules/platform";
if (platform.isIOS) {  // <--- Here you set the API KEY only when running on ios
GMSServices.provideAPIKey(globals.GOOGLE_API_KEY);
}


Select chart views on history page

Description:
We have ability to show various sensor data history on web. Let's bring this to mobile.

Add available sensors as selectable list to three dots menu
When user selects sensor from menu it will be shown as chart on history page
Default selection: temperature, humidity, pressure
User can set all sensors visible if he wants
Charts will be listed from top to bottom, for example user selects battery, acceleration X and RSSI from list - this is the same order as it is in selection list so top chart will be battery middle acceleration X and bottom chart will be RSSI
If more charts shown on page, make page scrollable

iOS card: https://github.com/ruuvi/com.ruuvi.station.ios/issues/1503

Spec needed

Closing this with https://github.com/ruuvi/com.ruuvi.station/issues/1099


Bind volume permissions issue

Hello,
I am having issue with oracle database and startup scripts, I have
  volumes:
        - ./db/oracleDump:/docker-entrypoint-initdb.d/startup

And the issue is that ls -la /opt/oracle/scripts/startup/ shows that all files inside the folder are owned by root.
drwxr-xr-x 2 root   root       4096 Jun 26 09:41 .
drwxr-xr-x 4 oracle dba        4096 Jun 25 06:44 ..
-rwxr-xr-x 1 root   root         25 Jun 26 09:41 .gitignore
-rwxr-xr-x 1 root   root<PHONE_NUMBER> Jun 13 08:10 dump.dmp
-rwxr-xr-x 1 root   root        195 Jun 27 08:27 start.sh

So launching it is impossible, Had anyone solved issue like this?
I have tried chown -R oracle /opt/oracle/scripts/startup/ inside the container, but it does not affect owership of the files inside.
Thank you for your help in advance.

Hi @lkiii,
What's the error that you are getting? All the files have the read and execute flags set for the others group so the container should be able to execute them just fine.

I have start.sh with
impdp "'"/ as sysdba"'" schemas=mro,ramco,biotime,navision,loc,fltsql_read,fin_read,middleware,flt_read,reports,quantum directory=/opt/oracle/scripts/startup/ dumpfile=dump.dmp

dump.dmp is in the same directory and when I run it I get
Connected to: Oracle Database 12c Standard Edition Release <IP_ADDRESS>.0 - 64bit Production
ORA-39002: invalid operation
ORA-39070: Unable to open the log file.
ORA-39087: directory name /OPT/ORACLE/SCRIPTS/STARTUP/ is invalid

Another issue is that this script is not being triggered during the startup of the container.

Hi @lkiii,
Thanks a lot for the additional information, that helps solve the mystery.
The issue you are facing is I believe misunderstanding on how directories work in Oracle. The database has the concept of DIRECTORY data dictionary objects that can be granted to the appropriate users and so forth, rather than just being exposed directly to the file system via a path.
When you invoked impdp with directory=/opt/oracle/scripts/startup/ the database is actually looking for a DIRECTORY data dictionary object with the name /OPT/ORACLE/SCRIPTS/STARTUP/ rather than just a location on the file system. However, the object does not exist, which is the reason why you are getting ORA-39087: directory name /OPT/ORACLE/SCRIPTS/STARTUP/ is invalid
In order for you to import the data you will have to connect as SYS (or / as sysdba as you are already doing in impdp as well) and issue:
CREATE DIRECTORY MY_IMPORT_DIR AS '/opt/oracle/scripts/startup';

to create a new directory object MY_IMPORT_DIR (replace the name as you see fit) and then issue the impdp command referencing this directory:
impdp "'"/ as sysdba"'" schemas=schema1,schema2,etc directory=MY_IMPORT_DIR dumpfile=dump.dmp

Hope this helps!
Thx,

Thank you I will try the solution. I will let you know about the outcome.

I have also another issue that startup script is not being executed during launch. As I understood starting container should execute *.sh located in /docker-entrypoint-initdb.d/startup?

Yes, either /docker-entrypoint-initdb.d/startup or /opt/oracle/scripts/startup. /docker-entrypoint-initdb.d is actually just a symbolic link to /opt/oracle/scripts (see setupLinux.sh).
You should see a line similar to this:
Executing user defined scripts
/opt/oracle/runOracle.sh: running /opt/oracle/scripts/setup/01_shellExample.sh
Environment: Linux 51f09a648c8e 4.1.12-94.3.8.el7uek.x86_64 #2 SMP Fri Jun 30 10:40:13 PDT 2017 x86_64 x86_64 x86_64 GNU/Linux

Thank you for the clarification, but I have no clue why my script is not being launched.
I have this line in my logs, maybe it stops execution of all startup scripts?  /opt/oracle/checkDBStatus.sh: line 27: sqlplus: command not found-

Yes, that could be the reason. Did you change anything else besides createAsContainerDatabase=false when building the image?

Closing due to lack of response. Please reopen if needed.


Hardcoded Google DNS

Why is the google dns server hardcoded? Every client has dns servers configured already...
Especially google might be a privacy issue...

Hello, @vordenken I believe you got a management service build from a work-in-progress branch.
The latest commits in that branch removed that piece of code, and we will be offering some public DNS options from the Dashboard code. So nothing will be created automatically.

Cool, thanks for the quick reply!


MAINT: Removed 'move_files' function from 'reads.py'

move_files was a helper function of annotate_reads_card that moved the files that where created by the analysis, to the allele and gene sample directories
Not using a helper function here has the benefit of making the tests easier. And for #21, additional files have to be added to the CARDAnnotationDirectoryFormat and with this setup it will be easier.

Hi Santiago, Michal told me to assign PRs also to you to divide the work load better.
But don't fear this is a really small one, not like the last one.

Hey @VinzentRisch. I ❤ small PRs ;). I'll take a look tmo.

Thanks :)
And yes I will always include it in the future I just thought the changes are so small that you don't have have to try to run it.

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (94069c6) 66.74% compared to head (8f4b64f) 66.70%.

:exclamation: Current head 8f4b64f differs from pull request most recent head 65695c1. Consider uploading reports for the commit 65695c1 to get more accurate results

Files
Patch %
Lines

q2_amr/card/reads.py
0.00%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #25      +/-   ##
==========================================
- Coverage   66.74%   66.70%   -0.04%
==========================================
  Files          13       13
  Lines         836      835       -1
==========================================
- Hits          558      557       -1
  Misses        278      278

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Figure out how to set DotnetCLIVersion automatically

We should figure out how to maintain the tooling SDK version at the N-1 version automatically.

 I see that both 2.1.3 and 2.1.2 have  2.1.301 in DotnetCLIVersion.txt. 2.1.1 has 2.1.300.  Is it just a previous update? And in that case, why do 2.1.3 and 2.1.2 have the same cli version?

N-1 means the previous SDK release, and yeah, we haven't been consistent in applying it. (Hence this issue. :wink:) The SDK being any previous update seems ok since it means people should be able to use that N-x built-from source SDK to build N, but ideally we'd use N-1 for simplicity and getting new toolset features (e.g. for perf and reliability).
/cc @dseefeld

[Triage] The process to update the SDK is automated by a release pipeline.  PRs are automatically created as part of the release.


Add Shakefile as possible build script

Closes #650
I implemented the Shakefile to be run directly from stack.
# one-stop-shop build everything
stack Shakefile.hs build-all

Features:

stack is the only build requirement
shake is used instead of make
should work on all platforms (not fully tested yet)
dist target works on windows out of the box, making it easier to create releases for windows
submodules are synced / updated at the start -> works after clean clone of project

Quirks:

shake.yaml is necessary to start the stack Shakefile.hs ... command before submodules are loaded.
On first execution, a compiler is downloaded. However, no compiler is downloaded more than once.

Tested on:

[X] Fedora
[X] Arch Linux
[x] Windows
[ ] MacOS
[ ] NixOS (works if you add pure: false to shake.yaml)

I've noticed there is a downside in requirement to download 8.6.2 to run shake script despite me only wanting to check help and build 8.4.4. Not sure if it's fixable.

I've noticed there is a downside in requirement to download 8.6.2 to run shake script despite me only wanting to check help and build 8.4.4. Not sure if it's fixable.

Yeah, having to download the compiler and dependencies to actually install something else is annoying.

I wonder if it's possible to get "whatever ghc is already available" from stack...

Or at least the ghc I'm intending to use anyway

@lorenzo @Anrock It is intended that the same resolver as in the global stack.yaml is used. Therefore, nothing extra is downloaded.
The initial download of the resolver and the ghc is necessary to eliminate all build exterior dependencies beside stack, which is a major benefit on non-linux platforms.

@power-fungus Can you clarify?
Suppose i have clean stack install without any ghc downloaded yet.
If i run stack Shakefile ... then stack uses resolver from shake.yaml with ghc-8.6.2 to run Shakefile.hs itself. So ghc-8.6.2 is downloaded in order to run Shakefile in addition to whatever ghc i need for building hie.
If i run make hie-8.4.4 it executes stack --stack-yaml stack-8.4.4.yaml ..., so stack downloads ghc-8.4.4 and nothing else.
Where does stack.yaml come in in case with Makefile?

@Anrock make hie-8.4.4 still requires the cabal target, which uses the local stack.yaml that requires ghc-8.6.2 as well.

@fendor i see now! I thought it was just overlooked.

I implemented the dist target in a platform independent way, e.g. tar package and zlib package.
I assume that we can use zlib, since git also requires that library and should be present on the system, anyways.
However, I have to test that first.
Tested on:

[ ] Arch Linux
[ ] Fedora
[ ] Windows
[ ] MacOs

I added the icu-macos-fix target.
It would be nice if someone can validate that the Shakefile.hs also works on MacOs.

The pull request should be ready for reviews now.

I invoked it as ./ShakeFile.hs, and got
-*- mode: compilation; default-directory: "~/mysrc/github/alanz/haskell-ide-engine/" -*-
Compilation started at Fri Dec 21 09:51:28

./Shakefile.hs
Warning: No want/action statements, nothing to do
Build completed in 0.00s

I think it should have a default target, that prints out the available ones, as happens if you just type make.
Invoking it as ./ShakeFile.hs build-all does kick off some activity though.

I agree it should have a default, the help message seems like a good idea.
I have to check that. In my opinion, the last ghc should be 8.6.2, but I might be mistaken.

The point is that the hie-8.2.1, hie-8.4.3 etc are all copies of the same
file, the last hie exe compiled
On Fri, 21 Dec 2018, 13:32 fendor<EMAIL_ADDRESS>wrote:

I agree it should have a default, the help message seems like a good idea.
I have to check that. In my opinion, the last ghc should be 8.6.2, but I
might be mistaken.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/haskell/haskell-ide-engine/pull/991#issuecomment-449365277,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAZAB8bI8a8wkBBSQL6FUEyLDdDqRtQvks5u7MbbgaJpZM4ZN1kb
.

If I understand you correctly, this behaviour should be similar to the Makefile.
cp '$(STACKLOCALBINDIR)/hie' '$(STACKLOCALBINDIR)/hie-$*'

We could copy it directly from the compile path, just like when executing the dist target.

@alanz The issue about the hie-versions not properly being copied should be resolved now.

One thing though, current master builds for 8.6.3 too.

I will update the pull request to also include 8.6.3.
I would also update the Readme to use the Shakefile.

Is this ready for merge yet?

Technically yes, but the README.md needs updates that one should use ./Shakefile.hs. This should be done tomorrow.
Also, it is not clear to me if we should use  system-ghc: true in shake.yaml now. I would like to test it tomorrow.
Moreover, should the Makefile and BuilAll.ps1  be deleted in this pull request to avoid confusion?

I would say we should add it, and keep the existing Makefile and .ps1 file.
And add a section saying we now have experimental Shake build support, and to please try it and report.
Then we can see what kind of feedback we get.

Ok, then I will revert the changes to the README.md and add a section for the Shakefile.hs.

README is updated, but I did not manage to use the option system-ghc: true correctly.
I think, it is ready to merge now.

Given #1008, I wonder if we should not use an LTS that is not for GHC 8.6.3.
Otherwise it looks good.  I think the build is failing because hackage was being restarted.

Makes sense, LTS-12.25 looks fine?


Refactor Astria overview and components sections

I don't have the time to thoroughly review the copy right now but the structure and links seem fine.
I think we should submit this as is, with a note that we'll need to update the website in 3 places as there are existing links to https://docs.astria.org/local-rollup/1-introduction