Move documentation inline and rework documentation

This commit is the beginning of an overhaul of the documentation for
retworkx. Previously the entire structure of the retworkx documentation
was hand built as rst files. When the project first started this was
fine because there was 1 class, PyDAG and only a few functions that used
PyDAG, but now that the project has started to grow dumping everything
into a single rst file hasn't scaled well. This resulted in several
problems primarily the formatting was all wrong, but also that the
structure was poorly organized. The first step this takes is moving the
rst files to inline docstrings, PyO3 is capable of exporting the rust
docstrings and function signatures to python so we should leverage that
capability, that way interactive help tools (like with ipython and
jupyter) can see the docs for everything, but also we can leverage
sphinx autodoc to help with the structure and formating. From there the
organization of the docs is reworked to split up the PyGraph, PyDiGraph,
and PyDAG into one section and then to put the algorithm functions into
another section.
TODO:

[ ] Get autosummary and sphinx layout to work correctly

Coverage remained the same at 85.121% when pulling 6b8a92d07f5653e87ab7fca3f2cf71848ea2dfe8 on mtreinish:rework-docs into 55176a359d5b99e8aba85c27fa42feabe029d822 on Qiskit:master.


tests: travis build fix

Coverage remained the same at 100.0% when pulling a417cf00f603cc83018f94d8bed5b33859768e25 on lnielsen:buildfix into f908749e8b7836f99d00625c991a643a1e844580 on inveniosoftware:master.


[IOTDB-1524] Support SELECT ... INTO ... clause

https://issues.apache.org/jira/browse/IOTDB-1524

Coverage decreased (-0.02%) to 68.144% when pulling 9067cf98729c6245417327df50783b3f782974c1 on select-into into 3c6925ce4746e9dfd3b2de1afcf5d6f49223454b on master.


Update Java Samples with try-with resources

@ajlam,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


[Feature] Can I combine multiple instructions into one event and undo them all at once?

Initial checklist

[X] Is this really a problem?
[X] I have searched the Github Issues for similar issues, but did not find anything.

Problem
我可以把多个指令合并成一次事件， 在执行撤销的时候一次撤销掉吗
假设我 发出了三个指令但是我只想记录成一次事件
Solution
有这样的方法 或者我怎么做可以模拟

https://univer.ai/zh-CN/guides/sheet/advanced/custom-command#undo--redo


Signify (Hue) LWB006 alias LWB014

Redo of #1476, but individually.

Need some more research as I think these are different generations of the bulb, having different power characteristics.
https://huehomelighting.com/key-differences-1st-2nd-3rd-gen-philips-hue-bulbs/
LWB006 seem to be a generation 2 light.
Found something about LWB014 on reddit: "I believe that the latest one available in the US is the LWB014. Contrary to previous versions it's a sizeable upgrade in terms of brightness as it delivers 840 lumen instead of 800 or 806", so it's definitely a different bulb in terms of brightness / lumen / power consumption.
Maybe you can do some more research as well.
But this PR seems to be a no go for me.

As commented above, power consumption is probably different for these light models so we cannot add the aliases. Closing this PR.


Is there a method similar to React.createElement？

I want to dynamically create html tags.

i got it.

Sure you found the answer you are looking for?
document.createElement is the cheeky answer but it isn't far off.
import { spread } from "solid-js/web"

function create(tag, props) {
  const el = document.createElement(tag);
  spread(el, props);
  return el;
}

That being said doesn't benefit from compiler optimizations. Same logic present in <Dynamic> control flow. If you can avoid spreads do so, so if it has known attributes better to just create it imperatively.. You can always directly insert DOM nodes in Solid as:
const div = <div />
console.log(div instanceof HTMLDivElement) // true

yes.  can satisfy！😬

@0x219 , what was the resolution you came to when you said i got it.?

@0x219 , what was the resolution you came to when you said i got it.?

The author's answer and the Dynamic component


Returning 0 tweets but it's impossible

Make sure you've checked the following:

[] Python version is 3.6; ✅
[] Updated Twint with pip3 install --user --upgrade -e git+https://github.com/twintproject/twint.git@origin/master#egg=twint; ✅
[] I have searched the issues and there are no duplicates of this issue/question/request. ✅

Command Ran
The class I made for Twint:

The implementation:

Description of Issue
Hello,
I am developing a live scraping tool using Twint library, where...

dates are supposed to update every n minutes in "window way" (end date at time t becomes new start date at t+1);
scraper is ran in that interval;
csv for that small interval is saved locally;
repeat.

However, Twint scrapes 0 tweets in an interval where it's OBVIOUS that there are tweets.
For instance, when I ran the scraper between 21:45 and 23:45, with a limit of 30000 tweets, I get the following count histogram of tweets found.

When I make the window shorter from 22:45 to 23:45 (code is the same, just the time delta is 60*60), 0 tweets are returned and the csv file is not saved locally.
This is impossible as the histogram above says that in that interval some people have made tweets which meet my search criteria.

Could someone please help me out with this? I googled already and found some results saying I have to remove "lang" parameter, I have to save every small batch locally (which is what I am already doing btw), and much more which turned out to be uneffective.
I am using Twint version 2.1.21 (the latest).
Also, another question is: could it be that Twint has problems downloading data when the interval is small (like 30 minutes, 10 minutes)?
Environment Details

I am using Windows, the code is ran locally on Jupyter Notebook.

I have the same issue, when interval is less then hour it's shows zero tweets ,
If you have any solution pls help me

Bump
I'm bumping this as a lot of people are having this problem. If I launch 10 scrapes, 9 of these return 0 tweets even when keywords are like "hello", which is obviously not possible.
Still is not resolved, is there a solution?


Allow creation of elements for invisible objects

Is your feature request related to a problem? Please describe.
I'm wanting to add temperature sensor labels to several rooms, but the only way I know of to do this is to add an object to that room and name it "sensor.living_room_temperature" for example.  The problem with this is now I have an unwanted object somewhere in my floorplan render.
Describe the solution you'd like
It would be great if we could create elements for objects that are marked as not visible in SH3D.
Describe alternatives you've considered
The only alternative that I know of is to pick an object in the room that you actually do want, and name it after the sensor.  This isn't a great solution as there are several spots where I would like the temperature reading to be located but I don't have an object in that spot in the render.  Adding an invisible object so that it does not show in the render, but still creates an element, would be perfect.

I just now found #89, I'll add my proposal as a comment there instead and close this.


Feature request: Support for configuring stats

I'm submitting a feature request
Webpack version:
4.x
Webpack Karma version:
3.0.0
Karma version:
2.0.2
Please tell us about your environment:
Windows 10
Current behavior:
Stats field in supplied configuration is ignored - default value is used when outputting stats
Expected/desired behavior:
Supplied stats field is used to modify output reported to karma.

What is the motivation / use case for changing the behavior?
I don't want the list of compiled modules in the output - it makes it harder to find compile errors.

Looks like this may not be possible - webpack nodejs interface does not support it.

Looking into configuring it on webpack dev middleware

palm to forehead I should read the README closer next time.


GPIO interrupt example

Hi there again. i love your code... at the moment i am playing with you GPIO classes and have noticed you allow to enable and disable interrupts.
My question would be, where are the Interrupt ISR's handled as i dont see any way for notification on interrupt triggering. normally you expose a callback function pointer or something

Let me look at this ... it is possible I haven't added interrupt handling ... if that's the case, I'll make that a priority to get added.

added some basic methods to the class will share a bit later with you…

On 12 Jul 2017, at 5:10 PM, Neil Kolban<EMAIL_ADDRESS>wrote:
Let me look at this ... it is possible I haven't added interrupt handling ... if that's the case, I'll make that a priority to get added.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/nkolban/esp32-snippets/issues/21#issuecomment-314800461, or mute the thread https://github.com/notifications/unsubscribe-auth/ADQtyOXNRjqSRY37cquLo1pPeQgUkfutks5sNOH5gaJpZM4OT-Ib.

I have the same issue.
My setup: INPUT_PULLUP.
When press button, many interrupts occurred continuously.

Correct me if I am wrong, but while the pin remains high you will be getting new interrrupts continiously. A single button press is a really long time for the processor, so when you exit your ISR re-enabling interrupts, it will fire again.


Auth user emails

What did you implement:

Added "fn authenticated_emails" and "UserEmail" struct in "src/users.rs", which fetches from github's "/user/emails"
Added a usage example of the new function in "examples/users.rs"

Closes: #277
How did you verify your change:

Since there isn't a generic test suite for api calls, I added it to "examples/users.rs" and eyeballed it.

What (if anything) would need to be called out in the CHANGELOG for the next release:

A new users feature that allows getting "/user/emails"

Looks good. Thanks @dt665m


Process completed with exit code 1.  even when code coverage succeed

Describe a bug
i'm currently pointing to master everything was working as expected but over the weekend something changed now all the builds exit with code 1 even when the code coverage command successfully complete

Is there a way to call uses: artiomtr/jest-coverage-report-action@master specifying a commit hash in master ? i think the latest commit broke everything 😞
Expected behavior
Exit command code to be success
Details

Action version:

OS, where your action is running (windows, linux):

action.yml file
```yml
# Insert your action.yml file here. Don't forget to remove all sensitive information (e.g. tokens)
```

Screenshots
<!-- If you encounter an incorrect coverage comment display, replace this comment with screenshot -->

<!-- If your action unexpectedly fails, please replace this comment with a screenshot of your console  -->

Additional context

This was actually my fault there was failing test ⚠ Closing this issue


dashcastx multi res (and usage in general) is broken

Almost there. The DASH muxer wrongly creates one adaptation set per media.


"Notice: Array to string conversion" when trying to use an array of filters

According to documentation:

filter can be array of filters in this case serialized field will be also array. For example if you add annotation @Bukashk0zzz\LiipImagineSerializableField(filter={"big", "small"})

However this seems to be broken or something. All I get is that error:

Notice: Array to string conversion

from class

Symfony\Component\Debug\Exception\ContextErrorException

If I put instead filter="small" it works and serves the image URL with the filter "small". But I can't find a way to serve all filters for the image (using VichUploader also) and I really need it for my API. Any other way or any clue to solve this?

Hi,
Example from project that now in production
@Bukashk0zzz\LiipImagineSerializableField(filter={"apartment_440", "apartment_1000"}, vichUploaderField="imageFile", virtualField="imageThumbs")

Thank you very much, works like a charm. Any chance to add it to documentation to make it crystal clear? So others doesn't make my mistake.
Documentation says:

Also there is another two not required options: vichUploaderField and virtualField

I guess it's the "not required" part that got me. Of course I knew that I needed vichUploaderField was required but the virtualField could have a part where it says "it's required if you're using an array of filters" or something similar.
And/or adding it in the bottom example.
Anyways, great work with the bundle. And thanks again for your quick help.
BTW, nice avatar, The Chemical Brothers FTW haha.

@migmolrod thanks. I will update doc.


