powershell -executionpolicy bypass

PS C:\WINDOWS\system32> powershell -executionpolicy bypass
Windows PowerShell
Copyright (C) 2015 Microsoft Corporation. All rights reserved.
Oops, something went wrong.  Please report this bug with the details below.
Report on GitHub: https://github.com/lzybkr/PSReadLine/issues/new
Last 0 Keys:
Exception:
System.IO.IOException: The handle is invalid.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.Console.GetBufferInfo(Boolean throwOnNoConsole, Boolean& succeeded)
at Microsoft.PowerShell.PSConsoleReadLine.Initialize(Runspace runspace, EngineIntrinsics engineIntrinsics)
at Microsoft.PowerShell.PSConsoleReadLine.ReadLine(Runspace runspace, EngineIntrinsics engineIntrinsics)

I am also receiving this error. Anything I can do to help diagnose?

Most likely a dupe of #158, reopen with additional information if you disagree.


Remove references to service classes in the src/test/java

With the migration to groovy style tests, we do not need the test service classes in test/java. The services are in the following package https://github.com/graph-quilt/graphql-orchestrator-java/tree/master/src/test/java/com/intuit/graphql/orchestrator
An example would be BookService
Delete these service class and migrate all the test referring to these services to groovy. Example of groovy test
https://github.com/graph-quilt/graphql-orchestrator-java/blob/master/src/test/groovy/com/intuit/graphql/orchestrator/integration/TopLevelStitchingSpec.groovy
More tests in this package

@ashpak-shaikh can I take this issue?


feat: add node 18 support

This library now supports Node 18.
Description

Node 18 added to all GitHub actions.
Fail-fast now activated on the any-pr action - it might fail on one version of Node, but build on another. It is helpful to know this.
Neon now upgraded to 10.1.
Code in macros.rs needed to be adjusted for the new version of Neon. For example, object property access has changed.
Dependencies like @mapbox/node-pre-gyp also updated.

~Tests for the changes have been added (for bug fixes / features)~

[x] The commit message(s) follow conventional commits
[x] Documentation has been added / updated (for bug fixes / features)
[x] Changes follow the contributing document.

Motivation and Context
Part of MATTR's push to Node 18.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Which merge strategy will you use?

[x] Squash
[ ] Rebase (REVIEW COMMITS)

@cowboy-bebug Fixing the clippy related stuff is a good idea, but out of scope for this work item. I agree the repo should lint. It should check formatting


Ignore archived upstream repos in show-open-prs-of-forks

Description
This alert is pretty sweat, but maybe ignore those PRs when the upstream already archived the repo will be even more awesome 😄
This repo has open issues/PRs, are you sure you want to delete everything?
Screenshot

Example URLs
https://github.com/PeterDaveHello/js-signals

when the upstream already archived the repo

I don't see any archived repos in your example URL and in its fork source. Did you see the alert when trying to delete the linked repo?

Oh, sorry, I must paste a wrong url, trying to find out the example needed here, I remember that I deleted already 😅

Okay now I found another example:

https://github.com/PeterDaveHelloKitchen/imagelayers-graph

Indeed! #6042 should fix that and also hide show-open-prs-of-forks, which has the underlying issue.

Is it possible to have a not that aggressive strategy that we can still know that there are PR opened, just with hints that the upstream was archived?

I don’t think you can do anything about PRs on archived repo.
The issue here specifically is that the deletion feature just depends on the element being on the page, it doesn’t make its own query. It’s not worth here to double the query (archived or not archived) just to avoid a warning (which is still correct)


Synced Immutable and plain exports (+ tests)

I stumbled across #2025 today, and thought I could try my hand at creating a PR to help out:

Mirrored exports in src/immutable.js that already existed in src/index.js:

getFormSyncErrors
getFormAsyncErrors
getFormSubmitErrors
registerField
unregisterField

Re-sorted position of autofill in src/index.js to be alphabetical and match the other files
Added tests to src/__tests__/immutable.spec.js that already existed in src/__tests__/index.spec.js:

arrayMove
arrayRemoveAll
autofill
FormSection
registerField
unregisterField

Added tests to both src/__tests__/index.spec.js and src/__tests__/immutable.spec.js that were missing for some exports

Fields
Form
getFormValues
getFormSyncErrors
getFormAsyncErrors
getFormSubmitErrors
isDirty
isInvalid
isPristine
isValid
isSubmitting
hasSubmitSucceeded
hasSubmitFailed
submit

Current coverage is 100% (diff: 100%)

Merging #2425 into master will not change coverage

@@           master   #2425   diff @@
=====================================
  Files          65      65
  Lines        1315    1315
  Methods         0       0
  Messages        0       0
  Branches        0       0
=====================================
  Hits         1315    1315
  Misses          0       0
  Partials        0       0

Powered by Codecov. Last update 3ee630e...eee5b9c

Wow. An embarrassing oversight on my part. 😳  Thanks! 👍

Released as v6.5.0.


gitlab ci allure generate job

What?
Add job with allure report generation and store generatedd allure-report as artifact
Why?
To make the next step with collecting allure-reports possible.

Done.


Question: innerHTML

Is there a directive or option to specify the inner html of an element if, for example, that "content" is stored in a global variable?

Can you give an example?

@dy
I have an element div#modal.
On page initiation it has innerHTML:
div#modal-header
div#modal-content
div#modal-footer

But after some user actions I have to change this content.
So, if i correctly understand, I have to change some variable and then div#modal will change it's innerHTML reactively

usually I either put :if condition or :hidden flag. If you need whole html content to be taken from somewhere else either use :ref=el :fx=el.innerHTML=str, or :html=otherElement

Ok. Thanx!


Please add license

Please add a license, as we would like to ship this software in NixOS. Thank you.

Done :)
Public-domain license CC0


Remove hardcoded path to choco.exe and use %ChocolateyInstall% / $env:ChocolateyInstall env variable to find it.

It should not be assumed that Chocolatey choco.exe is found in c:\programdata\chocolatey as it can be installed elsewhere.
Upon install, Chocolatey sets a system environmental variable pointing to where it was installed, this is %ChocolateyInstall%/$env:ChocolateyInstall and that environmental variable is best practice to find the location of choco.exe. v0.98(?) and before used c:\chocolatey
Also, Chocolatey doesn't install apps, it installs packages which in turn are most likely programs.

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

When TRMM installs choco, it requires a reboot to initialize the environment variables. That's why the path was updated to the current values (will work for onboarding tasks prior to agent reboot).
Although %ChocolateyInstall% may be best-practices for Chocolatey, this will reintroduce the problem of failing choco installs till a restart has occurred. I can also see if you have separate Choco installs along with TRMM managed choco installs. TRMM only installs Choco to the path in this script and is not customizable. What is going to be the final results of that Environment variable?
Do you have any other ideas on how we can handle all these issues?

ChocolateyInstall env variable is available immediately after installation of Chocolatey, I can't confirm how far back version wise this is true, but in recent versions (probably back to before .9x though) it's true and I just confirmed it in a VM with 2.2.2.
At the end of a Chocolatey install it runs C:\ProgramData\chocolatey\redirects\RefreshEnv.cmd which refreshes the environment ('natch) making the ChocolateyInstall and ChocolateyLastPathUpdate variables available to the shell as well as adding C:\ProgramData\chocolatey\bin to the path (foreshadowing here).
DISCLAIMER: I have not installed Chocolatey via TRMM. I've got only a few hours experience with TRMM. I tried the "Chocolatey - List Installed apps" script and it worked perfectly. I found the "Chocolatey - Install, Uninstall, List and Upgrade Software" script to be unintuitive as I couldn't get it to do anything and there is no option to view the script or get help with parameters for the script inside TRMM that I could find during my minutes of testing.... that's what led me to find the source of the actual script.
You mentioned "TRMM only installs Choco to the path in this script and is not customizable" - I haven't seen the script that installs Chocolatey yet. I just did a quick look through the community scripts and didn't see it. I'll have to go digging further later.
I need to do further research looking at and running the rest of the related scripts, but to handle these issues (I should have quoted your response questions and replied inline!) would be to change
$chocoExePath = "$env:PROGRAMDATA\chocolatey\choco.exe"

if (-not (Test-Path $chocoExePath)) {
    Write-Output "Chocolatey is not installed."
    Exit 1
}

to first test $env:ChocolateyInstall exists and set $chocoExePath if so. Second I would look for choco.exe in the the way you are doing it which will most likely be the place 99.99% of the time. The next logical way would be to use Get-ChildItem to find choco.exe (there are three, the one in bin "should" be used) which would be intensive as it should check all available drives; i.e. a locked down computer user might have Chocolatey installed in their user folder or possibly on a network $env:HOMESHARE.
Expounding on the choco.exe "should" be used from bin folder.... When Chocolatey installs it adds c:\programdata\chocolatey\bin to the path wherein lines a choco.exe, a shim that runs c:\programdata\chocolatey\choco.exe. It's probably best to point to the shim in the bin. I don't know any reason that it needs to use the shim as running the choco.exe from c:\programdata\chocolatey works just fine, BUT I'm guessing there IS a reason the devs configured Chocolatey by default to run the shim... Running choco.exe proper instead of the shim may have consequences during a Chocolatey program upgrade - I'm completely guessing here; just trying to keep Chocolatey as (ack hum - clearing throat getting ready for the pun...) "vanilla" as possible.

Can't beat a little white chocolate 😂

Choco install is either part of agent repo....or it's a payload delivered by TRMM. I'm guessing it's part of agent install though.
Community Library is just scripts for TRMM (designed to run from SYSTEM)...not specifically about Choco.

I'll try and retest virgin system with install to see if Choco in the env is still a problem or not. Been a year since I last did thorough testing.
The Choco install script used to warn about close/cmd/powershell windows after install to load env. Does it still do that?

Can't beat a little white chocolate 😂

Personally not a fan. :-) It's a misnomer. There is no chocolate in white chocolate! Nestlé attempting to FTW (fool the world).

Community Library is just scripts for TRMM (designed to run from SYSTEM)...not specifically about Choco.

100% understood. Repo is aptly and intuitively named.

The Choco install script used to warn about close/cmd/powershell windows after install to load env. Does it still do that?

Yepper depper.

WARNING: It's very likely you will need to close and reopen your shell
before you can use choco.
PATH environment variable does not have C:\ProgramData\chocolatey\bin in it. Adding...

Immediately post install (tested seconds ago):

dir env:

ChocolateyExitCode             0
ChocolateyInstall              C:\ProgramData\chocolatey
ChocolateyLastPathUpdate<PHONE_NUMBER>36589791
(results truncated)

