com.raytheon.viz.gfe requires bundle com.raytheon.uf.common.plugin.nwsauth but it could not be found

Installation failed.
Cannot complete the install because one or more required items could not be found.
 Software being installed: GFE Feature <IP_ADDRESS>601211519 (com.raytheon.viz.gfe.feature.feature.group <IP_ADDRESS>601211519)
 Missing requirement: GFE Plug-in <IP_ADDRESS>601211519 (com.raytheon.viz.gfe <IP_ADDRESS>601211519) requires 'bundle com.raytheon.uf.common.plugin.nwsauth 1.14.0' but it could not be found
 Cannot satisfy dependency:
  From: GFE Feature <IP_ADDRESS>601211519 (com.raytheon.viz.gfe.feature.feature.group <IP_ADDRESS>601211519)
  To: com.raytheon.viz.gfe [<IP_ADDRESS>601211519]
Application failed, log file location: /awips2/cave/configuration/1453426560719.log
restoring backup STARTED: Thu Jan 21 18:36:05 MST 2016

Installing com.raytheon.uf.viz.d2d.gfe.feature.feature.group <IP_ADDRESS>601211519.
Installation failed.
Cannot complete the install because one or more required items could not be found.
 Software being installed: D2D GFE Feature <IP_ADDRESS>601211519 (com.raytheon.uf.viz.d2d.gfe.feature.feature.group <IP_ADDRESS>601211519)
 Missing requirement: D2D GFE Feature <IP_ADDRESS>601211519 (com.raytheon.uf.viz.d2d.gfe.feature.feature.group <IP_ADDRESS>601211519) requires 'com.raytheon.viz.gfe.feature.feature.group 0.0.0' but it could not be found
Application failed, log file location: /awips2/cave/configuration/1453426684247.log
restoring backup STARTED: Thu Jan 21 18:38:09 MST 2016
restoring backup COMPLETE: Thu Jan 21 18:38:09 MST 2016
warning: %post(awips2-cave-viz-d2d-gfe-15.1.1-13.x86_64) scriptlet failed, exit status 1
  Installing : awips2-cave-viz-d2d-ui-awips-15.1.1-13.x86_64

nwsauth removed from gfe deps.


faq

mesh路由错误日志
What is the purpose of the change
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Please fix checkstyle

Codecov Report

Merging #10542 (dad74a7) into 3.1 (eb919e9) will decrease coverage by 0.38%.
The diff coverage is 62.50%.

@@             Coverage Diff              @@
##                3.1   #10542      +/-   ##
============================================
- Coverage     64.76%   64.38%   -0.39%
+ Complexity      396      395       -1
============================================
  Files          1331     1331
  Lines         56520    56482      -38
  Branches       8370     8341      -29
============================================
- Hits          36606    36364     -242
- Misses        15970    16155     +185
- Partials       3944     3963      +19

Impacted Files
Coverage Δ

...rpc/cluster/router/mesh/route/MeshRuleManager.java
95.45% <50.00%> (ø)

.../rpc/cluster/router/mesh/route/MeshRuleRouter.java
94.23% <50.00%> (ø)

...cluster/router/mesh/route/MeshAppRuleListener.java
86.53% <75.00%> (ø)

...mmon/serialize/fastjson2/FastJson2ObjectInput.java
0.00% <0.00%> (-62.50%)
:arrow_down:

...mon/serialize/fastjson2/FastJson2ObjectOutput.java
0.00% <0.00%> (-56.25%)
:arrow_down:

...dubbo/remoting/zookeeper/ZookeeperTransporter.java
50.00% <0.00%> (-33.34%)
:arrow_down:

...on/serialize/fastjson2/FastJson2Serialization.java
57.14% <0.00%> (-28.58%)
:arrow_down:

...ng/zookeeper/curator5/Curator5ZookeeperClient.java
43.86% <0.00%> (-25.48%)
:arrow_down:

...nt/metadata/ServiceInstanceHostPortCustomizer.java
65.78% <0.00%> (-21.06%)
:arrow_down:

...n/serialize/fastjson2/Fastjson2CreatorManager.java
46.66% <0.00%> (-20.00%)
:arrow_down:

... and 64 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Refactor ovnkube CLI into manageable commands and options

As it stands today ovnkube(1M) CLI is a kitchen sink with distinct commands (init-master, init-node, remove-node, and so on) implemented as global options. It is not clear what options goes with what and so on.
Ideally, we would like to have the ovnkube(1m) CLI to have subcommands for every major feature it implements.
$ ./ovnkube -h
NAME:
   ovnkube - run ovnkube to start master, node, and gateway services

USAGE:
   ovnkube [global options] command [command options] [arguments...]

VERSION:
   0.3.0

COMMANDS:
     init-master     initialize ovnkube-master
     init-node       initialize ovnkube-node
     net-controller  start the central controller that watches pods/services/policies
     remove-node     remove a node from the OVN cluster
     help, h         Shows a list of commands or help for one command

GLOBAL OPTIONS:
   K8S-RELATED OPTIONS
   --cluster-subnet value
   --service-cluster-ip-range value
   --k8s-kubeconfig value
   --k8s-apiserver value
   --k8s-cacert value
   --k8s-token value
   --cni-conf-dir value
   --cni-plugin value
   --win-hnsnetwork-id value

   OVN NORTHBOUND DB OPTIONS
   --nb-address value
   --nb-server-privkey value
   --nb-server-cert value
   --nb-server-cacert value

   <snipped>

$ ./ovnkube init-node -h
NAME:
   ovnkube init-node - initialize ovnkube-node

USAGE:
   ovnkube init-node [command options] [arguments...]

OPTIONS:
   --init-gateways
   --gateway-interface value
   --gateway-nexthop value
   --gateway-spare-interface value
   --gateway-localnet
   --nodeport

I am working on getting the CLI to be more streamlined.

I am going to submit a PR soon which fixes this issue.


feat(dispatcher): initial dependency injection

actual implementation will likely be a bit different, i just need to fiddle with the types in the canary build for now


init

Hello, Maintainers
I'm Lakshya, Learining Web dev and this is my first commit

@LakshyaKhilwani Welcome! good job :)

i merged your code into the main branch.


Bentham calculus encourages agents to move to low value cells

When experimenting with Bentham agents with infinite vision/movement, I noticed that many of them were moving to the very edges of the sugarscape rather than the center as expected. Here's a visualization of calculated ethical scores for the first agent to move in one of my sims:

I've normalized scores from red (worst) to green (best). It's clear that agents are being encouraged to move to cells with lower ethical scores. Here's a rough example for why I think this is happening:
value of cell is 0.5
10 neighbors
ethical score = 0.5 - 10(0.5) = -4.5
value of cell is 0.25
10 neighbors
ethical score = 0.25 - 10(0.25) = -2.25
The values aren't realistic but display the issue. Another piece of evidence contributing to this is that sims with only one agent seem to work correctly because there are no opportunity costs to subtract:

@nkremerh What are your thoughts on this?

This makes sense when agents can see the entire map. They will go to cells where they inconvenience nobody. When constrained to a far more local neighborhood, they don't have the opportunity to escape to the far-flung corners of the world. Instead, they have to determine the best local cell with the knowledge that they will cause some harm (at the moment, agents feel no remorse for this, but that could be an interesting topic for future study).


Proposed fix for CSS width of popup toolbar in #52

This would need compiling to be completed if approved - i'd love help if possible! Thanks!

thanks for sending this bugfix, Jeff! Let me know if you need me to do a new npm release with the fix included.

Hi thanks! I just don't have the full lib set up locally so I didn't do the
build process before submitting this, I think it could use a "npm build" or
whatever the minifying command is to reflect this into the min version of
the stylesheet. I'm happy to do it now that you approved it, but if it's
easier for you before releasing a new npm version I'd also appreciate the
help!
On Thu, Feb 21, 2019, 8:44 AM Justin Manley<EMAIL_ADDRESS>wrote:

thanks for sending this bugfix, Jeff! Let me know if you need me to do a
new npm release with the fix included.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Leaflet/Leaflet.toolbar/pull/53#issuecomment-466072617,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJ3ABZMZ_MfQEqhZ-2Koy-wu5-fwbks5vPsy1gaJpZM4bGD7P
.

yup, I already recompiled the CSS (see the master branch). Just lmk if you need a new npm release in order to integrate this into your workflow!

Oh awesome. Didn't see it in the PR so wanted to double check. Yes, a new
release would be helpful!
On Thu, Feb 21, 2019, 9:31 AM Justin Manley<EMAIL_ADDRESS>wrote:

yup, I already recompiled the CSS (see the master branch). Just lmk if you
need a new npm release in order to integrate this into your workflow!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Leaflet/Leaflet.toolbar/pull/53#issuecomment-466090689,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJ3apdpsZYJfuFKbLAXL4xZaEVu6uks5vPtfQgaJpZM4bGD7P
.

done! the latest version is now "0.4.0-alpha.2."
https://www.npmjs.com/package/leaflet-toolbar

awesome, thank you!
On Fri, Feb 22, 2019 at 2:08 AM Justin Manley<EMAIL_ADDRESS>wrote:

done! the latest version is now "0.4.0-alpha.2."
https://www.npmjs.com/package/leaflet-toolbar
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Leaflet/Leaflet.toolbar/pull/53#issuecomment-466296516,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABfJ8clPsyJuqq3YBcHGSlJAhgOHqOzks5vP5dggaJpZM4bGD7P
.


[SUPPORT] Cow non-partition table upsert action effect when  preCombineField field value change to Smaller (SparkSQL)

