Add LoadRunner parser

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@259f2fa). Click here to learn what that means.
The diff coverage is 82.85%.

@@            Coverage Diff            @@
##             master     #144   +/-   ##
=========================================
  Coverage          ?   68.25%
=========================================
  Files             ?       44
  Lines             ?     3824
  Branches          ?      662
=========================================
  Hits              ?     2610
  Misses            ?      974
  Partials          ?      240

Impacted Files
Coverage Δ

...son/plugins/performance/parsers/ParserFactory.java
63.85% <33.33%> (ø)

...on/plugins/performance/parsers/ParserDetector.java
75.8% <80%> (ø)

.../plugins/performance/parsers/LoadRunnerParser.java
88.88% <88.88%> (ø)

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 259f2fa...66fac82. Read the comment docs.

As you may be aware of there's nothing I can do about the Codacy warning without refactoring base class and all parsers.


pq: syntax error at or near "on"

Hi Brocaar
I've install your server(lora-gateway-bridge, loraserver, lora-app-server)
and all installed successfully
I create a node and set devaddr,nwskey,appskey and abp mode
I test this node by sending some data
but i've some error log :
time="2017-05-08T21:01:40+08:00" level=error msg="handle stats packet error: aggregate gateway stats error: insert or update aggregate error: pq: syntax error at or near "on""
This log sometimes appears,
but I have no idea what problem it is.
Could you help me?

Did you create a separate database for LoRa Server and LoRa App Server? It is probably that, or you forgot to run the database migrations (see --db-automigrate / DB_AUTOMIGRATE config flag).

Also, make sure you installed PostgreSQL 9.5+

Hi Brocaar
I am very sorry for not having to reply to you for a long time
because I am busy with my studies.
I check that I separate database for LoRa Server and LoRa App Server.
And I'm try to upgrade loraserver, lora-app-server component to last version,
the error log(pq: syntax error at or near "on")  no longer appears
But there another log say "type jsonb does not exist" when loraserver try to migration.
I try to install PostgreSQL 9.6 , but not work.
And I find that my system has both PostgreSQL 9.6 and PostgreSQL 9.3.
So I uninstall both of them and install PostgreSQL 9.6 again.
There is no error log!
Thank you brocaar, I resolve my problem!

Hi, guys! I just faced to a similar trouble, but the solution was instructive.
When I had been configuring the postgresql I missed the fact the version must be 9.5+, and then I installed 9.6 without removing old version 9.4. But apparently the new version use the neighbour port, If the old version is still working. I thinked I solve the problem, but the application have been use 9.4 version all the time instead 9.6, and I did'nt know about it. When I faced to this problem (syntax error), I decided to check the version is used now using netstat utility. Pay attention, if anyone faced to this problem, do not forget to delete the packets (postgresql) with configuration files! Otherwise, the newly installed version (e.g. 9.6) will  not use 5432 port (in my case it was 5433).


defaultsTo and required key do not reflect in database schema for postgresql

Node version: 10.16.0
Sails version (sails): 1.2.3
ORM hook version (sails-hook-orm): ^2.1.1
Sockets hook version (sails-hook-sockets): ^2.0.0
DB adapter & version (sails-postgresql): ^1.0.2

I have an existing postgresql database, and I want to replicate the same database using the sails-postgresql. But I am facing the following issues:

defaultsTo value does not reflect in the database schema. (eg. The isDeleted field in the attribute is of type boolean which defaults to false, but the schema does not reflect the same. Simillarly the meta field in the attribute is of type jsonb which defaults to empty object, but no reflection in the db schema)

Ideally, when I add the field required: true, then the db schema should show not null value in the Nullable column in the schema, but it shows nothing. (eg. the name field has required: true but the Nullable column is empty for the same)

The createdAt column is of type ref with column type timestamp(0) with time zone. The requirement is that, it should add the current timestamp (similar as default value NOW() in postgres). But autoUpdatedAt: true is not helping me with the same.

module.exports = {
  schema: true,
  tableName: 'organization',
  attributes: {
    id: {
      type: 'number',
      unique: true,
      columnName: 'id',
      autoIncrement: true
    },
    isDeleted: {
      type: 'boolean',
      defaultsTo: false,
      columnName: 'is_deleted'
    },
    name: {
      type: 'ref',
      columnName: 'name',
      columnType: 'text',
      required: true
    },
    createdAt: {
      type: 'ref',
      columnName: 'created_at',
      columnType: 'timestamp(0) with time zone',
      autoUpdatedAt: true,
    },
    meta: {
      type: 'ref',
      columnName: 'meta',
      columnType: 'jsonb',
      defaultsTo: {},
    },
  }
};

Schema built using sailsjs:

Original existing Schema:

Can you please let me know if I am making some mistake, or sailsjs is designed like this only???

Unfortunately there is quite a disconnect between how Waterline and the underlying database handle your data validations & constraints, with Waterline favouring to do it all itself.
I think for those fields that you defined with type ref you can hint to PG what the default should be, ie
    updatedAt: {
      type: 'ref',
      columnName: 'created_at',
      columnType: 'timestamptz not null default now()',
      autoUpdatedAt: true,
    },
    previousEmailAddresses: {
        type: 'ref',
        columnName: 'previous_email',
        columnType: `text[] default '{}'`, // Default array enforced @ db level
        defaultsTo: [], // Default empty array enforced @ JS level
    },

That previousEmailAddresses is just a PG text array I pulled from a working model of ours but might give you some inspiration. Using the above we end up with something like this:
            Column            |           Type           | Collation | Nullable |             Default
------------------------------+--------------------------+-----------+----------+----------------------------------
 updated_at                   | timestamp with time zone |           | not null | now()
 previous_email               | text[]                   |           |          | '{}'::text[]
...

Buyer beware: It's not The Sails Way, and will probably barf using the local disk adapter in tests / if you change to another underlying ORM / if Waterline changes how it handles these kind of attribute definitions in future.
Also you probably want to use autoCreatedAt: true on your createdAt attribute - it'll set the "now" timestamp in JS land before sending the INSERT to the database, and autoUpdatedAt will do some vooodo when you update an existing record.

Hi, @krisalay—if you're using the safe migration setting (migrate: 'safe—this should be used in production), then changes won't be made to your database when you run sails lift. To apply the changes, you can either set migrate: alter or migrate: drop (again, not for use in production), or you can manually update the database. When we migrate production data, we typically do so with a custom script (check out the relevant docs). For more on migration, see this docs page.


Added so that only the specified user are able to see the edit profile, voice settings and sound settings buttons. 👌

Fix #1936


License of weights

Hi,
first of all, thanks for the nice repository. I was wondering what is the license of the pretrained weights? Does the Apache 2.0 license cover them as well, or are they published under a different license?

Hi @DavHoffmann
Thanks for your kind words. Yes I believe the license covers the weights as well.

Great!
Thanks for the fast reply.


Adding ToShortDateString() and sisters to Date and TimeOfDay.

Added the following convenience methods to Date and TimeOfDay:

Date.ToShortDateString()
Date.ToLongDateString()
TimeOfDay.ToShortTimeString()
TimeOfDay.ToLongDateString()

#16

@mj1856, can you please review the changes I did above and let me know if you are OK with me merging the pull request. Then after that I will update the tests following your change of project to non-PCL.

Can you please rebase to remove the merge commit?
git fetch upstream
git checkout master
git rebase upstream/master
git push --force

Also, in the future, you might want to consider creating a feature branch in your fork for different items, that makes it much cleaner to do separate PRs for different issues simultaneously.

Thanks @mj1856. I changed ToIsoString() to use invariant culture and I rebased the repository as you suggested. I think we are good to merge this PR now, but I will leave this for you.


How can I serve static files?

This is probably not implemented, yet. Can you please advise me on a simple hack?

Hi! You can use either sendFile or sendFileExt. They will call expressjs' sendFile under the hood.

Thx!

@dancingrobot84 Express 4 ships with serve-static, located at express.static. I'd like to make a PR to include an FFI to this - what do you think?

@chexxor That would be good :)


fix mismatch json payload

#5
I think these will be able to work.

Looks like #6 and this are fixing same issue.

I will keep watching on that issue, thx.
But I think this might be a simplest solution depending on your current release.


How can I change the names of the three main tabs of the module?

I'm trying to change the name of the 'general' tab to display not the Content, but the Front Side. Hook works, new tabs are created. But I do not see any changes to the visual builder.
When I try to redefine the slug in the function settings_modal_tabs(), then the field disappears altogether. Tell me, please, are there any ways to change the name and slug of the main tabs?
public function init() {
$this->name = esc_html__( 'Hover Box', 'udp-ultimate-divi-plugins' );
	add_filter('et_builder_main_tabs', 'my_main_tabs');
	function my_main_tabs( $tabs ) {
	$tabs['general'] = esc_html('Front Side', 'udp-ultimate-divi-plugins' );
	return $tabs;
	};

	$this->settings_modal_tabs = array(
		'general' => array(
			'tab_slug' => 'Front Side',
			'name' => esc_html('Front Side', 'udp-ultimate-divi-plugins' ),
		),
	);
}

This array is main_tabs:

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[ ] Detailed description

Description should usually be 150+ characters.

[ ] Steps to reproduce included
[ ] Support Ticket URL included
[ ] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.


CUPS control files missing job-name's

When a single LPR command submits multiple files from CUPS
client to  CUPS print server the control file (c0000x-xxx) that
creates in /var/spool/cups dir  does not contain all the file
names (job-name). But, a data file (d0000x-xxx) for each file is created.
Sample LPR command
lpr -P blane /tmp/sample1.txt /tmp/sample2.txt /tmp/sample3.txt
Checking contents of /var/spool/cups/c00055 CUPS control file only one file
(job-name) sample1.txt exist. Missing are sample2.txt and sample3.txt
cd /var/spool/cups
/var/spool/cups# cat c00055
G attributes-charset utf-8H attributes-natural-language enE printer-uri
"ipp://localhost:631/printers/blaneB job-originating-user-name rootB job-name
sample1.txt I document-format text/plain! copies
All data files were created.
/var/spool/cups# ls
c00055      d00055-001  d00055-002  d00055-003  tmp
/var/spool/cups#

Looks like a duplicate of https://github.com/apple/cups/issues/4805
see in particular Michael Sweet's explanation https://github.com/apple/cups/issues/4805#issuecomment-211061981
I would not expect to much from the autogenerated job title.
I think if you need a particular job title, you must explicity
specify it in the command e.g. something like

