OAuth Endpoints Deprecated in 3.1, Discontinued in 3.4 GitHub Enterprise

Some OAuth breaking changes are coming upstream from GitHub Enterprise:
<EMAIL_ADDRESS>
We currently use an oauth access_token endpoint in Classy here:

The endpoint that we use doesn't have access_token as a parameter. Instead, it uses the access_token as a query string parameter.
https://docs.github.com/en/developers/apps/building-oauth-apps/authorizing-oauth-apps
This endpoint appears to still be valid without any issues. The deprecated endpoints are, instead:


Workaround for python layer in Windows

When Python layer is included in network, an exception occurred in caffe.exe (see this, related to LayerParamter having no to_python converter.  However, both pycaffe in Windows, and caffe/pycaffe in Linux worked without issue.  This workaround tries to run pycaffe in Windows when both of the following conditions meet.

At least one layer in the network has Type == 'Python'
platform.system() is 'Windows'
In other conditions, the usual caffe.exe will be used.

The codes were modified to follow suggestions.

Raise an exception when users requesting multiple-GPU, but the environment is Windows and python layer is included in the network.  Currently pycaffe does not have set_solver_count, and set_device accept single gpu_id.
When no gpu_id is specified, pycaffe tries not to set_device.
Re-arrange import platform.

Another code clean-up and re-arrange.  Thanks to Luke for the comments and better ideas to format the codes.

Tested on Ubuntu - doesn't cause any issues as far as I can tell.

An error occurred when I use python layer in windows caffe, the error information is as follows,
No to_python (by-value) converter found for C++ type: class caffe::LayerParameter
and I test the python layer in python console, the error again as follows
Boost.Python.ArgumentError: Python argument types in     Layer.__init__(DataLayer) did not match C++ signature:     __init__(struct _object * __ptr64, class caffe::LayerParameter)
Why did this error happen? How can I figure it?
Appreciate for your help.


Re-evaluate the Evaluate trait

I'd prefer evaluate, evaluate_checked and evaluate_with_default (with the option to set the default value for unknown variables) in the Evaluate trait. Setting unknown variables to false by default is a great way to amplify small bugs into hard to detect bugs. This would also affect evaluate_safe and evaluate_nonsafe. To a Python user, safe and nonsafe will be likely very foreign. We need to have a default evaluate method here (that can be also used to implement call) and throws an exception, and then evaluate_with_default in case they want a method that never fails. I'll prepare a PR for this.

Note to self:
evaluate: use false as default value if literal is not found
evaluate_checked: return Err if literal is not found
evaluate_with_default: pass default value if literal is not found


Username registering.

Allows players to register with their own usernames.
Chat now displays the username of the player that's chatting.

I can't comment on the specific line due to GitHub issues, but is it possible to make self.handle_message not need the message param? self.send_chat_message doesn't, for instance.


Prepare spring cloud azure 6.0.0 beta.1 release

As the title to prepare the Spring Cloud Azure 6.0.0 beta1 release.

API change check
APIView has identified API level changes in this PR and created following API reviews.
spring-cloud-azure-core

Due to the wrong target branch, use this pr https://github.com/Azure/azure-sdk-for-java/pull/31148  instead.


Finding Iris RGB Pixels

I have the Iris Landmarks Points( 5 : Centre, left, right, top, bottom). X,Y Coordinates
Can we Able to get the Entire Iris each Pixel(RGB)  Information, is there any specific Call i should Make.

Hi @Roopesh-Bharatwaj-K-R,  Could you please point us to the above code with any links which you are referring and also elaborate with complete details. Thanks!


Spanish number seen as valid in France and vice versa

Hi, I noticed that when country code was set to France some Spanish phone number could be valid in France. In instances where you absolutely need a local phone number this can prove to be a bit of an issue.

Country: FR
Example number(s) and/or range(s): 915211201 (ES number),<PHONE_NUMBER> (FR number)
Number type ("fixed-line", "mobile", "short code", etc.): "fixed-line" and "mobile" (only ones tested)
For short codes, cost and dialing restrictions: n/a
Where or whom did you get the number(s) from: they were both invented, based on the form of French and Spanish phone numbers
Link from demo (http://libphonenumber.appspot.com) showing error: http://libphonenumber.appspot.com/phonenumberparser?number=915211201&country=FR&geocodingLocale=fr-FR

@AKFourSeven,
The 'defaultCountry' means the country from where you are calling the number. When dialling 915211201 from Spain, it connects to local fixed-line number. 915211201 is a valid fixed-line number in Spain. See https://en.wikipedia.org/wiki/Telephone_numbers_in_Spain#Area_codes
Similarly for France 915211201 is a valid VOIP number. See https://en.wikipedia.org/wiki/Telephone_numbers_in_France
The numbers can overlap between the regions. It depends on from where you are dialing.
I'll close this as it's not an issue with the library. Please consider joining the discussion group if you'd like to discuss this further.

Okay but from a French standpoint this particular number "915211201" as it is, is viewed as incorrect whereas this one "0915211201" would be correct!
Is there an option to ensure the trunk prefix or force during the validation ?

@AKFourSeven,
Here '0' is the nationalPrefix and '915211201' is the subscriber number in France. While parsing we are lenient and remove country code, nationaPrefix, internationalPrefix etc. So to validate a number we only use the subscriber number. You can check 'Formatting Results' in demo to understand all the different formats.
No, there is not a way to tell the API to require the national-prefix when parsing.


Implementing a lighter Loader component for checkout

Use a lighter loader in checkout actions. The current checkout loader gives a feel like "something complicated happening in the background" which reduces total checkout experience

done


Media stops on Android background state

Video or audio automatically stops when activity goes to background. For example playing a video on YouTube or Vimeo

Load a Youtube Video on the webview and go to Home Screen. The video automatically stops.

It should be defined by a param like 'enableMediaOnBackground': true. If true then activate...

Adding the following code on RNCWebViewManager.java fix the problem

@Override
protected void onWindowVisibilityChanged(int visibility) {
if (visibility != View.GONE) super.onWindowVisibilityChanged(View.VISIBLE);
}

Screenshots/Videos:

Environment:

OS: Android
OS version: 10, LineageOS
react-native version: 0.63.2
react-native-webview version: 10.4.0

Can you tell me where I'd paste that, or what it should be changed to for the latest version? I keep getting errors when I try using it in the current version.

@flexgrip did you found nay solution?


Corrections banners

This adds banners to pages where there is a 'correction' to some factual data.
When this happens, previous versions of the page (which contained the error) get this banner:

And versions which contain the correction get this banner:

If neither of this banners appear, only then do we display this banner, which links to the latest version (if you're not already looking at it). This has moved up the page slightly to before the table of contents:

See https://trello.com/c/4GaHZRgR/1574-3-design-and-implement-banners-for-the-top-of-measure-versions-which-contain-factual-corrections-and-or-factual-errors-5

Should we maybe make the existing banner (blue bordered) more consistent with the new banners? Probably a blue strip on the left is good?
It feels weird to have three different kinds of banner altogether.

@samuelhwilliams how about this as a slightly more subtle version of the "there is a new version" available?

I feel this probably needs to be more prominent than the "This page corrects mistakes" banner?

It's more subtle, yep - although I wasn't too opposed to the big bold 'check that you're using the latest data'-esque message. It's still a third style of banner, so not sure it totally addresses the crux of my earlier review. But happy to iterate.

@frankieroberto @samuelhwilliams
Just a quickie to confirm that I'm happy with the words 👍
Personally, I'd have expected all banners to have had a border. I'm not sure the 2nd one ('This page corrects mistakes in a previous version') stands out as much as the others. Was there are reason why you opted for that design over a border?
In the same vein, I think I prefer the red border used in the design system error summary to the red background we've used here. (Mainly as I find it easier to read.)
Shall we have a chat after Friday's stand-up?

I think @spencerball's comment makes sense about using the Design System error summary thing. Maybe we can stick with that and apply it consistently for all three messages (just with different colours)?

@samuelhwilliams @spencerball Interesting comments, thanks.
My initial feeling was that there's a fair bit of difference in how prominent each of these should be:

The "This page contains a factual mistake" should be un-ignorable, given that nearly everyone looking at the page will want visit the corrected page instead
The "There is a new version of this page" should be prominent, given that most people will want the most up-to-date banner, but some may want to refer to previous year’s data.
The "This page corrects an error" should be a lot less prominent, given that most people will be viewing the page for the first time, and so the message has little consequence for them. The only people who really need to see the message are return visitors who may have used data from the previous version containing the error in their own work.

Given this, I'm not sure they should all use the same design? But we could reduce the number of variants a bit.
Here's some more options:

I think I was being a little reductive last night in trying to get them all to align - you make good justifications for having the 'factual mistake' banner stand out. I prefer the last design.
I'm still inclined to think the other two can be normal design system error summary bordered boxes.

How about using the Inset text pattern for the "This page corrects a mistake in the previous version"? I based it on that above, but with a green left border instead of grey. But I think the grey would be fine.

Sure.

@samuelhwilliams the test should at least check that the # reference in the link goes somewhere on the page? ie if we renamed the ID but didn't update the link, or vice versa, it should catch that?

@frankieroberto yep, that'd be a good test I think.


[5.0] Add whereNotNull to JoinClause

Fixes https://github.com/laravel/framework/issues/7165
I hope there is no hidden pitfall here

Thanks for sending this. Taylor should get to this within a few day's time. :)


Add e2e tests for Federated ReplicaSet cluster weighting/min/max

 Issue by quinton-hoole
Friday Sep 02, 2016 at 22:59 GMT
Originally opened as https://github.com/kubernetes/kubernetes/issues/32014

cc @jianhuiz As per https://github.com/kubernetes/kubernetes/pull/31904#issuecomment-244505671
Per design below.  Current e2e tests do not cover that functionality.
type FederatedReplicaSetPreferences struct {
    // Map from cluster name to preferences for that cluster. It is assumed that if a cluster
    // doesn’t have a matching entry then it should not have local replica. The cluster matches
    // to "*" if there is no entry with the real cluster name.
    Clusters map[string]LocalReplicaSetPreferences
}

type LocalReplicaSetPreferences struct {
     // There should be at least this amount of replicas in the clusters. 0 if not specified.
    MinReplicas int
    // There should be at most this number of replicas in the cluster. Unbounded
    // if not specified.
    MaxReplicas *int
    //  This weight should be used when spreading replicas across clusters
    Weight int
}

 Comment by dims
Monday Sep 05, 2016 at 23:23 GMT

@quinton-hoole Is this work still targeted for v1.4 milestone?

 Comment by quinton-hoole
Tuesday Sep 06, 2016 at 16:04 GMT

It's test code. It needs to be written and run before the release of v1.4.
But it doesn't necessarily need to be cherrypicked into the actual release
binary.
On Sep 5, 2016 4:24 PM, "Davanum Srinivas"<EMAIL_ADDRESS>wrote:

@quinton-hoole https://github.com/quinton-hoole Is this work still
targeted for v1.4 milestone?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/kubernetes/issues/32014#issuecomment-244820909,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJ6NAZky3P5V-VZNxxF60-PXUWVCe_KHks5qnKSpgaJpZM4J0JlA
.

 Comment by pwittrock
Thursday Sep 08, 2016 at 05:17 GMT

@quinton-hoole
I am moving this out of the blocking issues we are tracking for the 1.4 release.  If there is still 1.4 work to be done here that should be tracked in the weekly burndown meetings, move it back and provide daily updates to this issue.

cc @quinton-hoole


Support for Rails 7.1

To kick off the discussion: https://github.com/devise-two-factor/devise-two-factor/blob/main/devise-two-factor.gemspec#L25
Does anyone want to pick this up?
Cheers.

Will start testing https://github.com/devise-two-factor/devise-two-factor/pull/256

Is this going to happen?

I don't believe there is an issue here. The current version should allow for Rails v7.1 and v7.2. Closing for now unless I'm mistaken about something.


isto kube-inject breaking k8s.svc

After injecting via istioctl kube-inject, the existing application which was using kube api internally is getting unable to to connect to <IP_ADDRESS>:443 ( which is the default.svc), all inbound/outbound urls are routed via proxy.
The same application works without this error before kubernetes-inject.
Why is this behaviour ?

I am also seeing this. My application is written in go and has the following dependency in our glide.yaml:
- package: k8s.io/client-go
  version: 1.5.1
  subpackages:
  - 1.5/kubernetes
  - 1.5/pkg/api
  - 1.5/pkg/api/resource
  - 1.5/pkg/api/unversioned
  - 1.5/pkg/api/v1
  - 1.5/pkg/apis/batch/v1
  - 1.5/pkg/apis/extensions/v1beta1
  - 1.5/pkg/labels
  - 1.5/pkg/util/intstr
  - 1.5/rest

This may be a pod startup order/timing issue that should be dealt with in the app.
See related discussion here: https://groups.google.com/forum/#!topic/istio-users/euWH-cieiOc

the bigger issue here is auth/mtls which is probably why you can't talk to k8s api server if you turned auth on. Closing for  https://github.com/istio/pilot/issues/1015


RFC: numerics reform

This RFC is preparation for API stabilization for the std::num module.  The goal is to finish the simplification efforts started in @bjz's reversal of the numerics hierarcy.
Broadly, the proposal is to collapse the remaining numeric hierarchy in std::num, and to provide only limited support for generic programming (roughly, only over primitive numeric types that vary based on size).  Traits giving detailed numeric hierarchy can and should be provided separately through the Cargo ecosystem.
Thus, this RFC proposes to flatten or remove most of the traits currently provided by std::num, and generally to simplify the module as much as possible in preparation for API stabilization.
Rendered

The only code associated with this comment are the traits below:
pub trait RotateLeft<RHS> {
    type Output;

    fn rotate_left(self, rhs: RHS) -> Self::Output;
}

pub trait RotateRight<RHS> {
    type Output;

    fn rotate_right(self, rhs: RHS) -> Self::Output;
}

pub trait SwapBytes<RHS> {
    fn swap_bytes(self) -> Self;

    fn from_be(x: Self) -> Self;
    fn from_le(x: Self) -> Self;
    fn to_be(self) -> Self;
    fn to_le(self) -> Self;
}

I would like to bring attention to the particular use case of using these subsets of the current Int trait implemented for std::simd types (like i16x8, i32x4, i64x2, u16x8, u32x4, u64x2). The reason why you can't implement Int for SIMD types is that even though NumCast is easy to implement as the broadcast operation (a SIMD type composed of 8 copies of the same value), the reverse operations in ToPrimitive cannot be implemented in a satisfactory way (You could take the first element, or the last element, but neither of these are attractive options). So there is no natural implementation of ToPrimitive for SIMD types.
A second reason why these method subsets are useful to separate from the Int trait is that the current rotate_left and rotate_right methods strictly enforce RHS=usize, which is typically how they are used, but there are algorithms that require these methods where RHS=Self (meaning each SIMD component is rotated a different amount). This is already expressible in the current Shl and Shr traits, which are intimately connected to the rotation methods. Imaging Self below replaced with u32x4 in the following example specializations:
impl Shl<usize, Output=Self> for ... { ... }
impl Shl<Self, Output=Self> for ... { ... }
impl Shr<usize, Output=Self> for ... { ... }
impl Shr<Self, Output=Self> for ... { ... }
impl RotateLeft<usize, Output=Self> for ... { ... }
impl RotateLeft<Self, Output=Self> for ... { ... }
impl RotateRight<usize, Output=Self> for ... { ... }
impl RotateRight<Self, Output=Self> for ... { ... }

After some experimentation, it seems that Shl<Self, Self>, Shr<Self, Self>, RotateLeft<Self, Self>, and RotateRight<Self, Self> are already implemented for all SIMD types. It would be a great benefit for SIMD users if the other, more common shift and rotation methods were also built-in.
A side note (and perhaps outside the scope of this comment) is that RotateLeft and RotateRight could be renamed Rol and Ror as they can be implemented in single instructions on some architectures, even though they're not traditional C operations. Evidence can be found on the LLVM mailing list that others who have also considered them core operations.
In conclusion, there are at least 2 compelling reasons (the ToPrimitive reason and the RotateLeft reason) to separate out rotation operations from the Int trait so that SIMD types may benefit from sharing the same methods as primitive types.


Fix "error code" Typo (IDFGH-11162)

Fixed miss spelled words in WiFi files: "erro code" -> "error code"
Update Copyright year

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

sha=51e6d228b446a083d35b5b8e6ece1b9b7c127b10

merged to master： f67aa66067c348d483e9c49f8b544af35a46ebfe

Is there anything I can improve/fix, or why is the PR closed? :)

Thanks for your PR @nkaaf, your commit was merged in https://github.com/espressif/esp-idf/commit/2d6dcee1365992a7d48d2fd9ff75b6ed58a4a1ea (the link to the commit is right above). We sometimes have to rebase Github PRs before merging them into the master branch, and in this case Github doesn't associate the commit with the PR and doesn't mark the PR is "merged", even though effectively it is.

I understand, perfect. :)

I understand, perfect. :)


