In the PHP version, you can include templates from multiple locations

It would be nice to be able to pass an array of locations for the path option.

+1

+1

+1

+1

+1

+1

if I understood you are talking about namespaces, it is now working at the last version: https://github.com/justjohn/twig.js/blob/master/CHANGELOG.md
and I added a example on how to implement on the wiki

Namespaces are a great addition, but I am talking about cascading locations.
In php, you can specify an array of folders where to look for templates.
Twig will loop through the locations until it finds the template.
If I define my path as [folder-a, folder-b], Twig looks for template.twig first in folder-a and if it doesn't exist, it will cascade to look if it exists in folder-b.

in Express 4 we'd hope to set it up like
app.set('views',  ['path/to/files/', 'path/to/other/files'] ;

But when you do this, Twigjs doesn't see this as an array of 2 items. It tries to look in:
'path/to/files/,path/to/other/files'

Appears that the relativePath function assumes template.base to be a string, and not an array (which it is when multiple paths are defined in express.
I took a little stab at a fix, but it might require more of a rewrite than I'm able to do in order to support array paths.

Namespaces are a great addition, but I am talking about cascading locations.
In php, you can specify an array of folders where to look for templates.
Twig will loop through the locations until it finds the template.

Actually, they are the same thing. In the PHP Twig, there is a default "main" namespace. When you add paths to search for templates, you are adding paths to the main namespace. Here's the relevant docs: http://twig.sensiolabs.org/doc/api.html#built-in-loaders
There are 2 incompatibilities with Twig.js here.

Twig.js does not have a main namespace.
Namespaces can only have a single path. But namespaces should have an ordered array of paths.

Right now we are required to pass namespaces every time we call Twig.twig() which is kind of a pain, but manageable. However, if Twig.js adds a main namespace, it is going to have to manage a global registry for namespaces which means a user should be able to register their custom namespace once with Twig.js and not have to provide it every time it calls Twig.twig().

Here's more Twig docs. http://symfony.com/doc/current/cookbook/templating/namespaced_paths.html#multiple-paths-per-namespace
With Symfony, Twig.js' namespaces are configured by a YML file:
# app/config/config.yml
twig:
    # ...
    paths:
        "%kernel.root_dir%/../vendor/acme/themes/theme1": theme
        "%kernel.root_dir%/../vendor/acme/themes/theme2": theme
        "%kernel.root_dir%/../vendor/acme/themes/common": theme

In Twig without Symfony, you have to add namespace from within a loader. And then use that loader when creating the global Twig environment
$twig = new Twig_Environment($loader);

Or inserting it into the global twig with:
$twig->addLoader($loader);

Many Symfony-based apps use "services" to configure a Twig_Loader_Chain that includes several different loader types. So if you want custom namespaces, you register your custom loader with the services and it pushes it onto the loader chain stack. When a template is requested, Twig goes through each loader in the Twig_Loader_Chain stack and tries to find a match. Since your @mypath/template.twig will only match a namespace in your custom loader, your custom loader will be used to resolve any custom namespaces.
 So how do we add it to Twig.js?
Twig.js has an equivalent to Twig_Loader_Chain in that Twig.exports.twig() goes through a long if/then/else checking for which loader to use.
I'd rather not have to implement a custom loader that was essentially a fork of the fs loader just so I can add custom namespaces.
Ideally, we would be able to configure Twig.js' fs or ajax loader  by adding paths to the default namespace or adding custom namespaces. PHP Twig loaders have these methods for adding paths and namespaces: setPaths(), addPath(), and prependPath(). e.g. $loader->addPath('/etc'); adds a path to the main namespace and $loader->addPath('/etc', 'mynamespace'); adds that path to a custom namespace. If you want multiple paths in your namespace, you call addPath() or prependPath() multiple times. FYI, setPaths() only adds paths to the main namespace, you can't specify custom namespaces.  https://github.com/twigphp/Twig/blob/f0a4fa678465491947554f6687c5fca5e482f8ec/lib/Twig/Loader/Filesystem.php
However, in Twig.js, we aren't resolving namespaces inside loaders, we are doing it in twig.path.js, so our separation is already a little muddy.
The twig.path.js module is in charge of replacing namespaces with paths for fs and it completely ignores the concept of namespaces for the ajax loader. Wouldn't ajax users want a nice conversion of @mysite/template.twig into http://mysite/myfolder/template.twig? To be clear, I don't think multiple URLs would make sense, but a single path per namespace would be mighty handy.
So we have 2 options:

Export Twig.Templates.loaders, so users can use Twig.loaders.fs.addPath(), Twig.loaders.fs.setPaths(), Twig.loaders.ajax.addPath(), etc. That means Twig.path.parsePath() would just be calling the loader's findTemplate() method to get the correct path.
Add methods to Twig.path that resolves namespaces for any loader. That means you can't have a single namespace that resolves to different paths; e.g. "myStuff" resolving to a URL for ajax and to a file system path for fs. Users can use Twig.path.setPaths(), Twig.path.addPath(), and Twig.path.prependPath().

Option 2 is shorter for the user (and easier to implement), but option 1 is closer to what Twig PHP does.
I'm happy to do the PR for option 1.
Sorry for the long comment. Thoughts?

I'll counter your long comment with a short one :)
We go for whatever solution gets us closer to parity with Twig PHP. Adding sugar to said solution could be done in an additional module.

I'll see if I can get a PR for this done this week. @micahgodbolt and I are both at the same conference this week.

Any movement on this? It's been months and there doesn't appear to be a solution or an open PR

Bump :wave: Would love this too!

any updates ?


Implement w3rc CLI to look up providers for a CID

Implement a CLI with find command that looks up the list of providers
for a given CID. The command takes
Add testscripts for the CLI that assert expected error messages in
negative cases, and a positive case via a mock indexer server.
Relates to:

https://github.com/ipfs-shipyard/w3rc/issues/6

Closing for now until retrieval functionality is in place to implement a functioning CLI. Note, find is not needed as a separate top-level command.


Fixed #1024 - InstanceOfCheckForException being too general

There is a weird error with the Travis CI going on. This has happened more often recently.


Update badges for digiaonline org

Changes proposed in this pull request:

Update badges for digiaonline organisation
Also re-added the project to Scrutinizer and Coveralls, the old Nord ones hadn't triggered for a couple of years
Also re-added to Code Climate for good measure
Also consistently link to develop branches

Or shall we ditch develop and only use master?

Let's ditch develop after the current batch of PRs go in.


bug: Custom Dictionaries Uppercased fails to match

Describe the bug
We use a custom dictionary for other things I was hoping to use the same dictionary here. The issue is this dictionary is all upper case.
I expeted it to pass the test in all cases
List the steps that must be taken to reproduce this issue

use custom dictionary containing uppercase words
Run the test

Version
latest
Additional information
I have a functional fork that i would like to bring in where i am ToLower the dictionary words as they come through the Extract profanity function

I've just made it clear in https://github.com/TwiN/go-away/commit/b8dfd2b494133faf637012f9fcc6c38ab0a38b53, but to keep it short, all custom dictionaries are expected to be pre-lowercased.


RFP - IECookiesView

https://www.nirsoft.net/utils/iecookies.html
tag: nirsoft

In homepage it says

License
This utility is released as freeware. You can freely use and distribute it. If you distribute this utility, you must include all files in the distribution package including the readme.txt, without any modification!

This seems to allow us to embed the binary into the package? We need it because the URL is not version-dependent.

@FranklinYu Are you picking this up?
Whoever does pick this up, in the package LICENSE.txt file include that text and include a link to it on the website.

Status: Review
https://chocolatey.org/packages/iecv


Milestone 2: User interface

This pull request contains the mockup of the user interface for the TicTacToe game

We have fixed what was asked as we thought best, but the messages are contradictory and make not much sense to us:

[ ] game informs player if the selected move is a draw move (without computing draw): How are we going to know a draw move without computing it?
[ ] game informs player if the selected move is invalid(without computing draw): How are we going to know an invalid move without computing it?
[ ] game repeats all actions for the next player's move and for this, you will be needing a loop, preferable a while loop which needs to be terminated at some point (after a winning move or draw to be more specific). How are we going to start or end a loop if we cannot compute the winning conditions?

We have fixed what was asked as we thought best, but the messages are contradictory and make not much sense to us:

[ ] game informs player if the selected move is a draw move (without computing draw): How are we going to know a draw move without computing it?
[ ] game informs player if the selected move is invalid(without computing draw): How are we going to know an invalid move without computing it?
[ ] game repeats all actions for the next player's move and for this, you will be needing a loop, preferable a while loop which needs to be terminated at some point (after a winning move or draw to be more specific). How are we going to start or end a loop if we cannot compute the winning conditions?


Update app-service-logic-enterprise-integration-overview.md

added try it now para/link

Hi @jonfancey, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (jonfan). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@jonfancey : Thanks for your contribution to the Azure documentation! The author, @msftman, has been notified to review your proposed change.

Reviewed changes and confirm they should be added. #sign-off

Only the author of the article can use the sign off comment to add the ready-to-merge label.

There are 2 content developers for everything in app-service-logic: MandiOhlinger and MsftMan (Deon Herbert). #sign-off


Add support to python coverage reporting

Python coverage is a bit different here:
Python package file __init__.py can be empty, so upon submit, opencov cannot process the following coverage data correctly:
{
            "source": "",
            "name": "pkg/xxx/__init__.py",
            "coverage": [
                null
            ]
}

Hi thanks for reporting.
I do not have time to fix this right now, but I would be very happy to merge a PR.
Thank you.

@timonwong How do you configured your python project to send the report?
I did it like this:
export COVERALLS_HOST="http://localhost:4000"
export COVERALLS_REPO_TOKEN=Nk1Vck5aMnRXdlB4amFwVkplVi9Sd2JDTml2YU1CMzhib29TOEVKVQ==
coveralls

But only got a lot of "No source for ..." messages and a "Reporting 0 files" at the end.


Proofing tools installed but not selectable

Hi!
We successfuly deployed O2019 with proofing tools.
But we have just one issue.
Serbian (latin) proofing tools ARE installed but not working in office apps (Outlook, Word ...).

If this is not deploy error or install error but rather app error, can you please report it upstream?
Thanks!

Thanks the feedback--I've passed this issue to the product team to investigate.

@gregecslo.  You must enable that particular language under windows settings --time& language -- language, add that language. once installation completed, you will see proofing language

Hi!
We tried it but it doesn’t work.
We have numerous other proofing tools installed and they work just fine just Serbian doesn’t.
I still believe its a bug.
Regards, Gregor
From: VARADHARAJAN K<EMAIL_ADDRESS>Sent: Sunday, 23 June 2019 18:21
To: MicrosoftDocs/OfficeDocs-DeployOffice<EMAIL_ADDRESS>Cc: Grega<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/OfficeDocs-DeployOffice] Proofing tools installed but not selectable (#324)
@gregecslohttps://github.com/gregecslo. You must enable that particular language under windows settings --time& language -- language, add that language. once installation completed, you will see proofing language
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/OfficeDocs-DeployOffice/issues/324?email_source=notifications&email_token=AECKLCNSQF4HFC3SGGQX5MDP36PHVA5CNFSM4HA3WRP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYLCAJA#issuecomment-504766500, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AECKLCL3AHAYOOL2HRH6I3TP36PHVANCNFSM4HA3WRPQ.
[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/MicrosoftDocs/OfficeDocs-DeployOffice/issues/324?email_source=notifications\u0026email_token=AECKLCNSQF4HFC3SGGQX5MDP36PHVA5CNFSM4HA3WRP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYLCAJA#issuecomment-504766500", "url": "https://github.com/MicrosoftDocs/OfficeDocs-DeployOffice/issues/324?email_source=notifications\u0026email_token=AECKLCNSQF4HFC3SGGQX5MDP36PHVA5CNFSM4HA3WRP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYLCAJA#issuecomment-504766500", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

Hello!
This is not resolved, still unable to set proofing language to Serbian.

Hi.
Can someone please take a second look at this one?
This is still not resolved, proofing tools for Serbian LATIN still dont work. We deployed 13 others that do work ok, just this one doesnt.
Can you please reopen this issue?
Thanks!

Thanks for the feedback. I just passed this information along to the product team so they can investigate. In the meantime, I recommend working with support  to address this issue.

Quick update: the product team is investigating and looking at getting this fixed. I'll update with a status when I hear more. Thanks again for catching this.

Hi!
Issue is resolved and confirmed working.
Thanks again!

Glad to hear it--and thanks again for catching this.


feat: add libp2p and initial support for sync

Motivation
Hub's need a sync protocol to store and share data amongst each other. This is the first step towards implementing the p2p mechanism for sync.
Change Summary
Added libp2p and some scaffolding for creating a network of nodes.
Added Initial definitions for network protocol messages
Updated the repo to use ES modules instead of CommonJS
Merge Checklist
Choose all relevant options below by adding an x now or at any time before submitting for review

[x] The title of this PR adheres to the conventional commits standard
[ ] The PR has been tagged with the appropriate change type label(s) (i.e. documentation, feature, bugfix, or chore)
[ ] Changes to the protocol specification have been merged

Additional Context
Draft checklist:

[ ] No tests for the new node.ts file. Can't seem to run it with Jest after switching the project to use ES modules
[ ] Need naming suggestions for node and protocol members
[ ] Should protocol just live inside types? I separated it out to distinguish between "sdk" level API and pure internal

After getting this in order, I should be able to file a few different issues for a few different follow up tasks.

Codecov Report
Base: 90.61% // Head: 90.60% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (06bc7b0) compared to base (7fbc51f).
Patch coverage: 88.88% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #95      +/-   ##
==========================================
- Coverage   90.61%   90.60%   -0.02%
==========================================
  Files          22       23       +1
  Lines        1194     1203       +9
  Branches      250      251       +1
==========================================
+ Hits         1082     1090       +8
- Misses        105      106       +1
  Partials        7        7

Impacted Files
Coverage Δ

src/network/protocol.ts
88.88% <88.88%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


connection interrupts

need to test whether things recover after laptop hibernate or  loss of internet connection.

added a timeout of 4 seconds, but still occasionally issues with connection. need to look into it more

added 3 retries for network communication. timeout down to 1 second. seems stable enough


multiple go versions on Ci

FYI: setup-go@v4 does enable caching of GOCACHe by default. It means you can add fuzzing to CI - and progress (coverage cases) will keep between runs. Just limit it by -fuzztime


Incorrect percentages

I think the signalling/ucoming/non-signalling percentages are calculated incorrectly.
This screenshot shows the page, as well as a calculator showing the expected values:

PS Love this project!

@casey Thank you for reporting about this.
Yes, it looks like a regression was recently introduced, I'll address this and deploy an update soon.

PS Love this project!

I appreciate that! 🙏

Also why equal?

Foundry USA
6.02%
✅

Huobi.pool
6.02%

SlushPool
3.61%
✅

1THash
3.61%
🚫

ArkPool
1.20%
🚫

BTC.TOP
1.20%

SBI Crypto
0.60%
🚫

EMCDPool
0.60%
🚫

SpiderPool
0.60%

@hMsats Most likely because there aren't that many blocks mined yet for this period. I don't think there's any errors here.

@hsjoberg Yeah, that could be it. Just wanted it to get your attention just in case. Nice website!

@hMsats Thanks for caring and reporting. Cheers! 🙂


implement custom loss for multi-output training that ignores unknown outputs

If an output value for a data point is NaN, the corresponding output shouldn't impact the loss for that data point. Necessary for multi-output mass-spec + affinity predictors since for most points we know one output value but not the other.
@iskandr I'm down to implement this but just want to make sure you haven't already so we don't duplicate work?

I have an implementation using negative values as the sentinel (instead of NaN) because handling NaN's via Keras's backend seemed tricky. My old solution was to use an explicit Input to mask out each output but with multiple outputs this made for ugly/complex models (since each output must be wired to its input via a custom loss and the masks must be passed in by pepnet). Thinking about going back to that, but curious to see if you find a more elegant way to do things.

Specifically, I didn't find anything like keras.backend.isnan but we need that to properly mask out the losses.

For reference, my old masked loss was like this:
https://github.com/matthiasplappert/keras-rl/blob/master/rl/agents/dqn.py#L183
(currently lives in a notebook on my laptop)

Thanks @iskandr . Also possibly relevant (seems he got it working on theano backend?): https://github.com/fchollet/keras/issues/1628

Yeah, I was thinking about using T.isnan but I didn't like reaching underneath the keras backend abstraction. But maybe that's the cleanest solution.

This seems to work for me, although it's theano-specific
import keras.backend as K
import theano.tensor
def custom_loss(y_true, y_pred):
    squared = K.square(y_pred - y_true)
    loss = K.sum(
        K.switch(theano.tensor.isnan(y_true), 0.0, squared),
        axis=-1)
    return loss

I'll make a PR if this seems good to you @iskandr

Fixed by https://github.com/hammerlab/pepnet/pull/6


[android] add selected background to My Files selection mode

When in selection mode, anywhere in the app we should apply the same styling. In Settings -> Search engines we are using a light grey background for the selected items. I wanted to add the same to My Files -> Select All mode.
I tried and failed. Can someone help? It's probably super simple.

I'm afraid it's not super simple

@aldenml @gubatron  I do want the design to remain consistent in selection mode in different parts of the app, but after a day spent on trying to do this, I am giving up for now. Any pointers, clues on how to proceed?
I was able to make it happen by adding v.setSelected(true) to onFileItemClicked(View v), in order to activate the background selector but there was still few issues, like the listView being recycled and the selected background being applies to multiple list items as you scroll. Or the checkbox being set up separately in view_selectable_list_item and the background not being added under the checkbox.
The main issue though is with recycled views. As I am applying the selectable background to a view which is later reused by another item. And I am not sure how to make sure that the actual position is remembered. There are few similar issues people experienced but I was not able to successfully apply what I learned from their cases in FW. Would love your help.

in order to avoid any issues with the recycled views I suppose you gotta do your background change on the adapter's getView(), that's where I'd change the background, depending if the current view in the list view being returned is selected or not.

Test with master (https://github.com/frostwire/frostwire/commit/14662bade365c176fbd99466d8e04652c05bf207) and close if it's the expected


Mfarthin/add read xml

This isn't ready to merge yet.
Adds initial implementation of functions for reading mesh from xdmf in serial. Not intended to be a general solution yet. First goal right is to be able to generate hex mesh files from OpenFoam by first translating them to xdmf via foamMesh2vtk and then paraview.
Functions are in MeshTools.py right now
TODO

[x] test reading simple hex mesh from vtk
[x] fix c0q1 poisson example with RegularDomain
[x] fix c0q1 poisson example when read a hex mesh
[ ] test reading simplicial meshes in xdmf

@mfarthin , you mentioned that this is about ready to go in, didn't you?

@cekees I added the poisson based tests for the hex mesh stuff to the test/ci. They run on my machine ...
Assuming Shippable says it's ok then I think it's good to merge

OK, thanks. I've got to update the shippable image and insulated us from
changes in hashstack before the shippable badges are reliable again.
On Fri, Mar 13, 2015 at 4:36 PM, mfarthin<EMAIL_ADDRESS>wrote:

@cekees https://github.com/cekees I added the poisson based tests for
the hex mesh stuff to the test/ci. They run on my machine ...
Assuming Shippable says it's ok then I think it's good to merge
—
Reply to this email directly or view it on GitHub
https://github.com/erdc-cm/proteus/pull/229#issuecomment-79432716.

@cekees - It sounds like you've got this in hand, but let me know if I can help on the HashDist isolation angle.  One option would be to let HashDist control the install of Proteus itself, and to tell HashDist to look locally for the Proteus revision to build/test.  You could then hardcode in a HashDist "stack" profile that just requires proteus and nosetests.  Like the ones we have in air-water-vv (see the pelican branch) and roams.

@mfarthin The test is failing, but it runs fine on my machine when I just run that test. This is not the first time we've had  issues with strange behavior while running all the tests. I'm trying to debug it.

Ok, thanks. Let me know if there’s something I can look at to fix it. Matthew
On Mar 26, 2015, at 11:05 AM, Chris Kees<EMAIL_ADDRESS>wrote:

@mfarthin The test is failing, but it runs fine on my machine when I just run that test. This is not the first time we've had issues with strange behavior while running all the tests. I'm trying to debug it.
—
Reply to this email directly or view it on GitHub.

I think that should do it for now. We may want to add some file handling tools  to proteus/tests/util.py to make our test file i/o cleaner.  We could probably write something simple that would make it simpler to reference test input data and also store output data in per test output directories.

@cekees thanks for fixing the path.


Medium of reporting security vulnerabilities

Hi,
We are planning to use golden layout in one of our projects. So that we ran some security tools and came across some vulnerabilities in Golden layout. We are currently evaluating the report for false positive etc. What is the recommended medium of reporting such vulnerabilities ?
Thanks,
Tanya

Maybe you can post them here.
I'm using GL in production and it would be very helpful to know about security issues.

@TanyaM Any updates?

Closing issue due to no recent updates. Feel free to contact me or reopen the issue however


Migrate Sound.js to Typescript

Done. It's renamed to "DemoSampler" with a reduced functional range.


Improvements to grid editor

For making replacement of some layouts easier, we can now directly hide/show bricks in the grid editor

free camera
toggle between original and replaced models
highlight replaced and hidden bricks

Preview here: https://pr-554.lba2remake.net

The LGTM alerts are expected.


License in package.json

There are cases where the license is in the package.json file, for example: https://github.com/component/indexof . At least this doesn't show up in the licenseDeclared of the SPDX output.

I can't reproduce this as I just ran an analysis and it yielded:
---
SPDXID: "SPDXRef-DOCUMENT"
spdxVersion: "SPDX-2.2"
creationInfo:
  created: "2023-03-28T08:46:25Z"
  creators:
  - "Tool: OSS Review Toolkit - DOCKER-SNAPSHOT"
  licenseListVersion: "3.20"
name: "Unnamed document"
dataLicense: "CC0-1.0"
documentNamespace: "spdx://436ababa-c58d-4e0d-aa30-c5d3a6281d7b"
documentDescribes:
- "SPDXRef-Project-NPM-component-indexof-0.0.3"
packages:
- SPDXID: "SPDXRef-Project-NPM-component-indexof-0.0.3"
  copyrightText: "Copyright (c) 2011 TJ Holowaychuk<EMAIL_ADDRESS>    \ (c) 2012 TJ Holowaychuk"
  downloadLocation: "NONE"
  filesAnalyzed: false
  homepage: "NONE"
  licenseConcluded: "NOASSERTION"
  licenseDeclared: "MIT"
  name: "component-indexof"
  summary: "NONE"
  versionInfo: "0.0.3"

If you analyzed this package directly, did you maybe forget to enabled allowDynamicVersions (and ignored the respective error in the analyzer result)?

As a side note, https://github.com/component/indexof/blob/master/package.json is awkward as it contains the license key twice (here and here), but our JSON parser is able to tolerate that.

At least this doesn't show up in the licenseDeclared of the SPDX output.

I believe your specific issue has been addressed by https://github.com/oss-review-toolkit/ort/pull/6760.


Keyboard Manager doesn't correctly identify the Office and Emoji keys on a Microsoft Ergonomic keyboard

ℹ Computer information

PowerToys version: 0.33.1
PowerToy Utility: Keyboard Manager
Running PowerToys as Admin: no
Windows build number: [run "winver"] 19043.844

📝 Provide detailed reproduction steps (if any)

Tick Enable Keyboard Manager
Click Remap a key
Click the + button
Click Type
Press either the Office key or the Emoji key
Look at Key pressed

✔ Expected result
What is the expected result of the above steps?
The Office and Emoji keys should be reported as such.
❌ Actual result
What is the actual result of the above steps?
Currently the  Office and Emoji keys are reported as Win (Left) and Space respectively.
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

Duplicate of https://github.com/microsoft/PowerToys/issues/3194


Add Settings context menu

At first I thought i couldn't change the offset, or any settings at all. Just before I deleted it I found out that there were settings if you (double) click the tray icon.
Please make a context menu for Settings. Thanks!

Just Left Click the Icon, than you will see the Preference window.

I understand, as I figured it out on time, but it is not intuitive.

@KoalaBear84 I made a fork and fixed lots of bugs including this one.  You can visit my forked repo to install the latest version.
You can simply download the installer: https://doggy8088.github.io/carnac/publish/setup.exe
This version includes an Auto Update feature by ClickOnce.

Great, maybe you can also make PRs for it so it can be transferred back to the main repository.

I think the repo owner is out of this project for long time.  Let's why so many issues and PRs has no progress.

Yes, that is definately right and unfortunate.


Unable to load Windows tagging script

venv "D:\stable-diffusion-webui\VENV_DIR\Scripts\Python.exe"
Python 3.10.8 (tags/v3.10.8:aaaf517, Oct 11 2022, 16:50:30) [MSC v.1933 64 bit (AMD64)]
Commit hash: 78d6aef30249530d8cc0b5023b2f8bf8c86f8446
Installing requirements for Web UI
Launching Web UI with arguments: --medvram
Installing pywin32
Aesthetic Image Scorer: Unable to load Windows tagging script
LatentDiffusion: Running in eps-prediction mode
DiffusionWrapper has 859.52 M params.
making attention of type 'vanilla' with 512 in_channels
Working with z of shape (1, 4, 32, 32) = 4096 dimensions.
making attention of type 'vanilla' with 512 in_channels
Loading weights [40b9da7a] from D:\stable-diffusion-webui\models\Stable-diffusion\ma3.ckpt
Global Step: 487750
Applying cross attention optimization (Doggettx).
Model loaded.
Loaded a total of 0 textual inversion embeddings.
Embeddings:
Running on local URL:  http://<IP_ADDRESS>:7860
To create a public link, set share=True in launch().

venv "D:\stable-diffusion-webui\VENV_DIR\Scripts\Python.exe"
Python 3.10.8 (tags/v3.10.8:aaaf517, Oct 11 2022, 16:50:30) [MSC v.1933 64 bit (AMD64)]
Commit hash: 78d6aef30249530d8cc0b5023b2f8bf8c86f8446
Installing requirements for Web UI
Launching Web UI with arguments: --medvram
Installing pywin32
Aesthtic Image Scorer: Forcing prediction model to run on CPU
LatentDiffusion: Running in eps-prediction mode
DiffusionWrapper has 859.52 M params.
making attention of type 'vanilla' with 512 in_channels
Working with z of shape (1, 4, 32, 32) = 4096 dimensions.
making attention of type 'vanilla' with 512 in_channels
Loading weights [40b9da7a] from D:\stable-diffusion-webui\models\Stable-diffusion\ma3.ckpt
Global Step: 487750
Applying cross attention optimization (Doggettx).
Model loaded.
Loaded a total of 0 textual inversion embeddings.
Embeddings:
Running on local URL:  http://<IP_ADDRESS>:7860
To create a public link, set share=True in launch().
100%|██████████████████████████████████████████████████████████████████████████████████| 50/50 [00:14<00:00,  3.52it/s]
Total progress: 100%|██████████████████████████████████████████████████████████████████| 50/50 [00:19<00:00,  2.51it/s]
100%|██████████████████████████████████████████████████████████████████████████████████| 50/50 [00:11<00:00,  4.32it/s]
Total progress: 100%|██████████████████████████████████████████████████████████████████| 50/50 [00:13<00:00,  3.60it/s]
Save score as EXIF or PNG Info Chunk✔
Save score as tag (Windows Only)✔
Force CPU (Requires Custom Script Reload)✔
No function is added to the web ui interface

check where you put the extension. There should be a tools directory with files in there. If there's no files in tools, delete the extension and run git clone --recurse-submodules https://github.com/tsngo/stable-diffusion-webui-aesthetic-image-scorer extensions/aesthetic-image-scorer to pull both this repo and the tools submodule

Using M1 Mac. Also getting the error:
Aesthetic Image Scorer: Unable to load Windows tagging script
Repo clone was handled correctly.

The code is looking for tools/add_tags.py. Make sure the file is there.
If the file is there then try changing below in image_scorer.py:
try:     from tools.add_tags import tag_files except:     print("Aesthetic Image Scorer: Unable to load Windows tagging script")     tag_files = None
to:
from tools.add_tags import tag_files
let me know what the error message is

Ok that got it to progress to this new error:
Error loading script: image_scorer.py
Traceback (most recent call last):
  File "/stable-diffusion-webui/modules/scripts.py", line 157, in load_scripts
    compiled = compile(text, scriptfile.path, 'exec')
  File "/stable-diffusion-webui/extensions/aesthetic-image-scorer/scripts/image_scorer.py", line 23
    state_name = "sac+logos+ava1-l14-linearMSE.pth"
    ^^^^^^^^^^
SyntaxError: expected 'except' or 'finally' block

Did you remove try: line and also the except: the two lines afterwards? That error indicates try: is still there.
Replace this entire section
try:
    from tools.add_tags import tag_files
except:
    print("Aesthetic Image Scorer: Unable to load Windows tagging script")
    tag_files = None

with
from tools.add_tags import tag_files

This is how it should look afterwards:
if platform.system() == "Windows" and not is_installed("pywin32"):
    run_pip(f"install pywin32", "pywin32")

from tools.add_tags import tag_files

state_name = "sac+logos+ava1-l14-linearMSE.pth"

Try the latest version and also see https://github.com/tsngo/stable-diffusion-webui-aesthetic-image-scorer/issues/3 for some other issues that you may encounter

I'm still getting this error. I've deleted and pulled the latest a couple of times and same issue:
Aesthetic Image Scorer: Unable to load Windows tagging script from tools directory
M1 MacOS

Tried to load this and make the changes you mentioned but still experience the same
Aesthetic Image Scorer: Unable to load Windows tagging script
any updates on this issue?

I'm still getting the error regardless of installation method.


"Text Encoding" menu becomes truncated if more than one custom encoding file is present

If more than one custom encoding file is present, HexFiend will not show any custom encodings, nor the "Other..." option, in the Text Encoding menu.
That's really it. Steps to reproduce are to:

quit HexFiend if running
put two text encoding files in ~/Library/Application Support/com.ridiculousfish.HexFiend/Encodings
open HexFiend
click on Text Encoding menu
observe that no custom text encodings, nor the "Other..." menu item, are present.

As an aside, I love this feature. It has allowed me to use HexFiend as a disk editor for Apple II disk image files, which often store their ASCII with high bit set. (I'm sure this is an extremely common and important use case.) Is the feature documented anywhere outside of https://github.com/ridiculousfish/HexFiend/issues/95#issuecomment-397858715 ? I was happy to have stumbled across it.

Thanks to @ksherlock for the fix.


Updating cms-actions to support Silverstripe 5

We are currently preparing to migrate our site to Silverstripe 5 and checking & updating our dependencies.
However, our site is dependant on silverstripe-cms-actions and it has not yet been updated to support SS5.
Is such an update in the pipeline? If so, I would gladly offer my assistance in the process, once a procedure has been defined.

My plan is to migrate most of my websites over the summer so that's probably when I'll have a better look at it :-)
If you have time and energy to make a PR, I'd be more than happy to review it and start branch 1.4

I will certainly look into it. Do you plan on supporting SS4 as well as SS5 in 1.4 or on moving SS4 support into a separate branch?

if we can support both its better. as far as i can tell, it shouldn't not make a big difference from ss4 anyway.

released as 1.4.0, no major issues found so far


Syntax error - SELECT statement with SUM and CAST statement

Hello,
I just find this issue with abaplint (not a syntax issue in SAP). Looks like SUM / CAST combination in SELECT statements are unknown.

Best regards,
Dominik
P.S. Finding this issue in the abaplint repository check was not so easy because within abaplint result set the logic wasn't shown as failure directly. I only got this error line that method (also part of the class where SELECT statement was included) wasn't existing. So only class was identified as unknown only but no indicator why.

You can try to use this statement to reproduce in playground:
REPORT zfoobar.

    SELECT a~lgnum            AS lgnum,
           a~lgtyp            AS lgtyp,
           a~access_group     AS aisle,
           SUM( CAST( CAST( a~par_value AS NUMC ) AS INT8 ) ) AS loc_cnt_empty_total
      FROM /mycomp/stmval AS a
     WHERE a~lgnum = 'DE01'
       AND a~lgtyp = 'LG01'
       AND a~par_name = 'TEST'
     GROUP BY a~lgnum, a~lgtyp, a~access_group
      INTO CORRESPONDING FIELDS OF @ls_aisle_ptwy_index.

    ENDSELECT.

Thanks a lot!
Best regards,
Dominik

@dpuerner try again, 2.94.22 published

Thanks it's working - Thanks a lot for the very quick fix (as usual!) :-)


refactor(frontend): fix numerous details and UI elements

Closes #338
Changes

Users page:

Change edit icon to word.
Move edit icon from basic info.
Put paginator on bottom.
Reorder columns.
Fix gap between permissions heading and the list of permissions.

Change colours of top-level navigation to the DaisyUI primary theme.
Style more checkboxes with DaisyUI.
Vertical tabs use DaisyUI theme.
Remove dac details from release?

Users page

The text in the table (non headers) looks heavy (i.e. bold).. are they done at th rather than td?

The text in the table (non headers) looks heavy (i.e. bold).. are they done at th rather than td?

Ahh yes, I will unbold it. It is using td though.

Update: allow pressing the edit button even though the user cannot make changes - as the permission box already handles this.


Changes to support JEDI.

Various CMakeLists.txt have been modified to properly capture build vs install paths.  The previous was breaking in a JEDI configuration.

It would be good if @GEOS-ESM/chemistry-gatekeepers (or @mmanyin himself) to try this out. Make sure it works. It should (as it compiles) but always good to make sure we don't break GMI somehow.

This PR is zero diff for GMI


Improve Performance Test Coverage

Improve test coverage:

Add SimpleCov coverage
Create stories for missing tests
Run tests inside of a Rails application

The dev work side of things was addressed with #2140.
We'll revisit the story creation at a later time. Full coverage would require something akin to the multiverse tests and I'm hesitant to go that route.
As a team, we'll discuss strategy. I'm thinking we can focus the tests on specific areas of the agent (such as those that were recently improved to benefit GraphQL bulk load performance) and enforce an allowlist with .simplecov.


Text Search is  not working

I am trying to get multiple result from an ambiguous text search like food or grocery store by using GooglePlaceSearch but in suggestion i am not getting the search query , Here is the code snippet
query={{
key: 'my_api_key',
language: 'en',
location: this.props.lat+','+this.props.lng,        //specifing the current loc
radius:30000,        //search within 0km of your current loc
// origin:this.props.lat+','+this.props.lng,
}}
            GooglePlacesSearchQuery={{
                key: 'my_api_key',
                location: this.props.lat+','+this.props.lng,        //specifing the current loc
                radius:30000,        //search within 30km of your current loc

            }}

Did you get to make the radius work with the current location? To me it doesn't complain but it doesn't restrict it to the radius, it shows cities that are at 1000km from the end of the radius

@SyedTaha13 Can you please provide a full reproducible example of this by showing some code or better yet, linking to a snack?


Use Mocha & Karma for test cases

See #1036
This PR contains all the infrastructure needed:

Mocha is used for the test cases and as the runner.
mocha-sugar-free is used so that we can use the style of coding we are used to. For example it(), describe(), etc are globals in mocha, mocha-sugar-free lets you define them using require(). It also adds support for arrow functions, which mocha does not like. One extra feature I added which is convenient in jsdom is that you can exclude a test case from being run in the browser by passing options to mocha's it() function. Instead of having to blacklist/whitelist test files (what we did in the past)
Karma is used instead of selenium.
Browser tests are now being run both with and without a Worker. Previously tests were only run within a Worker. I created a karma plugin for this: karma-mocha-webworker
karma-browserify is used instead of having to generate a bundle ourselves. This plugin supports fast incremental rebuilds (which then trigger a fresh test run). Getting this to work actually took me the most time of everything because I had to fix a few issues in browserify and karma-browserify

So far I have converted one test case to mocha (test/jsdom/env.js). I made a small tool for myself which takes care of most of the conversion. Until all test cases have been converted, the mocha tests can live alongside the nodeunit tests (and the npm test commands and the travis build will run both).
Here is a successful travis build: https://travis-ci.org/Joris-van-der-Wel/jsdom/builds/107108618

Is the options object with async: true a mocha-sugar-free thing? I think mocha supports Promises natively, why not use those / make mocha-sugar-free also accept promises?
Will check the actual infrastructure commit later today.

Is the options object with async: true a mocha-sugar-free thing? I think mocha supports Promises natively, why not use those / make mocha-sugar-free also accept promises?

Yes that is a mocha-sugar-free thing. Normally mocha decides that your test is async by checking func.length > 0. I would rather make this explicit, therefor {async:true}.
Promises are also supported, simply return them (everything that works in mocha should work in mocha-sugar-free).
I have not used promises in env.js simply because we did not use them before. I could add them if that is a style we would like to follow, however it is a bit of a work because jsdom does not support them in its API:
it("explicit config.html, with overridden config.url", { async: true }, testCase => {
  env({
    html: "Hello",
    url: "http://example.com/",
    done(err, window) {
      t.ifError(err);
      t.equal(serializeDocument(window.document), "Hello");
      t.equal(window.location.href, "http://example.com/");
      t.equal(window.location.origin, "http://example.com");
      t.equal(window.location.search, "");
      t.equal(window.location.hash, "");
      testCase.done();
    }
  });
});

it("explicit config.html, with overridden config.url", () => {
  return new Promise( (resolve, reject) => {
    env({
      html: "Hello",
      url: "http://example.com/",
      done(err, window) {
        err ? reject(err) : resolve(window);
      }
    });
  }).then(window => {
    t.equal(serializeDocument(window.document), "Hello");
    t.equal(window.location.href, "http://example.com/");
    t.equal(window.location.origin, "http://example.com");
    t.equal(window.location.search, "");
    t.equal(window.location.hash, "");
  });
});

Why Chai? These days I tend to prefer the built-in assert. But I'm happy to be convinced if you think there's a killer feature we should be using.

It has more assertion functions like assert.isFunction(fn)
The plugins that chai has could also be useful:
Promises:
return assert.eventually.equal(Promise.resolve(2 + 2), 4, "This had better be true, eventually");

DOM:
expect(document.querySelector('main article')).to.have.attr('foo').match(/bar/)

And if we come up with useful assertions for jsdom, we could easily release them as a chai plugin, instead of using a hypothetical `test/util.js"
I do not feel strongly towards chai though, whatever works is fine with me.

Given that we haven't changed all our test names to follow the describe/it pattern, maybe use describe/specify instead?

Mocha uses the BDD interface by default (describe & it), it also offers the TDD interface (suite & test). Perhaps TDD would be more appropriate? We can also alias the functions to whatever we would like to use.

Regarding assertion library: those are fair arguments in favor of Chai; I'm happy to go with them. On the other hand, https://www.npmjs.com/package/power-assert looks pretty awesome....
Mocha allows specify as an alias of it, so I was just suggesting describe/specify.

Hmmm, nevermind about power-assert; I didn't realize it requires transforms of the test code. Chai it is!

Rebased & new build https://travis-ci.org/Joris-van-der-Wel/jsdom/builds/107894787

This looks good to me! It looks like the Travis failures might just be timeouts? The failures are on WPT which should not be an issue. I restarted the build to see.
Do you want to update CONTRIBUTING.md, or shall I? I can do it when I merge (probably the weekend).
Amazing stuff, really appreciated.

Thanks!,
Yea the travis build is suffering from timeouts a lot. Either saucelabs has no browsers available, or WPT timeouts somehow. Nice thing is that karma is able to retry starting the browser, it tries 3 times by default so that should help a bit with the suacelabs issue.
I can take care of CONTRIBUTION.md
btw, I just got test fixtures (html files, etc) to work in browers, will commit tommorow:
specify("should set namespace-related properties in HTML-SVG documents created by jsdom.env()", () => {
    return readTestFixture("jsdom/files/ns-svg.html").then(content => {
      const doc = jsdom.jsdom(content);
      testDocumentWithSVG(doc);
    });
  });

We could go even further if we use karma's proxy option (karma's http server will proxy a path to a different server).

btw, I just got test fixtures (html files, etc) to work in browers, will commit tommorow:

Oooh yesss that is super-great news. I was looking at all those tests that are skipped just because they use fixtures and being sad.

I'm going to work on the tweaks here and merge this now :)

This is merged!!!
One thing I noticed while updating Contributing.md is that it would be really nice if we could move the web platform tests to mocha. Then we can kind of lump all the nodeunit stuff as "these are old tests, run them only to check that you haven't broken everything".

Oh, and figuring out how to get the web platform tests running in the browser would be... amazing.


Node.prototype.emit is broken [Bug or Question]

Node.prototype.emit does not emit events globally any more.  It will only emit events to the node's children.  Is this a bug or intentional? (I didn't see any PR/Issue for this and the jsdoc still reads global)
Went in with this commit: https://github.com/Famous/engine/commit/35c6db1bd2dab6cbc5ea6b395d737630399e9df7#diff-20164165518dcfb14298730df6349123

This is intentional, sort of. I didn't mean for this to be a breaking change, I wanted to have a smoother deprecation towards the state that this is in now, but it seems to have gotten rushed in with the transform changes that landed in 0.6 and I missed the documentation.
The idea is that UIEvents flow upwards and custom events flow downwards, such that you can make a closed environment of events with controller classes listening to UI Events above, and view classes responding to state changes below. This is a fairly necessary change if famous is to be used as a component based application framework. If people are writing custom components and event dispatch is global by default, then suddenly we have to be concerned about namespace collisions. One cannot just drag in somebody else's code without figuring out what events they broadcast.
If you still want to broadcast events globally, the possibility exists by supplying the Dispatch with the path of a scene (which will just be a selector) like so:
Dispatch.dispatch('body', 'eventName', {hello: 5});

You can actually use that method to target any location in the scene graph, but I think the safest way to do that would be to make sure you are Dispatching to one of your components children and not to some random place.
There should definitely have been a more graceful switch to this design and I'm sorry for the inconvenience. I'll be issuing a PR ASAP to fix the documentation.

@DnMllr I couldn't agree more and that was the theory behind #319.  I guess I would have liked to see more of a generic emit(up)/broadcast(down) framework for passing events up the tree rather than restricting to DOM/GL events only. (I'm certain I can hack into that if necessary)
From my standpoint we can close this, however I would mention that all of your examples are now broken (or atleast the ones with events).  So from that standpoint, I will leave this for you to close if you wanted to track those changes.

Yes but why only the children of the node on the path?  Why not the node specified by path itself?  How is a non-UI Event to be sent only to a leaf node?

@steve-the-edwards  You can use Dispatch.emit to send to send events to a specific path in your subtree.

I think you were meaning Dispatch.dispatch(), yes?  Or Node.emit() both of which will send to children of that node, and not the node itself.  But that is what we are talking about here I believe :) : https://github.com/Famous/engine/issues/366

@steve-the-edwards you are correct.  I am going to close this for now and keep further conversation in #366


chore(deps): upgrade jenkins-x/jx3-versions to version 1.0.43

from: https://github.com/jenkins-x/lighthouse.git

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rawlingsj
You can assign the PR to them by writing /assign @rawlingsj in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Fixed link to demo page (note: also broken here http://www.abeautifulsit...

...e.net/blog/2011/02/jquery-minicolors-a-color-selector-for-input-controls/)

Still broken on the blog...  http://www.abeautifulsite.net/blog/2011/02/jquery-minicolors-a-color-selector-for-input-controls/ links to http://labs.abeautifulsite.net/projects/js/jquery/miniColors/

Fixed.


"-bedOut" argument does not work

It looks like both the documentation and the program are messed up here
In the documentation here for bedtools v2.26.0 getfasta, it lists the argument -bedOut:

Option
Description

-bedOut
Report extract sequences in a tab-delimited BED format instead of in FASTA format.

However, later in the documentation here, this is described as such:
Using the -tab option, the -fo output file will be tab-delimited instead of in FASTA format.
$ cat test.fa
>chr1
AAAAAAAACCCCCCCCCCCCCGCTACTGGGGGGGGGGGGGGGGGG

$ cat test.bed
chr1 5 10 myseq

$ bedtools getfasta -fi test.fa -bed test.bed -tab
chr1 5 10 AAACC

The -bedOut arg is not actually used here, so I am not sure what the syntax is supposed to be for this arg. Because, when I try to use it like this, it does not appear to work properly:
$ cat target_regions_head.bed
chr1	2985824	2985860
chr1	3102689	3103038
chr1	3160651	3160701
chr1	3301713	3301850
chr1	3313055	3313157
chr1	3319355	3319562
chr1	3321303	3321450
chr1	3322059	3322212
chr1	3327948	3329364
chr1	3331124	3331211

$ /local/apps/bedtools/2.26.0/bin/bedtools getfasta -fi "$hg19_fa" -bed tar
get_regions_head.bed -fo target_regions_head_fasta.bed -bedOut
Warning: the index file is older than the FASTA file.
chr1	2985824	2985860	chr1	3102689	3103038	chr1	3160651	3160701	chr1	3301713	3301850	chr1	3313055	3313157	chr1	3319355	3319562	chr1	3321303	3321450	chr1	3322059	3322212	chr1	3327948	3329364	chr1	3331124	3331211

$ /local/apps/bedtools/2.26.0/bin/bedtools getfasta -fi "$hg19_fa" -bed target_regions_head.bed -fo target_regions_head_fasta.bed -bedOut 1> /dev/null
Warning: the index file is older than the FASTA file.

$ head -3 target_regions_head_fasta.bed
TGCGATCCAAGGCGAGGGCGAGGAAGCTAGCCAAAA
TGACGGTGACGTTGTAAATAATATGTATGAGCCCAACCGGGACCTGCTGGCCAGCCACAGCGCGGAGGACGAGGCCGAGGACAGTGCCATGTCGCCCATCCCCGTGGGGCCACCGTCCCCCTTCCCCACCAGCGAGGACTTCACCCCCAAGGAGGGCTCGCCGTACGAGGCCCCTGTCTACATTCCTGAAGACATTCCGATCCCAGCAGACTTCGAGCTCCGAGAGTCCTCCATCCCAGGGGCTGGCCTGGGGGTCTGGGCCAAGAGGAAGATGGAAGCCGGGGAGAGGCTGGGCCCCTGCGTGGTGGTGCCCCGGGCGGCGGCAAAGGAGACAGACTTCGGATGGGAG
AAATACTGACGGACGTGGAAGTGTCGCCCCAGGAAGGCTGCATCACAAAG

Here, the fasta sequence has been written to the file, one per line, and the chromosome coordinates from the .bed file have written to stdout in a single line.
Its not supposed to do this, is it?

3 years later, I find this comment with the same issue. This is the exact thing I need but it doesn't work. Did you find an alternative?

Sorry, this has been fixed in master for awhile.  Just use -bedOut


Forbidden Response using SAS Token on a Single Table with Get-AzStorageTable

Using AzTable 2.04
Sas Token was created 3 different ways

Azure Portal - right Click on Table "Get SAS Token"
Powershell New-AzStorageTableSASToken
Azure Storage Explorer

All three SAS Tokens created on the Table ALONE fail
Using a SAS Token on the ENTIRE STORAGE ACCOUNT Always works
Code is:
Import-Module -Name Az.Storage
Import-Module -Name Az.Resources
Import-Module -Name AzTable
$AzStorageAccount = "storage_accountname"
$TableName = 'TableName'
$TableSasToken ="?st=2021-02-25T15%3A40%3A11Z&se=2022-01-01T04%3A59%3A00Z&sp=raud&sv=2018-03-28&tn=TableName&sig=mysig"
$StorageCtx = New-AzStorageContext -StorageAccountName $AzStorageAccount -SasToken $TableSasToken
$Table = Get-AzStorageTable -Name $tableName -Context $StorageCtx
throws
Forbidden
DEBUG: 1:04:10 PM - GetAzureStorageTableCommand end processing, Start 7 remote calls. Finish 7 remote calls. Elapsed time 1538702.92 ms. Client operation id: Azure-Storage-PowerShell-ee008953-9bf3-4536-9ad2-74a5b8742830.
DEBUG: AzureQoSEvent: CommandName - Get-AzStorageTable; IsSuccess - False; Duration - 00:00:01.8729751;; Exception - Microsoft.Azure.Cosmos.Table.StorageException: Forbidden
at Microsoft.WindowsAzure.Commands.Storage.Model.Contract.StorageTableManagement.DoesTableExist(CloudTable table, TableRequestOptions requestOptions, OperationContext operationContext)
at Microsoft.WindowsAzure.Commands.Storage.Table.Cmdlet.GetAzureStorageTableCommand.d__12.MoveNext()
at Microsoft.WindowsAzure.Commands.Storage.Table.Cmdlet.GetAzureStorageTableCommand.WriteTablesWithStorageContext(IEnumerable`1 tableList)
at Microsoft.WindowsAzure.Commands.Storage.Table.Cmdlet.GetAzureStorageTableCommand.ExecuteCmdlet()
at Microsoft.WindowsAzure.Commands.Utilities.Common.AzurePSCmdlet.ProcessRecord()
Request Information
RequestID:7b83f525-9002-0011-80a0-0b6c7f000000
RequestDate:Thu, 25 Feb 2021 18:04:08 GMT
StatusMessage:Forbidden
ErrorCode:
ErrorMessage:This request is not authorized to perform this operation.
RequestId:7b83f525-9002-0011-80a0-0b6c7f000000
Time:2021-02-25T18:04:08.4574077Z;
DEBUG: Finish sending metric.
DEBUG: 1:04:10 PM - GetAzureStorageTableCommand end processing.
hitting the URL in a browser works fine
using the same Table-level SAS Token against the REST API directly also works fine

hi @gwalkey , unfortunately, Get-AzStorageTable is cmdlet not support by the AzTables module, this is an official cmdlet from Az.Storage module and I suggest you to open an issue there.

Hey thanks, will do
I assumed this was your CMDLet
Too bad another MS piece of code is broken

One last comment, after the SAS token issue is fixed please change your line:
$Table = Get-AzStorageTable -Name $tableName -Context $StorageCtx

with
$Table = (Get-AzStorageTable -Name $tableName -Context $StorageCtx).CloudTable

yep. know about the sub-collection

Paulo, whats the deal with the Table module NOT being supported by MS?
Just trying to sell more Cosmos?

Update on documentation to account for the case where customer have a table sas token and how to obtain the CloudTable from it.
https://github.com/paulomarquesc/AzureRmStorageTable/blob/master/docs/README.md#troubleshooting


Make sure trigger mount is called in the correct order

I think this would solve the problem #582. Can someone please check it out?

Changes Unknown when pulling ab6530e8e2dd1d8cff7647eaada299e57ef8600e on AndreasHeintze:bug/mount-trigger-order into ** on muut:dev**.

Changes Unknown when pulling 680a56aa836deb819aebb44d1e2d3b80cbc5e6ae on AndreasHeintze:bug/mount-trigger-order into ** on muut:dev**.

It looks good but avoid to add public properties to riot. Can't you use a private variable to store the tags? And once the mount get triggered you should clean it to avoid memory leaks

#GianlucaGuarini I'll fix it later today.

Sorry, I did run out of time today. I will take a look at it tomorrow.

@GianlucaGuarini How can I see what those merge conflicts are?

good article, I would have just merged upstream into my working branch, but better to create a new one.

@rsbondi Well I tried, but I'm not doing so well here. Now I got a whole bunch of commits and there are still conflicts!

Coverage increased (+0.06%) to 91.69% when pulling cffdcb6573bb7f2e5e832acb74ccc87dbd842af3 on AndreasHeintze:bug/mount-trigger-order into e3e0f1481a0dc4d537931906d9c809df6d2a9957 on muut:dev.

How do I flatten this PR to only one commit? I've tried doing this: http://blog.steveklabnik.com/posts/2012-11-08-how-to-squash-commits-in-a-github-pull-request
..but it fails:
´´´
Andreas-MBP:riotjs andreasheintze$ git rebase -i upstream/dev
The previous cherry-pick is now empty, possibly due to conflict resolution.
If you wish to commit it anyway, use:
git commit --allow-empty

Otherwise, please use 'git reset'
rebase in progress; onto e3e0f14
You are currently rebasing branch 'bug/mount-trigger-order' on 'e3e0f14'.
nothing to commit, working directory clean
Could not apply 923c6f003b7673bc081d4baabb375a760d13dd64... Symple syntax checker for tag files
´´´

@AndreasHeintze Hi! How about recreate a new one as @rsbondi said? It seems easier than struggling on this PR.

@cognitom That is probably easier, I just wanted to learn how to do it, but if I don't find a way to fix this PR I will make a new one. Will do it later, got to go now.

thanks @AndreasHeintze for your contribute, I've solved the issue in a more elegant way using the riot.observable events.

Ok, can I see somewhere how you solved it? Does it solve this issue too? https://github.com/muut/riotjs/issues/334
I think @luffs issue needs to be taken into account too.

@AndreasHeintze see https://github.com/muut/riotjs/commit/9ecb46bb211142127766cab47f717310999fe689#diff-0204a5bfd4606caeeaf90c68f376aa65R106

Thanks! I like it.
What is the purpose of the change in the creation of _id?


Recently used folders in pane on left

What feature or improvement do you think would benefit Files?
Like Windows Explorer, it would be great to have recently and frequently folders in the Quick Access column on the left pane.
Requirements

This proposal will accomplish X
This proposal will accomplish Y
This proposal will accomplish Z

Files Version
<IP_ADDRESS>
Windows Version
10.0.19045.3448
Comments
No response

Thank you for your feedback, we don't have plans to show recent folders in the sidebar but they can be found on the home page.


Sentence Splitting Fails on fancy "s (“,”) and ()[]'s, too.

This is considered one sentence:
As recently as 1996, Pennsylvania was what I call the “tipping-point state.” Bill Clinton carried 21 states and the District of
Columbia by wider margins than he did Pennsylvania, giving him 256 electoral votes.

This is considered two sentences:
As recently as 1996, Pennsylvania was what I call the "tipping-point state." Bill Clinton carried 21 states and the District of
Columbia by wider margins than he did Pennsylvania, giving him 256 electoral votes.

The difference is the unicode quotations. Also, the splitting fails on a period then other symbols other than " and ', such as
{}, too.
In the regex for sentence splitting,
(?<!\d)
(?<![A-Z]\.)
(?<!\.[a-z]\.)
(?<!\.\.\.)
(?<!etc\.)
(?<![Mm]r\.)
(?<![Pp]rof\.)
(?<![Dd]r\.)
(?<![Mm]rs\.)
(?<![Mm]s\.)
(?<![Mm]z\.)
(?<[Mm]me\.)
(?:(?<=[.!?]) | (?<=[.!?]['"]))  [\s]+?  (?=[\S])

the problem is in the last line: ['"] should also include other things to consider.
Maybe it should be replaced with [\W] (non alpha-numeric?).

I should use unicodedata to fix this.
http://docs.python.org/library/unicodedata.html
http://www.peterbe.com/plog/unicode-to-ascii
For everything that is not ascii, replace with the unicodedata replacement IF there is one (for some strings, like ∫, there is no ascii equivalent).


Syntax error on _SAPS4_02 Schema Data Definition

Hi there,
I was trying to create the Audit policy based on _SAPS4_02 Schema Data Definition for a S/4HANA system with HANA 2.0 Rev 64
Replaced <SAPABAP1> with SAPHANADB as schema.
I get syntax error , SAP DBTech JDBC: [257]: sql syntax error: incorrect syntax near "ON": line 32 col 3 (at pos 701)
When I tried to recreate the audit policy via HANA Cockpit Auditing functionality , it seems that ON SCHEMA <SAPABAP1> does not appear to be a valid option.
Thanks
L2R

Hello L2R,
this syntax error indicates that your SAP HANA Version does not support this policy definition. The minimum version should be SAP HANA 2.0 SPS07.
best regards
jane

Thank you for the update. I did not notice any comments about HANA 2.0 SP07 as prerequisites for that policy.

Hi L2R,
please review the comments in
2_s4hana_hana_audit_policy_recommended.sql
directly above the definition of "_SAPS4_02 Schema Data Definition"  - line 50.
bst regards
jane


Add option to allow app directory globs from packages

For things like automatic loading of translation files, it may be nice to allow globs of files even when they are not within the package boundary. Right now the following fails if it's part of a module:
const en = fs.readFileSync(path.resolve('locales/en.json');

In my specific case, I was also interested in having an installed module be able to preserve a directory by doing:
fs.statSync(path.resolve('app-data-dir'));

But there may be more examples, like automatically loaded config files from the app/working directory.

even when they are not within the package boundary

I never really understood why the package boundary was necessary. This was something @guybedford was very keen on adding. It also matches the behavior of ncc.
However, I don't understand how your two examples are outside the package boundary? Perhaps you could add a unit test to show it fail for your use case?

The actual case where I ran into it was:
// HTTP handler for dynamic asset assembly.
// It reads `.next/chunks`.
node_modules/next-dynamic-bundle/dist/handler.js

// The actual chunk data
.next/chunks/*.js

NFT recognizes the reads in handler.js but discards them because they are for a directory outside of the package.
Code for the handler where it stats the chunk dir (which should cause a glob include of all files since it's a directory): https://github.com/azukaru/progressive-fetching/blob/bcc665cbf30934d3570c20d18c7860c658fe40b2/packages/next-dynamic-bundle/src/handler.ts#L20

The line you linked to is reading a dynamic path fs.statSync(chunkDir).
Can you try changing it to fs.statSync(path.resolve('.next', 'static', 'chunks')) and see if that works?

I tried a couple of variants. Debugging with the CLI directly showed that the glob is being done but then it is followed by "Ignoring X because it is outside of the package". So I'm pretty sure it's finding the assets successfully.

Have you tried setting the base option?
https://github.com/zeit/node-file-trace/blob/master/readme.md#base

The issue is that pkgBase is dynamic and afaict isn't the same as base. The condition I'm running into is this one: https://github.com/zeit/node-file-trace/blob/94a64aae0a4be8368717852de805f3627381a8e0/src/analyze.js#L735-L741

That condition can be changed to something more appropriate. The main thing it's trying to protect against is an invalid analysis inadvertantly globbing all of node_modules and defeating the point of tracing. So it's very much about balancing those things.
I'm not sure the best way to extend / adjust the logic here, it may come down to some trial and error on the analysis paths as well.

Suggestion for a revised logic: If the path starts with the overall base and the remainder doesn't include /node_modules/ at all, allow it? In addition maybe: The remainder has to start with a path segment that isn't node_modules. That should prevent accidentally including huge directory trees.

The specific case to be weary of is base/node_modules/pkg/node_modules/dep having a statement like fs.readFile(__dirname + '../../' + unknown) causing a glob of base/node_modules/pkg/.
The node_modules parsing itself is kind of a lazy way to do the package boundary - a package.json scope check could be another way to do the same thing. But the issue remains either way I think?
My concern is any logic to relax this will let the problematic case slip through.

I think the specific hole I'd like to punch is: Allow a glob relative to the project directory that "definitely" (minus symlinks) doesn't include node_modules. Your example should be rejected because the wildcard expression would include node_modules in the glob remainder.
To elaborate on my proposal, in semi pseudo code:
if (pkgBase) {
   const nodeModulesBase = id.substr(0, id.indexOf(path.sep + 'node_modules')) + path.sep + 'node_modules' + path.sep;
   if (!assetPath.startsWith(nodeModulesBase)) {
     // Allow after all if it's an app-level glob
     const appSubGlob = assetPath.substr(base.length + 1);
     const segments = appSubGlob.split(path.sep);
     if (
       !assetPath.startsWith(base) ||
       segments.length < 2 ||
       isGlobOrNodeModules(segments[0])
     ) {
       return false;
     }
   }
 }

Yes that sounds like it would work! If we can guarantee that whatever is being dived into is not a node_modules folder, then it must have been a folder the package knows exists below itself so is likely the correct intention.
Note the other thing to still be weary of is backtracking below the base (security risk), but that is still covered if there is a change of logic to this part since that check happens earlier I believe.
You happy to go ahead with a PR there further?

May find some time to get a PR together for this. Thanks for the buddy-check on the overall approach!

I also would like this feature. We Blitz framework code (inside node_modules) is reading config files in the app directory at runtime. Currently that is not detected by NFT and the config files are eliminated.


Add x64 support

Add x64 hook support.
Test in local.
The picture below is keepassXC x64

老哥你改字体的代码好稳，我加了个64位的hook就ok了

Because x64 inline hook is complicated, I'm planning supports only IAT and EAT hook, or using a third-party hook library such as PolyHook or Detours.
So your pull request will not be merged. But your winmm.asm is great, which uses macro to reduce duplicate code. I will use it.
Thanks for your work!


Accentuated chars in the title header aren't supported

:lady_beetle: Describe the bug

If I url encode accents (like é ç à è) in the title header they won't be rendered in the app.
Example: title "accentué" will look like "accentu%C3%A9" in the Android and web app.
:computer: Components impacted
Android and web app, probably iOS too but I don't have a device.
:bulb: Screenshots and/or logs

:crystal_ball: Additional context
It's not major, just a nice to have.
Thanks for the app, it's great!

ntfy supports utf-8 in headers. It's likely your library or language or OS or config not supporting it.
Please note that you can encode the headers with RFC 2047 if you don't get it to work and that'll always work. Or you can pass it via query param.
See the blue info box here https://docs.ntfy.sh/publish/#list-of-all-parameters as well as which query param to use if you choose to use that.
Feel free to drop by discord or matrix to ask for more help.


Warning: TypeError: 0, _$$_REQUIRE(_dependencyMap6ve-css-interop/jsx-runtime").jsx is not a function (it is undefined)

I've only initialized @rn-primitives/alert-dialog": "^1.1.0 in react-native cli project on v0.76.1. I no longer can run my app due to these errors.
(NOBRIDGE) ERROR  Warning: TypeError: 0, _$$_REQUIRE(_dependencyMap6ve-css-interop/jsx-runtime").jsx is not a function (it is undefined).
(NOBRIDGE) ERROR  Uncaught error: [TypeError: 0, _$$_REQUIRE(_dependencyMap6ve-css-interop/jsx-runtime").jsx is not a function (it is undefined)] {"componentStack": "
I'm using NativeWind for styling. My project was running fine until I added alert-dialog and I can no longer build it. I've tried downgrading, changing my nativewind version, reconfiguring my babel.config.js, cleaning and rebuilding. Also tried removing my themeContext etc. It's still giving me the same error. I think it'll give the same error for some other components too.

Hey @shikhinee, can you provide a minimal reproduction repo?


feat: integration of easysbc solutions

Prerequisites

[X] I have searched for existing issues that already include this feature request, without success.

Describe the Feature Request
integration of easysbc solutions in the manner of futbin solutions for SBCs
Describe the Use Case
solving SBCs
Describe Preferred Solution
No response
Describe Alternatives
No response
Additional Information
https://easysbc.io/

Is a feature of fut-enhancer not auto-buyer. Suggest moving this issue there.


Add code generator for the artifacts listing required by laxar

Move the code generation part of jpommerening/laxar-loader to this module.
Use callbacks for creating require calls (or actually reading the required module).

Upgrade info

The assetResolver API has changed. Instead of offering one method per artifact type, it now has two generic methods resolveAssets and resolveThemedAssets which take an artifact of any type, a list of assets to resolve and, in case of the themed version, a theme-artifact.

The artifactCollector now adds a field category which is used by the assetResolver to pick the correct theme sub-directory.

The artifactListing and serialize APIs have been added and can be used to build the artifact listing required by LaxarJS v2.

released with v0.3.0


importing the firebase-dynamic-link package produce error

If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "https": require.resolve("https-browserify") }'
- install 'https-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "https": false }

@kalpiBansal I'm not familiar with polyfill, so please can you provide an example. and please tell me if you have any possible solution.

`
./node_modules/firebase-dynamic-links/lib/firebase-dynamic-links.js:13:16-32 - Error: Module not found: Error: Can't resolve 'https' in '/Users/warren/Documents/dev/app-linkmate/node_modules/firebase-dynamic-links/lib'
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.
If you want to include a polyfill, you need to:
- add a fallback 'resolve.fallback: { "https": require.resolve("https-browserify") }'
- install 'https-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
resolve.fallback: { "https": false }
`
Build output when added to an angular project

I'm getting the same error in a React project, it's not to do with either Angular or React.  The error is because you're importing something from 'https' (here), but that package isn't global to everyone's environment and it's not being bundled in when you package your project for distribution.
I believe the solution would be to either explicitly add 'https' to the dependencies or to add a webpack config file to your project and (as the error above says) define a fallback to use when the package is installed.  I'll fork and give this a shot myself.

@evellior thanks for the clarification, I will try to solve the problem as soon as I find time.
meanwhile, please feel free to open a PR if you solved the problem.

@kalpiBansal can you please check if #8 solves the problem?


Replace page using morphing to avoid flicker

I'm trying to port PWA web-app to turbo native, main problem is bottom navbar which is flickering (actually whole page is flickering when using data-action="replace") as navigation is happening between different paths.
I believe this could be solved with morphing, but turbo is currently lacking way to replace page without navigation/reload/flicker.
Happy to contribute this change (assuming my understanding is correct)

This is already supported. See the demo site. Include this in your <head>:
<meta name="turbo-refresh-method" content="morph">

See demo app example:


KServe performance tutorial failed

Describe the bug
Running tutorial failed. Report had no output. Logs have
% kubectl logs job.batch/default-1-job
time=2023-02-27 18:25:23 level=info msg=task 1: run: started
time=2023-02-27 18:25:23 level=info msg=task 1: run: completed
time=2023-02-27 18:25:23 level=info msg=task 2: http: started
time=2023-02-27 18:25:23 level=error msg=fortio failed stack-trace=below ...
::Trace:: lookup sklearn-irisv2.default.svc.cluster.local on <IP_ADDRESS>:53: no such host
time=2023-02-27 18:25:23 level=error msg=failed to get results since fortio run was aborted
time=2023-02-27 18:25:23 level=error msg=task 2: http: failure
Error: lookup sklearn-irisv2.default.svc.cluster.local on <IP_ADDRESS>:53: no such host

Referenced service seems to exist.
To Reproduce
Run tutorial.
Additional context
On retry it was observed that the inference service took > 5 minutes to become ready. It may be that the readiness check failed to cause experiment to fail.

It appears that that the basic readiness check works. I set the timeout to 10s and increased the logging. I repeatedly see:
time=2023-02-27 21:35:44 level=trace msg=looking for resource (serving.kserve.io/v1beta1) inferenceservices: sklearn-irisv2 in namespace default
time=2023-02-27 21:35:44 level=trace msg=looking for condition: Ready
time=2023-02-27 21:35:44 level=error msg=condition status not True
followed by
time=2023-02-27 21:35:44 level=error msg=task 1: ready: failure

iter8 k report correctly identifies a failed task/experiment:
Experiment summary:
*******************

  Experiment completed: false
  No task failures: false
  Total number of tasks: 4
  Number of completed tasks: 0
  Number of completed loops: 1

Copy of slack comment:

I wonder if Fortio's

  -allow-initial-errors
        Allow and don't abort on initial warmup errors

should be exposed as a parameter in the http task. This might be a simple "fix" worth trying ... of course, more "warmup" behavior can also be defined in the task if this is insufficient.


System.Private.CoreLib: Could not load file or assembly 'Microsoft.AspNetCore.Http.Abstractions'

C:\Users\erics\AppData\Roaming\npm
--<EMAIL_ADDRESS> +--<EMAIL_ADDRESS> | --<EMAIL_ADDRESS>|   --<EMAIL_ADDRESS> --<EMAIL_ADDRESS>`--<EMAIL_ADDRESS>

I am also seeing this when trying to debug a new Functions project from a new install of VS2017 15.5.2.

I am getting the same error in VS2017 15.5.2 It seems related to the fact that the cli loads a certain version of the dll then you try to reference a different version. I am having a similar issue with WindowsAzure.Storage
Seems related to https://github.com/Azure/azure-functions-cli/issues/335 and https://github.com/Azure/azure-functions-cli/issues/253

Also had same issue with WindowsAzure.Storage 9.0 and 8.7. Had to downgrade to 8.6. Thanks for the workaround evgeny-n !
Did not have this issue with Azure functions V1 (.Net Framework). Occurred when I ported to V2 (.Net Standard 2.0)

Thanks @evgeny-n for your workaround. Updating the Azure Functions and Web Jobs Tools to the latest version might also help.

Am getting same error message when running azure functions project locally.  Targetting .net standard 2.0.
Using .net sdk functions 1.0.6 and it worked ok initially for the "Function1" freebie function.  Then added reference to Microsoft.AspNetCore.Authentication.JwtBearer and immediately get this error.
I also tried installing System.IdentityModel.Tokens.Jwt package [i just want to parse a jwt].  this works in local debugging but fails when I publish it to azure.

Just for fun, I get it on the actual Azure portal. The code is merely from the Azure samples.

I am getting something similar, but for cosmosdb trigger functions.
System.Private.CoreLib: One or more errors occurred. (Unable to load DLL 'Microsoft.Azure.Documents.ServiceInterop.dll': The specified module could not be found. (Exception from HRESULT: 0x8007007E)). Microsoft.Azure.DocumentDB.Core: Unable to load DLL 'Microsoft.Azure.Documents.ServiceInterop.dll': The specified module could not be found. (Exception from HRESULT: 0x8007007E)

The similar issue for the prerelease version of Azure.KeyVault
Could not load file or assembly 'Microsoft.Azure.KeyVault, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. Could not find or load a specific file. (Exception from HRESULT: 0x80131621)

Getting the same thing, after adding Microsoft.AspNetCore.Authentication.JwtBearer. VS 2017 15.7.1. Azure Functions and Web Jobs Tools 15.0.40502.0.

Same issue with <PackageReference Include="Microsoft.Extensions.Options" Version="2.0.1" /> down graded to <PackageReference Include="Microsoft.Extensions.Options" Version="2.0.0" /> to resolve the issue

Updating "Azure Functions and Web Jobs Tools" extension fixed the issue for me, although I am only trying to get it to run locally.
Steps:
In Visual Studio
Go to Tools > Extensions and Updates
In Extensions and Updates dialog, expand Updates > Visual Studio Marketplace
Choose Azure Functions and Web Jobs Tools
Select Update
Restart VS

@apostrophe , fixed what exactly? I have the latest and greatest of what's in the "Extensions and Updates" and I still see this when trying to use anything above EF Core 2.0
I have 15.0.40502.0 installed of "Azure Functions and Web Jobs Tools"

I have 15.0.40502.0 installed of "Azure Functions and Web Jobs Tools"

That's the same version I am have, but I'm not using EF Core.

I was able to get further by downgrading some of the nuget references to be the same as used by the tools, but  only got so far down that path. Not able to reference EF Core without getting similar errors.

Had this issue, got the updates (VS extension then Nuget then CLI tool then Storage emulator), issue stopped and function was working. Re-opened the project later in the evening, issue is back. No further update possible this time. Puzzling...

I have not had this issue for some time now. Currently running on v36 all nugets on latested(EF, AzStorage and AzEventHub).
Using VS 2017 as well.
@daviburg
Double check the host version. Maybe the AzHost is npm installed global????

A significant number of changes relating to Functions V2 GA have happened since this issue was originally opened and the underlying issue has been addressed. Anyone who is still experiencing issues should feel free to open separate issues.


Too many result sets

use cmd = Db.CreateCommand<"begin;
    update section set ord = ord + 1 where deck_id = @deck_id and ord >= @order;
    update section set ord = @order where id = @sectionId;
    commit;">(cs)

Npgsql supposedly reports there are 4 statements in this command, but neither begin nor end should have a result set of their own even though there is no problem at runtime.
https://github.com/npgsql/npgsql/issues/2873

Hmm well there are 4 statements, but only 3 queries :(
Von meinem iPhone gesendet

Am 27.02.2020 um 7:53 AM schrieb kerams<EMAIL_ADDRESS>
use cmd = Db.CreateCommand<"begin;
update section set ord = ord + 1 where deck_id = @deck_id and ord >= @order;
update section set ord = @order where id = @sectionId;
commit;">(cs)
Npgsql supposedly reports there are 4 statements in this command, but neither begin nor end should have a result set of their own even though there is no problem at runtime.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I did some digging in the bowels of Npgsql and it looks like Postgres gives no indication about the type of individual statements when getting schema. So while this issue is mildly annoying, I don't see a solution that would be worth the trouble...

Actually, there is one dirty thing we could do - use a simple regex to filter out 'control' statements and such.

   Might give feedback to the nice npgsql folks. They have been helpful in past when we’ve raised things that would be useful to have
Von meinem iPhone gesendet

Am 29.02.2020 um 9:09 AM schrieb kerams<EMAIL_ADDRESS>
Actually, there is one dirty thing we could do - use a simple regex to filter out 'control' statements and such.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Unfortunately, this is a limitation of Postgres itself :/. More in the comments https://github.com/npgsql/npgsql/issues/2873


check for suggestedActions length incorrect

When there was no text then suggestedActions check for content was incorrect.
Fixes #470
correctly check length of actions array.

Pull Request Test Coverage Report for Build 1022

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 69.478%

Totals

Change from base Build 1018:
0.01%

Covered Lines:
3511

Relevant Lines:
4838

💛  - Coveralls


feat: improve useSendDetails loading, early bail and debounce

Description
Started as a simple fix for early bailing out of estimating fees when funds are insufficient (see this sentry issue, but noticed our debouncing in useSendFees is borked out of sanity i.e:

Have a balance of 0.6 ETH
Type 0.5 which is happy
Now either clear the 0. part, or add a 9 at the end which would result in insufficient balance. The insufficient balance will be displayed for a second or so, only for
the previously debounced promise to settle, its side effects run, and the send modal being wrongly in a valid balance state.

The root cause is that while async functions may look like good ol' functions from a syntactic sugar point of view, they are promises, and promises are not cancellable.
Us calling .cancel() on the debounced "function" doesn't really do anything as far as canceling promises goes. This could be solved by using bluebird or other fancy APIs like AbortController
which both come with their set of issues (package and API overhead for the former, boilerplaty and limited to the fetch API for the latter).
Fixed by going on a proper refactor of useSendFees which:

decouples the network logic and the input change logic. handleInputChange's sole responsibility is now setting values in the form, and the newly introduced query handles fetching fees and error-handling
things are still debounced, though in a more simple fashion, by debouncing the amountCryptoPrecision input

And obviously fixes the issue this initially went for, which is early bailing out of estimating fees when there is insufficient balance.

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #6877
Risk

High Risk PRs Require 2 approvals

What protocols, transaction types or contract interactions might be affected by this PR?

Testing

Loading states are still happy i.e is still reflected immediately on user input
Insufficient funds checks are still happy:

Insufficient funds (either for token or native asset sends) are still surfaced with a "Insufficient funds" error
Token sends without enough funds for gas are still surfaced with a "Not enough native token for gas" error

Send max is still happy:

Token sends set the absolute max
Native asset still do fees deduction to set the actual max

Debouncing is actually working, and previous requests completing do not end up in a wrong state i.e

Having an initial input which settles to a happy state (enough funds), but a subsequent one that completes to a fail (not enough funds) correctly settles to a fail state visually, without a flash of wrong state
Having an initial input which settles to a sad state (not enough funds), but a subsequent one that completes to a happy (enough funds) state correctly settles to a happy state visually, without a flash of wrong state

Engineering

Input when not enough funds does not trigger an estimate request
☝🏽

Operations

Screenshots (if applicable)

Having an initial input which settles to a happy state (enough funds), but a subsequent one that completes to a fail (not enough funds) correctly settles to a fail state visually, without a flash of wrong state

https://github.com/shapeshift/web/assets/17035424/863a9398-7db4-4642-9aaa-b796951b6624

Having an initial input which settles to a sad state (not enough funds), but a subsequent one that completes to a happy (enough funds) state correctly settles to a happy state visually, without a flash of wrong state

https://github.com/shapeshift/web/assets/17035424/a946cea0-ef36-493d-9311-6984b773e441

Input when not enough funds does not trigger an estimate request

https://github.com/shapeshift/web/assets/17035424/d49a057b-c312-4d52-acf4-99a3db4f3ffe

Insufficient funds (either for token or native asset sends) are still reflected as "Insufficient funds"

Token sends without enough funds for gas are still surfaced with a "Not enough native token for gas" error

Token sends set the absolute max

Native asset still do fees deduction to set the actual max

Testing notes
✅ Send non-native max
✅ Send not max

🚫 NaN appearing when attempting to enter fiat amount (confirmed not in prod):
https://github.com/shapeshift/web/assets/125113430/72f0a8c9-07e1-40fe-b9df-f7cd7954a45a

🚫 Insufficient Funds appears when clearing the fiat amount:

🚫 Allows user to send 0 amount:
https://github.com/shapeshift/web/assets/125113430/93a5688e-f337-4fa1-8e3e-53528aa6899c

🚫 Allows user to enter negative amount, displays Insufficient Funds:

🚫 Unable to send max on native token:

Stellar testing @woodenfurniture, ty!

🚫 NaN appearing when attempting to enter fiat amount (confirmed not in prod)
🚫 Insufficient Funds appears when clearing the fiat amount

Both fixed in 271e1adb9e - https://jam.dev/c/d54a8ec9-29b1-46fc-808c-2adcc72d220c

🚫 Allows user to send 0 amount
🚫 Allows user to enter negative amount, displays Insufficient Funds

Both are also in develop/prod - quite wary about introducing more issues here so stacked out on top of this instead: https://github.com/shapeshift/web/pull/6930. See this Jam for the fix this PR introduces: https://jam.dev/c/542c2419-7c78-4339-a27e-77f63666e490 though I'll still need to test the send max negative amounts there.
My main concern is that negative amounts can sometimes be present when clicking max, fee deduction is expected (i.e native asset) but fees are greater than amount, this is valid and expected and we could break the error messaging here).

🚫 Unable to send max on native token

That one is specific to Arb fees which are always derp and very volatile by nature and isn't specific to this PR, see jam against prod (note, I monkey patched the staleTime of fees estimation to 2s for ease of testing), though this PR made the behavior more obvious (and actually correct, vs. the infinite loading state on prod):
https://jam.dev/c/b01da49b-485d-4f43-b6b5-59982a609d92
If you test with another native asset, things should be consistently happy:
https://jam.dev/c/a8121673-3cf4-43df-8fa0-a9987b09a256


The awscli tests for virtual style paths (dnsmasq fix)

The dnsmasq will now accept wildcard bucket name entries.
The container is spawned by

docker run -e "SERVER_ENDPOINT=<IP_ADDRESS>:9000" \ -e "DOMAIN=minio.com"  -e "ACCESS_KEY=minio" \ -e "SECRET_KEY=**REDACTED**" -e "ENABLE_HTTPS=0" \ -e "ENABLE_VIRTUAL_STYLE=1" minio/mint awscli

The DOMAIN provided in the env will be resolved to the host ip in SERVER_ENDPOINT
Fixes: #213

@nitisht @balamurugana  Changes updated.

@balamurugana  @nitisht  The changes are done .

@balamurugana  @nitisht  the changes are done

@balamurugana  The changes are done. Please check

LGTM.  Please resolve conflicts.


docs: openstack documentation preparation fixes

What is the purpose of this pull request?: Fix documentation broken links
What issue does this pull request address?: Fixes ##323
Notes for reviewers to consider: @renmak had some comments on this, @larryrensing noticed it originally.
Specific reviewers for pull request: @renmak @larryrensing

completed changes @wilkers-steve made.


Distinguish Between Piral Dev and Piral Instance Dev

Prerequisites

[x] Can you reproduce the problem in a MWE?
[x] Are you running the latest version?
[x] Did you perform a search in the issues?

Environment Details and Version
Using https://docs.piral.io.
Description
The general documentation includes a "Development" section, which, unfortunately, has nothing to do with the expected "development of a Piral instance", but rather with the "development of Piral itself".
We need to introduce some terminology to make clear what is what.
Steps to Reproduce

Open docs.piral.io
Go to the first tile - documentation
See "Development" noted in the sidebar / close to the bottom

Expected behavior
The development of Piral itself should maybe be constraint to the GitHub repository, e.g., the README files.
The following strategy could help:

The main README.md (root): informs about repo structure, important pages, services, ... - very quick intro
The README.md of "src" includes the development guide for Piral; referring also to the sub-README files
The README.md of "src/packages" lists these packages, their intentions, and whats necessary for their development + vision
The README.md of "src/pages" again, explains these repositories, their intentions, and vision
Same for README.md of "src/demos"
The README.md of specific packages (e.g., piral-vue) has documentation as accessible as possible for people coming to NPM

Actual behavior
Right now it is quite a mix. Not very user friendly.
Possible Origin / Solution
The documentation is growing and the organization needs to be more strict.
cc @lschoett  @iotcloudarchitect

Landed in  devel.


Feat: Shortcut to clear search bar

Resolves #241

To get the new feature working, run ALGCore initializeGlobalKeybinds in the workspace after pulling.


Observation shape in the custom marl environment

Dear Author,
I want to use the mappo algorithm in the custom multi-agent environment.
My observation consists of two parts: agent_state and global_state. Specifically:
The dimensions of agent_state are given by H1 × W1 × C1.
The dimensions of global_state are given by H2 × W2 × C2.
I noticed that the parameters of agent_obs_shape and global_obs_shape in MAVAC can be either integers or sequences:

https://github.com/opendilab/DI-engine/blob/7f951592a3d7b39c8fda44081fc40002f0ee27fc/ding/model/template/mavac.py#L43-L45

So, I set the observation shape in the config file:
model=dict(
            action_space='continuous',
            agent_num=n_agent,
            agent_obs_shape=[100, 100, 6],
            global_obs_shape=[100, 100, 6],
            action_shape=2,
        ),

However, the error:
Traceback (most recent call last):
  File "/data/projects/20240708/DI-engine/custom_procthor_env_hierarchy_top_view_multi_agent/procthor_env_ppo_subprocess_hierarchy.py", line 215, in <module>
    main()
  File "/data/projects/20240708/DI-engine/custom_procthor_env_hierarchy_top_view_multi_agent/procthor_env_ppo_subprocess_hierarchy.py", line 202, in main
    model = MAVAC(**cfg.policy.model)
  File "/data/projects/20240708/DI-engine/ding/model/template/mavac.py", line 84, in __init__
    nn.Linear(global_obs_shape, critic_head_hidden_size), activation,
  File "/home/wj/anaconda3/envs/di/lib/python3.9/site-packages/torch/nn/modules/linear.py", line 96, in __init__
    self.weight = Parameter(torch.empty((out_features, in_features), **factory_kwargs))
TypeError: empty(): argument 'size' must be tuple of SymInts, but found element of type tuple at pos 2

occurred in:

https://github.com/opendilab/DI-engine/blob/7f951592a3d7b39c8fda44081fc40002f0ee27fc/ding/model/template/mavac.py#L83-L88

when I use the observation shape mentioned above.
How can I utilize custom networks, or is there an alternative method for handling sequence-type inputs other than flattening the array to 1D?
Thank you for your assistance!
Best regards,
Juan

You can custom your own networks, and pass it into the input argument of DI-engine's policy. Here is the concrete implementation of this argument.

Hi, @PaParaZz1
Thank you for your reply.
Is there a plan to set encoder parameters in MAVAC that are similar to how it’s done in VAC?
https://github.com/opendilab/DI-engine/blob/7f951592a3d7b39c8fda44081fc40002f0ee27fc/ding/model/template/vac.py#L43
I think this would be convenient for network customization.
Thanks,
Juan

We will add this argument like that in VAC in a week.

Thanks, that’s good news! I’m looking forward to the update.

We have added this feature in the above commit.


Custom view doesn't show unique and foreign keys

System information:

Operating system (distribution) and version Win10
DBeaver version 21.0 EA

Connection specification:

Database name and version Vertica

Steps to reproduce, if exist:
Set connection view>HIde folders or show only schemas and tables
Expected: Unique keys and foreifn keys remain visible in table properties
Actual result: Columns and DDL tab is visible only, like in Simple view.

I cannot reproduce it for the Hide folders option. As for the Show only schemas and tables, it is expected behavior.


refactor: clean up unnecessary async functions

The readFile() and readConfig() helpers do not need to be async.
Doing so just adds complexity to their call sites.

Could you also rename the function to readFileSync?

Could you also rename the function to readFileSync?

I would push back on this. Why is this important? If it was, then the function previously should have been called readFileAsync(), right? Most functions in our code base are assumed to be synchronous.

None of the call sites got simpler here, so the reasoning isn't that convincing, but ok.
Every call site had the await keyword removed, plus in some cases some additional parentheses that was necessary to bind the await to the correct expression. Moreover, the calls stay in a stack, which makes debugging and stack traces simpler.

readFile is considered to be async in the std lib, so it's confusing when these calls don't have await or callbacks attached to them (that's why I converted it back to async in https://github.com/cloudflare/wrangler2/pull/538). If we're making this sync, I think we should add the Sync suffix.
The removal of await is, sure, less verbose. But errors and stacks stay even with async/ await statements (am I wrong here?). I think a bigger win is enclosing functions doesn't have to be async, which I can buy.
Happy with whatever call you take for either of these, it's fine.

OK, I can accept that the Node.js std lib uses readFileSync().
When debugging, once you hit an await point it is no longer possible to access the context in stackframes further up the chain.


No identity-based policy allows the ssm:GetServiceSetting action

Hello there,
we are facing the follow error, if we deploy the StackSet:
Resource handler returned message: "Resource of type 'AWS::SSM::Association' with identifier '4a65b230-3bb7-4b13-8f16-d97ac1c4e7ea' did not stabilize." (RequestToken: 6b9e0702-5379-664c-f454-f83c72e35757, HandlerErrorCode: NotStabilized)
After further investigation, we found this error in SSM automation:
Failure message
Step fails when it is Execute/Cancelling action. An error occurred (AccessDeniedException) when calling the GetServiceSetting operation: User: arn:aws:sts::210945550679:assumed-role/AutomationServiceRole-EnableDefaultSSM/Automation-52d6bfb4-f932-4d8d-9db3-c49e54b06e06 is not authorized to perform: ssm:GetServiceSetting on resource: arn:aws:ssm:eu-central-1:210945550679:servicesetting/arn:aws:ssm:eu-central-1:210945550679:servicesetting/ssm/managed-instance/default-ec2-instance-management-role because no identity-based policy allows the ssm:GetServiceSetting action. Please refer to Automation Service Troubleshooting Guide for more diagnosis details.
Is anyone else facing this error and unable to deploy the StackSet?

I fixed the issue with a DependsOn.
  automationRunbookEnableDefaultSSM:
    Type: AWS::SSM::Document
    DependsOn: automationServiceRole

Thanks @patrickznk, I hit the same problem and your fix worked.

Apologies for the delay in responding. It seems I was not receiving notifications for issues. The latest version of the CFN template added a DependsOn for the Automation IAM service role and removed the WaitForSuccessTimeoutSeconds parameter so the stack should stabilize during deployment. Please let me know if the issue persists.


powertoys Run does not run

Microsoft PowerToys version
0.82
Installation method
GitHub
Running as admin
No
Area(s) with issue?
PowerToys Run
Steps to reproduce
can not turn on
2024-07-03.txt
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

Duplicate of #33612.

/dup #33504