Updating 2.6 to 2.7 on tomcat7: works only if servlet container restarts.

I had a working 2.6 installation. After substitution of the war file on the tomcat7 container the servlet failed to start.
I downgraded back to 2.6 but got the same error.
So I upgraded again to the 2.7, the servlet works only after "service tomcat7 stop" + "service tomcat7 start". Without the stop+start, the release 2.7 fails to start (just as a further test I tried the 2.6 and the behaviour is the same: it starts only after a container stop+start); this is the catalina.out:

SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:/var/lib/tomcat7/webapps/gitbucket/WEB-INF/lib/logback-classic-1.0.13.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:/var/lib/tomcat7/webapps/gitbucket/WEB-INF/lib/slf4j-jdk14-1.6.4.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type [ch.qos.logback.classic.util.ContextSelectorStaticBinder]
04:33:12.245 [http-bio-8080-exec-101] INFO  org.quartz.impl.StdSchedulerFactory - Using default implementation for ThreadExecutor
04:33:12.251 [http-bio-8080-exec-101] INFO  org.quartz.simpl.SimpleThreadPool - Job execution threads will use class loader of thread: http-bio-8080-exec-101
04:33:12.268 [http-bio-8080-exec-101] INFO  o.quartz.core.SchedulerSignalerImpl - Initialized Scheduler Signaller of type: class org.quartz.core.SchedulerSignalerImpl
04:33:12.268 [http-bio-8080-exec-101] INFO  org.quartz.core.QuartzScheduler - Quartz Scheduler v.2.2.1 created.
04:33:12.269 [http-bio-8080-exec-101] INFO  org.quartz.simpl.RAMJobStore - RAMJobStore initialized.
04:33:12.270 [http-bio-8080-exec-101] INFO  org.quartz.core.QuartzScheduler - Scheduler meta-data: Quartz Scheduler (v2.2.1) 'DefaultQuartzScheduler' with instanceId 'NON_CLUSTERED'
Scheduler class: 'org.quartz.core.QuartzScheduler' - running locally.
NOT STARTED.
Currently in standby mode.
Number of jobs executed: 0
Using thread pool 'org.quartz.simpl.SimpleThreadPool' - with 10 threads.
Using job-store 'org.quartz.simpl.RAMJobStore' - which does not support persistence. and is not clustered.
04:33:12.270 [http-bio-8080-exec-101] INFO  org.quartz.impl.StdSchedulerFactory - Quartz scheduler 'DefaultQuartzScheduler' initialized from default resource file in Quartz package: 'quartz.properties'
04:33:12.270 [http-bio-8080-exec-101] INFO  org.quartz.impl.StdSchedulerFactory - Quartz scheduler version: 2.2.1
04:33:12.551 [http-bio-8080-exec-101] INFO  o.scalatra.servlet.ScalatraListener - The cycle class name from the config: ScalatraBootstrap
04:33:12.591 [http-bio-8080-exec-101] INFO  o.scalatra.servlet.ScalatraListener - Initializing life cycle class: ScalatraBootstrap
Dec 30, 2014 4:33:13 AM org.apache.catalina.core.StandardContext startInternal
SEVERE: Error listenerStart
Dec 30, 2014 4:33:13 AM org.apache.catalina.core.StandardContext startInternal
SEVERE: Context [/gitbucket] startup failed due to previous errors
04:33:13.357 [http-bio-8080-exec-101] INFO  o.scalatra.servlet.ScalatraListener - Destroying life cycle class: ScalatraBootstrap
04:33:13.357 [http-bio-8080-exec-101] INFO  org.quartz.core.QuartzScheduler - Scheduler DefaultQuartzScheduler_$NON_CLUSTERED shutting down.
04:33:13.358 [http-bio-8080-exec-101] INFO  org.quartz.core.QuartzScheduler - Scheduler DefaultQuartzScheduler$NON_CLUSTERED paused.
04:33:13.358 [http-bio-8080-exec-101] INFO  org.quartz.core.QuartzScheduler - Scheduler DefaultQuartzScheduler$_NON_CLUSTERED shutdown complete.
Dec 30, 2014 4:33:13 AM org.apache.catalina.loader.WebappClassLoader clearReferencesJdbc
SEVERE: The web application [/gitbucket] registered the JDBC driver [org.h2.Driver] but failed to unregister it when the web application was stopped. To prevent a memory leak, the JDBC Driver has been forcibly unregistered.
Dec 30, 2014 4:33:13 AM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads
SEVERE: The web application [/gitbucket] appears to have started a thread named [DefaultQuartzScheduler_Worker-1] but has failed to stop it. This is very likely to create a memory leak.
Dec 30, 2014 4:33:13 AM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads
SEVERE: The web application [/gitbucket] appears to have started a thread named [DefaultQuartzScheduler_Worker-2] but has failed to stop it. This is very likely to create a memory leak.
Dec 30, 2014 4:33:13 AM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads
SEVERE: The web application [/gitbucket] appears to have started a thread named [DefaultQuartzScheduler_Worker-3] but has failed to stop it. This is very likely to create a memory leak.
Dec 30, 2014 4:33:13 AM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads
SEVERE: The web application [/gitbucket] appears to have started a thread named [DefaultQuartzScheduler_Worker-4] but has failed to stop it. This is very likely to create a memory leak.
Dec 30, 2014 4:33:13 AM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads
SEVERE: The web application [/gitbucket] appears to have started a thread named [DefaultQuartzScheduler_Worker-5] but has failed to stop it. This is very likely to create a memory leak.
Dec 30, 2014 4:33:13 AM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads
SEVERE: The web application [/gitbucket] appears to have started a thread named [DefaultQuartzScheduler_Worker-6] but has failed to stop it. This is very likely to create a memory leak.
Dec 30, 2014 4:33:13 AM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads
SEVERE: The web application [/gitbucket] appears to have started a thread named [DefaultQuartzScheduler_Worker-7] but has failed to stop it. This is very likely to create a memory leak.
Dec 30, 2014 4:33:13 AM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads
SEVERE: The web application [/gitbucket] appears to have started a thread named [DefaultQuartzScheduler_Worker-8] but has failed to stop it. This is very likely to create a memory leak.
Dec 30, 2014 4:33:13 AM org.apache.catalina.loader.WebappClassLoader clearReferencesThreads
SEVERE: The webSLF4J: Class path contains multiple SLF4J bindings.

I used tomcat 7 to undeploy 2.7 and re-deploy 2.8. It was unable to startup again. Looking through the logs, the database file was locked by the tomcat process. Undeploying the servlet was not enough to free the database file so I had to stop and start tomcat in order for the new deploy of 2.8 to work.

@neandrake Yes, shutdown Tomcat at first to upgrade GitBucket.


github workflows without apt-get update will fail

Unless an apt-get update is run before the workflow, the workflow may (100% hit for me) fail.
Fixed by #1949

FYI issue

@markdedeuge were you seeing these failures when trying to run the github workflows on your local machine?

These were workflows running 'on github' as part of one of the docker PRs I had up.
Its caused by the github images (ubuntu etc) having some older sources cached. So unless you update them, certain packages will cause this issue.
Some google-ing revealed that this problem is not new and has happened in the past.
@SchrodingersGat i'll raise a PR today that sets the build images for each test to have a standard set of packages which should be easier to maintain.

This does not seem to impact the CI builds, I'll let @markdedeuge re-open if they are keen but otherwise, close for now.


Narrator does not announce position number of the variables mentioned in the expression.

A team member will update this issue with more details soon.

This is your friendly Microsoft Issue Bot. I created this issue automatically as requested by a team member.


[decomp] Handle another case of integer constant

(shl x 16) is sometimes used to generate an integer constant.

Pull Request Test Coverage Report for Build 944905953

4 of 9 (44.44%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 69.302%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

decompiler/IR2/bitfields.cpp
4
9
44.44%

Files with Coverage Reduction
New Missed Lines
%

goalc/listener/Listener.cpp
3
84.42%

Totals

Change from base Build 944817787:
-0.01%

Covered Lines:
36466

Relevant Lines:
52619

💛  - Coveralls


"Cilibia", "Gara Cilibia", "Mânzu", "Movila Oii: and "Poșta" are listed in the drop-down menu

Description:
"Cilibia", "Gara Cilibia", "Mânzu", "Movila Oii: and "Poșta" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Cilibia" in the search bar.

Expected results
The location is listed in the drop-down menu as "Cilibia, Buzău(Cilibia)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Mânzu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Mânzu, Buzău(Cilibia)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Movila Oii" in the search bar.

Expected results
The location is listed in the drop-down menu as "Movila Oii, Buzău(Cilibia)".

Step 8
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 9
Type "Poșta" in the search bar.

Expected results
The location is listed in the drop-down menu as "Poșta, Buzău(Cilibia)".


Remove unused pixel

Task/Issue URL: https://app.asana.com/0/inbox/361428290920650/1145593691874886/1145476565684961
Description:
Removes an unused pixel from the project.
Steps to test this PR:
An easy one! Just run the app & check that tests still pass.

Internal references:
Software Engineering Expectations
Technical Design Template

Thanks for the review!


BailHook default handler

What Changed
Added a default handler to SyncBailHook call to automatically handle non-bail cases
Why
Provide a default behaviour for when all the taps continue
Todo:

[ ] Add tests
[ ] Add docs
[ ] Add release notes

Release Notes
Allows for SyncBailHook to call with a default handler for when the taps do not bail and return nothing

📦 Published PR as canary version: 0.14.1-canary.33.352-SNAPSHOT

:rocket: PR was released in v0.15.0 :rocket:


[Bug] bubop - Not enough arguments provided

Trying to launch the latest version of the software fails.
System:

python v3.8.8
KUbuntu 21.10

Steps to follow:

pip3 install --user git+https://github.com/bergercookie/taskwarrior-syncall
(FYI, it installs bubop 0.1.6a)

~/.local/bin/tw_gcal_sync -c "GCAL_CALENDAR" -t "TW_TAG"

I get the following error:
Traceback (most recent call last):
  File "/home/petemir/.local/bin/tw_gcal_sync", line 11, in <module>
    sys.exit(main())
  File "/home/petemir/Apps/anaconda3/envs/newbase/lib/python3.8/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/home/petemir/Apps/anaconda3/envs/newbase/lib/python3.8/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/home/petemir/Apps/anaconda3/envs/newbase/lib/python3.8/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/petemir/Apps/anaconda3/envs/newbase/lib/python3.8/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/home/petemir/.local/lib/python3.8/site-packages/taskwarrior_syncall/scripts/tw_gcal_sync.py", line 88, in main
    check_required_mutually_exclusive(combination_name, custom_combination_savename)
  File "/home/petemir/.local/lib/python3.8/site-packages/bubop/cli.py", line 30, in check_required_mutually_exclusive
    raise NotEnoughArgumentsError()
bubop.exceptions.NotEnoughArgumentsError: Not enough arguments provided

Deleting lines 88-98 lets me run the program, but obviously, I would be missing some kind of additional validation.
Thanks!

Hi @Petemir , can you test with the devel branch? it should be fixed there.

Should also be available in the latest pre-release: pip install taskwarrior-syncall==1.2.0b2

@bergercookie now it works great, thanks!


Board listing fixes

Re-applies #69904 and re-fixes #70614

Error out if multiple boards use the same name
Ignore duplicate roots in list_boards/hardware

I would restore the original comment for the revert of the revert, maybe just mention that it's a second attempt in the message.

I would restore the original comment for the revert of the revert, maybe just mention that it's a second attempt in the message.

I think that's reasonable. Replaced with a cherry-pick.

The bug observed in #70614 has two facets, both of which should now be fixed.


initial support of custom query

initial support of custom query, find operation.
@ZhijunZhao,

@yungez,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Exception is thrown when the analytics option was opted out

Description
An exception is thrown when I opted for no analytics
Steps To Reproduce

Go to Terminal
Enter very_good --analytics
Getting the below exception

Missing argument for "analytics".
#0      CommandRunner.usageException (package:args/command_runner.dart:95:7)
#1      CommandRunner.parse (package:args/command_runner.dart:123:35)
#2      VeryGoodCommandRunner.run (package:very_good_cli/src/command_runner.dart:63:27)
#3      main (file:///Users/xxxxxxxxxx/.pub-cache/hosted/pub.dartlang.org/very_good_cli-0.1.1/bin/very_good.dart:5:54)
#4      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:299:32)
#5      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

Expected Behavior
In my opinion, instead of an exception message, a plain text saying "You have not opted for analytics. To enable it do xxx things"
Screenshots

Additional Context
I have just shared my opinion. If this is the expected message, please close this ticket.

Hi @NaagAlgates 👋
Thanks for opening an issue!
The --analytics option requires you provide the desired setting:
# enable analytics
$ very_good --analytics true

# disable analytics
$ very_good --analytics false

We'll make sure to improve this in the next release. Thanks for reporting it!🙏

Thank you @felangel ,
I missed the argument "true" or "false"
# disable analytics
$ very_good --analytics false

It is working now. :)


