Fix wrong path while merging a pull request

Description
Our testing revealed this issue which needs to be fixed. We probably need to rethink the strategy to ensure all paths for a certain repository are correctly prefixed as the current system cannot guarantee that.
This also removes a log message which is too verbose at this point.

Also I feel the need to say that the red shade for the bug label is very pretty


chore(ci): switch from next to latest release tag

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update 03_classification.ipynb

When I tried to run cell 2, I got the error "KeyError: "None of [Int64Index([    1,    21,    34,    37,    51,    56,    63,    68,    69,\n               75,\n            ...\n            59910, 59917, 59927, 59939, 59942, 59948, 59969, 59973, 59990,\n            59992],\n           dtype='int64', length=60000)] are in the [columns]". Apparently openml is returning a Pandas Dataframe here whereas it must have used to return a Numpy array; I had to make this change to get it to work correctly.

Hi @jdaviestx ,
Thanks for the PR. Since Scikit-Learn 0.24, fetch_openml() returns a Pandas DataFrame by default, instead of a NumPy array. Just pass as_frame=False to that function and everything will work fine. I think that's the simpler solution.
Hope this helps.


Peril code grouping

Explore whether adjustment to mapping file can address problems found in testing, with handling grouped peril codes in one model format (AIR) and separate peril codes in another (OED).
Documentation should highlight this as a known issue for this translation.

@tristanlloyd @benhayes21 Please could you add an example of the codes and the transformation direction requiring code splitting / direction requiring combination for this issue, to assist understanding the logic required to implement it in the mapping file. Thanks

The best reference is https://docs.air-worldwide.com/Database/CEDE/9.0/topic137.html
Grouped peril code: PWS equates to PWH+PSH Windstorm
PEQ  equates to PES+PFF+PTS+PLS+PSL All Earthquake perils
PFH equates to TS PFL+PSH Inland and Coastal Flood
Is the best way to deal with this, to include the original and destination in the peril code listing, i.e. transform the grouped code to string in the same way as we transform a single code to a single code (is any other logic actually needed?

Spliting these into multiple transformations is the way to go here, something like
transformation: 1
when: search(LayerPeril,‘PWH’)

is correct, is this not working?
The reason this wouldn't work
replace(search(LayerPeril,‘PWH’),’TRUE’, 1,’FALSE’,’’)

is that the first argument to replace must be a string and search returns a boolean.

Running this on a basic test does not appear to work as expected. Requires tests to confirm correct structuring of transformation using multiple where clauses, and documentation of how to perform these types of conditional statements with clear examples.
Issues found in running tests using the one-wat-transformation:

Using 'tab' to indent the where: clause, returns error: found character '\t' that cannot start any token
in "/Users/stufraser/OEDtransform/examples/one-way-transformation/A-B.yaml"

Using where: b >= 10 return error: Expecting: {'GT', 'IS', 'PLUS', '__ANON_0', 'STAR', 'AND', 'MINUS', '__ANON_2', 'SLASH', 'GTE', 'LT', '__ANON_1', 'OR', 'LTE'}

Using where: b = 10 return error: Expecting: {'GT', 'IS', 'PLUS', '__ANON_0', 'STAR', 'AND', 'MINUS', '__ANON_2', 'SLASH', 'GTE', 'LT', '__ANON_1', 'OR', 'LTE'}

Using where: b == 10 return error: Expecting: {'GT', 'IS', 'PLUS', '__ANON_0', 'STAR', 'AND', 'MINUS', '__ANON_2', 'SLASH', 'GTE', 'LT', '__ANON_1', 'OR', 'LTE'}

Using where: b GTE 10 return error: Expecting: {'GT', 'IS', 'PLUS', '__ANON_0', 'STAR', 'AND', 'MINUS', '__ANON_2', 'SLASH', 'GTE', 'LT', '__ANON_1', 'OR', 'LTE'}

Using where: b IS 10 return error: Expecting: {'GT', 'IS', 'PLUS', '__ANON_0', 'STAR', 'AND', 'MINUS', '__ANON_2', 'SLASH', 'GTE', 'LT', '__ANON_1', 'OR', 'LTE'}

Using when: b is not in [10] works but is not as intuitive as using where: b <> 10, for example

Running the following, the transformation succeeds with no error, but only the 1st transformation is completed; second one is blank.
- transformation: b + 10
when: b is not in [10]
- transformation: 99
when: b is 10
Running the following, the transformation succeeds with no error, but all values in d return 99
- transformation: 99
when: b is 10
- transformation: b + 10
when: b is not in [10]


Collision filtering: "from" after "to" results in endless loading spinner

What Happened
From user report:

I was trying to filter the collision data by date. By mistake, I entered a recent date for "from" and an old date for "to". Instead of telling me the input was invalid, MOVE is trying to process the filter. The loading does not stop until I cancel the filter.

What Should Happen
At least one of these three things should probably happen (pending design decision):

the user should be unable to set "from" after "to" (i.e. these dates are disabled);
there should be a clear, readable error message here;
"from" / "to" should be automatically swapped if this happens.

To Reproduce
Steps to reproduce the bug:

Go to any location with collisions, and "View Data";
Click "Filter" in the collisions section;
Set a date range with "from" after "to".

This has since been addressed by the new Global Filters implementation - marking closed.


opt_type_constraint_case getting stuck

the function opt_type_constraint_case(ast) is getting stuck at "Fix typename case from 'string' to 'String'"
It looks like the Element isn't getting replaced properly

Do you have a sample?

Here is a sample - I am having the same issue.
ADLastLogonToCSV.txt
I have taken your code and basically merged certain sections with my own - I am first converting .ps1 into raw AST, then calling optimizer in same way as yourself - I am getting infinite messages like below;
'[02-17-2023 11:52:51] [DEBUG] Fix typename case from 'string' to 'String''
This happens on a few files and I'm not sure why but it is getting stuck in a loop - I am trying to debug as well.

There was a problem with opt_type_constraint_case which change string to String and opt_type_constraint_from_convert which replace the type of TypeConstraintAst nodes with the static type of ConvertExpressionAst, in this case String to string.
fixed with 67e713c847b8cf79e54ede91227f3817bdefe32e

You are a boss - really, fixing a bug within 4 hours of having a sample is amazing, props to you!
I am using your tool in a combined ML workflow to analyze suspicious PowerShell files with both text classification and DNN learning - thank you very much for such an interesting approach to AST simplification/analysis.
(I confirmed that your patch mitigates the issue I was experiencing and this bug can probably be closed)


prevent crash on non Linux-based OS

related to https://github.com/lettuce-io/lettuce-core/issues/1786
on the latest version of netty-incubator-transport-native-io_uring class IOUring may throw an exception during initializing.
For example on Mac OS Redis connection doesn't work because of this exception https://github.com/netty/netty-incubator-transport-io_uring/blob/main/src/main/java/io/netty/incubator/channel/uring/Native.java#L277

hi @mp911de, sorry to bother you.  Could you take a look, please?
at the neighbor issue you told

we intend to pick up the latest io_uring version with our next release

this is, probably, the first issue you would meet, if you develop on Mac.
thank you in advance.

I'm somewhat confused why io_uring specifically causes issues. The native support code looks pretty similar to epoll and kqueue which do not seem to interfere. As a workaround, please do not include io_using when developing on unsupported platforms.
If we want to address the issue we need to make sure that we have a consistent behavior with the other libraries as well.

thanks, @mp911de for your feedback!
To be honest, I was surprised when I first time saw the code for checking the availability for IO_uring, I expected lettuce would catch any exception or even  Throwable, but for some reason, it caught only ClassNotFoundException...
Why? During initialization, everything might happen. I know other libraries throw nothing in static blocks, but, in my opinion, we were just lucky so far.
I can make the same adjustment for other providers if you bless =)

I was able to reproduce the issue with version 0.0.7. In version 0.0.8 that issue is fixed and handled properly within netty. So all that is left to do is upgrading to 0.0.8.


Search Modal shows non-snippeted value and search results page shows snippeted value, when it should be the other way around

Reported by @aandis during office hours
Showing the non-snippeted value in the search modal ends up cutting off highlighted terms, which leads to confusing UX

@aandis I've pushed out a fix for this in v0.10.0-0 of docusaurus-theme-search-typesense. Could you give it a shot now?

Looks good. Thanks!


Allow running go to definition on export and other keywords

🔎 Search Terms

go to definition

🕗 Version & Regression Information
Not a regression
⏯ Playground Link
No response
💻 Code
export class A {

    private z: string;

    readonly x: string;

    async a() {  }

    override b() {}
}

Try running go to definition or go to type definition on any of the keywords before the symbol name (export, private, override, async)
🙁 Actual behavior
No definitions fourd
🙂 Expected behavior
It should work the same as running go to definition on the symbol name
Additional information about the issue
No response

This is something that we run into with copilot's symbols links. I'm working on trying to make sure we always talk about the actual symbol's location instead of the range of the entire definition but thought it was worth bringing up for TS too


Ignore for specific path not working

According to documentation you can specify an array of glob patterns for certain path to be ignored. In practice, this does not work. I tried both with plain strings of glob patterns and with objects with a glob property.
This same patterns works if I include them in the options ignore array, so they should be right.
Here is my config:
  // copy custom static assets
    new CopyWebpackPlugin(
      [{
        from: 'node_modules/monaco-editor/min/vs',
        to: 'vs',
        ignore: [
          {glob: '**/language/html'},
          '**/language/css'
        ]
      }]),

Sorry, it was my bad, bad glob


httpyac cli does not exit when invoked with the same file twice

Steps to reproduce:

Create test.http file with following content:

POST https://httpbin.org/anything

Invoke httpyac from command line:

httpyac -a test.http test.http

The execution won't finish.
Note: Not that helps much, but this worked in a previous version of at least 7 months ago. I have several scenarios like the following:
httpyac -a file1.http file2.http file1.http

in which file1 queries my backend, then I execute file2 (to insert and change the assertion variable) and execute again the file1 to check.
Thanks a lot

seems to be an issue in lib hookpoint in setUniqueId. This method is stuck in an infinite loop.

Issue is fixed with v5.10.2. I would never have thought of using the same file twice.


Make Alpine instructions Dockerfile commands

I thought it'd be a little more clear if the instructions for Alpine were Dockerfile commands like in the "Using Tini" section.

