text input doesn't work

i actually fork imguiCS but when i rewrote everything under imgui.net it didn't help so how do i get the text input to work?

@makarhuyaker check the example projects provided in this repository. Input text works there.

@makarhuyaker check the example projects provided in this repository. Input text works there.

uh ...... so this is the problem I have an application in which I embed imgui and it looks like it intercepts keystrokes

sounds like an issue in your application and how it intercepts keystrokes rather than ImGui/ImGui.NET issue.

I already understood

I found out that this is most likely not a problem of my application, I am most likely not using input correctly, can you give a working code of the inputtext? i use private byte [] buff = new byte [10]

@zaafar

create a class level string as show below.
private string displayName = string.Empty;
then use it in the InputText as shown below.
ImGui.InputText("Display Name", ref this.displayName, 50);

what about the textinputbuffer class? i tried to use it like in sdl2 example but got AccessViolationException

ref this.displayName

argument 2 should not be passed with the ref keyword

forgot to say i'm using a fork of imgui.net imguics but wrote it here because the repository is zipped but when i tried to use imgui.net the text input also failed

forgot to say i'm using a fork of imgui.net imguics but wrote it here because the repository is zipped but when i tried to use imgui.net the text input also failed

https://github.com/0x0ade/ImGuiCS/blob/master/ImGuiSDL2CS-Example/src/YourGameWindow.cs#L67
That library was last updated in 2018. I would suggest you to use this lib rather than that one.

I already said that I tried to use the textinputbuffer class but got an AccessViolationException and about the second one, the fact is that I already wrote quite a lot of code and I just don't want to port it to imgui.net

I already said that I tried to use the textinputbuffer class but got an AccessViolationException and about the second one, the fact is that I already wrote quite a lot of code and I just don't want to port it to imgui.net

hmm, no idea in that case. What I would suggest is to clone that repository and see if examples are working over there. If yes, try to figure out what that repo example is doing that you aren't.

just now i created an empty xna project and checked the text input there and it doesn't work either all the same it's a problem imgui

now I will try with imgui.net

doesn't work either

yeah that repository/library is very old so make sense.
However, this repository/library has no connection with that repository/library.

yes I know but let's say I redid the code for imgui.net but my text input still won't work

maybe the problem is in the cimgui which I am using tomorrow I will try a new version

Without any code, this isn't an actionable issue. As stated above, the example project in this repository details exactly how to get input working. If you're using a different window/input system (or an entirely different set of bindings to dear imgui...), then I suggest you open an issue against the other library that you're using.

Without any code, this isn't an actionable issue. As stated above, the example project in this repository details exactly how to get input working. If you're using a different window/input system (or an entirely different set of bindings to dear imgui...), then I suggest you open an issue against the other library that you're using.

it really doesn't work with xna

I will upload my project to github


Allow an authenticated user to reset his password.

Hello,
When you're using the password reset link received by Mail or SMS when you're already connected, you'll have a 500 error and it's impossible to reset the password.
Access logs:
<IP_ADDRESS> - - [01/Nov/2020:12:22:09 +0000] "GET /login?pswdrst=TST-1-******************************&service=https%3A%2F%2Fcasserver%3A8443%2Foauth2.0%2FcallbackAuthorize%3Fclient_id%3Dclient%26redirect_uri%3Dhttps%253A%252F%252Fwww.client.com%26response_type%3Dcode%26client_name%3DCasOAuthClient HTTP/1.1" 302 -

<IP_ADDRESS> - - [01/Nov/2020:12:22:09 +0000] "GET /oauth2.0/callbackAuthorize?client_id=client&redirect_uri=https%3A%2F%2Fwww.client.com&response_type=code&client_name=CasOAuthClient&ticket=ST-2-**************** HTTP/1.1" 302 -

<IP_ADDRESS> - - [01/Nov/2020:12:22:09 +0000] "GET /oauth2.0/callbackAuthorize?client_id=client&redirect_uri=https%3A%2F%2Fwww.client.com&response_type=code&client_name=CasOAuthClient&ticket=ST-2-X************** HTTP/1.1" 500 14379

Cas logs:
2020-11-01 12:22:09,691 DEBUG [org.apereo.cas.web.FlowExecutionExceptionResolver] - <Ignoring the received exception [org.apereo.cas.ticket.InvalidTicketException] due to a type mismatch with handler [org.apereo.cas.support.oauth.web.endpoints.OAuth20CallbackAuthorizeEndpointController#handleRequest(HttpServletRequest, HttpServletResponse)]>
2020-11-01 12:22:09,691 DEBUG [org.apereo.cas.web.FlowExecutionExceptionResolver] - <Ignoring the received exception [org.apereo.cas.ticket.InvalidTicketException] due to a type mismatch with handler [org.apereo.cas.support.oauth.web.endpoints.OAuth20CallbackAuthorizeEndpointController#handleRequest(HttpServletRequest, HttpServletResponse)]>
2020-11-01 12:22:09,699 ERROR [org.springframework.boot.web.servlet.support.ErrorPageFilter] - <Forwarding to error page from request [/oauth2.0/callbackAuthorize] due to exception [null]>

When you're using the CAS Protocol and not oauth or oidc, the link is redirecting you to the service without error.
Fixed by adding the renew parameter to the link.
Regards,
Julien

you'll have a 500 error and it's impossible to reset the password.

Please update the description to indicate the error received. Thank you!

This also does not seem the right solution because renew is very CAS-protocol specific. There should be something in the flow that recognized the attempt as a reset attempt and ignored SSO, presenting a link, etc regardless of whether it's a CAS app or something else.

The link itself is very cas specific:
http://op-dev.ina.fr:8080/login?pswdrst=TST-*-********************-******************&service=the_service
The solution works even with the oauth or oidc protocol.

Sure, but it's not about "what works" now. It's about the appropriate usage of a protocol-specific flag to accommodate something irrelevant. There is hooks that allow control over the SSO session, regardless of what exists in the URL. That's what should be used here.
Also, this will not work if renew functionality is turned off.

So I'll not tell about the service parameter in the link. ^^
Ok i'll find a better solution. :)

If I could assist, (reviewed internals a bit), I think you should find a solution that relies on "SingleSignOnParticipationStrategy" components. This is what I meant by "hooks". These components determine whether existing SSO should be honored/used. You should in theory be able to create one that can access or reject SSO for password management, and it should recognize that SSO should be disabled if the incoming request is a password-reset request. HTH.

Thanks for the tip.
I have to tell that I'm not a Java developer, I'm really trying my best to do a correct job in all the PR that I've done here but sometime it's a bit complex. Do not hesitate to tell me what should be improved in it.

I have to tell that I'm not a Java developer, I'm really trying my best to do a correct job in all the PR I do here

No worries. You're doing a superb job, and this can be a complicated system. If there is something I can help with, I'll be sure to chime in. I can tell you that if you are deploying CAS, unless you have hired help, you are for all practical purposes a java developer :)

Hi,
I've just implemented the disabling of the sso for the flow with the SingleSignOnParticipationStrategy. if that's ok for you i'll add tests.
Regards,
Julien

if that's ok for you i'll add tests.

The structure looks good, yes. You will need to complete the body of the strategy to make sure SSO is disabled correctly (and not all the time) and then add tests.

I'm really sorry Misagh but I don't understand what you want. You want to disable the sso only in certain cases ?

You want to disable the sso only in certain cases ?

Yes. You need to disable SSO in scenarios that involve password management, password resets, etc. What you have now disables SSO for everyone all the time. That will not work. Think about the scenario you have: you have password reset link, and you launch into the password reset flow. That's when you need to disable SSO. Returning false all the time is going to disable SSO for every request all the time for everyone.

To assist you better, let's hold off on this one for now, and I'll take a look in the next few days to come up with better guidelines.

Got it. Related to supports.
Don't want to waste your time sorry!

Hi,
I added the disabling of the SSO for the reset password flow only, the tests and a PR for the 6.2 branch.
Take your time to review and thank you for having helped me with this, it's a little fix but it had been more complex to develop for me than all my work on the OIDC & OAuth 2.0 protocol here!
Regards,
Julien

No problem.  Let me know if I can help.
-- Misagh

Hi,
Again, thanks for your time, your help and your patience. Just want a little confirmation here.
I'm pretty sure the main reason of our problem is here:
            val createTgt = getTransitionableState(flow, CasWebflowConstants.STATE_ID_CREATE_TICKET_GRANTING_TICKET);
            createTgt.getEntryActionList().add(
                createEvaluateAction(String.join(DO_CHANGE_PASSWORD_PARAMETER, "flowScope.", " = requestParameters.", " != null")));

When you're not connected, an empty TGC is created and given to you when you're trying to access to the reset link page:
Set-Cookie
	TGC=; Max-Age=0; Expires=Thu, 01-Jan-1970 00:00:10 GMT; Path=/; HttpOnly

When you're already connected, the transitionableState STATE_ID_CREATE_TICKET_GRANTING_TICKET is never occurring and the STATE_ID_PASSWORD_RESET_SUBFLOW subflow is never accessed.
Is my understanding is correct?

Well, it's a bit difficult for to add this action. Would you mind to do it and I just push my new validation ?

I'll push my new code here. Would be easer for you to review and add the new action. :)

No problem. I probably won't get to this by end of next week. Stand by please. (Also, it might be best to close the PR for 6.2 atm until this is finalized; up to you)

No problem. :)

I fixed the tests.
Everything is working except, of course, when you're connected and the reset token is not valid. I'll wait for your fix on that point and will backport it in the PR for the 6.2 branch if you want.
Feel free to review and tell me if another improvements must be made.
Again, thanks for your help and your patience.

I've noticed you pushed the fix in the webflow today.
I've tested my last commit with your fix and everything is now working well. :)
Will backport it in the PR for the 6.2 branch.

I did some minor clean up here to make sure the strategy impl is consistent with others, and have also added UI/puppeteer tests to verify functionality. This is now merged. Thanks very much!

I've noticed, it's far better that way. Thanks!
Will take a look a the puppeteer tests, could be useful for my work on oidc.


onClick on PaymentRequestButtonElement does not appear to work.

onClick prop on PaymentRequestButtonElement does not appear to work.
E.g: no console.log fires when clicking the PaymentRequest (purple) button
<PaymentRequestButtonElement
    paymentRequest={paymentRequest}
    onClick={() => {
      console.log("clicked apple pay button")
    }}
/>

Hmm… just tried to replicate this on the latest version of this library and the onClick works as expected:
https://codesandbox.io/s/ecstatic-morning-vkvnz?fontsize=14&hidenavigation=1&theme=dark
Do you have any more details I can use to track down the issue?


Lightly modify new page style.

Responsive design
Darkmode support

You can see my style in my site: https://link.2to.fun/

Sorry, I pull this request by mistake


Add initial support for MDMA

Only memory-to-memory transfers
Simple example

This isn't yet part of #153, since it requires the nightly PAC (and the CI fails because of the nightly PAC combined with our rtc feature).

