pkg_zip failing on windows w/ Bazel 4.0.0

Error message:
PS C:\workspace> C:\bazel.exe build -s  //:protoc_release
INFO: Analyzed target //:protoc_release (21 packages loaded, 580 targets configured).
INFO: Found 1 target...
SUBCOMMAND: # //:protoc_release [action 'PackageZip protoc-3.19.1-x64_windows.zip', configuration: 9d96c5e0dbeafe11161fe69fc89d74bfea3cac17f680ef9ae77a57d635f7af83, execution platform: @local_config_pla
tform//:host]
cd C:/workspace/_build/out/execroot/com_google_protobuf
  SET PATH=C:\msys64\usr\bin;C:\msys64\bin;C:\Windows;C:\Windows\System32;C:\Windows\System32\WindowsPowerShell\v1.0;C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPo
werShell\v1.0\;C:\Windows\System32\OpenSSH\;C:\Program Files\dotnet\;C:\Users\ContainerAdministrator\AppData\Local\Microsoft\WindowsApps;C:\Users\ContainerAdministrator\.dotnet\tools;C:\Program Files\Nu
Get;C:\Program Files (x86)\Microsoft Visual Studio\2022\TestAgent\Common7\IDE\CommonExtensions\Microsoft\TestWindow;C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools\MSBuild\Current\Bin;C:\
Program Files (x86)\Microsoft SDKs\Windows\v10.0A\bin\NETFX 4.8 Tools;C:\Program Files (x86)\Microsoft SDKs\ClickOnce\SignTool;;C:\python;
    SET RUNFILES_MANIFEST_ONLY=1
  bazel-out/x64_windows-opt-exec-2B5CBBC6/bin/external/rules_pkg/private/build_zip.exe -o bazel-out/x64_windows-fastbuild/bin/protoc-3.19.1-x64_windows.zip -d / -t 315532800 -m 0555 --manifest bazel-out
/x64_windows-fastbuild/bin/protoc_release.manifest
ERROR: C:/workspace/BUILD:574:8: PackageZip protoc-3.19.1-x64_windows.zip failed: (Exit 1): build_zip.exe failed: error executing command bazel-out/x64_windows-opt-exec-2B5CBBC6/bin/external/rules_pkg/p
rivate/build_zip.exe -o bazel-out/x64_windows-fastbuild/bin/protoc-3.19.1-x64_windows.zip -d / -t 315532800 -m 0555 --manifest ... (remaining 1 argument(s) skipped)
Traceback (most recent call last):
  File "\\?\C:\Users\ContainerAdministrator\AppData\Local\Temp\Bazel.runfiles_1pl18lt_\runfiles\rules_pkg\private\build_zip.py", line 21, in <module>
    from private import build_info
ModuleNotFoundError: No module named 'private'
Target //:protoc_release failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 4.707s, Critical Path: 1.35s
INFO: 2 processes: 2 internal.
FAILED: Build did NOT complete successfully

Here's the BUILD target:
https://github.com/protocolbuffers/protobuf/blob/master/BUILD#L575
Any pointers? Any other context I can provide?
Python version: 3.9.8

OK. I took the issue back.
I need a lot more supporting info.

What release of rules_proto does this fail in, or what commit if from head? The WORKSPACE file matters as much as the BUILD.
Does it fail on linux too, or just windows

It's going to be hard to reproduce, since I don't have a working windows environment right now.
It also looks like it is a release before the last code restructuring, so you'll need to update what you pull, because we are not doing patch releases to the 0.x release line.

Also, are other python tools working for you with your particular combination of bazel and rules_python versions?

More things to try.
If you make it pkg_tar instead of pkg_zip, does it fail in exactly the same way? It should.
Can you send an ls -lR (or windows equivalent) of the unfurled rules_pkg repository.

What release of rules_proto does this fail in, or what commit if from head? The WORKSPACE file matters as much as the BUILD.

This has nothing to do with the rules_proto repo to be clear, this is the http://github.com/protocolbuffers/protobuf repo for the Protocol Buffers project.
The deps/versions of things are also checked in to GitHub here: https://github.com/protocolbuffers/protobuf/blob/master/protobuf_deps.bzl
We're using the officially released rules_pkg 0.5.1, haven't tested with HEAD.

Does it fail on linux too, or just windows

Only Windows, linux is working as expected.

It's going to be hard to reproduce, since I don't have a working windows environment right now.

I'd suggest a simple GCP VM, this is how I've been doing diagnostics.

other python tools working for you with your particular combination of bazel and rules_python versions?

This is the only python tool we've introduced, thus far.

If you make it pkg_tar instead of pkg_zip, does it fail in exactly the same way? It should.

Correct, it does fail in the same way.

Can you send an ls -lR (or windows equivalent) of the unfurled rules_pkg repository.

https://gist.github.com/perezd/1014425bfa1d0674c1269fbdcb2452ba

Thanks for that information.
If pkg_tar fails the same way, and the init.py is in private, as the ls shows, then this is a python path problem.  But we are using the same rules_python versions, and our tests pass on Windows, but yours don't, so there is no smoking gun.
I'm going to be tied up with BazelCon for the rest of the week.  I can potentially try some things out next week, but that is likely to be a busy/short week because of Thanksgiving.

I am using the embeddable package fwiw:
https://www.python.org/downloads/release/python-399/
Also, if there are any PATH things that may need to be setup, I likely
didn't set them up correctly/at all, do you have any suggestions?
On Mon, Nov 15, 2021 at 9:21 PM aiuto @.***> wrote:

Thanks for that information.
If pkg_tar fails the same way, and the init.py is in private, as the ls
shows, then this is a python path problem. But we are using the same
rules_python versions, and our tests pass on Windows, but yours don't, so
there is no smoking gun.
I'm going to be tied up with BazelCon for the rest of the week. I can
potentially try some things out next week, but that is likely to be a
busy/short week because of Thanksgiving.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_pkg/issues/475#issuecomment-969870627,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAFA5XQI62TM33C4JQHJ23UMHS6HANCNFSM5IB27QOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

On Tue, Nov 16, 2021 at 12:31 AM Derek Perez @.***>
wrote:

I am using the embeddable package fwiw:
https://www.python.org/downloads/release/python-399/

I don't think the python version should be the problem. AFAIK, import
did not change between 3.6 and 3.9

Also, if there are any PATH things that may need to be setup, I likely
didn't set them up correctly/at all, do you have any suggestions?

There should not be anything you should have to set. Bazel is supposed to
provide a good default environment for running py_binary targets.
On Mon, Nov 15, 2021 at 9:21 PM aiuto @.***> wrote:

Thanks for that information.
If pkg_tar fails the same way, and the init.py is in private, as the ls
shows, then this is a python path problem. But we are using the same
rules_python versions, and our tests pass on Windows, but yours don't, so
there is no smoking gun.
I'm going to be tied up with BazelCon for the rest of the week. I can
potentially try some things out next week, but that is likely to be a
busy/short week because of Thanksgiving.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/bazelbuild/rules_pkg/issues/475#issuecomment-969870627
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAAFA5XQI62TM33C4JQHJ23UMHS6HANCNFSM5IB27QOA
.
Triage notifications on the go with GitHub Mobile for iOS
<
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
<
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_pkg/issues/475#issuecomment-969876814,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXHHHBKKZBDK6ENPLJYEKDUMHUBXANCNFSM5IB27QOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Any updates here?

Not yet. We Bazelcon last week and Thanksgiving this week, I have not had a chance to even look at setting up a windows environment to test in.

OK! Let me know if I can assist when you start investigating.
On Wed, Nov 24, 2021 at 1:17 PM aiuto @.***> wrote:

Not yet. We Bazelcon last week and Thanksgiving this week, I have not had
a chance to even look at setting up a windows environment to test in.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_pkg/issues/475#issuecomment-978228076,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAFA5SPPL3LWETHCZVYUSLUNVJAFANCNFSM5IB27QOA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I tried this out in a local Windows dev environment I have lying around, and the first thing I encountered was:
> bazelisk.exe build //:protoc_release
ERROR: C:/$BASE/dev/protobuf/BUILD:570:15: in package_naming rule //:protoc_pkg_naming:
Traceback (most recent call last):
        File "C:/$BASE/dev/protobuf/protobuf_release.bzl", line 35, column 9, in _package_naming_impl
                fail("Unrecognized platform")
Error in fail: Unrecognized platform
ERROR: Analysis of target '//:protoc_release' failed; build aborted: Analysis of target '//:protoc_pkg_naming' failed
INFO: Elapsed time: 2.754s
INFO: 0 processes.
  This was easy enough to work around.  After doing so, the build of the zip worked fine for me.  I am using HEAD protobuf, rules_pkg 0.5.1.
Some differences between our environments:

I have VC 2022 tools installed locally, which requires use of bazel 5. (USE_BAZEL_VERSION=last_rc in my environment for bazelisk)
I have python 3.9 installed via the MSFT app store

Either of these may have made the difference.  I currently don't have the time to investigate further, but I hope this information helps.

That fail is in protobuf code, so I can't say if that is intentional or not, but it looks that way.
FWIW, rules_pkg is platform agnostic and will not take dependencies on any C++ compiler, so it won't matter if you use mingw or msvc. In any case, I don't have a C++ compiler on my machines, so I would be bit with the same problem.
For my testing, I would whack protobuf_release.bzl to default to win* if not macos or linux, but that might not be suitable for actually building protobuf.
This does indirectly answer the question you asked last week. "how can I help". You can find a reproduction of the problem that does not require a C++ complier. That would probably be something not trying to build protobuf.

Can you try checking out at this commit? https://github.com/protocolbuffers/protobuf/commit/4eb884054424def9348765898cc1ab2792826b15
I think you're running into some rules_pkg stuff we're setting up that's environment specific.
As for the doesn't rqeuire a C++ compiler, you can simply mod the build file to not include :protoc and you'd have that repro.

Can you try checking out at this commit? protocolbuffers/protobuf@4eb8840

I believe I already had it, but I synced to HEAD anyway today.

I think you're running into some rules_pkg stuff we're setting up that's environment specific.

I was, and was able to work around the problem OK.  Apparently it was fixed in protocolbuffers/protobuf@667d5e9e9f1a9ebc15cfcddb5f272a301d0f16b6 nearly identically to how I fixed it locally.

As for the doesn't rqeuire a C++ compiler, you can simply mod the build file to not include :protoc and you'd have that repro.

I was able to build the target without MSVC by making the suggested edits.  I am still not able to reproduce the problem at this time using either Bazel 4.0.0 or 4.2.1.
I am still using the copy of python (3.9.9) from the MSFT store, which may be the deciding difference.  There may some repository state that needs to be cleaned up (bazel clean --expunge?).

