[Backport] feat(cli): promote allow Integration update

Release Note
feat(cli): promote allow Integration update

Checks failure unrelated. Merging.


spaces proof for path in install.sh

Fixes #952

Closing this one, since #954 looks better. Thanks for the fix anyway!


Proof of concept: move cutin uturn waypoints generation from smarts to open-agent

Took a stab at decoupling cutin and uturn waypoint generation from smarts.
In summary:

Changed the original AgentBehaviour class which just has an aggressiveness value to be responsible for uturn and cutin waypoint generation.
Kept AgentBehaviour as part of AgentInterface. Gave WaypointsSensor access to AgentInterface.agent_behavior from vehicle.py
WaypointsSensor calls apply_custom_behaviour from AgentBehaviour
will pass all the needed information for uturn and cutin from WaypointsSensor to AgentBehaviour, such as self._road_network self._waypoints
self._road_network can be accessed from sim
self._waypoints can be accessed from scenario.waypoints

Since PR #722 haven't been merged and I expect there are new PR changes in Behaviour Model soon. I didn't proceed to change the details in def uturn_waypoints: def cut_in_waypoints: from referencing self.xxx in mission_planer to reference things in the new structure. It is better to make these changes after finalizing Behaviour model changes, and first making sure if this structure change makes sense
For now, this is just a proof of concept PR to show whether this new structure works.

Why is it necessary for the AgentInterface to has AgentBehaviuor as part of it? Why should WaypointsSensor have information about custom behavior at all? It seems to me that all that logic of adapting and filtering waypoints belongs to the internal of the agent. Maybe what’s needed instead is actually map observation that covers a broader area, map observation that includes adjacent lanes etc?
From: Jeffer Jingfei Peng @.***
Sent: Thursday, April 8, 2021 8:41 PM
To: huawei-noah/SMARTS @.>
Cc: Subscribed @.>
Subject: [huawei-noah/SMARTS] Proof of concept: move cutin uturn waypoints generation from smarts to open-agent (#749)
Took a stab at decoupling cutin and uturn waypoint generation from smarts.
In summary:

Changed the original AgentBehaviour class which just has an aggressiveness value to be responsible for uturn and cutin waypoint generation.
Kept AgentBehaviour as part of AgentInterface. Gave WaypointsSensor access to AgentInterface.agent_behavior from vehicle.py
WaypointsSensor calls apply_custom_behaviour from AgentBehaviour
will pass all the needed information for uturn and cutin from WaypointsSensor to AgentBehaviour, such as self._road_network self._waypoints ( though they are supposed to be private variable, will see what can be done)

Since PR #722https://github.com/huawei-noah/SMARTS/pull/722 haven't been merged and I expect there are new PR changes in Behaviour Model soon. I didn't proceed to change the details in def uturn_waypoints: def cut_in_waypoints:. It is better to make these changes after finalizing Behaviour model changes.
For now, this is just a proof of concept PR to show whether this new structure works.

You can view, comment on, or merge this pull request online at:
https://github.com/huawei-noah/SMARTS/pull/749
Commit Summary

initial commit
black

File Changes

M smarts/core/agent_interface.pyhttps://github.com/huawei-noah/SMARTS/pull/749/files#diff-09e41aaa060eed1eb0690b068f085ab0de6a9d7aae708a50c2cef760936d49f3 (10)
M smarts/core/mission_planner.pyhttps://github.com/huawei-noah/SMARTS/pull/749/files#diff-72668ccabb65e2bfe5e459aae7b67da9e3798e1fdb65014cb2c3c95d435a615f (322)
M smarts/core/sensors.pyhttps://github.com/huawei-noah/SMARTS/pull/749/files#diff-fc41655ffc7f7dc46a63dbf123288d7ab6cbc498eabdc7b3c0a99a57edbaad63 (20)
M smarts/core/vehicle.pyhttps://github.com/huawei-noah/SMARTS/pull/749/files#diff-3e1faf2c970b35f533e2632431d749d9bc9fc29f2b04ec2af6b799046fe8c843 (1)
M zoo/policies/open-agent/open_agent/init.pyhttps://github.com/huawei-noah/SMARTS/pull/749/files#diff-402baf40c60d691242bf4bd56b7d01224dfc8766de87118b7101c272e55ca163 (4)
A zoo/policies/open-agent/open_agent/agent_behaviour.pyhttps://github.com/huawei-noah/SMARTS/pull/749/files#diff-6040df8281e7819618d31a88dcd973ec73173dd30487e3d87b46ee97156c1462 (342)

Patch Links:

https://github.com/huawei-noah/SMARTS/pull/749.patch
https://github.com/huawei-noah/SMARTS/pull/749.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/huawei-noah/SMARTS/pull/749, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMXCDS7GUYIUV6JEBR7Q64TTHZEJFANCNFSM42T6UMMQ.

Why is it necessary for the AgentInterface to have AgentBehaviuor as part of it? Why should WaypointsSensor have information about custom behavior at all? It seems to me that all that logic of adapting and filtering waypoints belongs to the internal of the agent. Maybe what’s needed instead is actually map observation that covers a broader area, map observation that includes adjacent lanes etc? @Gamenot : You should remember that there has been a long standing request for adding broader map observation (e.g. for ULTRA). I do feel this is a similar need. Need for map observation (with waypoints being one kind of map info) in the context of behavior and mission is the issue here. Let's have a design review first!!!

sure, i guess we should try to have the agent generate its path purely based on observation in the act method. If needed to access self._road_network could be given to the OpEnAgent class as part of requirement in OpEnAgent class.
One other thing is the OpEnAgent class Act method currently still have all the leftover code for using OpEn. We should remove these code

Why is it necessary for the AgentInterface to has AgentBehaviuor as part of it?

We have a few targets regarding the AgentBehaviour: controller, trajectory generator, Agent, Actor. The AgentBehaviour is intended to be an interface that translates into parameters for the controller and trajectory generators. Since these values are

Why should WaypointsSensor have information about custom behavior at all?

We have a systemic problem of mixing up waypoints and trajectories. The WaypointsSensor is inappropriately named or inappropriately used since its actual role is to generate trajectories [pose, speed]. A WaypointSensor should not have any influence by behaviour but perhaps a trajectory generator might. Our problem here is that the WaypointSensor should return to being just a waypoint sensor again but almost everywhere the result is used assumes trajectories instead. Perhaps the start of the mistake was generating the equally spaced waypoints at each step.

It seems to me that all that logic of adapting and filtering waypoints belongs to the internal of the agent. Maybe what’s needed instead is actually map observation that covers a broader area, map observation that includes adjacent lanes etc?

The WaypointSensor currently generates branching trajectories from the current lane. And then there is a separate RoadWaypointSensor which finds the nearby static waypoints on adjacent lanes (both forward and oncoming) as well as the static route waypoints (but I believe that the route waypoints also got polluted by the dynamic equally spaced waypoints generator.)

@Gamenot : You should remember that there has been a long standing request for adding broader map observation (e.g. for ULTRA). I do feel this is a similar need. Need for map observation (with waypoints being one kind of map info) in the context of behavior and mission is the issue here. Let's have a design review first!!!

OK.

We have a systemic problem of mixing up waypoints and trajectories. The WaypointsSensor is inappropriately named or inappropriately used since its actual role is to generate trajectories [pose, speed]. A WaypointSensor should not have any influence by behaviour but perhaps a trajectory generator might. Our problem here is that the WaypointSensor should return to being just a waypoint sensor again but almost everywhere the result is used assumes trajectories instead. Perhaps the start of the mistake was generating the equally spaced waypoints at each step since I believe those augmented waypoints are actually just an attempt to generate appropriate trajectories.

Indeed. We are overloading Waypoints multiple times too much. We need a proper map observation API that is independent of the agent's mission route/goal or its motion trajectory. Design and design review first, please.

https://github.com/huawei-noah/SMARTS/tree/feature-extract_behavior_models ended up quite similarly although it also highlights some of the mangled use of waypoints.


Is it reason to discard this DDL "ALTER TABLE ...EXCHANGE PARTITION " ?

What did you do?

Now TiDB support EXCHANGE PARTITION, but TiCDC discard this DDL
[2022/07/18 14:01:24.260 +08:00] [INFO] [schema_storage.go:252] ["discard DDL"] [jobID=270] [DDL="ALTER TABLE employees EXCHANGE PARTITION p3 WITH TABLE employees1"] [namespace=default] [changefeed=mysql-test1]

What did you expect to see?
No response
What did you see instead?
If downstream is TiDB, discard this DDL will make query result different between upstream and downstream.
Versions of the cluster
Upstream TiDB cluster version (execute SELECT tidb_version(); in a MySQL client):
(paste TiDB cluster version here)

Upstream TiKV version (execute tikv-server --version):
(paste TiKV version here)

TiCDC version (execute cdc version):
(paste TiCDC version here)
Release Version: v6.2.0-alpha
Git Commit Hash: abc120caaf6bae9c64e9ed345618ecf303950409
Git Branch: heads/refs/tags/v6.2.0-alpha
UTC Build Time: 2022-07-17 14:25:40
Go Version: go version go1.18.2 linux/amd64
Failpoint Build: false

Seems that this  DDL can be expessed by 'DML' between different tables.
https://dev.mysql.com/doc/refman/5.7/en/partitioning-management-exchange.html
We can test this DDL by the example on the official website.

It's a bug.
Need investigate why DML don't dealed by ticdc.

/assign @asddongmen


Exception: Failed to load domain specification from

Rasa Core version:
rasa-core==0.8.6
rasa-nlu==0.11.5
Python version:
2.7
Operating system (windows, osx, ...):
MacOS X
Issue:
Exception: Failed to load domain specification from '/Users/anuranjit/MySpace/LM/restraunt_bot/<rasa_core.domain.TemplateDomain object at 0x10f6486d0>'. File not found!
Traceback
 File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/run.py", line 151, in <module>
    cmdline_args.credentials)
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/run.py", line 130, in main
    agent = Agent.load(model_directory, nlu_model)
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/agent.py", line 66, in load
    return cls(domain, ensemble, featurizer, _interpreter, _tracker_store)
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/agent.py", line 44, in __init__
    self.domain = self._create_domain(domain)
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/agent.py", line 243, in _create_domain
    return TemplateDomain.load(domain)
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/domain.py", line 426, in load
    if not os.path.isfile(filename):

