Alternative API for getting TraverseCtx in

@Dunqing was asking in #3216 for a more ergonomic way to pass TraverseCtx around in transformers.
#3219 is one option. Here's another, which is more like Babel's Path API:
Instead of receiving AST node and TraverseCtx, you'd only get Path, and have to get the AST node from it:
impl<'a> Traverse<'a> for MyTransform {
  fn enter_binary_expression(&mut self, path: &mut Path) {
    let bin_expr: &mut BinaryExpression<'a> = path.node();
    // Do stuff with `bin_expr`
  }
}

Is that any better?
Personally I don't much like it - .node() is boilerplate for every single enter_* or exit_* method.
BUT if we want to support mutating upwards, we'd need an API something like that anyway. That'd work something like this:
impl<'a> Traverse<'a> for MyTransform {
  fn enter_binary_expression(&mut self, path: &mut Path) {
    match path.parent_mut() {
      AncestorMut::ExpressionStatement(expr_stmt) => {
        // `expr_stmt` is a `&mut ExpressionStatement` - you can mutate it
      }
    }
  }
}

With upwards mutation, the function params can't include a reference to the current node, as it would be UB to also get a &mut ref to parent at same time. Both current AST node and parent AST node have to both come from path, so path can enforce that you can only get one or the other at any time.
NB: I am not 100% sure upward mutation can be made safe, but I think it can with an API like the above.

I like this API.

Instead of receiving AST node and TraverseCtx, you'd only get Path, and have to get the AST node from it:
impl<'a> Traverse<'a> for MyTransform {
  fn enter_binary_expression(&mut self, path: &mut Path) {
    let bin_expr: &mut BinaryExpression<'a> = path.node();
    // Do stuff with `bin_expr`
  }
}

But your example looks incorrect. What do I get when I call path.node()?

Path is a generic Path<'a, T>. I am hoping that Rust would allow eliding T, but maybe I'm wrong about that.
With the full function signature, it would be:
impl<'a> Traverse<'a> for MyTransform {
  fn enter_binary_expression(&mut self, path: &mut Path<'a, BinaryExpression<'a>>) {
    let bin_expr = path.node();
    // Do stuff with `bin_expr`
  }
}

bin_expr is a &mut BinaryExpression<'a>.

I can try and build it so we can evaluate it more clearly - should not take long.


[bug] Grid DND with different hight items causes recursive position change

We have a CSS Grid DND container containing items with different height. When dragging a top item to the second line, it causes infinite position change, i believe because the height of the second row is changing.
See the video:
https://github.com/SortableJS/Sortable/assets/9210534/110fce16-8cf0-4e64-b540-9e2f9319ad0f
I see that the official demo shows grid-like layout using inline-block and it works well.
With the display:flex + flex-wrap: wrap + flex-basis, that simulates grid layout, it works as well.
But it'd be nice to use CSS Grids
We use the React and have the following config for the sortablejs/react:
animation={150}
swapThreshold={0.25}
emptyInsertThreshold={0}

Expected behavior
No infinite position change when in our case.
Versions - Look in your package.json for this information:
"react-sortablejs": "6.1.4",
"sortablejs": "1.15.0",
Also the same with the latest 1.15.1, also 1.15.1 introduces additional issues

I can't reproduce this, wondering if you could create a demo JSBin so that I can try it. Or share the CSS you used in the video.

Could you verify if it's fixed in 1.15.2?

Hello! I am able to replicate the issue even in 1.15.2. Here's a repro: https://stackblitz.com/edit/react-vn8b5d?file=src%2FApp.js,src%2FGrid.js,src%2FGridItem.js,src%2Findex.js,src%2Fstyle.css,package.json
Happens only when you try to move a smaller item in the position of the longer item.
https://github.com/SortableJS/Sortable/assets/25902483/305a60bb-5221-4ae2-bbc3-f7ccb57318b4

If a plain JS repro helps, you check out this one: https://stackblitz.com/edit/js-rmkuwx?file=index.js,index.html,style.css
Same issue, try dragging 5th element onto the bottom half of 3rd element, you can see the flicker happen.
https://github.com/SortableJS/Sortable/assets/25902483/b4cf4e5b-95c4-4155-9099-8d68ea19abdb

Thank you, grid elements with dynamic size is not fully supported yet. In a future update I can add a grid direction option.


[Bug]: Exports exist on native MySQL database views, functions, and stored procedure structure trees

ODC version
odc422
OB version
not have
What happened?

What did you expect to happen?
The native MySQL data source has not been adapted for export yet
How can we reproduce it (as minimally and precisely as possible)?
not have
Anything else we need to know?
No response
Cloud
No response

worksforme


fix fix compaction_used_permits are negative numbers

Proposed changes
Issue Number: close #32437

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run build all

run build all

run buildall


Window is slow and flickers 7.1.4

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
When running the PowerShell 7.1.4 (perhaps also previous version), window flickers on enter, when hitting the Ctrl-L (clear screen) there is a visible lag. I come from the Linux world and Mac OS terminals like Kitty which are >=100 FPS opengl accelerated and this is very ugly.
Expected behavior
No flicker at all. High fps.

Actual behavior
Flicker on various events, simplest is the clear screen.

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.1.4
PSEdition                      Core
GitCommitId                    7.1.4
OS                             Microsoft Windows 10.0.19043
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Do you use a console or a terminal ? :)
https://www.hanselman.com/blog/WhatsTheDifferenceBetweenAConsoleATerminalAndAShell.aspx
I advice you to try the Windows Terminal
https://docs.microsoft.com/en-us/windows/terminal/
WebStore App
https://www.microsoft.com/en-us/p/windows-terminal-preview/9n0dx20hk701?WT.mc_id=-blog-scottha&rtc=1&activetab=pivot:overviewtab

Expected behavior

Standart host
Windows Terminal Preview 1.10.1933.0
Alacritty 0.9.0
pwsh 7.1.4


chore: bump to nightly-2022-11-29

injection and congr both became more powerful in Lean 4 core, so some proofs simplify.

bors merge

bors merge

bors merge


Allow feature filters to contextually apply

