RuntimeError: Input type (torch.cuda.FloatTensor) and weight type (torch.FloatTensor) should be the same

HI! I followed the <README--No network connection, cannot load these models?> When I try to run the run.py,I still got this error:
Traceback (most recent call last):
  File "/root/autodl-tmp/Depth-Anything-main/run.py", line 84, in <module>
    depth = depth_anything(image)
  File "/root/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/root/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/root/autodl-tmp/Depth-Anything-main/depth_anything/dpt.py", line 158, in forward
    features = self.pretrained.get_intermediate_layers(x, 4, return_class_token=True)
  File "/root/autodl-tmp/Depth-Anything-main/torchhub/facebookresearch_dinov2_main/vision_transformer.py", line 308, in get_intermediate_layers
    outputs = self._get_intermediate_layers_not_chunked(x, n)
  File "/root/autodl-tmp/Depth-Anything-main/torchhub/facebookresearch_dinov2_main/vision_transformer.py", line 272, in _get_intermediate_layers_not_chunked
    x = self.prepare_tokens_with_masks(x)
  File "/root/autodl-tmp/Depth-Anything-main/torchhub/facebookresearch_dinov2_main/vision_transformer.py", line 214, in prepare_tokens_with_masks
    x = self.patch_embed(x)
  File "/root/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/root/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/root/autodl-tmp/Depth-Anything-main/torchhub/facebookresearch_dinov2_main/dinov2/layers/patch_embed.py", line 76, in forward
    x = self.proj(x)  # B C H W
  File "/root/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/root/miniconda3/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/root/miniconda3/lib/python3.10/site-packages/torch/nn/modules/conv.py", line 460, in forward
    return self._conv_forward(input, self.weight, self.bias)
  File "/root/miniconda3/lib/python3.10/site-packages/torch/nn/modules/conv.py", line 456, in _conv_forward
    return F.conv2d(input, weight, bias, self.stride,
RuntimeError: Input type (torch.cuda.FloatTensor) and weight type (torch.FloatTensor) should be the same

This sort of error suggests that the model & image are being processed by different devices (e.g. cpu vs gpu). In this case, it looks like the input image is on the gpu while the model is on the cpu.
It should be a simple fix. However you're loading the model, you'll just need to add .to(DEVICE) onto the end of it before processing the input image. You can see an example of this from the original run.py script (near the end of line 34).

I found a solution on some forums：
depth_anything.load_state_dict(torch.load(f'./checkpoints/depth_anything_{encoder}14.pth'))

Add the following line of code to the end of the previous line：
depth_anything.cuda()

is
    depth_anything.load_state_dict(torch.load(f'./checkpoints/depth_anything_{encoder}14.pth'))
    depth_anything.cuda()

Thanks for the valuable discussion. Since the issue has been resolved, I will close it for now,


Full Support for WebAuthn 2?

Discussed in https://github.com/tweag/webauthn/discussions/121

Originally posted by ohri-anurag January 28, 2022
Hi,
I came across this library while searching for Haskell versions of the WebAuthn spec. I have following the repo's progress for the past few weeks, I see that it is actively being worked on. Thanks a lot for building this.
I want to know if this library has full support for WebAuthn 2. If not, are there any plans to support the full spec in the future?
Regards,
Anurag Ohri

Answered in #121


[interpolatable] WIP use uharfbuzz if available

For faster drawing. However, I'm hitting a lot of
node-count-differs errors.  I have not tracked the cause down yet.
I thought this line might be:
https://github.com/harfbuzz/harfbuzz/blob/df635ab78ae1d397396e3a0fe688b8a4674b2e8a/src/hb-draw.hh#L148
But disabling it in uharfbuzz didn't help.

@khaledhosny Any help here is appreciated.

Reopening to investigate so at least we know who's stealing a point from the outline!

you need to explicitly emit the implied closing line otherwise a duplicate line segment at the end of a contour can get eaten up, see
https://github.com/fonttools/fonttools/blob/9c154e81c26dcd94622e01a2f984256778096055/Lib/fontTools/pens/pointPen.py#L231-L247

and that's because SegmentToPointPen always pops the last node when path is closed and last == first
https://github.com/fonttools/fonttools/blob/9c154e81c26dcd94622e01a2f984256778096055/Lib/fontTools/pens/pointPen.py#L321-L323
so to compensate and make sure points are roundtripped when going points => segments => points you need to make sure the segment representation has explicit closing lineTos

Yeah but doing that in HB didn't seem to help my problem. I'll investigate more.

The fonttools RecordingPen and draw() work fine for me. HB is doing something different.


Create methods for GET since endpoints

Currenty methods only exist for POST since endpoints
Related: #1

Pages endpoint (all_pages) has three methods:
GET /page
GET /page/since/{timestamp}
POST /page/since/{timestamp}
Obviously all_pages is reserved for /page, but how to differentiate between the two since methods?
all_pages_since(timestamp) works really nicely for the GET method
I guess there's all_pages_since_paginated(timestamp, ...) but ugh verbosity is the bane of me.

What about having an internal function that's the same but with the pagination values?
all_pages_since.paginated(timestamp, ...)

Would probably be a good idea to provide a pagination generator that can be called sequentially to elicit limit after limit of pages.


PHP-CGI randomly stops running with no error log

Running wpn-xm for a while, the server does not respond, when accessing php files. Further investigation lead me to the following post:
http://serverfault.com/questions/579773/wt-nmp-php-cgi-randomly-stops-running-with-no-error-log
Quote:
Php-cgi may stop for many reasons (OS, memory, networking), not related to your PHP code. On Linux production servers, there is a process manager (php-fpm) that monitors/restarts/logs such crashed processes. WT-NMP server manager dose not do that (yet).
It seems that wpn-xm server manager does not do that either. Am I right?
It would be great, if server-manager detects the count of running php-cgi-Processes and restart them, if they are not started as configured.

Is there a fix for this yet?

Yes, this was fixed.

For older PHP version php-cgi-spawner is used.
Newer versions of PHP (if i remember correctly, the fix shipped with 7.1) do not need the spawner anymore.
the Nginx config was changed to provide a PHP upstream pool by default (several PHP instances, each with multiple childs, if needed)


getServersToValidate should not include nodes

getServersToValidate should not include nodes:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/master/master.go#L552
Other than as a proxy, we're trying to make the master not contact the nodes. Besides, this is what the node API is for. /componentstatuses should be reserved for cluster-level (master) components.
For more details, see discussion from #6597.
cc @fabioy

@faiboy are you working on this? If so, please assign to yourself.

Commenting here, since the other thread has mixed in code-review comments.
@bgrant0607 - I don't disagree necessarily on this, but here's the thing: it's nice to have a single end point to get all the status of all the things the master knows about. I was hoping to keep "componentstatus" as that place. I think it's ok for it to retrieve the node status from its internal state, but not having to do multiple API calls to get the data is good.
@davidopp - Sorry, no, I wasn't working on this at this moment.

We've been steadily eliminating master-to-node communication. I fully expect that we'll break the current implementation in the not-too-distant future. The right solution isn't to fix this, but to eliminate it. The node API already needs to implement this for ongoing operation. It shouldn't be implemented twice, especially two totally different ways.
Also, when one has lots of nodes, they'd drown out the master components, making problems harder to debug.
And, a UI would want to display master and node health separately, so it would be useful to have them separated. Parsing the names isn't the best way to do that. I also find it confusing to assign the nodes component names that are different from the actual node names.

I don't disagree with that. I think eliminating the master->node check is fine (and needed -- you're right that it won't scale).
What I do want, though, is to be able to make a single call and get all of the health status that the master knows. It'd simplify the monitoring a lot (and the master already knows it). Presumably, for node health, it could be as simple as "last status" and "timestamp" (the later to determine if the status is still good or stale).
If "components" is not a good name for this, we can change it. But having to make multiple calls to reconstruct the overall cluster state seems crudy.

A new endpoint that's just limited to master components SGTM. Then we can argue about how /validate should work separately.

Please don't change /validate until we have a chance to migrate GKE to use a new validation endpoint. I think it would be OK (even if it's more work on our end) to hit two endpoints instead of on as long as we can validate the master components and validate the nodes to ensure the cluster is properly functioning.

This is related to #6047

This should have been fixed with #7735. Closing.


Deployment should validate prepaid until

Now this validation is inside BillingService. I believe these condition:
if dep.prepaid_until.blank?
        # This should not happen - it means that we do not know when this deployment will run out of funds.
        # Therefore we cannot meaningfully bill it again and must return an error.
        Rails.logger.error("Unable to determine current payment validity date for deployment #{dep.id} belonging to appliance #{appl.id}. Skipping.")
        BillingLog.new(timestamp: Time.now.utc, user: appl.appliance_set.user,
          appliance: "#{appl.id.to_s}---#{appl.name.blank? ? 'unnamed_appliance' : appl.name}",
          fund: appl.fund.name, message: "Unable to determine current payment validity date for deployment.",
          actor: "bill_appliance", amount_billed: 0).save
        dep.billing_state = "error"
        dep.save
        raise Atmosphere::BillingException.new(message: "Unable to determine current payment validity date for deployment #{dep.id} belonging to appliance #{appl.id}")

can be changed and moved into Deployment validation.

@nowakowski any progress here?

@mkasztelnik Could you explain why you want it done this way? The trigger condition is only considered erroneous when BillingService is in operation; otherwise it can be valid (for example in a fresh deployment). Moving this method to Deployment will introduce strong coupling between Deployment and BillingService as I will need to pass all the variables required for the BillingLog entry. This seems to me like a bad practice.


[msal-browser] Adding exports and other changes for extensibility

This PR adds additional exported files and other changes to further support extensibility in other libraries.

Codecov Report

Merging #4459 (e86d7f7) into dev (dc27ea9) will not change coverage.
The diff coverage is 100.00%.

Flag
Coverage Δ

*Carryforward flag

msal-angular
96.39% <ø> (ø)

Carriedforward from dc27ea9

msal-browser
87.50% <100.00%> (ø)

Carriedforward from dc27ea9

msal-common
85.37% <ø> (ø)

Carriedforward from dc27ea9

msal-core
82.65% <ø> (ø)

Carriedforward from dc27ea9

msal-node
82.97% <ø> (ø)

Carriedforward from dc27ea9

msal-node-extensions
76.03% <ø> (ø)

Carriedforward from dc27ea9

