Refactor tag assertion to FatalError check & Fix unexpected increase of tag (fork)

This is just a fork of https://github.com/jonhoo/rust-imap/pull/231 so it can be finished and merged.

This change is

One test fails but it is not related to this PR.

Ready for a second review.

Let's get this merged

Awesome, thank you! Do you think it could be possible to release a new
alpha version? I need this fix to release a new version of mine.

Yep, it's coming shortly! I'm preparing it over here: https://github.com/jonhoo/rust-imap/pull/285
The specific doc changes I made are these: https://github.com/jonhoo/rust-imap/pull/285/commits/ed433ec5745a3dfb1aaecc22bbbe5d834d5c33dd

Published as 3.0.0-alpha.13 :tada:


Name versions on export

In GitLab by @jakebeal on Jan 27, 2020, 14:09
Google Docs allows naming of versions. When we export, it would be good to link the export to a version name (probably added automatically)

In GitLab by @jakebeal on Apr 27, 2020, 09:10
assigned to @tramyn

In GitLab by @jakebeal on Apr 27, 2020, 09:11
Finished during the implementation of golden files for #184

In GitLab by @jakebeal on Apr 27, 2020, 09:11
closed


Plugin is not working well on newest version of Atom after introducing Docks

Plugin doesn't work well with the newest Atom version because of new panels (docks): http://blog.atom.io/2017/05/16/atom-1-17.html
Moreover, it overlaps keymaps with core ones, so using ctrl+tab shortcut opens right pane instead.
Current behaviour: when you try to use tab switcher, black overlay with opacity appears and nothing more. After releasing tab, Atom's right dock appears (collision with  'ctrl-tab': 'pane:show-next-recently-used-item' - and you can't really rewrite that...)

Same issue for me, could you solve it?

Same issue here.


Problem using float type on Windows 10

(Using Vale 0.1.1)
Using the following test program float_operations.vale:
fn main() export {
  a = 42.9;
  b = 108.4;
}

Compiling this with python ..\valec.py build float_operations.vale gives the compilation error:

When pushing any of the Abort - Retry - Ignore buttons the terminal shows:
Using Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC.
Running: java -cp C:\Vale\Valestrom.jar net.verdagon.vale.driver.Driver build -o build.vast float_operations.vale
Running: C:\Vale\valec.EXE --verify --output-dir . build.vast
**valec couldn't compile build.vast:
Region override: assist
Warning: not using census when in assist mode!
Assertion failed: C->getType()->isFPOrFPVectorTy() && Ty->isFPOrFPVectorTy() && "Invalid cast", file C:\Users\Valerian\Downloads\llvm-project-llvmorg-11.0.1\llvm\lib\IR\Instructions.cpp, line 3069**
The same program running Ubuntu 20 under WSL in Windows compiles and runs without problems:

When running the test program round.vale from samples, I get the exact same error on Windows. However in Ubuntu 20 (WSL) I also get a compilation error:

Same error occurs when:

  pi = 3.1415;
  println(pi.str());

Fixed as of #208, and verified on windows!


Cannot find namespace 'Microsoft'.

Thanks for your hard work!
I've hit a compilation error and I don't know how to solve it. Can you take a look at this for me?
I'm using Angular 4.0.1, Angular-CLI 1.0.0, and your module @markpieszak/ng-application-insights 1.0.0-rc.1
Please let me know if you need more information.
ERROR in<EMAIL_ADDRESS>(4,23): Cannot find namespace 'Microsoft'.
<EMAIL_ADDRESS>(5,51): Cannot find namespace 'Microsoft'.
<EMAIL_ADDRESS>(37,14): Cannot find namespace 'Microsoft'.
<EMAIL_ADDRESS>(39,13): Cannot find namespace 'Microsoft'.
<EMAIL_ADDRESS>(70,24): Cannot find namespace 'AI'.
<EMAIL_ADDRESS>(1,1): Cannot find type definition file for 'applicationinsights-js'.

