Use the LATTICE planner fails - The planning Tab on Dreamview is disabled automatically.

Hi I am below my System Information:

OS Plataform: Ubuntu 18.04
Apollo Installed from Docker Images, following SVL tutorial: https://www.svlsimulator.com/docs/system-under-test/apollo-master-instructions/
Apollo Version: 6.0
Apollo installed from (source or binary):

Steps to reproduce the issue:
The first steps to reproduce the error is following the tutorial link above.
1 - Start the docker image: ./docker/scripts/dev_start.sh
2 - Enter the docker image: ./docker/scripts/dev_into.sh
Before build the workspace I tried to modify some files   to switch the standard ROAD_PUBLIC planner to use the LATTICE planner for a simple evasive maneuver against a  static obstacle. To achieve that I have changed the line 18 of the file planning_config.pb.txt:

Removed ROAD_PUBLIC and added LATTICE;

Afterwards I change the the line 30 of on_lane_planner_dispatcher.cc:

Changed the planner_type index from 0 to 3. Considering the number 3 is the one that defines the LATTICE planner on the planning_config.proto file:

3 - Then finally I have built the workspace: ./apollo.sh build_opt_gpu
4 - runned the bootstrap to activate the dreamview monitor: ./scripts/bootstrap_lgsvl.sh
5 - runned the cyber_bridge: ./scripts/bridge.sh
Unfortunately the planning tab on Dreamview is disabled automaticall when I turn it on, as you can check on this video:
https://youtu.be/wn6f2mhT8vY
I have done a lot of attempts:
https://youtu.be/DXmFH81SCps
If I access the Tasks panel on Dreamview

The planning module is bolded in red with the Console error:
"Raw IMU Message Delay: IMU message is 23949 cycle 24980 sec behind current time
"Raw GPS Message Delay GPS message is 23850 cycle 23805 behind current time".
Etc...
Accesing the Cyber_Monitor, I can check the planning module is not working (it is bolded in red too):

Entering the planning module channel I get:

"No Message Came"
Then If I press the planning tab again on Dreamview to activate the planning module, and look into the planning channel on cyber_monitor I get the message:

"Cannot Generate Message by Message Type"
Then I decided to look into the log files to comprehend the error. And I get the following output:

Also from the planning_config.proto I have noticed that the Planners PUBLIC_ROAD and NAVI have their configuration defined into this file. However the LATTICE has not:

Must I add a line of code: " message PlannerLatticeConfig {} in this file ?
If yes which parameters are required to define this planner?
I would like to know how to Use the LATTICE planner in Apollo 6.0.
What are the steps needed?
I also tried to use the planner in Apollo 3.0 however it was not possible to use this software alongside SVl simulator:
https://github.com/lgsvl/simulator/issues/1475
Someone know a tutorial how to set the rosbridge properly for Apollo 3.0 withing SVL simulator?
Thanks very much in advance

@marcusvinicius178 Hi,Can you use the lattice planner successfully now?
And if you know how to use lattice planner,  could you please tell me how to use it ?
Of course I know I should first change the line 18 of the file planning_config.pb.txt : ROAD_PUBLIC to LATTICE
Thanks a lot for your help!

