data table update columns to include protected APR

in our efforts to show protected APR, we would like to update the data table and include 2 new columns.
protected apr
unprotected apr
removing "fees" column as all pools should have the same fee setup (as of bip 4).
https://www.figma.com/file/3yED7o7TgoLwoEjV2EkRLL/Bancor-V2?node-id=1072%3A0
tool tips: (@nhindman23 )

protected apr
unprotected apr

Let's discuss this one @ashachaf

please note, we would like to keep the "fee" column.

What is Protected APR and Unprotected APR and how do we calculate that?
Is @nhindman23 going to post the tooltips then?

protected APR (we will calculate it based on 7days)
https://docs.google.com/document/d/1sE1AMPD7wfz1_kgb3jxEabNOUheTB2TfG1VCdid7teo/edit#heading=h.70bpehp3em80
(we have already done this for the data table and hide it, average of both reserves)
unprotected APR:

calculate the worth of pool token in primary and secondary reserve token today
calculate the worth of pool token in primary and secondary reserve token today-7days
take average {(primary value 1/primary value2 -1)10052} and {(secondary value 1/secondary value2 -1)10052}

how to calculate pool token to reserve amount?
pool token amount * ((primary reserve balance at the time of check)/(pool token supply at the time of check)) = primary reserve amount that is worth the pool token you are checking.

tool tips:
protected APR: 7d fees/liquidity annualized with Bancor’s impermanent loss protection
unprotected APR: 7d fees/liquidity annualized without Bancor’s impermanent loss protection

related to #558

this task is on hold for now

this task is on hold for now

doing other design updates to the table.
will revisit when relevant.


[Unity][CI] Update gpu and lint image

I tried to update the GPU image for unity CI in https://github.com/apache/tvm/pull/14291 but it doesn't seem to be working. Trying sending the change from apache and also including the recent lint image update https://github.com/apache/tvm/pull/14373. Not sure if this is the correct protocol for updating CI images for unity.
@tqchen @driazati

indeed a branch in apache would be the way to go, normally we use unity-staging for that


Add system install test for Amazon Linux

Supersedes https://github.com/astral-sh/uv/pull/1534

@charliermarsh this looks legitimately wrong? https://github.com/astral-sh/uv/actions/runs/8259113036/job/22592520139

Not necessarily. It depends on what sys.path is. Where does pip install into? It's possible that that's the right place, and the directory just doesn't exist yet. (I genuinely don't know, I'm just saying it's possible we're wrong to assume the directory has to exist in advance.)

In CentOS (I just have the image around locally):
[root@bf9d80fad201 /]# python3 -c "import sys; print(sys.path)"
['', '/usr/lib64/python39.zip', '/usr/lib64/python3.9', '/usr/lib64/python3.9/lib-dynload', '/usr/lib64/python3.9/site-packages', '/usr/lib/python3.9/site-packages']
[root@d4f5af1c6ac6 /]# ls /usr/local/lib/python3.9/site-packages
ls: cannot access '/usr/local/lib/python3.9/site-packages': No such file or directory
[root@d4f5af1c6ac6 /]# python3 -m pip list
Package    Version
---------- -------
pip        20.2.4
setuptools 50.3.2
[root@bf9d80fad201 /]# python3 -m pip install pylint
WARNING: Running pip install with root privileges is generally not a good idea. Try `python3 -m pip install --user` instead.
 Although why isn't /usr/local/lib/python3.9/site-packages on sys.path?

Oh yeah I read /usr/lib/python3.9/site-packages as /usr/local/... umm I'm not sure :)

Can you check that import pylint or whatever works in that Python?

Or you can share the Dockerfile and I can debug it.

[root@d4f5af1c6ac6 /]# python3 -c "import pylint" works
I'm just copy-pasting the setup commands for CentOS from the CI workflow

Interestingly now the sys.path includes it
[root@d4f5af1c6ac6 /]# python3 -c "import sys; print(sys.path)"
['', '/usr/lib64/python39.zip', '/usr/lib64/python3.9', '/usr/lib64/python3.9/lib-dynload', '/usr/local/lib/python3.9/site-packages', '/usr/lib64/python3.9/site-packages', '/usr/lib/python3.9/site-packages']

It gets added when you pip install something :)

Or, more concretely, when it exists
[root@f381759817d2 /]# python3 -c "import sys; print(sys.path)"
['', '/usr/lib64/python39.zip', '/usr/lib64/python3.9', '/usr/lib64/python3.9/lib-dynload', '/usr/lib64/python3.9/site-packages', '/usr/lib/python3.9/site-packages']
[root@f381759817d2 /]# mkdir -p '/usr/local/lib/python3.9/site-packages'
[root@f381759817d2 /]# python3 -c "import sys; print(sys.path)"
['', '/usr/lib64/python39.zip', '/usr/lib64/python3.9', '/usr/lib64/python3.9/lib-dynload', '/usr/local/lib/python3.9/site-packages', '/usr/lib64/python3.9/site-packages', '/usr/lib/python3.9/site-packages']

Done in #2413 instead

Thank you!


ConfigTree.php has PHP error

ConfigTree.php is throwing PHP Parse error:  syntax error, unexpected '.' line 20. This fixes it.

Thanks, @arthur24b6


[Bug] Example DB's overlap when user screen size is reduced

Description
Example DB's under Datasources overlap when screen size is reduced
Steps to reproduce the behaviour:

Go to Datasources and open up browser console
Increase console width and observe example DB's overlap with Databases

Important Details

Version: Cloud
OS: Win10
Browser: Chrome
Environment: Release

The Integrations grid needs to be responsive to different screen sizes.
Look at NewIntegrationsWrapper and MockDatasourceWrapper to start debugging the problem
Please read the contribution guidelines before working on this issue

Hi @hetunandu, would love to work on this.

It's yours @JubayerJoy. You can pick this up.
Please take note of the info Hetu has mentioned above.

Hi @ramsaptami,
can't reproduce this error. 😕
My Env is the same as yours.
Was this issue fixed?

Hey @JubayerJoy, thanks for reaching out.
You'll need to have at least one datasource present on your application since we currently list mock DB's on the top for a fresh application. So for our mock datasources to appear at the bottom, please add any datasource and then try to reproducing the case.
Just tried to reproduce this on my local and here's the screenshot of the overlap issue. Please do let me know if you need more information.

@ramsaptami Thanks for guiding,
Yap, was able to reproduce this. 👍

Will open a PR after solving this.
cheers 🎈


Offline mode

npm wizards, we need your help!
Per @pselle’s suggestion, we should try to find a way to make create-react-app work offline.
I’m not well versed in npm flags and likely won’t have time for this, so this is a perfect opportunity for you to contribute and significantly improve the user experience of Create React App.
Here is the behavior we’re looking for:

Existing requirement: create-react-app my-app should use the most recent version of react-scripts if you are online. We are not changing this: we always want the users to get the most recent one.
New requirement: create-react-app my-app should not crash offline if you ran it at least once before. It should be able to use locally cached react-scripts in this case, if it exists.
Bonus: create-react-app my-app should use cached react-scripts if it knows the latest npm version is cached locally. In other words, even if you are online, we shouldn’t just refetch the whole thing if we have it locally and we know it is the latest version. This is not required but would make a big difference for most people.
Acceptable tradeoff: we are fine with not installing the latest transitive dependencies. In other words it’s fine if react-scripts depends on<EMAIL_ADDRESS>then<EMAIL_ADDRESS>comes out, but we keep using the cached<EMAIL_ADDRESS>
How to achieve this? I have no idea.
Maybe this would be helpful: https://addyosmani.com/blog/using-npm-offline/.
We already use bundledDependencies (which is a bit unusual) as part of our build process so in reality react-scripts is being downloaded as a single tarball rather than through the usual npm process. Perhaps this can give us the leverage to make this better (e.g. by caching that tarball somehow).

I'm not sure how this fits with bundledDependencies but in npm >= 3.9 --cache-min=999999999 is pretty much exactly what you're looking for https://github.com/npm/npm/releases/tag/v3.9.0

I’m concerned about issues like this: https://github.com/npm/npm/issues/8581. If the solution involves --cache-min, and --cache-min causes weird crashes in some conditions, we must:

Understand why this happens and whether our users will be affected by this
Be resistant to such failures (e.g. by retrying npm install without that flag if it fails)

I believe that issue is exactly what 3.9 fixes. You could detect npm version and avoid the flag if running older than 3.9? I'd be happy to do a pr, is this the right call to modify?

Yes, this is the right call. What you suggest seems reasonable, please go for it!
It would also help if you could help me understand how to reproduce exactly that issue in < 3.9 so we can verify 3.9 indeed fixes it in our case.

