a problem with running Graphical UI

I ran a demo with the first sample(protein:6w70.pdb ligand:6w70_ligand.sdf) in graphical UI. But I ran it without any results and displayed an error message.
Show Error:----------------------------------------------------------------------------------------------------------------------------
[2024-04-26 12:41:32] [run_utils.py:124] INFO - Executing command: python3 inference.py --protein_path=/tmp/gradio/91e785aff143e89f8ac69b9fa79d954d8f9db1e9/6w70.pdb --ligand=/tmp/gradio/791e5fa3bf00a85ff5e9ad67106387c408ec2161/6w70_ligand.sdf --config=/mnt/data/sjb/chem/DiffDock/DiffDock-main/default_inference_args.yaml --no_final_step_noise --samples_per_complex=10 --out_dir=/tmp/tmpz48tjnka
[2024-04-26 12:42:09] [run_utils.py:144] ERROR - Command error:
/home/shijiabao/.conda/envs/diffdock/lib/python3.9/site-packages/torch/jit/_check.py:181: UserWarning: The TorchScript type system doesn't support instance-level annotations on empty non-base types in __init__. Instead, either 1) use a type annotation in the class body, or 2) wrap the type in torch.jit.Attribute.
warnings.warn("The TorchScript type system doesn't support "
0it [00:00, ?it/s]@> 2143 atoms and 1 coordinate set(s) were parsed in 0.02s.
/mnt/data/sjb/chem/DiffDock/DiffDock-main/datasets/parse_chi.py:91: RuntimeWarning: invalid value encountered in cast
Y = indices.astype(int)
@> 2143 atoms and 1 coordinate set(s) were parsed in 0.02s.
1it [00:03,  3.42s/it]
1it [00:03,  3.42s/it]
How should I solve this problem？
my package of the conda env:(as per your requirements)
torch                     1.13.1+cu117             pypi_0    pypi
dllogger      version: 1.0.0
e3nn                      0.5.0                    pypi_0    pypi
fair-esm      Version: 2.0.0
networkx                  2.8.4                    pypi_0    pypi
openfold     Version: 2.0.0
pandas                    1.5.1                    pypi_0    pypi
prody                     2.2.0                    pypi_0    pypi
pybind11                  2.11.1                   pypi_0    pypi
rdkit                     2022.3.3                 pypi_0    pypi
scikit-learn              1.1.0                    pypi_0    pypi
scipy                     1.12.0                   pypi_0    pypi
torch                     1.13.1+cu117             pypi_0    pypi
torch-cluster             1.6.0+pt113cu117          pypi_0    pypi
torch-geometric           2.2.0                    pypi_0    pypi
torch-scatter             2.1.0+pt113cu117          pypi_0    pypi
torch-sparse              0.6.16+pt113cu117          pypi_0    pypi
torch-spline-conv         1.2.1+pt113cu117          pypi_0    pypi
torchmetrics              0.11.0                   pypi_0    pypi
gradio                    3.50.2                   pypi_0    pypi

It's hard to understand what went wrong like this - can you provide the traceback surrounded in ``` quotes? It's easier to see what went wrong with that.

It's hard to understand what went wrong like this - can you provide the traceback surrounded in ``` quotes? It's easier to see what went wrong with that.
I don't know where the error occurred. I kept reporting errors yesterday, but today the project suddenly got through and I didn't make any changes to the code inside.I want to ask a question, how do you understand Rank 1 What about Confidence -0.02? What do the following numbers represent?

It's hard to understand what went wrong like this - can you provide the traceback surrounded in ``` quotes? It's easier to see what went wrong with that.

I don't know where the error occurred. I kept reporting errors yesterday, but today the project suddenly got through and I didn't make any changes to the code inside.I want to ask a question, how do you understand Rank 1 What about Confidence -0.02? What do the following numbers represent?


Feature requests

Could you please add nsfw (explicit and questionable) and a favourite button and make the tags clickable when viewing post tags. Thank you ❤

Hi, favourite feature is being planned, check the readme for more info.

make the tags clickable when viewing post tags.

Could you explain more what do you want it to do when the tags clicked?
For safe listing toggle, i won't enable the feature for an indefinitive time.


Different random variables for each device in monte-carlo simulation

Hi,
It looks like every device is getting independent/different random variables assigned in a monte-carlo simulation.
I also realised that the resistors were not getting any variation. This appears to be a mistake in the parameters_res_nom.spice. For example:
.param sw_sky130_fd_pr__res_high_po_rs = {325.0+corner_factor*0.0} + process_mc_factor*MC_PR_SWITCH*GAUSS(0,0.035,1)
GAUSS generates random values relative to the mean, which is 0. Therefore it always generates 0, I believe.

I'm not getting that.  Gaussian random variables calculated in a .param statement should be calculated once, not for every device.  And for the question above, process_mc_factor is 1 (which I think corresponds to 3 sigma variation), andMC_PR_SWITCH is 1 when the monte carlo corner is selected, so the parameter calculated should be (325 + (random number with zero mean)).

For the resistors I see in simulation no variation at all. The ngspice manual states that the gauss function generates random numbers relative to the mean. For that parameter it is generating 325 + (random number with 0 mean and 0 sigma) = 325. For most gaussian random variables in the pdk the agauss function is used, which generates random numbers with an absolute sigma value. I see no variation at all in the resistors with a monte-carlo simulation. By modifying the expression I could get the 3.5 % sigma variation.
I also see mismatch in transistors, which should not happen and when I place resistors in parallel and series to increase the area while still keeping the same equivalent resistance value the equivalent resistance variation is not 3.5 % 1 sigma, because every resistor is getting a different value assigned.


Standardize the way CF CLI commands are invoked

At the moment, CfCliService has 3 methods for invoking a command:

InvokeCfCliAsync
ExecuteCfCliCommand
RunCfCommandAsync

The first two are redundant (relics of an old scheme) & should be removed.
CmdProcessService has 2 methods for issuing commands:

InvokeWindowlessCommandAsync
RunCommand

Let's see if it's possible to combine these into a single method.

Update:
Turns out ExecuteCfCliCommand has a purpose after all: asynchronous methods can't be invoked within a lock statement, so it's helpful to have a synchronous alternative for those methods on CfCliService which amend the CF CLI config file (since those methods request a lock in order to make file changes)
InvokeCfCliAsync was duplicating the work of RunCfCommandAsync, so it was removed; RunCfCommandAsync is now the only asynchronous way to invoke CF CLI commands from CfCliService


setSortby not defined

I am updating the state of sortBy as below with setSortBy, but I notice error setSortBy not defined on both javascript and typescript environment. Any leads is appreciated.
Codesandbox:
https://codesandbox.io/s/restless-firefly-kmoeq?from-embed=&file=/src/App.js
From React-table documentation: setSortBy: Function(sortBy: Array<Object<id: columnId, desc: Bool = true>>) => void This function can be use to programmatically set the sortBy for the table instance
Is it possible to update the state of the sortBy using any other approach.

It's part of the table instance so, you should:
    ...
    rows,
    prepareRow,
    setSortBy,
    state: { sortBy }
  } = useTable(

Also, this seems to work for me, no need to wrap in an inner function:
setSortBy([{ id: 'firstName', desc: false }])

Thanks @david-potgieter, but I am not sure why itsnt working on previous sandbox link.
For those wanting to implement setSortby on previous versions useControlledState  would fix this.
const { rows, ...., prepareRow, state } = useTable({
  columns: columns,
  data: data,
  initialState: {
    sortBy: [{ id: "age", desc: false }]
  },
  useControlledState: state => {
    return React.useMemo(
      () => ({
        ...state,
        sortBy: [{ id: "firstName", desc: false }] // similar to setSortBy you can set here
      }),
      [state]
    );
  }
});

Also seeing the same issue - setSortBy is undefined on the table instance

Also seeing the same issue - setSortBy is undefined on the table instance

This issue is either:

Outdated
Stale
Fixed in v8
Unable to be fixed in v7 without breaking changes and is slated for v8

If you feel this is incorrect, please reopen a new issue with a codesandbox reproduction or more info regarding the issue.


Fix: Ensure OneDrive Accounts registry key is not null

Fixes #3230
Fix for OneDrive error in FullTrust log file of issue #3230

:shipit:

:shipit:


new cards

added 1,2,3,4! and Pictured as Perfect by Teennis

p.s. added 3 more cards
1,2,3,4! Overclocked Edition (1234overclock) by Teennis
ヘブンズバグ (heavensbug) by Xeno and Kin
ラッカンライア (rakkanliar) by Xeno

1234 overclocked and heavensbug need png versions for the shop

added the pngs now lmao


Weight Initialization & Layer-Specific Learning Rates

@amoudgl @sydney0zq After training some models on Caffe and checking the weights there, the only differences I found were:

Biases are initialized to 1 in Caffe (was not the case in PyTorch).
Weights are normally initialized with mean=0 and std= 0.005
Learning rates for weights and biases are different from the "overall learning rate". PyTorch does not provide support for lr_mult feature (in a direct way I mean).

The following update should show you a direct improvement on your training. For me, I get the same loss from step 100, that I used to get after ~80000 steps or so. Now, I get loss ~= 150 in the first 1000 steps which is very similar/comparable to the caffe model. Note, everything else is fixed. I am using the same training/batching code for both models.

Are you getting loss of ~150 on a batch of 11 samples? If I remember correctly, the current implementation also yields similar loss after 100 steps or so on a batch of 11 samples.

Nope. On a batch of 50 images.

What's your batching technique? Are you using the current implementation for forming batches or you implemented the one there in the author's GOTURN implementation?

Nope, I used the following code: https://github.com/devyhia/PY-GOTURN/commit/0a18a17ce68cead3237df3284a0bba6d87959672
You could check this branch out: https://github.com/devyhia/PY-GOTURN/tree/pytorch-train

Thanks a lot, @devyhia for this PR! I implemented the batch formation procedure of GOTURN in pytorch (#13) and I am able to reproduce the loss of ~150 after 1000 iterations as you stated. This patch indeed improves the training performance.


BUG: Unable to navigate to the Home page from the Menu page.

When we are on menu page and click on home page we are not able to navigate back  to home page. I can fix this issue
Video:
https://github.com/user-attachments/assets/5962b4e1-cf7e-43f6-9928-7c509fd48b0d

[x] I follow contributing guidelines
[x] I am gssoc'24 contributor.
[x] i want to work on this issue.

@sunny0625 Please mention the level of the issue . Otherwise I will not get the points.


📝 Create an Awesome list for reference architectures & compelling examples

github.com/sindresorhus/awesome/blob/main/create-list.md
https://github.com/SSWConsulting/awesome-clean-architecture

@Hona can we close this now?

Yes.


fix PyPi Description issue

the description field was set to what should be the long_description field -- making a bit of a mess on PyPi

that was fast!


External DNS cannot assume role on EKS built via eksctl

External DNS creates a restricted role for Route53 operations. Clusters created via eksctl cannot assume this role as they lack sts:AssumeRole permission.
It worked for a while due to Principal: "*" in assume role policy, removed via cdde8460.
https://aws.amazon.com/blogs/security/how-to-use-trust-policies-with-iam-roles/
We ask users to deploy eksctl clusters with iam.withAddonPolicies.externalDNS: true, thus this is the recommended way to access External DNS functions.
Propose to remove Terraform from External DNS component. @oginskis @akranga

AssumeRole is probably redundant there... I understand motivation for not to allow assumeRole for the instance profile. I have got a kind of similar issue with r53 when was schedulling deployments on eks... Workaround was the following.
hub ext eks policy attach -c anton-4 -f etc/policy.json

That means cluster administrator grants permissions to manage r53 outside of the components code. Happy to discuss

No solution is more secure than the other, yet the current one is harder to explain and taints the component with Terraform, making it not portable as is.

Agree... Primary motivation was the compatibility with control plane yet in expense of portability.

On EKS the official way to cloud is via OIDC.


Fix: project tribute error handling

prevent UI from crashing when a user without manager rights navigates to the tribute page

Note to @ivelin that this PR is ready for review per @cptnanon
https://discord.com/channels/962821474777972786/964949121804959774/1022209350191829103

@cptnanon this looks great. Would you like to add a check for when the user is disconnected. Currently this is what they see:

Also, how would you suggest we implement test coverage to prevent regressions?


Give logger provider a ProviderAlias attribute

Please add a ProviderAlias attribute to the logger provider so that a shorter name can be used in appsettings.json.

If I want to use a different log level for logging default and this logger provider, is there a better way other than using the long name in the appsettings.json? And what's the concern to have the dependency on Microsoft.Extensions.Logging? Just curious...

Provider Alias has been added to v3.


Bump to v1.2.0

Fixed

Bug: Handle exception when media attachments are geoblocked (403 Unauthorized). Closes #88
Regression: Take into account new structure of Twitter's archive (tweet.js changed to tweets.js)
Bug: Handle special media types that don't include link to media in some cases. Thanks @tewhalen! Closes #90
Bug: Align max_tweets minimum value (10) with the warning message and actual value. Thanks @nemobis!
Bug: Multiple video attachments cause HTTP 422 on Mastodon when posting. Closes #92
Bug: Exception when post was manually pinned on the Fedi account and later one was pinned on the Twitter account. Thanks @selyod-ka! Closes #104

Enhancements

Archive offline support, you no longer need a Twitter developer account or access to their API to process an archive. Closes #93
Mastodon limits, make URLs count as 23 characters (or the instance configured value) when determining if truncating is necessary. Closes #95
Added progress bars when gathering, processing and posting tweets
Pleroma and Mastodon rate limits, handle HTTP 429 more gracefully (wait until they reset and continue) Closes #75
Added fallback for unknown software on target instance. Closes #103

Added

RSS support, you can now use an RSS feed as a source of tweets to post. Take a look at the docs for more info. Closes #74
Thread support, now mirrored users should be able to reply to their own posts and create reply threads. Closes #91
Native retweet support, now users in your config should be able to reblog their own and each other's posts if they are on the same instance. Closes #91
bot config mapping, for setting the bot flag on the target account. Closes #96
--lockerfile argument, for specifying the path of the locker file that prevents collisions between multiple concurrent bot executions. Closes #98
guest mapping, for enabling the use of Guest Tokens (so you don't need a Twitter Developer account). If no twitter_token is present in your config it will default to true
proxy_pool mapping, list of proxies to use when being rate limited with Guest Tokens
proxy mapping, for enabling or disabling the use of proxies when using guest tokens
Caching IDs of posts published and skip mirroring tweet if associated published post is found to avoid duplicates. You can change this with the avoid_duplicates mapping. Closes #101
application_name mapping, for specifying the Fediverse application name to use as a filter when getting the date of the latest published post by the bot. Thanks @reorx!
Configuration wizard, which generates a minimal config if none is found
content_warnings, for specifying keywords that if found will add a content warning to the Fediverse post
custom_replacements for replacing any text (Twitter handles that differ from the Fedi handle, for example) Thanks @selyod-ka! Closes #105
software config mapping, for sidestepping the automatic detection of the type of software (mastodon, pleroma, misskey) running on the target instance and forcing a specific value. If your target is a fork of another project, for example, you would be able to force the bot to use whichever software (mastodon, pleroma, misskey) is the closest match for their API.

Hi @robertoszek,
Please allow me to ask a couple of questions regarding this release:

Native retweet support, now users in your config should be able to reblog their own and each other's posts if they are on the same instance.

Is the above compatible with the custom_replacements mapping feature? Considering the following:
custom_replacements:
"@twitter_user1": "@fedi_user1"
"@twitter_user2: "@fedi_user2"

On Twitter @twitter_user1 RTs @twitter_user2. Will the bot/instance realize they are retweeting each other if the custom_replacement mapping is being used?

application_name mapping, for specifying the Fediverse application name to use as a filter when getting the date of the latest published post by the bot. Thanks @reorx!

What problem does this solve?
Should all bots have different application names? Currently I have 3 mastodon users in place using the same application name (pleroma-bot). Will this be a problem?
Thank you for your time.

Hey @tomakun
Sure, let me try to answer the questions you had.

Is the above compatible with the custom_replacements mapping feature? Considering the following:

Short answer: Yes.
Long answer: The native retweet/thread feature of the bot does not check for the contents or text on the Fediverse post.
Instead it checks if the tweet ID returned by Twitter's API in the field "replied_to" or "retweeted" was published on the Fediverse instance.
It does so by searching the tweet ID on the posts.json file generated at the config file path.
So no matter if the text was changed (by adding a signature, a date, a replacement, etc.) the ID of the post on the Fediverse instance and the tweet ID will remain unchanged.
Regarding application_name:

Should all bots have different application names? [...] Will this be a problem?

Not really. It shouldn't be a problem, if those Mastodon users are just publishing posts with the bot.

What problem does this solve?

Let me try to explain, take this hypothetical example.
I'm an user who is using the bot with my personal Mastodon account.
I'm mirroring my Twitter account into my Mastodon account and it's working fine.
One problem though, the bot checks the date of the last post published on my Mastodon account.
But I happen to post often manually from my Mastodon client, so the bot wrongly assumes the last post I created from my client is the last one mirrored from Twitter (as it takes the date from the latest post).
So, here comes application_name, if I specify that the bot is publishing posts with the app name pleroma-bot for example, it can check which post was actually the latest one published with the bot.
No matter if I'm posting manually from other clients or from the Mastodon web app, it will continue mirroring from where it left off.

@robertoszek Many thanks for the explanations, this is all very clear. The application_name implementation is clever! Great stuff.
Thank you again for your time and support, I'll make sure to support myself via your other contributions channels.


Update CMake from vcpkg patch

This patch will simply make build of tests, example and documentation optional.
And CONFIG option is missing in find_dependency in Config.cmake

I understand the idea. But I don't think it's a good idea.
For example, when you include(CTest), it automatically set BUILD_TESTING to 'ON'. You can't set the name.
If you want to avoid clashes, you should pass EXCLUDE_FROM_ALL to add_subdirectory(units). So you will build only the needed target.

Sounds good. I also forgot about that consumers should use ./src, so it's all OK as is.

For example, when you include(CTest), it automatically set BUILD_TESTING to 'ON'. You can't set the name.

So it means that if someone will include(CTest) in the customer project before adding dependencies then all the dependencies will be built with tests and influence the compile times or possibly even make the compilation of the customer's project fail. I do not think it is a good idea. So many times I saw third-party unit tests in CTest that I never wanted to compile and run.
In general, I do not think it is a good idea. It explodes the number of CMake configurations to support and test. Why vcpkg cannot just use ./src/CMakeLists.txt for a build?

The choice about prefixing or not CMake global variables is complicated. I didn't find an official response about it.
If you don't prefix and you use add_subdirectory from another project, you will set variable from this subproject that you may don't want. The solution is to use the feature EXCLUDE_FROM_ALL.
If you prefix, then you break some features of CMake.
As units library looks to decide to prefix CMake variables, I will modify my PR according to this.

Is it possible for vcpkg to just use src/CMakeLists.txt directly? In such a case, no changes are needed, and the customer just gets the library code.

It seems that this PR is no longer needed as a patch to user ./src was applied to vcpckg.


Add a flag to Supermium 126 that returns the old address bar size that was in Chrome 68-18?

Go to chrome://flags/, search for "Omnibox Steady State Height", set the flag to "Disabled" and relaunch the browser; the vertical space taken by the address bar will be reduced; applicable to BOTH Sm 122/124; knowing how evil Google :rage: are towards anything "older", I expect this flag/setting to disappear in future Chrome releases...

Go to chrome://flags/, search for "Omnibox Steady State Height", set the flag to "Disabled" and relaunch the browser; the vertical space taken by the address bar will be reduced; applicable to BOTH Sm 122/124; knowing how evil Google 😡 are towards anything "older", I expect this flag/setting to disappear in future Chrome releases...

Thank you very much, I just didn't know that such a flag existed. The most important thing is that win32ss makes this flag disabled by default, as well as all elements of the new interface

Please don't remove the titles of your issues. People can no longer easily search for them.


[Suggestion] Is it time for col-xxl?

With newer screens getting bigger that easily goes double the col-xl width. I believe its time for col-xxl with more or less 1920px?

I'd say no out the box, but you can easily customize it on your code by changing the breakpoints map.

Nope, add it when you need it—all that can be easily generated from source code.

Ok, no. I get that I can add it when I need it, that's just like everything else. But, what's the reason behind "no"?


kie-issues#380: Replace the patternfly webjar in stunner-editors

Closes: https://github.com/kiegroup/kie-issues/issues/380
This PR adds a fixed version of the the patternfly 3.18.1 webjar.
It fixes the following vulnerabilities included in the original org.webjars.bower:

CVE-2022-24785, WS-2016-0075, CVE-2017-18214 & CVE-2022-31129<EMAIL_ADDRESS>brought on package tests (via cdnjs links).
CVE-2018-17567: jekyll.gem which is brought by the original patternfly build.

The (build) kie-tools-stunner-editors check has failed. Please check the logs.

The (build) kie-tools-stunner-editors check has failed. Please check the logs.

The (build) kie-tools-stunner-editors check has failed. Please check the logs.

The (build) kie-tools-stunner-editors check has failed. Please check the logs.

@paulovmr @jesuino would you mind taking a look? just added the patternfly webjar into kie-tools to solve some vulnerabilities there and changed the mvn dependency in some modules.

The (build) kie-tools-stunner-editors check has failed. Please check the logs.

The (build) kie-tools-stunner-editors check has failed. Please check the logs.

Closed in favor of https://github.com/kiegroup/kie-tools/pull/1834


Add responsiveness to contact us page

Make contact us page responsive. right now it not working make it responsive and display confirmation message

I want to work on this issue, please assign it to me,
and tag it hacktoberfest and gssoc labels. @tushargupta1504


Upgrade to support AWS provider v4

what

Upgrade to support AWS provider v4
Upgrade to latest s3 bucket module

why

This module is currently unusable in projects using AWS provider v4

/test all

/test all

/test all

/test all


Disable services collectors in default configuration

In environments where I do not care about service status, the services collector generates a huge amount of metrics when using the default install (as Windows comes with many services). I request that the Windows services collector be disabled by default, to keep the default set tidy and data-size-efficient.

We are unlikely to change defaults to suit your specific needs.
Please configure the collectors you need using -collectors.enabled.

This is not about my specific needs. I would think it to be in the interest of this software to provide reasonable defaults. It does not seem reasonable that the majority of the default metrics are relatively static metrics about services.
What is the rationale behind the current set of defaults?

I have the same opinion as martinlindhe.
A Monitoring agent should monitor services by default.

The rationale about the current defaults is that they are supposed to be available on all supported OS:es.
We can not offer a "set and forget" for monitoring. The user needs to think about what they want to monitor and base their decisions on that.

In my environment I need the textfile collector only and I thought by using "--collectors.enabled textfile" all other collectors are disabled but they are still enabled.
So, that option does not overrule the default collectors.
What about a "--collectors.disabled" instead?

@MichaelHellmann Hm, that would be a bug in that case - using --collectors.enabled should explicitly set only those collectors and disable all others.

Looking closer, we even have a unit-test for this. Could you paste your full invocation?

I installed it in my Windows environment by
msiexec /i C:\Prometheus\wmi_exporter-0.5.0-amd64.msi ENABLED_COLLECTORS=textfile
and the service looks like this
"C:\Program Files\wmi_exporter\wmi_exporter.exe" --log.format logger:eventlog?name=wmi_exporter --telemetry.addr :9182 --collectors.enabled textfile
On http://localhost:9182/metrics I see a lot of wmi_* data.

Thanks. If you find wmi_exporter_collector_success, which metrics are returned there?

I just reinstalled the service and it looks much better now :-)
I guess I made a mistake when I installed it the first time.
Now it looks like this.
Thanks a lot and sorry for the confusion.
wmi_exporter_build_info{branch="master",goversion="go1.11",revision="2766f0e3af9a7570d6b812eaf4bf262754c2d47f",version="0.5.0"} 1
# HELP wmi_exporter_collector_duration_seconds wmi_exporter: Duration of a collection.
# TYPE wmi_exporter_collector_duration_seconds gauge
wmi_exporter_collector_duration_seconds{collector="textfile"} 0.0010122
# HELP wmi_exporter_collector_success wmi_exporter: Whether the collector was successful.
# TYPE wmi_exporter_collector_success gauge
wmi_exporter_collector_success{collector="textfile"} 1
# HELP wmi_textfile_mtime_seconds Unixtime mtime of textfiles successfully read.
# TYPE wmi_textfile_mtime_seconds gauge
wmi_textfile_mtime_seconds{file="fa2ad.prom"} 1.544195285e+09
# HELP wmi_textfile_scrape_error 1 if there was an error opening or reading a file, 0 otherwise
# TYPE wmi_textfile_scrape_error gauge
wmi_textfile_scrape_error 0

:+1:


Remove Alamofire dependencies.

Fixes 71782
(IMPORTANT: Please do not create a Pull Request without creating a story in ClubHouse first)
Description of changes in release / Impact of release:
Remove alamofire dependencies.
Screenshots / gif (if customer facing):
(insert image here)
Value being added to the product or risk being removed by this release
(insert text here)
Changes to documentation (put link to pull request if some EXISTING docs page needs changes)

[ ] My change requires a change to the documentation.
(insert link here)
[ ] I have updated the documentation accordingly.
(insert link here)

Backout procedure
Deploy the previous release.

File README.md

Note that VGSCollectSDK use Alamofire and CardIO as dependencies. You should also link them to your project.


parmed md5 mismatch

Downloading https://github.com/ParmEd/ParmEd/archive/2.7.1.tar.gz
Success
MD5 mismatch: 'd19c063087de4d06652cf0ee3fcd5014' != 'f8387c2dc16ba627d2a3fcbf1909b48f'
Please note, that the Github archives are created on the fly and it is likely that it produces different outputs (eg. they change the compression flags etc.).

Is this a failure of the parmed recipe to build?


Button click events overcounting

I've played around with events on localhost and they record 1 button click correctly. However, when deployed on the web, I'm getting a single button click counting for 4-8+ events.

Are you manually sending the events or using the css classes?


Introduce transient profiles

Introduce transient profiles. These profiles are deleted automatically once the last window which uses it is closed.
You can create and associate a transient profile from

CLI by passing --profile-transient flag. Eg: code <folder-uri> --profile-transient
UI from Command Palette using command Create Transient Settings Profile

These will create a transient settings profile and associate it to the workspace. Once the workspace is closed (or uses different profile) this profile is deleted.
Fixes #154460
CC @bpasero

Are we thinking this is going to be used a lot from CLI over --profile? I am asking because we are a bit sensitive to what we put top level when you run code-insiders --help to not confuse the user with too many options. If we think this is only used in e.g. debug launch configs then maybe it should not be top level advertised?

Agreed. I assume --profile will be most commonly used from CLI than --profile-temp. I would move the latter into non-help args section.


Error during installation with bun

UnoCSS version
0.64
Describe the bug
D:\vue3-bun>bun add unocss
bun add v1.1.27 (267afa29)
error: moving "@esbuild/win32-x64" to cache dir failed
EPERM: Operation not permitted (NtSetInformationFile())
From: .38c7ed7b2ffafefe-00000001.win32-x64
To: @esbuild/win32-x64@0.23.1@@@1
error: InstallFailed extracting tarball from @esbuild/win32-x64
Reproduction
ewq
System Info
No response
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.

I test and install it on my local, and works fine, please check your BUN.


Fix typos and grammatical errors in V2 API doc

In this PR,

Fix typos and grammatical errors.

@NataliaTepluhina, May you please review it once.

Thank you so much for the PR! Unfortunately, a lot of these grammar fixes are either unnecessary or not really accurate. We appreciate the contribution though.


Attempt to parse predict input as JSON

We want to unescape things like \n that are escaped to \\n when serialized, but still fall back to the raw value if that fails for any reason (like the input not deserializing being to a string).

After a conversation, I'm not actually convinced this is the right thing...

@erbridge Thanks for jumping on this.
As we discussed, I think a better solution would be to follow the pattern of echo and other utilities, and add a -e flag (and its inverse, -E) to enable escape sequences. Attempting to do this automatically, such as by first parsing as JSON, could produce unexpected results (such as passing code like print("Hello,\nthere") as a text input).


Mozilla.Firefox-msix version <IP_ADDRESS>

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Why can't this just be part of the Mozilla.Firefox manifest?

The version number format of this msix package is different from the Mozilla.Firefox manifest. For example, Mozilla.Firefox 94.0 is Mozilla.Firefox-msix <IP_ADDRESS>

That's true, but due to Firefox's versioning scheme it shouldn't matter in practice (they only use to the second decimal point for any release), it shouldn't matter (<IP_ADDRESS> would upgrade to 94.0.1 in winget's logic).

That's true, but due to Firefox's versioning scheme it shouldn't matter in practice (they only use to the second decimal point for any release), it shouldn't matter (<IP_ADDRESS> would upgrade to 94.0.1 in winget's logic).

WinGet also "ignore"/remove zero's, so if we flipped the numbers <IP_ADDRESS> would not show up as a upgrade for version 94.0 (Its just good practice to keep the 0's if the versions are the same)
This can also be corrected in Manifest Version 1.1 with ease since the msix dosn't break winget

#34147

Why can't this just be part of the Mozilla.Firefox manifest?

Then how can the user choose which package to install between the exe package and the msix package?


Add BigQuery copy operator.

@mistercrunch has been allowing Google hook/operator commits without tests because 1) they're in contrib and 2) we're the only known people using them.
We do have "add tests" on the list of things to do, but it's kind of non-trivial right now due to the dependency on google cloud storage and bigquery.


Give labels individual colors

Coloring labels differently would go a long way towards making the calendar visually parseable and inviting

The Olin Library Color Palette is:
[
  '#000000',
  '#A7A9AC',
  '#ED037C',
  '#009BDF',
  '#26AAA5',
  '#349E49',
  '#F47920',
]

maybe a good starting point?

I've done some reading and come up with some other options:

Kelly's 22 colors of maximum contrast: https://gist.github.com/ollieglass/f6ddd781eeae1d24e391265432297538
i want hue tool: http://tools.medialab.sciences-po.fr/iwanthue/
color brewer: http://colorbrewer2.org/?type=qualitative&scheme=Paired&n=12#type=qualitative&scheme=Paired&n=12

Olin color palette:
olin_colors = ['#009BDF', '#A7A9AC', '#A7A9AC', '#000000',  # school colors
    '#E31D3C', '#750324',  # extended palette
    '#F47920',
    '#FFC20E', '#C05131',
    '#C0D028',
    '#8EBE3F', '#00653E',
    '#349E49',
    '#26AAA5', '#00677E',
    '#6BC1D3',
    '#009BDF', '#00458C',
    '#7B5AA6',
    '#C77EB5', '#511C74',
    '#ED037C'
]


Wiki site server redistricting access to the docs

The wiki is not accessible at all currently.

Closing, the Wiki was unavailable for emergency maintenance.  It's back up again.  See announcements on mailing lists: https://groups.google.com/g/dspace-community/c/BbMDHOHx6NM


get-childitem directory: shows wrong directory

Go to c:\windows directory and then get-childitem c:\users. -> directory: shows c:\windows, it should show c:\users.

This works fine for me. What exact version of Windows, what exact version of Powershell, what version of PSColor, did you install from here manually or from PowerShell Gallery, etc.?

win 10 1709 16299.309 x64
PSVersion   5.1.16299.251
Script     <IP_ADDRESS>    pscolor
Installed from powershell gallery, install-module..
PS C:\MaestroNG\Devel\Installation> Get-ChildItem c:\temp
Directory:  C:\MaestroNG\Devel\Installation

Mode                LastWriteTime     Length Name

d-----       24.10.2017     13.56        1   b
d-----       18.10.2017     11.33        1   bb

That's what I'm on as well. I can't even imagine a scenario where PSColor would be interfering with this.
PS C:\windows\system32> get-computerinfo -property windows*

WindowsBuildLabEx              : 16299.15.amd64fre.rs3_release.170928-1534
WindowsCurrentVersion          : 6.3
WindowsEditionId               : Professional
WindowsInstallationType        : Client
WindowsInstallDateFromRegistry : 2/27/2018 9:49:00 PM
WindowsProductId               : 00330-80000-00000-AA917
WindowsProductName             : Windows 10 Pro
WindowsRegisteredOrganization  :
WindowsRegisteredOwner         : Windows User
WindowsSystemRoot              : C:\Windows
WindowsVersion                 : 1709

PS C:\windows\system32> $psversiontable.psversion

Major  Minor  Build  Revision
-----  -----  -----  --------
5      1      16299  251

PS C:\windows\system32> get-module pscolor

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Script     <IP_ADDRESS>    pscolor                             Out-Default

PS C:\windows\system32> get-childitem c:\temp

    Directory:  C:\temp

Mode                LastWriteTime     Length  Name
----                -------------     ------  ----
-a----        5/16/2017   3:30 PM       2653 vrice.pfx

Well, I am able to reproduce this on my home computer as well. Similar specs. Exactly same result as with the work computer. I didn't have anything powershell related installed on my home machine.
Install-Module pscolor
Import-Module PSColor
PS C:> dir C:\Temp\
Directory:  C:\

Mode                LastWriteTime     Length Name

d-----        18.3.2018     23:12        1   NVIDIA
Here with debug:
(just dir c:\temp)
PS C:> C:\Users\veepee\Documents\test.ps1
Hit Line breakpoint on 'C:\Users\veepee\Documents\test.ps1:2'
[DBG]: PS C:>>
Stopped at: {
[DBG]: PS C:>>
Stopped at: if(($_ -is [System.IO.DirectoryInfo]) -or ($_ -is [System.IO.FileInfo]))
[DBG]: PS C:>>
Stopped at: FileInfo $_
[DBG]: PS C:>>
Directory:  C:\

Mode                LastWriteTime     Length Name

d-----        18.3.2018     23:12        1   NVIDIA
Stopped at: $_ = $null
[DBG]: PS C:>>
Stopped at: $steppablePipeline.Process($_)
[DBG]: PS C:>>
Stopped at: }
and one more time with and without the module..
PS C:\Users\veepee> dir c:\temp
Directory: C:\temp

Mode                LastWriteTime         Length Name

d-----        18.3.2018     23:12                NVIDIA
-a----        10.1.2018     18:16        2240808 fe1dbb5a98cb4443939b32b1858eec08-le64.cache-4
-a----       30.10.2016     13:08         231028 rep.html
-a----        9.12.2017      2:23           9734 t.txt
PS C:\Users\veepee> Import-Module PSColor
PS C:\Users\veepee> dir c:\temp
Directory:  C:\Users\veepee

Mode                LastWriteTime     Length Name

d-----        18.3.2018     23:12        1   NVIDIA
-a----        10.1.2018     18:16     2,14MB fe1dbb5a98cb4443939b32b1858eec08-le64.cache-4
-a----       30.10.2016     13:08   225,61KB rep.html
-a----        9.12.2017      2:23     9,51KB t.txt
PS C:\Users\veepee>

Weird. I have two Win10 VM's, both work as I showed above. I've never seen anything like that as long as I've been using PSColor (since I started playing with Win10, say six months or so). Sorry I can't be of more help. David (the author) might have an idea, but he doesn't frequent here very often these days.
Just for grins you might get the latest pscolor.zip from here and install it and see if it makes a difference. It has a change to allow the module to be removed (the way the original worked it borked the system if it was removed). It shouldn't matter — the example I posted earlier was from a stock PowerShell Gallery install just like yours.

I've got the same problem, and have experienced it right from day 1 (when I first installed PCColor many years ago). I've just installed the latest version (by extracting from the zip file). In running on Windows 10 Pro, build 10.0.16299
I've got a directory structure as follows:

The output of gci -r c:\pccolortest without the PSColor module...

The output of gci -r c:\pccolortest with the PSColor module...

(Disclaimer: I'm not the developer, just a contributor.)
 I've never had the issue (three different machines, multiple VM's), either with the original version of PsColor, or with the various updates I've made to it (some local only). I'm on Win10 1709 though, so it's possible (though unlikely IMO) that might make a difference.

On May 17, 2018, at 4:49 PM, Keith Fletcher<EMAIL_ADDRESS>wrote:
I've got the same problem, and have experienced it right from day 1 (when I first installed PCColor many years ago). I've just installed the latest version (by extracting from the zip file). In running on Windows 10 Pro, build 10.0.16299
I've got a directory structure as follows:
https://user-images.githubusercontent.com/3274184/40205588-b2c5708e-5a80-11e8-962b-b1cf937f688b.png
The output of gci -r c:\pccolortest without the PSColor module...
https://user-images.githubusercontent.com/3274184/40205478-4b2d3b1e-5a80-11e8-9a46-f3f1ccaae7aa.png
The output of gci -r c:\pccolortest with the PSColor module...
https://user-images.githubusercontent.com/3274184/40205522-6e0b5f44-5a80-11e8-8ac9-3d71d66bc83b.png


Deploying to Heroku

Have tried a few different things (a few too many times) and can't seem to get it to deploy to Heroku.  Either fails after compiling or in step 2/4 packaging dependencies. Maybe something wrong with my webpack config? 🤷♂

Heroku doesn't seem to be recognizing the Procfile

without buildpacks:
remote:        Bundle completed (47.21s)
remote:        Cleaning up the bundler cache.
remote: -----> Installing node-v10.14.1-linux-x64
remote: -----> Installing yarn-v1.12.3
remote: -----> Detecting rake tasks
remote: -----> Preparing app for Rails asset pipeline
remote:        Running: rake assets:precompile
remote:        yarn install v1.12.3
remote:        [1/4] Resolving packages...
remote:        [2/4] Fetching packages...
remote:        error<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
remote:        error Found incompatible module
remote:        info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
remote:        I, [2019-04-18T14:12:51.314277 #1439]  INFO -- : Writing /tmp/build_ff5bdf38f773c7315e2116f5504a4bbb/public/assets/application-13bf5de8a04c31ae70b41fb6c8feb4cedcac80a4025bdbc434e41c587df0a7e0.js
remote:        I, [2019-04-18T14:12:51.314672 #1439]  INFO -- : Writing /tmp/build_ff5bdf38f773c7315e2116f5504a4bbb/public/assets/application-13bf5de8a04c31ae70b41fb6c8feb4cedcac80a4025bdbc434e41c587df0a7e0.js.gz
remote:        I, [2019-04-18T14:12:51.320580 #1439]  INFO -- : Writing /tmp/build_ff5bdf38f773c7315e2116f5504a4bbb/public/assets/application-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.css
remote:        I, [2019-04-18T14:12:51.320713 #1439]  INFO -- : Writing /tmp/build_ff5bdf38f773c7315e2116f5504a4bbb/public/assets/application-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855.css.gz
remote:        Webpacker is installed 🎉 🍰
remote:        Using /tmp/build_ff5bdf38f773c7315e2116f5504a4bbb/config/webpacker.yml file for setting up webpack paths
remote:        Compiling…
remote:        Compilation failed:
remote:        /tmp/build_ff5bdf38f773c7315e2116f5504a4bbb/vendor/bundle/ruby/2.5.0/gems/webpacker-3.6.0/lib/webpacker/webpack_runner.rb:11:in `exec': No such file or directory - /tmp/build_ff5bdf38f773c7315e2116f5504a4bbb/node_modules/.bin/webpack (Errno::ENOENT)
remote:        	from /tmp/build_ff5bdf38f773c7315e2116f5504a4bbb/vendor/bundle/ruby/2.5.0/gems/webpacker-3.6.0/lib/webpacker/webpack_runner.rb:11:in `block in run'
remote:        	from /tmp/build_ff5bdf38f773c7315e2116f5504a4bbb/vendor/bundle/ruby/2.5.0/gems/webpacker-3.6.0/lib/webpacker/webpack_runner.rb:10:in `chdir'
remote:        	from /tmp/build_ff5bdf38f773c7315e2116f5504a4bbb/vendor/bundle/ruby/2.5.0/gems/webpacker-3.6.0/lib/webpacker/webpack_runner.rb:10:in `run'
remote:        	from /tmp/build_ff5bdf38f773c7315e2116f5504a4bbb/vendor/bundle/ruby/2.5.0/gems/webpacker-3.6.0/lib/webpacker/runner.rb:6:in `run'
remote:        	from ./bin/webpack:15:in `<main>'
remote:
remote:
remote:
remote:  !
remote:  !     Precompiling assets failed.
remote:  !
remote:  !     Push rejected, failed to compile Ruby app.
remote:
remote:  !     Push failed
remote: Verifying deploy...
remote:
remote: !	Push rejected to react-on-rails-spa.
remote:
To https://git.heroku.com/react-on-rails-spa.git
 ! [remote rejected] prod-deploy -> master (pre-receive hook declined)
error: failed to push some refs to 'https://git.heroku.com/react-on-rails-spa.git'

with Node > Ruby buildpacks:
remote: Compressing source files... done.
remote: Building source:
remote:
remote: -----> Node.js app detected
remote:
remote: -----> Creating runtime environment
remote:
remote:        NPM_CONFIG_LOGLEVEL=error
remote:        NODE_ENV=production
remote:        NODE_MODULES_CACHE=true
remote:        NODE_VERBOSE=false
remote:
remote: -----> Installing binaries
remote:        engines.node (package.json):  unspecified
remote:        engines.npm (package.json):   unspecified (use default)
remote:        engines.yarn (package.json):  unspecified (use default)
remote:
remote:        Resolving node version 10.x...
remote:        Downloading and installing node 10.15.3...
remote:        Using default npm version: 6.4.1
remote:        Resolving yarn version 1.x...
remote:        Downloading and installing yarn (1.15.2)...
remote:        Installed yarn 1.15.2
remote:
remote: -----> Installing dependencies
remote:        Installing node modules (yarn.lock)
remote:        yarn install v1.15.2
remote:        [1/4] Resolving packages...
remote:        [2/4] Fetching packages...
remote:        error<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
remote:        error Found incompatible module
remote:        info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
remote:
remote: -----> Build failed
remote:
remote:        We're sorry this build is failing! You can troubleshoot common issues here:
remote:        https://devcenter.heroku.com/articles/troubleshooting-node-deploys
remote:
remote:        Some possible problems:
remote:
remote:        - Node version not specified in package.json
remote:          https://devcenter.heroku.com/articles/nodejs-support#specifying-a-node-js-version
remote:
remote:        Love,
remote:        Heroku
remote:
remote:  !     Push rejected, failed to compile Node.js app.
remote:
remote:  !     Push failed
remote: Verifying deploy...
remote:
remote: !	Push rejected to react-on-rails-spa.
remote:
To https://git.heroku.com/react-on-rails-spa.git
 ! [remote rejected] prod-deploy -> master (pre-receive hook declined)
error: failed to push some refs to 'https://git.heroku.com/react-on-rails-spa.git'

Solved! Using the buildpacks, setting the versions, and/or removing<EMAIL_ADDRESS>fixed and successfully deployed! :)

Fixed with #14 and #15 😄


Use version range for tree-sitter dependency

Using this range avoids the Rust compiler error,
expected struct `tree_sitter::Language`, found a different struct `tree_sitter::Language`
perhaps two different versions of crate `tree_sitter` are being used?

which is encountered when using this crate along side tree-sitter-python, tree-sitter-javascript and tree-sitter-typescript. The range is consistent with the range used in those crates e.g. https://github.com/tree-sitter/tree-sitter-python/blob/master/Cargo.toml).

Thanks!


[text analytics] uncomment out multi page healthcare tests

Uncomment out these tests and make sure they work. Might need to update tests as well now that healthcare is multi page.
Might have to wait until GA of 3.1, since 3.1-preview.5 still can only return single page

There wasn't a customer ask to increase the document limit for v3.1 so it will remain at 10. Since service-side paging is set at 20 documents, pagination from the service will not be activated yet. Leaving these tests commented out and moving to backlog.

There's no reason to leave this issue open tracking this since the service has no future plans to do this. Going to remove the commented out tests  - we can use the PR to add them back if they are ever actually needed.


starting with testling

Hi!
I'm reading quick start guide and when I install testling and use a testling command:
$ browserify test/max.js | testling

this is output error:
events.js:85
      throw er; // Unhandled 'error' event
            ^
Error: spawn Xvfb ENOENT
    at exports._errnoException (util.js:746:11)
    at Process.ChildProcess._handle.onexit (child_process.js:1046:32)
    at child_process.js:1137:20
    at process._tickCallback (node.js:355:11)

This is my code so far:
https://github.com/xgbuils/testling-max-by
What's going on?

fixed https://github.com/substack/testling/issues/105


Missing config field checks

In receiver_config_unmarshal and in sender_config_unmarshal, for some fields we have checks if (tempObj == NULL) return -1, and for some we don't have them.
I've tried to add checks for missing fields, and for every field when I tried to add a check, test started failing, with an exception stating that sender or receiver config is invalid (i.e. the added checks are failing).
Below I list the fields which don't have checks (and I was unable to add them).
Receiver:

clock_source
resampler_backend
resampler_profile

Sender:

packet_channels
packet_encoding
clock_source
resampler_backend
resampler_profile
fec_encoding

They're enums and can be null, which correspond to "use default". We should add explicit check for those nulls.


Vertex Gemini Custom Model Feature

@HenryHengZJ - I've added Custom Model feature to ChatGoogleVertexAI node based on this example https://github.com/FlowiseAI/Flowise/pull/3088.
Tested and works great.
Hope that works / helps test experimental models released on Vertex!

Closing this PR and opening a new one with specific branch for this feature.


pilot-agent stats: pass query parameter to envoy

Please provide a description of this PR:
Support pass query parameter to envoy.
For example, currently when scape :15020/stats/prometheus?usedonly, the pilot-agent does not pass the usedonly query string to Envoy. This may result in collecting too many metrics and leads to inconsistent behavior compared to Envoy's /stats/prometheus?usedonly endpoint.
https://www.envoyproxy.io/docs/envoy/latest/operations/admin#get--stats?usedonly

Hi @masquee. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@masquee: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
1f3b6ef511d6043f04b170f2c1724f0fa80b244f
link
true
/test release-notes

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

@masquee: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
1f3b6ef511d6043f04b170f2c1724f0fa80b244f
link
true
/test release-notes

integ-ipv6_istio
1f3b6ef511d6043f04b170f2c1724f0fa80b244f
link
true
/test integ-ipv6

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

I am not 100% sure about this... this is changin the behavior, queries (in dashboards, alerts, etc) will be changed by this, as not found != 0, especially when doing things like rate
cc @keithmattix

If users doesn't include a query parameter like usedonly when using it, the behavior remains unchanged. However, if users includes it, the current behavior does not meet the user's expectations.

Yeah I'm inclined to believe that this is an opt-in change. Were you thinking something different @howardjohn?

Ah my bad, didn't realize the conditional. lgtm

/retest

@masquee: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
1f3b6ef511d6043f04b170f2c1724f0fa80b244f
link
unknown
/test unit-tests-arm64

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

/retest

@masquee: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
1f3b6ef511d6043f04b170f2c1724f0fa80b244f
link
true
/test unit-tests-arm64

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

/retest


Add support to allow variable name in path for maintain

do you want to update the readme with the changes you made as part of this PR? would make it easier for me to understand how it works 😄

do you want to update the readme with the changes you made as part of this PR? would make it easier for me to understand how it works 😄


Is there a way to add attachment file from content string

Can I add an attachment file from string of content ? (not from File instance)
I dont want to use Tempfile or File.
like: (and how should I add the file name?)
ticket.comment.uploads << params[:logs]
ticket.save!

Thanks

Hello @razbensimon
Zendesk api client uses Faraday as http client and the Faraday::UploadIO class to provide the uploading functionality. So theoretically, you could use StringIO class to pass a string. However, if you do that, you would still need to pass the filename in the request, which cannot be guessed from a StringIO instance.
The Upload Middleware provided within in the gem, gets these values from a File instance. So, unfortunately, if you want to use the gem as it is, you need to provide a File instance or a file path to the uploads accessor.
I am gonna close this issue, but feel free to open it again if you have any further questions.


Add role.permissions helper method, tidy export files for rbac classes

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.98%. Comparing base (34217a7) to head (2ae369f).
Report is 3 commits behind head on dev/1.28.

Additional details and impacted files
@@            Coverage Diff            @@
##           dev/1.28    #1455   +/-   ##
=========================================
  Coverage     91.97%   91.98%
=========================================
  Files           174      174
  Lines         14830    14848   +18
=========================================
+ Hits          13640    13658   +18
  Misses         1190     1190

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


Config File Mismatch

When config mismatch occurs, tell the user where the mismatch is:
for page in config.keys():
    section = config[page]
    section_new = config_new[page]
    for variable in section.keys():
        if section[variable] != section_new[variable]:
            print(page, variable)

I think the error you are talking about is this one in the notebook.py?

Yes I just didn't want to correct it at the time.

No worries, I can get around to this later on if it is still wanted

Added the following warnings to the compare config function.

This is excellent


WSDL derived datatypes in PHPDoc block

Hi,
So, I am trying to test the Demo code by introducing different WSDL data types (derived datatypes) and i'm getting the following error:
"Cannot add a complex type anyURI that is not an object or where class could not be found in "DefaultComplexType" strategy."
This happens when I add a new parameter to getProduct like "@param anyURI $test". How can I use WSDL derived datatypes? Using primitive datatypes works fine...
Many thanks in advance! :)

I think using the fully qualified class name should do the trick.

@viewflex I've got no .../anyURI class. "string" is a native datatype for both WSDL and PHP but "anyURI" is only a native datatype for WSDL, there is no native tive anyURI for PHP. So, in order to use a WSDL anyURI type should I create a PHP class with that name?

No, there is no anyURI type in PHP, but the WSDL anyURI type is essentially just a string. You could create a PHP class to define the type, but I'm not sure that's necessary.

Yes, you're absolutly right @viewflex thanks! :)
There's this image at w3.org that shows it.

Changing to array might work. If you feel like sharing your code I maybe I or someone could suggest a solution.

I followed your previous answer and everything went well @viewflex ! So for the record:

Fully qualified class name (full path to class) is required.
WSDL datatypes that in practice are text (token, anyURI, language, etc), can be defined as string.
For the arrays (string[], SomeComplexType[]) , I was able to make it work by changing the strategy to ArrayOfTypeSequence.

Working with these changes, I was able to use both objects and arrays with simple and complex types.
Everything is working fine! :)

That's great! Thanks for the update. :)


Bottom Right Fab Button Is not Fixed

Hello Again Zach, I have tried to add the bottom right fab button, and succeeded, but the button is not fixed to it's right corner, and on different displays it will show or not show... instead of being fixed in place, like the top fab button, which works great in this manner.
How can i fix this problem?
Thanks so much,
Shaul.

Hey Shaul, I see what you're seeing now. I've made note and this will be fixed in the upcoming release.

Thanks Zach, have a great day :)

@zachsoft was this fixed?


Questions about filters

Filters are for exact match only ? So:

case sensitive
no fuzzy matching

Can you explain why this query have results (have a filter on city):
/search?city=mmmmmm&q=rue

Here is why:
> TOKENIZE mmmmmmmm
m

;)

I'm still not understand filters. Since there are tokonized, why there are case sensitive?
/search?city=paris&q=rue des lilas

Give no result, but
/search?city=Paris&q=rue des lilas

give results (note the capital).

I setup an filter on "city", but the result depend on case of query
/search?city=Paris&q=rue%20des%20lilas

return results. But (note "p" in lower)
/search?city=paris&q=rue%20des%20lilas

return no result.
This one return result,
/search?city=m&q=rue

I'm almost sure I have no city named "m". And more over, results containts cities that haven't "m" in names.


Card size classes don't work correctly on cards without images

This is the code taken directly from the docs for a card with the simple addition of the small class:

          Card Title

I am a very simple card. I am good at containing small bits of information.
          I am convenient because I require little markup to use effectively.

          Projects

If the width of the card is narrow, you can see that the text at the bottom gets cut off, as well as adding a block of white space that is the size of what the image would be if there was one.

Additionally, with image cards that have text that overflows into the space that the card-action div occupies, it is not cut off. This is because the height of the card-action div isn't taken into account when giving the card-content and card-image heights.

Yeah, really annoying

I can confirm this issue.
Additionally, the card-content class is having issues wrapping text for non-image cards. Some of the letters may get truncated on various screen sizes.
Message me if any of you can close in on what's causing it and lets try to get this fixed.

I've been really busy with other projects lately, so I haven't had time to work on this, but I'll bet that using flexbox would solve the problems.

This happens because the heights of card-content and are static
fix:.
==SASS==
.card.small
    height: auto
    .card-content
        height: auto
    .card-action
        position: relative
==CSS==
.card.small {
  height: auto;
}
.card.small .card-content {
  height: auto;
}
.card.small .card-action {
  position: relative;
}

@rikato Thanks!

solved.


Querying TMX for certain author

With Xaseco there was the possibility to query for a certain author, as example:
/xlist auth:nadeo, to get all TMX tracks uploaded by Nadeo.
This function doesn't exist yet in trakman apparently (i was talking to wise and he told me to open a issue, so i'm requesting this as a feature.

22ad49a, e.g.: /xlist $a aropix. due to new tmx api limitations only full author names are supported (weird stuff from them imo)


Forbidding zombies from destroying AME and Gravity

A few changes to forbid zombies from destroying AME and Gravity. This is a stub, as I can't compile the project nor test it in-game. The general idea is to forbid zombies from attacking entities that contain specific components.

this could be extender to most consoles and ape/other machine that distribute power, chemdispenser/chemaster
they are supposed to be too dumb to know which machine is vital or not to the station

I think it should be put in CL so players do not wonder why they can't destroy them.

what about forbidding them from destroying any type of walls too? zombies should only go after crew members and not eating every walls of the station

breaking walls may be necessary to reach the crew and I don't think zombies ripping apart every wall is an issue rn

also even if this was the right way to go about it, having zombie reference ame directly is very very very very bad use an event

Which events do you refer to? Sorry, I'm not familiar with the engine yet, I basically started 5 hours ago xd

Gravity gen is already immortal.

Remember the shitter when making changes like this.

i died pls restart

this is the wrong solution. if you want to make these forbidden to destroy then make them electrocute on melee attack, like APCs/substations/SMES already do

It seems a bit too punishing to me, could get in the way during a legit fight. Are you sure?

not like zombies go out of their ways most of the time, spacing the station, destroying vital points and appliances of the station. i'm sure they'll start welder bombing if this merge on WizDen

not like zombies go out of their ways most of the time, spacing the station, destroying vital points and appliances of the station. i'm sure they'll start welder bombing if this merge on WizDen

If they do, I'll add more exceptions.

this is the wrong solution. if you want to make these forbidden to destroy then make them electrocute on melee attack, like APCs/substations/SMES already do

It seems a bit too punishing to me, could get in the way during a legit fight. Are you sure?

Yes.

this is the wrong solution. if you want to make these forbidden to destroy then make them electrocute on melee attack, like APCs/substations/SMES already do

It seems a bit too punishing to me, could get in the way during a legit fight. Are you sure?

Yes.

Could someone point me to the class which electrocutes zombies? I'm having a hard time finding it :(

Could someone point me to the class which electrocutes zombies? I'm having a hard time finding it :(

Have a look at the change which introduced it here : https://github.com/space-wizards/space-station-14/pull/16584

Could someone point me to the class which electrocutes zombies? I'm having a hard time finding it :(

Have a look at the change which introduced it here : #16584

So the file was already setup to electrocute on damage, but doesn't work for some reason o.O

the bug seems to be in ElectrocutionSystem.cs at line 269:
tryNode.NodeGroup is IBasePowerNet { NetworkNode: { LastCombinedSupply: > 0 } })

