bug(tcgc): getSdkHttpParameter() doesn't handle default value

Describe the bug
We try to set a endpoint parameter with default value in https://github.com/Azure/cadl-ranch/pull/598
@server(
  "{endpoint}",
  "Test server endpoint",
  {
    endpoint: string = "http://localhost:3000",
  }
)

The parameter can be fetched in TCGC, but default value is not parsed. Check the source codes of getSdkHttpParameter(), and it seems not handle default value: https://github.com/Azure/typespec-azure/blob/5c10b3e3524936610283b4044ddd01c73a22827f/packages/typespec-client-generator-core/src/http.ts#L294
Reproduction
Just try to parse the definition above.
Checklist

[X] Follow our Code of Conduct
[X] Check that this issue is about the Azure libraries for typespec. For bug in the typespec language or core libraries file it in the TypeSpec repo
[X] Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible example of the bug.

@iscai-msft i remembered that only when the default value is happened in server endpoint that we will use it as client default value, is that right? if so, it seems we missed this in current code.

Yep, it's only for api version and endpoint. This information is in .templateArguments though, so I don't think this is an issue. Closing. If it's still an issue can you reopen @archerzz

@iscai-msft for parameters in .templateArguments, looks like I could not get the defaut value unless I parse the __raw.defaultValue. Maybe we should set the .clientDefaultValue?

This is a good point, we purposefully don't take defaultValue into account unless for endpoint or api-version related information. I think we should take into account this information since this is endpoint related. I'm going to make a PR and merge it after I make sure people are ok with it. Thanks @archerzz


feat: Add workspaces documentation

Description

What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents

closes #254
Mobile & Desktop Screenshots/Recordings

Steps to QA

Added to documentation?

[ ] 📜 README.md
[x] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Notes for Updates

public by default
root of the website, they'll be redirected to personal workspace (if you haven't been onboarded)
close #253 with this issue
everytime you go back to a workspace, it's the last one that you've accessed

Paid v. free features

Private workspaces are paid features

Contributors

tracked contributors

Thanks, John! Updated.


fix #96 add nil check in connpool

Codecov Report

Merging #100 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #100   +/-   ##
======================================
  Coverage    22.2%   22.2%
======================================
  Files          40      40
  Lines        3499    3499
======================================
  Hits          777     777
  Misses       2684    2684
  Partials       38      38

Continue to review full report at Codecov.

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


Fix kube version again

With all these kubeVersion changes lately, it got borked and I guess I didn't notice because I had a bundle. This fixes the latest bundle call for the package controller without breaking the usage for the CLI. I also tried to clean up kubeVersion handling in general and avoid things like make it a string and then parse the string.

/lgtm


Can't access value from another 3rd party transformation

I'm trying to create my own field from the header using this package:
https://jcustenborder.github.io/kafka-connect-documentation/projects/kafka-connect-transform-common/transformations/examples/HeaderToField.headertofield.html
transforms.AppendServer.type=com.github.jcustenborder.kafka.connect.transform.common.HeaderToField$Value transforms.AppendServer.header.mappings=connect.hostname:STRING:server
However if I try to access $value.server in my filepulse filters, I don't get a value.
filters.SetKeyWithTag.type=io.streamthoughts.kafka.connect.filepulse.filter.AppendFilter
filters.SetKeyWithTag.field=$key
filters.SetKeyWithTag.value={{ concat($value.server, '-', $value.vendor, '-', $value.user, '-', $value.host, '-', $value.tag) }}
filters.SetKeyWithTag.overwrite=true

Hi @sscots, Connect Transfomers are executed internally by the Kafka Connect framework so after the filter-chain of Connect FilePulse.

