TypeError: Cannot assign array to property

Prerequisites
We have a SOAP method getProduct() where the response contains property "externalAttributes", which is defined as a Map struct in the WSDL:
<element name="externalAttributes" nillable="true" type="apachesoap:Map"/>
In the generated proxy class for the response, I see the property generated with the appropriate typehint:
/**
 * The externalAttributes
 * Meta information extracted from the WSDL
 * - nillable: true
 * @var \My\Namespace\Struct\Map|null
 */
protected ?\My\Namespace\Struct\Map $externalAttributes = null;

The Map struct class is properly generated and extends WsdlToPhp\PackageBase\AbstractStructBase.
Error message
Now, when I call the method doing the actual SOAP call, I'm getting this exception:

TypeError  Cannot assign array to property My\Namespace\Struct\ProductAttribute::$externalAttributes of type ?My\Namespace\Struct\Map

SOAP response
In the raw SOAP response, I see the data represented like this:
<externalAttributes href="#id55"/>
...a gazillion nodes later...
<multiRef xmlns:ns31="http://xml.apache.org/xml-soap" xmlns:soapenc="http://schemas.xmlsoap.org/soap/encoding/" id="id55" soapenc:root="0" soapenv:encodingStyle="http://schemas.xmlsoap.org/soap/encoding/" xsi:type="ns31:Map">\n
  <item>\n
    <key xsi:type="soapenc:string">LNG</key>\n
    <value href="#id97"/>\n
  </item>\n
  <item>\n
    <key xsi:type="soapenc:string">CDB</key>\n
    <value href="#id98"/>\n
  </item>\n
</multiRef>

Expectation
I would have expected the data to be parsed into the Map class.
What actually happens
While trying to interpret the response and hydrating the proxy class, the client seems to fail identifying the Map struct as such and simply converts the data into an associative array.
When I remove the type hints from the response struct class, the error message goes away and I can see the parsed data structure, which looks like this:
"externalAttributes": [
        "LNG" => SoapVar {#2857
          +enc_type: 0
          +enc_value: My\Namespace\Struct\ExternalAttribute^ {#2856
            #ID: 44
            #attributeId: 103
            #attributeName: "realaccount"
            #attributeType: "String"
            #productAttribute: My\Namespace\Struct\ProductAttribute^ {#2855}
            #systemId: "LNG"
          }
          +enc_stype: "ExternalAttribute"
          +enc_ns: "//productcatalog/beans"
          +enc_name: null
          +enc_namens: null
        }
        "CDB" => SoapVar {#2859
          +enc_type: 0
          +enc_value: My\Namespace\Struct\ExternalAttribute^ {#2858
            #ID: 517
            #attributeId: 103
            #attributeName: "realaccount"
            #attributeType: "String"
            #productAttribute: My\Namespace\Struct\ProductAttribute^ {#2855}
            #systemId: "CDB"
          }
          +enc_stype: "ExternalAttribute"
          +enc_ns: "//productcatalog/beans"
          +enc_name: null
          +enc_namens: null
        }
      ]

I am currently using PHP8.1, wsdltophp/packagebase:5.0.3 and PackageGenerator 4.1.7.
Is there a possible workaround other than to deactivate strict typing?

From my point of view, the generated classes match the WSDL definition.
I'm wondering if the SOAP response is well defined and if it does match the WSDL definition.
Maybe if I was able to send the request by myself, maybe I would be able to understand where the issue comes from and if it can be worked around.
Let me know then.

Thanks for looking into it. Unfortunately, the service is within a private network, so you can't access it from the outside. But if it helps, I could probably provide you with a complete protocol of a request/response cycle in private.


wl: Fix destroying CSDs in the correct order

On Wayland we implement Client-Side Decorations if the compositors do not
implement SSDs.  In that case, the destructors of the surfaces were called
in the wrong order, leading to a dereference of an already freed object.
We need to first destroy the subsurface before destroying the parent surface.
Related PR on kitty: https://github.com/kovidgoyal/kitty/pull/3066
Related issue on kitty: https://github.com/kovidgoyal/kitty/issues/3051

I think that the currently implemented CSD are unusable, at least on GNOME Shell. I proposed https://github.com/glfw/glfw/pull/1693 to implement "proper" client-side decorations on Wayland, including the usual functionality like min/max/close and additional fixes for resizing glitches.

I fully agree with you and would love to see your PR merged. However think it would be best to merge this simple fix first since it shouldn't take much time to review.


Reject uint64 Client.Point.Field values

