fix(graphql): resolve core-js version collision

This PR fixes a current issue where it's not possible to start the local dev server due to errors.
The problem is a (partial) dependency tree like this with the latest version of apollo-server-express (2.4.8):
$ yarn list --pattern core-js
├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>
The respective yarn why-output for core-js is:

$ yarn why core-js

=> Found<EMAIL_ADDRESS>info Reasons this module exists
   - "hops#@untool#webpack#@babel#polyfill" depends on it
   - Hoisted from "hops#@untool#webpack#@babel#polyfill#core-js"
info Disk size without dependencies: "7.66MB"
info Disk size with unique dependencies: "7.66MB"
info Disk size with transitive dependencies: "7.66MB"
info Number of shared dependencies: 0
=> Found<EMAIL_ADDRESS>info This module exists because "hops-graphql#apollo-server-express#apollo-server-core#@apollographql#apollo-tools#apollo-env" depends on it.
info Disk size without dependencies: "6.6MB"
info Disk size with unique dependencies: "6.6MB"
info Disk size with transitive dependencies: "6.6MB"
info Number of shared dependencies: 0

With the version range for apollo-server-express introduced by this PR, the (partial) dependency tree looks like this:
$ yarn list --pattern core-js
├─<EMAIL_ADDRESS>│  └─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>
And the respective yarn why-output for core-js is:

$ yarn why core-js

=> Found<EMAIL_ADDRESS>info Reasons this module exists
   - "graphql-extensions#@apollographql#apollo-tools#apollo-env" depends on it
   - Hoisted from "graphql-extensions#@apollographql#apollo-tools#apollo-env#core-js"
info Disk size without dependencies: "6.6MB"
info Disk size with unique dependencies: "6.6MB"
info Disk size with transitive dependencies: "6.6MB"
info Number of shared dependencies: 0
=> Found "<EMAIL_ADDRESS>info This module exists because "hops#@untool#webpack#@babel#polyfill" depends on it.
info Disk size without dependencies: "7.66MB"
info Disk size with unique dependencies: "7.66MB"
info Disk size with transitive dependencies: "7.66MB"
info Number of shared dependencies: 0

The two different versions of core-js basically swap places in the tree, which fixes the issue with the local dev server… 🙃🤪😭

An alternative fix that works for me, is to install jest-preset-hops. 😵

I still can't reproduce the error on my machine. 😕
It would be extremely helpful having the description of the steps which lead to the issue.

So @toomuchdesign and I did some more research on the topic. That's what we found:
This issue is related to the way Yarn hoists one of the two required core-js versions. Without jest & jest-preset-hops being required, there is exactly one package requiring<EMAIL_ADDRESS>— @babel/polyfill — and exactly one package requiring<EMAIL_ADDRESS>— apollo-env.
Yarn then decides to hoist<EMAIL_ADDRESS>to the root of the node_modules and install<EMAIL_ADDRESS>nested inside @babel/polyfill. Then during runtime @babel/polyfill fails to resolve its core-js-dependency correctly and ends up with the wrong version<EMAIL_ADDRESS>which makes it fail epically.
If jest and/or jest-preset-hops are installed, too, there are several packages requiring version ^2 of core-js, leading Yarn to rather hoist that one up to the root and put<EMAIL_ADDRESS>nested inside apollo-env. Then @babel/polyfill ends up with the correct version of core-js and everything is fine.
This problem does not occur when npm is used as a package manager…
Have a nice weekend everybody!

Similar issues

https://github.com/yarnpkg/yarn/issues/5705
https://twitter.com/dan_abramov/status/951931842273398784

My latest explanation consists of @babel/polyfill injecting its polyfills imports statement taking for granted that the globally-hoisted version of core-js is the one @babel/polyfill itself relies on (v2).
The polyfilled files are then supposed to resolve their core-js imports from their location with the provided module resolution strategy.
But if:

someone introduces an additional breaking version of core-js (in our case apollo-env) int the node modules tree
yarn decides that the new core-js version is the one to be globally hoisted

... the polyfilled files will be served with the wrong core-js version and not the one intended by  @babel/polyfill.
This issue precisely describes our case.

Wow, this is interestingly bad... Does all this not imply we could add corejs@2 as a peerDependency to one of our packages (maybe hops-graphql) to make sure our users install it manually?
If I understand correctly, this would resolve the issue at hand - but it would be a 'breaking fix', would it not?

We had a similar issue happening already with some styled component related package. In this scenario the added corejs polyfills where in the wrong version, which causes some polyfills to not exist.
The fix was to install corejs in the project itself.
I think it's fair to add corejs as peer dependency, but maybe in the main hops package as having the correct corejs version has so many implication.

The most solid solution I can currently think of solution also consists of keeping a core-js@^2 dependency in the project itself (as long as @babel/polyfill relies on that version).
Since @untool/webpack is the library responsible for configuring babel and making use of useBuiltIns option I suppose we might consider declaring a peer dependency there.

Discussion moved to @untool repo. See you there!
https://github.com/untool/untool/issues/297


AttributeError: 'Session' object has no attribute 'list_devices'

I am getting the error:
using the latest version of keras 2.1.1, I am trying to use the multi_gpu_model fonction and got an error:
so I ran:
from keras import backend as K
K.get_session().list_devices()

I get the error
AttributeErrorTraceback (most recent call last)
<ipython-input-6-0fe3a93aa06e> in <module>()
----> 1 K.get_session().list_devices()

AttributeError: 'Session' object has no attribute 'list_devices'

Also tried following https://github.com/fchollet/keras/pull/8377/files,
from tensorflow.python.client import device_lib
[d.name for d in device_lib.list_local_devices()]

it works fine
Any clues how to solve the problem ?

I also met this problem. What's wrong?

What's your version of TF? I assume you have an outdated version. I would
simply recommend upgrading to 1.4.
On 17 November 2017 at 19:13, TengfeiZhang<EMAIL_ADDRESS>wrote:

I also met this problem. What's wrong?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/8525#issuecomment-345414334,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArWb1OiVUIbN7VQ67tdSBz6Xzc2gOrHks5s3ktugaJpZM4QiqmV
.

How should I upgrade? My current version of 'tensorflow-gpu' is 1.2.0

Francois: Thank you !
adideshpande: Yes you need to upgrade to 1.4. Look at the tensor flow page depending on your OS. for tf 1.4, it will ask you to upgrade cudnn to 6.0...

Hi I am having the same issue:
AttributeError: 'Session' object has no attribute 'list_devices'
Making_A_Prediction_From_Sound.py:155: UserWarning: Update your `Conv2D` call to the Keras 2 API: `Conv2D(64, (3, 3), activation="relu", data_format="channels_first")`
  model.add(Conv2D(64, (3, 3), activation='relu', dim_ordering="th"))
Traceback (most recent call last):
  File "Making_A_Prediction_From_Sound.py", line 173, in <module>
    model = create_model('/home/pi/Desktop/heart-files/weights-best-031-0.88735.hdf5')
  File "Making_A_Prediction_From_Sound.py", line 155, in create_model
    model.add(Conv2D(64, (3, 3), activation='relu', dim_ordering="th"))
  File "/usr/local/lib/python3.5/dist-packages/keras/models.py", line 489, in add
    output_tensor = layer(self.outputs[0])
  File "/usr/local/lib/python3.5/dist-packages/keras/engine/topology.py", line 603, in __call__
    output = self.call(inputs, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/keras/layers/convolutional.py", line 164, in call
    dilation_rate=self.dilation_rate)
  File "/usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py", line 3180, in conv2d
    x, tf_data_format = _preprocess_conv2d_input(x, data_format)
  File "/usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py", line 3062, in _preprocess_conv2d_input
    if not _has_nchw_support():
  File "/usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py", line 270, in _has_nchw_support
    gpus_available = len(_get_available_gpus()) > 0
  File "/usr/local/lib/python3.5/dist-packages/keras/backend/tensorflow_backend.py", line 256, in _get_available_gpus
    _LOCAL_DEVICES = get_session().list_devices()
AttributeError: 'Session' object has no attribute 'list_devices'

I am following this tutorial:
https://github.com/samjabrahams/tensorflow-on-raspberry-pi/blob/master/GUIDE.md
Which version of tensorflow should I upgrade to and which version of bazel should I use.

You should upgrade to TF1.4.
Alternatively, we've added back legacy support for TF1.3 device listing in Keras 2.1.2. So you could also switch to using Keras 2.1.2 if you cannot update your version of TF.


Turtlebot4 lite Obstacle avoidance and performance issues

I am currently using RPI4 with 4GB for turtlebot4 lite.
I could use navigation to move the robot from Point A to Point B, but now i would like to add obstacle avoidance while moving from A to B. How to do that ? is it already implemented and or need to be enabled ?
I understand that, there is way to view stereo depth camera from robot_view by enabling in the rviz tool, but it hanging entire process if we enable the stereo depth camera ? Is 4GB ram is not sufficient to monitor the camera ?
While running navigation using
"ros2 launch turtlebot4_navigation localization.launch.py map:=office.yaml"
"ros2 launch turtlebot4_navigation nav2.launch.py"
it is always timed out waiting for odom server and unable fetch the odom topic.
How to resolve this issue as well ?

But in my turtlebot4 lite setup, obstacle avoidance is not working ? I have followed the same documentation as [https://turtlebot.github.io/turtlebot4-user-manual/tutorials/navigation.html]. Kindly advice
The access point used here is mobile hotspot and operating only in 2.4GHz , is it having any performance problem ?

Have you generated your own map? Which launch files are you using for navigating?
I would highly recommend using a 5GHz access point for the RPi4 especially if you are trying to stream video data. Even at the low resolution that the TurtleBot 4 uses, it can be quite taxing on network usage.

@roni-kreinin yes, I am using my own map generated from SLAM.
1.
$ ros2 launch turtlebot4_navigation localization.launch.py map:=my_room.yaml
$ ros2 launch turtlebot4_navigation nav2.launch.py
$ ros2 launch turtlebot4_viz view_robot.launch.py
using above three command for starting the navigation.
2. Another problem is even without camera, it is taking longer time activate navigation. after 5-10 minutes only it starting the navigation ?

What are the issues with obstacle avoidance? Is the robot driving into obstacles that you can see on the map? As for the activation time, are there any errors being reported by either localization or nav2 in the terminal?
Also, are you running Galactic or Humble?

Yes, it not detecting obstacles and trying drive into obstacles and failing to doso, if we manually clear the obstacle it going ahead.
We are using humble. I don't see any errors, but I will try to collect logs and share to you.
Are the launch files are correct?

The launch files are correct. Can you record your RViz while running navigation and share that here?

@roni-kreinin please find the attached logs of the three terminal after running the navigation. I could not collect the lengthy log but it should give you insight of the problem.
localization_launch.txt
nav2_launch.txt
RVIZ.txt

From the RViz errors it looks like there is an issue with the time sync between the RPi and the Create 3. Try calling turtlebot4-ntpd-sync from the RPi4 or accessing the Create 3 webserver and restarting ntpd from the Beta Features tab.

@roni-kreinin Thanks for pointing me in right direction, after turtlebot4-ntpd-sync, the obstacle avoidance is active in navigation and relatively the time for starting the navigation is less (still about 3-4 minutes of pose estimation).
But one problem is, the moment obstacle is detected and not cleared for 3-5 sec, then the navigation in the RVIZ getting feed back as aborted. And hence it is not moving to new target.
Also the initial pose delay is due to wi-fi speed ?

@roni-kreinin please find the below link for screen recorded video,
https://we.tl/t-sdS0ZHoqSt
I have uploaded in wetranfer due to size limitation. You can see the lag between pose and navigation and also obstacle abort problem at (9:22 minutes in video). Infact this time the response was much better :).
Please check and advise !

