PDFs generated by selectpdf.com slow to render in pdf.js

Attach (recommended) or Link to PDF file here:
Problem PDF: wikipedia-select-net.pdf
Good PDF for contrast: Wikipedia-chrome.pdf
Configuration:

Web browser and its version: Chrome 83.0.4103.116
Operating system and its version: MacOS 10.14.6
PDF.js version: 2.5.207, discovered in 2.1.266
Is a browser extension: No

Steps to reproduce the problem:

Use attached pdf "wikipedia-select-net.pdf" above or generate your own from https://selectpdf.com/

This pdf was generated from selectpdf.com. For contrast, a PDF generated using the chrome native HTML to PDF converter is also provided; this pdf has no slowness issues.

Open PDF in the demo, scroll around, see that the PDF is slow to load

What is the expected behavior? (add screenshot)
PDF pages should render quickly for PDFs created via Selectnet. The load time should more closely resemble the Chrome PDF.
I am filing this as a pdf.js bug because Selectnet PDFs are able to render quickly in other applications, e.g. Preview or Chrome
What went wrong? (add screenshot)
PDF takes excessive time to load

Related to the render time, is the excessive amount of memory consumed by the selectnet PDF. Example of memory comparison between selectnet and chrome PDF (see tabs for title)

My coworker found that:

the textLayer HTML element for each individual page contains the full text for the entire document, but with the positioning such that text elements not for that page are outside the viewport of that page. This is not the case for the PDF generated from Chrome. This would appear to explain the excessive memory use while viewing these PDFs because if you have an N page document, then the full content of those N pages is included in the textLayer of each of those N pages leading to an N^2 growth in memory consumption.

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

This PDF loads quickly for me using below version.  Macbook air circa 2015.  8GB ram.
const pdfjsVersion = '2.7.90';
const pdfjsBuild = '6728c8fa6';

Any solution for this? i am facing the same issue for pdf created with selectedPdf

Any solution for this? i am facing the same issue for pdf created with selectedPdf

PRs #12903, #12922, and #16062 have improved performance here, and at this point I think that we can consider this issue fixed.


update pulumi dependency

this PR updates the pulumi/pkg dependency to point to the latest commit that is on master so we can get through the codegen changes made from the following PRs:
https://github.com/pulumi/pulumi/pull/11937
https://github.com/pulumi/pulumi/pull/11811
this dependency update is also being done for the resourcedocsgen tool in the docs repo: https://github.com/pulumi/docs/pull/8480

I'd vote to wait until the new version of the CLI is released. Pinning to the commit could be causing the test failures.

yeah I would feel more comfortable waiting as well.

@zchase, the new version of pulumi has been released so I updated the dependencies. I also updated the resourcedocsgen tool in docs as well in this PR. I tagged you on that one as well. Should we consolidate both of these tools as they seemingly do the same thing. In the docs repo it says to use the resourcedocsgen tool to generate the api docs and in the registry it says to use the registrygen tool to do this. Is there a reason why we can't use just one of these or is there something that one does that the other does not?
cc @cnunciato


Analytics (Optional)