fix for issue #4249
tested by using the example client write code from https://godoc.org/github.com/influxdb/influxdb/client#example-Client-Write
once default, once replacing the value with a uint64
go test in /client passes, same as in the root of the repo.
(apologies if i've used the below incorrectly)

[ ] CHANGELOG.md updated
[ ] Rebased/mergable
[ ] Tests pass
[x] Sign CLA (if not already signed)

Thanks @arussellsaw - do you know if this type checking impacts performance?

Impacts it in a significant degree, that is.

I wrote a quick benchmark that essentially puts the client.Write() and result checking of TestClient_Write() into a b.N loop, here are the results i found:
with type checking: BenchmarkWrite	   10000	    136996 ns/op	    5982 B/op	      85 allocs/op
without type checking: BenchmarkWrite	   10000	    100105 ns/op	    5982 B/op	      85 allocs/op

Just ran a quick benchmark around client.Write() with a single integer point, one with, one without the checker function
Without checker
BenchmarkWrite	   10000	    151286 ns/op	    6979 B/op	     101 allocs/op

With checker
BenchmarkWrite	   10000	    161825 ns/op	    6997 B/op	     101 allocs/op

verdict: a bit, but not too much.

Cool. Would you consider adding that benchmark program to the PR?

it's basically a copy of TestClient_Write() with the write and checking in a b.N loop

Please rebase if you can and we will take another look. Thanks @arussellsaw

after being reminded by @e-dard in the Gophers Slack i've updated this PR! if you're still interested please review.

:bowtie:

tidied commits for good measure

My tests passed... can't find any actual go test failures. think it might be this?
Error deleting container: Error response from daemon: Cannot destroy container 2d2df6d5e8eafc4ca4439913f69d1a2babc5816351c24dde9ee87bb9b63068fc: Driver btrfs failed to remove root filesystem 2d2df6d5e8eafc4ca4439913f69d1a2babc5816351c24dde9ee87bb9b63068fc: Failed to destroy btrfs snapshot: operation not permitted

[build]

@e-dard spotted it:
 --- FAIL: TestServer_BackupAndRestore (5.38s)
    backup_restore_test.go:87: query results wrong:
            exp: {"results":[{"series":[{"name":"myseries","columns":["time","host","value"],"values":[["1970-01-01T00:00:00.001Z","A",23]]}]}]}
            got: {"results":[{"error":"database not found: mydb"}]}

Looks like TestServer_BackupAndRestore failed on the last run, which is a known flaky test (#5282)

@arussellsaw you need to add an entry to the CHANGELOG, which will hopefully kick off a healthy build.

done, hopefully correctly...? didn't see anything in the docs about updating it so i just followed existing style

TestServer_BackupAndRestore failed again. i'll wait for someone from influx to turn up and rebuild.

All good. @otoolep what are your thoughts?

oh seems like @otoolep isn't at influx anymore, @jwilder can i get your input?

is this thing on?

:+1:


Dynamic routes with required parameters are not generated correctly when router.trailingSlash is true

Versions

nuxt: 2.15.8
node: 16.17.0

Reproduction
router-test.zip (Sample project)

Steps to reproduce

Download the sample project that I created for this issue
Run npm run build command (Of course npm install first)
Go to .nuxt/routes.json
The route was generated with the path /users/:id
Go to nuxt.config.js and set trailingSlash to true in router config
Go to step 2 and repeat from there until step 4
When you reach 4. step, the route was generated with the path /users/:id?/ instead /users/:id/

What is Expected?
A route should be generated with the path /users/:id/
What is actually happening?
A route was generated with the path /users/:id?/ (with question mark)

We are approaching the Nuxt 2 EOL date (June 30, 2024) - see this article for more information. This is advance warning that I'm going to close this issue then, as it's currently marked as a Nuxt 2 related bug.
If it's a critical or security issue, please do comment and let me know, in case it is possible to address it before the EOL date.
If it's a an issue you think is relevant for Nuxt 3, please feel free to open a fresh issue (or just comment here so I can update labels, etc.). 🙏
Thank you for your understanding and for your contribution to Nuxt. ❤

It's the day, at last. Nuxt 2 is now marked end-of-life.
My apologies we never got round to resolving this bug in the 2.x branch, but we do have to draw a line somewhere. Again, if you think this is still relevant for Nuxt going forward, please feel free to open a fresh issue (or just comment here so I can update labels, etc.). 🙏


glimpse doesn't work

sc <- spark_connect(master = "local")
iris_tbl <- copy_to(sc, iris)
glimpse(iris_tbl)
# Observations: 150
# Variables: 5
# Error in if (width[i] <= max_width[i]) next :
#   missing value where TRUE/FALSE needed

It looks like the issue here (if I understand correctly) is that we dispatch to glimpse.tbl rather than a more appropriate method.
@hadley, our Spark tables have class:
Browse[3]> class(x)
[1] "tbl_spark" "tbl_sql"   "tbl_lazy"  "tbl"

Is there some package that should be providing a glimpse method for tbl_sqls or tbl_lazys? Or do we need to just implement glimpse ourselves for tbl_sparks?

Looking at glimpse.tbl I think it might work if we provide a type_sum.tbl_spark and request a change upstream to use tbl_vars() instead of names() around https://github.com/tidyverse/tibble/blob/81ea9560bfe9abc34210484720c13688ce3e1589/R/glimpse.R#L46-L47

That won't be quite enough -- the vapply() call would need to be changed too, as that will operate on the src and ops variables in the table object. I think there needs to be a separate method altogether for tbl_sqls.

@kevinushey right, just realized type_sum is applied to each column...

tbl_sql() should really get a glimpse method — can you please file an issue in dplyr?

Will do!


Use streamreader instead of string to load xml

fixes #985
If you have URL characters in your project path like C:\Project\Project%20\proj.csproj XMLTextReader.Read() method try to unescape these characters. In result the build is failed because it doesn't find the path. Workaround is to load xml from StreamReader instead of string.

Hmm, I wonder if the tests fail because the stream is not disposed. Can you try adding the stream in the using statement?

So . . . was the old way doing encoding detection based on XML magic, and the new way now isn't? Or is there another explanation for those test failures?

@rainersigwald, for now we use XmlTextReader to import the xml. XmlTextReader has the ability to read the encoding from string. But since we now read from a stream it can't read the encoding and use a default one. This is why these tests are failing.

I have added a commit to change from XmlTextReader to XmlReader. I've tested with url characters like %20 and all works fine

@dotnet-bot test this please

Ah, nice detective work @maddin2016.
Also

Starting with the .NET Framework 2.0, we recommend that you use the System.Xml.XmlReader class instead.

😆 Well, that class was deprecated so recently!

:thumbsup: :wink:

@dotnet-bot test this please
(the tests should fail after #1189 made the location stuff work again)

Actually, I think the tests were disabled in 430a0515214a9d61fb447ecd53a8ce944866c4c5.  @maddin2016 can you undo your changes to the ElementLocation_Tests.cs file?  The tests should be fixed now via #1189.

@maddin2016 also revert decd35176d01f022a85cf8a57fc4d8b681116186 please--the current state should fail since the column-number problem should now be fixed.

🙌 🎉

Thanks a lot for this contribution!


[Reproducers] Add FileCollector support to DependencyTracker

For reproducers in LLDB, we need to collect module dependency
information coming from the clang importer. However, we cannot override
the dependency collector, like we do in LLDB, because its interface is
completely hidden in the clang importer. The solution is to pass the
FileCollector through the DependencyTracker.

LLDB PR: https://github.com/apple/swift-lldb/pull/1836

I'm confused here. So now we're collecting the same list of files twice? That seems like there's a better answer somewhere else.
Also, this should probably go to master rather than master-next, no?

I'm confused here. So now we're collecting the same list of files twice? That seems like there's a better answer somewhere else.

Yes and no. The FileCollector for LLDB is global and we create more than once instance of the ClangImporter. The list of files in the FileCollector is the union of all the files seen by the clang importer (and everything else in LLDB).
I'm haven't checked this, but I can also imagine that the files needed by the reproducers don't have to be identical to the Swift dependency files. For example, in order to guarantee that clang behaves the same way during replay, we might have to capture module maps that aren't used.
Finally, I'm sure we could switch over Swift to use the FileCollector as well, though it would be more heavy-weight for the regular (non-reproducer) scenario. Currently a list of files is all we need (because we only generate the dependency files from it?) while the FileCollector keeps track of file mappings, to be able to write a VFS overlay in the end.
However, even if we did that, we'd need to pull in that information from Swift somehow. With the ClangImporter using LLDB's file collector, we're essentially being pushed the information as soon as it's available, similar to how clang does this. That makes things a lot easier when we're in a signal handler after a crash.

Also, this should probably go to master rather than master-next, no?

I can't work on master because I have dependencies in clang and llvm that shouldn't end up on the release branch. Technically stable could work, but that'd mean they'd no longer be aliases for the release branch and at this point in the release cycle it seems easier to just work on master-next.

Yeah, okay. LLVM and Clang stable really do benefit from staying basically the same as the release branch.
Can I take a look at the solution in Clang? If LLDB's using a custom subclass of DependencyCollector there, maybe Swift should either conditionally or always do that too. Or it could chain to one of those rather than talking directly to FileCollector. The way you've set things up here is okay, but it does feel ad hoc. (And doesn't have any tests in the Swift repo.)

Can I take a look at the solution in Clang? If LLDB's using a custom subclass of DependencyCollector there, maybe Swift should either conditionally or always do that too. Or it could chain to one of those rather than talking directly to FileCollector. The way you've set things up here is okay, but it does feel ad hoc. (And doesn't have any tests in the Swift repo.)

Yeah, for Clang we just subclass the ModuleDependencyCollectorAdaptor and basically make everything a NOOP, except for adding the files to our FileCollector. I'd be totally fine doing that here too, but it seems like we made quite the effort to hide the ClangImporterDependencyCollector, presumably because you don't want to leak the corresponding Clang header from ModuleLoader.h .

Yeah, that's pretty much the reason. (Also, hm, why ModuleDependencyCollector specifically for yours?)
Okay, I guess you're right: it makes more sense for Swift to expose an API that uses llvm::FileCollector than it does to expose one that uses clang::DependencyCollector, and it's not really that much extra work to make the other list we're not going to use.


Add occupancy apis to driver/result and driver/safe

Detailed here https://docs.nvidia.com/cuda/cuda-driver-api/group__CUDA__OCCUPANCY.html#group__CUDA__OCCUPANCY

can we get an update on this? is there a pr?

Addressed by pr above, thanks @Jark5455 !


Update dbmovement-scenario-goldenconfig.md

Just switched the order since self-serve env type is majority and customers without identifying the env type, getting into the incorrect section of the doc.

:white_check_mark: Organization membership check passed
Requester is not a member of the MicrosoftDocs organization.

@surekota, this is pending CLA approval before it can be published. Thanks!


bluebird? issues.

I thought I would keep powering through after the last issue. On the next section after adding Article.js there is an async issue that is past my pay grade. Upon connecting and running, you get a graphiql error that looks like this:
{"errors":[{"message":"A value undefined was yielded that could not be treated as a promise\n\n    See http://goo.gl/MqrFmX\n\nFrom coroutine:\n    at Promise.coroutine (/Users/klik/Projects/fuse-reindex-mongodb-tut/node_modules/bluebird/js/release/generators.js:194:17)\n    at Object.<anonymous> (/Users/klik/Projects/fuse-reindex-mongodb-tut/src/index.js:22:21)\n    at Module._compile (module.js:541:32)\n    at loader (/Users/klik/Projects/fuse-reindex-mongodb-tut/node_modules/babel-register/lib/node.js:144:5)\n    at Object.require.extensions.(anonymous function) [as .js] (/Users/klik/Projects/fuse-reindex-mongodb-tut/node_modules/babel-register/lib/node.js:154:7)\n    at Module.load (module.js:458:32)\n    at tryModuleLoad (module.js:417:12)\n    at Function.Module._load (module.js:409:3)\n    at Function.Module.runMain (module.js:575:10)"}]}
I tried following the error guidance here but like i said, i dont have the experience to figure out how to apply this yet. Any ideas?
Thank you.

This might be related.
https://github.com/babel/babel-loader

went back to it. just adding bluebird seems to do the trick. not getting additional issue right now. Thanks.


mapWriter, castWriter

This merge request includes 3 distinct but related additions:

mapWriterT definition modeled after one in Haskell
castWriterT definition. A special case of mapWriterT where mapping function is identity. Basically, it swaps Monoid in a monad keeping same types for state and carrier.
writer monad definition modeled after one in Haskell. Additionally mapWriter and castWriter definitions provided based on their transformer equivalents.

All definitions are organized into separate sections, so you can cherry-pick.
I have not specified universes implicitly since I am not comfortable with Universe Polymorphism. I tried it, but using mapWriter was causing "universe inconsistency" errors. Perhaps I did it wrong?

Are the implicit arguments and argument order correct for the definitions? It seems odd to have A before the Monoid instance.
Other than that, it looks pretty reasonable. The big issue with the monads is still the reasoning though.

I've changed arguments order using general pattern: W Monoid_W m A.
I am unsure if it better to have W W' Monoid_W Monoid W' or W Monoid_W W' Monoid W'

Thanks. I'm updating Coq now. Will confirm that it builds and merge when I finish.

I made a few cosmetic changes on the vzaliva-v8.6-writer branch. I'm not sure how to incorporate those changes here. I'm going to merge this and then merge that PR.


[CHORE] Keep all providers in one top level providers.tsx file

assign me sir

Assigned @wraith2009


Issue with k8s.io/docs/tasks/configure-pod-container/

This is a Bug Report

Problem:
Page does not contain links describing how to expose ports from a Pod.
Proposed Solution:
Add a link to another page.
Page to Update:
https://kubernetes.io/docs/tasks/configure-pod-container/

NA

/kind feature
/language en

/priority backlog


chore: add pr & issue templates

This adds PR and Issue templates that will be used when one creates a bug report/feature request issue, or a pull request.

Closes #29

are you going to also pr this into pengin, pilfer and docs/elusite as discussed in the issue?

are you going to also pr this into pengin, pilfer and docs/elusite as discussed in the issue?

Yes

good pr 11/10 would review again

LGTM. remember to yoink to other repos

I'll be waiting for your other 4 prs, glhf


Try and reduce the generated FCPXML size to increase performance

Currently I think there's a lot of duplicate data in the generated FCPXML, which could slow down FCPX when you're attempting to import thousands of clips.

This has been addressed in v1.0.1.


npm install of this project fails in a weird way

To Reproduce
I create this package.json
{
  "name": "foo",
  "version": "1.0.0",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "description": "",
  "dependencies": {
    "signify-polaris-web": "github:WebOfTrust/polaris-web"
  }
}

Then I run npm install of that project.  Then I try and import anything from that package and it fails.  Leading to discovering this:
daidoji@LAPTOP-5KEUASIG:~/foo$ tree node_modules/
node_modules/
└── signify-polaris-web
    ├── LICENSE
    ├── README.md
    └── package.json

1 directory, 3 files

All the files aren't being produced in this package for some reason in npm.  @lenkan points out in the vlei-developers meeting that: In this PR https://github.com/WebOfTrust/polaris-web/pull/26/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519L12 the package was changed so that it possibly no longer builds as it builds in a post-install script.
This issue is to track this in case anyone else runs into this in the future.  Two more logs attached for extra information.
daidoji@LAPTOP-5KEUASIG:~/foo$ npm install --verbose                                                                                npm verbose cli /home/daidoji/.nvm/versions/node/v21.5.0/bin/node /home/daidoji/.nvm/versions/node/v21.5.0/bin/npm                                                                          npm info using<EMAIL_ADDRESS>                                                                                npm info using<EMAIL_ADDRESS>                                                                                npm verbose title npm install                                                                                npm verbose argv "install" "--loglevel" "verbose"                                                                                npm verbose logfile logs-max:10 dir:/home/daidoji/.npm/_logs/2024-11-19T16_52_53_755Z-                                                                                npm verbose logfile /home/daidoji/.npm/_logs/2024-11-19T16_52_53_755Z-debug-0.log                                                                                npm http fetch POST 200 https://registry.npmjs.org/-/npm/v1/security/advisories/bulk 263ms                                                                                up to date, audited 2 packages in 722ms                                                                                found 0 vulnerabilities                                                                                npm verbose cwd /home/daidoji/foo                                                                                npm verbose os Linux <IP_ADDRESS>-microsoft-standard-WSL2                                                                                npm verbose node v21.5.0                                                                                npm verbose npm  v10.9.0                                                                                npm verbose exit 0                                                                                npm info ok                                                                                daidoji@LAPTOP-5KEUASIG:~/foo$ rm -r node_modules/                                                                                daidoji@LAPTOP-5KEUASIG:~/foo$ npm install --verbose                                                                                npm verbose cli /home/daidoji/.nvm/versions/node/v21.5.0/bin/node /home/daidoji/.nvm/versions/node/v21.5.0/bin/npm                                                                          npm info using<EMAIL_ADDRESS>                                                                                npm info using<EMAIL_ADDRESS>npm verbose title npm install
npm verbose argv "install" "--loglevel" "verbose"
npm verbose logfile logs-max:10 dir:/home/daidoji/.npm/_logs/2024-11-19T16_53_13_677Z-
npm verbose logfile /home/daidoji/.npm/_logs/2024-11-19T16_53_13_677Z-debug-0.log
npm http fetch POST 200 https://registry.npmjs.org/-/npm/v1/security/advisories/bulk 282ms
npm http fetch GET 200 https://codeload.github.com/WebOfTrust/polaris-web/tar.gz/74be1c3ba361c3c7c49d7cc083abaa90a5dea1b0 224ms (cache revalidated)

added 1 package, and audited 2 packages in 10s

found 0 vulnerabilities
npm verbose cwd /home/daidoji/foo
npm verbose os Linux <IP_ADDRESS>-microsoft-standard-WSL2
npm verbose node v21.5.0
npm verbose npm  v10.9.0
npm verbose exit 0
npm info ok

daidoji@LAPTOP-5KEUASIG:~/foo$ cat /home/daidoji/.npm/_logs/2024-11-19T16_53_13_677Z-debug-0.log
0 verbose cli /home/daidoji/.nvm/versions/node/v21.5.0/bin/node /home/daidoji/.nvm/versions/node/v21.5.0/bin/npm
1 info using<EMAIL_ADDRESS>2 info using<EMAIL_ADDRESS>3 silly config load:file:/home/daidoji/.nvm/versions/node/v21.5.0/lib/node_modules/npm/npmrc
4 silly config load:file:/home/daidoji/foo/.npmrc
5 silly config load:file:/home/daidoji/.npmrc
6 silly config load:file:/home/daidoji/.nvm/versions/node/v21.5.0/etc/npmrc
7 verbose title npm install
8 verbose argv "install" "--loglevel" "verbose"
9 verbose logfile logs-max:10 dir:/home/daidoji/.npm/_logs/2024-11-19T16_53_13_677Z-
10 verbose logfile /home/daidoji/.npm/_logs/2024-11-19T16_53_13_677Z-debug-0.log
11 silly logfile start cleaning logs, removing 1 files
12 silly packumentCache heap:2197815296 maxSize:549453824 maxEntrySize:274726912
13 silly logfile done cleaning log files
14 silly idealTree buildDeps
15 silly reify moves {}
16 silly audit bulk request { 'signify-polaris-web': [ '1.0.2' ] }
17 http fetch POST 200 https://registry.npmjs.org/-/npm/v1/security/advisories/bulk 282ms
18 silly audit report {}
19 http fetch GET 200 https://codeload.github.com/WebOfTrust/polaris-web/tar.gz/74be1c3ba361c3c7c49d7cc083abaa90a5dea1b0 224ms (cache revalidated)
20 silly packumentCache heap:2197815296 maxSize:549453824 maxEntrySize:274726912
21 silly ADD node_modules/signify-polaris-web
22 verbose cwd /home/daidoji/foo
23 verbose os Linux <IP_ADDRESS>-microsoft-standard-WSL2
24 verbose node v21.5.0
25 verbose npm  v10.9.0
26 verbose exit 0
27 info ok

I think I'm running into the same issue when trying to add a dependency in the KERI Auth browser extension so I can use its types/interfaces.  What I think is going on follows:
Issues Encountered:

dist Directory Missing After Installation:
When installing the package directly from GitHub (e.g., using "signify-polaris-web": "github:weboftrust/polaris-web#main" in my package.json), the dist directory is not present in the node_modules folder. This prevents TypeScript files and interfaces from being imported correctly.
Cause: The prepare script in the package.json currently runs husky but does not trigger a build process. As a result, the project is not built during installation.

Manual Steps Required for Consumption:
Currently, consumers may need to clone the repository, build it manually, or add workarounds in their own project to use polaris-web.

Suggestions for Improvement:

Update the prepare Script to Build Automatically:
Modify the prepare script to include a build step so that the dist directory is created during installation. For example:"scripts": {
  "prepare": "husky && npm run build"
}


Errors: game start and new level

The 2 errors seem to be occurring randomly. However, we need to investigate the nature of these errors in order to fix them. Moreover, in case of errors, any game progress should be saved if possible and the user has to be redirected to the main menu.

Some progress has been done on the new level bug but not yet fully complete. Need to focus on solving both issues for upcoming 0.6.2

Two main points to investigate and fix:

Issue on game start: JS error when attempting to draw characters. Occurs randomly for either new game or continuation. Perhaps, it's just that some character images might not have yet loaded at that stage and the context 2d method throws an error. Need to put an if statement to prevent this, as well as check that all promises are properly awaited!
Issue on level up is that after restoring game and completing level 1, the level may be incremented to level 3 and a dialog that you lost is displayed. Need to check the flow for that and fix that as appropriate.

To update ticket with progress regarding the above as we go along on this ticket.

Still occurring:
Prey.ts:80  Uncaught (in promise) TypeError: Failed to execute 'drawImage' on 'CanvasRenderingContext2D': The provided value is not of type '(CSSImageValue or HTMLCanvasElement or HTMLImageElement or HTMLVideoElement or ImageBitmap or OffscreenCanvas or SVGImageElement or VideoFrame)'.

To also refactor background.ts to simple functions rather than static class

"Only" the following error still to be fixed:
Prey.ts:80  Uncaught (in promise) TypeError: Failed to execute 'drawImage' on 'CanvasRenderingContext2D': The provided value is not of type '(CSSImageValue or HTMLCanvasElement or HTMLImageElement or HTMLVideoElement or ImageBitmap or OffscreenCanvas or SVGImageElement or VideoFrame)'.

Issue 2 from above is now (only) occurring when handling image 404s. To investigate the cause of this and resolve as most suitable.

Abandon development of 404 handling. Not really useful to invest time in that but rather in new features and fixing actual problems!!


Is there any size limitation with "Read All" buffer?

I have some serial commands which will output strings about 100 lines.
However, I can only read string about 50 lines when I use "Read All".
Is there any size limitation with "Read All" buffer? How can I read more lines from buffer with "Read All"?
If I use "Read until  string" instead, I can read all lines without problem.

As for as i know, There is no size limitation for read all.
But, If you want to limit the buffer you could use as this follow,
https://stackoverflow.com/a/50011066/12536466


Fixed grammar

Many thanks !

Thank you @rougier  for maintaining such a useful collection of problems. I would be using it for teaching numpy 😄


Fix segmaker task for new oss provider

Fix segmaker task for new oss provider (apache-airflow-providers-amazon==8.0.0)

@pankajastro I have copied back these changes in PR https://github.com/astronomer/astronomer-providers/pull/972

@pankajastro I have copied back these changes in PR #972 so that we can verify there that the tests pass.

I feel, ideally, we have merged this into your branch or cherry-pick the commit.

closing it


Missing a release of Ravedude

There is some new stuff in ravedude which isn't installable via cargo +stable install ravedude. Namely the promini-5v code added here.
It seems there is a release missing and the semver hasn't been updated.
To replicate:
> cargo +stable install ravedude
> ravedude --version
ravedude v0.1.5 (no git)
> ravedude promini-5v
thread 'main' panicked at .cargo\registry\src\index.crates.io-6f17d22bba15001f\ravedude-0.1.5\src\main.rs:90:47:
board not found

> cargo install --git https://github.com/Rahix/avr-hal ravedude --branch main
> ravedude --version
ravedude 190f2c3cb8d2
> ravedude promini-5v
Board SparkFun Pro Mini 5V (16MHz)

For what it is worth:

ravedude version 0.1.5 is from 2022-09-18
merge request #435 is from 2023-09-04

It seems there is a release missing and the semver hasn't been updated.

As I see it: There was no ravedude release since september 2022.

Fair enough, published ravedude version 0.1.6 with all the changes since the last release (see CHANGELOG).


websocket communication error

Issue

Every minute or so I receive this message
Log Messages

[11/15/2020, 4:42:37 PM] [ConBee II] ConBee II: websocket communication error <IP_ADDRESS>:443: cannot connect: ETIMEDOUT

Debug Files

homebridge-hue.json.gz

Maybe a rasbian firewall problem?

Yes, Port 443 was closed 😑🙄


77 token limit

Is it potentially possible to use compel? I tried to add it but it complains about the tokenizer. Not very good at coding, should it work or shouldn't I try?

In general, today, imho, segmoe gives the best results. I'm delighted)
No hiresfix, no upscale.

The outputs you shared look amazing! Its absolutely possible to use compel just as you would use it with diffusers, here is an example
For SDXL based SegMoEs:
from compel import Compel, ReturnedEmbeddingsType
from segmoe import SegMoEPipeline

t2i = SegMoEPipeline("segmind/SegMoE-4x2-v0", device = "cuda")
compel = Compel(tokenizer=[t2i.pipe.tokenizer, t2i.pipe.tokenizer_2] , text_encoder=[t2i.pipe.text_encoder, t2i.pipe.text_encoder_2], returned_embeddings_type=ReturnedEmbeddingsType.PENULTIMATE_HIDDEN_STATES_NON_NORMALIZED, requires_pooled=[False, True])

prompt = "Milky Way. Night sky with stars and silhouette of a standing happy man with yellow light. Space background, (sharp focus:1.2), extremely detailed, (photorealistic:1.4), (RAW image, 8k high resolution:1.2), RAW candid cinema, 16mm, color graded Portra 400 film, ultra realistic, subsurface scattering, ray tracing, (volumetric lighting), extreme contrast, intricate details, reflections on ice, reflections on water, water pouring down"
conditioning, pooled = compel(prompt)

img = t2i(
    prompt_embeds=conditioning,
    pooled_prompt_embeds=pooled,
    height=1024,
    width=1024,
    num_inference_steps=25,
    guidance_scale=7.5,
).images[0]
img.save(f"image.png")

For SD Based SegMoEs
from segmoe import SegMoEPipeline
from compel import Compel

t2i = SegMoEPipeline("segmind/SegMoE-SD-4x2-v0", device = "cuda")
compel = Compel(tokenizer=t2i.pipe.tokenizer, text_encoder=t2i.pipe.text_encoder)

prompt = "Milky Way. Night sky with stars and silhouette of a standing happy man with yellow light. Space background, (sharp focus:1.2), extremely detailed, (photorealistic:1.4), (RAW image, 8k high resolution:1.2), RAW candid cinema, 16mm, color graded Portra 400 film, ultra realistic, subsurface scattering, ray tracing, (volumetric lighting), extreme contrast, intricate details, reflections on ice, reflections on water, water pouring down"
prompt_embeds = compel(prompt)

img = t2i(
    prompt_embeds=prompt_embeds,
    height=1024,
    width=1024,
    num_inference_steps=25,
    guidance_scale=7.5,
).images[0]
img.save(f"image.png")

I hope this helps!

I hope this helps!

Thank you, it started, I will test it.

Tested it very quickly, it seemed to me that the quality had decreased, the accuracy of long prompts had increased, I took very long prompts for testing, perhaps they were initially “crooked”.
Tried to add - truncate_long_prompts = False
self.compel = Compel(
            tokenizer=[self.pipeline.pipe.tokenizer, self.pipeline.pipe.tokenizer_2],
            text_encoder=[self.pipeline.pipe.text_encoder, self.pipeline.pipe.text_encoder_2],
            returned_embeddings_type=ReturnedEmbeddingsType.PENULTIMATE_HIDDEN_STATES_NON_NORMALIZED,
            requires_pooled=[False, True],
            truncate_long_prompts = False
        )

And - max_embeddings_multiples=3
img = self.pipeline(
                prompt_embeds=conditioning,
                pooled_prompt_embeds=pooled,
                height=int(height),
                width=int(width),
                num_inference_steps=int(num_inference_steps),
                guidance_scale=float(guidance_scale),
                max_embeddings_multiples=3
            ).images[0]

ealing, appealing,:1.3), (provocative legwear:1.3), (full body photo:1.3), dynamic scene, action packed, solo, headdress, (v
oluminous petticoat:1.2 skirt black shiny satin), grand interior, baroque elements, elegant, detailed, 8k resolution, (Royal
ty:1.3), she is feeling furious, Fighter, subtle cheeks and Pouty lips and Symmetrical shaped face, in Mysterious Chaotic Tr
ansparent Unique Neon Lighting, The Chaotic Transparent Unique Neon Lighting is inspired by fantasy, pointe pose, Gray hair
styled as Bald, Cluttered Colorful Ruff, Funny Glasses, Sun in the sky, horizon-centered, Vivid (best quality, masterpiece:1
.2), photorealistic

Before:

After:


Within the support Config for SQL

Is there are reason why SSRS/RP is not listed as support role for all version of SQL above?
AKA likes this?
A central administration site
A primary site
A secondary site
And Reporting Points

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

ID: 1cb55272-b9ee-0407-d034-f6fc37c979a3
Version Independent ID: 9745517a-b005-84a8-d45a-fe02a636134f
Content: Supported SQL Server versions - Configuration Manager
Content Source: sccm/core/plan-design/configs/support-for-sql-server-versions.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

is Sql 2019 not support with the current branch?

@CraigB7708, offtopic for this specific issue, but I can answer: we'll add version 2019 when we've tested and verified it meets our customers' requirements for performance and function. That's in progress, so we'll update the doc when ready.

I submitted a change for both of these points 😉
Once the pull request merges, this issue will auto close.
I'm actually thinking of doing an out of band publish this afternoon, so you should see the changes live shortly


NOTIFICATION_APPLICATION_FOCUS_IN/OUT does not work in web exports on browser losing focus

Tested versions
Reproducible in 4.2.1.stable
Not reproducible in 3.5 and earlier (with NOTIFICATION_WM_FOCUS_IN/OUT)
System information
Godot v4.2.1.stable unknown - Arch Linux #1 SMP PREEMPT_DYNAMIC Fri, 05 Jan 2024 16:20:41 +0000 - Tty - GLES3 (Compatibility) - NVIDIA GeForce GTX 1060 (nvidia; 545.29.06) - Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 Threads)
Issue description
When detecting whether or not there is app focus on desktop builds, NOTIFICATION_APPLICATION_FOCUS_IN/OUT works as intended. It doesn't work in web builds though when the browser loses focus.
If there's another notification event I'm supposed to be listening on, I haven't found it or it hasn't been documented properly.
Steps to reproduce
Have a pause screen somewhere that shows up when the app loses focus or print it somewhere so that it's obvious this is happening.
func _notification(what):
	match what:
		MainLoop.NOTIFICATION_APPLICATION_FOCUS_OUT:
			get_tree().paused = true
			...do stuff on focus out
		MainLoop.NOTIFICATION_APPLICATION_FOCUS_IN:
			get_tree().paused = false
			...do stuff on focus in

Run this code in desktop. Works as expected. Tree pauses on focus out and resumes on focus in.
Run it in web export. Nothing happens when the browser loses focus. Tried it with changing tabs, switching to another app, changing workspaces, nothing happens.
Minimal reproduction project (MRP)
N/A

Your description lists NOTIFICATION_APPLICATION_FOCUS_IN/OUT but the title lists NOTIFICATION_WM_FOCUS_IN/OUT, which is it?
The former is only supported on desktop, not inluding Web

Your description lists NOTIFICATION_APPLICATION_FOCUS_IN/OUT but the title lists NOTIFICATION_WM_FOCUS_IN/OUT, which is it?
The former is only supported on desktop (as you can see in the documentation), not inluding Web AFAIK, and isn't the same as the latter, which is a distinct notification, can you try that one in 4.x? It is still there
Edit: updated the title to match the example code

No, it's not there anymore. There is NOTIFICATION_WM_ABOUT which is something else. The editor claims it can't find NOTIFICATION_WM_FOCUS_IN/OUT and won't build.

My bad I meant NOTIFICATION_WM_WINDOW_FOCUS_IN/OUT, though it seems the 3.x version maps to the application version, however the application version isn't supported on Web, as per the documentation

My bad I meant NOTIFICATION_WM_WINDOW_FOCUS_IN/OUT, though it seems the 3.x version maps to the application version, however the application version isn't supported on Web, as per the documentation

It doesn't find that either.

It's there, right in Node, see here

Okay, that works, thanks.


Issue while flashing Firmware

i am getting this issue while flashing the firmware on pinecil v2

If you still have the tip inserted in your pinecil, it might not work. Try following the same steps but remove the tip before plugging the device to your computer. Hope it helps!

If you still have the tip inserted in your pinecil, it might not work. Try following the same steps but remove the tip before plugging the device to your computer. Hope it helps!

Sir all the issues are with windows os when i try on ubuntu it flashes in 1 second


Document Developer Settings

Really liking the debug panel. I noticed it in Wharton's U2020, and was curious if yours is a clone or if you've modified it in some ways. I know there are a couple 3rd party libraries that are similar as well.
Would be nice to update the README to document it in some form.

QualityMatters has its own implementation of Developer Settings based on my
experience of how I do it in production apps.
You want to see more detailed explanation of how Developer Setting are
built in the app in the README?
On Thu, Dec 3, 2015, 23:37 John Shelley<EMAIL_ADDRESS>wrote:

Really liking the debug panel. I noticed it in Wharton's U2020, and was
curious if yours is a clone or if you've modified it in some ways. I know
there are a couple 3rd party libraries that are similar as well.
Would be nice to update the README to document it in some form.
—
Reply to this email directly or view it on GitHub
https://github.com/artem-zinnatullin/qualitymatters/issues/45.

--
@artem_zin

@artem-zinnatullin Yeah or even throw it into a library and put it up on jcenter :smiley:
Besides yours and whartons I've seen: https://github.com/palaima/DebugDrawer
Always like to see more out in the wild though to try out.

Okay, I'll describe basic things about Developer Settings in the README.
Not sure I want to turn it into a library because it should be easy to
setup and customizable at the same time. Also, it's just hard to decide
what to extract to the library and what not  ¯_(ツ)_/¯
On Fri, Dec 4, 2015, 00:17 John Shelley<EMAIL_ADDRESS>wrote:

@artem-zinnatullin https://github.com/artem-zinnatullin Yeah or even
throw it into a library and put it up on jcenter [image: :smiley:]
Besides yours and whartons I've seen:
https://github.com/palaima/DebugDrawer
Always like to see more out in the wild though to try out.
—
Reply to this email directly or view it on GitHub
https://github.com/artem-zinnatullin/qualitymatters/issues/45#issuecomment-161787383
.

--
@artem_zin


Migration Assistant cant connect to the shop when using IDN Domains

When using ASCII compatible domains (IDN - Internationalized Domain Name) for example in a Shopware 5 instance then you cannot connect since the domain is converted back automatically in the migration Assistant input form.
Example:
In Shopware 5 the domain the domain “examplé.com”  is used which is is the domain “examplé.com” .
In Shopware 6 in the migration Assistant when trying to input the domain “examplé.com“ it is converted into “examplé.com” but this cant connect since it is not the same as the one in shopware 5.

[public] Shopware automated response: This issue is linked to the internal issue: MIG-1048
[created from MIG-1048, comment 497690]


Support setting creation on settings config repository

:+1:

Can this be as simple as something like:
public function set($settingName, $settingValue)
{
    return $this->model->updateOrCreate([
        'name' => $settingName,
    ], [
        'value' => $settingValue,
    ]);
}

?

It does work and I can't think of anything else that it actually needs to do. Thoughts @GrahamCampbell?

Yeh, would probably be fine like that. :)

just name and value, rather than settingName and settingValue though

then can be: return $this->model->updateOrCreate(compact('name'), compact('value'));

On it :)

:+1:


Add a default space before extra options in dask command line

Fixes #156

Just a comment that is beyond the scope of this PR:  Keeping track of all the CLI args to dask-worker in a list_of_all_args (see the recommended way of passing args to subprocess) and eventually do " ".join(list_of_all_args) would be a general way of solving this.

@willirath this would probably be cleaner yes, but should we change the extra constructor kwargs to be also a list then?

but should we change the extra constructor kwargs to be also a list then?

My personal feeling says: Yes, passing CLI args as a sequence is standard any way.  But I'm not sure if that's representative.

Okay, so I chose to follow your suggestion @willirath.

Travis was also having a difficult monday apprently.

@willirath, since you've made the comment about using a list of args, a review would be welcome!

Few, thanks for that @willirath, no real testing from my part before your comments... Glad you saw those mistakes.

Thanks @willirath, feel free to take a look at other PR/issues if you find the time. It could be nice to have another core maintainer.

Thanks @willirath, feel free to take a look at other PR/issues if you find the time. It could be nice to have another core maintainer.

Sounds good.  I'm not sure I'll have much time for continuous work. But it sure looks like dask clusters and dask-jobqueue will play a increasingly role in my and my group's routine work. So count me in!

Nice !
We're all on the same situation about time 🙂 !


test for arcgis dependent portion of script

To catch issues like #34 somewhat automatically, add tests that check arcgis.py
Depends on an ESRI licence, but pytest has methods for skipping tests on given platforms https://docs.pytest.org/en/stable/how-to/skipping.html

Not immediately sure how to handle passing test data to arcpy.GetParameter(), so I'll put this to the side for now - there is very little to be tested in this script beyond the gdb/shp detection

It isn't an issue - I just missed spelling out / pasting a property of the path.
Detection of input file type could be removed entirely if the tool's parameters matched the script's parameters (ie path to data, name of layer rather than an arcpy feature class), but the interface isn't as nice then.

Understood. It would be nice to conserve the current concise interface if possible. Let me know when/if the code is stable enough to restart testing. Thank you!

No issues to resume testing.


Gauge unfilled section for full circle is not visible.

If the gauge is set to fullCircle and has a data value that is less than the max value, the unfilled section of the gauge is not visible.

Good catch! Please see #1604, I've added a few suggestions. :smile:

Sorry for the delay. Here's a JSFiddle showing the incomplete unfilled section of the 360 gauge:
http://jsfiddle.net/vkt2pvc9/


Version 0.4.5

Adding ndict
Addressing issue https://github.com/Lyncs-API/lyncs.io/issues/27

Codecov Report

Merging #28 (3926464) into master (0d4b0f7) will increase coverage by 0.20%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
+ Coverage   90.91%   91.12%   +0.20%
==========================================
  Files          11       11
  Lines         980     1003      +23
==========================================
+ Hits          891      914      +23
  Misses         89       89

Impacted Files
Coverage Δ

lyncs_utils/__init__.py
100.00% <100.00%> (ø)

lyncs_utils/extensions.py
97.53% <100.00%> (+0.31%)
:arrow_up:


Unable to resolve module ' react-native-facebook-login'

FBSDK Please Work
1 week from troubleshooting! same issue!! i just working on login with Facebook using react-native , i create more than 12 projects with different codes and methods i see it in internet no one love my problem.
In first project (i create it in past) the problem was same issue when i import FBSDK:
`import FBSDK, {LoginManager} from 'react-native-fbsdk'
[What i try]
reset cache , reset npm , delete modules , change require ,  install last version of react-native , install alpha6 , alpha 12 , restart mac , Xcode , quit simulation , npm start --reset-cache , rm -rf node_modules && npm install , watchman watch-del-all , kill process and restart npm , install non 8 , install npm 6 LTS
@ i follow this tutorials :
https://www.youtube.com/watch?v=rAXVKapP5cM
https://www.youtube.com/watch?v=rAXVKapP5cM
https://www.youtube.com/watch?v=aRJY7Hbe9jI
https://www.youtube.com/watch?v=yQi6F4Dh6gY
In Last project i make it very carefully by download example project named "react-native-fbsdk-master" from github by  dzhuowen ,

If you have project worked for you please upload the file or share code to see what is the problem.
@My Hardware spec:

Mac OS El Captain
npm 5.0.3
react-native-cli: 2.0.1
node.js v8.1.2
react-native-fbsdk

The 'react-native-facebook-login' worked for me by do this steps:
0- Create New react-native project then install npm by npm i
1- Download the SDK and unzip the archive to ~/Documents/FacebookSDK

the name of folder must to be FacebookSDK and this folder in Documents.

2- Make new folder named Framework
From Documents/FacebookSDK folder, drag the folder Bolts.framework, FBSDKCoreKit.Framework, FBSDKLoginKit.Framework, FBSDKShareKit.Framework into your Xcode Projects Framework folder.
3- Goto https://developers.facebook.com/
Make new App from Facebook Dev.  then input Bundle Identifier to get App ID ... skip this steps:
Configure your info.plist
Track App Installs and App Opens
Enable Sign in  then copy the AppID.
4- Go to react-native terminal and install react-native-fbsdk by
npm install<EMAIL_ADDRESS>--save
react-native link react-native-fbsdk
5- Go to your react-native app then open iOS folder, the direction now is YourApp/ios
make new file without extintion name Podfile open this file by textedit and put this code (don't forget to change YourApp to real name of your app):
# You Podfile should look similar to this file. React Native currently does not support use_frameworks!
source 'https://github.com/CocoaPods/Specs.git'

platform :ios, '8.0'

target '<YourApp>' do
  # Fixes required for pod specs to work with rn 0.42
  react_native_path = "../node_modules/react-native"
  pod "Yoga", :path => "#{react_native_path}/ReactCommon/yoga"
  pod "React", :path => react_native_path, :subspecs => [
    'Core',
    'RCTActionSheet',
    'RCTAnimation',
    'RCTGeolocation',
    'RCTImage',
    'RCTLinkingIOS',
    'RCTNetwork',
    'RCTSettings',
    'RCTText',
    'RCTVibration',
    'RCTWebSocket'
  ]

  pod 'GoogleMaps'  # <~~ remove this line if you do not want to support GoogleMaps on iOS

# when not using frameworks  we can do this instead of including the source files in our project (1/4):
#  pod 'react-native-maps', path: '../../'
#  pod 'react-native-google-maps', path: '../../'  # <~~ if you need GoogleMaps support on iOS
end

6- Change terminal dir to YourApp/ios  then apply this:
npm install pod
7- Now to to Xcode project that found react-native folder , then go to AppDelegate.m and put this code
replace all the file by the code above.
8- Back to YourApp dir in terminal , then Apply this command in terminal  npm install plist xcode adm-zip --save-dev
9- Then apply this cp ./node_modules/react-native-fbsdk/ios_setup.js ./ios_setup.js in terminal.
10- Next: node ./ios_setup.js [App ID] [App Name] , by change App ID to facebook id you get it , App name the same name you created in past (ex. YourApp).
11 - You must sure the Xcode like this:

be sure there is no bugs related to import xxxxxx.h !! this issues will build sometimes correctly but the app not working and FBSDK will show errors, step 5-6 to resolve  problem called #import <React/RCTBundleURLProvider.h> not found.

12- You must but ~/Documents/FacebookSDK to Build Settings under search path  like you see in this photo.

13- Now open index.ios.js download it with worked code 100%:
http://www.mediafire.com/file/tajh1izyx4j6b3v/index.ios.js
or follow this tips to write another code https://developers.facebook.com/docs/react-native/login
14- After replace your index.ios.js with downloaded in (13), open it and change this:
export default class ZApp extends Component
to your real app name
export default class YourApp  extends Component
Then save it!
15- Back to terminal Now Build App by react-native run-iso, the app must work 💯 %.
Best Regards
Tawfeeq Amro


Bug

Microsoft PowerToys version
0.66.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
General, Always on Top, Awake, ColorPicker, FancyZones, FancyZones Editor, File Locksmith, File Explorer: Preview Pane, File Explorer: Thumbnail preview, Hosts File Editor, Image Resizer, Installer, Keyboard Manager, Mouse Utilities, PowerRename, PowerToys Run, Quick Accent, Screen ruler, Settings, Shortcut Guide, System tray interaction, TextExtractor, Video Conference Mute, Welcome / PowerToys Tour window
Steps to reproduce
Auto pop-up
2023-01-07.txt
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/18166

For any users that are able to replicate this consistently, can you please update to 0.68.1 and disable thumbnail generation to verify if it fixes the issue? Please let us know on #18166!


Round to nearest instead of down when converting components to integers.

A pretty simple change, this makes it so when converting components to an integer format, we round it to nearest first, instead of just doing the conversion which will round towards zero.
Not happy about using the destination type's LIMITED to determine if it will be an integer, since that's not really appropriate semantics, but it is currently set if and only if the type is an integer, and every integer type would need it, and I'm not sure why there'd every be a limited non-integer type.  Could rename it, or add a separate property, but I'm not sure it's important enough for that.

Thanks, I appreciate it a lot! I have been meaning to adjust this whenever I pass by it, but I have had other things on my mind for a while... Anyway, rounding to nearest will make it slightly off in a different way. 0 and MAX will be smaller than the rest of the "bins", for example. A more correct formula would be (x * (MAX + 1)).min(MAX). The only time it would hit MAX + 1 is if it's larger than or equal to 1 and that bias will be negligible compared to floating point errors. Here are some more detailed explanations: https://stackoverflow.com/questions/1914115/converting-color-value-from-float-0-1-to-byte-0-255
As for the LIMITED constant, nothing prevents a third party from implementing the trait for a non-integer value. The formula I suggested clashed with the semantics as well, so I'm not sure what to do about it right now. I'm open for suggestions!

I actually agree with the second answer at the link you posted, that the way I did it is more accurate. :)
Fundamentally, I think reducing error is the goal, not having equal sized bins.  And I think converting from an int to a float is accurately done by simply dividing by MAX.  Not having equal sized bins when converting to a fixed range is a result of the desire for the fixed range to be able to exactly represent the end points - if not for that, the proper formula for converting from int to float would be, I think, (x * 2 + 1)/((MAX +1) * 2), and then the formula that you gave would both have equal sized bins and minimize error.  Does that make sense?
Um, still thinking about the LIMITED thing.  How important is backwards compatibility?  Is it okay if I find a solution that removes that from the public API?

It's fine to break things if you have a better setup in mind. :+1:

I actually agree with the second answer at the link you posted, that the way I did it is more accurate.

Do you mean the b=f*255.999 one? I guess it accomplishes more or less the same thing. I don't trust the input enough to skip the clamping, so multiplying with 256 or 255.999 would practically be the same thing. I should have written
let scaled = x * (cast(T::max_intensity()) + 1.0);
cast(clamp(scaled, 0.0, cast(T::max_intensity())))

as that's what it would end up being in our case.
The way the accepted answer from the link, and this formula, works is the same as b=f*255.999, but it leverages the fact that only 1.0 and above will be out of rage. That way it gets as close to the full range as possible while only being biased towards 255 by practically one value. Multiplying by 255.999 achieves the goal by trying to avoid getting too close to the upper limit, but by an arbitrary value, so the scale is still off (even if it's just by a fraction). Using 255.999 would probably be fine if that was all, but since we are still clamping we might as well go all the way. We are not really saving much computing power. Only one addition, as far as I can tell.

if not for that, the proper formula for converting from int to float would be, I think, (x * 2 + 1)/((MAX +1) * 2), and then the formula that you gave would both have equal sized bins and minimize error.

I think you will have to explain that one... :thinking:

Um, the answer I meant is the one that advocates for round(f * 255.0) - https://stackoverflow.com/questions/1914115/converting-color-value-from-float-0-1-to-byte-0-255/46575472#46575472
Okay, to explain more - say you're converting to and from just 2 bits, as that's easier to explain.  There's a couple methods you could use.  Method A sets the four available values to 1/8, 3/8, 5/8, and 7/8 - this means that when converting, you're never off by more than 1/8, but you can't exactly represent something at 0 or 1 - full white or black would be impossible to represent if this method was used for colors.  Method B has the available values at 0, 1/3, 2/3, and 1, which means that conversion could be off by 1/6, but you can exactly represent 0 or 1.
Method A means that conversion will have the same size bins for each value; that is how it minimizes the maximum possible error.  Method B does not have that property - the bins will be half size for the minimum and maximum values.  Nonetheless, colors stored as discrete values should use method B, so as to be able to properly represent full white and black.
Does that make sense now?

I think I get it now. Especially with the answer you really meant in mind. Reading that and the explanation again, I'm starting to think that you may be right. My main requirements are that

Values close to 0.0 should map to 0u8, and 0u8 should map to 0.0.
Values close to 1.0 should map to 255u8, and 255u8 should map to 1.0.

which both methods fulfill. Then I assumed rounding down was preferred because of familiarity, but I'm starting to wonder if it's important and if it's more important to avoid conversion errors. I want Quality Colors™ so maybe a low error rate is the way to go after all. As long as it doesn't create too "weird" colors compared to other software, because that may actually be worse than conversion errors. If you don't mind, you can try to add a test in palette/tests/convert for converting to u8. There doesn't seem to be one right now, but that would at least check against what's probably generally expected.

By the way, do you think reversing the relationship, so it's fn convert_from<T: Component>(other: T) -> Self would help with the semantics? Then only the target type need to care about rounding. I'm not sure why I didn't do that from the beginning.

Hi! Are you still interested in continuing this PR?

Yes!  I actually did work on it back in September, just need to get back to it.  Sorry about dropping it for so long and not communicating.

Don't worry about it! I have also had my mind elsewhere for a while. It's
nice to see that you are still on it. No stress, and let me know if you run
into any roadblocks.

Nice! It becomes a bit of a combinatoric monster, but I think it's a reasonable case for using macros to remove repetition. I'm super tired right now, but I will have another look tomorrow. Until then I have at least a couple of comments:

Oh, also I removed conversion from f32 to u32, as I don't think that makes sense, and removed u64 as a component type entirely. In general I think the point of using an integer type is to reduce the storage size; my though was if you're trying to convert from f32 to u32, you're probably making a mistake. But if I'm missing something, let me know.

The larger integer types are there to support formats with more range/precision. 16 bit components are not that unusual in photography, but 32 bit and even 64 bit may appear. Either way, I see no big downside with supporting them, other than the repetition (but we have macros).

Also want to update this to be based off the latest master branch, and maybe combine the two commits into one, but I haven't done that often enough to remember how, and didn't want to mess with it right now. But I'm aware it's something to be done; I may wait to see if there's other feedback to act on first, though.

You might as well wait with combining them until it's all green, but I would recommend rebasing your branch onto master to bring in the latest and greatest. Let me know if you need any help and I can either point you towards some resources or give you some useful commands. :slightly_smiling_face:

Hi! Long time, no follow-up from me (sorry). I just want to check what the status for this is, as it hasn't moved in a while, and maybe suggest rewinding it to only include the first commit. I would definitely appreciate to have the first one merged. The rest can be split out to a separate project, if it is to be continued, as it involves a bit more figuring out.

I have created a separate PR with only the first commit: #135. I'm going to close this one as inactive when that one passes the tests. Feel free to open a new PR if you decide to continue the trait project!


AT: S-7.1: As a user, I want my next class to be displayed in my calendar.

This acceptance test is in reference to issue #28
Acceptance Criteria
Given

that I am in the schedule page and that the app has access to the device’s calendar

Then

the list of my scheduled classes are displayed

Steps

Open the drawer menu on the left side of the screen.
Select “Schedule”.
Select “Allow” for permissions to your device calendar.

NOTE
4 uppercase letter followed by either (a space, dash or nothing) finally followed by 3 digits.

Example

ACCEPTED


java.lang.ClassCastException: java.lang.String cannot be cast to org.hl7.fhir.instance.model.api.IBase

Hi James,
we are experiencing this error while serializing to json or xml:
java.lang.ClassCastException: java.lang.String cannot be cast to org.hl7.fhir.instance.model.api.IBase
the stack trace points at:
ca.uhn.fhir.parser.BaseParser.encodeResourceToString(BaseParser.java:351)
Looks like somehow the parser is expecting an IBase but is getting a String. It should be no problem with the resource that I am serializing becuase everything was fine during the instantiation process.
In this case we are serializing a Bundle with a MessageHeader and Encounter resources.
Tested on HAPI versions 1.2, 1.3 and 1.4

Hi Hugo,
Would you be able to provide a code snippet illustrating what you're seeing?

Meanwhile I found the reason for this error was because of the name property of the patient resource.
This code is HAPI Fhir running on Javascript (Rhino, Mirth Connect).
var patient = new Packages.ca.uhn.fhir.model.dstu2.resource.Patient();

patient.setName(['Hugo']); //public Patient setName(List<HumanNameDt> theValue)
...
var outcomeString = parser.setPrettyPrint(true).encodeResourceToString(patient);

It is strange that this does not give an error while setting the name, only while encoding to string...

Hi Hugo,
patient.setName(['Hugo']);  looks wrong to me. That method accepts a list of HumanNameDt objects, but you're passing in a list of strings. Java's generics don't enforce their types at runtime, so the error isn't caught until the parser tries to actually use the value later on.
I don't know exactly the right syntax you'd need, but presumably it would be something like
var name = new Packages.ca.uhn.fhir.model.dstu2.composite.HumanNameDt();
name.addGiven('Hugo');
patient.setName([name]);

I understand the issue with generics... Has an improvement on the parser I think you could control these encoding errors and produce a detailed error message showing exactly wich property is failling during the encoding process.
I see this happening a lot of times during resource instantiation and it will be a pain to check each property set by hand to find the one that is causing the problem...
Keep up the great work.

Thanks for the suggestion! Checking in a fix now.


[AutoDiff] Handle struct_element_addr in differentiation.

Handle struct_element_addr in PrimalGen and AdjointGen.

struct_element_addr makes use of StructExtractDifferentiationStrategy,
along with struct_extract. The adjoint accumulation rules are similar.
This enables differentiation of structs with generic members. Add tests.
Gardening, edit doc comments.

Resolves TF-203.

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow


Cinergi facets in UI are incomplete

only Activity, Equipment and Feature3 are showing up under CINERGI facets.  The hierarchy construction is not working for some reason.

It does not like  order:{_term:'asc'}:
props:{size:1000,
order:{_term:'asc'}},

Looks like this is fixed by commit daab5a6ccf2ea8423284a3949da78a7316d4fefa.  The CINERGI keywords are still showing up sorted alphabetically which is good. Can the list show up without the top level 'Category' node in the facet tree?


BoringModel uses deprecated code

🐛 Bug
The BoringModel is out of date. This model is used to help generate bug reports, but it uses syntax that was deprecated as of 1.2.6. This should be a simple fix to update, but I also propose making legacy BoringModels for pre-1.2.6 versions, in case a user has an issue with old code.

Dear @tmcclintock,
Could you share which part of the BoringModel is depreceated ?
NB: The BoringModel doesn't perform any logic by default.
Best,
T.C

@tchaton the train/val/test_step functions all return a dictionary, even though they are supposed to use LightningModule.log_dict instead as of 1.2.6 (see the changelog bullet I linked to).

@tmcclintock - these step functions can still return a dict for further processing in the training/validation/test_epoch_end hooks

I see no reason to remove it from the BoringModel. It's perfectly fine to pass outputs from step to epoch_end hooks and it seems a good default implementation for a debugging model.

Sure thing. My mistake, I must have misread the functions and misunderstood. Sorry for the noise.


Message Signature to identify message sender and sending messages back

Resolves #119

Thank you for your big contribution :100: :)


data_retention_time_in_days as snowflake_object_parameter

Provider Version
v0.56.5
Terraform Version
1.3.9
Describe the bug
The documentation says that the data_retention_days argument is deprecated for Snowflake tables. When I use the data_retention_days argument under a table resource, Terraform tells me to use the snowflake_object_parameter resource instead. I used the snowflake_object_parameter resource, using "DATA_RETENTION_TIME_IN_DAYS" as the key. I set the value to 7. When I initially apply the table and object parameter, the data retention time for the table is correct. However, if I update the table on the next apply, the table's data retention time reverts back to 1 day, the default value for Snowflake time travel.
Expected behavior
When I initially apply the table and object parameter, the data retention time for the table is 7. If I update the table on future applies, the data retention time should stay 7 days, not revert back to 1 day.
Code samples and commands
https://github.com/AgilliticsLLC/Terraform_Bug

terraform init
terraform apply
Change the DESCRIPTION column's type to be VARCHAR(10) on table_and_object_parameter.tf.
terraform apply

Additional context
I opened an issue on Terraform's repo, and they redirected me to this repo.
https://github.com/hashicorp/terraform/issues/32713

Just to add to this ticket.
At first, the table data retention time in Snowflake will be as per the value set by the corresponding snowflake_object_parameter resource but the data_retention_days property of the snowflake_table resource will be set to 1 (in the state file), the default. At that point, subsequent executions of terraform apply, with or without changes in the configuration, will set the data retention time back to 1 in Snowflake since the snowflake_table resource is (now) configured as such. Obviously, the snowflake_object_parameter resource is then ignored (because respective entries in configuration and state file match?).
I suspect the same scenario repeats with other object parameters but I did not investigate any further.

Can confirm I am also experiencing this same issue where tables are flip flopping between retention periods. Clear mismatch between the value set in the object parameter and what is listed against the table itself in the state file.


unable to create wave charts for songs at amazon s3

When ever i am calling my s3 mp3 files to plot the waves of a songs. it gives me below error.
Failed to load resource: the server responded with a status of 404 (Not Found)
test.html:1 XMLHttpRequest cannot load http://neeraj-f0b1ea.s3.amazonaws.com/Desiigner - Timmy Turner (D…TQKEKGZSJDLKSA&Expires=1469689916&Signature=yfExCQd0CkPweIhDTxfQyHK1NEA%3D. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:9090' is therefore not allowed access.
Can you please guide.

Search in Google for "S3 how to enable CORS".

I am also facing the same issue. The audio is loading, I can here it but waveform is not creating. CORS is already added in S3. The audio is playing perfectly in audio tag.


https://github.com/FIWARE/node-red-contrib-FIWARE_official is not deprecated is new!

To move forward just fork the repo and open a PR to add something to the list or move it to the right section. Its very early days and only the top half of the list is properly formatted and in the right location so please follow  the style of that section as a guideline.
Thanks.

Fixed  d4f3e88


Change opossum sprites | Add new pet poppy the safety opossum

About the PR

Changes the opossum sprites to ones from baystation (it was the only ones i could fine and i cant sprite)
Also adds poppy, a safety opossum! Would go great as an engineering pet. (they dont have a pet right? only atmos does)
Since I know a bunch of people love Morty and his iconic sprite. I kept him in as an admin spawn
Little discussion
https://discord.com/channels/310555209753690112/346743671787356160/1174379481436270592  (deltanedas cant read this :troll:)
Why / Balance

Old sprites were only 2 directional and were not really in match of the art style. Neither are these really in the right art style but its better then what we have right now
Technical details

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Poppy

Morty/Possums

Breaking changes

Changelog

:cl: Vasilis

add: Added Poppy the Safety Opposum!
tweak: Opossums have received some new sprites.  I can already see the assassin's coming to my house

it's mortyover


CQs and INTO queries should allow for non-nanosecond precision

Feature Request
Proposal: [Description of the feature]
Allow users to specify the write precision for points written to InfluxDB with an INTO clause
Current behavior: [What currently happens]
Currently all CQs and ad hoc INTO queries write points with full nanosecond precision.
Desired behavior: [What you would like to happen]
We should be able to specify units other than ns for the timestamps, particularly us, ms, and s.
Use case: [Why is this important (helps with prioritizing requests)]
Compression and throughput are better with less precise timestamps.

This actually sounds like a bug. CQs are supposed to be idempotent and the only way that will be true is if the resulting timestamp is rounded to whatever bucket/precision the group by time on the CQ is.
At least, that's what the behavior used to be, once upon a time.

@beckettsean AFAICT, using precision rfc3339 in the CLI will display timestamps with  the same precision as what's stored.
A measurement that I know is stored with second precision:
> select n from go limit 1
name: go
--------
time			n
2016-03-31T16:54:14Z	50000

A measurement with microsecond precision:
> select bytes from hh_queue limit 1
name: hh_queue
--------------
time				bytes
2016-07-28T15:46:50.770082Z	191559

I'm not aware of a more efficient way to discover the stored precision of a point.

Thanks, @mark-rushakoff that works well. From my quick testing CQs do write with precision determined by the GROUP BY interval. Closing.


Breakpoint Delegate rework (part of improve errors)

I reworked the breakpoint delegate a little bit to prepare for re-working the errors in the gutter.

delegate methods made optional, as app might not use breakpoints.
deprecated the current delegates with the useless ForFile: syntax.
new methods have a sender. Unfortunately it meant assigning a weak reference to the gutter view so that delegates would know the sender. If an application manages multiple Fragaria instances, then previous implementation provides no means for the delegate to know which instance sent the message. This was done as a generic id called userData.
the demo application allows setting and clearing breakpoints in the gutter simply by clicking.

This was done in preparation for changing the NSButtons to NSImages, as now detection of the click in each line will have to discriminate between the breakpoint marker and the error image.
 It's innocuous, but it bugs me somewhat. Any suggestions on that point?

We can't do much about userData. I'd explicitly make it weakly retained though. If you want to get rid of the id typing, I'd prefer having it typed as MGSFragaria *; that would be in line with AppKit's convention of passing the object sending the notification to the delegate. Since the gutter view is private, passing back the gutter view would be inappropriate.

It's weak now. If this is the way it has to be, I'll probably rename userData to something more obvious, and set the type explicitly. Current implementation was meant to be as anonymous as possible and make the gutter view reusable, but if you're happy with keeping it private, being explicit is a much better choice.


Implement "Indent Model"

I didn't know that backspace could be improved. How wrong I was! I hope JetBrains roll out Clion backspace handling across all their products. I'd also love to see it in Rust Analyzer.
There's already special enter key handling so backspace should be possible...
I think the rules are that the first backspace takes you up a level in indentation, but then the second gobbles up all the whitespace to the start of the line and the line brake so that it appends onto the end of the previous line. This turns out to be what I wanted to do so much of the time - it feels so natural.
Small things make such a difference. Well done Clion for bringing a better backsapce to the world.

Yeah, open line / newline / backspace all can benefit from indentation info...
There are two pieces to the puzzle:

implement indentation model, so that we know what the indentation should be. I wonder whether this is actually blocked on formatting? It seems like we can start coding something up just for indent, and then re-rail it onto proper formatting engine...
implement editor-side hooks. We already have a hook for enter, the one for backspace can be added similarly

Heck, I think we should have this sooner rater than later, as it seems like our onEnter does not stress the editing as much as it should.

I have to say that the behavior described in the issue seems like something that should be handled by the editor and not be language-specific, but maybe I'm misunderstanding it :thinking:

It has to be language specific I think. For example, here's some Kotlin code
fun <T> frobnicate(a: T, b: T)
        where T : Comparable<T>,
              T : Runnable {

    frobnicate(a,
               b
    )
}

Here, the T's should be aligned according to code style, but b should be just indented. The backspace behavior is different for two:

Also, unrelated, but the gutter icons in IntelliJ are soooo much better than code lens :( Like this little recursion swirly it's just cute, and we can't have that in VS Code TT

I think it would be a bad idea to implement this in VSCode for the same reason that messing with multiline commends was a bad idea:

Users won't expect it, and it produces editing inconsistencies with editing different languages.
Messing with text editing in an extension causes performance issues in VSCode at the moment.

Users won't expect it, and it produces editing inconsistencies with editing different languages.

VSCode already does this. For Rust it often produces unexpected or plain wrong results like the following though:
let a = b
    .c();<|>

Pressing enter gives:
let a = b
    .c();
    <|>

That is too much indentation. Even when you already deindented the next line pressing enter decides that it needs to indent anyway:
let a = b
    .c();
<|>

gives
let a = b
    .c();

    <|>

This is annoying.

VSCode already does this.

Not sure exactly what you mean by "this", but it does seem that VSCode does some limited language-specific indentation. It is controlled by onEnterRules, e.g. Python, PHP.
It is regex based, and the actions are limited to the following, plus appending text (e.g. /// for comments)
	export enum IndentAction {
		/**
		 * Insert new line and copy the previous line's indentation.
		 */
		None = 0,
		/**
		 * Insert new line and indent once (relative to the previous line's indentation).
		 */
		Indent = 1,
		/**
		 * Insert two new lines:
		 *  - the first one indented which will hold the cursor
		 *  - the second one at the same indentation level
		 */
		IndentOutdent = 2,
		/**
		 * Insert new line and outdent once (relative to the previous line's indentation).
		 */
		Outdent = 3
	}

I don't think it is quite powerful enough to be useful in your example because you can only look back one line.
On reflection I think you are right that it would be nice to fix it, but I don't think the best way is by intercepting ever keypress in the language server because of the performance issues we've already seen.

As of vscode 1.53.0 OnEnterRule now supports: OnEnterRule.previousLineText so we can get at the previous line.


Disable animations

It would be good if you could disable the CSS animations easily. Although they look slick, it can sometimes not jibe too well with your particular site's overall style/aesthetic.
Maybe the animations could be split into a seperate CSS file that you can import, if you like them? simpleLightbox-animations.css

Right now you can use not minified CSS and easily edit or remove these animations


Create a go.mod file

In particular, have a look at the replace directives
#24479

closed by https://github.com/keybase/client/pull/24023


feat(dbm-services): hadb-api add query_switch_queue_by_uid

关联研发信息
dev_branch: feat/enable_query_switch_queue_by_uid/#2714
target_branch: v1.3.0


Added Delay to ActivityTypes to prevent need to use magic string

We have an context extension method which was using an activity type of "delay".  Given that this is part of the core SDK, I have brought this in line with other activity types and added it to the ActivityTypes class to prevent the need to use a magic string for comparison and hopefully avoid typo mistakes.

I forgot to mention, that the underlying issue here is that Delay isn't yet added into the protocol itself. The client-side hacks you're looking at are symptoms of that.
We plan to add it in, but it's in the backlog behind more important work...

Hi Chris. This makes sense, thanks for the context. How about we go for the ActivityTypesEx class for now and then we can just remove from that class as necessary when the swagger file gets updated. That way we get rid of the magic strings at least until the update and if the update doesn't happen soon then we have a way of handling more types if they are added in the meantime. If that makes sense then I'll update the PR later.

@garypretty
Thanks! That'll work perfectly.

Hi @cleemullins.  I have updated the PR and moved the "delay" type into an ActivityTypesEx as discussed, but I am referencing this directly at the moment, i.e. ActivityTypesEx.Delay, because I can't extent it as a partial due to AcitivityTypes not being a partial at present.  Is this something we can change easily? We can commit this now, but if we can make ActivityTypes a partial it would bring it into line with many other of the schema classes and allow us to extend it moving forward if we need to.
Anyway, let me know if we are good to merge as we are or if you want to move anything around.

Thanks Gary! This is perfect.
I'll see what we can do about getting the Swagger file updated to generate the ActivityTypes as a partial class. In the short term, what's here is very reasonable.


Fallback language not found with specific fallbackLng option

Hello,
We are using a specific 'fallbackLng' configuration which looks like this
i18next.init({     fallbackLng: {          'de-CH': ['fr', 'it'],          'zh-HANT': ['zh-HANS', 'en'],         'es': ['fr'],         'default': ['en']     } });
This configuration should be supported as mentioned in the official i18n documentation : https://www.i18next.com/principles/fallback.html
But when I look at the code (see below), this specific configurations does not seem to be supported
https://github.com/i18next/i18next-browser-languageDetector/blob/2b17200c2680c168a7484308cd294a6e2c8d90bc/i18nextBrowserLanguageDetector.js#L240
In my case, this specific line should be
return found || this.i18nOptions.fallbackLng[0] || this.i18nOptions.fallbackLng['default'];
Is there any way we can support this?
Thank you very much

argh...i see...we missed that in this place (while we updated that in eg. the detector for express-middleware: https://github.com/i18next/i18next-express-middleware/blob/master/src/LanguageDetector.js#L71)
Will do an update asap.

should be fixed in<EMAIL_ADDRESS>

Thank you very much ;)

What would be the correct way the fallback language to use when the language cookie is not set?
I looked here:
https://www.i18next.com/overview/configuration-options
https://www.i18next.com/principles/fallback
It says "lng - language to use (overrides language detection)", but I don't want to override language detection (from the cookie), so I shouldn't set lng.
But if I set fallbackLng, it will use this language as a fallback on a per-key basis, I don't want that.
Instead, I only want to fall back to a certain language if and only if the initial language detection (from the language cookie) fails.
How can I specify this? :)

fallbackLng will act as a fallback - not sure what you mean by on a per-key basis. If there is no cookie your i18next.languages will be set to whatever the fallback is -> means it will load the fallback and use it...as there is nothing else to use.

@jamuhl The i18next.init option fallbackLng is not a fallback for what you mean.

it will use this language as a fallback on a per-key basis

It is right. Please read the document "locals resolving" once more.
I think it should be implemented with a fallback option in detector options.

fallback acts on a per key basis means if a key is not found in primary language it will take it from fallback - if found not. it still loads the complete namespaces for fallback lng. Acutally you can read i18next.languages and you get the ordered array of locales used.

fallback acts on a per key basis means if a key is not found in primary language it will take it from fallback - if found not. it still loads the complete namespaces for fallback lng. Acutally you can read i18next.languages and you get the ordered array of locales used.

Agree exactly. So that i18nOptions.fallbackLng is a fallback for i18next key resolving, not browser language detection.
Below is my scenario:
import i18next from 'i18next';
import XHRBackend from 'i18next-xhr-backend';
import LanguageDetector from 'i18next-browser-languagedetector';
import {initReactI18next} from 'react-i18next';

const i18nextOpts = {
    debug: false,

    fallbackLng: ['common', 'en'],

    react: {
        useSuspense: false,
    },

    detection: {
        // I remove all detectors to make sure it will use the i18nOptions.fallbackLng
        order: [],
    },

    interpolation: {
        escapeValue: false,
    },

    backend: {
        // ...backend options
    },
};

i18next
    .use(XHRBackend)
    .use(LanguageDetector)
    .use(initReactI18next)
    .init(i18nextOpts);

I have pre-built three resource files: zh-CN.json, en.json, common.json. Like that:
// common.json
// All common translations will be put in common.json
{"common":{"button":{"english":"English", "chinese":"中文"}}}

// en.json
{"hello": "你好"}

// en.json
{"hello": "hello"}

It will support that,
// when i18next.language = "en"
i18next.t('hello') // => "hello"
i18next.t('common.button.english') // => "English"
i18next.t('common.button.chinese') // => "中文"

// when i18next.language = "zh-CN"
i18next.t('hello') // => "你好"
i18next.t('common.button.english') // => "English"
i18next.t('common.button.chinese') // => "中文"

Actually, when all detectors found nothing, it will return the i18nOptions.fallbackLng[0]. Based this line.
i18next.language => "common"
i18next.languages // => ["common", "en"]

But the "common" is a resource name, not a browser language.

so do not fill common as a language...that's what namespaces are for.  https://www.i18next.com/overview/configuration-options#languages-namespaces-resources --> fallbackNS
https://www.i18next.com/principles/namespaces#sample
your files should have path/naming like: /{lng}/{ns}.json

for react also checkout the sample: https://github.com/i18next/react-i18next/tree/master/example/react

namespace is common translations for a specific language. I need a common texts across multiple languages. Now I realize that it is not a i18n problem. It should be handled by myself. Thanks for your help. @jamuhl


using pipe api: delimiter option not working for array objects

I am having a bit of trouble.
when the callback of the transformer returns a multiple line output like
[ [ '1', '2', '3', '4' ], [ 'a', 'b', 'c', 'd' ] ]
(the transformer took a line from the parser and created 2 lines)
and i want to use a delimiter option other than comma:
var stringifier = require('csv-stringify')({delimiter: '\t'});
The pipe process does not seem to follow the delimiter option set, it always uses comma.
rs.pipe(parser).pipe(transformer).pipe(stringifier).pipe(process.stdout);
I noticed that in the callback api the array is iterated over, but in the pipe not sure where it iterates.

Please provide me with a sample or I will have to close the issue.


add minimal setup.py

This setup.py allows hipify to be installed as python package
# install from branch
pip install git+https://github.com/ROCmSoftwarePlatform/hipify_torch.git@hipify_pypa
# install from commit
pip install git+https://github.com/ROCmSoftwarePlatform/hipify_torch.git@a219f525801421bc84def818538d0f9177e0620a

jenkins retest this please

Addressed via https://github.com/ROCmSoftwarePlatform/hipify_torch/commit/1c074754365c63da8a137cd96ade860764d38e65


Evaluator Review: Neutroswap Milestone 3 #164

Review, approve, or deny https://github.com/eosnetworkfoundation/grant-milestones/pull/164

Closed per @nsjames approval for https://github.com/eosnetworkfoundation/grant-milestones/pull/164.


Canvas lock and drag mechanism

Hi,
I would like to be able to have the canvas locked so I can drag around anywhere with the mouse (and not move any nodes), but as soon as you want to move a node, you can just click/select it and then it'll unlock the canvas so you can move the node around.
But, as soon as node is unselected, the canvas is automatically locked again by default.
Is this possible?
thanks.

There are several ways in the i editor if you can't use events to block.
See the next issue:

https://github.com/jerosoler/Drawflow/issues/584

I've tried locking the editor (editor.editor_mode = 'view'), that works great, but then I can't detect when a node is selected to unlock the editor so you can drag the node because editor.on('nodeSelected') doesn't work or return value when the editor is locked in "view" mode.
Do you have any suggestions on how I can use "view" mode for editor, but be allowed to also select a node or drag a node if I click on it?
thanks.

Try with events:

https://github.com/jerosoler/Drawflow/issues/321

Thanks for response. I had a look at #321.
Using those events, how would I implement it?
So is the editor mode set to "view" by default, and then when do I call "changeMode()" ?
Thanks.

Hello
Example with event change on click target.
    const editor = new Drawflow(id);
    editor.editor_mode = 'view';
    editor.on("click", (e) => {

        if(e.target.closest(".drawflow_content_node") != null || e.target.classList.contains("drawflow-node")) {
            // Click on node content or parent
            // Exclude inputs and ouptus
            editor.editor_mode = 'edit';
            console.log('edit');
        } else {
            // Click ouput node
            editor.editor_mode = 'view';
            console.log('view');
        }
    });
    editor.start();
```

Thanks for the great example Jero.
It's close to the functionality I was after, but with your code I still cannot drag the node in "view" mode. I only want to change to "edit" mode if the user clicks/selects the node for editing.
So I tried changing the code a bit and the code below works and unlocks the editor, but I need it to auto-select the node the user clicked on:
`	editor.on("mouseUp", (e) => {
if(x_click === last_x && y_click === last_y && (e.target.closest(".drawflow_content_node") != null || e.target.classList.contains("drawflow-node"))) {
editor.editor_mode = 'edit';
console.log('edit');
} else {
editor.editor_mode = 'view';
console.log('view');
}
});

`
Can you suggest how I can auto-select the node once the user clicks on it in the above code.
thanks.

I don't quite understand the behavior you are looking for.
This is how the library selects a node:
         this.node_selected = this.ele_selected;
         this.node_selected.classList.add("selected");

Sorry if I didn't explain it clearly. let me try again....
Basically, I don't want to force the user to do extra clicks on "unlock/lock" buttons. So I'm trying to create a more intuitive/smarter way of combining the 2 editor modes.
I would like the editor to be in the locked "view" mode at all times. But as soon as user wants to edit a node, they do a simple "click" on a node and it selects/highlights the node (unlocks editor). Or if they want to drag a connection line then this works too.
I think it's not hard to achieve, we just need to detect 2 states (Drag and Click):

If there are no nodes selected and the user is dragging, then just pan the canvas around (just like "view" mode).
As soon as user "clicks" on a node, highlight the node and the user can now drag the node around (just like "edit" mode).
Detect if user is dragging a connection input/output and allow the user to connect to another node (just like "edit" mode).

At all other times (when there are no nodes selected) when user is dragging, the editor is always in "view" mode.
Hope that makes it a bit easier to understand.

Hello,
As I see that the predominant method is "editor".
I think it is better that he is active.
Which we force deselection of the node.
Hope this can help you:
    var id = document.getElementById("drawflow");

    const editor = new Drawflow(id);

    let x_click = 0;
    let y_click = 0;
    let node_selected = null;
    let restric_move = true;
    editor.on("clickEnd", (e) => {
        x_click = editor.pos_x;
        y_click = editor.pos_y;
        if(editor.node_selected !=  null && restric_move) {
          node_selected = editor.node_selected;
          editor.node_selected.classList.remove("selected");
          editor.node_selected = null;
          editor.ele_selected = editor.container.firstChild;
          editor.drag = false;
          editor.editor_selected = true;
        } else {
          node_selected = null;
        }
    })

    editor.on("mouseMove", (e) => {
      x_click = e.x;
      y_click = e.y;
    })

    editor.on("mouseUp", (e) => {
        if(x_click === editor.pos_x && y_click === editor.pos_y && node_selected !=  null) {
          editor.node_selected = node_selected;
          editor.node_selected.classList.add("selected");
          restric_move  = false;
        } else {
          restric_move  = true;
        }
    });
    editor.start();

Jero, that is amazing.
Yes, keeping the editor in "edit" mode and restricting the move is better way to think about the solution. Brilliant work!
Thanks so much for writing that code out, it works perfectly and exactly what I was trying to achieve.


0.0.4

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/119244
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.0.4 -m "<description of version>" aa6f300f26a5edd89fc04dfbd3e4f5b7df97ff01
git push origin v0.0.4


FileSystem Filter driver limitation when AccessMode is KernelMode

If the AccessMode parameter is KernelMode, the requested access is always allowed. If AccessMode is UserMode, the requested access is compared to the granted access for the object.
In the description of this function it is mentioned that when AccessMode is KernelMode, the request access is always granted. So even in the minifilter driver's POB_PRE_OPERATION_CALLBACK callback when some of the accesses are denied by the driver, those will be ignored by the OS and requested access will be granted? Could you please clarify

@SJKU , can you clarify which pages you want clarity on - please provide a link to the page for this issue and for issue #1227 .

Hi Iorihollasch,
This is link for the page
https://docs.microsoft.com/en-us/windows-hardware/drivers/ddi/ntifs/nf-ntifs-obopenobjectbypointer
At the bottom of the page it is mentioned that
If the AccessMode parameter is KernelMode, the requested access is always allowed. If AccessMode is UserMode, the requested access is compared to the granted access for the object.
When I tested with a minifilter driver that is masking off some of the access mask, it was not honored by the OS and the new returned handle has access that it has asked.

@sjku, we're still having a bit of difficulty parsing your issue. Based on what you posted on Q&A (which I discovered while researching this issue), the dev team is wondering whether you called FltSetCallbackDataDirty after amending the contents of the callback data structure. See my answer on Q&A, where I also posted additional resources for you if you are somewhat new to minifilter development. I'm going to close this issue out since it seems to not be a doc issue - feel free to reopen if you still feel the docs need updating.

Hi Iorihollasch,
I am not talking about preoperation or postoperation callback in the minifilter driver. I have registered for OB_CALLBACK_REGISTRATION in my filesystem filter driver class that is an "ActivityMonitor" category. So my driver has registered prehandle create callback MydriverPreHandleCreateCallback, so this function will be invoked when any kernel component calls the obopenobjectbypointer function.
ProcCreateOperation[0].PreOperation = MydriverPreHandleCreateCallback;
ObOpenObjectByPointer -> ObpCreateHandle -> ObpCallPreOperationCallbacks -> MydriverPreHandleCreateCallback
NTSTATUS ObOpenObjectByPointer(
[in]           PVOID           Object,
[in]           ULONG           HandleAttributes,
[in, optional] PACCESS_STATE   PassedAccessState,
[in]           ACCESS_MASK     DesiredAccess,
[in, optional] POBJECT_TYPE    ObjectType,
[in]           KPROCESSOR_MODE AccessMode,
[out]          PHANDLE         Handle
);
In my driver callback I am modifying the content of the POB_PRE_OPERATION_INFORMATION parameter for example removing PROCESS_CREATE_THREAD permission. So I am expecting that the handle that is returned to the caller at the end of the ObOpenObjectByPointer() will not have PROCESS_CREATE_THREAD permission. But when the AccessMode parameter of the obopenobjectbypointer is KernelMode, whatever my driver has modified that was not reflected when the handle returned to the caller. When I check the handle permission at the end of the ObOpenObjectByPointer function, the handle has PROCESS_CREATE_THREAD permission.
So I need clarification on this. Why even if the OB_CALLBACK_REGISTRATION callback modifies the content of the POB_PRE_OPERATION_INFORMATION parameter that was not honored by the OS.

@sjku, I assumed you were the author of this question on Q&A. Did you call FltSetCallbackDataDirty after modifying the content of POB_PRE_OPERATION_INFORMATION?

@SJKU , sorry for the delayed response. I have confirmed with the dev team that the behavior is expected and the documentation was correct: If AccessMode=KernelMode, the OS always allows the requested access, regardless of previously set limitations. I added your situation as a "for example" to the page.

Thanks Iorihollasch. Just to convey this the development team. Granting always access creates an issue. For example. in my filter driver, I add some restriction to the handle in the POB_PRE_OPERATION_CALLBACK callback. Any third party code with MS digitally signed driver can use KERNEL_MODE and gets the full access to the handle.