Crash With Nullable Columns During Grouping in WPA

I have tried the SQL Trace Events sample and tried to group by ClientProcessId. This crashes WPA with a NullReferenceException
>	Microsoft.Performance.DataEngine.dll!Microsoft.Performance.DataEngine.DataColumnView<int?, Microsoft.Performance.Shell.Bridge.ConvertToHdv.FunctionWrapper<int?, Microsoft.Performance.SDK.Processing.Projection.CompositionProjection<int, SqlPlugin.SqlEventWithRelativeTimestamp, int?, Microsoft.Performance.SDK.Processing.Projection.IndexReadOnlyProjection<SqlPlugin.SqlEventWithRelativeTimestamp, System.Collections.ObjectModel.ReadOnlyCollection<SqlPlugin.SqlEventWithRelativeTimestamp>>, Microsoft.Performance.SDK.Processing.Projection.FuncProjectionAdapter<SqlPlugin.SqlEventWithRelativeTimestamp, int?>>>>.PrepareGroupingIndicesIfNecessary(out ushort uniqueGroupCount, out ushort[] cachedGroupingIndices)	Unknown
 	Microsoft.Performance.DataEngine.dll!Microsoft.Performance.DataEngine.DataColumnView<int?, Microsoft.Performance.Shell.Bridge.ConvertToHdv.FunctionWrapper<int?, Microsoft.Performance.SDK.Processing.Projection.CompositionProjection<int, SqlPlugin.SqlEventWithRelativeTimestamp, int?, Microsoft.Performance.SDK.Processing.Projection.IndexReadOnlyProjection<SqlPlugin.SqlEventWithRelativeTimestamp, System.Collections.ObjectModel.ReadOnlyCollection<SqlPlugin.SqlEventWithRelativeTimestamp>>, Microsoft.Performance.SDK.Processing.Projection.FuncProjectionAdapter<SqlPlugin.SqlEventWithRelativeTimestamp, int?>>>>.GroupUsingGroupingIndicies(int[] indices, int start, int count, int level, Microsoft.Performance.DataEngine.GroupDescriptorOutputStream childGroups)	Unknown
 	Microsoft.Performance.DataEngine.dll!Microsoft.Performance.DataEngine.DataColumnView<int?, Microsoft.Performance.Shell.Bridge.ConvertToHdv.FunctionWrapper<int?, Microsoft.Performance.SDK.Processing.Projection.CompositionProjection<int, SqlPlugin.SqlEventWithRelativeTimestamp, int?, Microsoft.Performance.SDK.Processing.Projection.IndexReadOnlyProjection<SqlPlugin.SqlEventWithRelativeTimestamp, System.Collections.ObjectModel.ReadOnlyCollection<SqlPlugin.SqlEventWithRelativeTimestamp>>, Microsoft.Performance.SDK.Processing.Projection.FuncProjectionAdapter<SqlPlugin.SqlEventWithRelativeTimestamp, int?>>>>.Group(int[] indices, int start, int count, int level, Microsoft.Performance.DataEngine.GroupDescriptorOutputStream childGroups)	Unknown
 	Microsoft.Performance.DataEngine.dll!Microsoft.Performance.DataEngine.DataColumnView<int?, System.__Canon>.GroupCore(int[] indices, int start, int count, int level, Microsoft.Performance.DataEngine.GroupDescriptorOutputStream childGroups)	Unknown
 	Microsoft.Performance.DataEngine.dll!Microsoft.Performance.DataEngine.HierarchicalDataView._movable_ResolveGroup(int childVisibleColumnIndex, int level, ref Microsoft.Performance.DataEngine.GroupDescriptor groupDescriptor)	Unknown
 	Microsoft.Performance.DataEngine.dll!Microsoft.Performance.DataEngine.HierarchicalDataView.PartitionOnKeys(int keysCount, int preservedKeyCount)	Unknown
 	Microsoft.Performance.DataEngine.dll!Microsoft.Performance.DataEngine.HierarchicalDataView.RefreshOnDataUpdate(int keyCount, int preservedKeyCount)	Unknown
 	Microsoft.Performance.DataEngine.dll!Microsoft.Performance.DataEngine.HierarchicalDataView.OnDataUpdated()	Unknown
 	Microsoft.Performance.DataEngine.dll!Microsoft.Performance.DataEngine.DataView.EndDataUpdate()	Unknown
 	Microsoft.Performance.UI.DataEngine.dll!Microsoft.Performance.UI.DataEngine.HdvViewModel.CompleteApplyPreset(string presetName, Microsoft.Performance.DataEngine.GraphDataFormat newGraphDataFormat, bool ignoreInitialSelection, bool ignoreInitialExpansion, bool reinitializeFilter, bool initialTimeFilterRequestLocal)	Unknown
 	Microsoft.Performance.UI.DataEngine.dll!Microsoft.Performance.UI.DataEngine.HdvViewModel.ContinuePresetAfterGridModelInSync.AnonymousMethod__0()	Unknown
 	Microsoft.Performance.Windows.dll!Microsoft.Performance.Threading.BackgroundWorkManager.Post.AnonymousMethod__0(object _)	Unknown
 	System.Private.CoreLib.dll!System.Threading.QueueUserWorkItemCallback..cctor.AnonymousMethod__6_0(System.Threading.QueueUserWorkItemCallback quwi) Line 808	C#
 	System.Private.CoreLib.dll!System.Threading.ExecutionContext.RunForThreadPoolUnsafe<System.__Canon>(System.Threading.ExecutionContext executionContext, System.Action<System.__Canon> callback, System.__Canon state) Line 335	C#

When I make ClientProcessId in SqlEvent non nullable grouping works as expected. Is this a known bug?

Hi @Alois-xx,
Thank you for finding this. It appears to be an issue with WPA - not the SDK - so I will close this issue. Currently, WPA does not support grouping on nullable data types. A future release of WPA may allow this.
In the future, if the bug appears to be with WPA, you can use Feedback Hub to make sure the bug report gets to the appropriate team as soon as possible.

@mslukebo: I have sent feedback to WPA referencing this issue. But until nullable columns are supported shouldn't the sample be adapted to reflect the things which are actually working? It might take years until that backlog item is processed?

@Alois-xx,
Thank you for sending the feedback. The sample is going to remain as it is because

The column in question is a primitive type that may not be present for every data point. int? is the most appropriate type.
Samples highlight how to use the SDK and are not specific to WPA. Other SDK drivers may work fine with nullable columns.

Regardless, we hope to get this bug resolved as soon as possible. At the very least, WPA should not crash 😃. We are hoping to release more frequently as to not take years to fix.