Remove Plausible in lieu of direct server analytics managed by SUP. (Totally optional as long as the plugin doesn't jeopardize longevity or preservation efforts. In other words, if the plugin breaks, will it prevent the site from loading? If the site is emulated in a virtual machine will the call to the plugin prevent the rest of the site content from loading?)

This is not a plug in. It is simply a single call to Plausible's JavaScript. Were Plausible to go down, it would simply 404, and users would not notice.
The analytics are implemented here.
I'll think about whether to remove it or not. Thanks for the option.


Shorter versions of type creators like insets, point etc.

It would be nice if there was a support for a shorthand versions of these files like there is in CSS.
For example:
point(10, 10) => point(10)
insets(10, 10, 10, 10) => insets(10)
insets(10, 0, 10, 0) => insets(10, 0)
What do you think?

Good suggestion - shorter and more convenient ways of expressing things is always good. I'll add this in the next feature release.

From an outside perspective   Too much shorthand requires lots of rules that other authors need to know.   And it also if mirroring another spec you need to be concerned that you mirror it exactly.   I kind of liked that the markup was more precise.

Good point, one should of course not go crazy with adding shorthand. But as long as a shorthand form adds value and/or is in line with what exists for the standard web CSS, I think it's addition can be justified.


Running Portia with Docker

For windows machine I installed docker and this command "docker build -t portia ." also successfully  run . after that i received error while entering this section
docker run -i -t --rm
-v <PROJECT_FOLDER>/data:/app/slyd/data:rw
-p 9001:9001
--name portia
portia
how configure this command and what is <PROJECT_FOLDER>. Could please help on this to test.

There is a missing \.
https://github.com/scrapinghub/portia/pull/249

I have tried the following command which has showed in the error image, but it has still seem same error. Please let me know what are the steps would follow to run the portia for render javascript page.

Hope your help kind and need,
Best.

Hi,
you have to build the image first: docker build -t portia . in repository root.

Hi,
Actually i am checking in windows using docker.
Yes I ran this command before it in
c:protia> root. Its installed all the packages.

What's the output of docker images ?
On Sep 8, 2015 8:45 PM, "agramesh"<EMAIL_ADDRESS>wrote:

Hi,
Actually i am checking in windows using docker.
Yes I ran this command before it in
c:protia> root. Its installed all the packages.
[image: before]
https://cloud.githubusercontent.com/assets/11834931/9736269/ca65860e-565d-11e5-8e47-c070716ddb7e.jpg
—
Reply to this email directly or view it on GitHub
https://github.com/scrapinghub/portia/issues/269#issuecomment-138563950.

I cant analyze about the docker images and how can access it?

Just type "docker images" in the terminal where you type ask the docker
commands.
On Sep 8, 2015 10:23 PM, "agramesh"<EMAIL_ADDRESS>wrote:

I cant analyze about the docker images and how can access it?
—
Reply to this email directly or view it on GitHub
https://github.com/scrapinghub/portia/issues/269#issuecomment-138599191.

Hi
I got the image list,  thank you..

I checked two commands to run portia, but received error.

docker run -i -t --rm -v /<C:/portia/Projects>/data:rw -p 9001:9001 --name portia portia
docker run -i --tty=false --rm -v /<C:/portia/Projects>/data:rw -p 9001:9001 --name portia portia

is the above command correct..
Thanks in advance

Hi,
there are some problems with the command, try it like this:
docker run -i -t --rm -p 9001:9001 --name portia portia
The syntax you are using to mount the volume is not correct but it's not
necessary to do unless you want to modify the files inside portia.
2015-09-11 11:18 GMT+06:30 agramesh<EMAIL_ADDRESS>
Hi
I got the image list, thank you..
[image: docker images]
https://cloud.githubusercontent.com/assets/11834931/9807246/2a5f0014-586e-11e5-9cac-78489c637c55.jpg
I checked two commands to run portia, but received error.

docker run -i -t --rm -v //data:rw -p 9001:9001 --name portia portia
docker run -i --tty=false --rm -v //data:rw -p 9001:9001 --name portia
portia
[image: error]
https://cloud.githubusercontent.com/assets/11834931/9807251/31afb002-586e-11e5-8a2a-ace76e73f644.jpg

is the above command correct..
Thanks in advance
—
Reply to this email directly or view it on GitHub
https://github.com/scrapinghub/portia/issues/269#issuecomment-139451715.

Hi
thank you for response
here we faced some more new issues, we tried in many ways which more options
but shows error

If i delete the previous portia or Image ID, its displays error as " Unable to find image, Error: image library/portia:latest not found"

Hi, Run the command without -t and -i because your terminal doesn't have tty suport.
The image once build shouldn't be removed, but you need to remove the container before creating it again: docker rm -f portia

i am also running into these errors when trying to run with docker https://github.com/scrapinghub/portia/pull/337

Hi I installed portia-portia-16.02.2 using docker in windows, and i can open the portia UI in browser using http://<>:9001/static/index.html. And I use the command
docker run -i  --rm -p 9001:9001 --name portia portia
for loading portia, but the spdier didn't stored anywhere in my machine even i used the following command
docker run -i -t --rm -v <PROJECT_FOLDER>/data:/app/slyd/data:rw -p 9001:9001 --name portia portia. how could i resolve this.
thanks.

i tried many ways in windows using docker, i can create a spider in portia but the spider didn't store anywhere,
i use the following command to run portia

docker run -i --rm -v portia-data:/C/portia/portia/slyd/slyd/data/projects:rw -p 9001:9001 portia
docker run -i --rm -v portia-data:/C/portia/portia/slyd/slyd/data/projects:rw -p 9001:9001 --name portia portia
docker run -i --rm -v portia-data:/mnt/sda1/var/lib/docker/volumes/portia-data/_data:rw -p 9001:9001 portia

**and this is the log report of portia run : **
$ docker run -i --rm -v portia-data:/C/portia/portia/slyd/slyd/data/projects:rw -p 9001:9001 portia
2016-03-21 07:36:57+0000 [-] Log opened.
2016-03-21 07:36:57.488072 [-] Splash version: 2.0.3
2016-03-21 07:36:57.491375 [-] WARNING: Lua scripting is not available because 'lupa' Python package is not installed
2016-03-21 07:36:57.491988 [-] Qt 5.5.1, PyQt 5.5.1, WebKit 538.1, sip 4.17, Twisted 15.4.0
2016-03-21 07:36:57.492536 [-] Python 2.7.6 (default, Jun 22 2015, 17:58:13) [GCC 4.8.2]
2016-03-21 07:36:57.494495 [-] Open files limit: 1048576
2016-03-21 07:36:57.494871 [-] Can't bump open files limit
2016-03-21 07:36:57.693320 [-] Xvfb is started: ['Xvfb', ':1', '-screen', '0', '1024x768x24']
2016-03-21 07:36:58.671847 [-] /app/slyd/slyd/bot.py:25: scrapy.exceptions.ScrapyDeprecationWarning: Module scrapy.log has been deprecated, Scrapy now relies on the builtin Python library for logging.


Removes defer from imagery and terrain provider classes

This PR removes usage of defer.js from the *ImageryProvider.js* files.

Thanks for the pull request @sanjeetsuhag!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.

@ggetz Thanks for the explanation, it definitely helped clear things up! I believe the latest changes reflect the required architecture. Additionally, it also adds the changes to the *TerrainProvider.js classes. I have a couple of test related bugs that I am working through, but the core code is ready for a review pass.

Awesome, thanks @sanjeetsuhag! Good idea to keep this PR on the smaller size and just scoped to just the Providers. I'll give the non-test-related code a look.

@ggetz This is ready for review.

@ggetz The tests have been updated.

Looks great, thanks @sanjeetsuhag!


update docs to reflect the reorg

this works locally and there were not changes that should caus this error, merging for now and then will debug if this continrues to fail.


Drag query window outisde of Data Studio to it's own separate window.

Azure Data Studio Version:

Steps to Reproduce:
It is self explanatory, why can't I drag the query window outside of the application so I can make use of multiple screens when working with multiple queries, its a very simple thing that both Visual Studio and Management Studio have.

There is a open suggestion to allow this in vscode #10121 (which we are based on). It sounds like it is not a simple feature to add. I'm going to say this is an upstream issue.

This behavior is inherited from the vscode platform we are built on.  We'd prefer to see fixes like this taken upstream and then we can pick up in a future source code merge.  Thanks for opening this suggestion!


[WIP] Don't show the quoted query parameters in the filters UI

We can get filters from two places:

The list of aggregations on the catalogue API response
A specific filter applied by the user

In the latter case, suppose a user clicks through to a filtered search from a work page.  We need to include the filter they clicked on in the list of filters (so they can see/cancel it), but if it's an uncommon value it may not be in the API aggregations.  We need to add it manually.
A filter needs several things, including a value (which goes in the URL query parameter) and a label (shown to the user).
We combine these together inside filterOptionsWithNonAggregates. The previous signature was as follows:
function filterOptionsWithNonAggregates(
  options: FilterOption[],
  values: string[],
  showEmptyBuckets: boolean = boolean
)

That values is the list of currently selected filter values, which we need to construct the currently-applied filters which aren't in the aggregation. Unfortunately, because we only get a single string here, we need something that works for both value and label, which meant quoting the value before passing it to this function. Then users see quoted values in the interface, eww:

This PR modifies the signature of the function like so (roughly):
function filterOptionsWithNonAggregates(
  options: FilterOption[],
  values: { label: string, value: string }[],
  showEmptyBuckets: boolean = boolean
)

so callers can pass a separate label/value if necessary – thus removing the quote marks from the UI.
The actual change is a bit more involved to keep the calling of filterOptionsWithNonAggregates sensible, but that's the general gist. I've tested it locally and it seems to work.

holy size batman, where did that come from?

Closing in favour of #8513


Extra space added to multiline bracketed objects

When attempting to begin a multiline bracketed object, such as a list or dictionary, an extra space is added upon hitting return.

What steps reproduce the problem?

See Gif

What is the expected output? What do you see instead?
I expect that hitting return will place the cursor one space behind where it does.
Versions

Spyder version: 4.2.0
Python version: 3.8.5 64-bit
Qt version: 5.9.7
PyQt version: 5.9.2
Operating System name/version: Windows 10

On more experimentation, this only seems to be a problem with short variable names.  For instace:
mylist = [
1, 2, 3
]
Does not seem to suffer from this problem.

On more experimentation, this only seems to be a problem with short variable names.  For instace:
mylist = [
1, 2, 3
]
Does not seem to suffer from this problem.

Hey @Balter85, thanks for reporting. This really depends on the way you want to format your code. We add the extra space you mentioned to keep the indentation level of the previous line, which is what most people expects, but not in this case.
Fortunately, we now offer auto-formatting with Black, which could solve this issue automatically. To activate it, please go to
Tools > Preferences > Completion and linting > Code formatting

and change our formatting provider to black. After that, you can use the Ctrl+Alt+I keyboard shortcut to format your current file or selection.
Please do that and let us know if it works for you.

Hey @Balter85, thanks for reporting. This really depends on the way you want to format your code. We add the extra space you mentioned to keep the indentation level of the previous line, which is what most people expects, but not in this case.
Fortunately, we now offer auto-formatting with Black, which could solve this issue automatically. To activate it, please go to
Tools > Preferences > Completion and linting > Code formatting

and change our formatting provider to black. After that, you can use the Ctrl+Alt+I keyboard shortcut to format your current file or selection.
Please do that and let us know if it works for you.

Hi,
Thank you for the response.  While it doesn't solve this particular issue, I was unaware of CTRL-ALT-I which does make my life easier.  Thank you.  Feel free to close the issue.

Hi,
Thank you for the response.  While it doesn't solve this particular issue, I was unaware of CTRL-ALT-I which does make my life easier.  Thank you.  Feel free to close the issue.

I was unaware of CTRL-ALT-I which does make my life easier. Thank you

Glad you found our new auto-formatting functionality useful.

Feel free to close the issue.

Ok, closing then.

I was unaware of CTRL-ALT-I which does make my life easier. Thank you

Glad you found our new auto-formatting functionality useful.

Feel free to close the issue.

Ok, closing then.


Adding one more stat module : statsgroups

new module stat that can show informations with customers group details
On onther pull request I add the file

Picked to master as 569aa780e534a1a6dc7dafb53ca9e45df88ccf3d (together with the other commit). Thank you very much!


Redesign GYTxMonad and its instances to be able to submit transactions

Summary
Closes #316
This is a long overdue redesign of GYTxMonad. It adds the ability to submit transactions to the monad and splits it into a more lawful hierarchy. In particular:

GYTxQueryMonad - For common utxo and similar queries
Instance: GYTxQueryMonadIO

GYTxQueryMonad => GYTxSpecialQueryMonad - For uncommon queries such as protocol parameters, era history etc. (this could potentially be removed, see note below)
Instance: GYTxQueryMonadIO

GYTxQueryMonad => GYTxUserQueryMonad - For queries while acting as a user (having access to own wallet for querying purposes not signing purposes)
Instance: GYTxBuilderMonadIO - see below.

(GYTxSpecialQueryMonad, GYTxUserQueryMonad, Default (TxBuilderStrategy)) => GYTxBuilderMonad - For building transactions while acting as a user. This allows different instances to choose their own transaction building method. If TxBuilderStrategy is set to GYCoinSelectionStrategy (default), the implementation defaults to the pre-existing transaction building methods. Therefore, simply anyclass deriving this class will be enough to use the transaction building unchanged.
instance: GYTxBuilderMonadIO

GYTxBuilderMonad => GYTxMonad - For interacting with the blockchain in a way that mutates its state. E.g building and submitting transactions etc. Also is able to sign as a user.
Instance: GYTxMonadIO

(GYTxMonad (TxMonadOf m), GYTxSpecialQueryMonad m) => GYTxGameMonad m - This is an entirely new feature. This allows simulating multiple users interacting in an environment, building and submitting transactions etc. This is extremely useful in testing. Where we model an environment with many users interacting, waiting for events etc.
Each "game' monad has its own "tx" monad. The tx monad code can be lifted into the "game" monad and assigned a user to perform it as.
Instance: GYTxGameMonadIO

I suggest putting this brief note about the monad hierarchy in the docs so people can choose which monad they need, instead of always reaching for the most powerful one
Note the distinction between a TxQuery monad and the Tx monad. The query monad classes are meant to be non-mutating. In other words, if you are given a GYTxUserQueryMonad m => m a for example, and you run it (using a relevant interpreter) - you can be sure that it did not mutate anything on the chain. However, that guarantee does not exist for GYTxMonad.
This sort of a lawful type classes helps in "coloring" functions with types to define their expected behavior. Most code in the GY codebase (where the backend only builds transactions and passes it to frontend) can use just a query monad!
This makes running the monad instances (e.g GYTxMonadIO, renamed from GYTxMonadNode) much simpler. There is no traversable hack anymore. It's a simple monad interpreter. As you'd expect from any other proper monad in the ecosystem. As a result, transaction defining, building, signing, and submitting - all under one monad feels much more ergonomic.
I understand that this is a large PR with breaking changes, but I really think it's for the best. I will annotate most of the significant changes with comments in this PR to make reviewing easier. I will post a summary with each important commit.
Furthermore, almost all commits are self-contained (they are granular and they compile). The only exception is the commit titled "Add 'submitTx' and 'awaitTxConfirmed' under 'GYTxMonad' and refine monad hierarchy". The subsequent commit is required to build the project.
Migration guide
For code that uses submission after building, migration is a bit more work but has a simple pattern. This pattern can be observed in Privnet code, for example.
- txBodyPlace <- ctxRunI ctx (ctxUserF ctx) $ do
+ ctxRun ctx (ctxUserF ctx) $ do
      addr <- scriptAddress giftValidatorV1
-     return $ mconcat
+     txBodyPlace <- buildTxBody $ mconcat
           [ mustHaveOutput $ mkGYTxOut addr (valueSingleton goldAC 10) (datumFromPlutusData ())
           ]
- void $ submitTx ctx (ctxUserF ctx) txBodyPlace
+     submitTxBodyConfirmed_ txBodyPlace [ctxUserF ctx]

Code that previously used traversing functions such as rrunGYTxMonadNodeF in order to build a GYTxSkeleton contained within a traversable (e.g for interpeting GYTxMonadNode (Maybe (GYTxSkeleton v))) will simply use well-known helpers to traverse instead, making things much simpler:
- grabGiftsTx' <- ctxRunF ctx (ctxUser2 ctx) $ grabGifts  @'PlutusV2 giftValidatorV2
- mapM_ (submitTx ctx (ctxUser2 ctx)) grabGiftsTx'
+ ctxRun ctx (ctxUser2 ctx) $ do
+    grabGiftsTx' <- grabGifts @'PlutusV2 giftValidatorV2 >>= traverse buildTxBody
+    mapM_ (`submitTxBodyConfirmed` [ctxUser2 ctx]) grabGiftsTx'

For code that only built transactions but did not submit them, it is advised to replace the GYTxMonadNode usage with GYTxBuilderMonadIO. Instead of returning a skeleton, it should use buildTxBody on the skeleton. So you'd have: GYTxBuilderMonadIO GYTxBody. This can be run with runGYTxBuilderMonadIO to obtain the GYTxBody like before. Alternative, define a function buildAndRunGYTxBuilderMonadIO :: GYTxBuilderMonadIO (GYTxSkeleton v) -> GYTxBody, which is simply a composition of buildTxBody followed by runGYTxBuilderMonadIO.
- txBodyPlace <- runGYTxMonadNode nid providers addrs change collateral $ do
+ txBodyPlace <- buildAndRunGYTxBuilderMonadIO nid providers addrs change collateral $ do
      addr <- scriptAddress giftValidatorV1
      return $ mconcat
           [ mustHaveOutput $ mkGYTxOut addr (valueSingleton goldAC 10) (datumFromPlutusData ())
           ]

where:
buildAndRunGYTxBuilderMonadIO :: GYTxBuilderMonadIO (GYTxSkeleton v) -> GYTxBody
buildAndRunGYTxBuilderMonadIO act = runGYTxBuilderMonadIO $ act >>= buildTxBody

This way, a very simple find and replace action will complete most of the migration.
Extended ideas

There could be a GYTxMonad m => GYTxGameMonad m, which could allow a "game situation" where we have multiple users building and submitting contracts, interacting with one another etc. This would essentially have methods like waitUntilSlot, waitUntilBlock etc. There needs to be a function asUser :: (GYTxMonad m, GYTxGameMonad n) => User -> m a -> n a to facilitate the "game" flow. Added.
Related to #318 - GYTxBuilderMonadIO should actually have its envUsedSomeUTxO somehow exposed to GYTxMonadIO so that the submitTx implementation of GYTxMonadIO can reset it. I'd suggest making this field mutable. That way, a stateful flow can be achieved - which is necessary since envUsedSomeUTxO is meant to be stateful. I would discourage the usage of the state monad. Just utilize ReaderT _ IO which we already have.
Some day, this codebase should use an effect system instead. effectful will probably be a good choice. Or eff, if it's no longer experimental.

Type of Change
Please mark the relevant option(s) for your pull request:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Code refactoring (improving code quality without changing its behavior)
[ ] Documentation update (adding or updating documentation related to the project)

Checklist
Please ensure that your pull request meets the following criteria:

[x] I have read the Contributing Guide
[x] My code follows the project's coding style and best practices
[x] My code is appropriately commented and includes relevant documentation
[ ] I have added tests to cover my changes
[x] All new and existing tests pass
[ ] I have updated the documentation, if necessary

Testing
Please describe the tests you have added or modified, and provide any additional context or instructions needed to run the tests.

[ ] Test A
[ ] Test B

Additional Information
If you have any additional information or context to provide, such as screenshots, relevant issues, or other details, please include them here.

Summary of important commits:

Common MTL transformer instances (also remove ExceptT instance)
This adds some MTL transformer instances so downstream (user code) can easily compose with our monad in easy ways.
I have removed the ExceptT instasnce. Please read the associated note added in the commit.

Reorder GYInScript type variables
Mostly unrelated to the main purpose of the PR, but greatly helpful for users using TypeApps. This is a breaking change. Helped get rid of a verbose type signature in a privnet test.

Add ownChangeAddress and ownCollateral to GYTxMonad
Simple change that allows building transactions within the monad later on
Note the added law on someUTxO. See: #318

Remove BuildTxException, rename to GYBuildTxError
Consistent naming and exceptions mechanism
Since transaction building is part of the monad now, transaction build errors ought to also be part of the monad errors.

Add submitTx and awaitTxConfirmed to GYTxMonad
Main commit amending the monad class declaration and hierarchy. Upto GYTxUserQueryMonad denotes query monad capabilities (non chain mutating).
GYTxSpecialQueryMonad could potentially be removed. See note added in the commit.
Note the laws around submission and how it relates to someUTxO.

Make transaction building a part of GYTxMonad utilities
Main commit amending the monad instances and their usage.
This showcases what changes might be required in user code as well. See changes in Privnet code.

Use pre-provided submission utils rather than privnet defined ones.
Note that the submission functions only sign with the payment key. Previously, they checked whether stake key was required and added it if necessary. This is done manually now.
Upon inspection, it seems like this feature of detecting the necessity for stake key was not necessary since the Stake utils test do that on their own anyway.

Note how the Privnet code utilizes ctxRunQuery, ctxRun, and ctxRunBuilder etc. to limit expectations and mutation.

I suggest reviewing this PR commit by commit. Most of the changes to the Privnet code can be ignored. It's simply updating usage and its summarized in here. My commit summary above might help in reviewing as well.
Before, merging - I need to know how the parallel and chaining building could be amended. I'm thinking of removing the whole Traversable f thing from buildTxCore entirely and then exposing the parallel and chaining functions. They are currently in GeniusYield.TxBuilder but not exposed.

I just added GYTxGameMonad, which will be extremely beneficial for elegant testing code. I suggest you take a look! It's a really cool feature. It allows having multiple GYTxMonad code blocks be lifted into one environment and performed as different users (as given)!
https://github.com/geniusyield/atlas/pull/322/commits/a3a9ac35530867be6ec561094191917257d69ee2

I'd suggest making this field mutable. That way, a stateful flow can be achieved - which is necessary since envUsedSomeUTxO is meant to be stateful.
I am fine with it as I see merit in ReaderT design pattern.

Thanks @TotallyNotChase for much need contribution and also thanks for making review easy with thorough comments!

Before, merging - I need to know how the parallel and chaining building could be amended. I'm thinking of removing the whole Traversable f thing from buildTxCore entirely and then exposing the parallel and chaining functions. They are currently in GeniusYield.TxBuilder but not exposed.

What is the motivation of getting rid of Traversable f here? Unfortunately it's not clear to me the alternative implementation you are proposing. I am fine with just exposing

Before, merging - I need to know how the parallel and chaining building could be amended. I'm thinking of removing the whole Traversable f thing from buildTxCore entirely and then exposing the parallel and chaining functions. They are currently in GeniusYield.TxBuilder but not exposed.

What is the motivation of getting rid of Traversable f here? Unfortunately it's not clear to me the alternative implementation you are proposing. I am fine with just exposing

The Traversable thing is just a shoehorning thing. The same usability can be achieved by simply calling traverse on the result now. Because the monad is actually composable now. The reason it existed prior is because that composition was impossible before, and the only way to obtain the  a from GYTxMonad a, was to use Const and a shoehorned traversable in the implementation. This is completely useless now, and therefore there's no point in making the type signature more complicated.

I have removed the Traversable f parameter entirely in the last commit. Most places that were using it before were really just using it as either Identity (entirely unnecessary) or Const (no longer necessary, just run the monad). But anyway, the same usability as before can be achieved by essentially calling fSkeletons >>= traverse buildTxBody etc. within the monad. That's way simpler and conforming to the "usual" Haskell APIs.


Feature request: Token credential with local cache

We got a feature request from one of our customers.
Basically they're using token credential for authorization and they need different client options (like retry options) for different kind of requests so they need to initialize many clients for those requests.
The problem is the access token is cached in BearerTokenAuthenticationPolicy, which is created when initializing a blob client. So the access token cannot be shared among these blob clients even though requests are sent to the same storage account.
I'm looking for some way to fix this problem and while I'm looking at .Net code, I found SharedTokenCacheCredential. I think such a credential with local cache can work in this scenario.
@RickWinter @antkmsft What do you think? Can we provide the same credential in C++?

@Jinming-Hu, please help me to understand the request better. It sounds like what you want is to minimize token requests and token refresh requests. I.e. a case 100 clients are using the same credential, you call method of one of them, and it gets the token to reuse within the next hour. But currently that would be reused with only that one client, while all other 99 clients will send the same token request. And in an hour, all 100 of them will need to send a request to refresh the token.
What you would like to see is one request per (tenant id, client id, auth scope) to get the token initially, and a single request to refresh the token, not 100 of them. The cash is "local" - it only lives during process run time and is not shared between processes, and is not persistent after process quits.
Is this accurate?

it only lives during process run time and is not shared between processes, and is not persistent after process quits.

Cache within the process can meet this customer's requirement. I think it's much easier to implement because the cache can reside in memory, it involves no file operations or inter-process communication or synchronization. It's good enough for now.
But we should be aware of the fact that .Net has a more advanced implementation. Cache can be shared across processes with .Net. We should keep this in mind when designing the API interface, so that it can be easily extended if we do have such a feature request in the future.


DynamicRNNLayer cannot use GRUCell

When set cell_fn = tf.contrib.rnn.GRUCell, DynamicRNNLayer will complain TypeError: init() got an unexpected keyword argument 'state_is_tuple'.
Then I read the source code, and find that init() calls cell_fn(num_units=n_hidden, **cell_init_args) with the cell_init_args = {'state_is_tuple' : True} set.
In BasicRNNCell, BasicLSTMCell and LSTMCell, this is good, because they all have this parameter. However, GRUCell does not have it. And according to the Tensorflow docs, the 'state_is_tuple' parameter is True by default, so I think it's unnecessary to have it in the cell_init_args.
Do I get the right idea? Or the GRUCell is not suitable for Dynamic RNN by design?

@bcy you are right

I see that you have committed to master branch. I just pulled and tested, and it works now. Thanks.


React Native Image Prefetch Is Not Caching GIFs, And Falsely Says it Does (only tested on iOS)

Is this a bug report?
Yes
Have you read the Contributing Guidelines?
Yes
Environment
Environment:
OS: macOS Sierra 10.12.6
Node: 7.2.1
Yarn: 0.17.10
npm: 3.10.9
Watchman: Not Found
Xcode: Xcode 8.3.3 Build version 8E3004b
Android Studio: Not Found
Packages: (wanted => installed)
react: 16.0.0 => 16.0.0
react-native: https://github.com/expo/react-native/archive/sdk-23.0.0.tar.gz => 0.50.3
Steps to Reproduce

Using Image.prefetch, prefetch a GIF that is online such as the one here: https://78.media.tumblr.com/30cc9a2d916a3b93b0505686f3e16369/tumblr_nj6azbWpcP1twakhqo1_540.gif
Enter a view that shows the image
Although the image should be loaded from the cache, it is always loaded on the spot.

Expected Behavior
I expect prefetching to work with Animated GIFS.
Actual Behavior

I am building a card stack interface, and the cards have image backgrounds. Some of them are GIFs.
I am using the Image library from react-native to prefetch all images from the cards. The goal of this is to create a fast UX. -> Image.prefetch(url)
The JPGs and static images are preloaded fine -- they show up immediately as intended.
GIFs, however, are not. They are loading after the fact, live, once the view is rendered.
The output from the Image.prefetch(url) is a promise, and when identified it appears all caching is running. Even the GIF prefetch requests are returning true.
When the view showing the actual Image has the source.cache property set to only load the image if it's in the cache, the JPGs and such still show up (cache validated), and the GIFs do not.
I have only tested this on iOS, and have tested GIFs from multiple source URLs.

Reproducible Demo
This demo will show you exactly what I'm talking about. =)
https://snack.expo.io/BkMaU5iWG

I was shocked when I found that the gifs only displayed once on Android , But when I use a web-gif by uri reference, it worked well. And however the iOS does not meet the problem.

Hey Shiqkuangsan! I'm not sure what you mean -- does that have to do with prefetching/caching GIFs?

When I used xxx.gif in the 'asset' path of my project  by require-reference, the gif only displayed once, and stopped, behaved like a png, But when I changed the method to use uri: 'http:xxx/.../xxx.gif', the same gif workded well.  I don't know why.

Sounds weird, but unrelated to the bug I posted above.

So I came here and found your issue, thought you maybe also met this problem~But thank you all the same.

And as for your issue, I tested on Android. I enter a activity contained a web-gif prefetched from site, and than I exit my app, and turn off wifi, open app and enter the same activity, the gif works well. Does it means the prefetch method cached the gif ?

And I upload that problem-gif to a web, and than use uri:'xxx'  to display it, it does not worked, so I can say there is something with my gif, and I change to another gif, my problem solved.


✨ sequin report

#76  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @RTLS and the rest of your teammates on  Graphite


Any way to draw different colors along the normal direction of the line using an offset value?

I try to draw some more colorful line on the map but failed,it looks like it's not even supported in SKPath with SKPaint.
Like this, use to show administrative boundaries

or this,use to show transportation information in big resolution with different direction,I think using an offset value may be able to achieve that, but I can't find a simple way to do it.

Many thanks!

I think, that should possible with a shader in the SKPaint of the path. I would start with research in SkiaSharp or Skia. Perhaps you find there a solution.

This is mostly a SkiaSharp question so you could also ask it there (after you searched for previous questions on this topic of course). https://github.com/mono/SkiaSharp
Dash Patterns
It is possible to use dash patterns in SkiaSharp. We use this in our sample "Tests" | "Line", like this:
feature = new GeometryFeature(wktReader.Read("LINESTRING (7000000 12000000, 13000000 12000000)"));
feature.Styles.Add(new VectorStyle { Line = new Pen(Color.Blue) { PenStyle = PenStyle.DashDot } });

A screenshot of the sample copied from the blazor samples (https://mapsui.com/samples/):

Combining colors
As far as I know you can use only one color in such a draw operation. My way to go about this would be to have two draw operations. The first color be a solid line, and on top of that draw the dashed line, so the solid line fill the gaps in the dashed line.
Note, that for a good solution where the line looks the same while panning and zooming you need to take into account the offset (of the origin of the path is not visible in the image you need to correct for that) and the scale (the dash pattern is different for different scales to make it appear the same to the user).


Migrate this boilerplate project to flutter v2

Hi,
I am just wondering if this boilerplate project is planed to migrate to flutter 2.x ?
Thanks in advance

Hi @all,
I have updated the DI library for this boilerplate template, please go through feature/null-safety branch for more details. Feel free to reach out to me if you find any issues.
Note: I will merge the changes into master as soon as I am done testing and implementing other features.
Thanks.


chore: update README

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 76.19%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
256

Relevant Lines:
336

💛  - Coveralls


[Feature] Configure hatch for build, release, and bump version

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt-adapter functionality, rather than a Big Idea better suited to a discussion

Describe the feature
We've selected hatch as our build tool. Now we need to configure dbt-adapter so that hatch can build/release the application.
Describe alternatives you've considered
See ard-0001-build-tool.
Who will this benefit?
No response
Are you interested in contributing this feature?
No response
Anything else?
No response

We still need to configure version bumping.

Version bumping is covered in the release workflow ticket.


Update keyboardcleantool to 1.0

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

@miccal Thanks for the quick review. Addressed all change requests.

Thank you.


Fix light sprite size

Fixes: #2200

The sprite can only size to 128 as its max is seems

It's working for me. Try again? If not, let me know how you're doing it.

@sammyt291

Oh I was checking the issue instead of the PR. This all looks great, thanks a lot!


Is possibile to customize shape in line bars?

Is possibile to customize shape in line bars?
I need something like the charts here.
Thanks in advance,
Simone

Here's a few ideas:
https://github.com/chartjs/Chart.js/issues/3871
https://stackoverflow.com/questions/49910539/chart-js-add-icon-at-the-top-of-bar
https://stackoverflow.com/questions/52110684/adding-image-on-the-top-of-bar-in-chartjs

@benmccann thanks for your interest... i can say that is interesting the approach but my triangle is data-dependent. I need to change its base and height using  some external data.
I'm still looking for the right answer on the internet. If I'll find something i'll post it.
Thanks for the links :)


Update mockito from 5.2.0 to 5.3.1

Codecov Report

Merging #800 (7338418) into main (96afe71) will increase coverage by 0.05%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main     #800      +/-   ##
============================================
+ Coverage     92.45%   92.51%   +0.05%
- Complexity     2652     2653       +1
============================================
  Files           286      286
  Lines          5275     5275
  Branches        553      553
============================================
+ Hits           4877     4880       +3
+ Misses          255      253       -2
+ Partials        143      142       -1

see 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Typo in lab instructions for feature-workshop-prep

Should be "In this lab, we onboarded..."

Fixed


nans in gradient of scale_and_translate

I'd like to take gradients of scale_and_translate wrt the translation parameters.  Currently the gradients are nan whenever the translation is >1.  Example:
import jax
import jax.numpy as jnp
from jax.image import scale_and_translate

xin = jnp.r_[:16].reshape(4, 4) + 1.
xin = jax.device_put(xin)

def f(x):
    out = scale_and_translate(xin, shape=xin.shape, spatial_dims=(0, 1),
                              scale=jnp.array([1.0, 1.0]),
                              translation=jnp.array([x[0], x[1]]),
                              method='linear', antialias=True)
    return out

f((1.1, 1.2))  # raises an error if JAX_DEBUG_NANS=1

g = lambda x: jnp.sum(f(x))

print(jax.grad(g)( (1.1, 1.2)))  # nans

If JAX_DEBUG_NANS=1 is set, the script throws an exception at the evaluation of f:
Traceback (most recent call last):
  File "resize.py", line 15, in <module>
    f((1.1, 1.2))  # raises an error if JAX_DEBUG_NANS=1
  File "resize.py", line 9, in f
    out = scale_and_translate(xin, shape=xin.shape, spatial_dims=(0, 1),
  File "/home/pfister/miniconda3/envs/scico/lib/python3.8/site-packages/jax/_src/image/scale.py", line 219, in scale_and_translate
    return _scale_and_translate(image, shape, spatial_dims, scale, translation,
  File "/home/pfister/miniconda3/envs/scico/lib/python3.8/site-packages/jax/_src/image/scale.py", line 92, in _scale_and_translate
    w = compute_weight_mat(m, n, scale[i], translation[i],
  File "/home/pfister/miniconda3/envs/scico/lib/python3.8/site-packages/jax/_src/image/scale.py", line 69, in compute_weight_mat
    weights / total_weight_sum, 0)
  File "/home/pfister/miniconda3/envs/scico/lib/python3.8/site-packages/jax/_src/numpy/lax_numpy.py", line 5666, in deferring_binary_op
    return binary_op(self, other)
  File "/home/pfister/miniconda3/envs/scico/lib/python3.8/site-packages/jax/_src/numpy/lax_numpy.py", line 569, in true_divide
    return lax.div(x1, x2)
  File "/home/pfister/miniconda3/envs/scico/lib/python3.8/site-packages/jax/_src/lax/lax.py", line 352, in div
    return div_p.bind(x, y)
  File "/home/pfister/miniconda3/envs/scico/lib/python3.8/site-packages/jax/core.py", line 264, in bind
    out = top_trace.process_primitive(self, tracers, params)
  File "/home/pfister/miniconda3/envs/scico/lib/python3.8/site-packages/jax/core.py", line 606, in process_primitive
    return primitive.impl(*tracers, **params)
  File "/home/pfister/miniconda3/envs/scico/lib/python3.8/site-packages/jax/interpreters/xla.py", line 232, in apply_primitive
    return compiled_fun(*args)
  File "/home/pfister/miniconda3/envs/scico/lib/python3.8/site-packages/jax/interpreters/xla.py", line 351, in _execute_compiled_primitive
    check_special(prim.name, out_bufs)
  File "/home/pfister/miniconda3/envs/scico/lib/python3.8/site-packages/jax/interpreters/xla.py", line 370, in check_special
    _check_special(name, buf.xla_shape(), buf)
  File "/home/pfister/miniconda3/envs/scico/lib/python3.8/site-packages/jax/interpreters/xla.py", line 376, in _check_special
    raise FloatingPointError(f"invalid value (nan) encountered in {name}")
FloatingPointError: invalid value (nan) encountered in div

The issue is in this block:
https://github.com/google/jax/blob/c85e835951c68b4383dbc9ef25bef99ea996fd84/jax/_src/image/scale.py#L66-L69
in particular, the computaiton weights/total_weight_sum can involve a divide by 0, even though those locations get set to zero in the course of the jnp.where.
I'll submit a PR shortly with the "where inside of where" fix.

Assigning to @hawkinsp since Peter is shepherding the PR review

I have observed the same problem with the sinc function defined here.
A quick fix for the sinc case is given below. I would add this to another PR, but I've never done one and don't want to clutter with my inexperience :)
def safe_sinc(x: jnp.ndarray) -> jnp.ndarray:
    _check_arraylike("sinc", x)
    (x,) = _promote_dtypes_inexact(x)
    eq_zero = lax.eq(x, lax._const(x, 0))
    pi_x = lax.mul(lax._const(x, pi), x)
    safe_pi_x = where(eq_zero, lax._const(x, 1), pi_x)
    return where(
        eq_zero, _sinc_maclaurin(0, pi_x), lax.div(lax.sin(safe_pi_x), safe_pi_x)
    )

@astanziola Open a new issue or send a PR for sinc?
@lukepfister Perhaps. I wonder how often we want to apply the double-where trick just around a division (as opposed to other things as well).

@hawkinsp Sure, I'll do one this weekend


PsDscRunAsCredential is not passed to Set-ADGroup in xADGroup

Trying to manage an xADGroup resource using a PsDscRunAsCredential fails on certain operations. The creation of a group works, but when setting the properties of an existing group, we get "Set-ADGroup : Insufficient access rights to perform the operation". Debugging reveals that this occurs at the call to Set-ADGroup on line 387 of the MSFT_xADGroup.psm1 module. In inspecting the code, it appears that  $Credential never gets passed to the parameter list hashtable $setADGroupParams. Other calls like New-ADGroup succeed because they use a separate parameter list such as $adGroupParams which does appear to contain the credentials.

@stephenbonar That's correct. When you use PsDscRunAsCredential, the LCM is responsible for elevating the process the resource is executed in, to run under the specified credential (and the resource knows nothing/never sees this credential).
Have you tried swapping the PsDscRunAsCredential to Credential instead? This will call the *-ADGroup cmdlets with the credential parameter.

@stephentoub Having said that, I think you're correct in that fact that, if you do use the Credential parameter, it doesn't get passed to the Set-ADGroup cmdlet. That's a problem.

Yeah, I wrote PsDscRunAsCredential out of habit but I really meant Credential. I went back and looked at my DSC script and I was correctly referencing Credential there.

@stephenbonar You might have already seen, but I've submitted PR https://github.com/PowerShell/xActiveDirectory/pull/107 that should fix this. Would you mind reviewing it and seeing whether it fixes the issue?


Feature/improve yubikey usage

Improve the usage of yubikey for other av commands

Hey @nbrys - thank you for proposing this contribution! The one issue with moving from an alias to a function is that we lose the default auto-completion behavior.
Before
Type avl + space and then press the Tab key. You'll see your profiles appear as autocomplete suggestions:

After
On this PR, since avl (and others) moved to a function, autocompletion of profiles no longer works.

(you'll see in the screenshot it's suggesting foo.txt, a file in the demo directory, not my aws-vault profiles).
Solution
I'm happy to accept a changeset, I just want to make sure that we don't lose the autocomplete functionality that existed before this change. @reegnz submitted a PR a while back that focused on improving autocomplete, which might be interesting to take a look at: https://github.com/blimmer/zsh-aws-vault/pull/21

Thanks again for taking the time to write up this PR. Since this PR has been open for quite some time, I'm going to close it. If you'd like to circle back, I'd be happy to take a look again.


A problem about billing of cloud search.

In the cloud search instance page, the total fees per hour is less than the sum of all the fees of instances.However, the balance decrease rate seems to be faster than the sum of all the fees. What's wrong?Are there any other hidden fees in the service?
Thanks for helping me.

Hello, there is no hidden fee. The amount spent should be the same as the fees shown. Can I know your username so I can check it?
Thanks.

In the cloud search instance page, the total fees per hour is less than the sum of all the fees of instances.However, the balance decrease rate seems to be faster than the sum of all the fees. What's wrong?Are there any other hidden fees in the service? Thanks for helping me.

Hello again. Now when I looked at my own account, I saw the same problem. Total cost calculation is done incorrectly. Now I solved the problem. Thank you for feedback.


Does this work with SH10RT inverter?

Just got a sungrow inverter and battery and comms module... does this work with SH10RT etc

I think you'll have to try and find out...
But I heard they've updated the firmware for a while and hopefully they didn't change the configurations of modbus address.

will do... do you have a way to find the IP address easily? ;)

I remember I found it on the router.

Note the json on documentation is missing platform attribute which is throwing an error

showing some error in image above?

cannot view plugin logs due to some issue with the Plugin alias...
config below
    {
        "accessory": "SungrowInverter",
        "name": "My Solar Panels",
        "ipAddress": "192.168.1.XXX",
        "port": 502,
        "model": "SH10RT-V112",
        "serialNumber": "XXXXXXXXX",
        "refreshInterval": 1,
        "platform": "SungrowInverter"
    }

Sorry I don't have homebridge anymore... So I couldn't do some proper test.
The plugins was written too long ago. Perhaps it doesn't work the current version of homebridge anymore

Ok thanks so much
Richard Marr
Sent from my iPad
On 1 Mar 2024, at 1:41 pm, jerryzou @.***> wrote:

Sorry I don't have homebridge anymore... So I couldn't do some proper test.
The plugins was written too long ago. Perhaps it doesn't work the current version of homebridge anymore
—
Reply to this email directly, view it on GitHubhttps://github.com/jerryzou/homebridge-sungrow/issues/2#issuecomment-1972362690, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGHK4FFCGIZJEMKLA467LEDYV7TD3AVCNFSM6AAAAABEA6F2VSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZSGM3DENRZGA.
You are receiving this because you authored the thread.Message ID: @.***>


A600 support

Would be nice to add support for Amiga 600, or if it is allready supported, validate it by test and clarify the support in documentation and description of the project.

We're working on a A600 version of the hardware, and while it's still in the design phase, we can already now conclude it will only be able to support the Pi Zero form factor due to mechanical restrictions. More info will be posted once there is some real hardware produced. Probable time frame is Q1 2020.

Update: A600 version is now actively being worked on. We can already now see that there will be a couple of differences compared to the A500 version:

The memory will be increased to 2MB, thus allowing for all chip memory to reside on the A314. (prices for 2MB SRAM chips have dropped recently to apx the same levels as 1MB had previously)
Our intention is that programming (and re-programming) of the FPGA flash is made possible directly from the Raspberry Pi, without any additional hardware. Those who wish to continue using a classic JTAG interface can still do so, but will have to make a custom adaptor cable.

If the A600 version can only support a Pi Zero due to physical limitations, could it alternatively support a CM3 compute module?

It could, but it won't because we see it as a too big re-design for very little, if any, advantages. So the cost/performance ratio is not justifiable.

The cost is exactly the same as being able to use a full Pi 3 B+. It's more than a Pi Zero, but then the design brief calls using a Pi Zero a "design limitation"... ;)
The physical advantages are numerous.  Being able to place the connectors exactly where they are best suited. Being able to place only the connectors required. Lower power footprint by not using devices that are not installed. Extremely compact form.
I'd be happy to design this PCB.

@plasticuser If you feel up to designing and building a version of A314 for A600 using a CM3 compute module then of course that would be a very positive development :)
My initial thoughts about such a design is: how will you add back networking (wifi) and USB capabilities that are not available in CM3 (as compared to in Pi Zero)? And what is the combined cost of CM3 + additional chips? If the cost is not a lot more than a Pi3 then this sounds very interesting.