Hi @sscots, this issue has been resolved (https://github.com/streamthoughts/kafka-connect-file-pulse/issues/189). Kafka Connect FilePulse 2.4.1 is now available.


Support @implicitNotFound errors

Magnolia has pretty good missing implicit errors.
However custom @implicitNotFound messages defined by users are swallowed.
This came up on Gitter.

Are you thinking that we could read the appropriate @implicitNotFound annotation and use its text in Magnolia's output?

It should be simpler than that - use inferImplicitValue with silent = false and catch the error.
I'm not sure if this would slow us down though.

Although it probably wouldn't be too hard to read the annotation from the type only when we need to produce an error.


Config as model or much nicer config

As per title

Feature request for discord bot: an easy way to feature flag off cogs
while doing this

Feature request for discord bot: an easy way to feature flag off cogs
while doing this


Datepicker orientation

How can i control the orientation of the datepicker like in the non-angular-datepicker? I can't finy any option in the docs.

https://github.com/angular-ui/bootstrap#need-help


Histogram

Hi! Not sure if opening a new Issue here is the right way to go about this. I'd like to understand why the histogram created through t-digest loses so much information.
The original histogram with 500 breaks (every 0.05). Double peak is clearly visible.
.
Histogram created using t-digest centroids's means and counts. Double peak is not visible.

The raw data that I used includes 49220071 data points from 0 to 60.
This is the code I used to produce  histogram-from-tdigest
Is using Centroid's mean() and count() even the right way to go about this?
Thank you.

Ah. I can see something from your code.
Don't use the array digest.  Use the avltree digest.
Sent from my iPhone

On Mar 15, 2015, at 13:54, Grega Kespret<EMAIL_ADDRESS>wrote:
Hi! Not sure if opening a new Issue here is the right way to go about this. I'd like to understand why the histogram created through t-digest loses so much information.
The original histogram with 500 breaks (every 0.05). Double peak is clearly visible.
.
Histogram created using t-digest centroids's means and counts. Double peak is not visible.
The raw data that I used includes 49220071 data points from 0 to 60.
This is the code I used to produce histogram-from-tdigest
—
Reply to this email directly or view it on GitHub.

Another phoned in comment, t-digest makes no pretense of recreating densities.  It recreates the cdf. Try plotting cdf's against each other.  Also make sure compression factor is relatively high (200 or more) since you are interested in behavior far from the tails.
Sent from my iPhone

On Mar 15, 2015, at 13:54, Grega Kespret<EMAIL_ADDRESS>wrote:
Hi! Not sure if opening a new Issue here is the right way to go about this. I'd like to understand why the histogram created through t-digest loses so much information.
The original histogram with 500 breaks (every 0.05). Double peak is clearly visible.
.
Histogram created using t-digest centroids's means and counts. Double peak is not visible.
The raw data that I used includes 49220071 data points from 0 to 60.
This is the code I used to produce histogram-from-tdigest
—
Reply to this email directly or view it on GitHub.

I think I figured this out.  The plot that you have here obscures the spacing between centroids.  To recover the PDF from the t-digest you have to adjust the weight of the centroid according to the distance to the neighbors.

I tried with your previous suggestions, that is using avltree digest with high compression factor (200). This is the histogram (pdf) that it creates:

Indeed, the cdf is captured really well.
I didn't really fully understand your last comment. If I still want to plot pdf, how do I do what you described? Thanks!

Finishing this off ... yes, weighting the counts correctly is crucial to proper presentation of a PDF given a t-digest. The basic idea is that each centroid approximates the count over a particular range. To convert this to a rectangle in an approximated PDF, you need to convert the count and the range to get the slope of the CDF. Getting the bounds of the centroid is a bit tricky, but there is a pretty good way to do it:
Given: centroid index i, number of centroids n, counts k[i], positions x[i], smallest x_min and largest x_max samples seen
Wanted: left x_left and right x_right bounds of centroid
if i == 0:
x_left = x_min
else:
x_left = (x[i] * k[i-1] + x[i-1] * k[i]) / (k[i-1] + k[i])
if i == n-1:
x_right = x_max
else:
x_right = (x[i] * k[i+1] + x[i+1] * k[i]) / (k[i+1] + k[i])
Then the rectangle of the PDF can be drawn from x_left to x_right with height k[i] / (x_right-x_left)
Closing this for now.


Accessing context from Interactor::Organizer

I have something like this:
module Domains
  class Rename
    include Interactor::Organizer

    @interactors = [Domains::Install, Domains::UpdateName]
    if context.domain.ssl_cert.present?
      context.ssl_cert = context.domain.ssl_cert
       @interactors << SslCerts::Disable
    end

    organize @interactors
  end
end

But getting:

undefined local variable or method `context' for Domains::Rename:Class

Is there a way to access context from this class?

@michaelminter
It is possible, but I would not recommend it.  Would updating SslCerts::Disable to include a guard clause (e.g., return unless context.domain.ssl_cert.present?) and then always including it in Domains::Rename's organized interactors work for you instead?

Yeah, that's what I'm doing now as a workaround. But for low level insight, how do you call context in the organized interactor?

@michaelminter you call context on the instance, eg
class Organizer
  include Interactor::Organizer

  organize [A, B]

  def call
    # Access context here
    super
  end
end


Restructure project

What happened 👀
From @sleepylee suggestion, we have to restructure this project so that it's become a regular project and not a container of multiple projects.
Insight 📝
Running a script to generate a new project from the RxJava template and modified it to link to this repository.
Proof Of Work 📹
Able to run both staging and production correctly.

⚠ The failed checks will be resolved in #17

please make sure that it passes the CI (with our custom runner)
please update the CODEOWNER file to include only the people who involve in this project.

please make sure that it passes the CI (with our custom runner)
please update the CODEOWNER file to include only the people who involve in this project.

@sleepylee I fix the CI in this #17 PR. I think it will be more clear if we separate it to another PR. Is that seem okay to you?

@chornerman Have you started the runner? It seems CI still hasn't worked.


Model translation not applied everywhere

Hello,
Thank you for this trully amazing gem, it's saving me a ton of time.
I can also try to contribute with a PR to fix this issue if you can point me to the right direction.
So, I created several files to translate my models.
Example yml:
pt:
  activerecord:
    models:
      course:
        one: Curso
        other: Cursos
    attributes:
      course:
        name: Nome
        institution: Institutição
        external_id: ID Externo
        external_data: Dados Externos
        updated_at: Atualizado em
        created_at: Criado em

And motor-admin read them perfectly:

But there are places where the translation is not applied, such as the sub-menus:

Also, enum translations doesn't seem to work anywhere:

@bruno-berchielli i think that's just something that's not a part of rails i18n (please correct me if there is a way to translate scopes/associations in rails i18n).
But in your case you can simply rename resources in motor-admin - this should work if you're using only one language.

Hello @omohokcoj
Probably you are right about it not been part of the rails i18n API.
But I believe it could be simple to infer the model class from the has_many scopes and than apply the Rails i18n model name translation in the front-end.
e.g.
[8] pry(main)> Course.reflect_on_all_associations(:has_many).map { |z| z.name.to_s.singularize.camelcase.constantize }
[
    [0] class Curriculum < ApplicationRecord {
                    :id => :integer,
        :institution_id => :integer,
             :course_id => :integer,
                  :name => :string,
         :year_exercise => :string,
           :external_id => :string,
         :external_data => :jsonb,
            :created_at => :datetime,
            :updated_at => :datetime
    },
    [1] class ExternalTransferProcess < ApplicationRecord {
                            :id => :integer,
                :institution_id => :integer,
         :origin_institution_id => :integer,
                   :external_id => :string,
                 :external_data => :jsonb,
                    :created_at => :datetime,
                    :updated_at => :datetime,
        :parsed_transcript_file => :jsonb,
          :parsed_syllabus_file => :text,
                 :curriculum_id => :integer,
                        :status => :string,
                   :status_info => :jsonb,
              :origin_course_id => :integer
    }
]

Of course, that would work only with ActiveRecord::Associations::ClassMethods API , but that is probably the majority of the cases anyway.
Rename resources is an excelent alternative thanks, I was unaware of it.

@bruno-berchielli apparently the gem uses i18n rails model attribute name for association translations:
https://github.com/motor-admin/motor-admin-rails/blob/master/lib/motor/build_schema/load_from_rails.rb#L270
So maybe you can use it for i18n or just use Name field in the UI to rename associations.
Closing the ticket assuming the above suggestions can solve the issue.


Error while installing supergradients

💡 Your Question
pip install super-gradients==3.1.0 on ubuntu 20.04 python 3.8 is showing the following error:
× Building wheel for pycocotools (pyproject.toml) did not run successfully.
237.6   ¦ exit code: 1
237.6   ?-> [24 lines of output]
237.6       running bdist_wheel
237.6       running build
237.6       running build_py
237.6       creating build
237.6       creating build/lib.linux-x86_64-cpython-38
237.6       creating build/lib.linux-x86_64-cpython-38/pycocotools
237.6       copying pycocotools/cocoeval.py -> build/lib.linux-x86_64-cpython-38/pycocotools
237.6       copying pycocotools/mask.py -> build/lib.linux-x86_64-cpython-38/pycocotools
237.6       copying pycocotools/init.py -> build/lib.linux-x86_64-cpython-38/pycocotools
237.6       copying pycocotools/coco.py -> build/lib.linux-x86_64-cpython-38/pycocotools
237.6       running build_ext
237.6       Compiling pycocotools/_mask.pyx because it changed.
237.6       [1/1] Cythonizing pycocotools/_mask.pyx
237.6       building 'pycocotools._mask' extension
237.6       creating build/common
237.6       creating build/temp.linux-x86_64-cpython-38
237.6       creating build/temp.linux-x86_64-cpython-38/common
237.6       creating build/temp.linux-x86_64-cpython-38/pycocotools
237.6       gcc -Wno-unused-result -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -fPIC -I/tmp/pip-build-env-3h9be_du/overlay/lib/python3.8/site-packages/numpy/core/include -I./common -I/usr/local/include/python3.8 -c ../common/
maskApi.c -o build/temp.linux-x86_64-cpython-38/../common/maskApi.o -Wno-cpp -Wno-unused-function -std=c99
237.6       cc1: fatal error: ../common/maskApi.c: No such file or directory
237.6       compilation terminated.
237.6       /tmp/pip-build-env-3h9be_du/overlay/lib/python3.8/site-packages/Cython/Compiler/Main.py:381: FutureWarning: Cython directive 'language_level' not set, using '3str' for now (Py3). This has changed from earlier relea
ses! File: /tmp/pip-install-pp3evmo9/pycocotools_e098238c76834a67bdd5a1a7fda1e71b/pycocotools/_mask.pyx
237.6         tree = Parsing.p_module(s, pxd, full_module_name)
237.6       error: command '/usr/bin/gcc' failed with exit code 1
237.6       [end of output]
Versions
No response

Can you try with our latest release 3.1.3? We've upgraded dependency version for pycocotools recently which may fix this issue.

It works,Thank you!


Incomplete HTML response bodies

Noticed an unusually high number of pages without a <body> tag in the HTML response body. Here's a query to take a sample:
WITH req AS (
  SELECT
    page,
    response_body
  FROM
    `httparchive.all.requests` TABLESAMPLE SYSTEM (0.10 PERCENT)
  WHERE
    date = '2023-05-01' AND
    client = 'mobile' AND
    is_main_document AND
    is_root_page AND
    REGEXP_EXTRACT(response_body, r'(?i)(.*)<body') IS NULL
  LIMIT 10
),

pages AS (
  SELECT
    page,
    wptid
  FROM
    `httparchive.all.pages`
  WHERE
    date = '2023-05-01' AND
    client = 'mobile' AND
    is_root_page
)

SELECT
  page,
  wptid,
  response_body
FROM
  req
JOIN
  pages
USING
  (page)

For example, here's one WPT response body and you can see it cut off at mid-CSS: https://webpagetest.httparchive.org/response_body.php?test=230509_Mx1ZG_FBEG4&run=1&bodyid=741FA7D767C970D82CCE5621C0B68519
.et_animated.slideLeft{-webkit-animation-name:et_pb_slideLeft;animation-name:et_pb_slideLeft}@-webkit-keyframes et_pb_bounce{0%,20%,40%,60%,80%,to{-webkit-animation-timing-function:cubic-bezier(.215,.61,.355

The page seems to render fine in the test, as the filmstrip shows visual content and the waterfall is full of requests. Viewing source on the live page also shows complete HTML.
So maybe there's something in WPT or the HA pipeline that's cutting off the response body?

I'll look closer during the week in case there's something going on with the netlog body streaming but, just in case it's an issue, I just doubled the size of the agent SSD's from 10GB to 20GB (and got approval for the quota).
The test agent we use for manual tests had run out of disk space and was failing so it is running with a lot more space than the regular agents but it is also persistent for months while the agents are just alive for a day or so (and I'd expect much worse side-effects than some truncated bodies) but it doesn't hurt to have a bit more breathing room on the disks.

Besides the disks fix, I suppose the data was lost. So there is nothing else to do.
Closing.


Juggling check  for handling null and undefined

Check for null and undefined using the juggling method. In which both null and undefined handle in a single hit.

@BalajiJBcs Thanks for raising the PR.
But the file you are changing is an auto generated one by Autorest.
We have swagger enddoints and we run auto rest to generate this lib code.
So the manual changes to this file are overridden once we run the auto rest in future.
I am not sure, if we want to go ahead with this PR.
Do you have any concerns ?

Have closed the request based on @ajaykn  comments


Fix user related bug

This fixes issue happened when user is not authenticated. However, #55 is still there and will be taken care of in the next commit.


Customise colour of a single dot

We'd like to highlight one of the pages by setting it as a different colour. Mocked screenshot:

Without forking the library, how can I achieve this?

This wasn't possible out of the box so I've forked the library and customised for our needs


Add support for 10.0.14936.1000

termsrv.zip

What to do with this file? Normally we don't change this dll file. This was the reason for rdpwrappen insteat of changing dll files after each update from Windows

What to do with this file?

Inspect it for getting new offset values for INI file. (see README)

I am also having the same issue. It had been working fine for last few months but since the latest Windows 10 update last week, first it started showing "Not Listening" in RDPConf and later after uninstalling and re-installing, although it started showing "Listening" and "Fully Supported", I am unable to remote into it from other machines on the network. Please look into this, and let me know if you need more information/logs. Thanks

showing "Listening" and "Fully Supported", I am unable to remote into it from other machines on the network

Your issue is not the same. Check for firewall, etc.

@binarymaster

Your issue is not the same. Check for firewall, etc.

Thanks for your response. I did that already and ALL inbound connections are already allowed in Firewall. Also, the fact that I remote in to this machine daily (media server in attic) from my laptop and it broke the minute after installing the Win 10 update, was an indication to me that they were related. I am still able to ping it, can play media stored on it on TV, but can't RDP any longer. Looks like issue #166


refactor Implementation branching into its own module

:warning:  NOTE: this PR depends on #28 so it's base branch has been set to that PR's branch. switch the base to main before merging this

PRs #33 #34 and #41 were merged into this branch instead of being merged into main. re-doing this as PR #48


Commit 7907813 strata .json files, "invalid UTF-8 input in readChar()"

@anthonysena
Issue: Commit 7907813 led to the following error code when I ran the current version of the package: "Error in readChar(pathToJson, file.info(pathToJson)$size) :    invalid UTF-8 input in readChar()
The error only happened when creating strata cohorts right at the beginning of attempting to make them but none of the other cohorts threw the error. I'm not sure which strata cohort(s) was/were the problem, however. I do see in my "Covid19CharacterizationCharybdis.txt" file that the first strata cohort created is "Instantiation cohort Prevalent Type 2 Diabetes Mellitus (100.sql)" so perhaps that is the or one of the files at issue. Maybe all the strata .json files are loaded together though and if any of them have an issue the error is thrown.
Solution: I replaced the .json files from this commit with the .json files from commit ae5537d. The error went away and the strata cohorts were successfully created.
I know this package is no longer supported but this issue may be worth looking into if whatever caused this problem in commit 7907813 is a process used in the future which might re-create this issue in future packages.

Thanks @jatjasonthomas for reporting this issue.
Just wanted to confirm: have you rebuilt the package after pulling down the latest code? I was helping someone recently (with a different package) and they encountered the same problem with readChar and it was fixed by rebuilding the package which copied in all of the proper resource files, etc.
Thanks!

I'm experiencing the same problem using the release 1.4.2

Hi,
I'm having a bit of a challenge reproducing this problem so I'll need some assistance. Here is some code I'm hoping one of you can run to help identify the file(s) exhibiting the problem you are experiencing:
print(sessionInfo())
pathToJson <- file.path(getwd(), "inst/cohorts/")
lfilesJson <-
  dir(
    path = pathToJson,
    pattern = "\\.json$",
    full.names = TRUE,
    recursive = TRUE
  )

for (file in lfilesJson)  {
  tryCatch({
    fileContents <- ""
    fileContents <- readChar(file, file.info(file)$size)
  },
  error = function(e) {
    print(paste0("Problem File: ", file))
  })
}

The pathToJson variable may need adjustment to point to the contents of the Charybdis package. What I'm hoping to capture is the locale information you are using and which file(s) exhibit an issue when using the readChar function. For reference, here is the output of this code on my machine:
print(sessionInfo())
#> R version 4.1.0 (2021-05-18)
#> Platform: x86_64-w64-mingw32/x64 (64-bit)
#> Running under: Windows Server x64 (build 17763)
#>
#> Matrix products: default
#>
#> locale:
#> [1] LC_COLLATE=English_United States.1252
#> [2] LC_CTYPE=English_United States.1252
#> [3] LC_MONETARY=English_United States.1252
#> [4] LC_NUMERIC=C
#> [5] LC_TIME=English_United States.1252
#>
#> attached base packages:
#> [1] stats     graphics  grDevices utils     datasets  methods   base
#>
#> loaded via a namespace (and not attached):
#>  [1] digest_0.6.27     withr_2.4.2       magrittr_2.0.1    reprex_2.0.0
#>  [5] evaluate_0.14     highr_0.9         stringi_1.7.3     rlang_0.4.11
#>  [9] cli_3.0.1         rstudioapi_0.13   fs_1.5.0          rmarkdown_2.9
#> [13] tools_4.1.0       stringr_1.4.0     glue_1.4.2        xfun_0.24
#> [17] yaml_2.2.1        compiler_4.1.0    htmltools_<IP_ADDRESS> knitr_1.33
pathToJson <- file.path(getwd(), "inst/cohorts/")
lfilesJson <-
  dir(
    path = pathToJson,
    pattern = "\\.json$",
    full.names = TRUE,
    recursive = TRUE
  )

for (file in lfilesJson)  {
  tryCatch({
    fileContents <- ""
    fileContents <- readChar(file, file.info(file)$size)
  },
  error = function(e) {
    print(paste0("Problem File: ", file))
  })
}

Created on 2021-07-28 by the reprex package (v2.0.0)
Here you will see my session info and that my locale collation is LC_COLLATE=English_United States.1252. If there was an issue with reading the file via readChar then it would have appeared in the output.
If you could post your results to this thread, I'm hoping it will help to isolate the problem. We encountered a similar problem due to special characters in a different study package as referenced in this commit: https://github.com/ohdsi-studies/Covid19VaccineAesiDiagnostics/commit/e0917506f41765d75c8db7577739333e4e4788be

in my case, the json files are at /usr/local/lib/R/site-library/Covid19CharacterizationCharybdis/cohorts/
Running you code I found problems with
[1] "Problem File: /usr/local/lib/R/site-library/Covid19CharacterizationCharybdis//cohorts//118.json"
[1] "Problem File: /usr/local/lib/R/site-library/Covid19CharacterizationCharybdis//cohorts//199.json"
sessionInfo()
R version 4.1.0 (2021-05-18)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Debian GNU/Linux 10 (buster)

Matrix products: default
BLAS:   /usr/lib/x86_64-linux-gnu/blas/libblas.so.3.8.0
LAPACK: /usr/lib/x86_64-linux-gnu/lapack/liblapack.so.3.8.0

locale:
 [1] LC_CTYPE=C.UTF-8       LC_NUMERIC=C           LC_TIME=C.UTF-8
 [4] LC_COLLATE=C.UTF-8     LC_MONETARY=C.UTF-8    LC_MESSAGES=C.UTF-8
 [7] LC_PAPER=C.UTF-8       LC_NAME=C              LC_ADDRESS=C
[10] LC_TELEPHONE=C         LC_MEASUREMENT=C.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

loaded via a namespace (and not attached):
[1] compiler_4.1.0

a call to file * shows that all files are ASCII except these four
/usr/local/lib/R/site-library/Covid19CharacterizationCharybdis//cohorts//118.json: ISO-8859 text
/usr/local/lib/R/site-library/Covid19CharacterizationCharybdis//cohorts//143.json: ISO-8859 text
/usr/local/lib/R/site-library/Covid19CharacterizationCharybdis//cohorts//199.json: ISO-8859 text
/usr/local/lib/R/site-library/Covid19CharacterizationCharybdis//cohorts//200.json: ISO-8859 text
two of them are the problem ones

looks like problem with 118.json is the ö in "Sjögren's syndrome"  and 199.json "Klumpke-Déjerine paralysis"

Thanks @alabarga - this is hugely helpful! I'll work to update the package.


Gui.Top is extremely long

Recommend changing this to a button-ish type and drop down for selecting the show options

design was not a priority, i know that options has an awful, that was the fastest way to put the options there, it will be improved

with new UI in release 0.2.0, that options has been redesign, and is a frame (pop up) centered


Invalid .meta/config.json file for hello-world exercise

The .meta/config.json file has invalid  JSON (trailing comma). See https://github.com/exercism/website/issues/563

Thanks a lot! I had this feeling that JSON allowed trailing commas at some point. Seems it had to be JavaScript, or even PHP, a mess :-) That's why I fell in love with Red's delimiter-less blocks :-)

Officially, JSON does not allow trailing comma's, but some JSON parsers secretly allow it. Ours doesn't :)

Fixed by #12 .


[List] : List subheader does not honor theme properties

It seems like <List subheader="Today"> always have text color porp in dark format. Switching to dark theme should take the color prop from light theme.

not applicable anymore.


[Foxy] - colcon build C++ with Visual Studio 2022 - change CMake version

We are setting up ROS2 in our labs, with Visual Studio 2022 already installed. I try to build Cpp examples with colcon, however, it seems like the version of Cmake that is provided along with Foxy does not yet know the VS2022, so the build fails.  I know that according to the official docs, only VS up to 2019 are supported, however in our case it would be beneficial to avoid installing another VS.
ROS2 Foxy was installed from binaries with Chocolately according to: https://ms-iot.github.io/ROSOnWindows/GettingStarted/SetupRos2.html
Could please suggest any option (colcon argument or env variable) to change the Cmake.exe path during the colcon build?
My steps:
The initial error pointed me to the C:\opt\ros\foxy\x64\Lib\site-packages\colcon_cmake\task\cmake\build.py script:
…
--- stderr: cpp_pubsub
Traceback (most recent call last):
  File "c:\opt\ros\foxy\x64\lib\site-packages\colcon_core\executor\__init__.py", line 91, in __call__
    rc = await self.task(*args, **kwargs)
  File "c:\opt\ros\foxy\x64\lib\site-packages\colcon_core\task\__init__.py", line 93, in __call__
    return await task_method(*args, **kwargs)
  File "c:\opt\ros\foxy\x64\lib\site-packages\colcon_ros\task\ament_cmake\build.py", line 59, in build
    rc = await extension.build(
  File "c:\opt\ros\foxy\x64\lib\site-packages\colcon_cmake\task\cmake\build.py", line 87, in build
    rc = await self._reconfigure(args, env)
  File "c:\opt\ros\foxy\x64\lib\site-packages\colcon_cmake\task\cmake\build.py", line 163, in _reconfigure
    raise RuntimeError(
RuntimeError: Unknown / unsupported VS version '17.0'
…

Which was easily "fixed" by appending '17.0': 'Visual Studio 17 2022' to the dictionary:
supported_vsv = {
                '17.0': 'Visual Studio 17 2022',
                '16.0': 'Visual Studio 16 2019',
                '15.0': 'Visual Studio 15 2017',
                '14.0': 'Visual Studio 14 2015',
            }

A new attempt to build now suggests that the Cmake provided with the Foxy does not recognize the VS 2022 C++ compiler:
…
--- stderr: cpp_pubsub
CMake Error: Could not create named generator Visual Studio 17 2022

Generators
  Visual Studio 16 2019        = Generates Visual Studio 2019 project files.
                                 Use -A option to specify architecture.
  Visual Studio 15 2017 [arch] = Generates Visual Studio 2017 project files.
                                 Optional [arch] can be "Win64" or "ARM".
  Visual Studio 14 2015 [arch] = Generates Visual Studio 2015 project files.
                                 Optional [arch] can be "Win64" or "ARM".
  Visual Studio 12 2013 [arch] = Generates Visual Studio 2013 project files.
                                 Optional [arch] can be "Win64" or "ARM".
  Visual Studio 11 2012 [arch] = Generates Visual Studio 2012 project files.
                                 Optional [arch] can be "Win64" or "ARM".
  Visual Studio 10 2010 [arch] = Generates Visual Studio 2010 project files.
                                 Optional [arch] can be "Win64" or "IA64".
  Visual Studio 9 2008 [arch]  = Generates Visual Studio 2008 project files.
                                 Optional [arch] can be "Win64" or "IA64".
  Borland Makefiles            = Generates Borland makefiles.
…

By checking the version C:\opt\ros\foxy\x64\Scripts\cmake.exe --version I could see that the default vesion of CMake is 3.18.2. I hoped, that by installing a newer version of Cmake (3.23.3) and by prepending the PATH env var with its corresponding path (C:\Program Files\CMake\bin) would be sufficient to force Colcon to utilize the different version of Cmake. But it wasn't :) .
Could please suggest any option (colcon argument or env variable) to change the Cmake.exe path during the colcon build?

I guess I have found a solution, though it was so close that I should have checked it before :/ .
Looking at C:\opt\ros\foxy\x64\Lib\site-packages\colcon_cmake\task\cmake_init_.py pointed me in the right direction.
"""Environment variable to override the CMake executable"""
CMAKE_COMMAND_ENVIRONMENT_VARIABLE = EnvironmentVariable(
    'CMAKE_COMMAND', 'The full path to the CMake executable')

So by setting env var CMAKE_COMMAND I could change the path to CMake executable:
set CMAKE_COMMAND=C:/Program Files/CMake/bin/cmake.exe

colcon now succesfully builds my Cpp packages.
At least I hope this information might be of use to someone else :) .
(I guess its partially related to #356 )
I wonder if it is possible to update the build configuration to incorporate a newer version of CMake into Foxy distribution?

CMAKE_COMMAND is also mentioned in the colcon usage output:
colcon --help
usage: colcon [-h] [--log-base LOG_BASE] [--log-level LOG_LEVEL]
              {build,extension-points,extensions,graph,info,list,metadata,mixin,test,test-result,version-check} ...

options:
...

colcon verbs:
...

Environment variables:
  CMAKE_COMMAND         The full path to the CMake executable
  ...

:facepalm: I should have read this more carefully :D . thanks!
However, I wonder if it is possible to update the build configuration to incorporate a newer version of CMake into Foxy distribution so that it works out of the box for everyone?

@anion0278 ROS on Windows has a toolchain affinity, instead of ROS on Linux which has a distro affinity. Our binaries are compiled with the default toolchain for Visual Studio 2019, so it is required to be installed to build nodes for Noetic and Foxy. We can't change the toolchain for foxy; otherwise we would break existing deployments.
With Humble, we are moving to VS2022. Both can be installed in parallel.

Wrong button. Leaving open for consideration since installing VS2019 is getting difficult.

I'd like to mention that if anyone intends to utilize the VS 22 compiler with a newer version of CMake, it's crucial to select versions prior to 3.24. Specifically, I would recommend using version 3.23.3 (refer to the related issue: https://github.com/dusty-nv/jetson-containers/issues/181#issuecomment-1216802257). The newer versions of CMake introduce ament-related issues when building Turtlebot3 from source (Failed to find exported target names). Alternatively, it seems to be possible to apply a fix for ament: https://github.com/ros-planning/moveit2/issues/1570#issuecomment-1272390429.
To avoid confusion, I have edited CMake version in my initial message.


Support arm64 mongo releases automatic download

Add support for downloading arm64 mongo builds.
I went through the release archive and constrained automatically downloading arm releases to only existing ones.
Although rhel82 is in the list of supported os'es, it can't be provisioned yet as osNameFromOsRelease can't parse that name.

Merged, thank you, this will be rolled out with the new release


Add django-reversion

This PR fixes #430 by installing django-reversion.
When this change makes its way to Staging and then to Production, we'll need to run both of the following commands:

python manage.py migrate

for reasons I don't understand, migrations related to django-reversion are listed among the migrations when I run showmigrations, even without me having first run makemigrations. So, there are no migrations included in this PR. It seems to work fine anyways.

python manage.py createinitialrevisions

By adding django-revision to INSTALLED_APPS and to MIDDLEWARE, this ensures that all modifications made to objects via our views (and perhaps also via the admin interface, but not via the shell) are added as revisions. By setting BaseModelAdmin to inherit from reversion.admin.VersionAdmin, this adds a "History" button on all Change {Model} pages in the Admin area, where all modifications to the object in question are listed. It also adds a "Restore Deleted {Model}s" button allowing a user to view and to restore deleted objects.

for reasons I don't understand, migrations related to django-reversion are listed among the migrations when I run showmigrations, even without me having first run makemigrations

Third-party modules can come with their own migrations if they need to make changes to the database (Which reversion definitely does).
https://github.com/etianen/django-reversion/tree/master/reversion/migrations

Note that as I tried to run createinitialrevisions on staging, the command took too long and ssh quit on me:
root@da42ee733a02:/code/django/cantusdb_project# python manage.py createinitialrevisions
Creating revisions for century
- Created 54 / 54
Creating revisions for chant
client_loop: send disconnect: Broken pipe
client_loop: send disconnect: Broken pipe

I'll try running it again and see what happens. Just noting that we may need to babysit this and occasionally press a key or two while running this on Production. Not sure whether caffeinate would help. Maybe I should do some research into how long it takes for ssh to disconnect, and/or in what situations it does this.

update: I just tried running createinitialrevisions again on staging, and it seems the command ran to completion in the background:
Creating revisions for century
- Created 0 / 0
Creating revisions for chant
- Created 0 / 0
Creating revisions for differentia
- Created 0 / 0
Creating revisions for feast
- Created 0 / 0
Creating revisions for genre
- Created 0 / 0
Creating revisions for notation
- Created 0 / 0
Creating revisions for office
- Created 0 / 0
Creating revisions for provenance
- Created 0 / 0
Creating revisions for segment
- Created 0 / 0
Creating revisions for sequence
- Created 0 / 0
Creating revisions for RISM siglum
- Created 0 / 0
Creating revisions for source
- Created 0 / 0

Going in the admin interface, I can view a version history for sources. For all the sources I've checked, this is just the "Initial version." I've tried deleting several things and then recovering them, and it works just fine. So it seems that everything is as it should be.

 It will keep the session even if disconnected

Yes. Also screen -ls will list all the open screens;  Ctrl-A-D will detach a running screen and put you back at your main shell, then you can reconnect when you want to rejoin the session.

Also tmux is a newer, hipper alternative but I don't have it in my muscle memory yet. You might want to learn that if this is a new world.


Script tag in lit html not working

Description
The script tag in lit html is not executed
Steps to Reproduce

Write this code

html`<script type="text/javascript">alert('Foo')</script>`;
html`${unsafeHTML`<script type="text/javascript">alert('Foo')</script>`}`;

See this output...

Expected Results
An an alert box with „Foo“ should be pop up.
Actual Results
Nothing happens
It looks like that script tags are not executed when they are added via innerHTML (I saw the usage of innerHTML serveral times in the lit source).
https://stackoverflow.com/questions/51101473/script-inside-shadow-dom-not-working
Maybe that’s the reason?

But why would you want to do this instead of dynamically loading scripts for execution?

This is working as intended. Cloned scripts are not evaluated by the browser, and this is critical to our security story.

I want to do a webcomponent for Google Analytics.
The idea was to create with LIT the script tag to load the GA JS file.

Not sure how this rates against the security story, but you can use script nodes directly in lit-html, a la: https://stackblitz.com/edit/mytohf-frfwbd?file=my-element.js

Well, creating the script-tag in lit would be nice... but I’m using plan b, now
  updated(changedProperties) {
    if (['_id', '_accepted'].some(key => changedProperties.has(key)) && this._id && this._accepted === true) {
      this._initTrackingScript();
    }
  }
  _initTrackingScript() {
    if (!document.querySelector('#cy-ga-script')) {
      const scriptFileEl = document.createElement('script');
      scriptFileEl.setAttribute('async', '');
      scriptFileEl.setAttribute('id', 'cy-ga-script');
      scriptFileEl.setAttribute('src', `https://www.googletagmanager.com/gtag/js?id=${this._id}`);
      document.head.appendChild(scriptFileEl);
      _gtag('js', new Date());
      _gtag('config', this._id);
    }
  }