lp -d queue_name -t 'file1_file2_file3' file1 file2 file3

Not exact a dupe of issue #4805, but a similar cause.
CUPS supports multiple document jobs but does not support the IPP document object extension which would give you access to the names of individual documents in the job. And in particular the LPD protocol does not provide a way to name individual files in a job, just the overall name/title of the job.

Thanks for your response.
Srikanth
On 04/18/16 20:43, Michael R Sweet wrote:

Not exact a dupe of issue #4805
https://github.com/apple/cups/issues/4805, but a similar cause.
CUPS supports multiple document jobs but does not support the IPP
document object extension which would give you access to the names of
individual documents in the job. And in particular the LPD protocol
does not provide a way to name individual files in a job, just the
overall name/title of the job.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/apple/cups/issues/4806#issuecomment-211423979


Use placeholder loaders for a less jumpy table interaction experience

Because table content is paginated, simply changing the sort order in a column triggers a complete table refresh and displays a loader component while doing it, which creates a bit of a jumpy UX, imho.
To combat this problem, at work we use placeholder loaders such as this one (https://github.com/danilowoz/react-content-loader) , instead of traditional spinning / jumping / crawling loaders.
Supporting this kind of functionality would be a great addition to user experience.
EDIT:  an alternative approach would be to dim the current table rows until the new content is loaded. ( bitbucket uses this solution when you're browsing a paginated list of commits)

loader in list was added in version @1.2.0-beta.4 - btw thanks for the suggestion.


[request] multiple playlists and command to change.

An option to have multiple playlists and a command to change between them.
Add a setting in config to load xxxxxx.txt on bot load, so you can select the default one to load up.
Add command like !apl (AutoPlayList) so you could "!apl rock" to load autoplaylist rock.txt or "!apl 90s" to load playlist called 90s.txt

#43


Bug: Saving synchronized data from more than one device

Description:
Measuring a waveform with two or more devices.  Saving all of those measurements via the saving popup results in the same amount of files as devices were selected.
Bug: Those files have different sizes therefore not the same amount of data points.
Problem:
The OmnAIScopes are synchronized. Therefore when starting a measurement the OmnAIScopes start measuring at the same timestamp. They also stop at the same timestamp. As the sampling rate of both devices is the same, the same amount of data points is measured. For each device one file is created. As they both have the same amount of data points, if all data points that were measured are saved correctly the files will have the same size and the same amount of data points.
Case:
All datapoints are positive and between a range of 20  to 50 on the y-axis.
Reproducing the bug:
Connect two Scopes with USB-C cables to a Hub. Connect this Hub to the laptop. The Scopes are not connected to a voltage source.
Open the software.
Press the Buttons:
Search for devices --> Start (Play symbol button) --> Stop (stop symbol button) --> Save (Save symbol button)
In the opening Popup menu, open the section "Deviceslist(/Geräteliste)"
Select two devices in the drop-down menu --> Press Save (Speichern)

Go into the direction where the Application is saved.
Look in the build/Release/saves/ folder. Here the files should be saved. You should see that both files have a different size.

Files that were created when testing, the second file is always smaller than the first:
device1-2024-07-04T09-59.csv
data points: 81337
device2-2024-07-04T09-59.csv
data points: 67427
Expected Behavior:
Userflow:
--> Measuring a waveform with two or more devices , waveform in range 20  to 50 on the y-axis.
--> Pressing the save button
--> Selecting all devices in the drop down menu
--> Pressing save
Output:

The same amount of files as devices where selected
every file has the same amount of data
every file only saves the data from their corresponding specific device

Test that should work if the files are saved correctly
Connecting two / three / four devices that are not connected to a voltage source.
--> press the start button
--> press the stop button
Expected image :

--> Press the save button
--> select one device
Expected image:

--> Press save
Expected output:

The file should contain every measured data point from the measurement from the selected device.
--> Go back to the application
--> Press the save button again
--> select all connected devices
Expected image :

--> Press save
Expected output for two devices:

All files that are saved should contain the datapoints of their corresponding device. The files should all have the same size, as the same amount of datapoints was measured by each device.
This should work for connecting one or more devices up to 256.
Definition of done:
--> All described tests are fulfilled
--> All variable_names in the new code describe what the purpose of the variable
--> The new code fulfills the c++ standard
--> The new code fulfills the code standard for this project
--> All files that were changed are formatted with the given clang-format
If there are any questions regarding this issue feel free to ask in the comments of the issue.

Which version of the software did result in files with different sizes as shown in the Reproducing the bug section above, please?

The current master version : d0edbc2

https://github.com/skunkforce/OmniView/pull/167
The branch above works based on this and this  tests for the specifications above.


I have an issue with flutter doctor

[!] HTTP Host Availability
X HTTP host "https://pub.dev/" is not reachable. Reason: An error occurred while checking the HTTP host: The semaphore timeout period has expired.

@Steven20201 A semaphore timeout indicates that your operating system failed to connect to a network or retrying to connect. That's the reason you're not able to reach the https://pub.dev host
You can try some options from this link.

@Steven20201 it appears like this is an issue with your local setup, and not with the Flutter SDK.
If you believe your computer is able to reach pub.dev and the problem is with the Flutter SDK, please provide reproduction instructions and the full output of flutter doctor -v.


style(textlint): fix textlinterror

style(textlint): fix textlinterror
close #96

[Current coverage][cc-pull] is 94.90%

Merging [#102][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #102   diff @@
==========================================
  Files             7          7
  Lines          1255       1255
  Methods           0          0
  Messages         64         64
  Branches          0          0
==========================================
  Hits           1191       1191
  Misses           64         64
  Partials          0          0

Powered by Codecov. Last updated by [2df0793...cdf90a1][cc-compare]
[cc-base-branch]: https://codecov.io/gh/azu/JavaScript-Plugin-Architecture/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/azu/JavaScript-Plugin-Architecture/compare/2df079317e0fcbc08953efa867f9e912b65a32de...cdf90a1e0762b09c018655056287f63a4cf8df2f
[cc-pull]: https://codecov.io/gh/azu/JavaScript-Plugin-Architecture/pull/102?src=pr


Custom Titlebar Should Not Be Focusable

It is uncomfortable that we have to click on the editor to be able to type every time we drag the custom titlebar. A titlebar should have been non-focusable without doubt.

https://github.com/Microsoft/vscode/issues/30967


business central export settings model and service constraction

Summary by CodeRabbit

New Features

Introduced new settings and options for business central exports, enhancing customization for export operations.
Added support for additional export types, including journal entries and purchase invoices.

Enhancements

Improved the export settings service with new methods for posting settings and retrieving options.
Updated onboarding logic to better handle export settings for reimbursable and credit card expenses.

Bug Fixes

Adjusted the conditions for displaying the "Disconnect Business Central" option to align with updated export settings.

@coderabbitai review

Comments are fixed in https://github.com/fylein/fyle-integrations-app/pull/343


Use DataAnnotationLocalizerProvider for Display localization

Fixes #5275. By making Display localization use DataAnnotationLocalizerProvider we allow people to use a resource other than the one for the model.

@javiercn I believe you're the next most knowledgeable for localization since @dougbu is unavailable.

@javiercn I don't think so.

I meant @ryanbrandenburg

:up::date:

:up::date:

:watch:

Bah, sorry. Don't know how I managed to leave it a commit behind on github, but it's up there now.


fix: ci

Changes proposed in this pull request

Adds --privileged flag to docker start on tigerbeetle container start
patches pnpm packages where vulnerabilities were found (cherry-picked from https://github.com/interledger/rafiki/pull/2749) to resolve CI failure: Node Build / docker-build (linux/arm64, arm64, frontend) (pull_request) etc

Context

fixes recent ci failures
Seems to come from an upgrades version of docker in github. more context on this issue and getting to the resolution in this slack thread: https://interledgerfoundation.slack.com/archives/C06KZCKEEKW/p1718211242499809
see:

https://github.com/tigerbeetle/tigerbeetle/issues/1860
https://github.com/tigerbeetle/tigerbeetle/pull/1995

Checklist

[ ] Related issues linked using fixes #number
[ ] Tests added/updated
[ ] Documentation added
[ ] Make sure that all checks pass
[ ] Bruno collection updated

LGTM, but will wait also on @mkurapov to do a review.


[API] [BUG] daemonic processes are not allowed to have children

When try to run parsing, this error occured.
daemonic processes are not allowed to have children

https://github.com/pgjones/hypercorn/issues/191
Tried

Hypercorn : Daemon cannot have children issue
Daphne : Task is not finished
Uvicorn : nest_asyncio error (will not work with AutoRAG)

Wow. Just... wow

Just quart.run worked well to me.

uvicorn app:app --host <IP_ADDRESS> --port 5001 --reload --loop asyncio
This worked!! YEAHHHH
uvicorn is the GOD

close at #966


Implement Get User Input

This PR is my take on #52. It utilizes the same example program and test cases.
The test cases are concurrency safe and this can be checked by running: go test -race -run TestPrompt.
This PR only adds prompt functionality to read simple string. One in the future to select from a menu may follow.
In #51, there is a suggestion for a prompt that reads numbers. However I think a sink with the name Number() is more appropriate for the functionality similar to what String() does.

I wonder if it makes sense to have the prompt message be the contents of the pipe, and to rename this method something like ReadInput:
defaultPath := "/usr/local/foo"
path, err := script.Echo("Installation path: ").ReadInput(defaultPath).String()

What do you think?

It totally makes the prompting functionality more flexible as a pipe compared to it as a source. I was just following the previous PR's approach.
The question is should we still have a source prompt function that under the hood utilizes the pipe ReadInput(string)? Or is it considered bloat that way?
Anyhow, I'll keep Prompt(string, string) as a source for the time being and start working on ReadInput(string) the pipe.

Notwithstanding the excellent work done on this, I'm closing the PR with some reluctance, because I just can't seem to fit it into the general scheme of the script library. It's essentially about pipelines, and I haven't seen a use case where 'read from terminal' can reasonably be part of a pipeline. (The shell builtin read is never used this way, either.)


How to have translated validation error messages?

I see the error messages come from a .resx file. When using the library in my own project, is there a (recommended) way to add error messages translated in my own language? For example by adding a nl-NL resx file as an embedded resource or something similar?
I checked the wiki but found mostly information on how to use this library.
Thanks a bunch for a nice bit of open source, either way! <3

Indeed a <Filename>.<locale>.resx would need to be added to this repo and included as part of the build. If you are up for it, be my guest. I do think there may be some areas where strings possibly still need to be extracted into the resources files (some exceptions are very niche/edge case that I did not bother to add to resx file).

Also, if you just use DataAnnotations or FluentValidation, you can actually use your own generic messages:
DataAnnotations
[Iban(ErrorMessageResourceName = "MyResourceName", ErrorMessageResourceType = typeof(Resources))]
public string BankAccount { get; set; }

FluentValidation
RuleFor(x => x.BankAccount).Iban(ibanValidator).WithMessage("MyMessage")

Ooh nice, the latter comment's option I'll investigate, seems to be just what I need.
I could add the nl-NL resources to the library itself in a PR as well. And you'd be most welcome to ping me for translations of new/changed strings. But... I can also imagine as an OSS maintainer you'd prefer not to have the burden of (potentially stale) extra locales.
Let me know if you want nl-NL translation contributions from my end, or prefer to leave it as is.
PS. If we leave it as is I'll check your other suggestion and instead write up some docs for the repository or wiki for others to find.

Any contrib is appreciated, both to the wiki and code. Localization is fine, yes it can be cumbersome to maintain but I don't expect many textual changes. Plus, I can help with Dutch ;P


RuntimeError: Given input size: (1024x6x7). Calculated output size: (1024x0x1). Output size is too small

when i try to run the test_localizer.py with command:
python tools/test_localizer.py test_configs/thumos14/ssn_thumos14_rgb_bn_inception.py modelzoo/ssn_thumos14_rgb_bn_inception_tag-dac9ddb0.pth --out ./output/result.pkl --eval thumos14
the error occurs, like"
Traceback (most recent call last):   File "/home/user/pose/mmaction/tools/test_localizer.py", line 179, in <module>     main()   File "/home/user/pose/mmaction/tools/test_localizer.py", line 107, in main     outputs = single_test(model, data_loader)   File "/home/user/pose/mmaction/tools/test_localizer.py", line 30, in single_test     result = model(return_loss=False, **data)   File "/home/user/anaconda3/envs/pose/lib/python3.7/site-packages/torch/nn/modules/module.py", line 532, in __call__     result = self.forward(*input, **kwargs)   File "/home/user/anaconda3/envs/pose/lib/python3.7/site-packages/torch/nn/parallel/data_parallel.py", line 150, in forward     return self.module(*inputs[0], **kwargs[0])   File "/home/user/anaconda3/envs/pose/lib/python3.7/site-packages/torch/nn/modules/module.py", line 532, in __call__     result = self.forward(*input, **kwargs)   File "/home/user/pose/mmaction/mmaction/models/localizers/base.py", line 32, in forward     return self.forward_test(num_modalities, img_meta, **kwargs)   File "/home/user/pose/mmaction/mmaction/models/localizers/SSN2D.py", line 169, in forward_test     x = self.spatial_temporal_module(x)   File "/home/user/anaconda3/envs/pose/lib/python3.7/site-packages/torch/nn/modules/module.py", line 532, in __call__     result = self.forward(*input, **kwargs)   File "/home/user/pose/mmaction/mmaction/models/tenons/spatial_temporal_modules/simple_spatial_module.py", line 25, in forward     return self.op(input)   File "/home/user/anaconda3/envs/pose/lib/python3.7/site-packages/torch/nn/modules/module.py", line 532, in __call__     result = self.forward(*input, **kwargs)   File "/home/user/anaconda3/envs/pose/lib/python3.7/site-packages/torch/nn/modules/pooling.py", line 554, in forward     self.padding, self.ceil_mode, self.count_include_pad, self.divisor_override) RuntimeError: Given input size: (1024x6x7). Calculated output size: (1024x0x1). Output size is too small
@yjxiong could you help check what's happened? thanks a lot.

Hi, that problem is due to the image size of your THUMOS14 data, in ssn config we resize it to (340, 256) with resize_keep_ratio=True, so that maybe after resizing the short edge will be less than 224px. We have updated the config of ssn, to resize images to 256(short edge). You can try our new code, the problem should have been solved.

@kennymckormick it works...thanks a lot!


label:"help wanted"  Centralized documentation with Spring Boot and Netflix Eureka

Hi @dilipkrish
I have tried to create centralized documentation in case of microservices (Eureka & Zuul). My implementation is working correctly.
Just wanted to ask that the approach is correct or does springfox library has some other way to achieve the same?

Code : centralized-swagger-docs

If it works for you its the right approach :)

Thanks @dilipkrish
Could you please also help with the other question i wanted help with. Does library itself has something out of the box to support the centralized documentation in case of microservice environment (API gateway, Registry, services)


Fine-tune DeViT using voc_2007_trainval_all3_10shot.

Hello, thank you very much for your outstanding work. Your discussion about fine-tuning DeViT and achieving significant performance improvements is truly ingenious. I attempted to fine-tune DeViT myself, but encountered the following issue:
File "/media/A/code/lyl/devit-main/tools/../detectron2/modeling/meta_arch/devit.py", line 1182, in forward
_ = torch.gather(feats, 2, indexes[cmask].view(bs, spatial_size, num_classes - 1)) # N x spatial x classes-1
RuntimeError: shape '[1024, 49, 5]' is invalid for input of size 251713
My train_class_weight is as follows:
tensor([[ 0.0237, -0.0245, -0.0242,  ..., -0.0186, -0.0096,  0.0192],
[-0.0787, -0.0277,  0.0089,  ..., -0.0130, -0.0020,  0.0158],
[ 0.0051,  0.0063, -0.0180,  ..., -0.0065, -0.0335,  0.0469],
[ 0.0195,  0.0194, -0.0173,  ...,  0.0026, -0.0435,  0.0465],
[ 0.0136, -0.0001, -0.0225,  ...,  0.0150, -0.0459,  0.0157],
[ 0.0495, -0.0077, -0.0189,  ...,  0.0401, -0.0211,  0.0141]],
device='cuda:1')
The dataset has 6 base classes and 3 novel classes . Do you have any suggestions for resolving this issue? I greatly appreciate your response!

Could you provide more details about this error, such as the shapes of the tensors involved?

Could you provide more details about this error, such as the shapes of the tensors involved?
Thank you very much for your response! I have provided the specific error details, as shown in the image.

This class_weight has always been [6, 1024], but shouldn't it be [9, 1024] since my base classes plus the novel classes total 9?

The final error message is:
-- Process 1 terminated with the following error:
Traceback (most recent call last):
File "/home/com/anaconda3/envs/devit/lib/python3.9/site-packages/torch/multiprocessing/spawn.py", line 69, in _wrap
fn(i, *args)
File "/media/A/code/lyl/devit-main/tools/../detectron2/engine/launch.py", line 125, in _distributed_worker
main_func(*args)
File "/media/A/code/lyl/devit-main/tools/train_net.py", line 200, in main
return trainer.train()
File "/media/A/code/lyl/devit-main/tools/../detectron2/engine/defaults.py", line 496, in train
super().train(self.start_iter, self.max_iter)
File "/media/A/code/lyl/devit-main/tools/../detectron2/engine/train_loop.py", line 149, in train
self.run_step()
File "/media/A/code/lyl/devit-main/tools/../detectron2/engine/defaults.py", line 506, in run_step
self._trainer.run_step()
File "/media/A/code/lyl/devit-main/tools/../detectron2/engine/train_loop.py", line 273, in run_step
loss_dict = self.model(data)
File "/home/com/anaconda3/envs/devit/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/home/com/anaconda3/envs/devit/lib/python3.9/site-packages/torch/nn/parallel/distributed.py", line 1040, in forward
output = self._run_ddp_forward(*inputs, **kwargs)
File "/home/com/anaconda3/envs/devit/lib/python3.9/site-packages/torch/nn/parallel/distributed.py", line 1000, in _run_ddp_forward
return module_to_run(*inputs[0], **kwargs[0])
File "/home/com/anaconda3/envs/devit/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/media/A/code/lyl/devit-main/tools/../detectron2/modeling/meta_arch/devit.py", line 1182, in forward
_ = torch.gather(feats, 2, indexes[cmask].view(bs, spatial_size, num_classes - 1)) # N x spatial x classes-1
RuntimeError: shape '[1024, 49, 5]' is invalid for input of size 251468
Thank you once again for your excellent work and your response.

Since our model is pre-trained on COCO, we assume that the base classes correspond to COCO's classes, while the novel classes include all the classes from the new datasets. In the k-way k-shot setting, we train the model using k-shot instances and evaluate it on the test datasets.


bump: Add chromedriver binary in its updated version.

#minor
Description
This PR removes the chromedriver package and replaces its functionality using its binary to perform the browser unit tests.
Specific Changes

Removed  chromedriver package from browser-functional tests.
Added chromedriver binary into browser-functional.
Modified server path of nigthwatch chrome configuration.
Removed unnecessary jobs related to chromedriver in the browser-tests-build-ci pipeline.

Testing
The following image shows the browser-tests-build-ci pipeline running successfully after the changes.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 84.185%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
20334

Relevant Lines:
22872

💛  - Coveralls

Promoted /4596


in-copr --help now specifies that config needs to be set before usage.

Related to https://github.com/packit/packit/issues/1762

For transparency, I am going to rebase the branch for you and let it merge by Zuul.


[Snyk] Upgrade org.apache.maven:maven-plugin-api from 3.9.8 to 3.9.9

Snyk has created this PR to upgrade org.apache.maven:maven-plugin-api from 3.9.8 to 3.9.9.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.

The recommended version was released on 23 days ago.

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

/ok-to-test


[DAPHNE-#772] Fix Constant folding crashes for values of different types

Constant folding required the operands to have the same type. For instance, the following two DaphneDSL scripts would be perfectly amenable to constant folding, but make the DAPHNE compiler crash with the respective error messages:
print(1 * as.si32(1));
daphne: /daphne/thirdparty/llvm-project/llvm/lib/Support/APInt.cpp:226: llvm::APInt llvm::APInt::operator*(const llvm::APInt &) const: Assertion BitWidth == RHS.BitWidth && "Bit widths must be the same"' failed. ... Segmentation fault (core dumped)
FIxed it by comparing the bit width in the constFoldBinaryOp method. If they are not the same, the smaller one is getting type promoted to the bigger one (reusing the CastOp method to perfom this type promotion).
Aded testcases for binary op of different types

LGTM
Tests run fine
Thx for your contribution @ldirry


P70

I found a typo in "Finally, everything discusses above applies to scope hierarchies of arbitrary depths:"
Instead of "discusses" it should be "discussed".
Affects version 2015-02-23.

Fixed, thanks


Sort Imports command not following PEP8 strictly

Environment data

VS Code version: 1.56.0
Extension version (available under the Extensions sidebar): v2021.5.829140558
OS and version: Ubuntu 20.04
Python version (& distribution if applicable, e.g. Anaconda): Python 3.8.5
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): virtualenv
Value of the python.languageServer setting: pylance

[NOTE: If you suspect that your issue is related to the Microsoft Python Language Server (python.languageServer: 'Microsoft'), please download our new language server Pylance from the VS Code marketplace to see if that fixes your issue]
Expected behaviour
When using the Sort Import features in a script that contain standard libraries imports, third-party imports and local imports, I expect the output to conform with the imports section on PEP8, there should be a blank line between:

Standard library imports.
Related third-party imports.
Local application/library-specific imports.

Therefore, the desired output would be:
import functools
import os

from django.db import models
from rest_framework import viewsets

from myapp.models import Upload

Actual behaviour
Let's say I write the following imports when creating a Django app.
import os
import functools
from rest_framework import viewsets
from django.db import models
from myapp.models import Upload

After I run the Sort Imports command, the output is:
import functools
import os

from django.db import models
from myapp.models import Upload
from rest_framework import viewsets

This means VSCode is sorting the third-party imports together with my local app imports alphabetically.
Steps to reproduce:

Write any script that has standard library imports, third-party imports, and your local packages imports.
Right-click in the script and select Sort Imports

Logs

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

Before they change the language server to Pylance, Sort Imports function uses isort library.
At that time, we want to get the correct order should add --src=${workspaceFolder} to python.sortImports.args.
Now switch to Pylance, this setting still works.

We use isort for import sorting in all cases. So the behavior here depends on how isort was configured. If this is not sorting as intended please file a bug on isort

It started with an isse on VSCode, then on vscode-python extension and no it's supposed to go to isort :-1:


Release material-ui to npm

Problem description
0.15.2 – latest verion on npm
Steps to reproduce
Versions

Material-UI: 0.15.2
React:
Browser:

@oliviertassinari Please look

@dmfilipenko I don't have the access needed to do it. cc @hai-cea.

@dmfilipenko just published.

@hai-cea Thanks!


refactor(core): UUID suffix instead of Unix timestamp

I'm attempting parallel writes to the same target BigQuery table from multiple source taps (using Meltano). When a Unix timestamp is used as a suffix for the intermediate table, there is a significant chance that concurrent processes produce the same name for it (i.e., if they happen to call time.time() within the same second). This results in a scenario in which the first process to finish loading records drops the table, and a latter process raises an exception - e.g.: google.api_core.exceptions.BadRequest: 400 Not found: Table project:dataset.table_name__1699498892 was not found.
I'm not sure if there was another motivation behind using the Unix timestamp as a suffix, but if it was purely for the sake of uniqueness, I'd like to recommend using a UUID v4 suffix instead, which is effectively guaranteed to be unique across concurrent processes.

LGTM


Support go1.12 toolchain

https://golang.org/doc/go1.12#compiler requires changes to the way the stack is analysed.
See https://github.com/pkg/errors/blob/ba968bfe8b2f7e042a574c888954fccecfa385b4/stack.go#L136 in v0.8.1 for example.

I believe this is no longer required, Keith rolled back the change to runtime.Callers

See #200. I believe that Go 1.12 is supported by this package.


Maintainers of pkgs need to be lists not strings

Description
Relates to: https://github.com/NixOS/nixpkgs/pull/170992 #64
The daily builds been failing for a few days off nixpkgs/master: https://github.com/Nebulaworks/nix-garage/actions/runs/2295159292
The error message seems to be swallowed by what @kylerisse was seeing in #64
Looking into it more:
$ export NIX_PATH=nixpkgs=https://github.com/NixOS/nixpkgs/archive/053c592cf204627172540d8c2dd4cf11e770ae27.tar.gz
$ nix-shell --run 'bash -x ./publish-imgs' #same command run in daily.yml
$ nix-shell --run 'bash -x ./publish-imgs'
+ set -efo pipefail
+ registry=docker.io
+ nixtop=default.nix
+ getopts f:hs OPTION
+ shift 0
+ attr=imgs
+ BUILT=()
+ nix-instantiate --strict --json -A imgs default.nix

$ echo $?
1

Ok thats definitely related to the silent failure in #64 but lets run the nix-instantiate with a trace
Results in:
$ nix-shell --run 'nix-instantiate --strict --json -A imgs default.nix --show-trace'
warning: you did not specify '--add-root'; the result might be removed by the garbage collector
error: value is a string while a list was expected

       at /nix/store/b0db8fic9vxksg726bxzmrslpzsp87a5-source/pkgs/stdenv/generic/check-meta.nix:54:34:

           53|   hasNoMaintainers = attrs:
           54|     attrs ? meta.maintainers && (lib.length attrs.meta.maintainers) == 0;
             |                                  ^
           55|

       … while evaluating 'hasNoMaintainers'

       at /nix/store/b0db8fic9vxksg726bxzmrslpzsp87a5-source/pkgs/stdenv/generic/check-meta.nix:53:22:

           52|
           53|   hasNoMaintainers = attrs:
             |                      ^
           54|     attrs ? meta.maintainers && (lib.length attrs.meta.maintainers) == 0;

       … from call site

       at /nix/store/b0db8fic9vxksg726bxzmrslpzsp87a5-source/pkgs/stdenv/generic/check-meta.nix:311:13:

          310|     # Please also update the type in /pkgs/top-level/config.nix alongside this.
          311|     else if hasNoMaintainers attrs then
             |             ^
          312|       { valid = "warn"; reason = "maintainerless"; errormsg = "has no maintainers"; }

       … while evaluating 'checkValidity'

       at /nix/store/b0db8fic9vxksg726bxzmrslpzsp87a5-source/pkgs/stdenv/generic/check-meta.nix:288:19:

          287|   # Along with a boolean flag for each reason
          288|   checkValidity = attrs:
             |                   ^
          289|     {

       … from call site

       at /nix/store/b0db8fic9vxksg726bxzmrslpzsp87a5-source/pkgs/stdenv/generic/check-meta.nix:317:18:

          316|   assertValidity = { meta, attrs }: let
          317|       validity = checkValidity attrs;
             |                  ^
          318|     in validity // {

       … while evaluating 'assertValidity'

       at /nix/store/b0db8fic9vxksg726bxzmrslpzsp87a5-source/pkgs/stdenv/generic/check-meta.nix:316:20:

          315|
          316|   assertValidity = { meta, attrs }: let
             |                    ^
          317|       validity = checkValidity attrs;

       … from call site

       at /nix/store/b0db8fic9vxksg726bxzmrslpzsp87a5-source/pkgs/stdenv/generic/make-derivation.nix:403:14:

          402|
          403|   validity = checkMeta { inherit meta attrs; };
             |              ^
          404|

       … while evaluating the attribute 'outPath'

       at /nix/store/b0db8fic9vxksg726bxzmrslpzsp87a5-source/lib/customisation.nix:211:7:

          210|       drvPath = assert condition; drv.drvPath;
          211|       outPath = assert condition; drv.outPath;
             |       ^
          212|     };

       … while evaluating the attribute 'contents' of the derivation 'docker-layer-flasksample'

       at /nix/store/b0db8fic9vxksg726bxzmrslpzsp87a5-source/pkgs/stdenv/generic/make-derivation.nix:270:7:

          269|     // (lib.optionalAttrs (attrs ? name || (attrs ? pname && attrs ? version)) {
          270|       name =
             |       ^
          271|         let

       … while evaluating the attribute 'buildCommand' of the derivation 'docker-image-flasksample.tar.gz'

       at /nix/store/b0db8fic9vxksg726bxzmrslpzsp87a5-source/pkgs/stdenv/generic/make-derivation.nix:270:7:

          269|     // (lib.optionalAttrs (attrs ? name || (attrs ? pname && attrs ? version)) {
          270|       name =
             |       ^
          271|         let

A recent change upstream to pkgs/stdenv/generic/check-meta.nix seems to the expecting maintainers as a list (seems like thats the right expectation). Specially this line from the PR: https://github.com/NixOS/nixpkgs/pull/170992/files#diff-8d6de52a69f8de7972d373a09d7a0747f88017b71242cd7bc241d61dd431101dR54
Woudlnt you know it flasksample has such an error that aligns with why the docker-image-flasksample drv wont build: https://github.com/Nebulaworks/nix-garage/blob/4e2d274f907e48d798cd494d21992a4d809206a4/pkgs/flasksample/default.nix#L27
AC:

Fix our maintainers in all pkgs to be lists

Ah dang daily build is still failing (new error): https://github.com/Nebulaworks/nix-garage/actions/runs/2324386145
Will look into this conflict a little more shortly but not an issue for the time being for anything leveraging existing pins


Toolbar Icons Do Not Activate on Macos [BUG]

Describe the bug
When a tool is selected, the icon does not change to indicate that it is currently activated.

Apparently the way Macos renders buttons makes this impossible.  Perhaps changing the colour of the selected button will suffice.


Add "lifecycle" to GCS bucket fields enum.

/cc @garrettjonesgoogle
Per https://cloud.google.com/storage/docs/managing-lifecycles#check

Coverage increased (+0.0007%) to 84.343% when pulling 891fd4cc17c2a782152be075118fa42d4584039c on jerjou:lifecycle into 5dfb90466366370a0e0bfaeea476e4cfdb29b209 on GoogleCloudPlatform:master.


Refreshing the editor page with an unsaved template snippet loads the wrong template on refresh

This annoys the hell out of me. If you refresh the page it should re-use the same template. One way to resolve this is to make sure when we open an unsaved snippet we keep the template type in the url
https://github.com/user-attachments/assets/5d2ff280-8215-4cba-9db1-97fc46aa022a
/bounty $8

/attempt #195
I will update CodeEditor

Adding template state tracking using a URL query parameter
Preserving the template type across page refreshes
Loading the correct template content based on the URL parameter

/attempt #195


Update on Variants in Dictionaries?

https://github.com/Microsoft/AL/issues/913
Is there any news on when this might be possible?

It is on our backlog but currently not planned within the next 6 months.

+1

Any update on this? It has been over a year since I have heard anything about it.

@esbenk I know it's a long time since you made the comment but you mentioned that limited return types were the only blocker for more generic constructs of a dynamic length, like a List of Variant etc.

https://github.com/microsoft/AL/issues/913#issuecomment-344205033

I've been testing NextMajor and while complex types work, List of [Variant] and Dictionary of [Variant, Variant] don't work while e.g. array[16] of Variant has always worked for storing complex values.
Variant as a parameter and return value however works in runtime 7.0 and I'm really happy about that, opens up new possibilities. :)
Can we expect these generic constructs in the upcoming releases, since the main blocker has been solved?
procedure Foo(l: List of [Variant])
var
  v: Variant;
begin
  foreach v in l do
    if true in [
      v.IsText,
      v.IsInteger,
      // ... v.IsList/v.IsDictionary are still missing, see: #6481 and #5173
    ] then
      ValidOverloads(v); // or e.g. Message(Format(v, 0, 9));
end;

It is related, but the solution used for returning complex types it not directly applicable here. It is on our list, but we don't have any ETA for this.

Thanks for the insight Esben, you are all doing a really great job and I look forward to see the language progress even further. :)


Added Item BDD Test

If fail to start the server, do not forget to rebuild the container.
For each time BDD test, honcho start first then behave.


Add OncoKB

This work already exists in one of my jupyter notebooks somewhere, just need to copy the code over
Could be helpful for a metakb issue that @korikuzma is working on (although I think we've already implemented oncokb fetching for metakb before)

Just checked my notebooks and I hadn't actually implemented a functional version of this, although I had a few others.


Download link not working

Azure Data Studio Version:
OS Version: macos

Steps to Reproduce:

Download link not working.

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

The links seem to work fine to me so I'm closing this, please provide the link that's not working and what error you're setting if you're still seeing this and we can investigate!

The links seem to work fine to me so I'm closing this, please provide the link that's not working and what error you're setting if you're still seeing this and we can investigate!


Using redundant module alias import to make pyright happy

Related: https://github.com/microsoft/pylance-release/issues/2953#issuecomment-1168408943

I don't think this applies to this project.
The top-level imports are done correctly here. Unless mistaken.
Also, we don't use pyright to assess the typing,
I ran the tool using the master branch and got the following output:
/charset_normalizer/api.py
/charset_normalizer/api.py:80:29 - error: "previous_logger_level" is possibly unbound (reportUnboundVariable)
  /charset_normalizer/api.py:258:17 - error: "encoding_soft_failed" is possibly unbound (reportUnboundVariable)
 /charset_normalizer/api.py:442:33 - error: "previous_logger_level" is possibly unbound (reportUnboundVariable)
  /charset_normalizer/api.py:453:33 - error: "previous_logger_level" is possibly unbound (reportUnboundVariable)
  /charset_normalizer/api.py:495:25 - error: "previous_logger_level" is possibly unbound (reportUnboundVariable)

All of the error yields are false alarms there.
mypy infer correctly that those variables are bound at the given code path.
I am going to close this as I don't think it is relevant / fix something.
regards,


[SYSTEMML-872] Add new SystemML.py to various artifacts

Adds SystemML.py to the root of all artifacts where it makes sense.

@deroneriksson Since SystemML.py is likely going to be replace by a package structure once https://github.com/apache/incubator-systemml/pull/197 is merged in, can you please hold off this PR ? Once the PR 197 is merged, we will have a seperate Python package artifact (as a tar file).

@niketanpansare @dusenberrymw
There is going to be another artifact added to the project devoted to Python? Shouldn't this be packaged with the regular jar and standalone tar.gz/zip?

@niketanpansare @dusenberrymw
BTW I am fine with whatever approach you believe to be the best approach for Python users. If you think this PR should be closed w/o merging, that is fine. :-)

The shell script in the python package 'uploadToPyPI.sh' will create a tar.gz in dist/ directory. We can add that artifact into our release. Anyone can use command 'pip install ..tar.gz' and install SystemML.

It's probably a good idea to keep @lresende aware of the status of this to be sure that the release will follow Apache requirements.

I have created a JIRA: https://issues.apache.org/jira/browse/SYSTEMML-883
@lresende or @deroneriksson Do either one of you want to take it over ?

@niketanpansare I'm probably not a good choice for most Python tasks due to my lack of experience with it. :-)
Should I go ahead and close this PR?

No worries :) ...  I think we should not release the old SystemML.py (as it is deprecated) and only release the new Python package. Let's go ahead and close this PR.
@lresende commented on the JIRA that he will take a pass at this. If necessary, I too can help with it.