JCE cannot authenticate the provider BC` when running a JAR with assembly

In my scala project I use "org.bouncycastle" % "bcprov-jdk14" % "1.51" for cryptography.
If it test my project in Scala IDE it works fine. But once I make a JAR and try to run it via java -jar ... it throws an exception: Exception in thread "main" java.lang.SecurityException: JCE cannot authenticate the provider BC
This is a issue in stackoverflow, I met the same issue, I think this is assembly plugin's problem.

It's not a plugin problem. Please see the answer on Stackoverflow -https://stackoverflow.com/a/13726570/3827

the solution can be to provide the bcprov-*.jar via -classpath
java -cp chat-server.jar:/Users/prayagupd/.ivy2/cache/org.bouncycastle/bcprov-ext-jdk16/jars/bcprov-ext-jdk16-1.46.jar com.chat.server.ChatServer


Any Roadmap?

Would be great to see this demo evolving frequently, it has a lot of potential.
Any plans to add these features "streaming, pagination, benchmarks, and a more advanced security model..."

Hey,
thanks for the suggestion, we agree that those should be in there. We are currently doing some changes and currently are focusing our efforts on other areas. Give us some time to think about how and when to add these in.

Hey, thanks for your response.
I think it would be useful to communicate these plans to the community, so we have an idea of where this is going,
This could promote collaboration from users who want to do similar things.

Definitely, we'd have to think about those and come up with a plan in order not to overpromise. Thanks for the kind words on the potential of the repo.

We are currently working on a new flagship app that takes you from an empty folder/directory to a complete, productionized app. We're not getting rid of Fwitter, but we are directing efforts to support new features there.
Look for it to launch by October 31st!


Propose Link for Sovereign Region to clarify doubt

This is the first time I'm reading this documentation, and it might be confusing as to us "sovereign regions" refers to specific 3 data centers for Microsoft Azure.
As a customer or partner, it might be assumed  "sovereign regions" refers to any specific datacenter and their country of establishment. Example the entire phrase can be mistaken as such
"When you set a region, remember that Azure Cosmos DB respects sovereign and government clouds. That is, if you create an account in a [SouthEastAsia], you cannot replicate out of that [SouthEastAsia]. Similarly, you cannot enable replication into other sovereign locations from an outside account. "
I'm proposing a hyperlink to our official page, in case customers / partners need more details on what is " "sovereign regions"

@chunsiongtan : Thanks for your contribution! The author, @SnehaGunda, has been notified to review your proposed change.

@chunsiongtan thank you. We will merge your PR soon.

#sign-off


More unit tests for the  Nextcloud storage provider

Ah, I'll need to adapt some of the backend mock fixtures for the integration tests, will do that tomorrow.

All green now, @labkode please review


Analytic function does not originate from dependencies

I have run into an issue where I am unable to add a fully functioning analytic column to a table expression after filtering on a column not original to the table.
import ibis

t = ibis.table((('col', 'int32'), ), name='t')

# can be any added column e.g. t['col'].name('filter') raises an exception too
t = t[t.columns + [ibis.null().name('filter')]]

# can be any comparison, removing this line lets everything work
t = t[t['filter'].isnull()]

# removing this line lets everything work
t = t[['col']]

# Non analytic functions work here, i.e. (t['col'] + 1).name('count')
count = t.count().name('count')
t = t[t.columns + [count]]

# compiles fine and looks 100% correct
print ibis.impala.compile(t)

t[t.columns]  # fails here

Traceback (most recent call last):
  File "ibis_fail.py", line 14, in <module>
    t[t.columns]
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/types.py", line 607, in __getitem__
    return self.projection(what)
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/api.py", line 2058, in projection
    op = projector.get_result()
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/analysis.py", line 685, in get_result
    fused_op = self._check_fusion(roots[0])
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/analysis.py", line 725, in _check_fusion
    elif validator.validate(lifted_val):
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/analysis.py", line 764, in validate
    if self._among_roots(op.table.op()):
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/analysis.py", line 777, in _among_roots
    return self.roots_shared(node) > 0
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/analysis.py", line 782, in roots_shared
    if root.is_ancestor(node):
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/operations.py", line 1746, in is_ancestor
    reboxed = Selection(table, self.selections)
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/operations.py", line 1673, in __init__
    self._validate(dependent_exprs)
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/operations.py", line 1694, in _validate
    self.table._assert_valid(exprs)
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/types.py", line 576, in _assert_valid
    ExprValidator([self]).validate_all(exprs)
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/analysis.py", line 805, in validate_all
    self.assert_valid(expr)
  File "/pipedream/local/venv/deepfield-env/lib/python2.7/site-packages/ibis/expr/analysis.py", line 810, in assert_valid
    raise RelationError(msg)
ibis.common.RelationError: The expression ref_0
UnboundTable[table]
  name: t
  schema:
    col : int32

ref_1
Selection[table]
  table:
    Table: ref_0
  selections:
    col = Column[array(int32)] 'col' from table
      ref_0
    filter = NullLiteral[null]
      None

ref_2
Selection[table]
  table:
    Table: ref_1
  predicates:
    IsNull[array(boolean)]
      filter = Column[array(null)] 'filter' from table
        ref_1

col = Column[array(int32)] 'col' from table
  ref_2 does not fully originate from dependencies of the table expression.

I have time to look deeper into this tomorrow. If I'm doing something wrong or if there is a good place to start looking for a fix please let me know!

I actually just pulled master again and I'm getting this error now
support@docker-host-unknown:~/ibis$ python ibis_fail.py
SELECT `col`, count(*) OVER () AS `count`
FROM (
  SELECT `col`
  FROM (
    SELECT *
    FROM (
      SELECT `col`, NULL AS `filter`
      FROM t
    ) t2
    WHERE `filter` IS NULL
  ) t1
) t0
Traceback (most recent call last):
  File "ibis_fail.py", line 16, in <module>
    t[t.columns]
  File "/home/support/ibis/ibis/expr/types.py", line 607, in __getitem__
    return self.projection(what)
  File "/home/support/ibis/ibis/expr/api.py", line 2058, in projection
    op = projector.get_result()
  File "/home/support/ibis/ibis/expr/analysis.py", line 685, in get_result
    fused_op = self._check_fusion(roots[0])
  File "/home/support/ibis/ibis/expr/analysis.py", line 725, in _check_fusion
    elif validator.validate(lifted_val):
  File "/home/support/ibis/ibis/expr/analysis.py", line 764, in validate
    if self._among_roots(op.table.op()):
  File "/home/support/ibis/ibis/expr/analysis.py", line 777, in _among_roots
    return self.roots_shared(node) > 0
  File "/home/support/ibis/ibis/expr/analysis.py", line 782, in roots_shared
    if root.is_ancestor(node):
  File "/home/support/ibis/ibis/expr/operations.py", line 1741, in is_ancestor
    table = table.op().table
AttributeError: 'UnboundTable' object has no attribute 'table'

I will try to take a look tonight. Shouldn't be too hard to track down.

@maxmzkr Did this ever work for you in the past? A regression will be easier to debug.

I haven't tried it in any older versions

Cool, thanks.

In the case that anyone else runs into this in the mean time, you can do a projection directly with the string values and everything seems to work, so replace the last line with
t.projection(t.columns)

This seems to be enough to "fix up" the table. After doing the projection, everything works as expected.

@maxmzkr Very interesting. After I get to the bottom of this we should probably unify the code path here and make these expressions equivalent.


Add documentation for metric ui and prometheus

Signed-off-by: ariamarble<EMAIL_ADDRESS>Description
Adds documentation for metric ui and prometheus
Issues Resolved
fixes #1993
Checklist

[X] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Staging - https://ariamarble.github.io/observing-your-data/prometheusmetrics/

Tech review - @anirudha

Tech review complete, working on updates then sending for doc review.

@hdhalter I have addressed yours and Chris' comments. Can you review? Thanks!

@hdhalter please review my changes. thanks


Check for abandoned child start events

Description & motivation 💭
Checks for ChildWorkflowExecutionStartedEvents that don't have a corresponding closed event and displays them under Relationships > Children.
Screenshots (if applicable) 📸
Design Considerations 🎨
Testing 🧪
How was this tested 👻

[x] Manual testing
[ ] E2E tests added
[x] Unit tests added

Steps for others to test: 🚶🏽♂🚶🏽♀

[ ] Run samples-typescript > child-workflows to verify the UI loads and there isn't the following error ⬇

[ ] Verify child events show up as expected under Relationships > Children

when events are in ascending order
when events are in descending order
when all child events have been completed (no duplicates)
when some child started events don't have corresponding closed events

Checklists
Draft Checklist
Merge Checklist
Issue(s) closed
DT-1531
Docs
Any docs updates needed?

Just a general question: How can you abandon a child workflow? What's the situation where it doesn't complete/fail/cancel/terminate...

Just a general question: How can you abandon a child workflow? What's the situation where it doesn't complete/fail/cancel/terminate...

Looks like you can abandoned children if the parent needs to close but children are still running.

Also when I run samples-typescript / child-workflows, I don't get any errors (not on this branch). Do I need to do something to trigger the error?

See this comment.

ON HOLD: In testing it appears that children that haven't completed once a parent with an abandon child policy closes should continue to show up as pending children (even once the children complete). Awaiting further repro. info.


[zh] Translate reference/access-authn-authz/authentication.md

closes: #22943

/assign @haibinxie

@tanjunchen Thanks for the careful review. All feedbacks addressed.

/lgtm

/approve


SUNDIALS #1135

@atkawa7,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks for the PR!
The examples added an otherwise-unneeded dependency on python, so I disabled them.


Trouble building ubuntu core images. Error creating mount directory: mkdir /tmp/.../etc/resolv.conf file exists

Trying to build 64 bit ubuntu core image and getting the following error.
==> arm-image: Growing partition with 1024 M<PHONE_NUMBER> bytes)
    arm-image: mapping output-arm-image/image
==> arm-image: kpartx -s -a -v output-arm-image/image
==> arm-image: partitions: [/dev/mapper/loop1p1 /dev/mapper/loop1p2]
==> arm-image: partitions: [/dev/mapper/loop1p1 /dev/mapper/loop1p2]
    arm-image: Mounting: /dev/mapper/loop1p2
    arm-image: Mounting: /dev/mapper/loop1p1
==> arm-image: Mounting additional paths within the chroot...
    arm-image: Mounting: /proc
    arm-image: Mounting: /sys
    arm-image: Mounting: /dev
    arm-image: Mounting: /dev/pts
    arm-image: Mounting: /proc/sys/fs/binfmt_misc
==> arm-image: Error creating mount directory: mkdir /tmp/108605975/etc/resolv.conf: file exists
==> arm-image: fuser -k /tmp/108605975
Build 'arm-image' errored: Error creating mount directory: mkdir /tmp/108605975/etc/resolv.conf: file exists

==> Some builds didn't complete successfully and had errors:
--> arm-image: Error creating mount directory: mkdir /tmp/108605975/etc/resolv.conf: file exists

==> Builds finished but no artifacts were created.

the json file:
{
  "variables": {
    "home": "{{env `HOME`}}"
  },
  "builders": [{
    "type": "arm-image",
    "image_type": "raspberrypi",
    "qemu_binary": "qemu-aarch64-static",
    "iso_url" : "http://cdimage.ubuntu.com/releases/20.04/release/ubuntu-20.04-preinstalled-server-arm64+raspi.img.xz",
    "iso_checksum_type":"sha256",
    "iso_checksum":"48167067d65c5192ffe041c9cc4958cb7fcdfd74fa15e1937a47430ed7b9de99",
    "last_partition_extra_size" :<PHONE_NUMBER>
  }],
  "provisioners": [
    {
      "type": "shell",
      "inline": ["touch /boot/ssh"]
    },
    {
      "type": "file",
      "source": "/.ssh/id_rsa.pub",
      "destination": "/home/pi/.ssh/authorized_keys"
    },
    {
      "type": "shell",
      "inline": [
        "sed '/PasswordAuthentication/d' -i /etc/ssh/ssh_config",
        "echo  >> /etc/ssh/ssh_config",
        "echo 'PasswordAuthentication no' >> /etc/ssh/ssh_config"
      ]
    },
    {
      "type": "shell",
      "inline": [
	      "apt-get update",
	      "apt-get install -y zfs-dkms, atop",
	      "curl -SL https://github.com/prometheus/node_exporter/releases/download/v0.18.1/node_exporter-0.18.1.linux-armv7.tar.gz > node_exporter.tar.gz",
	      "sudo tar -xvf node_exporter.tar.gz -C /usr/local/bin/ --strip-components=1",
	      "cd /etc/systemd/system/",
              "curl -sSLo - https://raw.githubusercontent.com/prometheus/node_exporter/master/examples/systemd/node_exporter.service |sed -e 's|/etc/sysconfig/node_exporter|/etc/default/node_exporter|' |sed -e 's/^User.*//'  > node_exporter.service",
               "curl -sSLo - https://raw.githubusercontent.com/prometheus/node_exporter/master/examples/systemd/sysconfig.node_exporter  |sed -e 's/^OPTIONS/#OPTIONS/;aOPTIONS=\"\"' >/etc/default/node_exporter",
               "systemctl daemon-reload",
               "systemctl enable node_exporter.service",
               "systemctl start node_exporter.service"
      ]
    }
  ]
}```

Related to #58, #36 #22 ?
Using information from #36 I was able to get the build going. Then I saw #58 also reporting the same issue.
I have attached a pull request #64 to add a ubuntu core sample for others to avoid running into the same issue.


blog: March '22 Heartbeat

Status:  Ready for review @flippedcoder!  Links are being skipped, so I still need to check manually.

