UI: css refactoring to decrease bundle size

Replace @import to @use statements using sass-migrator
Move css classes which requires @import and @use statements to style.scss (global style)
Adapt Angular.json budgets

Before:
2022-09-06 15:05:21: webpack 5.70.0 compiled in 10791 ms (6f4a57ea380d1396)

Initial Chunk Files           | Names         |  Raw Size | Estimated Transfer Size
main.b6e1d2ae26ab676e.js      | main          |   3.45 MB |               271.61 kB
styles.c428f27bcabb0388.css   | styles        | 507.26 kB |                61.55 kB
polyfills.c3b42344daecbbf7.js | polyfills     |  33.80 kB |                10.87 kB
runtime.0277c5ce06eafbda.js   | runtime       |   1.13 kB |               643 bytes

| Initial Total |   3.98 MB |               344.65 kB

Build at: 2022-09-06T13:05:21.485Z - Hash: 6f4a57ea380d1396 - Time: 11226ms

After:
2022-09-08 16:45:16: webpack 5.70.0 compiled in 26178 ms (8ff57acc6d4d9197)

Initial Chunk Files           | Names         |  Raw Size | Estimated Transfer Size
main.407b1909032e3eb0.js      | main          |   1.58 MB |               230.75 kB
styles.d27ed3bbab3aa363.css   | styles        | 507.35 kB |                61.65 kB
polyfills.c3b42344daecbbf7.js | polyfills     |  33.80 kB |                10.87 kB
runtime.0277c5ce06eafbda.js   | runtime       |   1.13 kB |               643 bytes

| Initial Total |   2.11 MB |               303.89 kB

Build at: 2022-09-08T14:45:18.196Z - Hash: 8ff57acc6d4d9197 - Time: 28175ms

Fixes: https://github.com/aquarist-labs/s3gw/issues/42
Signed-off-by: Francesco Torchia<EMAIL_ADDRESS>

@votdev

After some tests I found that removing @extend from css classes reduces the bundle size almost as relocate them to src/styles.scss, and yes, I agree with you this last action it's not a good practice.

Regarding first commit , should we keep @import to @use replacing?

I found also, ~30% bundle size depends on those import from Material library:
@import<EMAIL_ADDRESS>
They be removed by a simple css refactoring. For instance:
  i.mdi::before {
    @extend .mdi-48px;
  }

to:
  .mdi-help-circle::before {
    font-size: 48px;
  }

What do you think about it?

IMO, we could:

Keep first commit and remove this
relocate bootstrap class in HTML files as you suggested
remove @mdi imports

  They be removed by a simple css refactoring. For instance:
  ```
    i.mdi::before {
      @extend .mdi-48px;
    }
  ```

  to:
  ```
    .i.mdi::before {
      line-height: inherit;
      font-size: 48px;
    }
  ```
  What do you think about it?

What i like to see is for the alert-panel.component for example:
Replace
.content-wrapper {
  @extend .d-flex;
  @extend .justify-content-start;
  @extend .align-items-center;
  @extend .overflow-hidden;

  i.mdi::before {
    @extend .mdi-48px;
  }

  .content {
    @extend .ms-2;
  }
}

by

.content-wrapper {}

<ngb-alert *ngIf="!noColor"
[dismissible]="false"
[type]="bsType">
<ng-container *ngTemplateOutlet="renderContent">

<ng-container *ngIf="noColor">
<ng-container *ngTemplateOutlet="renderContent">

<ng-template #renderContent>

  <=== will not work because class and [class] can not be used at the same time, must be fixed somehow