One thing I mentioned earlier on in the thread is I am using the embeddable package of python for windows, found here:
https://www.python.org/downloads/release/python-399/
Perhaps its  unique to this format? It's required for my use case (docker container)

Are there any other differences in your environment that come to mind?  Is there anything else special about the docker container?  Do you have a global bazelrc file that could be adding adding additional flags to bazel that might be influencing how the python rules are run?
It would also be very helpful to have an automated way to reproduce this that properly emulates your environment.  Would it be possible to provide one?  Other notes about the environment would also be helpful; I might have time to look into this a little more over the next few days.

Nothing unusual w/ bazelrc, just changes to output_base and repository_cache.. Also running w/ --batch but nothing that I would suspect could be related to this, it looks like:
startup --output_base=C:\\workspace\\_build\\out --batch
build --repository_cache=C:\\workspace\\_build\\repo_cache
build:offline --nofetch

We start bazel w/ a custom entrypoint.ps1 powershell script that looks like:
Start-Process -FilePath "C:\bazel.exe" -WorkingDirectory "C:\workspace" -NoNewWindow -ArgumentList "$args" -Wait

Nothing special there either really.
Our docker container is based on: mcr.microsoft.com/dotnet/framework/sdk:4.8-windowsservercore-ltsc2019
We simply extract the embeddable python package to C:\python and add it to $env:PATH using:
RUN setx /M PATH $($env:PATH + ';C:\python')

I was able to reproduce this locally using the embedded python package you mentioned earlier.  My conclusion is that this is not a bug in rules_pkg, but either something in bazel, or (more likely) your setup.
You probably should not be using the embeddable python binary, since it ignores various aspects of the environment by design, according to an https://bugs.python.org/issue28245 and the relevant documentation.
As it stands, the embeddable python binary is not compatible with Bazel: the bazel python stub launcher uses the PYTHONPATH environment variable to set the needed python module paths in the binary.  Since the embeddable python binary ignores the environment by design, it is never configured with the bazel-specified module paths, and thus will always fail when you use modules outside of the standard library.
See also https://github.com/bazelbuild/bazel/issues/7959, and the stub template itself for more details.
Local analysis:
PS C:\Users\...> $env:PYTHONPATH = "foo"
PS C:\Users\...> Get-Command python

CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Application     python.exe                                         3.9.915... C:\python\python.exe

Actually running said binary in that enviornment:
Python 3.9.9 (tags/v3.9.9:ccb0e6a, Nov 15 2021, 18:08:50) [MSC v.1929 64 bit (AMD64)] on win32
>>> import sys; sys.path
['C:\\python\\python39.zip', 'C:\\python']
>>> import os; os.environ['PYTHONPATH']
'foo'

PTAL and close if you agree.

This is an interesting thing to at least add to rules_pkg README, potentially. I'm not sure how else to get Python running in a docker container, perhaps there are other strategies I can try. Thanks for looking into this!


id: Fix output of 'kubectl rollout status'

This is the same fix as https://github.com/kubernetes/website/commit/6e93717597113c726cbcf8a7059e47f23e3e8a64 for id.
If running kubectl rollout status command, the output is like:
  deployment "nginx-deployment" successfully rolled out

The corresponding kubectl code also shows it according to https://github.com/kubernetes/kubectl/blob/e95e378e5972064b177a8e71eac2803f55c8c5df/pkg/polymorphichelpers/rollout_status.go#L89

Thank you @oomichi
/lgtm

/approve


Update application.properties

Won't merge, the original instructions are more clear than adding dummy values.


[FLINK-34901][connectors/jdbc]Update clause must EXCLUDED unique index.

As docs 'https://www.postgresql.org/docs/current/sql-insert.html', when we use ON CONFLICT, Update clause must EXCLUDED unique index.
Or it will throw exceptions.like " modification of distribution columns in OnConflictUpdate is not supported  Call getNextException to see other errors in the batch."

@1996fanrui Can you help me review?

Hi @Mrart , the CI is failed, could you rebase the main branch and check the CI?


Loss of Data - belongsTo disappearing

So I am having an issue for the 2nd time on an Emberfire project. Last time was almost a year ago when I was putting together a meetup preso on building a chat client in Ember and Firebase.
I have model's with belongsTo relationships and during local debugging the belongsTo keys in forge disappear and my relationship is gone.
I cannot reproduce it and it feels like it only seems to happen during heavy local debugging but it has happened half a dozen times times or so on this recent project so I feel confident it is real and not in my head.
I have found Angularfire issues posted with potentially similar data loss problems but none in this repo. And they all seem related to use of FB internals that I do not touch.
I am wondering if anyone else has experience this as it is very frustrating.

Does this happen when you delete a record locally? there is some code that tries to remove orphan hasMany links from the server on delete. Could this be it?

@tstirrat no not during delete (I think).
I believe it is happening when I am debugging a save after an edit and the browser is stopped at a debugger statement and then reloads after I make some live code changes and doesnt seem to actually hit the model.save() point. Perhaps it is a partial save of some sort?

Ahh, yes, it could be related to us saving relationship links in subsequent requests. This will be fixed in #337

Well that is good. This seems like such a big issue I am surprised (and thankful) no one else has hit it :-)

I wonder if this is related to #367 where the the belongsTo is nullified due to some inverse relationship sync issue, and when you save the model it deletes the belongsTo key.
Does that ring a bell? are you updating a hasMany and the later realizing the inverse belongsTo is null?

I'm running in to belongsTo relationships as well on 1.6.6. Worked fine on 1.6.0. The model is very simple:
// keyboard/model.js
export default DS.Model.extend({
  title: attr('string'),
  photo: belongsTo('photo')
});

// photo/model.js
export default DS.Model.extend({
  title: attr('string'),
  url: attr('string'),
});

If I create a keyboard, persist it, then create a photo, persist it then try to link it to a keyboard it all falls apart.
const { set } = Ember;

let self = this;
this.store.createRecord('keyboard', {title: 'someTitle'}).save()
.then((keyboard) => {
  self.createRecord('photo', {title: 'someOtherTitle'}).save()
  .then((photo) => {
    set(keyboard, 'photo', photo).save()
    .then((response) => {
      console.log('response', response);
    })
  })
})

The above results in {{keyboard.photo}} == null. The photo object is persisted, the keyboard is persisted, but the keyboard object does not have a photo key.
Now, if I define keyboard: belongsTo('keyboard') on the photo model, everything works fine. But the photo model will "belong" to many different model types, so it shouldn't have this sort of relationship.

Looks, it is a real issue, I have the same problem with belongsTo, hasMany. Firebase randomly remove relations, so data will be inconsistent. It is happening with async: true or without async also.

I ended up using a hasMany and just using {model}.firstObject for the moment. Granted you end up peppering your code with this.get('someModel').clear() and what not. I can look at issuing a pull request once this app launches

I solved it with manually remove the previous relationship and adding the new one after.
    saveAuthor(author, book) {
      book.get('author').then((previousAuthor) => {
        previousAuthor.get('books').then((previousAuthorBooks) => {
          previousAuthorBooks.removeObject(book);
          previousAuthor.save();
        });
      });

      // Setup the new relation
      book.set('author', author);
      book.save().then(() => author.save());
      book.set('isAuthorEditing', false);
    },

Working demo: https://library-app.firebaseapp.com/books

@ewalti @zoltan-nz can someone make a MCVE for me, so I can investigate? Thanks.

@tstirrat http://emberjs.jsbin.com/juwasa/edit?html,js,output
Start playing with Author names, change them with the select box (after select, they will saved immediately).

Are there any news on this issue?
I am having the same sort of problem.
When changing a route and loading objects from the store, the belongsTo relation is always null.

@florathenb can you post the two models?
Brendan

export default Model.extend({
label: attr('string'),
clearanceId: belongsTo('clearance'),
comment: attr('string'),
settings: attr('')
});
export default Model.extend({
comment: attr('string'),
priority: attr('number'),
timesheetIds: hasMany('timesheet'),
taskId: belongsTo('task'),
color: attr('string'),
colorBox: Ember.computed('color', function _colorComputed() {
return '';
}),
numericID: function(){
let id = this.get('id');
if (id) { return +id; }
}.property('id'),
prio: function(){
let prio = this.get('priority');
if (prio != null) { return +prio; }
else { return 99; }
}.property('priority'),
});

In my person model I have this:
organizations: DS.hasMany('organization', {async: true, inverse: 'people'}),
In my organization model I have this:
members:     DS.hasMany('person', { async: true, inverse: 'organizations'} ),
You don't have any inverse relationships setup but if you never need the ACTIVITY within the TASK that is likely OK. I am not sure on async: true I thought that was still a requirement although it is supposed to be the default. If this helps here is what I am doing where I have no loss.
When a person logs in I load all of their organizations.
When they add or edit an organization I save both sides the following way in my org-editor component:
saveOrganization() {
const self = this;
const organization = this.get('organization');
const person = this.get('person');
organization.set('updatedBy', person);
organization.get('members').pushObject(person);
organization.save().then(function(org) {
self.clearMessage();
person.get('organizations').pushObject(org);
person.save().then(function() {
self.sendAction('action');
});
});
},
@florathenb I hope that helps.

problem is, i lose the relation allthough i am not even changing or saving anything..
@brendanoh thanks for the answer anyway!!

I am not sure I understand what " i lose the relation although i am not even changing or saving anything." means.
At some point you need to save the data in firebase, then you see it in the firebase console but later when you load the activity the taskId is gone?

Been away due to family emergency. Finally getting back into things.
Often when you unexpectedly lose (or gain data) in the relationships on the client side without saving any data, it is usually from one of two things:

You have a hasMany defined in the other model that is being automatically set up as an inverse and this causes the belongsTo to be removed once the other model loads. To fix this you should try setting inverse: null or setting the inverse to the correct location. But you should check the next reason before doing this.
You have defined the relationship in both directions, but have only stored it on one side in the DB. When the hasMany side loads without the appropriate parent key present, it causes the Ember Data's inversion mechanics to erase the belongsTo side without warning.