This isn't yet part of #153, since it requires the nightly PAC (and the CI fails because of the nightly PAC combined with our rtc feature).


Add individual LTVs and LTIs as output

In the process and due to having to arrange a "preClearingResetCounters()" method, replace duality of counterVariable-actualVariable pairs by only actual variables.

Finally, decided against individually outputting LTV, LTI information, opting instead for relying on individual transaction data, whose storage will be made more efficient by splitting rental and sales data  into separate files.


⚠ SceneTime has degraded performance

In 687c4b6, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 14675 ms

Resolved: SceneTime performance has improved in bced61c after 6 minutes.


Support AttentiveFusion and V2X-ViT model

Hi team,
Thanks for your great work! Do you have plan to support AttentiveFusion model in OPV2V (ICRA2022) and V2X-ViT (ECCV2022) in your framework?
@RoboticsYimingLi @dekunma @simbaforrest

Haha Runsheng @DerrickXuNu , yes of course. In fact, we hope you and others could join this open-source effort! Do you want to be added to the core management team?
@dekunma is currently working on improving the readme about how everyone could join this effort freely via Git and GitHub. Basically you can add V2X-ViT models (and your dataset's dataloaders!) to this repo by filling a pull request.

Yeah, it would be great if I can be added to the core team!
Adding dataloader for opv2v/v2xset may take some time, but we could start from adding methods to this repo.

Thanks for adding me to this.
Yeah, I will invite my teammates as well! I agree that having a unified codebase would be great for the whole community. Letting the codebase support both datasets may take some time, but we can sync up the methods first to have the momentum. I will also adopt DiscoNet in OpenCOOD in the next few days, so people who want to try OPV2V in the near future can also compare it with DiscoNet. I will let Yiming be the reviewer for that.


Format of "road_event_feed_info" "version" property, possible enumerated type

The version property on the road_event_feed_info entity is described as

The specification version used to create the dataset

This is a text field with no further indication of how the version should be represented, that is, v2.0, V2, 2, 2.0, etc.
This should be standardized, either with clarifying text + a regex in the JSON schema or by creating an enumeration for all the possible feed versions.
My choice for a format for this field would be 2 if we are going to continue release only major versions, and 2.0 if we want to consider minor releases.

Comments received during the V3 Specification Update Subgroup meeting on 4/29/2020:

Lynne Randolph: I'm good with major.minor
Neil Boudreau, MassDOT: I think that makes the most sense
Derald W Dudley: +1 for major.Minor
Marysa Myers: Agreed, major.minor would be easiest and could avoid any inconsistencies using V, v, v., etc
Jim Williams (INRIX): Also agree with major.minor.
Ariel and David Craig to review the definition of major/minor versions of the specification

Resolved in #89


fix bug "expected 3 arguments"

Fix bug in method "add_auth".
Error: expected 3 arguments

fixes #194

@DavidVujic please merge & release asap :)

Great work @wareczek!
Can you also add a unit test, or example code that will show the effects of the currently missing parameter?


[2849] Don't save empty strings as course subjects

Context
https://trello.com/c/oFFhj2gs/2849-funding-is-cleared-when-only-course-dates-are-changed
Changes proposed in this pull request
This fixes a bug whereby if you re-entered a complete course section and changed the course_start_date, it would clear the funding section because the form changed course_subject_two from nil to empty string.
Guidance to review

Complete a trainee with a scholarship
Go back into course details and change the start/end date
Save
See that the scholarship information is still present and the funding section is still complete

 Funding is cleared when only course dates are changed


Hide DS4 Controller stuck enabled

I'm using the latest version (09.06.2020). I had issues in Fifa 20 and i saw an advice on reddit which was basically saying to enable "Hide DS4 Controller" option. When i tried so, DS4 Windows stopped detecting the device. The problem is, now windows is not detecting it too even when the DS4 driver is not installed.
The only thing that happens when i plug my ds4 is this weird thing that you can see at the bottom right of this video:
https://youtu.be/Ex9JtSKSJHk
I just want to know how can i get ds4 windows and my OS to recognize it again.

Issue solved.
Solution: Uninstall the driver, Open Devices and Printers, right click on Wirelles Controller, Change Settings, Enable Device.
After that you can open DS4 Windows again and install the driver. It should detect your DS4 from now.


"Boardwalk has stopped"

Version of Boardwalk:1.1
Version of Android:4.2.2
Device model:SM-T110
Description of the problem:When its done installing it goes to the menu and crashes leaving the message "Boardwalk has stopped"

When its done installing it goes to the menu and crashes leaving the message "boardwalk has stopped"

I am using a philips s388
Version of Android:4.4.2

Version of Boardwalk: 1.1

eu uso boardwalk 1.1 e meu android e 4.4.2 ele para

eu uso boardwalk 1.1 e meu android e 4.4.2 ele para


Potential Fix For Crashes Caused By Non-UTF-8 IRC Input

Hello,
I was having a lot of problems with crashes caused by non-UTF-8 IRC input. This won't hit most users because I'm using it in, shall we say, a completely insane channel, but perhaps this fix: 1) works 2) is desirable
I am using the next branch here.
Error examples, both triggered by the same IRC input at the same time:
Instance 1 running on a bare metal system (I call it "BM"):

Traceback (most recent call last):
  File "/home/shithead-x/virtualenvs/shithead-X-next/./gpt2_bot.py", line 6, in <module>
    bot.start()
  File "/home/shithead-x/virtualenvs/shithead-X-next/gpt2_bot/irc.py", line 65, in start
    lines = self.socket.recv(524288).decode("UTF-8")
UnicodeDecodeError: 'utf-8' codec can't decode bytes in position 508-509: invalid continuation byte

Instance 2 running in a virtual machine, which I call "VM":

Traceback (most recent call last):
  File "./gpt2_bot.py", line 6, in <module>
    bot.start()
  File "/home/shithead-x/virtualenvs/shithead-X-next/gpt2_bot/irc.py", line 65, in start
    lines = self.socket.recv(524288).decode("UTF-8")
UnicodeDecodeError: 'utf-8' codec can't decode bytes in position 508-509: invalid continuation byte

Potential fix: Change line 65 in irc.py to:
lines = self.socket.recv(524288).decode("UTF-8", errors='ignore')

Another option for errors= is 'replace' however I have not experimented with that. Previously under the conditions each of the bots would only last a few minutes before encountering the issue and crashing. I let both of them run overnight in those conditions without a single crash.
Credit should go to irc.libera.chat #python for helping me.
What do you think?

yep, nice, good catch.
do you want to submit a pull request, or are you happy for me to make the fix?

Good to hear! I don't really know what a pull request is, could you make the fix? Whichever is less work for you.
I am still having other, far less frequent crashes not related to this, is there somewhere we can talk (such as irc)?


Adjust threshold for splice variants.

Is your feature request related to a problem? Please describe.
The splice region is based on the sequence ontology setting. However, ACMG 2021 recommends different settings.
Describe the solution you'd like

Targeted regions should minimally include coding exons with sufficient intronic coverage to allow analysis of positions −1_−16 and +1_+5 as well as other regions with reported pathogenic variants (e.g., splice sites of noncoding exons, deep intronic variants).
-- ACMG 2021

Describe alternatives you've considered
N/A
Additional context

From internal helpdesk.
Next-generation sequencing for constitutional variants in the clinical laboratory, 2021 revision: a technical standard of the American College of Medical Genetics and Genomics (ACMG)

We now have "distance to exons". However, should be changed in bihealth/mehari.


fix: compatibility changes for v12

PS: I don't know if needs more changes i will check but for now this fix make the module work

Would love to see this get merged. Driving me crazy not having autocomplete since updating to v12.

The module work for me; @ghost-fvtt  can you just merge ? Thanks

@Biimok is there a way to use your module? The main branch seems dead.

You can clone my forked repo and link the built project to Foundry. The guide to do can be found at project README  @CaosFR


Question

Hi, thanks for the lib.
Is there a reason that there are two useEffect hooks used and not combined in one?  https://github.com/gorhom/react-native-portal/blob/master/src/components/portal/Portal.tsx#L31

Hey @telmen 👋
I asked the same question when I first saw it but it seems he prefered to handle didMount and didUnmount cycles seperately. Since this is an open source project, it might be more understandable for developers who uses classes.
But I would suggest using effect once and just pass named functions. @gorhom
useEffect(() => {
  onMount();

  return onUnmount;
}, [])

Closing this.


integration with or-tools in c++ with bazel

Could you please add some description at https://github.com/google/or-tools/tree/stable/bazel#integration instead of TODO.

There is nothing specific to or-tools there.
Le sam. 9 janv. 2021 à 16:02, yurij<EMAIL_ADDRESS>a écrit :

Could you please add some description at
https://github.com/google/or-tools/tree/stable/bazel#integration instead
of TODO.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/2328, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3IJC5OMVM63RYL3YKDSZBVWZANCNFSM4V3UT7AQ
.

There is nothing specific to or-tools there.
Le sam. 9 janv. 2021 à 16:02, yurij<EMAIL_ADDRESS>a écrit :

Could you please add some description at
https://github.com/google/or-tools/tree/stable/bazel#integration instead
of TODO.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/2328, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3IJC5OMVM63RYL3YKDSZBVWZANCNFSM4V3UT7AQ
.

@lperron the goal is to provide a sample/tempate project like we did for CMake
https://github.com/google/or-tools/tree/stable/cmake#integration
The best is to have a repo like https://github.com/or-tools/cmake_or-tools but for bazel based build.

@lperron the goal is to provide a sample/tempate project like we did for CMake
https://github.com/google/or-tools/tree/stable/cmake#integration
The best is to have a repo like https://github.com/or-tools/cmake_or-tools but for bazel based build.

@safronau something like this: https://github.com/or-tools/bazel_or-tools

@safronau something like this: https://github.com/or-tools/bazel_or-tools


Multiple GPU on the same server

This handy tool is a great addition to dask-distributed especially for those who re using tensorflow distributed. It would much better if there are options to specify GPU resources started by each dask tasks as in such situations like multiple workers resides on a same machine. When a tensorflow server initializes it grab all the available GPU RAM of all GPU cards availabe on a machine by default (as mentioned here). I have managed to achieve this by just adding:
import os
def start_and_attach_server(spec, job_name=None, task_index=None, dask_worker=None):
     os.environ["CUDA_VISIBLE_DEVICES"]=str(task_index)
     server = tf.train.Server(spec, job_name=job_name, task_index=task_index)
     dask_worker.tensorflow_server = server
     dask_worker.tensorflow_queue = Queue()
     return 'OK'

but this rudimentary, it will not check for whether it is on the same node or not which is essential and there is no clean interface to specify that as well. Is it possible to add such functionality in a more better way, so many tensorflow tasks can share a same node.

Do these docs suffice?
http://distributed.readthedocs.io/en/latest/resources.html
On Fri, Jul 14, 2017 at 2:46 AM, surajkamal<EMAIL_ADDRESS>wrote:

This handy tool is a great addition to dask-distributed especially for
those who re using tensorflow distributed. It would much better if there
are options to specify GPU resources started by each dask tasks as in such
situations like multiple workers resides on a same machine. When a
tensorflow server initializes it grab all the available GPU RAM of all GPU
cards availabe on a machine by default (as mentioned here
https://stackoverflow.com/a/34776814). I have managed to achieve this
by just adding:
import os
def start_and_attach_server(spec, job_name=None, task_index=None, dask_worker=None):
os.environ["CUDA_VISIBLE_DEVICES"]=str(task_index)
server = tf.train.Server(spec, job_name=job_name, task_index=task_index)
dask_worker.tensorflow_server = server
dask_worker.tensorflow_queue = Queue()
return 'OK'
but this rudimentary, it will not check for whether it is on the same node
or not which is essential and there is no clean interface to specify that
as well. Is it possible to add such functionality in a more better way, so
many tensorflow tasks can share a same node.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-tensorflow/issues/2, or mute the thread
https://github.com/notifications/unsubscribe-auth/AASszPvOKO8OdIxp--L6eVryO4Yc7sCgks5sNxzNgaJpZM4OX74s
.

Excellent, yes this will do.
I was just looking to the dask-tensorflow or rather tensorflow code itself constrain resources inline. However, the scheduler level resource constraining is much more straight forward.
Nice work, thank you.


BUG: Services Page has hover Error

Describe the bug
In the services page, when I hover over the services section the section changing to black.
To Reproduce
Steps to reproduce the behavior:

Go to 'Login page and login'
Click on 'services'
Scroll down to 'a bit'
See error

Expected behavior
The hover should be pop up the section or has clear hover effect.
Screenshots
https://github.com/Priyaaa1/StartConnect-Hub/assets/130668745/17651cca-81fc-40b6-9dc7-6121543a67a2

@MuraliDharan7 it has been resolved now


"Execute" permissions in "bin" folder are not preserved while building a wheel with PDM

[x] I have searched the issue tracker and believe that this is not a duplicate.

Make sure you run commands with -v flag before pasting the output.
Steps to reproduce
I'm using pdm build to build a wheel (https://github.com/pglet/pglet-python). The package includes "bin" folder with a number of executables in it. Executables are downloaded during CI build and their execute permissions are set with chmod: https://github.com/pglet/pglet-python/blob/main/appveyor.yml#L27-L42
Upon testing Python package built with PDM it looks like PDM drops "execute" permissions on files in bin directory while making a package with pdm build. Here is the recent version built with PDM: https://test.pypi.org/project/pglet/0.1.280/
On the other hand, the version originally built with setup.py preserves "execute" permissions: https://test.pypi.org/project/pglet/0.1.251/
Actual behavior
"Execute" permissions on executables in bin folder are dropped while building a wheel with pdm build.
Expected behavior
"Execute" permissions are preserved.
Environment Information
# Paste the output of `pdm info && pdm info --env` below:
PDM version:        1.12.6
Python Interpreter: /home/appveyor/venv3.9.8/bin/python (3.9)
Project Root:       /home/appveyor/projects/pglet-python
Project Packages:   /home/appveyor/projects/pglet-python/__pypackages__/3.9
{
  "implementation_name": "cpython",
  "implementation_version": "3.9.8",
  "os_name": "posix",
  "platform_machine": "x86_64",
  "platform_release": "5.4.0-1063-azure",
  "platform_system": "Linux",
  "platform_version": "#66~18.04.1-Ubuntu SMP Thu Oct 21 09:59:28 UTC 2021",
  "python_full_version": "3.9.8",
  "platform_python_implementation": "CPython",
  "python_version": "3.9",
  "sys_platform": "linux"
}

Thank you for the quick fix! I can confirm that file permissions are preserved within archive now and timestamps are accurate.
Just a quick not for others reading this issue: until a new version of PDM with this fix is released pyproject.toml could be updated as to pull the latest right from the repo:
[build-system]
requires = ["git+https://github.com/pdm-project/pdm-pep517.git"]
build-backend = "pdm.pep517.api"


Continuously report temperature - No sensor reported

Using the code snippet for Continuously reporting temperature sensor, no ambient temperature sensor is connected. However reading a single temperature value works fine. Is this library registering the driver for the Bmx280 on the board, because the sample doesn't cover it.
Repro steps:

Copy code snippet into onCreate.
Run activity on a Pi 3 (with working other functions)

Expected:

Observe logging of temperature sensor

Actual:

Observed logging only includes: "I/SensorManager: DYNS Register dynamic sensor callback"

No events triggered into the SensorEventListener instance

On further inspection (attempting to use the getDefaultSensor() ) no sensors exist.

It seems the snippet you linked to is missing a call to registerTemperatureSensor(). For now, please refer to the Activity code in the Bmx280 driver sample, which demonstrates the same use case.

@plattysoft Not quite a duplicate (at least I don't think so), I think @paul-matthews issue is the code snippet is missing a call to registerTemperatureSensor(), so the sensor never actually becomes available.


Players unable to retrieve their grave on others claims

Hi there,
I'm running a Sevtech Ages server and was previously using GriefPrevention and upgraded to GriefDefender after learning about it.
I've run into an issue where players can no longer collect their grave (Tombstone-4.0.0-1.12.2) if they happen to die on another players claim.
GriefPrevention used to allow this to happen if the player just crouched over their grave but GriefDefender wont allow it at all.
Is there something you can add to the config to allow a "bypass" so that players can access their graves without an Admin having to give them temporary claim trust?
Thanks for your time.

There is no need for a config to bypass this as you have full control of your protection. Unlike GP, GD hooks into contexts completely. Use https://github.com/bloodmc/GriefDefender/wiki/Debugging and add the source/target as an override globally.  Read https://github.com/bloodmc/GriefDefender/wiki/Contexts if you are unsure how contexts work.


Numeric only cells drop leading 0

This is a huge problem with most CSV editing tools, even Microsoft Excel.  I often am dealing with large CSV files that have a number of entries consisting of numbers that must retain a leading 0 digit.  I understand why it does this from a programming point of view, but it's very frustrating.

Also I had this problem, and I did a little investigation, a solution was found that by modifying one line code in CSVReader.m
`// regex = [NSRegularExpression regularExpressionWithPattern:[NSString stringWithFormat:@"^\s*[+-]?(\d+\%@?\d*|\d*\%@?\d+)([eE][+-]?\d+)?\s*$",_config.decimalMark,_config.decimalMark] options:0 error:NULL];
regex = [NSRegularExpression regularExpressionWithPattern:[NSString stringWithFormat:@"^\s*[+-]?([1-9]+\%@?\d*|\\%@?\d+|[0]+\%@+\d)([eE][+-]?\d+)?\s*$",_config.decimalMark,_config.decimalMark,_config.decimalMark] options:0 error:NULL];`
Jacob, thanks a lot for this simple and efficient application, could you please update it in the new version.

@YWangScience Could you explain in a bit more detail what your proposed change does?

@jakob The logic is if a value has no decimalMark (not a float) and it begins with 0, your program considers it as a string rather than a number. For example, 090618 is a string, shall not be cut to 90618. I simply separated \d to [1-9] and 0, surely you can construct a better regex.

@YWangScience thanks for explaining. This makes a lot of sense! (Reading regex expressions is not so easy....)
Would you like to make a pull request with your changes? (If you do, it would also be a good idea to add test cases to make sure that the proposed changes work, and I don't accidentally break them in the future).

@YWangScience Thank you so much! Your help saved me a lot of trouble.

@YWangScience Can you please submit as a pull request.

@jakob thanks for this tool and @YWangScience thanks for your investigation and solution for this issue. Can this fix please be updated into a new version of the product? This is preventing us using teh tool to work with mobile phone numbers, which include a leading zero by default in my country

Has this been merged? I just downloaded the app from the App Store and it is still messing up my leading 0's...

+41 on this. Table tool should not mess with data. This is one of the reasons we use CSV at all.

It's 2023 and I can't believe this is still an open issue.


Modification of the Menu "For Recommenders" --> "Preprints [or Reports] you're handling"

 Modification of the Menu "For Recommenders" --> "Preprints [or Reports] you're handling"


Handle empty location info in a stack frame

You can skip the ++ around the LocationOutput and just put it in the IO list. It will get flattened when printed anyway. This way it's much more efficient.
I can merge if you update this.


ModuleNotFoundError for msldap when starting pypykatz

When starting pypykatz i receive an ModuleNotFoundError for msldap. The package is of course installed.
──(kali㉿kali)-[/tmp]
└─$ pypykatz
Traceback (most recent call last):
  File "/usr/bin/pypykatz", line 33, in <module>
    sys.exit(load_entry_point('pypykatz==0.4.9', 'console_scripts', 'pypykatz')())
  File "/usr/lib/python3/dist-packages/pypykatz/__main__.py", line 16, in main
    from pypykatz.kerberos.cmdhelper import KerberosCMDHelper
  File "/usr/lib/python3/dist-packages/pypykatz/kerberos/cmdhelper.py", line 17, in <module>
    from pypykatz.kerberos.kerberos import get_TGS, get_TGT, generate_targets, \
  File "/usr/lib/python3/dist-packages/pypykatz/kerberos/kerberos.py", line 11, in <module>
    from msldap.commons.url import MSLDAPURLDecoder
ModuleNotFoundError: No module named 'msldap.commons.url'

This are my installed packages:
pip3 install minidump minikerberos aiowinreg msldap winacl
Defaulting to user installation because normal site-packages is not writeable
Requirement already satisfied: minidump in /usr/lib/python3/dist-packages (0.0.21)
Requirement already satisfied: minikerberos in /home/kali/.local/lib/python3.10/site-packages (0.3.5)
Requirement already satisfied: aiowinreg in /usr/lib/python3/dist-packages (0.0.7)
Requirement already satisfied: msldap in /usr/lib/python3/dist-packages (0.4.7)
Requirement already satisfied: winacl in /usr/lib/python3/dist-packages (0.1.5)
Requirement already satisfied: tqdm in /usr/lib/python3/dist-packages (from minikerberos) (4.64.1)
Requirement already satisfied: oscrypto>=1.2.1 in /usr/lib/python3/dist-packages (from minikerberos) (1.3.0)
Requirement already satisfied: asysocks>=0.2.2 in /usr/lib/python3/dist-packages (from minikerberos) (0.2.2)
Requirement already satisfied: unicrypto>=0.0.9 in /usr/lib/python3/dist-packages (from minikerberos) (0.0.9)
Requirement already satisfied: asn1crypto>=1.3.0 in /usr/lib/python3/dist-packages (from minikerberos) (1.5.1)
Requirement already satisfied: asyauth>=0.0.7 in /usr/lib/python3/dist-packages (from msldap) (0.0.9)

based on your traceback, you are using pypykatz version 0.4.9. The current public version is 0.6.3.
Since you're on kali linux, I'd advise you to create a virtual environment for python using the venv module and install pypykatz from github.
Example:
sudo apt install python3-venv
python3 -m venv pypykatzenv
source pypykatzenv/bin/activate
git clone https://github/com/skelsec/pypykatz.git
cd pypykatz
pip install setup.py

This example might not be fully correct as I just wrote it based on memory, bu you get the idea

I have the same issue with version 0.6.3

└─$ pypykatz
Traceback (most recent call last):
File "/usr/bin/pypykatz", line 33, in
sys.exit(load_entry_point('pypykatz==0.4.9', 'console_scripts', 'pypykatz')())
File "/usr/local/lib/python3.10/dist-packages/pypykatz-0.6.3-py3.10.egg/pypykatz/main.py", line 17, in main
File "/usr/local/lib/python3.10/dist-packages/pypykatz-0.6.3-py3.10.egg/pypykatz/kerberos/cmdhelper.py", line 19, in
File "/usr/local/lib/python3.10/dist-packages/pypykatz-0.6.3-py3.10.egg/pypykatz/kerberos/kerberos.py", line 7, in
ImportError: cannot import name 'KerberosUserEnum' from 'minikerberos.security' (/usr/local/lib/python3.10/dist-packages/minikerberos/security.py)

Tried on a fresh ubuntu wsl installation.

can confirm

Hey everyone,
Minikerberos released a new version yesterday.

The problem is related to the new version.
In order to fix the bug, you must install minikerberos version 0.3.5 :
python3 -m pip install minikerberos==0.3.5
Or, you can use pipx !
POC :
root@41ba79158f1b:/# python3 -m pipx install pypykatz
  installed package pypykatz 0.6.3, installed using Python 3.9.2
  These apps are now globally available
    - pypykatz
⚠  Note: '/root/.local/bin' is not on your PATH environment variable. These apps will not be globally accessible
    until your PATH is updated. Run `pipx ensurepath` to automatically add it, or manually modify your PATH in your
    shell's config file (i.e. ~/.bashrc).
done! ✨ 🌟 ✨
root@41ba79158f1b:/# python3 -m pipx inject pypykatz minikerberos==0.3.5
  injected package minikerberos into venv pypykatz
done! ✨ 🌟 ✨
root@41ba79158f1b:/# /root/.local/bin/pypykatz
usage: pypykatz [-h] [-v] {live,lsa,registry,crypto,kerberos,dpapi,ldap,rdp,parser,smb,version,banner,logo} ...
pypykatz: error: the following arguments are required: command
root@41ba79158f1b:/#

ImportError: cannot import name 'KerberosUserEnum' from 'minikerberos.security' (/usr/local/lib/python3.9/dist-packages/minikerberos-0.4.0-py3.9.egg/minikerberos/security.py)

root@41ba79158f1b:/pypykatz# python3 -m pip install minikerberos==0.3.5
Collecting minikerberos==0.3.5

[...TRUNCATED DATA...]

Successfully installed minikerberos-0.3.5

root@41ba79158f1b:/pypykatz# pypykatz
usage: pypykatz [-h] [-v] {live,lsa,registry,crypto,kerberos,dpapi,ldap,rdp,parser,smb,version,banner,logo} ...
pypykatz: error: the following arguments are required: command

Thank you very much @QU35T-code. Also the hint with pipx - tried it out and works like a charm.

Hello everyone!
I did the thing where I updated a module and went on a conference.
What is happening is the following: minikerberos module got updated to a breaking API version and since none of my modules have an upper cap on the required module versions that one slid in on new installs. Reason I haven't noticed is that mainline pypykatz is hosted on Porchetta Industries' git servers and my test version is also there.
I'm happy to see many ppl jumped in here with solutions, that's awesome! The official solution will be that I'll push the current private version of pypykatz here as well.
Some users might not be aware, but I'm also publishing the latest version of pypykatz bundled with a tool called octopwn which is WASM based, so it runs in your browser. When you have problems with LSASS parsing you can simply visit octopwn's website and parse your LSASS dumps there.

based on your traceback, you are using pypykatz version 0.4.9. The current public version is 0.6.3. Since you're on kali linux, I'd advise you to create a virtual environment for python using the venv module and install pypykatz from github. Example:
sudo apt install python3-venv
python3 -m venv pypykatzenv
source pypykatzenv/bin/activate
git clone https://github/com/skelsec/pypykatz.git
cd pypykatz
pip install setup.py

This example might not be fully correct as I just wrote it based on memory, bu you get the idea

Disclaimer:
I'm new to using Python so please bear with me
Windows 10 22H2
Python 3.11.2
Using Windows Terminal (PowerShell):
To create a virtual environment called "pypykatz_venv"
python -m venv pypykatz_venv
To activate aforementioned virtual environment
.\pypykatz_venv\Scripts\activate
To clone the pypykatz GitHub repo
git clone https://github.com/skelsec/pypykatz.git
To enter the cloned pypykatz directory
cd .\pypykatz\
To install pypykatz
pip install setup.py
^ Above fails with
ERROR: Could not find a version that satisfies the requirement setup.py (from versions: none)
ERROR: No matching distribution found for setup.py

Err...I googled around a bit and found the command:
python .\setup.py develop
which returns A LOT of output however it still seems to install "minikerberos" 0.4.0 even though it searches for 0.3.5 and 0.3.5 seems to be available:
https://pypi.org/simple/minikerberos/
Checking "pip list" returns:
minikerberos   0.4.0
Any ideas?

@InfiniteBSOD the issue is not on your end, what you did looks good. I was planning on uploading the new version of pypykatz when minikerberos package was released but that didn't happen until now.
Please try it again, it should fix everything.

Now with the new version is up on both pyp and github (and in the releases) pls someone on this thread verify it is solved so I can close this.

Tests with the new release seems to works fine. We can close this issue !
Thanks @skelsec !


v1.7.0 download files does not support PostgreSQL 13

v1.7.0 donwload files does not contain changes for support PostgreSQL 13. Due to network security reasons we could not clone. Hence we download v1.7 zip file and do make & make install the library and we ended up with error :
"Makefile:50: *** PostgreSQL 9.3 to 12 is required to compile this extension.  Stop.".
When we cloned repo, build was successfull.

Hi @smtspace
Columnar features have been integrated to citus extension. Please review the announcement at https://github.com/citusdata/cstore_fdw#important-notice-columnar-storage-is-now-part-of-citus


Request to publish extension to open-vsx.org

Since it is illegal to use Microsoft's VS Code Marketplace in non-Microsoft builds of Code OSS (like VSCodium, GitPod, Code-Server, OpenVSCode Server, Eclipse Theia, etc), could you please co-publish your extension to the Open-VSX marketplace ?
Wiki with instructions for publishing extensions
Doc on extensions and how to use the Open VSX marketplace

Hi, since GitHub issues are not officially monitored by Snyk support, we will be disabling "Issues" tab on this repository very soon.
Please raise the current issue directly by contact Snyk Support on the web or emailing us at<EMAIL_ADDRESS>


Initial app UI and responsive field monitor

It'll need some more work to make the field monitor look good and have all the information it needs, but I think it's a good start.

@Filip-Kin, looks like CI is failing as well.

@Filip-Kin were you going to address feedback? I can update the PR later if you'd prefer.

@Filip-Kin were you going to address feedback? I can update the PR later if you'd prefer.

I will be addressing the feedback, I just haven't found the time this week to work on it.

Alright, I'm going to go ahead and merge this now.


Implementation of Non-preemptive SJF scheduling

Description of Change
Non pre-emptive  SJF Scheduling algorithm
Added C++ implementation of SJF scheduling algorithm in the directory cpu_scheduling/
Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[x] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.

Notes:

put it in some directory instead of the root directory.
you haven't checked any of the checkboxes in your pull request
please add a description to the pr

https://github.com/TheAlgorithms/C-Plus-Plus/blob/master/CONTRIBUTING.md#Commit-Guidelines
read this contributing file thank you!

@realstealthninja I have done the following

@realstealthninja Could you please review the PR ? And could the hacktober label be put ?

I have done the necessary changes.

changed the file directory to /cpu_scheduling_algorithms
Now, the test function will generate 10 different testcases in which it will print  the before and after the SJF scheduling for each test case.

@realstealthninja Kindly review the PR and please accept it.

The approved label is not added

the approved label is not added and the hacktober has not accepted the repo. Could you tell why ?
…
On Thu, Oct 17, 2024 at 9:07 PM realstealthninja @.> wrote: @.* approved this pull request. Lgtm! — Reply to this email directly, view it on GitHub <#2801 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BBVMSXJBSFNZY5AOPNB33ZTZ37KTDAVCNFSM6AAAAABPUQSYHKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNZVGYYTINZTGE . You are receiving this because you modified the open/close state.Message ID: @.***>

I'm very sorry. I've added it now! :heart:  thanks again for contributing


FEATURE REQUEST: autoreload falco-talon when rules are updated

it could be very useful to give talon the ability to autoreload when the rules configuration file is updated
this way we don't need to rollout restart manually :)

The feature is in main branch.


What about Visual Studio Community?

My Help menu for VSC doesn't have the Report a Problem link -- maybe link me to that help page from here?

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

ID: 11bc8109-61f8-89c4-88f2-ea99c480eec9
Version Independent ID: 3fb40299-caeb-f933-c64c-1825cee9f418
Content: Report a Problem with Visual Studio for Mac - Visual Studio for Mac
Content Source: mac/report-a-problem.md
Product: visual-studio-mac
GitHub Login: @conceptdev
Microsoft Alias: crdun

All recent releases of Visual Studio Community should have the Help > Report a Problem menu item. If you do not see it, I would encourage you to update to the latest version via the Visual Studio > Check for Updates menu item. This will also ensure you have the most up-to-date release with recent bug fixes.
The Report a problem feature is also available on the 'welcome page' when you open Visual Studio for Mac:

The link for reviewing existing entries is https://developercommunity.visualstudio.com/ - we'll look at adding this to the page.


Improve documentation on ttl option

See: https://github.com/ipfs/go-ipfs/pull/5979

Thanks! Always nice to have documentation improvements as well.
(The timeout isn't your fault, so merging.)


Implement 508 Updates

Definition of Ready

[ ] Does this meet the INVEST criteria?
[ ] Does this have a clearly defined user?
[ ] Does this have an estimate?
[ ] Does this have acceptance criteria?
[ ] Does this have sufficient context (wireframes, business rules, etc.)?
[x] Is this traceable to a feature, epic, or PI?

Acceptance Criteria
Update data table to be 508 compliant
Update tab navigation to be 508 compliant
Update Select Facility section drop downs to be 508 compliant
From ticket #563
Hey @cvp-alexrobinson ,
As we discussed, the wireframes as is are ok but there will be necessary coding required to make them compliant in the final application. There are also decisions that would be easier to implement. Here are my notes:
The coding of the UI tabs will be important. They need to be "tab-able."
Be sure to model the tab coding against the aria specs, here are some resources
https://www.w3.org/TR/wai-aria-practices-1.1/#tabpanel
https://www.w3.org/TR/wai-aria-practices-1.1/examples/tabs/tabs-1/tabs.html
https://dequeuniversity.com/library/aria/tabpanel
http://accessibility.athena-ict.com/aria/examples/tabpanel2.shtml
Be sure to discuss tab order with dev and UX team—especially with multiple menus, tables, and accordions.
Tab order should be logical, up to down, left to right
Make sure the tab order is based on the order in the DOM, do not use tabindex because the order must also be logical in the DOM for screen readers
Make sure that scrollable areas can be scrolled using a keyboard.
If specific keystrokes are necessary to scroll, be sure to put visible directions on the page to communicate the keystrokes to the user
"Open" buttons or links should be labeled in the code with what they're actually opening. E.g. "Open Power Plant."
Yes, visibly it's ok to have the buttons say 'Open Tab' but programmatically it should be something like 'Open Tab for . This can be done with an aria label. See example 1 on https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/ARIA_Techniques/Using_the_aria-label_attribute.
Make the tabs only openable from the actual "open" buttons, anything else would be inaccessible or confusing
This would simplify. If the whole row is clickable, then just make sure that the keyboard only tabs to the open tabs buttons
Form fields should only submit input when a user has told it to submit.
This would simplify, otherwise you would need a visible message saying that a change on the page will occur when interacting with the forms, see https://www.w3.org/TR/2016/NOTE-WCAG20-TECHS-20161007/G13
Make sure that sections like the pagination area have hidden headings that identify them as such for screen readers.
The external "show/hide inactive" toggle method is preferred.
Correct, pagination in general will need care in keyboard accessibility and coding. See https://www.a11ymatters.com/pattern/pagination/ for an example. Right now one problem is that if a keyboard user activates a pagination page, 3 for example, visually the table is shifted, but when tabbing back, it goes to the last row of the table.
Other Notes
Be sure the Close tab option (X) is keyboard accessible and that it also has an accessible name (Close tab instead of X)
Be sure forms have programmatic names, right now they have visible labels but the labels are not coded with label tags and associated to the forms, see https://webaim.org/techniques/forms/controls
The dropdown with the active/inactive collapsible elements will be tricky, I'd suggest changing it to a checkbox outside that filters the dropdown values
Sortable tables need to be keyboard accessible and coded so that screen readers know if the column is sorted by ascending or descending, examples https://www.w3.org/TR/wai-aria-practices-1.1/examples/grid/dataGrids.html, https://dequeuniversity.com/library/aria/table-sortable
Definition of Done

[ ] Have all Acceptance Criteria been agreed to and validated by the scrum team?
[ ] Has the code been unit tested and peer reviewed?
[ ] Have the functional tests been executed?
[ ] Have all defects been reviewed and dispositioned, resolved, or deferred?
[ ] Has the user story been accepted by the Product Owner?
[ ] Is the user story ready to be deployed to the staging/integration environment?

Once template is complete and Anthony reviews, this ticket will be revisited.

Split into three tickets: #895, #897, and #899.


[FLOC-1849] Show stderr from admin.runner too.

https://clusterhq.atlassian.net/browse/FLOC-1849
This doesn't address the underlying issue (which is we don't interact with vagrant and virtualbox in a way that cleans up old images properly). But it does address the regression in #1294, where we stopped showing stderr from vagrant.
(See
http://build.clusterhq.com/builders/flocker%2Facceptance%2Fvagrant%2Ffedora-20/builds/2059
where it demonstrates the output).

This appears to work and doesn't seem to break anything. I've looked into some of the Twisted things used but I don't completely understand it. I've asked for some comments but I don't think that it is top priority that this is the shiniest code so merge when you and Buildbot are happy, or resubmit if there are any major changes.


Feedback: Make using preview (.NET9) opt-in

Thought about trying out the blog but now, 60 minutes plus later, it's still not running because of different .NET9 preview issues. I haven't used .NET9 before.
Surely solvable but not anything I'd like to put time into when my focus was to try out the blog, so for the future I'd appreciate preview stuff to be opt-in.

That could be an option - currently that is more of a documentation issue.
The basic idea is to use the "stable tags" like: https://github.com/linkdotnet/Blog/releases/tag/v8.0
There is no preview stuff in them, but that is not obvious at the moment, leading to those issues.
I am not sure whether it's straightforward to opt in (especially with some of the Nuget packages).
Another option is to have an unstable or devbranch.

But the project is bumped to .net9, how do you mean that stable tags should be used?
I guess that there could be a branch for unstable/preview target framework?
Some background on the problems:

First VS didn't find the .NET9 installation. Uninstalling and installing .NET9 didn't solve. Restart did solve.
Then this error: The analyzer assembly references version '<IP_ADDRESS>' of the compiler, which is newer than the currently running version '<IP_ADDRESS>'.. Solved by installing VS 2022 Preview. More on this issue.
Now I'm stuck with the error below, if you'd know the solution I'd appreciate it to get rolling with the blog :)

The main branch contains the latest and greatest state, i.e. net9.0-preview at the moment.
You can checkout the v8.0 tag to go to net8.0:
git checkout v8.0

This points to a commit where net8.0 is "still" used removing the need to install net9.0 and the latest Visual Studio Preview.

Thanks, that clarified! I'm now running the v8.0 tag.
Maybe something about this strategy would be nice in the README? In my experience master/main is the released/stable code so maybe not obvious for other people as well.

Yeah, that is what I meant about a documentation issue! I totally missed that, so thanks for bringing this up @modestotech
But going forward, the approach of using master/main as the stable version might be more suitable. I will make myself some thoughts.

I added a comment in the setup section explaining the difference between master and the tags.
In the future I will pivot towards master/main and a dev or unstable branch. As I already have unstable stuff on master I will do the cut once .net9 is stable.
Hope that makes sense. Thanks again for reporting @modestotech

Looks great @linkdotnet, well done although I think that requiring people that wants to contribute to use .NET previews isn't great (as you've written that contributors should use develop). Furthermore, if work are being done in the dev/unstable branch that requires preview features, you won't be able to merge into main until .NET9 has been released.
I'd keep main and dev with production ready stuff, not dependent on .NET preview, and an unstable branch besides those, that can be based on develop, and introducing new preview features, that can be merged into develop when MS released the next .NET version.
I might want to contribute but I don't want to "waste" time troubleshooting .NET preview stuff, therefore above would work for me.

The problem is that Blazor is "kind of new," and therefore, some features I really want to use are only available in the preview.
But maybe I can pivot in the future to take a more "stable" approach.

Fair enough! Creates a barrier for people that want to contribute but then it's for a good reason 👍


Add python serializer (ufo2flatfont)

So I generated python classes for UFOFontInfo by executing "flatc -p",
Then I wrote a code that reads a UFO and serializes it to a Flatfont (For now, just FontInfo).
Here in the python code ufoff.py, the main concern was skipping the process of calling all setter methods one by one.

but don't run the autoformatter on the python sources that are generated by flatbuffer itself, of course. there's no point reformatting autogenerated code


fix documentation to match code

Closes #159
based off of discussion in #158 .
Changes documentation in order to match Still's source code.

thank you!


External database: won't update currency + time watched

Everything works fine but it will not add currency to the external database. Viewers are added and levels are set correctly but currency and time watched don't get updated

There are known issues with the MySQL solution due to some limitations, this should be fixed in the next version, no ETA.


feat(consensus): develop aggregate crypto with overlord

What type of PR is this?
feature
What this PR does / why we need it:
Develop aggregate crypto with ophelia-bls-amcl crypto protocol.
Add some fields to the config file.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/LGTM


[Transaction] Method for adding a non-confidential issuance to a transaction

We need to add a new AddIssuance method to the *Transaction type of the transaction package.
This new function will have the following signature:
func (tx *Transaction) AddIssuance(arg AddIssuanceArg) (uint32, error)

It expects a single argument defined like:
type AddIssuanceArg struct {
   Precision uint,
   Contract *IssuanceContract,
   AssetAmount float64,
   TokenAmount float64,
   AssetAddress string,
   TokenAddress string,
}

About the fields of this type, it is important to note that:

the asset and token amount are expressed in BTC rather than Sat. After instantiating a new issuance object, the amounts in satoshis can be taken from the latter.
the function using these args will add some outputs to the transaction (as better explained below), so it will need to get the correct scripts from the provided addresses.

Following there are the steps that the new function have to do to correctly add an unblinded issuance to a transaction that must contain at least 1 input:

Check that transaction contains at least one input
Instantiate a new *TxIssuanceExtended.
a)  Check that AssetAmount and AssetAddress are defined
b) Check that TokenAmount and TokenAddress are defined if TokenAmount is > 0
Get the index of the first input of the transaction that has empty issuance
Generate entropy, asset (and token if defined) using the selected input hash and index and the provided Contract
Add the issuance to the designed input
a) Add an output to the transaction with asset amount, asset hash and script derived from asset address
b) Optionally add an output with token amount, token hash and script derived from token address if the provided TokenAmount is > 0
Finally, return the index of the selected input containing the issuance, or an error if any occurs