In the Supervised Learning Link tutorial
Also to add , in /Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/domain.py i returned the filename (which was a TemplateDomain object)
Now i got ,
Traceback (most recent call last):
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/run.py", line 151, in <module>
    cmdline_args.credentials)
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/run.py", line 135, in main
    agent.handle_channel(input_channel)
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/agent.py", line 126, in handle_channel
    processor.handle_channel(input_channel)
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/processor.py", line 60, in handle_channel
    input_channel.start_sync_listening(self.handle_message)
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/channels/console.py", line 52, in start_sync_listening
    self._record_messages(message_handler)
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/channels/console.py", line 45, in _record_messages
    self.sender_id))
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/processor.py", line 82, in handle_message
    self._handle_message_with_tracker(message, tracker)
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/rasa_core/processor.py", line 230, in _handle_message_with_tracker
    for e in self.domain.slots_for_entities(parse_data["entities"]):
  File "/Users/anuranjit/.virtualenvs/ml/lib/python2.7/site-packages/future/types/newstr.py", line 331, in __getattribute__
    return super(newstr, self).__getattribute__(name)
AttributeError: 'newstr' object has no attribute 'slots_for_entities'

Content of domain file (if used & relevant):

did you got the solution to this, i am facing the same issue.


About doc translation

I want to translate riverpod web to a Korean.
Are there any guidelines to follow for docs contribution? Or, can I just use docusaurus plugin and make a PR as usual?
Thanks

There's no particular guideline. Simply raise a PR that makes the necessary changes, and I'll accept it.


feature:  文档中心跳转路由改造 (closed #1975)

PR
CheckList（PR Assigners）
确认已完成以下操作：

[x] 已将分支与当前合入分支同步
[ ] 已在关联 Issue 中补充「功能自测」部分内容
[ ] 已更新关联 Issue 的状态 backlog -> todo -> for test -> tested（需通过测试同学测试） -> done
[ ] 将 PR 添加上 pr/reviewable

CheckList（PR Reviewers）
确认已完成以下操作：

[ ] 检查关联 Issue 「功能自测」部分内容
[ ] 必要时打回 pr/reviewable，要求重新修改
[ ] 及时 Approve

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (35d1ad1) 74.76% compared to head (fb36382) 74.68%.
Report is 4 commits behind head on v2.4.4-dev.

Additional details and impacted files
@@              Coverage Diff               @@
##           v2.4.4-dev    #2044      +/-   ##
==============================================
- Coverage       74.76%   74.68%   -0.08%
==============================================
  Files             404      404
  Lines           28188    28538     +350
==============================================
+ Hits            21075    21315     +240
- Misses           7113     7223     +110

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


maxList, maxListbBy, minList, minListBy

I was somewhat surprised to see that max[By] and min[By] are binary functions. I think unary-list variants would be much more useful, or maybe both options can be provided.

The problem is that R.maxList would be a partial function, since R.maxList([]) is undefined. Folding forces one to handle the empty list:
> R.reduce(R.max, -Infinity, [])
-Infinity
> R.reduce(R.max, -Infinity, [1, -2, 3, -4])
3

The function's curried nature is sometimes useful, too:
> R.map(R.max(0), [1, -2, 3, -4])
[ 1, 0, 3, 0 ]

The empty-list case is interesting in the context of ramda, because throwing isn't nice but returning a Maybe or an Either isn't an option.
The reduce implementation is nice, but I'm afraid it cannot be applied easily for implementing maxListBy, since you don't have -Infinity handy. For example, let's say you have a list of objects representing some sort of activities, each having a timestamp property, which an ISO-Date string. Now you wish to find the most recent activity. The by version would then be something like this
var toEpoch = pipe(moment, invoker("valueOf"));
pipe(prop("timestamp"), toEpoch);

But now what's you initial value for the reduce function? It cannot be -Infinity because if you pass that to maxBy, your by function ought to be branched to support that.

The problem remains that if these functions did not require an initial value, users would in many cases need guards to handle the empty list. For example:
R.isEmpty(xs) ? 0 : R.maxListBy(Math.abs, xs)

Instead we could have written:
R.reduce(R.maxBy(Math.abs), 0, xs)

The advantage of the latter is that we're using a total function rather than a partial function with a guard. The problem with guards is that it's easy to forget to include them, which can result in crashes at run time.
If one is able to determine that xs will never be [], one can use xs[0] as the initial value.

I acknowledge that. This is why I was referring to Maybe (which isn't not an option, unfortunately).
maxListBy doesn't have to be partial; it could take the fallback value (not that initial value - just the final fallback; this is similar to propOr which guards against null value, unlike prop).
Again, the problem with R.reduce(R.maxBy(Math.abs), 0, xs) is that the initial value isn't always in par with the fallback value, or it could be hard to provide. If i want the fallback to be null in the case of an object, i need to call reduce with an initial value for which the reducing function returns null.

The reduce approach certainly isn't perfect. I don't yet see a way to have our :cake: and eat it too. :)

If i want the fallback to be null in the case of an object, i need to call reduce with an initial value for which the reducing function returns null.

I don't quite follow (perhaps because I've stayed up very late writing code the past two nights). Could you provide an example?
By the way, for in-line code in Markdown single backticks are sufficient. ;)

I sometimes forget :) How did you notice?
So, let's go to my first example.
Here's a list:
var list = [
  { value: "foo", timestamp: "2015-08-13T06:22:39.501Z" },
  { value: "bar", timestamp: "2014-08-13T06:22:39.501Z" },
  ...
];

and now you want to find the most-recent entry in such list. If you know the list isn't empty, you could, indeed, do something like this:
var isoToEpoch = pipe(moment, invoker("valueOf"));
reduce(maxBy(pipe(prop("timestamp", isoToEpoch), list[0], list);

But if you don't know that the list isn't empty. You have to do it this way (assuming you want to return null in that case):
isEmpty ? reduce(...list[0]...) : null;

and that's because the initial value also goes through the function you pass to maxBy

Maybe is the answer, I believe:
> R.converge(R.lift(R.reduce(R.maxBy(x => moment(x.timestamp).valueOf()))), S.head, S.tail)(list)
Just({timestamp: "2015-08-13T06:22:39.501Z", value: "foo"})
> R.converge(R.lift(R.reduce(R.maxBy(x => moment(x.timestamp).valueOf()))), S.head, S.tail)([])
Nothing()

In the case of vanilla Ramda, I think we're stuck with <guard> ? ... : null.

I still need to catch up on lift :)
I think vanilla-ramda could just return undefined in this case (if such a function is provided). After all, it's probably the most appropriate use of `undefined. After all, the maximum value of an empty list is logically /undefined/.

I still need to catch up on lift :)

Conceptually lift and map are almost the same thing. R.map(fn) lifts a unary function to work on any functor (lists for instance).
R.equals(R.map(R.add(2))([1,2,3,4]),
         R.lift(R.add(2))([1,2,3,4])); //=> true

R.map takes a unary function. But you could also imagine a R.map2 that takes a binary function, an R.map3 that takes a ternary function and so on. If you make that into R.mapN you have the same as R.liftN. And R.lift just looks at the arity of the function you pass it and "selects" the correct map variant to use.

Probably not the best implementations, but this has worked for me:
import { pipe, split, toString, product, curry, maxBy, reduce, useWith, identity, head, tail } from 'ramda';

const NUMS = [415, 519, 145, 189];

const reduce1 = curry((fn, arr) =>
  reduce(fn, head(arr), tail(arr)));

// after http://hackage.haskell.org/package/base-<IP_ADDRESS>/docs/Data-List.html#v:maximumBy
// returns undefined for an empty list
const maximumBy = useWith(reduce1, maxBy, identity);

const numberProduct = pipe(
  toString, split(''), product);

maximumBy(numberProduct, NUMS);
// => 189

In our code base, I also chose the undefined behavior. I still think it's worth inclusion in the core lib. David's library can of course return Nothing in that case :)

What are your thoughts on this proposal, @CrossEye?

Trying to catch up on very old messages.

What are your thoughts on this proposal, @CrossEye?