Describe the problem you faced
table description:  a cow non-partition table (tblproperties with primaryKey and preCombineField) .
sql environment: spark sql
describe:
1.  if I action the insert or update action with same primary key  and preCombineField field value change to smaller, Unexpectedly the action effect (note: preCombineField  field value change smaller should not effect, but actual this is effect);
2. if I action the merge into action  with same primary key  and preCombineField field value change to smaller yet, it action not effect expectedly( note: I think this is normal);
the 1 and 2 abover upsert action seem to  conflict，Strangely enough.
To Reproduce
Steps to reproduce the behavior with SPARKSQL:
--step 1: create table
drop table hudi_cow_pk_cbfield_tbl;
create table hudi_cow_pk_cbfield_tbl (
id bigint,
name string,
ts bigint
) using hudi
tblproperties (
type = 'cow',
primaryKey = 'id',
preCombineField = 'ts',
hoodie.datasource.write.hive_style_partitioning = false
)
;
--step 2: insert into a recored with primaryKey=1, preCombineField=1000
insert into hudi_cow_pk_cbfield_tbl select 1, 'a0', 1000;
--step 3: 'insert' action with same primaryKey, but change the preCombineField value to the smaller value 100, Unexpectedly the action effect (note:is not normal)
insert into hudi_cow_pk_cbfield_tbl select 1, 'a0_new', 100;
select * from hudi_cow_pk_cbfield_tbl;
--step 4: 'update' action with same primaryKey, but change the preCombineField value to the smaller value 20, Unexpectedly the action effect yet (note:is not normal)
update hudi_cow_pk_cbfield_tbl set name='a1_new',ts=20  where id= 1;
select * from hudi_cow_pk_cbfield_tbl;
--step 5: 'merge into' action with same primaryKey, but change the preCombineField value to the smaller value 10,  the action effect(note: I think this is normal)
merge into hudi_cow_pk_cbfield_tbl as target
using (select 1 as id,'a1_merge' as name,10 as ts) as source
on target.id = source.id
when matched then update set *
when not matched then insert *
;
select * from hudi_cow_pk_cbfield_tbl;
Expected behavior
if the preCombineField value chagne to smaller, the  row data upsert action should not effect( i experiment in the cow-partition table, mor-table, it's normal)
Environment Description

Hudi version :0.11.1

Spark version :3.2.1

Hive version :3.1.0

Hadoop version :3.1.1

Storage (HDFS/S3/GCS..) :HDFS

Running on Docker? (yes/no) :no

cow use OverwriteWithLatestAvroPayload for default, but mor use DefaultHoodieRecordPayload.
hoodie.datasource.write.payload.class = 'org.apache.hudi.common.model.DefaultHoodieRecordPayload' in cow can achieve the effect you want

@KnightChess , i try it, it's effect. thank you~


Documentation for classParamIndentSize in indentation checker

classParamIndentSize was implemented in #153 and #154
Avoid confusion with tabSize parameter default value.
It wasn't clear why indentation=2 does not work with multiline case classes without looking into scalastyle code/tests.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@129957e). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #233   +/-   ##
=========================================
  Coverage          ?   87.17%
=========================================
  Files             ?       56
  Lines             ?     1263
  Branches          ?      113
=========================================
  Hits              ?     1101
  Misses            ?      162
  Partials          ?        0

Continue to review full report at Codecov.

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

Thanks!


Color conversion ergo

Hi! Thank you for helping out with Embedded Graphics development! Please:

[x] Check that you've added passing tests and documentation
[x] Add a simulator example(s) where applicable
[x] Add a CHANGELOG.md entry in the Unreleased section under the appropriate heading (Added, Fixed, etc) and appropriate crate (embedded-graphics, simulator, tinytga, tinybmp) if your changes affect the public API
[x] Run rustfmt on the project
[x] Run ./build.sh (Linux/macOS only) and make sure it passes. If you use Windows, check that CI passes once you've opened the PR.

PR description
I haven't thought about this too much, so if there's a reason not to add it let me know. At any rate, @almindor on Matrix was asking about making colour conversions easier.
Adds .into_raw() for PixelColor. Allows easier conversion from a colour type to its raw value. This is mostly sugar for the RawU1::from(color).into_inner() dance which is a bit of a pain. It can now be replaced by color.into_raw().
I'm not keen on the name into_raw(). Better ideas welcome! into_storage()? into_inner() like the Raw colours?

Looks good, into_raw is a bit funny name but I can't come up with anything that sounds any better. into_native comes to mind but that has other conotations, same for into_primitive...

I haven't thought about this too much, so if there's a reason not to add it let me know.

I'm not 100% convinced that this is necessary. Sure the long form isn't very user friendly, but conversion between colors and raw data isn't a very common operation either, at least in my experience. But adding this shouldn't harm anything and I'm totally OK with adding it, if you and @almindor think it's useful.
But I would like to change the way this is implemented: By adding an additional method to the PixelColor trait each implementation has to deal with this separately, even if the color can't be converted to RawData at all. By using an extension trait this function could be implemented for all PixelColors that can be converted to RawUx, with the additional benefit of not breaking the API:
pub trait PixelColorExt {
    type Storage;
    fn into_raw(self) -> Self::Storage;
}

impl<C> PixelColorExt for C
where
    C: PixelColor,
    C::Raw: From<C>,
{
    type Storage = <<C as PixelColor>::Raw as RawData>::Storage;

    fn into_raw(self) -> Self::Storage {
        C::Raw::from(self).into_inner()
    }
}

And into_raw should really get another name, because it doesn't convert into PixelColor::Raw.

Shouldn't it be possible to implement standard From for PixelColor<Each> in the macro? So you'd just do u16::from(color) in the end.
As for use cases this is pretty much required in drivers. It's a minor issue, just ergonomics really.

Yeah agreed, was hoping we could bikeshed some alternatives in this thread. into_storage might work? Although I'm not sure how much e-g internals one needs to know about to understand that storage means a u16 or whatever. The doc example on the trait helps though tbf.

into_storage isn't ideal, but I can't come up with something better. Should we also rename PixelColorExt to into IntoStorage then, because it only contains methods and types related to storage?
pub trait IntoStorage {
    type Storage;
    fn into_storage(self) -> Self::Storage;
}

This would leave the PixelColorExt name free for further extensions that don't require the C::Raw: From<C> trait bound.

Do you think we should also simplify the opposite conversion from storage to PixelColor?

Should we also rename PixelColorExt to into IntoStorage then

Yeah I like that. I've pushed a new commit with the renamed trait/method.

Do you think we should also simplify the opposite conversion from storage to PixelColor?

Maybe at some point, but the point of this PR was to address the currently-used code in various driver crates that converts PixelColors to their raw counterparts. Let's leave it for a future PR if we need it.

Maybe at some point, but the point of this PR was to address the currently-used code in various driver crates that converts PixelColors to their raw counterparts. Let's leave it for a future PR if we need it.

If we needed to implement this, IMO the opposite should also be implemented to provide some symmetry for the API. The opposite conversion is used in image crates like the tiny... crates (in a slightly different form because of the more generic u32 conversion). But I agree that this doesn't belong in this PR.

Thanks for the review!


Consider using webpack-merge to manipulate config

Webpack has a great tool for merging objects and arrays, respecting webpack config structure. Maybe we should use it, as it's an official, more maintainable way to manipulate the data we need.
https://github.com/survivejs/webpack-merge

Thanks for the suggestion! When I have time I'll take a look and see what it gives us and if it's worth the effort :)


[BUG] cant open bloxstrap [SOLVED]

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[x] My answer given in the checkbox above is a lie.

What problem did you encounter?
when im open bloxstrap , it got this error "System.UnauthorizedAccessException: Access to the path 'D:\Program Files\Bloxstrap\Logs' is denied."

i just reinstall and is solved


Specify ip version?

With v3:
redis.createClient({
  url: myUrl,
  family: 'IPv6',
})

I looked around and couldn't figure out how to specify the ip version. Is this no longer possible?

options.socket is passed to net.createConnection, which will pass it forward to new Socket and socket.connect, so basiclly you can set any option you want on the socket by including it in the socket object:
redis.createClient({
  url: 'redis://<IP_ADDRESS>:6379',
  socket: {
    family: 6
  }
});

redis.createClient({
  socket: {
    ip: '<IP_ADDRESS>',
    port: 6379,
    family: 6
  }
});

I'll close this issue once I'll update the docs & types.

Thanks!
The TypeScript types aren't too happy with that unfortunately. Am I doing something wrong?

@kentcdodds you are doing it right, the RedisSocketOptions type is incorrect... it was already fixed in e81bf64and it'll be released in<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

Do you have a release schedule published anywhere?


Automated syntax correction for solidity compiler in vs code

Feature
Is your feature request related to a problem? Please describe.

Because the VSCode solidity dev environment doesn't catch syntax and similar errors while you're coding, its pretty usual for me at least to make a bunch of typos,, missing ,s unbalanced brackets etc.  Hardhat requires a  a "compile" step which is quite tedious, because it only finds a few errors at a time, these need to be corrected, before you will get a new batch of also trivial errors.
So when you're done with the writing and run "hh compile" to find all the errors, you never get more than a few in a single pass, and you need to correct these and then run hh compile again which finds a new (but not complete) list of obvious errors.

Describe the solution you'd like

It would be super great if copilot could do the job that VS Code doesn't do and mark obvious syntax errors, or have some kind of syntax check function that would reveal all the errors and make suggestions for fixes, which you could just accept.  For me at least, 90% of the error corrections are trivial (missing closing brackets, missing keywords, etc.) all things which I think Copilot already recognizes, at least when its suggesting code.  Would save me a lot of time at least until the VSCode dev environment gets as good for solidity as it is for Typescript and simialr languages, where the errors are evident as soon as you type them.
Existing work

Errors of this syntax and typeos when writing pretty much any other strongly typed language Java or Typescript are marked in the UI with red squiggly's.  But for some reason, not in solidity.  I suspect has something to do with the the solidity compiler and the way integrated into VS Code.
Describe alternatives you've considered

The alternative is just running 'hh compile' about 15 times until you've got all the errors identified.
Additional context

@grapevinegizmos This issue does not seem related to the Fluid Framework. Can you clarify how Fluid is involved?

Apologies Tyler I thought I was posting in the Github Copilot suggestion box.


add fingerprint script to pip caching

Description
Enforce fingerprint of pip caching, this is pretty much last resort to force cache update, I was reluctant to go this path because it could potentially lead to a refresh of all caches, but it is a more appropriate way of setting up ci to recognize caching opportunity. (According to cirrus ci doc: By default the task name is used as a fingerprint value.)
Type of change

New feature (non-breaking change which adds functionality)

References
https://github.com/napari/napari/issues/1527
https://cirrus-ci.org/guide/writing-tasks/#cache-instruction
How has this been tested?

all tests pass with my change

Final checklist:

[X] My PR is the minimum possible work for the desired functionality
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] I have added tests that prove my fix is effective or that my feature works

cc @fkorotkov
@ziyangczi, @fkorotkov has been very helpful in the past with cirrus stuff...  if you have any specific how-to questions, I'm sure he can answer them

On the second glance it seems requirements/test.txt doesn't contain specific version of the dependencies which means you'll only invalidate the cache if a dependency will be added or removed. 🤔

Thanks @fkorotkov!  The issue we're trying to solve is here: https://cirrus-ci.com/task/5094878520541184?command=install#L65
we can certainly pin some of our test dependencies to force cache expiration... but looking at that error, do you think the issue is that the cirrus cache is problematic and just needs to be refreshed?  or that we actually have some issues in our requirements files that need fixing?

@tlambert03 this might be because you do pip install once again even for a cache hit. 🤔 I wonder if you need the extra pip install at all in case you already have it in populate_script. 🤷♂

That seems reasonable.  I think there was confusion as to whether the pip_cache was caching downloaded (but uninstalled) pip wheels (in which case we still needed to install after a cache hit), or whether it was actually caching the fully installed environment (in which case we just move on to testing)