You can retrieve the asset hash from the instance of *TxIssuanceExtended, through the method GenerateAsset.
Correct, but consider that the asset amount expected by the AddIssuance function is expressed in BTC, while the AssetAmount filed of the instance of *TxIssuanceExtended is instead in the confidential value format ([9]byte{}) and ready to be put in a tx output.
Yes, consider that the btcd's method you mentioned, unfortunately, is not reusable because it does not support addresses with prefixes different from the standard Bitcoin ones.

You can retrieve the asset hash from the instance of *TxIssuanceExtended, through the method GenerateAsset.
Correct, but consider that the asset amount expected by the AddIssuance function is expressed in BTC, while the AssetAmount filed of the instance of *TxIssuanceExtended is instead in the confidential value format ([9]byte{}) and ready to be put in a tx output.
Yes, consider that the btcd's method you mentioned, unfortunately, is not reusable because it does not support addresses with prefixes different from the standard Bitcoin ones.

@altafan @tiero shall we create util method PayToAddrScript in address.go?

shall we create util method PayToAddrScript in address.go?

I would call this new function ToOutputScript, so when it's used it looks like address.ToOutputScript, that seems pretty readable IMO.

Maybe we also need DecodeAddress, similar to btcutil.DecodeAddress to determine type of address and than based on type to generate script?

