It's not possible to access a kebab-cased attribute as camel-case and other way around in attrs

Version
3.2.22
Reproduction link
sfc.vuejs.org/
Steps to reproduce

Pass an aria-label to a component with :aria-label="foo" and :ariaLabel="foo"
Read the attribute in the component with the attrs parameter.

Parent.vue

<template>
  <Comp aria-label="This is an aria-label" />
  <Comp :aria-label="ariaLabel" />
  <Comp :ariaLabel="ariaLabel" />
</template>

Comp.vue

<template>

      kebab-case attr: {{ hasKebabCaseAriaLabel }}

      camelCase attr: {{ hasCamelCaseAriaLabel }}

</template>

Output:
kebab-case attr: true
camelCase attr: false
---
kebab-case attr: true
camelCase attr: false
---
kebab-case attr: false
camelCase attr: true

What is expected?
Vue should be able to infer an attribute regardless of is case as they are used camel-cased in the HTML.
aria-label in this example, will be used as camel-case in all three usages.
What is actually happening?
When aria-label is passed with kebab-case attrs.ariaLabel returns undefined and when it's passed with camel-case attrs['aria-label'] returns undefined.

I would say this is expected because that's how HTML attributes and are not adapted like props are when it comes to their naming. Also, this would be a breaking change

It's sad to hear it would be a breaking change because it's controversial to be able to pass attributes in both cases while creating elements but not possible to read them in the same way.
Here is an example to that uses both cases to create elements.
This would make library developers' lives easier.

@posva That's not the case. In SVG elements, not only the attributes are case sensitive, even the tags them selves are. This is a fundamental problem with Vue and that's a reason why I choose to create my own framework.

Not saying we aren't havving room to optimize SVG-related stuff, but I don't see how that's related to the specific issue here?

Not saying that there's no room to optimize SVG-related stuff, but I don't see how that's related to the specific issue here?

You're not realizing that tags appear in html can have and do have case sensitive attributes and tag names, which vue have completely no way to deal with.
Not mentioning XML namespaces which vue relies on for its v-xxx directives, users just can't deal with custom namespaces using vue.

Please open issues about these problems. They are adjecent to the issue at hand, bur not really related.

I'm aware of that Vue does handle SVG differently: it has a built-in list of tags and attributes that are marked as SVG related. That's a very stubborn way of telling if a tag should be svg: once SVG adds some new attributes to it and it's not on the built-in list, users will have no way but dirty hacks to get it working. One other way of the story is that most users just don't have the need to deal with SVGs and the part of code for handling SVGs are just built-in with no-way to strip them out for those who don't need them.
This is probably off-topic but I have to mention here:
The same thing applies to all other scenarios that Vue attemps to "take care" for users: Users won't have a chance to deal with custom namespaces/strange or non-standard runtimes/built-in keywords etc, and once users want to handle things more manually or more detailed, they will fail and have no way to solve the problem within the "beauty" of Vue.
Just think about the feeling of sitting on an automated viechale whose computer went mad but you have no way to take over.


#2115 user delete policy

Description
user delete policy
Related Issue : #2115

How Has This Been Tested?

When a user is deleted, change owner of the shared workspace.

If the shared workspace has a member, pick the owner.
If the shared workspace has no member, the shared workspace will be deleted.

If a resource is created by a user who is deleted, the creator is displayed Unknown user.

Private workspaces of deleted users is not displayed.

Delete the Delete button in the Members list.

Delete this user button is only displayed when Inactive is selected at the user detail page.

Need additional checks?
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

run build
deploy to 5

run build
deploy to 5

run build
deploy to 5


Error: Couldn't find preset "@babel/preset-env"

What should happen?
foundation-site should compile with gulp
What happens instead?
events.js:160
      throw er; // Unhandled 'error' event
      ^

Error: Couldn't find preset "@babel/preset-env" relative to directory "node_modules/foundation-sites/dist/js" while parsing file: node_modules/foundation-sites/dist/js/foundation.js
    at node_modules/babel-core/lib/transformation/file/options/option-manager.js:293:19
    at Array.map (native)
    at OptionManager.resolvePresets (node_modules/babel-core/lib/transformation/file/options/option-manager.js:275:20)
    at OptionManager.mergePresets (node_modules/babel-core/lib/transformation/file/options/option-manager.js:264:10)
    at OptionManager.mergeOptions (node_modules/babel-core/lib/transformation/file/options/option-manager.js:249:14)
    at OptionManager.init (node_modules/babel-core/lib/transformation/file/options/option-manager.js:368:12)
    at File.initOptions (node_modules/babel-core/lib/transformation/file/index.js:212:65)
    at new File (node_modules/babel-core/lib/transformation/file/index.js:135:24)
    at Pipeline.transform (node_modules/babel-core/lib/transformation/pipeline.js:46:16)
    at Babelify._flush (node_modules/babelify/index.js:27:24)
    at Babelify.<anonymous> (_stream_transform.js:118:12)
    at Babelify.g (events.js:292:16)
    at emitNone (events.js:86:13)
    at Babelify.emit (events.js:185:7)
    at prefinish (_stream_writable.js:505:12)
    at finishMaybe (_stream_writable.js:513:7)

Your Environment

Foundation version(s) used: 6.5.0

Checklist

[x] I have read and follow the CONTRIBUTING.md document.
[x] There are no other issues similar to this one.
[x] The issue title and template are correctly filled.

Please provide your package.json and gulpfile and .babelrc file.
You need @babel/preset-env.

yeah, just locking to previous version instead


Functional C

Here is a link to an old book on functional programming
https://research.utwente.nl/en/publications/functional-c
by Peter Hartelt
https://research.utwente.nl/files/5128727/book.pdf

would be a good addition. may want to note year and authors

I cloned the repo and added the book to free-programming-books.md
Would I just push that on master for the pull request?

give it a try


Sadly it does not build

16 | use esp_idf_svc::timer::{EspISRTimerService, EspTimer};
   |                          ^^^^^^^^^^^^^^^^^^
   |                          |
   |                          no `EspISRTimerService` in `timer`
   |                          help: a similar name exists in the module: `EspTimerService`

sadly I cannot build the project after adding your example code

Hi,
Some documentation missing, sorry. You need add in your sdkconfig.defaults file (at root of your rust project):
CONFIG_ESP_TIMER_SUPPORTS_ISR_DISPATCH_METHOD=y
CONFIG_ESP_TIMER_INTERRUPT_LEVEL=1

Version 0.2.0 released

Thank You very much!


conflict resolution should be opt-in, no longer default

It is too hard to do all at once.

👍

I see --ignore-conflicts in README.md

@Gaurav0 Yes, it should be --resolve-conflicts, so without the option it doesn't do anything for you.


adding bindings to AppProps

I like much this binding, but I can in pages/_app.res because no has AppProps binding.

