Make validation optional

Is your feature request related to a problem? Please describe.
Having the validation check using the schema is nice. However, sometimes it fails to run a "valid" (as is "it runs on GitLab") pipeline.
I believe it might also be an issue if you are using a private GitLab instance which might not support the latest version of the schema?
Describe the solution you'd like
Make all validation optional, or add an option to print a warning instead?
Describe alternatives you've considered
Version later than 4.46.1 fails because of invalid schema or max. includes check
The workaround for me is to manually update the code or use
npm i -g<EMAIL_ADDRESS>

Having the validation check using the schema is nice. However, sometimes it fails to run a "valid" (as is "it runs on GitLab") pipeline.

yeah unfortunately because gitlab seem to perform the schema validation on their backend
https://github.com/firecow/gitlab-ci-local/pull/1248#issuecomment-2146971764
to disable the schema validation
export GCL_JSON_SCHEMA_VALIDATION='false'

(Also, hard-coding it to 150 is not right. Maybe it should come from an env. variable as it can be configured in the GitLab admin) https://github.com/firecow/gitlab-ci-local/pull/1123#issuecomment-2152192377
Totally agree on this

@hverlin regarding https://github.com/firecow/gitlab-ci-local/pull/1123#issuecomment-2152192377

Can we just print a warning instead?

Currently, this also act as a safety net to breakout of circular includes.

I don't think the check here is accurate, as gitlab-ci-local fails to run a pipeline which works in our GitLab instance (at least it might count includes a bit differently?

could you try to hack gitlab-ci-local src code to show whats the MAXIMUM_INCLUDES that gitlab-ci-local calculated?

could you try to hack gitlab-ci-local src code to show whats the MAXIMUM_INCLUDES that gitlab-ci-local calculated?
@ANGkeith I am getting 156 includes :sweat_smile:

Other suggestion: maybe printing all the includes on error would help. In our case, it made pretty obvious which includes were included several times.
export class ParserIncludes {
  private static count: number = 0;
  private static allIncludes: Array<string> = [];

  static resetCount(): void {
    this.count = 0;
  }

  static async init(
    gitlabData: any,
    opts: ParserIncludesInitOptions,
  ): Promise<any[]> {
    this.count++;
    if (gitlabData?.include) {
      for (const incl of gitlabData?.include) {
        this.allIncludes.push(Object.values(incl).join(":"));
      }
    }
    assert(
      this.count <= MAXIMUM_INCLUDE + 1, // 1st init call is not counted
      chalk`This GitLab CI configuration is invalid: Maximum of {blueBright ${MAXIMUM_INCLUDE}} nested includes are allowed!.\n${
        this.allIncludes.sort().join("\n")
      }`,
    );

...

For the validation, I think the error message could be updated:
const validate = ajv.compile(schema);
const valid = validate(data);
assert(valid, chalk`Invalid gitlab-ci configuration! ${ajv.errorsText(validate.errors)}.\nJson schema validation failed. Dump the following to the pipeline editor to debug: ${yaml.dump(data)}`);

For large pipelines, dumping to the console is not ideal, maybe in a file under .gitlab-ci-local instead?
The error that I am locally that is different from GitLab pipeline editor is the following:
job:
  script: 'echo "test"'
  artifacts:

which should be invalid

In GitLab, this is what I see (valid and wrong at the same time!)

I have other repos which do execute on GitLab but are failing the validation. Their pipeline is also too big, so I think printing the full yaml directly to the console should be avoided.
I can make an MR if this works for you?

I have other repos which do execute on GitLab but are failing the validation. Their pipeline is also too big, so I think printing the full yaml directly to the console should be avoided.
I can make an MR if this works for you?

yup please go ahead 👍, i don't mind this new addition too
while waiting, you can get them to use some bash magic
gitlab-ci-local 2> /dev/stdout | grep --color=never . | tail -n +3 | pbcopy

For the validation, I think the error message could be updated:

In my opinion the error message isn't that helpful especially if there's alot of errors, so for ease of use we can just rely on pipeline editor.
out of the box, the some of the error message will also be quite convulated
eg.
Given the following .gitlab-ci.yml
foo:
  tags:
    - true
  script:
    - echo 1


is there a way to remove/make invisible the lower pane from the display?

As @Aya-S mentioned: https://discuss.streamlit.io/t/drawable-canvas/3671/25

Should be out next release :)


Replace ResizableTriggerIcon in ResizeTrigger to use @yamada-ui/lucide icon

Closes #3536
Description
Replaced ResizableTriggerIcon in ResizeTrigger to use @yamada-ui/lucide icon.
Current behavior (updates)

New behavior

Is this a breaking change (Yes/No):

Additional Information

Please review again @kaehehehe @hirotomoyamada.

Please review again @kaehehehe @hirotomoyamada.

No, please run pnpm install and commit pnpm-lock.yaml.

Thanks @hirotomoyamada, I have pushed the new pnpm-lock.yaml, please review.


bump sync plugin to v0.1.19

for https://bugzilla.redhat.com/show_bug.cgi?id=1461235
@bparees @dongboyan77 FYI

[test]

@bparees I also made a very small tweak to the README ... I'll wait a bit to see if you have time to review before merging

corrections pushed ... thanks @bparees

lgtm

[merge]

@stevekuznetsov uh.  i'm not sure how this merged so fast?  it didn't have passing tests for the most recent commit.


Hi PVPBET! Could I work with you?

Hi, I am senior full stack engineer who is very familiar with igaming development industry.
If you don't mind could I work with you?
I have attached some files so that you can understand my recent works.
My Discord Username: gohchenkai

Thank you for your interest. We’ll reach out if we have any future needs.


java.lang.IllegalStateException: Handler (android.os.Handler) sending message to a Handler on a dead thread

Hi and thanks for such a nice library! Checked this out today and saw the issue
Steps to reproduce:

.setShowPicker(true)
launch SandriosCamera with picker
tap on any of previews

01-05 17:58:57.256 22321-22641/? W/MessageQueue: Handler (android.os.Handler) {3a72e4e} sending message to a Handler on a dead thread                                                  java.lang.IllegalStateException: Handler (android.os.Handler) {3a72e4e} sending message to a Handler on a dead thread                                                      at android.os.MessageQueue.enqueueMessage(MessageQueue.java:543)                                                      at android.os.Handler.enqueueMessage(Handler.java:631)                                                      at android.os.Handler.sendMessageAtTime(Handler.java:600)                                                      at android.os.Handler.sendMessageDelayed(Handler.java:570)                                                      at android.os.Handler.post(Handler.java:326)                                                      at android.hardware.camera2.impl.CameraDeviceImpl.close(CameraDeviceImpl.java:1010)                                                      at com.sandrios.sandriosCamera.internal.manager.impl.Camera2Manager.closeCameraDevice(Camera2Manager.java:463)                                                      at com.sandrios.sandriosCamera.internal.manager.impl.Camera2Manager.closeCamera(Camera2Manager.java:442)                                                      at com.sandrios.sandriosCamera.internal.manager.impl.Camera2Manager.access$600(Camera2Manager.java:53)                                                      at com.sandrios.sandriosCamera.internal.manager.impl.Camera2Manager$4.run(Camera2Manager.java:225)                                                      at android.os.Handler.handleCallback(Handler.java:739)                                                      at android.os.Handler.dispatchMessage(Handler.java:95)                                                      at android.os.Looper.loop(Looper.java:148)                                                      at android.os.HandlerThread.run(HandlerThread.java:61)
Exception is handled, but will be good to fix.
Device - Nexus 5, Android 6.0.1

Exception is handled, but will be good to fix.
Device - Nexus 5, Android 6.0.1

@kosmonastya  I have the same device, I am unable to produce this issue. Can you please confirm which version you are using. Because I have tested this on various device. It would be great If you provide a sample project in which this issue is getting produced.

Was using 1.0.7
Created an empty project with just a button to call the lib - works like a charm, but in my big project - same issue. Give me a day, I'll try to find watta hell and report here then

I do not have a Nexus 5 Device. But I tried with other Samsung and Moto Devices running 6.0.1 and was unable to reproduce the error.
PS: You do not need to manage permissions as done in the sample app, The library does that for you.

Well, looks like not a major issue. I'll try to research a bit more, but
let's close it for now!
P.S. Thanks, I didn't notice the thing abt permissions ☺
On Fri, Jan 6, 2017, 12:52 Arpit Gandhi<EMAIL_ADDRESS>wrote:

I do not have a Nexus 5 Device. But I tried with other Samsung and Moto
Devices running 6.0.1 and was unable to reproduce the error.
PS: You do not need to manage permissions as done in the sample app, The
library does that for you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sandrios/sandriosCamera/issues/3#issuecomment-270831062,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE5fLbwzaCy5nGncIFAb-npS10KyZV_pks5rPcibgaJpZM4Lbhbd
.

This might be related to the issue. Steps to reproduce the problem:

Load a fragment with CameraFragment
Now open a camera activity.
Close/Destroy the camera activity.
PROBLEM: Camera fragment doesn't behave properly as background thread is stopped.

Workaround:
File : CameraManager

boolean isBackgroundThreadStoped();

File: BaseCameraManager

