Changed to cuDNN BatchNorm, use Bottle instead of SequenceWise

Changes to the benchmark as discussed on this issue #1. Also removed a module in favour of a torch standard module!

Thanks :)


Scroll to top button updated to match website theme

PR Description 📜
I have added a scroll to top button that becomes visible once the user starts scrolling down. It turns purple when the mouse is hovered over it. Clicking on the button scrolls the page to the very top. The scroll to top button has now been updated to correctly match the website's theme both in light and dark modes.
Fixes #1520

Mark the task you have completed ✅

[x ] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x ] I have performed a self-review of my own code or work.
[x ] I have commented my code, particularly in hard-to-understand areas.
[x ] My changes generates no new warnings.
[x ] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x ] I have added screenshot for website preview in assets/images
[ ] I have added entries for my game in main README.md
[ ] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @kaashikaagrawal ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @kaashikaagrawal , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Issues with tiling window manager.

Hey, I'm trying to run an old project using minifb. I'm having issues due to using a tiling window manager. I believe the window manager forces the window to be a specific size which the input buffer is too small to fill.
Sometimes the program will segfault, and other sometimes it will panic with
thread 'main' panicked at 'Failed to update window buffer.: UpdateFailed("Update failed because input buffer is too small. Required size for 1916 x 525 window (1x scale) is 4023600 bytes but the size of the input buffer has the size 1310
2 bytes")', libcore/result.rs:1009:5

Thanks.

hum.. if the window manager can force the window to a large size the buffer that is being sent int to minifb has to be large enough. Otherwise it may be possible to make sure to set that the window can't be resized when you create the window.
I'm not sure if the window manager will force it larger anyway and then the only way is really is to make sure to have large enough buffer that minifb will never read outside it

I have defined the window size to be 256x128, but as you can see from the panic message, the window is 1916x525. Is it possible to somehow fit the input buffer to the larger window and fill in the blanks?

There is no automatic way to do it currently. I think this is really an issue only on linux where there is no API for doing drawing rescaling to a windows so I have to do the code for scaling myself. Do you know if you set the scale flag of the window to true or false?

I tried this on Linux now with i3 (also tiling manager) using resize: false for the WindowOptions works there at least

Hey, I just tried setting resize: false in the WindowOptions and I still get a mix of input buffer to small and segfaults. I'm using Sway which might have something to do with it. If it works on i3, it should work on Sway...
Thanks for looking into this for me :+1:

It seems that Sway doesn't fully check the flags of the window then perhaps? At least in i3 I get a floating windows with the correct size that I have given it so I'm not sure what to do here.

It's probably an issue with Sway. I'll reference this issue and hopefully it can be fixed.

Alright. Sounds good :)


Test files mess with code completition

Is your feature request related to a problem?
Codesniffer tests have a lot of definitions that match common keywords and it messes up code completition (in PHPStorm):

names of test functions like func, true, false and null show up along with coresponding PHP keywords/constants/etc., sometimes even before them
in code in global scope variable types get combined (e.g. inside conditions) for variables with the same name (e.g. $container) even when proper phpdoc is defined in the file. In case of the DI container, we have defined method get() with generic return type which does not work (it is random, can't reliably replicate) because of the combined type and the PSR container (used in tests) having method with the same name.

When working with tens of packages, including third-party, working around the problem gets quite annoying. I am currently using drupal/core-vendor-hardening package to remove the test directories, with following config:
	"extra": {
		"drupal-core-vendor-hardening": {
			"squizlabs/php_codesniffer": [
				"tests",
				"src/Standards/Generic/Tests",
				"src/Standards/MySource/Tests",
				"src/Standards/PEAR/Tests",
				"src/Standards/PSR1/Tests",
				"src/Standards/PSR2/Tests",
				"src/Standards/PSR12/Tests",
				"src/Standards/Squiz/Tests",
				"src/Standards/Zend/Tests",
				"src/Standards/Generic/Tests",
				"src/Standards/Generic/Tests"
			]
		}
	},

Describe the solution you'd like

remove tests from released package
or move them into a single tests directory so they are easier to remove via Composer plugin

Additional context

[x] I have read the Contribution Guidelines and this is not a support question.
[x] I intend to create a pull request to implement this feature.

@mabar This is a known issue and will be fixed in the 4.0 release.
See https://github.com/squizlabs/PHP_CodeSniffer/issues/1908 and https://github.com/squizlabs/PHP_CodeSniffer/issues/548 for historical context and the reasoning for the decision to change this in the 4.0 release (and not before).


feat: document merging mechanism for cloud configs

relates to #2735 but also include all other cases for merging so it is clear for the user what will happen

One solution would be to allow the users to run a kairos-agent command and see how multiple files are combined in one. Maybe this is already possible. Let's check.

There is already this: kairos-agent config which prints out the merged configs.
One "strange" thing, in my test it had a top level key:
name: RPI configs

which comes from here and it probably just happens to be the last one that was merged (?)(thought the order of file indicates that others should have taken precedence).  They come with a filepath and a name: field which can be very useful to maybe create a tree of things that have been merged into the final config.

Done through:

https://github.com/kairos-io/kairos-docs/pull/270
https://github.com/kairos-io/kairos-docs/pull/274
https://github.com/kairos-io/kairos-agent/pull/550


Supply fully qualified names for loggers, making those work on Js

Logging frameworks such as oshai/kotlin-logging use invocations like this to automatically assign a logger name:
private val logger = KotlinLogging.logger {}

Behind the scenes, they use KClass.qualifiedName which does not produce a fully qualified name on Js.
This plugin could solve that, e.g.:

A file-level logger in MyFile.kt
package com.example

val logger = Logger(__QUALIFIED_CLASS__) // "com.example.MyFileKt"

A class-level logger
package com.example

class MyClass {
    companion object {
        val logger = Logger(__QUALIFIED_CLASS__) // "com.example.MyClass"
    }
}

In an internal project, I had created an annotation-based compiler plugin to generate logger names (also via a IrElementTransformerVoidWithContext derivative). The relevant qualified class name (including Kotlin's invisible file-level classes) was retrieved like so:
val qualifiedSurroundingClassName = allScopes.reversed().firstNotNullOfOrNull {
    it.scope.scopeOwnerSymbol.owner.qualifiedClassName()
}
if (qualifiedSurroundingClassName == null) {
    reportError("Could not find a qualified class name", call)
    return super.visitCall(call)
}

What should qualified name return in the top-level case on JS and native where there is no concept of an implicit file class like there is on the JVM?

A different way of phrasing that question is: Should we include MyFileKt in the qualified name when we don't support __TYPE__ at the same location?

From a plugin user perspective, I would not differentiate between platforms. For example, I have a YAML server-side logging configuration, which is directly applied at the backend, and transmitted to frontends on JVM and Js platforms. Since there are classes in commonMain which are used across platforms, it is essential to have platform-independent logger names. The above retrieval code does that on the JVM and Js.
Example configuration:
logging:
  com.example.service.FrontendServiceKt: DEBUG # this is a Js/JVM frontend class
  com.example.service.ClientSessionKt: DEBUG # this is a JVM backend class

It seems that qualifiedSurroundingClassName is similar to __TYPE__, except that it does not rely on visitor tracking, but asks the compiler directly at retrieval time. So it always returns a fully qualified class, since a file-level usually exists (at least the way I'm using the compiler via Gradle).

Oh, I forgot: For the above code to work, it needs to be supplemented with these extension functions:
    private fun IrSymbolOwner?.qualifiedClassName(): String? = when (this) {
        is IrFile -> "${fqName.asQualificationPrefix()}${name.substringBeforeLast('.')}Kt"
        is IrClass -> if (isLocal || isCompanion) null else fqnString(true)
        else -> null
    }

    private fun FqName.asQualificationPrefix(): String = if (isRoot) "" else "$this."

So it was actually me creating a name for that fictious file-level class, regardless of whether such a class is actually generated or not.


Problem in works with cmislib

Good morning
We have a problem in works with cmislib
When we trying to determine the contentFile of Document, appears an error ('base64' is not a text encoding; use codecs.encode() to handle arbitrary codecs)
version of python 3.6, version of CMIS 1.1
Can you help to us?
o = open("test.txt", "r")
uniqFolder1.createDocument('TestDocument1321.txt', contentFile = o, contentType='text/plain')

sure, I going to check and I fix it
Thank
Luca
2018-07-19 14:42 GMT+02:00 Anastasja997<EMAIL_ADDRESS>
Good morning
We have a problem in works with cmislib
When we trying to determine the contentFile of Document, appears an error
('base64' is not a text encoding; use codecs.encode() to handle arbitrary
codecs)
version of python 3.6, version of CMIS 1.1
Can you help to us?
o = open("test.txt", "r")
uniqFolder1.createDocument('TestDocument1321.txt', contentFile = o, contentType='text/plain')
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/luca-92/cmislib3/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/ARz9FcoVlXf16eUMPZcSA4W7anTZ9rbtks5uIH7QgaJpZM4VWU5v
.

--
[image: LiveTech s.r.l.] http://www.ilivetech.it/
Luca Marchetti
Data Scientist
M: +39<PHONE_NUMBER> <+393496442672>
E<EMAIL_ADDRESS>| W: www.ilivetech.it
https://www.facebook.com/luca.marchetti.35
https://www.linkedin.com/in/luca-marchetti-9a973b133/
https://plus.google.com/108140177087504250088?hl=it
https://www.instagram.com/luche11o/?hl=it

I upload the version 1.0.8 that now is working. Is a new library and I know
that there are bugs, apology me for this but obviously with your help it'll
become stable
Luca
2018-07-19 16:11 GMT+02:00 Luca Marchetti<EMAIL_ADDRESS>
sure, I going to check and I fix it
Thank
Luca
2018-07-19 14:42 GMT+02:00 Anastasja997<EMAIL_ADDRESS>
Good morning
We have a problem in works with cmislib
When we trying to determine the contentFile of Document, appears an error
('base64' is not a text encoding; use codecs.encode() to handle arbitrary
codecs)
version of python 3.6, version of CMIS 1.1
Can you help to us?
o = open("test.txt", "r")
uniqFolder1.createDocument('TestDocument1321.txt', contentFile = o, contentType='text/plain')
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/luca-92/cmislib3/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/ARz9FcoVlXf16eUMPZcSA4W7anTZ9rbtks5uIH7QgaJpZM4VWU5v
.

--
[image: LiveTech s.r.l.] http://www.ilivetech.it/
Luca Marchetti
Data Scientist
M: +39<PHONE_NUMBER> <+393496442672>
E<EMAIL_ADDRESS>| W: www.ilivetech.it
https://www.facebook.com/luca.marchetti.35
https://www.linkedin.com/in/luca-marchetti-9a973b133/
https://plus.google.com/108140177087504250088?hl=it
https://www.instagram.com/luche11o/?hl=it

--
[image: LiveTech s.r.l.] http://www.ilivetech.it/
Luca Marchetti
Data Scientist
M: +39<PHONE_NUMBER> <+393496442672>
E<EMAIL_ADDRESS>| W: www.ilivetech.it
https://www.facebook.com/luca.marchetti.35
https://www.linkedin.com/in/luca-marchetti-9a973b133/
https://plus.google.com/108140177087504250088?hl=it
https://www.instagram.com/luche11o/?hl=it

Hello.
Unfortunately, the error remained unchanged.
The content of the file "test.txt" is written in Latin.
o = open("test.txt", "r")
uniqFolder1.createDocument('TestDocument.txt', contentFile = o)

#error:
base64' is not a text encoding; use codecs.encode() to handle arbitrary codecs

Another variant:
o = open("test.txt", "rb")
uniqFolder1.createDocument('TestDocument.txt', contentFile = o)

#error:
'bytes' object has no attribute 'encode'

Thank you for your help!

very strange, because I'm tring and it's working with latin text!
If you want, send me the file or copy the entire error log in order to see
in more precise way where gives the error
2018-07-20 13:56 GMT+02:00 Anastasja997<EMAIL_ADDRESS>
Hello.
Unfortunately, the error remained unchanged.
The content of the file "test.txt" is written in Latin.
o = open("test.txt", "r")
uniqFolder1.createDocument('TestDocument.txt', contentFile = o)
#error:
base64' is not a text encoding; use codecs.encode() to handle arbitrary codecs
Another variant:
o = open("test.txt", "rb")
uniqFolder1.createDocument('TestDocument.txt', contentFile = o)
#error:
'bytes' object has no attribute 'encode'
Thank you for your help!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/luca-92/cmislib3/issues/1#issuecomment-406578989, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ARz9FULKZqXQklJULOjy-sgQlADXphphks5uIcVogaJpZM4VWU5v
.

--
[image: LiveTech s.r.l.] http://www.ilivetech.it/
Luca Marchetti
Data Scientist
M: +39<PHONE_NUMBER> <+393496442672>
E<EMAIL_ADDRESS>| W: www.ilivetech.it
https://www.facebook.com/luca.marchetti.35
https://www.linkedin.com/in/luca-marchetti-9a973b133/
https://plus.google.com/108140177087504250088?hl=it
https://www.instagram.com/luche11o/?hl=it

Hi,
I am getting the same error after specifying the contentFile parameter for createDocument() method.
Is there any workaround available for this issue? I am unable to proceed due to this.

Forked & fixed
https://github.com/sgacode/cmislib3

I am having some issues, using windows system
I am using :

Python 3.7
cmislib3==1.0.8

Maybe I have done something wrong in my code:

cmis imports

from cmislib.model import CmisClient

from CMIS_tools import settings

after taking credential and etc

root = repo.rootFolder

someFile = open('test.txt', 'r')

contentFile=base64.b64encode(bytes(someFile.read(), 'base64')))

someDoc = root.createDocument('Test Document', contentFile=someFile)

the error
someDoc = root.createDocument('Test Document', contentFile=someFile)
File "C:\Program Files\Python37\lib\site-packages\cmislib\atompub\binding.py", line 2778, in createDocument
contentEncoding)
File "C:\Program Files\Python37\lib\site-packages\cmislib\atompub\binding.py", line 1790, in createDocument
contentType, contentEncoding)
File "C:\Program Files\Python37\lib\site-packages\cmislib\atompub\binding.py", line 3996, in getEntryXmlDoc
fileData = contentFile.read().encode("base64")
LookupError: 'base64' is not a text encoding; use codecs.encode() to handle arbitrary codecs

