Python3 part10

Pull Request Description
What issue does this change request address?
Closes #71
What are the significant changes in functionality due to this change request?
Removal of old getpot reader in TreeStructure.
Changes to Rook to let it be used by PHISICS.
Allows coverage to be done from the tester.
Allows command prefix to be added to all commands the tester calls (to for example run them with qsubs).
Adding a differ that just checks for the file to exist.
Moved OrderedCSVDiffer, XMLDiff and DiffUtils to Rook from RAVEN.
Adds a system for different types of tests to be run with different command line parameters.
Now can wait a bit for files (needed for running a cluster where the filesystem can take some time to propagate all the files)
Smooth the output of the percentage of cpu used to reduce overloading the test machine.
Converted cluster tests to use Rook.
Added a test to make sure Rook can handle megabytes of output data.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?

Job Mingw Test on c600537 : invalidated by @joshua-cogliati-inl
seeing if fail was consistent

Job Mingw Test on c600537 : invalidated by @joshua-cogliati-inl
trying with -j4

Job Test mac on b857272 : invalidated by @joshua-cogliati-inl
seeing if fail of large_HDF5 test is consistent.

I have started on this review, sorry I haven't finished yet. I'll try to wrap it up soon.

Approved for merge pending successful completion of the regression tests.


[BUG] nan average_loss when running quantize_with_alpaca.py

Describe the bug
I tried to quantize Llama2-7b-chat using quantize_with_alpaca.py. But got nan loss for the last few layers.
When I try to use the model, it will only generate .
However, the model works well if using basic_usage.py
Screenshots

@PanQiWei

Did you solve it?I have the problem.

I  delete   pytorch_model.bin ,  which from origin Baichuan,and use pytorch_model.bin.index.json produced by combined model.The problem is solved!

@jaysonph Did you set the batch_size to values larger than 1? If so, I would suggest to set it to 1 which maybe more stable.

Hi @jaysonph, please provide reproduction info.


Define action points

For a given turn in the battle sequence,  players will have a limit of moves available to them and be limited by the AP limit to conceive a strategy. For instance,  moving up 3 squares would use 3 points,  using a magic attack would use 4 versus a physical attack that could be from 2 to 5 depending on the character and the weapon being used (if any).

Once a job role has been attributed to a character, right now, actions points are given to that character. But no system is using those points to manage a character's action in the battle system.
What I'm thinking of is actions should be definitive. Once an action was selected through the UI, the player can see the action range of the selected action is there are any and should also see the cost of those actions in the UI such as moving 3 squares up north takes up 3 points, using a spell or melee attack would take 1 point and a class' special attack would take 2 points.
Once the turn of a character ends and its turn come back again, the actions will be regenerated.


JBEHAVE-1476 Fix XSD location of Weld beans.xml

The changes have been cherry-picked to master: 130ee9e9f224e19003b695e1a8cbac5a69864841


use dep instead of glide

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8ca1f5d). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #48   +/-   ##
=========================================
  Coverage          ?   58.72%
=========================================
  Files             ?        5
  Lines             ?      235
  Branches          ?        0
=========================================
  Hits              ?      138
  Misses            ?       92
  Partials          ?        5

Continue to review full report at Codecov.

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


Revisiting mkimage-arch.sh

Overview
This pull request aims to clean up and slightly reduce the size of Arch Linux base images generated with mkimage-arch.sh. So far it does the following two things:

prevent installing more unnecessary packages
delete man pages of installed packages

