A11y_MySQLextensionforAzureDataStudio_New deployment - Azure Database for MySQL_Screen Reader:NVDA is not announcing  "accept terms of use" label in azure database for my sql dialog

GitHub Tags:
#A11yMAS; #A11yTCS; #A11ySev2; #DesktopApp; #GH-MySQLextensionforAzureDataStudio-Win32-Mar23; #MySQL extension for Azure Data Studio; #Win32; #WCAG1.3.1; #Info and Relationships; #NVDA;  #Screen Reader; #Benchmark; #JAWS; #Element:EditField;
"Please do not close this bug. This bug should only be closed by TCS, C+AI Accessibility or the Divisional Driver."
Environment and OS details:
Azure Data Studio Version: 1.42.0 (user setup)
Windows 11 Enterprise Version	22H2
Installed on	28-12-2022
OS build	22621.1413
Experience	Windows Feature Experience Pack 1000.22639.1000.0
NVDA Version 2023.1
JAWS version 2023
Pre-Requisite:

Install Azure data studio From here https://learn.microsoft.com/en-us/sql/azure-data-studio/download-azure-data-studio?view=sql-server-ver16&tabs=redhat-install%2Credhat-uninstall
Once installation done , Install MySQL Extension in Azure data studio
Tab Till  new connection and press enter
Enter a valid server name username and password and press connect

Repro Steps:

Tab till new deployment and press enter and Turn on nVDA
Tab till azure database for sql and press enter
Tab till accept terms of use
Verify Narrator/NVDA is announcing label or not

Actual Result:
NVDA is not announcing  "accept terms of use" label in azure database for my sql dialog
Issue repro with JAWS also
Expected Result:
NVDA should announce "accept terms of use" label in azure database for my sql dialog
Attachment:
Notes:
NVDA attachment:

This issue is external to MySQL extension as it is a common UI of Azure Data Studio.

As per comments mentioned https://github.com/microsoft/azuredatastudio/issues/18592 in this by design bug. Screen reader will read in Scan mode hence closing bug.


Local Preview Pulling from Live

Problem
We have multiple developers working on a shop.   Somehow when previewing the site using localhost:3000 we are getting code crossed.   The localhost preview is pulling files from the live theme.   One developer is seeing the others theme.liquid file on localhost.
Scope
Replication steps
Have two developers working on the same theme.
More Information

Are both developers' confg.yml files pointing at the same theme ID?  If so, then this is expected behaviour.  There's no local development in Slate, when you run slate watch it is always deploying to your site.  https://shopify.github.io/slate/commands/#watch

Sets up the watchers for all theme assets and deploys the compiled versions to your specified environment (default is development). Also opens a Browsersync window with live reloading after changing and saving a file.

The use of localhost:3000 is to facilitate Browsersync.  So if both developers are pushing to the same environment, they are going to see the same theme files when they run slate watch.

Closing for inactivity.

@NathanPJF I'd love to see this issue re-opened. Creating multiple themes for development isn't ideal. There has to be a better way to accomplish local development? Is there a fix for this in the works?

@jbsmith731 Using the config.yml file you can have multiple developers working on various branches or states of the same theme's code.  For example, your live, published theme could be the latest version of the production code, but you can duplicate the live theme to simultaneously working on various features.
live:
  password: 0000000aaaaaaa0000000aaaaaaa
  theme_id: "live"
  store: yourshop.myshopify.com
  ignore_files:
    - settings_data.json

staging:
  password: 0000000aaaaaaa0000000aaaaaaa
  theme_id: 12222222
  store: yourshop.myshopify.com
  ignore_files:
    - settings_data.json

new-feature-a:
  password: 0000000aaaaaaa0000000aaaaaaa
  theme_id: 13333333
  store: yourshop.myshopify.com
  ignore_files:
    - settings_data.json

new-feature-b:
  password: 0000000aaaaaaa0000000aaaaaaa
  theme_id: 14444444
  store: yourshop.myshopify.com
  ignore_files:
    - settings_data.json

Then if you're using git branches / pull requests each developer can merge their code/features into the staging or production ready code to be pushed to the correct published or unpublished theme.

Really appreciate you pointing this out. It does seem to be a bit of a workaround and not as intuitive. I assume it has to be done this way because shopify compiles the scss server side and there is no way around that currently?

@clarose098 It goes much deeper than that. 100% of the Shopify functionality runs directly on their servers, and it's not possible to run anything truly local.  Slate is a fantastic tool that allows you to sync and deploy code changes in real time to Shopify's servers and auto-update the browser using Browsersync (what you see with localhost).

What about tools like docker/vagrant? Could shopify release official VMs that replicate their setup?


Adding changelog and migrating doc to 4.1

Codecov Report

Merging #316 (eaac504) into master (de04aba) will decrease coverage by 1.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #316      +/-   ##
============================================
- Coverage     99.31%   98.29%   -1.03%
+ Complexity     1584     1418     -166
============================================
  Files           137      137
  Lines          3636     3628       -8
============================================
- Hits           3611     3566      -45
- Misses           25       62      +37

Impacted Files
Coverage Δ
Complexity Δ

src/AnnotationReader.php
83.53% <0.00%> (-16.47%)
65.00% <0.00%> (-7.00%)

src/Annotations/FailWith.php
87.50% <0.00%> (-12.50%)
5.00% <0.00%> (ø%)

src/Annotations/Right.php
88.88% <0.00%> (-11.12%)
5.00% <0.00%> (ø%)

src/Annotations/Decorate.php
88.88% <0.00%> (-11.12%)
5.00% <0.00%> (ø%)

src/Annotations/Security.php
89.28% <0.00%> (-10.72%)
16.00% <0.00%> (-3.00%)

src/Annotations/Autowire.php
92.30% <0.00%> (-7.70%)
6.00% <0.00%> (-1.00%)

src/Annotations/UseInputType.php
93.75% <0.00%> (-6.25%)
7.00% <0.00%> (ø%)

src/Mappers/RecursiveTypeMapper.php
93.84% <0.00%> (-1.03%)
76.00% <0.00%> (-6.00%)

src/InputTypeUtils.php
97.43% <0.00%> (-0.07%)
11.00% <0.00%> (ø%)

src/SchemaFactory.php
97.45% <0.00%> (-0.02%)
46.00% <0.00%> (-1.00%)

... and 5 more

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 de04aba...eaac504. Read the comment docs.


enc: add multi-reference-frame support.

Add multi-reference frames interface and test cases
Add multi-reference frames support for h264 and hevc.
Because of current reoder mechanism, multi-reference is only used
for forward reference. The max value of back referece frames is still
one.

Signed-off-by: Zhong Li<EMAIL_ADDRESS>

+1

merged as 72470d27ac93df1cf7df049bd34ee80450bc8ab4


Update Pages “aboutPage”

Automatically generated by Netlify CMS

Deploy preview for dds-mil ready!
Built with commit cdb7960dc4d73674128b306d181c5c6d10056f31
https://deploy-preview-124--dds-mil.netlify.app


Update README_CN.md

Fix Transaction param by README_CN.md

Codecov Report

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

@@          Coverage Diff           @@
##           master   #1179   +/-   ##
======================================
  Coverage    54.7%   54.7%
======================================
  Files          42      42
  Lines        7646    7646
======================================
  Hits         4183    4183
  Misses       2935    2935
  Partials      528     528

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 401f4ee...21c6606. Read the comment docs.

Codecov Report

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

@@          Coverage Diff           @@
##           master   #1179   +/-   ##
======================================
  Coverage    54.7%   54.7%
======================================
  Files          42      42
  Lines        7646    7646
======================================
  Hits         4183    4183
  Misses       2935    2935
  Partials      528     528

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 401f4ee...21c6606. Read the comment docs.


Misc cmake updates

force setting cached MLIR_DIR to honor command line argument
unset cached LLVM_DIR so it changes along with MLIR_DIR
surround third_party with set(CMAKE_MESSAGE_LOG_LEVEL NOTICE) and
set(CMAKE_MESSAGE_LOG_LEVEL STATUS) to mask out their cmake
output so we can see more clearly output by onnx-mlir only.
third_party cmake output can still be turned on by the --log-level
command line option
rename cmake option ACCELERATORS_TO_BUILD to ONNX_MLIR_ACCELERATORS
for consistency
allow accelerator lit tests even if accelerator code itself
cannot be built
fix bug in zdnn.cmake where ar was run even though libzdnn.so was
already up to date, causing concurrent builds to corrput libzdnn.a

Signed-off-by: Gong Su<EMAIL_ADDRESS>

Jenkins Linux amd64 Build #5399 [push] Misc cmake updates (#138... started at 10:07

Jenkins Linux ppc64le Build #4536 [push] Misc cmake updates (#138... started at 11:10

Jenkins Linux s390x Build #5415 [push] Misc cmake updates (#138... started at 11:07

Jenkins Linux s390x Build #5415 [push] Misc cmake updates (#138... passed after 1 hr 2 min

Jenkins Linux amd64 Build #5399 [push] Misc cmake updates (#138... passed after 1 hr 7 min

Jenkins Linux ppc64le Build #4536 [push] Misc cmake updates (#138... passed after 1 hr 9 min

@gongsu832
Question: is the MLIR_DIR removed? There are lots of references to it, for example in the docker files
ENV MLIR_DIR=/workdir/llvm-project/build/lib/cmake/mlir

or in our doc.
If we don't use the env var anymore, then all traces of it should be removed.


Error: value azurePushdownQuery is not a member of org.apache.spark.sql.SparkSession

I get this error when trying to run a custom query based on the example mentioned here:
https://github.com/Azure/azure-sqldb-spark#pushdown-query-to-azure-sql-database-or-sql-server
This is my code:
import com.microsoft.azure.sqldb.spark.config.Config
import com.microsoft.azure.sqldb.spark.query._

val createTableQueryString = s"""
  IF NOT EXISTS (SELECT * FROM sysobjects WHERE name = 'cohorts_analysis_manifest' and xtype ='U')
      USE testDb;
      CREATE TABLE testTbl
      (
          id bigint,
          name [varchar](255),
          ingestdate [varchar](10)
          CONSTRAINT [PK_testDb_testTbl] PRIMARY KEY CLUSTERED
          (
                    ingestdate DESC,
                    id ASC
          ) WITH (PAD_INDEX = OFF, STATISTICS_NORECOMPUTE = OFF, IGNORE_DUP_KEY = OFF, ALLOW_ROW_LOCKS = ON, ALLOW_PAGE_LOCKS = ON)
      )
"""
val createTableConfig = Config(Map(
  "url"            -> "myTestDbServer.database.windows.net",
  "databaseName"   -> "testDb",
  "user"           -> username,
  "password"       -> password,
  "queryCustom"    -> createTableQueryString
))

I faced the same issue. You need to import import com.microsoft.azure.sqldb.spark.query._ since the azurePushdownQuery is defined via implicit conversion.

I face the same issue even if I import "import com.microsoft.azure.sqldb.spark.query._"? I use Azure Databricks 5.4 with Apache Spark 2.4.3 and Scala 2.11 . All hints are welcome!