To reproduce you need a package a that depends on<EMAIL_ADDRESS>where a is not in the cache, and<EMAIL_ADDRESS>is in the cache. Then release<EMAIL_ADDRESS>and try to install a with ---cache-min=99999999. Instead of downloading the new version of b, npm will only try versions from the cache, and fail because there is no compatible version.
(Hope that helps, it's not a trivial issue...)

Yeah, I meant this in the context of react-scripts.
If you submitted a PR, I’d appreciate instructions to reproduce this exact issue with react-scripts just so I can verify that it was broken before 3.9 but fixed in it.

Oh, right, sorry. The user would have to have an old version of one of the dependencies of react-scripts in cache but not react-scripts itself. So if they had e.g<EMAIL_ADDRESS>in cache and tried to run npm install --cache-min=Infinity react-scripts (which depends on<EMAIL_ADDRESS>it would hit the issue.

Thanks for explanation! I’ll try reproing that if you submit a PR. 😉

Just double checking I'm editing global-cli/index.js? That's a pretty scary warning 😉

Yea, it’s meant to prevent accidental breaking changes since people almost never have to update the CLI, and since old CLIs must always work with new react-scripts even after breaking changes in react-scripts. Your change is safe tho.

Did anyone get to this issue? Does create-react-app work properly offline?

No but if you pick up https://github.com/facebookincubator/create-react-app/pull/375 and make it pass Travis, it could be in next version :-)

I could give it a try, seems like the ogher guy has been busy.  It's a matter of making travis tests pass?  Does this commit meet all the functional requirements?

I don’t think it meets https://github.com/facebookincubator/create-react-app/pull/375#issuecomment-237909277.

Are you doing this, @gafemoyano? If not, I'd like to try to resolve this as well.

We might have another solution to this soon.

Sorry I haven't been able to figure it out cause... well I don't have reliable internet access. For now I've just been keeping a clean project generated with the latest version and copy/pastying it when I need to start a new app. I know it's not really a solution to this, but it's good enough considering that upgrading between CRA versions hasn't been difficult at all.

Okay, check it out.
We should integrate with Yarn if it exists in the system:
https://github.com/yarnpkg/yarn
https://yarnpkg.com/
https://code.facebook.com/posts/1840075619545360
It supports offline mode.

Oh wow this came out literally  45 minutes ago, congrats!!
So will yarn be bundled with CRA? What happens if Yarn isn't in the system, should we still try to use npm's cache?

Yarn won't be bundled with CRA. We should change the CLI to detect if Yarn is present. If it is, we should install with Yarn, and offline mode will "just work". If not, we'll fall back to npm with current behavior.

Cool, I'll read up about Yarn and report back if I can handle the integration, unless @lukyth  wants to get started right away.

@gafemoyano It's fine. I just thought if you didn't want to do this anymore, I'd love to give it a try as well, since it's been sometimes since your last reply.

Please comment in https://github.com/facebookincubator/create-react-app/issues/896 if you work on this.

yarn supports offline mode only when the flag --offline is added, so we need to check if yarn failed and if it did, install with the flag. Probably should check the error to see exactly what failed.

Not sure what the status on this or on #896,i would love to work on this if possible

@chiptus since that issue is closed, I believe any discussion can be held here. Just please post a comment in #896 that you want to work on this and link back to this issue. We love contributions!

@Timer  it seems like #1423 is exactly this. I guess I will look for another issue to contribute to :)

Yea, https://github.com/facebookincubator/create-react-app/pull/1423 just fixed this!
Will be out in the next version which we plan to publish today.


Fix exported type resolution in commonjs

It is fine to resolve the types of exported classes in ES6:
export class C {
}
var c = new C()

But not for commonjs exported classes:
module.exports.C = class {
}
var c = new C() // should error

JSDoc type aliases are an exception since they are always exported implicitly, not by [property] assignment to module.exports or exports.
Fixes #24492

@mhegazy do you want this in 2.9 too?

Please port to release-2.9 as well.

OK, it's in 2.9 now too.


How to change contentView backgroundColor

I want to change contentView background color, but not cover a new view set background color
follow code not work
<PopupDialog
            style={{backgroundColor: 'transparent'}}
            ref={(popupDialog) => { this.popupDialog = popupDialog;}}
            width={Dimensions.get('window').width}
            height={Dimensions.get('window').height}
            haveTitleBar={false}
            dialogAnimation = { new SlideAnimation({ slideFrom: 'bottom' }) }
            onClosed={()=>{Actions.pop()}}
            >
              {this._popupView()}
            </PopupDialog>

The contentView still white background color

@bawn
You can pass a <View /> as the contentView that's mean you can set any content & any style  on the <View />. For example:
<PopupDialog
  ref={(popupDialog) => { this.popupDialog = popupDialog; }}
>
  <View
    style={{
      backgroundColor: 'red',
    }}
  >
    <Text>Hello</Text>
  </View>
</PopupDialog>

@jacklam718 Thanks, I mean i just to want change PopupDialog background color,  but not cover a new view set background color

@bawn
I see, I understand what you mean is.
You want to pass style to change the popupDialog style, is it right?

@jacklam718 YES

@bawn
Sorry it's not yet support 😔 I will make it support.
I will let you know when I updated.
Thanks for reporting this issue.

@bawn
Updated.
Please install the latest: 0.2.13 if any problems please let me know.

@jacklam718 Thanks

@jacklam718
Still can not change the background color
<PopupDialog
            style={{backgroundColor: 'red'}}
            ref={(popupDialog) => { this.popupDialog = popupDialog;}}
            width={Dimensions.get('window').width}
            height={Dimensions.get('window').height}
            haveTitleBar={false}
            dialogAnimation = { new SlideAnimation({ slideFrom: 'bottom' }) }
            onClosed={()=>{Actions.pop()}}
            >

            </PopupDialog>

@bawn the prop is dialogStyle rather than  style

@jacklam718 Thanks, It's worked


Update dataset.csv

Is this the right place to fix a typo that's surfacing in front end?
@psd or @mattlucht

Yes, it should be!


Add Basic Example

This changeset adds the most basic example for the crate.

Here is an example run on my fork: https://github.com/hendrikmaus/prefix_writer/actions/runs/1520462447
I reckon the workflows won't execute here unless you authorize them.


Influx write timeout on raspberry pi 4

InfluxDB version: 1.8.1
InfluxDB-python version: 5.3.0
Python version: 3.7.3
Operating system version: Raspbian GNU/Linux 10 (buster)

I am running both influxDB server and client on raspberry pi 4 (2GB Ram). My write query is timing out in every minute. Please let me know if you need any logs or any other information.

Same Issue. Happens 3x a day.


Implement organization of outputs

The extraction of visuals from PDF files should organize the outputs as follow:

[x] Name the directory for each thesis work using the repository-id
[x] Name each extracted visual by page and number, for example: p1-img_1
[x] Include the relative path to the directory as part of the thesis metadata

completed


Fixme fix

Coverage remained the same at 100.0% when pulling 2b4c68dbcc30e124af186dfe7a11e3c1b6b15c5a on fixme-fix into 47fa0fc16c587d6bd43ed7b5d334ec8cbcbcf6e8 on master.

Coverage remained the same at 100.0% when pulling c2f7b96048ef1d960b6a1edb6f6c4ae0b2dc4d99 on fixme-fix into 47fa0fc16c587d6bd43ed7b5d334ec8cbcbcf6e8 on master.


Fix typo in Ko

I found a typo in Ko, and fix it.
In English, it's like changing 'pplication' to 'application'.

Seems reasonable to me, thanks for the fix!


Can't Install Latest Version on an M1 Mac

I get this error when I try installing this on my mac.
Collecting unrar2-cffi==0.3.0
  Using cached unrar2-cffi-0.3.0.tar.gz (258 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [15 lines of output]
      Traceback (most recent call last):
        File "/Users/temp/.pyenv/versions/3.10.13/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/Users/temp/.pyenv/versions/3.10.13/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
        File "/Users/temp/.pyenv/versions/3.10.13/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
        File "/private/var/folders/hm/4jgkd02j2vx59g7v9sl7d1qr0000gn/T/pip-build-env-v8zmpzxv/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 325, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=['wheel'])
        File "/private/var/folders/hm/4jgkd02j2vx59g7v9sl7d1qr0000gn/T/pip-build-env-v8zmpzxv/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 295, in _get_build_requires
          self.run_setup()
        File "/private/var/folders/hm/4jgkd02j2vx59g7v9sl7d1qr0000gn/T/pip-build-env-v8zmpzxv/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 311, in run_setup
          exec(code, locals())
        File "<string>", line 12, in <module>
      ModuleNotFoundError: No module named 'buildconf'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

oh, i might need to fix unrar2 build again

Fixed it, currently waiting for CI runner https://github.com/noaione/unrar2-cffi/actions/runs/8369372376 to publish to pypi then I can bump it here.
I don't know if the ARM artifact for macOS would work though lol.

Should be fixed? Try the latest commit @mslmn

Sidenote, was the Oshot option removed from releases?

There's no option since it use filename to determine it.

Title - v01 - p000.jpg -> v01
Title - p000.jpg -> OShot/Oneshot

Try the latest commit https://github.com/noaione/nao-manga-rls/commit/61a4e1b01fe2019921b5f7aeb8da91aaaa16972b


Take up too much GPU memory

Neural-style python2.7 ubuntu ,with GPU0 . when training ,is about 1.8G GPU memory took up;
(with the default parameters)
So,what things can I do to reduce the GPU memory . ?

Try to set MXNET_BACKWARD_DO_MIRROR to 1. https://github.com/dmlc/mxnet/blob/2413a8ed421b5a917eed544b08d434839b8c812d/docs/how_to/env_var.md

Can I change the optimizer to Adam?  How can I do that in mxnet.when i use MXNET_BACKWARD_DO_MIRROR,It still need 1.7G GPUmemory. (but in tensorflow just 1.0G)~thanks

.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Remove Currency quick_withdraw_limit method (closes #692)

@yssavchuk Please, fix this:

Ok. It looks good but we need QA it.

@yivo I've tested looks like everything is okay


Add effective giving section, and Raise

Add "Effective Giving" section, branch of EA described in https://forum.effectivealtruism.org/topics/effective-giving?tab=wiki
Add 'Raise' to the "Effective Giving" section.

[x] Table of contents has been updated (if needed)
[x] Contents have been sorted alphabetically

Would it make sense to add the EA Market Testing Team to the "Effective Giving" section? (See: bit.ly/eamtt)


tests: ensure that we are deleting and seeding

📲 What
Delete and seed the database.

🤔 Why
Because data, and tests. But seriously, ensuring state before tests are run.
🛠 How
Interacting with the API, not with the Cosmos DB instance itself.
👀 Evidence
🕵 How to test
npm run test:e2e
✅ Acceptance criteria Checklist

[ ] Code peer reviewed?
[ ] Documentation has been updated to reflect the changes?
[ ] Passing all automated tests, including a successful deployment?
[ ] Passing any exploratory testing?
[ ] Rebased/merged with latest changes from development and re-tested?
[ ] Meeting the Coding Standards?

Woot thanks Ester! 🙌🏻


[Admin] Support running Emission from a local packager

works with https://github.com/artsy/emission/pull/338

So no other React subspecs were needed after all?

All subspecs are merged post-resolve, so Emission/AppHub bring their own subspecs along for the ride

I see. Ok, I can get with this 👍

Amazing work @orta !! This is going to make certain things so much easier!


Updated CSV and container values for new SG

This is the final step to making a new SGO release that support the new collectd metrics SmartGateway code.

I've tested this from source, but would like to retest with a full set of upstream builds.

 Produce a new SGO release that includes the new SG and bridge code

 Fix performance problems with NEW SG code

[csibbitt@laptop smart-gateway-operator (csibbitt-944-new-csv-for-v2.0.0) (⎈ |sgo-scorecard/api-stfsg2-lab-upshift-rdu2-redhat-com:6443/kube:admin)]$ operator-sdk scorecard
INFO[0000] Using config file: /home/csibbitt/cloudops/smart-gateway-operator/.osdk-scorecard.yaml
basic:
        Spec Block Exists                  : pass
        Status Block Exists                : pass
        Writing into CRs has an effect     : pass
olm:
        Spec fields with descriptors       : pass
        Status fields with descriptors     : pass
        Provided APIs have validation      : pass
        Owned CRDs have resources listed   : pass
        CRs have at least 1 example        : pass


[WIP] update should support limit 1 sub query

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1255/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1265/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1276/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1256/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1266/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1277/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1265/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1275/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1286/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1269/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1279/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1290/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/1329/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/1321/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/1344/

LGTM


Cable Status sensor is unreliable since last update

I had based all of my charging automations off this sensor because it was the quickest to update and most reliable.
Now this sensor can go an entire charging session without changing state from "unplugged" even if status is "charging" and it sais it is connected to the cloud

I agree the sensor is now unreliable, constantly changing it's status.

I will take a look and try to recreate, I've not seen this on my pod. Its coming into the school half term this week so it will be unlikely that I will be able to take a look until next week. Was there a certain time that it started happening? Did you restart your Home Assistant instance before seeing it?
Anything to help me re-create it would be awesome

Hi,
I have restarted home assistant a number of times and the problem continues. I am not sure how to recreate it.
Completely understand about half term. For me this issue has been on going for a while and not particularly problematic so not urgent.
Sorry I cannot be more helpful.

I need to clarify. The sensor also reports "connected" when it is not actually connected some times too. It is totally useless at this state.
This started happening after the first update that made PSL status report "pending" and the other new states.
I hope this helps

I also have this issue.

Another pod point with this issue. Mine in combination with a eCorsa, which only updates it's cable sensor when Stellantis can be arsed.

It seems to me, that what is happening here is that when you switch  "Charging Allowed" the cable status immediately changes to "unplugged" regardless of the actual plug status (is it renegotiating with the car?), and stays that way for about 5 minutes.
I use predbat to plan my carging slots, it refreshes every 5 minutes, so i was getting stuck in a doom loop of charging / not charging creating the behaviour you see above.
As a quick nasty fix, I created a helper that toggles according to the initial plug status, and then toggles off if the cable is reported unplugged for more than 30 minutes, and updated predbat to make decisions based on the smoothed sensor. It works, but feels nasty.

Final update:
This confirms it, I think:

Everytime "Charging Allowed" is switched, Cable Status is reset for +/- 5 Minutes. If you are doing your charging programmatically, you'll need to make a sensor to smooth it out.


Update Heroku Element Readme.md

Thank you for this buildpack. It works great with wicked_pdf when other buildpacks failed!
Your Heroku Elements Readme.md page does not include the "Changelog v3" description indicating this buildpack works with heroku-22, so I almost skipped testing it.
Please update your Heroku Elements Readme.md page if possible, helping others select your buildpack over the other non-working buildpacks.
Also, your GitHub "About" reading for this buildpack states "...and works on heroku-20 and heroku-18 stacks." could also be updated to include heroku-22.
Thanks again!

Thank you for your feedback and kind words! I am glad to hear that the buildpack worked well for you with wicked_pdf.
I was not aware of the Heroku elements page. I must have forgotten. And I don't know how to edit it. In case you are aware of how to change it you can contact me. Apart from that the GitHub readme and the about section have been updated.
Thanks for bringing this to my attention, and I really appreciate your support for this buildpack. I built it because of the exact same reason. If you have any other questions or concerns, please don't hesitate to reach out to us.


http: minor code clean up to the http filter manager

Commit Message: http: minor code clean up to the http filter manager
Additional Description:

simplify the code and remove unnecessary flag.
resolve clang tidy warning.

Risk Level: low.
Testing: n/a.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

/assign-from @envoyproxy/senior-maintainers

My understanding is that this will make some of the callbacks be called twice now for filters that are both an encoder and decoder. Was that protection unnecessary?

Long time ago, we only have decoder_filters_ and encoder_filters, this protection is necessary. But at #15172, we introduce a new containers to store the StreamFilterBase pointer for every unique filter. We can iterate the that container directly and needn't any additional branch.

Can you elaborate on the reasoning for this change?

I want to optimize the implementation of filter manager and reduce the scattered memory allocations of http filter chain creation. And this is a minor pre-PR to simplify that optimization PR.


Lowercase Z is not w

The keyboard example has 'Z' become 'w' as lowercase, instead of 'z'.
I believe it's from this line: https://github.com/felixmariotto/three-mesh-ui/blob/master/src/utils/Keymaps.js#L148

Thanks for pointing this out !
It's fixed by version 3.0.3


Allow Optional Header in Wizard Component

Some workflows do not require the header to display on every step of the wizard. Since each step dynamically changes what displays in the primary section of the wizard, the header does not always need to be present.
This issue is being logged to make allow for an option to disable the header in the wizard component.

@mmocniak-rh I feel like the Wizard title is important for the reasons @jeperry stated. It reinforces that intended outcome of the wizard. I agree that our design guidelines could be better and we should clarify what these fields are used for and how to reflect the current step as @jeperry suggests. I don't want to make the header optional, but I do have one question for you @mcoker . Description is optional, I believe. If a consumer of this chooses not to add a description, is the space collapsed or does that space just remain blank? If the later, we might want to make a change to reduce the space that the header consumes. Thoughts?

@mcarrano since it's optional, we balance out the layout when the description isn't present.

That's great @mcoker .

@mmocniak-rh thanks for submitting this, but after discussion with @mcoker and @kybaker we have determined that the Wizard header should not be optional and this issue will be closed. It is important that the Wizard always have a title to identify the action that they will take. Being able to hide the description as shown above helps compress the space required. Also note that the height and width of the wizard modal can be adjusted if you need to create more space for content.
A related documentation issue has been created to provide better guidance about this (https://github.com/patternfly/patternfly-org/issues/1454).


FOR DISCUSSION generic format string for parameters

As you can see from this PR some tasks allowed you to do things like:
brigade.run(commands.command,
            command="echo {host.name}")

Problem is that it was quite inconsistent. Some supported it while others didn't. So I started working on a generic solution for this and although it works I really don't like it as I think it's fragile and it's bound to break things eventually. I already had to handle ValueError exception because in one of my tests it was trying to format a junos configuration leading to an error...
So I was thinking in completely remove this functionality and change the pattern above for:
def echo_hostname(task):
    task.run(commands.command,
             command="echo {}".format(host.name))

brigade.run(echo_hostname)

Thoughts? Should we encourage this pattern or should we try to keep the old one (although inconsistent we can add as necessary) or should we try to make this generic mechanism for resilient? Maybe if we pass it to a jinja2 template it will be better? I think I personally prefer the echo_hostname pattern.

I was also thinking we could do something like this:
format_params = ["path", "src", "dst", (other "standard" parameters)]
params = {}
for k, v in self.params.items():
    if k in format_params:
            params[k] = format_string(v, self, **self.host)
    else:
        params[k] = v

Yeah, I don't like this.
brigade.run(commands.command,
            command="echo {host.name}")

I would be worried about the issues you referenced in your original discussion i.e. how hard it would be to make it generally reliable.
I am fine with your def echo_hostname(task): solution.
If we are going to do this, then maybe Jinja2, but I would be worried we were headed down the road of some of things I strongly dislike in Ansible.
So the key issue to clarify to people (assuming that I understand it properly) is when would inventory variables be available to use and what is the proper way to create tasks that need to reference inventory variables.

I am going to do some more experiments here.

Okay, I tested this some more both by putting the format_string directly into netmiko_send_config and by doing this:
def echo_hostname(task):
    return task.run(netmiko_send_config, config_commands=['hostname {}'.format(task.host.name)])

brigade = Brigade(inventory=SimpleInventory("hosts.yml", "groups.yml"), dry_run=True)
b = brigade.filter(group='cisco-ios')

result = b.run(task=netmiko_connection, num_workers=60)
result = b.run(echo_hostname, num_workers=60)
for k, v in result.items():
    print(v.result)

I probably prefer this pattern...but definitely trade-offs both ways.
I might try to experiment some more with it.

Can also do this:
brigade = Brigade(inventory=SimpleInventory("hosts.yml", "groups.yml"), dry_run=True)
b = brigade.filter(group='cisco-ios')

b.run(task=netmiko_connection, num_workers=60)
result = b.run(lambda t: t.run(netmiko_send_config,
                                               config_commands=['hostname {}'.format(t.host.name)]))
for k, v in result.items():
    print(v.result)

Yeah, there are many things we can do. However, I am starting to think what I proposed (https://github.com/napalm-automation/brigade/pull/53#issuecomment-353345543) is really convenient without breaking anything like this PR does. In any case, I have been working during the past week in some "real life" examples to illustrate this and other improvements. I will send a PR soon for broader discussion.


Working Profile Image Upload

Shoutout to Phillip for writing a great imgbb function!
This PR saw changes to app.py, profile.html, and _template.html, and the deletion of the temporary profile_pic static image folder.

The changes to app.py involved updating the image-saving code.
profile.html saw the source of the image change to imgbb instead of the local.
_template.html had the same change as profile.html, for the mini profile picture.
The profile_pic static folder was deleted.

I have switched to the current version of the function. Everything is working as expected.


Format About Page - Proposed fix Cybros/Cybros-Web-Application#30

I added a context for the partial footer call to hide the "about" section in the footer on the about page.
Also I added the class ".inner" to the whole about container to set a max-width.
Since I did not know what you mean by "links to coordinator's account" I did not include that one.

Thanks for the PR.
Let's wait for others to review this.

@davidbru  Thanks for contributing, Your code seems fine so I am merging it to the master, meanwhile you can go through other issues or find a new issue and help us resolve it. Happy coding. :)


Fixing a bug with path on firebase-list

There were some missing underscores on the path member of the class. Therefore it was being discarded. Also added the ability to define the path on construction (it makes the code cleaner on other places)

@franciscofsales thanks for the PR 👍


HidHide hides controller from DS4Windows if it points to DS4Windows linked through a directory junction

[x] I have searched open and closed issues for duplicates
[x] I am submitting a bug report for existing functionality that does not work as intended
[x] I have read https://vigem.org/Community-Support/
[x] This isn't a feature request or a discussion topic

Bug description
DS4Windows doesn't detect the controller if I add the application from the directory junction, meanwhile it works if I add it from the real directory
Steps to reproduce

Install ds4windows from Scoop.
Add the DS4Windows executable to the application whitelist from the directory junction (current)
Hide the controller
Open DS4Windows

Actual result: The controller is not detected in DS4Windows
Expected result: It is supposed to access the hidden controller.
Screenshots
Real directory:

Junction:

Machine info

CPU-Architecture: x64
Windows version: Windows 11 21H2
Software/driver version(s): 1.2.98
Additional Information
From https://forums.vigem.org/topic/579/hidhide-hides-controller-from-ds4windows-if-it-points-to-ds4windows-linked-through-a-directory-junction/3

The build I'm testing it on the screenshot is from https://github.com/CircumSpector/DS4Windows, maybe it is something you can look at?


Add method to check node repo for oldest installed version, for compi…

…le version
@bratseth please review.

Ah, commit message slightly wrong. Should also include:

and use that method for determining compile version (guess at version used to compile).


Last attempt with Logo and fonts

It takes really too long, we need to move on. Let's spend last one hour and try to come up with something that will stay next half a year or maybe a year.

Enough is enough. I'm not going to touch styles until I have at least 1K subscribers on YouTube :)))) For the family circle current design will be acceptable.


[AAP-5378] initial nginx config

initial nginx config added to both docker-compose and minikube

Launching additional containers is broken due to permissions on the docker unix socket:
Object { status_code: 500, detail: "DockerError(900, 'Cannot connect to Docker Engine via unix:///run/docker.sock [Cannot connect to unix socket /run/docker.sock ssl:default [Permission denied]]')", headers: null }

Websockets work for me in dev now.

I'll fix the container spawning problem in another PR.


Errors during build again

The issue https://github.com/millsp/ts-toolbelt/issues/125 valid again

I think this is a different issue. You need to upgrade to TypeScript 3.7.

Please let me know if you still have the same issues after the upgrade.


a11y: add aria-required on required fields

add aria-required on required fields and
add aria-described-by on error message to associate the field to the error message

@giuliaghisini  Can you please help me with that issue , I thought issue is regarding aria-required and aria-described in normal html tags , but this issue is something else , so please help me toh get there.

@giuliaghisini Can you please help me with that issue , I thought issue is regarding aria-required and aria-described in normal html tags , but this issue is something else , so please help me to get there.

When you are editing a content-type, for exaple, and you have some required fields in form, they doesn't have aria-required="true" attribute if it's a required field, and in case of some validation error, the error message isn't linked to the field, so the field has to have aria-described-by="id_of_the_wrapper_error_field_description" to enable screen reader to read error message for that field.
This attributes are required for accessibility of the interface.

Thanks @giuliaghisini  I will work on it.

I guess this is still not worked on? can i pick this one @giuliaghisini

@praveen-rikhari please read and follow First-time contributors, especially Things not to do, Contributing to Plone, and Contributing to Volto.


NUM_INTERRUPT parameter comments dont match CLIC Interupt ID ordering recommendations

Parameter section:
NUM_INTERRUPT  4-4096                          Always has MSIP, MTIP, MEIP, CSIP
doesn't match CLIC Interrupt ID ordering recommendations which has a recommendation of only 2 interrupts in M-mode only.
From Allen Baum email:
Always has MSIP, MTIP, MEIP, CSIP  <-- SSIP ?
We define almost all of these parameters in riscv-config by describing the RW properties of the appropriate CSRs (e.g. which bits are RW vs RdOnlyOne vs RdOnlyZero.
The config-struct may do this differently; we need to be able to translate back and forth.

closed with pull #216


Add missing rosdep dependencies

Add missing dependencies that should be installed using rosdep in order to compile correctly the ROS package.

Hi @jonazpiazu
Thank you for the PR.

I'm sorry I didn't see this.  I just created another PR for this issue: https://github.com/IntelRealSense/librealsense/pull/5523  Maybe if we join in bugging them they'll merge.  I added CI tests to try to defend this from braking in the future.

Closed as addressed in #5523


Another LateInitializationError issue even after implementing 'return Sizer'

Hello, I've checked many answers on here regarding this issue but I havent been able to solve it  :<
class MyApp extends StatelessWidget {
const MyApp({Key? key}) : super(key: key);
@override
Widget build(BuildContext context) {
return Sizer(
builder: (context, orientation, deviceType){
return GetMaterialApp(
navigatorKey: Get.key,
initialRoute: '/home',
getPages: [
GetPage(
name: '/home',
page: () => HomePage(),
binding: HomeBinding(),
),
],
);
},
);
}
}
which is run on Main with
void main() {
runApp(const MyApp());
}
Any help is very appreciated, Im looking forward to use this plugin :D

Also facing this problem, some solution?

In your main function try adding these lines:
WidgetsFlutterBinding.ensureInitialized();   await Future.delayed(const Duration(milliseconds: 200));


WIP: Added feature flag for ephemeral server module

What was changed
Added a feature flag for ephemeral server functionality.
Made it default to not break the existing usage.
Why?
To optimize compilation time for pure Rust usage of SDK.
Checklist

Closes #559

How was this tested:
Run the existing unit/integration/load tests,

Any docs updates needed?
No

Actually, this is not working,
https://github.com/temporalio/sdk-core/issues/559#issuecomment-1563586285

Closing this as alternative PR https://github.com/temporalio/sdk-core/pull/561 is agreed upon.


How to get the serialized version of the session cookie in 0.10?

In version 0.9 I could simply get the value of the session like this:
flask.session.serialize()

But after upgrading to 0.10 this is not working anymore.. what's the alternative? How can I get the session value?
(flask.request.cookies.get('session') is not good for me, because I would like to get the session right after login, so it's not part of the request yet)

app.session_interface.get_signing_serializer(app).dumps(dict(session))


How to write to GitHub Container Registry

ChatGPT provides me with the following code:
- name: Log in to GitHub Container Registry
  uses: docker/login-action@v1
  with:
    registry: ghcr.io
    username: ${{ github.repository_owner }}
    password: ${{ secrets.CR_PAT }}  # The PAT you saved as a secret

- name: Push to GitHub Container Registry
  run: |
    docker tag your-image-name:latest ghcr.io/${{ github.repository_owner }}/your-image-name:latest
    docker push ghcr.io/${{ github.repository_owner }}/your-image-name:latest

I'm skeptical of the first part because GitHub's documentation (https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-container-registry) says:

To authenticate to a GitHub Packages registry within a GitHub Actions workflow, you can use:

GITHUB_TOKEN to publish packages associated with the workflow repository.
a personal access token (classic) with at least read:packages scope to install packages associated with other private repositories (which GITHUB_TOKEN can't access).

I think ChatGPT is doing the latter approach while the former approach feels better suited for me?

ChatGPT suggests this code for authenticating with GITHUB_TOKEN:
- name: Log in to GitHub Container Registry
  uses: docker/login-action@v2
  with:
    registry: ghcr.io
    username: ${{ github.actor }}
    password: ${{ secrets.GITHUB_TOKEN }}  # Use the GITHUB_TOKEN for authentication

Login seems successful, let's now try pushing to the registry!

Docker is giving me an error because Bushwallyta271828 has an uppercase letter in it. That's annoying and I'd rather not change my GitHub username to fix a Docker bug. Is there a workaround?

Okay, I literally just changed my GitHub account name. Let's see if this works now.

That seems to work! I might be able to improve things (see other issues), but generally everything looks good!


Why is LICENSE.md executable?

executable file
https://github.com/rspec/rspec/blob/e1c2c6bd78c849d7956431331f32ba5092951dab/LICENSE.md
Could you please remove the executable bit?

PRs welcome

Closed by #52


when user add another option other uption dont update automaticly and these update  just after click on the search button

I don't understand this ticket, please chekc if this is resolved,


Traceback (most recent call last):
File "exp/s3dis/origin_multi-Ua-concat-latent_contrast-Ua-softnn-latent-label-l2-w.1/train.py", line 503, in
main()
File "exp/s3dis/origin_multi-Ua-concat-latent_contrast-Ua-softnn-latent-label-l2-w.1/train.py", line 127, in main
mp.spawn(main_worker, nprocs=args.ngpus_per_node, args=(args.ngpus_per_node, args))
File "/opt/conda/lib/python3.6/site-packages/torch/multiprocessing/spawn.py", line 247, in spawn
return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
File "/opt/conda/lib/python3.6/site-packages/torch/multiprocessing/spawn.py", line 205, in start_processes
while not context.join():
File "/opt/conda/lib/python3.6/site-packages/torch/multiprocessing/spawn.py", line 166, in join
raise ProcessRaisedException(msg, error_index, failed_process.pid)
torch.multiprocessing.spawn.ProcessRaisedException:
-- Process 0 terminated with the following error:
Traceback (most recent call last):
File "/opt/conda/lib/python3.6/site-packages/torch/multiprocessing/spawn.py", line 59, in _wrap
fn(i, *args)
File "/root/workplace/dhjeon/contrastBoundary-master/pytorch/exp/s3dis/origin_multi-Ua-concat-latent_contrast-Ua-softnn-latent-label-l2-w.1/train.py", line 263, in main_worker
loss_train, mIoU_train, mAcc_train, allAcc_train = train(train_loader, model, criterion, criterion2, optimizer, epoch)
File "/root/workplace/dhjeon/contrastBoundary-master/pytorch/exp/s3dis/origin_multi-Ua-concat-latent_contrast-Ua-softnn-latent-label-l2-w.1/train.py", line 326, in train
loss = criterion(output, target, stage_list)
File "/opt/conda/lib/python3.6/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/root/workplace/dhjeon/contrastBoundary-master/pytorch/model/pointtransformer_seg.py", line 24, in forward
loss_list += self.contrast_head(output, target, stage_list)
File "/opt/conda/lib/python3.6/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/root/workplace/dhjeon/contrastBoundary-master/pytorch/model/heads.py", line 251, in forward
loss = self.main_contrast(n, i, stage_list, target)
File "/root/workplace/dhjeon/contrastBoundary-master/pytorch/model/heads.py", line 222, in point_contrast
if not torch.any(point_mask):
RuntimeError: CUDA error: device-side assert triggered
/opt/conda/lib/python3.6/multiprocessing/semaphore_tracker.py:143: UserWarning: semaphore_tracker: There appear to be 56 leaked semaphores to clean up at shutdown
len(cache))
/opt/conda/lib/python3.6/multiprocessing/semaphore_tracker.py:143: UserWarning: semaphore_tracker: There appear to be 56 leaked semaphores to clean up at shutdown
len(cache))
/opt/conda/lib/python3.6/multiprocessing/semaphore_tracker.py:143: UserWarning: semaphore_tracker: There appear to be 56 leaked semaphores to clean up at shutdown
len(cache))
The above mentioned error occurs in the contrast boundary loss.
I checked that the code is working on 'origin_4gpu.yaml'.
Please give me a solution to this.

Hi,
it seems that the buffer for knn-search is not enough.
Have you tried to re-compile the pointops in this repo?

안녕하세요, knn-search용 버퍼가 충분하지 않은 것 같습니다. 이 저장소에서 pointops를 다시 컴파일하려고 했습니까?

I guess the previously compiled one was wrong Thank you for your reply.


Added undefined check for session

Hi thanks for making this module! There is an issue that if you delete the session during logout (using req.session.destroy()) then index.js will throw "Object Reference Not Set to an Instance of an Object" on line 10.

I can approve both the problem and the solution :thumbsup: Please merge

Sorry I hadn't noticed this, sometime github's notifications seem to fail......

Published v1.0.1, thanks!!


Updates supporting layer configuration to check for parent and childl…

…ayers

This isn't a breaking change, before this change you had to pass all layerids from parent to child example layer fiets en voetganger:

If you wanted to show the velo layer you had to pass [0,2] in the layerids array of the supporting layer.
This is still possible but now you can only pass [2] if you only want the velo layer or [0] if you want every layer inside the fiets en voetganger layer.


⚠ Bofrost.at has degraded performance

In 414da8c, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1089 ms

Resolved: Bofrost.at performance has improved in d4f0b6b after 7 minutes.


subscriptions not stopped on detach

Seems that subscriptions are not stopped properly on component detach.

That seems to be just a typo. Please consider adding some tests.

Thanks for pointing out the typo. I'll add some texts. :)


getitem/setitem putmask hooks

boolean and fancy index masking related routines can be sped up and are used very often.
Be nice if we could hook these routines.

We may have to hack the tp_getitem, tp_setitem to do this.

Yes, to replace "mask" and fancy-indexing, generically, we would likely need to replace the Python-level functions.   There might be a way to speed things up a bit by replacing functions on the "PyArray_ArrFuncs *f" member of the PyArray_Descr ("dtype") object.  There are functions there that also need to be swapped out to speed up sorting, and such.  See lines near 443 to 553 of ndarraytypes.h (though I may have an old version of the NumPy code checked-out).


Godoc for SDK functions

What this PR does / why we need it:
This adds godoc comments for the Client and Management SDK API functions we reference in the RFC.
Also includes some minor refactoring of api to make more uniform, minimize exports, closer to what Fabric does (e.g., no Interface suffix for interfaces) and get rid of some go-lint warnings.
Also added two missing .gitignores and a convenience .gitignore for the case you have a local copy of the helloworld example
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing changes and/or breaks backward compatability?:

Travis check complains about missing license headers in .gitignore, we can ignore this.

@bvavala any objections to merge? :)

Travis check complains about missing license headers in .gitignore, we can ignore this.

Strangely, it didn't complain when i tested in my VM.  But added it now also as exception to the licence check script

need to add some more exceptions for .gitignore

I too naively copy/pasted from the .gitmodules rule which i've added during the re-integration to solve another licence-header issue. Removed now the preceeding '^' made the rule match only on the top-level. Hopefully now it goes through ....


Popular website right click is in another language

When I right-click to organize a website, it appears in another language. But I don't know this language. So please make it English

It will be fixed in next version

Thank you


Create 0000-01-02-Priyanka-k24.md

created a branch, created a file and made a commit, and opened a pull request


Add an e2etest for testing restart policy

Fixes #639
This PR does the following things:

add replicas restart policy test by comparing the start time of the container in the pod
fixed the pod_selector in py/tf_job_client.py by modifying
labels["tf-replica-type"] = replica_type to
labels["tf-replica-type"] = str.lower(replica_type)
wait a time interval before fetching the pods' phase in py/k8s_util.py
fixed bugs in clean pod policy test

This change is

/hold
only add v1alpha2 test currently

/test kubeflow-tf-operator-presubmit

Coverage increased (+7.2%) to 71.739% when pulling 7fd61f0c72256de61a9743add5fe5be4502539d9 on ChanYiLin:master into 89e6f66aa05b9636b7f477f75465b9fdf94d846b on kubeflow:master.

Thanks for your contribution! Feel free to assign me or richardsliu to review.

/hold cancel

/assign @gaocegege @richardsliu

/lgtm

/retest

@richardsliu
HI, I have refactored the code and added more tests.
Please help me to take a look, thanks!

/lgtm
/approve


Allow for colons in autocomplete

Thanks for the plugin, works a charm.
However, small feature request allowing for colons.
User case: autocomplete for link types (using the Juggl plugin) in which I use (amongst others) the foaf link types. Aka, I use the foaf:document to highlight documents.
The autocomplete halts when typing the ':'. Would it be an option to escape the colon character?

You could add the : to the word character regex in the settings. Does that solve your issue?

That did it.
Thanks!

I have a similar problem: I would like to autocomplete user handles in the form @username so I simply added @ to the character regex but unfortunately it does not work. Is there something else I need to consider?
Thanks!

I have a similar problem: I would like to autocomplete user handles in the form @username so I simply added @ to the character regex but unfortunately it does not work. Is there something else I need to consider?
Thanks!

Seems to work fine for me. Can you elaborate on what your character regex looks like and where you're completing?

Ah, I tried again today and now it works! My regex is this a-zA-ZöäüÖÄÜß@:. Thanks a lot for this plugin, it is really helpful!


compile errors when trying to run example tests.

I'm trying to evaluate this shaclex library, but seem stuck in running the  first example in the README file.
When running sbt "run -d examples/shacl/good1.ttl --engine ShaClex" I get following errors/warnings
[info] Loading project definition from C:\Users\jad\git\shaclex\project
Missing bintray credentials C:\Users\jad.bintray.credentials. Some bintray features depend on this.
....
[info] Compiling 6 Scala sources to C:\Users\jad\git\shaclex\modules\srdfJena\target\scala-2.11\classes...
[warn] Class java.util.function.Predicate not found - continuing with a stub.
[warn] Class java.util.function.Predicate not found - continuing with a stub.
[error] Class java.util.function.Predicate not found - continuing with a stub.
...
[warn] 8 warnings found
[error] one error found
[error] (srdfJena/compile:compileIncremental) Compilation failed
[error] Total time: 8 s, completed 16-Dec-2016 15:20:16
error.txt

Couldn't reproduce this one.
My setup:
ᐅ sbt about
[info] Loading project definition from /Users/andrew/kth/tempcode/shaclex/project
[info] Set current project to shaclex (in build file:/Users/andrew/kth/tempcode/shaclex/)
[info] This is sbt 0.13.13
[info] The current project is {file:/Users/andrew/kth/tempcode/shaclex/}shaclex 0.0.55
[info] The current project is built against Scala 2.11.8
[info] Available Plugins: sbt.plugins.IvyPlugin, sbt.plugins.JvmPlugin, sbt.plugins.CorePlugin, sbt.plugins.JUnitXmlReportPlugin, sbt.plugins.Giter8TemplatePlugin, sbtassembly.AssemblyPlugin, sbtbuildinfo.BuildInfoPlugin, sbtrelease.ReleasePlugin, com.typesafe.sbt.SbtPgp, com.typesafe.sbt.site.SitePlugin, com.typesafe.sbt.site.SitePreviewPlugin, com.typesafe.sbt.site.SiteScaladocPlugin, com.typesafe.sbt.site.asciidoctor.AsciidoctorPlugin, com.typesafe.sbt.site.gitbook.GitBookPlugin, com.typesafe.sbt.site.jekyll.JekyllPlugin, com.typesafe.sbt.site.nanoc.NanocPlugin, com.typesafe.sbt.site.pamflet.PamfletPlugin, com.typesafe.sbt.site.preprocess.PreprocessPlugin, com.typesafe.sbt.site.sphinx.SphinxPlugin, com.typesafe.sbt.GitBranchPrompt, com.typesafe.sbt.GitPlugin, com.typesafe.sbt.GitVersioning, com.heroku.sbt.HerokuPlugin, pl.project13.scala.sbt.JmhPlugin, com.typesafe.sbt.SbtNativePackager, com.typesafe.sbt.packager.archetypes.AkkaAppPackaging, com.typesafe.sbt.packager.archetypes.AshScriptPlugin, com.typesafe.sbt.packager.archetypes.JavaAppPackaging, com.typesafe.sbt.packager.archetypes.JavaServerAppPackaging, com.typesafe.sbt.packager.archetypes.jar.ClasspathJarPlugin, com.typesafe.sbt.packager.archetypes.jar.LauncherJarPlugin, com.typesafe.sbt.packager.debian.DebianDeployPlugin, com.typesafe.sbt.packager.debian.DebianPlugin, com.typesafe.sbt.packager.debian.JDebPackaging, com.typesafe.sbt.packager.docker.DockerPlugin, com.typesafe.sbt.packager.docker.DockerSpotifyClientPlugin, com.typesafe.sbt.packager.jdkpackager.JDKPackagerDeployPlugin, com.typesafe.sbt.packager.jdkpackager.JDKPackagerPlugin, com.typesafe.sbt.packager.linux.LinuxPlugin, com.typesafe.sbt.packager.rpm.RpmDeployPlugin, com.typesafe.sbt.packager.rpm.RpmPlugin, com.typesafe.sbt.packager.universal.UniversalDeployPlugin, com.typesafe.sbt.packager.universal.UniversalPlugin, com.typesafe.sbt.packager.windows.WindowsDeployPlugin, com.typesafe.sbt.packager.windows.WindowsPlugin, bintray.BintrayPlugin, scoverage.ScoverageSbtPlugin, wartremover.WartRemover, spray.revolver.RevolverPlugin, org.scalajs.sbtplugin.ScalaJSJUnitPlugin, org.scalajs.sbtplugin.ScalaJSPlugin, play.twirl.sbt.SbtTwirl, sbtunidoc.Plugin, com.typesafe.sbt.SbtGhPages, org.scalastyle.sbt.ScalastylePlugin, com.typesafe.sbt.SbtScalariform, com.simplytyped.Antlr4Plugin
[info] sbt, sbt plugins, and build definitions are using Scala 2.10.6

Output:
[info] Running es.weso.shaclex.Main -d examples/shacl/good1.ttl --engine ShaClex
[info] 22:25:36,743 |-INFO in ch.qos.logback.classic.LoggerContext[default] - Could NOT find resource [logback.groovy]
[info] 22:25:36,744 |-INFO in ch.qos.logback.classic.LoggerContext[default] - Could NOT find resource [logback-test.xml]
[info] 22:25:36,744 |-INFO in ch.qos.logback.classic.LoggerContext[default] - Found resource [logback.xml] at [file:/Users/andrew/kth/tempcode/shaclex/target/scala-2.11/classes/logback.xml]
[info] 22:25:36,745 |-WARN in ch.qos.logback.classic.LoggerContext[default] - Resource [logback.xml] occurs multiple times on the classpath.
[info] 22:25:36,745 |-WARN in ch.qos.logback.classic.LoggerContext[default] - Resource [logback.xml] occurs at [file:/Users/andrew/kth/tempcode/shaclex/modules/server/target/scala-2.11/classes/logback.xml]
[info] 22:25:36,745 |-WARN in ch.qos.logback.classic.LoggerContext[default] - Resource [logback.xml] occurs at [file:/Users/andrew/kth/tempcode/shaclex/target/scala-2.11/classes/logback.xml]
[info] 22:25:37,000 |-INFO in ch.qos.logback.classic.joran.action.ConfigurationAction - debug attribute not set
[info] 22:25:37,013 |-INFO in ch.qos.logback.core.joran.action.AppenderAction - About to instantiate appender of type [ch.qos.logback.core.ConsoleAppender]
[info] 22:25:37,025 |-INFO in ch.qos.logback.core.joran.action.AppenderAction - Naming appender as [STDOUT]
[info] 22:25:37,039 |-INFO in ch.qos.logback.core.joran.action.NestedComplexPropertyIA - Assuming default type [ch.qos.logback.classic.encoder.PatternLayoutEncoder] for [encoder] property
[info] 22:25:37,134 |-INFO in ch.qos.logback.classic.joran.action.RootLoggerAction - Setting level of ROOT logger to INFO
[info] 22:25:37,134 |-INFO in ch.qos.logback.core.joran.action.AppenderRefAction - Attaching appender named [STDOUT] to Logger[ROOT]
[info] 22:25:37,135 |-INFO in ch.qos.logback.classic.joran.action.ConfigurationAction - End of configuration.
[info] 22:25:37,137 |-INFO in ch.qos.logback.classic.joran.JoranConfigurator@782a4fff - Registering current configuration as safe fallback point
[info]
[info] 22:25:38.640 [main] INFO  es.weso.shaclex.Main$ - Schema not specified. Extracting schema from data
[info] Result Solution
[info] :y +:S OK. Evidences: Checked maxCount(2) for predicate(<http://example.org/p>) on node <http://example.org/y>
[info]  Checked minCount(1) for predicate(<http://example.org/p>) on node <http://example.org/y>
[info] :x +:S OK. Evidences: Checked maxCount(2) for predicate(<http://example.org/p>) on node <http://example.org/x>
[info]  Checked minCount(1) for predicate(<http://example.org/p>) on node <http://example.org/x>
[info]
[success] Total time: 10 s, completed Dec 16, 2016 10:25:40 PM

Please check that SBT is run with JDK 1.8, not JDK 1.7.
To check:
ᐅ sbt console
[info] Loading project definition from /Users/andrew/kth/tempcode/shaclex/project
...
[info] Starting scala interpreter...
[info]
Welcome to Scala 2.11.8 (Java HotSpot(TM) 64-Bit Server VM, Java 1.8.0_77).
Type in expressions for evaluation. Or try :help.

scala>

I also couldn't reproduce the problem. As @berezovskyi says, it may be a problem with the Java version, as I am also using Java 1.8
Could you try to change to Java 1.8 and check if the problem remains?

Yes, the problem was with me using JDK1.7. So, problem solved. thanks

Thanks, I will update the information that appears in the README to indicate the need of JDK 1.8

The reason is that java.util.function.Predicate is part of lambdas, which appeared in Java 8. Sorry for not explaining from the start.


Get iptc from url image, return null (laravel storage)

Get iptc from url image, keep return null (laravel storage)
please help me, this my code for return null:

if try to get object from local path, working and fine.
I hate to be stuck @_@

You need to download the file to server and then need to give the path
 Storage::download('file.jpg');

The integration of the option to read data directly via URLs was a mistake. The task of this library is image manipulation. Reading data via HTTP should be done by a corresponding client. For this reason, Intervention Image Version 3 does not support reading URLs.


Website should be served over HTTPS

This service is absolutely fantastic! ♥ For security/performance reasons I am wondering if there are plans to introduce SSL.

@shushugah, thanks for the kind words, and I appreciate your concern. I've been thinking about this for a while. If it were free for me to have the app be under HTTPS then I would do it in a heartbeat -- would have done it long ago. Since that's not the case, I'm less inclined to do it.
Up to this point, building and running the site has only cost me my time, and incredibly little of that, I'm very hesitant to begin paying money every month just for TLS.

@jswanner We can use CloudFlare and do this for free.  Netlify can also be the host provider, and they offer free domain provisioning through Let's Encrypt. I've used both Cloudflare and namecheap as the domain provider

As of yesterday, the site is now served from Netlify via HTTPS and the diff data backing it is, as well.

Hot

I always love seeing an old issue get closed, because it was positively addressed ♥ thanks ya'll!

Happy to do it. 😄


Run unit tests inside browsers

With https://github.com/microsoft/vscode/pull/90010 we have the ability to run unit tests inside browsers, using playwright. To run them use this command
> node test/browser/index.js

Use the --run <glob>-option to narrow select tests by file path and and use the --browser <webkit|chromium|firefox>-option to select the target runtime (note that firefox currently doesn't work, https://github.com/microsoft/playwright/issues/842).
The plan is to make this part of our build but first we need to fix some test failures that seems to only fail in (some) browser types:
These test fail running inside browsers (chromium or webkit, node test/browser/index.js --browser chromium) @isidorn
  1) Debug - ANSI Handling Expected single sequence operation - /CHROMIUM/:
  2) Debug - ANSI Handling Expected single 8-bit color sequence operation - /CHROMIUM/:
  3) Debug - ANSI Handling Expected multiple sequence operation - /CHROMIUM/:

These test fail only when running node test/browser/index.js --browser webkit @alexdima
  1) MinimapCharRenderer letter d @ 2x - /WEBKIT/:
  2) Decoration Render Options css properties - /WEBKIT/:
  3) Decoration Render Options css properties, gutterIconPaths - /WEBKIT/:
  4) chunk based search #45770. FindInNode should not cross node boundary. - /WEBKIT/:
  5) FindController query options persistence wholeWord - /WEBKIT/:
  6) FindModel incremental find from beginning of file - /WEBKIT/:
  7) FindModel find model next - /WEBKIT/:
  8) FindModel find model next stays in scope - /WEBKIT/:
  9) FindModel find model prev - /WEBKIT/:
  10) FindModel find model prev stays in scope - /WEBKIT/:
  11) FindModel find model next/prev respects cursor position - /WEBKIT/:
  12) FindModel replace hello - /WEBKIT/:
  13) FindModel replaceAll hello - /WEBKIT/:
  14) FindModel listens to model content changes - /WEBKIT/:
  15) FindModel selectAllMatches - /WEBKIT/:
  16) FindModel issue #14143 selectAllMatches should maintain primary cursor if feasible - /WEBKIT/:
  17) FindModel issue #18711 replaceAll with empty string - /WEBKIT/:
  18) FindModel issue #27083. search scope works even if it is a single line - /WEBKIT/:
  19) Multicursor selection AddSelectionToNextFindMatchAction starting with single collapsed selection - /WEBKIT/:
  20) Multicursor selection Find state disassociation enters mode - /WEBKIT/:
  21) Multicursor selection Find state disassociation leaves mode when selection changes - /WEBKIT/:
  22) Multicursor selection Find state disassociation Select Highlights respects mode  - /WEBKIT/:

Those ansii tests were written as part of a PR. I do not have a good idea why they fail inside Chromium and work when run in VS Code. I tried debugging them with --debug but failed.
To unblock you I can move the tests under electron-browser.

@isidorn this is all new but with --debug --browser chromium you should see a browser in which you should be able to use devtools

@jrieken yes I tried that and I do see chromium, but none of the tests get rendered and the window closes before I can do anything (put a breakpoint).

Yes. In the renderer the tests run without mocha UX. What you can try is this: start with --debug, open dev tools, select "pause on script exec", wait for the pause and set breakpoints that you want/need

Yeah that works, just not very convenient as you have to be fast. Thanks!

FYI - you can now debug tests by simply opening the test/browser/renderer.html page in a browser. Use a uri like this, notics the m-query arguments (this is AMD module ids):
file:///Users/jrieken/Code/vscode/test/browser/renderer.html?m=vs/editor/test/browser/services/decorationRenderOptions.test&m=vs/editor/test/browser/services/decorationRenderOptions.test&m=vs/editor/test/common/model/pieceTreeTextBuffer/pieceTreeTextBuffer.test&m=vs/editor/contrib/find/test/findController.test

In the presence of test failures, the runner will also make an attempt to print the right link for you.

Actually it appears that there are still failures on Windows:
 1) Paths (Node Implementation) dirname - /CHROMIUM/:
     ReferenceError: __filename is not defined
      at Context.<anonymous> (file:///C:/Alex/src/vscode/out/vs/base/test/common/path.test.js:146:60)

  2) Paths (Node Implementation) extname - /CHROMIUM/:
     ReferenceError: __filename is not defined
      at Context.<anonymous> (file:///C:/Alex/src/vscode/out/vs/base/test/common/path.test.js:219:18)

  3) Paths (Node Implementation) resolve - /CHROMIUM/:

      AssertionError:
path.win32.resolve(".")
  expect="/"
  actual="\\"
      + expected - actual

      -1
      +0

      at Context.<anonymous> (file:///C:/Alex/src/vscode/out/vs/base/test/common/path.test.js:369:20)

  4) Paths (Node Implementation) basename - /CHROMIUM/:
     ReferenceError: __filename is not defined
      at Context.<anonymous> (file:///C:/Alex/src/vscode/out/vs/base/test/common/path.test.js:382:46)

  5) Resources relativePath - /CHROMIUM/:

      AssertionError: from file://server/share/some/ to file://server/share2/some/path
      + expected - actual

      -/share2/some/path
      +../../share2/some/path

      at assertRelativePath (file:///C:/Alex/src/vscode/out/vs/base/test/common/resources.test.js:210:20)
      at Context.<anonymous> (file:///C:/Alex/src/vscode/out/vs/base/test/common/resources.test.js:245:17)

  6) Debug - ANSI Handling Expected single sequence operation - /CHROMIUM/:

      AssertionError: Custom foreground class not found on element after foreground ANSI code #30.
      + expected - actual

      -false
      +true

      at file:///C:/Alex/src/vscode/out/vs/workbench/contrib/debug/test/browser/debugANSIHandling.test.js:135:21
      at assertSingleSequenceElement (file:///C:/Alex/src/vscode/out/vs/workbench/contrib/debug/test/browser/debugANSIHandling.test.js:83:13)
      at Context.<anonymous> (file:///C:/Alex/src/vscode/out/vs/workbench/contrib/debug/test/browser/debugANSIHandling.test.js:134:17)

  7) Debug - ANSI Handling Expected single 8-bit color sequence operation - /CHROMIUM/:

      AssertionError: Custom color class not found after foreground 8-bit color code 38;5;0
      + expected - actual

      -false
      +true

      at file:///C:/Alex/src/vscode/out/vs/workbench/contrib/debug/test/browser/debugANSIHandling.test.js:193:21
      at assertSingleSequenceElement (file:///C:/Alex/src/vscode/out/vs/workbench/contrib/debug/test/browser/debugANSIHandling.test.js:83:13)
      at Context.<anonymous> (file:///C:/Alex/src/vscode/out/vs/workbench/contrib/debug/test/browser/debugANSIHandling.test.js:192:17)

  8) Debug - ANSI Handling Expected multiple sequence operation - /CHROMIUM/:

      AssertionError: Foreground color class not found after multiple different ANSI codes.
      + expected - actual

      -false
      +true

      at file:///C:/Alex/src/vscode/out/vs/workbench/contrib/debug/test/browser/debugANSIHandling.test.js:288:17
      at assertSingleSequenceElement (file:///C:/Alex/src/vscode/out/vs/workbench/contrib/debug/test/browser/debugANSIHandling.test.js:83:13)
      at Context.<anonymous> (file:///C:/Alex/src/vscode/out/vs/workbench/contrib/debug/test/browser/debugANSIHandling.test.js:284:13)

The tests for path have been moved to the node-layer. @alexdima I believe that you are missing a fresh compile

@jrieken You're right. But after restarting gulp watch I see one test failure left:
  1) Resources relativePath - /CHROMIUM/:

      AssertionError: from file://server/share/some/ to file://server/share2/some/path
      + expected - actual

      -/share2/some/path
      +../../share2/some/path

      at assertRelativePath (file:///C:/Alex/src/vscode/out/vs/base/test/common/resources.test.js:210:20)
      at Context.<anonymous> (file:///C:/Alex/src/vscode/out/vs/base/test/common/resources.test.js:245:17)

@aeschli can you take a look at those ☝ test failures

Running node test/browser/index.js --browser webkit on Ubuntu with Chromium I only get the following test failure:
(HEAD ea8d1b7e548c7274e645f79598df3ca1d7b40db7)
1) MinimapCharRenderer letter d @ 2x - /WEBKIT/:

      [46,46,46,255,173,173,173,255,198,198,198,255,202,202,202,255,193,193,193,255,204,204,204,255,0,0,0,255,0,0,0,255] deepEqual [46,46,46,255,173,173,173,255,198,198,198,255,200,200,200,255,193,193,193,255,204,204,204,255,0,0,0,255,0,0,0,255]
      + expected - actual

         198
         198
         198
         255
      -  202
      -  202
      -  202
      +  200
      +  200
      +  200
         255
         193
         193
         193

@aeschli yes - this is Chromium and windows

I fixed the relativePath test failure

This PR adds browser-unit-tests to our continuous build: https://github.com/microsoft/vscode/pull/90153. There are still these failures

looks like linux fails to run webkit tests (see https://dev.azure.com/vscode/VSCode/_build/results?buildId=36640&view=logs&j=0da5d1d9-276d-5173-c4c4-9d4d4ed14fdb&t=db522a89-84be-55c1-d087-ef4dde315b47)
@alexdima windows with webkit seems to have decorations-test failure, https://dev.azure.com/vscode/VSCode/_build/results?buildId=36640&view=logs&j=2d2b3007-3c5c-5840-9bb0-2b1ea49925f3&t=4f94dfce-af9e-52fd-5eb8-ca3006505634
@alexdima Mac with webkit seems that have minimap-test failure, https://dev.azure.com/vscode/VSCode/_build/results?buildId=36640&view=logs&j=a5e52b91-c83f-5429-4a68-c246fc63a4f7&t=855ed636-0983-5316-9b28-7efa8f10a65b

@jrieken maybe something to consider: we support yarn smoketest --web --browser <type> for smoke tests. We could add a script like that for unit tests as well?

With the PR above there is yarn test-browser

@jrieken Could we make this one compatible with the other one, i.e.
node test\unit\browser\index.js --browser webkit --run src/vs/editor/test/browser/services/decorationRenderOptions.test.ts

does not work while it works in the other one, e.g.
scripts\test.bat --run src/vs/editor/test/browser/services/decorationRenderOptions.test.ts

This is superhelpful for me because I have a copy relative path plugin that I use on files and then I just paste the path

The --run argument is now supported as in other tests, there is not --run for workspace relative paths (with TS → JS trickery) and there is --glob for patterns

All tests green, PR merged, done


execute uncss with a javascript file

Is it possible to execute uncss against a javascript file? such that:
uncss libs/myscript.js media/css/mycss.css > output.css
I am trying it but output.css doesn't contain the css rules used by javascript file. For example, adding a class with js:
$(element).addClass('form-field-error');
I have a validation javascript that includes error styles when the data is wrong. I am trying to execute uncss against an array of different types of files: smarty templates (tpl), javscripts, ...

@crico That isn't possible, uncss actually runs your files as html in a headless web browser. Some types of template files might work if the template is valid html, but raw javascript will not work.
Currently, the best solution for including error styles is to use the ignore option.
Note that uncss will parse classes/selectors added by the javascript during the page load (when you run uncss on the html file), but not those dynamically added by user interaction.


[GSoC '18] Make unassign command accessible to org members only

This is part of https://github.com/coala/corobo/issues/551


firfox solution

i found the solution in firfox  source code
https://hg.mozilla.org/try/rev/08a4fafd6ac5
it works in 21H2 19044.2846

Hi,
Im sorry, I dont understand the issue.
Does pttb.exe not work with 21H2 19044.2846?

right ，bro
on my computer（21H2 19044.2846 ），pttb.exe  it not work .
but i found another way to pin taskbar in firefox  source code ，and it works .


bash: swa: command not found

Following all the steps precisely, including installation of static-web-apps-cli and functions-core-tools V3, i am still getting
bash: swa: command not found

when trying to run web app and API locally.
Here is my $PATH variable content:
$ echo $PATH
/mingw64/bin:/usr/bin:/c/Users/EDKN.SCDOM/bin:/c/Program Files/PowerShell/7:/c/Python39/Scripts:/c/Python39:/c/Program Files/AdoptOpenJDK/jre-<IP_ADDRESS>-hotspot/bin:/c/Program Files/Microsoft SDKs/Azure/Azure Dev Spaces CLI:/c/Program Files (x86)/Microsoft SDKs/Azure/CLI2/wbin:/c/WINDOWS/system32:/c/WINDOWS:/c/WINDOWS/System32/Wbem:/c/WINDOWS/System32/WindowsPowerShell/v1.0:/c/WINDOWS/System32/OpenSSH:/c/Program Files (x86)/Airtame:/c/Program Files/Intel/WiFi/bin:/c/Program Files/Common Files/Intel/WirelessCommon:/c/Program Files/Microsoft SQL Server/130/Tools/Binn:/c/Program Files/Microsoft SQL Server/Client SDK/ODBC/170/Tools/Binn:/c/Program Files (x86)/dotnet-core-uninstall:/c/Program Files/Calibre2:/c/Program Files/dotnet:/c/ProgramData/chocolatey/bin:/c/Program Files (x86)/sbt/bin:/c/Program Files (x86)/Google/Cloud SDK/google-cloud-sdk/bin:/c/Program Files/Azure Data Studio/bin:/c/Go/bin:/c/Program Files/Microsoft/Azure Functions Core Tools:/c/tools:/c/Program Files (x86)/dotnet:/c/Users/EDKN.SCDOM/AppData/Roaming/nvm:/c/Program Files/nodejs:/c/Program Files/PowerShell/7-preview/preview:/c/Program Files/PuTTY:/c/Program Files/PowerShell/7:/c/Program Files/Amazon/AWSSAMCLI/bin:/c/Program Files/nodejs:/c/Program Files/Microsoft/Web Platform Installer:/cmd:/c/Program Files/Docker/Docker/resources/bin:/c/ProgramData/DockerDesktop/version-bin:/c/Program Files/Microsoft Service Fabric/bin/Fabric/Fabric.Code:/c/Program Files/Microsoft SDKs/Service Fabric/Tools/ServiceFabricLocalClusterManager:/c/Users/EDKN.SCDOM/AppData/Roaming/.dotnet:%userprofile%/.dotnet/tools:%localappdata%/Programs/Microsoft VS Code/bin:/c/dapr:%localappdata%/Microsoft/WindowsApps:/c/Users/EDKN.SCDOM/AppData/Local/Pandoc:/c/Users/EDKN.SCDOM/AppData/Local/Programs/Microsoft VS Code Insiders/bin:/c/Users/EDKN.SCDOM/AppData/Local/Programs/Microsoft VS Code/bin

I should not have to spend an hour chasing down this failure in a quickstart that takes 10 min to complete.
And yes, the file 'swa' does exist at "C:\Users\EDKN.SCDOM\AppData\Roaming\npm\swa"

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

ID: 5689b389-ead5-825d-dde1-9a290bc0819f
Version Independent ID: 52a526a2-477a-dda4-02b4-eac1440811ed
Content: Add an API to Azure Static Web Apps with Azure Functions
Content Source: articles/static-web-apps/add-api.md
Service: static-web-apps
GitHub Login: @manekinekko
Microsoft Alias: wachegha

Hey, @Monte-Christo thank you for opening this issue. Looking at your $PATH variable, it looks like the folder where you installed swa, i.e. /Users/EDKN.SCDOM/AppData/Roaming/npm is not exposed by your $PATH variable. Please make sure to that add that folder to your $PATH variable and try running the swa command again. Thank you.

@manekinekko - does the doc need to add this step, or is the CLI being updated to take care of the path issue?

@craigshoemaker I think this is related to the installation of Node.js and npm and is outside of Docs scope. Here are the docs on npm: https://docs.npmjs.com/downloading-and-installing-node-js-and-npm.

You can push the blame around whichever way you like. When I follow a quickstart for beginners such as the one on Static Web Apps with Azure Functions, I expect the thing to work out of the box. If there is an issue with npm, then this needs to be in the docs as a heads-up. Just throwing a "command not found" into the user's face makes me not even want to continue.

Thanks for your feedback @Monte-Christo. I am going to call to #please-close this issue for now, but we will continue to refine this doc to remove any unnecessary dependencies.


Dropdowns do not update when values are set in backend

If I set my country dropdown's value to "Canada" in the backend, the dropdown shows the selected country as "Canada", but the region dropdown still displays "-" as if a country hasn't been selected.
Update
To get the region dropdown to populate, I had to do this:
document.getElementById('country').dispatchEvent(new Event('change'));

Thank you! This is exactly what I needed. I have a separate prepopulation script that puts a value in the country field, but it doesn't trigger the region specific list, so the region stays uninitialized, and results in not it being populated with the other given value.


this project

How long will this project be completed?

Is this ok? can be used?

How long will this project be completed?
Is this ok? can be used?

Thingsboard 3.0 UI app has been migrated to latest Angular, therefore this project will be discontinued.


Switch ordering of "Admin" and "User" survey fields in accept request admin view

Maybe it's better to have at first the "user" survey fields and then below the "admin" survey fields in the accpet request view. So that the admin can check at first the user inputs (if things are unclear ask more info from user), before going to set the admin inputs....what do you think?
Regards,
Stephan

Good suggestion, see you in the next release !

I confirm that it's a good point.
Go for it.

Done and merged in dev


TuriCreate crippled by dependencies on old MXNet

TuriCreate requires MXNet version 1.1.0 it has has serious issues. It's not compatible with fixed version to MXNet version 1.5.0. Because then it needs a new numpy and tensor. Can TuriCreate be updated to not require such outdated frameworks?
TuriCreate is crippled by MXNet and you should change your requirements or else no one will be able to create big models. I can't use the audio classifier past ~60K items before it dies.
see https://github.com/apple/turicreate/issues/2286

Hey @rplom, we are actively working on this issue.  Expect some resolution to it soon.

Amazing!  Thank you!  I'm working around it by using random_split with large numbers until I can grow the training set safely again.

Just copying some of the MXNet details here from the other ticket
Large Tensor Support
Before MXNet 1.5.0, MXNet supported a maximal tensor size of around 4 billion (2³²). This was due to uint32_t being used as the default data type for tensor size, as well as variable indexing. Now you can enable large tensor support by changing the following build flag to 1: USE_INT64_TENSOR_SIZE = 1.(Note this is set to 0 by default) This enabled large scale training for example large graph network training using Deep Graph Library.
see: apache/incubator-mxnet#9207
https://medium.com/apache-mxnet/apache-mxnet-1-5-0-release-is-now-available-4138f5233401

TuriCreate is API compatible with MXNet 1.5.0. TuriCreate even use 1.5.0 but only for Python 3.7 with macOS. We couldn't get it to work on Linux. We got nondeterminstic hangs in several but not all of our deep learning toolkits.
@rplom - are you using macOS or Linux? What toolkit are you using (i.e. what method are you calling .create(...) on)?

macOS X running your standard install with venv
I followed the requirements you have on the manual page.

When I got up around ~62K sounds (I don't know the exact number) I started hitting MXNet integer overflow issues.  I tracked down the issue in MXNet and built a 1.5.0 version with the USE_INT64_TENSOR_SIZE.  As soon as I ran Turicreate I got  warning about my numpy.  I updated that.  The next run I got an error with numpy wanting a new Tensorflow.  I successfully ran the model creation at that point, but I was concerned that even though "it worked" there were warnings and I wasn't sure if that would impact or degrade the model accuracy.  I decided to revert to the prescribed install versions and log this issue here.  I'm working around my upper bound problem by randomly splitting my training set to keep it under the threshold for overflow.  Ideally tho I can foresee 100K sounds to get the breadth I need for my application.
If you think it's safe to run those newer frameworks with Turicreate, then I will update to those.

How about numpy?  MXNet wants an updated number when I bump the version.


Regarding the 'Favorites' functionality

I'm sorry in advance if this is how the program is intended to work, i admit that it's been somewhat difficult for me to get to configure the program to suit my needs (besides, im somewhat new to *boorus so that only makes things harder)
Anyway, the problem is the following: im having issues with my Favorites. I've created a folder exclusively for them, and i want them to be saved using the "%artist%/%copyright%/%character%/%md5%.%ext%" syntax, to distinguish them from my non-favorite images.
Thing is: i) They're not being named by that syntax, instead it's using the Default file naming (which is currently "%copyright%/%md5%.%ext%") and ii) They're also being saved in the default folder. This point in particular isn't that much of a problem if it weren't for (i).
Any suggestions? The reason im asking is because i'd like to keep images from a show (for example) all piled together in one folder, while maintaining pictures from certain artists according to the character.
By the way (and this is unrelated to the title of this ticket) i was experiencing a couple of technical difficulties regarding extremely sluggish operation and a lot system resources consumed, basically it was taking too long for Grabber to do anything (even start up) and when it did, it consumed a lot of CPU power. I was unable to pinpoint the source of it, but thankfully it was fixed by un-installing the old version and installing the latest release in the root of the drive (on Windows 7). Maybe it had to do with my settings, i had a long list of blacklisted tags and added sources, but i can't say for sure.
Thanks for your time and all the effort put together into this program

Is the problem you're having happening during batch downloads?
And are you talking about the favorites like in the favorite tab or the favorites in the Save settings (where you can decide of a custom filename and file folder).
The two of them are currently not related. The first only decides of what appears on the "Favorites" tab while the second is used for the path and filename when you click the "Save (favorite)" in a details window (when you click a thumbnail in the picture list).
Your suggestion would be to automatically save the pictures in the "Favorites" tab to a different folder with a different filename?
As for your last issue, if you're saying it got fixed by reinstalling I guess it's ok? If you have this issue again please tell me, but I don't see any reason for the program to use a lot of system resources only to start up (batch download is another story though).

Is the problem you're having happening during batch downloads?
And are you talking about the favorites like in the favorite tab or the favorites in the Save settings (where you can decide of a custom filename and file folder).
The two of them are currently not related. The first only decides of what appears on the "Favorites" tab while the second is used for the path and filename when you click the "Save (favorite)" in a details window (when you click a thumbnail in the picture list).
Your suggestion would be to automatically save the pictures in the "Favorites" tab to a different folder with a different filename?
As for your last issue, if you're saying it got fixed by reinstalling I guess it's ok? If you have this issue again please tell me, but I don't see any reason for the program to use a lot of system resources only to start up (batch download is another story though).

Is the problem you're having happening during batch downloads?

Precisely, i didn't know that favorites were intended for particular images selected through the details window. Well, now i understand that's the expected behavior (Pick an image, and add it to favorites); makes a lot of sense really.

Your suggestion would be to automatically save the pictures in the "Favorites" tab to a different folder with a different filename?

My prior understanding was that Favorites applied to tags, such as ('artist_x') and the program would use the settings configured in the Options menu  to all pictures containing that tag in the favorites left-panel. Therefore, all images from selected sources (booru1, booru2) with the corresponding tax (artist_x) will be saved with the favorites_filename to the favorite_folder. Something like that.
What i expected out of "Favorites" is probably outside the scope of it. If i understand correctly now, favorites are images picked manually by clicking on the thumbnail and pressing the "+" button and selecting the Save (fav), Save and Close (fav) or Destination folder (fav).
Therefore, to achieve my original objective i should just change the filename syntax from "Options" directly ![Here] (http://a.loveisover.me/iktpzs.png) instead of using Favorites (please correct me if im wrong)
Maybe what i should have requested in the first place is some sort of Presets: adding the capability of having several different filename and save folder options configured at the same time, and being able to select them as will, so they can be used for different situations. For example Preset 1 would be named 'Artists' with syntax1 (%artists%/%md5%.%ext%) and folder ./Selected Artists; Preset 2 with syntax2, folder ./Series and so on; this way images could be kept organized following different criteria as the user sees fit. What do you think about it? Can it be done with the current set of tools Grabber has, or does it have to be implemented as a new option? If so let me know how hard it is to program so can keep the ticket open or close it.
Thanks again for your time and patience :+1:

Therefore, to achieve my original objective i should just change the filename syntax from "Options" directly Here instead of using Favorites (please correct me if im wrong)

If your objective is to have a different folder / filename for a specific batch download you can directly edit the batch download table in the "Downloads" tab.

Maybe what i should have requested in the first place is some sort of Presets: adding the capability of having several different filename and save folder options configured at the same time, and being able to select them as will, so they can be used for different situations. For example Preset 1 would be named 'Artists' with syntax1 (%artists%/%md5%.%ext%) and folder ./Selected Artists; Preset 2 with syntax2, folder ./Series and so on; this way images could be kept organized following different criteria as the user sees fit. What do you think about it? Can it be done with the current set of tools Grabber has, or does it have to be implemented as a new option? If so let me know how hard it is to program so can keep the ticket open or close it.

I guess it would actually be doable currently using a mix of custom tokens as well as token conditionals (see the Filename wiki page).
You could create a custom token with the tags you want, then add a conditional on that token for a custom filename. Another option would be to simply use Javascript filenaming but that implies knowing basics of Javascript.
My suggestion for you feature is to keep then custom token system, but add optional "filename" and "folder" fields for each custom token. If this field is set and one of the tags is found, it will use this filename and/or custom folder.
What do you think?

My suggestion for you feature is to keep then custom token system, but add optional "filename" and "folder" fields for each custom token. If this field is set and one of the tags is found, it will use this filename and/or custom folder.

Sounds like a great solution. Guess it's about time to learn how tokens work, because honestly, i have no idea. It'd be nice if you could update the documentation accordingly, if it's necessary.
Thanks a lot Bionus!
Cheers

Then I guess I'll add this in an upcoming update, 4.1.5 or 4.1.6. Since I have a lot of work on my hands right now I guess it'll be out around next week.
As for the tokens, there's not much to know. They're just the things with the % in them (%artist%, %copyright%, etc.), and they're all pretty much documented on the Filename wiki, as well as custom tokens.
Cheers

Don't worry, take your time, this time of the year is really busy for pretty much everyone, and this ticket is a suggestion after all, not a critical bug or anything like that (you've already made me happy with the promise that this will be implemented in the future :smile: )
Cheers


Timeout error

In iran bitcoin checker not worj and show time out error please change source address price not filter in iran

What exchange(s) are you using?
Does the issue reproduce in the latest version of Bitcoin Checker v2.57?

Outdated


Policy Groups in the policy file details

A policy file can be part of multiple policy groups.
Policy file details API:
GET /organizations/NAME/policies/POLICY_NAME/revisions/REVISION_ID
provides the information for the Policy but it does not include the names of the policy groups the policy file is part of.
This is needed to show the relation between policy file and policy groups.

Comment related to the issue - https://github.com/chef/chef-server/issues/2480#issuecomment-852664916

@PrajaktaPurohit The request is for API - GET /organizations/NAME/policies/POLICY_NAME/revisions/REVISION_ID. The changes should be made to https://github.com/chef/chef-server/blob/650c9713d2b54ce7caec78bfaa37d674e92cbfd0/src/oc_erchef/apps/oc_chef_wm/src/oc_chef_wm_named_policy_named_revision.erl#L106 right?

Policies can have many revisions. Policy need to have at least one revisions.

The policies table has the minimal details of the policy.
The policy_revisions table has the details of the policy of all its different revisions. The reference is the 'name' and 'policy_authz_id'

Policy can exist without group also. A policy group can have only one revision of a policy. A particular revision can be part of more than one group.

The policy_groups table has the minimal details of the policy group.
The policy_revisions_policy_groups_association table has the association between policy_revisions and  policy_groups. 'policy_group_authz_id' and 'policy_group_name' references policy_groups table and 'policy_revision_revision_id' and 'policy_revision_name' references policy_revisions.

There are two main ways in which we can create policies.

API - POST /organizations/[organization_id]/policies/[policy_name]/revisions
creates a policy of given revision without assigning to a group.
API - PUT /organizations/[organization_id]/policy_groups/[policy_group]/policies[policy_name]
creates a policy of given revision of it does not exits and associates it with the given group. If the policy exists but the revision is new, creates a policy with new revision and updates the association with the new policy revision.

The best APIs to add the requested details are

GET - /organizations/[organization_id]/policies/[policy_name]
GET - /organizations/[organization_id]/policies/[policy_name/revisions/[revision_id]

https://github.com/chef/chef-server/issues/2867 - merged.


fix #107

fix incorrect function nameGetBucketReferqer

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

Coverage decreased (-50.7%) to 40.567% when pulling 14c74c13da80d98c5d35ecbb9ed13d7ee20868ae on DennisMao:master into 36bf7aa2f916f274137aedd05d1d36f6a3c0f4a6 on aliyun:master.

谢谢你的反馈，下个版本我们将合入这个pr


⚠ Fortnite has degraded performance

In fa2ddfe, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1720 ms

Resolved: Fortnite performance has improved in a4a9944.