@roni-kreinin waiting for your feedback, after reviewing above video.

@roni-kreinin

No, I am waiting until ros2 topic list shows all the topics published by create3 and then only running the navigation.
But still it is taking 3-5 minutes to start the navigation.
During the course of booting the Create3 platform is showing RED in light ring for few seconds then giving chime sound. Is it normal ? because some times it is un responsive and not able to switch off on/off also ?
Yes, as i also mentioned in the video the obstacle behavior looks normal, but when there is obstacle present for little longer period, the navigation getting aborted. how to resolve this ?
Setup : ROS2 Humble 1.1 with Simple Discovery.
Also the turtlebot4-ntpd-sync need to be run every time we startup the ROBOT or only once enough ?
Still Am unable to use the Stereo camera at all, the moment it is enabled in RVIZ the entire process getting hanged.
Should i check Discovery server for better performance or simple discovery is good ?

This issue is being closed due to inactivity. If you are still experiencing the issue, feel free to reopen this ticket when you are ready to continue the troubleshooting process.

@roni-kreinin  I just want to know, by how the obstacle avoidance works in the turtlebot4, when i run these commands...
ros2 launch turtlebot4_navigation nav2.launch.py
ros2 launch turtlebot4_navigation localisation.launch.py map:=my_map.yaml
ros2 launch turtlebot4_viz view_robot.launch.py
with the help of RPI-Lidar or OAK-D camera or both?

@Vasanth28897 The short answer is just the lidar. The long answer is that those commands that you are referring to are launching Nav2. There are a lot of things that you can do with Nav2. All of the default Nav2 settings are contained in the config yaml files. You can modify this behaviour by making your own yaml files and passing them in when you launch nav2 and/or localization.

thank you @hilary-luo.  I am facing the same obstacle avoidance problem in the turtlebot4 i have when i run the nav2.launch.py like @stalinADAQ and @schenkch faced. I tried the time-sync command...it didn't work.

@Vasanth28897 I am going to need you to expand on what you mean when you are referring to the "same obstacle avoidance problem". Preferably can you do this in a fresh ticket? Time has changed since then, the software has been updated in various ways so that symptom may have a different reason now.

Alright, as @hilary-luo mentioned in  issue. The lidar cannot hits the costmap around the mapped dynamic obstacle(not in the map during generation)as you mentioned in this  issue. (Note: sometimes it hit the static obstacle also).

@Vasanth28897 If I am understanding correctly, you are seeing that the robot is not avoiding dynamic obstacles. This issue is fixed in 1.0.4. You can use it now if you build from source, or it will be released in the next update. The issue and the temporary fix were tracked here: https://github.com/turtlebot/turtlebot4/issues/247
If it is hitting static obstacles then either a) it had issues seeing them (not at lidar height or are too skinny, too reflective or see through etc.) and you may need to block off those areas or if it is not the case, b) you can tune your nav2.yaml parameters to better work for your environment. Different environments and different desired operations will require different parameters.

alright, thank you @hilary-luo for the clarification.


[java] Enable trust boundary violation tests for java