@sofroniewn shall we merge this one? I hope this will address it once and for all
we may want to update test dependency file to include version later if that becomes an issue too

@sofroniewn shall we merge this one? I hope this will address it once and for all
we may want to update test dependency file to include version later if that becomes an issue too

Yes, happy to approve and then merge. @ziyangczi will these changes be reverted after the problem is fixed, like the last approach, or are they meant to be permanent?

these ones meant to be permanent if it fixed the problem, fingers crossed

It passed!!

thanks for preserving here @ziyangczi!!


Unable to download from download url

I'm unable to reach the url HTTP://server1.xrdp.org/xrdp/X11R7.6 due to our corporate firewall.
Using the wayback machine i see looks like a folder of tar.bz2 files.  Are these files specifically this project or are they source for 3rd party libraries?
I'm able to get Github no issues.  So if it's just a matter of finding the sources/binaries for each file on another site I should be good to go.

Sources put there are for x11rdp the one of backend.  xrdp works together with backend X server.
You can choose either x11rdp or xorgxrdp. But I recommend to use xorgxrdp. It almost replaces x11rdp.
https://github.com/neutrinolabs/xorgxrdp
If you want to stay on x11rdp, here's alternative mirror for x11rdp sources.
https://xrdp.vmeta.jp/pub/xrdp/X11R7.6/

I was able to successfully build using xorgrdp

Those files are not specific to X11rdp. It should be possible to commit those files to the xrdp github repository, but that would increase the clone time and the amount of storage even for the users who don't need X1rdp.
The current .git directory (i.e. all objects that are downloaded when the repository is cloned) is 19 megabytes. The tarballs would add 56 megabytes.
It should be possible to split X11rdp into a separate repository and add the tarballs to that repository. But I'm not sure it's worth the trouble. xorgxrdp is a viable alternative to X11rdp that is much easier and faster to compile, and that is approaching X11rdp in features.

Umm… prebuilt tarballs are never, ever going into a source repository?

Right, that would have to be unpacked tarballs. One thing is certain, we should not do in with the xrdp repository.

As proski says, adding source tarballs which x11rdp depends on to the xrdp repository is not reasonable.


Add Binary Search Tree

Algorithm:
Binary Search Tree
DS:
Trees
Languages Supported:

C [ Assigned to @jivthesh ]
C++ [ Assigned to @jivthesh ]
C#
Java [ Assigned to @sd2001 ]
Javascript
Python [ Assigned to @sd2001 ]

Please can I start working on the binary search tree using Java and Python?

I have assigned you the Issue. You can start working on it @sd2001 💥
Please follow the general guidelines:

Make a Branch off your Master and work on that.
Make a proper README of how your Script works and what all libraries and modules you have used.
Add Proper Comments and focus on Clean Code, in practice.
Also, reference this Issue in your PR so that it becomes easy for us to review.

can I start working on the binary search tree using c and c++ ?

and python

You can work on it in C and C++ @jivthesh 🙌

@HarshCasper Can I start working on BInary Search Tree in Java with all inorder preorder and post order traversals

@HarshCasper Can I start working on BInary Search Tree in Java with all inorder preorder and post order traversals

I would like to implement both recursive and iterative methods for all the functions


Add support for connection tracking.

This series adds support for sending packets through a connection tracker,
which allows OVS to perform stateful firewalling functions. The functionality
added in this series works in conjunction with the interface that is available
in the latest upstream Linux kernel development releases that will become
Linux-4.3. The linux datapath backport is not included at this time.
The functionality is manipulated through a new action "CT" and several new NXM
fields: ct_state, ct_zone, ct_mark, ct_label. The CT action allows these fields
to be populated, and for connections that match the flow to be tracked. Later
patches in the series also allow metadata to be attached to these connections.
When a flow is sent through the connection tracker, there are two common
functions to perform. Firstly, match packets from port 1 and do a lookup:
ovs-ofctl add-flow br0 "in_port=1,actions=ct(table=1)"

When the "table" table is specified, the ct action performs similarly to an
output action, in that a copy of the packet is "output" to the connection
tracker. When the connection tracker has finished processing the packet,
processing will continue in the table specified. The connection tracking NXM
fields will be populated when the processing continues, which allows fields
such as the state to be matched on.
The connection state, as represented in the ct_state field, consists of a
collection of flags, including:

Tracked (trk): Connection tracking has occurred.
Reply (rpl): The flow is in the reply direction.
Invalid (inv): The connection tracker couldn't identify the connection.
New (new): This is the beginning of a new connection.
Established (est): This is part of an already existing connection.
Related (rel): This connection is related to an existing connection.

When the first packets for a new connection are sent through the connection
tracker, the ct_state will have the "+trk+new" flags set. The OpenFlow
controller may specify a policy to match new connections and allow or deny
them.
The second function of the ct action is to "commit" the connections. This
signals to the connection tracker that this connection should be tracked on an
ongoing basis, so that subsequent packets may be identified as belonging to
this connection. For instance, to allow new connections from port 1->2:
ovs-ofctl add-flow br0 \
    "in_port=1,ip,conn_state=+trk+new,action=ct(commit),2"

Later packets in the connection will have the "+est" flag set, so existing
connections may be allowed as so:
ovs-ofctl add-flow br0 "in_port=1,ip,conn_state=+trk+est,action=2"

In addition to the above, several other parameters may be provided to the ct
action:

zone: A 16-bit value or NXM field to retrieve the zone from. Each zone is an
independent connection tracking context. Connections which are committed to
zone A will not be remembered in zone B, unless the connection is also
explicitly committed to zone B.
eg: actions=ct(zone=1),ct(zone=NXM_NX_REG0[0..15])

exec: Execute a nested set of actions. This allows additional functions to be
performed as part of the connection tracking execution. In this series, the
set_field, reg_move and reg_load actions are supported with two connection
tracking metadata fields: ct_mark and ct_label.
eg: actions=ct(commit,exec(set_field:1->ct_mark))

alg: Specify an ALG to assist connection tracking. Some protocols consist of
multiple traffic streams that are impossible to associate without additional
context. This parameter provides that context to the connection tracker to
make it possible to track, for instance, FTP data connections.
eg: actions=ct(commit,alg=ftp)

Further examples are available in the commit messages for each patch, the
ovs-ofctl(8) man pages, and the traffic testsuite in tests/system-traffic.at.

v4 of the patch series is under review on the mailinglist now:
http://openvswitch.org/pipermail/dev/2015-October/060863.html


Slow vim paste (showing character 'P' temporarily)

Steps to reproduce

Open a file with at least one word inside.
Put cursor under the first letter in the word
Now, do this sequence quickly - Change word (cw) , exit insert mode (ESC) and paste (P).
Observe that paste is slightly slow and 'P' shows up for a split second before it disappears and the word is actually pasted.

Expected behaviour
I expect the paste that follows this sequence to be immediate and to not see the 'P' being displayed for a split second.
The glitch does not occur if word is deleted instead of changed (ie, use dw instead of cw).
Version of Vim
VIM - Vi IMproved 9.0 (2022 Jun 28, compiled Jul  2 2023 20:24:33) macOS version - arm64 Included patches: 1-1544
Environment
macOS
TERM = xterm-256color
shell = bash
Note: I tried to load vim without .vimrc using vim -N -u NONE test and the issue persists. Changing terminal emulator from Terminal.app to kitty also has no effect.

Thanks for the explanation @lacygoill .
The issue does go away by reducing ttimeoutlen.
I'm closing the issue.


Fixes #81.

Write tests to check that parent blocks are correctly passed into the reporter.
Otherwise :+1:


Create a looseInstanceOfClass to check FileBox and Puppet instances

When we are using the instanceof operator to check whether an instance is a Class, the instance must be exactly created by that Class, or instanceof will return false.
For example, if we created a fileBox with<EMAIL_ADDRESS>then we want to know whether the fileBox is the instance of the module file-box, but with the different version, say<EMAIL_ADDRESS>then the instanceof will return false.
Sometimes we just want to know whether the instance is from this module, we do not care about the version. That's the reason why we want to introduce the looseInstanceOf method.l
looseInstanceOfClass
For given instance and Class, This method only does two checks:

if instance instanceof Class is true, then return true
if instance.constructor.name is the same as the Class.name, then return true
else return false

Abstract Class & Private Constructor
If a class is an abstract class or the constructor is private, then we will not be able to infer the { new (...args: any): T } as expected.
The workaround is:
const looseInstanceOfFileBox = looseInstanceOfClass(FileBox as any as FileBox & { new (...args: any): FileBox })
const looseInstanceOfPuppet = looseInstanceOfClass(Puppet as any as Puppet & { new (...args: any): Puppet })

That's it!
We will use it to check the FileBox and Puppet in Wechaty.
Resolve: #1930

Should be available after Wechaty v0.51.4

move looseInstanceOfPuppet & looseInstancOfFileBox to wechaty-puppet npm module.


Remove pry-git dependency

Hasn't been updated in 4 years.
I'm not sure if this should be a 3.0.0, but it might be according to semver point 8:

Major version X (X.y.z | X > 0) MUST be incremented if any backwards incompatible changes are introduced to the public API. It MAY include minor and patch level changes. Patch and minor version MUST be reset to 0 when major version is incremented.

But I can change that.

I agree with you here. My only concern is to not mix code with version bump. Could you remove it so i can add a comment on top of it just to bump version?

Sure, I just left my PC, so I'll do it tomorrow.
On Mon, Sep 14, 2015, 19:01 Paulo Henrique Lopes Ribeiro <
<EMAIL_ADDRESS>wrote:

I agree with you here. My only concern is to not mix code with version
bump. Could you remove it so i can add a comment on top of it just to bump
version?
—
Reply to this email directly or view it on GitHub
https://github.com/plribeiro3000/jazz_fingers/pull/12#issuecomment-140144539
.

Done.


Record an event when provisioning request fails

What type of PR is this?

/kind feature
What this PR does / why we need it:
It produces an event when an error during provisioning request creation occurs.
Which issue(s) this PR fixes:

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

NONE

/test all

/retest-required

/assign @mbobrovskyi

/lgtm
Thanks!

/lgtm
/approve
Thanks!

/release-note-edit
MultiKueue: Record the ProvisioningRequest creation errors to event and ProvisioningRequest status.

I think we should drop the "MultiKueue: " - this is not used in MultiKueue. In fact, currently ProvisioningRequests and MultiKueue aren't working together.

I think we should drop the "MultiKueue: " - this is not used in MultiKueue. In fact, currently ProvisioningRequests and MultiKueue aren't working together.

Oh, you're right. Let me revise the release-note.

/release-note-edit
ProvisioningRequest: Record the ProvisioningRequest creation errors to event and ProvisioningRequest status.

/retitle Record an event and set status message when creation of provisioning request fails


