Building Python with --enable-shared on Linux

Environment

Server - Azure Pipelines

apgomes88/apgomes88 - 20181118.3 Test_Linux Job

Agent - Hosted

Pool:Hosted Ubuntu 1604 · Agent: Hosted Agent

Issue Description
Hi, people.
I'm building a multi OS desktop application with Python using pyinstaller. Pyinstaller requires building Python source code using the param --enable-shared (or, --enable-framework on Darwin) in order to access all its dependencies. On MacOS and Windows jobs is working fine but I'm having trouble with Linux build. I ran into the same problem with MacOS locally and I solved it downloading Python source code and doing:
./configure --enable-framework
make
make install

Now I have no idea if this is possible here. Can you help me with it?
- job: 'Test_Linux'
  timeoutInMinutes: 5
  pool:
    vmImage: 'Ubuntu 16.04'

  steps:
  - task: UsePythonVersion@0
    inputs:
      versionSpec: 3.7
      architecture: 'x64'

  - script: |
      sudo apt-get update -qy
      sudo apt-get install -y python3-dev python-dev python-pip
      sudo apt-get install -y espeak
      sudo apt-get install xvfb
      python -m pip install --upgrade pip
      pip install -r requirements.txt
      pip install -r requirements_test.txt
    displayName: 'Install prerequisites'

  - script: |
      sudo cp xvfb.init /etc/init.d/xvfb
      sudo chmod +x /etc/init.d/xvfb
      sudo update-rc.d xvfb defaults
      sudo service xvfb start
    displayName: 'Start X Virtual Frame Buffer'

  - script: |
      export DISPLAY=:10
      pytest
    condition: succeededOrFailed()
    displayName: 'Run tests'

  - script: |
      flake8 .
    displayName: 'Run lint tests'

  - script: |
      pyinstaller --clean pardal.spec
    displayName: 'Build'

Task logs
log_24_38.zip
Error logs
2018-11-18T13:36:55.5615673Z 15521 INFO: Python library not in binary dependencies. Doing additional searching...
2018-11-18T13:36:55.5992515Z Traceback (most recent call last):
2018-11-18T13:36:55.5993616Z   File "/opt/hostedtoolcache/Python/3.7.0/x64/bin/pyinstaller", line 11, in <module>
2018-11-18T13:36:55.5994918Z     load_entry_point('PyInstaller==3.4', 'console_scripts', 'pyinstaller')()
2018-11-18T13:36:55.5995834Z   File "/opt/hostedtoolcache/Python/3.7.0/x64/lib/python3.7/site-packages/PyInstaller/__main__.py", line 111, in run
2018-11-18T13:36:55.5996335Z     run_build(pyi_config, spec_file, **vars(args))
2018-11-18T13:36:55.5997468Z   File "/opt/hostedtoolcache/Python/3.7.0/x64/lib/python3.7/site-packages/PyInstaller/__main__.py", line 63, in run_build
2018-11-18T13:36:55.5998050Z     PyInstaller.building.build_main.main(pyi_config, spec_file, **kwargs)
2018-11-18T13:36:55.5998822Z   File "/opt/hostedtoolcache/Python/3.7.0/x64/lib/python3.7/site-packages/PyInstaller/building/build_main.py", line 838, in main
2018-11-18T13:36:55.6000016Z     build(specfile, kw.get('distpath'), kw.get('workpath'), kw.get('clean_build'))
2018-11-18T13:36:55.6000937Z   File "/opt/hostedtoolcache/Python/3.7.0/x64/lib/python3.7/site-packages/PyInstaller/building/build_main.py", line 784, in build
2018-11-18T13:36:55.6001464Z     exec(text, spec_namespace)
2018-11-18T13:36:55.6001828Z   File "<string>", line 25, in <module>
2018-11-18T13:36:55.6002536Z   File "/opt/hostedtoolcache/Python/3.7.0/x64/lib/python3.7/site-packages/PyInstaller/building/build_main.py", line 241, in __init__
2018-11-18T13:36:55.6003020Z     self.__postinit__()
2018-11-18T13:36:55.6003766Z   File "/opt/hostedtoolcache/Python/3.7.0/x64/lib/python3.7/site-packages/PyInstaller/building/datastruct.py", line 158, in __postinit__
2018-11-18T13:36:55.6004231Z     self.assemble()
2018-11-18T13:36:55.6004937Z   File "/opt/hostedtoolcache/Python/3.7.0/x64/lib/python3.7/site-packages/PyInstaller/building/build_main.py", line 573, in assemble
2018-11-18T13:36:55.6005499Z     self._check_python_library(self.binaries)
2018-11-18T13:36:55.6006310Z   File "/opt/hostedtoolcache/Python/3.7.0/x64/lib/python3.7/site-packages/PyInstaller/building/build_main.py", line 676, in _check_python_library
2018-11-18T13:36:55.6006783Z     raise IOError(msg)
2018-11-18T13:36:55.6007552Z OSError: Python library not found: libpython3.7.so.1.0, libpython3.7mu.so.1.0, libpython3.7m.so.1.0
2018-11-18T13:36:55.6008346Z This would mean your Python installation doesn't come with proper library files.
2018-11-18T13:36:55.6008839Z This usually happens by missing development package, or unsuitable build parameters of Python installation.
2018-11-18T13:36:55.6009120Z
2018-11-18T13:36:55.6009488Z * On Debian/Ubuntu, you would need to install Python development packages
2018-11-18T13:36:55.6010124Z   * apt-get install python3-dev
2018-11-18T13:36:55.6010842Z   * apt-get install python-dev
2018-11-18T13:36:55.6011662Z * If you're building Python by yourself, please rebuild your Python with `--enable-shared` (or, `--enable-framework` on Darwin)
2018-11-18T13:36:55.6012110Z
2018-11-18T13:36:55.6831329Z ##[error]Bash exited with code '1'.

Thanks for your feedback!  You are correct, we need to configure the Python installations on this image with this flag.  We have an issue tracking this in our image generation repo: Microsoft/azure-pipelines-image-generation#255.
I will try to get to this soon.  We will have to roll out a new version of the hosted image before you see the changes, which happens about once per month.

Closing as a duplicate -- please subscribe to the issue above to track.

Is the PR you referenced the correct one? It looks unrelated.
I have a similar issue with not being able to find the Python libraries, but I'm using cx_Freeze instead of pyInstaller. I think a workaround/solution would be to add the libraries to LDPATH on macOS and Linux, but I'm not too thrilled about doing that on my own pipeline since UsePythonVersion should set that up.


The IdleDispatchedHandlerArgs.IsDispatcherIdle property usage is unclear

if the IdleDispatchedHandler delegate returned by the callback has the IsDispatcherIdle property set to false, the work item immediately exits background processing until the event queue is empty again.

This sentence seems to be erroneous and confusing in these ways:

The IdleDispatchedHandler delegate is not returned by the callback. The callback is the IdleDispatchedHandler delegate.
I think this means to say that the IdleDispatchedHandlerArgs sent to the IdleDispatchedHandler delegate
By "the work item immediately exits", is this telling me that my callback should exit processing? If IsDispatcherIdle is false, then why was my callback even called? I thought it only should be called when the CoreDispatcher is Idle.

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

ID: cf4cafd3-1bf0-d95c-3ec8-92eb9c6ef6f1
Version Independent ID: 8b869f1c-e1e5-1740-bdd0-159464b26de0
Content: IdleDispatchedHandlerArgs.IsDispatcherIdle Property (Windows.UI.Core) - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.ui.core.idledispatchedhandlerargs.yml&version=GBlive&_a=contents
Product: windows
Git Login: wdg-dev-content
Microsoft Alias: wdg-dev-content

Apologies for the lengthy delay on this item.
1: You're right that the wording on this description is incorrect. I've updated it, and the changes should be live later today.
2: I believe that the text here is an accurate description of the behavior - if the dispatcher is not idle, the item will not process until the queue is empty (though this might be a back-up in case of unintended behavior?). Regardless, I've changed "exits" to "suspends," as I believe it gives the better impression here. Thank you.


feat: Allow unauthenticated feeds

This adds support for full application-enforced authentication, with optional IAP headers. Some UX will not be great yet (e.g. the 401 page is useless).
It adds tests for all existing routes that require auth, as well as pages requiring admin.
Pages which are allowed without auth are:

/about
/feed/$feedId.xml
/image-uploads/$

Contains a set of test utilities to easily assert on auth requirements, as well as helpers to mock identities.

This likely also fixes avatars in emails due to the auth change


Update appveyor.yml

Hi guibranco,
just trying to fix issue #29, please check it out

