Update folder names for stages

Use the user friendly official stage name instead of the id name for the stage music.
This PR will convert the existing folders to the new format, and handle 3 specific cases.
This PR addresses multiple cases:

No folders have been created at all
Folders exist but they are the old format
Folders exist and some are the new format, some are the old format

I have tested all 3 cases myself, but it would be nice if someone else could test it as well before I make a new release.

Test release
NickCustomMusicMod-v1-0-3.zip


time.AddDate() can't get Feb

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

$ go version
1.11

Does this issue reproduce with the latest release?
I don't try 1.12
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\hetao\AppData\Local\go-build
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\Users\hetao\go
set GOPROXY=
set GORACE=
set GOROOT=C:\Go
set GOTMPDIR=
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\hetao\AppData\Local\Temp\go-build476396638=/tmp/go-build -gno-record-gcc-switches

What did you do?

package main

import (
	"log"
	"time"
)

func main() {
	curTime := time.Now()
	lastMonthTime := curTime.AddDate(0,-1,0)
	log.Println("curTime:",curTime)
	log.Println("lastMonthTime:",lastMonthTime)
}

2019/03/29 16:11:39 curTime: 2019-03-29 16:11:39.1469807 +0800 CST m=+0.002000201
2019/03/29 16:11:39 lastMonthTime: 2019-03-01 16:11:39.1469807 +0800 CST
What did you expect to see?
If -1 I expect to see the time is Feb
What did you see instead?

From docs:

AddDate normalizes its result in the same way that Date does, so, for example, adding one month to October 31 yields December 1, the normalized form for November 31.

So if you do "03-29 - 1 Month" then you obtain 02-29 but there is no 29th of February so it is normalized to 1st of March

Closing because this is working as documented and expected.

 Obviously just because something is documented and expected doesn't mean it's right. It makes no sense that adding a month to January results in March. Momentjs and Java don't work this way. I had to write a Date library in C a long time ago and this wouldn't fly then. I understand why the developers are sticking to this narrative but I think it's silly.

Please see the comment I just added to #31145 (https://github.com/golang/go/issues/31145#issuecomment-479067967)


Unable to install Interchange

npm install -g nodebots-interchange
Issue:
Failed at the<EMAIL_ADDRESS>install script

I think I know what this is - what version of node are you running?

@tcreuro for the moment there's a dependency problem with serial port on node 10. I'll raise a PR with avrgirl to get that resolved though so if you can switch back to node 8 (eg using nvm) then you can install fine.

OK and thank you. I will downgrade from 10.6.0 to 8.

Also note that nvm is your friend here. I would highly recommend the use of
NVM to manage different node versions rather than a single install
On Fri, Jul 6, 2018, 13:27 maurice_trebels<EMAIL_ADDRESS>wrote:

Here is an article on how to downgrade for node 10 to 8 using brew.
https://medium.com/@katopz/how-to-install-specific-nodejs-version-c6e1cec8aa11
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nodebotsau/nbdau/issues/55#issuecomment-402914833,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHRo4IkNlUne7yWBT8265k-ypv3EgZ9ks5uDtkGgaJpZM4VDtlV
.

This should be fixed now as well as a result of this and related changes
https://github.com/johnny-five-io/nodebots-interchange/commit/3cc00ddd5be9583a9992668d99dd1d1805b585f9
NPM registry has been updated with newer version also.


Added data types to project and added minimal seed. Fixes #154

Added "TorchData" w/ "Model", "Tensor" and "Scalar" subtypes... We may also want to add text, etc
Thoughts?

We should probably add blob loaders for some of the specific data types... I am thinking tensors (for uploaded images) and probably models... Thoughts?


MPR [Bug]

Describe the Bug
First, congratulations to everyone who is involved in this project and has contributed to the success of the application.
You select a series that only has 1, 2 images (topogram, scout) and accidentally press the mpr function button.
The application displays a message that it is not possible to apply the protocol. Ok, it's correct. The problem is that this function does not check if the selected series meets the mpr rules (slice number, modality, it returns an exception.
After the exception is thrown, the user cannot correctly execute the function again. When you try to restore the preview mode, it is lost. To go back to working if the user presses the key with F5.
Steps to Reproduce
To play, open the study
'Neptune". Load the topogram/scout and click on the mpr function.
At that moment, the application will create the exception and display the message on the screen.
Now load the "body 3.0" series in 1x1 format and click again on the mpr function. The 3x1 layout will open correctly. Then click again on the mpr function to restore the 1x1 layout. You can not. Note that even the size of the image is wrong. There is an old bug already reported in the 1x3 format. The images are not the correct size.
Press F5 and open the "body 3.0" series, select the mpr, click again and it restores the layout correctly.
https://github.com/OHIF/Viewers/assets/3778492/76e9deb3-98ff-4753-beb5-947b9df48bcd
The current behavior
........
The expected behavior
.........
OS
Windows 11
Node version
20.3.0
Browser
Chrome

can you try on the viewer-dev.ohif.org ?

Just have a try with viewer-dev.ohif.org, the issues still exists: without refreshing the page, there is no way to disable MPR anymore in the end by following the reproduce steps.

This is now fixed in https://github.com/OHIF/Viewers/commit/221dedde5dd4df086276406a9fa2da1cc23b4eb1


ESP32 Dev Kit C and CC1101 module connection...