Generic type support in Workflow compiler

For Scalar type Generic, the compiler was missing support. Added this support.

You might want to double check the logic for how bindings are constructed with Generic values too. The reason I say this is because:

How we implemented a certain legacy type was with a Generic. And it works fine because it is never provided as a workflow input or as a static value. It always go from one task to another and works fine.

In our prototype I noticed the failure both when it was a Generic input to a workflow and when I statically set it on a task.

You might want to double check the logic for how bindings are constructed with Generic values too. The reason I say this is because:

How we implemented a certain legacy type was with a Generic. And it works fine because it is never provided as a workflow input or as a static value. It always go from one task to another and works fine.
In our prototype I noticed the failure both when it was a Generic input to a workflow and when I statically set it on a task.

Do you mean to check the runtime behavior? I did, Scalars are pretty simply copied over, by value


New package: TropicalNumbers v0.1.0

Registering package: TropicalNumbers
Repository: https://github.com/GiggleLiu/TropicalNumbers.jl
Created by: @GiggleLiu
Version: v0.1.0
Commit: 91c34a20e575ba15c2c46bf02f9035ebe93356c5
Reviewed by: @GiggleLiu
Reference: https://github.com/GiggleLiu/TropicalNumbers.jl/issues/1

There are two related packages SimpleTropical.jl and TropicalSemiring.jl. However, the tropical numbers in these packages contains an unnessesary extra field isinf. Which makes the number hard to fit the BLAS implementation for tropical numbers:
https://github.com/YingboMa/MaBLAS.jl/tree/myb/tropical

@GiggleLiu do you want to have this merged?  I think your comment blocked automerge

Yes, thanks @giordano


PowerShell script to upload libraries

Hi All, Does anyone have power shell script to upload libraries to the orchestrator?

Looking for the same. I know this is a old post. I use Jenkins and the uipath plugin to upload packages but haven't been able to figure out the best way to upload libraries when needed through automation.


[SPARK-38237][SQL] Rename back StatefulOpClusteredDistribution to HashClusteredDistribution

What changes were proposed in this pull request?
This PR proposes to rename back StatefulOpClusteredDistribution to HashClusteredDistribution. This PR retains the content of the classdoc for stateful operators in HashClusteredDistribution, along with new general content of the classdoc.
Why are the changes needed?
We figured out that in some case HashClusteredDistribution is still desirable other than stateful operators; HashPartitioning with subset of grouping keys can satisfy ClusteredDistribution, which means the cardinality of the subset of grouping keys technically defines the max parallelism. Increasing the number of partitions does not always help to solve the skew of the partitions.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing tests since this PR just renames a class.

Sigh, I pushed to the wrong repo... I'll push to my repo and submit a PR again. Sorry for the noise.

@HeartSaVioR Could you remove the branch https://github.com/apache/spark/tree/SPARK-38237 ?

Thanks for reminder! I just remove the branch.


UP-4677: Add ability to read/write the dynamic skin css files from/to…

… an AWS S3 bucket.

Changes Unknown when pulling 01807d925c7c3f67ca39cf2d1fd072e7f1859bb9 on groybal:UP-4677 into ** on Jasig:master**.

The changes here are all to the dynamic-skin portlet and the AWS stuff is optional and disabled by default.   This code has been in-use with CCC for a while now.  If anyone has any comments or suggestions, please get them in soon.   Barring downvotes, I will mostly likely try to get this merged soon.

Closing this PR.   See:  https://github.com/Jasig/uPortal/pull/735 instead


Added sAmountNumeric and sAmountNetNumeric to sOrder Mail

Description
Please describe your pull request:

What does it improve?
Makes easier to calculate with $sAmount or $sAmountNet in sOrder
Does it have side effects?
Nope

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?

How to test?
Make a order and show variables

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/#/issues/SW-17479.
Please use this issue to track the state of your pull request.


Discogs crashes beets 1.3.11

beets version 1.3.11
plugins: fromfilename, duplicates, lastgenre, mbsync, lastimport, fetchart, ftintitle, discogs
Python 2.7.9
OSX 10.10

Traceback (most recent call last):
  File "/usr/local/bin/beet", line 9, in <module>
    load_entry_point('beets==1.3.11', 'console_scripts', 'beet')()
  File "/usr/local/lib/python2.7/site-packages/beets/ui/__init__.py", line 953, in main
    _raw_main(args)
  File "/usr/local/lib/python2.7/site-packages/beets/ui/__init__.py", line 943, in _raw_main
    subcommand.func(lib, suboptions, subargs)
  File "/usr/local/lib/python2.7/site-packages/beets/ui/commands.py", line 876, in import_func
    import_files(lib, paths, query)
  File "/usr/local/lib/python2.7/site-packages/beets/ui/commands.py", line 853, in import_files
    session.run()
  File "/usr/local/lib/python2.7/site-packages/beets/importer.py", line 316, in run
    pl.run_parallel(QUEUE_SIZE)
  File "/usr/local/lib/python2.7/site-packages/beets/util/pipeline.py", line 301, in run
    out = self.coro.send(msg)
  File "/usr/local/lib/python2.7/site-packages/beets/util/pipeline.py", line 183, in coro
    func(*(args + (task,)))
  File "/usr/local/lib/python2.7/site-packages/beets/importer.py", line 1236, in lookup_candidates
    task.lookup_candidates()
  File "/usr/local/lib/python2.7/site-packages/beets/importer.py", line 567, in lookup_candidates
    autotag.tag_album(self.items)
  File "/usr/local/lib/python2.7/site-packages/beets/autotag/match.py", line 433, in tag_album
    search_album, va_likely)
  File "/usr/local/lib/python2.7/site-packages/beets/autotag/hooks.py", line 565, in album_candidates
    out.extend(plugins.candidates(items, artist, album, va_likely))
  File "/usr/local/lib/python2.7/site-packages/beets/plugins.py", line 344, in candidates
    out.extend(plugin.candidates(items, artist, album, va_likely))
  File "/usr/local/lib/python2.7/site-packages/beetsplug/discogs.py", line 141, in candidates
    self.reset_auth()
  File "/usr/local/lib/python2.7/site-packages/beetsplug/discogs.py", line 86, in reset_auth
    self.setup()

Config:
directory: /Volumes/Music/beets
library: /Users/sammcj/Music/beetslibrary.blb
import:
    move: yes
    write: yes
    resume: yes
    log: beetslog.txt
plugins:
    fromfilename
    ftintitle
    duplicates
    fetchart
    lastgenre
    mbsync
    lastimport
    discogs
match:
    strong_rec_thresh: 0.30
fetchart:
    auto: yes
lastfm:
    user: sammcj2000
    api_key: <removed>

It looks like the traceback may have gotten cut off (it's missing the exception). Any chance there's more to paste?

Hello,
I think I have the same issue. The complete traceback should be:
Traceback (most recent call last):
  File "/opt/oparent/virtenv_beet/bin/beet", line 9, in <module>
    load_entry_point('beets==1.3.11', 'console_scripts', 'beet')()
  File "/home/oparent/repositories/beets/beets/ui/__init__.py", line 953, in main
    _raw_main(args)
  File "/home/oparent/repositories/beets/beets/ui/__init__.py", line 943, in _raw_main
    subcommand.func(lib, suboptions, subargs)
  File "/home/oparent/repositories/beets/beets/ui/commands.py", line 876, in import_func
    import_files(lib, paths, query)
  File "/home/oparent/repositories/beets/beets/ui/commands.py", line 853, in import_files
    session.run()
  File "/home/oparent/repositories/beets/beets/importer.py", line 316, in run
    pl.run_parallel(QUEUE_SIZE)
  File "/home/oparent/repositories/beets/beets/util/pipeline.py", line 301, in run
    out = self.coro.send(msg)
  File "/home/oparent/repositories/beets/beets/util/pipeline.py", line 183, in coro
    func(*(args + (task,)))
  File "/home/oparent/repositories/beets/beets/importer.py", line 1236, in lookup_candidates
    task.lookup_candidates()
  File "/home/oparent/repositories/beets/beets/importer.py", line 567, in lookup_candidates
    autotag.tag_album(self.items)
  File "/home/oparent/repositories/beets/beets/autotag/match.py", line 436, in tag_album
    search_album, va_likely)
  File "/home/oparent/repositories/beets/beets/autotag/hooks.py", line 565, in album_candidates
    out.extend(plugins.candidates(items, artist, album, va_likely))
  File "/home/oparent/repositories/beets/beets/plugins.py", line 344, in candidates
    out.extend(plugin.candidates(items, artist, album, va_likely))
  File "/home/oparent/repositories/beets/beetsplug/discogs.py", line 141, in candidates
    self.reset_auth()
  File "/home/oparent/repositories/beets/beetsplug/discogs.py", line 86, in reset_auth
    self.setup()
TypeError: setup() takes exactly 2 arguments (1 given)

The session argument is not defined in the reset_auth method. Passing None removes the exception but then, the discogs plugin prompts for the token indefinitely.
I am using the latest discogs-client package: 2.0.2.
Thanks,

Hi,
the setup() problem has just been fixed, please pull master.
I recently implemented re-authorization, since the plugin used to lose it and not ask for it. I'll have a further look at whether the problem comes from us, the discogs python client or the discogs server.

Hmmm, I see:
discogs: API Error: 401: You must authenticate to access this resource. (query: )
which comes from:
self._log.debug(u'API Error: {0} (query: {1})', e, query)
In my case, va_likely (album might be VA) = true, and I have no album title, which creates an empty query. That's probably why there's an error?

PR #1351 sidesteps the problem, but other re-auth issues may arise.
It comes from discogs-client and oauth2. I actually have a fork of discogs-client there which used oauthlib instead and is python 2 & 3 compatible. The PR still hasn't been merged. I just pinged the owners again, and I think that if it's not merged soon it might be a good idea to move to the fork.

It looks like we can close this issue, although other problems clearly remain. Too bad the library is unmaintained.

The PR with discogs_client has been merged and a fresh install should use it, solving the re-auth problems presented here. Otherwise I'm waiting for the to produce a new tag (for a version newer that the current 2.0.2) and I'll specify it in beets' setup.py.

Nice! Thanks, @brunal. I agree we should require the latest release when one is out to PyPI.

Done! Please open a new issue if you have a problem.


Export PM jobs from PR project

Export requested from Claydson<EMAIL_ADDRESS>Export list https://drive.google.com/file/d/1CWeEE047aCY7nk4osoLQqYpLoRZTK_zL/view?usp=sharing

Export https://drive.google.com/drive/u/1/folders/1gDEVNFxL_lW6tAFhbHlOy2cqqmZ8Xc3q


Allow some way to get the JSON files from a script

With the ability to do it without auth for things in PUBLIC state (useful for httpd vuln pages for example)