A real world example: I have a chat structure where a user has openChatRooms: DS.hasMany('room', ..., and each room has creator: DS.belongsTo(user, .... openChatRooms is a subset of possible chatrooms that are actively joined.
Without setting inverse: null, a findAll('room') loads all rooms, and causes my openChatRooms to gain every single room where the creator was set to my user. This happens because each time a room is loaded, it tries to guess inverted rels, it sees a likely candidate user.openChatRooms thinking that is my list of user.createdRooms (which I haven't defined, and don't care about). It starts pushing room ids into the hasMany if they don't already exist.
I hope this helps you narrow down what might be happening in your case.
The moral of this story is: Try setting all of your relationships to inverse: null unless you really need the inverses. Only use inverses if you're truly going to need to traverse the relationship from the other side.. and if you do really need the inverse relationship, be explicit and save both records when you change any side of the relationship because Ember Data may silently nullify/change one side of the relationship.

Interesting, I have a (similar?) issue where the users in my db get deleted for no apparent reason. This happens sparsely/randomly and by looking at sentry logs I can see it happening right after a login, where a "no record found" error will pop.
The user model in my project has all the entities belonging to him and he as an entity belongs to no model, also all the relationships are inverse: null ..
I tried recreating the issue by spamming the login with logins/outs but nothing, i also tried many other stupid/random things someone could do, again nothing. We used to think that this error occurred only after a new deployment where a user wouldn't refresh his page and things would go wrong, but it seems that this is not the case.
"emberfire": "^2.0.10"
The login code is as follows
/app/controllers/sign-in.js

import Ember from 'ember';
import { task, timeout } from 'ember-concurrency';
import FindQuery from 'ember-emberfire-find-query/mixins/find-query';

const DEBOUNCE_MS = 400

export default Ember.Controller.extend(FindQuery, {
  application: Ember.inject.controller('application'),
  session: Ember.inject.service(),
  firebase: Ember.inject.service(),
  store: Ember.inject.service('store'),
  buttonLabel: 'Sign In',
  provider: "password",

  signInUser: task(function*() {
    try {
      yield this.get('session').open('firebase', {
        provider: this.get('provider'),
        email: this.get('email') || '',
        password: this.get('password') || '',
      });

      this.transitionToRoute('welcome');

    } catch (ex) {
      this.get('application').set('visible', true);
      this.get('application').set('alertMessage', ex.message);
      this.get('application').set('type', 'danger');
    }
  }).drop(),
});

and the logout
/app/controllers/application.js

import Ember from 'ember';
import firebase from 'firebase';

export default Ember.Controller.extend({
  firebase: Ember.inject.service(),
  actions: {
    signOut() {
     this.set('firebase.u.Ra.$',{});
      this.get('session').close().then(() => {
       this.transitionToRoute('/');
      });
    }
  }
});

I was forced to do this weird this.set('firebase.u.Ra.$',{}); cause after each logout we'd have many errors popping up for breaking references and such.
I don't know if everything done here is correct, or best practice, but our knowledge about emberfire grew as the project was being developed.
Should i post this here or create another issue maybe?

@frcake EmberFire has changed dramatically since my last issue of this type so I am not sure if this issue will help you in figuring your issue out.
this.set('firebase.u.Ra.$',{}); and the errors related to that seem super odd to me. Seems possible that those errors are related to this issue but dont know enough to say for sure.
What does your user model look like?

@brendanoh thanks for the fast response!
I too found it weird and difficult to overcome the errors on logout, others have the same issues too like https://stackoverflow.com/questions/38085030/closing-an-emberfire-torii-session-and-logging-out-throws-permission-denied-erro
(now that i see this post again, there's a newer answer, i might check that out too)
here's my user model
import DS from 'ember-data';
import { computed } from '@ember/object';

export default DS.Model.extend({
  email: DS.attr('string'),
  bankStreet: DS.attr('string'),
  name: DS.attr('string'),
  surname: DS.attr('string'),
  organizationType: DS.attr('string'),
  organizationDescription: DS.attr('string'),
  organizationName: DS.attr('string'),
  taxNumber: DS.attr('string'),
  taxRegion: DS.attr('string'),
  street: DS.attr('string'),
  region: DS.attr('string'),
  profession: DS.attr('string'),
  telephoneNumber: DS.attr('string'),

  // Associations
  pawnForms: DS.hasMany('pawn-form', { async: true, inverse: null }),
  deliveryReports: DS.hasMany('delivery-report', { async: true, inverse: null }),
  customers: DS.hasMany('customer', { async: true, inverse: null }),
  calendarEvents: DS.hasMany('calendar-event'),
  // Computed
  fullName: computed('name', function() {
    return this.get('name') + ' ' + this.get('surname')
  }),

  fullAddress: computed('street', function() {
    return this.get('street') + ' ' + this.get('region')
  })
});

All in all, it's very strange when things work nicely 99% of the time to have that weird errors pop out and delete an entity from the firebase ?!
An exception would look like this
Error: no record was found at https://asdf.firebaseio.com/users/EIRI3WWPy6TBb5nYcJ8MIBcw9eC2
  at ? (/assets/vendor-a274f5466e5371ddd606ba5896e5f204.js:10345:55)
  at _(/assets/vendor-a274f5466e5371ddd606ba5896e5f204.js:3765:25)
  at R(/assets/vendor-a274f5466e5371ddd606ba5896e5f204.js:3773:9)
  at C(/assets/vendor-a274f5466e5371ddd606ba5896e5f204.js:3771:43)
  at e.invokeWithOnError(/assets/vendor-a274f5466e5371ddd606ba5896e5f204.js:1628:237)
  at e.flush(/assets/vendor-a274f5466e5371ddd606ba5896e5f204.js:1615:172)
  at e.flush(/assets/vendor-a274f5466e5371ddd606ba5896e5f204.js:1631:15)
  at e.end(/assets/vendor-a274f5466e5371ddd606ba5896e5f204.js:1639:9)
  at _boundAutorunEnd(/assets/vendor-a274f5466e5371ddd606ba5896e5f204.js:1635:388)

There are also some other similar errors that originate from other routes now that i check again, pfff major headscratcher..!


auto select dd64 vs dd32

we can correctly select dd64 or dd32 by looking at bv.arch.

It uses bv.arch now. And it doesn't choose by looking at just the name. It looks at the integer size from bv.arch.

If you were worried that someone would try to use the plugin on the wrong arch type in general, it checks for that now in the correct place.

It uses bv.arch now. And it doesn't choose by looking at just the name. It looks at the integer size from bv.arch.

The way it currently works, it selects dd32 for binaries that are clearly x86_64, and listed as such in the editor. Your current code is broken.

To clarify, I mean that Binja returns a default in size of 4 even for x64.

You're right, bv.arch.default_int_size is not correct. It should be bv.arch.address_size. Fixed and thanks!

Ok, sounds good!

@redthing1
https://github.com/x64dbg/x64dbgbinja/releases/tag/v2.0.2
This version should percolate over to the plugin manager in BN soon.


Undefined is not a function on createLayer  - probably should be a different error?

With the following code: (which is a variant of http://docs.cartodb.com/cartodb-platform/cartodb-js.html#creating-visualizations-at-runtime)
        var mapOptions = {
            center: { lat: -34.397, lng: 150.644 },
            zoom: 5
        };
        map = new google.maps.Map(document.getElementById('map'),
            mapOptions);

        cartodb.createLayer(map, {
            user_name: 'adambaxter',
            type: 'cartodb',
            sublayers: [{
                sql: "SELECT * FROM sa4_paths_merge",
                cartocss: '#sa4_paths_merge {polygon-fill: #F0F0F0;}'
            }]
        }).addTo(map);
    }

    google.maps.event.addDomListener(window, 'load', init);

I get "undefined is not a function" on line 35499 of http://libs.cartocdn.com/cartodb.js/v3/3.12/cartodb.uncompressed.js
which corresponds to
var mapView = new MapType({
so probably https://github.com/CartoDB/cartodb.js/blob/0aea85812ead7e2ee0d27e451e7f6063f7747247/src/api/layers.js#L147
map is definitely a Google Maps object

Also getting a similar error when using a Leaflet object.

It might be because you need to put map = ... instead of var map = ...

Is google maps javascript included before including cartodb.js?
El 1/4/2015 5:25, "Andy Eschbacher"<EMAIL_ADDRESS>escribió:

It might be because you need to put map = ... instead of var map = ...
 Thanks @ohasselblad. I looked at that for a day without seeing the problem. I'll add use strict next time


GKE: Being able to choose the dataplane version (enabling ebpf based CNI plugin)

Hello,
Having a look to the current version of the API reference we're not able to set the Dataplane version.
This config item is used to enable a recent evolution of the GKE networking.
More information here
https://cloud.google.com/kubernetes-engine/docs/how-to/dataplane-v2
What are the API's items to change:
https://cloud.google.com/kubernetes-engine/docs/how-to/dataplane-v2#api
I would be happy to help if you can guide me to a few PR's as example.
Thank you

Having a look to the code. It seems that the API version used is pretty old which was released ~1 year ago.
google.golang.org/api v0.21.0

I'd love to update it but it will probably require many changes. How can I try my changes please ?

@Smana this should be addressed in https://github.com/crossplane/provider-gcp/pull/308, which I hope to complete very soon :)

The PR https://github.com/crossplane/provider-gcp/pull/308 has been merged and includes the parameter. Thanks


Bug in pythogorean calculator

Describe the bug

A clear and concise description of what the bug is.

Possible solution

Describe the solution you thought of.

Screenshots

A pythagorean triple is defined only for positive integers. Negative numbers can't be sides of a right triangle, therefore, aren't used in Pythagorean triples
i wud like to work on this issue

Go ahead
Deadline 17 May,2021


Appstudio update hacbs-test-project-5suk

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-5suk-on-pull-request-
cannot patch pipelinerun hacbs-test-project-5suk-on-pull-request-: failed to patch pipelinerun rhtap-demo-ctwv-tenant/logURL with hacbs-test-project-5suk-on-pull-request-tc2mb: pipelineruns.tekton.dev "hacbs-test-project-5suk-on-pull-request-tc2mb" not found


No length on password

Describe the bug
Hey when I try to set the password while creating account I noticed that you haven't kept any password limit.
You need to decrease password length :There are two reasons for limiting the password size. For one, hashing a large amount of data can cause significant resource consumption on behalf of the server and would be an easy target for Denial Of Service attack.
Normally all sites have a password minimum to maximum length like 72 characters limit or 48 limit to prevent Denial Of Service attack. in my sql but in weblate registration page there are no limitation. Let me know if you need any more details.
This is typically not DoS, but a vulnerability which may lead to DoS attack.
Screenshort

Assign this issue to me under GSSoC'23!

I can work on this issue. Please assign this to me

So, what u will do? add any kind of validation to it ? I think someone is already working on this!

Dear Monalika Patnaik,
I hope this email finds you well. As per your request regarding the password length limitation issue on the registration page, I have conducted an analysis and formulated a proposed solution. Here are the steps that I suggest to address this concern:
Conduct a thorough analysis: Review the current system architecture, password handling mechanisms, and potential impact of implementing a password length limit. This will help us understand the existing infrastructure and any limitations that need to be considered.
Determine an appropriate password length limit: Based on industry best practices and standards, I recommend enforcing a minimum and maximum password length that strikes a balance between security and usability. For example, setting a minimum length of 8 characters and a maximum length of 72 characters is commonly recommended.
Implement server-side validation: Modify the backend code of the registration page to enforce the desired password length limit. This validation should occur on the server side to ensure consistency and prevent any bypassing attempts by manipulating client-side scripts.
Update user interface and error handling: Enhance the user experience by updating the registration page's interface to reflect the password length requirements. Clear instructions and error messages indicating the password length limit should be provided during the registration process.
Communicate changes to users: If we have existing users, it is crucial to notify them about the updated password length limit and encourage them to update their passwords accordingly. Educating users on the importance of creating strong and secure passwords will help strengthen overall account security.
Perform testing and monitoring: Thoroughly test the updated registration process to ensure that the password length validation works as intended. Additionally, we should monitor the system for any potential performance issues or vulnerabilities, addressing them promptly.
Consider additional security measures: While addressing the password length limitation, it would be beneficial to explore other security measures, such as enforcing password complexity requirements and implementing multi-factor authentication (MFA). These measures will enhance the overall security of user accounts.
I believe implementing these steps will help us mitigate the password length limitation issue and enhance the security of the registration page. However, I recommend discussing these proposed changes with the development team to assess the feasibility and align them with our technical capabilities and priorities.
Please let me know if you have any further questions or if you require additional information. I'm available to discuss this solution further and work towards its implementation.
Thank you for your attention to this matter. Kindly consider assigning this issue to me.
Best regards,
Harshit Kumar Girdhar

Hey pls check i think it's been already resolved by adding validation to the sign up page..lmk if it doesn't!


[SPARK-8263][SQL] substr/substring should also support binary type

This is based on #7641, thanks to @zhichao-li
Closes #7641

Merged build triggered.

Merged build started.

Test build #39335 has started for   PR 7848 at commit b45377a.

Test build #39335 has finished for   PR 7848 at commit b45377a.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class SortArray(base: Expression, ascendingOrder: Expression)

Merged build finished. Test FAILed.

Merged build triggered.

Jenkins is down, fix the test and test it manually, merging this into master.

Merged build started.

Test build #39361 has started for   PR 7848 at commit 461b709.

Test build #39361 has finished for   PR 7848 at commit 461b709.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.


Add reuse window option

I want to use reuse_window option. So I add new environment value.
And I refactored to use the code command.

@kbshl Check plz :)

@kbshl I want you to review this pr :)


GoError: Exception (501) Reason: "EOF"

HI, @javaducky!
At the very beginning of using your library, I encountered difficulties that I can't find an answer to. I took your very first example, changed only the url in it (even without changing it, I get an error, which is logical)(I use other letters, but their count and case match my original).
`import Amqp from 'k6/x/amqp';
import Queue from 'k6/x/amqp/queue';
export default function () {
console.log("K6 amqp extension enabled, version: " + Amqp.version)
const url = "amqp://qa:Lalal765434!@jhvhg-fds094sd.zcxv.sad.com:1414/"

Amqp.start({
    connection_url: url
})
console.log("Connection opened: " + url)

const queueName = 'K6 general'

Queue.declare({
    name: queueName,
    // durable: false,
    // delete_when_unused: false,
    // exclusive: false,
    // no_wait: false,
    // args: null
})

console.log(queueName + " queue is ready")

Amqp.publish({
    queue_name: queueName,
    body: "Ping from k6",
    content_type: "text/plain"
    // timestamp: Math.round(Date.now() / 1000)
    // exchange: '',
    // mandatory: false,
    // immediate: false,
    // headers: {
    //   'header-1': '',
    // },
})

const listener = function (data) { console.log('received data: ' + data) }
Amqp.listen({
    queue_name: queueName,
    listener: listener,
    // consumer: '',
    // auto_ack: true,
    // exclusive: false,
    // no_local: false,
    // no_wait: false,
    // args: null
})

}
`
But, alas, it doesn't work and I get an error like:
 INFO[0000] K6 amqp extension enabled, version: v0.4.1 source=console ERRO[0002] GoError: Exception (501) Reason: "EOF" running at reflect.methodValueCall (native) default at file:///C:/Users/vxczvi/xczvzx/pvcxv/index.js:9:13(18) executor=per-vu-iterations scenario=default source=stacktrace
Yes, I saw issue  #5, but I didn't find anything useful there for myself
Thank you very much for your help!

So I added that I use the library to download a script written in LoadRunner java, which uses IBM and jms. I didn't make a mistake with the choice, can I use your library to achieve these goals?

Hello @Lamardo43!
Sorry for the confusion with this extension! It has fallen into neglect. The extension needs a complete re-write using the latest event model supported by k6. At this time, we intend to delist and deprecate the unless we can find another maintainer.
While any messaging backend supporting AMQP 0.9.1 should be supported, we cannot guarantee it at this time.


chore(package): remove transform-react-constant-elements

We decided not to use it at #731, but I missed the included dependency, this PR removes it.

Current coverage is 95.88% (diff: 100%)

No coverage report found for master at b7ffa6a.

Powered by Codecov. Last update b7ffa6a...2d4007c

Released in<EMAIL_ADDRESS>


Android example with MLC-LLM can't build with mlc-llm nightly package on MacOS x86-64

System Info
MacOS x86-64
AMD GPU
Information

[X] The official example scripts
[ ] My own modified scripts

🐛 Describe the bug
The nightly pip package for mlc-llm and mlc-ai on py311 MacOS x86-64 is old that the mlc-llm CLI doesn't include package as a command argument.
For users on these machines, they will need to build from source.
I think we should update the demo instructions here to explain this clearly.
Error logs
command
python3 -m mlc_llm package  --package-config mlc-package-config.json --output dist
output
------------------------- Usage -------------------------
usage: MLC LLM Command Line Interface. [-h] {compile,convert_weight,gen_config,chat,serve,bench}

positional arguments:
 {compile,convert_weight,gen_config,chat,serve,bench}
                       Subcommand to to run. (choices: compile, convert_weight, gen_config, chat, serve, bench)

options:
 -h, --help            show this help message and exit
------------------------- Error -------------------------
argument subcommand: invalid choice: 'package' (choose from 'compile', 'convert_weight', 'gen_config', 'chat', 'serve', 'bench')

Expected behavior
The command should work as expected to produce binaries.

cc. @tmoreau89

I  also get this  problem,   this is blocking my wok  now.  Who   has solution?

Closing since repo was updated


Carthage build fails on Xcode 10.x

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[x] I have read the wiki page and cheat sheet, but there is no information I need.
[x] I have searched in existing issues, but did not find a same one.
[x] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
What
Should KingfisherSwiftUI be built by carthage? If no, I can send a PR making this scheme not shared.
Reproduce

Have xcode 10.x installed and selected in terminal.
run carthage build with Kingfisher version 5.11.0.
KingfisherSwiftUI fails to build

Duplicated with https://github.com/onevcat/Kingfisher/issues/1291
Adding KingfisherSwiftUI as a scheme is intended. However, the fact that Carthage does not support build a certain scheme is the problem. We want to have full support for the latest version of Xcode, so you can have a local script to remove that scheme before building with Carthage.
See the #1291 and https://github.com/Carthage/Carthage/issues/1227#issuecomment-285800169 for more.

Oh, sorry about that. Seems like I need to update to XCode 11 :/


updating opengl calls with the modern versions

I was just about to psuh an update over Model, Nodes and Mesh. I'm delay with an issue in the texture loading. I'm thinking that this is goiing to be needed again then. When I can solve the issue we can review modern opengl calls in my code.


fix: differentiate max witness script size upon context

MaxWitnessScriptSizeExceeded is used in the context of SegwitV0 and Tap, where each of max witness script size differs. Moreover, even in the same context of SegwitV0, max witness script size differ whether it's standard or consensus rule. I just let MaxWitnessScriptSizeExceeded receive param usize to differentiate max witness script size upon context, which can fix the wrong err message of "The Miniscript corresponding Script would be larger than MAX_STANDARD_P2WSH_SCRIPT_SIZE  bytes." when SegwitV0 consensus and Tap context

In 5a3989c1d1ad2ad89f93242e2b31dbbcb69df479:
Can you instead add two named fields to the error variant, max and got?

Good idea! I've changed err message including got bytes.


[MM-43401] Add Banner to Communicate Delinquency to Cloud Customers

Summary
This PR adds a non-dismissable banner to the webapp when a subscription is delinquent. The banner starts off blue, until the 90 day mark, where it flips to red. The only way to dismiss this banner is to settle up with your delinquency.
Note: The design's specify the copy to be Update your billing information now to keep Professional features - the webapp doesn't have any notion of product history, it can only look at the currently subscribed product, so after the 90 day mark this would read Update your billing information now to keep Cloud Starter features, which I don't think is desired. For now, I've gone with Update your billing information now to keep paid features, will update when I hear back from John.
Ticket Link
https://mattermost.atlassian.net/browse/MM-43401
Related Pull Requests

Has server changes
Has CWS changes

Screenshots
Pre 90 day mark:

Post 90 day mark:

Release Note

/release-note-none


Little to no validation occurs on a Minion on create

Looking at the current code:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/api/validation/validation.go#L826
we do no validation on a minion.Spec value, therefore any initial capacity or external ids avoid validation.
this can also result in our node controller when paired with a cloud provider to create invalid nodes.

Should be fixed to use new genericetcd! :)

You'd still need to teach the validation strategy to run through the node spec right?

Yes

@kazegusuri

@bprashanth assigned this to myself. Let me know if you were planning to work on it.

oh, no, but @kazegusuri's been doing some node/etcd work off late so just make sure you both don't tread toes :)

ValidateMinion and ValidateMinionUpdate are called even for now, and they are called via strategy once #5441 is merged.
I haven't touched validations to Node yet, so please do it thanks.

The problem is validate minion doesn't validate the spec, it stops with objectmeta.
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/api/validation/validation.go#L844

I added two validations to Spec:

ExternalID: required.
Capacity: cpu and memory required. We could schedule pods that don't specify resources to nodes that don't publish resources. But it looks like we assume that these fields are present at other places in the code.

That will be a breaking change for api consumers.  We should double check and announce that.
----- Original Message -----

I added two validations to Spec:

ExternalID: required.
Capacity: cpu and memory required. We could schedule pods that don't
specify resources to nodes that don't publish resources. But it looks like
we assume that these fields are present at other places in the code.

Reply to this email directly or view it on GitHub:
https://github.com/GoogleCloudPlatform/kubernetes/issues/4992#issuecomment-85096217

I am fixing the tests and checking the checking the current usages. Capacity is the one thats worrisome.
@dchen1107 believes that these values are assumed to be required already in other subsystems. We'll verify before committing the change.
@smarterclayton What's the best list to announce the change?

https://github.com/GoogleCloudPlatform/kubernetes/blob/master/docs/devel/api_changes.md :)
----- Original Message -----

I am fixing the tests and checking the checking the current usages. Capacity
is the one thats worrisome.
@dchen1107 believes that these values are assumed to be required already in
other subsystems. We'll verify before committing the change.
@smarterclayton What's the best list to announce the change?

Reply to this email directly or view it on GitHub:
https://github.com/GoogleCloudPlatform/kubernetes/issues/4992#issuecomment-85103108

Heh, it doesn't describe where to discuss changes.  So we should also add that as well.  kubernetes-dev and adding to a changelog were also described.
----- Original Message -----

https://github.com/GoogleCloudPlatform/kubernetes/blob/master/docs/devel/api_changes.md
:)
----- Original Message -----