Thanks! That's a great idea :).
The README.md file needs to rebuilt from the template (I'll add a test for this in CI)  after changing the template — can you run cmake -B/tmp -H. from the source directory to do so?
If you don't have cmake installed, etc., let me know. I'm happy to amend your commit with that change.
Thanks again!
Cheers,

I was wondering if I needed to do that or if it was done by travis after a merge.  It's done now.

Thanks!


Use sensible buffer sizes for Windows named pipes

When creating a Windows named pipe one has to specify the buffer sizes. Previously we were specifying the maximum size, however this is unwise:
The MSDN docs say:

The input and output buffer sizes are advisory. The actual buffer size reserved for each end of the named pipe is either the system default, the system minimum or maximum, or the specified size rounded up to the next allocation boundary. The buffer size specified should be small enough that your process will not run out of nonpaged pool, but large enough to accommodate typical requests.

So this changes the values to 64k outbound and 16k inbound.

bors merge


maven target does not work

Hello,
I have add to my multimaven project into the root pom.xml the plugin with
    <build>
        <plugins>
            <plugin>
                <groupId>com.github.ferstl</groupId>
                <artifactId>depgraph-maven-plugin</artifactId>
                <version>4.0.1</version>
            </plugin>
        </plugins>
    </build>

I cannot run the plugin with mvn depgraph:aggregate I must run it with mvn com.github.ferstl:depgraph-maven-plugin:aggregate . Is it possible to run the plugin with the short version of the plugin?

Hi @flashpixx
For that, you need to edit your settings.xml:

com.github.ferstl

For further details, see the section "Run on the Command Line" in the README.md.
Cheers,
Stefan

I know this, but I would like to have got a project-based setup and not modifying my global settings.xml

This is not possible, because Maven does only look for plugins with the groupId org.apache.maven.plugins and org.codehaus.mojo by default: https://maven.apache.org/guides/introduction/introduction-to-plugin-prefix-mapping.html .
So unless you don't add com.github.ferstl to your maven settings, you can't use the "depgraph" prefix.
If you can't add the pluginGropu to your global Maven settings, maybe you can use a user-specific ~/.m2/settings.xml for that. You can also take a look at this GitHub project which allows you to define a project-specific settings.xml: https://github.com/gzm55/project-settings-maven-extension . I haven't tried if that works for pluginGroups.


Upgrade Go to 1.21.2

Proposal
New Go minor point release got released recently: https://groups.google.com/g/golang-dev/c/2C5vbR-UNkI/m/L1hdrPhfBAAJ
As a best practice let's upgrade our toolchain to Go 1.21.1
Will do it tomorrow/during PromCon this week, unless somebody has time to start the process now (help wanted!)
We can do patch release for LTS https://github.com/prometheus/prometheus/releases/tag/v2.37.9 and latest https://github.com/prometheus/prometheus/releases/tag/v2.47.0

/assign

Hello @bwplotka I would like to work on this issue should I change the version of Go now

LTS is 2.45 now

Hello I'd love to see this updated as it resolves several CVE:

CVE-2023-39318
CVE-2023-39319
CVE-2023-39320
from: https://groups.google.com/g/golang-announce/c/UXJQvKffcao

see https://groups.google.com/g/golang-announce/c/2dWHvJVFA9s/m/lF9Srr_QAAAJ

I'd like to take on updating to Go 1.21.2 when it's released on October 5th. Any specific guidelines I should follow?

Please review.
Thanks

Next one incoming:

We plan to issue Go 1.21.3 and Go 1.20.10 during US business hours on Tuesday, October 10.
These minor releases include PRIVATE security fixes to the standard library, covering the following CVE:
CVE-2023-39325
Following our security policy, this is the pre-announcement of those releases.

https://groups.google.com/g/golang-announce/c/XIujkcMT56U/m/zDqbx44vAwAJ

@cliffcolvin https://github.com/prometheus/prometheus/releases/tag/v2.47.1 was built with 1.21.1.

Nice, let's wait for Go 1.21.3 then.
Also for @Gokul2003g (thanks for your PR!) - Go module version does not matter here. What we have to change is the Go version Prometheus is built with e.g. https://github.com/prometheus/prometheus/blob/main/.promu.yml#L4 and perhaps https://github.com/prometheus/prometheus/blob/main/.github/workflows/ci.yml#L14
We probably should add some docs for contributors on how to upgrade Go version for our CI.

Note those only specify major version; the latest patch is picked automatically.
Generally we don't change anything to get the latest Go patch version.


Mobile staging

Coverage remained the same at 90.074% when pulling e1271666fe101865da0e886490c235411c3bec4e on mobile-staging into 7011ad7569174cee49a8112a510ca8f971d6a759 on master.


Remove confusing action button syntax and replace with events

Expected Behaviour
Currently you can invoke a function from a push on Android by passing in the full address to the function as part of the push. This is actually pretty hard to get across to people and for awhile now you could just register a custom event handler for your action button. We should make it so only the event driven way works.
Actual Behaviour
Both function path and event work.
Reproduce Scenario (including but not limited to)
Steps to Reproduce
See: https://github.com/phonegap/phonegap-plugin-push/blob/3bce4d4c81e738cb6557f1267807fb89406378a0/docs/PAYLOAD.md#action-buttons

Duh, I should make this consistent on iOS.


[DBZ-267] Enable images to run on OpenShift Online

Merged.  Thanks a lot, @jpechane!


Removing retries from classic output.

correct me if i wrong
but instead of figure out why our test need retries in this part of code
we just change code to remove retries from logs, but still retries?
but this is have no sense and smells bad


WRP-11279: Fix broken links referencing other props in documentation

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[ ] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
There are broken links that refer to other props in the documentation.
Broken links

https://enactjs.com/docs/modules/sandstone/Checkbox/#CheckboxBase.children
https://enactjs.com/docs/modules/sandstone/CheckboxItem/#CheckboxItemBase.icon
https://enactjs.com/docs/modules/sandstone/CheckboxItem/#CheckboxItemBase.indeterminateIcon
https://enactjs.com/docs/modules/sandstone/FlexiblePopupPanels/#Panel.nextButton
https://enactjs.com/docs/modules/sandstone/FlexiblePopupPanels/#Panel.prevButton
https://enactjs.com/docs/modules/sandstone/FormCheckboxItem/#FormCheckboxItemBase.icon
https://enactjs.com/docs/modules/sandstone/FormCheckboxItem/#FormCheckboxItemBase.indeterminateIcon
https://enactjs.com/docs/modules/sandstone/Icon/#IconBase.children
https://enactjs.com/docs/modules/sandstone/ImageItem/#ImageItemBase.placeholder
https://enactjs.com/docs/modules/sandstone/Picker/#PickerBase.changedBy
https://enactjs.com/docs/modules/sandstone/Picker/#PickerBase.decrementIcon
https://enactjs.com/docs/modules/sandstone/Picker/#PickerBase.incrementIcon
https://enactjs.com/docs/modules/sandstone/Popup/#Popup.onShow
https://enactjs.com/docs/modules/sandstone/RangePicker/#RangePickerBase.changedBy
https://enactjs.com/docs/modules/sandstone/RangePicker/#RangePickerBase.decrementIcon
https://enactjs.com/docs/modules/sandstone/RangePicker/#RangePickerBase.incrementIcon
https://enactjs.com/docs/modules/sandstone/Sprite/#Sprite.src
https://enactjs.com/docs/modules/sandstone/TabLayout/#Tab.icon
https://enactjs.com/docs/modules/sandstone/TimePicker/#TimePicker
https://enactjs.com/docs/modules/sandstone/TooltipDecorator/#TooltipDecorator.tooltipMarquee
https://enactjs.com/docs/modules/sandstone/TooltipDecorator/#TooltipBase.marquee
https://enactjs.com/docs/modules/sandstone/WizardPanels/#Panel.PrevButton
https://enactjs.com/docs/modules/sandstone/WizardPanels/#Panel.nextButton

Private broken links

sandstone\Button\Button.js

64,44
67,11
71,12
79,26
80,29
84,12

sandstone\Input\InputFieldDecoratorIcon.js 30,12
sandstone\internal\Picker\Picker.js

735,9
736,6

sandstone\VideoPlayer\VideoPlayer.js 1655,52

Resolution
Fix links to broken props in the documentation
Additional Considerations
Links
WRP-11279
Comments
Enact-DCO-1.0-Signed-off-by: Hoeun Ryu<EMAIL_ADDRESS>

The PR has been updated to fix additional broken links

https://enactjs.com/docs/modules/sandstone/FlexiblePopupPanels/#Panel.nextButton
https://enactjs.com/docs/modules/sandstone/FlexiblePopupPanels/#Panel.prevButton
https://enactjs.com/docs/modules/sandstone/Picker/#PickerBase.incrementIcon
sandstone\Button\Button.js 64,44, 67,11, 71,12, 79,26, 80,29, 84,12
sandstone\Input\InputFieldDecoratorIcon.js 30,12
sandstone\internal\Picker\Picker.js 735,9, 736,6
sandstone\VideoPlayer\VideoPlayer.js 1655,52

The PR has been updated to fix additional broken links

sandstone\internal\Picker\Picker.js 798,10, 834,10
sandstone\VideoPlayer\VideoPlayer.js 1665,14, 1691,11

The PR has been updated to fix typo

Button/Button.js: 64 (Icon -> icon)


bug: pip_url not consistent between .lock file and meltano.yml

Meltano Version
2.20.0
Python Version
3.8
Bug scope
CLI (options, error messages, logging, etc.)
Operating System
Linux - Ubuntu 22.04
Description
Given a plugin in a project and its upstream definition updated with a different pip_url, when updating the plugin in the project with
meltano lock --update <plugin>

the pip_url for the plugin entry in the meltano.yml is not updated (i.e a discrepancy between the plugin .lock file and meltano.yml entry).
This raises the following questions for me (not sure if answers to these are documented anywhere):

Is Meltano using the pip_url from the plugin .lock file or the meltano.yml entry during install?
Why is pip_url included in the plugin meltano.yml entry by default?

Might relate to #8176
Code
No response

Is Meltano using the pip_url from the plugin .lock file or the meltano.yml entry during install?

It should be using pip_url from meltano.yml. Otherwise there's a bug.

Why is pip_url included in the plugin meltano.yml entry by default?

The idea of having it in meltano.yml is that users can easily identify it and override it with a fork, for example. Though I don't know common of a need that is.

Yeah, so can confirm Meltano is using the pip_url from the meltano.yml during install like you said, so no issue there. I was mainly asking because when we pinned tap-shopify to v0.2.0 on Meltano Hub, a user was asking about how to update and I wasn't sure which pip_url would be used, since both point to the same commit currently.
I think #8176 could address this, but I'm not sure in what situation you would use meltano lock --update in then?

I think #8176 could address this, but I'm not sure in what situation you would use meltano lock --update in then?

At the moment it only serves to update plugin settings and capabilities, but ideally the pip_url should get updated too (like https://github.com/meltano/meltano/issues/6416)

Just caught up with Office Hours. I'm not really sure what the consensus was on pinning plugins to specific versions on Meltano Hub in the end - we do seem to be the only ones actively doing this... Although, the dbt utilities are pinned (e.g. dbt-postgres) and I assume whenever these are updated on the Hub to reference new versions, someone else will run into this same issue?
Our motivation for this is purely that it's more stable for users to be pinned to a specific version, and they can update whenever - manually or through whatever means Meltano provides. I agree that it's good to have the pip_url in the meltano.yml for clarity, but the fact that it can be different from the .lock file without a user having chosen to override it themselves is where I think this issue stems from.

My main problem with an unpinned pip_url for a plugin definition on the Hub is that there is no guarantee that the plugin won't break (i.e. as a user, I just want a version that works). Currently, there is no coupling between the version of the plugin that actually gets installed and the defined settings and/or capabilities - it could be invalided as soon as a breaking change is made and pushed to master/main, so the Hub definition must be kept up-to-date (or the user has to start overriding/redeclaring settings/capabilities in the meltano.yml). I realise that there is also a deeper issue here around the plugin not breaking, concerning dependencies of the plugin itself (discussed more in #6416).
Also, if a plugin's pip_url is overridden (pinned) in the meltano.yml, meltano lock --update becomes a useless (and potentially dangerous) command , since the Hub definition might be or could become incompatible with that pinned version.

@ReubenFrankel I was asking the question of whether its the hub/meltano's responsibility to pin the plugin vs the user's responsibility. You make a good point that its mixed right now since your variants and dbt imply that its the hub's responsibility but the majority of the other hub plugins offload that responsibility to the user.
I could see it making sense to update the pip_url in the meltano.yml anytime the lockfile is updated. Even if the hub pip_url isnt pinned, if someone asks to get an updated lock file then they'd probably also want the updated plugin code to go with it since the hub definition usually tracks the latest version. If we dont update the pip_url during a lock --update and the user has it pinned then its likely that new settings will be included in their lock file that arent supported by their pinned version. The only downside to this is that in most cases (excluding matatika and dbt) plugins get unpinned after a lock --update, so the user will need to remember to update and pin them again, which actually might be a good process to encourage.

I could see it making sense to update the pip_url in the meltano.yml anytime the lockfile is updated. Even if the hub pip_url isnt pinned, if someone asks to get an updated lock file then they'd probably also want the updated plugin code to go with it since the hub definition usually tracks the latest version.

This is actually how #8176 is implemented at the moment with meltano add --update, which is why I was asking about the overlap with meltano lock --update.

If we dont update the pip_url during a lock --update and the user has it pinned then its likely that new settings will be included in their lock file that arent supported by their pinned version. The only downside to this is that in most cases (excluding matatika and dbt) plugins get unpinned after a lock --update, so the user will need to remember to update and pin them again, which actually might be a good process to encourage.

In my mind, this feels a bit off because it would mean meltano lock operating outside the scope of just .lock files (as the command name implies). I'm curious on how you think it might be a good process for a user to re-pin after an update though, since it might be relevant to #8176.


Please help me out to solve the error

This is the code
from imageai.Detection import VideoObjectDetection
import os

execution_path = os.getcwd()

detector = VideoObjectDetection()
detector.setModelTypeAsRetinaNet()
detector.setModelPath(os.path.join(execution_path, "resnet50_coco_best_v2.0.1.h5"))

custom_objects = detector.CustomObjects(car=True, motorcycle=True, truck=True)

video_path = detector.detectCustomObjectsFromVideo(
            custom_objects=custom_objects,
            input_file_path=os.path.join(execution_path, "cars.avi"),
            save_detected_video=False,
            frames_per_second=30,
            log_progress=True)

print(video_path)

and this is the error
An error occured. It may be that your input video is invalid. Ensure you specified a proper string value for 'output_file_path' is 'save_detected_video' is not False. "
ValueError: An error occured. It may be that your input video is invalid. Ensure you specified a proper string value for 'output_file_path' is 'save_detected_video' is not False. Also ensure your per_frame, per_second, per_minute or video_complete_analysis function is properly configured to receive the right parameters.

@shivam9804 Do you have the latest version installed v.2.1.5?

Kindly install via the latest version via pip pip3 install imageai --upgrade
Also, ensure the input video is in the specified path

@OlafenwaMoses  Thank you for the response, it solved my problem.

Am happy to know that.


Alpha 5+ breaking changes

Various errors when using alpha 5+

Was going to replace /addons/ with latest PR versions, but realised some custom changes, so these fixes are project specific. At some point worth replacing entire /addon/ with repo defaults and working through the errors.


Cleanup using statements and namespaces in C# code

PR Summary
This will make the diff in future PR's cleaner.
This was done using Visual Studios new code cleanup feature that runs the formatter on all files and removes unused namespaces
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .cs, .ps1 and .psm1 files have the correct copyright header
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed and/or updated documentation
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

After a self-review, it seems the tool has left some unmerged changes in some namespaces like e..g in CompatibilityProfileLoader.cs


NIT-526 IAPS pre-prod deployment

closing as opa checks are not reporting status. will open a new pr with the same changes


Auto chart height resize not working

When the window resizes, I see the chart width update automatically, but the height does not change. I think this is because of the $$.selectChart.style('max-height', $$.currentHeight + "px"); bit on line 880. When I comment out this line, the chart height updates correctly.
This line forces the .c3 container element to have the max height $$.currentHeight, but on the window resize event, $$.currentHeight is set based on the height of that same element (see getCurrentHeight() and getParentHeight()), which means it gets stuck at the current value.
I can submit a pull request, but it's just a one line change.

Hello, I have the same issue. Looks like this is still in master branch. Has the change been made?

I'm having this issue as well

👍

any update on this ??

closing in favor of #1450

Do we have an update on this?

When the window resizes, I see the chart width update automatically, but the height does not change. I think this is because of the $$.selectChart.style('max-height', $$.currentHeight + "px"); bit on line 880. When I comment out this line, the chart height updates correctly.
This line forces the .c3 container element to have the max height $$.currentHeight, but on the window resize event, $$.currentHeight is set based on the height of that same element (see getCurrentHeight() and getParentHeight()), which means it gets stuck at the current value.
I can submit a pull request, but it's just a one line change.

Commenting out this line did not solve the issue. Is there another solution to this issue?


[INF] Switch from Sphinx to MkDocs

This resolves #894. The majority of votes from active maintainers is to move to mkdocs.
I am probably going to need a bit of help with this PR because of the sheer number of docstrings to reformat.
As promised in a comment below, these are the number of functions in each of the files.

[ ] janitor/timeseries.py 5
[ ] janitor/__init__.py 1
[ ] janitor/errors.py 0
[x] janitor/biology.py 1
[ ] janitor/functions.py 79
[ ] janitor/engineering.py 1
[ ] janitor/ml.py 1
[ ] janitor/io.py 1
[ ] janitor/chemistry.py 4
[ ] janitor/math.py 8
[ ] janitor/finance.py 8
[ ] janitor/utils.py 74

@ericmjl  Let me know where I can help.

@loganthomas and @samukweku I updated my original post! functions.py has the most meat in there. We might need to divvy that one up.

@ericmjl  Thanks for setting it up.
@loganthomas  @samukweku  I'll pickup the finance.py file as a starting point.

Next steps to note:

Jupyter notebooks, in this scheme, get moved out to another repo, so I am going to disable Jupyter nb builds.
Need docs previews to make sure everything is running properly.

🚀 Deployed on https://deploy-preview-897--pyjanitor.netlify.app

4. Mark tests as being turtle (slow), in preparation for splitting tests into two to make them run faster.
I totally missed that part


PlayDetails got an unexpected keyword argument

PlayDetails.__init__() got an unexpected keyword argument
       'disengagementnum'

Periodic tests are failing for PlayDetails

merged


Where are message attachments?

Hello,
I was trying to send email with pdf file but in mailin message event handler I get only these fields in attachments attribute of data variable:
 [ { contentType: 'application/pdf',
       contentDisposition: 'attachment',
       fileName: 'request.pdf',
       generatedFileName: 'request.pdf',
       contentId: '9d35d04b15f736a56b8ea607ea69fae4@mailparser' } ],

In examples there is always some content field with file data.. Am I missing something? How can I retrieve data of sent PDF? Can you help me please?

Ahh, my bad.. - There was problem in nodemailer with handling PDF file content..

Hi,
I'm having the same problem with sending 'png' attachments. Could you maybe post the solution you found?
Thanks!

Hi @rogerthealien - the problem was when sending an email with nodemailer - I put there something wrong (I think that it was wrong path to file) so it did not send file content..


Missing functions for binary searches

I'm missing some function for doing binary searches on arrays in GDScript.
Now, I'd add one, only there needs to be some discussion how and where. Should it be:
(1) a builtin GDScript function like clamp and max?
(2) an extension of Array? But that's a core extension and the core must stay minimal.
(3) some kind of utils package that people can use, like Python's bisect package
My personal favorite would be adding a bisect_left and bisect_right to Array.

I think it should be added to Array.

Merged now.


ImportError: cannot import name 'HTTPError'

System information

Have I written custom code (as opposed to using example directory):  No.
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):  Ubuntu 16.04 (Linux Mint 18)
TensorFlow backend (yes / no):  yes
TensorFlow version:  1.12
Keras version:   2.2.4
Python version: Anaconda Python 3.6
CUDA/cuDNN version:   9.2, 7.3.1
GPU model and memory:  2060, 6 gigs

Describe the current behavior
Traceback (most recent call last):
  File "curiosity.py", line 1, in <module>
    from tensorflow.python.keras.layers import Conv2D, MaxPooling2D, UpSampling2D, Input
  File "/home/ai/anaconda3/envs/drl/lib/python3.6/site-packages/tensorflow/__init__.py", line 24, in <module>
    from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
  File "/home/ai/anaconda3/envs/drl/lib/python3.6/site-packages/tensorflow/python/__init__.py", line 47, in <module>
    import numpy as np
  File "/home/ai/anaconda3/envs/drl/lib/python3.6/site-packages/numpy/__init__.py", line 187, in <module>
    from .testing import Tester
  File "/home/ai/anaconda3/envs/drl/lib/python3.6/site-packages/numpy/testing/__init__.py", line 12, in <module>
    from ._private.utils import *
  File "/home/ai/anaconda3/envs/drl/lib/python3.6/site-packages/numpy/testing/_private/utils.py", line 16, in <module>
    from tempfile import mkdtemp, mkstemp
  File "/home/ai/anaconda3/envs/drl/lib/python3.6/tempfile.py", line 45, in <module>
    from random import Random as _Random
  File "/home/ai/expeditionRL/expedition/curiosities/random.py", line 1, in <module>
    from expedition.curiosities import BaseCuriosity
  File "/home/ai/expeditionRL/expedition/curiosities/__init__.py", line 1, in <module>
    from .curiosity import BaseCuriosity
  File "/home/ai/expeditionRL/expedition/curiosities/curiosity.py", line 1, in <module>
    from tensorflow.python.keras.layers import Conv2D, MaxPooling2D, UpSampling2D, Input
  File "/home/ai/anaconda3/envs/drl/lib/python3.6/site-packages/tensorflow/python/keras/__init__.py", line 24, in <module>
    from tensorflow.python.keras import activations
  File "/home/ai/anaconda3/envs/drl/lib/python3.6/site-packages/tensorflow/python/keras/activations.py", line 24, in <module>
    from tensorflow.python.keras.utils.generic_utils import deserialize_keras_object
  File "/home/ai/anaconda3/envs/drl/lib/python3.6/site-packages/tensorflow/python/keras/utils/__init__.py", line 21, in <module>
    from tensorflow.python.keras.utils.data_utils import GeneratorEnqueuer
  File "/home/ai/anaconda3/envs/drl/lib/python3.6/site-packages/tensorflow/python/keras/utils/data_utils.py", line 38, in <module>
    from six.moves.urllib.error import HTTPError
ImportError: cannot import name 'HTTPError'

Code to reproduce the issue
import keras

This issue is not on TensorFlow/Keras end. Please take a look at similar issue raised on SO. For further questions related to '''ImportError: cannot import name 'HTTPError'''' you can try posting on stackoverflow. Thanks!


Undefined method errors on job edit

Originally created by Tom Smyth at 2018-08-08 19:12, migrated from redmine-#8737

Status changed from New to Closed

By Tom Smyth at 2018-08-08 23:19

Version set to 6.1

By Tom Smyth at 2018-09-27 20:02


Wood stairs are not craftable

Running 1.20.1 on Fabric/Quilt. Everything else seems fine, but both the greatwood stairs and silverwood stairs don’t have a crafting recipe associated with them. The only way to get them is to add them to inventory through commands or creative.

Recipes will get an update (will be remade from scratch) in 2.2.0 beta. I'm going to release it next week, stay tuned


Return download url without actually executing .download()

I've read the documentation and couldn't find such an option. I guess building url by filtering output of Video would be an option but is there a better way to do this? Thanks!

Hey, @hcbt thank you for your question.
The download URL is actually a bit hidden, and you're right that I should add a hint to the documentation. You can access each Streams download URL via the signature_cipher filed, which contains a url field.
I'll add a hint to the documentation in the next release.

Thanks!


Redesign the home page

The home page looks split into 2 fragments, which does not look good. We can join them and keep one whole page with "Work Day Scheduler", its description and enable light mode. Can I work on this issue under hacktoberfest'22?

This issue has been resolved in the latest version of the website, thereby closing it.


#new-87-detailed-report-message-for-groups

🗣 Description
Detailed report messages for Groups
Closes #87
💭 Motivation and context
This issue is part of epic #90
🧪 Testing
All reports and unit tests should pass all use cases
Note: Policy 7 is not updated with detailed report message. If it is applicable this can be handled as a separate issue post Barracuda release.
✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] If applicable, All future TODOs are captured in issues, which are referenced in the PR description.
[x] The relevant issues PR resolves are linked preferably via closing keywords.
[x] All relevant type-of-change labels have been added.
[x] I have read and agree to the CONTRIBUTING.md document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated to reflect the changes in this PR.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge Checklist