Hi!
I have an ESP32 Dev Kit C (https://www.az-delivery.de/en/blogs/azdelivery-blog-fur-arduino-und-raspberry-pi/das-zehnte-tuerchen) from azdelivery , I wanted to connect it to a CC1101 module the same one used for the D1 Mini, which is mentioned.
My current pinout is:
SCK MOSI MISO
ESP32 D18 D23 D19
CSN CC1101 > 27 ESP32 GIPO27
GDO0 CC1101 > 27 ESP32 GIPO21
GDO2 CC1101 > 27 ESP32 GIPO33
and PIN_RECEIVER_RST 32 not connected.
In the sensorCfg file, I selected the CC1101.
So far I have not had success as I do not see anything even with the basic example. My station is a Bresser 7002580.
Can I have some help? Thanks, Riccardo.

Hi,
which board did you select in the Arduino IDE?
Can you please provide the detailed compiler output?

Hi!
Usually with this card I select ESP32 Dev Module. With this tab the projects work normally.
I also tried with an ESP8266 MOD 12F board (nodemcu V3) but the board continues to restart once loaded.
In file included from C:\Users\riccardo.sordo\Documents\Arduino\BresserWeatherSensorBasic_1\BresserWeatherSensorBasic_1.ino:61: C:\Users\riccardo.sordo\Documents\Arduino\BresserWeatherSensorBasic_1\WeatherSensorCfg.h:551:48: note: '#pragma message: Receiver chip: [CC1101]'   551 | #pragma message("Receiver chip: " RECEIVER_CHIP)       |                                                ^ C:\Users\riccardo.sordo\Documents\Arduino\BresserWeatherSensorBasic_1\WeatherSensorCfg.h:552:170: note: '#pragma message: Pin config: RST->32, CS->27, GD0/G0/IRQ->21, GDO2/G1/GPIO->33'   552 | #pragma message("Pin config: RST->" STR(PIN_RECEIVER_RST) ", CS->" STR(PIN_RECEIVER_CS) ", GD0/G0/IRQ->" STR(PIN_RECEIVER_IRQ) ", GDO2/G1/GPIO->" STR(PIN_RECEIVER_GPIO) )       |                                                                                ^ Sketch uses 361893 bytes (27%) of program storage space. Maximum is 1310720 bytes. Global variables use 21952 bytes (6%) of dynamic memory, leaving 305728 bytes for local variables. Maximum is 327680 bytes.

I did some checks to understand if there are problems and I only added a few lines to the main code.
I see that the code is executed up to the line: ws.begin();
Then I no longer see anything about serial debugging.
This is true even if I disconnect the CC1101 board
`void setup() {
Serial.begin(115200);
Serial.setDebugOutput(true);
Serial.printf("Starting execution...\n");
initBoard();
Serial.printf("Starting initBoard...\n");
delay(2000);
ws.begin();
Serial.printf("Starting begin...\n");
delay(2000);

}`

The selected pins seem to be o.k. Did you connect the ESP32's MISO to the CC1101's MOSI and vice versa?
The SW gets stuck in an endless loop if the radio chip cannot be initialized.

https://github.com/matthias-bs/BresserWeatherSensorReceiver/issues/112

I pin selezionati sembrano ok Hai collegato il MISO dell'ESP32 al MOSI del CC1101 e viceversa? Il SW si blocca in un loop infinito se il chip radio non può essere inizializzato.

Hi!!
oops.. wait... do I have to connect MISO to MOSI and MOSI to MISO??

Yep!
MOSI: Master Out -> Slave In
MISO: Master In <- Slave Out
In this case, ESP32 is master and CC1101 slave.

Hi!
I tried 100 times even with different ESP32s, nothing to do as soon as you try to initialize the library it freezes. If I install OpenGatewayMQTT I am able to receive RF signals, so I rule out the possibility that the module is faulty or that the ESP32 has problems. There's no way to figure out where the code hangs?

Hi,
This is strange. Are you sure you have a 868 MHz version of the CC1101 module? Sometimes the ads are ambiguous. The chip is the same for 433 and for 868 MHz, but the RF circuit around it and the antenna are different.
Did you enable debug output in the Arduino IDE? You can also get additional debug output from RadioLib - see RadioLib documentation.

HI!!
I'm really starting to suspect that they gave me a 433 receiver, because if I set this frequency with OpenGatewayMQTT, I receive various data from various sensors nearby including a weather sensor. Yet the CC1101's enclosure is rated at 868Mhz. Can the module be forced to work on a different frequency?

N: Send on /RFtoMQTT msg {"active":3,"frequency":433.92,"rssithreshold":-82,"rssi":-80,"avgrssi":0,"count":0}
N: Update check, free heap: 114940E: Error on HTTP requestN: Update check done, free heap: 117040N: Send on /SYStoMQTT msg {"uptime":12,"version":"v1.7.0","disc":true,"ohdisc":false,"env":"esp32dev-multi_receiver","freemem":116688,"mqttp":"1883","mqtts":false,"msgprc":0,"msgblck":0,"maxq":0,"minmem":113688,"tempc":59.44,"freestck":5612,"eth":false,"rssi":-84,"SSID":"WiFiCS001","BSSID":"62:EB:B6:AF:A3:70","ip":"<IP_ADDRESS>","mac":"64:B7:08:C8:C7:10","modules":["WebUI","RF","RF2","RTL_433"]}
N: Send on /WebUItoMQTT msg {"displayMetric":true,"webUISecure":true,"displayQueue":0}
N: Send on /RTL_433toMQTT/Generic-Remote/41946 msg {"model":"Smoke-GS558","unit":41946,"t":118,"arn":"6e5bc5","ol":"ss Smoke and Heat Detector GS 558"," 558":-42,"rssi":1110996}
N: Send on /RTL_433toMQTT/Smoke-GS558/29406 msg {"model":"Smoke-GS558","id":29406,"unit":5,"learn":0,"code":"6e5bc5","protocol":"Wireless Smoke and Heat Detector GS 558","rssi":-42,"duration":1110996}
N: Send on /RTL_433toMQTT/Generic-Remote/41946 msg {"model":"Smoke-GS558","unit":41946,"t":118,"arn":"6e5bc5","ol":"ss Smoke and Heat Detector GS 558"," 558":-68,"rssi":1029997}
N: Send on /RTL_433toMQTT/Smoke-GS558/29406 msg {"model":"Smoke-GS558","id":29406,"unit":5,"learn":0,"code":"6e5bc5","protocol":"Wireless Smoke and Heat Detector GS 558","rssi":-68,"duration":1029997}

Yes, that seems to be the case. If the RF circuit is tuned to 433 MHz, you can't change it to 868 MHz without swapping parts. The antenna is another matter.Am 07.09.2024 09:52 schrieb sRspectrect68 @.***>:
HI!!
I'm really starting to suspect that they gave me a 433 receiver, because if I set this frequency with OpenGatewayMQTT, I receive various data from various sensors nearby including a weather sensor. Yet the CC1101's enclosure is rated at 868Mhz. Can the module be forced to work on a different frequency?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

I'm returning it and having it exchanged for one that is actually at 868. Thanks for the support.

Hi, in the end I had a Heltec Wireless Stick V3 sent to me, to avoid possible problems with the RF module. I selected the basic Test Sensor chetch, I set the relevant board in the Arduino IDE, and in the configuration I uncommented the line "// This define is set by selecting "Board: Heltec Wireless Stick(V3)" (SX1262) in the Arduino IDE:
#define ARDUINO_HELTEC_WIRELESS_STICK_V3". The compiler gives me an error: exit status 1
Compilation error: exit status 1 analyzing output I see: c:\Users\Programmazione\Documents\Arduino\libraries\BresserWeatherSensorReceiver\src\WeatherSensorCfg.h:421:31: error: 'DIO1' was not declared in this scope; did you mean 'GOD0'?
421 |     #define PIN_RECEIVER_GPIO DIO1

Hi,
you don't have to uncomment the line with the board definition, because this definition is set automatically by selecting the board in the Arduino IDE.
I can reproduce the compilation error - something has been changed in the board's pin definitions.

hi, ok, maybe I read the instructions wrong. In the cfg file, I don't have to change anything except the possible exclusion of the sensors. For the rest, do you mean that in your opinion they changed something in the esp32 module?

Right! Well, the change was made months ago, but I missed to integrate it into my code.
Here is the fix:
https://github.com/matthias-bs/BresserWeatherSensorReceiver/pull/194/files
You can change the cfg header file as shown above or pull the changes from GitHub.

Right! Well, the change was made months ago, but I missed to integrate it into my code.
Here is the fix: https://github.com/matthias-bs/BresserWeatherSensorReceiver/pull/194/files
You can change the cfg header file as shown above or pull the changes from GitHub.
Please create a new issue if required, because the topic has changed now.

Hi!!
Now it works, I receive data from the sensor, thank you very much for your assistance.
Riccardo.


Trinity Continuum 2ed - Bug fix - RollTemplate Icon

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[ ] The pull request makes changes to files in only one sub-folder.
[ ] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description
After the last merge of the Trinity Continuum 2e sheet for some reason, the roll templates are not showing up with a correct icon. As a result of this I'm doing a new PUSH in an attempt to solve the errors.

Character Sheet Info Roll20 Internal Use only.


Respect the SSL context's verify_hostname value

I stumbled upon an issue where http.rb wouldn't respect the verify_hostname value of the current OpenSSL::SSL::SSLContext.
Previously we would unconditionally call post_connection_check, whose only purpose is to verify hostnames, regardless of what verify_hostname was set to. This results in quite a bit of user confusion since one expects hostname verification to be handled by OpenSSL itself rather than by the HTTP library.
Digging through the blames, it looks like this was never implemented properly. Ruby's Net::HTTP fixed the same issue with this pull in January.
This simple change fixes the issue for me and brings http.rb in line with the other Ruby HTTP libraries.

This results in quite a bit of user confusion since one expects hostname verification to be handled by OpenSSL itself rather than by the HTTP library.

Yeah, it's a big mess. OpenSSL itself didn't implement hostname verification at all until quite recently, leaving all users of it to reimplement it poorly themselves (and as it were, I ended up rewriting the hostname verification code used by Ruby itself due to numerous problems).
Anyway, approved! Thanks for the fix.

Per #633 CI is broken (thanks Travis), but I'm going to go ahead and merge this.

Seems like this PR broke jRuby:

@ixti aah, unfortunate. Sorry about that.
Seems like it can at least be worked around with reflection.

jRuby has older version of OpenSSL gem bundled in.

https://github.com/httprb/http/pull/634/commits/f8dfb38181be02c973194657242590754eeed51e

I was suggesting something like:
if ssl_context.respond_to?(:verify_hostname) && ssl_context.verify_hostname
  @socket.post_connection_check(host)
end

@tarcieri :+1: Will do so!


Execute the portable version with custom path to Syncthing database

Is it possible to execute the portable version of SyncTrayzor with custom path to Syncthing database?

Sure, just set it in the settings.

Could you please explain more detailed, where I can set it?

Under the Syncthing tab, expand the Advanced section.


Restart a specific job executes all test cases

Hi
I'm using spot instances on GCP to perform my tests according to the technique listed below.
I have to restart the entire workflow if one of these instances is preempted during test execution. If I restart the failed run, all test cases are being executed in one VM, which takes a lot of time. Is it possible to restart one run to only execute the test cases that are assigned to the preempted VM?
    strategy:
      matrix:
        machines: [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]

@nitishvu

Your question seems to be related to the open issue https://github.com/cypress-io/github-action/issues/531

Closing in favor of #531


Updated links

Links to prefabs and demo scene have been changed to reflect new hierarchy.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/spectator-view.md
:white_check_mark:Succeeded
View

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


[#1] Create a bootstrap-cdi example application.

resolves #1

I have cloned the branch into my local branch and played with the example:

It works well. Thanks for the demo James!


⚠ Bofrost.be has degraded performance

In 9a79c7c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1627 ms

Resolved: Bofrost.be performance has improved in 0e6ac07.


Support data expiration timeout

Summary
Some data will expire after a period of time. We should support marking data stale as a timeout.
Acceptance Criteria

[x] Add option to expire data after x milliseconds
[x] Data is marked stale after x milliseconds
[x] Data is fetched after x milliseconds if lazy = false
[x] Determine if polling should be a first order property for easier dev experience

For now we'll implement polling with {staleTime: 10000, lazy: false}. Something about the staleTime API feels off. We shouldn't complicate it further by adding a separate parameter for polling.


Mesh network not being recognized on node

I created 2 nodes on 2 identical PI's, one in gateway mode and the other in internal mode.
When I look at available wifi networks I see the opposite node being advertised at 100% quality.
The OLSRD is running but the status page shows there are no mesh networks that are in range.
I run the daemon from the command line with various settings of debug, but I don't get a clue as to what is wrong.
Anyone have any ideas?
Thanks
Bob
Va3rjs

Having the same issue

I ordered some new wifi dongles to see if that solves,the problem........I suspect that the ones I have work for wifi on the Pi but there may be some issues with the associated drivers.........
I found iwconfig commands do not seem to work on the dongles I have........
I hope the new ones get here this week....I will update you when I they arrive.
Bob

Interesting, I have also ordered new dongles and am awaiting their arrival. Thanks for getting back to me. Look forward to hearing if it solves your issue.
Tyler

On Mar 22, 2016, at 5:56 PM, va3rjs<EMAIL_ADDRESS>wrote:
I ordered some new wifi dongles to see if that solves,the problem........I suspect that the ones I have work for wifi on the Pi but there may be some issues with the associated drivers.........
I found iwconfig commands do not seem to work on the dongles I have........
I hope the new ones get here this week....I will update you when I they arrive.
Bob
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub https://github.com/urlgrey/hsmm-pi/issues/64#issuecomment-200098408

Just received my new USB/WIFI dongles and now I can see other nodes in my MESH here at home...this is significant progress.......I will try setting up some services and see what happens.
Bob

Thats great news. Which dongles did you get?
Tyler

On Mar 24, 2016, at 12:36 PM, va3rjs<EMAIL_ADDRESS>wrote:
Just received my new USB/WIFI dongles and now I can see other nodes in my MESH here at home...this is significant progress.......I will try setting up some services and see what happens.
Bob
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub https://github.com/urlgrey/hsmm-pi/issues/64#issuecomment-200985641

I used one with the  Ralink 5370 chipset......
looks the same as the one in this picture:
HSMM-Pi Blog
  From: trrhodes<EMAIL_ADDRESS>
To: urlgrey/hsmm-pi<EMAIL_ADDRESS>Cc: va3rjs<EMAIL_ADDRESS>Sent: Thursday, March 24, 2016 5:15 PM
Subject: Re: [hsmm-pi] Mesh network not being recognized on node (#64)
Thats great news. Which dongles did you get?
Tyler

On Mar 24, 2016, at 12:36 PM, va3rjs<EMAIL_ADDRESS>wrote:
Just received my new USB/WIFI dongles and now I can see other nodes in my MESH here at home...this is significant progress.......I will try setting up some services and see what happens.
Bob
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub https://github.com/urlgrey/hsmm-pi/issues/64#issuecomment-200985641

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

I'm closing this ticket, as it seems to have been an issue with hardware compatibility.

Yes thanks it as the usb wifi dongle.
B
Sent from my iPad

On Apr 23, 2016, at 5:47 PM, Clayton Smith<EMAIL_ADDRESS>wrote:
I'm closing this ticket, as it seems to have been an issue with hardware compatibility.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub


make csi deployment optional in E2E

Describe the bug
currently, we deploy csi in E2E. so we cannot run the E2E against the already deployed cluster. if we can separate out the E2E and csi deployment, it will help us/others to run E2E tests and make sure things are working fine.

/assign @umangachapagain

This is already fixed, closing this one


Check that contact info for provider is correct

Contact info is incorrect for the following providers (likely also others):
UOC-BC - Martin doesn't work there any more
should be this: https://caldaslab.cruk.cam.ac.uk/bcape/
<EMAIL_ADDRESS>TRACE: https://gbiomed.kuleuven.be/english/research/50488876/54502087/Trace/contact-us
PPTC - contact is email of one person and 261 models are from different labs

For TRACE changes will come in once the data is pushed.

PPTC - need to rerun ETL to verify and then will push

For IRCC-CRC: Not sure if Andrea Bertotti, still works there. Can't find his details on the website (https://www.irccs.com/en/). Should this be changed to Livio Trusolino's contact details (https://www.irccs.com/en/translational-cancer-medicine) or should we drop them an email to confirm?


Content is not up to date

the 'pac pcf init' command does not automatically run 'npm install'. In fact it suggest the following.
Be sure to run 'npm install' or equivalent in this directory to install project dependencies.

I think the documentation is not up to date.

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

ID: e5338bf6-f6c2-fcf9-394f-e30871437de3
Version Independent ID: 03fed979-b86f-d08b-d0ee-3b83b39c0966
Content: Create a model-driven app field component in Microsoft Dataverse - Power Apps
Content Source: powerapps-docs/developer/component-framework/tutorial-create-model-driven-field-component.md
Service: powerapps
Sub-service: pcf
GitHub Login: @noazarur-microsoft
Microsoft Alias: noazarur

Thanks for your comment, @rezanid. This issue has been assigned to the doc author for review.

@rezanid
Thanks for your feedback.
Updating this to use the --run-npm-install parameter so that it will run npm automatically as advertised.


Fix doc_markdown lints in bevy_sprite

#3457 adds the doc_markdown clippy lint, which checks doc comments to make sure code identifiers are escaped with backticks. This causes a lot of lint errors, so this is one of a number of PR's that will fix those lint errors one crate at a time.
This PR fixes lints in the bevy_sprite crate.

bors r+


Is System Assigned Managed Identities supposed to be removed when removing assignment?

Hi.
I am trying to switch from a working deployment with User Assigned Managed Identity to System Assigned Managed Identities. I have understood that it should manage to also remove RBAC roles assigned in roles when you remove an assigment, but I am not sure. I am therefore creating this issue to get some more information.
When I changed from UAMI to SAMI, the RBAC roles where automatically added at the correct management group scope. Now i tried to remove the same assignment to see if it removes the RBAC roles. It does remove the assigment along with the SAMI, but the RBAC roles at management group is not detected for removal, hence I end up with a list of unkown  identities in IAM on the management group. This has been one of my reasons to using UAMI in other policy as code deployments, as we potentially end up with a lot of unkown users in IAM.

and

I am just curious if this is expected result, or if it should detect the removed roles. FYI there where 6 roles added when the assignment was changed from UAMI to SAMI, and I would expect the same 6 roles to be removed when i remove the assignment?
Hope i managed to explain ok and looking forward to hear from you.
Best Regards
Thomas

It should remove them yes - let me test and see if there is an issue - can you confirm that the account running the plan process has the correct permissions as documented -> https://azure.github.io/enterprise-azure-policy-as-code/ci-cd-pipeline/#role-assignments-for-cicd-spns

Hi. Thanks for the quick reply. You were spot on about the permissions graph permissions. Since I had been using UAMI, I never needed to remove roles, as it was the same role assigned to all anyway, but changing to SAMI requires the MS Graph permissions. Now it lists them as removal, but it then fails on the confirm, where it actually tries to remove the roles. Again it seems to fail in regards to graph, but also in regards to teams in the github org?

Hope maybe you can shed some light into what it actually tried to do here? Is it relying on reviewer permissions in graph? This wouldn't work in our setup as our github is not integrated with Entra ID. Also since I am using the same OIDC for both build and deploy, the app registration has the graph permissions in regards to the build phaze, but it seems something else is happening during deploy.
I am running in github with a simplified workflow, but with OIDC instead of credentials.
Best regards,
Thomas

Extra info.
I think, it's this that fails:
          if (Get-ChildItem $env:planFolder -recurse -include roles-plan.json) {
            gh pr edit $env:PR_NUMBER --add-label 'RoleDeployment' --remove-label 'PolicyDeployment'
            gh pr edit $env:PR_NUMBER --title 'Approval Required for Role Deployment' --remove-reviewer $env:REVIEWER
            gh pr edit $env:PR_NUMBER --add-reviewer $env:REVIEWER
          }

It seems the Job/Deploy does not have needed permissions to do these changes even if it has write-all permission.
If I understand correctly, this tries to:

remove existing label and add a new (this works)
change the title of the PR and remove the existing reviewer (does not work)
add the same reviewer again (does not work)

I guess the logic here is that it should update the label, title and reviewer of the PR in order to be approved again so that the roles-plan deployment will run.
There seems to be some different logics in the way that adding roles and removing roles are done in the deployment?
Is there any specific reason to why adding roles are done in the same deployment as creating policies, policy sets and assignments, while removing is separated in two deployments?

FYI. I have the same error when deploying new policyset and assigning them. First it deploys the policyset and the assignment, then it tries to update the PR, which fails, but then i can reapprove the same PR and it works. Gonna test if that also works for removing policy set and assignment with RBAC as well. At leas we know it works, it's just the github logics which do not.

I managed to solve it, but I had to make a change so it runs only once and needs to be approved only once. I added the second line so that both the PolicyPlan and the Rolesplan run in succession after being approved.

Can you also check to make sure the box below is ticked (it might be missing from the documentation) - I think this would fix the issue you were having above.

Can you also check to make sure the box below is ticked (it might be missing from the documentation) - I think this would fix the issue you were having above.

Unfortunately I have already tried that. After doing some googling it seems that github did a change 1-2 years ago limiting how the internal github.token works. I had an earlier post here as well where I also stated I needed to give the job write-all permissions to get it to work. Unfortunately I am running this all on a free version of github though, and if we had enterprise we could probably set it up with a private enterprise token instead. I don't like the PAT option as that is personal, but I also think that requires a paid plan. I am going to deploy this on a different tenant as well where they have enterprise github, so it will be interesting to see if it works better there.

I haven't tried using OIDC but I'll give the whole thing another test and see if I can work out what the issue might be, thanks for doing that investigation
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Thomas Stubergh @.>
Sent: Wednesday, November 22, 2023 6:42:03 PM
To: Azure/enterprise-azure-policy-as-code @.>
Cc: Comment @.>; Assign @.>; Subscribed @.***>
Subject: Re: [Azure/enterprise-azure-policy-as-code] Is System Assigned Managed Identities supposed to be removed when removing assignment? (Issue #423)
Can you also check to make sure the box below is ticked (it might be missing from the documentation) - I think this would fix the issue you were having above.
[image]https://user-images.githubusercontent.com/11281622/284775501-8148abed-c34d-43e3-a060-db9e46ba62fa.png
Unfortunately I have already tried that. After doing some googling it seems that github did a change 1-2 years ago limiting how the internal github.token works. I had an earlier post here as well where I also stated I needed to give the job write-all permissions to get it to work. Unfortunately I am running this all on a free version of github though, and if we had enterprise we could probably set it up with a private enterprise token instead. I don't like the PAT option as that is personal, but I also think that requires a paid plan. I am going to deploy this on a different tenant as well where they have enterprise github, so it will be interesting to see if it works better there.
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/enterprise-azure-policy-as-code/issues/423#issuecomment-1822256492 or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACWCJVSWWLMWDXUOAFNTID3YFWUEZBFKMF2HI4TJMJ2XIZLTSWBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVIZDOMRVHE4DSNBXHCSG4YLNMWUWQYLTL5WGCYTFNSBKK5TBNR2WLKRVHE2TCMJSGEYDQMNENZQW2ZNJNBQXGX3MMFRGK3FMON2WE2TFMN2F65DZOBS2YSLTON2WKQ3PNVWWK3TUUZ2G64DJMNZZJAVEOR4XAZNKOJSXA33TNF2G64TZUV3GC3DVMWUTGMZXGE3DEOBTHCBKI5DZOBS2K2LTON2WLJLWMFWHKZNKGIYDAMZRGAZTQOJYQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKRSG4ZDKOJYHE2DOOECUR2HS4DFUVWGCYTFNSSXMYLMOVS2UNJZGUYTCMRRGA4DDJ3UOJUWOZ3FOKTGG4TFMF2GK.
You are receiving this email because you commented on the thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Fix missing protocol in service when processing ANP named ports

When processing AdminNetworkPolicy and BaselineAdminNetworkPolicy named port rules, the ports section will not have protocol specified. Antrea agent should infer the protocol from the container spec so that rule can be enforced correctly in ovs.
Change has been verified with https://github.com/kubernetes-sigs/network-policy-api/pull/132

/skip-all

@Dyanngg could you backport it to release-1.13 as well?


Download other video resolutions

Currently the only 360p and 720p are the available resolutions to download both audio and video files. It's currently only downloading progressive streams; can anyone extend it to download all available resolutions?

Hi! Leave this issue to me


LinkedIn user profile industry code

Hi,
I'm sorry for reaching you like this but I'm struggling with problem I can't find an answer for ...
I'm developing an application using LinkedIn api, in documentation it is stated that company/industry value should be one of acronyms described here: https://docs.microsoft.com/en-us/linkedin/shared/references/reference-tables/industry-codes but instead of that I'm getting full industry name like "Information Technology and Services", is it possible to get it in "tech" format?
Thanks in advance for any response.

Never mind, I was using V1 api ;)


Adding release stage

Proposal for CI Publishing to crates.io
The following is a very simple solution to ticket #154 . It does not involve any changes to the existing CI infrastructure. It was based off a similar release cycle suggested for the BT-HCI repo/crate (https://github.com/embassy-rs/bt-hci/pull/22)
How it Works
Whenever a new tag is made in the format VX.X.X, the release workflow will activate. It consists of two jobs.

build: Which is identical to the tests,yml. Ensuring that every release runs our ci tests
release: This job will only run if the build job passes, It will first verify that the Github tag matches the config.toml tag. (Assumptions are made here, that both crates will increment their version at the same time) Only then will it continue on with the build process and then publish both a GitHub Release and crates.io release. All tags should match each other, allowing a single source of truth to be found.

What Other options did I try

I tried to get the workflow to depend on the tests.yml workflow but couldn't find a good way of doing it.
I did look into automated PR publishing, however launching on tag means we can choose when we deliver a next release in a low effort fashion. It does rely that main is always tested and in a good stage, otherwise the release workflow will fail (on purpose) preventing a release to crates.io

Testing

Since this requires access to the repos secrets, im unable to see this on my side. I did however make a repo to try out this process, it can be found here: https://github.com/ScottGibb/ci_playground. It does sucessfully publish crates to the staging crates.io https://staging.crates.io/crates/ci_playground. It was tested both in CI workflow and on terminal using the --dry-run command.

At the moment this Release cycle looks almost identical to the one proposed for  https://github.com/embassy-rs/bt-hci/pull/22. However there is a couple of questions that need to be clarified:

There are two crates in this repo, are these being published to crates.io as seperate crates?
- If this is true, how do we handle tagging of this repository? The way this current release flow works, the tagging is a crucial part of ensuring the single source of truth. We could tag the repo twice for each release, but this seems a bit cumborsome

On the Crates.io side, according to the rust book we should handle the cargo.toml like so

Multiple locations
It is possible to specify both a registry version and a git or path location. The git or path dependency will be used locally (in which case the version is checked against the local copy), and when published to a registry like crates.io, it will use the registry version. Other combinations are not allowed. Examples:

[dependencies]
# Uses `my-bitflags` when used locally, and uses
# version 1.0 from crates.io when published.
bitflags = { path = "my-bitflags", version = "1.0" }

# Uses the given git repo when used locally, and uses
# version 1.0 from crates.io when published.
smallvec = { git = "https://github.com/servo/rust-smallvec.git", version = "1.0" }

# N.B. that if a version doesn't match, Cargo will fail to compile!

At the moment this Release cycle looks almost identical to the one proposed for embassy-rs/bt-hci#22. However there is a couple of questions that need to be clarified:
* There are two crates in this repo, are these being published to crates.io as seperate crates?
  - If this is true, how do we handle tagging of this repository? The way this current release flow works, the tagging is a crucial part of ensuring the single source of truth. We could tag the repo twice for each release, but this seems a bit cumborsome

I assume you're referring to trouble-host-macros and trouble-host. Yes, those would both need to be released, but it should be based on the same tag IMHO since we cannot support changes across versions here. Also, it should only release creates that have actually changed, so if nothing changes in trouble-host-macros from 0.1.0 to 0.2.0, it should not be released.

* Do we also want to run the `tests` workflow as well, if this is hardware dependent whats the best way of handling that?

Yes, it should be run. Just running the test workflow should be enough.

On the Crates.io side, according to the rust book we should handle the cargo.toml like so

Multiple locations
It is possible to specify both a registry version and a git or path location. The git or path dependency will be used locally (in which case the version is checked against the local copy), and when published to a registry like crates.io, it will use the registry version. Other combinations are not allowed. Examples:

[dependencies]
# Uses `my-bitflags` when used locally, and uses
# version 1.0 from crates.io when published.
bitflags = { path = "my-bitflags", version = "1.0" }

# Uses the given git repo when used locally, and uses
# version 1.0 from crates.io when published.
smallvec = { git = "https://github.com/servo/rust-smallvec.git", version = "1.0" }

# N.B. that if a version doesn't match, Cargo will fail to compile!

This is for the crate consumer though, not for us?

I assume you're referring to trouble-host-macros and trouble-host. Yes, those would both need to be released, but it should be based on the same tag IMHO since we cannot support changes across versions here. Also, it should only release creates that have actually changed, so if nothing changes in trouble-host-macros from 0.1.0 to 0.2.0, it should not be released.

Ahh okay I think that makes sense, so each increment of this project would mean both crates increment at the same rate. So both projects have the same version number regardless of what code was updated?
As for not releasing to crates.io, does that mean that crates.io would only have for example v0.1.0 for trouble-host-macros but trouble-host might be on for example 0.5.0?

Yes, it should be run. Just running the test workflow should be enough.
Awesome! :)

This is for the crate consumer though, not for us?

I assumed it was for the publisher too? I know at the moment I cant run the --dry-run without specifying the trouble-host-macros crate, After specification it complains it doesn't exist which makes sense.
If the version isn't in the toml it spits out an error like so:
trouble/host on  Add-ci-publish-workflow [!] is 📦 v0.1.0 via 🐍 v3.13.0 via 🦀 v1.82.0 on ☁  (eu-north-1)
❯ cargo publish --dry-run
    Updating crates.io index
error: all dependencies must have a version specified when publishing.
dependency `trouble-host-macros` does not specify a version
Note: The published dependency will use the version from crates.io,
the `path` specification will be removed from the dependency declaration.

As for running the tests @lulf,

Yes, it should be run. Just running the test workflow should be enough.

Since that runs on every PR, is this what you mean by should be enough?

There are two crates in this repo, are these being published to crates.io as seperate crates?
Is there not a way to include both as packages under a single crate? My feeling is that trouble-host-macros doesn't really have meaning without trouble-host, so ideally should be packaged together with it.

I assume you're referring to trouble-host-macros and trouble-host. Yes, those would both need to be released, but it should be based on the same tag IMHO since we cannot support changes across versions here. Also, it should only release creates that have actually changed, so if nothing changes in trouble-host-macros from 0.1.0 to 0.2.0, it should not be released.

Ahh okay I think that makes sense, so each increment of this project would mean both crates increment at the same rate. So both projects have the same version number regardless of what code was updated?
As for not releasing to crates.io, does that mean that crates.io would only have for example v0.1.0 for trouble-host-macros but trouble-host might be on for example 0.5.0?

I thought some more about this, and I think we should tag them independently, it is a lot simpler:

trouble-host and trouble-host-macros are versioned independently
a git tag 'trouble-host-v0.1.0' is created when releasing trouble-host etc.

Yes, it should be run. Just running the test workflow should be enough.

Awesome! :)

This is for the crate consumer though, not for us?

I assumed it was for the publisher too? I know at the moment I cant run the --dry-run without specifying the trouble-host-macros crate, After specification it complains it doesn't exist which makes sense.
If the version isn't in the toml it spits out an error like so:
trouble/host on  Add-ci-publish-workflow [!] is 📦 v0.1.0 via 🐍 v3.13.0 via 🦀 v1.82.0 on ☁  (eu-north-1)
❯ cargo publish --dry-run
    Updating crates.io index
error: all dependencies must have a version specified when publishing.
dependency `trouble-host-macros` does not specify a version
Note: The published dependency will use the version from crates.io,
the `path` specification will be removed from the dependency declaration.

Ah, I see. Yes, we should add a version there.

I believe this is now at a stage for merging the workflow is as follows:

push a tag on the main branch (all other branch tags will be ignored)
the release cycle will trigger
the CI Tests will run
Depending on the tag pushed (trouble-host-vX.X.X/trouble-host-macros-vX.X.X) different release workflows will be launched and if the git tag matches the version on the corresponding cargo.toml, a github release alongside a cargo publish will be launched, otherwise the workflow will fail. If successful, the workflow will complete and the status will be green.

Current Issues that need to be resolved:

[ ] On crates.io we need a placeholder for trouble-host-macros, otherwise the workflows fail
[ ] naming convention for trouble-host-macros to be agreed (ive changed the name of the folder to trouble-host-macros instead of trouble-host-macro).
[ ] Confirmed workflow for hardware in the loop(HIL) tests. I struggled to get any linkage between the release flow and the HIL tests. At present we can assume that any code on main is HIL tested as all merges into main trigger the PR flow which trigger the HIL tests.

When merging, could we please squash the commits, due to the type of PR this is, it required a lot more commits than necessary

/test


Implement Data Analytics and Insights

Implement a data storage mechanism, which will store the total duration of time blocks and the date and time of starting, at the end of all the sessions.
Make a separate page which will show the corresponding data in daily, weekly and monthly format. Using line graphs and heatmaps.
Add an 'export as csv format' aswell.

Hello,
I really liked this project, simple and effective. If you don't mind I would like to contribute to it during hacktoberfest and build the analytics mechanism which will be effective. I have a good experience with python and the web.

@SpreadSheets600 Yes absolutely that would be great! Please share what do you plan to implement? Any inputs apart from my existing plan given above?

Hey, first thing first I would be working on some error handling using exceptions and implementing SQL databases to store the distraction focus time and more, even though I am not that good with the web I would be making a page showing the statistics using pandas and matplotlib.

@datavorous Hey I got a few cool ideas and suggestions apart form the data management, would you mind if I do it in one or create separate PR ? I am new to open source, if you guide me a bit that would be helpful. BTW I did sent you a friend request on discord.

Here this is a simple notification system that could be implemented :)

@datavorous Hello Again,
Sorry for pinging again, I am bit annoying but this project is honestly amazing. Well here's another suggestion which I am working on and is to track the distractions, like attempting to press buttons and moving mouse to the corners and track them as well.

@datavorous Hey I got a few cool ideas and suggestions apart form the data management, would you mind if I do it in one or create separate PR ? I am new to open source, if you guide me a bit that would be helpful. BTW I did sent you a friend request on discord.
 Here this is a simple notification system that could be implemented :)