```
Keep in mind that we need to be able to brand the whole UI. So prepare the code to allow easy customization for everything via an additional scss file.

  They be removed by a simple css refactoring. For instance:

i.mdi::before {
  @extend .mdi-48px;
}

to:

.i.mdi::before {
  line-height: inherit;
  font-size: 48px;
}

What do you think about it?

What i like to see is for the alert-panel.component for example:
Replace
.content-wrapper {
  @extend .d-flex;
  @extend .justify-content-start;
  @extend .align-items-center;
  @extend .overflow-hidden;

  i.mdi::before {
    @extend .mdi-48px;
  }

  .content {
    @extend .ms-2;
  }
}

<ngb-alert *ngIf="!noColor"
           [dismissible]="false"
           [type]="bsType">
  <ng-container *ngTemplateOutlet="renderContent"></ng-container>
</ngb-alert>
<ng-container *ngIf="noColor">
  <ng-container *ngTemplateOutlet="renderContent"></ng-container>
</ng-container>

<ng-template #renderContent>

      <ng-content></ng-content>

</ng-template>

by
.content-wrapper {}

and
<ngb-alert *ngIf="!noColor"
           [dismissible]="false"
           [type]="bsType">
  <ng-container *ngTemplateOutlet="renderContent"></ng-container>
</ngb-alert>
<ng-container *ngIf="noColor">
  <ng-container *ngTemplateOutlet="renderContent"></ng-container>
</ng-container>

<ng-template #renderContent>

      <=== will not work because class and [class] can not be used at the same time, must be fixed somehow

      <ng-content></ng-content>

</ng-template>

Keep in mind that we need to be able to brand the whole UI. So prepare the code to allow easy customization for everything via an additional scss file.

Sure, That' s what I did in my last fix, except for:

because mdi-48px class alone is not sufficient do correctly display the icon. The icon is not centered in the middle of the modal component.
See pics below, for example modal.component.html:

That's the reason why I put line-height: inherit; in a separated class.

Just rebased.


Remove old polling logic

We need to remove all the polling logic that exists. Historically this existed as we did not have a data node that allowed us to connect via websocket. This resulted in a lot of polling logic that can be much more easily written with subscriptions. These can be found in the project by searching for setInterval.

This caused a legitimate bug earlier today. An internal employee tried to created 2 withdrawals and due to the race condition tried to withdraw some money twice as the use maximum button on the withdrawals page had not been updated. This meant the withdrawal was rejected.


Components in docs

Adds components to documentation! So people can play with squiggle inside the documentation.
This also patches up dependencies so that you don't need to do anything other than build the rescript in squiggle lang to work with the docs.
I think I've fixed everything so that ci should pass, but we'll see

Yo! A successful CI (minus the netlify stuff)


DRT-4891 Generalise purgeExpired function. Rename QueueLoadMinute to …

…LoadMinute

looks good to me. happy to merge when you are.


code release

Hi, I read with great interest this work, is there any plan to release the code open source?
thank you

@tiralonghipol Hi,
I am beginnar in this coding language, did you run this code or not yet ?
Thank you


DenseNet Settings

Hi,
Thanks for open-sourcing the codes.
Could you please also reveal the code or the implementation details of the DenseNet-121 experiment?
DId you use the same resolution and the same temperature as the experiment of ResNetv2-121?

Hi,
Thanks for being interested in our work!
For DenseNet-121, we used the PyTorch implementation: https://pytorch.org/vision/main/_modules/torchvision/models/densenet.html#densenet121.
And we used the pre-trained model provided by PyTorch: https://download.pytorch.org/models/densenet121-a639ec97.pth
The resolution we used here is a 224x224 center crop. Everything else (including temperature) stays the same.
Let me know if there is any issue in reproducing the results!

Thanks! The results are almost the same when using the resolution of 224.

Awesome! Thanks for the follow up!


Transition ticketing system from Jira to Github Issues

Figure out how Github Issues works

Pls can you be more specific on the kind of help you would need on this issue?

reopening!

Pls can you be more specific on the kind of help you would need on this issue?

No help in particular - need to migrate tickets on our end - would love any feedback you have if you're able to run docker okay

we did it!


Changed CDN url for material iconic font

fixes #1080

Not sure what happened here @bradrydzewski, but the icons are not the same anymore:

Maybe we were using a different version then...


Please make it clear that the terraform state which exists will be nuked and replaced with the new local infrastructure state.

Hi @creikey,
Sorry to hear you had problem with tflocal.
Let me share my 2 cents on the topic even though I'm not a maintainer of the repo anymore:
Terraform itself is nuking your state if you provide another account's credentials. Period. That's the way it works and that's why you have the plan stage before any applys. Hashicorp emphasizes the vulnerability of the state file in its docs many times.
So by default it's the user's responsibility to isolate an environment ie into workspaces or into a different folder layout and double check if the changes are correct.
I personally can't think of a scenario where you'd be able to check an already existing state programmatically and tell if it was run against LocalStack or a valid AWS account as the stored resource ARNs could contain valid account numbers not just mumbo jumbo accounts.
The best LocalStack could do is provide in the docs and in the README a warning for terraform and any other tool that manages a separately stored statefile(ie Pulumi), but in the end everything comes down to the users and their actions.

@lakkeger
Yeah I agree I certainly made a mistake procrastinating more robust state management (will probably move it to a row in dynamo or an s3 file after this), I think tflocal as a tool should simply look for a local state file, check if it's already referring to local state infrastructure or not, and if it exists and isn't localstate add an extra Are you sure you want to proceed? You will lose current state before the apply.
I had assumed this tool used state that was separate from the existing local state files in the directory, based on some other language in the documentation

@creikey The problem with the suggested solution is, terraform has many options where to store your state, ie gitlab state storage or hashi cloud and that certainly would fall through with similar issues here.
Here the solution could be to list the resources in the background with terraform state list.
However, thinking a bit more on the issue the prompt would be a breaking change for anyone who would run terraform-local against LocalStack as it would prompt for every single time for input if you plan or apply something. 🤔 As a result any existing CI would break instantly that uses tflocal, which could be worked around by introducing a non-interactive flag/env variable, but this won't be the default. In case it's default this change would lose purpose.
As a major version this could be an option, but imo the UX for any normal user would become annoying by prompting twice for confirmation every single time.
Simply showing a warning makes no sense either as your plan stage with a bigger change rolls over this message in the console history. Showing it after the apply prompt makes no sense either.
So all in context of the above your best bet is better docs to raise awareness.
/cc @simonrw @MarcelStranak

@lakkeger  What do you mean? I'm suggesting this tool reads the tfstate it's about to modify, and if and only if the existing state points to non-tflocal resources, there is an explicit confirmation step.
Under none of the environments you listed, CI, running against local stack multiple times, etc does this warning get triggered, because in those cases, there are no non-tflocal resources saved. Normal users are not bothered because they aren't prompted twice, because the resources are seen to be tflocal by this tool.

To detect if a tfstate is managed by tflocal or not, it seems like the best bet would be to check for all zeros in the ARN:
            "arn": "arn:aws:dynamodb:us-west-2:000000000000:table/TestingUsers", (it's only like this with tflocal state)
Although this is a bit hacky

Also, the problem with the current UX is that, if you run tflocal apply, during terraform's confirmation step, there is absolutely no mention of the previous state being lost at all, everything is green and it seems as if local stack infrastructure is all being provisioned as plan (when what has just happened is all of your state has just been lost, that was there before)

Alternatively, ideally, you actually just make it so tflocal doesn't use the terraform.tfstate file, using the -state=path argument, instead please make it named local_terraform.tfstate or something

To detect if a tfstate is managed by tflocal or not, it seems like the best bet would be to check for all zeros in the ARN:
            "arn": "arn:aws:dynamodb:us-west-2:000000000000:table/TestingUsers", (it's only like this with tflocal state)
Although this is a bit hacky

You can set LocalStack to not deploy to the all 0 account so this is not a fool proof solution.

Also, the problem with the current UX is that, if you run tflocal apply, during terraform's confirmation step, there is absolutely no mention of the previous state being lost at all, everything is green and it seems as if local stack infrastructure is all being provisioned as plan (when what has just happened is all of your state has just been lost, that was there before)

Yes, this is the case as the state planned is done by terraform, terraform-local is not a fork of it but a wrapper if the plan is long, as I said it would roll out of your consoles buffer and you miss the message.

@lakkeger  What do you mean? I'm suggesting this tool reads the tfstate it's about to modify, and if and only if the existing state points to non-tflocal resources, there is an explicit confirmation step.
Under none of the environments you listed, CI, running against local stack multiple times, etc does this warning get triggered, because in those cases, there are no non-tflocal resources in the state. Normal users are not bothered because they aren't prompted twice, because the resources are seen to be tflocal by this tool.

You can't tell just from the state if it's LocalStack or other resources being involved.

Alternatively, ideally, you actually just make it so tflocal doesn't use the terraform.tfstate file, using the -state=path argument, instead please make it named local_terraform.tfstate or something

With this one you are right though, I didn't consider this option. This can work, only command parsing has to be added to the tool which currently is not a thing it does.


Room - Sign in RHP disappears after clicking on the "<" button

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.11-2
Reproducible in staging?: Y
Reproducible in production?: Y
Issue reported by: Applause - Internal Team
Action Performed:

Open Chrome in incognito
Navigate to https://staging.new.expensify.com/r/2091104345528462
Click on the "Sign in" button at the bottom right corner
Input any email and click on the "Continue" button
Click on the "<" button

Expected Result:
he RHP should display the page where I can input the email
Actual Result:
Sign in RHP disappears afte clicking on the "<" button
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/dcbb9b27-5572-41bd-a3fa-a8ebbd101e1f
View all open jobs on GitHub

@anmurali  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-vsp

Proposal looks good, but let's declare a function to handle the logic in onBackButtonPress instead of declaring it inline.

added regression and paid


eslint能不能关掉

[ ] I'm sure this does not appear in the issue list of the repository

Basic Info

Package Name And Version<EMAIL_ADDRESS>Browser: chrome<IP_ADDRESS>

eslint能不能关掉，怎么修改相关配置


stack rolled back

Stack creation rolled back due to ERROR "CodeBuildDeploy | CREATE_FAILED | Parameters: [Timestamp] must have values"

Same status

The issue is now resolved, the template cfntemplate.yml has been updated to add the mandatory timestamp parameter and synched with the s3 bucket . The launch template is working fine now.

confirmed, working as expected


'apply_resolutions' reports error at the end if any resolution failed

Before this change, 'apply_resolutions' always succeeded.
This changes behaviour of job 'scan_and_fix' and 'apply_resolutions'.
They will finish with error now in case of any resolution error.

#114092983
Signed-off-by: Tom Kiemes<EMAIL_ADDRESS>

Problem resolver task has error state and looks like:
, [2016-02-25 14:05:53 #13178] [task:55] ERROR -- DirectorJobRunner: Error resolving problem `6': Timed out pinging to c31b4b86-2e7d-4282-b1ba-26a55c4cc348 after 600 seconds
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.0000.0/lib/bosh/director/problem_resolver.rb:48:in `apply_resolutions'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.0000.0/lib/bosh/director/jobs/cloud_check/scan_and_fix.rb:37:in `block in perform'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.0000.0/lib/bosh/director/lock_helper.rb:13:in `block in with_deployment_lock'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.0000.0/lib/bosh/director/lock.rb:56:in `lock'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.0000.0/lib/bosh/director/lock_helper.rb:13:in `with_deployment_lock'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.0000.0/lib/bosh/director/jobs/cloud_check/scan_and_fix.rb:30:in `perform'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.0000.0/lib/bosh/director/job_runner.rb:99:in `perform_job'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.0000.0/lib/bosh/director/job_runner.rb:32:in `block in run'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh_common-1.0000.0/lib/common/thread_formatter.rb:49:in `with_thread_name'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.0000.0/lib/bosh/director/job_runner.rb:32:in `run'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.0000.0/lib/bosh/director/jobs/base_job.rb:10:in `perform'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/resque-1.25.2/lib/resque/job.rb:227:in `perform'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/resque-1.25.2/lib/resque/worker.rb:250:in `perform'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/resque-1.25.2/lib/resque/worker.rb:189:in `block in work'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/resque-1.25.2/lib/resque/worker.rb:166:in `loop'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/resque-1.25.2/lib/resque/worker.rb:166:in `work'
/var/vcap/packages/director/gem_home/ruby/2.1.0/gems/bosh-director-1.0000.0/bin/bosh-director-worker:75:in `<top (required)>'
/var/vcap/packages/director/bin/bosh-director-worker:16:in `load'
/var/vcap/packages/director/bin/bosh-director-worker:16:in `<main>'
D, [2016-02-25 14:05:53 #13178] [task:55] DEBUG -- DirectorJobRunner: (0.000478s) SELECT NULL
D, [2016-02-25 14:05:53 #13178] [task:55] DEBUG -- DirectorJobRunner: (0.000136s) BEGIN
D, [2016-02-25 14:05:53 #13178] [task:55] DEBUG -- DirectorJobRunner: (0.000690s) UPDATE "tasks" SET "state" = 'error', "timestamp" = '2016-02-25 14:05:53.131267+0000', "description" = 'scan and fix', "result" = 'Error resolving problem `6'': Timed out pinging to c31b4b86-2e7d-4282-b1ba-26a55c4cc348 after 600 seconds', "output" = '/var/vcap/store/director/tasks/55', "checkpoint_time" = '2016-02-25 13:55:00.631683+0000', "type" = 'cck_scan_and_fix', "username" = 'hm' WHERE ("id" = 55)
D, [2016-02-25 14:05:53 #13178] [task:55] DEBUG -- DirectorJobRunner: (0.002356s) COMMIT
I, [2016-02-25 14:05:53 #13178] []  INFO -- DirectorJobRunner: Task took 10 minutes 52.512512077 seconds to process.

Task 55 error

bosh tasks recent --no-filter shows:
+----+------------+-------------------------+-------+-------------------+--------------------------------------------------------------------------------+
| #  | State      | Timestamp               | User  | Description       | Result                                                                         |
+----+------------+-------------------------+-------+-------------------+--------------------------------------------------------------------------------+
| 65 | error      | 2016-02-25 14:16:51 UTC | hm    | scan and fix      | Error resolving problem `7': Timed out pinging to...                           |
| 69 | error      | 2016-02-25 14:09:59 UTC | hm    | scan and fix      | Unable to get deployment lock, maybe a deployment is in progress. Try again... |
| 68 | error      | 2016-02-25 14:08:59 UTC | hm    | scan and fix      | Unable to get deployment lock, maybe a deployment is in progress. Try again... |
| 66 | error      | 2016-02-25 14:06:59 UTC | hm    | scan and fix      | Unable to get deployment lock, maybe a deployment is in progress. Try again... |
| 55 | error      | 2016-02-25 14:05:53 UTC | hm    | scan and fix      | Error resolving problem `6': Timed out pinging to...                           |

I created a new one against develop branch.

New PR against the develop branch: https://github.com/cloudfoundry/bosh/pull/1152


Message error: unpacking of archive failed: cpio: Bad magic. when installing or updating package(s)

… occurs if a RPM package compressed with zstd is istalled or updated to on SailfishOS < 4.3.0.
A %define _binary_payload w2.xzdio in the spec file of the affected package resolves this issue.
Pulling this out of issue #165, because it is unrelated to the original issue content there.

Reference: FSO: SFOS 4.5.0 switches RPM compression. What does that mean for existing RPMs?


Initial investigation for WGE dark mode

Scope:

document the current state of WGE UI
any blockers?
suggest a plan/options to implement the dark mode in WGE with pros & cons

I wouldn’t begin thinking about adding Dark Mode to Enterprise in the near term, for all the reasons Josh listed. There is way too much inconsistency in the code base.
We need to decide what our UI code should look like across both projects, and then work towards that, in my opinion.

@jpellizzari I have to disagree.  These need to be solved, there will never be a good time to do it and it will only be harder in the future.  @LappleApple and @katya-makarevich let me know what help you need to get the FEs to coordinate and establish the right patterns.

@JamWils

These need to be solved

That is what I am advocating for. Without documenting best practices and holding people to them, how would these be solved?

Hi @JamWils, Jordan is working on a guide and I'm hearing that it should land quite soon. He's in touch with @yiannistri and @joshri.

Thanks @LappleApple.  Also, my last comment did undermine @jpellizzari and I apologize for my tone.  I meant to say can you and Katya coordinate with him so we can have the dark mode enterprise work unblocked.  As one of the lead FEs in the organization he should approve these patterns for the other engineers to follow moving forward.  Thanks for the extra clarification @jpellizzari!

No hard feelings. I realize I didn't offer any solutions in my comment.


Fixing destination directory issue with downloading test agent from non uri path

Description:
Dpwnload Test Agent powershell script is using robocopy to copy the non uri path to the required destination directory. But for the case of non URI destination directory was not even getting intialized leading to copy failure. As it was very rare scenario, and in almost every case path was provided as URi hence the issue was not hit earlier.
Fix:
Initializes Destination directory from destination path provided, as done in URI case.
Aslo fixed the syntax of a verbose message

Hi @vimegh, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Vishal Meghani). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


Displaying Error On Verification of Email

Describe the bug
On opening an account on the Eventyay Attendee App it requests an email verification for the account so opened and to do so a link is sent to the registered email. But on clicking the link it redirects to a web page which prints ERROR on the screen.
To Reproduce
Steps to reproduce the behavior:

Go to the Eventyay App.
As you open the app it asks for your email.
This is followed by  setting up of your account (eg: name , password etc.)
After doing so you will recieve a verification email on the registered account.
On clicking the link you will see the error message displayed on a web page.(Screenshots provided below)

Expected behavior
From my experience of using other applications i expected the application to give a confirmation message on the web page and then redirect me to the application.
Logs

Screenshots

Would you like to work on the issue?

[x] Yes
[ ] No
Other:

You have opened this issue in the Organizer App repo and then mentioned Attendee App in the description while the issue actually belongs to the frontend.

Please open the issue on the frontend repo.


Fix q002

Fixes #1775. Before implementing your solution I thought of a slightly simpler one. However, it will let this function pass:
def double_inside_single(a):
    'Double inside "single "'

If we want function to pass, my implementation works. But if we do not, then I can go with how you suggested I implemented this (I left how I would begin to handle it commented out). The bottom of the flake8-quotes documentation seems to suggest that this should pass:
https://pypi.org/project/flake8-quotes/

I went to check how flake8-quotes handles this, and it looks like out of the box they throw an error on what we are trying to fix.
'''
def double_inside_single(a):
"This is a test"
'''
Results in:
resources/test/fixtures/flake8_quotes/docstring_doubles_function.py:31:5: Q002 Single quote docstring found but double quotes preferred
So I will leave it up to you to decide on the quotes inside quotes edge case I found above. After that I will add the test fix and we can merge this thing!

Okay I'm struggling to understand the originating plugin's logic but here's my guess.
If you have Double enabled, these should be ok:
def f():
    """Docstring"""

def f():
    "Docstring"

def f():
    """Doc'string"""

def f():
    "Doc'string"

def f():
    'Doc"string'

def f():
    '''Doc"string'''

And these should error:
def f():
    '''Docstring'''

def f():
    'Docstring'

If you have Single enabled, the reverse should be true. So all of these should be ok:
def f():
    '''Docstring'''

def f():
    'Docstring'

def f():
    '''Doc"string'''

def f():
    'Doc"string'

def f():
    "Doc'string"

def f():
    """Doc'string"""

(I think we probably have the same bug as the library, because I copied the logic exactly.)

At first I made a complicated function:
fn contains_good_docstring(quote: &Quote, raw_str: &str) -> bool {
match quote {
Quote::Single => raw_str.contains("'''") || raw_str.contains("'"),
Quote::Double => raw_str.contains(""""") || raw_str.contains('"'),
}
}
But now im realizing we simplify it to this:
At first I made a complicated function:
fn contains_good_docstring(quote: &Quote, raw_str: &str) -> bool {
match quote {
Quote::Single => raw_str.contains("'"),
Quote::Double => raw_str.contains('"'),
}
}
And with this we can pretty much just take it back to the original function. But with a single character instead of triples.

Changes in! This should work as "if single there needs to be a single quote SOMEWHERE, and if double there needs to be a double quote SOMEWHERE".

@colin99d - If you're interested in a follow-up...
If we are using a triple quote, I don't think we should allow you to invert the quote style. That is, if you're preferring ", and you do this:
def f():
  '''This has a quote " in it.'''

I think, perhaps, that should be considered a violation, since you can just do this:
def f():
  """This has a quote " in it."""

So the flexibility we added here, IMO, should only apply when using single-quoted docstrings.

@colin99d - If you're interested in a follow-up...
If we are using a triple quote, I don't think we should allow you to invert the quote style. That is, if you're preferring ", and you do this:
def f():
  '''This has a quote " in it.'''

I think, perhaps, that should be considered a violation, since you can just do this:
def f():
  """This has a quote " in it."""

So the flexibility we added here, IMO, should only apply when using single-quoted docstrings.

Makes sense to me! Thanks for the explanation.

Another way of looking at it is... what would Black convert?
Black changes this to double quotes:
def f():
    '''Some "docstring.'''

But not this:
x = 'here is a quote " inside'

Another way of looking at it is... what would Black convert?
Black changes this to double quotes:
def f():
    '''Some "docstring.'''

But not this:
x = 'here is a quote " inside'

From now on I will just let what would black do answer formatting ambiguity questions I have.

In our case, though, we also the opposite, if the user has Quote::Single set :)


p and q dimension reversed in Traceback

Hi,
I am reporting a minor confusion, my traceback rshowed a SystemError when I called mmd.compute_p_value(statistic) with
mmd.set_null_approximation_method(sg.NAM_MMD2_GAMMA) :
SystemError: [ERROR] In file /feedstock_root/build_artefacts/shogun-cpp_1512688880429/work/shogun-shogun_6.1.3/src/shogun/statistical_testing/QuadraticTimeMMD.cpp line 439: Number of samples from p (1204) and q (2654) must be equal.

But the dimensions are reversed, p is of shape 2654 and q 1204. I doubled check using mmd.get_p() method

Thanks for the report!

I can pick this issue up.
@karlnapf can you please explain what needs to be done here?

Read the developer readme
find the place where the error message is raised
revert the numbers
send a PR

Fixed via #4745
Many thanks!


Could not find a version that satisfies the requirement tensorflow (from versions: ) No matching distribution found for tensorflow

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

What is the issue?

I got the same error message when installing tensorflow with the newest version of python (3.6) and the newest version of pip (10.0.1).

This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.
If you think we've misinterpreted a bug, please comment again with a clear explanation, as well as all of the information requested in the issue template. Thanks!


bug: show_symbol_type_toc stopped working with markdown==3.6.0

Description of the bug

After upgrading the markdown package to the most recent version, the type is not shown in the ToC anymore. I think this is due to https://python-markdown.github.io/changelog/#refactor-toc-sanitation ("Refactor TOC Sanitation").
To Reproduce

Enable show_symbol_type_toc: true in mkdocs.yml

Upgrade markdown to 3.6
pip install --upgrade "markdown"

ToC doesn't include the type anymore .. revert markdown to 3.5.2` for making it work again
pip install "markdown==3.5.2"

Expected behavior
show_symbol_type_toc works by showing the types in the table of contents (see screenshot):

Environment information

$ python -m mkdocstrings.debug  # | xclip -selection clipboard

- __System__: Linux-6.8.1-arch1-1-x86_64-with-glibc2.39
- __Python__: cpython 3.11.8
- __Environment variables__:
- __Installed packages__:
  - `mkdocstrings` v0.24.1

Markdown 3.6 is supported by the latest version of mkdocstrings-python, try upgrading it :)

Sorry for posting on v0.24.1 in the opening message; the CI tools are using the most recent versions. I've also upgraded on my localhost:
$ pip freeze | grep mkdocstrings
mkdocstrings==0.24.2
mkdocstrings-python==1.9.2

Still, I don't see the type symbols showing up in the ToC with markdown==3.6.

OMG I released the fix with commented out code... must be tired. Will push a new release (mkdocstrings 0.24.3).

We have the same issue.

Thank you so much, I highly appreciate it :bow:


AfFample2 implementation

Implements AFsample2, disabled by default

Also allows to increase the number of templates from 4 to 1000 which might be useful later


[WIP] [g-sim 2] (g + P)-sim

This is a follow-up for https://github.com/PennyLaneAI/qml/pull/1085, our new demo on g-sim.
Here, I show some original extensions of g-sim for product operators in an efficient fashion.
[sc-62859]

Merged both approaches in https://github.com/PennyLaneAI/qml/pull/1118


Yazi not opening in cwd if space in path

In Windows, when I call yazi.nvim, it goes to a parent folder that is the part of the path before a space appears. I'm not sure if it's possibly that spaces aren't being checked for, or that it sees it as an additional argument?
If I have the path C:\Users\Name\Notes\Third Year\Course Name, instead of opening in Course Name, it opens in Notes.
Calling Yazi directly in a terminal in Nvim opens in the correct location.

Hmm, I don't have access to a Windows machine but we can try to fix this together if you're up for a little experimentation.
I have some questions for you and then two ideas:

do you use WSL or WSL2? or is there something newer? Idk about windows development these days lol 😄
have you had issues with other neovim plugins? maybe there is something obvious that I have missed but others have solved.
do you think your choice of terminal emulator / neovide / something else could affect this? what are you using?
how do you start yazi.nvim? this probably doesn't matter but just want to make sure you don't have anything crazy going on

Ideas:

I implemented two new things: debug logging that can help diagnose this, as well as a potential fix for it in #69 . Can you try that out? You can do it with lazy.nvim by setting the option branch = "try-to-fix-windows-path-issue" in your yazi.nvim plugin spec
It might solve it right away, but in case not, you can follow the instructions in the PR and enable logging. :checkhealth yazi will show you where the log file is, and you can paste its contents here

Here are more detailed instructions:

Save this file as repro.lua
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify('./.repro', ':p')

-- set stdpaths to use .repro
for _, name in ipairs({ 'config', 'data', 'state', 'cache' }) do
  vim.env[('XDG_%s_HOME'):format(name:upper())] = root .. '/' .. name
end

-- bootstrap lazy
local lazypath = root .. '/plugins/lazy.nvim'
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    'git',
    'clone',
    '--filter=blob:none',
    'https://github.com/folke/lazy.nvim.git',
    lazypath,
  })
end
vim.opt.runtimepath:prepend(lazypath)
vim.g.mapleader = ' '

-- install plugins
local plugins = {
  'folke/tokyonight.nvim',
  {
    'mikavilpas/yazi.nvim',
    branch = 'try-to-fix-windows-path-issue',
    dependencies = {
      'nvim-lua/plenary.nvim',
    },
    event = 'VeryLazy',
    keys = {
      {
        -- 👇 choose your own keymapping
        '<leader>fy',
        function()
          require('yazi').yazi()
        end,

        { desc = 'Open the file manager' },
      },
    },
    opts = {
      open_for_directories = false,
      log_level = vim.log.levels.DEBUG,
    },
  },
}
require('lazy').setup(plugins, {
  root = root .. '/plugins',
})

vim.cmd.colorscheme('tokyonight')
-- add anything else here

Run the following command to enter neovim running the repro.lua script:
nvim -u repro.lua -c "lua require('lazy').update()"

You should now see neovim running. In this reproduction (repro) session,
yazi.nvim will now log messages to a log file. Execute the following command
in neovim:
checkhealth yazi

You should see something like this:
 yazi: require("yazi.health").check()

 yazi
 - yazi.nvim log file is at /Users/mikavilpas/git/yazi.nvim/.repro//state/nvim/yazi.log
 -     hint: use gf to open the file path under the cursor
 - OK yazi

Open the log file (e.g. by placing the cursor on the path and pressing gf)
and send me the contents.

The log under my configuration
[2024-05-13 12:56:53] DEBUG Opening yazi with the command: (yazi "C:\Users\Administrator\AppData\Local\nvim\lua\plugins\yazi.lua" --local-events "rename,delete,trash,move" --chooser-file "C:/Users/Administrator/AppData/Roaming/yazi/state/yazi_filechosen/yazi_filechosen" > C:/Users/Administrator/AppData/Roaming/yazi/state/yazi.nvim.events.txt)
[2024-05-13 12:57:00] DEBUG Opening yazi with the command: (yazi "C:\Users\Administrator\AppData\Local\nvim\lua\plugins\yazi.lua" --local-events "rename,delete,trash,move" --chooser-file "C:/Users/Administrator/AppData/Roaming/yazi/state/yazi_filechosen/yazi_filechosen" > C:/Users/Administrator/AppData/Roaming/yazi/state/yazi.nvim.events.txt)
[2024-05-13 12:59:02] DEBUG Opening yazi with the command: (yazi "C:\Users\Administrator\AppData\Local\nvim\lua\plugins\yazi.lua" --local-events "rename,delete,trash,move" --chooser-file "C:/Users/Administrator/AppData/Roaming/yazi/state/yazi_filechosen/yazi_filechosen" > C:/Users/Administrator/AppData/Roaming/yazi/state/yazi.nvim.events.txt)

I'll test this now. I think it's possibly got to do with vim.fn.filenameescape, as discussed in vim/vim#541.

Hmm, I don't have access to a Windows machine but we can try to fix this together if you're up for a little experimentation.

Ah, I can imagine this might make it trickier.

Do you use WSL or WSL2? or is there something newer? Idk about windows development these days lol 😄

I don't use either WSL or WSL2. Neovim actually works fine on native Windows now.

Have you had issues with other neovim plugins? maybe there is something obvious that I have missed but others have solved.

I think Neotree also had this issue before, discussed in nvim-neo-tree/neo-tree.nvim#1076 and they solved it in nvim-neo-tree/neo-tree.nvim#1077

Do you think your choice of terminal emulator / neovide / something else could affect this? what are you using?

I doubt it makes a difference. I use Powershell 7 inside Windows Terminal.

How do you start yazi.nvim? this probably doesn't matter but just want to make sure you don't have anything crazy going on

I've got it as an entry for Lazy.nvim. Originally, I called it without sending in vim.fn.getcwd(), but I added that to see if it would make a difference.
{
  "mikavilpas/yazi.nvim",
  dependencies = {
    "nvim-lua/plenary.nvim",
  },
  event = "VeryLazy",
  keys = {
    {
      "<leader>y",
      function()
        require("yazi").yazi(nil, vim.fn.getcwd())
      end,
      { desc = "Explorer Yazi" },
    },
  },
}

I implemented two new things: debug logging that can help diagnose this, as well as a potential fix for it in Try to fix windows path issue #69 . Can you try that out? You can do it with lazy.nvim by setting the option branch = "try-to-fix-windows-path-issue" in your yazi.nvim plugin spec

I will send the results of this and the guide in your next post soon.

I created a folder for your repro code, and ran Neovim using it. Unfortunately, calling the health check lead to:
yazi: require("yazi.health").check()

yazi ~
- WARNING yazi --version looks unexpected, saw 7[>q_Gi=31,s=1,v=1,a=q,t=d,f=24;AAAA\8Yazi 0.2.5 (386dd4c 2024-05-08)

- ERROR Failed to run healthcheck for "yazi" plugin. Exception:
  ...gram Files/Neovim/share/nvim/runtime/lua/vim/version.lua:174: attempt to index local 'version' (a nil value)

Possibly, the warning is due to having extra symbols in my prompt.
I'm going to test it with just changing the branch in my normal config, and see what the results of that are.

 I've never been able to figure out why, or how to remove them. It happens with any command, whether it's called through Lua, or directly.
For example:
echo system("ls")

also prints those weird characters. I'm not sure if they're an escape sequence.

Hmm, I read the discussion and it seems to me that this is some weird limitation in neovim itself.
Is it related to powershell? I have seen some neovim users on reddit report that wezterm has had really good windows support, it might be a good option to check out - although I don't know if that means a big change to your workflow.
If you happen to find a way this could be worked around, I would be willing to try it together. Let me know if you have any ideas.

Closing this for now, although if any ideas can be thought of, let's continue investigating.


⚠ Bofrost.at has degraded performance

In 18cef5f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1998 ms

Resolved: Bofrost.at performance has improved in 043b79b.


Update the design for the withdraw modal

This PR updates the withdraw modal to a structure similar to the deposit modal. It removes the old table format in favor of divs with flex layout. Cancel action now exists the modal immediately

fix display issue with mixer amount on modal 99999.99999999999 before merging

Merged with with new provider


simplify BlockingProcessStreamReader; fix tests

Update #1429.
This commit simplifies the main run() method.
Previous implementation checks whether the underlying Reader is ready to read
then either reads a line or sleeps,
catching InterruptedException to watch for any thread interruption.
There are subtle difficulties with this approach:

Even if the underlying Reader is ready to read, it might not have
enough bytes to form a line. It might still block.
It's not necessary to sleep. If the thread is interrupted while
reading. It should throw InterruptedIOException.
The method now reads in a loop, waiting for either exceptions or EOF.

The test class implements a mock Logger that logs to a data structure.
It then verifies that the data structure holds appropriate logs.
As implemented, this can cause a race, as two threads,
the writer and the verifier, run concurrently.
This commit fixes this by waiting for the writing thread to terminate
before verifying.

Coverage decreased (-0.01%) to 83.577% when pulling fca0cfeb5a2dd7790a72a5cdc5f6f7f76bce5d5d on pongad:stream-reader into cacf65e3fd1c4567a4a607c3d9d0ced2adc4d9cc on GoogleCloudPlatform:master.


Future import assertions syntax for importing JSON files

see https://github.com/osmlab/name-suggestion-index/issues/6014 for details, but basically:
Node is now (since v16.14+) requiring "assert" syntax when importing JSON as a module.
import fileJSON from 'file.json'  assert {type: 'json'};

So this is an issue to:

[x] add this syntax anywhere we import a JSON file
[x] drop support for older versions of node in package.json and the CI action
[x] remove eslint from the test command for now 😢

Just did this today - reminder that everyone working on this project probably now needs to upgrade their node.
nvm install 16.14
nvm alias default 16.14


Couch store security and maintenance

Perhaps two things could be added to the couch store database module:

security, e.g.: limit to a role
ongoing maintenance like compactation.

What do you think about that?

security, e.g.: limit to a role

Yea what were you thinking with this? Like adding roles to the _security document so only specific users had write access?

ongoing maintenance like compaction.

Yes it might be good to add some automatic/transparent processes so the end developer doesn't have to think about it. On the other hand, I set up autocompaction on my entire database so this would be a double-edged sword for me. I think this is okay as long as it is configurable and can be disabled.


VIC.5 docs reorg.

This PR takes care of most of the website reorganization for VIC.5. I took a fairly aggressive approach to removing documentation that was massively outdated.
After merging this, I'd like to initiate a standard that PRs include updates to the relevant docs.
Warning messages are printed on most Image Driver Pages:

@jhamman : How much review does this need? I don't really want to go through 125 files changed. Generally I am fine with this, so I am going to merge the PR and we can roll back what needs to be.

I think merging now (as you did).
Develop branch docs are now live: http://vic.readthedocs.org/en/develop/


EPIC: Data schema discussion ticket

Proposal for core model:
Extended version of the VFB model for images could work:
(C:Class:Neuron)<-[:INSTANCEOF]-(I:Individual:Neuron)<-[d:depicts]-(J:Image)
We could broaden the image slot to include other data types.
Evidence goes on the depicts edge - this could be an individual that aggregates information.
Experimental details can be connected to J.  I’m hoping we can link to external sources for this, as I worry we don’t have the resources to curate + I’m reluctant to wade in to details of experimental modelling.
With this approach we can add additional axiomatisation to ‘I’ that is useful for query/inference. As we learn more about I, we can add a more precise class.
This needs to be combined with a different approach for definitional (reference) data.

We already have individuals for dendrogram clusters with an exemplar (definitional) relationship to classes.

@cmungall  Comments please:  Should be be looking at BioLink instead?

Update - this will only be relevant if Allen adopts our pipeline infrastructure.


Envoy rejects the route config at start up time if a large number of xDS updates are happening

Title: *Envoy proxy rejects the route config at start up time if a large number of xDS updates are happening *
Description:
Envoy proxy rejects the route config at start up time if a large number of resources are there to configure from the control plane. For example if 1000 clusters and 5000 routes are to be created at start up time, it rejects the route configuration by returning the following error.
[libprotobuf WARNING external/com_google_protobuf/src/google/protobuf/text_format.cc:2166] type.googleapis.com/envoy.config.route.v3.RouteConfiguration: failed to parse contents
[2023-03-02 15:27:51.018][8][warning][config] [source/common/config/grpc_subscription_impl.cc:128] gRPC config for type.googleapis.com/envoy.config.route.v3.RouteConfiguration rejected: Unable to unpack as envoy.config.route.v3.RouteConfiguration: type_url: "type.googleapis.com/envoy.config.route.v3.RouteConfiguration"
value: << Route config goes in here >>

The value is filled with a bulky route config which spans over several hundred lines.
Even though it rejects the route config, eventually all the routes get created when I check from the config dump. This issue does not happen always for the above mentioned scenario with 1000 clusters and 5000 routes but happens like once in every 3 or 4 start ups. So for 1000 clusters and 5000 routes, the only issue is the start up time. Without the issue it only takes 1 to 5 seconds to configure the resources but with the issue it takes around 40 to 50 seconds to configure the resources.
The rate of this happening increases with the increase of resources. For 3000 clusters and 15 000 routes it happens always and sometimes it even takes like 20 mins to configure the resources.
We are using the Aggregated Discovery Service (ADS): SotW (aggregate stream for all resource types) to configure envoy proxy.
The following log shows envoy start up logs for a scenario with 1000 clusters and 5000 routes. From what I observed the following sequence of events occur.

Cluster manager gets initialised and starts adding the clusters.
Cluster manager sends cm init: all clusters initialized before initialising all the clusters (not sure whether it's an issue or the usual behaviour). In this scenario after 494 clusters, cluster manager sends cm init: all clusters initialized and starts adding the routes.
After that the lds adds the listeners and cluster creation and route creation happens at the same time. During this phase route config errors occur.  If I start up envoy proxy without any clusters and routes and then sequentially add clusters/routes with a sleep of 1 second between, this does not happen at all.  So I guess this is related to xDS protocol level concurrency or maybe I'm exceeding some buffer limit somewhere. Any idea about this issue ?
I can provide more logs, config or info and follow up with the issue if someone want to look into this further.

Yeah seems strange. cc @adisuissa

We are using envoy v1.25.1 and go-control-plane v0.11.0. I added xDS callbacks from the control plane side and went through the route config it provides to envoy proxy but couldn't find any issue with it.

That is very strange, and would like to understand better what is rejected as this seems more of a protobuf issue.
It may be interesting to look at what's being received and ensure that the contents is valid.
Do you use State-of-the-World or delta-xDS? For large scale configs it is better to use delta-xDS.
One thing to note is that if Envoy is subscribing to many RDS resources, there could be timeouts while waiting for these resources. For example after Envoy sends a request for 10000 routes, there is a timer started for each route (see config. When receiving a response with 10000 routes I think there may be a case where timeouts are triggered, and I wonder if what you are observing is related to these timeouts being triggered.

I'm also confused why the error comes from text_format decoder. Maybe some roundtrippnig issue to YAML?


bpf: Use ENABLE_L7_LB in complexity test

Add -DENABLE_L7_LB=1 to MAX_BASE_OPTIONS to pull that code in for load testing.
Fixes: #18894
Signed-off-by: Jarno Rajahalme<EMAIL_ADDRESS>

Rebased.

Rebased.

You will need to change files under bpf/complexity-tests/. The Makefile options are only used as the default options for compilation, not as the options for complexity tests.

ping @jrajahalme


Garrett - "Editati" button not functional on Safari

ENVIRONMENT: Validator.ui
URL: URL: Validator
Company_careers_url: Garrett
Desktop:
OS: Sonoma 14.6.1
Browser: Safari
Version: 17.6

STEPS TO REPRODUCE:

Type "Garrett" in the search field
Click on "Vizualizeaza Joburi" button
Scroll down to the "Indirect Site Buyer French" job listing
Click on "Editati" button
Select "hybrid" from the job type list
Click on "Editati" button

EXPECTED RESULTS:
The new values are added to the job listing details.

ACTUAL RESULTS:
The freshly inserted values are not being saved.
The popup window will not close unless clicking on the "x" icon.
Clicking the "x" icon does not save the new values.

ADDITIONAL INFO:
This issue has been noticed on several prepublished job listings: Program Manager, Senior EDI Specialist.
"Stergeti" button does not work on the prepublished job listings.

Fixed

Thank you!


chore: upgrade to Husky v7

What is the purpose of this pull request?

[ ] Documentation update
[ ] Bug fix
[ ] New rule
[ ] Changes an existing rule
[ ] Add autofixing to a rule
[x] Other, please explain:

What changes did you make? (Give an overview)
Used npx mrm@2 lint-staged to partially scaffold and re-ran prettier on all files

Trying to get this in now

Manually brought this in, thank you!

No worries


DataImport from SharepointWiki

We are planing to migrate existing sharepoint wiki data to Roadkill wiki. Is there any way to do that ?

You will have to do this by manually creating an importer. The schema is really simple in roadkill - just a pages table, and a page_content. It's not out of the box I'm afraid, and you may have problems with converting your Sharepoint markup to Markdown (or Creole but I'd advice sticking to Markdown).


Fix network retry integration test

Issue #, if available:
Fixes #1039
Description of changes:
Taken from @turan18's branch, this PR fixes the TestNetworkRetry logic.
The previous test worked strangely with iptables, and for some reason did not always process them. Additionally, the checks were not robust. So, even though it was always passing, in reality the errors were not being properly handled and thus always fell through. It also operated under the assumption that certain spans (specifically the ones associated with times binary) were fetched with background fetcher disabled, which we found to be incorrect. The setup also seemed to behave strangely on ARM machines, not catching the error and causing it to hang indefinitely. Still haven't figured out why yet.
This change fixes the logic by streamlining the read channels and only running one command, as no binaries should be expected to be fetched on container startup. This should also stop it from hanging as all buffered channel reads are now in select statements or goroutines.
This initially had issues and stayed hanging on ARM, but changing the timeout from 5 to 6 seconds seems to make it very consistent on ARM machines.
Thanks for the work done on this @turan18
Testing performed:
GO_TEST_FLAGS='-run TestNetworkRetry -count 1' make integration on ARM64 and AMD64
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

TL;DR I used testutil.LogMonitor to tie the stdout and stderr streams from sh.R to a new testreporter. This uses the same stdout and stderr that the logreporter uses, thus allowing sh.R's open pipes to be closed and the goroutines returned.
Ok, so I went back to the drawing board a bit and I think I found a solution I'm somewhat happy with. Instead of using our own goroutines that are not tied to any context or other structure, I opted to do what we previously did and just add a LogMonitor. This LogMonitor scans the given stdout and stderr readers and matches them with the writers for stdout and stderr, respectively. This fixes the issue we had previously with sh.R not properly exiting due to the pipes being left open.
I still kept the logic of not processing any of the data and just getting the raw string, though, i.e. any data from stdout = success, any data from stderr = failure. It felt a lot more elegant and removed the need to unmarshal into a json and whatnot.
This change was a comparatively significant refactor, and thus I opted to put it in its own commit so we can easily revert if needed, but I plan on squashing before we merge this.

The previous test attempted to check that both 1) reading files that are already downloaded works with no network and 2) reading files that are not already downloaded may or may not work depending on the retry policy/network interruption duration.

It's worth noting that, before changing to nerdctl run, I recall this test being fine. I was confused on why sh -c times worked before, though. Is it because with the entrypoint it would already have times loaded? But what's the guarantee there? Since as noted on ARM it did hang, which makes me wonder why that would be the case. Perhaps the image difference between architectures somehow made this a problem?

is there still value in a follow up test that verifies that the network can be interrupted after the image is downloaded?

Is this not what we're doing here though? Or do you mean fully download the image, wait for the background fetcher to finish, then run the snapshotter?

It's worth noting that, before changing to nerdctl run, I recall this test being fine. I was confused on why sh -c times worked before, though. Is it because with the entrypoint it would already have times loaded? But what's the guarantee there? Since as noted on ARM it did hang, which makes me wonder why that would be the case. Perhaps the image difference between architectures somehow made this a problem?

Yeah, the old logic was that by running the container before disabling the network, we force some amount of the image to be downloaded for sleep inf (probably span 0 which probably includes /bin/sh). We never had strong guarantees that would keep working, though.

is there still value in a follow up test that verifies that the network can be interrupted after the image is downloaded?

Is this not what we're doing here though? Or do you mean fully download the image, wait for the background fetcher to finish, then run the snapshotter?

The test now verifies that if we don't have the data we need and we have a network interruption, there will be retries until the network comes back and we can get the data.
The test that we lost is if do have the data we need, there is no network dependency and the container will work whether the network comes back or not (this is what we were trying to test with sh -c times although it didn't work reliably).
It doesn't have to be fully downloaded, but I think that makes it easy to make it more reliable. Assuming we had a way to determine that an image is fully downloaded, the test I'm imagining is:

Pull the image with the bg fetcher enabled
Wait for the image to be fully downloaded
Disable the network
run the container
Verify that the container runs successfully without requiring the network

I think it's possible. We could possibly just add something like this:
// Pull image
entrypointWithNetwork := ["sh", "-c", "times"]
sh.X(append(runSociCmd, "-d", "--rm", image, entrypointWithNetwork...)...)

// Disable network access here, or return if first test case
stdout, stderr, out := sh.R(append(runSociCmd, "-d", "--rm", image, entrypointWithNetwork...)...)) // In case any regression causes this to hang
// Setup listeners for this command

// Proceed with rest of test

There's no reasonable case for the first test to hang IMO, and the first run prefetches the required spans so the second command can then test our theory. WDYT?

LGTM. I don't think that blocks this PR, so we can defer that to a follow up issue.

Sure, will open that once this gets merged.

Squashed commits

I think it's possible. We could possibly just add something like this:

// Pull image

entrypointWithNetwork := ["sh", "-c", "times"]

sh.X(append(runSociCmd, "-d", "--rm", image, entrypointWithNetwork...)...)

// Disable network access here, or return if first test case

stdout, stderr, out := sh.R(append(runSociCmd, "-d", "--rm", image, entrypointWithNetwork...)...)) // In case any regression causes this to hang

// Setup listeners for this command

// Proceed with rest of test

There's no reasonable case for the first test to hang IMO, and the first run prefetches the required spans so the second command can then test our theory. WDYT?

Opened #1231 to track this


How to visualize the coco dataset

Hello, I wonder to know how to visualize the results?

You can refer to DINO visualization script as an example
https://github.com/IDEA-Research/DINO/blob/main/inference_and_visualization.ipynb


clean shutdown + default log level INFO

Two changes:

Implements CTRL+C and kill signal handler to kairos-server, which axum then uses to stop serving new connections and finish serving existing connections before shutting down.

Sets the default logging level to INFO, otherwise you can't see any logs unless you use the RUST_LOG environment variable.

If your using vscode please enable .editorconfig. It keeps the line endings consistent.
https://marketplace.visualstudio.com/items?itemName=EditorConfig.EditorConfig


waypoint: authorization selector

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
Describe alternatives you've considered
Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context

Ref: #42961

Is this done already?  Feels we have this selector in alpha.

This is being done via targetRefs now


where is result.json in the demo.py?

I came into this error:
File "./tools/demo.py", line 129, in
with open(save_path + '/results.json', 'w') as f:
IOError: [Errno 2] No such file or directory: './vis/data/results.json'

hi i got same problem
did you solved it?

hi i got same problem
did you solved it?

yeah, just create a folder 'data' under 'vis', it works for me, good luck!


Re: Missing Palauan word edchuul

Re: Missing Palauan word edchuul created by<EMAIL_ADDRESS>on 2017-01-20 16:21:10

<EMAIL_ADDRESS>replied, Here is the forwarded message because it looks like debugle cut it:
Hello , My Palauan friend gave me this nick name â€œ Edchuul â€œ . He said it is a title , meaning engineer , or someone who can fix anything . Sulang , Edchuul .


Using Table per Hierarchy with specification

I have two Classes called Product and Service that inherits from BaseProductService, TPH works fine but I dont know how to use it with specification and endpoints

What is the exact problem you are facing? I have tried TPC and it worked correctly. The issue I'm facing is OfType isn't supported yet for IQuerible in Specification

Sorry I figure it out, by calling the IRepository on the child class, because I couldn't find a proper documentation I was thinking that the way I was handeling the problem at hand could be wrong and I walking on a maze searching for a OfType alternative. Thanks.

When this feature is available in the next release. you can add a PostProcessingAction to filter the OfType.

That would be awesome


Add the first policy  which sends the request to the first available host

Hey,
as promised an implementation of the load-balancing algorithm first.
Should be help fixing:

https://github.com/mholt/caddy/issues/1463
https://github.com/mholt/caddy/issues/1103

Greetings

Thanks so much for your contribution, @pbeckmann !


stop words related to paedophilia appearing in random routes

Description of the Change
Removed words related to paedophilia from the dictionary used for random route generation.
Applicable Issues
#1261

@mkretz Thanks for making this PR!


[backport] Fix dali_util in imagenet example for fp16

Backport of #6342

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit c2a7fc6d10edd1771e27fe46aab88a90292b5ce4, target branch v5) succeeded!

@kmaehashi Would you check the failure on Travis?

LGTM! This PR will be tested by https://github.com/chainer/chainer-test/pull/494 .


chore [RUDOLPH-98] Use pdf library to check file without extension

Summary
Instead of manually checking filename with .pdf extension and return is_pdf = false -> use currently pdf library to read file content to decide if the file content is valid pdf

LGTM! I'd just suggest removing the logging when a file can't be loaded in pypdf. For a .doc, I see this:
INFO: Preparing to split document for partition.
ERROR: 'utf-8' codec can't decode byte 0xd0 in position 0: invalid continuation byte
WARNING: The file does not appear to be a valid PDF.
INFO: Partitioning without split.

If any non pdf is expected to hit this path, we can remove the scary ERROR and WARNING.


feat: Added joinUrl module

What does this change?
Adds the joinUrl function as a new module
Why?
This is used in DCR to prevent errors with missing or double slashes when concatenating strings to form urls and would be useful for other projects.

Coverage increased (+0.02%) to 99.469% when pulling 602a848fc2626d9d42d0b4fd731aee42048f71fa on oliver/joinurl into 9f25bea24275b8a4e4b6c96d5c72c2c0a2e48761 on main.

 joinUrl in @guardian/libs

I’d really like to know what the result of joinUrl('//example.com/','index.html') returns, as I fear it might not support an implicit protocol URL, starting with double-slashes.

	it('does not work with implicit protocol urls', () => {
		expect(joinUrl('//example.com/', '/index.html')).toBe(
			'/example.com/index.html',
		);
	});

I can add this test as an indication of the fact it won't support this 🤷♂. I can't think of a way to know when you'd want to preserve leading double slashes

Worth noting that we should probably generate some kind of warning for non-SSL pages?

Interesting point. I wonder if this kind of check would work better as a Github action higher up? Ot maybe a lint rule?

I can add this test as an indication of the fact it won't support this 🤷♂. I can't think of a way to know when you'd want to preserve leading double slashes

should it throw if it encounters one in this case? that way you can throw a meaningful error rather than return an invalid string?

// detect two or more `/` chars after a word boundary
const multipleSlashesInRoute = /\b\/{2,}/g;

export const joinUrl = (...args: string[]): string =>
	args.join('/').replace(multipleSlashesInRoute, '/');

This works beautifully! It achieves the same results but also allows for leading double slashes and allows for a trailing slash. And it does all this in a really simple, readable fashion (even as a regex 😉 )
Really great suggestion

From Regex101 on /\b\/{2,}/g :

\b assert position at a word boundary: (^\w|\w$|\W\w|\w\W)
\/ matches the character / literally (case sensitive)
{2,} matches the previous token between 2 and unlimited times, as many times as possible, giving back as needed (greedy)
g modifier: global. All matches (don't return after first match)

Could we use the native URL object instead (with the added benefit of a strongly typed objects)?


Concurrent Permissive Configuration

Adds support for configuring lane connections that are allowed to be concurrently permissive at the intersection level.
Adds a new data model for concurrent permissive lanes in the us.dot.its.jpo.conflictmonitor.monitor.models.concurrent_permissive package:

ConnectedLanes - A connected pair of lanes: ingress and egress
ConnectedLanesPair - A pair of connected lanes in the same intersection
ConnectedLanesPairList - A list of pairs of connected lanes that can be concurrently permissive

Adds a new intersection level property to MapSpatMessageAssessmentParameters:

concurrentPermissiveList of type ConnectedLanesPairList

Updates the Configuration REST API to be able to read, write, and delete configurations of the new data type, and also to be able to more generally handle configurations with complex data types by adding:

GenericJsonStringDeserializer - JSON Deserializer for generic types for the REST endpoint to use
WebConfg - Configures the REST endpoints to use the new deserializer.

Adds integration test scripts and unit tests to test reading and writing the new configuration types.
HTTP Test scripts are in:
/scripts/ConfigController_Tests/concurrent_permissive
Example POST request to configure concurrent permissive lanes for an intersection:
POST http://localhost:8082/config/intersection/10/101010/map.spat.message.assessment.concurrentPermissiveList
Content-Type: application/json

{
  "key": "map.spat.message.assessment.concurrentPermissiveList",
  "category": "",
  "roadRegulatorID": 10,
  "intersectionID": 101010,
  "value": [
    {
      "first": {
        "ingressLaneID": 1,
        "egressLaneID": 2
      },
      "second": {
        "ingressLaneID": 3,
        "egressLaneID": 4
      }
    },
    {
      "first": {
        "ingressLaneID": 9,
        "egressLaneID": 10
      },
      "second": {
        "ingressLaneID": 11,
        "egressLaneID": 12
      }
    }
  ],
  "type": "us.dot.its.jpo.conflictmonitor.monitor.models.concurrent_permissive.ConnectedLanesPairList",
  "units": "NONE",
  "description": "List of pairs of lane connections that are allowed concurrently permissive for an intersection.",
  "updateType": "INTERSECTION"
}

I updated the Conflict Monitor API for compatibility with the JSON type changes made in this PR.


Migrate all translations to formats.js

Checklist

[x] migrate translations of one or two pages;
[ ] guide PM to translate in Crowdin (doc?);
[ ] ~migrate rest translations;~ (split into tasks)

Breakdown into #3186 #3187 #3188 #3189 #3191


Update OptiFine

#53 Got closed
I might do it manually, could you assign this to me?

Sorry, I had to close the issue because it was basically getting hijacked. I'm sorry I haven't updated you on the topic in a while. I am working on a new version of the client, re-writing almost everything and using Optifine 1.8.8 HD U H8. It should come out fairly soon

Here's more info on the update https://telegra.ph/Whats-next-to-come-11-19

Let's just keep the latest optifine in the new version. let me add a SUPPORT.md based on GitHub's SECURITY.md

@OxideWaveLength the version of optimize used is basically ancient at this point.

just update yourself...


Grafana stuck on waiting - installing agent

Bug Description
In test run https://solutions.qa.canonical.com/v2/testruns/d4676f04-a422-47f1-a066-40a974baeebe, the deployment blocks with:
Model  Controller    Cloud/Region              Version  SLA          Timestamp
cos    popocatepetl  microk8s_cloud/localhost  2.9.37   unsupported  12:08:08Z

App           Version  Status   Scale  Charm             Channel  Rev  Address         Exposed  Message
alertmanager  0.23.0   active       1  alertmanager-k8s  edge      37  <IP_ADDRESS>  no
catalogue              active       1  catalogue-k8s     edge       4  <IP_ADDRESS>  no
grafana       9.2.1    waiting      1  grafana-k8s       edge      54  <IP_ADDRESS>  no       installing agent
loki          2.4.1    active       1  loki-k8s          edge      47  <IP_ADDRESS>   no
prometheus    2.33.5   active       1  prometheus-k8s    edge      86  <IP_ADDRESS>  no
traefik                active       1  traefik-k8s       edge      94  <IP_ADDRESS>    no

Unit             Workload     Agent  Address      Ports  Message
alertmanager/0*  active       idle   <IP_ADDRESS>
catalogue/0*     active       idle   <IP_ADDRESS>
grafana/0        maintenance  idle                       installing charm software
loki/0*          active       idle   <IP_ADDRESS>
prometheus/0*    active       idle   <IP_ADDRESS>
traefik/0*       active       idle   <IP_ADDRESS>

In the microk8s crashdump the pod logs for grafana are missing. There are some mentions of grafana in the microk8s logs, but nothing that indicates a clear failure to me.
Other logs and configs can be found here:
https://oil-jenkins.canonical.com/artifacts/d4676f04-a422-47f1-a066-40a974baeebe/index.html
To Reproduce
Deploy microk8s 1.26, add metallb, deploy cos
Environment
microk8s 1.26
cos-lite latest/edge
Relevant log output
https://oil-jenkins.canonical.com/artifacts/d4676f04-a422-47f1-a066-40a974baeebe/index.html

Additional context
No response

We did indeed get some messages complaining about memory and disk space. A larger VM seems to have fixed the problem. I will add the suggested kubectl output to the generated crashdumps.


Support for A2L Version 1.7.0

Hi! Is there maybe a chance you'll support the next version of the ASAM MCD-2 MC (ASAP2 / A2L) standard?

Hi,
I would love to but unfortunately I don't have access to the specifications.

pull request #37 added support on the lexer and parser level. This still needs to be interpreted and added to the object structure.


Merge Required! Updating ULRpng

Updating URLpng #21552
I want to change the LogoURL because i took the wrong Link
Error Automerge " duplicate token " appears but i dont know how to change fix this error.
"checkout the latest HEAD of main and then re-apply your change (simpler for git newbies but incurring a bit of duplicate work)"
is what im supposed to do but i dont understand how to do it i tried really hard..
new Link should be https://raw.githubusercontent.com/xLoopCreativeAndyx/-/main/logo.png
Original Pullrequest :#21191
May you please fix this manually Im looking to all Issues but i cant solve it myself PLS help me
@spacemandev-git @keone ty for your time

it just doesnt work at all pls merge this manaully im so lost

see this for help : https://github.com/solana-labs/token-list#duplicate-token


Rule-determined floating windows affect layout on creation

The LSP server for Emacs creates floating windows with tooltips containing useful information:

These frames are guaranteed to have an empty tile per https://github.com/emacs-lsp/lsp-ui/pull/252/, so following https://github.com/koekeishiya/yabai/issues/647 I used the rule
yabai -m rule --add app="Emacs" title="^\$" manage=off

This almost does the right thing, but the tooltip window is moved away from its original position and previous windows flash for a fraction of a second. If you add animation and increase the delay, you can see that yabai tries to tile the new window before floating it, which is pretty visually disorienting.

Fairly certain it should be title=^$ when matching an empty string. Rules are always applied before yabai attempts to tile the window.

Good catch, thanks. I was escaping the regex in my terminal when I initially tested it, but the behavior is the same after modifying .yabairc to use yabai -m rule --add app="Emacs" title="^$" manage=off and restarting the service.
I'll edit to include the config.

So you have this rule which makes all Emacs windows tile:
yabai -m rule --add app="Emacs" manage=on
From the docs:
manage=<BOOL_SEL>
Window should be managed (tile vs float).
Most windows will be managed automatically, so this should mainly be used to make a window float.

Does Emacs not tile correctly without that rule? If so, at least narrow down the types of Emacs windows that the rule will affect. I suppose on your scenario changing it to yabai -m rule --add app="Emacs" title!="^$" manage=on is more appropriate.

That's the fix! Thank you!
Correct config:
yabai -m rule --add app="Emacs" title!="^\$" manage=on
yabai -m rule --add app="Emacs" title="^\$" manage=off


getCharacterRanges seem to give inconsistent result

Dear @timdown ,
I have been able to replicate this problem but wasn't able to setup a plunkr. I will try to come back to create one later.
When there is the following html:

var

when the cursor is at the end of the
 block, the getCharacterRanges return position 12 instead of 4.
happens in Chrome.

This has been very consistent. I am going to dig into the source to figure out what's going on.
Any tips?

it seems dom.getNodeIndex somethings gives 0, sometimes gives the correct index of the root element in its' parent.
When it gives 0 the resulting start and end index is correct.
What is the purpose of this nodeIndex?

I think the bug is inside the boundary updater or the rangeBetween.rext() function.
basically when
rangeBetween.setStartAndEnd(parent, nodeIndex, this.startContainer, this.startOffset);
startIndex = rangeBetween.text(characterOptions).length;

The bug is in rangeBetween.text(characterOptions). This function returns a strange string that I have trouble finding in my html. (but should somehow come from my html).
uhm any tips?

I also get a DOM exception in safari.

Hi @timdown ,
I got the code to work by making the following changes:
                    if (rangeStartsBeforeNode) {
                        rangeBetween.setStartAndEnd(this.startContainer, this.startOffset, containerNode, 0);
                        startIndex = -rangeBetween.text(characterOptions).length;
                    } else {
                        rangeBetween.setStartAndEnd(containerNode, 0, this.startContainer, this.startOffset);
                        startIndex = rangeBetween.text(characterOptions).length;
                    }

Dear @timdown ,
I just submitted a pull request with the forementioned fix. Please let me know if this makes sense!!
Otherwise, I would also love to know if there is another way to fix this.
Cheers!!

I'm struggling to replicate the problem. Could you give me a bit more detail? more of the HTML and how to get the cursor in the problematic position would be ideal.

Hey Tim,
Here is an example html that is not editable:
<EMAIL_ADDRESS>The problem goes again if I refresh my editor.
If the
  block is just inserted, then rangy gives bad result as mentioned above.

Sorry if I'm being stupid but I just can't seem to get an error. On your page, I selected the final closing brace character, opened the console and called
rangy.getSelection().saveCharacterRanges(document.getElementsByTagName("pre")[0])
and got a character range with start 46 and end 47, which is correct. I used the same HTML in a unti test, tried placing the caret at various points and they all came out correct.
I'm sure there could well be a bug but I need to see it before fixing it or confirming your fix.

Hi Tim,
that page is actually using my patched version of rangy!!

Ahhh. I'll have another try based on your page later today then.

I had another go by wiping the rangy variable in the console and pasting in the current unpatched code. I still couldn't replicate the issue. Frustrating.

Hey Tim,
I will try to get back to this sometime this weekend. I suspect that it has
something to do with my app loading another copy of rangy inside of an
iframe.
Ge

Hi Tim, I tried to setup a jsfiddle for this, but the fiddle is working flawlessly.
Here is the link: http://jsfiddle.net/episodeyang/jk1s7adn/
I really want to solve this issue, would you mind have a look at my app? Here is the description:
<EMAIL_ADDRESS>

Okay, it is a problem with my app.
So what happened was the following:
Since my directive is working inside of an input/contenteditable element, each directive have to bind a handler to the parent focusable element.
Sometimes, the event that is passed into the event handler is undefined. this is when hell break loose.
Do you know why sometimes the event are undefined?
Right now I think this is the main issue, and after filtering out undefined events, my directive runs fine.

@timdown
YEAH!!!! I was able to reproduce the issue!!!!!!!

@timdown
Have you had a chance to look at this issue yet? the jsfiddle is able to reproduce the bug if you refresh often.
I am on a mac with Chrome.


fix: keyName column may not be present in pagination cursor

The laravel cursor parameter function throws an exception if the requested key is not present in the cursors' parameters.
However, when using withoutKeySort() on your CursorPaginator the key will not be there and we should ignore its absence.

Thanks for the PR. Any chance of a test that recreates the scenario?

Thanks for the PR. Any chance of a test that recreates the scenario?

done

Thanks! Released in 4.3.1.


InsecurePlatformWarning: A true SSLContext object is not available.

Hello, when trying to use livestreamer on OSX with the built-in python I get the following error:
livestreamer twitch.tv/sing_sing
[cli][info] Found matching plugin twitch for URL twitch.tv/sing_sing
/Library/Python/2.7/site-packages/requests-2.6.0-py2.7.egg/requests/packages/urllib3/util/ssl_.py:79: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
InsecurePlatformWarning
and then the program aborts. what is the issue here?

As the link in the error sugests, your Python version can't reliably be used for SSL connections. Either upgrade your Python or install an alternative SSL implementation.

I also have this problem on Mac OS, could you please give some direction how to fix it? I am not very skilled with python. Thanks

I had the same problem on Raspbian (Debian for Raspberry Pi). I was able to fix it by re-installing the requests package with this command:
pip install requests[security]

Assuming you have installed livestreamer through pip like me you can do the same. Although I have no idea how or even if these things work on Mac OS. Be warned: you need a working gcc and the Python header files installed.

Same problem here (Yosemite 10.10.3, python 2.7.9 installed. python -V gives 2.7.9).  I did:
sudo -H pip install --upgrade certifi pyopenssl requests[security] ndg-httpsclient pyasn1 pip sudo easy_install -U livestreamer livestreamer http://www.twitch.tv/day9tv best
and get:
[cli][info] Found matching plugin twitch for URL http://www.twitch.tv/day9tv /Library/Python/2.7/site-packages/requests-2.6.2-py2.7.egg/requests/packages/urllib3/util/ssl_.py:90: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.   InsecurePlatformWarning error: No streams found on this URL: http://www.twitch.tv/day9tv
Any clue somebody ?

After much tinkering (I'm not a coder, not a terminal god, etc. :x )  I found that for some reason it was working flawlessly on my MacBook Pro but I got this error on the Mac Mini. It was because at 1st, my installation of livestreamer got sent into my old Python folder (in /Library/Python/2.7/ etc.) and all the fix I did with Brew where in /usr/local/lib/python2.7 (upgrade of python, SSL & co). So I just deleted the old livestreamer install, launched the script again and FIXED. Hope this will help some people. Each case is VERY specific with so many possibilities.

chhe's solution worked for me (Ubuntu, installed via pip).

Getting this error
/usr/local/lib/python2.7/dist-packages/pip/vendor/requests/packages/urllib3/util/ssl.py:90: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
InsecurePlatformWarning
Error was resolved by
sudo pip install urllib3[secure] --upgrade

Install these packages if u get insecure platform warning or of you find somewhere ssl object not found Build essential,python dev,libffi-dev,libssl-dev
and then
run the command
sudo pip install urllib3
This should work

@97yogitha Thanks! urllib3 works for me!!

sudo pip install urllib3
can solve this question.
thanks

I have installed urllib3 and also upgraded but it is not resolved. It shows "This may cause the server to present an incorrect TLS certificate, which can cause validation failures. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#snimissingwarning."
What should i do so it run?

I have installed urllib3 and also upgraded but it is not resolved. It shows "This may cause the server to present an incorrect TLS certificate, which can cause validation failures. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#snimissingwarning."
What should i do so it run?

Thank you @chhe that worked for we as well


spanner: Add snippets for InstanceAdminClient

There were some errors in this request. I'll close this and send another separately.


partial content caching?

what would go wrong if i added HTTP_PARTIAL to CacheStrategy.isCacheable ?
thanks

Imagine two partial requests, one for the the range 0..100 and another for 100..200. The cache would need to reconcile the difference.


Problem on M1 Mac

Hi,
tested the 0.31-snapshot on M1 mac and it starts (after allowing in security manager) but then it runs into an error loop (s. attached file).
Hope that help. I have also downloaded the code and try to build it on my M1 Mac (with JavaFx 23).
Cheers Rainer

Can you please add the file?

Please see attached.
error.txt.zip

PS: I tried to build but failed with native image. it did run as a classical app.

This issue is probably related to #45. I don't have access to a M1 Mac at this time so I can't reproduce this.
2 Questions:

Does the app run normally when started with ./mvnw javafx:run -f sdkman-ui ?
If that works then can you run ./mvnw gluonfx:runagent -f sdkman-ui then close the app and then create a zip file of the sdkman-ui/src/main/resources/META-INF/native-image folder and attach it to this issue?

Hi,
I downloaded the main branch and set the GraalVM 17 as requested:

"./mvnw javafx:run -f sdkman-ui" -> works
"./mvnw gluonfx:runagent -f sdkman-ui" -> works
build with:  "gluonfx:build" -> fails -> s. attached error
alos META-INF attached
build_Error.txt
META-INF.zip

Tnx for the META-INF i've updated main, so it will become available as early access. Hope that will fix it.
The build error (linker error) you ran into is the same github actions is encountering, workaround is now also in the readme:
https://github.com/jagodevreede/sdkman-ui?tab=readme-ov-file#native

Hi,
I can now (with removing the .a file) compile and run it. However I still have the same problem with the your build as well as with my build (using your patch).

I think this is a problem related to the GraalVM Version. It seems this bug was only fixed with Version 21, while we are using the version 17 build. Going to test with other GraalVM Version.

I tried a new Gluon Version(s. https://github.com/gluonhq/gluonfx-maven-plugin/issues/189), still did not work ...

What version of osX are you testing? A college had the same issue but when he updated the os it was gone...

14.7.2


BUG?: PATCH needs an existing file.

Hi,
if I understand it properly the very basic flow of a Upload is as followed:

CLIENT sends HEAD request to ULR (e.g. /files/<FILE_ID>). SERVER responds with a 404 to indicate that the content does not exist.
CLIENT sends PATCH request to URL (e.g. /files/<FILE_ID>) with the full content of the file to upload.

I try to implement a client testing this very basic scenario and it does not work. I have added the source code to my server project to debug through and I found this line of code, which confused me:
https://github.com/tusdotnet/tusdotnet/blob/master/Source/tusdotnet/IntentHandlers/WriteFileHandler.cs#L197
This basically means that for the PATCH request the file must exist. It works well with the creation extension, but I thought that extensions are fully optional.

Hi,
Yes this is per the spec. creation is an optional feature of the protocol. You are free to create the resource to upload to with any type of request you wish if you do not wish to support creation. E.g. Vimeo is using a custom http request to create the upload resource and then uses tus for the actual file transfer.

Okay, thank you very much. I though that the Patch is like an implicit Create.

Unfortunately no. It will be in tus2 :)

Perfect. It works good with Create anyway. So I am fine with that.


Bad connecting

Once connected, the cell stops moving. It just stays there, even if a bigger cell runs over it.

guys please help me i dont want this mails -_-
2015-07-20 1:16 GMT+03:00 Anthony Thomas<EMAIL_ADDRESS>
Once connected, the cell stops moving. It just stays there, even if a
bigger cell runs over it.
—
Reply to this email directly or view it on GitHub
https://github.com/Apostolique/Agar.io-bot/issues/241.

Unsubscribe. There is a button on the right panel on this page.

I also have this issue. it will be fine one moment then super laggy the next. often the cell will stop, and when i press 'T' to control the bot, it will not let me. i have tested my internet connection that seems fine.

yea, there's nothing wrong with my internet either, its completely the script. either this script or the launcher is causing it.

This happens with me when I try to zoom.

Hey guys in the next days apostolique will work on a newer version and will try to fix this bugg.

awesome, thanks!!

I'm currently moving to a new house and I be able to work on this project until I'm fully settled. My computer and desk stuff aren't setup yet.


Office add-ins on mobile reccommends using deprecated (soon to be decomissioned) REST API

The office add-in mobile documentation directs users to use the Outlook REST APIs with their add-ins. The Outlook REST API was deprecated in 2020 and will be decommissioned in 2022.

https://devblogs.microsoft.com/microsoft365dev/outlook-rest-api-v2-0-deprecation-notice/

The makeEwsRequestAsync API is not supported on mobile since the mobile app uses REST APIs to communicate with the server. If your app backend needs to connect to the Exchange server, you can use the callback token to make REST API calls. For details, see Use the Outlook REST APIs from an Outlook add-in.

The deprecation notice recommends MS Graph as an altenative however the getAccessToken method provided by OfficeRuntime is part of the IdentityAPI 1.3 requirement set which is not supported on mobile.
The documentation needs updated with suggestion of what mobile office add-in developers should do instead of using the soon to be decomissioned REST API.

As a user of the REST API currently I'm not sure what the available migration path here.

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

ID: 564bc140-d31e-f042-bd30-b8140b7db416
Version Independent ID: 9953764b-39ca-d5c0-c1f8-5ca44ec2f472
Content: Outlook add-ins for Outlook Mobile - Office Add-ins
Content Source: docs/outlook/outlook-mobile-addins.md
Product: outlook
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Thanks for raising this concern @westconference.
@JuaneloJuanelo and @ElizabethSamuel-MSFT, could you please provide guidance around mobile alternatives for REST APIs?

Hi @westconference, we are currently reviewing our APIs to determine how they will be impacted by the REST API deprecation planned for 2022. We will communicate recommended guidance broadly in our documentation once we are done with our review.
Thanks.

Any updates?

Hi @ckollipa-adm,
We are still in the process of reviewing and updating our API documentation. I will update this issue as soon as this task has been completed.
In the meantime, please be advised that Outlook add-ins using REST V2 created before November 2022 are eligible for an exemption. If this is something you'd like to opt into, please register your add-in at https://aka.ms/RESTCheck.

Hi, @ckollipa-adm!  One note I have for you: you can call an Azure AD login page to get graph access for mobile devices.  However, I will note it is a much degraded user experiences vs using REST tokens or GRAPH SSO tokens.  Especially if you're add-in is meant to do small/quick actions related to the message.  The login page/process becomes the majority of add-in interaction time.
This demo has a login page fallback example: https://github.com/OfficeDev/Office-Add-in-samples/tree/main/Samples/auth/Outlook-Add-in-SSO/AttachmentDemo
Personally, I would much prefer that IdentityAPI 1.3 be added to the mobile clients and manual login only be needed for older clients, such as 2016.
@samantharamon the form you linked, can it be used for Private add-ins or is it only intended for add-ins that are in AppSource?

Great question, @GrumpyAL . I've verified that the form applies to both private add-ins and add-ins in AppSource.

@westconference @ckollipa-adm @GrumpyAL Thank you for your patience. Documentation on mobile Outlook add-ins using the REST service is still an ongoing task in review, but I wanted to provide an update. To add to my previous post, active add-ins that use the REST service will be granted an exemption to keep using the service until extended support ends for Outlook 2019 on October 14, 2025. This includes new add-ins that are developed after the decommission date in November. @JuaneloJuanelo provided an update on this in this month's Office Add-ins community call and answered questions, including mobile-related ones, which may interest you. Additional questions raised by the community during the call will be answered in a blog post and a link will be posted to Microsoft Office Add-ins community call, so watch out for that, too.

@samantharamon after reading through the August community call, I have a question.  Will this exemption process now just apply automatically or do we still need to opt in for any new or existing add-ins using the above form?  Our concern is that we have rare occasions where the appID may get changed before the add-in is deployed.

@GrumpyAL thanks for raising this question. The team is currently testing automatic add-in traffic identification, so that you won't need to submit the form for your new and existing add-ins to be exempted. During this testing phase, and especially since your add-in has certain conditions, I encourage you to submit a form to participate in testing. This will allow the team to identify and account for certain edge cases.

The office add-in mobile documentation directs users to use the Outlook REST APIs with their add-ins. The Outlook REST API was deprecated in 2020 and will be decommissioned in 2022.

https://devblogs.microsoft.com/microsoft365dev/outlook-rest-api-v2-0-deprecation-notice/

The makeEwsRequestAsync API is not supported on mobile since the mobile app uses REST APIs to communicate with the server. If your app backend needs to connect to the Exchange server, you can use the callback token to make REST API calls. For details, see Use the Outlook REST APIs from an Outlook add-in.

The deprecation notice recommends MS Graph as an altenative however the getAccessToken method provided by OfficeRuntime is part of the IdentityAPI 1.3 requirement set which is not supported on mobile.
The documentation needs updated with suggestion of what mobile office add-in developers should do instead of using the soon to be decomissioned REST API.

As a user of the REST API currently I'm not sure what the available migration path here.

Hi @westconference,
I wanted to provide an update on your original post. Since getAccessToken isn't supported in Outlook on mobile devices, we recommend using nested app authentication (NAA) instead to call services such as Microsoft Graph. NAA is currently in preview in Outlook on Android and on iOS. Here are some resources to help you learn more about NAA.

Enable SSO in an Office Add-in using nested app authentication (NAA)
Office Add-in Sample: Outlook add-in with SSO using nested app authentication (preview)
Nested app authentication and Outlook legacy tokens deprecation FAQ


Evaluate PushLink For Non-Release Sample Distribution

Auto-deployed via Maven plugin by Cloudbees Jenkins builds?

This is not feasible due to its requirement of the internet permission. It's also largely unnecessary for this type of library.