iOS Custom Schema

We tried Universal links with https and it worked on our IOS devices but using Custom Schema didn't work on all app states ex foreground, background and suspended.
Appreciate your support

we tried to test it on simulator as you mentioned but the same issue

We tried Universal links with https and it worked on our IOS devices but using Custom Schema didn't work on all app states ex foreground, background and suspended.
Appreciate your support

I'm having exactly the opposite problem, I am able to use custom links, but not uni links in iOS. I'm pretty sure I have everything set up properly, the entry in entitlements, and the domain association file on an https server, it will just open the browser instead. Any ideas?

Same problem. any solution?


Move formatting performed in beta into plugwise

In plugwise-beta we perform two formatting-actions:
        if self._unit_of_measurement == PERCENTAGE:
            measurement = int(measurement * 100)

and
        if self._unit_of_measurement == ENERGY_KILO_WATT_HOUR:
            measurement = round((measurement / 1000), 1)

These actions are moved into plugwise by this PR.
Plus some fixes in the use of constants in constants.py by the Stick-part of the code.

The changes may look a bit complex but I've made them like this with the idea in mind that, in the future, we move towards "push" instead of "poll".


[TASK] Import CSV BOM

create a livewire CSV import
the fields are as follows:
Sort Service (code) Label Component (code) Qty Unit Cost Unit Price
I need an import modal and a scipt which gives the user the lines where the service code or the product code does not exist

CSV Model
ordre,service_code,label,component_code,unit_code,qty,unit_cost,unit_price
1,COMPO,Task Example,SHT-1000,UNIT,10,5.00,10.00
2,COMPO,Task Example 2,PRD02,UNIT,15,4.50,9.00


(Documentation) After running from source on windows, meerk40t cannot connect to usb

Followed instructions here: https://github.com/meerk40t/meerk40t/wiki/Install:-Running-from-source
pyusb and the libusb k40 driver from ScorchWorks are both installed on my machine.

Hm. It really should have aborted that and tried the whisperer driver if the libusb driver failed. Maybe for long term I should give you a force switch for the config thing to let you manually choose your driver. So in the future if this comes up more, it'll be fixable without too much trouble.

This works fine for me with the precompiled executable - so I assumed there's some setup missing, somehow. Not 100% sure. For what it's worth, I've never installed the stock driver that came with the k40, only the scorchworks driver.

Yeah, when you plug it in the windows driver should load up. This is the CH341DLL driver which works fine, however, it's not loading. In that case it's supposed to try the LibUsb Driver which works with Whisperer. but somehow it's not doing that. It's supposed to switch between drivers depending on what works. The goal is to have it always work, but it looks like the windows driver failed and it failed in a way that prevented it from trying the other driver or simply didn't correctly attempt the fallback.
This needs to be checked before full release, I'll try deleting the dll and doing other things with it to see if I can replicate it.

Let me know if you need me to test out a patch

Followed instructions here: https://github.com/meerk40t/meerk40t/wiki/Install:-Running-from-source

I rewrote these when I installed to run from source on my own Windows laptop. I am not sure anyone other than you has tried them since, so there may well be a problem.
The Windows shortcut that is created does not create a console for console errors to appear on. You might want to create a small .cmd file as follows which you can run:
@Echo On
python meerk40t.py
pause

@tatarize The error message Could not find module 'CH341DLL.dll' (or one of its dependencies). Try using the full path with constructor syntax. is really not very helpful to the user - though it may be helpful to you because it seems to be finding the dll when running from an executable but not when running from source. You might want to ask @gsboylan to check that this dll is in the expected place when running from source.

The error message isn't mine, it's likely python with the call to ctypes.
https://stackoverflow.com/questions/65345213/could-not-find-module-or-one-of-its-dependencies-try-using-the-full-path-with
The real issue is that when hitting that it stops trying rather than goes on to the other driver.

Okay, I checked and that error there is apparently a FileNotFoundError. I should have added a commit to catch that.
Solid chance this commit addresses it: https://github.com/meerk40t/meerk40t/commit/7eff28a8e3e2fed07bdc6a77e34c150ad4da33e5

No good, and no logs out to console to clarify the error. Maybe something's swallowing the error?