Create a separate PR it would be helpful. Im new to OpenSource as well :)
Are you using win11toast for notifications? I couldn't make it work properly and got timed out errors repeatedly, let me know if it works properly in your implementation after some test runs.

@datavorous Hello Again, Sorry for pinging again, I am bit annoying but this project is honestly amazing. Well here's another suggestion which I am working on and is to track the distractions, like attempting to press buttons and moving mouse to the corners and track them as well.

That seems cool! I'll get back to you on discord tomorrow. :)

@datavorous i just finished working on the data visulaization, csv export and distraction tracking. It needs a lot of improvement that i will do ... later tomorrow and for now these are the update ....

I am opening a PR in few minutes, sorry it contains the notification stuff using plyer module since win11tost doesn't work well in windows 10 :)

@datavorous Hello Again, Sorry for pinging again, I am bit annoying but this project is honestly amazing. Well here's another suggestion which I am working on and is to track the distractions, like attempting to press buttons and moving mouse to the corners and track them as well.

That seems cool! I'll get back to you on discord tomorrow. :)

Great it would be helpful to communicate


Example Script for Seq2Seq QA?

🚀 Feature request
Hi!
I am trying to use prefix tuning to train a seq2seq question answering model (e.g. T5) but there is currently no example script for it. I have tried adapting the original transformers example but couldn't train the model properly - the resultant model's performance is unreasonably low.
This has been annoying us for weeks but we couldn't make much progress. We are relying on adapter-transformers for our research project, so it will be super helpful if a seqseq qa example script is available!
Thanks!