[ ] This PR has been smoke tested to ensure main is in a functional state when this PR is merged.
[ ] Squash all commits into one PR level commit using the Squash and merge button.

✅ Post-merge Checklist

[ ] Delete the branch to clean up.
[ ] Close issues resolved by this PR if the closing keywords did not activate.

@buidav this issue has other old branches, didn't delete those yet to avoid any conflicts /merge issues. This is current updated branch with all the updates. The other branches are good to delete.


New Release

Hi Team,
Wonder what's the release schedule/process from master?
Thanks

There is a release scheduled later this week.


feat: Update ui-router dependency

Type: feat
Platform: all
Is it possible to update current ui-router dependency to the next stable version?
Current used: 0.2.13
https://github.com/driftyco/ionic/tree/master/config/lib/js/angular-ui
Latest available: 0.2.15
https://github.com/angular-ui/ui-router
According to ui-router documentation it should not have any breaking change between these versions:
https://github.com/angular-ui/ui-router/blob/master/CHANGELOG.md
Perhaps another thing to consider is how manage these dependencies via NPM peer dependencies.

Greetings @thiagofelix!
I've closed this issue because my sensors indicated it was old and inactive, and may have already been fixed in recent versions of Ionic. However, if you are still experiencing this issue, please feel free to reopen this issue by creating a new one, and include any examples and other necessary information, so that we can look into it further.
Thank you for allowing me to assist you.