Got it - you're catching and throwing different error types. Let me put up a PR

https://github.com/meerk40t/meerk40t/pull/460

The actual problem here is that it should throw a failed error and not work and default back to the other driver, but apparently the connection refused says the driver was fine but didn't connect. I need to check the flow since the in theory that kind of failure should result in trying the other driver rather than stopping with the connection refused. Let me take another stab at it.

https://github.com/meerk40t/meerk40t/commit/962ab67230e674754d5d407340707fd6cd73e301
This should maybe correct it. #459 is going to hit some serious troubles. The import errors for the driver running aren't correct. It needs to catch the failure to load and raise it as an importError.
The relevant section is:
        handlers = []
        try:
            from .libusb import Handler as LibUsbHandler

            handlers.append(LibUsbHandler(channel=self.channel, status=self.status))
        except ImportError:
            self.channel(_("PyUsb is not installed. Skipping."))
        try:
            from .windll import Handler as WinHandler

            handlers.append(WinHandler(channel=self.channel, status=self.status))
        except ImportError:
            self.channel(_("No Windll interfacing. Skipping."))

In ch341.py the function tries to load the windll handler and the libusbhandler and when it fails to load those correctly it's supposed to move on the next bit and fail correctly. So if I can't load the LibUsbHandler because I don't have the driver or something, then failing should fail to register the handler. Same for WinDll, either load it or fail with import. The issue here is that it failed the wrong way and didn't just say No Windll interfacing. Skipping. -- Which would be the expected result there.

The DLL CH341DLL.dll depends on KERNEL32, USER32, SETUPAPI. I'd think your windows would have a lot more problems if those weren't functional.

I can confirm that 962ab67 fixes the issue.

Okay - Actually, I was incorrect, this commit does not fix the issue. I can still repro the issue on your most recent commit 962ab67.

@gsboylan Is this still an issue?

@tiger12506 no. Platform.release() return value of 8.1 for windows 8.1 which is a floating number not integer so gives error for os version with a floating value when try to convert the string to integer

Fix for ansi on 8.1 submitted as #872.

@Sophist-UK python3 documentation says that starting from python 3.8 winmode parameter was added to ctypes.WinDLL class which has a default value of RTLD_LOCAL so i fixed it setting winmode to RTLD_GLOBAL changing the code below in windll.py
Line 1-> from ctypes import c_byte, WinDLL
Line 151-> self.driver = WinDLL("CH341DLL.dll",winmode=0)
With this changes worked for me on python 3.9.10 but maybe you need to add an if statement to check if the python version is  equals or greater then 3.8 since winmode does not exists on prior versions.

@Sophist-UK sorry winmode has default value of None so i don't know if  it is ctypes.DEFAULT_MODE (wich is RTLD_LOCAL in windows OS) but it's certainly not RTLD_GLOBAL.
take a look https://docs.python.org/3.8/library/ctypes.html#ctypes.WinDLL

@dorsi90 Guiseppe - I am looking at this now. Thank you for researching this for us - you first logged this in July 2021 so apologies for not having looked at this. I have tried your code but I cannot test it on my development machine which doesn't have a K40 attached.
My research suggests that you are on the right track here. But I am not sure what LoadLibraryEx flags Python uses by default, nor which specific flag is causing the problem and since the Python documentation says that these flags are for security I think we should do the minimum that fixes this by adding a full path or removing only the flag that is causing the issue.
Are you willing to do some further research and testing? (I will create a PR for this using the above code on the basis that we are going to tweak it if necessary once the research is complete.)

@Sophist-UK I suggest not using the full path but keeping system variables search.
If it generate error for version prior to 3.8 because winmode not exists as parameter or just to or just to keep the old code then pathlevel.h defines PY_MAJOR_VERSION and PY_MINOR_VERSION so you can do something like this
if PY_MAJOR_VERSION >= 3 and PY_MINOR_VERSION >=8 :
"""new code"""
else
"""old code"""

Ok - I found the code for WinDLL and it uses _LOAD_LIBRARY_SEARCH_DEFAULT_DIRS from module nt as the winmode.
The documentation for the flags can be found here or here (both essentially the same) and they say:
LOAD_LIBRARY_SEARCH_DEFAULT_DIRS (0x00001000)

This value is a combination of LOAD_LIBRARY_SEARCH_APPLICATION_DIR, LOAD_LIBRARY_SEARCH_SYSTEM32, and LOAD_LIBRARY_SEARCH_USER_DIRS. Directories in the standard search path are not searched. This value cannot be combined with LOAD_WITH_ALTERED_SEARCH_PATH.

I took a look to see where CH341DLL.dll is located, and it appears that in the source it is located in /.github/workflows/win and included in the Windows executable with some clever build code. So I am not surprised that it is not found when running from source because:
a. Depending on the route you get the source, /.github is sometimes not included (i.e. Pypi/PIP install?)
b. Even if you have this directory, it is not included in the path.
So I am frankly surprised it works if you do winmode=0 - have you moved or copied the file?

It works because don't load the dll included in the .github folder but instead load the dll located in system32 which i think is installed when plug the k40 the first time.
LOAD_LIBRARY_SEARCH_DEFAULT_DIRS also search in system32 but are you sure that is the default paramenter value?
I had not seen that the dlls were also included in the github folder so at this point the full path to the github folder is the best choice as the doc suggest:

The winmode parameter is used on Windows to specify how the library is loaded (since mode is ignored). It takes any value that is valid for the Win32 API LoadLibraryEx flags parameter. When omitted, the default is to use the flags that result in the most secure DLL load to avoiding issues such as DLL hijacking. Passing the full path to the DLL is the safest way to ensure the correct library and dependencies are loaded.

Later I do some tests

@Sophist-UK to be sure i removed dll from system32 and from github folder and it also works without dll in folders so i don't know which path it use for load the ch341 dll :)
I need to write a code that print the path of the loaded dll