Or https://security.googleblog.com/2021/06/announcing-unified-vulnerability-schema.html

We have a get/post on /cve5/json/[id] existing, needs auth (but that's ok)

wrote a non-auth version for grabbing only-public issues


Exporting from the command line

Is it possible to run the exporter from the command line? I would like to run the it on the build.

sure you can, you can run blender with parameter
blender -b -P <python file>

in the python file, you write the code to open blender file, then export with exporter's options


Start creating altMe from talao wallet and crypto ssi wallet

complexity : infinite

dummy ticket used to explain Story Points


softLayer IBM s3

is it possible to add support for it? only missing env is ENDPOINT

@sarkistlt, can you send a PR for that?

Also, have you tried specifying the environment variable when running the container? It should support any environment variable supported by the aws-cli command.

let me test a couple things first, I think it will require new s3cfg file for IBM, but it's possible to have 2 conf file, lets say default and s3cfg-ibm, then if you want to use IBM all you need is add one more env. variable AWS_CONFIG_FILE=/s3cfg-ibm

Closed until OP sends more info.


The tree navigation filter does not show me only matches for the thing I typed

Type: Bug
This same problem happens in both the file explorer and the mongodb database explorer, both use the tree navigation control.
If I have a lot of databases and collections (or files) that have the word "Address" (Master_address, Primary_Address, etc. etc.) in the names of them as well as things that are suffixed by the abbreviation "_ADR", and I want to search for things ending in "_ADR", the tree navigation filter shows me results like this...

Master_Address
Primary_address
_WhoAddedRemarks
SomeThing_ADR
Sal_adressing

only the last two of which match the search criteria given. In my case this result list is actually a few hundred records long, and the things I am trying to filter for are interspersed throughout all the extra noise.
VS Code version: Code 1.71.2 (74b1f979648cc44d385a2286793c226e611f59e7, 2022-09-14T21:03:37.738Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.78GB (2.33GB free)

Process Argv
--crash-reporter-id 807d0387-caba-4b61-a1b4-330bf11683d8

Screen Reader
no

VM
0%

/duplicate of #116286

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


enable patchelf on windows for cross-compiling toolchains

Add handling of patchelf on windows, in case of cross-compiling toolchains (e.g. arm linaro on windows)

/azp run

Rerunning due to network mistake last night.


Remove "@Ignore", "enabled=false" and  "brokenOnJBallerina" from jBal unit tests

$subject.
This is used for disabling broken tests and ideally there shouldn't be any.

Most of the tests that were disabled with brokenOnJBallerina are enabled. There are few more. Moving them to fix on 1.1.

This is done. All test disabled test cases are enabled again.


Test is failed

I try to execute bundle exec rake spec locally, I had error.
Failures:

  1) Danger::DangerFindbugs with Dangerfile Create bug issues
     Failure/Error: expect(@my_plugin.bug_issues).not_to be_nil

     NoMethodError:
       undefined method `bug_issues' for #<Danger::Dangerfile:0x007fd8e5b0f6d8>
     # ./vendor/bundle/ruby/2.4.0/gems/danger-5.2.1/lib/danger/danger_core/dangerfile.rb:68:in `method_missing'
     # ./vendor/bundle/ruby/2.4.0/gems/danger-5.2.1/lib/danger/plugin_support/plugin.rb:23:in `method_missing'
     # ./spec/findbugs_spec.rb:48:in `block (3 levels) in <module:Danger>'

  2) Danger::DangerFindbugs with Dangerfile Send inline comments
     Failure/Error: expect(@my_plugin.send_inline_comment).not_to be_nil

     NoMethodError:
       undefined method `send_inline_comment' for #<Danger::Dangerfile:0x007fd8e5937cc0>
     # ./vendor/bundle/ruby/2.4.0/gems/danger-5.2.1/lib/danger/danger_core/dangerfile.rb:68:in `method_missing'
     # ./vendor/bundle/ruby/2.4.0/gems/danger-5.2.1/lib/danger/plugin_support/plugin.rb:23:in `method_missing'
     # ./spec/findbugs_spec.rb:55:in `block (3 levels) in <module:Danger>'

Deprecation Warnings:

Using `any_instance` from rspec-mocks' old `:should` syntax without explicitly enabling the syntax is deprecated. Use the new `:expect` syntax or explicitly enable `:should` instead. Called from /Users/ishikuranoboru/.ghq/github.com/noboru-i/danger-findbugs/spec/findbugs_spec.rb:52:in `block (3 levels) in <module:Danger>'.

If you need more of the backtrace for any of these deprecations to
identify where to make the necessary changes, you can configure
`config.raise_errors_for_deprecations!`, and it will turn the
deprecation warnings into errors, giving you the full backtrace.

1 deprecation warning total

Finished in 0.11623 seconds (files took 0.51359 seconds to load)
9 examples, 2 failures

I think, https://github.com/kazy1991/danger-findbugs/pull/3/files#diff-cb2124f377a7bb77eca4ecd8a4c0db42R77 changes makes bug_issues and send_inline_comment private, So we cannot call these methods by outside.
How about remove these test code?

@noboru-i
Thanks for reporting on the issue
Check it later, ASAP 👍


2024.4.4 update

dask-ml 2024.4.4
Destination channel: defaults
Links

{PKG-5134}
Upstream repository
Upstream changelog/diff
Relevant dependency PRs:

...

Explanation of changes:

update version, adjust filename for url. hatchling build system

So is this compatible with numpy 1 or 2? If the first, should it have an upper bound..? if the second, should it be >=2.0.0 <3 or something?


Modifications to get merge/Kintsugi work with OSX

Initial modifications to ethd and updated KINTSUGI.md to get eth-docker working with osx and docker-desktop

Hmm, no bonsai tries for merge?

FYI - the kintsugi_v1 network config won't work until this gets merged:
https://github.com/hyperledger/besu/pull/3347
(hopefully tomorrow if folks are online)


Messenger RNA not listed as a material entity

When validating SBML files from biomodels, we get the following warning whenever a species is annotated as messenger RNA:

The value of the 'sboTerm' attribute on a <species> is expected to be an SBO identifier (http://www.biomodels.net/SBO/). In SBML Level 2 prior to Version 4 it is expected to refer to a participant physical type (i.e., terms derived from SBO:0000236, "participant physical type"); in Versions 4 and above it is expected to refer to a material entity (i.e., terms derived from SBO:0000240, "material entity").
Reference: L2V4 Section 5
SBO term 'SBO:0000278' on the <species> is not in the appropriate branch.

It's possible that this is a validation problem, but I do think that messenger RNA is indeed a material entity (as well as being a 'functional entity'), and should be cross-added to that branch (like 'gene' is).

Hi,
today I encountered the same issue as I had several warnings that the SBO terms of my species were not in the appropriate branch. One of them was also the mRNA but in my oppinion also microRNAs should be included there.
What was the reasoning why it has been changed to material entity only in L2V4? In my oppinion microRNAs, mRNAs etc. would make it more precise.


Add max-width to text field part

Fixes #400

This change is

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks broke.

test/basic.html, line 330 at r1 (raw file):

          expect(datepickerWrapped.$.input.clientWidth).to.equal(120);
        });

This test fails, also in CI

Comments from Reviewable

Reviewed 1 of 2 files at r1, 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Reviewed 1 of 2 files at r1, 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable


A potential bug with respect to padding in convolve_s8_kernel3_inputch3_stride2_pad1_fpreq