Correct

shall we create util method PayToAddrScript in address.go?

I would call this new function ToOutputScript, so when it's used it looks like address.ToOutputScript, that seems pretty readable IMO.

Maybe we also need DecodeAddress, similar to btcutil.DecodeAddress to determine type of address and than based on type to generate script?

Correct

@sekulicd @altafan ACK

Actually, we made this function responsible for too many things, and making it working with the pset package as it is now will be quite complicated.
We should lighten this AddIssuance function so that by changing the 7th and 8th steps defined above.
Instead of creating and adding the outputs to the transaction, we should just create and return them along with the index of the input.
Also, the type of the returned input index should be changed to int so that we can return a -1 value when an error occurs.
To summarize, the returning types of this function should change like:
AddIssuance(...) (int, *TxOutput, *TxOutput, error) {...}

Note that the second returned output should be nil if the provided TokenAmount  is 0.


[PSET Input] Create

A method NewPsetInput accepting either a witness or non witness utxo and returning a *PsetInput must be added to the package in order to be able to create inputs.
Ref #17.

Should be possible to specify a sighHashType too. Could you add this in the spec? @altafan

Inspired by btcutil implementation, this would be our pset input type definition:
type PInput struct {
	NonWitnessUtxo     *transaction.Transaction
	WitnessUtxo        *transaction.TxOutput
	PartialSigs        []*PartialSig
	SighashType        txscript.SigHashType
	RedeemScript       []byte
	WitnessScript      []byte
	Bip32Derivation    []*Bip32Derivation
	FinalScriptSig     []byte
	FinalScriptWitness []byte
	Unknowns           []*Unknown
}