Add pipeline and trigger version

This will add pipeline and trigger version to the status
of the config CR

i see that this patch adds PipelineVersion and TriggersVersion.
Should we add ClusterTask Version as well. I understand that there is no label from where we can read this information at present.
But, I think it will be valuable for pir users
Either we can assume that there will be a label on our ClusterTasks (and pick one to read the label)
or we can take the pipeline vmajor.minor
cc @vdemeester

i see that this patch adds PipelineVersion and TriggersVersion.
Should we add ClusterTask Version as well. I understand that there is no label from where we can read this information at present.
But, I think it will be valuable for pir users
Either we can assume that there will be a label on our ClusterTasks (and pick one to read the label)
or we can take the pipeline vmajor.minor
cc @vdemeester

What would be the ClusterTask version ? (for example, for 1.1 release)

@vdemeester We don't have version label in cluster task. But we will cut the tag with the same as pipeline version v0.14.3

@piyush-garg
the updated status structure in conditions list or (status message) will be a great context while reviewing this.
could you paste the config status yaml after the installation is successful (in the comment thread or in pr description)

@nikhil-thomas
Status will look like
[
  {
    "code": "installed",
    "version": "devel"
  },
  {
    "code": "trigger-version",
    "version": "v0.6.1"
  },
  {
    "code": "applied-addons",
    "version": "devel"
  },
  {
    "code": "pipeline-version",
    "version": "v0.14.2"
  },
  {
    "code": "validated-pipeline",
    "version": "devel"
  },
  {
    "code": "applied-pipeline",
    "version": "devel"
  }
]

shall we separate the version as pipelines-version and triggers-version
something like
[
  {
    "code": "installed",
    "pipeline-version": "0.14.2"
    "triggers-version: "0.6.1"
  },
  {
    "code": "trigger-version",
    "pipeline-version": "0.14.2"
    "triggers-version: "0.6.1"
  },
  {
    "code": "applied-addons",
    "pipeline-version": "0.14.2"
    "triggers-version: ""
  },
  {
    "code": "pipeline-version",
    "pipeline-version": "0.14.2"
    "triggers-version: ""
  },
  {
    "code": "validated-pipeline",
    "pipeline-version": ""
    "triggers-version: ""
  },
  {
    "code": "applied-pipeline",
    "pipeline-version": ""
    "triggers-version: ""
  }
]

in addition, when we change the conditions to kubernetes style conditions later, the pipeline-version and triggers-version could become condition fields on status.
@vdemeester @sthaha

@nikhil-thomas
Now the status will look like
oc get config.operator.tekton.dev cluster -o json | jq '.status.conditions'
[
  {
    "code": "installed",
    "pipeline-version": "v0.14.3",
    "triggers-version": "v0.6.1",
    "version": "devel"
  },
  {
    "code": "applied-addons",
    "pipeline-version": "v0.14.3",
    "triggers-version": "v0.6.1",
    "version": "devel"
  },
  {
    "code": "validated-pipeline",
    "pipeline-version": "v0.14.3",
    "triggers-version": "",
    "version": "devel"
  },
  {
    "code": "applied-pipeline",
    "pipeline-version": "",
    "triggers-version": "",
    "version": "devel"
  }
]

@vdemeester @sthaha

/lgtm

/approve


Potential bug for s3 OrdinaryCallingFormat in python 3 environment

When I connected to s3 and setup calling_format as OrdinaryCallingFormat in Python 3.7 environment, s3.lookup(bucket_name) cannot return the bucket just created.
But it works in Py2 environment. Is this a bug?

Thanks for help.

Hi @PeterCui1991, all formats should be supported, as far as I know. This might need some debugging.
What kind of bucket name are you using?

Test-bucket-1
I had this issue only in py3 environment. It can run in py2
On Fri, Mar 13, 2020 at 4:04 AM Bert Blommers<EMAIL_ADDRESS>wrote:

Hi @PeterCui1991 https://github.com/PeterCui1991, all formats should be
supported, as far as I know. This might need some debugging.
What kind of bucket name are you using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spulec/moto/issues/2791#issuecomment-598599163, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AE7V5TTTJATXA7LBGRKVGR3RHHSKBANCNFSM4LEKYNYQ
.

@bblommers Hi is this issue related to bucket name? but this bucket name works in python 2 environment. What do you think?

@PeterCui1991 Bucket names cannot contain upper case characters (anymore), so I can't create that bucket to even try and reproduce it.
Using a lower case bucket name, the lookup function works fine for me in Python 2 and 3

Yes. I used lower case name. That capital T was because of email auto
convert. The bucket name is like this “test-bucket-1”
It still cannot return expected object.
Environment is like this:
Python 3.7
Boto == 2.49
Boto3 == 1.12.1
Botocore == 1.15.20
Moto == 1.3.14
On Tue, Mar 17, 2020 at 4:04 AM Bert Blommers<EMAIL_ADDRESS>wrote:

@PeterCui1991 https://github.com/PeterCui1991 Bucket names cannot
contain upper case characters (anymore), so I can't create that bucket to
even try and reproduce it.
Using a lower case bucket name, the lookup function works fine for me in
Python 2 and 3
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spulec/moto/issues/2791#issuecomment-599931481, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AE7V5TRWE5SZR5VNR6SV563RH4VJ7ANCNFSM4LEKYNYQ
.

We've just released Moto 3, where we've dropped support for boto2 altogether, so I'm going to close this.
I realize it's been a while - hopefully you did find a solution for this! Let us know if there's still anything we can do to help.


TypeError: First parameter has member 'readable' that is not a ReadableStream.

UnoCSS version
0.50.6
Describe the bug
When presetUno() was used in astro2.1.3, this issue occurred and required node18.x to be used to resolve it
Reproduction
https://github.com/qiuqfang/demo-astro-unocss
System Info
System:
    OS: macOS 13.2.1
    CPU: (8) arm64 Apple M1
    Memory: 196.02 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
