bug: [Ionic4.6.x] alertController.dismiss don't send data

Bug Report
Ionic version:
[x] 4.6.x
Current behavior:
expect dismiss be able to send data to onDidDismiss.
const alert = await this.alertCtrl.create({
  header: 'Header',
  message: 'Message',
  buttons: [
    {
      text: 'Close',
      role: 'cancel',
    },
    {
      text: 'Delete',
      handler: () => {
        alert.dismiss({ deleted: true });
      },
    },
  ],
});
await alert.present();
const { data } = await alert.onDidDismiss();
console.log(data);   // expect { deleted: true } , but undefined.

Expected behavior:
alertController.dismiss can send data
Other information:

<EMAIL_ADDRESS>is worked.
modalController is worked.

Ionic info:
   Ionic CLI                     : 5.2.0 (/usr/local/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.6.2
   @angular-devkit/build-angular : 0.13.9
   @angular-devkit/schematics    : 7.3.9
   @angular/cli                  : 7.3.7
   @ionic/angular-toolkit        : 1.5.1

Capacitor:

   Capacitor CLI   : 1.1.0
   @capacitor/core : 1.1.0

Utility:

   cordova-res : not installed
   native-run  : not installed

System:

   NodeJS : v10.16.0 (/usr/local/bin/node)
   npm    : 6.9.0
   OS     : macOS Mojave

The alert-controller handler dismisses the alert already, since the dismiss method is async and you are not waiting, it's actually happening in the next microtask. While this is a regression it was not using the Ionic APIs correct. Just return false to prevent the handler to close the alert with undefined data, before, alert.dismiss() happens!


'sync-back' and simple 'preparator' prototype for VagrantProvision

Important: I've not tested 'sync-back', nor preparator (but it should work). And I'm thinking about better solution for both...
The prepare step Itself will be in another PR.
Feel free to merge it if it works for you, we can enhance it later on.

Implement 'sync-back' and simple 'preparator' prototype.
Additionaly:

handle images properly
better debug/info: pass it (to super) every time
move init() to go()
abstract add_config_block
resume correctly with existing Vagrantfile
Better Vagrant 'config' handling

FTR: What is needed, additionaly to this PR:

https://github.com/vagrant-libvirt/vagrant-libvirt/issues/1058 for compatibility with system vagrant-libvirt on Fedora (dependency on fog-core)
decide to use either vagrant-scp or vagrant-rsync-back with git-ignore
decide whether to create an VagrantOrchestrator (not sure about the name) and inherit general methods from it (untrivial implementation, even now) for VagrantProvider and VagrantPreparer. (It would make sense to me, but we need some consensus.)

Looks good for now. Merged. I tried to enable sync-back, but failed with this error:
Run command '('vagrant', 'plugin', 'install', 'vagrant-rsync-back')'.
out: Installing the 'vagrant-rsync-back' plugin. This can take a few minutes...
err: Vagrant failed to properly resolve required dependencies. These
err: errors can commonly be caused by misconfigured plugin installations
err: or transient network issues. The reported error is:
err
err: conflicting dependencies fog-core (~> 1.43.0) and fog-core (= 2.1.2)
err:   Activated fog-core-2.1.2
err:   which does not match conflicting dependency (~> 1.43.0)
err
err:   Conflicting dependency chains:
err:     fog-core (= 2.1.2), 2.1.2 activated
err
err:   versus:
err:     vagrant-libvirt (> 0), 0.0.41 activated, depends on
err:     fog-core (~> 1.43.0)

That seems to be related to the fog-core issue mentioned above.

Yes, that's the linked issue; for quick workaround do:
$ sudo dnf remove vagrant-libvirt -y && vagrant plugin install vagrant-ilbvirt

Additionally, I'll  package vagrant-rsync-back into my COPR, possibly into Fedora later on.

I personally run the following:
➜  for p in ruby-devel rubygem-{formatador,excon,builder,ruby-libvirt} ; do dnf -q install -yb "$p" ; rpm -q "$p" ; done
ruby-devel-2.6.5-124.fc32.x86_64
rubygem-formatador-0.2.5-8.fc32.noarch
rubygem-excon-0.65.0-1.fc32.noarch
rubygem-builder-3.2.3-7.fc31.noarch
rubygem-ruby-libvirt-0.7.1-6.fc31.x86_64
➜  vagrant plugin install vagrant-libvirt
Installing the 'vagrant-libvirt' plugin. This can take a few minutes...
Fetching fog-core-1.43.0.gem
Fetching fog-xml-0.1.3.gem
Fetching fog-json-1.2.0.gem
Fetching fog-libvirt-0.7.0.gem
Fetching vagrant-libvirt-0.0.45.gem
Parsing documentation for fog-core-1.43.0
Installing ri documentation for fog-core-1.43.0
Parsing documentation for fog-xml-0.1.3
Installing ri documentation for fog-xml-0.1.3
Parsing documentation for fog-json-1.2.0
Installing ri documentation for fog-json-1.2.0
Parsing documentation for fog-libvirt-0.7.0
Installing ri documentation for fog-libvirt-0.7.0
Parsing documentation for vagrant-libvirt-0.0.45
Installing ri documentation for vagrant-libvirt-0.0.45
Done installing documentation for fog-core, fog-xml, fog-json, fog-libvirt, vagrant-libvirt after 1 seconds
Installed the plugin 'vagrant-libvirt (0.0.45)'!

Uninstalling the rubygem-fog-core help. Documented in 6a0dd20.

Yes, basically. fog* packages are the real conflict itself, as the error message states. But it is a temporary problem, as it will be solved upstream+downstream as I mentined earlier (and I've created a helpful error message in #30).
Uninstalling all vagrant-* plugins should be sufficient, as I can't think of any other reason you'd have fog-* gems installed (and the resolver probably won't use them if you don't have the plugin load them; Ruby itself supports multiple versions of gems installed).


OSFUSE-515: Set EXTRA_JAVA_OPTIONS to JAVA_OPTIONS by default when me…

…m limits are not set

@dhirajsb please note that these are generated files which are taken from https://github.com/rhuss/run-java-sh/blob/master/fish-pepper/run-java-sh/fp-files/java-default-options and hence will be overwritten by the next run of fish-pepper.

This file is also not supposed to export variables since it is not source in the script but the output is used. Hence every variable you export here wont make it into the run script.

Oops, sorry.  wrong image ;-) what I said is valid for the java image, for karaf I don't know how it works. btw, we should even try to unify the scripts for both images sometimes.


Attempt to fix share instagram on iOS

This PR attempts to fix share Instagram on iOS based on suggestion here https://stackoverflow.com/questions/11393071/how-to-share-an-image-on-instagram-in-ios
Note that: I don't have iOS device with Instagram in order to verify this.
Resolves https://github.com/CovidShield/mobile/issues/62

I also can't verify though

This doesn't work for me on device. I don't get the option to select instagram on the page and I can't select it in the share sheet. I can see Instagram on my other sharing apps.

Fixed. Posted a video here https://drive.google.com/file/d/1v37L3s8_l-Whh7zAOPbCKSepjsX4WPuP/view?usp=sharing
For instagram, it only allows share image. What image should we share? I am putting CovidShield logo as an example right now.
cc @kpeatt @AdamWhitcroft

It works!


As a Developer I can easily generate a new SQL-on-FHIR based data warehouse from a FHIR server that can be queried via an SQL Query Engine in an on-prem environment or via BigQuery on GCP.

#7
#8
#9
#10
#11
#12
#6


Prepare data and quality check adapatation to data class

Adaptation of CapsDataset to CapsDatasetConfig
Suppression of PrepareDataConfig
return_dataset function back in data.py file
adapt cli.py files of prepare-data, prepare-data-from-bidsandquality-check` to the new organization

NEED TO REBASE  !

Hello @camillebrianceau! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file clinicadl/commandline/pipelines/prepare_data/prepare_data_from_bids_cli.py:

Line 30:1: W293 blank line contains whitespace


Automatically view allure reports.

Automatically view allure reports after finishing tests while running tests on local machine.

As requested, here are the steps to implement this new feature. Kindly provide confirmation that you will own implementing this new feature. In that case you should fork the project > create a branch from 4.1.x > implement the feature > pull... from the latest 4.1.x from the main repo (not your fork) > open a pull request to merge your branch from your repo into 4.1.x from the main repo.

Create a new property named [openAllureReportAfterExecution] under execution.properties, with [true/false] possible values

Create a new method named [openAllureReportAfterExecution] under ReportManager. This method should have no inputs/outputs, and it should follow this pseudocode:
IF openAllureReportAfterExecution=true AND executionAddress=local
THEN use TerminalActions to run the generateAllureReport.bat or .sh file depending on the current OS

Call the new method as the last step in the onFinish method under the SuiteListener

Note: You can refer to ReportManager.writeGenerateReportShellFilesToProjectDirectory() to see how you can manage different OSs, and make sure to implement different commands to open the relevant files for linux/unix (.sh) and windows (.bat).

Greetings Mohab,
I am confirming that implementation.
If I need to ask any question which channel would you prefer?
Regards,
On Tuesday, March 31, 2020, 12:52:29 PM GMT+2, Mohab Mohie<EMAIL_ADDRESS>wrote:

As requested, here are the steps to implement this new feature. Kindly provide confirmation that you will own implementing this new feature. In that case you should fork the project > create a branch from 4.1.x > implement the feature > pull... from the latest 4.1.x from the main repo (not your fork) > open a pull request to merge your branch from your repo into 4.1.x from the main repo.

Create a new property named [openAllureReportAfterExecution] under execution.properties, with [true/false] possible values

Create a new method named [openAllureReportAfterExecution] under ReportManager. This method should have no inputs/outputs, and it should follow this pseudocode:
IF openAllureReportAfterExecution=true AND executionAddress=local
THEN use TerminalActions to run the generateAllureReport.bat or .sh file depending on the current OS

Call the new method as the last step in the onFinish method under the SuiteListener

Note: You can refer to ReportManager.writeGenerateReportShellFilesToProjectDirectory() to see how you can manage different OSs, and make sure to implement different commands to open the relevant files for linux/unix (.sh) and windows (.bat).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@MamdouhMansour kindly feel free to ask anything via messenger. And let's update the ticket by linking it to the PR once it's created.


Alamofire 5 disable caching

How to disable caching in Alamofire 5?

To disable caching you can either configure a URLSessionConfiguration with a nil urlCache reference, or set the cachePolicy of any URLRequests that you make.


UofT-DSI | Applying Statistical Concepts - Assignment 2

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Adding answers to the assignment.
What did you learn from the changes you have made?
I learned how to apply a linear regression models to data with multiple predictor variables.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
No
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
I had some issues with github, due to changes in the notebook files. I was unable to push (due to conflicts), and was unable to pull due to conflicts. With a little maneuvering I was able to resolve the conflicts. This was also an issue with my last submission, which didn't get pushed to my github due to conflicts. I didn't notice at the time, so my answers weren't my final draft.
How were these changes tested?
I looked at the range of mpg in the original data set, and compared this to check if the error value I got was reasonable.
A reference to a related issue in your repository (if applicable)
Checklist

[ ] I can confirm that my changes are working as intended

A2: Complete
I would suggest making the following changes below to complete the assignment in full
Q1 part(ii) - What is the 'variable type' of the response variable mpg (e.g., 'integer', 'category', etc.) and explain what it is in short? (1-2 sentences at max). Please add your explanation for completeness.
Q2 part (ii) More specifically, what concept in linear regression does the plotted line represent ? "Line of best fit"


Image(RGB format) to PencilSketch

hello sir,
Hope you may like this project.
kindly contact me, if upgradation is required in code, I will be really happy to do so.
If Possible please merge my pull request.
Thankyou
Regards

Hey, saw you submission. Can you please format the text which you write a bit and rename it from Discription_of_project.txt to README.md in the same directory.

Done sir

Done sir

Great, merging.


A few examples on "getting LRS statements."

Would it be possible to create a folder with several examples of pulling records from an LRS. Something like the following would be enormously helpful to what we are trying to accomplish in my company.
getStatements1.html
This would be an example of a getting LRS records. It would pull down all verbs with no data restrictions and no special formatting. It would look like a large data dump.
getStatements2.html
This would be an example of a getting LRS records but giving the user a way to specify what data they want. For example, I would like to be able sometimes to just grab Actor, verb, object, date/time and context activity (like category). It would also have a variable that can be adjusted that would allow for getting only a certain number of records.
getStatements3.html
This would be like getStatements2 but with additional features that would allow the user to limit data to a date range or a category name. It would also be parsed for a neat appearance.
Thank you very much for your consideration. ~ John M.

You mean beyond the extensive Readme?
https://github.com/adlnet/xAPIWrapper#get-statements

Yes, I couldn't get the example code working. I was hoping for at least one example that I could call up seeing how everything was laid out in the html document. This really helped me with learning how to send statements, etc. Thank you.

Ok I'll keep it on the list.
By the way, have you looked at this: http://adlnet.github.io/xapi-statement-viewer/   .. if you expand the 'Query Options' it provides you a list of GET statement options.

Here's where that viewer uses the xAPI Wrapper to do the GET Statements: https://github.com/adlnet/xapi-statement-viewer/blob/master/js/app/main.js#L215

https://gist.github.com/creighton/f83be9812bcf949e7b834041c2919da2
Is that close to what you're asking for in your getStatements1.html scenario.
Note most of this isn't really xAPIWrapper features. It's just getting the data from the LRS and manipulating it with JavaScript.

Hi Mr. Creighton, thank you. Would you consider modifying the file you created so that it shows not unique verbs as it does presently but a dump of actor, verb, and object? I do see how my request could be interpreted as unique verbs but what I really meant to say was a "dump of actor, verb, and object."
Thank you.

updated gist

Thank you. This is perfect and exactly what I was hoping for.


Mxnet prediction on cpu is very slow

i am trying get predictions for images using imagenet-21k-inception model through the cpp classification code but the prediction on cpu is very slow.
The prediction takes about 1.06 seconds for a single image and i tried to give images in batch but still there is no speed gain. I thought that giving images in batch will speed things up.
Is there something that i am doing wrong?
Is there a way to make predictions fast on cpu?

compile mxnet with openmp

I compiled mxnet with USE_OPENMP = 1
After some profiling i realized that MXPredForward takes only 0.04 seconds for a single image but
MXPredGetOutput takes a lot of time about 1 second for one image. This function copies the result data after prediction
( https://github.com/dmlc/mxnet/blob/5ff545f2345f9b607b81546a168665bd63d02d9f/src/c_api/c_predict_api.cc )
Any way to make this fast?

@tqchen

there are several things that can be done to speed things up.

Use openblas, which support multi-threading
Use NaiveEngine, that eliminates the thread scheduling

@tqchen thanks a lot for the help!
I used openblas with USE_OPENMP = 1  and used naive engine which speeded things up :)  The prediction now takes 0.55 seconds for a single image.
I have a few more questions -

I want to make predictions for a batch of 8 images using imagenet-21k-inception model in under a second. Is that possible?

In the documentation of the predict api (http://mxnet.readthedocs.org/en/latest/doxygen/c__predict__api_8h.html) it is written that num_input_nodes = 1 for feedforward.
How can batching be done for feedforward prediction and will batching speed things up?

When you create the operator, change the input shape to include batch dimension should enable batching. i.e.
instead of
(1, channel, height, width) change it to  (batch_size, channel, height, width)

@tqchen  I am able to predictions for batch of images but it takes 4 seconds to get predictions for 8 images ie. i did not get any speed gain through batching.
Is there any way to get prediction results faster for batch of images in cpu mode?
Right now i am using the c prediction api , will using the cpp interface lead to better performance?

The bottleneck is on the computation then. Using cpp interface won't speed things up. You can, however, try NaiveEngine. Note that 21k-inception is a bigger net than normal inception used in imagenet, so the speed could be justified

okay thanks :) @tqchen

i have tested predict in cpp,and find MXPredGetOutput is very slow(200ms), but in python executor.forward and get out is very fast(4.5ms), why?@tqchen

the python execution API is asynchronous, you can only measure the time after it is copied to numpy array

@careyjiang have your problem solved? Would you give me some advice?

@qt6636
I solve this proble by change MXPredGetOutput(out, output_index, &(data[0]), size) to:
MyPredictor* p = static_cast<MyPredictor*>(pred_hnd);//MyPredictor is the same as MXAPIPredictor in c_predict_api.cc std::vector<mx_float> data(size);//here size is the length of output from your net mxnet::NDArray& nd_out = p->out_arrays[output_index]; auto src = nd_out.data(); std::memcpy(&(data[0]), src.dptr_, size * sizeof(mx_float));
this works for me, and the result is correct.

@YunLexi When you did the code change, how did you update the include files and make files? In order to define the new struct, you need to include ndarray.h and all dependent header files, right? Among them, tensor.h try to include <mkl.h>. However I built mxnet with openblas, not sure why it is trying to look for mkl.h ...

@rhythm9229 could you tell me how to use openblas and naive engine?

@KeyKy build multithreaded openblas on your system and set USE_OPENMP=1 , and set environment variable MXNET_ENGINE_TYPE to NaiveEngine


env() Helper Post php artisan config:cache

Laravel Version: >5.2.0
PHP Version: N/A
Database Driver & Version: N/A

Description:
It seems that after running config:cache, the env() helper is unavailable for use, as it will always return null. Apparently you're just supposed to make sure you only use the env() helper from inside your configuration files, but surely this isn't really expected or correct behaviour.
The upgrade notes from 5.1 to 5.2 state:
If you are using the config:cache command during deployment, you must make sure that you are only calling the env function from within your configuration files, and not from anywhere else in your application.
If you are calling env from within your application, it is strongly recommended you add proper configuration values to your configuration files and call env from that location instead, allowing you to convert your env calls to config calls.
Steps To Reproduce:

Create an env variable.
Run php artisan tinker.
Run dd(env('YOUR_VAR'));.
You should see your variable.
Now exit artisan tinker.
Run php artisan config:cache.
Re-do steps 2-3.

Yeah this is intended behaviour and not a bug. This repo is for bug tracking so this issue can be closed. If you wish to make any proposal to improve this functionality you can open an issue on the laravel/internals repo

It seems very odd 'intended behavior' to return null values for variables with values.

@danjdewhurst yeah.. seems odd. could be a bug, because env helper working fine with fascgi_params after config:cache but not for the variables in .env file

but surely this isn't really expected or correct behaviour

Why not? this is the recommended behaviour since 5.1 as far as I remember

Closing since it's not a bug, but feel free to ping me or use the forum for discussion

To my understanding the env() helper doesn't even touch the cache? I really don't see how this isn't an issue.
It works just fine until you cache your configs. Why should caching your configs stop the environment helper from being able to pull values from your environment file?

because they can give you different values? if you change the .env the changes won't be reflected in the cached version?

I think that's the point. When you cache the values, they aren't available at all. This might be by design but it's confusing, and there are pieces of the docs that show env() helper calls in views - such as Cashier placing Stripe credentials in views for the JavaScript.
If you can't use env() in a view then it shouldn't be documented as such. It's pretty confusing that caching configs would remove values too - it might be intended but it's extremely unexpected in my opinion.

because they can give you different values? if you change the .env the changes won't be reflected in the cached version?

@Dylan-DPC
After running php artisan config:cache, I'm unable to pull out any values using env(). That's the issue.

You are supposed to only read environment variables in your config files, and never from any other place. Once you cache the configuration files dotenv will no longer be invoked, the .env file will never be read and the environment variables aren't set. This means that env() (and getenv()) will not find the values you are attempting to read.
To repeat; env() DOES NOT read from .env, it's a helper for getenv() which reads proces-wide environment variables.
You should still be able to read env('PATH'), an environment variable you should already have without the dotenv or .env stuff.

 Then there are ZERO env issues in production.
Source: https://github.com/vlucas/phpdotenv/issues/76#issuecomment-224899197

@sisve
Thanks for your comment. I do now fully understand what's going on, I just don't think it's very clear, if anything. And I'd still say it was odd, default behaviour.
If you were new to Laravel, and you were trying to pull out environment variables, you may not know about using configs, so you'd probably find yourself using the env() helper. That's all well and good until you cache the config and that helper now just returns null values.

If i'm not mistaken, it is mentioned in the docs to not use env. If a new dev doesn't read the docs,he can't blame the framework :stuck_out_tongue:

So it does
https://laravel.com/docs/5.5/configuration#configuration-caching
Still think it's strange. Maybe in the docs for helpers, it should be said that the command should only be used in configuration files?
https://laravel.com/docs/5.5/helpers#method-env

The functionality of the helper doesn't change at all when you have a cached configuration, it still reads from environment variables as stated. The information about side effects about a cached configuration should be in the configuration docs.
Also, we cannot change the docs for the builtin getenv() that works very similar to the helper function. The helper only adds some type casting for special string values to proper types.

You can submit a pr to the docs to have it included there as well

run php artisan config:clear if you don't want to use the config cache
else use run php artisan config:cache

I'm having a very weird bug here, where sometimes if you reload the page before it has finished loading the env() function doesn't work in a local environment when config files are not cached (executed php artisan config:clear before). Can anyone reproduce that problem? It's a bigger project running on laravel 5.6.7.

@navadon28 That sounds like something that happens on Apache on Windows. It's a problem where multiple concurrent requests are executed within the same process and share the same environment variables. This will also share environment variables between different sites (main side that calls api site on same Apache installation).
The workaround is to either use a cached configuration (and thus not use environment variables), use nginx+fpm, or use another Apache mpm. I'm not sure which mpm or configuration that would be.

A small hack for this problem:
https://stackoverflow.com/a/70871023/9731538

we always should config function not env function. and in the config file we should get the value from env file with env() function.


global-whitespace-mode is enabled but whitespace-mode is disabled

What did you expect to happen?
whitespace-mode is enabled globally using global-whitespace-mode
What actually happened?
whitespace-mode is nil even though global-whitespace-mode is t
Describe your attempts to resolve the issue
Based on https://github.com/hlissner/doom-emacs/issues/2673, I have the following lines in config.el:
(use-package! whitespace
  :config
  (setq
    whitespace-style '(face spaces tabs newline space-mark tab-mark newline-mark )
    whitespace-display-mappings '(
      (space-mark   ?\     [?\u00B7]     [?.])
      (space-mark   ?\xA0  [?\u00A4]     [?_])
      (newline-mark ?\n    [?¬ ?\n])
      (tab-mark     ?\t    [?\u00BB ?\t] [?\\ ?\t]))))

(global-whitespace-mode +1)

But every time I open any file in Doom, whitespace-mode is nil.
I also provide whitespace-enable-predicate var here (I did not set it tho), just in case it might be useful:
#[128 "\300\301\"\205\0\300\302\"\207"
      [apply doom-is-childframes-p
             #[0 "\303=\204$\0<\203 \0\242\304=\203\0	A>?\202!\0	>\202!\0\305\205A\0\n?\205A\0\306 \307H\310=?\205A\0\306 \307H\311=?\206A\0\306 \312\230\207"
                 [whitespace-global-modes major-mode noninteractive t not nil buffer-name 0 32 42 "*scratch*"]
                 2]
             nil]
      4 nil]

Steps to reproduce

Put the above config lines in config.el
Reopen Doom
Open any file
whitespace-mode should be nil and global-whitespace-mode should be t

System Information
https://pastebin.com/0fsnDTfG
Notes
Might be related to https://github.com/hlissner/doom-emacs/issues/5854, but the OP did not provide enough information.

As it turns out, global-whitespace-mode and whitespace-mode operate independently. The global mode bypasses the local mode and activates its underlying machinery directly, so testing whitespace-mode for a non-nil value won't actually tell you if it is in effect or not.
That said, I also discovered that my childframe predicate function was incorrectly negated, which prevents global-whitespace-mode from actually kicking in anywhere but child frames. I've fixed that in eefa0ec, which should resolve this issue (and #5854, for that matter).
Let me know if that isn't the case and I'll reopen this.

I can confirm that commit fixes this issue. Thank you for your hard work!


Devices count on site page does not include unracked devices

Issue type: Bug Report

Python version: Python 2.7.10
NetBox version: 1.9.0-r1

It appears that the devices count on site page does not include unracked devices, the following screencap lists 0 devices:

But then clicking on the devices count lists the unracked devices, as follows:

Seems to an issue with the way stats.device_count is calculated?
Thanks for any help you can provide.
Kind regards,
Matt

Looks like I missed some ORM references while implementing #198. Easy fix.


fix: Add common labels and annotations to pod spec inside deployment object

Description of the change
Add the labels defined in commonLabels to pods managed by the deployment object.
Existing or Associated Issue(s)
Fixes #110
Additional Information
n/a
Checklist

[x] Chart version bumped in Chart.yaml according to semver.
[x] Variables are documented in the values.yaml and added to the README.md. The helm-docs utility can be used to generate the necessary content. Use helm-docs --dry-run to preview the content.
[x] JSON Schema generated.
[x] List tests pass for Chart using the Chart Testing tool and the ct lint command.

WDYT about extending this to include commonAnnotations as well? Do you think it would be useful?

Frankly, I don't know if common annotations in pods would be useful. But since they are just annotations and already included in a lot of the other manifests, why not in the pod manifest as well? I added them to the pod spec now.

You need to regenerate README.md (update the version). It can be done by executing pre-commit run -a or helm-docs --chart-search-root=charts --template-files=./_templates.gotmpl --template-files=README.md.gotmpl

Done.

In order to merge this PR we will need you to update your commit with Developer Certificate of Origin (can be done by git commit -s option) and have it GPG signed

Done.

@vinzscam @tumido Are there any other changes you would like to see in this PR?


README.md refs va-v4

PR summary
docs: update README.md with links to va-v4 apidoc
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[x] Documentation content changes
[ ] Other (please describe)

What is the current behavior?
README links to deprecated apidoc
What is the new behavior?
README links to current apidoc
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
release unnecessary

:tada: This PR is included in version 1.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


iam service and region

iamClient := iam.New(&aws.Config{Region: "us-west-2"})
req := &iam.ListUsersInput{}
resp, err := iamClient.ListUsers(req)

always returns the following error:
Could not list IAM Users Credential should be scoped to a valid region, not 'us-west-2'.
However, the following works:
iamClient := iam.New(&aws.Config{Region: "us-east-1"})

This behavior is new with the recent development branch merge. Is it expected?  From what I read, IAM should be region agnostic.

@pdalinis the issue here is that we are generating the global endpoint but signing with the us-west-2 region. I'll take a look at this, thanks for reporting!

Thanks for reporting @pdalinis. The above commit should resolve incorrect credential scope errors when signing requests for services with global endpoints.

Confirmed it is working.  Thanks!


TypeError: Cannot read property 'text' of undefined

When trying out your example from the blog I get this error:
$ node index.js
/Users/jparnow/Documents/Unicorns/node_modules/scrape-it/lib/index.js:155
                    let value = typpy(cOpt.how, Function) ? cOpt.how($elm) : $elm[cOpt.how]();
                                                                                ^

TypeError: Cannot read property 'text' of undefined
    at iterateObj (/Users/jparnow/Documents/Unicorns/node_modules/scrape-it/lib/index.js:155:82)
    at iterateObject (/Users/jparnow/Documents/Unicorns/node_modules/iterate-object/lib/index.js:25:17)
    at handleDataObj (/Users/jparnow/Documents/Unicorns/node_modules/scrape-it/lib/index.js:119:13)
    at Function.scrapeIt.scrapeHTML (/Users/jparnow/Documents/Unicorns/node_modules/scrape-it/lib/index.js:172:12)
    at req (/Users/jparnow/Documents/Unicorns/node_modules/scrape-it/lib/index.js:28:27)
    at Tinyreq (/Users/jparnow/Documents/Unicorns/node_modules/cheerio-req/lib/index.js:21:9)
    at opt_callback (/Users/jparnow/Documents/Unicorns/node_modules/tinyreq/lib/index.js:57:9)
    at IncomingMessage.res.on.on.on (/Users/jparnow/Documents/Unicorns/node_modules/tinyreq/lib/index.js:89:13)

I’m using Node v6.0.0
It would be great if this could be fixed.

Well, actually the example was wrong (related discussion: #15). Sorry for that.

Thank you very much!

@z3to In #23 I added an additional fix to this: an error explicitly saying you have to pass the listItem, selector or have a context. :ship: 3.1.0 :tada:


Improved wgpu error handling, no more crashes through wgpu validation errors

What

Fixes #4457
solves the crash part of #4455

but keeping it open because I still want to check on the error message

There's a bunch of things that probably should have been separate PRs but by the time I ventured there it got too hard to entangle, so please bear with me and read this detailed list of changes carefully (if you understand this list you've done half the review ;-)):

re_renderer now uses cfg_aliases and cfg_if in various places to make cfg decisions more readable
ErrorTracker is now used in all build configurations. Previously, it was only active on debug native builds
The first thing RendererContext does now is to install global error handlers that feed into the "top level" ErrorTracker. Previously this was done delayed and only for debug+native
ErrorTracker never panics now
ErrorTracker no longer uses a frame counting heuristic (!!) to discard all errors, but instead relies on precise device timeline frame counters to discard individual errors

this fixes issues where previously an error would disappear and reappear without being logged
as commented several times in the code: Keep in mind the three staggered timelines of WebGPU

Content (your code)
Device (everything happening on wgpu::Device)
Queue (everything happening on the GPU, represented by wgpu::Queue)

Content and Device are mostly in sync on wgpu-core, but NOT on WebGPU in general!

split out the wgpu-core specific parts of ErrorTracker into its own module and fall back to simple string hashing for WebGPU

on WebGPU we're dealing with a JS interface that doesn't provide these types. Additionally, in a pure WebGPU build
(the only webgpu enabled build we have today), we don't even depend on wgpu-core!
Manual test of re_renderer samples shows that we now no longer crash on WebGPU either!

the wgpu-core error type handling is unchanged except for a change in label hashing

previously, in the absence of debug labels, error de-duplication would fail. In practice leading to error spam on release builds

Introduce  WgpuErrorScope utility for safe & fully covering wgpu error scopes

The resulting futures are handled with now_or_never on wgpu-core backend since we know this is safe. On WebGPU we hand off execution to the browser.

Each frame now has a top level WgpuErrorScope in order to catch all errors

this is not a strict requirement on wgpu-core backends since wgpu-core will always feed the fallback error handler. But Chrome/Dawn WebGPU does not. The spec allows for this behavior

Screenshot of a viewer in release mode with an intentionally broken mesh shader. Also tried other breakages like render pipeline format mismatches.
Things related, but NOT covered by this PR:

crashes on startup / adapter selection

this requires further improvements to egui: We need to improve the hooks that go into creating the egui render state. Need to consider allowing to create your own renderstate. This would also allow us to move the actual device/queue/adapter ownership to re_renderer

#4507

punting on this one. It's a nice-to-have extension of the work presented here

close look into #4455

the crash is almost certainly fixed now and replaced by a recoverable error 🥳, but I want to understand what sets the wrong viewport - either the one in egui we already fixed or the ViewBuilder (which surely could use some safety check)

Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[ ] I have tested the web demo (if applicable):

Full build: app.rerun.io
Partial build: app.rerun.io - Useful for quick testing when changes do not affect examples in any way

[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG

PR Build Summary
Docs preview
Examples preview
Recent benchmark results
Wasm size tracking

originally called frame_index_for_uncaptured_errors instead frame_index_device_timeline. But opted out of that since that's a bit too much of a claim & confusing as well and it's really only for that uncaptured_error callback 🤔


feat: add dataframe na functions

Codecov Report

Merging #169 (69d5cdf) into master (91b5631) will decrease coverage by 6.65%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #169      +/-   ##
==========================================
- Coverage   87.45%   80.80%   -6.64%
==========================================
  Files          23       24       +1
  Lines         231      250      +19
  Branches        7        1       -6
==========================================
  Hits          202      202
- Misses         29       48      +19

Impacted Files
Coverage Δ

.../spark/internal/codegen/DataFrameNaFunctions.scala
0.00% <0.00%> (ø)

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 91b5631...69d5cdf. Read the comment docs.


Make the matchesGoldenFile docs link to an explanation of how to create golden image files

Description
Make the matchesGoldenFile docs link to an explanation of using
flutter test --update-goldens.  Note that I'm linking to a page
that's specifically about updating the golden image files for
package:flutter, so this might not be the most appropriate, but
it's the best I could find.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

We should just put whatever documentation you think would be useful in the API docs directly, IMHO. No point linking elsewhere (especially if nowhere better exists currently).

(If we did add docs elsewhere, it should be on our Web site. The wiki is not something we should be pointing developers to, it's for contributors like ourselves.)


AJAX + Forward plugin return 500 Internal Server Error

I try to call chain of forward plugin in my action through ajax request for get only needed html content but in response return 500 error (in logs nothing).
I have two actions
public function oneAction()
{
	$request = $this->getRequest();
	$route = $this->getEvent()->getRouteMatch();
	$params = $route->getParams();

	$view = (new ViewModel())->setTerminal($request->isXmlHttpRequest());

	$params['action'] = 'two';
	$viewVm = $this->forward()->dispatch($route->getParam('controller'), $params);
	$view->addChild($viewVm, $params['action']);

	return $view;
}

public function twoAction()
{
	$request = $this->getRequest();
	$view = (new ViewModel())->setTerminal($request->isXmlHttpRequest());

	return $view;
}

and relative templates
// module/controller/one.phtml

Test one

// module/controller/two.phtml

Test two

This is how I send ajax request through firefox http://screencast.com/t/dYwuyPDm

Not sure if this bug is reported in proper repository. It is caused by:
https://github.com/zendframework/zend-view/blob/master/src/View.php#L232-L234
(when child model is terminated - setTerminal(true)).
When you comment out above lines in Zend\View\View.php it's gonna work as expected.
I couldn't find anything interesting in tests to understand why we need throw an exception in that case - there is only one test:
https://github.com/zendframework/zend-view/blob/master/test/ViewTest.php#L156-L169
After removing that condition all other tests pass, so I don't know what is the use case of this code.

@webimpress
I report this is general zendframwork repostitory but I asked move this issue here

when child model is terminated - setTerminal(true)

A model marked as "terminal" means that it should be rendered by itself, not as a child of another view model. What that means is that somewhere along the line, a view model was marked as terminal, but then added as a child to another view model.
And, in fact, we can see that above, as both actions do the following:
$view = (new ViewModel())->setTerminal($request->isXmlHttpRequest());

And in oneAction(), you then also have:
$view->addChild($viewVm, $params['action']);

after forwarding to twoAction().
I'm not entirely sure what you're trying to accomplish; is the final view supposed to contain the markup from both view scripts? or only one?

If you only want the child, you would just return the view it returns.
If you want both, you should not mark the child view model as terminal, and you will need to render it within the context of the parent view script (using echo $this->two;, as the rendered content will be captured to a value matching the child action, in this case).


Detection of Chant Mode

Description
We want to recognize the mode of the chant if there is any semitonal information available in the notation. We can detect this by looking at the finalis and the placement of the rhomboidal punctum.
Modes (Aquitanian notation)

Referecing the ECHOES - Project Report 2023

1st mode

The last note(D) is placed two steps below the line (the note on the line is F).
The rhomboidal punctum can be found exclusively one step below the line (E) or three steps above the line (B)

2nd mode

The last note (D) is placed on the line.
The rhomboidal punctum can be found exclusively one step above the line (E) or two steps below the line (B)

3rd mode

The last note (E) is placed two steps below the line (the note on the line is G).
The rhomboidal punctum can be found exclusively two steps below the line (E) or two steps above the line (B)

4th mode

The last note (E) is placed one step below the line (F).
The rhomboidal punctum can be found exclusively one step below the line (E) or three steps above the line (B)

4th mode, special case (showing greater antiquity)

The last note (E) placed on the line.
The rhomboidal punctum can be found exclusively on the line (E) or four steps above the line (B), or three steps below the line (B).

5th mode

The last note (F) is placed two steps below the line (A).
The rhomboidal punctum can be found exclusively one step above the line (B) or three steps below the line (E)

6th mode

The last note (F) is placed on the line.
The rhomboidal punctum can be found exclusively one step below the line (E) or three steps above the line (B)
It is rare, but occasionally in the 6th mode the rhomboidal punctum is used for the pitch A. The rhomboidal punctum signals the lower note of the semitone and, in this case, hints that the B is flat. In other words, the distance between A and B is a semitone. In the 6th mode the repercussio (=most repeated note) is A.

7th mode

The last note (G) is placed two steps below the line (B).
The rhomboidal punctum can be found exclusively on the line (B) or three steps above the line (E)

8th mode

The last note (G) is placed on the line.
The rhomboidal punctum can be found exclusively two step above the line (B) or two steps below the line (B)

Here is the summary of those previous rules in terms of MEI attributes and values:

IF
AND
THEN:
comments

last note
for every nc with tilt="se"

loc = -2
loc = -1   OR  loc = +3
mode = 1

loc = 0
loc = +1  OR  loc = -2
mode = 2

loc = -2
loc = -2   OR  loc = +2
mode = 3

loc = -1
loc = -1   OR  loc = +3
mode = 4

loc = 0
loc = 0    OR  loc = +4  OR  loc = -3
mode = 4
(rare case, earlier chants)

loc = -2
loc = +1  OR  loc = -3
mode = 5

loc = 0
loc = -1   OR  loc = +3
mode = 6

loc = -2
loc = 0    OR  loc = +3
mode = 7

loc = 0
loc = +2  OR  loc = -2
mode = 8

Let me know how this goes :)

Further things to keep in mind (don't worry about these now @notkaramel, these are just to keep in mind after we are done:

It is rare, but occasionally in the 6th mode the rhomboidal punctum is used for the pitch A. The rhomboidal punctum signals the lower note of the semitone and, in this case, hints that the B is flat. In other words, the distance between A and B is a semitone. In the 6th mode the repercussio (=most repeated note) is A.
Liquescence may be occasionally used to signal the lower note of the semitone (that is, E or B).

Would you mind providing a similar summary for the rules for mode detection for square notation, @elsinhadl ? I mean, similar to the way you provided them for the ECHOES report. Thank you!

Update 19/02/2024: I just pushed the feature to detect chant mode for Aquitanian notation.
@martha-thomae I have a question: for example, to statisfy mode 1, can the list of rhomboidal punctums (nc with tilt se) both has loc = -1 and loc = 3? Or is it strictly either only loc = -1 or only loc = 3, exclusively (xor relation).

@martha-thomae I have a question: for example, to statisfy mode 1, can the list of rhomboidal punctums (nc with tilt se) both has loc = -1 and loc = 3? Or is it strictly either only loc = -1 or only loc = 3, exclusively (xor relation).

Hi @notkaramel! It is both, it is an "inclusive or" (one, the other, or both).

@martha-thomae Thank you! I implemented it correctly.

Dear Martha and Antoine,
I have just added on p. 11 in the Project Report
https://docs.google.com/document/d/17bPC6Z9KSgKCxt_50-8Apsu1MAdyA-Pq13Hs2h778-0/edit?usp=sharing
the rules for identifying the mode of chants written in square
notation ("Teaching
the machine how to identify the mode of chants written in square notation").
Please let me know if anything is unclear.
Many thanks,
Elsa
________________________________________________________
PI of *Echoes from the Past: Unveiling a Lost Soundscape with Digital
Analysis
https://sites.google.com/fcsh.unl.pt/elsadeluca/echoes-research-project *
(2022.01957.PTDC)
Recent publication: The Materiality of Sound in ChantManuscripts in the
West
https://www.brepols.net/products/IS-9782503606149-1?fbclid=IwAR2gxS-ucQAfNZEgO9rjGSbBUcpXbSKBkictNRCHw5uqWjfIDCw0oyhYpso,
Brepols 2023
CESEM - Centre for the Study of the Sociology and Aesthetics of Music
IN2PAST – Associate Laboratory for Research and Innovation in Heritage,
Arts, Sustainability and Territory
School of Social Sciences and Humanities
NOVA University of Lisbon
https://sites.google.com/fcsh.unl.pt/elsadeluca/ |
*https://cesem.fcsh.unl.pt/en/pessoa/elsa-de-luca-2/
https://cesem.fcsh.unl.pt/en/pessoa/elsa-de-luca-2/ *
Antoine Phan @.***> escreveu (segunda, 19/02/2024 à(s)
18:50):

@martha-thomae https://github.com/martha-thomae Thank you! I
implemented it correctly.
—
Reply to this email directly, view it on GitHub
https://github.com/ECHOES-from-the-Past/mei-analyser/issues/8#issuecomment-1953018752,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMHC7ZVJURSZTQVMSB2NYZ3YUONIDAVCNFSM6AAAAABC6AYKK2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJTGAYTQNZVGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

There were a few issues found by @elsinhadl, so I made the following modifications on a separate branch:

We were missing a default mode=-1 case when there simply are no rhombuses at all in the chant (https://github.com/ECHOES-from-the-Past/mei-analyser/commit/aa9cc2e5d1d338016a129c62a7858c08e3d03c76)
There was a minor error in some of the flags for the location of rhombuses (the use of an ‘or’ instead of an ‘and’ statement) (https://github.com/ECHOES-from-the-Past/mei-analyser/commit/e2ca0cfded48c86a4afdcf54e4d2914a02b29f47)
I modified the change I just reported on item 1, so that it doesn't detect any mode (it returns mode = -1) not only when there are no rhombuses but also when there is only one rhombus in the whole chant. I made this change because @elsinhadl indicated that one rhombus is not enough to convey mode information (https://github.com/ECHOES-from-the-Past/mei-analyser/commit/e61c79a319fb6964033fd056defc89da9793feae).

I won't attempt to make a pull request until you have finished with the refactoring :)

Thank you @martha-thomae! I realized the logic that I implemented (using filter()) is incorrect and actually producing false results (it just coincidentally happens that most detectable chants are either mode 1 or 6, which is the first if block). There was a default mode=-1 at the very end of each condition blocks in fact, but I prefer your implementation to be honest :)
And yes, I will keep you updated once the refactoring of the interface is completed so we can discuss more about the mode detectioin.

There was a default mode=-1 at the very end of each condition blocks in fact

Yes, I saw this. And I kept this as well for any other cases that do not fall into the previous categories.

I realized the logic that I implemented (using filter()) is incorrect and actually producing false results (it just coincidentally happens that most detectable chants are either mode 1 or 6, which is the first if block).

Yes, I saw that. But it was just a small thing about negations and their interaction with OR / AND connectors. It was fine, the logic was there, it was just the wrong connector (so I added some comments for it to be more clear as well). It is working now. I checked everything against Elsa's table. I am just left with a few questions that I sent over email.

With the changes I detailed here, we have solved the issue of mode detection for the chants in Aquitanian notation. I went through the chants with Elsa and the algorithm detects the right mode for every single one of them (except for on where there is a scribal error, the scribe made a rhombus in the wrong place). So we are all good here. Maybe, for the future, including some extra information about how the chant mode was detected would be good (adding the number of rhombuses for each note, and the value of the finalis).

There were a few issues found by @elsinhadl, so I made the following modifications on a separate branch:

We were missing a default mode=-1 case when there simply are no rhombuses at all in the chant (aa9cc2e)
There was a minor error in some of the flags for the location of rhombuses (the use of an ‘or’ instead of an ‘and’ statement) (e2ca0cf)
I modified the change I just reported on item 1, so that it doesn't detect any mode (it returns mode = -1) not only when there are no rhombuses but also when there is only one rhombus in the whole chant. I made this change because @elsinhadl indicated that one rhombus is not enough to convey mode information (e61c79a).

I won't attempt to make a pull request until you have finished with the refactoring :)

I think we can write up a wiki page for mode detection, using all the information from this issue.


How to Use It?

How To use It?
i m adding this zip file provided by u in eclipse as external library. than trying to to add chipview by programically by i got this error:
The following classes could not be found:

com.plumillonforge.android.chipview.ChipView

i m new in android programming

Hi,
Do you include it as a project library ?
Please read instructions here for example : http://stackoverflow.com/a/9283084/328845

Thank u sir for ur reply
I  used it as an external library in my Android project .but I m not able
to implement chip interface in my project or any other interface...contain
in that zip file
On Jan 4, 2016 1:58 PM, "Flavien Norindr"<EMAIL_ADDRESS>wrote:

Closed #7 https://github.com/Plumillon/ChipView/issues/7.
—
Reply to this email directly or view it on GitHub
https://github.com/Plumillon/ChipView/issues/7#event-504507213.


Create Selenium UI Workflow Tests

Piyush, it would help the team to have these based on this sheet that Seth will give you access to here: https://docs.google.com/spreadsheets/d/1MxpNMTEkhSYptWsJxTy9RdF9kipxYm2jDFzAP-8GhdY/edit#gid=1413317393

Sure thing @ParkeBrown. I'll start working on it and keep you posted with progress. Thanks.

@iPiyushPatel no rush but sometime next week could you call out where you're tracking this?


NPE when retrieving virtual machine images

Retrieving the images from the germany region with beta3 fails with a NPE.
Example:
// Authentication to AzureEnvironment.AZURE_GERMANY
...
for (VirtualMachineImage img : azure.virtualMachineImages().listByRegion(Region.GERMANY_CENTRAL))
{
}

Stacktrace:
java.lang.NullPointerException
    at com.microsoft.azure.management.compute.implementation.ChildListFlattener$1.nextPage(ChildListFlattener.java:76)
    at com.microsoft.azure.PagedList.cachePage(PagedList.java:59)
    at com.microsoft.azure.PagedList.loadNextPage(PagedList.java:98)
    at com.microsoft.azure.PagedList$ListItr.next(PagedList.java:161)

US_WEST works fine from what I can tell.
GERMANY_CENTRAL fails already on listing publishers.
Internally, I see this failure:

A more explanatory error we get after that is:
INFO: {"error":{"code":"NoRegisteredProviderFound","message":"No registered resource provider found for location 'germanycentral' and API version '2016-03-30' for type 'locations/publishers'. The supported api-versions are '2015-05-01-preview, 2015-06-15, 2016-03-30, 2016-04-30-preview'. The supported locations are 'eastus, eastus2, westus, centralus, northcentralus, southcentralus, northeurope, westeurope, eastasia, southeastasia, japaneast, japanwest, australiaeast, australiasoutheast, brazilsouth, southindia, centralindia, westindia, canadacentral, canadaeast, westus2, westcentralus, uksouth, ukwest'."}}

Germany is not listed in the error. Not sure if GERMANY_* is not yet supported by this provider, or if it blocks access unless you have special permissions (accessing only from within Germany), in which case not sure how we would be able to test this..
@jianghaolu, @anuchandy - toughts?

also, @selevt - are you running this code from inside Germany?

@martinsawicki yes, running this from Germany

@jianghaolu, could it be related to the API version?  Looking at the error, I see the SDK is using 2016-03-30, while Germany was added much more recently...

@selevt Do you happen to have a dependency on an older version of client-runtime?

@selevt please confirm the azure-client-runtime version as Jianghao mentioned.
also could you try listing the publishers as mentioned below. We need your help to debug this since our test subscription is not enabled for German regions.

Azure azure = Azure
     .configure()
     .withLogLevel(HttpLoggingInterceptor.Level.BODY)
     .authenticate(credFile)
     .withDefaultSubscription();

for (VirtualMachinePublisher p : azure. azure.virtualMachineImages().publishers().listByRegion(Region.GERMANY_CENTRAL)) {
  System.out.Println(p.Region());
}

The reason for listing publisher is to narrow down the issue you are seeing, listing images in a region involves listing 4 levels of parent resources, but publisher listing is a single level call.
We have logging enabled for the above azure client via  HttpLoggingInterceptor.Level.BODY.
In the log,

Do you see any error?
If no error do you see the json response from the server that contains publisher list , if yes can you check and tell us the content of region property of a publisher?

@selevt I can reproduce it with everything latest. Seems like a bug in the ChildListFlattener. I'm investigating.

Fixed in a pull request. Will let you know when it's merged and released as a snapshot package.

This has been fixed and released since beta4.


Enables the emotion-babel-plugin to work with MUI v5

Having the @emotion/babel-plugin when working with emotion or in this case MUI can be a huge help. It enables features like contextual class names or using components as selectors. I also changed the plugin @vitejs/plugin-react-refresh to the official react plugin for vite.

This looks good to me. I'll pull it and make sure it works. Thanks!


Can't use proxy tools like scraproxy

Scraproxy accepts requests as HTTP but the HTTPS URL must be in the Location header, source:
http://docs.scrapoxy.io/en/master/advanced/understand/index.html#can-scrapoxy-relay-https-requests
go-colly doesn't seem to support this, if the URL is HTTPS and the only proxy available is HTTP, go-colly seems to just skip over it and not use a proxy. I've tested it, and it works perfectly when the website is just HTTP.
Would there be any way to get around this?

Scraproxy accepts requests as HTTP but the HTTPS URL must be in the Location header, source:
http://docs.scrapoxy.io/en/master/advanced/understand/index.html#can-scrapoxy-relay-https-requests
go-colly doesn't seem to support this, if the URL is HTTPS and the only proxy available is HTTP, go-colly seems to just skip over it and not use a proxy. I've tested it, and it works perfectly when the website is just HTTP.
Would there be any way to get around this?

Have you tried SOCKS4/5 to solve this problem?

If you are interested, Scrapoxy 4 is out:
Scrapoxy is a open source proxy aggregator, allowing you to manage all proxies in one place 🎯, rather than
spreading it across multiple scrapers 🕸.
Smartly designed for efficient traffic routing 🔀, Scrapoxy minimizes #bans and boosts success rates 🚀.
The tech stack is built on the latest NodeJS, Typescript, utilizing the NestJS and Angular frameworks.
Here are the key features:

☁ Cloud Providers with easy installation: Scrapoxy supports many cloud providers like AWS, Azure, or GCP.
🌐 Proxy Services: Scrapoxy supports many proxy services like Rayobyte, IPRoyal or Zyte.
💻 Hardware materials: Scrapoxy supports many 4G proxy farms hardware types, like Proxidize or XProxy.io.
📜 Free Proxy Lists: Scrapoxy supports lists of HTTP/HTTPS proxies and SOCKS4/SOCKS5 proxies.
⏰ Timeout free: Scrapoxy only routes traffic to online proxies to avoid inactive connection.
🔄 Auto-Rotate proxies: Scrapoxy automatically changes IP addresses at regular intervals.
🏃 Auto-Scale proxies: Scrapoxy monitors incoming traffic and automatically scales the number of proxies according to your needs.
🍪 Sticky sessions on Browser: Scrapoxy keeps the same IP address for a scraping session, even for browsers.
🚨 Ban management: Scrapoxy injects the name of the proxy into the HTTP responses.
📡 Traffic interception: Scrapoxy intercepts HTTP requests/responses to modify headers, keeping consistency in your scraping stack. It can add session cookies or specific headers like user-agent.
📊 Traffic monitoring: Scrapoxy measures incoming and outgoing traffic to provide an overview of your scraping session.
🌍 Coverage monitoring: Scrapoxy displays the geographic coverage of your proxies to better understand the global distribution of your proxies.
🚀 Easy-to-use and production-ready: Scrapoxy is suitable for both beginners and experts (Kubernetes / Helm).
🔓 Open Source: And of course, Scrapoxy is open source, under the MIT license.

Checkout https://scrapoxy.io/ !


Test python3.11

Close  #528
Checklist before merging

[ ] I added an appropriate entry to CHANGELOG.md
[ ] I merged main into the current branch.

Funny learning: when moving to Python3.11 something changes in the representation of an str,Enum object:
$ cat x.py
from enum import Enum

class MyEnum(str, Enum):
    KEY = "VALUE"

print(f"{MyEnum.KEY}")

$ python3.10 x.py
VALUE

$ python3.11 x.py
MyEnum.KEY


Repository

My Introduction

Name
Ben Langenberg
Class
Tech 3
About me
23 jaar en woont in Zaandam
Goals
Javascript leren en toepassen
Skill level
Beginner
GitHub username
https://github.com/benl95
GitHub repository
https://github.com/benl95/JS-Bootcamp.git

👋 Hi Ben, the link to your repo has been added to the admin sheet. Thanks! 🎉


the user ip should be passed as uip not uid

According to the Google Analytics Measurement Protocol Parameter Reference Guide, the IP override should be sent as 'uip' not as 'uid'.

:+1: thanks @vogelito


Add __dirname to pre-defined variables

See response https://github.com/AnWeber/vscode-httpyac/issues/129

The addition to the doc is great. I have explicitly added a link to the NodeJS Global Object. Thought was that the script behaves exactly like a normal NodeJS module. Only the currently defined variables and a redirection for all console are redirected to VSCode output in the Global Object are added. Thanks.

@AnWeber thx the clarification. yes mentioning it explicit helps JS newbies like me a lot

@rngtng that's why I find documentation so hard. Unfortunately, you still have your own point of view on. Therefore just ask if something is unclear


autoguess-tool install problem

Dear author.
I don't know if it's my problem.
I followed the instructions you gave to install the program. But two commands from your tool are not executed. I have attached the results of running these commands for you.
error.txt
Best wishes to you!

Hi Hamed,
Thanks for your interest in our tool. It seems you could not manage to install some solvers completely, e.g., Or-Tools, and Cadical. That is why you are getting these errors in the run time.
Try to reinstall the tool. But this time, please ensure you can successfully perform the instructions in the installation phase.
Hope that my comment helps! But, if you get any error during the installation again, please let me know the error in detail, e.g., which tool can not be installed based on our instructions and which OS you are using.

Prerequisite tools installed correctly. I have attached the results of reinstalling the tools for you...
install.txt

Hello Hamed,
Thanks for your useful feedback!
I tried to reinstall the tool and reproduce the results to figure out what is the problem.
I noticed that this kind of errors appear only if you run the tool with sudo or root privilege.
It is generally recommended to avoid running Python tools with sudo unless it's necessary.
For Autoguess you essentially do not need to use sudo.
If you have put the source file in a directory that requires a root privilege, you can simply move it to somwhere else where you do not need root privilege and then run the tool without sudo command.
I hope my comment helps and you can successfully run the tool. Feel free to report here if this solution does not solve the issue.
Thanks once more for your useful feedback :-)

Unfortunately, I could not install your program with method 3.
I installed dockerfile with sudo command. Only one command is not executed...
===========================================================================
sudo docker run --rm -it autoguess
root@fa283a6091f5:/home/tools/autoguess# python3 autoguess.py --inputfile ciphers/Example1/relationfile.txt --solver sat --maxguess 2 --maxsteps 5
Traceback (most recent call last):
File "/home/tools/autoguess/autoguess.py", line 211, in
main()
File "/home/tools/autoguess/autoguess.py", line 208, in main
startsearch(params)
File "/home/tools/autoguess/autoguess.py", line 26, in startsearch
search.search_using_sat(tool_parameters)
File "/home/tools/autoguess/core/search.py", line 78, in search_using_sat
gdsat = ReduceGDtoSAT(inputfile_name=parameters['inputfile'],
File "/home/tools/autoguess/core/gdsat.py", line 48, in init
self.supported_sat_solvers = list(solvers.SolverNames.cadical) + list(solvers.SolverNames.glucose4) + list(solvers.SolverNames.glucose3) + list(solvers.SolverNames.lingeling) + list(solvers.SolverNames.maplesat) + list(
AttributeError: type object 'SolverNames' has no attribute 'cadical'


First training run for v1

We are reserving capacity for next week to prepare the first training run

We have an initial model trained on 14M NAIP chips with the v1 architecture.


ENH: Expose v1 dataset overwrite

Closes #839

I think that's somewhat incomplete? There's no easy way of using that additional parameter. It would make sense to add overwrite in the DatasetConfig class.

Ah okay, didn't realise we wanted it exposed all the way up to the runner.py call!

Could also be good info about that flag to add that to the datasets.md docu file


Add a bug reporting functionality

Create a website/webpage which records information as requested below;
If you have other problems like the sound being quiet or the display being wrong in anyway, or if you can't get the stream to work, please send me an email with the following details;
* Browser Type (Chrome, Safari, IE, etc)
* Exact Browser Version (a screenshot of the about page would be awesome!)
* Operating System - as much detail as possible (IE Ubuntu Lucid 64bit)
* The speed and type of Internet connection you have. (IE ADSL, Cable, ADSL2 and 256k, 6M). You can find this information in your ADSL model.
* Please go to http://www.speedtest.net/ and send me the numbers the widget reports.
* Anything else you think is important.

This is meant to be used by people watching a live video stream.  They may be technical, but we can't rely on that, and really we want the process to be as automated as possible.  The more work the user has to do, the less they will use it.

When live events are streamed (like a conference) we will hear reports later of problems.  Generally that is too late and we won't know what the problem was, so we don't know what to fix so it doesn't happen again.   It would be nice if there was a system to capture the details.

@mithro @aps-sids I think this is good to be deployed. I didn't find any obvious bugs. The UI definitely needs to be improved and I've that on my priority list.


BUG: @font-face of Source Sans Pro

The current @font-face definition for Source Sans Pro seem wrong. Hits you when the font is installed as system font. For example: "Italic" is missing here:

Done with https://github.com/TYPO3-Documentation/t3SphinxThemeRtd/commit/842776346ec4707c55a1eec0f0561175fd77fd3e


Date format for relog.exe isn't accurate

For starters, it says "date and time must be in this exact format M/D/YYYYHH:MM:SS" when that is clearly only for US-style date locales.
Further, relog wasn't working for me because in its command line help it was quoting what I'd set in Windows 10 as the Short Date Format (because you have to do that to change the date format you see for the clock in the taskbar - a bug for someone else!?).
So relog printed this in its help: -b <ddd, d/MM/yyyy h:mm:ss[AM|PM]>
But gave "Invalid syntax: -b <ddd, d/MM/yyyy h:mm:ss[AM|PM]>" errors until I went and removed the ddd from the short date format. What's the point of quoting the exact short date format if you can't parse it!?
Anyway, notes to this effect and some examples wouldn't go amiss. Most of the world doesn't live in the USA.
Cheers,
Nick.

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

ID: 88c7179b-7544-073e-c4d9-0f88db2a4ef3
Version Independent ID: 7c71336f-7074-dceb-448d-f3d4d7aa377d
Content: relog
Content Source: WindowsServerDocs/administration/windows-commands/relog.md
Service: unspecified
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

@sicklittlemonkey : Sorry for picking up on this ticket so long after you first posted this. Looks like this could be a software related bug issue, at least that is the most likely scenario.
From what I have seen in most other tickets about software issues, not directly aiming at improving the document page, it is suggested to use the Feedback app to contact the Microsoft Developer team. If you still find that your issue needs to be called out in the documentation, please state which parts of the documentation needs to be changed by suggesting improvement details.
Just to add some more information based on my experience from interacting with tickets like this one:

Wish I could help, but this repository is not intended for regular support issues, support requests or support questions.
You may want to look elsewhere for answers, unless you have suggestions for improving one or more specific pages in the MS Docs repository.
Disclaimer: I am not affiliated with Microsoft or the Microsoft Docs teams here on GitHub.
The intention of this repository is to improve on the existing documentation by providing the users a way to report factual errors, improvement suggestions and request updates for new versions.
The MS Docs team members might not have time to answer your question, both due to the small size of the team and the big workload.
Try the following resources to get in touch with users who may be able to help you:

TechNet forums: https://social.technet.microsoft.com/Forums/en-US/home
MSDN: https://social.msdn.microsoft.com/Forums/en-US/home
Microsoft Community: https://answers.microsoft.com/

Seems to be resolved in commit https://github.com/MicrosoftDocs/windowsserverdocs/commit/ce6a687b3a8a71a87c9bb6257b3d575664c92747
#please-close


Autosuggestion is not invalidated when typing a * or ?

I'm having this problem with 0.4.0, but not 0.3.3: when the typed command includes a * or ?, the autosuggestion is not invalidated and starts crawling to the right: https://ludios.org/tmp/zsh-autosuggestions-async-issue.mp4
# zsh --version
zsh 5.1.1 (x86_64-ubuntu-linux-gnu)

.zshrc settings: https://gist.github.com/ivan/79de5e87210e8cf21e305bb4c30c4360#file-zshrc-L200-L206
(Originally reported at https://github.com/zsh-users/zsh-autosuggestions/pull/180#issuecomment-275911411.)

Only occurs with match_prev_cmd strategy.

I believe this is fixed on develop. Please try it out @ivan


Could not load type 'NSwag.AspNetCore.SwaggerUi3Settings' from assembly 'NSwag.AspNetCore, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=c2d88086e098d109'

My application is working fine previously with 13.9.0, After upgrading to 14.0.3, It is giving an error.
Previously we used app.UseSwaggerUi3() and now it is changed to app.UseSwaggerUi()

The issue is fixed after upgrading packages to 14.0.3


No way to set default folder for received shares

For now we should assume that the shares are always arriving in the "/Shared" folder and are not movable.

In reva and OCIS the folder is called /Shares

Some time ago we sorted out the acceptance test suites so that there are scenarios that test sharing with the share_folder set to Shares. When running against oC10, the share_folder in the config gets explicitly set to Shares. When running against oCIS, that is already the default, and the step that "sets the config" does nothing.
There is another set of sharing scenarios that has the oC10 default - share_folder is not set, and received shares appear in the root of the receiving user's file system. Those scenarios are only run with CI against oC10. They are not applicable to oCIS.
At some time, someone will need to decide if the hard-coded Shares folder in oCIS should have a configurable name, or...

In reva and OCIS the folder is called /Shares

Exactly. And there are no plans for now to make this configurable. Please adjust tests and remove from expected failures.

We have already adjusted the test suite so that all the sharing tests are done with a Shares folder for oCIS CI.
There are other tests that check the default oC10 sharing behavior - the received shares appear in the root of the receiver's file system. Those are all tagged notToImplementOnOcis
There should not be any tests in expected-failures related to this. @dpakach please get someone to double-check.

I found this from https://github.com/owncloud/ocis/blob/master/tests/acceptance/expected-failures-API-on-OCIS-storage.md

tests are marked @notToImplementOnOCIS in https://github.com/owncloud/core/pull/39117 so closing this issue


[Feature Request] Add EventSub webhook revocation

Somehow the revocation message type is missing. Would be nice if we can test the handling of those too!
https://dev.twitch.tv/docs/eventsub/handling-webhook-events#revoking-your-subscription

Added in #169 as --subscription-status flag on twitch event trigger.
However I'm leaving this open as an improvement request, since it should be moved to its own command, similar to twitch event verify

Debated over this when implementing #217 and decided the current system works best. Given the need to send over multiple transports (webhook, websocket) it wouldn't fit well. May re-evaluate in the future.


Make Training Portion of May 6th Bi-Weekly Available

Make Training Portion (last half) of May 6th Bi-Weekly Available for reply.
Can you also make a recommendation of where this replay should be posted? FINOS.org page? Docs site?

hi @brooklynrob @maoo please find the recording here https://www.finos.org/hubfs/FINOS/2020 Video Uploads/FINOS - Open Source Fundamentals - Part 1 - 06 May 20.mp4
huge shoutout to Grizz @grizzwolf for the great work. Please let me know if you have any comments

Great content! No strong opinions on where this should go, but I guess either www.finos.org or alloy.finos.org (one of the two). I'll defer to @aitana16 and @grizzwolf

The recording was posted to https://github.com/finos/alloy#1-review-the-existing-materials, closing this issue now.


tooltip z-index

Hello!
When I add tooltips to the elements in the dialog, they are not visible. z-index values are the same. Increase the value for the tooltip.
// Tooltip base styles
.tooltip {
position: absolute;
z-index: 9991; <<<<<<<<<<<<<<<<<<<<<
border-radius: $base-round;
opacity: 0;
overflow: hidden;
pointer-events: none;
}

Hi! Thanks for reporting. We'll try to rework the z-index management to avoid those errors, but for now I'll just increase the z-index of the tooltips.

LumX now has a depth management service.


error in "history.tcl" when compiled with tcl_readline

when compiled with tcl_readline
openroad xxx.tcl (I ran examples from the test folder)
it ran well, but it shows "application-specific initialization failed: error while autoloading "history": Error: history.tcl, 330 invoked "return" outside of a proc." at the end
I dig into it. It is because of tcl_readline.
remove tcl_readline form the configure and no more error.
btw:
example jpeg_sky130hd.tcl terminated due to
Design area 8883380 u^2 100% utilization.
if { [sta::worst_slack -max] < $setup_slack_limit } {
fail "setup slack limit exceeded [format %.2f [sta::worst_slack -max]] < $setup_slack_limit"
}
fail setup slack limit exceeded -5.24 < -1.5

I regularly build with tcl readline without any issue so it must be related to your setup.  Can you try it with a docker build?

I regularly build with tcl readline without any issue so it must be related to your setup. Can you try it with a docker build?

The docker version is fine. I tested it before.
The new build is local on a Ubuntu. It seems that history.tcl has some problems only on Ubuntu/Debian. I will to to see if I can find the root cause.

@yichungchen84 any further update?  If not we should close this.

@maliberty I actually see this on my Ubuntu 20 machine as well.

I can't reproduce this using Ubuntu on WSL2
% lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description: Ubuntu 20.04.2 LTS
Release: 20.04
Codename: focal
@rovinski since you can would you debug this a bit?

@maliberty I'm not really sure how to start tracking this down. I don't know where the history.tcl file exists. I do know that this only shows up for me when running an interactive shell - it doesn't happen when loading a script.
Here's the stack trace(s) from when the error is printed (breaking on syscalls to write):
Breakpoint 1, __libc_write (fd=1, buf=0x5555580cf838, nbytes=44) at ../sysdeps/unix/sysv/linux/write.c:25
25      ../sysdeps/unix/sysv/linux/write.c: No such file or directory.
(gdb) backtrace
#0  __libc_write (fd=1, buf=0x5555580cf838, nbytes=44) at ../sysdeps/unix/sysv/linux/write.c:25
#1  0x00007ffff7f30967 in ?? () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#2  0x00005555559ef7e6 in sta::ReportTcl::printConsole(char const*, unsigned long) ()
#3  0x00005555558d1e84 in sta::dbStaReport::printString(char const*, unsigned long) ()
#4  0x00005555559ef5d4 in encapOutputProc ()
#5  0x00007ffff7ed4140 in ?? () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#6  0x00007ffff7ed5bf8 in ?? () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#7  0x00007ffff7ed55ad in Tcl_WriteChars () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#8  0x00007ffff7ef01e2 in Tcl_MainEx () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#9  0x000055555573a8eb in main ()
(gdb) c
Continuing.
application-specific initialization failed:
Breakpoint 1, __libc_write (fd=1, buf=0x5555580cf838, nbytes=94) at ../sysdeps/unix/sysv/linux/write.c:25
25      in ../sysdeps/unix/sysv/linux/write.c
(gdb) backtrace
#0  __libc_write (fd=1, buf=0x5555580cf838, nbytes=94) at ../sysdeps/unix/sysv/linux/write.c:25
#1  0x00007ffff7f30967 in ?? () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#2  0x00005555559ef7e6 in sta::ReportTcl::printConsole(char const*, unsigned long) ()
#3  0x00005555558d1e84 in sta::dbStaReport::printString(char const*, unsigned long) ()
#4  0x00005555559ef5d4 in encapOutputProc ()
#5  0x00007ffff7ed4140 in ?? () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#6  0x00007ffff7ed5bf8 in ?? () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#7  0x00007ffff7ed5695 in Tcl_WriteObj () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#8  0x00007ffff7ef01f5 in Tcl_MainEx () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#9  0x000055555573a8eb in main ()
(gdb) c
Continuing.
error while autoloading "history": Error: history.tcl, 330 invoked "return" outside of a proc.
Breakpoint 1, __GI___libc_write (fd=1, buf=0x555557b0fa70, nbytes=1) at ../sysdeps/unix/sysv/linux/write.c:25
25      ../sysdeps/unix/sysv/linux/write.c: No such file or directory.
(gdb) backtrace
#0  __GI___libc_write (fd=1, buf=0x555557b0fa70, nbytes=1) at ../sysdeps/unix/sysv/linux/write.c:25
#1  0x00007ffff619000d in _IO_new_file_write (f=0x7ffff62ea6a0 <_IO_2_1_stdout_>, data=0x555557b0fa70, n=1) at fileops.c:1176
#2  0x00007ffff6191ad1 in new_do_write (to_do=1, data=0x555557b0fa70 "\nomponents of this program may be licensed under more restrictive licenses which must be honored.\nd\026d\027\204", fp=0x7ffff62ea6a0 <_IO_2_1_stdout_>)
    at libioP.h:948
#3  _IO_new_do_write (to_do=1, data=0x555557b0fa70 "\nomponents of this program may be licensed under more restrictive licenses which must be honored.\nd\026d\027\204", fp=0x7ffff62ea6a0 <_IO_2_1_stdout_>) at fileops.c:426
#4  _IO_new_do_write (fp=0x7ffff62ea6a0 <_IO_2_1_stdout_>, data=0x555557b0fa70 "\nomponents of this program may be licensed under more restrictive licenses which must be honored.\nd\026d\027\204", to_do=1) at fileops.c:423
#5  0x00007ffff6190835 in _IO_new_file_xsputn (n=1, data=<optimized out>, f=<optimized out>) at libioP.h:948
#6  _IO_new_file_xsputn (f=0x7ffff62ea6a0 <_IO_2_1_stdout_>, data=<optimized out>, n=1) at fileops.c:1197
#7  0x00007ffff6184541 in __GI__IO_fwrite (buf=0x7fffffffda60, size=1, count=1, fp=0x7ffff62ea6a0 <_IO_2_1_stdout_>) at libioP.h:948
#8  0x00005555565afd03 in spdlog::sinks::ansicolor_sink<spdlog::details::console_mutex>::log(spdlog::details::log_msg const&) ()
#9  0x000055555659de63 in spdlog::logger::sink_it_(spdlog::details::log_msg const&) ()
#10 0x000055555659d96a in spdlog::logger::log_it_(spdlog::details::log_msg const&, bool, bool) ()
#11 0x00005555558d20f2 in sta::dbStaReport::printString(char const*, unsigned long) ()
#12 0x00005555559ef5d4 in encapOutputProc ()
#13 0x00007ffff7ed4140 in ?? () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#14 0x00007ffff7ed5bf8 in ?? () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#15 0x00007ffff7ed55ad in Tcl_WriteChars () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#16 0x00007ffff7ef0209 in Tcl_MainEx () from /usr/lib/x86_64-linux-gnu/libtcl8.6.so
#17 0x000055555573a8eb in main ()

I'm guessing the ?? are probably important but I don't have debugging information in my libtcl.so

Note that this is similar to #102

@jjcherry56 any ideas?

I have seen issues with history.tcl issues in opensta over the years but frankly they are mysterious. history.tcl is one of the tcl libraries in the tcl install directory next to init.tcl.
I don't use tcl_readline because I run openroad inside emacs that has a real editor. I have never been able to get tcl_readline to compile in my environment.


Add fused matrix vector multiplication

This PR introduces a fused version of matrix multiplication that can be used when the same x vector is reused across multiple different Ws and Outs. This improves the speed by better reusing the x vector when it is in the cache.
Speed for single threaded sees a nice 10% boost. Unfortunately multi-threaded performance saw a 6% decrease on the 42M model. This makes me hesitant to call this a clear win because of this. I have made the macro generate normal matmul calls when OpenMP is enabled so that fused is only used in single threaded mode.
This idea was inspired by the work of Foundation42 in #94 and based off the fused matrix multiplication found in llama2.zig I had done previously.
The use of macros in C is quite ugly, if it does not fit in this repo then let it be an example for others to learn from and improve upon. I have tried to keep the code changes to a minimum.
Make commands
.PHONY: runomp
runomp: run.c
	$(CC) -Ofast -fopenmp -march=native run.c  -lm  -o run

.PHONY: runfast
runfast: run.c
	$(CC) -Ofast -o run run.c -lm -march=native

Results
Taken by averaging the shown runs below. CPU is an AMD 5900x.

model
command
master token/s
fused token/s
% increase

stories15M
runfast
508
563
10.8%

stories15M
runomp
2243
2589
15.4%

stories42M
runfast
184
207
12.5%

stories42M
runomp
336
314
-6.5%

Runfast Fused
❯ while true;  ./runfast_master stories15M.bin -t 0 | rg "tok" ; end
achieved tok/s: 504.566210
achieved tok/s: 508.045977
achieved tok/s: 505.720824
achieved tok/s: 511.574074
achieved tok/s: 512.761021
achieved tok/s: 508.045977
achieved tok/s: 516.355140
achieved tok/s: 510.392610
achieved tok/s: 515.151515
achieved tok/s: 502.272727
achieved tok/s: 510.392610
achieved tok/s: 498.871332
achieved tok/s: 512.761021
achieved tok/s: 501.133787
achieved tok/s: 509.216590
achieved tok/s: 502.272727

awk to get average:
echo "..." | awk '{ total += $3; count++ } END { print total/count }'

Runfast Master
❯ while true; ./runfast_fused stories15M.bin -t 0 | rg "tok" ; end
achieved tok/s: 572.538860
achieved tok/s: 568.123393
achieved tok/s: 571.059432
achieved tok/s: 563.775510
achieved tok/s: 565.217391
achieved tok/s: 565.217391
achieved tok/s: 556.675063
achieved tok/s: 565.217391
achieved tok/s: 553.884712
achieved tok/s: 571.059432
achieved tok/s: 559.493671
achieved tok/s: 556.675063
achieved tok/s: 565.217391
achieved tok/s: 549.751244

Runomp Master
llama.c-fork master\* 5s
❯ while true; ./runomp_master stories15M.bin -t 0 | rg "tok" ; end
achieved tok/s: 2600.000000
achieved tok/s: 2278.350515
achieved tok/s: 2351.063830
achieved tok/s: 1921.739130
achieved tok/s: 2483.146067
achieved tok/s: 2278.350515
achieved tok/s: 2351.063830
achieved tok/s: 2027.522936
achieved tok/s: 2255.102041
achieved tok/s: 2232.323232
achieved tok/s: 2232.323232
achieved tok/s: 2255.102041
achieved tok/s: 1811.475410
achieved tok/s: 2326.315789
achieved tok/s: 2210.000000
achieved tok/s: 2376.344086
achieved tok/s: 2145.631068

Runomp Fused
llama.c-fork master\*
❯ while true; ./runomp_fused stories15M.bin -t 0 | rg "tok" ; end
achieved tok/s: 2662.650602
achieved tok/s: 2630.952381
achieved tok/s: 2728.395062
achieved tok/s: 2511.363636
achieved tok/s: 2540.229885
achieved tok/s: 2797.468354
achieved tok/s: 2662.650602
achieved tok/s: 2695.121951
achieved tok/s: 2695.121951
achieved tok/s: 2662.650602
achieved tok/s: 2540.229885
achieved tok/s: 2695.121951
achieved tok/s: 2728.395062
achieved tok/s: 2728.395062
achieved tok/s: 2728.395062
achieved tok/s: 1613.138686
achieved tok/s: 2600.000000
achieved tok/s: 2402.173913
achieved tok/s: 2511.363636
achieved tok/s: 2662.650602
achieved tok/s: 2376.344086
achieved tok/s: 2695.121951
achieved tok/s: 2695.121951

Extra data on stories42M.bin
❯ while true;  ./runfast_master stories42M.bin -t 0 | rg "tok" ; end
achieved tok/s: 185.701831
achieved tok/s: 183.620690
achieved tok/s: 184.735473
achieved tok/s: 185.378590
achieved tok/s: 183.147034
achieved tok/s: 182.989691
achieved tok/s: 183.462532

❯ while true;  ./runfast_fused stories42M.bin -t 0 | rg "tok" ; end
achieved tok/s: 204.807692
achieved tok/s: 209.645669
achieved tok/s: 208.414873
achieved tok/s: 209.439528
achieved tok/s: 205.400193
achieved tok/s: 206.595538
achieved tok/s: 209.852217
achieved tok/s: 208.211144
achieved tok/s: 206.997085

 ❯ while true;  ./runomp_master stories42M.bin -t 0 | rg "tok" ; end
achieved tok/s: 359.797297
achieved tok/s: 321.752266
achieved tok/s: 347.471452
achieved tok/s: 357.382550
achieved tok/s: 325.688073
achieved tok/s: 353.233831
achieved tok/s: 333.333333
achieved tok/s: 315.088757
achieved tok/s: 346.905537
achieved tok/s: 308.695652
achieved tok/s: 331.259720

 ❯ while true;  ./runomp_fused stories42M.bin -t 0 | rg "tok" ; end
achieved tok/s: 328.197227
achieved tok/s: 335.433071
achieved tok/s: 309.143687
achieved tok/s: 309.593023
achieved tok/s: 333.333333
achieved tok/s: 309.593023
achieved tok/s: 286.675639
achieved tok/s: 324.200913
achieved tok/s: 299.157303
achieved tok/s: 294.198895
achieved tok/s: 314.623338
achieved tok/s: 334.379906

Cute! I learned something. But yes this is ugly. Thanks for the PR though!


Improve GameBananaHelper.IsOnline

About 45 minutes ago from me making this PR, a user from China using a VPN with TKMM asked for help in the Discord server because they were unable to use the GameBanana browser built into TKMM.
This change fixed their issue!
I noticed the Ping object's Send method had a timeout of one second.
After some thought it occurred to me that VPNs are notoriously slow. Even my own experience using a VPN from not the other side of the world, and I could tell that one second was far too short.
On top of that, I also ping GameBanana instead of google because Google can very easily be blocked in someone's country but GameBanana might not be. Same idea there as #48.
The timeout is up for change, however I figure 10 seconds is a good enough timeout for bad internet + VPN. If your internet is good enough this change effects nothing.

If this property is checked twice it will not check the connection again.
In this case, that's not an issue, but it's how I expect it to behave.
I think IsOnline should be a function named IsGameBananaOnline (my mistake, but I figure it's worth fixing now anyway).

I figured just keeping IsOnline was fine considering it's the GameBananaHelper; GameBananaHelper.IsGameBananaOnline() is kinda hell
IsOnline() then? or maybe just use a property and get rid of the lazy?

If this property is checked twice it will not check the connection again.
In this case, that's not an issue, but it's how I expect it to behave.
I think IsOnline should be a function named IsGameBananaOnline (my mistake, but I figure it's worth fixing now anyway).

I figured just keeping IsOnline was fine considering it's the GameBananaHelper; GameBananaHelper.IsGameBananaOnline() is kinda hell
IsOnline() then? or maybe just use a property and get rid of the lazy?

Either work, I think IsOnline() is good.

If this property is checked twice it will not check the connection again.
In this case, that's not an issue, but it's how I expect it to behave.
I think IsOnline should be a function named IsGameBananaOnline (my mistake, but I figure it's worth fixing now anyway).

I figured just keeping IsOnline was fine considering it's the GameBananaHelper; GameBananaHelper.IsGameBananaOnline() is kinda hell
IsOnline() then? or maybe just use a property and get rid of the lazy?

Either work, I think IsOnline() is good.

Keep lazy or not? I figured I'd make it lazy considering IsOnline is only ever checked in OnFrameworkInitializationCompleted


Update prom querier to be more generic

Changes
PromQuerier now takes in a number of new arguments allowing for example to pass in basic auth if your prom is behind an auth system.
Will now run multiple threads and pick randomly from the queries supplied.
Now run as either command line tool or as a server via flags.
Verification

have ran against our internal systems

@bwplotka when you get a chance could you have another pass of this


feat(ingest/dbt-cloud): use job-based graphql queries

The old type-based ones will be deprecated on Sept 7.
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Merging as FAILED tests/integration/vertica/test_vertica.py looks unrelated


Add PR template

What does this PR do?
Fixes #339
This PR adds a PR template for the Maesh project, and updates the issue template to remove the unnecessary whitespaces at the beginning of each line.
How to test it

Read it and ensure that it makes sense and fits the project
Look out for typos

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Mediapipe

Dear @DavidBoja
Thanks so much for your contributions.
Is there a way to use Mediapipe, to use it for your measurement SMPL anthropometry ? If that's possible what needs to be changed and how ?

As far as I know, Mediapipe works on images of humans, and finds various features from these, like joints, face detection, etc.?
You can use Mediapipe to estimate the joints from a subject in the image, and then use an existing HMR method to estimate the SMPL from the image. Then, you can use the SMPL-Anthropometry to get the measurements.

@DavidBoja But as you mentioned before I need a T or A pose.. this won't work ?

@DavidBoja Thanks so much for the reference github.
The main problem with all that work, Is I want to be able to view the measurement in Unity3D ( A game engine). Not sure how would that happen


Autodiscovered light now failing even with fixed as mode

This entity used to work but after a recent update is failing to create the power entity.
  - platform: powercalc
    entity_id: light.color_temperature_light_1   # Pantry WW/CW LED Strip
    standby_usage: 1.8
    fixed:
      power: 8.8

Logs:
2021-08-30 22:05:35 DEBUG (MainThread) [custom_components.powercalc.model_discovery] Auto discovered Hue model for entity light.color_temperature_light_1: (manufacturer=GLEDOPTO, model=GL-C-006)
2021-08-30 22:05:35 INFO (MainThread) [custom_components.powercalc.sensor] Model not found in library light.color_temperature_light_1: ('Model not supported', 'GL-C-006')

The autodiscovery is correct but the creation should not fail as I am specifying a fixed mode. Same thing if linear is used. The failure should only occur if LUT is specified or nothing is specified.

Thanks for reporting. You are correct, this should only fail in LUT mode. This should have gone broken after a lot of changes and cleanups I made in v0.5.0.
Will have a look into this tomorrow.

Should be resolved with 0.5.1

Yes fixed. Thanks!


fix: SyncProvider does not work after upgrading Jaspr to v0.12.0

Description
After upgrading Jaspr to v0.12.0, I discovered an issue where pages would not render with infinity browser loading when using SyncProvider. I confirmed that if I remove SyncProviderDependencies and preloadDependencies from the component, rendering itself is possible, but there are cases where data from SyncProvider is not rendered properly because preload does not proceed.
Steps To Reproduce

Download the Riverpod example
Upgrade Jaspr packages to v0.12.0 in the project
Serve the app
See infinity loading
Note: In this official example, the app appears to run properly even though I remove SyncProviderDependencies. However, when I actually used it in my projects, I found that in many cases, if SyncProviderDependencies was removed and preload did not proceed, the data was not rendered properly

Doctor Output
[✓] Jaspr CLI (Version 0.12.0)
  • Dart Version 3.3.4 (stable) (Tue Apr 16 19:56:12 2024 +0000) on "macos_arm64" at /opt/homebrew/Caskroom/flutter/3.19.6/flutter/bin/cache/dart-sdk/bin/dart
  • Running on macos Version 14.4.1 (Build 23E224) - Locale ko-KR
  • Analytics: Enabled

[✓] Current Project
  • Dependencies on core packages:
    • jaspr: ^0.12.0
    • jaspr_builder: ^0.12.0 (dev)
    • jaspr_riverpod: ^0.3.11
  • Rendering mode: server
  • Uses jaspr compilers: false
  • Uses flutter embedding: false

Expected Behavior
Pages should be rendered properly with synced data

Thanks I will look into it.