I recognize it would cost a "little" more. I'm estimating about $7-$9.
The fun bit is that I could supply the board with build time configuration options to lower the price point. Happy with ethernet? Cool! Want lots of I2C or SPI break outs on the board? Cool! 2MB SRAM? Yisss!
It is worth $10 on the final product price to have the four cores, and the option of SSD or eMMC - which is MUCH faster and more reliable. The CM4 is going to have a different pinout, but it would be easy to adapt the design at that point for the new connector.
So, yes, I'll happily do it, and I'd prefer to do it within your project than forking it and creating confusion and lots of extra work for myself ;)

@plasticuser That sounds great! Let us know if you have any questions or need help with anything.

I shall.

Fixed by commit c04a5190c468d0750ad19860da7e57ba250490ec.


Add float handling

https://github.com/openconfig/ondatra/blob/main/internal/gnmigen/gnmigen.go#L906

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 10 (10.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.3%) to 88.245%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pathgen/gnmigen.go
1
10
10.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
1569

Relevant Lines:
1778

💛  - Coveralls


MPFS TVS: driver and new uORB topic [1/4]

Added driver for MPFS TVS (temperature and voltage sensor) FPGA block and a new uORB topic (sensor_tvs). The driver is cherry-picked from 1.13_fc_only.
Reporting period is 500ms
TOPIC: sensor_tvs
 sensor_tvs
    timestamp: 276847808 (0.438826 seconds ago)
    device_id: 16711936 (Type: 0xFF, UNKNOWN:0 (0x01))
    temperature: 46.22501
    voltage: [987.87500, 1790.12500, 2499.87500]