Binaries:
    Node: 16.19.1 - ~/.nvm/versions/node/v16.19.1/bin/node
    npm: 8.19.3 - ~/.nvm/versions/node/v16.19.1/bin/npm
Browsers:
    Chrome: 111.0.5563.64
    Safari: 16.3

Validations

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

Facing this exact issue


[Bug]: Error in creating Replicator Database in rapidfort/couchdb-official

Name and Version
rapidfort/couchdb-official:3.3.3
Which runtime are you using to reproduce this issue?

[ ] Kubernetes
[ ] Docker Compose
[X] Docker

Is this issue reproducible on the original source image?
Reproducible
Could you please identify the category? Details in TROUBLE_SHOOTING.md
RF error
What steps will reproduce the bug?

Run the containers on docker using docker run -d --name rcdb-3 -e COUCHDB_SECRET=hello -e COUCHDB_USER=admin -e COUCHDB_PASSWORD=password -e ERL_FLAGS="-setcookie brumbrum" -p 5984:5984 -p 4368:4369 rapidfort/couchdb-official:3.3.3
Trying creating the replicator database using curl -X PUT localhost:5984/_replicator -u admin
It will prompt for password of admin user, the password is set when the container starts running

Are you using any custom parameters or values?
No response
What is the expected behavior?
{"ok":true}
What do you see instead?
The container crashes. Logs:

Additional information
Images used:
rapidfort/couchdb-official:3.3.3
couchdb:3.3
Sequence of events:

Edit:
in the above image at port 5985 the original couchdb image is running and at 5984 rapidfort image is running
the rapidfort image also returns ok:true but soon after crashes

Even on restarting, this behaviour persists

Due to error in creating the replicator db (it is a required step in docs), the cluster setup is not finished

Hi @utkarsh-1905, i tried to replicate, and on my end container never exited. There might be some difference in sequence of commands between us, but curl localhost:5984/_cluster_setup -u admin is failing.
Thanks for raising bug.

Ohh, that might be, but I tried multiple times and behaviour is the same, can you please mention your sequence of commands, i ll post mine

Here is what i tried for both images. (output is of RapidFort's image)
curl -X PUT localhost:5984/_replicator -u admin
# {"ok":true}
curl -X GET localhost:5984/_cluster_setup -u admin
# {"error":"unknown_error","reason":"undef","ref":228524404}
curl -X PUT localhost:5984/_replicator -u admin
# {"error":"file_exists","reason":"The database could not be created, the file already exists."}

Sorry @Kirandeep-Singh-Khehra , I left out a step, actually after we create a replicator db we need to restart the container to apply the changes and start replicator service, so after curl -X PUT localhost:5984/_replicator -u admin we need to docker restart rcdb-3 to apply the changes, when we restart, it exits, can you please reciprocate this?

While testing, I noticed a lot of API's are not working since the cluster setup is not working. For example  curl -X PUT http://localhost:5984/_node/_local/_config/uuids/algorithm -d '"random"' -u admin


dashboard-list.js sorts dashboards and updates page-header

Two simple changes to dashboard-list.js

Have the list of Dashboards sorted in alphabetical order via allTags.sort();
Fix page-header not updating bug by adding in title: 'Dashboards' in the route

This pull request was merged via https://github.com/getredash/redash/pull/1630


Support passing connection settings as URL parameters

It baffles me that for every aria2 server, we need another server to host a few static pages in order to use this UI. I personally have two aria2 servers and I don't want create my own instance of the UI server. I use http://ziahamza.github.io/webui-aria2/ and have to manually change the host address in connection settings back and forth between the two servers.
It would be great if we can specify the connection settings from the URL like this:
http://ziahamza.github.io/webui-aria2/?host=<IP_ADDRESS>&port=6800&rpcpath=/jsonrpc

Then we only need one URL per server. Which is very bookmark-friendly. We also don't need to set up our own server and use the fast GitHub CDN.
I'm not good with front-end stuff, but a quick Google search says it's possible with Javascript:
http://stackoverflow.com/questions/11582512/how-to-get-url-parameters-with-javascript

Any updated about this? Is this planned at all?

I'm no actively working on this.
I'm not entirely sure how to proceed because there is some security issues regarding transmitting the aria2 password/token unencrypted over internet, though.

I think this is a really useful feature.
Yaaw and aria-ng both support this feature (rest api):
yaaw simply transfer token/password as plain text , which is really a bad idea
aria-ng use base64 encode the token field, which is slightly better
Maybe we can use other encryption method (aes or rsa ) or we can just init anything else except token?


problems.

hey! I tried the game, there are some problems where the song  is uhh... fast forwarded. like- the music. I want to figure out how to fix it! thanks.

The music speed is normal on my end - did you accidentally change the song speed in the mods tab? 1x means normal speed


Unable to "/save" experience using the library

Describe the bug
I am sending "/save" command/message to Taskweaver to make it remember an experience. Taskweaver is failing at doing that.
To Reproduce

Run Taskweaver by importing as a library
After a couple of messages, send the "/save" command/message i.e., session.send_message("/save")
Wait for the response
{'id': 'round-20240205-472936-424d5z34', 'user_query': '/save', 'state': 'failed', 'post_list': [{'id': 'post-20240205-472936-e97ght638', 'message': '/save', 'send_from': 'User', 'send_to': 'Planner', 'attachment_list': []}]}

Expected behavior
I expect it to save the experience the way it does when run from Terminal
Environment Information:

OS: Windows
Python Version: 3.11.5
LLM that you're using: Llama

Hi, @ysid27 ! Did you use session.send_message("/save") to save the experience? Actually, all commands starting with/only support console chat and not through library usage or Chainlit UI.

Hi @zhangxu0307 ! I am using session.send_message("/save") through library. If session.send_message("/save") is not the way to save an experience through library, what is the way? Thanks!

So far, we only support user to use /save in the console chat (chat in command line).

Oh no! Are there plans to extend this feature to library?

A workaround method is trying to use the following code to save chat histroy directly:
session.memory.save_experience(exp_dir=session.config.experience_dir)
If it works, you will see a raw exp file saved in your expereince dir.


False-positives on “Perl 6 Types” page

| Name | Declarator | Source |

[ ] example | enum |
[ ] auto-punning | role |

These should not be listed.

also

[ ] Parameters | role

I think it's because of wrong =head3. let me try to fix it.


[Storage - File Datalake] test failing in nightly runs

Storage - File Datalake nightly test runs are failing with:

Error message:
The specified container does not exist. RequestId:11f5c3e0-e01e-0051-77a9-89573d000000 Time:2021-08-05T03:21:56.8490576Z
Stack trace:
RestError: The specified container does not exist.
RequestId:11f5c3e0-e01e-0051-77a9-89573d000000
Time:2021-08-05T03:21:56.8490576Z
at handleErrorResponse (D:\a_work\1\s\sdk\core\core-http\src\policies\deserializationPolicy.ts:266:17)
at parse.then (D:\a_work\1\s\sdk\core\core-http\src\policies\deserializationPolicy.ts:171:47)
at process._tickCallback (internal/process/next_tick.js:68:7)

For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1033589&view=results

Fixed with PR: https://github.com/Azure/azure-sdk-for-js/pull/19511.


I want to hidden the wrong information

Description
When I enter neofetch, it shows that /data/data/com.termux/files/usr/bin/neofetch: line 8682: /dev/stdin: No such file or directory
So what should i edit the config file to hidden it?
config is default.
If you're suggesting a new feature then just a description will suffice.
Neofetch version
4.0.1
Screenshot

## Config file
Verbose log

Run neofetch -vv 2> neofetchlog
Upload the contents of neofetchlog to pastebin, gist or equivalent.

This is fixed in master.

OK，great, thank  you.

A new release was also made. :)


Resolve issue 1819

Support polymorphic call to LocalRunner::send_message(...). Define LocalRunner::clients as a list containing the local WorkerNode initialized by Runner. Initialize LocalRunner::client_id to "local". Initialize LocalRunner::worker_count to 1.

A couple of things I forgot to mention; I fixed target_user_count because that does seem to be a bug, and user_count seems to have a time delay so that was causing some issues in getting the example we talked about to work.
Additionally, I wasn't sure with the polymorphic form if we should do anything with a client_id if subbmitted; currently, if submitted, it passes that ID through the message. I'm not sure if this could be a helpful feature used in some case or if it has the potential to break something else (as they use the client_id "local" everywhere for LocalRunner, which I have kept consistent).


[React/progress-bar] Fix progress control button responsiveness

Title and Issue number

Title: Fix progress control button responsiveness
Issue No. : #199
Code Stack:  REACT
Close #199

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md
[X] I have updated the Index.html file for my contribution

Closing due to inactivity


Make search box in header always be expanded even before click

https://github.com/yzernik/squeaknode/pull/1300


restart pipe mGLSurfaceView becomes three areas

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
D435

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win 10

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
Android.

SDK Version
2

Language
java

Segment
Android

Hello, when I restart the pipe, the first few times were normal, but later mGLSurfaceView becomes three areas. One of them should be the previous image. I don't know what to do
this  is  nomal

this  is  three areas

this  is  my  code

Hi @dota2mhxy   Could you provide more information about what you mean when you say there are three areas.  Do you mean that there should be two panels (the RGB and the depth) but then it becomes three panels (RGB and two depth).  Thanks!

yes，It used to be two panels, color and depth. After I restarted many times, it became three, two depths and one color. One depth seems to be an image left over from the past, not the current image information

@MartyG-RealSense

I compared your code to the Capture example.  The code of that project and your script looks similar, with the main difference being that some of the code blocks that are inside functions that are jumped to when called are moved so that a jump is no longer necessary.
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/android/examples/capture/src/main/java/com/intel/realsense/capture/MainActivity.java
This may introduce unforeseen errors though.  It may be better to use the Capture example's code and find a way to insert your own mStart RecordFab block of listener code into it.
Before doing that, you could try removing the index numbers from the cfg instructions.
cfg.enableStream(StreamType.DEPTH, width: 1280, height: 720, StreamFormat.ANY, framerate: 6);
cfg.enableStream(StreamType.COLOR, width: 1280, height: 720, StreamFormat.ANY, framerate: 6);

hello @MartyG-RealSense  .removing the index numbers from the cfg instructions   Is not feasible。this  is code  from
com.intel.realsense:librealsense:2.4.you can see  if you  not give  index，-1 is the default

Then, in this code, I actually made some modifications to the capture example, made some custom calculations for the functions of mstartrecordfab and mstopprecordfab buttons, and the displayed interface has always been mglsurfaceview. To prevent out of frames, I have to stop(), start()
The previous restarts were normal. About the third and fourth time, two depth maps appeared. I tried to move the camera and found that the color map and one depth map will change with the scenery, and the other depth map will not change. It may be a certain image. it seems to be the first copy of the depth map。You can see from the following two pictures that the top depth map is fixed, which is the copy of the  first depth map. The bottom depth map corresponds to the color map。