Facing the same problem here, trying to write pdf on repo.
The issue is in the funcion getEntryXmlDoc()
def getEntryXmlDoc(repo=None, objectTypeId=None, properties=None, contentFile=None, contentType=None, contentEncoding=None)
...
        fileData = contentFile.read().encode("base64")

contentFile is a binary stream (the result of read(filepath, 'rb'))
read() gives you the bytes of the binary stream
encode("base64") is not applicable on bytes --> ERROR! 'bytes' object has no attribute 'encode'
So, there is no way to store a binary file on repo using createDocument().
P.S. createDocumentFromString() uses createDocument() under the hood
the solution comes from @sgacode :
        fileData = contentFile.read()
        if isinstance(fileData, bytes):
            fileData = base64.b64encode(fileData).decode('utf-8')

@luca-92 could it be merged on master branch?

Moreover, as mentioned above by @Anastasja997 , when trying to write a text document through open(filename,'r') the error is in the .encode('base64') which does not exist in python3 (there is a base64 library now to do this job)


Use image as placeholder

Removes the placeholder div, applies same padding-top technique to the <img> instead
Removes the outer wrapper div making the <picture> the outer most element
Removes reliance on absolute positioning on the image
Fixes bug where Edge 18 was downloading both jpeg and webp with responsive images. This may be a side effect of way the code is now organized.

https://deploy-preview-301--thumbprint.netlify.com/components/service-card/react/
The ServiceCardImage examples seem to be getting cut off. Haven't dug into why though ^

The ServiceCardImage is actually not getting cut off but is now showing the full image. The issue is we're sending it an image that's not in a 8:5 aspect ratio and because the image is no longer using absolute to fill the space created by the parent the full image is showing. There's no way to force the image into that aspect ratio without using height (which we can't because it changes).
I think our options are:

Migrate ServiceCard images as the same time we release Image in website to use ImageServicePicture so we can send it an image in the correct aspect ratio. Maybe we whitelist 8:5 in ServiceCardImage to only accept that size.
Revert ServiceCardImage to not use Image and instead render the image as a background-image (which is how it was built originally to handle these odd image sizes) until we can update it properly via step 1.

Believe this is ready for review. My last couple commits...

Reverted to code prior to merging your branch
Removed containerAspectRatio object-fit example
object-fit now only added if height prop is populated
Updated docs on what containerAspectRatio does now
Updated ServiceCardImage to use 8-5
Updated changelog with breaking changes

Yes, Monday works.


Provide install instructions

What is the correct way of installing savi to a system location?
I copied build/savi-debug to /usr/local/bin/savi.
When I run savi in a directory containing a manifest.savi, I get:
Savi compiler error occured with message "Error opening directory: '/usr/local/core/declarators/meta': No such file or directory". Consider submitting an issue ticket.

When I copy the core directory to /usr/local/core it works. I'd suggest to search for these files in /usr/local/lib/savi/core instead.
Also savi init exists silently, not generating any files and not printing any hints to run savi init bin.

Right now the only expected way to install is with asdf, or with the release tarballs that asdf uses (which get automatically built for each platform by CI - though as you know, FreeBSD CI isn't working yet).
In that case, the tarball can be extracted anywhere - you only need to keep the file structure intact.
asdf is a convenient way to install because it makes shims for easily selecting a specified version, and you don't need to manually manage the PATH to find the savi binary - once you've set up asdf once.

I'd suggest to search for these files in /usr/local/lib/savi/core instead.

I don't want Savi to be searching for its files at an absolute file path - I want it to stay relative to the savi binary so that you can put the files anywhere you choose. At this time, installing in a global directory like /usr/local/bin is not the expected use case.
However, if you want to rearrange the relative paths in the tarball a bit so that it's a bit more friendly for a global installation (while still staying friendly to the main "anywhere installation" use case.

asdf doesn't seem to be easily available for FreeBSD :(
There is only support for ASDF, Advanced Scientific Data Format :).
You probably have code in the Makefile to produce that tarball right? I can run that locally and unpack the tar in a system directory or into $HOME/.local etc.
If I know which directories are needed by savi, I can just cpdup them accordingly.

asdf doesn't seem to be easily available for FreeBSD :(

@mneumann - did you try using the Git method of installation for asdf? Nothing on that page explicitly says FreeBSD is supported, but I do see some PR activity in the asdf repo that mentions FreeBSD, implying that it works on that platform.

You probably have code in the Makefile to produce that tarball right? I can run that locally and unpack the tar in a system directory or into $HOME/.local etc.

Right now, it's only in CirrusCI steps but I could likely move those steps into a Makefile target, then invoke that target from CirrusCI.

If I know which directories are needed by savi, I can just cpdup them accordingly.

Basically, at the time of this writing (which may change in the future) what you need is:

bin/savi (technically this can be named anything)
core (Savi source code for the core library, referenced during compilation)
lib/libsavi_runtime (built LLVM IR and debugging info for the runtime, referenced at the final stage of compilation)


Scroll to particular para inside WebView.

Hey Guys,
I have a url that contains multiple paragraph with p tag. I have to scroll to a particular para when opening the url with WebView.

still searching for solutions


Does not work with AoT compiling

I am using webpack, via the angular2-webpack-starter. Once I add this module AoT compile no longer works. Here is the error I get.
Error: Error encountered resolving symbol values statically. Calling function 'makeParamDecorator', function calls are not supported. Consider replacing the function or lambda with a reference to an exported function, resolving symbol Injectable in<EMAIL_ADDRESS>resolving symbol OpaqueToken in<EMAIL_ADDRESS>resolving symbol OpaqueToken in<EMAIL_ADDRESS>

@GenZodd I grabbed a fresh angular2-webpack-starter and added the logger and was not able to reproduce the error.  I went ahead and updated several dependencies as they were pretty out of date.  Try version 0.0.9 and let me know if you still have issues.

Thanks @Tgure! I cleaned up a few things as well and grabbed version 10 and AoT is now working.


Add CI jobs for other architectures

This commit adds ci jobs for testing on non-x86 architectures. Non-x86
architectures will only run a single python 3.7 job. Now that the
manylinux2014 spec [1] exists we can start publishing wheels for non-x86
architectures. Adding test jobs is the first step towards that goal.
[1] https://www.python.org/dev/peps/pep-0599/

All the new jobs worked except for s390x, I opened an thread on the travis community forum about this here: https://travis-ci.community/t/installing-rust-nightly-via-rustup-on-s390x-fails/7142 it might be a bug in rustup or an issue with the configuration of the s390x travis environment. Until there is a definitive answer I'm going to mark the jobs as allowed to fail and add the jobs for arm and ppc

It looks like switching to ubuntu 18.04 fixed the s390x jobs. So I've updated all the new non-x86 jobs to use bionic just to be consistent between all the environments