saluki>

saluki> tvs status
INFO  [tvs] TVS voltage 1.0 V: 987.875 mV
INFO  [tvs] TVS voltage 1.8 V: 1786.250 mV
INFO  [tvs] TVS voltage 2.5 V: 2499.875 mV
INFO  [tvs] TVS temperature:   47.663 Celcius
saluki>

board/ssrc submodules needs to be updated once those are reviewed

Reviewed submodules updated


bug: Refreshing the page after closing the 200 contributor guide causes flickering

Actual Behaviour
The pop-up window at the bottom right corner of the current home page of the official website will flash when the page is refreshed after being closed manually.
Expected Behaviour
There should be no flashing of the refresh page guide window after clicking on the close guide window
Steps to reproduce it

Click to close 200 contributors Lead

Refresh the page

200 Contributor guide component flashes

Screenshots of the issue

Would you like to work on the issue?

This issue is relatively easy to solve, I will send a PR


Content in adjacently tagged enums does/can not use default when missing

I want to do as shown in below example, but it doesn't seem to work. I feel like there is something I'm missing or serde is missing.
I'd solve this by making two enums, one that is the real enum, and one that is only used (and seen) in a custom deserialize. But I don't like that pattern, so I want to know if there's something that can be improved here.
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=2bf7a3b1c76f9fe9d7741dce671011cf
// [dependencies]
// serde = {version = "1", features = ["derive"]}
// serde_json = "1"