and  if  i  Unplug the USB interface, it will restart, and the new view will become normal

If the problem only occurs after the third / fourth pipeline start and an unplug-replug of the USB cable corrects it, a solution might be to perform a camera on each start.  In https://github.com/IntelRealSense/librealsense/issues/4091#issuecomment-561935062 an Android script that makes use of reset is provided.  Left-click on the Streamer.java.txt link in the comment to open the script.

Hi @dota2mhxy  Do you require further assistance with this case, please?  Thanks!

Thanks very much @dota2mhxy for the update!


Allows plugins to register controllers

Fixes #2.
The configureRoutes method on the Plugin-definition can be
overwritten/extended as desired. By default, it adds all
route-like files within the Resources directory, in a similar
manner the Kernel does so for the global routes within
/config/routes.

@janbuecker please merge :smile: :+1:

Thank you @EtienneBruines. We've merged it into our core.


UnionAll + First generates invalid SQL

Environment
Knex version: 0.95.1
Database + version: Postgresql 9.6
OS: Alpine docker running on Mac BigSur 11.2.1
Bug

The following knex query:

knex
  .unionAll(['table1', 'table2'].map((tablename) => knex(tablename).select().where('name', 'test')))
  .first()
  .then((result) => result);

generates the following sql
SELECT * UNION ALL SELECT * FROM "schema"."table1" AS "table1" WHERE "name" = $1 UNION ALL SELECT * FROM "schema"."table2" AS "table2" WHERE "name" = $2 LIMIT $3;

Error message
ERROR:  SELECT * with no tables specified is not valid at character 8

Reduced test code

https://runkit.com/ungrim97/6049dd2b75fafb00195923c2

Expectation

That code did (in version 0.21.17) generate the SQL:
SELECT * FROM "schema"."table1" AS "table1" WHERE "name" = $1 UNION ALL SELECT * FROM "schema"."table2" AS "table2" WHERE "name" = $2 LIMIT $3;

Which I believe is the correct SQL for this knex query. This appears to be a regression of functionality between version 0.21.17 and 0.95.1 when using first()
removing the first() and replacing the then with then(([result]) => result) works as expected in either situation as is a suitable work around for me for now

Released in 1.0.1.


Please update Jschema to latest SARIF-SDK

@lgolding a request from your fans

PR #129 updates SDK to 2.3.3.


Libraries

There are several people working on building libraries for Solid and no description of how they should work together in the process repository.
A panel is specifically for making proposals to the Solid specification for editorial review so a panel is not quite adequate.
Possibly a solution would be to have a section in the README of the process similar to solidproject website or test suite that describes the aim and where to list yourself if you choose to work in the Solid GitHub.
For example:
Solid Libraries
Anyone can write a Solid library based on the Solid specification. You can find an overview of Solid libraries in the the libraries repository. Librarians(link to GitHub.com/process/librarians.md) are responsible for keeping the overview of Solid libraries up to date as well as the libraries that can be found in the Solid GitHub account.

There is definitely a need to have a central place for information on libraries.  Putting a link within "process" is good in terms of alerting developers how to create a library but I think there needs to be a separate landing page telling developers what libraries are available, what the differences in capabilities and interfaces are, and why they might use one or another.   In other words a page devoted to using libraries, not creating them.  Perhaps something like:
Solid Libraries and Tools
There are a number of Solid libraries and toolkits which allow developers to create apps without having to re-invent the wheel on basic operations.  These include RDF parsing toolkits, login and authentication libraries, file-handling libraries, and GUI toolkits and component libraries.  An overview and comparision of libraries and toolkits may be found at X,  A cheatsheet of how-to perform basic operations in different libraries may be found at Y,  A list of library and toolkit developers may be found at. Z,

I would be willing to take the lead on organizing/expanding the library pages but it would be great to have a team of other developers.  If not a panel, then what?  There should be specified comm channels and a central place people can follow developments.
Currently, I know of four places libraries are discussed.  Are there others?
https://github.com/solid/libraries
https://github.com/solid/information/blob/master/documentation/developer-tools.md
https://vincenttunru.gitlab.io/tripledoc/docs/cheatsheet
https://github.com/inrupt/solid-lib-comparison

Fantastic! This would be really valuable @jeff-zucker
On second thought the overview could be done on solid.github.io repo so that it links to the website directly. @jeff-zucker maybe you could apply to be a Creator by submitting a pull to the process repo if you're interested?
The the Librarians could be added to the process readme:
#Solid Libraries
Anyone can write a Solid library based on the Solid specification. Librarians(link to GitHub.com/process/librarians.md) are responsible for keeping the  libraries that can be found in the Solid GitHub account up to date.

"Librarians communicate via gitter/solid/librarians"?

Thanks for picking this up @jeff-zucker! Yes, I think those four resources should probably be merged into one.

I think I found a fifth one: https://github.com/solid/solid-platform#libraries

Also https://forum.solidproject.org/t/soukai-a-different-way-to-query-pods/2105 should be added to the new list.

Thanks, @michielbdejong .  I've added that and a number of others at https://github.com/jeff-zucker/solid-libraries.

Amazing! Thanks @jeff-zucker this is really fantastic. @Vinnl @NSeydoux could you integrate this content into the website draft?

It's on my todo list, but so are many other things unfortunately. It would go here, I think.

This issue is resolved by the combination of https://github.com/solid/solid.github.io/pull/191 + https://github.com/jeff-zucker/solid-libraries/pull/6 + https://github.com/solid/information/pull/302 + https://github.com/solid/solid-platform/pull/7


Add auto doc feature

Should gen basic README.md with screen shots gathered from regression tests.

As of version 0.5.2 most pages have linked screen shots. Added app description and auto gen of files list page. Will add new pages going forward.


Update version of CsWinRT and Windows SDK projection (#4427)

Update CsWinRT version

A microsoft employee must use /azp run to validate using the pipelines below.
WARNING:
Comments made by azure-pipelines bot maybe inaccurate.
Please see pipeline link to verify that the build is being ran.
For status checks on the main branch, please use TransportPackage-Foundation-PR
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=81063&_a=summary)
and run the build against your PR branch with the default parameters.

/azp run

/azp run


[WIP] Empty commit to test on gpuCI

For debugging / testing #9432, and anything else that happens to be wrong on main at present.

gpuci run tests


Regex catches too much

Redirect with regex of \/([0-9]{4})\/([0-9]{2})\/(.*): /$3 needs to be more specific like kenst.com\/([0-9]{4})\/([0-9]{2})\/(.*): /$3

Use https://regex101.com/ to help


fixed a crash when an object with __toString() method is passed as templ...

...ate name

I use the class "Twig_Loader_Chain" as template loader in Symfony2 application. One of the embeded loaders is "Twig_Loader_Array". In Symfony2 is used "Symfony\Component\Templating\TemplateReferenceInterface" for internal representation of a template and it causes a crash.


Decorators for middleware ?

Hi
I was wondering if its possible to pass in custom middleware on a route or is it planned ?
or even better, protecting routes using security decorators ? separate library ?

protecting routes using security decorators ? separate library ?

I have a plans to create such library in perspective, but now I pretty busy with my other open source libraries.

I was wondering if its possible to pass in custom middleware on a route or is it planned ?

Right now there is no easy way to provide custom middleware, but I believe this is top required feature and Im planning to implement it soon.

HI @pleerock ,
First off all thank you for this amazing set of libs, great work!!
you will find in here an implementation that support Middlewares and a new param decorator for next function.
you can find a example in sample8 directory.
Tell me what you think about it, and if you're ok I will happily create a PR!
Cheers from Morroco

hi @Salimlou . Thank you very much for your contribution. Please create a PR from what you did and lets discuss implementation details there

PR created #6 , see you on the other side :)

released in 0.6.0


What font been used in the screenshot?

I was wondering what font been used in the screenshot? M+ or is it a patched version?

I'd guess Fira Code + Noto Sans

Fira Code + Noto Sans is what I'm using today (shown here):

But the screenshot on the README is at least two years old. I believe I was either using Inconsolata or Input Mono Narrow, and the variable-pitch font in the treemacs sidebar is Input Sans (from the same bundle as Input Mono).
I'd still be using Input if it were easier to automate its installation on NixOS.

Thanks, that's it.


My account menu for logged user should curl up after click on any position

What is the motivation for adding / enhancing this feature?
I think that it will more comfortable for user, when he developed My account menu (in right top corner) and chose any option, for example My newsletter, then he should be redirected on this page (it works properly) but menu should curl up.

What are the acceptance criteria
My account menu (for logged user)  should will curl up, when user click on any position in this menu.
Additional information
It works like that for example for Facebook ;)

It was just like that till 1.6 :)

verified


TypeError: not all arguments converted during string formatting

Command and output:
root@AI_Server:/home/sda/tianhaoli/AVDS#  cd /home/sda/tianhaoli/AVDS ; /usr/bin/env /root/anaconda3/envs/garak310/bin/python /root/.vscode-server/extensions/ms-python.debugpy-2024.4.0-linux-x64/bundled/libs/debugpy/adapter/../../debugpy/launcher 34521 -- /home/sda/tianhaoli/AVDS/app/garak/__main__.py --model_type huggingface --model_name meta-llama/Llama-2-7b-hf --probes malwaregen
garak LLM security probe v<IP_ADDRESS>.post1 ( https://github.com/leondz/garak ) at 2024-04-26T20:51:55.930111
📜 reporting to garak_runs/garak.b8f03591-73a8-4bda-b9b2-b7fa6f230355.report.jsonl
🦜 loading generator: Hugging Face 🤗 pipeline: meta-llama/Llama-2-7b-hf
2024-04-26 20:52:29.567091: I tensorflow/core/util/port.cc:110] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
2024-04-26 20:52:29.607328: I tensorflow/core/platform/cpu_feature_guard.cc:182] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 AVX512F AVX512_VNNI FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2024-04-26 20:52:34.258439: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
Loading checkpoint shards: 100%|████████████████████████████████████████████████████████████████████████████████████████████████████████| 2/2 [00:02<00:00,  1.19s/it]
🕵  queue of probes: malwaregen.Evasion, malwaregen.Payload, malwaregen.SubFunctions, malwaregen.TopLevel
probes.malwaregen.Evasion:   0%|                                                                                | 0/48 [00:00<?, ?it/s]Asking to truncate to max_length but no maximum length is provided and the model has no predefined maximum length. Default to no truncation.
probes.malwaregen.Evasion:   2%|██▏                                                                                | 1/48 [00:03<02:21,  3.01s/it]--- Logging error ---
Traceback (most recent call last):
  File "/root/anaconda3/envs/garak310/lib/python3.10/logging/__init__.py", line 1100, in emit
    msg = self.format(record)
  File "/root/anaconda3/envs/garak310/lib/python3.10/logging/__init__.py", line 943, in format
    return fmt.format(record)
  File "/root/anaconda3/envs/garak310/lib/python3.10/logging/__init__.py", line 678, in format
    record.message = record.getMessage()
  File "/root/anaconda3/envs/garak310/lib/python3.10/logging/__init__.py", line 368, in getMessage
    msg = msg % self.args