Motivation
Trust boundary violation tests were passing for Java
Changes

Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[ ] Relevant label (run-parametric-scenario, run-profiling-scenario...) are presents
[ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team

Hi @manuel-alvarez-alvarez ,
Wen your PR impacts only java (with a 100% certitude), prefixing the Pr title with [java] will run the CI only on java, saving some CI and cognitive time. See https://dd.slack.com/archives/C025TJ4RZ8X/p1713449788481689

Just saw the message in Slack, many thanks for the info as it's a really great feature :wink:


Drop AIX support

The gnupg module doesn't support AIX and we also cannot test on AIX.

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

this is cherry-picked from #164 to reduce the diff.


hcm: Add metadata specific the routed cluster in weighted_clusters.

Description:
Right now when a route points to multiple clusters, we don't have a good way of inferring  metadata
information specific to the routed cluster. more specifically when multiple clusters are in the route.
In solo.io we to get more info on the cluster chosen for routing, for example, in our transformation filter. we attach the transformation to perform to the route metadata.
To overcome that, in the route metadata we have a struct with an entry per potential cluster. This is
limited, as we can only have one cluster in the struct (it doesn't cover the case of traffic splitting to test a new transformation)
Desired Behavior:
Ideally one would to be able to have the transformation attached on the routed
cluster and be able to place the same cluster more than once in the list of weighted clusters (with different
metadata).
The most straight forward idea that comes to mind is to add a metadata field to
route.WeightedCluster.ClusterWeight, and expose it in the RouteEntry that the filter accesses.
We (the team at solo.io) can help with the implementation and submit a PR once we discuss and finalize the details.
Relevant Links:
https://github.com/solo-io/envoy-transformation

Sounds reasonable to me. @zuercher WDYT?

That seems reasonable to me as well.

great! i'm on it then.


make basic register_group instances t2.micro

These don't need to be t2.medium.

This has already been applied to test. #sns

Definitely.


[FEATURE] Option to go back to the previous item when showBottomSheetAt clicked while SSBottomSheet showing

Is your feature request related to a problem? Please describe.
When showBottomSheetAt has no proper widget to show (because SSBottomSheet will be showing) gets clicked, SSBottomSheet dismisses.
Describe the solution you'd like
I'd like to start animation to previously clicked SSBottomNavItem's index.
Describe alternatives you've considered

create a field named bottomSheetHistory
create a variable called _tempIndex to hold previously clicked SSBottomNavItem
in onTab

if index equals to showBottomSheetAt, save _tempIndex
else SSBottomSheet shows

in _onPressed(Offset offset)

if index equals to showBottomSheetAt and bottomSheetHistory is true,

_updateIndex(_tempIndex)
set _service.clickedIndex to _tempIndex for animation

else _updateIndex(index)

Sliding down the SSBottomSheet  doesn't animate the previously SSBottomNavItem's index.


Syntax Error?

/volume1/homes/name/syno_hdd_db.sh -showedits
/volume1/homes/name/syno_hdd_db.sh: line 8: syntax error near unexpected token `newline'
/volume1/homes/name/syno_hdd_db.sh: line 8: !DOCTYPE html
Am I doing something wrong? I'm trying to run this on an RS3618xs.
Cheers!

Did you edit the syno_hdd_db.sh file in notepad?
Or did you download the github webpage?
You should download the script via one of the 3 links shown in this picture:

Yep I'm an idiot... not sure why I didn't do that (i normally download from releases). Sorry!


As a user I want a clear visualization of how to control the car.

Create basic platform for app
Create outline for functionalities.
Create buttons
Give color and icons to buttons.
Create reactive buttons on input.


XML support User experience

This issue was created automatically, but XML support in serialization/deserilization has no user experience work (it's supposed to be transparent for users)

reopen for python feature to support xml in typespec.


Cannot pipe STDIN to REPL as expected

I would expect that redirecting STDIN to the REPL would execute the code given. For example;
With Erlang:
$ echo "1 + 1." | erl
Eshell V11.1.6  (abort with ^G)
1> 2
2> *** Terminating erlang (nonode@nohost)

With Elixir:
$ echo "1 + 1" | iex
Interactive Elixir (1.11.3) - press Ctrl+C to exit (type h() ENTER for help)
iex(1)> 2
iex(2)>

With LFE:
echo "(+ 1 1)" | rebar3 lfe repl
===> Verifying dependencies...
Eshell V11.1.6  (abort with ^G)
1> *** Terminating erlang (nonode@nohost)

Note that this may be an upstream LFE issue, but I thought to open the issue here first since the behaviour is different with the rebar3 lfe repl command and with the lfe command. For example, I get
$ echo "(+ 1 1)" | lfe
Eshell V11.1.6  (abort with ^G)
1> * 1: syntax error before: 1
1> *** Terminating erlang (nonode@nohost)

Which looks suspiciously like a regular Erlang shell to me. Sure enough,
$ echo "1 + 1." | lfe
Eshell V11.1.6  (abort with ^G)
1> 2
2> *** Terminating erlang (nonode@nohost)

But I do not get the same behaviour with the rebar3 lfe repl command:
$ echo "1 + 1." | rebar3 lfe repl
===> Verifying dependencies...
Eshell V11.1.6  (abort with ^G)
1> *** Terminating erlang (nonode@nohost)

I would be happy to try and help solve this issue, if someone could assist me in figuring out where to start.

I think the fact that the lfe command evaluates Erlang and not LFE is due to https://github.com/rvirding/lfe/issues/156. So then the issue with rebar3 lfe is that it does not run either Erlang or LFE code when reading from STDIN.

I think the fact that the lfe command evaluates Erlang and not LFE is due to https://github.com/rvirding/lfe/issues/156. So then the issue with rebar3 lfe is that it does not run either Erlang or LFE code when reading from STDIN.

@John-Goff sorry for getting back so late -- yeah, this won't work with the rebar3 LFE REPL. It's not actually due to the issue you linked to, though: the reason being that shells that run in rebar3 aren't OS processes; they are Erlang message loops with the "terminal display" being data that gets written to the user PID.
I'll close this as ... "can't fix" ;-)
You know, I've had so many issues with with the rebar3 LFE REPL (just ran into a new one this week, due to Erlang 26.0) that I'm sorely tempted to add a Makefile to the rebar3_lfe project templates and support make repl that starts up an OS process of the LFE REPL.
At which point the bug you linked would come into play (and might, in fact, be the culprit). Except in Erlang 26, where the shell has been refactored and now works for LFE:
echo '(+ 1 1)' | ./bin/lfe -nobanner

gives:
lfe> 2
lfe> *** ERROR: Shell process terminated! (^G to start new job) ***

All of this being said, you know about the eval capability of lfe/erl, right?
./bin/lfe -eval '(io:format "~p~n" (list (+ 1 1)))'
2

Was following up on an old email and just saw your reply to this! Yeah, it actually wasn't about eval-ing simple forms. It was instead about sending arbitrary forms to a long living repl process for creating a development environment in, vim at the time, but in the years since I opened this ticket I have switched to Emacs 😄 Oh how the turntables... Something or other.


complete CellIsOperators

Summary
TypeScript definition of CellIs operators was incomplete, it missed notEqual and some others.

I would ask maintainers to review and merge this PR or to suggest what should be fixed (I also need this change in my company project). Thanks a lot! 😊
CC.: @Siemienik, @alubbe, @guyonroche,


Proper transform interfacing.

The transforms shouldn't use a bunch of cryptic menu items but rather should just let you adjust the element in place. So you can drag it to the size and rotation you want. Perhaps with a button for flip horizontal or vertical.

This is added and complete in 0.6.0

This issue was corrected in 0.6.0. Which is now in full release.


returning false from validate does not cancel write

version: 0.14.11
node: v6.10.3
returning false from validate does not seem to cancel the write.  Is that a roadmap feature or maybe a documentation issue?  Could you have maybe meant that the particular value in question won't get set/"written"?  that's the way the code appears to be, but if I get a choice I'd rather it not write the entire record versus fail the "set" of a specific property.

Can you please elaborate and provide some sample source code?
Do you mean something like this:
const schema = lounge.schema({
  number: Number,
  minMax: { type: Number, min: 100, max: 200 }
})

const SModel = lounge.model('SModel', schema)

const obj = new SModel()
obj.minMax = 0
obj.save((err, doc) => {
  // ...
})

obj should not be saved because of the properties was unsuccessfully assigned?

Sure, and thank you for spending time on this.
const schema = lounge.schema({
username: String,
domain: {
type: Array,
arrayType: String
}
});
schema.index(['username', 'domain']);
const User = lounge.model('User', schema);
const user1 = new User({username: 'foo', domain: ['foo.com','bar.com']});
user1.save((err, doc) => {...});
64578f2a-62d8-4518-81d6-6f5f28a6efa2 {"username": "foo", domain: ["foo.com", "bar.com"]}
username_and_domainundefinedfoo_foo.com {"key": "64578f2a-62d8-4518-81d6-6f5f28a6efa2"}
username_and_domainundefinedfoo_bar.com {"key": "64578f2a-62d8-4518-81d6-6f5f28a6efa2"}
// delmiter should probably be defaulted...
const user2 = new User({username: 'foo', domain: ['bar.com']});
user2.save((err, doc) => {...});
b2694b7e-2bcc-44c3-a53a-29ada9177899 {"username": "foo", domain: ["bar.com"]}
Username_and_domainundefinedfoo_bar.com {"key": "b2694b7e-2bcc-44c3-a53a-29ada9177899"}
So even though there was already an index for foo_bar.com, when we stuck the second user the index got overwritten with the key of the new user.  so if I try to login as foo with a domain of bar.com I will fail because another user owns the index and almost certainly has a different password then me...  Now I could set indexType to Array and it would create an index like:
Username_and_domainundefinedfoo_bar.com {"key": [
"64578f2a-62d8-4518-81d6-6f5f28a6efa2",
"b2694b7e-2bcc-44c3-a53a-29ada9177899"
]}
... but that doesn't really accomplish the goal because now I'd have to compare each users password with the submitted password and it basically lets an evil user choose the same username as someone else and the same domain as that person and then begin to attack their account until they find a password that logs them in as someone else.
what I'm hoping for is "there's already a user with both the same username and domain", then fail to stick a second user.  I think what's maybe needed is to call upon the Model's findByUsernameAndDomain to see if there's an existing user and if there is then throw or next(null, false) or whatever; but I'd potentially have to call that method for each domain.  so if there's a user "foo" with 5 domains (unlikely) then I'd have to call findByUsernameAndDomain up to 5 times -- I'd presumably I'd exit early when I found a match.  I think it's a consequence of using upsert for everything.  I tried passing in a cas, but that obviously didn't work because the records are unique it's just the $ref that is not.
Ottoman, for example (and I seriously hate to keep comparing the two) allows you to set the type of index between refdoc (unique), view or n1ql.  So you'd do something like:
index: {
findByUsernameAndDomain: {
by: ['username', 'domain'],
type: 'refdoc'  // guarantees uniqueness even on a combined index
}
...
}

On Jun 27, 2017, at 1:16 PM, Bojan<EMAIL_ADDRESS>wrote:
Can you please elaborate and provide some sample source code?
Do you mean something like this:
const schema = lounge.schema({
number: Number,
minMax: { type: Number, min: 100, max: 200 }
})
const SModel = lounge.model('SModel', schema)
const obj = new SModel()
obj.minMax = 0
obj.save((err, doc) => {
// ...
})
obj should not be saved because of the properties was unsuccessfully assigned?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/bojand/lounge/issues/73#issuecomment-311472484, or mute the thread https://github.com/notifications/unsubscribe-auth/AMD_ntMZ-LCuyL2eT8SOgoAwpAoH4WGKks5sIWMjgaJpZM4OGdhv.

Hello, sorry it took so long to look at this. But I think you can accomplish what you wish to already with the existing API. You just need to query for the document first. A simple quick example
const schema = lounge.schema({
  username: String,
  domain: {
    type: Array,
    arrayType: String
  }
})

schema.index(['username', 'domain'])

const User = lounge.model('User', schema)

function createUser(username, domain, fn) {
  async.detect(domain, (d, asCb) => {
    User.findByUsernameAndDomain(username, d, asCb)
  }, (err, r) => {
    if (err) {
      return fn(err)
    }

    if (r) {
      return fn(new Error(`User with username ${username} and domain ${domain} already exists`))
    }
    const user = new User({
      username,
      domain
    })

    user.save(fn)
  })
}

createUser('Bob', ['foo.com', 'bar.com'], (err, saved) => {
  console.log(saved)
  createUser('Bob', ['foo.com'], (err, doc1) => {
    console.log(err.message)
    createUser('Bob', ['bar.com'], (err, doc2) => {
      console.log(err.message)
      process.exit(0)
    })
  })
})

Results in
{ username: 'Bob',
  domain: [ 'foo.com', 'bar.com' ],
  id: 'e1a65a64-5c43-4f62-9929-946757d1977c' }
User with username Bob and domain foo.com already exists
User with username Bob and domain bar.com already exists

Just to expand on this more, you can encapsulate this kind of logic in the pre save hooks and basically get exactly the type of behaviour you desire.


Add UUID_FORMAT config

PR adds the option to formating the current generated GUID in the available formats.

Closes #80

Seems like pre-commit needs an update. Please run pre-commit install, pre-commit autoupdate and pre-commit run —-all-files and commit changes. There’s is some MyPy errors that needs to be fixed too.
Ignore any integration flake8 errors.

I will have a look tomorrow when I’m back home 😊

I'd love a test that use it in a log message, but not critical if you don't have time to add it. I can do that later.
Looks good to me 😊 thank you so much!
(I'll merge when I'm on a computer, if you don't want to add the test)

@JonasKs I added some more tests. Please check if it is what you had in mind.

Perfect! 😊 Thank you so much!
I’ll merge if Sondre don’t beat me to it tomorrow!

Thank you so much! 😊 Should be released any minute on PyPI now.
https://github.com/snok/django-guid/releases/tag/3.3.0


when i ionic start new --type=ionic-angular,v4.5.3 node-sass is 404

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
What behavior are you expecting?
Steps to reproduce:

ionic start new --type=ionic-angular
v4.5.3 node-sass is 404

https://github.com/sass/node-sass/releases/download/v4.5.3/linux-x64-59_binding.node

http error 404 not found

Which @ionic/app-scripts version are you using?
cli packages：
@ionic/cli-utils : 1.16.0
ionic( ionic CLI): 3.16.0
System:
node: v9.0.0
npm: 5.5.1
os: linux 4.13(ubuntu 17.10)
misc:
backend: pro
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)

The required node-sass version is not ready for Nodejs 9. Please use the latest LTS release 8.9

thank you for your reply.


jrc_hela-22

used s3 (64nm seemed too low for this dataset)

proofread tif found here: /avetissiane/tiff_slices/jrc_hela-22/s3_amira_proofread.tif

/avetissiane/tiff_slices/jrc_hela-22/s3_amira_proofread.zarr

Thumbnail
/cellmap/nrs/data/jrc_hela-22/thumbnail_nuc.jpg
Ready for meshes

DOI created
https://doi.org/10.25378/janelia.26513071 https://figshare.com/s/1c4d0616e343a516c1c4


During emit, if shape signature for the file is same as version, then update it with emitted d.ts file

This change updates the signature if while computing affected file set we set it as version of the file (Whole text of the file instead of d.ts emit for perf imprrovement) using d.ts emit if it was done without using custom transformers. This will help with later updates since the shapes are initialized and smaller set of files will be updated..

In order to avoid performing .d.ts emit just for signature computation, we started using a signature based on the input text when no .d.ts file was available

Thats not what happens.. we have few optimizations when we would be computing huge set of d.ts files (like initial build) or when we are doing d.ts computation as the signature might change because it imports something with change in its d.ts indirectly. These were perf optimizations to speed up the process... Result of this is that when the file changes again, that is when we compute the d.ts for that file as well as any file that imports it to see if things have changed. Answer to that will be always yes since in first round we had sourceText and now the d.ts.. With this change, when we are emitting files and we detect that it is emitting declaration file and we are using sourceText as signature we update the signature with d.ts emit so that in next round we can correctly do the computation. This helps with not emitting some files unnecessarily as we are now have d.ts as signature and the d.ts computation from the change will match..

no custom transformers (why does this matter?)

Because custom transformers can change the generated output and we don't want and use that when we generate d.ts during signature computation

I don’t quite understand the expected behavior here or what’s being shown. nominal.ts exports a type that uses a global type declared in types.d.ts. A change to types.d.ts could cause a change to any file importing nominal.t

This was incorrect example and i am confused as well and will need to investigate. But the idea is that if .d.ts is not used as signature, sourceFile text and all the imports are used as if they are needed for determining d.ts impact.
To make it simple here:
/// moduleA.ts
import {} from "shared";
/// app.ts
import {} from "moduleA"

The theory is that if import of moduleA is needed when you emit app.d.ts for app.ts then change to share.ts such that share.d.ts is changed but doesnt change moduleA.d.ts can impact types in app as well app.d.ts
With #43314  we started using file text as signature in some cases for performance. And in that case we assume that all imports in the file are needed in .d.ts emit. So in that case any change to share.ts may unnecessarily update app.ts if it doesnt need moduleA import in its d.ts.
This change kind of avoids those unnecessary updates by updating the signature to d.ts emit during emit if we detect so.
https://github.com/microsoft/TypeScript/pull/48616/files#diff-65a637de3855967769ea5a0ed013a8b4cad6afccf07d094864172e80626b73d2L192 is better example showcasing this.

Ping @andrewbranch @amcasey  can you please review this again so i can get this in.
Since https://github.com/microsoft/TypeScript/pull/48616#discussion_r850866269 was old behaviour and will pop up on next change anyway, i dont think its blocking this PR. Creating a investigate issue for myself to track this.
The Thanks..


trying to open any .gv files crashes the program

Hi.   I'm trying to open .gv files created by wire visualization software.   When I open theses files,  the software inmediately crashes.  No error is given.  This is being tested in windows10.
The files I'm testing are here:  https://github.com/formatc1702/WireViz/tree/master/examples

Hi!
Thank you for the report, I will check it...
чт, 17 дек. 2020 г. в 08:44, dproldan<EMAIL_ADDRESS>
Hi. I'm trying to open .gv files created by wire visualization software.
When I open theses files, the software inmediately crashes. No error is
given. This is being tested in windows10.
The files I'm testing are here:
https://github.com/formatc1702/WireViz/tree/master/examples
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ArsMasiuk/qvge/issues/131, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEWY5KAJNACYJRIA2XBHECTSVGZF3ANCNFSM4U7F6CXQ
.

Well, the problem is QVGE does not support all of DOT features yet.
Your exapmles are using HTML labels which are one of not-yet-supported features.
So I will put this into TO-DO list...

crash has been fixed for 0.7, but HTML labels have to be completely supported in the future


unable to build lsquic

So, these are my steps to build lsquic
BORINGSSL_VERSION=b117a3a0b7bd11fe6ebd503ec6b45d6b910b41a1
mkdir -p $PLATFORM/boringssl
cd $PLATFORM/boringssl
ls .git/ || git init
git remote add origin https://boringssl.googlesource.com/boringssl
git fetch --depth 1 origin b117a3a0b7bd11fe6ebd503ec6b45d6b910b41a1
git checkout FETCH_HEAD
git remote remove origin
cmake -DBUILD_SHARED_LIBS=1 -DCMAKE_BUILD_TYPE=Release . && make
BORINGSSL=$PWD

cd ../..

LSQUIC_VERSION=2.27.4
rm -rf $PLATFORM/lsquic
git clone --depth 1 --branch v$LSQUIC_VERSION https://github.com/litespeedtech/lsquic.git $PLATFORM/lsquic
cd $PLATFORM/lsquic
git submodule init
git submodule update
cmake -DBORINGSSL_DIR=$BORINGSSL -DBORINGSSL_LIB_ssl=/usr/local/lib/ -DBORINGSSL_LIB_crypto=/usr/local/bin/ .

Here's what the /usr/local/lib/ looks like.
ls /usr/local/lib/
engines-1.1  libcrypto.a  libcrypto.so  libcrypto.so.1.1  libssl.a  libssl.so  libssl.so.1.1  pkgconfig  python2.7  python3.8

Here's what happens when I run make: https://bin.snopyta.org/?7be491e727f1a9e6#HAnp7UJemEAivPGFG8uYU3LCjsv5vBpjnqUzrzQ2RQVC
Without -DBORINGSSL_LIB_ssl=/usr/local/lib/ -DBORINGSSL_LIB_crypto=/usr/local/bin/, this is my output:
-- BoringSSL include directory /home/codespace/workspace/javacpp-presets/lsquic/cppbuild/linux-x86_64/boringssl/include
-- ssl library not found
-- crypto library not found
-- decrepit library not found
-- Found zlib: /usr/lib/x86_64-linux-gnu/libz.a
-- Found event: /usr/lib/x86_64-linux-gnu/libevent.a
-- sphinx-build not found: docs won't be made

Is there anything missing? I have openssl and libssl-dev installed.

You build BoringSSL but then link with system-installed OpenSSL?  Why not follow build directions in the README or .travis.yml?

You build BoringSSL but then link with system-installed OpenSSL?  Why not follow build directions in the README or .travis.yml?

I see that's a no brainer!
I found out the issue:
for some reason setting BORINGSSL_DIR isn't working for me, I worked around by setting BORINGSSL_INCLUDE instead.
codespace ➜ ~/workspace/lsquic (master ✗) $ cmake -DBORINGSSL_DIR=$BORINGSSL .
-- Build type: Debug
-- AddressSanitizer is OFF
-- Compiler flags:   -DLSQUIC_DEBUG_NEXT_ADV_TICK=1 -DLSQUIC_CONN_STATS=1 -Wall -Wextra -Wno-unused-parameter -fno-omit-frame-pointer -Wno-implicit-fallthrough -Wno-missing-field-initializers -O0 -g3
CMake Error at CMakeLists.txt:157 (MESSAGE):
  BoringSSL headers not found

-- Configuring incomplete, errors occurred!
See also "/home/codespace/workspace/lsquic/CMakeFiles/CMakeOutput.log".

I replaced -DBORINGSSL_DIR=$BORINGSSL with -DBORINGSSL_INCLUDE=$BORINGSSL/include and it worked.
Thanks for the help!

I see that's a no brainer!
I found out the issue:
for some reason setting BORINGSSL_DIR isn't working for me, I worked around by setting BORINGSSL_INCLUDE instead.
codespace ➜ ~/workspace/lsquic (master ✗) $ cmake -DBORINGSSL_DIR=$BORINGSSL .
-- Build type: Debug
-- AddressSanitizer is OFF
-- Compiler flags:   -DLSQUIC_DEBUG_NEXT_ADV_TICK=1 -DLSQUIC_CONN_STATS=1 -Wall -Wextra -Wno-unused-parameter -fno-omit-frame-pointer -Wno-implicit-fallthrough -Wno-missing-field-initializers -O0 -g3
CMake Error at CMakeLists.txt:157 (MESSAGE):
  BoringSSL headers not found

-- Configuring incomplete, errors occurred!
See also "/home/codespace/workspace/lsquic/CMakeFiles/CMakeOutput.log".

I replaced -DBORINGSSL_DIR=$BORINGSSL with -DBORINGSSL_INCLUDE=$BORINGSSL/include and it worked.
Thanks for the help!


Cannot add category to course

Bug Report
Problematic Behavior
When edit a course page and add category to the course.  Use the pop-up and select a category page.
When press save, it show "Select a valid choice. That choice is not one of the available choices." no matter select which category in the list.
Expected behavior/code
Save successfully and category add to course.
Steps to Reproduce

add category in categories page.
add new course in organization page.
edit new course and assign category.
select category and press save.

Environment

Richie version: 2.22.0
Platform: Linux

Additional context/Screenshots

Is this happening on the demo site generated by richie or another environment ?

It happens without demo data.

Did you create the master categorie such as Subjects (use it as page id if you want the search engine to pick it up) before creating the smaller categories such as Business management ? You aren't supposed to be able to add the master categories to course pages.
https://demo.richie.education/en/categories/


Cannot read properties of undefined (reading 'packageJson') when using npx chatgpt

Verify latest release

[X] I verified that the issue exists in the latest chatgpt release

Verify webapp is working

[X] I verify that the ChatGPT webapp is working properly for this account.

Environment details
$ npx --version
8.19.3

$ uname -a
Darwin ezekiels-mbp.lan 22.2.0 Darwin Kernel Version 22.2.0: Fri Nov 11 02:04:44 PST 2022; root:xnu-8792.61.2~4/RELEASE_ARM64_T8103 arm64

Describe the Bug
I can run npx chatgpt and npx chatgpt@latest from some directories, but in others I get an error.
~/git/replicate/scribble-diffusion bypass-vercel-image-optimization
$ npx chatgpt "write a shell command to convert input.mov to output.mp4, and speed it up 2x"

ffmpeg -i input.mov -filter:v "setpts=0.5*PTS" -an output.mp4

~/git/replicate/scribble-diffusion bypass-vercel-image-optimization
$ cd ~

~
$ npx chatgpt "write a shell command to convert input.mov to output.mp4, and speed it up 2x"
TypeError: Cannot read properties of undefined (reading 'packageJson')
    at main (file:///Users/z/.npm/_npx/060caca6003d4dba/node_modules/chatgpt/bin/cli.js:11:39)

~
$ npx chatgpt@latest "write a shell command to convert input.mov to output.mp4, and speed it up 2x"
TypeError: Cannot read properties of undefined (reading 'packageJson')
    at main (file:///Users/z/.npm/_npx/0395a9b5d614bda1/node_modules/chatgpt/bin/cli.js:11:39)

@transitive-bullshit maybe this is something with my env? Not sure. Let me know if there's something I can do to help debug.

Hmmm I'm using https://github.com/sindresorhus/read-pkg-up to read the package.json version. Probably overkill, but still I'm surprised.

Thanks @zeke. I think this should be fixed in https://github.com/transitive-bullshit/chatgpt-api/releases/tag/v4.7.1

Nice! It works 💛


Amplify not recognizing my artifact directory during deployment.

appId: d1ms7jt11oekyc
region: us-east-1
Hello,
I think I have my build settings incorrect.  I am having trouble getting my build (baseDirectory) recognized by the deploy step.  I am deploying a vanilla react.js app with a typical "npm run build" script that creates a "build" folder.  For some reason, Amplify can't locate the build folder even though it's been generated.  I even inserted some postBuild commands to ls la the directory contents and confirm the build directory is there.
Below is my log file.
amplify_output_slh.txt
Any insight to this is greatly appreciated.

hi @slhernandez can you share your buildspec?

Hi @kahdojay.  I am no longer using Amplify.  Decided to go in a different direction with my deployment.  I am going to close this issue.  Thanks!

@slhernandez sorry to see you go. Any particular reason why you moved away?

My problem was that variables wasn't recognised in artifacts section and I had a 'cd' command in preBuild. So adding full path it worked.
version: 1.0
env:
variables:
BUILD_FOLDER: app
OUTPUT_FOLDER: dist
frontend:
phases:
preBuild:
commands:
- cd ${BUILD_FOLDER}
- mkdir ./${OUTPUT_FOLDER}
- npm install
- npm audit fix
build:
commands:
- npm run build-release
postBuild:
commands:
- pwd
- ls ${OUTPUT_FOLDER}
artifacts:
baseDirectory: /app/dist
files:
- '/*'
cache:
paths:
- node_modules//*

@koprinkov thank you, this worked for my team.

I ran headfirst into this today. I updated Angular 11 to Angular 12 (locally), made no other changes, tested my serve and build commands, everything worked the same. There were no code changes, etc. I didn't expect any problems as the same exact configuration had worked many times before (the website is and has been live).
I committed the package.json changes, and the build failed, artifact directory not found. I tried everything basic I could think of (mostly altering the paths to things that might make sense), even committing the build directory in the place it was supposed to be.
There are no cd commands, it's very basic build. But I now have a web app that won't deploy because no matter what I try, an artifact directory that used to be fine can no longer be found.

A year later this is the 2nd result for my search (after  issue #892 ), adding this in case it's helpful to anyone:
Fixed by adding the preceeding / in artifacts: path to  baseDirectory: /dist in your amplify.yml


upeptide_mapper_1_0_0 fail with empty input files

When running upeptide_mapper_1_0_0 with an empty input file, it crashes with the following error:

File "/home/manuel/Gits/Ursgal/ursgal_fork2/ursgal/resources/platform_independent/arc_independent/upeptide_mapper_1_0_0/upeptide_mapper_1_0_0.py", line 234, in main
*class_etxra_args
File "/home/manuel/Gits/Ursgal/ursgal_fork2/ursgal/resources/platform_independent/arc_independent/upeptide_mapper_1_0_0/upeptide_mapper_1_0_0.py", line 776, in map_peptides
for match in self.automatons[fasta_name].iter(self.total_sequence_string[fasta_name]):
AttributeError: Not an Aho-Corasick automaton yet: call add_word to add some keys and call make_automaton to convert the trie to an automaton.

I'd suggest to just return an empty output file and maybe print a warning, since this could crash e.g. a machine offset sweep workflow otherwise
Best,
Manuel

I had that problem before as well and as a work around I put it in a try/except loop for e.g. the parameter sweep workflow. For this type of workflow just allowing the upeptide mapper to pass also would let percolator crash afterwards, so that would need to be solved then as well.
However, I would like it to crash in other workflows to know that something is wrong.
But I guess we could work on exit messages/error messages to deal with this?

I personally prefer the option to also make percolator accept empty files, however I see that this would mean pulling this through for every node and also that there are other valid opinions on that.
I also fixed that with a try/except condition for now, however you need to accept all AttributeErrors, which could also be something completely different, especially in such a big node.
If we agree on raising an Error, I'd like to have something escape an UrsgalError or something similar.
If we agree on this, I'd add a custom exception which will also be useful in future I think ( I know we had some discussions on custom erros, still a big fan of this concept :) )
Best,
Manuel

I also like the idea of having an UsrgalError or more specific errors for different cases.  This could then be well implemented in a workflow and handling these errors specifically, like having no peptides to process etc.

I can add a new exceptions.py file with a general UrsgalError and a NoPeptidesToMap error (or similar) and fix this specific case.
In future, we could inherit other errors from UrsgalError for other  cases.

Yes, sounds good to me

Just made a draft, please have a look if you like it.
The names of the exceptions are open for discussion (best directly in the PR), as well as the location of the tests for these custom exceptions

Agreement from dev meeting:
Instead of generating Ursgal exceptions we will use built-in exceptions to cover empty files and potentially other cases.
Ideally we can discuss here, which cases should be covered, if it should be done on wrapper and/or node and/or controller level, and which exceptions should be used for which cases.

What about ValueError for empty files?

exception ValueError
Raised when an operation or function receives an argument that has the right type but an inappropriate value, and the situation is not described by a more precise exception such as IndexError.

One could argue that the passed path has the right type (str) but points to an empty file, which could be interpreted as an inappropriate value.
I could not find anything else which seemed suitable in my opinion.

What about this one:

exception EOFError
Raised when the input() function hits an end-of-file condition (EOF) without reading any data. (N.B.: the io.IOBase.read() and io.IOBase.readline() methods return an empty string when they hit EOF.)

I would argue that's a bit more specific to he actual problem.
I mean, the ValueError would fit as well, but in terms of usability, it seems to me like ValueErrors might happen more often and at a lot of different places, so skipping them would be fairly dangerous.

I would prefer the EOFError as well. Seems like the best fitting case for empty files in my opinion.

Ok, I missed that one but I totaly agree that EOF is more suitable :+1:


-- cp option does not appear in the CLI’s help menu

It's there if you enter "help run".

@Asisayag seems to be ok.. please reopen if you think otherwise...


Add seifrajhi as approver and reviewer for Arabic localisations docs

This commit will add @seifrajhi  as an owner (approver) and reviewer for Arabic content on the website. This is based on the discussion here #44862.

Thanks for the PR @seifrajhi, will you please update the commit description to remove the @? Such keywords are not allowed in commit messages

/assign @adowair
/assign @mboukhalfa


Analyzer Plugin: 'resultColumn' called on null

Hello, thank you for the great library!
Trying to do anything inside a moor file results in the following error being reported in VS Code. The analyzer plugin is successfully running - I checked it in analyzer diagnostics.
Errors are also not highlighted in the moor file, I see them only in the moor_generator output in the terminal.
Versions

Visual Studio Code v1.45.1
Dart Code v3.10.2
Flutter SDK v1.17.1

Analyzer Info
The analyzer was launched using the arguments:

C:\Flutter\flutter\bin\cache\dart-sdk\bin\snapshots\analysis_server.dart.snapshot
--client-id=VS-Code
--client-version=3.10.2

Exception
PLEASE DO NOT REPORT THIS ON dart-lang/sdk! This should be reported via https://github.com/simolus3/moor/issues/new instead. Message was Error while working on file:///D:/Reso%20Coder/SHORTENED/git_repo.moor, error NoSuchMethodError: The getter 'resultColumns' was called on null.
Receiver: null
Tried calling: resultColumns

#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:53:5)
#1      CreateTableReader.extractTable (package:moor_generator/src/analyzer/moor/create_table_reader.dart:35:32)
#2      MoorParser.parseAndAnalyze (package:moor_generator/src/analyzer/moor/parser.dart:65:40)
#3      ParseMoorStep.parseFile (package:moor_generator/src/analyzer/runner/steps/parse_moor.dart:11:19)
#4      Task._parse (package:moor_generator/src/analyzer/runner/task.dart:84:31)
<asynchronous suspension>
#5      Task.runTask (package:moor_generator/src/analyzer/runner/task.dart:41:26)
#6      MoorDriver.performWork (package:moor_generator/src/backends/common/driver.dart:107:18)
#7      AnalysisDriverScheduler._run (package:analyzer/src/dart/analysis/driver.dart:1975:24)
<asynchronous suspension>
#8      AnalysisDriverScheduler.start (package:analyzer/src/dart/analysis/driver.dart:1898:5)
#9      new ServerPlugin (package:analyzer_plugin_fork/plugin/plugin.dart:100:29)
#10     new BaseMoorPlugin (package:moor_generator/src/backends/common/base_plugin.dart:16:47)
#11     new _MoorPlugin&BaseMoorPlugin&OutlineMixin (package:moor_generator/src/backends/plugin/plugin.dart)
#12     new _MoorPlugin&BaseMoorPlugin&OutlineMixin&FoldingMixin (package:moor_generator/src/backends/plugin/plugin.dart)
#13     new _MoorPlugin&BaseMoorPlugin&OutlineMixin&FoldingMixin&CompletionMixin (package:moor_generator/src/backends/plugin/plugin.dart)
#14     new _MoorPlugin&BaseMoorPlugin&OutlineMixin&FoldingMixin&CompletionMixin&NavigationMixin (package:moor_generator/src/backends/plugin/plugin.dart)
#15     new MoorPlugin (package:moor_generator/src/backends/plugin/plugin.dart:28:43)
#16     new MoorPlugin.forProduction (package:moor_generator/src/backends/plugin/plugin.dart:34:12)
#17     start (package:moor_generator/integrations/plugin.dart:7:34)
#18     main (file:///C:/Users/mreso/AppData/Local/.dartServer/.plugin_manager/bc0d343db770f933c8def53d1a73e1cb/analyzer_plugin/bin/plugin.dart:13:5)
#19     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:297:32)
#20     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

The moor file is rather simple:
CREATE TABLE git_repos (
  id INT NOT NULL PRIMARY KEY AUTOINCREMENT,
  user INT REFERENCES users (id),
  /*WRITING HERE TRIGGERS THE EXCEPTION*/
);

CREATE TABLE users (
  id INT NOT NULL PRIMARY KEY AUTOINCREMENT
);

Haven't tried it, but does the error go away if you define users before git_repos? I think the generator sorts tables by foreign key references, but maybe there's a bug there.

There were two issues here: The first one was a NPE in the sqlparser package when extracting the schema from a column definition without a typename (e.g. CREATE TABLE foo (bar);). We'll now interpret bar as a blob columns which matches sqlite's documentation.
Then there's another NPE in moor_generator when the schema reader throws an exception, which I've now fixed as well.
Both problems are fixed on develop now - I'll try to release a new version soon.


Update obsidian.css for light theme support

also adding to theme light declarations - only added to dark theme in previous commit (sorry)

My fault...I didn't find it either.😂😂😂


[Implement sniff] WordPress must be spelled correctly

Rule type: Error
Rule:

Spell “WordPress” correctly in all public facing text: all one word, with both an uppercase W and P.

Ref: Link to the Theme Review handbook page & subsection where the rule can be found, i.e. https://make.wordpress.org/themes/handbook/review/required/#naming
We do not need to make this just public facing as it is automated. The only exception that I can think of is when it is being used in a function then it can be all lower case.
To do:

[ ] Create unit tests
[ ] Create new sniff

A PR for the actual sniff has been pulled upstream https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/pull/909.
What would now be needed for TRTCS:

[ ] Merge back WPCS develop into "our" branch after the above PR has been merged
[ ] Add the new sniff to the TRT ruleset


Add example of FieldPath usage

The added fields and query show how to use FieldPath to distinguish between field names with ".", and a nested field.

@sdlin thanks for this PR!  I have no problem with the content however I can't really approve it in isolation.  That snippet will appear on this page:
https://firebase.google.com/docs/firestore/query-data/queries
That means:

The text would no longer match the snippet, since it says "The following query returns all cities with state CA:" but we'd also be showing a FieldPath snippet in the same block.
There are ~12 other snippets on the page which we'd want to update if we go forward with this.

For now can you break this snippet out into a totally separate one so that it won't appear in the docs right away?  Then we can decide if we want to invest in the other 5-13 snippets and get it in there.
I know that seems crazy but that's the pain of supporting so many languages!

@samtstern thanks for the quick review.
W/ regards to the first point, I modified the data and query so that only California cities are returned.  This still isn't precisely in-line with what the surrounding doc says ("with state CA").
And yeah totally get that by adding this to only one language, the snippets would be inconsistent across languages.
In any case, given my change which attempts to address the first point, let me know if you still think I should break this out.
Thanks!

@sdlin sorry I think I wasn't clear before.  Basically I don't think this FieldPath syntax belongs in the simple_queries snippet at all.  That snippet is often the first one a new Firestore user sees, it's not the time to introduce more complex syntax.
I'd say we can create a new snippet for this and then find a place for it in the docs.  Or maybe there's an existing more complex snippet which would be a good place to show this off.

Got it, thanks for the clarification!


detect openrisc core, support orbis32, orbis64, orfpx32, orfpx64, orvdx64

Thanks for your contribution !


AR "via/viaTable" relations:  sort by junction table field

Unfortunately, the problem with sorting by field in junction table is still exist.
For example we have three tables:
item:

| id | title  |
---------------
| 1  |  dog   |
| 2  |  cat   |
| 3  |  pup   |

item_has_value:

| item_id | value_id | rate |
-----------------------------
| 1       | 1        | 100  |
| 1       | 2        | 200  |
| 2       | 3        | 200  |
| 2       | 4        | 100  |
| 3       | 2        | 100  |

value:

| id | title  |
---------------
| 1  |  bow   |
| 2  |  wow   |
| 3  |  mew   |
| 4  |  miaow |

And these relations:
Item Model:

public function getValues()
{
        return $this->hasMany(Value::class, ['id' => 'valueId'])->via('itemValues');
}

public function getItemValues()
{
        return $this->hasMany(ItemHasValue::class, ['itemId' => 'id'])->orderBy(['rate' => SORT_DESC]);
}

And query to get linked values (properly sorted by rate field in juction table):
... ->getItems()->with('values')->...

It fails, because the orderBy applies only in the first sql query (as it mentioned in https://stackoverflow.com/questions/27690401/activerecord-where-and-order-on-via-table/27691992#27691992).
But if we add these relation in ItemHasValue model:
ItemHasValue Model:

public function getValue()
{
        return $this->hasOne(Value::class, ['id' => 'valueId']);
}

And perform these query:
... ->getItems()->with([
    'itemValues' => function($query) {
                $query->with('value');
            },
])->...

It will sort related values properly. But performs the exact same sql queries (as in the via relation).
So, question is - why does it happen, and why we can't sort properly by junction table field?
Maybe it is possible (in via / viaTable relations with sort) to add to the second query something like this: ... ORDER BY FIELD(id, 2, 1, 3, 4) ... or sort it internaly ?

Do you mean by "still exists" that the problem was going to be fixed by a pull request? If yes, which one?

No. I don't know any pull request which solves this problem. I mean that there was several issues and discussions about this problem, unfortunately without satisfactory solution. E.g.:
https://github.com/yiisoft/yii2/issues/16606
https://github.com/yiisoft/yii2/issues/10174
https://stackoverflow.com/questions/27690401/activerecord-where-and-order-on-via-table
https://forum.yiiframework.com/t/how-to-orderby-on-viatable-in-hasmany-model-relationship/80043

@degibons do you make Query class for every model?
If so, then you can do this in Query class

    /**
     * @param string $field
     * @return string
     */
    public function prependWithTableAlias($field)
    {
        /** @var ActiveQuery $this */
        list(, $alias) = $this->getTableNameAndAlias();
        return $alias . '.[[' . $field . ']]';
    }

    public function orderBy($columns)
    {
        array_map(function ($order, &$key) {
            $key = $this->prependWithTableAlias($key);
        }, $columns);

        return parent::orderBy($columns);
    }

Hi @degibons,
Though this is an old post, most probably you already got an answer/workaround for it ;) Today I faced exactly the same issue and found a solution that works perfectly without changing the DB-relation signature.
My solution as below:

Instead of using $this->hasMany use an ActiveQuery
Use the property $multiple on the query to indicate that this query will return an array of results.
Use leftJoin method, and define the junction table relation inside it.

Example:
`public function getValues()
{
$query = Value::find()->leftJoin(ItemHasValue::tableName(), ItemHasValue::tableName() . '. value_id = ' . Value::tableName() . '.id' )
->where([
// define all ur conditions here
])
$query->multiple = true;
return $query;
}`
I hope this can help you or anyone who faces this issue.
CC: @samdark can you please document this somehow, to help anyone who wants to order via a junction table and maintain the sort on the primary table.

Hi @degibons,
Though this is an old post, most probably you already got an answer/workaround for it ;) Today I faced exactly the same issue and found a solution that works perfectly without changing the DB-relation signature.
My solution as below:

Instead of using $this->hasMany use an ActiveQuery
Use the property $multiple on the query to indicate that this query will return an array of results.
Use leftJoin method, and define the junction table relation inside it.

Example:
`public function getValues()
{
$query = Value::find()->leftJoin(ItemHasValue::tableName(), ItemHasValue::tableName() . '. value_id = ' . Value::tableName() . '.id' )
->where([
// define all ur conditions here
])
$query->multiple = true;
return $query;
}`
I hope this can help you or anyone who faces this issue.
CC: @samdark can you please document this somehow, to help anyone who wants to order via a junction table and maintain the sort on the primary table.

@anasjaghoub that's quite specific case so I'm not sure where/how to docuement it best...

@anasjaghoub that's quite specific case so I'm not sure where/how to docuement it best...

I ran into this same issue of wanting to sort by a junction table with a dynamic relations. I was able to come up with a work around that does work correctly but I'm not sure if what I'm doing is "good" or "optimal" and if there is any arguments for or against my solution. If one of the Yii Devs can check my post at stack overflow and let me know your thoughts on this it would be very much appreciated. I love Yii and plan on using for pretty much all of my web apps going forward so I just really want to learn and understand the why behind this and the best way to handle this moving forward
https://stackoverflow.com/questions/69329769/yii2-dynamic-relational-query-junction-with-sort-uses-2-queries-instead-of-a-joi

We have the same issue. A leftjoin should work but its inefficient.
What if....
Yii generate the IN() query as usual but inlcudes an automatic order by FIELD(pk, 1,2,3)? Yii always adds then an order by FIELD with the same values as pasted inside the IN(). Since the IN ids are the ordered (because they returned ordered from the previous relation table) the order would be then correct, right?
So, when Yii generated the second (IN) query as: WHERE user.id IN (1, 2, 4, 10, 954, 979, 1080, 1102, 1116)  and adding an always order by FIELD (user.id, 1, 2, 4, 10, 954, 979, 1080, 1102, 1116)  would save the ordering, doesnt it?
Does that make any sense?

We have the same issue. A leftjoin should work but its inefficient.

if left join is inefficient, your indexes are bad.

What if....
Yii generate the IN() query as usual but inlcudes an automatic order by FIELD(pk, 1,2,3)? Yii always adds then an order by FIELD with the same values as pasted inside the IN(). Since the IN ids are the ordered (because they returned ordered from the previous relation table) the order would be then correct, right?
So, when Yii generated the second (IN) query as: WHERE user.id IN (1, 2, 4, 10, 954, 979, 1080, 1102, 1116)  and adding an always order by FIELD (user.id, 1, 2, 4, 10, 954, 979, 1080, 1102, 1116)  would save the ordering, doesnt it?
Does that make any sense?

No, as SQL does not care what order things are in... ORDER BY is for that

No, as SQL does not care what order things are in... ORDER BY is for that

Indeed, thats becuase the FIELD addition would work, since IN() does not care about order.
We tested that with an quick'n'dirty hack:
Index: vendor/yiisoft/yii2/db/conditions/InConditionBuilder.php
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/vendor/yiisoft/yii2/db/conditions/InConditionBuilder.php b/vendor/yiisoft/yii2/db/conditions/InConditionBuilder.php
--- a/vendor/yiisoft/yii2/db/conditions/InConditionBuilder.php
+++ b/vendor/yiisoft/yii2/db/conditions/InConditionBuilder.php	(date<PHONE_NUMBER>674)
@@ -91,7 +91,7 @@
             $column = $this->queryBuilder->db->quoteColumnName($column);
         }
         if (count($sqlValues) > 1) {
-            $sql = "$column $operator (" . implode(', ', $sqlValues) . ')';
+            $sql = "$column $operator (" . implode(', ', $sqlValues) . ') order by field('.$column.','.implode(', ', $sqlValues).')';
         } else {
             $operator = $operator === 'IN' ? '=' : '<>';
             $sql = $column . $operator . reset($sqlValues);

That would work in our case (for testing purposes). This shows what I mean.
So - bit it seems, that an addiotional leftJoin is the master solution on those issues (https://github.com/yiisoft/yii2/issues/17166#issuecomment-766326257)?

@Commifreak Unfortunately, this seems to be problematic as soon as an additional where clause exists. Right? Then all conditions gets wrapped into ( )...

@DrummerKH Does your solution really work? Which queries gets created by
public function getModles(): ActiveQuery
{
        return $this
            ->hasMany('model_table', ['id' => 'relation_id'])
            ->viaTable('via_table', ['owner_id' => 'id'])
            ->leftJoin('via_table via', 'via.relation_id = id')
            ->orderBy([
                'via.priority' => SORT_ASC,
            ]);
}

Within my app the second query contains the IN clause with the ids of the first query. But there is no constraint regarding the model id. Considering these values in the linking table (from the original post):
| item_id | value_id | rate |
-----------------------------
| 1       | 1        | 100  |
| 1       | 2        | 200  |
| 2       | 3        | 200  |
| 2       | 4        | 100  |
| 3       | 2        | 100  |

Item::find()->where(['id' => 3])->with('values')->one() would select (with the second query containing the IN clause) the records with value_id=2, which are two instead of one! The first one has rate value 200, which is wrong. In case of an item has only a single value entry, it doesn't matter, of course, since only the first one gets used and the others will be ignored. But if there are more, then the order may be wrong.
Correct me if I'm wrong (I'm confused anyway). At least in my code that doesn't seem to work as intended.

@samdark Is it possible that you or anyone implement that? The approach of @Commifreak looks promising but is not complete, afaik. It's to complex for me to provide a solution, sadly...
Or could you provide a working work around? I assume that the sugessted solutions doesn't work without restrictions, if I'm not wrong.

Found this answer which should reliably work. With this it should be:
public function getValues()
{
    $ids = $this->getItemValues()->select('itemId')->column();
    return $this->hasMany(Value::class, ['id' => 'valueId'])
        ->via('itemValues')
        ->orderBy([new Expression('FIELD (id, ' . implode(',', $ids) . ')')]);
}

public function getItemValues()
{
    return $this->hasMany(ItemHasValue::class, ['itemId' => 'id'])
        ->orderBy(['rate' => SORT_DESC]);
}

Can anyone confirm that this is a working solution that works in all situations? I think it should work as long as no other orderBy gets appended to the query. To be 100% sure a transaction might be needed as well.


Query upgrade

Remove unnecessary type, and use explicit type construction to expose meaningful field names at call sites.

Superseded by #1109


Adding alerting for 5XX status codes from application and LB

Same as in the profiles service. I'll run the cloudformation once this is merged to master

This PR was successfully deployed to production by CircleCI build 154 at 2017-03-24 09:54 GMT.


Upgrade Testing Framework to support dlopen & all documentation

Subject
Goal

Building the headers, including the testing suites and allow a currently unsupported function to be defined via stubs.

So that the testing suites can be written to the stubs, which will fail but will build.
Then the implementors of the new functions have a testing suite they can use to compare against implementation.

Upgrade the makefiles to build WEAK_SKELETON_SRC as a list of weak skeletons srcs that will be build into the main testing app, as real weak functions

Upgrade the makefile in the templates to support this

Upgrade the testing suites to test this

Upgrade the templates to also use dlopen as an example of how to remove the build time dependancy, and introduce a runtime dependancy.

Add a switch to disable the binary loading, and swap back to weak functions (by default the DLOpen will be used).

Add an option to if the library is not found, then use the weak functions, and display a warning? (Also a switch)

https://github.com/rdkcentral/ut-core/wiki/4.0.1:-Standards:-Dynamic-Library-Search-Order

Be aware that LD_LIBRARY_PATH should likely be used, because hardcoding the default paths may not be a good idea, considering the vendors are running the testing suites against their build system, and rootfs
Consider displaying the paths that your using to make it clear which libraries are being found.

Requirements

Upgrade the testing suites to support the new dlopen model
Upgrade all the testing suites makefiles to support the new model
Upgrade all the documentation to support the new model
Upgrade the checking scripts for missing functions, to ignore weak functions (New task)

Include the SKELETON_SRC as part of the ut-core build in both linux + arm building.
May need to separate as a SKELETON_SRC
Notes (Optional)
No response

Due to the future upgrade for AIDL interfaces, and the upgrades to the testing suite's we've decided to give examples of this under the testing model, but not to migrate the testing suites to use it.. As it won't be required for the future


Update pom.xml

@bhaskaryadav369 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


_example Failure to output

I run _example/main.go expected

but got

jsonType and timestamp output is wrong

This points to an incompatibility in returning JSON objects over the wire for in memory tables.
James, can you please take a look at what's going on here?

Big thanks for reporting this problem @Penglq. @JCOR11599 got the fix into main and it'll go out in our next release this week. Let us know if you hit any more issues with go-mysql-server and we'll be happy to jump on them.


Bad coloring?

Hey,
I was trying to color the image of women lena which has been used in image processing from the beginning using the full version. All of the result is pretty much bad.
this is the image and ur default images is also there.

outputs

I am not sure what is wrong!
Help me out.

I'm facing the same problem but my images are pink/violet instead of green

@emilwallner help us out man!

If you are having installation issues, try the ready-made version on FloydHub. If you are on a budget you can upload it to Google Colab, if you do, please ping me the link so you can help other people in your situation.


Bug fix 3.10/fix chaos test identifiers

Backport of:
https://github.com/arangodb/arangodb/pull/16876

Only unrelated known failures in Jenkins.


Add AWS dashboard to measure account creation

@c-castillo something like that. How many users are registered I think.

Thanks @oed for the clarification. I think in this case the users are measured by the number of rootStores created (not by address, since you can add N devices to your 3Box)

Yes, precisely 👍🏻

@msena @oed @dazuck
The WIP dashboard is located at
https://datastudio.google.com/open/1lrhZVm3o5LvSvGadibqozMbwWQhP7bIE

@c-castillo it's not letting me open the link, can you add our emails directly to the dashboard from data studio. I dont think we have permission


Error with dotnet 6.0 (samples/6.0/SwashbuckleSample)

dotnet build gets the following errors, lots of 'swagger' examples that I have found get the same errors (sorry if there is a simple fix for this I am missing)
C:\Users\jeff\dev\AspNetCore.Docs-main\aspnetcore\tutorials\web-api-help-pages-using-swagger\samples\6.0\SwashbuckleSample\SwashbuckleSample.csproj : error NU1100: Unable to resolve 'Microsoft.EntityFrameworkCore.InMemory (>= 6.0.0-rc.2.21480.5)' for 'net6.0'.
C:\Users\jeff\dev\AspNetCore.Docs-main\aspnetcore\tutorials\web-api-help-pages-using-swagger\samples\6.0\SwashbuckleSample\SwashbuckleSample.csproj : error NU1100: Unable to resolve 'Swashbuckle.AspNetCore (>= 6.2.3)' for 'net6.0'.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 14fce65f-1337-58cb-2193-8d8e0e62792d
Version Independent ID: 59222460-a971-e76e-4fe9-23d4d7d768ee
Content: Get started with Swashbuckle and ASP.NET Core
Content Source: aspnetcore/tutorials/getting-started-with-swashbuckle.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @zuckerthoben
Microsoft Alias: scaddie

Hi @jefflindholm. I just double-checked this, and it all appears to work fine for me. I even deleted all my cached NuGet packages to make sure. As a first step, can you run dotnet nuget list source and double-check that you see an enabled source for nuget.org?

That was it thanks, I forgot I was on the work network and nuget packages come from our manager, sorry

No worries. I'm glad it's working for you now.


Update aws-java-sdk-core to 1.12.422

Updates com.amazonaws:aws-java-sdk-core from 1.12.415 to 1.12.422.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.amazonaws", artifactId = "aws-java-sdk-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #455.


THRIFT-3774 The generated code should have exception_names meta info

Travis CI failed with
Broadcast message from root@testing-gce-d357df5d-d4c0-42df-9bcc-a6a7f62bb65e

	(unknown) at 10:58 ...

The system is going down for power off NOW!

The command docker build -q -t thrift-build build/docker/debian exited with 1.

How can I restart the check process?

Jenkins failed with SSL error: sslv3 alert certificate expired.
src/thrift/server/simple.d:143: Client died: thrift.transport.ssl.TSSLException@src/thrift/transport/ssl.d(223): SSL error: sslv3 alert certificate expired.
----------------
??:? pure @safe bool std.exception.enforce!(bool).enforce(bool, lazy object.Throwable) [0x515e16]
??:? void thrift.transport.ssl.TSSLSocket.checkHandshake() [0x5138d9]
??:? ulong thrift.transport.ssl.TSSLSocket.read(ubyte[]) [0x513457]
??:? ulong thrift.transport.buffered.TBufferedTransport.read(ubyte[]) [0x51c819]
??:? void thrift.transport.base.TBaseTransport.readAll(ubyte[]) [0x51be10]
??:? void thrift.transport.buffered.TBufferedTransport.readAll(ubyte[]) [0x51c9d1]
??:? thrift.protocol.base.TMessage thrift.protocol.binary.TBinaryProtocol!(thrift.transport.base.TTransport).TBinaryProtocol.readMessageBegin() [0x4f02ea]
??:? bool thrift.codegen.processor.TServiceProcessor!(async_test.AsyncTest).TServiceProcessor.process(thrift.protocol.base.TProtocol, thrift.protocol.base.TProtocol, std.variant.VariantN!(32uL).VariantN) [0x4ec275]
??:? void thrift.server.simple.TSimpleServer.serve(thrift.util.cancellation.TCancellation) [0x51a32b]
??:? void async_test.ServerThread!(thrift.server.simple.TSimpleServer).ServerThread.run() [0x4ec0fa]
??:? void core.thread.Thread.run() [0x54c731]
??:? thread_entryPoint [0x54c251]
??:? [0x17205181]
Build timed out (after 480 minutes). Marking the build as failed.
Build was aborted
[WARNINGS] Skipping publisher since build result is FAILURE
[TASKS] Skipping publisher since build result is FAILURE

How can I restart the check process?

Modify the pull request.

@Jens-G Thanks. I think I should do this after Jenkins's certificate fixed.

@laozhp has this been merged yet?

@laozhp I incorporated this into a namespacing fix here: https://issues.apache.org/jira/browse/THRIFT-3834

@jfarrell this PR can be closed - it was merged / fixed elsewhere.  Jira ticket is closed.


Update User Guide

User guide is the default AB3 template.
Let's add the following sections:

Introduction
Quick Start
Notes about features
FAQ
Command summary

Looks good to merge 👍


Request for Source Code and Assistance with Reproduction

We are focusing on your recent work and have been attempting to reproduce the reported results. However, we have encountered challenges when using the provided binary program, and we need your help to obtain the corresponding source code.
We noticed that you compiled afl-clang-fast using clang-6.0, but the benchmark you provided was compiled under clang-10. Therefore, we attempted to compile using clang-6.0, but ultimately we were unable to perform fuzz testing with the afl-fuzz you provided. We will demonstrate the specific error messages and trigger situations using the code and screenshots below.
Therefore, we would appreciate it if you could provide detailed information about the compilation to assist us in conducting our experiments. If it's convenient, providing the source code for replication would be even better.

export EDGE_POTENTIAL_PATH="abs_path"
export BELIEFFUZZ_CFLAGS="-res=$EDGE_POTENTIAL_PATH"
export BELIEFFUZZ_PATH="abs_path"

git clone https://github.com/axiomatic-systems/Bento4.git
cd Bento4
cmake -S . -B ./Build -DCMAKE_C_FLAGS="$BELIEFFUZZ_CFLAGS" -DCMAKE_C_COMPILER="$BELIEFFUZZ_PATH/afl-clang-fast" -DCMAKE_CXX_FLAGS="$BELIEFFUZZ_CFLAGS" -DCMAKE_CXX_COMPILER="$BELIEFFUZZ_PATH/afl-clang-fast++"
cmake --build Build
cd ../
$BELIEFFUZZ_PATH/afl-fuzz -m none -p -r -d -c $EDGE_POTENTIAL_PATH/edge_potential.txt -i fuzz_in -o fuzz_out_$(date +%m%d%H%M) -- ./Bento4/Build/mp4info --show-layout --show-samples --show-sample-data @@

For Bento4 compilation, you could try 3dc8d5, which is the exact version we used in our previous experiments :)


AICE Early version released! Thank you very much!

@ByBurton and @Sh0wdown
Thank you very much!
https://github.com/ByBurton/AI_Character_Editor/releases/tag/EarlyVersion

Thanks for the heads up.

I have a 2 question:
1 How to embed .aic file into the game?
2 Need to wait for the next unofficial patch to be released?

@Lolasik011

Please read my Readme on the repository page
You have to create one with my editor, and then move that created file into the stronghold crusader installation directory into the aic folder.
Yes.

@Lolasik011

Please read my Readme on the repository page
You have to create one with my editor, and then move that created file into the stronghold crusader installation directory into the aic folder.
Yes.

Ok.
Thanks again !

@ByBurton and @Sh0wdown
I have 2 more small questions :+1:  ^_^
1 Can I somehow translate your program into Russian?
2 Will the manual on the use of the program ?   :D)

I don't plan on integrating any languages besides English.
I won't write a manual, but after I am done with the functions, I will add tooltips that are shown if you hover over an option in my editor.

I don't plan on integrating any languages besides English.
I won't write a manual, but after I am done with the functions, I will add tooltips that are shown if you hover over an option in my editor.

Ok, I get it ... Thanks for the info.

So right now you can create and modify AIs but can't test them in-game yet?

@J0J0Jo Yes, the current UCP version has not yet been released.

@ByBurton @Sh0wdown
Hey guys, I want to write up instructions for those that want to apply the Sh0wdown/UnofficialCrusaderPatch. There are many Mac OS X users that purchased Stronghold Crusader HD. I tried running the exe patch with Wine Bottler but it doesnt work. Is it possible to tell us where to copy all the files in needed directories?

You are teasing us :D
I hope @Sh0wdown doesn't add to many things and just releases the next patch soon. In my opinion, not every parameter must be deciphered for the first iteration. The ones you have found out already open up so many possibilities. I think the wait is killing most people here ;)

Haha, you know. He was sick this week, so please bare with him.

You are teasing us :D

Yes us tease :)

Yeah nobody is demanding anything, we are super grateful for the patches :)

Many thanks for this, will be testing it out for a while!
Was trying to change Phillip to create pikemen there for his sieges and it didn't work though :( I think its because you can't set his poleturners to create pikes yet and you can't set him to buy in pikes yet.

Many thanks for this, will be testing it out for a while!
Was trying to change Phillip to create pikemen there for his sieges and it didn't work though :( I think its because you can't set his poleturners to create pikes yet and you can't set him to buy in pikes yet.

The next UTC version hasn't been released yet so the game atm has no way of reading the AIC files iirc.

@Panbutt That is true. Also you can't change workshops in my editor yet.

Unfortunately this doesn't seem to be working for me at all, I have tried simple changes such as giving Phillip archers as well as spearmen for his main attacks and giving Sultan slingers as well as Arab archers and Arab swordsmen for his attacks and I installed it in a new aic folder but the attacks are still just seem the same as vanilla.

How often do I have to write this: You would have to install it with the UCP, but the latest version is not out yet!

OK sorry I didn't realise, I had seen your statement before but I didn't know what you meant, I thought it would still make changes as a standalone feature.

Вообще-то. Мне кажется и я выржаю общее мнение, что вы делаете самый мощный и лучший мод в истории СТронгхолда. Так глубоко и тщательно его никто не изменял. Я восхищён вашими трудами. Я не обещаю, Но когда выйдет мой мод, надеюсь, вы смоежет взять всю звуковую часть мода. Она нейтральна к эпохам и времени. Звуки делал лично я давно. менял, подгонял и они сделаны классно.
Хотел узнать а кто может мне конкретно объяснить и ответить, что из того, что я предлагал у себя в теме - будет сделано? И что можно ещё будет доделать? Например можно ли добавить звуки новые к уже существующим?
Actually. It seems to me and I express the general opinion that you are doing the most powerful and best mod in the history of Stronghold. Nobody cheated him so deeply and thoroughly. I am delighted with your work. I do not promise, but when my mod comes out, I hope you can take the whole sound part of the mod. It is neutral to epochs and times. I personally made sounds for a long time. changed, customized and they are made cool.
I wanted to find out, and who can specifically explain to me and answer that what I offered in my subject line will be done? And what else can I finish? For example, is it possible to add new sounds to existing ones?

@Baummannn Equipment.

Checked the tool.
What are sortie units exactly?
And am assuming there can only be a max of 8 seige equipment for an attack or can there be more? Is it based on the number of engineers and then the AI divides them based on the ratio defined in the siege unit types? ( similar to crusaders 2).
Also what is the likelihood of finding out what controls tax levels and rations?

First of all, chill. Please head over to my repository, I will open an issue up there.
https://github.com/ByBurton/AI_Character_Editor/issues/13

Will do. Didnt mean to come across as aggressive:D

@Mizo5119
I don't think we can have more than 8 siege engines at the moment, because even without the mod, saladin and wolf send more engineers than necessary for their siege engines. This however does not result in the remaining engineers to build more siege engines. They rather just stand around unfortunately.


[Android] Unhandled exception

Received this exception  when I first run my project on Android
Java.Lang.NullPointerException: Attempt to invoke virtual method 'void android.widget.TextView.setTextColor(android.content.res.ColorStateList)' on a null object reference

I am getting close to release date. Please submit a demo reproducing the issue if you want a fix to be included. Thanks

I was failed to copy that issue on another project.
I can't provide the project I'm developing now,It will cause some commercial issue.
I thought maybe it was just somewhere wrong at my project.
I'll post it here If I find it out later.
Thx for your Reply.


Offline Release

Related issue:
https://github.com/Microsoft/vscode/issues/54354 Was worked stop for this?
https://github.com/MicrosoftDocs/intellicode/issues/102 Is actually for an extension but it is the same problem (download of schemas and none included by default) and is code by Microsoft too.
So it looks like a trend to require downloads after installation for features to work instead of providing a base copy of the schemas and features that dont actually require internet connection to work out of the box.
This feature is needed for highly protected networks with extremely limited access to the internet such as countries like China, corporate sensitive networks, or even government networks with limited to no access to the internet.
Maybe make the schemas their own package so you can download them and any extensions that needs them can use them/have the package as a dependency?

@jose-pr I can think of the following steps:

Have a machine without internet
Install VS Code
"something" does not work

What exactly is the "something" that does not work so I can route this issue to the right owner?


[Bug]: Crash when signing into LNMarkets with slashtag

Describe the bug
Bitkit displays 'Unexpected Error' screen with error: 'URL must be a string' when attempting to login in LN Markets at. Happens with QR or using SlashURL.
Reproduce

Go to https://lnmarkets.com/en/login/slashtag
Scan QR with Bitkit

Screenshots / Recording

Operating system
Android 13 TKQ1.220829.002
Bitkit version
2c5b07726fd35cb796a9566c47d0f6c36fda4ef4
Log output
No response

No longer crashing. @Slashtag is deprecated" message is shown. Closing.


Support for Arduino / AVR ATmega CPU ?

Hello, first thank you for excellent OLED library. I'm using it on ESP8266 a lot!
Now I'd like to use it on generic Arduino AVR Atmega MCU but I can't.
I made change to architectures=* in library.properties file, but there are many errors during compilation. Hmmm, apparently these settings matters.
So my question is: do you have plan to add support for AVR Atmega MCU?

Hi,
changes I made to run library on Mega2650 with SH1106 display controller:
library.properties:
Add "avr" to architectures.

Hi
I tried Todelec fix for an Arduino UNO (ATMega 328), and change the déclaration to :
SH1106 display(0x3c, A4, A5);
to take into account SDA and SCL pins of the UNO.
No errors in compilation, but the display remains blank.
Any ideas ?


Added "Migrate a React client-side application to server-side with Next.JS" post

@GusTTShowbiz Thanks for your comments! Errors are fixed


auth: correct range checking

Current permission checking allows access to a key b if a user has a
role that is granted to a range [a, b). The range shouldn't include
the end key so this commit let the checking logic excludes it.
/cc @soyking

Oops, I'll fix the CI later...

Closing this PR in favor of https://github.com/coreos/etcd/pull/6404


Question about the usage of connection.onInitialize

Hi there,
I have a general question about the usage of connection.onInitialize. Would it be correct to call it twice with different configs? (E.g. to keep code completion logic and its config together in one file, but other configs in other files.) Something like this:
// setup.ts
connection.onInitialize(() => ({
  capabilities: {
    textDocumentSync: documents.syncKind
  }
}));

// completion.ts
connection.onInitialize(() => ({
  capabilities: {
    completionProvider: {}
  }
}));

It seems to work, but I'm not sure if this is actually supported. I try to write a tutorial and this would help to keep a linear flow in the logic/explanations.
Is it also okay to call connection.onInitialize after connection.listen();? This seems to work as well.

@donaldpipowitch this actually installs a handler on the server side which is called when the client sends the initialize request to the server. You can modify the handler but it will not be called if the initialize requests has already been received. If it has not been received only the new handler will be called.
Closing. Please ping if something is still unclear.

Thank you for your answer. So my take away is: you should use onInitialize only once, because it overwrites the older handler and it can be used after listen, because it depends on the initialize request from the client. Right?
Thank you

Right.


feat(choices): use object as source iterating over its properties

We'll be able to use an object as the source for the choices (iterating over its properties)
Demo plunker
Pending:

[x] Need to add a watch to update choices when source object changes

Added docs here - https://github.com/angular-ui/ui-select/wiki/ui-select-choices

@dimirc It also does not work with 'nested' objects. The following setup:
	<ui-select ng-model="actionCtrl.model.action.delimeterType" style="width:100%">
		<ui-select-match>{{$select.selected}}</ui-select-match>
		<ui-select-choices repeat="type as (type, name) in actionCtrl.delimiterTypes | filter: $select.search">

		</ui-select-choices>
	</ui-select>

tries to bind all the properties of actionCtrl. http://www.screencast.com/t/jMN4BZiI

@pdegenhardt @dimirc I have resolved the issues and release 0.13.2

Can you please check out issue #1255  I opened yesterday?
I have the exact same problem you described and upgraded to v0.13.2 but still not working
Thanks

Hi amcdnl
Your Change #1208 broke the track by expression
Example: n in [42, 42, 43, 43] track by $index
https://github.com/angular-ui/ui-select/pull/1208
Greets Samuel


"Gârbova de Jos" is listed in the drop-down menu

Description:
"Gârbova de Jos" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Gârbova de Jos" in the search bar.

Expected results
The location is listed in the drop-down menu as "Gârbova de Jos, Alba (Aiud)".


feat: Add support collapse span

Ref https://github.com/apache/skywalking/discussions/12785 II. Collapse span
https://github.com/user-attachments/assets/f45ab2d8-5ee1-480a-907c-9c845cf97eaa

You need to sync to main repo, or wait for the other part is done?

You need to sync to main repo, or wait for the other part is done?

It will take some time for the other parts to be completed, let me sync first.


Batch Tracing from Folder ignores preset

First and foremost, I would like to thank you for your great work.
I have just spent hours trying to achieve batch processing with image tracing using Adobe tools, where I can choose my own preset, but to no avail. I also explored scripting, which was able to load my preset, but the number of colors was always set to 100, even though it is defined differently in the preset. In the end, I tried to define the parameters in the script itself, but I lost patience because the settings offer different possibilities and options than the sliders in Illustrator itself.
I was just about to go to bed when I found your scripts, which are brilliant. Unfortunately, there is a problem with batch processing from a folder where the selected tracing preset is being ignored. Everything works correctly with open files. I was also a bit confused because the script can only be opened when a file is open; otherwise, it does not respond.
Perhaps you have a solution to this problem or an idea of how to fix it?
Kind Regards

@kendesign thanks for appreciating my scripts.

The scripting has the problem of defining a color limit that is different from the setting in the Image Trace panel.  The Adobe developers seem to have mixed up the attributes. I plan to document this problem in a future blog post.
"can only be opened when a file is open" — it's a bug, I'll fix it in the update. Thanks for the report.
"selected tracing preset is being ignored" — in my tests, the Trasing preset is applied to the images from the folder. I haven't seen it ignored. Can you capture video?

Hey, sure: https://vimeo.com/833097003/224de91af7
Illustrator is up to date, 27.5.
It doesn't seem to recognize the preset at all.

@kendesign I have 27.5 installed and cannot reproduce this problem.  Thank you for trying to help!
Wich latest possible Version of Illustrator will do the trick?

@kendesign this is simple: the scripting language from the preset in your example also loads the values Colors: 100 instead of 10, and what looks white to you is RGB 253,253,253, so Ignore White does not work. This is the same bug in Adobe's scripting language.


Update org settings and help center "Notifications" terminology

Revised version of #22342.
Updates made:

Squashed Organization settings UI changes into 1st commit.
All help center and API documentation updates in 2nd commit.

Notes:

For the help center left sidebar title for the Organization language for automated messages and invitation emails, I went with the shortened version of Configure organization language. I felt like the article itself makes it clear what the setting is doing. See screenshot below.
I also used that shortened version for the header for the instructions block on that page. While I removed 'Configure' from the main article title and instead used Organization language for automated ...; again see same screenshot below.
This PR does not rename / redirect the URL / file name for change-the-default-language-for-your-organization.md; that will be done in a separate follow-up PR.
Confirmed that all 5 help center instruction blocks for the Organization settings under the new Automated messages and emails heading are updated.
Used git-grep to review all help center files with "notifications" in the text for possible updates related to this UI change.

Fixes: #22136.

Screenshots and screen captures:

Organization settings admin UI updates

Organization language for automated messages and invitation emails

Zulip notification bot

Invite new users

API realm_default_language documentation

API realm_notifications_stream_ID and realm_signup_notifications_stream_ID documentation

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

I don't feel qualified to review the API docs, but the rest looks good to me!
@timabbott do you want to take a look?

Discussion about possibly renaming Notification Bot is ongoing on CZO, but I think that can be a follow-up regardless.

This is great, merged, thanks @laurynmm!
I think the main follow-up that we might want to consider is changing the filename for the /help/ article and adding a redirect. I'm not sure I'd do that without first improving the system for declaring redirects to involve less work.

@timabbott - Sounds good. I've got that rename/redirect noted down as part of another issue I'm working on. And I have a draft PR up for improving the system for declaring redirects, #22444. It would be good to have eyes on that before I write the documentation about the updated system for contributors.


Update Vagrant file with port-forwarding

Missing, part please refer the challenge, in vagrant file
config.vm.network(
"forwarded_port", guest: 8000, host: 8000, host_ip: "<IP_ADDRESS>"
)

Resolved

Done