If I let the condition pass through, the game crashes at https://github.com/space-wizards/space-station-14/blob/4783a655a48f343fe297ca6679211d3426bd1b00/Content.Server/Electrocution/ElectrocutionSystem.cs#LL322C4-L322C4

Just making sure, you are not wearing captain gloves when testing the electrocution? They are insulated

Just making sure, you are not wearing captain gloves when testing the electrocution? They are insulated
Also probably better to join the discord for this and get help in the #howdoicode channel, I'm not an expert contributor but referencing ame in electrocution system seems like the wrong way to do it

don't zombies autodrop gloves

Just making sure, you are not wearing captain gloves when testing the electrocution? They are insulated
Also probably better to join the discord for this and get help in the #howdoicode channel, I'm not an expert contributor but referencing ame in electrocution system seems like the wrong way to do it

don't zombies autodrop gloves

Even if they do, I still feel like there is a better way then referencing ame in electrocution system directly

dont apc sub and smes have components for being electrified

dont apc sub and smes have components for being electrified

And so have AME shields, but it seems the current source doesn't make use of them, as I said: https://github.com/space-wizards/space-station-14/pull/16963#issuecomment-1571742420
No, my actual question here is why github fails to "build & test debug", while it works fine locally :\

I don't understand why it keeps failing here: https://github.com/space-wizards/space-station-14/actions/runs/5146945641/jobs/9266690219?pr=16963#step:10:16