I will close this PR and the accompanying JIRA.


Fixing and enabling some Projects tests

STS update changes:

The code in project.addScriptItem() doesn't join mixed-slash paths properly, but since I'm intending to introduce content params to let DacFx.Projects handle new files and deprecate addScriptItem() in an upcoming PR (it's currently not used in the oracle migration extension), I figured I'd update the test and let the code removal take care of the bug.

@SakshiS-harma

What did the STS update changed, that the tests are passing now?

There was yet another cross-plat path bug in the Projects library, fixed by https://msdata.visualstudio.com/SQLToolsAndLibraries/_git/DacFx/pullrequest/1073966.


Detection of circular dependencies

In the current ISO 19115-3 there are multiple circular dependencies, which means that implementation of them cannot be possible.
This means that we need to detect these circular loops and alert the authors to think hard and break them.

Example in conformance tests:
sources/ed2/sections/tables/yaml/19115-1-xml-cc.yaml:
  - name: Validation of XML instance for metadata basic information
    identifier: https://standards.isotc211.org/19115/-1/1/conf/metadata-xml/basic
    target: https://standards.isotc211.org/19115/-1/1/req/metadata-xml/basic
    dependencies:
    - https://standards.isotc211.org/19115/-1/1/conf/metadata-minimal-xml
    - https://standards.isotc211.org/19115/-1/1/conf/metadata-xml/common
    - https://standards.isotc211.org/19115/-1/1/conf/metadata-xml/multilingual

sources/ed2/sections/tables/yaml/19115-xml-min-cc.yaml:
  - name: Validation of minimum valid XML metadata instance
    title: Minimum valid XML metadata instance
    identifier: https://standards.isotc211.org/19115/-1/1/conf/metadata-minimal-xml
    target: https://standards.isotc211.org/19115/-1/1/req/metadata-minimal-xml
    subject: Metadata instance
    tests:
    - name: Valid minimum metadata instance
      identifier: https://standards.isotc211.org/19115/-1/1/conf/metadata-minimal-xml/valid
      targets:
      - https://standards.isotc211.org/19115/-1/1/req/metadata-minimal-xml/valid
      purpose: Verify that the minimal XML metadata instance document is well formed and valid
      method: Use an XML validation tool to determine if XML instance is valid using the XML schema `mdb.xsd`. See discussion of validation tools in <<scl_no_systematic_test>>.
      type: Validation
      dependencies:
      - https://standards.isotc211.org/19115/-1/1/conf/metadata-xml/basic
      - https://standards.isotc211.org/19115/-1/1/conf/metadata-xml/citation
      - https://standards.isotc211.org/19115/-1/1/conf/metadata-xml/common
      - https://standards.isotc211.org/19115/-1/1/conf/metadata-xml/extent
      - https://standards.isotc211.org/19115/-1/1/conf/metadata-xml/identification
      - https://standards.isotc211.org/19115/-1/1/conf/metadata-xml/multilingual

There's a very clear circular dependency here where:

https://standards.isotc211.org/19115/-1/1/conf/metadata-xml/basic depends on
https://standards.isotc211.org/19115/-1/1/conf/metadata-minimal-xml and vice versa

One of these must drop the other because this loop does not make sense.

BTW, I've removed this circular dependency from ISO 19115-3, which has been submitted. It's just hard to find issues like this when staring at a wall of text.

Implemented detection of cycles for each of parent-child, dependency (inherits), indirect_dependency, and implements, using TSort stdlib.


vulkan: accept device selection

read CL_DEVICE_TYPE_ when specified like other conformance tests
reject non-GPU device types

Discussed in memory subgroup call of October 22nd. We should reuse test harness functionality.

Merging as discussed in Memory subgroup call of October 22nd, 2024.


[Feature]: Cycle\Database\Query\SelectQuery::from missing "alias" parameter

Is your feature request related to a problem? Please describe.
When writing complex requests, especially with subqueries, Cycle generates unusable or outright wrong SQL queries.
Example:
<?php

        $arr = $database->select([
            "id",
        ])->from(
            $this->database()->select([
                "id",
                new Fragment("COUNT(*) OVER (PARTITION BY field1, field2) AS d"),
            ])->from('a_table')
        )
            ->where("field3", "=", new Parameter(1))
            ->where("d", ">", new Parameter(1))
            ->fetchAll();

The generated query looks like
SELECT `id`
FROM (SELECT `id`, COUNT(*) OVER (PARTITION BY tag, component) AS d
FROM `a_table`)
WHERE `field3` = ? AND `d` > ?;

which looks fine, but in fact is not correct MySQL.
Describe the solution you'd like
Simply adding subquery table alias … FROM (SELECT …) AS t WHERE … fixes that in an instant.
<?php

        $arr = $database->select([
            "id",
        ])->from(
            $this->database()->select([
                "id",
                new Fragment("COUNT(*) OVER (PARTITION BY field1, field2) AS d"),
            ])->from('a_table'),
            "t"
        )
            ->where("field3", "=", new Parameter(1))
            ->where("d", ">", new Parameter(1))
            ->fetchAll();

SELECT `id`
FROM (SELECT `id`, COUNT(*) OVER (PARTITION BY tag, component) AS d
FROM `a_table`) AS `t`
WHERE `field3` = ? AND `d` > ?;

Describe alternatives you've considered
Using ->innerJoin() method with appropriate ->on() bindings works around the deficiency, but defy the code readability.
Additional context
No response

You can achive this with Workaround ^_^
$select->from(
    new Fragment('('.$selectSub.') as table1'),
)

@roxblnfk may be we need to add some extra class for this case e.g. DerivedTable
DerivedTable
$subQuery = $this
    ->database()
    ->select([
        "id",
        new Fragment("COUNT(*) OVER (PARTITION BY field1, field2) AS d"),
    ])
    ->from('a_table')
$derivedTable = new DerivedTable(
    query: subQuery,
    alias: 't'
);

$arr = $database
    ->select(["id"])
    ->from($derivedTable )
    ->where("field3", "=", new Parameter(1))
    ->where("d", ">", new Parameter(1))
    ->fetchAll();

CTETable
Aloso we can add a new feature  CTETable which render sql -wuery with wtih
$subQuery = $this
    ->database()
    ->select([
        "id",
        new Fragment("COUNT(*) OVER (PARTITION BY field1, field2) AS d"),
    ])
    ->from('a_table')
$cteTable = new CTETable(
    query: subQuery,
    alias: 't'
);
$arr = $database
    ->select(["id"])
    ->from($cteTable)
    ->where("field3", "=", new Parameter(1))
    ->where("d", ">", new Parameter(1))
    ->fetchAll();

WITH t AS
(
  SELECT]
    `id`,
    COUNT(*) OVER (PARTITION BY tag, component) AS d
  FROM `a_table`
)
SELECT `id` FROM `t` WHERE `field3` = ? AND `d` > ?;