Investigate what happens to the app when the selected region flips its experimental status

What happens when the app switches from experimental -> prod -> experimental?
Does the app keep working properly?

(yes, it keeps working fine.)


enable SMB/UAM for a full 6 hours after any carb entry

With shorter DIAs and large meals with undercounted carbs, COB often decays to 0 before carb absorption finishes, and UAM cuts off after DIA hours.  This preference, if enabled, would allow UAM to remain active and allow oref1 to SMB if needed for 6 hours after any carb entry, unless disabled by a high temp target.

Tested successfully on our rigs.  Ready to merge if no objections.


Add new field to contact us form

📝 Changed routes:

/contact-us

(view full size)

And 1 other route:

/api/hubspot

Commit 72c427feabc638e2dc6f5e1613e9cdb617d2dfb4 (https://thirdweb-1l7joxzos.thirdweb-preview.com).


Gradle 7.6, Update Actions and fabric-loader.

Updates the GitHub Actions to make comments look a little better; Upstreams the upload-artifact to v3 which includes ubuntu being upstreamed to 22.04 in syncing configuration,
Also upstreams the fabric loader to be on 0.14.12 instead along with Gradle being on 7.6 which adds support for JDK 19 when self-compiling.

Unfortunately it doesn't look like the project is buildable with Gradle 7.6: https://github.com/Aizistral-Studios/No-Chat-Reports/actions/runs/3825701114/jobs/6508900982
I guess we'll have to live with 7.4 for a while longer.

Or use gradle 7.5.1 perhaps?

If it builds 🤷 https://github.com/Aizistral-Studios/No-Chat-Reports/actions/runs/3825746869/jobs/6508957371

Alright, it seems to work, so the answer is yes.

that has worked lol.


Permission groups section

I want to merge this change because...
adds frontend for https://github.com/mirumee/saleor/pull/5176
To be used with API branch:  feature/permission-groups
Screenshots

Pull Request Checklist

[x] All visible strings are translated with proper context.
[x] All data-formatting is locale-aware (dates, numbers, and so on).
[x] Translated strings are extracted to .pot file.
[x] Number of API calls is optimized.
[x] The changes are tested.
[x] Type definitions are up to date.
[x] Changes are mentioned in the changelog.

Blocked by https://github.com/mirumee/saleor-dashboard/issues/453

Note: something weird happened to schema, prettier maybe?


calculate_methylation_frequency crashes

Hi!
We successfully (?) called methylation using
 /home/qf0297/nanopolish/nanopolish call-methylation -t 8 -r mtdna_reads_alba.fasta -g rCRS_ME.fasta -b sorted_mtdna_alba.bam > methylation.tsv
This seemed to work fine, producing a large output file. When running
python ~/nanopolish/scripts/calculate_methylation_frequency.py -c 2.5 -i methylation.tsv > meth_frequencies.tsv
the terminal reports
 # error in calculate_methylation_frequency.py
qf0297@qf0297-ESPRIMO-P420:/media/qf0297/Volume/data_np/20170627_mtDNA_HEK/alba/fast5_alba$ python /home/qf0297/nanopolish/scripts/calculate_methylation_frequency.py -c 2.5 -i methylation.tsv > meth_frequencies.tsv
  File "/home/qf0297/nanopolish/scripts/calculate_methylation_frequency.py", line 76
    print "\t".join(["chromosome", "start", "end", "num_cpgs_in_group", "called_sites", "called_sites_methylated", "methylated_frequency", "group_sequence"])
             ^
SyntaxError: invalid syntax

How can this be resolved? Is this an issue in the output? The methylation.tsv looks quite plausible.
Thank you!
Stefan

What version of python are you using?

Python 3.6.1

Ok - I'll change that script to support python3-style prints. In the meantime it should work if you use python2

Fixed in 1ccd559

Hi!
I cloned nanopolish again, but it still gives the same error.
qf0297@qf0297-ESPRIMO-P420:/media/qf0297/Volume/data_np/20170627_mtDNA_HEK/alba/fast5_alba$ python ~/nanopolish/scripts/calculate_methylation_frequency.py -c 2.5 -i methylation.tsv > meth_frequencies.tsv   File "/home/qf0297/nanopolish/scripts/calculate_methylation_frequency.py", line 82     print "\t".join([str(x) for x in [c, s, e, sites[key].group_size, sites[key].called_sites, sites[key].called_sites_methylated, f, sites[key].sequence]])              ^ SyntaxError: invalid syntax qf0297@qf0297-ESPRIMO-P420:/media/qf0297/Volume/data_np/20170627_mtDNA_HEK/alba/fast5_alba$
But the script is however indeed the new one.
cat calculate_methylation_frequency.py
[...]
print("\t".join(["chromosome", "start", "end", "num_cpgs_in_group", "called_sites", "called_sites_methylated", "methylated_frequency", "group_sequence"]))

for key in sites:
    if sites[key].called_sites > 0:
        (c, s, e) = key.split(":")
        f = float(sites[key].called_sites_methylated) / sites[key].called_sites
        print "\t".join([str(x) for x in [c, s, e, sites[key].group_size, sites[key].called_sites, sites[key].called_sites_methylated, f, sites[key].sequence]])

The new nanopolish also gives this error when opening the help
qf0297@qf0297-ESPRIMO-P420:/media/qf0297/Volume/data_np/20170627_mtDNA_HEK/alba/fast5_alba$ python ~/nanopolish/nanopolish --help   File "/home/qf0297/nanopolish/nanopolish", line 1 SyntaxError: Non-UTF-8 code starting with '\xc0' in file /home/qf0297/nanopolish/nanopolish on line 2, but no encoding declared; see http://python.org/dev/peps/pep-0263/ for details
qf0297@qf0297-ESPRIMO-P420:/media/qf0297/Volume/data_np/20170627_mtDNA_HEK/alba/fast5_alba$ python ~/nanopolish/nanopolish variants   File "/home/qf0297/nanopolish/nanopolish", line 1 SyntaxError: Non-UTF-8 code starting with '\xc0' in file /home/qf0297/nanopolish/nanopolish on line 2, but no encoding declared; see http://python.org/dev/peps/pep-0263/ for details


Issue: Chromadb document deletion not working

Issue you'd like to raise.
I am trying to delete a single document from Chroma db using the following code:
chroma_db = Chroma(persist_directory = embeddings_save_path,
embedding_function = OpenAIEmbeddings(model = os.getenv("EMBEDDING_MODEL_NAME"),
chunk_size = 1,
max_retries = 5)
)
chroma_db._collection.delete(ids = list_of_ids)
chroma_db.persist()
However, the document is not actually being deleted. After loading/re-loading the chroma db from local, it is still showing the document in it.
I have tried the following things to fix the issue:
I have made sure that the list of ids is correct.
I have tried deleting the document multiple times.
I have tried restarting the Chroma db server.
None of these things have worked.
I am not sure why the document is not being deleted. I would appreciate any help in resolving this issue.
Thanks,
Anant Patankar
Suggestion:
No response

🤖 AI-generated response by Steercode - chat with Langchain codebase
Disclaimer: SteerCode Chat may provide inaccurate information about the Langchain codebase.
It seems like you are trying to delete a document from the Chroma collection using the _collection.delete() method. However, the proper method to delete a document from the Chroma collection is delete_document(). You can use this method as follows:
chroma_db = Chroma(persist_directory=embeddings_save_path,
                  embedding_function=OpenAIEmbeddings(model=os.getenv("EMBEDDING_MODEL_NAME"),
                                                      chunk_size=1,
                                                      max_retries=5)
                  )

# Delete the document
for document_id in list_of_ids:
    chroma_db.delete_document(document_id)

# Persist the changes
chroma_db.persist()

Make sure to replace list_of_ids with the list of document IDs you want to delete. This should properly delete the documents from the Chroma collection.
You can continue the conversation at https://steercode.com/

I have the same question, @anant-patankar , did you solved the problem?

Ensure the list of ids are strings.
 def delete(self, ids: Optional[List[str]] = None, **kwargs: Any) -> None:

The steercode solution does not work. It gives me the following error:
AttributeError: 'Chroma' object has no attribute 'delete_document'

delete collection
Chroma(collection_name="chroma_db", embedding_function=embedding).delete_collection()

you can also pass client ti it if using client/server mode.

Just replace function name delete_document with delete, there is no delete_document function in Chroma class.
this is the code snippet of delete function inside Chroma class:
def delete(self, ids: Optional[List[str]] = None, **kwargs: Any) -> None:
"""Delete by vector IDs.
    Args:
        ids: List of ids to delete.
    """
    self._collection.delete(ids=ids)

Hi everyone, chiming in on this,
I tried what you suggested and used
collection.delete(     ids=collection.get().get('ids') )
but when looking at the SQLlite database, I can still see the entries in the table 'embedding_fulltext_search_data'.
I also tried:
chroma_client.delete_collection("test")
and the data is not shown with:
collection.get()
but it seems to still remains in the database (table: 'embedding_fulltext_search_data'), which eats up a lot of memory when deleting frequently.
Is there a way to complete remove ids and corresponding data from the database, or completely remove en entire collection?
Thank you,
Cornelius

Hi - has anyone found a solution yet? I'm facing the same issue.

Same issue here. I'm calling the endpoint api/v1/collections/<collection_name> method DELETE but I get only delete the entry in collections table, all the documents, metadata and embedding_fulltext_search are still in the sqlite database

Reference the below code. This works after the update to chroma, where now it uses SQLite instead of duckdb. Based on the file_name the deletion will take place.
`import chromadb
import os
from langchain.vectorstores import Chroma
persist_directory = "Database\chroma_db\"+"test3"
if not os.path.exists(persist_directory):
os.makedirs(persist_directory)
Get the Chroma DB object
chroma_db = chromadb.PersistentClient(path=persist_directory)
collection = chroma_db.get_collection(name="langchain")
Get the metadata list
metadata_list = collection.get()['metadatas']
print(metadata_list)
To get the filenames from the metadata, where the filename is stored inside the dictionary as the index - 'source'
file_names = []
for metadata in metadata_list:
filename = metadata['source'].split('\')[-1]
if filename not in file_names:
file_names.append(filename)
print(file_names)
the below will fetch the ids of the all the document chunks from this doc1 file name
print(collection.get(where={"source": "uploads\temp_files\doc1.pdf"})['ids'])
the below deletes all the chunks from the doc1 file
collection.delete(
where={"source": "uploads\temp_files\doc1.pdf"}
)`

I have strong thoughts that will not delete documents store in embedding_fulltext_search* tables because in these tables there are no ids that would enable filtering via collection id nor document id. E.g. embedding_fulltext_search even have just one column which is the document itself, without any ids

If you want to delete documents by IDs, consider the following code:
It worked perfectly for me.
Provide a document /file name; based on that, it will aggregate ID's associated with that file name and start deleting them.
from chromadb.config import Settings

persist_directory = "./testing"
if not os.path.exists(persist_directory):
    os.makedirs(persist_directory)

# Get the Chroma DB object
chroma_db = chromadb.PersistentClient(path=persist_directory, settings=Settings(allow_reset=True))
collection = chroma_db.get_collection(name="docs_store_v2")

# Function to delete documents by IDs
def delete_documents(ids):
    if ids:
        # Delete the documents by IDs
        collection.delete(ids=ids)
        print("Documents have been deleted from the collection.")
    else:
        print("No documents found with the given filename.")

# Function to prompt user for resetting the database
def reset_database(client):
    confirm = input("Do you want to reset the database? (y/n): ")
    if confirm.lower() == 'y':
        client.reset()
        print("Database has been reset.")
    else:
        print("Database reset cancelled.")

# Function to prompt user for deleting the collection
def delete_collection(client):
    confirm = input("Do you want to delete the collection? (y/n): ")
    if confirm.lower() == 'y':
        collection_name = collection.name
        client.delete_collection(name=collection_name)
        print(f"Collection '{collection_name}' has been deleted.")
    else:
        print("Collection deletion cancelled.")

# Get all documents in the collection
db_data = collection.get()

# Extract metadata
metadatas = db_data['metadatas']
ids = db_data['ids']

# Display all source file names present in the collection
print("Source file names present inside the collection:")
source_file_names = set(metadata.get('source') for metadata in metadatas)
for source_file_name in source_file_names:
    print("- " + source_file_name)

# Get the filename from the user
filename = input("\nEnter the filename you want to delete (e.g., 'example.txt'): ")

# Find document IDs with matching filename
ids_to_delete = [id for id, metadata in zip(ids, metadatas) if metadata.get('source') == filename]

# Delete the documents with matching IDs
delete_documents(ids_to_delete)

# Print the updated list of files in the collection
print("\nUpdated files in the collection:")
updated_metadata_list = collection.get()['metadatas']
updated_file_names = set(metadata.get('source') for metadata in updated_metadata_list)
for updated_file_name in updated_file_names:
    print("- " + updated_file_name)

# Ask the user if they want to delete the collection
delete_collection(chroma_db)

# Ask the user if they want to reset the database
reset_database(chroma_db)

Hello I am also worried about this bug as well. I have followed the above to remove my documents collection.delete(ids=ids) but I am still seeing db data as well as the folder that is created when embeddings are made (has files like: data_level0.bin, link_lists.bin, i dont know what these are).
I am noticing embedding data in embedding_fulltext_search etc records with plain text just being left behind. All the other document data has been removed, but this still remains. Also the folder with db files remains.
Everything else in the db seems to be removed successfully except these two things.

any update? it seems that chroma DB still include deleted data!

I've observed that during document upload, the file gets split into multiple chunks, resulting in multiple documents being uploaded with identical IDs. Consequently, when we try to delete a document, only the first matching document from the top order is deleted, while the remaining documents remain untouched.

This is a serious issue for us. We are trying to delete outdated documents and replace them with updated documents in an active vector store.
The following bug.py script deletes and adds 5000 documents to a vector store:
#!/usr/bin/env python
import chromadb
import numpy as np
import subprocess
import time

NUM_DOCS = 5_000
EMBEDDING_SIZE = 1000
VS_PATH = "./vs_test"

# disk usage in human readable format (e.g. '2,1GB')
du = lambda path: subprocess.check_output(["du", "-sh", path]).split()[0].decode("utf-8")

# create/open the vector store
client = chromadb.PersistentClient(VS_PATH)
collection = client.get_or_create_collection(name="test")

# delete existing documents
ids = collection.get()["ids"]
print(f"Deleting {len(ids)} existing documents...")
start_time = time.time()

if ids:
    collection.delete(ids=ids)

print(f"{collection.count()} documents after deletion.")
end_time = time.time()
print(f"Document deletion runtime: {round(end_time - start_time)} seconds")

# add new documents
ids = [str(id) for id in range(NUM_DOCS)]
embeddings = [list(np.random.normal(size=EMBEDDING_SIZE)) for id in ids]
print(f"Adding {len(ids)} documents...")
start_time = time.time()

collection.upsert(ids=ids, documents=ids, embeddings=embeddings)

end_time = time.time()
print(f"{collection.count()} documents after addition.")
print(f"Document addition runtime: {round(end_time - start_time)} seconds")

# print on-disk size
print(f"Vector store size: {du('./vs_test')}")
print("")

When I run this five times:
rm -rf ./vs_test/ && \
  ./bug.py && \
  ./bug.py && \
  ./bug.py && \
  ./bug.py && \
  ./bug.py

I get increasing addition runtimes and on-disk vector store sizes:
Deleting 0 existing documents...
0 documents after deletion.
Document deletion runtime: 0 seconds
Adding 5000 documents...
5000 documents after addition.
Document addition runtime: 8 seconds
Vector store size: 44M

Deleting 5000 existing documents...
0 documents after deletion.
Document deletion runtime: 2 seconds
Adding 5000 documents...
5000 documents after addition.
Document addition runtime: 9 seconds
Vector store size: 86M

Deleting 5000 existing documents...
0 documents after deletion.
Document deletion runtime: 2 seconds
Adding 5000 documents...
5000 documents after addition.
Document addition runtime: 19 seconds
Vector store size: 129M

Deleting 5000 existing documents...
0 documents after deletion.
Document deletion runtime: 2 seconds
Adding 5000 documents...
5000 documents after addition.
Document addition runtime: 27 seconds
Vector store size: 171M

Deleting 5000 existing documents...
0 documents after deletion.
Document deletion runtime: 2 seconds
Adding 5000 documents...
5000 documents after addition.
Document addition runtime: 30 seconds
Vector store size: 214M

On our production vector store with ~55k documents, the document addition time grew to 11 minutes and the on-disk size grew to 4.2 GB after several deletion/addition cycles.

Hello all!
I had the same problem in production and it was very serious for our company!
We add collections with many vectors/documents and update them very often.
The problem is that if you take a closer look at the SQLite3 database, all the deleted information with deleted links (foreign keys) keeps adding up, but the DB keeps getting bigger and bigger.
In a short space of time we reached over 13 GB in the ChromaDB database folder and the server memory was exploding!
I found a strange, and temporary, solution by testing numerous solutions....
Here it is:
            ids = chromaColl.get()['ids']
            if ids :
                chromaColl.delete(ids)
            del chromaColl
            _chromadb.delete_collection(collectionName)

Why is it absolutely necessary to call these 2 deletions in order to empty the data correctly?
Thank you !

@jczic - thanks very much for sharing this! Does this allow the documents to be deleted and refreshed while there are active connections (with the understanding that those connections have a brief window of reduced data)?

@chrispy-snps
I don't know, but in any case it allows you to delete a collection correctly.
You need to recreate it afterwards if you want to update everything.
You can also try opening the chroma.sqlite3 files with this great open-source software: SQLiteBrowser.ORG to browse the data organised by ChromaDB :)