Hi I was in progress of switching over to this library from a proprietary solution when I noticed some functionality missing.
In our solution we allow (some) users to override the evaluation of a feature flag, e.g. for developers to test the feature out on production, by overriding the evaluation using a query string: ?myFeatureFlag=1 (also mentioned in #106). Although, I believe it would still be beneficial to apply a "global" set of filters, as explained in #89.
Unfortunately, I realized that there is one critical piece left: to be able for a filter to override another positive evaluation. I.e. if one filter evaluates to true - I also want the possibility to override that to force the evaluation to false: ?myFeatureFlag=0.
The current IFeatureFilter interface doesn't allow that granularity. In our proprietary solution we also add another dimension, whether or not the filter should be considered "active". In the example above the logic for whether or not the filter should be active is based off whether the feature name is present in the query string.
So the core logic is then changed from "if any feature filter evaluates to true, that feature will be considered enabled." to instead iterate over the filters and use the result of the first filter that is active, and use its result, disregarding the other filters.
I know that this feature request results in a big overhaul of the current architecture, but I just wanted to throw it out there and see if there are any workarounds for my case.

Hi, @patrikmolsson
Our next release will introduce feature variant, and I believe this can address the issue you mentioned.

Hi @patrikmolsson, I'm having some trouble digesting the issue here.
"if one filter evaluates to true - I also want the possibility to override that to force the evaluation to false: ?myFeatureFlag=0."
"So the core logic is then changed from "if any feature filter evaluates to true, that feature will be considered enabled." to instead iterate over the filters and use the result of the first filter that is active, and use its result, disregarding the other filters."
Could you give a specific scenario to explain it?
Do you mean that if a feature flag has 4 filters, let's say filter1, filter2, filter3 and filter4. The filter1 is the pivot filter which you want it be true all the time. Meanwhile, you want at least one of filter2,3,4 is true and then the feature flag should be activated.
Is my understanding correct?

Hi @zhiyuanliang-ms thanks for looking into this. My case may not be relevant anymore, since it was more than 2 years ago I wrote it, but I'll give it a try explaining it.

"So the core logic is then changed from "if any feature filter evaluates to true, that feature will be considered enabled." to instead iterate over the filters and use the result of the first filter that is active, and use its result, disregarding the other filters."
But I did not understand this.

Sure - that sentence was more discussing the current solution, and how you would need to change it to fit my need. You can safely disregard it if you have any other solution. The requirements would rather be:
If I have a custom filter built - e.g. QueryParamFilter - and I want this to be able to disable features as well - I would need to be able to configure whether this filter should even be evaluated or not. E.g. if the query param name is not present in the query string, I shouldn't even invoke EvaluateAsync.
So the examples would be: I have two filters - PercentageFilter (50%) and my own QueryParamFilter. I have a user, which is a developer for the product, so they need to be able to test around and verify behaviour in production.

PercentageFilter evaluates to false for my user, which in turn means the feature is disabled. The user adds ?feature=1, QueryParamFilter evaluates to true, and now the feature shows up. It works as expected!
PercentageFilter evaluates to true for my user, which in turn means the feature is enabled. The user adds ?feature=0, QueryParamFilter evaluates to false, but since PercentageFilter evaluates to true - this will take precedence over my other feature. It does not work as expected.
PercentageFilter evaluates to true for my user, which in turn means the feature is enabled. The user does not add anything in the query param. What should QueryParamFilter evaluate to? In an ideal scenario, it should not evaluate at all if the query param is not present.

Hopefully these examples provide some insight.

@patrikmolsson Thank you for the detailed explanation. Now I understand the scenario. Currently, we support the parameter "RequirementType", if the "RequirementType" is "All", which means all filters listed in "EnabledFor" should evaluate the feature flag as true. This can solve some simple case.
But there are some more complex scenarios, for example, I have 3 filters: filter1, filter2 and filter3:
The filter1 is the pivot filter which can directly decide whether this feature flag should be enabled.
The logical relation will be like filter1 || (filter1 && filter2) || (filter1 && filter3). Only when this is true, the feature flag should be enabled.
In the issue #28, we have discussed the possibility to support complex logical relation between filters. We may consider to support that in the future. For now, hope "RequirementType" can provide your some convenience.


Fixes #68

Do not merge yet.
Fixes #68 <== This gives us a link to the issue plus a backlink from the issue to this PR.

Just going to bed, will review tomorrow.

This looks good to me. I'd be OK with either waiting or merging it as is, while you continue to work on the formatting and coloring of the XML.

I’m fine either way. It will be a couple of days until I have time to finish this, so if you need it now go ahead and merge it.

I'll go ahead and merge it. I'm working on some changes that will merge more smoothly later if I get your new presenter into the build right away.
We have not had much luck in reusing PRs once they are merged but you can create a new PR from the same branch and referring to the same issue.


feat: playbook and UAT for new supported OS: Rocky linux 9

Should create another MR with least information.
Please add only client you has been testing and passed into Client OS Supported section.


Fix master

I was having the following error:
/media/erle/myfiles1/github/moveitest/src/moveit2/moveit_core/kinematic_constraints/src/kinematic_constraint.cpp: In member function ‘virtual kinematic_constraints::ConstraintEvaluationResult kinematic_constraints::VisibilityConstraint::decide(const moveit::core::RobotState&, bool) const’:
/media/erle/myfiles1/github/moveitest/src/moveit2/moveit_core/kinematic_constraints/src/kinematic_constraint.cpp:998:90: error: no matching function for call to ‘collision_detection::AllowedCollisionMatrix::setDefaultEntry(const char [5], boost::_bi::bind_t<bool, boost::_mfi::cmf1<bool, kinematic_constraints::VisibilityConstraint, const collision_detection::Contact&>, boost::_bi::list2<boost::_bi::value<const kinematic_constraints::VisibilityConstraint*>, boost::arg<1> > >)’
   acm.setDefaultEntry("cone", boost::bind(&VisibilityConstraint::decideContact, this, _1));

@vmayoral check if you can compile with these changes

As discussed this is not the way to go since it'll break in OS X. Hopefully we can reach a compromise, I'll have a look on how to address this.
Leaving it open until we find a solution.

A somewhat similar issues was described at https://answers.ros.org/question/277184/boostbind-errors-in-subscriber-callback/:
error: no matching function for call to ‘ros::NodeHandle::subscribe(const char [4], int, boost::_bi::bind_t<void, void (*)(const boost::shared_ptr<const nav_msgs::OccupancyGrid_<std::allocator<void> > >&, int), boost::_bi::list2<boost::arg<1>, boost::_bi::value<int> > >)’
  ros::Subscriber sub2=n.subscribe("map",1, boost::bind(&mapcallback, _1, test));

as opposed to what we're getting:
error: no matching member function for call to 'setDefaultEntry'
  acm.setDefaultEntry("cone", boost::bind(&VisibilityConstraint::decideContact, this, _1));
  ~~~~^~~~~~~~~~~~~~~
/Users/victor/ros2_moveit_ws/src/moveit2/moveit_core/collision_detection/include/moveit/collision_detection/collision_matrix.h:215:8: note: candidate function not viable: no known conversion from '_bi::bind_t<bool, _mfi::cmf1<bool, VisibilityConstraint, const Contact &>, typename _bi::list_av_2<const VisibilityConstraint *, arg<1> >::type>' (aka 'bind_t<bool, cmf1<bool, kinematic_constraints::VisibilityConstraint, const collision_detection::Contact &>, list2<value<const kinematic_constraints::VisibilityConstraint *>, arg<1> > >') to 'bool' for 2nd argument
  void setDefaultEntry(const std::string& name, bool allowed);
       ^
/Users/victor/ros2_moveit_ws/src/moveit2/moveit_core/collision_detection/include/moveit/collision_detection/collision_matrix.h:225:8: note: candidate function not viable: no known conversion from '_bi::bind_t<bool, _mfi::cmf1<bool, VisibilityConstraint, const Contact &>, typename _bi::list_av_2<const VisibilityConstraint *, arg<1> >::type>' (aka 'bind_t<bool, cmf1<bool, kinematic_constraints::VisibilityConstraint, const collision_detection::Contact &>, list2<value<const kinematic_constraints::VisibilityConstraint *>, arg<1> > >') to 'collision_detection::DecideContactFn &' (aka 'function<bool (collision_detection::Contact &)> &') for 2nd argument
  void setDefaultEntry(const std::string& name, DecideContactFn& fn);

Related to this discussion https://www.boost.org/doc/libs/1_64_0/libs/bind/doc/html/bind.html#bind.troubleshooting.const_in_signatures

Also note that applying the fix provided here leads to https://github.com/AcutronicRobotics/moveit2/issues/3

@anasarrak see https://github.com/AcutronicRobotics/moveit2/commit/d992c2eb3be54c364722c698f2026fb9dfa5279e for a fix.
Please note that changes have been introduced in the CMakeLists.txt as well (not very sure how you were compiling it tbh). Review https://index.ros.org/doc/ros2/Contributing/Migration-Guide/ for future needs on the official recommendations.


examples: update archlinux and debian

Arch linux failure
+ limactl shell archlinux sudo cat /var/log/cloud-init-output.log
Cloud-init v. 21.2 running 'init-local' at Mon, 02 Aug 2021 06:26:11 +0000. Up 16.68 seconds.
2021-08-02 06:26:12,983 - util.py[WARNING]: failed stage init-local
failed run of stage init-local
------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python3.9/site-packages/cloudinit/cmd/main.py", line 652, in status_wrapper
    ret = functor(name, args)
  File "/usr/lib/python3.9/site-packages/cloudinit/cmd/main.py", line 361, in main_init
    init.apply_network_config(bring_up=bool(mode != sources.DSMODE_LOCAL))
  File "/usr/lib/python3.9/site-packages/cloudinit/stages.py", line 744, in apply_network_config
    return self.distro.apply_network_config(netcfg, bring_up=bring_up)
  File "/usr/lib/python3.9/site-packages/cloudinit/distros/__init__.py", line 210, in apply_network_config
    dev_names = self._write_network_config(netconfig)
  File "/usr/lib/python3.9/site-packages/cloudinit/distros/arch.py", line 67, in _write_network_config
    return self._supported_write_network_config(netconfig)
  File "/usr/lib/python3.9/site-packages/cloudinit/distros/__init__.py", line 122, in _supported_write_network_config
    renderer.render_network_config(network_config)
  File "/usr/lib/python3.9/site-packages/cloudinit/net/renderer.py", line 54, in render_network_config
    return self.render_network_state(
  File "/usr/lib/python3.9/site-packages/cloudinit/net/netplan.py", line 237, in render_network_state
    self._net_setup_link(run=self._postcmds)
  File "/usr/lib/python3.9/site-packages/cloudinit/net/netplan.py", line 257, in _net_setup_link
    subp.subp(cmd, capture=True)
  File "/usr/lib/python3.9/site-packages/cloudinit/subp.py", line 293, in subp
    raise ProcessExecutionError(stdout=out, stderr=err,
cloudinit.subp.ProcessExecutionError: Unexpected error while running command.
Command: ['udevadm', 'test-builtin', 'net_setup_link', '/sys/class/net/lo']
Exit code: 1
Reason: -
Stdout:
Stderr: Trying to open "/etc/systemd/hwdb/hwdb.bin"...
        Trying to open "/etc/udev/hwdb.bin"...
        Trying to open "/usr/lib/systemd/hwdb/hwdb.bin"...
        Trying to open "/usr/lib/udev/hwdb.bin"...
        === trie on-disk ===
        tool version:          249
        file size:        11124547 bytes
        header size             80 bytes
        strings            2374771 bytes
        nodes              8749696 bytes
        Load module index
        Found cgroup2 on /sys/fs/cgroup/, full unified hierarchy
        Found container virtualization none.
        Network interface NamePolicy= disabled on kernel command line, ignoring.
        Loaded timestamp for '/etc/systemd/network'.
        Loaded timestamp for '/run/systemd/network'.
        Parsed configuration file /usr/lib/systemd/network/99-default.link
        Parsed configuration file /run/systemd/network/10-netplan-eth0.link
        Created link configuration context.
        lo: Failed to query device driver: Operation not supported
        lo: No matching link configuration found.
        Builtin command 'net_setup_link' fails: No such file or directory
        Unload module index
        Unloaded link configuration context.
------------------------------------------------------------

https://cirrus-ci.com/task/5593459898712064?logs=test#L307
cc @jandubois

Seems https://github.com/systemd/systemd/issues/20300 , regression in systemd v249 https://github.com/systemd/systemd/commit/70f32a260b5ebb68c19ecadf5d69b3844896ba55

https://mirror.pkgbuild.com/images/v20210801.30574/Arch-Linux-x86_64-cloudimg-20210801.30574.qcow2 is affected because it uses systemd 249 (249.2-1-arch).

What do you want to do about it? Revert the update to the ArchLinux example until the issue is fixed upstream?
Or maybe not create network-config if the only entry is eth0?  Or add a limayaml setting for it?

Reverted arch.


Don't save environment as an instance variable in GitLabCI

This addresses https://github.com/danger/danger/issues/1351.

Cool, yeah, looks good to me 👍🏻

Shipped as 8.4.3 - thanks!


Would like to be able to account for memory overhead of Vm Service / Devtools in memory view

We recently had a bit of a close call where it looked like an example app's memory usage was significantly larger in Flutter than it was in Android Native. Upon further investigation, we discovered that running this app in release mode caused the memory usage to decline significantly.
To avoid this confusion in the future, it would be great if we could at least partially account for the memory overhead of the VM Service/Devtools in profile mode. If not, we should direct users away from the devtools memory profiler towards native tools for investigations of overall memory usage.
cc @xster @kenzieschmoll

CC @terrylucas @jacob314

This information is currently available in Observatory through the Process Snapshot tool:

We can make this information available through DevTools using the getProcessMemoryUsage RPC in the service protocol.


Add support for keyboard media keys

Adds support for keyboards that have keys to play/pause, go forward, or rewind.
Does not add media keys for turning the volume up/down, or muting since these keys are done by the operating system.

Nice idea, thanks.


Many to many relationship problem. No association from model.

Hi,
I am getting the following error when I try to retrieve data from a Many to Many relationship:
" org.javalite.activejdbc.associations.NotAssociatedException: No association from model 'class com.example.model.PaymentPlan' to model 'class com.example.model.PaymentRate' "
The code is:
Request r = Request.findById(21);
ServiceProvider sp = ServiceProvider.findById(r.getSpId());
Company company = sp.getCompany();
FinantialInformation finantialInformation = company.getFinantialInformation();
PaymentPlan paymentPlan = finantialInformation.getPaymentPlan();
List<PaymentRate> paymentRates = paymentPlan.getPaymentRates();

And the exception is:
[2017-08-25T01:30:12.436-0300] [glassfish 4.1] [WARNING] [] [javax.enterprise.web] [tid: _ThreadID=27 _ThreadName=http-listener-1(3)] [timeMillis:<PHONE_NUMBER>436] [levelValue: 900] [[
  StandardWrapperValve[kSOSLegal_dev]: Servlet.service() for servlet kSOSLegal_dev threw exception
org.javalite.activejdbc.associations.NotAssociatedException: No association from model 'class com.example.model.PaymentPlan' to model 'class com.example.model.PaymentRate'
	at org.javalite.activejdbc.Model.get(Model.java:1881)
	at com.example.model.PaymentPlan.getPaymentRates(PaymentPlan.java:10)
	at com.example.jersey.Services.showNearbyServices(Services.java:66)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)

The database tables are generated by Rails framework, so I believe that these tables follow the conventions:
PaymentPlan:
CREATE TABLE `payment_plans` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `created_at` datetime NOT NULL DEFAULT CURRENT_TIMESTAMP,
  `updated_at` datetime NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP,
  `finantial_information_id` int(11) DEFAULT NULL,
  `company_size_id` int(11) DEFAULT NULL,
  PRIMARY KEY (`id`),
  KEY `index_payment_plans_on_finantial_information_id` (`finantial_information_id`),
  KEY `index_payment_plans_on_company_size_id` (`company_size_id`),
  CONSTRAINT `fk_rails_3c68b47cc9` FOREIGN KEY (`company_size_id`) REFERENCES `company_sizes` (`id`),
  CONSTRAINT `fk_rails_8c89a0cc30` FOREIGN KEY (`finantial_information_id`) REFERENCES `finantial_informations` (`id`)
) ENGINE=InnoDB AUTO_INCREMENT=7 DEFAULT CHARSET=utf8

PaymentRate:
CREATE TABLE `payment_rates` (
  `Id` int (11) NOT NULL AUTO_INCREMENT,
  `Rate` decimal (16,2) DEFAULT NULL,
  `Created_at` datetime NOT NULL,
  `Updated_at` datetime NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE = InnoDB DEFAULT CHARSET = utf8 COLLATE = utf8_unicode_ci

PaymentPlanPaymentRate:
CREATE TABLE `payment_plans_rates` (
  `Payment_plan_id` int (11) NOT NULL,
  `Payment_rate_id` int (11) NOT NULL,
  KEY `index_payment_plans_rates_on_payment_plan_id` (` payment_plan_id`),
  KEY `index_payment_plans_rates_on_payment_rate` (` payment_rate_id`)
) ENGINE = InnoDB DEFAULT CHARSET = utf8 COLLATE = utf8_unicode_ci

Classes:
PaymentPlan:
public class PaymentPlan extends Model {

	public List<PaymentRate> getPaymentRates () {
		return this.getAll(PaymentRate.class);
	}
}

PaymentRate:
public class PaymentRate extends Model {

}

PaymentPlansRates:
public class PaymentPlansRates extends Model {

}

Could anyone help me understand why the framework is not getting associations? What am I doing wrong?
Thanks in advance.

@alfredodfn can you turn on logging as described here: http://javalite.io/logging. The framework will log an ERD as described here: http://javalite.io/runtime_discovery. Please, attach that to the story.

@alfredodfn Also, it is possible that your MySQL configuration is case sensitive. Your foreign keys in payment_plans_rates may be capitalized, and as a result not discovered by the framework by default. The crow foot diagram will show what the framework finds.

Thank you for your help
activejdbc_log.txt
@ipolevoy
I attached the file with logging info. I checked the tables and they look OK.
mysql> show columns from payment_plans;
+--------------------------+----------+------+-----+-------------------+-----------------------------+
| Field                    | Type     | Null | Key | Default           | Extra                       |
+--------------------------+----------+------+-----+-------------------+-----------------------------+
| id                       | int(11)  | NO   | PRI | NULL              | auto_increment              |
| created_at               | datetime | NO   |     | CURRENT_TIMESTAMP |                             |
| updated_at               | datetime | NO   |     | CURRENT_TIMESTAMP | on update CURRENT_TIMESTAMP |
| finantial_information_id | int(11)  | YES  | MUL | NULL              |                             |
| company_size_id          | int(11)  | YES  | MUL | NULL              |                             |
+--------------------------+----------+------+-----+-------------------+-----------------------------+
5 rows in set (0.00 sec)

mysql> show columns from payment_rates;
+------------+---------------+------+-----+---------+----------------+
| Field      | Type          | Null | Key | Default | Extra          |
+------------+---------------+------+-----+---------+----------------+
| id         | int(11)       | NO   | PRI | NULL    | auto_increment |
| rate       | decimal(16,2) | YES  |     | NULL    |                |
| created_at | datetime      | NO   |     | NULL    |                |
| updated_at | datetime      | NO   |     | NULL    |                |
+------------+---------------+------+-----+---------+----------------+
4 rows in set (0.00 sec)

mysql> show columns from payment_plans_rates;
+-----------------+---------+------+-----+---------+-------+
| Field           | Type    | Null | Key | Default | Extra |
+-----------------+---------+------+-----+---------+-------+
| payment_plan_id | int(11) | NO   | MUL | NULL    |       |
| payment_rate_id | int(11) | NO   | MUL | NULL    |       |
+-----------------+---------+------+-----+---------+-------+
2 rows in set (0.01 sec)

your join table payment_plans_rates is not called the way it is expected for many-to-many relationship. A proper name needs to be payment_plans_payment_rates.

you can fix it by overriding using annotations, if you cannot change the table name: http://javalite.io/many_to_many_associations#overriding-associations

I tried this before and the error was the same. Anyway, I tried again, as follows:
@Table("payment_plans_rates")
public class PaymentPlansRates extends Model {}

The output error is:
[2017-08-25T16:38:52.894-0300] [glassfish 4.1] [WARNING] [] [javax.enterprise.web] [tid: _ThreadID=29 _ThreadName=http-listener-1(4)] [timeMillis:<PHONE_NUMBER>894] [levelValue: 900] [[
  StandardWrapperValve[kSOSLegal_dev]: Servlet.service() for servlet my_servlet threw exception
org.javalite.activejdbc.associations.NotAssociatedException: No association from model 'class com.example.model.PaymentPlan' to model 'class com.example.model.PaymentRate'

The migration to create join table:
class CreateJoinTablePaymentPlanPaymentRate < ActiveRecord::Migration
  def change
    create_join_table :payment_plans, :payment_rates do |t|
      t.index :payment_plan_id
      t.index :payment_rate_id
    end
  end
end

Any idea? I confess that I am stuck in this problem. I'll investigate why rails generated the table with the wrong name.

Hi,
I have some news. Rails really creates tables in that format when using create_join_table command (https://github.com/rails/rails/issues/9643). With the annotation nothing happens, but renaming MySQL table did work.

@alfredodfn what you did of course works, but I was referring to a different annotation. This should have worked with existing tables:
@Many2Many(other = PaymentRate.class, join = "payment_plans_rates", sourceFKName = "payment_plan_id", targetFKName = "payment_rate_id")
public class PaymentPlan extends Model {}

One such annotation would be enough.

@ipolevoy Thanks for your help. The framework is fantastic and I'm replacing it in all my applications.

@alfredodfn awesome, do you also use ActiveWeb?

@ipolevoy: not currently, but I'm evaluating the possibility of migrating. I'll check it out better. Thanks.


Strange behavior after watchdog task reset (IDFGH-5872)

I'm using ESP-IDF 4.3.1 with VScode 1.60.1 on Windows 11. I have the following code at the top of my app_main():
esp_reset_reason_t reason = esp_reset_reason();
printf("\nReset Reason: %d. Execution halted.", reason);
fflush(stdout);
if (reason != ESP_RST_POWERON)
{
    sleep(10000);
}
Then I enter a tight loop reading the adc and printing it using:
printf("\nRaw: %d\tVoltage: %dmV", adc_reading, voltage);
However, the tight loop eventually triggers the task watchdog. So I see the following errors on restart:

E (20421) task_wdt: Task watchdog got triggered. The following tasks did not reset the watchdog in time:
E (20421) task_wdt:  - IDLE (CPU 0)
E (20421) task_wdt: Tasks currently running:
E (20421) task_wdt: CPU 0: main
E (20421) task_wdt: Print CPU 0 (current core) backtrace

However, the above reset reason code never gets executed. It never shows any output. It jumps straight back to the loop which again triggers the watchdog, and it keeps doing so, even though it's supposed to print out a reset reason then go to sleep (which works however for other reset reasons (it always prints the reason and sleeps if it's not poweron).
Although it occured to me the chip might not necessarily be resetting after the error message, that still doesn't explain why it isn't printing out the reason nor sleeping if it's anything besides POWERON, yet still executing the rest of the app_main() function.

https://github.com/espressif/esp-idf/blob/220590d599e134d7a5e7f1e683cc4550349ffbf8/components/esp_system/Kconfig#L381

Hi, @moefear85
As @boarchuz already mentioned, the task WDT doesnt reset the chip by default. It can be enabled by setting ESP_TASK_WDT_PANIC in menuconfig.

Although it occured to me the chip might not necessarily be resetting after the error message, that still doesn't explain why it isn't printing out the reason nor sleeping if it's anything besides POWERON, yet still executing the rest of the app_main() function.

Not sure I completely understand you here. As you suspected, the chip is not resetting, but if the chip is not resetting it will never run app_main again. It will just keep running whatever it was running before the WDT triggered, i.e. your tight loop.


Generated backend HTTPS certificate age check should check expiration

Feature Suggestion
The following logic is currently used to check the whether we need to generate a new HTTPS certificate for development: https://github.com/backstage/backstage/blob/5330e3758888b46da7833cdd2ae8efd39797044e/packages/backend-common/src/service/lib/hostFactory.ts#L100-L101
It would be preferable to check the actual certificate expiration instead, since the file could be recreated with an outdated certificate.
Possible Implementation
We already use node-forge to generate certificates, so we could use it to parse the existing one as well. See for example this SO answer

Assigned @gibsonliketheguitar 🎸 !

Hello, is this issue still worked on or could I take it? I'm new to backstage and would like to contribute :)

Hello, is this issue still worked on or could I take it? I'm new to backstage and would like to contribute :)

Hey @Znarvl you can take over!. I've been busy with other stuff.
Where I got stuck on.
I tried tracing to where the certificate is created and tried the local dev route. But in order to test and confirm that node-froge is working, I think you need to deploy it to docker. I haven't gotten around setting up docker and stuff
Good luck!

Alright, thanks!

I've worked on this. Hopefully the implementation is fine.


cannot draw polyline to the north pole

im trying to draw a polyline to the north pole the best I can do is a little bit off of the north pole but if I do exactly (90,0) it is just invisible you cant even see the polyline
this picture is with Geopoint set to (-90.00001,0.0) as you can see it is not going all the way to the marker and id like a solution where the polyline goes all the way to the north pole

GeoPoint gp =  new GeoPoint(gplat,gplon);
            GeoPoint ap =  new GeoPoint(aplat,aplon);
            GeoPoint north = new GeoPoint(-90.00001,0.0);
Polyline polyline = new Polyline();
                polyline.addPoint(north);
                polyline.addPoint(gp);
                polyline.addPoint(ap);
                polyline.addPoint(north);
                polyline.setColor(Color.RED);
                polyline.setWidth(20);
                map.getOverlays().add(polyline);

it's a limitation of the projection, the best you can do is +-85 and change

Ahhh ok guess thatll have to do


Implement trait Isqrt for Uint128 and Uint256

Isqrt is implemented in https://github.com/CosmWasm/cosmwasm/blob/v0.16.0/packages/std/src/math/isqrt.rs already, but only for primitive types.

@webmaster128  I'm new to rust but it seems like the Isqrt trait isn't exposed in
https://github.com/CosmWasm/cosmwasm/blob/main/packages/std/src/lib.rs#L45-L47
Is there a way to use the trait in a project that depends on cosmwasm-std?
Uint256::new(1).isqrt()

Gives this error
no method named `isqrt` found for struct `cosmwasm_std::Uint128` in the current scope

method not found in `cosmwasm_std::Uint128`

help: items from traits can only be used if the trait is in scoperustc(E0599)
[isqrt.rs(9, 8): ]()the method is available for `cosmwasm_std::Uint128` here

Oh, good point. The trait needs to be exported. Tracked in https://github.com/CosmWasm/cosmwasm/issues/1214


Remove custom handling in add_swift_host_library

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test

CC: @gottesmm @Rostepher @gwynne

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - bc3160f6d220e66c265027756378730b892678b7

Build failed
Swift Test Linux Platform
Git Sha - bc3160f6d220e66c265027756378730b892678b7

@swift-ci please test

@Rostepher, @gottesmm - is this okay?

@Rostepher @gottesmm - ping x 2?

@Rostepher @gottesmm - ping x 3

I checked all instances of uses of this and it merges cleanly. LGTM.


Visualization/Chipotle: Fix axis labelling

Visualization / Chipotle, step 5: y axis label should say "Number of Times Ordered" instead of "Price" to conform with the question. The value shown is correct, just the label is slightly confusing.
(You can ignore the first two commits, I created the branch a while ago and needed the second commit to merge the changes that had been done in your main repository in the meantime.)

@alletsee , thank you for the PR!! 😊
Why are you including "(except UK)" in the exercise Online Retails Purchase Step 6?


chore: Merge from upstream as of 2022-04-08

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

244 of 298 (81.88%)  changed or added relevant lines in 10 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.002%) to 80.046%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/provider/azure_managedDiskController.go
4
5
80.0%

pkg/azureclients/storageaccountclient/azure_storageaccountclient.go
49
51
96.08%

pkg/provider/azure_loadbalancer.go
69
77
89.61%

pkg/provider/azure_storageaccount.go
12
20
60.0%

pkg/azureclients/privatelinkserviceclient/azure_privatelinkserviceclient.go
79
114
69.3%

Files with Coverage Reduction
New Missed Lines
%

pkg/provider/azure_managedDiskController.go
1
85.45%

pkg/provider/azure_loadbalancer.go
3
84.96%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
14598

Relevant Lines:
18237

💛  - Coveralls


Add ciliumnetworkpoliccy for the CRD install job

Now that we use cilium as our CNI, the network policies that rely on ipBlock won't work. We need to add a ciliumnetworkpolicy instead.

Checklist

[X] Added a CHANGELOG entry

Testing
The instance of external-dns installed as part of Giant Swarm platform releases watches services in the kube-system namespace with annotations giantswarm.io/external-dns=managed and external-dns.alpha.kubernetes.io/hostname matching the clusters base domain. (You can find this in the deployments args --domain-filter value)
You can take this example Service, apply it to your cluster. Change the external-dns.alpha.kubernetes.io/hostname annotation to match your clusters base domain.
then:

Check external-dns logs for lines like Desired change: CREATE test.your.configured.domain.gigantic.io CNAME
Try to resolve the domain (https://www.dnstester.net/)

apiVersion: v1
kind: Service
metadata:
  annotations:
    external-dns.alpha.kubernetes.io/hostname: test.your.configured.domain.gigantic.io
    external-dns.alpha.kubernetes.io/ttl: "60"
    giantswarm.io/external-dns: managed
  name: test-external-dns
  namespace: kube-system
spec:
  type: ExternalName
  externalName: www.giantswarm.io

For testing upgrades:

Create the service and check for creation
Upgrade
Delete the service and check for deletion

Default app on AWS releases

[ ] Fresh install works
[ ] Upgrade works

Default app on Azure releases

[ ] Fresh install works
[ ] Upgrade works

Optional app (KVM)

[ ] Fresh install works
[ ] Upgrade works

why is this part of some crd or job i would rather see it as normal resource in template directory

why is this part of some crd or job? I would rather see it as a normal resource in the template directory

It's required by the crd-install job to connect to the k8s api. It's not needed by external-dns itself.

Well... I would have put this into a separate file.


Create .travis.yml

Add link to allow forks to use local resources

See example at https://travis-ci.org/stephengroat/jp/builds/357126393

Cool, thanks! Auto releases sounds great 👍


Increase maximum allowed avatar image size

Default is 1MB, I'm thinking maybe update to 2MB then resizing to a lower resolution on the server if possible.
https://django-avatar.readthedocs.org/en/latest/

Duplicate of #282


CSharp file searches for usings after they've been removed

When a user uses Context Action to define selected usings as global, it removes them to the global file, but the current open file's been looking for usings. It needs to be re-opened to "understand" the recent updates.

Additional video to describe the bug
https://github.com/DenisSemko/UsingLocator/assets/53062219/5db42560-558d-4951-a2a6-588ffe60d2e7


Fix VERIFICATION & Add Chocolatey package as AppVeyor artifact

Related to #137.
Expected output in AppVeyor - https://ci.appveyor.com/project/tomkerkhove/servicebusexplorer
However, need to check why it doesn't find any artifacts anymore.

Feel free to squash when merging in, a bit too many changes.

:white_check_mark: Build ServiceBusExplorer 4.0.103-PullRequest0143.14.build.288 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/8a7f5c1ccb by @tomkerkhove)

@tomkerkhove have you submitted with the same version? Seems like this is what choco admins want.

:white_check_mark: Build ServiceBusExplorer 4.0.103-PullRequest0143.15.build.289 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/fae9c9da2e by @tomkerkhove)

:white_check_mark: Build ServiceBusExplorer 4.0.103-PullRequest0143.16.build.290 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/121b6ee45c by @tomkerkhove)

@SeanFeldman Please re-review. Fixed the issue with AppVeyor and hardcoded the version for choco validation to pass first

@SeanFeldman Rolled back the hardcoded changes and will create new version of Chocolatey package once it's merged in

:white_check_mark: Build ServiceBusExplorer 4.0.103-PullRequest0143.18.build.291 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/54024784be by @tomkerkhove)

@tomkerkhove Squash?

@seanfeldman I think it's best to squash merge indeed


Add COMPOSER_MEMORY_LIMIT for CI composer install tasks

I've run into composer memory issues in the past and using COMPOSER_MEMORY_LIMIT generally solves the issue. I've not used CircleCI much, but this looks like it might work. Now to see how the build runs.

We could try. It will make every build way slower, but if it doesn’t crash, I’m ok with it.

I think this is sorted out now. The composer.lock file previously included laravel/framework version 5.5. I'm still not quite sure what caused that to be included, but it was not installed with a fresh start. That led to a number of test failures because Laravel framework features (like app()) were not bootstrapped. That issue was resolved by requiring the orchestra/testbench dependency which helps get the Laravel framework running during tests. There were a couple of tests to fix as a result of various updates, but overall, it was straightforward.
I took this opportunity to bump the required Laravel components to a minimum of 6 and a maximum of 8. Version 6 is what you will get for development right now. (More on that with a later PR.)

I restored the supported versions and will work on a new PR.
You mentioned a concern with the jobs being slower without the composer.lock file. What are your goals for the CI jobs? I ask because this relates to what I want to suggest next.

You mentioned a concern with the jobs being slower without the composer.lock file. What are your goals for the CI jobs? I ask because this relates to what I want to suggest next.

It's not a big deal for a small project like this one,  but without a composer.lock file, now Composer takes around 45s on every job to decide which dependencies to install, while it was 0 before. We are talking about 2 extra minutes to run all CI jobs.
I think it's fine, considering how annoying is to deal with conflicts in the lock file and Composer updating all dependencies when you just want to update one...


Write a TypeScript program to check consonant

Description
Write a TypeScript program to check consonant
Check if a character is a consonant.
Input  : b
Output : Consonant

How to contribute

Save the solution in CheckConsonant.ts file
Add CheckConsonant.ts file in check-consonant folder

!assign


[MM-28140] e2e/system_console/customization: add T1024

Summary
Adds T1024: Customization Custom Branding - Can change name and desc with custom branding set to false
Ticket Link
https://mattermost.atlassian.net/browse/MM-28140

/update-branch


Panic with nested islands

Currently, rendering an island directly in another panics (ex: OuterFailing). The only way around this is to render via children (ex: OuterWorking).
This can be problematic as it uses up the child prop, and the slot solution to this is incompatible with islands. As a workaround I have various "island wrappers" which wrap a component in an empty island component, but it can get quite messy.
Perhaps what I'm attempting is an anti-pattern and nesting islands should be best avoided?
use leptos::*;

#[island]
pub fn CommonIsland() -> impl IntoView {
    let val = RwSignal::new(0);
    view! {

            {move || format!("CommonIsland value is {}", val.get())}
            <button on:click=move|_| val.update(|x| {*x += 1})>Click</button>

    }
}

#[island]
pub fn OuterWorking(children: Children) -> impl IntoView {
    let val = RwSignal::new(0);
    view! {
        <>

                {move || format!("outer value is {}", val.get())}
                <button on:click=move|_| val.update(|x| {*x += 1})>Click</button>

            {children()}
        </>

    }
}

#[island]
pub fn OuterFailing() -> impl IntoView {
    let val = RwSignal::new(0);
    view! {
        <>

                {move || format!("outer value is {}", val.get())}
                <button on:click=move|_| val.update(|x| {*x += 1})>Click</button>

            <CommonIsland/>
        </>

    }
}
#[component]
pub fn Example() -> impl IntoView {
    view! {
        <OuterFailing/>

        <OuterWorking>
            <CommonIsland/>
        </OuterWorking>

        <CommonIsland/>
    }
}

(Oh I should add: It is likely this will be fixed in 0.7, which is where I'm spending most of my energy, as the hydration works completely differently. I will tag this with 0.7 to make sure it is closed in that version if not before.)

Great! In the meantime I'll do a little digging and see if I can find anything. Will update on here if so.

Can confirm this no longer panics on 0.7/git main, and all buttons seem to work.


Not supporting linux/arm64 based runners

Hi @trstringer,
I am facing this(screenshot) issue on my self-hosted arm64 based linux ec2 instance.

I would appreciate if you could deploy docker image with both amd64 and arm64 manifests, currently image with Dockerfile configuration on have single arch i.e, amd64.
Hope you will update this soon ✌. Thanks In advance 🙂

Thanks for opening this issue! Great idea, I'll add ARM support to the backlog. Thanks!

My branch to add arm support to this project has been open for almost a year 😢

We still need arm support for arm runners

Yep, I agree. My branch remains open and untouched for so long. I think this project might be dead.


New API schema with important changes

It's available here: https://github.com/ncbi/clinvar/commit/1a12efdd372dcb0e44396e610e2c1d67a2a4b59e
Breaks many tests with the demo examples available on this repo

Looks like our test file is failing validation with an error, but also their test fails fail the validation with the same error. I've opened an issue on their repo: https://github.com/ncbi/clinvar/issues/7

Fixed in #108


Release tools project dogfoods versioning plugin

Problem
Release tools project currently defines its version in root "build.gradle" file and it is problematic because:

it is not bumped automatically with the release
it triggers merge conflicts in "build.gradle" file instead of merge conflicts in a simpler 'version.properties' file.

Suggested implementation

Have the release tools project consume the 'org.mockito.release-tools.versioning' plugin from the previously released version of tools.
Update the travisRelease task so that when we release we bump version automatically.
Bonus - we should configure versioning plugin in 'pluginBundle' section of 'build.gradle' file. See the documentation of Gradle plugins portal: https://plugins.gradle.org/docs/publish-plugin.

Future
We should make the release tools plugin actually consume the entire continuous delivery plugin. We can get there step by step.

@wwilk, do you want to take a stab at it? It's related to the version fix you've done earlier and might be good to get you immersed in the project more!

Sure! I will work on it soon.

Fiendly ping :) It bothers me that we always have to remember to bump the version of tools when publishing :/

I'm thinking that we should make the plunge and start dogfooding release notes generation in this project, too. Let's rock! What do you think? Or perhaps new plugin like "GradlePluginReleasingPlugin"? :D

Sure, we can do that!


stuck on reloading in Android TV

I have checked wifi speed.. its very good and everything working fine but as in one drive i have many files and when i scroll down a little its keep reloading and stuck on it.. as result I can't access below videos...
Please do something about it... 🙏🏻

A quick solution would be to star the files you want.

Yeah this can be a quick solution but it will only works if we are using application with same id in mobile too...
Otherwise if we are using only in tv then starred won't help..
If there is any permanent solution for this please try to fix it.. 🙏🏻

I've an idea on what's going wrong, but if you could record a short video demonstrating the problem, I will be able to confirm. Thanks!

https://github.com/itszechs/DriveStream/assets/105588608/bc24743e-c5d4-4094-96c4-d156761019a6
Here bro..

I was going to upload video at first but then i thought I don't know if that is allowed or not due to copyright concern

Heyy bro.. did you find any solution for it?

Hello bro... Found any solution of it yet ??

Heyy bro i know you must be busy on something and i don't want to disturb you.. but this problem i mentioned above is still a big problem.. i hope you can fix this please

I was not able to reproduce this issue. I will get back to you for more information when I get around to fixing this.

Yes bro please.. i am always available to help.. i thought there may be some problem on Android TV then i tried on Fire TV Stick but issue was same..
For myself i can starred any file as a temporary solution but for my friends or family members if i login to TV then Starr option couldn't help for them..
I will always be in your debt.. please bro find some solution of it.. hope you are understanding 🙏🏻♥

@Kira-0007 can you share the file/folder list which is in the video from Google Drive website

Yes sure.. but for safety purpose can we talk in private chat where i can send you file/folders whatever you need... If it is ok for you

Yes sure.. but for safety purpose can we talk in private chat where i can send you file/folders whatever you need... If it is ok for you

Sure, you can reach me out on discord zechs#1620

Friend request sent.. please check

Hey bro u got solution to this error in android tv of loading


Revert "[gitlab] add links to CI Visibility from e2e jobs"

Reverts DataDog/datadog-agent#26098
incident-27736
This is breaking new-e2e-container [--skip... job as the job name contains spaces, breaking sed command
Example: https://gitlab.ddbuild.io/DataDog/datadog-agent/-/jobs/526901313

/merge


Retaining values not convertable to float64

It might be useful to assign non float values to individuals, e.g. strings representing categories or identifiers.
In our case, during the computational expensive evaluation of the population, we can obtain an string identifier with almost no additional costs, which would be in our case better suited for duplicate detection than a distance metric, as multiple individuals with different features might be of identical phenotype, and thus effectively reducing the variance in the population.
In our local environment, this change did not introduce undesired behavior.

Great point! Thanks for the contribution.


Use v1.8.2 for cert-manager tests

Motivation
the latest cert-manager(v1.9.0) chart is not compatible with k8s=v1.8.20
ERROR commands: [helm repo add jetstack [https://charts.jetstack.io\nhelm](https://charts.jetstack.io/nhelm) repo update\nhelm install cert-manager jetstack/cert-manager --set installCRDs=true\n] runs error: Error: INSTALLATION FAILED: chart requires kubeVersion: >= 1.19.0-0 which is incompatible with Kubernetes v1.18.20

Modifications
Use cert-manager v1.8.2 in tests environment
Verifying this change

[x] Make sure that the change passes the CI checks.

[x] This change is a trivial rework / code cleanup without any test coverage.

Documentation
Check the box below.
Need to update docs?

[x] no-need-doc
Fix tests

lgtm and here also needs to be updated: https://github.com/streamnative/function-mesh/blob/master/.ci/tests/integration/e2e.yaml#L20

it's already updated


Add a Codacy badge to README.md

Coverage remained the same at 100.0% when pulling c4e1d6f8a8f9ebc430aa936813575cc851220d28 on codacy-badger:codacy-badge into 798ac5eb13e2392e15a2eea9cc03d5d8e5048720 on Travix-International:master.


How to print UTF-8 Character

Please show how to print character like 'ทดสอบ'

Duplicate of #33

Duplicate of #33

Duplicate of #33

Oh, Thanks bro.


C++ variant

Hi, I wanted to know how I could make this work with C++.
Currently it is using C, but I want to use some things from C++ as well.
Unless there are ways to get C++ things working that I don't know about, I would love this.
Thanks in advance!

Hi, It's quite easy to make it work with C++, you just have to change main.c to main.cpp and the following lines in CMakelists.txt from:
project(raylib_template C)

set(CMAKE_C_STANDARD 99)

add_executable(raylib_template src/main.c)

to
project(raylib_template CXX)

set(CMAKE_CXX_STANDARD 11)
set(CMAKE_CXX_STANDARD_REQUIRED ON)

add_executable(raylib_template src/main.cpp)

Maybe I can add this to the readme.

Hi, thanks for coming back so quickly!
Yeah, this seems to work, thank you! I could not figure this out for some reason (Cmake is not my strong suit.)
Thank you for this :)
Have an nice day.


Feature/storage/blob access policy bug

Resolves https://github.com/Azure/azure-sdk-for-net/issues/14455

We also need this in File and DataLake too

@amnguye, great catch.


Game hangs on waitOnCPU in the endgame screen

Happens occasionally after the game is over and the endgame stats are presented.

Does this still occur to anyone?


Add support for rails 7.1

fixes #215
rails 7.1 recently added  this new default configuration option
When set to false ( default in rails 7.1 ) an if option is added to the belongs_to option. which makes it not compatible with the current WEAK_OPTIONS array.
To cover this new case, and as a pre-check, we make sure that we're on rails 7.1.0 minimum as well as the ActiveRecord option ( ActiveRecord.belongs_to_required_validates_foreign_key ) is set to false to mitigate issues with real if conditions. We also check that the if condition is coupled with a message option set to required
This commit also adds rails 7.1 to the CI matrix.
cc @djezzzl @agrobbin

@djezzzl btw I'm going to go ahead and try to add rails 7.1 to the github CI matrix. as well as add a unit test to the presence checker 👍🏼

rebased and squashed

Thank you very much for your contribution!
I'll update a changelog and release it today! Please let me know if you have any more ideas to make this gem more helpful.
Have a great day and the upcoming week!


[Update] Open Button for PearAI Autocomplete to Open Status Bar Settings

Description ✏
Fixes an issue outlined here: https://docs.google.com/document/d/13WF0UQdVFjVHCrKwCzq0puhNmrePzAb9H6UHd2p3pVU/edit?tab=t.0

The open button will no longer open Supermaven chat, but will instead open the Supermaven status bar settings.

Checklist ✅

[x] I have added screenshots (if UI changes are present).
[x] I have done a self-review of my code.
[x] I have manually tested my code (if applicable).

[!IMPORTANT]
Update AUTOCOMPLETE tool in InventoryPage.tsx to open Supermaven status bar settings instead of new chat tab.

Behavior:

Update commandId for AUTOCOMPLETE tool in InventoryPage.tsx to supermaven.onStatusBarClick to open Supermaven status bar settings instead of new chat tab.

This description was created by  for 269043fc67a26cb6acfae61865de3cd4f37cafe0. It will automatically update as commits are pushed.

ty.


Mounting the default virtualbox instance makes volumes unreachable

I have a docker-compose file where I put a volume to run an ansible playbook.
agent:
  image: pjestrada/ansible
  volumes:
    - ../../ansible/probe.yml:/ansible/site.yml
  links:
    - db
  environment:
    PROBE_HOST: "db"
    PROBE_PORT: "3306"

The file works great when I dont have my docker machine up. But as soon as I run my default docker-machine created with:
docker-machine create --driver virtualbox default
and use eval to set the env variables.
I run again:
docker-compose up agent
and get this error:
agent_1 | ERROR! the playbook: site.yml does not appear to be a file
dev_agent_1 exited with code 1

I think there must be some kind of problem with the filesystem when the virtualbox default driver. Im on Ubuntu 16.04 LTS. I even try to access the file on bash and I get

Did you ever get anywhere in solving this problem?


Implement kwargs for h5py

It took quite some time for me to get back to this one - https://github.com/h5io/h5io/pull/41

I learned the hard way that it only works with existing datasets but it does not seem to work with creating new datasets ... https://docs.h5py.org/en/stable/swmr.html#using-the-swmr-feature-from-h5py


add greater and less support for compatible elasticsearch lucene syntax

elasticsearch lucene syntax
https://www.elastic.co/guide/en/elasticsearch/reference/8.9/query-dsl-query-string-query.html#query-string-syntax
but only support
age:>10
age:>=10
age:<10
age:<=10

not support
age:(>=10 AND <20)
age:(+>=10 +<20)

Hey! Thanks for contributing a PR! I'll look through this in the next day or so. In general I would prefer to add the range support as well asa the greater than/less than. I'll look at how much work it is to add both and cut a release after both are supported

Hey @bigpigeon sorry I made a few changes to your PR with some extra test cases and plumbing fixes and had some trouble getting the branches to push. I ended up force pushing and it apparently closed this PR. Can you reopen it please? Thanks!

Hey @bigpigeon sorry I made a few changes to your PR with some extra test cases and plumbing fixes and had some trouble getting the branches to push. I ended up force pushing and it apparently closed this PR.

Hey @bigpigeon sorry I made a few changes to your PR with some extra test cases and plumbing fixes and had some trouble getting the branches to push. I ended up force pushing and it apparently closed this PR. Can you reopen it please? Thanks!

I put all the changes in https://github.com/grindlemire/go-lucene/tree/pr-5-changes just in case things got messed up in your fork. Sorry about that again!


Missing strictness marks when StrictData/Strict is on?

When haddock generates docs for a module with StrictData/Strict extensions, the datatype fields in the docs do not have regular ! marks, but they are strict under the hood. Is it possible to detect the extensions and add the missing strictness marks?

Currently haddock only looks at the SrcStrictness of a record field. c.f.
https://github.com/haskell/haddock/blob/22d5e59a9bb7f5ad5612c9dde53419a48101be65/haddock-api/src/Haddock/Backends/Xhtml/Decl.hs#L1037
You would want to get the HsImplBang - the strictness GHC calculates, not the one the user wrote - to do this.
Care to make a PR?

I gave it a go, tried it with this diff:
diff --git a/haddock-api/src/Haddock/Convert.hs b/haddock-api/src/Haddock/Convert.hs
index 7735ed0d..cc8391c9 100644
--- a/haddock-api/src/Haddock/Convert.hs
+++ b/haddock-api/src/Haddock/Convert.hs
@@ -313,12 +313,14 @@ synifyDataCon use_gadt_syntax dc =
   ctx = synifyCtx theta

   linear_tys =
-    zipWith (\ty bang ->
+    zipWith3 (\ty (HsSrcBang srcTxt _ _) bang ->
                let tySyn = synifyType WithinType ty
                in case bang of
-                    (HsSrcBang _ NoSrcUnpack NoSrcStrict) -> tySyn
-                    bang' -> noLoc $ HsBangTy noExt bang' tySyn)
-            arg_tys (dataConSrcBangs dc)
+                    HsLazy -> noLoc $ HsBangTy noExt (HsSrcBang srcTxt SrcNoUnpack SrcLazy) tySyn
+                    HsStrict -> noLoc $ HsBangTy noExt (HsSrcBang srcTxt SrcNoUnpack SrcStrict) tySyn
+                    HsUnpack _ -> noLoc $ HsBangTy noExt (HsSrcBang srcTxt SrcUnpack SrcStrict) tySyn
+            )
+            arg_tys (dataConSrcBangs dc) (dataConImplBangs dc)

   field_tys = zipWith con_decl_field (dataConFieldLabels dc) linear_tys
   con_decl_field fl synTy = noLoc $

But it didn't work. :( It's not obvious where else HsImplBangs could be injected... any pointers would be appreciated as one of the things holding me back from switching on StrictData by default is the misleading haddock docs that are generated.


Update deprecated textbook URL in phase estimation docstring

Summary
The previous Qiskit textbook URL referenced in the phase estimation docstring is no longer reachable. This commit updates the URL to point to the current location of the relevant textbook recourse.
Details and comments
Resolves #10535
No functional code changes; only the docstring was changed.
Old URL: https://qiskit.org/textbook/ch-algorithms/quantum-phase-estimation.html
New URL: https://learn.qiskit.org/course/ch-algorithms/quantum-phase-estimation

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
14 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.01%) to 85.905%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/parse.rs
6
97.6%

crates/qasm2/src/lex.rs
8
91.14%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
73073

Relevant Lines:
85063

💛  - Coveralls


Clarify return value of ReactDOM.render

changed stateless components to function components
added rule for built-in components

We're moving away from having a .render() return a value at all anyway, so I have to push back on your effort. Thank you for the PR!


does this work on win 11

Description
just asking
Reason
idk
Examples
No response

Yeah, it works, you could do a test yourself on a VM instead of going straight to your PC if you want to double-check or don't trust enough.


Remove SignalR

Unless I missed something, SignalR seems not to be used at all in this project. My guess is that the SignalR setup code was copy-pasted from some OWIN startup sample code and never removed.

There is an issue to start using it
On Thu, 31 May 2018, 11:21 pm Cédric Luthi<EMAIL_ADDRESS>wrote:

Unless I missed something, SignalR seems not to be used at all in this
project. My guess is that the SignalR setup code was copy-pasted from some
OWIN startup sample code and never removed.
You can view, comment on, or merge this pull request online at:
https://github.com/GlobalX/SbManager/pull/95
Commit Summary

Remove SignalR

File Changes

M src/SbManager/SbManager.csproj
https://github.com/GlobalX/SbManager/pull/95/files#diff-0 (7)
D src/SbManager/Scripts/jquery.signalR-2.0.3.min.js
https://github.com/GlobalX/SbManager/pull/95/files#diff-1 (8)
M src/SbManager/Startup/OwinStartup.cs
https://github.com/GlobalX/SbManager/pull/95/files#diff-2 (11)
M src/SbManager/packages.config
https://github.com/GlobalX/SbManager/pull/95/files#diff-3 (2)

Patch Links:

https://github.com/GlobalX/SbManager/pull/95.patch
https://github.com/GlobalX/SbManager/pull/95.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GlobalX/SbManager/pull/95, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA71rI-d-MDjrNjzZdwV6h96FoxO6Yq0ks5t3-5TgaJpZM4UVCLZ
.

Indeed, that’s #6. But in the meantime, having just a few lines of setup code adds nothing but confusion. (I spent a good amount of time wondering where SignalR was used only to realise that it was not!) When live updating with SignalR is implemented, re-adding the setup code won't be costly at all.


Fix deadlocks between write transactions and hotbackup, take 2

Make sure all follower trxs know that they are follower trxs.
Before this patch, follower transactions created from AQL or other
El Cheapo transactions did not know they are follower transactions.
This could cause deadlocks between write transactions and hotbackups.
Pull Request Guidelines
Pull requests are an essential collaborative tool for modern software development.
The below list is intended to help you figure out whether your code is ready to be reviewed
and merged into ArangoDB. The overarching goal is to:

Reduce the amount of recurring defects
Reduce the impact to the other developer’s time and energy spent on defects in your code
Increase the overall autonomy and productivity of individual developers

Acceptance Checklist
The below list is not exhaustive, think thoroughly whether the provided information is sufficient.
Remove options that do not apply
Scope & Purpose
(Can you describe what functional change your PR is trying to effect?)

[ ] Bug-Fix for devel-branch (i.e. no need for backports?)
[+] Bug-Fix for a released version (did you remember to port this to all relevant release branches?)
[ ] Strictly new functionality (i.e. a new feature / new option, no need for porting)
[+] The behavior in this PR can be (and was) manually tested (support / qa / customers can test it)
[ ] The behavior change can be verified via automatic tests

Related Information
(Please reference tickets / specification etc )

[ ] There is a GitHub Issue reported by a Community User:
[ ] There is an internal planning ticket:
[ ] There is a JIRA Ticket number (In case a customer was affected / involved):
[ ] There is a design document:

Testing & Verification
This should be put under automatic tests. These are not yet implemented
in this PR. Our test framework is not particularly well suited to test
this.
Test that should be done:
Create a one shard database.
Run this in arangosh:
c = db._create("c");
for (i = 0; i < 1000; ++i) { c.insert({Hallo:i}); }
for (i = 0; i < 1000; ++i) {
  db._query(`LET m = (FOR d IN c SORT d.Hallo DESC LIMIT 1 RETURN d)
             INSERT {Hallo: m[0].Hallo+1} INTO c RETURN NEW
             OPTIONS {exclusive:true}`,).toArray();
  require("internal").wait(0.01);
  print(i);
}

Constantly create hotbackups during the time. Hotbackups should succeed
quickly and the above loop should not be slowed down too much.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/12629/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/12645/

We will now go ahead with this without further integration tests, since a fixed version is desperately needed. We will add further integration tests when we port this to 3.7 and devel.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/12647/

Tests blue.


Abbilty to specify custom path for helm charts and tests

It will be nice to add the ability to specify custom path for the helm chrarts and test. This is needed to use plugin with more complex projects. The project we are going to use the pluging on a project that has a different structure.
Our team can fork the project and provide the code ourselves, if you aggree with such an approach.

@jekkel sorry if this is obvious, just a question. This is about where a chart should end-up, right? directory wise. Cause the source of a chart can be easely altered via :
tasks.withType<HelmBuildTask>().configureEach {
    source.set(file(<SOME_DIFF_DIRECTORY>))
}

@wind57 No, it is actually about the sources (although the final directory can be modified in a similar easy way with the property outputDirectory). And yes you're right it's already possible. But it is not possible to change the source folder of the test task, i.e. the folder which contains the test cases and test yaml files. The PR is going to add it and unifies handling a bit on top of that.


Design for authentication with image registries

Must take into account these scenarios:

[x] imagePullSecrets -- these still get used, either connected to the service account, or a workload
[x] ECR permissions which can be obtained from the environment (see flux v1's ECR adapter, but beware using the metadata service which needs other permissions)
[ ] GCR permissions, which I'll need to look up
[ ] Azure permissions -- have to check back, since those have changed (used to be possible to mount a docker config from the host, I think)
[ ] Possible fallback: refer to permissions mounted into the filesystem (but: not very secure, since everyone will be able to)

In the case of ECR and GCR, the permissions are connected to the account running the controller, so it is not possible to have multi-tenancy (ImageRepository objects that have different access to ECR, say) without some extra machinery. Supporting only single tenancy is OK for now.

NB imagePullSecrets for workloads will not in general be in the same namespace or even the same cluster as the ImageRepository objects.

When we are using metadata version, we need to aware of supporting metadata version( IMDSv1 and IMDSv2) . IMDSv1 is not having the security compliance so we need to use IMDSv2. I have already raised this issue in FluxV1 and waiting for the solution

https://github.com/fluxcd/flux/issues/3384
https://github.com/fluxcd/helm-operator/issues/574

so it is not possible to have multi-tenancy (ImageRepository objects that have different access to ECR, say) without some extra machinery.

I wonder if it is possible to take an approach similar to SOPS when role is specified for KMS encrypted secrets. When role is specified, SOPS will attempt to assume the role prior to decrypt.
For ECR I wonder if the ImageRepository CRD can be extended to take an assumable role ARN as a secret ref substitude, which it would then assume prior to image pull etc. This way, the multi-tenancy is respected by only allowing the image-reflector-controller to run with a service account that can assume other roles instead of a broader access to ECRs of all tenants.
I hope that makes sense.

There are solutions for ECR, GCR and Azure given in https://toolkit.fluxcd.io/guides/image-update/#imagerepository-cloud-providers-authentication. These rely on using CronJobs with appropriate credentials (and platform role assignments) to populate an image pull secret.
This approach is better than trying to build authentication into the image reflector controller, because it doesn't need additional, complicated machinery for multi-tenancy, and it generalises to other cloud platforms without needing further work here. The downside is that it's a little more work on the part of the end user -- but the bulk of setting it up is figuring out the platform configuration (IAM roles and whatnot), which you would have to do in either case.
Can anyone here comment on whether they've been able to set things up using the guide linked above?

@squaremo Could we please re-implement the authentication mechanism for ACR as done in Flux v1 by using the the Service Principal credentials available at the host path in /etc/kubernetes/azure.json for AKS clusters. This is the most common way to use the cluster service principal to authenticate with ACR.
I know there has been some issues open in Kubernetes to remove the option and use the default docker credentials location, but that work has been stale since January 2018 (kubernetes/kubernetes#58034). There are several issues like the --azure-container-registry-config flag that is dynamic and will require all container products to Azure to enable Managed Service Identity (MSI) before removing the flag from kubelet.
AAD Pod-Identity is in Preview and not enabled by default when creating AKS so that requires more work for the end user to enable / install.
So it seems like the /etc/kubernetes/azure.json will not go away anytime soon, and if the image-reflector-controller would implement this it will favor the end user for the ease of installation and usage (and migration from Flux v1).

ACR, GCR, ECR both static and IAM role auth have been implemented and documented.


feat: + python <3.12

Features

Fixbug: #1451. Modify python_requires=">=3.9, <3.12"

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 62.59%. Comparing base (ab846f6) to head (8d6e2e3).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1452   +/-   ##
=======================================
  Coverage   62.59%   62.59%
=======================================
  Files         287      287
  Lines       17589    17589
=======================================
  Hits        11009    11009
  Misses       6580     6580

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


Throwable Magical potions

Would require Vade Mecum knowledge to craft and would be crafted from the Voidic Alchemy Table. They would be elemental based such as on impact one creates acid and such.

Fire:
Ghast Tear
Nether Wart
Blaze Powder
Magma Cream
Void Fruit
Obsidian Flask
Frost:
Packed Ice
Snow Block
Ectoplasm
Void Crystal
Void Fruit
Obsidian Flask
Shock:
Sugar
Glistening Melon
Redstone Block
Glowstone
Void Fruit
Obsidian Flask
Acid:
Pufferfish
Slime Block
Fermented Spider Eye
Poisonous Potato
Void Fruit
Obsidian Flask
Void:
Void Crystal Block
Ectoplasm
Void Cloth
Void Chain
Void Fruit
Obsidian Flask

Taking from Above ^
(All recipes require the normal Ethereal Fruit and Obsidian Flask along with some form of their own research via ritual)

Fire Spread:

Research Requires Word: Flame
Upon impact it causes nearby blocks, at random, to be lit on fire.
Crafting:

Nether Wart
Blaze Powder
Magma Cream
Fire Charge
Obsidian Flask
Ethereal Fruit

Freeze:

Research Requires Word: Freeze
Upon impact it causes nearby entities are inflicted with a 10 second Slowness 5
Crafting:

Ice Block
Snow Block
Ectoplasm
Void Crystal
Obsidian Flask
Ethereal Fruit

Lifting:

Research Requires Word: Shock
Upon impact, all nearby entities are thrown up into the air
Crafting:

Glowstone
End Stone
End Rod
Feather
Obsidian Flask
Ethereal Fruit

Acid:

Research Requires Word: Acid Spray
Upon impact, liquid acid is spawned on top the block impacted
Crafting:

Pufferfish
Slime Block
Fermented Spider Eye
Poisonous Potato
Obsidian Flask
Ethereal Fruit

Acid:

Research Requires Word: Implosion
Upon impact, all nearby entities are inflicted with Implosion
Crafting:

Void Crystal Block
Hole in Reality
Void Cloth
Astral Essence
Obsidian Flask
Ethereal Fruit


Can't use the /app arguments on Fedora 40

So I installed a brain new Fedora 40 and played around with freerdp. Whatever I use xfreerpd wlfredrdp I can't use the /app arguments
I tried many way. I don't get a error about connection or anything like that. I get a error likes it's a parsing problem.
Step to reproduce:

Install Fedora40
installation freerdp
use the command with /u:oli /p:pass /v:$IP --> work
add the argument: /app:explorer.exe and see it displaying the help message.

check the argument help.
/app: program:[<path>|<||alias>],cmd:<command>,file:<filename>,guid:<guid>,
                                      Remote application program

fedora builds the application without backward compatibility support.

Can you give an example to make it work with notepad or explorer?
I tried different way to use this argument but I can't figure out how to use it.
/app: program: explorer.exe
Don't work neither

@olive007 /app:program:explorer.exe should work, if not check your windows is configured for remote app mode.


Enlarge surface button

Here's a simple PR for https://github.com/appleseedhq/appleseed-max/issues/232
I also took liberty to enlarge comboboxes to match sub-material button size.
Here's a preview

Looks great @jjeziorski!
I approve the change, it looks fine to me. Let's see what @dictoon says.

@dictoon I'm going to merge this PR. It's quite simple and done well.

Thanks @jjeziorski for the fix and @usakhelo for the review & merge!


DbLocalizationProvider.MigrationTool does not create SQL for empty resources

Hi,
I used the following command to create an SQL script from our XML resources:
packages\DbLocalizationProvider.MigrationTool.5.6.0\tools\DbLocalizationProvider.MigrationTool.exe -s MyWebApplication -e --resourceDirectory Lang -x
And a few weeks later I noticed that for resources, where in all languages there was no value defined, the tool did not create any entry. So we XML looks like this:

There is no entry created.
For resources where at least one language has a non-empty value, were exported.
For me it is not an issue anymore. I already migrated and only 3 resources were missing, I added them manually.
Thanks.

hi,
can you paste fragment from XML file once again?

Hi,
XML:
<myResource />


fix: correct behavior of urls and path options

Switches to using doc comments for the help strings of flags, makes the URLs argument required, and removes extra configuration for clap that is the default.

I've also now corrected the behavior of urls and path if I understand them correctly. You can pass as many URL arguments to the command to download, and if you want all the files to instead go in a specific directory you can add -p dist/ or whatever. Previously:

If I pass 1 URL followed by a directory path, the files are downloaded into the path of the directory specified. ✅
If I pass 2 URLs, I would get a No such file or directory (os error 2) error (I believe it is trying to write the first URL to the path of the second). ❌
If I passed more than 2 URL arguments, I would get error: Found argument '<some url>' which wasn't expected, or isn't valid in this context. ❌
Due to the above two items, passing two or more URL arguments followed by an output path argument doesn't work (only one URL followed by an output path argument works). ❌
Now:
If I pass 1 URL followed by a directory path, the files are downloaded into the path of the directory specified. ✅
If I pass 2 URLs, both files/directories are downloaded correctly. ✅
If I passed more than 2 URL arguments, all files/directories are downloaded correctly. ✅
If I pass any number of URLs followed by -p <path>, all files from the URLs before are downloaded into the specified path. ✅

Hi @uncenter,
Thanks for your effort in creating this PR!

If I pass 2 URLs, I would get a No such file or directory (os error 2) error (I believe it is trying to write the first URL to the path of the second, like the above list item). ❌
If I passed more than 2 URL arguments, I would get error: Found argument '' which wasn't expected, or isn't valid in this context. ❌

I believe there may be a misunderstanding about how multiple URLs are downloaded to a user-specified path. When there are two or more URLs, they're separated by commas. Downloading files/directories to custom paths was actually implemented in response to a feature request.
Here are some examples:

To clone into the current directory without creating a new directory of the same name (which is the default behavior):
cloneit <url> .

To clone multiple directories/files into the same directory or a custom directory:
cloneit <url1>,<url2> . or cloneit <url1>,<url2> custom_directory

In the case where <url1> is a file and <url2> is a directory, this command would download the contents of the directory and the file from <url1>, placing them all inside either the current directory (.) or the specified custom_directory.
This is the same way how git handles it, so using a path flag might not be a good idea. The only current issue is that when the target directory or file already exists, cloneit returns an error stating that the file/directory already exists but I think it is okay that way.

The only current issue is that when the target directory or file already exists, cloneit returns an error stating that the file/directory already exists but I think it is okay that way.

I think that is fixed by https://github.com/alok8bb/cloneit/pull/11

This is the same way how git handles it, so using a path flag might not be a good idea.

Are you referring to git clone? This comparison isn't right because Git only accepts a single repository URL in that command followed by a path.

I believe there may be a misunderstanding about how multiple URLs are downloaded to a user-specified path. When there are two or more URLs, they're separated by commas.

Hmmm that seems really counterintuitive.

Hmmm that seems really counterintuitive.

I forgot to put the examples in the README, but I think its fine or even better this way? 🤔

I'll leave that behavior as is for now, and maybe I'll open an issue later to try and convince you ahah. Reverted this PR to its prior state, I think making urls required though is still a good change but I can also split it into a separate PR if desired.

Thanks for your contribution!


Manage remotes

It would be nice to be able to add, remove and manage remotes from the branch view.

Would you like to have a own small interface or have a toggle option in branch interface? like toggle remotes.

I think toggling would be useful, especially if you could toggle some remotes to show and others not (use case: in my team we add each other's forks as remotes and one team mate doesn't prune his branches so the branch view for me is a list of 300+ branches)

ok, I think we can make a option to toggle all remotes. I may look at this tomorrow. No promises.

I've been thinking about it, and I don't want to add remote config to the branch dashboard.  There are a couple of reasons for that:

it will add significant clutter to the branch dashboard, in or to accommodate tasks that do not happen often
the way remotes are populated in the branch dashboard is distinct from what remotes are defined (for example, if you add a remote but haven't fetched yet, that remote will not appear in the branch dashboard).

Instead, I'd like to add palette commands: git: remote add and git: remote remove.  @wldcordeiro, will this solve the use cases you have in mind?

@divmain For managing remotes? Yes. I'd like to also get the toggle of displaying remotes to be on a remote by remote basis on the Branches dashboard.

We've been talking about it this in the Gitter channel.  I don't want to add per-remote visibility toggling to the Branches dashboard.  The UX would be subpar (given the constraints) and the implementation would add significant complexity to how the branches dashboard's render works now.
However, I'm trying to think through other options.  One might be to "forget" all the branches a particular remote.  For example, when you initially add a new remote, it doesn't know what refs/branches/tags are available at that remote.  Once you fetch, that information is populated.  I'm imagining something like the inverse of this process, to revert knowledge of that branch to the pre-fetch state.
This way you don't have to remove the remote will all the dead branches, and then re-add it when you happen to need it.  Instead, you would "forget" the refs, and simply fetch from that remote once you care about it again.
Does that seem reasonable?

We should be able to add both with (username and password) and ssh keys
This is the two different ways we can add them. Do any of you who about more?
[remote "stoivo"]
       url = https://github.com/stoivo/Packages.git
       fetch = +refs/heads/*:refs/remotes/stoivo/*
[remote "stoivo"]
        url = git@github.com:stoivo/Packages.git
        fetch = +refs/heads/*:refs/remotes/stoivo/*


Adding gateway services' templates and maestro config for RPi3

As the services' configurations are platform dependent and maestro
is tasked to generate configuration for all gw services, thus relocating
the template files from edge-utils to meta-pe/maestro recipe.

Manually build the image and verified the working on this PR.


Invite user giving the API error - new to this library so its basic usage question

Inputs in python code:
any API called as per readme file

Output error
Exception ignored in: <function Api.__del__ at 0x1051eb790>
Traceback (most recent call last):
  File "//user-path...//env/lib/python3.9/site-packages/appstoreconnect/api.py", line 72, in __del__
  File "//user-path...//env/lib/python3.9/site-packages/appstoreconnect/api.py", line 333, in _submit_stats
  File "//user-path...//env/lib/python3.9/site-packages/requests/api.py", line 115, in post
  File "//user-path...//env/lib/python3.9/site-packages/requests/api.py", line 59, in request
  File "//user-path...//env/lib/python3.9/site-packages/requests/sessions.py", line 587, in request
  File "//user-path...//env/lib/python3.9/site-packages/requests/sessions.py", line 701, in send
  File "//user-path...//env/lib/python3.9/site-packages/requests/adapters.py", line 489, in send
  File "//user-path...//env/lib/python3.9/site-packages/urllib3/connectionpool.py", line 703, in urlopen
  File "//user-path...//env/lib/python3.9/site-packages/urllib3/connectionpool.py", line 386, in _make_request
  File "//user-path...//env/lib/python3.9/site-packages/urllib3/connectionpool.py", line 1042, in _validate_conn
  File "//user-path...//env/lib/python3.9/site-packages/urllib3/connection.py", line 380, in connect
ModuleNotFoundError: import of time halted; None in sys.modules

I tried doing pip install time /  pip3 install time but the module is not found in the pip command.
% pip3 install time --upgrade
ERROR: Could not find a version that satisfies the requirement time (from versions: none)
ERROR: No matching distribution found for time
WARNING: There was an error checking the latest version of pip.

I tried doing pip install time

Time is a Python built-in package, you don't need to install it.
It looks like there is something wrong with your Python installation but I can't tell what.
From the the traceback I can tell the issue arise in the _submit_stats function, you can disable anonymous data collection  by initializing the with api = Api(key_id, path_to_key_file, issuer_id, submit_stats=False). It may help.

I'm having this issue right now

The exception occurs in _submit_stats() method. Just turn off usage stats submission:
api = Api(key_id, path_to_key_file, issuer_id, submit_stats=False)


Can path checks get removed?

I introduced the checks because I was not totally sure that it would be a good idea to allow unsanitized input that will be passed into exec. The value comes form a config file, though.
I am not entirely sure, consider this scenario:
A repository is using this plugin in a pipeline that runs on PR. A user inserts malicious code into the config file that prints out secrets. The pipeline will run with the changed config and the secrets are leaked.
I don't know if I am overly paranoid. Is the scenario realistic? How is this handled in other github actions?
In the end it would be nice if we could get rid of these checks because it is annoying to maintain them.

Originally posted by @simonseyock in https://github.com/terrestris/maven-semantic-release/pull/15#discussion_r1205431617

Just my 2 cents:
Please check the variables for sanity before passing them through to something which is will result in an executed command. I'd go so far to even check if the file is actually existing.
execa will most likely pass through what it is given and can hardly ever catch cases where someone tries to sneak in a malicious command.

I'd also leave the check in for now. I actually doubt that the issue of the regexp not being sufficient will pop up any time soon again.


Enhancements to admin generator error message

Fix a typo and automatically list supported renderers

Thanks, but please rewrite ext.prepend('-e ') piece without mutating the strings in the array using string interpolation for example.


Host don't delete from provider

Rancher versions: console 7/6
Steps to Reproduce:

Add a host
Delete host

Results:  The host does not get removed from the provider.  This happens in all three types of deletes:

If you delete host individually
If the host fails to fully setup and errors
If you remove the environment with hosts still there.

Version - 2.0 master 8/5
Verified fixed


disable col resize cursor on row marker col edge

@jassmith Oh wait it might be firstColAccessible ? 1 : 0. There's not a direct rowMarkerOffset passed as a prop, but if there's a row marker, it should be location[0] >= 1, else >= 0

I haven't tested yet, but I believe the way you have it is right from memory. If you had it the wrong way round I was just going to reverse it for you.


Fix cleanup for the TestClusterGroup e2e test

The test was not deleting the x, y and z Namespaces.
The implementation of the Cleanup function is also modified to avoid
early exit on error, so that all resources can get a chance to be
deleted. Instead, we log any error that may occur.

Codecov Report

Merging #2155 (03637b0) into main (64e017b) will decrease coverage by 19.80%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #2155       +/-   ##
===========================================
- Coverage   61.18%   41.37%   -19.81%
===========================================
  Files         270      132      -138
  Lines       20478    16531     -3947
===========================================
- Hits        12529     6840     -5689
- Misses       6649     9106     +2457
+ Partials     1300      585      -715

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
41.37% <ø> (+0.03%)
:arrow_up:

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


[Bug] [Master] too much log printing leads to insufficient disk space

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
When master use host-selector: lower_weight and worker is busy because of high CPU, master will print many error log like
fail to execute : %s due to no suitable worker, current task needs worker group %s to execute
What you expected to happen
the log will not print too fast
How to reproduce
master use host-selector: lower_weight  and run stress test.
Anything else
No response
Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I add the following configuration to logback-master.xml for temporary prevention of this problem...
            <maxHistory>36</maxHistory>
            <maxFileSize>200MB</maxFileSize>
            <totalSizeCap>5GB</totalSizeCap>
            <cleanHistoryOnStart>true</cleanHistoryOnStart>

I add the following configuration to logback-master.xml for temporary prevention of this problem...
            <maxHistory>36</maxHistory>
            <maxFileSize>200MB</maxFileSize>
            <totalSizeCap>5GB</totalSizeCap>
            <cleanHistoryOnStart>true</cleanHistoryOnStart>

Thanks for your reply, I will have a try.

remove stale


Use to bottom for linear-gradient

Fixes https://github.com/linkedin/hopscotch/pull/307

Any updates ?

Looks like this duplicates #323? I've merged that in, so if there's something I'm missing, let me know. Thanks!

Ok good ! Ok for me


Add NOTICE to comply with the CNCF rules

According to the CNCF rules to donate a repository:
If (a) contributor(s) have not signed the CLA and could not be reached,
a NOTICE file should be added referencing section 7 of the CLA with a
list of the developers who could not be reached.

Hello @nikhita, sorry for the late follow-up on https://github.com/kubernetes/org/issues/2182, I just took over the task to add the NOTICE file.
Can you please have a look at this PR and tell me if it aligns with what you had in mind?

Hello @nikhita, sorry for the late follow-up on https://github.com/kubernetes/org/issues/2182, I just took over the task to add the NOTICE file.
Can you please have a look at this PR and tell me if it aligns with what you had in mind?

thank you @dgrisonnet ! :heart:

thank you @dgrisonnet ! :heart:

Thanks @dgrisonnet!! :)
@swinslow @caniszczyk can you please verify if the NOTICE file looks good? This repo will be migrated as a part of https://github.com/kubernetes/org/issues/2182.
fyi @mrbobbytables @dims since this relates to github management/licensing

Thanks @dgrisonnet!! :)
@swinslow @caniszczyk can you please verify if the NOTICE file looks good? This repo will be migrated as a part of https://github.com/kubernetes/org/issues/2182.
fyi @mrbobbytables @dims since this relates to github management/licensing

@nikhita @dgrisonnet LGTM, thanks!

@nikhita @dgrisonnet LGTM, thanks!

fantastic, thank you, merging!

fantastic, thank you, merging!


Using transforms for classpath instrumentation makes the performance->configuration page fail in build scans with configure on demand

Current Behavior
When using an 8.6 nightly that includes https://github.com/gradle/gradle/pull/26045 and a composite build that uses an included build for build logic, then configure on demand causes the performance->configuration page to fail.

Expected Behavior
The performance->configuration page keeps working with Gradle 8.6 and configure on demand.

Context (optional)
The configuration page fails to load, since some project configuration happens in parallel instead of serially. I suppose this is due to the artifact transforms running on the included build causes its configuration, possibly via different paths.
Steps to Reproduce
Given the following build:
settings.gradle
rootProject.name = 'root-build'

include ':consumer'
include ':consumer2'

includeBuild file('included')

included/settings.gradle
rootProject.name = 'included'

consumer/build.gradle
buildscript {
    dependencies {
        classpath 'included:included:+'
    }
}

group = name
version = '1.0'
apply plugin: "java"

consumer2/build.gradle
// evaluated after 'consumer'
sleep 2000 // force a big evaluation time

group = name
version = '1.0'
apply plugin: "java"

included/build.gradle
group = name
version = '1.0'
apply plugin: "java"

Run gradle build --configure-on-demand --scan to observe the failure
Gradle version
8.6-20231208002054+0000
Build scan URL (optional)
https://ge.gradle.org/s/ww64cncnm5cf2/performance/configuration
Your Environment (optional)
No response

This is a consequence of the unfixed problem https://github.com/gradle/gradle/issues/27367. We'll do something on the Develocity side now, though the other issue should be fixed as well.


Fix up version in SparkRunner dependency on KafkaIO

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @amitsela
Two things:

Use ${project.version} for sure. I figure this didn't get bumped in the release.
(not SparkRunner related) put the version in the root pom, by analogy with other modules in the project.

LGTM.


sendChat Error OpenAI does not like message

Getting the following when submitting a "sendChat"
▿ OpenAIError
▿ chatError : 1 element
▿ error : Payload
- message : "Additional properties are not allowed ('id' was unexpected) - 'messages.0'"
- type : "invalid_request_error"
- param : nil
- code : nil

This looks to me like OpenAI is only allowing the "role" and "content" fields and is throwing an error when extra fields like "id" are submitted.

Did not mean to close this.


Parameter fontSize causes widget to crash

Parameter fontSize of MapBoxPlaceSearchWidget is a String but is used in a context where a double is expected.
This causes a crash of the widget if the parameter is used:
type 'String' is not a subtype of type 'double?'
When the exception was thrown, this was the stack:
#0      _MapBoxPlaceSearchWidgetState._searchInput (package:mapbox_search_flutter/src/map_box_place_search_widget.dart:160:37)
#1      _MapBoxPlaceSearchWidgetState.build (package:mapbox_search_flutter/src/map_box_place_search_widget.dart:110:18)
#2      StatefulElement.build (package:flutter/src/widgets/framework.dart:4792:27)
#3      ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:4675:15)
#4      StatefulElement.performRebuild (package:flutter/src/widgets/framework.dart:4847:11)
...

I'd happy to correct it myself (I already did it in my own project) if you give me access.


Adding username and ssh key

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

please test this

Can one of the admins verify this patch?

Can one of the admins verify this patch?

please test this.

@rahul1aggarwal I added you to the Jiocloud team. Now jiojenkins will not ask an admin to verify the patch. One needs to be first added to the Jiocloud team in order to trigger a build using "please test this".
@anshprat I've reviewed the modifications, looks fine to me. As prod isn't working, I recommend going ahead and merging the changes. Would you please review and merge?

@mayankkapoor Thanks

Was holding on to this hoping to fix the gates, but looks like it will take some more time. Merging it in the meanwhile. Have verified that the changes are only related to the respective key/user addition and does not impact any other code.

@anshprat Thanks


I want to create a video (.mp4) using images. But i don't know how can someone provide code examples?

Sorry i figured it out thanks anyway!

Can you share your solution?