Yepper depper.

WARNING: It's very likely you will need to close and reopen your shell
before you can use choco.
PATH environment variable does not have C:\ProgramData\chocolatey\bin in it. Adding...

So then nothing's changed, which isn't surprising. As this is just how programs and their runtime env in the os work. Would love to use best-practices of having a proper choco env var...but because:

TRMM agent install installs choco and its env
trmm agent doesn't restart/reload it's runtime env...it won't have the envs yet without a trmm agent .exe/service reload
Someone might have onboarding scripts that trigger right away before agent/PC restart

I don't think updating scripts to use this path method. Am I missing anything?

Closing. Will create support issues in the field to use choco env vars.


OAProject should implement ICustomTypeDescriptors/IDispatch

OAProject from CPS should be extended to implement  ICustomTypeDescriptors/IDispatch which should provide common project properties.
More info

Fixed by #1842


So many warnings!

My current project has an in-tree HAL implementation. I now can't see any of the build errors because they're hidden among umpteen pages of deprecation warnings for the v1 hal traits.
Is there any easy way to shut these warnings up?

Does #![allow(deprecated)] help?

v1/v2 traits have been removed in v1.0.0 (#177) which I believe resolves this issue


Remove author annotations

We've decided to no longer add author annotations since we can't add everyone who contributes to the list.
Instead, refer to the Github contributor list.

LGTM.


Update for the Monster Module

Here is the updated spreadsheet for my Monster Module, which is already included in Kobold Fight Club. As this new version is both more balanced and includes more monsters than the previous one, the existing spreadsheet should be updated to match this one.
https://docs.google.com/spreadsheets/d/1WSwYu3iWXYBy-7xsx1qhomJEynPKWMOU32MnDNZS17w/edit?usp=sharing
PDF Link: https://drive.google.com/file/d/0B-g9vLTX0eHKWDZHZ05QWkVNZVU/view
Reddit Thread Link: https://www.reddit.com/r/UnearthedArcana/comments/95nd8m/the_monster_module_ac_now_doubled_in_size_to_48/

So... bit of a log-jam here.
I've decided to move away from the "master sheet" paradigm, and instead make it easier to share and load whatever content people want. The problem with this specific thing is that, since it's an update to content that's already in the master sheets, there are a lot of duplicated monsters and KFC currently doesn't handle that well.
This sheet won't be able to be added until #149 or (more likely) #105 are implemented.


Broken Section Linking in Answer Key

Currently lettered sections in the answer key link back to the first instance of that letter in the book, usually in the first chapter.

The links for the practice problems now correctly link back to the chapter sections.


chore(server): What if we remove autotest/Interface

Checklist

[x] the contributor license agreement is signed
[x] commit message follows commit guidelines
[ ] tests are included
[ ] screenshots are included showing significant UI changes
[ ] documentation is changed or added

Description of change

AFAICT this is almost a duplicate of the Interface folder in the root
directory. But if not, we can always jump back to the old
openequella/openEQUELLA-autotests repository and bring it back - if we
can't find the merge commit to revert.
I'd be inclined to say if we end up with a green build, then this should be an easy merge.
Also, this will remove a significant number of CheckStyle errors. :wink:

Initial CodeBuild attempt failed on a single ScalaCheck test - seems like a flake.

I reckon it's good to go although AdvancedScriptControlTests failed.

Flake on testNavScriptingObject in com.tle.webtests.test.contribute.controls.asc.AdvancedScriptControlTests.
Merging.


Add multiline to syntax info

Add multiline info to the syntax names this allows to get more useful information from
synIDattr(synID(line("."), col("."), 0), "name")

I need this info for a better version of gq that I'm working on in this pull request: https://github.com/hynek/vim-python-pep8-indent/pull/29

Do you want me to fix the indentation again?


FALCON-1976 and FALCON-1977:  Remove hadoop-2 profile and move unit t…

Removed hadoop-2 profile and moved falcon-unit to addons

@vrangan I have tried to build the Falcon with the patch in pull request. When I use "mvn clean install" build fails in examples module. It looks like that testng dependency is required explicitly now.
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.16:test (default-test) on project falcon-examples: groups/excludedGroups require TestNG or JUnit48+ on project test classpath -> [Help 1]
But when I use "mvn clean install -Ptest-patch", build succeeds . Earlier also it succeeded by using hadoop-2 profile.

@peeyushb   I got that error without test-patch earlier also.  Let me see if there is a way to to fix it or is it specific to maven versions as removing the profile did not change the dependencies being managed

Failures are unrelated to this change.   Passes successfully locally.

+1 build works for me, will commit if there are no further comments.

[INFO] --- maven-assembly-plugin:2.6:assembly (default-cli) @ falcon-main ---
[INFO] Reading assembly descriptor: src/main/assemblies/assembly-src.xml
[INFO] Copying files to /Users/bvellanki/SourceCode/apache-falcon/falcon/target/falcon-0.10-SNAPSHOT-notes
[WARNING] Assembly file: /Users/bvellanki/SourceCode/apache-falcon/falcon/target/falcon-0.10-SNAPSHOT-notes is not a regular file (it may be a directory). It cannot be attached to the project build for installation or deployment.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Apache Falcon ...................................... SUCCESS [02:15 min]
[INFO] Apache Falcon UI ................................... SUCCESS [  8.062 s]
[INFO] Build Tools ........................................ SUCCESS [  4.759 s]
[INFO] Apache Falcon Java client .......................... SUCCESS [ 31.087 s]
[INFO] Apache Falcon Metrics .............................. SUCCESS [ 20.956 s]
[INFO] Apache Falcon Hadoop Dependencies .................. SUCCESS [ 14.534 s]
[INFO] Apache Falcon Test Utility ......................... SUCCESS [  8.538 s]
[INFO] Apache Falcon Commons .............................. SUCCESS [02:09 min]
[INFO] Apache Falcon CLI client ........................... SUCCESS [ 12.894 s]
[INFO] Apache Falcon Oozie EL Extension ................... SUCCESS [ 11.549 s]
[INFO] Apache Falcon Embedded Hadoop - Test Cluster ....... SUCCESS [ 23.295 s]
[INFO] Apache Falcon Sharelib Hive - Test Cluster ......... SUCCESS [  2.058 s]
[INFO] Apache Falcon Sharelib Pig - Test Cluster .......... SUCCESS [  0.993 s]
[INFO] Apache Falcon Sharelib Hcatalog - Test Cluster ..... SUCCESS [  0.777 s]
[INFO] Apache Falcon Sharelib Oozie - Test Cluster ........ SUCCESS [  0.368 s]
[INFO] Apache Falcon Test Tools - Test Cluster ............ SUCCESS [  0.341 s]
[INFO] Apache Falcon Messaging ............................ SUCCESS [ 27.373 s]
[INFO] Apache Falcon extensions ........................... SUCCESS [ 21.795 s]
[INFO] Apache Falcon LIfecycle Module ..................... SUCCESS [ 24.443 s]
[INFO] Apache Falcon Oozie Adaptor ........................ SUCCESS [01:09 min]
[INFO] Apache Falcon Scheduler ............................ SUCCESS [01:23 min]
[INFO] Apache Falcon Acquisition .......................... SUCCESS [  0.984 s]
[INFO] Apache Falcon Distcp Replication ................... SUCCESS [ 16.677 s]
[INFO] Apache Falcon Retention ............................ SUCCESS [ 16.594 s]
[INFO] Apache Falcon Archival ............................. SUCCESS [  0.950 s]
[INFO] Apache Falcon Rerun ................................ SUCCESS [ 23.557 s]
[INFO] Apache Falcon Prism ................................ SUCCESS [02:04 min]
[INFO] Apache Falcon Unit ................................. SUCCESS [02:44 min]
[INFO] Apache Falcon Examples ............................. SUCCESS [  1.629 s]
[INFO] Apache Falcon Hive Replication ..................... SUCCESS [01:10 min]
[INFO] Apache Falcon HDFS Snapshot Replication ............ SUCCESS [02:11 min]
[INFO] Apache Falcon Web Application ...................... SUCCESS [18:32 min]
[INFO] Apache Falcon Documentation ........................ SUCCESS [  8.045 s]
[INFO] Apache Falcon Distro ............................... SUCCESS [ 45.134 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 39:18 min
[INFO] Finished at: 2016-05-23T13:14:42-07:00
[INFO] Final Memory: 313M/798M
[INFO] ------------------------------------------------------------------------
Shutting down hive metastore.
[INFO] Shutdown hook executing
[INFO] Shutdown hook complete
bvellanki@HW11014:~/SourceCode/apache-falcon/falcon (master)$

Command used to build after applying patch is
"mvn clean verify assembly:assembly -Ptest-patch"

Build succeeded. +1.


[WIP] Fix build after eslint updates

[WIP] Fix build after eslint updates

Stack created with Sapling. Best reviewed with ReviewStack.

#983
#982
#981
#980
-> #979
#978
#977
#976
#975
#974

Unintended duplicate push; vscode extension not ready yet


Model::Load should not emit duplicate child element warning for  elements

<plugin> elements should be ignored when validating a model for name uniqueness, but Model::Load emits a warning if it encounters multiple <plugin> elements with the same name attribute.
Related: ignitionrobotics/ign-gazebo#164

In my opinion, ign-gazebo is misusing the //plugin/@name attribute by storing a non-unique class name there. I think it would be cleaner to use a custom attribute for the class name.
In general I think the parser shouldn't bother about things like //plugin/@name, but this attribute is in the spec saying that it must be unique. So if we're going to change the parser to eliminate this warning, then I think we would also need to change the spec.
Another option is for ign-gazebo to start using a custom element for plugins, like <ign-gazebo:system> or something.

Fixed by #721


Split update

Split Update
This is a small update to Split. It brushes up Cargo for the recent cargo shuttle update, adds in the warden's locker, cleans up some wires, and changes atmos tech spawn from [1, 2] -> [2, 3].

Fixes #9084

Tossing in another set of fixes for issues that are impacting rounds negatively that don't require extensive work.
-Funny substation by atmos fixed.
-CE now has RCD
-PA has HV Wire under it roundstart. Y'know back in MY day earlier this year, Packed never did this, we had to lay our HV wiring ourselves...


Im 217

This...Is...Sparta!

I get this error for product "100-105": SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data. For those that work, the location history works fine. IM-219 tested, the dates look fine.

Thanks, updated for when there's no locations currently.


Apify.call() is missing an option to set up memory

I would like to add this, it seems straightforward. It is in the client.
The thing is if we want to somehow control the memory to be 2 to the power of something. Right now in API it is not controlled at all, you can run the act with memory 1234.

We let use enter any value in SDK/client. If the memory won't be a power of 2 then server throws an error which appears in actor log. I think that this is better behavior than having some validation in SDK, client and server.

Fixed in new version that will be shortly released as BETA


Issues with file transfer during CLEM workflow

Post-transfer file name of XLIF files generated incorrectly

Path to file incorrect; /data/ is missing from the file path.

Logic for CLEM context not working correctly (murfey.client.contexts.clem).

Number of files present in series does not match that deduced from XLIF file.

"Failed to transfer file" message occurs even when file is successfully transferred (murfey.client.tui.app).

Related to presence of spaces in file path; conditions for handling file strings need to be tightened.

Analyser.post_transfer just calls the post_transfer method on whatever the Context has been determined to be


Update astropy-helpers to v2.0.8

This is an automated update of the astropy-helpers submodule to v2.0.8. This includes both the update of the astropy-helpers sub-module, and the ah_bootstrap.py file, if needed.
A full list of changes can be found in the changelog.
This is intended to be helpful, but if you would prefer to manage these updates yourself, or if you notice any issues with this automated update, please let @bsipocz or @astrofrog know!
Similarly to the core package, the v3.0+ releases of astropy-helpers require Python 3.5+. We will open automated update PRs with astropy-helpers v3.1.x only for packages that specifically opt in for it when they start supporting Python 3.5+ only. Please let @bsipocz or @astrofrog know or add your package to the list in https://github.com/astropy/astropy-procedures/blob/master/update-affiliated/helpers_3.py

Coverage remained the same at 29.124% when pulling 7bbb1c8f8eb4bbcdccc4ad79f588f900b4e8801a on bsipocz:update-helpers-v2.0.8 into d2217f5694c8518a03931a49a5bb00bd467f9a06 on spacetelescope:master.


WCAG 1.4.3: (ExpenseItDemo/ Table labels "Expense Type", "Description", "Amount" on data grid in 'Create Expense Report' view) [High Contrast White] Contrast ratio failure for regular text

The following accessibility issue needs investigation.
App: ExpenseItDemo
Element path: Table labels "Expense Type", "Description", "Amount" on data grid in 'Create Expense Report' view
Issue Details: Contrast ratio failure for regular text. Details:
Font Size = 10
Font Weight=bold
How To Fix: https://www.w3.org/TR/UNDERSTANDING-WCAG20/visual-audio-contrast-contrast.html
This accessibility issue was found using Accessibility Insights for Windows, a tool that helps debug and find accessibility issues earlier. Get more information and download this tool at https://aka.ms/AccessibilityInsights.


Fix/checkout row

Hi again István!
I've been working into the fixes you mention plus some others I encountered along the way.
The product page worked as intended however, the in the /cart, the "+" and "-" buttons where messing things up. They always add and concatenating amounts instead of overriding. For example, if I have 2 elements of X book, when I pressed + it add 1 in our page (so 3) but 5 (2+3) in Shopify's checkout. The same happened for the "-" because it doesn't remove elements unless you send a negative number.
This is done and fix here:
      const quantityToAdd = item ? parsedQuantity - item.quantity : parsedQuantity;

      const res = await client.checkout.addLineItems(checkoutID, {
        variantId,
        quantity: quantityToAdd,
      });

The "jumps" you mentioned where caused because the cart changed the order of the elements once updated (likely because of some code from the guide). I fixed it by sorting the elements alphabetically:
       cart.sort((a, b) => a.product.title.localeCompare(b.product.title)).map((item, index) => (
            <ProductRow item={item} key={item.product.title} />
          ))

Probably this overkilling and we may get rid of it if we debug carefully the update cart code. For now it works.

P.S: please don't buy more beers or gifts. It's ok for me, specially if the yoga classes are low in holidays. Consider it as a holiday gift for you 😄 . We can further discuss terms once the shop is released or exchange some vouchers for free yoga classes 🤣
Namaste!

Forgot to mention: to test it, remove the local storage from your browser to avoid getting wrong data and to do a fresh start (or try an incognite view):

Select the cart and delete


Bihaox inferschema

also I got this error upon testing:
java.lang.RuntimeException: attribute date already exists in the Schema: [text, date]
	at edu.uci.ics.texera.workflow.common.tuple.schema.Schema$Builder.checkAttributeNotExists(Schema.java:383)
	at edu.uci.ics.texera.workflow.common.tuple.schema.Schema$Builder.add(Schema.java:218)
	at edu.uci.ics.texera.workflow.common.tuple.schema.Schema$Builder.add(Schema.java:237)
	at edu.uci.ics.texera.workflow.operators.typecasting.TypeCastingOpDesc.getOutputSchema(TypeCastingOpDesc.java:70)
	at edu.uci.ics.texera.workflow.common.workflow.WorkflowCompiler.$anonfun$propagateWorkflowSchema$7(WorkflowCompiler.scala:155)```

I rewrite the inferToken so I think it is more readable

@bihaox Thanks! Can you please rebase this branch on top of master and I will review again. Note that the LocalCsvFileScanOp is now renamed to CSVScanSourceOp.

@Yicong-Huang It seems like the CSVScanSourceOpDesc code is changed to Scala. I will try to convert them.

@Yicong-Huang It seems like the CSVScanSourceOpDesc code is changed to Scala. I will try to convert them.

Ok. please let me know if you need help on converting to Scala, or anything related to rebase/merge.

This PR
Infers the schema (columns' AttributeType) when reading from a CSV file by reading first N(default = 100) records
AttributeType
By default, when we read from the csv. file, all data types are AttributeType.String.
There will be a a Ranking in current AttributeType
We before had 4 types:
Integer, Double, Boolean, Timestamp
And we added 2 types:
Long, Timestamp
Logic:
Infer Order: Integer->Long->Double->Boolean->String/Timestamp:
we will try to parse the data by Infer Order
For example:
we will first try to do parse the data to the Integer, if it is success, we will assume the data is Integer
if it throw out exception, it will continue to try to parse the data to the Long....
Priority of the attributeType: String/Timestamp->Boolean->Double->Long->Integer
Then, when we infer several rows data, we will try to merge them by Priority of the attributeType
For example:
[String, Integer, Double, Integer]
[Integer, Integer, Double, Long]
[Long, Integer, Long, Long]
will become
[String,Integer,Double,Long]
Currently, there is no way to directly show column's type
To view current column's type
We can use Type Casting operator to view column's type
Just connecting them like this

We can see input column's Attribute Type on the right panel.

Can you update this description, the types are not correctly listed.

I changed order.


Configurable basepath/url

Description:
adds prefix to url in python and react
Steps to Check:

configuring prefix in python
in init.py's PrefixMiddleware class update prefix
EX.  init(self, app, prefix = '/demo'):

configuring prefix in webpack config
update URL_PREFIX in webpack.config.js
EX. URL_PREFIX = '/demo';

This has been fixed by alternate implementation #207 & #208


Loss of the discussions tab and its content

The Discussions tab was previously active on on this repo and contained some useful resources, tips, and tricks for working with egg. It appears to have disappeared and those resources along with it, although some of them are still showing up in google results for now (and 404ing).
Perhaps it was taken down for good reason, but I'd encourage those discussions to be restored if possible - even if you point people to the zulip for new discussions! Or alternatively, perhaps they could be archived somewhere.

Okay! I've restored them but pinned a message to use zulip instead.


Merged the latest project-template

Fixed TravisCI configuration
Fixed PHPUnit runnings
Minor fixes for the README

:+1:


CI/CD workflow to publish helm chart

Many thanks for submitting your Pull Request :heart:!
What this PR does / why we need it: CI/CD flow to publish helm chart
Special notes for reviewers: need another step to publish into https://artifacthub.io/
Additional information (if needed):

Hi @robinson, thanks a lot for your PR !
But as you can see, it doesn't pass the DCO requirements. Could you please sign/verify and sign off your commits ?

Btw, should this PR be merged in main or in gh-pages ?

Btw, should this PR be merged in main or in gh-pages ?

the gh-pages branch use to create a pages in order to publish the artifacts. The helm release workflow should be run in main branch.


Research - Hosting

Describe the solution you'd like

A clear way of seeing the different hosting options available

Describe alternatives you've considered

AWS
Heroku

Additional context

A list of different hosting options for different types of applications
What would the cost be of each of the options
What's the scaling like with these options (1 user / 10 users / 100 users / 1000 users etc)
Downtime/Availability?
Serverless?
Bandwidth?
Integration/Deployments?

Still worth keeping this for now. Chances are Azure / AWS - with the possibility of using Serverless / Functions.


New version: DavidMoore.IPFilterUpdater version 3.0.5-beta

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: Open in CodeFlow

Hi @vedantmgoyal2009,
This manifest adds Apps and Features entries that aren't present in previous PR versions. Should these entries be added to the previous versions also?
(Automated response - build 485.) Hi @vedantmgoyal2009,
This manifest adds Apps and Features entries that aren't present in previous PR versions. Should these entries be added to the previous versions also?


Misalignment on member invitation modal

Package version (if known): v9, v10
Describe the bug
The invitation modal has various alignment issues (identified with red rulers below):

I guess the fact that I can choose myself at all is a tiny bug. Also my preferences have my email hidden, so is it normal that I can see my email there?


Main Scene Camera does not run correctly

The main scene has two game objects with camera in it. One of them is disabled, but neither is tagged as main camera. When activating play mode, the example script throws an error in line 26, when trying to access the main camera.
There are also two other leftover game objects in the scene, one with a blank name and one called Missing Prefab. Some of them have missing script components on them.
My suggestion would be to remove the obsolete objects from the scene, tag the main camera correctly and remove the second one.
Also, since the sound is played as a pure 2D sound, the camera doesn't need to be moved in the example script at all, so the line could probably be removed completely.

Main Scene is cleaned now.

Main Scene is cleaned now.


How to include more dataset ?

I am currently working and have got results on Sysu Dataset(through some other algorithm). What changes do I make in the code so as to run this on Sysu. I have made Sysu.py just like dukemtmc.py and placed the dataset at the same place as other dataset. I have also included it in init.py (__factory) of the datasets folder. Whenever I try running the sh scripts/pretrain.sh Sysu market1501 resnet50 1 it show the error that only defaults datasets "dukemtmc", "msmt", market1501 can be used.

Did you run python setup.py —install after changing the code?

Did you run python setup.py install after changing the code?

Ohh. Thank you. I forgot to run it again. So any changes I make in the original code I have to do that python setup.py install. Got it :-)


hdodenhof CircleImageView doesn't work for  19 Api !!

If you want help, you need to provide more details about your issue. Feel free to reopen with at least a description and preferably some code snippets and/or stack traces.


[FEATURE] Write API Wrapper Function for Getting Concept Set Values

Using the ATLAS API, getting the value for a concept set would be crucial going forward. To do this, we need to build a wrapper function around the conceptset GET string. To do this, I imagine doing the following steps:

Review: https://github.com/JuliaHealth/OHDSIAPI.jl/blob/2c92bfbdde31a14ac12604ccfa4a048c605638de/src/OHDSIAPI.jl#L33 to see how building a wrapper function would work
Go to: http://webapidoc.ohdsi.org/
Navigate in the API documentation to conceptset > id > expression > /
Test an example within your browser (executing this should give a nice example: https://atlas-demo.ohdsi.org/WebAPI/conceptset/1874094/expression)
Write wrapper function around this result
Write small test suite around function; see here for an example: https://github.com/JuliaHealth/OHDSIAPI.jl/blob/main/test/getters.jl

And that should do it!

Closed via #5 -- thanks @Farreeda !


Uncaught DOMException: Failed to read a named property 'addEventListener' from 'Window'

Hi,
I'm getting this error when trying to use react-drawio in an actual application.
Uncaught DOMException: Failed to read a named property 'addEventListener' from 'Window': Blocked a frame with origin "https://my-application-domain" from accessing a cross-origin frame.
Any idea how this can be fixed? Is there somewhere that is using trying to add a direct listener as opposed to postMessage?
Thanks.

I believe this was coming from elsewhere


Arguments passed to smarttransformer constructor now pass to _get_tra…

Fixes issue #9
See issue thread for additional comments. Fully resolved with minor changes.

Pull Request Test Coverage Report for Build 98

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 89:
0.0%

Covered Lines:
1004

Relevant Lines:
1004

💛  - Coveralls


Spotify Autoplaylist

So is there a way for the autoplaylist to read the URI from my spotify playlist? So that i can easily manage the auto played songs by adding/removing in the Spotify library?
I have added Spotify and i can use uri's for request's, but it wont read in the auto file...
Help is much appreciated!

It is not currently possible to add Spotify URIs to the autoplaylist. They are only supported through playing them with a command.

I'd love to be able to save the queue made from a spotify playlist, or have the same spotify -> youtube conversion for pldump

Duplicate of #1587


Create target repository within Azure DevOps

The target repository for Project  is created within Azure DevOps

Created target with GitHub repository in Azure Devops


In the discord.js Command there is ESLint Error

In the discord.js Command, there is ESLint Error that causes thee, Travis.CI to say that the build is failing. This is a horrible choice to have the build fail for one command. Please fix this.

fixed 😎


Allow setting custom fields on embedded requests

For #96

@jspaetzel Any update on this? I'd like to get back on a standard release rather than my patched version. Thanks!

Hey @codylerum sorry for the wait! I wanted to spend a little more time looking at this. I've opened a PR that's kinda similar to this one if you want to take a look. https://github.com/hellosign/hellosign-java-sdk/pull/100/files?utf8=✓&diff=split&w=1
This should add support for custom_fields in embedded requests, signature requests, and unclaimed drafts which all support that field.
I'm going to spend a little longer on tests but feel free to take a test drive and let me know if you have any feedback.


Work in Progress?

This seems to be WIP for last year. Is there some deadline on when it will be ready for production use?

Sorry for the radio silence on this. The development got stalled but we have concrete plans to reboot the development process on this in October. Still, it will be at least a few more months until the client will be in a usable state.

Hello, the new client has been made publicly available today — sorry for the long silence, again!


== null should be forgiven when surrounded by extra parens

As discussed in issue #13 , x == null is a useful construct that evaluates to true only if x is undefined or null. The only way to avoid a warning currently is to enable boss:true.
It would be useful if you could also suppress this warning by surrounding the comparison with an additional pair of parenthesis. This would match another similar case, described in the documentation for boss:

You can silence this error on a per-use basis by surrounding the assignment with parenthesis, such as:
for (var i = 0, person; (person = people[i]); i++) {}

I think == null is now only controlled by the eqnull option.
Try this out on the website.
/* jshint eqeqeq: true, eqnull: true, boss: false */

var a = 1 == null;

for(var b = 1; b == null; b++) {

}

Since eqnull only controls this very specific point, is it necessary to add a specific type of override? I would vote it is fine as it is.

Well, then the same question could be raised about boss for assignment as expression, right ?
Personally I like being able to silent those cases by using extra parens because it's explicit in the code and somewhat points out that the author knows what he's doing (even if he have a ~/.jshintrc with eqnull).
When working on a collaborative project, it's then easier to spot responsible usage vs potential mistake.
My 2 cents.

The "supress-via-parens" pattern is incompatible with the singleGroups option. If we decide that eqnull is somehow not sufficient, we shouldn't consider unnecessary parenthesis an acceptable solution because it's not valid in singleGroups: true contexts.

Well, then the same question could be raised about boss for assignment as expression, right ?

I would think eqnull is more likely to want a consistent approach, where as a for loop in a specific scenario might need an override more often.
What would code look like if every rule was ignored via parens and you used it? It would look pretty bad.  Also what about false positives - would we require 2 parens in case someone was already using parens for aesthetic or ordering reasons.

I'm happy with the eqnull option which I didn't realise existed, so don't keep this open on my account.


section "fishManagement" text content

Nearly 100 lakes in Wisconsin are predicted to support naturally reproducing walleye populations even under extreme warming conditions.

These  lakes are resilient to climate change, and should be protected from other stressors such as habitat loss, invasive species, or overfishing to maximize the potential for continued walleye production.

Some lakes that are unlikely to  support natural walleye reproduction will continue to be suitable for adult walleye, and fishable walleye populations could potentially be maintained in these lakes via stocking.

Quality fishing opportunities for largemouth bass and other warm water species will be possible in lakes that become unsuitable for walleye.

Actually, the picture of Jon with both fish should be here, I think.

And maybe the Mille Lacs background picture should be on this one too. I was confusing the two sections.

OK, just saw these comments, so you can ignore my questions on the other issue about background pictures

Pic of jon w/ fish and pic of Ada w/ bass in this section? might make sense since the "coexist" and "bass dominated" are two of the bright spots coming out of this section.

Sounds fine, then I guess the new Mille Lacs pic in the other section? Maybe it is best to just work it up however you think it works best, then comment and change as needed.

Have added the text, but we should deal with the organization of pictures and backgrounds as a separate issue


How to use local docker image?

Hi.
When I run my first agent, it is very slow pulling images automatically by the program. Then I downloaded the docker image openai/universe.flashgames with latest version  from docker hub by myself.  However it still pull the docker image when I run the program. And I want to know is there any solution to use the local docker image.

Hi, @paive, I have the same situation with you, slow pulling image of flash games, unfortunately Docker does not support proxies, so may I ask how do you finally make it?

Hi, have you solved this problem? If not, you can refer next steps.
First, download the universe flash games images from docker hub,  you can use 'docker images' in terminal to view the repository and tag. Maybe the repository is 'openai/universe.flashgames' and tag is 'latest' or '0.20.21' or any others.
Second, make sure the version of container that would be created by universe, just run the example of universe and you can find one line of output is 'Creating container: image=quay.io/openai/universe.flashgames:0.20.21', maybe the tag is others.
Third, modify the repository and tag according to second step, for example is use 'docker tag openai/universe.flashgames:latest  quay.io/openai/universe.flashgames:0.20.21'.
Anyway, it is essentially rename the image and its tag.
If it does not help you solve the problem, welcome for further discussion.

发件人: Bingo Du<EMAIL_ADDRESS>发送时间: 2017年3月21日 11:35:21
收件人: openai/universe
抄送: paive; Mention
主题: Re: [openai/universe] How to use local docker image? (#157)
Hi, @paivehttps://github.com/paive, I have the same situation with you, slow pulling image of flash games, unfortunately Docker does not support proxies, so may I ask how do you finally make it?
―
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/openai/universe/issues/157#issuecomment-288052089, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AII_iAVH6qKJVP4E2UBLyH8zXFz37sWUks5rn7X5gaJpZM4MOEZV.


Warn if expired token is used to log in

Looks like resin-cli successfully logs in with an expired token but then none of the commands actually work: https://github.com/resin-io/resin-cli/issues/447 Should resin login time expired tokens be rejected, or warned about?
 Front conversations

Great point, if it's the case indeed the warning must be issued and such token must be dropped.

The key changes are here https://github.com/resin-io-modules/resin-token/issues/29

Released in 5.8.0


Check for profile picture override when viewing avatar.

When a user has VRCPlus and sets a Profile Picture Override the current avatar thumbnail URL will be overwritten to VRChat's default Robot avatar, it will now return "Avatar is unreachable!" in this case.

LGTM! Thanks for the contribution.


Implement local PDP for Vapor integration

Work in progress

@mkasztelnik This functionality is meant to be utilized by https://github.com/VPH-Share/atmosphere-vph/pull/19 which represents "the other half" of this PR.

What about new created ATs, where is the place appropriate rights are added (UserApplianceType is created)?

Is it ready for second round?

@mkasztelnik It is now!

Some small remarks/questions if tested locally I think we should push it into prod and improve when needed.


Clean up atom constructor

Here's a take on cleaning up the structure and return typings in atom.ts:

extract inner basicAtom function
extract inner derivedAtom function
extract inner customAtom function
remove atomConstructor
collapse atom into simply returning basicAtom, derivedAtom or customAtom depending on input
add file to test return types of atom (this "passes" both on master and on this branch)

No behaviour change, this is all about tidying up types and making the code easier to reason about.

@merisbahti I also see this as a preparatory step for the dev tools support; by having clearer atom construction code it will be easier to add eventual new parameters ("store" id, atom description, etc) needed for dev tools.


TS-4627: support TSRemapOSResponse in ts_lua

@jpeach pls take a look. Thanks.
I want to add enhancement to ts_lua so I can write script to take advantage of the TSRemapOSResponse() related functionality.
Basically it would allow me to do something when origin server fails to connect (such as DNS error, timeout, etc)

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/398/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/294/ for details.

@jpeach , I have done the cleanup according to the comments. please take a look. thanks.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/436/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/329/ for details.

@jpeach cleanup is done accordingly. pls review.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/446/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/447/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/340/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/341/ for details.

Loods good 👍


Intent Guide

1/ Intent guides not highlight when I focus in selection scope.
=> Please change color of intent guide when focus some where.
Ex: Change color instant of "IF statement" when I focus beside of IF.
2/ When I open big html file, when I try to collapse or uncollapse each tag element, VS Code will be "lag", crashed or Not response I can't do anything next :(

The first issue is dup of https://github.com/Microsoft/vscode/issues/9069
About the second one, we already create tasks for that https://github.com/Microsoft/vscode/issues/3725


Update FreeBSD dependencies

how is jq a hard dep?

$ meson setup hyprland_build hyprland
The Meson build system
Version: 1.1.0
Source dir: /tmp/hyprland
Build dir: /tmp/hyprland_build
Build type: native build

hyprland/meson.build:2:12: ERROR: Program or command 'jq' not found or not executable

A full log can be found at /tmp/hyprland_build/meson-logs/meson-log.txt

huh. Meson. Maybe @fufexan did something. Fair.

huh. Meson. Maybe @fufexan did something. Fair.

Yeah, remember when we implemented version reading from props.json?

right.

Yeah, remember when we implemented version reading from props.json?

Sounds incredibly painful, why make it into json instead of just a version file containing a single line with the version?

Yeah, remember when we implemented version reading from props.json?

Sounds incredibly painful, why make it into json instead of just a version file containing a single line with the version?

I don't remember the real reason, but I think we wanted to be future proof in case another variable needed to be added.

Has this turned out to be a problem in practice? What other information might need to be read into the build system itself?

Has this turned out to be a problem in practice? What other information might need to be read into the build system itself?

No. We just over engineered it seems. 🤷♂


Suggestions for new features

Hello,
being new to this hobby I find Sondehub to be incredibly useful. But there are a couple of issues and features missing for the site to be perfect.

Adding a way to record sonde location manually. Ideally by putting a marker into the map and filling the info.
Big portion of the sondes are landing into areas where there is no network coverage.  For example the sonde V5050436 I was searching for today - the landing zone is about 5 kilometers East of last recorded frame (more frames received later by TTGO). I wanted to mark the spot for others for future attempts of recovering but had to walk about a kilometer away to get to signal and in the end the position was marked on the spot of last received frame. Car location was way off too.
Search button missing in mobile version of the website. There is no way of searching for sondes older than 12 hours on mobile. Filling the ID to the address link is hit or miss, sometimes it works, sometimes it doesn't open anything. Quite crippling while on foot in field.
Missing sonde information in mobile version of the website. You have to open Plots to search for sonde frequency. At least the transmitting frequency value should be visible right when the sonde information is opened.
Adding button for mobile version of the website to change to desktop version. It is very painful to use the full desktop version on phones but sometimes you need to access the full version to get the info previously mentioned as you cannot access it on mobile version.
Adding button for location refresh. Think of a button like Google Maps have. There should be a similiar one to refresh your position on a map. Sometimes I use the Sondehub in field to locate myself and measure how far I am from the recorded dropzone. Reloading website works in about 40% of cases, other times it either puts me on last location or removes my location alltogether. This also makes troubles with recovery announcing as it can throw you WAY off your real location.

I'm not a programmer so please take these notes as ideas for future development. Thank you

A few notes:

Sonde recovery - we are working towards a separate recovery reporting website, which would have the ability to 'adjust' the last reported position to where you know it actually landed.
Mobile limited data issues in general - This is very much a tradeoff in screen real estate. I think almost everything you are after is available by just rotating your phone into landscape mode.
Location Service issues - Enable chase car mode. This enables higher precision and more frequent updates of your position.. and also lets others know that you are out chasing.

Here's some screenshots from my phone:
Portrait mode:

Landscape mode (search box, frequency, etc all now visible):

Browser settings showing 'Desktop Site' option... though I don't think it's necessary:

Hello, another solution, use a tablet, mobile or portable it is much better with a 10" screen or more

Hello,
thanks for suggestions. Yeah, the landscape mode is something I didn't try because I never use landscape mode for anything. That is a good point.
The desktop mode is not working on iPhones, only on Android. I don't use the web on Android because it is kinda poorly optimised. While iPhone is refreshing butter smooth on 60FPS, Android does it in single digits and lags very badly. Both phones are comparable quality and performance-wise. You can compare it yourselves by number of received and processed messages.
Android:

And it feels very choppy, it lags a lot and moves poorly
iOS:

Smooth like butter

What phone is the android phone? Pixel 8a renders ok for me. There's probably not many more performance improvements we can do as we are somewhat bound by leaflets performance - its just the shear number of elements we  are trying to update on the map becomes a problem.
Not sure why desktop mode wouldn't be working on ios. I think we've removed most of our ios specific code. I don't have an iphone to test with however so will probably need someone else to look into.
You shouldn't need to refresh the page to get updated location if chase is enabled - however your browser might not be sending us the latest position for some reason

Separated out the issues we can work on into seperate issues so we can track them individually


e2e Test Startup is Flaky

Sometimes in the e2e test some container do not start.

As can be seen here


VK_EXT_metal_surface extension is not reported correctly

When calling vkCreateMacOSSurfaceMVK() I get a message that it's obsolete and I should be using vkCreateMetalSurfaceEXT() from VK_EXT_metal_surface. However that extension is not reported as supported via vkEnumerateInstanceExtensionProperties.
Looking at the output from vulkaninfo, I see that the extension is reported but incorrectly as a device extension. There are also some other instance extensions like VK_KHR_surface as well as VK_MVK_macos_surface that are incorrectly enumerated as device extensions.
I tested with the SDK 1.1.114 loader and vulkaninfo, and with the MoltenVK both from that SDK as well as built from latest source.

The loader does not yet support that extension.

MoltenVK does not correctly discriminate between the lists returned by the two calls vkEnumerateInstanceExtensionProperties() and vkEnumerateDeviceExtensionProperties(). It should.

@baldurk The reporting of the instance and device extensions is now fixed in PR #723 . Please retest and close this issue if it works.
As @cdavis5e indicates, the reporting of VK_EXT_metal_surface works for MoltenVK, but is not supported by the loader yet.

I can test that the device extension list doesn't include any instance extensions and vice-versa, but it sounds like the core issue that I reported is still a problem if I can't use VK_EXT_metal_surface.
Maybe the message about deprecation should be removed until VK_EXT_metal_surface is usable?

it sounds like the core issue that I reported is still a problem if I can't use VK_EXT_metal_surface.

That core issue needs to be fixed in the layers implementation in the SDK. If you have not done so yet, can you post an issue there, please?

Maybe the message about deprecation should be removed until VK_EXT_metal_surface is usable?

VK_EXT_metal_surface is usable within MoltenVK...which is where the deprecation message is originating. Although we encourage the use of the SDK, a number of developers use MoltenVK by itself.

It's not an SDK wide issue so I looked instead on the Vulkan-Loader repository, and it seems like there's already an issue for this - https://github.com/KhronosGroup/Vulkan-Loader/issues/207

VK_EXT_metal_surface is usable within MoltenVK...which is where the deprecation message is originating. Although we encourage the use of the SDK, a number of developers use MoltenVK by itself.

I understand that MoltenVK can be used directly without the loader, but what I mean is that it seems premature to deprecate the MVK surface extensions when for many people the new extension they have been deprecated in favour of is not available.
As you say that is the encouraged path use the loader, so hopefully most users are doing that. Having a deprecation notice that can't be fixed when using the encouraged path is kind of confusing. It doesn't do any harm to users calling MoltenVK directly to not get a deprecation notice yet.

it seems premature to deprecate the MVK surface extensions when for many people the new extension they have been deprecated in favour of is not available.

The offending log message has been removed.
The VK_EXT_metal_surface is being added to the loader for the next SDK release.
Closing this now.


menu problems

Howdy! I love the idea of the enchantments and really look foward to using them but in the 'menus' you have everything just seems to appear as a stone block. Is there a fix to this? I thought it was my resource pack at first but the problem persisted through the default texture pack

This has to do with the old 1.12 MC IDs being in the configs. But when updated to the new names, the gui will let players take the items out. So we need a 1.13.1 update


Invalid content /en/testnets/erc20-converter/erc20-using-converter/erc20-address-setup/

Where does the issue occur?
Link to the file or location on the website where the issue occurs. Optionally add screenshots for context.
Describe the issue
A clear and concise description of what the issue is.
Expected changes
A clear and concise description of what you expected to change.
Additional context
Add any other context about the problem here.
"Daedalus" is typed in ERROR in #4 where MetaMask should be typed. This is the section on how to Connect Metamask wallet

AGIX ERC20 converter is now live on mainnet. Please see updated tutorials here: https://bridge.singularitynet.io/


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2803

Questions ?
Please refer to the Backport tool documentation


pythonpath modification

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Closes #1343
What are the significant changes in functionality due to this change request?
Removed update of the pythonpath if not needed

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Job Test CentOS 7 on 713b7c8 : invalidated by @alfoa

Job Test qsubs on 713b7c8 : invalidated by @alfoa

Job Test Fedora 31 on 713b7c8 : invalidated by @joshua-cogliati-inl
failed tests/framework/CodeInterfaceTests/genericInterfaceParallel

Some more explanations or work needs to be done before issue #1343 can be closed.


emit nodejs package

@aep Do you have a general summary about this PR? It looks interesting

@jwerle hopefully this is a start. feel free to open a bug if you have questions
https://github.com/aep/zz/wiki/using-zz-in-other-ecosystems

@aep Looks like a great start. Thanks!


Receiver chat list does not get chat if the receiver hasn't opened the chat, messagesArray comes empty

Hi, Are the users not being linked in the get chat list service?

@kiritoxcii just after login, sdk will download only latest message for each contact. Historical messages will be downloaded once tapped on contacts.
Can you please let me know more detail, what you are trying to achieve and what code block you are using?


Don't allow clicking "Upload" if no file is selected

Select a file to upload
Reload page
File gets un-selected, but can still try to create an upload. (Results to an error, but still creates an entry in uploads)

Livestream Form also suffers from a similar problem, where users can submit without picking a replay or selecting a form.
Let's do this after the Unlisted PR to avoid conflicts.


Reduce output bundle size

Currently the output from webpack for the client side mapping code is quite large (2-3MB on production mode). This is expected as its a large application, but this generates warning in webpack and could be better optimised by either reducing the bundle size further, or by using splitting behaviour.
At the time this was a low priority and a single 2-3MB js file was considered good enough, but this could be improved.
The warnings are turned off in the webpack.prod.js.

Using the source-map-explorer package produced this visualisation

There are some areas for improvement there


Fix KNeighborsTimeSeries object has no attribute d

I hope this fixes the issue I created #122

Hello @thusithathilina! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file tslearn/neighbors.py:

Line 69:80: E501 line too long (89 > 79 characters)

Fix the line length in #124


Wagtail Compatibility: "Uncaught ReferenceError: pgettext is not defined at recurrence.js:1036"

I'm trying to get this working with the Wagtail CMS. I've got a basic Wagtail model - a Wagtail Page inherits from a Django models.Model:
class Event(Page):
    """
    Model for storing recurring events.
    """

    description = models.TextField()
    start = models.TimeField()
    end = models.TimeField()
    recurrences = recurrence.fields.RecurrenceField()

    content_panels = Page.content_panels + [
        FieldPanel('description'),
        FieldPanel('start'),
        FieldPanel('end'),
        FieldPanel('recurrences'),
    ]

    def __str__(self):
        return '{0}'.format(
            self.title,
        )

Many fields which "work" in the Django admin also work in the Wagtail admin. However, I'm getting the following JavaScript error:
recurrence.js:1036 Uncaught ReferenceError: pgettext is not defined at recurrence.js:1036

As you can see from the screenshot, it does partially instantiate:

Clicking "Add Rule" gives the following error:
recurrence-widget.js:1122 Uncaught TypeError: Cannot read property '0' of undefined
    at recurrence.widget.RuleYearlyForm.init_dom (recurrence-widget.js:1122)
    at recurrence.widget.RuleYearlyForm.init (recurrence-widget.js:1108)
    at recurrence.widget.RuleYearlyForm (recurrence-widget.js:1101)
    at recurrence.widget.RuleForm.<anonymous> (recurrence-widget.js:961)
    at recurrence.js:905
    at Object.foreach (recurrence.js:876)
    at recurrence.widget.RuleForm.init_dom (recurrence-widget.js:958)
    at recurrence.widget.RuleForm.init (recurrence-widget.js:732)
    at recurrence.widget.RuleForm (recurrence-widget.js:710)
    at recurrence.widget.Widget.add_rule_panel (recurrence-widget.js:533)

I realize this is not what django-recurrence was initially designed for, but it'd be great if we could get this working as well. I figured I'd start an issue in case anyone had experience getting this to work with Wagtail, or could point me in the right direction as I start digging. Thank you!

Thanks to some help from Torchbox, I've got this working. It can use some CSS tweaks, but the key was to include the jsil8n library:
import recurrence.fields
import datetime

from django.db import models

from wagtail.wagtailcore import hooks
from wagtail.wagtailcore.models import Page
from wagtail.wagtailadmin.edit_handlers import FieldPanel

class Event(Page):
    """
    Model for storing recurring events.
    """

    description = models.TextField()
    start = models.TimeField()
    end = models.TimeField()
    recurrences = recurrence.fields.RecurrenceField()

    content_panels = Page.content_panels + [
        FieldPanel('description'),
        FieldPanel('start'),
        FieldPanel('end'),
        FieldPanel('recurrences'),
    ]

    def __str__(self):
        return '{0}'.format(
            self.title,
        )

@hooks.register('insert_editor_js')
def editor_js():
    return '<script src="/admin/jsi18n/"></script>'

Hopefully this helps the next person looking!

@gabrielwolf
may I ask if you got it to work? I followed all steps in this thread including the django-admin url part but my console throws me this error:
TypeError: django.jQuery is not a function for the first line of the recurrence-widget.init.js. This seems to be very basic but I am noob. JQuery loads correctly though…

+1


Remove SDK-specific sub pages, redirect to GitHub

This removes the outdated sub-pages for SDK'S, and redirects the "Go to Docs" button directly to GitHub instead.
The sub-pages serves no purpose, because the contents is a clone of the SDK-specific Readme.

This PR breaks existing URLs, and negatively affects SEO, they should be redirected to their new content at least.

@ariesclark Thanks, old SDK sub-sites will now redirect as well.


File Load Error for .gitignore

Describe the bug
Sagemaker Studio Lab can't seem to load my gitignore file. The repository I am working on is https://github.com/pproe/spiral-galaxies. As you can see, .gitignore exists in the root directory, but when clicking Git > open .gitignore via the toolbar, Studio Lab presents the error "File Load Error for .gitignore file or directory '/spiral-galaxies/.gitignore' does not exist".
To Reproduce
Steps to reproduce the behavior:

Open a git repository in Studio Lab.
Click on 'Git' in toolbar menu.
Click on 'open .gitignore'.
See error

Expected behavior
Studio Lab should open up my .gitignore file.
Screenshots

Error displayed when attempting to open .gitignore

Proof .gitignore file does exist in directory

Desktop:

OS: Windows 11
Browser: Firefox
Version: 110.0.1 (64-bit)

Additional context
A workaround is simply accessing the .gitignore file via nano or vim in the terminal. However, this is not ideal as it goes against the core premise of Studio Lab being a beginner-friendly entry point for machine learning beginners.
Simplified workaround for others coming across this problem:

Click 'New Launcher' button (big button on top left or  press 'Ctrl + Shift + L')

Under 'Other' select the 'Terminal' option.

In your terminal type nano .gitignore.

Make changes to your gitignore file.

When finished editing, press 'Ctrl + X', then 'Y', then 'Enter'.

Dear @pproe , thank you for raising the issue and using the Studio Lab for your repository!
We confirmed that .gitignore can not open. We will investigate this problem.
As you said, we can edit .gitignore by nano. Or else, we can open it by double click if you show the hidden file.

Settings > Advanced Settings > File Browser > check the Show hidden files
Generate the setting file by jupyter server --generate-config
Edit generated home/studio-lab-user/.jupyter/jupyter_server_config.py.

Write c.ContentsManager.allow_hidden = True. We can search c.ContentsManager by F6 key in nano editor.

Restart the Jupyter Server

Reference: Jupyter Lab / Working with Files

Dear @pproe , we have now fixed this issue. Thank you for your contribution!


Cannot specify WebSocket extensions

The change introduced here breaks the possibility for client applications to specify WebSocket extensions.
A client application may need to specify a WebSocket extension in the same way it may need to specify a WebSocket protocol, for example:
Sec-WebSocket-Extensions: identity, fragment, custom
If OkHttp wants to add the permessage-deflate extension, it should merge it with the extensions provided by the client application.

I don’t think our API lenda itself to user-provided extensions. Negotiating which order to apply extensions in is awkward. Similarly it’s awkward to expose the message flags for read and write.
That said, I would like to know about the other extensions so I can understand the use case. Got a link to the CometD test?

https://github.com/cometd/cometd/blob/master/cometd-java/cometd-java-websocket/cometd-java-websocket-jetty-server/src/test/java/org/cometd/websocket/ExtensionNegotiationTest.java

https://github.com/cometd/cometd/blob/f3cf4d9ffe352082a57b038aa72a664ed6cdab58/cometd-java/cometd-java-websocket/cometd-java-websocket-jetty-server/src/test/java/org/cometd/websocket/ExtensionNegotiationTest.java

I don't think there are enough useful web socket extensions to justify the complexity of a user-provided extensions mechanism.
https://www.iana.org/assignments/websocket/websocket.xhtml#extension-name

I'm going to close this as working as intended. We don't support user-supplied extensions because the additional complexity of doing so correctly is very high, and the benefit is very low. This is similar to the policy in browser-based websocket APIs.

@swankjesse thanks, I believe that is fair enough.
IIRC CometD had a case where the WebSocket communication was using a custom extension known to both server and client - they won't be able to use the OkHttp transport but they have other transports.

What did the extension do? I assume most interesting things you'd do are more portable and work better in the payload than in extensions.


Mermaid no longer created nodes with classes?

I'm trying to upgrade from an older version. I used to create nodes that looked like this:
A[content]
But this syntax no longer seem to work. This is crucial my my usage, is there a way to create nodes items with classes?

It should be working. Could it be something else like the chrome 48 issue?
Follow this link to the live editor, for me this renders fine:
http://knsv.github.io/mermaid/live_editor/#/view/Z3JhcGggVEIKQVtjb250ZW50XQ

Thanks for the prompt reply!
I tried the following:
graph TD;
A[city]-->B
	A-->C;
	B-->D;
	C-->D;

Which works. This however, doesn't:
graph TD;
A[city]-->B
	A-->C;
	B-->D;
	C-->D;

From what I was able to gather, the nodes are not parsed correctly, so I get a the following error:

Error: Invalid value for  attribute viewBox="0 0 -Infinity -Infinity"

That's a result of the nodes not being found, so getExtremes doesn't have anything to run on, which leaves the sizes unchanged (at the end of that function the parameters are set to -Infinity).

I am having the same problem. I upgraded to latest version to get the Chrome 48 fix, but now my diagrams don't draw. The problem occurs when there is an equals sign in the node label. All of my nodes have spans with id's, titles, etc. The example below breaks because of the two equal signs in the span tag.
graph LR
A(Top Node);
A-->B
B-->C
C-->A
D-->C

I confirm that this is still an issue:
https://mermaidjs.github.io/mermaid-live-editor/#/edit/Z3JhcGggVEIKQVs8c3BhbiBjbGFzcz0ibm9kZSI-Y2l0eTwvc3Bhbj5d/error/UGFyc2UgZXJyb3Igb24gbGluZSAyOgouLi5oIFRCQVs8c3BhbiBjbGFzcz0ibm9kZSI-Y2l0eTwvc3Bhbj5dCi0tLS0tLS0tLS0tLS0tLS0tLS0tLS1eCkV4cGVjdGluZyAnU0VNSScsICdORVdMSU5FJywgJ1NQQUNFJywgJ0VPRicsICdHUkFQSCcsICdESVInLCAnVEFHRU5EJywgJ1RBR1NUQVJUJywgJ1VQJywgJ0RPV04nLCAnc3ViZ3JhcGgnLCAnZW5kJywgJ1NRUycsICdTUUUnLCAnUFMnLCAnUEUnLCAnKC0nLCAnLSknLCAnRElBTU9ORF9TVEFSVCcsICdESUFNT05EX1NUT1AnLCAnTUlOVVMnLCAnLS0nLCAnQVJST1dfUE9JTlQnLCAnQVJST1dfQ0lSQ0xFJywgJ0FSUk9XX0NST1NTJywgJ0FSUk9XX09QRU4nLCAnLS4nLCAnRE9UVEVEX0FSUk9XX1BPSU5UJywgJ0RPVFRFRF9BUlJPV19DSVJDTEUnLCAnRE9UVEVEX0FSUk9XX0NST1NTJywgJ0RPVFRFRF9BUlJPV19PUEVOJywgJz09JywgJ1RISUNLX0FSUk9XX1BPSU5UJywgJ1RISUNLX0FSUk9XX0NJUkNMRScsICdUSElDS19BUlJPV19DUk9TUycsICdUSElDS19BUlJPV19PUEVOJywgJ1BJUEUnLCAnU1RZTEUnLCAnTElOS1NUWUxFJywgJ0NMQVNTREVGJywgJ0NMQVNTJywgJ0NMSUNLJywgJ0RFRkFVTFQnLCAnTlVNJywgJ1BDVCcsICdDT01NQScsICdBTFBIQScsICdDT0xPTicsICdCUktUJywgJ0RPVCcsICdQVU5DVFVBVElPTicsICdVTklDT0RFX1RFWFQnLCAnUExVUycsICdFUVVBTFMnLCAnTVVMVCcsIGdvdCAnU1RSJw
It seems that HTML tags are supported but you cannot add attributes to those tags.
I will investigate when I have more time.

I managed to give my nodes/edges some properties by using the following chars:
graph TD
  %% My nodes-----------------------------------------------------------------------------------------------
  10000633(fa:fa-camera-retro My content here)

The code above works if I use the mermaid.initialize method only, but when I try to use mermaidAPI.render method, the page keeps loading forever and nothing shows up.
Currently, I'm trying to find an alternative for the problematic characters when using render.

Just solved it by using single quotes in JS (and properly escaping them):
var insertSvg = function(svgCode, bindFunctions){
element.innerHTML = svgCode;
};
var graphDefinition = 'graph TD\n' +
        '10000633(fa:fa-camera-retro Node 1)\n' +
        '10000562(Node 2)\n' +
        '10000633-.->|goes to|10000562';

mermaidAPI.render('graphContent', graphDefinition, insertSvg);

It's a nice workaround. In the long term we should improve the parser so it could have better HTML support

It's a nice workaround. In the long term we should improve the parser so it could have better HTML support

Is there any news on this improvement?
I have searched for better support for HTML tags in the node contents, but as fare as I can see, the problems still exists in the latest version (8.0.0-rc.8)...

Given how long ago this issue's activity died out, I'm closing it.
This allows us to maintain a fresh list of issues the community is interested in.


[language] Parser cleanup

Motivation
I noticed a few small things to clean up related to the new parser for the IR compiler. There were some documentation and comments that still referred to the old files, and we can avoid a lot of unnecessary type parameters by removing the unused UnrecognizedToken error.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
There are no functional changes here. I ran the functional tests to make sure that I didn't inadvertently break something.

@bors-libra r=tnowacki

:pushpin: Commit 67ec5b6 has been approved by tnowacki

:hourglass: Testing commit 67ec5b62ec567320a82329ebf4760394fd6f2375 with merge 1f5c808edbf801082b18e5a834ee89e65fae3a38...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: tnowacki
Pushing 1f5c808edbf801082b18e5a834ee89e65fae3a38 to master...


no localhost in list of servers

After change, missing staging/localhost
[
"Sparta Offline System",
"home.spartascience.com"
]

fixed by: https://github.com/sparta-science/connect/commit/3da29168bdb783dca657b5b0b241b89befe95706


Consistent disposal of receivers across adapters

Few items:

every adapter has a disposeDataReceiver function
all disposeDataReceiver functions handle uuid deleting creating a single point of uuid deletion per adapter
always end with the disposal of the receiver to ensure no repeat receivers on retries
cancelation and the end of a subprocess both result in the same function call to cleanup

label "skip-test" added for now until tests can be properly written that watch the order in which events happen. Only delayed to get bug fix in


Have the next slide slightly appear?

Hello there,
I love the library bu I was wondering if there were anyway to have the next slide slightly appear?
Thank you for your help/tips

Hello :) If by "slightly appear" you mean how fast next slide appears over the previous one please look at the live demo. There you will find on the right side property called slider-transition-duration. Play with it with different values to see if that is what you want to achieve. If it does, you will have to overwrite that property using raw CSS or create your own copy of slider styles in SCSS and modify it from there. Alternatively you could also create your own instance of component that you've chosen(e.g. autoplay.js) and adjust to your needs.

Thank you very much by the detailed reponse however I wasnt clear enough. What I meant next slide appears a little bit on the side. Like the example below.

Thanks for the details! As far as I am concerned repo consider only single slide examples. If you would like to achieve that effect you definietly would have to make some changes to the slider behaviour. I'm assuming that within the example you shown above by "next slide appears a little bit on the side" you mean only that next image fragment(not the text titled "A New Era of Glamour"). I've thought about one tricky idea which you could try.. If we look at e.g. autoplay.js you will find function called renderSlides which looks like this:
  const renderSlides = () => {
    return slides.map(slide => {
      return (
        <div
          key={slide.src}
          style={{ backgroundColor: slide.backgroundColor }}
          data-src={slide.src}
        />
      );
    });
  };

If you would like the user to see the incoming slide you would have to change that function first so instead of single slide it returns two(current one and the next one). Map function allows to refer to the next index e.g. stackoverflow(remember about case when referencing last image from slider to point to the first index not index + 1). This way each "slide" would have current one and the next one and you would only have to create appropiate styles for that incoming slide by using for e.g. clip property to crop image and then position it :)
I didn't try it but if it sounds promising feel free to try it :)
Alternatively you can look after ready solutions that instantly allows to have multiple slides like react-multi-carousel from YIZHUANG(click here for preview, I've thought about the example on the bottom of the page)

This is great! Thank you very much for your detailed answer. Might go with the multi-carousel library


Claims - ClaimantAddress

In the printing object, there are options to set both claimantAddress and payorAddress when creating and modifying claims. RB, however, only allows the setting of the payor's address, not the claimant's. Contrarily, there is an option to set the claimant's address on the Template, which seems more logical. It raises the question: why would one want to set different claimant addresses for claims being created? This inconsistency may lead to confusion and should be addressed.

Meeting TN-FMÞ-VH7-20 discussed that this had probably been intended back in the day to anticipate the possibility that large corporates could vary the claimant address printed on slips. This is however not being used currently by the banks, so the meeting agreed on this being dropped from the specification.


Arrays: multiple calls to 'set' or 'splice' messes up Collection keys

There was a question today in the Polymer G+ community page...
How do I swap two elements in an array
It appears that multiple sets or splices causes some problems w/ Collection keys. Eg: 1st set or splice works okay, 2nd fails.
Here is a jsbin:  http://jsbin.com/rupati/edit?html,output
Maybe this is related to / the same as:  https://github.com/Polymer/polymer/issues/2248
The console shows this error:
TypeError: Cannot read property '0' of undefined
    at Polymer.Collection._parseKey

The errors with the splice & set cases are a side effect of https://github.com/Polymer/polymer/issues/1913 (dom-repeat: Duplicate values result in unsynchronized data).  Since there is a point in time where one item is duplicated in the array (before the 2nd item overwrites the first), the collection can get out of sync.
Workaround is to ensure there are never duplicate items. e.g. splice both out, splice both back in.

@azakus : your workaround assumes duplicate items in an array is an error, which is not always true (for example, the user sets the values of the array).  In this example:
https://jsbin.com/qixufahego/edit?html,output
the javascript console shows:
Key for array[0] key=#1
Key for array[1] key=#1
setItem(key0, "toto")
Key for array[1] key=undefined

and the first and last lines are wrong.
The problem seems to be that keys in pmap are compared by value instead of by reference.  If array items are wrapped in objects though they would be compared by references.  This seems to do the trick:
http://jsbin.com/wubarenufi/edit?html,output
The javascript console now shows:
Key for array[0] key=#0
Key for array[1] key=#1
setItem(key0, "toto")
Key for array[1] key=#1

So another workaround if duplicate primitives are required is to wrap them in objects.
This wrapping could be done inside Collections.  In the ctor, instead of:
this.store = userArray.slice();

do:
this.store = [];
for (var i = 0; i < userArray.length; ++i) {
  store.push({value: userArray[i]});
}

However, methods like getKey() and removeKey() become O(n) running time.  It also requires that calls to remove() and getKey() be passed items in the array and not copies.  Functions like Polymer.set() and Polymer.notifyPath() seem to do that already.

Leaving this as an example that worked for me for function that moves elements up/down by an index:
        columnPosition: function (event) {
            var ix = event.detail.index;
            if (event.type === 'column-position-minus') {
                if (ix > 0) {
                    var left = this.splice('columns', ix - 1, 1)[0];
                    var right = this.splice('columns', ix - 1, 1)[0];
                    this.splice('columns', ix - 1, 0, right);
                    this.splice('columns', ix, 0, left);
                }
            }
            else if (ix < this.columns.length - 1) {
                var left = this.splice('columns', ix, 1)[0];
                var right = this.splice('columns', ix - 1, 1)[0];
                this.splice('columns', ix, 0, right);
                this.splice('columns', ix + 1, 0, left);
            }
        },

@rogerta This is a documented limitation of the data binding system in 1.x: we don't support duplicate array items (and by extension, arrays of primitives are not well-supported). The recommendation here is to wrap primitives into objects so they're unique.
https://www.polymer-project.org/1.0/docs/devguide/model-data#work-with-arrays

(Looks like you already updated your jsbin, but I'm leaving the comment here for other folks who may come across this issue.)

Closing this issue as the original JSBin shows that it is working as intended. Polymer.Collection was removed, which was the root cause for this issue 🎉


Should MultiMap.asMap() exist?

I'm struck that Multimap.toJavaMap() exists, but Multimap.toMap() does not.
multimap.keySet().toMap(Function.identity(), key -> multimap.getOrElse(key, Set())) is a bit much.
Should there be a new method here to make it simpler (similar to #1274)?

Hi @skestle,
sorry for the late answer.
We can add it to master (v0.9.x).v1.0 is still in the design stage.

I wanted to help, but I might be missing something: Multimap already has a method called asMap(), which appears to do what I'd expect. I found only one test, which turns Multimap { 1 => 2, 1 => 3 } into Map { 1 => [2, 3] }. This at least suggests exactly the same thing as @skestle seems to expect.
Were you look for something else, @skestle?


Allow reuse of layer rendering code without creating a map

This is part of an effort to make the layer rendering code a bit more reusable.  For the most part, layer rendering methods are provided what they need in the frame state, and they do not need access to a map or a map renderer.
Previously, the layer renderer's image loading handler would call map.render() directly.  Now layer renderers are observable and they fire a change event on image load (if otherwise ready).
The WebGL layer renderers can be refactored to be given an ol.webgl.Context instead of a map renderer.  This would replace all calls to mapRenderer.getContext() and mapRenderer.getGL() within the layer renderers.  However, the WebGL tile layer renderer is still tied to the map renderer through its use of the tile texture queue and cache.  If someone was motivated to change this, the tile renderer could be passed the queue and cache instead of maintaining a reference to the map renderer to access them.  Until then the WebGL layer renderers are a special case in that they need a reference to a map renderer.

Good changes. Looking forward to reworking the WebGL renderer. Please merge.


Integration Test - Release - 2019/03/25 19:56:48

Auto create for integration test. IssueCommandFlow

@PRMergerTest3 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


News / Library Spotlight Box Shape

It looks like the library spotlight box and recent news boxes were at one point separate, but they are now merged together leaving a notch at the top and bottom of the unified box.

This will be resolved when #798 is, but leaving open until then so I am sure of it.


Release 1.2.1

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d90ac88). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #19   +/-   ##
=========================================
  Coverage          ?   13.73%
  Complexity        ?      113
=========================================
  Files             ?       49
  Lines             ?     4565
  Branches          ?      391
=========================================
  Hits              ?      627
  Misses            ?     3885
  Partials          ?       53

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 d90ac88...0ca5158. Read the comment docs.


⚠ Shakedown Social - Homepage has degraded performance

In e53f6fd, Shakedown Social - Homepage (https://shakedown.social) experienced degraded performance:

HTTP code: 200
Response time: 17104 ms

Resolved: Shakedown Social - Homepage performance has improved in 6f01f16 after 2 minutes.


Filter duplicate enabled chains. Happens if user enables a chain and we disable that chain in an update so the user could end up with Ethereum mainnet enabled twice since ETH mainnet is the fallback

Bad merge commit title since it took the 2nd commit's message 😢


Add support for description field in heads

Description field text will show up on the admin page for easier documentation and a good help for users

Please let me know if i've missed something when it comes to where to document that the description tag is a viable option on the heads, and/or I need to add it in more places.

Hi Mikael, thanks for the patch!
Adding a new, optional field to all heads is easy, you just have to add the value 'description' to the IMPLICIT_OPTIONAL_PROPS array at the top of lib/heads.js.
I added it locally to check your change, and I'm not sure myself about the current solution, it's pretty non-obvious that clicking on a row will show a description. Another thing that bothers me is that because of how the CSS is made, the description row will take one of the alternating row colours, so that when the description is collapsed, there will be two rows with the same background colour. Finally, clicking on the attached/detached button produces a quirky behaviour, because the page is reloaded...
Have you seen it working, what do you think? Maybe it would be better to show the description somehow separately, like in a tool tip?

Hi
Thanks for getting back to me :)
I'm back at the office on Monday and I have loads of things during the weekend. I can absolutely look into it. Adding support for jquery in the admin will help a lot too.
Here's what I wanted to achieve:

Easy documentation for people to understand the endpoint and its function
Add a test part below for inline testing

I'll get back more next week.
Have a good weekend.
// Mikael X Johansson
Hennes & Mauritz
Skickat från min iPhone

6 mar 2015 kl. 21:57 skrev Esteban Manchado Velázquez<EMAIL_ADDRESS>Hi Mikael, thanks for the patch!
Adding a new, optional field to all heads is easy, you just have to add the value 'description' to the IMPLICIT_OPTIONAL_PROPS array at the top of lib/heads.js.
I added it locally to check your change, and I'm not sure myself about the current solution, it's pretty non-obvious that clicking on a row will show a description. Another thing that bothers me is that because of how the CSS is made, the description row will take one of the alternating row colours, so that when the description is collapsed, there will be two rows with the same background colour. Finally, clicking on the attached/detached button produces a quirky behaviour, because the page is reloaded...
Have you seen it working, what do you think? Maybe it would be better to show the description somehow separately, like in a tool tip?
—
Reply to this email directly or view it on GitHub.

I thought about jQuery when I started writing the admin UI, but decided against it because I didn't have a lot of use for it and wanted to keep the page as simple as possible. Depending on what we end up doing here, it might be worth adding it.
I'm unsure about the inline testing, though. How useful is it, do you have use cases of the top of your head? Are you thinking about the people writing the plugins, or the people using them? I can't imagine that a lot of people using the plugins would need the inline tester. In comparison, I think I'd rather integrate some way to read the logs created by the "logger" plugin, from the admin UI. That helps understand what's going on when there are problems, which seems more useful for the plugin users.
Anyway, looking forward to your ideas!

Hi
So Monday is here, just wanted to explain how we use your tool here, also Gustav Weslien you have been talking to (about the proxy) is my Colleague and we’re working on this together.
We use the tool as a mock-service for our internal APIs which means that a lot of users are using the mock for testing which gives why I need the documentation and testing tool, we use headers for authentication due to the fact that our aged internal APIs use that and not oAuth2.
My plan if you approve is to add jQuery to the admin interface, fix the documentation in a good way and maybe add support for markdown documents to be placed and then rendered in place of some sort of documentation for a fully fletched system inside robohydra.
My current solution (forget the look on it right now) reads markdown files on our end and parses the html into the description field i’ve added, this is working pretty ok but as said the look will be improved, just kicked it together to show you my thoughts before i finished it.
Sincerely
MIKAEL X JOHANSSON
H & M HENNES & MAURITZ IT - CMS Team
ÅRSTAÄNGSVÄGEN 13, 117 43 STOCKHOLM, SWEDEN
PHONE: +46 8 5780 6257  MOBILE: +46 735 405301
EMAIL<EMAIL_ADDRESS>FOLLOW US ON:
www.hm.comhttp://www.hm.com/
www.facebook.com/hmhttp://www.facebook.com/hm
www.twitter.com/hmhttp://www.twitter.com/hm
www.youtube.com/hmhttp://www.youtube.com/hm
The information contained in this e-mail message may be privileged, confidential, and protected from disclosure. Any unauthorized use, printing, copying, disclosure or dissemination of this communication may be subject to legal restriction or sanction. If you think that you have received this e-mail message in error, please reply to the sender and delete this message from your computer.

I'll drop you an e-mail, I need a bit more information to wrap my head around the exact usage, and you may not want to publish that information here and/or this thread will be cleaner without that discussion...

Sounds good, please do :) removed some messages

Ah, no need to remove, I just didn't want to add a lot more messages while I understood exactly what you need. I'm a bit slow sometimes, you know? :-)
Anyway, message sent, check your spam folder just in case (in happens sometimes)...

Did you get my e-mail?

OK, closing for now, as there hasn't been any activity for over a year.


Produce .NET Standard 2.0 packages now that it is final

.NET Standard 2.0 now is now final (https://github.com/dotnet/announcements/issues/24).
@jskeet Is this something you/we have on our plans?  I'm happy to test this on our libs.
cc: @Jonathan34

I'd wait until .NET Core 2.0 is out, to be honest - netstandard2.0 is just the specifications, effectively.
When .NET Core 2.0 SDK comes out, we'll almost certainly want to update our build procedures to use that anyway (no more dotnet restore required - yay!) but we'll still want to target netstandard1.x. We should probably test on netcoreapp2.0 as well as netcoreapp1.0...

Sounds good, I'll close this for now and we can revisit when .Net Core 2.0 is final.


Incorrect type signature for RPCProviderModule.enable

✅ Prerequisites

[x] Did you perform a cursory search of open issues? Is this bug already reported elsewhere?
[x] Are you running the latest SDK version?
[x] Are you reporting to the correct repository (magic-sdk)?

🐛 Description
The return type for RPCProviderModule.enable is Promise<string>, but the actual value returned is Promise<string[]>.
🧩 Steps to Reproduce

[First Step]
[Second Step]
[and so on...]

🤔 Expected behavior
The type signature should reflect the actual return type.
😮 Actual behavior
The type signature reflects string instead of the correct string[].
💻 Code Sample
None.
🌎 Environment

Software
Version(s)

magic-sdk

Browser

yarn

Operating System

:rocket: Issue was released in<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>:rocket:


Review cache data structure for feasibility to insert new data into it

Review cache data structure for feasibility to insert new data into it; right now it only stores arrays.
Spec: https://docs.google.com/document/d/1kZ5PS_cnUNo9gvop28qd92stZSr-bzzHtp6NK1nKDPw/edit?usp=sharing

Closing this card with a new card that's more descriptive.


Parsing the quotes on logfmt format

I'm trying to parse quotes on logfmt format but, the value after the quote become new key with value "True".
iimport logfmt from logfmt import format import io raw_text=io.StringIO("ts=1579739313\"80007168") rawdata=logfmt.parse(raw_text) for i in rawdata:     print(i)
output
{'ts': '1579739313', '80007168': True}

Thanks for reporting this issue. This project has moved to https://github.com/wlonk/logfmt-python and the folks over there are likely best equipped to address this.


Missing Error Definitions or References in 9 Swagger Specs

Missing Error Definitions or References in 9 Swagger Specs

Data Lake Catalog
DNS
Graph RBAC
Logic
Notification Hubs
Resources Features
Resources Lock
Search
Service Fabric

Notification Hubs REST API throws several error codes however its Swagger does not define any of these.
Search REST API throws several error codes however its Swagger does not define any of these.
Azure Active Directory Graph API throws several error codes however its Swagger does not define any of these.
So on and on ...

Which error codes are you referring to? Please be specific or the issues will be long lived and difficult to determine if they are completely resolved.

@devigned - if you go to their corresponding REST API specs, then you can see it. I provided some examples in the first comment

@Azure/adx-autorest-contributors and @Azure/openapi-advisors any thoughts on this issue?

@devigned Two thoughts, one specific to Search, one general:

The Search Swagger specs are not what I would call "production-ready" yet. We have an issue open to decide what to do about this: https://github.com/Azure/azure-rest-api-specs/issues/195  It would be great to close on it so we can be omitted from these sweeps until we are ready.
From a codegen standpoint, would adding the additional error codes change anything? For example, in C#, AutoRest basically assumes that success means return a response, while failure means throw a CloudException. I'm confident we have already captured all the success cases in the Search specs. Other than for documentation purposes, is there any value in explicitly enumerating all the possible failure cases in the specs?

https://github.com/OAI/OpenAPI-Specification/issues/566 <-- to help follow 3.0 suggestions

We should have a shared (common) set of schema, a base error should be returned by default (non-success responses) specialized by the error code via a polymorphic discriminator. This will provide the information to specialize the initialized class as well as the to document all of the possible error responses.

Closing since this is being tracked here - https://github.com/Azure/autorest/issues/1523.