🚀 Feature request
Hi!
I am trying to use prefix tuning to train a seq2seq question answering model (e.g. T5) but there is currently no example script for it. I have tried adapting the original transformers example but couldn't train the model properly - the resultant model's performance is unreasonably low.
This has been annoying us for weeks but we couldn't make much progress. We are relying on adapter-transformers for our research project, so it will be super helpful if a seqseq qa example script is available!
Thanks!

+1 have you solved yet?


Add cards and fixes a bug with Double Team

Adds the cards:
Trapping Pit, Extremespeed, Triple Slash, Guardian Mode, Refuge, Marching Song.
Fixes a bug with double team where it would expire at the start of next turn, instead of the end of the current turn.


Setups Final Page Modifications for User Manual Screen

Task linked: CU-3hday9z Setup Platform for User Manuals


Create GitHub Issues from a subset of checks.

Adds logic to GHIssuesNotifier._generate_issues that
looks for a checks configuration element.

If the configuration element is found with a value,
only generate GitHub issues based on any checks
contained therein.

Increments version where applicable as I'd like
to use the changes ASAP.

[x] Tick to sign-off your agreement to the Developer Certificate of Origin (DCO) 1.1

What

Adds logic to allow GitHub Issue notifications based on a subset of checks in a given accreditation.