@gam6itko yes it is also my main idea about it
See https://github.com/cycle/database/issues/139#issuecomment-1783856089


Updated property name for active items

Fixed property name from data to active. Should fix #307.

thanks :)


Unique arguments with substitution tags

Is it possible to set a unique argument for each e-mail I send, when sending to multiple recipients and when using substitution tags?
Currently using the following code which works fine to send to multiple users, with substitution tags:
$mail = new SendGrid\Mail();
$mail->setFrom($email)
    ->setFromName($from)
    ->setSubject($subject)
    ->setHtml($body)
    ->setTos($recipients)
    ->addSubstitution("__recipientid__", $replacementIDS)
    ->addSubstitution("__recipient__", $replacementNames);

I'd like to be able to also add something like: ->addUniqueArgumentSubstitution('my_unique_key', $arrayOfValues);

Hi @drawmyattention,
Do you mean something like this? https://github.com/sendgrid/sendgrid-php/blob/master/examples/helpers/mail/example.php#L72

Perfect. I was using an old version of the package, which did not have the Personalization class.
Was there a way to do this without that class?

@drawmyattention,
No, the personalization class is fundamental to the new v3 design.
You can dig deeper here: https://sendgrid.com/docs/Classroom/Send/v3_Mail_Send/personalizations.html
Thanks!