[ ]  There are no grammatical errors, jargon, or obvious stylistic issues
[x]  Have images been run through tiny.png?
[x]  Are all lines 80 characters or less?
[x]  Are Oxford commas observed?
[x]  Content Creator has done a complete review for typos/language after taking a break from content
[ ]  links checked in piece
[x]  Have reviewed what the piece looks like on your own [machine using yarn develop](https://dvc.org/doc/user-guide/contributing/docs#development-environment)

@rogermparent Having link issues with Towards Data Science links.

This is ready to go except for the links that I don't know what to do about.  If it can't go today, I need to change the dates on everything and delete a section (tomorrow's meetup). @shcheklein @jorgeorpinel @rogermparent

@jendefig LGTM ... I've approved it, feel free to Squash and merge when you are ready.

@shcheklein has approved, but hold. I may add a statement for the Community about Ukraine.

I investigated the towardsdatascience links and had a rough time pinpointing what's wrong, but I think I found it: TDS returns 503 when given node-fetch's default user agent.
How I diagnosed it:

link-check CLI fails with the same code 503
the links are fine in a browser
default curl returns 200 (OK)
Lychee (a well established link checking library) succeeds when given a towardsdatascience link

After getting a hunch that something in our request was the problem, I snooped into the requests link-check was sending by running the link check CLI with NODE_DEBUG=http. Nothing else looked off, so I tried using our link-check's user agent on curl by running the command curl -A 'node-fetch/1.0 (+https://github.com/bitinn/node-fetch)' -L --head -X HEAD https://towardsdatascience.com/, which finally returned a HTTP/2 503- the failing code we're looking for.
Pinging @julieg18 since this info should be helpful where she's looking into our link checker right now. I should be able to get together an interim solution if we need it, but weird problems like this are a big reason why I'd personally throw my hat behind using a popular battle-tested library like lychee for our planned link-check revamp.

@julieg18 @rogermparent Need some help here.
I just put in a message on Ukraine at the top.  I want to include the headings and bulleted links inside the details.  Is that possible?  I had them all aligned left and that didn't work and then I tabbed it all in, but that didn't work either.
Alternatively I could make a new notion page with all those links and make it public and just put that link in there.  LMK.
@shcheklein @jorgeorpinel Could use review of the message please. 🙏

I just put in a message on Ukraine at the top. I want to include the headings and bulleted links inside the details. Is that possible? I had them all aligned left and that didn't work and then I tabbed it all in, but that didn't work either.

Nevermind!  @julieg18 @rogermparent Apparently tabbing them in did work! 🎉

This has been approved by @shcheklein.  There are some new links that have the same problem as Toward Data Science ones, but they work in the deployment.  Can someone please merge this for me? @julieg18 @rogermparent?


High CPU usage

Type: Performance Issue

Behaviour
CPU usage should be minimal. Ideally less than 5%.
These pyenv-which and pyenv-exec processes are spawned as soon as VSCode is opened. Takes a few minutes before they start hitting 100%. They also never terminate when you completely close VSCode. Manually terminating the processes then restarting VSCode will do the same thing over again.
Steps to reproduce:

Open VSCode
Open a Python project or two
Make sure the Python extension is enabled
Wait about 5 minutes for your CPU to be pegged
Kill VSCode, observe the pyenv-exec processes with ms-python.vscode-pylance in the -I argument
Reopen VSCode and now more of the same processes are running, but CPU is completely pegged

Diagnostic data

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

Extension version: 2024.13.2024080701
VS Code version: Code 1.92.0 (b1c0a14de1414fcdaa400695b4db1c0799bc3124, 2024-07-31T23:26:45.634Z)
OS version: Darwin arm64 23.6.0
Modes:

Python version (& distribution if applicable, e.g. Anaconda): 3.8.19
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Poetry
Value of the python.languageServer setting: Default

User Settings

languageServer: "Pylance"

experiments
• enabled: false

Installed Extensions

Extension Name
Extension Id
Version

autoDocstring - Python Docstring Generator
njpwerner.autodocstring
0.6.1

Better Jinja
samuelcolvin.jinjahtml
0.20.0

Django Support
almahdi.code-django
0.5.4

ES7+ React/Redux/React-Native snippets
dsznajder.es7-react-js-snippets
4.4.3

Even Better TOML
tamasfe.even-better-toml
0.19.2

GitHub Actions
github.vscode-github-actions
0.26.3

GitHub Copilot
GitHub.copilot
1.220.1035

GitHub Copilot Chat
GitHub.copilot-chat
0.18.1

GitHub Pull Requests
GitHub.vscode-pull-request-github
0.95.2024080708

GitHub Theme
GitHub.github-vscode-theme
6.3.4

GitLens — Git supercharged
eamodio.gitlens
15.2.3

Go
golang.go
0.42.0

Go Template Support
jinliming2.vscode-go-template
0.2.1

HashiCorp HCL
hashicorp.hcl
0.5.0

HashiCorp Terraform
hashicorp.terraform
2.33.2024073012

IntelliCode
VisualStudioExptTeam.vscodeintellicode
1.3.1

JavaScript Debugger
ms-vscode.js-debug
1.92.0

JavaScript Debugger Companion Extension
ms-vscode.js-debug-companion
1.1.3

Kubernetes
ms-kubernetes-tools.vscode-kubernetes-tools
1.3.16

Live Share
ms-vsliveshare.vsliveshare
1.0.5936

Markdown Preview Mermaid Support
bierner.markdown-mermaid
1.23.1

Material Icon Theme
PKief.material-icon-theme
5.8.0

Modelines
chrislajoie.vscode-modelines
2.0.6

Moonlight
atomiks.moonlight
0.11.1

One Dark Pro
zhuangtongfa.material-theme
3.17.2

Packer
4ops.packer
0.3.0

Prettier - Code formatter
esbenp.prettier-vscode
10.4.0

Pylance
ms-python.vscode-pylance
2024.8.1

Python
ms-python.python
2024.13.2024080701

Python Debugger
ms-python.debugpy
2024.10.0

Table Visualizer for JavaScript Profiles
ms-vscode.vscode-js-profile-table
1.0.9

Tailwind CSS IntelliSense
bradlc.vscode-tailwindcss
0.12.5

vscode-proto3
zxh404.vscode-proto3
0.5.5

vscode-styled-components
styled-components.vscode-styled-components
1.7.8

YAML
redhat.vscode-yaml
1.15.0

System Info

Item
Value

CPUs
Apple M1 Max (10 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
10, 9, 9

Memory (System)
32.00GB (0.20GB free)

Process Argv

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    3	   164	 79220	code main
    0	    66	 79221	   gpu-process
    0	    33	 79222	   utility-network-service
    0	   262	 79223	window [1] (metrics.py — beam-client)
    0	   229	 79224	window [2] (app.py — beta9)
    0	    66	 79348	ptyHost
    0	     0	 79354	     /bin/zsh -il
    0	     0	 79427	     /bin/zsh -i
    0	     0	 79492	     /bin/zsh -i
    1	    98	 79349	shared-process
    0	    66	 79350	fileWatcher [2]
    0	    66	 79351	fileWatcher [1]
    0	   131	 79352	extensionHost [1]
    0	     0	 79417	     /Users/npetrovic/.vscode/extensions/ms-python.python-2024.13.2024080701-darwin-arm64/python-env-tools/bin/pet server
    0	     0	 79444	       bash /Users/npetrovic/.pyenv/libexec/pyenv-exec python -c import json, sys; print('093385e9-59f7-4a16-a604-14bf206256fe');print(json.dumps({'version': '.'.join(str(n) for n in sys.version_info), 'sys_prefix': sys.prefix, 'executable': sys.executable, 'is64_bit': sys.maxsize > 2**32}))
    0	    33	 80132	     /Applications/Visual Studio Code.app/Contents/Frameworks/Code Helper (Plugin).app/Contents/MacOS/Code Helper (Plugin) /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/json-language-features/server/dist/node/jsonServerMain --node-ipc --clientProcessId=79352
    0	    66	 80574	     electron-nodejs (bundle.js )
    0	     0	 81170	       bash /Users/npetrovic/.pyenv/libexec/pyenv-exec python3 -I /Users/npetrovic/.vscode/extensions/ms-python.vscode-pylance-2024.8.1/dist/bundled/files/get_pytest_options.py
    0	     0	 81177	       bash /Users/npetrovic/.pyenv/libexec/pyenv-exec python3 -I /Users/npetrovic/.vscode/extensions/ms-python.vscode-pylance-2024.8.1/dist/bundled/files/get_pytest_options.py
    0	    33	 81061	     electron-nodejs (server-node.js )
    0	    33	 82170	     electron-nodejs (tailwindServer.js )
    0	    98	 79353	extensionHost [2]
    0	     0	 79416	     /Users/npetrovic/.vscode/extensions/ms-python.python-2024.13.2024080701-darwin-arm64/python-env-tools/bin/pet server
    0	     0	 79445	       bash /Users/npetrovic/.pyenv/libexec/pyenv-exec python -c import json, sys; print('093385e9-59f7-4a16-a604-14bf206256fe');print(json.dumps({'version': '.'.join(str(n) for n in sys.version_info), 'sys_prefix': sys.prefix, 'executable': sys.executable, 'is64_bit': sys.maxsize > 2**32}))
    0	    33	 80076	     /Applications/Visual Studio Code.app/Contents/Frameworks/Code Helper (Plugin).app/Contents/MacOS/Code Helper (Plugin) /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/json-language-features/server/dist/node/jsonServerMain --node-ipc --clientProcessId=79353
    0	     0	 80395	     /Users/npetrovic/.vscode/extensions/hashicorp.terraform-2.33.2024073012-darwin-arm64/bin/terraform-ls serve
    0	    66	 80530	     electron-nodejs (bundle.js )
    0	     0	 81185	       bash /Users/npetrovic/.pyenv/libexec/pyenv-exec python3 -I /Users/npetrovic/.vscode/extensions/ms-python.vscode-pylance-2024.8.1/dist/bundled/files/get_pytest_options.py
    0	     0	 81193	       bash /Users/npetrovic/.pyenv/libexec/pyenv-exec python3 -I /Users/npetrovic/.vscode/extensions/ms-python.vscode-pylance-2024.8.1/dist/bundled/files/get_pytest_options.py
    0	    33	 82521	     electron-nodejs (server-node.js )

Screen shots
VSCode running

VSCode closed/quit

Process explorer capturing the high CPU

Things tried

Completely removed pyenv and deleted the ~/.pyenv directory.
Tried an older version of pyenv (v2.3.21)
Wiped out my .venv directories for my projects and reinstalled with poetry
Disabled the Python extension in VSCode, resolves the issue, but then there's no Python support
Disabled the Python Debugger, isort, and Ruff extensions in VSCode
Disabled the Pylance extension, this looks like it resolves it, but after having VSCode open for about 5 minutes, htop shows pyenv-which is still hitting 100% at least every other second, but the process is no where to be found in the process explorer.
Confirmed that pyenv-which pyenv-exec processes continue to spawn as long as VSCode is opened.
Set the Python: Language Server setting to None, issue still happens
This looks identical to this issue on python-environment-tools - https://github.com/microsoft/python-environment-tools/issues/134

@nickpetrovic is this with particular pyenv environment or just anything in general? Could you share the logs from Output > Python?

If I remove pyenv entirely, it goes away. Doesn't matter which python project I open up. (See my "Things tried" section.
In my screenshots section, you can see the parent process of pyenv-which is /Users/npetrovic/.vscode/extensions/ms-python.python-2024.13.2024080901-darwin-arm64/python-env-tools/bin/pet server.
Here are the logs you asked for.

2024-08-09 14:22:33.537 [error] Reading directory to watch failed [Error: ENOENT: no such file or directory, scandir '/Users/npetrovic/Projects/beam-client/.pixi/envs'
	at Object.readdirSync (node:fs:1509:26)
	at t.readdirSync (node:electron/js2c/node_init:2:11418)
	at /Users/npetrovic/.vscode/extensions/ms-python.python-2024.13.2024080901-darwin-arm64/out/client/extension.js:2:583006
	at /Users/npetrovic/.vscode/extensions/ms-python.python-2024.13.2024080901-darwin-arm64/out/client/extension.js:2:583197
	at Array.map (<anonymous>)
	at d.initWatchers (/Users/npetrovic/.vscode/extensions/ms-python.python-2024.13.2024080901-darwin-arm64/out/client/extension.js:2:582915)
	at async d.ensureWatchersReady (/Users/npetrovic/.vscode/extensions/ms-python.python-2024.13.2024080901-darwin-arm64/out/client/extension.js:2:539326)] {
  errno: -2,
  code: 'ENOENT',
  syscall: 'scandir',
  path: '/Users/npetrovic/Projects/beam-client/.pixi/envs'
}
2024-08-09 14:22:33.537 [error] Dir "/Users/npetrovic/Projects/beam-client/.pixi/envs" is not watchable (directory does not exist)
2024-08-09 14:22:33.571 [info] Default formatter is set to charliermarsh.ruff for workspace /Users/npetrovic/Projects/beam-client
2024-08-09 14:22:33.580 [info] Python interpreter path: ./.venv/bin/python
2024-08-09 14:22:33.728 [info] > ~/.pyenv/bin/pyenv which python
2024-08-09 14:22:33.728 [info] cwd: .
2024-08-09 14:22:34.669 [info] Starting Pylance language server.

I think I have the same issue when I'm working remote with my Raspberry Pi. The CPU overload goes that high, that the SSH connection breaks and I can not reconnect.

This (specifically the pixi error) should get addressed by https://github.com/microsoft/vscode-python/pull/23937 . Try the latest pre-release.

Duplicate of https://github.com/microsoft/vscode-python/issues/23906


feat: use charset utf8 by default

I am getting the same error
nginx: [emerg] unknown directive "charset" in /app/nginx/nginx.conf:13
nginx: [emerg] unknown directive "charset" in /app/nginx/nginx.conf:13
nginx: [emerg] unknown directive "charset" in /app/nginx/nginx.conf:13
nginx: [emerg] unknown directive "charset" in /app/nginx/nginx.conf:13
nginx: [emerg] unknown directive "charset" in /app/nginx/nginx.conf:13

You may need to clear your repo cache using repo:purge-cache.
At worst, I'll revert the directive addition, and it'll be up to the user to include that on their end.

That fixed it! Also I think no need to revert, if anyone hits this error and repo:purge-cache doesn't fix it they can just pin their buildpack to the previous release like https://github.com/dokku/buildpack-nginx#v13


Fix an issue with the fastify/node server crash

An invalid hexadecimal string in a data parameter in a decrypt method crashes the server process.

Thank you for taking the time to find this issue and submit this PR.
Very much appreciated!

Merged and now available on NPM on versions 1.2.1 and up.


Restructuring "robots.json"

The robots.json needs to be altered for supporting a new Entity-Layout

The documentation and configuration is still  missing

Documntation is added.


Tweak Light3D property hint to be more accurate for LightmapGI usage

For LightmapGI, Bake Mode works as it did in 3.x with BakedLightmap.

It's not clear of the added "Direct + Indirect" and "Indirect Only" apply only to LightmapGI or also to VoxelGI and SDFGI.

It's not clear of the added "Direct + Indirect" and "Indirect Only" qualifiers apply only to LightmapGI or also to VoxelGI and SDFGI.

Would replacing slashes with commas make it easier to understand?

Static (VoxelGI, SDFGI, LightmapGI Direct + Indirect)
Dynamic (VoxelGI, SDFGI, LightmapGI Indirect Only)

@Calinou Either of your suggestions looks fine to me as a non-native English speaker

@Calinou Either of your suggestions looks fine to me as a non-native English speaker

Nevermind, I realized property hints can't contain , within them as that's the separator.


rm -rf fails with 'invalid argument' while building on overlayfs, CentOS 7.1

I'm unable to build docker images on an up-to-date CentOS 7.1 machine using ext4-backed overlayfs.
 May be, you can try upgrading your kernel?
root@ubuntu:~# uname -a
Linux ubuntu 3.19.0-25-generic #26~14.04.1-Ubuntu SMP Fri Jul 24 21:16:20 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

root@ubuntu:~# docker build .
Sending build context to Docker daemon 982.5 MB
Sending build context to Docker daemon
Step 0 : FROM busybox
latest: Pulling from busybox
cf2616975b4a: Pull complete
6ce2e90b0bc7: Pull complete
8c2e06607696: Already exists
busybox:latest: The image you are pulling has been verified. Important: image verification is a tech preview feature and should not be relied on to provide security.
Digest: sha256:38a203e1986cf79639cfb9b2e1d6e773de84002feea2d4eb006b52004ee8502d
Status: Downloaded newer image for busybox:latest
 ---> 8c2e06607696
Step 1 : RUN mkdir /tmp/a; touch /tmp/a/b
 ---> Running in 4879bc1d0f02
 ---> 7c24446f9cb0
Removing intermediate container 4879bc1d0f02
Step 2 : RUN rm -rf /tmp/a
 ---> Running in 11469f761368
 ---> 90cb3abd8ff6
Removing intermediate container 11469f761368
Successfully built 90cb3abd8ff6

Also, rm -rf against stuff in a previous layer doesn't make sense, since all the layers are required to make the image, the lower layer is still kept.

Not that this isn't an issue...

@cpuguy83 Apologies if I've misunderstood, but the example I provided is very simplified for the purposes of the bug report. In my particular use case, I'm hitting this while doing some cleanup of various files after a package installation. (a RUN yum install.. followed by a RUN rm -rf ...).
I think this pattern does make sense and will be present in quite a lot of dockerfiles.

@lewispeckover yum is a known issue with overlay.

@cpuguy83 This is not at all related to the rpm db issues that yum has though, which, as I understand it, are due to it's unusual way of double-opening the db files.

@cpuguy83 @lewispeckover so, can we say that it was fixed in kernel?

Seems like that has been fixed in the latest CentOS kernels:
] uname -r
3.10.0-327.18.2.el7.x86_64

] docker build - <<EOF
FROM busybox
RUN mkdir /tmp/a; touch /tmp/a/b
RUN rm -rf /tmp/a
EOF
Sending build context to Docker daemon 2.048 kB
Step 1 : FROM busybox
 ---> 2b8fd9751c4c
Step 2 : RUN mkdir /tmp/a; touch /tmp/a/b
 ---> Using cache
 ---> cef03d9cb0a5
Step 3 : RUN rm -rf /tmp/a
 ---> Using cache
 ---> d2aa44ea8cd7
Successfully built d2aa44ea8cd7

This works on a modern kernel, so I think you will have to file an issue with Red Hat.

I had the same issue. It can be solved by formatting the underlying XFS filesystem with ftype=1. See https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html/7.2_Release_Notes/technology-preview-file_systems.html for more information.
$ ~/docker/test% uname -a
Linux hkgdev01 3.10.0-327.36.3.el7.x86_64 #1 SMP Mon Oct 24 16:09:20 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
$ ~/docker/test% cat Dockerfile
FROM busybox
RUN mkdir /tmp/a; touch /tmp/a/b
RUN rm -rf /tmp/a
$ ~/docker/test% docker build -t test .
Sending build context to Docker daemon 2.048 kB
Step 1 : FROM busybox
 ---><PHONE_NUMBER>dc
Step 2 : RUN mkdir /tmp/a; touch /tmp/a/b
 ---> Running in c32da72a770e
 ---> bd3bbe3cfd11
Removing intermediate container c32da72a770e
Step 3 : RUN rm -rf /tmp/a
 ---> Running in 9a8d92c028a5
 ---> ed378a7113ae
Removing intermediate container 9a8d92c028a5
Successfully built ed378a7113ae

Yes, this looks like the issue with ftype not being set on XFS. The current version of Docker shows this information in the output of docker info;
Storage Driver: overlay
 Backing Filesystem: xfs
 Supports d_type: false

Running overlay (or overlay2) on XFS without d_type is known to cause issues. Docker currently "allows" it for backward-compatibility, but will block it in future, you'll see a warning being logged about this as well when starting the daemon;
overlay: the backing xfs filesystem is formatted without d_type support,
which leads to incorrect behavior. Reformat the filesystem with ftype=1
to enable d_type support.
Running without d_type support will no longer be supported in Docker 17.12.

Also see these issues, which contain some information https://github.com/moby/moby/issues/10294, https://github.com/moby/moby/issues/29873.
The RHEL release notes mentions this; https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html/7.2_Release_Notes/technology-preview-file_systems.html

Note that XFS file systems must be created with the -n ftype=1 option enabled for use as an overlay. With the rootfs and any file systems created during system installation, set the --mkfsoptions=-n ftype=1 parameters in the Anaconda kickstart. When creating a new file system after the installation, run the # mkfs -t xfs -n ftype=1 /PATH/TO/DEVICE command. To determine whether an existing file system is eligible for use as an overlay, run the # xfs_info /PATH/TO/DEVICE | grep ftype command to see if the ftype=1 option is enabled.
Note that OverlayFS provides a restricted set of the POSIX standards. Test your application thoroughly before deploying it with OverlayFS.

I'll go ahead and close this, but feel free to comment

@thaJeztah i'm having this issue, even when not using xfs with overlay and d_type is true
Storage Driver: overlay
Backing Filesystem: extfs
Supports d_type: true
Logging Driver: json-file
Cgroup Driver: cgroupfs```

@hc000 what distro are you running on? I know on RHEL/CentOS, only XFS can be used as an underlying filesystem; https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html/7.2_Release_Notes/technology-preview-file_systems.html, not sure if that's related - do you have steps to reproduce?

Only XFS is currently supported for use as a lower layer file system.

@thaJeztah I can concur with @hc000; I'm seeing this same issue on CentOS 7.2. I selected overlayfs as the storage driver in my Docker config file and started to see this problem.
[user@kaiser ~]$ uname -a
Linux kaiser 3.10.0-327.36.2.el7.x86_64 #1 SMP Mon Oct 10 23:08:37 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

Server Version: 17.03.1-ce
Storage Driver: overlay
 Backing Filesystem: extfs
 Supports d_type: true
Logging Driver: json-file
Cgroup Driver: cgroupfs

@otherjason see my comment above; you probably need XFS as backing system; also CentOS used a rolling release model, which means that now CentOS 7.3 replaced 7.2 and 7.2 is EOL

@hc000 @otherjason update kernel from CentOS updates repository, if you didn't it already. 3.10.0-327 has problems with overlayfs AFAIK. It was fixed in later versions. 3.10.0-514 is fine, for example.


mill.scalalib.GenIdea/idea fails with netty-transport-native-epoll as dependency

mill.scalalib.GenIdea/idea fails with the following error when executing on a project that has netty-transport-native-epoll as a dependency.
1 targets failed
mill.scalalib.GenIdea.idea java.nio.file.NoSuchFileException: ~/.cache/coursier/v1/https/repo1.maven.org/maven2/io/netty/netty-transport-native-epoll/4.1.46.Final/netty-transport-native-epoll-4.1.46.Final-linux-x86_64.pom

It's complaining about a missing pom file which has a linux-x86_64 suffix. Looking into the ~/.cache/coursier.../4.1.46.Final directory, we can see the said pom file does not exists, whereas a pom file without the suffix exists.
 ~/coursier/v1/https/repo1.maven.org/maven2/io/netty/netty-transport-native-epoll/4.1.46.Final:
   netty-transport-native-epoll-4.1.46.Final-linux-x86_64.jar
   netty-transport-native-epoll-4.1.46.Final-sources.jar
   netty-transport-native-epoll-4.1.46.Final.pom

The issue seems to come from the logic in GenIdeaImpl looks for a POM extension with the same filename as the jar which does not cater for classifiers.
We think one solution to resolve this issue is to change the code to look for any file with a POM extension in the same directory as the jar.
More detailed information by @zainab-ali can be found here https://github.com/lsug/lsug-website/issues/90
Would you like us to PR this change? Also, thank you for working on and maintaining this build tool. Our project has benefited a lot from this project. :slightly_smiling_face:

Can you show how you defined the dependency? Did you used the classifier? In case you did not used a proper classifier, you should first try if using it fixes the issue. Something like this:
ivy"io.netty:netty-transport-native-epoll:4.1.46.Final;classifier=linux-x86_64"

If that doesn't help, yeah, a PR would be great. But the proper name of the pom file should be calculated from the maven/coursier coordinates. Picking random pom files will open more issues eventually instead of fixing them.

@kxa14 Could you try this branch? https://github.com/lefou/mill/tree/idea-sources
The issue is, that we, in the place where the idea generator fails, only see jars but don't have their ivy dependencies at hand, so we must play a guess game to find the source jars. In this branch I tried to guess a little bit better.

I just merged PR #1195.
@kxa14 Your feedback would be much appreciated.

@lefou Sorry for the late reply. Yes, it worked when I ran mill.scalalib.GenIdea/idea with the fix. I can see on the log file that it's parsing the right pom file
Parsing POM /home/kaixin/.cache/coursier/v1/https/repo1.maven.org/maven2/io/netty/netty-transport-native-epoll/4.1.46.Final/netty-transport-native-epoll-4.1.46.Final.pom ...

Thank you for looking into this.

Fixed by #1195.


SvelteKit Auth helper: PKCE flow doesn't work with generateLink (type magiclink)

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
I migrated from v0.9 to v0.10 following this guide, reviewing my auth code overall, not just the migration part.
https://supabase.com/docs/guides/auth/auth-helpers/sveltekit
but this:
const genLink = await supabase.auth.admin.generateLink({
      type: 'magiclink',
      email: email,
      options: {
         redirectTo: '.../callback'
      }
    })

doesn't seem to work anymore.
genLink.data.properties.action_link doesn't contain the code parameter, which I assume should be present for exchangeCodeForSession in the callback server-side code.
What am I doing wrong?
To Reproduce

Generate a magic link with auth.admin.generateLink via Supabase JS client with Service Key
Sent it via custom email service to the user (we use Postmark)
User clicks on the link
User is redirected to the callback route with exchangeCodeForSession
url.searchParams.get('code') returns nothing

I inspected the generated URL as present in the email sent to the user and it has the correct redirect_to parm, but doesn't have the code parm.
UPDATE: I also made sure to include this, when creating the client, but no luck:
auth: {
    flowType: 'pkce',
  },

Expected behavior
Authenticate user upon clicking on the link directly on the server, as intended by the new PKCE flow.
It was working fine previously with generateLink just to be clear.
System information

Version of supabase-js: latest
Version of SvelteKit Auth Helper: v0.10

Based on what I see in the code, this method doesn't support pcke flow. I don't know if it's planned.

Running in to this issue as well. Flow is hardcoded to pcke in the client and the "url is unsupported"

Did anyone of you got this to work by updating @supabase/gotrue-js ? I updated it to 2.45.0 but I am still not receiving the code.

No I didn’t get it to work either.
On Mon, 24 Jul 2023 at 8:14 PM, Tobias @.***> wrote:

Did anyone of you got this to work by updating @supabase/gotrue-js ? I
updated it to 2.45.0 but I am still not receiving the code.
—
Reply to this email directly, view it on GitHub
https://github.com/supabase/auth-helpers/issues/610#issuecomment-1648053221,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWM6RCSXDCOXV3LCMYHUI3XR2C4NANCNFSM6AAAAAA2OTYDUE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

For me, using the signInWithOtp works with the callback going to /auth/callback for the supabase.auth.exchangeCodeForSession().
However, as also described above, when using auth.admin.generateLink as following:
await supabaseAdmin.auth.admin.generateLink({
        type: 'magiclink',
        email,
        options: {
            redirectTo: `${url.origin}/auth/callback`
        }
    })

it seems the callback to /auth/callback does not contain any search params. Instead, I get #access_token hash on the url and it still does not trigger onAuthStateChange. - It does not log in the user.

Also update to the latest version but it still does not work.

Indeed, same story here.

A temporary workaround is to manually set the user session based on the access token:
const searchParams = new URLSearchParams($page.url.hash.replace('#', '?'))

const { error } = await supabase.auth.setSession({
    access_token: searchParams.get('access_token'),
    refresh_token: searchParams.get('refresh_token')
})

However, I'd much more like this to be fixed so it works the intended way.

bump!

thanks @kamerat for the work around! I remember doing exactly this "trick" 1.5 years ago with SvelteKit, this time with next.js.
Hope we can get an update soon to fix the issue.

We have published a new guide on how to handle this from the email perspective and framework side of things. Note that this guide does not mention supabase.auth.admin.generateLink() but the same setup will still work. Here is a link from the part that you would need to follow https://supabase.com/docs/guides/auth/server-side/email-based-auth-with-pkce-flow-for-ssr#create-api-endpoint-for-handling-tokenhash

Thanks to the guide mentioned by @silentworks, I figured it out and documented the process here: https://catjam.fi/articles/supabase-generatelink-fix

We have published a new guide on how to handle this from the email perspective and framework side of things. Note that this guide does not mention supabase.auth.admin.generateLink() but the same setup will still work. Here is a link from the part that you would need to follow https://supabase.com/docs/guides/auth/server-side/email-based-auth-with-pkce-flow-for-ssr#create-api-endpoint-for-handling-tokenhash

So the documentation points to changing the invitation link to doing something manually. This make sure the endpoint recieves the right data. But documentation says that verifyOtp online needs an option with two fields, but in reality it needs an option with three fields. Depending on the type. If email, it needs the email address itself.
The blog post mentioned above does also not work in my case. I tried using the generall strategy to fit for sveltekit but to no luck
The work around did also not work for me as I primarily work on the server side and was hoping to reach the server first

@vongohren I'm not sure what the blog post you linked to contains but my link to the guide is what you should follow. I also have a working example in my waiting-list project which uses SvelteKit and the verifyOtp method https://github.com/silentworks/waiting-list/blob/main/src/routes/auth/confirm/%2Bserver.ts#L22

@silentworks thanks for the reply! As for blog post, it was the post above mine by wyozi, that tried to elaborate on what is going on.
As for your code, you are using verifyOtp, without any email or phone value? How is this possible? I get an error? Is it because of my version of the library?
The only different think I do from you is that i generateLink(), type:invite, and take the token hash from there and send as an email through an i18n email function.
But that should not make any difference?
What is the expected outcome? The user loads the link, the server verifys and gives user a session so he can access authroized urls right?

Closing this one out as I believe it has been resolved by the link to the guide I posted above.

Thank you @wyozi -- were are still using the /pages router and no server components and were able to adapt your example relatively easily!


Bug fix for Remove and add tests

This PR fixes a bug with Remove caused by t.root not being assigned to the returned value (in the case where the only node in a tree is removed from the tree).
This PR also adds a small but fairly comprehensive test.

Thanks for the fix.


Updated g_y_annual min to 0.08 in default_parameters.json

This PR updates the g_y_annual minimum value in default_parameters.json to 0.08.
cc: @jdebacker

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 71.31%. Comparing base (e799452) to head (141053f).

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #959   +/-   ##
=======================================
  Coverage   71.31%   71.31%
=======================================
  Files          20       20
  Lines        4953     4953
=======================================
  Hits         3532     3532
  Misses       1421     1421

Flag
Coverage Δ

unittests
71.31% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Update line interpolate point

Still need to add python bindings. What's the shapely name for this again?

Added AnyPrimitiveBroadcastInput; should now be able to handle float broadcast arguments

TODO: docstrings, accept numpy arrays for broadcasting


new rules for eslint

Removed a few errors, brought different warnings. Working in a different branch on fixing all of the errors.

This PR also adds validation for our naming convention.


Create the Dossier Review Page

This PR only involves creating the UI for the Dossier Review Page. The backend functionalities for the page are not complete.
This closes #243
Demo:
https://github.com/JamesPartsafas/ConcordiaCurriculumManager/assets/66841718/64b5327a-e211-4646-ab26-4592e239acc7

Demo:
https://github.com/JamesPartsafas/ConcordiaCurriculumManager/assets/66841718/f5e3eac3-e6e9-4fcd-8f65-ff649f67e3df


How to add another route / page

Hey there,
I'm trying to add another page and I'm getting misc errors. Ex: "This route doesn't match any handlers"  https://puu.sh/tIE9D/c26c9ac186.png. Another example is that on submitting a form on the front end the backend the endpoint gives a 404. Previously it gave a 403 auth error and I followed your wiki to solve it. Any full stack / full page life cycle tutorial for the app would be appreciated.
Thanks

Thanks for opening an issue.

Any full stack / full page life cycle tutorial for the app would be appreciated.

It would be great if someone wants to add a wiki page on this. I think figuring it out would be part of the fun. In general though, we're not doing anything fancy or that isn't documented in the appropriate modules. Here is a brief overview.
Server:

We're using hapi as the web server, so follow the trails in ./server.js, ./server/*.
One thing that can be confusing is that we use the glue module to compose the server, see ./manifest.js.

Client:

In it's default state, the front-end is essentially 5 single page apps, some with their own routes and some without.

You can see this in the webpack configuration.

We're using react-router on the front-end.

Adding a whole new entry point will require a new server side render see `./server/www/
But adding a page to /login/, /account/ or /admin/ should just require a new entry in the appropriate routes.jsx file.

I hope that helps get you going.


First issue (using issues for todos and ideas)

Took me long enough to use the Issues feature on GitHub, but I'm gonna try to use it to use it to document todos and ideas. I've tried to use the README.mds in the project's root and in the nvim directory so that I could do everything in the terminal, but its kinda clunky to quickly jot something down. Also tried using the Projects tab, but its web-only. Using Issues should be the best of both worlds, and I can use it on my phone with the GitHub app! I thought people only use it for actual projects and real issues, but better late than never.

Using issues is going pretty well, I even managed to get a GitHub workflow that automatically adds new issues to my private Project (#21). This is the way to go.


Check GHC version on launch

Logs to the hie log, logs to the lsp log and shows a message to the user if the HIE ghc version doesn't match up with the project ghc version, should hopefully close #459 and #414, and many more support requests

This looks like it needs to be updated for the Semigroup / Monoid change that GHC 8.4 introduced.


Awral

@jvgeiger @dmocko @shugongwang, I've marked this as a draft pull request to get feedback.
Please take a look at the current implementation of AWRAL 6.0.0. I have followed the general template design and merged the latest version of master to my forked repository.
The Australian domain is implemented at 0.05 degree resolution.
I have generated a testcase, see: LISF/lis/testcases/surfacemodels/land/awral.6.0.0
The input and output tarballs are on the er4 project on Raijin:
/g/data/er4/LIS/AWRAL_testcase

@bmcandr @dmocko @shugongwang I've reviewed the comments and tried to address all of them bar the documentation as there is a comprehensive model description report available that I thought we could include? See: http://www.bom.gov.au/water/landscape/assets/static/publications/AWRALv6_Model_Description_Report.pdf

Thanks @wendyksharples! I do not have access to Raijin so I cannot get the testcase inputs/outputs. Is there a way that you can share these files with me so I can test? @dmocko @shugongwang, maybe you can pull them over to discover if you have Raijin access?

Hi @bmcandr.  We're exploring another way to get the testcase.  Once I have it, I'll let you know.

Also, compile fails on discover due to the addition of the libraries -lsz -ltirpc on line 826 in Config.pl.

Hi @bmcandr I took out the libraries and will just have to keep a separate copy of the compilation script for our system unless you advise otherwise.

Hi @wendyksharples, I have updated the docs to add the configuration and model output options described in your sample config/model output files. A rendered PDF is attached here and the AWRA-L additions can be found on pages 55, 58, 141, and 190.


Replace Lidgren.Network?

Lidgren is unmaintained, seems to have some issues, and doesn't have stuff like IPv6 support.
On one side we can try to maintain it, on the other we can maybe look for a replacement. This library got linked in Lidgren's google group: https://github.com/RevenantX/LiteNetLib

Alternative point though, many people have successfully, and continue to successfully, use Lidgren in their applications without issues.

There's a PR on the lidgren repo that adds ipv6 support
The multi-client per IP thing is an implementation error, i.e. our fault.

IPv6 support in a lone PR is not something I'd trust though.

Better than re-implementing our entire netcode using something lower level than what we're using now.

There's always the option of replacement instead of rolling our own.

Looking it over it looks like litenetlib is better in the long run because:

We don't have to fix the ipv6 implementation
We don't have to fix other lingering issues
litenetlib is supposedly more efficient

@gbasood

👍 from litenetlib creator

🤔

This is obviously gonna be a really long term goal, so not something we should be working on yet.
That said it may be worth making an abstraction layer on top of Lidgren so we can swap it out more easily later if we have to (interfaces for things like the incoming and outgoing message types)

Honestly whoever does it is going to have a great time doing so since there doesn't seem to be straight parallels between any part of Lidgren we're currently using and litenetlib
Also litenetlib isn't documented, and there are no tutorials related to it and game dev, so you're going to end up learning from the code

@outbools i am collecting feedbacks. Because not all users write to me (i understand). Documentation exists (in XML) (and covers almost all features except very obvious). And after reading this thread i added .chm file.
I am tried to make my library very similar to lidgren3 in many aspects :)

Now this is some service.

The IPV6 issue was fixed in https://github.com/space-wizards/space-station-14/pull/204. There does not seem to be any actual issues with Lidgren, so unless something comes up there is no point taking the effort to replace it.


Add haptics feedback when playing chess moves

@Fouad-sys Can we close this issue ? I think it's done.


batched inference for grammatical score

I noticed that the lm_score code processes a single sentence at a time. This is pretty slow if you're processing a large amount of data. I wrote a batched version, though it's a bit ugly. This increases processing speed by about 8x on a single 3090
import torch.nn.functional as F

def get_lm_score(sentences, batch_tokens=42000):

    def score_batch(batch, tokenizer, model):
        inputs = tokenizer(batch, padding=True, truncation=True, return_tensors="pt").to(device)
        batch_scores = []

        with torch.no_grad():
            labels = inputs["input_ids"].clone()
            labels[inputs["input_ids"] == tokenizer.pad_token_id] = -100
            out = model(input_ids=inputs["input_ids"], labels=labels, attention_mask=inputs["attention_mask"], token_type_ids=inputs["token_type_ids"])
            logits = out['logits']

            for j in range(labels.shape[0]):
                loss = F.cross_entropy(logits[j].view(-1, tokenizer.vocab_size), labels[j].view(-1))
                batch_scores.append(math.exp(loss.item()))

        return batch_scores

    model_name = 'bert-base-cased'
    model = BertForMaskedLM.from_pretrained(model_name).to(device)
    model.eval()
    tokenizer = BertTokenizerFast.from_pretrained(model_name)
    lm_score = []

    # sort sentences by length for optimal padding (getting the tokens takes too long so using string length as approximation)
    sentences_flat = []
    for sent in sentences:
        for s in sent:
            sentences_flat.append((s, len(s)))

    sentences_flat.sort(key=lambda x: x[1], reverse=True)

    batches = []

    current_batch_count = 0
    current_batch = []
    for sent in sentences_flat:
        current_batch.append(sent[0])
        current_batch_count += sent[1]
        if current_batch_count > batch_tokens:
            batches.append(current_batch)
            current_batch_count = 0
            current_batch = []

    if len(current_batch) > 0:
        batches.append(current_batch)

    score_dict = {}

    for batch in tqdm(batches):
        batch_score = score_batch(batch, tokenizer, model)
        for j, sent in enumerate(batch):
            score_dict[sent] = batch_score[j]

    for sentence in sentences:
        if len(sentence) == 0:
            lm_score.append(0.0)
            continue
        score_i = 0.0
        for x in sentence:
            if x in score_dict:
                score_i += score_dict[x]
            else:
                score_i += 10000
        score_i /= len(sentence)
        lm_score.append(score_i)
    return lm_score

Thank you Jack! Can you request a pull?

ah there are two things I'm not sure about.
huggingface seems to have changed their api for the model.forward call - the above code works for transformers 4.20 (the latest one) but not the one in this repo (3.3.1) The code would have to be changed if you want to keep the current transformer version.
the batched code requires a new parameter for either a batch size or number of tokens per batch. This parameter would need to be set depending on how much vram you have. I'm not sure how you'd like to expose this option in your code.


Need help with ln -sf

There is a simple workaround for haraldF/homebrew-kf5/kf5-ktexteditor build failure
ln -s  /usr/local/share/kservicetypes5/ ~/Library/Application\ Support/
as mentioned by me here.
The problem is that I can't replay that command in brew file. I had tried both ln_sf and system "ln", "-sf",. The both command can't find corresponding target path. The problem persists when I add system "mkdir" "-p" before this command.

Homebrew doesn't support writing outside of the prefix during installation. Bugs with https://github.com/haraldF/homebrew-kf5 should be discussed there.


Code Quality: Fix flakey e2e test should be able to use existing template for new story

Context

Summary
Fix flakey e2e test and update accessibility so that it passes axe tests.
Relevant Technical Choices
n/a
To-do
n/a
User-facing changes
n/a
Testing Instructions

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

Tests should pass

Reviews
Does this PR have a security-related impact?
n/a
Does this PR change what data or activity we track or use?
n/a
Does this PR have a legal-related impact?

Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This code is covered by automated tests (unit, integration, and/or e2e) to verify it works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #9665

summary

Ignore this 🤦

Had to update the color swatch grid so that it would be accessible but look the same as before.

Before
After

Everything should be working now and accessibility is improved 👍


Evaluate altstore.io

Is your feature request related to a problem? Please describe.
Users can't install the App without compiling it for themselves using Xcode.
Describe the solution you'd like
A easier way to install the app.
Describe alternatives you've considered
None.
Additional context
Test out altstore.io and see if it can be used to install and run the app.

Hi, I'will try to help with the AltStore integration. I do have an paid Apple Development Account, so theoretically I could also try submitting the app or making TestFlight Builds for Testers.

Hi, at the current stage I think an official App Store release is not wise, mainly due to the implemented trick to keep the app running in the background.
I'd really appreciate if we can setup some routine to serve it through Altstore though.

AltStore deploy is pretty simple actually. In Xcode we can compile an .ipa file, which a user can side load via AltStore. The only problem is that the compiled .ipa file contains ones signing certificate, which in my case is from my company. I wouldn't like to release that one for AltStore deploy. I will investigative creating the .ipa file without signing.

Sounds good. What are the implications when using an own (e.g my own) certificate?
Thx in advance for helping out! I greatly appreciate it.

Using yours should be just fine. As I understand you are using a free developer account. If I’d sign the App using my distribution certificate anyone could install the app using just the .ipa (clicking a link with the file embedded would install it automatically).
The worst that could happen is that apple cancels your Developer Account for violating their ToS for publishing signed apps outside of the App Store. Just as a precaution I’d never publish a ipa file with my own (or your) signing certificate.
It shouldn’t be that difficult to sign without an certificate, I’ve done it before for different projects, but that was a couple of years ago.

This is an unsigned version of the current master branch. I've tried it using AltStore and it works for me.
Rooms.ipa.zip

So do I understand correctly, that we can now have something like a release page on Git to provide the latest pre-build *.ipa files?


Add "integration-testing" topic

Seems like an important topic is missing from the list :)

Thanks, @bsideup !


Create an RDF4J plugin JAR

Depends on: #45
We should have a way to easily enable support for Jelly in RDF4J applications, without having to recompile the thing. There should be a JAR published with all releases of Jelly (not necessarily on Maven, maybe just on GitHub) that includes all dependencies of jelly-core and jelly-rdf4j, except RDF4J itself. Then using the plugin is as simple as adding it to the classpath.
This is basically the same issue as #46, only this one is about RDF4J.

Resolved in #97


Old version of lodash?

H There,
Getting an error when I install gulp-sass. Anyway around or should I ignore it?
OSX: 10.10.5
npm 2.11.2
node: v0.12.5
gulp:  3.9.0
gulp-sass 2.1.1
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained. Upgrade to<EMAIL_ADDRESS>/node_modules/gulp-sass/node_modules/node-sass/node_modules/gaze/node_modules/globule/node_modules

We're aware of this warning. You can ignore it.

thanks @xzyfer.


New Label: franz

https://meetfranz.com/
messaging app for WhatsApp, Facebook Messenger, Slack, Telegram and others

Thank you!


Fix head bobbing exploit

Vanilla Minecraft has this exploit when you walk in a tunnel, in 3rd person. The camera bobbing allows the player to see through the roof, sort of like x-ray. I think it's reasonable to fix this glitch someday.

Less present but still occurs on CE 3.0 Snapshot 2.1
Currently, this only seems to happen on FOV above 107 based on limited testing.
It is also possible to glitch out the side camera view, I was able to see caves and ores through walls using this method.


(Monthly) Scheduled transactions do not appear to follow intended schedule

Steps to reproduce the behaviour

Create monthly scheduled transaction, set to repeat on "same day each month"
Create a 2nd monthly schedule transaction, but on a different date

Expected behaviour
I would expect the first scheduled transaction to repeat every month on specified date of transaction 1, and the second schedule transaction to occur later the following month on the different date of transaction 2.
Example:
Transaction 1 =  Aug 1, repeat on Sep 1
Transaction 2 = Aug 16, repeat on Sep 16
Actual behaviour
The actual behavior appears to be that all monthly scheduled transactions take place on Sep 1.
It also appears getting the scheduled transactions to actually run is sporadic and inconsistent; sometimes schedule transactions just never run.
Software specifications

GnuCash Android version: 2.2.1
System Android version: 6.0.1
Device type: LG Nexus 5

@rivaldi8 Will your new JobScheduler help here?

@codinguser I don't think so. Those changes were mainly for the restrictions of Android 8. I'd say it's more likely something to do with the code deciding when to run the transactions.

I can confirm this bug on

GnuCash Android version: 2.4.0
System Android version: 8.0.0
Device type: Samsung Galaxy A3


Release v3.8.0

Depends on #177, #173, #178

Hey @albertyw - I am going to close this one out, we have a tool that will do this for us once we create a new release.


Updated default values for iOS and README.

Perhaps we should double check that default values are the same for Android & iOS.

Been so long since I've worked on this but quickly looking at the android source I don't think I ever set default values. The cardview just had it's defaults, I think Walker added the default stuff for iOS to keep it close to the default android look. Not a bad idea to set defaults for both platforms as long as it's a consistent look with the values for both android and iOS (meaning that radius =2 isn't radically different on the different platforms).

It's ok, I just updated the ones for iOS, made them the same as it was in the readme file. You can updated them if you want. The PR includes the documentation for shadowRadius.


fix: stream offset overflow issue in 32-bit system

Stream offset may be transfer from uint64_t to size_t when gen stream frame. In 32bit system, size_t means unsign int. This may cause stream offset error when offset larger than 2^32.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.shiwen.lc seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add one more test case for sort command

Other than what the title states, redundant code in sort command has also been removed. Code annotations to facilitate collating has also been added.

Pull Request Test Coverage Report for Build 52

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.005%) to 91.917%

Totals

Change from base Build 44:
-0.005%

Covered Lines:
1501

Relevant Lines:
1633

💛  - Coveralls

Coverage was lowered due to removal of redundant line. The particular line of code will be called during run-time, but what was being returned was not utilised anywhere in the class file.


Specify somewhere that core.autocrlf = true is required on Windows for working on detekt itself

Expected Behavior

Cloning and running gradle build works on Windows
Observed Behavior

This test fails due to the assertion line 25 which expects the system's line ending, i.e. \r\n on Windows. However, in order to have \r\n line endings on clone, you need to set the core.autocrlf Git option to true, otherwise the test fails because the file uses \n line endings.
Steps to Reproduce

Set core.autocrlf to false or input on Windows
Clone the detekt repository
Run gradle build

Context

This makes the onboarding developers (e.g. me) confusing, as a seemingly random test fails for what appears to be no reason. This degrades the developer experience.
A fix could be to tell Windows user to use the following command in the README and CONTRIBUTING files:
git config --global core.autocrlf true

Your Environment

Version of detekt used: Main branch
Version of Gradle used (if applicable): Wrapper version from detekt (6.8.3)
Gradle scan link (add --scan option when running the gradle task): N/A
Operating System and version: Windows 10
Link to your project (if it's a public repository): N/A

I agree, we have an issue here but I'm not sure if that's the way to fix it. I think that the problem is the test. We can't suppose that the same file in windows will have other end of line. But I'm not sure how can we fix it without losing coverage...

I use Windows but I have core.autocrlf set to true, so I never saw this problem. But true is sometimes problematic for other projects I work on so asking people to change a global setting isn't the right approach.
Instead we can have OS-specific tests and/or skip this one on Windows. I'm not worried about loss of coverage since the code coverage task in CI is run on Ubuntu which wouldn't be affected.

Well, that test checks that on windows it uses one end of line and on Linux it uses other. Maybe we can copy this test and add to both of them a skip. One only runs on windows and the other runs on Unix. This way on ci we have all of them running because we run in the 3 different OS and the contributors doesn't find this issue.


FATAL ERROR: HomeSpan\src\HAP.cpp:29:10: fatal error: sodium.h: No such file or directory

I created a sketch and uploaded it! Everything worked fine! The next time I tried to upload a sketch, I got the following error:
HomeSpan\src\HAP.cpp:29:10: fatal error: sodium.h: No such file or directory
   29 | #include <sodium.h>
      |          ^~~~~~~~~~

To create and download I use Arduino IDE 2.2.1, ESP32-C3 board
MY SKETCH:
#include <HomeSpan.h>
#include "Garage1.h"

void setup() {
  Serial.begin(115200);
  homeSpan.setPairingCode("11122333");
  homeSpan.begin(Category::GarageDoorOpeners,"GarageDoors");

  new SpanAccessory();
    new Service::AccessoryInformation();
      new Characteristic::Identify();
    new BUTT_G1(12);
}

void loop() {
  homeSpan.poll();
}

Garage1.h
struct BUTT_G1 : Service::Switch {

  int pinG1;
  SpanCharacteristic *power;

  BUTT_G1(int pinG1) : Service::Switch() {

    power = new Characteristic::On();
    this->pinG1 = pinG1;
    pinMode(pinG1, OUTPUT);
  }

  boolean update() {

    digitalWrite(pinG1, power->getNewVal());
    delay(800);
    digitalWrite(pinG1, false);
    power->setVal(false);
    return(true);
  }
};

It seems that the Arduino-ESP32 library itself may be corrupted.   Are you using the Arudino IDE?  If so, recommend you delete the board manager and follow the instructions on the Arduino-ESP32 GitHub site to re-install the latest production version of the Arduino-ESP32 library (2.0.14) .  Sodium.h is part of that library.

It seems that the Arduino-ESP32 library itself may be corrupted. Are you using the Arudino IDE? If so, recommend you delete the board manager and follow the instructions on the Arduino-ESP32 GitHub site to re-install the latest production version of the Arduino-ESP32 library (2.0.14) . Sodium.h is part of that library.

I reinstalled the board library and nothing changed! Only I have the latest version 2.0.13

Everything worked out after reinstalling the Espressif ESP32 board library! Thank you


Fix permissions for jenkins-schema.sh

Jenkins can't execute this file at the moment, so the tests won't run.
https://ci.dev.publishing.service.gov.uk/job/govuk_business_support_finder_schema_tests/1/console

👍


Fix zerver.tests.test_messages.MessageDictTest.test_bulk_message_fetching failing when run alone

See 50d67d9a135a361158813710c411dc9df087daf9 for context.

@timabbott could I work on this issue ?

Yep, go for it!  The failure is likely caused by some sort of test setup work that another test does being needed by this test; I haven't looked in detail though.


Check if the capability is shared before making requests

Description
Currently we do not check whether an EthPeer supports the request that we are sending, e.g. when we are sending a GetStorageRangeFromPeerTask to a peer we should check whether that peer has the capability to do SNAP.
We need to think about at what point this should be checked. It should be done when we are selecting the peers for the retrying tasks, but we might need to do that as well for the simple tasks (e.g. GetStorageRangeFromPeerTask).

current PR https://github.com/hyperledger/besu/pull/6609

closing as duplicate of #6349


Removed USD component from SDFormat and move to gz-usd

Signed-off-by: ahcorde<EMAIL_ADDRESS>🎉 New feature
Removed USD component from SDFormat and move to gz-usd
There are some reason to move this component:

It's using the FUEL directory structure to get some models https://github.com/gazebosim/sdformat/issues/1051
External contributors asking for permission in SDFormat (easy to given then permissions in another repository)
Other converters are in a different repo https://github.com/gazebosim/gz-mujoco
This module is not yet released
Less time on CI

New location https://github.com/gazebosim/gz-usd
Summary
Checklist

[ ] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

the test/usd directory

That's still there. Remove them here and make sure the tests are also present in the new repo


Make logbook-httpclient optional

Description
Support for Apache HttpClient 4 has been removed in Spring Framework 6.0, and replaced with Apache HttpClient 5. Because of that logbook-httpclient should not be added as a default dependency to logbook-spring-boot-autoconfigure.
Motivation and Context

Addresses #1491
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

:+1:


Constify SideSet::new and add a convenience method for a nop instruction

The implementation of nop follows the c-sdk.

LGTM, with or without the nop function.

You design API, but users drive evolution of that API.
If the end-users of your system think your low-level abstraction is right for them, they will use and extend it - whether you think it's correct or not.
Instruction encoding (even pseudo-instruction) is the lowest level abstraction, and in this crate it's being handled at the highest level. Whether it's the correct place or not, it feels wrong to discard this information so early and prohibit the use of this abstraction for other tooling.
My opinion is that constifying SideSet::new() is worth doing, that the addition of NOP to the assembler seems perfectly reasonable based on evidence provided.
If it proves to add less value to the codebase than it costs (in terms of compile-time, maintenance, documentation) I will be happy to approve a PR to remove Assembler::nop at a later date.

@jannic just needs a trailing comma


Fix Curve Editor crash with null curve

Applies bitsawer's fix https://github.com/godotengine/godot/issues/78080#issuecomment-1585644608
I copied the bug from GradientTexture2DEditor, so the PR fixes the unsafety there too, although it doesn't play out anywhere at the moment.
Bugsquad edit:

Fixes #78080

Mew Pur Pur edit:

Sorry for forgetting to do the "fixes" thing, bugsquad.

Thanks!


Investigate automating lein installation with distutils

It might be possible to make lein installation an automated step using distutils. This is worth investigating because it could make setup that much easier. java would still be a requirement, but that can safely be assumed to be handled by operating system package managers. Alternatively, we could just bundle a shell script with the quickstart project that downloads & installs lein.

The conda recipe I put together in #105 will automatically install Leiningen, and I think that is probably the more appropriate way to solve this issue for now.  This also won't be a factor if/when we get rid of the Clojure part (as per #136).  In light of both of these facts, I'm closing this for now.


We were getting ugly output if no models were present on the system

Ugly output like this:
Traceback (most recent call last):
File "/usr/local/bin/ramalama", line 36, in
main(sys.argv[1:])
File "/usr/local/bin/ramalama", line 20, in main
ramalama.init_cli()
File "/usr/share/ramalama/ramalama/cli.py", line 71, in init_cli
args.func(args)
File "/usr/share/ramalama/ramalama/cli.py", line 210, in list_cli
name_width = max(len("NAME"), max(len(model["name"]) for model in models))
now it's cleaned up so it looks like:
NAME MODIFIED SIZE

LGTM
But could you add a test in the future.


Replaced alias_method_chain with Module.prepend

Summary of changes made:

Removed class ActionDispatch::Routing::RouteSet (/lib/devise/rails/routes.rb)

Created file /lib/devise/rails/route_set.rb and defined module ActionDispatch::Routing::RouteSetFinalizeWithDevise and added

ActionDispatch::Routing::RouteSet.prepend(ActionDispatch::Routing::RouteSetFinalizeWithDevise)

My bad! This change has already been taken care of in PR https://github.com/plataformatec/devise/pull/3714.
Closing this now.


Should not distribute the JSON validation code

Currently, it is compiled with the project and packaged with it. Ideally, it would not be shipped together.

Maybe it should. We might use it to validate third-party JSON as game mods in the future and fail early.


Add third button for online submission of permits

Steps to Reproduce

Next to view and download buttons, add a pencil type icon for permits that have an option for submitting online

Actual Behavior
(Explain what actually happens.)

Added new column, Online Submission, in permit table dataset.

@samanthawolfe there should be another column with the URL for the online submission right? Two new columns?


Client Events

Hello!
Is there any chance we can get client events added to this? Or if it already is added can you please explain how to send them? Trying to avoid using the official pusher python client as that needs you secret key now.
Thank you so much! :D
https://pusher.com/docs/channels/using_channels/events/#triggering-client-events

My Bad, found the trigger function on channel, thank you! :)


Upgraded react native to 0.18.1 and react redux to 4.0.6 and removed unnecessary files from source control

react native 0.18.1 is out and now depends on react from npm so we can use react redux 4+
also .gradle folder was commited to source control even though it is ignored, removed it so ignore rules would actually apply to it.

forgot to run react-native upgrade, will submit a new PR after i do that


Is there a Non-Ad supported version for pay?

Do you sell a non-ad supported license?

I would love to hear any news about a paid version. Ads are a deal breaker for me. Thanks!

me too. thx.

So there is clearly interest in a non-ad supported versions. Can anyone from the team respond as to if  there are any plans for this?

I think there is something wrong in this library! the owner update it but he don't care about any question regards to this matter!
My app need a library like this, but I don't show ads for my users, how can I use this !!

Completely Agree. There are a lot of devs who need a library like this and are willing to pay for it. Ad supported will never work for most of these devs.


How we can use components?

How we can use components? We does not have completed code, What to import before using any components?

@ayoubkhan558 it is a React library, so...

Follow the instructions here to install the library in your project
Every time that you want to use a component you need to import it:

import { Button } from 'flowbite-react';

Be happy!

@ayoubkhan558 it is a React library, so...

Follow the instructions here to install the library in your project
Every time that you want to use a component you need to import it:

import { Button } from 'flowbite-react';

Be happy!

Thank you :)
Why there is no documentation for this on https://flowbite-react.com ?
It should have complete documentation for every component how to import it from 'flowbite-react' and use it.