use serde;
use serde_json;

#[derive(serde::Deserialize)]
pub struct Foo {
    pub bar: i64,
}

#[derive(serde::Deserialize, Default)]
pub struct NoContent {}

#[derive(serde::Deserialize)]
#[serde(tag = "tag", content = "content")]
#[serde(rename_all = "snake_case")]
pub enum Data {
    WithContent(Foo),
    WithOptContent(Option<NoContent>),
    // There is no way to make this work.
    WithoutContent(#[serde(default)] NoContent),
}

#[test]
fn deserialize() {
    let things = vec![
        r#"{"tag":"with_content", "content": { "bar": 10}}"#, // This works fine
        r#"{"tag":"with_opt_content"}"#, // This works because the content is wrapped with an Option
        r#"{"tag":"without_content" }"#, // This fails
    ];
    for i in things {
        serde_json::from_str::<Data>(i).unwrap_or_else(|_| panic!("failed on {:?}", i));
    }
}

Please take this question to one of the resources listed in https://github.com/serde-rs/serde/tree/v1.0.135#getting-help. Sorry that no one was able to provide guidance here.


Create Selection Sort in C

Thanks for contributing. It would be great if you could add description and also the issue number.

Thanks for contribution. Please, add the issue number in the description.

@Bhawna11agg : Thanks for contributing
Please use our contribution guidelines (https://github.com/fnplus/interview-techdev-guide/blob/master/CONTRIBUTING.md) for reference


distance wrong in triplogger

Hello!
I roughly see double the distance logged compared to my track in iSailor and can‘t find the issue.Where to start ?
Thanks Ron

How often do you update triplogger log, in milliseconds ?
And did you recalculate meters from triplogger to nautical miles in iSailor ?
1 m = 0.00054 nm


Author index page should have a link to delete an author

Scenario
Given an author named 'Alan Turing'
When users visit the authors index page
Then it should have link to delete an author
Hints
To delete a record following the REST paradigm, a DELETE request needs to be made.
As not many browsers support this directly, the DELETE call is made using JavaScript.
Newer versions of Rails support this using the turbo dependency.
Install it using rails importmap:install turbo:install stimulus:install (answer 'y' to overwrite existing files).
The following data attributes (read by JS) turn the usual GET request of a link into a DELETE (with confirmation messge):
<%= link_to 'Delete', author_path(author), data: {turbo_method: :delete, turbo_confirm: "Are you sure?"} %>
After an author is deleted Author.count should have decreased.
A destroy controller action is needed.
Here is the relevant section (5.13) of the Rails Guide. (Please be aware that the link_to syntax of the guide is outdated, see code above.)
Error

Expected to find css "a[data-turbo-method='delete'][href='/authors/1']" but there were no matches

18 exercise tests have passed. There are 44 in total. You will solve multiple at once towards the end.

Sorry, it's not working for me yet:
Scenario
Given an author named 'Alan Turing'
When users visit the authors index page
Then it should have link to delete an author
Error

Expected to find css "a[data-turbo-method='delete'][href='/authors/1']" but there were no matches

If you have problems solving this task, please don't hesitate to contact the teaching team!


Revert "Make command word case insensitive"

Reverts CS2103AUG2017-T10-B3/addressbook-level4#11

Hopefully this can undo. Because all of the built in tests were designed to fail if the command words are not lowercase.
To pass all the tests a lot of work needs to be done to rewrite all the test cases involved.


How to apply only for desktop version (no mobile)

Hey there, thanks for your plugin.
I am using it successfully with the desktop version of my project, but now I'm having problems with the mobile version (implemented with bootstrap) since it shouldn't work due to the different nature of the scroll usage.
Any suggestion how to selectively show the sticky sidebar depending on screen resolution?
Thanks in advance.

You can use stickySidebar.destroy(); below a specific screen size to disable the scroll.
Example:
if( $(window).innerWidth() < 750) {
stickySidebar.destroy();
}

I used the opposite logic: create only if bigger than.
Thanks, solved!

@Opanga @masiorama https://abouolia.github.io/sticky-sidebar/#minwidth

I completely missed that option! Thanks @abouolia !

@abouolia  I don't think the minwidth is working~


Fixed typo— "represents"

Thank you. I'll update it manually sometime later since it requires regenerating the static search index too.


Improve and fix some stuff in the upload.py file

When uploading a video there are some failure errors that occurs frequently. The aim of this PR is to fix those fails to have a uploading rate of 100%
Content of the PR:

Adds the cyan color as a debug color
Makes the remove_split_window skippable if the user wants it
Adds an uploading_wait to avoid a Timeout Exception whan a video takes more than 60s to get uploaded
Adds a Timeout Exception in the _set_video function to have better exception handling
Fixes the Something went wrong with the date picker bug (might be temporary)
Fixes the Failed to set schedule: Message: element not interactable bug

Good To Know:

I am currently working on the Failed to set schedule: Message: element not interactable:
Edit: Did some investigation, this error occurs when hour_to_click gets clicked and hour_to_click.is_displayed() is false (the wanted hour is not visible on the screen). So that means that the ScrollIntoView script does not work properly.
Edit 2: Noticed that relaunching the script when hour_to_click.is_displayed() is false solves the problem. Might be useful as a temporary fix but I will keep investigating to find the origin of the bug
Edit 3: The first scrollIntoView script is called too fast which can sometimes break. A time.sleep(1) solves the problem, I have pushed the fix. I don't know if it is a good idea to use time.sleep with selenium though

I have left a comment about an error than happens sometimes when trying to upload the video. Indeed we are waiting for a //video but it is never found. I have fixed this thing on my fork but did not push the changes because it might not be consistent with the way this package is coded.

I have not precised it but of course I am open for a review whenever you will have some time. @wkaisertexas


Added transaction receipts for slashings/rewards

Codecov Report

Merging #46 into master will decrease coverage by 1.19%.
The diff coverage is 0.66%.

@@            Coverage Diff            @@
##           master      #46     +/-   ##
=========================================
- Coverage   37.73%   36.54%   -1.2%
=========================================
  Files          20       20
  Lines        3230     3336    +106
=========================================
  Hits         1219     1219
- Misses       1873     1979    +106
  Partials      138      138

Impacted Files
Coverage Δ

primitives/state.go
15.56% <0%> (-1.46%)
:arrow_down:

beacon/statemanager.go
79.84% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Test of thread pool example plugin

The thread_pool example plugin (psi) has some tests, but it's not built nor running. Do we want to maintain this test or KIWF?
https://github.com/apache/trafficserver/tree/master/example/plugins/c-api/thread_pool
https://github.com/apache/trafficserver/tree/master/example/plugins/c-api/thread_pool/test/SDKTest

@masaori335 please create a PR to remove these plugins


ENH: empty pytorch cuda cache after termination

Is your feature request related to a problem? Please describe
Pytorch will cache GPU memory for fast allocation in the future, which is not suitable for xinference since we have other backends that use GPU memory.

solved


Fix broken json logging

Fixes https://github.com/cert-manager/cert-manager/issues/6768
In https://github.com/cert-manager/cert-manager/pull/6587 (https://github.com/cert-manager/cert-manager/pull/6587/files#diff-8f2f7b96261d29c4991c0025fde03c2edb0411a9aa7afaa834b5cf19837b5b2eL41), I updated the global Log definition in log.go.
I based the updated code on the deprecation message of the klogr.NewWithOptions() command that we were using.
It seems like the proposed replacement textlogger.NewLogger(textlogger.NewConfig()) does not get updated when we parse & set the logging flags.
This PR fixes this bug by using klog.Background() instead.
Before:
$ k -n cert-manager logs cert-manager-78dd8d6947-br2h8
{"ts":1708504242845.5793,"caller":"clientcmd/client_config.go:618","msg":"Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.","v":0}
I0221 08:30:42.851118       1 options.go:248] "enabling all experimental certificatesigningrequest controllers" logger="cert-manager"
I0221 08:30:42.851135       1 options.go:253] "enabling the sig-network Gateway API certificate-shim and HTTP-01 solver" logger="cert-manager"
I0221 08:30:42.851148       1 controller.go:89] "enabled controllers: [certificaterequests-approver certificaterequests-issuer-acme certificaterequests-issuer-ca certificaterequests-issuer-selfsigned certificaterequests-issuer-vault certificaterequests-issuer-venafi certificates-issuing certificates-key-manager certificates-metrics certificates-readiness certificates-request-manager certificates-revision-manager certificates-trigger certificatesigningrequests-issuer-acme certificatesigningrequests-issuer-ca certificatesigningrequests-issuer-selfsigned certificatesigningrequests-issuer-vault certificatesigningrequests-issuer-venafi challenges clusterissuers gateway-shim ingress-shim issuers orders]" logger="cert-manager.controller"
{"ts":1708504242853.3452,"caller":"leaderelection/leaderelection.go:250","msg":"attempting to acquire leader lease kube-system/cert-manager-controller...","v":0}