TypeError: not all arguments converted during string formatting
Call stack:
  File "/root/anaconda3/envs/garak310/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/root/anaconda3/envs/garak310/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/root/.vscode-server/extensions/ms-python.debugpy-2024.4.0-linux-x64/bundled/libs/debugpy/adapter/../../debugpy/launcher/../../debugpy/__main__.py", line 39, in <module>
    cli.main()
  File "/root/.vscode-server/extensions/ms-python.debugpy-2024.4.0-linux-x64/bundled/libs/debugpy/adapter/../../debugpy/launcher/../../debugpy/../debugpy/server/cli.py", line 430, in main
    run()
  File "/root/.vscode-server/extensions/ms-python.debugpy-2024.4.0-linux-x64/bundled/libs/debugpy/adapter/../../debugpy/launcher/../../debugpy/../debugpy/server/cli.py", line 284, in run_file
    runpy.run_path(target, run_name="__main__")
  File "/root/.vscode-server/extensions/ms-python.debugpy-2024.4.0-linux-x64/bundled/libs/debugpy/_vendored/pydevd/_pydevd_bundle/pydevd_runpy.py", line 321, in run_path
    return _run_module_code(code, init_globals, run_name,
  File "/root/.vscode-server/extensions/ms-python.debugpy-2024.4.0-linux-x64/bundled/libs/debugpy/_vendored/pydevd/_pydevd_bundle/pydevd_runpy.py", line 135, in _run_module_code
    _run_code(code, mod_globals, init_globals,
  File "/root/.vscode-server/extensions/ms-python.debugpy-2024.4.0-linux-x64/bundled/libs/debugpy/_vendored/pydevd/_pydevd_bundle/pydevd_runpy.py", line 124, in _run_code
    exec(code, run_globals)
  File "/home/sda/tianhaoli/AVDS/app/garak/__main__.py", line 15, in <module>
    main()
  File "/home/sda/tianhaoli/AVDS/app/garak/__main__.py", line 11, in main
    cli.main(sys.argv[1:])
  File "/home/sda/tianhaoli/AVDS/app/garak/cli.py", line 486, in main
    command.probewise_run(generator, probe_names, evaluator, buff_names)
  File "/home/sda/tianhaoli/AVDS/app/garak/command.py", line 212, in probewise_run
    probewise_h.run(generator, probe_names, evaluator, buffs)
  File "/home/sda/tianhaoli/AVDS/app/garak/harnesses/probewise.py", line 106, in run
    h.run(model, [probe], detectors, evaluator, announce_probe=False)
  File "/home/sda/tianhaoli/AVDS/app/garak/harnesses/base.py", line 93, in run
    attempt_results = probe.probe(model)
  File "/home/sda/tianhaoli/AVDS/app/garak/probes/base.py", line 189, in probe
    attempts_completed.append(self._execute_attempt(this_attempt))
  File "/home/sda/tianhaoli/AVDS/app/garak/probes/base.py", line 138, in _execute_attempt
    this_attempt.outputs = self.generator.generate(this_attempt.prompt)
  File "/home/sda/tianhaoli/AVDS/app/garak/generators/base.py", line 91, in generate
    return self._call_model(prompt, generations_this_call)
  File "/home/sda/tianhaoli/AVDS/app/garak/generators/huggingface.py", line 104, in _call_model
    raw_output = self.generator(
  File "/root/anaconda3/envs/garak310/lib/python3.10/site-packages/transformers/pipelines/text_generation.py", line 240, in __call__
    return super().__call__(text_inputs, **kwargs)
  File "/root/anaconda3/envs/garak310/lib/python3.10/site-packages/transformers/pipelines/base.py", line 1167, in __call__
    logger.warning_once(
  File "/root/anaconda3/envs/garak310/lib/python3.10/site-packages/transformers/utils/logging.py", line 329, in warning_once
    self.warning(*args, **kwargs)
Message: 'You seem to be using the pipelines sequentially on GPU. In order to maximize efficiency please use a dataset'
Arguments: (<class 'UserWarning'>,)
malwaregen.Evasion                                                                malwaregen.AnyCode: SKIP  ok on    0/   0
malwaregen.Payload                                                                malwaregen.AnyCode: SKIP  ok on    0/   0
malwaregen.SubFunctions                                                           malwaregen.AnyCode: SKIP  ok on    0/   0
malwaregen.TopLevel                                                               malwaregen.AnyCode: SKIP  ok on    0/   0
📜 report closed :) garak_runs/garak.b8f03591-73a8-4bda-b9b2-b7fa6f230355.report.jsonl
📜 report html summary being written to garak_runs/garak.b8f03591-73a8-4bda-b9b2-b7fa6f230355.report.html
✔  garak run complete in 191.72s

Cause by error in third-party module:
https://github.com/huggingface/transformers/commit/c82b38a3e2c8af69fef08e371189766a87ed6159

quickfix is to upgrade transformers >= 4.40.0, e.g.
python3 -m pip install "transformers>=4.40.0"


iterables and toHaveBeenCalledWith result in RangeError: Maximum call stack size exceeded

🐛 Bug Report
If toHaveBeenCalledWith is used with an iterable, RangeError: Maximum call stack size exceeded will be thrown.
To Reproduce
Steps to reproduce the behavior:
Run
class Blah {
  *[Symbol.iterator]() {
    yield this;
  }
}

const video = {
  play(x) {
    console.log('received', x);
    return true;
  },
};

test('plays video', () => {
  const val = new Blah();
  const spy = jest.spyOn(video, 'play');
  const isPlaying = video.play(val);

  expect(spy).toHaveBeenCalledWith(val);
  expect(isPlaying).toBe(true);
});

This results in:
 FAIL  __tests__/test.js
  ● Console

    console.log __tests__/test.js:9
      received Blah {}

  ● plays video

    RangeError: Maximum call stack size exceeded

      17 |   const isPlaying = video.play(val);
      18 |
    > 19 |   expect(spy).toHaveBeenCalledWith(val);
         |               ^
      20 |   expect(isPlaying).toBe(true);
      21 | });
      22 |

      at Object.toHaveBeenCalledWith (__tests__/test.js:19:15)

Test Suites: 1 failed, 1 skipped, 1 of 2 total
Tests:       1 failed, 17 skipped, 18 total
Snapshots:   0 total
Time:        1.401s, estimated 2s
Ran all test suites.

Expected behavior
The test passes.
Link to repl or repo (highly encouraged)
Code given above
Run npx envinfo --preset jest
Paste the results here:
It results in an error for me. I don't think it's what you want, but I will paste it here anyway.
(node:27012) UnhandledPromiseRejectionWarning: TypeError: Cannot read property '1' of null
    at a.run.then.e (/home/user/.npm/_npx/27012/lib/node_modules/envinfo/dist/cli.js:2:96634)
(node:27012) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:27012) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Here's the version in package.json
"jest": "^23.5.0",

Minimum reproduction in Jest's own test suite:
diff --git i/packages/expect/src/__tests__/utils.test.js w/packages/expect/src/__tests__/utils.test.js
index 6a5e7d909..8ba302d89 100644
--- i/packages/expect/src/__tests__/utils.test.js
+++ w/packages/expect/src/__tests__/utils.test.js
@@ -13,6 +13,7 @@ const {
   emptyObject,
   getObjectSubset,
   getPath,
+  iterableEquality,
   subsetEquality,
 } = require('../utils');

@@ -150,3 +151,17 @@ describe('subsetEquality()', () => {
     expect(subsetEquality(undefined, {foo: 'bar'})).not.toBeTruthy();
   });
 });
+
+describe('iterableEquality()', () => {
+  test('self-returning iterator does not explode', () => {
+    class Iter {
+      *[Symbol.iterator]() {
+        yield this;
+      }
+    }
+
+    const val = new Iter();
+
+    expect(iterableEquality(val, val)).toBe(true);
+  });
+});

From what I can understand it recurses forever here: https://github.com/facebook/jest/blob/c9893d2f1cdbc98655ca446d2a49f8c77a42b4be/packages/expect/src/utils.js#L183-L188
Not sure how to handle it. Bail after say 1000 iterations? If val.next() returns val? If it returns itself n amounts of times?

@thymikee @rickhanlonii thoughts?

I'm no expert about this, but would checking reference equality helps?

Yeah, I think so. But for how many iterations should we do that before bailing? Maybe somebody has an iterator delivering fibonacci numbers - there are cases for infinite generators. Not sure what would give the best DX here

I think bailing after 1000 makes sense, we can up it if necessary

Hi, I am a student of computer and software engineering and I choose this issue to work on in my software engineering subject so I'm no expert on this. As I was trying to understand this issue I looked into the comments and saw that the problem may be fixed if the cycle referenced above is changed. Would you give me some tips and help? Thanks a lot!


specs: VEP-2448: VDK Run Logs: Simplified And Readable

Why?
Part of the run logs improvement initiative https://github.com/vmware/versatile-data-kit/issues/2408
https://github.com/vmware/versatile-data-kit/issues/2451
https://github.com/vmware/versatile-data-kit/issues/2448
What?
Create VEP document and fill out Summary, Motivation and Goals sections
How has this been tested?
n/a
What type of change are you making?
docs

@tozka I've re-worded the requirements section and added a TODO to the Summary section. Please, review again.


Routine QCMDEXC in QSYS2 not found with specified parameters. (42884)

What is the system looking for here?
This system is a v5.4 system.
Thanks.
Darryl Freinkel.

⚠ REMOVE THIS LINE AND ANY SENSITIVE INFORMATION BELOW! ⚠

Context
Version

Code for IBM i version
2.12.1

Visual Studio Code version
1.92.0

Operating System
win32_x64

Active extensions
COBOL (cobol): 10.7.23
Code Coverage for IBM i (code-coverage-ibmi): 0.1.7
Code for IBM i Walkthroughs (vscode-ibmi-walkthroughs): 0.5.0
Configuration Editing (configuration-editing): 1.0.0
Db2 for IBM i (vscode-db2i): 1.1.1
Emmet (emmet): 1.0.0
Git (git): 1.0.0
Git Base (git-base): 1.0.0
GitHub (github): 0.0.1
IBM i Debug (ibmidebug): 2.0.1
IBM i Notebooks (vscode-ibmi-notebooks): 0.0.5
JSON Language Features (json-language-features): 1.0.0
Merge Conflict (merge-conflict): 1.0.0
Node Debug Auto-attach (debug-auto-launch): 1.0.0
TODO Highlight (vscode-todo-highlight): 1.0.5
TypeScript and JavaScript Language Features (typescript-language-features): 1.0.0

