Updated README for Clarity on Params

The readme can be clearer to present the fact that there are parameters for specifying continuous variables as we only knew about it after diving into the actual code. The continuous_cols is not resented in Optional Arguments either. I hope this adds clarity for your users and readers. Thanks!

Credits to @gkeechin for pointing this out - might help other devs, hopefully -


cc2538 makefile, configure file not found

I just begin to follow the first steps to compile the examples for the cc2538 chipset, and I found these errors:
/bin/sh: 1: /mnt/c/Users/JaimeBohigas/openthread/examples/../configure: not found
I looked for configure file but on the repository is missed. How can I fix the problem??
Thank you very much.
Kind regards

It would help if you referenced the steps that you actually tried and their output.  In particular, did you follow the steps in this README.md
?  Also, make sure you have the GNU toolchain installed (autoconf, automake, libtool)

Dear all, specially Mr. jwhui,
I have followed the steps on the README.md, and also I use the bash on windows toolchain.
I verified I have installed the autoconf, automake and libtool too.
The libtool version I have got is 2.4.6-0.1
The automake version I have got is 1:1.15-4ubuntu1
the autoconf version I have gor is 2.69
This is the report I have on compile time
jaimegm@DESKTOP-95UCL84:/mnt/c/Users/JaimeBohigas/openthread$ make -f examples/Makefile-cc2538
CONFIG   cc2538...
(cd build/cc2538 && /mnt/c/Users/JaimeBohigas/openthread/examples/../configure INSTALL="/usr/bin/install -p" CPP="arm-none-eabi-cpp" CC="arm-none-eabi-gcc" CXX="arm-none-eabi-g++" OBJC="cc" OBJCXX="" AR="arm-none-eabi-ar" RANLIB="arm-none-eabi-ranlib" NM="arm-none-eabi-nm" STRIP="arm-none-eabi-strip" CPPFLAGS="-fdata-sections -ffunction-sections -Os -g -DOPENTHREAD_PROJECT_CORE_CONFIG_FILE='"openthread-core-cc2538-config.h"' -I/mnt/c/Users/JaimeBohigas/openthread/examples/../examples/platforms/cc2538/  -mcpu=cortex-m3 -mfloat-abi=soft -mthumb " CFLAGS="-fdata-sections -ffunction-sections -Os -g -DOPENTHREAD_PROJECT_CORE_CONFIG_FILE='"openthread-core-cc2538-config.h"' -I/mnt/c/Users/JaimeBohigas/openthread/examples/../examples/platforms/cc2538/  -mcpu=cortex-m3 -mfloat-abi=soft -mthumb " CXXFLAGS="-fdata-sections -ffunction-sections -Os -g -DOPENTHREAD_PROJECT_CORE_CONFIG_FILE='"openthread-core-cc2538-config.h"' -I/mnt/c/Users/JaimeBohigas/openthread/examples/../examples/platforms/cc2538/  -mcpu=cortex-m3 -mfloat-abi=soft -mthumb -fno-exceptions -fno-rtti " LDFLAGS="-fdata-sections -ffunction-sections -Os -g -DOPENTHREAD_PROJECT_CORE_CONFIG_FILE='"openthread-core-cc2538-config.h"' -I/mnt/c/Users/JaimeBohigas/openthread/examples/../examples/platforms/cc2538/  -mcpu=cortex-m3 -mfloat-abi=soft -mthumb -nostartfiles -specs=nano.specs -specs=nosys.specs -Wl,--gc-sections " --host=arm-none-eabi --prefix=/ --exec-prefix=/cc2538 --enable-cli-app=all --enable-ncp-app=all --with-ncp-bus=uart --enable-diag --with-examples=cc2538 --with-platform-info=CC2538 --enable-linker-map )
/bin/sh: 1: /mnt/c/Users/JaimeBohigas/openthread/examples/../configure: not found
I how It can show the problem.
Thank you very much.
Kind regards.

Did you run ./bootstrap then make -f examples/Makefile-cc2538?  What was the output, if any, from ./bootstrap?

Dear all,
Finally I have finished the make procedure.
I made a mistake because I tried to execute the file bootstrap before I had all the necesary programs.
Thank you very much.

@jaimebohigas whitch files did you mean ?
I have the same problem and i can't solve the issue.
Please how did you exactly manage to figure it out  ?

@jaimebohigas whitch files did you mean ?
I have the same problem and i can't solve the issue.
Please how did you exactly manage to figure it out ?

Did you follow the directions in examples/platforms/cc2538/README.md?


[UI]: Migrate RTK Query for Adapters Config

Notes for Reviewers
This PR fixes #
Migrating the dataFetch of adapters config to RTK query for the following api's:

GET: /api/system/adapters
GET: /api/system/availableAdapters
GET: api/system/adapters?adapter=adapterLoc
POST: /api/system/adapter/manage
DELETE: /api/system/adapter/manage?adapter=adapterLoc

Screen Recorder
https://github.com/user-attachments/assets/73c1e85d-42f2-4d02-9174-2a21a961932f
Signed commits

[x] Yes, I signed my commits.

@senthil-athiban let's discuss this PR during the Meshery dev meeting call, if you would. Please add it as an agenda item to the meeting minutes.


Validate frequency of days

This issue is caught in a Timed Trigger type for a Tekton pipeline.
When I set the cron expression 0 8 */80 * * for a job and it shows At 08:00 AM, every 80 days there. But actually, there is no such cron expression that will work for every 80 days. The expression above means At 08:00 on every 80th day-of-month, but eventually, this job runs on the 1st of every month. We could probably add validation for such scenarios when the frequency of the day (or any other component) exceeds the maximum length.
We use cron-validator to validate the cron expression and it doesn't seem to consider it as an incorrect expression. If the frequency of the day (like in this example 80) exceeds the length of days of that month (28 or 30 or 31), it should not pass it as a valid expression because the actual frequency happens on the 1st day of the next month, not on every 80 days.
The code being used:
const cron = require("cron-validator");
cron.isValidCron(cronString, { alias: true });

Expected: cron.isValidCron(...) should return false when the frequency exceeds the maximum length.
Routed here based on the recommendation from cronstrue.
PS: Tested with the latest version, it doesn't seem to address this issue.

According the cron syntax spec, there is no limit to the steps values (aside from the divide by 0 you filed earlier), which I assume was to make it easier to parse at the time. I could add the limits though I don't have the time for this feature right away.
You could try out the cron-validate library for the time being, as they already have a way to specify limits I think (though not sure if they apply to steps).


NeuroDesk Installation tutorial for the cloud

It might be nice to have a tutorial for folks on how to run NeuroDesk on the cloud

Dear @likeajumprope,
I described this as part of the docker setup:
https://brainhack.org/brainhack_cloud/docs/docker/
and it's described in detail here:
https://www.neurodesk.org/docs/neurodesktop/getting-started/cloud/

I am adding a separate section now so that it's more obvious

ok, done - can you check @likeajumprope?

Looks great! Thanks @stebo85


model training script and dataset

Great Work!
Do you have the code for model training? Where can I download the training data?

See the demo page for source of training code.
Dataset for training will not be shared as it is too large.


Kubernetes branch

online version demo not work

@WhiteBite - thanks for the PR, could you please add yourself to the contributors list: https://github.com/yugabyte/yugabyte-db/blob/master/CONTRIBUTORS.md
@nchandrappa - could you please review the change. cc @ameyb


Use ResizeObserver where available for Breadcrumbs and ButtonGroup

🙋 Feature Request
In any component that supports collapsing items based on size, we should use ResizeObserver instead of window resize events if possible. This will make it handle cases where part of the UI changes size without the window changing size (e.g. opening/closing a sidebar). We'll need to document that if you want to support older browsers, you'll need to include a polyfill. By default, we could fall back to window resize events, as the polyfill would only be needed for specific use cases.
💁 Possible Solution
Use ResizeObserver to handle resize events in Breadcrumbs and ButtonGroup, and whatever other components we add going forward that need this behavior. Perhaps we could create a reusable hook to handle this.
🧢 Your Company/Team
RSP

Would love to work on this

Awesome @Anenth! Let us know if you have any questions. 😀


Reducer import error

I'm using react-boilerplate which uses webpack 2. I installed redux-form(version 6.0.1) and did this:
import { reducer as form } from 'redux-form/immutable';
Note that I only tried to import the reducer, I did not even try to use it, just imported. And after building and running my application I get the error in browser console:
 SubmissionError.js?3964:17Uncaught TypeError: Super expression must either be null or a function, not object
I thought that maybe the problem is in immutable version of redux-form and tried to import the reducer of not immutable version and got the same error.

https://github.com/mxstbr/react-boilerplate/issues/920

https://github.com/mxstbr/react-boilerplate/issues/920

Can we reopen this issue?

This is an issue with redux-form and what it specifies in the jsnext:main field, not an issue with react-boilerplate.
Please add an issue on the redux-form repo and tell them to properly use the jsnext:main field like all other package authors. /cc @erikras

Just bumped on the same issue, would be great to see the fix 👍

I'm experiencing the exact same issue.

No ideas on how to solve this?

@luandro It's not a fix but https://github.com/mxstbr/react-boilerplate/issues/868

v6.0.2 should fix this issue.

@erikras, I just updated to v6.0.2 but I'm still getting the same error:
?d41d:7 Dynamic page loading failed TypeError: Super expression must either be null or a function, not object(…)
Any thoughts? Mind re-opening this?

@luandro after npm i --save redux-form
npm run build:dll
then npm run start again in react-boilerplate

Worked! Thanks @sourcesoft