The readme is straight-up wrong, the command should be sqlDBQuery instead of push down:
https://github.com/MicrosoftDocs/azure-docs/issues/14640

@Axxeption I used sqlContext.sqlDBQuery(config) to issue a pushdown query instead

The option presented by @B4PJS is the correct way to address this. Closing the issue.


Compilation error with version 3.2.1

Hello,
I got a few compilation errors when trying to build with last version.
Capture of Xcode errors:

Hello @gsl-sebastienpecoul
Thank you for raising the issue, we will take a look and provide a fix.
In the mean time, we have remove the 3.2.1 tag, we will add it back when the issue is fixed.


[.github] Add CI support for fckmtp

Because why not ¯\_(ツ)_/¯

@TheMorc Ah sorry I just fixed it xD

Yeah, I just noticed it and closed the pull request


Remove version: gmbertani.Echoes version 0.50

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/echoes/files/Windows binaries/Install-echoes-0.50-Win64.exe
OK
(Automated message - build 923)

URL: https://sourceforge.net/projects/echoes/files/Windows binaries/Install-echoes-0.50-Win64.exe
OK
(Automated message - build 923)

Close with reason: Package still available;


Update CHANGELOG.md

What type of PR is this?

correction

What this PR does / why we need it:
Removing -wip

Another minor adjustment @bigludo7


CLIENTS-565

Ensure that conflict resolver passed into builder is used.
Fixes #21 (CLIENTS-565)

:+1:


Bash points on tribe and Player info

THe bash doesnt add up to whats in game it looks like it tracks from the start of data grabbing vs actual total in game

As we discussed on discord this is intended.
With the most recent changes, as the title on graphs says, we are presenting the difference in bash points the last 3 days.
The reason I change it is that the previous graphs was mostly showing flat lines and it was very difficult to distinguish any differences.
One solution that I had in mind was plotting the offsets just like right now, but reporting the actual values on the y-axis. Unfortunately this wouldn't work since every type of bash point starts from a different spot. So if we want to follow this solution we will have to split the graph into 3 different ones for each type of bash point (OBP, DBP, TBP).


Sync Wii Clock when connecting to MKWSP servers

To have more accurate ghost timestamps, to allow the creation of any other mkwsp feature that uses the clock, and doesn't secure networking need an accurate time on the device too or is that not how that works

I'd recommend using something like SNTP here; the NTP pool is more than adequate.


New example for an Orquesta workflow

This is an example for an Orquesta workflow using the core.ask pack to query multiple parameters. I will also create a PR in the st2docs repository to include this into the documentation.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Marcel Weinberg seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

As far as I can tell the failed Travis CI build is not caused by the new example and rather an issue with the infrastructure. Please let me know if I'm wrong and there is anything I can do to fix the issue.

@winem Thanks for the contribution! This looks like a useful example :+1:
@m4dcoder Can you please take a look if that workflow looks good to you or there are any recommendations?


input cannot read function keys on apple silicon mac

Software Versions

Python: 3.9
OS: Mac OS 12.0.1
Kivy: 2.1.0 dev0
Kivy installation method: pip install git+https://github.com/kivy/kivy.git

Describe the bug
the input object cannot read function keys (such as space and shift) and arrow keys. it read a-z keys no problem.
The issue tracker is a tool to address bugs only (search known bugs: https://git.io/vM1iE).
Please use the #support Discord channel at https://chat.kivy.org/ or Stack Overflow for
support questions, more information at https://git.io/vM1yQ.
Expected behavior
input.is_key_down('a')
input.is_key_down('left')
the first line works, second doesn't
To Reproduce
input.is_key_down('a')
input.is_key_down('left')
Code and Logs and screenshots
Additional context

Hi @JerryYang666 ,
Can you add some additional code and context?

In kivy/core/window/init.py Line 157
The method
def _on_window_key_down(self, instance, keycode, scancode, text,modifiers)
This method will be called when a key is pressed
In the method, the variable 'keycode' when I press a letter key, such as 's'
(115, 's')
which is normal.
However, if I press a function key, such as 'up', the variable 'keycode' becomes:
(1073741906, '')
which is not as expected.

Just tested, this problem is only replicable in an earlier version of 2.1.0.dev0. It's not replicable in any version after 2.1.0.dev0. I am marking this issue as resolved.


Talks doesn't support .background

Host any present slide with .background image.
It won't be shown for your presentation.
Other images seem to work fine.
Is this a bug or feature?

Is this an issue with the present command? Or just talks.godoc.org?

Just talks.godoc.org

Background works fine when present is launched locally.

I'm seeing this too. There seems to be some re-processing of presentations happening which is not re-mapping .background URLs to point at github instead. See
https://github.com/golang/gddo/blob/9d8ff1c67be5c2da6321c656913e1fe355991138/gosrc/present.go#L37-L39
Looks like this would not be too hard to fix.

See https://go-review.googlesource.com/c/gddo/+/133235

@nezorflame (or anyone else), do you know of a sample talk that has a slide with .background that can be used to reproduce the bug?

@dmitshur sure, take mine: https://talks.godoc.org/github.com/nezorflame/golang-talks/2018/05/execution-tracing-in-go1.11/main.slide#1
Source for it: https://github.com/nezorflame/golang-talks/tree/master/2018/05/execution-tracing-in-go1.11

Any thoughts on the CL https://go-review.googlesource.com/c/gddo/+/133235 ?

@mmcloughlin I read it and tested it locally, everything works fine on my workstation.

The new version is deployed now, so .background should be working. Thank you for the contribution!

Thank you @dmitshur . It now works fine on the public URL of my slideshow.

We can also test this with the sample presentation that was also included in the CL. This was intended to exercise all possible features of the present tool.
https://talks.godoc.org/github.com/golang/gddo/gosrc/testdata/sample.slide
We do indeed see the .background correctly, however it seems the the vector graphic is broken.

I'm getting a 403 from https://rawgit.com/golang/gddo/master/gosrc/testdata/gopher.svg
https://rawgit.com says "RawGit has reached the end of its useful life".


whats-is-the-difference-between-jpg-and-JPG ?

Short description of what this resolves:
https://stackoverflow.com/questions/52123415/whats-is-the-difference-between-jpg-and-jpg
test checks upper-case format images JPG
Changes proposed in this pull request:

Ionic Version: 1.x / 2.x / 3.x / 4.x
Fixes: #

Thank you for the PR! I am not going to merge this as I don't think it is something we need to test for. You should always write the path to an image using the capitalization that matches it exactly as it is saved. For example, if you have an image named RABBIT.jpg, you wouldn't link to it by writing:
"preview/assets/rabbit.jpg",

it would have to be:
"preview/assets/RABBIT.jpg",

🙂


Add a proper testsuite

The current code base only has the doctests from the standard library HashMap. We should add a proper testsuite.
Possible sources of inspiration:

https://github.com/blt/bughunt-rust
https://github.com/bluss/indexmap/blob/master/tests/quick.rs

I've adapted indexmap's tests/quick.rs for the standard HashMap interface over in https://github.com/jonhoo/griddle/blob/master/tests/quick.rs, which may also be helpful. My plan is to try and keep the two synced, while adding some additions that are specific to the standard HashMap.

https://github.com/jakubadamw/arbitrary-model-tests is an evolved version of https://github.com/blt/bughunt-rust - it autogenerates all the verification code in a proc macro, all you have to do is supply it with a reference implementation of the data structure to compare against. It already includes a HashMap example using std::HashMap.

Which is basically hashbrown :p Finding a way to integrate it here would be amazing though! And I'll see if I can't take a stab at getting it set up with griddle too.

@jonhoo, I'm the maintainer of that project (which now has a proper name: rutenspitz), so if I could be of any assistance, please do cc me. :)
In the past I have used it to fuzz hashbrown, which is when I found the issue #113. I was however deemed the expected behavior. 🙂 The fuzzer can nonetheless be used as part of a regression suite, perhaps.


Abillity to specify host in PfsClient()

Hi,
PfsClient() presumes you're only connecting to localhost.  I've added host="localhost" as a kwarg in PfsClient() so you can connect, e.g., from within the same k8s cluster that pachd is running in.
https://github.com/kalugny/pypachy/pull/3
Thanks!

Merged your pull request. Thanks!


[5.8] Validate container alias registration

The Container::getAlias() method currently throws LogicException. This exception is forwarded to many other container methods that call getAlias() and there are a lot of conflicts with the container interface.

\Illuminate\Container\Container::when() >>> breaks container interface.
\Illuminate\Container\Container::resolved() >>> breaks container interface.
\Illuminate\Container\Container::addContextualBinding() >>> breaks container interface
\Illuminate\Container\Container::extend() >>> breaks container interface
\Illuminate\Container\Container::rebinding()
\Illuminate\Container\Container::resolve()
\Illuminate\Container\Container::resolving() >>> breaks container interface
\Illuminate\Container\Container::afterResolving() >>> breaks container interface
\Illuminate\Container\Container::getExtenders()
\Illuminate\Container\Container::forgetExtenders()

Let's validate container alias registration to avoid these conflicts. And I think it's reasonable to add @throws \LogicException to \Illuminate\Contracts\Container\Container::alias(). Each class has to implement this logic.

Maybe the message at this phase should be changed into a more meaningful one.
like:
"[{$abstract}] cannot be aliased to itself.

I know. So, let him decide. Thanks!

I agree with this change but we should definitely document this in the upgrade guide.


class "Pdp\Rules" not found with laravel 8

Pdp version: 6.2.0
PHP version: 8.1.7
OS version: Ubuntu 20.04
Here is error log
[2023-01-11 10:24:07] local.ERROR: Class "Pdp\Rules" not found {"exception":"[object] (Error(code: 0): Class \"Pdp\\Rules\" not found
Here is my code
use Pdp\Rules;
use Pdp\Domain;
$publicSuffixList = Rules::fromPath('/home/joe/www/laravel_project/storage/public_suffix_list.dat');
$domain = Domain::fromIDNA2008($dn);
$result = $publicSuffixList->resolve($domain);
return $result->secondLevelDomain()->toString();

I have use composer require jeremykendall/php-domain-parser:^6.0
and in autoload_static.php it could see that 'Pdp\\Rules' => __DIR__ . '/..' . '/jeremykendall/php-domain-parser/src/Rules.php',
I have test in windows platform, and it could be used, but in ubuntu, it occurs this problem,
I makesure that did the following command
php artisan route:clear
php artisan route:cache
php artisan octane:reload

Do I miss any steps to use domain parser? Or something errors in it?

To prevent that forget enable intl ext, here if my loaded extension with get_loaded_extensions()
["Core","date","libxml","openssl","pcre","zlib","filter","hash","json","pcntl","Reflection","SPL","session","standard","sodium","swoole","mysqlnd","PDO","xml","bcmath","calendar","ctype","curl","dom","mbstring","FFI","fileinfo","ftp","gd","gettext","iconv","igbinary","imagick","imap","intl","exif","mysqli","pdo_mysql","Phar","posix","readline","redis","shmop","SimpleXML","soap","sockets","sysvmsg","sysvsem","sysvshm","tokenizer","xmlreader","xmlrpc","xmlwriter","xsl","zip","Zend OPcache"]

Finally I found the solution.
I stop the octane server, start again, and it worked.
I think the problem that reload the octane it wouldn't reflash the packages that I install?
Maybe but needs more confirm.


[NFC] specs: fix list formatting on SIGNATURE_SPEC

No functional changes.
Summary
Fixes the list formatting in the SIGNATURE_SPEC.md file. Previously, the list wasn't rendered correctly due to the lack of a line break and incorrect indentation.
Signed-off-by: William Woodruff<EMAIL_ADDRESS>

Codecov Report

Merging #2030 (04c191d) into main (6a298db) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2030   +/-   ##
=======================================
  Coverage   26.13%   26.13%
=======================================
  Files         127      127
  Lines        7432     7432
=======================================
  Hits         1942     1942
  Misses       5235     5235
  Partials      255      255

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 6a298db...04c191d. Read the comment docs.


Retire the old Makefile and make Makefile.port "Makefile"

Is there a reason why the original Makefile is being held on to? Maybe the Makefile.port should be Makefile and the old Makefile should be removed or renamed.

There's no particular reason. At first I was keeping it to see if I still could build the N64 ROM, but that's impossible by now. I'll probably remove it.


[Meta] PHP 7.1 features

PHP 7.1.0 Beta 1 is released with a feature stop so we can list the things we need to take care of or could support.

[ ] void return type RFC
[ ] Iterable pseudo-type accepting any array or object implementing Traversable RFC
[x] Support Class Constant Visibility RFC
[ ] Added a form of the list() construct where keys can be specified RFC
[ ] Added [] = as alternative construct to list() = RFC
[ ] Added nullable types. RFC
[ ] String offset access now supports negative references, which will be counted from the end of the string RFC
[ ] ASCII 0x7F delete control character is no longer permitted in unquoted identifiers in source code
[ ] multiple catch RFC
[x] rand() aliased to mt_rand() and srand() aliased to mt_srand()

Sources:
https://github.com/php/php-src/blob/php-7.1.0beta1/NEWS
https://github.com/php/php-src/blob/php-7.1.0beta1/UPGRADING
Thanks to all who worked on PHP7.1!

ping @dshafik it seems the support for class constant visibility is missing from;
https://github.com/php/php-src/blob/php-7.1.0beta1/UPGRADING
it is listed in https://github.com/php/php-src/blob/php-7.1.0beta1/NEWS#L373 (PHP 7.1.0alpha1).
Am I correct to assume it will still be part of 7.1?

I Added multiple-catch  to the list

perfect, filling in the blanks :)

Also, it seems that the fixer for spaces in ternary condition also adds a space in nullable type hints :(

Added nullable types. RFC

we didn't add support for them yet. feel free to contribute !

@keradus Yes I'm just saying that it needs to update a current fixer too :)

we do sth like https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/master/tests/Fixtures/Integration/misc/PHP7.test so we try to figure out which fixers need to be updated as well for new syntax.

Thanks for the heads-up ;)

I think that thanks to #2196 and #2205 the iterable support is there. Please verify @SpacePossum

I think we miss the logic to fix the casing of iterable in (return) type hinting? not sure though I haven't checked as I messed up my PHP7.1 VM ATM.

Do we have a fixer for that in first place? I don't think so

Don't think so either, having case-fixing-fixers for std-classes and interfaces would be nice though.
Anyway, with that in mind I leave it to others to make the call if we have support for iterable or not :)

