Client Specific Apps Like Salesforce

Hi Tom,
Thank you for the great package Django-tenants. I am implementing a project with it, a saas app. The challenge I am facing now is to give the clients the possibility to add their own apps. What I mean, I could allow the client X to add an app "Finances", but when I will append the new app to the TENANT_APPS list, it will be available for all the other clients, not just the client X. That behavior would be annoying for the other clients who are not necessarily interested in it. Is there a way to do that with Django-tenants now? Just like Salesforce does, give the opportunity to add app/objects/tabs and so on for a specific client. I can create the logic for that the only concern is how to display the app only for the client who created it not all the clients.
I hope the question is clear and you will find a moment to help me.
Greetings from Haiti!

I don't know if this will fit your use-case perfectly, but I have a series of (mainly Boolean) fields on my Client model to enable and disable features
And then in my templates I have stuff like
{% if request.tenant.xyz_enabled %}
 XYZ
{% endif %}

@adnathanail thank you. I am not sure if that can make it. I need to avoid application  of migration of clients created specific app to all the other clients.

@wisnercelucus Ah ok why is that?

To give full power to the clients. That would allow the software to be used in any industry. One client could use it to build School Management System while another to build Hospital Management for example. So the industry could be dynamic. I would provide regular and standards apps like "Lead, Contact, Sales, etc, like Salesforce does" and they would add other apps according to their specific need. Do you know Salesforce? https://www.salesforce.com/

Ah wow - I figured it wouldn't be a problem running the migrations on all clients as the overhead of having loads of completely empty tables is practically nothing
However, if you are saying you want to give your users the ability to write their own custom apps then I can see why you wouldn't want peoples custom migrations running on everyones dbs
I would firstly say that I think you're taking on a ginormous task here! But these could help you out:
This is the code for djangos migrate command
https://github.com/django/django/blob/c60524c658f197f645b638f9bcc553103bfe2630/django/core/management/commands/migrate.py
And this is the code from this library that calls a django management command from code as opposed to the command line
https://github.com/tomturner/django-tenants/blob/bb133c4dbbe8e9c3d1db36ef913df1888f1be315/django_tenants/models.py#L156
And you can do things like
./manage.py migrate_schemas whatever_app --schema=xyz
So whenever a user created app needs to run a migration then only run it on their schema
You may get warnings saying you have unapplied migrations, but as long as custom user apps never depend on each other then I reckon it might work

@adnathanail Wow, helpful! I will dive into it. I know, it sounds insane what I want to do but this is how you know you are on the right way in the voice of Larry Page/Ellison. Loll

@adnathanail I will try both approaches, first, I will apply all migrations first and following your logic, I will decide to display or not the custom apps to their owners.
I only, and that was the concern about migrating system-wide, think that migrating for all schemas each time a client adds a new custom app can slow down the whole system. But, considering the second approach, what if in the future, I decide to add new standard apps application-wide, won't that force execute all pending migration to all schemas?
I know, there will always be challenges and I will have to come with my contribution to improving the tools but want to take advantage of experiences haha.

Sounds good!
I'm not 100% sure what you mean?
Do you mean when you run migrate_schemas from the command line in order to migrate your custom app on everyones sites?
Because you could just do ./manage.py migrate_schemas custom_app without specifying the schema and it would run the migrations for just that app across all schemas (as long as none of the models in that app depend on models in a customers app)

@adnathanail Great. Thank you. I am going to put it in practice.


[Question]: Playwright throwing error INVALID PRINT SETTINGS ERROR for large values of pageheight or pagewidth in pdf options. Is there any limit?

Your question
Hi,
I tried generating pdf with values pageHeight=8.5, pageWidth=1234567890 and also with pageHeight=1234567890, pageWidth=11 .
In both the scenarios playwright threw excpetion INVALID PRINT SETTINGS ERROR (Protocol error (Page.printToPDF): Show invalid printer settings error) .
I wanted to know what is the limit of page height and page width that are permitted?
Also, is there a way to generate a pdf with default pdf height and width in case such large values are sent?
I have seen such implementation in other implementations like CEF (chromium embedded framework) .

What are you trying to achieve by setting such page sizes?

Also, is there a way to generate a pdf with default pdf height and width in case such large values are sent?

No, you should compute desired page size when calling Page.pdf() at which point you could pick whatever default works for you.

What are you trying to achieve by setting such page sizes?
This is just a negative case scenario, in case the client enters such huge value.

No, you should compute desired page size when calling Page.pdf() at which point you could pick whatever default works for you.

Okay. Do we have any limits for page height and width ?  Can you suggest what are the max values of page height and width that we should provide to playwright

Do we have any limits for page height and width ? Can you suggest what are the max values of page height and width that we should provide to playwright

I don't see explicit limits in the Chromium base (well, the number has to fit into double). Also running with the numbers you provided does produce a pdf for me, can you share you code snippet?
    Path path = Paths.get("/tmp/out.pdf");
    page.pdf(new Page.PdfOptions().setPath(path).setWidth("1234567890").setHeight("1234567890"));
    long size = Files.size(path);
    System.out.println(size);

Here is the generated document info:

Hi Yury,
I missed to point out earlier, but I am passing the values of pageheight and pagewidth with "inches" format i.e.<PHONE_NUMBER> inches not<PHONE_NUMBER> pixels.
I think in your case by default it is creating pdf with<PHONE_NUMBER> pixels.
page.pdf(new Page.PdfOptions().setPath(path).setWidth("1234567890in").setHeight("1234567890in"));

The following snippet fails at 2^25 and passes at 2^24 so you can probably use "16777216in" as an upper limit, also bear in mind that there may be limitations imposed by the viewer.
  for (let i = 20; i < 30; i++) {
    const size = 1 << i;
    console.log(`trying size: 2^${i} = ${size}`);
    const outputFile = '/tmp/output.pdf';
    await page.pdf({path: outputFile, height: size + 'in', width: size + 'in'});
  }

For more precise estimate of the max limit you'll need to look into Chromium codebase or maybe ask this question in crbug.com


[FLINK-7524] Remove potentially blocking behaviour from AbstractClose…

…ableRegistry
What is the purpose of the change
This PR removes potential for blocking behaviour in all CloseableRegistries.
Verifying this change
Change is tested in AbstractCloseableRegistryTest::testNonBlockingClose.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes)

Documentation

Does this pull request introduce a new feature?  (no)

CC @aljoscha

Merged in 0073204b25.


Add support for Che when provisioning a Kabanero Foundation instance

The inclusion of Che as part of the Kabanero foundation allows enterprises to develop microservices leveraging remote Kubernetes resources.  Additionally, it will be a preparation step for enabling multi-architecture implementations, such as IBM p- and z-architectures.

Assignee: Melissa and Jay

My understanding is that this epic needs to move out to the next release (0.3.0) because we are not going to advertise the Che support until the Appsody extension is ready.

Hi Tim, noted documentation needed but didn't see a documentation work item linked in this epic, are we still looking for documentation assistance for 0.3?

The operator work for Che integration has completed.


Proxy password contains square bracket "[" will failed with error "Invalid IPv6 URL" from urllib3

Summary.
Expected Result
When I have passed a proxy with square bracket "[" in the password, the request session should work.
Actual Result
There will be error "Invalid IPv6 URL" generated by urllib3 which requests is based on.
Reproduction Steps
from requests import Request, Session

proxies = {
    'http'<EMAIL_ADDRESS>    'https': 'http://username:passw[ord@proxy-server:8081',
    }

s = Session()

req = Request('POST', auth_url, data=login_data)
prepped = s.prepare_request(req)
resp = s.send(prepped, proxies=proxies)
print(resp.cookies)

======The error message======
    resp = s.send(prepped, proxies=proxies)
  File "/usr/local/lib/python3.7/site-packages/requests/sessions.py", line 622, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/requests/adapters.py", line 410, in send
    conn = self.get_connection(request.url, proxies)
  File "/usr/local/lib/python3.7/site-packages/requests/adapters.py", line 303, in get_connection
    proxy = prepend_scheme_if_needed(proxy, 'http')
  File "/usr/local/lib/python3.7/site-packages/requests/utils.py", line 894, in prepend_scheme_if_needed
    scheme, netloc, path, params, query, fragment = urlparse(url, new_scheme)
  File "/usr/local/Cellar/python/3.7.0/Frameworks/Python.framework/Versions/3.7/lib/python3.7/urllib/parse.py", line 368, in urlparse
    splitresult = urlsplit(url, scheme, allow_fragments)
  File "/usr/local/Cellar/python/3.7.0/Frameworks/Python.framework/Versions/3.7/lib/python3.7/urllib/parse.py", line 417, in urlsplit
    raise ValueError("Invalid IPv6 URL")
ValueError: Invalid IPv6 URL
======End of the error message======

System Information
$ python -m requests.help

{
  "chardet": {
    "version": "3.0.4"
  },
  "cryptography": {
    "version": "2.2.2"
  },
  "idna": {
    "version": "2.7"
  },
  "implementation": {
    "name": "CPython",
    "version": "3.7.0"
  },
  "platform": {
    "release": "17.5.0",
    "system": "Darwin"
  },
  "pyOpenSSL": {
    "openssl_version": "1010008f",
    "version": "18.0.0"
  },
  "requests": {
    "version": "2.19.1"
  },
  "system_ssl": {
    "version": "100020ff"
  },
  "urllib3": {
    "version": "1.23"
  },
  "using_pyopenssl": true
}

This command is only available on Requests v2.16.4 and greater. Otherwise,
please provide some basic information about your system (Python version,
operating system, &c).

Hi @Hugh----, the stacktrace you provided shows this error is from the standard library urllib module rather than urllib3. RFC 3986 states the allowable ABNF for a URI's authority component which doesn't include gen-delims (i.e. square brackets).
You'll need to percent-encode the password before using it in the URI  in order to get things working as expected.


Issue building Dockerfile.kinetic

Hi Cartographer team!
I encounter this issue when I try to build a Docker container for ROS Kinetic using Dockerfile.kinetic
The command used to build it is docker build -t cartographer-kinetic -f Dockerfile.kinetic .
And the error message I get is below.
How can I begin troubleshooting this?
Thank you.
In file included from /catkin_ws/src/cartographer/cartographer/transform/timestamped_transform.h:21:0,
                 from /catkin_ws/src/cartographer/cartographer/mapping/pose_extrapolator_interface.h:28,
                 from /catkin_ws/src/cartographer/cartographer/mapping/pose_extrapolator.h:25,
                 from /catkin_ws/src/cartographer/cartographer/mapping/pose_extrapolator.cc:17:
./cartographer/transform/proto/timestamped_transform.pb.h:177:30: error: expected ‘}’ before end of line
./cartographer/transform/proto/timestamped_transform.pb.h:177:30: error: expected declaration before end of line
ninja: build stopped: subcommand failed.
Command failed, exiting.
Base path: /catkin_ws
Source space: /catkin_ws/src
Build space: /catkin_ws/build_isolated
Devel space: /catkin_ws/devel_isolated
Install space: /opt/cartographer_ros
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
~~  traversing 4 packages in topological order:
~~  - cartographer (plain cmake)
~~  - cartographer_ros_msgs
~~  - cartographer_ros
~~  - cartographer_rviz
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
The packages or cmake arguments have changed, forcing cmake invocation

==> Processing plain cmake package: 'cartographer'
==> Creating build directory: 'build_isolated/cartographer/install'
==> cmake /catkin_ws/src/cartographer -DCMAKE_INSTALL_PREFIX=/opt/cartographer_ros -G Ninja in '/catkin_ws/build_isolated/cartographer/install'
-- The C compiler identification is GNU 5.4.0
-- The CXX compiler identification is GNU 5.4.0
-- Check for working C compiler using: Ninja
-- Check for working C compiler using: Ninja -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler using: Ninja
-- Check for working CXX compiler using: Ninja -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Build type: Release
Files /catkin_ws/build_isolated/cartographer/install/AllFiles.cmake and - differ
-- Found PythonInterp: /usr/bin/python (found version "2.7.12")
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- Found GMock: gmock_main;-lpthread
-- Boost version: 1.58.0
-- Found the following Boost libraries:
--   iostreams
--   regex
-- Found required Ceres dependency: Eigen version 3.2.92 in /usr/include/eigen3
-- Found required Ceres dependency: glog
-- Performing Test GFLAGS_IN_GOOGLE_NAMESPACE
-- Performing Test GFLAGS_IN_GOOGLE_NAMESPACE - Success
-- Found required Ceres dependency: gflags
-- Found Ceres version: 1.12.0 installed in: /usr with components: [LAPACK, SuiteSparse, SparseLinearAlgebraLibrary, CXSparse, SchurSpecializations, OpenMP]
-- Found Lua: /usr/lib/x86_64-linux-gnu/liblua5.2.so;/usr/lib/x86_64-linux-gnu/libm.so (found version "5.2.4")
-- Found Protobuf: /usr/local/lib/libprotobuf.a (Required is at least version "3.0.0")
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.29.1")
-- Checking for one of the modules 'cairo>=1.12.16'
-- Found Sphinx: /usr/bin/sphinx-build
-- Configuring done
-- Generating done
-- Build files have been written to: /catkin_ws/build_isolated/cartographer/install
==> ninja -j16 -l16 in '/catkin_ws/build_isolated/cartographer/install'
<== Failed to process package 'cartographer':
  Command '['ninja', '-j16', '-l16']' returned non-zero exit status 1

Reproduce this error by running:
==> cd /catkin_ws/build_isolated/cartographer && ninja -j16 -l16

The command '/bin/sh -c cartographer_ros/scripts/install.sh --pkg cartographer &&     cartographer_ros/scripts/install.sh --pkg cartographer --make-args test' returned a non-zero code: 1

I am also experiencing this issue when I run "catkin_make_isolated --install --use-ninja" after following this tutorial.
nvidia@nvidia-desktop:~/catkin_ws$ cat /home/nvidia/catkin_ws/build/CMakeFiles/CMakeError.log
Determining if the pthread_create exist failed with the following output:
Change Dir: /home/nvidia/catkin_ws/build/CMakeFiles/CMakeTmp

