Using digits to make predictions

Hi
If I trained my network using caffe and I have the depoly.prototxt and the training output (.caffemodel)
can I use digits to make predictions?
Thanks for your time

Hello, you can upload your pretrained model using the "Pretrained models" tab on the main page. You will need to train a DIGITS model using your pretrained model if you wish to use DIGITS to make predictions. Note also that you will need to select compatible pre-processing options (mean subtraction, crop, ...)

Thanks for your prompt response
If I'm working on Nivida server, Should I download the model at my desk and upload it again to digits?
Is there a way to use path at the server?
Thanks

You can't point to the file paths, you need to upload the files using the web interface.

Ok, I uploaded the model
but what should I do after that? I thought it will give me link to make predictions to test images.

Ok, I got it
Thank u so much for your help

The problem is I'm getting this error:
ERROR: Your deploy network is missing a Softmax layer! Read the documentation for custom networks and/or look at the standard networks for examples.
My val_traing prototxt:
name: "VGG_ILSVRC_16_layers"
layers {
name: "data"
type: DATA
include {
phase: TRAIN
}
transform_param {
crop_size: 224
mean_file: "/home/eng_rkotb/DIGITS/digits/jobs/20161127-013943-824e/mean.binaryproto"
mirror: true
}
data_param {
source: "/home/eng_rkotb/DIGITS/digits/jobs/20161127-013943-824e/train_db"
batch_size: 32
backend: LMDB
}
top: "data"
top: "label"
}
layers {
name: "data"
type: DATA
include {
phase: TEST
}
transform_param {
crop_size: 224
mean_file: "/home/eng_rkotb/DIGITS/digits/jobs/20161127-013943-824e/mean.binaryproto"
mirror: false
}
data_param {
source: "/home/eng_rkotb/DIGITS/digits/jobs/20161127-013943-824e/val_db"
batch_size: 25
backend: LMDB
}
top: "data"
top: "label"
}
layers {
bottom: "data"
top: "conv1_1"
name: "conv1_1"
type: CONVOLUTION
convolution_param {
num_output: 64
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv1_1"
top: "conv1_1"
name: "relu1_1"
type: RELU
}
layers {
bottom: "conv1_1"
top: "conv1_2"
name: "conv1_2"
type: CONVOLUTION
convolution_param {
num_output: 64
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv1_2"
top: "conv1_2"
name: "relu1_2"
type: RELU
}
layers {
bottom: "conv1_2"
top: "pool1"
name: "pool1"
type: POOLING
pooling_param {
pool: MAX
kernel_size: 2
stride: 2
}
}
layers {
bottom: "pool1"
top: "conv2_1"
name: "conv2_1"
type: CONVOLUTION
convolution_param {
num_output: 128
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv2_1"
top: "conv2_1"
name: "relu2_1"
type: RELU
}
layers {
bottom: "conv2_1"
top: "conv2_2"
name: "conv2_2"
type: CONVOLUTION
convolution_param {
num_output: 128
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv2_2"
top: "conv2_2"
name: "relu2_2"
type: RELU
}
layers {
bottom: "conv2_2"
top: "pool2"
name: "pool2"
type: POOLING
pooling_param {
pool: MAX
kernel_size: 2
stride: 2
}
}
layers {
bottom: "pool2"
top: "conv3_1"
name: "conv3_1"
type: CONVOLUTION
convolution_param {
num_output: 256
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv3_1"
top: "conv3_1"
name: "relu3_1"
type: RELU
}
layers {
bottom: "conv3_1"
top: "conv3_2"
name: "conv3_2"
type: CONVOLUTION
convolution_param {
num_output: 256
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv3_2"
top: "conv3_2"
name: "relu3_2"
type: RELU
}
layers {
bottom: "conv3_2"
top: "conv3_3"
name: "conv3_3"
type: CONVOLUTION
convolution_param {
num_output: 256
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv3_3"
top: "conv3_3"
name: "relu3_3"
type: RELU
}
layers {
bottom: "conv3_3"
top: "pool3"
name: "pool3"
type: POOLING
pooling_param {
pool: MAX
kernel_size: 2
stride: 2
}
}
layers {
bottom: "pool3"
top: "conv4_1"
name: "conv4_1"
type: CONVOLUTION
convolution_param {
num_output: 512
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv4_1"
top: "conv4_1"
name: "relu4_1"
type: RELU
}
layers {
bottom: "conv4_1"
top: "conv4_2"
name: "conv4_2"
type: CONVOLUTION
convolution_param {
num_output: 512
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv4_2"
top: "conv4_2"
name: "relu4_2"
type: RELU
}
layers {
bottom: "conv4_2"
top: "conv4_3"
name: "conv4_3"
type: CONVOLUTION
convolution_param {
num_output: 512
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv4_3"
top: "conv4_3"
name: "relu4_3"
type: RELU
}
layers {
bottom: "conv4_3"
top: "pool4"
name: "pool4"
type: POOLING
pooling_param {
pool: MAX
kernel_size: 2
stride: 2
}
}
layers {
bottom: "pool4"
top: "conv5_1"
name: "conv5_1"
type: CONVOLUTION
convolution_param {
num_output: 512
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv5_1"
top: "conv5_1"
name: "relu5_1"
type: RELU
}
layers {
bottom: "conv5_1"
top: "conv5_2"
name: "conv5_2"
type: CONVOLUTION
convolution_param {
num_output: 512
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv5_2"
top: "conv5_2"
name: "relu5_2"
type: RELU
}
layers {
bottom: "conv5_2"
top: "conv5_3"
name: "conv5_3"
type: CONVOLUTION
convolution_param {
num_output: 512
pad: 1
kernel_size: 3
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "conv5_3"
top: "conv5_3"
name: "relu5_3"
type: RELU
}
layers {
bottom: "conv5_3"
top: "pool5"
name: "pool5"
type: POOLING
pooling_param {
pool: MAX
kernel_size: 2
stride: 2
}
}
layers {
bottom: "pool5"
top: "fc6"
name: "fc6"
type: INNER_PRODUCT
inner_product_param {
num_output: 4096
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "fc6"
top: "fc6"
name: "relu6"
type: RELU
}
layers {
bottom: "fc6"
top: "fc6"
name: "drop6"
type: DROPOUT
dropout_param {
dropout_ratio: 0.5
}
}
layers {
bottom: "fc6"
top: "fc7"
name: "fc7"
type: INNER_PRODUCT
inner_product_param {
num_output: 4096
}
blobs_lr: 0
blobs_lr: 0
}
layers {
bottom: "fc7"
top: "fc7"
name: "relu7"
type: RELU
}
layers {
bottom: "fc7"
top: "fc7"
name: "drop7"
type: DROPOUT
dropout_param {
dropout_ratio: 0.5
}
}
layers {
name: "fc8_mini"
bottom: "fc7"
top: "fc8_mini"
type: INNER_PRODUCT
inner_product_param {
num_output: 100
}
blobs_lr: 1
blobs_lr: 1
}
layers {
name: "loss"
type: SOFTMAX_LOSS
bottom: "fc8_mini"
bottom: "label"
top: "loss/loss"
}
layers {
name: "accuracy/top1"
type: ACCURACY
bottom: "fc8_mini"
bottom: "label"
top: "accuracy@1"
include: { phase: TEST }
accuracy_param {
top_k: 1
}
}
layers {
name: "accuracy/top5"
type: ACCURACY
bottom: "fc8_mini"
bottom: "label"
top: "accuracy@5"
include: { phase: TEST }
accuracy_param {
top_k: 5
}
}
I don't know what is the problem
It is executable using caffe but I can't execute it using digits

Sorry which part of ERROR: Your deploy network is missing a Softmax layer! Read the documentation for custom networks and/or look at the standard networks for examples do you not understand?

What should I do to fix this error ? what is the correct syntax ? I'm already using softmax layer.

See for example:
https://github.com/NVIDIA/DIGITS/blob/digits-5.0/digits/standard-networks/caffe/lenet.prototxt#L169-L175


default codec for kafka is plain

Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/

I wonder if this sentence shouldn't just be removed since it duplicates information already in the same document.
Anyway, LGTM.


Additions and sort

added a few I've hit on in the past and sorted the list so it was easier to read and contribute to.

look who is on the board <3
merged


Multiple columns in Big Number Chart

Can we add multiple columns to big number chart ?

Thanks

Nope, you'd want to add multiple Big Number charts.
Or, if you want icons/emoji, you could probably get clever with the Handlebars plugin (though it's admittedly a bit beta-esque)


Display 'No problems' in the status bar when no errors and warnings

Fixes #51010

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: 1AmNegan sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@stevencl can you cast your vote on this. I am actually not a fan of changing this since it will make the status bar more unstable when moving from 0 -> n or from n -> 1.
Please assign back if you think this is something we should merge in.

@bpasero FYI.

I'm not convinced that this is better than what we have today, although I would like to learn more about what makes the current counter distracting when there are zero errors and warnings. The other problem is that this text would need to be localized.

Let's put on the UX agenda.

Closing this as it is not intended.


Update/Clarify our Documentation Guidelines

The documentation guidelines that have been transferred from the GitHub Wiki to the sphinx docs are outdated with our practices in some respects. In particular, @behackl has noted in #1137:

https://github.com/ManimCommunity/manim/pull/1137#discussion_r604287102
https://github.com/ManimCommunity/manim/pull/1137#discussion_r604287886
https://github.com/ManimCommunity/manim/pull/1137#discussion_r604289847

The clearer we can be on how we expect contributions regarding documentation, the easier reviews on documentation will go.

I think this can be closed now.


Feature api bugfixes

Description:

Small documentation cleanup
https://github.com/OpenTrons/opentrons/issues/292

Lowercase Pipette axis arg by default
https://github.com/OpenTrons/opentrons/issues/315

Check List:

[x] Are there breaking changes in the API and how are they being communicated?
No
[x] Who is reviewing your code?

Merged into #324


Hide chatbot access on mobile

Make sure the widget is hidden on mobile as well as the global navigation item

@mruwnik I think you wanted to take this but feel free to reassign to me if you don't


Fixes error | TypeError: string indices must be integers

Description

Motivation and Context

How has this been tested?

Screenshots (if appropriate)
Types of changes

Changes visible to users:

[ ] Bug fix (prefix: fix - non-breaking change which fixes an issue)
[ ] New feature (prefix: feat - non-breaking change which adds functionality)
[ ] Breaking change (prefix: feat!! or fix!! - fix or feature that would cause existing functionality to not work as expected)

Internal changes:

[ ] Refactor (prefix: refactor - non-breaking change which only improves the design or structure of existing code, and making no changes to its external behaviour)
[ ] Tests (prefix: test - additions and improvements to unit tests and the smoke tests)
[ ] Infrastructure (prefix: chore - examples include GitHub Actions, issue templates)

Checklist

[ ] My code follows the code style of this project and passes yarn run lint.
[ ] My change requires a change to the documentation.
[ ] My change works!

can close issue #833 after its been merged


App:dexDebug?

Hello,
First of all, thanks for this awesome lib, it's really hard to find something clean and flexible.
I create a whole web service api using this lib but, when I try to run the app, this happens:
**Error:Execution failed for task ':app:dexDebug'.

com.android.ide.common.process.ProcessException: org.gradle.process.internal.ExecException: Process 'command '/Library/Java/JavaVirtualMachines/jdk1.7.0_75.jdk/Contents/Home/bin/java'' finished with non-zero exit value 2**

I'm using Gradle to import the lib and, if I remove the dependency, the app works.
Do you have any idea?
Thank you!
Best wishes;

I have no issue with the latest version 1.1.3 on my android project. Can you try to clean and rebuild project and see whether it solves problem?

For some reason an unused library was causing a conflict. I just removed and it worked!
Thank you.

Alright! Thanks so much to using Fuel. I will close this as it is solved somehow.


Update the photo and video importers of Koofr and Backblaze to log bytes (#1195)

Update the photo and video importers of Koofr and Backblaze to log bytes transferred

Summary: Updated the photo and video importers of Koofr and Backblaze to log the number of bytes transferred

Updated

Fix formatting

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


train my datasets , IndexError: too many indices for array: array is 1-dimensional, but 2 were indexed

File "/home/ubuntu/EfficientDet.Pytorch-master/datasets/augmentation.py", line 113, in call
annots[:, :4] *= scale
IndexError: too many indices for array: array is 1-dimensional, but 2 were indexed

There are images without labels.


Login

Add a basic login page, which allows a user to log in - then creates a PHP session reflecting the fact that they are logged in. This will allow #3 to be ready, as it requires the user's login details to know where to save files.

The login details are stored on a MySQL database in the server's localhost. However, the database has not been set up with tables to store the data yet.

I have set up the users table. There are three columns:

INT AUTO_INCREMENT ID - A user ID.
TEXT Email - The user's email address, used as part of login process.
TEXT Password - The password used in conjunction with the email to log in.

Closed by #18


Incomplete and misleading information about configuring Pay-per-minute

Hi!
We are trying to configure audio conferencing pay-per-minute. This article states that "If you have Audio Conferencing pay-per-minute licenses, you don't have to assign Communications Credits licenses separately to each user specifically for Audio Conferencing usage". The article it leads to in point 3 https://docs.microsoft.com/en-us/microsoftteams/set-up-communications-credits-for-your-organization tells to add both communication credits license and audio conferencing license to user. This article https://docs.microsoft.com/en-us/microsoftteams/audio-conferencing-pay-per-minute states "Pay-per-minute requires your organization to have Communications Credits enabled with a license assigned to each user in order for Audio Conferencing to work." but it does not say which licenses.
When we try to setup audio conferencing for users in Teams admin panel https://docs.microsoft.com/en-us/MicrosoftTeams/manage-the-audio-conferencing-settings-for-a-user-in-teams then we do not see audio conferencing settings. When we try to add the communication credits to users in Office 365 license panel then we get error that we also need to assign audio conferencing license which we do not have as this is licensing not per user but per communication credits.

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

ID: 41c6c2e6-a1c7-dd0c-dbe3-0072497282d5
Version Independent ID: 1b09ca3d-f45e-5540-40b4-8c3b7176fc81
Content: Try or purchase Audio Conferencing in Office 365 for Microsoft Teams - Microsoft Teams
Content Source: Teams/try-or-purchase-audio-conferencing-in-office-365-for-teams.md
Service: msteams
GitHub Login: @tonysmit
Microsoft Alias: tonysmit

@Larryngolog Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@Larryngolog  Thank you for raising this issue! I'll be looking into it!

@tonysmit Being the page author for this piece of documentation, I feel you might have a more detailed knowledge about this issue that could prove to be helpful.

@TokyoScarab - I've been out of the content side for a while. I do probably have additional information, but things change very quickly. I've added the PM to see if he can help with this.
@ruizoscar - can you answer their question here please?

@ruizoscar - Any updates on this? Thanks

@ruizoscar - Any updates on this? Thanks

@ruizoscar - Any updates on this? Thanks

@ruizoscar - Any updates on this?  The world would like to understand this and it is completely unintelligible at the moment.  Clicking any link in the docs for the pricing for Teams calling leads you to the Teams main page which only states $4 for Audio Conferencing and no mention of Communications Credits.  I think it's time to be transparent on this item.  People don't mind paying if they know up front, but this is very confusing and unanswered for months.

@ruizoscar - Any updates on this? Thanks

I'm checking with the PMs on this. Please hold and I will report back here. @fafan @ruizoscar

@tonysmit Were you able to find anything from the PMs?

@TokyoScarab @Larryngolog - sorry about the long delay here. Honestly, I lost track of this issue. It's true that Communication Credits are required for Pay-per-minute usage scenarios. We need to update this doc to align correctly.
Communication Credits is a separate license (requires no payment) that must be assigned to any AC user to support these scenarios:

User has an Audio Conferencing license assigned. Communication Credits are required for a meeting participant to dial out to someone else and have them join the meeting.
User has a Pay-per-minute license. Communication Credits are required for this because it's how we charge customers for this service.
Communication Credits are required when you have a toll free phone number that has been ported over or purchased by Microsoft for an organization.

So for Pay-per-minute scenarios, you must do the following:

Create a conferencing bridge with a phone number for dial-in (or use a shared number). Note - if you have a toll free number, Communications credits must be set up but you don't have to assign any licenses to users.
Set up Communication Credits for your org. Add credit card details and number of minutes. Also can set thresholds.
Get the Pay-per-minute licenses from an account rep, and assign this license to any user that will be using PPM.
Assign a Communications Credits license to the same user that will be using PPM.

Update: I updated the content below to reflect the issues you have above:
https://docs.microsoft.com/en-us/microsoftteams/audio-conferencing-pay-per-minute
https://docs.microsoft.com/en-us/microsoftteams/try-or-purchase-audio-conferencing-in-office-365-for-teams
These changes should be live in a few hours.


refactor(go/adbc/driver): add driver framework

Fixes #996.

@zeroshade, how does the direction here look?

I think what I'll have to do is invert things: make driverbase.NewDatabase also take an interface as an argument, where the interface is all the driver-specific code, and then implement the ADBC methods inside driverbase (delegating/forwarding to the supplied interface as needed)

Either that, or continue down this path, with the vendor-specific side delegating calls to the generic side, and moving common state into the generic structs as I find it.

The problem with both versions is that there is shared state and delegation, e.g. if we go with the take-an-interface version, we need to provide the interface methods with state like the allocator and the logger still. But I think I'm still going to go that way.

@zeroshade this is ready. It does have a breaking API change (driver structs are now hidden in favor of a NewDriver constructor). I didn't refactor connection/statement which is where I expect the most changes to be made, this is just to set up the framework.


PCIe x1 connector

請問圈起來的P & N pin 是原廠線路否有畫錯筆誤

正確

Sorry, not something I can answer as the Pi4J project doesn't provide PCIE access.

抱歉，我無法回答，因為 Pi4J 項目不提供 PCIE 訪問。

sorry, I don't understand what you mean, I want to express whether the original circuit has drawn P(+)&N(-) backwards,THX

Sorry, not something I can answer as the Pi4J project doesn't provide PCIE access.

Sorry, not something I can answer as the Pi4J project doesn't provide PCIE access.

sorry, I don't understand what you mean, I want to express whether the original circuit has drawn P(+)&N(-) backwards,THX

Sorry, not something I can answer as the Pi4J project doesn't provide PCIE access.

sorry, I don't understand what you mean, I want to express whether the original circuit has drawn P(+)&N(-) backwards,THX

I think this is a question that will get a better answer on https://raspberrypi.stackexchange.com/


"Authorize Twitter" button does authorization and then nothing

"Authorize Twitter" button does authorization, it redirects to Twitter and I click "Authorize App", but then I get back to the same page and nothing changed - including the "Authorize Twitter" button which still sits there. When I click it now, however, nothing happens. Did I do the "Publish your handle" part wrong? (What exactly does that even mean, though? A link to the mastodon page, or @user@host, or what exactly will work? That seems to be a bit unclear.) Did the authorization itself even work? In overall, the clearness of the instructions and more importantly the feedback on which step I did correctly or didn't yet, seems a bit lacking.

I'm seeing the same thing. The Authorize Twitter button seems to work but when I return nothing happens. Fedifinder worked fine a couple of weeks ago and the old version still works for me. Tried it in Edge and Firefox, same result in both browsers.


Got command error in LunarVim

Hi, I used the default configs provided and when run, got the following error.

I selected some text in visual mode and ran Silicon command, that's how I got the error.
Am I missing something? 🤔

Hi there,
yes, you are missing the prerequisite silicon binary, which needs to be installed on your system and in your $PATH. The upstream project is here: https://github.com/Aloxaf/silicon
But you can install it on a mac with brew install silicon or if you have installed rust, the compilation and installation is very easy as well: cargo install silicon.
Maybe I should emphasize in the readme how the tool can be installed...
Could you try to install silicon on your system and tell me how it went?
Thanks a lot!
Michael.

It worked, however I had to remove NF from font = "VictorMono NF=34;Noto Emoji=34" since I couldn't find VictorMono font with NF postfix. Maybe I am missing something.

Maybe that is enough for me, therefore I would close this issue. Feel free to reopen, if you need more details from me.

Thank you! Yes, the font setting was merely an example, silicon would still fallback to some other builtin font. I guess I did not put too much emphasis on constructing a typeface recommendation, because I think every terminal user has a strong opinion what the perfect font/typeface is and in the end in 90% of the time, the font setting would be overridden by users 😏
The NF stands for NerdFont. That is a project that embeds a lot of special unicode characters in normal fonts, so that neovim and plugins look nice. You can download many prepatched fonts from their website: https://www.nerdfonts.com/font-downloads


Wip

Description:

Changes:

Other checks:

[ ] I have tested all my changes thoroughly.

I see absolutely nothing wrong with this PR, carry on king

I made an oopsie


[BUG] bulk copy has lost support for spatial columns (geometry/geography)

Support for spatial columns was added (for us, thank you!) in https://github.com/Microsoft/mssql-jdbc/issues/533
Sadly it appears that when https://github.com/Microsoft/mssql-jdbc/issues/655 was fixed (also for us - I have regrets) by https://github.com/Microsoft/mssql-jdbc/pull/657 it killed the Bulk Copy support for spatial columns (we believe because VARBINARY columns are supported by Bulk Copy but spatial columns are not).
Creating repro sample code will take at least a little effort; I'll hold off on that until you say you need it.  Hopefully it's pretty straightforward!

Hi @ravenAtSafe, thanks for reporting this issue. The team is aware that geometry/geography datatypes are currently not supported for bulk copy. We will investigate if we can support this feature again without breaking the existing functionalities and let you know.

Thanks @peterbae; we will stick with the preview driver for now.

Is there any scope for this enhancement in the near future?

For what it's worth we ultimately upgraded to the latest driver and were able to convince bulk copy to handle geometry with only minimal tweaks - it looks like we just claim we are working with VARBINARY and all is well.

@karan-shell this enhancement is in our backlog it will be considered along with other bugs and features when we plan for the next semester. We take into consideration the severity of the bug and community demand for the feature when we do the triage so please upvote anything that you would like to see added to the driver.
Thanks


Flash timeout

Hi,
Espflash works fine when I am trying to program Default Uart pins on esp32s3 with cp2104 but When I use the Jtag I get timeout error. Does esp flash support this?

Yes it does support usb serial JTAG. I use it everyday, so it should work. Could you give us more details about your environment, package versions and finally the full output from espflash, thanks!

Hardware : USB_N --> GPIO19
USB_P --> GPIO20
My Command:
export ESP_ARCH=xtensa-esp32s3-espidf
espflash /dev/ttyACM0 target/${ESP_ARCH}/${BUILD_MODE}/ee
ERROR:
Serial port: /dev/ttyACM0
Connecting...
^[Chip type:         ESP32-S3
Crystal frequency: 40MHz
Flash size:        8MB
Features:          WiFi, BLE
MAC address:       f4:12:fa:44:a8:10
Error: espflash::timeout
× Communication error while flashing device
╰─▶ Timeout while running WriteReg command

Duplicate of https://github.com/esp-rs/espflash/issues/171. Please install espflash from master until the next release. cargo install espflash --git https://github.com/esp-rs/espflash should do the trick.

thanks for your response. It would be my pleasure to help to develop more feature on this great app.
Best Regards
@MabezDev


cmd/go: go build *.go does not account for file suffixes

go version go1.7 darwin/amd64
$ ls
lib.go lib_darwin.go lib_linux.go
$ head -1 lib_darwin.go
package lib # no build tags
$ head -1 lib_linux.go
package lib # no build tags
$ go build *.go
#some error about same function defined twice
$ go build && echo $?
0

This is working as expected.  If you explicitly list the files to build, the go tool builds exactly those files.  Build constraints only apply when looking at an entire package in a directory.


Inconsistency between DateField model field and the fieldpanel in the admin interface

Hi guys,
Just noticed a minor bug with the ways dates are formatted. In one of my page models I have a date field declared as:
...
date = models.DateField("Post date", default=date.today())
..
BlogPage.content_panels = [
    FieldPanel('title', classname="full title"),
    FieldPanel('date'),
    ...

When I add a new BlogPage through the admin interface:

The default date is properly set but with the wrong formatting (uses / instead of - characters). Because the page-editor.js uses the format format: 'Y-m-d' when working with the datetimepicker, the value is not recognised.
As a workaround I have to override the init method to something like:
def __init__(self, *args, **kwargs):
    super(BlogPage, self).__init__(*args, **kwargs)
    self.date = self.date.strftime("%Y-%m-%d")
```

Not sure it's due to my config or wagtail. Maybe someone else's with the same problem?

Cheers!

Oh, I think with the changes you've made to https://github.com/torchbox/wagtail/blob/master/wagtail/wagtailadmin/widgets.py should be solved as you  force the widget date input with:
class AdminDateInput(WidgetWithScript, widgets.DateInput):
    # Set a default date format to match the one that our JS date picker expects -
    # it can still be overridden explicitly, but this way it won't be affected by
    # the DATE_INPUT_FORMATS setting
    def __init__(self, attrs=None, format='%Y-%m-%d'):

Will wait for the next release then :)

Yep, this is (hopefully...) covered by #1018.

@jordij - as a side note, I think you have a bug here:
date = models.DateField("Post date", default=date.today())

You probably want:
date = models.DateField("Post date", default=date.today)

Python defaults are evaluated exactly once, so the default value in this case will end up being the date on which you started the CMS, but if it runs for a year, it will still be that date. See http://effbot.org/zone/default-values.htm
Django helps you out with this by detecting if the argument to default is callable (function, lambda, static method). See https://docs.djangoproject.com/en/1.7/ref/models/fields/#default

It's all good, thanks!


Error in Ymatrix when building model from RAW file

Hi, I have a 4-bus test system with an off-nominal tap position transformer. The resulting Ymatrix is wrong. This occurs when I use a raw (PSSE) file. However, when I build the same system with Matpower, I have no problems. Please see attached.

TVC_SystemMATPOWER.csv
TVC_SystemRAWPSSE.csv

It looks like the data has a problem, the r and x values for the tap transformer "BUS4-BUS3-i_4" are 0.0. I am not sure how PSS/e or Matpower handles these cases internally but we don't support it.

I added some protection to detect these cases just in case in PR #854

This transformer has a reactance value different from zero in the sav file. Something is wrong when PSSE converts from sav to raw file.

@tavovalmo1983 The issue is that in the data set the base voltage of the transformer is set to 0.0 and the parser does a per-unit transform of the transformer from its base to the system base. Since the base voltage is set at 0.0, the transformation results in a base impedance of 0.0. I opened another PR to catch this issue.


show amount of completed items

if you have time/feel like it, I would be very pleased if a counter could be added for how many of the items you already have
This is useful to know if I should build more of (especially) the intermediate types or if I  can burn some 'needed' recipes
(like if I have 6 frost striders I might want to burn a few to make space in the inventory, or if I only have 1 invulnerable I might want to already make a second)
Any way great tool, and a big improvement over the native one. i was actually skipping the mechanic before I found this tool so thanks a lot :1st_place_medal:

Clicking the PRIO-label opens a surplus list which should also calculate the surplus in intermediate mods (please tell me if it doesn't). To balance out branches, you can hold-click a recipe in the prio-list to see a tree-view with the quantities of base and intermediate mods.
If a different kind of list/counter could be beneficial, please let me know and also specify. I need to know how a feature should work and could be helpful/useful before I implement it (I don't always understand it from a concept alone, I need to 'see' the train of thought).

I saw what yoou mean and that is nice i think a counter IN the prio list would be nice though (so you can see at a glance how many of the prio ones you have ready and wether you can 'waste' some while building up to a larger combination
(i mean the green ones in the top)

Alright, there'll be an update in an hour or so that revamps the UI a bit and covers what you suggested. Just needs a little bit more testing. Thanks for your feedback.


BitSet.getAndSet() should expand the bitset on demand [HPPC-86]

Issue: HPPC-86 (migrated from JIRA), created by Dawid Weiss (@dweiss), resolved Apr 08 2014

Comment by Dawid Weiss (@dweiss) (migrated from JIRA)
Doesn't make sense to cater for this one.

Comment by Stanisław Osiński (@stanislawosinski) (migrated from JIRA)
I actually had this use case quite a few times, usually in tight loops. I'm not sure what the speed gain would be though.

Comment by Dawid Weiss (@dweiss) (migrated from JIRA)
If you do, call:
    /**
     * Ensure that the long[] is big enough to hold numBits, expanding it if necessary.
     * getNumWords() is unchanged by this call.
     */
    public void ensureCapacity(long numBits)

in your loop. It's what getAndSet would have to do anyway, so embedding it inside getAndSet is not going to be any improvement (and it's going to be a perf. penalty for those not needing it).


init method does not responds in cordova android method

The init method does not work properly it executes properly for one application and does not respond for another application within same environment
Following warning / Error / info messages were found in LogCat
Could not find method com.facebook.Session.getAccessToken, referenced from method com.ccsoft.plugin.CordovaFacebook.prepareAccessTokenInfo
DexOpt: unable to optimize static field ref 0x0745 at 0x304 in Lcom/ccsoft/plugin/CordovaFacebook;.execute
I/dalvikvm(12811): DexOpt: unable to optimize static field ref 0x0743 at 0x310 in Lcom/ccsoft/plugin/CordovaFacebook;.execute
I/dalvikvm(12811): Failed resolving Lcom/ccsoft/plugin/CordovaFacebook$16; interface 973 'Lcom/facebook/Request$Callback;'

You are either using an unsupported version of Facebook SDK or not included Facebook SDK properly in your project. Most likely the latter.
The error seem to say the plugin cannot find a method from Facebook library.


Change width in styles.css

Could you please add this change when you are committing something else?
In styles.css please change the numeric value in l.  273 from 300 to 750.
.metadata { 	width: 300px; } to  .metadata { 	width: 750px; }

done with d6b95da


disable plugin roadiehq-scaffolder-backend-module-utils-dynamic

Description
Disable plugin roadiehq-scaffolder-backend-module-utils-dynamic by default
Which issue(s) does this PR fix

Fixes #801

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

There's no reason for this plugin to be disabled by default, since it doesn't require any configuration. We should search the reason of the error reported in the corresponding issue somewhere else.

It must be either a very recent change, or some misconfiguration of in the environment (helm-chart / image inconsistency ?)

cc @tumido

Got past this error - turns out I was setting the runAsUser manually to 1001 and so preventing my own access to the file, see https://github.com/janus-idp/backstage-showcase/issues/801#issuecomment-1820755428.
Thank you for your help!


Procure upgraded Acquia cloud account

Description
Definition of done

[ ]
[ ]
[ ]

Attachments
title

Dupe of #21 on scrum board


Xprize finals

[ ] to be rebased on top of https://github.com/robotology/walking-controllers/pull/117

These commits have been already ported in master
https://github.com/robotology/walking-controllers/commit/f2eaceed6081df94b39264f9575c7b0ebced2c14
https://github.com/robotology/walking-controllers/commit/43586c7efd7e1295ba5b3be22be4f7710725c2c3
closing


SPRT-126: Implement the language switcher in the pukara.es website

@gnaomi0 I have some considerations:

The base url cannot change, because is part of the github pages configuration
Okay. Sorry, I didn't know.

I don't understand why you are exposing new ports in docker-compose file for this implementation
If I don't expose other ports, I cant see the english and portuguese version. Now I changed the url again and if the ports 1314 and 1315 are not declared it returns an error. Is this okay?

The implementation doesn't accomplish the task criterias (Ex.: the switcher must be a flag...)
I'll make some changes.

When I open locally (localhost:1333) , I got an error page doesn't exists. By default the page is in Spanish, so we don't need specify /es
Okay.

@eddymdz  check my last comments here and jira.

@eddymdz let me know if we need some adjustments


Update flag-icon-css to flag-icons library.

flag-icon-css library is now deprecated and changed to flag-icons.
Please update this.

Posted PR at https://github.com/azzamasghar1/ion-intl-tel-input/pull/36

@azzamasghar1


Mark version param as deprecated in server.pp comments

Thanks @binford2k !

Whoops! I totally meant to push this branch to my own fork, lol!


Add Stable Diffusion 3.5 Large bf16 support

Feature Idea
I downloaded Stable Diffusion 3.5 Large in bf16 https://civitai.com/models/882666/sd35-large-google-flan with builtin clips.
I edited metadata of model to Stable Diffusion 3.5.
Try to put to unet, Stable diffusion, diff
backend #0 failed to load model with error: Model loader for sd35LargeGoogleFLAN_large3CLIPFLANBF16.safetensors didn't work - are you sure it has an architecture ID set properly?
In command line I also see nonsence message
[Warning] Your system memory usage exceeded 98.2% just before the backend process failed. This might indicate a memory overload.
 [Warning] You appear to have a sufficient pagefile (96.9 GiB), so you might have too many background processes, or you might just be trying to run too much.
 [Warning] Consider closing background processes, or greatly expanding your pagefile size.
[Warning] Or, reduce the size of what you're trying to run. If you're running an FP16 or FP8 model, consider a quantized variant like GGUF Q4

I never had 96.9 GiB pagefile. Moreover, creator of checkpoint writes "The Full BF16 model runs at an amazing speed even on my 8GB card". My card is RTX 4070. 32 GB RAM. 10000 Megabytes pagefile.
Other
Windows 11 24H2

The pagefile size detection I think is wrong (something weird in how the windows api does virtual memory). The memory usage error message regardless only appears if the process crashed, which is usually genuinely a memory error if there's no other error (memory overload errors for some reason fail silently, which is why I have the memory check and warning there). If there's a different error above it, that earlier error is the real problem.
Moreover, creator of checkpoint writes "The Full BF16 model runs at an amazing speed even on my 8GB card"
This person is silly.
SD 3.5 large is an 8B model, and BF16 is 16 bits aka 2 bytes per parameter, ie 8 times 2 is 16 billion bytes, ie 16 gigabytes of VRAM to fit the model at all (plus more space needed for activation data and all). The model does not fit in VRAM and must be offloaded to system memory, which means it can't run nearly as fast as a model that fits in VRAM.
(Also the whole google flan insertion is a bit silly but that's a separate topic)
An FP8 version of the SD35Large model will offload too, but much less. A GGUF could fit entirely within your card, and is the best bet.
For models with multiple components built-in (Textencoders, vae, ...) you want Models/Stable-Diffusion
Also the architecture ID error means it didn't have an architecture ID in the model and wasn't able to autodetect it (possibly because they did weird things to the model format there). You can click the ☰ menu next to the model in the Models browser and click "Edit Metadata" and set the architecture ID to the correct value.


[1.20.1] adding drawn to controller bug

so ive noticed a bug in the latest version of AMT9 with adding draws to a controller if i have the linking tool in multi mode when i select an area it changes the mode to remove draw and wont allow me to add multiple draws at once i have to change it to single select and add them that way

Adding to this since it may be related and with the same mod ATM9 mod pack.
I moved the controller, re-linked the tool but every time I try to link a drawer in multi or single it just says "Removed drawer from the controller". Can not seem to re link any thing now.

Can you change the mode by right-clicking in the air with the linking tool?

There is probably a mod messing with this interaction and it will be impossible for me to find


Use of Javascript

Issue Type: Performance Issue
I am trying to run my Javascript code as if it was in the browser console. However it is not performing the same, and I would like to know why. With-in the "Chome developer tool" my code performs as intended. But when ran in Visual Studio is reads errors, I want to be able to output funcations as the console would.
VS Code version: Code 1.43.1 (fe22a9645b44368865c0ba92e2fb881ff1afce94, 2020-03-18T07:01:20.184Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
AMD FX(tm)-4300 Quad-Core Processor             (4 x 3817)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.99GB (4.14GB free)

Process Argv

Screen Reader
no

VM
0%

Workspace Info
|  Window (JS-dev.js - Untitled (Workspace) - Visual Studio Code)
|    Folder (TheDevStar): 7 files
|      File types: pdf(2) js(1) css(1) html(1) jpg(1)
|      Conf files:;

PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar> n
n : The term 'n' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify
At line:1 char:1

n
~

CategoryInfo          : ObjectNotFound: (n:String) [], CommandNotFoundException
FullyQualifiedErrorId : CommandNotFoundException

PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar> function bmiCalculator (weight, height) {

return interpretation;

}
At line:1 char:25

function bmiCalculator (weight, height) {

                    ~

Missing ')' in function parameter list.
At line:1 char:39

function bmiCalculator (weight, height) {

                                  ~

Unexpected token ')' in expression or statement.
+ CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : MissingEndParenthesisInFunctionParameterList
At line:1 char:1

var weight = 86.1826;

The 'var' keyword is not supported in this version of the language.
+ CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : ReservedKeywordNotAllowed

The 'var' keyword is not supported in this version of the language.
+ CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : ReservedKeywordNotAllowed
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
At line:1 char:1

var n = weight / (height * height);

The 'var' keyword is not supported in this version of the language.
+ CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
+ FullyQualifiedErrorId : ReservedKeywordNotAllowed
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
n : The term 'n' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify
that the path is correct and try again.
At line:1 char:16

n = Math.round(n * 10) / 10;

CategoryInfo          : ObjectNotFound: (n:String) [], CommandNotFoundException
FullyQualifiedErrorId : CommandNotFoundException

PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar> if (n <= 18.4) {

}
At line:1 char:7

if (n <= 18.4) {

  ~

CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
FullyQualifiedErrorId : RedirectionNotSupported

PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar> if (n >= 18.5 && n <= 24.9) {

n = (n + " normal weight")

}
The token '&&' is not a valid statement separator in this version.
At line:1 char:20

if (n >= 18.5 && n <= 24.9) {

               ~

CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
FullyQualifiedErrorId : InvalidEndOfLine

PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar> if (n >= 25 && n <= 29.9) {

n = (n + "over weight")

}
The token '&&' is not a valid statement separator in this version.
At line:1 char:18

if (n >= 25 && n <= 29.9) {

             ~

CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
FullyQualifiedErrorId : InvalidEndOfLine

PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar> if (n >= 30 && n <= 34.9) {

n = (n + " obese")

}
The token '&&' is not a valid statement separator in this version.
At line:1 char:18

if (n >= 30 && n <= 34.9) {

             ~

CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
FullyQualifiedErrorId : InvalidEndOfLine

PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar>
PS C:\Users\The core unit\Desktop\Code file storage\TheDevStar> if (n >= 35) {

}
n : The term 'n' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify
that the path is correct and try again.
At line:1 char:5

if (n >= 35) {

~

CategoryInfo          : ObjectNotFound: (n:String) [], CommandNotFoundException
FullyQualifiedErrorId : CommandNotFoundException


Drone does not move in Flightmare simulator

Hello! Thank you and your team for your project!
I have faced a problem with the following thing. When I start a simulation with python3 main_ros.py --config_file config/config_flightmare.yaml with Flightmare launched before in other terminal by authors' command roslaunch flightros rotors_gazebo.launch the drone moves to the initial pose with height mentioned in config. And that's it. It does not plan any trajectory.
Can you give an advise how to fix it?
Btw, I tried to download your binary files by curl -LO https://phenoroam.phenorob.de/geonetwork/srv/api/records/be36e0bc-bb15-4e7c-b341-5754670ff29c/attachments/RPG_Flightmare.zip and it crushes every time in 2-3 seconds.
With Potsdam and RIT18 everything works fine.

same here. I think the authors are not maintaining the repo anymore. since the provided link for the RPG_Flightmare.zip is now blocked, I have downloaded the original zip file from the flightmare github. However, this seems to not have the proper sensor topics since the code stucks at "data_rgb = rospy.wait_for_message("/rgb", Image, timeout=None)" and "rostopic echo /rgb" outputs "WARNING: no messages received and simulated time is active. Is /clock being published?" error.
I assume the modification the authors made in the custom zip-file was crucial for running this repo.

It turns out we have to modify the camera settings in the original cpp files to publish ros topics of '/rgb','/segmentation' by ourselves. you can refer to the 'camera' example code. However, even though I've managed to run the code, the FOV of the camera is still unknown but this I think I can conduct some trial and error to find the sweet spot

Hello! Thanks for your advice!


Implement mechanism to be able to halt the bridge

Being able to halt to bridge at consensus level could be useful in case we find some issues

solve by https://github.com/crypto-org-chain/gravity-bridge/pulls?q=is%3Apr+is%3Aclosed


Release 0.14.4

Checklist

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

I updated the requirements in accordance with the source repo.

@conda-forge/gluonts this is now ready for review


Can we restrict result to specific instead of generic ?

*Can we restrict result to specific instead of generic ? **
For example in Austrailia if we are looking for an address it should only show addresses with sepecific name or number
Burton Street, Cannington WA, Australia    //this is a generic address user don't know where this position actually is so we need to identify a street number as well or a places name on this street . map should not allow to select this kind of specific address.
it should allow addresses like below.
12 Burton Street, Cannington WA, Australia
or
abc police station, Burton Street, Cannington WA, Australia
or
ABC Coffee Shop, Burton Street, Cannington WA, Australia
if some has worked on this or any success story kindly share your code example . TIA

As per the Google Autocomplete docs, you can't specify more than one type in a query.
You can try and filter the results manually in renderRow depending on your requirements.


Revert "OCPCLOUD-2623: Merge https://github.com/kubernetes-sigs/cluster-api-provider-vsphere:v1.10.0 (41eeb09) into master"

Reverts openshift/cluster-api-provider-vsphere#40

/lgtm
/approve

/tide refresh

/override ci/prow/e2e-vsphere
/override ci/prow/e2e-vsphere-serial
/override ci/prow/e2e-vsphere-techpreview
/override ci/prow/regression-clusterinfra-vsphere-ipi-techpreview
Because this was a mistaken merge and a clean revert.


Revert "feat: mStable template"

This reverts commit 3438cf7997b5ef78edf811a4e9620eae7c418820.

Hi @tota999, this is the 3rd ticket that you're opening without explaining yourself. I will proceed with contacting GitHub support.


docs: fix reference for the jobName in a matrix

Fix the example since jobName is the one with the matrix values.
See https://github.com/v1v/otel-upload-test-artifact-action/blob/fc344a1ce206d702f748ae1418d0b426ae582c30/.github/workflows/code-quality-pr-check.yml#L46-L47

LGMT


[Help]: The converted sound goes to two extremes

I am doing voice conversion task, and the converted voice is not quite right. If the target is a male voice, the converted voice is all very low "cow", and if the target is a female voice, the converted voice is all relatively sharp. I think it is the problem of network structure, so may I ask how to deal with it.

已解决


[New Sniff] Make sure widget_title filter has all parameters

One thing I often run across is the following in custom widgets.
$title = apply_filters( 'widget_title', $title );

There's 2 missing parameters there.  Ref: https://developer.wordpress.org/reference/hooks/widget_title/
When themes call this, it should be like so:
$title = apply_filters( 'widget_title', $title, $instance, $id_base );

They need to make sure that all 3 parameters are correct and passed to filters.  Otherwise, it breaks plugins that rely on those 2nd and 3rd parameters.

This issue raises a number of questions from an implementation point of view:

Is this the only filter where a check like this would need to take place or are there / will there be more actions/filters where the parameters will need to be checked ?
Does the check need to be for the amount of parameters passed or also need to go into the content ? (If the answer would be 'content', that would be neigh impossible as variables can be named differently across themes.)

Questions from an implementation perspective:

Do we need to create a re-usable abstract class which handles a large part of the logic and can be extended for individual actions/filters or small groups thereof ?
If this will be the only one, we can just use a hard-coded check, if not, we may need to have an extensible array property listing the hooks or in the case of the abstract, possibly a function instead.

Also: is there a rule in the handbook covering this or should it be added ?

Is this the only filter where a check like this would need to take place or are there / will there be more actions/filters where the parameters will need to be checked ?

Not sure if there are others at the moment.

Does the check need to be for the amount of parameters passed or also need to go into the content ? (If the answer would be 'content', that would be neigh impossible as variables can be named differently across themes.)

Amount is doable and would make theme devs aware of the issue.

Do we need to create a re-usable abstract class which handles a large part of the logic and can be extended for individual actions/filters or small groups thereof ?
If this will be the only one, we can just use a hard-coded check, if not, we may need to have an extensible array property listing the hooks or in the case of the abstract, possibly a function instead.

There's few cases of themes needing to call core WP hooks like this.  It's really a poor implementation on core's part to be honest.
I really can't think of another example off the top of my head, but I'll leave it open to others.

Also: is there a rule in the handbook covering this or should it be added ?

We don't currently have a specific rule to this specific case.  It falls under the more general "Use template tags and action/filter hooks properly" guideline.  See: https://make.wordpress.org/themes/handbook/review/required/#core-functionality-and-features

If custom widgets are no longer allowed this would be a non issue?

Perhaps, but we haven't gotten rid of custom widgets yet. My theme in particular relies on my custom widgets for its flexibility.

We could maybe add a warning sniff about possible depreciation of custom widgets, but I guess this depends on how the gutenberg widget thing will progress...

We need to see how relevant this issue is in the context of widgets being moved to core blocks.
Also, we got the info about unused parameters in actions and hooks in general, but that sounds more like a WPCS issue to me.
Thoughts @jrfnl?

Also, we got the info about unused parameters in actions and hooks in general, but that sounds more like a WPCS issue to me.

I agree. I think someone should probably check if there is a WPCS issue open for this, and if not, open one.
I imagine it would be good to have one generic sniff which checks that if any of the WP core hooks are invoked in a plugin/theme (or in core itself) that the amount of parameters passed is the same as in the original hook definition used in Core.
This would be a horrific sniff to maintain though unless we also create a second (non-public) sniff to scan core on each release which would update the (public) sniff, see https://github.com/WordPress/WordPress-Coding-Standards/issues/1803 for more information about that principle.

I like that idea, especially if we would have the extra utility sniff in the linked issue 🙂 That would help with the maintenance. I'll open up an issue upstream, and close this one 👍


Apply a recursive chown on data folder

To be able mount and custom scripts and folders inside the data folder.
#78

@the-lazy-fox

To be able mount and custom scripts and folders inside the data folder.

Recursive chown will take too much time, that's why we only chown the base folder. You have to chown those files yourself as stipulated in the README:

Note that the volumes should be owned by the user/group with the specified PUID and PGID. If you don't give the volumes correct permissions, the container may not start.


Simplified Transaction Context

Adds implementation and unit tests for a simplified transaction context and a new Addresser trait.
Came from here: https://github.com/hyperledger/sawtooth-sabre/pull/87, now following the correct process through the Cargill Sabre fork. All comments had been addressed from the requested changes before closing.

This work is being moved over to the Transact repository. Here is the initial pr for that move: https://github.com/hyperledger/transact/pull/27


Default behavior of tempfile.mkdtemp(suffix=None, prefix=None, dir=None) changed in Python 3.12

Description
We use a module to create Lambda layers built in Docker. Recently, I tried changing the pip_tmp_dir to a path relative to the module (${path.module}) to avoid constant discrepancies in the plan between my local environment and our CI/CD pipeline. I noticed differences between the local run and the CI/CD pipeline run.
On my computer, the Python version is 3.12, but our pipelines use 3.10. I found that there was a change in Python 3.12: https://docs.python.org/3/library/tempfile.html#tempfile.mkdtemp
It states: "Changed in version 3.12: mkdtemp() now always returns an absolute path, even if dir is relative."
This change is beneficial for us as it allows the use of a relative path for pip_tmp_dir, thus avoiding constant changes in the plan due to ${path.cwd} differences.
Could you suggest a way to achieve the same behavior with older Python versions? Additionally, is there any concern about always returning the absolute path in this file: https://github.com/terraform-aws-modules/terraform-aws-lambda/blob/master/package.py#L123?
If your request is for a new feature, please use the Feature request template.

[x] ✋ I have searched the open/closed issues and my issue is not listed.

⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Module version [Required]: 7.7.0

Terraform version: 1.9.2

Provider version(s):

provider registry.terraform.io/hashicorp/aws v5.57.0
provider registry.terraform.io/hashicorp/external v2.3.3
provider registry.terraform.io/hashicorp/local v2.5.1
provider registry.terraform.io/hashicorp/null v3.2.2

Reproduction Code [Required]
module "s3fileloader_lambda_function_layer" {
  source  = "terraform-aws-modules/lambda/aws"
  version = "7.7.0"

  create = local.create_s3fileloader

  create_function = false
  create_package  = true
  create_layer    = true

  layer_name  = "${local.lambda_name}-pip-requirements"
  description = "Layer for supporting 's3fileloader' lambda function"

  compatible_runtimes = ["python3.12"]
  runtime             = "python3.12"
  artifacts_dir       = "${path.root}/builds/lambda_layers/"

  build_in_docker = true
  docker_additional_options = [
    "--network", "host"
  ]

  source_path = [
    {
      path             = "${path.module}/s3fileloader"
      pip_tmp_dir      = "${path.module}/s3fileloader"
      prefix_in_zip    = "python"
      pip_requirements = true
    }
  ]

  store_on_s3               = true
  s3_prefix                 = "lambda_layers/"
  s3_object_storage_class   = "STANDARD"
  s3_bucket                 = module.s3fileloader_lambda_function_sources.s3_bucket_id
  s3_server_side_encryption = "aws:kms"
  s3_object_tags            = local.tags

  recreate_missing_package     = false
  trigger_on_package_timestamp = false

  tags = local.tags
}

Steps to reproduce the behavior:
Run the module definition above with python3.10 and python3.12.

Expected behavior
The Lambda layer was created correctly.

Actual behavior
Python 3.10:
The script fails here, presumably due to the relative path: https://github.com/terraform-aws-modules/terraform-aws-lambda/blob/master/package.py#L1108
[2024-07-11T09:21:27.996Z] module.s3fileloader_lambda_function_layer.null_resource.archive[0] (local-exec): FileNotFoundError: [Errno 2] No such file or directory: './s3fileloader/terraform-aws-lambda-4jrqazb4/requirements.txt'
[2024-07-11T09:21:21.436Z] module.s3fileloader_lambda_function_layer.null_resource.archive[0] (local-exec): zip: creating './builds/lambda_layers/50440c451250a62316026ec292815a320d9bbca0100cf05b80febf54fd137adc.zip' archive
[2024-07-11T09:21:21.436Z] module.s3fileloader_lambda_function_layer.null_resource.archive[0] (local-exec): Installing python requirements: ./s3fileloader/requirements.txt
[2024-07-11T09:21:21.436Z] module.s3fileloader_lambda_function_layer.null_resource.archive[0] (local-exec): > mktemp -d terraform-aws-lambda-XXXXXXXX # ./s3fileloader/terraform-aws-lambda-4jrqazb4
[2024-07-11T09:21:21.436Z] module.s3fileloader_lambda_function_layer.null_resource.archive[0] (local-exec): > cd ./s3fileloader/terraform-aws-lambda-4jrqazb4

Python 3.12:
The script does not fail.
module.s3fileloader_lambda_function_layer.null_resource.archive[0] (local-exec): zip: creating './builds/lambda_layers/50440c451250a62316026ec292815a320d9bbca0100cf05b80febf54fd137adc.zip' archive
module.s3fileloader_lambda_function_layer.null_resource.archive[0] (local-exec): Installing python requirements: ./s3fileloader/requirements.txt
module.s3fileloader_lambda_function_layer.null_resource.archive[0] (local-exec): > mktemp -d terraform-aws-lambda-XXXXXXXX # /home/user/src/bitbucket/company/Terraform-AWS/tf-scripts/.terragrunt-cache/J5cPGP4LIGaljTfIVUC_seYg5ek/37ujJTU5vwGX_XeoGQN7WeHvNDI/s3fileloader/terraform-aws-lambda-79x0pniy
module.s3fileloader_lambda_function_layer.null_resource.archive[0] (local-exec): > cd /home/user/src/bitbucket/company/Terraform-AWS/tf-scripts/.terragrunt-cache/J5cPGP4LIGaljTfIVUC_seYg5ek/37ujJTU5vwGX_XeoGQN7WeHvNDI/s3fileloader/terraform-aws-lambda-79x0pniy

Terminal Output Screenshot(s)

Additional context

Hi,
interesting finding about tempfile.mkdtemp() behavior change in python 3.12!
A few findings below:

This change is beneficial for us as it allows the use of a relative path for pip_tmp_dir, thus avoiding constant changes in the plan due to ${path.cwd} differences.

I do not believe pip_tmp_dir is taken into account when computing the hash of the lambda content, is it?
However, one should probably avoid to point pip_tmp_dir to the same location as the source path like in your reproduction case, as it will probably include temporary artifacts in the output, and thus alter the hash value.
Also, if pip_tmp_dir is not set at all, I believe the path is normally already absolute with all python versions, e.g. /tmp/terraform-aws-lambda-xxxxx on Linux:

If dir is not None, the file will be created in that directory; otherwise, a default directory is used. The default directory is chosen from a platform-dependent list, but the user of the application can control the directory location by setting the TMPDIR, TEMP or TMP environment variables. There is thus no guarantee that the generated filename will have any nice properties, such as not requiring quoting when passed to external commands via os.popen().

# python3.10
Python 3.10.14 (main, Mar 22 2024, 15:42:52) [GCC 13.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import tempfile
>>> tempfile.mkdtemp()
'/tmp/tmpscec8nji'

# python3.11
Python 3.11.9 (main, Apr 15 2024, 18:24:23) [Clang 17.0.6 ] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import tempfile
>>> tempfile.mkdtemp()
'/tmp/tmprp7_k1h3'

# python3.12
Python 3.12.4 (main, Jun 17 2024, 16:05:13) [GCC 13.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import tempfile
>>> tempfile.mkdtemp()
'/tmp/tmpnsxqju_s'

In the end, we should probably recommend that pip_tmp_dir must be an absolute path, and outside the source path.
WDYT?

Hi @sassdavid, sure, please open a PR!

This issue has been resolved in version 7.7.1 :tada:


ResourcesOperations.check_existence returns Bad Request

Package Name: azure-mgmt-resource
Package Version: 20.0.0
Operating System: Linux/Ubuntu 18.04
Python Version:  3.6.9

Describe the bug
Using the method check_existence returns azure.core.exceptions.HttpResponseError: Operation returned an invalid status 'Bad Request'
To Reproduce
exists = resource_mgmt_client.resources.check_existence(
    resource_group_name=resource_group,
    resource_provider_namespace="Microsoft.Compute",
    parent_resource_path="",
    resource_type="virtualMachines",
    resource_name=vhub_name,
    api_version="2019-10-01"
)

Expected behavior
Should return True/False

@ksridhar57 Thanks for your feedback, we'll investigate asap.

@ksridhar57 I could see the same behavior, when I use the resource type as 'virtualMachines'.

Additionally, when I use the api version as 2021-04-01 then I get Operation returned an invalid status 'Method Not Allowed' error.

I am checking on internally on this one and get back to you.
Thanks
Saurabh

Hi @service team, it seems that the parameter is not valid for service. Please provide some doc or examples for customers to refer.

@ksridhar57 Sorry for the delay on this.  As per internal discussion this looks like a limitation as check_existence is a head call and HEAD is not supported by Compute for a VM url.
Workaround is to use GET instead of HEAD when it's a VM. However, using a GET does not have a Boolean return type instead you will get a typed return type (i.e. resource exists) if resource exists or an exception ResourceNotFoundError if it does not.

Please let me know if you have any other questions.
Thanks
Saurabh


Use ECP CI

Use CI testing platforms provided by ECP or ECP participating labs.

Moving to 1.0.0

@hategan We are using the provided gitlab. Is this what you mean?

With the sunset of ECP, this is not relevant any more.


refactor(react-sdk/js-web-sdk): fix various typos

This PR corrects the following typos:

"gaurantee" -> "guarantee"
"sucess" -> "success"
"userPromiseResovler" -> "userPromiseResolver"

It also updates the README to correctly list the latest version of the React SDK and JS Web SDK packages.

Looks like I don't have permission to request reviewers, so I'm tagging y'all here instead: @asaschachar @mikeng13 @jordangarcia
Thanks!

Thanks for your contribution. Would you mind signing the CLA form as stated in our Contributing guide please: https://github.com/optimizely/fullstack-labs/blob/master/CONTRIBUTING.md

Signed 🙂

@mikeng13 Just wanted to follow up on the status of this PR. Are we waiting for @mjc1283 to approve this as well?

Yes sorry @NaughtyMC . I'm just need @mjc1283 to bless it as well 😄


Support ".." is alias-path while setting a dict-state

@mkundu1. Please consider reassigning within the team.


explicitly cast content length to int

Else the unit tests fail with php7

Not just the unit tests also the output of assets.

I wanted to provide the same change. Can you rebase / fix the conflict so the merge can be done?

Please rebase/resplve conflicts, atm not mergable as is

rebased

Thx!


original input msg obj gets lost

The original input msg object with all other information (for example return node information) is getting lost.
The better way would be to update msg.payload and keep the other information.

The original input msg object with all other information (for example return node information) is getting lost. The better way would be to update msg.payload and keep the other information.

Please update with information on what you try to do. If possible add an example showing the faulty behaviour.

Input object:

Output object:

The tcp node creates a new msg object, instead of changing the payload. Therefore all other information like "_linksource" are getting lost. Put the tcp node between link_in and link_out nodes (to create a function) to see the behaviour. I stored this manually in the screenshot below.

I see your point too ;-)
I was used to this behaviour of the standard TCP request node, so it would be nice to have this feature implemented.
Since im not a js programmer but only a "stupid" node red user, I can´t help really much on this.
But I think I´ve found the code of the built in tcp node (especially line 646 and following):
https://github.com/node-red/node-red/blob/master/packages/node_modules/%40node-red/nodes/core/network/31-tcpin.js


Create Rigol_DG800

Driver for Rigol DG800 waveform generator

@BenMowbray Thanks for your contribution. However, in all 4 it seems like the files do not end in a .py could you fix that and we can have a look


Mark kmain() as never returning

This may help catch the bug you mentioned in https://www.youtube.com/watch?v=iTSx-8qK4Hw where you forgot to include loop { }, so kmain immediately returned to the boot assembly which continued to hlt ; This should never happen.

Thanks so much for this! Great idea.


Layout array

Closes #418 and #339
I adapted and integrated part of the old branch 'sites' into the current version of the code. The following changed in comparison to the old branch:

sites.py was a module at simtools. I renamed it to layout_array_builder and put inside simtools/layout since it is closely related to simtools/layout_array. I intend to expand this same module to include functions which create an array, according to a specific shape, for instance, the arrays we will need for the atmospheric tool (#360). This will still come, but I am already justifying its name;

I made the tests and the documentation of the functions that were missing in the original branch;

changed the mpatches of MST and the sizes, because the mpatches.Rectangle for the MST wasn't making sense to me. The sizes of the circles now reproduce in scale the differences in sizes of the telescope types. Furthermore, the MST mpatches.circle is filled to distinguish it more clearly from the LST (see image attached). Naturally in the image, the legend is malfunctioning, but this would be subject to a change in the legend_handlers.py. Let me know if you prefer it implemented here.

The next step is to write the application for plotting the array, which I can do once we agree on these changes here. Thank you.

Codacy still complains about this.
Minor icon MINOR
Code Style
Method could be a function
def telescope_layout_file_to_dict(self, file_name):

I am wondering if we should, instead of using these plot functions in a class, to have a simtools/util/layout.py ? What do you think @orelgueta ?

You can simply remove the self argument and add a @staticmethod decorator. We have quite a few of those if you want an example. (I started a review but will finish it tomorrow.)

You can simply remove the self argument and add a @staticmethod decorator. We have quite a few of those if you want an example. (I started a review but will finish it tomorrow.)

thanks. that is indeed a good idea

Codecov Report

Merging #429 (e18b692) into master (8cdb37a) will increase coverage by 0.34%.
The diff coverage is 94.70%.

@@            Coverage Diff             @@
##           master     #429      +/-   ##
==========================================
+ Coverage   81.27%   81.62%   +0.34%
==========================================
  Files          70       72       +2
  Lines        6931     7057     +126
==========================================
+ Hits         5633     5760     +127
+ Misses       1298     1297       -1

Impacted Files
Coverage Δ

simtools/layout/layout_array_builder.py
90.24% <90.24%> (ø)

tests/conftest.py
93.75% <93.33%> (-0.08%)
:arrow_down:

simtools/util/names.py
81.88% <100.00%> (+0.26%)
:arrow_up:

simtools/visualization/legend_handlers.py
61.05% <100.00%> (+10.52%)
:arrow_up:

tests/unit_tests/layout/test_layout_array.py
100.00% <100.00%> (ø)

...sts/unit_tests/layout/test_layout_array_builder.py
100.00% <100.00%> (ø)

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

Array plotting is something that you would really benefit from using ctapipe's functionality so we do it in one common way.  We have both coordinate systems  like ctapipe.coordinates.EastingNorthingFrame and  ctapipe.visualization.ArrayDisplay that could be used here.  I worry that we will and up with slightly different conventions, which should be avoided.
The only feature missing so far is to map the correct telescope names instead of simtel numbers, but we have an issue opened for that.
Using the ctapipe.instrument model even automatically can create an ArrayDisplay:
source.subarray.peek()

Hi Karl,
I agree using ctapipe plotting and coordinate transformation capabilities would be great. We are just a bit hesitant at the moment with adding ctapipe as a dependency. I am sure it will happen at some point (or perhaps some "core" ctapipe package), but it would probably happen when we need to use the analysis capabilities first like for muon analysis.
BTW, if we wanted to use the ctapipe array plotting for example, it would mean we would have to also use the subarray functionality. That in itself would involve a rethinking of our current telescope model in gammasim-tools. Not necessarily a bad idea, but it's not a simple "import and plot" decision.

I agree here with Orel - duplication is definitely bad, but it would be a large effort to adapt gammasim-tools to the ctapipe style.
Maybe easier to start introduce a DPPS core package with minimal interface for those standard tasks.

Hi all, thanks for the discussions. Indeed we should try to avoid duplications of large amounts of code (maybe small internal functions would be ok?), though I am not sure at the moment how is the best way to avoid it.
@orelgueta, I implemented all the changes we discussed here and in the mattermost, please have a look again once the tests pass. Thank you.

Hi @GernotMaier , @orelgueta . I worked out most of the suggestions. I argue in some cases against the suggestions and in some cases I could not find a quick solution (in the case of configuration, I am not quite used to this module yet). Please have a look at the opened conversations. Thank you!

Codacy complains about this eval
https://github.com/gammasim/gammasim-tools/blob/8e5fd1396057fb047b03275cb161fcf7bac6edbd/simtools/util/names.py#L797
Including another dependency (import ast) as it suggests doesnt seem to be a wise solution either. what do you think?

Codacy complains about this eval
https://github.com/gammasim/gammasim-tools/blob/8e5fd1396057fb047b03275cb161fcf7bac6edbd/simtools/util/names.py#L797
Including another dependency (import ast) as it suggests doesnt seem to be a wise solution either. what do you think?

I think you can do * u.Unit(unit) instead.

Thanks for the changes @VictorBarbosaMartins!
I answered some points but will do a more thorough review on Monday most likely.

I agree here with Orel - duplication is definitely bad, but it would be a large effort to adapt gammasim-tools to the ctapipe style.
Maybe easier to start introduce a DPPS core package with minimal interface for those standard tasks.

ctapipe is the core package.  I don't really understand the hesitancy to use it as one.  It is a very small package with minimal required dependencies (and most are the same).   You can't really separate the visualization from the coordinates from the instrument model, as they are all connected.
But this can be done incrementally, by for example simply using ctapipe's visualization tools to start, and then later other parts like the config system.  I would not suggest migrating everything.
The reason this is important is from a lesson learned"from HESS (and probably others).  Due to a slight difference in conventions between the simulation software and the data processing software, there were numerous bugs in the software that persisted for years without being caught, event though making a single plot using common plotting utilities would have caught them instantly.   We should avoid that.

Thanks for this - a few minor comments and considerations.
tests/unit_tests/visualization/test_legend_handlers.py is empty - this is what you want? Hard do test?

Ah I just saw this one comment. It is in this PR empty but I started implementing it in #434 and Orel said it is better to finish this PR first.

We might have discussed it offline, but I repeat my question about the change of the API for the init function of LayoutArray. What do I miss here to understand why it is needed? Neither site or mongo_db are needed in the init and also not for when using the class as a simple converter of a ecsv file. If both of you insist, then we can do it - but I would like to understand...

I agree that site is not always needed and we can make it optional (in case it is given in the file). The MongoDB info is a bit more necessary in case you want to limit the info in the file to just the site. However, we can make that optional as well in case all of the location information is given in the file (as is done now). Supporting both options would definitely make the class more flexible. The only question is whether this flexibility is necessary. I will anyway change the site requirement since it's an easy change. With the MongoDB info I will wait for your opinion since generally I don't mind the dependence on the DB.

Thanks @GernotMaier!
@VictorBarbosaMartins, please wait for me to do what I mentioned in the previous comment with the site and DB config before merging. It should be fairly simple and since it's part of this same PR, I think it should be done here.

I removed the requirement to provide the site and the MongoDB connection details in the case the info exists in the layout file. However the change discussed in #436 is necessary for the completeness of this PR I think. Nevertheless, @VictorBarbosaMartins, please check my changes and if you are happy with them I think we can merge. The rest can be done in separate PRs.


README typo in the community session

Mainling lists: https://lists.edgexfoundry.org/mailman/listinfo
Mainling → Mailing

I'm editing the README now, so I'll make this fix. Thanks Cloud


Skill : Drive Stat

Added the Drive Skills : Allies within 2 spaces receive +4 to their respective stat

Indentation issues still exists, use four space rather than TAB on Indentation.

Indentation issues still exists, use four space rather than TAB on Indentation.

I think it should be good now. Notepad++ tell me the file no longer contain any TAB character

It is recommanded to use vscode rather than notepad since vscoder may handler properly on code style


Sort GDNative api.json

Not 100% sure if this does the trick, it looks like its sorting the classes properly but it may need more to sort the methods and properties.
I kept running into problems with added api.json to our godot_headers repository, every time the order of things is different generating massive diffs.
Once we're sure this does the right thing it'll fix #27755

Seems fine, as long as generation time does not explode, but it should be fine

It seems to be sorting the classes correctly but I'm not 100% sure if its sorting the rest. It's obviously sorting on whatever sort function is implemented on the type of record contained within the list so it might just be sorting on something else then name.
It generated pretty fast for me, it's not that much data to process and its something you only run once in awhile.

Ok, I don't think this actually sorts but I can't really follow the logic here.
So I call the sort function on the list. Looking at list.h that ends up calling sort_custom.
Inside of sort custom it builds a list of element pointers that it can sort and then uses the AuxiliaryComparator structure to define the compare function it uses.
That does a compare using the original class, so best I can tell is that it will call operator() on the class.
Now these are implemented inside of structs but they don't seem to exist on the class itself.

Ok, I figured it out and everything now sorts properly as far as I can tell.
I am not 100% sure about the sort functions I've added for MethodInfo and PropertyInfo, I'm surprised they don't exist yet and maybe should move into object.h so they are more widely available.
Other then that, I'm pretty happy with this and I think it can be merged.

Thanks!

Cherry-picked for 3.1.2.

Cherry-picked for 3.0.7.


Add proposal for root signature in clang.

Move discussion from https://github.com/llvm/llvm-project/pull/83933

I think the latest update mostly captures the requirements. Are there changes you think need to be factored into the design based on the requirements?

I'm not sure what to do for the future versions.
And we have discussion about the IR representation. I'll update once we got conclusion on that.

I'm not sure what to do for the future versions. And we have discussion about the IR representation. I'll update once we got conclusion on that.

My read of the IR representation that you're proposing is that it is pretty tightly linked to the binary encoding that gets put into the container file. I think we should decouple those representations.

@python3kgae, the documentation PR that you closed on LLVM had a lot of really valuable information. It's really a shame to drop that. Can you please do as I suggested here and put up a PR with just the documentation of the root signatures without the implementation design?
It would also be extremely helpful to have the grammar for the local and global root signatures so that we can understand the required parsing impact.

@python3kgae, the documentation PR that you closed on LLVM had a lot of really valuable information. It's really a shame to drop that. Can you please do as I suggested here and put up a PR with just the documentation of the root signatures without the implementation design?
It would also be extremely helpful to have the grammar for the local and global root signatures so that we can understand the required parsing impact.

Sure. I can do that.
Where should this separate doc live?
And any more issue with current PR?

Sure. I can do that. Where should this separate doc live?

I can see two docs being useful. The "Usage" and "Root Signature Grammar" sections of your document seem like they should live under clang/docs/HLSL, and the "Serialized Format" section could live in llvm/docs/DirectX
Related: I filed an issue on myself to document the DXContainer data that I implemented (https://github.com/llvm/llvm-project/issues/88775).

And any more issue with current PR?

We need the grammar for local and global root signatures documented here or in the PR to LLVM.
I'll give the document here a more detailed read today to see if I have any other questions.

Superseded by #209.


Prepared MonologBundle for Symfony 4

In Symfony 3 the public parameter for service definitions is defaulted to true, while in Symfony 4 it is defaulted to false.
In order to keep MonologBundle working with both major versions, all we need to do is to explicitly set that parameter on all service definitions.
This Pull Request adds public="true" to all service definitions that didn't specify it.

Symfony 4 is released :tada:

Hi, do you know when this change is going to be merged in master ?

Hmm, we don't technically need this: it's ok that the logger became private: people will see deprecation warnings about it in 3.4 (that it will become private, so stop fetching it directly from teh container) and can use dependency injection instead.
So, the question is, do we want this or not?


Introduce magic construct identifier that does not add to the logical ID hash

Right now, as soon as you extract functionality out of a construct and move it to a separate construct that is now wrapped by the original one, all logical IDs change for the already deployed resources. That is generally now what you want.
A simple solution is a magic construct identifier (like "Resource" is today) that does not get added to the name nor the hash (today, "Resource" does not get added to the name but it DOES get added to the hash).
This is not introduce a chance of conflicts, since if the parent's logical ID was unique, the new child's logical ID must be unique as well, and no other child can have the same logical ID (because the magic identifier must be unique among siblings).
In fact, we could (should?) change "Resource" to be this unique marker... with the downside that all logical identifiers change, all tests break, and all customers have to replace all resources in all of their stacks.

Changing the semantics of "Resource" is probably too invasive.
How about one of these:

"Wrapped"
"Default"
"Index" (<-- we're web tech!)

Default sounds good.


[beta] Improve example for "Applied Visual Design: Use the u Tag to Underline Text"

Challenge use-the-u-tag-to-underline-text has an issue.
User Agent is: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.76 Safari/537.36.

Google

Google was founded by Larry Page and Sergey Brin while they were Ph.D. students at Stanford University.

    Larry Page

      Sergey Brin

Much like #12831, I think the example used in this challenge could be improved. Since the  tags already have a underline, there's no clear change from adding the  tags.
What do you think about changing this challenge to ask campers to wrap the words "Ph.D. students" like Ph.D. students?
This text formatting makes no sense practically - but it makes a better example.
To make this change:

Update the instructions.
Update the tests.
Update the challengeSeed for the following challenges.

This PR #12860 demonstrates what this update could look like.

hello

stiffness Chan industry Bhutan Ellen it and hairy and some other parts

@Greenheart - agreed, I think putting the tag on text to show the formatting helps show what it does better than placing it on the (already underlined) links. 👍  We can open this up as Help Wanted, or I could tackle it tomorrow morning 😄

I could fix this now, I've got nothing else to do :smile:

Awesome 🎉

@HKuz Should I add solutions while I'm at it? I suppose they are used when running npm test to verify that all challenges in the seed works properly?

You can add solutions 👍

@HKuz Nice, I'll fix that when I'm done with my current PR


changed scope of d4 var to work with how browserify compiles

Did you manually change the references to d4 in the compiled file? Because if so that won't work with the way the lib is compiled from parts.

no, i only changed the base.js file and compiled with grunt

Ok the compiled file looked a bit odd. Let me test your branch locally but if everything works i'll make a new release. Thanks!

One final question, is there a way to give a minimal browserify test for this? How did you confirm this worked?

good question. I found this tool browserify-test and ran this test which passed:
var expect = require( 'chai' ).expect;

describe('browserify test', function() {
  it('browserify should compile d4', function() {
       var d4 = require( '../lib/d4.js' );
      expect(d4).to.not.be.an('undefined');
  });
});

let me know if that works.

Awesome, can you integrate that with the existing test suite and append the PR you sent me?
Thanks again for your hard work!
Best Regards,
Mark

On Mar 29, 2016, at 10:47 AM, Andrew Lomax<EMAIL_ADDRESS>wrote:
good question. I found this tool browserify-test https://github.com/alekseykulikov/browserify-test and ran this test which passed:
var expect = require( 'chai' ).expect;
describe('browserify test', function() {
it('browserify should compile d4', function() {
var d4 = require( '../lib/d4.js' );
expect(d4).to.not.be.an('undefined');
});
});
let me know if that works.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub https://github.com/heavysixer/d4/pull/45#issuecomment-202968779

ok. no problem. I need to add grunt-browserify and a new task to run during grunt test. is that fine with you?

Perfect, I’ll confirm it works for me and then merge.

On Mar 29, 2016, at 12:04 PM, Andrew Lomax<EMAIL_ADDRESS>wrote:
ok. no problem. I need to add grunt-browserify and a new task to run during grunt test. is that fine with you?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub https://github.com/heavysixer/d4/pull/45#issuecomment-203000163


Create 'behind the scenes' for FreeCEN

Use https://www.freereg.org.uk/cms/information-for-coordinators/behind-the-scenes as a basis (initially with replacement of 'FreeREG' with 'FreeCEN'
Economic 0, Urgency 3, strategic 9, Cost 9: Priority = 21

@geoffj-FUG is this still needed as a web page, or will it be part of the handbook?

The Handbook is primarily a How to ... It does not cover generalities.
The Handbook is now 107 pages and growing. Behind the scenes sounds like general information. The Handbook will be daunting.
That is why I am creating 2 more documents - a Quickstart Guide to Transcribing which is really for experienced transcribers and a Ready Reference which I suspect will be the reference for most transcribers. Both are small, easily printable and dont create the fear that the Handbook will to a newbie.
The answer to your question is that if this is the Duty Statements that are currently there we need them to still be there. They will not be in the Hnadbook at this point.
Geoff

Thanks, @geoffj-FUG
Looks like a fairly quick job (famous last words) I'll move into sprint backlog as I may need to be in the office, but not doing much over the weekend.

@geoffj-FUG could you send me the duty statements and I'll see whether they are best added here, or elsewhere.
Draft is here: https://test3.freecen.org.uk/cms/behind-the-scenes

Also needs two coord roles merging, as per #847.

I have not merged the two coords roles but - I hope - explained that both are combined in the single role of Coord, but with actions on the different buttons.
Does the table need editing @Captainkirkdawson to make it clearer which buttons have tools for managing the team and its members, and which for managing the county?
@geoffj-FUG is there anything that a volunteer would expect to be told about FreeCEN which is not in the handbook, and not here?

Pat
I have picked out a couple of Coordinator Actions and duplicated them in the Handbook. That is where they will probably look first.
Do you want to add Download Handbook to the Actions table?
I am not aware of anything that is not in the Handbook or not in ‘Behind the Scenes’. The Handbook is dynamic so anything that pops up can be easily added.
Geoff
From: PatReynolds<EMAIL_ADDRESS>Sent: Tuesday, November 3, 2020 8:36 PM
To: FreeUKGen/FreeCENMigration<EMAIL_ADDRESS>Cc: geoffj-FUG<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [FreeUKGen/FreeCENMigration] Create 'behind the scenes' for FreeCEN (#376)
I have not merged the two coords roles but - I hope - explained that both are combined in the single role of Coord, but with actions on the different buttons.
Does the table need editing @Captainkirkdawson https://github.com/Captainkirkdawson  to make it clearer which buttons have tools for managing the team and its members, and which for managing the county?
@geoffj-FUG https://github.com/geoffj-FUG  is there anything that a volunteer would expect to be told about FreeCEN which is not in the handbook, and not here?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/376#issuecomment-721035122 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFJO2G7JGSYQAEDW7A3SN7MJZANCNFSM4EF43VGQ .  https://github.com/notifications/beacon/AKCPIFMJQSLMKXZOCS64QB3SN7MJZA5CNFSM4EF43VG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFL5B64Q.gif

Pat
Coordinator Actions link from Behind the scenes page is not working. Nor are the links below it.
Geoff
From: PatReynolds<EMAIL_ADDRESS>Sent: Tuesday, November 3, 2020 8:36 PM
To: FreeUKGen/FreeCENMigration<EMAIL_ADDRESS>Cc: geoffj-FUG<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [FreeUKGen/FreeCENMigration] Create 'behind the scenes' for FreeCEN (#376)
I have not merged the two coords roles but - I hope - explained that both are combined in the single role of Coord, but with actions on the different buttons.
Does the table need editing @Captainkirkdawson https://github.com/Captainkirkdawson  to make it clearer which buttons have tools for managing the team and its members, and which for managing the county?
@geoffj-FUG https://github.com/geoffj-FUG  is there anything that a volunteer would expect to be told about FreeCEN which is not in the handbook, and not here?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/376#issuecomment-721035122 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFJO2G7JGSYQAEDW7A3SN7MJZANCNFSM4EF43VGQ .  https://github.com/notifications/beacon/AKCPIFMJQSLMKXZOCS64QB3SN7MJZA5CNFSM4EF43VG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFL5B64Q.gif

@PatReynolds
A number of observations.

Your paragraph on the county and syndicate roles is fine.
I am surprised that neither the REG nor the CEN versions talk about the 4th communication action. Specifically the Communicate action that allows the user to: The Communicate action allows you to communicate directly with members of the FreeREG team. Every user has it.
The coordinator help buttons on the right; only the first is active; the others are not relevant to CEN and should be replaced with the link to the Handbook
The coordinator role page really is quite dated and will raise more problems and issues than it resolves.
The Actions are the REG actions not the CEN actions which are quite different as is their order (list sent off line)
Is this table too much for 90% of the users???

working on this now
could you remove, please, @Captainkirkdawson
now story #1054
working on this now
is the table (see https://test3.freecen.org.uk/cms/draft-behind-the-scenes) too much, @geoffj-FUG

Text ready for proofing (https://test3.freecen.org.uk/cms/draft-behind-the-scenes)
This is currently seen as a sub-page for Coordinators (see links on right) but is it needed as a sub-page for Transcribers and others too?  If so, should links on right to be to Help for Coordinators, Help for Transcribers, and ????

assigned Javier 12/12/2020

Pat
Table looks OK to me.
Whilst it needs interpretation I think it serves its purpose. We know what these things mean.
Geoff
From: PatReynolds<EMAIL_ADDRESS>Sent: Saturday, December 12, 2020 10:41 PM
To: FreeUKGen/FreeCENMigration<EMAIL_ADDRESS>Cc: geoffj-FUG<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [FreeUKGen/FreeCENMigration] Create 'behind the scenes' for FreeCEN (#376)
assigned Javier 12/12/2020
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/376#issuecomment-743750198 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFKNDDOAMEVCUE2PFJ3SUNQEDANCNFSM4EF43VGQ .  https://github.com/notifications/beacon/AKCPIFOT7XXBWHA7GH3PHR3SUNQEDA5CNFSM4EF43VG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFRKLUNQ.gif

@PatReynolds I've spotted and corrected some simple typos: a hyphen that should be an em-dash, a missing capital letter and a missing full stop. (I thought it easier to do than to explain!)
I will leave it to others to comment on the content as I'm a little out of touch.

On further reflection, there are clearly some Actions missing from the table, but there is not much wriggle room to add them in. Maybe remove and list separately the Actions that are common to all Members?


Setup vue frontend

sets the router to point to an index.html
added files from rooms-frontend's production build into web/assets folder
updated some of the CSP config to allow the external graphql API and blobs

:thinking:


Fingerprint generation from custom dataset

Hi, first of all great work. I am trying to use this model to generate some fingerprints for a single movie audio (let's say 2 hours long). I trained the model, following your instructions, and for the -s argument in the python run.py generate command I used the directory containing the movie audio. Then the fingerprints are generated, all good. I then proceed to read the produced memmap, I convert it into a numpy array and it's of size (16353,128) with duration=1 and hop=0.5 as parameters. All good until now. However, when I inspect the produced fingerprints, they are repeated every 125 fingerprints, meaning that it produces 16353/125=131 batches which are the same. Why is that? Don't you crop the audio file into segments and then pass it through the model, or should I do that beforehand? Thanks!

Hi @AspiringDeveloper00, sorry for late reply. I currently don't have the environment set up to run the code. However, based on my guess, this part of the code is related to the repeating behaviour you mentioned.
https://github.com/mimbres/neural-audio-fp/blob/058d812df3787a7e000c6f595e200fd2e15ee348/model/dataset.py#L312
_ts_n_anchor should be set the same as 125, assuming your test batch size defined in config was 125.
If _ts_n_anchor is 1, it will create the same clones (of single segment) in the batch.
It's been too long since I tried using a custom dataset, so I can't remember well. I hope this provides some help in solving the problem.

Hey, thanks for answering. I figured it out, it was because I transferred the checkpoints to another machine to generate the fingerprints there and possibly I came across a bug. I retrained the model on the new machine and everything is good!