How many of these fixers destroy the codebase with the new syntax? For now, could we skip bad cases until they're properly implemented?

go ahead and check it / skip it / fix it

What I was really asking is if anyone's aware of anything really broken. I guess I'll find out. :)

what has been reported was already fixed ;)

Optional parameters are still pretty much broken, is anyone working on that?
Also, even with -vvv it only shows the linting failed, not how and where. Got me confused at first.

Optional parameters are still pretty much broken, is anyone working on that?

not that I'm aware of, so please help :)

Also, even with -vvv it only shows the linting failed, not how and where. Got me confused at first.

Hits me as well once and awhile, would be great to improve the reporting. For linting we do rely on the runtime (i.e. PHP or HHVM), would be worth it to check out if we can retrieve more details from it when linting fails.

Hmm just started looking into it when I realized my test case is completely invalid - the linting was failing because my local Ubuntu is running PHP 7.0 but I'm running a 7.1 application inside Docker. Might have another look at this later on.
The hidden linter errors should be pretty easy to fix, they're just being eaten at Runner.php:144. The LintingException there contains the actual PHP parser error, in my case "PHP Parse error: syntax error, unexpected '?' on line 179.". That's pretty much all you need.

Strongly disagree. if they would not be handled there, they would break the flow and not allow to properly fix all other files. Solution would be to pass more information to ErrorManager

There's no disagreement, I didn't propose not catching the exception there, I said they were being eaten there. The details just need to be logged properly.

so, we do sth useful with it - catch it and handling it, only thing is that one would want to see more information about it.
anyway, nothing related to php 7.1

True that, moved to a separate issue.

I'm closing the meta ticket as the vast majority has been done; patching fixers to not break code, new rules, etc.
There might be more cases we could support better and for this I would like you all to open dedicated tickets.
Thanks!

This issue is linked as a blocker in #2512, but it seems done to me... The opening comments has void and Iterable as unchecked stuff, but I see:

#2740 done (but not merged?) for void
Iterable not done
Cna we have more details so we can help on these matters?