Example on extracting data from post or put

I'm unclear on how to extract data from a post or put using this library; I seem to only find empty dictionaries. Could you add an example?
Thanks!

No problem, I'll add one to the README and this issue shortly.

Here is my example set up
main.swift
let app = Blackfish()

app.get("/") { (request, response) in
    response.render("index.html")
}

app.put("/") { (request, response) in
    print(request.data)
    response.redirect("/")
}
app.post("/") { (request, response) in
    print(request.data)
    response.redirect("/")
}

app.listen(port: 3000) { error in
    if error == nil {
        print("App listening on port 3000")
    } else {
        print("Error")
    }
}

Then, using curl to post data:
curl --data "test=2&name=foo" http://<IP_ADDRESS>:3000
I get the following output:
App listening on port 3000
["test": "2", "name": "foo"]

Using PUT gives me the same output
curl -X PUT --data "test=2&name=foo" http://<IP_ADDRESS>:3000
["test": "2", "name": "foo"]

Added to the README.md as well, let me know how it goes and if all is okay, we'll close the issue :)

That was useful, thanks. I thought NSURLSession did more than it does for uploads; that's what I get for trying two new things at once.

No problem, if you need to do file uploads make sure you use the Multiparser middleware that comes built in: Multipart


Release

Thanks for writing this package, I'm currently using cherry picked functionality from this repo since I only utilise a few Cognito features in my Elixir app.
Are you planning on releasing this package? It would be nice to have all the functions documented and managed through mix if I ever plan to implement more Cognito related functionality. I noticed that you already reserved the cognito-idp repo under the ex_aws github organisation.

Same here. Is this planned on being released?


Some loaders appear more than once

Hi, when I use this plugin ,the output shows some loaders , like scss-loader,  appear many times, so why this happens, what does it mean? I appreciate anyone's help.
screenshot:

This has been raised a couple of times before. Basically:
This is expected output, but it is just inherently confusing. The way webpack runs, the different loaders run in different configurations, and without setting SMP's granularLoaderData option, what SMP outputs is also, unfortunately, quite confusing!
I've raised https://github.com/stephencookdev/speed-measure-webpack-plugin/issues/51 to hopefully address the confusion


Implement selector feature using 'select' annotation

Tested: make test

/assign @yiqigao217
/assign @adrianludwin

/retest

Ran 2 of 58 Specs in 7.171 seconds
SUCCESS! -- 2 Passed | 0 Failed | 0 Pending | 56 Skipped
PASS | FOCUSED
FAIL	sigs.k8s.io/multi-tenancy/incubator/hnc/internal/reconcilers	7.222s