@ayoubkhan558
After installing Flowbite and Flowbite-react with all stated in documentation quick start
You can use https://flowbite-react.com  and here's an example:
import React from "react"
import { Sidebar } from 'flowbite-react';
import { HiChartPie, HiViewBoards, HiInbox, HiUser, HiShoppingBag, HiArrowSmRight, HiTable } from "react-icons/hi";
import { BiBuoy } from 'react-icons/bi'
import logo from './logo.png'

function App() {
  return (

    <div className="w-fit bg-gray-50">
      <Sidebar aria-label="Sidebar">
        <Sidebar.Logo
          href="#"
          img={logo}
          imgAlt="Flowbite logo"
        >
          Khasni
        </Sidebar.Logo>
        <Sidebar.Items>
          <Sidebar.ItemGroup>
            <Sidebar.Item
              href="#"
              icon={HiChartPie}
            >
              Dashboard
            </Sidebar.Item>
            <Sidebar.Collapse
              icon={HiShoppingBag}
              label="E-commerce"
            >
              <Sidebar.Item href="#">
                Products
              </Sidebar.Item>
              <Sidebar.Item href="#">
                Stores
              </Sidebar.Item>
              <Sidebar.Item href="#">
                Costomers
              </Sidebar.Item>
              <Sidebar.Item href="#">
                People
              </Sidebar.Item>
              <Sidebar.Item href="#">
                Grossists
              </Sidebar.Item>
              <Sidebar.Item href="#">

              </Sidebar.Item>
            </Sidebar.Collapse>
            <Sidebar.Item
              href="#"
              icon={HiViewBoards}
            >
              Kanban
            </Sidebar.Item>
            <Sidebar.Item
              href="#"
              icon={HiInbox}
            >
              Inbox
            </Sidebar.Item>
            <Sidebar.Item
              href="#"
              icon={HiUser}
            >
              Users
            </Sidebar.Item>
            <Sidebar.Item
              href="#"
              icon={HiShoppingBag}
            >
              Products
            </Sidebar.Item>
            <Sidebar.Item
              href="#"
              icon={HiArrowSmRight}
            >
              Sign In
            </Sidebar.Item>
            <Sidebar.Item
              href="#"
              icon={HiTable}
            >
              Sign Up
            </Sidebar.Item>
          </Sidebar.ItemGroup>
          <Sidebar.ItemGroup>
            <Sidebar.Item
              href="#"
              icon={HiChartPie}
            >
              Upgrade to Pro
            </Sidebar.Item>
            <Sidebar.Item
              href="#"
              icon={HiViewBoards}
            >
              Documentation
            </Sidebar.Item>
            <Sidebar.Item
              href="#"
              icon={BiBuoy}
            >
              Help
            </Sidebar.Item>
          </Sidebar.ItemGroup>
        </Sidebar.Items>
      </Sidebar>
    </div>

  );
}