Would be nice if I could add the script tag with lit, after I want do the banner which asks for the tracking permission also with lit. But the way I do it now works great. It’s just a little bit inelegantly ;)


ISSUE 6 - App store review

#6 App Store review
Notes
The prompt appears every 10 times the user opens the application.

@georgealegre fixed conflicts!

@georgealegre ready for merge! no conflicts


Hopefully fix formatting of bulleted list.

A bulleted list in the Installing C++ page looks fine when rendered in GitHub as markdown, but I guess Redcarpet or something might be parsing/rendering slightly differently, because it shows up like this on the website:

I am guessing that my fix will fix things but I didn't actually test it out.

Thanks!

Thanks @kytrinyx!
I've been noticing more instances of this same bug--at least once more in the C++ docs, and once in the JS docs--which makes me wonder if the problem should actually be fixed further upstream? I mean, GitHub does render the markdown in a way that will lead contributors to assume things will look okay if they don't put an empty line between a paragraph and a bulleted list...
This bug seems particularly odd to me given that I'd assumed GitHub uses a lot of the same underlying code as exercism, like Redcarpet, to render the markdown. Ah well. I could also just continue to file PRs like this when I run into the bugs, it's all good either way. :grin:

If you have time to look at why the missing blank line causes the problem here, that would be great.