 public boolean isBackgroundThreadStoped() {
        return backgroundThread == null;
}

File: Camera1Controller and
  @Override
    public void onResume() {
        Log.i(TAG, "onResume: ");
        if (cameraManager.isBackgroundThreadStoped()) {
            cameraManager.initializeCameraManager(configurationProvider, cameraView.getActivity());
        }
        cameraManager.openCamera(currentCameraId, this);
    }

File: Camera2Controller
  @Override
    public void onResume() {
        if (camera2Manager.isBackgroundThreadStoped()) {
            camera2Manager.initializeCameraManager(configurationProvider, cameraView.getActivity());
        }
        camera2Manager.openCamera(currentCameraId, this);
    }

I/SandriosCameraFragment: onPause:
I/Camera1Controller: onResume:
I/Camera1Manager: openCamera:
I/Camera1Controller: onDestroy:
I/BaseCameraManager: stopBackgroundThread:
I/SandriosCameraFragment: onResume:
I/Camera1Controller: onResume:
I/Camera1Manager: openCamera:
E/Camera1Manager: openCamera backgroundHandler is null
D/Camera1Manager: Error starting camera preview: Attempt to invoke virtual method 'android.hardware.Camera$Parameters android.hardware.Camera.getParameters()' on a null object reference
D/Camera1Manager: Error starting camera preview: Attempt to invoke virtual method 'android.hardware.Camera$Parameters android.hardware.Camera.getParameters()' on a null object reference


redirectUrl is http when it should be https

The following code should create the redirectUrl as https but creates it as http if the site is accessed through http:
public IActionResult ExternalLogin(string provider, string returnUrl = null)
{
	var redirectUrl = Url.Action("ExternalLoginCallback", "Account", new { ReturnUrl = returnUrl }, protocol: "https");
	var properties = _signInManager.ConfigureExternalAuthenticationProperties(provider, redirectUrl);
	return Challenge(properties, provider);
}

I am using Nginx  as a reverse proxy that proxies external https calls to http://localhost:5000. The site does therefor think that it is accessed through http even though it is accessed externally with https.
The Challenge method seems to convert the https redirectUrl  to http. Is this intended or is it a bug?

I seem to have the same thing...

Found the solution!
Add this to your startup (before app.UseIdentity*):
            app.UseForwardedHeaders(new ForwardedHeadersOptions
            {
                ForwardedHeaders = ForwardedHeaders.XForwardedFor | ForwardedHeaders.XForwardedProto,
                ForwardLimit = null,
                RequireHeaderSymmetry = false
            });

Edit /etc/nginx/proxy_params to make it look like this:
proxy_redirect off;
proxy_set_header Host $host;
proxy_set_header X-Real-IP $remote_addr;
proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
client_max_body_size 10m;
client_body_buffer_size 128k;
proxy_connect_timeout 90;
proxy_send_timeout 90;
proxy_read_timeout 90;
proxy_buffers 4 256k;
proxy_buffer_size 128k;

And make your host look like this:
# Workaround for ASP.NET Core
map $http_x_forwarded_proto $thescheme {
        default $scheme;
        https https;
}

server {
        listen 80;
        listen [::]:80;

        root /var/www/html;

        index index.html index.htm index.nginx-debian.html;

        server_name xxxxxxxxxxxxxx;

        location / {
                return 301 https://$host$request_uri;
        }
}
server {
        listen 443 ssl http2;
        listen [::]:443 ssl http2;
        ssl_certificate xxxxxxxxxxxxxxx;
        ssl_certificate_key xxxxxxxxxxxxxxxxxxx;
        ssl_trusted_certificate xxxxxxxxxxxxxxxxxxxx;
        include /etc/nginx/snippets/ssl.conf;

        root /var/www/html;

        index index.html index.htm index.nginx-debian.html;

        server_name xxxxxxxxxxxxxx;

        location / {
                proxy_pass http://localhost:5000;
                limit_req zone=one burst=10;
                include /etc/nginx/proxy_params;
                proxy_set_header X-Forwarded-Proto $thescheme;
        }
}

Thank you. I had followed https://docs.microsoft.com/en-us/aspnet/core/publishing/linuxproduction before and had to add the following to my config file:
        proxy_set_header X-Forwarded-Proto $thescheme;
        proxy_redirect off;
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
...
map $http_x_forwarded_proto $thescheme {
        default $scheme;
        https https;
}

It now looks like
server {
    ...
    location / {
        proxy_pass http://localhost:5002;
        proxy_http_version 1.1;
        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Connection keep-alive;
        proxy_set_header Host $host;
        proxy_cache_bypass $http_upgrade;

        #Added everything bellow
        proxy_set_header X-Forwarded-Proto $thescheme;
        proxy_redirect off;
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
    }
}

map $http_x_forwarded_proto $thescheme {
        default $scheme;
        https https;
}

I did not have to change anything in the startup file so it continues to look like
app.UseForwardedHeaders(new ForwardedHeadersOptions
{
	ForwardedHeaders = ForwardedHeaders.XForwardedFor | ForwardedHeaders.XForwardedProto
});

Should the documentation be upgraded with this information?

I had to overwrite scheme manually. ForwardedHeaders would not do it.
Here is the code:

app.Use( ( context, next ) => {
                if ( context.Request.Headers.TryGetValue( "X-Forwarded-Proto", out StringValues proto ) ) {
                    context.Request.Scheme = proto;
                }
                return next();
            } );


EXC_BAD_ACCESS v0.26

Hello all
I have a project with some pods and I updated it
I have a "EXC_BAD_ACCESS (code=1, address=0x0) in this lines:
"objc_setAssociatedObject(objectRequestOperation, RKOperationStartDate, [NSDate date], OBJC_ASSOCIATION_RETAIN_NONATOMIC);"
(RKObjectRequestOperation.m)
"[[NSNotificationCenter defaultCenter] postNotificationName:RKObjectRequestOperationDidStartNotification object:weakSelf];"
(RKObjectRequestOperation.m)
"block" inside

(void)setExecutionBlock:(void (^)(void))block
{
__weak __typeof(self)weakSelf = self;
TKState *executingState = [self.stateMachine stateNamed:RKOperationStateExecuting];
[executingState setDidEnterStateBlock:^(TKState *state, TKTransition *transition) {
[weakSelf.operation didChangeValueForKey:@"isExecuting"];
dispatch_async(weakSelf.dispatchQueue, ^{
block();
});
}];
}
(RKOperationStateMachine.m)

Any ideas??
I need help with this issue.
Thanks

UPDATE INFO:
Reading this another issue
https://github.com/RestKit/RestKit/issues/2356
I edit two files:
File: RKOperationStateMachine.m
Method:

(void)setExecutionBlock:(void (^)(void))block
{
// __weak __typeof(self)weakSelf = self;
__typeof(self)weakSelf = self;
TKState *executingState = [self.stateMachine stateNamed:RKOperationStateExecuting];
[executingState setDidEnterStateBlock:^(TKState *state, TKTransition *transition) {
[weakSelf.operation didChangeValueForKey:@"isExecuting"];
dispatch_async(weakSelf.dispatchQueue, ^{
block();
});
}];
}

File: RKObjectRequestOperation.m
Method:
// Designated initializer

(instancetype)initWithHTTPRequestOperation:(RKHTTPRequestOperation *)requestOperation responseDescriptors:(NSArray *)responseDescriptors
{
NSParameterAssert(requestOperation);
NSParameterAssert(responseDescriptors);
self = [super init];
if (self) {
self.responseDescriptors = responseDescriptors;
self.HTTPRequestOperation = requestOperation;
self.HTTPRequestOperation.acceptableContentTypes = [RKMIMETypeSerialization registeredMIMETypes];
self.HTTPRequestOperation.acceptableStatusCodes = RKAcceptableStatusCodesFromResponseDescriptors(responseDescriptors);
self.HTTPRequestOperation.successCallbackQueue = [[self class] dispatchQueue];
self.HTTPRequestOperation.failureCallbackQueue = [[self class] dispatchQueue];
  **// __weak __typeof(self)weakSelf = self;
  __typeof(self)weakSelf = self;**
  self.stateMachine = [[RKOperationStateMachine alloc] initWithOperation:self dispatchQueue:[[self class] dispatchQueue]];
  [self.stateMachine setExecutionBlock:^{
      [[NSNotificationCenter defaultCenter] postNotificationName:RKObjectRequestOperationDidStartNotification object:weakSelf];
      RKIncrementNetworkActivityIndicator();
      if (weakSelf.isCancelled) {
          [weakSelf.stateMachine finish];
      } else {
          [weakSelf execute];
      }
  }];

....
....
This changes works for my project.
I am interesting in know if this changes have sense
Thanks.

Surprisingly it worked for me too. Is it the right approach ?

Just taking away __weak can result in a memory leak.
See https://dhoerl.wordpress.com/2013/04/23/i-finally-figured-out-weakself-and-strongself/

@CesarRN   Great.


Pexip app crash during long duration calls or multiple short duration calls

Describe the bug
Pexip app crash during long duration calls or multiple short duration calls.
Issue Description: Pexip app crashes / closes during long hours or multiple short duration calls.
Issue: Only Video call and no screen sharing
OS: Android 12
H/w: NXP i.MX 8M Plus
lowmem.txt
We did test other video platform calls like zoom on same device and don’t observe similar issues.
Attached are the app and tombstone logs.
To reproduce
Make multiple video long duration calls with out re-booting the device.
Expected behavior
Pexip app should not crash.
Version
0.12.0
Additional context
No response

Thanks for the log, it looks very interesting - the first call takes about 1.5 hours, but subsequent calls do not exceed 2 minute duration:
2023-04-20 15:18:43.945   592-1118  ActivityTaskManager     system_process                       I  START u0 {act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x10200000 cmp=com.pexip.sdk.sample/.SampleActivity bnds=[8,454][260,552]} from uid 10074
2023-04-20 16:43:05.392   219-219   lowmemorykiller         pid-219                              I  Kill 'com.pexip.sdk.sample' (2248), uid 10086, oom_score_adj 0 to free 130572kB rss, 33708kB swap; reason: device is low on swap (236kB < 61436kB) and thrashing (200%)

2023-04-20 16:43:15.122   592-2868  ActivityTaskManager     system_process                       I  START u0 {act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x10200000 cmp=com.pexip.sdk.sample/.SampleActivity bnds=[8,454][260,552]} from uid 10074
2023-04-20 16:44:00.831   219-219   lowmemorykiller         pid-219                              I  Kill 'com.pexip.sdk.sample' (2982), uid 10086, oom_score_adj 0 to free 121888kB rss, 15580kB swap; reason: device is low on swap (0kB < 61436kB) and thrashing (201%)

2023-04-20 16:45:51.380   592-607   ActivityTaskManager     system_process                       I  START u0 {act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x10200000 cmp=com.pexip.sdk.sample/.SampleActivity bnds=[8,454][260,552]} from uid 10074
2023-04-20 16:47:11.236   219-219   lowmemorykiller         pid-219                              I  Kill 'com.pexip.sdk.sample' (3375), uid 10086, oom_score_adj 0 to free 93080kB rss, 15792kB swap; reason: device is low on swap (0kB < 61436kB) and thrashing (210%)

2023-04-20 16:52:51.115   592-3562  ActivityTaskManager     system_process                       I  START u0 {act=android.intent.action.MAIN cat=[android.intent.category.LAUNCHER] flg=0x10200000 cmp=com.pexip.sdk.sample/.SampleActivity bnds=[8,454][260,552]} from uid 10074
2023-04-20 16:54:08.702   219-219   lowmemorykiller         pid-219                              I  Kill 'com.pexip.sdk.sample' (3768), uid 10086, oom_score_adj 0 to free 126288kB rss, 15720kB swap; reason: device is low on swap (0kB < 61436kB) and thrashing (200%)

oom_score_adj 0 indicates that the app is in foreground and should have a extremely low probability of getting killed by lmk.
App memory usage at the time of death seems fairly reasonable (around 130 Mb for the first call, even lower for other calls). Profiling may help rule out memory leak, but just from the memory usage I'd say it's not happening in the app itself.
My guess is that there may be some underlying OS issue at play here (leaky codecs?) because the OS appears to struggle allocating enough RAM for the app.
Are you able to provide the following info for further investigation:

Hardware specs of the device
adb shell free output
Memory profiling of the app
A list of codecs present on the device, with their names, supported formats and whether they're hardware or not

Additionally, what commit did you checkout to test this? The issue description lists 0.12.0 (released at the end of May), but the logs indicate that they were recorded earlier (Apr 20). It'd be great if you could run these tests against the latest available commit.

Thanks for the response, while we work on the suggested analysis, please let us know if Pexip Android SDK has a call back when a participant leaves the conference. How the Android app knows about participant events like a new participant joined, left etc.

Thanks for the response, while we work on the suggested analysis, please let us know if Pexip Android SDK has a call back when a participant leaves the conference. How the Android app knows about participant events like a new participant joined, left etc.

It's currently not available. I suggest that you open a feature request for discussion/tracking.


AliasResolutionError with a suspected fix.

Describe the bug
I encountered this when using mkdocstrings on a fairly large mono-repo style codebase with mkdocs-monorepo.
The python handler from mkdocstrings would eventually trickle down to griffe.loader.resolve_module_aliases and would fail for a certain file.

Traceback
    data: CollectorItem = handler.collect(identifier, options)
  File "/Users/deepandas/.pyenv/versions/3.8.13/envs/mkdocs_test/lib/python3.8/site-packages/mkdocstrings_handlers/python/handler.py", line 199, in collect
    unresolved, iterations = loader.resolve_aliases(implicit=False, external=False)
  File "/Users/deepandas/.pyenv/versions/3.8.13/envs/mkdocs_test/lib/python3.8/site-packages/griffe/loader.py", line 217, in resolve_aliases
    next_resolved, next_unresolved = self.resolve_module_aliases(module, implicit, external)
  File "/Users/deepandas/.pyenv/versions/3.8.13/envs/mkdocs_test/lib/python3.8/site-packages/griffe/loader.py", line 357, in resolve_module_aliases
    sub_resolved, sub_unresolved = self.resolve_module_aliases(
  File "/Users/deepandas/.pyenv/versions/3.8.13/envs/mkdocs_test/lib/python3.8/site-packages/griffe/loader.py", line 357, in resolve_module_aliases
    sub_resolved, sub_unresolved = self.resolve_module_aliases(
  File "/Users/deepandas/.pyenv/versions/3.8.13/envs/mkdocs_test/lib/python3.8/site-packages/griffe/loader.py", line 357, in resolve_module_aliases
    sub_resolved, sub_unresolved = self.resolve_module_aliases(
  [Previous line repeated 1 more time]
  File "/Users/deepandas/.pyenv/versions/3.8.13/envs/mkdocs_test/lib/python3.8/site-packages/griffe/loader.py", line 335, in resolve_module_aliases
    if not implicit and not member.is_explicitely_exported:
  File "/Users/deepandas/.pyenv/versions/3.8.13/envs/mkdocs_test/lib/python3.8/site-packages/griffe/dataclasses.py", line 827, in __getattr__
    attr = getattr(self.target, name)
  File "/Users/deepandas/.pyenv/versions/3.8.13/envs/mkdocs_test/lib/python3.8/site-packages/griffe/dataclasses.py", line 936, in target
    self.resolve_target()
  File "/Users/deepandas/.pyenv/versions/3.8.13/envs/mkdocs_test/lib/python3.8/site-packages/griffe/dataclasses.py", line 964, in resolve_target
    raise AliasResolutionError(self.target_path) from error

However, I followed a hunch and was able to fix it magically by monkeypatching a fix in expressions:
https://github.com/mkdocstrings/griffe/blob/2c3b0cea0c9a216fc0a25445e0dbe9619a15a199/src/griffe/expressions.py#L32-L33
I simply added a path to allow str types
def __eq__(self, other: Name | Expression | str) -> bool:  # type: ignore[override]
        if isinstance(other, str):
            return self.full == other
        return self.full == other.full  # noqa: WPS437

To Reproduce
I haven't been able to find a demonstrable repro on a smaller scale, which makes me believe this could also stem from a weird import order issue from the large codebase I am trying to handle.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
System (please complete the following information):

griffe version: 0.24.1
mkdocstrings version: 0.8.0
Python version: 3.8.13
OS: MacOS

Additional context
I haven't played around with the library too much and wasn't able to figure out the underlying reason. Looking for more context into this and if possible, to accommodate the simple fix?

Hello, thanks a lot for the detailed report, for the time you took to investigate, and for the suggested fix :rocket:!
I'm very curious as to how you found that this change would fix the issue. I guess you ran a debugging session?
I also guess your monorepo is not public, and therefore I cannot debug on my end as well, or try to find a MRE :sweat_smile:?

Thanks for the swift response! Appreciate you looking into it.

I'm very curious as to how you found that this change would fix the issue. I guess you ran a debugging session?

Honestly, I didn't spend a lot of time into it, and my hunch originated from the member.is_explicitly_exported check on the traceback. Following that trail led me to this line in dataclasses.py:
https://github.com/mkdocstrings/griffe/blob/2c3b0cea0c9a216fc0a25445e0dbe9619a15a199/src/griffe/dataclasses.py#L409
The type hints in the initializer suggests exports is set[str] or list[str | Name], qualifying the __eq__(self, other) on Name for the list[Name] case when other is of type Name, but not for str.

I also guess your monorepo is not public, and therefore I cannot debug on my end as well, or try to find a MRE 😅?

Unfortunately, yes, it's a private repo, and I did put in some effort into setting up a minimal repro, but that didn't work. I am hoping the traceback, and my line of thought above could assist you in some way?

Should be fixed by 37ae0a2, thanks again @deepandas11. I'll try to pack a few more things before the next release.


Are you coding to help fluffypony to steal some more money?

https://tinyurl.com/e7u3kadu (changed link to removeddit since /r/monero mod removed to try to cover it up - maybe fluffy himself?)
Monero: BUT HE APOLOGISED TO SOME OF THEM AFTER LAUGHING AT THOSE IDIOTS!

Spam or truth? There is only one way to find out. Look into the sources yourself.

tari-project/tari#2944

tari-project/tari#2940

tari-project/tari#2938

tari-project/tari#2934

tari-project/tari#2943

tari-project/tari#2939

tari-project/tari#2937

tari-project/tari#2933


3D touch

Are this library is 3D Touch ?

Yes sir, you can implement 3D Touch with your custom level of screen pressure

@michelelacorte , thank you my bro 👍
Can you write simple code for show image in your library? for example sort images with grid layout, when click on this images and use forceTouch show this image in dialog. such as instagram.
with this simple understand your codes for all users. ❤

Please open question on http://stackoverflow.com/ for this.


BC6H MSE calculation error

There appears to be an error when calculating MSE for BC6H, where with the following files
https://send.firefox.com/download/dcc3a03777702c22/#ARpj0hUbj52fHzQkqv96Tw
Compressonator is saying MSE is 1.6773
Where as the correct MSE should be 0.0414 from what I can tell from other programs.

When comparing two different EXR files (no DDS involved to decode), the MSE is calculated the same. Thus, perhaps this is an issue with float textures in general?

Hi @Zelex, can you provide a reference link to the "other programs" you used to determine the 0.0414 value. Thx.

I'll make one for you, 1 sec.

https://send.firefox.com/download/990dd959f0492c5d/#X0pnxs-4ZgpxYvlx-_Ssmg

@Zelex Thank you for the reference.


Time doesn't change?

Hi,
Thanks for the library, it works well but I am facing an issue that I hope you can help me with.
I have the datetime picker working on both Android and iOS, but I've just noticed it's not updating the time.
For example, I opened my project at around 10:30 this morning, but now at around 12:40 the time is still showing as 10:30.
If I reload Expo, the time will update, but it will stay the same up until I reload it again.
Any idea why this may be happening, please?
Thanks,

Hi @KemikalGeneral, the time doesn't update "live" because it's evaluated the moment the component is mounted. So if you keep the app in the background, it will stay the same.
Depending on how you're handling your flow, maybe instead of immediately opening the modal when you click on your "show" button (or whatever you're using to opening the modal), in the opening flow you can:

set a variable in the state with the current time
use that variable in the date prop of the picker, which is the initial date selected in the picker
after the variable has been set, show the picker

Either way, this is a bit out of the scope of the project, so it would be better to ask the question on stackoverflow 👍

Hi @mmazzarolo,
Sure, thanks for that.


Popup: is misplaced in scrollable containers

Bug Report
Might have been caused by #3607
Steps

Create Popup which has wider width than its container
Make the container overflow: scroll
Popup is wrongly positioned

Expected Result
Position should be relative to Popup's trigger. Should respect position prop
Actual Result
It seems that the right side of the Popup is positioned to the edge of the container, which pushes the left side further left, out of the trigger position. With overflow undefined it works as expected. See the testcase :)
Version
0.87.2
(0.87.1 works as expected)
Testcase
https://codesandbox.io/s/semantic-ui-react-kxrk7

It's definitely caused by #3607, but a fix provided there was correct. You can change boundariesElement element via popperModifiers:
https://popper.js.org/popper-documentation.html#modifiers..preventOverflow.boundariesElement
<Popup
  popperModifiers={{ preventOverflow: { boundariesElement: "window" } }}
/>

https://codesandbox.io/s/semantic-ui-react-zf6pd
For me it looks like an edge case: positioning was correct only arrow is broken, but we can't fix this on our side 😭

passing boundariesElement fixed the issue for me :)

Can you please consider adding this to the docs?
It do not consider this as an edge case to be honest. Having an overflowing popup within a scroll area is really common. Basic example is a dialog, with a simple popup that is overflowing the content.

but we can't fix this on our side 😭

Have you ever considered, setting this attributesfor popper in case someone explicite sets the mountNode of the popup to the body?
I mean, if I am using the body, I expect the positioning based on the window and not on a scrollcontainer or dialog where the Popup is used in the end.


draft issue #520: provide option to customize stock extensions

This is a draft for #520 to allow for a customization of the default extensions.

refactored out the extension loading code from the PebbleEngine.Builder into the ExtensionRegistryFactory
provide a ExtensionCustomizer which basically allows to access extensions before loading

So I came up with "ExtensionCustomizer" to allow filtering out or change whatever functionality is provided by an extension. It currently only targets the baked in extensions - not sure if it would make much sense on the user provided extensions anyway.
On the one hand it looks like it leaves the configuration API completely intact and doesn't degrade things like allowOverrideCoreOperators to an implementation detail.
On the other hand I'm not too happy with the current version as now other implementation details, e.g. how core functionality is split over extension classes (core, i18n, escaper) kind of leaks out as you have to know the extension class to apply the customizer.
Of course the whole target class thing can be dropped and there could be just one single customizer that is wrapped around every extension before it is loaded ... while thinking about it - i think this is would would provide the cleanest API and won't break things for users if stuff is re-organized internally at some point.. I'll rework it.
Feedback is much appreciated!

Can you provide some documentation for this new functionality ?

@ebussieres Ok, I added further JavaDoc - is that okay so far?

I mean add some documentation for pebble website. Doc is here : https://github.com/PebbleTemplates/pebble/tree/master/docs/src/orchid/resources

Oh sure, I'll check this out!


Selected actions for "swipe right" not performing as intended

Worked well for me until update today (7/28). Now the "Good / Show" action actually functions as "Easy / Show", and the "Easy" action does nothing at all. There may be more broken ones.

Should be fixed for V1 scheduler in the latest update.

Before the recent update, ankimote looked for the text 'good' on a button and then clicked the button. That didn't work for non-english languages, or when people were using addons like 'Button Colors' or 'Auto Ease Factor', so the new update instead directly signaled answer button 3, which is always 'good' with the V2 scheduler but can sometimes be 'easy' with the V1 scheduler.
Has now been fixed to detect when the V1 scheduler is being used in learning/relearning mode.

Please let me know if your issue is resolved.


Make sure ServiceXDataSet is deep-copyable/pickable?

Actually:

The first step is to decide if it should be deep-copyable. If that makes sense.
Second, understand how hard it is to deep copy it.

This closely relates to the question: should you be able to pickle the object? This is especially true since it holds a password.

Until we get a specific usecase, we aren't going to work on this. It comes down to the ServiceXDataset is both a resource and a dataset name. It doesn't make a lot of sense to deepcopy a resource holder object. For now suggested workaround is to implement the __deepcopy__ method in the object that holds onto a ServiceXDataset and have it not deep copy, but just share a reference.


Fix MariaDB indexes renaming

MariaDB does not support renaming index on the latest LTS, we need to drop/recreate them.

Even if MariaDB I don't think it is directly supported by Laravel - it simply works as any drop-in replacement - you are right even in case of MySQL 5.6, which should be supported.
It's seems to me an issue to be addressed directly on framework.
https://github.com/laravel/framework/issues/37161
If my issue will be considered valid, I think that this test will fail.
https://github.com/laravel/framework/blob/0117d797dc1ab64b1f88d4f6b966380ea7def091/tests/Database/DatabaseMySqlSchemaGrammarTest.php#L285-L293
Anyway, IMHO a workaround should be rely on doctrine/dbal Table::renameIndex.

Hi @matteotrubini, thanks to pointed me the laravel issue. According to Dries' comment, this won't be changed into laravel though.
This PR (and others into the winter's plugins) follows an issue reported via Discord.
Also thanks to pointed me the Doctrine method, I'll modify the commit to not re-invent the wheel (and my wheel was weak in front of Doctrine's one).

Even with version 2.0.1 I've got error with winter:up command and MariaDB 10.3.28 :
SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'index `rain
  lab_translate_attributes_model_id_index` to `winter_translate_attr...' at line 1 (SQL: alter table `winter_translate_attributes` rename index `rainlab_translate_attributes_model_id_index` to `winter_trans
  late_attributes_model_id_index`)

@damsfx This is fixed in latest dev-main, there has been no release with this fix yet.

Tough Monday, I hadn't noticed the detail.
Updated composer.json file ... everything works.
Sorry for the inconvenience.


Errorr s1001 streaming with plex

Hello,
i'm really new with iptv streaming in plex.
I've just set up my m3u file as a tuner via Threadfin and when i try to stream any channel with plex i have errorr s1001 (network)
I tried with all Stream Buffer options but nothing seems to change.
I don't know if i have to ask here or on plex forum.
Can anyone give me some help?
Tnx

Have you tried playing one of the urls in VLC? You can open the m3u in a file editor

Yes, i can play in VLC.
And i can edit m3u file with word.
I have this as first lines:
#EXTM3U
#EXTINF:-1 tvg-id="" tvg-name="INFO EVENTI" tvg-logo="http://XXXXXXX.XXXXXXX.XXXX:6923/images/XXXXXXXXXXXXX.jpg" group-title="ITALIA [XXXXXX]",INFO EVENTI
http://XXXXX.XXXXX.XXXX:XXXX/XXXXX/XXXXXXX/XXXXXX

Tnx fot your hints.
Following documentation i changed ffmpeg option in TF
from
-hide_banner -loglevel error -i [URL] -c copy -f mpegts pipe:1
to
-hide_banner -loglevel error -i [URL] -c:a libmp3lame -vcodec copy -f mpegts pipe:1
And now it seems to work.
I have to check if all the channels work.


Made floating point values be printed with all necessary precision to…

… JSON

The tests unfortunately were higher level and can't be committed into soa.  But the code was tested and has been fairly thoroughly used since.  Plus, I used the same solution as Python used for the same problem.
+1


Use File Scheme in FileSystemMasterTest

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.4 please


[main] Format Go code

Cron -knative-prow-robot
/cc knative-extensions/eventing-writers
/assign knative-extensions/eventing-writers
Produced by: knative-extensions/knobots/actions/update-gofmt
Details:

/retest-required

/retest-required

/retest-required

/test reconciler-tests

@Cali0707 I am currently investigating this failed reconciler test, because the kafkasource dispatcher cannot be woke up. It is impacting all the PRs right now.

@Leo6Leo I think @pierDipi is working on a fix here: https://github.com/knative-extensions/eventing-kafka-broker/pull/3423

/retest

Done in https://github.com/knative-extensions/eventing-kafka-broker/pull/3422


Development

[ ] Read the Leaderboard API documentation to learn how to use this API.
[ ] Create a new game with the name of your choice by using the API.
[ ] Implement the "Refresh" button (receiving data from the API and parsing the JSON).
[ ] Implement the form "Submit" button (sending data to the API).
[ ] Use async and await JavaScript features to consume the API.
[ ] No styling is required.

Hi @medaminedev66,
Thank you for the review, I have read the documentation attached to the Git flow, and I followed the Git Flow but there is a difference in the naming I think you expected.

The dev_leaderboard is a branch to the main, which you expected to be named the development branch.
The development branch is a branch to the dev_leaderboard, which you expected to be named the feature branch.
In this case, the created pull request is comparing the development branch ( feature ) with the dev_leaderboard branch ( development ).


PMC deps.json error

Add-Migration InitialCreate  Update-Database
When running the above PMC command on a project with a UNC path, I get the "The specified deps.json [..] does not exist" error. The deps.json file does exist in the bin folder, but the project is unable to locate it. I moved the project to various UNC locations and received the same error each time.
When I moved the project to a non-UNC path, the PMC command executed successfully. The project worked in the UNC location on each tutorial step up to this point.
The image on the first page of the tutorial shows the project created in the C:\ drive, but it does not specifically say that is required. Are there some setups missing on step 1 to configure this project to work with UNC paths?

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

ID: 1d975a83-1550-b284-d0d8-7fe2ea206ff5
Version Independent ID: 48e81b42-7c2b-afbf-71e6-d4f442a4bd1f
Content: Part 4, add a model to an ASP.NET Core MVC app
Content Source: aspnetcore/tutorials/first-mvc-app/adding-model.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

non-UNC paths are beyond the scope of this tutorial.


update the document for semicolon in TypeSpec-Project-Generate.ps1

Despite the document says we could use semicolon to separate multiple options in this parameter, but it will not work if we just do this:
dotnet build /t:GenerateCode /p:typespecAdditionalOptions="csharpGeneratorPath=./autorest.csharp/artifacts/bin/AutoRest.CSharp/Debug/net6.0/AutoRest.CSharp.dll;debug=true"

which eventually calls the TypeSpec-Project-Generate.ps1 script.
The option after the semicolon is somehow discarded. But if we escape this semicolon, everything works fine
dotnet build /t:GenerateCode /p:typespecAdditionalOptions="csharpGeneratorPath=./autorest.csharp/artifacts/bin/AutoRest.CSharp/Debug/net6.0/AutoRest.CSharp.dll%3Bdebug=true"

Fixes https://github.com/Azure/autorest.csharp/issues/3362

@m-nash @benbp @weshaggard  could you also take a look?
I wonder if we have other options or other places to fix to let this work as is instead of escaping this character.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

Close this PR since the issue only happens when we are using msbuild
I will open another PR to update document somewhere else


Cannot convert condition in circuit with multiple classical registers to instruction

Information

Qiskit Terra version:
Python version:
Operating system:

What is the current behavior?
Appending a circuit with multiple conditional registers to an instruction raises an exception (from to_instruction method), however using compose instead of append seems to work.
A related issue is that the "working" circuit from compose assembles to incorrect qobj (see issue #6583)
Steps to reproduce the problem
Take basic teleport circuit as example
def teleport_circuit():
    """Teleport qubit_in to qubit_out"""
    # It would be nice to be able to do this without using
    # register objects

    # Teleport qubit 0 -> 2
    qr = QuantumRegister(3)
    c0 = ClassicalRegister(1)
    c1 = ClassicalRegister(1)
    teleport = QuantumCircuit(qr, c0, c1)
    teleport.h(qr[1])
    teleport.cx(qr[1], qr[2])
    teleport.cx(qr[0], qr[1])
    teleport.h(qr[0])
    teleport.measure(qr[0], c0[0])
    teleport.measure(qr[1], c1[0])
    teleport.z(qr[2]).c_if(c0, 1)
    teleport.x(qr[2]).c_if(c1, 1)

Appending this to another circuit with
qc = QuantumCircuit(3, 2)
qc.append(teleport_circuit(), range(3), range(2))

raises
~/git/qiskit/qiskit-terra/qiskit/converters/circuit_to_instruction.py in circuit_to_instruction(circuit, parameter_map, equivalence_library)
    116             else:
    117                 raise QiskitError(
--> 118                     "Cannot convert condition in circuit with "
    119                     "multiple classical registers to instruction"
    120                 )

QiskitError: 'Cannot convert condition in circuit with multiple classical registers to instruction'

This error seems to come directly from teleport_circuit().to_instruction() which gets called by append.
If you do compose it produces a circuit, however the conditional values appear incorrect on the drawer and in the assembled qobj (see #6583)
qc = QuantumCircuit(3, 3)
qc = qc.compose(teleport_circuit(), range(3), range(2))
print(qc.draw())

                    ┌───┐┌─┐
q_0: ────────────■──┤ H ├┤M├──────────────
     ┌───┐     ┌─┴─┐└┬─┬┘└╥┘
q_1: ┤ H ├──■──┤ X ├─┤M├──╫───────────────
     └───┘┌─┴─┐└───┘ └╥┘  ║  ┌───┐  ┌───┐
q_2: ─────┤ X ├───────╫───╫──┤ Z ├──┤ X ├─
          └───┘       ║   ║  └─╥─┘  └─╥─┘
                      ║   ║ ┌──╨──┐┌──╨──┐
c: 3/═════════════════╩═══╩═╡ = 1 ╞╡ = 2 ╞
                      1   0 └─────┘└─────┘

What is the expected behavior?
to_instruction should work for any circuit by flatting registers.
Suggested solutions
The implementation in compose seems to work, so maybe how it is implemented could be used for building the flattened circuit definition in the instruction-ified circuit.

For now, I think raising is the right behavior here. Instruction conditions can span a full ClassicalRegister or, after #6018, individual Clbits, but not yet slices of registers. One possible short term workaround would be to repeat the conditional instruction, conditioned over the possible states of the remaining bits of the flat register, in sort of an 2^N-way OR, but this doesn't scale especially well.

@kdk In that case compose should also raise an error rather than appear to work (see the other issue for it returning incorrect result)


[locale] Update bg.js

Fixed cardinal noun. Sigh…

@ichernev, will you take a look at this change please?

Coverage decreased (-0.01%) to 88.398% when pulling 55a961d4443b7d1565d7df6643b79aa5c8506985 on StoyanDimitrov:patch-1 into 13a61b285c095bda7ea8e33156090ea5ccfeaef1 on moment:develop.

Well, I managed to make pretty nice little mess here. @kraz, can you fix some of it for me please?

Looks good, just remove the locale/bg.js from this change (it is autogenerated).

Merged in 01e59f213b1c82d274f40ba10cb01274795d6183


Older algorithms in periodsearch

There are several period-finding algorithms in periodsearch.py that rely on cuvarbase or gcex imports. Should these older algorithms be removed since we prefer to use periodfind algorithms, or is it ok for this code to remain?

@bfhealy I suggest to remove these.


Bug:Issue on Search page

Has this bug been raised before?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Where did you find this bug?
production
Version of BioDrop (for example "v1.2.3")
v2.83.8
Description
These two search values don't match.Please see the attached snapshot.
Screenshots

Do you want to work on this issue?
Yes
If you want to work on this issue...
No response

Thank you for raising. I noticed that you want to work on this issue, but again you have not explained how you would go about this.  In future if you want to work on the issue please complete the if you want to work on this issue section.  If you are unsure of how to approach it, then we will open the question to other contributors and will assign based on that.
On this occasion I will assign this to you and here is what you need to do...
The discrepancy relates to:

badges result: has no filter (for example, includes tags from every Profile)
search result: has filters applied (for example isEnabled: true)

The source files to look at are:

tag api file pages/api/discover/tags.js
search for get profiles pages/api/profiles/index.js

hi @sumitunique777, If you do not resolve this issue?  I want to try to solve this issue.
would you assign to me? @SaraJaoude
I am new in open source. if any guidance for me. Please tell me it's very helpful for me

hi sumitunique777, If you do not resolve this issue? I want to try to solve this issue. would you assign to me? SaraJaoude
I am new in open source. if any guidance for me. Please tell me it's very helpful for me

@aarjun8060 this issue was assigned to @sumitunique777 3 days ago - it is not very collaborative to chase them to resolve the issue, especially as we give our contributors two weeks to start work.
As you are new to open source and looking for guidance, I would suggest that you start out by reading this project's Contributing Guide.  This will explain how this project is run.

The results are coming incorrect because the filters return the record even if there is not any tags list available by comparing the search string with username and another fields

Will you please suggest some ideas how can I approach this issue?

Will you please suggest some ideas how can I approach this issue?

@sumitappinfoinc thank you for your interest but this issue is assigned to someone else. Please feel free to collaborate once a PR is raised.

Great discussion, but I am going to close this, because the search results does not just search the tags but also other profile fields - that is why the numbers do not match


FlowTappedBackgroundColor is not working

i have set the FlowTappedBackgroundColor to red color, but when I tapped my item, the Background color seems to be remain the same default color. Any idea?

Same for me 😢

You have to remove default os rows highlighting first: https://github.com/daniel-luberda/DLToolkit.Forms.Controls/tree/master/FlowListView#how-can-i-disable-entire-row-highlighting-when-tapped


Fix repo name used when uploading jars

Coverage remained the same at 100.0% when pulling 211dee3d1ed73f1d70863da808b3f63d30c70a40 on fix_repo into c0e91c6f7fb23ed1fffda21e7e5bc0ed34c665c7 on main.


Feature Request: some way to pass hyperparameters out of transforms

Do you want to hand tune your models? Most folks don’t because it’s slow. So we get into hyperparameter optimization.
In Haiku, we define a model like
def forward(x):
   return MyModule(hyperparameters)(x)

model = hk.transform(forward)

Where hyperparameters is a dict or class to record decisions modules make when they “pull” choices like “number of layers” “latent code dimensionality” “block 2 n_heads” ... we cannot enumerate all possible hyperparameters ahead of time because certain ones depend on the value of earlier ones and there are just too many combinations
Those hyperparameters become “stuck” inside the forward function, unless we make hyperparameters a mutable datastructure and mutate it inside all our MyModule.init methods. Then, the forward function has deeply nested side effects when defined, so you cannot share the decisions across a group of agents because they will mutate shared state, and the haiku tracer needs to know to ignore these init hyperparameter mutations, which I’m assuming it does but not 100% sure. Mutating hyperparameters at definition time is also confusing for engineers because there’s many “init” steps at different times
How can we make an elegant way for “forward” methods to pull hyperparameters and return their decisions?

Hey, this seems like a good use case for creating a custom hooks, in Elegy we created a custom elegy.transform to enable custom hooks like elegy.add_loss and elegy.add_metric. I imagine you could create an API like this:
def forward(x):
    a = get_hyperparameter("a")
    b = get_hyperparameter("b")
    return MyModule(a, b)(x)

model = custom_transform(forward)

hyperparameters = dict(a=1, b=2)

model.init(..., hyperparameters)
model.apply(..., hyperparameters, ...)

But creating you own transform is not that straight forward, you can check our implementation here.
Now that you mention it I think this would be a good feature for Elegy. I it would also be nice if we could make a transform general enough so users could create their own hooks.


added froque as maintainer to powershell plugin

Description
The link to plugin repository: https://github.com/jenkinsci/powershell-plugin
pinging @chrisalbrecht for confirmation.

Submitter checklist for changing permissions

Always

[x] Add link to plugin/component Git repository in description above

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

Merge permission to GitHub repository

[x] Check this if newly added person also needs to be given merge permission to the GitHub repo.

@froque all good!

@slide anything else required ?

@chrisalbrecht @slide I don't believe I have commit access to https://github.com/jenkinsci/powershell-plugin repository

Fixed


[AnVIL Catalog] Build AnVIL Catalog off of the DUOs API instead of Terra workspaces

Need
Terra ingest/data workspaces in GCP are being retired. As an alternate source of the workspace related information we can use the  DUOs API instead. We may still need the dbGaP API for some of the fields.
AnVIL Catalog Mapping
https://docs.google.com/document/d/1HxPhJlzhYfoLTaeLZXJsc_RQN7DAibq3KtwNkPSHOI0/edit#heading=h.zgake2jchq9a
DUOS APP Home Page
https://duos.broadinstitute.org/home
Current AnVIL Catalog Script
https://anvil.terra.bio/#workspaces/anvil-datastorage/Clever_Canary_AnVIL_Dataset_Catalog_Dev/analysis/launch/Copy of Create Dataset Catalog Report From Terra APIs.ipynb
Approach

[ ] Review DUOs API and determine what additional DUOs API fields would be needed to match the current dataset catalog display.
[ ] Determine how to request anvil-only studies and see study metadata from DUOS.
[ ] Review DUOs Auth capabilities, login with dev account and make requests to get the list of anvil datasets and the dataset metadata.
[ ] Build the current consortia, study, and workspace lists
[ ] Run the Catalog locally off of the new lists.
[ ] Determine deployment architecture to allow for the catalog to refresh as DUOS is updated.

Completed.


Clear structure for koa.js app

I find very interesting document saying how we should structure the application for node.js app based. Have a look.
https://github.com/focusaurus/express_code_structure

@hung-phan your change on master on refactor the middlewares looks good. I think we can improve this part https://github.com/hung-phan/all-hail-the-R/blob/master/config/initializers/middlewares.js#L88
  applyMiddlewares(app, middlewares.bodyParser, settings.bodyParser);
  applyMiddlewares(app, middlewares.logger, settings.logger);
  applyMiddlewares(app, middlewares.compress, settings.compress);
  applyMiddlewares(app, middlewares.conditional, settings.conditional);

Should we use a for here? Given that the order of the middleware matters a lot, maybe we should change the middleware settings to an array of object.

:+1: make sense. Let me do it


Export transpiled components

This will allow projects to leverage components like SidebarLayout if desired.

The only reason I didn't do this was because within x0, the src directory is included in babel loader, but this change would make the components work wherever

Going to close for the time being since it appears to be an issue with global v local x0 installs.


Add R.lensFind

The intention here is a mash up of R.find and R.lens.
Questions I asked myself before submitting this:

Is the same effect straightforward by composing existing functions? Not really.
Has someone already proposed a function like this? Not that I can see. However, I found out after fact that this S.O. question is answered with a great lens written by Scott Christopher, which only tells me:

a). Other people probably need to do this sort of thing.
b). Scott already wrote a lensMatching function. Did he or someone else already consider adding it? Maybe I can close this and he can propose it? :)

Ugh, spacing issue. Will resubmit.


operator: add node selection constraint (amd64 arch)

In order to make controllers consistent, I add a nodeselector constraint of daemonset to dlb, fpga, qat too.
Signed-off-by: Hyeongju Johannes Lee<EMAIL_ADDRESS>

@hj-johannes-lee looks good to me. One thing though: can we avoid code duplication by creating a function and calling it from all plugins?

@bart0sh I see.! But, where would the function have to be?

where would the function have to be?

Anywhere where it can be imported from I guess. pkg/controllers would be a good place in my opinion.

BTW, there are other candidates that would reduce code duplication. Some of them are even already wrapped into the functions, for example setInitContainer: 1, 2, 3, addVolumeIfMissing etc.

Naming is too difficult, please suggest more proper names for the file and function.

@hj-johannes-lee It looks much better now!

Codecov Report

Merging #764 (8272ea7) into main (ef354f7) will increase coverage by 0.22%.
 Consider uploading reports for the commit 0a71e25 to get more accurate results

@@            Coverage Diff             @@
##             main     #764      +/-   ##
==========================================
+ Coverage   54.59%   54.82%   +0.22%
==========================================
  Files          39       39
  Lines        3515     3473      -42
==========================================
- Hits         1919     1904      -15
+ Misses       1489     1465      -24
+ Partials      107      104       -3

Impacted Files
Coverage Δ

pkg/controllers/dlb/controller.go
19.64% <ø> (ø)

pkg/controllers/dsa/controller.go
11.11% <ø> (-2.83%)
:arrow_down:

pkg/controllers/fpga/controller.go
19.04% <ø> (ø)

pkg/controllers/gpu/controller.go
36.72% <ø> (+0.07%)
:arrow_up:

pkg/controllers/qat/controller.go
22.78% <ø> (ø)

pkg/controllers/sgx/controller.go
13.68% <ø> (-2.83%)
:arrow_down:

Continue to review full report at Codecov.

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

lgtm except of couple of nits I was commented on

@bart0sh I commented two times that I do not understand why making const one line instead of 3 lines is related to this pr, but I think you couldn't see them. So, I just deleted my comments and did as you said.!

why making const one line instead of 3 lines is related to this pr,

As far as I can see in 2 out of three cases you are making changes (removing amd64 const) that affect the whole block (which justifies moving to a single line const. That remaining one case is ok to be taken in this pr to make it consistent with others.

@mythi
My point was.. that I do not know why there has to be another pr for removing a space to make all plugins look the same.. but there is no need for the matter of changing 3 lines into 1 line... Including only necessary changes in a single pr is, to me, too difficult.. :(

@mythi My point was.. that I do not know why there has to be another pr for removing a space to make all plugins look the same.. but there is no need for the matter of changing 3 lines into 1 line... Including only necessary changes in a single pr is, to me, too difficult.. :(

you can move cosmetic changes into the separate commit I guess. Mixing them into the 'main' commit(s) make reviewing (and therefore merging) harder. This is, btw, exactly what we have here.

@hj-johannes-lee please resolve conflict in DSA controller and this PR is good to go from my point of view.


Speaker Post Needed - GDG Cloud Gandhinagar  16 Sept 2023

Goal: To Create the Poster For GDG Cloud Gandhinagar Speaker post
[What you'd like created]
Info: Speaker1
Name: Mehul Patel

Speaker2
Name: Kaivaliya Shah

[What it will be used for/where it will go]
Style Information: Please use the standard format
[What you want it to look like, link to references, similar things etc.]
Deadline: Friday 15th Sept'23.
[When would you like this done by]
Tag: Design Needed
[Design Needed, Developer Needed, Question, Support Needed]

Closing this one!


(likely good first issue) Chats should have an emoji selector

Is your feature request related to a problem? Please describe.
I like to send DMs (chats) that sometimes include emoji. Sometimes I do this from the Web client. It would be nice to add them directly.
Describe the solution you'd like
An emoji selector similar to the one provided by the New Post interface or something else at the level of convenience of an emoji keyboard on a mobile app.
Describe alternatives you've considered
Copying and pasting emoji from other text interfaces that supply an emoji keyboard or selector.
Additional context
Not really much else to say. On mobile this is easy to do relying on the platform emoji selector in the keyboard presumably, but the web doesn't have this.

Resolved via #5091


Dockerfile: S6 updated, duplication removed

S6 updated to v<IP_ADDRESS>
S6 verified via automatic gpg signature, instead of manually-generated sha256
S6 and Goss installed from dedicated scripts, removed obvious Dockerfile duplication

Unrelated:

Ubuntu 17.10 removed, Canonical pulled support July 2018

Remove - docker build -f Dockerfile-ubuntu-17.10 . from .travis.yml

LGTM 👍


Sentinel Root Fix

Description

closes: #XXXX

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #234 (a8216a5) into main (f4484c9) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #234   +/-   ##
=======================================
  Coverage   79.48%   79.48%
=======================================
  Files         115      115
  Lines        6834     6834
=======================================
  Hits         5432     5432
  Misses       1024     1024
  Partials      378      378

Impacted Files
Coverage Δ

...ght-clients/07-tendermint/types/consensus_state.go
100.00% <ø> (ø)

...dules/light-clients/07-tendermint/types/upgrade.go
72.88% <100.00%> (ø)

This should be backported


Feature: loop

This PR proposes a loop option which enables animation looping on a set of keyframes and a start delay option to delay the start of an animation.
Both of these options are set on the velocity directive:

And can also be initialised via the velocityGroup directive:
<sn-velocity-group data-loop="true" data-keyframes="{ '#elem1': { 'keyframes': [{ 'properties': { 'opacity': '1' }, 'options': { 'duration': '1000' } }], 'startDelay': '200' } }">

This requires a new format for the keyframes object to allow each animation element in the group to have it's own start delay. This does not break the old format, this directive now accepts either format.

looks good. :+1:


cmd/go: Proposal: Modules without releases have the newest code fetched when go get -u is ran.

Currently modules that have no releases do not fetch the latest code, even when go get -u is run.  Essentially there is a user-hostile edge-case in go get workflow:
If the dependency has releases, then use go get -u, else use go get module/path@master
The code that was present on the first go get of the module is still used until a new release is cut and go get -u is run.  It is possible to manually update the module without a release by doing go get module/path@master, but I think it would make sense to see this behavior be the default for modules without releases.
Steps to reproduce

Create a repository and init a module for it
Do not cut a release of this repository
Use unreleased module and repository as a dependency in another program
Update the unreleased module's code
Try to go get -u the unreleased module from your other program
Notice that the build does not contain the newest code from the unreleased module

tag this with modules and Proposal maybe?

If I understand correctly, this is already the intended behavior.  If there are no tagged versions (or if all tagged versions are excluded), it will find the commit that HEAD points to, and it will encode that as a pseudo-version. If HEAD points to something different than before, you will get a new pseudo-version.
If you're using a proxy (for example, if you're using go1.13beta1, GOPROXY is set to https://proxy.golang.org,direct by default), you may not see new commits right away. The <module>/@latest endpoint is cached (I think the TTL is roughly an hour). So temporarily setting GOPROXY=direct or running go get example.com/mod@<commit> is the right thing to do there. #32870 discusses this.
Note that go get -u is changing behavior in go1.13, so if you're proposing a change in behavior in go get -u specifically, not go get <module>@latest, please explain the difference. In go1.13, go get -u will upgrade modules providing packages transitively imported by the package in the current directory.

Thanks for your reply jayconrod.  I am not suggesting a change to go get <module>@latest.  That functionality works as intended.
I think the existing behavior is expected and works well in all circumstances where module releases exist, but not when the module maintainer hasn't cut a release yet (effectively been running the module as version 0.0.0).  In that case, the consumer of the module will unexpectedly not get the newest code in the repository through go get -u.  Instead, they will get the exact same code they already have without a warning of any kind.  They need to somehow know that the upstream module hasn't tagged any releases, so they need to run a different command entirely (go get example.com/mod@latest).
I am proposing that, in the case of there not being any releases for a module, go get -u will always effectively run go get example.com/mod@latest.
If the maintainer has not specified a stable release of their module (0.0.0), we should assume that things are still changing rapidly and without any guaranteed stable release.  Therefore, it should be sensible to always fetch the newest code in the repository when go get -u example.com/mod is ran.
If we do not adopt this behavior, then developers have to know the hidden logic I described before:

If the dependency has releases, then use go get -u, else use go get module/path@master

I am proposing that, in the case of there not being any releases for a module, go get -u example.com/mod will always effectively run go get example.com/mod@latest

This is what go get -u does though. go get -u pkg upgrades modules that provide packages transitively imported by the package pkg. For these modules, it is as if you ran go get example.com/dependency@upgrade. The @upgrade suffix is like @latest except if the current version is newer than the "latest" version (for example, if the current version is a pre-release or a chronologically newer pseudo-version), the module won't be upgraded.

I just simulated this using a couple repos.  I made a repo with module and a repo with an app that uses it.
I did the following with these two repos:

Push a version of the module that says First commit
Build my test-app binary for the first time
Note that the following go mod was created:

module github.com/integrii/test-app

go 1.13

require github.com/integrii/go-module-test v0.0.0-20190820213515-828ef694dc70

When using the app, it of course prints First commit
I then changed the master branch print statement to say Second commit
I then ran go get -u in my test-app repository
The go.mod file for test-app remained the same
The test-app binary still produced First commit when rebuilt

In order to actually get the Second commit string to build into test-app, I must run go get -u github.com/integrii/go-module-test@update (or @master or @latest).
Then, I found out today that by waiting for an hour or two, then re-running go get -u, I got more messaging about downloading and extracting files where previously I only had finding.
go: finding github.com...
go: downloading github.com...
go: extracting github.com...

Perhaps the behavior I am seeing (that you say already exists) only exists after some time has passed.  My organization is not running any kind of module proxy and I have no environment variables configured to change the default proxy settings.
So my ask remains the same, but with the additional qualifier that reproducing this requires you to act relatively quickly between making updates to an un-tagged module and running go get -u.

Thanks for posting this example. From your description, I believe this is the same as #32870, so I'll close this issue.
As mentioned above, go1.13 uses a module proxy by default. If you have no other setting, you should see that go env GOPROXY prints https://proxy.golang.org,direct, which means that the Go command will try to fetch the module from the Go module mirror first, and if that fails, it will attempt to access the origin repository directly.
You can work around this issue by setting GOPROXY=direct when running go get commands, or, more narrowly, by setting GONOPROXY=github.com/integrii/test-app.

Thanks @jayconrod !  Having this issue de-duped is valuable and I appreciate your careful consideration.


Show a blue background when launch/exit game in steamVR Home everytime.

When I launch game in SteamVR Home, I saw a blue background everytime. It's only show like one second before disappear.
Description

ScreenShot:

Hardware
Quest 2
Installation
ALVR Version: Lastest version
SteamVR Version: 1.24.7

Does the game launch at all? Did you try other games?

Already sloved by myself, close this issue.


Simpler Jekyll searching

Comment thread for post Simpler Jekyll searching.
Comments made before 18 February 2021 were imported from Disqus using an import script. The datetimes and authors of the imported comments do not reflect the values of the original comments.

Comment № 1 originally by wileywiggins at 2018-07-07 03:55:03 UTC.
This is working for me up to the point of the script hiding the other categories on the search page. You don't seem to be using this on your site so I couldn't really get a look at it working properly. My search page is just search.html in the root of my jekyll directory, so do I need to put anything in the SEARCH_PATH variable? If I make it "search.html" The script errors out at line 47 with "TypeError: undefined is not an object (evaluating 'tagsContainer.style')".
Thanks for posting this, the liquid bit to write out the search page is useful either way.

Comment № 2 originally by Alex Pearce at 2018-07-10 12:02:41 UTC, replying to a comment by wileywiggins at 2018-07-07 03:55:03 UTC.
I am still using this technique, e.g. searching for the JavaScript tag.
I also have search.html in the root of my repository. So, as your setup sounds so familiar to mine, you should be able to copy-paste the JavaScript I'm using.
For your error, it sounds like tagsContainer is undefined. This object is defined as tagsContainer = document.querySelector('.tag-index'), so chances are you don't have a DOM element with the required class name, so the querySelector call evaluates to undefined.

Comment № 3 originally by wileywiggins at 2018-07-11 22:20:50 UTC, replying to a comment by Alex Pearce at 2018-07-10 12:02:41 UTC.
Ah, it's working now. I needed enclosing divs with .tag-index and .category-index classes. You might include one in your html snippit if you have the time. Thanks for your help!

Comment № 4 originally by Alex Pearce at 2018-07-13 06:35:51 UTC, replying to a comment by wileywiggins at 2018-07-11 22:20:50 UTC.
Ah, good spot, thanks! Updated.

Comment № 5 originally by Madhusoodan Pataki at 2020-09-14 12:06:25 UTC.
Came here to check how to implement the search page. This is exactly how I did for my archive page. But had to write a little javascript
https://mmpataki.github.io/...
Only thing missing here is, search by just words (not any tags or categories)


Improve discover(): perform discovery on all NICs/IPs

This PR changes the way that discover() determines the IP addresses to use for multicast discovery. It replaces the use of gethostbyname() with a comprehensive list of IPv4 addresses obtained using the ifaddr library. This should cover cases where the host has multiple NICs and/or multiple IP addresses associated with those NICs, and prevent some of the common problems encountered when using discovery.
Addresses #764.

Coverage increased (+0.1%) to 57.559% when pulling 50adb13da33d74f48f10c2ec6bc4610c141d4a6f on pwt:discover_multi_IP into 784d95b42910e2634d7826ae0df16f461d8a8ea9 on SoCo:master.

Works fine on my setup with multiple addresses on an interface. Probably don't need the None default on create_socket any longer.
def create_socket(interface_addr=None):

My PC just have a single interface, so I don't how much it is worth, but I can confirm that discovery on the S1 system still works with this PR.

My PC just have a single interface, so I don't how much it is worth, but I can confirm that discovery on the S1 system still works with this PR.

Thanks for testing, @KennethNielsen. It's now been tested on multihomed and aliased systems, and on Windows, Linux, and macOS. Not seeing any issues (and it will certainly address some existing limitations).


Query pdf files instead of images

Feature request  / 功能建议
Dear CogVLM's authors,
Thank you for your outstanding work on MLLM.
In the demo, we can only query pictures. Is it possible to make the model process pdf files?
Motivation / 动机
I want to implement the model in a personal project I'm currently working on
Your contribution / 您的贡献
Thank you

pip install pdf2image


Publish to PyPA

It would be much more convenient to install this library with pip install.

Hi. Yep, will do next week

Currently, as solution you can directly install you can install:
python3 -m pip install git+https://github.com/seriyps/ru_number_to_text

Currently, as solution you can directly install like: python3 -m pip install git+https://github.com/seriyps/ru_number_to_text

мне выдало просто Python

Strange, exactly in windows this does not working. Same result.
Try next, tested now on Win10 with python3 + git installed:
pip3 install "git+https://github.com/seriyps/ru_number_to_text"
Without git, I think it possible just unpack repo locally and run inside:
pip3 install .


I'm sorry, but...

Hello. Can I use SVG library inside Node.js engine? I want use vector graphics in my multi-player game.

Please change the title to something useful.
For your question look into #464.
Short answer: it currently doesn't work but we age working on it. However it won't come soon.


apiVersion on SCC for lab 6 + refresh warning in lab 6.3

I've changed the apiVersion attribute becasue the raw file gave me errores when trying to import it on opc 4.

LGTM


Update README.md to remove stale install instructions, instead pointing to os_setup.md.

Change README.md to point to install instructions, since the install instructions were stale (mentioned 0.5.0), and are likely to continue to be stale with respect to os_setup.md.

LGTM


[App] Mobilum Swap Error When Fetching Prices

Lots of users have been reporting an issue with Mobilum. This should be treated as high priority since it's preventing less technical US users from buying Klima.
I found that the issue only occurs when selecting "Swap Klima". It looks like there's an issue in the widget when it tries to fetch the Klima price in USDC.
"Swap Klima" selected:

"Swap Klima" not selected:

@ShimonD-KlimaDAO mobilum team says this is fixed now. Can you confirm on your end?

@Atmosfearful this is now resolved, thanks. Closing.


Change Containerisation

CS++ containerises and deploys our systems in a different way to Redbrick.
Most notably, we reverse proxy to a container, instead of hosting it in (or bootstrapping it with) NGINX or other proxies.

Keeping it as it is, really smart way of deployment by Jed!


Improve DNSResolver log

We are using Tuweni in Besu, and some users find the following log line misleading
2022-02-22 16:44:04.285+00:00 | Timer-0 | INFO  | DNSResolver | Resolved 0 nodes

since they think that something went wrong with the DNS resolver, while everything is fine, since what is actually happening, is that there are no new nodes to resolve, since the sequence is not increased, looking at the debug log:
{"timestamp":"2022-02-22T17:47:29,325","level":"DEBUG","thread":"Timer-0","class":"DNSResolver","message":"ENR tree root seq enrtree-root:v1 e=ZPZNBATGXWZOFW33QLGJWTVHM4 l=FDXN3SN67NA5DKA4J2GOK7BVQI seq=2941 sig=6URRq9MWZL_q9W2zasLUPL21Lw7ND5LAjR-X9pEpLVtkURllhjbRAmdTNDHo20t1bOfAgf2V88Kj4xHGEjBeTAE=.seq is not higher than 2941, aborting","throwable":""}

A proposal to improve this log is to not log when nodes is empty, or when the sequence is not increased.

Sounds good - please send a patch.


Correct syntac for go2rtc camera without camera.entity "Could not determine suitable engine"

Checklist:

[X ] I updated to the latest version available
[X ] I cleared the cache of my browser

Release with the issue:  6.0.beta5
Last working release (if known):
Browser and Operating System: Chrome
Description of problem:
Hi; im testing your frigate card beta 5, it allows to use go2rtc camera streams, but i'm not sure if the correct syntax is right? I dont have an camera entity in HA, i want to use the stream directly, but i get the error in below screenshot
Testing the stream directly in go2rtc does work

Frigate card diagnostic information
        - type: custom:frigate-card
          cameras:
            - live_provider: go2rtc
              go2rtc:
                # modes:
                  # - mse
                stream: tuya_pepe_sale

Additional information:

tried also like below, but gives a black screen
        - type: custom:frigate-card
          cameras:
            - go2rtc:
                url: https://xxx
                stream: tuya_pepe_sale

You are telling the card what live video provider to use, but not what kind of camera to use ("camera engine"). Normally, you'd give it an entity and it can figure it out from that, if you don't give an entity you need it extra information. Since you're using go2rtc, I assume this is a Frigate camera? Try something like this:
type: custom:frigate-card
cameras:
  - engine: frigate
    frigate:
      camera_name: office
    go2rtc:
      stream: office

Relevant documentation: https://card.camera/#/configuration/cameras/README

Hi, thnx for the info, but it's not a frigate camera...
The stream is defined in go2rtc.yaml itself, nowhere else....

Maybe I can use as engine: "auto" or maybe engine: "go2rtc" ?

Try: generic . Not at a computer now, if that doesn't work I'll take a look tomorrow!

Ok, will try that on Monday, I'm a weekend away, I keep you posted ;-)
Thnx

Aha ,  cool, going to test this on monday, appreciate the feedback!!!

tested and working, thnx!!


Minify assets

I was checking if the CSS and JS files are minified in production by looking at the CSC209 instance and it seems like none of them are, although they're supposed to be when I look at production.rb. Is there something up with this?
EDIT: actually, are those instances running the equivalent of the 1.0.x branch?

I doubt this is happening at UofT any time soon.


Update how-to-connect-group-writeback.md

Hello, i added "V1" to the docs to differentiate between this and the Group WriteBack V2 that is in private preview. This document has requirements that no longer apply in V2.

@TosinlufPM : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Undefined reference during build of reason-skia (esy bootstrap, windows)

Hey there, after attempting to build the project from source (i have all of the dependencies installed) on Windows 10, i get this error when running esy bootstrap :
error: build failed with exit code: 1
  build log:
    # esy-build-package: building: reason-skia@github:revery-ui/reason-skia#6b459c7
    # esy-build-package: pwd: C:\Users\Issy\.esy\source\i\reason_skia__f0ccb717
    # esy-build-package: running: "refmterr" "dune" "build" "-p" "skia"
    x86_64-w64-mingw32-gcc src/wrapped/c/c_stubs.o
    cc1: warning: command line option '-std=c++14' is valid for C++/ObjC++ but not for C
    c_stubs.c: In function 'reason_skia_make_sdl2_interface':
    c_stubs.c:30:47: warning: passing argument 2 of 'gr_glinterface_assemble_gl_interface' from incompatible pointer type [-Wincompatible-pointer-types]
           gr_glinterface_assemble_gl_interface(0, reason_skia_sdl2_get);
                                                   ^~~~~~~~~~~~~~~~~~~~
    In file included from c_stubs.h:1:0,
                     from c_stubs.c:6:
    C:/Users/Issy/.esy/3_/i/esy_skia-7ed409d0/include/c/gr_context.h:40:34: note: expected 'gr_gl_get_proc {aka void (* (*)(void *, const char *))(void)}' but argument is of type 'void * (*)(void *, const char *)'
     SK_C_API const gr_glinterface_t* gr_glinterface_assemble_gl_interface(void* ctx, gr_gl_get_proc get);
                                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    c_stubs.c:30:7: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
           gr_glinterface_assemble_gl_interface(0, reason_skia_sdl2_get);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
          ocamlc src/wrapped/lib/raw_bindings.o
    cc1: warning: command line option '-std=c++14' is valid for C++/ObjC++ but not for C
    x86_64-w64-mingw32-gcc src/wrapped/c/dllskia_wrapped_c_stubs.dll (exit 1)
    (cd C:/Users/Issy/.esy/3/b/reason_skia-77127284\default/src/wrapped/c && C:\Users\Issy\AppData\Roaming\npm\node_modules\esy\node_modules\esy-bash\.cygwin\bin\x86_64-w64-mingw32-gcc.exe -O -mms-bitfields -shared -o dllskia_wrapped_c_stubs.dll c_stubs.o -luser32 -lskia -lSDL2 -lstdc++ -LC:/Users/Issy/.esy/3_/i/esy_sdl2-2.0.10006-d4b1b631/lib -LC:/Users/Issy/.esy/3_/i/esy_skia-7ed409d0/lib)
    c_stubs.o:c_stubs.c:(.text+0x42): undefined reference to `sk_canvas_draw_rect'
    c_stubs.o:c_stubs.c:(.text+0x5d): undefined reference to `gr_glinterface_assemble_gl_interface'
    collect2: error: ld returned 1 exit status
    x86_64-w64-mingw32-gcc src/wrapped/stubgen/ml_types_stubgen.exe
    cc1: warning: command line option '-std=c++14' is valid for C++/ObjC++ but not for C
          ocamlc src/wrapped/bindings/.SkiaWrappedBindings.objs/byte/skiaWrappedBindings__Skia_generated_type_stubs.{cmi,cmo,cmt}
    File "src/wrapped/bindings/skia_generated_type_stubs.ml", line 149, characters 4-78:
    Warning 11: this match case is unused.
        ocamlopt src/wrapped/bindings/.SkiaWrappedBindings.objs/native/skiaWrappedBindings__Skia_generated_type_stubs.{cmx,o}
    File "src/wrapped/bindings/skia_generated_type_stubs.ml", line 149, characters 4-78:
    Warning 11: this match case is unused.
          ocamlc src/wrapped/lib/.SkiaWrapped.objs/byte/skiaWrapped__Skia_generated_stubs.{cmi,cmo,cmt}
    File "src/wrapped/lib/skia_generated_stubs.ml", lines 1307-1308, characters 2-26:
    Warning 11: this match case is unused.
        ocamlopt src/wrapped/lib/.SkiaWrapped.objs/native/skiaWrapped__Skia_generated_stubs.{cmx,o}
    File "src/wrapped/lib/skia_generated_stubs.ml", lines 1307-1308, characters 2-26:
    Warning 11: this match case is unused.
          ocamlc src/wrapped/lib/skia_generated_stubs.o
    cc1: warning: command line option '-std=c++14' is valid for C++/ObjC++ but not for C
    skia_generated_stubs.c: In function 'skia_wrapped_stub_83_sk_data_get_data':
    skia_generated_stubs.c:732:17: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
        void* x697 = sk_data_get_data(x696);
                     ^~~~~~~~~~~~~~~~
    skia_generated_stubs.c: In function 'skia_wrapped_stub_88_gr_glinterface_create_native_interface':
    skia_generated_stubs.c:762:29: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
        gr_glinterface_t* x713 = gr_glinterface_create_native_interface();
                                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    skia_generated_stubs.c: In function 'skia_wrapped_stub_126_sk_surface_get_props':
    skia_generated_stubs.c:1175:31: warning: initialization discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
        sk_surfaceprops_t* x1069 = sk_surface_get_props(x1068);
                                   ^~~~~~~~~~~~~~~~~~~~
    error: command failed: "refmterr" "dune" "build" "-p" "skia" (exited with 1)
    esy-build-package: exiting with errors above...

  building reason-skia@github:revery-ui/reason-skia#6b459c7
esy: exiting due to errors above

Most of these seems to be avoidable warnings, the only real problem being the ld error :
    (cd C:/Users/Issy/.esy/3/b/reason_skia-77127284\default/src/wrapped/c && C:\Users\Issy\AppData\Roaming\npm\node_modules\esy\node_modules\esy-bash\.cygwin\bin\x86_64-w64-mingw32-gcc.exe -O -mms-bitfields -shared -o dllskia_wrapped_c_stubs.dll c_stubs.o -luser32 -lskia -lSDL2 -lstdc++ -LC:/Users/Issy/.esy/3_/i/esy_sdl2-2.0.10006-d4b1b631/lib -LC:/Users/Issy/.esy/3_/i/esy_skia-7ed409d0/lib)
    c_stubs.o:c_stubs.c:(.text+0x42): undefined reference to `sk_canvas_draw_rect'
    c_stubs.o:c_stubs.c:(.text+0x5d): undefined reference to `gr_glinterface_assemble_gl_interface'
    collect2: error: ld returned 1 exit status

It would seem that the error comes from undefined references. After looking in the sources, it does indeed seem that these are never defined (Maybe in SkiaWrappedBindings.re ? Having no knowledge of Reason, im unsure if it is a definition or not). I should mention that deleting .esy and redownloading everything didn't have an impact on the results.
Is this issue a problem on my side ? Thanks for any help provided.

The error is here:
  c_stubs.o:c_stubs.c:(.text+0x42): undefined reference to `sk_canvas_draw_rect'
    c_stubs.o:c_stubs.c:(.text+0x5d): undefined reference to `gr_glinterface_assemble_gl_interface'

it is really weird.
Can you try to build revery-ui/esy-skia ? just clone and type esy

This issue is almost a year old and sadly i switched machines by then. I was running on an Intel iGPU (iris plus 655) which is pretty obscure, so running into GPU support issues make sense. Nowdays it compiles on my Arch system with an AMD Picasso V11 GPU and on a Win 10 dual boot. I can try to rebuild on the old machine in case theres a need for a test.

No need to boot old machine I just tried to unlock. I am in the process of updating skia so it should be more resilient when I have fully updated. You can close the issue now. Thanks for your answer


Programming books

I could add a few good books on ML

@ahmeddz39 can you elaborate on what the issue is?

hello sir, I want to add some books on competitive programming.

I can add some excellent and most-needed programming books.

can I add some useful programming books?

can I add some useful programming books which is useful for beginners.

I want to contribute, pls assign me? @shrn01

Feel free to raise a PR
Only books that are allowed for free access by the licence are accepted. Please do not share pirated links

Hello Maintainer,
i want to contibute on it with my idea.
can you assign it to me

Hello sir , can you assign this issue to me?

Hello sir
i would like to contribute with ny ideas.
i'll be glad if you assign it to me .
thank you..

Could you please assign it to me?

Could you please assign to me? Can I add Android Books

I have some Machine Learning books. Could you assign this task to me Sir?

Hey guys, if you have a free resource in mind that you want to add to the list, please raise a PR. No need to have a issue for that.
I'm closing this issue, feel free to reopen if there are any doubts


Let non-viewable Popup Annotations inherit the parent's Annotation Flags if the parent is viewable

Fixes http://www.pdf-archive.com/2013/09/30/file2/file2.pdf.
Note how it's not possible to show the various Popup Annotations in the above document.
To fix that, this patch lets the Popup inherit the flags of the parent, in the special case where the parent is viewable and the Popup is not.
In general, I don't think that a Popup must have the same flags set as the parent. However, it seems very strange to have a viewable parent annotation, and then not being able to view the Popup.
Annoyingly the PDF specification doesn't, as far as I can find, mention anything about how this case should be handled, but this patch seem consistent with the actual behaviour in Adobe Reader.
@timvandermeij How do you feel about this patch? Should we consider fixing this edge-case, or are we more worried about breakage elsewhere.
Please note: I've not attached a test-case, nor run the tests, since I wanted feedback on the general idea of the patch before spending a lot of time on this.

I think the general idea here is fine. Okular is also able to properly view this file, so I think it makes sense to fix this, even though (luckily) it's very much an edge case. The situation where the parent is visible but the popup is not is sufficiently rare that I'm not worried about breakage in other areas.

/botio test

/botio-linux preview

/botio makeref

Thank you for working on this!


bootstrap-select

Like:
https://silviomoreto.github.io/bootstrap-select/examples/

Or like this:
http://fezvrasta.github.io/bootstrap-material-design/#dropdown-menu

@sldobri thank you for the suggestion. I think we will use @rafaeldev's example because Material Kit was built over Bootstrap Material Design of Fezvrasta. Right now we are working for the PRO version of this kit then after we release it we will add the select in the Free version too.
Best,
Alex

Has this made into free version?

@uhfath bootstrap Select is a plugin that is included only in Material Kit PRO.

Thanks for the info.


Add observability-bundle

Towards https://github.com/giantswarm/giantswarm/issues/24246

Did you have time to test?

Not Yet!

@giantswarm/team-hydra Do we have to release this?

Yes. And if you want current MCs to use it, they need to be upgraded to this version.

@fiunchinho doers this also apply to cluster-aws?

@mcbenjemaa can you take care of the release? :)


Bug: SyntaxError: Cannot use import statement outside a module

Kept getting this when I was running tests using jest
SyntaxError: Cannot use import statement outside a module

After much investigation found out that supertest uses superagent on the latest version which is not recommended
<EMAIL_ADDRESS>Please use v8.0.0 until https://github.com/visionmedia/superagent/issues/1743 is resolved

Had to fix it by adding resolution to my package.json
"resolutions": {
        "superagent": "8.0.0"
    },

Hopefully<EMAIL_ADDRESS>comes out this week and no one has the same issue

Closed cause superagent:8.0.3 is out which fixes this
Similar to #798


Add error message to "Data Series" section

Demo
https://github.com/iterative/vscode-dvc/assets/43496356/fc25e368-50d7-4028-9388-cc7f6dbd4c79
To Do

[ ] Fix bug where errors don't show unless dvc.yaml changes in session
[ ] Tests

Part of #5087

[ ] Fix bug where errors don't show unless dvc.yaml changes in current session

Figured out the issue. We actually don't know what type a plot is unless it's built at least once in the current session. If the user starts a new session or reloads the window with a broken plot, the extension will not know the broken plot type and we can't show the error in the Data Series section:
https://github.com/iterative/vscode-dvc/assets/43496356/2fcd5c67-41b5-495c-9909-bdeb8fb380c7
One option to fix this is to show the error messaging above the sections below the ribbon:

We also could just leave it as is but it seems buggy 🤔 Any other ideas? cc @iterative/vs-code, @shcheklein

One option to fix this is to show the error messaging above the sections below the ribbon:
We also could just leave it as is but it seems buggy 🤔 Any other ideas? cc @iterative/vs-code, @shcheklein

Going to open a new pr that adds the error message above the sections and close this one for now. We could at least have a generic error message above all sections as a first iteration and come back to this later if we find a solution for the bug.


Add user repository using the Auth0 management API

Please keep in mind that this is a public package, licensed under MIT
Proposed changes
Fully tested UserRepository with Facade. This can be used to retrieve user details from Auth0, by e.g. ID. The repository was copied from PSM. That can use this version from now on. Facade has been added to ensure very easy testing for the applications that use this package. Using the facade, the UserRepository can be swapped in Laravel's service container by a fake implementation. From that moment on, no API calls will be made to Auth0 and a mocked implementation is used. Thus, greatly improving testability in the applications.
Also, this uses Azure Pipelines now instead of Travis.
Types of changes
What types of changes does your code introduce to this repository?
Put an x in the boxes that apply

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

Testing
Explain here how to run your unit tests, and explain how to execute manual testing.
Unit testing
As usual. See readme. Use composer test-coverage for code coverage.
Manual testing
Use as package in Laravel application.
Further comments
Package is public.

@SquarePlayn thanks for the detailed review, and fixing my copy-paste errors :)


[BugFix] prevent running out-of-space when selecting disk for new tablet

What type of PR is this：

[X] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

prevent running out-of-space when the disk selected for new tablet beyond the limit.

check the available disk space is enough  for new tablet
sort disk by available space left

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[X] I have checked the version labels which the pr will be auto backported to target branch

[X] 3.0
[X] 2.5
[ ] 2.4
[ ] 2.3

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5