The problem is back with 6.2 unfortunately :( @erikras

this problem is backed to version 6.2 @erikras

I'm having the same issue, and I could fix the problem after changing package.json file of es6-error package like the following.
"jsnext:main": "./lib/index.jsnext.js", to "jsnext:main": "./lib/index.js",

I'm also having this problem with 6.2.
The problem seems to be that ExtendableError is null or something else: https://github.com/erikras/redux-form/blob/d6bec04e5f7f9e5f263af8d393497ca6c7ebdb78/src/SubmissionError.js#L1-L3

still having issues with this on 6.2

Thanks a ton guys! I got my redux-form 6.2 working with react boilerplate thanks to this fix.

I'm not very knowledgable on webpack but here's some speculation:
It's caused by redux-form specifying a jsnext:main property but we are importing redux-form/immutable directly, hence bypassing the specified "jsnext:main": "./es/index.js" (this goes on to only include files in the /lib/* dir).
Exactly why this breaks I don't know but _es6Error gets doubly wrapped with .default which causes the exception since a constructor function is expected and not and object.

The reason it worked in previous versions was that es6-error": "^3.1.0(which was used by redux-form) was built with a different jsnext setup (I think it was untranspiled) making me think it was two wrongs that cancelled each other.
https://github.com/rollup/rollup/issues/126 seems to suggest that it might be needed to publish two separate packages two work around this.
Regarding react-boilerplate and https://github.com/mxstbr/react-boilerplate/issues/1244 RBP is setup to always favor jsnext entires. Changing the order of resolve.mainFields and put main before jsnext:main in the webpack config fixes the issue to (but I don't know what else it changes)

I think the cause is addressed by https://github.com/webpack/webpack/pull/3268 https://github.com/webpack/webpack/issues/3168
Basically Webpack fails to handle imports of es6 modules from es5 code which is what's happening above when the jsnext:main version of es6-error is required from inside the es5 version of redux-form.

I run into to this regression as well in 6.2. @erikras, this is definitely a regression.
The workaround is still going to internals/webpack/webpack.base.babel.js and swap 'main' and 'jsnext:main' as below.
mainFields: [
  'browser',
  'jsnext:main',
  'main',
]

to:
mainFields: [
  'browser',
  'main',
  'jsnext:main',
]

@carlos-peru yupp, that's what I'm doing atm, but I think that effectively disables webpack tree shaking.

If the suggested change the only way to workaround this? I don't like making deep changes to template internals. Mostly because I tend to forget about them.
In any case, thanks to @jacobk and @carlos-peru for the clean explanation about what is happening and how to workaround it.

@erikras Any ideas on this issue? It's still present in 6.4.3

PRs are welcome.

Any news on this ?

@erikras
I also got this issue.

Reached up till this issue due to the issue initialValues not getting rendered in Field even though the store is updated with initialValues/values.
I'm currently having this issue.
Thanks,
Sudheesh


Cache node names in ComparingFields, ComparingProperties

Check List:

Fixes #3157, #3158
Unit tests : YES
Javadoc with a code example (on API only) : NA
PR meets the contributing guidelines

Following the contributing guidelines will make it easier for us to review and accept your PR.

Integrated thanks @ykardziyaka !


Allow marking test cases as skipped, so that they are executed, but marked as skipped in the test result xml

Hi
I could not find a way to mark a unit test with a special ignore tag, so that the test is executed, but marked as skipped in the produced test result xml file.
The issue Skipping tests #355 describes how to hide a curtain test case. In that case the test is not executed when running from command line and therefore the test result is not in the report.
The issue Allow skipping tests at run-time. #395 is neither what I'm looking for.
Can anybody tell whether this is really not possible or if there's a workaround for that?
If not, it would be really great to have this feature implemented.
Cheers,
Stephan

How about the tag [!mayfail], would that work for your use case?

Hi @lightmare
Thanks for the response. This special tag is quite interesting, but does not fully what I need.
If I use the [!mayfail] tag, the output with the console reporter resp. the number of failed as expected tests looks how I could use it.
===============================================================================
test cases:   219 |   218 passed | 1 failed as expected
assertions: 62878 | 62877 passed | 1 failed as expected

When using the junit reporter (what I need, because I'd like to show the results on a TeamCity) you cannot distinguish anymore between failed and failed as expected test case.
Junit would support the test case status skipped, this is the preferred status in my use case. But I think this would not fit to the failed as expected result.

Interesting.
You are correct that its doesn't map cleanly on to a JUnit status (well, I assume you are - I never did find a spec). Not sure what the best mapping should be - but I'm inclined to map it to a pass.
But you mention you want to use this with TeamCity. There is a TeamCity reporter that is much better for that (you do have to #include the TeamCity reporter header explicitly in the same file that has CATCH_CONFIG_MAIN or CATCH_CONFIG_RUNNER).

@stidch did you try the TeamCity reporter in the end?
I was just looking at it and realised it had a similar problem, so I've fixed it.
I'll leave this ticket open as the issue still exists in the JUnit reporter (will try and get to that next week)


Only animate selectable background if we want to

Update the getSelectableBackground from FastAdapterUtils to use the parameter animate.

@Fuhrmann looks good. thanks


Remove user from github scope

For issue ga4gh/dockstore#1126 Remove user from GitHub scope.  On a fresh database, GitHub account info gets automatically synced upon first visit.  New workflows/tools are automatically added via refresh all.

Codecov Report

Merging #189 into develop will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           develop     #189   +/-   ##
========================================
  Coverage    63.08%   63.08%
========================================
  Files           95       95
  Lines         2790     2790
  Branches       364      364
========================================
  Hits          1760     1760
  Misses         836      836
  Partials       194      194

Impacted Files
Coverage Δ

src/app/shared/dockstore.model.ts
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Add instructions on how to install with conda

I  have added instructions on how to install with conda.
I think the install instructions would be simpler if you didn't suggest doing the install through a notebook. I haven't changed that though...

Please correct the package name and remove the ">" characters from the commands.
I wrote the instructions for the notebook to allow beginners to install and test the package in the same environment, and introduce the "!" command that I think is very handy for Windows users. I assume more advanced users can get it installed on their own. I'm open to suggestions for improving the instructions though.

Done.
Finding the best way to target beginner is extremely tricky. That is also my experience. So maybe you are onto something with the notebook approach. Although I guess most users trying PyMeasure would know what a command prompt/terminal is.
I have had some luck creating a tutorial as a Jupyter Notebook for one of my first projects. https://github.com/AnyBody-Research-Group/AnyPyTools/blob/master/Tutorial/00_AnyPyTools_tutorial.ipynb
The cool thing about that approach is that I could bundle the tutorial with the conda package. So users would get a live version of the tutorial after installing my package with conda.

I've had a chance to run the instructions in the notebook, and found it requires the --yes argument.
!conda config --add channels conda-forge
!conda install pymeasure --yes

The print out is pretty badly formatted, so I don't think that conda in the notebook is worth pointing people towards as @melund mentioned. Since its preferred that people move to the in-place installation (pip install -e .) version for contributing features, I want to keep this consistent. For now, lets keep the pip instruction, and add a sub-section of "Installing PyMeasure" that briefly describes the alternative installation using conda-forge.
@ddale, do you have any thoughts on the best way to instruct beginners to install the package?

You are right. IPython and the notebook, in particular, is not well suited for working with CLI's like pip or conda. And in general, it is not the best practice to introduce to new users. Let them install PyMeasure from the command line.
I am not a big fan of pip.  And suggesting people to use pip with the anaconda python distribution is a terrible idea. Conda will not know of the packages that were pip-installed and you can easily end in a situation where the dependencies of PyMeasure are suddenly broken when users install something else.
I guess pip install -e is the new python setup.py develop. That is the only use of pip that makes sense when you are using the Anaconda python distribution.
I would really suggest that you just make the instructions for different scenarios. The first one could be targeted, beginners. I.e. Anaconda python distribution and the conda commands.  The second one could be installing with pip for people who don't want to use Anaconda.

I agree with all of this.

On Aug 5, 2016, at 3:18 PM, Morten Enemark Lund<EMAIL_ADDRESS>wrote:
You are right. IPython and the notebook, in particular, is not well suited for working with CLI's like pip or conda. And in general, it is not the best practice to introduce to new users. Let them install PyMeasure from the command line.
I am not a big fan of pip. And suggesting people to use pip with the anaconda python distribution is a terrible idea. Conda will not know of the packages that were pip-installed and you can easily end in a situation where the dependencies of PyMeasure are suddenly broken when users install something else.
I guess pip install -e is the new python setup.py develop. That is the only use of pip that makes sense when you are using the Anaconda python distribution.
I would really suggest that you just make the instructions for different scenarios. The first one could be targeted, beginners. I.e. Anaconda python distribution and the conda commands. The second one could be installing with pip for people who don't want to use Anaconda.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@ddale, do you have any thoughts on the best way to instruct beginners to install the package?

I think the easiest way for beginners is to use Anaconda Navigator: https://docs.continuum.io/anaconda/navigator. I'll look into supporting it on Monday.

@ddale. That is a good option for non-tech users. It is also very easy to do. All that is needed is to add an app section to the recipe. See how we do it for xonsh. https://github.com/conda-forge/xonsh-feedstock/blob/master/recipe/meta.yaml#L41
The only downside is that the navigator is still a little unfinished. Icon and the detailed descriptions don't work but should be fixed soon. See this conversation: https://github.com/ContinuumIO/anaconda-issues/issues/937#issuecomment-234980373

Thats a good idea @ddale, but I agree with @melund that Navigator is not that polished. It may be a good solution in the future when it comes out of beta.
@melund, can you update the docs to use the "Anaconda prompt" and conda instructions? Please make sure to correct the package to pymeasure.

@cjermain I have updated the instruction. Hope this is better

@melund, thanks!


License

Hi,
is the code Free Software?  I cannot seem to find a license declaration.  I'd like to use part of the code in another project and it would be good to know under what license these files can be used.
Thanks!

Thanks for the reminder, I keep forgetting this.


Live map - key format changed

As of today evening the live map parsing failed completely for me. The key format was changed from 'x_y' to (apparently) '(x,y)'
Can someone verify this?

I can confirm this changed behaviour - very strange. At 19:00 it worked without any problem. Now at ~22:30 the format is changed without any notification in the GS.com forums...

As its is only internal it does not matter. As the do not interpret the keys, the generation format is irrelevant to the gc map. It seems we will need to interpret the grid as well, to avoid such pitfalls

Is this or #1216 the reason why I experienced no caches shown on live-map for basic member with NB 0203 today ?
Also for PM it is quite clear that only the nearby search on live-map seems to work.

@Lineflyer this one. #1216 is the reason that c:geo tells you that the cache is not published if you have a version that includes this on only


Add Terraform hello world example

What does the program do ?
Prints Hello world once you run terraform apply

In what programming language it is written?
Terraform

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


MAID-1866: Make tests deterministic

This modifies SeededRng to give the same deterministic results on different architectures also when using choose(). new_rng() is modified to return a SeededRng so that calls to choose() on the returned generator call the fixed function.

Thanks for the pull request, and welcome! The MaidSafe team is excited to review your changes, and you should hear from @ustulation (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTOR.md for more information.


refac: Update anonymisation notebook with asserts

Description
Updated anonymization notebook with asserts and quality checks
Pull-request quality

[x] The title adheres to this guide
[x] Tests are written and executed locally
[x] Subsystem tests have been tested (by manually deploying to dev_002)
[ ] Documentation has been updated
[ ] The integration event catalog has been updated
[ ] C4 diagrams have been updated and Team Outlaws has been informed about relevant changes

Ran notebook manually on dev-002 here: https://adb-4044020512035994.14.azuredatabricks.net/?o=4044020512035994#notebook/3397088863440150


[bzip2] Fix installation when VCPKG_BUILD_TYPE is set

Commit 458c20e94b9f558b523f187ce403bd062e131cdb added vcpkg_fixup_pkgconfig()
that breaks installation when VCPKG_BUILD_TYPE is set:
-- CHECK_LIB_bz2d_DEBUG:CHECK_LIB_bz2d_DEBUG-NOTFOUND
CMake Error at scripts/cmake/vcpkg_fixup_pkgconfig.cmake:239 (message):
  Library "bz2d" was not found! If it is a system library use the
  SYSTEM_LIBRARIES parameter for the vcpkg_fixup_pkgconfig call! Otherwise,
  correct the *.pc file
Call Stack (most recent call first):
  scripts/cmake/vcpkg_fixup_pkgconfig.cmake:348 (vcpkg_fixup_pkgconfig_check_files)
  ports/bzip2/portfile.cmake:51 (vcpkg_fixup_pkgconfig)
  scripts/ports.cmake:79 (include)

Signed-off-by: Vitalii Koshura<EMAIL_ADDRESS>

Closing as a duplicate of #12759


[Critical] XSS Vulnerability

Jobs description vulnerable because this input accept markdown format. You should unsupport markdown, should only accept a few HTML tag like bbcode.
Proof of concept: http://pyistanbul.org/jobs/100
Attacker can hijack admin sessions via using XSS => https://omercitak.com/xss-zafiyeti-ile-oturum-calma-session-hijacking/

@om3rcitak Jobs app is completely removed. Did you see same vulnerability in any other place?

@ulgens No, I didn't see any other page. You can close this issue.

@om3rcitak Thanks a lot.


Thaumcraft & PneumaticCraft integration

Adds Thaumcraft essentia and Pneumaticraft pressure support to Compact Machines.
Thaumcraft:
Thaumcraft essentia is stored in a buffer similar to fluids, gases, and energy. To display it, I split the Energy bar in the same way the fluid bar is shared with gasses. Essentia suction is "48", meaning it will draw essentia from unlabeled jars, but not labeled jars.
I used the "hopping" functionality to do the "per tick" stuff that needed to be done for essentia handling. Since the tick was needed on both the inside and outside simultaneously, I added a new hopping mode that does this, mode 4. This mode is not selectable from the GUI.
PneumaticCraft:
Pressure is just passed through using pre-existing methods in the PneumaticCraft API.
Known Issues:
When a pressure tube is connected to the outside of a Compact Machine, and not connected to anything else, it does not leak air like it is supposed to. This is due to an issue in PneumaticCraft, that has been reported to the author: https://github.com/MineMaarten/PneumaticCraft/issues/705
Fixes #43

Nice! I'll pull this in asap, currently rather busy. Need to put a few hours in myself.


Error occurred during creating affordance map

Hello, I am recently running your code in ubuntu16.04, ros kinetic, and v-rep 3.6.1. When I start the simulation of V-rep scene, and run the command "python main.py --is_train=True" to perform training, the following error occurs. It seems that the affordance map cannot be created in the former step of "os.system(affordance_cmd)". Would you please help me with this issue? Thanks a lot! The log is as follows:
(active-perception) ys@huang:~/Active-Perception/DQN$ python main.py --is_train=True
/home/ys/anaconda3/envs/active-perception/lib/python3.6/importlib/_bootstrap.py:219: RuntimeWarning: compiletime version 3.5 of module 'tensorflow.python.framework.fast_tensor_util' does not match runtime version 3.6
return f(args, **kwds)
/home/ys/anaconda3/envs/active-perception/lib/python3.6/site-packages/h5py/init.py:34: FutureWarning: Conversion of the second argument of issubdtype from float to np.floating is deprecated. In future, it will be treated as np.float64 == np.dtype(float).type.
from ._conv import register_converters as _register_converters
--- Successfully load vrep ---
[] Use GPU Fraction is : 0.6000
2019-09-25 20:43:31.029367: I tensorflow/core/platform/cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2 FMA
[*] Current Configuration
{'Lua_PATH': '../affordance_model/infer.lua',
'batch_size': 16,
'ckpt_dir': './dqn/checkpoint',
'cnn_format': 'NHWC',
'discount': 0.6,
'end_metric': 0.85,
'env_name': 'Act_Perc',
'ep_end': 0.2,
'ep_end_t': 10000.0,
'ep_start': 1.0,
'history_length': 4,
'inChannel': 4,
'is_sim': True,
'is_train': True,
'learn_start': 200.0,
'learning_rate': 0.001,
'learning_rate_decay': 0.96,
'learning_rate_decay_step': 200,
'learning_rate_minimum': 0.00025,
'max_reward': 1.0,
'max_step': 20000,
'memory_size': 2500,
'min_reward': -1.0,
'model_dir': './dqn/model',
'scale': 50,
'scene_num': 24,
'screen_height': 128,
'screen_width': 128,
'target_q_update_step': 50,
'test_scene_num': 6,
'test_step': 200,
'train_frequency': 4}
Simulation started
Connection success!
[] Initialize the simulation environment
[] Build Deep Q-Network
[] Build Q-Evaluate Scope
[] Build Q-Target Scope
[] Build Weights Transform Scope
[] Build Optimize Scope
[] Build Summary Scope
[] Initial All Variables
[] Loading checkpoints...
[] Load SUCCESS: ./dqn/checkpoint/dqn_model_ckpt-13400
[] Assign Weights from Prediction to Target
[] Random init the scene 13 with 0 object removed
/home/ys/torch/install/bin/luajit: cannot open <../affordance_model/model.t7> in mode r  at /home/ys/torch/pkg/torch/lib/TH/THDiskFile.c:673
stack traceback:
[C]: at 0x7fb4af384440
[C]: in function 'DiskFile'
/home/ys/torch/install/share/lua/5.1/torch/File.lua:405: in function 'load'
../affordance_model/infer.lua:36: in main chunk
[C]: in function 'dofile'
...e/ys/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:150: in main chunk
[C]: at 0x00405d50
Traceback (most recent call last):
File "main.py", line 88, in
tf.app.run()
File "/home/ys/anaconda3/envs/active-perception/lib/python3.6/site-packages/tensorflow/python/platform/app.py", line 48, in run
_sys.exit(main(_sys.argv[:1] + flags_passthrough))
File "main.py", line 72, in main
agent.train()
File "/home/ys/Active-Perception/DQN/dqn/agent_18.py", line 50, in train
screen, reward, action, terminal = self.env.new_scene()
File "/home/ys/Active-Perception/DQN/simulation/environment.py", line 483, in new_scene
self.screen, self.local_afford_past, self.location_2d = self.camera.local_patch(self.index, (self.screen_height, self.screen_width))
File "/home/ys/Active-Perception/DQN/simulation/environment.py", line 211, in local_patch
raise Exception(' [!] !!!!!!!!!!!  Error occurred during creating affordance map')
Exception:  [!] !!!!!!!!!!!  Error occurred during creating affordance map

Thank you for your interesting in our project.
I'm not sure if you have fixed this bug. But I will still provide you with the solution.
The solution is: our project based on a model called "Affordance model" from http://arc.cs.princeton.edu/, for details, you can refer to this project. The main purpose for this "Affordance model" is: given a RGBD image for current scene, it can output a heatmap over location, in which the higher value stands for higher confidence when operating on this location.
Our project is to introduce reinforcement learning into robotic grasping, i.e. let agent learn a policy to optimize the scene with certain action until we obtain a good heatmap from "Affordance model".
Here is the detail for the solution:

plz download the "Affordance model" from https://github.com/andyzeng/arc-robot-vision, notice we use suction-based grasping verstion of the "Affordance model" ( http://vision.princeton.edu/projects/2017/arc/downloads/suction-based-grasping-snapshot-10001.t7 )
Place the model ( the .t7 file ) in correct path, so the cmd in python file can execute.

If you have any questions, plz feel free to contact me. I will try my best to respond quickly!

Thanks for your warm help! I will follow your instructions to re-run the demo.

You are welcome. Sorry, it's not a mature repo, which can be used directly. I'll help you to debug all the stuff out.

感谢您的热烈帮助！我将按照您的指示重新运行演示。

Hi, I meet the same question as yours. And I haven't solved it.
I copy suction-based-grasping-snapshot-10001.t7 into affordance_model and ./simulation/afford_h5. .
However, it still doesn't work.
Maybe I should install Torch.
I want to ask you whether you have solved it following the links, and which folder  I should copy  .t7 into.

感谢您的热烈帮助！我将按照您的指示重新运行演示。

Hi, I meet the same question as yours. And I haven't solved it.
I copy suction-based-grasping-snapshot-10001.t7 into affordance_model and ./simulation/afford_h5. .
However, it still doesn't work.
Maybe I should install Torch.
I want to ask you whether you have solved it following the links, and which folder I should copy .t7 into.

Hi, I have loaded the model of suction-based-grasping-snapshot-10001.t7 in this way:

put "suction-based-grasping-snapshot-10001.t7" in the folder of "affordance_model";
modify the file "affordance_model/infer.lua" at line 21 like this: modelPath = '../affordance_model/suction-based-grasping-snapshot-10001.t7';

these steps can load the model successfully.

感谢您的热烈帮助！我将按照您的指示重新运行演示。

Hi, I meet the same question as yours. And I haven't solved it.
I copy suction-based-grasping-snapshot-10001.t7 into affordance_model and ./simulation/afford_h5. .
However, it still doesn't work.
Maybe I should install Torch.
I want to ask you whether you have solved it following the links, and which folder I should copy .t7 into.

Hi, I have loaded the model of suction-based-grasping-snapshot-10001.t7 in this way:

put "suction-based-grasping-snapshot-10001.t7" in the folder of "affordance_model";
modify the file "affordance_model/infer.lua" at line 21 like this: modelPath = '../affordance_model/suction-based-grasping-snapshot-10001.t7';

these steps can load the model successfully.

Thank you very much, I know what caused it. I haven't find the correct path and set the arguments

感谢您的热烈帮助！我将按照您的指示重新运行演示。

Hi, I meet the same question as yours. And I haven't solved it.
I copy suction-based-grasping-snapshot-10001.t7 into affordance_model and ./simulation/afford_h5. .
However, it still doesn't work.
Maybe I should install Torch.
I want to ask you whether you have solved it following the links, and which folder I should copy .t7 into.

And you also need to install Torch7, to run the .t7 model

感谢您的热烈帮助！我将按照您的指示重新运行演示。

Hi, I meet the same question as yours. And I haven't solved it.
I copy suction-based-grasping-snapshot-10001.t7 into affordance_model and ./simulation/afford_h5. .
However, it still doesn't work.
Maybe I should install Torch.
I want to ask you whether you have solved it following the links, and which folder I should copy .t7 into.

And you also need to install Torch7, to run the .t7 model

Yeah, now I'm struggling to install Torch7 and then test it.

感谢您的热烈帮助！我将按照您的指示重新运行演示。

Hi, I meet the same question as yours. And I haven't solved it.
I copy suction-based-grasping-snapshot-10001.t7 into affordance_model and ./simulation/afford_h5. .
However, it still doesn't work.
Maybe I should install Torch.
I want to ask you whether you have solved it following the links, and which folder I should copy .t7 into.

And you also need to install Torch7, to run the .t7 model

Yeah, now I'm struggling to install Torch7 and then test it.

In the beginning, I paid a lot to convert the torch model to pytorch model, which will speed up the training process and save the trouble caused by torch. But I failed, for a certain structure in this model can't be transformed...

Ｈi, I have installed Torch7 and Lua5.1, and the *.t7 is in the correct folder.
Then I type

python main.py --is_train=True or python main.py --is_train=False --is_sim=True

There are still some errors.

It seems that my GPU is not enough to test codes.
However, it still exits when I set the batch_size=4 in ./DQN/config.py.
Have you met this one?
Here is the config about my GPU:

How much GPU memory does your program need?

PLZ try to change the gpu fraction param here. This param is to assign how much gpu memory to DQN part, maybe you can set it with a smaller number to leave more space for Affordance Map, which will take about 2.5G memory, as far as I remembered.

PLZ try to change the gpu fraction param here. This param is to assign how much gpu memory to DQN part, maybe you can set it with a smaller number to leave more space for Affordance Map, which will take about 2.5G memory, as far as I remembered.

I set

flags.DEFINE_string('gpu_fraction', '50/100', 'idx / # of gpu fraction e.g. 1/3, 2/3, 3/3')

Now it works well.
I don't know how to thank you. Thank you for your help in my testing.

You are welcome. It's not a mature repo, without detailed documents, which may cause you much more time to use it. Hope you can get motivated by this project and explore something more.


Removed non-ascii characters in comment block.

I can see  a repeated sequence of \302\240\ in comment block.

Joe, could you please check that? Is it only in that file or is it also present in other files?

There are others (all examples files). I'll replace them globally later.


Add Support for Spring Native

Are there any plans to support spring native?

All native efforts are for now taken care of in https://github.com/spring-projects-experimental/spring-native/ directly, so we're not able to fix this in this repository right now. Also, GraphQL Java itself might need to ship reflection hints as a first step.
Feel free to create an issue in the spring-native project. Spring GraphQL is not yet released so I'm not sure this is a high priority for now.
Thanks!


NEXT-7687 Fixes #832

1. Why is this change necessary?
The function did not work at this moment.
2. What does this change do, exactly?
Use the existing data that is passed through the order object instead of reloading is.
3. Describe each step to reproduce the issue or behaviour.
Create credit nota. The list of invoices will not load.
4. Please link to the relevant issues (if any).
https://github.com/shopware/platform/issues/832
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

hey @runelaenen
please add a short message to your commit, what was changes exactly
the fixes #123 should only be seen as command for github, and not as information

Oh must have forgotten it. I normally always do that :)
I've updated it.

QA approved :heavy_check_mark:

Thank you for your contribution :blue_heart: :tada:


Tag for v0.0.3?

Can you point to a commit corresponding to latest 0.0.3 (I'm stuck on Python 3.8)

Hi @dwillhelm, unfortunately there is not really a usable version before 0.0.4. Regarding python 3.8 and python 3.9. Is there no possibility to upgrade to 3.9?
cc @bertiqwerty

Unfortunately I am stuck with 3.8 for the moment.
But I am grateful to see #213! 😄

Hi @dwillhelm, You should be able to use Python 3.8 now if you install from the main branch.

Hi @dwillhelm, You should be able to use Python 3.8 now if you install from the main branch.

Great news! Thanks @bertiqwerty @jduerholt


Draft: TS integration

I have no idea what I'm doing regarding IO script itself, please give any feedback regarding bot script itself or TS integration 🤔.
P.S. For some reason IO generally behaves very badly, diving heroes, etc, I assume it's not caused by my changes?

Hey ThirVondukr, first of all thanks a lot for brining in the typescript-to-lua dependency and set this up. I think this is a great start.
Also, thanks for raising this to check with me first before adding a lot of things to it.
It can be a lot better if we can put the source code in different paths like below, and set the generated path of the lua files to the relative bots folder, such that we can keep the code base clean for better maintenance and less confusion. for example:
root
│   README.md
│   .gitignore
│   [other config files]
└───bots (lua scripts)
│   │
│   └───BotLib
│       │   hero_wisp.lua
│   ...
└───typescript (ts scripts)
│   │
│   └───interfaces (all interfaces)
│       │   bots_general.d.ts
│       │   ...
│       └───valve_scripting (interfaces from Valve https://developer.valvesoftware.com/wiki/Dota_Bot_Scripting)
│           │   globals.d.ts
│           │   unit.d.ts
│           │   ability.d.ts
│           │   ...
│
│   └───bot_lib (all bot logic in ts)
│       │   hero_wisp.ts
│       │   ...
│
│   └───lib
│       │   [other ts utilities]
│

For some reason IO generally behaves very badly

This is because IO is buggy due to missing default bot logic from Valve. This is not because of your change, it's a bug on Valve side - those buggy heroes like IO will only simply go to a dummy target point and don't do anything that makes sense unless we override the relative behavior.
I fixed those buggy heroes by implementing a simple override layer for laning and attack logic. This is also to be improved.
I wonder how you wrote the ts code for IO. Did you write it on your own, or used some tools for help? I like the idea of
const roleItemBuyList: { [key in BotRole]: string[] } = {
    pos_1: defaultBuild,
...

I think since we are doing this refactoring, a more complex mapping may make more sense:
{
pos_1 -> item-build, ability-build, talent-build, etc,
pos_2 -> item-build, ability-build, talent-build, etc,
}

A general feedback on new util-like methods such as:
function HasHealingEffect(hero: Unit) {
    const modifiers = [

please first check if there's such method in bots/FunLib/jmz_func.lua, or bots/FunLib/utils.lua, if not then add it to a generic utils file since it can be shared across different files.
Please add a brief intro to the ts files you add, so we can share the context what to expect from those files to avoid mis-use or mis-add/mix of functionality.
And you dont need to keep the bots/BotLib/hero_wisp.reference.lua file for reference. we can always ref the historical commits.

I just rewrote the IO code to be more type safe 🙂
Regarding using different paths - maybe it would be better to do that after all the code is migrated to TS?
For builds it would be better to use an interface/object:
interface BotBuild {
  items: {
    buy: string[]
    sell: string[]
  }
  ...
}

{
pos_1 -> {...},
pos_2 -> {...},
}

Also it would be great to talk with you outside of github to collaborate easier.

For builds it would be better to use an interface/object

good point, yeah we can do that too. I have not been using ts for a long time, hopefully this big nested entity won't be too hard to use.

Regarding using different paths - maybe it would be better to do that after all the code is migrated to TS?

We should do it in the beginning, because we know it's anyway should be achieved for the long run and I dont want any conflicts or accidental changes to the ts files when we and potentially other people later working together on this project. Maybe something like following:
// in tsconfig.json:
{
  "compilerOptions": {
    "target": "ESNext",
    "module": "CommonJS",
    "strict": true,
    "outDir": "tempBuild",  // Temporary directory for all compiled Lua files
    "rootDir": ".",       // Base directory for TypeScript source files
  },

// Add a build.js file:
const fs = require('fs');
const path = require('path');

const folders = [
  { src: 'tempBuild/bot_lib', dest: 'bots/BotLib' },
  { src: 'tempBuild/fun_lib', dest: 'bots/FunLib' },
];

folders.forEach(({ src, dest }) => {
  fs.mkdirSync(dest, { recursive: true });
  fs.readdirSync(src).forEach(file => {
    const srcPath = path.join(src, file);
    const destPath = path.join(dest, file);
    fs.renameSync(srcPath, destPath);
  });
});

// Optionally, clean up the tempBuild directory
fs.rmSync('tempBuild', { recursive: true, force: true });

// package.json:
{
  "scripts": {
    "build": "tstl && node build.js"
  }
}

can add me in discord: forest0xia

In case you or anyone else wants to know more about the bugged heroes or bugs from Valve side for bots support, check this post, I had shared part of the bugs in reddit: https://www.reddit.com/r/DotA2/comments/1ezxpav/a_note_to_valve_official_regarding_bot_scripts/

Bro, following up on this. We likely will have more conflicts as we both keep modifying same files for our updates. What is your plan to fix conflicts and merge a initial verison?

Hey, sorry, but I was a bit busy recently, we don't have to migrate whole project in one go, we can solve current conflicts and merge the changes, we could use generated lua in existing lua files that we didn't yet migrate.

#No worries man. Take care and take your time. I'm also busy recently lol and just had an orthopedic surgery.
To clarify my point for the previous comment was: to minimize the conflicts that can cause unexpected behaviors. Please keep your changes as minimal as possible for the initial integration with TS.
Obviously we are aligned on this already, but I'd like to make it clear so you are not adding more or converting more files while trying to wrap up your work for the commit.

I think I can solve the conflicts with utils, version and hero_wisp files and let you know

@forest0xia I've solved a couple of conflicts in utils and wisp script, can we merge the changes or is there something you'd like to change?

Decided to just provide optional defaults for enum values in case these globals are nil
https://github.com/ThirVondukr/dota2bot-OpenHyperAI/blob/a6cd1577799b6ae229bd35dff62d59cc5c9c1452/bots/lib/dota/enums.ts#L71

Impressive work @ThirVondukr ! Thanks for making this change <3

Apparently there were some errors in utils.ts and other place in the merged commit. I just fixed them. I also refactored the file structure, and moved jmz_func to typescript folder for now. Let's start from the current state to use ts.
Btw, BotBehavior needs further refactoring. We need to break it down. It's not adding value but simply making unnecesary structure for now.  It will be harder to manage and harder to review later for bots that has very large BotBehavior. Take a look at Invoker lua.
Also, as we've discussed, we want to optimize code structure for different positions. for example pos1 should have a specific item-build, ability-build, talent-build etc. so pos1 wisp should get a pos1 build for all things (e.g. item-build[1], ability-build[1], talent-build[1]), and pos2 wisp should get a pos2 build for all things. Let's add this later as well.

@ThirVondukr ^^


dont exit in sanity checker on stimulus_reflex:install

Type of PR
Bug Fix
Description
I just noticed, that when you create an new Rails app with the current master (45e12ed), that the sanity checker kicks in and aborts the rails stimulus_reflex:install script because it can't find the stimulus_reflex npm package in the package.json. This package isn't included yet, because this installer is responsible for adding it in the first place.
/cc @RolandStuder
Why should this be added
New users shouldn't be stopped by the sanity check on their first hands-on with StimulusReflex.
Checklist

[x] My code follows the style guidelines of this project
[x] Checks (StandardRB & Prettier-Standard) are passing
[x] This is not a documentation update

Good catch!
I could have sworn that #262 covered this case but I realize that it covered generators not rake tasks.
My only comment, which isn't a blocker, is that I wonder if there's a way we could standardize the approach taken by called_by_installer? and called_by_generate_config?. The answer could very well be "nope!"
I don't build a lot of tooling like this, so I'm learning in real-time.


Modernized libflann ubuntu rule.

There were no rules for the modern distros, whereas there were plenty of (now) uselsess rules for discontinued ones.
https://packages.ubuntu.com/source/bionic/flann

Okay, I updated the rule to just add the asterisk for ubuntu, and I also updated debian packages according to https://packages.debian.org/search?suite=default&arch=amd64&searchon=names&keywords=libflann .
Is there actually no automatic mechanism on the ROS build farm that would validate at least if rosdep apt keys exist on a new ubuntu distro supported by rosdistro?


(QENG-3920) Global host config CLI configuration support

This commit adds CLI support for specifying global configuration
settings that will be included in the CONFIG section.
For example:
  $ beaker-hostgenerator --global-config {master=headless} redhat7-64m

  ---
  HOSTS:
    redhat7-64-1:
      pe_dir:
      pe_ver:
      pe_upgrade_dir:
      pe_upgrade_ver:
      hypervisor: vmpooler
      platform: el-7-x86_64
      template: redhat-7-x86_64
      roles:
      - agent
      - master
  CONFIG:
    nfs_server: none
    consoleport: 443
    master: headless
    pooling_api: http://vmpooler.delivery.puppetlabs.net/

@waynr for review

I'd like to cut a release with these changes as soon as this is merged.
This should allow for QA/modules to get rid of most/all of their static config files.

@waynr Shoot, just realized we don't quite handle non-string values properly.
If any of the arbitrary key-value settings had a non-string value (such as a boolean or number), the resulting YAML will be strings.
For example:
centos6-64{value-should-be-a-number=123\,value-should-be-a-boolean=true}
---
HOSTS:
  centos6-64-1:
    <...>
    value-should-be-a-number: '123'
    value-should-be-a-boolean: 'true'
    <...>
CONFIG:
  <...>

Those values should be:
    value-should-be-a-number: 123
    value-should-be-a-boolean: true

Ok I fixed the datatype problem and updated the test fixture to exercise it.

@nwolfe can you add a test that demonstrates individual host settings override global settings?

can you add a test that demonstrates individual host settings override global settings?

That's not the behavior.  The global settings are settings for the CONFIG section.  They're not "global" as in they apply to all hosts implicitly.

@nwolfe I see what you mean now, my bad.

👍

:+1:


Version change to 0.3.1

This change is

r? @Fraser999
(maidsafe_highfive has picked a reviewer for you, use r? to override)

r? @dirvine

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Server shows as running even though it's not

When I run the status, this is the output:

arkserver@server:/home/nick$ arkmanager status
Running command 'status' for instance 'main'
Server running:   No
Server listening:   Yes
Server Name: Dino Fight Club - (v265.51)
Players: 0 / 10
Active Players: 0
Server online:   Yes
ARKServers link:   http://arkservers.net/server/<IP_ADDRESS>:27015
Server build ID:   2066619

It also shows in the server list in Steam. That makes no sense to me. This is after rebooting the entire server computer, too, and not having input any commands. I don't remember arkmanager being set to start on boot and it's causing a lot of problems with me logging in. I am also not able to issue the "stop" command since it thinks there is no server running. On top of that, if I issue the "start" command, THEN issue a "stop" command, I end up with the same output as above again. Something is going horribly wrong and I don't know what it is.
If it means anything, I recently moved my server files from the "nick" user to the "arkserver" user on my server because it kept going down. I also updated the permissions and all that. It has been working fine all week since doing that until now and I can't figure out what's going on.
On further looking, when I run the "top" command, I see this;

1499 nick      20   0 8760056 5.228g  60364 S  71.7 34.5   2:37.93 ShooterGameServ

So it's definitely starting for some reason. How do I stop it? Why is it doing this?

What is the output of systemctl status arkmanager@main?
To stop this service from starting at startup, try:
systemctl disable arkmanager

The output (this is after killing the ShooterGameServ process via top) is:

nick@server:~$ systemctl status arkmanager@main
â<EMAIL_ADDRESS>- Daemon to start an ark server instance
Loaded: loaded<EMAIL_ADDRESS>static; vendor preset: enabled)
Active: inactive (dead)
Aug 27 22:22:10 server arkmanager[1442]: Setting breakpad minidump AppID = 346110
Aug 27 22:22:14 server arkmanager[1442]: 2017-08-27 22:22:14: Bad PID ''; expected '1499'
Aug 27 22:26:53 server arkmanager[1442]: Using binned.
Aug 27 22:26:53 server arkmanager[1442]: 4.5.1-0+UE4 7038 3077 402 7
Aug 27 22:26:53 server arkmanager[1442]: /home/buildbot/buildslave/steam_rel_client_linux64/build/src/clientdll/applicationmanager.cpp (3004) : Assertion Failed: CApplicationManager::GetMountVolume: invalid index
Aug 27 22:26:53 server arkmanager[1442]: /home/buildbot/buildslave/steam_rel_client_linux64/build/src/clientdll/applicationmanager.cpp (3004) : Assertion Failed: CApplicationManager::GetMountVolume: invalid index
Aug 27 22:26:53 server arkmanager[1442]: /home/buildbot/buildslave/steam_rel_client_linux64/build/src/clientdll/applicationmanager.cpp (3155) : Assertion Failed: m_vecInstallBaseFolders.Count() > 0
Aug 27 22:26:53 server arkmanager[1442]: CtrlCHandler: Signal=15
Aug 27 22:26:53 server arkmanager[1442]: /usr/local/bin/arkmanager: line 1110:  1499 Aborted                 (core dumped) "$arkserverroot/$arkserverexec" "$arkserveropts" "${arkextraopts[@]}"
Aug 27 22:26:53 server arkmanager[1442]: 2017-08-27 22:26:53: exited with status 0

Also annoying is that a lot of the stuff we had put up is now gone, but I suspect that could be related to an Extinction Core update. The only other thing I have done recently is delete the old ARK folder from the nick user.
I am fine with the server starting a startup if it is going to work correctly, but as of now that does not seem to be the case.

1499 nick 20 0 8760056 5.228g 60364 S 71.7 34.5 2:37.93 ShooterGameServ

That looks like it's still starting as nick rather than as arkserver.
Please check that steamcmd_user=arkserver in arkmanager.cfg
Also if there is a User setting in<EMAIL_ADDRESS>check that it is set to arkmanager rather than nick

arkmanager.cfg was set correctly, but the other setting was still on nick. Hopefully that should do it. Thanks (again)!
Totally unrelated, but I've not had to restore from a backup before. When Extinction Core updated, a lot of our dinos and structures disappeared, so I want to restore from a backup. I have the backups, just not sure what to do with them...

The backups made using arkmanager backup contain the .ark (map), .arktribe (group) and .arkprofile (player) and .arktributetribe files from the saved arks directory (default Saved/SavedArks), and the GameUserSettings.ini and Game.ini files from Saved/Config/LinuxServer


feat: add RemoveCompileScope recipe

Closes #2888

Github is acting up for some reason. I am trying to re-request a review from @jkschneider and @timtebeek
Thanks!

Looking at the implementation and current set of tests my worry is if we would correctly handle cases where there is a parent with a dependencyManagement section, setting an explicit scope, that is overridden with compile in a project pom.xml. That parent can also be defined outside the current project, and might need to be resolved & evaluated.

Looking at the implementation and current set of tests my worry is if we would correctly handle cases where there is a parent with a dependencyManagement section, setting an explicit scope, that is overridden with compile in a project pom.xml. That parent can also be defined outside the current project, and might need to be resolved & evaluated.

Are you aware of any other recipe that is doing this? Or is this a foundational need of rewrite?
If you can share any example that would be helpful


Deprecate xmlutils, jsonutils and stringutils modules

Description:
We need to support all the functions in xmlutils, jsonutils, and stringutils modules in lang modules.  Check which functions are currently missing in lang modules and check the possibility of adding them. If functions are supported in the lang module, deprecate them.

[ ] stringutils [#25100]

stringutils module is already removed hence removing the stringutils label

This is done. Hence closing the issue


Results returned without ElasticSearch running

I'm a bit puzzled by the fact that my app is returning results, without having any instance of ElasticSearch running...
For instance, I do:
Tag.search("stackoverflow").results.first
And it actually returns an #Elasticsearch::Model::Response::Result instance, containing "stackoverflow".
Here a gif of the issue: http://i.imgur.com/8jGB5x1.gif
Where does it come from?
Thanks.

I had setup my client to connect to bonsai.io when deploying to heroku and forgot about it completely... My bad!


import Atomics won't compile in Playgrounds

I am trying to import swift-atomics into a Playground.  This fails to compile the shims because Playgrounds don't (and as far as I can tell, won't) link libswiftCore.dylib.  This is similar to Issue #8 in that its dying in the same place, unfortunately the work arounds don't work here.  There is mention of using Unmanaged to do this in: SR-13708.
Given that the only use of these two functions is in:
extension Unmanaged {
  internal func retain(by delta: Int) {
    _sa_retain_n(toOpaque(), UInt32(delta))
  }

  internal func release(by delta: Int) {
    _sa_release_n(toOpaque(), UInt32(delta))
  }
}

I've been giving thought to trying to replace _sa_retain_n(void *object, uint32_t n) and void _sa_release_n(void *object, uint32_t n) with Swift versions that do what the comment in SR13708 suggests, but wanted to check and 1) see if that is actually advisable and 2) ask why the comment:
"Looping over Unmanaged.retain/release doesn't seem like a viable option, although I'm sure that would work too."
indicates both that it "doesn't seem viable" and "would work too" .
Information

Package version: 1.0.2
Platform version: MacOS 12
Swift version: swift-driver version: 1.62.1 Apple Swift version 5.7 (swiftlang-<IP_ADDRESS>.7 clang-14<IP_ADDRESS>)
Target: arm64-apple-macosx12.0

Checklist

[X] If possible, I've reproduced the issue using the main branch of this package.
[X] I've searched for existing reports of the same issue.

Steps to Reproduce
git clone https://github.com/CSCIX65G/AtomicsExample/tree/main/Sources/AtomicsExample, try to run the Playground
Expected behavior
The playground should compile and run.
Actual behavior
The playground fails to compile

Ok I went ahead and implemented the looping construct and now my fork passes all the tests and can run in playgrounds.
I need to run the tests on both the modified and unmodified code to try to quantify how much of a performance hit this is.  It seemed noticeably slower, but that could be just that I was watching it.
As a side note, it seems like a pretty big oversight that the std lib doesn't implement these two methods in Unmanaged so that none of this is necessary in a library like swift-atomics.  It also seems bad that I have to use a fork to demonstrate the use of swift-atomics in a playground.
Any thoughts or advice for other solutions are welcome.

Looping over swift_retain/release technically doesn't break correctness, but if the loop survives into the generated binary, then the resulting code will have unacceptably bad performance, so this isn't a viable workaround, unless the relevant LLVM optimization pass replaces the loop with a direct call to swift_retain_n/swift_release_n.

I had sort of figured that out.  So I have a branch that implements it this way which I use for teaching about atomics using playgrounds and use the main branch version for actual compiled code.

Hm. This package needs these two functions only to work around https://github.com/apple/swift/issues/56105.
Fixing that bug would allow the package to remove these, although only if it is compiled using a toolchain that includes the fix.
(The package would still need the headers, though, so this probably wouldn't help with #62. To fix that, we likely need to wait until we have native atomics in the Swift toolchain.)

Planning for getting rid of the C module has started in https://github.com/apple/swift-atomics/pull/74. However, this won't land until some Swift compiler & stdlib work is done (and is shipping), so the current workaround will need to remain in place for a while.

As a stopgap workaround, #95 attempts to switch to using dlsym to access these on Darwin platforms, hopefully bypassing this issue.

#96 and #97 explore two more ways to resolve the problem: hiding the calls inside inline assembly, or using inline assembly directives to smuggle in an extra linkage.
Of the three attempts, #97 seems the least invasive, so that looks like the most promising option.

I verified that #97 will resolve the "symbol not found" issues for _swift_retain_n and _swift_release_n.
However, I'm still seeing subsequent missing symbol issues about _sa_retain_n and _sa_release_n. These are evidently due to the build system deciding to build _AtomicsShims as a standalone framework, but then failing to link it with the client binary. It seems this is a common failure for packages with C language targets; for example, I managed to reproduce it with swift-nio's NIOConcurrencyHelpers product (which imports a C module called "CNIOAtomics", similar to "Atomics" importing "_AtomicsShims"). Unfortunately I don't know a good workaround.


brew tap fortran-lang/fpm fails

Below is the transcript of what happens when I execute the command provided in this repository's README.md.  I don't have trouble cloning any other repositories.  I have 2FA set up and don't have trouble cloning any other repositories.
% brew tap fortran-lang/fpm
==> Tapping fortran-lang/fpm
Cloning into '/usr/local/Homebrew/Library/Taps/fortran-lang/homebrew-fpm'...
Username for 'https://github.com': rouson
Password for<EMAIL_ADDRESS>
remote: Support for password authentication was removed on August 13, 2021. Please use a personal access token instead.
remote: Please see https://github.blog/2020-12-15-token-authentication-requirements-for-git-operations/ for more information.
fatal: Authentication failed for 'https://github.com/fortran-lang/homebrew-fpm/'
Error: Failure while executing; `git clone https://github.com/fortran-lang/homebrew-fpm /usr/local/Homebrew/Library/Taps/fortran-lang/homebrew-fpm --origin=origin --template=` exited with 128.

Oops.  I used the wrong brew tap command.


I cannot get it running

I'm getting the following error message while starting fail2ban after adding the pterodactyl files
Failed during configuration: Bad value substitution: option 'action' in section 'pterodactyl-sftp' contains an interpolation key 'logpath' which is not a valid option name. Raw value: '%(action_mwl)s'

The original action_mwl implies that fail2ban send the log as mail.
You can add logpath = /var/log/pterodactyl/wings.log in the pterodactyl-sftp jail.
However, pay attention to the size of this log file... it could be huge.


remove parenthesis from asBool, asUInt methods

A lot of chisel code does not use parenthesis with these methods, but since they are declared with parenthesis, this leads to a bunch of warnings. We should consider following the common usage patterns.

I believe @jared-barocsi is working on this?

Yup. This will be fixed with: https://github.com/chipsalliance/chisel3/pull/2124


[EPIC] wasmCloud OTP Platform Support

This issue serves as a larger epic tracking the release artifacts we produce, platforms wasmCloud is compatible with, and some specific dependencies that are necessary to run on specific platforms.
Current Release Artifacts

Release Binary
Operating System
Architecture
Static
Platform Dependencies
CI/CD

aarch64-linux.tar.gz
Linux
aarch64
❌
libcrypto 1.1, glibc 2.31+
✅

aarch64-macos.tar.gz
MacOS
aarch64
❌
<EMAIL_ADDRESS>❌

x86_64-linux.tar.gz
Linux
x86_64
❌
libcrypto 1.1, glibc 2.31+
✅

x86_64-macos.tar.gz
MacOS
x86_64
❌
<EMAIL_ADDRESS>✅

x86_64-windows.tar.gz
Windows
x86_64
❌
Windows 11 Home
✅

amd64-linux-static.tar.gz
Linux
x86_64
✅
N/A
✅

Current remaining work in the release artifacts section

[ ] Automatic CI/CD building of the aarch64-macos.tar.gz artifact
[ ] Replace downloads of dynamically linked hosts with static hosts when available to maximize compatibility

Future Release Artifacts

Release Binary
Operating System
Architecture
Static
Platform Dependencies
CI/CD

WIP #593
MacOS
x86_64
🚧
N/A
🚧

TODO
MacOS
aarch64
🚧
N/A
🚧

TODO
Linux
aarch64
🚧
N/A
🚧

Current remaining work in the future release artifacts section

[ ] Static x86_64 MacOS builds to remove openssl dependency
[ ] Static aarch64 MacOS builds to remove openssl dependency

#616 is addressing many of these, just keep an eye out

First, a note: static builds are not supported by Erlang at all. It is not possible to statically-link Erlang OTP, therefore static builds are technically impossible today if we keep using OTP.
Static builds would only be possible if we used an alternative Erlang implementation to compile wasmcloud-otp, e.g. https://github.com/GetFirefly/firefly , which is experimental and does not have good support for Elixir yet
In https://github.com/wasmCloud/wasmcloud-otp/pull/616#issue-1689293596 we get artifacts with minimum possible amount of dependencies (NOTE, that e.g. Musl self-extracting binary is, indeed, statically-linked, but the actual executables are not, and cannot be, as outlined above), that is:

MacOS: none
Ubuntu/Debian:

ca-certificates - self-explanatory

Alpine:

libgcc - that's actually a NIF dependency, which is added by the Rust compiler https://github.com/rust-lang/rust/issues/82521

Windows: none

Currently aarch64 GNU build requires GLIBC 2.32, which is available on Ubuntu 22.04, but not on Ubuntu 20.04. x86_64 GNU build depends on an earlier GLIBC, which is provided by Ubuntu 20.04.
All of these builds are produced by CI (including the Erlang distribution). MacOS ERTS are available pre-built at https://github.com/rvolosatovs/otp/releases/tag/OTP-25.3.2, the rest are automatically built if necessary, but in overwhelming majority of cases it should be fetched from cache

For all intents and purposes, I think we have a great set of platforms here. I'm going to add the pinned label so that it won't get cleaned up by our stale issue bot and leave it as a good piece of information.


Unable to open the secured Website Applications inside docker,even after passing Cookies

Hi
I am able to open an applications which are not secured inside the docker successfully when we try to open secured applications with cookies inside docker using webdriverio.
We are not able to able any.There are no logs getting generated inside the docker
Request your Help on this issue

You may want to use VNC while a browser in the container is running to see what is going on. Its possible that browser needs to disable SSL verification to access a site.

As for logs - did you set debug: true and dockerLogs: './?

As for VNC Due to security reason I won’t  be able to install
Logs
I tried to add code for   docker compose up logs in docker file.
But NO luck

Few questions:

Are you using Docker to run Selenium image and trying to access external site to test?
Is secure site you're trying to access running with self-signed certificate?

1 . I am using Docker to run selenium webdriverio scenarios to test the application .not external site
As selenium image

Yes , the application I am trying to access has self signed  certificate

In that case it is quite possible that a browser inside container is blocking self-signed certificate sites just like it does in your browser (when it prompts to accept an unrecognized certificate). I believe in Chrome there is a flag which ignores custom certificates (i.e. --allow-insecure-localhost). Try passing that and see what happens.

Hi
Do we need to pass along  the url or we need to pass this in Dockerfile while creating the image of selenium stand-alone chrome

No, those are passed-in as capabilities section of configuration for a browser.
capabilities: [{
    browserName: 'chrome',
    chromeOptions: {
         args: ['ignore-certificate-errors','disable-web-security']
    }
}]

Hi,
I added as per your suggestion with additional security features as shown in screenshots
But i am still not able to open the applications,I am getting the error like below(attached screenshot)

sorry, I can't see those attachments

Also try adding flags with dashes
capabilities: [{
        browserName: 'chrome',
        chromeOptions: {
            args: ['--ignore-certificate-errors', '--disable-web-security']
        }
}]

I added  the commands with dashes
Now I am getting the error message as
Logs show as
Url name is  “data;”
Title is showing as blank
Looks like the chrome browser is open but the application is not able to open  inside docker

This means it can not access it by url. If your app is running on your machine, then you can not use localhost for baseUrl since inside docker it means a different thing. So you need to use either an IP of your machine on your network or map that IP to a dockerhost name like explained here

@anil8040 any progress on getting this working?

Hi,
Still unable to open the secured website applications.
Other Application such as Google (I am able to open inside docker  and retrieve the title)
That is what i am worried  😟 .How come i am able to open Google  but not secured application inside docker
I am using Webdriverio V4,Node version 7
Is it Possible to start selenium-standalone chrome server inside linux machine and run scripts using webdriverio  (Without docker) ??

So the reason you have no problems opening up Google is because it has a valid certificate. Your site on the other hand uses self-signed certificate which Chrome's security prevents from opening it up. If you look deeper into Chrome CLI flags you may find one that works for you or bypasses security all together (I thought the ones I sent you would do that).
As for running selenium-standalone on linux - yes you would just install webdriverio, selenium-standalone service and Chrome - the test will run against locally installed Chrome.

@anil8040 is this resolved? Anything else I could help you with?

Hi
I added the docker host details which you told
.Now I am getting logs But still the issue exists
It Shows Blank when I tried to retrieve title of the application

Perhaps its a matter of adding waitFor in your test to ensure page has loaded?

@anil8040 have you figured it out, or should I close this issue?

I still have the issue. Unable to retrieve the element value

Closing due to inactivity


iphone 6s plus system update to 9.3,v-model  click more times have mistakes

iphone 6s plus system update to 9.3,v-model  click more times have mistakes.

Read the issue reporting guidelines.


After use, the mobile phone fever is serious, please check what is the reason

After use, the mobile phone fever is serious, please check what is the reason

iOS is more obvious

@yoyo89757001 Thank you for your feedback! 😄
flutter_tilt v2.1.2 improved this issue.
Cause of mobile phone overheating: High Energy Impact.
Before

Default configuration:

Disable sensors: Tilt(tiltConfig: const TiltConfig(enableGestureSensors: false))

After

Default configuration:

Disable sensors: Tilt(tiltConfig: const TiltConfig(enableGestureSensors: false))

Thanks for the improvement, I need to test the new version

@yoyo89757001 I have tested Android and iOS platforms.
Overheating is reduced.
If the problem persists, I will reopen the issue. 😄


CI Workflows various fixes

This PR attempt to fix various issues on the GHA workflows:

Dependency latest was failing due to a recent OSL update, I updated the code adding #ifdef to handle the changes, tagging @lgritz for visibility.
Platform latest failing on ubuntu-latest / Clang combo, proposing to disable those 2 jobs for the time being until the underlying issue gets fixed (https://github.com/actions/runner-images/issues/8659). Also updated the C++ standard to 23 to stay "bleeding edge".
Wheel workflow Windows failure required an update of cibuildwheel version. Updating the upload artifact action to remove the deprecation warning also required a change in the artifact layout, now each wheel is upload as a separate artifact you can find and download on the workflow summary page.
Updating the Node JS based action to remove deprecation warnings yielded an issue when running in ASWF docker images (<= 2022 IIRC) due to incompatible GLibC version, see https://github.com/AcademySoftwareFoundation/OpenColorIO/actions/runs/7836526047/job/21384368269, might be something to raise on the CI working group.
The analysis workflow is still failing at the upload report step, discussing with @jfpanisset it seems like a security feature of Github where PR can't have access to secrets to avoid leakage, will have to see if the fix works after merging.
Update the CI on Linux to add VFX 2024 Reference Platform new containers (thanks @jfpanisset), there is currently an issue with OIIO's Ptex dependency which might need a fix in the OCIO Docker image. Revert this change, will wait for the next ASWF Docker release.
Fixed a likely issue of incorrect Pybind11 usage mentioned in #1939 which seem hard to reproduce but ran into it while updating Platform latest to C++23 (which required a Pybind11 version update) on the Windows jobs.

My only comment is whether we should include the GIL release in this PR? https://github.com/AcademySoftwareFoundation/OpenColorIO/pull/1945/files#diff-6c87f76bfb1f06a48bb762f97632eee604f4d8cd2e852347c3b46209a056086a
Those seems to be beyond pure CI workflow changes.
If people are happy with it then fine.

@KelSolaar, given the large delays in getting PRs merged, we all sometimes include small miscellaneous improvements into a larger PR. It's not ideal, but it's a response to the current state of affairs where we don't have a sufficient number of reviewers.

@doug-walker : All good with me and I understand the frustration! It is one of the reason I have been hesitant to move colour to the ASWF and a more formal review process, it certainly hinders the development velocity.

Merging for good measure because the aforementioned note was the only comment I had!

@doug-walker : All good with me and I understand the frustration! It is one of the reason I have been hesitant to move colour to the ASWF and a more formal review process, it certainly hinders the development velocity.

Such rules are made by every project TSC individually to pick the right set of procedures for their project and team. There is no ASWF-mandated policy on how reviews and merges work.

I'm sorry, I see I was tagged here, but I didn't notice this PR until today. Apologies if you wanted me to look at something here and I failed to reply. (I do "watch" this project, but sometimes when I'm busy, I'm a little eager with the delete key if I don't see a PR title that I think is highly relevant to me.)

@KelSolaar Agree the GIL changes could have been separated in a different PR, I included these here because it fixed the WIndows Platform Latest workflow when I switched them to C++23.
@lgritz No worry at all! I tagged you because I made some OSL related fix for the unit tests.


feat: expand GraphQL search fields

Currently we only support ID we should expand that to support more properties
Events:

ID
Name
Version
Release
PlatformID
Package

EventReceivers:

ID
Name
Version
Type
Fingerprint

EventReceiverGroups:

ID
Name
Version
Type

Now returns a list of things, but does not include the additional requested parameters

Need to clean pointer shenanigans as part of this Issue

If we need slice-pointers, at least keep them out of the storage pkg. Resolvers can hit storage methods and indirect further as necessary.
Alternatively can make schema arrays non-null, e.g., event_receivers(id: ID!): [EventReceiver]!. Then you are safe to return slices directly. This makes the query signature ambiguous for get-by-id scenarios; if an id doesn't exist, is that an error or do I get an empty array? To cover that, I'd suggest a dedicated id query:
type Query {
  # If no receivers meet criteria, should return empty (but not-null)
  # Later expand for pagination
  event_receivers(input: <input type>): [EventReceiver]!
  find_event_receiver(id: ID!): EventReceiver
}

Idk graph best practices but that seems most intuitive to me.

#82


Add toolbar icon for Add note

I wanted to add this to the toolbar as I use it a lot, and noticed that it wasn't possible. Please add this option.

Just to add, the same for the iPad implementation; next to the icons for adding a link and an image.

I'll consider this for inclusion in a future release.

This has been added and will be in the upcoming 2.0.9 release.


Fixes Issue with Loading bzlead

This fixes an issue when trying to apply changes to the se-scrap-recycling recipe

Thanks!


Updates examples for aws-sdk-js v2 -> v3

Which Cloudflare product does this pertain to?
R2
Existing documentation URL(s)

https://developers.cloudflare.com/r2/examples/aws-sdk-js/

Section that requires update
All of them. Installation, imports and usage are slightly changed.
What needs to change?
https://aws.amazon.com/blogs/developer/modular-packages-in-aws-sdk-for-javascript/ here an info on what changed + how npm package structure has evolved.
How should it change?
I could send a PR for use-cases already covered on the cloudflare R2 documentation page mentioned above 🤔
Additional information
No response

@DeTeam if you wanted to fork the repo and submit a PR to make those updates, that would be much appreciated! 😀

Awesome @dcpena!
Give me a few days, I will tinker with a few examples today/tomorrow and then will submit a PR 🙏


Cairo won't start

I don't know what the issue is worked yesterday

i have done a uninstall reinstall with no luck

bug appears to have solved itself


Add support for section help (with basic interpolation support)

Add a div.small.help-block to each dynamic section which looks up an optional static panel and renders it as markdown with the dynamic interpolation support (via the context object).
This will allow  us to provide deeper links with non-Waltz systems by allowing creation of hyper links with dynamic portions (i.e. [extSys](http://foo.bar/entity/id/${ctx.entity.id}) would render with the trailing part of the path correctly filled in.

cc'ing @panterose


CSS file is properly commented

Technical Acceptance Criteria: 40%
Satisfies all of the preceding acceptance criteria plus the following code improvements:
Application's CSS file is properly commented.

Updated file to include comments on each CSS declaration


Collect ISBN #’s for entire list.

We need to collect and confirm ISBN #’s for every book on the curriculum.

Happy to do this if needed. However, I think if you get the linking correct then this should be sufficient.

This is now completed in the fork,

Thank you! Please submit a PR for that when you get a chance.

Done. Shall we close this thread?

Thanks! Closed.

Now done.
Looking forward to working with you.
Thx.
On Fri, 1 Jul 2022 at 12:23, Chris, LICSW @.***> wrote:

Thank you! Please submit a PR for that when you get a chance.
—
Reply to this email directly, view it on GitHub
https://github.com/unshakenme/OSPAT/issues/1#issuecomment-1171856565,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZWWJQU76A6P4UTDSWHXSXDVRZJDTANCNFSM5K7CEX4Q
.
You are receiving this because you commented.Message ID:
@.***>

Likewise. Thanks for the help thus far!


[WIP] Fix Restore file to read-only node (DecomposedFS)

test
fixed?

apiTrashbin/trashbinSharingToShares.feature:82
✅

apiTrashbin/trashbinSharingToShares.feature:83
✅

apiTrashbin/trashbinSharingToShares.feature:102
✅

apiTrashbin/trashbinSharingToShares.feature:82103
✅

I think this looks good. But what is with @butonic's remark: #1913 (comment)?

It's a bigger cookie to eat 🍪

@refs please rebase and drep the regex based UUID format check.

@labkode 🍰 ready for review and mergie mergie 👯


Expose TranslationServer::standardize_locale add_default param publicly

Comparing locales can have surprising outcomes since it standardizes locales with defaults. For example, zh and zh_CN result in an exact match since the defaults change them both to zh_Hans_CN. Expose the add_default parameter publicly with a default of false so the fully standardized locale can be inspected.
Closes: https://github.com/godotengine/godot-proposals/issues/11119

@bruvzg could I get another review? Thanks!

Thanks!

Comparing locales can have surprising outcomes since it standardizes locales with defaults. For example, zh and zh_CN result in an exact match since the defaults change them both to zh_Hans_CN. Expose the add_default parameter publicly with a default of false so the fully standardized locale can be inspected.
Closes: godotengine/godot-proposals#11119


Updated dapr enable/disable to current spec.

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

Haven't been able to test this yet, but old code was trying to change dapr under properties instead of under properties configuration.

lgtm


Add Host Authorization checkbox for Property Managers

Issue:

bcgov/entity/issues/23745

Description of changes:

On Review and Confirm page - add a checkbox when Property Manager exists
Add validation, test update

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

/gcbrun

Temporary Url for review: https://strr-ui-dev--pr-239-uxhsc5yl.web.app

@dimak1 Should the checkbox still be visible if it is property host and selected yes for adding property manager info?

@dimak1 Should the checkbox still be visible if it is property host and selected yes for adding property manager info?

@shaangill025 hmm I check only for hasPropertyManager flag. @andyyanggov could you please confirm when is the checkbox is shown? Thanks

@andyyanggov could you please confirm when is the checkbox is shown? Thanks

It's intended for the PM to declare that they're authorized to do this on the host's behalf, so it should be hidden if the host themself is filling out the form (regardless if they hire a PM or not).

@shaangill025 based on Andy's answer, the checkbox is hidden correctly. What you think?

based on Andy's answer, the checkbox is hidden correctly. What you think?

With the preview url, I see the checkbox:

@shaangill025 ok i see the issue now, thanks for catching it.

/gcbrun

@shaangill025 updated, thanks!

Temporary Url for review: https://strr-ui-dev--pr-239-uxhsc5yl.web.app

/gcbrun

Temporary Url for review: https://strr-ui-dev--pr-239-uxhsc5yl.web.app

/gcbrun

Temporary Url for review: https://strr-ui-dev--pr-239-uxhsc5yl.web.app

/gcbrun

Temporary Url for review: https://strr-ui-dev--pr-239-uxhsc5yl.web.app

/gcbrun

Temporary Url for review: https://strr-ui-dev--pr-239-uxhsc5yl.web.app


Improve the documentation before 1.0.0 is released

Improve
https://astro-sdk.readthedocs.io/en/1.0.0b1/
Goal
Improve the content itself.
Acceptance criteria

[ ] Add section about supported technologies: https://github.com/astronomer/astro-sdk#supported-technologies
[ ] API Reference and Changelog should be after the Getting Started section on the left menu
[ ] Configuration section: a summary of all available parameters and configuration
[ ] Things to hide in the API Reference

databases
utils

[ ] Have a Concepts section where we explain:

File
Tables (temporary table)
Operators/Decorators & Task Flow API
Details about native aql.load_file
Append x merge

[ ] Create section "Examples" - include the explanation of the example of dynamic task mapping

Duplicate of documentation epic


add css properties to center the buttons

I did not put accent to much on design this time. The main is JS and the buttons.

This PR close #3


Unable to set access token in header with React Query client

Describe the bug
I'm trying to use ts-rest's React Query client (https://ts-rest.com/docs/react-query/v5-setup).
The requests I'm making need to include an Authorization header with Bearer ${token}, where token is an asynchronously generated access token.
Since access tokens expire after some time, the header needs to be set at the time of the request with a valid token.
The issue that I'm running into is that it doesn't seem possible to set the header at the time of the request when using ts-rest's React Query.
Option 1: baseHeaders in initTsrReactQuery
export const tsr = initTsrReactQuery(contract, {
  baseUrl: 'http://localhost:3333',
  baseHeaders: {},
});

The header can't be set as a base header on initTsrReactQuery because <tsr.ReactQueryProvider> wraps the application so the same header will be used throughout the entire lifecycle of the application, during which access tokens may expire.
Option 2: headers in useQuery
const { data, isPending } = tsr.posts.get.useQuery({
    queryKey: ['posts'],
    queryData: {
      headers: {
        authorization: `Bearer ${token}`,
      },
    },
});

The header can't be set when calling useQuery because while the token may be valid now, it may be expired when re-fetching the data at a later time.
Suggestion
It seems that the only way to support setting the header at the time the request is made is to allow passing in a function to baseHeaders or headers. (In my case, it'd need to support asynchronous functions as well since the token is generated asynchronously.)
Comparison: tRPC supports passing in a function to set custom headers: https://trpc.io/docs/client/headers
It seems that I've run into a limitation in the React Query client made by ts-rest, but I'm new to ts-rest and React Query / Tanstack Query so please let me know if I'm misunderstanding something or if there's a solution I'm overlooking!
ts-rest version
3.50.0

You can pass a function to baseHeaders, but not to headers for individual queries or mutations. This is on purpose, because passing any auth tokens either directly as a string, or a function will technically mean you need to include it as part of your Query Key somehow.
I believe the common and recommended approach is to handle all auth stuff outside of React Land entirely. So all the auth stuff, handling 401 errors, etc. should happen opaquely to your React code. You can achieve this using a custom client. You can change any parameters going to tsRestFetchApi(), and you can intercept the response to handle auth errors, token refresh, etc. before passing the response back to the client.
I think the TSR RQ docs need to clearly indicate this though, so I'll make sure to edit the docs now, and maybe add some example in the repo to show how auth is done later.

You can pass a function to baseHeaders, but not to headers for individual queries or mutations. This is on purpose because passing any auth tokens as a string means you need to somehow encapsulate it as part of your Query Key. Furthermore, passing any kind of function, whether for auth or other purposes, just makes the query non-deterministic and complicates formulating a well functioning query key.
I believe the common and recommended approach is to handle all auth stuff outside of React land entirely. So passing tokens, handling auth errors, etc. should happen opaquely to your React code. You can achieve this using a custom client. You can change any parameters going to tsRestFetchApi(), and you can intercept the response to handle auth errors, token refresh, etc. before passing the response back to the client. This provides very similar functionality to axios interceptors.
I think the TSR RQ docs need to clearly indicate this though, so I'll make sure to edit the docs now, and maybe add some example in the repo to show how auth is done later.

Thanks Gabrola! Appreciate the clarification and the link to the custom client docs.
For anyone who stumbles on this issue in the future, the custom client docs that Gabrola linked has an example on how to generate an access token asynchronously (e.g. when using auth from Firebase or Supabase) since baseHeaders takes in functions but not async functions: https://ts-rest.com/docs/core/custom#instantiate-the-client-but-access-a-token-during-runtime