Updates for experimental data handling

update of Functions/Plot/init.py to consier experimental data
is_contour option for 'pcolor' plot of plot_3D
z_min and z_max considered  for 'pcolor' plot of plot_3D

Test are ok

Thank you for your contribution :)


Benchmarking docker not stopped if a run fails

mlagility.api.devices.BenchmarkException: Error: Failure to run model using ORT - ERROR:root:Subprocess failed with command: docker run -d --rm -v tmp_cache/sample/x86_benchmark:/app --name mlagility-onnxruntime-mlas-ep mlagility-onnxruntime-image and error message: docker: Error response from daemon: Conflict. The container name "/mlagility-onnxruntime-mlas-ep" is already in use by container "5a5e5eb76a6069b3a8c600d9d597951c8673db404c87b90ba82d51ba1b8a9a7d". You have to remove (or rename) that container to be able to reuse that name.

Container needs to be removed in any case so the container name does not conflict with the next run.

I am able to reproduce this problem by raising a ValueError within the container:
Traceback (most recent call last):
  File "/net/home/jfowers/miniconda3/envs/mlaof/bin/benchit", line 33, in <module>
    sys.exit(load_entry_point('mlagility', 'console_scripts', 'benchit')())
  File "/home/jfowers/mlagility/src/mlagility/cli/cli.py", line 483, in main
    args.func(args)
  File "/home/jfowers/mlagility/src/mlagility/cli/cli.py", line 94, in benchmark_command
    benchmark_model(
  File "/home/jfowers/mlagility/src/mlagility/api/model_api.py", line 117, in benchmark_model
    perf = cpu_model.benchmark(backend=backend)
  File "/home/jfowers/mlagility/src/mlagility/api/ortmodel.py", line 19, in benchmark
    benchmark_results = self._execute(repetitions=repetitions, backend=backend)
  File "/home/jfowers/mlagility/src/mlagility/api/ortmodel.py", line 74, in _execute
    devices.execute_ort_locally(
  File "/home/jfowers/mlagility/src/mlagility/api/devices.py", line 623, in execute_ort_locally
    raise BenchmarkException(
mlagility.api.devices.BenchmarkException: Error: Failure to run model using ORT - Traceback (most recent call last):
  File "/home/jfowers/mlagility/examples/cli/onnx/tmp_cache/sample/x86_benchmark/setup_ort.py", line 131, in run
    perf_result = execute_benchmark(onnx_file, docker_name, num_iterations)
  File "/home/jfowers/mlagility/examples/cli/onnx/tmp_cache/sample/x86_benchmark/setup_ort.py", line 74, in execute_benchmark
    raise ValueError(f"Execution of command {cmd} failed with stderr: {stderr}")
ValueError: Execution of command docker exec mlagility-onnxruntime-mlas-ep /usr/bin/python3 /app/run_ort_model.py --onnx-file /app/onnxmodel/model.onnx --iterations 100 failed with stderr: b'Traceback (most recent call last):\n  File "/app/run_ort_model.py", line 90, in <module>\n    run_ort_profile(\n  File "/app/run_ort_model.py", line 26, in run_ort_profile\n    onnx_session.run([output_name], input_feed)\n  File "/usr/local/lib/python3.9/dist-packages/onnxruntime/capi/onnxruntime_inference_collection.py", line 200, in run\n    return self._sess.run(output_names, input_feed, run_options)\nonnxruntime.capi.onnxruntime_pybind11_state.InvalidArgument: [ONNXRuntimeError] : 2 : INVALID_ARGUMENT : Unexpected input data type. Actual: (tensor(double)) , expected: (tensor(float))\n'

And then the subsequent run of benchit produces the "name collision" error above:
Info: Benchmarking on local x86...
Traceback (most recent call last):
  File "/net/home/jfowers/miniconda3/envs/mlaof/bin/benchit", line 33, in <module>
    sys.exit(load_entry_point('mlagility', 'console_scripts', 'benchit')())
  File "/home/jfowers/mlagility/src/mlagility/cli/cli.py", line 483, in main
    args.func(args)
  File "/home/jfowers/mlagility/src/mlagility/cli/cli.py", line 94, in benchmark_command
    benchmark_model(
  File "/home/jfowers/mlagility/src/mlagility/api/model_api.py", line 117, in benchmark_model
    perf = cpu_model.benchmark(backend=backend)
  File "/home/jfowers/mlagility/src/mlagility/api/ortmodel.py", line 19, in benchmark
    benchmark_results = self._execute(repetitions=repetitions, backend=backend)
  File "/home/jfowers/mlagility/src/mlagility/api/ortmodel.py", line 74, in _execute
    devices.execute_ort_locally(
  File "/home/jfowers/mlagility/src/mlagility/api/devices.py", line 623, in execute_ort_locally
    raise BenchmarkException(
mlagility.api.devices.BenchmarkException: Error: Failure to run model using ORT - ERROR:root:Subprocess failed with command: docker run -d --rm -v /home/jfowers/mlagility/examples/cli/onnx/tmp_cache/sample/x86_benchmark:/app --name mlagility-onnxruntime-mlas-ep mlagility-onnxruntime-image and error message: docker: Error response from daemon: Conflict. The container name "/mlagility-onnxruntime-mlas-ep" is already in use by container "f8ddd7f10cb961a538b970988a9ea674b24abeb22403f9e85ef57e9e3718a41a". You have to remove (or rename) that container to be able to reuse that name.
See 'docker run --help'.


Error Program Execution Failure No user tests were run.

Challenge http://freecodecamp.com/challenges/waypoint-iterate-with-javascript-for-loops has an issue. Please describe how to reproduce it, and include links to screenshots if possible.

@tiagomf1 thanks for posting this issue!
Please, provide the description of your issue/code/screenshot, thanks!.

Please review the Guidelines for Contributing, thank you!.

I open again and run normaly... thanks :)

OK, closing this issue, feel free to reopen it if you still have a problem.


Use AWS provider v4

This may take some work, because the modules we're using likely expect us to be on AWS provider v3.

This is important for long-term maintainability.


requestLegacyExternalStorage fails for older SDK versions

AndroidManifest.xml:3: error: No resource identifier found for attribute 'requestLegacyExternalStorage' in package 'android'

Deleting that part from the xml fixes it. I don't have enough knowledge around Android to know if you can just remove it without consequences. Try on a few SDK versions below 29. Currently on 24 and it fails.
Btw this project is amazing. Every few years I google 'Android + C` and everyone says it's impossible. Heh.

You cannot remove it if you want to access files at all if targeting Android 29+.  It's frustrating, but, I know no other way forward.  We could make the default target 28, I suppose... But that is absolutely the lowest supported by the store.

@MrOneTwo if you ever do find a better solution, please feel free to open a new issue, but rest assured, google will find this thread so any poor souls who run past it will see your answer.


Listing tabs Vertically?

I'm trying to implement this in a personal project and while I've easily styled the tabs to appear vertically and to the left of the tab content, I'm not sure how to go about getting a drop-down when the tabs extend outside of the container height. I'm very novice with javacript/jquery, so I've been unable to do this on my own. Any help / usefull comments are appreciated!

Hey @alishabussart. Do you have a demo I can look at? JSBin or Codepen?

@gabrieltomescu Thanks for the reply! Here's a JSBin, hopefully I got all of the necessary components in there.
http://jsbin.com/kakejoluzo/1/


Fix typos in the blueprint

Fixes a few typos in the blueprint. I have also noticed that there is a broken link at the beginning of section 5.6, but I don't know where exactly that one is supposed to go so I haven't changed it.

Thanks!


Hello developer, Support for dreame_vacuum r9441

Dear developer, I have a Dreame vacuum cleaner robot. After checking, I found that my model is dreame_vacuum_r9441, which is not in the compatible list.
This product is an upgraded version of S30 Pro Ultra, released on August 6, 2024. Compared with S30 Pro Ultra, the difference is: the suction power is upgraded to 17000Pa. Newly designed base station self-cleaning. 75° hot water dynamic washing and mopping. Side brush, mop, and roller brush can be lifted in three directions.
Screenshot of the device model displayed on the router backend

Image from Dreame APP

Thank you for your hard work and dedication in developing this open source project.

Hello developer

Dear developers, will you adapt dreame_vacuum r9441?

I hope developers can help adapt r9441, thank you. I will use the payment support for this project

+1，thanks

+1


before and after result share link

A url that contains all of the options setting, that can be shared, to show what exactly one person was looking at.
(@jakearchibald please edit if this does not make any sense)

Closing this issue. Sharing image data as dataURI is bad idea (also probably size limiting).
We can perhaps create something like Glitch app based on Squoosh that takes pre-set setting.


Decompose transform matrix in native (for android).

This diff translates implementation of transform matrix decomposition from JS to java. This is to support offloading animations of transform property, in which case it is required that we can calculate decomposed transform in the UI thread.
Since the matrix decomposition code is not being used for other platform I went ahead and deleted parts that are no longer being used.
Test plan
Run UIExplorer Transform example before and after - compare the results

By analyzing the blame information on this pull request, we identified @cpojer and @ericvicenti to be potential reviewers.

cc @astreet

Thanks, looks good! Thanks for preserving the test!
@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.

Import failed, we're actually using the JS matrix transforms in internal code... can you un-delete it and resubmit?

I tried to merge this pull request into the Facebook internal repo but some checks failed. To unblock yourself please check the following: Does this pull request pass all open source tests on GitHub? If not please fix those. Does the code still apply cleanly on top of GitHub master? If not can please rebase. In all other cases this means some internal test failed, for example a part of a fb app won't work with this pull request. I've added the Import Failed label to this pull request so it is easy for someone at fb to find the pull request and check what failed. If you don't see anyone comment in a few days feel free to comment mentioning one of the core contributors to the project so they get a notification.

@astreet I reverted changes in MatrixMath and also restored JS test for MatrixMath, but long term I think it would be better if you extract MatrixMath somewhere else in order to use it in the internal code cause it's going to confuse people :)

@facebook-github-bot shipit

I tried to merge this pull request into the Facebook internal repo but some checks failed. To unblock yourself please check the following: Does this pull request pass all open source tests on GitHub? If not please fix those. Does the code still apply cleanly on top of GitHub master? If not can please rebase. In all other cases this means some internal test failed, for example a part of a fb app won't work with this pull request. I've added the Import Failed label to this pull request so it is easy for someone at fb to find the pull request and check what failed. If you don't see anyone comment in a few days feel free to comment mentioning one of the core contributors to the project so they get a notification.

This is weird, the diff isn't updating internally so it keeps failing with the same error. I'm going to try to close that diff out and re-land this.

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


Add Language Data's Comment Tokens to Support Toggle Commenting Shortcuts in the Editor

Previously, hitting shortcuts like Cmd+/ would not toggle comments in the editor. This is normally supported out-of-the-box with codemirror with the default keymap, but it wasn't working because it wasn't able to get the language data config. This PR adds the comment tokens to the language data so it can find it when it tries to toggle the comments.
Demo vid:
https://github.com/KittyCAD/modeling-app/assets/9722062/439ff676-968f-4bf6-92bb-60dd5c657cde

this is dope

Dope! was meaning to look into this.
It was working with the default setup, but went missing when we added our lsp.


add #remove method to the Feature class

I thought it might make sense for it to be available on the feature level:
flipper[:feature].remove

This brings of up the question of what does remove mean. Adapter#remove is for removing the feature from the set of known features, but I would assume Feature#remove would also clear the gate values and such. Also, we currently don't have Feature#add, which would be the reciprocal method of #remove. I think for these reason I'm inclined to just let the api use the adapter remove method and not add remove to feature. What are your thoughts on this?

Hmm that's a good point. I've been mostly using this api:
flipper.class                  # => Flipper::DSL

flipper[:feature]              # calls Feature.new(name, @adapter, ...)
flipper[:feature].enable       # calls adapter.enable(self)
flipper[:feature].disable      # calls adapter.disable(self)
flipper[:feature].enable(user)

So it seems that almost every adapter method is exposed through DSL, except for #remove.
And I just noticed that it wasn't obvious to me how to remove redis data when you stop using the feature.
I don't have a strong preference either way though.

I lied. Adapter#remove is for removing feature from set of known features and clearing gate values. Totally forgot. My bad. I think the only thing this is missing is adding the remove method to the DSL with an accompanying spec and changing the web action to use feature#remove instead of adapter#remove. If you have time to knock those out, go for it. If not, I'll try to get it done in the coming weeks.

Thanks John! I've just added the DSL#remove method. Please let me know if I missed anything, I'm happy to make any other changes.


Inconsistent state possible after middle-click on refresh icon on Firefox 56

Version
2.5.2
Reproduction link
http://neofoodclub.fr/repro.html
Steps to reproduce

Be on Firefox 56 or more
Check the checkbox.
Middle-click on the refresh icon in the toolbar.

What is expected?
A consistent state, either unchecked and false, or checked and true.
What is actually happening?
The state is checked and false.

On Chrome, the resulting state is unchecked and false.
On Firefox 55 or less, the resulting state was checked and true.
Happens on https://vuejs.org/v2/guide/forms.html#Radio with all inputs except the textarea and text ones.
The number inputs result in a consistent state too.
I couldn't make a jsfiddle for it, because Firefox doesn't attempt to restore input state after a midlle-click refresh on jsfiddle.

Tried on Safari 11.. it's checked and true

On Firefox 57.0b7, the event "input" (but not the event "change") is triggered on the checkbox in the newly opened tab after a middle-click refresh.

I tested:

vue doesn't react to "input" event on the checkbox.
The model is updated (text next to the checkbox changes) when I fire a "change" event on the checkbox.

So basically:

vue.js updates select, checkbox input and radio input elements after a "change" event, but not after a "input" event.
Firefox follows MDN and only fires the event "change" when the change is committed by the user. On middle-click refresh, the change is automatically done by Firefox and therefore not by user. Consequently, only the "input" event is fired.

I'm wondering what the reasoning is behind the first point. What would happen if we just changed these 3 'change' by 'input' ? This would probably need a lot of browser compatibility checking.

After trying out possible solutions, I'm closing this as a wontfix. IMO this is something that should be fixed in Firefox:

The behavior of normal refresh and middleclick-refresh are inconsistent
It fires input and input only in this one specific case when every other browser fires change only.

It is technically possible for Vue to address this behavior, however:

It introduces non-trivial complexity into the codebase
If listening to both input + change in FF, it causes all listeners to be fired twice, which is non-efficient and may cause unnecessary re-renders if microtasks fire between input and change.
If only use input event for all those input types in FF, then it breaks user code that expects to trigger v-model updates by manually dispatching change events.

So, the tradeoff isn't really worth it for such a niche case.


comments in commits neither loading nor being counted correctly

I am using the last version of the application from the play store. I have noticed that it's always showing zero comments in any commit in a private repository (I've not tried with public ones). if I try to open the commit, comments are simply not loaded. if I post a comment, it briefly stays on bottom then it just disappears. I am using android 4.2.2 on the nexus 4.

Fixed


fix(kms): have EncryptionKeyBase implement IEncryptionKey

Fixes #1530
Pull Request Checklist

[ ] Testing

Unit test added
CLI change?: coordinate update of integration tests with team
cdk-init template change?: coordinated update of integration tests with team

[ ] Docs

jsdocs: All public APIs documented
README: README and/or documentation topic updated

[x] Title and Description

Change type: title prefixed with fix, feat will appear in changelog
Title: use lower-case and doesn't end with a period
Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"

[ ] Sensitive Modules (requires 2 PR approvers)

IAM Policy Document (in @aws-cdk/aws-iam)
EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
Grant APIs (only if not based on official documentation with a reference)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

awslint would have caught this if the name of the construct would have been Key (which it should be).


[Monaco] Improve .gitignore definition

Description of the new feature / enhancement
Current version:
This is the current .gitignore Monarch definition:
https://github.com/microsoft/PowerToys/blob/4badf2d24a64390a3f80d78b2963e1aaaa5ff20b/src/common/FilePreviewCommon/Assets/Monaco/customLanguages/gitignore.js#L1-L13
It has plenty of space for improvement:

[/^\s*!.*/, 'invalid'] marks all lines starting with a ! as invalid. Negation is supported by gitignore, so why is it considered invalid?
[/^\s*[^#]+/, "tag"] why can't there be a # in a pattern? Just tested it, # is absolutely valid in a gitignore pattern.
This tokenizer marks many invalid patterns that are easy to catch as valid.

My improved version:
return {
  defaultToken: 'invalid',

  tokenizer: {
    root: [
      [/^#.*$/, 'comment'],
      [/.*((?<!(^|\/))\*\*|\*\*(?!(\/|$)))/, 'invalid'],
      [/(^\s*(?:\\\s|\S)+)((?:\s+(?:\\\s|\S)+)*)/, ['tag', 'invalid']],
    ],
  },
};

Tested in Monaco playground
Scenario when this would be used?
Previewing .gitignore
Supporting information
No response

neat, if you want to submit, 100%

[/^\s*!.*/, 'invalid'] marks all lines starting with a ! as invalid. Negation is supported by gitignore, so why is it considered invalid?

It makes it red, that's why.

But thank you for your improved version

Hello @Aaron-Junker, I understand your thinking. It makes sense to me, I just don't think that's how you're supposed to use invalid. Differentiating negations is a neat feature, but anything else that's actually invalid also has the same color. I've been thinking for a while, but I didn't come up with anything. Let my know if you have any ideas, or if I should keep it the way it was before. Thank you

Can we define our own color and distinguish between invalid rule (red) and negative rule (purple) or something else?

Hello @Aaron-Junker, I understand your thinking. It makes sense to me, I just don't think that's how you're supposed to use invalid. Differentiating negations is a neat feature, but anything else that's actually invalid also has the same color. I've been thinking for a while, but I didn't come up with anything. Let my know if you have any ideas, or if I should keep it the way it was before. Thank you

Can we define our own color and distinguish between invalid rule (red) and negative rule (purple) or something else?

Hello @htcfreek, yes we can, I already have it working. I don't know what color to choose though. It is possible to have a different one for light and dark theme, but it would be much easier to have a single value for both. Do you have any recommendations? There are many shades of purple, you know :)

I know. 😅 Don't thought about.

This was released wit v0.82.0.


v1.54.0 Cannot find module 'cronstrue'.

Hi @bradymholt
Just connected your package to my Angular project!
npm install cronstrue
import { Component } from '@angular/core';
import cronstrue from 'cronstrue';

@Component({
    selector: 'my-app',
    template: `<router-outlet></router-outlet>`,
})
export class AppComponent implements OnInit {
    constructor() { }

  ngOnInit() {
     console.log('cronstrue', cronstrue.toString("* * * * *"));
  }
}

Error appears when building a project:
ERROR in src/app/app.component.ts(5,26): error TS2307: Cannot find module 'cronstrue'.

@vadost Can you see if you have the same problem is version 1.51?  npm install<EMAIL_ADDRESS>

@bradymholt Yes, the same

@bradymholt I'm sorry to trouble you! The problem was that the node_modules folder was cached in my docker container! Everything is working!


Running Examples - Plot not showing

Hi,
When running examples only the first iteration and their respective point placements appear on the plot. The following iterations do not appear and I am unsure why.
Thank you in advance!

Can you provide some more information? What examples? Is it all of them? What environment? Are you running them from the command line? Any other information that you have would be helpful.

Is your issue similar to https://github.com/siavashk/pycpd/issues/30?

Yes, my issue was the same #30 thank you so much!

I had the same plotting issue: the first iteration was plotted, but others were blank.
Changing the code as follows worked for me:
pic_num =1

def visualize(iteration, error, X, Y, ax):

    global pic_num
    fig = plt.figure(pic_num)
    ax = fig.add_subplot(111, projection='3d')

    ax.scatter(X[:, 0],  X[:, 1], X[:, 2], color='red', label='Target')
    ax.scatter(Y[:, 0],  Y[:, 1], Y[:, 2], color='blue', label='Source')
    ax.text2D(0.87, 0.92, 'Iteration: {:d}'.format(
        iteration), horizontalalignment='center', verticalalignment='center', transform=ax.transAxes, fontsize='x-large')
    ax.legend(loc='upper left', fontsize='x-large')

    pic_num +=1

(To make it 2D:  ax = fig.add_subplot(111) )

Hi, I have the same problem and I'm using Spyder with python 3.6. Only the result for the first iteration is showed as shown in the attachment.
Thank you in advance.

@sandwich25 this seems to be a compatibility issue between matplotlib and spyder. I do not use spyder as an integrated development environment (IDE), so I cannot give you a definitive answer. The solution is probably similar to https://github.com/siavashk/pycpd/issues/30#issuecomment-531438477.
I would suggest looking into the plotting options in your IDE. The answers in this Stackoverflow question might help you.


Confused about the main menu feature

...What does it do exactly? I thought it would arrange my applications — all applications I have installed — in a circle, categorized like the menus I would use for program access traditionally.
I am using the MATE desktop. Is there some optional deps for GNOME Shell I need to include so GNOME Pie can do its thing correctly rather than showing me absolutely nothing?

Found the problem:
[WARNING] Failed to look up menu_file for "applications.menu"
But now what's the solution?

Figured out a good temp fix, at the very least. It's not the best way of doing things but this helped:
sudo ln -s /etc/xdg/menus/gnome-applications.menu /etc/xdg/menus/applications.menu
Not sure why Manjaro does that. But now I wonder, what's the proper way to fix it? Because using a symbolic link feels like cheating...

I found some information here: https://developer.gnome.org/menu-spec/. It seems that there could be various menu definitions, one for each desktop environment. Gnome-Pie could check whether applications.menu exists and if not search for an alternative.
But in the meantime, your workaround seems reasonable...

Hey Simon, just wanted to chime in and say that gnome-menus from extra is required to be installed for use on Arch. That's what renders the option of gnome-applications.menu to be available in /etc/xdg/menus.


Dbeaver not getting records change on Table

Good afternoom,
There is a strange behavior with my connections.
This happens following this steps.

Connecting on database
Use SELECT command to query some data.
Use an application to create or to update a record.
Execute select from step 2 again. (You will observe that the result of SQL query remais the same)
Disconnect on database
Reconnect on database
Execute select from step 2 again (You will observe that the result of SQL now has the right information)

This behavior starts with the last version I have updated.
I am using Oracle database connecting with service name.
I'll be greatfull for any help,
Thanks

Hi there,
It seems from uncommitted data, are you using transaction control under your connection on DBeaver and/or on your application?
Changes only will be visible after they're committed.

There is no Repeatable read option in my DBeaver.
I very sure that all data was already committed and DBeaver just show the correct data after reconnect.

Same for me! Most of the times I'm not getting the actual data from the database except I'm doing a manual reconnect. This problem is there since I'm Using Dbeaver (~ 6 Months).
To be sure that the correct data are displayed, I have to reconnect manually.
System: Linux (Ubuntu 16.04)

Recently, I found something that helps. We do not need reconnect to see correct data, you just have to close all tabs e open again.
It is not the best way, at least we do not have to reconnect al the time.

Hmm, thanks for your help. But nevertheless it's a big issue because we can't be sure if the displayed data is up to date or not.

In fact DBeaver doesn't have any kind of data cache so I still think it is related to transactions.
Does this problem exists in auto-commit mode?

Serge,
I could not verify if that happens with auto-commit because I am not use this feature but I am sure that is not related to transactions.
You see, I have a Java App that changes information in database, after the java application ends I just see the new information on DBeaver after reconnect or after close and reopen an SQL editor.
Yesterday, I did something and I think that I won't have this problem anymore. I exported all my connections, close deaver, deleted the folder $HOME/.dbeaver4, reopen dbeaver and imported the connections that I had exported. After that, the problem stops to occur. I hope that stays this way.

You see, I have a Java App that changes information in database, after the java application ends I just see the new information on DBeaver after reconnect or after close and reopen an SQL editor.

This makes me think that this is still related to transactions. Because reconnect/DBeaver restart ends transaction in DBeaver and it can see changes made by other applications (this actually depends on database and transaction isolation level).
You could try to turn autocommit on or just perform manual commit or rollback operation (in the main toolbar) to see changes made by other applications.

It looks like this question was solved. It will be closed. If you think that it is actual just let us know.

Wow,
I've recereated the .dbeaver4 directory, and copied over only the connections & project.
The GUI is MUCH snappier and responsive. (also many errors about missing icons are gone)
However the commit is again lost/missing after restart.
Steps:

set to manual commit

execute INSERT INTO (...) query

during query execution this popup appears:

statistic page shows query completed, updated 7 rows

observe the Commit button is greyed out (maybe the DB forces the commit?)

also observe the query is green and Result = Success in the Query Manager

switch to another tab and run a SELECT to verify the inserted data is there

restart DBeaver -> data is not there. -.-

It is always reproduceable.
I have created a new connection, and there it is not reproducible. maybe it's a good idea to start with a fresh profile after the major Eclipse version update...

compared the entries in .dbeaver-data-sources.xml,
and the only difference seems to be:
<property name="autocommit" value="true"/>
that might be causing some unexpected state

if the connection type is changed to "Development", then the commit is not lost with the existing DB connection
but with the new connection it doesn't make a difference, it can be set to "Production" and the commit is there (after restarting DBeaver)

@orbanbalage I think this deserves a separate ticket.
BTW is this Invalidate notification happens always? Usually it i shown when connection was broken by some reason (and transaction will be obviously rolled back - this might be the reason why data is not committed)


underscore was missing from the method name

Nice catch - merging in. Will publish updated version to NPM right now.


Audit logs (password changes, consent changes, logins)

We should have a view where users can see what happened to their account. This includes:

login/logout events
password changes
new consent granted

This is the kind of screens you can find on myaccount.google.com for this:


Implement IDisposable for ReactiveDictionary

Sorry for delay response.
Thanks, good improvement.
I think it needs some fixes, same behaviour as ReactiveProperty.
But okay, I merge it and I'll fix some bahviours.

Ah sorry.
It's same as ReactiveProperty, completely ok.

returns empty observable on subsribe event when ReactiveDictionary was disposed.


consider having LVM calculate metadata size

See https://github.com/projectatomic/docker-storage-setup/pull/171#discussion_r87871678

I think my bottom line on this is - we're on track to switch to overlay2.  I'm not sure how much I want to poke at tweaking devicemapper.

apart from switching to overlay2, current logic is working and nobody has complained. So why to change that unnecessarily. If we face a real issue or data which shows that lvm takes much smarter decisions, then I will be inclined to make the change. Need a good reason to make the change.

ok, let's close this then as we have decided not to make the change


[WIP] CLI bats integration tests

Run: bats cli.bats
TODO:

[ ] write more tests
[ ] include in CI builds

Bats test performing differently for CI Linux than my local OSX machine:
Local:
~/dev/etc/emerald-cli tests/cli-bats ⟠ bats cli.bats
 ✓ [meta] succeeds: set env var and tmp dir EMERALD_BASE_PATH
 ✓ succeeds: --version
 ✓ succeeds: --help
 ✓ succeeds: new --chain=testnet [empty options]
 ✓ succeeds: new --chain=mainnet --security=high --name='Test account' --description='Some description'
 ✓ succeeds: list
 ✓ succeeds: update
 ✓ succeeds: strip
 ✓ succeeds: hide && unhide

9 tests, 0 failures

CI: https://travis-ci.org/ethereumproject/emerald-cli/jobs/273274694
14.33s$ bats cli.bats
1..9
ok 1 [meta] succeeds: set env var and tmp dir EMERALD_BASE_PATH
ok 2 succeeds: --version
ok 3 succeeds: --help
ok 4 succeeds: new --chain=testnet [empty options]
ok 5 succeeds: new --chain=mainnet --security=high --name='Test account' --description='Some description'
ok 6 succeeds: list
not ok 7 succeeds: update
# (in test file cli.bats, line 102)
#   `[ "$status" -eq 0 ]' failed
ok 8 succeeds: strip
not ok 9 succeeds: hide && unhide
# (in test file cli.bats, line 155)
#   `[ "$status" -eq 0 ]' failed
The command "bats cli.bats" exited with 1.

Will test on my local linux,

update, hide, and unhide don't require password input, right?

yeap, no password required

Merged in new PR #21


How to handle Apple REJECT when using SVProgressHUD

We still found your app uses or references the following non-public APIs:
NSBlock, UIInputSetContainerView, UIInputSetHostView, UIKeyboard, UIPeripheralHostView, UIStatusBarDataNetworkItemView, __NSCFURLSessionConfiguration

Hm. 1) We do not use NSBlock, UIStatusBarDataNetworkItemView, and __NSCFURLSessionConfiguration. 2) UIInputSetContainerView, UIInputSetHostView, UIKeyboard, and UIPeripheralHostView are not used by class instantiation, they are only referenced by name NSClassFromString. We do not modify the view hierachy and all my submission to the App Store were fine.

However, maybe we could change to code to listen to UIKeyboardDidShowNotification.

I got my app rejected:

UIInputSetContainerView, UIInputSetHostView, UIKeyboard, UIPeripheralHostView, UITabBarButton```

We do not use UITabBarButton.

Okay, but the UIInputSetContainerView, UIInputSetHostView, UIKeyboard, UIPeripheralHostView is "used". I don't Apple is rejecting this code. What would be the solution to it?

I also received this: Your app uses or references the following non-public APIs:
UIInputSetContainerView, UIPeripheralHostView, UIInputSetHostView, UIKeyboard. I  would remove these code for submitting.

@yanpanpan I did a "solution" and I'm waiting for Apple review. As soon as it works I'm gonna open a PR.

Is there any other way to determine the keyboard height during runtime? See the following sequence.

Keyboard is visible <=> All Notifications already have been fired
HUD should be shown for the first time <=> HUD gets allocated
Now we need the keyboard height without notifications

This sequence would is not possible without the current implementation. I see that there "might" be a problem and as a "quickfix" I could remove the code fragment. Nonetheless, there need to be a solution for the sequence described above.

My app was rejected again with the same problem even changing all comparations to [NSStringFromClass([possibleKeyboard class]) isEqualToString:.

Ok, the only reasonable solution is thus to remove the corresponding code. However, the issue of determining the keyboard height still remains. There must be a way to get the keyboard frame without actively listening for notifications.

@honkmaster if I remove the code on my side, will it be okay or it can cause problems?

You can just return 0, will give no problems. However, the Hud might be displaced if the HUD is shown if an keyboard is already present.

I think I will try the solution presented here.

I check IQKeyboardManager. THey use the same technique. However, they obfuscate  their use of "private" API by checking prefix and suffix of the class.name and not the whole string ...
  if (![subViewName hasPrefix:@"UI"]) continue;
  if (![subViewName hasSuffix:@"InputSetHostView"]) continue;

@honkmaster that's a bypass. it could work, but I don't know if it's a complete solution. I think it's worth a try.

Is it safe to use SVProgressHUD since there has been issues related to rejection of app by Apple?

I implemented a bypass and someone has to give app review feedback if it works. By the way: I did not have a rejection for an app submitted last week.

I see, in my case, I can't risk another rejection. So I just removed the code inside the method. But if anyone can test it, it would be good to know.

I used the lasted version,but i got my app  rejected.   @honkmaster
Your app uses or references the following non-public APIs:
UIPeripheralHostView
UIKeyboard
UIInputSetContainerView
UIInputSetHostView

Did you use the latest code?

Yes,so i set the "visibleKeyboardHeight" to 0 without other code.Then the app already passed.

Just as an FYI: I got two apps through app review with the latest modifications with no problem.


IA-2153: Make the difference between allowed-to-create children types and children-type to show in Iaso

Have a way on mobile to differentiate the org unit type we would like to allow people to suggest adding, and those that are fixed and are not in the scope of suggestions for additions (ie regions, districts)
Related JIRA tickets : IA-2153
Self proofreading checklist

[x] Did I use eslint and black formatters
[x] Is my code clear enough and well documented
[x] Are my typescript files well typed
[x] New translations have been added or updated if new strings have been introduced in the frontend
[x] My migrations file are included
[ ] Are there enough tests
[ ] Documentation has been included (for new feature)

Changes

model and migration
duplicate code for sub unit type creation
adding custom tooltip

How to test
Create or edit an org unit type.
You should be able to add sub unit types displayed and sub unit types allowed for the creation in the mobile app
I have added same check on circular hierarchy as the one sub unit types
Print screen / video
https://github.com/BLSQ/iaso/assets/12494624/2aa2dcec-fe79-4bcb-bb1d-28077d5acf59

I have fixed the black issue, we have changed the black version in main and the format is not 100% the same.


Can we make the AOP Proxy Classes not catch all Throwable?

It's kind of a hassle to constantly catch and unwrap the Invocation Exception to rethrow the Runtime exception I'm expecting.

Can you provide an example / context for this question. We need to handled checked exceptions etc so perhaps include an example of what is generated for an given method signature and then maybe it is more obvious why it's an issue?
Cheers, Rob.

Say I got an aspect called ScopedValueMDC if I place it on an HTTP client method, they will no longer throw the expected HttpException on 5xx errors since that is wrapped inside of an invocation exception.
@Singleton
@Client
public interface ApiClient {

  @Get
  @ScopedValueMDC("funny picture")
  HttpResponse<InputStream> call(@Header("Accept") String accept);
}

Now I need more logic to extract the HttpException.
    try {
      final Sus github;
      final var contributors = github.contributors("rbygrave", "junk");

    } catch (final InvocationException iex) {
      if (iex.getCause() instanceof final HttpException ex) {
        if (ex.getCause() != null) {
          // some IO/Interrupted exception happened while sending request.
        } else {
          // Use HttpException utility methods to access response status and handle/deserialize
          // error body
        }
      }
    }

Perhaps we could do an instance of check in the generated proxy method and simply rethrow the checked exception?
@Singleton
@Client
public interface ApiClient {

  @Get
  @ScopedValueMDC("funny picture")
  HttpResponse<InputStream> call() throws CheckedException;
}

we would have the proxy method throw the same checked exception
  @Override
  public java.net.http.HttpResponse<java.io.InputStream> call() throws CheckedException {
    var call = new Invocation.Run<>(() -> super.call(accept))
      .with(this, call0, accept);
    try {
      call0Timed.invoke(call);
      return call.finalResult();
    } catch (InvocationException e) {
      throw e;
    } catch (Throwable e) {

      if(e instanceof RuntimeException ex) throw ex;
      else if (e instanceof CheckedException ex) throw ex;
      else throw new InvocationException(e);
    }
  }

Ah, I just noticed that this behavior is there when you explicitly add throws to the method signature. It generates like this:
  @Override
  public java.net.http.HttpResponse<java.io.InputStream> call() throws CheckedException {
    var call = new Invocation.Run<>(() -> super.call())
      .with(this, call0, accept);
    try {
      call0Timed.invoke(call);
      return call.finalResult();
    } catch (InvocationException | CheckedException e) {
      throw e;
    } catch (Throwable e) {
       throw new InvocationException(e);
    }
  }

when I modify a method signature with throws

Can we just add a catch clause for rethrowing runtime exceptions?

So currently the generated code is:
    ...
    } catch (InvocationException | CheckedException e) {
      throw e;
   ...

And we instead like it to include RuntimeException like:
    ...
    } catch (RuntimeException | InvocationException | CheckedException e) {
      throw e;
   ...

And the reason for this is .... when the client api method throws a RuntimeException it is currently wrapped as an InvocationException  and this is a pain because ...[some reason].
Noting that when the client api does not throw a CheckedException we want to generate similar code but it just does not include the CheckedException.
Is that correct?

Basically yeah

oh yeah, looks like there is a bug with this. when the method sig throws Exception it fails to compile because.
    ...
    } catch (InvocationException | Exception e) {
      throw e;
   ...

is invalid.

To kind of state the obvious, InvocationException is a RuntimeException ... so we could almost swap out InvocationException with RuntimeException here in this catch block except for detected bugs where the method throws Exception (or Throwable) in which case we need to suppress catching the RuntimeException.


Issue in css at 767px

the style is not properly been applied because at 767 there is error window

Provide the site's link and also describe your issue in detail please @Yash121l

Actually i was just searching a sidebar for my project and then i find this when i see there live preview by cloning it i find out that there is sum issue with css the the code writer for making it responsive is not writing properly and at exactly at 767 Px it get breaks and the user see navbar like (which i show in the main fork).

isn't it working fine? @Yash121l

I dont know but it is looking like this in mine

change the min-width: 767px to min-width: 768px in styles.css file in the following section of code
@media screen and (min-width: 767px) {
.container {
margin-left: 7rem;
margin-right: 1.5rem;
}
.nav {
position: fixed;
left: 0;
background-color: var(--container-color);
box-shadow: 1px 0 4px hsla(228, 81%, 49%, .15);
width: 84px;
height: 100vh;
padding: 2rem;
transition: .3s;
}
.nav__logo {
display: flex;
}
.nav__logo i {
font-size: 1.25rem;
color: var(--first-color);
}
.nav__logo-name {
color: var(--title-color);
font-weight: 600;
}
.nav__logo, .nav__link {
align-items: center;
column-gap: 1rem;
}
.nav__list {
display: grid;
row-gap: 2.5rem;
margin-top: 10.5rem;
}
.nav__content {
overflow: hidden;
height: 100%;
}
.nav__toggle {
position: absolute;
width: 20px;
height: 20px;
background-color: var(--title-color);
color: #fff;
border-radius: 50%;
font-size: 1.20rem;
display: grid;
place-items: center;
top: 2rem;
right: -10px;
cursor: pointer;
transition: .4s;
}
}


thing where the bot crashes when modal run

yeah so try to run the command and you'll see!!

what do i do??

You can maybe try to not be bad at the programming language, by that I mean restart the bot!

Thank you, that worked!!

You can maybe try to not be bad at the programming language, by that I mean restart the bot!


enable val_indices feature in user Dataset

The val_indices argument here https://github.com/apple/pfl-research/blob/develop/pfl/data/dataset.py#L164 is currently not implemented. fix it.
The correct behavior should be if val_indices are provided, then split method should split the user dataset into a train and val partition dataset objects according to those indices and not according to the input arguments of split.
This is useful if you have static partitions within each individual user.
(115345691)

Hi @grananqvist I can take up this and implement if no one is working on it currently

sure!


Update example.py

Disconnect the number of parallel workers (numW) from the number of simulations (num) to perform. Up to now, num controls the number of parallel workers as well as the number of simulations.

obsolete

obsolete, see #2


Some updates to help ease installation

Because of a couple of the pre-reqs now you should update SQL to SP4
This tool was SUPER useful to help install pre-reqs https://gallery.technet.microsoft.com/ConfigMgr-2012-R2-e52919cd

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

ID: 4e9b7de4-5860-4eda-a0e5-d34f69a63fd5
Version Independent ID: b3140976-dc93-deab-1f90-d5479c8403af
Content: Set up your lab - Configuration Manager
Content Source: sccm/core/get-started/set-up-your-lab.md
Product: configuration-manager
Technology: configmgr-other
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Thanks @kabluton for the feedback. We should probably revise this article with more recent versions all around.

I'm closing this since it's in the retired repo and there's another issue filed on the article in the new repo (see the link above). Still hope to revise the article, just consolidating the feedback. If you want to receive updates on the issue, follow the link to MicrosoftDocs/MEMDocs#159 and select the button in the right side menu to subscribe to notifications.


Move from ratelimit.proto to rds.proto in data-plane-api

https://github.com/envoyproxy/data-plane-api/blob/master/api/rls.proto

@junr03 description seems wacky, do you mean that we want to migrate envoy's ratelimit service proto dependency from
https://github.com/envoyproxy/envoy/blob/master/source/common/ratelimit/ratelimit.proto
to
https://github.com/envoyproxy/data-plane-api/blob/master/envoy/service/ratelimit/v2/rls.proto
?
I can work on this once the intention is clarified.


Do not error in optimizer if resulting schema has different metadata

Which issue does this PR close?
Closes https://github.com/apache/arrow-datafusion/issues/4346
Note this PR is almost all tests
Rationale for this change
Optimizer passes do not necessarily keep nullability and metadata the same between passes
See https://github.com/apache/arrow-datafusion/issues/4346 for more detail
What changes are included in this PR?

Make the check added in https://github.com/apache/arrow-datafusion/pull/4233  less stringent by only checking field name and data type.

Are these changes tested?
Yes
Are there any user-facing changes?
No

@jackwener  and @mingmwang  can you please take a look?

I would suggest to add the schema equal method to arrow-rs add the new method for Field,  DataType and Schema.
Maybe we can call the new method semantic_equals().

I would suggest to add the schema equal method to arrow-rs add the new method for Field, DataType and Schema.
Maybe we can call the new method semantic_equals() or semantic_eq() .

This is a good idea -- I will do so

As this error is blocking my upgrade of DataFusion into IOx my plan is to file tickets for the follow on work and merge this one in (I will also try and prepare a PR to support the nested types as well)

Follow up in arrow-rs: https://github.com/apache/arrow-rs/issues/3199 (will add this link in comments in a follow on PR)


Support multiple eBPF programs in the same section

Describe the feature you'd like supported
Quentin Monnet pointed out two years ago that libbpf v1.0 now allows multiple programs in the same section (Andrii's blog again)).
Also, the prog_array.c sample in the ebpf-samples repository does this.  (Other existing samples might too.)
Proposed solution
Using a section name is not sufficient.  Instead use the C function name.
Support is blocked on https://github.com/vbpf/ebpf-verifier/pull/638 being merged first.
Additional context
bpf2c support is covered by issue #1460
This issue covers ebpfapi.dll and ebpfsvc.exe support.

Duplicate of #1440


Add (unchecked (int)) to enum values greater than 32 bits)

Some manifest files have values greater than 32 bits, i.e. 0x80000000.  Add an (unchecked) type when creating an enum in these cases.

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

Codecov Report

Merging #586 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #586   +/-   ##
=======================================
  Coverage   17.49%   17.49%
=======================================
  Files         216      216
  Lines      124823   124823
  Branches    12046    12046
=======================================
  Hits        21841    21841
  Misses     102118   102118
  Partials      864      864

Flag
Coverage Δ

#2017
17.49% <ø> (ø)
:arrow_up:

#Debug
17.49% <ø> (ø)
:arrow_up:

#Release
100% <ø> (?)

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 0be8cbd...68f8eba. Read the comment docs.


Use IndexedDB to store assets hashes

To allow changed update strategy to be used with any files names (even without changing them on file update) ServiceWorker should know current hash (easy, just generated map) and old hash (hard, need to store somehow on the client) of the file.
Storing custom headers in request does not work (Headers are immutable). IndexedDB should work.

Done in v3.


[BARX-454] Ensure the Agent is not installed when DD_INSTALL_ONLY is set

For deb/ubuntu:
apt-get install will try to start the services after installing them. One way to stop it is to set RUNLEVEL=1 but it won't work with recent distributions. The recommended way is to use a policy-rc.d script which is done in this PR. However it will not work for ubuntu chroots (known limitation).
For RPM packages:
These post install scripts try to install the Agent. We set SYSTEMD_OFFLINE=1 when running them.
Another solution would be to just stop the service after installation.

A quick note before reviewing, I suppose you mean not launched in the PR title :grin:


OpAtomicSMax behaves as unsigned

It seems that MoltenVK executes OpAtomicSMax as its unsigned counterpart, and similarly for OpAtomicSMin. I discovered this while triaging vkd3d test failures (specifically, test_atomic_instructions()), but I also created a "minimal" (in the Vulkan sense) example which is attached (it doesn't properly do the portability subset extension dance, but hopefully that shouldn't be relevant). I current don't know any "real life" application impacted by the problem.
On MoltenVK (Apple Silicon M2 MacBook) the test program prints:
Final value: ffffffff

while I would expect (and I can see on other implementations) zero.
Hopefully the Vulkan boilerplate is obvious enough and you only need to care about the SPIR-V source and main function. The attachment has extension .txt because for some reason .cpp is not allowed.
I noticed that the generated MSL code indeed changes if I change the SPIR-V operator, but the result ends up being the same; with OpAtomicSMax I can see:
uint _17 = uint(u0.atomic_fetch_max(uint(0), int(4294967295u)).x);

and with OpAtomicUMax:
uint _17 = imageAtomicMax(u0, 0,<PHONE_NUMBER>u);

I don't know enough about Metal to say whether the first result is correct.

This should be filed on SPIRV-Cross.

I can confirm this is fixed in 1.2.9.


Fix to prevent node from moving out of workspace

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes

A node can be moved out of the right or bottom edge of workspace using Move Selection to Right/Down action.
This PR try to fix the issue.

Checklist

[x]  I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 68.326% when pulling 02308f9e2fce251ab8a40adc8a68566cc035e349 on node-red-hitachi:fix-node-move into f760354e821c464295285651b38d0acdcdc1443d on node-red:master.


Error during fine-tuning EVA object detector

Hello,
I am fine-tuning EVA on my custom dataset. I ran into the following error (it also happens when fine-tuning on COCO):
File "train.py", line 187, in <module>
  main(args)
File "train.py", line 169, in main
  trainer.train(0, cfg.train.max_iter)
File "/home/appuser/eva_repo/det/detectron2/engine/train_loop.py", line 149, in train
  self.run_step()
File "/home/appuser/eva_repo/det/detectron2/engine/train_loop.py", line 421, in run_step
  self.grad_scaler.scale(losses).backward()
File "/home/appuser/.local/lib/python3.7/site-packages/torch/_tensor.py", line 255, in backward
  torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
File "/home/appuser/.local/lib/python3.7/site-packages/torch/autograd/__init__.py", line 149, in backward
  allow_unreachable=True, accumulate_grad=True)  # allow_unreachable flag
File "/home/appuser/.local/lib/python3.7/site-packages/torch/autograd/function.py", line 87, in apply
  return self._forward_cls.backward(self, *args)  # type: ignore[attr-defined]
File "/home/appuser/.local/lib/python3.7/site-packages/fairscale/nn/checkpoint/checkpoint_activations.py", line 331, in backward
  outputs = ctx.run_function(*unpacked_args, **unpacked_kwargs)
File "/home/appuser/eva_repo/det/detectron2/modeling/backbone/vit.py", line 289, in forward
  x = self.attn(x)
File "/home/appuser/.local/lib/python3.7/site-packages/torch/nn/modules/module.py", line 1051, in _call_impl
  return forward_call(*input, **kwargs)
File "/home/appuser/eva_repo/det/detectron2/modeling/backbone/vit.py", line 139, in forward
  attn = add_decomposed_rel_pos(attn, q, self.rel_pos_h, self.rel_pos_w, (H, W), (H, W))
File "/home/appuser/eva_repo/det/detectron2/modeling/backbone/utils.py", line 133, in add_decomposed_rel_pos
  Rh = get_rel_pos(q_h, k_h, rel_pos_h)
File "/home/appuser/eva_repo/det/detectron2/modeling/backbone/utils.py", line 100, in get_rel_pos
  z = rel_pos[:, i].view(src_size).cpu().float().numpy()
RuntimeError: Can't call numpy() on Tensor that requires grad. Use tensor.detach().numpy() instead.

The error is caused by this line:
https://github.com/baaivision/EVA/blob/098c493af0b31499ee2930beb09edcc38fac2974/det/detectron2/modeling/backbone/utils.py#L100
I fixed it by changing it to:
z = rel_pos[:, i].view(src_size).cpu().float().detach().numpy()

After this change, the training runs without issue and the loss decreases steadily.
But I am not sure if I understand the full implications of this change.
Calling .detach() means the gradients are not updated for this Tensor. Or is that not an issue for this call? Did you not get this error during training?
I am running EVA inside docker with CUDA 11.1, Python 3.7, torch 1.9.0, torchvision 0.10.0, mmcv-full 1.6.1. But I doubt this is a versioning issue

Hi @CamielK, thanks for trying EVA and reporting this issue.
The current rel pos interpolation implementation is from BEiT, which we find can bring ~0.2 AP improvement if evaluated with a higher resolution (e.g., 1536px) than training / fine-tuning (e.g., 1280px) compared with the original rel pos interpolation implementation in ViTDet.
When we training / fine-tuning with a higher resolution, e.g., from 1024px on O365 to 1280px on COCO, we actually use the original rel pos interpolation implementation in ViTDet.
I think your hack (i.e., z = rel_pos[:, i].view(src_size).cpu().float().detach().numpy()) is also OK if you want to fine-tune at a higher resolution (because this implementation can bring improvement during evaluation even without fine-tuning). I also suggest you try the original implementation in ViTDet. But I think there is no much difference in terms of the performance.
Finally, all these implementations interpolate rel pos on the fly for every forward / backward pass, which I believe it is better to interpolate only once at the time one initializes the model. I will also work on this as long as I have time / GPUs.

I checked out the latest commit and the training is much faster now. I have not done extensive tests but it appears to be almost twice as fast with the ViTDet implementation. I will defintely test the BEiT approach during final eval to compare AP performance
Thanks for the quick fix!


Error in bbox calculation

Describe the bug
Non-manhattan rotations in subcomponents lead to incorrect values in bbox_np. It appears that the subcomponent's rotated bounding box contributes to the total instead of the subcomponent's geometry.
To Reproduce
import gdsfactory as gf
import numpy as np

@gf.cell
def pyramid():
    comp = gf.Component()
    comp.add_polygon(((0, 0), (1, 1), (2, 0)), layer=(1, 0))
    return comp

def test_off_axis_bbox():
    comp = gf.Component()
    subcomp = comp.add_ref(pyramid())
    subcomp.drotate(45, center=(0, 0))

    rt2 = np.sqrt(2)
    # The triangle should form the upper-left corner of a rt2-width square.
    assert np.allclose(comp.bbox_np(), ((0, 0), (rt2, rt2)))
    # Error: ((-rt2 / 2, 0), (rt2,  rt2 * 1.5)) != ((0, 0), (rt2, rt2))

Expected behavior
The bounding box should always be calculated directly from geometry, not from the bounding boxes of child components.

The same appears to be true for the gf.kdb.DBox produced by comp.dbbox().

I would argue this is not a bug. It's purely how you define a bbox.
Also, to my knowledge, no fab I know of really allows you to have instances like this (or off-grid instances in general)
If you want the true bbox, flatten the ref in, or use the various tools like recursive shape iterators. Note though, that will be extremely expensive compared to the standard bbox calculation.
If you still think this is absolutely a bug, I would suggest to open an issue on klayout as the behavior is fully inherited from klayout.


[WIP] Feature/ttf 466 style a z page

This isn't 100% quite yet, but is close enough to merge it in for testing tonight? @Koded what do you think of this? Is this against everything?? Feels a bit naughty but I would like to sort the /drug/cannabis page and this is more important for testing


Update required

Please update the version of nordic UART to the latest

Please, use nRF Toolbox -> UART. nRF UART is in Nordic Playground and it's not fully supported.
Do you have any specific use case you need it for?

Check out this https://github.com/NordicSemiconductor/Android-BLE-Library/ library. Good luck!
Also, easy example is nRF Blinky https://github.com/NordicSemiconductor/Android-nRF-Blinky.


Add backup reminder in workaround

Hello!
An addition I think it's worthwhile to add to the workaround procedure - A reminder to create a backup before starting.
Cheers!
BR's
Ruben S

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

ID: 5853ce95-f163-0739-0a57-50aab1719bdd
Version Independent ID: b1536434-3675-a736-22e8-28e5fea49f06
Content: Steps for in-place system upgrades for supported Windows 10 Azure VMs and workarounds for unsupported versions - Virtual Machines
Content Source: support/azure/virtual-machines/in-place-system-upgrade.md
Service: virtual-machines
GitHub Login: @genlin
Microsoft Alias: genli

@rsantosms, Agreed. I will add a backup reminder to the article.  Thanks for the feedback.

Fix https://github.com/MicrosoftDocs/SupportArticles-docs/issues/685

Thanks!


add: angular-buch logo

The first German book about Angular: https://angular-buch.com
...2nd edition is coming soon!

Thank you for this PR, but we only add logos that just use the Angular logo itself, not with extra text.
So I sadly cannot add this one.

You should change that rule! 😆

Sorry my friend, that is something I'm not really willing to do :-)


Workspaces: support for building multiple versions

Using workspaces/packages will allow multiple versions of the web server app to be maintained while the current arrangement only allows 1 Vite app to be built. This is mostly in preparation for the upcoming v3 revision with a much improved HA inspired UI.

Static files for v1 / v2 should still be built and deployed like they are currently
Adding another version package should be simple after this change set
Added the ability to override the proxy target via PROXY_TARGET environment variable
Updated documentation for the new arrangement

Worth nothing that according to the netlify build/deploy process (which presumably uses this change set), it seems to work just like it did before which is expected behavior.


Issue remains with edge cases

I still seem to be hitting the same scenario where client._keepaliveTimer === null at line 19 in index.js
client._keepaliveTimer.reschedule(client._keepaliveInterval)

It's hard to recreate the exact conditions as this is in our production system, but the packet which is being handled at that moment is...
{"cmd":"puback","retain":false,"qos":0,"dup":false,"length":2,"topic":null,"payload":null,"messageId":30407}
...in case that helps to make sense of it. Otherwise I have to do some fairly heavy lifting to figure out the full sequence of events leading up to this failure.
Here's the stack trace...

TypeError: Cannot read property 'reschedule' of null
    at handle (/Users/cefn/Documents/code/imagination/git/aedes/lib/handlers/index.js:19:27)
    at Parser.enqueue (/Users/cefn/Documents/code/imagination/git/aedes/lib/client.js:204:3)
    at Parser._newPacket (/Users/cefn/Documents/code/imagination/git/aedes/node_modules/mqtt-packet/parser.js:30:10)
    at Parser.parse (/Users/cefn/Documents/code/imagination/git/aedes/node_modules/mqtt-packet/parser.js:42:48)
    at DestroyableTransform.nextBatch (/Users/cefn/Documents/code/imagination/git/aedes/lib/client.js:55:23)
    at emitReadable_ (node_modules/websocket-stream/node_modules/readable-stream/lib/_stream_readable.js:448:10)
    at emitReadable (node_modules/websocket-stream/node_modules/readable-stream/lib/_stream_readable.js:444:5)
    at readableAddChunk (node_modules/websocket-stream/node_modules/readable-stream/lib/_stream_readable.js:187:9)
    at DestroyableTransform.Readable.push (node_modules/websocket-stream/node_modules/readable-stream/lib/_stream_readable.js:149:10)
    at DestroyableTransform.Transform.push (node_modules/websocket-stream/node_modules/readable-stream/lib/_stream_transform.js:145:32)
    at WebSocket.onmessage (node_modules/websocket-stream/stream.js:74:11)
    at WebSocket.onMessage (node_modules/websocket-stream/node_modules/ws/lib/WebSocket.js:414:14)
    at Receiver.onbinary (node_modules/websocket-stream/node_modules/ws/lib/WebSocket.js:804:10)
    at node_modules/websocket-stream/node_modules/ws/lib/Receiver.js:533:18
    at node_modules/websocket-stream/node_modules/ws/lib/Receiver.js:357:7
    at node_modules/websocket-stream/node_modules/ws/lib/PerMessageDeflate.js:217:5
    at afterWrite (_stream_writable.js:346:3)
    at onwrite (_stream_writable.js:337:7)
    at WritableState.onwrite (_stream_writable.js:89:5)
    at afterTransform (_stream_transform.js:79:5)
    at TransformState.afterTransform (_stream_transform.js:54:12)
    at Zlib.callback (zlib.js:614:5)

As a reference, the same production scenario completes with Mosquitto without errors and we're only engaging with Aedes as a client through MQTT.js (with no deeper integration) so I don't think we're doing anything unusual from our side.

Oh, Aedes is at best beta software. It did not go through all the hardening that real-world did with Mosca.
Anyway, PR is coming for you to test.

Oh, Aedes is at best beta software. It did not go through all the hardening that real-world did with Mosca.
Anyway, PR is coming for you to test.


Allow refreshing tokens without typ field

The standard "typ" field is not mandatory for JWTs. The token refresh
implementation does not actually rely on the "typ" field in case a
custom TTL is provided, but the set_ttl function header mandates the field.
Instead of matching in the function header, move fetching the token type
into the branch where it is actually necessary.

Hi @urmastalimaa
Looking good, do you by any chance a reference to the docs for this?

Hey @Hanspagh, can you clarify what docs are you referring to?

Just in case, this is the "typ" field documentation in the JWT RFC: https://tools.ietf.org/html/rfc7519#section-5.1

Thank you. This was what I was looking for. This looks perfect.


Minor improvements to SwiftLanguageServer.swift

Introduce a new sub-logging scope for the implicitly triggered publish diagnostics notification

Just makes it clearer why some logs are being triggered if you know that it’s for the publish diagnostics notification.

Don’t log a fault if the publish diagnostics notification is cancelled

Cancelling the publish diagnostics notificaiton because there was an edit is totally normal and nothing to be logged at the fault level.

Format SwiftLanguageServer.swift

@swift-ci Please test


Can you run fixed effects or set check_rank=False in IV2SLS?

I have a panel with multiple time periods per entity. I would like to run IV2SLS with entity effects as well as other effects.
Normally, in PanelOLS, I would set entity_effects=True and other_effects=df['other_effect'] to use the fixed effects that I need. However, these parameters do not seem to be present in IV2SLS. How can this be done?
If this has not been implemented yet, is there a way to set check_rank=False like you can in a PanelOLS estimation?
I tried to create my own fixed effects by hand using pandas.get_dummies. However, the estimation stops because one of my matrices does not have full rank.

You could probably purch the FE from your Y, Z and X, and then run standard IV on the purged data..


[backport 1.1.latest] add python version and upgrade action (#6204)

Backport https://github.com/dbt-labs/dbt-core/commit/c3ccbe3357d15e45153c70b62bea626d37fc0d05 from https://github.com/dbt-labs/dbt-core/pull/6204.

@emmyoop since this is for the logging schema check (and isn't required) plus this is just a change to the Actions file, I'm ok if you merge this PR with those failures. We can investigate those separately


Prefix Sums approach

@iamehran I wish to add the prefix sums approach in both java and C++

done


Add support for computed_field in Table component.

Fixes: #320

@sydney-runkle @samuelcolvin I've reviewed this PR, yet maybe one of you wants to take a quick look before merging. It's a pretty small/simple fix. Thanks!


[cfid-5063] UAA Release Monit Healthcheck Ignores Ports 8080 & 8443 - Story Id: 116659675

Link: https://www.pivotaltracker.com/story/show/116659675
Labels:
Requester: Caroline Taymor
Owners: Madhura Bhave
Estimate: 0.0

The monit health check for UAA release only checks port 8989. However, UAA also listens on 8080 and 8443, but those ports open up later than 8989 during the startup process.
The health check should cover all ports used by UAA to avoid cases where callers assume the port is open, when it is actually still closed.
https://github.com/cloudfoundry/uaa-release/blob/develop/jobs/uaa/monit
We think the following change causes a greater chance of this occurring because of the port being available earlier for monit to see it: https://github.com/cloudfoundry/uaa-release/commit/5c1d0f3dba401c9f5910201ba5786152cbddcc70

Link: https://www.pivotaltracker.com/n/projects/997278/stories/116659675/comments130563653
User: Aakash Shah

We've been seeing this in our CI more frequently lately - ping @stummidi
Comment Id: 130563653

Link: https://www.pivotaltracker.com/n/projects/997278/stories/116659675/comments132285939
User: Caroline Taymor

Could you please let the OpsManager team know when this is released by commenting on our follow up story? Thanks.
#118344031
Comment Id: 132285939

Link: https://www.pivotaltracker.com/n/projects/997278/stories/116659675/comments134134579
User: Matt Royal

@stummidi This is still causing reliability problems in Ops Manager 1.7 and above. Can you please prioritize this?
Comment Id: 134134579


Support convert_from(b: bytea, src_encoding: text) -> text

Is your feature request related to a problem? Please describe.
Convert data b from original encoding specified by src_encoding into text.
Describe the solution you'd like
Postgres's doc:
https://www.postgresql.org/docs/9.1/functions-string.html
convert_from(string bytea, src_encoding name) -> text
Convert string to the database encoding. The original encoding is specified by src_encoding. The string must be valid in this encoding.
convert_from('text_in_utf8', 'UTF8')
Example:
postgres=# select convert_from('\x4346464558'::bytea, 'utf8');
 convert_from
--------------
 CFFEX
(1 row)

Describe alternatives you've considered
dev=> SELECT encode('\x4346464558', 'escape')
dev-> ;
 encode
--------
 CFFEX
(1 row)

encode('\x4346464558', 'escape') achieves the same thing if users are using utf8-compatible encoding for the converted string. ASCII is compatible to utf8 so it also works in this case.
43 in hex is equal to 67 in decimal, which corresponds to 'C' in ASCII.
46 in hex is equal to 70 in decimal, which corresponds to 'F' in ASCII.
46 in hex (again) is 'F'.
45 in hex is equal to 69 in decimal, which corresponds to 'E' in ASCII.
58 in hex is equal to 88 in decimal, which corresponds to 'X' in ASCII.
Additional context
No response

If not intentionally being PG 9 compatible, the latest doc is here:
https://www.postgresql.org/docs/16/functions-binarystring.html#FUNCTIONS-BINARYSTRING-CONVERSIONS
Which of the encodings do we intend to support?
https://www.postgresql.org/docs/16/multibyte.html#MULTIBYTE-CONVERSIONS-SUPPORTED


limit webapp icon size

In some cases the webapp displays a very large icon. The css code used to limit the icon size is not applied due to recent re-arrangement of the html template structure in hs_core.

@zhiyuli please check if this is fixed, and if it is close.

resolved on www. closed


Feature/SONYRT-1291

Platforms affected
What does this PR do?
What testing has been done on this change?
Checklist

[ ] ICLA has been signed and submitted to<EMAIL_ADDRESS>[ ] Reported an issue in the JIRA database
[ ] Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
[ ] Added automated test coverage as appropriate for this change.

We will send out another PR from the feature/SONYRT-1292 branch. Closing this PR for now


Console does not refresh

Hi,
with latest version 1.4.3 and 1.8.9 of hangfire the console does not refresh itself.
I have make an hard reset of the cache, i also set the consoleoption with PollInterval = 500 (i hope it is ms)
Any one can help me?
Thanks

duplicate: #128


Update CODEOWNERS

Remove Chris and add Bryan as a codeowner, affects default reviewers for PRs.

Haha! No it's okay! But I'll be keeping track. We may use it for our ground tools at AS o.o Will keep you guys posted.

Haha! No it's okay! But I'll be keeping track. We may use it for our ground tools at AS. Will keep you guys posted.

That would be very cool. Happy to add you back as a code owner at any point – figured you didn't want to be a default reviewer for stellar at the moment.


Minimize test work

This branch does a couple of things.

It fixes runtime errors caused by duplicated/overlapping inclusion of libraries in the main apk and the test apk.  The test apk should only include test-specific content (classes, resources, libraries)
It skips proguard on the test project altogether.  Given (1), the need to run proguard on the test project is pretty dramatically reduced.

With this branch, I'm finally able to successfully run some basic ProviderTestCase2 stuff from sbt.
The downside:  It's a little bit hacky (note the use of the skipApkLibDependencies key).

Quick note:  This doesn't allow for library dependencies added in the test project.
The test project dependsOn the main project.  This means that at runtime, when packaging the test project, there's no way to distinguish between dependencies added by the main project and those added by the test project.  I have to dex all dependencies, or none, into the test apk.  Dexing all dependencies would cause conflicts with the main apk, so for now it's none.
I don't really have a better idea than removing the project dependency relationship, but that would involve touching a lot more than I care to touch at the moment, and would likely leave current builds broken.


New Release

Checklist

[x] App update process has been followed
[x] Target branch is develop
[x] Application version has been bumped

You can find the changelog here.

Resolve https://github.com/LedgerHQ/app-stellar/issues/82

Codecov Report
Attention: Patch coverage is 88.15490% with 156 lines in your changes are missing coverage. Please review.

Project coverage is 89.42%. Comparing base (3d5896b) to head (bc565ff).

Files
Patch %
Lines

libstellar/parser.c
89.81%
83 Missing :warning:

libstellar/printer.c
85.15%
72 Missing :warning:

libstellar/base64.c
94.11%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##           develop      #84       +/-   ##
============================================
+ Coverage    76.87%   89.42%   +12.55%
============================================
  Files           14        5        -9
  Lines         2378     2742      +364
============================================
+ Hits          1828     2452      +624
+ Misses         550      290      -260

Flag
Coverage Δ

unittests
89.42% <88.15%> (+12.55%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Perhaps we need to disable the default CodeQL scan configuration. Otherwise, the CodeQL CI will not pass.
https://docs.github.com/en/code-security/code-scanning/troubleshooting-sarif-uploads/default-setup-enabled

Hello @overcat , ty for your PR
Could you please open a PR to update the ledger database (https://github.com/LedgerHQ/ledger-app-database) for Stellar in order to have the guideline CI green ?
Is there any test for the new device ? I didn't find any snapshots..
Last remark, you can probably:

use useCaseHomeAndSettings instead of useCaseHome + useCaseSettings
use useCaseAddressReview instead of nbgl_useCaseReviewStart + nbgl_useCaseAddressConfirmation

We have a good example in our boilerplate app. It would be more simple & easier to maintain (with the guarantee to be aligned with the design guidelines)

Hi @tdejoigny-ledger

Could you please open a PR to update the ledger database (https://github.com/LedgerHQ/ledger-app-database) for Stellar in order to have the guideline CI green ?

I added it a few days ago, and now CI has passed.

Is there any test for the new device ? I didn't find any snapshots.

We use zemu for e2e testing, but it does not yet support Flex, so we may need to wait some time before adding it.
The other four devices all have end-to-end testing.

Last remark, you can probably:
use useCaseHomeAndSettings instead of useCaseHome + useCaseSettings
use useCaseAddressReview instead of nbgl_useCaseReviewStart + nbgl_useCaseAddressConfirmation
We have a good example in our boilerplate app. It would be more simple & easier to maintain (with the guarantee to be aligned with the design guidelines)

I have made the corresponding modifications. Because I also want to make some changes in the UI aspect, originally I planned to submit them together at that time, but now let's first submit this part.
CodeQL test failure is likely due to repository configuration issues: https://github.com/LedgerHQ/app-stellar/pull/84#issuecomment-2114257826

Hi @tdejoigny-ledger

Could you please open a PR to update the ledger database (https://github.com/LedgerHQ/ledger-app-database) for Stellar in order to have the guideline CI green ?

I added it a few days ago, and now CI has passed.

Is there any test for the new device ? I didn't find any snapshots.

We use zemu for e2e testing, but it does not yet support Flex, so we may need to wait some time before adding it. The other four devices all have end-to-end testing.

Last remark, you can probably:
use useCaseHomeAndSettings instead of useCaseHome + useCaseSettings
use useCaseAddressReview instead of nbgl_useCaseReviewStart + nbgl_useCaseAddressConfirmation
We have a good example in our boilerplate app. It would be more simple & easier to maintain (with the guarantee to be aligned with the design guidelines)

I have made the corresponding modifications. Because I also want to make some changes in the UI aspect, originally I planned to submit them together at that time, but now let's first submit this part.
CodeQL test failure is likely due to repository configuration issues: #84 (comment), it is running well in my repository: https://github.com/lightsail-network/app-stellar/actions/runs/9869736057

Ok ty @overcat I will approve & merge this PR


Check go version before export GCFLAGS for debug.

When DEBUG is set to 1, for go higher than or equal to 1.10, add
pattern 'all=' in the flag, to apply '-l -N' to all the dependencies.
Refs:

https://golang.org/doc/go1.10#build
https://github.com/golang/go/blob/release-branch.go1.10/src/cmd/go/internal/work/build.go#L113-L127

/assign @geeknoid
Would you help to review this change @geeknoid ? Many thanks!

I'll check the reasons why e2e & race tests fail

Move to master, not critical

@costinm Should I make another PR to the master branch and close this one?
By the way, I see there is a $VER_TO_INT awk command to convert go version into an integer(like: 1.9->10900), which makes less code change to check go version. I'll use it instead

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

Needs approval from an approver in each of these files:

OWNERS [rshriram]

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

@crystaldust: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
780d90739d644a84a2ffb52dffbf96e8429d923e
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[4.0] Fix strideable methods for large unsigned values

This is cherry-picked from #11917
Explanation: Fixes a bug where calling advance(by:) with valid inputs can unexpectedly error.
Scope of Issue: Bug fix only, no API change.
Risk: Minimal
Reviewed By: Max Moiseev
Testing: This passes the automated test suite and includes expanded tests for the behavior fixed here.

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - d613e36bb5af78bb0d90bd801152b614d67ea2bf

Build failed
Swift Test Linux Platform
Git Sha - d613e36bb5af78bb0d90bd801152b614d67ea2bf


Update PRIVACY POLICY.md

Hi;
Privacy Policy Update.

Removal of unnecessary elements, additions and corrections of existing ones.
Normally it covers everything.

To merge if appropriate and at a time that suits you.

Very comprehensive, thanks.


Add Development section to readme

Preview: https://github.com/stephanlachnit/moderncv/blob/p/readme-improvements/README.md

Incidently, should we have a platform to communicate? A slack/discord/something, since you're speaking of "loosely structued team" it would probably help to have a way to ask for help, or for people interested in joining to contact us on how to get started.

Hm, don't know, opinions @cryptointerest @mefuller @gcorrad?
I personally am a fan of matrix.org (i.e. https://app.element.io/), since it's more open source than the others, but I'm also not sure if there would be much to talk about that couldn't be handled on GitHub anyway. But I'm open for it if you think it's useful.

I also prefer matrix to other platforms.

Ok, I'm going to merge this now and created #91 for discussing the idea of a communication platform.


[DashboardLayout and Account] A Few Issues

Steps to reproduce
<DashboardLayout
  defaultSidebarCollapsed
  slots={{ toolbarAccount: () => <></>, sidebarFooter: SidebarFooter }}
>

const SidebarFooter = ({ mini }) => (
  <Account
    slotProps={{ preview: { variant: mini ? "condensed" : "expanded" } }}
  />
);

Current behavior
Is there a more effective way to disable the toolbarAccount? From my tests, the only method to disable it is as shown in the POC. It might be beneficial to introduce a hideAccount prop for the DashboardLayout. Additionally, when the Account is set as a sidebarFooter, the arrow connecting the Avatar to the menu is pointing in the wrong direction, see the POC.
Expected behavior
No response
Context
No response
Your environment
  System:
    OS: Windows 10 10.0.19045
  Binaries:
    Node: 20.5.0 - C:\Program Files\nodejs\node.EXE
    npm: 10.2.5 - C:\Program Files\nodejs\npm.CMD
    pnpm: Not Found
  Browsers:
    Chrome: Not Found
    Edge: Chromium (129.0.2792.65)
  npmPackages:
    @emotion/react: ^11.13.3 => 11.13.3
    @emotion/styled: ^11.13.0 => 11.13.0
    @mui/base:  5.0.0-beta.60
    @mui/core-downloads-tracker:  6.1.5
    @mui/icons-material: ^6.1.5 => 6.1.5
    @mui/lab: ^6.0.0-beta.13 => 6.0.0-beta.13
    @mui/material: ^6.1.5 => 6.1.5
    @mui/private-theming:  6.1.5
    @mui/styled-engine:  6.1.5
    @mui/system:  6.1.5
    @mui/types:  7.2.18
    @mui/utils:  6.1.5
    @mui/x-charts: ^7.22.0 => 7.22.0
    @mui/x-charts-vendor:  7.20.0
    @mui/x-data-grid: ^7.22.0 => 7.22.0
    @mui/x-date-pickers: ^7.22.0 => 7.22.0
    @mui/x-internals:  7.21.0
    @toolpad/core: ^0.8.1 => 0.8.1
    @toolpad/utils:  0.8.1
    @types/react:  18.3.3
    react: ^18.3.1 => 18.3.1
    react-dom: ^18.3.1 => 18.3.1
    typescript:  4.9.5

Search keywords: account dashboardlayout

The sidebar footer variant is still in progress at https://github.com/mui/toolpad/pull/4255 - feel free to share any feedback on the PR itself.
We discussed the hideAccount prop once earlier and felt like the current way is sufficient for the time being and not worth introducing a new prop on DashboardLayout


schema.rb not being generated correctly after migration

My migrations are not creating the correct entires in my schema.rb file.
See my example migration:
create_table :my_table, { id: false } do |t|
      t.uuid :id, primary_key: true

      t.timestamps
end

And this generates an entry in my schema.rb, without my uuid primary key
create_table "my_table", force: :cascade do |t|
    t.datetime "created_at"
    t.datetime "updated_at"
  end

I then need to manually change my shema.rb to the following in order for it to work correctly:
create_table "my_table", id: false, force: :cascade do |t|
    t.uuid        "id",                limit: 16, primary_key: true
    t.datetime "created_at"
    t.datetime "updated_at"
end

Is there something existing I can enter into my migrations to get this to work?

@cameronbourgeois what version of Rails are you using?
We just upgraded from Rails 4.0 to Rails 4.2, and now every time we run a migration, in the schema file every table that had a uuid 'id' column loses its id column entirely, even if the migration itself is for something completely unrelated.
If our problem is the same thing as yours, that suggests the problem isn't with your migration syntax, it's with how this gem interacts with ActiveRecord in Rails 4.2.
@jashmenn, this repo hasn't had any commits in 10 months - is it still being maintained by anyone and is there a reasonably chance that anyone will be able to look into this?

@patrick-gleeson I've had this issue in both Rails 4.2.5 and 4.2.6. It does makes sense that this may be a compatibility issue with newer versions of ActiveRecord since this gem seems to have fallen out of maintenance.
My work-around was to use structure.sql instead of schema.rb by adding the following line to config/application.rb:
config.active_record.schema_format = :sql

See http://edgeguides.rubyonrails.org/active_record_migrations.html#schema-dumping-and-you

Ah nice one! Yep, that's working for us too as a workaround.

The problem is in the migration syntax.
The example in the README, as used above:
create_table :my_table, id: false do |t|
      t.uuid :id, primary_key: true
      t.timestamps
end

This does not work in rails 4.2!! Instead, you need to use:
create_table :my_table, id: false do |t|
      t.uuid :id, primary: true, null: false
      t.timestamps
end

This will generate the following line in your db/schema.rb:
t.uuid     "id",           limit: 16,  null: false

@patrick-gleeson - I've added you as a contributor! Feel free to make commits directly!


Support for GDAL 3

Whilst working on #11 it is also apparent that some changes are necessary to support GDAL 3.
Functions removed in OSR API:

OSR_FixupOrdering()
OSR_Fixup()
OSR_StripCTParms()

Additionally, results for the PHPUnit-Tests must be updated, particularly w.r.t. coordinate axis order handling.

excellent point!  (GDAL 3.4.0 is the goal)

@jmckenna With all 4 open PR applied I can now compile and run the unit tests on both PHP 7.3 with GDAL 2.4 and PHP 8.0 with GDAL 3.0.
Let me know how you get on! If it all works for you then I'll merge everything and tag a release.

Closed with #15


Cow: Watch dandelion[\W_]*+studio

Cow requests the watch of the watch_keyword dandelion[\W_]*+studio. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

dandelion[\W_]*+studio has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Need to update composer-cli commands for new version

In Step 2 the following needs to be changed:
Replace:
 composer runtime install --card PeerAdmin@hlfv1 --businessNetworkName events
with
 composer network install -a events.bna -c PeerAdmin@hlfv1
And change:
 composer network start --card PeerAdmin@hlfv1 --networkAdmin admin --networkAdminEnrollSecret adminpw --archiveFile events.bna --file networkadmin.card
To:
 composer network start --card PeerAdmin@hlfv1 --networkAdmin admin --networkAdminEnrollSecret adminpw --file networkadmin.card --networkName events -V 0.0.1

Resolved using the latest command-line switches/arguments supported by Hyperledger Composer 0.19.5.


Bug: Next Launch not loading on Wear Watch face

Device: Moto 360 (wear 1.4, android 6.0.1, Nexus 5 on 6.0.1 connected)
Bug: On the watch face, the next launch indicator never loads, continuously shows "Syncing Next Launch..."
Edit: SLN version 1.4.2

Fixed for good in the latest Alpha build #68


Update Set-SPWebApplication.md

Add new paragraph in description to infer user that IIS binding settings should be respecified when updating the binding of an IIS web site.

Docs Build status updates of commit 07370e0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-server-ps/sharepoint-server/Set-SPWebApplication.md
:white_check_mark:Succeeded
View (sharepoint-server-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@mingzhangMicrosoft Gentle reminder.

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

File
Status
Preview URL
Details

sharepoint/sharepoint-server-ps/sharepoint-server/Set-SPWebApplication.md
:white_check_mark:Succeeded
View (sharepoint-server-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 118a048:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-server-ps/sharepoint-server/Set-SPWebApplication.md
:white_check_mark:Succeeded
View (sharepoint-server-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Multiplication by non-square matrix should drop labels

On 0.10.7 I get:
julia> rand(2,3) * ComponentArray(x=[1,2,3])
2-element ComponentVector{Float64} with axis Axis(x = 1:3,):
 3.0820225302525293
 3.921995265502881

julia> [rand(2,3) * ComponentArray(x=[1,2,3])]
1-element Vector{ComponentVector{Float64, Vector{Float64}, Tuple{Axis{(x = 1:3,)}}}}:
Error showing value of type Vector{ComponentVector{Float64, Vector{Float64}, Tuple{Axis{(x = 1:3,)}}}}:
ERROR: BoundsError: attempt to access 2-element Vector{Float64} at index [1:3]

When you multipy a ComponentArray by a rectangular matrix, you've changed the dimensionality so it doesn't make sense to preserve the labels. Additionally, you end up with an object whose labels and actual length are inconsistent, which can lead to e.g. the printing error in the second line.
Afaict the solutions are 1) drop the labels for non-square matrices, although a side-effect is that by definition Matrix * ComponentVector is no longer type stable, or 2) drop the labels for both square and non-square matrices, in which case it can be type stable, and if the user wants labels they need to label the matrix explicilty and hence do ComponentMatrix * ComponentVector. Not sure which is the better choice.

Oh, this is a definitely a bug. Even for square matrices this is wrong. For example, if the matrix was a ComponentMatrix, the behavior works as expected (i.e. it only takes the labels from the first axis of the matrix):
julia> x = ComponentArray(x=1:3);

julia> y = ComponentArray(y=4:6);

julia> z = ComponentArray(z=7:9);

julia> yz = y * z'
3×3 ComponentMatrix{Int64} with axes Axis(y = 1:3,) × Axis(z = 1:3,)
 28  32  36
 35  40  45
 42  48  54

julia> yz*x
ComponentVector{Int64}(y = [200, 250, 300])

This shouldn't be happening:
julia> getdata(yz)*x
ComponentVector{Int64}(x = [200, 250, 300])

The labels should come from the first axis of the matrix here and when that matrix doesn't have labels on the first axis (or, in this case, any axis), the result should be a plain Vector.
I'll have to see what's going on in the broadcasting machinery that's causing this issue.

Ah, here's the issue. The default matrix by vector multiplication in LinearAlgebra creates the output vector like this:
similar(x, TS, axes(A,1))

That calls this method in ComponentArrays:
function Base.similar(x::ComponentArray{T1,N,A,Ax}, ::Type{T}, dims::NTuple{N,Int}) where {T,T1,N,A,Ax}
    arr = similar(getdata(x), T, dims)
    return ComponentArray(arr, getaxes(x))
end

The reason this doesn't happen with A::ComponentMatrix * x::ComponentVector is because that case is handled explicitly here. There are basically two options to go from here:

Handle all multiplication and left/right division by AbstractMatrix and AbstractVector explicitly like this. I really don't want to do this, though. This is how things were set up in the past and it led to method ambiguity hell. In essence, ComponentArrays will no longer be able to multiply with other nonstandard arrays (like StaticArrays) without explicit overloads. It's pretty unmaintainable.
Make Base.axes return the Axis of the ComponentArray (which I've already been meaning to do for a while). I'm pretty convinced this is the correct way to handle it. I don't remember why this didn't work when I tried it before, but I'm sure I can work though it.


Attempt HEAD when resolving backlink, fall back to GET

I had to stop using HEAD for backlinks because the XHR requests just sometimes don't work. It would be better to try HEAD, and fall back to a GET using the onerror handler if it breaks, so that we're at least usually not downloading useless bytes.

Switched back to HEAD in 2.0 and haven't seen issues, I think it was the combination of that and the extra headers I was sending.


Mac OS - sudo node install_chaincode.js  - gives error

Trying to setup Marbles demo locally.
Gives error in this step.
It is searching for ...-priv file in main directory /Users/Govind/.hfc-key-store/.
While the file is in Users/Govind/Documents/032017Official/learning/marbles/config/crypto/prebaked/...
Have already put this in my .profile as mentioned in the tutorial:
export PATH=$PWD/bin:$PATH

info: Loaded config file /Users/Govind/Documents/032017Official/learning/marbles/config/marbles_local.json
info: Loaded creds file /Users/Govind/Documents/032017Official/learning/marbles/config/blockchain_creds_local.json
info: Lets install some chaincode - marbles v4
info: First we enroll
info: [fcw] Going to enroll with admin cert!  peer_urls=[grpc://localhost:7051], channel_id=mychannel, uuid=marbles-Docker Compose Network-mychannel-fabric-peer-org1, orderer_url=grpc://localhost:7050, msp_id=Org1MSP
events.js:160
throw er; // Unhandled 'error' event
^
Error: ENOENT: no such file or directory, open '/Users/Govind/.hfc-key-store/cd96d5260ad4757551ed4a5a991e62130f8008a0bf996e4e4b84cd097a747fec-priv'
at Error (native)

Copied the contents of /fabcar/hfc-key-store into a newly created folder Users/Govind/.hfc-key-store.
Now it works.


Kit and Yiyang's Poetry Slam Turtlehack

@EternalFootman

Really well done!  The writing experience on your program is definitely very natural feeling since there's no prompt, just a space for me to type.  And your "".join(list[:20]) logic is nicely implemented. :star2:


Small pd.link improvements

Reduced binary size for [pd.link]
Future proofed signal header
Fixed buffer overrrun bugs with large bufsize values
Removed some files I accidentally added

@timothyschoen do you know anything about these 'bot' update PRs I'm getting? weird...


Make it possible to check if we have previously been granted permission

In some situations, it might be desirable to run code such as this
if( Push.Permission.has() ){
  Push.create( 'Notification!' );
} else {
  // Something else
}

Right now this might not work as expected because hasPermission isn't set to the correct value if we load a page where we have previously been granted permission.
This PR just initializes hasPermission to true if we have previously been granted permission.

Coverage increased (+0.2%) to 70.0% when pulling 242fa6f23298af71e79a83bf8e620b6748ad97fd on kokarn:master into a8b79803a4da9ad4e0ade75a1ef13c871860b662 on Nickersoft:master.

Hey, thanks for your contribution! I'm not sure if you noticed, but hasPermission is already defined at the top of the file. However, you're PR does simplify the whole has() function of the Permission object, which I like, so what I would recommend is returning Permission.get() === Permission.GRANTED directly from the self.Permission.has() function and removing all traces of hasPermission in the main file. This includes the two assignments made in the request() function's switch statement. After that's done it should be good to merge.

Coverage increased (+0.9%) to 71.25% when pulling a4e4e708f20b9ad94c822224c81f5b5718b23ca9 on kokarn:master into be9c6dc6b9b7e812ae9172472ffd332d30da5047 on Nickersoft:master.

There, updated. Sorry it took so long.
I'm having some trouble with one of the tests. It used to test if hasPermission was updated but because we don't use that anymore it fails.
The functionality should ofc be tested but my limited knowledge of Karma is in the way.
As I understand it, I should make my test mock an updated of the property I want to test.
This makes it pretty hard because it's all different properties in different browsers.
Think you can help me out?

 Easier to just submit a new PR, brb


Can't access omnibar in full screen safari when chunkwm is running

If I make Safari full screen, everything is fine.
Until I make a new window, ⌥⌘N or ⇧⌥⌘N and then i can't access the Safari url bar for some reason.

Pretty safe bet here is that you're using window_float_topmost which has always had its little quirks. Nothing much you can do there, some applications don't work well with it, especially in native-fullscreen mode. Pressing ⌘L to focus the address bar should still work, though.

@dominiklohmann ⌘L is exactly what doesn't work

Have you ever looked at your ~/.chunkwmrc file?
You can find a reference of all options for the tiling plugin here: https://github.com/koekeishiya/chunkwm/blob/master/src/plugins/tiling/README.md

@dominiklohmann "signal dock to make windows topmost when floated"—I'm not sure what this means

"floating" is a window state for windows that are either not resizable or floated when in a tiled space or all windows when in a floating space. "topmost" means that even when you focus another window, that window remains visible over the background windows.

That fixed it