:x: Build pagarme-sdk-dotnet 1.0.120 failed (commit https://github.com/guibranco/pagarme-sdk-dotnet/commit/db00751937 by @viktoriussuwandi)

Hi @guibranco,
How about this PR ?
Should it close without merge, or anything need to add/improve ?

@viktoriussuwandi I am updating the files, just wait. I tracked all Prs that you submitted, just wait, no need to push a comment on every single one.

:white_check_mark: Build pagarme-sdk-dotnet 1.0.126 completed (commit https://github.com/guibranco/pagarme-sdk-dotnet/commit/d149a3a179 by @guibranco)

Thanks @guibranco !


Translate in the controller

How are you supposed to use the Translator inside the controller, i.e. when you are returning a JsonModel and want to translate error messages? Should there be a controller plugin, or is there another approach that should be taken here?

Thanks! its working
$translator = $this->getServiceLocator()->get('Translator');
$translator ->translate("your text");


Blog Art: Professional Services 24/02/05

Related Issue
No response
Description
Title of Article: Should You Invest in Professional Services for Your Node-RED Development?
Considerations:

Where will the image be displayed/shared, product? blog post? social media? presentation? website?

Inspirations:

Hi @gdziuba I just added it to the PR

Thank you @Yndira-FlowForge.


Unable to enter input for ibmcloud login

Related issue - when the spinner is stuck in this position, I'm unable to kill it. I tried all the using Cmd/Ctrl C inputs

thanks for the issue. indeed, the bash-like plugin does not yet support stdin communication. we should have this support soon.


Copy binary from upstream image

@ jedisct1 has a Docker image for minisign.
This way, the final image could be cached, most certainly outperforming building it every time.

Ooo, thanks @bnoctis!


PubSub does not resubscribe to channel on reconnect

If Redis goes down, Redix will try to reconnect with a backoff strategy. However after successful reconnect PubSub subscriptions are not restored, neither :connect message received.

What version of Redix are you on? Can you provide a reproducible example?

Redix v0.9.3
The simplest example I can think of: GenServer subscribes to a channel.
Steps to reproduce:

Spawn a GenServer, and subscribe to PubSub channel
Turn off Redis
Redix will print ERROR messages about connectivity issue, and try to reconnect with a backoff delay
Turn on Redis
Redix reconnects, but GenServer does not resubscribe to a channel

I'm not sure resubscription is expected behaviour in this case on after reconnect. However I would expect at least :connect message, so I can handle subscription manually.
What do you think?
P.S. You can use this example project from another issue: https://github.com/paveltyk/pub-sub-issue-132

@paveltyk according to this test https://github.com/whatyouhide/redix/blob/7fa54612414a13e2cd3dd37da3dbe000d94b20e4/test/redix/pubsub_test.exs#L203-L220, if the connection drops and then is established again, all the pids that were subscribed are resubscribed. If you encountered this problem, could you please write up a test that makes sure this is easy to reproduce? Thanks!

@whatyouhide Will that gist work? I've checked with v0.9.3
https://gist.github.com/paveltyk/d46b589a8bca600fb6f8399ce0023370

@paveltyk do you mind giving master a try for this? Should be fixed.

@whatyouhide yes. Now it works. Thank you.


Track node role and next in line

Each topic process needs to know who to replicate their writes to and each server process needs to know the role it has when receiving requests for reads or writes.

#27 added the knowledge of who to replicate to in topics/chains, but roles still need to be enforced.

fixed in #29


Add new stable version for ARE

Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.

Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.

Please fill this.
The first commit should be an exact copy of the files of the previous API version in the folder of the new API version. Any changes should be made in the following commits. This way reviewer can compare what changed in this new API version.
Please fix all the required failing checks + Swagger Avocado failing check too .


Replace jQuery document load event handler with javascript native version

Motivation and Context

Replaces all .ready() jQuery method calls with an event listener on document handling the DOMContentLoaded event

Your Changes

Description:

Replaced all .ready() jQuery method calls with an event listener on document handling the DOMContentLoaded event or just executes the code that used to exist in the callback of the .ready() method calls (depending on the context)

Type of change (select all that apply):

[x] Refactoring (internal change to codebase, without changing functionality)

Testing

Went through each individual context and tested that the behaviour was unchanged

Questions and Comments (if applicable)

Checklist

[x] I have performed a self-review of my own code.
[x] I have verified that the pre-commit.ci checks have passed.
[ ] I have verified that the CI tests have passed.
[ ] I have reviewed the test coverage changes reported on Coveralls.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
36985

Relevant Lines:
39836

💛  - Coveralls


kernel: enable OverlayFS

Motivation for this change
It would be nice to support OverlayFS out-of-the-box.
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

See: https://www.kernel.org/doc/Documentation/filesystems/overlayfs.txt
Partial fix for #15879

Note to self: we might also want to enable this patch by default: https://github.com/NixOS/nixpkgs/commit/84c0098117e329ef370153a1ecaa19ee72f336b5

We might also want to take a look at https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1478578 (patch).

Note CVE-2015-1328:

Philip Pettersson discovered a privilege escalation when using overlayfs
mounts inside of user namespaces. A local user could exploit this flaw to
gain administrative privileges on the system.

Not sure if all of our kernels are new enough to be unaffected. Need to check that out before we enable overlay mounts in unprivileged containers.

@cstrahan overlayfs is already available, in fact it's being used with dockerTools. Why do you want it builtin instead of installed as module?

@lethalman

@cstrahan overlayfs is already available, in fact it's being used with dockerTools. Why do you want it builtin instead of installed as module?

Oh, oops. I checked cat /proc/filesystems and didn't see "overlay" listed, so I figured I needed to configure the kernel to support it. Hmm... I'm curious:

Did I just miss it? Or
Must one first modprobe overlay? Or
Does mount -t overlay ... somehow automatically load the overlay kernel module?

As a side note, I suppose I could do something like this to see what fs kernel modules are available:
$ ls $(nix-build -A linuxPackages.kernel)/lib/modules/*/kernel/fs/**/*.ko.xz | grep overlay
/nix/store/knbclla3rfx9w4m8k3l9kdipwg0qhkqb-linux-4.4.11/lib/modules/4.4.11/kernel/fs/overlayfs/overlay.ko.xz

My guess is that mount causes the appropriate kernel module to be loaded. I don't know if mount does that itself (perhaps by looking for a <FSTYPE>.ko ), or if the kernel does that in the respective syscall.

Mount requires a certain filesystem type, and the kernel calls modprobe to load it.

This can be closed, right?


Error: Removed API method: PDFDocumentLoadingTask.then, use the promise getter instead.

error with pdfjs-dist version

Doesn't seem to be pdfjs-dist, it worked for me on 2.x before the latest updates.
Code also looks ok, it returns loadingTask.promise as expected for 2.x at
https://github.com/FranckFreiburger/vue-pdf/blob/master/src/pdfjsWrapper.js#L362

@Davidbr17, can you provide us a stack trace ?

app.js?id=48141a9ecccaa7c1f0fa:110200
Error: Removed API method: PDFDocumentLoadingTask.then, use the promise getter instead.
at PDFDocumentLoadingTask.then (11.2acad0fb94df56ba92b0.js:11179)
at VueComponent.pdfRender (34.ad82936246c758c265ed.js:1046)
at VueComponent.handler (34.ad82936246c758c265ed.js:975)
at Watcher.run (app.js?id=48141a9ecccaa7c1f0fa:112870)
at flushSchedulerQueue (app.js?id=48141a9ecccaa7c1f0fa:112614)
at Array. (app.js?id=48141a9ecccaa7c1f0fa:110292)
at flushCallbacks (app.js?id=48141a9ecccaa7c1f0fa:110218)
logError @ app.js?id=48141a9ecccaa7c1f0fa:110200
globalHandleError @ app.js?id=48141a9ecccaa7c1f0fa:110191
handleError @ app.js?id=48141a9ecccaa7c1f0fa:110151
run @ app.js?id=48141a9ecccaa7c1f0fa:112872
flushSchedulerQueue @ app.js?id=48141a9ecccaa7c1f0fa:112614
(anonymous) @ app.js?id=48141a9ecccaa7c1f0fa:110292
flushCallbacks @ app.js?id=48141a9ecccaa7c1f0fa:110218
Promise.then (async)
timerFunc @ app.js?id=48141a9ecccaa7c1f0fa:110245
nextTick @ app.js?id=48141a9ecccaa7c1f0fa:110302
queueWatcher @ app.js?id=48141a9ecccaa7c1f0fa:112706
update @ app.js?id=48141a9ecccaa7c1f0fa:112846
notify @ app.js?id=48141a9ecccaa7c1f0fa:109048
reactiveSetter @ app.js?id=48141a9ecccaa7c1f0fa:109373
proxySetter @ app.js?id=48141a9ecccaa7c1f0fa:112933
click @ 34.ad82936246c758c265ed.js:1616
invokeWithErrorHandling @ app.js?id=48141a9ecccaa7c1f0fa:110166
invoker @ app.js?id=48141a9ecccaa7c1f0fa:110487
original._wrapper @ app.js?id=48141a9ecccaa7c1f0fa:115850

When you use vuePdfNoSssNoWorker.vue or vuePdfNoSss.vue you will find this error. Because it requires pdfjs-dist/build/pdf.js which will throw the error. Once the file does the loadDocument job, it will throw the error

@FranckFreiburger any updates on this issue?

any updates on this issue? This is suddenly breaking a production site.

Seems like changing pdf.createLoadingTask(uri) to pdf.createLoadingTask(uri).promise solves the issue in my case


Update extensions and logo for one of our tokens

This is an update to a coin I had added myself in the past.

@keone would it be possible to merge this update to one of our tokens?

added in https://github.com/solana-labs/token-list/commit/fb09a6b2113b42aca4c2f51b9f02b91becc091b8


Statsbomb event parser

WIP

if this gets merged, please provide me with a paypal I will buy you a beer for that feature

if this gets merged, please provide me with a paypal I will buy you a beer for that feature

All credits to @DaanGro!
My currency is in GitHub stars :wink:
It would also be of great help if you could open some issues when you face bugs using StatsBomb!

if this gets merged, please provide me with a paypal I will buy you a beer for that feature

Haha cheers! No need of course ;)


barcharts crash on zoom with a lot of data

thanks for this great library port!
When a set a lot of data in barcharts and apply zoom in/out the app stop with the error below. im using the demo app in my iOS device iPhone 7.
stack trace in xcode:
2019-09-30 23:41:26.337733-0300 Runner[1930:141345] flutter: Another exception was thrown: Unsupported operation: Result of truncating division is Infinity or NaN

Could you tell me more details, which barchart and  how many data , i'll have a try


Omnioutliner typo

Missing quote in "url" line.

Something went very wrong, there. Please try again.


Learning Content: Just Add Water

This tutorial is part of the Building Realistic Worlds tutorial section. It should cover how the watermask works, and how to work with the Water plugin for Unreal to create ponds, lakes, etc.
Primary objectives:

Brief explanation of watermask
How to use the watermask
When to use the watermask vs hand-placed water
How to set up water plugin
Using water plugin - for example, place a lake over an existing lake

This is touched on in the FAQ. I don't feel like we need to make a whole new tutorial for what remains. We can re-open this issue if anyone disagrees.


Remove the automatic link to Shepherd test results

Bikeshed currently adds an automatic test widget to CSSWG specs, both on drafts.csswg.org and on W3C /TR. This used to be very helpful, giving an implementation report broken out by spec section.
Given that:

The tests that it reports on are now missing more than 2 years of additions, corrections and updates
The reports rely on manual test running by an actual person, which I previously saw as a good use of my time and now don't because running a known-incorrect test has no value
The tests and the reports give an inaccurate and very negative view of the state of implementation
There is no prospect of the Shepherd test situation improving (the recent "month of work" to stop the drafts server falling over all the time did not include any work on the test area)
The presence of the widget encourages people to run tests, and they will then draw inaccurate conclusions or even change implementations "to pass" known incorrect tests.

Then adding this link to all CSSWG specs has negative value and I am asking that Bikeshed remove (comment out code that adds) this widget. If at some future time it starts working again it can always be added back.
6237 well maintained tests? Or 118 out of date ones?

Related:

https://github.com/tabatkins/bikeshed/issues/2216

Agreed, done. I'll cut a new release in a little bit.

\0/


Adding Selenium Code Base

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@vladbailescu @tyge68 @absarasw can you please help review this pr.
thanks,


[Long runtime tests] Link the Objective-C runtime.

The long-running runtime tests depend on the Objective-C runtime on
Darwin, so link it directly rather than relying on various uses of
__builtin_available to link CoreFoundation for us.

@swift-ci please test


On the user consent page, clicking the cancel button produces a 500 error code

Describe the bug
In the project example, click Cancel on the user consent page, return to the previous page, and click the authorization code  button again, reporting error code 500.
message-client error message：
org.springframework.web.reactive.function.client.WebClientResponseException$Forbidden: 403 Forbidden from GET http://<IP_ADDRESS>:8090/messages
	at org.springframework.web.reactive.function.client.WebClientResponseException.create(WebClientResponseException.java:313) ~[spring-webflux-6.0.0-RC4.jar:6.0.0-RC4]
	Suppressed: reactor.core.publisher.FluxOnAssembly$OnAssemblyException:
Error has been observed at the following site(s):
	*__checkpoint ⇢ 403 FORBIDDEN from GET http://<IP_ADDRESS>:8090/messages [DefaultWebClient]
Original Stack Trace:
		at org.springframework.web.reactive.function.client.WebClientResponseException.create(WebClientResponseException.java:313) ~[spring-webflux-6.0.0-RC4.jar:6.0.0-RC4]
		at org.springframework.web.reactive.function.client.DefaultClientResponse.lambda$createException$1(DefaultClientResponse.java:213) ~[spring-webflux-6.0.0-RC4.jar:6.0.0-RC4]
		at reactor.core.publisher.FluxMap$MapSubscriber.onNext(FluxMap.java:106) ~[reactor-core-3.5.0.jar:3.5.0]

Expected behavior
When the user clicks the cancel button it should return to the authorization page without the 500 error code. The user can then reselect the Authorization Code or Client Credentials.
To Reproduce
When i use the custom-consent-authorizationserver example, i can solve this problem by updating the  consent.html
from onclick="cancelConsent(); to onclick="javascript:history.back(-1);".
.................
  <button class="btn btn-link regular" type="button" id="cancel-consent" onclick="javascript:history.back(-1);">
                          Cancel
  </button>
 .................

But when users don't want to customize the page, it's not clear how to solve the problem.

Thanks for getting in touch, but it feels like this is a question that would be better suited to Stack Overflow. We prefer to use GitHub issues only for bugs and enhancements. Feel free to update this issue with a link to the re-posted question (so that other people can find it) or add a minimal sample that reproduces this issue if you feel this is a genuine bug.
The sample is working as expected. Place a breakout in AuthorizationController.authorizationCodeGrant() and then inspect the JWT claims via authorizedClient.getAccessToken().getTokenValue() and you will see the claims:
{
  "sub": "user1",
  "aud": "messaging-client",
  "nbf":<PHONE_NUMBER>,
  "iss": "http://localhost:9000",
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>
}

Notice there is no scope claim because of the canceling of the authorization consent.
The access token is then passed to the resource server and triggers the 403 error.


group by column_name in accepted_values

resolves #3905
Description
Group by index is not ANSI SQL and not supported in every database engine (e.g. MS SQL). Use group by column_name in shared code.
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

Closing and re-opening to trigger adapter integration tests. Assuming they all pass, we can sneak this in for v0.21.0-rc1.

@sdebruyn Looks like I don't have permission to resolve changelog conflicts by pushing to your fork. If you can manage to do that in the next hour, we might just be able to get this in under the v0.21 wire. If not, this will have to wait for a future release.

@jtcohen6 resolved :)