@GinnyJI I think the problem is that you checked in some focused tests? So the tests you focused on passed, but the test suite failed because (I'm guessing) you're not supposed to check in focused tests.

Ran 2 of 58 Specs in 7.171 seconds
SUCCESS! -- 2 Passed | 0 Failed | 0 Pending | 56 Skipped
PASS | FOCUSED
FAIL	sigs.k8s.io/multi-tenancy/incubator/hnc/internal/reconcilers	7.222s

@GinnyJI I think the problem is that you checked in some focused tests? So the tests you focused on passed, but the test suite failed because (I'm guessing) you're not supposed to check in focused tests.

Oh good point. Just removed the focused test.

/lgtm
/hold
/assign @adrianludwin

Tested: make test

Can you please update this commit message with more information (e.g. link to the design doc, summary of the changes) as well as more information about the testing? E.g. why is make test sufficient?

/retest

tracked at #1123

/lgtm
feel free to remove hold if we don't need the issue for managed-by (I think we plan to have a PR for it in the next day or two?)

feel free to remove hold if we don't need the issue for managed-by (I think we plan to have a PR for it in the next day or two?)

Yup I'm working on it now.
/hold cancel

/retest


[Deepak Foundation] Move decision logic changes to live app

[x] Move following story changes to live app
https://app.zenhub.com/workspaces/avni-impl--support-5cf8e458bf08585333fd64ac/issues/gh/avniproject/deepak-foundation/15
[x] Run the script required to make change in existing observations
[ ] Perform sanity testing

Bundle Before upload

we can able to see a card more than once in the dashboard

Know prod issue: gender search in the search filter will not select or deselected properly
https://github.com/avniproject/avni-client/issues/1055

@nupoorkhandelwal deletion of card not worked. Voided  dashboard_section_card_mapping as per uat org


refactor: migrate the project from SLS v2 to v3

Migrated the Serverless Framework from v2 to v3
Bump serverless-iam-roles-per-function to v3.2.0
Bump serverless-layers to v2.5.4

The serverless-export-env plugin v2.x has a bug that causes circular dependencies and breaks the serverless deployment. I'm currently thinking on a workaround for that.

Managed to make the deployment work by downgrading the serverless-export-env plugin back to v1.4.0.


Enhance file structure

changes

separate code and style from markup and make index.html only contain HTML
change serial Results Div to be auto scrollable
change baud input to be baud selection
add 5px padding to #LineToSend Input
add check if the browser has no support for web serial Api before  connection

While I am generally happy about splinting up a project to have better organisation on this particular one I really want to keep the source at less than 200 lines of code with comfortable amounts of white space for easy reading for the main example. That why I did it all in one HTML file.
Would you consider making the changes in a separate directory labeled "advanced"?
Also on my screen the CSS seems to make the height of the div for displaying data too tall and the page now scrolls on my screen rather than it being a clean non scrolling single page application.
I do plan to add a capability of shortcut buttons that the user can set up for custom strings to send as shortcuts. This is going to required that a more advanced version be put together and splitting it up in a more proper way would be required for that so the split up version would be great for that.

Would you consider making the changes in a separate directory labeled "advanced"?

Instead of crating new directory Could you make another branch called dev or development and I will push in this branch ... and when you fell happy with the changes you can merge it with the master branch

I do plan to add a capability of shortcut buttons that the user

also I have some feature idea like multi device communication with connection ID /or naming ... if you like we can code together to add more features

What is the reason to swap the baud input item to a select

I was guessing that no need to use a non-standard speed, but I will change as what it was

The reason I asked for a new folder was because it is hosted using github pages and wanted to keep the home page simple but link to a more sophisticated version.


How to deploy OpenPai with Nginx on sub-path?

Organization Name:

Short summary about the issue/question:
Currently, I can deploy OpanPai in sub-paths, but I need to write a lot of sub_filter rules, as shown below.
    location /openpai/ {
        proxy_pass https://MasterIP/;
        sub_filter 'https://MasterIP/' 'https://$host/openpai/';
        sub_filter '/env.js' '/openpai/env.js';
        sub_filter 'restServerUri: \'/rest-server\'' 'restServerUri: \'/openpai/rest-server\'';
        sub_filter 'marketplaceUri: \'/marketplace/api\'' 'marketplaceUri: \'/openpai/marketplace/api\'';
        sub_filter 'prometheusUri: \'/prometheus\'' 'prometheusUri: \'/openpai/prometheus\'';
        sub_filter 'grafanaUri: \'/grafana\'' 'grafanaUri: \'/openpai/grafana\'';
        sub_filter 'k8sDashboardUri: \'/kubernetes-dashboard/\'' 'k8sDashboardUri: \'/openpai/kubernetes-dashboard/\'';
        sub_filter 'alertManagerUri: \'/alert-manager\'' 'alertManagerUri: \'/openpai/alert-manager\'';
        sub_filter '/home.html' '/openpai/home.html';
        sub_filter '/assets/img/' '/openpai/assets/img/';
        sub_filter 'src:"/' 'src:"/openpai/';
        sub_filter 'url:"/' 'url:"/openpai/';
        sub_filter '"/job-detail.html' '"/openpai/job-detail.html';
        sub_filter 'href:"/' 'href:"/openpai/';
        sub_filter 'href="/' 'href="/openpai/';
        sub_filter '"/index.html' '"/openpai/index.html';
        sub_filter '"/submit.html"' '"/openpai/submit.html"';
        sub_filter 'href = `/submit.html`' 'href = \'/openpai/submit.html\'';
        sub_filter '/marketplace/api' '/openpai/marketplace/api';
        sub_filter 'url(/assets' 'url(/openpai/assets';
        sub_filter '["/' '["/openpai/';
        sub_filter 'link:"/' 'link:"/openpai/';

        proxy_set_header Accept-Encoding "";
        sub_filter_types *;
        sub_filter_once off;
        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Host $host;
    }

These sub_filter rules are created by myself. The method is that when I find inaccessible resources in the browser, I find the source code as a reference for creating sub_filter rules.
With these rules, the service can be accessed normally, but there are a few places where the behavior is abnormal.

The sub menu cannot be focused;
The default state of k8s Dashboard is nothing.

These two problems can be explained with the following animation.

What I want to seek from the community is whether there is a simpler solution to configure the Nginx subpath as an OpenPai proxy. If not, how can the above two problems be solved?
Brief what process you are following:

How to reproduce it:

OpenPAI Environment:

OpenPAI version: v1.6.0
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Hardware (e.g. core number, memory size, storage size, GPU type etc.):
Others:

Anything else we need to know:

So far, the following configuration works well.
    location /openpai/ {
        proxy_pass https://MasterIP/;
        sub_filter 'https://MasterIP/' 'https://$host/openpai/';
        sub_filter '/env.js' '/openpai/env.js';
        sub_filter '/rest-server' '/openpai/rest-server';
        sub_filter '/marketplace/api' '/openpai/marketplace/api';
        sub_filter '/prometheus' '/openpai/prometheus';
        sub_filter 'grafanaUri: \'/grafana' 'grafanaUri: \'/openpai/grafana';
        sub_filter '/grafana/' '/openpai/grafana/';
        sub_filter '/kubernetes-dashboard' '/openpai/kubernetes-dashboard';
        sub_filter '/alert-manager' '/openpai/alert-manager';
        sub_filter '/home.html' '/openpai/home.html';
        sub_filter '/dashboard.html' '/openpai/dashboard.html';
        sub_filter '/job-list.html' '/openpai/job-list.html';
        sub_filter '/virtual-clusters.html' '/openpai/virtual-clusters.html';
        sub_filter '/user-view.html' '/openpai/user-view.html';
        sub_filter '/cluster-view' '/openpai/cluster-view';
        sub_filter '/plugin.html' '/openpai/plugin.html';
        sub_filter '/log-manager' '/openpai/log-manager';
        sub_filter '/job-detail.html' '/openpai/job-detail.html';
        sub_filter '/index.html' '/openpai/index.html';
        sub_filter '/submit.html' '/openpai/submit.html';
        sub_filter '/user-profile.html' '/openpai/user-profile.html';
        sub_filter '/favicon.ico' '/openpai/favicon.ico';
        sub_filter '/assets/font' '/openpai/assets/font';
        sub_filter '/assets/img' '/openpai/assets/img';
        sub_filter '' '';
        sub_filter '/marketplace/api' '/openpai/marketplace/api';

        proxy_set_header Accept-Encoding "";
        sub_filter_types *;
        sub_filter_once off;
        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Host $host;
    }


sap.m.Slider tickmarks layout collapses for zoom levels less than 100%

OpenUI5 version:
1.100.2 (latest)
Browser/version (+device/version):
Chrome 100.0.4896.127
URL (minimal example if possible):
https://sapui5.hana.ondemand.com/#/entity/sap.m.Slider/sample/sap.m.sample.Slider
Steps to reproduce the problem:

Open the above URL (the slider sample page)
Change the zoom level to anything below 100%

What is the expected result?
For sliders that have tick marks, the tick marks layout doesn't break/collapse
What happens instead?
The ticks don't fit and the layout collapses.
Any other information? (attach screenshot if possible)

Hi Elena,
We decided that the <100% isn't a critical use case (especially given that the control is still usable even with a collapsed layout) and this can therefore be closed.
btw. the zoom level at which the layout breaks <100% seems to be inconsistent and dependent on different factors such as monitor or browser.
Best,
Richard


[Feature Request] - Tap Custom URLs

Summary of Request
As a user of homebrew, I want the ability to tap a custom formula repository that is not hosted on GitHub. This is advantageous for organizations that use Homebrew and store formula repositories internally, but for whom GitHub is not an option. Therefore, this role needs the ability to pass a custom URL to the brew tap command. For context, the usage in man brew is brew tap [--full] user/repo [URL].
Existing usage of the homebrew_tap module today:
# Tap.
- name: Ensure configured taps are tapped.
  homebrew_tap: "tap={{ item }} state=present"
  with_items: "{{ homebrew_taps }}"

According to ansible-doc homebrew_tap, there is a url option (added in version 2.2) that can be added or omitted as appropriate:
- url
        The optional git URL of the repository to tap. The URL is not assumed to be on GitHub, and the protocol doesn't have to be HTTP.
        Any location and protocol that git can handle is fine.
        `name' option may not be a list of multiple taps (but a single tap instead) when this option is provided.
        [Default: (null)]
        version_added: 2.2

Example usage w/ role vars:
homebrew_taps:
  - caskroom/versions
  - { name: mycompany/homebrew, url: https://example.company.com/homebrew-example.git }

Accept Criteria for Completion

[ ] Users can tap custom URLs
[ ] Backwards compatible with the existing usage of homebrew_tap which does not supply a URL (hint: it should be omitted.)

(I can get started on this soon, but I wanted to save this to an issue so I wouldn't forget 👍)


Deprecate Social Amplificator classes + methods

Changes
Officially deprecating methods/classes for the Social Amplificator feature.
References
Removed in #607

Codecov Report

Merging #612 into master will increase coverage by 0.28%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #612      +/-   ##
============================================
+ Coverage     36.01%   36.29%   +0.28%
+ Complexity     1270     1266       -4
============================================
  Files            52       52
  Lines          3968     3937      -31
============================================
  Hits           1429     1429
+ Misses         2539     2508      -31

Impacted Files
Coverage Δ
Complexity Δ

WP_Auth0.php
22.45% <ø> (ø)
62 <0> (ø)
:arrow_down:

lib/admin/WP_Auth0_Admin_Advanced.php
30.55% <ø> (+2.96%)
60 <0> (-4)
:arrow_down:

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 da69a55...be0c56a. Read the comment docs.


Vg 5783 login screen consent text

+1

Codecov Report

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

@@           Coverage Diff           @@
##           master     #123   +/-   ##
=======================================
  Coverage   79.77%   79.77%
=======================================
  Files         117      117
  Lines        7697     7697
  Branches     1158     1158
=======================================
  Hits         6140     6140
  Misses       1557     1557

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 a5456fb...b235555. Read the comment docs.


Indicate that we forbid unsafe code

Makes this show up as clean on cargo-geiger

@Soveu Any chance you could review this?

Huh, why I haven't noticed that earlier? :eyes:
I think we could change the version to 0.1.1 in the same PR.

I think we could change the version to 0.1.1 in the same PR.

Sounds good! I bumped to v0.1.1 in Cargo.toml.

Bumped it to 0.1.1 on crates.io :)


onBlur function clears the input

When the value of the autocomplete input field is updated via Javascript instead of typing the value directly, the onBlur function clears the text of the input. Commenting the line 2409 of this file https://github.com/algolia/autocomplete.js/blob/master/dist/autocomplete.js#L2409 seems to solve the problem.
Is this a normal behavior or there is some setting I have to update in order to avoid it?

How are you updating the field via JavaScript? You need to use autocomplete.js's setVal (or .autocomplete('val', myVal) if jQuery) function.
Let me know!

Thanks for your response. I was assigning the input value using the JQuery function val(), but after see your comment I updated the functionality to use setVal and it fixed the issue.

Thanks for your response. I was assigning the input value using the JQuery function val(), but after see your comment I updated the functionality to use setVal and it fixed the issue.

Good to know!


feat: support GraphQL all type args

One day, while I was doing AST parsing, I came across this repository and found that the support for all GQL types was incomplete. I made improvements on my end to support parameter parsing for all GQL types.

[x] Added test cases passed.
[x] Added code coverage cases, all covered and passed.

有一天我在做AST解析时找到了这个仓库，发现对所有的GQL的类型支持不完善，我这边进行了完善。 可以支持所有GQL类型的参数解析

[x] 新增测试用例通过
[x] 新增代码覆盖用例全部覆盖且通过

#3 fixed closed

This is great! Thanks!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add typescript definitions for getStorybook()

2.35.3 add getStorybook() in 86d5819a7b8843a1331c33eb7bef17bae0d98006, but Typescript definitions were not updated.
Also configure was missing altogether, added in 	0e9d7de
Issue:
stories.ts(6,23): error TS2339: Property 'getStorybook' does not exist on type 'typeof "./node_modules/@kadira/storybook/config/storybook"'.

when using getStorybook().
What I did
Added TS definition.
How to test
Run
test.ts, e.g.:
import * as storybook from '@kadira/storybook';
import './.storybook/config.js';

console.log(storybook.getStorybook());

with tsc.

Updated to not overwrite the changes from https://github.com/storybooks/react-storybook/pull/680

@ndelangen any chance to get this in if I rebase it again?

Yeah, I think so!

👍 @ndelangen rebased!

ta!

any chance to get this in a release @ndelangen?

Yes this will get released in 3.0.0, I don't have a date for that yet, should be an alpha release soonish. My guess this at the end of this month or in the beginning of next month.


assert the circuitBreakerForceClosed property

The motivation here is the "propertyValue_circuitBreakerForceOpen" is currently being checked to make sure there is a value.
It would make sense that the "propertyValue_circuitBreakerForceClosed" value also be checked

Looks good to me.

Thanks @lholmquist !


elo compile error

单独编译elo模块，出现这个错误，我的系统是16.04，请问是什么原因呢
[ 50%] Building CXX object CMakeFiles/localization_test.dir/localization_test.cc.o
[100%] Linking CXX executable localization_test
../lib/libcamera_localization.so: error adding symbols: File in wrong format

I have the same problem, The error:
[ 50%] Building CXX object CMakeFiles/localization_test.dir/localization_test.cc.o
[100%] Linking CXX executable localization_test
../lib/libcamera_localization.so: error adding symbols: File in wrong format
collect2: error: ld returned 1 exit status
CMakeFiles/localization_test.dir/build.make:141: recipe for target 'localization_test' failed
make[2]: *** [localization_test] Error 1
CMakeFiles/Makefile2:67: recipe for target 'CMakeFiles/localization_test.dir/all' failed
make[1]: *** [CMakeFiles/localization_test.dir/all] Error 2
Makefile:83: recipe for target 'all' failed
make: *** [all] Error 2

你好，这个localization_test 是要自己写的算法吗？

localization_test 是百度提供的，我是想先运行跑起来看看
在 2018-03-09 14:17:05，"maziqiqi"<EMAIL_ADDRESS>写道：
你好，这个localization_test 是要自己写的算法吗？
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

请问安装成功了吗？我也想安装，不过很多地方不明白。

我换了Ubuntu 14.04 和 indigo的环境，但是还是同样的错误；不太明白arrach64平台具体是什么要求
在 2018-03-12 11:17:10，"maziqiqi"<EMAIL_ADDRESS>写道：
请问安装成功了吗？我也想安装，不过很多地方不明白。
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Please refer to the latest documentation to resolve the issue.
Closing this issue as there has been no communication in a while. Please feel free to open it if you have additional questions.


Bato not working

Source information
Host cannot be found
Source language
English
Steps to reproduce
Bato host cannot be detected for tachimanga

Expected behavior
Open of manga list
Actual behavior
Nothing is detected
Mihon/Tachiyomi version
Tachimanga V3.3
Android version
IOS
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

This is your network issue and has nothing to do with the extension.

The Bato.to extension has several mirrors you can pick from the extension settings, so if the default site doesn't work you should change it and check.
In any case, it's not an issue with the extension. If changing the mirror doesn't help it's better if you ask in #support on Discord for easier troubleshooting.


Unable to run the demo locally on Ubuntu 20.04

Hi,
I am trying to run the demo locally, however yarn install keeps throwing following error:

Current setup:
Ubuntu 20.04
Node - 16.14.0
yarn - 1.22.19
Steps to reproduce:

Clone the animo demo repo using https://github.com/animo/animo-demo.git
Run yarn install in the root

Any help would be appreciated. Thanks! :)

Hi! It looks like it cannot find libindy. This is a native library required for hosting the demo.
Hopefully this guide can help you: https://aries.js.org/guides/getting-started/installation/nodejs

Hi @blu3beri, you are right and thanks for your response.
I was missing the libindy library but the AFJ documentation for linux didn't work out for me. However, the following Dockerfile commands solved my issue.
`RUN apt-get update -y && apt-get install -y
software-properties-common
apt-transport-https
curl
# Only needed to build indy-sdk
build-essential
libindy
RUN apt-key adv --keyserver keyserver.ubuntu.com --recv-keys CE7709D068DB5E88
RUN add-apt-repository "deb https://repo.sovrin.org/sdk/deb bionic stable"
nodejs
RUN curl -sL https://deb.nodesource.com/setup_16.x | bash
yarn
RUN curl -sS https://dl.yarnpkg.com/debian/pubkey.gpg | apt-key add - &&
echo "deb https://dl.yarnpkg.com/debian/ stable main" | tee /etc/apt/sources.list.d/yarn.list
install depdencies
RUN apt-get update -y && apt-get install -y --allow-unauthenticated
libindy
nodejs`


PPPoE using incorrect IP Address

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
After major upgrade from 21.7 to 22.1, PPPoE connection ignores ISP assigned address (observed in packet capture) and uses the first Virtual IP (IP Alias).  The "WAN Address" is now the second usable IP in my /28 block.  Had to create manual NAT rules to force use of first address which some remote systems are set to allow.
Tip: to validate your setup was working with the previous version, use opnsense-revert (https://docs.opnsense.org/manual/opnsense_tools.html#opnsense-revert)
To Reproduce
Steps to reproduce the behavior:

Boot/Reboot firewall.
Start packet capture
Disconnect/Reconnect PPP connection
See ISP assigned address of x.x.x.161
Check public IP address, which is x.x.x.162

Expected behavior
"WAN Address" should be the IP assigned from ISP which has been working for almost 2 years
Describe alternatives you considered
Have tried adding VIP of .161, moving to the top, but it seems to be ignored.  This VIP works if used in a manual outbound NAT rule.  Have considered removing all VIPs and rebooting.  However I have many rules which are referencing these so it will require some more downtime to test.
Screenshots

Relevant log files
ppps.log
Environment
Software version used and hardware type if relevant, e.g.:
OPNsense 22.1.3-amd64 (FreeBSD 13.0-STABLE, OpenSSL 1.1.1m 14 Dec 2021)
Running on Sophos SG310
Intel(R) Core(TM) i3-4330 CPU @ 3.50GHz (2 cores, 4 threads)
Intel I210 Gigabit Network Connection x8
Intel I350 Gigabit Fiber Network Connection x2

@utahbmxer do you have data points for 22.1, 22.1.1 and 22.1.2 ?

I don't as I applied all updates back to back before noticing.  I am planning on reinstalling 21.7 and then stepping through each one to try and pinpoint which release this started in.  I'll try and do this later tonight (located in US) and report back.

@fichtner I reinstalled 21.7 (used config importer), then upgraded to 21.7.8.  Confirmed everything was working as expected.  Upgraded to 22.1 and things still worked correctly.  I tried to use opnsense-update tool (opnsense-update -bkpr 22.1.1) to try and do minor updates, but it jumped to 22.1.3.  How can I target those other releases to get data around those?

@utahbmxer in the firmware settings tab select flavour "(other)": 22.1/MINT/22.1.1/OpenSSL
this way you can step through each update. I think this one might have started in 22.1.3.

Looks like you're correct, @fichtner.  Loaded my other SG310 (eventually wanting to setup CARP, pppoe?) up to 22.1 and started going through.
22.1.1 - No issue noticed, uses correct IP
22.1.2 - No issue noticed, uses correct IP
22.1.3 - Issue started, seems to use first VIP instead of provider assigned IP
22.1.4 - (for fun) Issue still present, however I noticed that in the GUI, it seems to display correctly.  Dashboard / Interfaces widget shows correct IP.  Interfaces > Overview > WAN seems correct as it shows the primary IP, followed by all the VIPs preceded with a +.  However when I do 'curl http://ifconfig.me' or check in a browser, it shows the first VIP.
Going to keep this firewall racked but plug my cables back into the other one on 21.7.8.  Let me know if you want me to check anything else.

@utahbmxer I've added back a change that seems to make this order enforcement happen although in general it would be nicer if the kernel had some sort of understanding for a primary address other than "first found. 57097e2
Can you try it?
# opnsense-patch 57097e2

@fichtner sorry for the delay.  I just applied that patch against 22.1.4_1 and reconnected my pppoe connection without a reboot (wasn't sure if that was needed).  Initial tests show it using the correct IP now.


Add resources list to documentation.md

Add an external resources section to documentation.md with these resources:
https://www.writethedocs.org/guide/
https://guides.github.com/features/mastering-markdown/

Make it similar to the external resources table that we have in the main README.md


Additional R Shiny Best Practice

As part of DfE Dashboard Standards review, we are looking to drive "best practice" in Shiny apps/content in general.
Suggestion is to build on the documentation already in the analyst guide to point people towards useful resources. The Creating R Shiny dashboards section is most appropriate place for this but we can also look to expand out to Rmarkdown and Quarto.
This will include a few aspects:

Recommended resources for getting started
File structure
Creating tables
Databasing
Testing
Theming

 Analysts Guide: Additional RShiny best practice

Make use of shinyModules to segregate your app into reusable modules.
Use functions like reactive(), observe(), eventReactive() to reduce your apps high computation.
Make use of modal dialogues and notifications for a clean and modern outcomes of your app.


Seems src/bindings#extractNamespace behaviour is not expected

Hi, I'm reading the code, and I found the code in function extractNamespace
https://github.com/ktsn/vuex-class/blob/ddefbb3f125f174beb3678067423bf4a2966f412/src/bindings.ts#L128-L130
will append / to the tail of namespace.
So if I call @State('name', { namespace: 'userInfo' }), it will execute mapState('userInfo/', { [...]: 'name' })
I think this isn't what I expected.
Did I missed any thing?

Why doesn't it expected? It would bind to userInfo namespace.

I'm so sorry to reply so late. I used to be too busy in the past to have the energy to check the code.
At last I found it's my fault, the vuex version I am using is too old to work with newest vuex-class.
Now my problem has resolved. Thanks for your work :P
And apologize again for my recklessness.


Create a Readme for Avatar

How to create a Readme for Avatar ?

You need to copy paste the buttons code in  Tailwind Playground
Wrap all of them in a div and then add a class='flex justify-center items-center h-screen '  you can use flex-col too as per component size.
Take a screenshot of that and add it to the readme in the table format
Make sure to enclose the screenshot under anchor tag so that one can just choose the Avatar and after clicking that image will be redirected to the code of that Avatar.
NOTE:- Make sure each component screenshot is of equal size so that it fits in the layout

For Reference you can see Button Readme

@RavindraP04 are you working on it ?

No, @Dibyendu303 If you want you can work on it.

Please assign me @Rajdip019


Missing browser support information

Great work. It would be great to have some browser support information for that library.

This uses CSS3 animations. Probably on older browsers the animations won't work.
http://caniuse.com/#feat=css-transitions
Contributions adding this to docs would be welcome! :memo:

Blazing fast response. I appreciate it. 👍 I'm going to use it in a project and I'll be back to you if I found any problems regarding browser support. Thanks!

@miduga Great! ✨  If you see any way to improve the functionality, feel free to contribute. I wrote this code few years ago. 😁

/cc 🔔 @miduga


tsserver fails with "Cannot read property 'path' of undefined"

TypeScript Version:  2.9.2
(I am pretty sure this still repros against latest as the affected code is the same. The way Google has things setup makes it hard to test against the latest version.)
Search Terms: Cannot read property 'path' of undefined
Stack:
TypeError: Cannot read property 'path' of undefined
at ProjectService.updateNonInferredProjectFiles (.../node_modules/typescript/v2_9_2/lib/tsserver.js:95205:43)
at ProjectService.addFilesToNonInferredProjectAndUpdateGraph (.../node_modules/typescript/v2_9_2/lib/tsserver.js:95154:22)
at ProjectService.createConfiguredProject (.../node_modules/typescript/v2_9_2/lib/tsserver.js:95172:22)
at ProjectService.openClientFileWithNormalizedPath (.../node_modules/typescript/v2_9_2/lib/tsserver.js:95566:44)
at IOSession.Session.openClientFile (.../node_modules/typescript/v2_9_2/lib/tsserver.js:97028:37)
at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (.../node_modules/typescript/v2_9_2/lib/tsserver.js:96199:31)
at .../node_modules/typescript/v2_9_2/lib/tsserver.js:97699:88
The offending line is https://github.com/Microsoft/TypeScript/blob/878bf80a48716e80063919fad1c83d54259b1cb0/src/server/editorServices.ts#L1578.
scriptInfo is undefined as it is returned here: https://github.com/Microsoft/TypeScript/blob/a1746d4cfe357ce8f38619e46578f817dfc03f9c/src/server/editorServices.ts#L1861
Even just looking at those two lines it's pretty clear that updateNonInferredProjectFiles should handle undefined coming back from getOrCreateScriptInfoNotOpenedByClientForNormalizedPath.
In my particular case where this exception is being triggered, a build tool at Google is leaving behind a .d.ts symlink that points to a file which no longer exists. This .d.ts file is referenced directly in the "files" array in the tsconfig.json.
I didn't do enough experimenting to know whether it's sufficient to trigger this issue by simply referencing a file that doesn't exist in the tsconfig files array, or whether having a symlink pointing to a file that doesn't exist is required.
The user-facing result of this exception is that for some files vscode will find lots of problems that don't exist (such as underlining modules it can't find or that experimental decorators aren't enabled).
I'm working around it for now by adding if (!scriptInfo) continue; before the scriptInfo.path reference. I haven't noticed any issues running with this change.
Thanks for taking a look!
Chris

You can upload logs using

With "typescript.tsserver.log": "verbose" enabled
Restart vscode or vscode insiders
Reproduce the problem
Run code --upload-logs on the command line (or code-insiders --upload-logs if using vscode insiders) and follow instructions
Share the log file ID

This error shows up in the tsserver's log (using export TSS_LOG='-level verbose -file /tmp/tss.log'). Based on my understanding this log isn't returned to vs code (at least nowhere I've seen). I'm using a tsserver on a remote machine so I'm not sure if vs code would have access to this log to upload it.
I already triaged the bug though, maybe someone can just try to repro using the symlink like I described...

@scriby i cannot repro the issue you mentioned. Also you shouldnt be hitting this given
https://github.com/Microsoft/TypeScript/blob/878bf80a48716e80063919fad1c83d54259b1cb0/src/server/editorServices.ts#L1565
where it checks if file is not present on the disk and handles that case.
tsserver log would be the only way we can decode this.  Please try getting us the log to be able to investigate this further.

Since we haven't heard about this yet, closing it. We can open it when we have more info


Update docker configurations

This PR aims to update the docker use in the project.

Coverage decreased (-1.6%) to 90.15% when pulling dc059cf915d41f5e71eb6fdd961676425abb8884 on JonathanAlcantara:master into fa1886988b7c54a418900549af8e25acd96ddf35 on backstage:master.

Coverage decreased (-1.6%) to 90.15% when pulling dc059cf915d41f5e71eb6fdd961676425abb8884 on JonathanAlcantara:master into fa1886988b7c54a418900549af8e25acd96ddf35 on backstage:master.


Normalized and Improved the Legacy filter

Filter before had amplitude issues.

Lets get this fixed before people start downloading it.


Prepare the Serenata apps to be deployable in a Kubernetes cluster

As the Serenata run in the same infrastructure used by the Querido Diario project we need to converge both projects to use the same underlying infrastructure. There is an ongoing process of migrating the Querido Diario workload to run on top of a Kubernetes cluster. So, we need to adapt the Serenata app to be deployable in the cluster as well.
The first step for this is to update the script to be able to build the container image and make it available in some registry.

@andreformento is working on this.

@sergiomario can you assign the issue to @andreformento? I cannot do it.

@jvanz I also can't sign the issue for him until he makes a comment here on the thread. Github has this rule.
@andreformento Can you make a comment here?

sorry for delay @jvanz @sergiomario


Button styling

Change the button styling to match this mockup. Click "inspect" in the lower right corner to view the CSS. You may need to request access. You can ignore everything about the mockup except the button styling

On hover, the button background color should turn white. The text should remain the same color.

I would love to work on this, can I?

@abnazhor yes, thank you. Let me know if you need any more details

Thanks again @abnazhor


Commands have no insert_non_send_resource method

Bevy version
0.8
What you did
I'd like to use Commands to insert a non-send resource.
fn foo_system(mut commands: Commands) {
    commands.insert_non_send_resource(RefCell::new(String::from("Hello World")));
}

What went wrong
Expected: No error.
Actual:
error[E0599]: no method named `insert_non_send_resource` found for struct `bevy::prelude::Commands` in the current scope
  --> src\main.rs:16:14
   |
16 |     commands.insert_non_send_resource(RefCell::new(String::from("Hello World")));
   |              ^^^^^^^^^^^^^^^^^^^^^^^^ help: there is an associated function with a similar name: `insert_resource`

Additional information
You cannot create custom commands because Command requires it to be Send + Sync.
The workaround is to use an exclusive system:
fn foo_system(world: &mut World) {
    world.insert_non_send_resource(RefCell::new(String::from("Hello World")));
}

You cannot create custom commands because Command requires it to be Send + Sync.

I'd expect built-in commands to have the exact same limitation, no?

So in order to get this working, we'll probably need to pass in a function that can be called and generates the nonsend resource later on demand.
The signature would be something like
create_nonsend<N, F: FnOnce() -> N>(&mut self, builder: F) {}

I'd expect built-in commands to have the exact same limitation, no?

Well according to the docs (highlight for emphasis):

A queue of commands that get executed at the end of the stage of the system that called them.
Commands are executed one at a time in an exclusive fashion.

Maybe my assumptions were incorrect, but I would think that if this is possible with an exclusive system, then it would be possible with Commands albeit with extra delay.
Also after reading the docs I found another workaround. I'm updating the original.
fn foo_system(mut commands: Commands) {
    commands.add(|world: &mut World| {
        world.insert_non_send_resource(RefCell::new(String::from("Hello World")));
    });
}

You can't insert a non send resource with a command, because that would require sending the resource from one thread to another. Systems can run on any thread and when you call insert that is then initializing the resource on a thread that is not the main thread. Then when the command is run on the world, it inserts the resource into the world which is owned on the main thread. This violates the non sendiness of the resource.
The workarounds that you've listed are what I would recommend. It might be possible to allow init_non_send_resource, since that doesn't need to initialize the resource immediately, but I'm not sure what internal changes might be needed.

With the move away from storing !Send resources in World, closing this issue as this would is impossible to implement soundly as initially proposed, and likely short lived as an API even if we did a closure based approach.


Standalone filter by uuid

Background
In third-party mode (i.e. Elasticsearch as backend), we can filter by UUID in GraphQL to retrieve an object by it's uuid. E.g.:

However, in standalone, this is not possible yet:

It does not seem to recognize uuid as a "special" prop and in the inverted index there is no entry for "uuid" which is not a custom user defined prop
Goals

[ ] A search like the above example returns the object with the correct UUID

Technical Overview

According to the package structure outlined in the Contributor Guide Weaviate has a strong separation between API, Business Logic and Persistence. As a result, we do not need to alter the API or Business Logic levels. As soon as the persistence logic passes its tests, this feature should work in standalone mode
There is already a test (pre-existing and copy/pasted from the "old" elasticsearch implementation). It is currently skipped (commented) as it has not been implemented yet: https://github.com/semi-technologies/weaviate/blob/00747bf2a51fe97433c760cb946ccb3125ce09b9/adapters/repos/db/filters_integration_test.go#L227-L232 Note: While this test should pass once the feature is complete, it is not ruled out that more tests have to be added to test specific portions of this feature.
The test mentioned above is an integration test. See the integration test section in the contributor guides for more details.
Normally we use the inverted index for where filters in standalone. However, the uuid field is not a user-configurable field, but is always present for each element. It is therefore used as the key when storing the object itself. So instead of using the inverted index, we can simple use the object store directly. Due to this special case, this has not yet been implemented.

@antas-marcin I think this issue will be a good first issue for you. I've thus prepared it for tomorrow's onboarding session. The reasons I think this is a good first issue for you are:

It touches upon a lot of topics (Standalone vs. Third-Party, Inverted index, etc.). Some of those might be foreign to you for now, but they will make more sense as part of tomorrow's onboarding.
It is reasonably small, so you can complete your first feature quickly.


[apt-get] Could not get lock /var/lib/apt/lists/lock

Error occured when several self-hosted runners on the same machine executes in parallel.
—————————————————— Package tasks using: ruby/setup-ruby-pkgs 1.33.2
apt-get update
apt-get update
  sudo apt-get -o Acquire::Retries=3 -qy update
  Reading package lists...
  E: Could not get lock /var/lib/apt/lists/lock. It is held by process 497290 (apt-get)
  E: Unable to lock directory /var/lib/apt/lists/
  Error: Command failed: sudo apt-get -o Acquire::Retries=3 -qy update

Is it possible to add action's input param like apt-get-retry-count or retry-count to allow custom configuration of retries count?

I'll have a look this weekend.

several self-hosted runners on the same machine executes in parallel.

Aren't they in VM's?  Just wondering why the lock...


Please invite me to the GitHub Community Organization

Name
Manish Rawat
Discord Username (if applicable)
Manish#2183
Additional Context
i met eddie first time on youtube. when i was learning ablout git and github. so from that time being i follow eddie everywhere.
the members of the community are just awesome. they all are so much welcoming and encouraging..  that is the one of the reason why i wanna join eddiehub community

i didn't receive any invitation

help me guys. how to join ?

check your spam folder, but you can also accept the invite from our GitHub Org page https://github.com/EddieHubCommunity

thank you eddie. finally found it.


add PerPage param to ListProjectOptions

This option is missing in many calls I believe, and I especially need it here.

PerPage and Page are available on the embedded ListOptions struct:
Ref: gitlab.go#237

@Gnappuraz, @zaquestion is perfectly right... I will close this PR as the fields you are looking for are already added using the embedded ListOptions struct.
Feel free to open an issue if you have any additional issues or questions regarding this!

Thanks for pointing me to the right spot. Sorry I couldn't find it before.

No problem at all! 👍


feat: enable basic runner

Background
This PR enables basic runner by commenting out some dependencies.
Changes

[x] Added runner, runner menu items
[x] Integrated runner to the request pane
[x] Updated timer to support updating step name in place
[x] Refactored sendAction to make it can be reused

Ref: INS-4197, INS-4222, INS-4223

@ihexxa was running this locally and one thing I noticed is that when you just boot it up, and create a design doc, it restarts the app when navigating to the collection page. Please see the Loom link - https://www.loom.com/share/f6f851ca31964ff2b0bee118c28ecd63?sid=e9a9c9c3-4e18-4852-95b6-2f7c3bc84d6c
Steps to reproduce:

run Insomnia locally
create a design doc
navigate to the collection page

Expected result:

it should be navigated to Collection page without restarting the app

Actual result:

it restarts the app
it stays in the Design page instead of navigating to Collection page

I wonder if this is related to this branch or not.

@marckong Let me schedule a sync at sometime.

Note: abstracts the send action in order to use different exception handling.

@filfreire It is no longer needed, let me close it.


Draft: Adds fallback to culture defined on node

This update fixes a null exception when the scope has no language set.
Explanantion
For invariant content the language property is always null for the variant. The call to SetCulture will throw a null exception when passed a null value. In that case we need to set the language on the scope by retrieving the culture information for that node and use that to set the language.

I was too quick with this pull request. The current code only works for a node that has culture set on itself. Running into issues with calling ancestors. So changed the status to draft for now.

Updated the pull request. It now thandles these cases:

retrieve language from culture defined on current node or ancestor
when all nodes set to inherit culture, it has fallback to the default language

I moved the fix into the controller since there was already some culture resolving taking place.
The case where culture is retrieved from domains is now used also for GetMarkupForBlock.

Fixes #18 and #20


Add "Link local CSS-File"-Feature

Adds the feature to select a HLJS-Stylesheet on your computer and load it into the Discord DOM. Thought would be a neat little feature if you want to select a custom hljs stylesheet that is located on your computer (using the link feature with a file-link like: file://C:/User/Desktop/hljs.css did not work for me)

Applied your requested changes!


[feat] Allow to increese/decrese zoom level

Actually pressing Ctrl + + or Ctrl + -  doesn't increeses/decreses zoom level, we should allow it, the zoom is already stored in user preferences automatically: https://stackoverflow.com/questions/46526734/where-does-electron-save-previous-zoom
Optionally: save zoom level also to json config

Seems that zoom is already possible using Ctrl+Shift++ and zomming out with Ctrl+- shortcuts


Can't run iOS project

I have an issue while running iOS example project

Is the library ready for the usage or not yet?
Thanks in advance!

This is not a released project and has not been published to pub.dev yet


Fix line-height in task list on multi-line titles

See https://github.com/Nitrino/flatly_light_redmine/commit/0b91a70f2226a414a41d8619b0c86b9e8d4306d5

👍


user error fix

fixed "Uncaught ReferenceError: user is not defined @ account-controller.js:95"

lgtm


dist/src/index.js

I do not find the "server/dist/src/index.js" file

Hi, you need to install typescript:
>>npm install -g typescript
and in then transpile the typescript code to javascript with (while being in the server folder)
>>tsc

Sorry, but i have error:
src/controller.ts(22,51): error TS2345: Argument of type '{ initial: States.CONNECTING_TO_SERVER; events: [{ name: Events.CONNECTED; from: States.CONNECTIN...' is not assignable to parameter of type 'StateMachineConfig<States, Events>'.
Types of property 'events' are incompatible.
Type '[{ name: Events.CONNECTED; from: States.CONNECTING_TO_SERVER; to: States.LOADING_GAME; }, { name:...' is not assignable to type '[{ name: Events; from: States | States[]; to: States; }]'.
Types of property 'length' are incompatible.
Type '3' is not assignable to type '1'.

I took a fresh clone and followed the instructions on the readme and I can't reproduce your error :/
The error in the browser is of course because there already was an error in transpiling typescript.

Thanks for the answer.


Migrations: Update adapter to support callback for when a new collection is being processed

Tasks

[ ] Add a callback parameter to the process_each_document adapter function

References

You can see the callbacks that AdapterBase subclasses have access to today, in: adapter_base.py
https://github.com/microbiomedata/nmdc-schema/blob/aa028108b4450f2ae0c8cff486f3b38a026725af/nmdc_schema/migrators/adapters/adapter_base.py#L11-L16

I don't remember why I wanted this to exist. Closing until I remember.


Major bug: events are not all retrieved

In some situation, events are not all printed

bug still present

ok in v0.3.3


Implement "Provide better answer" also for Discord

See implememtations (provide.better.answer)
src/main/java/com/wyona/katie/integrations/msteams/MicrosoftMessageSender.java
src/main/java/com/wyona/katie/integrations/slack/SlackMessageSender.java

Please see screenshot of current Slack implementation


Support iOS 8.0

Does this library support iOS 8.0, after i install this library in my project support 8.0 it will get error like the code did not convert to swift 3
If I change my project development target to 9.0 it's ok

@jacks8x I have lowered the base to iOS 8 and fixed some of the Swift 3 warnings. Please try updating your pod and project. Cheer! 🍻

Sorry about that, missed one other thing. Please use v2.0.3 and let me know if you have any problems. Cheers! 🍻


It's not finished

Why not Grant

You are correct, it is not finished yet since it was made about 5 minutes ago.

Close the issue

Oh wait I can do it


Update cats-effect, cats-effect-laws, ... to 3.3.7

Updates

org.typelevel:cats-effect
org.typelevel:cats-effect-laws
org.typelevel:cats-effect-testkit

from 3.2.9 to 3.3.7.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Applied Scalafix Migrations

org.typelevel:{cats-effect,cats-effect-laws}:3.3.0 (created no change)

github:typelevel/cats-effect/v3_3_0?sha=series/3.x

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel" } ]

labels: library-update, early-semver-minor, semver-spec-minor, scalafix-migrations, commit-count:1

Superseded by #442.


Normalization Issue

@matmarti There is something wrong with the energies in the ENSDF systematics data (see plot below).  This seems to be true for all cases right now.

No clue how to actually attach pictures here, but I have attached an updated picture using the inputs you did. We now have the code plotting the ground state no matter what spins you input so that the energies match up better. We can talk about this further at the meeting tomorrow, but this is where the program is at. If you download the master again then you should get the same result


feature request: allow entity as severity value

Right now, the severity values are fixed. It would be nice to optionally make these values dynamic by setting an entity as with the target value.

This is apparently already possible (although undocumented).


contextcmd is not run with absolute path of file

title.
for example, xfilesctl can run with file .profile as an argument, but that won't work due to PWD. is this intentional because xfilesctl will be spawned in the pwd of the xfiles gui?

is this intentional because xfilesctl will be spawned in the pwd of the xfiles gui?

Yeah, xfilesctl is spawned with its current working directory as the directory where the selected files are at.
So there's no need to give the absolute paths, as they are already relative to where the script is invoked.
That was done in this commit.
If you want the absolute paths in the script, you can prefix the arguments with $PWD.
(Or the commit can be reverted, if proved necessary to have the absolute paths instead).

it 100% should be absolute paths instead, this can be unsafe..

Done.


block production test

:+1: Looks good. Let's merge this and then I'll integrate it with my deployment branch and get it to run.


add command time duration plugin

Display the last command duration in prompt.
The function only display seconds and minutes.

The last commit resolve conflict following the merge of https://github.com/Bash-it/bash-it/pull/1681

This looks like an improved version of #1651, which had some flaws.
Since you're writing the start time to a file, this feature will likely introduce some overhead. Any thoughts on setting an environment variable instead?
One other issue that I could see is that multiple shells would use the same file, wouldn't they? So if you have a couple of long-running commands in multiple terminal windows, wouldn't they all write to the same file?

I did not see https://github.com/Bash-it/bash-it/pull/1651.
It has the advantage to define a plugin and several helpers so every theme can use the duration.

Since you're writing the start time to a file, this feature will likely introduce some overhead. Any thoughts on setting an environment variable instead?

I will test if I can make it work this way

One other issue that I could see is that multiple shells would use the same file, wouldn't they? So if you have a couple of long-running commands in multiple terminal windows, wouldn't they all write to the same file?

I used a tmpfile instead of an environment variable because I have several term open / command running.
The tmp file is created with the term PID as a suffix, so there is one tmp file by open term.
$ cat /tmp/bashit_theme_execution_341704
1603255567

Let me see if I can take the good parts of https://github.com/Bash-it/bash-it/pull/1651 and make a general plugin that can work for every theme.

Thanks for the explanation with the PID, that makes sense! The approach in #1651 did not work in the end, since the start time was always set to the end of the last command - not taking any wait time by the user into account... It looks like your _pre_exec function should help with that...

Since you're writing the start time to a file, this feature will likely introduce some overhead. Any thoughts on setting an environment variable instead?

This is not possible, using PS0 to set the start time of a command happen in a subshell, so we can't retrieve the variable after.
The best way to do this seem to be with tmp files.
The generic way to define a tmp file seem to be:
tmpdir="${TMPDIR:-/tmp}"

We can use mktemp to define a unique tmp file with
tmpfile=$(mktemp "${TMPDIR:-/tmp}"/tmp.XXXXXXXX)

I'll update the pull request with those information and try to do a working plugin.

I finally had the time to work on this feature.
I took the good ideas from #1651 and made a plugin.
The plugin can be enabled with:
bash-it enable plugin cmd-duration

In your theme, you can call _command_duration function to retrieve the command duration and use it in your PS1:
    command_duration=$(_command_duration)
    PS1="${command_duration}"

To have better results with time spent on commands, you have to call _command_duration function at the very beginning of your prompt function.
Two var can be defined in your theme to style the output:
COMMAND_DURATION_COLOR="$bold_blue"
COMMAND_DURATION_ICON='  '

This is still a a work in progress, but feedback are welcome.

@nwinkler Any feedback on this PR ?
I have some time this week end to document the feature if needed.

I've added:

a better time precision: deciseconds
a way to configure the minimum time in seconds before displaying the command duration

With COMMAND_DURATION_MIN_SECONDS=1

On a sidenote, would you agree to create a linting PR for your theme as well?

Yes, I was planning to do that after this PR is merged.

Some minor change in 9732bbe1713348e4463b51c7f8d36cd3a74fd8eb to add better theming ability to command duration output.
We can now choose an icon and a color for the text.
Choosing a color for icon only can be done like that:
COMMAND_DURATION_ICON="$bold_blue  $normal"

I added the command duration to powerline theme in 384b31da98af683f2d90e7524518e7cdecc25a77 as it's a popular one:

I'm reverting the commit for this PR, because powerline theme is modified in master.

f1d27faa537ecab6832ab178a49642c16bc2dafa remove deciseconds precision for command duration > 1 minute.
Before:

After:

@nwinkler any feedback on this PR ?

I merged the master branch in this PR to resolve a conflict.


Server Library Compatibility Issue

ROS 1 version: Noetic
Ubuntu version: 20.04.6
Robot system version: 4.2.2 (pandas)
Robot / Gripper Server version: 5/3
libfranka version: 0.9.2
franka_ros version: 0.9.1
When I run the communication test (sudo communication_test <IP_ADDRESS>), I receive the following error:
libfranka: Incompatible library version (server version: 5, library version: 4).
Please check https://frankaemika.github.io for Panda system updates or use a different version of libfranka.

According to the compatibility docs, my installed versions of the libraries should be correct, so I am not sure what the issue is. Any ideas?

Hi,
Please update sub-modules and try again.
git submodule update --init --recursive
Thanks!

I ran git submodule update --init --recursive within  libfranka and recompiled the library. I'm still running into the same issue.
However, I have discovered that I can run the examples within libfranka (like libfranka/examples/communication_test.cpp).  The incompatibility problem just occurs when I try to use communication_test with  franka_ros.

Solution(s)
I fixed the issue. It turns out there were a couple problems going on:

I had the ros-noetic-libfranka library installed on my machine which was interfering which my version of libfranka built from source like in this other issue.
SOLUTION: I uninstalled the library using sudo apt remove ros-noetic-libfranka and recompiled my catkin_ws.

I didn't build the libfranka library into a debian package since that step was marked as optional in the installation guide. This caused franka_ros to try to use  libfranka 0.8.0 (which I didn't install) instead of the correct 0.9.2 version when I ran catkin_make -DCMAKE_BUILD_TYPE=Release -DFranka_DIR:PATH=../libfranka/build. This other issue was similar to the problem I was facing.
SOLUTION: I built the debian package using the following commands in my libfranka/build directory:
cpack -G DEB
sudo dpkg -i libfranka-0.9.2-x86_64.deb

Then in my catkin_ws, I could compile franka_ros properly using:
catkin_make -DCMAKE_BUILD_TYPE=Release
source devel/setup.sh

Documentation
I ended up running everything in a Docker Container. The directions and Dockerfile for setting up this process are here.


ROE-1339 Should fail validation if both due diligence options are sup…

…plied.
https://companieshouse.atlassian.net/browse/ROE-1339
Introduced dedicated code to validate both due diligence dtos because the web never sends a null value only an empty one so it is the content that needs to be checked.

there's 3 code smells to be looked at

I think this needs some discussion, I am not sure what the right strategy is here.


Memcache expire does't work in special cases

What steps will reproduce the problem?
https://github.com/yiisoft/yii2/issues/17063#issue-402056858 , this problems arise for many people.
    $resultSet = Yii::$app->cache->set('test', 'ok', 60);
    var_dump($resultSet);  //ture
    $resultGet = Yii::$app->cache->get('test');
    var_dump($resultGet);  //false

Because memcached internally uses a clock based on the monotonic clock, php time() uses sytems timestamp.
If memcached has been running for a long time since startup,In framework/caching/MemCache.php#L298 $expire probably behind memcached internally timestamp, so the problem happened.
see https://github.com/php-memcached-dev/php-memcached/issues/368#issuecomment-359137077
https://github.com/yiisoft/yii2/blob/f0de56b81e4fb6731304fd070f8d2f72a040e52e/framework/caching/MemCache.php#L298
What is the expected result?
I suggest $expire use the number of seconds, not Unix timestamp, although this will cause $expire may not exceed 30 days, but can be solved this problems
What do you get instead?
Additional info

Q
A

Yii version
2.0.24

PHP version
7.3.12

Operating system
centos 7

That changes behavior very significantly so I'm not sure it's a good idea overall.

@yiisoft/reviewers what do you think?

It shouldn't hurt if we do this only for $expire smaller than 30 days.

I like @rob006 solution. We can reliably detect the intent in this case. If value is less than 2592001 then it's duration. Otherwise it's timestamp.


Consider renaming data_file field  on FileScanTask to maintain consistency and  avoid confusion

I have been running some tests with the TableScan API, and I think that there is something a bit puzzling about naming and access there.

TableScanBuilder's build() returns a TableScan.
TableScan's plan_files() returns a stream of FileScanTask

so far so good...

FileScanTask has a field named data_file, but that field doesn't return DataFile. Rather, it returns a ManifestEntry (see here).
ManifestEntry also has a field named data_file, which does return an actual DataFile type, but it does not expose it (see here). ManifestEntry only exposes some of DataFile fields via public methods.

Therefore, I think there are 2 issues here:

Because we do have a DataFile type, I think its confusing that FileScanTask has a field named data_file, which returns a ManifestEntry rather than a DataFile.
The DataFile fields and methods are public but are not effectively accessible via TableScan, because ManifestEntry "hides" DataFile itself.

It seems that the original design was based on the idea that ManifestEntry should encapsulate and hide DataFile. If we want to address this, I think we have 2 choices here:  The less optimal choice, in my view, is to follow this encapsulation logic and only expose DataFile fields via public methods on ManifestEntry.
The other option, is to rename the data_file field on FileScanTask to manifest_entry whereas the data file fields will be accessed like: file_scan_task.manifest_entry().data_file().some_property
I have a small PR that shows the latter suggestion. Please let me know if this sounds like a good direction and I will post it for your consideration

The other option, is to rename the data_file field on FileScanTask to manifest_entry whereas the data file fields will be accessed like: file_scan_task.manifest_entry().data_file().some_property

This makes the most sense to me.

Hi, @a-agmon Thanks for raising this. Let me add some background here, the reason the field is named as data_file is that, in fact for each FileScanTask we have not only data, but also deletion tasks, e.g. position deletes and eq delete files. Currently they are not included because the logic of applying and pruning deletion files is a little complicated, and we
decided to focus on read only data first.

The other option, is to rename the data_file field on FileScanTask to manifest_entry whereas the data file fields will be accessed like: file_scan_task.manifest_entry().data_file().some_property

This is reasonable to me, but I have a small suggestion to rename data_file to data_manifest_entry, what do you think?

This is reasonable to me, but I have a small suggestion to rename data_file to data_manifest_entry, what do you think?

Can we not simply use manifest_entry instead of data_file and have the delete_file named explicitly like that, once we get there? I think this is how the java impl does it (I only took a quick look - so bear with me if I'm mistaken)? https://github.com/apache/iceberg/blob/main/api/src/main/java/org/apache/iceberg/FileScanTask.java

This is reasonable to me, but I have a small suggestion to rename data_file to data_manifest_entry, what do you think?

Can we not simply use manifest_entry instead of data_file and have the delete_file named explicitly like that, once we get there? I think this is how the java impl does it (I only took a quick look - so bear with me if I'm mistaken)? https://github.com/apache/iceberg/blob/main/api/src/main/java/org/apache/iceberg/FileScanTask.java

Sorry I don't quite get your meaning, do you mean to use manifest_entry for data file, and deletes for deletions?

Hi, @a-agmon Thanks for raising this. Let me add some background here, the reason the field is named as data_file is that, in fact for each FileScanTask we have not only data file, but also deletion files, e.g. position deletes and eq delete files. Currently they are not included because the logic of applying and pruning deletion files is a little complicated, and we decided to focus on read only data first.

The other option, is to rename the data_file field on FileScanTask to manifest_entry whereas the data file fields will be accessed like: file_scan_task.manifest_entry().data_file().some_property

This is reasonable to me, but I have a small suggestion to rename data_file to data_manifest_entry, what do you think?

Sounds good. Will submit a PR for us to consider.

Hi,
Please see this PR and let me know what do you think
https://github.com/apache/iceberg-rust/pull/300

Sorry I don't quite get your meaning, do you mean to use manifest_entry for data file, and deletes for deletions?

Yes. Something like that.

Not sure if that is what you mean, but DataFile has a content field which is basically an enum that indicates whether its Data, PositionDeletes or EqualityDeletes
so that can be accessed via data_manifest_entry.data_file().content()


Remove wrong code from JarParser.java.

refactor JarParser.java to remove inappropriate code.

I can not merge this PR because it is marked as Draft

I still can not merge this pull request until you remove the draft state of this PR

Add associations support and refactor JarParser.
Also add a new package for builders.


Upgrade google.golang.org/grpc to 1.63.0

Which problem is this PR solving?

Fix #5330

Description of the changes

Upgrade google.golang.org/grpc to 1.63.0
Fix lint
WIP: Fix test

How was this change tested?

make lint
make test (need to fix)

Checklist

[x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md
[x] I have signed all commits
[ ] I have added unit tests for the new functionality
[ ] I have run lint and test steps successfully

for jaeger: make lint test
for jaeger-ui: yarn lint and yarn test

Honestly, I have no idea at the moment.
But looking at the CI message, it looks like something is wrong with the connection.
They all end in timeout and bind: address already in use in the gRPC TLS test.
https://github.com/jaegertracing/jaeger/actions/runs/8602860896/job/23573382151?pr=5336#step:7:81
https://github.com/jaegertracing/jaeger/actions/runs/8602860896/job/23573382151?pr=5336#step:7:235
My guess might be Dial and NewClient. The former has passthrough as the default scheme, and the latter has dns. I'll try to see if I can fix it.

don't look at the last error message. I see that these tests timeout (the first error in the logs):
$ go test -test.timeout=10s -v ./cmd/agent/app/reporter/grpc/
..
panic: test timed out after 10s
running tests:
	TestBuilderWithCollectors (10s)
	TestBuilderWithCollectors/with_collector_connection_status_ready (10s)


Make sure tasks are loaded

running mix harness when asdf_elixir_version for a harness_process_manager has been set to 1.15.8-otp-25 results in the following error:
❯ mix harness
==> nimble_options
Compiling 3 files (.ex)
Generated nimble_options app
==> harness_process_manager
Compiling 1 file (.ex)
Generated harness_process_manager app
==> harness
Compiling 2 files (.ex)
error: module Projection is not loaded and could not be found. This may be happening because the module you are trying to load directly or indirectly depends on the current module
  lib/evolver.ex:4: Evolver (module)

== Compilation error in file lib/evolver.ex ==
** (CompileError) lib/evolver.ex: cannot compile module Evolver (errors have been logged)
    (elixir 1.15.8) expanding macro: Kernel.use/2
    lib/evolver.ex:4: Evolver (module)

further tinkering revealed that not all harness tasks are available when running "harness.compile" inside Mix.Tasks.Harness.run/1
in mix codebase the private fetch/1 function doesn't find the appropriate task modules for Harness.Compile and Harness.Check, maybe some others also:
https://github.com/elixir-lang/elixir/blob/4e6d67c169d75476677a768f5cff553a3c8f288b/lib/mix/lib/mix/task.ex#L316
but if we load tasks from the archive install location, then all task modules are available, and mix harness runs without issues

the proposed solution might not be the only one, but seems the simplest to me, and I'm open for any suggestions

further testing reveals that even with this change, we will still need to modify our harness libraries, more precisely the mix.exs template
I've found that both of these things work:
  def application do
    [
      mod: {<%= @app |> pascal() %>.Application, []},
      extra_applications: [:logger, :runtime_tools, :sasl],
      included_applications: [:harness]
    ]
  end

and
  defp aliases do
    Code.ensure_compiled(Harness)
    Code.ensure_compiled(Harness.Manifest)
    Code.ensure_compiled(Mix.Tasks.Harness.Check)

    [
      "ecto.setup": ["ecto.create", "ecto.migrate"],
      "ecto.reset": ["ecto.drop", "ecto.setup"],
      compile: ["harness.check", "compile"],
      test: ["ecto.reset --quiet", "ecto.migrate", "test"]
    ]
  end

without one of those "harness.check" task is not available:
❯ mix bless
==> event_listener
Generated event_listener app
Running compile with args ["--warnings-as-errors", "--force"]
==> extrack_translation_pm
Generated extrack_translation_pm app
** (Mix) The task "harness.check" could not be found. Did you mean "archive.check"?

there might be a nicer way to this, but I wasn't able to find it

I don't have a strong intuition on which is better based on how they work internally or what's more idiomatic for Elixir. so all I'm left with is a preference for the ensure_compiled inside aliases since it at least puts the solution near the thing that depends on it in the code.

yeah, same, although I'd wrap those in a function and just have it like that


Sash dragging moves too slow

The Sash dragging motion does not work fluence as I expect (compares to visual studio code).
Some corner cases I have already excluded:

[x] When the renderer process only renders a single Sash object, still moves too slow. So I believe it is not a general performance issue to the whole program.
[x] I remove all the dynamic import CSS files, still moves too slow.
[x] I used the same main.ts and sash.ts codes in another basic electron program, that program still works perfectly. So I believe it is not an issue from the main process or the Sash implementation.
[x] Used the same package.json in that basic electron program, still works. So it is not any version issues.

 Now it is commented out.


fix: spark can't use options on PHP 7.4

Description
Fixes #5835
Follow-up #5206
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[x] User guide updated
[x] Conforms to style guide

I'm okay with this breaking change given that it is inherently bugged.

Added changelog.


fix: #19

As title said.

[ ] Documentation
[x] Unit Tests
[ ] Code complete

wxml-parser is eslint-plugin-wxml's direct dependency, we also need test eslint-plugin-wxml (but you don't need care this, i will do it).


Why does importing of other tasks not work in Enter-Build block?

As mentioned here, I try to use Enter-Build for initialization:
Enter-Build {
. '..\Common.tasks.ps1' # this contains some_task
. '..\Other.tasks.ps1'

$var1 = $param1
...
}

When I run it, I get an error:
ERROR: Task 'Main': Missing task 'some_task'.
When I place dot-sourcing statement just in the script, not using Enter-Build block, then it works.

Enter-Build is called after adding all tasks and before invoking these added tasks. Thus, tasks cannot be added in Enter-Build. In other words, "adding tasks" is not "initialization".

When I place dot-sourcing statement just in the script, not using Enter-Build block, then it works.

This is the correct way of adding tasks.

TODO: Think about failing on adding tasks after loading build scripts.

v5.6.2 fails on adding tasks after loading build scripts.


Cannot find package 'minimist' on node packages

Hello,
I'm working on a CLI to generate boilerplates for React apps. For this I'm using Plop. Everything works locally, but when I try to use the published package from npx (or installed globally with npm) I get this error:
node:internal/modules/esm/resolve:839
  throw new ERR_MODULE_NOT_FOUND(packageName, fileURLToPath(base), null);
        ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'minimist' imported from<EMAIL_ADDRESS>    at packageResolve (node:internal/modules/esm/resolve:839:9)
    at moduleResolve (node:internal/modules/esm/resolve:908:18)
    at defaultResolve (node:internal/modules/esm/resolve:1038:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:554:12)
    at ModuleLoader.resolve (node:internal/modules/esm/loader:523:25)
    at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:246:38)
    at ModuleJob._link (node:internal/modules/esm/module_job:126:49) {
  code: 'ERR_MODULE_NOT_FOUND'
}

Any idea why?
The npm package can be found here https://www.npmjs.com/package/@julseb-lib/julseb-cli
This is how I use minimist:
import path from "node:path"
import minimist from "minimist"
import { Plop, run } from "plop"

const args = process.argv.slice(2)
const argv = minimist(args)

import { dirname } from "node:path"
import { fileURLToPath } from "node:url"

const __dirname = dirname(fileURLToPath(import.meta.url))

Plop.prepare(
    {
        cwd: argv.cwd,
        configPath: path.join(__dirname, "scripts/plopfile.js"),
        preload: argv.preload || [],
        completion: argv.completion,
    },
    env => Plop.execute(env, run)
)

Thanks!

You have it listed as a dev dep instead of a runtime dep (also child_process should not be a dep, it's a built-in module)


Corrected rotation

Rotation functionality was rotating counter-clockwise. The CSS transform function rotates clockwise, and I've adjusted the rotationTransformationMatrix to match.

Hmm. Cant we change it in context2d?

No, this transform is the source of the rotation being made counter-clockwise rather than clockwise. We do not have reference to the rotation parameter on the context('2d') function.

Yeah, we can decompose the transformation matrix etc..
I am just worried, that we are changing the  behaviour of the API.


Property title have an incomplete content

Describe the bug
Items from database have an incomplete title
To Reproduce
Node version: v18.2.0
Notion JS library version: 1.0.4
Steps to reproduce the behavior:
Expected behavior
That my title have a full content
Screenshots

My output from my code
const getProducts = async () => {
  const database_id = "";

  const db = await notion.databases.query({
    database_id,
  });

  const records = [];

  for (const iterator of db["results"]) {
    const entry = iterator["properties"];

    const data = {
      Title: entry["Title"].title[0].text.content,
      Price: String(entry["Price"].number),
      Category: entry["Category"].select.name,
      Brand: entry["Brand"].select.name,
      Status: entry["Status"].select.name,
    };

    if (data.Status === "True") records.push(data);
  }

  return records;
};

[
  {
    Title: 'P',
    Price: '550',
    Category: 'Jeans',
    Brand: 'Urban Pipeline',
    Status: 'True'
  },
  {
    Title: 'P',
    Price: '650',
    Category: 'Jeans',
    Brand: 'Magellan',
    Status: 'True'
  },
  {
    Title: 'P',
    Price: '550',
    Category: 'Jeans',
    Brand: 'Magellan',
    Status: 'True'
  },
  {
    Title: 'Pantalón Vestir',
    Price: '550',
    Category: 'Jeans',
    Brand: 'Croft & Barrow',
    Status: 'True'
  },
  {
    Title: 'Pantalón - Corte Bota  - 2 pzas - 32x32',
    Price: '550',
    Category: 'Jeans',
    Brand: 'Urban Pipeline',
    Status: 'True'
  }
]

Hi @zavbala , thanks for the bug report. Bugs with the API should be emailed to<EMAIL_ADDRESS>not opened as an issue in this repo. As a result I'm closing this issue.
That said, title is an array of rich text objects and it looks to me like your code is only taking into account the first element of each title array. If your page titles happen to contain hyperlinks, mentions, or formatting, those will get truncated by your code. Hope that helps.


[Question] Why there is "xterm" in the name of this code ? Does it fit well with xterm.js ?

Hi,
I'm working with Xterm.js but at this moment I have some troubles to open X terminals with X streams.
So I typed : xterm multiple streams on Google and I got https://www.npmjs.com/package/websocket-stream-xterm who is your code. Then I searched into it but I can't find any xterm things...
So could you help me to understand that ?
Thank you

We do not maintain that package. I am not sure why it points here.

Okay so it's nice to know. Have a nice day


Add note command

Format: note INDEX o/[NOTE]

NOTE can be left blank, which will clear the Person's note (if any)
NOTE can take in any alphanumeric, punctuations and whitespaces.

First, the NoteCommandParser object parses the command and creates a NoteDescriptor object (under NoteCommand as a static class) using Note and LastModifiedTime objects. Note can be empty, in which case it is just wrapping an empty String. Then it creates a NoteCommand that takes in the Index of Person to be noted and the NoteDescriptor. Through NoteCommand#execute(), replaces the original Person object (by Index), after editing the Note and LastModifiedDateTime object in the Person.
Confirmation string is then printed, which is just "Index: , Note: ".
A note that is too long will cause it to be truncated, in which we have two ways to resolve:

Make PersonViewPanel scrollable
Set a character limit
Still in draft phase, actual PR pending

Closes #173

Decided to set character limit at 1000. See the UI above for how the Ui looks like with note of 1000 characters.

The PR should be ready for review now.


ARROW-14853: [C++][Python] Improve error message for missing function options

Some compute function have required options (because there is no reasonable default value).
Currently they would raise a cryptic error:
>>> pc.partition_nth_indices([1,2,3])
Traceback (most recent call last):
  ...
ArrowInvalid: Attempted to initialize KernelState from null FunctionOptions

With this PR, the error becomes slightly more insightful:
>>> pc.partition_nth_indices([1,2,3])
Traceback (most recent call last):
  ...
ArrowInvalid: Function 'partition_nth_indices' cannot be called without options

Based on PR #11955.


Retry marking CARRY4_VPR as whitebox

Rebased a PR (https://github.com/SymbiFlow/symbiflow-arch-defs/pull/2082) previously made by Dan Ravensloft
Signed-off-by: Dan Ravensloft<EMAIL_ADDRESS>

I don't have a problem with trying, but what changed such that we expect this to return a different result?

My understanding was there would be an install issue which would cause CI to fail otherwise.

My understanding was there would be an install issue which would cause CI to fail otherwise.

I believe last time we tried this it caused an "as expected" packing issue.  The solution was then to remove the black/whitebox completely so that the cell was opaque to abc.  This resulted in a smaller negative (or eligible) CPD impact, and was merged.

My bad, then I will close.


Gulp updates

Updates gulp and uses the much more helpful gulp-help instead of gulp-task-listing:

@tsmith512 if you could pull this down and make sure an npm i works I'd appreciate it!

I'll just assume this works and find out I'm wrong later :grin:


adding Cibyl to the tool section

In the case it would be "awesome" enough by your standards, it's located here :
https://github.com/senselogic/CIBYL

Yes. Please open a PR for that.


SliverWithKeepAliveWidget docs

customer-testing seems to keep failing on https://github.com/flutter/flutter/pull/69523 with dependency conflicts:
|
| ---- Log transcript ----
|
| FINE: Pub 2.12.0-3.0.dev
|
| MSG : Resolving dependencies...
|
| SLVR: fact: flutter_svg is 0.19.1
|
| SLVR: derived: flutter_svg
|
| SLVR: fact: flutter_svg requires Flutter SDK version >=1.24.0-6.0.pre <2.0.0
|
| SLVR: conflict: flutter_svg requires Flutter SDK version >=1.24.0-6.0.pre <2.0.0
|
| SLVR: Version solving took 0:00:00.077470 seconds.
|
|     | Tried 1 solutions.

Testing to see if this happens for me as well.

Customer testing seems to pass, so it's likely due to some inconsistency in the working branch's state when the test failure appears. I'll close this PR now.