Why

Not all checks in a given accreditation necessarily need separate GitHub Issues created for them.

How

Add logic to GHIssuesNotifier.__generate_issues to look for a non-empty checks configuration element within a given accreditation and then only generate issues for the specified checks if it's found.

Test

Manually tested it against GitHub project to make sure it created a card and/or updated an existing issue.

Context

Closes #93
Closes #37

I think this also satisfies #37.  I've gone ahead and added it to the PR description.


hard coded PYTHONPATH to /home/Patrick/python/pi3d

In viewer_display.py line 2 there is a hard coded extension to PYTHONPATH.
sys.path.insert(1, '/home/patrick/python/pi3d')

I propose for debug reasons to change PYTONPATH on console with
export PYTHONPATH=/home/Patrick/python/pi3d:$PYTHONPATH

Or in visual studio code make use of launch.jason
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "name": "picframe",
            "type": "python",
            "request": "launch",
            "justMyCode": false,
            "module": "picframe.start",
            "args": ["~/picframe_data/config/configuration.yaml"],
            "env": {
                "DISPLAY": ":0.0",
                "DYLD_FALLBACK_LIBRARY_PATH": "/opt/x11/lib",
                "PYTHONPATH": "/Users/herbe/Development/pi3d:$PYTHONPATH"
            },
        }
    ],
}