Run Build Command:"/usr/bin/ninja" "cmTC_ac935"
[1/2] Building C object CMakeFiles/cmTC_ac935.dir/CheckSymbolExists.c.o
[2/2] Linking C executable cmTC_ac935
FAILED: cmTC_ac935
: && /usr/bin/cc   -rdynamic CMakeFiles/cmTC_ac935.dir/CheckSymbolExists.c.o  -o cmTC_ac935   && :
CMakeFiles/cmTC_ac935.dir/CheckSymbolExists.c.o: In function `main':
CheckSymbolExists.c:(.text+0x14): undefined reference to `pthread_create'
CheckSymbolExists.c:(.text+0x18): undefined reference to `pthread_create'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.

File /home/nvidia/catkin_ws/build/CMakeFiles/CMakeTmp/CheckSymbolExists.c:
/* */
#include <pthread.h>

int main(int argc, char** argv)
{
  (void)argv;
#ifndef pthread_create
  return ((int*)(&pthread_create))[argc];
#else
  (void)argc;
  return 0;
#endif
}

Determining if the function pthread_create exists in the pthreads failed with the following output:
Change Dir: /home/nvidia/catkin_ws/build/CMakeFiles/CMakeTmp

Run Build Command:"/usr/bin/ninja" "cmTC_7fa55"
[1/2] Building C object CMakeFiles/cmTC_7fa55.dir/CheckFunctionExists.c.o
[2/2] Linking C executable cmTC_7fa55
FAILED: cmTC_7fa55
: && /usr/bin/cc -DCHECK_FUNCTION_EXISTS=pthread_create  -rdynamic CMakeFiles/cmTC_7fa55.dir/CheckFunctionExists.c.o  -o cmTC_7fa55  -lpthreads && :
/usr/bin/ld: cannot find -lpthreads
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.


yum repository redirects to different URL

Hi.
It seems that RPMs hosted on the packages.adoptium.net yum repository are actually stored on some S3 buckets hosted by JFrog
While this can work in most situations, it complexifies in cases where servers are not allowed by firewalling to download from any URL.
I have allowed the flow from my server to https://packages.adoptium.net but it then redirects RPM urls to https://jfrog-prod-usw2-shared-oregon-main.s3.amazonaws.com/...
While I understand that you can't do much about this, could you at least document somewhere the exhaustive list of URLs to allow for the repository to be usable ?

I'm not sure we can easily document this as we are at the mercy of whatever JFrog has and they may change the server names that they use, so it would always run the risk of being out of date.


Implement a scheduled task to archive activity data

In a production deployment of IoT server, activities and relevant data keep accumulating over time.  So tables such as DM_OPERATION, DM_DEVICE_OPERATION_RESPONSE, and DM_ENROLMENT_OP_MAPPING will grow in size eventually. Due to this reason, reading activities will get slower.
To address this, a scheduled task needs to implemented to run once a day and move all records that are older than 30 days to another set of archival tables. This helps the system to retain activities only for the last month and will make the getActivities() endpoint faster to some extent.
There will be two scheduled tasks, both run exactly once a day.
Task #1 performs following on DM_OPERATION, DM_DEVICE_OPERATION_RESPONSE and DM_ENROLMENT_OP_MAPPING tables.

Read all records that are older than 30 days.
Insert them into corresponding tables in a new database. Those tables will be suffixed with "_arch".
From source tables, delete all records that are older than 30 days.

Task #2 performs following on the tables with "_arch" prefix.

Read all records that are older than 30 days( this retention period can be customized) and delete them.

These operations will be implemented with atomicity in mind. During the data move, records will be broken into a set of batches and they will be inserted and deleted in a transactional manner, batch after batch. We'll have to do some load tests and determine a suitable value for the batch size.

Closing the issue since it has been moved to product-iots repository at [1]
[1] https://github.com/wso2/product-iots/issues/1394


Add configurable fix for SRG-APP-000108-AS-000067

Rule Title:  The application server must alert the SA and ISSO, at a minimum, in the event of a log processing failure.
STIG ID: SRG-APP-000108-AS-000067
Rule ID: SV-46473r2_rule
Vuln ID: V-35186
Check Content:
Review application server log configuration.  Verify the application server sends alerts to the SA and ISSO in the event of a log processing failure.
If the application server is not configured to meet this requirement, this is a finding.

I'm going to leave this issue open for now until the gf-audit module has a (default) solution for this, but I believe this should be implemented outside of the gf-audit project and be a concern of whatever logging framework/system is used (org.geoint.gf.audit.log.AuditLogger implementation), as this rule is a broader concern and not just for GF.


Broken Link

Broken link - https://docs.substrate.io/rustdocs/v3.0.0/sp_api/trait.Core.html
From page - Runtime Apis

Hi @muharem ,
The recipes site is no longer being maintained. I wasn't able to spot the broken link. You can use this link thogh:
https://paritytech.github.io/substrate/master/sp_api/trait.Core.html


prettier config is missing

@Noitidart you probably used some prettier config to format the JS files but it's not in the repo, do you think you can add it?
Also, I'm planning to upgrade the lib to the latest version of the underlying native library, which will probably result in some non-exactly-small changes. You okay with that? Thanks.

Hey @vonovak I actually didn't use prettier, but I support going that way. We should also probably remove flow support and switch to TypeScript.
Thanks for asking. An upgrade would be awesome, but hopefully all existing features remain.

hi, thanks for comment. Removing Flow is currently not what I'm after - I just want to upgrade the underlying native lib and make any necessary changes to the api.

but hopefully all existing features remain

actually, there will be some removals. eg. progress dialog is removed and some more options are removed.


"window.newWindowDimensions" ignored if a Folder is opened inside the editor in Linux

I've got window.newWindowDimensions set to inherit. It mostly works, unless a folder is currently opened inside VS Code. In that case, if my window is maximized, it'll restore to the default position and state when I open an existing file from my system's file manager.

VSCode Version: 1.21.0
OS: Arch Linux with Cinnamon 3.6.7 and GTK 3.22.30.

@marc2377 can you share a screencast how this looks for you?

Sure.
Now, I did notice it only happens when the Code window is minimized, either by clicking the minimize button or by using the "Show desktop" button in the DE (or, alternatively, using the Ctrl + D shortcut)

@marc2377 I cannot reproduce from our insiders release, can you try?  You can give our preview releases a try from: https://code.visualstudio.com/insiders/

I'll try it, on Thursday probably. Will let you know.

I did that.

Strange.
I'm gonna set up a virtual machine and try a desktop environment other than Cinnamon then.

Any update?

Not yet but I'll check this weekend and report back


Version 1.0.0 crashes the app on load

Hello,
it took me a huge amount of time to realize that your version 1.0.0 crashes on launch the app.

Sorry for the late reply.
Is the issue still persists? If yes can you add some error logs?

The potential fix has been made in V 2.0.0,
Closing this due to no activity.


Create an online interpreter that can be embedded into online books / blogs

Similar to the interpreter/compiler system built into Rust By Example
This service could then be re-used by a number of tutorials / books.

That already exists: glot.io
Perl6.Party uses it. Click on any code example and you can modify/run it: http://perl6.party/post/Perl-6-S---Substitution-Operator

Did you just embed the glot.io iframe in your blog? Under the share link?
<iframe src='https://glot.io/snippets/emfuaraio4/embed' frameborder='0' scrolling='no' sandbox='allow-forms allow-pointer-lock allow-popups allow-same-origin allow-scripts' width='600' height='400'></iframe>
for example ^

That's another way of doing it, yes.
You can also use Glot.IO's API too (we got a module). Perl6.party just uses it under the hood to get execution results and codemirror for the frontend.


API-6545 - Add example for multi-file OAS specs

Maybe we could extract responses from components in application.yaml next time


Add software review badge

@pachamaltese sorry I clicked too rapidly and made a commit to main instead of a PR https://github.com/ropensci/censo2017/commit/3222e05cef355935600391155f4f6a4e381ba03a
Could you please knit the README for adding the badge? Sorry again.

Actually the badge should be
[![Status at rOpenSci Software Peer Review](https://badges.ropensci.org/414_status.svg)](https://github.com/ropensci/software-review/issues/414)

Furthermore, can you confirm you had seen

Should you want to acknowledge your reviewers in your package DESCRIPTION, you can do so by making them "rev"-type contributors in the Authors@R field (with their consent).  More info on this here.

And therefore chose not to? No judgement either way, I'm asking as I'm collecting info for the newsletter.

Dear Dr. Salmon
Thanks a lot. I'm slowly back into normal activity after medical rest.
Of course I'm creating the MD file again. The authors were added as rev, after asking them.
Best,
M Vargas


fix: array_resize null fix

Which issue does this PR close?

Closes #13200 .
Rationale for this change

What changes are included in this PR?
Added null check for array_resize

Are these changes tested?

Are there any user-facing changes?

I kept two null checks in the array_resize function: one in the array_resize_inner function and one in the general_list_resize function. The first null check is needed for the original problem of handling cases like SELECT array_resize(arrow_cast(NULL, 'List(Int8)'), 1);. This scenario can't be checked by the second null check because the array has a length of zero. With a length of zero, it conflicts with the null builder, as the null counts as a length of one, which isn't the expected value when constructing the result. This is due to the null array and the data array needing to have matching lengths.

error[E0599]: no method named `offsets` found for reference `&GenericByteViewArray<StringViewType>` in the current scope
   --> /Users/bytedance/.cargo/registry/src/index.crates.io-6f17d22bba15001f/arrow-string-53.2.0/src/length.rs:142:50
    |
142 |             Ok(bit_length_impl::<Int64Type>(list.offsets(), list.nulls()))
    |                                                  ^^^^^^^
    |
help: there is a method `offset` with a similar name
    |
142 |             Ok(bit_length_impl::<Int64Type>(list.offset(), list.nulls()))
    |                                                  ~~~~~~

For more information about this error, try `rustc --explain E0599`.
error: could not compile `arrow-string` (lib) due to 1 previous error
warning: build failed, waiting for other jobs to finish...

I got this error when running query in this branch 🤔

lgtm  @alamb

Thanks @jonathanc-n @findepi @jatin510


INT-B-19074

B-19074
Summary
Govt. came back after completion of B-19034 with a request to add "payment" to a sentence for the naval branches, and to add a paragraph before the final sign-off that informs the customer that they may need to fill out a DD Form 1351-2. This PR adds these to the .txt and .html templates, adds a relevant link to the constants.go file, and updates the tests to match the new content.
How to test
Follow the steps given in #12090 and ensure that the sent email content matches the new template.
If you chose a naval branch, the first line should now read (Change in bold): "You can now log into MilMove and view your payment packet; however, you do not need to forward your payment packet to finance as your closeout location is associated with your finance office and they will handle this step for you."
And all branches should now have a new line before the final one, that reads: "Please be advised, your local finance office may require a DD Form 1351-2 to process payment. You can obtain a copy of this form by utilizing the search feature at www.esd.whs.mil".

Also, just a question regarding the Origin address and Destination address - I've noticed that zip code is not included in the address when shown. Not sure if it matters, but just something I've noticed.
EDIT - Nvm I think I just answered my own question. I guess they don't really care about the zip.

EDIT - Nvm I think I just answered my own question. I guess they don't really care about the zip.


combining PR #724 (handling 25hr in ilevels) and #729 (studies with nonwear during night)

Hi @jhmigueles this PR attempts to merge the work I did in #729 with your PR #724, because they overlap in relation to g.analyse.perday I think it is best to try to combine before merging to master.
As part of this PR I made the following changes:

I moved your new code related to vari2 to a local function to createNewVari inside g.analyse.perday. This to isolate the new code from the rest of the code to ease readability and to reduce duplication.
I added if statements to make sure vari2 is only calculated if it is used later on, this to avoid unnecessarily doing the calculations.
Renamed object difference to deltaLength to make it a bit more specific.
I moved the creation of recorded down until after object varnum has been standardised to 24 hours.

For me the fixes I intended to make for #729 still work, but it may be good if you could also test that this does not break the fixes you tried to make in #724.
Two more general concerns:

When AskedToWear247 is set to FALSE the code assumes that the accelerometer is not worn during the night. However, in the recordings I have seen in the past with such study protocols there are almost always individuals who still wear the accelerometer 24/7 for some or all of the recording days. My concern is that those days will cause a significant bias because suddenly ilevels and time spent in inactivity will be calculated across the entire day. It would be good to find a way to account for this. One approach may be to detect these days and omit them. For example, by only including days with between 16 and 20 hours of wear time. However, the exact thresholds may be hard to justify, so maybe not. Further, it may be necessary to consider the timing of the non-wear.

There is a lot of new code, but this is not covered by unit-tests. Do you see a way to expand the unit-tests or create new unit-tests to also cover the new functionality?

What do you think?

Checklist before merging:

[x] Existing tests still work (check by running the test suite, e.g. from RStudio).
[ ] Added tests (if you added functionality) or fixed existing test (if you fixed a bug).
[ ] Updated or expanded the documentation.
[x] Updated release notes in inst/NEWS.Rd with a user-readable summary. Please, include references to relevant issues or PR discussions.
[ ] Added your name to the contributors lists in the DESCRIPTION and CITATION.cff files.

@jhmigueles The more I think about this the more I am wondering what this adds to the functionality GGIR already has.
When the accelerometer is worn during the night my advise used to be to use the following combination of input argument:

ignorenonwear = FALSE # to allow non-wear to be classified as sustained inactivity bout, in that way the night can be treated as sleep
HASPT.ignore.invalid = FALSE # to allow non-wear to be included in the guider detection, in that way night can still be included as sleep
includedaycrit =10 # to include days with a lot of non-wear
includenightcrit = 10 # to include nights with a lot of non-wear
do.imp = FALSE # leave all non-wear as they are. If calibration is poor then we could add an extra option to replace non-wear by zero.
relyonguider = TRUE # to let the guider decide on onset and waking time, because with all the possible sustained inactivity bouts during the day the guider is probably most reliable.

The advantages of this approach are:

This can handle recordings where participant did take of the sensor on some night but not on others.
Time spent in intensity levels can all stay in part 5 and does not become scattered across part 2 and 5.
The separation of sleep stays in part 3/4 and is not also done in part 2.
Avoids expansion of code and arguments.

Hi @vincentvanhees, here a couple of thoughts:

I think the main advantage of the "new" approach is that the user can get the estimates of the behaviors in part 2, and skip parts 3, 4, and 5, which are less meaningful considering that the participants were instructed not to wear the accelerometers during sleep.
Still, something needs to be done in those days in which participants still wear the accelerometer during nights even if they were instructed not to. So maybe your proposal makes more sense.

Also, I have tested your proposed arguments in some files in which participants were instructed not to wear the accelerometers during nights. Accelerometers were worn in the hip in this project, and I have tried this with both the HASPT_algo = "HDCZA" and HASPT_algo = "HorAngle".
HDCZA algo:

Even if I would expect this algorithm to work better when sensor.location = "wrist", it has performed well in this test. For this participant, who adhered to the protocol and only wore the accelerometer during waking hours, GGIR was able to find SPTs and focus the analysis on the waking hours:

BUT, if a certain day, the participant did not wear the accelerometer until afternoon, then the SPT is constrained to consider the wake up time at noon:

For other participant, who did not adhere to the protocol (i.e., he/she wore the devide during night), GGIR was not able to find the night, likely because of the high amount of nonwear time the day before. This makes that the day 7 in the pic below is included in the analysis and almost all the SPT is misclassified as inactivity:

HorAngle algo:

Here GGIR is not able to correctly identify the SPTs (even if accelerometers were worn on the hip). Neither in participants adhering nor in those who did not adhere to the protocol. Here an example:


Gitlab CI is not recognizing the git command used to retrieve branch name

I'm using this scan executer in Gitlab CI where its not able to recognize the branch names.Please refer the image.
https://github.com/koliveira15/nx-sonarqube/blob/d75fcf10e95db811250f0ea69981e768cb47b7fb/packages/nx-sonarqube/src/executors/scan/utils/utils.ts#L105

Following might help to understand why its not working in Gitlab CICD.
https://gitlab.com/gitlab-org/gitlab-runner/-/issues/4557

@ak274 seems like the issue is specific to Gitlab runner. Handling every type of CI environment is not in scope for the plugin. I would be open to changing the branches parameter to branch. That way consumers can get their own branch, respective of their CI environment.

I have observed that if you remove sonar.branch.name param from the scanner function altogether.. the GitLab CI automatically detect branch name and pull requests.

This behaviour also happens in GitHub's CI/CD pipelines. Running with branches: false the scanner tries to autoconfigure, but fails because sonar.branch.name is still passed, even though the value is '', resulting in the following:
...
INFO: Load project branches
INFO: Load project branches (done) | time=108ms
INFO: Load project pull requests
INFO: Load project pull requests (done) | time=113ms
INFO: Load branch configuration
INFO: Detected branch/PR in 'GitHub Action'
INFO: Auto-configuring pull request '2287'
INFO: ------------------------------------------------------------------------
INFO: EXECUTION FAILURE
INFO: ------------------------------------------------------------------------
INFO: Total time: 15.413s
ERROR: Error during SonarScanner execution
ERROR: A pull request analysis cannot have the branch analysis parameter 'sonar.branch.name'
ERROR:
INFO: Final Memory: 6M/24M
INFO: ------------------------------------------------------------------------

Opened PR #21 to omit the param if the option is false or unset.

@tstackhouse your PR has been published under version 2.1.1

Sweet! Thank you for pulling in my change. I’ll give it a try on Monday :)

Just to follow up, I pulled in the update and it worked like a charm :D The PR detection is now working perfectly in GitHub CI.  Thanks again!


Fail to install jsregexp on Windows11

Could you check whether these help?

Could you check whether these help?

yeah，I have tried almost every methods，including luarocks、MSYS2 and so on, however, every method has its own problem that I can not solove

Could you go into the directory luasnip is in and run make install_jsregexp manually, and post the whole output?
Unfortunately I don't have a windows setup currently, so I can't test this stuff myself :/

Could you go into the directory luasnip is in and run make install_jsregexp manually, and post the whole output? Unfortunately I don't have a windows setup currently, so I can't test this stuff myself :/

The meaning of Chinese in the output is that system cannot find the files

Hey, as this is probably the issue someone finds when googling it. I also had the same problems and solved it with the help of https://github.com/L3MON4D3/LuaSnip/issues/986#issuecomment-2060360732
Also to to make it more convinient I build a powershell script I can run.
What is needed:

Powershell (7) (winget install --id Microsoft.Powershell)
msys2 (https://www.msys2.org/ - https://github.com/msys2/msys2-installer/releases)

Than I added somewhere in my user home dir ( c:/users/MYUSERNAME/ps1scripts/luasnip_jsregexp_build.ps1) this powerhell script:
$find = '$(CC) $(LDFLAGS) $^ $(LDLIBS) -o $@'
$replace = '$(CC) $^ $(LDLIBS) -o $@ $(LDFLAGS)'

$file = '.\deps\jsregexp\Makefile'
If (!(Test-Path $file)) {
	throw '\deps\jsregexp not found'
}
(Get-Content $file).replace($find, $replace) | Set-Content $file
$file = '.\deps\jsregexp005\Makefile'
If (!(Test-Path $file)) {
	throw '\deps\jsregexp005 not found'
}
(Get-Content $file).replace($find, $replace) | Set-Content $file

$env:CHERE_INVOKING = 'yes' # Keep current Directory
$env:MSYSTEM = 'UCRT64' # https://www.msys2.org/docs/environments/
C:\msys64\usr\bin\bash -lc 'pacman --noconfirm -Syu'
C:\msys64\usr\bin\bash -lc 'pacman --noconfirm -S mingw-w64-ucrt-x86_64-luajit mingw-w64-ucrt-x86_64-make mingw-w64-ucrt-x86_64-gcc'
C:\msys64\usr\bin\bash -lc 'mingw32-make CC=gcc CFLAGS="-I/ucrt64/include/luajit-2.1 -O2 -Wall -fPIC" LDFLAGS="-shared -lluajit-5.1" install_jsregexp'

For the build option in lazy or wherever:
build = "pwsh " .. vim.fn.expand('$HOME/ps1scripts/luasnip_jsregexp_build.ps1'),

Run it manually in nvim-data/lazy/LuaSnip :
& "$env:USERPROFILE/ps1scripts/luasnip_jsregexp_build.ps1"

Hope that helps. :)

Hey, as this is probably the issue someone finds when googling it. I also had the same problems and solved it with the help of #986 (comment)
Also to to make it more convinient I build a powershell script I can run. What is needed:

Powershell (7) (winget install --id Microsoft.Powershell)
msys2 (https://www.msys2.org/ - https://github.com/msys2/msys2-installer/releases)

Than I added somewhere in my user home dir ( c:/users/MYUSERNAME/ps1scripts/luasnip_jsregexp_build.ps1) this powerhell script:
$find = '$(CC) $(LDFLAGS) $^ $(LDLIBS) -o $@'
$replace = '$(CC) $^ $(LDLIBS) -o $@ $(LDFLAGS)'

$file = '.\deps\jsregexp\Makefile'
If (!(Test-Path $file)) {
	throw '\deps\jsregexp not found'
}
(Get-Content $file).replace($find, $replace) | Set-Content $file
$file = '.\deps\jsregexp005\Makefile'
If (!(Test-Path $file)) {
	throw '\deps\jsregexp005 not found'
}
(Get-Content $file).replace($find, $replace) | Set-Content $file

$env:CHERE_INVOKING = 'yes' # Keep current Directory
$env:MSYSTEM = 'UCRT64' # https://www.msys2.org/docs/environments/
C:\msys64\usr\bin\bash -lc 'pacman --noconfirm -Syu'
C:\msys64\usr\bin\bash -lc 'pacman --noconfirm -S mingw-w64-ucrt-x86_64-luajit mingw-w64-ucrt-x86_64-make mingw-w64-ucrt-x86_64-gcc'
C:\msys64\usr\bin\bash -lc 'mingw32-make CC=gcc CFLAGS="-I/ucrt64/include/luajit-2.1 -O2 -Wall -fPIC" LDFLAGS="-shared -lluajit-5.1" install_jsregexp'

For the build option in lazy or wherever:
build = "pwsh " .. vim.fn.expand('$HOME/ps1scripts/luasnip_jsregexp_build.ps1'),

Run it manually in nvim-data/lazy/LuaSnip :
& "$env:USERPROFILE/ps1scripts/luasnip_jsregexp_build.ps1"

Hope that helps. :)

I'm on Windows 10. This solved it for me. Thank you so much! I was futzing around with mingw-w64-x86_64-lua and linker commands (of which I know little about). Should have come here first. My msys2 make output looked like this:
`$ make install_jsregexp
git submodule init
git submodule update
make "CC=cc" "INCLUDE_DIR=-I'/c/users/kortantic/AppData/Local/nvim-data/lazy/luasnip/deps/lua51_include/'" LDLIBS="" -C "deps/jsregexp"
make[1]: Entering directory '/c/users/kortantic/AppData/Local/nvim-data/lazy/luasnip/deps/jsregexp'
...
cc -shared jsregexp.o cutils.o libregexp.o libunicode.o  -o jsregexp.so
C:/msys64/ucrt64/bin/../lib/gcc/x86_64-w64-mingw32/13.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: jsregexp.o:jsregexp.c:(.text+0x11):
undefined reference to luaL_checklstring'
C:/msys64/ucrt64/bin/../lib/gcc/x86_64-w64-mingw32/13.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: jsregexp.o:jsregexp.c:(.text+0x24):
undefined reference to luaL_optlstring'
...
collect2.exe: error: ld returned 1 exit status
make[1]: *** [Makefile:17: jsregexp.so] Error 1
make[1]: Leaving directory`
I removed a couple hundred lines of missing lua references. Should be enough to help anyone seeing similar output. I'd recommend throwing the luasnip_jsregexp_build.ps1 in vim.fn.stdpath('config') or a child directory to help install neovim on another windows system if needed.

Is there any precompiled jsregexp for Windows?

Maybe fixed by #1252


Research custom allocators (jemalloc/mimalloc) if they could bring better performance

@mkaput commented on 2.07.2024, 13:11:23:

It looks like jemalloc brought improvements to Rust Analyzer:

https://github.com/rust-lang/rust-analyzer/issues/1441
https://github.com/rust-lang/rust-analyzer/pull/17007

It would be awesome to verify that one of these two would also improve CairoLS. For this, we need a reproducible benchmark code, though.

@mateusfigmelo commented on 17.10.2024, 01:27:15:

Could you plz give me some more crates like jemallocator, snmalloc, mimalloc and so on?
I need to pick global allocator crates as many as possible and select which allocator is the best for our project performance.

@mkaput commented on 17.10.2024, 17:31:42:

No idea. What were the errors?

@mateusfigmelo commented on 17.10.2024, 17:57:14:

FROM lukemathwalker/cargo-chef:latest-rust-1 AS chef
WORKDIR /app

# Builds a cargo-chef plan
FROM chef AS planner
COPY . .
RUN cargo chef prepare --recipe-path recipe.json

FROM chef AS builder
COPY --from=planner /app/recipe.json recipe.json
...

It is a part of my Dockerfile.
I got errors with this part in snmalloc-sys.

@mkaput commented on 17.10.2024, 18:25:33:

out of thin air, I have no idea what might be wrong

@mateusfigmelo commented on 17.10.2024, 18:35:31:

That's it.

@mateusfigmelo commented on 17.10.2024, 20:23:02:

I added command to install cmake in docker container, and built again.
As a result, the error is diappeard.
😄


System.ComponentModel.Win32Exception: System.ComponentModel.Win32Exception: Access is denied.   Stack Trace:at System.Diagnostics.Process.Kill()

Hii Folks,
I am running UI Testing testcases for Desktop application(C#,UWP).My testcases are getting passed and sometimes it fails due to
"Access is denied".I am actually using a method to close UI after running testcase.
[TestCleanup]
//to close ABC UI
public void UIClose()
{
//Close the ABCGrouper after running test cases
foreach (var process in Process.GetProcessesByName("ABCUserInterface"))
{
process.Kill();
}
}
Thanks & Regards
Md Azhar Ashraf
7908140840

Hi @alakshmi1030 If I remember correctly, Access is denied exception usually occurs when the RDP session gets minimized or disconnected.
Maybe try the solutions provided here: https://github.com/microsoft/WinAppDriver/issues/1510

@shoaibmansoor,It's a different issue.It happenened in odd times and can't predict its behaviour.
Your provided solution may be suitable for #1611.It has same issue like #1510

Do you get these when running remotely, locally or both? I have a similar issue when trying to clean up old winappdriver sessions remotely. Also, try calling a process.Dispose() before your process.Kill()

I don't get issue like this on local machine.Whenever I run testcases on remote machine using pipeline or manually on self hosted machine.It passed sometimes,and sometimes gets failed with this issue.Actually I debug in code and found out that after checking assert statement,we call a method to quit session in which software close,then  UIClose() executes.


Mouse appearing on second screen in Issue #26338 still occurs after waking up laptop from sleep

Microsoft PowerToys version
0.78.0
Installation method
WinGet
Running as admin
Yes
Area(s) with issue?
Mouse Without Borders
Steps to reproduce
Use mouse without borders as usual.
Put the host laptop to sleep and let the secondary laptop keep running.
Wake up the host machine after a period of time.
I then encounter issue #26338
✔ Expected Behavior
Mouse should hide
❌ Actual Behavior
Mouse appears on the center top of the other screen
Other Software
No response

ever since this update a few days ago been having similar issues 2024-02 Cumulative Update for Windows 11 Version 22H2 for x64-based Systems (KB5034765)

It's happening here in 0.78, I would much appreciate a workaround.


Increase options page width

Another issue with my slightly-wider font. Now inputs are wrapping to the next line:

Following the suggestion in Chrome's options page docs, I set an absolute width for the body. It's a little larger than the minimum size needed for everything to look right for me (430px minimum, I went with 450px).

This is the same fix as PR #7 but it sets the body width. I figured setting div.options width was more appropriate.


Zephyr firmware broken when using full RAM, atleast on nRF SoCs

Describe the bug
Non-functional samples in Zephyr when samples or application use all of the RAM, atleast in nRF SoCs. Hey, I want to use every bit of my precious SoC!
A sample (hello_world) worked correctly if there was ~37 bytes free on nRF52840, and similar free bytes on nRF53 network core. Tested on these two to fail. Have not tested if the failure is on all vendor SoCs supported by Zephyr.
Is there any hidden issues if this a symptom in Zephyr kernel/build, that can cause faulty applications that are in productions, in field and/or with customers.
Discovered as part of #80852.
Use the below snippet in hello_world sample, reduce the value 37 anything lower down to 0 to reproduce a non-functional hello_world sample.
The number 5696 is the RAM usage of the hello_world sample without the use of ram_fill variable.
Ozone debugger disconnects during calls to z_sys_init_run_level(INIT_LEVEL_PRE_KERNEL_2); i think, reporting Bus Faults.
diff --git a/samples/hello_world/src/main.c b/samples/hello_world/src/main.c
index c550ab461cb..632383f2fdc 100644
--- a/samples/hello_world/src/main.c
+++ b/samples/hello_world/src/main.c
@@ -6,6 +6,8 @@

 #include <stdio.h>

+static volatile uint8_t ram_fill[256 * 1024 - 5696 - 37] __attribute__((retain));
+
 int main(void)
 {
        printf("Hello World! %s\n", CONFIG_BOARD_TARGET);

To Reproduce
Steps to reproduce the behavior:

Change hello_world sample to use 100% RAM
mkdir -p build/hello_world; cd build/hello_world
cmake -GNinja -DBOARD=nrf52840dk/nrf52840 ../../samples/hello_world
ninja flash
See, there is no output on console

Expected behavior
Sample to functional as expect when using full RAM available on nRF SoCs.
Impact
Annoyance, run-time application failure on near RAM usages. No build failure of imminent issue.
Lost over 21 hours of this week's Bluetooth LE Audio testing, and possibly many man hours of those interested in its results and analysis of the failure.
Logs and console output
None
Environment (please complete the following information):

OS: Linux
Toolchain: zephyr-sdk-0.16.8
Commit SHA or Version used: c3466b14d09

Additional context
NA

Maintainers or release managers are free to remove the release blocker label, if this is an acceptable release notes "Known Issue" candidate.

The reason for this is that you have a heap enabled with the default value of use all available RAM, it seems that this will fail if there is actually no RAM left (arguably that is expected). CONFIG_COMMON_LIBC_MALLOC=y enables malloc (enabled by default), and CONFIG_COMMON_LIBC_MALLOC_ARENA_SIZE=-1 uses all remaining RAM for it. If you disable the heap or set the size to 0 then it successfully boots.

Triaged internally

cc @andyross @peter-mitsis , can you help take a look at this issue reported.
@cvinayak is this a regression?

@mmahadevan108 The issue might always have been, I have not tested if older release has same issue.

@cvinayak , I am removing the release blocker label from this PR as from discussion with the release team this does not seem like a release blocker.

@cvinayak can this be closed now?

I don't think so. The issue here is that we run out of RAM even when the reported RAM allocation is less than 100%. That means that we cannot find RAM issues at build time, and thus don't catch various issues in CI. This will also be an issue for users expecting to be able to use RAM up to 100%, where in reality they may only be able to use 99.5% or so.

As @Thalley has explained, the build should fail if 100% of the RAM cannot be utilized at runtime.

hey @cvinayak, I'll lower this to medium since it does not look like there's a plan to fix this for this release, feel free to readjust if you think that's not the case


Ternary conditional Support on Column formula

Hi,
Setup: Elasticsearch/Kibana v 7.5.2 | Plugin:
Great work with the plugin..
I am trying to plot a range with colors on the value feild.
value< 0.15 ## column value with RED color
value>1.5 ## column value with YELLOW color
0.15>value<1.5 ## column value with GREEN color
This worked fine for me, just that i need that extra Yellow to show along with red & green
col2>0.15? 'green': 'red'

I believe i fixed it, Thanks again to your documentation
Here it is
col2<0.15? 'red': 'green' and 1.5>col2>0.15? 'green': 'yellow'
Would welcome a better way to do though

Hi,
This is the formula that I advice for your need:
col2<0.15 ? 'red' : (col2>1.5 ? 'yellow' : 'green')

Worked like a charm! Thanks so much for that.
Not sure if i should continue with one more question, regarding the decimal precision
The value returned should be upto 3 decimal points.
Desired: 1.825
Computed column: 1.8246377548692734
Attached is the snapshot for your reference

Well, to have the expected column format, I invite you to:

have a first computed  column named color and no template
have a second computed column named “daily avg-FV”, with formula “col2”, “Number” format, “0,0.000” pattern, and this template:
{{value}}
have 3 in “hidden columns” field (to hide the color)

Perfect, however the  did not work for me.
Managed to get that sorted with this

{{value}}

Thanks for all the help @fbaligand . Happy to close the issue

Nice to see that it finally works as you expect!


[rubocop] Correct offenses

Now that rubocop is actually running on the integration rake files.
Mostly auto-corrections + splitting long lines

CI's passing, merging :)
Running RuboCop...
Inspecting 70 files
......................................................................
70 files inspected, no offenses detected


Add buttons in Markdown activity to show/hide Editor/Preview

Add buttons in the toolbar to show/hide Editor or Preview pane

Hello I am new to open source I would like to work on this how to get started

Hi,
A good start is to understand the Activities part of the README file. https://github.com/llaske/sugarizer#activities
You could also follow the video tutorial to create an activity https://www.youtube.com/watch?v=31CCAYy0pLE&feature=youtu.be
Finally you have to understand how the Markdown activity work.

Thanks for providing the youtube link it was very helpful can you please tell me where can I get more information about Markdown activity.

--Some questions--
1)When I hide preview area, should the input area expand and take the entire width?
2)Should it be allowed to hide both of the areas ie., input and preview at the same time?

It has no sense to hide both areas so the non hide area should take the whole space

Opened a pr, plz check

Fixed in https://github.com/llaske/sugarizer/commit/5f642290f14bc60c947decef239566a09d9ffca3


[Flowbroker] Cron node - Validations

@raulnegreiros:
This regex seems don't follow the field specification:
"The cron time string is five values separated by spaces. From the most left value to the righest one, we have minute (0-59), hour (0-23), day of month (1-31), month (1-12 or names), and day of week (0-7 or names, 0 or 7 are sunday). You may use multiple values separated by comma e.g 0,10,20,30,40,50, * * * * ."
The multiple values are not supported
https://github.com/dojot/flowbroker/pull/151#discussion_r284205052
And
https://github.com/dojot/flowbroker/pull/151#discussion_r284205422

Ticket analysis meeting (March, 6th): Medium.

Ticket analysis meeting (May, 22nd): Out of roadmap column.

The cron time expression is validated. If the expression is invalid, there is an indication that the cron node is not configured correctly.


Is this project live?

Is this project live or no longer being developed?

the intention is to continue it, but i've been busy with many things, so it is on hold for now.


Show more details about interface on hover

interface Interface {
    a: string
}

type Type = {
    a: string
}

// Hovering over `Interface` shows only `interface Interface`
let a:Interface

// Hovering over `Type` shows `Type = { a: string }`
let b:Type

In general interfaces are longer. For many types (e.g. Window) we'd end up showing an unreasonably large tooltip that would be unuseful.


Improve change log

API change check for Azure.Messaging.ServiceBus
API changes are not detected in this pull request for Azure.Messaging.ServiceBus


Account balance shows "Loading"

No changes! Account balance shows "Loading" after decrypting!

@Tobito19 I've checked this with a couple of my accounts and have no issues. If you would like to see your balance, please use a blockchain explorer such as https://etherchain.org/ or http://etherscan.io/


Migration to landscape2

Hi! 👋
We've been working on a new version of the landscape generating software, landscape2. This new version relies on the same landscape.yml file as the main data source, so nothing changes regarding adding or updating entries in the landscape.
The CNCF landscape has already been successfully migrated to landscape2, and we'd like to start migrating more landscapes 🙂
We are already building the OpenSSF landscape using landscape2, you can check it out at https://openssf.landscape2.io/. It'd be great if you could give it a try when you have a chance and, whenever you are ready, we can proceed with the migration.
The process is very simple and would involve the following steps:

[ ] Add/update some DNS entries [1]
[x] Stop background automated processes that write to this repository (i.e. CNCF bot)
[ ] Unlink this repository from Netlify
[ ] Update README file and clean up some files no longer needed in this repository
[ ] Setup the landscape2-validate-action to validate the landscape.yml file on PRs (optional)

We can take care of the last 4 points, you'd only need to update the DNS entries.
Landscape2 supports embedding a view with items in a category/subcategory in another website, but it works slightly different than landscape1. If parts of this landscape are embedded using this mechanism on other websites, we'd appreciate if you could let them know about how it works now.
Please ping me if you have any question 😉
[1] We’d need to know the hostnames you’d like to use to prepare our end to listen on them before switching the DNS (i.e. landscape.openssf.org). Once we are ready we'll provide you with the destination for the DNS entries so that you can update them.

Can we test it out with<EMAIL_ADDRESS>first before proceeding?

We are having issues with v2 and the at risk flag. We would like to monitor the v2 before trusting it completely as well. It looks like the landscape2 is pulling the correct at risk flagged member but not the new member it should.

Could you share some more details about the problem? Not sure what you mean by the "at risk flag", but we'd be happy to look into it if there's any issue 🙂

I see. But I'm afraid that's not related to landscape v2. V2 feeds from the same landscape.yml file and at the moment it's built daily. At build time, we'll use that data file as it is at that moment. But if there is something that's automatically adding or removing some entries from the data file they may not show up properly.
If you need us to build it manually at any time please let us know. https://openssf.landscape2.io it's being built daily and should be up to date (within that time frame).

Our current code for the landscape:

Currently, the only thing that the OpenSSF landscape is meaningfully used for is the display here: https://openssf.org/about/members/
That uses an iframe which displays this view: https://openssf.org/about/members/
Is there a similar view presented in landscape2?  I don't see anything displayed at https://openssf.landscape2.io/pages/members.

Yes! It works slightly different but it should be easy to migrate. Please see:
https://openssf.landscape2.io/embed-setup
https://github.com/cncf/landscape2/blob/main/README.md#embeddable-views

We have tried several iterations of this in the WP site and could not get it to look good on both mobile desktop with the current iterations. We can explore more later with time but do not have the bandwidth at the moment. Some of our testing is below
The code provided gives just one looks like this on desktop

meanwhile on mobile

The only way I saw to fix this was setting the size of each element and that was functional on Desktop

However mobile did not show anything still.

Hi @hythloda 👋
Did you enable iframe-resizer? It works slightly different than in the previous landscape (please see the demo referenced below).

[!NOTE]
In addition to the customization options available in the embed setup view, it's also possible to embed views using iframe-resizer. This feature can be enabled by adding iframe-resizer=true to the embed url (demo).

Is the source code of that website publicly available? If you still run into issues after trying the suggestion above, we'd be happy to take a look at it and submit a PR if needed 🙂

We did try the iframe-resizer and it still had a scroller

Hi @hythloda  👋

We did try the iframe-resizer and it still had a scroller

The code you shared seems to be working fine -no scroll-, you can see it in action in this codepen: https://codepen.io/cynthiasg/pen/QWPOdYm (I applied some minor changes to the styles to make it look similar to your current site, but it's mostly the same).
I'd be happy to assist further if you could point me to the site's source code. Please note that if you have multiple iframes you need to call iFrameResize(...) for each using the corresponding id. You can also see how this is being used in the Kubernetes or CNCF's website, just in case it helps.

Must be a wordpress plug issue then here is what I see after uploading it
screen-capture.webm

I have tried various ways of getting it to work with the rawJS block and it always looks the same with the small scrolling window. We have plans on potentially migrating to a new site and landscape1 works fine until such a time.
Attempt:

Attempt:

Attempt:


feat: change the default retry strategy

This PR updates the default strategy for retrying SendData commands. For details see #997
fixes: #997

Technically not breaking, but can result in worse behavior if node responses often get lost.


Make change "at cursor" (@cursor) or "within selection" (@selection)

Feature request: https://github.com/cline/cline/discussions/473
Feature
This PR add 2 new keywords to use in the prompt:

@cursor: when used, the code generated by the prompt should be added at the cursor current position in the current file opened in editor.
@selection: when used, the code generated by the prompt should replace the code in current selection in the current file opened in editor.
In both cases, the rest of the file should not be modified.

Exemples
@cursor
@cursor add a new paragraph that says "hello"

This should create a new paragraph exactly at the cursor position .
@selection
Select a whole function in the current file
@position add a parameter "max_items" and limit the behavior of the function regarding.

This update only the selected function with a new parameter "max_item" and change the code accordingly.
Additional info
When @selection is used and there is no selection in the code, fall back to @cursor.

Hi @PhunkyBob thank you for taking the time to open this PR, I'm impressed you got something like this to work! I'm avoiding adding too many options to the context mention menu, and I think this feature may not be as useful anymore now that Cline v3 does diff editing for targeted edits. You can now just ask him to replace a line or section instead. Apologies for getting back so late, I moved to a new city and was stretched a bit think this last month.


Fixed: checks for valid, but unconfirmed account + test function

Awesome, thanks for this contribution. Note that to keep the project's commit list cleanly tagged, I will not be able to merge this directly, instead I will do a rebase of the repository. I will close this PR once I apply the changes.
Thanks!

@frank-u I have incorporated the API fix and test you provided in the correct place in the history. Note that you will need to reset or re-clone the repository to see these changes (the commands are in the "How to work with the example code" section in the preface of the book).


Feature Check - ARM Builds

Hi! I think I can update the github action to build for both X86_64 and various ARM archs.
If I opened up a PR that did that, would you be likely to accept it?
Just wanted to check first in case there was a specific case against it.

I would be happy to accept a clean PR that adds ARM builds!


Keep track of when files are last seen in the cache

Summary
And remove cached files that we haven't seen for a certain period of time, currently 30 days.
For the last seen timestamp we actually use an u64, it's smaller on disk than SystemTime (which size is OS dependent) and fits in an AtomicU64 which we can use to update it without locks.
Test Plan

Looks reasonable to me! I'll let @MichaReiser approve

Looks good. Can you please add the test plan in the PR summary? Ideally, we would have a unit test for this as well but I can see how this may be difficult.

Added a test: https://github.com/astral-sh/ruff/pull/5214/files#diff-80a9c2637c432502a7075c792cc60db92282dd786999a78bfa9bb6f025afab35R532-R579.


Dependencies added programmatically by plugins are detected by the migration

Dependencies such as com.squareup.sqldelight:runtime-jvm or org.jetbrains.kotlin:kotlin-android-extensions-runtime aree added by their respective plugins and therefore do not need any manual intervention by the users

There's no way to ensure the dependencies have been added by a plugin or by code in the build files.
That means there's a need for rules for every case to be handled.
If the cases where we do this is limited, we can embed these rules, but there might be some that are very special cases.
Do you think we should provide an API to allow that to be user defined?

I think this issue is not happening with the new refreshVersionsMigrate Gradle task.
I think we'll soon drop the original fully manual migration task in favor of the aforementioned refreshVersionsMigrate Gradle task, which will make this issue obsolete.


Visual Studio 2022 Support

Since RC has been released this week and we even know a release date for RTM I wanted to know if / when you plan a version that's compatible with Visual Studio 2022 :-)

I've created a Pull Request for that
https://github.com/luminous-software/time-savers/pull/12


Use the vanniktech plugin to configure Maven publishing

I've looked at buildSrc, as I'm trying to re-configure to publish also to a private repository, and I think you can simplify the configuration a lot by using https://vanniktech.github.io/gradle-maven-publish-plugin/.
You can take antlr-kotlin as example:
https://github.com/lppedd/antlr-kotlin/blob/7fc16cceaf384d64191f86483bc5692356f6322e/antlr-kotlin-runtime/build.gradle.kts#L51-L61

Here is another example use in case it helps.


Fixing issues with debugger from VS Studio where after a few nested calls to features on other folders the debugger didn't stop on the breakpoint

Description
This was a painful one for me and I just stumbled upon the solution by change. When your feature files call other feature files the debugger was never great and most of the times didn't stop (especially if they were on other sub folders etc). It has always been painful for UI automation as we reuse a lot of features and functions etc.
The problem was that the path of the breakpoint was on this format: C:\test\feature\folder..\called-feature.feature and the breakpoints are all stored with absolute paths - C:\test\feature\folder\called-feature.feature . Paths.normalize() takes care of that. In addition VS Studio some times passes the path with the driver letter in lower case (e.g. c:\test\feature\folder\called-feature.feature) and so the lookup in the Map also fails.
Case is important because of the BREAKPOINTS.get(). I wondered for a few minutes if we should make it insensitive but that can create problems in Mac/Linux (although why would you have the same file with two different cases in the same folder).

Relevant Issues : (compulsory)
Relevant PRs : (optional)
Type of change :

[ ] New feature
[x] Bug fix for existing feature
[ ] Code quality improvement
[ ] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

fyi @ptrthomas @ivangsa

fyi @ptrthomas @ivangsa


plans for 1.1 release

Now that Python 3.8 is nearing final release, we need to make an official release here that incorporates the import fix from #436.
While we could do a 1.0.4 release just to include this fix, I think we have enough new things to go ahead with a 1.1 release.
There are only three open PRs marked 1.1:
#439: I want to get this bug fix merged. It is nearly there and I can help finish it soon.
#395 has not been updated recently and can be pushed back
#478 it is ready if anyone gets a quick chance to review, but not a big deal to push back if not
Other open issues are potentially removing the cameraman image and potentially adding a code of conduct. I don't think either of those necessarily should block a release, though.

All sounds good to me. CoC would be good; don't really care about the cameraman image, no one is complaining about that one. I'll have a look at gh-478.

If we have PyWavelets follow the recommendations in NEP29, it looks like we would drop Python 3.5 support from the 1.1.0 release. I don't have a strong opinion either way on dropping it now or soon after the release.

Perhaps good to drop it straight after v1.1.0, to not leave scikit-image or other downstream hanging.

Trying out wheel building over at MacPython/pywavelets-wheels#9
Once that is working, I think we should be ready to tag 1.1.0.

old issue, closing.


bug(indent): Incorrect indent guides for continuation indents

Did you check docs and existing issues?

[X] I have read all the snacks.nvim docs
[X] I have updated the plugin to the latest version before submitting this issue
[X] I have searched the existing issues of snacks.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.10.2
Operating system/version
Ubuntu 22.04.5 LTS x86_64
Describe the bug
The indent module displays multiple indent guides when the continuation indent is bigger than the regular indent.

Steps To Reproduce
Edit a file with a continuation indent, e.g.:
public class Main {
  public static void main(String[] args) {
    System.out
        .append('a')
        .append('b');
  }
}

Expected Behavior
Only one indent guide should be displayed under System.out. Other plugins (e.g., indent-blankline.nvim) handle this case correctly.

Repro
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
  spec = {
    {
      "folke/snacks.nvim",
      opts = {
        -- Indent guides without scope highlighting.
        indent = { scope = { enabled = false } },
      }
    },
    {
      "lukas-reineke/indent-blankline.nvim",
      main = "ibl",
      cond = false, -- Enable this and disable `snacks.nvim` for correct behavior.
      opts = {
        -- Indent guides without scope highlighting.
        scope = { enabled = false },
      },
    },
  },
})

I think I added this correctly, but do let me know if you see unexpected results.

Works great, thanks!


How to conduct HOTA MOTA IDF1 MT ML FP FN IDs FPS evaluation

For the results with CLEAR metrics, you can run tracking scripts [such as, mot17_track_cfg.py] to get the results.
For the results with HOTA metrics, you can refer to 'tools/eval_hota.sh', modify your own file paths, and run it.


visualizing the coverage, paint the basic blocks instead of instrumented lines

Is your feature request related to a problem? Please describe.
Current fuzzing coverage visualization approach works good for the experienced users but looks confusing for the newcomers.
Describe the solution you'd like
Let's highlight covered basic blocks instead of instrumented lines. From the UI perspective it means instead of the lines we'll highlight line1:col1-line2:col2 blocks.
The easiest first step I see is to focus on clang builds and:

Enable llvm-addr2line usage. It provides line:col pairs. Current addr2line gives only line.
Add column support to our symbolizer.
Add functionality to detect the basic block end. I think union of basic block starts and function ends will be a good first approximation (will it be enough?).
Add UI changes.

GCC coverage visualization is out of scope.
Additional context
The alternatives considered:

Move to clang coverage. Rejected because we don't want to lose our custom features line the ability to see the line context programs.
Reuse llvm perf counters based coverage. It looks interesting but will be quite invasive and will take the time for llvm changes.
Use llvm basic block sections. It will also be quite invasive because currently kernel doesn't support it.

CC @ramosian-glider @dvyukov

gVisor already supports range based coloring. It means the work scope is smaller than I expected.


Generic SignatureCheck Verifier

This PR introduces a new Verifier implementation calledSignatureCheck. It is similar to the existing Sr25519Signature except that it is generic over Substrate's notion of crypto Pair.

Am I missing something, or can we just ditch the Sr25519SignatureCheck struct and provide a type Sr25519SignatureCheck = SignatureCheck< sr25519::Pair>; with doc comments?

Yes, keeping the original Sr25519SignatureCheck is redundant at this point. I had originally left it there to show how you can easily use your own signature scheme even if it doesn't already exist in Substrate's code.
Maybe it makes more sense to do this for something like RSA https://lib.rs/crates/rsa which really doesn't exist in Substrate's code.
One potential problem is that the wallet uses sc-keystore, which only supports certain types of crypto not including RSA. So maybe the wallet just doesn't support rsa then; I guess that would be fine.


DO NOT MERGE! Fast Forward stable to 028de9d11341335c5446f9a00ffe3055aab81ea6.

This PR will be merged automatically once checks are successful.

Jenkins job manager/postsubmit passed


Main workflow discovery

Currently the main workflow if there are multiple in the same group of files is chosen in the 'dumb' way of the first one we come across.
Ideally we would look at the relationships between the workflows and choose the largest to display (or perhaps even add an additional step to ask the user if it is not clear (though this brings up additional issues with caching if future users wanted a different workflow - would the ID change somehow?)

Yeah, it would be really nice to be able to override this.

For example, by providing the full URL to the workflow:https://github.com/NCI-GDC/gdc-dnaseq-cwl/blob/master/workflows/dnaseq/transform.cwl


[Bug Fix] fix a bug in encode/decode in group

What do these changes do?

As titled.
Related issue number

Fixes #1965

Codecov Report

Merging #1966 (3efe79b) into main (2f7d1ec) will decrease coverage by 3.80%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1966      +/-   ##
==========================================
- Coverage   43.91%   40.10%   -3.81%
==========================================
  Files          89       89
  Lines        9820     9820
==========================================
- Hits         4312     3938     -374
- Misses       5508     5882     +374

Impacted Files
Coverage Δ

python/graphscope/tests/unittest/test_java_app.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/deploy/hosts/cluster.py
29.41% <0.00%> (-58.83%)
:arrow_down:

python/graphscope/analytical/app/java_app.py
24.24% <0.00%> (-44.95%)
:arrow_down:

python/graphscope/framework/utils.py
40.10% <0.00%> (-14.85%)
:arrow_down:

python/graphscope/framework/app.py
69.95% <0.00%> (-10.33%)
:arrow_down:

python/graphscope/client/rpc.py
81.48% <0.00%> (-8.34%)
:arrow_down:

python/graphscope/framework/graph_schema.py
52.45% <0.00%> (-7.11%)
:arrow_down:

python/graphscope/tests/conftest.py
48.21% <0.00%> (-4.65%)
:arrow_down:

python/graphscope/client/session.py
63.75% <0.00%> (-3.22%)
:arrow_down:

python/graphscope/analytical/udf/utils.py
36.17% <0.00%> (-3.20%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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


update Project URL on Puppet Forge and changelog

Could you please update the Project URL on https://forge.puppet.com/elastic/elastic_stack which currently points to https://github.com/jarpy/puppet-elastic-stack and it doesn't exist?
Could you also fill the changelog as is very useful to see how this project proceeds, please?

Good points, thanks. Both of these are important items that were lost in the transition from prototype to production.


sap-ui-core.js.map and all other core map files load -dbg files

OpenUI5 version:
I discovered this in SAPui5 1.44.16, and 1.46.9
I'm not sure if this problem also exists on openUI5, but I figured, this is the fastest way to report any UI5 bug in general.
It's a massive issue for me, using SAPUI5 in cordova, from visual studio.
All core-map-files reference dependencies to the -dbg.js files. even if I load a core without debugging enabled.

This means that my visual studio attempts to load -dbg files into it's debugging memory.
But as I always strip the -dbg files, that fails. so it attempts to load the minified files. which fails as well, because I strip those as well, in favour of the library files.

Up to version 1.44 (tested) none of this was an issue, as the core immediately loaded my preload files (as specified in my bootstrap). But now, with the map files referencing -dbg resources, my visual studio remains irresponsive for about 10 minutes, before I can start debugging.
Any ideas?

Either switch off source maps (IE11 fails on that, so maybe Visual Studio also fails), or remove the source maps from your Cordova package as well. As you remove the -dbg files, the source map files are without any value for you.

I followed up on that and removed all map files.
My loading process is now back up to the speeds I'm used to!
Thanks!
I noticed that the library_preload.js.map files suffered the same.
Is this by design, or a small lapsus in the release procedure?

This is by design.
The purpose of the sourcemap files is to allow the browser (or any debug environment like Visual Studio) to display the original sources instead of the name-mangled and merged bundle files (like sap-ui-core.js, library-preload.js or Component-preload.js in UI5). To make this possible, sourcemap files basically contain a mapping from text fragments in the bundle to the corresponding fragments in the source file(s). In addition, the browser needs to know where to find the original sources. Sourcemaps therefore refer to the original sources by URL. By using data:-URLs, the original sources even can be embedded into a sourcemap file. For UI5 we decided against this embedding to keep the footprint smaller, but refer to the already existing -dbg files.
If the -dbg files are removed from the deliverable, the sourcemap references become broken and the sourcemaps make no more sense. That's why I suggested to remove them as well in such a scenario.


Add is_auction and set it to true for lots by followed artists

Problem
Seeing unrelated lots in lots by followed artists page.
Solution
Follow up on changes in Gravity and MP we need to now set is_auction to true when asking for lots in live auctions by followed artists.
Depends on https://github.com/artsy/metaphysics/pull/923

Tests all pass, it's just a cross-fork PR, so it's fine to merge 👍


Unit test for edges

Add unit tests to make sure that parent-child relationships worked OK.

Done and merged into development branch


data: Update Viveksati5143.json

Added another milestone

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

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

Screenshots

Note to reviewers

Congrats on the new milestone, Vivek! 🎉

Thankyou!


[WIP] Zip64 writer+reader

Opened to track progress, not much to see yet. Closes #7103

@Sija Maybe you could advise. I changed Writer#add to be more predictable in its behavior ("always use data descriptor with a block, do not use data descriptor without and assume the Entry is correct"). The problem is in the reading of the entries. The current reader in the stdlib assumes it is possible to read the ZIP file "straight ahead", which in a way is a good idea if you want to, say, open a ZIP right when it is being uploaded to you by the client (neat feature! smart! no buffering!) It is however almost impossible to do it correctly in the presence of data descriptors - your only way to catch where the file entry body ends and the file descriptor begins is to scan for the 0x08074b50 byte sequence. In practice it can (and will) also occur in file bodies. It will also occur inside ZIPs inside ZIPs and so forth.
The consensus so far seems to be that it is not possible to open such ZIP files without having a seekable input. Our solution was to actually read the ZIP from the "tail", starting with the central directory, using that to recover offsets and sizes within the entire ZIP file being read, and then address those offsets directly. This works very well. We ended up only using the "straight ahead" reading for recovering truncated ZIP files where no central directory could be found.
With the changes I made, there is a bit of an embarrassing situation that we are trying to read our own written file with the reader and failing, because the reader is trying to do the impossible (do straight ahead read with data descriptors). Possible courses of action:

Scan for the data descriptor signature in the input stream and hope for the best (this is not a good approach, really)
Change the ZIP reader to require a seekable IO. This is a breaking API change, but a necessary one. Then rewrite the entire reading code for "from tail" seeks. Doable, but change will be much larger and this Reader class alone in zip_tricks is pushing a few hundred lines
Make the existing reader fail explicitly when it encounters an entry with bit 3 set in the local headers, since once you have hit one you don't really know how many bytes you need to skip to end up at the next entry

If I have to do both reader and writer changes at the same time I will pretty much have to do 2 + 3, and the changes are big. I would prefer doing first the writing correctly, then the reading in a separate PR. But the problem, if not addressed, will remain - that it is impossible to reliably forward-decode a ZIP file in presence of data descriptors. In zip_tricks we split the two code paths completely and it is up to the user of the code to decide whether to use the central directory reading approach (much faster, much more reliable and allows random access to entries) or use straight-ahead parsing (potentially unreliable, may lead to extracting corrupt data unless every entry is buffered over the entire remaining body of the ZIP and then verified for length).

@julik that's a good question!  In the meantime I wish you a great New Year! 🍾🎊🌲

I think that probably the best approach is to support both option 2 and option 3 as modes, with option 2 as the default.

@RX14 🤷♂ Let me know when there is a decision on this. I would be happy to maintain a Zip shard, but I got no idea what the current requirements regarding API compatibility requirements are etc

I still think a modular approach is better. It should be a library that can be included, or not, in to the stdlib.

@julik If there's a good reason, then compatibility should be broken, but obviously kept at a minimum. So changing to require a seekable IO is fine, since it's the most sane thing to do. I'd still like an option to read straight-ahead if the user can assert that there are no data descriptors, but that can come in a later PR if you wish.
If it's at all easier for you to split into reader+writer PRs then by all means do so.
Splitting into a shard can come later, we should move forward with this PR for now.

Note that Zip::File already requires an IO to be seekable. I think requiring that for Zip::Reader is fine too. In fact it also happens that in Go it's exactly like that.

Closing in favor of https://github.com/crystal-lang/crystal/pull/11396


Add canonical links to web site

The Druid website is hosted at a number of URLs. The canonical URL should be https://druid.apache.org/. To help Google out we should include canonical URLs: https://support.google.com/webmasters/answer/139066

I'll take a look

Adding to everything but the docs: https://github.com/apache/incubator-druid-website-src/pull/67


chore: refactor input loop in get_exec_command_args

Description
Refactor 3 loops on inputs in substrapp in only one
How has this been tested?
CI
Checklist

[x] changelog was updated with notable changes
[ ] documentation was updated

/e2e --tests sdk,substrafl,frontend

/e2e --tests sdk,substrafl,frontend

End to end tests: :hourglass_flowing_sand: STARTING

/e2e --tests sdk,frontend,substrafl --benchmarks mnist,camelyon

End to end tests: :hourglass_flowing_sand: STARTING


Update footer copy

What is the context?
When we become a bank, we need to align all legal text to mention Zopa Limited and Zopa Bank Limited.
What needs to be done?

The copy of the footer component in the React library needs to be updated
The link structure needs to be updated
The React component needs to be used in all Shopfront pages

The copy change required is
FROM
© Zopa Limited 2019 All rights reserved. 'Zopa' and the Zopa logo are trade marks of Zopa Limited. Zopa is a member of CIFAS – the UK's leading anti-fraud association, and we are registered with the Office of the Information Commissioner (No. Z8797078).
Zopa Limited is incorporated in England & Wales (registration number 05197592), with its registered office at 1st Floor, Cottons Centre, Tooley Street, London, SE1 2QG. Zopa Limited is authorised and regulated by the Financial Conduct Authority, and entered on the Financial Services Register under firm registration number 718925.
Contact Zopa on 020 7580 6060. Calls may be monitored or recorded.
TO
Zopa Limited is authorised and regulated by the Financial Conduct Authority, and entered on the Financial Services Register (718925). Zopa Bank Limited is authorised by the Prudential Regulation Authority and regulated by the Financial Conduct Authority and the Prudential Regulation Authority, and entered on the Financial Services Register (800542). Zopa Limited (05197592) and Zopa Bank Limited (10627575) are both incorporated in England & Wales and have their registered office at: 1st Floor, Cottons Centre, Tooley Street, London, SE1 2QG.
© Zopa Bank Limited 2019 All rights reserved. 'Zopa' is a trademark of Zopa Bank Limited.
Zopa is a member of Cifas – the UK’s leading anti-fraud association, and we are registered with the Office of the Information Commissioner (ZA275984, Z8797078).
The link structure needs to be changed
FROM

TO

I believe this was addressed and released here.

Thanks for updating this Lluis!


Impossible to use generator with field file

I encoutered the error "Invalid argument supplied for foreach()" when I use the command :

php artisan infyom:api post --fieldsFile=fields_sample.json

The file fields_sample.json is in the root directory, it's the original sample.

try
php artisan infyom:api Post --fieldsFile='resources/model_schemas/fields_sample.json'

Thanks for you reactivity phillip.
This fix doens't help me.

My bad I  meant to also help you with the foreach() problem but forgot to tell you. Everytime I  get this error there is a syntax error or wrong field format in my fieldsfile. check that or maybe post it and I  will take a look.

It's the original file ..

it seems like the problem is with JSON format. it should be in an array. Just like this file.
it should be like,
[
{
},
{
}
]

Thanks you @mitulgolakiya,the problem is indeed from comment, I post the correct file :

[
{
"fieldInput": "id:increments",
"htmlType": "",
"validations": "",
"searchable": false,
"fillable": false,
"primary": true,
"inForm": false,
"inIndex": false
},
{
"fieldInput": "title:string,20",
"htmlType": "text",
"validations": "required",
"searchable": true,
"inForm": true,
"inIndex": true
},
{
"fieldInput": "email:string",
"htmlType": "email",
"searchable": true
},
{
"fieldInput": "post_date:dateTime",
"htmlType": "date"
},
{
"fieldInput": "words:integer",
"htmlType": "number"
},
{
"fieldInput": "attachment:string",
"htmlType": "file"
},
{
"fieldInput": "password:string",
"htmlType": "password",
"inIndex": false
},
{
"fieldInput": "post_type:string",
"htmlType": "select:Type1,Type2,Type3,Type4",
"validations": "",
"searchable": true
},
{
"fieldInput": "post_day:enum",
"htmlType": "enum:Mon,Tue,Wed,Thu,Fri,Sat,Sun",
"validations": "",
"searchable": true
},
{
"fieldInput": "post_author_gender:string",
"htmlType": "radio:Male,Female",
"validations": "",
"searchable": false,
"fillable": true,
"primary": false,
"inForm": true,
"inIndex": false
},
{
"fieldInput": "body:text",
"htmlType": "textarea",
"validations": "",
"searchable": false,
"fillable": true,
"primary": false,
"inForm": true,
"inIndex": false
},
{
"fieldInput": "private_post:string",
"htmlType": "checkbox:Yes",
"validations": "",
"searchable": true,
"fillable": true,
"primary": false,
"inForm": true,
"inIndex": false
},
{
"fieldInput": "is_active:boolean",
"htmlType": "bool-checkbox:checked",
"validations": "",
"searchable": true,
"fillable": true,
"primary": false,
"inForm": true,
"inIndex": false
},
{
"fieldInput": "is_published:boolean",
"htmlType": "toggle-switch",
"validations": "",
"searchable": true,
"fillable": true,
"primary": false,
"inForm": true,
"inIndex": false
}
]

@jackleblack great. so is it now resolved or still you are getting problem ?

My bad guys I  should have removed that before push it was meant as notes of what was being done. I  meant to remove it but guess it got pushed before I  could remove it. Sorry.


setting keys and shortcuts concurrently results in problems

Microsoft PowerToys version
0.72.0
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce

As shown in the picture, I want to use Windows like a Mac. So I switch Ctrl and Alt. But I also set the shortcut to fix the Alt-Tab shortcut in the original place. But the setting is not working as imagined.
✔ Expected Behavior
Ctrl and Alt are switched. And the Alt-tab stays the same.
❌ Actual Behavior
Ctrl and Alt are switched. But Alt-tab is changed to Ctrl-tab.
Other Software
No response

This isn't concurrency, key to key takes priority and will directly effect the shortcut remapping.  We do directly call this out in the help document.


feat(questions): add image input question type

Type:

[ ] CI/CD: helm, docker & CI/CD adjustments.
[x] feature: new capabilities.
[ ] fix: bug, hotfix, etc.
[ ] refactor: enhancements.
[ ] style: changes in styles.
[ ] other: docs, tests.

What's the focus of this PR:
How to review this PR:
Related work items
Before submitting this PR, I made sure:

[ ] There is no lint error in the code
[ ] Build process passes successfully
[ ] There are some tests

Will it be stylable enough? Otherwise LGTM!

It will be stylable the same way as normal inputs for now. I think it should be enough, at least for DAZN. Later when we add the image previewer we may need to create a varian just for the image_input type


DS-2067: declare $modules before use

This avoids build errors in some bundlers, eg. with NextJS.
These changes have been tested by Dragan in his application and they compiled successfully.
Closes DS-2067

Looks good to me @smorrisods!

:tada: This PR is included in version 5.0.0-alpha.3 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


tandoor-recipes: Add runtime dependencies

Description of changes
Closes #287956.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.
Plus test to verify executing the main command.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/how-to-serve-tandoor-recipes-from-a-sub-path/32451/2


cmd/compile: nil pointer dereference for receivers has wrong line number

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

go version go1.6.3 darwin/amd64

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"

What did you do?

https://play.golang.org/p/qNwU6hT1YS
Force nil-pointer dereference on receiver in method.   The nil-dereference is reported as happening on the method call - line 25 - rather than on the actual deference which is inside the method (line 12).

What did you expect to see?

0
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0xffffffff addr=0x0 pc=0x20146]
goroutine 1 [running]:
panic(0x13f4a0, 0x1040a038)
/usr/local/go/src/runtime/panic.go:481 +0x700
main.main()
/tmp/sandbox599323765/main.go:12 +0xe6

What did you see instead?

0
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0xffffffff addr=0x0 pc=0x20146]
goroutine 1 [running]:
panic(0x13f4a0, 0x1040a038)
/usr/local/go/src/runtime/panic.go:481 +0x700
main.main()
/tmp/sandbox599323765/main.go:25 +0xe6

This is a duplicate of #10152 and caused by inlining.

What happens if you disable inlining? -gcflags=-N
On Fri, 22 Jul 2016, 10:24 David Howden<EMAIL_ADDRESS>wrote:

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

go version go1.6.3 darwin/amd64

What operating system and processor architecture are you using (go
env)?

GOARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"

What did you do?

https://play.golang.org/p/qNwU6hT1YS
Force nil-pointer dereference on receiver in method. The nil-dereference
is reported as happening on the method call - line 25 - rather than on the
actual deference which is inside the method (line 12).

What did you expect to see?

0
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0xffffffff addr=0x0 pc=0x20146]
goroutine 1 [running]:
panic(0x13f4a0, 0x1040a038)
/usr/local/go/src/runtime/panic.go:481 +0x700
main.main()
/tmp/sandbox599323765/main.go:12 +0xe6

What did you see instead?

0
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0xffffffff addr=0x0 pc=0x20146]
goroutine 1 [running]:
panic(0x13f4a0, 0x1040a038)
/usr/local/go/src/runtime/panic.go:481 +0x700
main.main()
/tmp/sandbox599323765/main.go:25 +0xe6
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/16462, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAcA87Ce-2j-sSqT0qzYnldkSJqrOFlks5qYA3JgaJpZM4JSVwQ
.

@davecheney Still incorrect with inlining disabled.
$ go build -gcflags=-N example.go
$ ./example
0
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x231f]

goroutine 1 [running]:
panic(0xda980, 0xc82000a0f0)
	/usr/local/go/src/runtime/panic.go:481 +0x3e6
main.main()
	example.go:25 +0x26f

-N does not disable inlining, -l does.

Yeah, sorry.  Just seen that too.  With inlining disabled it shows the correct position.
$ go build -gcflags="-N -l" example.go
$ ./example
0
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x205d]

goroutine 1 [running]:
panic(0xda920, 0xc82000a0f0)
	/usr/local/go/src/runtime/panic.go:481 +0x3e6
main.(*Example).Value(0x0, 0x0)
	example.go:12 +0x1d
main.main()
	example.go:25 +0x12a

My mistake, I thought -N was a superset of -l. Show's how much I've used
that flag.
On Fri, Jul 22, 2016 at 2:41 PM, Dominik Honnef<EMAIL_ADDRESS>wrote:

-N does not disable inlining, -l does.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/16462#issuecomment-234453101, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAcA3qTCn_f6XMCJELLMHRihEbrOV73ks5qYEoFgaJpZM4JSVwQ
.

Closing as dupe of #10152, please comment if you think it's not.


thcrap_configue is feaking out

i cant use it at all
it looks ok for a little bit, but it never stops listing the patches

it may also be worth noting that i am using wine, but this is the first time it has ever done something like this

Which OS do you use?
If this is MacOS, this is a known issue, and we have no idea how to fix it, at least until some developer with an Apple device manages to debug this. In the meantime, Touhou Patch Center's standalone patches should still work. If you want a different set of patches rather than just the English translation, just tell us, and I'll create such a package.
If this is Linux, could you please tell us the exact version of your distribution, and your version of Wine?

yeah this is OSX EL capitan with wine 1.7.53. i might be able to try to fix it, but i only know c#
maybye in the meantime i could have (all of) the standalone lang_en-lol patches? that's the main reason i got thpatch
i can try to make it work, i have a mac, i have a compiler, i have the curiosity

Alright, since the basic patch downloading and updating functionality should still be working, this should do:
thcrap bootstrapped en-lol.zip
Extract this into the thcrap directory and put the correct paths to the game executables into the .bat files. (Absolute paths have to match the virtual filesystem of Wine, but relative paths work too.)
After launching one of the .bat files, thcrap will then download the rest of the patch data for that specific game in the background, so it might take a while before everything appears as it should. You can check thcrap_log.txt for the progress of that. (thcrap does have an on-screen console that can be enabled in the en-en-lol.js file, but that'll probably be just as broken on OSX as thcrap_configure is.)

After the release of the next thcrap build, I'm actually planning to move the project away from Visual Studio I'll ping you once this is done (I guess it'll take at least a month from now).

visual studio is fine. the best way to fix it would be to put it on a windows form instead of a console. wine should work perfectly there. if i ever get my visual studio to work, i can help: i'm good at utilizing the .NET framework
and thank you.

Should be fixed with the new graphical configuration tool.


WIP - Select component

Screenshot

Designs
https://www.figma.com/file/RqENxZWAzGiEWM7COch1Sc/Radius-Design-Kit?node-id=542%3A6317

@turquoisemelon you can probably set pointer-events: none on the ExpandMore icon.

@turquoisemelon I would also recommend adding Select story to form-control/field.stories

A few of the stories need to be updated to account for bg name change. Check out chromatic for breaking changes.

A few of the stories need to be updated to account for bg name change. Check out chromatic for breaking changes. And the theme decorator.

Sure, will do 👍 One last thing I need to do is kitchen sinking the component to sample app. Should be good to go once I do those.

Also this can totally wait till Monday. No rush.