Hi @blacksheep0303 yes! I also thought it was necessary more changes. But just change this line is enough to change the planner. I believed I was not having success because the Lattice planner never started. But then I tried it without using the LGSVL simulator, I have just used the simulation on SIM_CONTROL mode (You can activate this mode in Panel Tasks. Then the simulation will happen just on Dreamview...and the lattice planner will probably work
This happened to me because I do not have a powerful machine, and the Lattice planner requires more RAM  memory or GPU (or both), then it escaped from my computer capability run the Lattice planner and the simulator simultaneously. This is why it seems don't be working...but just change this line you said me is enough!!!

@marcusvinicius178
Thanks a lot for your help.Your help is very useful.I try to  use the simulation on SIM_CONTROL mode and LGSVL without other obstacles, and the lattice planner worked.
However , if there are other obstacles , the lattice planner progress will die when the planned path intersects the predicted trajectory of the obstacle. The error is "st_boundary.cc:374] Check failed: !lower_points_.empty() StBoundary has zero points".
So,briefly speaking,I can only run the lattice planner when there are no obstacles. Have you ever met this problem ?
By the way,I think the path generated by PUBLIC_ROAD planner is better than lattice planner ,so I haven't debugged lattice planner recently.

The LATTICE works also on simulation mode....you just need to switch to it after begin using the Road Public planner. But don't restart the simulation...
I still didn't try Lattice with obstacles.


StackTrace text is unselectable

it's annoying to have to copy the entire stack trace to the clipboard just to paste it into another app in order to obtain the part that I actually want. it would be nice if scoop used a readonly textbox for the stack trace, rather than a normal label, so that text can be selected and copied directly from there.

Sorry for the late response, but this is already possible if you press the "toggle selection mode" icon next to the search icon. You can then select the portion that you want.

Bit weird that you have to activate a special mode to do this. But thanks!

Yeah, I'm not happy with the app design. That's usually what happens when working with several-years-old written by someone else. The technical debt builds up.
I've been wanting to completely rewrite and redesign it from the ground up for a long while (because the code is terribly designed and hard-to-understand in several places), but trying to keep up with yearly Android releases and all their API changes is so tiring. Life stuff, school, and other priorities also get in the way, and lower-importance hobby stuff like this takes the backburner. Also, there was a change in Android 13 that severely restricts this app's functionality (which further dampens my motivation). A majority of open issues are just a result of the app's flawed design that would take a bunch of rewriting to fix, and I don't want to do that. I'd rather just throw it all out and start anew. Ironically, that would require me to dedicate a bunch of time to studying Compose, but I'd be willing to do that.
Sorry for the dump, but I felt it was pretty relevant lol.

It might be worth looking at alternatives like LogFox and port missing functionality there. They found a way to cope with the restrictions.


Handle inlined sources in d.ts maps to support goto definition and other such functionalities

Found as part of #35014

@DanielRosenwasser @RyanCavanaugh Do we want to handle source maps where source code is inlined for goto def etc. And if yes we need to figure out what the interaction with editor is going to look like since file a.ts that editor refers to may not same as what file a.ts source map says is.
This is definitely a feature like thing so doesnt make sense to put in 4.0.1 so removing the mile stone for you to triage.
Thanks


using message rooms to send specific notifications

Naming Rules
Name your PR like this: ISSUENUMBER-TITLE-YOURNAME
Description
Write a summary of your change. Explain why you have made it.
Related to
Make sure you include the issue number with a hash sign # so Github can link this PR to the right issue, like this:
Fixes #1
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have carefully reviewed my own code
[ ] I have commented my code
[ ] I have updated any documentation

@gregdyke Please review the changes we worked on together yesterday.


Android app and Web3 support

Hey,
I have two questions is there an Android app and can this hardware wallet be used for web3 Ethereum uses?

Okay that's cool :) looking forward to it.


Bcrypt hook user schema

Adds hashing prehook function in User model for saving hashed passwords in db

I still need to compare hashed passwords for login


randomcats: minor tweaks for release

cache gif and nongif data separately
add cataas.com credit to manifest

@MrRobot245 FYI!


There is no .js entry code in published npm package.

As the subject.

Hmm, how do you mean? package.json defines "main": "./lib", and the lib directory contains an index._coffee file (which gets compiled to index.js pre-publish).
And indeed, you can see it works here:
https://github.com/zoomhub/zoomhub/blob/a5fbec5cea75011dc409e49af9766f8a41f27a2d/lib/worker._coffee#L23

@jacobbubu I think @aseemk answered your question but please reopen if not :smile:


Flag popo update locations

Description
Include a summary of the changes and the related issue. List any dependencies that are required for this change.
Linked Items:
Closes/Fixes/Resolves #XYZ
Added

Describe any new features.

Changed

Describe any changes in existing functionality.

Deprecated

Describe any deprecated features.

Removed

Describe any removed features.

Fixed

Describe any bug fixes.

Security

Describe any security-related changes.

Checklist:

[ ] I updated project documentation to reflect my changes.
[ ] My changes generate no new warnings.
[ ] I updated new and existing unit tests to account for my changes.
[ ] I linked the associated item(s) to be closed.
[ ] I bumped the version.
[ ] I added the labels corresponding to my changes.

If you merge the latest popo-first-party-plugin-comments-pm branch, if might eliminate some of the flake8 errors.

That would be mixing work areas and task scopes. This is just flagging within validators.


Multiple objects with same value of primary key field.

I've got object with children of the same class. I wrote mapping and set primary keys.
Here is my mapping: https://gist.github.com/AlloyDev/c59a0e40dbd0fd0e9c52
I map objects with + (NSArray *)arrayOfObjectsFromExternalRepresentation:(NSArray *)externalRepresentation                                           withMapping:(EKManagedObjectMapping *)mapping                                inManagedObjectContext:(NSManagedObjectContext*)context method.
For some reason I received objects with same clusterId field.

You don't actually need separate mapping for recursive mappings - they are supported out of the box. You can simply call following:
[mapping hasMany:[RNDCluster class] forKeyPath:@"related" forProperty:@"childClusters"];

It was my first implementation. In that case I also have multiply clusters with same clusterId. One with clusterBody and childClusters and one without.
I think that this may be due to some clusters could be root cluster and also a child cluster for other root clusters.

I'm not sure what is the issue here. Is the data in objects different(except for clusterId)?
Can you post an example of your JSON?

Wrote small test project: https://github.com/AlloyDev/EasyMappingIssue103
As you can see cluster 30637176 duplicates

Resolved issue via manual mapping withValueBlock for related clusters. But also we need to create mapping for related clusters without mapping related clusters, children of some root clusters will be overwritten otherwise.
Updated test repo: https://github.com/AlloyDev/EasyMappingIssue103
And gist: https://gist.github.com/AlloyDev/c59a0e40dbd0fd0e9c52

I'm not sure i understand the issue here.

Your JSON contains duplicate entries for clusterId - 30637176 and 30645762 exist on parent and on child level
What happens from MagicalRecord standpoint, is when you call MR_deleteAllMatchingPredicate, it selects following objects - http://d.pr/i/eLGq, which results in deleting every single RDNCluster object

I'm not sure i understand the issue here.

Your JSON contains duplicate entries for clusterId - 30637176 and 30645762 exist on parent and on child level
What happens from MagicalRecord standpoint, is when you call MR_deleteAllMatchingPredicate, it selects following objects - http://d.pr/i/eLGq, which results in deleting every single RDNCluster object

Yes, problem is here. I thought, that everything is ok and manual mapping works fine, but sometimes it breaks on real data. The issue is EM creates objects with same primary key in case same objects on parent and child level.
Everything is ok here, I just tested deleting and forgot to remove this check.

Are you saying that example project does not have this issue?
We need to replicate this behaviour to write a reliable test. Can you narrow down on which JSON you are getting this?

Yes, I thought that I had solution.
To present I resolve issue by mapping all fields except related via EM and then just set parent-child relations manually. Also I wrote test for it in project.
Example of test: https://gist.github.com/AlloyDev/438405e7dfd007784317
Json for test: https://gist.github.com/AlloyDev/ae2e38fe1fccdb027a2f

Can you update example project with this code?

Updated project. There are three common implementations in RDNCluster+RDNMapping except manually adding parent-child relations.

Thanks you for taking time and making detailed example and tests.
I have spent some time researching this stuff, but it seems you have duplicated clusterID's once again in your JSON - http://d.pr/i/qaXf
It's not clear how EasyMapping would even deal with these kinds of JSON. Even if we were caching objects before their full initialization, behavior would still be incorrect - you would be replacing parent object content with the child one.
It seems you are having issue with your back-end, not with EasyMapping. You probably should not get duplicated clusterID on children objects in the way you are receiving.
I'm closing this issue for now, cause it seems EasyMapping if behaving correctly( or at least reasonable, given this kind of JSON).
Feel free to reopen this issue or create a new one, if there is duplication in different scenarios.


bootstrap-st2express on AWS

Tried to run AIO installer on AWS (EC2) Ubuntu Server 14.04 LTS (HVM)
Preparation steps
# set hostname
echo 'ec2-54-246-35-90.eu-west-1.compute.amazonaws.com' > /etc/hostname
# set FQDN as a first line of /etc/hosts
echo -e "<IP_ADDRESS> ec2-54-246-35-90.eu-west-1.compute.amazonaws.com\n$(cat /etc/hosts)" > /etc/hosts

sudo service hostname restart

root@ip-10-106-9-138:/home/ubuntu# hostname
ec2-54-246-35-90.eu-west-1.compute.amazonaws.com

Installer result
Notice: /Stage[main]/Profile::St2server/Exec[register all st2 content]/returns: executed successfully
Notice: /Stage[main]/St2::Packs/St2::Pack[hubot]/Exec[install-st2-pack-hubot]/returns: 2015-10-07 20:56:22,727  WARNING - Auth API server is not ava    ilable, skipping authentication.
Notice: /Stage[main]/St2::Packs/St2::Pack[hubot]/Exec[install-st2-pack-hubot]/returns: ERROR: HTTPSConnectionPool(host='ec2-54-246-35-90', port=9101    ): Max retries exceeded with url: /actions/packs.install (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnectio    n object at 0x7f5484148610>: Failed to establish a new connection: [Errno -2] Name or service not known',))
Error: st2 run packs.install packs=hubot    returned 1 instead of one of [0]
Error: /Stage[main]/St2::Packs/St2::Pack[hubot]/Exec[install-st2-pack-hubot]/returns: change from notrun to 0 failed: st2 run packs.install packs=hu    bot    returned 1 instead of one of [0]
Notice: /Stage[main]/St2::Packs/St2::Pack[hubot]/File[/opt/stackstorm/packs/hubot/config.yaml]: Dependency Exec[install-st2-pack-hubot] has failures    : true
Warning: /Stage[main]/St2::Packs/St2::Pack[hubot]/File[/opt/stackstorm/packs/hubot/config.yaml]: Skipping because of failed dependencies
Notice: /Stage[main]/St2::Packs/St2::Pack[st2]/Exec[install-st2-pack-st2]/returns: 2015-10-07 20:57:39,374  WARNING - Auth API server is not available, skipping authentication.
Notice: /Stage[main]/St2::Packs/St2::Pack[st2]/Exec[install-st2-pack-st2]/returns: ERROR: HTTPSConnectionPool(host='ec2-54-246-35-90', port=9101): Max retries exceeded with url: /actions/packs.install (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f08d3566610>: Failed to establish a new connection: [Errno -2] Name or service not known',))
Error: st2 run packs.install packs=st2    returned 1 instead of one of [0]
Error: /Stage[main]/St2::Packs/St2::Pack[st2]/Exec[install-st2-pack-st2]/returns: change from notrun to 0 failed: st2 run packs.install packs=st2    returned 1 instead of one of [0]
Notice: Applied catalog in 925.76 seconds

███████╗████████╗██████╗      ██████╗ ██╗  ██╗
██╔════╝╚══██╔══╝╚════██╗    ██╔═══██╗██║ ██╔╝
███████╗   ██║    █████╔╝    ██║   ██║█████╔╝
╚════██║   ██║   ██╔═══╝     ██║   ██║██╔═██╗
███████║   ██║   ███████╗    ╚██████╔╝██║  ██╗
╚══════╝   ╚═╝   ╚══════╝     ╚═════╝ ╚═╝  ╚═╝

  st2 is installed and ready to use.

First time starting this machine up?
Visit https://<IP_ADDRESS>/setup to configure StackStorm
Otherwise, head to https://ec2-54-246-35-90.eu-west-1.compute.amazonaws.com to access the WebUI

If you would like to use the CLI interface, please log out and
login to your session again. This is necessary to properly source
StackStorm ENV variables
root@ip-10-106-9-138:/home/ubuntu# 2015-10-07 20:57:56,692  WARNING - Auth API server is not available, skipping authentication.
ERROR: HTTPSConnectionPool(host='ec2-54-246-35-90', port=9101): Max retries exceeded with url: /actions/core.local (Caused by NewConnectionError('<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x7f684cec4410>: Failed to establish a new connection: [Errno -2] Name or service not known',))

ne^C

Both EC2 instance public DNS https://ec2-54-246-35-90.eu-west-1.compute.amazonaws.com/ and public IP https://<IP_ADDRESS>/ points to WEBUI.
But for setup https://<IP_ADDRESS>/setup/ I get 502 Bad Gateway

Also, might be good info:
root@ip-10-106-9-138:/home/ubuntu# wget https://<IP_ADDRESS>/setup --no-check-certificate

--2015-10-07 21:26:58--  https://<IP_ADDRESS>/setup
Connecting to <IP_ADDRESS>:443... connected.
    WARNING: certificate common name ‘ec2-54-246-35-90’ doesn't match requested host name ‘<IP_ADDRESS>’.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: https://<IP_ADDRESS>/setup/ [following]
--2015-10-07 21:26:58--  https://<IP_ADDRESS>/setup/
Reusing existing connection to <IP_ADDRESS>:443.
HTTP request sent, awaiting response... 502 Bad Gateway
2015-10-07 21:26:58 ERROR 502: Bad Gateway.

Also got 502 for /setup:
root@ip-10-106-9-138:/home/ubuntu# wget https://<IP_ADDRESS>/setup/ --no-check-certificate

--2015-10-07 21:32:18--  https://<IP_ADDRESS>/setup/
Connecting to <IP_ADDRESS>:443... connected.
    WARNING: certificate common name ‘ec2-54-246-35-90’ doesn't match requested host name ‘<IP_ADDRESS>’.
HTTP request sent, awaiting response... 502 Bad Gateway
2015-10-07 21:32:18 ERROR 502: Bad Gateway.

Fixed by #155. Please confirm?

Yes, worked with update-system after fix.
Just to be sure, trying 100% clean install and closing.

Everything worked.


Adding LICENSE file to project

fixes: #83

squash commits, otherwise looks good

Sure! You can squash and merge it.

@kshithijiyer I should have clarified - please squash your commits. With a few exceptions, we require that contributors squash their commits before we merge each PR.

@kshithijiyer I should have clarified - please squash your commits. With a few exceptions, we require that contributors squash their commits before we merge each PR.
Done.

/ok-to-test

/hold
Issues with Imagestreams and builds - needs investigation of the OpenShift controller manager

Builds tests are referencing an obsolete Imagestream. Need to fix that
first.
On Sat, Jul 20, 2019, 8:17 AM OpenShift CI Robot<EMAIL_ADDRESS>wrote:

@kshithijiyer https://github.com/kshithijiyer: The following test
failed, say /retest to rerun them all:
Test name Commit Details Rerun command
ci/prow/e2e-aws-builds a65b770
https://github.com/openshift/builder/commit/a65b7700537389b91d59b671fd5e69285e39dcac
link
https://prow.svc.ci.openshift.org/view/gcs/origin-ci-test/pr-logs/pull/openshift_builder/85/pull-ci-openshift-builder-master-e2e-aws-builds/304 /test
e2e-aws-builds
Full PR test history
https://prow.svc.ci.openshift.org/pr-history?org=openshift&repo=builder&pr=85.
Your PR dashboard
https://prow.svc.ci.openshift.org/pr?query=is:pr+state:open+author:kshithijiyer.
Please help us cut down on flakes by linking to
https://github.com/kubernetes/community/blob/master/contributors/devel/flaky-tests.md#filing-issues-for-flaky-tests
an open issue
https://github.com/openshift/builder/issues?q=is:issue+is:open when you
hit one in your PR.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/openshift/builder/pull/85?email_source=notifications&email_token=AALXWFVIEBDKOXWBXZ76DG3QAL65DA5CNFSM4IDYCF52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2NNDRQ#issuecomment-513462726,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALXWFUNMCZJ22IFNJTLGLDQAL65DANCNFSM4IDYCF5Q
.

/retest

/hold
Issues with Imagestreams and builds - needs investigation of the OpenShift controller manager

Don't we need to unhold it?

/hold cancel


test-lib: add ct_random_string function

Please take a look, this should work fine for environments with blocked sigpipe..

lgtm, thanks! Merging


Multiple iOS app uses same Twitter App for Login

Our company has a framework that uses TwitterKit.
All of our iOS apps are using this framework.
Our aim to doing this to manage users in only one Twitter Application.
I implemented this framework to all of our apps
When I try to Login with Twitter from Application B, it redirects to Twitter iOS application.
Twitter asks me "Do you want to give permission to Application B ?"
After I accept the permission, Twitter App redirects to Application A.
But it should redirect me to Application B.
I am using Twitter iOS Application in my iPhone 6S Plus
iOS version: 11.2.2
Twitter iOS App version : 7.15
TwitterKit version 3.3.0
TwitterCore version 3.1.0

@ahmadsallam no we I didn't. To solve this I have used old TwitterKit version in my projects.
When you press login, it redirects to safari instead of Twitter iOS app, and then it redirects back to the correct application.

Many thanks 🙏 my friend


Take screenshots of virtual level-ups

Is your feature request related to a problem? Please describe.
When achieving a virtual level (100+), no screenshot is taken as there is no level-up widget presented.
Describe the solution you'd like
When getting an XP drop that achieves a virtual level, a screenshot should be taken.

Screenshot of exactly what? You get no message, no widget, no particles, so the screenshot will be basically screenshot of you doing some skilling/combat with no context

I don't see how the widget is especially interesting. What you are doing is the purpose of the screenshot, otherwise we would just write it to a text file.

Well yes, but you are taking screenshot of nothing in this case. E.g there is no indicator that you achieved said level on the screenshot except file name.

Your taking a screenshot of what you are doing, the indicator not important.

Yeah, I guess I kind of agree with db. Given that there's not really an indicator, it would just be some random screenshot. Didn't really think too much about this request haha


Implement change detector for StatefulSet

Description
In case, when user changes configs in an existing NATS CR, there may be a situation where we need to properly edit the statefulSet, like changing PVC storage, changing replica count from 3 to 1, etc.
depends on: #33

We don't need it anymore as the upgrade from Kyma NATS to NATS-manager is already working.
https://github.com/kyma-project/nats-manager/issues/33#issuecomment-1562479024


multiprocessing enhancements

Enhancements to the multiprocessing package and related modules, especially for the intersection of surfaces with regular grids.

Codecov Report

Merging #514 (122d5b1) into master (73124d6) will increase coverage by 0.14%.
The diff coverage is 65.59%.

@@            Coverage Diff             @@
##           master     #514      +/-   ##
==========================================
+ Coverage   79.07%   79.21%   +0.14%
==========================================
  Files         176      176
  Lines       29056    29120      +64
==========================================
+ Hits        22975    23068      +93
+ Misses       6081     6052      -29

Impacted Files
Coverage Δ

resqpy/model/_model.py
96.28% <ø> (+1.35%)
:arrow_up:

resqpy/multiprocessing/multiprocessing.py
28.30% <11.11%> (-1.11%)
:arrow_down:

resqpy/multiprocessing/wrappers/grid_surface.py
77.77% <69.62%> (-22.23%)
:arrow_down:

resqpy/fault/_grid_connection_set.py
84.84% <100.00%> (ø)

resqpy/grid/_regular_grid.py
85.31% <100.00%> (+0.10%)
:arrow_up:

resqpy/model/_xml.py
91.13% <100.00%> (+0.83%)
:arrow_up:

resqpy/olio/uuid.py
61.42% <0.00%> (-4.29%)
:arrow_down:

resqpy/surface/_surface.py
72.46% <0.00%> (+5.64%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


[RTC-478] Propagadate peer crash reason

ref: https://github.com/jellyfish-dev/protos/pull/16
https://github.com/jellyfish-dev/membrane_rtc_engine/pull/368
Acknowledging the stipulations set forth:

[ x ] I hereby confirm that a Pull Request involving updates to the Software Development Kit (SDK) has been smoothly merged, currently awaits processing, or is otherwise deemed unnecessary in this context.
[ x ] I also affirm that another Pull Request, specifically addressing updates to the documentation body (commonly referred to as 'docs'), has either been successfully incorporated, is in the process of review, or is considered superfluous under the prevailing circumstances.

CI fails @Rados13

We need to merge proto first


[DOCFIX] Fix COS_ALLUXIO_BUCKET Description

Fix https://github.com/Alluxio/alluxio/issues/12677

Can one of the admins verify this patch?

Can one of the admins verify this patch?

alluxio-bot, merge this please

alluxio-bot, merge this please


GEOMESA-2934 Fix kryo serialization errors expanding large buffers

Signed-off-by: Emilio Lahr-Vivaz<EMAIL_ADDRESS>

we/I should cherry-pick this back to 2.4.x and 3.0.x


Apps using Socket Mode cannot be submitted to the App Directory

Apps using Socket Mode cannot be submitted to the App Directory
Socket Mode is not intended for publicly distributed apps.
Please enable HTTP Request URLs for your app before submitting to the App Directory.

socket mode removed - in master - only used for development on local running node with ngrok to remote - see https://slack.dev/bolt-js/reference App options developerMode:

boolean to activate the developer mode. When set to true the logLevel is automatically set to DEBUG and socketMode is set to true. However, explicitly setting these two properties takes precedence over implicitly setting them via developerMode.  Furthermore, a custom OAuth failure handler is provided to help  debugging. Finally, the body of all incoming requests are logged and  thus sensitive information like tokens might be contained in the logs.  Defaults to false.


[#1878] User definable duplication for DeduplicationHandler

This is a draft to solve #1878
Feel free to comment on the changes I made!
Although the tests ran through without modification, this change is not backwards compatible when you look at it closely, because it changes the signature of several methods (changed visibility from private to protected, add parameters etc.).

Thanks!


write diffs only

idea for a feature I have made use of to compress the output a lot & also make versioning easier for my use cases

One note about this, I implemented the option as a true/false macro, but it might be useful to be able to control this behavior with an optional argument instead - in my own project, the "write diffs only" works for every case, but I could see a situation where you want to have fully verbose exports for some mixed cases in one project, like for save data or online communication, etc.
I didn't want to mess with the implementation that deeply though, so I kept it as a separate macro that's easy to turn on/off and ignore.

Thank you for this PR! Merging was going to be a pain so I rewrote this myself


Update imports to remove all std to core

Replaced instances of std::marker::PhantomData and std::mem::MaybeUninit with their core equivalents. This change affects various source files and examples to ensure consistency and potentially reduce dependency overhead.
This will allow no-std crates to use this crate
Fixes #24
Replaces all instances of std imports with core imports
PR Checklist

[x] Tests
[x] Documentation
[ ] Pallets, that require benchmarks, are added to scripts/assets/pallets.txt (Pallet require benchmark if it has WeightInfo config type)

LGTM! Thanks 👍


Better support for pause editable ConfigEntry

Added:
Option to set a Config in Main Menu only, Pause only or Main & Pause menus
Register with custom submenu name: allows user to categorize their config if they want
Usage Example

Looks mostly good, although I don't think the RegisterWithPauseAction method is necessary
You should just be able to use the configEntry.SettingChanged event or the configFile.SettingsChanged event to accomplish the same thing, plus it'll work with other methods of changing the config

Changes are done, indeed the setting change event handler is the way to go, Im working on that for DebugUtilityMod
The custom name and location of the config entry is still a nice addition imo

LGTM


IT-related changes pulled out of PR #12368

The new IT PR has struggled to get a clean build due to the complexity and flakiness of the Druid build system. It is ironic that the attempt to fix the ITs has been stymied by the fragility of the existing ITs. Out of desperation, the big PR is being broken up into smaller chunks. This one provides changes made to the existing ITs to support the new ITs.
The first set of changes make the "custom node role" code usable by the new ITs. No functional change, just some clean-up. There are also some documentation updates.
The second change is more substantial. The new ITs use the "failsafe" Maven plugin, which is the sister plugin to the "failsafe" plugin Druid uses for unit tests (UTs). The venerable -DskipTests flag will thus skip both ITs and UTs. But, when running the new ITs, we want to skip the UTs, but run the ITs. A common solution is to use separate flags for ITs and UTs.

-DskipITs skips the integration tests but runs unit tests.
-DskipUTs skips unit tests but runs the "new" integration tests.

The existing Druid profile, -P skip-tests is expanded to skip both ITs and UTs.
In this PR, -DskipTests and -DskipUTs have the same effect. However, once the new ITs are in place, the meanings will differ, as explained above.
The key purpose of this PR is to test the new setup in Travis separately from the other changes in PR #12368.

This PR has:

[X] been self-reviewed.
[X] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[X] has been tested in the Travis Druid build (the purpose of this PR.)

Build is basically clean except for an unrelated IT failure. @kfaraz please either restart that test, or use your judgment that the failure is not relevant to the changes here.

Thanks for the changes, @paul-rogers . I have merged the PR.


cargo-zigbuild: 0.19.4 -> 0.19.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/messense/cargo-zigbuild/releases.
meta.description for cargo-zigbuild is: Tool to compile Cargo projects with zig as the linker
meta.homepage for cargo-zigbuild is: https://github.com/messense/cargo-zigbuild
meta.changelog for cargo-zigbuild is: https://github.com/messense/cargo-zigbuild/releases/tag/v0.19.5
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.19.5 with grep in /nix/store/ivxpipf7hdvhihw4zysgmz57y0n13900-cargo-zigbuild-0.19.5
found 0.19.5 in filename of file in /nix/store/ivxpipf7hdvhihw4zysgmz57y0n13900-cargo-zigbuild-0.19.5

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

cargo-zigbuild

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ivxpipf7hdvhihw4zysgmz57y0n13900-cargo-zigbuild-0.19.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cargo-zigbuild https://github.com/r-ryantm/nixpkgs/archive/28ede1d90751472c4371b36da9664d35275be3af.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/28ede1d90751472c4371b36da9664d35275be3af#cargo-zigbuild

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ivxpipf7hdvhihw4zysgmz57y0n13900-cargo-zigbuild-0.19.5
ls -la /nix/store/ivxpipf7hdvhihw4zysgmz57y0n13900-cargo-zigbuild-0.19.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

cargo-zigbuild

Maintainer pings
cc @figsoda for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/messense/cargo-zigbuild/releases.
meta.description for cargo-zigbuild is: Tool to compile Cargo projects with zig as the linker
meta.homepage for cargo-zigbuild is: https://github.com/messense/cargo-zigbuild
meta.changelog for cargo-zigbuild is: https://github.com/messense/cargo-zigbuild/releases/tag/v0.19.7
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.19.7 with grep in /nix/store/7yjr0254vrywb4kd3vbz21v1dg31i096-cargo-zigbuild-0.19.7
found 0.19.7 in filename of file in /nix/store/7yjr0254vrywb4kd3vbz21v1dg31i096-cargo-zigbuild-0.19.7

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

cargo-zigbuild

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/7yjr0254vrywb4kd3vbz21v1dg31i096-cargo-zigbuild-0.19.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cargo-zigbuild https://github.com/r-ryantm/nixpkgs/archive/b6dc676f2a6c83ca9245bb112ccd277c1b70648d.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b6dc676f2a6c83ca9245bb112ccd277c1b70648d#cargo-zigbuild

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/7yjr0254vrywb4kd3vbz21v1dg31i096-cargo-zigbuild-0.19.7
ls -la /nix/store/7yjr0254vrywb4kd3vbz21v1dg31i096-cargo-zigbuild-0.19.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

cargo-zigbuild

Maintainer pings
cc @figsoda for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@ofborg build cargo-zigbuild cargo-zigbuild.passthru.tests


Enable and Disable on Responsive view

When switching between mobile and desktop, disable and enabling is not working.

Hi!
Could you send the example code with the reproduction of this issue?

Hi,
if ($(window).width()  > 990) {
    if ($('.js-sticky-widget').length) {
        var stickyElements = Sticksy.initializeAll('.js-sticky-widget', {topSpacing: 120, listen: true})
    }
}

function onResizeStickySection() {
if ($(window).width() < 991) {
if ($('.js-sticky-widget').length) {
// Sticksy.disableAll();
var stickyElements = Sticksy.disableAll();
}
} else {
if ($('.js-sticky-widget').length) {
Sticksy.enable();
}
}
}
$(window).on('resize', onResizeStickySection);
Issue:
When resizing the browser window manually, I could not reinitialize the sticky elements.

Sticksy is a Global object. Unfortunately, it still doesn't have enableAll method. I think I can implement it on this weekend.
In your case, you just call 'enable' static method which doesn't exists. enable() and disable() methods available only for Sticksy instances .
Currently, you can turn them on manually with foreach loop.

Any update about this issue?


Feat/civ 3263

JIRA link (if applicable)
CIV-3263
Change description
Added changes so that I2P changes to Judge referral or Proceeds in heritage system depending on the claim's traits
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

claimant_response before the changes

claimant_response after the changes

claimant_response_spec before the changes

claimant_response_spec after the changes

I noticed that even when SDO is enabled and the case is kept online the last event is still 'Notify RPA on case handed offline' - is that correct? If so, I am happy to approve.

I noticed that even when SDO is enabled and the case is kept online the last event is still 'Notify RPA on case handed offline' - is that correct? If so, I am happy to approve.
Edit: Redundant as of latest commit [7029c8e] - will approve.

You are right, there should be another gateway to decide between offline and continuous notification


Date::parse now doesn't accept String

(Other parse methods as well, I think)
While it used to accept it and now doesn't, there's no real reason for it other than some method signatures changing during the big rewrite - so it needlessly breaks some code that used to work in 0.2.
The solution is simple: take impl AsRef<str> instead of &str.

Is there some reason you can't just do take a reference to the string? The intent was to prevent users from needlessly moving or cloning the value.

There's nothing sinister in doing Date::parse(something_that_returns_a_string()), no? I think there's tons of valid use cases that shouldn't really be 'prevented' from a 3rd party library's standpoint.
If users want to move a string, ok then - it means they don't need it anywhere else anymore - in which case there's no difference in passing a ref or moving it since it's going to be dropped shortly anyway.
Taking &str means "give me a reference and nothing else, up to you to extract it"; impl AsRef<str> is more like "give me something that can produce a reference, don't care what it is really" which is far more ergonomic. (my 2c, anyway)


Need proper validation in model path

Closes: RHOAIENG-3314
Description
Added condition to throw error with path is invalid, also reduced the field debounce timeout to 500ms
Valid path

Path with special chars

Path with only slash

Path with multiple slashes in between

Path starting with slashes are valid, but the slashes will be removed after submitting

How Has This Been Tested?

Open deploy model
Enter path with different special chars and check the error.

Test Impact
Added cypress test to check path with different values
Added unit test for string util function
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Commits have been squashed into descriptive, self-contained units of work (e.g. 'WIP' and 'Implements feedback' style messages have been removed)
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

If you have UI changes:

[x] Included any necessary screenshots or gifs if it was a UI change.
[x] Included tags to the UX team if it was a UI/UX change (find relevant UX in the SMEs section).

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

@vconzola

@vconzola can you review this pls.

Path starting with slashes are valid, but the slashes will be removed after submitting
This is a nit, but for the above case, if there's only one leading slash, it should be ignored and removed after submitting. If there are multiple leading slashes, that should be an "Invalid path format." Also, note, there should be a period after the word "format" in this error text.

@vconzola made your suggested changes, pls re-check

Once @vconzola lgtm it we can approve it, great work here!


Add Sensair S8 UART library

To easily interface with a Sensair S8 sensor
Author: @jcomas
Repo: https://github.com/jcomas/S8_UART

Thanks @Belphemur.
I'll add a note to myself here that there is an effort to resolve the issue blocking the merge of the PR: https://github.com/jcomas/S8_UART/issues/2
So we can leave this open until it is resolved.

@per1234 Thanks !
The developer has been unresponsive for close to a week now. I'll see if I can contact him by email.

Next days I'll test the new changes before I release the final version 1.0.0.

@jcomas Great to hear !
Thank you again for your work, I've been using it in my fork of AirGradient (https://github.com/Belphemur/AirGradient) , and it's working flawlessly for now.

Added v1.0.0 tag, first release.
Is it ok "v1.0.0" tag for "version=1.0.0" in library.properties?

Hello @jcomas

Is it ok "v1.0.0" tag for "version=1.0.0" in library.properties?

Yes. The Library Manager system doesn't care at all what the name of the tag is, so you are welcome to use any tag name format you like.
Although Git tags are the unit of release for Arduino Library Manager, all versioning is done according to the value of the version field in the library.properties metadata file at that tag. So the important thing is that the version field value be unique and compliant with the requirements specified here.


Some Feature Requests

The ability to manage SM users from the GUI. Add, delete, etc.
The ability to disable the sign up page

User registration can be disabled with a config option:
SECURITY_REGISTERABLE = True

I agree that SM needs a user management and role system with admins/users.

Thanks

SECURITY_REGISTERABLE is not conducive to a secure workflow to add users. The ability to confirm users after signup  before they get access would be much more helpful.

+1 for a role system with admins/users. Additionally defining roles in a way that would limiting which AWS accounts any given user could see or modify.

I'd like to see the SECURITY_REGISTERABLE = False added to the Quick Start guide under "Productionalizing Security Monkey"
It'd also be nice if setting SECURITY_REGISTERABLE would not only disable that but also hide the Register link on the main login page.   If someone browses to the register HTML form they should be told registration is disabled.
Perhaps a better option would be to extract out user management to the web server engine level instead.   It'd be nice if I could tie my LDAP authentication into it instead or any other mechanism I choose that my web server supports.

User Management has was merged in PR #317.
SSO was added in PR #354.
@rbeede - I'll accept a PR for the quickstart guide to set SECURITY_REGISTERABLE = False.
Closing this issue now.


Create thank-you.md

Trello card
https://trello.com/c/WbNIcjb1/2542-create-new-thank-you-page-for-intentions-survey

Hi @gemmadallmandfe here's the page (my first creation!) for people who have completed the survey. I've just asked Nicola Beech about what they send (i.e. do they call it a 'survey'? Wil update when I find out.
Any comment welcome though.

Have spoken to Nicola - wording is now aligned. I haven't shared it with the team... do I need to?

@peteryates How do we suppress this page from appearing in the nav?

Have spoken to Nicola - wording is now aligned. I haven't shared it with the team... do I need to?

Looks good to me in terms of content @Tomcowan1 - don't think it needs sharing with anyone else other than Nicola.


packageOverrides in ~/.nixpkgs/config.nix overwrites /etc/nixos/configuration.nix

Issue description
If I as a user, set a packageOverrides in my ~/.nixpkgs/config.nix, it will fully overwrite the packageOverrides in /etc/nixos/configuration.nix.
So say I overrode package go in /etc/nixos/configuration.nix. Then I overrode package neovim as a user in ~/.nixpkgs/config.nix. Now if I do nix-env -i go as the user, I will not get my overridden package from /etc/nixos/configuration.nix, I will get the regular package.
Is there any way to make it so the packageOverrides in ~/.nixpkgs/config.nix merges with the packageOverrides in /etc/nixos/configuration.nix?
Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...) 17.03pre99759.f66d782 (Gorilla)
Nix version: (run nix-env --version) nix-env (Nix) 1.11.6
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion) "17.03pre99759.f66d782"

nix-env by itself knows nothing about /etc/nixos/configuration.nix, which seems OK to me – that way it works more similarly on NixOS and elsewhere. The value for the currently running system isn't normally stored anywhere.
I just share the stuff "manually", i.e. I have /etc/nixos/nixpkgs-config.nix that is used by configuration.nix and I symlink that from ~/.nixpkgs/config.nix. (For systems used by multiple humans you could import and modify it instead.)  I'm not very fond of "merges magically", because I don't think there's any simple merging function that would satisfy most people.

Why symlink? If it is used by configuration.nix, shouldn't apply globally? Did you mean you are importing it in ~/.nixpkgs/config.nix?
As for merging, what is wrong with handling collisions like: any packages defined in ~/.nixpkgs/config.nix take precedence over the ones configuration.nix? What would not be satisfactory about that?

I have a symlink: ln -s /etc/nixos/nixpkgs-config.nix ~/.nixpkgs/config.nix, but importing would do basically the same thing.

But why even symlink or import? If it's used by configuration.nix, it's applied globally already.

nix-env by itself knows nothing about /etc/nixos/configuration.nix

argh, I read that but misinterpreted. I understand why it's not working now, thanks! But what about the merging the way I described in my previous comment? Why would someone object or even expect anything different?

I know of no universal way of "merging" two attrsets that would satisfy everyone.  I personally think it would perhaps be better to keep this explicit or configurable, as nixpkgs configuration commonly contains even functions...

Are there any updates to this issue, please?

Going to close as it has been a long time and I'm not sure if this is relevant anymore.


API updates to reflect latest HYAS Insight release.

Please check the following conditions for your PR.

[ ] apiDefinition.swagger.json is validated using paconn validate command.
[ ] apiProperties.json has a valid brand color. Invalid brand colors are #007ee5 and #ffffff.

@dougelliott Not sure how to add you as a reviewer.


Implement Process Class

Small functions implemented.
Things to note:

There is no way to serialize references because AS has a lack of reflection
Every reference must be marshalled manually by the end user, unless it is an Array<T> where T is a number type
TypedArrays and Arrays have backing buffers, and that's the data we are copying, but the marshalling service needs overhead to assemble it
StaticArrays have a single heap allocation, and it should be easy to implement
Simple values, because the stack cannot be reference must be boxed manually, but are also very easy to marshal

The overhead for starting a process with arrays is at least a heap allocation and a memcopy. (TypedArray or Array dataview buffer needs to be marshalled in combination with the callback.)
The overhead for running a child process with arrays is a few heap allocations (depending on the type) and a memcopy. This is all very unfortunate, and is done rather trivially in Rust, but that's because Rust handles stack references behind the scenes.
All in all, mostly done. Just need to add a few more implementations for Process.start().
Any ideas?

Seeing that checkmark turn green makes me smile.

I believe I should open up another pull request for something that might function like a Worker with a single channel. Still fleshing out ideas, but having access to create a Process will make higher level abstractions possible!  I'm satisfied with this request as-is, and I've flagged it for review.

Thanks for the contribution, this looks great! I like the shape this is taking and the CI is a nice touch.
I think that it makes a lot of sense to add a higher level abstraction like Worker, especially because the JS world is familiar with this concept of a Workers without memory sharing. Once we can use this implementation for "real-world" apps, we can start abstracting away common patterns like this. I'm not too worried to get it perfect right out of the gate.


BackAndroid was deprecated in 0.44

There is need to use BackHandler instead

@farwayer true! feel free to make a PR :)

done :)


Getting NoSuchMethodError: method not found: 'toString' on null in Production for Flutter web

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Not sure how to reproduce.
Expected results
Facing this issue only on production when deploying to firebase hosting. I guess similar to https://github.com/flutter/flutter/issues/101647
Actual results
Got Following Error:
NoSuchMethodError: method not found: 'toString' on null main.dart.js:31459

on main.dart.js:31459

Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration

Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.5, on macOS 13.4 22F66 darwin-arm64, locale en-NP)
    • Flutter version 3.10.5 on channel stable at /Users/fatdog/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 796c8ef792 (2 weeks ago), 2023-06-13 15:51:02 -0700
    • Engine revision 45f6e00911
    • Dart version 3.0.5
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/fatdog/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.79.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.66.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 13.4 22F66 darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 114.0.5735.106

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Hi @coder-Aayush, does the application work fine locally in profile and release modes? If it does not work in profile mode, can you share the message you get?

hey, yes app works perfectly file on profile and release modes.

@danagbemava-nc i deploy same codebase in cPanel it seems working fine. May be problem of Firebase Hosting, not sure.

Can you check the network panel of your browser to see if any files fail to load when you open the application on firebase hosting?

Other pages are working completely fine on firebase hosting, but some of the pages with same code is throwing the error message.

What pages are throwing the error? And what do you do on that page? Can you try to provide a minimal sample so that we can properly investigate this issue?

I found! In my case problem was shimmer package.

@SerhiiPtitsyn what was the issue?

I don't know. The error disappear when i don't use Shimmer widget. May be it's a flutter bug.

@danagbemava-nc you can leave this ticket open for further investigations.

Hi @coder-Aayush, you mentioned some pages work fine while some others do not. What packages are you using on those pages? And if you remove them(if you can), do you still get the issue with firebase hosting?

Can't figure it out, sometimes it renders and sometimes not

Hi @coder-Aayush, if you're not using any crash monitoring tool like sentry or firebase_crashlytics, can you try setting one of them up to see if they catch some more details about the crash message? You may also find what exactly is causing the error


createPullReplication how to set the since query param

I start listening to PullReplication like that:
val manager = new Manager(new JavaContext(), Manager.DEFAULT_OPTIONS)
  val sgUrl = new URL("http://test-machine:4984/sync_gateway/")
  val auth =  AuthenticatorFactory.createBasicAuthenticator("ombo", "master1")
  val db = manager.getDatabase("sync_gateway")

  db.addChangeListener(new Database.ChangeListener(){
    override def changed(event: ChangeEvent): Unit = {
      println(event.getChanges)
    }
  })

  val push = db.createPullReplication(sgUrl)
  push.setContinuous(true)
  push.setAuthenticator(auth)
  push.start()

it always pull the last seqId and start from there. how I can set it to number I need?

Hi @igreenfield,
Your client side code uses default setting. So, I recommend you to check server side setting, Sync Gateway or CouchDB?
Thanks,
Hideki

Hi @hideki
I will clarify my question:
The driver save the last revision it fetch, it checkpoint this value to doc in the sync-gateway db.
when you restart the program it takes this value and continue from there.
the question is:
When you add changeListenr if it failed to process the change the message still mark as handled so how I can change this behavior?
Thanks
Izek

Hi @igreenfield,
As you wrote in your comment, CBL ignore the Exception from ChangeListener.changed(...).
https://github.com/couchbase/couchbase-lite-java-core/blob/master/src/main/java/com/couchbase/lite/Database.java#L2389-L2400
Current version of CBL does not provide to override this part of code. So you could modify this part of implementation, and build CBL by yourself.

Hi @igreenfield,
CBL automatically stores check point value into the database at end of the replication. When replication is launched, new replication skips already pull replicated changes. So, since parameter is not necessary to be exposed as public API.
Thanks!


"Trust a workspace" not granular enough

Issue Type: Feature Request
"Trust a workspace" not granular enough.
I trust the files in the folder that I wrote, but not necessarily every file in my node_modules directory (which may contain many thousands of packages). I don't know that npm has implemented "VS Code extension hacking" as one of their package audit checks.
This will be true for all my projects. However, I don't seem to be able to exclude subdirs or subdirs by pattern. The pattern repeats across other package management situations (e.g. pyenv).
VS Code version: Code 1.57.0 (b4c1bd0a9b03c749ea011b06c6d2676c8091a70c, 2021-06-09T17:22:31.215Z)
OS version: Darwin x64 20.5.0
Restricted Mode: No

I'm looking for a middle ground between "I don't care about anything" and "I want a readonly viewer"
I don't trust most extensions not to do reckless things with content from the repositories i'm looking at. And I understand that VSC has decided that it can't warrant that most extensions won't be reckless.
That said, I'd like to be able to use the Terminal (mostly to run a handful of my own scripts) with lots of repositories that I don't directly trust. I do trust my own tooling (and if my tooling is broken, that's my problem, a problem I need to fix, independently of VSCode encouraging me to run that tooling in a window not labeled VSC).
In terms of granular features that I want to be able to use, I have just two:

Terminal
Being able to use the git staging gui tool (which I also can't use without enabling trust) -- It's shinier than git add -i. I'm fine w/ not being able to use the git commit features of the VSC gui (there are enough quirks about it that it's not really great for my use cases).

Obviously, I could imagine eventually identifying a handful of other items I'd want in intermediate trust.
Note: I do have a larger set of extensions that I actually use for repositories that I do trust, so it isn't simply a matter of "just uninstall everything you think is reckless".


Feature: Time type

If we have Date field in partition key, It would be perfect to have Time (without Date) type.
Now it is possible to emulate time with UInt32, but it is quite unreadable and not easy to use.

Time type will have almost no advantages over DateTime.
In memory, we need more than two bytes for representation of Time up to seconds resolution, and due to alignment and for simpler arithmetic, it will be four bytes, just as for DateTime.
On disk, full DateTime will compress to similar size as Time, because many consecutive dates in DateTime column and higher bytes in binary representation of DateTime will be the same.

I suppose that Time type with ms (or nano?) instead of UInt32 would help to track events more precise and would have some sugar to make work with time more accurate.
For better understanding, could you please show how to work with time in current realization?
SELECT event WHERE date = '2017.02.02' AND time < '1970-01-02 12:00:00' // without ms or nano
SELECT event WHERE date = '2017.02.02' AND time < '12:00:00.000' // with ms
SELECT event WHERE date = '2017.02.02' AND time < 43200000 // with UInt32

Agree. Special time type with custom resolution will be way more convenient.

Now we have DateTime64 data type that support up to nanosecond precision.
We still don't have dedicated Time (without Date) type but it is "won't fix".


0.2.5 breaks scalars with block notation

Hi,
Version 0.2.5 seems to break scalars that use block notation but with custom tags.  For example:
---
hmm: !example.com,2002/type |
  hello

The above document would parse on 0.2.4, but 0.2.5 raises an exception:
did not find expected whitespace or line break while scanning a tag at line 2 column 6

Strangely, the following two documents both work on 0.2.4 as well as 0.2.5:
---
hmm: |
  hello

---
hmm: !binary |
  hello

I bisected and found that this was introduced in #179.  Before #179, all three of the above documents would successfully parse.
I can't tell if this is intentional, so I'm not 100% sure how to fix tests.  The tests in Psych are failing with 0.2.5 but not 0.2.4.  Some of the failures are obvious problems in Psych's test suite, but this one isn't so obvious.
Thanks!

The comma is invalid in tags, so that is an intended change.
https://yaml.org/spec/1.2/spec.html#c-ns-shorthand-tag
https://yaml.org/spec/1.2/spec.html#ns-tag-char
[40] | ns-tag-char | ::= | ns-uri-char - “!” - c-flow-indicator

And flow-indicator contains the comma:
https://yaml.org/spec/1.2/spec.html#c-flow-indicator
So, commas are only allowed in tag handles or inside of <>:
!<tag:clarkevans.com,2002:invoice>
%TAG !yaml! tag:yaml.org,2002:

Sorry if it's breaking a use case.
I know that many libraries allow it, mostly because they were ported from libyaml.
The reference parser and yaml-cpp also don't accept the comma.
The use case for the fix was this testcase:
https://github.com/yaml/yaml-test-suite/blob/master/test/WZ62.tml
{
  foo : !!str,
  !!str : bar,
}

libyaml thought that the comma is part of the tag, and then the input is invalid.
I didn't actually expect people using commas in local tags...

The comma is invalid in tags, so that is an intended change.

Cool

I didn't actually expect people using commas in local tags...

TBH I don't know if anyone actually is.  These tests were ported from Syck as part of its implementation suite, so I don't think they say anything about real world usage.  I'll remove the commas from the suite in Psych.
Thanks for the quick response!

I did some more research. In YAML 1.1, the comma is indeed allowed:

[44] | ns-tag-char | ::= | ns-uri-char - “!”

I guess there was a areason for changing it in 1.2.
The correct thing would be to parse it depending on the YAML version, but currently there is
no handling of the version (and also no default version setting) in libyaml.
I guess @ingydotnet would have to decide what to do in this case.
Many libyaml based libraries are/were allowing this, so people out there might be relying on it.

Thanks @perlpunk for this investigation, and to all of the libyaml contributors.
I confirm that Sup has been relying on this: we have used YAML tag !supmua.org,2006-10-01/Redwood/Maildir for years. (Issue https://github.com/sup-heliotrope/sup/issues/577 - credit to @krerkkiat for identifying this.)
We're intending to modify Sup to be YAML 1.2 compliant, but adding this comment in case you wanted a real-world example of a project that depended on the YAML 1.1 behaviour.


gnome3.gtk+3: add broadwayd to path

Motivation for this change
I want to use broadwayd on nixos.
Things done
Added broadwayd to the output of the gtk+3 derivation

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 116053 at 2805076e run on aarch64-linux 1

432 packages marked as broken and skipped:

agdaPackages.iowa-stdlib
aldor
aqemu
asymptote
betaflight-configurator
cassandra_2_1
cassandra_2_2
colmapWithCuda
devede
electron_3
...

2917 packages skipped due to time constraints:

AusweisApp2
MIDIVisualizer
R
abcl
abiword
adapta-gtk-theme
adementary-theme
adoptopenjdk-hotspot-bin-13
adoptopenjdk-hotspot-bin-15
adoptopenjdk-icedtea-web
...

30 packages built successfully:

adoptopenjdk-bin (adoptopenjdk-hotspot-bin-11 ,openjdk11-bootstrap)
openjdk15-bootstrap (adoptopenjdk-hotspot-bin-14)
openjdk8-bootstrap (adoptopenjdk-hotspot-bin-8)
appgate-sdp
geoclue2
glib-networking (gnome3.glib-networking ,gnome3.glib_networking)
gnome3.gnome-desktop (gnome3.gnome_desktop)
gnome3.gnome-session-ctl
gtk3 (gnome3.gtk ,gtk3-x11)
gssdp
gtkspell3
gupnp
gupnp-igd
libcanberra-gtk3
libdbusmenu-gtk3
libnice
libproxy
libwnck3
networkmanager
nice-dcv-client
openconnect (openconnect_gnutls)
pantheon.elementary-icon-theme
pantheon.granite
prs
rofi
stoken
vscode
vscode-with-extensions
xfce.libxfce4util
xfce.xfce4-dev-tools (xfce.xfce4_dev_tools)

4 suggestions:

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/development/libraries/gtk/3.x.nix:69:5:
   |
69 |     ./patches/3.0-immodules.cache.patch
   |     ^

warning: missing-patch-comment
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/development/libraries/gtk/3.x.nix:71:5:
   |
71 |     (fetchpatch {
   |     ^

warning: unused-argument
Unused argument: gmp.
Near pkgs/development/libraries/gtk/3.x.nix:29:3:
   |
29 | , gmp
   |   ^

warning: unused-argument
Unused argument: x11Support.
Near pkgs/development/libraries/gtk/3.x.nix:35:3:
   |
35 | , x11Support ? stdenv.isLinux
   |   ^

/rebase-staging

/rebase staging

It would be nice if you would wait for ofborg to finish the eval after a rebase. It can always be that something unrelated makes the eval check fail.

If it's unrelated to the changes, there's no point in holding back the merge. Also eval succeeded with the changes applied to master.

Also eval succeeded with the changes applied to master.

Which does not always mean it needs to succeed on staging which could have more changes which make this PR fail.


Allow for using external logback.xml configuration

I'd like to add some custom appenders (like Whisper) that require specialized configuration. Does dropwizard allow you to move all logging configuration outside of yaml to logback's own xml configuration file?
If you don't please consider doing so. Otherwise you are trying to unnecessarily wrap another library into your yaml and will always play catchup to logback.

FWIW, I have not found dropwizard 0.8 to discover any AppenderFactory implementations from the classpath, but adding a custom META-INF/services/io.dropwizard.logging.AppenderFactory has an effect.  Bit clunky and a lot of extra work to create the custom factory compared to simply using logback.xml however...

This is the only reason we maintain our own fork of dropwizard.  We like everything else, but having native support for a logback.xml file is still far superior for our needs then what we can do in YAML.
Our fork basically removes the dropwizard logging functionality and now we just rely on plain old logback and life is good.

@skradel / @brentryan - care to share that particular code?

@mjallday Check out this repository - https://github.com/Wikia/dropwizard-logstash-encoder . Contains an implementation. Be sure to check out the open pull requests as well.

+1

@mjallday Here's our fork of dropwizard that allows us to just use the native logback system with dropwizard.  https://github.com/cvent/dropwizard/tree/v0.7-cvent
With this we simply pass in -Dlogback.configurationFile=logback.xml on startup and we have full logging capabilities.
Makes it easy for us to use open source json logging encoders for our integration with splunk
    <encoder class="ch.qos.logback.core.encoder.LayoutWrappingEncoder">
        <layout class="ch.qos.logback.contrib.json.classic.JsonLayout">
            <jsonFormatter class="ch.qos.logback.contrib.jackson.JacksonJsonFormatter">
                <!-- prettyPrint is probably ok in dev, but usually not ideal in production: -->
                <prettyPrint>false</prettyPrint>
            </jsonFormatter>
            <timestampFormat>yyyy-MM-dd'T'HH:mm:ss.SSS'Z'</timestampFormat>
            <timestampFormatTimezoneId>UTC</timestampFormatTimezoneId>
            <appendLineSeparator>true</appendLineSeparator>
        </layout>
    </encoder>


Bitcoin formatter issues PHP notice with small amounts

When trying to print a fraction of a bitcoin with the bitcoin formatter, if the number of digits requested is too few (I think where it would effectively round to zero, since the subunit for bitcoin is so large), a string offset warning is issued.
<?php
require 'vendor/autoload.php';
use Money\Currency;
use Money\Currencies\BitcoinCurrencies;
use Money\Formatter\BitcoinMoneyFormatter;
use Money\Money;

$btc = new Money(100, new Currency('XBT'));

$btcFormatter = new BitcoinMoneyFormatter(2, new BitcoinCurrencies());
echo $btcFormatter->format($btc);

Notice: Uninitialized string offset: -3 in DIR/vendor/moneyphp/money/src/Number.php on line 199
My composer.lock file says this is with v3.0.0.

Thanks, fixed in current master


Team 18 Review (VnV Report): Formatting: list of figures not required

@CSchank
Summary
There is no figures throughout the document. The "List of Figures" section is empty and redundant.
Deadline
April 4th
Estimated Work Hours
2 min

Resolved in commit id: 82a9b4ec4dfe3751fcaa257d9e3d4cf4948dde69


[SPARK-20918] [SQL] Use FunctionIdentifier as function identifiers in FunctionRegistry

What changes were proposed in this pull request?
Currently, the unquoted string of a function identifier is being used as the function identifier in the function registry. This could cause the incorrect the behavior when users use . in the function names. This PR is to take the FunctionIdentifier as the identifier in the function registry.

Add a new function listFunction to FunctionRegistry

def listFunction(db: String): Seq[FunctionIdentifier]

How was this patch tested?
TODO: add extra test cases to verify the inclusive bug fixes.

Test build #77519 has started for PR 18142 at commit 201787f.

Test build #77520 has started for PR 18142 at commit 3f253f3.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77520/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77519/
Test FAILed.

Test build #77538 has started for PR 18142 at commit e8a534a.

Test build #77538 has finished for PR 18142 at commit e8a534a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77538/
Test PASSed.

Merged build finished. Test PASSed.

Test build #77602 has started for PR 18142 at commit 794de15.

Test build #77602 has finished for PR 18142 at commit 794de15.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77602/
Test PASSed.

retest this please

Test build #77815 has started for PR 18142 at commit 794de15.

Test build #77815 has finished for PR 18142 at commit 794de15.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77815/
Test PASSed.

Test build #77825 has started for PR 18142 at commit 5635c27.

Test build #77825 has finished for PR 18142 at commit 5635c27.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77825/
Test PASSed.

LGTM, merging to master!

Guys - please in the future separate bug fixes with refactoring. Don't mix a bunch of cosmetic changes with actual bug fixes together.

I am leaving a note (at least to myself) since it looked anyhow caused behaviour change.
With this statements in Hive side:
CREATE TABLE emp AS SELECT 'user' AS name, 'address' as address;
CREATE DATABASE d100;
CREATE FUNCTION d100.udf100 AS 'org.apache.hadoop.hive.ql.udf.generic.GenericUDFUpper';

Hive
hive> SELECT d100.udf100(`emp`.`name`) FROM `emp`;
USER
hive> SELECT `d100.udf100`(`emp`.`name`) FROM `emp`;
USER

Spark
Before:
scala> spark.sql("SELECT d100.udf100(`emp`.`name`) FROM `emp`").show
+-----------------+
|d100.udf100(name)|
+-----------------+
|             USER|
+-----------------+

scala> spark.sql("SELECT `d100.udf100`(`emp`.`name`) FROM `emp`").show
+-----------------+
|d100.udf100(name)|
+-----------------+
|             USER|
+-----------------+

After:
scala> spark.sql("SELECT d100.udf100(`emp`.`name`) FROM `emp`").show
+-----------------+
|d100.udf100(name)|
+-----------------+
|             USER|
+-----------------+

scala> spark.sql("SELECT `d100.udf100`(`emp`.`name`) FROM `emp`").show
org.apache.spark.sql.AnalysisException: Undefined function: 'd100.udf100'. This function is neither a registered temporary function nor a permanent function registered in the database 'default'.; line 1 pos 7

MySQL
This change causes a inconsistency with Hive although looks consistent compating to MySQL.
mysql> SELECT `d100.udf100`(`emp`.`name`) FROM `emp`;
ERROR 1305 (42000): FUNCTION hkwon.d100.udf100 does not exist
mysql> SELECT d100.udf100(`emp`.`name`) FROM `emp`;
+---------------------------+
| d100.udf100(`emp`.`name`) |
+---------------------------+
| Hello, user!              |
+---------------------------+
1 row in set (0.01 sec)

@HyukjinKwon Thanks for the note! I think this behavior is better, I'm adding a release_note tag to the JIRA ticket, so that we don't forget to mention it in release notes.

I agree with this change too for clarification.

BTW, this is changed at Spark 2.3.0. How did we handle this before?

hmm, then it's too late. Maybe we can add it in Spark 2.3.2, cc @jerryshao

I see. Thanks for the note.

@cloud-fan, should we update migration guide as well?

After a second thought, isn't it a bug?
hive> SELECT `d100.udf100`(`emp`.`name`) FROM `emp`;
USER

This clearly violates the SQL semantic: the string inside backticks should be treated as a string literal. I think we should update the JIRA ticket to explain this bug, but we don't need to put it in a release notes or migration guide.
I'll update the ticket.

This clearly violates the SQL semantic: the string inside backticks should be treated as a string literal.

BTW, I believe there's no particular standard for backticks themselves since different DBMS uses different backtick implementations.

One explicit problem here is, we claim Hive compatibility in Spark. The difference should be explained when we are clear on this.

BTW, I believe there's no particular standard for backticks themselves since different DBMS uses different backtick implementations.

You are right, but SQL standard does define how to quote identifiers, by using "xyz". Databases usually support one more syntax to quote identifiers, e.g. backticks, square brackets, etc.
So for this case, I think it's obvious that users want to quote the function name, and we have a bug.

One explicit problem here is, we claim Hive compatibility in Spark.

Do we? Sometimes we follow hive behaviors, but we never guarantee that IIRC.

We do not need to follow Hive if Hive does not follow SQL compliance. Our main goal is to follow the mainstream DBMS vendors.
BTW, we can enhance our parser to recognize the other symbols (e.g., double quotes) as the quotes instead of forcing users to choose backtick. cc @maropu

I mean https://spark.apache.org/docs/latest/sql-programming-guide.html#supported-hive-features and https://spark.apache.org/docs/latest/sql-programming-guide.html#unsupported-hive-functionality
The issue is also related with VIEW support as well. Should be good to note.
I don't mean we should necessarily follow Hive's behaviour for clarification .. Also, I agree with this change ..

Spark SQL is designed to be compatible with the Hive Metastore, SerDes and UDFs.

This is different from Spark can run any Hive SQL. Spark can load and use Hive UDFs, with the right SQL syntax.
Anyway I'm happy to see a PR to note it. It's good to be verbose in the doc. But I won't treat it a must-have.

Yea, I didn't mean it super seriously @cloud-fan - I just left a comment in case for a better documentation since I see many users go from Hive to Spark.

@gatorsmile btw, we've already file a lira to track these kinds of all the SQL-compiliant issues? not yet?


[FEATURE] [v7] Return executeAsync from useAction and useOptimisticAction hooks

See #147

Implemented in v7.0.0-next.46


Tableview issue not generating pdf

`/var/mobile/Containers/Data/Application/D845549F-E560-4940-BA52-3F6229ED32F5/Documents/customDirectory' for writing: Is a directory.
deflateEnd: error -3: (null).
2017-09-18 17:40:44.584464 pdfdemo[554:107175] CGPDFContextCreate: failed to create PDF context delegate.
Sep 18 17:40:44  pdfdemo[554] : CGPDFContextCreate: failed to create PDF context delegate.
Error

Did you solved this issue ?

The problem with this is that the file path is wrong and you can put it somewhere else.
such as:
 do {
     let dst = NSTemporaryDirectory() + "/sample_tblview.pdf"
     try PDFGenerator.generate(self.tableView, to: dst)
     openPDFViewer(dst)
} catch let error {
     print(error)
}


Crash when filename contains non-existent directory

If a file logger's filename contains a directory, creating the logger crashes if that directory does not exist.
For example:
tinylog.writer2 = rollingfile
tinylog.writer2.filename = logs/log.txt
tinylog.writer2.backups = 10
If I run this in a directory which does not yet contain a logs/ folder, the program crashes.
Given that the logfile itself is created at start-up, it seems reasonable that any folders in the filename should also be created.

Thanks for reporting! I can reproduce this bug for all file based writers. Will be fixed in the next release!

Preview of fixed version: http://www.tinylog.org/downloads/tinylog-1.0.1.zip

Fixed in tinylog 1.0.1


Physics collision system

Currently adding a collision system to RigidEntities. Will commit later.

Done. It's a little bit buggy, but works well enough for now.


"Add support for secure token storage"

Token storage change in latest release of gh
This is a message from the GitHub CLI team, maintainers of gh, writing to inform you that the most recent release of gh contains changes which may affect your extension. The latest release introduces the feature of storing authentication tokens in the system keyring (encrypted storage) instead of in a plain text file. The keyrings that are supported are:

Keychain on macOS

GNOME Keyring on Linux (Secret Service dbus interface)

Wincred on Windows

This has huge security benefits for the users of our tool and was one of our oldest outstanding issues. Unfortunately this change has the potential to break extensions that rely on utilizing the users authentication token to work.
In order to have continued compatibility with gh there are some actions you, as an extension author, need to take. These actions will depend on the implementation of your extension.
Extensions built in Go using go-gh:

Upgrade your go-gh version to v1.2.1, the latest version.

This can be done using go get<EMAIL_ADDRESS>

Verify that in your extension retrieval of the user authentication token is done using the  auth.TokenForHost function.

If you were previously accessing the authentication token using any other method it will no longer work.
Automatic resolution of the authentication token when using the API clients will continue to work without changes.

All other extensions:

Verify that in your extension retrieval of the user authentication token is done by shelling out to the gh auth token command.

If you were previously accessing the authentication token using the gh config get command, reading the configuration file directly, or any other methods it will no longer work.

As of right now storing the authentication token in the system keyring is an opt-in feature, but in the near future it will be required and at that point if the changes above are not made then your extension will be broken for all users. If you have any questions/concerns about this change please feel free to open a discussion in the gh repo.
Thanks, The GitHub CLI Team

e.g.
# this is nushell
~> star-history dtolnay
Error: GitHub auth token is not set up.

(Expected config file: C:\Users\CAD\AppData\Roaming\GitHub CLI\hosts.yml)

Run `gh auth login` to store a GitHub login token. The `gh` CLI
can be installed from <https://cli.github.com>.

If you prefer not to use the `gh` CLI, you can instead provide
a token to star-history through the GITHUB_TOKEN environment
variable. Head to <https://github.com/settings/tokens> and click
"Generate new token (classic)". The default public access
permission is sufficient -- you can leave all the checkboxes
empty. Save the generated token somewhere like ~/.githubtoken
and use `export GITHUB_TOKEN=$(cat ~/.githubtoken)`.

~> gh auth status
github.com
  ✓ Logged in to github.com as CAD97 (keyring)
  ✓ Git operations for github.com configured to use ssh protocol.
  ✓ Token: gho_************************************
  ✓ Token scopes: admin:public_key, gist, read:org, repo

~> with-env [GITHUB_TOKEN (gh auth token)] { star-history dtolnay }
# works

(just for clarity, I'm not part of and this isn't from the gh team; I've just copied the message they did provide to gh extensions)


Rename file based on language

Hello,
With command:
jimaku videofile.mkv --language=fi
The file name should be like this:
videofile.mkv
videofile.fi.srt <--
Can this be added?

Hello @ronilaukkarinen,
Sure, this is released in v0.1.1!

That was fast, thanks! 👍


Does directxtex support D2D1?how to save bitmap from ID2D1HwndRenderTarget?

i wanna save bitmap from ID2D1HwndRenderTarget in kinect v2 bodybasics demo
thanks for help

DirectXTex and the rest of my libraries are focused on Direct3D scenarios. Direct2D supports WIC directly, so that's probably the most direct solution. See StackOverflow.


Log in my account

User Story Description
As a user
I want to log into my account
So I can so I can start interacting with the app
Steps to Follow (optional)

[ ] #106
[ ] -
[ ] -

Additional Considerations
Any supplemental information including unresolved questions, links to external resources, screenshots, etc.

Fixed...
https://github.com/chingu-voyages/v39-bears-team-11be-a/blob/main/controllers/userController.go
The User provides the email and password he used to create the account.
This information is verified and he gets authenticated.


merge dev into master

Codecov Report
Patch and project coverage have no change.

Comparison is base (c105252) 53.23% compared to head (48d91bc) 53.23%.
 Consider uploading reports for the commit e2c6b27 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2464   +/-   ##
=======================================
  Coverage   53.23%   53.23%
=======================================
  Files         117      117
  Lines        9633     9633
=======================================
  Hits         5128     5128
  Misses       4505     4505

Flag
Coverage Δ

unittests
53.23% <ø> (ø)

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

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


Pacemaker Cluster - created tags don't showed up if other filter selected

Testing 3rd round
Server version: v0.6.0+git.dev293.1639737346.f1029ae
Go to QA server: http://<IP_ADDRESS>/clusters
Add some tags for hana_cluster at first. They show up as expected.
Now select any other filter like cluster type or name, the created tags env1, env2, env3 don't show up.

v0.7.0: issue persists

0.7.0+git.dev39.1640009634.6e57174: fixed: issue can be closed.


Bug: Multi resource yaml does not work since v0.26.1

Hi!
It seems that version 0.26.1 had introduced a bug processing multi resource yaml files.
How to reproduce
Create a file such as example.yaml:
apiVersion: monitoring.coreos.com/v1
kind: PodMonitor
---
apiVersion: v1
kind: Secret
data:
  username: ref+sops://secrets.enc.yaml/#username

Running vals eval -f manifests/example.yaml outputs:
sops: successfully retrieved key=secrets.enc.yaml
null

On version 0.26.0 it works correctly:
sops: successfully retrieved key=secrets.enc.yaml
apiVersion: monitoring.coreos.com/v1
kind: PodMonitor
---
apiVersion: v1
data:
  username: ****
kind: Secret

@jkroepke could you see this issue? maybe introduced in https://github.com/helmfile/vals/pull/150.

@jkroepke hello.


document.location.reload () to render new translation?

Hi,
More of a question really than an issue but in /src/app/structure/header-controls/header.language.component.ts  there is a call to document.location.reload () do we really need to reload the entire application to get the view to re-render the translation change?
Currently on the application I am building when a user changes language the current view is not re-rendered. However if they then navigate off to another page ( without a refresh ) the correct translation is rendered on the new page.
I've used i18next in the past with React and when hooked up to the store the views just automatically re-render. Apologies I'm pretty new to angular.

@WebStew  Short answer to you question: you don't have to call location.reload(), but i would not recommend it =)

However if they then navigate off to another page ( without a refresh ) the correct translation is rendered on the new page.

You found a bug: top navigation have been always refreshed page (without use angular router navigation mechanism). Fixed now =)
Now navigation would not make you see correct language in a top bar, only in components inside .
i18next is a pure pipe that means that it will not rerender until input params would change (key or options), the correct language will render only for new pipe instances.
As i know in react you can call redraw of whole application, in angular you don't have this option. The only way to do it are through hacks: check it out
The main reason why i prefer to reload page on locale change is that some UI components in your application could not change their format at runtime (date/datetime pickers or moment locale). You just set its locale on an application startup and do not trying to make 3rd party component dynamicaly configurable.
Or you could have use some other pure pipes that doesn't know anything about i18locale change.
Another problem is that you can't change LOCALE_ID dynamicaly (once it was resovled), this token could be used for localize angular controls and stuff...
And finally page reload can be positive for app performance. It would clean all memory after refresh =)
You must be wondering why is that i18next is not impure pipe than? Here is the answer:
https://github.com/Romanchuk/angular-i18next/pull/11#issuecomment-364725022

Many thanks for the response @Romanchuk. I appreciate the links to the history on your decisions as well. I'll do some more reading around the pipes and revert.

@WebStew
Depends of your project use cases.
Mostly user wants to change language before moving forward.
Use i18next language auto detect to set user preferred language.
Anyway i recommend to use router for main sections of the site, it would help to return back after user manual refresh (F5) and also could be helpfull for analytics and metrics (GA, yandex, application insights). If you have lists with filters, you might want to users share links that contain filter params.
Options for lang change:

Show confirmation dialog before change language
Show lang choose right after app init
If user lang stores in db you can move language change functionality to some sort of setting. Without afraid of losing state.
Just reload if your system is read only (or has just a few small forms)
If your system is not tolerant to refresh at all. Than use one of the hacks to redraw whole application

Okay thanks @Romanchuk. I definitely agree. User will probably only change language at the beginning and possible only ever once as well.


[18273] Refactor enum EntityKind::DOMAIN with EntityKind::DOMAIN_ENTITY

In order to avoid a compilation issue with clang #184 a refactor of EntityKind::DOMAIN with EntityKind::DOMAIN_ENTITY.
This implies an API Break.

Also checked on macOS Ventura 13.4, Xcode 14.3.1.
Issue is legacy BSD extensions to the C standard defined in MacOSX.sdk/usr/include/math.h.
If Fast-DDS-monitor is similarly patched the monitor builds and runs on macOS (good to have!).


Mention the Alt- shortcut in the documentation

Otherwise it's not obvious how to access these shortcuts.

Found it in the docs!

@rob-smallshire where in the doc please? I'm looking for it too :)

Ah thanks @ellie ! Saw it -- I was looking in the documentation website, not the README. At some point I hit another key combination that allowed to switch between Globa/Host/something else, but I don't remember which, and again can't find it. Not likely to use this specific one much yet at this stage, though.

Ahh thanks for highlighting that! We totally need to improve the docs there, the website is super new!
Ctrl-r will cycle the filter modes 😁


Only modified files should be written on 'Save All'

All open files are written on 'Save All', causing unnecessary triggers of change watchers, for example.

Looks like a duplicate of https://github.com/atom/atom/issues/2871.


Kanban Extra Settings Missing Hover Effect

Describe the bug
No hover effect on Kanban Card Extra settings dropdown
To Reproduce
Steps to reproduce the behavior:

Go to Kanban
Click on '...'
You will see that there is missing hover effect

Expected behavior
Add hover effect
Screenshots

added in pr #197


[SPARK-35789][SQL] Refine lateral join syntax to only allow subqueries

What changes were proposed in this pull request?
This PR is a follow-up for SPARK-34382. It refines the lateral join syntax to only allow the LATERAL keyword to be in front of subqueries, instead of all relationPriamry. For example, SELECT * FROM t1, LATERAL t2 should not be allowed.
Why are the changes needed?
To be consistent with Postgres.
Does this PR introduce any user-facing change?
Yes. After this PR, the LATERAL keyword can only be in front of subqueries.
sql("SELECT * FROM t1, LATERAL t2")

org.apache.spark.sql.catalyst.parser.ParseException:
LATERAL can only be used with subquery(line 1, pos 26)

== SQL ==
select * from t1, lateral t2
--------------------------^^^

How was this patch tested?
New unit tests.

Test build #139886 has started for PR 32937 at commit 243d80e.

Test build #139886 has finished for PR 32937 at commit 243d80e.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139886/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44416/

thanks, merging to master!


[FLINK-35472] Improve tests for Elasticsearch 8 connector

https://issues.apache.org/jira/browse/FLINK-35472

Per discussion in this PR (thanks to @reta), it makes the tests more reusable if we use parameterized tests. It requires some changes of the existing tests, which includes:

Make base test class parameterized with secure parameter. As JUnit 5 has limited support for parameterized tests with inheritance, we can use the ParameterizedTestExtension introduced in Flink, see this doc
Manage the test container lifecycle instead of using the managed annotation @Testcontainers and @Container so that the test containers can be used as a singleton for all tests in the suite
Create and use common methods in the base class that concrete test classes can be mostly parameter-agnostic
This JIRA intends to not change any logic or functionality. Instead it focuses on tests refactoring for more reusable tests and future proof.

@reta @snuyanzin Could you take a look? Thank you!

@snuyanzin could you help review on this? Andriy and I don't have write permissions. Thank you!

thanks for the contribution @liuml07
the ci is still failing, can you please have a look ?

Thank you @snuyanzin for taking another look.
I see a lot of errors like this, and not sure if this is causing the compile & test to timeout

Warning:  Could not transfer metadata org.apache.flink:flink-test-utils-junit:1.18-SNAPSHOT/maven-metadata.xml from/to apache.snapshots (https://repository.apache.org/snapshots): transfer failed for
... (repeat multiple lines) ...
Error: The action 'Compile and test' has timed out after 50 minutes.

I have rebased from the main branch and will check again.

Error:  Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.0:testCompile (default-testCompile) on project flink-connector-elasticsearch-base: Compilation failure
Error:  /home/runner/work/flink-connector-elasticsearch/flink-connector-elasticsearch/flink-connector-elasticsearch-base/src/test/java/org/apache/flink/connector/elasticsearch/sink/ElasticsearchWriterITCase.java:[485,20] org.apache.flink.connector.elasticsearch.sink.ElasticsearchWriterITCase.TestMailbox is not abstract and does not override abstract method shouldInterrupt() in org.apache.flink.api.common.operators.MailboxExecutor

We didn't change the code over there for 6/7, and they don't fail locally. Interesting why it's failing for the PR.

Rebasing from main branch.


ENH: Moving Ridge out of preview

Description
This PR moves Ridge out of preview namespace to main sklearnex. It drives sklearnex Ridge away from daal4py by switching underlying implementation to onedal4py, which enables GPU support in turn. Minor usage of daal4py remains in case of individual (non-scalar) penalties since oneDAL still does not support such scenario. Performance comparison has been recorded on CPU:

Performance comparison on GPU is irrelevant since there was not real GPU implementation before.
Separate ticket to update documentation has been created.

PR completeness and readability

[x] I have reviewed my changes thoroughly before submitting this pull request.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have updated the documentation to reflect the changes or created a separate PR with update and provided its number in the description, if necessary.
[x] Git commit message contains an appropriate signed-off-by string (see CONTRIBUTING.md for details).
[x] I have added a respective label(s) to PR if I have a permission for that.
[x] I have resolved any merge conflicts that might occur with the base branch.

Testing

[x] I have run it locally and tested the changes extensively.
[x] All CI jobs are green or I have provided justification why they aren't.
[x] I have extended testing suite if new functionality was introduced in this PR.

Performance

[x] I have measured performance for affected algorithms using scikit-learn_bench and provided at least summary table with measured data, if performance change is expected.
[x] I have provided justification why performance has changed or why changes are not expected.
[x] I have provided justification why quality metrics have changed or why changes are not expected.
[x] I have extended benchmarking suite and provided corresponding scikit-learn_bench PR if new measurable functionality was introduced in this PR.

/intelci: run

/intelci: run

Private CI errors seem unrelated

/intelci: run

This should also be updated - https://github.com/intel/scikit-learn-intelex/blob/main/doc/sources/preview.rst?plain=1#L49-L76, remove ridge from bottom part and replace it with EmpiricalCovariance or IncrementalPCA in snippet

With the CI fixes merged, please rebase @DDJHB

/intelci: run

/intelci: run

/intelci: run

@david-cortes-intel does this PR look properly aligned with #2083 (specifically see changes from commit: 4e18d8a#diff-ed94dee2dafa5407c12f73420aaee87ee7748a2fedeeea8d4bd7b1d50fbf4a53)

Yes, changes look aligned - I had made a mistake in the condition namings which this PR is correcting.

/intelci: run

/intelci: run