Prices have nothing to do with my changes, I don't understand why github keeps failing here: https://github.com/space-wizards/space-station-14/actions/runs/5146945641/jobs/9266690219?pr=16963#step:10:16

Someone merged a broken PR so now master is broken and tests are failing on all PRs.

@mirrorcult @PJB3005 I'm waiting for your approval o.O

Ignore the reviewers tab it's meaningless.

Shouldn't you just make a friendlyfire faction component.

Do you mean these classes? They seem to be for the AIs, not prevent friendly fire between players :
https://github.com/space-wizards/space-station-14/blob/9852bba84776ad914f5607bb00a393c0cd0aff29/Content.Server/NPC/Components/FactionComponent.cs#L8
https://github.com/space-wizards/space-station-14/blob/821f4cc3d4427510869894ea3a97d88a1bd418f7/Content.Server/NPC/Systems/FactionSystem.cs#L10

Thank you for the PR.  For what it's worth, here are my thoughts on zombies:
Regarding the friendly fire portion:  I am in favor of preventing friendly fire between zombies.  It's very easy for zombies to get confused who is a zombie and who isn't, and I've accidentally been mowed down as a zombie by a mob of friendly zombies myself.  If the hit effect is prevented entirely, it will hopefully cue zombies that they're attacking a fellow zombie and they'll focus their efforts elsewhere.
Regarding the AME portion:  Short version is I agree an electric shock as suggested above (or otherwise something that stuns) would be the best route.
Long version is that fundamentally, it's not very engaging to RP as a braindead creature.  Entertaining the first round, sure, but after that it doesn't stay engaging because the player is not supposed to make any meaningful decisions as a zombie.  Without any meaningful decisions or interactions, gameplay becomes stale and I think is a major contributing factor to why people disliked zombie rounds.
I think less-dumb zombies would be much more interesting to play as.  Limited communication and abilities, but still able to work together and pose a challenge to the station.  Zombies should be easily defeated, their principle strategies being either to quickly sneak in and convert humans, or to work together to take down larger targets like infrastructure/sec.
To that end, a non-lethal shock would still allow zombies to work together to make decisions to go after infrastructure should they wish to go that route -- while requiring teamwork or otherwise a lot of time between stuns.
Here are some specific design thoughts on zombies vs infrastructure:

If a single human armed with one of the tools found in the engi area can't reliably defend the AME against a single zombie, the zombie stats should probably be further nerfed to a point where this works out.  (Last time I played a zombie round, it was basically impossible to survive as a human without a ranged weapon.)
Additionally, a zombie, perhaps even a couple zombies, probably shouldn't be able to get through an electrified door easily.  It should take a large group effort / considerable time to get through these.  Perhaps knocking them out for an extended time after each bite to give humans time to notice.
Finally, if AME is taken down, perhaps it should be easier to call in for rescue (or even mechanisms to repair AME) so players aren't stuck indefinitely on a doomed station?

Thank you for the PR. For what it's worth, here are my thoughts on zombies...

Thanks for your comment. Some of your suggestions are outside the scope of this PR, but I agree with you. I'd also add that I don't like playing as zombie, I'd prefer them to be AIs, but...

@metalgearsloth can this PR be merged? It works. The only thing I'm sorry is the fact you mentioned, that I had to hardcode AMEengine in ElectrocutionSystem.cs, but it's a temporary fix to an existing bug that I don't know how to fix myself :(

temporary fix pls merge :pleading_face: :pray:

change the name and description, considering the implementation is completely different.

change the name and description, considering the implementation is completely different.

Name matches current implementation, what do you mean? o.O