@jtcohen6 any chance we could patch this change into v0.20.0? Without this change, the accepted_values test isn't usable for dbt-sqlserver~=0.20.0, and we're currently blocked from upgrading to v0.21.0 due to https://github.com/dbt-msft/dbt-sqlserver/issues/164


[WIP] Archivist reports progress

👩🏫

Part of: https://github.com/wellcometrust/platform/issues/2634


3.6 - Ensure Firewall is Configured to log

line 1703
If organizational score is 1 or true, check status of client
if [[ "${auditResult}" == "1" ]]; then
if [[ "$osVersion" == "12."* ]]; then
I'm not sure how to add support for OSVersion 13.  I'd like to have an addition rather than a replacement 12 for 13.

if [[ "${auditResult}" == "1" ]]; then
if [[ "$osVersion" == "12."* ]] || [[ "$osVersion" == "13."* ]]; then
I modified and it seem still work in Ventura


Updated validations.py python script. closes #1

updated validations.py python script.
Fixed the behavior of validate_user function in validations.py
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

@googlebot I signed it!


Merge to publish

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.github/ISSUE_TEMPLATE/documentation-issue.md
:white_check_mark:Succeeded

docs/rest/terms-of-use.md
:white_check_mark:Succeeded

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


chore(ci): optimize bench base

This could take quite a long time to push before. Try just the binaries.  Some CI improvements.

Benchmark results
Metrics with a significant change:

note_successful_decrypting_time_in_ms (32): 593 (+18%)
note_trial_decrypting_time_in_ms (64): 58.6 (+95%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
676
676
676

l1_rollup_calldata_gas
6,412
6,424
6,412

l1_rollup_execution_gas
585,745
585,757
585,745

l2_block_processing_time_in_ms
1,337 (+1%)
4,709 (-1%)
9,252 (-3%)

note_successful_decrypting_time_in_ms
225 (-8%)
:warning: 593 (+18%)
939 (+3%)

note_trial_decrypting_time_in_ms
27.7 (-37%)
101 (-10%)
:warning: 58.6 (+95%)

l2_block_building_time_in_ms
10,400
36,990 (-1%)
73,844 (-1%)

l2_block_rollup_simulation_time_in_ms
10,283
36,594 (-1%)
73,081 (-1%)

l2_block_public_tx_process_time_in_ms
4,902
19,108 (-1%)
38,321 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,580
26,803 (-3%)

note_history_successful_decrypting_time_in_ms
1,149 (+3%)
2,230

note_history_trial_decrypting_time_in_ms
41.9 (-4%)
77.3 (-1%)

node_database_size_in_bytes
18,624,592
34,996,304

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
145 (+1%)
44,281
26,164

private-kernel-inner
152
71,648
26,164

private-kernel-ordering
111 (-1%)
50,830
39,026

base-parity
3,983 (+3%)
128
311

root-parity
1,046 (-7%)
1,244
311

base-rollup
13,458
110,464
861

root-rollup
49.4 (-1%)
4,359
725

public-kernel-app-logic
79.6
47,599
40,661

public-kernel-tail
112 (-1%)
53,372
7,125

merge-rollup
10.5 (-1%)
2,568
861

public-kernel-teardown
76.7 (+4%)
47,599
40,661

public-kernel-setup
75.4 (-1%)
47,599
40,661

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.1 (+2%)
16.0 (-2%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.589 (+1%)
0.495 (-2%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
46.1
72.7 (+1%)
232
446
881 (+1%)
1,734
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.473
0.448 (+1%)
0.421
0.417
0.417 (+1%)
0.414
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
54.7
107
337
670 (+1%)
1,314
2,611 (-1%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
105
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.483
0.482
0.458
0.461 (+1%)
0.454
0.453 (-1%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.4

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.536

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
40,372
496,534

Transaction size based on fee payment method

Metric
native fee payment method
fpc_public fee payment method
fpc_private fee payment method

tx_with_fee_size_in_bytes
905
1,161
1,377

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
1,184 (+3%)
819
3,997 (+4%)

Metric
1 public data writes
2 public data writes
3 public data writes
4 public data writes
5 public data writes
8 public data writes

tx_sequencer_processing_time_ms
420 (-1%)
289 (+5%)
740 (+4%)
367 (+4%)
1,210 (-1%)
430 (+3%)


Minor simplification of ApproveChaincodeDefinition logic

Type of change

Improvement (improvement to code, performance, etc)

Description
Minor simplification to streamline the code path.
Related issues
https://jira.hyperledger.org/browse/FAB-17441

/ci-run


TCA-958 sync fcc -> dev

Merge latest from freeCodeCamp:main into our dev branch.

Yes, just put this up, but I'll wait with the merge.


feat: support css-modules select-text parse and codegen

feat: support css-modules select-text parse and codegen
Description:
css-modules include  ":local and :global" features  is important for modern front-end project. css-loader actually used
"post-css", now I ported the feature and tests with swc, this is a hard work for review. Thanks for your work, hope get feedback :)
BREAKING CHANGE:
nothing
Related issue (if exists):

 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.zhushijie.jayson 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.

Implemented, look at swc_css_modules crate and the css_modules option for parser, anyway thank you for the PR and sorry for delay


Swift 2.0 Conversion

Bare minimal to compile in swift 2.0 and remove and associated warnings
from var to let as per the new code analyzer.
Disregard if you've done this and decline it, but I needed this for myself and wanted the chance to see if it helps you.

Thanks for sharing mitchins! Merging!

How do I update this for cocoa pods?

You can add a new version in the pod spec file, and then tag or branch that version in GIT.
—
Sent from Mailbox
On Fri, Sep 25, 2015 at 12:12 PM, Kyle Emmanuel<EMAIL_ADDRESS>wrote:

How do I update this for cocoa pods?
Reply to this email directly or view it on GitHub:
https://github.com/ortuman/SwiftForms/pull/69#issuecomment-143103866

Hi Mitchell
I’ve just uploaded the new version to the pods repository. Tell me if you have any other problem trying to integrate it. And thanks for the feedback. Appreciate it.
--
Miguel Ángel
Sent with Airmail
En 24 de septiembre de 2015 en 22:01:37, Mitchell Currie<EMAIL_ADDRESS>escrito:
You can add a new version in the pod spec file, and then tag or branch that version in GIT.
—
Sent from Mailbox
On Fri, Sep 25, 2015 at 12:12 PM, Kyle Emmanuel<EMAIL_ADDRESS>wrote:

How do I update this for cocoa pods?
Reply to this email directly or view it on GitHub:
https://github.com/ortuman/SwiftForms/pull/69#issuecomment-143103866
—
Reply to this email directly or view it on GitHub.


Add AcquisitionDuration in JSON metadata for run1

A reviewer of the submitted paper is asking to compute CNR per unit time, which requires knowing the volume acquisition duration. This is a valid JSON field, and I suggest to add it for all subjects in this repository. Note that only 'run1' needs to be updated because the processing code will only look at 'run1' to fetch that information.

@jcohenadad I found some info here: https://www.ncbi.nlm.nih.gov/pmc/articles/PMC3045184/
I checked SCT, but it doesn't seem like we have this feature yet.
Are we supposed to compute the CNR ? Should we do it like the paper (this was done in brain)?
What is the deadline ?

I checked SCT, but it doesn't seem like we have this feature yet.

right, we don't need that feature in SCT (too specific)-- the code is already there

Are we supposed to compute the CNR ?

yes but this is unrelated to this issue-- this issue is just about adding AcquisitionDuration

What is the deadline ?

ASAP (this should take you 10min to add this field for all subjects)-- thanks!


Feature Request: Downloading Features as GeoJSON or OSM

Hi,
I'm a big fan of waterwaymap.org and the way it visualizes natural waterways and downhills. It's incredibly useful!
I was wondering if it would be possible to add a feature that allows users to download the displayed features (rivers, streams, etc.) in either GeoJSON or OSM format. This would be incredibly helpful for integrating the data with other GIS tools and workflows.
Thanks for your time and consideration, and for creating such a valuable resource.

I'm glad you like it! I'd love to help people make maps with this sort of data. All the scripts to make it are in the Makefile which is called from make-planet.sh. It's osm-lump-ways which actually generates the data.
I'm not able to provide an interactive “Download Current View” like you ask for. WWM.org is a static website that displays maps from a statically hosted .pmtiles file. The vector tile data has been processed and a little simplified.
One issue is that the data files are very large! Before it gets converted to vector tiles each map is an ca. 20 GiB GeoJSONSeq file.
But! Here's a first attempt: https://data.waterwaymap.org/planet-upstreams.csv.zst (data updated daily, and open data under the ODbL, contains OSM data)
it's a 6.5 GiB zstd compressed CSV. Uncompressed 52 GiB. 3 columns: end_nid (integer), from_upstream_m (float) & geom (text). The geom has a 2 point linestring in WKT. It's a line from point A to point B. There's 4½ billion of them. from_upstream_m  is the total upstream in point B. end_nid is the OSM node that this eventually flows into.
ogr2ogr can convert this to another format (don't forget -oo GEOM_POSSIBLE_NAMES=geom!). You can also do geo filtering. e.g. here's how I get a nice 233 MiB file for Ireland (in 25 min)
ogr2ogr ie-upstream.gpkg planet-upstreams.csv -oo GEOM_POSSIBLE_NAMES=geom -clipsrc -10.546875 51.158677 -5.350342 55.509971

I can make maps of this in QGIs, like this:

or

CSV is not practical to load in (e.g.) QGis, however it compresses really well. I'm open to other suggested file formats.
What do you (& others) think of this file? Does this data file work for you?

I just tried:
mdione@ioniq:~/src/projects/elevation/data/amapanda$ wget https://data.waterwaymap.org/planet-upstreams.csv.zst
--2024-12-21 22:43:46--  https://data.waterwaymap.org/planet-upstreams.csv.zst
Resolving data.waterwaymap.org (data.waterwaymap.org)... <IP_ADDRESS>, <IP_ADDRESS>, 2606:4700:3033::ac43:9693, ...
Connecting to data.waterwaymap.org (data.waterwaymap.org)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 500 Internal Server Error
2024-12-21 22:43:51 ERROR 500: Internal Server Error.

Works now...

Weird, FYI it's hosted on Cloudflare's R2 cloud storage (like all of data.waterwaymap.org files). I don't have direct access to that web server, but I'd trust Cloudflare. Otherwise I can't tell you what's up. Maybe just try again?
On So, 22 Dez 2024 10:42 +01:00, Marcos Dione @.***> wrote:

Works now...
—
Reply to this email directly, view it on GitHub
https://github.com/amandasaurus/waterwaymap.org/issues/74#issuecomment-2558395136,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAAMC2MFLRENRNX3V7VC2T2G2CRDAVCNFSM6AAAAABTVRFBK6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJYGM4TKMJTGY.
You are receiving this because you commented.Message ID:
@.***>

--
Amanda


About the Full-dynamic-range capacity of the structural mask

Hi~ @pwangcs
Thanks for sharing your wonderful work! I have some question about the dynamic range issue in video CS.
As mentioned in the paper (Sec3.2--Full dynamic range (FDR)), the decreased dynamic range  in video CS is origined from the fact that multiple frames  are integrated into a single measurement. So the equivalant dynamic range for each of the reconstructed frame would be several times lower than the sensor's physical dynamic range.  I agree on this point.
However, I think, even using the proposed 'structural mask', this problem still exists —— each measure is sill the composite of multiple original frames, and thus the equivalant dynamic range for the reconstructed frames will still decreased. The structrual mask's feature——the summation of the temporal weight equals to 1 —— can only guarantee that different pixels have the same exposure time thus making the measurement looks more smooth and will not generate local satureatation.  But I cannot figure out why it can increase the dynamic range.
The paper mentioned that It means that the brightness range of measurement is equal to that of each of video frames regardless of B. Therefore, the proposed structural mask keep the dynamic range of video SCI in line with that of the used sensors, i.e., full dynamic range (FDR).. Emmm, this conclusion seems not so logical, as what matters is the relation between the measure's physical dynamic range and the reconstructed video's equivalent dynamic range.  Actually, the dynamic range is physically determined by the full-well capacity of the pixel circuits. As a whole, video CS decouples multiple frames from a single real-captured measurement. It seems that the decreased dynamic range for the reconstructed videos is a physical defect, which can hardly be overcome by mask design.
Could you give some explanation on this issue? Thanks!

Hi, dawnlh, pixel's physical bit depth directly determines the dynamic range of captured measurement rather than reconstructed video. Using coded aperture/exposure (namely mask) to improve dynamic range is a useful approach in the field of computational imaging.
Considering a special state, namely video SCI for a static scene. 10 same video frames need to be compressed into a measurement image. Without any optical modulation, compressed measurement's pixel range is 10× larger than each video frame. With binay mask modulation, compressed measurement's pixel range is ~5× larger than each video frame. With structural mask modulation, compressed measurement's pixel range is the same as each video frame. Compressed measurement's pixel range is determined by the used sensor, usually 0~255 for 8-bit sensor. Using different modulation strategies would affect the reconstructed dynamic range.
I hope the above statement answers your question well. You can also refer to our OL paper [https://doi.org/10.1364/OL.499735].

Thanks for your prompt answer!
10 same video frames need to be compressed into a measurement image. Without any optical modulation, compressed measurement's pixel range is 10× larger than each video frame.
In this case, you assume that the video frame keeps their original intensity when integration,  which causes 10x larger value and might cause saturation/clip and information loss (thus sacrify the dynamic range). But in physical capture, we will generally avoid saturation by decreasing the light throughput. So the intensity for each integrated video frame will be decreased as well, and the corresponding mask sequence for each pixel should be [1/10, 1/10, ..., 1/10], instead of [1, 1, 1,..., 1], which  also satisfy the porposed sutructural mask's sum-to-one condition. (Perhaps, the scale of the mask's summation value doesn't matter so much? As the physical capture process will adpat/normalize them according to the light brightness)
So, emmm, from the perspective of the overall information entropy, if the physical capture porcess doesn't introduce saturation,   i.e, no information loss, and the sensor's physical full-well-capacity determines the upbound of information captured., why changing the mask can increase the dynamic range. (I mean, if the reconstructed video frames have the same dynamic range as the sensor, there should be  some 'more' details of information). I hope I make my confusion clear, and thanks for your explanation!

Sensor's physical bit depth determines captured pixel range but captured inforamtion capacity. Following the above setting, compressed measurement is limited in 0-255, so video frames need to be 0-255/10 to avoid overexpose, leading to low dynamic range. In this case, using structural mask modulation, video frames keep in 0-255, namely full dynamic range.
One key objective of computational photography is to capture visual informartion better via physically limited hardwares. Please refer to some papers related to high-dynamic-range imaging via coded aperture/exposure.

Ok, thanks for your patient explanation.


add support for OpenShift Routes

This commit adds support for defining OpenShift routes in a Kedge
file.
Routes can be defined like the following at the root level of a
Kedge file
routes:

name: route1
host: route1-myproject.<IP_ADDRESS>.nip.io
port:
targetPort: 8080
to:
kind: Service
name: httpd

This is implemented by essentially merging RouteSpec and ObjectMeta
and adding an array of the resulting struct at the root level under
the field "routes"
Adds docs, tests.
Fixes #238

[ ] add docs
[ ] add tests
[ ] add examples

ping @kadel @cdrage @surajssd - I've added docs, tests and an example.
PTAL!
I might not be online tomorrow and day after (19th, 20th October), so feel free to edit the PR (edits are allowed by maintainers)
I've also broken changes into different commits, so should be easier to review.
If you folks need to merge, use the "Squash and merge" button, just in case :tada:

Just a few notes regarding docs, and test. The code itself looks good.


docs: include xForwardedFor param on getRequestIP

Adds a missing parameter on the docs for getRequestIP(). May be worth backporting to v1 docs too.
https://github.com/unjs/h3/blob/7324eeec854eecc37422074ef9f2aec8a5e4a816/src/utils/request.ts#L297-L307

Thanks for PR dear @RihanArfan. It is caused by an issue in automd not preserving some params.
Since it is an autogenerated content, your changes sadly will be lost after next lint pnpm automd.
If you want to contribute and fix it, it is more than welcome I can give you some pointers 👍🏼


Another Potential Enhancement; Charts

This could be another enhancement if not already available and if so, my fault. I was thinking about basic charting. Essentially something along the plotting disk growth over time. Say have a variable that allows a hashtable to be passed; and in that hashtable one could pass the fields for the X and Y coordinates. I might already be there and I have not see it or missed it, totally my fault. As I have said before the module is AWESOME, keep up the good work.
Thanks,
Forensicsguy20012004

The charts in Excel work off the data in a sheet. So, would if you could create a sheet with this data and then tell it to create a chart from it, is that what you're thinking?

Yep, that is exactly what I was thinking of. Have the data-sheet produce the chart and then if the end-user wants hide the data-sheet. For the record, I apologize, for the all enhancement requests/ideas. I just love your module.
Forensicsguy20012004

This is great feedback. I haven't exposed charting outside of pivot tables. I need to prioritize this.

So, let's say we take this one more step, bear with me. We have all of these tabs/data-sheets that would then be used to generate another tab/data-sheet for the chart/graph; instead of all these individual charts/graphs; let's put them all on one single tab/data-sheet. So essentially we would be creating a dashboard. One more step, there would be some drop-downs and an end-user would select some other way to slide-n-dice the data and the charts/graphs would change. As you probably see, the idea would be to truly create interactive dashboards based on tabs/data-sheets. How I envision using this is pulling data from MSSQL to populate the individual tabs/data-sheets.  Either way the module is AWESOME, keep up the great work.
Thanks,
Forensicsguy20012004

I'm playing around with adding this:

That looks AWESOME. This is fantastic work. You have really done some amazing work here.

So, here is kind of a snap-shot of what I was thinking. All of this data is driven off a single spreadsheet and all of these "charts" would be indicators.
Mind you this is all made up data in the screenshot but it would give you an idea of what I mean. Then a "slicer" could be added that would auto-adjust the charts. Let me know what you think.

Thanks, that's great. Are the Total Databases and Distinct Machines just cells with different fonts?

Yeah, they are just different cells and different fonts. But I think you kind of see where I am going with this? Essentially creating a PowerBI dashboard but all within Powershell code driven off maybe a single or a couple tabs/data-sheets. Seems monumental.

Hi @forensicsguy20012004, I'd like to dive a little deeper. If you ok with it, please email me at finked hotmail.
Thanks
Doug

Sent over an email to you from my gmail account. Look forward to chatting with you.


feat: Make NDK stack traces more standard

What does this PR do?
This changes the stack frame format to:
#{frame_number}  pc {relative_address}  {module_name} ({symbol_name}+{symbol_offset})

This is a more standard format of stack traces seen in examples and Android tombstones.
Also remove a few extra object creations / memory allocations.
refs: RUM-1896
Motivation
We're looking to start work on NDK crash symbolication and wanted a format that matched with what is in other examples. While the previous format has all the information we needed for symbolication, it wasn't quite right about what was offset by what. This is a more correct format.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

We're currently processing your upload.  This comment will be updated when the results are available.


do_ica() fails if BIO* channels are presents instead of ECG/EOG

Especially in older .fif files, the automatic ica fails since it is searching for ecg/eog channels, but they are not present ...
For our MEGIN system an easy fix is
` #%% fix first the eog/ecg naming mess
if 'BIO003' in raw.ch_names:
raw.set_channel_types(
{'BIO001': 'eog',
'BIO002': 'eog',
'BIO003': 'ecg',})
    mne.rename_channels(raw.info, {'BIO001': 'EOG001',
                                    'BIO002': 'EOG002',
                                    'BIO003': 'ECG003',})`

But dunno yet how to generalise it in other systems ... also in other Elekta/MEGIN ones it might be that e.g. BIO001 is ECG instead, since there was no standard naming back then ... one solution is to leave the fix to the users, more elegant would be to have a general solution for everyone, like if no ecg present search for ecg activity in the non MEG files (?!), or maybe better, if no ECG and/or EOG channels present then use just use some smart (template?) MEGs for the EOG, and all MEGs (?) for ECG ...

Can you give me the path to one of the files? I can post some ideas after having a look :)

here:
/home/gdemarchi/Work/Data/cdk_migraine/raw/controls/19800908IGDB_woCI_rest.fif


Update to 3.2.2

This change is

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

Project coverage is 69.65%. Comparing base (2469d89) to head (e9fbe47).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #221   +/-   ##
=======================================
  Coverage   69.65%   69.65%
=======================================
  Files         374      374
  Lines       31298    31298
  Branches     4383     4383
=======================================
  Hits        21801    21801
  Misses       8479     8479
  Partials     1018     1018

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Supports retina images in card

Picking up small issue from https://github.com/artsy/force/pull/6264 and https://github.com/artsy/palette/pull/776

LGTM!


How does stream::reduce with 3 arguments work?

the one that takes in identity, accumulator and combiner as arguments

I think the best way to think about this is that upon insertion of the parallel(), the pipeline is segmented into a few portions. Then, the accumulator is performed on the individual segments, before the combiner performs the specified operation combining the result again. It is best to keep in mind that even when performing this, the operations must be associative. We do not know how parallel() treats the pipeline and segments it, so as long as the operation as a whole (accumulator and combiner) is not associative, we will have a chance of obtaining indeterministic results. E.g. (12 + 34 != 1 + 2*3 + 4). Hence, even if the 2 individual operations are associative, the combined operation need not be


Add vendor support for vorwerk robots and fix zone retrieval

Breaking Change:

As the vendor default remains as neato this is not a breaking change for neato users.
Description:
This PR has a few small changes:

Bump pybotvac to 0.0.15
Add vendor support for Vorwerk robots (tested by community users: https://community.home-assistant.io/t/neato-component-add-vendors-in-order-to-allow-support-for-vorwerk-vacuum/121176/29?u=dshokouhi)
Fix #21796

Related issue (if applicable): fixes #21796
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#9882
Example entry for configuration.yaml (if applicable):
neato:
  username: username
  password: password
  vendor: vorwerk

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Thank you for your review @MartinHjelmare :)

Forgot to mention but we can also close this feature request in the forum once we merge this :)
https://community.home-assistant.io/t/please-integrate-vorwerk-vr200-vacuum-cleaner-robot-in-home-assistant/92042/4


Merging with another core extensions library

What do you think about merging with tiny library like http://github.com/hbakhtiyor/rcore-ext?

I'm always open to collaboration. Any concrete ideas you have in mind? Sorry about the super slow response!


Update README

Configurations for Java and installation with vim-plug.

Nice improvements. Thanks!

:+1:


AssertionError: Nesting violated for default stack of <class 'tensorflow.python.framework.ops.Graph'> objects

I used flask, and then called the interface. No error was reported the first time, but this error will definitely be reported the second time I called it.
Exception ignored in: <generator object Estimator.predict at 0x000001C3F2AF6AB0>
Traceback (most recent call last):
File "C:\Users\admin123\PycharmProjects\sd_flask.venv\lib\site-packages\tensorflow_estimator\python\estimator\estimator.py", line 618, in predict
with tf.Graph().as_default() as g:
File "C:\Users\admin123\AppData\Local\Programs\Python\Python310\lib\contextlib.py", line 153, in exit
self.gen.throw(typ, value, traceback)
File "C:\Users\admin123\PycharmProjects\sd_flask.venv\lib\site-packages\tensorflow\python\framework\ops.py", line 5821, in get_controller
with super(_DefaultGraphStack,
File "C:\Users\admin123\AppData\Local\Programs\Python\Python310\lib\contextlib.py", line 153, in exit
self.gen.throw(typ, value, traceback)
File "C:\Users\admin123\PycharmProjects\sd_flask.venv\lib\site-packages\tensorflow\python\framework\ops.py", line 5633, in get_controller
raise AssertionError(
AssertionError: Nesting violated for default stack of <class 'tensorflow.python.framework.ops.Graph'> objects

me too

Same issue, Anyone find solution.

我也遇到了这个问题

me too


Move to GH Actions

We switched to GH actions in other repos, so it might be beneficial to do it here as well for consistency.
This is for upgradability and various maintainers being able to do help here.

I can work on it, can you assign it to me?


Rework how connections work to remove the use for the transfer packets and multiple ports

Now for the internal Geyser instance, you no longer need another port and it smoothly transitions from the menu into Geyser.
I also cleaned-up the config, this will mean you have to remove any old options from the config file.
https://streamable.com/2lm8wf

After some testing from @Greaper88 on a public test instance this has no regressions and works well.


nm.profile: rewrite NM profile activation

This patch is rewriting the NM profile activation in order to use UUID
to identify which profile should be activated instead of using connection
id or device.
In addition, the ConnectionProfile class is being renamed to Profile and moved
to profile.py. A following PR will rewrite nm.active_connection.py into
profile.py in order to unify the NM profile management.
Signed-off-by: Fernando Fernandez Mancera<EMAIL_ADDRESS>

@cathay4t you can do a pre-review. The patch needs some adjustments for tests and for the activation but the structure is quite stable now. Thanks!

Found a better way for doing this. Will update soon. Thanks for all the comments.


Why Highcharts.Highcharts?

For access to methods & properties from the Highcharts library you can use Highcharts.Highcharts. For example, the Highcharts options are available via Highcharts.Highcharts.getOptions().

Why the extra level?

Also, is how does one use highcharts-release as a peer dependency?

First Highcharts is a react component, the nested one is a Highcharts library
See https://github.com/kirjs/react-highcharts/pull/61 for an example

I'm closing the issue, feel free to re-open if needed

In new version if the docs it's ReactHighcharts.Highcharts to make it more clear

@kirjs Thanks!


syntatic run with key not after autosave

hello, i want to stop syntastic from running after each file save(don't work very well with autosave plugin) and bind sysntastic to a key, i'm new to vim so i tryed to read the syntatic vim file, but withtout any result.
thanks and sorry for my english

Did you consider reading the manual?


allow {} in ldap.search-filter

Scope & Purpose
(Please describe the changes in this PR for reviewers - mandatory)

[ ] :hankey: Bugfix (requires CHANGELOG entry)
[x] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[x] :book: CHANGELOG entry made

Backports:

[ ] No backports required
[x] Backports required for: 3.7

Related Information
(Please reference tickets / specification / other PRs etc)

[x] Docs PR:
[x] Enterprise PR: https://github.com/arangodb/enterprise/pull/636
[ ] GitHub issue / Jira ticket number:
[ ] Design document:

Testing & Verification
(Please pick either of the following options)

[ ] This change is a trivial rework / code cleanup without any test coverage.
[x] The behavior in this PR was manually tested
[ ] This change is already covered by existing tests, such as (please describe tests).
[x] This PR adds tests that were used to verify all changes:

[ ] Added new C++ Unit tests
[x] Added new integration tests (e.g. in shell_server / shell_server_aql)
[ ] Added new resilience tests (only if the feature is impacted by failovers)

[ ] There are tests in an external testing repository:
[ ] I ensured this code runs with ASan / TSan or other static verification tools

Link to Jenkins PR run:
Documentation

All new features should be accompanied by corresponding documentation.
Bugs and features should furthermore be documented in the CHANGELOG so that
support, end users, and other developers have a concise overview.

[ ] Added entry to Release Notes
[ ] Added a new section in the Manual
[ ] Added a new section in the HTTP API
[ ] Added Swagger examples for the HTTP API
[ ] Updated license information in LICENSES-OTHER-COMPONENTS.md for 3rd party libraries

External contributors / CLA Note
Please note that for legal reasons we require you to sign the Contributor Agreement
before we can accept your pull requests.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/13714/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/13714/

In "ldap.search-filter" the pattern "{USER}" will be replaced by the username

In "ldap.search-filter" the pattern "{USER}" will be replaced by the username

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/13716/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/13716/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/13721/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/13721/

Tests blue except unrelated failure.

Tests blue except unrelated failure.


[TC-DM-1.2], [TC-DM-1.4] -  Events can't be verified

Problem
Events can't be verified - shut-down, leave event

expected behavior : after the execution of events no more events or commands or attributes can't be executed
actual behavior : after the execution of events other commands can be executed
steps to reproduce : sudo ./chip-tool basic read-event start-up 1 0
sudo ./chip-tool basic read-event shut-down 1 0
sudo ./chip-tool basic read-event leave 1 0

I don't understand what this issue is about, exactly...  read-event for the "startup" event will generally work and other things can happen after that, of course.
Reading the shutdown and leave events will generally return nothing, because those only happen when the server is shutting down or being decomissioned, so is no longer around to be read from.  Subscribing to them should work, though.
@sowmyassp can you please describe the exact behavior you expected and why you expected it?

@bzbarsky-apple ----Yes reading the shutdown and leave events will generally return nothing but after that if we read any other basic attribute or send any command it should not pass as after shutdown event DUT will be closed right?

If the device is shut down, yes...
At this point, this issue could benefit from:

Clear steps to reproduce.  The steps listed above do not involve shutting down the DUT, so I have no idea what they have to do with "after shutdown" behavior.
A clear description of where (after which step) expected behavior and actual behavior diverge.

@bzbarsky-apple  After shutdown event i.e., sudo ./chip-tool basic read-event shut-down 1 0
the connectivity should close and if we try to read for example ./chip-tool basic read vendor-name 1 0  it should fail as we did a shut-down event before.

sudo ./chip-tool basic read-event shut-down 1 0
After event I tried to read vendor-name
sudo ./chip-tool basic read vendor-name 1 0
After this command it should fail because in step 1 the window should be closed

After shutdown event i.e., sudo ./chip-tool basic read-event shut-down 1 0

Reading the shut-down event does not make anything shut down.

the connectivity should close

No, it should not.


DirectB2S Preview does not support Grill/DMD settings

The image should preview the matching configuration.
E.g. if a grill is hidden, it should be hidden from the generated image too.

Additional issues will raise, but for now we have the preview. Let's see what else needs to be fixed there.


allow custom options merge strategies

try to fix #56

Thanks for the PR - this is done in a more straightforward fix in 2.1.1


Hunt Showdown——Hunt: Showdown 1896

Engine change. Can you update it.

I updated the offset,I game into the black screen.cry

A problem with my game, I don't know if my updated offset will work,I can't log in to the game

uint64_t SystemGlobalEnvironment = 0x26782F8;
uint64_t EntitySystem = 0xC0;
uint64_t pSystem = 0x90;
uint16_t ObjectCount = 0x0;
uint64_t EntityList = 0xC0;
uint16_t ObjectCountOffset = 0x40092;
uint64_t EntityListOffset = 0x40098;
uint64_t CameraBase = 0x928;
uint64_t ViewMatrixOffset = 0x230;
uint64_t CameraPosOffset = 0x2F0;
uint64_t ProjectionMatrixOffset = 0x270;
Where else do I need to update the offset？？？？？？？？？？？？？？


Support CertifyCreation for Seal/Unseal

Changed Seal() Unseal() Reseal() APIs to accept SealingConfig and CertifyConfig
Removed duplicated tests in ./gotpm

I saw some changes may have conflict/duplicate with #55 , I will update this PR when #55 is merged.
Known Issue:

When try to seal/reseal with an empty pcr selection, seems like the computePCRSessionAuth() is not yielding the correct Auth digest, which will cause the unsealing to fail. Currently I am by passing this by calling getPCRSessionAuth() directly from the TPM()

@googlebot I fixed it

Closing. Create a new PR #60


Dialogue after video cutsence

Describe your problem here.
Trying to figure out how to get dialogue to work after video
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

Put a script that plays a video onto your song chart and another script that runs the dialogue, each one will be named how you want to. It's kinda more easy that what it seems.


Users Cannot Set Their Location Using the Map in Enatega Multivendor Customer App

Issue: Users Cannot Set Their Location Using the Map in the Customer App
Description:
A significant issue has been identified in customer app, where users are unable to set or update their location using the map feature. This functionality is crucial for ensuring accurate delivery and providing users with a seamless experience.
Impact:
Delivery Accuracy: Inaccurate location information can lead to delivery issues and user dissatisfaction.
User Convenience: Users may find it inconvenient when they cannot set their location using the map, resulting in a suboptimal user experience.
Steps to Reproduce:

Open the customer app.
Attempt to set or update your location using the map feature.
Observe that the map does not respond or allow you to select your location accurately.

Expected Resolution:
The customer app should be reviewed and adjusted to ensure that users can set or update their location accurately using the map feature. This functionality is crucial for ensuring efficient service and a convenient user experience. Addressing this issue promptly is essential for user satisfaction and the app's functionality.

Resolved!


no map generated using kinect2

I want to run cartographer with kinect2. Here are my configuration file and launch file.
<launch>
  <param name="/use_sim_time" value="true" />
  <node name="squash_depth_camera" pkg="depthimage_to_laserscan"
      type="depthimage_to_laserscan">
    <param name="output_frame_id" value="kinect2_ir_optical_frame"/>
    <remap from="image" to="/kinect2/qhd/image_depth_rect" />
    <remap from="camera_info" to="/kinect2/qhd/camera_info" />
  </node>
<!--   <node name="squash_depth_camera" pkg="pointcloud_to_laserscan"
      type="pointcloud_to_laserscan_node">
    <remap from="cloud_in" to="/kinect2/qhd/points" />
  </node> -->
  <node name="cartographer" pkg="cartographer_ros"
      type="cartographer_node" args="
          -configuration_directory $(find cartographer_ros)/configuration_files
          -configuration_basename kinect2.lua"
      output="screen">
  </node>
  <node name="rviz" pkg="rviz" type="rviz" required="true"
      args="-d $(find cartographer_ros
          )/configuration_files/kinect2.rviz" />
  <node name="playbag" pkg="rosbag" type="play"
      args="--clock $(arg bag_filename)" />
</launch>

include "map_builder.lua"
include "trajectory_builder.lua"

options = {
  map_builder = MAP_BUILDER,
  trajectory_builder = TRAJECTORY_BUILDER,
  map_frame = "map",
  tracking_frame = "kinect2_ir_optical_frame",
  published_frame = "kinect2_ir_optical_frame",
  odom_frame = "odom",
  provide_odom_frame = false,
  use_odometry = true,
  use_laser_scan = true,
  use_multi_echo_laser_scan = false,
  num_point_clouds = 0,
  lookup_transform_timeout_sec = 0.2,
  submap_publish_period_sec = 0.3,
  pose_publish_period_sec = 5e-3,
}

MAP_BUILDER.use_trajectory_builder_2d = true
MAP_BUILDER.num_background_threads = 7

TRAJECTORY_BUILDER_2D.min_range = 0.1
TRAJECTORY_BUILDER_2D.max_range = 4.0
TRAJECTORY_BUILDER_2D.missing_data_ray_length = 2.0
TRAJECTORY_BUILDER_2D.use_imu_data = false
TRAJECTORY_BUILDER_2D.motion_filter.max_time_seconds = 0.3
TRAJECTORY_BUILDER_2D.motion_filter.max_distance_meters = 0.2
TRAJECTORY_BUILDER_2D.motion_filter.max_angle_radians = math.rad(3.)
TRAJECTORY_BUILDER_2D.ceres_scan_matcher.translation_weight = 70
TRAJECTORY_BUILDER_2D.ceres_scan_matcher.rotation_weight = 300

TRAJECTORY_BUILDER_2D.submaps.resolution = 0.035
TRAJECTORY_BUILDER_2D.submaps.num_range_data = 120
SPARSE_POSE_GRAPH.optimize_every_n_scans = 120
SPARSE_POSE_GRAPH.constraint_builder.min_score = 0.82
SPARSE_POSE_GRAPH.constraint_builder.sampling_ratio = 1.
SPARSE_POSE_GRAPH.constraint_builder.ceres_scan_matcher.covariance_scale = 3e-5

return options

I can see the laser scan data transformed from kinect2 depth image, but I cannot see any map generated. No map data is published.
I have uploaded these two files in my forked directory: https://github.com/CTTC/cartographer_ros
And a small bag recording data published by kinect2 is available here

Thanks, @Entropy512  I misunderstood the meaning of provide_odom_frame.Now I have set use_odometry to false as well. And I can get map message now, but it's a pretty bad map. As imu and odom are optional in 2D case, I am only using a kinect2 now.  So I only have these frames available: kinect2_ir_optical_frame, kinect2_rgb_optical_frame,kinect2_link. No other frames are available.  In this case, how should I set the map_frame, tracking_frame, and published_frame? If I use the lua file shown above, the /scan data shown in rivz is always blinking. It shows Transforms Ok and For frame [kinect2_ir_optical_frame]: No transform to fixed frame [map].  TF error: [Could not find a connection between 'map' and 'kinect2_ir_optical_frame' because they are not part of the same tree.Tf has two or more unconnected trees.] alternately.

You are doing 2D SLAM with kinect? Are you projecting the points into a plane before passing it to Cartographer somehow?
map_frame should be "map", tracking_frame should be the frame which's position you are interested in. Set published_frame to the same thing.
You do not need to mention /map in your TF tree, but it seems something else is misconfigured. Did you run roswft?

@SirVer  Thanks for your reply. Yes, I am trying 2D SLAM with kinect (no other sensors for now). I followed the turtlebot example which convert the depth image into laser scan using depthimage_to_laserscan.
I can run the program now. It's just that the map does not look good (not even close). I assume that is because no imu or odom data is available.
Also, I tried your advice, and here is the output of roswtf:
Found 2 warning(s).
Warnings are things that may be just fine, but are sometimes at fault

WARNING The following node subscriptions are unconnected:
 * /squash_depth_camera:
   * /clock
 * /cartographer:
   * /tf_static
   * /clock
 * /rviz:
   * /tf_static
   * /clock
   * /map_updates

WARNING Received out-of-date/future transforms:
 * receiving transform from [/cartographer] that differed from ROS time by<PHONE_NUMBER>.67s
 * receiving transform from [/kinect2] that differed from ROS time by<PHONE_NUMBER>.71s

Found 3 error(s).

ERROR /use_simtime is set but no publisher of /clock is present
ERROR TF re-parenting contention:
 * reparenting of [kinect2_ir_optical_frame] to [kinect2_rgb_optical_frame] by [/kinect2]
 * reparenting of [kinect2_ir_optical_frame] to [map] by [/cartographer]

ERROR TF multiple authority contention:
 * node [/kinect2] publishing transform [kinect2_ir_optical_frame] with parent [kinect2_rgb_optical_frame] already published by node [/cartographer]
 * node [/cartographer] publishing transform [kinect2_ir_optical_frame] with parent [map] already published by node [/kinect2]

These errors disappeared after I set publish_tf to false in kinect driver. Thanks for your suggestion.
So now I only have these two questions:

If I add imu sensor, will this improve the 2D map?
If I move the kinect sensor vertically (change its height in a robot) while running the slam algorithm, will cartographer still be able to generate descent map? If it is fine, does that require imu sensor?

If I add imu sensor, will this improve the 2D map?

Maybe. But it seems unlikely that your map is very bad now and you fix it with just adding IMU. I expect that your setup is poorly tuned.

If I move the kinect sensor vertically (change its height in a robot) while running the slam algorithm, will cartographer still be able to generate descent map? If it is fine, does that require imu sensor?

No, not if you use 2D SLAM. This violates the 2D assumptions. You could try doing 3D SLAM with your depth camera, but that requires an IMU.

Thank you all. @SirVer @Entropy512

Sorry for the long time to ask.... Could you tell me are you using this package to build map in real-time, or just record data from kinect and then process the data and build a map?
Thank you for your patience, waiting for your replying..


Improve the Android experience.

Godot version
all
System information
Android
Issue description
Since godot was ported to Android, the development team has been working to improve the experience of using the Android engine. While the experience has greatly improved since the original version, and has benefited most of the student developer community, there is now a huge problem: how do we share the apps we've worked so hard to make? Currently, the related Settings exported by Android are deliberately hidden, which becomes a huge barrier to sharing. It's also discouraging for our particular group, and as an engine, I think the "export" function is one of the important features it has, and it's part of the experience. So far, most of the official replies to "export on Android" are "Android does not have the necessary conditions for export such as sdk". But as far as I know, now a number of components including Android sdk and ndk have corresponding implementations on Android. There are even applications that use them (e.g. AIDE: an Android integrated development environment that runs on the Android side and can export JAVA or Cpp into an apk that can be installed). Therefore, I think the lack of sdk and ndk should not be the reason to cut down the Android export function. At the same time, as a part of the use experience, I also hope that we can share the fruits of our labor more easily, rather than let the other side to carry out several tedious steps and finally run my "app" in the debug mode of the engine like me.
Steps to reproduce
null
Minimal reproduction project
null

This seems more appropriate for https://github.com/godotengine/godot-proposals

I know it's a big project to improve Export on Android, but anyway, let's start with move the button back to where it was

Duplicate of https://github.com/godotengine/godot/issues/73130.


Remove all non-standard CommandDests

DestTransform
DestCombiner
DestTupleTable

Each of these can probably be replaced fairly cleanly by using DestTuplestore and just writing those tuples out as they currently are in the current CommandDests.

While this obviously requires a bit of extra iteration since output rows aren't written directly to the receiver, after some initial work it seems like this model will actually help quite a bit with 853.

Yes, that's going to be a pretty killer feature. I think first step is going to be ordered stream processing. Which we can add as another "level", since it'll slow things down a bit.


kconfiglib: Expose parsing location to Python preprocessor functions

Update Kconfiglib (and menuconfig/guiconfig, just to sync) to upstream
revision f2ce282eca, to get this commit in:
Allow preprocessor user functions to access the parsing location

Just requires making Kconfig.filename/linenr public.

'lineno' would be a more standard name, but be consistent with
MenuNode.linenr.

This can be used to give friendly errors in
scripts/kconfig/kconfigfunctions.py, e.g. for
https://github.com/zephyrproject-rtos/zephyr/pull/18752.
Some minor optimizations are included too.

ping @carlescufi


colab

there are many great colabs
I tried some and managed to install the extensions correctly however I struggled with enabling api as I'm not a programmer
Please consider adding simple colab
thanks

I tried this(new) plugin today, but I was not able to make it work (it crash webui for some reason), tomorrow il try again. If this plugin works il make Colab for it
You can use https://github.com/danilw/Krita-StableDiffusion-WebUI-Colab "old" plugin and Colab for it, it does work.

Send me links to a few of the collabs you are using, I will try and figure out why it crashes.
Actually, you are able to register additional APIs without needing the --api flag, but I decided to only activate my additional API when the --api flag is provided to stay inline with expected behaviour.

actually I managed to get it working with some help
I used this colab its models are low size but still generates same results and works faster
here is the --api fix
here is my fixed colab

but I'm still facing problem with gradio it keeps getting timed out
so I used ngrok and seems to do fine so far

Nice. Anyways, maintaining collabs is actually out of scope for me. Since the plugin is an extension now, it should work for any collab that properly supports extensions and the --api/--ngrok flags.


Added Framer Motion Navbar

Please Review and Merge and kindly give it level 3
@jayk-gupta
https://github.com/jayk-gupta/web-resources-project/assets/112967201/507f12f8-db73-4cac-aa5d-99b5540ea9b2

please pull changes before pushing, you are working on old UI

please pull changes before pushing, you are working on old UI

already pulled what is the issue explain


Support React 18

Hello everyone 👋 ,
This module supports React 17 only but I'd like to use it with React 18.
I'm using npm 8 and I don't want to install deps with --legacy-peer-deps so it'd be great if you could let this module to be used with React 18.
Thank you in advance.

Sure. This should be a pretty trivial change. 👍

@CharlesStover Thank you for your extremely fast response. You're the best 👍


ZoomToRectangle - options for which mouse click to zoom

the rectangle is drawn with onMouseDown, but it would be nice to choose which mouse click to aactually draw it since we want it with left click only.

Thanks for submitting the issue, we will consider adding the feature to our next release.

Solved in master branch - next release will be out very soon :)


NoSuchMethodError: The getter 'node' was called on null at FixProcessor._addFix_createGetter (fix_internal.dart:2171)

From crash reporting:
NoSuchMethodError: The getter 'node' was called on null
	at Object.noSuchMethod	(dart:core )
	at FixProcessor._addFix_createGetter	(fix_internal.dart:2171 )
	at <asynchronous gap>	(async )
	at FixProcessor.compute	(fix_internal.dart:468 )
	at <asynchronous gap>	(async )
	at DartFixContributor.computeFixes	(fix_internal.dart:69 )
	at EditDomainHandler._computeDartFixes	(edit_domain.dart:656 )
	at <asynchronous gap>	(async )
	at EditDomainHandler._computeServerErrorFixes	(edit_domain.dart:763 )
	at EditDomainHandler.getFixes	(edit_domain.dart:283 )
	at EditDomainHandler.handleRequest	(edit_domain.dart:387 )
	at AnalysisServer.handleRequest.<fn>.<fn>	(analysis_server.dart:333 )
	at _PerformanceTagImpl.makeCurrentWhile	(utilities_general.dart:258 )
	at AnalysisServer.handleRequest.<fn>	(analysis_server.dart:329 )
	at runZoned	(dart:async )
	at AnalysisServer.handleRequest	(analysis_server.dart:328 )
	at ByteStreamServerChannel._readRequest.<fn>	(byte_stream_channel.dart:189 )
	at _PerformanceTagImpl.makeCurrentWhile	(utilities_general.dart:258 )
	at ByteStreamServerChannel._readRequest	(byte_stream_channel.dart:179 )
	at ByteStreamServerChannel.listen.<fn>	(byte_stream_channel.dart:120 )

fixed by https://dart-review.googlesource.com/c/sdk/+/127422


Panini tests do not pass on Ubuntu with pytest command

On Mac and Windows pytest command works just fine - and all tests pass,
But on Ubuntu platform tests have some problems

The problem is inside test_middleware_manager.py


MiAIRR record on fairsharing.org

Hi All,
Who is the owner of the MiAIRR record on fairsharing.org.
https://fairsharing.org/bsg-s000689
I noticed that the site is using the redirect to the SFU domain name rather than using the airr-community.org address. It should probably be changed from:
http://airr.irmacs.sfu.ca/working_groups/minimal_standards
to:
http://airr-community.org/working_groups/minimal_standards
Brian

Steve Kleinstein created the fairsharing.org record.

Alejendra from oxford (http://www.oerc.ox.ac.uk/people/alejandra) is managing this portal and she suggested us (in one telecon) to update the standards page with documentation link and templates files  as well.

The fairsharing.org record currently refers to http://miairr.org, which redirects to this repository. The last decision on this topic from MiniStd WG (call on 2018-08-10) was to use http://miairr.airr-community.org, which also redirects here. The link http://airr-community.org/working_groups/minimal_standards does not work anymore, as it redirects to TAbS hence the path component breaks.
We should contact Steve once the current domain dispute is settled.

Contacted Steve today, asking whether he wants to continue updating the record or transfer this task to MiniStd. Will F/U on this here...

@bussec ping

Re-contacted Steve just now...

It turns out that the records cannot be "pushed" to other people, so I just sent a "claim" request for that record to fairsharing.org.

I now have access to the record, updated as many links as possible. Two questions before we can close this:

@bcorrie : The deep link you mentioned above (http://airr-community.org/working_groups/minimal_standards) is still not working. I cannot remember whether there was any final decision on this when we discussed the domain issues, but this is the URL we put into the NI paper...
@schristley @laserson : Is there a fairsharing record for the DataRep Standardized Representation? If yes, let me know and I will link it.

Is there a fairsharing record for the DataRep Standardized Representation?

Not that I'm aware of.

@bcorrie : The deep link you mentioned above (http://airr-community.org/working_groups/minimal_standards) is still not working. I cannot remember whether there was any final decision on this when we discussed the domain issues, but this is the URL we put into the NI paper...

@bussec I believe you will need to talk to the Comms Committee about this, not sure where this has landed and if anything has been done since I was last involved in the URL redirect discussion - which was about this time last year...

@schristley @bcorrie I now included the DataRep (Front Immunol 2018) paper in the record. I thought about making the record a more general "AIRR standards" record, but realized that FAIRsharing makes a distinction between "Formats" and "Reporting Standards". So the record for the DataRep Standard is a new issue (#306). I close this ticket now, as the original question has been resolved.


Forked Repos Fail to Load

Hey there!
First of all, this is a great app! I definitely find myself doing these manual searches more often than I'd like to. Having a one-click action to jump to the commit and/or PR in GitHub website is definitely a time saver!
I'd like to report a bug, though: for forked repos, it tries opening the commit/PR in the forked repo instead of the original repo. It should probably be able to pick up whether the PR belongs to the original PR or one of its forks.
Thanks for all the effort to build this great tool!

That's good to know, thanks!


mkchain fails to run minikube's docker

https://github.com/tqtezos/tezos-k8s/blob/0b2ed250ae44ec23f492533130081dc511c1ecb5/mkchain/tqchain/mkchain.py#L31
executing the following
mkchain $CHAIN_NAME --zerotier-network $ZT_NET --zerotier-token $ZT_TOKEN

results in the error
bozon@buba test % minikube start
😄  minikube v1.16.0 on Darwin 11.1
✨  Using the hyperkit driver based on existing profile
👍  Starting control plane node minikube in cluster minikube
🏃  Updating the running hyperkit "minikube" VM ...
🐳  Preparing Kubernetes v1.20.0 on Docker 20.10.0 ...
🔎  Verifying Kubernetes components...
🌟  Enabled addons: storage-provisioner, default-storageclass
🏄  Done! kubectl is now configured to use "minikube" cluster and "default" namespace by default
bozon@buba test % eval $(minikube docker-env)
bozon@buba test % mkdir mkchain && cd mkchain
python3 -m venv .venv
source .venv/bin/activate

(.venv) bozon@buba mkchain %
(.venv) bozon@buba mkchain % pip install mkchain==0.0.1
Collecting mkchain==0.0.1
  Using cached mkchain-0.0.1-py3-none-any.whl (5.0 kB)
Collecting pyyaml
  Using cached PyYAML-5.3.1.tar.gz (269 kB)
Using legacy 'setup.py install' for pyyaml, since package 'wheel' is not installed.
Installing collected packages: pyyaml, mkchain
    Running setup.py install for pyyaml ... done
Successfully installed mkchain-0.0.1 pyyaml-5.3.1
WARNING: You are using pip version 20.2.3; however, version 20.3.3 is available.
You should consider upgrading via the '/Users/bozon/Documents/TQ/test/mkchain/.venv/bin/python3 -m pip install --upgrade pip' command.
(.venv) bozon@buba mkchain % export PYTHONUNBUFFERED=x
(.venv) bozon@buba mkchain % export CHAIN_NAME=avb
(.venv) bozon@buba mkchain % mkchain $CHAIN_NAME --zerotier-network $ZT_NET --zerotier-token $ZT_TOKEN
Traceback (most recent call last):
  File "/Users/bozon/Documents/TQ/test/mkchain/.venv/bin/mkchain", line 8, in <module>
    sys.exit(main())
  File "/Users/bozon/Documents/TQ/test/mkchain/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 125, in main
    "genesis_chain_id": get_genesis_vanity_chain_id(),
  File "/Users/bozon/Documents/TQ/test/mkchain/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 48, in get_genesis_vanity_chain_id
    run_docker(
  File "/Users/bozon/Documents/TQ/test/mkchain/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 19, in run_docker
    return subprocess.check_output(
  File<EMAIL_ADDRESS>line 420, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File<EMAIL_ADDRESS>line 524, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'docker run --entrypoint flextesa --rm registry.gitlab.com/tezos/flextesa:01e3f596-run vani "" --seed 4X4M58LLL043JZWS --first --machine-readable csv' returned non-zero exit status 127.

I don't think 'eval $(minikube docker-env)' does what it intends to do. In an attempt to address the error above I run
brew install docker
brew link docker

and it seemed to have solved the immediate problem and progressed further to the following error:
(.venv) bozon@buba mkchain % mkchain $CHAIN_NAME --zerotier-network $ZT_NET --zerotier-token $ZT_TOKEN
Traceback (most recent call last):
  File "/Users/bozon/Documents/TQ/test/mkchain/.venv/bin/mkchain", line 8, in <module>
    sys.exit(main())
  File "/Users/bozon/Documents/TQ/test/mkchain/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 141, in main
    keys = gen_key(args.docker_image)
  File "/Users/bozon/Documents/TQ/test/mkchain/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 38, in gen_key
    return {"public_key": extract_key(keys, 1), "secret_key": extract_key(keys, 2)}
  File "/Users/bozon/Documents/TQ/test/mkchain/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 27, in extract_key
    return keys[index].split(b":")[index].strip().decode("ascii")
IndexError: list index out of range

I don't think 'eval $(minikube docker-env)' does what it intends to do. In an attempt to address the error above I run
brew install docker
brew link docker

and it seemed to have solved the immediate problem and progressed further to the following error:
(.venv) bozon@buba mkchain % mkchain $CHAIN_NAME --zerotier-network $ZT_NET --zerotier-token $ZT_TOKEN
Traceback (most recent call last):
  File "/Users/bozon/Documents/TQ/test/mkchain/.venv/bin/mkchain", line 8, in <module>
    sys.exit(main())
  File "/Users/bozon/Documents/TQ/test/mkchain/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 141, in main
    keys = gen_key(args.docker_image)
  File "/Users/bozon/Documents/TQ/test/mkchain/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 38, in gen_key
    return {"public_key": extract_key(keys, 1), "secret_key": extract_key(keys, 2)}
  File "/Users/bozon/Documents/TQ/test/mkchain/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 27, in extract_key
    return keys[index].split(b":")[index].strip().decode("ascii")
IndexError: list index out of range

This 'index out of range' error might be a different issue from what was originally reported here. In any case this problem seem to appear intermittently, got it again today:
(.venv) bozon@blob aryeh % mkchain --docker-image tezos/tezos:v8-release av8
Traceback (most recent call last):
  File "/Users/bozon/Documents/TQ/aryeh/.venv/bin/mkchain", line 8, in <module>
    sys.exit(main())
  File "/Users/bozon/Documents/TQ/aryeh/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 141, in main
    keys = gen_key(args.docker_image)
  File "/Users/bozon/Documents/TQ/aryeh/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 38, in gen_key
    return {"public_key": extract_key(keys, 1), "secret_key": extract_key(keys, 2)}
  File "/Users/bozon/Documents/TQ/aryeh/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 27, in extract_key
    return keys[index].split(b":")[index].strip().decode("ascii")
IndexError: list index out of range

This 'index out of range' error might be a different issue from what was originally reported here. In any case this problem seem to appear intermittently, got it again today:
(.venv) bozon@blob aryeh % mkchain --docker-image tezos/tezos:v8-release av8
Traceback (most recent call last):
  File "/Users/bozon/Documents/TQ/aryeh/.venv/bin/mkchain", line 8, in <module>
    sys.exit(main())
  File "/Users/bozon/Documents/TQ/aryeh/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 141, in main
    keys = gen_key(args.docker_image)
  File "/Users/bozon/Documents/TQ/aryeh/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 38, in gen_key
    return {"public_key": extract_key(keys, 1), "secret_key": extract_key(keys, 2)}
  File "/Users/bozon/Documents/TQ/aryeh/.venv/lib/python3.9/site-packages/tqchain/mkchain.py", line 27, in extract_key
    return keys[index].split(b":")[index].strip().decode("ascii")
IndexError: list index out of range


add metrics for op influence and tolerant resource

It can be merged after https://github.com/pingcap/pd/pull/1888

PTAL @disksing @shafreeck @nolouch


Update code for latest Ocean (cloud-client 0.11)

Fix #52.

Not sure what's up with windows tests, though. (Unrelated to this PR)


Library under development is not installed into virtual environment in CI

Using a CI setup such as
variables:
  UV_VERSION: 0.4
  PYTHON_VERSION: 3.9
  BASE_LAYER: bookworm-slim

image: ghcr.io/astral-sh/uv:$UV_VERSION-python$PYTHON_VERSION-$BASE_LAYER

...

doc:
  stage: build
  script:
    - uv sync
    - uv run sphinx-build -b html docs docs/_build/html

The uv run sphinx-build -b html docs docs/_build/html fails because uv does not install the library under development into the virtual environment in the uv sync step.
The uv.lock file shows
[[package]]
name = "libunderdev"
version = "2024.1"
source = { virtual = "." }
...

In local dev, uv sync installs the library into the virtual environment.

I don't think it does install it locally if it's marked as "virtual".
What's your pyproject.toml look like?
Have you read the build system documentation?

Thanks for the quick response and pointers. Indeed I was looking at the wrong Terminal and thus pyproject.toml.
Having
[build-system]
requires = ["hatchling"]
build-backend = "hatchling.build"

in pyproject.toml works as expected. Sorry for the noise.

No prob, thank you for following up.


Issue deallocate: Ptr

Hi,
like the title, in all two sample, the object Ptr present issue when program end: in memory deallocation phase.
My system configuration is:

Win 8.1
VS2013
OpenCV 3 beta.

error is:
HEAP CORRUPTION DETECTED
I have not this error if declare static Ptr and run app in release mode.
Sorry for my English.
Nicola
I have attached screenshot error.

Hello, could you please add the software that lead to this crash so we can check if this still exists in the current repository?


Print WARN messages to stderr instead of stdout

When running ag on some path, it spits a lot of:
WARN: Skipping ./long/path. Use the --depth option to search deeper.

I'm use to quiet grep(1) by appending 2>/dev/null, but it just doesn't
work here because they're printed on standard output.
Move the warning to stderr so that it can be easily silented if
necessary.

I agree that the current behavior is undesirable, but I think it would make more sense to change the long path warning to an error. Here's how I think of the log levels:

DEBUG: Verbose debugging stuff.

LOG: Normal stuff.

WARN: Bad, but not so bad that it needs to go to stderr.

ERR: Pretty bad.

While log_warn isn't used anywhere else in the codebase, it could be handy in the future. Better to allow for different behavior than to make it log_err in all but name.

I'm usually not a big fan of logs, also for this reason.
I think I would move everything to stderr.
In my opinion stdout should be used for the actual output of the application, stderr for everything "meta". As an example, I might want to parse the output of grep/ag for whatever purpose, and I expect "messages" to not interfere with my file-descriptor. Let's say I do: ag something | wc -l. That'd be kind of awkward.

I agree that WARN should go to stdout. Otherwise e.g. piping output into xargs breaks!

Wait, did you mean stdout or stderr? Having errors (or even warnings for that matter) spitted in stdout just makes it impossible to script properly

Oh right I meant stderr!

On Nov 30, 2015, at 1:13 PM, Antoine Pelisse<EMAIL_ADDRESS>wrote:
Wait, did you mean stdout or stderr? Having errors (or even warnings for that matter) spitted in stdout just makes it impossible to script properly
—
Reply to this email directly or view it on GitHub.


Testing with Adium not Okay

Hi guys,
I've just came across xmpp-server and hitten my first try with it. I wanted to test it against a simple jabber client, Adium, and I get some difficulties..
That's what I did :
First of all, in server.js, I've disactivated the authenticate part (i call cb() without any verification) and logged any try of authentication. I don't care about auth for my use..
I've launched my adium client and try to connect as, for instance, foo@localhost with password bar. Adium warns me about the fact that my connection is not encrypted, my auth log is hitten and I'm sure that the authenticate callback is called.
But Adium remains in state connecting.
has someone any hints about it ?

Yes. Adium is a bit excessive and wants the server to send an initial presence as well as a roster before considering the user connected. You may want to check that the server does indeed send those, but I dno't think it does.
Check with Psi that has XML logging.

Oh thanks for your answer.
I understand. Actually, I forgot to mention – shame on me – I also disactivated the Roster and Presence modules : this can explain that !! Maybe it works with'em, but I won't try, because it's sufficient for my need without.
I'll try Psi, this could be really helpful ! thanks.


Added smithing plugin.

So this plugin is really rough. I wrote is sometime last year, and was going to clean it up and have not found the time to do so. If someone would like to work on it, feel free.

I thought I would just clarify, this plugin is in working order, the code may be a bit messy, but it is fully functional.

@tlf30 sorry for the delay. Are you still looking for a review on this PR? I'm looking to get it closed off and can provide review feedback or pick it up myself.

Hello @garyttierney I will be at work in the artic circle for the next three weeks. If you or someone wants to tackle cleaning it up, it would be much appreciated.

This has some useful stuff in but isn't in a good state to merge. I'm going to close this PR and when we look to add smithing we can work off this.
Thanks!


Added server connection status check

Summary
Details and comments
closes #23

Looking Good !, that's a huge UX enhancement

Thank you!

Appreciate it @farhadjaman ! Good work !

Thank you @gobbledy-gook !


Add native-select and Java API refactor

Uncomment "native-select" Section (Section 7)
Move Java Remote API Section include into Java API doc (Section 7)
Rearrange API sections such that Local and Remote are next to each other (Section 7)
Rename "Java API (Embedded)" to "Java API (Local)" (Section 7)

Thanks