The leaderboard is missing in https://paperswithcode.com/sota/automated-theorem-proving-on-leandojo

Check again but found the leaderboard is removed completely, closed...


Viewing dashboard microk8s -MACosX

How do I view the kubernetes dashboard on the Mac OSx with the brew installation of the microk8s.
In multipass I input microk8s kubectl proxy so I can attempt to view the dashboard at http://localhost:8001/api/v1/namespaces/kube-system/services/https:kubernetes-dashboard:/proxy/ however I'm having trouble.
I've tried port forwarding on Virtual Box with a rule for host port: 8001 and guest port: 8001 but that's not working either.
I thought maybe I can try ssh -L 8001:localhost:80 username@hostname, but I don't know the username and hostname of the multipass login.
Please help.

I also tried the instructions on the microk8s website and it results in Client sent an HTTP request to an HTTPS server

I got this to work by https:// instead of http.


Window open

Could somebody implement the function that i can open the window? Or is the API blocking that?

It is always red if i do so

Hi,
All of my Velux shutters works fine with target_position :-)
Log entry: {"status":"ok","time_server":16374*****}
But for my Velux windows, only target_position 0 has an effect, ie the windows are closed.
All other values has no effect and results with an error.
Log entry: {"status":"ok","time_server":16374.....,"body":{"home":{"id":"5eb3e9cf3fc5e9**********"},"errors":[{"code":9,"id":"563613**********"}]}}
Is this is a Velux security setting, that they don't allow to open wondows this way?
Do you have a solution for this problem?

Yes opening needs a pairing with the router.
An idea is to add the windows in homekit and use the new homekit adapter
https://forum.iobroker.net/topic/48570/test-alpha-homekit-controller-0-0-x

Thanks for your answer!
I’ll be looking at this homekit-controller adapter, because pairing with the router isn't an option, is it?

I dont have a router and was not able to see the traffic

I connected the Velux direct to the iobroker with the homekitadapter and everything works fine. Now I us homekit for window movments and velux for the rest.

When using the Homekit-Adapter, will it also work (for windows and blinds), when the Velux-Gateway has no internet access?

No it won't. Velux need internet for no reason.

I gave it a try yesterday. After 5-6 attempts I could pair the gateway with homekit. Then I cut the internet connection from the velux gateway, which I could verify by checking with the velux active app saying "connection to gateway lost". I was still able to control windows with homekit adapter. Today morning I power cycled the gateway since I need to move it to another place and afterwards no homekit connection was possible, unless the device answered my ping requests. When I allowed internet connection via firewall for a few minutes the gateway was available on homekit again.


Error in chaining two consecutive @count

I want to count the number of groups having the same number of elements. In R, this can be archived using a consecutive call of count:
library(dplyr)
df <- tibble(id=1:10, group = c(1,1,1,1,1,2,2,2,2,2))
df |>
  count(group) |>
  count(n)
#> Storing counts in `nn`, as `n` already present in input
#> ℹ Use `name = "new_name"` to pick a new name.
#> # A tibble: 1 × 2
#>       n    nn
#>   <int> <int>
#> 1     5     2

Indeed, there are 2 groups having 5 elements each. However, this can not be archived with Tidier.jl:
using Tidier
df = DataFrame(id=1:10,group=[1,1,1,1,1,2,2,2,2,2])
@chain df begin
    @count(group)
    @count(n)
end
# ERROR: ArgumentError: column :n in returned data frame is not equal to grouping key :n
# Stacktrace:
#  [1] _combine_prepare_norm(gd::DataFrames.GroupedDataFrame{DataFrame}, cs_vec::Vector{Any}, keepkeys::Bool, ungroup::Bool, copycols::Bool, keeprows::Bool, renamecols::Bool, threads::Bool)
#    @ DataFrames ~/.julia/packages/DataFrames/58MUJ/src/groupeddataframe/splitapplycombine.jl:104
#  [2] _combine_prepare(gd::DataFrames.GroupedDataFrame{DataFrame}, ::Base.RefValue{Any}; keepkeys::Bool, ungroup::Bool, copycols::Bool, keeprows::Bool, renamecols::Bool, threads::Bool)
#    @ DataFrames ~/.julia/packages/DataFrames/58MUJ/src/groupeddataframe/splitapplycombine.jl:52
#  [3] _combine_prepare
#    @ ~/.julia/packages/DataFrames/58MUJ/src/groupeddataframe/splitapplycombine.jl:26 [inlined]
#  [4] combine(gd::DataFrames.GroupedDataFrame{DataFrame}, args::Union{Regex, AbstractString, Function, Signed, Symbol, Unsigned, Pair, Type, DataAPI.All, Between, Cols, InvertedIndices.InvertedIndex, AbstractVecOrMat}; keepkeys::Bool, ungroup::Bool, renamecols::Bool, threads::Bool)
#    @ DataFrames ~/.julia/packages/DataFrames/58MUJ/src/groupeddataframe/splitapplycombine.jl:861
#  [5] macro expansion
#    @ ~/.julia/packages/TidierData/hI0Pq/src/TidierData.jl:324 [inlined]
#  [6] top-level scope
#    @ REPL[3]:70

Moved this issue to the TidierData.jl package because Tidier.jl is a meta-package.


Cancellation tokens are not used in implementations

Looking at issues like this:
https://github.com/StefanMaron/BusinessCentral.LinterCop/issues/284
It seems there are some rules that may be long running. I'd suggest invoking
context.CancellationToken.ThrowIfCancellationRequested() in reasonable checkpoints in the code .
This ensures that if the user types something while the rule is running, the compiler will not have to wait for long running rules to finish and should improve the perceived performance of all rules.

That seems like a good idea to do. Is that something you're willing to create a PR for? Or maybe can you share an example of an implementation of this?

Am I doing this right?

Ive added some cancellation tokens within the foreach loops, unfortunately this decreases the performance.
Before

After

It could have a beneficial effect on working in VS Code, where modifying the AL code would interrupt this rule from executing the current run and start a new one.
I'm afraid to solve the performance issue on this, is to improve the code itself. This is a complex rule covering all kind of exceptions, which makes is hard to implement changes without breaking the rule.

With the improvements brought by @dannoe, it seems that cancellation tokens aren't needed anymore.

Oops, that's not right! 😳
The cancellation tokens are now implemented and with the other code improvements together, this has resolves the performance issues.

Oops I missed this in a sea of notifications.
Awesome work, as you mentioned before it's about VSCode perceived performance. Cancellation tokens can't increase performance but they can make the diagnostics snappier because the rule will be cancelled faster on code changes.
It made a big difference on the microsoft AL analyzers.
My suggestion is to always employ cancellation tokens on all rules and the best place to check for it is in larger bodies of work. So it's not worth checking for it on every single tree node but it's worth checking it on every symbol or application object.


Make big social button text customizable

Currently the header of small social buttons is customizable, but the prefix text of big social buttons text ('Login with') is not.
This PR makes it customizable via dict.

Not exactly sure what's with the build failure 😕

Thanks! I'll take a look at the build failure.

Thank you @thameera !! 👍 I really will benefit from your work, in case anybody needs to change the buttons (hackish version), check out #116 while we wait for the merge.

We're actually merging Passwordless and Lock into a single project. You can follow the progress here. There'll be no active development on this project anymore. We'll fix security issues only for the near future.


Add  rel="noopener noreferrer" to sample privacy hyperlinks

Fixes #257
References:

https://owasp.org/www-community/attacks/Reverse_Tabnabbing
https://stackoverflow.com/a/50709724

Thanks for the PR!


Latest releases of Histoire and Nuxt are not compatible