You may be able to find the path using a ctype utility (see "Finding shared libraries]:
from ctypes.util import find_library
find_library("CH341DLL.dll")

@Sophist-UK ctypes.util.find_library function returns None so can't find the dll but WinDLL("CH341DLL.dll",winmode=0) works like a charm i don't understand why

Have you rebooted since you removed CH341DLL.dll from system32? It may still be loaded in memory and so not reloaded.

I doubt that Python caches the file - but it is normal not to remove a DLL from memory immediately you say you want it unloaded and so if it has been loaded already, it probably stays loaded.

After reboot the system is the same as you can see in picture the first line is find_library response which is None but dll is loaded.

Tested again with old code windll.LoadLibrary("CH341DLL.dll") just to be sure and the error occurs.

@Sophist-UK the dll was located in the Script folder of the virtualenv and if i delete the dll from this folder not working again.
So windll.LoadLibrary not work, WinDLL works only with winmode=0 but only if the dll is located in the Script folder and maybe in Lib folder of the virtualenv that starts the process.
I think the best solution is to add the full path as you suggested before

There are only a few locations that it will find the DLL automatically. System32 is the obvious one. The scripts folder in a venv works because the python.exe is located there. Did you try putting the DLL back in system32 if it was originally there and seeing if it loaded with or without winmode=0?

The alternative is for MK to move it from /.github/workflow/win to somewhere like /meerk40t/extras where it will be included in Pypi installs / Download source zip/tar files / git, and adjust the windows executable build script accordingly.

Yes i restored the dll in the system32 folder the problem is that winmode=0 search in system32 only if the app is located in the same partition where the OS is installed so if my OS is located in C and the meerk40t also is located in C it loads the dll as expected otherwise works only if dll is in the Script folder.
windll.LoadLibrary("CH341DLL.dll") does not work in any way.
The user friendly option is the full path but you can also use winmode=0 specifying in the documentation to place Meerk40t folder in the same partition where the OS in installed or copy CH341DLL.dll from system32 or github folder into Script folder generated by virtualenv.
Another option could be retrieve the partition where OS is installed and adding it to the searching path but i don't know how to do that.

You will need to look at the documentation already linked to to see what flags to use. There is one for searching system32. But you first need to work out which of the flags is needed in addition the lack of which was not allowing it to be loaded at the start.

Yes it say that LOAD_LIBRARY_SEARCH_DEFAULT_DIRS flag that is available in LoadLibraryEx can search in APPLICATION_DIR, USER_DIR and SYSTEM32 and this flag can also be used for WinDLL class in winmode parameter wich is perfect but doesn't work for me.

The winmode parameter is used on Windows to specify how the library is loaded (since mode is ignored). It takes any value that is valid for the Win32 API LoadLibraryEx flags parameter. When omitted, the default is to use the flags that result in the most secure DLL load to avoiding issues such as DLL hijacking. Passing the full path to the DLL is the safest way to ensure the correct library and dependencies are loaded.

The only parameter that for me search correctly in system32 is winmode=0 (only if application is placed anywhere in the partition where OS is installed).
Also note that winmode=0 is the default value for python 3.8 then is set to none starting from 3.9 version.
Which option do you prefer to use?
I can do some tests and if you want you can send me some code to test

import nt
self.driver = WinDLL("CH341DLL.dll", winmode=nt.LOAD_LIBRARY_SEARCH_DEFAULT_DIRS | nt.LOAD_IGNORE_CODE_AUTHZ_LEVEL)

self.driver = WinDLL("CH341DLL.dll", winmode = 0x00001000 | 0x00000010)

self.driver = WinDLL("CH341DLL.dll", winmode = 0)

All tests are made with CH341DLL.dll only present in system32 and github folders.

Ok. I think we might now have enough information to get the driver to load providing that it is in a fixed number of places. I think that there might need to be a broader discussion about how to provide both CH341 support and LibUSB support on all platforms in both 32 and 64 bit o/s and 32/64 bit python, and when run from executables and from various types of source so we can provide a single unified experience for users.

From executable i haven't encountered any problems maybe the dll is embedded in the EXE file.
However during my tests there have been several crashes not related to this that can be easily fixed. I can open new issues?

The DLL is totally built into the exe file. But, also it's using 32 bit python internally to make sure that stuff works, so it basically freezes a venv in the file.

Yes. But the DLL is not available if you are running from downloaded source or pypi (can't recall which) because it is located in a directory which isn't included. And we need to provide 64 bit dll too so we can create a 64 bit version. And we should handle libusb similarly.

Yeah, the dll isn't really being distributed, just included at points. I'm not really sure about the 64 bit version stuff or whether that stuff is able to work. @tiger12506 might be pretty reasonable to figure such things out. Mac and Linux is still pretty consistent, and we might want to do some work on figuring out a pure 64 bit version. But basically you need the same bit driver as the python version, and it could be the case that the calls in the code itself are for the 32 bit bit python dll for the windll version. If that's the case then the dll calls would be different for 64 bit and 32 bit and so any 64 bit python would have an issue using the windows driver which could explain the issue here.

I think it checks libusb first, so in theory if you installed that it would work even if the windows driver thing is weird. And yeah the exec files has them built in since we didn't want to deal with intricate problems from folks who run the exe files.

AFAIK this has not been completed.
The suggestion was to move the CH341A.dll to somewhere in the source tree so it is included in run from source and not only included in exes. Then the DLL can be loaded if run from source not just if run from exe.

I am no longer developing so I am no longer running from source - and since I only ran from source on my normal laptop and not my K40 controller, I never had the need personally.
My comment was for the benefit of anyone else who might want to run MK from source on Windows - like the OP did.


[QA] A grey overlay appears on the backpack

Explorer Alpha build versions:
0.25.21
Issue Description:
When tapping multiple times the letter M and I, you are able to see a grey overlay when opening the backpack again.
STR:
1.Open the build 0.25.21
2. Validate that you will appear on the Genesis Plaza
3. Press M and I multiple times
4. Verify that a grey overlay appears on the backpack
Expected behaviour:
No grey overlay should appear when tapping those keys at the same time for multiple times
Current behaviour:
A grey overlay appears on the backpack when pressing the keys ''M'' and ''I'' multiple times
Evidence:
https://github.com/user-attachments/assets/6f37b834-a543-4265-aa6c-48adf198dd8b
Additional Notes:
Windows 11pro
RAM 16 GB

Unable to repro on latest versions (current v0.46.0)


Can't download all the followers of a profile

Hello, i'm trying to download all the follower's profile names (millions) from a specific profile. Here's the code:
import instaloader
from time import sleep
from datetime import datetime

now = datetime.now()
print("Starting at {}".format(now))

L = instaloader.Instaloader()
L.login(MY_PROFILE, MY_PASSWORD)

profile = instaloader.Profile.from_username(L.context, "prada")
follow_list = []
count=0
file = open("prada_followers.txt","w")

for followee in profile.get_followers():

    follow_list.append(followee.username)
    file.write(follow_list[count])
    file.write("\n")
    if count != 0 and count%1000==0:
        print("{} followers, sleep 60 seconds...".format(count))
        sleep(60)
        now = datetime.now()
        print("Restarting at {}".format(now))

    count=count+1

file.close()

After about 10k followers i get this:
HTTP Error 400 (Bad Request) on GraphQL Query. Retrying with shorter page length.

Did you have a similar problem? Is there a solution?

HTTP error 400 sounds like your account has been blocked by instagram. please check that.

HTTP error 400 sounds like your account has been blocked by instagram. please check that.

I checked from my app and my account is still working


Enable setting default Downloader

This PR will allow setting the default Downloader.
Motivated by the discussion at JuliaLang/Pkg.jl#3211
TODO

[x] add tests
[x] update documentation

Request feedback on:

function naming (currently default_downloader!
Should this be part of the exported API?

Not sure what the preferred workflow is for this repo.  This PR is simple enough that I would suggest squashing it down to a single commit . I can do that prior to review or later just before a possible signoff

This seems reasonable to me from both the naming and functionally perspectives. Normally I would say "just set the global variable" but since there's a lock, we really shouldn't encourage that.

Hi @StefanKarpinski --

reverted spurious formatting changes
rebased
squashed to single commit

lmk if there is anything else.  thanks

pinging to see if this can be merged -thanks!


feat(Data/Set/Function): add MapsTo.map_nonempty

Extracted from #8160.

I'm not 100% sure about the name; suggestions welcome.

Thank you for the review.
bors r+


How to fix Illegal mix of collations for operation 'like'

I'm getting Illegal mix of collations for operation 'like' in CI 3.1.11 & mysql 5.6

Hi ,
I think your problem is MySQL related. Can you check the link below
https://stackoverflow.com/a/3029483/6333300


RFC: Show more replies in Following (different heuristic)

Alternative to the heuristic https://github.com/bluesky-social/social-app/pull/4871 based on community feedback.
This fixes the double-reply loophole and actually opens up the default Following heuristic to be a bit more broad than the current default one on the website. We'd include not only replies between people you follow (which is the current default), but also replies where you follow only the replier and the thread OP, or only the replier and the grandparent. In other words, we'd show the reply if there's at least one other person you'd recognize looking at the context above that reply.
In practice this means you'd see banter where you don't follow some people as long as there's one more reason to display them (e.g. you follow the OP, the parent, or the grandparent).
For example, I don't follow Jeremy but this still surfaces Stephanie's reply to Jeremy (because I follow Eustace):

Here, I don't follow Anna, but the thread has Amy and Rahaeli (both of whom I follow) so I see it too:

I like this. Seems like a good compromise. A one-off reply won't be shown, but if there's a whole subthread it probably will be.


Make pv names consistent with web gui names

A copy of https://github.com/PandABlocks/PandABlocks-ioc/pull/18, but branched off of the improved testing branch.

On a related note, have you seen the discussions happening in Ophyd at the moment regarding similar changing of block names? See this comment.

Yep; just to chime in, I do think it makes more sense for the PandA to have either names and numbers on their blocks or just names, e.g.
PCAP
PULSE1
PULSE2
SEQ1
SEQ2
but not, for example,
sfp2_sync_in
sfp2_sync_in1
as seems to be happening here

Yep; just to chime in, I do think it makes more sense for the PandA to have either names and numbers on their blocks or just names, e.g.
PCAP PULSE1 PULSE2 SEQ1 SEQ2
but not, for example,
sfp2_sync_in sfp2_sync_in1
as seems to be happening here
In fact if you have a look at that whole thread it looks like there's a mismatch between the panda blocks on malcolm and what we see via PVI. And I wonder if we can explain why 🤔

I'll chat to  @coretl about what we want our behaviour to be. In this PR I've caused the softioc to throw an error if the user tries to have a number in the block name (rather than passing number=).

To update this ticket, the outcome was that we should never have both sfp2_sync_in and sfp2_sync_in1, that is a bug. All entries should be under sfp2_sync_in. @evalott100 is working on a PR to fix this.


First time deno task start fails with Deno 2.0: error: Import assertions are deprecated. Use with keyword, instead of 'assert' keyword.

I followed the Saaskit README and cloned the repository, set up .env as described, and when I ran deno task start I got:
➜ deno task start
Task start deno run --unstable-kv -A --watch=static/,routes/ --env dev.ts
Watcher Process started.
error: Import assertions are deprecated. Use `with` keyword, instead of 'assert' keyword.

import emojis from "./all.json" assert { type: "json" };

  at https://deno.land/x/emoji@0.3.0/emoji.ts:2:1

Watcher Process failed. Restarting on file change...

Not long ago, in 2.0 RC, the above was a warning (I checked the code) in Deno, but now in 2.0 it's an error (for good reason, the syntax is deprecated and unsupported).

I tried deno task update and the results after this are the same.
I also tried a clean Fresh project with deno run -A -r https://fresh.deno.dev and deno task start ran without issue, so the error must not be from Fresh.
Also the x/emoji module is a peer dependency and I don't know how to check which module is using it. Version 0.3.1 fixes the use of the assert keyword for import assertions, replacing it with with but I don't know how I could fix this or which package to upgrade or open a PR with...
To Reproduce

Go to the Saaskit README.md
Follow the steps to create a new Saaskit project up until and including starting the project
See error

Expected behavior
The app should have started locally for the first time when following the README instructions correctly.
Screenshots

Desktop (please complete the following information):

OS: macOS
Browser Chrome
Version 129.0.6668.90 (Official Build) (arm64)

Smartphone (please complete the following information):

Device: N/A
OS: N/A
Browser N/A
Version N/A

Additional context
N/A

I found out how to view dependencies with deno info. Sorry, new to Deno. Trying it out for the first time.
file:///Users/REDACTED/Code/retinfo/dev.ts (253B)
└─┬ file:///Users/REDACTED/Code/retinfo/fresh.config.ts (769B)
  └─┬ file:///Users/REDACTED/Code/retinfo/plugins/blog/mod.ts (667B)
    ├─┬ file:///Users/REDACTED/Code/retinfo/plugins/blog/routes/blog/[slug].tsx (1.35KB)
    │ ├──<EMAIL_ADDRESS>*
    │ ├──<EMAIL_ADDRESS>*
    │ ├─┬<EMAIL_ADDRESS>(6.94KB)
    │ │ ├─┬<EMAIL_ADDRESS>(787B)
    │ │ │ ├─┬<EMAIL_ADDRESS>(28B)       <-------
    │ │ │ │ └─┬<EMAIL_ADDRESS>(4.34KB)
    │ │ │ │   ├──<EMAIL_ADDRESS>(1.1KB)
    │ │ │ │   ├──<EMAIL_ADDRESS>(548.4KB)
    │ │ │ │   └──<EMAIL_ADDRESS>(3.82KB)

I stripped out the blog module and the project now starts successfully!
➜ deno task start
Task start deno run --unstable-kv -A --watch=static/,routes/ --env dev.ts
Watcher Process started.
The manifest has been generated for 22 routes and 3 islands.

 🍋 Fresh ready
    Local: http://localhost:8000/

Might the blog module need updating?
gfm 0.6.0 is the latest version.

gfm is a GitHub flavored Markdown renderer, supporting GitHub style emojis, which is why it depends on the emoji lib that is causing the error on this issue.

this is fixed when https://github.com/denoland/saaskit/pull/684 lands

#688 merged, so this is good to close now.

Resolved by #688

This doesn't seem to be fixed https://github.com/denoland/saaskit/issues/689

So it isn't solved :) .... the #688 was closed in favour #684 and that is still open.
I've merge request to fix only that https://github.com/denoland/saaskit/pull/692/commits/6c6399bd1b6a8fec351725be1700013d37df17c3 but it doesn't pass tests as the CI is broken what I solved by https://github.com/denoland/saaskit/pull/694


When you try to import scenes to an existing project, they don't show up unless you refresh

Seems like probably the same root cause as https://github.com/azavea/raster-foundry/issues/2082
Uploading:

Without refresh:

After refreshing:

Like I said in the other issue, we don't create the scenes immediately. We need to create a websocked notification service of some sort, and listen for events along the lines of "project.scene.created" in the relevant controllers.

I think that we may want to (once more users are involved) think about how to present placeholder scenes or at least a notification within the project edit area (like the AOI messages) that an upload is being processed and will be inserted into that project.
I think the most important thing isn't that the scenes show up right away but that we acknowledge that their action actually did something.


[FEATURE] add dedicated set_runtime_parameters to Task, Step and LLM

Is your feature request related to a problem? Please describe.
Currently, it can be difficult to grasp and combine all of the runtime parameters for each component.
Describe the solution you'd like
something like
class Magpie:
    def set_runtime_parameters(self, arg_a = 1, arg_b = 2):
        super.set_runtime_parameters({arg_a: arg_a, arg_b: arg_b})

Describe alternatives you've considered
go into the docs.
Additional context
NA.

For the first version of distilabel 1.0.0, @burtenshaw suggested something like:
with Pipeline() as pipeline:
     task = MyTask(...)
     my_other_step = OtherStep(...)

if __name__ == "__main__":
    pipeline.run(parameters=[
        task(param1="...", param2="..."),
        my_other_step(...)]
    )

The idea is to implement the __call__ method for each Step and add there all the runtime parameters as arguments with precise type hints. This way, the IDE will assist you and tell you the name of the runtime parameter and the type. __call__ should return a Tuple containing the name of the step and the runtime parameters.

@gabrielmbmb I think either way would work very nicely and really helps w.r.t. development progress. Given that this is a repetitive task we might outline this for each step/task/llm and let people contribute for hacktoberfest as proposed by Omar in the advocacy channel. WDYT @gabrielmbmb.

@gabrielmbmb @plaguss wdyt about creating some issues for community members to pick up?


Automate textbook publication

When the textbook repository is put online, we would like to be able to link to an equation in other documents/pages with pandoc. Also create a "textbook" repo that has only that textbook. Make Pandoc run on Travis CI.

Kamino closed and cloned this issue to AguaClara/Textbook


[Improve] KafkaExtractNode support more StartupMode

Description
Written variously: I think KafkaExtractNode should support more KafkaScanStartupMode. Currently, only two types of KafkaExtractNode are supported, including EARLIEST_OFFSET and LATEST_OFFSET, But in many cases topicPartitionOffset is specified, and this parameter is already reserved in KafkaSource,I'd like to contribute this code if I can
InLong Component
InLong Sort
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

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

Yeah sure, thanks for participating, note that now SPECIFIC_OFFSETS is currently support by KafkaDynamicTableFactory. Please be aware of that the specificStartupOffsets should also be passed to the factory when doing so

I want to do this  @dockerzhang

Yeah sure, thanks for participation, note that now SPECIFIC_OFFSETS is currently support by KafkaDynamicTableFactory. Please be aware of that the specificStartupOffsets should also be passed to the factory when doing so

Ok, I'll take care of that


zero (0) value not appear when use format in DevExtreme grid

Bug Report

Package versions:
devexteme version: 18.2.7
devextreme-angular version: 18.2.7

Steps to reproduce:

add format= '#,###.##' in grid column like this:
   <dxi-column                     dataField="yourDataField"                      dataType="number"                      format="#,###.##">     <dxo-format type="fixedPoint" [precision]="2"></dxo-format>     </dxi-column>
and then type number that include '0' or paste number that include '0'.
Current behavior:

The zero value not appear.
Expected behavior:

The zero value must appear.
Screenshots:

This is expected behavior. It is necessary to add a leading zero to the format to show 0 in a column. So, you can set the format in this way to resolve the issue:
format="#,##0.##"

OK thanks @LexDX


docker run fetches npm dependencies each time

Describe the bug
whenever a want to generate html template based on provided docker command, it goes to fetch all node dependencies again.
How to Reproduce
as provided in docs
Expected behavior
the docker file should be organized in a way that it can use the installed npm modules in container volume

Thanks for the issue @mbande. This is actually expected behaviour because each template defines which dependencies are required and are installed when you run the container, however as soon as the container shutsdown it forgets about it. If you want to preserve the dependencies you could make a volume bind to (I am assuming it is where it places the dependencies) /usr/local/lib/node_modules inside the container.

@mbande hey, do you have more questions? does it work for you better after mounting volume with deps?

I would argue a container image should not need to pull a slew of software dependencies from the network in order to function.
Our use case only includes @asyncapi/html-template so we have created a custom image that has a Dockerfile line of:
RUN ag /app/sample_contract.yaml @asyncapi/html-template
I would happily paste the rest of the Dockerfile but I don't own the IP rights and am not authorized to share it (closed source project).
I am guessing others would prefer this approach also. I don't know the full breadth, but some kind of general pre-cache for the Docker image seems appropriate to me.

Release 1.2 enables support for latest node and npm and fixes all known installation issues. In case you see some other issues, please create a new issue. Please use --debug flag when you experience issues with the generator and in the issues mention the troubleshooting logs. Thanks!


Updates ExternalURL info

Updates ExternalURL info to make its usage more explicit
Closes: #???
Description
The description for ExternalURL was unclear. This PR hopes to make its usage more explicit by expanding on the example.

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the standards mentioned here.
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

[ ] Update the version in package.json when there is a change in the smart contracts

Addressed this in #163


Fix incorrectly added "await" when there is an assignment to a variable

The following line:
promise = $('.a').click();

was incorrectly transformed into:
await promise = page.locator('.a').click();

This PR makes sure await is no longer added in that case:
promise = page.locator('.a').click();

@fbasso Thank you for your review!


Convert Mian and Postgress deploy from Dialog to NotebookWizard

Description:
This PR changes the MIAA and Postgres instance deploy experience from the dialog tab to the notebook wizard.
Here are screenshots of the new wizards:
SQL Miaa

Postgres Instance

This PR fixes #12577
Testing: End to end testing of MIAA and Postgres instance deployment scenarios

Pull Request Test Coverage Report for Build 270268587

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

Totals

Change from base Build 269776218:
0.0%

Covered Lines:
19144

Relevant Lines:
45868

💛  - Coveralls

I assume you'll be following up in a separate PR to add the "Script + Run" button to these wizards?

I assume you'll be following up in a separate PR to add the "Script + Run" button to these wizards?

Yes.


Improve type checking in parser

A set of small changes that should improve the state of type checking in the manifest parser, peg parser and supporting files.
Note: Please review this one commit at a time, I would split them but they're all tiny.

Previous test failure was due to flakey test failures (present on main). Filed b/178327127

Previous test failure was due to flakey test failures (present on main). Filed b/178327127


[Done] DRAFT: Change order of items to newest > oldest #205

Fixed #205

@mrjamieeast please can you review?

Works for me - thanks


fix gridRow defaultValue logic

fromPairs does not work.
https://github.com/GriddleGriddle/Griddle/commit/b65bfe8460848f6e3a1a7942c4b1c3ae3c9f4255#diff-486930079516b14efdf7e2572b02c647R83

I want to use griddle 0.5.0 but this regression prevent from it.
Could you check it ? and I hope this patch is released.

Awesome -- thanks so much for adding this. I'll try to test this out tonight

@ryanlanciaux Thanks! It's not a hurry, I use my fork now.
When the latest is released, I use :)

rebased, but I'm not sure why build/Griddle.js diff is so huge :(


Simple stuff, setting up python and installing dep

Please review my bash code

Waiting for pull request #6

Waiting on #7 . Fixes the problems.

Why we don't need pyc nor 1000000 lines

whoops i should've put that in the gitignore my bad

sure


Date field not carrying over from employees submission to supervisors copy =>  3. Date/duration of agreement.

Background:
We have received inquiries again regarding the Date field appearing empty when the supervisor receives the employees copy.
The date is visible in the employees version but not when it reaches the supervisor/manager.
Steps to Reproduce:
Tested as a supervisor receiving both a 2 day and 3 day submission for approval.
Actual Behavior:
Both emails request came in a timely matter and were approved with the date field data appearing.
Expected Behaviour:
Appears to be hit and miss.  We have two emails in the inbox. One at 10:25 am from Thursday, and another at 4:18 pm, both are missing the Date field that the employee  filled out. Date field is filled out correctly and sent (since it is a mandatory field).  The issue is the date does not always carry over to the supervisors copy to approve (null field).
Environment: Prod
[Uploading... JenniferDouglasDateMissing.msg]
[Uploading... DanieleWilliamsonDateMissing.msg]

Still under investigation - we are getting reports of some user cases where this is happening.
Date implementation is still there in service side pdf generation.

The issue was that the minDate validation rule was triggered during the pdf generation and didn't let the date value passes to those fields. It only happened when a submission was signed off one or more days after the original employee submission.
It was fixed by in the pdf generation server by removing the minDate validation rule.

@HasanSifat Deployed to DEV, ready to be tested.

Simen to push to Prod

@HasanSifat This has now been deployed to production

tested successfully in PROD! Ready to be closed


Please pull fix

Hello,
Quite a few of us are waiting for this fix to be pulled:
https://github.com/zfcampus/zf-oauth2/pull/120#issuecomment-202807065
Thanks!

Bump

Looks like it's merged, but there's no new release yet? @ezimuel, can we get a working build please!

This still has not been released. I cannot see any blockers affecting this?
Ping @weierophinney @ezimuel

Fixed in https://github.com/zfcampus/zf-oauth2/releases/tag/1.3.2
(@chateaux please close this issue)


Use execution versioning for Move tools

Description
Removing explicit usages of move dependencies in favor of sui-execution.
That way we can version code and tests and use whichever version of Move we care about.
This specific PR is about sui-move-build.
I was piling PRs in #15030 and then realized those can go on their own and they will be even more clear. Moreover those are independent of creating a current (deployment) and latest (working) branches.
So there...
I need help to check that what I am doing is the right way to approach things.
I have added a version field to BuildConfig which defaults to None indicating the current production version.
If defined the given version will be used for everything versioned via sui-execution.
It would be great to hear if what is here makes sense...
@amnn @tnowacki @tzakian @stefan-mysten
Test Plan
Curent tests

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

talk to @amnn and @tzakian as well and let's have a conversation at the next Move meeting about this.
Assuming last is the production (mainnet/testnet) version and latest the working version, we are considering the 2 options:
1- version all things that matter (definitely what is there today for execution + framework + compiler + ???) between last and latest and go through the effort/pain (not clear yet what the effort is but not minor at all) to be able to run both on CI, CLI, locally, etc.
2- leave things as they are, possibly clean up a bit more and make it easy to run an experiment as if the experiment was latest (current meaning of latest)
not clear what is better all things considered.
Let's think about it and talk next week


[BEAM-12504] Introduce WaitConnectTransaction in SpannerIO

The current SpannerIO creates transaction at the beginning of pipeline run. This causes SpannerIO error by session not found if the pipeline runs long with ReadAll which gets ReadOperation as input .
This change introduces a delayed transaction creation. It makes a pipeline able to create Spanner transaction when it is needed (e.g. just before ReadAll).
SpannerConfig config = ...

PCollection<ReadOperation> readOperations = ...

PCollectionView<Transaction> transaction = readOperations.apply(SpannerIO.waitCreateTransaction());

PCollection<Struct> users = readOperations
  .apply(SpannerIO.readAll().withSpannerConfig(config).withTransaction(transaction))

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

@pabloem Can you take a look at this PR?

sorry about the delay. Looking...

@baeminbo - Pablo will be out of office until August. I do not know the level of urgency on this PR. If it can wait, that is fine. Otherwise, do you who else might review this?
/cc @chamikaramj

Hi, this is not urgent. Thanks!

I'm back from vacation! I'll take a look now

@baeminbo doesn't it make sense to use WaitCreateTransaction in all cases? So maybe just update CreateTransaction to always wait?

Hi @pabloem, I changed the code to CreateTransaction can accept PCollection so that it will wait until the input PCollection is closed. Accordingly, I updated the PR title and summary.

Run Java PostCommit

LGTM thanks


fix drilldown library error

the version in the link was ommited, causing problems when making a drilldown.

Hi @fevrec ,
Thank you for your contribution!
The change looks good.

Your welcome, thanks for maintaining this project :-)


Fix infinite loop error when defining protobuf index with large value

(66 for example).
Define protobuf metadata file when field index is big enough, we started
to have infinite loop with field index 66:
syntax = "proto2";

message Example
{
    required string id = 1;
    optional string id1 = 2;
    optional string cur = 66;
}

When calling serializeToString, php zend api zend_hash_index_find will
be stuck in infinited loop forever with our values, this loop
introduced in zend internal function zend_hash_index_find_bucket:

static zend_always_inline Bucket *zend_hash_index_find_bucket(const
HashTable *ht, zend_ulong h)
{
	uint32_t nIndex;
	uint32_t idx;
	Bucket *p, *arData;

	arData = ht->arData;
	nIndex = h | ht->nTableMask;
	idx = HT_HASH_EX(arData, nIndex);
	while (idx != HT_INVALID_IDX) {
		ZEND_ASSERT(idx < HT_IDX_TO_HASH(ht->nTableSize));
		p = HT_HASH_TO_BUCKET_EX(arData, idx);
		if (p->h == h && !p->key) {
			return p;
		}
		idx = Z_NEXT(p->val);
	}
	return NULL;
}

Something happens within our hashtables values which causes idx
bigger than nHashTableSize, so if we build php in debug mode,
ZEND_ASSERT will be fail, otherwise, we will stuck in infinite loop
forever.

With same protobuf metadata file, if we use protoc-gen-php to gen php
file and then use it like this:

<?php

require_once("BidResponse.php");

$brp = new BidResponse();
// call setCur() first
$brp->setCur("aaa");
$brp->setid(1);
$brp->setid1(2);

$data = $brp->serializeToString();

var_dump($data);

we will have problem descibed in 1. If we use setCur() function first,
infinite loop will be happend inside PHP_METHOD set() function, which
is also introduced by zend_hash_index_find descibed in 1.
This patch rewrite pb_get_value to not to use zend_hash_index_find, but
looping through our values until we find needed index. In our test it
fixed both problem described above.
Looping through values can be slow, but in my opinion,
zend_hash_index_find_bucket also use loop, this solution removes a
loop and use another loop, so it's ok for me.

@whatvn Thank you for your report and fix. But complexity of your implementation is O(n^2). There might be a better solution for the issue.

I know this implementation may slow down serialization and set "just a little bit", because almost protobuf metadata file won't have so much field to make for loop run forever.
Actually in my test implementation, in serializationToString I dont have to call pb_get_value in order to get field value, but set method has to.

Hi guys.
Just set cur's index to (index_of_id1 + 8 * n) where n is any integer number...

Check this solution.
--- a/protobuf.c
+++ b/protobuf.c
@@ -798,7 +798,8 @@ static int pb_assign_value(zval *this, zval *dst, zval *src, zend_ulong field_nu
                goto fail2;
        }

-       *dst = tmp;
+    //*dst = tmp;
+    ZVAL_COPY_VALUE(dst, &tmp);

        zval_ptr_dtor(&field_descriptors);
        return 0;

*dst = tmp copies zval::u2 union that contains next member used in the zend_hash_index_find_bucket():
idx = Z_NEXT(p->val);

More info here: http://jpauli.github.io/2016/04/08/hashtables.html#collisions-resolution

Awesome! I haven't checked but I think this will gonna work. Thank for opening my mind 👍

@serggp what do you think?

#syntax = "proto2";
message BasePush {
}
enum EmDeviceDataType {
EDDT_manufatureSvr = 0;
EDDT_wxWristBand = 1;
EDDT_wxDeviceHtmlChatView = 10001;
}
message RecvDataPush {
required BasePush BasePush = 1;
required bytes Data = 2;
optional EmDeviceDataType Type = 3;
}
//php
$recvDataPush = new RecvDataPush();
$recvDataPush->setBasePush(new BasePush());
$recvDataPush->setData('33078001a224');
$recvDataPush->setType(EmDeviceDataType::EDDT_manufatureSvr);
try {
echo bin2hex($recvDataPush->serializeToString());  //The null value is returned here
} catch (Exception $e) {
echo $e->getMessage();
}


Module link is broken (missing)

First link "Module: addsadministration"
Does not work, it does not exist.

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

ID: 611e18f8-fcbb-a4ea-4601-8f3f57530a35
Version Independent ID: 6184cd5c-5b5d-3ee2-df27-a9fc0f6440c8
Content: Get-ADGroupMember (addsadministration)
Content Source: docset/windows/addsadministration/Get-ADGroupMember.md
Product: w10
Technology: powershell-windows
GitHub Login: @andreabarr
Microsoft Alias: v-anbarr

The issue has been addressed with PR #1618.


Release 5.0.2

Hello @hammzj !
Our team is working on other tasks with higher priority right now, but we plan to release it in 3-4 business days.
Keep updated!

@AmsterGet No problem, just checked recently and saw it was a pull request -- thanks for the update!


Table report - Turning a URL into a report action creates invalid button

When turning a table column into a report action, and that column contains URL strings, the generated buttons show Object instead of the actual URL:

Likely this is cause by the new Needle styled links, which are a React object rather than a simple text string or link.

This is fixed, closing this issue.


add mesh lighting controls

This PR adds mesh lighting controls

Example of setting the light direction to follow the camera:
https://user-images.githubusercontent.com/1120672/154949532-d7af239d-8c01-4038-a2ca-ac380e6267b4.mov

Awesome man, this looks really great! Trying to finish up the API stuff now so we can get both in :)