Thanks for your great work!
When reading through your code and learn from it, we encountered several confusing lines which may be a potential bug.
In file convolve_s8_kernel3_inputch3_stride2_pad1_fpreq.c, the function 'convolve_s8_kernel3_inputch3_stride2_pad1_fpreq', padding is handled by checking if it's the first column and row, such as
if (base_idx_y != -1) { 				if (base_idx_x != -1) { //load all for now and unroll all
However, this means that the last column and row are not padded properly.
The following picture illustrates what we mean. The yellow blocks illustrate the items that you padded, while the red ones illustrate the items that you potentially missed.

Is this a bug?
We're looking forward to your response! Thank you in advance!

Thank you!
I have just read your code again. When input shape is even, this function is correct, because only the first column and row are needed to be padded.


add support for attaching to pids by-number (v3)

v2:
* define isInterger as a function sibling to init_session instead of a in-line declaration of the function in an else block (dpnishant , https://github.com/dpnishant/appmon/pull/44#issuecomment-323856974)

no problem. updated now.

Thank you for the PR!


Python scripts

Did a few things

Added a folder 'python_subscripts'. In the main folder there is the main python file.
Changed the matlab scripts folder to 'matlab_subscripts'.

In the main matlab file, changed 'genpath' to 'addpath'.

Changes from breakout3 are also here, so address breakout3 pull requrest first.

Nothing against Gary's code, but I'd put money on attendees encounting system compatibility issues when they try to execute these Python scripts. That's the most common issue I've dealt with teaching high school and college CS classes. I could refactor Gary's code to be cross compatible with Python versions 2 and 3, and doesn't depend on third-party packages that the user has to remember to install. Then attendees could do this whole tutorial with the Python version that shipped with their Macbook.

We specifically say use the anaconda distribution of python. That should remove all the compatibility issues if they get the latest version of anaconda.

Regarding this PR, issue #9, and issue #22, I think we should hold off on python/R scripts until we at least get some survey results back (setting 26th-ish cutoff). It may just be an enhancement and not a priority. When we cross this bridge, I think ease of download (full anaconda vs miniconda vs just the python that comes with your machine) and compatibility should be prioritized. The focus of this tutorial is version control, and the simplest examples could be typos in *.txt files; no need to get carried away just yet.

So the point of the first scripts is that the DON"T run. The base scripts we give them are supposed to have errors in them.

I mean even your solution doesn't compile without errors.

Is there a bug list? I'm re-introducing intentional errors now.

Look at the Matlab scripts and try to make the bugs the same

All that sounds fine to me. Those Python scripts include a hashbang #! that specifies their executable, so you don't actually need to preface each command with python. But if users do, it will still work. So it's all good.
Ok, I'm gonna merge now. 🤞

I saw the #!, but couldn't get it working without the explicit python in win10 anaconda powershell. On my mac I got it working without 'python' after troubleshooting some permission denied errors.


[FEATURE REQUEST] Why is there no support for transports?

Is your feature request related to a problem? Please describe.
I've created a pino transport for Logtail. I was hoping to use this package to integrate my NestJS logging to Logtail. The issue I have here though is that it seems this package is heavily involved into http context aware logging only. What about logging in other
Describe the solution you'd like
Allow the ability to pass pino options directly to this package and use the pino logger for everything, not just HTTP.
Describe alternatives you've considered
Creating my own module.

there is pinned issue for this feature already

@iamolegga My apologies. I need this for a project I'm working on, if you can detail a design to take, I can work on a PR if this suits you.

No worries. Let's keep the discussion in #803

if you can detail a design

will answer there


AMBARI-26128: Add UnlockExperimentalVMOptions to Hive startup parameters in Ambari

What changes were proposed in this pull request?
Currently, the Hive startup parameters in Ambari do not include the -XX:+UnlockExperimentalVMOptions flag. This omission causes errors when using the G1 garbage collector, preventing Hive from starting successfully. To resolve this issue, we propose adding the missing flag to the HADOOP_OPTS environment variable.
Rationale:

The -XX:+UnlockExperimentalVMOptions flag is required for certain G1 GC optimizations.
Without this flag, Hive fails to start when configured to use the G1 garbage collector.
This change ensures compatibility with G1 GC and prevents startup failures.

How was this patch tested?
manual test and CI
(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)
Please review Ambari Contributing Guide before opening a pull request.

Does Hive recommend this? Usually this param is for experimentation purpose only and not recommended for production use right?

experimentation

Does Hive recommend this? Usually this param is for experimentation purpose only and not recommended for production use right?

Currently, if Hive uses JDK 8 with G1 GC, the JVM will fail to start without enabling this parameter.


Layman's Guide Link no longer valid

In docs/source/contents.rs there is a reference to the Layman's guide, pointing to a domain that no longer exists (rsasecurity.com) using a protocol that is outdated  (ftp).
There are many copies of this document available, some of them guaranteed to be stable. Update the docs.

Patch in  #50.


Revert "Add new feature "back to last position""

Reverts spgennard/vscode_cobol#72
I need to revert the changes because of the following observed issues:

line cache is not cleared when a new document is opened/closed
only the line is saved not the column
in getSectionOrParaLocation, the line is saved before we get a successful match

close but needs some more work.

Ok spgennard.
I'm going to take a look at this issues.
Thanks.

I think you need to keep a dictionary indexed by the filename, which is
cleared on an onDidCloseTextDocument event.
If the "remember" location function is exported, I can also tie this into
other features that moves around the documents.
On Sun, 9 Jun 2019 at 16:19, Rodrigo Augusto<EMAIL_ADDRESS>wrote:

Ok spgennard.
I'm going to take a look at this issues.
Thanks.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/spgennard/vscode_cobol/pull/73?email_source=notifications&email_token=ABNWWGH7QBLAKE2CJ2YXAKLPZUNRXA5CNFSM4HWHOVFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXIMEUY#issuecomment-500220499,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABNWWGABOMP3Q2FJTVL6FELPZUNRXANCNFSM4HWHOVFA
.

--
Stephen Gennard
<EMAIL_ADDRESS>


Skill Matrix - Request Support Page Dropdowns

Sample
Skills Matrix
Author(s)
@jmcgee7
Question
I cannot seem to get the dropdowns on the request support page to work

The data is loaded in the other screens and working fine.
I have all of the same columns in my MatrixNetwork

and they are indexed. I don't think it is a data connection issue.

Hi,
Did you get this working? I have the same issue.

I did not, I have abandoned this as a result. It was exactly what I was looking for as a solution but I couldn't get it to function


mri, initial offsets

allow the consumer to choose an initial offset position on mri.  this can be either :start, :end, or a fixnum position.  would do the same for the jruby client if it was thought valuable, but I'd need to go poke into the interface.

After this change is merged, I'll cut a new release to rubygems.org since my magic auto-deploy stuff isn't entirely working correctly

updated, thanks for the review!

yup, sorry.  fixed now.

0.23.242 contains this change

awesome, thanks much @rtyler


MicroProfile 4.1 Compatible Certification Request

[x] Organization Name ("Organization") and, if applicable, URL:
IBM
[x] Product Name, Version and download URL (if applicable):
Open Liberty <IP_ADDRESS>-beta Source jar
[x] Specification Name, Version and download URL:
MicroProfile 4.1
[x] Public URL of TCK Results Summary:
MicroProfile 4.1 TCK results
[x] Any Additional Specification Certification Requirements:
No
[x] Java runtime used to run the implementation:
Java 8, 11
[x] Summary of the information for the certification environment, operating system, cloud, ...:
See the TCK results for details
[x] By checking this box I acknowledge that the Organization I represent accepts the terms of the EFTL.
[x] By checking this box I attest that all TCK requirements have been met, including any compatibility rules.

The TCK summary page references Jakarta EE TCK results page, that describe tests against a different version of OpenLiberty. If this version intends to claim compatibility with Jakarta EE TCKs, it should include proper evidence for this version.
I am not clear what requirements were settled on, for compatibility claims with MicroProfile 4.1 and the required Jakarta EE component specifications but, for me, it would be better to just omit these since they are not really relevant. If there are results for <IP_ADDRESS>-Beta, it would be excellent to include them.
In short: I would either drop this reference to <IP_ADDRESS>, or provide a reference for <IP_ADDRESS>-beta.

@edbratt thanks for reviewing this! The settlement on the compatibility was in the MP 4.1 spec itself: only require passing the 8 MP spec tcks. I will drop the reference to <IP_ADDRESS> for Jakarta EE tck results to avoid any confusion. I'll get this done on Monday as I will be off tomorrow.

Before I log off for my PTO, I managed to do the changes to remove the Optional Jakarta EE section and hopefully the change will be visible tomorrow.


migrate to io.FS when Go 1.16 is out

I think we can migrate once its out... we would only need to write some sort of test fs... or wrap afero.FS in a io.FS... not sure yet...
But the endgame IMHO is for the exposed API to use io.FS

We should definitely do this. However, we have to keep in mind that that change would imply that nfpm and goreleaser also require Go 1.16 to build which may put off the Linux crowds with their slow distros. Personally, I can't be bothered with waiting for them to upgrade their stuff, but we should still keep that in mind.

hmm I think that's a non-issue for users using the goreleaser binaries...
also, goreleaser always use the latest (I always update as soon as the new version is out), and this time, with m1 support coming on 1.16, it will not be different anyway... so.. IMHO... it should be fine 🤔

Yeah the users are fine but I meant the contributors with Linux distributions that ship with compilers a few versions behind.
But, again, I'm with you and I think that's not really our problem.

done, since idk when


virtuallist doesn't render new items after using deleteAllItems()

Hi
I'm having trouble with virtual list. when I use deleteAllItems() on my virtual list it clears all items, but after appending new items via appendItem() I can see that new items are added to virtual list items property (they exist in items property of my virtual list) but they are not rendered and now shown in the list. also onItemBeforeInsert() doesn't fire up anymore when appending items after using deleteAllItems().
This problem also happens if you delete all items of virtual list using deleteItem() and delete them one by one.
What is interesting is that if you leave at least one item in the virtual list and don't clear it completely appending new items and everything else works just fine.
Steps to reproduce:

Create a virtual list.
Append items to it.
Delete all items.
Append new items.

Expected results:
New items should be rendered and shown.
Actual result:
New items are added to "items" property of virtual list and yet not rendered and shown.
Additional info:
I'm using virtual list with dynamic height function and template property for rendering items.
If there is any more information needed I'll be glad to provide it.

Is this issue still actual as i can not replicate it? Works fine for me

I have the same problem.
It delete only the first item of the list. Using deleteItem don't update display. Using update work only one.
Quite problematic

Could you provide some details and a codepen / jsfiddle please?

After some code reading and console.log usage, i think I understand what is happening in my case.
I create a Virtual List and I use the callback "renderItem"
renderItem: function (index, item) {
On my template for items, I inject the "index" value on a "onclick" action of an html element.
So, the deleteItem function in Framework7.js is called with index = 2 (i just pass index value without manipulate it).
Look at the  console.log(index); I put near the "splice" function .
The console output "20" !! So splice is called with splice(20,1).
The 20 is "built" by  index = index + indexShift; line. It does 2 + 0 = 20 as text.
To have the good behaviour, I have to call deleteItem with a parseInt(index) parameter.
I don't understand why the index value that was given by the renderItem: function (index, item) { callback must be "parseInt"ed
I spent 2 hours understanding what happened. But now it works
   ```
 // Delete
        vl.deleteItems = function (indexes) {
            var prevIndex, indexShift = 0;
            for (var i = 0; i < indexes.length; i++) {
                var index = indexes[i];
                if (typeof prevIndex !== 'undefined') {
                    if (index > prevIndex) {
                        indexShift = -i;
                    }
                }
                index = index + indexShift;
                prevIndex = indexes[i];

                // Delete item
                console.log(index);
                var deletedItem = vl.items.splice(index, 1)[0];

                // Delete from filtered
                if (vl.filteredItems && vl.filteredItems.indexOf(deletedItem) >= 0) {
                    vl.filteredItems.splice(vl.filteredItems.indexOf(deletedItem), 1);
                }
                // Update cache
                if (vl.params.cache) {
                    var newCache = {};
                    for (var cached in vl.domCache) {
                        var cachedIndex = parseInt(cached, 10);
                        if (cachedIndex === index) {
                            delete vl.domCache[index];
                        }
                        else if (parseInt(cached, 10) > index) {
                            newCache[cachedIndex - 1] = vl.domCache[cached];
                        }
                        else {
                            newCache[cachedIndex] = vl.domCache[cached];
                        }
                    }
                    vl.domCache = newCache;
                }
            }
            vl.update();
        };

If you've got a fix please open a PR 😄

In fact I just "feed" the deleteItem public function using parseInt(index) et not index. And all is working perfect now.
If I have time I will try to understand why the renderItem callback give me a non Integer index. I think it would better to fix on this side rather than parseInt at the end.
I will update


Cannot assume IAM profiles on windows

On behalf of a slack user.
In Windows powershell, Granted fails to assume either an IAM profile with credentials in the .credentials file or a profile in the .config file which references a source_profile
e.g
config
[profile creds]
region = ap-southeast-2
[profile dev]
region = ap-southeast-2
source_profile = creds
credentials
[creds]
aws_access_key_id=
aws_secret_access_key=

Hi Josh, as mentioned, here is my config the same as requested above. Additionally, I've added role_arn in the profile:
config
[default]
region = ap-southeast-2
output = json
[profile creds]
region = ap-southeast-2
[profile dev]
region = ap-southeast-2
role_arn = arn:aws:iam::1234:role/GrantedAdmin
source_profile = creds
credentials
[creds]
aws_access_key_id = XXX
aws_secret_access_key = XXX
region=ap-southeast-2


Question: How to overlay on each page?

Hi, I know about set(_ container: PDFContainer, absoluteOffset: CGFloat)
to overlay a large text item over a page. How to do this on each page? I want to “watermark” each page with the same text.

Hi, Watermarks have been requested in #231 and extensively discussed too, and there was an WIP pull request by @chrisgonzgonz #241 using a delegation pattern.
I will close this issue in favour of #231 so please read through what has been discussed there already. Will consider taking over the PR and finishing it myself


[Feature Request] Documentation: how to build the tank outside of the gui?

I didn't see it in the documentation, so I don't know if it is a doc missing or a feature request...
Is it possible to inject my own tank from either another location (local folder or a github repository)?
How could I share my tank to run somewhere else (at friends) ?

The easiest way is to just copy *.tank.js file to /tanks folder and update of /tanks/index.json file. It is not described in docs but I will add it

documented in https://github.com/jamro/jsbattle/blob/master/docs/dev_guide/adding_bundled_tanks.md

This doesn't seem to work.  My new tank doesn't show up in the list if I refresh the page.  And it doesn't even show up if I rebuild.  Has something change? Should I open a bug for this?


Deployment on Bielefeld OpenStack failled

Hi,
Issue to lift a kubernetes cluster. Tried with terraform 0.7.8 and 0.7.9.
Yours, Steffen
root@host-192-168-0-5:~/KubeNow# terraform apply openstack # deploy the cluster
There are warnings and/or errors related to your configuration. Please
fix these before continuing.

Warnings:

  * template_file.master_bootstrap: using template_file as a resource is deprecated; consider using the data source instead
  * template_file.edge_bootstrap: using template_file as a resource is deprecated; consider using the data source instead
  * template_file.node_bootstrap: using template_file as a resource is deprecated; consider using the data source instead

No errors found.
Instead, your Terraform state file has been partially updated with
any resources that successfully completed. Please address the error
above and apply again to incrementally change your infrastructure.

OpenStack API version 2.0

As reported in https://github.com/hashicorp/terraform/issues/10130, a workaround to this issue consists in using Terraform 0.7.7.

Setting wontfix as it is a problem caused by https://github.com/gophercloud/gophercloud. Hopefully it is going to be solved soonish.

A workaround is to specify flavor_id instead of flavor_name, or image_id instead of image_name, also reported in hashicorp/terraform#10130

@sneumann is this still a problem?

Fixed by Terraform upgrade to version > 0.9.4

closing then 🙂


Apiai

Coverage decreased (-10.6%) to 88.376% when pulling caa18314f649d4a1b7803d0037a6428331f53113 on rmberrios:apiai into 24b096746c9cb7898dfbe08b421d4408e8233e9e on mediarain:dev.

Wow, i don't really wanna merge all this work into dev yet. We still need to test some more, have a session to check how we're doing things and see if this is really the direction we want to go

But why close it? i was working on the PR


Added asynchronous dismissal of actions

@MichaelCampbell @htinlinn

:+1:

this looks good but was there any reason we have viewController as a property in YoshiActionResult instead of having it as an associated value in YoshiResult?

@htinlinn at this point, no. I can just refactor out YoshiActionResult and just have the function return YoshiResult. Would you guys want me to do that here or in a subsequent PR? This would be a breaking change.

I vote subsequent PR, @htinlinn?

@Haud Conflicts

still a :+1: for me

alright. it makes sense to do this in a subsequent pr.

:+1:


Unexpected Message in 17-stateful-communication example

Prerequisites

[X] I checked the documentation and found no answer to my problem
[X] I checked the existing issues and made sure there are no similar bug reports

Category
Bug (unexpected behavior)
Expected Behavior
I do not expect an error
Observed Behavior
INFO:     [chit_agent]: Received message: Hello!
Please enter your message:

hi
ERROR:    [chit_agent]: Received error message from agent1qwvecfwc255pfqqwtjznh9qqk6skl77xc6fzw8mr3ppfex32sr0kcad62n4: Unexpected message in dialogue: text='hi'
NoneType: None

To Reproduce
Run agent 4, then agent 3 withpython agent4.py etc

Predefined Dialogue (Agent 3 & 4)
Agent 3 and 4 are given a graph of the same ChitChat Dialogue but most of the interactions are automated and not exposed to the user. Only one interaction to start the dialogue and the cyclic ChitChat message is required to use the graph.

Version
latest, main
Environment Details (Optional)
No response
Failure Logs (Optional)
No response
Additional Information (Optional)
No response

I just checked out and tested with the latest uAgents version and main branch and can confirm that the example works as intended.

Making sure that you have the required packages installed and invoking the agents with:
python agent3.py

and
python agent4.py

respectively.
Please check your environments and see if you've changed the predefined dialogue or anything else in the examples folder.

Closing this issue as completed.


Add missing F# console related samples from api/system.console page

Summary
Add missing F# console related samples. These changes cover all code samples that can be found on the System.Console API documentation page.
In light of the conversation in https://github.com/dotnet/docs/issues/16105 I'm going to plan on creating separate merge requests for each grouping of F# samples that I cover in order to avoid unnecessary merge conflicts due to restructuring.
Partially addresses dotnet/dotnet-api-docs/issues/3571

Thanks @cartermp, I'll take a look and grey to address these tonight. Thanks for the helpful feedback.

@cartermp, I believe I have all of the requested changes completed along with some extra refactoring and formatting fixes. When you get the chance please let me know if you see any remaining changes that need to be made.


Self-destructing comments

Is your feature request related to a problem? Please describe.
Some stackers manually delete their comments after some amount of time for privacy reasons (I assume).
This is bad UX and we could maybe accommodate their needs better.
Describe the solution you'd like
We could help them by providing a setting in their settings and/or during posting/commenting that will delete their comment automatically.
We could charge extra sats for that since I think we don't want everyone to delete all their posts and comments after some amount of time, lol. The faster it gets deleted, the more sats it requires?
Describe alternatives you've considered

a script that we could provide which they can run to do it themselves

Additional context
https://stacker.news/items/245650

Another name: "ephemeral comments"

Forgetfulness is the only mechanism we humans have for gaining wisdom, so they say.. uhhhhh anyway so yeah I agree, lets do this 🤓

I am considering picking this up. I have a few thoughts/questions:

The title of this issue specifies comments, but should this apply to posts, too? IMO it should apply to comments and all post types.
What is the logic around charging more to delete it sooner? I'm not arguing against it, I don't have a strong opinion either way, I'm just curious what would be best and why.
What kind of options should be supported for when to delete the item? Maybe we follow the search pattern and have a few options (a day, a week, a month, a year), and then a custom date/time picker for something specific?
For posts, the placement of the UX seems fairly intuitive, probably another option in the options accordion section. But for comments, how might we expose this option in a clean way? I know @huumn likes to keep the Reply component clean :)
Do we want this to be a user-level setting or should it just be specified on a per-item basis?
I'm curious if folks have any ideas around implementation of the deletion. My first thought would be to store a deleteAt timestamp in the item table and have a job that runs every minute (in the worker, maybe?) that "deletes" all items whose deleteAt has passed (not delete the row, but convert the title/content to deleted by author, as we do today). Maybe there's a better implementation idea floating around?

The title of this issue specifies comments, but should this apply to posts, too? IMO it should apply to comments and all post types.

Yes, the symmetry is nice.

For posts, the placement of the UX seems fairly intuitive, probably another option in the options accordion section. But for comments, how might we expose this option in a clean way? I know @huumn likes to keep the Reply component clean :)

Perhaps it's a directive in the text itself? Maybe it's a "builtin" bot, in other words acts like a bot but isn't a bot? @delete in 7 days ... we could then parse the text for @delete mentions.

I'm curious if folks have any ideas around implementation of the deletion. My first thought would be to store a deleteAt timestamp in the item table and have a job that runs every minute (in the worker, maybe?) that "deletes" all items whose deleteAt has passed (not delete the row, but convert the title/content to deleted by author, as we do today). Maybe there's a better implementation idea floating around?

I think it might be better to schedule a job to run when an individual item is supposed to be deleted.
INSERT INTO pgboss.job (name, data, startafter) VALUES ('deleteItem', jsonb_build_object('id', NEW.id), now() + interval '10 minutes');

we could then parse the text for @delete mentions.

checks if delete is a reserved name here and realizes it is
Where did you find that list of reserved names btw? Looks really thorough.

Google?

Google?

haha, could have thought of that. Sometimes, I think too complicated or think you're some kind of wizard, haha

For example, assume everyone does that and SN becomes a wasteland

I don't love runaway hypotheticals. If people delete a lot of stuff, we'll just try to make it interfere with other people's experience less.

at least I thought it would be against the interests of SN, @huumn might disagree

Stacker's interests are SN's interests. But when the interests of stackers conflict, do we favor the interests of producers or consumers? I say if content creators want to delete stuff, it's their stuff to delete. If other people don't like that, they should make better content than the people deleting stuff.

Thank you both for the feedback. I will get to work on this and we can iterate/adjust as we see fit!

For example, assume everyone does that and SN becomes a wasteland

I don't love runaway hypotheticals. If people delete a lot of stuff, we'll just try to make it interfere with other people's experience less.

Makes sense that we can approach this issue when it actually happens. So I agree, it's a runaway hypothesis which makes it not a good argument

at least I thought it would be against the interests of SN, @huumn might disagree

Stacker's interests are SN's interests. But when the interests of stackers conflict, do we favor the interests of producers or consumers? I say if content creators want to delete stuff, it's their stuff to delete. If other people don't like that, they should make better content than the people deleting stuff.

Also makes sense, I can see how my idea with variable fees is actually worse than the strictly hypothetical case of too many people using/"abusing" this feature. This even excludes the amount of complexity we would introduce just for that. I also think now there is no such thing as "abusing the system". If you can abuse the system, the system is just bad, lol
As you said, if there is a problem with something, the solution is not to make the experience worse but to make it better :)


Fixes #3253 - make User-Agent configurable

Adds a configuration option to overwrite the default User-Agent header that is send at least by the calendar and news module. Allows other modules to use the individual user agent as well.
The configuration accepts either a string or a function:
var config =
	{
		...
		userAgent: 'Mozilla/5.0 (My User Agent)',
		...
	}

or
var config =
	{
		...
		userAgent: () => 'Mozilla/5.0 (My User Agent)',
		...
	}

Thank you for this PR!
First thing, the test fails are unrelated to your changes, we had problems with one unit test which are now fixed on develop, so can you please merge develop into your branch (or rebase on develop), thanks.
I'm not sure if we need the new getUserAgent() function because we store defaults in js/defaults.js, so AFAIK would be the normal way to add the entry
userAgent: `Mozilla/5.0 (Node.js ${Number(process.version.match(/^v(\d+\.\d+)/)[1])}) MagicMirror/${global.version}`,

in this file and then use this property with config.userAgent in the modules and the tests.
But before refactoring this may @rejas can review and tell us what he prefers ...

I'm not sure if we need the new getUserAgent() function because we store defaults in js/defaults.js, so AFAIK would be the normal way to add the entry
userAgent: `Mozilla/5.0 (Node.js ${Number(process.version.match(/^v(\d+\.\d+)/)[1])}) MagicMirror/${global.version}`,

in this file and then use this property with config.userAgent in the modules and the tests.

In the beginning I thought about that as well, but it felt a little bit odd to have all those variables in a configuration string by default. Also I wasn't sure when those variables are available and in case someone wants to use other runtime variables in here, that might lead to issues - that's why I added the possibility to use it with a callback as well.
I also thought about, wether it would make sense to add a parameter (e.g. the module name, url, ...) to the method, so that it could be possible to use different user agents based upon those parameters.
Based on those thoughts i decided to wrap it in a utility method to allow easier, non-breaking, changes in the future. But if you prefer to consume the config.userAgent directly, I'm also fine with it 😉

and I forgot something yesterday because config.userAgent doesn't work, the module only gets his own config section, not the global one (so the default userAgent had to be set in every module).
So I'm fine with merging this.

since when does config. not work?
you have to pass it from browser side to node helper,

may I'm wrong but AFAIK you can't get e.g. config.language in the node_helper.js of the module (not without special coding)

yes, but 'special' is tiny..
from my MMM-Config
  notificationReceived: function (notification, payload, sender) {
    // once everybody is loaded up
    if (notification === "ALL_MODULES_STARTED") {
      // send our config to our node_helper
      this.config.address = config.address;
      this.config.port = config.port;
      this.config.whiteList = config.ipWhitelist;
      this.sendSocketNotification("CONFIG", this.config);   // this sends to node_helper
    }
  },

so for using config.userAgent instead of the implemented function in this PR we had to do this.config.userAgent= config.userAgent; in the 2 modules.
I'm fine with both solutions.

and then in node_helper to use it, its  payload.config.userAgent (or whatever variable payload is stored in)
from the sockerReceivedNotifications.
and you can get it ALL with
this.config.global_config = config
we have no cross module protection on the config data


[NFC] [AutoDiff] Gardening.

@swift-ci Please test tensorflow

Merging to unblock progress, happy to address feedback later.


[core] Upgrade to angular 2.2.0

Signed-off-by: Matt Hippely<EMAIL_ADDRESS>

@mathisscott - I am going to close this PR and re-submit, after all browsers are tested.


Bulk Inserts

We have been using ex_machina for seeding bulk data. I was thinking it would be a good feature to add insert_bulk functionality to make that process faster. How does that sound to you?
If this sounds like a feature you guys want, I can work on it in a PR.
Thanks for writing this library :smile:

Hi @aditya7iyengar, thanks for bringing up this idea. It's interesting you're using ex_machina for bulk inserts. Just curious, is this dummy data you're inserting for development?
Since ex_machina is mostly used in tests, I'm not quite sure bulk inserts would be a great feature to add. Instead, what I might do in that scenario is use something like Ecto's insert_all. Do you know if that would be good for your use case?

Hi @aditya7iyengar I'm going to close this for now. If this is still something of interest, let me know! Glad to reopen this if the previous comment doesn't make sense for your use case.


Fixed the return result of ShardingSuffixs cannot be correctly matched when using string type table suffix.

…ot be correctly matched when using string type table suffix.
Fixed the problem that the return result of ShardingSuffixs cannot be correctly matched when using string type table suffix.

[x] Do only one thing
[x] Non breaking API changes
[x] Tested

What did this pull request do?

User Case Description

@jinzhu Can you help me, thanks?

Thank you.

any plan on releasing this ?


Support throwing Java exceptions in interface implementation

Another thing that users might want to do is throw actual Java exceptions. This seems to be useful actually, and we could use the same Dart throw keyword to achieve this by checking if the exception thrown is of type JException.

Some pointers in case you want to fix this @Anikate-De
Here is the place where we throw DartExceptions: https://github.com/dart-lang/native/blob/main/pkgs/jni/java/src/main/java/com/github/dart_lang/jni/PortProxy.java#L82-L90
Currently we only check if the result is a DartException and only then we throw it. But we want to throw every type of exception. This works because the DartException class is private and we can be sure that no one is returning an instance of it (so the only way we get it as a result is if someone has thrown).
To add support for all kinds of exceptions. We can either:

Add another object to the result array, where result[0] is the pointer, result[1] is the actual result and result[2] is an integer enum, for example 0 when it's a normal returned value and 1 when it's a thrown error. Or
Add a nullable Throwable cause to DartException and fill it if we throw a Java throwable from Dart. Then check if cause is not null and if so throw the cause instead of the top level DartException.

Option 2 is probably faster to implement.
In both cases you will need to check if the thrown object is a JObject and is an instance of java.lang.Throwable in the generated code. https://github.com/dart-lang/native/blob/main/pkgs/jnigen/lib/src/bindings/dart_generator.dart#L415-L417

Thanks for the pointers!
I am still a little confused, unfortunately.
I tried following Option 2

2. Add a nullable Throwable cause to DartException and fill it if we throw a Java throwable from Dart. Then check if cause is not null and if so throw the cause instead of the top level DartException.

private static final class DartException extends Exception {
  Throwable cause;

  private DartException(String message, Throwable cause) {
    super(message);
    this.cause = cause;
  }
}

@Override
public Object invoke(Object proxy, Method method, Object[] args) throws Throwable {
  Object[] result = _invoke(port, isolateId, functionPtr, proxy, getDescriptor(method), args);
  _cleanUp((Long) result[0]);
  if (result[1] instanceof DartException) {
    Throwable cause = ((DartException) result[1]).cause;
    if (cause != null) {
      throw cause;
    } else {
      throw (DartException) result[1];
    }
  }
  return result[1];
}

I then updated the dartjni.h to include the cause
JniResult DartException__ctor(jstring message, jthrowable cause);

Likewise updated dartjni.c
But here is the problem I faced:
Won't this https://github.com/dart-lang/native/blob/main/pkgs/jnigen/lib/src/bindings/dart_generator.dart#L415-L417, always throw a JniException?
Reference: https://github.com/dart-lang/native/blob/a44ef95ee3fcd2e39281b1abe3e405072e602790/pkgs/jni/lib/src/accessors.dart#L107-L111
I'm confused about how I could check whether it is a JObject and then a java.lang.Throwable
Also, I'm still unable to workaround java.net.SocketTimeoutException for ok_http with this approach :(
I can send a Draft PR if you'd like to check it out, just need to add some comments.

Thanks for working on this!

Won't this https://github.com/dart-lang/native/blob/main/pkgs/jnigen/lib/src/bindings/dart_generator.dart#L415-L417, always throw a JniException?

No, that's just returning the new DartException object. You just have to add the cause to it as well as the message: https://github.com/dart-lang/native/blob/main/pkgs/jni/lib/src/jni.dart#L239-L244
To regenerate the bindings for jni, run the script in tool/generate_ffi_bindings.dart. And after changing the code generation in jnigen, you can generate all the bindings again using tool/regenerate_all_bindings.dart (in jnigen).
After the regeneration of FFI bindings, DartException__ctor will get an extra parameter. There are some signatures that you need to also change in C. But it's easier for me to comment on a drafted PR for those.
Back to the codegen, if the caught exception e is an instance of java.lang.Throwable then you can use that for the cause argument, otherwise you can pass null.

I'm confused about how I could check whether it is a JObject and then a java.lang.Throwable

e is JObject && Jni.env.IsInstanceOf(e.reference.pointer, jThrowableClass.reference.pointer)

You can temporarily put final jThrowableClass = JClass.forName('java/lang/Throwable'); somewhere for example in internal_helpers_for_jnigen.dart. Until we have a full fledged JThrowable in package:jni.

I can send a Draft PR if you'd like to check it out, just need to add some comments.

Please do. I'll assign you to this issue!


Running 'gulp' on project is resulting in error "No gulpfile found"

After making the npm install and bower install if I try to run the
gulp to build the project, it results in an error "No gulpfile found".
Looking at the web how to resolve that you can find tips to create a gulpfile.js, but then running gulp is not actually building the project.

Theresa a gulpfile in the main repo. Do you have it in your checkout?


it still no mudule when i Successfully installed utils.misc

from utils.misc import thresh_OTSU, ReScaleSize, Crop
ModuleNotFoundError: No module named 'utils.misc'

Hi @qwertliwen
Sorry for replying slowly, and we have uploaded misc.py. Please check it!
Best!


Put rustflags in .cargo/config instead of environment variable

It's easy to forget setting env variables.

Looks great. I'll merge once tests are passing (ignore clippy I have a different PR that has that fixed).


fix CI ingress-nginx installation

The file we relied on previously to detect the latest ingress-nginx version was removed, so I changed it to use the release tags.
https://github.com/kubernetes/ingress-nginx/pull/8778

Note that the upgrade job is expected to fail because it tries to install based on what is currently in main, which is broken, and the reason for this PR.


Maven build issue

While executing the command
! mvn clean package appassembler:assemble
I get the following error:
BUILD FAILURE [ERROR] Failed to execute goal org.codehaus.mojo:appassembler-maven-plugin:2.0.0:assemble (default-cli) on project anserini: Execution default-cli of goal org.codehaus.mojo:appassembler-maven-plugin:2.0.0:assemble failed: Plugin org.codehaus.mojo:appassembler-maven-plugin:2.0.0 or one of its dependencies could not be resolved: The following artifacts could not be resolved: net.java.dev.stax-utils:stax-utils:jar:20060502, stax:stax:jar:1.1.1-dev: Could not transfer artifact net.java.dev.stax-utils:stax-utils:jar:20060502 from/to central (https://repo.maven.apache.org/maven2): Connection reset -> [Help 1]
I am using Colab, and Java 11.
result of
!java -version
openjdk version "11.0.10" 2021-01-19
OpenJDK Runtime Environment (build 11.0.10+9-Ubuntu-0ubuntu1.18.04)
OpenJDK 64-Bit Server VM (build 11.0.10+9-Ubuntu-0ubuntu1.18.04, mixed mode)

Hi @Tooba-ts1700550
We have a demo of Anserini on Colab.
https://github.com/castorini/anserini-notebooks/blob/master/anserini_robust04_demo.ipynb
I just run this and it works.
See if it can help?

Having heard no follow-up, closing...


Use links to new Rust book instead of old

There are several places where there are links to the first edition of the Rust book. These links should probably be updated to point to the new Rust book. See for example the links in this section: https://os.phil-opp.com/vga-text-mode/#a-println-macro
I could take care of this if you agree that it should be done

Hi, thanks for creating this issue and offering your help!
I tried to update all links that point to a "There is a new edition of the book and this is an old link." site when creating the second edition, but a few seem to slipped through. I'm happy to merge PRs that update them.
However, there are a few links that deliberately point to the first edition of the Rust book, because the topic is no longer explained in the second edition (these should point to the first edition directly, not to the "There is a new edition of the book and this is an old link." landing page).

I think this can be closed now