util: allow configuring VAULT_BACKEND for Vault connection

It seems that the version of the key/value engine can not always be
detected for Hashicorp Vault. In certain cases, it is required to
configure the VAULT_BACKEND (or vaultBackend) option so that a
successful connection to the service can be made.
The kv-v2 is the current default for development deployments of
Hashicorp Vault (what we use for automated testing). Production
deployments default to version 1 for now.

Show available bot commands
These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

/retest ci/centos/<job-name>: retest the <job-name> after unrelated
failure (please report the failure too!)
/retest all: run this in case the CentOS CI failed to start/report any test
progress or results

/retest ci/centos/upgrade-tests-cephfs

@mergifyio rebase

@humblec @Madhu-1, it seems Mergify dropped the approvals here :disappointed:

@mergifyio rebase

/retest ci/centos/mini-e2e-helm/k8s-1.19

/retest ci/centos/mini-e2e-helm/k8s-1.19

Failed with #2264

@mergifyio backport release-v3.3


kibosh example chart should have something meaningful in plans/default.yaml

Right now it is empty and its unclear what if anything can be put into the plan yaml. The documentation should also talk about how to use plans.yaml.

We've added a new example here: https://github.com/cf-platform-eng/tile-generator/tree/master/examples/kibosh-mysql/mysql with a simple example of multiple plans, with different configuration.
I'll add a story for improving the plans.yaml and plans/.yaml documentation, but the quick answer is each plans/.yaml file contains plan-specific overrides for your values.yaml file. There is a brief blurb in the kibosh readme, but I agree it should be improved.