msal-react
92.70% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files
Coverage Δ

lib/msal-browser/src/app/ClientApplication.ts
96.29% <ø> (ø)

...ser/src/interaction_client/SilentAuthCodeClient.ts
79.31% <ø> (ø)

...owser/src/interaction_client/SilentIframeClient.ts
91.42% <ø> (ø)

...rc/interaction_client/StandardInteractionClient.ts
98.87% <ø> (ø)

...ib/msal-browser/src/app/PublicClientApplication.ts
100.00% <100.00%> (ø)

lib/msal-browser/src/cache/BrowserCacheManager.ts
93.03% <100.00%> (ø)

lib/msal-browser/src/index.ts
100.00% <100.00%> (ø)

...er/src/interaction_client/BaseInteractionClient.ts
98.03% <100.00%> (ø)

...msal-browser/src/interaction_client/PopupClient.ts
100.00% <100.00%> (ø)

...l-browser/src/interaction_client/RedirectClient.ts
100.00% <100.00%> (ø)

... and 8 more


Fix ruby 3.x Regexp.new deprecation

👋 A couple a minor fixes here in the hopes of moving further towards ruby 3.x:

Deprecate 2.6/2.7x (latest bundler versions support ><IP_ADDRESS>)
Ruby 3.3 deprecates the third argument in Regexp.new
reverts 23a87fd64 fix for jruby @ujifgc

This fixes #2278


Add support for triple slash single line comments for indicating documention blocks

Instead of using
/**
* My comments go here
* Second line comment
* third line comment
*/

I would like to use
/// My comments go here
/// Second line comment
/// Third line comment