API /dcim/interfaces does not handle interfaces on virtual machines correctly

Environment

Python version:  3.6.8
NetBox version:  2.5.9

Steps to Reproduce

Create a virtual machine with an interface.
Use the API to list interfaces (you may specifically search by name or whatever, just ensure that you get the interface at the virtual machine returned).

Expected Behavior
The API returns the virtual-machine with the corresponding data, just like it does for a device if a returned interface belongs to a device.

Observed Behavior
There is no reference to a virtual-machine returned. Instead the interface is returned with a "device: null" entry.
Background
I realized this when I gathered up all interfaces via the API, keeping track of the devices: surprisingly I encountered multiple interfaces with the same name on the same "device", then realized they all belonged to the device "null". I realized, ok, no devices, these are virtual machines - but no virtual-machine data is returned.
In fact, this may even be deeper buried: "device" is a required field when doing a POST for an interface (see api_dcim_interfaces_create in api/docs/) . No way to specify a virtual machine instead, which should be there.

I believe that is working as intended.
/api/virtualization/interfaces/ is what you are looking for, there is a separate API call for the virtual machine interfaces.

Ah, right, I missed that, oups. But...
Should interfaces of virtual machines returned at all when querying for interfaces under /api/dcim/interfaces? To return them under /api/dcim/interfaces (with device = null) when they are under /virtualization/interfaces is not really helping (at least, it fooled me ;-o).
If I query under /api/virtualization/interfaces I only get the interfaces belonging to virtual machines, and not the interfaces under dcim with virtual-machine = null. To have a different behavior in these two trees seems inconsistent.

Should interfaces of virtual machines returned at all when querying for interfaces under /api/dcim/interfaces?

No. Marking this as a bug.


Site Cookie won't get created or modified resulting in strange behaviour

For an external deployed site when I log in as user X, log out and log in as user Y, I get data belong to  user X showing and when I log out I get an error. (not resolved)

If I force delete my site cookie and refresh the app login page the cookie doesn't get created and I cannot log into the site at all. (Resolved)

When I access the same web site  on VPN I don't get the issue. I  only get this from external Url.
I wrote stackoverflow post here:
http://stackoverflow.com/questions/37297925/stange-cookie-behaviour-between-two-external-domain-sites-with-two-identical-asp

What cached data are you seeing? What error are you seeing when you logout?
For point 2 the login page doesn't create any cookies until you actually login. A refresh doesn't do this.. What do you mean by you are unable to login? Are you seeing errors?

Its difficult as it is on production server that I'm not given access to from external url so I can't see errors. On dev server which is supposed to be duplicate of production I don't get the issue.
Having a site work while I'm on a VPN but not when I'm on it is what I don't understand.

We haven't seen such an issue either, we can't think of what would cause it. Logging out deletes the cookie entirely, logging in creates a complete new cookie. Is there a proxy in the way anywhere?
On production could you configure debug logs to a file to try to catch any exceptions?
Again when you say you can't login what do you mean? What happens? Is the login page there? Does the form submit? What happens if the form submits? Are there errors?
There's nothing in our code that cares about IP addresses, so why it works on a VPN is a huge puzzle, unless you've customised login code, or have some sort of IP filtering or blocking in place.