This also seems to stem from the way Chroma is used, particularly in multithreaded/asynchronous mode.
(see https://github.com/chroma-core/chroma/issues/1908)
For my part, I don't use asyncio or FastAPI in Python, but I'm in production with MicroWebSrv2 🙂

Hello all, I was having the same problem : I deleted some documents and I was trying to query my collection and I was having "None" values and this helped :
from chromadb.api.client import SharedSystemClient

client._system.stop()
SharedSystemClient._identifer_to_system.pop(client._identifier, None)

Adding this after the delete documents solved my problem 🙂


Create a BoundingCircle from 3 vertices

Fixes: https://github.com/PurpleKingdomGames/indigo/issues/528

bounding-circle-from-three-vertices.webm
Proof of the pudding.


Fixes to get LPC4330 GCC based builds to run

I was getting Hard Faults in even the simplest of samples before I made
these fixes.

WaitUs() did nothing on optimized builds.  I added the volatile
qualifier to the cyc variable to make sure that the delay loop
doesn't get optimized out.
I removed the #ifdef which skipped the fpuInit() call when building
with GCC.

Is anything using WaitUS() ?


Use compare cache for dimensions in incremental index

NavigableMap in IncrementalIndex frequently compares values to sort input rows. If cardinality is not that high, we can cache the compare result and use it again.

Marked WIP because need modification on DimensionsSpec to accept per dimension configuration. With 2K cache (About 2M for each dimension), got 5~10 % performance enhancement of on-heap ingestion.
BEFORE (msec)
6590
4454
4774
4713
4619
4583
4884
4545
4566
4381
4565
4442
AFTER (msec)
4502
4007
4461
4430
3909
4020
4211
4178
4224
4136
4381
4072

added compareCacheSize in StringDimensionSchema

Looks good to me, would be good to have some more unit tests for the cache.

@navis can you think of edge cases where this cache may actually cause performance to be worse?

@xvrl when the cardinality exceeds cache capacity, the effectiveness will start degrade. I'm thinking of disabling it when some threshold exceeds (3x of capacity?). But generally it should not affect performance in any time.


Fix dark theme for properties panel

Fixes #1245
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

This is how the side panel looks with all the changes

Looks good, only nit is that the inverted carat for the runtime image selection is a bit washed out compared with the rest of the icons

@akchinSTC Good catch, I will create a follow up issue


There's no way to style react-slick component

I'm trying to reproduct this in my own project but the style it's to crazy:
https://react-slick.neostack.com/docs/example/simple-slider
Here a screenshot without import react-slick style because the style turns my page crazy.

My codesanbox replicated example only work because it don't have another components / styles.
The same code is as in the screenshot above.
https://codesandbox.io/s/m7k3l5503j

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

I solved that:
https://stackoverflow.com/questions/40914808/react-slick-not-rendering-slides
https://stackoverflow.com/questions/38269293/displaying-react-components-in-carousel-using-react-slick


html editor replacement

Describe the solution you'd like
Replace the current editor with an wysiwyg-editor. An option could be the following: https://iliyazelenko.github.io/tiptap-vuetify-demo/#/simple

Simple TryOut of the editor can be found here: https://iliyazelenko.github.io/tiptap-vuetify-demo/#/simple
The current implementation can be found in /features/admin/features/the-contact-points/components/EditContactPoint

Other options include:

https://www.npmjs.com/package/vue2-editor
https://vueup.github.io/vue-quill/

This one seams to work: https://github.com/chmln/vue-wysiwyg

More are here: https://medium.com/@mounoydev/vue-component-rich-text-editor-418579cb5858


Change args to list

Could this be the change requested by #446 ?

Changed all *args arguments from core_schema.py to use type list instead

@realDragonium yes, this is what @adriangb was talking about in that issue. Looks good to me but I'll defer to the others.

Codecov Report

Merging #508 (8102d72) into main (8b2be89) will increase coverage by 0.01%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #508      +/-   ##
==========================================
+ Coverage   94.58%   94.59%   +0.01%
==========================================
  Files          93       93
  Lines       12014    12047      +33
  Branches       25       25
==========================================
+ Hits        11363    11396      +33
  Misses        644      644
  Partials        7        7

Impacted Files
Coverage Δ

pydantic_core/core_schema.py
96.74% <100.00%> (+0.01%)
:arrow_up:

... and 4 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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

great, thank you so much.


WSL NullType should be a special type variable so it can unify with any pointer type

Migrated from https://bugs.webkit.org/show_bug.cgi?id=176235:
At 2017-09-01T18:09:40Z<EMAIL_ADDRESS>wrote:
...

At 2017-09-02T21:08:49Z<EMAIL_ADDRESS>wrote:
Created attachment 319740
the patch

At 2017-09-02T21:29:41Z<EMAIL_ADDRESS>wrote:
Created attachment 319741
the patch

At 2017-09-02T21:10:36Z<EMAIL_ADDRESS>wrote:
Comment on attachment 319740
the patch
View in context: https://bugs.webkit.org/attachment.cgi?id=319740&action=review

Tools/WebGPUShadingLanguageRI/Test.js:628
+gc();

I'll remove this.  I put it in there out of curiosity.  Turns out this program allocates about 10MB, so it doesn't get to GC yet even with all these tests.

At 2017-09-03T17:25:19Z<EMAIL_ADDRESS>wrote:
Comment on attachment 319785
the patch
View in context: https://bugs.webkit.org/attachment.cgi?id=319785&action=review

Tools/WebGPUShadingLanguageRI/Test.html:3
+WSL Test

So we're forbidding load() inside any of these js files?

At 2017-09-03T17:43:40Z<EMAIL_ADDRESS>wrote:
Landed in https://trac.webkit.org/changeset/221549/webkit

At 2017-09-03T16:42:23Z<EMAIL_ADDRESS>wrote:
Created attachment 319785
the patch
Added facilities for NullType to figure out what type it got unified with.

At 2017-09-27T19:41:52Z<EMAIL_ADDRESS>wrote:
rdar://problem/34693782


tinyxml2.pc.in: restore prefix to lib/include dir

Since d89d6d9551e5343de4f70ac534f61c3057713f92, the tinyxml2.pc file is missing the install prefix from the includedir and libdir variables. Here is what I see on my system:
prefix=/usr/local/Cellar/tinyxml2/7.0.0
exec_prefix=${prefix}
libdir=lib
includedir=include

Name: TinyXML2
Description: simple, small, C++ XML parser
Version: 7.0.0
Libs: -L${libdir} -ltinyxml2
Cflags: -I${includedir}

This restores the prefix to those variables within the pc file.
Also, I noticed this after testing the new homebrew release of 7.0.0, so I would request a 7.0.1 patch release with this change, assuming that it is accepted.

For reference, here is the previous version of the pc file:
prefix=/usr/local/Cellar/tinyxml2/6.2.0
exec_prefix=${prefix}
libdir=/usr/local/Cellar/tinyxml2/6.2.0/lib
includedir=/usr/local/Cellar/tinyxml2/6.2.0/include

Name: TinyXML2
Description: simple, small, C++ XML parser
Version: 6.2.0
Libs: -L${libdir} -ltinyxml2
Cflags: -I${includedir}

Even if you don't have time to make a patch release, merging this will make it easier to get this patch applied to the homebrew formula, since patches that have already been accepted upstream are preferred.


Revert "chore(deps): update dep typescript from 4.3.5 to v4.6.2"

This reverts commit 1f83593e851cd4aebcd6ef5166b68f6c0a1d10c2.
Will disable typescript from being automerged.

Interesting: I guess https://github.com/artsy/force/pull/9622 went green because of Github being down. And so it got merged.


Subdivision surfaces proposal for Godot Engine

Followup proposing a gltf extension for division surfaces similar to https://github.com/KhronosGroup/glTF/pull/1620 for quads in 3d characters

Postponed. Needs to be done on the Godot Engine org.


Problems in python 2 version

Describe the bug
I found that lack of future imports and use of f-strings in python 2 version. But there can be more errors like that. I did not went through the entire code.
To reproduce
anime-dl --help will give the error due to no print function was imported. And when I fix that I got f-string error with the same command.
anime-dl --help   # This is just a one scenario

I just tried.. :)