export default App;


[test] Tweak class_getImageName to correctly handle 2018 OS betas

I messed up the condition here—targetEnvironment(device) is ignored and should be written !targetEnvironment(simulator)—but even besides that I didn't actually have the right conditions for which iOS devices this passes on and which it doesn't. Rather than trying to perfectly match this with an XFAIL, just skip the parts of the test that depend on having a new enough 2018 OS build if we, well, don't have it.
rdar://problem/42398849

@swift-ci Please smoke test and merge


EmrEtlRunner: move output directory empty checks to prior to staging step

Because otherwise this scenario can happen:

Job fails at shred step
Next job starts, moves files to staging, dies following output check
Job is manually restarted from shred step
Job succeeds, archives all files in staging, including the files from bullet point 2 above, which haven't been processed

Moving forwards because this issue is a real pill


Return underlying transaction hash of a snapshot

Users and bot developers currently cannot relate their snapshot objects returned by Mixin API to underlying transactions in the Mixin Kernels. However, knowing those undecorated data behind your somewhat centralized API service can give people more confidence about the integrity and consistency of Mixin software services as a whole.
So please return transaction hashes in /snapshots.

Good approach and the whole API should all be refactored to make the API and Kernel data more connected. Now we just added snapshot_hash and transaction_hash to the /snapshots/:ID API, this gives people some way to ensure the API is correctly based on the Kernel graph.
Those new fields should only be available for those assets enabled for Kernel graph, and they are not available until the snapshot confirmed by the Kernel graph.

Great. And I also found those new fields in /transfers/trace/:trace_ID.
However, by comparing an API snapshot with its Kernal data, I realized that memos of snapshots were not saved on-chain, or say, on-graph. But many future MTG (Mixin Trusted Group) services would probably be based on that, such as the current 4swap. Making memos or hashed memos available in their Kernal snapshots may help those services be more trusted.
So please consider saving memos on-chain.

Transfers through Mixin Messenger now encodes memo in the mainnet extra.
type MixinExtraPack struct {
	T uuid.UUID
	M string `msgpack:",omitempty"`
}

M is the memo, and T is the trace id.


systemservices/deployer: improve logs

I reverted back logging messages like in the first implementation of SS. I lowercased the log messages for convience. Added total system services number. Removed use of %q because their ' escaped during logs are printed which is hard to read. And removed some extra logs to cut the noise.

@antho1404 @krhubert can you review again?