next('router') is ignored.

It seems to app is ignoring next('router').
https://github.com/expressjs/express/blob/master/lib/router/index.js#L177 here is layerError is checking for any error or if err is equal to 'router'.
and then below this variable used to skip next routes if layerError is equel to 'router'  https://github.com/expressjs/express/blob/master/lib/router/index.js#L195 but it newer will be equal to this, it will have error message or null;
Is there a typo and should be:
var layerError = err;
or
if (layerError === null && err === 'router') {
  setImmediate(done, null)
  return
}

?

The implementation is correct. They are ignored at certain levels on purpose :+1: For example when you are in a middleware, next('route') does nothing because a route is not being executed. Similar rules for next('router')
They jump out of the current route or router.

So did understand wrong this functionality?
var express = require('express'), app = express(), r = express.Router();

r.get("/new", function( req, res, next ) {
  console.log('static path');
  next('router');
});

// route to trigger the capture
r.get('/:id', function (req, res, next) {
  console.log( 'dynamic path' );
  next();
})

app.use(r);

app.use((req, res) => {
    res.send('end');
});

app.listen(3001, function () { })

And on this code, request on /new should match with both routes?

If code above matches both routes, how can i skip second in first route?

When I run your above code, it logs static bath and skips the dynamic path log then sends the response with end.  Is this not the specific behavior are you looking for?


suspend: Remove dupe kill ${SSH_AGENT_PID}

This kill is already handled by the trap which is setup immediately
after ssh-agent is executed and always executed when this script exits (in
either success or failure).
By having this duplicate kill command at the end of the script, the
process was killed before the trap, and the trap produced this error in the
job output:
/tmp/hudson8936147716169821220.sh: line 1: kill: (457) - No such process

Deployed. I had to re-enable and run the default-dsl-job for the suspend jobs to be updated.


Defining users in xml files passed to spec.configuration.files

My team is currently migrating our local development from running the clickhouse executable on the machine to a kubernetes deployment. We have a set of user xml files we use to define users for local testing. Is there a way to pass these xml files into the cluster configuration? I attempted to use spec.configuration.files, but while the files are present the users do not seem to be created. Is the only path to defining users through the spec.configuration.users YAML property?

try to use
spec:
  configuration:
    files:
      users.d/file_name.xml: |
        <xml content>

That worked, thanks! Is this somewhere in the documentation? Want to make sure to pin it.

look to
kubectl explain chi.spec.configuration.files


Signature overrides other Plugin

Hello,
i got a similar error as in https://github.com/folke/noice.nvim/issues/173 , so i tried to disable it via config.lsp.signature.enabled = false.
when i then go into a file the signature help from the other plugin is not visible anymore

if is is enabled i got following, it seems noice overlays the one from the other plugin (ray-x/lsp_signature.nvim)

Any idea what could be wrong here?
greetings

It's the other way around. The hidden one is Noice, so you did disable Noice in the first screenshot

Also, don't set the overrides, because they might aloso affect other plugins

in the first screenshot i did set :
 require("noice").setup({                                                                                lsp = {                                                                                signature = {                                                                                enabled = false,                                                                                },                                                                                },                                                                                })

That's what I mean. In the first screenshot noice signature is indeed disabled. So what you see is not  Noice.


test change to test bors ng

Fix ?
Link to ticket: No linked ticket
What has been done

How to test

Acceptance criteria
Todo

[ ]
[ ]

Notes

bors r+


SeedPicker UI Revamp

Currently SeedPicker's UI Confict With BetterLobbies (https://thunderstore.io/c/lethal-company/p/Ryokune/Better_Lobbies/)

I was afraid the UI of seedpicker would clash with another mod at some point.
I'd love to create a lasting solution for this. (not just moving the UI around so it doesn't overlap)
I've not played with many other mods, do you have any ideas on how I can redesign the UI so it would work with most other mods?

this is one idea i thinked

another idea is put a UI config,that allow players to move UI to another position (but this may require extra depedencies)

I will try to slightly move the UI for now and think of a more permanent solution later

Also Confict With LethalConfig btw

@FlooflesDev oh i have an idea, u can use LethalConfig to update the mod config file in real-time without restart the game, should be can work right?, so u don't have to worry about UI issues

@FlooflesDev oh i have an idea, u can use LethalConfig to update the mod config file in real-time without restart the game, should be can work right?, so u don't have to worry about UI issues

I was thinking about checking LethalConfig as well. Though I'm a bit worried that using LethalConfig will make quick seed changes a bit more difficult.
One of the ideas I came up with was to make a custom keybind to show LCSeedPicker's UI and make that keybind configurable with LethalConfig.
I'll also take a second look into integrating it with the Terminal.
I can hopefully pick this issue up again sometime in the coming days!  :)

highly sug not create another ui for this mod, in the first place Lethal_Admin or other simlar like that mods need some UI space, the second is bind a keybind actually not necessary, specially when u r using steam controller input, u will have a huge terrible experience, open on-screen keyboard to type command in the terminal is already painful, but i think that better than to bind another keybind

highly sug not create another ui for this mod, in the first place Lethal_Admin or other simlar like that mods need some UI space, the second is bind a keybind actually not necessary, specially when u r using steam controller input, u will have a huge terrible experience, open on-screen keyboard to type command in the terminal is already painful, but i think that better than to bind another keybind
Edit: actual keyboard still necessary in this game, i won't use on-screen keyboard if i can use my actual keyboard

Good points. I'll try to come up with something that works the best for everyone.
In the end I could always implement multiple solutions and give players a choice in the config


Add VAPTCHA SMS provider

https://www.vaptcha.com/document/install.html#短信接口

@rdp-studio can you make a PR to support it?

@rdp-studio can you make a PR to support it?

Yes, but I need some time.

OK, take your time. This issue will be closed for now.


Fix typo

Many thanks for your awesome work! 👏
In setup, I found a typo, so send this pull request

Sorry for replying
Thanks for the fixed typo, I merged it!


double-beep: Watch wijdanshahid.pk

double-beep requests the watch of the watch_keyword wijdanshahid\.pk. See the MS search here and the Stack Exchange search here.

wijdanshahid\.pk has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Zoe in Charcoal HQ


Add Min, Avg & Max humidity in Daily()

Hi everyone,
I love using the Python Meteostat API for getting access to meteo data through Daily() function. Unfortunately, Daily() doesn't include Min, average and Max humidity but Hourly() have humidity data.
Today, if i want to get the min, avg and max humidity data, i have to use Hourly() in order to have the 24 hours data for extract the min, avg and the max data of humidity of the day.
Python code:
hourly_data = Hourly(location, start_date, end_date)
min_humidity = hourly_data.dict['_data']['rhum'].min()
avg_humidity = hourly_data.dict['_data']['rhum'].avg()
max_humidity = hourly_data.dict['_data']['rhum'].max()
Daily() documentation: https://dev.meteostat.net/python/daily.html#api
Hourly() documentation: https://dev.meteostat.net/python/hourly.html#api
Is it possible to add min, avg and max humidity directly in Hourly() result ?
Thank you

Hi @Vhaume,
It's true - there's a lack of humidity data in the daily dumps. Same for dew point and weather conditions. While we could certainly add the suggested aggregations, it would come at the cost of performance regression. It's similar to this issue: https://github.com/meteostat/routines/issues/5
With aggregated data we have to decide which parameters matter most. When I built the bulk data interface & the Python lib, I choose the ones which seemed obvious to me. However, I'm aware that this is heavily use case-dependent.
I created an issue #62 for parameter requests/discussion.


Apparent error in multihead attention calculation

System information

Have I written custom code (as opposed to using a stock example script
provided in Keras):

No

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Colab (and others, but can be demoed on colab)

TensorFlow installed from (source or binary):
Colab default

TensorFlow version (use command below):
tensorflow version 2.5.0
keras version 2.5.0

Python version:
sys.version_info(major=3, minor=7, micro=11, releaselevel='final', serial=0)

Bazel version (if compiling from source):
NA

GPU model and memory:
CPU

Exact command to reproduce:
See this colab:  https://colab.research.google.com/drive/19hs764rkf2pgM49nNkhlw-ojzchxdhTa?usp=sharing