@alisterburt , this is ready for review after #31 is merged

I fixed the transform issue and exposed the widget as a plugin. We are now up to 5 plugins!


[mongoose] add support for mbedtls and update the way it selects wich tls engine with defines

I don't have time to improve this further, nor to check how to properly do it, so i'm leaving this here as-is to notify people of the changes in that library
mongoose now define tls engine this way:
#define MG_TLS_NONE 0     // No TLS support
#define MG_TLS_MBED 1     // mbedTLS
#define MG_TLS_OPENSSL 2  // OpenSSL
#define MG_TLS_BUILTIN 3  // Built-in
#define MG_TLS_CUSTOM 4   // Custom implementation

#ifndef MG_TLS
#define MG_TLS MG_TLS_NONE
#endif

https://github.com/cesanta/mongoose/blob/a619de05087735fc71e173eb27da3e3535039360/src/tls.h#L3-L11
This pr comply with this change and add support for mbedtls

https://learn.microsoft.com/en-us/vcpkg/contributing/maintainer-guide#do-not-use-features-to-implement-alternatives :/


Fix: Update Kolnovel & Agitoon

Updates for Kolnovel chapter scraping and Agitoon popularNovels screen

How do I get the fixed version of kolnovel

How do I get the fixed version of kolnovel

You could download a dev build from here.