I'm pretty sure I relied on github's rendering to check the markup on these docs.  Thanks for the fixes!


[Bug]: Broken link on 'React Server Components' page RE: 'not supported on the client'  (why-cant-i-use-async-components-on-the-client)

Summary
At the bottom of the 'React Server Components' page is the line:

Since async components are not supported on the client, we await the promise with use.

But the https://react.dev/reference/rsc/server-components#why-cant-i-use-async-components-on-the-client link doesn't actually go anywhere.
Page
https://react.dev/reference/rsc/server-components
Details
No response

PR submitted.


fix: update to eye v10.11.2

fix: update to eye v10.11.2

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

npm package (@latest dist-tag)
GitHub release
v16.10.2

Your semantic-release bot :package::rocket:


How Can I Use ClientCertificates in HttpClientFactory.CreateClient

From @kugarliyifan on August 28, 2018 17:32
i use Microsoft.Extensions.Http library ,in my scene,only before call SendAsync function ,i can know use which X509Certificate,it just like that,i has many customer config data,when call the webapi,need to send the the customer's clientCertificate before call httpclient's sendasync function ,,but HttpClientFactory.CreateClient can't pass the new handler to client,   how can i solve the scene???any one can help me please
Copied from original issue: dotnet/corefx#32005

From @davidsh on August 28, 2018 17:47
@karelz - Where is HttpClientFactory repo?  This needs to be moved there.  I think it is ASP.NET.

From @kugarliyifan on August 28, 2018 17:51
@davidsh    i view the HttpClientFactory repo issues,,,too few member,,,,,,,

Client certificates need to be passed to the HttpClientHandler when it's constructed via https://docs.microsoft.com/en-us/dotnet/api/system.net.http.httpclienthandler.clientcertificates?view=netframework-4.7.2
I'd recommend creating a separate named client for each different cert you want to use. You can configure the 'client handler' for each named client separately.
services.AddHttpClient("foo").ConfigurePrimaryMessageHandler(() =>
{
    var handler = new HttpClientHandler();
    handler.ClientCertificateOptions = ClientCertificateOptions.Manual;
    handler.ClientCertificates.Add(...);
}


ComposedFixture: Get this working for Couchbase and ServiceFixture

This scenario needs some TLC.

I renamed the fixture StartInAction() methods to StartAsComposed() to make it clear what these are for and I also added this method to the fixtures that needed them.  I also enhanced the unit tests to demonstrate how to do this.


Updating different files in the same repo at the same time return 409 conflict error

I am trying to use the githubpy library in my setup. My requirement is to update multiple different files in the repo. The request to update the files in git happens almost at the same time.
Sample code is like below. This is called with different file names from different places.
github_client = github.GitHub(access_token=access_token)
file_content = base64.b64encode(some_data).decode()
response = (github_client.repos(owner)(repo)('contents')(file_name)
                .put({'message': message,
                      'content': file_content,
                      'sha': file_sha,
                      'branch': branch}))

The first file update is successful but the subsequent file updated fail.
With this we always get the error. Error 409: Conflict
May i know is there a limitation on the githubpy library in doing this operation ?

Hi,
githubpy is only a wrapper for the REST APIs. You can use GitHub(access_token='xxx', debug=True) to log and execute the curl command manually. If curl failed then there is something wrong with the args.


feat: added pagination to hosting & app idea pages

#19 Added Pagination to hosting and app ideas pages

How about showing?

Showing 10 from 200 entries?  kind of message?

I will merge this for now, lets track the above feature in separate issue.

Thanks @yogesh7401 Wonderful work.


Error

Hey, it looks like BeReal changed something again. Posts on all third-party sites are no longer loading. Can you please try to fix it? :)

Sadly this problem has no easy fix. A header Bereal-Signature is being enforced now, and the only feasible way to crack this header is to decompile the application and reverse engineer it. I'll try to reverse engineer it, though I cannot promise anything!

Hey @Jabolol, if you haven't found a solution yet, the GitHub user BZHugs provided a solution for the BeFake project. I recommend reaching out to him on X so that he can give it to you as well

Hello! Thanks for letting me know. Nonetheless I had already reverse engineered the BeReal app and got my custom headers, which are already being used here.
I used this downtime to create a new and robust API system, based on the /api/execute endpoint, yet all of the heavy lifting is done in client.ts, so the app is working once again.
Thanks for your input, much appreciated, and sorry for the inconvenience!

Hey @Jabolol,
I think it broke again, can you fix it please :)


DBBigint and DBInt scaffolding as NumericField

I think the use cases of just having these scaffold as a TextField is far more than that of a NumericField
Not so much an issue per-say, there are several more scenarios where one would not want their Int or BigInt nicely formatted with commas
I just see myself having to replace these fields with TextFields via a getCMSFields() override soooo much more often than I would vise-versa
Maybe opinionated, maybe not :)

Cut the baby in half: how about using injector services to define what gets scaffolded? The you can override.

Yep, would be happy to accept a PR which makes the scaffolded fields injectable via their own service name. Until then: Closing as a low priority enhancement


Rpc contracts services should be disabled before node sync up complete

Description:
When new BP/FullNode first launch or restart and in sync up block process, rpc contracts service should be disabled to avoid sync up error and rpc request error.

Close issue due to fixed.


How to contribute?

hey,
are you looking for contributors? do you have any item that i can start off with?

@kvijayva Thanks!
https://github.com/ofek/hatch#contributing meta would be easy. that one issue is the hardest hence the bounty

@ofek ok, i can take a look.
do you have any feature enhancements in mind that i can pick up, apart from this issue?


Sway doesn't start with Lima or Panfrost

First of all, many thanks for your help so far. I use a ROCK64 single-board computer (RK3328 Quad-Core ARM Cortex A53 64-Bit Processor)

ARM Mali-450MP2 Dual-core GPU
OpenGL ES 1.1 and 2.0, OpenVG1.1