FYI, for everything in pages/* I use JS to avoid problem with JS export etc.
So I use proxy pages for actual pages, but keep JS for _*.js next files.
Feel free to contribute to the bindings if you like !


Improve wording in compound assignment with augmented addition challenge

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no): Closes #10998

Description

Improved wording through a code example to explain the problem.
closes #10998

LGTM :+1:
Thanks for contributing to the most awesome open source community :tada:


vox activation not working

This xontrib is useful with terminals like kitty which don't remember the last session. But when this xontrib is loaded, my custom script to activate the vox environment based on the current location is not getting fired. I suspect it is because the events.on_cd is not fired.

Hi @jnoortheen! Thank you for reporting this!
Is there a way to repeat this (with description of expected and got behavior)?
But if you already have a fix PRs are very welcome!

Yes you could use the autovox's hooks and check if that works after enabling this xontrib.
Also I have a fix, I will create a PR.


Creating Training Polygons - Single or Multi-AOIs

Simple Version:
I guess my question is, can I have multiple 1x1km grid AOIs per poly/grid file?  For example,  '010101_poly_2022.gpkg' might have five AOIs with plot bounds.  Or am I allowed only a single AOI per image region?
More Detail:
I am setting up training data for TZ.  I'm getting confused, from my reading there was miss match between the example data and the docs. But maybe I am wrong.
In /cultionet/data, the region code doesn't  appear to be related to a {region} defined by the image folder name, and each only seems to contain a single AOI  - I had assumed this meant that there might be more than one AOI geopackage file per region - which seems impossible given the naming conventions in the doc

In docs, training dataset seems to refer to single file potentially with multiple AOI polygon features for each {region} - where region is defined by the folder name of the image tile.
project_dir:
  user_train:
    '{region}_grid_{time_series_end_year}.gpkg'
    '{region}_poly_{time_series_end_year}.gpkg'

So single file with single 1x1 AOI, single file with multiple AOIs, or multiple files with single AOI each?
@jgrss

@mmann1123 I would recommend having multiple files with a single AOI each. The initial versions of cultionet were setup to support files with multiple AOIs, but that support has sort of faded and it's not well tested.
Sorry if the docs are unclear. The example with training data is meant to show a single grid/polygon pair for one site, but the number of sites is unlimited. I will update the README to show something like:
Train data format:
project_dir:
  user_train:
    '{region}_grid_{time_series_end_year}.gpkg'
    '{region}_poly_{time_series_end_year}.gpkg'

Example:
project_dir:
  user_train:
    'site1_grid_2021.gpkg'
    'site1_poly_2021.gpkg'
    'site1_grid_2022.gpkg'
    'site1_poly_2022.gpkg'
    'site2_grid_2020.gpkg'
    'site2_poly_2020.gpkg'
    ...

@jgrss  Ok so the region doesn't need to be in the poly name. Got it. Thanks!

Sorry, it does actually. The {region} in the template needs to be replaced with a region name, which can be a unique string or 6-character integer (e.g., 000001). In the second example, the regions are 'site1' and 'site2'.
project_dir:
  user_train:
    'site1_grid_2021.gpkg'
    'site1_poly_2021.gpkg'
    'site1_grid_2022.gpkg'
    'site1_poly_2022.gpkg'
    'site2_grid_2020.gpkg'
    'site2_poly_2020.gpkg'
    ...

See an updated version of the README.

Oh, maybe you meant that '{region}' doesn't need to be in the filenames? Anything within and including {} is meant to illustrate the format and should be replaced with your names and years. E.g., '{region}_grid_{time_series_end_year}.gpkg' should be 'A_grid_2022.gpkg' for region A, year 2022.

Ok I think I am seeing the issue now. So the {region} has to match the image folder and the name of the grid or poly.
time_series_vars:
      image1:
          evi2:
             2021001.tif
             2021014.tif
      image2:
          evi2:
              2021001.tif
              2021014.tif
project_dir:
   user_train:
       'image1_grid_2022.gpkg'
       'image1_poly_2022.gpkg'
       'image1_grid_2022.gpkg'
       'image2_poly_2022.gpkg'

What threw me off is I thought you said that I could only have one box of training data in a given 'poly.gpkg'.  So I was creating multiple .gpkg for the same image/year. @jgrss


i18next — Replace lng=en to /en/

Next.js
Example source: https://github.com/zeit/next.js/tree/canary/examples/with-react-i18next
What's best way for replacing lang-param in url from ?lng=en to /en/
Necessary for ssr till load the page
Default: http://localhost:3000/catalog?lng=en
Need: http://localhost:3000/en/catalog

As per the issue template questions should be posted on spectrum.chat/next-js

https://spectrum.chat/next-js?thread=31adf3bd-8711-4bd1-91b7-04b598fae291


Adding error handling if the image is unusable.

I wrapped the processing in a try/catch so if an error occurs, the user is notified and can recover.

This change is

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

.gitignore, line 11 at r2 (raw file):

dist/
.idea/
demos/bower_components/

you shouldnt have to do this now that we use yarn (remove bower_components from your directory)

demos/imagenet/imagenet.ts, line 156 at r2 (raw file):

    await this.math.scope(async (keep, track) => {

      this.clearOutput();

I think the underlying problem is that we actually call animate() and then fromPixels() before the webcam is actually ready. Instead of catching the error, can we fix the root cause (not animating when the webcam / image element is not ready yet)?

Comments from Reviewable

Review status: 0 of 2 files reviewed at latest revision, 2 unresolved discussions.

demos/imagenet/imagenet.ts, line 156 at r2 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
I think the underlying problem is that we actually call animate() and then fromPixels() before the webcam is actually ready. Instead of catching the error, can we fix the root cause (not animating when the webcam / image element is not ready yet)?

The issues doesn't appear to be with the webcam but with the images. I noticed the beer image doesn't work no matter what you do, we are receiving an error: Error: The output # of rows (55.5) must be an integer. Change the stride and/or zero pad parameters.  I will look into that more.  The webcam is working fine from my testing.

Comments from Reviewable

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

.gitignore, line 11 at r2 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
you shouldnt have to do this now that we use yarn (remove bower_components from your directory)

Done.

Comments from Reviewable

Sorry to drop this - we fixed this in a PR that was related! Thank you for the contribution :)


Bug

Challenge http://www.freecodecamp.com/challenges/bonfire-mutations has an issue.
This is the code that should work
function mutation(arr) {
for (var i = 0; i < arr[1].lenght; i++)
if (arr[0].indexOf(arr[1][i]) == -1)
return false;
return true;
}
yet it doesn't for the first test case mutation(['hello', 'hey']);

Sorry, but I think your code is incorrect.

Well, an explaination as to why it doesn't work would be great.

@Valdrinit Looks like you have a typo. arr[i].lenght should be arr[i].length. That is atleast one error I see. There may be more. Good luck with the rest of the bonfires. Also, you will often find a quicker responses using the help chat room https://gitter.im/freecodecamp/help

@Valdrinit Sorry for being so brief, I should've at least directed you to the help chatroom. A very common but major error that I've fallen for a few times myself is to put the else part of the if statement inside the for loop. This way, the for loop doesn't loop all the way through, but returns a value after checking the first value only. This is easier to see if you use curly braces {} around your if statements.

Sorry, but that's not it. My function returns false as soon as it finds a letter in the second argument that can't be found in the first one; when the loop is done, meaning the first argument contains all the letters of the second one, it returns true.
It seems that the typo was the true problem.


can you please add test code ?

can you please add test code ?
thanks !

There's example code that do coverage test.
https://github.com/jamesruan/sodium/blob/master/sodium_test.go


Add judy array as storage type

Add and benchmark judy array as possibly faster storage.
@torkleyy I noticed you were interested in adding it as storage so I figured I would create issue for it so it doesn't get forgotten.

I took a whack at this and hit a couple snags, first of which was that the current rudy crate's RudyMap has no remove(). After switching to a git checkout (and building with nightly), I got everything wired up behind a feature flag, and found five of the usual storage tests fail against RudyMap.
I pushed this to a rudy branch but have no immediate plans to open a PR.

The bug in Rudy has been fixed in master. I ran the tests locally on the rudy branch of specs and all tests pass.

@WaDelma @torkleyy Would you like me to open a PR now (as an optional dependency pointing to git), or would you prefer me to wait for a followup rudy release?

I would prefer if I didn't accidentaly introduce the bug you encountered :P
We cannot release new version of specs before the fixed version of rudy is released, so I would prefer waiting for it.

A PR would be nice, yeah :+1:

RudyStorage landed in #222, so this issue can be closed.

Probably not, we still need proper benchmark results and a secrion in the book.

Ah! You are correct. This is what I get for trying to cover all my tabs with time-limited Internet access :-) Cheers!


new warning -Wlto-type-mismatch with LTO enabled (gcc8)

The openSUSE project is planning to use LTO (Link Time Optimization). So I tried to build version 2.0.0-dev5 (df280a7ff) for openSUSE Tumbleweed with LTO by adding "-flto=4" to the CFLAGS. Basically it worked, but I got a new warning:
...
[   76s] [ 93%] Linking C shared library libfreerdp-shadow2.so
[   77s] /home/abuild/rpmbuild/BUILD/FreeRDP-master/channels/client/tables.c:37:13: warning: type of 'smartcard_DeviceServiceEntry' does not match original declaration [-Wlto-type-mismatch]
[   77s]  extern UINT smartcard_DeviceServiceEntry(void);
[   77s]              ^
[   77s] /home/abuild/rpmbuild/BUILD/FreeRDP-master/channels/smartcard/client/smartcard_main.c:724:6: note: type mismatch in parameter 1
[   77s]  UINT DeviceServiceEntry(PDEVICE_SERVICE_ENTRY_POINTS pEntryPoints)
[   77s]       ^
[   77s] /home/abuild/rpmbuild/BUILD/FreeRDP-master/channels/smartcard/client/smartcard_main.c:724:6: note: type 'struct DEVICE_SERVICE_ENTRY_POINTS *' should match type 'void'
[   77s] /home/abuild/rpmbuild/BUILD/FreeRDP-master/channels/smartcard/client/smartcard_main.c:724:6: note: 'smartcard_DeviceServiceEntry' was previously declared here
...

The complete build log:
osTWbuild_lto.zip

Retrying with newer GCC might resolve the warning. I saw this warning in my project with GCC 8.5 and it went away when I use GCC 12. Looking at the OpenSUSE build at https://build.opensuse.org/package/view_file/X11:RemoteDesktop/freerdp/freerdp.spec?expand=1, looks like LTO is currently disabled there. If I try to enable it and rebuild, I don't see the warning any more
My build logfile from https://build.opensuse.org/build/home:jdanek:branches:X11:RemoteDesktop/openSUSE_Tumbleweed/x86_64/freerdp/_log can be viewed at
freerdp_os_tumbleweed_flto_auto.txt

@jiridanek : openSUSE only ships freerdp 2.*. The type mismatch occurs in the master branch alias freerdp3.
I checked the build log after openSUSE switched the default compiler for Tumbleweed to gcc12 and the type mismatch warning is still there; in freerdp3.

@hendwolt I compiled here on fedora 38 with clang-16 as well as gcc 13.1.1 and did not get any lto-type-mismatch warnings.
closing, repoen if you can still reproduce.


added email scraping

Description

Added email scraping file

Resolves: [Issue Number]
Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

[ ] Have you adhered to the repository's defined coding convention rules?
[ ] Have you updated the 'documentation.md' file with the method/function documentation?
[ ] Have you sent a message along with the result or response?
[ ] Have you used the try-catch technique?
[ ] Has the method/class been added to the documentation (md file)?

Screenshots

Uploading a screenshot illustrating the approach you developed for validation is required.

Additional Notes/Comments

Any additional remarks or suggestions concerning this pull request are welcome.

I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

Hey @kharshita590 | Any update on this PR?


train multiprocess issues

Hi, thanks for your excellent work and I am interest in. But when I try to run your code(don't change anything), the first iteration is ok, and it can print the information about reward and so on. when enter into 2 iter, the main process is also ok, but the followed childs process is always run without cpu usage, and finally the problem was located at policy_vs_net, It's gonna stay stuck here without forward output,  so have you meet this problem? Thanks!

ummm, sorry I am not sure here. Can you change to another computer and try?


Schema compare preview pane is not fully visible

Issue Type: Bug
I did a schema compare between 2 DB's and when I select a row in the result screen, the preview pane below is not showing all info.
Azure Data Studio version: azuredatastudio 1.16.1 (16801c522df68e63218b39ccb5b4a7431f9bf667, 2020-03-23T22:06:16.901Z)
OS version: Darwin x64 18.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4980HQ CPU @ 2.80GHz (8 x 2800)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.23GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

schema-compare
Mic
1.3.0

@Depechie How big is the window? A current limitation of schema compare is that the height is a maximum of 800 pixels.

I was running Azure data studio on a 25" screen with 2560*1440 resolution full screen.
So the window itself could not have been more than 800. I would guess it was around 100.

@Depechie I'm not able to repro this on my mac with the same build. Does this repro every time and for all rows? If the window is less than 800 pixels tall, there should be a scrollbar on the right side. Is that there or missing?

hey @kisantia seems i can't reproduce either... when I do the compare now, the window is indeed not covering the full height ( guess the 800px limit ). But it is much bigger than when I reported the problem.
So it occured for sure, but I can't reproduce it straight away.

Hmm that's strange, but good it isn't happening anymore. Please reopen this issue if it starts happening again consistently. Thanks!


Created slider in editor for Paddle speed and Ball speed and set Y senstivity to 1.0.

Unity information about Senstivity setting in Input Manager reads thus:
"For keyboard input, a larger value will result in faster response time. A lower value will be more smooth. For Mouse delta the value will scale the actual mouse delta."  They don't well document the affect on axis controls but all documentation I've found said the value should be set to 1.  During playtesting it felt as though setting it to 0.8 did decrease the sensitivity, but also seemed to decrease response time.
I created slider to test decreasing the speed of the paddles as well as the speed of the ball.  I then did play testing to find values that felt like a good balance between speed and control, and set those values in the start function of their respective scripts.

Be aware that values set in the slider will return to defaults every time a win state is called.
It felt good to me, but my reaction time may be a little slow because I'm old :P  Feel free to test and give feedback.

Furthermore, if it turns into a matter of preference it is possible the give the player access the these values in one form or another(like an actual slider or just a difficulty setting, etc.)

I don't really like testing the game in the editor, so I'm not the best person to test this.
@lightsoda @osse101 want to take a look?

You can test the changes with a build.  The sliders are optional, for if
you want to tweak the settings.  I used them while playtesting so I could
quickly modify things until they felt right.
On Apr 30, 2017 1:26 AM, "Cheeseness"<EMAIL_ADDRESS>wrote:

I don't really like testing the game in the editor, so I'm not the best
person to test this.
@lightsoda https://github.com/lightsoda @osse101
https://github.com/osse101 want to take a look?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Double-Fine-Game-Club/pongball/pull/165#issuecomment-298212333,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE1A8AhYEYjYBQ5_XWlPvtsqsrg2Ui7cks5r1BubgaJpZM4NMdB0
.

Confirming that the sliders, etc. are behaving as needed is an important part of reviewing whether or not changes are appropriate to merge or whether the PR needs more work.

-Gamepad sensitivity
I didn't see any difference between (lower sens + higher paddle speed) vs (1 sen + low paddle speed), so I think you're right in keeping sensitivity at 1 here, and it is paddle speed to adjust.  I did not notice reduced response time with lower sensitivity, other than lower paddle speed.  The numbers you choose, feel too fast to me.
-sliders
I was playing with paddle speed numbers and wanted to go below the range limit and the slider stopped me, so the number spread needs to be really opened up.  Not sure how useful it would be.

@osse101 The min values are 70% of previous values for min velocity and
paddle speed.  75% for max velocity.  Max value of the ranges are the
previous values.  The default values are close to the middle of those
ranges.  Changing the range values is an easy change.  What range would you
suggest?
What controls are you testing with?
What values felt good, for you, for paddle speed, max velocity and min
velocity during your testing?  The risk of the lowering the values too far
is that the game could feel too slow or less exciting.  Additionally
lowering paddle speed can give you better control but also reduce your
ability to move quickly across the goal line, for quick saves.
I noticed that most of the time, when I first start playing it feels way
too fast, but as I play longer I adjust.  Having some sort of ramp up in
speed would probably be ideal, but that's (probably?) a post AF
consideration.
For now, I think we should just find a good default balance that feels
satisfying to the broadest range of people who would play our game.
On Apr 30, 2017 3:41 AM, "Michael McKenzie"<EMAIL_ADDRESS>wrote:

-Gamepad sensitivity
I didn't see any difference between (lower sens + higher paddle speed) vs
(1 sen + low paddle speed), so I think you're right in keeping sensitivity
at 1 here, and it is paddle speed to adjust. I did not notice reduced
response time with lower sensitivity, other than lower paddle speed. The
numbers you choose, feel too fast to me.
-sliders
I was playing with paddle speed numbers and wanted to go below the range
limit and the slider stopped me, so the number spread needs to be really
opened up. Not sure how useful it would be.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Double-Fine-Game-Club/pongball/pull/165#issuecomment-298217196,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE1A8Hg3YnWddJImKrOLjZsYPPy9zrh8ks5r1Ds9gaJpZM4NMdB0
.

In-editor I'd like to see Paddle speed slider go ~0.4 to 1.3.  0.4 because 0.7 felt too fast still.  1.3 because I want the option of trying a faster paddle speed.  Ball speed I haven't played with.  Max ball speed isn't actually used AFAIK.
An in game slider would be much more limited, +/-15% of what we decide on.
I was playing with gamepad, but I liked how keyboard play felt.
The current in-game values are what feel good to me and I don't want to retune them again before Humble Bundle release.  Its a balance between shortest meaningful movement and how long it takes to cross the defense zone, and also how accurate your stop is.
Powers are intended to disrupt your opponents play giving them panic moments, if that is what you are looking for.  We do need some way to prevent never ending games.

I'll change the range values, but it will probably have to wait about 8-12
hours at a minimum.
.7 is a full 30% slower than current in-game speeds.  Unless sensitivity
does essentially the same thing, in which case it would be 10% slower.
On Apr 30, 2017 9:36 AM, "Michael McKenzie"<EMAIL_ADDRESS>wrote:

In-editor I'd like to see Paddle speed slider go ~0.4 to 1.3. 0.4 because
0.7 felt too fast still. 1.3 because I want the option of trying a faster
paddle speed. Ball speed I haven't played with. Max ball speed isn't
actually used AFAIK.
An in game slider would be much more limited, +/-15% of what we decide on.
I was playing with gamepad, but I liked how keyboard play felt.
The current in-game values are what feel good to me and I don't want to
retune them again before Humble Bundle release. Its a balance between
shortest meaningful movement and how long it takes to cross the defense
zone, and also how accurate your stop is.
Powers are intended to disrupt your opponents play giving them panic
moments, if that is what you are looking for. We do need some way to
prevent never ending games.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Double-Fine-Game-Club/pongball/pull/165#issuecomment-298232768,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE1A8LVbdIYJsMNafHseKEmca3KEYz-lks5r1I5ogaJpZM4NMdB0
.

The current in-game values are what feel good to me and I don't want to retune them again before Humble Bundle release.

Yeah, I don't think we should be making any unnecessary changes at this point, and shouldn't rush this PR on account of the Humble release.
That said, I really would love to get further real-world playtesting feedback (that is feedback that isn't from any of the core developers and isn't against AI) on the current handling. Without that, we're flying blind a little bit.

What kind of controller are you using?
On Apr 30, 2017 8:47 AM, "Stewart Martin"<EMAIL_ADDRESS>wrote:
I'll change the range values, but it will probably have to wait about 8-12
hours at a minimum.
.7 is a full 30% slower than current in-game speeds.  Unless sensitivity
does essentially the same thing, in which case it would be 10% slower.
On Apr 30, 2017 9:36 AM, "Michael McKenzie"<EMAIL_ADDRESS>wrote:

In-editor I'd like to see Paddle speed slider go ~0.4 to 1.3. 0.4 because
0.7 felt too fast still. 1.3 because I want the option of trying a faster
paddle speed. Ball speed I haven't played with. Max ball speed isn't
actually used AFAIK.
An in game slider would be much more limited, +/-15% of what we decide on.
I was playing with gamepad, but I liked how keyboard play felt.
The current in-game values are what feel good to me and I don't want to
retune them again before Humble Bundle release. Its a balance between
shortest meaningful movement and how long it takes to cross the defense
zone, and also how accurate your stop is.
Powers are intended to disrupt your opponents play giving them panic
moments, if that is what you are looking for. We do need some way to
prevent never ending games.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Double-Fine-Game-Club/pongball/pull/165#issuecomment-298232768,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE1A8LVbdIYJsMNafHseKEmca3KEYz-lks5r1I5ogaJpZM4NMdB0
.

I playtested this with a 17 yo, 15 yo, 10 yo and 42 yo.  My feedback is
based upon theirs and mine.
On Apr 30, 2017 9:03 AM, "Stewart Martin"<EMAIL_ADDRESS>wrote:
What kind of controller are you using?
On Apr 30, 2017 8:47 AM, "Stewart Martin"<EMAIL_ADDRESS>wrote:
I'll change the range values, but it will probably have to wait about 8-12
hours at a minimum.
.7 is a full 30% slower than current in-game speeds.  Unless sensitivity
does essentially the same thing, in which case it would be 10% slower.
On Apr 30, 2017 9:36 AM, "Michael McKenzie"<EMAIL_ADDRESS>wrote:

In-editor I'd like to see Paddle speed slider go ~0.4 to 1.3. 0.4 because
0.7 felt too fast still. 1.3 because I want the option of trying a faster
paddle speed. Ball speed I haven't played with. Max ball speed isn't
actually used AFAIK.
An in game slider would be much more limited, +/-15% of what we decide on.
I was playing with gamepad, but I liked how keyboard play felt.
The current in-game values are what feel good to me and I don't want to
retune them again before Humble Bundle release. Its a balance between
shortest meaningful movement and how long it takes to cross the defense
zone, and also how accurate your stop is.
Powers are intended to disrupt your opponents play giving them panic
moments, if that is what you are looking for. We do need some way to
prevent never ending games.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Double-Fine-Game-Club/pongball/pull/165#issuecomment-298232768,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE1A8LVbdIYJsMNafHseKEmca3KEYz-lks5r1I5ogaJpZM4NMdB0
.

I playtested this with a 17 yo, 15 yo, 10 yo and 42 yo. My feedback is based upon theirs and mine.

Definitely a good start! :D

I tried rebasing this.  Hopefully it worked ok?

I found some more information on sensitivity here https://docs.unity3d.com/Manual/ConventionalGameInput.html
It says: "Speed in units per second that the the axis will move toward the target value. This is for digital devices only."
I'm not sure what that means, though.  Does "digital devices" mean that it has no affect on the analog joystick axis?  What is the target value?  The value the axis returns is 0 to 1 for up and 0 to -1 for down.  Is that what they mean by target value?  If the value is speed in units per second, does that mean a higher value is faster or slower?
I've tried asking on Unity Answers but have received no reply.

@osse101 Be aware that the values on the sliders get reset, every win state, back to their original values set in void start().  You probably read that in the first post, but I'm reiterating just to be sure.


Erase in splines mode

Hi,
thanks for your lib, it's awesome. We are using it in a real-time manner in our app : Shuff
We have however, one main issue, we would like to have an "erase splines" mode.
We are currently doing so by using the splines mode in white, but we now are using image in background so it's no more possible.
Do you have any workaround in order to do so?
Thank you !

Which feature do you need?

Remove shapes with a curve boundary.
Erase image content inside a curve boundary.

In addition, thank you for telling your application to use this lib, I hope your application becomes better!

The image is behind the drawview, so what I need is remove content of the drawview like a rubber.
So if you want to erase the whole screen you would have to "paint" the whole screen with your finger.
I hope you understand because it is not easy to describe !

On 10 Aug 2015, at 03:00, Zhang Yungui<EMAIL_ADDRESS>wrote:
Which feature do you need?

Remove shapes with a curve boundary.
Erase image content inside a curve boundary.

In addition, thank you for telling your application to use this lib, I hope your application becomes better!
―
Reply to this email directly or view it on GitHub.

I would like to draw "eraser" curve, a curve that remove the content.
If the background is white , it's easy : splines mode with white color, but my background is transparent now, and drawing with clear color is not working.

On 10 Aug 2015, at 11:25, Zhang Yungui<EMAIL_ADDRESS>wrote:
Do you want to draw white and opaque curves on the background image?
―
Reply to this email directly or view it on GitHub.

You can draw the background image with a clip area generated from outline curves and the view boundary.

Thanks for your help!
Could you help me with the step that returns an area according to a shape ?
Le 10 août 2015 à 13:14:30, Zhang Yungui<EMAIL_ADDRESS>a écrit:
You can draw the background image with a clip area generated from outline curves and the view boundary.
—
Reply to this email directly or view it on GitHub.

I've add an example at SplinesEraser.
The following is the mainly code (CmdDrawSplinesEraser::touchEnded):
NSArray *paths = [[GiViewHelper sharedInstance] exportPathsForShape: dynshape()->toHandle()];
UIBezierPath *path = paths[0];
CAShapeLayer *shapeLayer = [CAShapeLayer layer];

shapeLayer.path = path.CGPath;
[imageView.layer setMask:shapeLayer];

It's so difficult that I've taken about 10 hours.
You can fork and change the example to make it better.


Updating mapping form doesn't bring girl's details

VHT module.
Button Update should bring girls records from the db

The value of the uuid in the mapping form in the VHT flavor doesn't match the value the MainActivity uses to check whether it is a registration form.
Unless I am mistaken, the mapping form should be the same in both flavors. I updated the value in the vht version to reflect that.


Fix use-after-free in SILCombine

SILCombine maintains a worklist of instructions and deleting of instructions is valid only via callbacks that remove them from the worklist as well. It calls swift::tryDeleteDeadClosure which in turn calls SILBuilder apis like emitStrongRelease/emitReleaseValue/emitDestroyValue which can delete instructions via SILInstruction::eraseFromParent leaving behind a stale entry in SILCombine's worklist causing a crash.
This PR adds swift::emitDestroyOperation which correctly calls the appropriate InstModCallbacks on added/removed instructions. This comes from swift::releasePartialApplyCapturedArg which was handling creation of destroys with callbacks correctly.

I tried to change the SILBuilder apis emitStrongRelease/emitReleaseValue/emitDestroyValue such that they don't do any "folding". Even though there are no changes in benchmarks, several tests need to be updated. Because there are no optimizations in GuaranteedARCOpts at -Onone that can optimize the same way as the "folding" in these apis. I'll do this cleanup at a later time. cc @gottesmm

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - d7086c513021003c81e3154195231999c4b2aa23

Build failed
Swift Test Linux Platform
Git Sha - d7086c513021003c81e3154195231999c4b2aa23

@swift-ci test

Build failed
Swift Test Linux Platform
Git Sha - f05accac0653779e3ca571b0f0af7fb9a1de8918

@swift-ci test

@swift-ci test OS X platform

@swift-ci test Windows platform

Build failed
Swift Test Linux Platform
Git Sha - f05accac0653779e3ca571b0f0af7fb9a1de8918

Build failed
Swift Test OS X Platform
Git Sha - f05accac0653779e3ca571b0f0af7fb9a1de8918

@swift-ci test


Jest encountered an unexpected token    SyntaxError: Unexpected token import

🐛 Bug Report
In my jest tests, when i attempt to run jest, this is output:
FAIL  test/product.test.jsx
  ● Test suite failed to run

    Jest encountered an unexpected token

    This usually means that you are trying to import a file which Jest cannot parse, e.g. it's not plain JavaScript.

    By default, if Jest sees a Babel config, it will use that to transform your files, ignoring "node_modules".

    Here's what you can do:
     • To have some of your "node_modules" files transformed, you can specify a custom "transformIgnorePatterns" in your config.
     • If you need a custom transformation specify a "transform" option in your config.
     • If you simply want to mock your non-JS modules (e.g. binary assets) you can stub them out with the "moduleNameMapper" config option.

    You'll find more details and examples of these config options in the docs:
    https://jestjs.io/docs/en/configuration.html

    Details:

    /Users/resource/BeiKe/pikachu-static-front-owner/node_modules/glue-redux/es/index.js:1
    ({"Object.<anonymous>":function(module,exports,require,__dirname,__filename,global,jest){import _defineProperty from 'babel-runtime/helpers/defineProperty';
                                                                                ^^^^^^

    SyntaxError: Unexpected token import

      1 |
      2 |
    > 3 | import { createGlue, gluePair } from 'glue-redux';
        | ^
      4 | import confirmLoan from './confirmLoan';
      5 | import Certification from './certification';
      6 |

      at Runtime.createScriptFromCode (../../.nvm/versions/node/v8.16.0/lib/node_modules/jest/node_modules/jest-runtime/build/index.js:1059:14)
      at Object.<anonymous> (client/glue/glue.js:3:1)

my .babelrc:
{
  "presets": [
    "@babel/preset-react",
    [
      "@babel/preset-env",
      {
        "useBuiltIns": "usage",
        "corejs": 3,
        "targets": {
          "chrome": 48,
          "ie": 9,
          "browsers": [
            "> 1%",
            "last 5 versions"
          ]
        }
      }
    ]
  ],
  "plugins": [
    "@babel/plugin-proposal-class-properties",
    "@babel/plugin-syntax-dynamic-import",
    "@babel/plugin-proposal-object-rest-spread"
  ]
}

my package.json
{
  "jest": {
    "transform": {
      "^.+\\.jsx?$": "babel-jest"
    },
    "moduleFileExtensions": [
      "js",
      "jsx"
    ],
    "moduleNameMapper": {
      "\\.(jpg|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga|css|less|scss)$": "identity-obj-proxy"
    },
    "testPathIgnorePatterns": [
      "<rootDir>/node_modules/"
    ],
    "transformIgnorePatterns": [
      "/node_modules/(?!@babel/runtime|glue-redux)"
    ],
    "collectCoverage": false
  },
  "dependencies": {
    "antd-mobile": "^2.2.11",
    "axios": "^0.18.0",
    "babel-jest": "^25.2.3",
    "chai": "^4.2.0",
    "child_process": "^1.0.2",
    "classnames": "^2.2.6",
    "core-js": "^3.6.4",
    "enzyme": "^3.11.0",
    "enzyme-adapter-react-16": "^1.15.2",
    "eslint-friendly-formatter": "^4.0.1",
    "fastclick": "^1.0.6",
    "glue-redux": "1.1.8",
    "history": "^4.7.2",
    "js-pinyin": "^0.1.9",
    "lodash": "^4.17.10",
    "node-schedule": "^1.3.2",
    "prettier": "^1.14.0",
    "progress": "^2.0.3",
    "prop-types": "^15.6.2",
    "react": "^16.4.1",
    "react-dom": "^16.4.1",
    "react-loadable": "^5.4.0",
    "react-redux": "^5.0.7",
    "react-router": "^4.3.1",
    "react-router-dom": "^4.3.1",
    "react-router-redux": "^5.0.0-alpha.9",
    "react-sticky": "^6.0.3",
    "redux": "^3.5.2",
    "redux-thunk": "^2.3.0",
    "weixin-js-sdk": "^1.4.0-test"
  },
  "devDependencies": {
    "@babel/cli": "^7.0.0",
    "@babel/core": "^7.0.0",
    "@babel/plugin-proposal-class-properties": "^7.0.0",
    "@babel/plugin-proposal-object-rest-spread": "^7.0.0",
    "@babel/plugin-syntax-dynamic-import": "^7.8.3",
    "@babel/plugin-transform-member-expression-literals": "^7.0.0",
    "@babel/plugin-transform-property-literals": "^7.0.0",
    "@babel/plugin-transform-runtime": "^7.0.0",
    "@babel/polyfill": "^7.8.7",
    "@babel/preset-env": "^7.0.0",
    "@babel/preset-react": "^7.0.0",
    "@babel/preset-stage-2": "^7.8.3",
    "@babel/runtime-corejs2": "^7.0.0",
    "autoprefixer": "^7.1.1",
    "babel-eslint": "^9.0.0",
    "babel-loader": "^8.0.0",
    "babel-plugin-import": "^1.12.0",
    "babel-plugin-transform-es2015-modules-commonjs": "^6.26.2",
    "clean-webpack-plugin": "^0.1.19",
    "copy-webpack-plugin": "^5.0.3",
    "css-loader": "^0.28.11",
    "eslint": "^4.19.1",
    "eslint-config-standard": "^12.0.0",
    "eslint-config-standard-react": "^7.0.2",
    "eslint-loader": "^2.0.0",
    "eslint-plugin-flowtype": "^3.2.0",
    "eslint-plugin-import": "^2.17.2",
    "eslint-plugin-jsx-a11y": "^6.0.3",
    "eslint-plugin-node": "^9.1.0",
    "eslint-plugin-promise": "^4.1.1",
    "eslint-plugin-react": "^7.13.0",
    "eslint-plugin-standard": "^4.0.0",
    "file-loader": "^1.1.11",
    "html-webpack-plugin": "^3.2.0",
    "identity-obj-proxy": "^3.0.0",
    "less": "^3.0.4",
    "less-loader": "^4.1.0",
    "mini-css-extract-plugin": "^0.4.1",
    "optimize-css-assets-webpack-plugin": "^3.2.0",
    "postcss-flexbugs-fixes": "^3.3.1",
    "postcss-loader": "^2.1.5",
    "px2rem-loader": "^0.1.9",
    "redux-devtools": "^3.4.1",
    "redux-devtools-dock-monitor": "^1.1.3",
    "redux-devtools-log-monitor": "^1.4.0",
    "source-map-loader": "^0.2.3",
    "style-loader": "^0.21.0",
    "svg-inline-loader": "^0.8.0",
    "uglifyjs-webpack-plugin": "^1.2.7",
    "url-loader": "^1.0.1",
    "vconsole": "^3.3.0",
    "webpack": "4.15.0",
    "webpack-bundle-analyzer": "^3.3.2",
    "webpack-cli": "^3.3.11",
    "webpack-dev-server": "^3.1.4",
    "webpack-manifest-plugin": "^2.0.3"
  }
}

envinfo
$ npx envinfo --preset jest

System:
    OS: macOS Mojave 10.14.3
    CPU: (4) x64 Intel(R) Core(TM) i5-5350U CPU @ 1.80GHz
  Binaries:
    Node: 8.16.0 - ~/.nvm/versions/node/v8.16.0/bin/node
    Yarn: 1.13.0 - /usr/local/bin/yarn
    npm: 6.4.1 - ~/.nvm/versions/node/v8.16.0/bin/npm

Input Code
None
Expected behavior/code
None
Possible Solution
None
Additional context/Screenshots
None
How should I solve this problem? thx

Please provide a minimal repro we can investigate. Closing until having that. Sounds like something cache related, try jest --clearCache, remove node_modules, install and try again

This happens to me when I run jest [options] rather than npm test (mapped to jest [options]). In other words, using a globally installed jest instead of a local one. The global jest ignores your jest config (as far as I can tell). Should a global jest pass control to the local one (grunt/gulp/webpack do this) or at least use a local jest config if there is one? (It would be nice if it did . . . I have no idea if it should be, but it isn't.)


Using composer require command to install package

Changed log

Using the composer require command to install this package.
If using the original approach, we need to run composer install or composer update commands after editing the composer.json file.

It's not easy and complicated. To simplify the approach, using the composer require command instead.

Thanks @peter279k! Looks good to me. :)


add txt and name to query params, remove Type check

addresses https://github.com/xenolf/lego/issues/275

LGTM
=== RUN   TestNewDNSProviderValid
--- PASS: TestNewDNSProviderValid (0.00s)
=== RUN   TestNewDNSProviderValidEnv
--- PASS: TestNewDNSProviderValidEnv (0.00s)
=== RUN   TestNewDNSProviderMissingCredErr
--- PASS: TestNewDNSProviderMissingCredErr (0.00s)
=== RUN   TestCloudFlarePresent
--- PASS: TestCloudFlarePresent (2.20s)
=== RUN   TestCloudFlareCleanUp
--- PASS: TestCloudFlareCleanUp (4.14s)
PASS
ok  	github.com/xenolf/lego/providers/dns/cloudflare	6.355s


1.4.0 Wrapper pull request

Version being tested in latest betas being tested by the community (see issues #445 and #451)
It includes almost all the pull requests currently pending, so a lot of new improvements and bug fixes.
Which pull requests it does not include

#319
#368
#414
#448 (last one as of the writing of this pull request)

New Features:

[FEATURE] Embedded photos in chat messages (#407)
[FEATURE] Image preview when pasting or adding images via selector (#398)
[FEATURE] 'Help -> About' window (#423)
[FEATURE] Add startup screen and fix window position on startup (#418)
[FEATURE] Switching between hide window and clear input when pressing 'escape' key (#409)
[FEATURE] Feature Request: Notification Privacy (#386 and #399)
[FEATURE] Add "show popup notifications" option to tray (#393)
[FEATURE] Debugging: All errors from main window are shown in command line (#426)

Bug fixes

[FIX] Fix on not fitting images in conversations list (#430)
[Fix] menu icon in dark scheme (#433 and #437)
[Fix] fallback to initials when thumbnail image not available #428
[FIX] fix deploy.sh and add --no-zip option (#424)
[FIX] Fix 'event.keyIdentifier' depreciation in favour of 'event.key' (#419)
[FIX] fixes three bugs with emojis (#412 and #413)
[FIX] menus had a typo 'getAccelarator' should be 'getAccelerator' (#403)
[FIX] Make hide the window (close to tray) work on all platforms if "Close (#397)

Experimental (now fully working)

[Experimental] Shows who has seen each message group (#429)

wait, i am un-doing my mess

@averissimo how can i talk with you privately ? maybe I could add you on yakyak ?

@davibe find my email in any of my commits (same email I use for hangouts)
ps. I actually tried sending you an email two days ago, but your personal server returned an SERVFAIL


ci: disable commitlint mergify rule

currently, PR merging is blocked due to the commitlint issue. disabling commitlint on the release branches now. more details at
https://github.com/ceph/ceph-csi/pull/2342
Signed-off-by: Madhu Rajanna<EMAIL_ADDRESS>

@Madhu-1 dont we need to just disable it on release-* branches ?

@Madhu-1 dont we need to just disable it on release-* branches ?

We have different mergify rules for different branches. This PR is disabling on the release branches only.

@Madhu-1 dont we need to just disable it on release-* branches ?

We have different mergify rules for different branches. This PR is disabling on the release branches only.

@Madhu-1 sure, I was asking that, cant we keep it disabled only in a particular branch and its section , for example : on release-34 just disable it on https://github.com/ceph/ceph-csi/blob/release-v3.4/.mergify.yml#L294 and thus ... instead of disabling it on devel for all the branches.

@Madhu-1 dont we need to just disable it on release-* branches ?

We have different mergify rules for different branches. This PR is disabling on the release branches only.

@Madhu-1 sure, I was asking that, cant we keep it disabled only in a particular branch and its section , for example : on release-34 just disable it on https://github.com/ceph/ceph-csi/blob/release-v3.4/.mergify.yml#L294 and thus ... instead of disabling it on devel for all the branches.

AFAIK this is not the way Mergify works. It always picks the configuration from the default branch. Adding rules to release branch won't help, if it works happy to make changes

AFAIK this is not the way Mergify works. It always picks the configuration from the default branch. Adding rules to release branch won't help, if it works happy to make changes

Yes, I think that is correct. We only need to adjust the configuration in the default branch (devel in our case). There is no use in changing the configuration in any of the other branches.


Add "Send as Mail" Feature to Toolbar - Alternatively Choose Action at Download

Hello,
Until Firefox 101.01 it was possible, to still use the "Choose What to do with the file" when You clicked on the "Download" Button inside PDF.JS Toolbar. - I ´ve already tried known Workarounds from here https://support.mozilla.org/en-US/questions/1370409 to bring back this feature without any effect.
I guess the true issue is somewhere in the area of how PDF.JS parses the output - since there are instances when you simply click on a PDF download link, and it still behaves as wished (probably due to the above mentioned workarounds)
The main reason is, that there needs to be a way to send a File from the Viewer by mail, without the need to handle the download manually first. So I would like to ask for these following features, which of each I would see as good solution

Is there any known way, to bring back the old way of behaviour of the "Download" Button inside of a PDF.JS viewer, so we can again choose if we want to open it in Full Acrobat or Save it.
Implement a "Send as Attachment by Mail" Button
Implement a direct "Open in Full Version of Acrobat" Button

Any ideas? I list this as an issue, as it is actually workflow dependant

Is there any known way, to bring back the old way of behaviour of the "Download" Button inside of a PDF.JS viewer, so we can again choose if we want to open it in Full Acrobat or Save it.

Not that I'm aware of unfortunately, since this is the result of changes on the browser side in Firefox (rather than in this repository).

2. Implement a "Send as Attachment by Mail" Button

Duplicate of #5527

3. Implement a direct "Open in Full Version of Acrobat" Button

Duplicate of #2553


once id.loc.gov/vocabulary/organizations avail as queryable dataset from QA, add to auth list and update templates

This is a placeholder issue for https://github.com/LD4P/qa_server/issues/376.

duplicate of #2587 , closing


Allow installation/use without flash attention dependency

Nice repo! Thanks for sharing.
Apologies in advance if I misunderstood something, but I wanted to use faesm with just the SDPA upgrade and found that I was unable to install or run it without flash-attn.
I made a couple of small changes as a workaround:

Change setup.py to make flash-attn an optional dependency
Make imports from the rotary and utils modules conditional on having flash-attn installed

Nice catch! Did u run the example code on the readme? Did it pass? Would love to merge if the code runs!
Thank you : )

Nice catch! Did u run the example code on the readme? Did it pass?
I was able to install and run the example code after making these changes and it seemed to work fine. I haven't tried running your tests yet but would be happy to do that, will report back!

That's amazing! thanks Alex!
It would be great to run the benchmark test, which will generate an efficiency comparison figure. Would like see how SDPA vs. official ESM.
Cheers,
Fred

Looks like test_compare_esm.py::test_esm_vs_faesm_numeric and benchmark.py::test_esm_vs_faesm_benchmark both pass if I set use_fa=False so hopefully everything is in order.

Beautiful! I think we can still get ~30% reduction if we use the pytorch SDPA. Not bad :)


.csproj settings don't work during Visual Studio Rebuild or Clean

TypeScript for Visual Studio 2015:  2.3.3
Visual Studio Enterprise 2015
Version 14.0.25431.01 Update 3
During a Rebuild, the following .csproj settings do not work; however, they do appear to work during a Build:
<TypeScriptEnabled>false</TypeScriptEnabled>
<TypeScriptCompileBlocked>true</TypeScriptCompileBlocked>

Since we are required to have the output .js files in source control, we see the following error for every file that is not checked out, due to the TypeScript being built during a Rebuild:
12>C:\Program Files (x86)\Microsoft SDKs\TypeScript\2.3\build\Microsoft.TypeScript.targets(326,5): error MSB3061: Unable to delete file "<filepath removed>.js". Access to the path '<filepath removed>.js' is denied.

For now we have to perform a Build after every Rebuild, but it would be nice if these options worked during Rebuilds.
P.S.  I just learned that the same errors also occur during a Clean.

rebuild just calls clean then rebuild. looks like we are not checking the flags on clean.

UPDATE: 06/19/2017
Today, I ran into another problem and opened #16630.  Fortunately, @yuit was able to help me fix the issue, which appears to be just one more issue with the ComponentModelCache folder and corrupted contents.  After deleting that folder and allowing VS 2015 to rebuild the contents, that issue was fixed.
Then I tried a rebuild, and guess what?  This issue no longer appears to be an issue.  I'll continue to watch for the issue, but this issue may also be related to ComponentModelCache folder corruption issues.

@Kaelum I don't think clearing the mef cache is what did it. I verified that this issue still exists under certain conditions.
At the moment, the trick to getting Clean/Rebuild to ignore your .js files is to delete the tsc.out file in the output directory (e.g. obj/Debug). This file gets generated when the project is built.
Can you tell me a bit more about your build process? When do you actually build your TypeScript files?
Note: Any reason you have  <TypeScriptEnabled> set to false? That's an internal flag only used by the language service, not the build.

@minestarks, I did delete the entire solution folder and reload from TFS, so that would have deleted the file.  If it pops up again, I'll try what you've described.
Our build process is fairly simple.  Since the output is transpiled code, we don't allow our build servers to build it, and it is only done within VS upon file save.  We have siloes for each application, and a dedicated tsconfig.json for each application in a folder under ~/Scripts/apps, and support code in folders under ~/Scripts/modules.  We use outFile to place the transpiled output, and declaration, into the ~/Scripts folder.
We used the following .csproj settings based on statements made in other threads, as we never want any TypeScript to be transpiled when we do a build.  Only file saves should transpile the TypeScript:
<TypeScriptEnabled>false</TypeScriptEnabled>
<TypeScriptCompileBlocked>true</TypeScriptCompileBlocked>

If the first setting doesn't need to be used, or shouldn't be, let me know.  The overall application is an enterprise application built on ASP.NET MVC,  Before we started using TypeScript, we had a mess left by a contractor, which required us to use Babel and many other tools that we don't want to be tied to.  The code that I wrote was extremely easy to convert over, but we're still trying to just get the basics working in this other code.

Thanks @Kaelum. I would go ahead and remove TypeScriptEnabled for A) it's not used by the build, just the VS language service and B) the Microsoft.TypeScript.targets import sets that right back to true anyway so it probably has no effect.
The bug with TypeScriptCompileBlocked and Clean will be fixed in the next release.

@minestarks I removed the line from the project today.  Thank you for fixing this so quickly.
Have a great week!


Error when running meth_phaser_parallel

Hi,
I tried to run the example cmd of:
    ./meth_phaser_parallel -b test_data/HLA.R10.haplotagged.bam -r test_data/GCA_000001405.15_GRCh38_no_alt_analysis_set.chr6.fna -g test_data/LSK.filtered.gtf -vc test_data/HLA.R10.phased.vcf.gz  -o test_data/work -k -1 -ml -2

it raises an error of
File "/home/_env/anaconda3/envs/mp_env/lib/python3.10/subprocess.py", line 1863, in _execute_child     raise child_exception_type(errno_num, err_msg, err_filename) FileNotFoundError: [Errno 2] No such file or directory: 'methphasing'
any clue to fix it? thanks.

Please put the executable methphasing into bashrc for meth_phaser_parallel to call it. exact command: export "PATH=/path/to/methphaser/:$PATH"
Thanks for noticing, I will put MethPhaser to conda for easier usage.

Got it, many thanks.


Vickel: Watch blisscarrentals.com

Vickel requests the watch of the watch_keyword blisscarrentals\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

blisscarrentals\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Document gpio for f373 and f3x8

Some GPIO peripherals do not have a LCKR register, so gpio_v2 can not be
applied.
~GPIO[AH] and GPIO[BD] could be merged. For now, I've considered not to, as
they are split the same in the corresponding svd file. I can merge them,
if that's desired.~
See RM0313 8.4.8 and RM0316 11.4.8

Ok, this case is a bit confusing:

x= A, B and D in STM32F303xB/C and STM32F358xC devices, x= A, B, C, D and F in
STM32F303x6/8 and STM32F328x8 devices and x = A, B, C, D, E, F, G, H in
STM32F303xD/E devices.

A GPIOC LCKR file should be available for the STM32F328x8 but not for the
STM32F358xC. The stm32f3x8.svd file does not contain a LCKR description at all.

Ok, this case is a bit confusing:

x= A, B and D in STM32F303xB/C and STM32F358xC devices, x= A, B, C, D and F in
STM32F303x6/8 and STM32F328x8 devices and x = A, B, C, D, E, F, G, H in
STM32F303xD/E devices.

A GPIOC LCKR file should be available for the STM32F328x8 but not for the
STM32F358xC. The stm32f3x8.svd file does not contain a LCKR description at all.
And the STM32F398 is not mentioned at all. Is there a way to resolve this with patched svd's?

Yes, GPIOC LCKR should be available for the STM32F328.
To me the most sensible solution is to add LCKR register to GPIOC in stm32f3x8.yaml.
Then you can use gpio_v2.yaml for stm32f3x8.
And to document GPIO for stm32f373 without too much duplication, you could do something like this:
in gpio_v2.yaml:
_include:
 - gpio_v2_common.yaml

"GPIO*":
  LCKR:
    "LCK[0123456789]*":
      Unlocked: [0, "Port configuration not locked"]
      Locked: [1, "Port configuration locked"]
    "LCKK":
      NotActive: [0, "Port configuration lock key not active"]
      Active: [1, "Port configuration lock key active"]

add a gpio_v2_f373.yaml:
_include:
 - gpio_v2_common.yaml

"GPIO[ABD]":
  LCKR:
    "LCK[0123456789]*":
      Unlocked: [0, "Port configuration not locked"]
      Locked: [1, "Port configuration locked"]
    "LCKK":
      NotActive: [0, "Port configuration lock key not active"]
      Active: [1, "Port configuration lock key active"]

and create a gpio_v2_common.yaml, with all the current content of gpio_v2.yaml except for the LCKR things

Thanks for the help :)
I think, creatinga gpio_v2_common is the way to go. I will include your
suggestions.

Yes, GPIOC LCKR should be available for the STM32F328.
To me the most sensible solution is to add LCKR register to GPIOC in
stm32f3x8.yaml.
Then you can use gpio_v2.yaml for stm32f3x8.

That's what I would like to do as well. The stm32f3x8.svd however does not include a LCKR description for the GPIOC periphery.
Ignoring this, it would fail at:
python3 scripts/svdpatch.py devices/stm32f3x8.yaml
Traceback (most recent call last):
  File "scripts/svdpatch.py", line 893, in <module>
    main()
  File "scripts/svdpatch.py", line 886, in main
    process_device(svd, root)
  File "scripts/svdpatch.py", line 864, in process_device
    d.process_peripheral(periphspec, device[periphspec],
  File "scripts/svdpatch.py", line 406, in process_peripheral
    p.process_register(rspec, register, update_fields)
  File "scripts/svdpatch.py", line 669, in process_register
    raise MissingRegisterError("Could not find {}:{}"
__main__.MissingRegisterError: Could not find GPIOC:LCKR
make: *** [Makefile:39: svd/stm32f3x8.svd.patched] Error 1

I've two options now:

Do not include GPIOC LCKR for the stm32f328
Or patch the svd, so GPIOC LCKR get's added?

I've two options now:

Do not include GPIOC LCKR for the stm32f328
Or patch the svd, so GPIOC LCKR get's added?

I was actually suggesting option 2.
To achieve this, you should add something like this in stm32f3x8.yaml:
GPIOC:
  _add:
    LCKR:
      description: GPIO port configuration lock register
      addressOffset: 0x1C
      access: read-write
      fields:
        LCKK:
          description: Lok Key
          bitOffset: 16
          bitWidth: 1
        LCK15:
          description: Port x lock bit y (y=0..15)
          bitOffset: 15
          bitWidth: 1

and so on...
BTW: you forgot to add the gpio_v2_common.yaml and gpio_v2_f373.yaml file in your last commit...

Okay, nice. I did apply option 2. However, I wonder what happens, lckr is accesssed when using e.g. the stm32f398 or similar, which do not have the LCKR register.

Rebased and fixed the typo. Thank you for the review @adamgreig :)

bors r+


zuul -> eureka client, authorization header lose

Spring Boot Version:2.0.0.RC1
Spring Cloud Version: Finchley.M6

Do not have this problem on a version

Zuul proxy didn't forward header

close this issues。
https://github.com/spring-cloud/spring-cloud-netflix/blob/master/docs/src/main/asciidoc/spring-cloud-netflix.adoc#cookies-and-sensitive-headers


Dispose disposable HttpContext

HttpContext is IDisposable so should be disposed at the end of the request.
Would suggest Func<IFeatureCollection, Task> application change to Func<IFeatureCollection, Task<IDisposable>> application in IServerFactory.Start
The Server Factory should then return the HttpContext from the closure: https://github.com/aspnet/Hosting/blob/dev/src/Microsoft.AspNet.Hosting/Internal/HostingEngine.cs#L88 to allow it to be disposed at the end of the request.

I don't think we ever really thought though it. I don't know why it was ever marked as IDisposable in the first place, we haven't found a use for it before now.  @benaadams's attempted use of it highlights some of the potential issues, such as non-deterministic lifetime.
I'd recommend removing IDisposable rather than trying to call it.

:+1: for removing IDisposable from HttpContext if possible.

Sounds good, good riddance IDisposable!


Follow upstream change in cost_calculator.py

Follow chia-blockchain Chia-Network/chia-blockchain@275969ddd  (Chia-Network/chia-blockchain#9964)

@arvidn :)

it looks like there may be a few more places this function is called that need to be sorted out for the linter

@arvidn Good call. I only removed this one which really broke cdv, but missed one in the inspect command. Updated.

@Quexington I have pushed a commit which should be compatible with both 1.2.11 and trunk. Can you please re-review?

I think this should stay its own repo, we want to move to chia-blockchain being mostly about consensus and full node stuff and this repo would be a move contrary to that direction.

I want to clarify that I use "mono repo" as a term of art, not to refer to a single git repository. The mono repo I've worked with had every project in its own mercurial repository, but a master repository tying them together at specific revisions. One  advantage of a setup like that is that if chia-blockchain introduces a change that breaks chia-dev-tools (or any other of our projects depending on it), CI would barf, unless chia-dev-tools is also changed to stay-up-to-date.

The approach you suggest is a head repo which ties together versioned submodules of subcomponents; AFAIK a monorepo is not that (don’t know the name for it though).

It can be, given the right tooling.
With large organizations, it's not scalable to have everyone clone the complete codebase, let alone the complete history (as is the case with git)

It can be, given the right tooling.

Sorry to be offtopic @arvidn, but I think that monorepo is not the name for this what you're describing. That is, versioned submodules tied together in a main repo. It's not a matter of tooling, but of naming of the approach. When you have versioning of subcomponents instead of versioning the entire thing, atomically, you do not have a monorepo. AFAIK and IMHO.

With large organizations, it's not scalable to have everyone clone the complete codebase, let alone the complete history (as is the case with git)

I disagree. If can Google, Microsoft and Linux can do it (what I call a monorepo, no submodules), it's obviously doable. Git 2.25 introduced sparse checkout. Microsoft also uses a monorepo for their entire Windows codebase and has a custom sparse checkout git extension for Windows called VFSForGit which retrieves objects from the git monorepo on-demand.

think that monorepo is not the name for this what you're describing.

It has all the properties listed on wikipedia's "monorepo" article, so I think it does. Keep in mind that when I say "appropriate tooling" I specifically mean tooling that implements the monorepo properties.

When you have versioning of subcomponents instead of versioning the entire thing, atomically, you do not have a monorepo

With the appropriate tooling, you can tie together the entire thing, atomically. It doesn't matter whether the underlying storage for "the thing" happens to use submodules or not. (But I think doing so makes it more suitable for open source development on github).

I disagree. If can Google, Microsoft and Linux can do it (what I call a monorepo, no submodules), it's obviously doable.

Being doable doesn't imply scalable. I don't have first hand experience from google, but wikipedia states that their repository is 80 terabytes. I very much doubt that all 27000 engineers at google has a complete copy of that repository checked out.

Git submodules have given me quite a lot of pain on my projects, so personally I strongly prefer the strict monorepo approach, at least for actively developed subcomponents

Free your mind and imagine better tooling :)
At one company I used to work for we used the (closed source, internal) predecessor to git-meta. It definitely fulfills all the properties of a monorepo (according to wikipedia) and even lacks some of the drawbacks wikipedia lists too.
Anyway, we (and maybe the python world) seems pretty set on using pypi as the intermediary to tie everything together, so there might be issues making releases with a monorepo setup as well.
The clear benefits would be that all of our projects could be tested together, in lock step. Currently they can't (not in CI at least)

I think that the better tooling is already here. It’s git subtree, which can be used to extract a subcomponent from the monorepo into a child repo if needed (with all of the history which touches the subcomponent), and it can also pull back changes from a dowstream child repo into the monorepo. The important part here being that the monorepo version is the first-class citizen which is developed atomically in the monorepo along with all of its dependencies and dependants, it looks just like a regular folder, and all it takes to make a change is a single commit and push to the monorepo. Updating the child repo, if having one is needed, can be automated (for e.g. open sourcing just that subcomponent and being able to pull from downstream).
I think that until your codebase is an order of magnitude larger than Linux (for which plain git still works fine), being concerned with scalability of the plain monorepo is premature optimization, and is like thinking how to kill a squat with a bazooka.
At work, I work on a medium-sized C++ codebase (the checkouted trunk is around 800 MB large, though some chunk of that is 3rd party blobs). When I pushed for an often-modified important subcomponent to be moved from a submodule to the main repo, everyone was insanely happy that they didn’t have to do the “commit, push submodule; update submodule; commit, push main repo” dance anymore. It’s now just commit and push to monorepo and simplies things dramatically.


Can't insert text fetched from an API

I find this tutorial very useful but I want to try more things and for example I want to insert text that is retrieved from an API, but this can't seem to work and I don't know why even though I know the text is being retrieved. At first, I thought it was because it is done asynchronously but I don't know the reason anymore. I would be glad if I could get some help

@SofiaOUANES Can you respond with the following information so we can better help you? Thanks.
Article URL
On what page did you find the problem?
Describe the problem
A clear and concise description of what is incorrect, unclear, or missing in the document.
Screenshots
If applicable, add screenshots to help explain your problem.


Use array short syntax

Thanks for the PR! I'd hold this one off either until:

shortly before the 1.3 release or at least until existing pull requests are merged
until support for existing releases that support PHP < 5.5 ends to avoid re-introducing verbose array syntax.

Merged manually in https://github.com/alcaeus/mongodb/commit/49433f0e4f9cb00f49a73e579bb36e32e348ebd6.

Thanks!


Move markdown parsing from the client to the server

Right now, when a task (née pipeline, #28) runs to completion (either successfully, or as a failure), the output of the last step that ran is considered to be the final output of the task.
This output is considered to be regular text which can be formatted using markdown.
The GraphQL API currently returns this text in its raw form, at which point the web client parses it as markdown, and converts it into HTML to show it on the screen.
This works, but specifically for the web client, this results in a binary that is 50% larger than it could be, because the markdown parser library used loads in the entire Unicode character table, which adds 160 kB of extra data (see https://github.com/blendle/automaat/issues/27#issuecomment-509297055).
To solve this, I propose we move the markdown parsing to the server, and let the clients dictate which format they want the output to be returned in.
The API currently looks like this:
https://github.com/blendle/automaat/blob/2d9bc81bec96ae606c2680090469376f3926b68a/src/server/schema.graphql#L227-L238
I propose the following change:
type TaskStep {
   id: ID!
   name: String!
   description: String
   processor: Processor
   position: Int!
   startedAt: DateTimeUtc
   finishedAt: DateTimeUtc
   status: TaskStepStatus!
-  output: String
+  output: StepOutput
   task: Task
}

+ type StepOutput {
+   text: String!
+   html: String!
+ }

This allows for a couple of improvements:

Markdown parsing happens server-side, improving client performance and reducing the Wasm binary size by 50% for the web-client.
Clients can ask multiple formats from the server, which makes it easy to support #17.
We can add new output formats in the future if needed, without introducing breaking changes to the API.

side note: the regular output is called text instead of markdown, since any step output can be formatted as markdown but doesn't have to be.

This is implemented as of 68cc532c71de8b9b001f00d5f8164fd5e8c75bc4 and de7ef8506421e10193a9ee2cde59d6eb4596530c.
As a result of this change, the Wasm binary has been reduced from 460 kB to 320 kB
(120 kB compressed) 🎉.


Width option has some problems

Hi,
first thanks for this plugin! I have found couple of problems with the width option. The label is always a bit wider than set width and also the caps are not taken into account, both combined makes it quite hard align labels.

Thanks for taking a bit of a deeper look at this.
I kind of tweaked some scaling factors a bit so the widths kinda matched up to mm. But it looks like it's a bit off.
Different end caps also will always increase the width currently.


Express Route Private Peering support for VMWare replication traffic

As per the Step 2 of the Replication Process (Link: https://learn.microsoft.com/en-us/azure/site-recovery/vmware-azure-architecture#replication-process), "Traffic replicates to Azure storage public endpoints over the internet. Alternately, you can use Azure ExpressRoute with Microsoft peering.".
However, the document "Replicate on-premises machines by using private endpoints" (link: https://learn.microsoft.com/en-us/azure/site-recovery/hybrid-how-to-enable-replication-private-endpoints) explains how to use Private Endpoints for ASR VMWare to Azure replication. So, this means that Express Route Private Peering is also supported?

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

ID: 9f8b6d0d-22ec-65e3-c21f-6a40fd2f1787
Version Independent ID: dc284d51-3502-b8f4-5eb6-1473a56b14ca
Content: VMware VM disaster recovery architecture in Azure Site Recovery - Classic - Azure Site Recovery
Content Source: articles/site-recovery/vmware-azure-architecture.md
Service: site-recovery
GitHub Login: @jsuri
Microsoft Alias: jsuri

@rjosephp
Thanks for your feedback! We will investigate and update as appropriate.

@rjosephp Please, note the following:
https://docs.microsoft.com/en-us/azure/site-recovery/concepts-expressroute-with-site-recovery
“Site Recovery replicates data to an Azure Storage account or replica Managed Disk on the target Azure region over a public endpoint. To use ExpressRoute for Site Recovery replication traffic, you can utilize Microsoft peering or an existing public peering (deprecated for new creations). Microsoft peering is the recommended routing domain for replication. Note that replication is supported over private peering only when private ends points are enabled for the vault.”
https://docs.microsoft.com/en-us/azure/site-recovery/hyper-v-azure-common-questions#can-i-replicate-to-azure-with-expressroute
Can I replicate to Azure with ExpressRoute?
Yes, ExpressRoute can be used to replicate VMs to Azure. Site Recovery replicates data to an Azure Storage Account over a public endpoint, and you need to set up Microsoft peering for Site Recovery replication. After VMs fail over to an Azure virtual network, you can access them using private peering.

@rjosephp Checking in to see if the above answer helped. Let us know if there are still any additional issues we can help with.

@SadiqhAhmed-MSFT -thank you for the response and it helped. I also would suggest that if you could this information to the Step 2 of the replication process, that would help to avoid any confusion. Thank you!

@rjosephp Glad to know it helped. We will share your valuable feedback with the content owner to review and update as appropriate.
@v-ksreedevan & @SharmisthaRai Can we consider the feedback for doc-enhancement?

@rjosephp Closing this issue for now as no action pending from my end.
@v-pgaddala & @SharmisthaRai Appreciate if you could review the feedback and update the doc if applicable.


[Feature Request] Latest and OS tags for releases

Is your feature request related to a problem? Please describe.
The unix installations are currently a bit clicking back and forth.
And we're tearing people into the terminal anyways, so let's make ArcCommander fully installable from there.
Describe the solution you'd like
Is it possible to automagically get linked to the latest arc releases for a specific OS:
https://github.com/nfdi4plants/arcCommander/releases/latest/macos
https://github.com/nfdi4plants/arcCommander/releases/latest/linux
Additional context
I've tried to write out the linux installation a bit more click-and-copy.
We'd just have to replace the "v0.2.1-linux.x64/arc" by e.g. "latest/linux.x64/arc" in the wget.
Assuming that wget is installed on linux by default (not an expert).
Install ArcCommander on Linux

Download latest release
wget https://github.com/nfdi4plants/arcCommander/releases/download/v0.2.1-linux.x64/arc

Make executable
chmod a+x arc

Move to suitable place (e.g. in your home directory or to /home/bin/ to make it accessible for all users)
mkdir ~/ArcCommander # Create folder "ArcCommander" in your home directory
mv arc ~/ArcCommander # move executable to that folder

Add that place to your path. This allows you to start the ArcCommander from any folder.
export PATH=$PATH:~/ArcCommander

Test that ArcCommander is properly installed
arc --version # check the current version

mkdir ~/testArc; cd ~/testArc # create a test folder in your home directory and change into it
arc init # initialize arc

cd ~/; rm -rf testArc # remove test folder

Is it possible to automagically get linked to the latest arc releases for a specific OS:
https://github.com/nfdi4plants/arcCommander/releases/latest/macos
https://github.com/nfdi4plants/arcCommander/releases/latest/linux

Afaik that's not possible, unfortunately.
But we could move all releases into one release per OS, meaning that we'd always override the current one.
Downside: Impossible to get older releases without building the repo yourself.

The URL is set to a specific release. But I think we can enhance the current GitHub action to also change this line in the README.md. Otherwise, we'd have to update the README.md with every release manually.
Besides that, I like your suggestion.

Shit, I had to look up afaik. :older_man:
If the gh action can take care of this, that would be perfect. Would probably also be easier for user trouble-shooting with the version specified rather than "latest".

Reminder to automate installation (instructions). #136


Pane > Left Nav > References don't match the items in the picture

Menu Button is listed as Nav Items, Nav Items are listed as something else, etc. Seems like Menu Button was attributed as 6 in the list when it should have been 1, and all references which follow are off by one. 0/1 index problem? :-)

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

ID: 745ab6c5-cfbe-a56f-dccd-ab55123f9223
Version Independent ID: 54455a9f-a92f-e462-558d-78fb696dba8c
Content: Navigation view - UWP app developer
Content Source: windows-apps-src/design/controls-and-patterns/navigationview.md
Product: windows
GitHub Login: @serenaz
Microsoft Alias: sezhen

Fixed - thanks for the catch!


chore: Added extra stylelint settings

Added settings that cause linting failures when turned on.

Hi @shannonmoeller, could you take a look at my PR when you get a chance?
I noticed a few settings were causing linting errors in our project when using prettier + stylelint.

Thanks :)

lgtm. Thanks and sorry for the late response.

Published as v2.0.0


upgrade to ghc-9.6.1

the minimum build compiler becomes 9.2.2 (9.2.1 was a broken release). https://github.com/ndmitchell/hlint/issues/1468 is the only known release blocker. update: this pr bumps cmdargs to 0.10.22

This looks great! I've built it locally and can confirm it builds with GHC 9.6.

Thanks!


[$500] Expense - Copy to clipboard on the system message for paid expense copies negative amount

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.3.95-5
**Reproducible in staging?:**Y
**Reproducible in production?:**Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
**Issue reported by:**Applause - Internal Team
Slack conversation:
Action Performed:

Navigate to staging.new.expensify.com
Go offline
Create a workspace
Go to workspace chat > + > Request money
Create a manual request
Click Pay elsewhere
On the expense report, right click on the system message for paid expense
Click Copy to clipboard
Paste it on the composer

Expected Result:
The correct paid amount is copied
Actual Result:
The negative paid amountn is copied
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/0e67e23b-3d0d-417a-98af-f84ffc5c2ad7
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ca92a3fdee611a1f
Upwork Job ID:<PHONE_NUMBER>745327616
Last Price Increase: 2023-11-03

Proposal
Please re-state the problem that we are trying to solve in this issue.
Expense - Copy to clipboard on the system message for paid expense copies negative amount
What is the root cause of that problem?
When paying money, we will create new optimistic data for new report action
https://github.com/Expensify/App/blob/7b546b0470b34bcaf5b9ee85e5011fff42443fc4/src/libs/actions/IOU.js#L2423-L2433
And we assigning iouReport.total to originalMessage.amount field. It causes this bug because iouReport.total is negative. Note that this bug doesn't happen online because BE overwrite reportActions.originalMessage.amount field to positive
What changes do you think we should make in order to solve the problem?
We need to update optimistic data like BE reponse
https://github.com/Expensify/App/blob/7b546b0470b34bcaf5b9ee85e5011fff42443fc4/src/libs/actions/IOU.js#L2423-L2433
we should update
        iouReport.total * -1,

What alternative solutions did you explore? (Optional)

@dukenv0307 @DylanDylann Why is the message showing correctly but not with the copied text?

@mollfpr When displaying the message. We manually convert negative value to positive value here
https://github.com/Expensify/App/blob/7b546b0470b34bcaf5b9ee85e5011fff42443fc4/src/libs/ReportUtils.js#L1473

@mollfpr Currently we have 2 places to save amount

iouReport.total (this field is nagative value, if we use this field we need to convert it to positive value)
reportActions.originalMessage.amount (the BE return this field is positive value but in the optimistic data we are setting a negative value for it. It caused this bug)

Sorry for the delay.
I just tested the issue again and confirmed that the cause explained by @DylanDylann is correct. API PayMoneyRequest response on the reportActions_ is returning a positive value on the amount. Aligning the optimistic data seems the correct solution for me. So the proposal from @DylanDylann looks good!
🎀 👀 🎀 C+ reviewed!

Awesome! Thanks @mollfpr
@Li357 can you confirm if this decision looks good? https://github.com/Expensify/App/issues/30856#issuecomment-1803743436

Taking a look now!

Agree with @DylanDylann's approach here, assigned! Thanks!

@mollfpr The PR is ready for review https://github.com/Expensify/App/pull/31292

I'm going ooo until Monday Nov 27th so going to assign another BZ teammate to monitor this until I'm back. Thanks!
status - reviewing PR

[@mollfpr] The PR that introduced the bug has been identified. Link to the PR:
[@mollfpr] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

I don't think we have an offending PR. This might be related to how the BE handles the params and returns it to FE with a negative value on request money.

[@mollfpr] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

The regression step should be enough.

[@mollfpr] Determine if we should create a regression test for this bug.
[@mollfpr] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Open the App and sign in
Go offline
Go to workspace chat > + > Request money
Create a manual request
Click Pay elsewhere
On the expense report, right-click on the system message for paid expense
Click Copy to the clipboard
Paste it on the composer
Verify that the correct paid amount is copied
👍 or 👎

@Christinadobrzyn Could you give the payment summary so I can do the manual request? Thank you!

Payouts due:
Issue Reporter: NA
Contributor: $500 @DylanDylann (in Upwork)
Contributor+: $500 @mollfpr (in NewExpensify)
Upwork job is here. Internal posting - https://www.upwork.com/ab/applicants/1720467002745327616/job-details

Paid @DylanDylann in Upwork based on this payment structure!  I think this can be closed!


Improve search

Before:

After:

@Tamba32 Can you update the screenshots to provide an example with the same search string and also comment on what issue this change addresses?

🚢


Fix Spotbugs in azure-core

This PR is lacking clarity on why this change was made. Without context, all I am seeing is that our logging will be slower now, and I don't know why. Future developers will not have any context, either in the code or in this PR. Please be sure to create PRs with proper descriptions of the intent of the change, and ideally include appropriate inline code comments about why things are being done the way they are.
All reviewers too - this should be baseline expectations to ensure we are not doing our future selves and teammates a disservice by under-documenting things.

This PR is lacking clarity on why this change was made. Without context, all I am seeing is that our logging will be slower now, and I don't know why. Future developers will not have any context, either in the code or in this PR. Please be sure to create PRs with proper descriptions of the intent of the change, and ideally include appropriate inline code comments about why things are being done the way they are.
All reviewers too - this should be baseline expectations to ensure we are not doing our future selves and teammates a disservice by under-documenting things.

The descriptions are limited intentionally.


Does not connecting to velocity

Im trying connect to velocity, but get "Disconnecting"
I enabled debug mode and get this:
[22:59:39 WARN] [net.tcpshield.tcpshield.velocity.TCPShieldVelocity]: Debug : unknown[unknown/ip] was disconnected because no proxy info was received and only-allow-proxy-connections is enabled. Raw payload = "SERVER IP"
[22:59:39 ERROR]: net.tcpshield.tcpshield.util.exception.phase.HandshakeException: An exception occured during the handshake process
[22:59:39 ERROR]:       at net.tcpshield.tcpshield.TCPShieldPacketHandler.handleHandshake(TCPShieldPacketHandler.java:163)
[22:59:39 ERROR]:       at net.tcpshield.tcpshield.velocity.handler.VelocityHandshakeHandler.handleEvent(VelocityHandshakeHandler.java:56)
[22:59:39 ERROR]:       at net.tcpshield.tcpshield.velocity.handler.VelocityHandshakeHandler.onProxyPing(VelocityHandshakeHandler.java:41)
[22:59:39 ERROR]:       at net.tcpshield.tcpshield.velocity.handler.Lmbda$45.execute(Unknown Source)
[22:59:39 ERROR]:       at com.velocitypowered.proxy.event.UntargetedEventHandler$VoidHandler.lambda$buildHandler$0(UntargetedEventHandler.java:47)
[22:59:39 ERROR]:       at com.velocitypowered.proxy.event.VelocityEventManager.fire(VelocityEventManager.java:598)
[22:59:39 ERROR]:       at com.velocitypowered.proxy.event.VelocityEventManager.lambda$fire$5(VelocityEventManager.java:479)
[22:59:39 ERROR]:       at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130)
[22:59:39 ERROR]:       at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630)
[22:59:39 ERROR]:       at java.base/java.lang.Thread.run(Thread.java:831)
[22:59:39 ERROR]: Caused by: net.tcpshield.tcpshield.util.exception.parse.InvalidPayloadException: length: 1, payload: [SERVER IP], raw payload: SERVER IP
[22:59:39 ERROR]:       at net.tcpshield.tcpshield.TCPShieldPacketHandler.handleHandshake(TCPShieldPacketHandler.java:100)
[22:59:39 ERROR]:       ... 9 more

You seem to be trying to directly connect with only-allow-proxy-connections enabled.

You seem to be trying to directly connect with only-allow-proxy-connections enabled.

I fixed this, I writes on discord support server and I forget to close issue) Sorry!


Disable intermittent lock tests

Purpose
Few lock tests are failing intermittently. Hence disabling for the moment and to be fixed later.
#22987 is created to track the disabled test fixes

Codecov Report

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

@@           Coverage Diff           @@
##           master   #22991   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

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 efcb688...c04b9d9. Read the comment docs.

Please don't merge this PR until #22993 #22995 pass.


Capacitor 3 Support

Is your feature request related to a problem? Please describe.
The Capacitor 3 RC is out! 🎉 This library should support the new version
Describe the solution you'd like
Follow the plugin upgrade guide here: https://capacitorjs.com/docs/v3/updating/plugins/3-0
Additional context
The migration is pretty easy, I've done it for a couple plugins. Happy to assist if you need assisting!

Yes, please!


pull request for e2e test at 01/10/2017 08:45:06

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Bug:  fails with no-such-method error when translation key is an integer

Description
fails with an exception when generating from translation file like this:
    errors:
      to_homepage: Back to homepage
      400: Invalid request
      404: Page not found

How to reproduce
run json generator
Software:

Gem version: 4.0.0.alpha2
Ruby version: 3.0.3

i am planning to fix this and submit PR

@honzasterba this actually an error on https://github.com/fnando/glob. I just fixed and released a new version. Can you please try and see if is there any other fix required?

thanks! I updated glob and now the error is gone.


UX: Lost on next steps because of UTF-8 error

What were you trying to do
Encrypt an .mp4 file with passphrase.
It worked with a different password so I guess the error is about the passphrase. I was not using anything that does not work in a normal password. Are there any requirements that weren't explicit?
What happened
Type passphrase (leave empty to autogenerate a secure one):
(typed a password)
Error: stream did not contain valid UTF-8

[ Did rage not do what you expected? Could an error be more useful? ]
[ Tell us: https://str4d.xyz/rage/report

Can you tell me the following:

Operating system
Keyboard layout
Terminal application
Whether what you typed included any non-Latin characters


cache served microformats2 responses

...with a short expiration. ideally include the cache expiration in the headers and/or content itself.
alternatively, serve them from the stored response JSON in the datastore. drawback is that we don't get updated content.

done, yay. oldest issue we've closed in a long time. :P
oh and for the record, serving from stored fesponses would be tricky since we always want to serve the current silo response, or 404/410 if it's missing, etc. this should usually still work with stored responses, since we update them during poll if they change, but only every poll period, so they could be stale, and we also may often miss updating them entirely right now since we don't see responses past the first page. that's #657.


Fixed broken links in the docs and in contributing.md

Fixed Reducing Boilerplate link in Recipes > Structuring Reducers > Refactoring Reducers Example
Fixed links to the examples page and a link to the docs in contributing.md

Thanks!

I actually need to figure out how to move the docs folder back to /docs.  I know we kept it that way with RTK's repo, and I missed that they got moved here as part of the DSv2 upgrade.


[UI] "Add Cluster" button to be always available

Current Behavior
Users who have one or more Kubernetes clusters connected sometimes wish to add another cluster under Meshery's management.  They can do so on the Connections page or that could do so under the Kubernetes context switcher in the upper right corner.
Users that have existing Kubernetes cluster connections are not shown the convenient option of the "Add Cluster" button that they will see when there are not clusters connected.
Desired Behavior
As a matter of convenience, it would be nice if the "Add Cluster" button was ever-present.
Acceptance Tests
Whether a user has zero or more Kubernetes cluster connections, the Add Cluster button will be available.
Mockups

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Hey @winkletinkle can you assign this issue to me ?

@winkletinkle can you assign this issue to me

@winkletinkle can you assign this issue to me

Verify that the "Add Cluster" button is displayed both when there are no clusters connected and when one or more clusters are connected.
Ensure the UI consistently shows the "Add Cluster" button in relevant sections, such as the Kubernetes context switcher.


add vuex-persist

@kazupon done !

@nickmessing  done !
Please squash and merge, as it has become 3 commits now.

@championswimmer, that's what we always do ;)

@kazupon, any objections?


Different results in playground vs standalone REST client

I got the correct using Insomnia, but not using the playground client. Tried using existing database and following the steps outlined below.
This issue pertains to:

[ ] GraphQL Playground - Electron App
[x] GraphQL Playground

What is the expected behavior?
{
  "data": {
    "loggedInUser": {
      "id": "cizjh26dujm7f0190j1jovkhb"
    }
  }
}

What is the actual behavior?
{
  "data": {
    "loggedInUser": null
  }
}

What steps may we take to reproduce the behavior?

yarn add global graphcool
mkdir graphcool-test && cd $_
graphcool init
graphcool add-template graphcool/templates/auth/email-password
Uncomment the appropriates lines in the files edited by 3
graphcool deploy
Navigate to your Simple API url in the browser
Use the signupUser mutation which returns a token
Use the token as a Header with the name Authorization
Use the loggedInUser query

Did you use Bearer in front of the token? Also, can you check the network tab and grab the actual request being executed?

That was the issue, cheers... Not sure how I missed that.