Remote system

Setting
Value

IBM i OS
n/a

Tech Refresh
n/a

CCSID Origin
65535

Runtime CCSID
-2

Default CCSID
?

SQL
Disabled

Source dates
Disabled

Enabled features

/usr/bin
/QSYS.LIB

attr
QZDFMDB2.PGM

iconv

ls

setccsid

tar

Shell env

Variants
{
  "american": "#@$",
  "local": "#@$"
}

Errors
[
  {
    "command": "/QOpenSys/usr/bin/qsh",
    "code": 1,
    "stderr": "CPD0043:  Keyword SRCSTMF not valid for this command.\nCPF0001:  Error found on RUNSQLSTM command.",
    "cwd": "/"
  },
  {
    "command": "/usr/bin/attr \"/QIBM/ProdData/IBMiDebugService/bin/DebugService.env\" CCSID",
    "code": 1,
    "stderr": "attr: 001-2248 Error found getting attributes for file /QIBM/ProdData/IBMiDebugService/bin/DebugService.env.  No such path or directory.",
    "cwd": "/home/A4G_SSH"
  },
  {
    "command": "export BUILDLIB=\"QGPL\" && export CURLIB=\"QGPL\" && export USERNAME=\"a4g_ssh\" && export HOST=\"<IP_ADDRESS>\" && export HOME=\"/home/A4G_SSH\" && export WORKDIR=\"/home/A4G_SSH\" && export LIBLS=\"A4G21XSF A4G21XSFS QTEMP QGPL\" && env",
    "code": 1,
    "stderr": "bsh: BUILDLIB=QGPL: is not an identifier",
    "cwd": "/home/A4G_SSH"
  }
]

We do not support anything below 7.3.

Pity, there are so many companies that could use this. I am not looking for all functionality. Just to be able to use the explorer and edit the members. No debug, SQL or anything like that.
Thanks, Darryl

Pity, there are so many companies that could use this. I am not looking for all functionality. Just to be able to use the explorer and edit the members. No debug, SQL or anything like that.
 The explorer has been around for ages, so if it worked for you with an old version, just go back to it.

Thank you for the information. I will try it later today.
Questions:

Can I have the older and current versions on the same PC?
How would I set the older version to not update to a newer version?

Thank you.
Darryl Freinkel
A4G
Telephone:<PHONE_NUMBER>
From: Sébastien Julliand @.>
Sent: Friday, August 2, 2024 5:13 PM
To: codefori/vscode-ibmi @.>
Cc: a4ginatl @.>; Author @.>
Subject: Re: [codefori/vscode-ibmi] Routine QCMDEXC in QSYS2 not found with specified parameters. (42884) (Issue #2210)
Pity, there are so many companies that could use this. I am not looking for all functionality. Just to be able to use the explorer and edit the members. No debug, SQL or anything like that.
 The explorer has been around for ages, so if it worked for you with an old version, just go back to it.
image.png (view on web) https://github.com/user-attachments/assets/b67cb27a-576c-4ce5-9a7b-70cbf595bab3
—
Reply to this email directly, view it on GitHub https://github.com/codefori/vscode-ibmi/issues/2210#issuecomment-2266150726 , or unsubscribe https://github.com/notifications/unsubscribe-auth/A4EMVL2V4CHH3QSRNFFVL53ZPPY4NAVCNFSM6AAAAABL3RG3Z6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRWGE2TANZSGY .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/A4EMVL2DOGAFT7MFHXT42GLZPPY4NA5CNFSM6AAAAABL3RG3Z6WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUHCK3UM.gif Message ID: @.*** @.***> >

Thank you for the information. I will try it later today.
Questions:

Can I have the older and current versions on the same PC?
How would I set the older version to not update to a newer version?

No, you can't by default. But looking at the code CLI option, there is a --extensions-dir option that lets you define the extensions' location. So I guess you could create a dedicated shortcut to open VSCode, adding --extensions-dir path_to_a_dir to the command line so this particular shortcut would open VSCode with another set of extensions. If it doesn't work, maybe have a look at the Portable Mode: https://code.visualstudio.com/docs/editor/portable#_enable-portable-mode

Disable the Auto Update for the extension.

Thank you
Darryl Freinkel
A4G
Telephone:<PHONE_NUMBER>
From: Sébastien Julliand @.>
Sent: Monday, August 5, 2024 4:33 PM
To: codefori/vscode-ibmi @.>
Cc: a4ginatl @.>; Author @.>
Subject: Re: [codefori/vscode-ibmi] Routine QCMDEXC in QSYS2 not found with specified parameters. (42884) (Issue #2210)
Thank you for the information. I will try it later today.
Questions:

Can I have the older and current versions on the same PC?

How would I set the older version to not update to a newer version?

No, you can't by default. But looking at the code CLI option, there is a --extensions-dir option that lets you define the extensions' location. So I guess you could create a dedicated shortcut to open VSCode, adding --extensions-dir path_to_a_dir to the command line so this particular shortcut would open VSCode with another set of extensions. If it doesn't work, maybe have a look at the Portable Mode: https://code.visualstudio.com/docs/editor/portable#_enable-portable-mode
image.png (view on web) https://github.com/user-attachments/assets/dddbb7d9-9fb8-44e7-81f2-6e090139a48c

Disable the Auto Update for the extension.
image.png (view on web) https://github.com/user-attachments/assets/c97b3c35-6a47-45b0-a059-c8a0c732f92d

—
Reply to this email directly, view it on GitHub https://github.com/codefori/vscode-ibmi/issues/2210#issuecomment-2269867480 , or unsubscribe https://github.com/notifications/unsubscribe-auth/A4EMVL4MKDZTYIZ6QP3R3XDZP7ORJAVCNFSM6AAAAABL3RG3Z6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRZHA3DONBYGA .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/A4EMVLZUF235IAQO5KODX2DZP7ORJA5CNFSM6AAAAABL3RG3Z6WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUHJNW5Q.gif Message ID: @.*** @.***> >


Support mount options in Volume Plugins

Currently, the only way to pass configuration info to a driver is via create. This causes create to be overloaded, especially in the case of dynamic mounting instructions. For example, an NFS volume may be mounted on one host as r and on another as rw. Calling create repeatedly on shared resources in order to pass along mount/attach instructions has a tendency to warp driver design.
https://docs.docker.com/engine/extend/plugins_volume/#/volumedriver-mount
The proposed addition is an "Opts" object on the Mount request.
/VolumeDriver.Mount
Request:
{
    "Name": "volume_name",
    "ID": "b87d7442095999a92b65b3d9691e697b61713829cc0ffd1bb72e4ccd51aa4d6c",
    "Opts": {}
}

Cheers,
Ted

but you can still mount a volume read-only? -v some-volume:/some/path:ro

The original read-write/read-only example was probably not the best as I guess this option applies to the mount in the container, not the mount on the host itself.
The use case we have in mind is the distributed file system use case.  In this case, you usually need to do things like pass  the DFS address and the credentials key file, etc.   And because you can mount multiple volumes per container this really does needs to be passed as mount options.
By way of an example, we could extend the -v option like this:-
-v some-volume:/some/path:ro,dfs-addr=<IP_ADDRESS>:9999,keyfile=/etc/conf/dfs/creds.keyfile

or, we could add an additional --volume-opt (-o) like this:-
-v some-volume:/some/path:ro \
-v some-other-volume:/some/path:rw \
-o some-volume:dfs-addr=<IP_ADDRESS>:9999,keyfile=/etc/conf/dfs/creds_a.keyfile \
-o some-other-volume:dfs-addr=<IP_ADDRESS>:9999,keyfile=/etc/conf/dfs/creds_b.keyfile \

Anyway, hopefully you get the point.  For DFS use cases we would like to add mount options.


Fix/sender method error

The API was calling the Sender constructor and passing in the initial_capacity which was erroring out since it wasn't defined in the function.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: DaniGlez:x: Sam ReskalaSam Reskala seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


How to add list view webpart (Promoted links webpart) to wiki page(Home Page)

###Notice: many issues / bugs reported are actually related to the PnP Core Library which is used behind the scenes. Consider carefully where to report an issue:###

Are you using Apply-SPOProvisioningTemplate or Get-SPOProvisioningTemplate? The issue is most likely related to the Provisioning Engine. The Provisioning engine is not located in the PowerShell repo. Please report the issue here: https://github.com/officedev/PnP-Sites-Core/issues.
Is the issue related to the cmdlet itself, its parameters, the syntax, or do you suspect it is the code of the cmdlet that is causing the issue? Then please continue reporting the issue in this repo.
If you think that the functionality might be related to the underlying libraries that the cmdlet is calling (We realize that that might be difficult to determine), please first double check the code of the cmdlet, which can be found here: https://github.com/OfficeDev/PnP-PowerShell/tree/master/Commands. If related to the cmdlet, continue reporting the issue here, otherwise report the issue at https://github.com/officedev/PnP-Sites-Core/issues

Reporting an Issue or Missing Feature
Please confirm what it is that your reporting
I would like to add promoted links list view web part to my home page using CSOM for which i am not able to export list view as XML. i am using "AddWebPartToWikiPage" extension.
Expected behavior
Please describe what output you expect to see from PnP-PowerShell Cmdlets
Actual behavior
Please describe what you see instead. Please provide samples of HTML output or screenshots
Steps to reproduce behavior
Please include complete code samples in-line or linked from gists
Which version of the PnP-PowerShell Cmdlets are you using?

[ ] PnP PowerShell for SharePoint 2013
[ ] PnP PowerShell for SharePoint 2016
[ ] PnP PowerShell for SharePoint Online

What is the version of the Cmdlet module you are running?
(you can retrieve this by executing Get-Module -Name *pnppowershell* -ListAvailable)
How did you install the PnP-PowerShell Cmdlets?

[ ] MSI Installed downloaded from GitHub
[ ] Installed through the PowerShell Gallery with Install-Module
[ ] Other means

fixed


add HTML5mode support

I also added .idea in .gitignore as I use Webstorm.
I think the idea is good, but it probably needs a refactor to a seperate file for the config. Let me know what you think and I'll update it.

@samvloeberghs Thanks for contribution!
But as for .idea - this should be done in your global gitignore. So please revert this commit.
Also please squash your commits.

O well I keep sucking at rebasing :) Will create a new pull request!

Cool!