I'm not quite sure what constitutes a proposal here.  The code from @raine?  That's almost exactly what I do when I want this sort of behavior.  I don't know that we'd want to reintroduce reduce1 (see #20) on the public API, but obviously we could do it internally if desired.
I'd be happy to have the *List[By?] functions.  I wasn't thrilled with the change from variadic (or list?) based max/min to binary ones, even if I saw the reasons for it.  It would be nice to have these back.  But I think I would prefer to insist that the user supply a reasonable initial value than try to infer it.  Thus:
var list = [
  { value: "foo", timestamp: "2015-08-13T06:22:39.501Z" },
  { value: "bar", timestamp: "2014-08-13T06:22:39.501Z" },
  ...
];

var isoToEpoch = pipe(moment, invoker("valueOf"));
reduce(maxBy(pipe(prop("timestamp"), isoToEpoch),
       {timestamp: "0000-00-00T00:00:00.0Z"}, list
);

But. then again, it has been a long time since #20, and @buzzdecafe and I were working blind, as the only ones working at all with the code-base, so perhaps it's time to reconsider reduce1.

the bigger question is whether we should integrate algebraic types, which i am starting to think might be worth doing. ramda-sanctuary merge?

closed for inactivity


No working

Hi I try pip3 install xmppwb  and look install xmppwb

Requirement already satisfied (use --upgrade to upgrade): xmppwb in /usr/local/lib/python3.4/dist-packages Requirement already satisfied (use --upgrade to upgrade): aiohttp in /usr/local/lib/python3.4/dist-packages (from xmppwb) Requirement already satisfied (use --upgrade to upgrade): pyyaml in /usr/local/lib/python3.4/dist-packages (from xmppwb) Requirement already satisfied (use --upgrade to upgrade): slixmpp in /usr/local/lib/python3.4/dist-packages (from xmppwb) Requirement already satisfied (use --upgrade to upgrade): chardet in /usr/local/lib/python3.4/dist-packages (from aiohttp->xmppwb) Requirement already satisfied (use --upgrade to upgrade): aiodns>=1.0 in /usr/local/lib/python3.4/dist-packages (from slixmpp->xmppwb) Requirement already satisfied (use --upgrade to upgrade): pyasn1 in /usr/local/lib/python3.4/dist-packages (from slixmpp->xmppwb) Requirement already satisfied (use --upgrade to upgrade): pyasn1-modules in /usr/local/lib/python3.4/dist-packages (from slixmpp->xmppwb) Requirement already satisfied (use --upgrade to upgrade): pycares>=1.0.0 in /usr/local/lib/python3.4/dist-packages (from aiodns>=1.0->slixmpp->xmppwb)

But by example I do xmppwb --help and I got:
 File "/usr/local/bin/xmppwb", line 9, in <module>     load_entry_point('xmppwb==0.2.5', 'console_scripts', 'xmppwb')()   File "/usr/lib/python3/dist-packages/pkg_resources.py", line 351, in load_entry_point     return get_distribution(dist).load_entry_point(group, name)   File "/usr/lib/python3/dist-packages/pkg_resources.py", line 2363, in load_entry_point     return ep.load()   File "/usr/lib/python3/dist-packages/pkg_resources.py", line 2088, in load     entry = __import__(self.module_name, globals(),globals(), ['__name__'])   File "/usr/local/lib/python3.4/dist-packages/xmppwb/core.py", line 17, in <module>     from xmppwb.bridge import XMPPWebhookBridge, InvalidConfigError   File "/usr/local/lib/python3.4/dist-packages/xmppwb/bridge.py", line 87     async def handle_outgoing(self, outgoing_webhook, msg):             ^ SyntaxError: invalid syntax
Any ideas how to solve that

As mentioned in the readme xmppwb requires Python 3.5+ but you seem to have Python 3.4. So you need to upgrade Python first.


Support for http streaming audio?

Is your feature request related to a problem? Please describe.
I actually need a feature to setup a streaming for a web radio, which have an https url (https://onair20.xdevel.com/proxy/nslradiotv?mp=/;stream)
Describe the solution you'd like
All actually present features but with possibility to setup an http url as of assetPath
Describe alternatives you've considered
Cordova Plugin media-streaming

duplicate of https://github.com/capacitor-community/native-audio/issues/16


server.js fails on WebStorm 12 EAP with node version 3.8.0 with message:

No longer relevant, dropped Node backend from project


Setting php namespace in proto fails

Setting the option php.namespace in proto files fails with Option "php_package" unknown. The option is described in the repo here. I'm using protoc-gen-php to generate the proto files.
I have tried these as the name of the option, all of them get similar error messages though.

php.namespace
php_namespace
namespace
php.package
php_package
package

I'm trying to achieve the same result as the java option https://github.com/grpc/grpc-go/blob/master/examples/helloworld/helloworld/helloworld.proto#L33 here.

gRPC has switched to use the official Google protobuf implementation since v1.1.0 has been released. Please check whether this option is supported there. If you need this option, please file an enhancement request to that repo.

Implemented in https://github.com/grpc/grpc/pull/11886.


Multiline and returnLabelType="send' not working together?

Description
On android, we unfortunately cannot have the returnLabelType="send" when using multiline.
So it makes it impossible to replicate a "snapchat-like" textinput for example, where we can send our message from the keyboard while having the multiline in place as we type.
There was this old issue about the same: https://github.com/facebook/react-native/issues/33193
and for strange reason it was marked as "resolved" by facebook team when it's not?
Steps to reproduce
1- add prop multiline to textinput
2- add prop returnLabelType='send' to textinput
3- returnlabeltype is ignored
React Native Version
0.73.5
Affected Platforms
Runtime - Android
Output of npx react-native info
System:
  OS: Windows 11 10.0.22631
  CPU: (20) x64 12th Gen Intel(R) Core(TM) i7-12700KF
  Memory: 20.26 GB / 31.86 GB
Binaries:
  Node:
    version: 18.18.2
    path: C:\Program Files\nodejs\node.EXE
  Yarn: Not Found
  npm:
    version: 9.8.1
    path: C:\Program Files\nodejs\npm.CMD
  Watchman: Not Found
SDKs:
  Android SDK: Not Found
  Windows SDK: Not Found
IDEs:
  Android Studio: AI-221.6008.13.2211.9514443
  Visual Studio:
    - 17.8.34309.116 (Visual Studio Community 2022)
Languages:
  Java: 11.0.18
  Ruby: Not Found
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.73.5
    wanted: ^0.73.5
  react-native-windows: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: Not found
  newArchEnabled: Not found

Stacktrace or Logs
no crash/error

Reproducer
https://github.com/react-native-community/reproducer-react-native
Screenshots and Videos
No response

Hi @pierroo,
It appears you meant returnKeyType instead of returnLabelType, as it isn't documented or present in the codebase. To achieve your desired functionality, you can set the submitBehavior prop with either submit or blurAndSubmit values on the TextInput component.

<TextInput
  multiline={true}
  returnKeyType="send"
  submitBehavior="submit"
/>

This prop isn't documented, but has been introduced in #33653 and available since version v0.71.
type SubmitBehavior = 'submit' | 'blurAndSubmit' | 'newline'

It works perfectly, thank you so much @jakex7 !


Allow any sender

Instead of relying only on the list of granted phones, we could allow anyone to send commands through messages.  This can be achieved by allowing the user of the application to set a password in the application itself.  I explain it by an example.  Suppose the password I set is "cao".  If I want to get my phone location, I can send
rc get location
using a granted phone, or I can send something like
rc cao get location
using any phone.
This should be secure, because only the user of the application knows the password.
This could be useful in situations like:
-->  The owner of the granted phone changed number and I forgot to update the list of granted phones in the application
-->  I lost my phone and I am alone.  I can't contact any of the owners of the granted phones because I don't have a phone.  But I can ask help to a random stranger.

I would rephrase this as: implement password based authentication for remote commands.

I think the use cases described here can be done with the new feature/module "Grant Phone Remotely".


Change kubeconfig current context through the menu

It would be really convenient to have a dropdown menu where you can select the kubeconfig context that you want to use.
There is already a dedicated app to do that, but it would be nice to also have it: https://github.com/nsriram/KubeSwitch

Was done in https://github.com/code-ready/tray-macos/pull/107


Address form update

country changed to za from pl

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


Support Resume from Replay

Totally failing right now.

I found this issue with resumed replays. Haven't dug into it much but here is a stacktrace and its pretty obvious what file it blew up on:
 (master * u=)$: python sc2analyzer.py '../WCS 2015 Season 1/SSL/Challenge/Roundof32_Classic_Cure_1Set(After Recover).SC2Replay'
Traceback (most recent call last):
  File "sc2analyzer.py", line 48, in <module>
    main()
  File "sc2analyzer.py", line 32, in main
    replay = sc2reader.load_replay(filename, debug=True)
  File "/home/nibz/projects/sc2replay_proc/venv/local/lib/python2.7/site-packages/sc2reader/factories/sc2factory.py", line 85, in load_replay
    return self.load(Replay, source, options, **new_options)
  File "/home/nibz/projects/sc2replay_proc/venv/local/lib/python2.7/site-packages/sc2reader/factories/sc2factory.py", line 137, in load
    return self._load(cls, resource, filename=filename, options=options)
  File "/home/nibz/projects/sc2replay_proc/venv/local/lib/python2.7/site-packages/sc2reader/factories/sc2factory.py", line 146, in _load
    obj = cls(resource, filename=filename, factory=self, **options)
  File "/home/nibz/projects/sc2replay_proc/venv/local/lib/python2.7/site-packages/sc2reader/resources.py", line 309, in __init__
    engine.run(self)
  File "/home/nibz/projects/sc2replay_proc/venv/local/lib/python2.7/site-packages/sc2reader/engine/engine.py", line 174, in run
    for new_event in (event_handler(event, replay) or []):
  File "/home/nibz/projects/sc2replay_proc/venv/local/lib/python2.7/site-packages/sc2reader/engine/plugins/context.py", line 228, in handleUnitDoneEvent
    self.logger.error("Unit {0} done at {1} [{2}] before it was started!".format(event.killer_pid, Length(seconds=event.second), event.frame))
AttributeError: 'UnitDoneEvent' object has no attribute 'killer_pid'


Unable to Add Certain Games via URL

Describe the bug
When trying to add certain games, the program is unable to find them. When then proceeding to try and add them via URL it will popup with "Adding game from url" and then stop. I've noticed that this only happens on games with 0 reviews on f95.
To Reproduce
Steps to reproduce the behavior:

Attempt to add a game with 0 reviews via URL
Choose the folder on computer
Input the url the game is located

Hi @Satir12, thanks for the feedback!
I investigated the problem: this happens because some information about the game is present in a (hidden) field on the page that exists only in the presence of reviews.
This problem is actually the fault of the APIs (here) that I have developed to extrapolate the data of the platform, an API that I have completely rewritten in the last few months.
Since I intend to rewrite YAM around September and the fix for this problem would take me some time (which unfortunately I don't have right now), I would recommend that you wait for someone to write a review on the game in question and then try to add it.
It is not exactly the best of solutions but right now it is the only thing that comes to me to suggest to you.


the libary dot't work  when i set css_moudles

thanks for your libary ,In most cases it works fine.  in react v^16.5.2 , i set css_moudles, if i don't set  ::-webkit-scrollbar  Series of property, it's dose'nt work; beccause scrollbars allways display none in css

￼￼


How to configure ES7 support?

I can't wait (!) to use async/await features offered by ES7.
I apologize for my simple question: What is the preferred way to add ES7 support to the vuepack config?

Most modern browsers support async/await by default. If you need to support older browsers, consider using babel-plugin-transform-async-to-promises because this plugin does not use regenerator-runtime which will bloat the bundle size.


half-body image effect is poor

The half-body image effect is poor, and the effect is still not good after padding. Do you have any suggestions?

Could you please provide an example image, as well as the image after padding, and the result image with the mesh overlay?


Camera zoomed in

Why is camera zoomed in, once scanning controller is loaded?

Hi @alexandremne,
I found some time today to look into this.
TLDR: The image isn't zoomed, but the layers' aspect ratios are different.
Compare the images from the system camera and the scanner:

iPhone camera
Scanner

You'll notice that the aspect ratio of the scanner is slightly taller, so more of the layer will be cropped off at the left and right edges. The image will be larger because of this.
There's nothing really we can do about this other than changing the UI so the layer has a similar aspect ratio as the camera.
I think we're open to this option if you'd like to submit a pull request :+1:.
The resizing strategy is configured here.

Oh, I get it. Ok, thank you for clarification. I will take a look and get back to you if I come up with a solution.

:+1: That sounds nice. I'll close the issue, but feel free to reopen if you like.


NPM install fails

NPM install is failing because the package.json refers to bin/commands.js which is not part of the repository.
Error message:
npm ERR! Windows_NT 10.0.14965
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\nick\\AppData\\Roaming\\npm\\node_modules\\npm\\bin\\npm-cli.js" "install"
npm ERR! node v7.2.0
npm ERR! npm  v4.0.3
npm ERR! path C:\Users\nick\Source\Repos\foobar\node_modules\dynamicsnode\bin\commands.js
npm ERR! code ENOENT
npm ERR! errno -4058
npm ERR! syscall chmod

npm ERR! enoent ENOENT: no such file or directory, chmod 'C:\Users\nick\Source\Repos\foobar\node_modules\dynamicsnode\bin\commands.js'
npm ERR! enoent ENOENT: no such file or directory, chmod 'C:\Users\nick\Source\Repos\foobar\node_modules\dynamicsnode\bin\commands.js'
npm ERR! enoent This is most likely not a problem with npm itself
npm ERR! enoent and is related to npm not being able to find a file.
npm ERR! enoent

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\nick\Source\Repos\foobar\npm-debug.log

Suggested resolution ideas:

Allow the compiled files to be a part of the repository?
Explore how other node modules built with typescript are shipped.. is this standard?
Change the bin to use the typescript file instead?

I will explore these options, but it may be difficult to test.

Attempting to duplicate this fails. It appears to happen only when installing using a github install.
Please hold off while I investigate further.

Confirmed when installing via npm install "crisfervil/DynamicsNode#master"

I don't think the 3rd option would work.
To be honest, I've never tried to install it from github.
Why do you want to do this? If you want to test a modified version you are working on, maybe is easier to use the npm link command https://docs.npmjs.com/cli/link

Let's close this ticket, looks like I can install from a local directory.
You've been so incredibly helpful with these tickets. Thank you!

Thank you @nhance


Python editor messes up whitespace and line endings, causes numerous merge conflicts

This issue has been moved from a ticket on Developer Community.

I urge you not to dismiss this as a cosmetic issue because this is a significant problem that triggers numerous merge conflicts as different people open these messed up files and their IDE's fix the errors silently, including Visual Studio in some cases.
When the source file is set up to use LF line endings, pressing Enter when the caret is not at the very end of the line inserts CRLF. Only Python editor does this - TypeScript and C++ work fine. When the same file is opened in Visual Studio again, it asks whether mixed line endings should be fixed (may also be silent), causing conflicts with other people working with the same source files.

Similarly, just typing source code inserts those empty indentations, which other editors correct, also causing merge conflicts. Again, TypeScript and C++ editors don't exhibit this behavior.
Please, look into this. I lost count how many times pulling from a repository ended up in a conflict because of these whitespace changes.

Original Comments
Feedback Bot on 7/9/2024, 06:40 PM:
(private comment, text removed)

Original Solutions
(no solutions)

This is a duplicate of https://github.com/microsoft/PTVS/issues/7598. I agree it could be causing some trouble and definitely needs attention. Meanwhile, can you try navigating to Tools -> Options -> Environment -> Documents and disabling Check for consistent line ending on load as a temporary workaround?


i18n(zh-cn): fix inconsistencies with the en doc [i18nIgnore]

Description (required)
Update zh-cn content to keep consistent with en.
Related issues & labels (optional)

Closes #
Suggested label: i18n

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any broken links you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Vercel 🥳

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.

Lunaria Status Overview
🌑 This pull request will not trigger status changes.

Learn more
Lunaria automatically ignores changes on specific PRs by adding a ignored keyword in its title. Found: i18nIgnore.
You can change this by either removing the keyword above from the PR's title, or modifying the ignoreKeywords property in your Lunaria configuration file.

Tracked Files

Locale
File
Note

zh-cn
tutorial/4-layouts/1.mdx
Localization changed, will be marked as complete.

zh-cn
tutorial/6-islands/2.mdx
Localization changed, will be marked as complete.

zh-cn
tutorials/add-view-transitions.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


[bug] LazyFormatted values don't get evaluated properly when they are not strings

Describe the bug
Values using @format don't get evaluated properly in data structures that are not strings.
Instead the returned value is <dynaconf.utils.parse_conf.LazyFormat object at 0x105737a90>.
To Reproduce
Steps to reproduce the behavior:

Having the following config files:

In settings.toml:
[default]
VAL = "val"
TEST = ["a", "@format {this.VAL}]

Having the following app code:

/path/src/app.py
from dynaconf import settings
print(settings.TEST)
...

Expected behavior
The result should be ["a", "val"].
Instead the second value doesn't get lazily evaluated and I get: ["a", <dynaconf.utils.parse_conf.LazyFormat object at 0x105737a90>]
To get the value of the second element, I would have to do print(settings.TEST[1](settings))
Environment (please complete the following information):

OS: MacOS

fixed on #405


Regenerate protos

This PR, along with google/trillian#584 gets us back on track for being able to generate protos correctly.
With regard to the grpc-gateway runtime.AnnotateContext issue:

etcd, which trillian uses, vendors grpc-gateway to it's latest released version v1.2.2
Trillian generate pb.gw.go file accordingly
Which means KT must do the same.
It's a good idea to use release versions rather than HEAD.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #705   +/-   ##
=======================================
  Coverage   57.74%   57.74%
=======================================
  Files          38       38
  Lines        2530     2530
=======================================
  Hits         1461     1461
  Misses        778      778
  Partials      291      291

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 2defcd2...23b1205. Read the comment docs.


instancetype: Reject Matcher updates without updating RevisionName

/area instancetype
What this PR does / why we need it:
As set out in issue #9132 requests to update the Name of an {Instancetype,Preference}Matcher were always accepted regardless of the RevisionName also being updated. This would result in an older and likely unwanted ControllerRevision being used containing an outdated or completely wrong Instancetype or Preference.
With this change the VM mutation webhook will now reject requests to update the target Name of a {Instancetype,Preference}Matcher without also updating the RevisionName.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #9132
Special notes for your reviewer:
Release note:

 Requests to update the target `Name` of a `{Instancetype,Preference}Matcher` without also updating the `RevisionName` are now rejected.

/cherrypick release-0.59

Thanks!
/lgtm
/retest

/retest-required


LUCENE-10502: Use IndexedDISI to store docIds and DirectMonotonicWriter/Reader to handle ordToDoc

follow up of https://github.com/apache/lucene/pull/792 and https://github.com/apache/lucene/pull/870

Hi, @mayya-sharipova , such merge operation is new to me, I am not sure if I could add an entry to CHANGES.txt correctly, so I could add entry after this PR merged or could you be nice to help me out.

@LuXugang Right,  you can add an entry to CHANGES.txt after this PR is merged.

@LuXugang I've merged your PR to main.
Other things we need to do:

Please create a PR against main add an entry to CHANGES.txt  under the Lucene 9.2.0 release.
After 1) is merged,  we need to cherry-pick all these changes to 9.x branch.

@mayya-sharipova , @jtibshirani  Thanks for your reviews!

Please create a PR against main add an entry to CHANGES.txt under the Lucene 9.2.0 release.

Addressed in https://github.com/apache/lucene/pull/881.


volumeAvailableCapacityForImportantUsageKey returns 0 when no user is logged in

Thanks so much for updating mist-cli to use volumeAvailableCapacityForImportantUsageKey instead of systemFreeSize to properly calculate free space in a way that matches Finder!
But, there is one odd quirk with volumeAvailableCapacityForImportantUsageKey which is that it returns 0 when no user is logged in. I don't think this means volumeAvailableCapacityForImportantUsageKey should not be used since it is the better/correct thing to use when a user is logged in, but if someone tries to run mist-cli in a scripted way when no user is logged in there would need to be a fallback since currently mist-cli would not be able to download anything since it would always think there is no free space when there could be.
I would suggest falling back on volumeAvailableCapacityKey which matches what systemFreeSize returns and works properly when no user is logged in. Since you're already using let values: URLResourceValues = try url.resourceValues(forKeys: [.volumeAvailableCapacityForImportantUsageKey]) you can easily add .volumeAvailableCapacityKey to the array of keys you're retrieving and then check if values.volumeAvailableCapacityForImportantUsage is 0 and use values.volumeAvailableCapacityKey instead if so to properly get some free space value whether or not a user is logged in.
Here is some more info about this issue this same described solution with code in JavaScript for Automation (JXA): https://macadmins.slack.com/archives/C0619KJQY/p1663642814450849

I have a fix in https://github.com/ninxsoft/mist-cli/pull/130, @PicoMitchell if you have time I would love some fresh eyes to test and confirm everything is working as expected 🙏

I believe that new code would use the volumeAvailableCapacityKey when Mist or running as root even if a user is actually logged in when volumeAvailableCapacityForImportantUsage could be used.
I would recommend not checking the running user or anything and instead always retrieve both values and then check if volumeAvailableCapacityForImportantUsage is 0 and fallback on volumeAvailableCapacityKey if so. Doing it that way would guarantee always using volumeAvailableCapacityForImportantUsage when it has been properly loaded.

Here is a JXA example of how volumeAvailableCapacityForImportantUsage still works properly when run as root as long as a user is currently logged in:
sudo osascript -l JavaScript -e 'console.log($.NSUserName().js); ObjC.deepUnwrap($.NSURL.fileURLWithPath("/").resourceValuesForKeysError(ObjC.wrap([$.NSURLVolumeAvailableCapacityForImportantUsageKey]), ObjC.wrap())).NSURLVolumeAvailableCapacityForImportantUsageKey'

When run from Terminal, that will output "root" for the NSUserName since the osascript command is run as sudo and then also output the current available space from the volumeAvailableCapacityForImportantUsage key.

@PicoMitchell thanks for the info, works for me!


DOC:Correct the image in the provided

What's wrong with the existing documentation

Add ScreenShots
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Can u pls add the level label also mam


Merge branch 'release/3.0-preview6' -> 'master'

Gave up on #15943 after messing up the bot's branch. Let's do the no-op merge manually.

Ping reviewers. All checks have passed.


Cleaned up includes. Removed duplicate code in LoadHits(). Fixed bug …

…where two PFPs in different cryostats are assigned the same color/grouping. Moved global vars to plot_tools.h

good for pull

Good


Promisify all the methods in Redis

Can I use the following code to promisify all the methods inside Redis and will they work?
const redis = require('redis');
const client = redis.createClient();
const { promisify } = require('util');
const asyncClient = {};

for(let fn in client.__proto__) {
    if (client.__proto__[fn]) {
        asyncClient[fn] = promisify(client[fn]).bind(client);
    }
}

why not try it and find out!


Power World License Interactions and selecting license type

Using limited licenses, can lead ESA to using a license when the add-on isn't needed.
For example, having a PW normal & transient license, but only steady state power flow is needed for ESA calculations. When running ESA the transient license is assumed to be taken, so anyone else using the same group of licenses would be unable to run transient (assuming 1 transient and multiple base PW licenses).
Basically adding choose Add-ons (from the selection of choose add-ons when opening PW) in the classes.saw.SAW() command would be the recommendation to fix this issue.

It has nothing to do with ESA. You should register the SimAuto with the proper license.


Develop G35 interface

The 63-bulb limitation is physical, but it shouldn't affect development of LightPrograms. Develop a G35 abstract class that's implemented by the current G35 concrete class, then adapt the programs to use that interface. This will make it easy to implement a G35 aggregator, which takes multiple strands and presents them as a single logical strand to the programs.
TL;DR: it'd be cool to be able to string together multiple strands and have them act as one.

you are doing AMAZING work Thankyou on behalf of everyone who is trying to use this for all your hard work!

Thanks, Curt! Only two days left before this crosses over from Christmas-appropriate activity into obsession!

G35StringGroup has been implemented and is working fine.


[gardening] Two small refactorings

First commit refactors the move diagnostic emission code in the move function address checker into its own function instead of inline in preparation for adding some more complex diagnostics via pattern matching.
I found an inline instance of the function getDebugVarName() and just replaced the inlined version with a call to the out of line function.

@swift-ci smoke test

@swift-ci smoke test linux platform


Open Source LLMs using ollama

🐛 Bug Report
🔎 Describe the Bug
Give a clear and concise description of the bug.
I tried to run the open source model using ollama but in the AgentsOps dashboard it is not posting the llm calls. Can you fix this for any open source LLM used either through ollama or Groq or some other way.
🔄 Reproduction Steps
List the steps to reproduce the behavior.
🙁 Expected Behavior
Describe what you expected to happen.
Integrate the open source LLMs also to post the data of what's happening under the hood.
📸 Screenshots
If applicable, add screenshots to help explain the problem.
🔍 Additional Context
Provide any other context about the problem here.
Thank you for helping us improve Agentops!

Hey, Ollama support was added recently. You can use it by pulling the github repo and installing the manually to get the latest changes.
To install the latest version of agentops, you can follow these steps

Clone the project

git clone git@github.com:AgentOps-AI/agentops.git

cd agentops
pip install -e .

Hey @sprajosh I'm also currently reviewing this bug as well, I'll let you know if there's anything I find

@albertkimjunior I think you misunderstood what I said. I have already added Ollama support. Since the version has not been updated, I was suggesting maybe pull the code from github and install the latest package manually.
Or, is there a bug in the ollama models?

@sprajosh No I understood. I am also looking at the Ollama support pull request that @siyangqiu approved to double-check if there are any possible issues.

Ollama support was merged to the main branch, but it is not currently released to pypi. There is an unrelated bug blocking the release ☹. We'll have it released shortly

Ollama support is release in v0.2.5


Allow user to also create namespaces when creating a project

Currently when setting up a project the user has to complete the initial set of requirements, save, and then one by one add namespaces in a separate view.
It would be a nice improvement to allow the user to specify one or more namespaces in the create project view to also create.
Notes

Would need to think of an alternative to tabs in tabs (namespace create shows two tabs - Container Resource Limit and the usual labels/annotations)
Namespace limits supplied in the project should be handled by namespace container limits (defaults, collectively exceeding limits)
Would need to nicely communicate project save success but specific namespace failure

+1 to this

This is pretty useful (the UX is normally always create projects --> create namespaces), however difficult to neatly present.
Create Project

Create Namespace

Could something like the new workloads and multi-container ux could work here too? A horizontal tab containing allowing the user to add zero to many namespaces?


Fixup windows stdlib path handling

StackFrames seem to contain the paths from the buildbot if they are from a stdlib. Replace them with the real stdlib path and avoid using regex for it so we don't have to escape windows path separators. Fixes #6.
@jonniedie if you're able to confirm this fix that would be great, thanks.

It works! Thanks!

Thanks for checking.


Some new images are keeping embeds and text URL

Looks like it might be just new images? I don't know how

Fixed; see #138


fix: regex lightning is surrounded by period

What does this PR do?
allows foo-lightning.my.salesforce.com login urls
@W-13639832@
What issues does this PR fix or reference?
https://github.com/forcedotcom/cli/issues/2241

QA Notes

before
 ➜  sfdx org:login:web --instance-url="https://mycompanyname-lightning.my.salesforce.com"
Error (1): Invalid instance URL. It should NOT be a lightning domain.

Try this:

If you specify an --instance-url value, this value overrides the sfdcLoginUrl value in your sfdx-project.json file.

To specify a My Domain URL, use the format https://yourcompanyname.my.salesforce.com.

To specify a sandbox, set --instance-url to https://MyDomainName--SandboxName.sandbox.my.salesforce.com.

✅ :  after opens a browser with that domain

✅ : reads from sfdx-project.json correctly
  "sfdcLoginUrl": "https://mycompanyname-lightning.my.salesforce.com",
✅ : deploy a new domain in a scratch org with "lightning" in the name, and then do a round trip org web login

This is still not working for domains like https://company—uat.sandbox.lightning.force.com. I am still getting the same error that domains with lightning text are not allowed.

@melvincanafranca can you please  open a new issue? https://github.com/forcedotcom/cli/issues/new?assignees=&labels=investigating&projects=&template=bug_report.md&title=
make sure to follow the template so we can ensure you are using a version that includes this fix 👍🏼 .


Bug: "Preview Table in VS Code" does not use bigqueryRunner.tree.projectIds with multiple projects

Thanks for all your work and providing this super helpful extension :)
While using it today I found the following bug:
Previewing tables in VSCode (via Bigquery Runner: Preview Table in VS Code) works well for all tables in the main project (i.e. set in bigqueryRunner.projectId).
However, when the table is in an alternative project from bigqueryRunner.tree.projectIds, it seems that the extension is looking for the tables in the project set in bigqueryRunner.projectId, rather than using the project set in bigqueryRunner.tree.projectIds and thus returns an error if the table is not found or it could even return the wrong table if a table with the same name exists in the main project.

@dzion Thanks for using BigQuery Runner.
bigqueryRunner.tree.projectIds is used only for tree display.
The use of bigqueryRunner.projectId when executing the query is as specified.
If you want to query for datasets in a project other than bigqueryRunner.projectId, you can explicitly specify the projectId in the query, such as select * from projectId.datasetId.tableId.

Hi @minodisk,
I think we might be talking about different things.
I was indeed referring to the tree display (where you can execute the preview table in VS Code command from).
The tree view uses the correctly specified bigqueryRunner.tree.projectIds to show datasets, tables and their schemas. However, when you click on a table in the tree view in a different project and try to preview a table, the query is send against the project set in bigqueryRunner.projectId, rather than the project in the tree.
Anyways, if this is your expected behavior, than maybe disabling the preview table in VS Code for the tree viewer for tables not in the main project would be a good idea, as it doesn't work currently.

@dzion Thank you for sharing the detailed situation. I think it is totally a bug and I will fix it when I have more time.

Fixed in v1.21.11.

Works perfectly on my end too.
Many thanks 💪 🥳


Storing RASH documents via browser

By using the Core layer of RAJE (i.e. by opening a RASH document that includes the editor script with a browser) it should be possible to save the RASH document edited by means of the usual save button of the browser. There are two possible alternatives (in order of preference):

we store the RASH_source HTML by intercepting the save event in the browser and by subtituting the HTML page visualised with the RASH_source;
we store the RASH_view (which is visualised) adding a meta tag saying explicitly that the HTML document is a RASH_view file.

In the latter case, RAJE core is expected to recognise the RASH_view file, and to convert it back as a RASH_source (storing it in an appropriate variable). If this option will be chosen, then it should result in appropriately extending ROCS so as to handle the case of submitting RASH_view documents.

Using Chrome
All TinyMCE needed plugins aren't added in the result package


MP dolichol-linked oligosaccharide biosynthetic process (GO:0006488)

shouldn't
dolichol-linked oligosaccharide biosynthetic process (GO:0006488)
have the parent
N-linked glycosylation (GO:0006487)????

http://www.thefog.ca/research.html

I am picking up a few mappings from various sources to GO:0006487 for
dolichol-linked oligosaccharide biosynthetic process (GO:0006488)
and that doesn't seem incorrect.

I guess it depends on how much you want part_of to cover. The oligosaccharides get built up on dolichol and then transferred to the substrate protein, so one could argue that GO:0006488 precedes and is required for (but ends before) N-linked glycosylation.

Well do they have any other purpose except for glycosylation (this is referring to the process rather than the specific MF of transferring the  dolichol-linked oligosaccharide.
I guess it comes down to what users would expect to see?

Wikepedia (I know it isn't authoititve but its a starting point)
descibes the process  biosynthesis of N-linked glycans occurs via 3 major steps

Synthesis of dolichol-linked precursor oligosaccharide
En bloc transfer of precursor oligosaccharide to protein
Processing of the oligosaccharide

Reviews seem to agree with this.


generate less pages on centos 7

windows:
$ hugo version
Hugo Static Site Generator v0.55.6-A5D4C82D windows/amd64 BuildDate: 2019-05-18T07:57:00Z
$ hugo
Building sites … WARN 2019/07/19 10:32:29 found no layout file for "HTML" for "section": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:29 Page's .Hugo is deprecated and will be removed in a future release. Use the global hugo function.

                   | EN
+------------------+----+
  Pages            | 41
  Paginator pages  |  0
  Non-page files   |  0
  Static files     | 10
  Processed images |  0
  Aliases          | 14
  Sitemaps         |  1
  Cleaned          |  0

Total in 59 ms

centos 7
Hugo Static Site Generator v0.55.6 linux/amd64 BuildDate: 2019-05-20T08:32:58Z
Building sites … WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomyTerm": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "section": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomyTerm": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "home": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "page": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "page": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "page": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "page": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "page": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "page": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "taxonomy": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.
WARN 2019/07/19 10:32:34 found no layout file for "HTML" for "page": You should create a template file which matches Hugo Layouts Lookup Rules for this combination.

                   | EN
+------------------+----+
  Pages            | 17
  Paginator pages  |  0
  Non-page files   |  0
  Static files     |  3
  Processed images |  0
  Aliases          |  0
  Sitemaps         |  1
  Cleaned          |  0

Total in 113 ms

windows:41 centos: 17

Thanks for getting in touch, but it feels like this is a question  that would be better suited to the Hugo Discussion Forum.  We prefer to use GitHub issues only for bugs and enhancements. Feel free to update this issue with a link to the re-posted question (so that other people can find it) or add some more details if you feel this is a genuine bug. You may also find the answers you search for in Hugo Documentation.


Revised: New lib and test files for proximity sensor interface on Tock kernel

Accidentally shut down the first pull request. Sorry!  I have revised the changes suggested by @ppannuto and have ran ./format_all.sh and reverted changes to the simple-ble library.
Added: proximity.c , proximity.h , tests/proximity/main.c
Modified examples/sensors/main.c
Tested with the kernel here: https://github.com/arjundeopujari/tock-1.git branch nano33-apds9960 (should be in tock/tock master in a while)

Absolutely I can.  Thanks for taking the time to review!

@ppannuto Done.

@ppannuto  No problem.  It is always a pleasure working on a great new Open Source IoT project.


drivers: modem: hl7800: fix file open in fw update

File open function for firmware update requires a flags parameter to be passed in.

Fixes https://github.com/zephyrproject-rtos/zephyr/issues/28745

Fix is fine, but we should add a build test to tests/drivers/build_all

Sounds good. What are your thoughts on how to approach this?

Add a board file overlay in the boards folder? (pinnacle_100_dvk can be used to test the HL7800 driver).
Add a new hl7800.conf that exercises all HL7800 driver options? testcase updated to use hl7800.conf with board pinnacle_100_dvk.

Sounds good. What are your thoughts on how to approach this?

Add a board file overlay in the boards folder? (pinnacle_100_dvk can be used to test the HL7800 driver).
Add a new hl7800.conf that exercises all HL7800 driver options? testcase updated to use hl7800.conf with board pinnacle_100_dvk.

I don't know if we can get away with doing something similar to i2c.dtsi called modem.dtsi and have the hl7800, ublox-sara-r4, and wncm14a2a in there.  Than have a modem.conf that enables those drivers (similar to ethernet.conf).

Change looks good, and I agree with @galak we should have a build test of this driver.

Sounds good. What are your thoughts on how to approach this?

Add a board file overlay in the boards folder? (pinnacle_100_dvk can be used to test the HL7800 driver).
Add a new hl7800.conf that exercises all HL7800 driver options? testcase updated to use hl7800.conf with board pinnacle_100_dvk.

I don't know if we can get away with doing something similar to i2c.dtsi called modem.dtsi and have the hl7800, ublox-sara-r4, and wncm14a2a in there. Than have a modem.conf that enables those drivers (similar to ethernet.conf).

Due to the driver relying on a UART and multiple IO, couldn't see how to do it like that. Added the test for the HL7800 driver and made use of the pinnacle_100_dvk because it already supports the HL7800.

Change looks good, and I agree with @galak we should have a build test of this driver.

@mike-scott get I get a re-review, thanks!

@galak can I get a second look and get this merged?

can you rename modem_hl7800.conf to modem.conf

are you sure? the settings in there are specific to the hl7800 modem. Different modems may have different settings.

are you sure? the settings in there are specific to the hl7800 modem. Different modems may have different settings.

I am, as I can follow up this PR w/having the test build support for multiple modems at the same time.

@rerickson1 / @mike-scott here's a PR based on the build_all change in here that enables additional modem support:
https://github.com/zephyrproject-rtos/zephyr/pull/29211
I would like to squash it down to one commit for the build_all changes, and one for the build bug fix, but left it as is for now for review purposes (and signed-off-by ack if I squash it all down) from @rerickson1 .


multiple files named [liability.launch] in package [robonomics_liability]

After
nix build -f release.nix
. ws/devel/setup.bash
roslaunch robonomics_liability liability.launch

on NixOS or if used Nix package manager, appear trouble below:
[de@nixos:~/robonomics_comm]$ roslaunch robonomics_liability liability.launch
RLException: multiple files named [liability.launch] in package [robonomics_liability]:
- /home/de/robonomics_comm/result/share/robonomics_liability/launch/liability.launch
- /nix/store/sh8ljpq5wy1vqmp9z4adb9ck7xfi6b42-robonomics_comm-master/share/robonomics_liability/launch/liability.launch
Please specify full path instead
The traceback for the exception was written to the log file

Currently you can run launch file directly, I'm working on similar issues around ros package finder.

Fixed


Update image docker.io/ollama/ollama to v0.4.2

This PR contains the following updates:

Package
Update
Change

docker.io/ollama/ollama
patch
0.4.1-rocm -> 0.4.2-rocm

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Error loading page

Hello.
I have followed the guide published by you (https://shanlh.github.io/vue-tinder/guide/bing-images.html), but when loading the page, the page appears blank with the following error in the console:
http://prntscr.com/uwbry1
Thanks in advance.

Did you set the key for items same as the prop keyName?


README.md corrections and clarification on develop

Here's the changes on the develop branch as requested.

Hi,
I'm not sure why the last entry is on the commit list.  Feel free to let me know if this is a problem I need to look into.

Closing this request and creating a better one in its stead.


About license

I found this amazing highly optimized filecoin sealing algorithm. I plan to use it in my code, but I didn't see any open-source license on GitHub repository. Could you please let me know what your open-source license is?

We will be adding an apache 2 license soon.

Added!


when disableGlobalTransaction is true, no available service 'null' found

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Ⅰ. Issue Description

Ⅱ. Describe what happened
If there is an exception, please attach the exception trace:
Just paste your stack trace here!

Ⅲ. Describe what you expected to happen
Ⅳ. How to reproduce it (as minimally and precisely as possible)

xxx
xxx
xxx

Ⅴ. Anything else we need to know?
Ⅵ. Environment:

JDK version :
OS :
Others:

https://www.cnblogs.com/javashare/p/12529434.html

detail https://www.cnblogs.com/javashare/p/12529434.html

beacuse vgroup_mapping or vgroupMapping
detail https://www.cnblogs.com/javashare/p/12529434.html


haskell-docs-4.2.5 conflicts with descriptive-0.9.3

I used your manual to customize Emacs. Package descriptive-0.9.3 was installed as dependency to hindent.
http://lpaste.net/127029

This doesn't appear to be a conflict per-se; my guess is that a dependency somewhere wants an older version of aeson/mtl.
Maybe also run the same command with -v3 to get more information as to why older versions of mtl and aeson are chosen?

Is this still a problem?

As it's been almost two years without any responses I'm closing this; please re-open if this is still a problem.


ISPN-1435 - JCache interceptors need to be serializable

https://issues.jboss.org/browse/ISPN-1435

Kevin, I assume this is done so that the AS can passivate the beans and hence we're at the mercy of the serialization/marshalling requirements of AS, right?
Even then, wouldn't it be better to make them extend Externalizable rather than Serializable so that we can control what's exactly marshalled of the interceptors?

Kevin, I assume this is done so that the AS can passivate the beans and hence we're at the mercy of the serialization/marshalling requirements of AS, right?

You're right. A CDI bean with a passivation capable scope should have serializable interceptors (required by the specification).

Even then, wouldn't it be better to make them extend Externalizable rather than Serializable so that we can control what's exactly marshalled of the interceptors?

I see what you mean.
The interceptors dependencies have the application scope. In this case the  CDI implementation will take care of the serialization process. In fact, the proxy will be serialized, which doesn't hold a reference to the bean. For more information, you can see the chapter 6 and section 5.4 of the specification. IMHO, we don't need to control the serialization process for now.

Thanks Kevin :)


[Feature request] ignore list for custom-classes

The custom-classes is a nice way of finding invalid classes, but it would be nice with the option of supporting some custom classes. Right now it's all or nothing.
As an example, I've added a .focus-outline to our CSS that gets flagged as custom:
@layer components {
  .focus-outline {
    @apply focus:outline-none focus-visible:ring-2 focus-visible:ring-aqua;
  }
}

Great idea, this should not be complicated to implement.

There you go @thebuilder, v1.6.0 freshly released on both github and npm with the optional whitelist option available.
Thank you for sharing your feature request 👍

Amazing! Thanks for that, and great plugin so far. Seems to be working great🙌


Dummy ENtry [DO NOT MERGE - DUMMY PR FOR TESTING]

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-libraries-for-java

Automation for azure-sdk-for-go

Test Comment added by code.

Test Comment added by code.

Test Comment added by code.

Test Comment added by code.


Attach current auth token to the client config

@lholmquist  what do you please think about it? I realized I wasn't able to obtain current token value from the config or am I wrong again? :)

was this meant for the 1.x branch?

@lholmquist yes. It's open against 1.x if I am not mistaken.

Just looking at this,  it looks like you will only be able to access the token after you make a request to an API.
i haven't tested it yet

Yes it's likely true. I already switched to v2.1.0 so this is not a blocker for me. Maybe we can close it? On the other hand it might helpful...at least little bit:-)

closing for now,  if needed we can re-open


Docs: Added documentation on how to retrieve keys from AWS KMS.

@innovia Thanks for the PR. Can you please add a release note as described here: https://github.com/kubernetes/community/blob/master/contributors/guide/release-notes.md
/assign @innovia
/ok-to-test

/assign @JoshVanL
/ok-to-test

ping

ping

It's just docs - can we get this done please?

/assign @simonswine
/unassign
/unassign @innovia


Deploy Checklist: New Expensify 2023-09-18

Release Version: 1.3.71-8
Compare Changes: https://github.com/Expensify/App/compare/production...staging
This release contains changes from the following pull requests:

[x] https://github.com/Expensify/App/pull/22986
[x] https://github.com/Expensify/App/pull/25398
[ ] https://github.com/Expensify/App/pull/25512
[x] https://github.com/Expensify/App/pull/25560
[ ] https://github.com/Expensify/App/pull/25564
[x] https://github.com/Expensify/App/pull/25761
[x] https://github.com/Expensify/App/pull/26217
[ ] https://github.com/Expensify/App/pull/26602
[x] https://github.com/Expensify/App/pull/26677
[ ] https://github.com/Expensify/App/pull/26747
[x] https://github.com/Expensify/App/pull/26811
[x] https://github.com/Expensify/App/pull/26816
[x] https://github.com/Expensify/App/pull/26839
[x] https://github.com/Expensify/App/pull/26853
[ ] https://github.com/Expensify/App/pull/26854
[x] https://github.com/Expensify/App/pull/26873
[x] https://github.com/Expensify/App/pull/26884
[x] https://github.com/Expensify/App/pull/26931
[x] https://github.com/Expensify/App/pull/26936
[x] https://github.com/Expensify/App/pull/26947
[ ] https://github.com/Expensify/App/pull/27088
[ ] https://github.com/Expensify/App/pull/27137
[ ] https://github.com/Expensify/App/pull/27139
[x] https://github.com/Expensify/App/pull/27166
[ ] https://github.com/Expensify/App/pull/27180
[ ] https://github.com/Expensify/App/pull/27190
[x] https://github.com/Expensify/App/pull/27261
[x] https://github.com/Expensify/App/pull/27279
[ ] https://github.com/Expensify/App/pull/27287
[x] https://github.com/Expensify/App/pull/27289
[x] https://github.com/Expensify/App/pull/27309
[x] https://github.com/Expensify/App/pull/27317
[x] https://github.com/Expensify/App/pull/27323
[x] https://github.com/Expensify/App/pull/27327
[x] https://github.com/Expensify/App/pull/27335
[x] https://github.com/Expensify/App/pull/27340
[x] https://github.com/Expensify/App/pull/27356
[x] https://github.com/Expensify/App/pull/27364
[x] https://github.com/Expensify/App/pull/27369
[x] https://github.com/Expensify/App/pull/27379
[x] https://github.com/Expensify/App/pull/27388
[x] https://github.com/Expensify/App/pull/27395
[x] https://github.com/Expensify/App/pull/27408
[x] https://github.com/Expensify/App/pull/27411
[x] https://github.com/Expensify/App/pull/27421
[x] https://github.com/Expensify/App/pull/27422
[x] https://github.com/Expensify/App/pull/27423
[x] https://github.com/Expensify/App/pull/27426
[x] https://github.com/Expensify/App/pull/27438
[x] https://github.com/Expensify/App/pull/27452
[x] https://github.com/Expensify/App/pull/27460
[x] https://github.com/Expensify/App/pull/27462
[x] https://github.com/Expensify/App/pull/27465
[x] https://github.com/Expensify/App/pull/27478
[x] https://github.com/Expensify/App/pull/27479
[x] https://github.com/Expensify/App/pull/27483
[x] https://github.com/Expensify/App/pull/27484
[x] https://github.com/Expensify/App/pull/27491
[x] https://github.com/Expensify/App/pull/27496
[x] https://github.com/Expensify/App/pull/27498
[x] https://github.com/Expensify/App/pull/27576
[x] https://github.com/Expensify/App/pull/27623
[x] https://github.com/Expensify/App/pull/27637
[x] https://github.com/Expensify/App/pull/27741
[x] https://github.com/Expensify/App/pull/27760
[x] https://github.com/Expensify/App/pull/27793

Deploy Blockers:

[x] https://github.com/Expensify/App/issues/27560
[x] https://github.com/Expensify/App/issues/27669
[ ] https://github.com/Expensify/App/issues/27673
[ ] https://github.com/Expensify/App/issues/27677
[x] https://github.com/Expensify/App/issues/27678
[ ] https://github.com/Expensify/App/issues/27684
[ ] https://github.com/Expensify/App/issues/27710
[x] https://github.com/Expensify/App/issues/27712
[ ] https://github.com/Expensify/App/issues/27713
[ ] https://github.com/Expensify/App/issues/27722
[ ] https://github.com/Expensify/App/issues/27724
[ ] https://github.com/Expensify/App/issues/27754
[ ] https://github.com/Expensify/App/issues/27774
[ ] https://github.com/Expensify/App/issues/27777
[x] https://github.com/Expensify/App/issues/27789
[ ] https://github.com/Expensify/App/issues/27791
[ ] https://github.com/Expensify/App/issues/27802

Deployer verifications:

[ ] I checked the App Timing Dashboard and verified this release does not cause a noticeable performance regression.
[ ] I checked Firebase Crashlytics and verified that this release does not introduce any new crashes. More detailed instructions on this verification can be found here.
[ ] I checked GitHub Status and verified there is no reported incident with Actions.

cc @Expensify/applauseleads

:rocket: All staging deploys are complete, @Expensify/applauseleads please begin QA on version https://github.com/Expensify/App/releases/tag/1.3.71-5 :rocket:

@thienlnam we still don't have Android build. Can you pls double check

Ah crap, looking now - going to try to rebuild Android

We're running into an issue right now, @AndrewGable is looking into it!

Android should be fixed starting version 1.3.71-6

@thienlnam Regression is just 2% left
https://github.com/Expensify/App/pull/26602 is failing with https://github.com/Expensify/App/issues/27712
https://github.com/Expensify/App/pull/26854,https://github.com/Expensify/App/pull/27139, https://github.com/Expensify/App/pull/25512, https://github.com/Expensify/App/pull/25564 #27088 are failing with https://github.com/Expensify/App/issues/27713
https://github.com/Expensify/App/pull/27137 is failing with https://github.com/Expensify/App/issues/26308 - not fixed
https://github.com/Expensify/App/pull/27287 is failing with https://github.com/Expensify/App/issues/27677
https://github.com/Expensify/App/pull/27340 is failing with https://github.com/Expensify/App/issues/27715 - repro in production, checking off
https://github.com/Expensify/App/pull/27421 is failing with https://github.com/Expensify/App/issues/27723 - repro in production, checking off
https://github.com/Expensify/App/pull/27479 is failing with https://github.com/Expensify/App/issues/27707- repro in production, checking off
https://github.com/Expensify/App/pull/26217 is failing with https://github.com/Expensify/App/issues/27775 - repro in production, checking off
#27491 is failing with #27800 - repro in production, checking off
#26747 - for android slot, our team does not have a small height device with in scope OS. Could you check internally?

@thienlnam Regression is completed.
#27180 is failing with #27815 - Repro in production. Okay to check it off?
There are open blockers

https://github.com/Expensify/App/issues/27774 has a fix that was merged.


Providing package argument doesn't change generated documentation links

When providing a package argument to change the package of the generated files, the actual class package is changed correctly, but the documentation is linked. For example:
package com.foo.client;

/**
* Generated REST service builder for {@link com.foo.server.Api}.
*/

I don't understand the issue. Why should the link change ? It is the base class used to generate the builder.

Sorry, you are correct. The link showed up as an error in my IDE and I posted this issue without thinking things through. The real issue is that I am using a multi-project Gradle setup and the client code does not depend on the server code, and so cannot find the source for the documentation link.


[Enhancement]: BigQuery should play nice with spring-cloud

Module
GCloud
Proposal
Problem:
Spring cloud uses the streaming API. The mock of the docker image supports this, but it needs to know, on which port it runs.
Idea:
The container should start on a port, that is free on the host, so that the Streaming API works.
Details in:
https://joerg-pfruender.github.io/software/testing/2023/11/01/spring-bigquery-localstack.html

There is no need to raise a PR and issue at the same time.

See https://github.com/goccy/bigquery-emulator/issues/160#issuecomment-1808939362


Update process-software-bitmaps-with-opencv.md

The following using statements were required in the app:
using OpenCVBridge; //Wasn't required
using Windows.Storage;
using Windows.Storage.Streams;
using Windows.Graphics.Imaging;
using Windows.UI.Xaml.Media.Imaging;

Added this to help users. This was all I required to get the sample app going.

@drewbatgit  : The PR passed checklist review. Please evaluate further and approve/complete the PR if appropriate. Thanks!

Hello, thanks for providing feedback on the docs. The code snippet you highlighted wasn't intended to be a full class or even function that can be compiled as-is. It's just standalone code that you could paste into an app. That said, I think your idea to call out the namespaces used is still a good one. This particular article uses an automated code snippet feature where the code itself lives in an actual project in the repo. So, just to keep things consistent, I went ahead and added the namespaces as this type of snippet. So, you will see that the namespaces now appear in the live page. But, that means that I'm going to close your pull request instead of accepting it. Thanks again!


Rearrange deployment artifacts and examples.

Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

@wongma7 The tests are failing on gometalinter. How to overcome this failure?


Fixes #2689: Replaced all anonymous classes with lambdas

Description
Replaced all anonymous classes with lambdas in the whole project
Related issues and discussion
Fixes #2689

@deniger Please review this PR.

@RajaVamsi11 please, could fix spme conflicts ? I didt the same refactoring in another PR ( merged a bit too quickly...).
Thanks

@deniger Thanks you for the commits and merging the PR.


Switch protractor to minijasminenode2

I signed it!

Why do you need this? We're using "jasmine": "2.3.1" to support jasmine2, and 'minijasminenode2' is essentially obsolete. See http://www.protractortest.org/#/jasmine-upgrade.
Closing for now. Please open an issue if there is any lingering problems.


Conversion of HTML tags to newlines

Some HTML tags should be converted to newlines, others not. This is important to have ideal input for underlying tokenizers / line splitters / sentence detectors. Some examples I created using the NIF Converter:
hello
world gets converted to helloworld, should get converted to hello\nworld. So in this example the
 tag should get converted into a newline.
Highlight in text. gets converted to Highlight in text without newlines which is totally ok.
Question: How can we create a list of tags that should produce newlines? I think a newline should be generated by each HTML tag that produces a linebreak when being rendered to HTML, like p, div, br.  But there might by others. This can also be influenced by CSS.

I could not find a list of all HTML tags that produce newlines. Considering the HTML5 specification:

All elements from "<IP_ADDRESS>.3 Sectioning content" and "<IP_ADDRESS>.4 Heading content" should produce newlines.

These tags should produce newlines also:

article
aside
br
canvas
code
fieldset
hr
iframe
img
ol
p
pre
table
ul
li
tr
td

I do not know where to find an authorative list of HTML tags that should produce newlines. Apache Tika does this well, maybe we can find inspiration there.

Via email we agreed with @katia-vistatec that she will consider either converting all these tags to newlines or only the most important if all the tags is too much work.

Hi,
and what about the tag title?
Should it produce new lines also?

and what about the tag title? Should it produce new lines also?

Yes, good idea!

This is the list of hml tags I am considering for producing new lines:
h1, h2, h3, h4, h5, h6, p, div, pre, title, article, section, address, footer, table, tr, td, thead, tbody, th, caption, ul, li, ol, dl, dt, dd, br, hr, img, iframe, canvas.

Pushed change to repository

I submit a small input file:
curl -X POST --header "Content-Type: text/html" --data "
hello world

abc

xyz
" "http://api-dev.freme-project.eu/current/toolbox/nif-converter?outformat=turtle"

It produced
@prefix xsd:   <http://www.w3.org/2001/XMLSchema#> .
@prefix itsrdf: <http://www.w3.org/2005/11/its/rdf#> .
@prefix nif:   <http://persistence.uni-leipzig.org/nlp2rdf/ontologies/nif-core#> .
@prefix dc:    <http://purl.org/dc/elements/1.1/> .

<http://freme-project.eu/#char=14,19>
        a                     nif:String , nif:RFC5147String , nif:Phrase ;
        nif:anchorOf          "\nabc\n"@en ;
        nif:beginIndex        "14"^^xsd:nonNegativeInteger ;
        nif:endIndex          "19"^^xsd:nonNegativeInteger ;
        nif:referenceContext  <http://freme-project.eu/#char=0,25> ;
        dc:identifier         "2" .

<http://freme-project.eu/#char=0,13>
        a                     nif:String , nif:RFC5147String , nif:Phrase ;
        nif:anchorOf          "\nhello world\n"@en ;
        nif:beginIndex        "0"^^xsd:nonNegativeInteger ;
        nif:endIndex          "13"^^xsd:nonNegativeInteger ;
        nif:referenceContext  <http://freme-project.eu/#char=0,25> ;
        dc:identifier         "1" .

<http://freme-project.eu/#char=20,25>
        a                     nif:String , nif:RFC5147String , nif:Phrase ;
        nif:anchorOf          "\nxyz\n"@en ;
        nif:beginIndex        "20"^^xsd:nonNegativeInteger ;
        nif:endIndex          "25"^^xsd:nonNegativeInteger ;
        nif:referenceContext  <http://freme-project.eu/#char=0,25> ;
        dc:identifier         "3" .

<http://freme-project.eu/#char=0,25>
        a               nif:String , nif:Context , nif:RFC5147String ;
        nif:beginIndex  "0"^^xsd:nonNegativeInteger ;
        nif:endIndex    "25"^^xsd:nonNegativeInteger ;
        nif:isString    "\nhello world\n \nabc\n \nxyz\n"@en .

I wonder why it produces three annotations. Is this correct?
When I try to submit a longer request with SB50.txt then the request never finishes. This seems to be a new bug:
curl -X POST --header "Content-Type: text/html" --data<EMAIL_ADDRESS>"http://api-dev.freme-project.eu/current/toolbox/nif-converter?outformat=turtle"

Hi,
if SB50.html is the same file used to reproduce issue #35 I think there's some bug in internationalization that prevent processing this file correctly or maybe something not implemented. When I used this file (see issue 35) for testing with e-tilde translation service there was a response error from e-tilde service.

This seems to be related: https://github.com/freme-project/e-services/issues/17#issuecomment-230841515

For the request that does not finish I created this bug report: https://github.com/freme-project/basic-services/issues/59
I think the rest of the issue is solved.


Baklava: Set min width and min height to window

add minWidth and minHeight properties to the createMainWindow() function.

This is what the min-width and height looks like, it is also one pixel above the breakpoint to switch to mobile view.

please use proper commit messages (yarn commit or yarn commit:signed)

there is already a pr that deals with this and more #2518


Sequence dictionaries should include SequenceRecords from second-in-pair reads

I just hit an error:
htsjdk.samtools.SAMFormatException: Error parsing text SAM file. MRNM 'chr10' not found in any SQ record; Line 152

trying to load a SAM file, output by ADAM, that contains an @SQ record for the contig the read in question is on ("chr11"), but not that of its mapped mate ("chr10").

Wow, I think I was just bit by the same underlying issue in a very differently-manifesting way.
tl;dr: why does SequenceRecord.fromADAMRecord require reads to be the first of a pair?
if ((!rec.getReadPaired || rec.getFirstOfPair) && (rec.getContig != null || rec.getMateContig != null)) {
  …
} else
  Set()

I was observing very strange behavior where ShuffleRegionJoin and BroadcastRegionJoin would return different results, even on trivial datasets (e.g. 1 read, 1 interval); the former would return an empty RDD.
At first I was omitting the SequenceDictionary argument to ShuffleRegionJoin (which from what I can tell should not be allowed; I'll file a separate issue for that), resulting in an empty SequenceDictionary and empty results. After catching that, I was constructing a SequenceDictionary from my RDD[AlignmentRecord], but still getting empty results, apparently due to my RDD only having 2nd-of-pair reads in it.
I'm going to update this issue title to describe what I think the problem is.

Fascinating! I would say that's a bug in SequenceDictionary. The no-SequenceDictionary version of the ShuffleRegionJoin should compute the SequenceDictionary using the ADAMSequenceDictionaryRDDAggregator class. If the SequenceDictionary implementation was correct, your results should match.
It looks like the behavior in the SequenceDictionary came in via https://github.com/bigdatagenomics/adam/commit/6622e89168ba962b5b87ab955766a4d0d6e92967. @mlinderm any thoughts about this?

Yea, so I also want to talk about the no-SequenceDictionary version of ShuffleRegionJoin; it seems like it currently always returns no intersections? Unless I'm mistaken, I really shot myself in the foot due to that (and this) for a few hours tonight, I think we should kill that overloaded version and the sd singleton.
It's not obvious to me how we would work a ADAMSequenceDictionaryRDDAggregator in there, either...

Ah, I see what you're saying. You're correct there. I think the correct way to do this might involve rethinking the RegionJoin trait and going from there.

Partially answering my own question, you mean we should use ADAMSpecificRecordSequenceDictionaryRDDAggregator to basically get the SequenceDictionary-relevant fields out of whichever object we are joining, by ~reflection?
That sounds bad. Maybe I'm missing something, but it seems like the magic required to auto-generate the SD inside SRJ would be too magical, and brittle.

Ah, I just got what you meant by rethinking the RegionJoin trait: the no-SD API is inherited.
So, up until today I was looking at / using the 0.16.0 versions of the region-joins, where there is no pretense of a super-trait.
I think we could/should just go back to that. The use case strawman: "I have this RegionJoin that could be a ShuffleRegionJoin or a BroadcastRegionJoin and I don't know or care which one it is" doesn't seem to be that substantiated yet (please correct me if I'm wrong, I basically only paged in all of this stuff Saturday :) ).
The semantics/constraints of Shuffle vs. Broadcast are different enough that you kind of have to deliberately choose which one to use in each situation, so I don't think removing the super-trait is a very high cost, or even a cost at all.

(suggest we port further discussion, if any, to #697)


Add explicit mention of beta server to login screen of beta product flavour

We could make it more explicit in the login screen of the beta product flavour that the user is logging in to the beta servers (and that they will need a beta account).
Rationale: to prevent issues like #1052 (we have had several questions about this), and I sometimes forget myself.

Done in #1102


Update minimum supporter Java version to Java 8

The targeted Java version is currently (according to the readme) Java 1.6.
It has been out of support for a long while now, and so has Java 7.
I propose that we update the minimum required Java version to Java 8, to enable developers to use new functions of the language.
I would be interested in doing a PR for this, as well as run a preliminary code analysis to find outdated patterns we can replace with the version change.

It might even prevent a whole lot of problems arising in the future ...

I agree with setting the min version to 1.8

I am preparing a PR that sets the Maven target to 1.8 as well as fixes most of the errors/warnings/code smells found by Jetbrains code analysis tools.
Further refactoring will likely be needed, but I feel like that's a good start. The PR will obviously be big and thus require a decent review before merging, though.

The update went thru, so I'm going to close this


Main merge and regenerate screenshots

Summary
Merging in main to `split-date-picker-inputs` branch and regenerating some screenshots.
Also fix where terra-form-input was overwriting some css in terra-date-picker. Fixed by increasing specificity on css applied to terra-date-picker inputs.

Closes #
Deployment Link

https://terra-framework-deployed-pr-#.herokuapp.com/
Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra

@BenBoersma Would it be easier to merge main on the split-date-picker-inputs branch, then issue a PR with only the new changes you add on top of it. The way this PR is now is it has all the other changes that are already in main and that makes it harder to identify the actual changes to review.

@BenBoersma Would it be easier to merge main on the split-date-picker-inputs branch, then issue a PR with only the new changes you add on top of it? The way this PR is now is it has all the other changes that are already in main and that makes it harder to identify the actual changes to review.

@benbcai Merged main into split-date-picker-inputs branch and then remerged split-date-picker-inputs into this one. only differences should now be my changes and the regenerating of screenshots.


Docs: replace most internal links with Hugo references

Motivation and context

This helps detect broken links, as Hugo will complain if a reference does not resolve to a page.
In fact, we already had a bunch of broken links, which this patch also fixes.
A few links I had to keep as-is for various reasons:

Links from the docs/_index.md, because they are inside a shortcode, and as far as I can tell, you can't put shortcodes in shortcode parameters.
The refresh URLs in home.html and 404.html, for complicated reasons.
Links to autogenerated API documentation pages due to some known problems with generating those pages for previous releases.

How has this been tested?

hugo --destination ../public/cvat --config config.toml
Checklist

[x] I submit my changes into the develop branch
[ ] I have created a changelog fragment
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Bleh. Doesn't seem like there's a way to fix the Remark problems easily. :/ I'll close this for now.

@mdacoca Could you review and make sure I redirected the broken links to the correct pages? Those fixes are all in the second commit.


Use safe_filename instead of file_system.py

https://github.com/pytube/pytube/blob/a32fff39058a6f7e5e59ecd06a7467b71197ce35/pytube/streams.py#L255C1-L255C45
The original pytube already applied this change
CLI always failed on video with special char in the title
Test results:
https://github.com/JuanBindez/pytubefix/pull/311#issuecomment-2453479178

Alright seems u just can't read python