After:
$ k -n cert-manager logs cert-manager-78dd8d6947-64kcb
{"ts":1708504356076.2844,"caller":"app/controller.go:288","msg":"configured acme dns01 nameservers","v":2,"logger":"cert-manager.controller.build-context","nameservers":["<IP_ADDRESS>:53"]}
{"ts":1708504356076.3547,"caller":"clientcmd/client_config.go:618","msg":"Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.","v":0}
{"ts":1708504356085.3008,"caller":"options/options.go:248","msg":"enabling all experimental certificatesigningrequest controllers","v":0,"logger":"cert-manager"}
{"ts":1708504356085.322,"caller":"options/options.go:253","msg":"enabling the sig-network Gateway API certificate-shim and HTTP-01 solver","v":0,"logger":"cert-manager"}
{"ts":1708504356085.3352,"caller":"app/controller.go:89","msg":"enabled controllers: [certificaterequests-approver certificaterequests-issuer-acme certificaterequests-issuer-ca certificaterequests-issuer-selfsigned certificaterequests-issuer-vault certificaterequests-issuer-venafi certificates-issuing certificates-key-manager certificates-metrics certificates-readiness certificates-request-manager certificates-revision-manager certificates-trigger certificatesigningrequests-issuer-acme certificatesigningrequests-issuer-ca certificatesigningrequests-issuer-selfsigned certificatesigningrequests-issuer-vault certificatesigningrequests-issuer-venafi challenges clusterissuers gateway-shim ingress-shim issuers orders]","v":0,"logger":"cert-manager.controller"}
{"ts":1708504356085.3403,"caller":"app/controller.go:435","msg":"serving insecurely as tls certificate data not provided","v":1,"logger":"cert-manager.controller"}
{"ts":1708504356085.345,"caller":"app/controller.go:102","msg":"listening for insecure connections","v":2,"logger":"cert-manager.controller","address":"<IP_ADDRESS>:9402"}
{"ts":1708504356085.5166,"caller":"app/controller.go:182","msg":"starting leader election","v":2,"logger":"cert-manager.controller"}
{"ts":1708504356085.534,"caller":"app/controller.go:129","msg":"starting metrics server","v":2,"logger":"cert-manager.controller","address":"[::]:9402"}
{"ts":1708504356085.5508,"caller":"app/controller.go:175","msg":"starting healthz server","v":2,"logger":"cert-manager.controller","address":"[::]:9403"}
{"ts":1708504356087.196,"caller":"leaderelection/leaderelection.go:250","msg":"attempting to acquire leader lease kube-system/cert-manager-controller...","v":0}

Kind
/kind bug
Release Note
BUGFIX: Fixes issue with JSON-logging, where only a subset of the log messages were output as JSON.

/cherrypick release-1.14

/kind bug

Does this bug fix affect any released version of cert-manager?


SteamOS doesn't boot after 3.5 update

I set up rEFInd yesterday with steamos and windows 11. Everything worked perfectly. Today I decided to update SteamOS to 3.5. Some things happend and as of now I'm able to launch SteamOS using Volume + and Power and everything works Windows aswell.
When I turn on steamdeck I get the rEFInd menu but when I choose SteamOS icon it doesn't work. Windows works just fine tho and my NTFS sd card too. Is there a way to fix this?

Can you manually boot into SteamOS and show me the output of efibootmgr and what your refind.conf file looks like?

Sure I have to type efibootmgr into Konsole?

Yes. I need to see that efibootmgr output and compare it to your refind.conf file. You can see the contents of the in use refind config file with sudo nano /esp/efi/refind/refind.conf.

I see the problem. Change the line in the SteamOS stanza from firmware_bootnum 0001 to firmware_bootnum 0000.
Then press ctrl+x followed by 'y' to exit nano with changes saved. Reboot and all should be fixed.

I think I do something wrong when I do this and press Y I'm stil in konsole and when i exit it and type this again it still shows firmware_bootnum 0001

You may need to run this first before editing sudo steamos-readonly disable.
Or you can manually edit or recreate the refind.conf file in the /home/deck/.SteamDeck_rEFInd/GUI/ folder (hidden folder, by the way) and perform the Install Config step from the GUI.

It works! But right now when I shut down SteamOS it shutdowns and goes to black screen (screen still have light on) and I have to long press Power button for screen to shut down. Do you know why it happends or it's just minor inconvinience

I just checked and it doesn't happen on Windows

I'm not sure about that. Maybe some issue with 3.5 since that's not released as stable yet. If you're happy with the resolution, then please close the issue. Thanks.

Right that might be it. Thank you very much!


MFA Issue in LAB_03_Ex2_Configure_Conditional_Access_policies_for_AVD_ADDS

Module: 03
Lab/Demo: Exercise 2
Task: 04
Step: 04
Description of issue
aduser5 still asks for MFA
This happened with most of the students in my class

**
Conditional Access Policy

**

Repro steps:

Hi David,
the What if functionality of Azure AD CA should give a clear indication of the expected outcome - did you give this a try?
rgds
Marcin

Hi Marcin,
Several of the people STILL asks for MFA, the instructions say it shouldn't because it is Hybrid joined,