I am fixing the tests and checking the checking the current usages.
Capacity
is the one thats worrisome.
@dchen1107 believes that these values are assumed to be required already in
other subsystems. We'll verify before committing the change.
@smarterclayton What's the best list to announce the change?

Reply to this email directly or view it on GitHub:
https://github.com/GoogleCloudPlatform/kubernetes/issues/4992#issuecomment-85103108

Fixed by #5929


Question on sharing service

So I have a question and not an issue (I apologize if this is the wrong spot!), but I have a UserPermission service that extends ObservableStore.
My question is, if I am on a roles component, is there a way to hit the StateChanged without adding the customerService to the constructor of the roles service?
Also, is there a way to subscribe to only one of the state changed services instead of all of them? so something like userPermissionService.StateChanged.UserPermissions.subscribe(x={do fancy stuff here})?
Thanks in advance!

I'm not clear on the first part of the question so let me walk through a scenario and see if that helps. Observable Store sends out change notifications made by a given service. So, if your UserPermission service changes something with setState(), any subscribers hooked to userPermissionService.stateChanged would get those changes. They wouldn't get changes made to the store through CustomersService unless they called customersService.stateChanged on it and subscribed. In sum, each service that extends Observable Service only sends out changes made by that particular service.
If you want to get changes made to the store by any service you can use the myServiceName.globalStateChanged option.
To know if a specific property changed such as userSettings, use the stateWithPropertyChanges: Observable<StateWithPropertyChanges<T>> option (https://github.com/danwahlin/observable-store#store-api). Here's an example:
this.userPermissionsService.stateWithPropertyChanges.subscribe(stateWithProps => {
    if (stateWithProps && stateWithProps.stateChanges.userSettings) {
      // Do something since the target property did change
    }
});

Doing that works well when you only want to run some code when a specific property changes.
As a heads-up, the latest version has a new function called getStateProperty() (see the link above for more details). Instead of retrieving (and cloning) the entire store, it'll let you pick a specific property from the store:
// Somewhere in a service
this.getStateProperty<YourPropertyType>('userSettings');

Thanks Dan! Sorry for the confusion on the question, but you definitely answered all of the questions I had.
Thanks!


typo in 5.11 test

Nurlan,
Thank you for the pull request! I have examined the test case and I do believe that -1 is vacuously palindromic just as +1 is since they are only one character with a sign attached. I went further and took my solution code from doing that same problem a couple months ago and uploaded it to my solutions branch which can be viewed here (that test case passes):
https://github.com/gardncl/elements-of-programming-interviews-solutions/blob/master/primitives/src/main/java/CheckIfIntegerIsPalindrome.java
According to a quick search online it seems that -1 is a palindrome, could you please make a case for this? Just because my code passes that test does not mean that it is correct, so please do help if I am incorrect.
Thank you!

Clay,
I'll put here 5.11 problem's description:
In this problem, you are to write a function which determines if the decimal representation of an integer is a palindromic string. For example, your function should return true on inputs (here some examples), and false on the inputs -1, 12, 100, and<PHONE_NUMBER>.

Thank you! Accepted.


Retain all metadata

Conversion should retain all the metadata

How can I do this?

There are these libraries:
https://github.com/polyfloyd/rust-id3
https://github.com/jameshurst/rust-metaflac
Or use ffmpeg as converter.
ffmpeg -i input.flac -ab 320k -map_metadata 0 -id3v2_version 3 output.mp3


Concept of Distribution Group in App Center doesn't now exist?

Has Teams replaced Distribution Groups as I can't find the "New Group" group button from the top of the screen?

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

ID: 5a807b36-339c-dd76-a6bb-132005991516
Version Independent ID: 8ffb33e3-09fd-fe41-742f-a9f4d11bf901
Content: Manage App Center Distribution Groups - Visual Studio App Center
Content Source: docs/distribution/groups.md
Service: vs-appcenter
GitHub Login: @JoshuaWeber
Microsoft Alias: JoshuaWeber

Okay, I have now found it in the Distribute menu, so it does exist :)