Here is the relevant part of my app.module file:
import { ApplicationInsightsModule, AppInsightsService } from '@markpieszak/ng-application-insights';
@NgModule({
  imports: [
    ...
    ApplicationInsightsModule.forRoot({ instrumentationKey: 'My-Key' })
  ],
   ...

  providers: [
     ...
    AppInsightsService,
   ...
  ]

Here is the relevant part of my component:
import { AppInsightsService } from '@markpieszak/ng-application-insights';
...

constructor(
   ...
   private appInsightsService: AppInsightsService) {}

...
doSomething() {
     this.appInsightsService.trackPageView('WelcomePage')
}

I am having the exact same issue when trying to upgrade my app to ng4

Also experiencing the same problem

Does adding "@types/applicationinsights-js":to thedevDependencies` in the package.json help fix this one? I be,I've I need to add it to the dependencies of the library here!

Adding @types/applicationinsights-js to the devDependencies does the trick indeed. I haven't tried to add to your library though.  Only to my ng4 app.

Yeah I think that'll handle it, let me add it in the project and push a new version to npm!

Published 1.0.0, let me know if there's still any problems 👍
@aguerot @hakonamatata @MrTimms

@MarkPieszak I no longer see this compile time error. I am seeing a couple of compile time warnings and a run time error when I do an --aot build. I'll start a new thread for that issue.


multi-character delimiter not supported

Q
A

Version
9.6.1

Question
Hello!
I have to generate a file that uses three characters as delimiter (don't ask why -_- ).
Any reason thephpleague/csv doesn't support multi-character delimiters?
An exception is thrown at
League\Csv\AbstractCsv::setDelimiter() expects delimiter to be a single character ;@/ given

  at C:\www\illico_feed_generator\vendor\league\csv\src\AbstractCsv.php:397
    393▕
    394▕             return $this;
    395▕         }
    396▕
  ➜ 397▕         throw new InvalidArgument(sprintf('%s() expects delimiter to be a single character %s given', __METHOD__, $delimiter));
    398▕     }
    399▕
    400▕     /**
    401▕      * Sets the field enclosure.

Thanks!
A.

@aand18 thanks for using the library. As explained by the exception only single character are supported as delimiter. Nevertheless, you may use the stream filter capabilities of the package if you want to use multi-bytes delimiter characters. For how to do it please refer to stack overflow or any other PHP help forum as this is an expected behaviour.

@nyamsprod thank you for pointing me to some solutions!
Sorry if may fail to see the bigger picture, I'm not intimately familiar with the source code.
Looks like it shouldn't make a difference, just splitting a string by three chars instead of one.
I was wondering if there was maybe a historic reason to force single byte?
Thanks

@aand18 parsing and generating CSV is done using underlying PHP mechanism and it is not done using splitting characters please refer to PHP documentation about CSV functions to see how it is done.


Published 0.6.1 AMIs are ECS optimized

RancherOS Version: (ros os version)
0.6.1
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
AWS
I had to search for what appears to the the correct AMI on the AMI marketplace.
Edit: I've only tested the AMI for us-east-1: ami-af7d0bb8  It appears the correct AMI for us-east-1 is ami-a8d2a4bf.

@ryansch Thanks! I had mixed up the tables between our Amazon ECS ones and regular ones. I've updated it now.


Sitecore Publishing Service

Great job!
Would be nice if you guys could create a Dockerfile(example) for Publishing Service

Check out PR  in docker-images repo for publishing service. Includes the /modules/ps-assets as well as how to add the publishing service to sql and cm roles.
I think docker-examples is more for using images rather than building them.

When publishing service is supported in provided base images, we will add an example here.


Query returns incorrect records in associations when the foreign key is omitted from select

Not sure if this was already considered or fixed but it seems to be pretty important to be worth checking.
Environment

Elixir version: Elixir 1.8.2 (compiled with Erlang/OTP 21)
Database and version: PostgreSQL 10.12
Ecto version: 2.2.12
Database adapter and version: postgrex 0.13.5
Operating system: elementary OS 5.1.2 Hera

Current behavior
Consider a query which returns records with two levels deep of associations and uses a select: clause. If the select: does not include the relevant foreign key field, Ecto silently accepts the query and returns the result where parent records have incorrect children (and grandchildren) attached. The children get attached to parents in such a way that all parents receive the same single child which happened to be returned first.
When the resultset does not include the foreign key field, Ecto, naturally, cannot associate the records correctly (which it should know), but it silently proceeds nevertheless, and returns an inconsistent result.
This does not happen when there is no select: clause because then all fields are included and the result reconstruction proceeds correctly. This also does not happen in a simpler parent-child query where the children are simply not included in the result.
Schema
  +-------------------+          +-----------------+          +------------------+
  | Patient           | 1      1 | User            | 1      n | Document         |
  |                   +--------->+                 +--------->+                  |
  +-------------------+          +-----------------+          +------------------+

  schema "patients" do
    field :first_name, :string
    field :middle_name, :string
    field :last_name, :string

    belongs_to :user, User
  end

  schema "users" do
    field :email, :string

    has_one :patient, Patient, on_delete: :delete_all
    has_many :documents, Document, on_delete: :delete_all
  end

  schema "documents" do
    field :filename, :string

    belongs_to :user, User
  end

No select in the query
  # Note that the query starts from the Patient and not from the User,
  # that is in the `belongs_to` -> `has_one` -> `has_many` order.
  # This is dictated by the realities of the schema and may or may not
  # be relevant to the issue.

  def patient_summary_query do
    documents_query = from d in Document

    from p in Patient,
      join: p_u in assoc(p, :user),
      preload: [
        user: {
          p_u,
          documents: ^documents_query
        }
      ]
  end

SELECT p0."id", p0."first_name", p0."middle_name", p0."last_name", p0."user_id", u1."id", u1."email" FROM "patients" AS p0 INNER JOIN "users" AS u1 ON u1."id" = p0."user_id" []
SELECT d0."id", d0."filename", d0."user_id", d0."user_id" FROM "documents" AS d0 WHERE (d0."user_id" = ANY($1)) ORDER BY d0."user_id" [[24, 23, 1, 21, 2]]

The result is correct with all the associations in their right places:
[
  %Patient{
    __meta__: #Ecto.Schema.Metadata<:loaded, "patients">,
    first_name: "Jerrod",
    id: 1,
    last_name: "Gislason",
    middle_name: "Ansel",
    user: %User{
      documents: [
        %Document{
          __meta__: #Ecto.Schema.Metadata<:loaded, "documents">,
          filename: "test1.jpg",
          id: 1,
        },
        %Document{
          __meta__: #Ecto.Schema.Metadata<:loaded, "documents">,
          filename: "test1.jpg",
          id: 2,
        },
        %Document{
          __meta__: #Ecto.Schema.Metadata<:loaded, "documents">,
          filename: "test1.jpg",
          id: 3,
        }
      ],
      id: 1,
      ...
    },
    user_id: 1
  },
  %Patient{
    __meta__: #Ecto.Schema.Metadata<:loaded, "patients">,
    first_name: "Patient3",
    id: 4,
    last_name: "Test",
    middle_name: "3",
    user: %User{
      documents: [],
      id: 23,
      ...
    },
    user_id: 23
  },
  %Patient{
    __meta__: #Ecto.Schema.Metadata<:loaded, "patients">,
    first_name: "Patient4",
    id: 5,
    last_name: "Test",
    middle_name: "4",
    user: %User{
      documents: [],
      id: 24,
      ...
    },
    user_id: 24
  }
]

Select skips the foreign key
  def patient_summary_query do
    documents_query = from d in Document

    from p in Patient,
      join: p_u in assoc(p, :user),
      preload: [
        user: {
          p_u,
          documents: ^documents_query
        }
      ],
      select: [
        :id,
        :first_name,
        :middle_name,
        :last_name,
        user: [
          :id,
          :email,
          documents: [
            :id,
            :filename
          ]
        ]
      ]
  end

SELECT p0."id", p0."first_name", p0."middle_name", p0."last_name", u1."id", u1."email" FROM "patients" AS p0 INNER JOIN "users" AS u1 ON u1."id" = p0."user_id" []
SELECT d0."id", d0."user_id", d0."filename", d0."user_id" FROM "documents" AS d0 WHERE (d0."user_id" = ANY($1)) ORDER BY d0."user_id" [[24, 23, 1, 21, 2]]

Note how the result lists the user_id: nil but the user is in fact attached, the same one to every patient:
[
  %Patient{
    __meta__: #Ecto.Schema.Metadata<:loaded, "patients">,
    first_name: "Jerrod",
    id: 1,
    last_name: "Gislason",
    middle_name: "Ansel",
    user: %User{
      documents: [],
      id: 24,
      ...
    },
    user_id: nil
  },
  %Patient{
    __meta__: #Ecto.Schema.Metadata<:loaded, "patients">,
    first_name: "Patient3",
    id: 4,
    last_name: "Test",
    middle_name: "3",
    user: %User{
      documents: [],
      id: 24,
      ...
    },
    user_id: nil
  },
  %Patient{
    __meta__: #Ecto.Schema.Metadata<:loaded, "patients">,
    first_name: "Patient4",
    id: 5,
    last_name: "Test",
    middle_name: "4",
    user: %User{
      documents: [],
      id: 24,
      ...
    },
    user_id: nil
  }
]

A query with the correct select clause
  def patient_summary_query do
    documents_query = from d in Document

    from p in Patient,
      join: p_u in assoc(p, :user),
      preload: [
        user: {
          p_u,
          documents: ^documents_query
        }
      ],
      select: [
        :id,
        :first_name,
        :middle_name,
        :last_name,
        :user_id,          # <- added
        user: [
          :id,
          :email,
          documents: [
            :id,
            :filename
          ]
        ]
      ]
  end

Expected behavior
It would be nice if Ecto did not attach associated records when there is not enough data to do so. Ideally, it would also be nice to have a warning that the field on which it attempts to match the records is not specified in the select.

Unfortunately this is really hard to verify in practice because all the preloader sees is that fields are nil, and it is completely fine for them to be nil as it means they have no associated data. We do document you need to be careful to select all relevant data when using custom queries - it is an advanced feature and it requires attention when using it. In any case, thanks for opening the issue!

@josevalim I understand the detection for the purpose of warning may be unfeasible, but Ecto does return inconsistent data, which is a different story.

In an attempt to track why data gets unrelated I was able to introduce a warning. The secret was looking at the data, and not in trying to validate the query!

@josevalim Excellent! Thanks!

hey @josevalim, we are seeing this error occur in our phoenix service, and are not using select: in our queries. it seems non-deterministic. we're doing a chained preload of a few associations, deep, our preload tree looks like (where letters are record types)
A
  belongs_to(B)
  belongs_to(C)
    belongs_to(B)

The error seems to be getting raised for the first level B reference. I'm not sure, but seems possible the fact the record type B is involved at two levels of the heirarchy might be an edge case. Haven't been able to repro in a test case yet, but might be obvious to an ecto expert what is going on. For now, we're going to roll back to previous version since this doesn't seem like a real problem ecto is raising, but the error mis-firing. (Though maybe it is surfacing a non-deterministic bug!)

Hi @gfodor! 👋
I can see the repo where you see this failure is public. Can you please give me a commit SHA and a command to run that reproduces the failure? Then I can take a look at it and cut a new Ecto version.

Cool thanks! Unfortunately this one doesn’t have a repro at the moment,
it’s non deterministic in production. I’ll see if I can come up with one.
Was fishing to see if the “Feynman technique to solving problems” was
potentially applicable given the preload chain I wrote above :)
On Sun, Apr 19, 2020 at 2:00 AM José Valim<EMAIL_ADDRESS>wrote:

Hi @gfodor https://github.com/gfodor! 👋
I can see the repo where you see this failure is public. Can you please
give me a commit SHA and a command to run that reproduces the failure? Then
I can take a look at it and cut a new Ecto version.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/elixir-ecto/ecto/issues/3246#issuecomment-616080541,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABVW5EJNZH56D7TWPKNYSLRNK4TDANCNFSM4LJKRRTA
.

Hi @josevalim 👋, just want to follow up the issue @gfodor mentioned above.
The case is when building a nested FormData from changeset.
# Document has_many :attachments ( in document changeset, we do preload :attachments)

changeset = Document.changeset(%Document{attachments: [%Attachment{}]})
# <- And it will error out
(RuntimeError) association `attachments` for `Document` has a loaded value but its association key `id` is nil. This usually means `id` was not selected in a query

How should we handle this?

@technicalcapt does it come with a default attachment for view purposes? So you have something to show by default? Maybe you can do that only in your new/edit actions and now when you actually persist your changeset? I.e. not on create/update.

In any case, I have migrated this to a warning, so everyone has a easier time migrating to it and can fix things without a broken app. Thanks for the input everyone!

@josevalim unfortunately viewing is not the case. We wanted to create the form dynamically with two level nested from changeset.
I will investigate more about this and yeah a warning will work.
Thanks a lot!


question: change timezone in config/app.php?

I am not sure, if I should change 'timezone' in config/app.php to my local timezone? How is it actually used?

Hi @jeyca,
Really, You should not edit any of the files in config/, Only make config changes via the .env file.
The timezone config item you have found is not actually shown often, Most dates shown to the user are in a relative format.

If you have access to the php.ini file you can set the timezone - http://php.net/manual/en/datetime.configuration.php#ini.date.timezone

Will now close this since question was answered and no further response was requested.


why training iteration is 220K

Hi, thanks for your great work.
I am confused that training iteration mentioned in your paper is 220k instead of 150k(80k for flying chairs and 70k for flying things).
Can you help me out here?

Please see the updated version of the paper:
http://www.contrib.andrew.cmu.edu/~gengshay/wordpress/wp-content/uploads/2019/11/vcn.pdf
We train on chairs/things with more iterations in the final version, but it turns out not all numbers in the paper have been updated accordingly. Thanks for pointing it out.

Please see the updated version of the paper:
http://www.contrib.andrew.cmu.edu/~gengshay/wordpress/wp-content/uploads/2019/11/vcn.pdf
We train on chairs/things with more iterations in the final version, but it turns out not all numbers in the paper have been updated accordingly. Thanks for pointing it out.

thanks for replying


No auto-build after command plugin modifies source files

Describe the bug
While having the "auto-build project" feature on, I noticed a command plugin changing the files doesn't trigger a new build.
To Reproduce
Trigger a command plugin which modifies Swift files.
Observe no new build is triggered.
Expected behavior
A new build is triggered.
Environment
VS Code Version: 1.93.1
Additional context
Add any other context about the problem here.

The same thing is true for "Swift: Clean Build Folder" as well.


docs wran

Describe the bug

Reproduction

System Info
https://cn.vitest.dev/guide/features.html

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

CN docs has a separate repo: https://github.com/vitest-dev/docs-cn


[stable/airflow] private git repo doesnt seem to work

BUG REPORT
I am trying to setup this chart using a private repo
the doc says you can use private repo by uploading private ssh keys as kube secrets
however, i noticed that the git-clone.sh is not starting ssh-agent
so the initContainer is still failing on authentication
i am missing a step to start ssh-agent?
chart: stable/airflow

Nope!  I created a password-less key so I didn't have to deal with ssh-agent, since it's mostly intended for GitHub repo-specific deploy keys with read-only access, not for uploading your primary SSH key.

i am using a passphrase-less read-only ssh key as well but i am not able to get it to work without starting the ssh-agent

Seem like a configuration issue.

@kalyanaramansanthanam were you able to figure this out? How do you start the ssh-agent, and if it is not needed, what is the proper configuration to do this?

can't help here since we have shelved using this chart for now and decided to take a DIY approach to managing the airflow service


Fix open button position

https://github.com/cardstack/boxel/assets/12637010/8f1234ba-8352-4b35-9d82-9aae74b869ec

[x] Fix the "Open" position button.
[x] Fix the "Open" button z-index problem. The fix requires some refactoring so we can put an overlay button in each card.

Is it time yet to consider a dedicated library for this like https://popper.js.org/ ?

@jurgenwerk I think so.

@lukemelia I decided to use ember-velcro instead of calculating the position of the overlay button. https://github.com/cardstack/boxel/pull/429/commits/5e8f1dea0200ea269633a29c3cf54f1ba9ac6233


support NAT loopback in masquerade mode.

What this PR does / why we need it:
Issue: when use masquerade for vm. I want to access vm by pod IP in my vm (known as NAT loopback/hairpin). But current nftable rule can not. Current DNAT just only applied to eth0 interface, which just do DNAT for external traffic. But NAT loopback fails.
Fix: The DNAT rule should apply to all interface.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Special notes for your reviewer:
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[x] PR: The PR description is expressive enough and will help future contributors
[x] Code: Write code that humans can understand and Keep it simple

Release note:

Support NAT loopback in masquerade mode.

/test pull-kubevirt-unit-test
/test pull-kubevirt-e2e-k8s-1.28-sig-network


Fix code scanning alert - Inclusion of functionality from an untrusted source

Tracking issue for:

[ ] https://github.com/judaicalink/judaicalink-labs/security/code-scanning/14

Fixed in beta.


distgit: add SOURCE_DATE_EPOCH env var

Ref. https://reproducible-builds.org/docs/source-date-epoch/
Also https://github.com/openshift/machine-config-operator/pull/744#issuecomment-492046675

/lgtm


[cloud_functions]Remove internal FIR methods

Internal methods caused build errors https://github.com/flutter/flutter/issues/21973 and they should not be used anyway!

Looks like this was addressed in fc5fab0e40050c1d9f0ae30792eacbe9af120606 but didn't remove the internal header reference; I've opened https://github.com/flutter/plugins/pull/1747 for that.


Folder Upload

I'm probably missing it but I've been digging through the docs for a while, figured I would ask here as if I'm having trouble finding maybe someone else is. Is there support for folder uploads or will I just have to build a simple wrapper? We have folder uploads so I was hoping to be able to work with files and folders using all GCP native functions but not a big deal if there is not support for this at this time.
Thanks!

Hey @1tron1, we unfortunately don't have support for this at the moment - but it would be an awesome feature.
If you'd like to open a PR that would be fantastic, otherwise I am sure we can get on it soon.

Sure thing. Quick question though. I was uploading files with a simple wrap:
php 	function upload_file($storage,$queue,$file){ 	  $bucket = $storage->bucket($queue); 	  $bucket->upload( 	    fopen($file, 'r')); 	}
Was wondering if with the bucket-> upload method there is a way to specify "folder path") into the bucket, or if I have to use bucket->object->insert() which I have seen done? Going to need to know to do this, and wanted to know if there was a nice terse method, the ones I have seen are a bit long.

@dwsupplee Will add this soon, quick unrelated question though. What is the best way get the file contents from a GCP bucket in php?

Assigning it to @vishwarajanand as they can decide on the FR. The mentioned PR was closed citing a reason.


[Bug]: Error Update AL Go System File (401 Unauthorized)

AL-Go version
5.3
Describe the issue
Hello,
I'm trying to update the AL Go System Files, but I keep getting an error Unauthorized. Do not know the reason, the last time I was able to sync without any problem was a month ago.
Downloading BcContainerHelper latest version from Blob Storage
Import from C:\ProgramData\BcContainerHelper\6.0.22\BcContainerHelper\BcContainerHelper.ps1
BcContainerHelper version 6.0.22
BC.HelperFunctions emits usage statistics telemetry to Microsoft
Running on Windows, PowerShell 5.1.20348.2652
Applying settings from D:\a\D365BC\D365BC.github\AL-Go-Settings.json
TemplateUrl: https://github.com/microsoft/AL-Go-PTE@main
TemplateSha:
DownloadLatest: True
The remote server returned an error: (401) Unauthorized.
Unauthorized
...retrying in 1 minute
...retrying in 1 minute
Retry failed as well
The remote server returned an error: (401) Unauthorized.
Unauthorized
Applying settings from D:\a\D365BC\D365BC.github\AL-Go-Settings.json
Applying settings from D:\a\D365BC\D365BC.AL-Go\settings.json
No settings found in D:\a\D365BC\D365BC.github\Update AL-Go System Files.settings.json
No settings found in D:\a\D365BC\D365BC.AL-Go\Update AL-Go System Files.settings.json
No settings found in D:\a\D365BC\D365BC.AL-Go\DesarrolloBC.settings.json
Enabling Microsoft telemetry...
Error: Unexpected error when running action. Error Message: The remote server returned an error: (401) Unauthorized.   Unauthorized , StackTrace: at InvokeWebRequest, D:\a_actions\microsoft\AL-Go-Actions\v5.3\Github-Helper.psm1: line 112 <- at DownloadTemplateRepository, D:\a_actions\microsoft\AL-Go-Actions\v5.3\CheckForUpdates\CheckForUpdates.HelperFunctions.ps1: line 31 <- at , D:\a_actions\microsoft\AL-Go-Actions\v5.3\CheckForUpdates\CheckForUpdates.ps1: line 72 <- at , D:\a_temp\904b80da-900a-4996-aa7a-25bc84b1fb26.ps1: line 3 <- at , D:\a_actions\microsoft\AL-Go-Actions\v5.3\Invoke-AlGoAction.ps1: line 17 <- at , D:\a_temp\904b80da-900a-4996-aa7a-25bc84b1fb26.ps1: line 2 <- at , : line 1
Error: Process completed with exit code 1.
Expected behavior
To update the system files correctly.
Steps to reproduce
Create a new action, specifying the needed parameters and execute.
Additional context (logs, screenshots, etc.)
No response

Can you check if:

You have a secret called GHTokenWorklow in your repo secrets (or an org secret shared with your repo)
That secret isn't expired


fix: escape html correctly

The .text constructor was a misnomer, as this was not a "text node" in the sense that HTML/DOM defines the phrase.
This renames text to raw (which it appears is only needed in one place) and adds a new text (with escape-on-output semantics) that is used in almost every other place.
Without this change,
def foo («<script>alert("hello world")</script>» : Nat) : Nat := 0

results in javascript injection.
The instance : ToString Html is removed because this makes it too easy to accidentally doubly-escape something.

Do you consider also removing Coe String Html? I don't know if it's implicitly used.

The top bar is now visually broken and using the search feature does not work anymore, probably because some escaping broke stuff. As reported here by users: https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/Documentation.20loading.20issue/near/443714091

I think I can look at this later, but not today...

I fixed this in #192


⚠ ClamAV has degraded performance

In a3eaec2, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 207 ms

Resolved: ClamAV performance has improved in b3a5e14 after 18 minutes.


Error: tx parse error

Hi, All!
I'm a beginner in javascript, and i try to create and send a transaction using js sdk, I get an error message. Here is the code:
const BnbApiClient = require('@binance-chain/javascript-sdk');
const api = 'https://testnet-dex.binance.org/'; /// api string
const bnbClient = new BnbApiClient(api);
var rawTx = {
account_number: 1
, chain_id: 'Binance-Chain-Nile'
, memo: 'my memo'
, msg: [
{
"type": "CreateOrderMsg"
, "data": {
"test": "test_data"
}
, "value": {
"id": "19F0A5A49A790C68B4C0C1092A5B0B07A33C1850-55844"
, "ordertype": 2
, "price": "96773"
, "quantity": "4400000000"
, "sender": "bnb1r8c2tfy60yxx3dxqcyyj5kctq73ncxzsfndxk4"
, "side": 1
, "symbol": "COTI-CBB_BNB"
, "timeinforce": 1
}
}
]
, type: 'StdTx'
, sequence: 29
, source: 0
};
const privateKey = BnbApiClient.crypto.generatePrivateKey()
const publicKey = BnbApiClient.crypto.generatePubKey(privateKey);
console.log('publicKey: ', publicKey);
let msg_tx = rawTx;
var tx = new BnbApiClient.Transaction(rawTx);
signature_trx = tx.getSignBytes(tx);
console.log('signature_trx: ', signature_trx);
tx = tx.addSignature(publicKey, signature_trx);
console.log('1 tx: ', tx);
tx = tx.sign(privateKey, msg_tx);
console.log('2 tx: ', tx);
bnbClient.chooseNetwork('testnet');
bnbClient.setPrivateKey(privateKey);
bnbClient.initChain();
result = bnbClient.sendTransaction(tx, true);
console.log('3 result: ', result);
Error message:
....
3 result:  Promise {  }
(node:36004) [DEP0079] DeprecationWarning: Custom inspection function on Objects via .inspect() is deprecated
(node:36004) UnhandledPromiseRejectionWarning: Error
at /Users/alexandrdenisov/Downloads/node_modules/@binance-chain/javascript-sdk/lib/utils/request.js:71:19
at process._tickCallback (internal/process/next_tick.js:68:7)
(node:36004) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)
(node:36004) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
(node:36004) UnhandledPromiseRejectionWarning: Error: tx parse error
at /Users/alexandrdenisov/Downloads/node_modules/@binance-chain/javascript-sdk/lib/utils/request.js:71:19
at process._tickCallback (internal/process/next_tick.js:68:7)
(node:36004) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 4)
What's wrong? Please help me

Also i try:
sz_tx = tx.serialize();
result = bnbClient.sendRawTransaction(sz_tx, true);
But result and error the same


gnupg package does not provide gpg program

despite the bash hook suggesting that it should.
[anders@gurney:~] $ gpg
The program ‘gpg’ is currently not installed. It is provided by
several packages. You can install it by typing one of the following:
  nix-env -i gnupg
  nix-env -i gnupg1compat
[anders@gurney:~] $ nix-env -i gnupg
installing ‘gnupg-2.1.2’
[anders@gurney:~] $ gpg
The program ‘gpg’ is currently not installed. It is provided by
several packages. You can install it by typing one of the following:
  nix-env -i gnupg
  nix-env -i gnupg1compat
[anders@gurney:~] $ nix-env -e gnupg
uninstalling ‘gnupg-2.1.2’
[anders@gurney:~] $ nix-env -i gnupg1compat
installing ‘gnupg1compat-0’
[anders@gurney:~] $ gpg
gpg: Go ahead and type your message ...
^C
gpg: signal Interrupt caught ... exiting

That's because there are multiple derivations called gnupg. Any 2.x gnupg
provides only thr gpg2 binary while gnupg 1.4 provides gpg. Gnupg1compat
provides gpg2 as gpg.
On Mar 7, 2015 5:28 PM, "anderspapitto"<EMAIL_ADDRESS>wrote:

despite the bash hook suggesting that it should.
[anders@gurney:~] $ gpg
The program ‘gpg’ is currently not installed. It is provided by
several packages. You can install it by typing one of the following:
nix-env -i gnupg
nix-env -i gnupg1compat
[anders@gurney:~] $ nix-env -i gnupg
installing ‘gnupg-2.1.2’
[anders@gurney:~] $ gpg
The program ‘gpg’ is currently not installed. It is provided by
several packages. You can install it by typing one of the following:
nix-env -i gnupg
nix-env -i gnupg1compat
[anders@gurney:~] $ nix-env -e gnupg
uninstalling ‘gnupg-2.1.2’
[anders@gurney:~] $ nix-env -i gnupg1compat
installing ‘gnupg1compat-0’
[anders@gurney:~] $ gpg
gpg: Go ahead and type your message ...
^C
gpg: signal Interrupt caught ... exiting
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/issues/6706.

It seems like sticking a version number into a binary name is a very "un-nixy" thing to do, even if they do so by default. Can we just change it? It seems like an artifact of a world where all packages share the same namespace.

That might create issues where programs who think they need gnupg-2 try to call gpg2. As long as it's common (as of https://www.gnupg.org/documentation/manuals/gnupg/Invoking-GPG.html) to name the binary in a gnupg-2 installation gpg2, we should stick to it

For reference #1126

#1126 seems to be exactly the same as what I'm reporting. Feel free to close as a dup.

Duplicate


Distinguishing the Java version for installing on the Mac OS X

Install Java on the Mac OS X with version node['java']['jdk_version']

Thanks!


Improve MySQL database detection and creation

A better way to test for MySQL being available is to use a mysqli connection.
Compared to the current implementation, this has the advantages of:

not requiring the MySQL client to be installed.
will work with non-local databases.
will work on Windows and Linux.

Test for MySQL server availablity
<?php
$servername = "<IP_ADDRESS>";
$username = "root";
$password = "";
$port = 3306;

// Create connection
$conn = new mysqli($servername, $username, $password, null, $port);

// Check connection
if ($conn->connect_error) {
    die("Connection failed: " . $conn->connect_error);
}
$conn->close();

Create a new database
// Create database
$sql = "CREATE DATABASE {$dbname}";

if ($conn->query($sql) === TRUE) {
    echo "Database created successfully";
} else {
    echo "Error creating database: " . $conn->error;
}

I love this!


No jobdetail information for workflows

Impact of the bug
P&R ACDC databases need information regarding exitCodes to display information about failures across the system and submit ACDCs.
Describe the bug
The wmsstatsserver jobdetail endpoint appears empty for some workflows, some times.
This  is how we read it, but even navigating to it via browser shows an empty page, so I don't believe it's an issue of how we read it.
How to reproduce it
Some examples:

https://cmsweb.cern.ch/wmstatsserver/data/jobdetail/pdmvserv_task_BPH-RunIISummer20UL18GEN-00237__v1_T_230926_104347_8763
https://cmsweb.cern.ch/wmstatsserver/data/jobdetail/wangz_ACDC1_task_GEN-Run3Summer22EEwmLHEGS-00479__v1_T_231201_223414_5800
https://cmsweb.cern.ch/wmstatsserver/data/jobdetail/cmsunified_task_EXO-RunIIFall18GS-05548__v1_T_230915_125606_6488

Expected behavior
Here is some examples of something that I would expect

https://cmsweb.cern.ch/wmstatsserver/data/jobdetail/llavezzo_task_EXO-Run3Summer22MiniAODv4-00112__v1_T_231110_180302_2038
https://cmsweb.cern.ch/wmstatsserver/data/jobdetail/pdmvserv_task_GEN-Run3Summer22EEwmLHEGS-00361__v1_T_231012_195915_7504

There is an extra issue which might be related. The Exit code isn't properly parsed so it leads to:

Error message without line changing
example: https://cms-unified.web.cern.ch/cms-unified/report/cmsunified_ACDC0_task_EXO-RunIIFall17GS-07823__v1_T_231115_005644_458

No Exit code:
example: https://cms-unified.web.cern.ch/cms-unified/report/wangz_r-0-Run2017G_DoubleMuon_UL2017_MiniAODv2_BParking_240104_200314_2623

Another example: https://cms-unified.web.cern.ch/cms-unified/report/wangz_ACDC1_Run2018D_EGamma_UL2018_MiniAODv2_BParking_240108_221907_5689
No error report

Another example with a special case, which may help resolve this problem.
https://dmytro.web.cern.ch/dmytro/cmsprodmon/workflows.php?prep_id=task_HIG-Run3Summer22MiniAODv4-00153
In this WF, ACDC2 and ACDC5 shows 8001 error with error "[a] Fatal Root Error: @SUB=TTreeCache::FillBuffer".
ACDC3 and ACDC4 don't have exit code.
It's the same error and somehow in ACDC3 and ACDC4 there is nothing returned.

@lucalavezzo @z4027163 after investigating this workflow

No Exit code:
example: https://cms-unified.web.cern.ch/cms-unified/report/wangz_r-0-Run2017G_DoubleMuon_UL2017_MiniAODv2_BParking_240104_200314_2623

I can see from submit8 that we actually had the same issue as the one reported in:
https://github.com/dmwm/WMCore/issues/11771#issuecomment-1793165161
That issue has been resolved with a 2nd iteration fix, but it was not backported to the agents that we are currently swapping out of production (version 2.2.5).
The new agent that started getting rolled into production by the end of the past week - version 2.3.0 - should have this 2nd iteration fix and I do not expect to have job report data loss again.
For the moment, I would suggest to close this issue; and if we see lack of job details for workflows that started after Jan 20th, 2024, we should then reopen this issue.
For the workflows missing job details, unfortunately we cannot recover those details. So I would suggest to use the documents in the ACDC server for any followup workflow recovery.
Please let us know what you think and/or if you have any questions.

Hi Alan,
Thank you for the explanation!
However, I want to clarify something: the wmstatsserver pages for these workflows (linked in issue description) are completely empty, it's not that we don't see exitCodes, we don't see anything. Is that exactly the same issue as missing exitCodes?
We will monitor this in the next couple of weeks to see how it develops, as workflows that started on the 20th of Jan will begin to finish. Can we please keep the issue open until we can confirm that this is issue is really gone?
Thank youo,
Luca

@lucalavezzo Hi Luca, yes, it's the same problem, but triggered in a slightly later stage in the component logic. The impact is that those documents are not retried (they are larger than the limit of 8MB) and we don't even try to make them smaller.
With the latest fix, we will try to reduce the file, if possible, and upload it to couchdb.

@amaltaro Just so that we understand on our end, are there cases then in which no exitCodes are expected? (e.g. very large documents). We really depend on this information for some fundamental aspects in our databases, like the lists tasks in each workflow, so we need to understand whether we can expect that this will work 100% of the time, or we need to build systems to deal with the hopefully occasional issue that we would expect. FYI @z4027163

@lucalavezzo I reviewed the two PRs fixing this issue:
https://github.com/dmwm/WMCore/pull/11502/files
and
https://github.com/dmwm/WMCore/pull/11786/files
and I am almost sure that we will have something reported in WMStats (including the exit code). It's not impossible that I am mistaken and/or missing something as well.
If you can identify error messages missing in WMStats, then we shall discover that in the coming weeks/months. What do you think?

Perfect, thank you very much for all your detailed responses :)
We will keep an eye on this issue and report back to you.

Hello @amaltaro,
This workflows Failure rate is 46% but no Logs for a workflow in wmstats.

cmsunified_task_PPD-Run3Summer22wmLHEGS-Backfill-00016__v1_T_240125_170259_2990

Hi @hassan11196 ,
we restarted AgentStatusWatcher component for voms0281, and this makes the jobdetails log appear in WMStats.
Thanks,
Andrea

Ciao @anpicci,
Thank you for your prompt response!
Does this mean that we can expect temporary 'outages' in the jobdetails from time to time?
Or is this a leftover issue that was fixed by the latest fix that Alan mentioned, which for some reason wasn't applied to this job/agent?
Again, we are just trying to understand how to set up our systems to best interplay with yours.
Best,
Luca

@lucalavezzo this is an unexpected issue with database replication. We can definitely not prevent it from happening in the future, or having one of the agent's components down. So yes, there could be a period of time where job documents don't get replicated to WMStats, but the system usually recovers itself before you can even spot those.

@amaltaro thank you very much for the information. If it's okay, we will keep this ticket open for another week or two just to be 100% sure, but I am glad this issue seems resolved.

@lucalavezzo we discussed this issue in today's WMCore meeting and Hassan thinks it can now be closed. I am going to close this out, but feel free to reopen it if this same problem occurs again. Thanks!

@amaltaro Thank you, sounds good to me.


proposal: sync: add a HasRun method to sync.Once

In some cases, a different code path should be taken depending on if sync.Once() has been executed (example below). It is possible to work around this issue by using a mutex, but Once can return this information more efficiently
func (o *Once) HasRun() bool {
    return atomic.LoadUint32(&o.done) == 1
}

Use case:
In gonum's implementation of a multivariate Normal distribution, we only compute and store the covariance matrix if necessary. Most commonly, users need to compute probabilities and generate random numbers, neither of which actually need the covariance matrix. Computing and storing the matrix is O(n^3) operation and O(n^2) respectively, which can be a significant additional cost. We have a setSigma method that uses a sync.Once to compute and store the covariance matrix if it's actually required. This is fine, except there are also operations which only need a single element of the covariance matrix, for example getting the marginal distribution along the single dimension. If the covariance matrix has already been computed (due to some other method call), then it's an O(1) operation to return the marginal. If it has not been computed, then a single element of the covariance matrix can be computed in O(n) time, instead of forcing the entire (n^3) operation of the covariance matrix.
At the moment, we have the code
func (n *Normal) MarginalNormalSingle(i int, src *rand.Rand) distuv.Normal {
	var std float64
	if n.sigma != nil {
		std = n.sigma.At(i, i)
	} else {
		// Reconstruct the {i,i} diagonal element of the covariance directly.
		for j := 0; j <= i; j++ {
			v := n.lower.At(i, j)
			std += v * v
		}
	}
	return distuv.Normal{
		Mu:     n.mu[i],
		Sigma:  math.Sqrt(std),
		Source: src,
	}
}

// setSigma computes and stores the covariance matrix of the distribution.
func (n *Normal) setSigma() {
	n.once.Do(func() {
		n.sigma = mat64.NewSymDense(n.Dim(), nil)
		n.sigma.FromCholesky(&n.chol)
	})
}

These two methods cannot be called concurrently, because checking that n.sigma == nil directly races against n.Once setting n.sigma.  It is possible to work around this using a mutex, with something like
func (n *Normal) sigmaNil() bool {
    n.RLock()
    b := n.sigma == nil
    n.RUnlock()
    return b
}

and a similar transformation to setSigma, but it would be really nice to just have the MarginalNormal code be
func (n *Normal) MarginalNormalSingle(i int, src *rand.Rand) distuv.Normal {
   var std float64
   if n.once.HasRun() {
   	std = n.sigma.At(i, i)
   } else {
   	// Reconstruct the {i,i} diagonal element of the covariance directly.
               std = ...
   }
}

This seems ripe for misuse. And even in your example code it looks like you're doing it wrong.
Your code permits two goroutines to both see HasRun() return false and then do duplicate initialization work.
You should instead do:
func (n *Normal) MarginalNormalSingle(i int, src *rand.Rand) distuv.Normal {
     n.once.Do(n.initStd)
     std := n.std
     ....
}

func (n *Normal) initStd() {
    ....
}

I don't understand. Your initStd (which is our setSigma), computes the entire covariance matrix, which computes O(n^2) matrix elements and takes O(n^3) time. If the covariance matrix is not set, instead MarginalNormalSingle computes one matrix element in O(n) time. That is a significant cost savings. Yes, it's possible that two different goroutines could compute the same marginal, which would duplicate work, but it takes O(n^2) goroutines to compute the same marginal before you lose overall.

Sorry, I didn't read all your code. I just responded looking at your final snippet.
You can work around this without any new API. Just use atomic.StoreInt32 in your 	n.once.Do(func() { and LoadInt32 to check whether it's done.
I don't think we want to encourage such racy checks. We also didn't accept Mutex.TryLock in #6123.

You can work around this without any new API. Just use atomic.StoreInt32 in your n.once.Do(func() { and LoadInt32 to check whether it's done.

By which you mean replicating the code that's in sync.Once?

Agreeing with @bradfitz . This should not go in.
sync.Once is here exactly to handle concurrent invocations correctly. Every time you provide a simple predicate that doesn't also atomically handle an action depending on the predicate, you are producing racy code in a concurrent environment.
If you still wanted that predicate, you can do exactly as Brad suggested: simply use an atomic store to set the predicate in the function you provide to sync.Once.

Okay.

Sorry, I thought I understood yesterday, but I don't think I do anymore.
@bradfitz says that "my code permits two goroutines to see HasRun() as false and do duplicate initialization work". This is not true. sync.Once ensures the initialization work only happens once. In my case here, the code in sync.Once is expensive, but does not necessarily need to happen. If it has happened, then the code uses that work as a shortcut, but if it has not happened it can avoid the work by avoiding the data modified by sync.Once entirely.
@griesemer : Isn't having sync.HasRun exactly atomically handling an action depending on the predicate? If the predicate has run, then it does one way, and if it hasn't, it avoids that memory altogether. HasRun is (proposed as) atomic, so it's not a race condition in the "should set off the race detector" sense of the word.
I'm not necessarily trying to change the argument, but you both seem to be suggesting that the above code is bad. I don't understand why. It's not a race condition if the check happens atomically, and it increases the efficiency (in big O) for almost all executions of the code.

@btracey Think about the code you'd write using the HasRun predicate: Even if HasRun returns false, by the time the caller checks the result of HasRun (which is still false), the code may actually have run (and if you'd call HasRun again it would return true) - hence the race condition.

Right, I understand that. This is why there needs to be protection with an atomic or mutex.
The code that uses the answer returns the same number in either condition, it just uses a different means to acquire the number. On the one hand, I can see the argument that this kind of unpredictability is bad, but on the other  using sync.Once in the first place doesn't seem all that different -- it's impossible to tell if a specific goroutine will get to use the fast path or have to wait for the call to f() to conclude.
If the argument is just "this is too dangerous a tool to expose", then okay, but it seemed like there was an additional "and your code is bad and you should change it" and I'm trying to understand why.

@btracey It's a dangerous tool to expose exactly because it can't be used correctly without yet another synchronization mechanism. So why not use just one mechanism instead?

Well, with HasRun exposed it would be the same synchronization mechanism (the done field in Once), while without the method another (or altogether different) synchronization mechanism is required. However, I understand your point that HasRun gives the impression the call either has or hasn't run, while in reality all that can be said is that the call either has definitely run or it has maybe run.


Clicks pass through when refocussing a window that's lost focus on OS X

Standard usability patterns on OS X dictate that clicking on a window which is not focussed will refocus the window without triggering any action on that window (except for the close/minimize/maximize Buttons). You can observe this pattern on Safari, App Store, Slack, Atom, Spotify, Mail, and almost all other Mac apps.
VS Code seems to have overlooked this usability pattern. I've attached a video to demonstrate how this creates a bad user experience.
Disruptive.zip

VSCode Version: 1.24.1
OS Version: OS X 10.13.5

Steps to Reproduce:

Open VS Code
Open a new application (VS Code will lose focus)
Click a button/anywhere on VS Code (while it is not focussed)
Button click is received

Expected Behavior:

Open VS Code
Open a new application (VS Code will lose focus)
Click a button/anywhere on VS Code (while it is not focussed)
Button click is ignored, VS Code regains focus
Click again to trigger button click

Does this issue occur when all extensions are disabled?: Yes

@IsaiahJTurner you will see that this pattern is not "standard" by any means if you e.g. run TextEdit and see that you can click on elements in the toolbar without making the Window focused first. The same is true with Xcode for example.
We decided that it makes sense for an editor to be usable without making the window focus first. And it seems other editor related applications behave the same from Apple. If you do not like that you can turn it off via "window.clickThroughInactive": false


wip: docs(README.md): minor improvements

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

/kind release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area tests

/area proposals

/area CI

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
/milestone 0.36.0
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM, let me know when ready to move out of wip!

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [LucaGuerra,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


OS Management Extension Not Available in Rancher v2.9

In a fresh install of Rancher v2.9 (not Rancher Prime) I can't find the OS Management Extension, I see only Kamaji as an available extension.
I saw in this issue https://github.com/rancher/dashboard/issues/11372 that it suggests the OS Management Extension is only available in Prime, however when I look at the "Detailed Comparison" of Rancher SKUs here: https://www.rancher.com/products/rancher-platform
It shows all three versions should have OS Management with Elemental

Can you please clarify and make the appropriate fix in one place or another?

@KarmaPoliceT2 The built Elemental extension is only available for Prime:

The source code is open source, so you can build the extension and load that into Rancher (Rancher or Rancher Prime).

Please re-open if you have more questions


Could not request Youtube.

Le bridge Youtube ne fonctionne plus, et indique "Could not request Youtube".
Testé sur mon instance ainsi que celle de Mitsu.

Youtube a (encore) blacklisté les adresses IP de OVH: il demande un captcha pour accéder. La bridge fonctionne sans problème quand je teste en local (Free). Pas de solution en code donc.

Je relance ici parce que je suis chez ovh et mon serveur est banni mais sans captcha cette fois.
Our systems have detected unusual traffic from your computer network. Please try your request again later. Why did this
happen?
This page appears when Google automatically detects requests coming from your computer network which appear to be in violation
of the Terms of Service. The block will expire shortly after those requests stop.
This traffic may have been sent by malicious software, a browser plug-in, or a script that sends automated requests. If you
share your network connection, ask your administrator for help — a different computer using the same IP address may be
responsible. Learn more
Sometimes you may see this page if you are using advanced terms that robots are known to use, or sending requests very
quickly.
Time: 2015-11-19T12:50:01Z
URL: http://www.google.com/
Une idee du delai ou d'un moyen de contourner ?


RDKBDEV-2999: drop unnecessary buffer zeroing

Reason for change:
drop unnecessary buffer zeroing
Code Cleanup.
Test Procedure: Sanity.
Risks: None.
Signed-off-by: Andre McCurdy<EMAIL_ADDRESS>

Hi @guto86 ,
Can you please review and merge this PR?

Internal ticket for tracking https://ccp.sys.comcast.net/browse/RDKCOM-5142


No logo shown during reboot / shutdown

Related to https://github.com/balena-os/meta-balena/issues/1291


RDM-2168 Definition store API : Use Liquibase through pipeline

JIRA link (if applicable)
RDM-2168
Change description
Definition store API : Use Liquibase through pipeline
Does this PR introduce a breaking change? (check one with "x")
[x] Yes
[ ] No

Merged as part of #23


Permissions of translations after download are bad

Describe the bug
I was trying to set up a new way of building our app with the latest fetched translation from Crowdin.
I added the following action:
     - name: ⤵ Download translation from Crowdin
         uses: crowdin/github-action@v1
         with:
           upload_sources: false
           upload_translations: false
           download_translations: true
           create_pull_request: false
           push_translations: false
         env:
           CROWDIN_PERSONAL_TOKEN: ${{ secrets.CROWDIN_TOKEN }}

Very reasonable, I would say (although it is weird, that creating a PR is the default, instead of opting in.
However, after this step my build failed with:
$ lingui extract && lingui compile
node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

[Error: EACCES: permission denied, open 'src/locales/en-GB/messages.po'] {
  errno: -13,
  code: 'EACCES',
  syscall: 'open',
  path: 'src/locales/en-GB/messages.po'
}

That was unexpected. After adding some more debugging I got to see the file permissions created:
app/src/locales/nl:
total 464K
565681 4.0K drwxr-xr-x 12 runner docker 4.0K May 31 11:01 ..
565689 4.0K drwxr-xr-x  2 runner docker 4.0K May 31 11:04 .
545200 448K -rw-r--r--  1 root   root   448K May 31 11:04 messages.po
545199 8.0K -rw-r--r--  1 runner docker 5.5K May 31 11:01 countries.json

The .po file is created by Crowdin, the countries.json is from the repo. As we can see the .po file is now belonging to root (not the current user, "runner") and because of that they are no longer writeable explaining the issue above.
To Reproduce
Steps to reproduce the behaviour:

Download translations files with GitHub action
Try to open these files for writing

Expected behavior
Downloaded files shoudl have sane default permissions and ownership

For other people encountering this, my work around is to add the following line after the action has run:
- name: 🧱Fix permission situation Crowdin created
        run: |
          sudo chmod -R 666 app/src/locales/**/*.po

@Bertg do you have any ideas on how to implement this on the action side? The user's files might have different permissions and we'd like to avoid adding a new option to the action configuration. Also, I don't recall similar problems with permissions so it might be something more specific to your setup. Thank you!

@andrii-bodnar Honestly I have no idea here. We have a very plain setup. We just run as runs-on: ubuntu-latest and use uses: actions/checkout@v4 to check out our source code. We have no special permissions.

In this setup when I run touch some_file in a bash script, my files are created with runner docker permissions. (both before and after running the crowdin action). I think the action should make new files using the same permissions the default bash scripts would.
The files were initially runner docker and after the action is run they are root root. As a bare minimum the action could try to keep the permissions the same if it is going to overwrite existing files.

If I were to guess, this may be part of the Docker setup for this action? Maybe something can be changed there?

I just found a similar discussion in the past - https://github.com/crowdin/github-action/discussions/181#discussioncomment-6525429
It looks like having a separate step to customize the permission is fair in this case. For most Action users, the current behavior seems fine


But what is a pipeline version?

I've read the sdk-overview and Using the SDK to create a pipeline and a pipeline version  but I am still none-the-wiser regarding what pipeline version is. The name suggests its iterative changes on an existing pipeline. But what is a pipeline version yaml where do I learn about it and how to use it?

/assign
In Kubeflow Pipelines 0.2.5 and higher, you can add a new version of a pipeline rather than creating a completely new pipeline record for each version. Here is an example that uses the SDK to create a pipeline and then add a new version of that pipeline.


Consider Switch to GRAAL-VM

Incorporates 64_bit Java Virtual Machine into .exe
Avoids Java runtime environment installation
Requires 64-bit adaptor .DLL

Joe to confirm with additional adapters. (Vector next).

Vector does not support a 64 bit RP1210 adapter.

Is there a 32-bit equivalent?

There is no 32 bit equivalent.

I guess we wait then, and ask Peter, when that might happen


Testsuite does not work on wildfly because DiscoveryServlet casts non File to File

14:46:58,960 ERROR [io.undertow.request] (default task-29) UT005023: Exception handling request to /testsuite/discovery/app_pages: java.lang.ClassCastException: org.jboss.vfs.VirtualFile cannot be cast to java.io.File
	at com.sap.openui5.DiscoveryServlet.listClasspathResources(DiscoveryServlet.java:243) [testsuite-utils-1.30.2.jar:]
	at com.sap.openui5.DiscoveryServlet.init(DiscoveryServlet.java:94) [testsuite-utils-1.30.2.jar:]
	at javax.servlet.GenericServlet.init(GenericServlet.java:244) [jboss-servlet-api_3.1_spec-1.0.0.Final.jar:1.0.0.Final]
	at io.undertow.servlet.core.LifecyleInterceptorInvocation.proceed(LifecyleInterceptorInvocation.java:117) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at org.wildfly.extension.undertow.security.RunAsLifecycleInterceptor.init(RunAsLifecycleInterceptor.java:79)
	at io.undertow.servlet.core.LifecyleInterceptorInvocation.proceed(LifecyleInterceptorInvocation.java:103) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.core.ManagedServlet$DefaultInstanceStrategy.start(ManagedServlet.java:220) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.core.ManagedServlet.getServlet(ManagedServlet.java:163) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.ServletHandler.handleRequest(ServletHandler.java:84) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.FilterHandler$FilterChainImpl.doFilter(FilterHandler.java:130) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at com.sap.openui5.CORSFilter.doFilter(CORSFilter.java:64) [testsuite-utils-1.30.2.jar:]
	at io.undertow.servlet.core.ManagedFilter.doFilter(ManagedFilter.java:60) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.FilterHandler$FilterChainImpl.doFilter(FilterHandler.java:132) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at com.sap.openui5.CacheControlFilter.doFilter(CacheControlFilter.java:55) [testsuite-utils-1.30.2.jar:]
	at io.undertow.servlet.core.ManagedFilter.doFilter(ManagedFilter.java:60) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.FilterHandler$FilterChainImpl.doFilter(FilterHandler.java:132) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.FilterHandler.handleRequest(FilterHandler.java:85) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.security.ServletSecurityRoleHandler.handleRequest(ServletSecurityRoleHandler.java:61) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.ServletDispatchingHandler.handleRequest(ServletDispatchingHandler.java:36) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at org.wildfly.extension.undertow.security.SecurityContextAssociationHandler.handleRequest(SecurityContextAssociationHandler.java:78)
	at io.undertow.server.handlers.PredicateHandler.handleRequest(PredicateHandler.java:43) [undertow-core-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.security.SSLInformationAssociationHandler.handleRequest(SSLInformationAssociationHandler.java:131) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.security.ServletAuthenticationCallHandler.handleRequest(ServletAuthenticationCallHandler.java:56) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.server.handlers.PredicateHandler.handleRequest(PredicateHandler.java:43) [undertow-core-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.security.handlers.AbstractConfidentialityHandler.handleRequest(AbstractConfidentialityHandler.java:45) [undertow-core-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.security.ServletConfidentialityConstraintHandler.handleRequest(ServletConfidentialityConstraintHandler.java:63) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.security.handlers.AuthenticationMechanismsHandler.handleRequest(AuthenticationMechanismsHandler.java:58) [undertow-core-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.security.CachedAuthenticatedSessionHandler.handleRequest(CachedAuthenticatedSessionHandler.java:70) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.security.handlers.SecurityInitialHandler.handleRequest(SecurityInitialHandler.java:76) [undertow-core-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.server.handlers.PredicateHandler.handleRequest(PredicateHandler.java:43) [undertow-core-1.1.0.Final.jar:1.1.0.Final]
	at org.wildfly.extension.undertow.security.jacc.JACCContextIdHandler.handleRequest(JACCContextIdHandler.java:61)
	at io.undertow.server.handlers.PredicateHandler.handleRequest(PredicateHandler.java:43) [undertow-core-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.server.handlers.PredicateHandler.handleRequest(PredicateHandler.java:43) [undertow-core-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.ServletInitialHandler.handleFirstRequest(ServletInitialHandler.java:261) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.ServletInitialHandler.dispatchRequest(ServletInitialHandler.java:247) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.ServletInitialHandler.access$000(ServletInitialHandler.java:76) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.servlet.handlers.ServletInitialHandler$1.handleRequest(ServletInitialHandler.java:166) [undertow-servlet-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.server.Connectors.executeRootHandler(Connectors.java:197) [undertow-core-1.1.0.Final.jar:1.1.0.Final]
	at io.undertow.server.HttpServerExchange$1.run(HttpServerExchange.java:759) [undertow-core-1.1.0.Final.jar:1.1.0.Final]
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) [rt.jar:1.8.0_45]
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) [rt.jar:1.8.0_45]
	at java.lang.Thread.run(Thread.java:745) [rt.jar:1.8.0_45]

Hi ManuelB,
thanks for your feedback. Responsible developer will look into your fix suggestion.
Regards,
Cahit

Hi Manuel,
interesting dirty change... ;-) I will have a look into it. As we are already dirty we could be even dirtier as you also write it in the comment. But I want to test this out before I merge it...
Thanks for the PULL request - I will come back to you soon...
Thanks and nice weekend,
Peter

Hi Peter,
so the fix will prevent the error message but I was still not able to get the testsuite running on wildfly.  all_test was still empty.
/Manuel

Hi Manuel,
thanks for the feedback - this means it would be better to not include the fix and do an instanceof check of the returned object from the file field of the collection to avoid this problematic cast. I will try to check it with wildfly 9 (the servlet distribution)...
Cheers,
Peter

Hi Peter,
yes, if a core developer can test this on wildfly would be awesome.
I am using wildfly 8.2 but probably 9 will produce the same result.
/Manuel

Hi Manuel,
the implementation is not really simple. If we fully want to support wildfly vfs to list the resources we need to properly use the API of the vfs to drill down the resources. I think a custom implementation for JBoss might be better. This is my first impression now. I can use reflection to drill down the resources but I need to collect more experience with that. Maybe it would be better to enhance the DiscoveryServlet to allow to add custom implementations to list the resources...
Best regards,
Peter

Hi Peter,
the easiest way would be to add a embedded server to the pom which automatically downloads itself and runs the test suite. So it shouldn't be necessary to bring an own server.
Examples are:
Glassfish embedded:
https://docs.oracle.com/cd/E19798-01/821-1754/gijhs/
Wilfdfly maven plugin:
https://docs.jboss.org/wildfly/plugins/maven/latest/
Jetty maven plugin:
http://www.eclipse.org/jetty/documentation/current/jetty-maven-plugin.html
Examples for applications that bring their own app server can be found here:
https://github.com/ManuelB/facebook-recommender-demo/blob/master/pom.xml
and here:
https://github.com/ManuelB/gengo-webcrawler/blob/master/pom.xml
Hope that helps
Manuel

Hi Manuel,
thanks for the links and samples. I setup the server locally and debugged it today - but I had the feeling that doing this with reflection is really ugly and thus I would prefer a specific implementation for JBoss/Wildfly of the DiscoveryServlet which makes use of the VFS API. Right now the coding is independent from proprietary API and I am not in favor to make it dependent.
Do you know the cargo-maven2-plugin? https://codehaus-cargo.github.io/cargo/Maven2+plugin.html - I am in favor of it and internally we are using this plugin to start the servers. We concentrate on Tomcat/Jetty/SAP Servers mainly - that's why we did not run it on JBoss/Wildfly till now.
I am thinking on a solution but as said I am in favor of enabling support to be able to implement specific adapters for individual containers with proprietary API to access the resource like the VFS in Wildfly.
Best regards,
Peter

Hi Peter,
from an architecture perspective I would recommend to remove the whole need for a full blown java ee server with a web profile for testing a javascript client framework instead of implementing adapters for the different servers.
Currently I don't have the necessary time to make the test suite working without a server but being able to start the test suite with the cargo plugin would help me a lot.
/Manuel

Hi Manuel,
it also works with Grunt & the connect-openui5 module - without the Java server stack. Also those JSON files being returned can be put just in the proper location instead of executing the scan logic to find the test files. So it would even be possible to run it without the server side logic - but for dev purposes where the people just want to add additional pages and they get discovered automatically it is simpler to have some dynamic logic there. If you are interested in the static approach I can just provide you some guidance how to set this up... In addition I can also think about enhancing the POM to start the test server (Jetty) with Maven in the testsuite itself.
What do you think?
BR,
Peter

Hi Peter,
starting an embedded jetty would be awesome.
/Manuel

Hi Manuel,
I recently submitted a change which allows you to start an embedded Jetty server for the OpenUI5 testsuite. It uses the cargo-maven2-plugin. Please have a look here: 5abd94af920018a535daa4a02664bca04a3301c0
Is this what you need?
Best regards,
Peter

Hi Manuel,
I fixed the code so that it will not fail with an exception anymore and provided a workaround to start an embedded Jetty server for OpenUI5.
I will close the issue now.
Thanks for reporting and BR,
Peter

@petermuessig good job!

Just tested it. Full instructions:
$ git clone https://github.com/SAP/openui5.git
$ cd openui5
$ mvn install
# ...
$ cd src/testsuite-utils
$ mvn install
# ...
$ cd ..
$ cd testsuite
$ mvn install
# ...
$ mvn  org.codehaus.cargo:cargo-maven2-plugin:run
...
[INFO] [talledLocalContainer] Jetty 7.6.17.v20150415 started on port [8080]
# Open http://localhost:8080/testsuite/test-resources/testsuite/testframe.html afterwards

Enjoy you own personal openui5 testsuite.
If you can't build it your own go here:
https://openui5.hana.ondemand.com/test-resources/testsuite/testframe.html

Just a short note in my opinion you just move to Java 8 and Jetty 9.3.1. VersionEye can check if you are using out dated libraries. As far as I know Java 6 is not supported by oracle anymore.


1.2.0 Feature: Realtime: Queryable Connection State

One of a family of pull requests addressing #3.

Apologies for the noise with https://github.com/ably/features/pull/21/commits/899b3172e7d2afb3e153d95607d945968bbc3398 but I had a change of heart in respect of use of this naming. The word "queryable" was too open to interpretation, so I've renamed to keep it simpler and hopefully as a result clearer.


#7   Added the score feature correctly

Thank you for the correct procedure. I'm still a beginner in version control so that happened by mistake. From next time I'll take care that these sort of issues don't take place.
Once again sorry again for causing any kind of trouble.
Thank You

Changed the font and removed the constants in blit. Now, the score can change size according to DEFAULT_RECT_SIZE.  Removed score from Game class and added  it to main.

You didn't fix the style, so I made another commit on your branch


Reevaluate server.rs error message generation

Since the move to message id invariants this no longer works as expected:
https://github.com/shotover/shotover-proxy/blob/58ed58e02ca067407d7c1247b11230444f6c7f41/shotover/src/server.rs#L750-L769
We should investigate what the best way forward is here.

Resolved by https://github.com/shotover/shotover-proxy/pull/1591


Generating inner classes

When I try to generate java classes from schemas containing definitions of inner classes, they simply get ignored. For example the following input
{
    "TestInnerClass":{
        "title":"TestInnerClass01",
        "type":"object",
        "additionalProperties":false,
        "properties":{
            "var":{
                "type":"string"
            }
        }
    }
}

and the same goes for another case that I need to get working
{
    "definitions":{
        "TestInnerClassA":{
            "title":"TestInnerClass01",
            "type":"object",
            "additionalProperties":false,
            "properties":{
                "var":{
                    "type":"string"
                }
            }
        },

         "TestInnerClassB":{
            "title":"TestInnerClass02",
            "type":"object",
            "additionalProperties":false,
            "properties":{
                "var":{
                    "type":"string"
                }
            }
        }
    }
}

You need to refer to these schemas in some way (e.g. $ref). This plugin wont do anything with inner nodes of this kind.


Bump zip build minimum version options

The zip builder does not support mixed minimum platform versions. Since AppCheck requires iOS 11.0, bumping the zip and Carthage distribution minimum iOS versions.

I'll explore a less radical fix today.

Closing in favor of #7966


UI - Missing link to Pipeline Run in the the Releases tab

Expected:
Once a release is been triggered, a Pipeline Run link should appear in the Releases tab
Actual:
The Pipeline Run link is empty

Looking at the downstream production deployment, it seems to be the same.

Ah, it is likely rbac. If you don't have rights to view things in that release workspace, then the link is absent. If you do, then HAC should render it. (True in the downstream RH deployment.) Ping an admin of that workspace and they can give you view rights there.


Please help to find the right configuration

Hi @bsorrentino , as we briefly discussed at https://github.com/bsorrentino/langgraph4j/issues/50, I'm trying to split common logic and my flow. Could you please do me a favor and help to figure out where the problem is?
Simplified what I want:

to have 2 agents, each with own tasks to resolve (let's say greetings_agent and story_teller)
to have an orchestration which decides which agent to call

for now I made for that purpose yet another agent that makes the orchestration through prompt

do you happen to know how to make it better?

depending on user's request, the one of 2 agents (either greetings_agent or story_teller) is called
be able to include LangChain4j-native tools into the grapth so that if specific agent is called, it may trigger one or many tools for better result

What I have now:

Agent to greet
Agent to tell a story
Agent for orchestration

My graph configuration looks like the following
new StateGraph<>(SCHEMA, stateSerializer)
                .addNode("orchestrator", orchestrator)
                .addNode("story_teller", story_teller)
                .addNode("greetings_agent", greetings_agent)
                .addConditionalEdges("orchestrator",
                        routeOrchestration,
                        mapOf( "good_story", "story_teller",
                                "greet", "greetings_agent" ))
                .addEdge(START, "orchestrator")
                .addEdge("story_teller", END)
                .addEdge("greetings_agent", END);

My schema is
private static final Map<String, Channel<?>> SCHEMA = Map.of(
            "intermediate_steps", AppenderChannel.<IntermediateStep>of(ArrayList::new));

It's quite not clear how routeOrchestration should look like. Now I use this mocked variation that every time should chooce the same edge
public class RouteOrchestration implements AsyncEdgeAction<State> {

    @Override
    public CompletableFuture<String> apply(State state) {
        var orchestrationEdge = state.agentOutcome()
                .map(agentOutcome -> {
                    return "good_story";
                })
                .orElse("greet");

        return completedFuture(orchestrationEdge);
    }
}

During execution I'm getting the same result (because of implementation of RouteOrchestration) but my questions are:

Does the orchestration have to be done through dedicated agent? Or you may suggest something beter?
Does the schema look valid? Seems like yes, even taking into account that I will add the tools into the flow.
How to implement RouteOrchestration right?

Where can I get the orchestration result?
One more State is needed?

Does another schema is needed for orchestration?

Thank you in advance! :)

Hi @pakamona I'll arrange a notebook asap

Hi @bsorrentino thanks a lot.

Hi @pakamona I've added the notebook for providing a simple sample to help you.
Let me know


Error loading

https://www.racedepartment.com/threads/hardware-steering-lock-simhub-plugin.197410/page-5#post-3676507


(CEV2-266) Problem: Missing ClientRPC documentation UI

Added postman collection for supported endpoints
Added JSON RPC Methods Documentation

/staging

/staging

/staging

/staging

/staging

/staging

/staging


Installing with minimum-jaxlib fails for Python 3.11

Description
Trying to install jax v0.3.25 on Python 3.11 via pip with the minimum-jaxlib extra fails to resolve a jaxlib version:
pip install "jax[minimum-jaxlib]==0.3.25"

gives output
ERROR: Could not find a version that satisfies the requirement jaxlib==0.3.22; extra == "minimum-jaxlib" (from jax[minimum-jaxlib]) (from versions: 0.3.24, 0.3.25)
ERROR: No matching distribution found for jaxlib==0.3.22; extra == "minimum-jaxlib"

.
The problem appears that only jaxlib versions >= 0.3.24 are available on PyPI for Python 3.11.
Possible fixes:

update minimum-jaxlib to 0.3.24
make jaxlib 0.3.22 available for python 3.11
instead of freezing any particular jaxlib version via minimum-jaxlib, maybe an extra compatible-jaxlib that defines a version range of known good jaxlib versions and is updated with every jax release would be a better option. For jax v0.3.25 it would now be jaxlib >= 0.3.22, <=0.3.25 for example. That would give a bit more freedom to which particular version to resolve from the package index, while still restricting it to one that is known to work with that particular jax version.

What jax/jaxlib version are you using?
jax v0.3.25, jaxlib v0.3.22
Which accelerator(s) are you using?
CPU/any
Additional system info
Python 3.11, Ubuntu Linux, Anaconda
NVIDIA GPU info
No response

@hawkinsp @yashk2810 @nouiz @yhtang for viz

Metaquestion: why are you installing the minimum jaxlib at all?
This particular problem will resolve itself when we bump the minimum, which we are technically free to do at any time, but we typically try to not do it right after a new release just in case the new release is problematic.

I mostly need this in automated tests that verify that my project still works with older jax versions. I cannot use the cpu extra/target for that as it would just install the newest (and likely incompatible) jaxlib version.

You should be able to use the cpu extra along with a version constraint, e.g.,
pip install "jax[cpu]==0.3.24"

should install jax 0.3.24 along with whatever jaxlib that version of jax pinned.
Is there a particular reason you care about the jaxlib version and not just the jax version?

It seems I have missed the change in jax version v 0.2.27 that freezes the jaxlib version. I set up my tests before then and there had the problem that jax would always pull the latest jaxlib version available, which tended to cause incompatibilities for older versions of jax. So it seems that now I can switch to using the cpu extra then..
Remaining questions is then, why is there still a minimum-jaxlib extra?


Use hostname command as fallback way of getting build host name

getHostName() may fail for number of reasons, for example when
networking is disabled or network is down.
hostname command is a portable way of determining host name,
which doesn't depend on networking to be available.

For more details, see: http://stackoverflow.com/a/7800008
For example, Fedora build servers don't have networking available, not even local loopback interface (localhost). As a result host name in Gradle build is always set to "unknown". This patch fixes the problem.

Merged, thanks for the contribution.


Update WebSocketTransportHybi.php

see issue:
https://github.com/Devristo/phpws/issues/94

Thanks! Good catch!

Hello,
First of all, thank you for merging previous merge request.
Сould you see my new request?
https://github.com/Devristo/phpws/pull/96
2016-12-21 10:29 GMT+03:00 Devristo<EMAIL_ADDRESS>
Thanks! Good catch!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Devristo/phpws/pull/95#issuecomment-268455373, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AXaIE89b-p_TWYuB6_vPUwak5Ji_rDm3ks5rKNVAgaJpZM4LMlY8
.


Please open search results in a new tab when using right click and search from code

https://developercommunity.visualstudio.com/t/please-open-search-results-in-a-new-tab-when-using/763875

/needsMoreInfo
The link refers to Visual Studio, not VS Code. If your request is for an enhancement to VS Code please give more detail about what you want. Pasting an annotated screenshot may help.


docs(k4k8s) add changelog for Kong Enterprise k8s <IP_ADDRESS>

/lgtm

@h3xar0n Please merge when you are ready. Thanks!


CDPS-854: Surfacing lastModifiedAt and lastModifiedBy in the API

Codecov Report
Attention: Patch coverage is 72.72727% with 3 lines in your changes missing coverage. Please review.

Project coverage is 81.37%. Comparing base (c7181f6) to head (a777768).

Files
Patch %
Lines

...prisonperson/dto/response/PhysicalAttributesDto.kt
75.00%
2 Missing :warning:

...gital/hmpps/prisonperson/jpa/PhysicalAttributes.kt
66.66%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #50      +/-   ##
============================================
- Coverage     81.47%   81.37%   -0.11%
  Complexity      333      333
============================================
  Files            55       55
  Lines          1058     1063       +5
  Branches        101      101
============================================
+ Hits            862      865       +3
  Misses          124      124
- Partials         72       74       +2

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


Add plausible

See here: https://dash.plotly.com/external-resources
and
<script defer data-domain="digest.neurobagel.org" src="https://plausible.neurobagel.org/js/script.js"></script>

Bumping this. We should make it an option / only include it if it is our deployment. But the implementation should be quick

I'd say we don't do this as it seems excessively involved, i.e. requires us to edit the template dash uses (iiuc): https://dash.plotly.com/external-resources#usage


Update to protoc-gen-go-aip-test 0.15.0

@thall 🤔 was chore not triggering a release?

@thall I realized I used chore ... 🤔 was chore not triggering a release?

correct, not optimal.. use feat instead

Fixed.


Add skip ssl param

Ref #42 - this PR adds skip_ssl_verification to the source configuration. I have not verified that this actually works, other than that it does not break regular functionality - will need someone to test this on their own GHE with an untrusted certificate.

Thanks! I'll wait with merging this one until we find someone who can test it in the wild!

@itsdalmo, it looks good to me. The hardest part was finding users to test on GHE for you.


sf::Shader is not moveable

Subject of the issue
It is not possible to return sf::Shader from function.
Your environment
Any environnement
Steps to reproduce
#include <SFML/Graphics.hpp>

sf::Shader loadShader() {
    sf::Shader shader;
    /* Probably more code here */
    return shader;
}
int main() {
    sf::Shader myShader = loadShader();
}

Expected behavior
The sf::Shader should not be copiable but should be moveable.
Actual behavior
Compilation error.
Solution
The compilation error comes from the implementation of NonCopyable. NonCopyable class mean also non moveable.
Trivial change in sf::Shader or code design change with NonCopyable to accept move semantic, but may / may not break other classes.

We know that move semantics are not implemented in SFML, what we need is someone to actually do it ;)

See this forum thread for discussion on C++ standards.

Just as a workaround for now - you can use std::unique_ptr<sf::Shader> for non-moveable objects and do something like this:
#include <SFML/Graphics.hpp>

std::unique_ptr<sf::Shader> loadShader() {
    auto shader = std::make_unique<sf::Shader>();
    /* Probably more code here */
    return shader;
}

int main() {
    auto myShader = loadShader();
}

The solution isn't perfect, but I think it's wrong to add move constructor only for one class (and before we "officially" say that we support C++11 or newer in the SFML)

sf::Shader is now moveable


Add now, daysInMonth. Enhance isLeapYear

Changed log

Implement the now and daysInMonth methods.
Enhance isLeapYear and formatsSeconds methods.

@peter279k  thanks you so much


(QENG-1852) install_puppet unable to install windows msi (regression)

busted curl command on windows
busted msiexec command for installation
i cry myself to sleep every night

LGTM.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/Beaker Combined Smoketest/519/
:green_heart: Test passed.

Updated based upon review comments.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/Beaker Combined Smoketest/520/
:red_circle: Test failed.

Failure caused by test timeout, not releated to patch.

@puppetlabs-jenkins retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/Beaker Combined Smoketest/526/
:green_heart: Test passed.


More code reuse for RHTAP Multi-CI rego

New revision uses j.validate_schema to check that the attestations look correct.


WIP: Adding CLI argument option to specify location of settings.yaml

Created for the resolution of issue: #45
Signed by<EMAIL_ADDRESS>Thanks to @lyfofvipin for helping me with changes.

@ntkathole It is ready for review but it does need to be tested with a valid settings.yaml file.

Ping @bovem !

What would you be needing to test it ?


Remove stale test suites and update some flags.

Delete iree_special_models (migrated to https://github.com/iree-org/iree/tree/main/experimental/regression_suite/shark-test-suite-models, outdated and failing in this repo: https://github.com/nod-ai/SHARK-TestSuite/actions/workflows/test_iree.yml)
Delete iree_tests/benchmarks/sdxl (same as previous item)
Delete turbine_tank (unmaintained since https://github.com/nod-ai/SHARK-TestSuite/pull/78, not running on CI anywhere I can see)
Update XFAILs so test_iree.yml passes again

Ping?

Cool go for the delete. I also branched off in case


Android: direct message , UI issue for textfield + send

Steps to Reproduce
The attachment is clear:

open messaging
click on a contact
click on the textfield to send a message
go back
open the conversation again
the input field to send a message is in the wrong place

Attachments
https://github.com/user-attachments/assets/af0d9013-8b1f-401c-bf9b-7e36a6a02689
What platform(s) does this occur on?
Android
Device Info
Samsung Galaxy Fold 3
What version of the app are you using?
1.93.0
Additional Information
It is probably related to the fact the component for sending the message is moved up when the keyboard is shown but if you go back, the keyboard is hidden but the component is not moved again at the bottom of the window

@marcomaroni-github thanks :D

I should consider to test it also on IOS to confirm is not available also there.

I am no longer able to reproduce the issue on Android devices (both emulator and physical device). I presume the issue has been fixed in the latest version.
@crazycoder1999 can you please confirm if it's still an issue and close out the ticket if it's already fixed? Thanks.

Maybe either https://github.com/bluesky-social/social-app/pull/6739 or https://github.com/bluesky-social/social-app/pull/6727 helped.

I believe so, I used to be able to reproduce the issue and it's no longer happening.

Well done! it works correctly now!


Slow Docker on ZFS

RancherOS Version: (ros os version) 1.3.0
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
(drilled in different virtual environments) proxmox (with ceph) and vsphere 5.0
(only SATA)
use 8 core and 8gb memory
configure docker with zfs
https://rancher.com/docs/os/v1.2/en/storage/using-zfs/
docker it's very slow!!!
rancher@rancher:~$ time docker run -it --rm alpine hostname
6c002f5347a9

real	0m12.343s
user	0m0.068s
sys	0m0.052s

i try to create (pool raid0 with 3 disk)

Next, I ran it on my machine with ssd in the rancheros virtual machine
[rancher@node-1 ~]$ time docker run -it --rm alpine hostname
bcd821579152

real	0m6.787s
user	0m0.012s
sys	0m0.004s

@kingsd041 When you have a chance, please help to confirm this issue.

4-5 seconds on ssd with 6 cores and 24 gb (proxmox)

@niusmallnan  I reproduced this issue. I tested using different docker versions (17.09.1-ce, 17.12.1-ce) and different ros versions (1.1.0, 1.2.0, 1.3.0)

This is indeed an issue, I use ZFS as the default storage in rancheros v1.4.1, the test results are as follows:
root@ip-172-31-3-37:~# ros -v
version v1.4.1 from os image rancher/os:v1.4.1

root@ip-172-31-3-37:~# docker -v
Docker version 18.03.1-ce, build 9ee9f40

root@ip-172-31-3-37:~# docker info| grep 'Storage Driver'
Storage Driver: zfs

root@ip-172-31-3-37:~# time docker run -it --rm alpine hostname
1fb430f27c25

real	0m4.285s
user	0m0.062s
sys	0m0.008s

Using zfs storage on ubuntu16.04 only requires 0.926s
root@ip-172-31-8-46:/etc/docker# cat /etc/issue
Ubuntu 16.04.5 LTS \n \l

root@ip-172-31-8-46:/etc/docker# docker -v
Docker version 18.03.1-ce, build 9ee9f40

root@ip-172-31-8-46:/etc/docker#  docker info| grep 'Storage Driver'
Storage Driver: zfs

root@ip-172-31-8-46:/etc/docker# time docker run -it --rm alpine hostname
d9e8f3c0b67a

real	0m0.926s
user	0m0.056s
sys	0m0.012s

Yeah... I just got bit by this. Been working on deploying a RancherOS server on bare metal and am getting basically what's described here. Two raidz2 vdev's of SSD's.
version v1.4.2 from os image rancher/os:v1.4.2
Docker version 18.03.1-ce, build 9ee9f40
Storage Driver: zfs
time docker run -it --rm alpine hostnamez
real	5m8.353s
user	0m0.096s
sys	         0m0.013s
My pool has ashift=12 and I also set recordsize=32k for the docker dataset.

This is not a performance issue with zfs itself.
When we use ros s up zfs, the zfs command is installed like this:
[root@ip-172-31-3-240 rancher]# which zfs
/sbin/zfs
[root@ip-172-31-3-240 rancher]# cat /sbin/zfs
#!/bin/sh

exec system-docker run --rm --privileged \
                --pid host \
                --net host \
                --ipc host \
		-v /mnt:/mnt:shared \
		-v /media:/media:shared \
		-v /dev:/host/dev \
		-v /run:/run \
		zfs-tools $(basename $0) "$@"

In fact it will run a container to run the zfs parameters.
If we run zfs list on RancherOS, it will be called more times than Ubuntu.
If we use zfs as the storage driver, the zfs command will be called when we run a container, so we will see Slow Docker on ZFS.
The performance of the zfs file system should have no effect.

Thanks, that's good news. This does affect the responsiveness of Docker administration tools though. Portainer, for example, takes forever on any page that needs to fetch Docker info, and unfortunately even times out sometimes :( (otherwise it'd be a little less of an issue)
I have Portainer installed in system-docker, and I mount the Unix socket of user docker, which produces these results. Granted, I have not gotten a chance to test the actual responsiveness of containers deployed in that user docker yet.

I am considering providing a pre-compiled zfs, which will be the same as other OS, but this does increase the maintenance difficulty.


[Guide] Consolidate php articles and remove duplicates

Description
This consolidates and improves the PHP folder in the guides.
Includes:

Combining 2 articles on cookies
Combining articles on syntax and fixing typos
Improving articles and combining them and creating new articles in appropriate places in loops
Combining and improving 3 articles on switch statements.

Checklist

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

@shamamaz Thanks for your contribution.
We want to err on the side of having more, more atomic articles, rather than fewer.
Could you open a new series of smaller pull requests - one per article - so we can more easily QA and merge these?


Feature/grid editor change annotation value

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] The Unreleased section in CHANGELOG.md has been amended to reflect the changes in this PR
[x] Build (mvn verify) was run locally and any changes were pushed
[x] The pull request is against the correct branch (master for bug fixes, develop for new functionality)

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: #96
What is the new behavior?

Annotation values can be changed in the grid editor

Does this introduce a breaking change?

[ ] Yes
[x] No

Does this introduce new dependencies?

[ ] Yes
[x] No

If this introduces new dependencies:

[ ] Dependencies and citation templates have been updated where necessary

Other information

Checklist for maintainers
Releases

[ ] Do NOT push the green merge button on GitHub.
Follow the latest version of the developer/maintainer docs for making releases.
[ ] License and citation information for dependencies are complete (each dependency has a folder in /THIRD-PARTY/).

Thanks for approving this, will merge once CI passes :).


unable to parse address

Do you want to request a feature or report a bug?
Bug
What did you do?
Update to traefik 1.6.2 from 1.5.4
What did you expect to see?
That everything would work as expected
What did you see instead?
A lot of logs with the following messages:
time="2018-05-29T09:27:15Z" level=error msg="unable to parse address: <IP_ADDRESS>1, <IP_ADDRESS>: can't parse IP from address <IP_ADDRESS>1, <IP_ADDRESS>"

Configuration
Relevant bits with forwarded headers.
[entryPoints]
  [entryPoints.http]
  address = ":80"
    [entryPoints.http.redirect]
      regex = "^http://(.*)"
      replacement = "https://$1"
    [entryPoints.http.forwardedHeaders]
      trustedIPs = ["<IP_ADDRESS>/22", "<IP_ADDRESS>/16"]
  [entryPoints.internal]
  address = ":8081"
    [entryPoints.internal.forwardedHeaders]
      trustedIPs = ["<IP_ADDRESS>/22", "<IP_ADDRESS>/16"]

Output of traefik version: (What version of Traefik are you using?)
v1.6.2

It seems to me that there has been some regression in the way that traefik parses multiple ip address, as is the case in our environment when traefik itself is behind an external load balancer. This worked prior to v1.6. Looking at the commit history I am guessing that it is related to the changes introduced in d2766b1 or edb53rd.

As far as I can see, prior to v.1.6.0 the X-Forwarded-For header was not parsed at all. Looking at the logic right now https://github.com/containous/traefik/blob/master/whitelist/ip.go#L60 it seems like the parsing logic is not able to handle the case where there are multiple comma separated ips in the X-Forwarded-For header as is the case here.


Isochrone toggle broken

The isochrone intervals can't be hidden anymore. It only hides the outline of the isochrone.

Hi
any update on the issue above please - really need this option for a piece of work which involves emergency response planning
thank you


restore gamma correction after save

At the end of save() method, the gamma correction should be restored on source_image.
In case if further save calls on the same image / object, those will get darker with each save.
Simple fix:
imagedestroy($dest_image);

// restore gamma correction on source_image
if ($this->gamma_correct) {
  imagegammacorrect($this->source_image, 1.0, 2.2);
}

return $this;

It's closed in #132