Describe the bug
Versions:
histoire, @histoire/plugin-nuxt, @histoire/plugin-vue : 0.15.8
nuxt: 3.2.3
Hi!
When running story:dev with the latest releases of Histoire and Nuxt, an Iframe containing the rendered app.vue file gets displayed below the actual story (See image below). If you scroll down the page on the repro, you will notice that the same page is also displayed below the viewport.
A few errors are also thrown at startup, but don't seem to prevent the app from running.
Sourcemap for "/__resolved__virtual:$histoire-theme.css" points to missing source files
stories/test.story.vue 4302ms (setup:31ms, execute:3183ms, run:1088ms)
[nuxt] error caught during app initialization ReferenceError: document is not defined
    at normalizeContainer (file:///home/projects/nuxt-starter-7hjhxl/node_modules/@vue/runtime-dom/dist/runtime-dom.cjs.js:1616:21)
    at app.mount (file:///home/projects/nuxt-starter-7hjhxl/node_modules/@vue/runtime-dom/dist/runtime-dom.cjs.js:1537:27)
    at initApp (file:///home/projects/nuxt-starter-7hjhxl/node_modules/nuxt/dist/app/entry.mjs:55:14)
Collect stories end 17800ms
Collect stories start all
stories/test.story.vue 96ms (setup:57ms, execute:1ms, run:38ms)
Collect stories end 108ms

Thanks for this awesome project, can't wait to use it with Nuxt!
Reproduction
https://stackblitz.com/edit/nuxt-starter-7hjhxl
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    @histoire/plugin-nuxt: ^0.15.8 => 0.15.8
    @histoire/plugin-vue: ^0.15.8 => 0.15.8
    histoire: ^0.15.8 => 0.15.8

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

Additional reproduction here: https://stackblitz.com/edit/histoire-nuxt3-starter-eb2jnf
(This is the same stackblitz i proposed in #471)
I also reproduced it locally by cloning this repo and making the appropriate updates (instead of displaying the demo page it also displays what's inside the app.vue template, being hi there in the local nuxt example).
Note: if adding pages templates you would get 404s because of nuxt's vue-router errors not recognising histoire routes.


AttributeError: 'list' object has no attribute 'merge'

 cat in.txt | pdml2json > /dev/null
Traceback (most recent call last):
  File "/usr/bin/pdml2json", line 9, in <module>
    load_entry_point('pdml2flow==1.0', 'console_scripts', 'pdml2json')()
  File "/usr/lib/python3.5/site-packages/pdml2flow-1.0-py3.5.egg/pdml2flow/__init__.py", line 81, in pdml2json
  File "/usr/lib/python3.5/site-packages/pdml2flow-1.0-py3.5.egg/pdml2flow/__init__.py", line 102, in pdml2frame
  File "/usr/lib/python3.5/site-packages/pdml2flow-1.0-py3.5.egg/pdml2flow/__init__.py", line 115, in start_parser
  File "/usr/lib/python3.5/xml/sax/expatreader.py", line 110, in parse
    xmlreader.IncrementalParser.parse(self, source)
  File "/usr/lib/python3.5/xml/sax/xmlreader.py", line 125, in parse
    self.feed(buffer)
  File "/usr/lib/python3.5/xml/sax/expatreader.py", line 210, in feed
    self._parser.Parse(data, isFinal)
  File "/build/python/src/Python-3.5.1/Modules/pyexpat.c", line 414, in StartElement
  File "/usr/lib/python3.5/xml/sax/expatreader.py", line 322, in start_element
    self._cont_handler.startElement(name, AttributesImpl(attrs))
  File "/usr/lib/python3.5/site-packages/pdml2flow-1.0-py3.5.egg/pdml2flow/pdmlhandler.py", line 47, in startElement
AttributeError: 'list' object has no attribute 'merge'

Input is a normal pdml file (2GB).
Any Idea what that could be?
I can send you the pcap file (100mb) via email/och, if you need it.

This could happen if a frame contains a field ending in .raw or . show preceeded by a similar field not ending in .raw or .show. But this should not happen.. Added debug messages in commit 6e3db40ecc95a2acc0eb31cf04527541dfd3a17a which might help solving this issue. Can you please re-run the test in debug and send me the last couple of debug messages before the program crashes.

also this is a frame-local problem. Thus attaching the last frame beeing processed before the script crashes could help a lot.

Sure I can try to fetch those debug messages for you. Or if you don't want to wait, you just run the dump I sent you (29.4 via email) yourself....

[Debug:<PHONE_NUMBER>.162531]  field: smb.write.mode.raw
Traceback (most recent call last):
  File "/usr/bin/pdml2json", line 9, in <module>
    load_entry_point('pdml2flow==1.0', 'console_scripts', 'pdml2json')()
  File "/usr/lib/python3.5/site-packages/pdml2flow-1.0-py3.5.egg/pdml2flow/__init__.py", line 81, in pdml2json
  File "/usr/lib/python3.5/site-packages/pdml2flow-1.0-py3.5.egg/pdml2flow/__init__.py", line 102, in pdml2frame
  File "/usr/lib/python3.5/site-packages/pdml2flow-1.0-py3.5.egg/pdml2flow/__init__.py", line 115, in start_parser
  File "/usr/lib/python3.5/xml/sax/expatreader.py", line 110, in parse
    xmlreader.IncrementalParser.parse(self, source)
  File "/usr/lib/python3.5/xml/sax/xmlreader.py", line 125, in parse
    self.feed(buffer)
  File "/usr/lib/python3.5/xml/sax/expatreader.py", line 210, in feed
    self._parser.Parse(data, isFinal)
  File "/build/python/src/Python-3.5.1/Modules/pyexpat.c", line 414, in StartElement
  File "/usr/lib/python3.5/xml/sax/expatreader.py", line 322, in start_element
    self._cont_handler.startElement(name, AttributesImpl(attrs))
  File "/usr/lib/python3.5/site-packages/pdml2flow-1.0-py3.5.egg/pdml2flow/pdmlhandler.py", line 48, in startElement
AttributeError: 'list' object has no attribute 'merge'

exactly what I expected:
[Debug:<PHONE_NUMBER>.162531]  field: smb.write.mode
[...]
[Debug:<PHONE_NUMBER>.162531]  field: smb.write.mode.raw

Don't know how to fix this properly atm. I've to name the fields somehow If these names are beeing used by tshark later on I will run into a conflict. So I could:

don't appen the value when the exception is raised
rename raw to something extremly complex which is never used as fieldname
do something else..

added a test (currently skipped) which replicates the issue we're facing here. Note that pdml2flow does not raise this exception if this behaviour is split into two frames, as shown here. So is this (same as in test_issue-7.1):

    "information": {
      "raw": [
        123,
        {
          "raw": [
            321
          ]
        }
      ]
    }

how you'd like to have to output formatted? Please tell me and I'll try to implement it.

The proposed output is fine for me. I don't really care about the output format, as long as no exception is raised.


Improve cipherUpdate

Don't use unsafe import for functions which pass control to C for unbounded amount of time since then a Haskell thread that runs them can't be preempted. In fact, it might be best to import everything as safe and change some functions to unsafe later as an optimization pass (if it's even needed).

Don't make a copy of the input text - when performing operations on large ByteStringS, duplicating the memory used by them is not desirable. This also applies to other relevant functions that use asBytesLen where unsafeAsBytesLen is sufficient. ByteStringS are pinned, so doing this is fine.

ping

Pong! I've been a little too focused on the X509 stuff lately. This will be included in the next update; I left off doing it immediately because there is also this PR to consider but that PR has breaking changes and more, whereas I can pull this in now.


Suggestion - Warped and Crimson slabs and planks and stairs

I believe they could have a texture that somewhat makes almost invisible the lines that make them look like planks. Since they are pilotable and would bring an add of interesting colors for details on ships. The idea is to make it look more like bricks or "stoney" like instead of wood like as it is now!
See this example! it could look a lot better after changing those planks lines!

@Gutin1 what do you think

It could probably be done. I don't really see anyone using them as wood on this server.


Gradle takes long to initialize before the build even starts

When I run Gradle build on my Windows10 laptop (gradle wrapper or not) it just sits there for up to 10-15 seconds before something is logged indicating start of Configuration phase.
Debug log starts earlier than Configuration phase (about 3 seconds after execution)  and shows some gaps but I have no clue whether there is a way to speed things up
2021-02-15T10:49:53.835+0100 [INFO] [org.gradle.internal.nativeintegration.services.NativeServices] Initialized native services in: C:\Users\username\.gradle\native
2021-02-15T10:49:53.916+0100 [LIFECYCLE] [org.gradle.launcher.cli.DebugLoggerWarningAction]
#############################################################################
   WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING WARNING
   Debug level logging will leak security sensitive information!
#############################################################################
2021-02-15T10:49:53.962+0100 [DEBUG] [org.gradle.internal.nativeintegration.services.NativeServices] Native-platform posix files integration is not available. Continuing with fallback.
2021-02-15T10:49:57.197+0100 [DEBUG] [org.gradle.launcher.daemon.client.DaemonClient] Executing build f3347a0e-2f9e-4b6b-abcd-b720fb6a167d in daemon client {pid=15440}
...

I do not know whether this is gradle bug or some corporate goodness on my laptop but I was hoping there was a way to understand what is gradle doing that takes so long. Any hints on how to debug this?
Expected Behavior
Debug log should help identify what is taking long time during gradle initialization
Current Behavior
Debug log starts after 3 seconds and has gaps but I was unable to determine what is the problem. I assume some resource takes time to get.
Context
With 30 projects in Eclipse workspace, it can take Buildship ages to initialize task lists and maintain projects in bulk in general.
Steps to Reproduce
I am pretty sure this is something to do with corporate pseudo security measures. I have the problem i general when running gradle builds. Nothing special.
Your Environment
Windows10, Gradle 6.8.3 (but I tested today with versions down to 3.5). AdoptOpenJDK Java8 (Same for AdoptOpenJDK Java11)
Build scan URL for "gradlew tasks" for empty project created by "gradle init"

without --debug: https://gradle.com/s/dalu5tkvujqxq
with --debug: https://gradle.com/s/rqzacdg433kpu

this went mostly away and sice there is no help with debugging  am closing this


[Adaptive Cards - Customer Credit Request]: All the edit form fields are not associated with their respective labels.

Target Platforms
Other
SDK Version
105.0.1300.0
Application Name
Adaptive Cards
Problem Description
Test Environment:
OS Version: Dev (OS build 21390.1)
Browser Version: 93.0.961.10 (Official build) dev(64-bit)
URL: Designer | Adaptive Cards
Screen Reader: Narrator
Pre-requisites:
Copy the complete json code in ''Card Payload Editor". (Please refer attachment).
Repro Steps:
Launch the above URL.
Navigate to header control ‘Designer’ using tab key and activate through enter key.
Navigate through the edit fields present.
Observe the issue.
Actual Result:
All the edit form fields are not associated with their respective labels.
Expected Result:
All the edit form fields should be associated with their respective labels.
Observation:
Issue is also repro with JAWS and NVDA.
User Experience:
Screen readers users will face difficulty if All the edit form fields are not associated with their respective labels.
MAS Reference:
Headings and Labels - Liquid (microsoft.com)
Have feedback to share on Bugs ? Please help fill Trusted Tester Bug Feedback (office.com) “
Screenshots
https://user-images.githubusercontent.com/96283152/181273116-5c4b68ab-9ba6-465c-aeae-f20560414809.mp4
https://user-images.githubusercontent.com/96283152/181274364-74644888-8912-4327-9192-9fb0b03c5837.mp4
Card JSON
{
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "type": "AdaptiveCard",
  "version": "1.2",
  "body": [
    {
      "type": "ColumnSet",
      "columns": [
        {
          "type": "Column",
          "width": 2,
          "items": [
            {
              "type": "TextBlock",
              "text": "Please enter the below details to create Customer Credit request",
              "weight": "bolder",
              "wrap": true
            },
            {
              "type": "TextBlock",
              "text": "Invoices (Number - Amount(Currency)) *",
              "wrap": true
            },
            {
              "type": "Input.ChoiceSet",
              "id": "InvoiceNumber",
              "style": "expanded",
              "isMultiSelect": true,
              "choices": []
            },
            {
              "type": "TextBlock",
              "text": "Total Number of Hours to be Billed",
              "wrap": true
            },
            {
              "type": "Input.Number",
              "id": "InvoiceHours"
            },
            {
              "type": "TextBlock",
              "text": "Total Number of Units to be Billed",
              "wrap": true,
              "isVisible": "${HasUnitBasedDemands}"
            },
            {
              "type": "Input.Number",
              "id": "InvoiceUnits",
              "isVisible": "${HasUnitBasedDemands}"
            },
            {
              "type": "TextBlock",
              "text": "Total Amount of Expenses to be billed",
              "id": "InvoiceAmountTextBlock",
              "wrap": true
            },
            {
              "type": "Input.Number",
              "id": "InvoiceAmount"
            },
            {
              "type": "TextBlock",
              "text": "Transaction From Date"
            },
            {
              "type": "Input.Date",
              "id": "FromDate"
            },
            {
              "type": "TextBlock",
              "text": "Transaction To Date"
            },
            {
              "type": "Input.Date",
              "id": "ToDate"
            },
            {
              "type": "TextBlock",
              "text": "Comments",
              "wrap": true
            },
            {
              "type": "Input.Text",
              "id": "Comments",
              "isMultiline": true,
              "isRequired": true
            }
          ]
        }
      ]
    }
  ],
  "actions": [
    {
      "type": "Action.Submit",
      "title": "Submit",
      "data": {
        "id": "CreditRequest_Submit"
      }
    }
  ]
}

Sample Code Language
No response
Sample Code
No response

Update: This should likely be handled in the larger designer overhaul discussed here (https://github.com/microsoft/AdaptiveCards/issues/7436).
We've had some changes to focus on peers instead of card elements while in design mode. Since the peers now get focus, we need to determine how screen readers should announce the peers while in design mode.
This specific issue still occurs when in preview mode.

Hello @Shireen2000,
This bug is fixed in schema 1.3 with the addition of the label property. By using labels for Inputs instead of TextBlocks, the form fields will be correctly associated with the text and Narrator should not have an issue.
Here is an example of the modified card payload.
{
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "type": "AdaptiveCard",
  "version": "1.3",
  "body": [
    {
      "type": "ColumnSet",
      "columns": [
        {
          "type": "Column",
          "width": 2,
          "items": [
            {
              "type": "TextBlock",
              "text": "Please enter the below details to create Customer Credit request",
              "weight": "bolder",
              "wrap": true
            },
            {
              "type": "Input.ChoiceSet",
              "id": "InvoiceNumber",
              "style": "expanded",
              "isMultiSelect": true,
              "choices": [],
              "label": "Invoices (Number - Amount(Currency)) *"
            },
            {
              "type": "Input.Number",
              "id": "InvoiceHours",
              "label": "Total Number of Hours to be Billed"
            },
            {
              "type": "Input.Number",
              "id": "InvoiceUnits",
              "isVisible": "${HasUnitBasedDemands}",
              "label": "Total Number of Units to be Billed"
            },
            {
              "type": "Input.Number",
              "id": "InvoiceAmount",
              "label": "Total Amount of Expenses to be billed"
            },
            {
              "type": "Input.Date",
              "id": "FromDate",
              "label": "Transaction From Date"
            },
            {
              "type": "Input.Date",
              "id": "ToDate",
              "label": "Transaction To Date"
            },
            {
              "type": "Input.Text",
              "id": "Comments",
              "isMultiline": true,
              "isRequired": true,
              "label": "Comments"
            }
          ]
        }
      ]
    }
  ],
  "actions": [
    {
      "type": "Action.Submit",
      "title": "Submit",
      "data": {
        "id": "CreditRequest_Submit"
      }
    }
  ]
}

Additionally, see the Important note about accessibility that has been added to our documentation site: https://adaptivecards.io/explorer/Input.Text.html.
Let me know if you have any questions!


in sync with transformers, removing style_doc utils and using doc-builder instead

What does this PR do?

In sync with transformers, removing style_doc utils and using doc-builder instead

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Snyk checks request a fixed version of d3-color

I'm using s3-scale version 4.0.2, which uses the following dependencies<EMAIL_ADDRESS>›<EMAIL_ADDRESS>›<EMAIL_ADDRESS>this causes the Snyk PR Check in my repo to fail due to d3-color Regular Expression Denial of Service (ReDoS), which is fixed on<EMAIL_ADDRESS>What's the road map for upgrading d3-scale with the required d3-color version?

d3-interpolate requires "d3-color": "1 - 3", you just need to upgrade.


Dev/prechecks

Add all check function scattered through out spim repos and added a check read/write function as well that uses fios

@micahwoodard it looks like some of the "local=external directory" code logic made it into this branch. Let's first merge #20 then merge #18. After that, let's update this branch against main, and those extra changes should disappear. If that works, we should be able to merge this too afterwards.


Create header component for home page

This is supposed to be an open source guide where we help people by showing them active open source projects of open source organizations along with their recent issues.
Please keep the above theme in mind when you design the react function component.

what should include a header component like , logo of bossy helper , organisation button  or anything else ??

We can add the logo later. Think of a catchy phrase to display on top. It should make it clear that it is a guide to open source. Include some nice design or image alongside. Let me know if you want to work on it.

Yeah I'm interested ...
Just thinking what can be the good way.


Unhandled error in list enrollments

Version Info:
NAME             VERSION
ziti             v0.27.9
ziti-controller  not installed
ziti-prox-c      not installed
ziti-router      not installed
ziti-tunnel      not installed
ziti-edge-tunnel not installed

Issue:
Running ziti edge list enrollments on a vanilla installation results in the below error.
error: error listing https://<IP_ADDRESS>:8443/edge/management/v1/ext-jwt-signers in Ziti Edge Controller. Status code: 404 Not Found, Server returned: {
    "error": {
        "cause": {
            "code": "UNHANDLED",
            "message": "path /edge/management/v1/ext-jwt-signers was not found"
        },
        "code": "NOT_FOUND",
        "message": "The resource requested was not found or is no longer available",
        "requestId": "uzbEz5O1J"
    },
    "meta": {
        "apiEnrollmentVersion": "0.0.1",
        "apiVersion": "0.0.1"
    }
}

Expected:
List of enrollments managed by the edge controller.
Additional Information:
GET /enrollments succeeds with exit status 200.

no longer reproducible in 0.32.0


Add RDS Instance Storage Not Encrypted query #1456

RDS Instance Storage should be encrypted or kms_key_id should be defined.
Closes #1456

Scan submitted to Checkmarx

Checkmarx SAST - Scan Summary & Details
Cx-SAST Summary
Total of 4 vulnerabilities 0 High 3 Medium 1 Low 0 Info
Violation Summary
 3 Medium
View more details on Checkmarx UI
Cx-SAST Details

Lines
Severity
Category
File
Link

30
Medium
Path_Traversal
cmd/builder/main.go
Checkmarx

83
Medium
Denial_Of_Service_Resource_Exhaustion
test/main_test.go
Checkmarx

52
Medium
Denial_Of_Service_Resource_Exhaustion
pkg/kics/service.go
Checkmarx


🐛 BUG: Building Worker with D1 and Nuxt 3

Which Cloudflare product(s) does this pertain to?
D1, Wrangler
What version of Wrangler are you using?
2.9.1
What operating system are you using?
macOS 13.2
Describe the Bug
Overview:
Worker cannot be built with D1 bindings and Nuxt 3.

Nuxt 3 built & deployed to Workers - success.
Nuxt 3 with KV bindings built & deployed to Workers - success.
Nuxt 3 with D1 bindings cannot be run in dev or published.

Nuxt builds fine with the cloudflare Nitro preset.
KV bindings work AOK, however, D1 doesn't but I'd have expected similar behaviour.
Error:
✘ [ERROR] No matching export in ".output/server/index.mjs" for import "default"

    ../../../../../private/var/folders/6c/6l0q50813qd2t24z8zztts2r0000gn/T/tmp-2814-oylSIpoFBDZf/d1-beta-facade.entry.js:5:7:
      5 │ import worker, * as OTHER_EXPORTS from "/Users/me/src/templates...
        ╵        ~~~~~~

✘ [ERROR] Build failed with 1 error:

  ../../../../../private/var/folders/6c/6l0q50813qd2t24z8zztts2r0000gn/T/tmp-2814-oylSIpoFBDZf/d1-beta-facade.entry.js:5:7:
  ERROR: No matching export in ".output/server/index.mjs" for import "default"

Steps to Reproduce:

Create a Nuxt project.
Create a Wrangler project.
Integrate the two - clone the repo here already setup.
Run npm install.
Run wrangler kv:namespace create tester.
Run wrangler d1 create tester.
Use your Cloudflare Account ID along with the IDs generated from the wrangler commands to update the wrangler.toml file's bindings of D1 and KV.
Run wrangler dev or wrangler publish and you get the error.
Comment out the D1 bindings line and re-run wrangler dev or wrangler publish and the error no longer exists.

The existing Cloudflare preset for Nitro is based on the service worker format, so D1 won't work.
I've been working on a preset for the module format: https://github.com/unjs/nitro/pull/681

change the NITRO_PRESET from cloudflare to  cloudflare_module
https://nitro.unjs.io/deploy/providers/cloudflare#cloudflare-module-workers


Play 2.7.0, Scala 2.13.0-M5, Scala 2.11.12

:tada:

does it mean that new release of Deadbolt is comming soon? ;)

Should happen soon, yes.


e2e-tests.sh fails if teardown function isn't defined

https://github.com/knative/hack/blob/abd085fc43de80d6344a88788ae6b926c48c908e/e2e-tests.sh#L31-L33
If test_teardown and knative_teardown isn't defined, the script exits with error instead of passing properly.
/kind bug

Affects https://github.com/knative-sandbox/reconciler-test/pull/273

/assign


NPC knock-back can push them through walls

check collision layer bits and layer masks for the NPC.
It should respect the layers for environment objects, like walls and platforms.

I think this is solved. We switched NPC knock back away from tweens and into move_and_slide with iframes.


vendor/ dir

I love godef, but it doesn't seem to notice when a vendor/ directory is being used.

I ran into the same thing when using glide.
I found this PR (which was closed without merging). The code here works with vendor dirs: https://github.com/rogpeppe/godef/pull/25

@aloucks helpful thankyou. I wonder why folks seem to think that #33 fixed the issue, when apparently it does not?

I tested this by reverting to d712540 and it still didn't work, so this hasn't been broken by subsequent commits to this repository.  Perhaps this has been broken by changes in more recent Go versions?  I only tested against 1.8, as I don't currently have a simple way to test against previous versions.
Thoughts @rogpeppe?


md5_client -b mode error?

./md5_client -s 127.1:12345 -f echo_client -f echo_server -b. In parameter -b mode, no matter you specify 1 or more files, the first file always reports an error, lsquic_stream_read: Connection reset by peer, I should How to solve this problem, thank you in advance

It is not a very important example application.
I guess you need to run it in a debugger and figure it out if you have to make it work.


These credentials do not match our records.

I'm trying to get Adldap Auth to work and I get the follow message on my login screen when I try to login with valid creditentials:
These credentials do not match our records.

I do not get any connection errors from Adldap so I think my config is correctly set up.
I also tried doing a $ad->search()->all(); and the result is an empty ArrayCollection.
I'm not quite sure where this issue lies and how to debug this further.

Hi @SuperDicq,
Are you using the default app/Http/Auth/AuthController that comes pre-installed with Laravel?
Also, do you have APP_DEBUG=true in your .env? If you don't you won't see any connection errors from Adldap.

Yes, I'm using the default AuthController that I've modified for usernames.
class AuthController extends Controller
{
    /*
    |--------------------------------------------------------------------------
    | Registration & Login Controller
    |--------------------------------------------------------------------------
    |
    | This controller handles the registration of new users, as well as the
    | authentication of existing users. By default, this controller uses
    | a simple trait to add these behaviors. Why don't you explore it?
    |
    */

    use AuthenticatesAndRegistersUsers, ThrottlesLogins;

    protected $username = 'username';

    /**
     * Create a new authentication controller instance.
     *
     * @return void
     */
    public function __construct()
    {
        $this->middleware('guest', ['except' => 'getLogout']);
    }

    /**
     * Get a validator for an incoming registration request.
     *
     * @param  array  $data
     * @return \Illuminate\Contracts\Validation\Validator
     */
    protected function validator(array $data)
    {
        return Validator::make($data, [
            'name' => 'required|max:255',
        	'username' => 'required|max:255|unique:users',
            'password' => 'required|confirmed|min:6',
        ]);
    }
}

And yes I do have APP_DEBUG set to true.

Ok, are you using OpenLDAP or ActiveDirectory?
Also, can you post your adldap_auth.php configuration?

We're using ActiveDirectory.
Here is my adldap_auth.php file.
<?php

return [
    'username_attribute' => ['username' => 'samaccountname'],
    'login_fallback' => false,
    'password_key' => 'password',
    'login_attribute' => 'samaccountname',
    'select_attributes' => [
        //
    ],

];

Hmmmm everything looks fine... Can you try inserting this at the top of your app/Http/routes.php file and see what the result is?
$username = 'myusername';
$password = 'mypassword';

dd(Adldap::authenticate($username, $password));

The output is true

Ok, can you try inserting this at the top of your app/Http/routes.php file and see what the result instance is?
dd(Adldap::users()->find('myusername'));

Need to see if an instance of Adldap\Models\User is returned.

The response here is false

Ok, looks like this may be the issue.
Can you try:
dd(Adldap::search()->where('cn', '=', 'My Name')->get());

And see what instances are returned (if any)?

It seems like an empty ArrayCollection.
ArrayCollection {#263 ▼
  -elements: []
}

Hmm, do you have a base_dn set in your config/adldap.php file? Or have SSL / TLS enabled?

	'connection_settings' => [

		'account_suffix'        => '@qlic',
		'domain_controllers'    => ['OFFICE.QLIC.NL'],
		'port'                  => 389,
		'base_dn'               => 'dc=QLIC-SRV01',
		'admin_username'        => '****',
		'admin_password'        => '****',
		'follow_referrals'      => false,
		'use_ssl'               => false,
		'use_tls'               => false,
		'use_sso'               => false,
	]

Can you try:
dd(Adldap::getRootDse());

And check the attribute rootdomainnamingcontext? Your base_dn needs to match this value.

Fixed it.
My base_dn was not correct.
I found the right base_dn by typing in dsquery computer in cmd.exe on the server.
Thanks for your help.

Great! Glad you've solved it!
No problem at all. If you encounter any further issues please create an issue. I'd be glad to help.
I'll create a troubleshooting document so I can keep track of these issues.

I have solved this issue. Click https://github.com/vermaboys/change-error-message-of-login-laravel5.4


Any tips to speed up quantized Whisper inference on Android?

Hello, running q80 quantized whisper on Android (Pixel 7) is taking around 15 seconds for 5 seconds of audio. Is there any way to speed this up that I might not be aware of or is it just because candle isn't as optimized as something like whisper.cpp yet? whisper.cpp took around 3 seconds or less if I remember correctly. Although this was with a q40 model. Thanks.

I'm not super familiar with how cross-compiling to android works, and what simd instructions/blas library could be available on these platforms. Most likely the compiled binary is not benefiting from these and this would explain the slowness (on normal builds and wasm, we have some specific build setup and code to hopefully use simd instructions).
I just added a CANDLE_DEQUANTIZE_ALL environment variable that will force using the standard matmul rather than the quantized one, could you try running your tests with this set to 1 just in case?

Wow, enabling that resulted in it taking only around 4.5 seconds!

Interesting, thanks for reporting this back. There are multiple things at play here and I'll have to dig a bit deeper to understand what is going on.

It could be that the quantized matmul doesn't detect the simd instructions but the normal matmul does (unlikely).
Using Q8_0 is slower than Q4_0, it's supposed to be optimized but maybe we've missed something.
The quantized matmul isn't as smart as the unquantized one when in comes to cache locality. In GPT like architectures, it's usually not an issue but the whisper encoding step might not have the same properties.

Btw I was a bit off on the amount of time, I wasn't actually testing with 5 second audio but more like 2 seconds with 3 seconds of no sound. But I just tried and its still about the same difference, just saying in case you try and get slower than 4.5 seconds

Q4_0 is even slower (without CANDLE_DEQUANTIZE_ALL set to 1) so it can't be that. Is there anything more I can do to try and figure out the cause of the problem? I could provide the source code of my app if needed because I'm going to open source it anyways.

@soupslurpr can you share how you have made it running?
When I build cargo it always fails with such as errors:
error: instruction requires: fullfp16
error: could not compile gemm-f16 (lib) due to 11 previous errors

@rbrus are you sure you are using the Android NDK to compile?
For example, in .cargo/config.toml I specified:
[target.aarch64-linux-android]
ar = "C:/Users/user/AppData/Local/Android/Sdk/ndk/26.1.10909125/toolchains/llvm/prebuilt/windows-x86_64/bin/llvm-ar.exe"
linker = "C:/Users/user/AppData/Local/Android/Sdk/ndk/26.1.10909125/toolchains/llvm/prebuilt/windows-x86_64/bin/aarch64-linux-android21-clang.cmd"

Thanks @soupslurpr . This seems to be working ,it just fails to build it. I am wondering if there is some issue with fp16 build for Android?

@rbrus what version of candle?

Also, are you building for the aarch64-linux-android target?

And note that the config I provided probably needs to be changed for your Windows username

Yes, and it always fails with the same error:
`error: instruction requires: fullfp16
--> /home/sus/.cargo/registry/src/index.crates.io-6f17d22bba15001f/gemm-common-0.17.0/src/simd.rs:1940:18
|
1940 |                 "fmul {0:v}.8h, {1:v}.8h, {2:v}.8h",
|                  ^
|
note: instantiated into assembly here
--> :1:2
|
1    |     fmul v0.8h, v1.8h, v2.8h
|     ^
error: could not compile gemm-f16 (lib) due to 11 previous errors
warning: build failed, waiting for other jobs to finish...`
I run it on Ubuntu 22.04 upgraded recently to 23.10. The version of rust and cargo and candle are most recent. I have setup it all today.

So you changed the config.toml to go to the NDK you have downloaded?

Yes, exactly, to NDK 25.2.9519653.

No, maybe try NDK 26.1.10909125?

Also perhaps try adding this to the [target.aarch64-linux-android]
rustflags = [
"-C", "target-feature=+fp16,+neon",
]
I think I needed this before and had the same error as you, but it isn't needed for me anymore.

@soupslurpr  after changing the NDK it built! Huh, thanks for help!
By any chance, do you have a project which inference with the rlib?

@rbrus great!
I do have a project I'm working on for running whisper speech to text on Android using Candle, but I'm not working on it currently as the speed is still too slow.

I found one of the issues. Quantized whisper is still slower, but setting the threads to 2 increases speed for both immensely.
This works because by default Rayon uses a thread for each CPU I think? The Pixel 7, which is the device I'm running on has 8 CPUs. 2 of them are high performance, 2 are still high but more for efficiency, and the other 4 are efficiency.
Whisper.cpp gets the number of threads by getting the number of the lowest frequency cores, and subtracting it by the total core count. This results in it using 4 threads. Android doesn't like this for the non-foreground app, and slows down everything.
By setting the number of threads to the number of the highest frequency cores, it is more efficient and Android doesn't limit it.

Sorry I was wrong, I should've done more testing

@soupslurpr how did you build candle with target as android I am getting openssl error. I am on mac and have setup the environment variable like below:
export AR="/Users/akashsingh/.NDK/arm64/bin/llvm-ar" export CC="/Users/akashsingh/.NDK/arm64/bin/aarch64-linux-android-clang"

Can you provide me steps to compile for android I am trying to run phi-2 on a old redmi note 7 pro android device
? We can put it in this discussion if anyone else would be interested? https://github.com/huggingface/candle/discussions/2081

Just tested and this issue actually happens with x86_64 too. Tested on Windows and the quantized Whisper is way slower than the unquantized / the one using the CANDLE_DEQUANTIZE_ALL set to 1 so it isn't Android specific.


Fix first assignment to _min/max_value in Curve bypassing min < max check

Currently, the very first time one assigns to either _min_value or _max_value in Curve, it bypasses the min < max condition:

This PR implements "bulk load" for curve limits by adding a "fake" property called _limits, which loads an array containing all the values. These are assigned to the proper variables without checking for min < value, as it should be during deserialization. The min < max condition is still ensured by the set methods for the min/max values, so that we always serialize min < max values.
This solution was taken from https://github.com/godotengine/godot/pull/29959#issuecomment-504783824, and extracted from #67857 as a first step towards reviving it. It also sets the stage for other limits to be added (such as the domain ones) without having to replicate the current, buggy solution.
Long-winded explanation of the problem to serve as reference in the future:
The reason for the current behavior is that the min < max check is disabled the first time each property is written to. The idea is that this happens during deserialization, so that the checks don't conflict with the default values of 0 and 1. Suppose serialized values are min=5, max=10. When deserializing min first, and trying to set it, the condition that min < max(default) is 5 < 1, which fails.
However, if the Curve has default min/max values, those properties aren't serialized in the first place, and, therefore no first time assignment happens deserialization. This makes it so the first time the user attempts to assign to these limits, the checks are not guaranteed.
Downsides of approach
GDScript recognizes the properties tagged as PROPERTY_USAGE_INTERNAL & related methods, which perhaps it shouldn't. In this case, it is unintended since these properties are "fake" and mean to be used just for de/serialization. On the other hand, there might be some GDScript code out there that makes use of internal properties. I can imagine some addons, through GDExtension or GDScript, would do it. See https://github.com/godotengine/godot/issues/25651 for discussion on whether PROPERTY_USAGE_INTERNAL properties should be exposed to GDScript.
Warning: This contains a minor compatibility break in that _min/max_value are no longer serialized. However, they are still deserialized because the resource loaders don't check that the property they are reading must have the PROPERTY_USAGE_STORAGE flag.

Can't we just do it like...

Set min to -INF
Set max to the proper value
Set min to proper value

Tangentially related: #72250

@MewPurPur your logic is correct, but the actual issue isn't so much how to avoid the check, but instead knowing when to avoid it! The current implementation uses extra variables to disable checks the first time a variable is assigned to, which is often (but not always, as in this bug) during deserialization.
This PR's solution is exclusively used during de/serialization, so it addresses the when to use the checks perfectly. If we do move towards having curves with arbitrary domains (the other linked PR), it also scales really well to adding _min/max_domain with _min_domain < _max_domain to this :)

Tangentially related: #72250

Actually looks to be the exact same issue. I'll see if I can't replicate this solution there as well. Probably worth standardizing one way or another.
The downside of this approach is that somehow GDScript recognizes the properties & related methods, which perhaps it shouldn't, since they are meant to be PROPERTY_USAGE_NO_EDITOR | PROPERTY_USAGE_INTERNAL. But there might be some GDScript code out there that makes use of them, so we should figure out a solution.

I think this should be solved, somehow, with a new property hint. That hint would take care of not setting values in a wrong order or not setting them one by one. It's like having linked properties in a sense.
The current implementation technically breaks compatibility for serialized data.

Thanks for looking at this! :)
It does break compatibility (it's forward-compatible but not backwards-compatible), but not in a way that affects the curve data. These limits are used exclusively for drawing the curve in the inspector, so it's not necessarily a huge deal. Users could manually change domain values to be able to see the whole curve. We could also update this PR to enforce the domain (when not found) to include min and max points in the curve.
I do think it'd be lovely if the core had a way to handle mutually-dependent properties though!
I'm not immediately seeing a solution with property hints though. A load order hint wouldn't work (e.g. always load min before max): in this case, the order in which the max or the min needs to be loaded depends on whether it's above or below the default values.
Is there a way to bypass setters and immediately assign to a property? That could be the way to go, perhaps.

I'm not immediately seeing a solution with property hints though. A load order hint wouldn't work (e.g. always load min before max): in this case, the order in which the max or the min needs to be loaded depends on whether it's above or below the default values.

The hint could possibly act as a delay for setting values until after all of the values are parsed, then calling a special setter to set them together. As an example, at least.

Bypassing setters might be hard, because ClassDB doesn't support reflection on values directly, as far as I know.
What might work instead is if Resource had a new pair of methods:
private:
    virtual void _before_load();
    virtual void _after_load();

These are called by ResourceFormatText and ResourceFormatBinary before and after all the setters are called, respectively. These classes would need to be declared friends of Resource, or the methods would need to be public instead of private.
Resources with complex interdependencies between their properties can then do whatever they like in these callbacks, e.g. set a flag to temporarily suppress validity checking in setters.
This particular case doesn't even need a flag though. In _before_load(), min and max could be set to -INF and INF, and in _after_load(), if either of them is still infinity, it's set to its default value, 0 or 1.

My initial thought was to create a new macro called ADD_PROPERTY_OVERRIDE_LOAD() or something like that which further specified two setters to be used only when loading a resource. I haven't quite been able to figure out how many changes in core this would entail, but feels more in line with how other types of properties are added?
I feel like I would prefer this to a more stateful solution like the one you suggested (which can introduce threading issues, though I don't think that would happen in this particular case), even though what you suggest is perhaps less intrusive?

What kind of threading issues are you thinking of? No other references exist to the resource yet, since it's newly being loaded.
Adding new "properties of properties" is more heavyweight, because every property in ClassDB will need to have fields for it, even if they are rarely used. I'm not sure it's an actual problem, but the benefit needs to be higher to outweigh the cost. PropertyInfo has a comment stating something like "talk to Juan before adding more fields here" :)
Also, wouldn't you want to validate the properties after they've all been loaded? You'd need some kind of "loading done" callback for that anyway.
Could someone from the core dev team maybe comment? Maybe there are other cases in the engine that could make use of this API and help inform its design?

Maybe we can use ADD_LINKED_PROPERTY to indicate their relationship and their setting priority.

Some updates on this! I love documenting stuff so bear with me or skip to TL;DR :)
I looked into the approach suggested by @ttencate with pre-load and post-load callbacks.
A post-load hook _finish_loading() can be called from ResourceLoader::_load(), which is agnostic to the specific loader being used. Now the problem is how a Resource knows it's in loading mode. There are two options:

Resource, or Curve more specifically, could have a member _is_loading that is initialized to true. But then every time a Resource/Curve is created manually, the code must also manually call _finish_loading() . This feels like a sure way to get bugs ❌
Use a pre-load function _start_loading() as suggested. However, there's no loader-agnostic way to call this function, since the loaders themselves instantiate the resources from ClassDB, and that's when you need to call the hook. Thus, if we want this at the Resource level, then every single loader would need to call the preload hook at the right time, or we would get inconsistent behavior across different resources, some supporting the preload/postload hooks and some not ❌
Perhaps it is possible to create a loader that's specific to resources that need these functions called without too much code duplication? ❓

TL;DR all other suggested approaches have pitfalls that would require remembering to call specific functions at specific places in the codebase, which is a sure way to create bugs or inconsistent behavior. I'm currently looking into creating a loader specific to Resources that need these callbacks.

Been thinking about this too. There are only two resource loaders in the engine, right? And I don't think writing a custom loader in a midule or GDExtension is all that common. So option 2 is not so bad imo. Not perfect, but very simple and non-invasive.
We could wrap the creation of the resource in a virtual function on the abstract ResourceLoader base class, which would do the ClassDB thing and call the pre-load hook. That makes it clearer that it should be done for any type of resource loader.

So unfortunately there are 18 subclasses of ResourceFormatLoader that I can find, including ResourceFormatLoaderText and ResourceFormatLoaderBinary, but also some for JSON, crypto stuff, images, translations, scripts, textures, etc. There are many ways in which they work: some of them load directly in the load() function, some of them offload the loading work to another class (e.g. ResourceFormatLoaderText offloads to ResourceLoaderText), some use ClassDB::instantiate(), some directly memnew(), or Ref<>().instantiate(), etc.
So I don't believe we want the load hooks to exist at the Resource level, because that means we'd need to change every loader in non-trivial ways to accommodate functionality only one or two resources need.
I've been digging into ResourceLoaderText::load() it uses ClassDB to instantiate both subresources as well as the main resource itself. I haven't looked very deeply into ResourceLoaderBinary::load(), but I only see a ClassDB::instantiate() once there.
I'm going to try to make a subclass of Resource called LoadHookResource that can be used by resources that need this functionality. This should allow maintainers to add support for these hooks as needed in different loaders, and we could start with just the two you suggested. I'm going to try to update #88224 with this approach.

I hope that only ResourceLoaderText and ResourceLoaderBinary are relevant. Those are the ones that work by dynamically calling setters, for which it isn't known ahead of time which setters will be called. Others, such as ResourceFormatLoaderGDScript or ResourceFormatLoaderCompressedTexture2D, have more knowledge of the type of resource they're loading, and can collaborate with that type accordingly.

Rebased.
I do think this is the only sensible solution, and it's on the path towards getting #67857 revived, which I'm interested in doing.
There's a deeper discussion of this in #88224, but all other alternatives seem to require nontrivial changes to ClassDB, properties or all loaders. This general approach should also work for any Object that needs to be deserialized, including Range which appears to have the same problem despite being a Node and not a Resource.
⚠ this is forward-compatible but not backwards-compatible: min_value and max_value will be lost when going back to previous version ⚠  However, these are just visual aids and do not affect behavior of curves, so it's very minor ⚠

Closed as this is included in #67857.


Android API 21 throws UnsatisfiedLinkError

Getting the following exception on Android API 21 when trying to establish a connection via sqlite-jdbc-crypt.
Caused by: java.lang.UnsatisfiedLinkError: dlopen failed: cannot locate symbol "__register_atfork" referenced by "/data/app/io.toxicity.core.data.cipher.driver.test.test-2/lib/x86_64/libsqlitejdbc.so"...
at java.lang.Runtime.loadLibrary(Runtime.java:371)
at java.lang.System.loadLibrary(System.java:989)
at org.sqlite.core.NativeDB.<clinit>(NativeDB.java:44)
at org.sqlite.SQLiteConnection.open(SQLiteConnection.java:283)
at org.sqlite.SQLiteConnection.<init>(SQLiteConnection.java:70)
at org.sqlite.jdbc3.JDBC3Connection.<init>(JDBC3Connection.java:28)
at org.sqlite.jdbc4.JDBC4Connection.<init>(JDBC4Connection.java:19)
at org.sqlite.JDBC.createConnection(JDBC.java:104)
at org.sqlite.JDBC.connect(JDBC.java:77)
at java.sql.DriverManager.getConnection(DriverManager.java:179)

Unsure if this has anything to do with this project or somewhere else downstream (xerial/sqlite-jdbc or Willena/sqlite-jdbc-crypt), but wanted to ping you here to start the process.
Does __register_atfork ring any bells? Thanks!

cc: @Willena

Actually. This issue exists with xerial/sqlite-jdbc.
Closing!


Consolidate project data to one yaml file

This PR consolidates all the project data to one yaml files as done in https://github.com/typelevel/typelevel.github.com/pull/274 but without the categorization, which could perhaps come later in a separate PR.
It also makes sure cats comes first in the list of cats-... named projects.

I've kept changes to a minimum here, but I feel like this would make it easier to maintain reasonable category values across all the projects. Or ensure that all projects have particular metadata.

Just a quick note to self, if we go with this approach we'll want to update the migration template: https://github.com/typelevel/governance/blame/main/.github/ISSUE_TEMPLATE/2-project-migration.md#L22

I think the single file is fine just ease of ordering etc.  We can just keep an eye on if this becomes a pain.


Luvenda Translation of the dashboard

Hello,
I'm interested to contribute to this project, I have not as yet found a way in. However, I would like to offer translations into Tshivenda / Luvenda language, similar to what was done with Xitsonga.

I have asked the creator of the Xitsonga version to comment.

@rudzanimulaudzi have you been able to start?

Please accept my apologies for the reply @rudzanimulaudzi  and @vukosim. It gapped my mind som how. It is important we translate this important data to all languages in SA.
The approach we took for Xitsonga was 1) get a go ahead from Dr. Vukosi. 2) find translations for all labels on https://dsfsi.github.io/covid19za-dash/. 3) Created graphs for free using Google data studio.
On going, we manually update the datasets (CSV) as the main repo is updated. There might be a better way of doing this but we were pressed for time. I am happy to assist if you need help with translations or Google

@rudzanimulaudzi Can you respond.

Hi Dr. Vukosi, for some reason I missed the response from @Sneidon. I am keen, sounds like it's just manual updates of labels then recreating graphs with data studio.
I can on Sunday and probably complete it the weekend that follows or on the 16th of June.
@vukosim are you able to send me a "copy" of the data studio site (does google allow that?) that I can update? trying to take the next steps.

@rudzanimulaudzi let me help with the data studio part. Just send a meeting request for any day, it shouldn't take 30 min to setup.

Great, thanks @Sneidon . I got your email address.

Thanks Dr. Vuko & Sneidon.
Will "launch" the dashboard tomorrow: https://rudzanimulaudzi.github.io/covid19/

Thanks @rudzanimulaudzi and @Sneidon
Closing this as we now have http://luvenda.com/covid/

@rudzanimulaudzi congrats and thanks @vukosim.


Misleading warning for ?? throw.

The code x ?? throw 0 gives the following errors:
ERROR|SYNTACTIC_ERROR|...file.dart|3|8|Expected to find ';'
ERROR|SYNTACTIC_ERROR|...file.dart|3|8|Expected an identifier

This is misleading since the real solution to the errors is to add parentheses around the throw 0 expression.
Listing what you expect to find is great, but it should be complete then, and '(' is also possible here.

Will become a duplicate of https://github.com/dart-lang/sdk/issues/24891 when the analyzer integrates with the CFE.

And so it is done. Duplicate of #24891.


Move giving.cu.edu/essentialcu HTML/CSS/JS and remove redirect.

this is related to https://github.com/CUCentralAdvancement/essential-cu/issues/85 and https://github.com/CUCentralAdvancement/essential-cu/issues/83
That HTML was created by E3 as a stand alone page. There were some Views created to dynamically link the Drupal nodes, but that was never used.
With https://github.com/CUCentralAdvancement/essential-cu/issues/76, I wasn't able to find a way to have the index.html resolve at the directory root. Going to continue looking at that.

@alexfinnarn @mattroush I think this is good enough since it's the same HTML/CSS/JS and no worse than what's on giving.cu.edu now. The header/footer options can obviously be improved, but I can't see any issues in how next.js is routing this or how Fastly is caching it.
Even without Fastly, https://developers.google.com/speed/pagespeed/insights/?url=https%3A%2F%2Fgiving.cu.edu%2Fessentialcu%2F scores the Pantheon site a 22 and the review app without Fastly https://developers.google.com/speed/pagespeed/insights/?url=https%3A%2F%2Fessential-cu-feature-10-guqo2x.herokuapp.com%2Findex.html a 34.

@kreynen I think it looks great, and definitely feels faster.

@alexfinnarn any issue with merging this in now and adding the wildcard redirects on the giving.cu.edu side? It won't generate a lot of traffic, but would be more than 0 traffic through the prod instance and Fastly config.

Let's have @tskillern review it on review app before we merge. Tim -- this is to move essential cu microsite from giving.cu.edu to this new essential.cu.edu site. You can see it and give it whirl here: https://essential-cu-feature-10-guqo2x.herokuapp.com/index.html
@kreynen @alexfinnarn

Seems like a lot of files for one page, but...if it works, who cares at this point? I wouldn't think the scss files are needed if the CSS is already compiled, but whatevs.

@mattroush looks good to me. Thanks.
cc @kreynen @alexfinnarn

Deployed without issue to staging, but first attempt at deploying to production failed with...
       Installing Cypress (version: 5.0.0)

       [13:48:25]  Downloading Cypress     [started]
       [13:48:26]  Downloading Cypress      0% 0s [title changed]
       [13:48:26]  Downloading Cypress      1% 10s [title changed]
       [13:48:26]  Downloading Cypress      3% 7s [title changed]
       [13:48:26]  Downloading Cypress      5% 6s [title changed]
       [13:48:26]  Downloading Cypress      8% 5s [title changed]
       [13:48:26]  Downloading Cypress      11% 4s [title changed]
       [13:48:26]  Downloading Cypress      14% 4s [title changed]
       [13:48:26]  Downloading Cypress      14% 4s [failed]
       [13:48:26] → The Cypress App could not be downloaded.

       Does your workplace require a proxy to be used to access the Internet? If so, you must configure the HTTP_PROXY environment variable before downloading Cypress. Read more: https://on.cypress.io/proxy-configuration

       Otherwise, please check network connectivity and try again:

       ----------

       URL: https://download.cypress.io/desktop/5.0.0?platform=linux&arch=x64
       Error: read ECONNRESET

       ----------

       Platform: linux (Ubuntu - 18.04)
       Cypress Version: 5.0.0
       The Cypress App could not be downloaded.

Made no changes. Deployed again. Worked.


embed without an _id

Versions 2.1.8 removed the embedded_object_id configuration parameter, issue #740, however that was a global configuration. Is there any documentation on how to embed a document which does not have an _id on a per model basis?

Mongoid requires the ids to be on the embedded documents in order for it's relationships to function properly, as well as atomic updates since the $ positional operator in MongoDB is not fully-functional. Therefor we do not provide a way to do this out of the box and cannot make any guarantees of everything working properly if the ids are removed.


Special handling for NaN

Not sure on details but it will likely have to be specially handled, as it should be rejected in modern proposals: https://github.com/tc39/how-we-work/blob/main/normative-conventions.md#number-taking-inputs-should-reject-nan

We could replace this: https://github.com/Richienb/proposal-math-clamp/blob/fbf4b5a1173996cfc896ebdcd6c6a3e747d79e29/spec.html#L18C6-L18C54
with If _max_ is neither *undefined*, *null* nor *NaN*, then
and do something similar for min

if the value is NaN, either throw, or return NaN (i prefer "return NaN", and i think that convention doesn't necessarily constrain us here)
if the lower bound is NaN, throw
if the upper bound is NaN, throw

Fixed in 513523edf172b71060b96249e90be8c7db997e9d

Reopening to allow for discussion

if the value is NaN, either throw, or return NaN (i prefer "return NaN", and i think that convention doesn't necessarily constrain us here)

I would prefer for this to throw - but that's mostly because I wish that all operations that currently return NaN would throw instead - NaN is, after all, nothing more than a way to represent an error in a calculation, and I would rather that errors throw earlier instead of causing weird bugs later.
In practice, from my understanding, it's better for operators such as division to return NaN instead of throwing an error simply because it's more performant that way (at least, that's what I've heard). But I don't know if the same argument applies to a clamping function - especially if we're considering it to throw when other arguments are NaN..

I think that there's definitely a potential for Math.clamp to be used in a hot path along with other math operations, which would mean that the code already has to guard against NaN, so there's no added hazard here from it.


nsc_events_android-01-creating_an_event

Attempting to integrate frontend and backend by creating an event in the add event page. Still a work in progress so this is just a rough draft pr

@Robel-003 Did we still want to keep this open or close this out?

@Robel-003 Did we still want to keep this open or close this out?

@Seiyaroo we can close this out. Doesn't need to be merged at the moment, still a work in progress

https://youtu.be/HsUxkJ3yBTQ link check video

@Robel-003 what is the status of this? Did we want to review and try to merge this?

Nice work! I was able to add an event but had to follow your instructions and change the following line from @isnotempty() to @IsOptional():

After entering the correct values into the other 4 fields you mentioned, I was able to add an event and view it in my MongoDB database:

I think we should create a user story with 2 issues. 1 issue for this PR and another issue for the nsc-events-nestjs repo so we can change that 1 line above.

@taylorpapke I've created the user story and issues for this pr and assigned it to this pr. If it all looks good, I can go head and merge this to main

@SeattleColleges/leadership Can we get another review on this when someone gets a chance. Would like to get this merged in before we resume work on android next sprint.


Prevent migration failures when re-deploying to test

Description
As a developer deploying to test, I can only run a DB migration once. This breaks test if I need to modify the migration before merging to main, because changing the migration will change the checksum. It would be great to have a way to not fail deploys to test due to migrations.
Proposed solutions

Wipe the DB before each deploy
Automatically roll back the migrations (is this possible?)

As of 6/29, this has occurred 3-4 times. If it happens again/ in the future, we can revisit taking action.

this is 80% solved with https://github.com/CDCgov/prime-simplereport/issues/2177
It would be nice to have an automated solution. However with the rollback it should be a rare and minor inconvenience


Investigate reducing the cost of DynamoDB in the matcher

I think I've got all the easy bits here.

I think I've got all the easy bits here.

We've decided the way to do this is to switch to provisioned throughput for the lock table, which will get scaled up when we're reindexing and scaled down when not. This will get rolled into a larger piece of scaling work the next time we do a reindex.

We've decided the way to do this is to switch to provisioned throughput for the lock table, which will get scaled up when we're reindexing and scaled down when not. This will get rolled into a larger piece of scaling work the next time we do a reindex.

Currently this costs ~$50–$60 per reindex. The time taken to tune the provisioned throughput would likely wipe out several reindexes worth of savings, so I'm going to close this as done for now.


Optimize our CI pipeline.

A few things, in no particular order:

[ ] 1. we should investigate Azure Pipelines: Windows, macOS, Linux, free for OSS
[ ] 2. we should kill all 32-bit builds because it's 2019 and they just waste time
[ ] 3. we should check formatting and not just regular tests
[ ] 4. might be worth switching to cargo-make to  ⭐ synergize ⭐with the rest of the NF crates

Did nr. 2 in #94.


[fix][txn] fix ack with txn compute ackedCount error

issue:
reproduce:

redeliver one consumer received message of a batch message, will remove this message from this consumer pending acks
if this message is removed and doesn't redeliver to another consumer or this consumer, we ack another message in the same batch
to compute the consumer unackedCount will get an incorrect batch size, because the message don't in any consumer pending acks

Motivation
get correct batch size to compute the consumer unackedCount
Modification
use the correct batch size from the messageIdDate
Verifying this change
add the test
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

Does this pull request introduce a new feature? (yes)

If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

If a feature is not applicable for documentation, explain why?

If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

[x] doc-not-needed

@Technoboy- transaction ack command will carry the batchSize, if the batchSize is not in pendingAcks(the message redeliver hasn't been sent to any consumer), we will not get the correct batch size. So I think in Transaction ack, we uniformly use command carried batch size is ok

@Technoboy- Ping
@congbobo184 Please resolve the conflicts.

@Technoboy- Ping
@congbobo184 Please resolve the conflicts.


Razorpay enhancement.

Added feature for enabling donations through Razorpay
Added Glossary button on the front end in the consultation summary page

We have addressed all review comments


initialIndex without animation

When i use initialIndex in during init lory, i have animation to slide.
How can i see slide without animation

You can turn off the initial slide animation by disabling all transitions on the lory element via CSS first and enabling them again after lory was initialized.
You could for example add a no-animations class and remove it after lory was initialized.
CSS:
.no-animations * {     transition: none; }
And then add in your JS (before initializing lory!):
document.addEventListener('after.lory.init', function() { yourLoryElement.classList.remove('no-animations'); });


Try posting to matrix again

@wm75 Sorry about that!
I will fix it soon!

Hmm ...

@arash77 can you revert the auto-generation of the processed_files for now and add it back in later when it's properly done?


angular to react configuration migration guide

What this PR does / why we need it:
Adds how-to guide for angular plugins migrating to react
Fixes grafana/grafana#79200

Hi, great job. I have completed a first pass editorial review. Please review the latest commit with proposed changes and ensure that no inaccuracies have been introduced. I will begin a second pass later this week or when the PR gets out of "Draft" state.

@josmperez ready for a second pass!

not sure why this is blocked with a request pending, if we're good to go Joe i'll force the merge.

Okay, sounds good
On Tue, Feb 6, 2024 at 8:37 PM Brian Gann @.***> wrote:

not sure why this is blocked with a request pending, if we're good to go
Joe i'll force the merge.
—
Reply to this email directly, view it on GitHub
https://github.com/grafana/plugin-tools/pull/664#issuecomment-1931267372,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK5BGRCRGT7G4Q5K7TV6WXLYSMAHJAVCNFSM6AAAAABBYWIE2CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZRGI3DOMZXGI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Accordion UI Bug

Blue border highlighted when open, but when back to being closed, bottom blue border is hidden while the left, right and top borders remain highlighted in blue.
Shouldn't they all be deselected when the accordion closes?

Expected Behavior

Your Environment

I tested this on the Base UI website, accordion section.

Tech
Version

Base UI
v?.?.?

React

browser

[x] I have searched the issues of this repository and believe that this is not a duplicate.

This is the element's 'focus' indicator. Pressing 'tab' should focus the next accordion panel and display the border. The hidden bottom-border is not ideal, but we should maintain the borders in general


Can't use older Rhai versions anymore

Related to #828 (the proposed solution to which I wanted to benchmark), there's another issue with the breaking change (which is automatically used unless pinned due to semantic versioning of minor versions):
Even if I pin the rhai crate version, Rust will automatically pick the latest rhai_codegen version which is not compatible with older versions:
rhai v1.16.3
├── ahash v0.8.7
│   ├── cfg-if v1.0.0
│   ├── const-random v0.1.17
│   │   └── const-random-macro v0.1.16 (proc-macro)
│   │       ├── getrandom v0.2.12
│   │       │   ├── cfg-if v1.0.0
│   │       │   └── libc v0.2.153
│   │       ├── once_cell v1.19.0
│   │       └── tiny-keccak v2.0.2
│   │           └── crunchy v0.2.2
│   ├── getrandom v0.2.12
│   │   ├── cfg-if v1.0.0
│   │   └── libc v0.2.153
│   ├── once_cell v1.19.0
│   └── zerocopy v0.7.32
│   [build-dependencies]
│   └── version_check v0.9.4
├── bitflags v2.4.2
├── num-traits v0.2.17
│   [build-dependencies]
│   └── autocfg v1.1.0
├── once_cell v1.19.0
├── rhai_codegen v1.17.0 (proc-macro)
│   ├── proc-macro2 v1.0.78
│   │   └── unicode-ident v1.0.12
│   ├── quote v1.0.35
│   │   └── proc-macro2 v1.0.78 (*)
│   └── syn v2.0.48
│       ├── proc-macro2 v1.0.78 (*)
│       ├── quote v1.0.35 (*)
│       └── unicode-ident v1.0.12
├── serde v1.0.196
│   └── serde_derive v1.0.196 (proc-macro)
│       ├── proc-macro2 v1.0.78 (*)
│       ├── quote v1.0.35 (*)
│       └── syn v2.0.48 (*)
├── smallvec v1.13.1
│   └── serde v1.0.196 (*)
└── smartstring v1.0.1
    ├── serde v1.0.196 (*)
    └── static_assertions v1.1.0
    [build-dependencies]
    ├── autocfg v1.1.0
    └── version_check v0.9.4

Pinning an older version of rhai_codegen doesn't work either:
# cargo tree -v -p rhai
    Updating crates.io index
error: failed to select a version for the requirement `rhai_codegen = "~1.16.0"`
candidate versions found which didn't match: 1.17.0, 1.6.0, 1.5.0, ...
location searched: crates.io index
required by package `foo v1.0.0 (/Users/egger/Desktop/Source/foo)`
perhaps a crate was updated and forgotten to be re-vendored?

Okay, pinning is possible, if one knows that up to rhai 1.16.0, rhai_codegen 1.6.0 needs to be pinned.

Version 1.16 uses rhai_codegen version 1.6.0

Just pin rhai_codegen also. Those two crates go in pairs.

Yeah, one needs to know that rhai_codegen 1.6.0 needs to be used, which is not obvious or great. Ideally rhai would depend on a fixed version of rhai_codegen and not run into the same semver foul as rhai itself.

Yeah, one needs to know that rhai_codegen 1.6.0 needs to be used, which is not obvious or great. Ideally rhai would depend on a fixed version of rhai_codegen and not run into the same semver foul as rhai itself.

True...  but that would remove the capability of putting out compatible patch versions of rhai_codegen.

Hi, we use cargo-generate in our dependency tree for the dioxus-cli crate and rhai being a dependency means our CLI no longer compiles.
https://github.com/DioxusLabs/dioxus/issues/1884
I hate to be that guy... but maybe put up a 1.17.1 and yank 1.17.0 with the compatibility fixes?

Yeah, one needs to know that rhai_codegen 1.6.0 needs to be used, which is not obvious or great. Ideally rhai would depend on a fixed version of rhai_codegen and not run into the same semver foul as rhai itself.

True... but that would remove the capability of putting out compatible patch versions of rhai_codegen.

I'm reading https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#tilde-requirements differently.
If you specify a dependy version of "~1.17.0" for rhai_codegen, then you could still release a rhai_codegen 1.17.1, 1.17.2, ... and it would be used.

I'm reading https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#tilde-requirements differently.
If you specify a dependy version of "~1.17.0" for rhai_codegen, then you could still release a rhai_codegen 1.17.1, 1.17.2, ... and it would be used.

True, but that prevents me from putting out a new backwards-compatible version with new features (that are just not used in the older version).  Those versions will necessarily bump the minor number.
Say I have 1.5.0.  I put out 1.6.0 with new features (that older Rhai versions do not use) and improvements.  Technically speaking older Rhai versions are free to use this 1.6.0 as it is backwards compatible.
But of course you can say backwards compatible versions should always be on the patch number...

Observationally (and sorry if this isn't helpful to add to the conversation), from a semver point of view:

if rhai depends on rhai_codegen
and the latest version of rhai_codegen is no longer compatible with older versions of rhai
then this indicates that rhai_codegen is no longer BC and should get a major version bump
and the newer version of rhai should point to that

Aside, I wish instead of Major.Minor.Patch they'd used Breaking.Feature.Fix, partly because its easier to understand but mostly cos then its BFF 😊 ❤

True...  semver is a simple concept but quite difficult to stick to completely...
Usually there are times when you forget (or make a breaking change without knowing).
In this case, it is unfortunate that I cannot specify a minimum rhai version to work with that rhai_codegen -- that would create a dependency cycle.  Otherwise I can simply lock that version to a minimum rhai version and be done with it.

But, come to think of it more clearly, you probably would be right.  rhai_codegen should be bumped to the next major version if it is non-backwards compatible.  That way, things will "just work" for people, but that also means that it is difficult to modify a certain part of it in non-compatible ways and still stay within a minor release.

Since rhai_codegen is an internal package really, I wouldn't at all be concerned with minor complications in the choice of a different versions. The important bit here is not to cause surprises for people using rhai. I would suspect is rather unusual for people to even declare a specific rhai_codegen dependency, but it is very typical to select a rhai version and somewhat common for that version to be pinned at least via Cargo.lock, so that is that case that should work by default.

True.  I'll have to give it some thoughts to come to the best arrangement.

I just want to chime in that breaking semver breaks all downstream consumers of rhai.
https://github.com/cargo-generate/cargo-generate/issues/1118
https://github.com/DioxusLabs/dioxus/issues/1884
Nether cargo-generate nor dioxus should have to yank released versions to set seemingly unrelated associated dependencies. Being on 1.x.x of rhai means that cargo is free to pick whatever version it wants provided that the version it's picking is newer than what's specified in cargo.
Since rhai is on 1.x.x all new features must be additive. If you want to make breaking changes while on a 1.x.x those breaking changes need to be made on a 2.x.x. That's just the tough reality of being on 1.x.x and having consumers of the published crate.

I plan to yank rhai_codegen 1.17.0 and put out rhai_codegen 2.0.0.
Rhai 1.17.0 should be OK as it'd find 1.6.0 which should work just OK -- the old API are deprecated not removed.
There may be warnings about using deprecated API's though...  I'll check about those.

rhai_codegen is bumped to 2.0.0 while rhai is updated to 1.17.1.
Thanks everybody for flushing out this issue.


orator.exceptions.query.QueryException: connection already closed

Hi
I am using orator db.connection to make an insert query, it works fine but after a while, it starts throwing "connection already closed" exception.
I dig out some code, and it looks like there are some string matching to find out the possible exception so it can reconnect in the following code:
https://github.com/sdispater/orator/blob/23daa78892ff5d1db8111bba098452248abc7ee4/orator/connections/connection.py#L346
But it looks like, the Postgres is having a different exception string "connection already closed", that does not exist there? Is that right or I am missing something else?
Here is the exception trace:
  File "/usr/local/lib/python3.5/dist-packages/orator/connections/co
nnection.py", line 236, in insert
    return self.statement(query, bindings)
  File "/usr/local/lib/python3.5/dist-packages/orator/connections/connection.py", line 34, in _run
    e, query, bindings, wrapped
  File "/usr/local/lib/python3.5/dist-packages/orator/connections/connection.py", line 341, in _try_again_if_caused_by_lost_connection
    raise QueryException(query, bindings, e)
orator.exceptions.query.QueryException: connection already closed

Thanks

any update on this issue??

I simply override this function and additional message for the Postgres connection close.


Open Folder

Type: Bug
When you open a new Window of visual code, the button 'Open Folder' isn't working. Even the close button in top left side.
VS Code version: Code 1.76.0 (92da9481c0904c6adfe372c12da3b7748d74bdcb, 2023-03-01T10:23:45.993Z)
OS version: Darwin x64 21.6.0
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-1068NG7 CPU @ 2.30GHz (8 x 2300)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
2, 2, 2

Memory (System)
32.00GB (0.51GB free)

Process Argv
--crash-reporter-id dfbc1fe0-1182-4b74-857c-79cfbcd1586f

Screen Reader
no

VM
0%

Extensions (46)

Extension
Author (truncated)
Version

rails-snippets
Hri
1.0.8

vscode-base64
ada
0.1.0

vscode-languagetool
ada
3.8.0

vscode-languagetool-en
ada
3.8.0

vscode-standardjs
che
1.4.1

githistory
don
0.6.19

gitlens
eam
13.3.1

json-tools
eri
1.0.2

prettier-vscode
esb
9.10.4

code-runner
for
0.12.0

beautify
Hoo
1.5.0

references-plusplus
jri
0.0.5

Lua
key
0.0.9

vscode-codemetrics
kis
1.26.1

rpm-spec
Lau
0.3.2

sftp
lix
1.12.10

vscode-rubycommentdoc
mat
1.0.1

escape-quotes
mil
1.10.0

vscode-remote-workspace
mkl
0.44.0

prettify-json
moh
0.0.3

compare-folders
mos
0.22.3

vscode-docker
ms-
1.24.0

isort
ms-
2022.8.0

python
ms-
2023.4.0

vscode-pylance
ms-
2023.3.10

jupyter
ms-
2023.2.1000592019

jupyter-keymap
ms-
1.1.0

jupyter-renderers
ms-
1.0.15

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

cpptools
ms-
1.14.4

printcode
nob
3.0.0

yard
pav
0.4.0

ruby
reb
0.28.1

vscode-statusbar-json-path
ric
2.0.0

erb
Rie
0.1.1

markdown-preview-enhanced
shd
0.6.7

lua
sum
3.6.15

remove-blank-lines
tha
1.1.1

vscode-lua
tri
0.12.4

haml
vay
1.0.1

expand-selection-to-scope
vit
0.2.0

vscode-ruby
win
0.28.0

collapsed-expand-empty-tag
won
0.0.5

autocomplete-english-word
wus
0.1.7

JavaScriptSnippets
xab
1.8.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30669237
azure-dev_surveyone:30548225
vsccc:30610678
pyindex848:30662994
nodejswelcome1cf:30587006
282f8724:30602487
pyind779:30671433
89544117:30613380
pythonsymbol12:30671437
a9j8j154:30646983
vsctsb:30662444

\gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

@lramos15 Hi, I guess it was something related to caching, you can close the ticket.
I used the Activity Monitor to force close visual code and it solved my issue.
If I'll see it again I'll let you guys know. Thanks.
PS. my mac had some crash right before the issue, probably related.