This is under distribution group settings, what even does this do?


App devs can use DCAP attestation for OE enclaves on Windows

The existing OE support for Windows only depends on the old Intel AESM stack and is not yet integrated with new DCAP dependencies necessary for remote attestation features in OE. To enable this, we need to:

[ ] Add DCAP dependencies and configuration instructions for Windows to OE project
[ ] (External) Ensure that Azure/Intel DCAP provider libraries are available.
[ ] Enable USE_LIBSGX configuration for Win32 code branches
[ ] Enable full report tests on Windows

@soccerGB

bors try

@CodeMonkeyLeet I think this is done. Can we close it?

Completed


Roll Engine from a9efb963de52 to 25262ad2d606 (2 revisions)

https://github.com/flutter/engine/compare/a9efb963de52...25262ad2d606
2021-08-24<EMAIL_ADDRESS>Roll Skia from 0f36d11c1dcf to 6f90c702e6c8 (4 revisions) (flutter/engine#28280)
2021-08-24<EMAIL_ADDRESS>Roll Dart SDK from abecc4bc4521 to 72f462ed2d34 (6 revisions) (flutter/engine#28279)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Hide monitor in maintenance on status page if the monitor is not list in the status page

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
UI Feature
🔖 Feature description
I use 3 monitors. (M1 M2 M3).
I create 2 status page (SP1 and SP2)
I add the M1 monitor on the SP1 status page (for customer 1)
I add all monitor on the SP2 page (for me, admin user)
When i put the M2 monitor in maintenance, there is a blue caption on the Status page SP2 with the text of the maintenance.
But on the SP1 page, the maintenance caption is still visible.
The customer 1 did not understand why he see maintenance information about monitor which are not list on the SP1 page
✔ Solution
I think the maintenance caption should be visible only if the monitor in maintenance is to be show on the status page.
❓ Alternatives
No response
📝 Additional Context
No response

@Lurikbolding I tried to reproduce your issue. However, I didn't get the same results. Here are some pictures of the way I set the stuff up:
Can you make sure you have the same settings? If it still doesn't work, could you provide more detailed steps to reproduce?

Pictures

Maintenance Settings (IMPORTANT)

Monitors

SP1

SP2

Hello
Sorry i did not see your first message.
I don't success to reproduce the issue. So i think i made a mistake the first time.


Remove youtube embed and replace with link card to page with embed

Fixes load issue on pages with large amounts of TV episodes

Closing with fixes, but will actively monitor for the captcha notice above. It appears to be a residue issue from the previous embed logic.


AutoAWQ_kernels 安装失败，是怎么回事呢

如果是   pip install -e .    安装卡到如下地方一直等待

Installing collected packages: autoawq_kernels   DEPRECATION: Legacy editable install of autoawq_kernels==0.0.6+cu124 from file:///mnt/chunshengwu/code/github/LLM-Drop/src/llmtuner/compression/quantization/AutoAWQ/AutoAWQ_kernels (setup.py develop) is deprecated. pip 25.0 will enforce this behaviour change. A possible replacement is to add a pyproject.toml or enable --use-pep517, and use setuptools >= 64. If the resulting installation is not behaving as expected, try using --config-settings editable_mode=compat. Please consult the setuptools documentation for more information. Discussion can be found at https://github.com/pypa/pip/issues/11457   Running setup.py develop for autoawq_kernels

如果是   pip install -e . --use-pep517，报错:
   File "/tmp/pip-build-env-663db4nq/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 320, in run_setup           exec(code, locals())         File "<string>", line 2, in <module>       ModuleNotFoundError: No module named 'torch'

emm，等了好久pip install -e .  是正确的


sqlite: 3.37.0 -> 3.37.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/sqlite/versions.
meta.description for sqlite is: "A self-contained, serverless, zero-configuration, transactional SQL database engine"
meta.homepage for sqlite is: "https://www.sqlite.org/"
meta.changelog for sqlite is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
/nix/store/nizqw1bm9whfjip5vg74rijfzjl9rnvv-sqlite-3.37.1-bin/bin/sqlite3 passed the binary check.
1 of 1 passed binary check by having a zero exit code.
1 of 1 passed binary check by having the new version present in output.
found 3.37.1 with grep in /nix/store/nizqw1bm9whfjip5vg74rijfzjl9rnvv-sqlite-3.37.1-bin
found 3.37.1 in filename of file in /nix/store/nizqw1bm9whfjip5vg74rijfzjl9rnvv-sqlite-3.37.1-bin

Rebuild report (if merged into master) (click to expand)

63129 total rebuild path(s)

37964 package rebuild(s)

37036 x86_64-linux rebuild(s)
26092 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath

ArchiSteamFarm
AusweisApp2
CHOWTapeModel
ChowKick
CuboCore.coreaction
CuboCore.corearchiver
CuboCore.corefm
CuboCore.coregarage
CuboCore.corehunt
CuboCore.coreimage
CuboCore.coreinfo
CuboCore.corekeyboard
CuboCore.corepad
CuboCore.corepaint
CuboCore.corepdf
CuboCore.corepins
CuboCore.corerenamer
CuboCore.coreshot
CuboCore.corestats
CuboCore.corestuff
CuboCore.coreterminal
CuboCore.coretime
CuboCore.coretoppings
CuboCore.coreuniverse
CuboCore.libcprime
CuboCore.libcsys
DisnixWebService
EBTKS
EmptyEpsilon
Fabric
LAStools
LASzip
LASzip2
Literate
MACS2
MIDIVisualizer
MMA
NSPlist
OSCAR
OVMF
OVMFFull
PlistCpp
QuadProgpp
R
SDL
SDL2
SDL2_gfx
SDL2_image
SDL2_mixer

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nizqw1bm9whfjip5vg74rijfzjl9rnvv-sqlite-3.37.1-bin \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A sqlite https://github.com/r-ryantm/nixpkgs/archive/f91334a9a24aed957d2039a2817ed8acbb87be34.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nizqw1bm9whfjip5vg74rijfzjl9rnvv-sqlite-3.37.1-bin
ls -la /nix/store/nizqw1bm9whfjip5vg74rijfzjl9rnvv-sqlite-3.37.1-bin/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @edolstra @np for testing.

@ofborg build sqldiff sqlite-analyzer

@ofborg build sqldiff sqlite-analyzer

@ofborg eval

@ofborg build sqldiff sqlite-analyzer


Existing names with duplicate counter IDs get added to AKAs but not Events

An object with a duplicate original ATLAS name will also have a duplicate counter.
Duplicate counter entries in the Events table are not allowed.  Result is:

First Event gets recorded
All subsequent events are not added to Events (even though coordinates are different), but ARE added to AKAs.

Suggested solution:

Update the error message for duplicate Events (currently incorrectly reporting duplicate AKAs).
Record all events that create a duplicate Event ID and save them somewhere. We will need these later.
Once all the historic data has been added, add the duplicate events as new events (meaning they'll be tacked onto the end of each year). They can't be added inline because this will violate the event ID of other existing events.

The problem with the above solution is that removing the counter from the insert (to add the objects at the end) works fine, but the AKA fails to insert because the internal object_id and survey_database are unique together. Solution is to change the survey_database for the duplicates to be something like "atlas4_dup".


builders unable to git clone after rake setup:delivery_build_nodes

While trying to replace existing delivery builder node with new builder nodes, running 'rake setup:delivery_build_nodes', a new builder_key is generated and dropped to /var/opt/delivery/workspace/etc/builder_key on each builder nodes; however builder's public key (as see through delivery UI) is not changed; it cause git clone fail on builder nodes;
manually copy public key that match the new builder_key into UI fixed the problem.

@ghao-ooyala thank you for reporting this issue. May I ask how you deleted the build-nodes? The only place we generate the builder key is in the setup_delivery_server.rb recipe so I wonder how did that run to do this behavior. :smile:

Salim: long story but here it goes.  We had 3 builders initially. Then we decided to created 3 new ones having different aws security groups. I edited ooyala.json and replaced the 3 old IPs with the 3 new IPs. I deleted the cache directory on the provisioner, ran setup:delivery_build_nodes then ran setup:cluster. Then we noticed the builders don't all show up when we ran "knife node status."  I ran reconfigure on the server and somehow the 3 builders showed up.
However Geng noticed that traffic still went to the 3 old builders instead.  That was when I modified ooyala.json again, putting back the 3 old IPs plus the 3 new IPs for a total of 6 builders. I didn't wipe out the cache this time, and just ran the regular builder nodes and cluster setup commands.  Then all the builders couldn't connect to the delivery server anymore.

@taimai @ghao-ooyala For curiosity; how did you clean the cache?

rm -rf .chef/delivery-cluster-data
rm -rf .chef/delivery-cluster-data-ooyala
chef exec rake maintenance:clean_cache

@taimai unfortunately the two rm commands is the main cause of this failure. When you ran them you deleted all the data of the cluster that lets the provisioning node manage the infrastructure.
The proper way to clean the cache is just by running the rake command you ran above.

I knew that was a bit aggressive :)

However I think the main issue here is that if a pub key already exists in the GUI, it won't be overwritten by a new pub key. That was why we keep getting "permission denied" on the git clone command.


Fix httplib API calls for empty body #126

Changes
Adapted the httplib interface calls to correctly pass std::string() instead of nullptr, plus simplifications.
Affected use-cases
Threw an exception when POST, PUT, etc. was called with empty request object.

Awesome 💯


OAK-9208 - Log unexpected writes to the paths designated as part of a non-default mount

OAK-9208 - Log unexpected writes to the paths designated as part of a non-default mount

@trekawek  - would you have time to review this?

The Travis error seems to come from the OSGi ITs, but it does not look related to my changes
ERROR: Bundle org.apache.jackrabbit.oak-segment-tar [45] Error starting file:/tmp/1600698042537-0/bundles/org.apache.jackrabbit.oak-segment-tar_1.35.0.SNAPSHOT.jar (org.osgi.framework.BundleException: Unable to resolve org.apache.jackrabbit.oak-segment-tar [45](R 45.0): missing requirement [org.apache.jackrabbit.oak-segment-tar [45](R 45.0)] osgi.wiring.package; (osgi.wiring.package=com.oracle.svm.core.annotate) Unresolved requirements: [[org.apache.jackrabbit.oak-segment-tar [45](R 45.0)] osgi.wiring.package; (osgi.wiring.package=com.oracle.svm.core.annotate)])

I assumed lazy consensus :-) and committed.


for #675 COM_QUERY procedure

Complete COM_QUERY procedured.
Test result is similar to COM_STMT_EXECUTE.

Codecov Report

Merging #754 into dev will decrease coverage by 0.44%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##              dev     #754      +/-   ##
==========================================
- Coverage   64.57%   64.12%   -0.45%
==========================================
  Files         483      483
  Lines        9406     9470      +64
  Branches     1546     1556      +10
==========================================
- Hits         6074     6073       -1
- Misses       3000     3065      +65
  Partials      332      332

Impacted Files
Coverage Δ

...bc/proxy/backend/mysql/MySQLPacketQueryResult.java
0% <0%> (ø)
:arrow_up:

...ysql/packet/command/text/query/ComQueryPacket.java
0% <0%> (ø)
:arrow_up:

...backend/common/StatementExecuteBackendHandler.java
0% <0%> (ø)
:arrow_up:

...proxy/backend/common/SQLExecuteBackendHandler.java
0% <0%> (ø)
:arrow_up:

...ce/OrchestrationShardingDataSourceFactoryBean.java
66.66% <0%> (-11.12%)
:arrow_down:

Continue to review full report at Codecov.

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


Indentation is different in when entry of Trailing commas

Expected Behavior of the fomat
Invalid:
fun isReferenceApplicable(myReference: KClass<*>) = when (myReference) {
    Comparable::class,
    Iterable::class,
    String::class, // trailing comma
    -> true

    else -> false
}

Valid:
fun isReferenceApplicable(myReference: KClass<*>) = when (myReference) {
    Comparable::class,
    Iterable::class,
    String::class, // trailing comma
        -> true

    else -> false
}

Context
Complies with Kotlin coding conventions.
See: https://kotlinlang.org/docs/coding-conventions.html#when-entry

I don't understand exactly what's the request here. A rule is doing it's work wrongly? You are talking about format so this issue should be open to ktlint. They are the ones that have the logic, we only wrap those rules.

This issue is waiting for authors feedback since 30 days. Please comment providing the requested feedback or this issue will be closed in 7 days.


Update Wavlake embed URL, add support for album & artist links

As described in the title, I've created a new subdomain for embeds from Wavlake to simplify the URL parsing. This update also adds support for links to artist and album pages (in addition to tracks).

Updated with suggested changes @SamSamskies


Role Throttling?

When Executing the following series of command in sequence...
bot.removeMemberFromRole(user, message.server.rolesOfUser(user), function(err) {
  if (err) console.log(err);
  bot.addMemberToRole(user, theRole, function(err) {
    if (err) console.log(err);
  });
});

The code is supposed to remove all roles from a user, then add back a single role.
Sometimes only one of these commands properly executes. It is intermittent and not consistent (sometimes this works correctly). I have witnessed several executions where a user ends up with a role, ends up without a role being removed and in some cases the role is both properly added and a different role is removed.
Should be fairly easy to replicate. I do not run this pair of commands multiple times, usually once in a long while. I.E: I am not running the above sequence chained together to hit a rate limit.

Note, #472 and #476 weren't attempting to implement the above commands nested properly for async behavior.
I have attempted to structure the code in numerous ways to work around this issue, but at this point its either a problem with the Discord API (likely) or the library in question.

I can confirm removing multiple roles at the same time from one user does not work as intended, though I have not tried in the last month. What I think you have to do to "fix" this though it's probably a lib issue is remove the roles one after another, in the callback.
Bassically the easiest way to do that would be an async recursive function, that removes a random roll, then calls itself till their are no roles left. it would have to return a promise. An example implementation with psudo code would be.
function removeAllRoles(user, server) {
    return new Promise((resolve, reject)=>{
    removeARandomRole(user, server).then(()=>{
        if(remainingRoles > 0) {
            resolve(removeAllRoles(user, server));
        } else {
            resolve(true);
    })
}

If that makes any sense.

Both requests are trying to over write each other, you need to give one request time to be executed (@macdja38's solution fixes this)


Add API support for querying tags and commit details

I'm currently trying to integrate Gogs support for the DUB package registry (D's package manager), which needs to be able to list all tags of a repository, as well as some details about a commit (most importantly, the commit date). Branches already work, so listing tags looks like a simple addition.

+1

+1

did this feature supported? tags related api


Incompatible with Storybook 8.0.6

After upgrading Storybook to the latest v8.06, I get the following warnings in the CLI:
WARN The following packages are incompatible with Storybook 8.0.6 as they depend on different major versions of Storybook packages:
WARN -<EMAIL_ADDRESS>WARN  Repo: https://github.com/break-stuff/wc-storybook-helpers#readme

I am also experiencing this on 8.0.8

Thank you for letting me know! This is likely because we also support v7 APIs, but we should be able to fix this.

This should now be fixed in release 2.0.2

Closing for now. Let me know if you run into any issues.

Got around to trying out the latest but getting the following
Here is my full story
import { Meta, StoryObj } from '@storybook/web-components';
import './scoped-notification.slds';
import { NjcSldsScopedNotification } from './scoped-notification.slds';
import { getWcStorybookHelpers } from 'wc-storybook-helpers';
const { events, args, argTypes, template } =
getWcStorybookHelpers('nsc-scoped-notification');
const meta: Meta = {
title: 'Components/ScopedNotification/Slds',
component: 'nsc-scoped-notification',
args,
argTypes,
parameters: {
actions: {
handles: events,
},
},
};
export default meta;
type Story = StoryObj<NjcSldsScopedNotification & {slotContent: string}>;
export const Example: Story = {
render: (args) => template(args),
args: {
slotContent: 'Slot',
theme: 'success',
},
};


After esp_restart chip won't restart

We are running tests on ESP32 (IDF v2.1) and after a while when we do a esp_restart the module never restarts unless we do a HW reset. It is hard to reproduce but we have made some observation. The tests where it happens after a while are usually short, doing something then restarting the module, some tests are writing to one label in nvs and every test are erasing that label just before the restart.
What we have tested:

Toggling some pins in esp_restart_noos procedure it seems like it never gets to the end (seems to hang somewhere around esp_dport_access_int_deinit).
Adding a delay between esp_cpu_stall and esp_dport_access_int_deinit made it take longer before it happened, so long we thought it was fixed but then we got a problem with tests that used wifi, see point 3
Doing a esp_wifi_stop in the before calling esp_restart made it take longer before it happened, made us think the delay and this where the solution until it happened again
The power consumption is normal when it happens, tried with external power supply
The problem isn't completely solved by doing a HW reset, when it has booted up after HW reset we can reproduce the problem quicker. Only a complete wipe of flash restores the devices to normal.
Swapped out the chip for new ones, problem still exists.

Looking into similar reports for latest git version. There are three related issues here, i am not sure which one you are hitting:

esp_dport_access_int_deinit will wait on a critical section. Depending on what the other CPU is doing, this wait may never complete. In master branch there is a new function, esp_dport_access_int_abort, which will not wait for critical section and will just disable DPORT register access protection.

There is a small possibility that when esp_cpu_stall(other_core_id); is called, deadlock will happen. Other CPU may become stalled at the time when there is a D(or I)-bus transaction active. This may prevent current CPU from accessing the same bus, effectively stopping the program execution. The current solution (which we are still verifying) is to reset the other CPU before stalling it, e.g.

    SET_PERI_REG_MASK(RTC_CNTL_OPTIONS0_REG,
            other_core_id == 0 ? RTC_CNTL_SW_PROCPU_RST_M : RTC_CNTL_SW_APPCPU_RST_M);

Finally, there is an issue that rtc_clk_cpu_freq_set may disable the PLL before the switch from PLL to XTAL clock has completed. We're running more tests to be sure we choose the right value this time, but you may try increasing DELAY_CPU_FREQ_SWITCH_TO_XTAL_WITH_150K value from 20 to 40 (that's in rtc_clk.c). It is unlikely that you are hitting this case though, as by the time when rtc_clk_cpu_freq_set is called, RTC watchdog is already enabled.

Finally, moving the RTC watchdog init procedure to the top of esp_restart_nonos will help recover the device if issues 1 or 2 happen. (This is also something we are testing now.)

Thanks @igrr !!
We have confirmed that at least number one is happening, we also notice that configuring our flash with 3.3V instead of 1.8V makes it easier to reproduce.
Is it safe to call esp_dport_access_int_abort instead of esp_dport_access_int_deinit in esp_restart_noos and ignore where the other CPU is stalled

Sorry, i have misread your last sentence, it was in fact a question.
Yes, it is safe to call esp_dport_access_int_abort provided that the other CPU is stalled. As mentioned in point 2, before stalling the other CPU you have to use the SET_PERI_REG_MASK... bit from my previous comment before stalling the other CPU, to make sure that the other CPU is not in the middle of s32ci1 instruction.

@igrr no worry apparently i forgot the question mark


keybase help gpg 'keybase pgp select' needs to be updated

keybase help gpg and keybase pgp help select contradict.
keybase help gpg gives outdated 'keybase pgp select' advice -- according to keybase pgp help select steps (3) and (4) below no longer occur by default, and now require the --import flag, according to keybase pgp help select.  Also keybase pgp select --no-import no longer works.
keybase help gpg gives:
'keybase pgp select' - Pull a PGP key out of your GPG keyring, and "select" it
  for use on keybase. This will: (1) sign it into your signature chain with your
  local device key; (2) push this signature and the public PGP key half to the
  server; (3) copy a version of secret key to your local keybase keychain;
  and (4) encrypt this copy with your keybase passphrase via local-key security
  (see "keybase help keyring"). Keybase takes steps (3) and (4) so that subsequent
  PGP operations (like sign and decrypt) don't need to access your GPG keyring again.
  Once running this command, you wind up in state similar to that following
  "keybase pgp gen" above. The difference is that you've used gpg to generate
  the key rather than keybase.

keybase pgp help select gives:
DESCRIPTION:
  "keybase pgp select" looks at the local GnuPG keychain for all
  available secret keys. It then makes those keys available for use with keybase.
  The steps involved are: (1a) sign a signature chain link with the selected PGP
  key and the existing device key; (1b) push this signature and the public PGP
  key to the server; and if "--import" flag is passed: (2a) copy the PGP secret half
  into your local Keybase keyring; and (2b) encrypt this secret key with Keybase's
  local key security mechanism.

  By default, Keybase suggests only one PGP public key, but if you want to,
  you can supply the "--multi" flag to override this restriction. If you
  want your secret key imported into the local Keybase keyring, then use
  the "--import" flag. Importing your secret key to Keybase keyring makes
  it possible to use Keybase PGP commands like "pgp decrypt" or "pgp sign".

  If you don't want to publish signature chain link to Keybase servers, use
  "--no-publish" flag. It's only valid when both "--no-publish" and "--import"
  flags are used.

  This operation will never push your secret key, encrypted or otherwise,
  to the Keybase server.

OPTIONS:
  --multi	Allow multiple PGP keys.
  --import	Import private key to the local Keybase keyring.
  --no-publish	Only import to Keybase keyring, do not publish on user profile.

Thanks, here's a PR: https://github.com/keybase/client/pull/14900

Thanks for the report, it's fixed in master.


Chasing camera sort of stottering

Hi, when I use the chase camera it is not fluently riding when going fast. If I change it to orbit it is 100% fluently.
Am i doing something wrong maybe or is it a known issue?

Hello. I can't really reproduce this issue.
Does it also happen to you when you play the example scenes included in the addon?

The chase camera is the only one that calculates interpolation for the distance to the car, so it's the only one that could have these issues.
Can you reproduce the issue in a completely new empty project?
As in just: start new project -> install the addon (and turn it on in project settings) -> run the example scenes.
If not, then it might be a problem in your current project. And in that case, if you could send me your project or link me to a repo, I could take a look at it, to see if I can figure out what could be the issue.

I will try this evening. Maybe I found something because when I drive the car in that level which is on a Terrain3D ground I see that FPS is mostly around 55 and I read that under 60 it could jitter?
I will try something if I can raise the FPS somehow and see what happens then.

I found it, was my worldenvironment what was not good. I already saw the fps was bad and I searched why that was. Now a new worldenvironment created and the fps stays good. The car is now acting like it should. Sorry for the misunderstanding. Your cameras are working great :)

Nice to hear. :)
I'm gonna leave this open for now, until I take a deeper look at the camera code. I don't think this should happen even with low frame rates, so there might still be something wrong there.

I also are testing with different other interactions like with simplegrasstextured and especially the interactive part that grass is gone around the car. You have to disable a layer on the cull mask so the camera wont see a particulair mesinstance. Only the orbit camera disables this, other cameras will not disable the view of it. Have to figure out why.

It looks like disable a layer of 3d camera not working, am i right?

Ah, yes, most of the cameras use an internal camera object for their actual effect. They only inherit from Camera3D because some of them (e.g. the cockpit camera) don't use child cameras, but still require previewing in the editor.
https://github.com/Skaruts/racing_cameras/blob/eb2af2d3a7a07e8e09169bc093aba8942fbb5c4c/addons/sk_racing_cameras/src/cameras/race_cam.gd#L50
So, currently, you'll have to change the internal _cam settings (through code) to achieve that kind of thing.
I'll see if I can figure out a way to expose the internal camera's settings, somehow.

I thought this would help:
func setCullMaskValue(): 	_cam.layerNumber = 17 	_cam.value = false
But I am wrong :(

Try this function:
https://docs.godotengine.org/en/stable/classes/class_camera3d.html#class-camera3d-method-set-cull-mask-value

Thanks,
If I do it in the _on_ready it works:
func _on_ready() -> void: 	_cam_pos = Vector3(0, 0, default_distance) 	_cam.position = _cam_pos 	_pivot2.rotation.x = deg_to_rad(-25) 	**_cam.set_cull_mask_value(17, false)**


Colorbar tooltips

otherwise creates this problem which we do not want

i've also added ChromeHeadless in this PR , UTs run faster now 👍

Codecov Report

Merging #651 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #651      +/-   ##
==========================================
+ Coverage   99.71%   99.71%   +<.01%
==========================================
  Files         261      261
  Lines        6683     6698      +15
  Branches     1561     1563       +2
==========================================
+ Hits         6664     6679      +15
  Misses         19       19

Impacted Files
Coverage Δ

src/components/colorBar/ColorBar.spec.tsx
100% <100%> (ø)
:arrow_up:

src/components/colorBar/ColorBar.tsx
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[10] How to use compat with preact@10

Hi,
there is no preact-compat@4 on npm, but it get's released under preact@10 as preact/compat. So the right way to use compat now is to replace the old preact-compat with preact/compat and remove the preact-compat dependency?
If that's true, then it should be documented in the README and the README shouldn't just link to the "outdated" (from the point of view of preact@10) preact-compat repo.

At this point Preact X is in alpha so updating the docs would be a bit premature. Your assumption is right preact-compat is getting replaced with preact/compat.

Agree that updating the README right now would only lead to confusion. We should do that when we're going gold 👍


Create docker image with service

@logachev just to double check, all of the deleted files are the full .net framework versions correct?


Expose a Configure button in the terminal dropdown

Idea: Show a config button in the terminal dropdown in order to increase discoverability of the settings.
cc @kieferrm

implemented by #110874


ADS: Internal Features Settings migration

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

feature/david/ads_release

PR #2576   👈

This comment was auto-generated by Graphite.


[BUG] Moving a token hindered by tooltip

Describe the bug
When selecting a token to move, the tool-tip does not disappear once the token is dragged. Compare this to the ruler use where once you begin drawing a distance, the tool-tip goes away so that you can see clearly where you are measuring to.
To Reproduce
Steps to reproduce the behavior:

Click on a token to move it and observe the tool-tip as you move the token.
Click on a token to measure with the ruler, once your mouse starts to move to measure, the tool-tip goes away.

Screenshots
If applicable, add screenshots to help explain your problem.
Please complete the following information:

OS: Windows 10 and OS X Big Sur
Browser: Chrome
Game version: 9.280
Game system and version: dnd5e 1.6.3 and Token Tooltip Alt 3.1.5

Additional context
Add any other context about the problem here.
An export of your tooltip settings
Please provide an export of your settings, found by going into the Tooltip manger and clicking Export.
{"gmSettings":{"default":{"items":[{"disposition":"FRIENDLY","items":[{"value":"Test","icon":"","isFunction":false,"expression":false,"isNumber":false,"color":"#30dfa8"}]},{"disposition":"NEUTRAL","items":[]},{"disposition":"HOSTILE","items":[]}],"static":{"displayNameInTooltip":true,"useAccentEverywhere":false,"accentColor":"#92152c","tokenDispositions":["FRIENDLY","NEUTRAL","HOSTILE"],"useAccentColorForEverything":false}},"character":{"items":[{"disposition":"FRIENDLY","items":[]},{"disposition":"NEUTRAL","items":[]},{"disposition":"HOSTILE","items":[]}],"static":{"displayNameInTooltip":true,"accentColor":"#0698d1","useAccentColorForEverything":false,"tokenDispositions":["FRIENDLY","NEUTRAL","HOSTILE"]}},"npc":{"items":[{"disposition":"FRIENDLY","items":[]},{"disposition":"NEUTRAL","items":[]},{"disposition":"HOSTILE","items":[]}],"static":{"displayNameInTooltip":true,"accentColor":"#d136db","useAccentColorForEverything":false,"tokenDispositions":["FRIENDLY","NEUTRAL","HOSTILE"]}},"vehicle":{"items":[{"disposition":"FRIENDLY","items":[]},{"disposition":"NEUTRAL","items":[]},{"disposition":"HOSTILE","items":[]}],"static":{"displayNameInTooltip":true,"accentColor":"#e159f4","useAccentColorForEverything":false,"tokenDispositions":["FRIENDLY","NEUTRAL","HOSTILE"]}}},"playerSettings":{"default":{"items":[{"disposition":"OWNED","items":[]},{"disposition":"FRIENDLY","items":[]},{"disposition":"NEUTRAL","items":[]},{"disposition":"HOSTILE","items":[]}],"static":{"displayNameInTooltip":"OWNED","useAccentEverywhere":false,"accentColor":"#eeb391","tokenDispositions":["OWNED","FRIENDLY","NEUTRAL","HOSTILE"],"useAccentColorForEverything":false}},"character":{"items":[{"disposition":"OWNED","items":[]},{"disposition":"FRIENDLY","items":[]},{"disposition":"NEUTRAL","items":[]},{"disposition":"HOSTILE","items":[]}],"static":{"displayNameInTooltip":"OWNED","accentColor":"#1dea42","useAccentColorForEverything":false,"tokenDispositions":["OWNED","FRIENDLY","NEUTRAL","HOSTILE"]}},"npc":{"items":[{"disposition":"OWNED","items":[]},{"disposition":"FRIENDLY","items":[]},{"disposition":"NEUTRAL","items":[]},{"disposition":"HOSTILE","items":[]}],"static":{"displayNameInTooltip":"OWNED","accentColor":"#68a05d","useAccentColorForEverything":false,"tokenDispositions":["OWNED","FRIENDLY","NEUTRAL","HOSTILE"]}},"vehicle":{"items":[{"disposition":"OWNED","items":[]},{"disposition":"FRIENDLY","items":[]},{"disposition":"NEUTRAL","items":[]},{"disposition":"HOSTILE","items":[]}],"static":{"displayNameInTooltip":"OWNED","accentColor":"#0c4953","useAccentColorForEverything":false,"tokenDispositions":["OWNED","FRIENDLY","NEUTRAL","HOSTILE"]}}}}

this can be greatly ameliorated by adding the 'refreshToken' hook to the array in removeTooltipHandlers() in TTAHooks.js, like so (~ lines 57-59?):
    ['preUpdateToken', 'refreshToken', 'canvasPan', 'deleteToken'].forEach((hook) => {
      addHookHandler(hook, HOOK_TYPE.ON, TooltipFactory.removeTooltips.bind(TooltipFactory));
    });

there's a few knock-on effects I found undesirable, but I also solved those by moving both 'refreshToken' and 'preUpdateToken' to a second array directed at a different callback I added to TooltipFactory.js (removeControlledTooltips(token) {if (token.controlled) this._removeTooltips();}); it was a problem for some specific shenanigans I'm pulling, but is probably entirely unnecessary for most people, and maybe even unwanted


Improve formatting

Normalized how the cli.go package is formatted in the README.

Thanks for the clean-up!


Always use one layout view for files, regardless of content.

What feature or improvement do you think would benefit Files?
I like "Details" view in layout. Always. The unpredictability of the other layouts is unhelpful (its slower to scan rows than to scan columns), as well as lack of details, e.g. date, file size. My choice, I know. (Its the first thing I change in a new Windows  installation)
So add an option to "fix" the view, with a selection of preferred view.
Maybe a "default" option in the Layout selection (the three little squares icon).
Requirements
Amend the UI to permit a selection. Check for this choice at point of display.
Files Version
Version: 2.4.40
Windows Version
Windows 11 Pro 22H2 22621.1265
Comments
Windows Explorer gets this wrong: it does not properly apply to every folder, even when that choice is made (e.g. new USB devices).
(It is possible I missed this option in the various menus :( )

I think this is already implemented in Settings > Folders > Sync layout and sorting preferences across directories


if repeat option for the panels are in use, they are not rendered in pdf.

Hi,
if repeat option for the panels are in use, they are not rendered in pdf.
Only one panel is rendered (which is base).

This is a duplicate of #55 and #79. Since the reporter uses the grafana dashboards API to get the dashboard definition, I am not sure how that is supposed to interact with templating (repeating rows).
Currently there is no workaround for this, but I have not investigated it in detail. If you want to add more insight please comment on #79.


Post processing source links in generated typedoc

Post processing source links in generated typedoc
Description

The shell script which generates the Node.js SDK API documentation now does some postprocessing to change the source code links. The typedoc tools by default inserts the git repository of the developer at the time of execution. The shell script has a variable DOCURI which needs to be adjusted to the final URI when the code is released. The default value just points to the current master branch which might from the release source code eventually. Upon release the should be generated with a URI using the respective release branch.
Motivation and Context

Source code links in generated API doc for Node.js SDK point to developers git repo.
How Has This Been Tested?

Regenerated doc (run sdk/node/makedoc.sh) and checked that links point to master branch on github and not my developer repo
Checklist:

[x] I have added a Signed-off-by
[x] Either no new documentation is required by this change, OR I added new documentation
[x] Either no new tests are required by this change, OR I added new tests
[n/a] I have run goimports, go vet, and golint. I have cleaned up all valid errors and warnings in code I have added or modified. These tools may generate false positives.  The goal is clean, consistent, and readable code.

Signed-off-by: Thomas Eirich<EMAIL_ADDRESS>

LGTM
Reviewed-by: Greg Haskins<EMAIL_ADDRESS>


Switch search button to outline

This keeps it from changing size on hover

@gabalafou can you review this one? For some reason I can't request a review from you in the usual GitHub way

I see, this is included in https://github.com/pydata/pydata-sphinx-theme/pull/1564
I hope that one can be merged soon!


Support Ceph/RBD

Hi, developers
I am a IaaS engineer and I also focus on Ceph/RBD. Does ZStack support Ceph/RBD as backend storage ? Thanks !
And I create a QQ Group: 410185063, named "ZStack中国社区", welcome to anyone to join us. Thanks.
Best Regards,
Star Guo

Thanks for initializing ZStack user group in QQ!

would like to know the answer also...

Hi @samos123 , Ceph will be supported in ZStack 0.9 in this Sept. For more ZStack features plan, please find out ZStack roadmap in http://zstack.org/blog/roadmap.html

this has been supported in 0.9


package.json should have main attribute

It would be a lot cleaner for users of the package if it had a main attribute as described in this SO answer.
https://stackoverflow.com/questions/22512992/node-js-package-json-main-parameter
I think the main should be:
'main': 'build/js/messenger'

I found a work around with webpack.
given the current state of the package.json, you have to do this:
import "messenger-hubspot/build/js/messenger.js";
but if you add this to the plugin section of your webpack config, you can work around the lack of a main tag:
        new webpack.NormalModuleReplacementPlugin(/^messenger$/, 'messenger-hubspot/build/js/messenger.js'),
then you can just do this to import the module:
import "messenger"


[backport 2.0.x] Optimize TSI series iteration

Backport the fix for  #22314 to the 2.0 branch

Duplicate of #22518


Article comparing performance of go-linq vs others

For maintainers, as FYI:
https://medium.com/wesovilabs/koazee-vs-go-funk-vs-go-linq-caf8ef18584e

The benchmark seems inaccurate. I'll write it to the guy.

@ahmetb @cleitonmarx guys, I've published a reply to the post.
https://medium.com/@kalansoft/hi-iván-corrales-solera-8459e3eed4c6

Thanks that’s cool I didn’t understand the specifics but thanks for taking the time.

@ahmetb the idea is that koazee utilizes a push model. So it manipulates data the same order you write functions. E.g. if you take an array of 100 elements, filter it and take just 1 element - it will filter all the data into a new array first and take the first element afterwards. While go-linq uses a pull model and goes the other direction, so in the example above it will look through the initial array until the first element that satisfies the condition. No extra work.
If you finally decide to move go-linq to a team account I will make a second repo with benchmark comparison to the competitors. Also, I have plans to work on go-linq performance - introduce hints similar to rust iterators. Having a separate repo will help me to measure and show the performance difference.

Honestly since we're mostly in maintenance mode and there'll be 1 repo, creating a go-linq org might not be all that useful. If you would like to write an article, you can create a personal repo and publish benchmarks there as well?


Readme should tell the user to install briefcase before telling them to use it

This is already fixed:
$ mkdir beeware
$ cd beeware
$ python3 -m venv venv
$ source venv/bin/activate
(venv) $ pip install --pre beeware

Confirmed; looks like we've fixed this along the way.


Support ZendService-ReCaptcha v3

This supports v2 of the ReCaptcha API and update the CS standards to zend-coding-standard.

@weierophinney: Documentation added along with BC methods and keys for privKey & pubKey.


How can i get the old value of of input type number on @change event

I want to deduct the quantity of the item to another quantity
For Example if
Item1.quantity = 100
Item2.quantity = 1000
this quantity will change on the @change event listener
if i deduct 2 in item1
item1.quantity = 98
item2.quantity = 1002
how can i achieve this
it seems that i dont get the old value
                        :ref="item.stock"
                        :name="item.stock"
                        v-model.lazy="item.quantity" type="number" @change="deductQuantity(item,$event)">

Please make sure to read the Issue Reporting Guidelines before opening new issues. Questions should be posted on the forum or StackOverflow. Thanks.

You can use a watcher on the value (item.quantity)


[code-infra] Closer sync with eslint config of codebase

A step to simplify the eslint config in the codebase.

Did you see that this eslint is in the process of deprecating its config format?

The new format seems considerably easier to understand in terms of extending lint configs.

I briefly looked at it, but saw we are on ESLint v8 not v9 so ignored. I hated these files so much: https://github.com/mui/material-ui/blob/next/packages/api-docs-builder/.eslintrc.js. I'm happy it won't be possible anymore.


No module named 'airdialogue'

Hi, Thanks for your excellent work.
I double-checked the projects.
I think we're still missing some documents or packages.
Looking forwards to your reply.
Best,
Xiaowen

Oh sorry, I just updated the requirements. You need to run: "pip install airdialogue-essentials"


Can't change require line?

In current version of riot-tag-loader, var riot = require('riot') is hard-coded (line).
If we want Riot tags to treat as ES6 Modules, require line should be import riot from 'riot'.
The line shoud be configured, at least the selection of require/import.

this issue was solved in the new v4.0.0 version