See

https://github.com/pivotal-cf/docs-tiledev/pull/24
https://github.com/cf-platform-eng/kibosh/blob/master/README.md#changes-required-in-chart


Add theming system

using hashes and functions, so one can easily create interchangeable themes or even libraries.

So I'm not really super interested in adding this to axis, as it's more of a general utility belt and the UI stuff is just for basic prototyping with the intent that you'd be writing your own custom visuals for most sites rather than just ripping a theme like bootstrap. But I know @corysimmons is pretty interested in working on a library that does theming stuff, and has started some work on a couple -- maybe he would be able to tell you more!

I think i did not express myself clear enough, i was having the idea of storing a $theme object containing keys like foreground, background, highlight etc and using functions like color('background') to get a specific color.
In short terms, abstracting away the actual names of colors to easily change the color scheme of the whole design.

I started work on a themeable UI kit but haven't touched it for a while. Feel free to check it out: https://github.com/corysimmons/switch

Hey @FranzSkuffka -- you can do this by just adding your own version of the _settings.styl file :grin:

As always, you are right.
Don't be like that to me :D


Add table of Org plans to My Dashboard

UX wireframe @JEK-III http://h90c3m.axshare.com/#g=1&p=my_dashboard
Add a separate table for all the plans marked w/Org or Public visibility for a specific Org if the user has an Org affiliation. This table should appear below the My plans table on My Dashboard page.
Note blue notepad functional specs. UX recommendation to make the Org table a less salient color for production services. Okay to leave gray in Roadmap.
Add Notifications area to this issue also?

this looks good @jollopre - will create separate issue for formatting long tables w/page breaks. closing this issue.


Code-Based Linking (CBL) for Other Devices and Platforms -Device Authorization Request fail!!!!

curl -k -d 'response_type=device_code&client_id={{amzn1.application-oa2-client.37b63f01091146249651b5774523982b}}&scope=alexa%3Aall&scope_data=%7b%0d%0a++%22alexa%3aall%22%3a+%7b%0d%0a++++%22productID%22%3a+%22%7b%7bmy_Device%7d%7d%22%2c%0d%0a++++%22productInstanceAttributes%22%3a+%7b%0d%0a++++++++%22deviceSerialNumber%22%3a+%22%7b%7b%7d%7d%22%0d%0a++++%7d%0d%0a++%7d%0d%0a%7d' -H "Content-Type: application/x-www-form-urlencoded" -X POST https://api.amazon.com/auth/O2/create/codepair
{"error_description":"The request has an invalid parameter : client_id","error":"InvalidValue"}

Hi,
How did you remove this error?


feat(slp): Added slp-mutable-data to gatsby-web-wallet

PR related to this trello task --> https://trello.com/c/7X9QxXYe

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added a placeholder for FrontEnd.

Creating a placeholder PR for Front End. Do not merge yet.

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes #52.

Type of change 📝

[ ] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev (Flutter, Java, Kotlin)
[ ] Backend Dev (Java, .NET, Python)
[ ] C/CPP
[ ] Competitive Programming
[ ] Cyber Security
[ ] DSA
[ ] Database
[ ] Datascience with Python
[ ] Datascience with R
[x] Frontend Dev (HTML, CSS, JS)
[ ] Frontend Dev (React, Angular, Vue)
[ ] Golang
[ ] Interview Prep
[ ] Java
[ ] JavaScript
[ ] MERN
[ ] Machine Learning
[ ] Open Source
[ ] Python
[ ] Rust
[ ] Statistics
[ ] UI/UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

LGTM 👍🏻

@praveenscience  I think you need to create PR for your branch i.e FEWD

@Aryamanz29 Ah, got it! :)


feat(brand-icons): add two new brand icons

add two new brand icons
Description
Added two new brand icons. minutes and quick-and-easy.
Requirements
Needed to change the svgo dependencie back to version 1.3.0 because the command npm run optimise:svgs would break when trying to commit, with the error:

Storybook link: (once netlify has deployed link provide a link to the component)
Design link: design
Screenshot:

Checklist:

[x] The commit messages follow the convention for this project
[ ] I have provided an adequate amount of test coverage
[ ] I have added the functionality to the test app
[x] I have provided a story in storybook to document the changes
[ ] I have provided documentation in the notes section of the story
[ ] I have added any new public feature modules to public-api.ts

@AlexandruHoisan I've approved but would you mind creating an issue re svgo please so that we don't forget?

@elenagarrone Done. :)


Add configurable https redirection to mid-tier

Coverage remained the same at 26.531% when pulling<PHONE_NUMBER>05c7bcd97eb4b9215d17a433cb9db4 on mars-lan:master into 713a51db1c631eab9a14421f9ae3efb2dfb0739b on linkedin:master.

Codecov Report

Merging #1166 into master will increase coverage by 0.49%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1166      +/-   ##
==========================================
+ Coverage   33.38%   33.87%   +0.49%
==========================================
  Files         199      199
  Lines        4889     4889
  Branches     1325     1325
==========================================
+ Hits         1632     1656      +24
+ Misses       3257     3233      -24

Impacted Files
Coverage Δ

...mponents/datasets/containers/dataset-compliance.ts
62.29% <0%> (+1.63%)
:arrow_up:

wherehows-web/app/services/notifications.ts
63.75% <0%> (+28.75%)
:arrow_up:

Continue to review full report at Codecov.

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


Method name check

PR has multiple bugs. I'll try to fix them real quick.

Oh I kind of see what happened.
Of the three commits on this PR, commit 743ac20 seems perfect, but the later 2 commits broke everything.

Closing because the work was reproduced in #14


Deprecate legacy pulse builder command

Summary
This addresses issue #11152 .
This PR deprecates

qiskit.pulse.builder.call_gate

qiskit.pulse.builder.cx

qiskit.pulse.builder.u1

qiskit.pulse.builder.u2

qiskit.pulse.builder.u3

qiskit.pulse.builder.x

qiskit.pulse.builder.active_transpiler_settings

qiskit.pulse.builder.active_circuit_scheduler_settings

qiskit.pulse.builder.transpiler_settings

and, modifies related tests in

test/python/pulse/test_builder.py
test/python/pulse/test_block.py
test/python/pulse/test_builder_v2.py
test/python/transpiler/test_calibrationbuilder.py

Details and comments
Refer to Issue: #11152
fixes #11152

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

10 of 10 (100.0%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.02%) to 86.951%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
2
90.91%

qiskit/pulse/library/waveform.py
3
93.75%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
74384

Relevant Lines:
85547

💛  - Coveralls

@nkanazawa1989 , please see if this is ok :)