These changes reduce the image size by 16.1 MB or ~5% (from 309.7 MB to 293.6 MB).
Note: Image size is the VIRTUAL SIZE as reported by docker inspect on a btrfs file system. The test images were built around 2014-12-21 00:49:28 UTC.
Details
Deleting man pages
The script already skips installing man-db and man-pages to save on disk space, however it fails to delete man pages that belong to the installed packages. rm -r /usr/share/man/* accomplishes just that, and reduces the image size by about 11.9 MB.
Revising list of installed packages
This is more for cleaning up than to save a few megabytes, but it also further reduces the image size by 4.2 MB.
The following is a revised list of packages that should not be installed and a short explanation on why they should be left out, along with a few notes on packages that cannot be left out. Packages denoted with an * are already ignored in the current version of mkimage-arch.sh, the rest are new additions.

linux*, systemd-sysvcompat - Containers do not boot.

systemd - procps-ng (pkill, pgrep, etc.) depends on it.

openresolv - resolv.conf is managed by the docker daemon.

netctl,dhcpcd,iproute2 - Network interface is managed by the docker daemon.

inetutils - Contains telnet and ftp.
iputils - Contains ping.

pciutils*, pcmciautils*, usbutils - Hardware management is done by the host.

jfsutils*, xfsprogs*, reiserfsprogs* - File systems are managed by the host.

e2fsprogs - pam, a base package indirectly depends on it.

lvm2*, mdadm*, cryptsetup*, device-mapper - Storage devices are managed by the host.

man-db*, man-pages* - Documentation not useful inside a container.

texinfo - pacman indirectly depends on it.
licenses - Should not be left out, see here

s-nail* - Unlikely to be widely used.

For completeness, here are the members of the base group that are installed.
bash, bzip2, coreutils, diffutils, file, filesystem, findutils, gawk, gcc-libs, gettext, glibc, grep, gzip, inetutils, iproute2, iputils, less, licenses, logrotate, nano, pacman, perl, procps-ng, psmisc, sed, shadow, sysfsutils, tar, util-linux, vi, which.
Note: groff has been removed from the ignore list because it is actually not installed by base (it's in base-devel).
Feedback wanted
The patched script builds a working image, however it's possible that I overlooked an unnecessary package or removed an important one. I linked all package names to the respective pages on the Arch Linux web site to make it easier to review the changes. Let me know what you think.

I personally think the images should be clean as well and not include editors etc, but @tianon is the main deciding factor here

I'm definitely +1 on removing editors, etc. from base images, but sometimes upstream isn't as privy to that (see Ubuntu as an example).

awesome info - bookmarking for docs :)

Thanks guys! I updated the pull request to remove nano and vi (shaving off another 1.6 MB) and updated the OP to reflect the changes.
@tianon let me know if you want me to squash the commits into one before merging.

With the list of @tianon, those packages are being installed into the image:
bzip2
coreutils
diffutils
e2fsprogs
file
filesystem
findutils
gawk
gcc-libs
gettext
glibc
grep
gzip
inetutils
iputils
less
licenses
logrotate
pacman
perl
procps-ng
psmisc
sed
shadow
sysfsutils
tar
texinfo
util-linux
which

If we ignore the fact, that dockerfiles may depend on them, or they may be useful for most of the users, these packages could also be removed: diffutils, file, gettext, grep, inetutils, iputils, logrotate, psmisc, sysfsutils, tar, and which.
procps-ng has to be removed after the pacman keys were populated because procps-ng contains pkill. Removing this package also removes the systemd package.
locale-gen, which is called from the script requires sed.
When leaving procps-ng and sed in, my image is 285,4MB in size.

@tianon good point. Amended.

@dasJ so, in both Debian and Gentoo, there's an understanding in package metadata that packages in the base set are not required to be listed explicitly in dependencies of other packages -- is that true in Arch as well?  ie, are packages allowed to assume that basic tools like pkill, sed, grep, etc are available without explicitly depending on them?

This PR as-is LGTM anyhow, but I think there's clearly room for further discussion about the exist list of packages to purge. :+1:

@dasJ You left out bash from the list, but other than that it's correct.

"If we ignore the fact, that dockerfiles may depend on them, or they may be useful for most of the users..."

I think this is the perfect argument against removing those packages.
file, inetutils, iputils, procps-ng, ps-misc, sed, sysfsutils, tar, which provide binaries that are too essential to be removed. An image without them would barely work. It would be impossible to derive new images from such a base. Many shell scripts would fail as well.
I'm not sure about logrotate, diffutils and gettext.
PS.: The list of installed packages in the OP was not correct. I missed e2fsprogs and texinfo, and included iproute2. It should be fixed now.

LGTM

@tianon That is not true for Arch.
Packages list all run-time dependencies even if they are in base. For example device-mapper lists systemd, both of which are in base. And so does the docker package. I think that's because in Arch base is a suggestion rather than a requirement and users are encouraged to customise even by leaving out packages they don't need (e.g. jfsutils).
Note: There is a similar rule for packages built from source. They should not list compile time dependencies that are already in the base-devel group (packages required for building packages, e.g. make and gcc).

Ah cool, thanks for clarifying!


Make importable as GovUKFrontendJinja.init_app

https://github.com/alphagov/govuk-frontend-jinja/blob/master/govuk_frontend_jinja/flask_ext.py#L10
GovUKFrontendJinja.init_app ?
It doesn't seem to matter but does follow convention?

The documentation for Flask seems to suggest that either form is good http://flask.palletsprojects.com/en/1.1.x/extensiondev/#initializing-extensions, I went for the function because I couldn't see a need for a class.


Fix unhide bug

Codecov Report

Merging #270 (85cf6df) into master (65f0269) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #270      +/-   ##
============================================
- Coverage     70.79%   70.74%   -0.06%
  Complexity      814      814
============================================
  Files           111      111
  Lines          2736     2738       +2
  Branches        365      365
============================================
  Hits           1937     1937
- Misses          689      691       +2
  Partials        110      110

Impacted Files
Coverage Δ

.../address/logic/commands/UnhidePatientsCommand.java
0.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Option to allow duplicate downloads between member and tag searches

Subject TL;DR. Unless there's a config setting I have missed. More often than not I have images that exist between member and tag lists but currently it will only download a given image from one source and ignore the other. It'd be awesome to be able to allow this.

Assuming you download the image to different folder, try to set:
overwrite = True
alwayscheckfilesize = True

in config.ini

This was discussed in #107, and the solution there was to allow various config options by which to specify alternate config.ini and db files.
Sounds like a more sophisticated/simplistic option became available at some point. So...using the same database, and checking the file size, the program will overwrite [its null option]?
I'm a little dense tonight, but I'm a bit confused by how overwrite interacts with the file size check...just to confirm, if I run multiple searches or searches of different kinds, these two options will have the same effect as though the database was unaware of previous downloads? The alwayscheckfilesize option always confused me, lol ^^;;;
Very glad to see you're still active on Github. Transferring your blog to the Dutch blogger might work, since nothing on pixiv violates Dutch law, and ahem a group I'm aware of has run a small translation blog there without harassment for a good 4 years now. There are also a few reliable and fairly inexpensive Dutch web sites that don't mind adult content of this type. Not that you need one, but just letting you know about an option.

overwrite and alwayscheckfilesize = False
if already exists in DB, skip download.

overwrite = True, alwayscheckfilesize = False
always download even file exists in DB and the target file path.

overwrite = True, alwayscheckfilesize = True
always download, except file already exists in target file path with the same size.

overwrite = False alwayscheckfilesize = True
no impact, similar with 1st behaviour.

@Nandaka Is this outdated information?

overwrite = True, alwayscheckfilesize = True
This will check the filesize if file already exists in target file path, but it will overwrite even if the filesize matches.

overwrite = False alwayscheckfilesize = True
This will check the filesize, but it will NOT overwrite if the filesize match. This will ignore the DB, so it's not similar with the 1st behaviour.


Fix yet another in-page link

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/develop-your-app/OnScreenKeyboard.md
:white_check_mark:Succeeded

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


Add support for OS version range

Allows specifying the version (a range of versions) of the target simulator.

I'm fixing CI currently

If you rebase and rebuild the dist files it CI is working again.

Thanks @mxcl, all rebased and CI passes now. Yet I'm not sure that the check I've added was run.

It was not run because you did not use the matrix value os-version in the job.
I would also request os-version be renamed platform-version to match the platform key.

Good catch about the use in job, fixed. And also renamed to platform-version.


Add formatting instructions

The README did not mention the extension supports formatting of .smithy files. This change advertises the formatting feature and adds instructions for use
Tested auto formatting in VS code by adding and removing the block in settings.json and attempting to format via the command palette. Formatting worked when the block was there, and did not do anything when the block was omitted.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Can you provider information on how did you test it?
Thanks!

Hey @yasmewad, I added my ad-hoc testing method to the description above

Tested auto formatting in VS code by adding and removing the block in settings.json and attempting to format via the command palette. Formatting worked when the block was there, and did not do anything when the block was omitted.


[Fixes #371] Add Github Actions CI pipeline

fixes #371
This patch adds GitHub actions CI pipeline.
CI pipeline:

Creates multiple environments
Installs requirements and HUB
Lints code with Black
Runs pytest and codecov

This looks great! @ADI10HERO we need to this all fixed, let us know how we can help to make all the checks.

[ ] add credentials for s3, azure, GCP to be able to test those external services, feel free to point me where to add those creds and I will do it similar to CircleCI
[ ] To fix the issues, @edogrigqv2 do you think we should get rid of lmdb dependency?
[ ] ubuntu 64x has this issue AttributeError: module 'hub.api' has no attribute 'objectview'
[ ] AttributeError: module 'numcodecs' has no attribute 'MsgPack'
[ ] c:\hostedtoolcache\windows\python\3.7.9\x64\lib\posixpath.py:248: ModuleNotFoundError on windows
[ ] assertion errors on windows regarding in unit8
[ ] ModuleNotFoundError: No module named 'pwd' not found on windows
[ ] More issues.

@ADI10HERO there are two points, which I want to add:

Why are you running once with and without optional dependencies the same test because it pyetest runs in both cases the exactly the same tests. I would recommend if that not cause some of your fail tests.
Also I can recommend adding https://github.com/darrenburns/pytest-clarity which provides a better visualization of the fail tests.

@ADI10HERO there are two points, which I want to add:

Why are you running once with and without optional dependencies the same test because it pyetest runs in both cases the exactly the same tests. I would recommend if that not cause some of your fail tests.
Also I can recommend adding https://github.com/darrenburns/pytest-clarity which provides a better visualization of the fail tests.

We removed 3.9 and pytests in order to test installation on cross-platform. @Anselmoo will add two feature requests to make sure Hub is properly working across all OS and 3.9 python.


Memory leak when using back button to switch between pages

Reproduction
https://stackblitz.com/edit/vitejs-vite-mnz3ja?file=src%2FApp.vue
Steps to reproduce the bug
Tested in Chrome 131 in incognito mode
Vue version: 3.5.12
Vue router version: 4.4.5

Build the app and start it:

npm run build
npm run preview

The issue is reproducible in dev and production mode
Steps to reproduce:
Open DevTools → Memory

go to /
click on the first link
Classic Margherita Pizza (#1)
click back (browser button)

You should be on the home page
Click gc, and start profiling with Allocations on the timeline.
Then:

go to a recipe that wasn't visited before
go back (via a browser button)
repeat steps 1, 2 two-three times
go to /
go to page 3 (link at the top of the page)

Run garbage collection again.
Then stop memory profiling
Expected behavior
When navigating:

from home page (1) to recipe page (2)
from recipe page (2) to home page (3)

Memory allocated for home page (1) should be cleared
Actual behavior
Memory allocated for home page (1) isn't cleared
DevTools screenshot:

Additional information
When wrapping RouterView in Suspense:

  <Suspense>
    <RouterView v-slot="{ Component }">
      <component :is="Component" />
    </RouterView>
  </Suspense>

The memory is cleared after navigation

When lazy loading recipe in /pages/recipes/[id].vue
const recipe = ref(null);

(async () => {
  recipe.value = await apiRecipeGetById({ id, router });
})();

The memory is cleared after navigation

Originally tried reporting in https://github.com/nuxt/nuxt/issues/29854

i think Suspense should be within RouterView. Does it still reproduce if so?
https://vuejs.org/guide/built-ins/suspense.html#combining-with-other-components

When Suspense is inside RouterView:

  <RouterView v-slot="{ Component }">
    <Suspense>
      <component :is="Component" />
    </Suspense>
  </RouterView>

The memory seems to be retained
Moving Suspense outside RouterView resolves the issue, but vue docs do say to put Suspense inside

Have you ensured this doesn't happen with a dynamic component within Suspense (without Vue Router)?

I played around with this a little today. I'm not sure I have a proper solution, but what I found may be interesting.
If I change the imports for the route components to be asynchronous, and change nothing else, you can see the memory for each recipe page is released when clicking the back button.
// main.js

- import IndexView from './pages/index.vue';
- import PageThree from './pages/3.vue';
- import RecipeId from './pages/recipes/[id].vue';

const routes = [
-  { path: '/', component: IndexView },
+  { path: '/', component: async () => await import('./pages/index.vue') },
-  { path: '/3', component: PageThree },
+  { path: '/3', component: async () => await import('./pages/3.vue') },
-  { path: '/recipes/:id', component: RecipeId },
+  { path: '/recipes/:id', component: async () => await import('./pages/recipes/[id].vue') },
];

I think this may be because your actual page components, linked to the route, are asynchronous, rather than this being handled by async components inside the router view itself, and the nested Suspense is not waiting for Component to resolve.
You can see in Nuxt they actually implement the internal RouterView a bit differently with suspense logic, etc.
I believe with the setup you have in the reproduction, you technically should have two <Suspense> components, with the child instance also having the suspensible prop set to true so that it will independently wait for its child nodes to resolve, rather than defering to the outermost Suspense parent; essentially something like this:
<template>

    <Suspense>
      <RouterView v-slot="{ Component }">
        <Suspense suspensible>
          <component :is="Component" />
        </Suspense>
      </RouterView>
    </Suspense>

</template>

With this template change in App.vue, and reverting the changes in main.js (the await imports) you can see that the memory is again properly released when navigating back from the recipe page.
@danielroe @posva this feels like something that should be solved by documentation, but I'll defer to you all in case I missed something 🍻

I managed to reproduce this without vue router

Vue only reproduction
code: vue playground
Reproduction steps:
1. click Next button (top of the page)
2. click Previous button (top of the page)
Open DevTools → Memory.
Start Allocation instrumentation on timeline
1. click next
2. click previous
3. click next
4. click on any link
5. click previous
6. click on any link
7. click next
8. click previous
For navigations 3. and 5. memory is not cleared. These are the steps where the links were clicked
For other navigations (1., 2., 7., 8.) memory is cleared

I think the issue is due to https://issues.chromium.org/issues/376777343#comment9.
Similarly to https://github.com/vuejs/core/issues/12306#issuecomment-2458809020

For me changing component imports to be asynchronous and wrapping RouterView with Suspense didn't resolve the issue.
Wrapping component :is="" with a div in App.vue:
<template>

    <RouterView v-slot="{ Component }">
      <Suspense>

          <component :is="Component" />

      </Suspense>
    </RouterView>

</template>

Seems to resolve the issue. I'm not sure why

For me changing component imports to be asynchronous and wrapping RouterView with Suspense didn't resolve the issue.

It resolved for me 🤔 but not doing both.

Wrapping component :is="" with a div in App.vue:
<template>

    <RouterView v-slot="{ Component }">
      <Suspense>
+

          <component :is="Component" />
+

      </Suspense>
    </RouterView>

</template>

Seems to resolve the issue. I'm not sure why

I did notice a warning in your original reproduction from Suspense saying it expects a single child.

Thanks for taking a deeper look. Since this can be reproduced without the router and it doesn't seem to come from it, I will close this.
FYI when I tested on Edge, I don't always see a memory leak.


Hitting fake notes can result in scores over 100%

Hitting fake notes in a chart makes the score higher - with good accuracy sometimes over 100%. This error is only local though, in EtternaOnline the score seems correct.
To Reproduce
Steps to reproduce the behavior:

Play a chart with fakes that you can play accurately (like >99%)
Hit the fakes accurately as well
Watch the score rise to over 100%

Expected behavior
I would expect the game to totally ignore hitting fakes.
Screenshots

Fakes are completely impossible to hit in gameplay rewrite.
Fake notes are ignored by the calculator, but taps within fake regions are not. Still needs work.
Primary issue fixed by 140af604678120ed17322bdc6ec92786d0b972ef
closed when merged


Various Derby URLs fail in core.jdbc/connection

e.g.
user=> (with-open [conn (jdbc/connection "derby:/path/to/db;user=dot;password=D^1@?pw*qwK")] (jdbc/execute conn "insert into foo values ('hello')"))
URISyntaxException Illegal character in path at index 37: derby:/path/to/db;user=dot;password=D^9@?pw*qwK  java.net.URI$Parser.fail (URI.java:2848)

user=> (with-open [conn (jdbc/connection "derby:memory:derby2pg;create=true")] "create table foo(bar varchar(10))")
SQLException No suitable driver found for jdbc:derby://  java.sql.DriverManager.getConnection (DriverManager.java:689)

Both of these urls work fine e.g.
user=> (with-open [conn (DriverManager/getConnection "jdbc:derby:/path/to/db;user=dot;password=D^9@?pw*qwK")] )
nil

user=> (with-open [conn (DriverManager/getConnection "jdbc:derby:memory:foobacle;create=true")] )
nil

In the first case, it looks like the code that attempts to make the connection is passing the string to java's URI class and fails because of some character in the password. This isn't that big of a deal for me to work around, because I can create the connections with DriverManager and just pass them on to the library. However, it seems like rather than forcing users of the library to do that, the library should not try to build a URI out of the raw string and simply create the connection based on the raw string.

Thanks!


Third Party Application Access

Listed below is the current third-party applications that have access to the team. We'd love to clean these up. Could the people would enabled this third-party access chime in with what the access is used for and if the integration can be removed?

Travis CI for Open Source: @deiu
Gitter Public Repo Access: @dmitrizagidulin
Gitter Webhooks: @dmitrizagidulin
ZenHub: @dmitrizagidulin
Slack: @dmitrizagidulin
Coveralls: @dan-f
Coveralls Pro: @deiu
Docker Hub Registry: @deiu
Snyk: @dmitrizagidulin
Sourcerer: @RubenVerborgh
Gitter Private Repo Access: @justinwb
GitKracken: @megoth
Docker Hub Builder: @michielbdejong
Travis CI: @RubenVerborgh
Zenodo: @RubenVerborgh
LifeScope: @timbl
Codecov: @RubenVerborgh

Hi @jaxoncreed, as far as I'm concerned none of those integrations might actually still be in use. You can safely remove them.

I have a kanban board just for myself that uses Zenhub. We're not actually actively assigning tasks and stuff at the moment (though I think we should), so it isn't very active, but it is a nice-to-have as I juggle many different projects.

Docker Hub Builder can be removed. We used it to build test-suite images but since we're on a freebie account at Docker that's no longer supported. We now build those docker images locally on our laptops and then push from there to Docker Hub.

All of mine can be removed. (Though let me know if something breaks about our Gitter :) )

Following up on:

Slack: @dmitrizagidulin

All of mine can be removed. (Though let me know if something breaks about our Gitter :) )

Removed.

Travis CI: @RubenVerborghCodecov:

Removed.

Codecov: @RubenVerborgh

Removed.
Also removing unlisted Jira and Sonar as they are not used or documented by solid org.

To chime in for my Integration with GitKraken: I still use it to handle my Git needs, so please don't remove it.


runbooks website uses inappropriate SSL certificate

the website URL is https://runbooks.starkandwayne.com while the SSL certificate is issued to  *.run.pivotal.io

it is fixed now


Creating branch issues

Too many conflicts in code that wasn't R so copied this manually from a clean copy of main


resolved issue #2542 with position of map icon

Removed MapIcon from right card and added it to the bottom card
Fixes #2598 Put map component in bottom sheet of upload activity
Changes made

Modified upload_bottom_card.xml
Removed toggle functions for right card layout
Accordingly modified UploadActivity.java, UploadPresenter.java, UploadView.java

Codecov Report

Merging #2601 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #2601      +/-   ##
=========================================
+ Coverage     2.7%    2.7%   +<.01%
=========================================
  Files         258     258
  Lines       12283   12275       -8
  Branches     1113    1112       -1
=========================================
  Hits          332     332
+ Misses      11925   11917       -8
  Partials       26      26

Impacted Files
Coverage Δ

...va/fr/free/nrw/commons/upload/UploadPresenter.java
11.17% <ø> (+0.24%)
:arrow_up:

...ava/fr/free/nrw/commons/upload/UploadActivity.java
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@maskaravivek @nicolas-raoul @ujjwalagrawal17 The PR is complete and up for review. Can you please review :)

@vanshikaarora Can you attach a screenshot?

@vanshikaarora Can you resolve the merge conflicts?

Sure  @maskaravivek :) I just made the changes and the PR is ready for review :)

@vanshikaarora Can you resolve the merge conflicts?

@maskaravivek I have resolved the merge conflicts Kindly review :)

@vanshikaarora Can you attach a screenshot?

@maskaravivek here is a screenshot for the current position of map icon

Thanks for approving the PR @maskaravivek  :)


RandomState based on xorshift*

As mentioned in #514, the Random module was rewritten using RandomState and the faster xorshift * algorithm was adopted.

The random number generator that originally used mt19937 will be migrated to moonbitlang/x later on.

The next implementation transition will be Lagged Fibnoacci, which will be completed later by @CAIMEOX.


Libraries/Geolocation/RCTLocationObserver.m 208-222 line commented. I…

…ssue: https://github.com/facebook/react-native/issues/19057
Thank you for sending the PR! We appreciate you spending the time to work on these changes.
Help us understand your motivation by explaining why you decided to make this change.

Test Plan
navigator.geolocation.getCurrentPosition stop working after we add NSLocationAlwaysUsageDescription and NSLocationWhenInUseUsageDescription to plist.

Related PRs
https://facebook.github.io/react-native/docs/geolocation.html already update.

Release Notes

Apple stopped accept applications without NSLocationAlwaysUsageDescription description in .plist file.
[CATEGORY] [TYPE] [LOCATION] - Message

IOS Apple do not accepting apps without NSLocationAlwaysUsageDescription description.

You should remove the code rather than comment it out.
Your test plan seems incomplete. How did you test that this fixes the issue?

Closing after several months without a test plan. Feel free to send another PR with a description of how the change was tested, if you're interested in landing this change.


Parsing a different custom model

First of all, many thanks for your work, this area seems not very talked about and your work has been invaluable to my progress.
I have had a look at the provided ONNX model and the code that is used to parse and predict with it.
I am now looking to modify this to work for my own ONNX model, but have found one major difference in the model shape, and I am not sure how to get around this issue.
I have an extra "output" with shape [1, 25200, 20] and I'm unsure what to do with it.
Also the rest of the output is of a slightly different format [1,3,80,80,20] vs [1, 52, 52, 3, 85] for example.
See my netron output of my ONNX

I exported my model from a Yolov5 format using the export.py from Ultralytics, as follows:
sudo python3 export.py --data  <.yaml> --weights <.pt> --img 640 --batch 1 --opset 9 --include onnx --simplify
Any help or guidance anyone can give would be greatly appreciated.
Edit: I am looking at parsing and inference in C#

Hi @Westycoot, thanks for your message.
Just to make sure, did you have a look at the below (from the readme)?
YOLO v5 in ML.Net
Use YOLO v5 with ML.Net
Thanks to raulsf6, deanbennettdeveloper and keesschollaart81

See the discussion in the Tips for modifying this for YoloV5 issue.
See the yolo-v5-incl branch
See keesschollaart81's gist

The yolo v5 model you'll find in this branch look to be similar yo yours, with the  [1, 25200, xx] layer.
Let me know if this is useful

Hi @BobLd after some work, I realised I had pulled the wrong project in and now have detection working using the above. Many thanks for your hard work. I have now tried porting this to Hololens, where I wish to do the detection, but it seems the onnxruntime isn't supported by this device. So I am now looking into Windows.AI.MachineLearning to do this.
While this is separate to this issue I have posted above, do you know how the model format version is produced? For example my version is ONNXv7 but I need it to be v4 or v5.
Many thanks.

Hi @Westycoot, I think what your are refering to is the opset_version. When you convert your model to onnx format, you should be able to set the version, e.g.:
    torch.onnx.export(model,
                      [image],
                      model_path,
                  ->  opset_version=11,
                      do_constant_folding=True,
                      ...)

I've used that here.
Also, I guess you might be able to translate one onnx model to another in a different opset_version, I've never done that. Maybe try to google that

Hi @Westycoot try setting offset version to 12 .....currently ML.Net framework does not support opset version 13
The inference script is pretty similar to yolov5 small you will just need to change output dimensions at a few places in the files
You can safely ignore those extra output dimensions like 454, 495 and 536 if you only want the box predictions
Please refer to this inference script to get an idea https://github.com/BobLd/YOLOv4MLNet/blob/yolo-v5-incl/YOLOv4MLNet/DataStructures/YoloV4Prediction.cs


Blue green deployment

Hey @bharat-hexad, I'm not sure what this is...

Hey @bharat-hexad, I'm not sure what this is...

I am so sorry for this. The PR got sent to you by mistake.Closing it

@bharat-hexad no worries at all! Have a great day.


Login Screen

Is it possible to activate the login screen? Cheers, Dave

It's already active, but we bypass it by default for you. If you navigate to http://localhost:/ui/_login, you can get to it. You could also run edgedb ui --print-url and just copy the portion of the URL up to the ?.
Closing now, but feel free to re-open if that doesn't fully address your question.


ImportError: ('requires pygraphviz ', 'http://pygraphviz.github.io/')

Rasa Core version:0.11.3
Python version: 3.6.4
Operating system (windows, osx, ...):Windows 10 64xbit OS
Issue:I'm already installed pygraphviz1.5 but when i try to run sample rasa_core concert-bot visualize.py file I'm facing following error
C:\Users\Aarbor 01\rasa_core\examples\concertbot>python visualize.py
C:\ProgramData\Anaconda3\lib\site-packages\pykwalify\core.py:99: UnsafeLoaderWarning:
The default 'Loader' for 'load(stream)' without further arguments can be unsafe.
Use 'load(stream, Loader=ruamel.yaml.Loader)' explicitly if that is OK.
Alternatively include the following in your code:
import warnings
warnings.simplefilter('ignore', ruamel.yaml.error.UnsafeLoaderWarning)
In most other cases you should consider using 'safe_load(stream)'
data = yaml.load(stream)
Processed Story Blocks: 100%|███████| 7/7 [00:00<00:00, 7265.56it/s, # trackers=1]
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\lib\site-packages\networkx\drawing\nx_agraph.py", line 132, in to_agraph
import pygraphviz
File "C:\Users\Aarbor 01\AppData\Roaming\Python\Python36\site-packages\pygraphviz_init_.py", line 58, in
from .agraph import AGraph, Node, Edge, Attribute, ItemAttribute, DotError
File "C:\Users\Aarbor 01\AppData\Roaming\Python\Python36\site-packages\pygraphviz\agraph.py", line 22, in
from . import graphviz as gv
File "C:\Users\Aarbor 01\AppData\Roaming\Python\Python36\site-packages\pygraphviz\graphviz.py", line 28, in
_graphviz = swig_import_helper()
File "C:\Users\Aarbor 01\AppData\Roaming\Python\Python36\site-packages\pygraphviz\graphviz.py", line 24, in swig_import_helper
_mod = imp.load_module('_graphviz', fp, pathname, description)
File "C:\ProgramData\Anaconda3\lib\imp.py", line 243, in load_module
return load_dynamic(name, filename, file)
File "C:\ProgramData\Anaconda3\lib\imp.py", line 343, in load_dynamic
return _load(spec)
ImportError: DLL load failed: The specified module could not be found.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "visualize.py", line 15, in
output_file="graph.png", max_history=2)
File "C:\ProgramData\Anaconda3\lib\site-packages\rasa_core\agent.py", line 602, in visualize
should_merge_nodes, fontsize)
File "C:\ProgramData\Anaconda3\lib\site-packages\rasa_core\training\visualization.py", line 323, in visualize_stories
persist_graph(graph, output_file)
File "C:\ProgramData\Anaconda3\lib\site-packages\rasa_core\training\visualization.py", line 228, in persist_graph
expg = nx.nx_agraph.to_agraph(graph)  # convert to a graphviz graph
File "C:\ProgramData\Anaconda3\lib\site-packages\networkx\drawing\nx_agraph.py", line 135, in to_agraph
'http://pygraphviz.github.io/')
ImportError: ('requires pygraphviz ', 'http://pygraphviz.github.io/')
Content of domain file (if used & relevant):

What was the command you ran to install pygraphviz?

python setup.py install --user --include-path="C:\Program Files\graphviz-2.38\include" --library-path="C:\Program Files\graphviz-2.38\lib"  @akelad

what setup.py is this?

its pygraphviz-1.5 set.py file

can you try importing pygraphviz from the python command line?

Hi, I have the same issue.
Akelad, do you ask to use "pip install pygraphviz"? If it's the case, then it's not working on windows and that's why Aarbor1 used this way to install pygraphviz.
But even if it's not with pip, when using "pip show pygraphviz", we correctly see that pygraphviz 1.5 is installed, but we're facing the issue mentionned above when trying to use the interactive learning command (or anything asking for visualisation).

then what is the solution for it.

There's a lot of problems with pygraphviz in Windows, in fact for some versions of Windows it's entirely incompatible. I'm not sure there is a solution to this, perhaps someone has found a solution on Stackoverflow before...
Though, what error are you getting for the interactive learning command? That shouldn't require that.

Hi,
Sorry for the delay, here is the error I got when lauching the interactive learning
Traceback (most recent call last):
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\networkx\drawing\nx_agraph.py", line 133, in to_agraph
    import pygraphviz
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\pygraphviz\__init__.py", line 58, in <module>
    from .agraph import AGraph, Node, Edge, Attribute, ItemAttribute, DotError
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\pygraphviz\agraph.py", line 22, in <module>
    from . import graphviz as gv
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\pygraphviz\graphviz.py", line 28, in <module>
    _graphviz = swig_import_helper()
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\pygraphviz\graphviz.py", line 24, in swig_import_helper
    _mod = imp.load_module('_graphviz', fp, pathname, description)
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\imp.py", line 243, in load_module
    return load_dynamic(name, filename, file)
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\imp.py", line 343, in load_dynamic
    return _load(spec)
ImportError: DLL load failed: %1 n’est pas une application Win32 valide.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\train.py", line 239, in <module>
    additional_arguments)
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\train.py", line 185, in train_dialogue_model
    training_data = agent.load_data(stories_file, **data_load_args)
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\agent.py", line 472, in load_data
    debug_plots)
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\training\__init__.py", line 56, in load_data
    debug_plots)
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\training\generator.py", line 168, in __init__
    self.story_graph.visualize('story_blocks_connections.pdf')
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\training\structures.py", line 546, in visualize
    visualization.persist_graph(graph, output_file)
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\rasa_core\training\visualization.py", line 228, in persist_graph
    expg = nx.nx_agraph.to_agraph(graph)  # convert to a graphviz graph
  File "C:\Users\a.vescera\AppData\Local\Programs\Python\Python36\lib\site-packages\networkx\drawing\nx_agraph.py", line 136, in to_agraph
    'http://pygraphviz.github.io/')
ImportError: ('requires pygraphviz ', 'http://pygraphviz.github.io/')


Installation Bug on MacOS Big Sur - Binary cannot be verified.

I'm opening this issue because:

[X] I have found a bug

My Go version is: (go version go1.16.4 darwin/amd64)

My GOPATH is set to: (/Users/shuvayan/mygo)

During the installation of qrcp on MacOs Big Sur,
I downloaded the binary and extracted and put the binary in path, after doing chmod +x.
However on running
qrcp --help

MacOs comes with a prompt
"qrpc" cannot be opened as the developer cannot be verified.

Thats not a bug but Apple trying to sell expensive stuff.
Please follow https://github.com/hashicorp/terraform/issues/23033#issuecomment-540417305

Hey thanks it worked.


Fix generateShortOrCompleteClassname

Looks good.
Can you also add a test for this case so that I don't make this mistake again? Thanks!
I hope this one gets patched fast :)

As mentioned by @veewee, a test case is strictly required for fixes to be merged.

Yes! Nice to see it fixed so quick!

@qRoC I really want to get this in, but I really need a test case, for a couple of reasons:

I don't know how to reproduce the issue.
The project needs to ensure it doesn't break in the future.

Any chance you can get a test in ASAP so I can do a 3.0.4 release, please?

yes


i2c_master_slave/i2c_master_slave_dma examples have undocumented requirements

The examples are designed to run without external hardware, but they still seem to require pull-up resistors (which makes sense). At least I think this is what makes the difference. The only board I can get them to run on is part of a non-trivial test stand, so I can't be sure.
The solution here is to verify that pull-up resistors are actually what's missing (10 kOhm on the SCL and SDA pins should work) and either document that in the examples, or figure out if the built-in pull-up resistors can be configured in software (never tried that).

This is probably also the case for the i2c_eeprom example.


GPG encryption

Add the ability to encrypt file(s) with GPG before uploading.
Change Details

Added two environment variables:

NEW: GPG_ENABLE

Description: Toggle the GPG feature.
Default: FALSE

NEW: GPG_PUBKEY

Description: Path to the public key for HPH encryption.

CHANGE: ZIP_PASSWORD

Default: "WHEREISMYPASSWORD?" -> "" (empty str)
Reason: It is no longer necessary to be compulsory as GPG encryption is an alternative.

P.S. I'm not quite familiar with shell script. Please feel free to make any further changes.

Thank you for your pull request.
There's another matter I believe is worth discussing, regarding the environment variable GPG_PUBKEY.
The environment variable GPG_PUBKEY specifies the path to the GPG public key, and users are required to mount the GPG public key file into the container when using it. However, I think it might be preferable to have the public key path determined internally rather than by the user.
Additionally, the /config/ path is defined as the configuration folder for rclone. While placing the GPG public key inside it is feasible, the provided docker-compose.yml file already directs /config/ to the vaultwarden-rclone-data volume, which could lead to confusion. Therefore, specifying it as /gpg/key.pub might be a better choice.
Furthermore, since containers can be launched using non-root users, mounting a file could present permission issues for these users.
So, I'm wondering if it's possible to have GPG_PUBKEY filled with the content of the public key rather than the file, and then use a script to write this content into the container. This approach could potentially avoid permission issues. Users who prefer to mount the file directly would need to handle any permission problems themselves. I'm not familiar with the gpg command, so I don't know whether it's possible to use the public key content directly instead of a public key file.
We can discuss this matter first and come to a solution. I would like to handle the implementation, and then you can assist with the review afterward.

@ttionya Fair enough. Filling GPG_PUBKEY with the public key directly sounds good. Although it seems gpg doesn't accept inline key content directly, I reckon it wouldn't be difficult to save the key as a file with cat in the script.
Again, I'm not really familiar with all these either. I just feel it would be great if this project could have GPG encryption, as all contents in a password manager backend should be considered sensitive data, while some of them (e.g. attachments) are not internally encrypted by Vaultwarden.

Alright, I will modify GPG_PUBKEY to accept a public key string instead of a file path.
Once you've made the changes based on the review, I'll go ahead and merge this.

@kongwy ,
I noticed that this pull request has had no new commits over 10 days. Are you willing to make code changes based on the review feedback?

@ttionya Is there a plan to advance this PR? GPG encryption is very useful for me, I am looking forward to it. Thank you for everything you've done. Thanks.

Moved to PR #152 .


[doc][Parameters-Tuning] Fix the document about num_leaves in case of max_depth=6

https://lightgbm.readthedocs.io/en/latest/Parameters-Tuning.html

For example, when the max_depth=6 the depth-wise tree can get good accuracy, but setting num_leaves to 127 may cause over-fitting, and setting it to 70 or 80 may get better accuracy than depth-wise.

Where does this number 127 come from?
Other sentence says:

we should let it be smaller than 2^(max_depth)

2^6 = 64. 70 and 80 are larger than 2^(max_depth). The sentence doesn't make sense.
I think it should be:

For example, when the max_depth=6 the depth-wise tree can get good accuracy, but setting num_leaves to 63 may cause over-fitting, and setting it to 40 or 50 may get better accuracy than depth-wise.

welcome to create a PR to fix it : )

Thanks for your feedback. I wanted to confirm that before creating a PR.  I will try.

Merged https://github.com/Microsoft/LightGBM/commit/1a1262cceb1121287eb44c60c491f0dbaee8f015


Added PR template and Issue template

This PR contains addition of PR template and issue template suitable for this project and I have moved the previous PR template into the .github folder and renamed it to PR template2.
This PR is in reference to #31
Kindly review and merge this PR :)

There was some semantic error with the PR so I closed it. I am resending the PR ASAP

There was some semantic error with the PR so I closed it. I am resending the PR ASAP


Integrate Codecove

https://codecov.io

After investigate found out we cannot run OpenCover, which required by Codecov; on other platform than Windows. So we need #123 to be done first so we have CI for Windows.


E0d/add discourse

Adding Discourse to the metrics dashboard.

@alangsto if you have a moment I'd appreciate a quick look at this code to add Discourse to the metrics dashboard that you created.

@alangsto if you have a moment I'd appreciate a quick look at this code to add Discourse to the metrics dashboard that you created.


Add "UI Output", "Console" and "Docs" labels to playground

Similar to embeds:

@johnpryan Can this one be closed out?

yep!


[Question] Priority with Enhancements

I see a lot of enhancement requests coming in. FanOut, Facebook, etc.
I am still learning, but I hope to participate when I get a better grasp of everything. Until then, do we have some way to prioritize the backlog? Is there any way to upvote enhancements (user voice) or something?
For example, I would put FaceBook before FanOut, due to the amount of exposure it would garner the project.

UserVoice is here

I moved all of the enhancement requests to UserVoice so they can be voted on by the community.


no javascript in data, only on Android, works on iOS

javascript in data like this
      "data": {
        "infive": "{{var now = new Date(); var then = new Date(now.getTime()+5*60000); then = then.toISOString(); return then;}}"
      }

works fine in iOS, but not in Android.

I just tried this, and it works. There is really no reason it should work on iOS but not on Android, so if this is true we really need to fix it. But I need more info since I can't replicate this.
Please share:

Your full code (that's working on iOS but not on Android)
Screenshot of the app not working (with the console logs)
Your OS/device spec

p.s.
In most cases where something works on iOS but not on Android, it's because the JSON is not in a valid format. The iOS is very forgiving when parsing JSON but Android is very strict, so in many cases if you have invalid JSON it wouldn't even get to the point of render. You could check to make sure the JSON is in a valid format by trying JSON Lint.

Ethan, thanks.
My JS is correct, my problem is: there is no javascript in head/data, on Android only.

Hello @j2l. Can you post a minimal json app to reproduce the bug ?
@gliechtenstein needs a simple way to reproduce it. I can look at it too but we need the json app.

@realitix, simply paste data in the head and call it from template or root body.
Test it on both iOS and Android.
Thanks

@j2l I have tried that
{
  "$jason": {
    "head": {
      "data": {
        "infive": "{{var now = new Date(); var then = new Date(now.getTime()+5*60000); then = then.toISOString(); return then;}}"
      },
      "title": "JS demo",
      "actions": {
        "$load": {
          "type": "$util.alert",
          "options": {
            "title": "test",
            "description": "{{infive}}"
          }
        }
      }
    }
  }
}

And it displays the content of infive (without interpreting javascript code). I don't understand what you are trying to achieve. Please paste your json, we can't help otherwise...

now test it on iOS and report.

@j2l please post the full JSON. Because a lot of the ways you use the JASON markup is recursive, the context really matters. The same piece of data object can function completely differently depending on how it's used with others.

I tested before writing a ticket with complex and simple contexts.
Why don't you test it on iOS to see result?
Here's a full JSON, to test on iOS:
{
  "$jason": {
    "head": {
      "data": {
        "infive": "{{var now = new Date(); var then = new Date(now.getTime()+5*60000); then = then.toISOString(); return then;}}"
      },
      "title": "JS demo",
      "actions": {
        "$load": {
          "type": "$util.alert",
          "options": {
            "title": "test",
            "description": "{{infive}}"
          }
        }
      }
    }
  }
}

Spoiler:

En même temps, je fais au plus court : titre et premier texte en une ligne.
Mais je teste tout avant, donc faut juste tester pour comprendre :)
An eval() should do the trick but it must have a secure way.

@j2l this is an edge case I hadn't thought about when I added the data attribute. The data attribute is meant to be used as just that, a static piece of data. The use case I had in mind was when you're delivering a huge JSON markup from the server, in many cases you can separate them into data and template, reducing the overall file size significantly.
That said, since this is something I haven't thought about before I spent some time to think about whether to officially support this.
Theoretically it's possible to "fix" the android code so that the dynamic expression works, but I don't think it's a good idea because it makes things complicated and I feel like it may be a source of potential bugs. In fact, even on iOS there may be problems if you use this method extensively because it was not meant to work that way.
Plus this is something you can achieve using actions (If you're trying to set a constant variable, you can just use the $set action to set a local variable on $load event).
But I might be wrong and there are indeed cases where this is the only way you can do things. If you are running into a situation where you can't solve the problem this way, please let me know, and I'll try to figure something out.

Nice answer @gliechtenstein. Indeed, it's not documented like that. Maybe you can close the issue for now @j2l ?

I should have called it [low hanging feature].
It shouldn't stay inconsistent.
Javascript is expected to be executed where it is written.


[tool_crash] ProcessException: The directory name is invalid. Command: C:\opt\flutter_windows_2.0.5-stable\flutter\bin\cache\dart-sdk\bin\pub.bat, OS error code: 267

Command
flutter doctor

Steps to Reproduce

Followed the instruction on https://flutter.dev/docs/get-started/install/windows
Run flutter doctor

Logs
ProcessException: The directory name is invalid.
Command: C:\opt\flutter_windows_2.0.5-stable\flutter\bin\cache\dart-sdk\bin\pub.bat, OS error code: 267
#0      _ProcessImpl._start (dart:io-patch/process_patch.dart:390:33)
#1      Process.start (dart:io-patch/process_patch.dart:36:20)
#2      LocalProcessManager.start (package:process/src/interface/local_process_manager.dart:39:20)
#3      ErrorHandlingProcessManager.start.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:638:33)
#4      _run (package:flutter_tools/src/base/error_handling_io.dart:532:20)
#5      ErrorHandlingProcessManager.start (package:flutter_tools/src/base/error_handling_io.dart:638:12)
#6      _DefaultProcessUtils.start (package:flutter_tools/src/base/process.dart:472:28)
#7      _DefaultProcessUtils.stream (package:flutter_tools/src/base/process.dart:491:35)
#8      _DefaultPub.batch (package:flutter_tools/src/dart/pub.dart:282:34)
<asynchronous suspension>
#9      _DefaultPub.get (package:flutter_tools/src/dart/pub.dart:222:7)
<asynchronous suspension>
#10     PubDependencies.update (package:flutter_tools/src/cache.dart:774:5)
<asynchronous suspension>
#11     Cache.updateAll (package:flutter_tools/src/cache.dart:562:9)
<asynchronous suspension>
#12     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1109:7)
<asynchronous suspension>
#13     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1009:27)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#15     AppContext.run (package:flutter_tools/src/base/context.dart:149:12)
<asynchronous suspension>
#16     CommandRunner.runCommand (package:args/command_runner.dart:197:13)

[✓] Flutter (Channel stable, 2.0.5, on Microsoft Windows [Version 10.0.19042.928], locale en-CH)
    • Flutter version 2.0.5 at C:\opt\flutter_windows_2.0.5-stable\flutter
    • Framework revision adc687823a (6 days ago), 2021-04-16 09:40:20 -0700
    • Engine revision b09f014e96
    • Dart version 2.12.3

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\christian.loosli_wab\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] Connected device (3 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
    • Chrome (web)                • chrome        • web-javascript • Google Chrome 89.0.4389.128
    • Edge (web)                  • edge          • web-javascript • Microsoft Edge 89.0.774.77

! Doctor found issues in 1 category.

Flutter Application Metadata
No pubspec in working directory.

$ where flutter dart
C:\opt\flutter_windows_2.0.5-stable\flutter\bin\flutter
C:\opt\flutter_windows_2.0.5-stable\flutter\bin\flutter.bat
C:\opt\flutter_windows_2.0.5-stable\flutter\bin\dart
C:\opt\flutter_windows_2.0.5-stable\flutter\bin\dart.bat

duplicate of https://github.com/flutter/flutter/issues/80418

My solution was to install the dart sdk. I don't know why this is not mentioned in the installtion guide.
After I installed the dart sdk flutter doctor worked. So make sure you have dart.exe at the end of your path
$ where flutter dart
C:\opt\flutter\bin\flutter
C:\opt\flutter\bin\flutter.bat
C:\opt\flutter\bin\dart
C:\opt\flutter\bin\dart.bat
C:\opt\dart-sdk\bin\dart.exe


Fixed #20889 -- Prevented BadHeaderError when Python inserts newline

Workaround for http://bugs.python.org/issue20747.
In some corner cases, Python 2 inserts a newline in a header value
despite maxlinelen passed in Header constructor.

Can we simplify this when we drop Python 2? Would be helpful to add a comment about that or an item to #23919.

The original issue is indeed solved in Python 3. The fact that we check for newlines before passing the value to Python functions seems more logical to me. Then about the bytes/unicode conditionals, I'm not sure this will change when dropping Python 2.


Fix property wrappers with tuples.

In case of a tuple as value type, the initializer and setter takes the tuple elements as separate arguments. This was just not handled in the assign_by_wrapper instruction lowering.
rdar://problem/53866473

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - aa76b74b02d4fddd8640229a7a525a2f0dc94c9a

Build failed
Swift Test Linux Platform
Git Sha - aa76b74b02d4fddd8640229a7a525a2f0dc94c9a

@gottesmm Do we have any utilities that we can reuse here?

@swift-ci test

@swift-ci test

@slavapestov I had the same thought and did some search in the code base, but couldn't find anything useful (which doesn't mean it does not exist).


Load image from album

hi,
I return to my previous question regarding the use of "UIImagePickerController"
I use "CLEffectTool" to apply a "GPUImageDissolveBlendFilter" filter on the image in the editor.
This filter requires me to provide an additional image (overlay) I wish to load it from my photo album, so I need to use UIImagePickerController instead of Bundle.
I can not find any place I have to call UIImagePickerController and then recover the selected image for use in my filter.
Here is a sample code in my customEffect.m :

(UIImage*)applyEffect:(UIImage*)image
{
inputImage = [UIImage<EMAIL_ADDRESS>//// I will wish that inputImage comes from my photo album
GPUImagePicture *imageSource = [[GPUImagePicture alloc] initWithImage:image];
[imageSource processImage];
GPUImageDissolveBlendFilter *blendFilter = [[GPUImageDissolveBlendFilter alloc]init];
GPUImageExposureFilter *brightNessFilter = [[GPUImageExposureFilter alloc] init];
GPUImageContrastFilter *contrastFilter = [[GPUImageContrastFilter alloc] init];
[(GPUImageDissolveBlendFilter *)blendFilter setMix:_radiusSlider.value];
[(GPUImageExposureFilter *)brightNessFilter setExposure:_brightnessSlider.value];
[(GPUImageContrastFilter *)contrastFilter setContrast:_contrastSlider.value];
[imageSource addTarget:blendFilter];
[imageSource addTarget:brightNessFilter];
[imageSource addTarget:contrastFilter];
[imageSource processImage];
UIImage *quickFilteredImage1 = [blendFilter imageByFilteringImage:inputImage];
UIImage *quickFilteredImage2 = [brightNessFilter imageByFilteringImage:quickFilteredImage1 ];
UIImage *quickFilteredImage = [contrastFilter imageByFilteringImage:quickFilteredImage2 ];
[imageSource removeAllTargets];
imageSource = nil;
blendFilter = nil;
brightNessFilter = nil;
contrastFilter = nil;
return quickFilteredImage;
}

Could you help me with some advice or with some sample code I am beginner and I'm struggling to find the right method, thank you in advance for your attention

I asked this before as well, how do it become if you try changing to setup?

ok I still try and tell you, thank's

Le 12 août 2015 à 02:28, Sho Yakushiji<EMAIL_ADDRESS>a écrit :
I asked this before as well, how do it become if you try changing to setup?
—
Reply to this email directly or view it on GitHub.

UIImagePickerController in setup it's ok,  so   _blendImage = [info objectForKey:UIImagePickerControllerOriginalImage];
_blendImage declared as @property (nonatomic, strong)  UIImage *_blendImage; in CLEffectTool
how to pass _blendImage to  (UIImage*)applyEffect:(UIImage*)image
in CLCustomEffectTool

I think it's not my issue. It is totally up to you. But if it were me I'd probably do it below.
- (id)initWithSuperView:(UIView*)superview imageViewFrame:(CGRect)frame toolInfo:(CLImageToolInfo*)info
{
    self = [super initWithSuperView:superview imageViewFrame:frame toolInfo:info];
    if(self){
        // The rest of your method implementation.

        UIImagePickerController *picker = [[UIImagePickerController alloc] init];
        UIViewController *vc = [[[UIApplication sharedApplication] keyWindow] rootViewController];
        while (vc.presentedViewController) { vc = vc.presentedViewController; }
        [vc presentViewController:picker animated:YES completion:nil];
    }
    return self;
}

i tried then
I had in CLCustomEffect.m
@implementation CLCustomEffect
{
UIImage *_blendImage;
...
}
#pragma mark- ImagePicker delegate

(void)imagePickerController:(UIImagePickerController *)picker didFinishPickingMediaWithInfo:(NSDictionary *)info
{
_blendImage = [info objectForKey:UIImagePickerControllerOriginalImage];

}

(UIImage*)applyEffect:(UIImage*)image
{
GPUImagePicture *imageSource = [[GPUImagePicture alloc] initWithImage:image];

...
GPUImageDissolveBlendFilter *blendFilter = [[GPUImageDissolveBlendFilter alloc]init];
UIImage *quickFilteredImage = [blendFilter imageByFilteringImage:_blendImage];

}
at runtime I get a crash
Assertion failure in -[GPUImagePicture initWithCGImage:smoothlyScaleOutput:],iOS9/_Demo/CLImageEditorDemo/SDKs/GPUImage/framework/Source/iOS/GPUImagePicture.m:74
2015-08-12 11:06:24.404 MyApp[22255:986445] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Passed image must not be empty - it should be at least 1px tall and wide'

Did you write properly also other part? Sample is a minimum code, so it doesn't available by copy and paste.

I checked my code but - (id) initWithSuperView: (UIView *) superview imageViewFrame (CGRect) frame TOOLINFO (CLImageToolInfo *) info
thank you is the right solution

?

it’s ok for me Yackle still have the same issue I will review my code :)
Merci beaucoup :))

Le 12 août 2015 à 12:05, Sho Yakushiji<EMAIL_ADDRESS>a écrit :
?
—
Reply to this email directly or view it on GitHub https://github.com/yackle/CLImageEditor/issues/115#issuecomment-130246332.


image-builder documentation correction in reference/artifacts

What happened: Documentation leaves user in /tmp resulting in permissions errors running image-builder.
What you expected to happen: User to be moved back to their home directory to build image.
How to reproduce it (as minimally and precisely as possible): Follow the image-builder instructions as-is on various sections in this article.
Anything else we need to know?: Encountered this while attempting to build an AMI. Spoke with others in #aws-eks-anywhere-interest internally who saw this issue too. I'm attaching a log file so you can see what you encounter when running in /tmp When switching back to /home/image-builder the issue is resolved. Submitting a fix.
Environment:

EKS Anywhere Release: N/A
EKS Distro Release: N/A

Matt Henslee / @henslma
AWS Professional Services

Fixed in #4556


Rare permission denied error when opening or manipulating worlds

This happens rarely when opening a world too quickly after starting mcedit or sometimes when editing using larger brushes right after loading, going to call this one system specific unless someone confirms it, Windows 7.

When I try to run MCEdit Unified on Linux, I come up with this error in terminal:
Starting MCEdit...
*** APP DIR: /home/mythicalz10n/MCEDITNEW/mcedit-<IP_ADDRESS>
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/tmp/release-master/MCEdit-Unified/main.py", line 38, in
File "/tmp/release-master/MCEdit-Unified/mcedit.py", line 13, in
File "/tmp/release-master/MCEdit-Unified/splash.py", line 5, in
File "/tmp/release-master/MCEdit-Unified/directories.py", line 243, in
File "/usr/lib/python2.7/os.py", line 157, in makedirs
mkdir(name, mode)
OSError: [Errno 13] Permission denied: '/home/mythicalz10n/.mcedit/MCEdit'
Press any key to close.
How do I fix this?


dark-mode-added

solved the issue of dark-mode

I recently made some new changes to the project which is causing this conflict . You can pull the latest changes from this original master branch and push it again after resolving the conflict. If you have any question or need any help ,You can ask me right here.

Okay
On Sat, 26 Sep 2020, 11:59 pm Rocktim Saikia<EMAIL_ADDRESS>wrote:

I recently made some new changes to the project which is causing this
conflict . You can pull the latest changes from this original master branch
and push it again after resolving the conflict. If you have any question or
need any help ,You can ask me right here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RocktimSaikia/Contribute-101/pull/265#issuecomment-699531470,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM7NHZETDCMELQMQKPDUZP3SHYXJRANCNFSM4R3AHPPA
.


[Feature] Pushover.net Notifications

General

[X] My feature does not exist in the latest version of MySpeed.
[X] I have checked that my feature has not been suggested by anyone else.

 I would love to see Pushover as a option for notifications. They use a simple JSON API. Thanks!

Hey, happy to hear that you like the project. I'll take a look at the pushover.net api and implement the integration. Thanks for your suggestion!

Plus 1


Add links section and css

What is contained in this PR:

config for links
generated css for links
link added to editor with controls

Please address this build error too:

I think we're close to merge! Thanks man

@blackmann kindly check again.


add migr file

Commits should be squashed.

I would add to the description an explanation about what's going on in here.
Maybe something like

Created Django apps [head]

Created 3 Django apps: Tasks, Manager, Employees.
Note: most of the files are auto-generated.

In regard to the flake8 errors - when fixing it, I think it will be better to add some dummy code (rather than comment out the lines)

Commits should be squashed.
I would add to the description an explanation about what's going on in here.
Maybe something like

Created Django apps [head]

Created 3 Django apps: Tasks, Manager, Employees.
Note: most of the files are auto-generated.

In regard to the flake8 errors - when fixing it, I think it will be better to add some dummy code (rather than comment out the lines)

sabab


How to calculate the maximum image size

Given the available RAM in GiB, how to calculate the image size  (Maximum side length) ?
Regards

No idea yet, I can give you figures that i've come up with playing with this project so far.
output resoultion - RAM usage
~1280x800 - around 24 GB
~1920x1200 - around 50 GB
I have not testet bigger sizes, it seems that it uses 24 GiB per megapixel of output.

Jakub<EMAIL_ADDRESS>kirjoitti 16.3.2016 kello 15.49:

No idea yet, I can give you figures that i've come up with playing with this project so far.
output resoultion - RAM usage
~1280x800 - around 24 GB
~1920x1200 - around 50 GB
I have not testet bigger sizes, it seems that it uses 24 GiB per megapixel of output.

I made quick tests on smaller image sizes (CPU only), with the following results. Note that I did not wait through all the iterations, only to see that the increase settled down.
Using VGG19, default images, default layers
image size:	resident memory
320			1.7G (peaks at 2.8G during startup)
480	 		3.3G
640			5.7G
720			7.1G
800			8.8G
960			12G
Testing at 800px, changing style scale to 2: 8.8  =>  20G
Testing at 800px, dropping all but one style layer: 8.8G  => 7.6G
Testing with VGG16 (yes, sixteen) with FC layers removed,* using the default layers
800 px		7.9G (from 8.8G with VGG19)
Testing with nin-imagenet-conv, 1 content layer and 3 style layers
800 px		1.8G
640 px		1.2G
Don’t know how these CPU based results correlate with GPU, but from this it appears that with VGG19 running with 2G RAM would fail because of the memory peak during initialization. Running with nin-imagenet should be possible, there seems to be no memory peak but rather a steady increase up to the values given. But nin-imagenet-conv is no direct replacement, it produces different-looking results and requires tweaking the settings.
These results are all using L-BFGS. I found (to my surprise) that the effect of ADAM regarding memory usage was quite small. It seems to me that the smaller networks with L-BFGS work better with lower memory usage than using VGG19 with ADAM.
Hannu
http://liipetti.net/erratic

As far as I could see, the memory reduction was not so much due to the removal of the FC layers but because VGG16 has fewer convolutional layers.


Remove new keyword from style guide

The Flutter style guide uses the new keyword in multiple code examples. It is my understanding that this is no longer needed as of Dart 2 (source).
I suggest that the new keyword can be removed from the code examples in the style guide. For example,

new ListView.builder(...);
new EdgeInsets.TRBL(0.0, 8.0, 0.0, 8.0);
Foo f = new Foo(...);
new PopupMenuItem<String>(...)

could be written as

ListView.builder(...);
EdgeInsets.TRBL(0.0, 8.0, 0.0, 8.0);
Foo f = Foo(...);
PopupMenuItem<String>(...)

@Hixie
What do you think of this proposal?

Removed in https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo/5c279fe8d7b52d31ebcf05385f69c636a6b8e676


List view does not update rows

On refresh the Listview does not update it's values. When looking at logs in the debugger, values are changing so it should prompt the scene to render again with the new data.

Nothing seems to want to update 😓

892b6cc fixes this problem but also created a new problem.


sonarr_series_imdbid

After this commit d9f831f16850c290b2752fb8542d98fd8ff9be06 I keep getting errors when Sonarr is executing postSonarr.py:
20-10-4 22:19:41.5|Error|postSonarr.py|Traceback (most recent call last):
20-10-4 22:19:41.5|Error|postSonarr.py|  File "/scripts/sickbeard_mp4_automator/postSonarr.py", line 97, in <module>
20-10-4 22:19:41.5|Error|postSonarr.py|    imdb_id = int(os.environ.get('sonarr_series_imdbid'))
20-10-4 22:19:41.5|Error|postSonarr.py|ValueError: invalid literal for int() with base 10: 'tt1086761'

I have tested with different episodes and different series and nothing works now.

733570cccf27f8bc25a28f52969b530733c3d23c
That should fix it, my mistake thought it reported imdb id as a number not a string

733570c
That should fix it, my mistake thought it reported imdb id as a number not a string

Tested and everything works again :)

Sweet thanks, good catch


libffi: bump version to fix aarch64 issue #114

@jianshu93 Can you confirm that this works for you?

yes confirmed.
Jianshu


remove unused apigen.neon

Follow up to #161

Coverage decreased (-0.04%) to 79.872% when pulling feb1d126b68a3f2d6587265bdc0f87dd210cc970 on TomasVotruba:patch-1 into 5c01aa20e28355c30aa8de7626c8093e5a44a6a1 on hirak:master.


feat: added example for setting up new emissions per second and distribution end

I would better pin to a current block on Polygon (where LM is active), to skip the setup steps that are not required for both changing emissions and changing the distribution end


Fix for a no-longer valid yaml file

I found a yaml file which no longer satisfies the existing syntax. I put a fix. I set knn to 10, as that's what is mostly used in other yaml files.

Thanks for the update. I'll put the knn value back to 7 as this file was used to generated benchmark results.


vcv-rack: Init at 0.6.2b

Motivation for this change
VCV Rack is an interesting and relatively new Eurorack emulator.
There are a few problems I have with packing it, so your help would be appreciated.
Currently the following points need to be addressed:

[x] Get installation working (there is no make install target, but it builds in a nix-shell)
[x] Check whether all derivation dependencies are really needed
[x] Better names for the overridden dependencies, i.e. with commit date
[x] Check if we have permission to use the VCV name. Edit: I asked via mail and got an OK. Any ideas how to document that permanently?

It would be nice to fix these points:

[ ] Fetch all dependencies via Nix facilities, i.e. don't use fetchSubmodules. This would also allow to specify a tag instead of a revision for fetchFromGitHub
[x] Cleanup/sort derivation arguments
[x] Check if we can update the overridden dependencies in general
[x] Maybe override optimization, as -march=nocona  doesn't match the Nix default
[x] Address glfw error when building against master on nixos-18.03 systems (See https://github.com/NixOS/nixpkgs/pull/42679#issuecomment-400852126)

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

When building against the nix master branch I get an error when I execute the Rack executable.
On my system with the nvidia driver:
./Rack: symbol lookup error: /nix/store/yck3b43w1xg7v7pg9cx3ycms5hgxc7sv-libGL-1.0.0/lib/libGL.so.1: undefined symbol: __GLXGL_CORE_FUNCTION

On my system using the intel driver a get a more arcane error, that no GLXFBConfigs were returned. I will post it asap.
It seems that there is an incompatibility between some gl lib on master and nixos-18.03. I'm not sure though if this is just a result of using a more recent version of glfw in this derivation, or whether it also happens with other packages.

I'm getting an error when I try to run it:
X Error of failed request:  BadAtom (invalid Atom parameter)
  Major opcode of failed request:  20 (X_GetProperty)
  Atom id in failed request:  0x0
  Serial number of failed request:  178
  Current serial number in output stream:  178
terminate called without an active exception
[1]    2629            result/bin/Rack

I'm on latest unstable, happens with both an intel graphics card and an nvidia one, here's the output with -d: https://gist.github.com/e30fe7337f10abb942b24ac8882bc1a2
I'm using xmonad on these systems, not sure if that could be a reason.

@Infinisil You don't have a minimal NixOS config for xmonad on hand by chance that I can run with VirtualBox and/or NixOps? :p
I'm really not sure what could cause that... My issue with the GL driver seems to come from building in a nix shell, i.e. without sandboxing.
With sandboxing it works. Have you tried it with sandboxing enabled and building the package via nix-build?


Ace Security fix PDF link for mod 3 and Mod 4

Ace Security fix PDF link for mod 3 and Mod 4

Ace Security fix PDF link for mod 3 and Mod 4


close subscription even if there is available element in GQ in Drain

Applied in sumbit1 and seems to be working.


Allow for passing of selectedTitleStyle as TabNavigator.Item prop

Thanks @skevy!

Published to npm as 0.2.7.


Update README with min version of gcc.

The gcc-9 mentioned before only had c++17 compatibility as experimental feature, in particular library completeness. Suggest g++ 10 as minimum (which is already very old at this point, so should not be a problem).

Codecov Report
Patch and project coverage have no change.

Comparison is base (7d301d5) 92.85% compared to head (8c8c057) 92.85%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1977   +/-   ##
=======================================
  Coverage   92.85%   92.85%
=======================================
  Files         355      355
  Lines       26272    26272
=======================================
  Hits        24395    24395
  Misses       1877     1877

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add description to index to display commands when hubot help is called

When you type hubot help it doesn't display any standup commands. I've added the normal standup help command to make it easier to find the modules commands for chat room users.

Thanks for the PR!

:+1: no problem!


[firebase_auth]: Sign in with Apple invalid OAuth response

Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
Auth
Which platforms are affected?
iOS
Description
Sign in with apple no longer works when upgrading firebase_auth (5.2.0) and firebase_core (3.4.0) to the latest version via the signInWithCredential function, but works using signInWithProvider and AppleAuthProvider().
The following error appears: [firebase_auth/invalid-credential] Invalid OAuth response from apple.com.
I'm also using the latest version of the "sign_in_with_apple" package.
It works perfectly with the previous version, 5.1.3 for firebase_auth and 3.3.0 for firebase_core.
Reproducing the issue
import 'dart:convert';
import 'dart:math';
import 'package:crypto/crypto.dart';
import 'package:firebase_auth/firebase_auth.dart';
import 'package:sign_in_with_apple/sign_in_with_apple.dart';

String generateNonce([int length = 32]) {
  const charset =
      '0123456789ABCDEFGHIJKLMNOPQRSTUVXYZabcdefghijklmnopqrstuvwxyz-._';
  final random = Random.secure();
  return List.generate(length, (_) => charset[random.nextInt(charset.length)])
      .join();
}

String sha256ofString(String input) {
  final bytes = utf8.encode(input);
  final digest = sha256.convert(bytes);
  return digest.toString();
}

Future<UserCredential> signInWithApple() async {
    final rawNonce = generateNonce();
    final nonce = sha256ofString(rawNonce);

    final appleCredential = await SignInWithApple.getAppleIDCredential(
      scopes: [
        AppleIDAuthorizationScopes.email,
      ],
      nonce: nonce,
    );

    final oauthCredential = OAuthProvider("apple.com").credential(
      idToken: appleCredential.identityToken,
      rawNonce: rawNonce,
    );

    return await FirebaseAuth.instance.signInWithCredential(oauthCredential);
  }

Firebase Core version
3.4.0
Flutter Version
3.24.1
Relevant Log Output
[firebase_auth/invalid-credential] Invalid OAuth response from apple.com

Flutter dependencies

Expand Flutter dependencies snippet

dependencies:
- firebase_analytics 11.3.0 [firebase_analytics_platform_interface firebase_analytics_web firebase_core firebase_core_platform_interface flutter]
- firebase_auth 5.2.0 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 3.4.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_crashlytics 4.1.0 [firebase_core firebase_core_platform_interface firebase_crashlytics_platform_interface flutter stack_trace]
- firebase_messaging 15.1.0 [firebase_core firebase_core_platform_interface firebase_messaging_platform_interface firebase_messaging_web flutter meta]
- firebase_performance 0.10.0+5 [firebase_core firebase_core_platform_interface firebase_performance_platform_interface firebase_performance_web flutter]
- firebase_remote_config 5.1.0 [firebase_core firebase_core_platform_interface firebase_remote_config_platform_interface firebase_remote_config_web flutter]

Additional context and comments
No response

Issue solved. Simply pass accessToken: appleCredential.authorizationCode to the OAuthProvider(“apple.com”).credential() method.

Issue solved. Simply pass accessToken: appleCredential.authorizationCode to the OAuthProvider(“apple.com”).credential() method.

Thanks, it worked for me!

Issue solved. Simply pass accessToken: appleCredential.authorizationCode to the OAuthProvider(“apple.com”).credential() method.

Thank you!


Better check for the existence of window

The current check for window doesn't cover the scenario were window is null:
https://github.com/chaijs/type-detect/blob/master/index.js#L104
Setting window to null bypasses the current window check and errors when trying to read location off of it.
While I would encourage users to set the window to undefined when they're resetting in tests, not everyone will.  This is a small PR to fix that.

Also I've looked through each of the test files and can't see where I can actually test this scenario with the current karma setup for browser tests.  If anyone can offer advice, it's more than welcome!

Hey @christemple thanks for the PR!
This is fair. Don't worry about karma tests - travis and appveyor run a handful of browsers to give us confidence.
If you could rebase your commit message to be prefixed with fix: instead of Type fix: this will help our robots determine how to release this change.

Oh, actually also @christemple while you're rebasing - could you omit the .gitignore change? Ideally if you're running a tool which adds files to the folder which arent ignored - these should go in your global ~/.gitignore - not our project one.

@keithamus I've removed the .gitignore change and amended the commit message :+1:

If @meeber or @lucasfcosta are around they should review this and merge it 😄


Feature Request: Delete Connection

Hello,
Would it be possible to extend the plugin so that existing connections (on video destinations) can be deleted?
Robin

Hey Robin,
thanks for your feature request. What are you looking for exactly?

Delete the connection to one specific destination? This is already possible by selecting the source "Delete connection"(1002) or "No connection to destination"(1001)
Deleting all video connections This is a feature which could be developed.

Cheers Nico

Please response to my question so that I can close this issue or create a feature request 😄

No response yet and there is a way to delete video crosspoints.


No NetworkExtraMemberNode for information=route_marker

This is supposed to solve #302. I followed the steps as in #243.

I have added commits that should allow route markers as route members. This is also common in some regions.


[App] skopeo

It would be nice if arkade supports a skopeo installation.
skopeo is a command-line utility that performs various operations on container images and image repositories.
here is for detail: https://github.com/containers/skopeo

Is the a static binary for it? As I remembered, it needed several C libraries for GPG etc and had to be installed through brew.

yes you are right

Let's close this. And it's weird because I had the same idea as you yesterday and was thinking how we can manage it.
Perhaps printing a message "use brew"
Or finding another similar tool that doesn't need C libraries to work? @mattmoor had a suggestion for a similar tool.

There is more than one tool available for working registries
One from Google :
https://github.com/google/go-containerregistry/blob/master/cmd/crane/doc/crane.md
One from regclient :
https://github.com/regclient/regclient

crane is pure Go, so go get should just work(tm).
There's also a public multi-arch image here:
# crane manifest gcr.io/go-containerregistry/crane | jq .
{
  "schemaVersion": 2,
  "mediaType": "application/vnd.docker.distribution.manifest.list.v2+json",
  "manifests": [
    {
      "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
      "size": 750,
      "digest": "sha256:a7df9d17f321c11f8d14d6ceeb325c7a2e38959109f18e4dd45a7dd2873b07ba",
      "platform": {
        "architecture": "amd64",
        "os": "linux"
      }
    },
    {
      "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
      "size": 750,
      "digest": "sha256:59a61be66dfdee8cfb7d24e64901b2c6eb8b33f140495ac717509ba093fe60c8",
      "platform": {
        "architecture": "arm64",
        "os": "linux"
      }
    },
    {
      "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
      "size": 750,
      "digest": "sha256:24eb901541a7c443f0a1807c00f24e3ba8c0988b80cdb8be3114aedf8678a1f4",
      "platform": {
        "architecture": "ppc64le",
        "os": "linux"
      }
    },
    {
      "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
      "size": 750,
      "digest": "sha256:fa58c6ae7ceb51459c74a40b3787ebf396bed54cf64e3294c572baee37b9615a",
      "platform": {
        "architecture": "s390x",
        "os": "linux"
      }
    }
  ]
}

There is a static binary available on the releases page at this crane repo.
See for macOS:
https://github.com/google/go-containerregistry/releases/download/v0.1.4/go-containerregistry_Darwin_x86_64.tar.gz


fixed governance rules

Looks good :)
governancerules.md should be named README.md to match the other modules.

renamed to README.md


Configuring CausalVAE v1.2 for Training

Hello,
I am currently training a CausalVAE model from scratch. According to the report for CausalVAE v1.2, the previous version's separated up/down temporal/spatial sampling methods have been replaced by a unified approach in v1.2. Could you please provide guidance on how to set the configuration file correctly for CausalVAE v1.2 to ensure it aligns with this unified sampling method? Thanks!
对于Causalvae v1.2要如何配置config.json呢？十分感谢！

https://huggingface.co/LanguageBind/Open-Sora-Plan-v1.2.0/blob/main/vae/config.json

Due to code updates, some parameters in this config file are invalid, but it does not affect usage.

请问您训练CausalVAE v1.2顺利吗？下面这个问题您是怎么解决的呢
Specific Issue: a missing file error

File "Open-Sora-Plan/opensora/models/causalvideovae/model/losses/lpips.py", line 7, in
from ...utils.taming_download import get_ckpt_path
ModuleNotFoundError: No module named 'opensora.models.causalvideovae.utils.taming_download'

It appears that the taming_download module is missing from the opensora.models.causalvideovae.utils directory.


Infra to get pod cgroup metadata

Consolidated the agent changes with the datapath changes - https://github.com/cilium/cilium/pull/20492.


[NG] Fixes datagrid height on page size changes

Closes #1084

Note: I saw some image churn and replaced three images for basic custom cell css regression.
Signed-off-by: Matt Hippely<EMAIL_ADDRESS>

Actually, I do have one question: why the update to the reference screenshots with no visible changes? I thought we were finally past that.

I thought we were finally past that.

You and me both.
There was a 1px shift in only those three images. It looks much like what I saw after we upgraded the angular version to 4.3.0 but before I had actually installed it. What I mean is I brought changes from master that updated package.json to angular 4.3.0 and new reference images but I hadn't installed it yet. Many tests failed much like these until I installed the angular 4.3.0 code.
I'm also having clang issues atm but will update here once tests are happy.

In working through the clang issues I reverted the images and was able to get the regression tests to pass without updating the reference image.


lxcfs: 6.0.0 -> 6.0.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for lxcfs is: FUSE filesystem for LXC
meta.homepage for lxcfs is: https://linuxcontainers.org/lxcfs
meta.changelog for lxcfs is: https://linuxcontainers.org/lxcfs/news/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 6.0.1 with grep in /nix/store/cx2wf4v9z84q695drp5zy3cl4saai774-lxcfs-6.0.1
found 6.0.1 in filename of file in /nix/store/cx2wf4v9z84q695drp5zy3cl4saai774-lxcfs-6.0.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

lxcfs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cx2wf4v9z84q695drp5zy3cl4saai774-lxcfs-6.0.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A lxcfs https://github.com/r-ryantm/nixpkgs/archive/05bd0a0d8aff908ed31d51e0697d4f606ec0200e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/05bd0a0d8aff908ed31d51e0697d4f606ec0200e#lxcfs

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cx2wf4v9z84q695drp5zy3cl4saai774-lxcfs-6.0.1
ls -la /nix/store/cx2wf4v9z84q695drp5zy3cl4saai774-lxcfs-6.0.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

lxcfs

Maintainer pings
cc @aanderse @adamcstephens @jnsgruk @megheaiulian @mkg20001 for testing.

@ofborg build lxcfs lxcfs.passthru.tests

x86
/nix/store/95c4s4jrm40ic1nr6a3j1pa5mkp8j585-vm-test-run-container-legacy-init
/nix/store/gza6l8a793ip8vsqf7y4app5ald4rj83-vm-test-run-container-systemd-init


[libra framework] Remove entity-based freezing.

This PR removes per-entity-based freezing, and we now only have per-account freezing.
If an account is frozen:

Money can't be withdrawn from the account.
Money can't be sent to the account.
Transactions can't be sent from the account.

Closes #4986

[move-prover] Move Prover test (language/move-prover/cargo test) on commit a9bda9e did fail. See details.

Does this happen after rebasing? Perhaps, @emmazzz knows how to fix the precondition violation error in Libra.move?

Although it's unrelated to this PR, it seems that the comments right above LibraAccount::make_account does not match well.

Working on fixing the precondition failures in LibraAccount right now. IIRC the failures in Libra.move are from a different PR.

:umbrella: The latest upstream changes (presumably d8dbbd1fb54980314d3dd25b6cdb021045bd7bc2) made this pull request unmergeable. Please resolve the merge conflicts.

If https://github.com/libra/libra/pull/4972 manages to land, maybe try rebasing--managed to get all prover tests passing there.

@bors-libra r=sblackshear

:pushpin: Commit 723f30d has been approved by sblackshear

:hourglass: Testing commit 723f30d40596479aa1ee9f926c9043a6fd34c09b with merge 51d6489678a601db9f378d7eb914a6095e0cfb5d...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: sblackshear
Pushing 51d6489678a601db9f378d7eb914a6095e0cfb5d to master...


Feature: Multi-GPU support (SLI / NVLink)

I haven't seen SLI on the roadmap, is it doable as part of this project?

Not to speak for anyone at Nvidia or this project's contributors but SLI is deprecated functionality at this point.
So even if it's technically possible to support a deprecated technology on the only two RTX cards capable of it (3090 and 2080); only the 3090 SLI/Nvlink might be able to handle path tracing and even then its probably not competitive with a single 4090. I wonder how many dozens of people are out there that could afford a 3090 SLI setup and haven't already upgraded to the 4090.

Thank you for reaching out. We don't currently plan to implement SLI support with Remix. If you have other questions or suggestions, please open another ticket and we'll be happy to take a look.