Fixed with e55bd854e97bc0d9f15f526ef6e6adc2afb26d60


Gopass doesn't syncing with repository

I've installed gopass on Ubuntu 18.04 host, everything is working fine except one thing.
When I run syncing command:
$ gopass sync
Sync starting ...
[<root>]
   WARNING: Mount uses RCS backend 'noop'. Not syncing!
All done

I didn't encountere such problem on Mac OS
Update:
When I'm creating/inserting password, it didn't add gpg file to repository, just put that file in ~/.password-store

I think you need to set gitcli as the syncing backend.
If you type gopass config and check the path, I think it should say noop there, replace it with gitcli.
In linux the config is located in ~/.config/gopass/config.yml.

Thanks
That is exactly what I needed

How to set path using gopass config command

That's fixed in master, but not yet part of any release.


completion icons

Completions
CompItemKinds for icons:
. - Class
value() -> Variable
-st-named -> (var -> Variable, cls -> Class)
: -> Enum
:vars -> Keyword
:import -> Keyword
-st-* directive -> Keyword

Closed. We use the same icon for everything.


FreeType check?

Has a check that validates that a font can be rasterized by FreeType been considered already?
I'm asking because AnchoGX.ttf version 1.000 (from http://beatrizl.com/project/ancho) crashes freetype-py v2.2.0.
fontview v0.3.1 also doesn't render it, I assume because of a FreeType failure.
import freetype
face = freetype.Face("AnchoGX.ttf")
face.set_char_size(48 * 64)
face.load_char('S')
bitmap = face.glyph.bitmap
print(bitmap.buffer)

The snippet above crashes with this error,
freetype.ft_errors.FT_Exception: FT_Exception:  (too many function definitions)

And Rubik-Italic.ttf crashes with,
freetype.ft_errors.FT_Exception: FT_Exception:  (stack overflow)

It would be nice to gather more info about this crash and also report it to the FreeType dev team.

So far, FreeType is the only tool I found that cannot handle these fonts. I'll open an issue at freetype-py.

This specific error message means that the hinting code on this font exceeded freetype limits.

https://github.com/servo/libfreetype2/blob/f801061ee57dac0937cccffb0f7bf3fa120208f5/freetype2/src/truetype/ttinterp.c

The feedback on the linked issues suggests that the fonts are malformed.
@felipesanches I'm interested in adding a FreeType check; it will have a dependency on freetype-py. Do you have any preference on which profile it should go into, and any guidance on how to implement it?

Chiming in with my vote for this also (and willingness to help implement). Considering the widespread use of FreeType, I think it would be helpful to have a check that actually exercises fonts in the rasterizer. Worth noting too that other implementations that appear to work are probably just silently failing and attempting to continue using the malformed font while skipping bad glyphs/hints/etc.

@felipesanches I'm interested in adding a FreeType check; it will have a dependency on freetype-py.

That's OK.

Do you have any preference on which profile it should go into?

Sounds like a check for the universal profile ;-)

and any guidance on how to implement it?

I'm not sure. A long time ago there was a check that relied on a fork of MSFontValidator that tapped into a patched freetype that exposed those hinting bytecode errors. But it was very unpleasant to work with these patched dependencies. Not sure if you are planning to do something similar to that, or if you have something else in mind.
Anyway, I'll surely be glad to review a pull request and offer you some guidance if there's anything in our codebase you need help with.

FWIW, exc->maxFDefs is read from the font's cvt table. The freetype message just indicates the font is broken, in that some of values in its cvt table is just wrong (too low). It is not a freetype limit, it is just a wrong number in the font itself.

@miguelsousa the Microsoft documentation is quite clear about E6045 about what's wrong with the font and what the corrective action from the font designer is https://github.com/HinTak/Font-Validator/blob/b8394cb446c1c4963e88f6f2242c6847378ca1b0/GenerateFValData/OurData.xml#L5163

@miguelsousa likewise, the description for E6016 is quite clear that maxp table in the font is wrong.
And what you need to do to correct the font: https://github.com/HinTak/Font-Validator/blob/b8394cb446c1c4963e88f6f2242c6847378ca1b0/GenerateFValData/OurData.xml#L4833 .
This is original Microsoft documentation , written by Microsoft folks - you would see that my only change to this part is documenting how freetype differs from the Microsoft rasterizer in situations where the font is broken in multiple ways.

@miguelsousa and there is a "official" python binding https://github.com/FontVal-extras/freetype-py to the freetype-pretending-to-be-Microsoft-rasterizer backend.

@HinTak thanks for the info.


Enhanced OpenJSCAD.org code with Touch device support.

Folks. I think this pull request is no longer required. I believe that OpenJSCAD.org has support for touch events, and supports all touch devices. Please correct me if wrong.


chore: update dependency semantic-release-pub to 0.2.1

:tada: This PR is included in version 2.0.1 :tada:
The release is available on:

v2.0.1
GitHub release

Your semantic-release bot :package::rocket:


Skip test "image status should support all kinds of references" for docker runtime

Refer #473
Skip the test for docker because dockershim refuses the image inspect requests for digest-only references.

The test is also making a few PRs won't be able to pass the CI.

Merging this since it's blocking a few other PRs. Would investigate whether it should be fixed in dockershim.


Ecommerce - Tracking Product Action Event

to be able to track product action events like:

add to cart
remove from cart
view product detail
product click

by using trackProductActionEvent without using transaction

@cbrevik Any news on this? #150 mentions a PR would be great, and this one seems to be a fine candidate.

I’ve been rewriting the API surface, which will also support a lot more ecommerce stuff: https://github.com/idehub/react-native-google-analytics-bridge/tree/rewrite-api-surface
It’s slated for a release as soon as I am able. Needs some more testing first.

Closed with ead3e84faf4a8fb4255c0d31d6b6418ade633a6b


Implement Artwork Offer

--need to work on this

Already done, kindly check the main branch as I've already merged the latest updates.

As per my latest update, viewing all the detail screens are already done. What's left to do now is implement the make an offer, and hire artist.

--most ui + logic taken care of by @ajcatindig


Improve the design of the social media icons in the footer of the GirlScript Chennai website. We need Instagram,Twitter,LinkedIN,Facebook

I would like to work on this issue as part of GSSOC'2020. I would be highly obliged if you can grant me permission to work on this issue @smaranjitghose

i would like to work on this issue.

I would like to work on this issue


Feature/refactor a star

We've refactored the A* implementation. Would you mind having a look, @wahllca?
Closes #3

Looks good to me :+1:


Roll Flutter Engine from e29263212bfd to 824b5a7fb799 (4 revisions)

