We have completed the physical website.

physical web is not enough. We have to further complete physical website architecture, for to consider and use.References are as follows:
https://play.google.com/store/apps/details?id=com.adist.drivethru&hl=zh-TW
http://youtu.be/C-cLDUhitAc
So, you can better physical web concept.

Ok, please tell us what you suggest. what improvements are you proposing?

google is a good company, but in this concept at least 2 years behind us.
Because it is innovation and invention, so there are some new word must be explained:
physical website / a site is placed on the location. without the Internet, only when necessary, and then connect IoT exchange of information.
WiFi LBS physical website / geographically through WiFi way to provide all the known sites of Web services, without the Internet.
physical website URL / use the local language name of the store or service through WiFi broadcasts.
physical website browser / only a App, you can see all the physical website, or wearable equipment operators and physical website specifications.
Cloud application site is now mainstream, very suitable for people in the home or business, do some virtual trading or centralized management areas, conditions for large-scale systems and the high rate of flow, and I want to try to solve the real trading, so use integrated application solutions O2O do the actual situation. In fact, to solve real application, not only the connection or not, there is an important issue is the operating cost.
Application entity is when we are not home or business, but out in the real world environment in a region or businesses, the number of traffic usage is low, can only operate with a low cost way. Today all the web servers are concentrated in cloud, is the concept of concentration. in order to solve the physical environment of the application, we will move directly to a web server each entity store business is decentralized concept application in the physical world does not need a large centralized server , but scattered small server to each store.
Using solid way to achieve offline web server can also connect the O2O, but should be offline to online, in order to really solve the problem of substance use e-commerce.
Alibaba e-commerce bottleneck logistics, physical stores over the counter, using the system after phone / tablet, you can store in the physical electronic orders, O2O (offline to online) will become the bottleneck in the kitchen, you and Like Alibaba, delivery speed problems in the background, so the physical store efficiency in the use of the system after Alibaba achieve equivalent results, it can save manpower reception, kitchen manpower increase and accelerate the meal rate / rate increase turn tables.
The same Apple is a good company, but ibeacon not a good way to solve the problem of the physical environment.
My English is not good, hope you can understand my consciousness.
From: scottjenson<EMAIL_ADDRESS>Sent: Friday, May 08, 2015 11:22 AM
To: google/physical-web
Cc: cgj531
Subject: Re: [physical-web] We have completed the physical website. (#370)
Ok, please tell us what you suggest. what improvements are you proposing?
—
Reply to this email directly or view it on GitHub https://github.com/google/physical-web/issues/370#issuecomment-100087047 .  https://github.com/notifications/beacon/ALvn5ZZE0CorAreclY1eN4MtqcYARn9aks5oHCM9gaJpZM4ETOHA.gif

The most important is a physical web server, the production cost of less than US $ 100, and no other fixed expenses. Entity store can use very low-cost and simple system to join the business. In the prior art methods are way is to use the web server operation cost at least US $ 10000 or more. use the cloud network to pay a very high cost and maintenance, and also has a high risk.
Our technology is the Wi-Fi base station and sites combined and placed in the actual location, to reach the site information service, location and integrated WiFi connection, and make special arrangements designed on SSID, become a new and innovative service system and the cost is only one-tenth or less of the old methods. the product service systems can help a physical store, use WiFi LBS concept of network technology, do e-commerce and marketing advertising (POP mode), but without the Internet and network use of fixed costs, and maintenance and operation is very simple. easily could have its own website and e-marketing advertising trading capabilities, increase revenue / reduce manpower and provide better service.
From: scottjenson<EMAIL_ADDRESS>Sent: Friday, May 08, 2015 11:22 AM
To: google/physical-web
Cc: cgj531
Subject: Re: [physical-web] We have completed the physical website. (#370)
Ok, please tell us what you suggest. what improvements are you proposing?
—
Reply to this email directly or view it on GitHub https://github.com/google/physical-web/issues/370#issuecomment-100087047 .  https://github.com/notifications/beacon/ALvn5ZZE0CorAreclY1eN4MtqcYARn9aks5oHCM9gaJpZM4ETOHA.gif

Sounds like you are creating something very exciting, but you've misunderstood the concept of what we mean by "The Physical Web".
From our main readme:

"Our core premise is that you should be able to walk up to any “smart” physical object (e.g. a vending machine, a poster, a toy, a bus stop, a rental car) and interact with it without first downloading an app."

and

"At its base, the Physical Web is a discovery service: a smart object broadcasts relevant URLs that any nearby device can receive. This simple capability can unlock exciting new ways to interact with the Web."

By definition, we explicitly want to interact with the web.  From there, its up to the web app developer to create experiences.
Perhaps you can adapt your payment processing system for this use case.. but if you think you think you have created something better.. truly best of luck to you!

There must be physical website.
From: Michal Mocny<EMAIL_ADDRESS>Sent: Friday, May 08, 2015 8:06 PM
To: google/physical-web
Cc: cgj531
Subject: Re: [physical-web] We have completed the physical website. (#370)
Sounds like you are creating something very exciting, but you've misunderstood the concept of what we mean by "The Physical Web".
From our main readme:
"Our core premise is that you should be able to walk up to any “smart” physical object (e.g. a vending machine, a poster, a toy, a bus stop, a rental car) and interact with it without first downloading an app."
and
"At its base, the Physical Web is a discovery service: a smart object broadcasts relevant URLs that any nearby device can receive. This simple capability can unlock exciting new ways to interact with the Web."
By definition, we explicitly want to interact with the web. From there, its up to the web app developer to create experiences.
Perhaps you can adapt your payment processing system for this use case.. but if you think you think you have created something better.. truly best of luck to you!
—
Reply to this email directly or view it on GitHub https://github.com/google/physical-web/issues/370#issuecomment-100211616 .  https://github.com/notifications/beacon/ALvn5Vy1xSg9GyA5SXDsMMzgm4f0Yg6Uks5oHJ4QgaJpZM4ETOHA.gif

Glad I took the time to answer nicely.

(I've now blocked this user.)  I believe this should prevent his ability to interact with these repos.


feat: Support dedicated CPU placement

Codecov Report

Merging #19 (e1d27a2) into main (00cb1dc) will increase coverage by 3.38%.
The diff coverage is 72.30%.

@@            Coverage Diff             @@
##             main      #19      +/-   ##
==========================================
+ Coverage   26.79%   30.17%   +3.38%
==========================================
  Files           2        3       +1
  Lines         780      845      +65
==========================================
+ Hits          209      255      +46
- Misses        526      545      +19
  Partials       45       45

Impacted Files
Coverage Δ

pkg/cpuset/cpuset.go
72.30% <72.30%> (ø)

pkg/controller/vm_controller.go
24.33% <0.00%> (-0.27%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us.


About dataset part and trainer part of fairseq2

Until now, I don't seem to have found any details about the specific example of dataset organizing and training from scratch , Would fairseq2  intend to provide a separate trainer library and dataset library like lightning and accelerate ,datasets from hugging face.? I think that fairseq2 mainly focuses on model details like transformers library from hugging face now.
I'd like to decouple training and modeling so that it's more conducive to updating the training part quickly, one of the main pain points of self-supervised large models at the moment is that training is time-consuming and slow。
While, I hope that fairseq2 has a good recipe organized for example, maybe like speech open source projects (i.e. kaldi, espnet,wenet and so on), to solve fairseq example is unfriendly and some details missing for new users.

Have the same issue as above ^
Are there any examples or documentation regarding how to train from scratch based on a pre-determined architecture?
(Related: https://github.com/facebookresearch/fairseq2/issues/305)

@shanguanma @sidharthrajaram
Please take a look at our tutorial in the nightly documentation. We will move them to the stable version soon. Thanks!


Duplicated symbols with Whole Module Optimization

Imported SwiftBond 4.0.1 to my project by adding .swift files but get "3 duplicated symbols" linker error between "UITextView+Bond.o" and "EventProducer.o". Fixed it by removing "UITextView+Bond.swift" from the project.

It turns out that the problem occurs when Whole Module Optimization turned on.

Have you tried cleaning you project? Sounds like an issue with Xcode...

Sure. I'm on Xcode 7.0, OS X El Cap.

Turned WMO on again and now it found duplicate with FontAwesome..
Output:
duplicate symbol __TTSg5O10AppName10ModelState___TFC10AppName13EventProducerP33_D6B553B696813232F95C039DE50201B312dispatchNexturfGS0_q__Fq_T_ in:     /Users/user/Library/Developer/Xcode/DerivedData/AppName-fpxzofbepdulhpdcemeqatauwvqq/Build/Intermediates/AppName.build/Debug-iphoneos/AppName.build/Objects-normal/armv7/FontAwesome.o     /Users/user/Library/Developer/Xcode/DerivedData/AppName-fpxzofbepdulhpdcemeqatauwvqq/Build/Intermediates/AppName.build/Debug-iphoneos/AppName.build/Objects-normal/armv7/EventProducer.o duplicate symbol __TTSg5SS___TFC10AppName13EventProducerP33_D6B553B696813232F95C039DE50201B312dispatchNexturfGS0_q__Fq_T_ in:     /Users/user/Library/Developer/Xcode/DerivedData/AppName-fpxzofbepdulhpdcemeqatauwvqq/Build/Intermediates/AppName.build/Debug-iphoneos/AppName.build/Objects-normal/armv7/FontAwesome.o     /Users/user/Library/Developer/Xcode/DerivedData/AppName-fpxzofbepdulhpdcemeqatauwvqq/Build/Intermediates/AppName.build/Debug-iphoneos/AppName.build/Objects-normal/armv7/EventProducer.o ld: 2 duplicate symbols for architecture armv7 clang: error: linker command failed with exit code 1 (use -v to see invocation)

Swift compiler is full of issues...


Minor performance improvements to MarshalBlittable

This PR includes a couple of performance improvements to MarshalBlittable<T>:

Use sizeof(T) instead of Marshal.SizeOf<T>
On .NET 6+, replace a temporary pinned GCHandle with MemoryMarshal.GetArrayDataReference

Is there a difference between Buffer.MemoryCopy and array.AsSpan().CopyTo(new Span<byte>(data.ToPointer(), byte_length));?


Request For Comment: Emoji

Vim doesn't handle emoji correctly. Most of terminal emulators including xterm have been drawn emoji with half width character in default. So many multi-byte users troubled. We can't see emoji on vim correctly.

As you know emoji was defined as ambiguous width on unicode.org. (fix1: in latest, it's narrow, but may be fixed in PRI318, fix2: if PRI318 will be accepted, all of emoji will be wide) So vim doesn't handle emoji as double width. https://github.com/vim/vim/blob/master/src/mbyte.c#L1267 .
However, most of ASCII users may not know, unicode.org recomend emoji as square aspect ratio.
http://unicode.org/reports/tr51/

Current practice is for emoji to have a square aspect ratio, deriving from their origin in Japanese. For interoperability, it is recommended that this practice be continued with current and future emoji.

iTerm2 provide option to switch narrow and wide.
Preferences -> Advanced -> Unicode East Asian Ambiguous character are wide

And macvim render emoji as wide. https://github.com/macvim-dev/macvim/pull/187
So I hope that vim also can handle emoji as wide possibly. I don't have stuck about implementation of handling emoji widths to treat as double. We are okay if vim provide way to render emoji as double width. Even though if it is new option 'emoji' or something with false in default for doing that. For example,
Suggestion1: load EastAsianWidth.txt
set<EMAIL_ADDRESS>
user can modify data.
Suggestion2: new value for ambiwidth
Currently, it take two values single or double. Then new value 'fulldouble' for fix this issue.
Suggestion3: new option 'emoji'
https://gist.github.com/mattn/00dc87d2e8cf1dd3131d
This implementation refered unicode data from 1.0 to 6.0. This patch handle emoji as wide.

AFAIK, most of fonts expect the square aspect ratio for rendering of emoji as wide. For example, smily face ( :smile: :sushi: or etc ). At least, I don't want to eat sushi rendered as narrow width.

Would be nice this feature. Actually I have so much trouble with this too:

The better option is the 3 I think. Others chars worked fine?

Adelar da Silva Queiróz wrote:

Would be nice this feature. Actually I have so much trouble with this too:

The better option is the 3 I think. Others chars worked fine?

To get this moving forward we need to convince others that these Emoji
characters are to be considered full width.  I asked Thomas Dickey.
He suggested we should also get wcwidth() to behave this way.  I hope
someone can look into that.
--
Computers make very fast, very accurate, mistakes.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

FYI, Proposed Update UAX #11, East Asian Width (PRI318) notes:

Review Note: In terms of Unicode Version 8.0, and although not yet reflected in the EastAsianWidth.txt data file, this will result in the following 774 characters being changed from Neutral to Wide: 231A..231B, 23E9..23EC, 23F0, 23F3, 25FD..25FE, 2648..2653, 267F, 2693, 26A1, 26AA..26AB, 26BD, 26CE, 2705, 270A..270B, 2728, 274C, 274E, 2753..2755, 2795..2797, 27B0, 27BF, 2B1B..2B1C, 2B50, 1F004, 1F0CF, 1F300..1F320, 1F32D..1F335, 1F337..1F37C, 1F37E..1F393, 1F3A0..1F3CA, 1F3CF..1F3D3, 1F3E0..1F3F0, 1F3F4, 1F3F8..1F43E, 1F440, 1F442..1F4FC, 1F4FF..1F53D, 1F54B..1F54E, 1F550..1F567, 1F595..1F596, 1F5FB..1F64F, 1F680..1F6C5, 1F6CC, 1F6D0, 1F6EB..1F6EC, 1F910..1F918, 1F980..1F984, and 1F9C0; and the following 25 characters being changed from Ambiguous to Wide: 2614..2615, 26BE, 26C4..26C5, 26D4, 26EA, 26F2..26F3, 26F5, 26FA, 26FD, 2757, 2B55, 1F18E, and 1F191..1F19A.

And:

Emoji style standardized variation sequences behave as though they were East Asian Wide, regardless of their assigned East_Asian_Width property value.

Thanks!
https://github.com/vim/vim/commit/3848e00e0177abdb31bc600234967863ec487233

The vertical movement is showing a strange behavior.
With DirectX enabled:

Without DirectX:

Others operations are working fine.
Vim 7.4.1627
SO: Windows 10

It seems that width of cursor is correct, But seems that the width of cells are bit storange to me.

This happen with all "no default" chars. In this case the chars are taken from vim-webdevicons plugin:

With ambiwidth double and single the behaviour is the same.

I think that this issue can be closed and open another to solve the rendering problem, since the cause isn't exactly related with the last changes.


Graphics.generateCanvasTexture() Bug

There's a bug when converting a graphics object to a texture with generateCanvasTexture.
Here's a Pen that demos the bug: CodePen
The triangle displayed on the right side shows the Graphics object.
The triangle displayed on the left side shows the generated texture.
The corners of the texture's triangle are cut off, making the corners much less sharp and making the triangle a little smaller.

P.S. - We're using the latest release of Pixi (4.4.1).
And if anyone has a workaround we'd love to try it :)

I use plain html5 2d api for those things. Then "Texture.fromCanvas(myGeneratedCanvas)"

I think that the workaround would be either making a RenderTexture from two layered triangle fills, or using the Canvas2D api directly to generate the canvas. Since the extra space needed for sharp line joins isn't easily predicted, I imagine that a fix for this would just be givinggenerateCanvasTexture a padding parameter for cases like this.

Thanks for the quick suggestions guys 👍
We've got it working using the Canvas2D API and PIXI.Texture.fromCanvas and loving the nice crisp graphics.
One maybe-issue we're having thou is that we need to create a new canvas element for each texture, because if we clear the canvas after creating the texture and sprite, the sprite ends up being empty.
Does this sound right?
Thanks again: )

You can upload it to GPU and then re-use the canvas:
renderer.textureManager.updateTexture(myTexture)

It happens when you first time draw sprite, so if you re-used canvas before update, it will upload newer version. Do that manually and it'll work.

Thanks Ivan, this led me in the right direction for our use case (which is slightly more complicated than I let on). After some experimentation and RTFM'ing I ended up using texture.baseTexture.update() instead, which seems to more thoroughly update the texture's properties (height, width, and resolution being important in our case).
Thanks again, this solved the issue for us - and quicker than I dared hope :)

@ivanpopelyshev
Thanks for the update  about the PIXI Canvas Renderer. My current approach was crashing w/ WebGL disabled (no textureManager). I don't mind requiring WebGL, but am glad to know that I need to check for it :)

FYI, after some more tinkering with @ivanpopelyshev's suggestions I found an approach that seems to be working for both WebGL and Canvas-fallback rendering, and allows the off-screen canvas to be cleared and re-used.
  texture.baseTexture.update();     sprite.texture = texture;     renderer.render(stage);     offScreenContext.clearRect(0, 0, offScreenCanvas.width, offScreenCanvas.height);
Below incorporates both techniques, using the WebGL version when possible.
  if (renderer.textureManager) {         renderer.textureManager.updateTexture(texture.baseTexture);     }     else {         texture.baseTexture.update();         sprite.texture = texture;         renderer.render(stage);     }     offScreenContext.clearRect(0, 0, offScreenCanvas.width, offScreenCanvas.height);

Sorry my first time using the "Insert Code". It it supposed to condense like that?

Hi @ivanpopelyshev. I'm trying to re-use a canvas to create multiple textures, using the method you showed me, but it's not working after all. I made a very simple little CodePen that tries do draw a green square, update the texture, and then draw a red square, but both squares are always the same color. Can you please take a peek if you get a chance? I'm not sure what else to try..

Oh my!
"fromCanvas" caches stuff, and it won't update texture that already exists in cache.
new PIXI.Texture(new PIXI.BaseTexture(offScreenCanvas));

Awesome! Thanks @ivanpopelyshev. BTW, when I redraw a texture (on resize), if I call "texture.destroy(true)", set all my sprites' texture property to the new texture and don't keep any extra references to the object in my code, that should take care of freeing up the old texture's resources, right?


Fix compiler warnings in unit tests.

This commit fixes compiler warnings of the type [-Werror=range-loop-construct]
Here is one example:
test/common/router/config_impl_test.cc:2207:27: error: loop variable 'header' of type 'const string&' {aka 'const std::__cxx11::basic_string<char>&'} binds to a temporary constructed from type 'const char* const'

@hq6 could you fix DCO please

@hq6 could you fix DCO please

Done.


Some advice for detecting song lyrics

Thanks for all the work you guys are doing here -- this is a fantastic project!
My usecase is pretty simple and I am unsure how to apply WhisperKit to it so I hope to get some advice from the experts here.
I am writing an app that will show song lyrics and "flip the page" when you are done with a sentence or two in order to show the next page of lyrics.
What I'm trying to do is have WhisperKit detect where in the lyrics the person is, and flip the page on its own without user interaction. What would be the best way to achieve this?
I understand that WhisperKit can detect all speech, but I want to help it by showing it the text it is meant to match the speech to.
Thanks in advance.

Thanks for the note! This sounds like a very interesting use case. What is the format of your song+lyric data? My understanding is that most songs have timestamped lyric data so I am keen to understand the challenge. WhisperKit can certainly help timestamp each word in the lyrics to your audio if the original data doesn't have this metadata.

The challenge here I think, is that these songs are not the kind you'd find on Spotify. They are old songs and they can be sung in several different ways too, so I don't really have original audios of how they're supposed to sound..
From some initial testing, I can see that WhisperKit is very good at detecting words being sung -- but it can produce results that are incorrect, hence my question whether I can restrict the words it should be listening to based on the lyrics I have.
Thank you for your help @atiorh!

Understood! In that case, you can implement a custom SuppressTokensFilter which was added in #14 that should take the list of tokens to suppress before each transcribe call.

Thank you @atiorh I will give that a try.


"The element 'createTransactionRequest' in namespace 'AnetApi/xml/v1/schema/AnetApiSchema.xsd' has invalid child element 'refId' in namespace

Getting error response while making payment , how can I solve this?Any help please
Request :
[
"createTransactionRequest":
["refId": "123456",
"transactionRequest":
["payment": ["opaqueData": ["dataDescriptor": "COMMON.ACCEPT.INAPP.PAYMENT", "dataValue": "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx="]],
"shipTo": ["state": "TX", "city": "Pecan Springs", "zip": "44628", "address": "12 Main Street", "lastName": "Bayles", "firstName": "China", "company": "Thyme for Tea", "country": "USA"], "userFields": ["userField": [["name": "MerchantDefinedFieldName1", "value": "MerchantDefinedFieldValue1"], ["name": "favorite_color", "value": "blue"]]], "poNumber": "456654", "billTo": ["state": "TX", "city": "Pecan Springs", "zip": "44628", "address": "14 Main Street", "lastName": "Johnson", "firstName": "Ellen", "company": "Souveniropolis", "country": "USA"], "transactionType": "authCaptureTransaction", "lineItems": ["lineItem": ["itemId": "1", "name": "vase", "description": "Cannes logo", "quantity": "18", "unitPrice": "45.00"]], "amount": "1", "customerIP": "<IP_ADDRESS>"],
"merchantAuthentication":
["name": "xxxxxxxxxxx", "transactionKey": "xxxxxxxxxxxxx"]
]
]
Response:
{
messages = {
message = (
{
code = E00003;
text = "The element 'createTransactionRequest' in namespace 'AnetApi/xml/v1/schema/AnetApiSchema.xsd' has invalid child element 'refId' in namespace 'AnetApi/xml/v1/schema/AnetApiSchema.xsd'. List of possible elements expected: 'merchantAuthentication' in namespace 'AnetApi/xml/v1/schema/AnetApiSchema.xsd'.";
}
);
resultCode = Error;
};
}

Hi @Junaidtt, Have you got any success implementing this? I'm having the same issue.

Yes, Firstly referring  Codable will give you more idea.
struct Transaction: Codable {
let createTransactionRequest: CreateTransactionRequest
}
struct CreateTransactionRequest: Codable {
let merchantAuthentication: MerchantAuthentication
let refId: String
let transactionRequest: TransactionRequest
}
struct MerchantAuthentication: Codable {
let name: String
let transactionKey: String
}
struct TransactionRequest: Codable {
let transactionType: String
let amount: String
let payment: Payment
}
struct Payment: Codable {
let opaqueData: OpaqueData
}
struct OpaqueData: Codable {
let dataDescriptor: String
let dataValue: String
}
You will be more clear on this Stackoverflow question.

Thanks for replying. Actually I've already  done the same but getting following json object:
["createTransactionRequest": {
merchantAuthentication = {
name = "";
transactionKey = "";
};
refId = "123456";
transactionRequest = {
amount = 20;
payment = {
opaqueData = {
dataDescriptor = "COMMON.ACCEPT.INAPP.PAYMENT";
dataValue = "";
};
};
transactionType = authCaptureTransaction;
};
}]
Here I'm getting  transactionType in wrong place(I think its alphabetical in order). Can you please help me fix this issue.

I solved a similar problem of mine by using the exact ordering of keys as present in the documentation.

I solved a similar problem of mine by using the exact ordering of keys as present in the documentation.

yes this works, thanks!


Added option to have a separate parse format and date format

I added the option to provide a parse format that is different from the date format. This is useful when passing in a date in a different format from the one that you want to display. For example, this allows the user to pass in a date string in ISO format but display it in MM/DD/YYYY HH:mm z format. I have added an example (ng2-utils-7) to your test app. Without parseFormat, opening the datepicker shows an incorrect date; with parse-format, the datepicker shows the correct date, and after selecting a new date the input is formatted according to the given dateFormat.

I am hesitant to accept parseFormat. Instead you can pass a Date as your input,  so that you don't need to worry about parse-format. As you know already, ngModel value doesn't have to be string.

Thanks for your response.
If I get a date string like
dateWithTimeStr = '2017-01-15T14:22:00-06:00';
and convert it to a Date
dateWithTime = moment(this.dateWithTimeStr, 'YYYY-MM-DDTHH:mm:ssZ').toDate();
logging the value shows Sun Jan 15 2017 15:22:00 GMT-0500 (Eastern Standard Time)
but when I open the datepicker it shows today's date, because in the component init the call to .getTime() returns NaN for some reason, so it defaults to today.
Also, I would like to avoid converting to a Javascript Date object because I don't want to always show the local timezone.

Thanks for the explanation. Could you update README.md before I accept this PR?

Absolutely, should be all set. Let me know if there's anything else. Thanks.


SyntaxError with Addon Using included Hook

Hello,
My team is using your package in the component files of an addon like so:
/addon/components/some-component/component.js
import Ember from 'ember';
import hbs from 'htmlbars-inline-precompile';

export default Ember.Component.extend({
  layout: hbs`IMPORTANT STUFF {{yield}}`
});

This works great until I attempt to add an included hook to the addon index.js file:
/* jshint node: true */
'use strict';

module.exports = {
  name: 'neat-addon',

  included(app) {
    this._super.included(app);
  }
};

As soon as I add that included hook I get this error attempting to run the addon or use it inside of an Ember app:
File: inline-precompile/components/test-component/component.js
The Broccoli Plugin: [Babel] failed with:
SyntaxError: inline-precompile/components/test-component/component.js: Line 9: hbs should be invoked with a single argument: the template string

export default Ember.Component.extend({
  layout: hbs(_templateObject)
});

Running:
ember-cli: 2.9.1
node: 6.9.1
os: darwin x64

with:
"dependencies": {
  "ember-cli-babel": "^5.1.10",
  "ember-cli-htmlbars": "^1.0.10",
  "ember-cli-htmlbars-inline-precompile": "^0.3.6"
},

Any ideas why using the included hook in the addon would cause this error?
Thanks for the awesome tool!

You are not invoking super quite right, though I'm not certain if that is the fundamental issue.
It should be:
this._super.included.apply(this, arguments);

YASSSSSS. Thank you @rwjblue !!!
So is this always the correct way to invoke super for this hook? In the ember-cli guide at Developing Addons and Blueprints  there are examples with:

no _super
this._super.included(app);
-return this._super.included(app); (for an in-repo addon)

I'd be happy to submit a PR updating the docs for ember-cli if this is always the correct syntax: this._super.included.apply(this, arguments);
Thanks again for the SUPER fast response.

Yes, those examples are wrong. I'd love a PR fixing.

Closing since this seems resolved now


Fix UI wrap text issue

Codecov Report

Merging #203 (b5dad20) into master (89ce5d3) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #203   +/-   ##
=========================================
  Coverage     55.38%   55.38%
  Complexity      785      785
=========================================
  Files           166      166
  Lines          3284     3284
  Branches        381      381
=========================================
  Hits           1819     1819
  Misses         1363     1363
  Partials        102      102

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 89ce5d3...b5dad20. Read the comment docs.

Nice! now the text will be fully dispalyed.


qgis-ltr: 3.34.5 -> 3.34.6

Description of changes
Update QGIS LTR to the latest patch version.
List of changes: https://github.com/qgis/QGIS/compare/final-3_34_5^...final-3_34_6

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 305331 run on x86_64-linux 1

1 package built:

qgis-ltr


http://framework7.ios/ :)

What does it mean?

It means that you have wrong urls in the description ;) Take a look at:
`The premium and free iOS-icons font for Framework7.
The font is developed to be used with Framework7, but you can use it wherever you see it fits, personal or commercial. It is free to use and licensed under MIT.`


Wrong answer of sum of A.P. when last term is given

When we are calculating the sum of AP by knowing the last term result is coming wrong.
I would like to correct this issue. Please assign this to me.

@RaveenaBhasin You can go ahead with this :)
Deadline 27th March 11:59 PM IST

I have made the PR kindly review it


Granularity in compare and equal?

Steps to reproduce
Timex.equal? ~D[2017-07-27], ~D[2017-07-28], :months
# => true
Timex.equal? ~D[2017-07-27], ~D[2017-08-01], :months
# => true
Timex.equal? ~D[2017-07-27], ~D[2017-08-26], :months
# => true
Timex.equal? ~D[2017-07-27], ~D[2017-08-27], :months
# => false

Description of issue
I only expected the first statement above to return true, however the 2nd/3rd also returned true. My assumption was the granularity above would be used to truncate the two dates and see if they were in the same calendar month. But instead, it's being treated as  "is it within a month?"
Is this by design? The documentation doesn't appear to specify exactly how the granularity is used, and other date/time libraries I've used (e.g. Moment.js's isSame() on the frontend or using date_trunc in Postgres) would count all three August comparisons above as false

Yes this is intentional - comparisons are performed based on the difference between the two values, and granularity affects the interpretation of the difference. Put more simply, if the granularity unit, e.g. :months is larger than the difference between the two values, then they are effectively equal for that comparison.


customized HTML_BLOCK_TAGS Parser option seems not taken into account

Hi,
I tried to add a custom block tag, to allow code insertion in a custom webcomponent. Here is my test.md file:
# test

<warp10-warpscript-widget>
<%
  <'
  This **code** is not markdown
  $list <% %> FOREACH

//There should not be no <p> here

%>
</warp10-warpscript-widget>

**markdown again**

I use these options :

    ArrayList<String> userTags = new ArrayList<>(Parser.HTML_BLOCK_TAGS.getFrom(null));
    userTags.add("warp10-warpscript-widget");

    DataHolder options = new MutableDataSet()
      .set(HtmlRenderer.INDENT_SIZE, 2)
      .set(Parser.HTML_BLOCK_DEEP_PARSER,true)
      .set(Parser.HTML_BLOCK_DEEP_PARSE_BLANK_LINE_INTERRUPTS,false)
      .set(Parser.HTML_BLOCK_DEEP_PARSE_FIRST_OPEN_TAG_ON_ONE_LINE,true)
      .set(Parser.HTML_BLOCK_DEEP_PARSE_BLANK_LINE_INTERRUPTS_PARTIAL_TAG,false)
      .set(Parser.HTML_BLOCK_TAGS,userTags)  //TODO : does not work
      .set(TablesExtension.COLUMN_SPANS, false)
      .set(TablesExtension.APPEND_MISSING_COLUMNS, true)
      .set(TablesExtension.DISCARD_EXTRA_COLUMNS, true)
      .set(TablesExtension.HEADER_SEPARATOR_COLUMN_MATCH, true)
      .set(TablesExtension.CLASS_NAME, "table table-striped table-sm")
      .set(WikiLinkExtension.IMAGE_LINKS, true)
      .set(Parser.EXTENSIONS,
        Arrays.asList(
          TablesExtension.create(),
          AttributesExtension.create(),
          TocExtension.create(),
          AutolinkExtension.create(),
          AsideExtension.create()
        )
      );

Whese these options, if I include my code in , it works. But custom HTML_BLOCK_TAGS has no effect.
<div>
<warp10-warpscript-widget>

//There should not be no <p> here
**no md**

%>
</warp10-warpscript-widget>
</div>

I used 0.32.18 for my tests.
I can't find out what I did wrong... In my tests, It seems I need the html deep parser to get rid of the <p> in the output.

@pi-r-p, you need html deep parser because CommonMark standard ends an HTML block on a blank line. Deep html parsing closes the HTML block on a matching closing tag.

OK, that's close to my guess.
And up to code, deep parser does not take into the customs HTML_BLOCK_TAGS.
Is it easy for you to add HTML_BLOCK_TAGS as an input of the html deep parser ? I may be able to do it myself and make a PR, but it will took me a lot of time to dive in your project.
If you do not want to add this feature, just close this issue !
If you are open to the idea but no time to do it, would you accept a PR ?

@pi-r-p, PR's are always welcome.
I am adding this little bit and making a release since I forgot to pass html block tags to deep parser constructor.
BTW, the block tags should contain all default tags since it is a complete set not only additional tags. So you need to create a new set with default tags new HashSet<String>(Parser.HTML_BLOCK_TAGS.getFrom(null)) then add your tags.
You only need to add your tags to block tags if you allow starting an HTML block with your tag since in your example you start with div which creates an HTML block.

@pi-r-p, repo updated, maven update in progress

@vsch  Crazy good reactivity !
OK, I will test in a few hours.

It's now working perfectly with the same source code as in my first post.
Many thanks for your reactivity !


How I can disabled the contextmenu event of ngx-contextmenu item?

When I right click the ngx-contextmenu item,browser default context menu shows up.
How can I disable this ?

There's no good way to do this right now.  The closest you can get is something like this:
<context-menu>
  <ng-template contextMenuitem>
    <div (contextmenu)="$event.stopPropagation();$event.preventDefault()">Something</div>
  </ng-template>
</context-menu>

This will stop the context menu on the context menu item itself, but not on the padding around it.

How can I open the context menu on right click  as well as on left click ?


Can't resolve paths from index.html after npm run build

After building an app into /dist folder index.html references all resources (minified .css, .js and images) as /resource_name instead of ./resources_name. Manually replacing / to ./ in webpack.config.prod.js:output.publicPath or in index.html fixes the problem.
Am I doing something wrong that I have to do it manually?
P.S. The issue does not affect the dist server.
Node version: 6.9.2
npm version: 3.10.9
Operating system: macOS 10.12.3
Command line used: npm run build (excluded linting and testing)
Steps to reproduce: npm run build

You should change the public path in webpack.config.prod.js: https://github.com/coryhouse/react-slingshot/blob/master/webpack.config.prod.js#L23

Could you post more of a snippet of the code? It is hard to tell if that is intended or not.

@kwelch sure!
Here's index.html generated by build:
<!DOCTYPE html><link href="https://fonts.googleapis.com/css?family=Roboto" rel="stylesheet"><meta http-equiv="X-UA-Compatible" content="IE=edge"><meta charset="utf-8"/><title>Twitch viewer</title><link href="/main.bacb06aca645ee7750eca079e5cdd5a2.css" rel="stylesheet">

href for the CSS bundle and src for the js bundle start with / which results in 404 when I run it "in production" or on the local server.
If the publicPath is changed from / to ./ (as suggested by @mnoleto and mentioned in the first post) it works properly.

I am glad to hear that, that worked for you. It does seem odd as both slingshot and my configuration I am unable to replicate what you are saying.
Even further, it is odd that only the css would 404 but the main.js would render properly.
Is this happening on your machine with a clean clone of slingshot?

@kwelch
I probably didn't explain myself properly, sorry for that. All of the resources ref'd from index.html give 404, including main.js, main.css, and a png image that is referenced from main.js bundle. I've deleted some changes from the original clone, but I cannot think of anything that could affect the path resolution. I'll do more testing on a completely clean clone and post it here.
It was pretty straightforward to fix and thank you guys for helping. However, I wasn't able to find anything in FAQ so I decided to bring it up here.

I would appreciate a PR to update the docs. I am glad it is working. Do you feel this is a direct bug in slingshot? If not is this be available to be closed.

@kwelch
I'll try to do some more testing over the weekend to make sure that it's not my fault. Hope it can wait a couple of days.

No rush at all. I only ask to help ensure we are able to help the next user that comes across this.
Thank you for you time.

@kwelch
I figured out what my problem was. When serving dist folder to either GitHub Pages or on a local server (I'm using the server provided with WebStorm) I had no control over the location of the served files. E.g. on Github /repo-name/ was appended and I missed this point. As the result, all minified files were requested from /main.js(.css)' in the index.htmlinstead of/repo-name/main.js`.
Is there a good reason to use the absolute path (/) in this case? I think that a relative path (./ or '') would be more logical since the webpack is configured to bundle everything to one folder anyways. Am I missing something?
P.S. if you find it useful I'd like to make a PR. It'd be my first commit to an open source project ever ;)

I defer to @coryhouse. I think there is value in having a how to deploy section to the docs.

I agree that a deploy section would be useful, and likely expose us to opportunities for improvement. I'd welcome a related PR.

I created a short note in FAQ on this topic.


gitter room needs repo integration

This is for @kevin1024 who is the only one with permissions.
How about adding repo integration to https://gitter.im/kevin1024/vcrpy ?  Click the tools button at the top right of the chat window, go through the oauth sequence, etc.  The default list of notifications is fine.
I had temporarily added vcrpy-unittest notifications to the room but they don't call out the repo name in the notification so I removed them to avoid confusion.

Thanks @kevin1024 !


Passing Twist message from Ros2/foxy to Teensy3.2 using micro-ros

I am running Ros2/foxy under Ubuntu 20.04 and microros for Arduino on a teensy 3.2.
I seem to be able to pass simple messages like int32 or float32 across the interface, but I have tried several more complex messages like geometry_msgs::msg::Twist and I can't get it to function.  It hangs up every time I try to pass the message.  I have looked at the ROS2 created message using ros2 topic echo, and the message looks fine.  But it hangs up the teensy processor on the other end.  I have attached the code.  I use the same code for transferring a single float value and for passing the Twist message, except that I change the message type from std_msgs::msg:Float32 to geometry_msgs::msg::Twist every place it is needed.  The following is the code.  I would appreciate any help I can get resolving this.  iT hits the callback function, turn the LED off, and then hangs up.
#include <micro_ros_arduino.h>

#include <stdio.h>
#include <rcl/rcl.h>
#include <rcl/error_handling.h>
#include <rclc/rclc.h>
#include <rclc/executor.h>
#include <LiquidCrystal.h>
#include <geometry_msgs/msg/twist.h>

rcl_subscription_t sub_cmd;
geometry_msgs__msg__Twist* msg_cmd;

rclc_executor_t executor;
rclc_support_t support;
rcl_allocator_t allocator;
rcl_node_t node;
rcl_timer_t timer;
double cmd[4];
int num;
#define LED_PIN 13

#define RCCHECK(fn) { rcl_ret_t temp_rc = fn; if((temp_rc != RCL_RET_OK)){error_loop();}}
#define RCSOFTCHECK(fn) { rcl_ret_t temp_rc = fn; if((temp_rc != RCL_RET_OK)){}}

  const int rs = 19, en = 18, d4 = 17, d5 = 16, d6 = 15, d7 = 14;
  LiquidCrystal lcd(rs, en, d4, d5, d6, d7);

void error_loop(){
  while(1){
    digitalWrite(LED_PIN, !digitalRead(LED_PIN));
    delay(100);
  }
}

void subscription_callback(const void * msgin)
{
  digitalWrite(LED_PIN, LOW);

  const geometry_msgs__msg__Twist * msg_cmd = (const geometry_msgs__msg__Twist *) msgin;

  cmd[0]=msg_cmd->angular.x;
  cmd[1]=msg_cmd->angular.y;
  cmd[2]=msg_cmd->angular.z;
  cmd[3]=msg_cmd->linear.x;

  lcd.setCursor(0,3);
  lcd.print(cmd[0];
}

void setup() {
  set_microros_transports();

  lcd.begin(20,4);
  lcd.clear();
  lcd.setCursor(0,0);
  lcd.print("Setup");
  pinMode(LED_PIN, OUTPUT);
  digitalWrite(LED_PIN, HIGH);

  msg_cmd=geometry_msgs__msg__Twist__create();

  delay(2000);

  allocator = rcl_get_default_allocator();
  lcd.setCursor(0,1);
  lcd.print("1");

  //create init_options
  RCCHECK(rclc_support_init(&support, 0, NULL, &allocator));
  lcd.print("2");

  // create node
  RCCHECK(rclc_node_init_default(&node, "Arm_node", "", &support));
  lcd.print("3");

  // create subscriber
  RCCHECK(rclc_subscription_init_default(
    &sub_cmd,
    &node,
    ROSIDL_GET_MSG_TYPE_SUPPORT(geometry_msgs,msg,Twist),
    "Arm_cmd"));
  lcd.print("4");

  // create executor
  RCCHECK(rclc_executor_init(&executor, &support.context, 1, &allocator));
  RCCHECK(rclc_executor_add_subscription(&executor, &sub_cmd, &msg_cmd, &subscription_callback, ON_NEW_DATA));
  lcd.print("5");
}

void loop() {
  delay(100);
  lcd.setCursor(10,1);
  lcd.print(++num);
  RCCHECK(rclc_executor_spin_some(&executor, RCL_MS_TO_NS(100)));
  lcd.setCursor(10,2);
  lcd.print(num);
  lcd.setCursor(0,3);
  lcd.print(cmd[1]);

}

Issue template

Hardware description:
RTOS:
Installation type:
Version or commit hash:

Steps to reproduce the issue

Expected behavior
Actual behavior
Additional information

You should not be using geometry_msgs__msg__Twist__create() since it uses dynamic memory. In micro-ROS types should be handled with care, check this tutorial for more information: https://micro.ros.org/docs/tutorials/advanced/handling_type_memory/

Pablo, Thanks, but that didn't make any difference.  Actually, I am thankful it didn't work, or I would have worried a lot about the micro-ros code.  I did find the problem though.    I guess it is true what they say.  If you have enough monkeys with enough keyboards, they will eventually reproduce the works of Shakespeare.  The error was here.
rclc_executor_add_subscription(&executor, &sub_cmd, &msg_cmd, &subscription_callback, ON_NEW_DATA)
In the code, msg_cmd was declared as a pointer, and so the line of code above was passing the address of the pointer rather than the pointer.  I removed the & in front of msg_cmd and it works fine.
rclc_executor_add_subscription(&executor, &sub_cmd, msg_cmd, &subscription_callback, ON_NEW_DATA)
That' what I get for using exmple code and not paying enough attention to it.
Thanks again

On 11/22/2021 10:31 PM Pablo Garrido<EMAIL_ADDRESS>wrote:

You should not be using geometry_msgs__msg__Twist__create() since it uses dynamic memory. In micro-ROS types should be handled with care, check this tutorial for more information: https://micro.ros.org/docs/tutorials/advanced/handling_type_memory/

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/micro-ROS/micro_ros_arduino/issues/584#issuecomment-976198766 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AWT3SINBBLA67Q2S254GKJTUNMYLXANCNFSM5ISLJFEA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .


UnsatisfiedLinkError

Hello,
While trying out your example app I have ran into a problem. On Android i get the UnsatisfiedLinkError
whole stacktrace:
com.cloudwebrtc.webrtcexample E/org.webrtc.Logging: NativeLibrary: Failed to load native library: jingle_peerconnection_so
    NativeLibrary: java.lang.UnsatisfiedLinkError: dalvik.system.PathClassLoader[DexPathList[[zip file "/system/framework/org.apache.http.legacy.boot.jar", zip file "/data/app/com.cloudwebrtc.webrtcexample-zASNzno6_zbJwbicSb0Dpg==/base.apk"],nativeLibraryDirectories=[/data/app/com.cloudwebrtc.webrtcexample-zASNzno6_zbJwbicSb0Dpg==/lib/x86_64, /system/fake-libs64, /data/app/com.cloudwebrtc.webrtcexample-zASNzno6_zbJwbicSb0Dpg==/base.apk!/lib/x86_64, /system/lib64]]] couldn't find "libjingle_peerconnection_so.so"

The app works great on iOS, but I cant get around this on Android. Every suggestion would help.

you need import android/libs/*.jar files

import exactly how? it is in the build.gradle of webrtc module if that is what you mean

@SnowcatSVK
From the log analysis, does not support the x86_64 architecture so, you are using the ARM architecture?
If not, you can apply doc/patchs/m63_patch_for_flutter_webrtc_plugin_v0.patch to recompile webrtc M63 to support x86_64 architecture.

@cloudwebrtc first i tried it on x86_64 emulator and when i saw the logs i have tried it on pixel 2, where i get the same error, only there it said it couldnt find armeabi_v8a, so then i guess i would have to recompile it for both x86_64 and armeabi_v8a?

@SnowcatSVK Yes, you can try to recompile it for both x86_64 and armeabi_v8a(arm64).
Normally, if you run the Android native app, armv7 library can run in the arm64 platform, and x86 library can run in x86_64 architecture. As far as I know flutter is still unable to run in x86 or x86_64 even arm64 platform, you need to exclude the errors caused by it.

ok then, i will try to recompile it and see if that helps, thank you for your time

Could some notes about this eventually be added to the readme ? Ir even better a make file to see how to do this sort of ISA architecture shuffling ?
Just because its quite complex to know what to do on android to get it working in Sim and on real device.

Ok, I will update README.md and add x86_64 and arm64 architecture to the next commit.

@SnowcatSVK We have recompiled this native sdk, support armv7, arm64, x86, x86_64 on android, you can view the latest commit.

@cloudwebrtc Just tested it out and everything works rather nicely, thank you very much for your help


Sprite/ColorRect: using "onready var" reference won't visually update the rect_size/scale during runtime

Godot version: 3.2.3 stable
OS/device including version: Win64
Issue description:
I have a node 2D with a ColorRect as child and want to change the rect_size in code:
this works
extends Node2D

func _process(delta):
	if Input.is_action_just_pressed("ui_accept"):
		$ColorRect.rect_size.x += 10
		print($ColorRect.rect_size.x)

this does not work
extends Node2D

onready var color_rect_x = $ColorRect.rect_size.x

func _process(delta):
	if Input.is_action_just_pressed("ui_accept"):
		color_rect_x += 10
		print(color_rect_x)

Both will print the rect_size changes correctly in the output panel, but in the onready var example, the visual representation of the ColorRect does not change. As expected, assigning the ColorRect to the variable in _ready() has the same effect as onready var.
Same applies to Sprite:
works
extends Node2D

func _process(delta):
	if Input.is_action_just_pressed("ui_accept"):
		$Sprite.scale.x += 0.1
		print($Sprite.scale.x)

does not work
extends Node2D

onready var sprite_scale_x = $Sprite.scale.x

func _process(delta):
	if Input.is_action_just_pressed("ui_accept"):
		sprite_scale_x += 0.1
		print(sprite_scale_x)

Unless I misunderstand something, your code examples are not equivalent at all.
In your work examples, you increment the scale on the node directly, so yeah it updates the node.
In your does not work examples, you put the scale in a member var, and increment the member var. This will obviously not  update the node. Integer variables are not references, so that's expected.

Closing per @Zylann's comment.

Thank you for the thorough explanation, @Zylann , that totally makes sense!


Can I use it with custom serviceworker events?

I'm currently sending and receiving push notification with serviceworker, while caching website too.
I'm considering migrating to this plugin if it supports adding custom serviceworker events. As far as I know, one serviceworker can be attached to one web scope, so I cannot use my own serviceworker with offline-plugin.
So can I use it with push notification events?

Yeah. use ServiceWorker.entry option. If will include a require(...) to that file in generated SW.

@martinop just provide the correct path.
ServiceWorker: {
        entry: path.resolve(__dirname, 'push.js'),
        ...
}

thanks! @dsaket


compressing each file has a bug

When i have many folders and i want to compress each folder into a zip, the below issue happens.
Screencast from 27-09-24 03:17:21 PM IST.webm
The issue is that anything after . is cut.
See that "mistral7B_0", the decimal and after got deleted.
please fix this.

I have updated the script to ensure that file suffixes are only removed for files, not directories.
Thanks for bringing this up! 😉

Yep now it works just fine! Thanks so much!
Do check just in case if any other pre-made actions of yours might face this.
Thanks!


[mxlinux] Drop identifiers

Looks like it was wrong, see #5356.

Update proposed in #5356, merging.


pkg: devicemapper: fix typo in function declaration

6990b76a696dd265674f4c2973f25755a6485f05 introduced a typo in function
declaration, this patch fixes that.
Actually @shishir-a412ed had a build error while building master with this typo while I hadn't (we're both on fedora 22 but he ran ./hack/make.sh dynbinary and me just make)
Reproduced error is (on fedora 22 with AUTO_GOPATH=1 ./hack/make.sh dynbinary):
 ---> Making bundle: dynbinary (in bundles/1.9.0-dev/dynbinary)
Created binary: bundles/1.9.0-dev/dynbinary/dockerinit-1.9.0-dev
Building: bundles/1.9.0-dev/dynbinary/docker-1.9.0-dev
# github.com/docker/docker/pkg/devicemapper
.gopath/src/github.com/docker/docker/pkg/devicemapper/devmapper_wrapper_deferred_remove.go:14: undefined: CDmTask
.gopath/src/github.com/docker/docker/pkg/devicemapper/devmapper_wrapper_deferred_remove.go:18: undefined: CDmTask

Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

ping @vdemeester also

@runcom waw.. how did I miss that :sweat: :sob:

/cc @vbatts to merge :)

Do we need a separate Janky to catch these on future?

@thaJeztah devicemapper build on Jenkins, not shown here with other builds,  was failing

LGTM

@runcom thanks, all good then :-)


Added 2 Jupyter Notebooks to Examples to perform 2D and 3D shepp-loga…

Added 2 Jupyter Notebooks to Examples to perform 2D and 3D Shepp-Logan phantom generation, followed by projection and reconstructions.
This will add 2 more examples to the Documentation based on demo_2D_shepp_logan.py and demo_3D_shepp_logan.py

Wenrui, I'm not sure how to access the Jupyter Notebooks script.
I deleted the demo_simple.py. I want to base our examples on the 3D microscopy and SL demos.

I put all Jupyter Notebooks scripts to ./svmbir/docs/source/ipynb. You can
use pycharm to open them.
[image: Access Jupyter Notebook.png]
Charles A Bouman<EMAIL_ADDRESS>于2020年11月17日周二 下午3:40写道：

Wenrui, I'm not sure how to access the Jupyter Notebooks script.
I deleted the demo_simple.py. I want to base our examples on the 3D
microscopy and SL demos.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cabouman/svmbir/pull/91#issuecomment-729196400, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJDA4ULVDC5CQM5TTYZRD3SQLNTHANCNFSM4TZBZKRQ
.


Use Emacs 29.1 for testing

Using snapshot was causing issues with package-lint not being able to find the respective compat lib.

We could start using snapshot once https://github.com/purcell/package-lint/pull/262 gets merged.


Update package "DataMercury" and change the name to "MercurySM" and some things

Fix open json empty files.
Fix undo, after saveIn in the same file.
Some optimizations

LGTM


Added Web3 Content

Update broken link from Docker Section.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Put lists in alphabetical order, correct spacing.
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

@davorpa check the updated web3 content.
Looking forward to hearing from you.

also, section need to be in same order as index

Hi @davorpa ,
Thank you for the suggestion.
Check out the recent updates!!


[Feature Request] Add way to change the prefix for each icon

Describe the feature and why you wanted
Hi there, in Font Awesome 5 have different prefix for the icon, would you mind adding feature that can add prefix to each icon
For example:
We will use icon: "fab-accusoft" instead of:
icon: "accusoft"
    iconPrefix: "fab-",

Excuse me,  I read the doocs you provided and what I saw is that the front awesome 5 needs fa- prefix.
I am providing an option which can add this prefix on all of your icon.
If you are suggesting to change the default value, I am afraid I will not accept it.
The reason I am using icon- for default is that most of my theme users including myself are chinese, and we use iconfont.
If you have other problems, welcome to decribe   more specificly.

Sorry, Mr.Hope. This is quite misleading. I don't mean to change the default value
Let's take a look at the picture below:

As you see, each icon have different prefix: fas,far,fal,fad,fab which mean that you have to set different prefix to every individual icon instead of add this prefix on all icon

Got it, currently I am on a trip, I will try to provide better support after I return home.

I was thinking to make a better enhancement, but I am not catching up some great ideas. If you are using only one style of icon in your site, I think it's possible for you to set iconPrefix to fas fa- or far fa- and so on. Then it will be easy for you to set the prefix. If you are using different icons accross the site, I can not provide with some options for you to group those icons with same style.
The only thing I can enhance is that:

Providing three optional options: navbarIconPrefix sidebarIconPrefix pageIconPrefix.
Let all the icon key support both string and string[]

I will consider the second one as a good enhancement, But I am not sure we really need the first one.

The first one is useless anyway.
The second look better than the first one
Actually their is a small workaround for this:
iconPrefix:" " //Avoid adding prefix
icon: "fas fa-camera"
icon: "far fa-camera" //Set it manually
// Don't know if adding a boolean to disable iconPrefix setting may be good as well

Hey there, below is my code:
  private get iconPrefix(): string {
    const { iconPrefix } = this.$themeConfig;

    return iconPrefix === "" ? "" : iconPrefix || "icon-";
  }

So I think you can just use iconPrefix: "" to unset the default prefix.
Besides, i am not sure if the style class must be placed before the icon type class, so,
// config
iconPrefix: "fa-"

icon: "camera fas"
icon: "camera far"

would be better.

Also, I think again for the second improvement, and I decide not to do it.
Before:
JS:
icon: "fas fa-camera"

Front matter:
icon: fas fa-camera

After:
JS:
icon: ["fas", "fa-camera"]

Front matter:
icon:
  - fas
  - fa-camera

It's making configuration more complicated.

Besides, i am not sure if the style class must be placed before the icon type class, so I think

// config
iconPrefix: "fa-"

icon: "camera fas"
icon: "camera far"

would be better.

I think so, because all fontawesome's icon have fa- and this way look better

I am going to close this issue since all the problems have been discussed.


Revert "Recaptcha integration master pr"

Reverts firebase/firebase-ios-sdk#11231

@andrewheard and I will work on the revert PR.

Replaced by #11617


Missing option to administer users

There should be a user administration menu which would allow to remove users entirely from an installation. The user administration menu could for reasons of simplicity be implemented as a special project.

@tinne
Would you like this to be implemented as an additional (small) web app that is distributed along with the main war and simply connects to the database and allows the administrator to delete any user?

A minimum complete solution could come as a separately deployed or separately accessible web app. Else, administrator could be a special role to every user (in a in-memory setting, the first user created only would automatically become administrator) and user administration a separate page in the normal application.
Ideally, an administrator could enable and disable administratorship on other users, delete users and set (overwrite) user passwords.

@tinne
I have implemented the required functionality in a separate web app.
It is possible for you to try it out and tell me if it anything should be adjusted or changed?
It can be compiled from this PR #298 or I can also send a .war file of the administration menu if that's more convenient :).

I tried the PR: it runs into an error due to an incompatible database schema - kindly provide a flyway script. See the attached schema-fix-issue in budgeteer-admin.log.

I have added a migration script that will update the database (as it is in branch release commit 73c7cc5370e7682de563d0fbac3b4bcd204438e1 on  an Oracle db) . Everything should be working now.

see my comment in https://github.com/adessoAG/budgeteer/pull/298

Creating a new test database, I now got a running user admin installation, thanks. I try the productive database administration later.
The role options is somewhat clumsy to use and not easy to understand.  Especially the drop-down menu with the checkboxes is hard to use and it takes time to understand that you cannot untick both boxes. I recommend to use two separate checkboxes without any pull-down around.
Who is allowed to log into the admin menu? Is this just the first user ever to register? This should be explained somewhere.
As concerning our other discussion, one could as well make the user name edible in the user list. But what is the use of having the role “admin only” for a user and project? It seems to behave as if the user and both roles anyway.
The default action in “Are you sure you want to delete this project?” should not be “Yes.” Nor should it be in any other these security dialogues. Also, add a “irrevocably.“
The security question for a user is ill-formulated. Instead of “Are you sure you want to remove this user from the project?” it should be “Are you sure to irrevocably remove this user from Budgeteer?”
If I fail to select a user (or there is none), the error message for a click on Add User to this Project / Add User should rather be “You need to select a user to perform this action.”
No verification mail is ever sent, nor do I find any error regarding this in the logs (using standard log settings).
What is the anticipated functionality considered to be removed from the Project Administration? Is there any?

The role options is somewhat clumsy to use and not easy to understand. Especially the drop-down menu with the checkboxes is hard to use and it takes time to understand that you cannot untick both boxes. I recommend to use two separate checkboxes without any pull-down around.

I see, I will change the drop-down to simply checkboxes.

Who is allowed to log into the admin menu? Is this just the first user ever to register? This should be explained somewhere.

The first user to log into the admin menu becomes the only one allowed to access it (untill other users are also made admins).

As concerning our other discussion, one could as well make the user name edible in the user list. But what is the use of having the role “admin only” for a user and project? It seems to behave as if the user and both roles anyway.

Yes, the project roles work a bit differently. A user can be both admin and user or only an admin, there is no difference. I have made this possible because of the way roles work in keycloak. (users can have many different roles). I also thought this design would be good if additional roles are to be added in the future (for example something between user and admin for example). But I agree that this is a bit confusing. Also, I will make the usernames editable.

The default action in “Are you sure you want to delete this project?” should not be “Yes.” Nor should it be in any other these security dialogues. Also, add a “irrevocably.“

The security question for a user is ill-formulated. Instead of “Are you sure you want to remove this user from the project?” it should be “Are you sure to irrevocably remove this user from Budgeteer?”

If I fail to select a user (or there is none), the error message for a click on Add User to this Project / Add User should rather be “You need to select a user to perform this action.”

I will adjust everything accordingly,

No verification mail is ever sent, nor do I find any error regarding this in the logs (using standard log settings).

In order for the email functionality to work, a mail server must be set up. How to do this is explained here: https://github.com/adessoAG/budgeteer/wiki/Set-up-local-mail-server-for-development-(Windows) and in a README in the project repo. However, these instructions are Windows-only and I'm guessing the Budgeteer runs on a Linux based server. I'm not sure how to handle this. (but I don't think we would want to publish any adesso specific data on github).

What is the anticipated functionality considered to be removed from the Project Administration? Is there any?

After an admin is made a user, they can no longer access the administration menu.
After a project admin is made a user, they can no longer acess the Project Administration Menu (the button in the Dashboard disappears). This was somewhat bugged before (likely when you tested it), but I have fixed it since. I really did not want to give you a buggy build to test, but unfortunately some bugs only come up some of the time or things sometimes break after making changes, without me noticing (and Unit tests don't always catch these errros).

The default action in “Are you sure you want to delete this project?” should not be “Yes.” Nor should it be in any other these security dialogues. Also, add a “irrevocably.“

The security question for a user is ill-formulated. Instead of “Are you sure you want to remove this user from the project?” it should be “Are you sure to irrevocably remove this user from Budgeteer?”

If I fail to select a user (or there is none), the error message for a click on Add User to this Project / Add User should rather be “You need to select a user to perform this action.”

The role options is somewhat clumsy to use and not easy to understand. Especially the drop-down menu with the checkboxes is hard to use and it takes time to understand that you cannot untick both boxes. I recommend to use two separate checkboxes without any pull-down around.

As concerning our other discussion, one could as well make the user name edible in the user list.

These issues have been fixed. If you want I will also replace the dropdown in the Project Administration with checkboxes.

Yes, the project roles work a bit differently. A user can be both admin and user or only an admin, there is no difference. I have made this possible because of the way roles work in keycloak. (users can have many different roles). I also thought this design would be good if additional roles are to be added in the future (for example something between user and admin for example). But I agree that this is a bit confusing. Also, I will make the usernames editable.

Having specific roles like this is very much appreciated. I only would like to make a bit more explicit:
Introduce separate roles for system admins (system wide) and project admins (per project). Make this sort of stuff edible by the admin menu. I suggest to add an action link [turn into system admin] to all users in “All users in this Budgeteer install” and a link [revoke system admin rights] to all other system admins in the list.
If project admin implies project user, ok. Just add a handler that enables the user role once the admin role is activated, and deletes the admin role once the user role is deleted. Thus people will have visual feedback on this dependency. Else, also use action links for all actions [grant admin rights], [revoke admin rights], [remove from project].
With regards to the mail setup, I suggest to first explain the general setup options and - potentially as an addendum - explain how to install a test server, as enough people got a productive mail server around they could use, and even more so in production environments. Additionally, this information should be transformed to an example application.properties and duplicated to the new client.

Introduce separate roles for system admins (system wide) and project admins (per project). Make this sort of stuff edible by the admin menu.

That is how it is currently. Project roles and System admin roles are separate. A project admin only has access to the administration for a project. System wide admins have access to the administration app as well as access to all projects. So a system admin can set permissions on users in a project and set other users as system wide admins as well.

I suggest to add an action link [turn into system admin] to all users in “All users in this Budgeteer install” and a link [revoke system admin rights] to all other system admins in the list.

I will add this.

If project admin implies project user, ok. Just add a handler that enables the user role once the admin role is activated, and deletes the admin role once the user role is deleted. Thus people will have visual feedback on this dependency. Else, also use action links for all actions [grant admin rights], [revoke admin rights], [remove from project].

Yes, I didn't think about just having both checkboxes be checked. I will do it like that.

With regards to the mail setup, I suggest to first explain the general setup options and - potentially as an addendum - explain how to install a test server, as enough people got a productive mail server around they could use, and even more so in production environments. Additionally, this information should be transformed to an example application.properties and duplicated to the new client.

I will try to write instructions for setting up a mail server in a linux environment. But I will add this in another PR. Could you please clarify about this part:

enough people got a productive mail server around they could use,

This means setting up an additional mail server isn't nessecary?

This means setting up an additional mail server isn't nessecary?

You need an SMTP server / mail transport agent, which is integrated to the global e-mail infrastructure and would accept mail via the SMTP protocol from your Budgeteer server (via the Java Mail API). An additional mail server can be used if your productive mail infrastructure would not exist / allow you to dock in. E-Mail is traditionally split into several parts and the MTA should not be mixed up with a POP3/IMAP/proprietary mail storage/access server or mail user agents (vulgo e-mail clients).

I suggest to add an action link [turn into system admin] to all users in “All users in this Budgeteer install” and a link [revoke system admin rights] to all other system admins in the list.

I have implemented this. To me it looks and feels the best so far.

If project admin implies project user, ok. Just add a handler that enables the user role once the admin role is activated, and deletes the admin role once the user role is deleted. Thus people will have visual feedback on this dependency

This has been implemented as well.
As for the emails, it seems things aren't simple as I had thought. I will look into it.


ModuleNotFoundError: No module named 'compat'

Next import error:
  File "AnimeDL.py", line 9, in <module>
    import sites
  File "C:\Users\x\Desktop\anime-dl-master\anime_dl\sites\__init__.py", line 1, in <module>
    import sites.crunchyroll
  File "C:\Users\x\Desktop\anime-dl-master\anime_dl\sites\crunchyroll.py", line 10, in <module>
    from external.utils import bytes_to_intlist, intlist_to_bytes
  File "C:\Users\x\Desktop\anime-dl-master\anime_dl\external\__init__.py", line 1, in <module>
    import external.aes
  File "C:\Users\x\Desktop\anime-dl-master\anime_dl\external\aes.py", line 5, in <module>
    from .utils import bytes_to_intlist, intlist_to_bytes
  File "C:\Users\x\Desktop\anime-dl-master\anime_dl\external\utils.py", line 37, in <module>
    from compat import (
ModuleNotFoundError: No module named 'compat'

working on my end. Just downloaded bunch of videos. Let me know if Py script still causes some funky errors. After that I'll push the windows binary. PyCharm doesn't show these errors. Gawd dammit PythonPath :/

New one now haha:
Traceback (most recent call last):
  File "AnimeDL.py", line 9, in <module>
    import sites
  File "C:\Users\Administrator\Desktop\anime-dl-master\anime_dl\sites\__init__.py", line 1, in <module>
    import sites.crunchyroll
  File "C:\Users\Administrator\Desktop\anime-dl-master\anime_dl\sites\crunchyroll.py", line 10, in <module>
    from external.utils import bytes_to_intlist, intlist_to_bytes
  File "C:\Users\Administrator\Desktop\anime-dl-master\anime_dl\external\__init__.py", line 1, in <module>
    import external.aes
  File "C:\Users\Administrator\Desktop\anime-dl-master\anime_dl\external\aes.py", line 5, in <module>
    from utils import bytes_to_intlist, intlist_to_bytes
ModuleNotFoundError: No module named 'utils'

Just pushed an update from desktop. It closed automatically. Re-opening, please try again and let me know. Should work now. Please make sure you're running on Python 3

Still get the same error after the update:

C:\Users\Administrator\Desktop\anime-dl-master\anime_dl>python --version
Python 3.6.0

Okay, please check again. PyCharm uses PYTHONPATH and the imports work just fine inside it. Outside it, THIS happens. (Learned something new). Check again and let me know...

Fixed utils back to compat however:
Traceback (most recent call last):
  File "AnimeDL.py", line 9, in <module>
    import sites
  File "C:\Users\Administrator\Desktop\anime-dl-master\anime_dl\sites\__init__.py", line 1, in <module>
    import sites.crunchyroll
  File "C:\Users\Administrator\Desktop\anime-dl-master\anime_dl\sites\crunchyroll.py", line 10, in <module>
    from external.utils import bytes_to_intlist, intlist_to_bytes
  File "C:\Users\Administrator\Desktop\anime-dl-master\anime_dl\external\__init__.py", line 1, in <module>
    import external.aes
  File "C:\Users\Administrator\Desktop\anime-dl-master\anime_dl\external\aes.py", line 5, in <module>
    from external.utils import bytes_to_intlist, intlist_to_bytes
  File "C:\Users\Administrator\Desktop\anime-dl-master\anime_dl\external\utils.py", line 37, in <module>
    from compat import (
ModuleNotFoundError: No module named 'compat'

son of a ...  :/

Oh yeah I understand, glad you're active and willing to work through these problems.

Check this update and let me know if its working or not...

No more errors but running --help/-h and I get no output, assume that's just because running it on windows.

Try downloading an episode. Don't forget to put ffmpeg in the directory of this script, or in the system path.

No good:
C:\Users\Administrator\Desktop\anime-dl-master\anime_dl>python AnimeDL.py -i http://www.crunchyroll.com/kemono-friends/episode-8-ppp-live-728273 -u username -p password

C:\Users\Administrator\Desktop\anime-dl-master\anime_dl>

This :
python __main__.py -i http://www.crunchyroll.com/kemono-friends/episode-8-ppp-live-728273 -u username -p password

Alright that did it, worked successfully.

NICE! Finally! And thank you so much for pointing this out and helping me fix this shiznit :)


The repository with name 'pytorch-training' does not exist

When running the
%%bash
./build_and_push.sh sagemaker-d2-train-sku110k latest Dockerfile.sku110ktraining

cell there is a docker error
repository<PHONE_NUMBER>77.dkr.ecr.us-east-1.amazonaws.com/pytorch-training not found: name unknown: The repository with name 'pytorch-training' does not exist in the registry with id '663822598777'

I think you might be able to solve this by updating the DockerFile.sku100ktraining
ARG REGION
FROM<PHONE_NUMBER>84.dkr.ecr.us-east-1.amazonaws.com/pytorch-training:1.6.0-gpu-py36-cu101-ubuntu16.04

Please check again the file Dockerfile.sku110ktraining, there is no reference to the call that you mentioned  and the id of the ECR registry is<PHONE_NUMBER>84.


Display elapsed time at end of scan

Description
We already measure scan duration in telemetry, but we don't display it to the end-user. As a user, I find it useful to know how much time it took for the tool to perform an analysis of my entire infrastructure. This tiny UX improvement would also highlight performance optimizations such as cache for resources enumeration (#421).
Example

it works well! can you explicit the wording for the user, like "Scan took 9s" (I find "Done in" too vague for someone not aware of the intention here). Thanks!

or better @sundowndev your wording on the HTML output is better: "Scan Duration: 8s"

LGTM!
[...]
Found 13 resource(s)
 - 76% coverage
 - 10 covered by IaC
 - 1 not covered by IaC
 - 2 missing on cloud provider
 - 0/10 changed outside of IaC
Scan duration: 9s


[Windows] WebGL support added

Summary
WebGL support added. WebView can show now WebGL content.
Read more from https://docs.microsoft.com/en-us/microsoft-edge/webvr/webvr-in-webview
See the updated example app:

Compatibility
Windows
React Native Windows 0.62.x
Checklist

[x] I have tested this on a device and a simulator

Fixes issue #1621

Test release build fails in command npx react-native run-windows --root example --arch x64 --release --no-packager --no-deploy --logging
i think that there might be missing attribute --use-react-native-windows ?

@jamonholmgren is WebGL support needed? I need it for my app...

@Titozzz  is WebGL support needed? I need it for my app...

I'll forward the issues to the windows team

I'll forward the issues to the windows team

Thanks!  My another PR is also very important, could you forward this also https://github.com/react-native-webview/react-native-webview/pull/1617

@NickGerleman  and @asklar this PR is ready from my side now. I will start to fix my another PR about messaging now...

I will fix the conficts soon

@Titozzz @NickGerleman  @asklar  this PR is also ready for merge. I just fixed conficts with master (my messaging impl)

Git hub action bot did close my issue for this PR. Does this mean that i have to remove this PR?

Git hub action bot did close my issue for this PR. Does this mean that i have to remove this PR?

I think @jamonholmgren can resurrect it if needed. looks like the tests were failing?

Hi guys, any plan to merge this? The failed Windows CI seems to be not related to this. It seems to be failing for every Pull requests.  Maybe should not block this pull requests.
Thanks for the great work.

Hi guys, any plan to merge this? The failed Windows CI seems to be not related to this. It seems to be failing for every Pull requests.  Maybe should not block this pull requests.
Thanks for the great work.

@asklar  Do i have any change to get this PR merged?

If i have any change i may once again get pull from master and test before merge


Do you face similar issue of App Store review rejection for privacy info.plist strings?

Hi @ivanvorobei , I'm the owner and creator of PermissionsSwiftUI, a permission request and display library large inspired by your SPPermissions. Despite the different codebase and rewrite in SwiftUI, I believe this can be a common issue that both our libraries share.
Recent, a developer submitted a bug report to my library stating that:
TL;DR
Due to App Store's review guidelines, any user data API referenced in code, even if not used must require an info.plist string. As a result, a permission request library would contain API references to all the different permissions, but most users only need some of the permissions. It will be redundant and inconvenient to add every single info.plist string.

I only use Location, notification permissions from this swift package. Yet I am asked to include all of these different type of usage description strings. I believe this is because this swift package uses all of the permission types in PermissionsManager. Is there a way for me to not include all usage descriptions?

ITMS-90683: Missing Purpose String in Info.plist - Your app's code references one or more APIs that access sensitive user data. The app's Info.plist file should contain a NSContactsUsageDescription key with a user-facing purpose string explaining clearly and completely why your app needs the data. Starting Spring 2019, all apps submitted to the App Store that access user data are required to include a purpose string. If you're using external libraries or SDKs, they may reference APIs that require a purpose string. While your app might not use these APIs, a purpose string is still required. You can contact the developer of the library or SDK and request they release a version of their code that doesn't contain the APIs.

In your README.md, you say that "If you use xliff localization export, keys will be create automatically." And you also provided all the privacy string keys. Does your library's users face similar issue of their apps using SPPermission being rejected from the App Store? If not, do you mind explaining in further detail how did you tackle this issue? How to automatically generate info.plist keys? Thank you very much for taking the time to read this, and I greatly appreciate any help or idea.

Good day!
I have same issue maybe 3 years ago, because library shouldn't import all modules. You can check my Podspec. When user install my library, will import only usage code (for location example) and don't imports other modules.

Thank you very much, you are such a kind and helpful person. I will probably implement a similar solution in my library.


Auutomation -> providers should use  ExternalAutomationManager not the AnsibleTower provider.

Changes needed in both core and ui repos.
@miq-bot assign @kavyanekkalapu
@miq-bot assign @agrare

core pr -> https://github.com/ManageIQ/manageiq/pull/21740


[Event Request] codeunit 7312 "Create Pick" - OnAfterCalcInventoryAvailableQty

Please add a new event OnAfterCalcInventoryAvailableQty in procedure CalcAvailableQty
        // For locations with pick/ship and without directed put-away and pick
        GetItem(ItemNo);
        AvailableQtyBase := WhseAvailMgt.CalcInvtAvailQty(Item, Location, VariantCode, TempWhseActivLine);

        OnAfterCalcInventoryAvailableQty(Item, Location, VariantCode, SourceType, SourceSubType, SourceNo, SourceLineNo, SourceSubLineNo, AvailableQtyBase); // <-- NEW EVENT

        if (CreatePickParameters."Whse. Document" = CreatePickParameters."Whse. Document"::Shipment) and WhseShptLine."Assemble to Order" then
            WhseSource2 := CreatePickParameters."Whse. Document"::Assembly
        else
            WhseSource2 := CreatePickParameters."Whse. Document";

    [IntegrationEvent(false, false)]
    local procedure OnAfterCalcInventoryAvailableQty(Item: Record Item; Location: Record Location; VariantCode: Code[10]; SourceType: Integer; SourceSubType: Option; SourceNo: Code[20]; SourceLineNo: Integer; SourceSubLineNo: Integer; var AvailableQtyBase: Decimal)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Produced/Consumed tags

Am I correct in assuming that Produced/Consumed tags is not available?

I'm not sure I about  your request, but produced/consumed is a viable option when you have to exchange  data between  PLCs.
If needed you can use pycomm to read/write multiple PLCs at the same time, and that include the tags shared within the produced/consumed configuration


fix latex math on MuJoCo Documentetion

Description
Fixes https://github.com/Farama-Foundation/Gymnasium/issues/820
vim macro used:
  c  "q   xi{math} `^[nr`<80><fd>an

Type of change

[X] Documentation only change (no code changed)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Screenshots
Please attach before and after screenshots of the change if applicable.

Checklist:

[X] I have run the pre-commit checks with pre-commit run --all-files (see CONTRIBUTING.md instructions to set it up)
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

fixed without this PR see:
https://github.com/Farama-Foundation/Gymnasium/issues/820


List reviewer statistics of group's private repository

Hi :)
Thanks for your neat tool!
I am unsure if this is actually an issue of this tool / the github API or the configuration on my end.
I am unable to list the reviewer statistics of a private repository of a group I am a member of.
I've got access to the repo (push / pull / view etc), and I am trying to access it with this tool via a private access token (which grants me access to my private repos).
Have you got any experience with this? I have limited access over the group's setting though (so it might be the case that access via a meber's access token is somehow disabled if that is even possible).
Best regards
Julius
edit:
It turns out that I don't have access to my own private repos either.  Did I misconfigure my access token?

Ok this was plain stupidity from my end :D I forgot to check the correct box ... :D Thanks anyway


Make motion blur + screen space reflections use prepass

That’s awesome, no excuse for me to implement full metallic/roughness workflow for SSR!!

playground I used for tests : http://localhost:1338/Playground/index-local.html#PIZ1GK#169


miri: improve current situation

In https://github.com/rust-osdev/multiboot2/pull/128, I added basic miri support. Due to the following issues, miri can't embrace all it's funcitionality:

Each test that uses the cast_tag (transitively) fails, as miri can't guarantee that the memory is valid, unfortunately
when something like struct_as_bytes() ... cast is done, there are alignment issues as the returned Vec is of course only aligned to a one byte boundary... so far, in tests this never was a problem, luckily. At least on x86.

I'm 99.9% sure that the current crate is memory-safe. Unit tests, integration tests, at least a partial miri coverage. However, having miri on your side is a benefit, of course.

I found a minimal reproducer for the miri issue:
use std::mem::size_of;
use std::ops::Deref;
use ptr_meta::Pointee;

#[repr(C, align(8))]
struct Align8<T>(T);

impl<T> Deref for Align8<T> {
    type Target = T;

    fn deref(&self) -> &Self::Target {
        &self.0
    }
}

#[derive(Debug, PartialEq, Eq)]
#[repr(u32)]
enum TagType {
    Foo = 1,
    Dst = 2,
}

#[repr(C)]
struct BaseTag {
    typ: TagType,
    size: u32,
}

// Tag type: 0x1
#[repr(C)]
struct FooTag {
    base: BaseTag,
    val: u64,
}

// Tag type: 0x2
#[derive(ptr_meta::Pointee)]
#[repr(C)]
struct DstTag {
    base: BaseTag,
    // embedded string slice
    name: [u8],
}

impl DstTag {
    fn name(&self) -> &str {
        core::str::from_utf8(&self.name).unwrap()
    }
}

fn cast_tag<'a, T: ?Sized + Pointee<Metadata = usize> + 'a>(tag: &'a BaseTag) -> &'a T {
    let ptr = core::ptr::addr_of!(*tag);
    unsafe { &*ptr_meta::from_raw_parts(ptr.cast(), tag.size as usize - size_of::<BaseTag>()) }
}

mod tests {
    use super::*;

    #[test]
    fn test_parse_tag() {
        let bytes = Align8([
            // <begin of tag foo>
            TagType::Foo as u32 as u8, // tag foo: type
            0x0,
            0x0,
            0x0,
            16, // tag foo: size
            0x0,
            0x0,
            0x0,
            0x37, // tag foo: val
            0x13,
            0x0,
            0x0,
            0x0,
            0x0,
            0x0,
            0x0,
            // <end of tag foo>
            // <begin of tag dst>
            TagType::Dst as u32 as u8, // tag dst: type
            0x0,
            0x0,
            0x0,
            16, // tag dst: size
            0x0,
            0x0,
            0x0,
            b'h',
            b'e',
            b'l',
            b'l',
            b'o',
            b' ',
            b'!',
            b'\0', // tag foo: name
            // <end of tag dst>
        ]);

        // Parsing of Tag Foo
        let base_tag_ptr = bytes.as_ptr().cast::<BaseTag>();
        let base_tag_ref = unsafe { &*base_tag_ptr };
        assert_eq!(base_tag_ref.size, 16);
        assert_eq!(base_tag_ref.typ, TagType::Foo);
        let tag_foo = unsafe { &*base_tag_ptr.cast::<FooTag>() };
        assert_eq!(tag_foo.val, 0x1337);

        // Parsing of Tag Dst
        let base_tag_ptr = unsafe {
            bytes
                .as_ptr()
                .add(tag_foo.base.size as usize)
                .cast::<BaseTag>()
        };
        let base_tag_ref = unsafe { &*base_tag_ptr };
        assert_eq!(base_tag_ref.typ, TagType::Dst);
        assert_eq!(base_tag_ref.size, 16);
        let tag_dst = cast_tag::<DstTag>(base_tag_ref);
        assert_eq!(tag_dst.name(), "hello !\0");
    }
}

This patch solves the miri issues:
0a1
> use ptr_meta::Pointee;
3d3
< use ptr_meta::Pointee;
51,53c51,60
< fn cast_tag<'a, T: ?Sized + Pointee<Metadata = usize> + 'a>(tag: &'a BaseTag) -> &'a T {
<     let ptr = core::ptr::addr_of!(*tag);
<     unsafe { &*ptr_meta::from_raw_parts(ptr.cast(), tag.size as usize - size_of::<BaseTag>()) }
---
> fn cast_tag<'a, T: ?Sized + Pointee<Metadata = usize> + 'a>(
>     backing_mem: &'a [u8],
> ) -> &'a T {
>     let base_tag = unsafe { backing_mem.as_ptr().cast::<BaseTag>().as_ref() }.unwrap();
>     unsafe {
>         &*ptr_meta::from_raw_parts(
>             backing_mem.as_ptr().cast(),
>             base_tag.size as usize - size_of::<BaseTag>(),
>         )
>     }
97c104
<             // <end of tag dst>
---
>                    // <end of tag dst>
118c125,127
<         let tag_dst = cast_tag::<DstTag>(base_tag_ref);
---
>         let offset = base_tag_ptr.cast::<u8>() as usize - bytes.as_ptr() as usize;
>         let bytes_slices = &bytes[offset..];
>         let tag_dst = cast_tag::<DstTag>(bytes_slices);

A lively discussion on Reddit pointed out that my assumption was wrong. My assumption was that one can access memory outside the scope of a function if the caller level ensures that memory is valid. However, rustc only looks at functions and not the whole call tree to improve performance during compilation.
Here are the options I have:

go with approach B (see patch above)
make cast_tag a macro
Split the type Tag into CommonHeader and GenericTag, where the latter is a DST with a [u8] field after the common header. This ensures that every cast only operates on memory that the function also has access to by its inputs.


Remove .dependabot.yml file

I don't think this is particularly helpful or useful, especially since the majority of these PRs sit unmerged. Since we have Dependabot security notifications turned on, we'll still be notified of upgrades necessary for security reasons.

Agreed. 👍

Thanks for the review!


systemd-run guide does not work for LKVM stage 1

rkt hangs with systemd-run using the KVM stage 1 without a TTY.
When used with a TTY to run interactively, rkt behaves normally.
Environment
rkt Version: 1.6.0
appc Version: 0.8.1
Go Version: go1.6.1
Go OS/Arch: linux/amd64
Features: -TPM
--
Linux 4.5.2-coreos x86_64
--
NAME=CoreOS
ID=coreos
VERSION=1032.1.0
VERSION_ID=1032.1.0
BUILD_ID=2016-05-05-1941
PRETTY_NAME="CoreOS 1032.1.0 (MoreOS)"
ANSI_COLOR="1;32"
HOME_URL="https://coreos.com/"
BUG_REPORT_URL="https://github.com/coreos/bugs/issues"
--
systemd 229
+PAM +AUDIT +SELINUX +IMA -APPARMOR +SMACK -SYSVINIT +UTMP +LIBCRYPTSETUP +GCRYPT -GNUTLS -ACL +XZ -LZ4 +SECCOMP +BLKID -ELFUTILS +KMOD -IDN

What did you expect to see?
cgroups stage 1
$ sudo systemd-run --unit run-rkt-coreos-stage1 --slice=machine `pwd`/rkt-v1.6.0/rkt run --stage1-name=coreos.com/rkt/stage1-coreos:1.6.0 docker://alpine:3.3 --exec sh -- -c "echo Successful"
Running as unit run-rkt-coreos-stage1.service.
$ journalctl -o cat -u run-rkt-coreos-stage1.service
Started /home/core/rkt-v1.6.0/rkt run --stage1-name=coreos.com/rkt/stage1-coreos:1.6.0 docker://alpine:3.3 --exec sh -- -c echo Successful.
image: using image from local store for image name coreos.com/rkt/stage1-coreos:1.6.0
image: using image from local store for url docker://alpine:3.3
networking: loading networks from /etc/rkt/net.d
networking: loading network default with type ptp
[35289.382487] sh[5]: Successful

KVM stage 1 with a TTY
$ sudo systemd-run --tty --unit run-rkt-kvm-stage1-tty --slice=machine `pwd`/rkt-v1.6.0/rkt run --stage1-name=coreos.com/rkt/stage1-kvm:1.6.0 docker://alpine:3.3 --exec sh -- -c "echo Successful"
Running as unit run-rkt-kvm-stage1-tty.service.
Press ^] three times within 1s to disconnect TTY.
image: using image from local store for image name coreos.com/rkt/stage1-kvm:1.6.0
image: using image from local store for url docker://alpine:3.3
networking: loading networks from /etc/rkt/net.d
kvm: network unit created: "interface-eth0.service" in "stage1/rootfs/usr/lib/systemd/system" (iface="eth0", addr="<IP_ADDRESS>/24")
[    6.430600] sh[102]: Successful
[    7.893618] reboot: Restarting system with command '<rkt: obligatory restart>'

  # KVM session ended normally.

What did you see instead?
$ sudo systemd-run --unit run-rkt-kvm-stage1 --slice=machine `pwd`/rkt-v1.6.0/rkt run --stage1-name=coreos.com/rkt/stage1-kvm:1.6.0 docker://alpine:3.3 --exec sh -- -c "echo Successful"
Running as unit run-rkt-kvm-stage1.service.
$ journalctl -o cat -u run-rkt-kvm-stage1.serviceStarted /home/core/rkt-v1.6.0/rkt run --stage1-name=coreos.com/rkt/stage1-kvm:1.6.0 docker://alpine:3.3 --exec sh -- -c echo Successful.
image: using image from local store for image name coreos.com/rkt/stage1-kvm:1.6.0
image: using image from local store for url docker://alpine:3.3
networking: loading networks from /etc/rkt/net.d
kvm: network unit created: "interface-eth0.service" in "stage1/rootfs/usr/lib/systemd/system" (iface="eth0", addr="<IP_ADDRESS>/24")
[   61.346986] INFO: rcu_sched detected stalls on CPUs/tasks:
[   61.346986]  (detected by 0, t=60002 jiffies, g=-266, c=-267, q=0)
[   61.346986] All QSes seen, last rcu_sched kthread activity 60002<PHONE_NUMBER>-4294668425), jiffies_till_next_fqs=3, root ->qsmask 0x0
[   61.346986] rcu_sched kthread starved for 60002 jiffies! g18446744073709551350 c18446744073709551349 f0x2 s3 ->state=0x0

I have probably other version of systemd-run (v229) because on my machine this parameter have --pty name and there is no mentioned --tty parameter.

@jellonek can we block the lkvm flavour from getting to this point if tty is not available?

Yes, we should do this in init.

@jellonek See systemd/systemd#3054 for why it's -t or --tty not --pty on systemd v229. If you use --pty with v229, you get systemd-run: unrecognized option '--pty' despite what the documentation says.
Would also appreciate some doco around how to run KVM with TTY in a non-interactive fashion. The main focus of the systemd-run documentation is to run non-interactive:

To start a "daemonized" container that forks the container processes into the background

Running with -t starts in interactive mode.  If you're going to do that, you might as well just run rkt directly.

Thanks for pointing me about --tty. Now I downloaded sources and in fact I see this tty long option.
For this moment as obvious workaround I see for example using screen (as in good old days) as pty provider. KVM flavor at this point can not be started without pty (but I did not had a time to try this under qemu branch).

@jellonek what do you think we should do? document this?

at the start, someone on intel side should test this against qemu WIP branch. if this will still not work, yes, we should document this.
other thing is to block init in such situation with appropriate error message.

@jellonek which QEMU branch should I be testing, and is there any likelihood it'll merge soon?
In the meantime, can we document this given LKVM doesn't look like it's going away?

Look at https://github.com/coreos/rkt/pull/2952
@jjlakis will you document this?

I've just tested this with 6041ab657526817212fbecc839f67aa97363a988 for coreos, qemu and lkvm and all appear to be working correctly.
@tjdett - could you confirm this now works for you?

Yes, this seems to be working now.


for rails5

@misopeso Would be awesome if these were accepted, because right now this gem is causing Rails 5 upgrade to fail.


Cartoon network clone

This website is a cartoon network clone website made using html and css
Tech stack:- HTML , CSS
Deployment link :- https://charu1603.github.io/Cartoon-N/

@charu1603 are you a SWOC participant?