I just tried :)

I just tried.. :)
Currently using python 3. Great app..


Add step in creating traceback

With the experimental version of Avant-Idle, it is possible to show the same traceback multiple times, by changing the verbosity level.  However, the traceback thus shown has the language "hard-coded".
By adding an extra step between the extraction of the information and the formatting, it might be possible to show a given traceback in different languages.

This is probably not doable in any reasonable way given how the generic and especially the specific cause are processed.
In reality, a user would almost certainly never need to switch back and forth between languages.


remove unnecessary things from build

Issue #, if available:
Description of changes:

Remove unnecessary files
Clean up build
Switch to cmake for log4cplus build instead of download/loading/using auto tools

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report
Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 16.48%.  Consider uploading reports for the commit 6c2ab32 to get more accurate results

Files
Patch %
Lines

samples/kvssink_gstreamer_sample.cpp
0.00%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1151   +/-   ##
========================================
  Coverage    16.48%   16.48%
========================================
  Files           50       50
  Lines         7018     7018
========================================
  Hits          1157     1157
  Misses        5861     5861

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


MAE computation is different from Ref-NeRF

Hi, I have been running experiments on Shiny Blender recently, and I want to compare my results with yours. But I find that your MAE computation is different from Ref-NeRF.
Ref-NeRF averages MAE weighted on alpha*acc:
https://github.com/google-research/multinerf/blob/5b4d4f64608ec8077222c52fdf814d40acc10bc1/internal/ref_utils.py#L45-L50
https://github.com/google-research/multinerf/blob/5b4d4f64608ec8077222c52fdf814d40acc10bc1/eval.py#L156-L163
But you average MAE on all pixels.
https://github.com/half-potato/nmf/blob/3eb60392cb7b0958bcd38b4f46663596a897dfa0/renderer.py#L375-L376
Since transparent pixels' MAE is 0, final results would be much smaller than Ref-NeRF's.

Thanks for bringing this to my attention. I will recalculate the values and update the paper.

I have updated the arxiv paper.

Thanks for your update.
I have encountered another problem when reproducing the experiment on helmet. I run the experiment with:
python train.py -m expname=v38_noupsample model=microfacet_tensorf2 dataset=helmet vis_every=5000 datadir={dataset dir}
But it seems to fail to learn the correct geometry and normal vector.

I'm not very clear on whether the novel view synthetic results in Table 1 were trained on HDR or original images from Shiny Blender. Would the use of HDR images influence the results?

This is probably caused by the incorrect mixing mode. Can you check the output config.yaml to see if the diffuse mixing mode is set to "fresnel"?
diffuse_mixing_mode: "fresnel"
If it isn't you can set this by passing model.arch.model.diffuse_mixing_mode="fresnel"

I'm using the latest version of code, where the diffuse_mixing_mode is set "fresnel" by default.
By comparing the output config.yaml with your relighting experiment's config.yaml, I have found that only by setting field.smoothing to 1, can the correct normals be obtained.
The command I use:
python train.py -m expname=smoothing model=microfacet_tensorf2 dataset=helmet field.smoothing=1 vis_every=5000 datadir={dataset dir}

Thanks for the help with fixing the default settings!


New strictTemplates option causes issues with data-* attributes

It seems the fix for #1413 is not complete, I'm seeing data-* errors with the latest Volar and strictTemplates on.

See https://github.com/johnsoncodehk/volar/issues/1418#issuecomment-1179656286.
You need to define HTMLAttributes interface to allow data-*, see https://github.com/johnsoncodehk/volar/issues/1077#issuecomment-1146573602.

A suggestion to manually add definitions for all attributes that are already valid HTML attributes doesn't really make sense to me. This applies both to data-* and aria-*.
And I suppose mine #1871 is a duplicate.

I understand that there might be technical difficulties with solving it but if Volar strives to be user-friendly official Vue language server then it can't require everyone to do such basic things.
We should at least keep some issue open for it so that it's clear that it's a bug (or something to solve at some point) and not make this suggestion an official response.
Maybe the solution would be for volar to include the list of known attributes (would work for aria-* at least).

FYI it should handle by dataAttributes and htmlAttributes options after v1.0: https://github.com/johnsoncodehk/volar/blob/19cc0f56865fa3827420b15e85f4ab26e9850e18/vue-language-tools/vue-language-core/schemas/vue-tsconfig.schema.json#L39-L48


Added last compare and codecs writer.

@bjlittle just pep8 to fix please

Hmmm I keep needing to re-run some of the pk3 image tests, that are failing on travis and they shouldn't ... I see this as very bad behaviour indeed. Particularly as I've not touched them, which means to me the image testing is unstable and everyone is going to be stung by this ...

Hmmm I keep needing to re-run some of the pk3 image tests (twice now), that are failing on travis and they shouldn't ... I see this as very bad behaviour indeed. Particularly as I've not touched them, which means to me the image testing is somewhat unstable and everyone is going to be stung by this ...

@bjlittle I agree, there is a follow up activity here which we need to make time for.  do we have an issue on github for this? could we have one and link to it from here please
i think we should consider this outside of the 1.11 milestone, we can cut 1.11 with this, but I think it is technical debt which we have created anew, in trying to reduce our overhead.  perhaps the v2 milestone is the appropriate place for this?

@marqh I've captured this in issue https://github.com/SciTools/iris/issues/2195

@marqh Agreed. LGTM :smile:


Update CLI script M1 Support when github actions allows

Currently github actions don't support pushing darwin arm distros. The CLI script currently works around this by pulling the darwin amd distro. Once github actions supports darwin arm the CLI script will need to be updated to pull the correct distro.
Appropriate build scripts will need to be updated as well so we actually push a darwin arm distro
https://github.com/actions/runner/issues/805

https://github.blog/changelog/2024-01-30-github-actions-macos-14-sonoma-is-now-available/


Parameters from URL path are not decoded.

Parameters from URL path are not decoded. e.g. users/{value}. Shouldn't we decode them?

@zboralski could you please elaborate more how this affects the package and for users? How did you notice this?