I use actual Alpine Linux with Mainline Kernel 5.6.2.
In kernel config:
CONFIG_DRM_LIMA=m
CONFIG_DRM_PANFROST=m
All other graphics settings are the same as in the Kernel configuration from Armbian.
If i tried Lima for Mali 400/450 (and also Panfrost but it seems that Panfrost is not supported from Mali 400/450) i get the following errors:
[backend/drm/renderer.c:20] Failed to create GBM device
[backend/drm/backend.c:196] Failed to initialize renderer
[backend/backend.c:198] Failed to open DRM device 9
[backend/drm/backend.c:152] Initializing DRM backend for /dev/dri/card1 (lima)
[backend/drm/drm.c:50] DRM universal planes unsupported
[backend/backend.c:198] Failed to open DRM device 10
[backend/backend.c:339] Failed to open any DRM device
[backend/noop/backend.c:51] Creating noop backend
[sway/server.c:46] Unable to create backend
I tried to run Sway with xdg_runtime_dir.sh and without it (with elogind+dbus) same result.
With rockchip_dri.so I get the same errors but Sway starts and Sway CPU usage is good.
When i use swrast_dri.so (without rockchip_dri.so) I get the same errors but Sway starts and Sway CPU usage is bad: 40-90%.
I noticed it seems that there is no /dev/drmX.
Permissions and ownership seem to be ok.
User is in the video group.
I have used also mesa-dri-lima and mesa-dri-panfrost (and also tried all other possibly missing packages from Alpine Linux mirror).
From dmesg:
[    9.557182] lima ff300000.gpu: IRQ pmu not found
[    9.558192] lima ff300000.gpu: IRQ ppmmu2 not found
[    9.558792] lima ff300000.gpu: IRQ ppmmu3 not found
[    9.559362] lima ff300000.gpu: IRQ ppmmu4 not found
[    9.559953] lima ff300000.gpu: IRQ ppmmu5 not found
[    9.560502] lima ff300000.gpu: IRQ ppmmu6 not found
[    9.561041] lima ff300000.gpu: IRQ ppmmu7 not found
[    9.561615] lima ff300000.gpu: gp - mali450 version major 0 minor 0
[    9.562433] lima ff300000.gpu: pp0 - mali450 version major 0 minor 0
[    9.563238] lima ff300000.gpu: pp1 - mali450 version major 0 minor 0
[    9.564168] lima ff300000.gpu: IRQ pp2 not found
[    9.564687] lima ff300000.gpu: IRQ pp3 not found
[    9.565199] lima ff300000.gpu: IRQ pp4 not found
[    9.565707] lima ff300000.gpu: IRQ pp5 not found
[    9.566223] lima ff300000.gpu: IRQ pp6 not found
[    9.573411] lima ff300000.gpu: IRQ pp7 not found
[    9.580299] lima ff300000.gpu: l2 cache 8K, 4-way, 64byte cache line, 128bit external bus
[    9.587547] lima ff300000.gpu: l2 cache 64K, 4-way, 64byte cache line, 128bit external bus
[    9.596117] lima ff300000.gpu: bus rate = 163840000
[    9.603650] lima ff300000.gpu: mod rate = 163840000
[    9.611439] [drm] Initialized lima 1.0.0 20190217 for ff300000.gpu on minor 1
sway+lima.txt
sway+swrast.txt

Also transferred to wlroots, because it's related to that instead of sway.

Yeah, looking at Mesa a bit more, I think it needs to be built with -D gallium_drivers=kmsro,lima and I think that'll work. Running sway with WLR_DRM_DEVICES=/dev/dri/card0 may be required too.

With rockchip_dri.so (from Alpine Linux mesa-dri-kmsro package, I think it's proprietary) I get the same errors but Sway starts and Sway CPU usage is good, but with mouse sway works only with "WLR_NO_HARDWARE_CURSORS=1".
I haven't build mesa myself. Actual with and without "WLR_DRM_DEVICES=/dev/dri/card0", and without "xwayland disable" in sway config, lima works only with firefox (according to the information in Firefox, wayland and lima is in use with firefox, but it's still a lot slower than with Xorg!) but according to the sway log, lima does not work with Sway, with "xwayland disable" lima does not work with firefox (firefox information says x11 is in use and not wayland) and also not with sway.
firefox+xwayland_disable.pdf
firefox-without-xwayland_disable.pdf

I think it's proprietary
Nah, it's in upstream mesa.
The kmsro is basically a glue between display controllers and render-only GPUs. When you use rockchip_dri.so, it loads the lima driver in the background.

but according to the sway log, lima does not work with Sway
We currently don't handle render-only GPUs properly in wlroots which is why it says it fails, but because of kmsro, we should be able to do everything through rockchip's device node, and shouldn't need to go through the lima device node directly. This is why I said to use WLR_DRM_DEVICES to just use the rockchip node (/dev/dri/card0).

I don't really know much about any firefox-related issues here.

Ah ok that's why it works with firefox, at first I didn't understand that.

We don't really handle some of these split display controller (/dev/dri/card0 : rockchip) and GPU (/dev/dri/card1 : lima) configurations too well at the moment, unless they appear to us as one device.
Mesa does seem to have a rockchip_dri.so DRI driver, which maybe will work? I don't know if alpine has a package providing that. You may have to build Mesa yourself.