"webedit temporary fix pls merge 🥺 🙏 "

If you mean the AME Shields bug fix, I could exclude it from the PR and focus it just on the zombie vs zombies stuff, but there is no need to be sarcastic.


Added minZoom and maxZoom to props

In my case, I needed the maxZoom prop because all markers are really close each other, and the clusters didn't revealed very close markers...
PS: Please review and document new prop uniqueIdProperty...

We purposely kept this as TODO, we already are tweaking min/max zoom for fixing strange SuperCluster behavior under certain circumstances.

Cool!


Vex can now be called as a module

I found this to be really valuable when testing.
$ python -m vex ....

vex is a command line utility. No reason for "python -m vex" when "vex" does the same thing.


Add further debug info to flaky test output

Add more debug info to be able to diagnose the flakiness of JavaApplicationDeploymentIntegrationTest.can run application.

@bot-gradle test QFL

OK, I've already triggered the following builds for you:

QuickFeedbackLinux build

@bot-gradle test and merge

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


AOT fails

Hey folks,
Getting following error will compiling with ngc
Error: Error encountered resolving symbol values statically. Could not resolve angular2-modal relative to .../node_modules/angular2-modal/plugins/bootstrap/modal-context.d.ts.
Can any one looks into this.
Regards,

Duplicate of #273
I