Make Deno 2.0 compatible

Issue
This package is not compatible with Deno 2.0, I made a couple changes, which make it work again.
Details

I changed the window global variable to the new globalThis.
added some import flags to Makefile
added needed override statements to test and others.

CheckList

[x] Has been tested (where required) before merge to main. (ran tests locally)

Considering to remove Opine tests, since its deps do not work with Deno 2.0

I've thrown out several tests, mainly because Opine, which seems to play a crucial role in testing is deprecated/ not executable in Deno 2.0 because of deprecated import assertions.  Maybe I'll open a pull request, which addresses this issue in Opine' s repo..


Minimum python version is probably 3.9

I just tried with python3.8 on Ubuntu 20.04, and it fails with the following error:
AttributeError: module 'argparse' has no attribute 'BooleanOptionalAction'
Which is fixed by just using a more recent python interpreter (3.9 in my case).

Oh absolutely! I totally forgot to adjust the README for that. That change came with the latest merge to main.. A README update is in the works where I’ll update the minimum requirement for the Python version!
thanks for the reminder :D!

Fixed in #49


deal with matrix parameters in model selection

@karlnapf
take a look at this

Let me know what travis says and then I'll merge

Changes Unknown when pulling a1c329ccc2fd84ea9f905d5292a929ba81a78c50 on yorkerlin:gradmodelselection into ** on shogun-toolbox:develop**.

@karlnapf
the error is not related to this PR

@karlnapf
Please merge this PR

Cheers!


Fix flaky snapshot test by disabling rent

Problem
I've seen the snapshot test fail due to eager rent collection adjusting account balances that are later asserted in the test. This only happens about 1/4000 runs because it requires a randomly generated keypair to have a pubkey in the slot (1-102)/432,000 partition for eager rent collection
Summary of Changes
Use rent free config in test
Fixes #

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

Project coverage is 81.8%. Comparing base (03b5965) to head (e7fac42).

Additional details and impacted files
@@            Coverage Diff            @@
##           master     #856     +/-   ##
=========================================
- Coverage    81.8%    81.8%   -0.1%
=========================================
  Files         851      851
  Lines      231744   231745      +1
=========================================
- Hits       189692   189654     -38
- Misses      42052    42091     +39