Describe the problem
The multihead attention layer should multiply the k and q matrices by the value, take the dot prod, and then combine the resulting weighted sum to get the output.  However, upon testing with a random input, and freshly initialized weights (which should also be random, the output is identical at every time step, despite the attention heads having plausible, and varying values per time step.
When an attention mask is used - the values for every head (which were expected to become the reweighted first part of the sequence, with the masked part having zero weight) now become identical across all time steps.

The values which appear identical are not actually identical.  The differences are just extremely tiny and not visually perceptible, which was unexpected.


Update Nuxt to v3.13.1

Fixes

Description

This PR re-adds the changes for updating Nuxt to v3.13.1. I removed the changes related to i18n for easier testing.
Testing Instructions

The CI should pass, and the app should work as expected. Since the server folder was moved, it is important to check that the server routes work as usual (/healthcheck, /robots)
Checklist

[ ] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (ov just catalog/generate-docs for catalog
PRs) or the media properties generator (ov just catalog/generate-docs media-props
for the catalog or ov just api/generate-docs for the API) where applicable.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Drafting because there are typing problems that we've previously seen and couldn't identify the origin: the component doesn't have $route property in this PR

@obulat just curious why the pinia reversion was necessary. This looks good though

@obulat just curious why the pinia reversion was necessary. This looks good though

Just to minimize the number of changes in  one PR, to make reverting easier if necessary


Unified build script.

To build an iso you can:
git clone https://github.com/rancher/os
cd os
./scripts/build-all

this will clone all the repositories that are needed (to the same dir level as the os repo), and then build each project in turn, adding the resulting artifacts into the assets dir of the repos that need it (ie, it avoids tagging, downloading and pushing/uploading).
TODO:

[x] redo the conditional rebuilding to use git commit hash.
[ ] add os-packer building to build and upload test AMI's
[ ] make a zfs service image so zfs works with mainline
[ ] alert if the buildroot&kernel config has changed - helps when upgrading
[ ] contributing docs - once the docs PR merges.

DEFERED TO NEXT PR

bundle up the normally un-shipped images and put them into the dev/test iso to be pre-loaded.
figure out how to get the services to use the preloaded images....

depends on:

https://github.com/rancher/os-kernel/pull/33
https://github.com/rancher/os-base/pull/41

Can we split this up into a few PRs? I'm not sure why this depends on a new kernel and the dev ISO might get tricky.

yeah, i was thinking about splitting https://github.com/rancher/os-kernel/pull/33 - so the kernel change is in a separate PR - so :+1: (this PR doesn't contain kernel changes - and I'll squash when things are ready to merge.

well damn.
it seems that the integration tests will always fail here where I work - due to pulling console images taking a very long time over my DSL.
in the interests of making things work, I'm going to have to see if i can set up a local registry for the tests, so they can run anywhere.


Fix races in resolve_address and resolve_address_posix tests

This should fix internal test flakinesss bugs: b/179683282, b/171497698, and b/179083103
The polling loops in each of these tests currently exits when args->done_atm has been set. And this is set by grpc_resolve_address callbacks before acquiring args->mu. So there's a race where pollset_work can wake up due to timeout, exit, and then destroy args->mu, right before we try to acquire args->mu and kick the pollset.

I've updated with a tentative fix for merge conflicts with https://github.com/grpc/grpc/pull/25404, so this will need to wait for https://github.com/grpc/grpc/pull/25404 to merge before tests start passing.

squashed down commits

interop ALTS tests: https://github.com/grpc/grpc/issues/25242

artifact build windows completed OK but result is still "pending" for some reason (https://source.cloud.google.com/results/invocations/ac1e85af-2474-4eab-8eb4-ed00164de33a/targets)
Bazel C/C++ Dbg Macos: https://github.com/grpc/grpc/issues/25432


.net azure web app startup command multiple apps

Is there a way to run multiple startUp apps?
I have currently
dotnet MyApp.Web.dll
but I also need MyApp.Api.dll.
How can I do that?

@ColonyCreative
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!

@ColonyCreative
Thanks for sharing the feedback. Since this channel is for driving improvements towards Docs, could you please provide the URL of the specific documentation that your feedback is applicable to? That way, we can make sure your feedback is addressed by the right team.

@ColonyCreative
Thanks for your feedback and your contribution to Azure docs.
Feedback for this repository is moving away from GitHub to a system specific to the Microsoft Learn platform. Issues for this repository will soon be disabled, and additional comments from GitHub will no longer be possible. However, we are now tracking and triaging this issue in the new feedback system.
To learn more about our feedback systems, please see Provide feedback for Microsoft Learn content.

@ColonyCreative
We are going to close this thread if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


[cherry-pick] pick index compaction fix and temporarily disable inverted index on unique key

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

wait for formal fix for index compaction


add unsafe from_data_unchecked

This proposes an unsafe from_data_unchecked for ListArray<O> and Utf8Array<O>.
The rationale for this change is that I cast the inner type of the list data and create a new ListArray, where the offsets are unchanged (only cloned).
let child = list.values();
let offsets = list.offsets();
let child = cast::cast(child, &child_type)?;
let list = ListArray::from_data(child_type, offsets.clone(), Arc::new(*child), list.validity.clone());

There was a name collision on the Utf8Array so maybe another function name is better.

Did we observe a performance difference when e.g. casting?

I should have read the code. I thought that yesterday I read somewhere that check_offsets checks if all offsets are monotonously increasing. So that would be O(n). But it only does a few sensible asserts.
I am closing this one, as that would be needless optimization.


 Prismic preview toolbar overflows on the right on mobile, cant close preview

Versions

nuxt-edge: v2.16.0
@nuxtjs/prismic: v1.2.6
node: v14.18.0

Reproduction
Currently dont have a repro on this, wondering if it's because of my CSS stylesheet, seems to have encountered this really early on.
What is Expected?
I should be able to see the close preview X on the mobile.
What is actually happening?
It's being overflowed on the right. The preview functionality works well for me, but I'm wondering if I'm the only one facing this issue on mobile where I can't click the close preview button.
Here's how it looks for me, I've tried to edit by using global CSS but it doesn't work because the CSS is injected via a shadow root.
Is there anything I can do to reveal the X button to close my preview on mobile?

Hey there, thanks for opening an issue!
This is indeed a known issue of the toolbar, we're tracking it internally.
In the meantime preview can be closed by switching to paysage mode.
I'll let you know when we have progress on it :relaxed:


Saying Hello World in Python Notebook

Printed Hello World in Jupyter Notebook

@bhaveshlohana Could you merge this PR.

@bhaveshlohana could you merge this PR


forEach not working

unable to iterate through db records

The tests for .forEach() are passing and there have been no other reports of .forEach() failing, so I'm going to assume that the issue was resolved.


Add "top padding" to image only

I'm expanding on the PinView extension, such that when the image is loaded it also generates 1-X pins, these pins can be dragged and placed on a position on the image to "Mark the spot".
However as you can see, these pins are displayed on-top of the image and while visible on this particular image will likely not have enough contrast on others to see them properly. I'm hoping to find a solution that would allow me to simply push the start of the image down something like 50px.
I tried using padding-top for this but it also affects the left right and bottom spacing.
There was also this "mega-hack":
canvas.clipRect(canvas.getClipBounds().left, 50, canvas.getClipBounds().right,
canvas.getClipBounds().bottom, Region.Op.REPLACE);
    super.onDraw(canvas);
canvas.clipRect(canvas.getClipBounds().left, 0, canvas.getClipBounds().right,
canvas.getClipBounds().bottom, Region.Op.REPLACE);

But that of course clips part of the image itself.

Just as i posted this i gave this a shot:
canvas.translate(0, 50);
super.onDraw(canvas);
canvas.translate(0,-50);

While it does actually appear to work, still feels like a massive hack!

Why not give the pins an outline or drop shadow so they'll show up, and/or draw a semi-opaque dark circle underneath them until they're moved?
Padding is kind of a tricky concept when you can zoom in to the image, so it's hard to get the right behaviour for everyone. It's not really well supported and generally expects equal top/bottom padding.

Yeah, that's what i've resorted to for the time being but it's not really my call sadly, hopefully the feedback is positive and i won't have to spend another 2 days looking for solutions aha.
It's definitely a tricky one, i even tried using a Picasso transformation to modify the source image it self, drawing a white bar along the top for the markers to sit on, but as you mentioned, it wasn't consistent when zooming in/out.

Looks like they were happy! My only problem now is that because they're pins, the top left corner of the view is where it's being scaled around, meaning when zooming the tip of the arrow never aligns with the correct part of the image

Is that something you can fix by offsetting the image to place the arrow in the correct place? The sample code does that.

It is indeed something i can do... thanks for that, no idea why i didn't see it, saved me my evening!


Use isomorphic useLayoutEffect

What kind of change does this PR introduce? (check at least one)

[x] Bugfix (non-breaking change which fixes an issue)
[ ] Feature (non-breaking change which adds functionality)
[ ] Code style update
[ ] Refactor (refactoring or adding test which isn't a fix or add a feature)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

Did you test your solution?

[x] I lightly tested it in one browser
[ ] I deeply tested it in several browsers
[ ] I wrote tests around it (unit tests, integration tests, E2E tests)

Problem Description
When we do initial animations we usually use useLayoutEffect to wait for the DOM to render since useEffect sometimes makes the website flash for a very small amount of time (like 50ms) which is noticeable on the first hard refresh.
Using useLayoutEffect throws a warning with nextjs because it has no use in the server
Solution Description
Swap useLayoutEffect that you want to wait for the DOM to render with useIsomorphicLayoutEffect
Someone smarter than me explained it here
https://medium.com/@alexandereardon/uselayouteffect-and-ssr-192986cdcf7a
Side Effects, Risks, Impact

[x] N/A

Aditional comments:
You should still use useEffect as normal
We could also just create our own hook since this is essentially just this:
const useIsomorphicLayoutEffect =
  typeof window !== 'undefined' ? useLayoutEffect : useEffect

:spinthink:

@neo did some research and this PR is still valid, basically nextjs adds a display: none to the body until the js loads and that's why the flashing doesn't happen 🙏 🔍 with LayoutEffect

@neo did some research and this PR is still valid, basically nextjs adds a display: none to the body until the js loads and that's why the flashing doesn't happen 🙏 🔍 with LayoutEffect

Amazing!


new component backdrop

hey @karishmashuklaa pls check :D
the prop relative is also added (changes the position from fixed to relative for backdrop) because the backdrop was taking the width of entire canvas and the props were not visible on docs page of storybook

@Saurav-Singh-Rauthan LGTM 👍

ref #107


Resources with similar labels will be locked in a random order

This is designed to prevent over-usage on the first numerical resources and to uniformly distribute load between resources.

Do i understand correctly that a resource with the requested label will be selected at random?
In my use case, i have generic and specialized resources, and when a generic resource is requested, that request could be fulfilled by either a generic resource or a specialized resource. There are fewer of the specialized resources, so i like to keep those unused when possible and available for a request for a specialized resource. To give an example: monochrome and color printers. The color printers have the monochrome label as well, but one would like to keep them free so a color print does not have to wait for some monochrome print when monochrome printers have been idling the whole time.
Right now these priorities can be configured somewhat through the resource order.

@clonejo Good example, this does make this harder to use with the current pull request...

As @clonejo suggest, I would be happier with a configurable solution. In FreeStyle projects, there is the possibility to provide a script to select a resource from a label, which seems a bit too powerful for my taste, but it was already there when I adopted the project :smile: - I'd suggest exploring the idea of providing "selections strategies" via an extension point and we can provide some useful ones inside this plugin and people who want to shoot themself in the foot can provide more complex strategies via additional plugins. The most obvious basic ones:

first() - the current default
random() - the one you want to add
preferOtherLabel(String) - the one @clonejo suggested
Sounds good?

Wait, what - are the scripted selectors not exposed in the pipeline step? That would be a bummer to migrate some of our older jobs into the new era :\
We are testing against devices, which we lock for a test. Sometimes a new firmware build is not that good, so a device has to be restored manually - but as far as the lockable resources are concerned, the virtual item is available, matches a label and can be assigned. So the legacy jobs responsible for that have a script to check if the candidate resource is actually usable and only assign those to a job run, to avoid failures just because the test device was AWOL.
The example with printers reminds me of (uh, my) RFE in issue #81 to add weights to the matched resources, as determined by the custom script or perhaps also by extended syntax of resource label definition. In the printer example above, monochrome-only printers might list a label token say "monochrome:1" and color printers would include a "monochrome:2". Then the lock step called for a job which needs a label:"monochrome" printer, gets a list of items with sorted by priority (all 1's, then all 2's, ...) and can pick a random one of those with highest priority currently available.

What about new parameter randomOrder to allow this scenario. Per default false that means no body see difference.
The idea is not wrong.


Bug: case-insensitive treatment of single-word view cells in Windows

PHP Version
8.2
CodeIgniter4 Version
develop
CodeIgniter4 Installation Method
Git
Which operating systems have you tested for this bug?
Windows
Which server did you use?
apache
Database
No response
What happened?
This test (CellGeneratorTest) currently fails on Windows:
    public function testGenerateCellSimpleName()
    {
        command('make:cell Another');

        // Check the class was generated
        $file = APPPATH . 'Cells/Another.php';
        $this->assertStringContainsString('File created: ' . clean_path($file), $this->getStreamFilterBuffer());
        $this->assertFileExists($file);
        $this->assertStringContainsString('class Another extends Cell', $this->getFileContents($file));

        // Check the view was generated
        $file = APPPATH . 'Cells/another.php';
        $this->assertStringContainsString('File created: ' . clean_path($file), $this->getStreamFilterBuffer());
        $this->assertFileExists($file);
        $this->assertSame("<div>\n    <!-- Your HTML here -->\n</div>\n", $this->getFileContents($file));
    }

This is because on Windows, Cells/Another.php is treated the same as Cells/another.php. Thus, only one file is generated, the class file. The view file is skipped.
Steps to Reproduce
Run the test above in Windows. Linux and MacOS are not affected.
Expected Output
I think we should add a check if the cell name is a single word and it is run through Windows. If that's the case, we forbid single word as cell name through a nice error message.
Anything else?
No response

On the default file system on macOS also case-insensitive. So the test also fails.
See https://github.com/codeigniter4/CodeIgniter4/pull/7471#issuecomment-1531329060

I think we should add a check if the cell name is a single word and it is run through Windows. If that's the case, we forbid single word as cell name through a nice error message.

Do you mean to add the check in make:cell?
If so, I think just displaying an error message for single words is fine, regardless of platform.
To begin with, now we get an error on Windows/macOS because we can't create the HTML file.
On Linux we can create the files without error, but if we move to a Windows/macOS server
it won't work.


My bot doesn't move when i do "collect "

This is explained in https://github.com/PrismarineJS/mineflayer/issues/2611 -- i made that issue

Stale 👍


shape control

Dear Author,
Thank you for your work!
Could you please let me know where I can set the shape control?
Best regards

I took a look at the data processing part, it seems to turn the mask into a box, and then I try to use the mask instead of the box for inference, and the generated object will still be outside the mask.


Add Arabic translation

Added new translations to the Arabic locale file.

feel free to re-open if you fix the PR, thanks


JavaScript Interview Questions [ Intermediate Level ]

A curated list of JavaScript questions that helps learners to improve their understanding of the language and develop the skills they need to be successful JavaScript developers.

hello, please assign me this issue under hacktoberfest 2023. i have understanding of javascript language.

sir could you please tell me whether the interview questions are going to constitute a different chapter or whether they'll be in the existing chapters?

sir could you please tell me whether the interview questions are going to constitute a different chapter or whether they'll be in the existing chapters?

There will be a dedicated chapter with various levels of questions. The chapter page will provide an overview of the content and list the different levels of questions. Clicking on each level will redirect the user to the corresponding set of questions, similar to the example shown here.


Alternative shasum 0e063fd87d5b0a4f68dbd35da95b2018748f88eb for InstallESD.dmg for OS X 10.10.5

Downloaded with the App Store yesterday (2016-06-17), the result and a discrepancy was noted at http://apple.stackexchange.com/a/246331/8546
Today I trashed that copy, emptied the trash, restarted the Mac, gained a fresh copy and obtained the same result from shasum:
sh-3.2$ date ; uptime ; cd /Applications/Install\ OS\ X\ Yosemite.app/Contents/SharedSupport/
Mon 18 Jul 2016 18:53:47 BST
18:53  up 58 mins, 2 users, load averages: 2.81 3.78 3.65
sh-3.2$ shasum InstallESD.dmg ; sw_vers
0e063fd87d5b0a4f68dbd35da95b2018748f88eb  InstallESD.dmg
ProductName:	Mac OS X
ProductVersion:	10.12
BuildVersion:	16A239m
sh-3.2$

The discrepancy
0e063fd87d5b0a4f68dbd35da95b2018748f88eb for downloads today and yesterday differs from
ef5cc8851b893dbe4bc9a5cf5c648c10450af6bc at https://github.com/drduh/OS-X-Security-and-Privacy-Guide/blob/master/InstallESD_Hashes.csv

The first response to http://forums.macrumors.com/posts/23136438 also shows (in a screen shot)
0e063fd87d5b0a4f68dbd35da95b2018748f88eb
Without performing an installation from what I downloaded, my first guess is that Apple has distributed an updated installer for an operating system with an unchanged build reference.

Is it not true that mounting a DMG file often changes the hash?

DMG files don't change when mounted. You're getting a different hash because 10.11.6 was released today. Feel free to PR InstallESD_Hashes.csv with the new values.

10.11.6

That's El Capitan.
This issue is for the values for 10.10.5 Yosemite. If there's to be a PR then it may be useful to know the version and build of the .app that contained the earlier .dmg
relate

Actually, @drduh, it is quite often the case that DMG files ARE MODIFIED when mounted! Don't believe me? Try it yourself...
$ wget -q 'https://dist.subgraph.com/downloads/Vega64.dmg'
$ md5 Vega64.dmg
MD5 (Vega64.dmg) = b828ad1d67e522cfc0faf9ba7daca009
$ hdiutil attach Vega64.dmg
/dev/disk2          	                               	/Volumes/Vega
$ md5 Vega64.dmg
MD5 (Vega64.dmg) = e37b2ac0aecbbe0bf024ea4594ebe920
Q.E.D.

I was not sure but @9Yg1rxeSeha90ZU1 comments confirm my idea.
Some DMGs must be burn on CD/DVD before mounting it or they/their content will not work, I think it was for some apple stuff, long time ago...

@grahamperrin My apologies, I had misread your bug title. Security Update 2016-004 is probably what's causing the discrepancy. I would be surprised if the build version isn't updated, however. Can you install and check, please?
It appears you're correct, @9Yg1rxeSeha90ZU1:
$ xxd Vega64.dmg > before
$ hdiutil mount Vega64.dmg >/dev/null
$ xxd Vega64.dmg > after
$ diff before after
69c69
< 0000440: 0000 01cb 0000 1812 0000 0000 0000 0001  ................
---
> 0000440: 0000 01cb 0000 1813 0000 0000 0000 0001  ................

However, that file is read/write image, unlike InstallESD.dmg, which is UDIF read-only (you can check with hdiutil info or hdiutil imageinfo), so that's unlikely the cause here.

I'm also getting 0e063fd87d5b0a4f68dbd35da95b2018748f88eb / 6a2d563b89d4c2733e8ff087c16db0caaa6594375ac835b3110df4306edb2459 for the latest 10.10.5 InstallESD.dmg.
Interestingly enough, each file on the mounted volume is identical, though.
$ diff <(find /Volumes/OS\ X\ Install\ ESD/ -type f -exec shasum -a 256 {} \+ | sed 's%/Volumes/.*/%%') <(find /Volumes/OS\ X\ Install\ ESD\ 1 -type f -exec shasum -a 256 {} \+ | sed 's%/Volumes/.*/%%') ; echo $?
0

More random poking about:
$ stat -f %z *dmg
5718943631
5718940459

$ hdiutil info | grep block
blockcount      : 11760932
blocksize       : 512
blockcount      : 11760932
blocksize       : 512

$ diff <(tail -c 512 InstallESD-latest.dmg | hexdump -C) <(tail -c 512 InstallESD-current.dmg | hexdump -C)
2c2
< 00000010  00 00 00 00 00 00 00 00  00 00 00 00 00 59 71 97  |.............Yq.|
---
> 00000010  00 00 00 00 00 00 00 00  00 00 00 00 00 59 71 9e  |.............Yq.|
9c9
< 000000d0  00 00 00 00 00 00 00 00  00 00 00 01 54 d7 ef 46  |............T..F|
---
> 000000d0  00 00 00 00 00 00 00 00  00 00 00 01 54 d7 ef 4d  |............T..M|

Here is a mysterious Pastebin from February with MD5 sums of both DMGs: http://pastebin.com/CDrPPjsj
I can't explain the discrepancy at this time.

I suggest we checksum partition/slice, instead of or in addition to InstallESD.dmg container.


@agent.on_query issue

Prerequisites

[X] I checked the documentation and found no answer to my problem
[X] I checked the existing issues and made sure there are no similar bug reports

Category
Bug (unexpected behavior)
Expected Behavior
The query should be processed and agent should acknowledge that it has received the query
Observed Behavior
While following the example for on_query on the link: https://fetch.ai/docs/guides/agents/intermediate/handlers
The server could not process the TestRequest request on the "/endpoint" POST endpoint and threw a status code 422 Unprocessable Entity. After investigation, I found that changing the class passed to TestRequest from "Model" to "BaseModel" [ from pydantic import BaseModel ] made the request process successfully. I checked the uAgents "Model" class, it adds some fields which were removed from Pydantic BaseModel v2.
Original code:

Curl return:

Server log:

Changed to BaseModel:

Curl return after change:

Server log after change:

Kindly let me know if this issue is observed by everyone or if there is something different about my setup.
To Reproduce
No response
Version
v0.15.2
Environment Details (Optional)
No response
Failure Logs (Optional)
No response
Additional Information (Optional)
No response

Hi @tanaygodse! You are correct in that the above code snippet will not work due to the current changes in pydantic model handling. We'll resolve these soon in the core framework and make sure that model handling will be as seamless as before.
As for the error itself, please check out the repository examples folder where you can find the updated query snippet: https://github.com/fetchai/uAgents/tree/main/python/examples/16-on-query-proxy
I'm going to move this issue to the docs repo for the docs to be updated and thank you for your input.


Add sonarcloud workflow

Fixes #595 (fully this time)

@DiamondJoseph I do not have one, plausibly @callumforrester  signed up for the account for the DLS org and the key can be found / generated there

I did not...
You will have to figure out who owns it and approach them

need to add org key

now debugging this

08:59:50.762 ERROR main component does not belong to specified organization

which surprisingly only pops up after analysis has run succesfully for 40 seconds


feat: add permission enforcement and permission e2e tests

Pushed a fix that fixes the errors that were created during merging. should now be in the correct state.


missing all table structure or migrations scripts.

please advice.

@cymonx I have now added the migration files to the index :)


ReferenceError on page load

Uncaught ReferenceError: onMarsRightNow is not defined at 45-exercises.js:47 should not be the first thing that users see on first page load.

Resolved by https://github.com/gocodeup/45-js-exercises/pull/4

Resolved by https://github.com/gocodeup/45-js-exercises/pull/4


dataset

Hi, can you share the dataset?

Thank you for your interest in the code base.
I added links to the dataset in the README file.


bugfix/timestamp_cast

Are you a current Fivetran customer?

Fivetran created PR
What change(s) does this PR introduce?

Casts the created_date and closed_date fields within the stg_salesforce__opportunity model to a timestamp using the dbt_utils.type_timestamp() macro. This is needed for Redshift users that see these fields being synced as timestamptz as the downstream date functions do not work with the timestamptz datatype.
Did you update the CHANGELOG?

[X] Yes

Does this PR introduce a breaking change?

[ ] Yes (please provide breaking change details below.)
[X] No  (please provide an explanation as to how the change is non-breaking below.)

This PR simply casts timestamp fields and using dbt_utils macro. This will not result in a breaking change for existing users.
Did you update the dbt_project.yml files with the version upgrade (please leverage standard semantic versioning)? (In both your main project and integration_tests)

[X] Yes

Is this PR in response to a previously created Bug or Feature Request

[X] Yes, dbt Slack issue raised
[ ] No

How did you test the PR changes?

[X] CircleCi
[X] Local (please provide additional testing details below)

I have also asked the customer who raised the issue to test out the change and confirm if the issue still persists.
Select which warehouse(s) were used to test the PR

[X] BigQuery
[X] Redshift
[X] Snowflake
[X] Postgres
[X] Databricks
[ ] Other (provide details below)

Provide an emoji that best describes your current mood

🥗
Feedback
We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your feedback on our existing dbt packages or what you'd like to see next.

Thanks so much @fivetran-sheringuyen 😸


[staging] No hover state for Download Tomogram / All Annotations

Steps to Reproduce

Click on [Download...] button
Hover on Download Tomogram and Download All Annotations
Pay attention to hover state
Compare to design

Actual Results
No hover state
Expected Results
Options should be greyed out on hover
Design

Evidence
https://github.com/chanzuckerberg/cryoet-data-portal/assets/63799148/f18b5b73-bcbc-4397-9939-67608dcca246

This defect is no longer relevant with the new download modal


Create a test environment for deepspeech version2 on github

Description
This PR is to create a specific test environment for automating deepspeech version 2 tests.
Fixes #1032
Type of change
Please check all relevant options.

[x] Improvement (non-breaking)
[x] Bug fix (non-breaking)
[ ] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Testing
Please describe the tests that you ran to verify your changes. Consider listing any relevant details of your test configuration.

[ ] Test A
[ ] Test B

Test Configuration:

OS
Python version
ART version or commit number
TensorFlow / Keras / PyTorch / MXNet version

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Codecov Report

Merging #1035 (01e5207) into dev_1.6.1 (b2c8955) will decrease coverage by 0.79%.
The diff coverage is 60.00%.

@@              Coverage Diff              @@
##           dev_1.6.1    #1035      +/-   ##
=============================================
- Coverage      62.55%   61.76%   -0.80%
=============================================
  Files            220      220
  Lines          19056    19049       -7
  Branches        3469     3469
=============================================
- Hits           11921    11765     -156
- Misses          5818     5974     +156
+ Partials        1317     1310       -7

Impacted Files
Coverage Δ

...ion/imperceptible_asr/imperceptible_asr_pytorch.py
7.04% <0.00%> (+0.07%)
:arrow_up:

art/preprocessing/audio/l_filter/numpy.py
68.91% <100.00%> (-0.82%)
:arrow_down:

art/preprocessing/audio/l_filter/pytorch.py
64.40% <100.00%> (-0.60%)
:arrow_down:

...estimators/speech_recognition/tensorflow_lingvo.py
16.74% <0.00%> (-66.52%)
:arrow_down:

art/estimators/tensorflow.py
73.73% <0.00%> (-2.03%)
:arrow_down:

art/estimators/estimator.py
68.68% <0.00%> (-2.03%)
:arrow_down:

art/utils.py
64.46% <0.00%> (-0.59%)
:arrow_down:

art/attacks/evasion/brendel_bethge.py
20.30% <0.00%> (-0.37%)
:arrow_down:

art/estimators/classification/scikitlearn.py
80.11% <0.00%> (+0.77%)
:arrow_up:


Fix a misspelled word in the code

The word "fourth" (meaning number four in a list) is consistently misspelled as "forth" (meaning onward) in the code. I've gone ahead and replaced all occurrences with no issue, but I'm afraid it's a little more complicated than that, as there are a few parallel branches being worked on at the time, so I thought the next best thing was to let you know.
final AutoDisposeProvider<String> futureProvider = provider;

expect(provider.first, 42);
expect(provider.second, 'x42');
expect(provider.third, .42);
expect(provider.forth, false); // <--- here
expect(provider.fifth, ['x42']);

final String result = container.read(
  FamilyProvider(
    42,
    second: 'x42',
    third: .42,
    forth: false, // <--- here
    fifth: ['x42'],
  ),
);

expect(
  result,
  '(first: 42, second: x42, third: 0.42, forth: false, fifth: [x42])', // <--- and here
);

// and a bunch more just like it

Let's face it; most likely nobody cares about this. It's not a bug, it's not a missing feature, it's barely even noticeable if you look hard enough, and it most definitely isn't an issue for end-users of the library. I was just going through the recent changes for the codegen component and it was simply bothering me.

Thanks for the report!


chore(STONEINTG-542): update Apache licence info in all source files

Maintainers will complete the following section

[ x] Commit messages are descriptive enough (hints)
[x ] Code coverage from testing does not decrease
[ ] Controllers diagrams are updated when PR changes controllers code  (if applicable)

/retest

/retest

The changes are looking good so far, but the files under git/github also need the license added to them.

Thanks, I wrote one note about having copyright owner in each file, that's something we miss almost everywhere.
It can be separate PR

I covered it while i was checking everything

The changes are looking good so far, but the files under git/github also need the license added to them.

Added.

merged


41 pinfu

close #41

Current coverage is 78.85%

Merging #107 into master will decrease coverage by -0.11% as of 995cd1b

@@            master    #107   diff @@
======================================
  Files           45      45
  Stmts         1041    1069    +28
  Branches       258     266     +8
  Methods          0       0
======================================
+ Hit            822     843    +21
- Partial         57      64     +7
  Missed         162     162

Review entire Coverage Diff as of 995cd1b

Powered by Codecov. Updated on successful CI builds.


Slight timestamp mismatch between individual results and combined json

eg: Yuna, WA

"processed_date": "2023-07-30T20:51:05.657736",

"processed_date": "2023-07-30T20:51:05.609951",

results/combined-suburbs.json shows
  {
   "name": "Yuna",
   "processed_date": "2023-07-30T20:51:05.657736",
   "announced": false,
   "announced_date": null,
   "address_count": 22
  }

But results/WA/yuna.geojson shows
{
 "type": "FeatureCollection",
 "generated": "2023-07-30T20:51:05.609951",
 "suburb": "YUNA",
 "features": [

This is because when the combined file is updated, it uses
found_suburb.processed_date = datetime.now()

Rather than the datetime from the file.

Add fix for this to the PR about NBN dates
https://github.com/LukePrior/nbn-upgrade-map/pull/221/commits/e10510268f23c195f059c60d28285b9796bf6242

Fixed


Declaration of ContainerAwareCallbackFactory::get must be compatible with CallbackFactory::get(): CallbackInterface

Hi
I have next bundle version at my composer.lock:
{
            "name": "winzou/state-machine-bundle",
            "version": "v0.3.1",
...
            "require": {
...
                "winzou/state-machine": "~0.3"
            },
}

After doing composer update
  - Updating winzou/state-machine (0.3.3 => 0.4.0): Downloading (100%)

I see
In ContainerAwareCallbackFactory.php line 35:

  Compile Error: Declaration of winzou\Bundle\StateMachineBundle\Callback\Con
  tainerAwareCallbackFactory::get(array $specs) must be compatible with SM\Ca
  llback\CallbackFactory::get(array $specs): SM\Callback\CallbackInterface

I guess winzou/state-machine-bundle:v0.3.2 and winzou/state-machine-bundle:v0.4.3 should contain something like this (0.4.0+ excluded):
            "require": {
...
                "winzou/state-machine": "^0.3"
            },

rather than ~0.3

Hi.
The same situation is on Sylius(master or 1.7), When i try update the project, i get similar error, that ContainerAwareCallbackFactory needs return type.

Nice catch. I've added the 2 tags, can you confirm it's all good on your side? Thanks!

Works correctly. Thanks for the very quick help 👍

Thanks guys!


Buff Duration Multiplier Issue

Heyyyyyyyy it's your boi Dragon back with another issue from Discord
Describe the bug
From Trixter: Folks have been getting that in errors for "permanent" buffs with absurdly high multipliers, It normally doesn't break anything, but I'd like to make it stop erroring when I have a chance
Additional context
Trixter wanted a Github issue, so he didn't forget about it.

removed the part of  the log file I pasted here because Github doesn't like to format stuff nicely

[ToyBox] [Error] BuffCollection_AddBuff2_patch - duration: 10675199.02:48:05.4775807 - ticks:<PHONE_NUMBER>854775807 * 1 => 9.223372E+18
...
[ToyBox] [Error] System.OverflowException: Arithmetic operation resulted in an overflow.

Went and checked the log from discord. Looks like someone is multiplying long.max by a float representation of 1 and then getting back slightly more than long.max. The code has a try-catch around it, so evidently someone knew it was a problem before, and that's why it's not actually causing problems.


add more commands

support ssh_credentail.

Thanks for your contribution!

sorry for long time no response. and  what's the meaning of PR? and what can i do ?


usb_cam: 0.3.4-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository usb_cam to 0.3.4-0:

upstream repository: https://github.com/bosch-ros-pkg/usb_cam.git
release repository: https://github.com/tork-a/usb_cam-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.5.26
previous version for package: null

usb_cam
* Installs launch files
* Merge pull request #37 from tzutalin/develop
  Add a launch file for easy test
* Add a launch file for easy test
* Contributors: Russell Toris, tzu.ta.lin

I know this not a good behavior, but I have take over release repository with out original maintainers's permission
https://github.com/bosch-ros-pkg/usb_cam/issues/53
If you think we should have more conversation on this process, feel free to close this PR
c.f. https://discourse.ros.org/t/add-usb-cam-to-ros-kinetic-main-repository/607/2


Bouncing Mechanics are frequently off

Big example is if I do a big drop and hit an enemy, I sometimes just kill it and then immediately land. I'll be investigating this behavior soon.

Clearing assignment from myself as I haven't had much time to actively contribute lately. Hoping to resume working on new gimmicks soon though.

I fixed this problem in my own project by modifying the code in EnemyBase.gd by changing this code at line 19:
if (i.movement.y < 0 or i.global_position.y-(i.velocity.y*delta) > global_position.y):
to:
if (i.movement.y < 0):
The second test in the if statement seems to return true in inappropriate circumstances, which causes the problem of the player sometimes simply landing after destroying an enemy. It also just seems unnecessary to me; is there ever a situation where you wouldn't want the player to bounce if they had downwards velocity? I cut this second test out and bouncing works perfectly.
This is just for enemies; monitors have their own problems, and the way I fixed them in my project is a little hacky, but I think it works fine. If you like, I can make a PR for this.

This should be completely fixed by now. Worked on it a couple weeks back and found some inaccurate math and a faulty elif statement to be the cause.

Since this issue hasn't been mentioned in a while and we've made steps to address it, I'm going to mark this issue as fixed. I've had no more issues with many attempts.


Question about string encoding

Hi folks,
first of all, thanks for the great library!
I am currently working on internationalization of my project that integrates OpenEXR. I tried to find out from the documentation and the source what encoding OpenEXR accepts/returns for its stings but did not find anything definitive. The only references to string encoding I found were in the docs for StandardAttributes.rst where some camera metadata attributes should be UTF-8 encoded. But I failed to find something more general.
So the question is: Is there a convention in the project that readers and writers of the file format are sticking to? Or is it just "unspecified", like ASCII works for sure, anything beyond is "up to the implementation"? I guess I can just put UTF-8 or ISO/IEC 8859-1 encoded strings into it without breaking anything while writing, but without a common standard I can't rely on viewers to interpret this correctly.
Can you give me any hints at best practices? I am especially interested into encoding of header names, header attributes (names, StringAttribute), channel names and slice names.
Many thanks!

I think the short answer is not to expect any tool to display strings correctly if they aren't simple ASCII characters: they aren't common so wouldn't have had much testing.
There is standardized naming for channels (e.g. R/G/B must be used for red/green/blue and '.' is a layer separator). In theory that would allow UTF-8 encoded channel names, but some encodings wouldn't be compatible with that standard. I don't think there's anything else in the library that enforces any encoding. StringAttributes are just arbitrary arrays of bytes. However, code written in languages with more explicit string encoding may well break.

That's exactly what I feared. But at least it is somewhat documented now in this repository. Nevertheless, thank you for the quick answer!


Post-build validation fails: "cannot execute LINK.EXE"

Describe the bug
The post-build validation step fails with an error.
Environment

OS: Windows 10
Compiler: Visual Studio 2019 version 14.26.28801

To Reproduce
Steps to reproduce the behavior:

vcpkg.exe install fmt:x64-windows --debug
See error

Expected behavior
The build should succeed.
Failure logs
Running command:
   "C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Tools\MSVC\14.26.28801\bin\HostX86\x86\dumpbin.exe" /exports "C:\dev\vcpkg\packages\fmt_x64-windows\debug\bin\fmtd.dll"
 failed
[DEBUG] C:\dev\vcpkg\toolsrc\src\vcpkg\postbuildlint.cpp(397)

Additional context
If I try to run the command above manually, I receive the error:
> "C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Tools\MSVC\14.26.28801\bin\HostX86\x86\dumpbin.exe" /exports "C:\dev\vcpkg\packages\fmt_x64-windows\debug\bin\fmtd.dll"
DUMPBIN : error : cannot execute LINK.EXE; errno = 0 (2)

> where link.exe
INFO: Could not find files for the given pattern(s).

As there is no LINK.EXE program on the PATH. The command succeeds if I run it within a Visual Studio developer console:
> "C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Auxiliary\Build\vcvars64.bat"

> where link.exe
C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Tools\MSVC\14.26.28801\bin\Hostx64\x64\link.exe

> vcpkg.exe install fmt:x64-windows --debug
...
-- Performing post-build validation
-- Performing post-build validation done
...

Similarly, running vcpkg env also gives me a valid environment.
I've tried it on 3 separate development machines with the same result, so I'm not sure how it works for anybody else.
After a brief look in the source, I see some code that's calling the Visual Studio vcvarsall.bat script to set up an environment (e.g. in make_build_env_cmd) so I'm guessing that the dumpbin program should also be called within the context of that environment?

@chris-t-w, I can't repro this issue locally.
Beofore you manually execute the command, you need set the VS relate environment by executing ""C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Auxiliary\Build\vcvars64.bat"
E:\VCPKG\boost_173\vcpkg>"C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Auxiliary\Build\vcvars64.bat"
**********************************************************************
** Visual Studio 2019 Developer Command Prompt v16.6.0
** Copyright (c) 2020 Microsoft Corporation
**********************************************************************
[vcvarsall.bat] Environment initialized for: 'x64'

E:\VCPKG\boost_173\vcpkg>where link.exe
C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.26.28801\bin\Hostx64\x64\link.exe

E:\VCPKG\boost_173\vcpkg>"C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.26.28801\bin\HostX86\x86\dumpbin.exe" /exports E:\vcpkg\boost_173\vcpkg\packages\fmt_x64-windows\debug\bin\fmtd.dll
Microsoft (R) COFF/PE Dumper Version 14.26.28805.0
Copyright (C) Microsoft Corporation.  All rights reserved.

Dump of file E:\vcpkg\boost_173\vcpkg\packages\fmt_x64-windows\debug\bin\fmtd.dll

File Type: DLL

  Section contains the following exports for fmtd.dll

    00000000 characteristics
    FFFFFFFF time date stamp
        0.00 version
           1 ordinal base
          81 number of functions
          81 number of names

    ordinal hint RVA      name

Beofore you manually execute the command, you need set the VS relate environment

@PhoebeHui that was my point: the dumpbin command will fail due to LINK.EXE not being on the PATH unless it is run within a VS environment.
As far as I understand, vcpkg does not have to be run inside such an environment. Indeed, if I run with --debug then I can see that it runs this script itself to get a list of the environment variables (which are presumably used to create a build environment):
[DEBUG] CreateProcessW(cmd /c ""C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Auxiliary\Build\vcvarsall.bat" -vcvars_ver=14.2 amd64   2>&1 <NUL" & echo cdARN4xjKueKScMy9C6H& set)

This then raises the question, if dumpbin has to be run inside a VS environment, how is it supposed to work in the vcpkg post-build validation step, which, as far as I can tell, isn't executing with such an environment set up?
Do you have a LINK.EXE anywhere on your PATH before running vcvars64.bat?
As mentioned, I've tried this on three independent environments so I'm confused as to what I'm doing wrong. Here's the powershell script I used on all of them:
$vcpkg_dir = "C:\dev\vcpkg"

New-Item -ItemType directory -Path $vcpkg_dir
git clone https://github.com/microsoft/vcpkg $vcpkg_dir
cmd.exe /c "$vcpkg_dir\bootstrap-vcpkg.bat" "-disableMetrics"

[Environment]::SetEnvironmentVariable("VCPKG_ROOT", $vcpkg_dir, "Machine")

& "$vcpkg_dir\vcpkg.exe" install fmt:x64-windows

Every time, the same thing:
Running command:
   "C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\VC\Tools\MSVC\14.26.28801\bin\HostX86\x86\dumpbin.exe" /exports "C:\dev\vcpkg\packages\fmt_x64-windows\debug\bin\fmtd.dll"
 failed

@chris-t-w, vcpkg call vcvarsall.bat to set the VS environments inside, but out of the box, the environments doesn't perserve, so when you execute the command manually, it can not find the LINK.exe.
I don't have LINK.EX before running vcvars64.bat. I tried your powershell script, it works well for me.
So you tried to build it with powershell, Command, and Visual Studio developer console, and only Visual Studio developer console install it successfully, right?

After re-installing Visual Studio, the issue seems to have gone away. I'm not really sure what happened, but something seems to have been weird with the linker in the installation.


add David as contributor and rerender

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

requirements: run: semver>=2.0.0,<3.0.0 must contain a space between the name and the pin, i.e. semver >=2.0.0,<3.0.0

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender


Create export target when building without ros

Currently the ROS_VERSION is always set from the environment variable, if present. To allow building without ROS support on systems where ROS is installed, only set ROS_VERSION from environment when not explicitly set in cmake. This means calling cmake with -DROS_VERSION=0 will disable ROS support even when ROS is installed on the system.
Add export target for shared library when building with ROS_VERSION=0. This means the library can be used in a cmake project by simply using find_package(sick_scan_xd) and target_link_libraries(<project-name> sick_scan_xd::sick_scan_shared_lib). This also means calling  SickScanApiLoadLibrary(sick_scan_api_lib); in the application is not necessary anymore as the shared library is linked during compile time.
I only tested this on Ubuntu with a sick_tim_5xx lidar sensor and building with -DLDMRS=0 -DSCANSEGMENT_XD=0 -DROS_VERSION=0.

Many thanks for your pull request! We will merge it into one of the next releases. Until then we leave the PR open as a remainder.

Thank you for your contributions. We included them in Version 2.9.0.


Volumes mounted by Windows on paths aren't available to WSL

Windows Build Number
Microsoft Windows [Version 10.0.19043.1165]
WSL Version

[X] WSL 2
[X] WSL 1

Kernel Version
5.10.16
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps

Create a new volume, such as by plugging in an external drive.
Using Windows' Disk Management utility, mount the volume to an empty NTFS directory instead of a drive letter.
Attempt to access the volume from a WSL guest.

Expected Behavior
The volume should be accessible from the WSL guest.
Actual Behavior
The guest sees the directory as a symlink. Dereferencing this symlink, or attempting to stat it, fails with a generic "input/output error" (EIO, according to Ruby).
Diagnostic Logs
No response

I got similar problem.
My host: Windows 11 Pro.
Linux version: Ubuntu 20.04
I mount a vhdx put on a USB drive.
It works fine for awhile. But at some point it will complain "Input/output error."
The files on the whole USB drive becomes read-only.
All I can do is to eject my USB drive, and then connect again and mount the vhdx again.

Has there been any progress on this issue? According to the best solution for this problem, I can access the local file, but the path under wsl is different from the path under windows, which is very inconvenient.

Yeah, SUBSTed drives basically don't work in wsl, although I can force-mount them and get around it during a session, no new mount point needed:
In CMD:
D:\>wsl
<3>WSL (318) ERROR: CreateProcessParseCommon:711: Failed to translate D:\
brbill@brbox:~$ ls /mnt/d
brbill@brbox:/mnt$ sudo mount -t drvfs D:\\ /mnt/d
[sudo] password for brbill:
brbill@brbox:/mnt$ ls /mnt/d
folder1        license.txt     readme.txt


Firewall Control Module/Backdoor

Ability to create or take down firewall

Could see this being either actually - a firewall module that explicitly allows the port you're trying to use through the firewall, and a firewall backdoor that gives the user complete control over the firewall. Yeah this should be two separate things


project-posenet and others not working with enterprise-eagle-20200724205123

We have 32 Coral Dev Boards and most of the demos now fail after flashing enterprise-eagle-20200724205123 because edgetpu https://github.com/google-coral/edgetpu has been branded Legacy and replaced by libedgetpu
mendel@k3s-tpu-09:~/project-posenet$ python3 simple_pose.py  Traceback (most recent call last):   File "simple_pose.py", line 18, in <module>     from pose_engine import PoseEngine   File "/home/mendel/project-posenet/pose_engine.py", line 20, in <module>     from edgetpu import __version__ as edgetpu_version ModuleNotFoundError: No module named 'edgetpu'
Are there any plans to update the example code soon ?
Can i substitute libedgetpu easily ?

Hello, yes, we're working on a fix for this as our old API was deprecated.
For now, please use my fork: https://github.com/Namburger/project-posenet
More details here:
https://github.com/google-coral/project-posenet/issues/42#issuecomment-738026058

Closer !
simple_pose.py works now but pose_camera.py is not happy about the input size
ValueError: input size=925444, expected=924963.
mendel@k3s-tpu-04:~/project-posenet$ python3 simple_pose.py
Inference time: 19 ms
Pose Score:  0.60673916
KeypointType.NOSE    x=210  y=152  score=1.0
KeypointType.LEFT_EYE x=223  y=138  score=1.0
KeypointType.RIGHT_EYE x=199  y=137  score=1.0
KeypointType.LEFT_EAR x=244  y=135  score=1.0
KeypointType.RIGHT_EAR x=183  y=129  score=0.8
KeypointType.LEFT_SHOULDER x=269  y=169  score=0.8
KeypointType.RIGHT_SHOULDER x=160  y=174  score=1.0
KeypointType.LEFT_ELBOW x=281  y=255  score=0.6
KeypointType.RIGHT_ELBOW x=153  y=253  score=0.9
KeypointType.LEFT_WRIST x=237  y=332  score=0.6
KeypointType.RIGHT_WRIST x=163  y=304  score=0.5
KeypointType.LEFT_HIP x=261  y=317  score=0.1
KeypointType.RIGHT_HIP x=173  y=309  score=0.1
KeypointType.LEFT_KNEE x=223  y=339  score=0.3
KeypointType.RIGHT_KNEE x=200  y=342  score=0.3
KeypointType.LEFT_ANKLE x=186  y=407  score=0.1
KeypointType.RIGHT_ANKLE x=188  y=410  score=0.2
Pose Score:  0.54908466
KeypointType.NOSE    x=400  y=145  score=1.0
KeypointType.LEFT_EYE x=418  y=129  score=1.0
KeypointType.RIGHT_EYE x=383  y=127  score=1.0
KeypointType.LEFT_EAR x=458  y=110  score=0.9
KeypointType.RIGHT_EAR x=371  y=121  score=0.2
KeypointType.LEFT_SHOULDER x=496  y=168  score=1.0
KeypointType.RIGHT_SHOULDER x=361  y=150  score=0.8
KeypointType.LEFT_ELBOW x=463  y=291  score=1.0
KeypointType.RIGHT_ELBOW x=331  y=245  score=0.8
KeypointType.LEFT_WRIST x=340  y=305  score=0.9
KeypointType.RIGHT_WRIST x=234  y=332  score=0.5
KeypointType.LEFT_HIP x=485  y=312  score=0.1
KeypointType.RIGHT_HIP x=364  y=327  score=0.1
KeypointType.LEFT_KNEE x=470  y=336  score=0.0
KeypointType.RIGHT_KNEE x=245  y=329  score=0.1
KeypointType.LEFT_ANKLE x=444  y=381  score=0.0
KeypointType.RIGHT_ANKLE x=188  y=410  score=0.2
mendel@k3s-tpu-04:~/project-posenet$ python3 pose_camera.py
/home/mendel/project-posenet/gstreamer.py:15: PyGIWarning: Gst was imported without specifying a version first. Use gi.require_version('Gst', '1.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
/home/mendel/project-posenet/gstreamer.py:15: PyGIWarning: GstBase was imported without specifying a version first. Use gi.require_version('GstBase', '1.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
/home/mendel/project-posenet/gstreamer.py:15: PyGIWarning: GstVideo was imported without specifying a version first. Use gi.require_version('GstVideo', '1.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
/home/mendel/project-posenet/gstreamer.py:15: PyGIWarning: Gtk was imported without specifying a version first. Use gi.require_version('Gtk', '3.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
Loading model:  models/mobilenet/posenet_mobilenet_v1_075_481_641_quant_decoder_edgetpu.tflite
Detected Edge TPU dev board.
Gstreamer pipeline:  v4l2src device=/dev/video0 ! video/x-raw,width=640,height=480,framerate=30/1 ! decodebin ! glupload ! glvideoflip video-direction=identity ! tee name=t
t. ! queue max-size-buffers=1 leaky=downstream ! freezer name=freezer ! glsvgoverlaysink name=overlaysink
t. ! queue max-size-buffers=1 leaky=downstream ! glfilterbin filter=glbox name=glbox ! video/x-raw,format=RGB,width=641,height=481 ! appsink name=appsink emit-signals=true max-buffers=1 drop=true
Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/lib/python3.7/threading.py", line 917, in _bootstrap_inner
self.run()
File "/usr/lib/python3.7/threading.py", line 865, in run
self._target(*self._args, **self._kwargs)
File "/home/mendel/project-posenet/gstreamer.py", line 180, in inference_loop
output = self.inf_callback(input_tensor)
File "pose_camera.py", line 148, in run_inference
return engine.run_inference(input_tensor)
File "/home/mendel/project-posenet/pose_engine.py", line 96, in run_inference
edgetpu.run_inference(self._interpreter, input_data)
File "/usr/lib/python3/dist-packages/pycoral/utils/edgetpu.py", line 169, in run_inference
_check_input_size(memory.size, expected_input_size)
File "/usr/lib/python3/dist-packages/pycoral/utils/edgetpu.py", line 141, in _check_input_size
input_size, expected_input_size))
ValueError: input size=925444, expected=924963.

We have 32 Coral Dev Boards and most of the demos now fail after flashing enterprise-eagle-20200724205123 because edgetpu https://github.com/google-coral/edgetpu has been branded Legacy and replaced by libedgetpu
mendel@k3s-tpu-09:~/project-posenet$ python3 simple_pose.py Traceback (most recent call last): File "simple_pose.py", line 18, in <module> from pose_engine import PoseEngine File "/home/mendel/project-posenet/pose_engine.py", line 20, in <module> from edgetpu import __version__ as edgetpu_version ModuleNotFoundError: No module named 'edgetpu'
Are there any plans to update the example code soon ?
Can i substitute libedgetpu easily ?

I doubted. I tried removing pycoral, and adding legacy packages but couldn't resolve decencies. I tried going back to Day13 version but that also failed in reflashing

I ran into this issue with the new repo after running pose_camera.py
oading model:  models/mobilenet/posenet_mobilenet_v1_075_481_641_quant_decoder_edgetpu.tflite
Detected Edge TPU dev board.
 Use gi.require_version('Gst', '1.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
/home/mendel/project-posenet/gstreamer.py:15: PyGIWarning: GstBase was imported without specifying a version first. Use gi.require_version('GstBase', '1.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
/home/mendel/project-posenet/gstreamer.py:15: PyGIWarning: GstVideo was imported without specifying a version first. Use gi.require_version('GstVideo', '1.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
/home/mendel/project-posenet/gstreamer.py:15: PyGIWarning: Gtk was imported without specifying a version first. Use gi.require_version('Gtk', '3.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
Loading model: models/mobilenet/posenet_mobilenet_v1_075_481_641_quant_decoder_edgetpu.tflite
Detected Edge TPU dev board.
Gstreamer pipeline: v4l2src device=/dev/video0 ! video/x-raw,width=640,height=480,framerate=30/1 ! decodebin ! glupload ! glvideoflip video-direction=identity ! tee name=t
t. ! queue max-size-buffers=1 leaky=downstream ! freezer name=freezer ! glsvgoverlaysink name=overlaysink
t. ! queue max-size-buffers=1 leaky=downstream ! glfilterbin filter=glbox name=glbox ! video/x-raw,format=RGB,width=641,height=481 ! appsink name=appsink emit-signals=true max-buffers=1 drop=true
Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/lib/python3.7/threading.py", line 917, in _bootstrap_inner
self.run()
File "/usr/lib/python3.7/threading.py", line 865, in run
self._target(*self._args, **self._kwargs)
File "/home/mendel/project-posenet/gstreamer.py", line 180, in inference_loop
output = self.inf_callback(input_tensor)
File "pose_camera.py", line 148, in run_inference
return engine.run_inference(input_tensor)
File "/home/mendel/project-posenet/pose_engine.py", line 96, in run_inference
edgetpu.run_inference(self._interpreter, input_data)
File "/usr/lib/python3/dist-packages/pycoral/utils/edgetpu.py", line 169, in run_inference
_check_input_size(memory.size, expected_input_size)
File "/usr/lib/python3/dist-packages/pycoral/utils/edgetpu.py", line 141, in _check_input_size
input_size, expected_input_size))
ValueError: input size=925444, expected=924963.

I got here too after connecting the output:
python3 pose_camera.py
/home/mendel/p2/project-posenet/gstreamer.py:15: PyGIWarning: Gst was imported without specifying a version first. Use gi.require_version('Gst', '1.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
/home/mendel/p2/project-posenet/gstreamer.py:15: PyGIWarning: GstBase was imported without specifying a version first. Use gi.require_version('GstBase', '1.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
/home/mendel/p2/project-posenet/gstreamer.py:15: PyGIWarning: GstVideo was imported without specifying a version first. Use gi.require_version('GstVideo', '1.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
/home/mendel/p2/project-posenet/gstreamer.py:15: PyGIWarning: Gtk was imported without specifying a version first. Use gi.require_version('Gtk', '3.0') before import to ensure that the right version gets loaded.
from gi.repository import GLib, GObject, Gst, GstBase, GstVideo, Gtk
Loading model:  models/mobilenet/posenet_mobilenet_v1_075_481_641_quant_decoder_edgetpu.tflite
Detected Edge TPU dev board.
Gstreamer pipeline:  v4l2src device=/dev/video0 ! video/x-raw,width=640,height=480,framerate=30/1 ! decodebin ! glupload ! glvideoflip video-direction=identity ! tee name=t
t. ! queue max-size-buffers=1 leaky=downstream ! freezer name=freezer ! glsvgoverlaysink name=overlaysink
t. ! queue max-size-buffers=1 leaky=downstream ! glfilterbin filter=glbox name=glbox ! video/x-raw,format=RGB,width=641,height=481 ! appsink name=appsink emit-signals=true max-buffers=1 drop=true
Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/lib/python3.7/threading.py", line 917, in _bootstrap_inner
self.run()
File "/usr/lib/python3.7/threading.py", line 865, in run
self._target(*self._args, **self._kwargs)
File "/home/mendel/p2/project-posenet/gstreamer.py", line 180, in inference_loop
output = self.inf_callback(input_tensor)
File "pose_camera.py", line 148, in run_inference
return engine.run_inference(input_tensor)
File "/home/mendel/p2/project-posenet/pose_engine.py", line 96, in run_inference
edgetpu.run_inference(self._interpreter, input_data)
File "/usr/lib/python3/dist-packages/pycoral/utils/edgetpu.py", line 169, in run_inference
_check_input_size(memory.size, expected_input_size)
File "/usr/lib/python3/dist-packages/pycoral/utils/edgetpu.py", line 141, in _check_input_size
input_size, expected_input_size))
ValueError: input size=925444, expected=924963.

Sorry guys, I forgot to push some important changes :)
https://github.com/Namburger/project-posenet
please do a git pull and try again, I'm traveling the first few days of this week and forgot my tpu at home so I can't really test this, but I'll try to get this repo fixed as soon as I get back!

Sorry guys, I forgot to push some important changes :)
https://github.com/Namburger/project-posenet
please do a git pull and try again, I'm traveling the first few days of this week and forgot my tpu at home so I can't really test this, but I'll try to get this repo fixed as soon as I get back!

It's working now! thank you.

Sorry guys, I forgot to push some important changes :)
https://github.com/Namburger/project-posenet
please do a git pull and try again, I'm traveling the first few days of this week and forgot my tpu at home so I can't really test this, but I'll try to get this repo fixed as soon as I get back!

@Namburger Does the synthesizer.py work for you with the pycoral one?

Yes synthesizer.py runs. I need to find some speakers to hear if its working.
Any way to get audio to default to HDMI output ?

Yes synthesizer.py runs. I need to find some speakers to hear if its working.
Any way to get audio to default to HDMI output ?

It also runs for me but I don't hear anything. When it first runs, it does generate some noise but that's about it

Closing the issue as examples got updated with pycoral API.


HassanTeslim007: Added a New Game (Wordle)

Added a new wordle game and tried to conform to existing UI designs and theme.

@harshptl14 what do you think?


Setting to disable scroll beyond last line

Lapce by default scrolls beyond the last line. Most IDEs have a setting for disabling this. It would be less jarring and make for a smoother user interface for some users who accidentally scroll past the last line frequently.

I feel that Lapce already does this

Do you mean that one should not be able to scroll beyond line 176 in the above example?

It can be turned off now in the latest release v0.1.2


"Install Wagtail in just 7 lines" doesn't works

I try to install wagtail following the instruction on the front page:
pip install wagtail
wagtail start mysite
cd mysite
pip install -r requirements.txt
python manage.py migrate
python manage.py createsuperuser
python manage.py runserver

Sadly it fail already on pip install wagtail:
Could not install packages due to an EnvironmentError: [Errno 13] Permission denied: '/usr/lib/python3.6/dist-packages/modelcluster'
Consider using the `--user` option or check the permissions.

I'm using a clean EC2 instance on AWS started from https://aws.amazon.com/cloud9/ with Python 3.6.7. I guess I should use a virtual environment.
The point is: if it can break so easily why showing those instruction? I just wanted to test wagtail and launched an VM in the cloud to try it, this give me a bad first impression.

No, I was able to run the same commands and get Wagtail up and running with only those 7 commands. The only issue you are facing is that you need to run those commands with adequate permissions. In my case, I ran those commands with 'sudo' permissions and they went through just fine. May be you could try the same?

I would recommend against installing with sudo, as this will install into a location normally reserved for operating system files - it's more difficult to clean up and risks breaking your system in ways that are hard to fix.
The smallest change that would work would be to replace pip install with pip install --user in both locations. While it's a less common approach, it makes for a nice quick demo.

I like the idea of using pip install --user - it's a good middle ground that avoids the two extremes of giving people advice that contradicts real-world best practices, versus "here's how to install Wagtail in 7 commands once you've read a whole load of documentation about Python virtualenvs" :-) And for people who are already familiar with virtualenvs, it should hopefully be self-explanatory enough that they should be able to confidently adapt the instructions accordingly. Thanks @BenSturmfels!
Any objections to this approach? I'll go ahead and update the page if not.

Why not adding virtual env directly? It doesn't matter if it's 7 or 10
commands. What matters is the commands works and that I can try something I
don't understand without risk for my own machine (so sudo is certainly to
avoid).
On Thu, Feb 14, 2019, 12:33 Matt Westcott<EMAIL_ADDRESS>wrote:

I like the idea of using pip install --user - it's a good middle ground
that avoids the two extremes of giving people advice that contradicts
real-world best practices, versus "here's how to install Wagtail in 7
commands once you've read a whole load of documentation about Python
virtualenvs" :-) And for people who are already familiar with
virtualenvs, it should hopefully be self-explanatory enough that they
should be able to confidently adapt the instructions accordingly. Thanks
@BenSturmfels https://github.com/BenSturmfels!
Any objections to this approach? I'll go ahead and update the page if not.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/wagtail/wagtail/issues/5047#issuecomment-463594864,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKA6tMh11AUequDJRXNexvdoUNrFlBnks5vNUmIgaJpZM4adK6E
.

@gagarine That's definitely something we can consider, if we can come up with a 3-or-so line snippet for setting up a virtualenv that reliably works across all machines irrespective of what is or isn't pre-installed - or, failing that, is self-explanatory enough that a user can adapt them to their setup without in-depth knowledge.
(For starters, that would probably imply using Python 3's venv rather than virtualenv - the latter just has too many system-specific variations.)
These instructions will always have to assume some kind of baseline starting point, both in knowledge and what's pre-installed on the system (even if that's just "knowing how to open and use the command line"). Reducing that baseline is a worthwhile goal, but that rapidly gets harder the less we assume, especially if you're aiming to make it a 'mechanical' process that someone can follow without understanding what's going on.

If venv is ommitted from the 7 lines, then it may be worth making mention of it on the Your First Wagtail Site page.
Worth noting that the 7 lines are on the home page and also duplicated on the Developers page.

I come to realize venv and package management is really the place where
python is hard for new user. Not because those concept are particularly
hard but because most python projects assume you already know them. In top
of that their is a lot of confusion between different tools (venv,
virtualenv,...). So nobody really tell you "this is the way to go".
For beginners I think it's absolutely necessary to include those kind of
things. To do that we certainly have to limit the beginner introduction to
python 3. Limiting is nice for beginners, as their is already so many
things you have to understand.
On Tue, Mar 26, 2019, 06:50 Ben Sturmfels<EMAIL_ADDRESS>wrote:

If venv is ommitted from the 7 lines, then it may be worth making mention
of it on the Your First Wagtail Site
http://docs.wagtail.io/en/latest/getting_started/tutorial.html page.
Worth noting that the 7 lines are on the home page and also duplicated on
the Developers page.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wagtail/wagtail/issues/5047#issuecomment-476484967,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKA6hNvL3m3E5K5EXBRsZR-mxa6L7S2ks5vabUmgaJpZM4adK6E
.

Thanks very much for the feedback @gagarine. We discussed some options in the last core team meeting and agreed on this:

the link goes to https://docs.python.org/3/tutorial/venv.html. Do you think this gives the right level of help to beginners?

@tomdyson yeah that good!

I've updated the wagtail.io homepage, as well as the documentation (in #5210), with thanks to @tomdyson. The instructions on the wagtail.io homepage note that you need a virtual environment without providing a link or instructions (because putting a link into the plaintext code block isn't trivial), but if you click through to the install guide, it has full instructions including venv.
I think that we should probably change that "Install guide" link to go directly to the docs (probably the Getting Started page) rather than to the Developers page.

I think that we should probably change that "Install guide" link to go directly to the docs (probably the Getting Started page) rather than to the Developers page.

👍 . We should make sure that both the homepage and Developers page point to the /stable/ version of the docs rather than /latest/ too.

Submitted for moderation the update to the homepage link. (The Developers page is locked.)

Sorry for the delay in approving! Links on homepage and developer page are now updated. (I've linked to the tutorial rather than the 'getting started' index page, as the tutorial is the one with the detailed instructions on virtualenvs - we ought to do something about that duplication really...)


build: add release-it to facilitate builds

Problem
Releases are currently manual, tedious and prone to human error. While fully automated releases would be great, creating and maintaining such scaffolding would require planning. A much easier alternative would be to automate as much of the process for the publisher as possible
Closes #274
Solution
Port release scaffolding from passport-saml for consistent publishing across all repos in the node-saml organization

npm install --save-dev release-it gren
add release-it config and changelog run script from passport-saml

Are we going to release this on NPM using the @node-saml organization too, or will we continue with the package as it currently is on NPM? What we did with node-saml and passport-saml is we put up a deprecation notice for the unscoped packages and re-published under an NPM organization. It might make sense to do that here.

@cjbarth It's up to you! We've just made one major release with breaking changes not too long ago, so we may as well rename the package (though we would have to copy over the DT types accordingly), since users will have to test before upgrading anyway.

I'm on board with making releases easier. Also, do we have someplace where we can manage release goals or have a discussion about the direction of this project?

@cjbarth @djaqua - I would like to have this PR merged so that we can move ahead with releases. As suggested, I will rename the package to @node-saml/xml-crypto

@LoneRifle , I made a few changes to this to use the latest package for generating changelogs and then added the first changelog. Have a look and LMK what you think. If you approve, I'll land it.

Can't approve something I raise myself! Go ahead and merge.


Pay Button active while waiting for cardTokenizeResponseReceived

Describe the feature you'd like to request
Right now the credit card pay button remains active while waiting for the cardTokenizeResponseReceived to be called and there is no indication that the button has been pressed / anything is happing.
Describe the solution you'd like
It would be great if the button was disabled and some sort of loading indication was displayed in order to inform the user that they don't need to press the button multiple times.
Describe alternatives you've considered
N/A

Thank you for this issue! We will be working on this and keep it you updated


fix(cloudmux): remoteFile metric

What this PR does / why we need it:
fix remotefile metric
If yes, write the release branches name in the below format and submit the related cherry-pick PR:

release/3.9

/cc @ioito

/lgtm


[IST0137] sample-bookinfo-argocd-server is associated with multiple services using targetPort "8080" but different ports: [80 443].

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
I am facing
Warning [IST0137] (Deployment sample-bookinfo/sample-bookinfo-argocd-server) This deployment sample-bookinfo-argocd-server is associated with multiple services [sample-bookinfo/sample-bookinfo-argocd-server sample-bookinfo/sample-bookinfo-argocd-server] using targetPort "8080" but different ports: [80 443].
Warning [IST0137] (Deployment sample-petclinic/sample-petclinic-argocd-server) This deployment sample-petclinic-argocd-server is associated with multiple services [sample-petclinic/sample-petclinic-argocd-server sample-petclinic/sample-petclinic-argocd-server] using targetPort "8080" but different ports: [80 443].
This waring is caused by the ports on the argocd server
It is possible to remove this warning according to the doc, but argo-cd is installed via Helm and deployment/service (argo-cd server deployment and service) cannot be controlled.
These warnings disappear in version 1.20.0, but I want to fix it without waiting for the upgrade.
Version
istioctl version: 1.20.0
control/data plane version: 1.18.5

Additional Information
I changed manually service ports and it was fixed but I want to control via helm values

# Analyze the current live cluster and suppress PodMissingProxy for pod mypod in namespace 'testing'.
istioctl analyze -S "IST0103=Pod mypod.testing"

You can use -S to suppress warnings.

Not sure if I understand, doesn't modifying the Helm values solve the problem? Or if you're looking to suppress the message, the analyze command has the -S flag or galley.istio.io/analyze-suppress annotation.

Not sure if I understand, doesn't modifying the Helm values solve the problem? Or if you're looking to suppress the message, the analyze command has the -S flag or galley.istio.io/analyze-suppress annotation.

like this, I can not disable https port from here, If I remove manually this port this warning will be removed so I need to disable this port using helm values or with another way

@qodirovshohijahon I think you should ask argo community instead of Istio, as we have no control over the manifest. You may also consider using things like Kustomize for overrides.


delombok + LGTM

https://quotidian-ennui.github.io/blog/2019/10/28/lombok-lgtm/

Using jabel instead
https://github.com/bsideup/jabel

Actually, let's support both.


Special Handling for recreating VCL windows

Take simple browser demo, and add this button to it
procedure TMainForm.btnReCreClick(Sender: TObject);
begin
  PostMessage(WVWindowParent1.Handle, CM_RECREATEWND, 0, 0);
end;

Run it and see the browser disappearing.
Indeed, why would Chrome know about so deep VCL pecularities.
I had to make a special case for it in CEF when mantaining our legacy Delphi 2007 app. And if you would say our app does something crazy... yes, it does. But.. but it does :-D

Perhaps we would need to make it hardened in this library too, especially since you seem to separate visual and non-visual parts, perhaps should be easier.
I did it in CEF by merely blocking the reactiong to this event on the CEF HWND parent level.
Maybe with this library we could do better, like [ab]using SetWindowLong to temporarily change the Win32-level parent before doing VCL-mandated TControl recreation.
Perhaps i would see to it later... Recording the issue so it would not be forgotten.

RecreateWnd destroys the handle and the web browser thinks the application is being destroyed.
Chromium and Edge close the browser when the parent window is destroyed.
Try setting TWVBrowserBase.ParentWindow to a different parent. Setting the property causes the WebView to re-parent the main WebView window to the newly provided window.


fix prepare_data bug

bug that resulting very small shuffled corpora and unclosed files.

After update all the modules, now I get the same error always at the end of the data file. Maybe this problem is not from 'prepare_data.py'. @orhanf, Could you check 'get_tr_stream' in stream.py? Thanks!

@heeyoulchoi can you please uninstall and clone a recent version of fuel since that is causing the problem. Please also check this thread #36

I am going to merge upon clearing #44  out of the way.

Oops, actually it has to rebased to include changes from #44 that make tests pass. @orhanf ,  can you please do this?

okay @rizar, done

I merged it, which I should not have done, because somehow it contains my recent commits, that is you rebased it wrongly. I reverted the merge, and created a new PR #45 to merge you changes properly.

@heeyoulchoi , as far as I understand after this PR the dataset created by prepare_data.py will contain more training examples, and I remember you said that you have troubles with overfitting.


transform model superglue onnx to engine failed

After building the libsuperpointglue_lib.so and superpointglue_image, I use the command like this:
./superpointglue_image ../config/config.yaml ../weights/ ../image/12.jpg ../image/34.jpg
but  an  error  occurred like this, "Error in SuperGlue building engine. Please check your onnx model path",
I'm sure the onnx model path is true, then add log in the C++ file, the error is parser->parseFromFile, it return false,
why the superpoint is right, and the superglue is not?

You may need to regenerate engine files on your inference platform.


Question About Pseudo-Label Generation and Code Location

Hello,
Thank you for sharing your work.
Could you please guide me on the following:

Where can I find the code related to generating pseudo-labels?
How do I run the code for pseudo-label generation? Are there any specific dependencies or setup instructions I should follow?

I appreciate your assistance and look forward to your guidance.

Hi, thanks for your interest in our work.

The pseudo labels can be generated either on-the-fly during training or offline. The training script for the pseudo label proxy task is here, where I generate pseudo label offline to have faster training speed. If you would like to train pseudo labels online, you can change the run_training_proxy.py by replacing the BasicTransform to ProxyTransform. Basically, the GetProxyLabeld in ProxyTransform.py explains everything on how the pseudo labels are generated.
As mentioned above, you can either generate the pseudo labels offline using GetProxyLabeld and use the run_training_proxy directly, or modify the run_training_proxy.py by using ProxyTransform. No additional dependencies are required.
Let me know if you have any other questions.