What is "What if functionality of Azure AD CA " ? where to find it? thanks

Hi David,
pls refer to https://learn.microsoft.com/en-us/azure/active-directory/conditional-access/what-if-tool
rgds
Marcin

Hi David,
pls let me know if you run into this problem again - this seems to work for me without any issues
rgds
Marcin

Hi,
20% of the class had this issue.
David Papkin
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: polichtm @.>
Sent: Friday, March 31, 2023 9:12:36 PM
To: MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual-Desktop @.>
Cc: David Papkin @.>; Author @.>
Subject: Re: [MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual-Desktop] MFA Issue in LAB_03_Ex2_Configure_Conditional_Access_policies_for_AVD_ADDS (Issue #231)
Hi David,
pls refer to https://learn.microsoft.com/en-us/azure/active-directory/conditional-access/what-if-tool
rgds
Marcin
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual-Desktop/issues/231#issuecomment-1491907105, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF2LQITWTEHHBY5EH6JLI5LW63J4JANCNFSM6AAAAAAWM3IBWM.
You are receiving this because you authored the thread.Message ID: @.***>

Will see what happens next class
David
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: polichtm @.>
Sent: Wednesday, April 12, 2023 7:41:40 PM
To: MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual-Desktop @.>
Cc: David Papkin @.>; Author @.>
Subject: Re: [MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual-Desktop] MFA Issue in LAB_03_Ex2_Configure_Conditional_Access_policies_for_AVD_ADDS (Issue #231)
Hi David,
pls let me know if you run into this problem again - this seems to work for me without any issues
rgds
Marcin
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual-Desktop/issues/231#issuecomment-1505123509, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF2LQIRDVPPCEVGIMLLWPXDXA2IHJANCNFSM6AAAAAAWM3IBWM.
You are receiving this because you authored the thread.Message ID: @.***>


Conflict error when installing package for SPFx 1.16.1

Category
[ ] Enhancement
[ ] Bug
[x] Question
Version
Please specify what version of the library you are using: [  3.12.0  ]
Please specify what version of the SPFx you are using: [  1.16.1  ]
Expected / Desired Behavior / Question
Create a new project with SPFx v1.16.1 using yeoman (yo @microsoft/sharepoint), and install the spfx-controls-react package, the installation should successful without any errors.
Observed Behavior
There were a lot of peer dependency, conflict errors in the console related to @fluentUI.
For screenshots, please find the attachment.
Steps to Reproduce

Create a new project with SPFx v1.16.1
Install the spfx-controls-react package using npm commands

Thanks!
Aditya Reddy.
spfx-controls-react error.docx

Hello @sallaadithya
This controls library is itself a SharePoint project. And the SPFX version of the project is 1.16.1.
This is a common pitfall of creating reusable sharepoint framework component libraries. SPFX libs should be set as peer dependencies out of SPFX project while having a SPFX project for testing, but such pattern are a lot harder to implement.
To sum up, I think you should stick to the 1.16 version of SPFX in your project. Unless you actually need the fixes that came from the patch version 1.16.1, it will be easier I guess.

Hi @sallaadithya,
Does the answer provided by @stevebeauge is what you've expected?
FYI I've tested installing latest version (3.15.0 which is based on SPFx 1.17.3) on a SPFx 1.17.4 project and no peer dependancies are displayed:


Usability: validate an initial draft structure for the pattern usage doc template

### Tasks
- [x] Shey / Zoe Independently review MTK research and pull out key themes/ideas
- [ ] Shey to do additional desk research on what competitors are doing
- [ ] Meet Zoe / Shey
- [ ] Zoe to generate structured discussion guide for interviews (e.g., what content is required in technical docs, what do users need to know) - aim for early May
- [ ] Collate & bring together to draft MDX template

@shey-v & I catching up tomorrow

Docs to review and draw themes/ideas out of
Pattern Lib Workshop:
https://www.figma.com/file/YPpNOKLf4WQDbOifDafoL5/Pattern-Library-Workshop?node-id=0-1&t=wlASsaqdSxJpg6FY-0
MTK Pattern Lib:
https://www.figma.com/file/53Sf8iOEq22zimhg1iKjdK/MTK-pattern-library?node-id=93-104559&t=f1aRNryanJO8vKky-0

Progress continuing, (see tasks above)

Discovery should cover exploration of and refining of definition of pattern

Discovery and research ongoing - meeting scheduled to catch up with Zoe this Weds to discuss findings

Zoe & Shey met to discuss independent desk research, found similar themes and now working on outlining a draft structure for the patterns page.

Research consolidated and shared with Robin. Shey working on competitor analysis.

Work in progress - drafting out a proposal for the general structure of patterns docs based off of research done.

Reached out to Oren Dayan and Julie East (Risk & Payments) to gather further insights on what is required from our consumers for pattern docs. In meantime, I'm putting together a draft template for various content blocks or 'sections' for the template in Figma.

@shey-v  i'm going to split this out since the usability aspect is on hold / no longer planned for this Q

May 12 - work in progess

May 17 - have socialised with Ben, Robin, James and Jo for feedback. Will be pausing this piece of work until the Nav item component is ready, which will be my main focus.


Create presentation materials for visiting lecture

There is visiting lecture on 29 September at TUT about Real-time open web technologies. Design a presentation about the given topic.
Definition of done

[ ] Create topic outline for presentation.
[ ] Organize topic outline for flow.
[ ] Create slides for each topic.

[ ] If possible, ask for peer review on slides.

[ ] Rehearse slides for flow.
[ ] Deliver lecture.

@brylie Could you provide a link to your material?

The slides are available on Github and can be viewed in online presentation mode](https://gnab.github.io/remark/remarkise?url=https%3A%2F%2Fraw.githubusercontent.com%2Fbrylie%2Frealtime-open-web%2Fmaster%2FREADME.md#1).


Document external auth token exchange

Documentation for this PR https://github.com/medplum/medplum/pull/1691/files
Proposal: should live in this document https://www.medplum.com/docs/auth/external-identity-providers

Yes, agree.
Stretch goal would be to create an alternate graphic of the "Auth Flow" diagram:

Current image: https://www.medplum.com/assets/images/external-identity-providers-79441e8d1e3cc003fa6ed4e0b06117ca.webp
Source: https://docs.google.com/presentation/d/1l92v3kMWQmMVhVRE8m1UavSpoK0Cz0MxPndTWqQDq30/edit#slide=id.g226463c901f_0_0

Steps 3-15 would all be replaced with one new step.

The end user makes a call to the /auth/exchange endpoint with their external access token
Medplum verifies the external access token with the external auth server
a. Optional details: Medplum verifies that the ClientApplication identityProvider details are valid
b. Optional details: Medplum calls the external auth /userinfo endpoint to verify credentials
On success, Medplum performs the login (similar to existing step 10)
Then send the Medplum access token back to the client (similar to existing steps 14 and 15)

Fixed by #1804


[typo]typo for module collations

At line 4472 of sql-2003-2.bnf,<module collation> may be typo for <module collations>.
<SQL-client module definition> ::=
		<module name clause> <language clause> <module authorization clause>
		[ <module path specification> ]
		[ <module transform group specification> ]
		[ <module collation> ]
		~~~~~~here~~~~~~~~~~~~~~~~
		[ <temporary table declaration>... ]
		<module contents>...

At line 4487 of sql-2003-2.bnf,<module collations> is defined,but <module collation> is not found.
<module collations> ::= <module collation specification>...

Hi
Looking at the usages of that term, I would prefer altering lines 4487
and 5753 to use 'module collation', i.e. in the singular.
The reason for my decision is that on line 4487 there is an ellipsis
(...) which indicates plural/repetition elsewhere in the doc. See lines
4421 and 4446 (and elsewhere) for example, and they don't use plural
terminology.
What do you think?

Ron Savage
savage.net.au
On 2020-03-10 22:25, GCer-Hidenori wrote:

At line 4472 of sql-2003-2.bnf, may be typo for .
 ::=

[  ]
[  ]
[  ]
[ <temporary table declaration>... ]
<module contents>...

At line 4487 of sql-2003-2.bnf,<module collations> is defined,but <module collation> is not found.

<module collations> ::= <module collation specification>...

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub [1], or unsubscribe [2].

Links:
[1]
https://github.com/ronsavage/SQL/issues/19?email_source=notifications&email_token=AAAFH6EMRMU756ATF5OIMOTRGYPSVA5CNFSM4LE4XDZKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IT3MMUQ
[2]
https://github.com/notifications/unsubscribe-auth/AAAFH6GQAHU3VSGHAHE72A3RGYPSVANCNFSM4LE4XDZA


kyocera-ecosys-3500-4500: init at 9.0

New package with drivers for various Kyocera printers.
Based heavily on the kyodialog package.
Description of changes
New driver package for some Kyocera printers, based on the drivers from https://www.kyoceradocumentsolutions.com/download/model_en.html?r=119&s=25&m=391&p=83 and the kyodialog package at https://github.com/NixOS/nixpkgs/tree/master/pkgs/misc/cups/drivers/kyodialog
This is my first attempt at sharing a package; I have it building and working on my test NixOS VM, but I'm still getting the hang of the Nix language, and I'm not 100% certain of the changes I've made as part of rewriting this to work as a derivation rather than an import (or even if those are the right words for what I've done…)
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg eval

Please follow the instructions at https://github.com/NixOS/nixpkgs/actions/runs/9165436983/job/25198875210?pr=313244

Done!

Additionally, please add a commit that adds yourself as a maintainer.

Also done! I'd been wary of doing this as I didn't want to commit to significant ongoing work, but it looks like that's not likely in practice here.

Finally, printer drivers often have cups somewhere in their name. Can I suggest cups-kyocera-ecosys-3500-4500 as the package name?

I've gone for "cups-kyocera-3500-4500"; I'd initially started working on this to get drivers for my own Kyocera Ecosys printer, but the driver package includes drivers for several printers from different lines too. I have no idea what Kyocera's conventions are for naming their printers or packaging their drivers, but the common theme for this package seems to be them having 3500, 4000 or 4500 in the name.


CORE-9630 Finality takes a transaction builder

UtxoLedgerService.finalize and ConsensualLedgerService.finalize both
now take in their respective transaction builders instead of signed
transactions.
The respective finality flows now sign the builders.
toSignedTransaction has been made internal and removed from the API.
The "cant sign twice" code has been removed from the builders because
we control the signing of them, making the code redundant (as it was there
to help developers).

The interop team mentioned that they will need a way to save draft transactions somehow. I think that means that they will need a way to get a hold on the transaction id somehow before signing it. I am not too sure about the details and its current priority, but if the transaction builder gets finalized instead of the signed transaction, this will be more tricky.
@jmacmahonr3 @mnesbit

am not too sure about the details and its current priority, but if the transaction builder gets finalized instead of the signed transaction, this will be more tric

Hi @vlajos, @lankydan & @mnesbit
The ability to advertise an unsigned tx or a draft tx as we call it, is critical for alot of cross network patterns and will be our biggest ask for 5.1

Ok, @mnesbit, @blsemo what do we want to do in this case? If we want to provide the functionality that @jmacmahonr3 is asking for then we either: scrap this PR or introduce a callback that executes before signing the transaction. I'd say leaving the current code is probably best because the callback is just a different way of signing it before passing it into finalize. That being said, if most people are ok with the builder being passed in, then having the callback to cover @jmacmahonr3's scenario doesn't sound too bad.

Hi @vlajos, @lankydan & @mnesbit The ability to advertise an unsigned tx or a draft tx as we call it, is critical for alot of cross network patterns and will be our biggest ask for 5.1

Unsigned as in completely unsigned? Or is signed by the initiator ok? In the latter case, just scrapping this PR would do it, in the former, we need to introduce a new intermediate format (or the callback Dan mentioned).

Completely unsigned. We need something that represents a transaction proposal that a later notarized transaction (or fully signed transaction ) can be proven to be fulfilling. In classic c4's utxo ledger this has traditionally meant we have a notary's signature over the tx id

Closing this PR. Discussed with @blsemo @mnesbit to not do this as not to cause issues with interop.


Updated typo for remove function

In the code example, the wrong name was used. Should be onRemove.

Docs Build status updates of commit 6faf44b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

BingMaps/v8-web-control/map-control-api/customoverlay-class.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Evaluate GDScript dynamically

I know about #8003, but that seems to be barking up the wrong tree to be honest...
I'm building an application in godot and I'm wanting to provide a way for users to extend the application with "addons". So for example, say I want to allow developers to add their own custom UI to my application... Well, I was hoping that I'd be able to expose GDScript directly to them, so they can simply execute GDScript, in runtime to build a custom UI.
I know this isn't exactly a traditional use case of the game engine, and I'm only really planning to use Godot due to the 3d engine which it provides... The programming language isn't super important as it is likely plausible I can build my own interprated language in GDScript anyway, but it would be somewhat nice if there were native support for running GDScript in runtime.

Well, you can do that already by creating a GDScript resource and setting its source code. Then you can instance it and call its functions like any other script.

Huh? How do you add a random GDScript on the file system to the resources?

yep that looks right. There's other ways too
http://docs.godotengine.org/en/3.0/classes/class_gdscript.html?highlight=gdscript
var MyClass = load("myclass.gd")
var instance = MyClass.new()
assert(instance.get_script() == MyClass)

@sancarn please close when you can.

i'd also recommend you just have your users learn gdscript. I'm not sure how I'd go about sandboxing users...

@pgruenbacher this works as a dynamic way to execute scripts, but load only works on the game's own resources AFAIK

good point... what we'd need to test is if load works on the User data.... my guess is load("user://myclass.gd") should still work... So I'd say yes.
as for sandboxing, I think this is mostly possible. The gdscript::init() method will inject Global variables into the scope of the script. We could maybe propose a GDScriptBox resource subclass that does not inject global variables automatically ( but still allows for manual injection). That will prevent game users from accidentally making changes to the root node or Engine.

Huh? Can't you simply load from a file system path. E.G. load("/some/file/dir") ?

you can try. I haven't tested.

Just tested, you can! :D
Though I have looked around a bit and I wouldn't even know how to do a directory tree in gdscript... I can understand it... It's not common practice for games to be looking at the file system in general...
Anyway...
Conclusions
var MyClass = load("some/cool/file.gd") #this can also be a user:// or res:// path
MyClass.staticMethod()
var instance = MyClass.new()
#dostuff

or can also use new gdscripts
var script = GDScript.new()
script.source_code = "func say_hello():\n\tprint(\"Hello!\")"
script.reload()
var script_instance = script.new()
script_instance.call("say_hello")

Thanks both for the input!

Hi guys, may I reopen the topic?
What about now in 2022... I now previous answers are still working but is there a way to sandbox dynamically evaluated scripts?

Hi guys, may I reopen the topic? What about now in 2022... I now previous answers are still working but is there a way to sandbox dynamically evaluated scripts?

This is being tracked in https://github.com/godotengine/godot-proposals/issues/389. There are no immediate plans to work on this for 4.0 though.

Hi guys, may I reopen the topic? What about now in 2022... I now previous answers are still working but is there a way to sandbox dynamically evaluated scripts?

This is being tracked in godotengine/godot-proposals#389. There are no immediate plans to work on this for 4.0 though.

does that mean that the dynamic evaluation is still working in Godot 4 or was it disabled?


KMS client activation data

This doesn't really apply to home users since they don't use KMS activation, but I figured maybe it could still be included in this script, because I use it myself. 😸
Turn off KMS Client Online AVS Validation

This policy setting lets you opt-out of sending KMS client activation data to Microsoft automatically. Enabling this setting prevents this computer from sending data to Microsoft regarding its activation state.
If you disable or do not configure this policy setting, KMS client activation data will be sent to Microsoft services when this device activates.

I haven't done any tests to see if this is affected if you have already disabled telemetry, but I just end up setting it to be safe 😆

You can disable it and use HWIDGEN instead which does not even connect to the internet at all.
Even on a legitimate system this can be turned off because AVS Validation is a part of the telemetry process to solve possible "activation problems".


[Smoke Tests] PSTN smoke tests break the js - communication pipeline

This is a subtask of #21344
Failing tests:

getPurchasedPhoneNumber.js
getPurchasedPhoneNumbers.js
updatePhoneNumberCapabilities.js

Related ADO task: https://skype.visualstudio.com/SPOOL/_workitems/edit/3076112

Smoke tests have been disabled globally for now, future work is tracked here: https://github.com/Azure/azure-sdk-for-js/issues/24440


feat: added new release action to create semantic releases

This action is used to create releases in a number of other repositories.

https://github.com/mozilla-it/deploy-actions/tree/main
https://github.com/mozilla/terraform-modules

Am I right that we'd use semantic-release to automatically generate changelogs in our Github releases?
If so, looks like that relies on us using conventional commits. Perhaps we should document that somewhere in this project that we expect people to use conventional commits.
Another way we could achieve this is by using Github labels + configuring the way automatically generated release notes are generated.


Font not being applied in Action bar - support library 23.2.1

Hi, I have tested new support library 23.2.1 with Caligraphi 2.1.0 and font is not applied in ActionBars

Title
SearchView widget

Menu overFlow is ok.
Screenshots:
support library 23.1.1 http://snag.gy/AVbgn.jpg
support library 23.2.1 http://snag.gy/HvsKr.jpg
Code is here https://github.com/sonavolob/ABIconTest.

@sonavolob try the snapshot, that has a fix for Toolbars.

@chrisjenx Actually I have no experience with Maven so I have downloaded last snapshot from Calli Git repo and compiled it separately. Than I used aar file from "Calligraphy-master\calligraphy\build\outputs\aar" and imported it to ABIconTest project.
Result is the same...

I think you should switch to use Toolbar as recommended by majority of android devs, Calligraphy works fine with toolbar in my case.

@staroflead Thank you for directing me. Replacing ActionBar for Toolbar did the thing (it messed up some other things, but corrected problems with Caligrhaphy).

@sonavolob you are welcome

Closing. 2.2.0 should work with the Toolbar, just to Clarify, I will not be supporting the ActionBar


dns: hook DNSBackend for A/AAAA records lookup instead of using Authority records

Configure coredns to use lookup logic directly from DNSBackend instead
of using Authority records.
This prevents creating copies of A/AAAA record entries and we follow
desired logic for lookup.

@mheon Following prevents creating copies and makes sure that we use lookup logic from DNSBackend as it is.


Clarify Primary Dependency Management Tool in pyproject.toml

Issue Description:
The pyproject.toml file for the "llm_vm" project currently includes configurations for both Poetry and Setuptools as potential dependency management tools. This setup may lead to confusion among contributors and users. This issue aims to clarify the choice of the primary dependency management tool for this project.
Proposal:

Choose a Primary Tool: We should decide whether Poetry or Setuptools will serve as the primary dependency management tool for the "llm_vm" project. Having a single primary tool simplifies the development process and avoids potential conflicts.

Update the Configuration: Once a decision is made, update the pyproject.toml file to reflect the choice. Remove or comment out any sections related to the tool that won't be used as the primary one.

Document the Choice: In the project's documentation or README, we should clearly state which dependency management tool is being used as the primary one. Provide instructions on how contributors and users should interact with it.

Discussion:

Are there specific reasons for using both Poetry and Setuptools in the same file for this project?
What benefits or advantages does each tool bring to the "llm_vm" project?
Are there any potential drawbacks or conflicts in using both tools simultaneously?

This issue is intended to ensure clarity and consistency in managing dependencies for the "llm_vm" project. Please share your thoughts and suggestions regarding the choice of the primary dependency management tool.
Thank you for your input!

i didnt see this ticket till now, i've been using setup tools features because it was slightly easier to figure out the documentation, and the poetry one was an experiment left dangling.
We can take either path, as long as theres a clear case.


File upload support

Some hackathons may require file upload support. Hence it will be great if we could add file upload routes also in this repository.

Already supported: https://github.com/sahat/hackathon-starter/blob/master/app.js#L173

Currently it is local device, I was suggesting something like cloud storage.

If you are looking for cloud storage integration, the basis for Google Drive integration is at https://github.com/sahat/hackathon-starter/blob/master/app.js#L179 .  You can modify the related controller to perform various functions including file uploads.
Cheers :)


fix selecting http(s) agent

Hello.
I'm trying to use speedtest.net behind corporate proxy.
I reproduced issue #52
This pull request fixes the issue in my environment.
According to readme of http-proxy-agent and https-proxy-agent,
agent should be selected based on the protocol of the request url.

Thanks! :+1:


Can photo album be customized and translated into Chinese, just like wechat

The default album Recent is in Chinese
Lisk Recent show 图片和视频 Camera show相机

#28 .


Use remote address if local address is null

I think it's better to use stream_socket_* everywhere instead of socket_*, so we don't require the socket extension additionally.

Thanks!


ARXIVNG-1215 ARXIVNG-1270 configure URLs, add help info

Most of ARXIVNG-1270 was done along with ARXIVNG-1259; this PR cleans up some of the leftovers in file upload help text. Imperfect but better.
I chose to place these particular URLs in the submit repo directly because they seemed to apply only within the submission process; happy to move them to base if they are deemed more general.
Screenshot of the new help info for file upload:

Pull Request Test Coverage Report for Build 356

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 70.136%

Totals

Change from base Build 354:
0.07%

Covered Lines:
1132

Relevant Lines:
1614

💛  - Coveralls