Thanks @MozammilQ the PR itself looks good to me. Unfortunately (fortunately?) we are now on bit tricky period for the first major version release. This means main branch doesn't accept code deprecation, and they should go to stable/0.46 branch instead. Guide is here: https://github.com/Qiskit/feedback/discussions/36
Since this PR targets the main branch, you can immediately remove all code to deprecate, and move the deprecation code you wrote for the PR to another PR targeting stable/0.46. If you feel this workflow is troublesome, I'll merge this PR as-is and take care of the rest of procedure. What do you prefer?

@nkanazawa1989 , I will first move the deprecation code which is currently in this PR to another PR against stable/0.46,
and then, remove the deprecated code here in this PR.

@nkanazawa1989 , PR #11249 , is against stable/0.46,
now, I will remove the deprecated code here, (against main)

I am not done yet! I will let you know when  I am done :)


Snippet thumbnail shows gist description smartly

Currently, the snippet thumbnail shows gist description completely, which sometimes is too long for a thumbnail card.

Proposal:

Only pick up the title. However, GitHub Gist doesn't provide the "title" concept in its Gist description section. We need to invent the concept ourselves, like putting the title inside brackets[title] description.
Set a maximum lines and truncate the rest.

fixed:https://github.com/hackjutsu/Lepton/commit/3df8c28bcac174b956c92801768327afd5c0d5dc


Fix an outdated comment

Fix the outdated comment in ServerCompletionQueue constructor