I personally use block comments (i.e. /* ... */ ) for temporary commenting out code, not for documentation. This is because block comments cannot be nested, so I will always run into a problem where only part of the code gets commented out, forcing me to comment code out in parts or to use line comments (i.e. // ...) over multiple lines.
triple slash can be used to distinguish comments you want to have documented from the other comments in a similar fashion as /** is used now. (doxygen for instance supports this).

JSDoc includes a commentConvert plugin to support exactly this feature. See the documentation to learn how to enable plugins.

@hegemonic is the commentConvert plugin you mentioned still available somewhere?

maybe this... https://gitlab.cas.mcmaster.ca/theateam/FloppyFishGroup/-/blob/c953e275b9f61d66c4485b7331bcf933305b0c1e/JavaSriptDocs/jsdoc/plugins/commentConvert.js


Take down node modules from the repo!!

Use interactive rebase, and please take this out.

Add a .gitignore lol.

@KUHOO-S close this issue. It has been sorted in pull request #3


Unable to run the code

Hi!
Your work and repo are exciting! However, I am unable to reproduce your result successfully by following the command line you gave in the repo.
I would like to run the CelebA experiment.
The first issue is that, I think the command line  python data.py --data celeba --data_dir './data/ missing a apostrophe at the end?
Second is that... in celeba.py, the code cannot identify the path for the data and extra information (i.e list_landmarks_align_celeba.txt though they are not important here.).
After I change the relative path to absolute path, it starts to run. However, after 11 iterations, it reports another error from line 53 in cacheloader.py:
RuntimeError: The expanded size of the tensor (15000) must match the existing size (8500) at non-singleton dimension 0.  Target sizes: [15000, 2, 3, 32, 32].  Tensor sizes: [8500, 2, 3, 32, 32].
Meanwhile, another critical issue is that, the final variance is always INF or NAN. Do you have any clue?
Again, I really appreciate that you can share the repo with the community, and I am looking forward for your reply!

the command line is
python data.py --data celeba --data_dir '../data/'

Exactly! After I fixed this command line there still exists errors w.r.t path in celebA.py, but it is not so critical problem anyway, I can fix it by simply modify the path on my own.
The issue is mainly about the error happened after 11 iterations of training.
Meanwhile I noticed that the author fixed another bug raised in this issue (INF and NAN of variance occur during training) in the latest command! Thank you!

Thank you for raising this and apologies I did not see the initial message earlier

the data path issue will be corrected, this was an oversight as I did not want to put the full path with my username in case of anonymity issues
the inf variance issue was due to some tests trying inverse time in the network, these should not have been committed to the master branch and have since been addressed
I will look into the other error regarding size issue. I did not experience this and I suspect due to a bug when cache batch size is not a factor of the size of the dataset.
the celeba data loader was a bit naively copied from a previous paper, including the additional data files, I will check this and clean it. We did not notice an error when running, but it depends on downloading files from Google drive which may cause issues
We will soon release a cleaner version of the code in Jax which is under development. There has been a lot of progress in terms of training techniques.

Thank you! I am looking forward to your new version!

Just to be clear the celeb A issue is the filepath and you need to provide the correct one, I will make this more obvious if it was not clear. I cannot reproduce your error once I correct this.


fix README

@khawkins Hey, this is a minor change our technical writer asked for, please approve, thanks.


use markdown for description pane

for #20358
I decided to make it even better so that user can click on the links directly.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.01%) to 41.636%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/book/bookTreeView.ts
2
37.97%

extensions/notebook/src/jupyter/serverInstance.ts
4
75.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
27872

Relevant Lines:
62365

💛  - Coveralls


Addition of female population percentage for Afghanistan

Hi,
I want to add the female population percentage for Afghanistan.
Can I go ahead ?

Sure @ashitaprasad .. you can go ahead and raise a PR.


Scoping js variables to avoid polluting global scope

I've updated the tracking code with the latest generated by Matomo. The main difference resides in the scoping of variables declared when inserting the <script> tag. Previously it was polluting the global scope and wasn't recommended.

awesome, thank! 🙏


Use updated pysteam version

Fix for: https://github.com/scottrice/Ice/issues/449
Update Ice to use new pysteam from: https://github.com/scottrice/pysteam/pull/14.
Now that the launch options field has been added to shortcuts.vdf Steam seems to bug out if you have parameters in the exe field.
This puts just the exe location in the "exe" field and the formatted command into "launch_options"
Also adds all the extra data to all the Shortcut tuples for tests etc

@HEspoke so because pysteam is seperate library you have to install both locally to get it working. Another user has made a zip of an exe you can just download that has both here https://github.com/scottrice/Ice/issues/449


bluesky

adding a bluesky social to the sharing.json for my own purposes.

I don't really know what I'm doing but I tried adding Bluesky. it seems to work locally for me.

I am hoping to add Bluesky social sharing to the theme.


Scroll up force SwipeRefreshLayout to refresh

Scroll up force SwipeRefreshLayout to refresh, however, scroll down works properlly

any update on this?

@nmvictor 's solution worked for me but for clarity it was necessary to remove the internal ListView from its parent.
  ListView interalListView = headerListView.getListView();
  headerListView.removeView(internalListView);
  swipeRefreshLayout.addView(internalListView);

Hey @grossmae
can you see why my code is not working please ?
 mSwipeRefreshLayout = new SwipeRefreshLayout(getContext());
        RelativeLayout.LayoutParams swipeParams = new RelativeLayout.LayoutParams(RelativeLayout.LayoutParams.MATCH_PARENT, RelativeLayout.LayoutParams.MATCH_PARENT);
        swipeParams.addRule(ALIGN_PARENT_TOP);
        mSwipeRefreshLayout.setLayoutParams(swipeParams);
        mSwipeRefreshLayout.setColorSchemeResources(R.color.Yellow, R.color.android_green, R.color.honeycombish_blue, R.color.RedLight);

        ListView internalListView = headerListView.getListView();
        headerListView.removeView(internalListView);
        mSwipeRefreshLayout.addView(internalListView);``


Disabling prettier-loader when webpack starts

[x] I tried using the latest stable version of prettier-loader

[x] I've read readme

[x] Enhancement: speedup webpack start up by disabling prettier-loader (just for bootstraping)

In a project, i have a lot of HTML files defined as entry files thanks to html-webpack-plugin.
Loaders for HTML files are prettier-loader and html-loader.
Each HTML file needs 2 sec of processing by prettier (triggered by prettier-loader.
Cold start will go higher and higher everytime we add new entry HTML files.
As HTML are supposed to be prettified each time we save a file, the processing at startup is unneeded.
Disabling / skipping prettifying at startup would make the startup process way faster.

Naive (still working) approach: we could wait x milliseconds before enabling prettier.
let skip = true;
setTimeout(() => {
  console.log("prettier is now enabled")
  skip = false
}, 5000)

module.exports = async function(source, map) {
  this.cacheable();
  const callback = this.async();

  if (skip) {
    return callback(null, source, map);
  }

A demo project has been created : https://github.com/astik/prettier-loader-demo-issue-29

I think timeout with hardcoded value is not so good.
But we can create Set of files paths that should be processed.
On start Set will be empty.
After that we can check, if filepath is not in set, skip check, and add it to set.
On next handling this Set will containt this file path, and all prettier processing will work.
And of course we should hide this feature by new flag in loader options.
Ok?


Request abort race conditions

It seems that it is possible for an abort signal to be lost in some scenarios.  Perhaps request should always check whether a signal has been aborted before listening?

This has been found and corrected - the abort signal's event handler was being removed too early, meaning that it would be received and ignored.


Animation of the player's sprite

When trying to create an animation of the player's sprite, it gives out an error. I restarted the Android Studio, updated the packages of Flame. The problem is not in links, but in the package of Flame. Created a new project and tried to make an animation, the result is the same. Flutter also does not see the package 'Package: Flame/images.dart', although on the site it is used as an example

Below is the problem with exporting

@ZagumennikovYaroslav  This doesn't look like a bug in flame. The code you are referring to is for standalone use of image cache. But in your code you probably just want to use the default image cache. Instead of image you need to use gameRef.image.

well, then why does he write in line 53 and 54 the indefinite name "assets" and "images"

well, then why does he write in line 53 and 54 the indefinite name "assets" and "images"

Your code is in Player.onLoad and at that place images and assets are not defined anywhere. To use an object, it should first be created somewhere and should be accessible at the place where you are writing your code.
images and assets are members of the Game class. So you access them you'll have to get hold of your game class.
https://github.com/flame-engine/flame/blob/c6827a45dfecf261d001518e5088d3c676f05ae6/packages/flame/lib/src/game/game.dart#L23
https://github.com/flame-engine/flame/blob/c6827a45dfecf261d001518e5088d3c676f05ae6/packages/flame/lib/src/game/game.dart#L28


fix(angular): slide change render

https://stackoverflow.com/questions/65570802/angular-swiper-not-working-when-using-ngif

Merged, thanks!

Merged, thanks!


viewport.bounds returns illegal coordinate sets for large dimensions

bounds returns an illegal "top left" coordinate set (minX < -180) when the first member of the dimensions array passed in is larger than a certain threshold. This happens for valid viewport (centerpoint) coordinates.
We found this bug when a call to the mapbox geocoding API was returning an error: BBox minX value must be a number between -180 and 180. The minX value was less than -180, and was coming from the geo-viewport bounds function.
Example arguments that cause the bug: viewport.bounds([-98.5556199, 39.8097343], 3, [2892, 480], 512)
Bounds returned: [ -225.615234375, 21.861498734372567, 28.564453125, 54.1109429427243 ]
Failing test case available here: https://github.com/mapbox/geo-viewport/compare/master...coopy:geo-viewport:master#diff-9d2f3f2a83cca94e2e2185c0173c9170a80fdd3428bdf1c6f580f9d23f2fbe01R133

The longitude of the map's bounds can be greater than 180 (or less than -180). It will occur when the map is stretched and you can see multiple maps. Here's a visual example:

In your input it looks like the X/width of the canvas is ~3000px, which would correspond to the top left corner's latitude being at -225.61, which is correct even if it isn't technically a valid longitude value.
If I'm understand correctly, you passed the direct output of the bounds to the API and it gave you that error?
You'll want to cap or wrap the values for these cases.

Thank you for the detailed answer – that makes sense, though it wasn't immediately intuitive. I no longer have access to the code that produced this issue, but perhaps the call I was making to bounds happened before zoom level was set, and the map was "repeated" like in your example.
Since bounds usage together with the geocode API is likely common, a note in both sets of docs to encourage clamping could be helpful.
Closing!


[EntitySource] Modify entity source interface

The "Get" function of the entity source did not return an error and instead only a nil entity when its not found. This PR modifies the same, to make it return an error.
Reason:
The method could be implmented with a client that fetches bundle contents from the cluster. The error returned from the client needs to be propagated. Even if its from an offline key-value store, it could be upto the library users to ignore the returned error.
More info: https://github.com/operator-framework/operator-controller/pull/141#discussion_r1163271945

Looks good to me, but I'll defer to @perdasilva to make sure we aren't missing some important context about this.

(Something seems off with the merge requirement for someone from operator-framework/deppy-maintainers. Me, @perdasilva, @ankitathomas , and @everettraven have all approved, and apparently none of us are on that team. I'm going to use my admin powers to merge anyway.


homebridge 2.0 support

Is your feature request related to a problem? Please describe:
Homebridge will be releasing new version that will introduce breaking changes to some plugin
Describe the solution you'd like:
@lynxcs  What is the plan for supporting Homebridge 2.0 - will the plugin be impacted and will it be easy to make it compatible with 2.0.

Hi, currently there is no plan to support homebridge 2.0, at least from me. I will happily accept PR's if they appear, but in general - this repo/project is not maintained anymore, since I don't have access to CT200 device nor the motivation to keep working on it.

Hi, currently there is no plan to support homebridge 2.0, at least from me. I will happily accept PR's if they appear, but in general - this repo/project is not maintained anymore, since I don't have access to CT200 device nor the motivation to keep working on it.

I will fork the code to review it and check for break changes, and if i found something i can fix it and push it for your review.

@lynxcs  i check the code and there is no sign of old code that is depreciated in Homebridge 2.0 , so it can be used with coming homebridge 2.0.
cheers

That's fantastic, closing this issue.


refactor: Refactor data scratch-remote storage

Fixes #1753
Test Plan

How do we know the code works?

Code is refactored according to the description in #1753
Command flank auth login runs without any problems

Checklist

[x] Unit tested

@jan-gogo please confirm. More changes have been made.

@jan-gogo please confirm. More changes have been made.

@Sloox please check comment and apply changes according to it

@piotradamczyk5 issues addressed. Please review.


ML Notebooks should use Plotly.NET

Replace XPlot with Plotly.NET.
Blocked on..

Plotly.NET getting signed. https://github.com/plotly/Plotly.NET/issues/175
Also waiting on C# improvements https://github.com/plotly/Plotly.NET/issues/285

Both should be resolved: The newest releases have strong names and basic native C# bindings are available as well (https://github.com/plotly/Plotly.NET/pull/294, https://www.nuget.org/packages/Plotly.NET.CSharp/).

@kMutagene thoughts on the interactive package depending on the CSharp package?

@JakeRadMSFT what would be the benefit of that other than only referencing one package in C# notebooks. Ideally I would like to keep these packages separated, as that would also mean that an F# notebook using the interactive package would install an unused dependency.

Resolved with: https://github.com/dotnet/csharp-notebooks/pull/49


Link from German "About us" page to page for "Hendrik Ebbers" switches language to English

By switching the page to German (DE) (1) and going on the "About us" ("Über uns") page you will find a link to "Hendrik Ebbers" in the first paragraph. Clicking on that link will switch the page to English (EN). The page should stay in German language.

Link to German "About us" / "Über uns" page: https://open-elements.com/de/about/

I have found the fix for the issue
Fix the href in the file "content/about/index.de.md" to redirect to "/de/about-hendrik.html"  instead of just "/about-hendrik.html"

fixed by https://github.com/OpenElements/open-elements-website/pull/115#pullrequestreview-2347255267


kvm: Fix double-escape issue while creating rbd disk options

This fixes issue introduced in c3554ec31dafbdfaa0ed646afb17a6f3378571f5
which enable block of code that will double escape rados host/monitor
port.
Seen in logs:
2021-01-04 14:21:35,412 INFO  [kvm.storage.LibvirtStorageAdaptor] (agentRequest-Handler-4:null) (logid:c5187751) Creating volume 125a9ebb-de4c-49cf-b9a7-9249c4af53b9 from template d9eb3660-23e5-4b37-85ec-b6ad4978bf22 in pool 509bd564-b7e3-3d5f-b84b-b799055b11ca (RBD) with size (5.00 GB)<PHONE_NUMBER>

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

@blueorangutan package

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@GabrielBrascher had a discussion about this. As the port usually is 6789 the escaping wouldn't happen. Now the if-statement changed and this causes it to escape most of the time.
Gabriel is testing this to see if this fixes it.

@GabrielBrascher had a discussion about this. As the port usually is 6789 the escaping wouldn't happen. Now the if-statement changed and this causes it to escape most of the time.
Gabriel is testing this to see if this fixes it.

@rhtyd just adding a small change that is necessary in order to have all tests passing.
At line: https://github.com/apache/cloudstack/blob/261750c191f36ca93f0b8a030b00730b4a448c29/plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/KVMPhysicalDiskTest.java#L28
It needs to be updated from
... monitor\\\\:8000:auth_supported=...
to
... monitor\\:8000:auth_supported=...

@rhtyd just adding a small change that is necessary in order to have all tests passing.
At line: https://github.com/apache/cloudstack/blob/261750c191f36ca93f0b8a030b00730b4a448c29/plugins/hypervisors/kvm/src/test/java/com/cloud/hypervisor/kvm/storage/KVMPhysicalDiskTest.java#L28
It needs to be updated from
... monitor\\\\:8000:auth_supported=...
to
... monitor\\:8000:auth_supported=...

@DaanHoogland @rhtyd @wido tests are looking good :+1:
Additionally, from my tests, this also fixes https://github.com/apache/cloudstack/issues/4498.
Once the test class is updated this is good to go.

@DaanHoogland @rhtyd @wido tests are looking good :+1:
Additionally, from my tests, this also fixes https://github.com/apache/cloudstack/issues/4498.
Once the test class is updated this is good to go.

@GabrielBrascher can you send a PR to this one to include the test fix?

@GabrielBrascher can you send a PR to this one to include the test fix?

code lgtm
thanks @GabrielBrascher for testing.
@div8cn could you please test it as well ?

code lgtm
thanks @GabrielBrascher for testing.
@div8cn could you please test it as well ?

thanks @rhtyd @GabrielBrascher @weizhouapache
This patch has fixed the double-escape .
Now copy template to primary storage is running
2021-01-05 01:07:56,411 DEBUG [kvm.storage.LibvirtStorageAdaptor] (agentRequest-Handler-2:null) (logid:a8028c86) copyPhysicalDisk: disk size:(356.96 MB) 374304768, virtualsize:(2.44 GB)<PHONE_NUMBER> format:qcow2 2021-01-05 01:07:56,412 DEBUG [kvm.storage.LibvirtStorageAdaptor] (agentRequest-Handler-2:null) (logid:a8028c86) The source image is not RBD, but the destination is. We will convert into RBD format 2 2021-01-05 01:07:56,413 DEBUG [kvm.storage.LibvirtStorageAdaptor] (agentRequest-Handler-2:null) (logid:a8028c86) Starting copy from source image /mnt/466f03d4-9dfe-3af4-a042-33a00dae0e97/40165b83-896c-4693-abe7-9fd96b40ce9a.qcow2 to RBD image rbd/40165b83-896c-4693-abe7-9fd96b40ce9a 2021-01-05 01:07:56,413 DEBUG [utils.script.Script] (agentRequest-Handler-2:null) (logid:a8028c86) Executing: qemu-img convert -O raw /mnt/466f03d4-9dfe-3af4-a042-33a00dae0e97/40165b83-896c-4693-abe7-9fd96b40ce9a.qcow2 rbd:rbd/40165b83-896c-4693-abe7-9fd96b40ce9a:mon_host=<IP_ADDRESS>\:6789:auth_supported=cephx:id=hyperx:key=AQAywfFf8jCiIxAAbnDBjX1QQAO9Sj22kUBh7g==:rbd_default_format=2:client_mount_timeout=30  2021-01-05 01:07:56,414 DEBUG [utils.script.Script] (agentRequest-Handler-2:null) (logid:a8028c86) Executing while with timeout : 10800000 2021-01-05 01:08:16,919 DEBUG [cloud.agent.Agent] (agentRequest-Handler-4:null) (logid:248c7e0b) Processing command: com.cloud.agent.api.GetStorageStatsCommand

thanks @rhtyd @GabrielBrascher @weizhouapache
This patch has fixed the double-escape .
Now copy template to primary storage is running
2021-01-05 01:07:56,411 DEBUG [kvm.storage.LibvirtStorageAdaptor] (agentRequest-Handler-2:null) (logid:a8028c86) copyPhysicalDisk: disk size:(356.96 MB) 374304768, virtualsize:(2.44 GB)<PHONE_NUMBER> format:qcow2 2021-01-05 01:07:56,412 DEBUG [kvm.storage.LibvirtStorageAdaptor] (agentRequest-Handler-2:null) (logid:a8028c86) The source image is not RBD, but the destination is. We will convert into RBD format 2 2021-01-05 01:07:56,413 DEBUG [kvm.storage.LibvirtStorageAdaptor] (agentRequest-Handler-2:null) (logid:a8028c86) Starting copy from source image /mnt/466f03d4-9dfe-3af4-a042-33a00dae0e97/40165b83-896c-4693-abe7-9fd96b40ce9a.qcow2 to RBD image rbd/40165b83-896c-4693-abe7-9fd96b40ce9a 2021-01-05 01:07:56,413 DEBUG [utils.script.Script] (agentRequest-Handler-2:null) (logid:a8028c86) Executing: qemu-img convert -O raw /mnt/466f03d4-9dfe-3af4-a042-33a00dae0e97/40165b83-896c-4693-abe7-9fd96b40ce9a.qcow2 rbd:rbd/40165b83-896c-4693-abe7-9fd96b40ce9a:mon_host=<IP_ADDRESS>\:6789:auth_supported=cephx:id=hyperx:key=AQAywfFf8jCiIxAAbnDBjX1QQAO9Sj22kUBh7g==:rbd_default_format=2:client_mount_timeout=30  2021-01-05 01:07:56,414 DEBUG [utils.script.Script] (agentRequest-Handler-2:null) (logid:a8028c86) Executing while with timeout : 10800000 2021-01-05 01:08:16,919 DEBUG [cloud.agent.Agent] (agentRequest-Handler-4:null) (logid:248c7e0b) Processing command: com.cloud.agent.api.GetStorageStatsCommand

@DaanHoogland the only change necessary to pass all tests was the following commit: https://github.com/PCextreme/cloudstack/commit/cc6f7f7d560972213415f0130e3fb8d410c6ace6
for reference, I added it under ceph-double-quote-fix-test branch.

@DaanHoogland the only change necessary to pass all tests was the following commit: https://github.com/PCextreme/cloudstack/commit/cc6f7f7d560972213415f0130e3fb8d410c6ace6
for reference, I added it under ceph-double-quote-fix-test branch.

@blueorangutan package

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2523

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2523

ok, looks good, I'll await travis and assume that will do. Given that there is no ceph automation and we have manual confirmation, i'll create a new rc today.

ok, looks good, I'll await travis and assume that will do. Given that there is no ceph automation and we have manual confirmation, i'll create a new rc today.

Travis is green and we've manual tests from @div8cn and me, as well as lgtm from Daan and Wei. Merging this, let's cut RC4 after this.

Travis is green and we've manual tests from @div8cn and me, as well as lgtm from Daan and Wei. Merging this, let's cut RC4 after this.


Failed to locate VS2015 after installation

Hi,
As mentioned in the readme, I have to install VS2010+ in order to run nexe on Windows and I did that by installing VS2015. However, when running the nexe command I'm still faced with the same following errors on my NodeJS command prompt:
----> vcbuild [make stage]
Looking for Visual Studio 2015
Looking for Visual Studio 2013
Failed to find Visual Studio installation.
What would be the main cause for this problem?

Hi! This isn't handled by nexe. This is handled by node.js (I presume you're using 4.1+) Can you build regular node?

I have the same issue. I tried installing VS2015, VS2010 and VS2013 but none of it helped.
Any suggestion?
(I'm using Windows 10)

I can't reproduce on W8.1
Can't access 10 at the moment, but I'd take this up with node.js

I am running Windows 10 x64 and node.js found Visual Studio 2015 Community fine. Ensure that you have ran Visual Studio at least once (although I know that you probably have already). Otherwise, I can't reproduce.

I installed only the VS2013 after removed all VS from my system and now it works perfectly. I have no idea what was wrong at first.
Nice product! Keep up the good work! ;)

Glad this was resolved! Feel free to report any other issue you have! I try to work on this project when I can :)

I got this same error with VS2015, had to open VS -> new project -> install visual c++ tools

@mark-archer same here, thank you.

@mark-archer thanks for the solution, I just wonder how did you find it.

I forget.  I think I just assumed based on the error that it needed the c++ compiler but couldn't find it and then I assumed that could be gotten by installing the c++ tools.  Glad it's helping other people.

@mark-archer
it works!!!!!!thanks you a lot


feat: print error msg & stack on preset normalization error

Summary
When loading a jest-preset.js, which does some computation or load other packages, errors can occur.
This PR will help preset-developers to see the error message and stack. Currently only the misleading error message Module jest-preset-angular should have "jest-preset.js" or "jest-preset.json" file at the root. is displayed.
Inspired by https://github.com/thymikee/jest-preset-angular/pull/230, kudos to @thymikee
Test plan
Unit test added

Changed Regex /---.*---/s to /---[\s\S]*---/ so they also run in node v6.
Is the changelog entry ok?

Update 2: Simplified expected error message, as node v6 stacktraces it differently


"lid" variable

Super cool.
I might be wrong here, but in your while loop testing if the lid has been opened an closed, you wait for the value of "lid" to change. But "int lid" is just the pin number to read the lid state, not the lid state itself. Furthermore, there doesn't seem to be a reading of that pin at each iteration of the while loop for it to ever change. Am I missing something?

Good question, it is true that by looking at the code like this, it seems strange.
The trick is that for the moment I keep this code in master, it is that it functions corectly, and that it turns on the attiny.(well I have not yet succeeded on the attiny ;) And I expect that the version of psxdev support the attiny for the paser in master, and the code of both versions is not mergeable, (too much divergence).
P.S. I just gathered the code I didn't really take part in development.


feat(kafka): add producer config capabilities for connections

Description
For the sidecars using the existing kafka client configurations, after a connection has been closed by the client after the default connection idle timeout of 9 minutes, then if that sidecar goes to write and publish a message they will see a write: broken pipe err. This is due to the connection being deemed idle, but the sidecar has no idea, and so when it tries to publish, then it is doing so to a closed connection to which we see that error. It is quite confusing for end users, and so these metadata fields allow us to keep a connection alive and establish a refresh for the topic metadata such that the connection may be kept alive indefinitely if desired.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

I can update docs after I get initial feedback here pls :)

Looks like main still has rabbit mq issues based on these test results

unrelated failure due to stable components certification test for bindings.rabbitmq

ready!

this is now a dependency: https://github.com/dapr/kit/pull/88

ready

I've had a couple of issues with broken pipe so this fix will be well received :)

ready

ping for review please

ready for review please

@sicoyle please see latest comments from @JoshVanL. We should be good to merge once addressed

addressed feedback. Thanks @JoshVanL

ready @ItalyPaleAle Thanks for the extra feedback!


Replace com.google.common.base.Function (partly)

Replace com.google.common.base.Function (partly)

Are you making these changes by hand @pks-1981 ? You might want to have a look at these Guava recipes. I'm in the process of covering Optional there as well.

Are you making these changes by hand @pks-1981 ? You might want to have a look at these Guava recipes. I'm in the process of covering Optional there as well.

Yes, only with my hands) I'm have more fun in doing it myself...

Are you making these changes by hand @pks-1981 ? You might want to have a look at these Guava recipes. I'm in the process of covering Optional there as well.

Yes, only with my hands) I'm have more fun in doing it myself...

That's fine as well; even more appreciative of contributions made that way; Let's hope it's merged quickly without conflicts then!

@timtebeek @tomakehurst waiting for review)

@tomakehurst
maybe I'll merge my two MR?
if you're not planning merge soon...

@pks-1981 I'm on holiday this week, so please bear with me. Appreciate you opening these, and I plan to review them soon.

One test fails in a single configuration, likely to be a flaky one
ProxyAcceptanceTest > fixedDelaysAreAddedToProxiedResponses() FAILED
    java.lang.AssertionError:
    Expected: a value equal to or greater than <300L>
         but: <299L> was less than <300L>
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:6)
        at com.github.tomakehurst.wiremock.ProxyAcceptanceTest.fixedDelaysAreAddedToProxiedResponses(ProxyAcceptanceTest.java:558)
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.github.tomakehurst.wiremock.common.xml.Xml (file:/D:/a/wiremock/wiremock/build/classes/java/main/) to constructor com.sun.org.apache.xalan.internal.xsltc.trax.TransformerFactoryImpl()
WARNING: Please consider reporting this to the maintainers of com.github.tomakehurst.wiremock.common.xml.Xml
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

One test fails in a single configuration, likely to be a flaky one
ProxyAcceptanceTest > fixedDelaysAreAddedToProxiedResponses() FAILED
    java.lang.AssertionError:
    Expected: a value equal to or greater than <300L>
         but: <299L> was less than <300L>
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:6)
        at com.github.tomakehurst.wiremock.ProxyAcceptanceTest.fixedDelaysAreAddedToProxiedResponses(ProxyAcceptanceTest.java:558)
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.github.tomakehurst.wiremock.common.xml.Xml (file:/D:/a/wiremock/wiremock/build/classes/java/main/) to constructor com.sun.org.apache.xalan.internal.xsltc.trax.TransformerFactoryImpl()
WARNING: Please consider reporting this to the maintainers of com.github.tomakehurst.wiremock.common.xml.Xml
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

"blinking" test?
Checked on my Windows 11.


Fix ceph_all groups as hosts children

Adds children under hosts for ceph_all groups.
Signed-off-by: Nathan Pawelek<EMAIL_ADDRESS>

Thank you for submitting a pull request to this project!
Once all check jobs have passed and the pull request has received a
sufficient number of reviews, the repository's gate jobs can be triggered by
adding a comment of :shipit: to this pull request.
When the gate jobs have completed successfully, this pull request will get
merged automatically.


Update build.gradle

for fixing - Namespace not specified. Specify a namespace in the module's build file.

Kindly merge this as a priority to resolve the namespace issue.

Kindly merge this as a priority to resolve the namespace issue. @shaqian

update your pubspec.yaml for work around

  flutter_tflite:
    git:
      url: https://github.com/tech-ramakant/flutter_tflite.git
      ref: Namespace_not_specified_fix


Documentation for common gotchas

Inspired by a discussion on https://github.com/thoughtbot/Curry/pull/23 I have written up documentation on some common gotchas with using Argo. I put these into a separate markdown document that is linked under the "Basic usage" section.
Lemme know if this is on the right track of what y'all would expect from such a document. up for any kind of revisions to correct tone or meaning. There are probably more gotchas that could be added later. I mostly added things I've run into and scanned issues for some common mistakes.
NB: Please ignore all the commits... I used github's web interface to make these edits.

@mbrandonw Thanks man this is awesome!!!!
One quick comment, would you mind creating new lines at 80 characters per line? This would make it easier for us to leave you more focused comments.

@tonyd256 for sure, doing that and fixing a few mistakes. brb

@tonyd256  Ok done, and squashed commits.

@mbrandonw this is looking great. We should leave this open for a bit to get feedback from the rest of @thoughtbot/ios team and from @thoughtbot/copy-editing

All of this is really great. Made some comments, going to try to get more feedback from grammar people.

This is a fantastic write-up.

thanks for the notes! i'll be looking at this soon and take a pass at it

@mbrandonw Do you have time to finish this up? I'd be happy to take it over and apply fixes for the comments if you don't.

@mbrandonw Do you have time to finish this up? I'd be happy to take it over and apply fixes for the comments if you don't.

so sorry for being absent! if you could help out it would be much appreciated. i'm having a hard time finding spare time at this moment :/

@mbrandonw No worries! I'm happy to take it over.

Oh hey the revised version of this got merged so we can close it. Huzzah! 🎉


[SUGGESTION] maybe do not store FiledName into data?

Is your feature request related to a problem? Please describe.
i make a test store 1-million data in 2 files. one is CTS,one is CodeTask.

    public class CTS {
        public CTS(string code, int taskId) {
            id = code;
            d = taskId;
        }
        [LiteDB.BsonId]
        public string id;
        public int d;
    }

       public class CodeTask {

            public CodeTask(string code, int taskId) {
                Code = code;
                TaskId = taskId;
            }

            [LiteDB.BsonId]
            public string Code;

            public int TaskId;
        }

im got 2 files.

theres about 7mb size different.
if litedb make a field map(hold),maybe will be better ?

another  problem is when im query some field not in the data,it seems LiteDB will do full table scan ?
emm.

Hi @sgf, the size difference is due to the indexing system. LiteDB uses SkippedLists as the indexing system, which generates a random sized linked pointer array. So if you recreate these same databases again with the same amount of records, you should get different sized datafiles

Hi @sgf, the size difference is due to the indexing system. LiteDB uses SkippedLists as the indexing system, which generates a random sized linked pointer array. So if you recreate these same databases again with the same amount of records, you should get different sized datafiles

im just want to say,im open the db file,and its store the Field Name in there(every recored).

Hi @sgf, the size difference is due to the indexing system. LiteDB uses SkippedLists as the indexing system, which generates a random sized linked pointer array. So if you recreate these same databases again with the same amount of records, you should get different sized datafiles

im just want to say,im open the db file by notepad++ ,and its store the Field Name in there(every recored).

Ok, yes, this is by design. The object is serialized into a document on Bson format, which records the name of each property. This is a feature so you have the flexibility to store a modified object within the same collection without having to change its structure.


Uneven indent should not report on test case/keyword names

Originally posted by @mydeveloperplanet in https://github.com/MarketSquare/robotframework-robocop/discussions/283
We're iterating keyword and test cases by objects and saving it's lines for further indent analysis. In case if next object (like Documentation) is in the same line as test case / keyword name (which should be allowed - it's shown as example in user guide) it will be reported as having uneven indent.

@mydeveloperplanet
I've compared your code with examples in User Guide and done extra verification with Robocop. It actually looks like Robocop works fine.
See example from this section https://robotframework.org/robotframework/latest/RobotFrameworkUserGuide.html#pipe-separated-format . Whenever we're defining [Documentation] in the same line as keyword/ test case name we also align everything to that 'column':
| *** Test Cases *** |                 |               |
| My Test            | [Documentation] | Example test. |
|                    | Log             | ${MESSAGE}    |
|                    | My Keyword      | ${CURDIR}     |

Alternative is starting it in new line line in normal style:
*** Test Cases ***
My Test
    [Documentation]    Example test.
    Log                         ${MESSAGE}
    My Keyword           ${CURDIR}

In case of your code it could look like that:

or like that:

What do you think, do you agree?

Well, I agree with the layout you propose, but when I change it like 'in case of your code it could look like that', I receive the issue '1007 Line is under-indented'.
In case of 'or like that' I get numerous other issues, but the first one is '0202 Missing documentation in test case'.
That is why I am quite puzzled how I should resolve the issue.

Looks like my changes in PR #286 did fix '1007 Line is under-indented' that happens when I use code shown in 'like that' (I just checked in 'vanilla' code and I got the same result as you, then I've checked in PR code). Meaning that this problem should go away after next release. For now you can of course --exclude 1007
And you're right that 'or like that' is not valid code - I'm not that familiar with the pipe style thus I made syntax errors. It should be:
| Add Employee         |
| | [Documentation]      | Verify adding an employee
| | [Setup]              | Clear Employees List
| | Add Employee         | first_name=John | last_name=Doe
| | ${output}            | Retrieve Employees List
| | Should Be Equal      | ${output}       | ['John Doe']
| | [Teardown]           | Clear Employees List

(One pipe for test case / keyword name and two pipes for its body). This example should work before and after PR #286.

I am still in the experimenting phase, so this is just fine. Thanks for helping me out!


Creator POST Handler

A handler for POST requests (creating a resource) that allows the creator of the resource to automatically have access to the resource by creating the corresponding ACL.
Documentation can be found here.
@uvdsl Ping for deployment

Thanks! It's an item on the TODO now... :)


Events documentation and per_page param

What is the current behavior?
The docs for the Events API says the following:

Events support pagination, however the per_page option is unsupported. The fixed page size is 30 items. Fetching up to ten pages is supported, for a total of 300 events. For information, see "Traversing with pagination."

However, each of the API's does document a per_page parameter and it does seem to work fine:
$ curl -s https://api.github.com/repos/wayofthepie/ctlogs-cli/events?per_page=12 | jq '. | length'
12

What changes are you suggesting?
It seems like the section mentioning per_page is unsupported should be removed, if per_page is indeed meant to work for the Events API.

Thanks for opening an issue @wayofthepie! I'll triage this for the right team to take a look 👀

@wayofthepie Thanks for catching this discrepancy! You are correct that this paragraph should be removed. ✨ You or anyone else is welcome to open a PR to address this issue.

@wayofthepie Thanks for catching this discrepancy! You are correct that this paragraph should be removed. ✨ You or anyone else is welcome to open a PR to address this issue.

I am new with this could I able to contribute in this work as I am still learning

@KrKeshav09 Take a look at the contributing.md to learn how to contribute in this repo. You can open a discussion if you have any questions or check out issues with the help wanted label. :sparkles:

Please check PR #4362


doc: move USB documentation to connectivity

doc: move USB documentation to connectivity
Mostly moved and reorganised content to the Connectivity chapter,
separated API references. Fill in some gaps in the USB device
documentation.
doc: connectivity: move USB-C device stack to USB chapter
This looks a bit lost in its own USB-C chapter,
move it to USB under the Power Delivery heading.
@sambhurst @keith-zephyr please take a look at last patch, I can drop it if you have any objections.

Very welcome change!


packaging.version.InvalidVersion and DeprecationWarning (leading to) errors during testing

Hi! I'm packaging molecule for Arch Linux.
When building 3.2.0 as a package I ran the test suite against the built package. Unfortunately two tests are failing (during collection). These worked with 3.1.5 (the last version I packaged, but that was also on python 3.8).
Issue Type

Bug report

Molecule and Ansible details
ansible 2.10.3
  config file = /etc/ansible/ansible.cfg
  configured module search path = ['/home/dave/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python3.9/site-packages/ansible
  executable location = /usr/bin/ansible
  python version = 3.9.0 (default, Oct  7 2020, 23:09:01) [GCC 10.2.0]
molecule 3.2.0 using python 3.9
    ansible:2.10.3
    delegated:3.2.0 from molecule

Molecule installation method (one of):

source

  sed -e '/addopts/d' -i setup.cfg

  python setup.py build
  export PYTHONPATH="${PWD}:build:${PYTHONPATH}"
  pytest -v src/molecule/test/unit/

Ansible installation method (one of):

OS package

Detail any linters or test runners used:
Desired Behavior
All unit tests pass
Actual Behaviour
============================= test session starts ==============================
platform linux -- Python 3.9.0, pytest-6.1.2, py-1.9.0, pluggy-0.13.1 -- /usr/bin/python
cachedir: .pytest_cache
metadata: {'Python': '3.9.0', 'Platform': 'Linux-5.9.10.a-1-hardened-x86_64-with-glibc2.32', 'Packages': {'pytest': '6.1.2', 'py': '1.9.0', 'pluggy': '0.13.1'}, 'Plugins': {'xdist': '2.1.0', 'forked': '1.3.0', 'verbose-parametrize': '1.7.0', 'testinfra': '6.1.0', 'mock': '2.0.0', 'html': '3.0.0', 'metadata': '1.11.0',
 'helpers-namespace': '2019.1.8'}}
rootdir: /build/molecule/src/molecule-3.2.0, configfile: setup.cfg
plugins: xdist-2.1.0, forked-1.3.0, verbose-parametrize-1.7.0, testinfra-6.1.0, mock-2.0.0, html-3.0.0, metadata-1.11.0, helpers-namespace-2019.1.8
collecting ... collected 431 items / 2 errors / 429 selected

==================================== ERRORS ====================================
____________ ERROR collecting src/molecule/test/unit/test_shell.py _____________
/usr/lib/python3.9/site-packages/packaging/version.py:57: in parse
    return Version(version)
/usr/lib/python3.9/site-packages/packaging/version.py:298: in __init__
    raise InvalidVersion("Invalid version: '{0}'".format(version))
E   packaging.version.InvalidVersion: Invalid version: '/usr/lib/python3.9/site'

During handling of the above exception, another exception occurred:
src/molecule/test/unit/test_shell.py:23: in <module>
    from molecule import shell
src/molecule/shell.py:121: in <module>
    if pkg_resources.get_distribution("futures"):
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:465: in get_distribution
    dist = get_provider(dist)
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:341: in get_provider
    return working_set.find(moduleOrReq) or require(str(moduleOrReq))[0]
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:884: in require
    needed = self.resolve(parse_requirements(requirements))
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:757: in resolve
    env = Environment(self.entries)
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:968: in __init__
    self.scan(search_path)
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:1000: in scan
    for dist in find_distributions(item):
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:2056: in find_on_path
    path_item_entries = _by_version_descending(filtered)
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:2026: in _by_version_descending
    return sorted(names, key=_by_version, reverse=True)
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:2024: in _by_version
    return [packaging.version.parse(part) for part in parts]
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:2024: in <listcomp>
    return [packaging.version.parse(part) for part in parts]
/usr/lib/python3.9/site-packages/packaging/version.py:59: in parse
    return LegacyVersion(version)
/usr/lib/python3.9/site-packages/packaging/version.py:127: in __init__
    warnings.warn(
E   DeprecationWarning: Creating a LegacyVersion has been deprecated and will be removed in the next major release
_______ ERROR collecting src/molecule/test/unit/command/test_converge.py _______
/usr/lib/python3.9/site-packages/packaging/version.py:57: in parse
    return Version(version)
/usr/lib/python3.9/site-packages/packaging/version.py:298: in __init__
    raise InvalidVersion("Invalid version: '{0}'".format(version))
E   packaging.version.InvalidVersion: Invalid version: '/usr/lib/python3.9/site'

During handling of the above exception, another exception occurred:
src/molecule/test/unit/command/test_converge.py:24: in <module>
    from molecule.shell import main
src/molecule/shell.py:121: in <module>
    if pkg_resources.get_distribution("futures"):
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:465: in get_distribution
    dist = get_provider(dist)
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:341: in get_provider
    return working_set.find(moduleOrReq) or require(str(moduleOrReq))[0]
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:884: in require
    needed = self.resolve(parse_requirements(requirements))
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:757: in resolve
    env = Environment(self.entries)
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:968: in __init__
    self.scan(search_path)
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:1000: in scan
    for dist in find_distributions(item):
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:2056: in find_on_path
    path_item_entries = _by_version_descending(filtered)
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:2026: in _by_version_descending
    return sorted(names, key=_by_version, reverse=True)
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:2024: in _by_version
    return [packaging.version.parse(part) for part in parts]
/usr/lib/python3.9/site-packages/pkg_resources/__init__.py:2024: in <listcomp>
    return [packaging.version.parse(part) for part in parts]
/usr/lib/python3.9/site-packages/packaging/version.py:59: in parse
    return LegacyVersion(version)
/usr/lib/python3.9/site-packages/packaging/version.py:127: in __init__
    warnings.warn(
E   DeprecationWarning: Creating a LegacyVersion has been deprecated and will be removed in the next major release
=========================== short test summary info ============================
ERROR src/molecule/test/unit/test_shell.py - DeprecationWarning: Creating a L...
ERROR src/molecule/test/unit/command/test_converge.py - DeprecationWarning: C...
!!!!!!!!!!!!!!!!!!! Interrupted: 2 errors during collection !!!!!!!!!!!!!!!!!!!!
============================== 2 errors in 0.90s ===============================

full build log:
molecule-3.2.0-build.log
I have tried:

installing to temporary location and running the tests suite against that installation (same result, but different reported directory for the "version"
skip collection of failing tests (remaining tests run successfully)
skipping tests and installing the package (the version output works)

For comparison, here's the output when running the tests against a temporary installation of molecule (but disabling the two test files in question):
  local _python_version="$(python -c 'import sys; print(".".join(map(str, sys.version_info[:2])))')"
  local _test_dir='test_dir'

  # install into test location
  mkdir -vp "${_test_dir}"
  python setup.py install --skip-build \
    --optimize=1 \
    --root="${_test_dir}"

  export PYTHONPATH="${PWD}/${_test_dir}/usr/lib/python${_python_version}/site-packages/:${PYTHONPATH}"
  export PATH="${PWD}/test_dir/usr/bin:${PATH}"

  # skip collection of broken pkg_resources tests: https://github.com/ansible-community/molecule/issues/3004
  pytest -v \
         --ignore="${_test_dir}/usr/lib/python${_python_version}/site-packages/molecule/test/unit/command/test_converge.py" \
         --ignore="${_test_dir}/usr/lib/python${_python_version}/site-packages/molecule/test/unit/test_shell.py" \
         "${_test_dir}/usr/lib/python${_python_version}/site-packages/molecule/test/unit/"

Full build log:
molecule-3.2.0-build.log

With 3.2.1 this situation deteriorated further. I am now unable to run the test suite, even if I ignore molecule/test/conftest.py as described in the previous comment. I will not be able to upgrade molecule on Arch Linux until this is resolved.
molecule-3.2.1-build.log

This error is still present in 3.2.2 which is why I am not able to upgrade to that version either.
Any hints on how to resolve this issue?

This error is still present in 3.2.2 which is why I am not able to upgrade to that version either.
Any hints on how to resolve this issue?

Hm, searching further it seems this is related to https://github.com/pypa/setuptools/issues/2466
It looks like molecule needs to be patched to work properly on py39.

Hm, searching further it seems this is related to https://github.com/pypa/setuptools/issues/2466
It looks like molecule needs to be patched to work properly on py39.

Hey @dvzrv molecule is already CI tested against py39 over at https://github.com/ansible-community/molecule/runs/1678619552?check_suite_focus=true. Haven't looked thoroughly at this ticket but just an FYI. I also use molecule on py39 on my own testing.

Hey @dvzrv molecule is already CI tested against py39 over at https://github.com/ansible-community/molecule/runs/1678619552?check_suite_focus=true. Haven't looked thoroughly at this ticket but just an FYI. I also use molecule on py39 on my own testing.

With a little help I figured out that the filterwarnings setting for pytest is the problem here, as it raises warnings as errors. Is there any reason to apply that setting?

@dvzrv That option is there on purpose as we want to consider any deprecation warning an error. As our CI is passing that is not a bug. This prevents us from adding code that makes use of deprecated APIs for example.  When using molecule at runtime warnings can appear and will not be errors.
Shortly we are very strict when running tests but less strict in production, as we want to avoid breaking user workflows.


DockerV2: Invalid base image name with multi-stage builds using COPY --from

Required Information
Question, Bug, or Feature?
Type: Bug
Enter Task Name: DockerV2
list here (V# not needed):
https://github.com/microsoft/azure-pipelines-tasks/tree/master/Tasks/DockerV2
Environment

Server - Azure Pipelines
Agent - Private

Issue Description
Sample Dockerfile to reproduce this issue:
FROM golang:1.16 AS builder
...
FROM alpine:latest
COPY --from=builder . .

The COPY --from=builder command is picked up  as FROM command resulting in an invalid base image name in this function:
https://github.com/microsoft/azure-pipelines-tasks/blob/14e609d787a7c29d12b4fe4cb275464dd0db5b19/common-npm-packages/docker-common-v2/containerimageutils.ts#L66
Task logs
invalid reference format

Comments like # Copy files from A to B are also taken into account when trying to get the base image name. The current implementation seems to be very immature.
Greetings!

Docker FROM seems very well defined. It's more than a little naive to just look for the word "from". A simple regex would fix this

Please don't close this issue. It is not fixed. I may look at providing a solution myself


Repo sync for protected CLA branch

The pull request is created from main637782301052707697 to main to fix git push error for protected CLA branch

@shsagir : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Update README.md => Question 23 #742

@jakeherp This is the new PR as i couldn't reopen the previous one


[Question❓] Foreground transparency issue

Hello Thomas!
I am using Pymatting to remove green screen background from portraits of people.
I noticed that there is an issue with foreground estimation (I use estimate_alpha_lkm and estimate_foreground_ml) - on some images there are transparent pixels inside the foreground, even though the trimap looks correct. Do you have any idea what could cause this issue and how to solve it? Are there any parameters that need modifying?
I attach here an example - a raw image, its trimap and cutout. On this image the issue is mostly with the neck area and other skin areas (you need to download the image and open it with some image viewer application to actually see it). On other images it could happen that a part of clothing is partially transparent. It is very subtle and hard to notice but it causes some issues for me further on.
Thanks in advance for your help! I really appreciate all you have helped me with!
Let me know if you need any other details from me!
Best regards,
Tatiana

Yes, this definitely makes sense to me! I will test and see if it improves the performance!
Thanks again for your help!


Working with System.Xml.Serilaziation on WebGL

Hello, I implemented your file browser system into my project which uses .xml files to save objects and their properties in the scene. It works perfectly on editor and windows build (for which i am very thankful) but it does not work on WebGL. I wanted to ask if you have any idea about the problem.
Thanks

It turned out to be about Xmlserializer and .NET Api Compatibility Level. Completely different thing. XmlSerialization with 4.x does not work on WebGL. Should be 2.0


Feature/metadata additional fields

What
Added additional fields and ux changes to dataset and version metadata pages.
How to review
All fields should now work correctly with the dataset api. Check that all fields can be created, saved, changed etc.
Note - currently unable to delete any data that's stored as an array i.e. alerts, changes. API change is required.
Who can review
Anyone.

Looks good, the metadata screens have come a long way in this PR! A few things to note, not for you alone, but for all of us to come back to at some point:

If you edit a dataset, add it to a collection, then preview, the back link takes you to the version metadata screen. Should it take you back to the dataset metadata screen?
If you edit the dataset title the 'Dataset title` heading also changes. Do we want to keep this as the original and only update it once it's been saved?
I'm getting a warning somewhere (I think the dataset metadata screen?) about an uncontrolled input.
I think we need to make lots of choices about what inputs are disabled. For example, should I even get the option to change an edition input back to empty after it's been selected? Also, should we scroll users to the top of the page where there are all of the validation errors listed, because it could be off screen if not. Etc etc etc.
I got a 404 when PUTting versino metadata changes, I think due to my own local issues, but there were no errors messages and I went to the datasets screen without knowing that anything hadn't worked. We probably need to show some notification to show something when wrong.

I think it's worth merging this PR in and just making tickets / having a chat with UX about what we want to do with the above comments It feels like we want to know whether these screens actually work for the users before we start polishing them / bug fixing.


Property 'captureStackTrace' is not a function

Hello. I am not a direct user of verror, only indirect via restify, and I have this problem:
When running tests against my rest api and an assertion-error is thrown the entire test runner breaks down because of this error:
/home/ludwig/myproject/node_modules/restify/node_modules/verror/lib/verror.js:147
	Error.captureStackTrace(this, ctor || this.constructor);
	      ^
TypeError: Property 'captureStackTrace' of object function Error() { [native code] } is not a function
  at [object Object].WError (/home/ludwig/myproject/node_modules/restify/node_modules/verror/lib/verror.js:147:8)
  at [object Object].HttpError (/home/ludwig/myproject/node_modules/restify/lib/errors/http_error.js:44:12)
  at [object Object].RestError (/home/ludwig/myproject/node_modules/restify/lib/errors/rest_error.js:41:15)
  at new module.exports.(anonymous function) (/home/ludwig/myproject/node_modules/restify/lib/errors/rest_error.js:86:19)
  at Server.<anonymous> (/home/ludwig/myproject/node_modules/restify/lib/index.js:111:18)
  at Server.emit (events.js:106:17)
  at Domain.onError (/home/ludwig/myproject/node_modules/restify/lib/server.js:775:18)
  at Domain.emit (events.js:95:17)
  at Async$_consumeLateBuffer [as _consumeLateBuffer] (/home/ludwig/myproject/node_modules/bluebird/js/main/async.js:93:27)
  at Async$_consumeFunctionBuffer [as _consumeFunctionBuffer] (/home/ludwig/myproject/node_modules/bluebird/js/main/async.js:80:10)
  at Async$consumeFunctionBuffer (/home/ludwig/myproject/node_modules/bluebird/js/main/async.js:40:14)
  at process._tickDomainCallback (node.js:463:13)

I really don't understand much of this but I notice that in some parts of the verror code there is a check: ìf (Error.captureStackTrace) before using that function. This does not exist at the specific line (147) but if I add a check at that point my problem disappears and the assertion error message is correctly printed.
Is this a check that can be added to the code to solve my problem?

This should be fixed by my refactoring for #10, but I'm still not sure why Error.captureStackTrace isn't present when this happens.  Do you understand why that's happening?

Sorry, this was a long time ago. I can't really remember the issue and I haven't run across it since we last spoke.

Closing as a dup of #10, which should have fixed this.


[Doc] Explain the reason for Authentication.Policy.Password.Reset.Time.In.Days property

In doc [1] we have to mention why we need to configure Authentication.Policy.Password.Reset.Time.In.Days property and what will happen if we increase or decrease the value.
[1] https://docs.wso2.com/display/ISCONNECTORS/Configuring+Password+Policy+Authenticator

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


Donatorform

Naming Rules
Name your PR like this: ISSUENUMBER-TITLE-YOURNAME
Description
Write a summary of your change. Explain why you have made it.
created react component to display the donator form
create API endpoint: POST /api/laptop_donation
created laptop donation table on the database
the data from the form is now being saved to the database.
Related to
Make sure you include the issue number with a hash sign # so Github can link this PR to the right issue, like this:
Fixes #1
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have carefully reviewed my own code
[ ] I have commented my code
[ ] I have updated any documentation

@gregdyke , please review my code for the donation form.

Sorry to be difficult. Could you format the code with eslint instead of prettier? (if you turn of autosave and then manually save it should happen automatically). (The reason is that other files are formatted with eslint and it will make PRs harder to read if different formatters are applied everywhere)

Merging as I think all that remained was styling stuff that can be dealt with separately.


Revert "Allow the client to specify custom context"

Reverts richtier/alexa-voice-service-client#8

Codecov Report

Merging #10 into master will decrease coverage by 0.54%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master      #10      +/-   ##
==========================================
- Coverage   94.06%   93.51%   -0.55%
==========================================
  Files          11       11
  Lines         219      216       -3
==========================================
- Hits          206      202       -4
- Misses         13       14       +1

Impacted Files
Coverage Δ

avs_client/avs_client/client.py
100% <100%> (ø)
:arrow_up:

avs_client/avs_client/connection.py
95.74% <100%> (ø)
:arrow_up:

avs_client/avs_client/device.py
66.66% <50%> (-33.34%)
:arrow_down:

Continue to review full report at Codecov.

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


Getting metric value for each user

Hi, i am currently working on a project where I am trying to compare different algorithms together to see if the results are statistically significant. How can I get the HR, NDCG and MRR values for each user instead of getting one value for the results.
For example, if my dataset has 10 users in the test set, how can I retrieve all the 10 MRR,NDCG and HR values for each user instead of an average. As of right now, the model is returning "INFO  test result: OrderedDict([('recall@10', 0.0233), ('mrr@10', 0.0098), ('ndcg@10', 0.0125), ('hit@10', 0.025), ('precision@10', 0.0025)])". But, I want each metric for all the users in the dataset.
Please let me know how to get that.
Thank you!

Is there an update on this issue?

Hello @ajaykv1 , sorry for the late reply.
We reuse RecBole's implementation in the evaluation module. So you need to modify some of your RecBole code to do this, or rewrite it under RecBole-CDR. Take the example of modifying RecBole code:

You first need to find the metrics that you want to calculate for each user, take Hit@10 for example.
Locate the Hit class in RecBole/recbole/evaluator/metrics.py.
Then you can find result in calculate_metric function, this is metric for each user.

    def calculate_metric(self, dataobject):
        pos_index, _ = self.used_info(dataobject)
        result = self.metric_info(pos_index)   ###This is metric result for each user
        metric_dict = self.topk_result('hit', result)
        return metric_dict


time: ParseInLocation does not adjust timezone offset when seconds are not explicitly set

What version of Go are you using (go version)?

$ go version
go version go1.18.4 darwin/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/mark/Library/Caches/go-build"
GOENV="/Users/mark/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS="-mod=mod"
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/mark/go/pkg/mod"
GONOPROXY="github.com/flypay/*,github.com/faetools/*"
GONOSUMDB="github.com/flypay/*,github.com/faetools/*"
GOOS="darwin"
GOPATH="/Users/mark/go"
GOPRIVATE="github.com/flypay/*,github.com/faetools/*"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.18.4/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.18.4/libexec/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.18.4"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/mark/go/src/github.com/golang/go/src/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/rw/2cbtm9v95bz86c0ktyjwj2_80000gn/T/go-build2898766309=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I ran into an issue where a time.Time object was seconds off after having been converted into a string using Format with RFC3339 and back using ParseInLocation.
https://go.dev/play/p/5t8wa3b_ahv
What did you expect to see?
The test above succeeding, i.e. when I explicitly state my desired location that go is smart enough to adjust the seconds.
What did you see instead?
The test failing.
This is because the parsed time zone differs mere seconds from the desired time zone and RFC3339 expresses a time zone only in hours and minutes.

The Z0700 represents a fixed time offset, not an actual timezone location. You would either need a named timezone or a the timezone offset with seconds to accurately represent your time.
second zone offset: https://go.dev/play/p/ivhN_nsQ_qB
named zone: https://go.dev/play/p/pVPflGCeBDk
See also #37277

cc @rsc

Thank you, @seankhliao. Yes, I realize that. The expectation was that ParseInLocation is smart enough to work with the standard format RFC3339.
BTW I encountered the error when writing custom JSON marshalling and unmarshalling of an object that has a field for the time and a timezone string. For the API client to work, times have to be marshalled using standard RFC3339.

It seems to me that this problem occurs because the tzdata database says that the timezone for Asia/Shanghai in 1900 is 8:05:43, with a note saying "Milne gives 8:05:43.2 for Xujiahui Observatory time; round to nearest."  However, the time format defined by RFC 3339 does not support seconds in the timezone, only hours and minutes.  So printing a time that has a timezone offset measured in seconds produces unavoidable data loss.  You are suggesting that we try to compensate for that, but I don't see how to do that in a way that is always accurate.
So I don't think that there is anything to do here.  Code that needs to worry about these old dates for which the timezone database uses second offsets can check the timezone offset via t.Zone, and compensate for unusual cases.

Closing as there appears to be nothing to do


Consider updating the docgen.mjs script that updates configuration-reference

Opening this issue about the docgen.mjs script to be able to auto-update configuration-reference.md as a scheduled github action.
Ideas mentioned with this:

Schedule it as 1 time per day / two days ( to avoid the complexity of listening on changes from withastro/astro types )
Updating the current docgen.mjs to be compatible with github CI
Converting automatically https://docs.astro.build/ to relatives links
Automatically merge changes on the repo ( and be sure it trigger the i18n CI loop )

We have a live test-case available:
"The configuration documentation says the default value of trailingSlash is always. But it's wrong, the default value is ignore https://docs.astro.build/en/reference/configuration-reference/#trailingslash"

"The link inside the config ref is absolute, can cause issues with dev environnements and redirecting when unnecessary https://github.com/withastro/docs/blob/main/src/pages/en/reference/configuration-reference.md?plain=1#L266"

Wait, is this the same as #209 @Chrissdroid ? or related? If so, maybe you can add info there.

we could merge this here for the first case but i think it's more suitable to make a separate thing for the "CLI Flag" automatisation
( which could be pushed toward with changes from this )

Good catch, @sarah11918. Do we prefer moving this to #209 to keep all docgen-related ideas in one place? I can copy this over to that issue in that case.

If it's easier together because it's targeting the same script, perfectly fine to copy over stuff from here and just keep going ahead in #209 . If you want to keep the specific ideas separate, I have no problems having two different issues.

OK! I’ve moved this over there. That way if someone’s looking at one of these they might pick up the others if it makes sense to tackle more than one at once. Thanks so much for collating the different thoughts here @Chrissdroid 💜


Repo sync for protected CLA branch

The pull request is created from main638076754345658787sync_temp to main to fix git push error for protected CLA branch

Learn Build status updates of commit da0c909:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Windows 11 is Crashed while playing YouTube in Chrome

Microsoft PowerToys version
0.76.2
Installation method
Microsoft Store
Running as admin
No
Area(s) with issue?
General
Steps to reproduce
When Error is occurred
Windows 11 is Crashed while playing YouTube in Chrome
Error Message
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.22631.0
IntPtr Length: 8
x64: True
Date: 1/30/2024 1:06:50 AM
Exception:
System.Runtime.InteropServices.COMException (0x80263001): {Desktop composition is disabled} The operation could not be completed because desktop composition is disabled. (0x80263001)
at Standard.NativeMethods.DwmExtendFrameIntoClientArea(IntPtr hwnd, MARGINS& pMarInset)
at System.Windows.Shell.WindowChromeWorker._ExtendGlassFrame()
at System.Windows.Shell.WindowChromeWorker._UpdateFrameState(Boolean force)
at System.Windows.Shell.WindowChromeWorker._HandleDwmCompositionChanged(WM uMsg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Shell.WindowChromeWorker._WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Interop.HwndSource.PublicHooksFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
✔ Expected Behavior
Chrome should work as it is.
❌ Actual Behavior
Windows is crashed.
Other Software
2024-01-30.txt

/dup #30507


Add to hosted workshop capability to create resources in project.

This replicates functionality from learning portal and user workspace configurations.
Need to fix up labels on hosted workshop to match convention on others.

Decided not to do this. Better to use user-workspaces config for this.


chore: update translations for de locale

Summary
This updates translations for the de locale
Testing
👀

Hi @Criptic!
You've previously been responsible for updating de translations.
Would you mind updating the translations in this pull request by either:

Adding suggestions for each translation in this pull request. click here for more information on adding suggestions.
Updating the attached CSV and attaching it to the pull request

de-update-dec2023.csv

Hi @scottdover, here is the translated csv file
de-update-dec2023.csv

Thank you, @Criptic :)


lineColor prop

I added a prop for lineColor, and used CSS variables to update the class.
closes #66

Codecov Report

Merging #92 (5854503) into master (9524971) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master       #92   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           14        16    +2
  Branches         5         5
=========================================
+ Hits            14        16    +2

Impacted Files
Coverage Δ

src/VerticalTimeline.js
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

why isn't this merged?

why isn't this merged?

+1 I'd like to change dynamically the line color to align with the dark/light theme and using only static CSS without access to the elements does not allow me to do this :/

+1 This would be really good, please merge

@stephane-monnot is this something whic is going to implemented?


[Feature request] toast message

I guess I could make a notification and use it as a toast message but ideal would be to have it already positioned in lower right and above all other elements.
https://docs.microsoft.com/en-us/windows/uwp/design/shell/tiles-and-notifications/adaptive-interactive-toasts
Possibilty to set position would be ideal but not necessary in the first run.
The cool spinner components you made could be placed inside such a message and positioned in center to be used as a "saving to unit, waiting for response" information.
Just ideas I'm trowing out here.

We don't have any js support in Bulma (so in this theme), but I think some basics can be implemented using just the css.

Just the css is enough for me.
Den fre 25 maj 2018 08:43Muhammad Ubaid Raza<EMAIL_ADDRESS>skrev:

We don't have any js support in Bulma (so in this theme), but I think some
basics can be implemented using just the css.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mubaidr/bulma-fluent/issues/28#issuecomment-391957648,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AZeBYptrNhespuDd_ygmOdjuOnkS8cpjks5t16f4gaJpZM4UNU_W
.

https://github.com/mubaidr/bulma-fluent/commit/e9b4d9a8b39b402d9110a0d83407c26717481689
Please check the notifications section in demo for information.

Ah, great!

Moved to https://github.com/mubaidr/bulma-addons


⚠ sandbox-api.piste.gouv.fr healthcheck has degraded performance

In d06ac6d, sandbox-api.piste.gouv.fr healthcheck (https://sandbox-api.piste.gouv.fr/cassation/judilibre/v1.0/healthcheck) experienced degraded performance:

HTTP code: 200
Response time: 2645 ms

Resolved: sandbox-api.piste.gouv.fr healthcheck performance has improved in 15c6c2f.


Move repository to rust-lang

Hello, the current rustasync organisation is considered deprecated, and the Rust Programming Language organisation is trying to consolidate managing Rust's GitHub organisations, as such we'd like you to consider moving your repository to the main rust-lang organisation instead.
How Do I Move To rust-lang?
You won't be able to transfer your repository directly, you'll need to transfer your repository to @Mark-Simulacrum, who will then move your repository to the rust-lang organisation. They will also be able to assist with joining the organization if you aren't a member already.
cc/ @ibaryshnikov

@ibaryshnikov I'm unfamiliar with most of the teams on the rust-lang repo, but I would assume it should be possible to create a new one (possibly under "website") and add @ibaryshnikov?

A thing to note here is that the website seems to be hosted from gh-pages. This means that if we initiate the transfer, the actual site might go down as the url no longer resolves. To that extent we should probably coordinate when to transfer this.
The steps are:

I transfer this repo to myself
I initiate the transfer request to @Mark-Simulacrum
The request is accepted, and @Mark-Simulacrum now moves the repo to rust-lang

The only question is how to update our DNS records at this point. @gnunicorn I believe you still administrate these? An option would be to transfer these to the rust-lang infrastructure (@alexcrichton is the point person for DNS specifically here), or else just it'd be much appreciated if the record could be updated.
Ideally we could do this next week Tuesday, which is before the async/await release which might seen an increase in traffic. I've proposed moving "areweasyncyet" then too.
It'd be much appreciated if folks could share how they feel about this!

I think one simplification would be that if you make me an admin on this repository I can likely transfer it directly, which could help with DNS. If we keep hosting this site on GitHub pages though I wouldn't expect DNS to be affected (though I could be wrong :)

I wouldn't expect DNS to be affected (though I could be wrong :)

Hah, no I think you're right and I was thinking about this the wrong way. Oops, bit tired on this Saturday haha.
I'll make grant you admin rights to both repos, and let you move from there!

Looks like I have the necessary permissions, thanks! I will proceed with the move on Tuesday to give time for people to give feedback and also to try and ensure we have folks on hand to fix things if this does cause breakage.

Note that it's unfortunately not trivial to transfer *.rs domains, so we'd ideally not tweak the DNS registration, but if still desired send me an email and we can sort out the details.

Okay moved repository over and invited @ibaryshnikov; closing this issue. Thanks!


NATS logging driver support

Rancher version:
1.4
Docker already supporting NATS logging driver (https://docs.docker.com/engine/admin/logging/overview/), but I do not see it in Rancher interface.

docker 1.13 has NATS support, but in docker 17 NATS logging driver removed without any comments
https://github.com/docker/docker.github.io/pull/2240

The NATS driver was never integrated into Docker and should not be in
the docs.

I guess we're going with not adding it then if it doesn't exist...


Create a CD

We should do the same as in https://github.com/massalabs/massa-as-sdk/tree/main/.github/workflows

Actually, there is already a CD workflow that publish as, as-tester and transformer. We could improve it with a script that pushes a commit to bump the new version.
Moreover, it is currently not possible to publish only on package of the three. @gregLibert do you think it's a concern?

We now have a CD:

https://github.com/massalabs/as/pull/22/files#diff-f824c6e5a2b0ac16ea55eb73c74d51966cc1c3d6e7ad71d6e4c0b13f0a13346c

This PR will improve monorepo CD: https://github.com/massalabs/as/pull/58

@longgamma-massa for QA, we will see if it work when we will create the release :) but we are confident because it is the same way we know it's work for other packages

@longgamma-massa CD works as expected https://www.npmjs.com/search?q=%40massalabs


fix(openfga): allow service and deployment annotations

Description
Specify .annotations and .service.annotations in the chart, that allow adding additional annotations to the deployment and to the service resources, respectively.
References

#35
Review Checklist

[ ] I have clicked on "allow edits by maintainers".
[x] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[x] The correct base branch is being used, if not main
[x] I have added tests to validate that the change in functionality is working as expected

@PedroArvela for documentation purposes, please add the new annotations to the values.schema.json file as well.

Added to the json schema!


fix core llp tlv test for Arista

Include InterfaceEnabled deviation.
Configure ATE before DUT to get Neighbour data on DUT faster.

Pull Request Functional Test Report for #1856 / 531f4da9938e68f48caac93dd6dbf955f86c2c65
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

RT-6.1: Core LLDP TLV Population

Cisco 8000E

RT-6.1: Core LLDP TLV Population

Cisco XRd

RT-6.1: Core LLDP TLV Population

Juniper cPTX

RT-6.1: Core LLDP TLV Population

Nokia SR Linux

RT-6.1: Core LLDP TLV Population

Openconfig Lemming

RT-6.1: Core LLDP TLV Population

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 47.655%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1341

Relevant Lines:
2814

💛  - Coveralls


Add error code when throwing UserStoreException

Purpose
Add error code when throwing UserStoreException when updating the claims of a user who are in a read-only userstore to capture the error scenario for the fix: wso2-support/carbon-identity-framework#1810
Resolves: wso2/product-is#12382
When should this PR be merged
This PR should merge before https://github.com/wso2/carbon-identity-framework/pull/3704

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/1211878729

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/1211878729
Status: success


Can't set image background (while can in cygwin)

I can't set image background in  minttyrc file, but I can do this in cygwin.

It works if you provide the right path.

Please be more specific.


Fix: ESlint

Remove unused imports in the create-remix package

This is already fixed in #611

Thanks for your work here, the code has changed a lot since this and I think it addresses the issues.


Set the static IP for nodes

Hi Team,
Some customers are request dedicate static IP for their nodes managed by RackHD. I have tried to do that by adding host section in dhcpd configuration file. By doing that, the nodes can get static IP as expected. However, the boot sequence became failed after the change. That is to say, using the dynamic IP, nodes can boot to platypus via RackHD, while using static IP(configured in dhcpd.conf), nodes failed to boot platypus via RackHD.
The configuration I added for static IP:
             ```subnet <IP_ADDRESS> netmask <IP_ADDRESS> {
                   range <IP_ADDRESS> <IP_ADDRESS>;
                   # Use this option to signal to the PXE client that we are doing proxy DHCP
                   option vendor-class-identifier "PXEClient";
                   option routers <IP_ADDRESS>;
                   #host ob1a {
                   #  hardware ethernet 00:60:16:6F:9E:34;
                   #  fixed-address <IP_ADDRESS>;
                   #}
                   }```

            Can you help me understand how the static IP configuration impacted the workflow?

@yyl8815  - we need a little more information to assist you. When you say the boot sequence failed, can you share what worked and what didn’t?
Presumably you initiated a PXE boot sequence - did the node get the the IP address and request/download iPXE from RackHD?
There after it will immediately request an iPXE script (a profile) typically associated with your workflow. Did it get the iPXE script (also known as profile)?
Are there any errors in the on-taskgraph or on-http logs while doing this process that indicate a failure to render a template - or something else with an extended traceback?
-joe