Back port force-sending of email

Back port the ability to force sending of email form the config file.

@akaul ?

@thebigjc r+


External dataset re-downloaded when a different scene from a previously downloaded dataset is used.

Steps to reproduce:

Invoke nerfbaselines with scene A from dataset X, eg. external://mipnerf360/garden
Invoke nerfbaselines with scene B from dataset X, eg. external://mipnerf360/kitchen

Expected behavior
Dataset is downloaded on first invocation, cached download is used on second invocation
Actual behavior
Dataset is downloaded for each scene separately.

Hi, by default, when you use --data external:// only the specified scene is downloaded and cached locally. Unfortunately, for mipnerf360 this means a zip file with multiple scenes is downloaded and only the specific scene is extracted (to save space on disk). If you want to download all scenes from a dataset use the following command: nerfbaselines download-dataset external://mipnerf360. See here: https://nerfbaselines.github.io/docs/cli.html#nerfbaselines-download-dataset This will download can cache all scenes so that consecutive commands do not need to download anything.

Thanks for the reply, I understand. In my opinion it would be best if the download would be cached even when downloading the individual scene. But I fully understand that that's likely not a priority.

That is the case currently, the downloaded scene is cached


blur does not hide the menu

In version 3.01 (with elm 0.17) the menu is not hidden if the focus is removed. Selecting an item still hides the menu so I guess the "onBlur" in the div is not working anymore.

Yes, this is an issue with the virtualdom elm implementation. didn't realize it was a problem here as well as in #13. Thanks for letting me know!

I saw that this was resolved using an onBlur event, then subsequently changed to handle it a different way... though using 3.2.0 the menu isn't going away on blur for me.

@demsullivan thanks for the catch! I was manually testing and missed this! Solved Esc, but not blur. 3.2.1 should have the fix. click events fire after blur (on release of the mouse), onMouseDown will allow the input to still be filled in when selecting a menu item with the mouse.
This just means: time to write some tests! 😅


Credit Transfer | Remove the ability for BCeID users to see the Review status |3|