Just realized I no longer have write-access :(.. Wanted to assign this to @vjpai for review (as per his comment here: https://github.com/grpc/grpc/pull/16992#discussion_r228033869)

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

@sreecha I'm glad to proxy for you until we figure out the committer story. ( calling @hsaliak )

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 2,015,303      Total (=)      2,015,303

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
11,115,279      Total (=)     11,115,279

 No significant differences in binary sizes

Corrupt JSON data (indicates timeout or crash):
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.new: 10
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.old: 10

[microbenchmarks] No significant performance differences


Feature Request: Search on current project's files

Expose the command to search files in the current project directly so that you can bind a shortcut directly to it. As I see currently you have to invoke the menu, find the current project again and then be able to search on its files.

also a option to disable the find files search instantly after the project has been selected

@matu3ba When you are editing a big project sometimes :Telescope find_files will only list files under your current working directory. This plugin always has the feature to fuzzy search over files of the current project regardless of the current working directory. The problem is that it only exposes one command that I can bind to (:lua require'telescope'.extensions.project.project{}<cr>) unless I am mistaken that's the only telescope command it has.
Another problem with the b\C-b thing is that you have to first type the name of the project you are on. What I want is to be able to already search for the files of the current project with a single command.
I guess this project isn't currently keeping track of the last selected project so that would be needed for this, and then expose a command to directly have the file exploring telescope from that menu.

Ah, I get it.
Instead of require'telescope'.extensions.project.project{}<cr> you want
require'telescope'.extensions.project.project{}.all()<cr>
and
require'telescope'.extensions.project.project{}.current()<cr>
Yes, this would help. I am planning this for running scripts (local project) vs editing scripts (all projects). But having it upstream would make things simpler.

Any news on this issue? I'm a convert from doom-emacs and Visual Studio Code. In Doom, I could just hit <Leader> p f or <Leader> <Leader> to search for a file in the current project. I used this all the time!
In VS Code this is covered by hitting Ctrl-P. The files even come up sorted according to recency.
Having this functionality work as frictionless as possible in Neovim is a hard requirement for my workflow. And it's already covered by tools like CommandT or CtrlP. However, I think that Telescope is the way to go from this point on, so having a fast "jump to project-file" is crucial from my point of view.

Hey Tim, I am in the process of figuring out and communicating the intended direction to become maintainer.
There are 3 blockers on the refactor (#75) for this:

[ ] setting up 2 different entry points/modes with 1 for editing projects and 1 for using projects and prefix relevant functions to make clear if they modify telescope-project.txt and telescopoe-workspaces.txt
[ ] expose the path of current project as lua function, so users can define their own keybindings
[ ] encode the behavior of :lcd,:cd and ":nocd" into the picker call functions of telescope-project
[ ] (no real blocker) provide example keybindings

If you want to move things forward, open a PR for 1. so we can discuss the design and iterate further.

@matu3ba I'm not a lua-programmer at all and will have difficulties making time to implement this feature. I can't promise anything, but I'll at least take a look at the sourcecode.

@tim-hilt Hey Tim, lua is fairly easy (you have 2 type of tables and functions).
I am still using this for logging:
local fp = assert(io.open("/tmp/tmpfile", "w")) --DEBUG
for index,tables in pairs(repo_paths) do        --DEBUG
  fp:write(index)                               --DEBUG
  fp:write(", ")                                --DEBUG
  fp:write(tables)                              --DEBUG
  fp:write("\n")                                --DEBUG
end                                             --DEBUG
fp.close()                                      --DEBUG

local fp = assert(io.open("/tmp/tmpfile", "w")) --DEBUG
for index,tables in ipairs(repo_paths) do       --DEBUG
  fp:write(index)                               --DEBUG
  fp:write(", ")                                --DEBUG
  fp:write(tostring(tables))                    --DEBUG
  fp:write("\n")                                --DEBUG
end                                             --DEBUG
fp.close()                                      --DEBUG
-- deleting lines ending with DEBUG :g/.*DEBUG$/del

also a option to disable the find files search instantly after the project has been selected
any news on this one?

also a option to disable the find files search instantly after the project has been selected

This bit already exists, see <C-l> when selecting a project.
As for the other issue, if you use Git for your projects you might find Telescope's builtin for git-files more useful than the regular file finder, and it is scoped to the project root.


[🤔To clarify] - installation process

Currently, the installation/build process is quite complex, and also the Microsoft Graph Developer Proxy Plugin repo is needed as the plugin has direct reference to msgraph-developer-proxy-abstractions.dll in order to implement the proxy interface needed for the plugin.
I should clarify with the dev team what is the recommended approach here and if maybe this should be delivered as NuGet

@sebastienlevert, @waldekmastykarz, @garrytrinder, @gavinbarron as promised I am moving this topic here 😉.
So to sum up what @sebastienlevert already explained to me:

currently there is no support (like any command or plugin store) that will do the installation process of a custom plugin to the mgdp and additional guidance in wiki will be added
the msgraph-developer-proxy-abstractions may be delivered in future as NuGet. Currently it is not and the recommended approach is to... hmm ? ref either project or directly the dll from the mgdp

Is there anything else someone would like to add?

I think Seb meant if you could please create an issue in our repo so that we track the work we need to do 😊  You're bringing up a very good point that abstractions should be published as NuGet. Since we're early on this is not the case, which complicates building plugins, but it's definitely something that we need to have in place. We'll also look at the end to end process of discovery, acquisition and enablement of plugins. This is another thing that we need to iron out before GA. Thank you for bringing it up! 👏

aaaaj, sorry for that @waldekmastykarz and thanks for your response.
I will move those two points as new issues in the mgdp repo 👍 and will link those to this issue so that I may track the progress in future.

monitor:
👉 https://github.com/microsoftgraph/msgraph-developer-proxy/issues/225
👉 https://github.com/microsoftgraph/msgraph-developer-proxy/issues/226

Thanks for re-routing!


Clock has Compass Hud / No disable option for Clock Hud

Mod Version:
1.11.2 - <IP_ADDRESS>
Forge Version:
Forge-<IP_ADDRESS>2
Link to crash log (if applicable):
N/A
Description:
While holding the Clock, you get the Compass bar render like with the regular Compass HUD.
I tried changing the rendering style of the Compass HUD and it only happens with the styles 0 to 3.
The Wind Rose style (4 to 6) doesn't render on the Clock.
I have checked this with the previous version aswell (1.11.2-<IP_ADDRESS>) and it is only with the current version.
As a side note, I also noticed there isn't an option to disable the Clock HUD separately.
Would it be possible to have a config option for it?

Anything in your shield slot?  And there isn't an option to disable the clock separate.  It is tied to the compass.  I could add it though I am curious as to your thinking on it.

Nope, my offhand slot is empty.
Tested with and without a compass and/or clock in my inventory or offhand.
To be sure, I also created a new instance with only Forge and Dynamic Surrounds.
Same results.
And my reason isn't much special, but I would like to disable the Compass HUD and keep having the Clock HUD.
I have a mod (RPG HUD) in my modpack that renders a compass bar to the top of the screen when there is a compass in your inventory.

OK.  Found the bug, and added an option.  Will be in the next BETA.

Thank you so much for the great support!
I really like your mod and I hope more users will use it.

Pushed v<IP_ADDRESS> BETA that has a fix.


Can't configure default resource

Brocoli irods3 catalog client could use a default resource. Same as irodsDefResource in .irodsEnv

Ok. (used only for output files)


adding metadata option

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
feature
add an option for metadata

What is the current behavior? (You can also link to an open issue here)
unable to invoke outputOptions(); because ffmpeg is not accessible. Instead of making it public, I decided to add an option.

What is the new behavior (if this is a feature change)?
metadata option added so that you don't need to another ffmpeg operation for it.

const option = {
    followNewTab: true,
    fps: 30,
    ffmpeg_Path: null,
    videoCrf: 23,
    autopad: {
      color: 'black' | '#35A5FF',
    },
    aspectRatio: '4:3',
    metadata: {
      title: 'Awesome video',
      comment: 'example comment'
    }
  };

let recorder = new PuppeteerScreenRecorder(page, option);

Other information:
Thank you for your great work. 👍

This is a nice additional option to have, would definitely support this merge.
I have also written a patch to enable custom options to outputOptions(), as I needed that myself.

Thank @im0505  for your PR. Looks good to me. Appreciate your contribution.
Apologies for my delayed response on this.


change request: managing form actions

Hi Lukasz,
Thanks for this project!
This is not  real issue, just a change request/proposal. Please labels it accordly, if you want.
Do you maybe foresee to implement also the RASA 2 form particular action management?
If I can I'll contribute, with maybe more examples, e.g. to set a slot to a value, etc.
Minor stuff:

I'd call define method with the alias on, maybe more js-idiomatic.
I'd add info for beginners on readme, about how to run stuff (rasa run actions, etc.)

Please close this issue if you want.
Thanks again
Giorgio

Hello @solyarisoftware,
Thank you for taking time to provide feedback. I'll take in consideration adding the Rasa Forms custom actions (I'll look into it maybe as soon as this weekend). With regards of the other notes:

I agree with you that on is more js-style, might as well change it in the next version
Since the custom connector is the only thing actually needed to connect the node-action-server with Rasa itself, I thought I'd keep the README as simple as possible to get started. The user I had in mind when doing this had already used rasa before. However, I can add a brief explanation of how to start everything from scratch 👍

Thanks for the feedback!
About the form actions server, maybe the repo https://github.com/rohitnairtech/nodejs-formaction-sdk-rasa could give inspiration but it's old (previous RASA 2.0).
In general, a README with a full running example could help users.
You could also just supply a domain.yml and a README.md in the directory of your example examples/http: https://github.com/Lykos94/rasa-node-action-server/tree/master/examples/http
# domain.yml
version: "2.0"

nlu:
- intent: greet
  examples: |
    - Hi
    - Hey!
    - Hallo
    - Good day
    - Good morning
- intent: ask_weather
  examples: |
    - what's the weather
    - what's the weather in [Genoa](city)
    - i want to know the [milan](city) weather of today
    - i want to know the weather of today
    - tell me the [Vienna](city) weather forecast
    - hows the weather today
- intent: city_inform
  examples: |
    - milan
    - Genoa
    - rome
    - Berlin
    - Vienna
    - Turin
    - Naples
    - london
    - new york
    - Dar Es Salaam
    - Nairobi
    - Mogadishu

intents:
  - greet
  - ask_weather
  - city_inform

entities:
  - city

slots:
  city:
    type: text
    influence_conversation: true

responses:
  utter_greet:
    - text: "Hey there!"
  utter_ask_city:
    - text: "which city you want to check for?"
  utter_default:
    - text: "Sorry, I didn't get that, can you rephrase?"

actions:
  - weather_city_today_action

stories:

 - story: 1. greet and weather with city specified
   steps:
   - intent: greet
   - action: utter_greet
   - intent: ask_weather
   - action: weather_city_today_action

 - story: 2. greet and weather with city NOT specified
   steps:
   - intent: greet
   - action: utter_greet
   - intent: ask_weather
  - slot_was_set:
    - city: None
   - utter_ask_city
   - city_inform
   - action: weather_city_today_action

config:
  store_entities_as_slots: true

session_config:
  session_expiration_time: 60  # value in minutes
  carry_over_slots_to_new_session: true

BTW, please double check if syntax of the above example is formally correct and it make sense.


Implement .view() method of jax.numpy arrays

This is really exciting! I'm guessing this won't be in the cards in the near future, if ever, but I remain hopeful :)

Yeah, view() here is a bit of a misnomer, because the result of this is not a view but a transformed array. But the casting behavior of np.ndarray.view() is something I've been missing...

Haha, you are forgiven; although what is the hard part about providing a true view? Is it the support for TPU/GPU? I guess there would need to be a much more holistic approach to keep track of references to memory.


Add DevOps / continuous release / production environment / code & release process / ...

I can't find the proper term for that, but it would be awesome to add an awesome list of tools and resources for the "code & release process" (which is usually called DevOps these days).  This would include topics related to source control management, continuous integration, testing, provisioning, deployment, containers, release orchestration, cloud, analytics, monitoring, security, collaboration... and many more?

What about this awesome list : https://github.com/AcalephStorage/awesome-devops
You can contribute if you want to add stuff.


🐛 Fix incorrect column name

Required items, please complete
Change(s): Two analytics were referring to the incorrect column, using uppercase URL instead of the projected Url
Reason for Change(s): Fix the broken analytics
Version Updated: ✅
Testing Completed: ✅

Thanks @pemontto !


Merge remote-tracking branch 'luigirizzo/master' (MERGE WITH NEW CENTOS RPMS)

See https://jira.logrhythm.com/browse/NM-1139
NOTE TO REVIEWER: To test this build you will need to install the latest CentOS rpms which Dan has staged in artifactory under the 3.8.1 directory. I tested it on a development VM.

@KjellKod, this is on-hold until after I have tested MTU size 2048 for netmap. If that is successful, there will be a second pull request in nm-core to make that change.


feat: option to add Gitpod configuration

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[ ] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[ ] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[x] I have added a convincing reason for adding this feature, if necessary

Other information
Nobody requested it specifically, but here it is, an option in create-tauri-app for creating a basic Gitpod configuration. I've seen enough people talk about it that I decided to develop it. Not sure if this is how we'd want to develop such an option, but in my testing at least it works like a charm.

Closed because I noticed #297 is accidentally based on my dev branch instead of a feature branch. So for simplicity sake it'd be better if that one gets merged first so I can rebase this branch, then reopen the PR

Not gonna lie, I am kina on the fence on this one. The option is definitely cool but only a very small subset of users is going to need it, so my proposal is to keep it as a CLI option only.


🐛 [Firefly III] Default email and password

Description
After installing its asking for a Email and password
Reproduction steps
1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

Addon Logs
dd-on: Firefly iii
 A free and open source personal finance manager
-----------------------------------------------------------
 Add-on version: 6.1.22
 You are running the latest version of this add-on.
 System: Home Assistant OS 13.2  (amd64 / qemux86-64)
 Home Assistant Core: 2024.11.2
 Home Assistant Supervisor: 2024.11.2
-----------------------------------------------------------
 Please, share the above information when looking for help
 or support in, e.g., GitHub, forums
-----------------------------------------------------------
 Provided by: https://github.com/alexbelgium/hassio-addons
-----------------------------------------------------------
/etc/cont-init.d/00-global_var.sh: executing
APP_KEY='******************************'
CONFIG_LOCATION='/config/addons_config/fireflyiii/config.yaml'
DB_CONNECTION='sqlite_internal'
silent='true'
Timezone set from Etc/UTC to America/New_York
/etc/cont-init.d/01-config_yaml.sh: executing
Setting permissions for the config.yaml directory
Load environment variables from /config/addons_config/fireflyiii/config.yaml if existing
If accessing the file with filebrowser it should be mapped to /homeassistant/addons_config/fireflyiii/config.yaml
---------------------------------------------------------
Wiki here on how to use : github.com/alexbelgium/hassio-addons/wiki/Add‐ons-feature-:-add-env-variables
... no env variables found, exiting
/etc/cont-init.d/01-custom_script.sh: executing
Execute /homeassistant/addons_autoscripts/fireflyiii.sh if existing
Wiki here : github.com/alexbelgium/hassio-addons/wiki/Add-ons-feature-:-customisation
/etc/cont-init.d/99-run.sh: executing
[23:49:10] INFO: Backuping APP_KEY to /config/addons_config/fireflyiii/APP_KEY_BACKUP.txt
[23:49:10] WARNING: Changing this value will require to reset your database
[23:49:10] INFO: Defining database
[23:49:10] INFO: Using built in sqlite
[23:49:10] INFO: Defining upload folder
[23:49:10] INFO: Please wait while the app is loading !
[23:49:11] WARNING: Silent mode activated. Only errors will be shown. Please disable in addon options if you need to debug
> Illuminate\Foundation\ComposerScripts::postAutoloadDump
[Wed Nov 20 23:49:27.666893 2024] [mpm_prefork:notice] [pid 299:tid 299] AH00163: Apache/2.4.62 (Debian) configured -- resuming normal operations
[Wed Nov 20 23:49:27.666951 2024] [core:notice] [pid 299:tid 299] AH00094: Command line: 'apache2 -D FOREGROUND'

Architecture
No response
OS
HAos

Hi, it is likely that you already had an account. Delete the folder /homeassistant/addons_config/fireflyiii using the add-on filebrowser

That helped me.
Thanks so much


Update iot-hub-devguide-security.md

Replace SHA1 thumbprint in C# sample with a SHA256 thumbprint.

@rajeevmv : Thanks for your contribution! The author, @dominicbetts, has been notified to review your proposed change.

@rajeevmv - thanks for the contribution.

#sign-off


Update THC Prepare to use QROAMClean

Use QROAMClean for THC prepare. Hit some issues with resource estimation notebook and fixes a bug with LCUBlockEncoding junk registers which may be sided (at least I think this is the desired behaviour).
BlackBoxPrepare does not work with sided registers either so I removed this from the resource estimation notebook (xref #1381)
I updated the call graph slightly to remove CirqGateAsBloq references and makes sure the decomposition matches the call graph.
I removed the PrepareTHC circuit diagram as the new QROAMClean bloq results in way too many junk registers making it unsightly (or really it just fails by default: #1380)

xref #1381 and #1380

summoning @tanujkhattar


Make annotation tool determine current container, when creating a container annotation

We need to resolve this hack here, as we have different containers in different projects.
https://github.com/substance/substance/blob/master/src/surface/annotation_tool.js#L193-L194
Possible solutions:

require container annotations to have a getContainerName() method. however this would mean we can not use the same annotation in multiple containers.
in Annotation Tool we have a surface instance. Can we ask that for the container name?

I will try to solve myself first, and let you review.

Pls review: https://github.com/substance/substance/commit/0c93e84cbd7a579a324812e5ed83439d8238c0ea


Write City of Boston Data utils

functions for parsing the data, aggregating based on individual candidates, precincts, etc

[x] Get display names for precinct, ward
[ ] Convert ward/precinct object into an array of candidates
[ ] ???


again

Please provide 2 different pull requests.
one - only first commit "this is my first code in python"
second - both others "today I do the requirements file and dataframe, please read this file first"


server: Use tracing_subscriber's TestWriter in tests

… so that logs are only printed for failing tests.

Does this help with my desire to log response bodies on status code mismatch? You mentioned some extra config was needed to get tracing to show up in tests (from the test code itself maybe?).
Either way, this seems good. Less output is better.

This is just for the tracing subscriber that's installed for the server task. If the test code itself emits tracing events, those currently go nowhere because there is no subscriber. I can post a follow-up PR to change that (make the tests themselves also use TestWriter, but no extra otel stuff like we have for the server).


chore: update toolchain

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#305  👈
#304
#303
#302
#301
#300
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite


didyoumean: 1.1.0 -> 1.1.3

Description of changes
https://github.com/hisbaan/didyoumean/releases/tag/v1.1.3
Closes #184088.
Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 186557 run on x86_64-darwin 1

1 package built:

didyoumean


Select. 'PortalDispatchContext' cannot be null

Hello.
When I use Select.Viewport inside Select.Content I get the error:
'PortalDispatchContext' cannot be null, please add 'PortalProvider' to the root component.

Is it a bug or I missed something?

How'd you end up solving this?

add 'PortalProvider' to the root component.

import {
  PortalProvider,
} from '@tamagui/portal';

etc...

+1 still doesn't work for me despite adding PortalProvider in the root