https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/2512 is about supporting PHP7.2 this issue is about 7.1.
We support PHP7.1 for a while (7.2 was dropped here https://github.com/FriendsOfPHP/PHP-CS-Fixer/commit/d6ec4dd3e8d5b79995b8b45fe904c58b9eb3fce9#diff-b5d0ee8c97c7abd7e3fa29b9a27d1780R17 ), which is why this issue is clossed.
Please discuss PHP 7.2 @ https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/2907


Increased build number because of PR

I believe you're supposed to increase the build number when there's a new commit on master, but not a new version.
PR merged without increasing version number:
https://github.com/agronholm/sphinx-autodoc-typehints/pull/55
Fixes a compatibility issue with newer sphinx.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

noarch packages can't have selectors. If the selectors are necessary, please remove noarch: python.

@TiemenSch PR agronholm/sphinx-autodoc-typehints#55 was upstream and the package built here with your bump won't be different. We need to wait for a new release.


type: "list" in prompt.js is not working...?

I want to select an item from list by prompt. But type: "list" looks not working... Please help me.
I made prompt.js like this.
module.exports = [
    {
        type: "list",
        choices: ["atoms", "molecules", "organisms", "templates", "pages"],
        name: "direcrory",
        message: `What's atomic directory?`,
    }
};

I execute to create new file. The prompt is below.
What's atomic directory?("atoms", "molecules", "organisms", "templates", "pages") ›

I expected like list.
What's atomic directory?("atoms", "molecules", "organisms", "templates", "pages")
❯◯ atoms
 ◉ molecules
 ◯ organisms
 ◯ templates
 ◯ pages

I have the same problem, did you solve it?

To solve your solution use the "Enquirer" module and destructure the "Select object from it. From there you can create a new list with choices. There is no type called list.
Code example provided below from (https://github.com/enquirer/enquirer#select-prompt).
Issue can probably be closed.
const { Select } = require('enquirer');

const prompt = new Select({
  name: 'color',
  message: 'Pick a flavor',
  choices: ['apple', 'grape', 'watermelon', 'cherry', 'orange']
});

prompt.run()
  .then(answer => console.log('Answer:', answer))
  .catch(console.error);

❤

To be specific for above example,
const { Select } = require('enquirer');

const selectPrompt = new Select({
  name: 'color',
  message: 'Pick a flavor',
  choices: ['apple', 'grape', 'watermelon', 'cherry', 'orange']
});
module.exports = {
  prompt: ({ prompter, args }) =>
  selectPrompt.run()
  .then(({ color }) =>
        prompter.prompt({
          type: 'input',
          name: 'folder',
          message: `Please type your color ${color} again:`
        })
      )

}


sddm: link whole sddm directory, not just themes

Motivation for this change
SDDM has a number of other important files in there, which won't be linked otherwise. Also, this way other packages can now contribute translations etc.
(This currently doesn't bite you if you use most DEs because of https://github.com/NixOS/nixpkgs/issues/47173)
I tested this by building a system without my DE and it now links the paths properly where it didn't before.
@jtojnar finally remembered to do this.
Things done

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

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

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

@GrahamcOfBorg test sddm


Feat: Create default fragments

What's the purpose of this pull request?

How it works?
This PR is part of the extension API initiative, focused on creating default GraphQL fragments for customization in the stores. Two default fragments, 'plp' and 'pdp', were added and called in ServerCollectionPageQuery and ServerProductDetailPageQuery, respectively.
After adding the fragments, the "yarn generate" command was used to create the types.
How to test it?
To test this feature, you can navigate to the 'package/core' directory and execute the command 'yarn dev'. Once done, you can utilize your preferred client to run the following CURL command:
curl --request POST \
  --url http://localhost:3000/api/graphql \
  --header 'Content-Type: application/json' \
  --data '{"query":"fragment plp on Query {\n\tcollection(slug: $slug) {\n\t\tid\n\t}\n}\n\nquery ServerCollectionPageQuery($slug: String!) {\n\t...plp\n\tcollection(slug: $slug) {\n\t\tseo {\n\t\t\ttitle\n\t\t\tdescription\n\t\t}\n\t\tbreadcrumbList {\n\t\t\titemListElement {\n\t\t\t\titem\n\t\t\t\tname\n\t\t\t\tposition\n\t\t\t}\n\t\t}\n\t\tmeta {\n\t\t\tselectedFacets {\n\t\t\t\tkey\n\t\t\t\tvalue\n\t\t\t}\n\t\t}\n\t}\n}\n","operationName":"ServerCollectionPageQuery","variables":{"slug":"brand"}}'

Everything should work normally in the store navigation.
References
Task

Isn't search a PLP?

The search is a client side query inside a PLP component. That's one of the reason we shouldn't use just 1 fragment per PLP/PDP. We need to create a multiple fragment/query per page approach.


add support for Laravel 8

This PR adds Laravel 8 compatibility without removing Laravel 7 compatibility. So it's an alternative to https://github.com/cviebrock/eloquent-sluggable/pull/524
Tests seem to be running fine:

Browser tests too:

Don't know which way you'd rather go @cviebrock - let me know.
Cheers!

I went the other way, but thank you!


the formula is messed up after selecting with the mouse

Selecting the formula with mouse, from right to left, causes the formula to become completely disordered
Example: this formula in demo page, basic mode, 1+2+3*(4+5), is converted in 1+352+*+)(4

duplicate #26


How to select all the available columns in the KQL

The documentation I am referencing: Get started with Azure Monitor Log Analytics#filter-the-results and Get started with Azure Monitor Log Analytics#select-columns-to-display
I am query the log of api management in Azure portal, however, I found the default columns (top-level-diagnostic-logs-schema) are not that much useful.

TenantId
SourceSystem
MG
ManagementGroupName
TimeGenerated
Computer
RawData
identity_claim_http_schemas_xmlsoap_org_ws_2005_05_identity_claims_upn_s
Category
OperationName
ResultType
CallerIPAddress
identity_claim_http_schemas_microsoft_com_identity_claims_objectidentifier_g
identity_claim_appid_g
identity_claim_xms_mirid_s
id_s
clientInfo_s
subnetId_s
requestUri_s
ResourceId

What should I do to select all the available columns(could be found under the arrow to the left of each record) that are used for API Management Diagnostic Logs without typing one by one?
{
    "isRequestSuccess": "",
    "time": "",
    "operationName": "",
    "category": "",
    "durationMs": ,
    "callerIpAddress": "",
    "correlationId": "",
    "location": "",
    "httpStatusCodeCategory": "",
    "resourceId": "",
    "properties": {
        "method": "",
        "url": "",
        "clientProtocol": "",
        "responseCode": ,
        "backendMethod": "",
        "backendUrl": "",
        "backendResponseCode": ,
        "backendProtocol": "",
        "requestSize": ,
        "responseSize": ,
        "cache": "",
        "cacheTime": "",
        "backendTime": ,
        "clientTime": ,
        "apiId": "",
        "operationId": "",
        "productId": "",
        "userId": "",
        "apimSubscriptionId": "",
        "backendId": "",
        "lastError": {
            "elapsed": "",
            "source": "",
            "scope": "",
            "section": "",
            "reason": "",
            "message": ""
        }
    }
}

I also find SELECT ALL is disabled without project operator in the query, it's inconvenient

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

ID: 773e07ff-6bd8-47b2-5b83-75c7246693e6
Version Independent ID: 8723d5a6-ac93-0dba-b2b8-83905d807d67
Content: Get started with Azure Monitor Log Analytics
Content Source: articles/azure-monitor/log-query/get-started-portal.md
Service: log-analytics
GitHub Login: @bwren
Microsoft Alias: bwren

@ToMakeSense Thanks for the Comment. We are actively investigating and will get back to you soon.

@ToMakeSense To get all available columns, you an use search * as detailed here.

We will now close this issue. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

@femsulu let's make it clear, I just want to find a way to use the Service-specific schemas template(in the case, it's apim), not use the Top-level diagnostic logs schema. I don't want to type each column property with project operator all the time.


Ss437 438 439 labwhere enhancement

Included in: https://github.com/sanger/sequencescape/pull/1333 which also contains the fixes for my javascript.


Non-admin users can't determine what failed jobs are.

When logged into Huginn on a fairly recent checkout (5101cd5a8c6dced63742b080fdeb4b7d59a80d56) from time to time the "X job(s) recent failures) icon will pop up in the menu bar at the top next to the Account drop-down.  In theory, it can be clicked on, but in practice clicking on it as a non-admin user does nothing.  It would be very handy if at least one of the following was possible:

Due to the fact that all users on a Huginn instance see this warning, it would be useful for a user to click on it and see whose agent it was that failed.
Only the user whose agent failed would see that notification, and know that something was wrong.  Clicking on the warning would display the "Background Jobs" page, but  that page would say which agent(s) failed and provide a way to jump directly to them with a click for troubleshooting.
Clicking on the warning would display the agent ID that was experiencing problems.
It might also be useful for agents to emit special "Something went wrong!" events that could then be caught by another agent and processed (I'd like to be able to text myself if something goes sideways, for example).

SlackAgent seems to be working more often than not these days.  I haven't tried the Dropbox agents yet because I'm still monitoring it through RSS.

Should I close this?

I'd rather you not, because I still can't get any information about failed jobs (I'm currently at 50 failed recent jobs behind the  button) unless I log out and then log back in as the admin user.  I'm dealing with it at the moment but I can't be the only person who's running into this UX problem.

Worked around this by making my personal account an admin.  It's bad practice to run with elevated privileges all the time but I'm the only person making use of this particular instance.


Know when NSNumber is nil or zero.

Example
We have two cases:

There are no setting in Firebase Config for key.
There is "0" in Firebase Config for key.

Problem:
With last change, that it's always NSNumber exists can't figure out is setting exist or just setted to "0".
Want:
Return optional value.

Discussed with the team, the recommendation for now is to check FIRRemoteConfigSource which will indicate whether the value is user-defined or not (FIRRemoteConfigSourceStatic indicates no value exists)
Hope that helps!


External fairseq model loading issue

Hi @ghchen18 ,
Thanks for the code. Your work is very interesting.
I am trying to load a translation model trained on fairseq v0.10.0 which, as expected, is giving errors since your paper is trained on fairseq v0.9.
Is there any way to load the v0.10 model (since this version is in existence since Nov 2020) ?
Thanks,

Hi, Ayushbits,
what are the giving errors? Are they caused by the mismatched names? If debug step by step, the model may be loaded.

Hi, thank you for your reply @ghchen18 . I'm also working with @ayushbits on same thread.
This is the error I got:
RuntimeError: Error(s) in loading state_dict for TransformerModel:
Missing key(s) in state_dict: "decoder.embed_out".
Unexpected key(s) in state_dict: "decoder.output_projection.weight".
size mismatch for encoder.embed_tokens.weight: copying a param with shape torch.Size([32104, 1536]) from checkpoint, the shape in current model is torch.Size([32105, 512]).
size mismatch for encoder.layers.0.self_attn.k_proj.weight: copying a param with shape torch.Size([1536, 1536]) from checkpoint, the shape in current model is torch.Size([512, 512]).
size mismatch for encoder.layers.0.self_attn.k_proj.bias: copying a param with shape torch.Size([1536]) from checkpoint, the shape in current model is torch.Size([512]).
size mismatch for encoder.layers.0.self_attn.v_proj.weight: copying a param with shape torch.Size([1536, 1536]) from checkpoint, the shape in current model is torch.Size([512, 512]).
size mismatch for encoder.layers.0.self_attn.v_proj.bias: copying a param with shape torch.Size([1536]) from checkpoint, the shape in current model is torch.Size([512]).
size mismatch for encoder.layers.0.self_attn.q_proj.weight: copying a param with shape torch.Size([1536, 1536]) from checkpoint, the shape in current model is torch.Size([512, 512]).
size mismatch for encoder.layers.0.self_attn.q_proj.bias: copying a param with shape torch.Size([1536]) from checkpoint, the shape in current model is torch.Size([512]).
size mismatch for encoder.layers.0.self_attn.out_proj.weight: copying a param with shape torch.Size([1536, 1536]) from checkpoint, the shape in current model is torch.Size([512, 512]).
size mismatch for encoder.layers.0.self_attn.out_proj.bias: copying a param with shape torch.Size([1536]) from checkpoint, the shape in current model is torch.Size([512]).
size mismatch for encoder.layers.0.self_attn_layer_norm.weight: copying a param with shape torch.Size([1536]) from checkpoint, the shape in current model is torch.Size([512]).
size mismatch for encoder.layers.0.self_attn_layer_norm.bias: copying a param with shape torch.Size([1536]) from checkpoint, the shape in current model is torch.Size([512]).
size mismatch for encoder.layers.0.fc1.weight: copying a param with shape torch.Size([4096, 1536]) from checkpoint, the shape in current model is torch.Size([2048, 512]).
...
...
I have also trained a model from scratch using your code and compared the state['model'] keys with state['model'] keys of fairseq v0.10.0 model ( the translation model which I'm trying to load), Although decoder.output_projection.weight is present in v0.10.0 model which i can get rid of, I couldn't find the key "decoder.embed_out" in both of them.
I have used the custom transformer architecture for v0.10.0 model using --user-dir tag.
Can you help me with the possible reason for the same and the corresponding solution?
Thanks.

Hi, Piyush,
I think the error is not caused by the fairseq version difference.  It is caused by the difference of model structure and dictionary  between pretrained model and the v0.9.0 code.
You can check the difference of the task dictionary (for example the first and last several tokens).  and set the values of args encoder_ffn_embed_dim,  encoder_embed_dim, decoder_ffn_embed_dim,  decoder_embed_dim, to make them consistant between your pretrained model and the v0.9.0 code.  you may also need to set share_all_embeddings as True.
Hope these helps,
Best,

Hi @ghchen18,
I tried loading another model (itransformer.wmt18.en-de) which has the same encoder_ffn_embed_dim , encoder_embed_dim etc. still I'm getting same error as above i.e. Missing key(s) in state_dict: "decoder.embed_out".  I could not find this key in the model trained using your code.
Errors
RuntimeError: Error(s) in loading state_dict for TransformerModel:
Missing key(s) in state_dict: "decoder.embed_out".
size mismatch for encoder.embed_tokens.weight: copying a param with shape torch.Size([32070, 512]) from checkpoint, the shape in current model is torch.Size([32071, 512]).
size mismatch for decoder.embed_tokens.weight: copying a param with shape torch.Size([32070, 512]) from checkpoint, the shape in current model is torch.Size([32071, 512]).
I'm not sure why I'm getting this one extra row (32071 --> 32071), maybe because of this missing key. Can you look into this?
Thanks

Hi,
The embed_out is used in fairseq/models/transformer.py#L559, you can check the config of  share_input_output_embed (expected to be true).
You can print the first/last several tokens in your dictionary to check the extra dictionary token (e.g. dictionary.symbols[:20]).
Hope these help.
Best

Hi,
I am able to get rid of the error Missing key(s) in state_dict: "decoder.embed_out" by setting the arg share_all_embedding=True and share_input_output_embed=True.
The only error that remains is:
size mismatch for encoder.embed_tokens.weight: copying a param with shape torch.Size([32104, 1536]) from checkpoint, the shape in current model is torch.Size([32104, 512]).
I'm passing args --use-dir with model architecture and dimensions (the model architecture has embed_dim as 1536).  As per my knowledge, the model should see dimension from dir provided in --user-dir and accordingly set the tensor, but somehow it is not happening. If these dimensions are needed to be changed in the code, can you please point me to that file?
Thank you.

Hi,
you can try to set the dimension in the scritps, by --encoder-embed-dim 1536  --decoder-embed-dim 1536 .  If not work, you might have to debug step by step.
Best,

That's good.


Wasm: CalliIntrinsics.Call is generating invalid calli

https://github.com/dotnet/corert/blob/42f1bc077327d6e3647861bec00cfe0458187071/src/System.Private.CoreLib/src/System/Runtime/InteropServices/Marshal.CoreRT.cs#L200-L203
This is currently a problem for the wasm backend.  With the input
    // something to exercise CalliIntrinsics.Call<T>
    struct NonBlittableToMarshal
    {
        internal int a;
        internal object b;
    }

    static void TestMarshalStructToPtr()
    {
        StartTest("Test marshal struct to ptr");
        NonBlittableToMarshal s = new NonBlittableToMarshal();
        s.a = 1;
        s.b = null;
        var pParms = Marshal.AllocHGlobal(Marshal.SizeOf(typeof(NonBlittableToMarshal)));
        Marshal.StructureToPtr(s, pParms, false);
        EndTest(true);
    }

This fails at runtime with an invalid signature message, with ASSERTIONS=2, it produces:
Invalid function pointer 4692 called with signature 'iiii'. Perhaps this is an invalid value (e.g. caused by calling a virtual method on a NULL pointer)? Or calling a function with an incorrect type, which will fail? (it is worth building your source files with -Werror (warnings are errors), as warnings can indicate undefined behavior which can cause this). This pointer might make sense in another type signature: as sig "vi" pointing to function _Internal_CompilerGenerated__Module____ManagedToNative_HelloWasm_Program_NonBlittableToMarshal,

This is saying that there is a method with a void (int) signature at the right function index, but it does not match the called convention which is int (int,int,int).  There is a method produced with a vi signature, although it looks incomplete and throws an exception
define void @"Internal_CompilerGenerated__Module___<ManagedToNative>HelloWasm_Program_NonBlittableToMarshal"(i8*) {
Prolog:
  br label %Block0

Block0:                                           ; preds = %Prolog
  %LoadAddressOfSymbolNode = load i32*, i32** @__Str_Struct__NonBlittableToMarshal__6058104189200C7DE718BE6B8F3608243B48FAFC59CD4893FB0EFFADF26C30EA___SYMBOL
  %1 = getelementptr i8, i8* %0, i32 8
  %Temp0_ = getelementptr i8, i8* %0, i32 8
  %LoadeeType = load i32, i32* bitcast (i32** @__EEType_S_P_CoreLib_System_Exception___SYMBOL to i32*)
  call void @S_P_CoreLib_System_Runtime_RuntimeExports__RhNewObject(i8* %1, i8* %Temp0_, i32 %LoadeeType)
  %2 = getelementptr i8, i8* %0, i32 12
  %Temp0_1 = getelementptr i8, i8* %0, i32 8
  %CastPtrTemp0_ = bitcast i8* %Temp0_1 to i8**
  %LdTemp0_ = load i8*, i8** %CastPtrTemp0_
  %3 = getelementptr i8, i8* %2, i32 0
  %CastPtrTypedStore = bitcast i8* %3 to i8**
  store i8* %LdTemp0_, i8** %CastPtrTypedStore
  %CastPtr = bitcast i32* %LoadAddressOfSymbolNode to i8*
  %4 = getelementptr i8, i8* %2, i32 4
  %CastPtrTypedStore2 = bitcast i8* %4 to i8**
  store i8* %CastPtr, i8** %CastPtrTypedStore2
  call void @S_P_CoreLib_System_Exception___ctor_0(i8* %2)
  call void @llvm.trap()
  unreachable
}

And the iiii signature method would match the calli inside
                        CalliIntrinsics.Call<int>(
                            marshalStub,
                            ((void*)((IntPtr*)unboxedStructPtr + offset)),  // safe (need to adjust offset as it could be class)
                            (void*)ptr                                      // unsafe (no need to adjust as it is always struct)
                        );

With an the extra i for the shadow stack.  I guess the question starts with how does the marshal code get generated because it shouldn't terminate in a throw I imagine?

The internal object b; field takes us down the COM marshalling path. I think this is just the compiler going down the path where it failed in the middle of generating marshalling code.
CoreRT compiler can't do COM. If you just need something that is not blittable, use e.g. a string.

Thanks, moving on Wasm is confused by the different arguments to the generated method for the managed to native conversion, in this case:
.method void ManagedToNative__NonBlittableToMarshal(valuetype [HelloWasm]Program/NonBlittableToMarshal&, valuetype Internal.CompilerGenerated.__NativeType__NonBlittableToMarshal&) cil managed
{

And the call from the CalliIntrinsics:
  IL_0003:  calli       int32(void*, void*)

The first are ByRefType and the second PointerType, which normally would be no issue, both pointers.  But in Wasm it makes a decision about whether a variable needs to placed on the shadow stack for conservative GC purposes using:
        private static bool CanStoreTypeOnStack(TypeDesc type)
        {
            if (type is DefType defType)
            {
                if (!defType.IsGCPointer && !defType.ContainsGCPointers)
                {
                    return true;
                }
            }
            else if (type is PointerType)
            {
                return true;
            }

            return false;
        }

And each case returns a different value, ByRefType are on the shadow stack, and PointerType are passed normally.  I assume that this logic is ok, and ByRefTypes do indeed need to be scanned when looking for roots so maybe I need so special case by identifying somehow the generated method (ManagedToNative__NonBlittableToMarshal) and using some logic, like, "I know this is only going to be called from CalliIntrinsics and put its arguments on the llvm stack".  This would of course mean that the parameters are not scanned for GC reporting, which might be ok for this scenario - if there's nothing else referencing the managed struct then its ok to collect.
This accounts for 2 of the "missing" parameters on the call, the other is the return type, the generated method is void but CalliIntrinsics.Call<int> is int32.  Is it the case that some generated methods do return something and in cases like this one the return value is never used so its not an issue?

Yeah, the calli is illegal. Per the ECMA-335 spec, the signature needs to match the destination. I've submitted #7954 to hopefully unblock you. The sig doesn't match exactly either (callsite is ref byte, but the actual method is ref X where X is the struct type), but it should be good enough for the WASM backend.
I think the existing shape was just legacy wart (Project N at some point only supported calli intrinsic that return a value, and doing pointer math with refs used to be impossible with C#).

Thanks a lot.


Feature/issue 994

Sample for ga4gh/dockstore#994  .  Should further update everything at some later date.  The code in the .ts file is mostly the same as markdown except all ` must be escaped

Codecov Report

Merging #129 into develop will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop    #129      +/-   ##
==========================================
+ Coverage    64.87%   64.9%   +0.02%
==========================================
  Files           95      95
  Lines         2551    2553       +2
  Branches       322     322
==========================================
+ Hits          1655    1657       +2
  Misses         723     723
  Partials       173     173

Impacted Files
Coverage Δ

...g/downloadcliclient/downloadcliclient.component.ts
91.66% <100%> (+0.75%)
:arrow_up:

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 f046bc2...52ac137. Read the comment docs.

Moved everything except the textarea and the button to the .ts file as markdown.  Lower the line count, same file as all the variables (cwltool version, etc).  More importantly, it removes all the html tags for readability.


Access violation error with 1.0.12

With reference to this project :
github.com/philstopford/Eto.Veldrid
updating the spirv from 1.0.9 to 1.0.12 yields an access violation :
for Direct3D11 or OpenGL backends.
Build/run the WPF project and things should explode with 1.0.12 (1.0.9 is configured in the project and should work fine). The shaders are in the project.
{{{
System.OverflowException
HResult=0x80131516
Message=Array dimensions exceeded supported range.
Source=Veldrid.SPIRV
StackTrace:
at Veldrid.SPIRV.SpirvCompilation.CompileVertexFragment(Byte[] vsBytes, Byte[] fsBytes, CrossCompileTarget target, CrossCompileOptions options)
at Veldrid.SPIRV.ResourceFactoryExtensions.CreateFromSpirv(ResourceFactory factory, ShaderDescription vertexShaderDescription, ShaderDescription fragmentShaderDescription, CrossCompileOptions options)
at VeldridEto.VeldridDriver.CreateResources() in D:\development\git\Eto.Veldrid\src\Eto.VeldridSurface\VeldridDriver.cs:line 1117
at VeldridEto.VeldridDriver.SetUpVeldrid() in D:\development\git\Eto.Veldrid\src\Eto.VeldridSurface\VeldridDriver.cs:line 1026
at VeldridEto.MainForm.SetUpVeldrid() in D:\development\git\Eto.Veldrid\src\Eto.VeldridSurface\MainForm.cs:line 170
at VeldridEto.MainForm.set_FormReady(Boolean value) in D:\development\git\Eto.Veldrid\src\Eto.VeldridSurface\MainForm.cs:line 37
at VeldridEto.MainForm.<.ctor>b__12_0(Object sender, EventArgs e) in D:\development\git\Eto.Veldrid\src\Eto.VeldridSurface\MainForm.cs:line 48
at Eto.PropertyStore.TriggerEvent[T](Object key, Object sender, T args)
at Eto.Forms.Control.OnShown(EventArgs e)
at Eto.Forms.Control.Callback.OnShown(Control widget, EventArgs e)
at Eto.Wpf.Forms.WpfWindow`3.b__29_7()
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
at System.Windows.Threading.DispatcherOperation.InvokeImpl()
at MS.Internal.CulturePreservingExecutionContext.CallbackWrapper(Object obj)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at MS.Internal.CulturePreservingExecutionContext.Run(CulturePreservingExecutionContext executionContext, ContextCallback callback, Object state)
at System.Windows.Threading.DispatcherOperation.Invoke()
at System.Windows.Threading.Dispatcher.ProcessQueue()
at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
at System.Windows.Threading.Dispatcher.LegacyInvokeImpl(DispatcherPriority priority, TimeSpan timeout, Delegate method, Object args, Int32 numArgs)
at MS.Win32.HwndSubclass.SubclassWndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam)
at MS.Win32.UnsafeNativeMethods.DispatchMessage(MSG& msg)
at System.Windows.Threading.Dispatcher.PushFrameImpl(DispatcherFrame frame)
at System.Windows.Application.RunDispatcher(Object ignore)
at System.Windows.Application.RunInternal(Window window)
at VeldridEto.MainClass.Main(String[] args) in D:\development\git\Eto.Veldrid\src\gui\Eto.Veldrid.Wpf\Program.cs:line 124
}}}

Hey @philstopford , did you resolve this on your end, or should I still investigate?

Combination of user error in build config (special sauce) and a change in 1.0.12 that some clarification would help with  :
Rob (the upstream developer of the Eto Veldrid project) had issues with OpenGL rendering blank until he changed references to "ViewMatrix" to "vdspv_0_0", and then changed "ModelMatrix" to ""vdspv_1_0"
Is that expected/required?

Yes, that is an accidental but expected breaking change in the 1.0.12 version right now. I am hoping to fix that soon so that the new behavior is opt-in instead of automatic. Getting an access violation before compilation even finishes sounded very different. Until the fix is in, updating the references in code will work.

The access violation was due to some variable usage to define the spirv version, and it seemed to be falling back to 1.0.0 because I missed the special sauce needed. That resulted in the access violation. This coincided with the 1.0.12 breaking change, I think, so two causes for one common point of failure.


Update the what-is-awkward to align with (and include) the video.

Only the doctest needs to pass here (and I'm not sure it's testing the JupyterBook part). The real test will be to see if Netlify can run it.


set unique useragent for setup-chainctl

@imjasonh can you rebase and fix the conflict and then we merge this :) thanks!


Add AgeSource example

BigtableSource is closer to the example shc repo provides, but it's a bit abstract. I put together this sample for the demo, and figured it probably wouldn't hurt to add it here.

LGTM

@dpebot merge on green


Adding Utf8String and ConverAssignRowNullable

Adding changes needed for handling sql UTF8 marshaling/unmarshaling and allowing reading values from SQL if they are nullable. This can happen sometimes if the tables get new columns and some of the items are left nullable. Default values are expected in this case, which is what's expected.

The isUtf8 field in String is actually a misnomer.  If you look at inside BuildValue, pretty much all string is treated as "isUtf8".


Adaptative classification trainer

The classification trainer only works for classifications with 5 or more classes. The purpose of this PR is to make it adaptive

Ok I understand your point I will remove the case of binary classification. Yes sorry the test is comming :)

Sorry to bother, I still have a few suggestions 😅

I like suggestions, it makes me learn :) I made the changes, tell me if it's good for you ?


Failed to open /dev/mem: Permission denied

Hi all,
I am using hyperion to drive my WS2812b which was working well until suddenly the lights keep staying off.
hyperiond /etc/hyperion.config.json is showing me the following:
Application build time: Mar  6 2015 20:57:44
QCoreApplication initialised
Selected configuration file: /etc/hyperion.config.json
ColorTransform 'default' => [0; 133]
Device configuration:
{
"colorOrder" : "rgb",
"name" : "MyPi",
"type" : "ws2812b"
}
Failed to open /dev/mem: Permission denied
Segmentation fault
I guess the issue is not with hyperion but with some broken permissions on my PI. Unfortunately I'm quite new to Linux so I'd appreciate if someone has a hint on how to solve this.
Many thanks in advance!

you could try in SSH
mount -o remount,rw /dev

Hi,
unfortunately that also doesn't work. Quite weird, espacially since the problem started from one moment to the other... I think I'm just gonna rewrite the image to the SD. That should do the job.
Thanks for trying to help!


Parametrise script folder

It would be nice to give users possibility to select where they would like scripts to be installed.
Could you, please, parametrise script folder like ${include_dir} for user parameters?
Thank you.

Hi Grundic,
Thanks you for making time to create this issue.
What do you mean with parametrise?
You can use with the current module:
class { 'zabbix::agent':
  server      => <IP_ADDRESS>,
  include_dir => '/usr/local/scripts',
}

Kind regards,
Werner

Hello, Werner!
Sorry for unclear issue report.
What I meant is configuration of LLD script:
zabbix::userparameters { 'lld_snort.sh':
  script => 'puppet:///modules/zabbix/lld_snort.sh',
}

Correct me if I'm wrong, but according userparameters.pp, lld_snort.sh script will always be placed to /usr/bin. And for example, I would like it to be placed under /usr/local/bin.
Thank you.

Hi,
I've pushed an fix for this almost an week ago to github.
You can use it like this now:
zabbix::userparameters { 'lld_snort.sh':
  script     => 'puppet:///modules/zabbix/lld_snort.sh',
  script_dir => '/usr/local/bin'
}

Default, '/usr/bin' is used.
Will close this ticket to.
Kind regards,
Werner

Thank you very much, will test it soon.


Implement Autocomplete spectre components

Hello,
I have implemented the Autocomplete components. Following the philosophy of the Spectre library the components don't include any javascript and it's up to the user to  add it. On the other hand, I also included an example component with JS which uses the library components in the assets folder of the autocomplete package. That component is not expose in the package, it's just an example for people to use.
Regards,
Ángel

@0aps added you as a contributor of this package (65ded02177054bdd444774a70d21087334b541ff).


Change in header tags and color theme

I have changed the texts and the color theme as there were too much text in the home page.
This PR fixes #118

Hey @Aditi323 , your commit has conflicts through we won't be able to merge.
We prefer the current one, if there is anything else you would like to work on pls go ahead:)


TypeError: fns.shift(...) is not a function

Reposting the issue from here https://github.com/alanshaw/it-pipe/issues/1

Circuit relay has been rewritten since this issue, closing as stale


Current ctx is no longer indicating the current context with kctx to list the contexts

This used to work just as the front page text gif demo'd when I was using Bash, but after switching to oh my zsh. What it brings up is a selector style menu but it doesn't indicate the current context, as seen below, it always has the cursor on ftp context :(

So I am wondering what caused it to have this behavior or how do I fix it?

What it brings up is a selector style menu

I'm assuming this behavior is consistent in both bash/zsh, it just looks at whether fzf is installed.

but it doesn't indicate the current context

what version are you on? v0.6.3 has fixed this: https://github.com/ahmetb/kubectx/releases

I'm assuming this behavior is consistent in both bash/zsh, it just looks at whether fzf is installed.
Yes, it is! I realized that I did just recently installed fzf plugin and behaviors on both shells are consistent. Thanks for the tip!

what version are you on? v0.6.3 has fixed this: https://github.com/ahmetb/kubectx/releases
I was on v0.6.2 and since upgrade to v0.6.3, and yes it does seem to highlight the current context in a different color (a little bit harder to discern though I guess I can tweak _KUBECTX_FORCE_COLOR env variable to highlight is a brighter vivid color of yellow?) Eitherway, thank you!

_KUBECTX_FORCE_COLOR is the wrong switch.
See KUBECTX_CURRENT_FGCOLOR, KUBECTX_CURRENT_BGCOLOR.


PBS video site broken again

PBS has changed their site to jwplayer.
I am working on a fix.

Fixed in pull request #174


Folder - documentation edited (check ???'d questions)

Done. Approve?
From: Vincent Pun<EMAIL_ADDRESS>Sent: Wednesday, September 5, 2018 4:41 PM
To: sourced/deploymentmanager-samples<EMAIL_ADDRESS>Cc: Alexander Levitski<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [sourced/deploymentmanager-samples] Folder - documentation edited (check ???'d questions) (#41)
@sourced-vince requested changes on this pull request.

In community/cloud-foundation/templates/folder/README.mdhttps://github.com/sourced/deploymentmanager-samples/pull/41#discussion_r215418165:

+- Install gcloud
+- Create a GCP project, set up billing, enable requisite APIs
+- ??? roles/permissions ???

Grant the resourcemanager.folderAdmin or resourcemanager.folderCreatorhttps://cloud.google.com/resource-manager/docs/access-control-folders IAM role to the project service account

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/sourced/deploymentmanager-samples/pull/41#pullrequestreview-152679540, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Aokzc8MraD_BpNU1jXt6dgMCOeGhdAbAks5uYDbHgaJpZM4WZp5h.


[Feature Request] Add suspend version of getLastUpdateState()

I'd like to suggest adding a suspend version of the getLastUpdateState() function.
Since Kotlin Coroutines are widely used, having a suspend version would simplify working with the API and make it more Kotlin idiomatic.
suspend fun BazaarUpdater.getLastUpdateState(
    context: Context
): UpdateResult = suspendCoroutine { cont ->
    getLastUpdateState(context = context) { result ->
        cont.resume(result)
    }
}

scope.launch {
   val result = BazaarUpdater.getLastUpdateState(context)
}

Thank you, Ali, for your suggestion. We've actually considered this before. If you take a look at getLastUpdateState, you'll see that it retrieves the scope from your context using the retrieveScope(context) function. Our goal was to ensure ease of use, but this can potentially create conflicts with the inner scope that we've drawn from context. Moreover, making getLastUpdateState suspendable could lead to memory leaks if developers mistakenly use the wrong scope.


Resize panes showing orthoviews and 3d view

Enable users to resize the panes.
This doesn't seem possible now...

This is indeed not supported --- a patch would be welcome though.


Maybe you need to add onSubmit: onTextSubmit in ChatInputFieldContainerWidget

Its an awesome plugin, and I have test it and its work!!! Just maybe you forgot this line. :)
Version 1.0.0 beta

thank you so much for the feedback <3
I'm still working on flutter 3 migration and adding some new features and bug fixes
that's why it's in beta

Waiting your next release.


ERROR: Cannot install -r requirements.txt (line 2) and pathway because these package versions have conflicting dependencies.

I'm also experiencing the same issue when I was trying to install the requirements.

Facing the same issue

Hello @arjun-ms @chriz-ty @munish0838.
Could you please let us know what Python version you're using? You can check it by executing python --version in a command line.
I have checked the dependencies of the llm_app library and I see that they refer to certain versions of pathway. It may be the case that the version you have installed is too old. Could you try to uninstall pathway and llm_app and run the pip install -r requirements.txt again?
On the other note, using a clean virtualenv to install the dependencies should also help.

Hey @zxqfd555-pw !
I am using Python 3.10. I believe the error I encountered is due to having installed this on a Windows machine, whereas the pathway package currently supports only Linux and macOS.
Now, I am attempting to run it on Docker.

Yeah, I tried to attempt in docker and it worked! But now there is another problem that I'm facing. The external url and the network url are not working..

Could you please let us know what Python version you're using? You can check it by executing python --version in a command line.
I have checked the dependencies of the llm_app library and I see that they refer to certain versions of pathway. It may be the case that the version you have installed is too old. Could you try to uninstall pathway and llm_app and run the pip install -r requirements.txt again?
On the other note, using a clean virtualenv to install the dependencies should also help.

Hii @zxqfd555-pw , my python version is Python 3.12.1
Still finding the same error, even tried to make a new environment for it

see conclusion of https://github.com/pathway-labs/dropbox-ai-chat/issues/5


Consolidate ModelBuilder InferenceData getters (and setters)

Think the ModelBuilder class itself could just get the getters (and setters) for:

prior
prior_predictive
fit_result (posterior)
posterior_predictive

They are defined here for mmm:
https://github.com/pymc-labs/pymc-marketing/blob/664b5caef1f20785a8f73cac3c4f83a13ceeeee7/pymc_marketing/mmm/base.py#L266-L290)
and here for clv:
https://github.com/pymc-labs/pymc-marketing/blob/664b5caef1f20785a8f73cac3c4f83a13ceeeee7/pymc_marketing/clv/models/basic.py#L242-L256
Would the clv modules benefit from this? @ColtAllen
I'm leaning more toward standardization of code and having informative error messages

Would the clv modules benefit from this? @ColtAllen I'm leaning more toward standardization of code and having informative error messages

CLVModel overwrites quite a few of the inherited ModelBuilder methods because the latter is built around an "X,Y" variable convention that doesn't apply to the CLV models. That said, the idata methods could benefit from some cleanup, particularly if it gets rid of the following warning that pops up during testing:
~/site-packages/arviz/data/inference_data.py:1538: UserWarning: The group fit_data is not defined in the InferenceData scheme
This warning is probably extraneous from the ModelBuilder class, but also worth mentioning:
~/site-packages/pymc/model/core.py:518: FutureWarning: All coords are now mutable by default. coords_mutable will be removed in a future release.

This warning is probably extraneous from the ModelBuilder class, but also worth mentioning:
~/site-packages/pymc/model/core.py:518: FutureWarning: All coords are now mutable by default. coords_mutable will be removed in a future release.

Yeah, I think we should get rid of these. Have this in mind #669

CLVModel overwrites quite a few of the inherited ModelBuilder methods because the latter is built around an "X,Y" variable convention that doesn't apply to the CLV models. That said, the idata methods could benefit from some cleanup, particularly if it gets rid of the following warning that pops up during testing:

Maybe there can be two versions of the ModelBuilder? There can be some base class that will handle this common logic of getters and setters. Then two child classes with the X at initialize and X,y in the methods.
Just an initial thought...


[Feature][Zeta Engine UI] Add zeta engine UI.

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
https://summer-ospp.ac.cn/org/prodetail/248940398?list=org&navpage=org
Project Overview
Project Name: Zeta Web UI Page Management
1. Project Background
Currently, SeaTunnel's self-developed engine, Zeta, does not have a web UI that can intuitively display the system's health status and job execution status. Therefore, the main goal of this project is to develop a web UI for Zeta to address these issues.
2. Core Features
2.1 Dashboard
Description: Provides an overview page that intuitively displays basic system information and job summaries.
Features:

Displays the number of running jobs, completed jobs, and failed jobs.
Shows system version and resource usage (number of slots, version, etc.).

2.2 Job Management
Description: Provides detailed job management features. Users can view, start, pause, and stop Zeta jobs.
Features:

Lists all Zeta jobs, including job name, status, start time, duration, etc.
Supports filtering jobs by status, name, time, etc.
Job details page displaying detailed information about the job, including DAG (Directed Acyclic Graph), runtime, parallelism, etc.
Supports operations to start, pause, and stop jobs.
Provides job log viewing functionality with support for searching and filtering log content.

2.3 Task Management
Description: Provides management features for individual Zeta tasks, allowing users to view task execution status and performance metrics.
Features:

Displays a list of tasks, including task name, status, start time, duration, etc.
Task details page showing detailed information and execution status of the task.
Provides performance metrics for tasks (throughput, runtime, etc.).

2.4 Cluster Management
Description: Manages and monitors the Zeta cluster, viewing resource usage and health status of the cluster.
Features:

Displays a list of cluster nodes, including node name, status, role (worker/master), CPU and memory usage, etc.
Node details page showing detailed information and running status of the node.

3. Acceptance Criteria

Zeta UI functional design document.
Zeta UI code PR, which needs to be finally merged into the main branch.
Zeta UI functionality meets the requirements and runs normally.
Zeta UI usage documentation.

4. Reference
https://seatunnel.apache.org/docs/2.3.5/seatunnel-engine/about
https://seatunnel.apache.org/docs/2.3.5/seatunnel-engine/rest-api

中文描述

项目概述
项目名称: SeaTunnel Zeta Engine UI 开发
1. 项目背景
目前SeaTunnel 自研引擎Zeta还没有一个 ui页面可以直观的展示系统的健康状况以及作业的执行情况，所以本题的主要目的就是为Zeta 开发一个web 页面解决这些问题。
2. 核心功能
2.1 仪表盘
描述: 提供一个总览页面，可以直观展示系统基本信息和作业的总览。
功能点:

显示正在运行的作业数、已完成的作业数、失败的作业数。
展示系统版本和资源使用情况（slot数、版本信息等）。

2.2 作业管理
描述: 提供详细的作业管理功能，用户可以查看、启动、暂停和停止Zeta作业。
功能点:

列表展示所有 Zeta 作业，包含作业名称、状态、开始时间、持续时间等信息。
支持按状态、名称、时间等条件筛选作业。
作业详情页面，展示作业的详细信息，包括 DAG（有向无环图）、运行时长、并行度等。
支持启动、暂停和停止作业的操作。
提供作业日志查看功能，支持搜索和过滤日志内容。

2.3 任务管理
描述: 提供对单个 Zeta 任务的管理功能，查看任务的执行情况和性能指标。
功能点:

展示任务列表，包括任务名称、状态、开始时间、持续时间等。
任务详情页面，展示任务的详细信息和执行情况。
提供任务的性能指标（吞吐量等信息）。

2.4 集群管理
描述: 监控 Zeta 集群，查看集群的资源使用情况和健康状态。
功能点:

展示集群的节点列表，包含节点名称、状态、角色（worker/master）、CPU 和内存使用情况等。
节点详情页面，展示节点的详细信息和运行状态。

3.完成标准

Zeta UI功能设计文档
Zeta UI代码PR, PR需要最终merge到主分支
Zeta UI功能满足需求说明，正常运行
Zeta UI的使用文档

4.参考
https://seatunnel.apache.org/docs/2.3.5/seatunnel-engine/about
https://seatunnel.apache.org/docs/2.3.5/seatunnel-engine/rest-api
https://github.com/apache/seatunnel/issues/6932
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

您好，我想接下这个Feature点的开发，并且以此为契机加入seatunnel开源社区


Global BindingBehavior default

Is your feature request related to a problem? Please describe.
We do not want anyone changing the schema of our graphQl endpoint by accident, so we tell each graph type to explicitly bind all of the fields for that type. However, new developers might not know to set that explicit flag and accidentally expose fields they did not mean to include in the external schema.
Describe the solution you'd like
Ideally we'd have a spot in the ISchemaConfiguration that calls BindFields(Explicit) on each object type as it is being constructed for the schema.
Describe alternatives you've considered
My current implementation is to manually set Explicit for each of our object types
I'd be glad to work on this, but it doesn't seem like there's an obvious place for this to be handled, so I'd need a little guidance on where in the object type creation process we'd do such a check.

Let me have a look at this. We are planning to open up the schema building process so that you can add your own discovery service, how the casing of fields should be etc.

This is related to  #320

Let me work a few more days on the schema builder design. I think once I got this we both could have a look how we break up the descriptor services. The thing here is that I do not want to add some static defaults that would impact all schemas in the same process.
I would rather like to define it on a schema base. So, basically that you could say this specific schema behaves in a certain way and has certain defaults.
What do you think?

For the meantime you could create a base class like the following:
public class ExplicitObjectType<T>
        : ObjectType<T>
    {
        protected override void Configure(IObjectTypeDescriptor<T> descriptor)
        {
            descriptor.BindFields(BindingBehavior.Explicit);
        }
    }

Yeah I would agree. If I understand the code base correctly, an ISchemaConfiguration is scoped to a single schema, which is why I would think we'd have a method on that that can set and provide the default BindingBehavior as the schema is being constructed. The Base class is a good idea, I may try that.

accidentally closed.

I am moving this to version 0.8.0 since we will focus on enhancement in the type system.

This is implemented with 10


remove cli from linter - folder was deleted recently

all of
FILTER_REGEX_EXCLUDE: cli/.*
in
https://github.com/GoogleCloudPlatform/pubsec-declarative-toolkit/blob/main/.github/workflows/linter.yaml

https://github.com/GoogleCloudPlatform/pubsec-declarative-toolkit/issues/863


Bluetooth services and characteristics missing from hex file

The following simple application should result in the three specified Bluetooth services being included in the resultant binary:

The corresponding hex file is attached in the zip file.
microbit-Bitty-Logger-fie.zip
After installing, pairing and resetting, examining the Bluetooth services on the micro:bit using nRF Connect we see we have:

which includes the accelerometer service but does not include the magnetometer or temperature services.
Looking at the characteristics in the accelerometer service:

we can see that it only contains the Accelerometer Data characteristic. It is missing the Accelerometer Period characteristic.
See https://lancaster-university.github.io/microbit-docs/resources/bluetooth/bluetooth_profile.html
This is may be due to insufficient memory being reserved via the gatt_table_size property in config.json
Recommended setting is:
"gatt_table_size": "0x700"

We already have this setting specified in the bluetooth project.

This is the yotta config generated by PXT:
{
  "config": {
    "microbit-dal": {
      "bluetooth": {
        "enabled": 1,
        "private_addressing": 0,
        "advertising_timeout": 0,
        "tx_power": 6,
        "dfu_service": 1,
        "event_service": 1,
        "device_info_service": 1,
        "eddystone_url": 1,
        "eddystone_uid": 1,
        "open": 0,
        "pairing_mode": 1,
        "whitelist": 1,
        "security_level": "SECURITY_MODE_ENCRYPTION_NO_MITM"
      }
    },
    "gatt_table_size": "0x700"
  },
  "dependencies": {
    "microbit": "lancaster-university/microbit#v2.0.0-rc7"
  }
}

gatt_table_size looks good. Something else must be pushing GATT memory use over the edge. No need for eddystone_url or eddystone_uid to be enabled unless the corresponding block is used. Maybe that's the issue. I can check with C/C++ tomorrow if I get time but that's all I can think of right now. It's easy to test though... use nRF Connect to discover the services on the micro:bit. Use the "Refresh Services" function to ensure you're looking at data from the micro:bit as opposed to cached GATT data from earlier access to the same micro:bit.
If we can't see the issue between us, we'll have to consult LU. Good be unrelated to config.... other changes in the DAL might be a factor (theoretically).

I think I see the problem. The gatt_table_size attribute is misplaced. It should be at the same level as the "bluetooth" object like this:
{
    "microbit-dal": {
      "bluetooth": {
        "enabled": 1,
        "private_addressing": 0,
        "advertising_timeout": 0,
        "tx_power": 6,
        "dfu_service": 1,
        "event_service": 1,
        "device_info_service": 1,
        "eddystone_url": 1,
        "eddystone_uid": 1,
        "open": 0,
        "pairing_mode": 1,
        "whitelist": 1,
        "security_level": "SECURITY_MODE_ENCRYPTION_NO_MITM"
    },
    "gatt_table_size": "0x700"
  },
  "dependencies": {
    "microbit": "lancaster-university/microbit#v2.0.0-rc7"
  }
}

I tested with the original config.json posted above and recreated the problem. I guess the effect is that the gatt_table_size property is ignored and the default of 0x300 applied instead.

The fix is in beta.

Tested and working. Thanks @pelikhan


Gen Bitstream Vivado observation

If it says files unchanged, it means vivado did not 'register' that you changed the file. Does not seem to happen when using built in editor. May just be temporary bug

Solution
Refresh hierarchy manually when this bug occurs. open file in local editor and see that changes are there. can also see it if the file is open in local editor and no reload option is present (file outdated)

Vivado does have a few bugs like this, yes.  Best solution I found is the manual refresh you're talking about.


Remove network-connected and default test-related Maven profiles

Fixes #1073

+1!

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1345/
Test PASSed.

Merged as 496add6f99e5378f0e2df0c03a1d7db0cddf01c9. Thanks @heuermh!


chore(): add sort module to universal app

Adds the MdSortModule to the universal app. This means that it will be part of the prerender check on the CI.

Note: It makes sense to check the mdSort module because it includes components and directives that may cause problems in the server platform.

please rebase

@mmalerba Done.


shows 'Dependabot version updates aren't configured yet' even when it is configured

Package ecosystem: maven
dependabot.yml content
https://github.com/rabobank/shadow-tool/blob/main/.github/dependabot.yml

Even though I have a dependabot.yml file configured that should apply to all dependencies. What did I do wrong

nevermind can be closed as adding the following to our action solved it
    - name: Maven Dependency Tree Dependency Submission
      uses<EMAIL_ADDRESS>


Enable Clang static analyzer build.

The changes are pretty simple, except for the tweaks to speed up
the build.  This is a fix for #10.

PTAL.
We should wait until the PR finishes to see how long the builds take, we might want to disable the static analysis for PRs.

SGTM on disabling static analysis for PRs, just run it on the master branch, no need to slow down PR review/merge process.

PTAL.  I think I addressed all your comments, but I have rebased so many times I might have missed some.  Oh, and we need to wait for it to pass the tests, of course.

I am going to close this PR because the build is still too slow, it just went from 37 minutes to 50+ minutes and aborted on my branch.

Per offline conversation, please resurrect the part to allow folks to manually run this from the CLI, but leave out the part that automatically runs it on Travis CI — it will still be useful for us in the future to be able to easily run this on our workstations, even if it can't run automatically on Travis CI.


Add continuous delivery infrastructure

Once we have CI (#28), it would be great to automate deployment of the image to docker. I would love that just pushing a tag would trigger an automated process. Alternatively would be great if just merging to main trigger tagging and push automatically to docker.

#33 should solve this.

Small issue left with Android image, hopefully #35 should fix it.


[GitHub] Fix Travis-CI GitHub rate limiting

Attempt to fix #979 limit on GitHub request.
Travis-CI Environment variable github_token will need to be set for this to work though.

Thanks for this. I'd rather not add one-off logic to deal with our internal uses. While it'd be nice to provide a way to load secrets from the environment (rather than the filesystem) I'd want to do that in a holistic way. I opened #1095 which is an alternate approach. See what you think.


feat: Add clearer ui for asset download permissions

Description
Depending on its organisation, a user doesn't always have the permissions to download a function or opener. Currently, the user could still click on the download button when he didn't have the permissions, it's just that nothing would happen.
This PR aims for a clearer UI by disabling the download button and adding an informative tooltip when the user doesn't have permission to download.
Fixes FL-1072

It seems to me similar code is repeated multiple times, but I don't see an easy way to refactor and you're the expert

It seems to me similar code is repeated multiple times, but I don't see an easy way to refactor and you're the expert

No you're right ! We could make it a custom hook, and turns out there already was one i forgot about. Refactored 👍


Add a phantom type to caracterize Bytes

So I've done here what I proposed in option (3) of #12 , consisting of adding a phantom type to Bytes. As a result we now know what each and every Bytes contain, so this is much more readable. As a bonus, the BytesMap type now also has that phantom type for the key of the map, which also make it much more readable.
type Bytes a = Bytes String

type BytesMap k v = BytesMap (Dict String v)

singleton : Bytes k -> v -> BytesMap k v

This means I was able to get rid of the Hash module, as now all hashes are just Bytes with an appropriate phantom type.
type Credential
    = VerificationKeyCredential (Bytes CredentialHash)
    | ScriptCredential (Bytes CredentialHash)

{-| Phantom type for 28-bytes credential hashes.
This is a Blake2b-224 hash.
-}
type CredentialHash
    = CredentialHash Never

As you can see in that example, most phantom types used for all the Bytes type are opaque (constructor not exposed) and also defined using the Never type in their only constructor. This makes extra sure that no value of type CredentialHash can ever be created, and this can only serve as an annotation for another type with an "phantom type hole", such as Bytes a.
There are 1 or 2 exceptions like in places where there is Bytes Address for example where Address is also a concrete type that can be used on its own. But I think it's clear enough that it should not create any issue.
As much as I can, I've tried to create adequate phantom types for their corresponding bytes. I hope I haven't misinterpreted some of the types from what I've read in the previous code and in the CDDL.


[gatsby-source-graphcms]: Problem building GraphCMS nodes // Error: Bad Request

Public API, with default permissions (Read all models, Published stage)
https://api-ca-central-1.graphcms.com/v2/ckrikr0k50viq01xs8vhc5984/master
Commented out all queries to ensure that I didn't accidentally add non-existent properties. Still get this error with no idea what is happening -- the only thing I have setup is in the gatsby-config.js file:

This should be resolved in the latest next release. I believe it was to do with RichText fields having embeds enabled causing this, and swallowing the error.


Update directv.com.ar

Channel data stilll needs a lot of work, some channels have changed IDs and others are not available anymore on DirecTV. I'll try to help but can't promise anything.


UserWarning: RNN module weights are not part of single contiguous chunk of memory. and how to generate probability of a setence

I am getting the following warning:
UserWarning: RNN module weights are not part of a single contiguous chunk of memory. This means they need to be compacted at every call, possibly greatly increasing memory usage. To compact weights again call flatten_parameters().
I am using pytorch 0.20 and python 3.5
Also, how do you generate a probability of a given sentence?

Hi @dixiematt8,
This is an expected UserWarning and isn't a problem. As the weights are changed before each call of the LSTM, they'd need to be compacted anyway.
Re: probability of a given sentence, that is not defined. As language is infinite in the sentences it can generate, there's no well defined probability distribution. You can see which sentences may be more of less likely but that's about the best I could suggest.
For that you'd multiply the probabilities (or sum the log probabilities for a numerically more stable approach) of each of the words according the model's prediction and select the sentence with the larger probability.
Hope that helps!


Add extension docs for copyright, eula and support info (CustomizationsMetadata)

Should include implementation change in https://github.com/cloudfoundry-incubator/stratos/pull/3964


Send a response action gets created with Send an Activity as the name

When a Send a response action gets created, the name field on the property pane says Send an Activity. But the node title says Send a response.

I've also seen in a couple areas that overridden labels are not getting applied. See attached. Adaptive Dialog should actually say the name of the Bot (TodoWithLuis)

@cwhitten that particular issue is due to malformed dialog data.

@sangwoohaan what is the suggestion here? Which label should we use?

@a-b-r-o-w-n the example was a vanilla creation from sample. it's unclear what is malformed.

The sample has
{
  $designer: {
    $designer: {
      ...
    }
  }
}

I will update it.


Migrate docs to use Intellij instead of Eclipse

Similar to level 1

@limmlingg can do this one too?


Installing on Mac

I have tried to follow the directions provided.
After downloading macos binary from releases, I am unable to open it. I get an error:

How do I install vessel after downloading it?
Thank you.

It's a binary. You should move it to a location that's on your path and rename it to vessel. You probably also need to run chmod +x vessel on it.

here is what I have done.
❯ ls $HOME
Applications/ Exercism/     Movies/       Public/       gist/         smlformat/
Desktop/      Github/       Music/        bin/          node_modules/ vessel*
Documents/    Heroku/       Pictures/     docs/         npmbin@
Downloads/    Library/      Postman/      dotfiles/     polyml/

Im in fish shell:
~/.config/fish master* ⇡
❯ set -Ua fish_user_paths $HOME/vessel

Running vessel init gets me ```sh
❯ vessel init
fish: Unknown command: vessel

Though it works if I run vessel init from the same directory that its in. So likely i am setting the path wrong....

Thanks.
❯ wget -O $HOME/bin/vessel https://github.com/kritzcreek/vessel/releases/download/v0.4.1/vessel-macos
~/Downloads/MOKOTO/calculator2 master
❯ set -Ua fish_user_paths $HOME/bin/

~/Downloads/MOKOTO/calculator2 master
❯ fish
~/Downloads/MOKOTO/calculator2 master
❯ which vessel
/Users/mandalarian/bin//vessel


Fix some tree command outputs in the Quickstart guide

Some of my changes in #41 were overwritten by #40. Fixing this here and including the top level directory in the output of the tree command.

Closing in favour of https://github.com/FuelLabs/fuel-docs/pull/43


tests: net: socket: tcp: Add test for an interface down

A reproducible case to the behavior when during a connection the interface goes down and all data is lost, the TCP connection should properly terminate. This test helps to reproduce the issue of https://github.com/zephyrproject-rtos/zephyr/issues/44473

I addressed a few comments, but worryingly bumped in some issues with the test_v4_send_recv_large.
In my PC I got the following log:
===================================================================
START - test_v4_send_recv_large
 PASS - test_v4_send_recv_large in 18.452 seconds
===================================================================
START - test_v4_send_recv_large

    Assertion failed at WEST_TOPDIR/zephyr/tests/net/socket/tcp/src/main.c:341: test_v4_send_recv_large: (send_bytes bigger then 0)
Error sending 256 bytes on top of 45568, got -1 in iteration 178, errno 12
 FAIL - test_v4_send_recv_large in 12.617 seconds
===================================================================

It looks to be the CI timed out on this test:
INFO    - -/__w/zephyr/zephyr/twister-out/mps2_an385/tests/net/socket/tcp/net.socket.tcp.preempt/handler.log--
INFO    - 104/104 mps2_an385                tests/net/socket/tcp/net.socket.tcp                FAILED Timeout (qemu)
INFO    - -----/__w/zephyr/zephyr/twister-out/mps2_an385/tests/net/socket/tcp/net.socket.tcp/handler.log------

===================================================================
START - test_v4_send_recv_large

INFO    - -----/__w/zephyr/zephyr/twister-out/mps2_an385/tests/net/socket/tcp/net.socket.tcp/handler.log------

In my PC I got the following log:

What platform? I can't reproduce this locally on qemu.

The failed tests I have seen on multiple platforms, the fail at my PC was with the Qemu X86. Back then I had CONFIG_NET_BUF_RX_COUNT=64
CONFIG_NET_BUF_TX_COUNT=64
And a k_yield() after the send.
In the meantime, I reverted this to by removing the mentioned lines.
The fault on the CI platform came at varying platforms. In this particular case on mps2_an385.
The challenge was that is did only sometimes failed, I did not see a reproducible case. It gives the indication that something is not totally right. It would help if we can build into a reproducible testcase, this would aid solving the issue. Any suggestions in which direction to look to build a reproducible failing testcase?

@ssharks Ok, I've reproduced the issue with
CONFIG_NET_BUF_RX_COUNT=64
CONFIG_NET_BUF_TX_COUNT=64

but, with no k_yield() after send - the test works fine if yield is present. I've briefy identified the issue to be RX net_pkt starvation, for some reason RX pkts are not freed. I'll investigate further to see what is the reason of this.

@ssharks I've submitted a PR with a fix (https://github.com/zephyrproject-rtos/zephyr/pull/45018) - it turned out to be a deadlock between TCP/net_context mutexes, introduced recently with SO_SNDBUF option implementation. Please give it a shot

Nice find I tried, but could not easily reproduce the problem.


Implement IDisposable in bidi/server streaming call wrappers

As reported on https://github.com/grpc/grpc-dotnet/issues/2086, if we don't dispose or read server streaming calls to completion, the process will leak memory.
The underlying gRPC call implements IDisposable, but our wrappers (ServerStreamingBase and BidirectionalStreamingBase) don't, which makes it a) harder for callers to do the right thing; b) harder to spot that anything needs disposing.
We should at least implement IDisposable, and possibly add helper methods to make it easy to consume all responses.

This was implemented in #692. (We didn't include any helper methods to consume all responses.)


[0.7.0-Alpha] How to generate all reports (html, xml) together ?

Describe what you would like to clarify about Kover
Not able to see "koverMerged" option anymore. How to generate all reports (html, xml) together ?
and where to mention xml {} , html{} properties ?

Hi,
to generate HTML and XML reports, call Gradle tasks koverXmlReport koverHtmlReport together.
To configure reports of JVM or multiplatform project, use the extension
koverReport {
    xml {  }
    html {  }
}

Please also refer to migration guide or example projects


Menu Items: Make installation from menu clearer

A lot of feedback tend to show that there was a confusion regarding what was doing what between "Debug/Install from lockfile" and "Install Dependencies". This moves the lockfile installation out of the debug section as it is not a debug-only feature and is used quite frequently.
Moreover, this clarifies the messages to better explain the difference between the installation methods.

By moving it first aren't we modifying the default user behavior?
See also

for reference


the next step of background color

I have updated other color now.

@xiaoshaotongzhi please resolve the conflicts first