https://trello.com/c/7xogFspj/757-credit-transfer-remove-the-ability-for-bceid-users-to-see-the-review-status-3
This card is to remove BCeID users ability to see transactions (credit transfers & PVR) that have a Reviewed status.
For Credit Transfer Proposals, the transaction should remain visible but simply stay in the Signed 2/2 state (status & breadcrumb).
For PVRs, the transaction should remain hidden while in the Reviewed state. Meaning, BCeID users can only see the transaction after it has been approved by the Director.

 Credit Transfer | Remove the ability for BCeID users to see the Review status |3|


feat(nn): add CausalConv1d

Proposed changes
This pull request introduces CausalConv1d, a 1-dimensional causal convolution layer designed for time series tasks such as WaveNet and Temporal Convolutional Network.
The CausalConv1d layer ensures that each output at time ( t ) is computed using inputs up to and including time ( t ), without depending on future inputs. This is crucial for maintaining causality in time series predictions.

Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the necessary documentation (if needed)

This is really nicely implemented. Since it's a fairly simple modification of Conv1D that's used only in a few specific cases, let's hold off on adding it to mlx.nn.
We can revisit this if it becomes a common need.


Consider adding 3DoF controls to support Oculus Go

Would you like to look into that and submit a pull request? I speculated on how that would be possible in this technical write-up here: Space Rocks—WebXR tech deep dive. Search for the phrase “add 3DoF support” to find the relevant paragraphs.


[USGS]

Modified USGS harvester from PR #432 (PEP8 fix)

@MerlinZhang Nice! Looks like you have scrapi/processing/cassandra.py still modified tho... I'd look into putting those back the way they were.
Also if you could add to your PR description that you're also modifying travis-dist and local-dist just so we have record of what's changing


Java 11 support

Could you please make this compatible with java 11?

Yes, I think that it makes sense to update it to JDK 11+ since Presto recently (a few days ago) moved they product tests to JDK 11 as well.
I'll try to take a look in the next few days.

Done in https://github.com/rchukh/presto-querylog/pull/15
Available in 0.3 version - https://github.com/rchukh/presto-querylog/releases/tag/0.3

Just to be clear - it should have been compatible with JDK 11 before as well.
At least according to Travis - https://travis-ci.org/rchukh/presto-querylog/jobs/568324696
Now it requires JDK 11 and I have checked that it actually works with Presto running on JDK 11.


0.2.0 AppStore Compatibility

0.2.0 enacted "Changed: XCTFail is no longer restricted to DEBUG builds."
It seems like Apple does not like this in production. When submitting for review I got the response that included: "Non-public API usage - The app references non-public symbols in ---: _swift_FORCE_LOAD$_XCTestSwiftSupport."
In order to compile my project I also need to set "Enable Testing Search Paths" to Yes or I get the undefined symbol _swift_FORCE_LOAD$_XCTestSwiftSupport
Should this package only target testTargets in SPM?

Ah interesting. We can revert that change and explore other ways of restoring test functionality outside debug builds in the future.

Thanks!

Just released 0.2.1! Please let us know if you have any more issues.

@JacksonUtsch I know it's been some time but do you remember how this App Store submission failure occurred? Was it immediately during the submission process? Or did it come later on in the review period.
We're revisiting this because we'd love to make XCTestDynamicOverlay work in release builds, so it'd be helpful to know what Apple is on the watch for and how.

@stephencelis This occurred after a successful submission, I'm pretty certain it was automated. The error code given was ITMS-90338. There are online positing of issues with people utilizing private APIs having the same issue.
I'd be interested to know the use case of having this work in release builds rather than no-op.

@JacksonUtsch Being able to run tests on release is valuable, and possible using vanilla XCTest tools, but because we currently compile everything out of RELEASE builds, no code using XCTestDynamicOverlay can be run in release at all.

Hi @JacksonUtsch, sorry to resurrect such an old issue for the second time, but we are once again looking into the possibility of removing #if DEBUG from this library.
Upon re-reading your original issue it seems that your problem may have actually been unrelated to the 0.2.0 release that removed the #if DEBUG. It seems like you explicitly turned on the "Enable Testing Search Paths" setting in your Xcode project, which force links XCTest with the application target. That seems to be the actual reason your application was rejected, not due to the symbols that ship in this library.
Can you explain a bit more why you turned on "Enable Testing Search Paths"?

"Enable Testing Search Paths" was enabled to get the compiler to build when I got the error from adopting 0.2.0 Undefined symbol _swift_FORCE_LOAD$_XCTestSwiftSupport. I was able to reproduce this error after some digging. If you import this branch into a project and make sure xctest is at 0.2.0 you should get the same error.

Hi @JacksonUtsch, thanks for the repro, but I don't think this is related to xctest-dynamic-overlay at all. I think this is related to the fact that you are importing XCTest directly here. Can you try removing that and see if it fixes your issue?
The if #DEBUGs should have no bearing on your library at all since it is building for debug anyway.

@mbrandonw Hmm, I reluctantly agree 🙈. I removed the import earlier and still had the issue but this time I clean and rebuild and the error goes away!
I now wonder what the sequence of events that happened originally to make it seem like it was a issue with 0.2.0. All the better to move forward with release support then 🙃


ARM support

Can you provide version for ARM? I want to use this library on Allwinner H3 CPU (Quad-Core ARM Cortex-A7) under Linux.

aarch64?

aarch64?
@ailike
No, we need libssp for 32-bit ARM (ARMv7).

Any update on ARM support for Nvidia's Jetson TX2/Nano?

@ailike, any news about linux-version for arm? We really need it!

Bump

I'd like to see something I can use on an STM32


68 add logo for the package

Pull Request for addition of the logo. The specifications are in the following line in the README.Rmd file:

For reference, the dplyr logo has a height of 138 and lacks a specified width. Our logo is specified to be a bit larger, just because of the lettering, but can easily be scaled back down.


syntax error in requirements.txt

Hi, upura,
I found a very tiny error in "requirements.txt".
The required packages should be separated by a new line, and a comma is unnecessary here.
If run pip install -r requirements.txt for the present file, following error will occur.

pip._vendor.packaging.requirements.InvalidRequirement: Invalid requirement, parse error at "','"

Please consider removing commas from that file.
Thank you.

@resnant Thanks for your issue!


ci: test with go1.20

Coverage: 99.666% (-0.2%) from 99.863% when pulling a4ae796c4e6ca2614061fe9d3fd9c923ed623e93 on go1.20 into d0c499ef9c7044cdb7bb8a0cdad1ca78d16939ee on master.


Close image file before adding to IPFS

Coverage remained the same at 27.422% when pulling 80bc1e929c15039a4d910e87b8f64f54f1dd7ac3 on images into 5746bf5b641b29f79f7fa3f8c3cb3e909831518e on master.


Can not find module "uiohook-napi" after installation .exe

I am using electron forge with webpack and typescript, uiohook-napi is working properly in my development platform. Compiling the program is also working. But when I tried to install the .exe file this error is appearing.

My enviroment
uiohook-napi Version: ^1.5.3
Environment name and version: nodejs 20.9.0, electron: 29.1.5,  electron-forge: ^7.3.1
Operating System and version: win32 x64

I think you need to force electron-forge to exclude uiohook-napi from the ASAR archive for it to work in packaged form. I am doing this using packagerConfig in my forge configuration, see https://www.electronforge.io/config/configuration#electron-packager-config. You can explicitly specify uiohook-napi to not be packed in the ASAR using the asar unpack or unpackDir options.

Hey @marcelblum, I appreciate your help. Unfortunately, I tried what you suggested, but I'm still encountering the same error 😔.
I've sharing my forge.config file for your reference. Could you please review it and let me know if I've made any mistakes? I'm relatively new to Electron, so I apologize in advance for any beginner errors.
Thanks again!
packagerConfig: {
    asar: {
      unpack: "node_modules/uiohook-napi/**", // Exclude uiohook-napi from ASAR
    },
  },

You can test it by looking in the folder structure of your packaged app to see if the files are outside the ASAR as desired. If successful you should find the excluded files under \resources\app.asar.unpacked\. But actually if you want to exclude a whole folder you can use unpackDir instead of unpack. unpackDir: "node_modules/uiohook-napi" should do it.

I have tried both unpackDir: "node_modules/uiohook-napi"  and unpack: "node_modules/uiohook-napi/**" together as well as separately, but nothing works, stil the same error.
what i've tried so far:
 asar: {
      unpack: "node_modules/uiohook-napi/**", // Exclude uiohook-napi from ASAR
    },

asar: {
      unpackDir: "node_modules/uiohook-napi"
    },

asar: {
      unpack: "node_modules/uiohook-napi/**",
      unpackDir: "node_modules/uiohook-napi"
    },

I am using forge make command for creating packaged app and I cannot find \resources\app.asar.unpacked\, I can only see \resources\app.asar

if you're not seeing a \resources\app.asar.unpacked folder then something's not right in the way you have your packagerConfig set up. Either the config is not being properly passed to forge or there's some syntax error or something. As a quick test you can just try asar: false which should result in no asar being created, instead there should just be a \resources\app\ folder containing your source files (side note: this is also a good test to confirm that the asar was the problem preventing uiohook-napi from working in the first place). If you still have a \resources\app.asar file even with asar: false then your packagerConfig is being ignored or overridden.

Hello all!
I'm having the same issue here: https://github.com/Dygmalab/Bazecor/tree/removeFocusAPI (open software with GPLv3 license so that you can compile it yourselves with yarn make)
I found this plugin from Forge to make use of the asar packager, but it fails either way.
https://www.electronforge.io/config/plugins/auto-unpack-natives
This only fails in Windows platforms, in Mac it's working perfectly when packed.

@batoolfatima2135 did you manage to solve the issue on your side?

so, just for the record, I found the reason.
After updating xstate to v5, electron-forge started failing in the packaging process, to solve it, I had to update Typescript package from v4 to v5, but that broke the Windows native package's prebuilt binaries compilation.
The issue was within the filter in forge.config.ts at the packageAfterPrune function was due to a change in how the RegEx detected the prebuilds path to remove the precompiled binaries from other platforms than the one being compiled, it deleted them all, thus failing to execute when packaged.
Changing the filter to look for the platform independently solved this issue and maintained the optimization.

that's great you found solution @alexpargon, actually on my side I just used vite instead of webpack and it started working


PCBC-327 and PCBC-328

PCBC-327: Query parameter 'group_level' is not compatible with 'group'.
PCBC-328: range() method causes 'bad_request: invalid UTF-8 JSON: {{error,insufficient_data},[]}'