https://github.com/flutter/engine/compare/e29263212bfd...824b5a7fb799
2024-02-02<EMAIL_ADDRESS>Revert "[Fuchsia] Execute most of the testing/fuchsia/test_suites.yaml on debug and release builds" (flutter/engine#50291)
2024-02-02<EMAIL_ADDRESS>Move linux_fuchsia to staging (flutter/engine#50288)
2024-02-02<EMAIL_ADDRESS>Add the focus state related methods to the platform dispatcher (flutter/engine#50273)
2024-02-02<EMAIL_ADDRESS>Add a link the different possible Android virtual device configs (flutter/engine#50267)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC matanl@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Container unable to validate certs to perform OAuth

I've been experience an intermittent issue (that's becoming more frequent), where cloud sql proxy can't auth via OAuth. It continues along and says it's ready for new connections, but it's not.
In the logs, I see something like this:
Get https://www.googleapis.com/sql/v1beta4/projects/xxxxx-11111/instances/staging?alt=json&prettyPrint=false: oauth2: cannot fetch token: Post https://oauth2.googleapis.com/token: dial tcp: i/o timeout

I've seen other people talk about this years ago now, but there was never any clear resolution in their discussions. It just seemed to magically go away.
I was presented with this issue again and had a chance to look into the container a bit. Outgoing connections are working, but not for SSL. Another container in the same pod can make outgoing HTTPS requests. So I don't think there's any sort of networking issue with the pod itself at least.
/ $ wget -O - https://oauth2.googleapis.com
Connecting to oauth2.googleapis.com (<IP_ADDRESS>:443)
wget: SSL/TLS certificate is not being validated!

Of course on Alpine:
/ $ wget --no-check-certificate -O - https://oauth2.googleapis.com/token
Connecting to oauth2.googleapis.com (<IP_ADDRESS>:443)
wget: can't execute 'ssl_helper': No such file or directory

I've mounted the ssl certs volume as described in a readme somewhere (it wasn't before actually and it still ran). I can see a list of certs in /etc/ssl/certs. This didn't help. There are certs in ca-certificates.crt as well. Is it perhaps an authority that needs to be updated?
I'm also running this as a sidecar. I just realized other readmes about running as a service. I'm unsure if that matters here. It just seems that the container that can't connect to HTTPS for the OAuth, which I'd assume would still be an issue whether it's a sidecar or a service.
Any ideas?

What image/version are you using for the proxy? Can you provide more info on how you are deploying the proxy?

gcr.io/cloudsql-docker/gce-proxy:1.15 as a sidecar.

Noticed the build.sh Alpine 3.5. Current seems to be 3.11, but more importantly there's a related issue: https://github.com/docker-library/official-images/issues/2773 ... For some the apk command to add certs and then update them seemed to work, but for others it did not.

Any reason you haven't upgraded to 1.16?

I've since upgraded to 1.16. The timeout is new though, it hasn't happened before regardless of version. Still hasn't occurred in our production environment either. I'm not certain it's tied to a version because of how intermittent it has been. Also not sure the newer Alpine version would help, though I did see that related issue that was also intermittent. The other container in the pod can make HTTPS requests so the pod and networking should all be fine. It has GCP support stumped too.

This seems similar to #357, which also was unable to get an oauth token. (there's a different specific error here, it appears that the request to https://oauth2.googleapis.com/token is failing here compared to a request to the metadata server failing in the other issue.)
Where is your pod running? Like, which cloud and which cloud product?
How is the proxy set up to authenticate? --credential_file?
What resources do you have allocated to the pod? You could try increasing the allocation of CPU/memory if it is relatively low.
If #357 really is related, perhaps there's a common root cause within Google's backend auth stack.

The resource allocation is what I was suspicious of as well.
However 1.16 image switched to a distroless base image - so if alpine was the culprit I would expect that version bump to resolve the issue.

Everything is running in GKE, there's two separate projects. In the Secrets, there's the instance credentials JSON for a service account.
Unfortunately, it's a painfully slow trial and error. I'll see how 1.16 works as it was a recent update. Good to know the base image isn't going to be an issue. Again, I'm just searching for related issues between everything that's in use. Being able to tick items off the list is great.

@tmaiaroto is this issue resolved?

Can confirm, latest version worked. Or something else changed, but all good. Thanks!


check whether handling of large numbers in YAML is consistent with docker-compose

No, looks like docker-compose uses PyYAML, which will use python integers for big integers. It appears that floats are interpreted in the same fashion, however, since the data type for go/kube-compose and python/docker-compose is both IEEE754 binary fp


Tweaked Dataset.plot docs to better describe quiver

Based on issue #5002, I did a few tweaks to make it clearer which parts of the generated documentation apply to quiver, and which apply to scatter. I also removed references to contourf because that is now documented in a separate location.

Thanks @cspencerjones


Updated readme with steps to use the package.

Hello
I am a student. I had used this repository in an NLP based project. Initially, I faced some difficulty in using this package. So, now I have updated the readme with steps to use the package so that it would be easier for other people to use.
Thank You.

Thanks for this. 👍


[WIP] Support range format ProtoPort

Enable the ProtoPort parameter to support range formats when the user inputs as follows:
reject(all, tcp/1022-1024)

It will works as 3 rules below:
reject(all, tcp/1022)
reject(all, tcp/1023)
reject(all, tcp/1024)

This feature is particularly beneficial for blocking a wide range of contiguous ports.
# Ban Bittorrent Protocol
- reject(all, tcp/6969)
- reject(all, tcp/6881-6889)
- reject(all, tcp/10001-65535)

I definitely like the idea, but I don't think this implementation of splitting each port into a separate rule is ideal...we should instead change the internal rule structure to support range ports.


Documentation

The README doesn't describe how to use the library. It should have basic examples of defining a variant type, defining and using a visitor, and so on..

Yes please. That's the reason we're pointing to the Boost.Variant docs for our users.
There are differences between Boost.Variant and mapbox/variant that are not documented (e.g. member function vs free standing get, no is<T> in Boost.Variant, and so on). It would be great to document not only the usage but also the differences to popular alternatives such as Boost.Variant.

@jfirebaugh @daniel-j-h - good call


[2i2c: temple] Investigate user report of kernel crashes

Context
Reported via: https://2i2c.freshdesk.com/a/tickets/494
On our current week's project we are getting an inordinate number of kernel crashes. The data set is larger than others we use at 6.6 mb.
Proposal
No response
Updates and actions
No response

Initial log exploration link https://cloudlogging.app.goo.gl/uvmQ9KNvDpnyCzSq7

I think you can also add resource.labels.container_name="notebook" to the query, as kernels are in the user's container, which is identified by the name 'notebook'

The most common cause of kernel restarts is notebooks hitting the memory limit.  This is memory usage graph for last week (https://grafana.pilot.2i2c.cloud/d/MMHgC_Qnz/cluster-information?orgId=1&viewPanel=10&from=now-7d&to=now)

And here is memory usage per-user as a heatmap:

It's not entirely clear there's a lot of people hitting the limit, but some people definitely are.
We also have an OOM killer graph, that reports some killings - but we can't be sure if those are from temple or elsewhere (the recent ones are definitely from oceanhackweeks):

The users have a 256M Guarantee and a 1G limit (which is the default for all of basehub, and i see no overrides in temple's config). Perhaps because the nodes are getting fuller, they are getting killed before they reach 1G as well.
So two options here are:

Increase the memory guarantee, to maybe 512MB.
Increase the memory limit, perhaps even temporaryily, to 2G.

I think in the longer term, this should be written into contracts (cc @colliand and @jmunroe) as this will increase our cloud costs.

[ ]  Roll back #2274 after March 4

No OOM kills since guarantee bump. Putting in waiting as pending until after March 4


nixos/systemd-networkd-vrf: implement working TCP test on a 5.x kernel

Motivation for this change
By design, VRFs allow route-leaking for forwarded packages, but not for
local processes using a socket. While it was possible to leak such TCP
traffic through a VRF on a 4.x kernel, this behavior was considered
wrong and got fixed in Linux 5.x[1].
From now on, local unix sockets must run in the VRF itself using
ip vrf exec[2] which basically injects a BPF program into the VRF and
drops elevated networking capabilities by default for the specified
command.
[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=3c82a21f4320c8d54cf6456b27c8d49e5ffb722e
[2] https://man7.org/linux/man-pages/man8/ip-vrf.8.html
Things done

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

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

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

@GrahamcOfBorg test systemd-networkd-vrf


docs: relationship between "registerType", "onNeedRefresh" and "onRegistered".

From reading the guides I personally don't understand the relationship between "registerType", "onNeedRefresh" and "onRegistered".
For example, if I set registerType: 'autoUpdate':

Do I still need to configure "onNeedRefresh" with a call to "updateSW"? or is that redundant?
Do I still need to follow the "Periodic Service Worker Update" guide and set an interval inside "onRegistered"?

If I guess correctly, the "auto-update.html" and "prompt-for-update.html" guide are mutually exclusive, while the "periodic-sw-updates.html" is relevant for both of them.
Am I right?
Thanks for this great plugin.

@danielyogel I use strategy for both strategies and registerType options: former for build time and latter for client behavior/strategy

onNeedRefresh will never be called since you're using auto update client strategy (registerType): it will be called only when using prompt client strategy and a new sw detected. onOfflineReady will be called once the sw is registered first time: the callback will be called in both client strategies.
onRegistered will be always called if provided from sw registration callback, just check https://github.com/antfu/vite-plugin-pwa/blob/main/src/client/build/register.ts#L83

Yes, autoUpdate and prompt client strategies (registerType) are mutually exclusive, you can only configure one client strategy on plugin options.
About the periodic sw updates:
While your users are using the app, there is no way to hint the browser there is a new app version (you can deploy it at any time). Periodic sw updates will just ping the server to check if the sw has changed: once sw change the browser will try to install it: if using autoUpdate client strategy then the sw will be activated and all clients (page/tabs/another browser nstances) using it will be reloaded, while using prompt client strategy the sw will keep in skip waiting state awaiting user confirmation, once the client/user confirm its installation the browser will proceed with its activation (the same process previously described for autoUpdate client strategy.
I don't use periodic sw updates: you can use server push notifications instead but not all browser support it. periodic sw updates is the simplest strategy to keep the app updated.
Depending on the type of application it may be necessary that you have to use it:

For example, if the application is something critical, you can use web sockets and register all clients to be notified (it would be similar to server push notifications), so once a new version of the application is deployed, you send a notification to all clients and it would only be necessary to add the logic to update the app (for example, saving the reference to the sw via onRegistered and calling it when the notification is received from the web socket).
On the other hand, if it is not something critical, users usually turn off the computer or close the browser or if it is on the mobile, put the browser in the background, once the application is navigated, it is usually reloaded and if there is a new version the browser will take care of activating the update process

you can take a look at this issue (take your time, it is huge): https://github.com/antfu/vite-plugin-pwa/issues/33
check https://github.com/antfu/vite-plugin-pwa/issues/33#issuecomment-792680487 and a few comments I made in response

@userquin Many thanks for the detailed answer!

I use strategy for both strategies and registerType options

I don't really understand this sentence... :\  I've defined only "registerType", anyway.

I don't use periodic sw updates:

I understand I may just leave it off, and the app eventually will update when navigated, but what about "installed" (added to home screen) apps?

you can use server push notifications instead

If I do decide to use periodic updated, using push and web sockets -  how do I know that an asset was changed in the backend, do I do it manually by inspecting the files?
const updateSW = registerSW({
  onRegistered(r) {
    r && setInterval(() => {
      r.update()
    }, intervalMS)
  }
})

What exactly r.update() does? is this call just refreshes current browser? is it triggering some request to the server to check for new assets?
Thanks again!

@danielyogel
There are 2 plugin options: strategies  (default is generateSW) and registerType (default is prompt), I use strategy to refer both, former for build time and latter for client behavior.
r.update() will ping the sw url to the server (via GET or HEAD), if changed then will proceed to install it in the background without refreshing the page: browser will start download new app version (all assets listed on the sw precache manifest), once pre cache manifest dowenloaded, it will emit new sw available event and depending the registerType will proceed as explained on previous entry.
vite will emit your assets using some hash algorithm (sha256) and so all stuff unchanged will be generated with the same hash. Just check the sw.js on your app on source dev tools, see precacheAndRoute() method (seescreenshot  below).  Check also this article https://css-tricks.com/vitepwa-plugin-offline-service-worker/, you will find some reference how the pwa works (the cache and versioning). You can try to build your app twice, the content will be the same (if nothing changed).
If you will use web socket, you'll need to add some functionality on your app to allow send you the web socket event as administrator, and then send the event once the new app is deployed and available. If you send that web socket event and the r.update() result is that the sw is not changed, then there will not be app update (just like if you refresh the page and no new app deployed/changed).

Thanks for the detailed response!
A few more questions, if that's fine:

If I use registerType: 'autoUpdate', and never configure onRegistered at all - will the app still update when reloaded - or I have to call onRegistered(r) {r.update()} at least once? (I'm not interested in periodic update).
When I use registerType: 'autoUpdate' - does that can cause a flickering when loading the app, everytime? I see some flickering when loading my webapps which are configured this way.
Does registerType: 'autoUpdate' updates the webapp immediately when loading it? or does it takes some time?

@danielyogel

onRegistered is optional so you don't need to worry about it, the app will be auto updated with a page reload: the check for new sw will only occur when entering the page or a page refresh (this is why we include the periodic check for updates)
should be only a flickering only when received an updated and reloading the page
no, the time will depend on the app size: when the sw detects a new version, it will try to install all new resources (downloading all resources), then, the current sw will fire a page reload forcing the new sw version to take the control of all opened client/tabs where the app is running


refactor: Member.has_role

What type of pull request is this?

[X] Non-breaking code change
[ ] Breaking code change
[ ] Documentation change/addition

Description
Refactored Member.has_role to use all() function and generator expression
Changes

refactored Member.has_role, no behavioral changes, improved performance is expected

Checklist

[X] I've formatted my code with Black
[X] I've ensured my code works on Python 3.10.x
[X] I've tested my code

I like the code. i dont like this bit:

You can remove the Role typehint from the methods roles argument because Role meets Snowflake_Type

I like the code. i dont like this bit:

Lol, sorry, somehow missed those. Though they are all done

You can remove the Role typehint from the methods roles argument because Role meets Snowflake_Type

Agreed, though given same typehint is used in other methods too, it's out of scope of current PR IMO


bug fix null pointer exception in get partition visible version rpc

Proposed changes
Issue Number: close #xxx
fix null pointer exception in get partition visible version rpc

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

run buildall

run cloud_p1

run buildall


Spurious networking errors

Hi @crohr, first and foremost just wanna say thanks for creating this awesome project! It's my first day trying this out and I'm pretty impressed by how easy it is to get working (mostly) in under an hour.
However, I have noticed spurious networking errors occurring in some of our jobs quite consistently. Not sure if it's because of the highly parallel nature of our workflow that's pushing some network/os limits, yet such issues are very rare with the default github runners.  Wondering if you have any idea what may cause this?
We are using the default cloudformation stack values, and our jobs are running on one of these two configs:

runs-on: [runs-on,runner=2cpu-linux-x64,"run-id=${{ github.run_id }}"]
runs-on: [runs-on,runner=4cpu-linux-x64,"run-id=${{ github.run_id }}"]

Some error logs coming from our python tests:

Hi @marqueurs404, could you tell a bit more about your configuration: region, private mode or not?
That's the first time I see this kind of feedback (and many thousands of runners are launched every hour), so I suspect it is specific to your workflow. Can you tell more about what hosts it is trying to reach?


Question: How to search for two string in different lines (accepting line breaks between them)

Hi  @PeterBooker  - sorry for asking you such a basic question, but I can’t find a working search with line breaks on http://wpdirectory.net
I started with a basic search like this: https://wpdirectory.net/search/01EJEZ66ZANYGE43SCMVJW96EA
to find themes with the issue of checking the wrong comment_type (the change of comment_type is explained here: https://make.wordpress.org/core/2020/07/29/miscellaneous-developer-focused-changes-in-wordpress-5-5/).
All hints on stackexchange don’t work for this regex variant ...
I tried many ...
with [/s]
https://wpdirectory.net/search/01EJEVTGYD4W4VJFVYQ0RV2JNG
https://wpdirectory.net/search/01EJEW571RFRFK7J5WJWGWVJX9
https://wpdirectory.net/search/01EJEXRMBC27VKYZ4VBFZHEMPW
https://wpdirectory.net/search/01EJEX5ETGBR21NZY9SQTDF0A6
with (?ms)
https://wpdirectory.net/search/01EJEVXJE4ZCTNJD5M7QCVWB6P
with [\r|\n]
https://wpdirectory.net/search/01EJEXC1YDDFBWA6E9FDQ9P020
https://wpdirectory.net/search/01EJEXB4HQDH82E823H7374QKQ
https://wpdirectory.net/search/01EJEWER1D6T675A8C7PYJ4XDC
Any help to find the two strings which are in one file but spread on different lines is is much appreciated!

@Zodiac1978 I believe this syntax can be used to match newline characters:
first_string(?s).+second_string.
And yes, I realize this issue was opened 4 years ago.

Hey @anton-vlasenko
appreciate the ping! After 4 years, I still not found a way to search this way ...
Tried your approach: https://wpdirectory.net/search/01JFCEQ52XZ5X7FMVKXJH4TF8W
But zero results.
But I think this should have found this part of Antispam Bee:
https://github.com/pluginkollektiv/antispam-bee/blob/master/antispam_bee.php#L254-L262
In PHP it works:
https://regex101.com/r/u6RYGx/1
Unfortunately this does not seem to work on https://wpdirectory.net

Another example from my initial plan to find broken themes:
https://wpdirectory.net/search/01JFCFEDWTJJ5JFAM8AM5ZPVJ5
This search should have found the Yoko theme:
https://themes.svn.wordpress.org/yoko/1.2.5/functions.php

Hmm, I'm confused now because I wonder how this query is working
https://www.wpdirectory.net/search/01JFAHJT537STY2RME331GR8W8
while other similar queries (example) clearly don't.
That's why I thought the synthax above should do the trick.
Maybe I'm missing something very obvious.


removed reference to PSReadline.dll-help.xml in csproj file

enables building without error in VS

That's not the right change - the file is generated by PlatyPS - the fix should be to build that file.

I see, so PlatyPS builds the xml from a md file.


Fixes a problem that would not allow you to import using swift

I just tried to use PureLayout in swift, but it gave me an error. Something like this:
could not build objective-c module
It may be a bug with swift or frameworks, but if you have private headers they will be include anyway if you add them to the targets, which I believe causes the above error. I have now unchecked "add to target"  and it seems to solve the problem.
Take a look and see if you can agree with me.

Hey Christian,
Can you provide a little more detail on how you have integrated PureLayout into your project? Are you using CocoaPods or Carthage, or are you manually dropping the source files in?
Better yet, if you can toss up a sample project/workspace that reproduces the issue that would help me figure out what the problem might be most easily.
The PureLayout-Internal.h header is actually an important header to include in the framework. It can be used by other projects that depend on accessing the internals of PureLayout. So you may be right that something needs adjustment in the targets/visibility, but removing it altogether isn't going to be the right solution.

@chrene Any updates on this?

Closing this PR. Please let me know if this is still a problem.