Could we instead fix this by making amounts above the rent-exempt minimum?
I think we had an issue on the old repo (although I'm struggling to find it atm) about turning rent on by default in all tests because of concerns about coverage of real-cluster behavior. I won't fight this too hard, because fixing flake is good. But a new Rent::free seems like a step backward.


Publish

Docs Build status updates of commit a7d6364:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/create-sso-office-add-ins-nodejs.md
:white_check_mark:Succeeded

docs/includes/mac-cache-folders.md
:white_check_mark:Succeeded

docs/reference/objectmodel/preview-requirement-set/office.context.mailbox.diagnostics.md
:white_check_mark:Succeeded

docs/reference/objectmodel/preview-requirement-set/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/preview-requirement-set/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/preview-requirement-set/office.context.mailbox.userProfile.md
:white_check_mark:Succeeded

docs/reference/objectmodel/preview-requirement-set/office.context.md
:white_check_mark:Succeeded

docs/reference/objectmodel/preview-requirement-set/office.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.1/outlook-requirement-set-1.1.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.2/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.3/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.4/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.5/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.5/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.6/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.6/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.7/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.7/office.context.mailbox.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.8/office.context.mailbox.item.md
:white_check_mark:Succeeded

docs/reference/objectmodel/requirement-set-1.8/office.context.mailbox.md
:white_check_mark:Succeeded

docs/testing/debug-office-add-ins-on-ipad-and-mac.md
:white_check_mark:Succeeded

docs/testing/sideload-an-office-add-in-on-ipad-and-mac.md
:white_check_mark:Succeeded

docs/testing/testing-and-troubleshooting.md
:white_check_mark:Succeeded

docs/testing/troubleshoot-manifest.md
:white_check_mark:Succeeded

docs/tutorials/excel-tutorial.md
:white_check_mark:Succeeded

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


Please emphasize that it's cannot be installed with python >= 3.7

I think it needs to be emphasized more

@mlv-dev
You can update requirements.txt (replace all == with >=).


SharePoint and OneDrive Functional Testing

💡 Summary
As part of Halibut release, to ensure that the tool's update is accurate, detailed functional testing has been planned. Each policy will be functionally tested to check that all pass/fail scenarios have been correctly processed by the tool using existing automated functional test plans.
This issue functionally tests each policy of SharePoint/OneDrive product of ScubaGear tool. The main objective is to test SharePoint and OneDrive policies and ensure that ScubaGear tool is providing the correct result for each possible configuration option.
Motivation and context
This functional testing ensures that ScubaGear results of pass/fail are accurate based on the latest version of the code.
Implementation notes
For each SharePoint and/or OneDrive policy do the following:

[ ] Configure the automated testing and run against each tenant type for which a variant testplan is present.  Run the automated test plans (e.g. regular, g5, etc.) and note the results.
[ ] For each test failure, investigate and note the cause of the failure and log any new issues needed to resolve

For details on how to use and execute the functional tests, please see the Testing documentation in Testing/Readme.md or reach out to the dev team via other channels.
Acceptance criteria

[ ] All existing test plan variants for the product have been successfully run
[ ] Test plan results have been reviewed and any issues resolved or reported as new issues (Link created issues below)

The failure against GCCHIGH maybe related to an existing issue #622
Since its an existing issue against functional test process for a single tenant type - this will not stop Halibut release.


feat: Optimize Join Penalty Calculation Based on Build Side Data Volume

Proposed changes
This PR introduces an optimization that adjusts the penalty applied during join operations based on the volume of data on the build side. Specifically, when the number of rows and width of the tables being joined are equal, the materialization costs are now considered more accurately. The update ensures that joins with a larger dataset on the build side incur a higher penalty, improving overall query performance and resource allocation.

run buildall

run buildall

run buildall

run buildall


Support labels

To keep PR titles cleaner, we use labels. Could we add configuration for this?

Hi James, the WIP app is already looking at labels, see https://github.com/gr2m/wip-bot#usage. Does that work for you? I try to avoid configuration to keep the code base simple

FWIW, our organization uses the label "Work in Progress" rather than "wip".  It might be worth adding that to the regex in containsWIP.

Yes, I’m a big fan of avoiding abbreviations where possible. Feel free to send a pull request for that

:tada: This issue has been resolved in version 2.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


no kind "InferenceServiceList" is registered for version "serving/v1beta1"

/kind bug
What steps did you take and what happened:
[A clear and concise description of what the bug is.]
I'm using fake client using NewSimpleClientset.
and testing List function
list, err := suite.fakeClient.Serving.List(v12.ListOptions{})
s.NoError(err)

this error occurs
 no kind "InferenceServiceList" is registered for version "serving/v1beta1" in scheme "pkg/runtime/scheme.go:100"
What did you expect to happen:
I expect fake list function work
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
other fake functions work well

KFServing Version: 0.8.0
h- OS (e.g. from /etc/os-release): MacOS

I have the kserve go client API has not been updated for a while.

no longer valid
/close


How to do anchor links within a single page

Hello, really enjoy docsify. Is there any way to be to have anchor links within a single page?
My use case is that I generate markdown API documentation automatically from JSDoc, and would like to be able to just add that as a single page within docsify project / site. The generated markdown automatically generates a "table of contents" that's a listing of all functions and classes of all the API within the project at the top of the file, and the table of contents links to the actual documentation for those that are elsewhere later in the file.
So for example it may end up being something like:
MyClass
...

### MyClass

I can't seem to get this to work even if I try modifying the anchor links or setting data-id properties.
Is this possible to accomplish using docsify somehow (even with some hacking or manual manipulation of the generated markdown).
If not perhaps point me in the right direction and I can try and do a PR.
Thanks

You need generate markdown syntax. Such as
[MyClass](#MyClass)

It will not be compiled if it is HTML.

Hmm maybe I didn't explain the issue correctly. It seems that docsify actually parses the file and sets the links to non-existent / invalid path. Lets say you have markdown file api.md and in it we have something like this:
See documentation for [MyClass](#myclass)

And this should jump to elsewhere later in api.md. Docsify seems to create a link http://localhost:3000/#/?id=myclass when the correct link is http://localhost:3000/#/api?id=myclass. I am trying to see if there is a workaround so it actually jump within the actual page, rather than trying to look for a path / route within docsify app. Hope this helps.

It really is a bug. Do you have time to fix it?

From some debugging it seems that this fixes it, but I am not sure about the complete effect my changes...?
function toURL (path, params) {
  var inline = path[0] === '#';
  var route = parse(replaceSlug(path));

  route.query = merge({}, route.query, params);
  path = route.path + stringifyQuery(route.query);
  path = path.replace(/\.md(\?)|\.md$/, '$1');

  if (inline) path = currentPath + path;

  return cleanPath('#/' + path)
}

Basically we detect that the path / href starts with '#' indicating usage of inline anchor / href link like
[MyClass](#myclass)

It will generate a link http://localhost:3000/#/api?id=myclass
And then later in file if we have

### MyClass

It will correctly link and jump to that anchor. Not sure if this is the correct approach.

Yes, I think it is right.

It'd be neat to be able to apply this mapping to regular  tags too, because there are some cases where it is not possible to put the a[]() syntax.
For example, I want to make some tables, but GitHub-flavored markdown does not support cells with multi-line content, so I need to use
 and due to the nesting, I can not place markdown inside the  elements. But I can only place  elements.


NPE If i add Fact with null value and use annotation

When we use annotation in pojo for argument and in facts i push null for one or more fact, there is NPE when easy-rules try to call method.
This issue is corrected in PR  #137

Hi @bpoussin
Thank you for reporting and fixing this issue 👍 Congrats on your First PR!
I added you to the contributors list: cf66bd6eb169f11d43551fb7abe2f9097da2bca9
I'm releasing v3.2 soon. Your fix will be included. Stay tuned 😉
Kind regards,
Mahmoud


Update to support Minecraft 1.21 on Fabric

Please update to support Minecraft 1.21 on Fabric. Thanks!

i agree


what are the C++ types I can use in function arguments for sol::state::set_function() ?

I've got weird runtime errors when I tried things like this, for example:
lua.set_function("loadImage",[](char ch, std::string const &imageFile) { ... });
I had to change to this, to make it work:
lua.set_function("loadImage",[](std::string ch, std::string imageFile) { /* use ch[0] here */ });
lua code: loadImage("A","a.png")
also is there any way to bind user data in function arguments? It did not work for me as well.

There's no char type in lua. "A" is a string. Even if it's just one char long (it's one byte long), it's a string. We can add an overload and converter for this, just to make it easier to deal, but... it's kind've hazardous. For example, if someone has a UTF8 encoded string, and they ask for the char back... do we just yank the first byte, or do we try to decode the string to get the first "character"? What about wchar_t? char32_t? char16_t ? sol would need to include an encoder and a decoder with it, and then we'd also have to specify what we push these characters as (utf8? utf16? utf32?, etc. etc...).
Binding userdata in function arguments should just work™, if it's a class type. If you want an actual userdata from lua and not the class type, try using sol::userdata to get the raw void* pointer.

#define SOL_CHECK_ARGUMENTS

#include "sol.hpp"
#include <iostream>

int main(int argc, char* const argv[]) {
	sol::state lua;
	lua.open_libraries(sol::lib::base);

	struct bark {
		int x = 500;

		bark() {}
		bark(int x) : x(x) {}
	};

	lua.set("b", bark{ 200 });

	lua.set_function("fx", [](bark& b) { std::cout << b.x << std::endl; });
	lua.script("fx(b)");
	return 0;
}

Works for me. Note that you don't have to set a usertype / metatable to at the very least pass it around like an opaque handle: it just works. Please tell me if something specifically goes wrong.


OBF - Grey scores are displayed for cosmetics

OBF - Grey food scores are displayed for cosmetics

Fixed by #3745

Fixed by #3745


[Feature] Cooldown

Hello i love the plugin thanks
i was wondering if you can add an way to have cooldown on Player Command and Console Command, instead of having players spamming it
thanks

It will, in an update i am working on.


New command: _crud

Is your feature request related to a problem? Please describe.
Create read update delete wiki pages. (Probably not create and update, wikilink command does read, so really just move, revert, and delete)

move: move a page. Accepts arg - -noredirect
delete: deletes a page. Duh.
revert: revert the latest edit to a page.

Bulk close due to archive of repo.


FPGA: new reference design and linear algebra library - Cholesky-based matrix inversion

Adding a New Sample
Description
This pull request adds both:

a new linear algebra header library (streaming_cholesky_inversion.hpp) that performs matrix inversions based on the Cholesky decomposition.
a new reference design that exercises this new library.

Checklist
Administrative

[x] Review sample design with the appropriate Domain Expert: @mdbtucker

Code Development

[x] Implement coding guidelines and ensure code quality. see wiki for details
[x] Adhere to readme template
[x] Enforce format via clang-format config file
[x] Adhere to sample.json specification. https://github.com/oneapi-src/oneAPI-samples/wiki/sample-json-specification
[x] Ensure/create CI test configurations for sample (ciTests field) https://github.com/oneapi-src/oneAPI-samples/wiki/sample-json-ci-test-object
[x] Run jsonlint on sample.json to verify json syntax. www.jsonlint.com

Security and Legal

[ ] OSPDT Approval (see Project Manager for assistance)
[x] Compile using the following compiler flags and fix any warnings, the falgs are: "/Wall -Wformat-security -Werror=format-security"
[ ] Virus scan

Review

[ ] Review DPC++ code with Paul Peterseon. (GitHub User: pmpeter1)
[ ] Review readme with Tom Lenth(@tomlenth) and/or Project Manager
[ ] Tested using Dev Cloud when applicable

@pmpeter1 thanks, fixed now
@tomlenth do you mean the  tags? These appear in all FPGA READMEs. If you think these should be removed, we can make a bulk change to all the FPGA READMEs rather than having it fixed it but not in the other ones?

@tomlenth I'll do a bulk change then!
@praveenkk123 Can you have a look at this PR? It has been reviewed internally before, and I've manually validated it


Autocomplete does not work for nested files

While editing my pure haxe (no OpenFL, Haxeflixel, etc) autocomplete stops working while in files not in the folder that build.hxml is in.  Autocomplete works perfectly fine for all files in the same folder as build.hxml.
Screenshot of console:
https://i.gyazo.com/833078222224afbe9137fef03c09233a.png
See the line that says 'Display file was not found in class path'
Any ideas?

The --cwd is not putting it in the source file's folder, just the builds folder. Obviosly resetting the HXML file and everything isn't fixing the issue.

I cannot tell without knowing the contents of your hxml file and your directory structure, but I just tested and autocomplete works find on any file that is in the class path, not only the directory of the hxml file. Of course the file still needs to be inside a directory referenced with -cp some/directory.

Ah, -cp was needed. I am however a little confused, I never had -cp in my hxml file before, and my project compiled it just fine. The main.hx file was in the same folder as build.hxml, and it accessed all the other files just fine.
I only have around 8 folders in my project (it is a medium sized project), and in order for autcomplete to work, I have to list every sub folder. Not a big deal, not at all, but it seems like it would be tedious for larger projects. Anyhow, thanks for your help!
Screenshot of working build.hxml and file structure.
https://i.gyazo.com/7a3ec859f1ba3c3c60776ce77283cabd.png
Thank you for your help!

You should not need -cp for sub directories, as long as they are valid haxe packages. I am not sure it's a good idea to use subfolders with capital letters as well.
You can find more informations here: http://haxe.org/manual/type-system-modules-and-paths.html
Closing because it is not really an issue caused by atom-haxe.


You Absolute Legend`

First, Thanks for all your hard work and efforts! You are a Legend :)
This error is caused by the final line, you have sentencepiece 0.2.0 which is incompatible. image-reward 1.5 requires timm==0.6.13, but you have timm 1.0.8 which is incompatible.
ArgosTranslate was unable to install due to the previous installation of a newer Dependancy.
error copy:
aeiou 0.0.20 requires soundfile<=0.10.2, but you have soundfile 0.12.1 which is incompatible. argostranslate 1.9.3 requires sentencepiece==0.1.99, but you have sentencepiece 0.2.0 which is incompatible. image-reward 1.5 requires timm==0.6.13, but you have timm 1.0.8 which is incompatible.
From the argostranslate github, it seems that the 0.2.0 was a fix for newer python, regardless we can change the sentence piece deps to 0.1.99 and then it will require the correct argotranslate 1.9.3
https://github.com/argosopentech/argos-translate/issues/405

I reduced the requirements for Sentencepiece, to match argostranslate 1.9.3
sentencepiece>=0.1.99

TLDR, Sentencepiece required by ArgosTranslate, so i checked the compatibility and the error was resolved (for me at least) with this change. maybe this will help someone if they also run into it,,

all loaded fine :) Thanks again !

But my json file won't load, I don't know why, everything seems to be installed


substring does not support UTF-16

"🍕".substring(0, 1);
> "�"

This issue came up because a user inputted an emoji into a data field. Attempting to render Text("🍕".substring(0, 1)) in Flutter results in:
flutter: ══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY ╞═════════════════════════════════════════════════════════
flutter: The following ArgumentError was thrown during performLayout():
flutter: Invalid argument(s): string is not well-formed UTF-16
flutter:
flutter: When the exception was thrown, this was the stack:
flutter: #0      ParagraphBuilder.addText (dart:ui/text.dart:1157:7)
flutter: #1      TextSpan.build
package:flutter/…/painting/text_span.dart:172
flutter: #2      TextPainter.layout
package:flutter/…/painting/text_painter.dart:352
flutter: #3      RenderParagraph._layoutText
...

A substring method that works on code points rather than UTF-16 code units might be inefficient. Eg, taking a substring near the end of a very long string would be slow, because it would have to iterate through most of the string counting code points.
The substring documentation doesn't make it clear that the method operates on code units. Can we update it to explain this, similarly to the good explanation given on the [] operator?

I am having the exact same problem. My case is I want to programmatically delete (backspace) a text input which may have emojis. So far, my workaround is as follows:
var s = "abc😀";
var sRunes = s.runes;
print(String.fromCharCodes(sRunes, 0, sRunes.length-1));

And make sure users do not input those emojis which have length 4.

Hi, just FYI, I have found a better workaround. Check my stackoverflow's answer code: https://stackoverflow.com/a/56135774/348719

I have found another workaround , is more respource expensive but so far it works with all emojis.
In my particular case i wanted to do a subtring [EX from index 0  to 16], and count an emoji as an individual character , however it was just getting half of the text due to the emojis in it
My workaround is this one
Create a function called runeSubtring() like this one
String runeSubstring({String input , int start , int end}){
  String finalString = ''; //initialize the string
  List individualRunes = input.runes.toList(); //convert the string to a list of runes
  individualRunes.sublist(start,end).forEach((rune) { //"substring" the list
          String character = String.fromCharCode(rune); //convert the list back to the string one by one
          finalString = finalString + character;
});
  return finalString; //return the substring
}

and just use it like this when you like
String example = r'Example \ud83d\ude13  Example \ud83d\ude13';
String result = runeSubstring(input: example,start: 0,end:10);
String resultEnd = runeSubstring(input: example,start: 11); //from 11 to the end

If the text is really big, you should do input.runes.toList(); outside the function since it will leverage the charge of converting text to runes and to a list everythime the function is called.

See https://github.com/dart-lang/sdk/issues/28404 and https://github.com/dart-lang/language/issues/34,  long discussions about making correct String manipulation  easier. We will probably close this issue as being one example of the bigger issue.
@LiteCatDev String.fromCharCodes takes an Iterable of rune values, so you can simplify your code to:
String runeSubstring({String input, int start, int end}) {
  return String.fromCharCodes(input.runes.toList().sublist(start, end));
}

this SO might be related https://stackoverflow.com/questions/54483177/handling-grapheme-clusters-in-dart

https://github.com/dart-lang/language/issues/685 is our current attempt at supporting this via a package; closing the present issue in favor of that

Similar issue I need to solve have posted problem here
https://stackoverflow.com/questions/68518125/flutter-dart-how-to-trim-if-special-characters-are-present
please give your inputs thanks

Use package:characters: https://pub.dev/documentation/characters/latest/


Compilation error in version 0.3.1 using MSVC 19.38 and Windows-on-arm

Hi @jeremy-rifkin. I stumbled upon this error, and I wonder whether your library supports Windows-on-arm yet?
backward-cpp\backward.hpp(1168,29): error C2039: 'Eip': is not a member of '_CONTEXT'
backward-cpp\backward.hpp(1169,32): error C2039: 'Esp': is not a member of '_CONTEXT'
backward-cpp\backward.hpp(1170,32): error C2039: 'Ebp': is not a member of '_CONTEXT'
see declaration of '_CONTEXT'
And this is CONTEXT object, defined in winnt.h for ARM64_NT:
typedef struct DECLSPEC_ALIGN(16) DECLSPEC_NOINITALL _ARM64_NT_CONTEXT {
//
// Control flags.
//
/* +0x000 */ DWORD ContextFlags;

//
// Integer registers
//

/* +0x004 */ DWORD Cpsr;       // NZVF + DAIF + CurrentEL + SPSel
/* +0x008 */ union {
                struct {
                    DWORD64 X0;
                    DWORD64 X1;
                    DWORD64 X2;
                    DWORD64 X3;
                    DWORD64 X4;
                    DWORD64 X5;
                    DWORD64 X6;
                    DWORD64 X7;
                    DWORD64 X8;
                    DWORD64 X9;
                    DWORD64 X10;
                    DWORD64 X11;
                    DWORD64 X12;
                    DWORD64 X13;
                    DWORD64 X14;
                    DWORD64 X15;
                    DWORD64 X16;
                    DWORD64 X17;
                    DWORD64 X18;
                    DWORD64 X19;
                    DWORD64 X20;
                    DWORD64 X21;
                    DWORD64 X22;
                    DWORD64 X23;
                    DWORD64 X24;
                    DWORD64 X25;
                    DWORD64 X26;
                    DWORD64 X27;
                    DWORD64 X28;
/* +0x0f0 */        DWORD64 Fp;
/* +0x0f8 */        DWORD64 Lr;
                } DUMMYSTRUCTNAME;
                DWORD64 X[31];
             } DUMMYUNIONNAME;
/* +0x100 */ DWORD64 Sp;
/* +0x108 */ DWORD64 Pc;

//
// Floating Point/NEON Registers
//

/* +0x110 */ ARM64_NT_NEON128 V[32];
/* +0x310 */ DWORD Fpcr;
/* +0x314 */ DWORD Fpsr;

//
// Debug registers
//

/* +0x318 */ DWORD Bcr[ARM64_MAX_BREAKPOINTS];
/* +0x338 */ DWORD64 Bvr[ARM64_MAX_BREAKPOINTS];
/* +0x378 */ DWORD Wcr[ARM64_MAX_WATCHPOINTS];
/* +0x380 */ DWORD64 Wvr[ARM64_MAX_WATCHPOINTS];
/* +0x390 */

} ARM64_NT_CONTEXT, *PARM64_NT_CONTEXT;

Hi @alexbichkov, I don't have a windows arm machine to test with however when adding cpptrace to vcpkg I was able to get it working for ARM. The StackWalk64 unwind code does also have specific handling for arm: https://github.com/jeremy-rifkin/cpptrace/blob/448c325d2e9d1c6e28749202c1dca7ec546cca6a/src/unwind/unwind_with_dbghelp.cpp#L83-L93
I'm a bit confused by the error message you provided though, as that is from backward-cpp which is a different library.

Surely it's a different library and I got confused. Sorry to bother you.

All good, take care