This is off subject here, I just upgraded to php5.6.6 and the php-couchbase 2.0.4 lib..
The example:
// array(1) {
//   ["name"]=>
//   string(5) "Frank"
// }
Returns it as:
// object(stdClass)#644 (1) {
//   ["name"]=>
//   string(5) "Frank"
// }
The only way to not get it like that, is to serialize on upsert(), and unserialize on get().. but that won't work as I have an existing couchbase db, with unserialized objects.
I'm either thinking this is an issue with my php5.6.6 compile, or just this new php-couchbase lib. I've tried all versions to 2.0.0.. no luck.  I can give more details, but my configure was:
./configure' '--enable-mbstring' '--with-curl' '--with-openssl' '--with-xmlrpc' '--enable-soap' '--enable-zip' '--with-mysql' '--enable-fpm' '--with-gd' '--with-pdo-mysql' '--with-jpeg-dir' '--with-zlib'

It looks like objects with libcouchbase2 are being json encoded by default.. whereas before, they were serialized.. which makes this backwards incompatible?  Unless there's a setting somewhere I'm unaware of?

Solved the problem with doing:
$COUCHBASE_DEFAULT_ENCOPTS['cmprtype']=COUCHBASE_COMPRESSION_ZLIB;

I am going to close this now as I believe that 5f32dbb6839373531ec5ccbb53dc4c3690d7d6bd fixes these issues.
Cheers, Brett

a94bc61868133be018c6c0a292eda281add2dc4a is related as well.

In terms of your php-ext-couchbase compatibility issue, I actually would suggest that you build your own object with the same properties as $COUCHBASE_DEFAULT_ENCOPTS, and then use setTranscoder to switch to a custom transcoder which internally calls couchbase_basic_encoder_v1.
Example:
$MY_ENCOPTS = array(
    'sertype' => COUCHBASE_SERTYPE_JSON,
    'cmprtype' => COUCHBASE_COMPRESSION_ZLIB,
    'cmprthresh' => 0,
    'cmprfactor' => 0
);
function my_encoder($value) {
    global $MY_ENCOPTS;
    return couchbase_basic_encoder_v1($value, $MY_ENCOPTS);
}
$bucket->setTranscoder("my_encoder", "couchbase_default_decoder");

So I'm unable to get old encoded objects..  it just returns a null value..
key=22000847u5_3_
CouchbaseMetaDoc Object
(
[error] =>
[value] =>
[flags] => 52
[cas] => Resource id #61
)
but if I look in the couchbase web console for the document '22000847u5_3_', i see:
"GVoAAHicvTxrj+O4kX9F6QMOt0B6mhRJPdwIkuxudpNNdgZ3s5v5KMg2261rWfJIcrc7k/nvKb4kSqb8kDoBBuixSBbrzapiSR8WdHHza82rmwXDiy+1+Nm87vjNfd2OwH/9xU22vrnPFj5l8JMsbvbid72IFje+jxCKaCh+xoubPK2bpOa8ENOjCNHQ930YwkSPrdNXexxFPo1hPFzcrCqeNlzuEwZhQGLMFExYUrdrGIWHAcwWkGrxhAjwgOKmzNfJMs3TYgUErBc4jJEYQnqoKZs0FwM+YSjQix7K0lqULcwK+Vyv0E+Bgl2ZFY29hx/FONSgeMGrzetgEe6eZ1tumOIzNli1LIt9LSBSsxVPq7xHDsEaoBpqycFEwgIOPlQZL9bJS5UWm5wnaSP3YyiOaCznCGby1dNjejwLs ............................. (trimmed) ................ "


.Net: Discussion: Idea behind output_variable & json_schema?

I am looking through the source and trying to figure out what the use case is for output_variable and the json_schema of the type (C# source, Microsoft.SemanticKernel.OutputVariable)
It is not in use anywhere that makes sense to me, is this for future-proofing for a potential scenario where the kernel abstracts enforcement of an output schema when the LLM is being used in json mode?

Yes, this is for future proofing. We will eventually use this information for code-based planners so that the LLM knows what the return type is for complex objects.


Kubernetes Services for Egress Traffic fails on 1.4.0-alpha.0

Bug description
Working through the Kubernetes Services for Egress Traffic example at https://preliminary.istio.io/docs/tasks/traffic-management/egress/egress-kubernetes-services/
Under the Use a Kubernetes service with endpoints to access an external service section, step #6 shows accessing the wikipedia service from the Istio sidecar pod with Istio's mTLS disables. This step should get a valid response, but end up getting a exit code 35
Expected behavior
Example should pass
Steps to reproduce the bug
Follow the instructions for the task at https://preliminary.istio.io/docs/tasks/traffic-management/egress/egress-kubernetes-services/
Version (include the output of istioctl version --remote and kubectl version)
1.4.0-alpha.0
How was Istio installed?
Since the installation specified in the example is broken, I used:
helm template install/kubernetes/helm/istio --name istio --namespace istio-system \
    --values install/kubernetes/helm/istio/values-istio-demo.yaml \
    --set global.controlPlaneSecurityEnabled=true \
    --set global.mtls.enabled=true | kubectl apply -f -

Environment where bug was observed (cloud vendor, OS, etc)
IKS, but problem was replicated elsewhere (though not sure where)

I did try the same example on 1.3.3 using the 1.3 docs (demo profile) and it fails with the same error code.

@vadimeisenbergibm could you help check this out when you have some time?

@linsun Sure, will do.

@linsun @ericvn The issue is probably related to https://github.com/istio/istio/issues/17112, that bug should be fixed first.

As an FYI, this was merged into preliminary.istio.io with https://github.com/istio/istio.io/pull/4710 and cherry-picked back to istio.io (default 1.3 branch) with https://github.com/istio/istio.io/pull/5157

As an FYI, this was merged into preliminary.istio.io with https://github.com/istio/istio.io/pull/4710 and cherry-picked back to istio.io (default 1.3 branch) with https://github.com/istio/istio.io/pull/5157. Per discussion, reverted in 1.3 for now with https://github.com/istio/istio.io/pull/5254.

I think we will need #17112 fixed and cherry-picked back into 1.4 (how big is that effort), or we want to revert it out of the 1.4 documentation as well (unless we can fix it so that it works (with the mentioned workaround?)).

@ericvn let me check https://github.com/istio/istio/issues/17112, will update you.

17112 is not a bug as far as I can tell, so please revert/fix the docs

@ericvn https://github.com/istio/istio.io/pull/5261 should fix this issue.

@vadimeisenbergibm I did verify the new page instructions work. Thank you.


Tight layout options

This would add the feature of https://github.com/lukelbd/proplot/issues/64, the ability to set hspace between all rows, wspace between all columns, or both, based on the calculated maximum value in ProPlot's layout routine.
I am not sure if this follows what you were suggesting in this comment. It seems to work, but the gridspec's idea of hspace and wspace don't seem to match how the figure shows up (see my example, maybe I am printing the wrong thing).
See in action here.

10 years later... looks good to me, thanks! Here's an example with this feature turned off and on:
fig, axs = plot.subplots(ncols=3, share=False, wequal=False)
axs[2].format(yformatter='null')
axs[1].format(ylabel='ylabel\nylabel\nylabel')
fig.show()

fig, axs = plot.subplots(ncols=3, share=False, wequal=True)
axs[2].format(yformatter='null')
axs[1].format(ylabel='ylabel\nylabel\nylabel')
fig.show()

Will add this to the changelog + update the contributors list

(Closes #64)


Enable HTTPS/SSL on rasa core server

Rasa Core version: 0.11.3
Python version: 3.6.2
Operating system (windows, osx, ...): windows 10
Issue:
I am trying to hit RASA Core server (http) from my site (https), I am getting the below error

Mixed Content: The page at 'https://mysite.com/index.php#' was loaded over HTTPS, but requested an insecure XMLHttpRequest endpoint 'http://:5005/conversations/default/respond'. This request has been blocked; the content must be served over HTTPS.

I need to enable SSL in the RASA Core server, please share any docs?
Content of domain file (if used & relevant):

hi i am looking for running over nginx any .conf file or link that can help.

create ssl certificate and key using certbot. and replace the key file and crt file location respectively
rasa run --ssl-certificate /etc/letsencrypt/live/(Website)/fullchain.pem  --ssl-keyfile  /etc/letsencrypt/live/(Website)/privkey.pem  --enable-api -m model


Can't write files after receiving

Every time I receive a transferred file the app crashes and I see the following error in the logs:
java.lang.SecurityException: Unsupported path /storage/emulated/0/Download/<redacted filename>

Environment: v1.0.3 from the GitHub release on LineageOS 17.1 (Android 10) on a Sony Xperia XZ1C
I've also seen the app crash in the same way on a Pixel 2 running stock Android 10 but can't check the logs to verify it's the same crash (but it looks like it).
On both devices the app didn't ask for any permissions. Newer Android versions have locked down writing to the SD card more tightly so the app might need to request the storage permission in order to work. See https://developer.android.com/training/data-storage

This is a duplicate of issue #9. Will close this and comment there on the current state.

Whoops, I excluded that one while looking for an existing issue solely based the title. Sorry for the noise!


Fix issues with character encoding in non-UTF-8 environments

If LANG on the system is not UTF-8 and the output contains special characters, then mob_recon fails to read the mob_typer files with the following error 'UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 824: ordinal not in range(128)' . This does not occur when LC_ALL=UTF-8 is explicitly set for the environment. All MOB-suite functions which read data from disk should check character encoding and handle these cases.

The failure occurs at line 200 of mob_recon link where readlines() function takes system character encoding that is non-UTF8. Will try to fix it with pandas dataframe function or open(filename,'r',encoding='utf-8') with encoding parameter.

We had addressed this issue in the new release 2.0.2 by providing explicit UTF-8 encoding  in all open functions. In addition in mob_recon module we've changed code to use pandas dataframe with utf-8 encoding. See line 191

By having non-ASCII characters in the Host range files host_range_literature_plasmidDB.csv, host_range_ncbirefseq_plasmidDB.csv, this means that any stray print statements involving the data from these files has a chance to error out due to non-ascii characters. I have stripped the non-ascii characters from both files and we will need to update the code to properly address non-ascii characters at run time for all input data to be safe.

During host range database production, the non-ascii characters (i.e. Ê, é, ú, ñ) were intentionally introduced during copy and paste from the literature sources. The non-ASCII characters in host_range_ncbirefseq_plasmidDB.csv and host_range_literature_plasmidDB.csv could cause unpredictable behaviour on systems not supporting the unicode characters. I've went through the files replacing the non-ASCII characters with their equivalents.
I've used grep to highlight and find characters outside ascii hexadecimal character range x00-\x7f.
Also in vi it is possible to search for non-ascii characters via /[^\x00-\x7F] command.
grep --color='auto' -P -n "[\x80-\xFF]" host_range_ncbirefseq_plasmidDB.csv
11856:CP022018,8798Ê,57.11525347,-,-,-,-,-,-,-,-,N

I've patched both files and submitted commit to the string_encoding branch

closing this for now but will re-open if it pops up again