What I don't understand is if I delete all cookies (tested on all desktop browser and mobile phone browser) and want to log in again, it doesn't allow me. The form posts to the server. the Result back is login failed. Running a database trace and nothing. The page refuses to create a new cookie. (On the VPN and working dev server over the internet  it does create cookie)
The signing in is the first step after the post
    [HttpPost]
    [AllowAnonymous]
    [ValidateAntiForgeryToken]
    public async Task<IActionResult> Login(LoginViewModel model, string returnUrl = null)
    {

      ViewData["ReturnUrl"] = returnUrl;
       if (ModelState.IsValid){
             SignInResult result = await _signInManager.PasswordSignInAsync(model.Username, model.Password, false, lockoutOnFailure: false);

SignInResult  has no field to explain why it failed. Just Success Yes or No. No exception is thrown either.
If I restart the IIS worker process I can log in with the  again by immediately clicking the Login button without even refreshing the Login page.
I don't have any IP filtering or settings. I can't explain why it works on VPN but not over internet.
I have customized Identity to make use of Guids instead of the default string
  public class ApplicationUser : IdentityUser<Guid>

Apart from that my code is taken from  the default template.
Everything  works  on the VPN on production server, has the issues over internet. It behaves consistent when at office / or using 3G from my android phone with its mobile browser or my home ISP across all browsers. The VPN/ direct internet behavior is consistent across multiple devices and networks.
So it isn't a client browser proxy.
VPN over internet  works but internet direct  has the issue over the  same https url. A dev server with the same IIS version, Windows version and .net version and exact same site deployed does not have the issue over the internet

I also compare using fiddler between being on the VPN and not after deleting the cookie when logging in as the same user
On the VPN:

ASP.Identity Cookie "MySiteCookie" is POSTed with the login.
First Page after login GET contains the ASP.NET Identity Cookie "MySiteCookie"
-Response body of my first page after login is 7,705 bytes Chunked transfer encoding is used. GZIP

Not on the VPN:

ASP.Identity Cookie is not POSTed with the login although it exists in the site lists of cookies.
A unknown cookie name is posted instead with a name of Nue9LI0c0tY
First Page after login GET contains the ASP.NET Identity Cookie "MySiteCookie"
-Response body of my first page after login is 7,705 bytes, Response Body is 34494 bytes. Not chunked. not HTTP compressed

Can you try enabling debug logging within the app? @HaoK any more ideas? This is pretty bizarre.

Regarding the production server treating cookies differently direct on the internet I cannot explain. One difference I picked up in fiddler is that when on the VPN or to the dev server the content gets encoded/zipped but not to production via the internet. Not sure why IIS behaves this way or if it is related to the problem
Regarding the strange behavior when not on the VPN
It is basically is two issues. The one issue of when deleting the cookie a new cookie doesn't get created. I found the the reason for this was the Identity cookie being in  conflict with Microsoft. AspNet Session
I  found an answer googling here. Setting a dummy session value before login  worked somewhat but I decided to remove using sessions and Microsoft. AspNet Session from my application to eliminate any other conflicts. So I'm not experiencing this when.
The 2nd issue of logging with user X, logging out, log in with user Y and getting user X details still always occurs on VPN. I suspect the cookie isn't updated or something gets cached.
The errors I get sometime User.IsAuthenticated= false and User.GetUserId() .is null
and
await _signInManager.SignOutAsync();

can throw error.
To not get errors I commented out the Authorize attribute and put the code in a conditional if
if (User.Identity.IsAuthenticated) { await _signInManager.SignOutAsync(); }

The issue I get seems to be getting the wrong User.GetUserId. The errors I don't often get it is only sometimes when User.GetUserId() becomes null and as a result  my code failing because it is unexpected
The only similar issue I could find googling solution is to she cookie validateInterval to 0 however that property doesn't exist anymore from what I could find in Identity 3 RC1

Why is the identity cookie in conflict with the session cookie?  As a first step, can you revert any changes you have to the cookie properties in general, the defaults should be fine in general, and would be a good thing to remove as possible causes of any weirdness you are seeing.

I can only conclude the issue is to do with caching. This might explain why over VPN I don't get the issue. There must be a server caching https server traffic.
I log in as user X. Access some page. Then log out.
Log in as user Y. then all the pages that user X accessed previously shows user X details instead of user Y.
If I access a page that user X hasn't accessed  it will show user Y's page. Errors a get is because the cached page contains user X's details but the server know I am logged in a user Y.
For example when I log out. Because the page thinks I am user X but the server thinks I am user Y I get a subsequent error

closed as proxy/cache related


Python intellisense is showing nonsense

Environment data

Language Server version: v2023.3.20
OS and version: ubuntu 22.04
Python version (& distribution if applicable, e.g. Anaconda): 3.9.16 (miniconda)

Code Snippet

x=[1,2,3]
x[]

Repro Steps

New a py file
Type x=[1,2,3]
Type x[

Expected behavior
Do not give any suggestions ( or only intergers for list, keywords for dict)
Actual behavior
Showing a lot of irrelevant  suggestions

Duplicate of https://github.com/microsoft/pylance-release/discussions/2599

@zhong-yy , btw, you should go up vote that discussion item, since we use vote counts to help prioritize enhancements.


Rewrite Project

Rewrite the project from the ground up.  Closes #596
Steps

[x] Update data extractor to the latest version and fix known issues.
[x] Fix issues in java_string.
[x] Rework interface of valence_nbt to support redesign of valence_protocol.
[x] Remove valence_generated.

Computing everything statically doesn't work in a world where more and more registries are networked.
Compile times were too long. We can generate stuff from the JSON at runtime if we need to.

[ ] Redesign valence_protocol and update to 1.20.4.

Reorganize packets into a single module. (Minecraft shares some packets between protocol stages now, so this accommodates for that).
Introduce McRead and McWrite traits similar to PacketByteBuf in vanilla.
Remove bespoke derive macro for "encodable" and "decodable" types and write all the packet impls out manually.
Remove dependence on valence_generated, bevy_ecs, valence_math.
Consider merging valence_text and valence_ident into valence_protocol since we don't have valence_generated anymore.

[ ] Update the packet inspector and get it to a usable state.

Create "packet registry" in valence_protocol so the inspector can access all packets programmatically.

[ ] Rewrite main server crates.

Replace bevy_ecs with evenio. Use event driven design described in #596.
Use monoio + ureq on Linux and MacOS, tokio everywhere else.
Handle switching between configuration phase and play phase.
Get basic chunk stuff working at least.
Combine the miscellaneous crates together like weather, player_list, boss_bar, registry, etc.

[ ] Pick a linear algebra crate (glam, vek, ultraviolet, something else?)
[ ] Port some of the examples over.
[ ] Update valence_anvil (or leave that for follow up?)
[ ] Update CI scripts.

Given that you are making a bunch of changes and performance is important, have you considered adding benching to CI? I feel like I have seen Rust projects that have good performance regression detection in CI, but I might need to do a bit of searching.
This would be nice because performance would be repeatable unlike

if I have singleplayer Minecraft open on my machine at the same time I'm running bench_players, the performance actually improves.

https://github.com/valence-rs/valence/issues/323

@andrewgazelka I'm open to the idea, but it seems like a lot of work to do it correctly. For starters we would need dedicated CI runners and a decent suite of benches. Definitely something to think about after the rewrite is finished.

The criterion crate sounds like it may be useful here, unless ya'll have your own favorite (micro)benchmarking tools.

@andrewgazelka I'm open to the idea, but it seems like a lot of work to do it correctly. For starters we would need dedicated CI runners and a decent suite of benches. Definitely something to think about after the rewrite is finished.

That makes sense. Why would you need dedicated CI runners, though? Do you think performance would not be predictable using the default CI runners, which run in a VM on the same machine as other runners?

If consistent environments are available (e.g. on your own machine), it may be worth it to use a microbenchmarking crate like criterion to check for performance regressions.

For the lin alg crate, I'd say either glam or nalgebra.

Closing this since I'm not happy with the direction this was going and it's too much to do in one PR anyway.


Create CODE_OF_CONDUCT.md

Related Issue
Changes you made :

Couldn't find the Code of Conduct on the repository, hope this helps!

Describe the changes :

Check list :

[x] I have followed code style of this project.
[x] I have read the CONTRIBUTING document.
[ ] All the aspects mentioned in the issue are covered.
[x] The website is properly working after my changes.
[x] 🌟 ed the repo

Type of change :

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Output Screenshots :

Hey @Arun9739! Thank you for guidance!!


###Material Stepper

Bug, feature request, or proposal:
I am using angular material stepper , so i have use stepper in 5 steps, all are the steps including forms like textbox, i have use angular reactive forms.When i hit on 2nd step by default display all required fields validations
What is the expected behavior?
Expectecd is Display all validations when i touch any one of textbox.
What is the current behavior?
I have 5 steps in mat-stepper, each steps having form fields.I am doing this using reactive forms.So Each steps i have divided into formArray, for eg. 1st step->controls.formArray[0] etc.
In this always call all the required validations.
And I have already set [linear]="true"
What is the use-case or motivation for changing an existing behavior?
Use Case is , 1st i want to validate all the fields in stepper when i touch in textbox.
I have use formArray for each steps, so 5 steps five FormArray.I want to validate all the fields using formArray and also in stepper
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
angular 6,angular material 6.4.6

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.
(this is a canned response)

why i am troubleshoot that questions in stack overflow?
i have problem on angular material stepper,so that i am posting here


tf.math.is_non_decreasing outputs incorrect result when input is an uint tensor

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.14.0
Custom code
Yes
OS platform and distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
When giving tf.math.is_non_decreasing a decreasing tensor with dtype=uint32. This API incorrectly output True instead of False.
Standalone code to reproduce the issue
import tensorflow as tf
x = tf.constant([10,9], dtype='uint32')
print(x)
out = tf.math.is_non_decreasing(x)
print(out)

Relevant log output
No response

Hi @drewshark ,
Thanks for reporting this. The issue persists with all unsigned dtypes i.e 'uint8','uint16','uint32','uint64' which are causing overflow (i.e negative values being overflowed to positive end as unsigned integers can't hold -ve numbers) when the input (x) is of non_decreasing type.
The overflow is happening from this line of code.
https://github.com/tensorflow/tensorflow/blob/4dacf3f368eb7965e9b5c3bbdd5193986081c3b2/tensorflow/python/ops/check_ops.py#L1947
We will check and submit a fix for this. Thank you!


Add a move_to function

Add a move_to function that makes use of the bresenham line algorithm to move diagonally

The scroll functions could probably make use of this for less code overall.


Namechange MeshData -> PixelData

What do you guys think @skalarproduktraum this would make you happy?
@krzysg ?

Do we need that change? I mean, I do not have any objections to change it - if this is MeshData, PixelData or even GridData it's completely OK for me. (It can be even IdoNotGiveAShitAboutPixelDataSinceIuseApr class ;-) ).

hahahaha, yeah this is more of a response to Ulriks dislike + it woudl also align with the language we use in the paper, and avoid any ambiguity.

Go for PixelData. It is pixel data you're representing and therefore the least confusing naming for external people 👍

Everything seems to work fine after changes so closing.


Feature - Added new unit_test_writer

Description
This pr introduces a stand alone function which allows for basic unit test code creation, via utilisation of a simple config dict, and uses CSVs files as inputs. The output after running is a new .py file containing the script for the testing.
Improvements available on request:

I can add extensive commenting to the file if required
Currently the script outputs a generic test function at the end of the .py file, this can be improved if I have a template of what's preferable
The column_type override is currently limited to 'string' & 'float', this can be extended to other types
A different output directory can be added for the outputted .py file, currently it goes to the same location as the input files folder

Peer review
Any new code includes all the following:

Documentation: docstrings, comments have been added/ updated.
Style guidelines: New code conforms to the project's contribution guidelines.
Functionality: The code works as expected, handles expected edge cases and exceptions are handled appropriately.
Complexity: The code is not overly complex, logic has been split into appropriately sized functions, etc..
Test coverage: Unit tests cover essential functions for a reasonable range of inputs and conditions. Added and existing tests pass on my machine.

Review comments
Suggestions should be tailored to the code that you are reviewing. Provide context.
Be critical and clear, but not mean. Ask questions and set actions.
These might include:

bugs that need fixing (does it work as expected? and does it work with other code
that it is likely to interact with?)
alternative methods (could it be written more efficiently or with more clarity?)
documentation improvements (does the documentation reflect how the code actually works?)
additional tests that should be implemented

Do the tests effectively assure that it
works correctly? Are there additional edge cases/ negative tests to be considered?

code style improvements (could the code be written more clearly?)

Further reading: code review best practices

@gavindeeppahl  & @AnneONS:
Here are a couple of my thoughts:

Refactor main() function to accept parameters instead of hard-coding them. This makes the function more flexible and reusable.

Use argparse for command-line arguments: Add argparse to handle command-line arguments, allowing users to specify their own parameters when running the script. I guess this is only necessary if you want to call from command line and not script.

Remove if __name__ == "__main__" command function and call it with their own arguments from another script.

I'm assuming you'd want to run with users calling own arguments from another script? So you can ignore argparse example.
import argparse
import logging
from pathlib import Path
import json

def main(csv_path: str, files: list, function_name: str, column_type_override: dict) -> None:
    """Initialise configuration and process CSV files for unit testing.

    This function sets up the configuration with paths, filenames, and function names,
    and then calls `process_dataframe` to handle the CSV files and generate the test
    code.

    Parameters
    ----------
    csv_path : str
        The path to the directory containing the CSV files.
    files : list
        A list of filenames to process.
    function_name : str
        The name of the function to generate tests for.
    column_type_override : dict
        A dictionary to override column types.

    Returns
    -------
    None
    """
    config = Config(
        csv_path=csv_path,
        files=files,
        function_name=function_name,
        column_type_override=column_type_override,
    )

    process_dataframe(config)

def run_from_command_line():
    parser = argparse.ArgumentParser(description="Process CSV files for unit testing.")
    parser.add_argument("--csv_path", type=str, required=True, help="Path to the CSV files directory.")
    parser.add_argument("--files", nargs='+', required=True, help="List of CSV filenames.")
    parser.add_argument("--function_name", type=str, required=True, help="Name of the function to generate tests for.")
    parser.add_argument("--column_type_override", type=str, required=True, help="Column type overrides in JSON format.")

    args = parser.parse_args()

    # Convert column_type_override from JSON string to dictionary
    column_type_override = json.loads(args.column_type_override)

    main(args.csv_path, args.files, args.function_name, column_type_override)

# Example usage:
# if __name__ == "__main__":
#     run_from_command_line()

Usage Instructions
Option 1: Running from the Command Line
Users can run the script from the command line with their own parameters:
python -m rdsa_utils.helpers.unit_test_writer --csv_path "path/to/csv" --files "input1.csv" "expected_output.csv" "fail_output.csv" --function_name "new_function" --column_type_override '{"string": ["period", "reference"], "float": ["602"]}'

Option 2: Creating a Custom Script
Users can create their own Python script to call the main() function with custom arguments:
from rdsa_utils.helpers.unit_test_writer import main

main(
    csv_path="path/to/csv",
    files=["input1.csv", "expected_output.csv", "fail_output.csv"],
    function_name="new_function",
    column_type_override={"string": ["period", "reference"], "float": ["602"]}
)

This approach provides flexibility and allows users to customise the parameters as needed.


fix: several fixes and a new simplest C test

fix: exclude common directories
fix: logging reports the correct file location on Python 3.8+
fix: SKBUILD_PROJECT_NAME should be canonical
tests: move test examples to project subdirectory
tests: add test for simplist C-only like in README

@jcfr, @thewtex I hate writing things like 24ed940c07b8ce7796deb126496dac2300486e61 - at least it's supposed to be fixed in PyPy 7.3.10. Would it make sense to add a workaround in FindPython for PyPy < 7.3.9 though?
TL;DR: FindPython considers SOABI to be the source of truth, and it's been wrong in PyPy for years - it's missing the platform tag - while the other two related variables (SO & EXT_SUFFIX) are fine. PyPy will not load something without the platform tag, so you can't load an extension made from FindPython without renaming the extension.
https://foss.heptapod.net/pypy/pypy/-/issues/3816

It turns out this has been wrong so long it masked an issue with packaging, which means pip won't support correcting this value, so the change had to be reverted. Packaging can special case PyPy (just like it does for CPython), and then this won't matter, but for now, SOABI is unreliable and can't be used to compute the correct extension. I believe it either needs to be special cased or fixed in CMake as well. As it is, CMake's FindPython can't produce a binary that PyPy can load as an extension.
I wrote the workaround for pybind11 a year or two ago, I wish I'd pushed to get it fixed then.

Opened https://gitlab.kitware.com/cmake/cmake/-/issues/24121 (turns out I'd opened one two years ago too, and a few other people have opened one since).


Markdown docs for both gjirafa and malltv bid adapters

Adding @willviles and @agon-qurdina to confirm the changes to the Gjirafa docs.

LGTM!

These adapter released with 4.8

It was missing the pbjs:true header field

Just created new PR (#2392) for this.


ENH: optimize: c1 and c2 of scalar_search_wolfe to cg and newton-cg

Reference issue

closes #19075
follow up #18977
What does this implement/fix?

As requested exposes  parameters c1 and c2 for cg and newton-cg as well.
Additional information

comments hopefully addressed

Have fixed the incorrect c1 and moved the location of the check

LGTM thank you @j-bowhay


Expose dict values as list

Just adds to_list() methods to DotDict and DataIO so the dictionaries can be case to value lists. The order is explicitly not guaranteed, but it winds up being the order items are added to the underlying dict.
As requested in #38.
Closes #40.

======================================================================
ERROR: test_connection (unit.test_io.TestDataIO.test_connection)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/pyiron_workflow/pyiron_workflow/tests/unit/test_io.py", line 68, in test_connection
    self.input.x = self.input.y
    ^^^^^^^^^^^^
  File "/home/runner/work/pyiron_workflow/pyiron_workflow/pyiron_workflow/io.py", line 76, in __setattr__
    self._assign_value_to_existing_channel(self.channel_dict[key], value)
  File "/home/runner/work/pyiron_workflow/pyiron_workflow/pyiron_workflow/io.py", line 92, in _assign_value_to_existing_channel
    channel.connect(value.channel)
  File "/home/runner/work/pyiron_workflow/pyiron_workflow/pyiron_workflow/channels.py", line 126, in connect
    raise ChannelConnectionError(
pyiron_workflow.channels.ChannelConnectionError: x (InputData) and y (InputData) share a generic type but were not a valid connection. Check channel classes, type hints, etc.

Riiight. Because the two tests each had the same name, only one of them was getting run and it wasn't this one. This actually does break expectations -- trying to assign input to input should raise an error.


feature: Feature set management allow to add new variable

Feature Request
Describe the Feature Request

Once the created feature set is saved, new variables cannot be added but can be deleted. So, allow to add variables

This is already implemented


Can't set breakpoint when use babeljs

FYI

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/21487255-can-t-set-breakpoint-when-use-babeljs?utm_campaign=plugin&utm_content=tracker%2F195817&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F195817&utm_medium=issues&utm_source=github).

Fixed in 0.10.2.
Feel free to reopen this issue, if I'm wrong here.


Material View poor Tab Bar rendering

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Framework7 Version:
▶ bower info framework7
bower cached        https://github.com/nolimits4web/Framework7.git#1.4.2

Platform and Target: CORDOVA/ANDROID

What you did
Took your tab-bar template (http://framework7.io/examples/) and used it as a base for the app
Expected Behavior
Content on the page is offset to not overlap with the tabs above it.
Actual Behavior

As you can see from the screenshot, the tab bar completely obstructs the title and the content. On iOS this isn't an issue because tab bar renders on the bottom.

On a related note, this tab bar on Android looks almost identical to a nav bar, yet iOS ones are distinct. Material design guidelines allow for a similar nav bar on Android as well, see here: https://material.google.com/components/tabs.html#tabs-usage. Note the use of images, is this something that can be added to Android version?

It works fine in the Kitchen Sink example, there is probably an error in your code.
Can you provide some runnable example?
You can use one of these templates for that:

iOS JSFiddle/Codepen
Material JSFiddle/Codepen


Unable to use transformation matrices outside TouchDesigner

Hello again Darien,
I have been experimenting with TDAzureMerger a bit more and got stuck in trying to use the generated transformation matrices outside of TouchDesigner for perspective alignment.
I have successfully aligned 3 Azure Kinects using TDAzureMerger. I extracted to a .py file each of the transformation matrices available at project1>AzureCalibration>Azure[N]>mtx (where N is either 2 or 3, since Azure1 is the master and does not need any transformation).
I have exported point clouds from my Kinects using Open3D and loaded them to CloudCompare (a software similar to MeshLab).
However, when I apply a transformation to the subordinate perspectives, the alignment is incorrect and does not match the quality of the one achieved inside TouchDesigner...
Individual Point Clouds inside CloudCompare:

Manual Alignment using CloudCompare Tools:

Automatic Alignment using TDAzureMerger:

Application of the transformation matrix generated by TDAzureMerger on the original subordinate point cloud inside CloudCompare:

NOTE: the clothes are different because the images were taken on different days, but the setup did not change in the mean time.

How can I export the transformation matrices correctly? Is there an additional transformation done inside TouchDesigner that I am missing?
Thank you for your time.

Hi Filipe,
If you use the matrices generated by the TDAZureMerger that should be it indeed. There's nothing else being applied to the data beyond those transforms, so it sounds like you are doing the right thing.
I'm not sure of what CloudCompare does if you pass those matrices. It should be a simple linear transformation, so they should match one to one. I don't have at the moment kinects to try this out unfortunately. What are the differences number-wise between the matrices generated from CloudCompare and the TDAzureMerger?

@FilipePires98
Did you manage to solve this? I am planning on using Darien's component to do registration inside Unity.

Hi @Zulex and @FilipePires98
So sorry I missed this message by a long shot! I don't have notifications enabled for Github and cleaning up stuff for the coming year I realized I had these in here.
The used case with CloudCompare is interesting because in my case it works fine. What I did for one production is having frontal calibration done by my tool and calibration of the back camera with CloudCompare using as reference the same one. Once exported, the matrix of CloudCompare used as transform for the point-cloud of the kinect gives correct results. So I do not think that the coordinate spaces or the matrix are dissimilar. There is perhaps something else going on.
I'll be happy to check this on a call or if you send me a patch after the holidays. I'll be back in the studio around 14th January

Hi @DarienBrito ,
Thank you for your message. Unfortunately, I am no longer working with TD for Azure Kinect calibration. It is possible that the cause of this was related to some parameter(s) inside CloudCompare, idk.
Thank you anyways for making yourself available, but by this time it will no longer justify spending more time on this issue. If you wish, you may close it. :)

Alright! Thanks for letting me know Filipe. Happy new year and good luck with it all!


All functions should be stored in the same GitHub repository? Or at least in the same place. Meaning, a uniquely named function should only exist in one place, no matter how many projects use it.

This way, a change or improvement in one piece of code will be able to propagate to other projects. If multiple versions are desired, then just rename the function!

Each person in a lab could have their own branch of the common code. Then, when  a paper is published from that code, they merge it with the lab's main code (and also save the export from the PGUI).


Fix memory leak bug when directbufferedinput is enabled

Fix issue #8204
The root cause is the speculative fetch of DirectBufferedInput. Like if we have a SQL:
select A where B=0
It may prefetch ColumnChunk A where all B acually isn't 0. Then when the task finished, the fetch is still pending and hold the memory block.
The solution is to wait all pending requests finish before descror return.

@rui-mo Can you review?

@mbasmanova @xiaoxmeng should we just wait in the destructor of DirectBufferInputs or should we add close() function to the class chain? I do think we should call the close function to make sure the resources are released before they are destroied. If so we need to add
TableScan::close() -> DataSource::close() -> HiveDataSource::Close() -> SplitReader::close() -> Reader::close() -> ParquetReader::close() -> ReaderBase::close() -> BufferedInput::close() -> DirectBufferedInput::close()
Then we need to take a look of other DataSources, Readers and BufferedInputs. TableScan::close is added by PR #8181 and is called when driver finish current pipeline.

@zhztheplayer

@rui-mo  will continue this PR to add the close function to the call chain, as well as UT.

Let's close the PR since it has several conflicts, Rui will start a new one later.


Wipe stored secrets when "save in keychain" is unchecked

Resolves DESKTOP-863.
Not sure how I should approach error handling and user feedback here. Copied a console.log handler from another action. Thoughts?
:eyeglasses: @keybase/react-hackers

Ok, I've updated with your suggestions, thanks!
:repeat: @chrisnojima @MarcoPolo

👍


Reduce test execution time. #1452

pinpoint rpc test execution times (42.449s -> 21.249s)
ping @jaehong-kim

reviewed-by @jaehong-kim


Please Added multiple Client ID, Client Secret, and Refresh Token

iam request for add multiple Client ID, Client Secret, and Refresh Token in index.js
example :
Account 1
client_id: "202264815644.apps.googleusercontent.com",
client_secret: "X4Z3ca8xfWDb1Voo-F9a7ZxJ",
refresh_token: "", // 授权 token
Account 2
client_id: "202264815644.apps.googleusercontent.com",
client_secret: "X4Z3ca8xfWDb1Voo-F9a7ZxJ",
refresh_token: "", // 授权 token
.....
.....
Account 10
client_id: "202264815644.apps.googleusercontent.com",
client_secret: "X4Z3ca8xfWDb1Voo-F9a7ZxJ",
refresh_token: "", // 授权 token
i think this is solution for gdrive limit
so if account 1 limit, automatically change to account 2
thanks
regard

have a pull request with this updade @rikyuking , from the other code gdindex, um can get the code/array to use random service accounts, multiples, on onde index.
const serviceaccounts = [
{
    "type": "service_account",
    ...
    "client_x509_cert_url":...
},
{
    "type": "service_account",
    ...
    "client_x509_cert_url":...
},
... many as need
];
const randomserviceaccount = serviceaccounts[Math.floor(Math.random()*serviceaccounts.length)];
...

client_id: "",, #important keep empty for service accounts
client_secret: "", #important keep empty for service accounts
refresh_token: "",, #important keep empty for service accounts
service_account: true,
service_account_json: randomserviceaccount,

remeber, u need to use the pull request with the js updated to use this modifications.
using multiples service accounts, like u need.


adds Sans font

this PR adds option for sans font, to be used, not just mono.

In my opinion, users should be able to add various fonts on their own.
Instead of including an extra font in the template out of the box, it would be more beneficial to provide a documentation post explaining how to add different fonts. This approach helps reduce unnecessary bandwidth usage for downloading additional fonts.
Besides, fonts are typically added inside src/layouts/Layout.astro in this repo.


json files don't get installed with pip install .

  File "/tvm-fork/python/tvm/relay/backend/contrib/soma_dory/codegen.py", line 201, in soma_dory_compiler
    converter = onnx_manager(codegen.dory_graph, config_file, '')
  File "/usr/local/lib/python3.8/dist-packages/dory/Hardware_targets/Diana/Diana_TVM/HW_Parser.py", line 42, in __init__
    with open(os.path.join(file_path, "pattern_rules.json")) as f:
FileNotFoundError: [Errno 2] No such file or directory: '/usr/local/lib/python3.8/dist-packages/dory/Hardware_targets/Diana/Diana_TVM/pattern_rules.json'

I installed this with pip install . from the root of the repository
Current workaround is to install with pip install -e
@ABurrello maybe this is due to an issue with setup.py?
CC @maartenvds

I will investigate on it. Also I update the README to install with -e.


Setting value of the slider.

Hi,
Thanks for a great addition to Semantic UI.
Is there a way to dynamically change the value of the slider? I would like to animate 10 pixels per second until the user grabs the slider. After the user release the slider it will start auto sliding again.
Edit: I could re instantiate the slider over and over but not sure thats a good way.
$('#my-range').range({         min: 0,         max: 10,         start: 5     });

@nilsi I agree, that would be a hack and not the best way to set the value.
I just updated the code so that you can now set the value like so:
$('#range').range('setValue', 17); // Sets slider with id 'range' to value 17

I have documented this in readme as well.
Thanks!

Thanks a lot! Works great. A small feedback would be to change it to $('#range').range('set value', 17); since that is more Semantic-ish. :) Closing this.

@nilsi  Thanks for pointing that out! I changed the query to set value.


Update PPP for medajet

Codecov Report

Merging #298 (f964ede) into master (0546c50) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #298   +/-   ##
=========================================
  Coverage     73.64%   73.64%
  Complexity      760      760
=========================================
  Files           105      105
  Lines          2436     2436
  Branches        325      325
=========================================
  Hits           1794     1794
  Misses          519      519
  Partials        123      123

Continue to review full report at Codecov.

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


Create new table for zabbix monitoring

I reconsidered that is better to have a table also for zabbix data is too messy to recover info dynamically.
New table will be:
customization_monitoring_zabbix
Columns:
id
customization_id (reference to the customization table)
host (string)
item (string)
history (string)
sortfield (string)
limit (integer)
description (string)
chart_type (string)
unit (string)
As always Db and java

Done


Support omitting time values in SimpleSegmentNameGenerator

Related to https://github.com/apache/pinot/issues/11649
We use the normalizedDate segment name generator for an append table w/ a time column. The generators add a min/max time value to the segment name (ex. example_table_2023-09-22_2023-10-02_46). We had a user backfill their batch jobs overwriting existing data. Some of the upstream data had changed as part of this backfill which caused the min/max time value in the segment name to also change. This caused inconsistent data since some new segments didn't replace thier old ones.
This PR adds param omit.timestamps.in.segment.name to the SimpleSegmentNameGenerator to omit time values from the segment name. I've only added this to the simple generator since it doesn't seem intuitive to have the normalizedDate generator omit timestamps. We plan to create segments w/ this generator by including the execution date for the batch run (ex. example_table_20230928_46) since it will give us a consistent set of segment file names as long as the # of input files is the same.
I've updated the unit tests. It seems like the best place to write an end to end test might be in https://github.com/apache/pinot/blob/master/pinot-plugins/pinot-batch-ingestion/pinot-batch-ingestion-standalone/src/test/java/org/apache/pinot/plugin/ingestion/batch/standalone/SegmentGenerationJobRunnerTest.java Let me know if that's desired.
Long-term, it seems ideal that we'd use https://docs.pinot.apache.org/operators/operating-pinot/consistent-push-and-rollback for re-running segment creation on a given day so users aren't contrained on how the input data is structured.
cc @Jackie-Jiang

Codecov Report

Merging #11650 (77fbb58) into master (4226ea0) will increase coverage by 0.00%.
Report is 1 commits behind head on master.
The diff coverage is 81.25%.

 Click here to find out more.

Files Changed
Coverage Δ

...ache/pinot/segment/local/utils/IngestionUtils.java
28.32% <0.00%> (ø)

...not/spi/ingestion/batch/BatchConfigProperties.java
0.00% <ø> (ø)

.../apache/pinot/spi/ingestion/batch/BatchConfig.java
70.00% <80.00%> (-1.02%)
:arrow_down:

...t/spi/creator/name/SimpleSegmentNameGenerator.java
92.30% <87.50%> (+1.39%)
:arrow_up:

...tion/batch/common/SegmentGenerationTaskRunner.java
70.76% <100.00%> (+0.92%)
:arrow_up:

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

cc @snleee @swaminathanmanish

@swaminathanmanish thanks for reviewing! updated the description to include motivation and responded to your comment

Related to #11649
We use the normalizedDate segment name generator for an append table w/ a time column. The generators add a min/max time value to the segment name (ex. example_table_2023-09-22_2023-10-02_46). We had a user backfill their batch jobs overwriting existing data. Some of the upstream data had changed as part of this backfill which caused the min/max time value in the segment name to also change. This caused inconsistent data since some new segments didn't replace thier old ones.
This PR adds param omit.timestamps.in.segment.name to the SimpleSegmentNameGenerator to omit time values from the segment name. I've only added this to the simple generator since it doesn't seem intuitive to have the normalizedDate generator omit timestamps. We plan to create segments w/ this generator by including the execution date for the batch run (ex. example_table_20230928_46) since it will give us a consistent set of segment file names as long as the # of input files is the same.
I've updated the unit tests. It seems like the best place to write an end to end test might be in https://github.com/apache/pinot/blob/master/pinot-plugins/pinot-batch-ingestion/pinot-batch-ingestion-standalone/src/test/java/org/apache/pinot/plugin/ingestion/batch/standalone/SegmentGenerationJobRunnerTest.java Let me know if that's desired.
Long-term, it seems ideal that we'd use https://docs.pinot.apache.org/operators/operating-pinot/consistent-push-and-rollback for re-running segment creation on a given day so users aren't contrained on how the input data is structured.
cc @Jackie-Jiang

Thanks for adding detailed notes. Did you consider using FixedSegmentNameGenerator, where you can exactly specify the segment name that you want ?
BatchConfigProperties.SegmentNameGeneratorType.FIXED:
        return new FixedSegmentNameGenerator(_segmentName);

Yes, we still want to use the global sequence ID which isn't supported by the FixedSegmentNameGenerator. Another option here is to update FixedSegmentNameGenerator to support appending the sequence ID. Let me know if that approach is preferred.

Yes, we still want to use the global sequence ID which isn't supported by the FixedSegmentNameGenerator. Another option here is to update FixedSegmentNameGenerator to support appending the sequence ID. Let me know if that approach is preferred.

Thanks ! Lets go with SimpleSegmentNameGenerator itself.  FixedSegmentNameGenerator can just be for a specific names set by the user, without the other attributes.

@swaminathanmanish updated the name. let me know if there's other feedback that needs addressing.

@Jackie-Jiang thanks for the review. addressed your comments.

ah thanks for catching those @Jackie-Jiang. i fixed them up.

pushing an empty commit to trigger CI again. the test failure seemed unrelated.


Added js-cjs-async-await template

This pull request makes the following changes

Fixes collabo-community/contributor-issue-report#8

General checklist

[x] File or folder now contains changes as specified in the issue I worked on
[x] I have linked the issue I worked on to this pull request submitted by me

Testing checklist
Replace this text with the testing checklist from the issue this pull request fixes.
Ping @code-collabo/node-mongo-api-boilerplate-templates

The P.R. is ready @collabo-community/review-backend-api

The P.R is ready for review @collabo-community/review-backend-api

@Danbaba1 are all endpoints now working?

@Danbaba1, you have not updated the

@Danbaba1, please remove the .vscode folder from the template.

@collabo-community/review-cli, @Ifycode, @Festusali, @tylerjusfly.
I'm okay with the changes, Please help take a look so it can be merged as necessary.

@all-contributors add @Danbaba1 for code


In create list, filter for community vs core packages

Great package!  My typical use is for troubleshooting package conflicts, so I'd really like to be able to quickly create a bundle for all the packages that aren't core.
Shift-select to do a whole bunch at once would also be pretty handy!

+1 - Also wish we could search when creating a .package-switch.cson file.

This is a good feature. I will try to add it in the next versions.


how to use the Integration ,when i add navidrome is not thing to do..

In this initial version, we only listed the media on HA. Like Radio Browser, but soon I'll implement it as a device to obtain some status. I still need to research a bit about Navidrome's Jukebox to see if it will be possible to control it.

This is a great project! Looking forward to seeing it come to fruition! 👍🏻

Yes, they can now be seen in the media library, but it indicates that the music is not compatible. We hope it can start up normally so that our navidome server can be perfectly integrated into homeassistant

Thanks for the integration, it looks very promising!


Uncaught (in promise) DOMException: Registration failed - A subscription with a different applicationServerKey (or gcm_sender_id) already exists; to change the applicationServerKey, unsubscribe then resubscribe.

I am getting the above error on my server.

Hi, try the latest release.

I am getting same error, what should I do?


Remove extra header for beta token limit

Expanded token window is out of beta
https://x.com/alexalbert__/status/1825920737326281184

Thank you for bringing this up! Removed in v1.3.43


SwiftUI 4 NavigationStack naming conflict

I'm using MacOS Ventura Beta with SwiftUI 4. The problem is that Apple introduced NavigationStack in SwiftUI 4 and it conflicts with the NavigationStack<T: NavigationCoordinatable> from stinsen library. Hence i'm getting the following error:
Generic type 'NavigationStack' specialized with too few type parameters (got 1, but expected 2)
As a quick fix i think you could rename NavigationStack to SNavigationStack or StinsenNavigationStack

This is simply fixed in https://github.com/rundfunk47/stinsen/pull/75

As @LePips says, this is temporary fixed with a namespace. However, this item should still be open - a long term solution should be to rename it completely. Any suggestions? I'm not a huge fan of adding a "Stinsen"-prefix...
Maybe just Stack would be fine?

I think Stack would be problematic both in usage and in context. It would conflict with a basic Stack if a developer ever had one since NavigationStack is already public. For context, it's usually better to have the name indicate its domain or purpose, especially since it's for navigation.

I totally agree with @LePips , naming it just Stack would certainly create confusion

With the release of Xcode 14 at time of writing, this is become an immediate "issue". I'm not bothered too much by my single usage of Stinsen.NavigationStack but something should be decided.
@rundfunk47 Have you been able to work on anything in regards to the latest comment in https://github.com/rundfunk47/stinsen/issues/29? If not, that's completely fine.

A more pressing issue is that the navigation title styles (.inline, .large) are no longer consistent on push with Stinsen when targeting iOS 16. They will default to .large.


Update the documentation regarding the new Mapsui.Native package

The quickstart guides for WPF & Android now require the Mapsui.Native package instead of the Mapsui base package (the latter previously also included the native UI parts).

Your first commits adds a lot of documents. Looking at a few it seems that they should have there in the first place. A lot of documents are missing in the current api on github (they used to be there). Do you know what is happening?
Even Mapsui core is missing

Publishing the new api docs is supposed to be an automatic process.

Discussed this with @janusw. It seems the files added in the PR were supposed to be here in the first place. Apparently something went wrong in the past building the docs. Could be on a local machine or could be on the build server.


filter metadata on field date/datatime

Hello,
When I put data in the db, I store embeddings  and metadata like filename, ext, size and modification date.
In some cases, I want old files or new files close to an query embedding.
When I search documents with python sdk, I use embeddings and some other criteria like the modification date.
Today the filter support only meta_contains.
It could be nice to support comparison like meta["date"] <= ... or meta["date"]> ...
For now, I use this workaround:
generate a big meta_contains dict

I use Psycopg3 :
from pgvecto_rs.sdk import PGVectoRs, Record
from uuid import UUID

# doc: https://github.com/tensorchord/pgvecto.rs-py/blob/main/src/pgvecto_rs/sdk/client.py
class Client:

    def __init__(self, config: dict, table: str):
        url = f'postgresql+psycopg://{config["username"]}:{config["password"]}@{config["host"]}:{config["port"]}/{config["database"]}'
        self.client = PGVectoRs(
            db_url=url,
            collection_name=table,
            dimension=1024,
            recreate=False
        )

It's hard to support all kinds of filter. If you prefer such interface, SQLAlchemy might be a better choice which has full support of such primitives


[BUG]: Practice Snippet Getting Loaded Twice

Description

On the Practice page, the snippet is getting loaded twice and it is causing a glitchy change from one snippet to another.
A potential fix would be to get only one snippet or implement a loading state.
(optional) What browsers are you seeing the problem on?
Microsoft Edge
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I think I can fix this


Akka Timed Out Error

When I create messages using JMS/SQS
I get following error
akka.actor.ActorSystemImpl - Request timeout encountered for request [POST /queue/queue1 Strict(129 bytes)]
15:58:52.689 [elasticmq-akka.actor.default-dispatcher-20] INFO  akka.actor.DeadLetterActorRef - Message [org.elasticmq.MessageData] from Actor[akka://elasticmq/user/$a/$c#1667985933] to Actor[akka://elasticmq/deadLetters] was not delivered. [9] dead letters encountered. If this is not an expected behavior, then [Actor[akka://elasticmq/deadLetters]] may have terminated unexpectedly, This logging can be turned off or adjusted with configuration settings 'akka.log-dead-letters' and 'akka.log-dead-letters-during-shutdown'.

After creating a queue with:
aws sqs --region eu-central-1 --endpoint-url http://localhost:9324 create-queue --queue-name lol

And testing message receival with WaitTimeSeconds from range (0..21):
for i in {0..21};
    do
    echo $i;
    aws sqs --region eu-central-1 --endpoint-url http://localhost:9324 receive-message --queue-url http://localhost:9324/queue/lol --wait-time-seconds $i;
    done

and:
for i in {0..21};
    do
    echo $i;
    curl -v --data-binary "Action=ReceiveMessage&WaitTimeSeconds=$i" http://localhost:9324/queue/lol
    done

I've never got an akka timeout in response. It seems that it's working correctly.
Please mind that, despite you can set akka.http.server.request-timeout in application.conf the max. value for WaitTimeSeconds cannot be higher than 20s - this value is validated here.

You won't be able to reproduce it unless you would slow down your system (but it's hard to reproduce). In my case, I had many containers open and probably low memory.
Default elasticmq akka.http.server.request-timeout is 21s. Many clients have set WaitTimeSeconds=20. This means elasticmq has 1 second to send a response. If system resources are very low, it's possible to exceed 21s timeout.
Anyway increasing akka.http.server.request-timeout helped in my case.

@michalbogacz thanks for the clarification. For future readers, remember you're still able to modify akka.http.server.request-timeout directly in application.conf configuration file.


Not returning body after 2.4.0

Hi, did anyone had any problems not returning body inside requisition and only returning the status after the realease of 2.4.0 yesterday?
Node v16.18.1
azure-functions-core-tools: "^4.0.4895"

I couldn't reproduce this issue locally but had it happening on one of my Function apps hosted in Azure. I've removed the  applicationinsights package from my application and deployed it to Azure. Working again now.
Seems to be some kind of issue happening with this package used with Azure functions. I was on v2.3.5, not even v2.4.0 and it was happening.

Likely the same issue as #1063


Simplify on-boarding process for new users

Meta issue to track the simplification of the on-boarding process.

Create release page https://github.com/mimblewimble/site/issues/103. With possibly Keybase signed release.
Create a page similar to https://bitcoin.org/en/choose-your-wallet. Listing third party wallet/implementations (must be open source) with a disclaimer.
Avoid going on GitHub as much as possible.

Note that 1. and 2. can be combined.

Add a get started page similar to https://z.cash/get-started/

It looks like someone is working this and it looks great (https://grin.mw/get-started).  However, wanted to caution that it might be unwise to have a prominent link on the website go to an un-moderated wiki page (Buy Grin --> https://github.com/mimblewimble/docs/wiki/Community-projects#exchanges).  I would have created a pull request with a suggested solutions... but I can't think of a better solution right now because the way it was done makes sense... it just exposes the site to abuse.  When people land on a wiki they should know right away they are headed to a wiki.

I'm considering this fixed for now, with further improvements welcomed in new PRs and issues. Closing


Support credential-less fileshare datastore

Description
Supporting credential-less fileshare datastore creation/updation, other datastores already support it. The REST interface for fileshare datastore supports it as well, modifying the schema to support from sdk side too.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-ml


Add Kotlin syntax support

Please add Kotlin syntax support. I can't find Kotlin syntax support in VS Code.
My VS Code about info :
Version: 1.61.0 (user setup)
Commit: ee8c7def80afc00dd6e593ef12f37756d8f504ea
Date: 2021-10-07T18:13:09.652Z
Electron: 13.5.1
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19043

Did you try using the Extensions view to search for extensions related to Kotlin?


Added in some background information for zero shot text classification + helpful links

This might sound confusing but zero-shot-classification is the NLP task (there's no zero-shot-text-classification) sorry for the confusion, I told about this to the team (I came across this multiple times)
would be nice if you could make your changes to zero-shot-classification page instead. @ckgresla

Not at all! thank you for clarifying, I saw the separate entry for zero shot image classification and thought that the team wanted to verbosely distinguish between the two -- updated the doc!

thank you for all of the great comments and recommendations! I ran through both files again and tried to incorporate the feedback to make the information better, hopefully this rendition is a better fit.

1st public PR ever, so my apologies for any inconveniences

I see most of my comments are now addressed, there's only one left (simplifying the description) and then we can merge!
Also, can you resolve conflicts? I wrote a guide here if you don't know how to do it, please let me know if you have any problems or questions 🙂 https://huggingface2.notion.site/Contribution-Guide-19411c29298644df8e9656af45a7686d

Thank you for getting the note and the nice guide on merging! I tried to simply the description again -- hopefully this is a nice way of phrasing things!

I went through each of the comments and tried to make the appropriate changes -- thank you for your patience with this PR!

@ckgresla what is your HF username? will give credits!


A EQL filter for result

Sometimes the results are too big to understand by scrolling thru its pprint in the result pane.
Would be great to have a EQL filter (like datomic pull) to narrow down nested maps structures.

Maybe we can use something like this https://github.com/juxt/pull ?

Implemented here e0dae7cc1f6335a568919c53c419c517038caef7 and documented here https://flow-storm.github.io/flow-storm-debugger/user_guide.html#_eql_pprint_visualizer


change function

@anseki Is there a function that changes an element's div?

Hi @nSanhuezaKrino, thank you for the comment.
What is "element's div"?
Do you mean that you want to use an element other than <div> element?
You can specify any element that has a bounding-box.
https://anseki.github.io/plain-draggable/#constructor

for example if I want to change an from one   to another

for example if I want to change an input from one div to another div

@anseki for example if I want to change an input from one div1 to another div2

Hi @nSanhuezaKrino , i will prefer  you control that by yourself, after you clickd the input ,at the same time you move this input into document.body , then you can drag . and after you decide put it into the target div2, after you grip, at the same time you insert the input to target div2.

@anseki I understand, thank you

@kaysaith1900, thank you for your help.
@nSanhuezaKrino, So, was the issue solved?

No reply came, then this abandoned issue is closed.


How can I quantize my trained model into GGUF format?

I trained a model using the script from Kohya, using the Dreambooth method. How can I convert it to Q8?

The readme in the tools folder should have the step by step on how to convert it. Feel free to reopen this issue if you run into any problems.

thanks，it's work


Infinite loop when positioning inline image (was: massive CPU load and error with "Java heap space")

Release 2.0.0 eats excessive CPU for longer time and ends up with:
Java heap space
Use --trace to show backtrace
This occurs using jRuby <IP_ADDRESS> as well as using ruby 3.1.2.
command is
asciidoctor-pdf -o movex-cdc.pdf movex-cdc.adoc
The used adoc file can be found here to reproduce the problem:
https://gitlab.com/otto-group-solution-provider/movex-cdc/-/blob/master/doc/movex-cdc.adoc
With asciidoctor-pdf 1.6.2 it works without problems.

The heap space error itself is a red herring. The real problem is that it is getting stuck doing an operation. If I cut the document down to this line, I can reproduce it:
image:er_model.svg[format=svg,opts=inline]

The hang may have something to do with the logic for resizing an inline image. But that also points us to the actual problem in the document. This should be a block image, not an inline image. If you turn it into an block image, you will be able to convert the document using Asciidoctor PDF 2.
image::er_model.svg[format=svg,opts=inline]

See https://docs.asciidoctor.org/asciidoc/latest/macros/images/#block-image-macro
I'll have a closer look as to why it is getting stuck when it's an inline image.

I figured it out. The text arranger was not able to fit a full height inline image on a new page, and thus would keep trying on the next page after that. I have added a protection against this in general, then tweaked the computation for the positioning of a full height image so that it does fit on the next page.

@mojavelinux, Thank you very much for your quick response and your useful hints

You're welcome! You can now access this fix in the 2.0.1 release, though I still think you'd be better off using a block image in this case.


✍ Ruby: Implementing custom formatters

As reported in https://github.com/appsignal/appsignal-ruby/issues/321, we need more thorough documentation for implementing custom formatters.

See also changes made in https://github.com/appsignal/appsignal-ruby/pull/397
about how to set this up


CMD should be masked array not clipped and pushed

I don't know what this means anymore.

Since masked arrays are found to negatively impact performance, I'm closing this issue.


Restart Camera After ShutDown

I'm using the scanner for bulk barcode scanning, I used the StartScanning and StopScanning methods IZXingScanner interface to define how I want the scanner to work, i.e It wont leave the camera view screen after scanning one barcode.
The camera shuts down when its scans a barcode, which makes sence, I wrote a method that opens the camera, sets caemra previewDisplay, camera previewCallback, CameraDisplayOrienrtation and StartPreview so i can continue to scan until I press the "done" or back button.
Please could you add a new method "StartCamera" in the  ZXingSurfaceView class, I can submit my code to you, you review it and see if you can merge it in.
Regards

I have the same situation where I need to validate the barcode that was read before scanning stops.  Would you be willing to share your modifications?

The next release will have continuous scanning (the code is already in master if you want to try it)

Thanks for the tip.  I tried it out and it worked.
From: Jonathan Dick<EMAIL_ADDRESS>Sent: Wednesday, July 15, 2015 2:22 PM
To: Redth/ZXing.Net.Mobile
Cc: Bill MacAdam
Subject: Re: [ZXing.Net.Mobile] Restart Camera After ShutDown (#139)
The next release will have continuous scanning (the code is already in master if you want to try it)
—
Reply to this email directly or view it on GitHubhttps://github.com/Redth/ZXing.Net.Mobile/issues/139#issuecomment-121719842.


Create workspace based app insights

Leverages log analytics create step, which is required to migrate App Insights to the Workspace Based. Here is some documentation on that. This was already being done in Functions, so the step already existed.
It also uses the connection string property rather than the instrumentational key since that is the recommended path going forward.
Another reference: https://learn.microsoft.com/en-us/azure/azure-monitor/app/create-workspace-resource
How to test:

Right-click "App Service" node and select "Create Web App..."
Fill out the prompts
Check to see if a log analytics workspace is created and linked to app insights
Check in the portal for the app insight component and make sure it's workspace-based (it'll tell you that you need to migrate it)
Try it with advanced too

I'm running with advanced because there are no resources currently available to test on basic.
It seems like I'm never prompted about creating a log analytics workspace, but the output logs indicate it's being defaulted to an existing resource.  Is this intended behavior?

(I'm running with advanced because there are no resources currently available to test on basic.)

What's the error? If you increase the sku, that will usually resolve the issue.

It seems like I'm never prompted about creating a log analytics workspace, but the output logs indicate it's being defaulted to an existing resource. Is this intended behavior?

Yeah, there's nothing to really pick and as far as I know, no benefit to really having a bunch of workspaces. I'll confirm with the App Service team, but that is the intended behavior.  You must have had one created with a function app at some point.

What's the error?  If you increase the sku, that will usually resolve the issue.

Are you using the free tier?

It reproduces on all the tiers offered via quick picks


Incorrect quotes reflection api flags set on Java symbols

Compiler version
3.3.3, 3.4.0, c8c3bde0270db83c2950c79cbe68267af7cfbc69 (main)
Minimized code

Main.scala:
@main def main() =
  Repro[JavaClass]

Macro.scala
import scala.quoted.*

object Repro {
  inline def apply[A]: Unit = ${ applyImpl[A] }

  def applyImpl[A](using Type[A], Quotes): Expr[Unit] = {
    import quotes.*, quotes.reflect.*
    println(TypeRepr.of[A].show + " " +  TypeRepr.of[A].typeSymbol.primaryConstructor.flags.show)
    '{ () }
  }
}

JavaClass.java:
public class JavaClass {}

Output
JavaClass Flags.JavaDefined | Flags.Local | Flags.Method | Flags.Private | Flags.PrivateLocal

Expectation
The Flags.Private and Flags.PrivateLocal should not be a part of the flagset of a public constructor method
Related: https://github.com/scalalandio/chimney/issues/484#issuecomment-2027060505

The issue lies in our java outline parser, which always produces a dummy constructor:
package <empty> {
  object JavaClass() {}
  class JavaClass private[this](x$1: scala.Unit) extends Object {
    def <init>() = _root_.scala.Predef.???
  }
}

The  method is the actual constructor found by the parser (so for this case it's public, if we were to add a private java constructor it would become private etc.), but there is no way to locate it with the quotes.reflect methods.
In scala 2 no such system was used, so after parsing we would get:
package example {
  object JavaClass extends  {
    def <init>()
  };
  class JavaClass extends _root_.java.lang.Object {
    def <init>()
  }
}

and primaryConstructor would return the correct method.
Another issue here is the fact that if we split compilations into 2 runs, I believe we read the java class file instead of using our source outline parser, which results in obtaining the correct primaryConsructor

There was some issue with Scala 2 primaryConstructor and Java as well - https://github.com/scala/scala/blob/d76229ef6f6b2e79422d5990ba2deb4aed340277/src/reflect/scala/reflect/api/Symbols.scala#L960 - so I am not surprised.
Ideally, this behavior would be fixed, but if it cannot, I'd hope that the proper workaround could be documented, ideally in APIs documentation with some code examples. For Scala 2/Scala 3 cross-compilable macros I'd like to be able to align the behavior or at least be able to document the inconsistencies and their sources.
E.g. if it was possible to:

verify that it's a Java class (Flags.JavaDefined?)
always get a list of all constructors (sym.declarations.filter(_.isConstructor)?)
sort them by position and assume that the first one is primary (kinda sketchy but at least deterministic - Ordering.Option(Ordering.fromLessThan[Position] { (a, b) =>      a.startLine < b.startLine || (a.startLine == b.startLine && a.startColumn < b.startColumn)    }))

it would be... something. For classes with a single, default constructors (like most beans) it should be basically deterministic behavior, for classes with multiple-constructors... well, we there could be some best effort.


Added retries for httpd.conf

Added 3 retries for httpd.conf, since otherwise it gives an intermittent error of the browser not rendering the text

Not to fan the fire or anything, I agree with Jeff that this might be an issue and we should probably fix. Does it happen in Jwala ? Like when you click httpd.conf manually it does not show up sometimes ?


Move sampling to query

from the Matcher.__init__ method. Doing it during indexing may led to invalid pairing

max_size param was removed from init in #53


Log file Rotation - How to set the Total size of files in the target directory, in bytes, upon which the oldest file will be deleted.

Hello
This lib is really great one to use.
Here is one of the thing -where i am looking some help to implement the Total size of files in the target directory, in bytes, upon which the oldest file will be deleted.
I am looking always 10files with 2MB size in maximum, then the oldest will delete .

I have my call-back with a static counter.


docs: add JobSet API reference

Fix issue: https://github.com/kubernetes-sigs/jobset/issues/566
How to generate documents:
root@VM-0-3-ubuntu:/home/ubuntu# cd jobset
root@VM-0-3-ubuntu:/home/ubuntu/jobset# export PATH=$PATH:/usr/local/bin/go/bin
root@VM-0-3-ubuntu:/home/ubuntu/jobset# make generate-apiref
make: go: Permission denied
cd /home/ubuntu/jobset/hack/genref/ && /home/ubuntu/jobset/bin/genref -o /home/ubuntu/jobset/site/content/en/docs/reference
I0701 10:24:47.111766 1227157 main.go:142] Parsing go packages in sigs.k8s.io/jobset/api/jobset/v1alpha2
I0701 10:24:50.923557 1227157 main.go:325] Output written to /home/ubuntu/jobset/site/content/en/docs/reference/jobset.v1alpha2.md
root@VM-0-3-ubuntu:/home/ubuntu/jobset# Connection to <IP_ADDRESS> closed by remote host.

API reference autogenerated  based on this project: https://github.com/kubernetes-sigs/reference-docs/tree/master/genref

@googs1025  I checked out this PR and previewed the output docs locally, is the {#jobset-x-k8s-io-v1alpha2-JobSet} next to the JobSet expected?

I checked the API Reference of kueue and got the same result. This is the result of the automatic document generation.
here

/lgtm
/approve
Thanks!


New API proposal-Shutdown Service Status

This API allows users to obtain the current shutdown status of a certain phone number, including Voice/SMS in shutdown, Voice/SMS out shutdown, and data service shutdown.
Anti fraud: When receiving SMS or phone calls from a suspended number, it may be a disguised fraudulent number, which can provide security risk alerts.
Customer service support: If a user reports that they are unable to receive calls or text messages, customer service personnel can determine whether the problem is caused by the phone voice function being shut down by checking the phone's status.
This is from GSMA drop4 priority list
PR is here: 32

Based on the results of the discussion about the overlap between ShutdownStatus and DeviceStatus in Device Status .I believe they are two different APIs.
Additionally, based on the suggestions discussed, I think changing the name to SubscriptionStatus would be more accurate.

As agreed in the group discussion (please correct if mistaking):

Shutdown Service pending to create repo. Backlog admins will get in touch with API supporters to find CodeOwner/Maintainers, and will follow similar strategy as previous APIs. Name change to be confirmed (Subscription Status).

As agreed in the conversation, KYC family meetings can be used as follow-up tracker for the hole set of APIs (KYC-Match, KYC-Fill-In, Age Verification, Tenure, Number Recycling, Shutdown Service, final decision on meetings will take place after some running time.
@camaraproject/api-backlog_maintainers

@TEF-RicardoSerr
This issue can be closed

Please @chinaunicomyangfan collect the codeowners and maintainers from your side and from Huawei and China Telecom

@TEF-RicardoSerr  I have already sent the codeowner and maintainer list of Subscription Status via email. I am not sure if you have received the email, so I just sent it again.My colleague ShuoRan will serve as the owner to undertake the related work of this API.
Here is the list of personnel:
@ShuoRan ChinaUnicom is the code owner from ChinaUnicom
@philipxujin  is the maintainer from Huawei.
@DanXu-ChinaTelecom is the maintainer from China Telecom.


Add support for Python 3.13

The Python 3.13 release candidate is out now! :rocket:
The Release Manager has issued a call to action:

We strongly encourage maintainers of third-party Python projects to prepare their projects for 3.13 compatibilities during this phase, and where necessary publish Python 3.13 wheels on PyPI to be ready for the final release of 3.13.0. Any binary wheels built against Python 3.13.0rc1 will work with future versions of Python 3.13. As always, report any issues to the Python bug tracker.

LGTM!


Add resource attributes

Adding hypertrace resource attributes, and updating telemetry attributes

I this something hypertrace expects? If not, I would not add these kind of
attributes and instead pass attributes in the resource section on the agent
config.
On Tue, 6 Apr 2021, 22:16 David Roth, @.***> wrote:

@.**** approved this pull request.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/hypertrace/goagent/pull/113#pullrequestreview-629372158,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYAVYG6HAQYKW2VN5M73THNT25ANCNFSM42OYQSAQ
.

I this something hypertrace expects? If not, I would not add these kind of attributes and instead pass attributes in the resource section on the agent config.
…
On Tue, 6 Apr 2021, 22:16 David Roth, @.> wrote: @.* approved this pull request. — You are receiving this because your review was requested. Reply to this email directly, view it on GitHub <#113 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAXOYAVYG6HAQYKW2VN5M73THNT25ANCNFSM42OYQSAQ .

We need to set TelemetrySDKLanguageGo, but the last two we can remove.