the split works well for etnaviv (render node) and dcss or mxsfb (display controllers so  i wonder what the difference is.

the split works well for etnaviv (render node) and dcss or mxsfb (display controllers so i wonder what the difference is.

I mean that when we currently assume that every DRM device has at least some modesetting capabilities. When we enumerate and attempt to open every GPU, there will be an error in the log saying using the GPU failed, but will continue on working with the other node + kmsro.
There is more we could do to handle render-only GPUs more gracefully.
Anyway I'm going to close this, because I think they've got it working.


Extraneous create-react-class dependency

react-redux still has create-react-class  as a dependency but doesn't use it anymore.
Therefore it could be removed.

It's used right here: https://github.com/reactjs/react-redux/blob/fcc28ff487b85e64da6d0b3d0c9014597d3585cf/test/components/connect.spec.js
It should be dropped down to a devDep.


io_placer.py fix for ZeroDivisionError

If there's a side that doesn't have any ios the script breaks because of ZeroDivisionError

You literally just could've reviewed #1164


Use new Redigo repo

The original redigo repo (github.com/garyburd/redigo) has been archived. It is now being actively supported at github.com/gomodule/redigo. I updated all references to garyburd/redigo to use gomodule/redigo instead.

Also, it appears that the various gorilla/sessions stores (CookieStore and FilesystemStore) when MaxAge <= 0. Figured I'd get that into this PR too.

Thanks for the fixes, much appreciated.


Content authors cannot read text "config" node

I customize the "text" component by copying it in my project. Specifically, my edits related to the default enabled plugins during editing.
Everything works if you are Admin, but if you use an user member of "content-authors" group (or more specifically "contributor" group), you can't see any customization, so you cannot use configured plugins.
Admin

Content Author

I think this problem is due to a too restrictive deny permission that prevents users from reading the configuration of the inplace editing node (apps/*/config/*)

I think it's necessary to change the text component by using a configuration structure like this:

Thanks @kele23 for your report - we can confirm the described behavior. The solution you provide solves the issue, however, it should work with the existing config set up, as documented at https://docs.adobe.com/docs/en/aem/6-3/administer/operations/page-authoring/rich-text-editor.html.
Therefore this is seen more as an AEM product issue. If you would like to follow-up you could create a support issue at https://helpx.adobe.com/marketing-cloud/contact-support.html

@kele23 - I'm closing this issue given the previous comment regarding it being a product related issue.


Difference on the acoustics when changing materials?

Hi all,
I don't know if it is just me, but I cannot find a proper difference in the room acoustics when changing materials. For example, metallic materials should turn to a more reverberant environment because the attenuation rate is lower; with carpet material, the other way around.
I have also tried the Custom material and, in my opinion, I can't see a difference.
Am I doing something wrong? Because that is not working for me.
Thank you!

Did you pre-export and generate the probes after making the changes?

I pre-exported the scene. I haven't generate the probes, I thought it didn't matter if I was using Real-time?

We are working on a small project with 4 people and have the same issue. Sometimes changes to materials etc. seem to be ignored by steam audio (e.g. I couldn't hear a source as soon as it's around a corner even when I set up the scene for maximum reverb...)
One time it worked for me when I used git clean directly after I pulled changes from github, but another project member reported that he tried that and still has the problem.
Maybe some temporary/cached files aren't overwritten and are still reused every time?

Update: For him worked "git clean -fx", too. Maybe this can be a workaround for @ariadnasc if the issue has the same cause. (Or if he doesn't use git: cleaning all unity-generated temporary files in the project directory)

Yeah, you're right about the probes. After pre-exporting it on mine it seemed to work just fine between carpet and concrete. It echoed for concrete, and was pretty dry on carpet. Do you have your reverb enabled on your listener?

@ariadnasc Oh yeah real-time you don't need the probes. How complex is your scene? Is all of your geometry the same material in the space?

@Luux I tried doing that but it still doesn't work. I have also tried with different audios but nothing seems to change.
@wderstine right now I am testing with a simple room made of cubes, and a wall in the middle (another cube) to test the occlusion. And yeah, I am trying Custom material, changing the absorptions to see if there is any difference... but still none.

Did you use the git command or did you delete the files manually? If case 1: did you use a .gitignore for unity (like https://github.com/github/gitignore/blob/master/Unity.gitignore)?
(Just in case: did you add the Phonon Listener script and set it to realtime, too?)

@Luux I erased them manually because I am not using git.
I have the Phonon Listener added in an FPS Controller. I have the Real time reverb selected.

Is your geometry script attached to a parent of all of your cubes with export all children checked?
Btw which version of Steam Audio do you have? I just recently had to update to the 2.0-beta.4 release from .2 because I was having some issues and it seemed to help. Although, I have the .1 release on another comp and it was working fine.  I don't know if it'll help or not.

@wderstine All my cubes are attached to the same parent, yes. And I have that option checked. I really think I am not missing anything, I followed the manual step by step.
I started testing Steam Audio 2 or 3 days ago so I have the 2.0-beta.4 as well.

I am using Unity on a OSX, I don't know if that changes anything or there are some incompatibility problems? My Unity is 5.5.2f1 free version.

Yeah, I'm always forgetting little things like that so I just try to double check. I'm on a PC so if there is some issue with Mac then I won't be able to help. I'm gonna keep looking into it and see if I find anything that isn't OS related.

@ariadnasc Does increasing the Indirect Mix Fraction on the Phonon Listener help? We're also trying to reproduce the issue, and to figure out if this is a macOS-specific thing or not.

@lakulish have you found out if it is an macOS thing?
However, I am going to try to increase that parameter this afternoon and I'd let you know if that helps.

@lakulish I have increased the Reverb Mix Fraction (I couldn't find the Indirect Mix Fraction in my Phonon Listener) and I don't appreciate any difference. I used a custom material with all the absorption parameters to 0 and another with all of them to 1. I also tried with Wood and Metal but no difference

@ariadnasc I haven't been able to reproduce this on macOS. I tried a custom material with all parameters set to 0, followed by all parameters set to 1, and things seem to be working fine. Can you confirm whether you're pre-exporting the scene after changing the materials?

@lakulish yes, I am pre-exporting the scene after changing the materials.

@ariadnasc Can you try deleting the .phononscene file (which is created under Assets/StreamingAssets) and pre-exporting again? Also, are you modifying the default material, or a material component attached some geometry? If the latter, are you modifying the materials for all geometry that has a material component?

@lakulish I deleted the .phononscene file but still the same.
I have a room made with cube geometry and a wall, all of them linked to an Empty GameObject (which is the parent). I clicked on Export all children. All materials are the same for all the different walls of the room.

@ariadnasc Can you try the following scenario?
Remove Phonon Material from all children object and use global material to change material settings. Pre-export scene. Does that change reverb, now?
There is currently a bug we are fixing when material is attached to children objects. Basically, either just use global material or do not use Export All Children option until we provide a fix.

@achandak I have already just one material in the parent object (not in the childrens), and the reverb does not change.
If I unselect the Export All Children option it doesn't work because then Vertices and Triangles are 0.
Am I understanding it wrong?

@ariadnasc - No, your response is fine. Do the following then.

Select Export All Children.
Remove Phonon Material from the GameObject with Phonon Geometry. (now, only Global Material is used, see Window > Phonon for Global Material settings).
Change the Global Material and pre-export scene. Change it again to see if reverb changes.

@achandak I just did that, and used Custom reverb (one set to 0 and another set to 1) and it's just the same. It is very weird. I would try it on a windows PC to see if it works, but I don't have any.

@ariadnasc - At this time, we do not have any more ideas on what might be going on. Our next update will be out next week, try it out and see if that fixes your issue.
If you continue to face the same issue and are willing to share a project where we can reproduce the issue, we can look into that further.

@achandak That's fine, thank you very much.
If the problem persists, I will upload a sample project so you can reproduce the issue.

@ariadnasc - We have fixed issue with materials on macOS. This was a problem on other platforms as well, but may not manifest itself depending on how your scene geometry is organized.
See release notes here.


cesium 3d tiles cannot be seen after packing

I have successfully packaged the application with cesium3d tiles, and running the packaged application locally can see cesium3d tiles, but cesium3d tiles do not display after migrating to the server. ue5.3 cesium form ion

We will need a lot more detail in order to help you. For example:

What version of Cesium for Unreal? (5.3 is the Unreal version, not the plugin version. Go to Edit -> Plugins -> Geospatial to find the plugin version)
What sort of tileset? Where is it being loaded from?
What platform?
Are there any messages in the log?

But this is the wrong place for such matters. GitHub issues are for confirmed bugs and concrete, actionable feature requests. Please follow up on the community forum instead.


Feature Request: Improve VSCode plugin and publish it in the Marketplace

Implement the same (or most of) features that IntelliJ have

Ah, there is an issue already. Because I was wondering where the "Incredible IDE Integration" in VSCode is...
Maybe a note would be good, that some parts of the "Incredible IDE Integration" is not available in all Plug-Ins (see https://pkl-lang.org/index.html).

For preference:
Same issue, different repo.

I wonder what's holding this back? Are there actually blockers for this? Can we speed up the process?

This is something that's progressing; I get that it's frustrating that this is so slow. Apologies here; we will hopefully provide updates soon.


Cant set route middleware and prefix

It would be nice to control which middleware to use on the route and its prefix.
Sending a PR for this in a minute

$ git push origin issue#27
remote: Permission to ichtrojan/laravel-location.git denied to cacpmw.
fatal: unable to access 'https://github.com/ichtrojan/laravel-location.git/': The requested URL returned error: 403


How to run or build it on a Raspberry Pi?

Hi,
while the provided release binaries work perfectly fine on macOS (Darwin), I can't get it to work on a Raspberry Pi (Raspberry OS, Debian GNU/Linux 11 bullseye). The linux binary is not compatible.
I tried to install it manually by first installing golang with sudo apt install golang and afterwards I tried to run
go install github.com/mattn/bsky@latest
package github.com/mattn/bsky@latest: cannot use path@version syntax in GOPATH mode

Any help appreciated.

Got it to work, at least on a Raspberry Pi 3 (not 4 though).
Needed steps:

Uninstall golang from apt-get (because it's outdated)
sudo apt-get remove --purge golang && sudo apt-get autoremove
Install latest golang version manually

curl -o go.tar.gz "https://dl.google.com/go/$(curl -sS https://go.dev/VERSION?m=text | head -n1).linux-armv6l.tar.gz"
sudo tar xf go.tar.gz -C /usr/local
echo 'export PATH=$PATH:/usr/local/go/bin' | sudo tee -a /etc/profile
source /etc/profile

install gcc for arm architecture
sudo apt-get install gcc-arm-linux-gnueabihf
checkout project and build it with make

git clone https://github.com/mattn/bsky.git
cd bsky
CC=arm-linux-gnueabihf-gcc make


Comprehensive Source Tracking

Bevy code can get very spread out. It's not uncommon for very well separated pieces of machinery to have to coordinate, often across crate boundaries. Bevy has great tools to accomplish this sort of coordination, but it is often hard to inspect.
The first attempt at improving runtime inspection was made in https://github.com/bevyengine/bevy/pull/14034, which added optional tracking of exactly where every component is accessed or mutated, as an extension of the change detection system. But there's still several other places we don't track this information.

[ ] #16776
[ ] #15607
[ ] #16047
[ ] Add source location tracking to resource access, though this may happen as part of reunification with components.

Each of these work items should be tackled one PR at a time to help them get merged in smoothly :)

Thanks for helping with triage & de-duplication @SpecificProtagonist <3.


Update razer-synapse to 1.47

Thank you for the contribution. It was merged directly as ddbcca1a18e644c87ea32c8bbd9576b847058f38 to keep commit history cleaner. Your contribution is still credited to you.


Move locale files into locales folder

This will keep locale files more organized in one place.
@fetlife/product

@reneklacan great initiative! Thanks for cleaning it up!


Web Application

In the past two meetings we have discussed adding some kind of API to the website for searching our classified data. One obvious way of doing this would be to postprocess our classified data to generate something like a static JSON database which could be used for a dynamic Web interface.
I have a little bit of experience using the Backbone.js library which provides a Model-View-Presenter interface over a JSON REST API. This library should definitely be on our radar, though there are of course alternatives with different programming models.
In response to @lewismc's comments today, note that the output format of classification (i.e., ENVI headers) does not necessarily tie us to using that format for public-facing data. It would be simple enough to convert to other formats after the fact.
Assigning to @xiaomei7 for now to go along with the website stuff. We probably won't need to worry about this until we have made more progress with classification and correlation, although it would be possible to implement some sort of skeleton code.

Setting as Blocked in the project board for now. We will need more details on just what we want to be able to search for. An example mentioned in Friday's meeting was "images that contain a particular surface cover type".

one possible parameter to categorize imagery and classifications is to query geographic boundaries. GDAL uses a standard text serialization to represent these. this would allow the user to search for images that correspond to a particular region. note, however, that would arguably be beyond the scope of our original project specification.

Unassigning from @xiaomei7 and reassigning to myself per https://github.com/capstone-coal/capstone-coal.github.io/issues/6. This issue is still blocked, but we can start thinking about possibilities now that we have an AWS instance.

closing this issue per today's meeting as @lewismc is implementing a science data system.


Add CRC32 to DynamoDB responses

Hi, when we point aws-sdk-v2-go's DynamoDB client at the Moto server, the SDK complains that the responses don't have the X-Amz-Crc32 header. This PR adds the decorator which populates those checksums on the responses from the DynamoDBv2 API.

 The tests were failing because of an existing issue where the CRC32 wasn't computed properly in Python2.
This should be fixed now... 🤞

Amazing, thank you!

This is now part of moto >= 1.3.17.dev259


TE: Update timeseries client package to be consistent with comparison client package. Refactor shared code into ResponseParserUtils.

LGTM


What about books with videos excerpts?

Will we be able to download the video as well?

Duplicate #140, kept the newest one.


chore(wallet): test aa wallet support article feature

https://github.com/thematters/matters-web/pull/3315


[BUG] iOS: Converting into WEBP via attachment results in larger files, but pasting is fine.

I hit this as well. the original size in album was 1.2 MB but it results in 9.3 MB.
Seems this happen in mobile side. Desktop seems work fine.

Closing this for now.
New toggle in 1.2.3 allows files to be at bay.  https://github.com/xRyul/obsidian-image-converter/issues/83#issuecomment-2466454350


SVG options are not optional

Thanks for adding TypeScript typings!
I just created a circle and wanted to set the stroke but discovered that the options parameter requires almost all fields to be supplied. Is this expected behaviour?
This is the interface for SVG Options:
export interface Options {
  maxRandomnessOffset: number
  roughness: number
  bowing: number
  stroke: string
  strokeWidth: number
  curveTightness: number
  curveStepCount: number
  fill: string | null
  fillStyle: string
  fillWeight: number
  hachureAngle: number
  hachureGap: number
  simplification?: number
}

For some reason, only simplification is optional.

Your'e right. My mistake. I made it so because internally all values are available using default values.
I did not consider people using these options. Will fix it shortly. (On holiday this week)
Thanks for reporting.

Cheers, @pshihn! Want me to just make a quick PR for Options? I haven't delved too much into roughjs yet, so I don't know of any other related issues, but I'll be happy to add some question marks and submit it as a PR 😉

Resolved with https://github.com/pshihn/rough/pull/83

Available in v2.2.4


Small Windows HTTP Server package improvements

@blairmcg using this server I was able to get 3000 req/sec in example1 removing the log to transcript, pretty impresive. And it appears to be stable without memory consumption. I think that it can be used as replacement for swazoo. Do you have any comments about your motivation to include this package?

@fxgallego, I started working on it because Andy needed a high performance HTTP server. It didn't get very far because the need went away, and also because I spent more time improving the type library code generation in order to generate a better wrapper than I did working on the package.
I tend to think of external component wrappers having 3 levels:

Basic ExternalLibrary library wrapper for the API, and ExternalStructures for any structures it uses (in this case, a lot). This is a low-level layer, and hard to use. I generally try to auto-generate these from a type library, building one of those if necessary by creating IDL from the C header files. Doing a good job of this requires adding quite a lot of extra metadata attributes to the information in the .h files. The advantage of doing this is that regenerating the wrapper to reflect changes is easy, and it avoids mistakes in sizing, choice of external types, etc, that often result in memory corruption and crashes when trying to use the API.
Hand-written extensions and refactorings of the first level wrapper to make it more usable from Smalltalk, e.g. marshalling between standard Smalltalk types and byte structures etc. This layer should be easier and more natural to use, but still doesn't have any higher-level abstractions, so is inconvenient and verbose.
Proper abstractions for the contents in the library that hide all the external interfacing details so that the consuming developer is not aware that the component isn't implemented in entirely Smalltalk, and has convenient high-level classes that are easy and natural to use.

I would say this work is somewhere between 1 and 2. I have a type library that defines the complete API (this isn't pushed to GitHub). I'd started experimenting with the generated API and adding bits of level 2 along with the examples. I didn't explore the whole API, so its likely that some parts of the generated API have not had all the necessary metadata added to the typelib to generate the best quality low-level API functions/structures. I also haven't fleshed out or written examples for all the functionality of the API/component, as it is quite extensive. I haven't even started on higher-level abstractions - e.g. it needs a proper HttpRequest class that hides most of the details.
There is still a lot of work to do to make this a replacement for Swazoo, although maybe it would be possible to replace some of the lower-level layers of Swazoon with this (I don't know how practical that would be).

@blairmcg thanks for your detailed explanation.
We currently use swazoo to serve a REST API. We've patched swazoo in several places in order to rid off several bugs causing crashes and hangs. Sadly, we can't figure out the reason of this problems because they aren't easy to reproduce so we don't know in which part is the problem, could it be in process, error handling, sockets, semaphores.
We'll try to implement our API using this server. Maybe we can borrow some ideas from Swazoo for the 3rd level layer or directly replace the swazoo low level server loop. I let you know if we need some extra functionality that's supported by the api and currently not implemented in Dolphin.

BTW, I've found an easy way to reproduce the Dolphin crash with a swazoo server being stressed by the apache benchmark tool. Below the error details. This two errors where generated by two different sessions. I don´t want to report an issue for this but perhaps this helps to identify a hidden bug. Please, don't hesitate to ask me for more details.
********************************************************************************
************************** Dolphin Crash Dump Report ***************************

10:04:17, 10/10/2017: Dolphin7.exe caused an unhandled Win32 Exception C0000005
at 10002C02 in module 10000000 (C:\Users\scalvo\dev\Dolphin\DolphinVM7.DLL)

*----> Exception Parameters <----*
00000001
0697FFFC

*----> CPU Context for thread 0x1c20 <----*
EAX = 06980000 EBX = 10060390 ECX = 1078AA30
ESI = 10787810 EDI = 10787810 EIP = 10002C02
ESP = 00C0F774 EBP = 00C0F790 EFL = 00210246
CS = 0023 SS = 002B DS = 002B
ES = 002B FS = 0053 GS = 002B

*----> VM Context <----*
Process: {C17F0210:size 276365657 words, suspended frame 0363075D, priority 3, callbacks 0
last failure 0:nil, FP control 80017, thread 00000000}
Active Method: WS2_32Library>>accept:addr:addrlen:
IP: 070B68B0 (0)
SP: 03630784
BP: 03630750 (276365633)
ActiveFrame: {0363075C: cf 1078B150, sp 00000000, bp 03630750, ip 0, WS2_32Library>>accept:addr:addrlen:}
	receiver: a WS2_32Library
	arg[0]: a ExternalHandle(00000860)
	arg[1]: a SOCKADDR_IN
	arg[2]: a DWORD

New Method: WS2_32Library>>accept:addr:addrlen:
Message Selector: accept:addr:addrlen:

*----> Stack Back Trace <----*
{0363075C: cf 1078B150, sp 00000000, bp 03630750, ip 0, WS2_32Library>>accept:addr:addrlen:}
	receiver: a WS2_32Library
	arg[0]: a ExternalHandle(00000860)
	arg[1]: a SOCKADDR_IN
	arg[2]: a DWORD

***Unable to generate complete exception report (c0000005)

***** End of crash report *****

********************************************************************************
************************** Dolphin Crash Dump Report ***************************

18:44:26, 11/10/2017: Dolphin7.exe caused an unhandled Win32 Exception 80000003
at 100063B9 in module 10000000 (C:\Users\scalvo\dev\Dolphin\DolphinVM7.DLL)

*----> Exception Parameters <----*
00000000

*----> CPU Context for thread 0x32dc <----*
EAX = 5E7F035D EBX = 0AB80210 ECX = 5E7F035C
ESI = 105ACE40 EDI = 001D40C0 EIP = 100063B9
ESP = 00C0FDB4 EBP = 00C0FDFC EFL = 00200202
CS = 0023 SS = 002B DS = 002B
ES = 002B FS = 0053 GS = 002B

*----> VM Context <----*
Process: {FF7F0210:size 398458964 words, suspended frame 5E7F0345, priority 3, callbacks 0
last failure 0:nil, FP control 80017, thread 00000000}
Active Method: ProcessorScheduler>>enableInterrupts
IP: 07274166 (6)
SP: 5E7F035C
BP: 5E7F0344 (398458942)
ActiveFrame: {5E7F0344: cf 5E7F0329, sp 5E7F035C, bp 5E7F0344, ip 7, ProcessorScheduler>>enableInterrupts}
	receiver: a ProcessorScheduler

New Method: ProcessorScheduler>>yield
Message Selector: yield

*----> Stack Back Trace <----*
{5E7F0344: cf 5E7F0329, sp 5E7F035C, bp 5E7F0344, ip 7, ProcessorScheduler>>enableInterrupts}
	receiver: a ProcessorScheduler

{5E7F0328: cf 5E7F0305, sp 5E7F033C, bp 5E7F0320, ip 2, ProcessorScheduler>>gpFault:}
	receiver: a ProcessorScheduler
	arg[0]: a ByteArray
	stack temp[0]: nil

{5E7F0304: cf 5E7F02D9, sp 5E7F0318, bp 5E7F02FC, ip 16, [] in ProcessorScheduler>>vmi:list:no:with:}
	receiver: a ProcessorScheduler

{5E7F02D8: cf 5E7F02B9, sp 5E7F02EC, bp 5E7F02D4, ip 18, BlockClosure>>ifCurtailed:}
	receiver: [] @ 0 in nil
	arg[0]: [] @ 24 in ProcessorScheduler>>vmi:list:no:with:

{5E7F02B8: cf 5E7F028D, sp 5E7F02CC, bp 5E7F02A8, ip 27, ProcessorScheduler>>vmi:list:no:with:}
	receiver: a ProcessorScheduler
	arg[0]: -553648004
	arg[1]: a LinkedList
	arg[2]: 5
	arg[3]: a ByteArray

{5E7F028C: cf 107F2570, sp 5E7F02A0, bp 5E7F028C, ip 0, Process>>primTerminate}
	receiver: a Process(HTTPConnection>>interact base 5E7F0000 in Process>>shutdown sp=5E7F0288 ip=15 list=10050010)

***Unable to generate complete exception report (c0000005)

***** End of crash report *****


Remove commented code in compute_data.py

For cleanliness and it wouldn't be uncommented by a user for a purpose.

[ ] Check comments in all the .py files


user_character.isMain does not have a default value

when trying to add a new api key to a new install, on the page where you enter it, it doesn't allow you to go to the character select page instead giving an error about how there's no default value for isMain.
confirmed the lack of default in MySQL:
mysql> show columns in user_character;
+-------------+------------+------+-----+-------------------+----------------+
| Field       | Type       | Null | Key | Default           | Extra          |
+-------------+------------+------+-----+-------------------+----------------+
| id          | int(11)    | NO   | PRI | NULL              | auto_increment |
| created     | timestamp  | YES  | MUL | CURRENT_TIMESTAMP |                |
| updated     | timestamp  | YES  | MUL | CURRENT_TIMESTAMP |                |
| active      | tinyint(1) | NO   | MUL | 1                 |                |
| userId      | int(11)    | YES  | MUL | NULL              |                |
| apiId       | int(11)    | YES  | MUL | NULL              |                |
| characterId | int(11)    | YES  | MUL | NULL              |                |
| isMain      | tinyint(1) | NO   | MUL | NULL              |                |
+-------------+------------+------+-----+-------------------+----------------+

after running alter table user_character modify column isMain tinyint(1) not null default 0; I was able to input an api key and select a main.
https://github.com/exodus4d/pathfinder/blob/master/app/main/model/usercharactermodel.php#L60

Oh thanks, i forgot the default value for that column.
Set nullable to false without providing any default value, does not make sense. Unfortunately I don´t see those errors on my installations.
To make it more consistent, i changed all default column values for bool* fields (which are actual converted into tinyint(1)`) from bool (true/false) into (1/0).

great, thanks! I've confirmed this is now resolved in the develop branch.
(btw, if you ever want a docker container to test things with... https://github.com/a-tal/pathfinder-docker)


Use vendored taglib for swig

This is working towards deterministic rake swig (see https://github.com/robinst/taglib-ruby/issues/136).

let rake swig use the taglib installed by rake vendor
use TagLib 1.11.1 in rake vendor (the checked-in wrapper files were already based on 1.11.1)
normalize @SWIG comments so they don't depend on where the developer installed swig

@robinst is there anything else we need to do in this PR before we merge?

No, you can go ahead and merge :)


ModbusRTU result: E4, Mem: XXXXX  RTU Timeout

Hi!
I'm testing this project as a link between https://sourceforge.net/projects/qmodmaster/ and https://pypi.org/project/pyModSlave/. As hardware, I'm using a NodeMCU 1.0 and a USB/TTL converter.
When I make any request from the TCP client, I receive an RTU Timeout.
I've already tried using hardwareSerial, baudrates of 9600 and 19200 and powering the board with a regulated supply @5 V-1 A.
Both programs work normally when I communicate them directly with the ESP, either reading from the RTU slave or receiving reads from the TCP client.
When I use ESP as a bridge, the RTU slave receives the request normally and responds correctly in < 5 ms.
Please, any hint on what to test or what else to investigate? Thanks.

my current code is
#include <ESP8266WiFi.h>
#include <ModbusTCP.h>
#include <ModbusRTU.h>

ModbusRTU rtu;
ModbusTCP tcp;

IPAddress srcIp;

uint16_t transRunning = 0;
uint8_t slaveRunning = 0;

uint16_t ModbusBaudRate = 19200;

const char* ssid     = "<SSID>";
const char* password = "<PASSWORD>";

Modbus::ResultCode cbTcpRaw(uint8_t* data, uint8_t len, void* custom) {
  auto src = (Modbus::frame_arg_t*) custom;

  srcIp = src->ipaddr;
  slaveRunning = src->slaveId;
  transRunning = src->transactionId;

  rtu.rawRequest(slaveRunning, data, len);

  return Modbus::EX_SUCCESS;
}

Modbus::ResultCode cbRtuRaw(uint8_t* data, uint8_t len, void* custom) {
  tcp.setTransactionId(transRunning);
  tcp.rawResponce(srcIp, data, len, slaveRunning);

  transRunning = 0;
  slaveRunning = 0;
  srcIp = (0,0,0,0);
  return Modbus::EX_PASSTHROUGH;
}

void setup() {
  WiFi.begin(ssid, password);
  while (WiFi.status() != WL_CONNECTED)
    delay(1000);

  tcp.server();
  tcp.onRaw(cbTcpRaw);

  Serial.begin(ModbusBaudRate, SERIAL_8E1);
  rtu.begin(&Serial);
  rtu.master();
  rtu.onRaw(cbRtuRaw);
}

void loop() {
  rtu.task();
  tcp.task();
  yield();
}

yet, no change.


Support for dry-validation v1.0?

Is there support planned for dry-validation v1.0? As of now it isn't compatible.

Hi! That's right. See https://github.com/bloom-solutions/dry-validation-matchers/issues/17


ExchangeOnlineManagement "Exception: Cannot access the variable '$HelpFileNames' because it is a private variable" when using a session configuration file

Using version 3.3.0 of the ExchangeOnlineManagement module and running pwsh with a configuration file I get the following error:
Exception: Cannot access the variable '$HelpFileNames' because it is a private variable.
ExchangeOnlineManagement version 3.2.0 has the same issue.
I've prepared a simple reproduction scenario which also works in Windows, in reality, we have a more complex configuration file and setup in Docker.
Using the attached files (exo.zip) it can be reproduced by running:
pwsh -ConfigurationFile configuration.pssc -Command ./exo.ps1
We've experimented with overriding the Export-ModuleMember cmdlet which seems to suppress this issue but in turn causes other problems. It may have something to do with how the module is loaded when using a session configuration file.

@nicot4e, you should open a support ticket about this. Report back here with any information that can help improve the documentation.


Feign Targeter is @Autowired, causing NPE because of lifecycle issues.

When i add dev-tools dependency FeignClientFactoryBean throws NullPointerException.
I don't know very well how dev-tools works but i think Spring doesn't see anymore @Autowired annotation for field targeter.
error.txt
In my gradle build there are:
    dependencyManagement {
        imports {
            mavenBom 'io.spring.platform:platform-bom:Athens-RELEASE'
            mavenBom 'org.springframework.cloud:spring-cloud-dependencies:Camden.RELEASE'
            mavenBom "org.springframework.boot:spring-boot-parent:1.4.1.RELEASE"

        }
    }

    dependencies {
        //Cloud
        compile 'org.springframework.cloud:spring-cloud-starter-config'
        compile 'org.springframework.cloud:spring-cloud-starter-eureka'
        compile 'org.springframework.cloud:spring-cloud-starter-ribbon'
        compile 'org.springframework.cloud:spring-cloud-starter-feign'
        compile 'org.springframework.cloud:spring-cloud-starter-hystrix'

        compile("org.springframework.boot:spring-boot-starter-security")

        compile group: 'org.springframework.boot', name: 'spring-boot-devtools'

        testCompile group: 'org.springframework.boot', name: 'spring-boot-starter-test'
    }

In the Brixton release the code was:
	private static final Targeter targeter;

	static {
		Targeter targeterToUse;
		if (ClassUtils.isPresent("feign.hystrix.HystrixFeign",
				FeignClientFactoryBean.class.getClassLoader())) {
			targeterToUse = new HystrixTargeter();
		}
		else {
			targeterToUse = new DefaultTargeter();
		}
		targeter = targeterToUse;
	}

in Camden there is @Autowired.
In either Brixton or Camden if you put in class FeignClientFactoryBean the following code:
    @Autowired
    private PackageDto packageDto;

where PackageDto isn't a bean the application doesn't throw any Exception. In this case without dev-tools spring throws org.springframework.beans.factory.UnsatisfiedDependencyException: Unsatisfied dependency expressed through field 'packageDto'.

3991c8c5f78b6c2d781c5c4b0e86c961378e6fc9 reintroduced @Autowired to FeignClientFactoryBean 😢 .

A quick workaround for this issue appears for be importing the FeignAutoConfiguration.class, which I assume configures it earlier:
@SpringBootApplication
@ImportAutoConfiguration(FeignAutoConfiguration.class)
@EnableFeignClients
public class Application {

This is a sample project recreating the issue: https://github.com/Turbots/spring-cloud-feign-autowiring-problem
It seems that the error occurs from the moment we add the spring-boot-devtools dependency, which changes the autoconfiguration order I guess


add pt1.6 cpu/gpu in CI

See https://github.com/open-mmlab/mmdetection/blob/master/.github/workflows/build.yml for reference


External JS and/or CSS is not combined/minified when minify set to auto mode

Very simple! In my deve environment I've a bunch of local (to my webserver/domain) JS and a couple of references to external resources (namely: facebook's sdk.js and google's analytics.js).
While the former are packed/combined/gziped neatly, the latter is totally ignored.
Th behavior is totally different if I chose "manual" minify mode. Everything is combined as expected.
The problem is that my set of javascript/js can vary greatly upon different conditions, so I cannot "dynamically" change the w3tc manual minify settings each time my set of JS changes.
Ideal behavior would be, as said, that the "auto" minify option would work as expected.
In the mean time, as a workaround, I'm forced to copy the external resources locally.
Thanks for any suggestion/help and, most of all, thanks for your job.

Webserver ngix (ver?)
PHP 7.1
MySQL 5.6
WordPress 7.1
W3 Total Cache plugin <IP_ADDRESS>.4

In the minify page setting there is the minify option Include external files/libraries

It works. Sorry! I didn't notice the option.
Thanks again.
I'm closing the issue


Proposed navigation

Add Leaderboard to main Nav when implemented.
Add to footer:

about us
faq
contact us
social icons
Partnerships and ambassadors

Confirmed, it's meanwhile there!


Introduce FieldVec and improve Polynomial and InvalidResidueError

This introduces a new internal type FieldVec which is basically a backing array for checksum residues with some fiddly logic to work in a noalloc context.
Unlike ArrayVec or other similar types, this one's semantics are "an unbounded vector if you have an allocator, a bounded vector otherwise". If you go outside of the bounds without an allocator the code will panic, but our use of it ensures that this is never exposed to a user.
It also assumes that it's holding a Default type which lets us avoid unsafety (and dismisses the potential performance cost because this array is never expected to have more than 10 or 20 elements, even when it is "unbounded").
Using this backing, it improves Polynomial and removes the alloc bound from it; using this, it puts Polynomial into a new InvalidResidueError type, which will allow users to extract the necessary information from a checksum error to run the error correction algorithm. (The resulting change to an error type makes this an API-breaking change, though I don't think that anything else here is API-breaking.)
The next PR will actually implement correction :).

Clicked "resolve" on the conversations we agreed were not actionable.
Force-pushed to address the other comments.

Second force-push was to fix a bug converting residues to polynomials which I noticed when working on the next PR.

Ok, I have working error correction code now based on this PR :). I will stop iterating on it until somebody reviews it.

Is a further non-math review useful by me, I can do it if so. I don't want to bother the PR with petty concerns if not.

I think it'd be valuable, yeah, if you're willing to take the time.

I'll make a coffee :)

Improved doccomments and added a bunch of unit tests. Also added missing Index<Range<usize>> impl on fieldvec.


Add a 'Copy To Clipboard' button in code examples

Add an icon on the top right corner so that when clicked it copy the code to your clipboard

Hey @ihsavru can I get assigned to this issue?


Unify #[pyfunction] and #[pyfn]

It seems confusing to me to have two different attributes, #[pyfunction] and #[pyfn], which both wrap Rust functions to create python functions. Pyo3 users need to know which applies in two different contexts.
I'd argue we should merge these. Then there's only one attribute which pyo3 users need to know about. It would also help us to simplify / refactor the macro code if we didn't need two different parsing systems.
I think this would need to go in the following steps:

Allow #[pyfunction] to be used in place of #[pyfn]
Change the docs to describe #[pyfunction] everywhere
Change #[pyfn] to expand to #[pyfunction]
(Later on) change #[pyfn] to emit an error, stating the #[pyfunction] invocation to use instead
(Much later on) remove #[pyfn]

I've already opened a PR #693 to achieve step 1.

But... are they so confusing?
For me, the different looks clear from the guide.
Or am I too accustomed?

The guide is clear enough on the differences. It's only a little confusing - but mostly I think for new users of pyo3.
Another downside to having two different attributes is that they might drift apart over time if we're not careful. For example, if #692 lands, then the way that each of these specify the python name is different:
#[pyfn(m, "foo")]
fn bar() { }

vs
#[pyfunction]
#[name = "foo"]
fn bar() { }

It seems like a good idea to me to simplify the codebase by unifying these attributes.

@kngwyu I would not say they are confusing if you read the guide, but I agree with @davidhewitt - these two procedural macros apparently do the same thing but in two different contexts.
Consider that even if the documentation clarifies this, it's definitely going to be hard to remember which one to use in which context unless you do it basically every day, so you'll have to look it up every time. Readers of the code who aren't necessarily reading the documentation will wonder why one or the other is used, etc.
If it's possible to unify them into a single macro, IMO, it should definitely be done.

I'd rather introduce
#[pymodule]
mod mymod {
    #[pyfunction]
    fn myfunction(py: Python, ...) {}
}

and make pyfn soft-duplicated.

I do like this new syntax for #[pymodule] very much, and would be happy to go down that road. How would you propose classes and attributes to be added to a #[pymodule] in this design? I'm thinking something like this:
#[pymodule]
mod mymod {
    #[pyfunction]
    fn myfunction(py: Python, ...) { ... }

    #[pyclass]
    struct MyClass { ... }

    #[pymethods]
    impl MyClass { ... }

    #[pyattribute]
    static myattr: i32 = 5;
}

I do like this new syntax for #[pymodule] very much, and would be happy to go down that road. How would you propose classes and attributes to be added to a #[pymodule] in this design? I'm thinking something like this:
<omitted>

I think attributes should be const rather than static since the value accessible from Rust is not the same variable as the value accessible from Python. If non-const attributes are needed, they can be added in the module #[init] function:
#[pymodule]
mod my_mod {
    #[pyclass]
    struct MyClass { ... }

    #[pyfunction]
    fn myfunction(py: Python, ...) { ... }

    #[pyattribute]
    // can override name on all module members using #[name]
    #[name = "MY_PYTHON_NAME"]
    pub const MY_CONST: &str = "My str";

    #[init]
    fn my_init(py: Python, module: &PyModule) {
        // custom init code; called after adding all other module members
        // can add non-const attributes here using module.add()
    }

    // add in external #[pyclass], #[pyfunction]
    #[pyclass]
    #[name = "NameStillWorksHere"]
    pub use path::to::external::MyClass;
    #[pyfunction]
    pub use path::to::external::my_function;
}

I think attributes should be const rather than static

I did consider const, but wondered if that might be misleading because nothing in Python is ever really const 😄 . I would be happy to go with it if that's the consensus.
I really like your idea of pub use to add extra items to the module.

I'm sorry I noticed #[proc_macro] mod m { ... }  is unstable today(https://github.com/rust-lang/rust/issues/54727).
@programmerjake

I experimentally implemented a similar syntax, but had some problems:

use function; raises not used warning
pub use function; requires function is pub

I'm sorry I noticed #[proc_macro] mod m { ... }  is unstable today(rust-lang/rust#54727).

It's stable (in the latest releases): rust-lang/rust#64273
What's unstable is non-inline mod:
#[my_proc_macro]
mod my_external_mod;

It's unstable because they haven't yet decided if the proc-macro will see mod my_external_mod; or mod my_external_mod { ... }.

@programmerjake

I experimentally implemented a similar syntax, but had some problems:

use function; raises not used warning

just use the name that the use statement introduces. alternatively decorate it with the appropriate #[allow(...)].

pub use function; requires function is pub

pub or pub(<path>) should be used only when the user specifies it, it should have no effect on the generated Python bindings, all it should do is make the Rust function visible to other Rust code. What makes the function visible to Python is the #[pyfunction] annotation, which should be independent of Rust visibility.

@programmerjake

It's stable (in the latest releases): rust-lang/rust#64273

Wow, thank you for letting know that :+1:

just use the name that the use statement introduces
How about removing the whole use statement?

just use the name that the use statement introduces
Another idea: How about removing the whole use statement?

I would leave the use statement in there since the imported function could be used from the user's Rust code:
#[pymodule]
mod my_mod {
    #[pyfunction]
    use my::other_fn;

    #[pyfunction]
    fn my_fn() -> i32 {
        other_fn(123)
    }
}

Proc-macros on mod are stable in 1.48.0, so we could make a pass at implementing a new pymodule now.

Indeed, I'm dangerously tempted to try to do this very soon! If anyone else wants to play with it, feel free to claim this and have a go and I'll be a happy reviewer / co-designer 😄


Manage multiple displays

Description of the new feature / enhancement
A tool to see which windows are open on which displays.
I have multiple monitors, which I do not always have switched on, or sometimes I am using one or two of my monitors on my laptop instead of my desktop. Windows sometimes opens windows on the monitors which are switched off or being used on another computer, this is very annoying. I would like a tool to see the open windows on my main monitor, and be able to move them between my monitors.
Scenario when this would be used?
Windows opens windows sometimes on the monitors which are switched off or not in use, this is very annoying as I cannot see or use the window.
Supporting information
No response

What about pressing Win+Tab, or the button in the Taskbar?