publishChecks step: the conclusion should be none when status is not completed

Currently, the publishChecks step has a default success value for the conclusion, however, it should reset the conclusion to be none when the status is queued or in_progress. Otherwise, users may have to explicitly set the conclusion to be none since, like GitHub, it will automatically set the status to be completed when conclusion is provided.
See the context: https://github.com/jenkinsci/checks-api-plugin/issues/70#issuecomment-765606154

Hi @XiongKezhi, If this fix is not super urgent, can I work on this?

Hi @XiongKezhi, If this fix is not super urgent, can I work on this?

go ahead! thanks!

go ahead! thanks!

@XiongKezhi, is this issue fixed, or am I checking at the wrong place?
https://github.com/jenkinsci/checks-api-plugin/blob/64dc0a6c55c704c1d716d5532ed16cce9358584c/src/main/java/io/jenkins/plugins/checks/status/BuildStatusChecksPublisher.java#L128-L129
https://github.com/jenkinsci/checks-api-plugin/blob/64dc0a6c55c704c1d716d5532ed16cce9358584c/src/main/java/io/jenkins/plugins/checks/status/BuildStatusChecksPublisher.java#L152-L153

BuildStatusChecksPublisher is the wrong place. You need PublishChecksStep.

Thanks, @KalleOlaviNiemitalo. To be clear, I shouldn't change the conclusion variable's default value but need to add a condition to check whether status is queued or in_progress, right?

I don't really know what is the best way to implement this. Anyway, please test that the pipeline snippet generator behaves in a sensible way after your changes.

Thanks, @KalleOlaviNiemitalo. To be clear, I shouldn't change the conclusion variable's default value but need to add a condition to check whether status is queued or in_progress, right?

yes, you should add a condition to check the status when setting the conclusion.

Ok sure. Thanks a lot, both of you.

@XiongKezhi, could you please let me know if you have any guide to follow on setting up the plugin for local testing? I referred to the Consumers Guide and Implementation Guide, but it's a bit unclear for me. It would be better if you can help me with this.

Sorry, those are not guides for users, they are for other plugin developers.
So when you are trying to test this plugin locally, first you need to set up a Jenkins instance for remote debugging. If you are using a Jenkins war file, startup it with the java options like (for more, see here):
-Xdebug -Xrunjdwp:transport=dt_socket,address=<IP_ADDRESS>:8000,server=y,suspend=n

Then you can set up your remote debugger to attach to it.
Finally, you need a Jenkins project to run the code of this plugin:

Set up the GitHub app credentials as in this guide
Use the GitHub app credentials you added to Jenkins in the last step in one of your projects. I have a multi-branch project set up like this (you should use the repository that your GitHub app is installed on):

And since you are going to modify the publishChecks step, so you should have a repository with a Jenkins pipeline calling publishChecks.

personally I just change the dependency in the github checks repo to a SNAPSHOT dependency, build this project with mvn install -P quick-build and then run the github checks project with mvn hpi:run using the debug option in my IDE
or just using unit tests for some changes where I don't actually need GitHub

Hi @XiongKezhi, I have pushed some changes, please check that and let me know your thoughts.
https://github.com/jenkinsci/checks-api-plugin/pull/82

fixed by https://github.com/jenkinsci/checks-api-plugin/pull/82


[5.0]Independ connection option for each connection name

Hi.
In current 5.0.17, the ability to use Redis Client option create one extra client and all client inherit the same option.
My edit makes each client have their own option and there is no extra client being created.
Before:

After:

Sample Configuration:
	'redis' => [

		'cluster' => false,
		'default' => [
			'host'     => '<IP_ADDRESS>',
			'port'     => 6379,
			'database' => 1
		],
		'phpiredis' => [
			'host'     => '<IP_ADDRESS>',
			'port'     => 6379,
			'database' => 1,
			'options' => [
				'connections' => [
					'tcp'  => 'Predis\Connection\PhpiredisStreamConnection',
				]
			],
		],

	],

Isn't this breaking?

@GrahamCampbell what do you mean if its breaking?

@blacklizard If we don't change the code of other application it'll fail.

Breaking - does the config file need to change.

@RomainLanz , @GrahamCampbell
Now i get it, you don't need to change anything in config file, it will still work. This is just for whom who want extra option when creating Predis Client.


i18n(de): update guides/data-fetching.mdx

Description (required)

update guides/data-fetching.mdx

Related issues & labels (optional)

#8210
#9240
Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

de/guides/data-fetching.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Rollup plugin support for SSR dev mode

Is your feature request related to a problem? Please describe.
When I run vite it uses rollup-plugin-svelte in my vite.config.js file to build my .svelte files. When I try to do vite build --ssr src/entry-server.ts --outDir dist/server followed by node server it does not seem to leverage the rollup-plugin-svelte in my vite.config.js because I get:
Error: Failed to parse source for import analysis because the content contains invalid JS syntax. You may need to install appropriate plugins to handle the .svelte?import file format.
    at formatError (/node_modules/vite/dist/node/chunks/dep-e56807a6.js:53347:46)
    at TransformContext.error (/node_modules/vite/dist/node/chunks/dep-e56807a6.js:53343:19)
    at TransformContext.transform (/node_modules/vite/dist/node/chunks/dep-e56807a6.js:46225:22)
    at async Object.transform (/node_modules/vite/dist/node/chunks/dep-e56807a6.js:53558:30)
    at async transformRequest (/node_modules/vite/dist/node/chunks/dep-e56807a6.js:60171:29)
    at async ssrLoadModule (/node_modules/vite/dist/node/chunks/dep-e56807a6.js:71337:10)
    at async Promise.all (index 0)
    at async ssrLoadModule (/node_modules/vite/dist/node/chunks/dep-e56807a6.js:71343:5)
    at async /server.js:61:11

It seems like vite.ssrLoadModule isn't executing rollup-plugin-svelte when it encounters a .svelte import
For now I just made it run always in prod mode: https://github.com/benmccann/vite-svelte/blob/6863508453b2e51011c3cc1b8fa77e4f74fd4bc6/server.js#L52
Describe the solution you'd like
Hopefully I could just reuse the existing rollup-plugin-svelte since it seems like Vite plugins mostly extend the Rollup plugin API
Another thing that would be nice is to pass some kind of flag to the vite.config.js because I need to know whether to generate the SSR or DOM code. I managed to make it work, but my solution is a little hacky:
https://github.com/benmccann/vite-svelte/blob/a9346d957ca2630c0a9c8d4cdf17d7b9dc189c07/vite.config.js#L6
Describe alternatives you've considered
This might just be an issue with me not quite being sure how to use the API
Additional context
There's been a lot of buzz about using Vite and Svelte together. We have a PR pending in rollup-plugin-svelte that enables using Svelte with HMR in Vite. I'd love to see if we can get SSR working as well

To get the plugin to work with SSR it will need Vite specific compat because the same plugin will be used for both client and ssr transforms. Vite passes an additional ssr flag to resolveId, load and transform hooks: https://github.com/vitejs/vite/blob/main/packages/vite/src/node/plugin.ts#L114-L134

To get the plugin to work with SSR it will need Vite specific compat because the same plugin will be used for both client and ssr transforms. Vite passes an additional ssr flag to resolveId, load and transform hooks: https://github.com/vitejs/vite/blob/main/packages/vite/src/node/plugin.ts#L114-L134

Thanks for the pointer Evan! I'll check it out

Thanks for the pointer Evan! I'll check it out


Bug in media query challenge

Describe your problem and - if possible - how to reproduce it
Following the steps and typing in the correct answer needed to pass the challenge still
gives me an error.
Add a Link to the page with the problem
https://learn.freecodecamp.org/responsive-web-design/responsive-web-design-principles/create-a-media-query
Tell us about your browser and operating system

Browser Name: Chrome
Browser Version:
Operating System: Windows 7

If possible, add a screenshot here

@RReddVar
I just tried this challenge and it worked for me.

Add a media query, so that the p tag has a font-size of 10px when the device's height is less than or equal to 800px.

I believe that your media query is for 800px or greater not 800px or less.
I used this for my media query and passed
@media (max-height: 800px)

@alhazen1 Okay, I tried it and that did work. Thank you for helping me with that! Think the developers need to fix the instructions to show that a media query for 800px and max is required, and not the opposite. :P

@RReddVar  You're very welcome.
When you have time come back to comments and select the button to close this issue.


Use Custom View inside AppBar

Is there any way, I can customize the AppBar Widget to look like that of Google PlayStore AppBar?
Something like this form would be appreciated
appbar: AppBar(
  child: CustomView()
)

I am guessing you just want a whole new widget to replace the whole AppBar and make it custom
import 'package:flutter/material.dart';

class CustomAppBar extends StatelessWidget {
  final String title;

  GradientAppBar(this.title);

  @override
  Widget build(BuildContext context) {
    return Container(
      height: barHeight,
      decoration: BoxDecoration(color: Colors.blue),
      child: Center(
        child: Text(
          title,
        ),
      ),
    );
  }
}

and then inside the Scaffold you can just
return Scaffold(
      body: Column(
        children: <Widget>[
          GradientAppBar("treva"),
        ],
      ),
    );

Let me know if your question meant for something else

This approach will be cumbersome. It would be like rebuilding the current Flutter AppBar Widget. I want to be able to pass a CustomView to the AppBar Widget through an extension like this
new Scaffold(
       appBar: AppBar(
              child: CustomView()
       )
);

You could try using AppBar(title: CustomView()), but if that's not working you probably want to just create your own special AppBar widget.  AppBar, as designed, is meant to make it easy to conform to the material specification rather than support specializations of it.  It's probably possible to do that as is, but it also wouldn't be terribly difficult to reuse a lot of the AppBar code to achieve your own effect (which may not quite be the Material spec but might be close!).
I suspect you'd get better support on this on StackOverflow, or one of the other channels mentioned at https://flutter.dev/community.  Please feel free to re-open this issue if there's something missing here though, or if you really are looking for a feature request for the app bar!

It works fine by passing CustomView() through the title. Thanks @dnfield


Update test snapshots

The current snapshot that is expected in the tests is not up to date, meaning the tests fail. This PR updates the snapshot to be in sync with the current version of the code, no functional changes were made.
This was making subsequent changes to the actual logic hard to read since updating the tests brings also these updates that were missing.

Hmm, something is wrong here. Maybe it's my local development that has issues. Checking

Yup, my local setup was the one that was wrong. Closing


Uvcvideo patching issue

| Required Info |
|---------------------------------|------------------------------------------- |
| Camera Model                       | SR300 |
| Operating System & Version |   Ubuntu 18.04  |
| Kernel Version (Linux Only)    |  5.0.0-32-generic  |
| Platform                                 | PC  |
| SDK Version                            |  2.29.0  |
| Language                            |  C++  |

I am following the Linux Ubuntu installation guide: https://github.com/IntelRealSense/librealsense/blob/master/doc/installation.md
When running the ./scripts/patch-realsense-ubuntu-lts.sh script, it gives me the following error

Failed to insert the patched module. Operation is aborted, the original module is restored
Verify that the current kernel version is aligned to the patched module version

I make use of kernel version 5.0.0-32-generic. Is there a patch i can run similar to the kernel 4.16 patch?

@markgrts, the script you're using is the correct one for LTS kernel 5.0.  There could be several reasons for rejecting the patched module, like. Secure Boot or Ubuntu packages inconsistency.
Please post the kernel log trace when the mentioned error occurs for review (dmesg)

dmesg gives me:

[  495.611605] usbcore: deregistering interface driver uvcvideo
[  495.931326] media: Linux media interface: v0.10
[  495.931613] Lockdown: modprobe: Loading of unsigned module is restricted; see man kernel_lockdown.7
[  495.960839] videodev: Linux video capture interface: v2.00

@markgrts , googling the error identifies this error as one of the symptomps of Secure Boot being enabled.
In order to load patched kernel modules this feature must be disabled.
There are references how to check the Secure Boot  state of the  on Linux - https://www.centos.org/forums/viewtopic.php?t=47224, but to disable it you'll have to enter BIOS settings #2815,#1641

Secure Boot was enabled. After disabling it, it worked. Thank you very much!


About ALMA's very bad benchmark results

Thank you for your amazing work.
I have a question because ALMA's results are very poor in the paper.
When I look at the results for WMT 20 en -> zh in Table 5, it shows a very low score (11.3) for ALMA, but this is very strange. ALMA gets a high score (39.3) on WMT 22 and even the WMT 20 test dataset used in the benchmark is used for training ALMA.
This is my personal guess, but it seems that the prompts template used for ALMA's inference were not used.

prompt example: "Translate this from Chinese to English:\nChinese: 我爱机器翻译。\nEnglish:"

Hi, I also find it strange when I reproduce them, but I directly follow their official infer commands and prompt template (same as your comment above), I also tried different `max_new_tokens' from default 20 to 150.
So I am also confused what's going with it, willing to communicate with you if any new findings. Thank you.


mobile: Fix clang-tidy issue in jni_impl.cc

This fixes this clang-tidy issue.
Clang-Tidy: 'event_tracker_global_ref' declared with a const-qualified typedef; results in the type being '_jobject *const' instead of 'const _jobject *

Risk Level: low
Testing: unit tests
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

/assign @abeyad

/retest


[Bug]: Cannot register API models (OpenAI, Azure OpenAI) from within the company network

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

RAGFlow workspace code commit ID
a88a184
RAGFlow image version
1b817a5b(v0.14.1~62) slim
Other environment information
No response
Actual behavior
I trying to set up a ragflow container within our company network according to the readme.
I have rewritten the nginx expose port and added proxy and no_proxy settings to the .env file to get to the outside network, since nginx was already running on the same server.
.env
~~~~~~~~~~~~
EXPOSE_NGINX_PORT=50080
EXPOSE_NGINX_SSL_PORT=50443

http_proxy=http://cache.xxx.xx.xx:1234
https_proxy=http://cache.xxx.xx.xx:1234
HTTP_PROXY=http://cache.xxx.xx.xx:1234
HTTPS_PROXY=http://cache.xxx.xx.xx:1234
no_proxy=localhost,<IP_ADDRESS>,...,ragflow,redis,es01,minio,mysql,infinity,kibana,kibana-user-init
NO_PROXY=localhost,<IP_ADDRESS>,...,ragflow,redis,es01,minio,mysql,infinity,kibana,kibana-user-init

docker-compose.yaml
~~~~~~~~~~~~~
services:
  ragflow:
    depends_on:
      mysql:
        condition: service_healthy
    image: ${RAGFLOW_IMAGE}
    container_name: ragflow-server
    ports:
      - ${SVR_HTTP_PORT}:9380
      - ${EXPOSE_NGINX_PORT}:80
      - ${EXPOSE_NGINX_SSL_PORT}:443
~~~~~~~~~~~~~

The container started up without any problems, and the registration of the self-hosted xinference model worked, but API models such as OpenAI and Azure OpenAI could not be registered.
Is there some other setting I need to do?
Expected behavior
Ability to register models from within the company network
Steps to reproduce
After modifying .env and docker-compose.yaml as above,
Start the container according to the readme

Additional information
Container logs in question
2024-12-05 13:30:25,545 INFO     18 <IP_ADDRESS> - - [05/Dec/2024 13:30:25] "GET /v1/user/info HTTP/1.1" 200 -
2024-12-05 13:30:25,550 INFO     18 <IP_ADDRESS> - - [05/Dec/2024 13:30:25] "GET /v1/llm/my_llms HTTP/1.1" 200 -
2024-12-05 13:30:25,556 INFO     18 <IP_ADDRESS> - - [05/Dec/2024 13:30:25] "GET /v1/user/tenant_info HTTP/1.1" 200 -
2024-12-05 13:30:25,588 INFO     18 <IP_ADDRESS> - - [05/Dec/2024 13:30:25] "GET /v1/llm/factories HTTP/1.1" 200 -
2024-12-05 13:30:31,939 INFO     18 Retrying request to /chat/completions in 0.871602 seconds
2024-12-05 13:30:32,818 INFO     18 Retrying request to /chat/completions in 1.506051 seconds
2024-12-05 13:30:34,334 INFO     18 <IP_ADDRESS> - - [05/Dec/2024 13:30:34] "POST /v1/llm/add_llm HTTP/1.1" 200 -

I'm not quite sure how to set proxy in docker container. But this should not be miss.

Accessing OpenAI API in banned regions with HTTPS_PROXY env is disallowed by the OpenAI.
RAGFlow team will not to try to support such use case.


(Stats) User feedback that we are not capturing all his data

Cool initiative, but something is broken it lists way too little data. https://sourcerer.io/rogeralsing Go: 881 loc. listed in the report only.
see attached screen shot

Indeed, very little useful data is going to be found for me because mine is all tied up in the commits I make to organization-owned projects. And supposedly, I gave sourcerer permission to access those repos, so it's not clear why it doesn't see it.

I am facing a somewhat similar issue - only one repo of mine is being processed. 103 repos, 5 OKs, one fail and the rest are skipped. Among the processed ones, there is only one repo for the stats. Here's my profile:
https://sourcerer.io/blitzkraft
I am not sure if I should open a new issue for this.

@BlitzKraft it looks like for many commits GitHub does not know your email addresses. They are attributed to "BK Bolisetty". Sourcerer relies on email addresses for commit attribution. If you don't want to make your email public, could you please send us any message from it to<EMAIL_ADDRESS>This will work as a confirmation for you, and we'll associate the email address with your account manually. After that, the attribution should work fine.

@rogeralsing : your profile seems pretty complete now. you may want to re-do sourcerer.io/start to update it. you can also use your pic from github now. please let us know if there still an issue.

@anatolykislov could you please look into the issue @mcg1969 is seeing?

Some questions, why is it listing 2015-2016 only?

I started doing Kotlin last year, and that is on the report.
I've been doing Go for multiple years, and that is mostly missing (it says 800loc, which is incorrect)

(from my org-repo https://github.com/AsynkronIT/protoactor-go)
Or is it only including direct personal repos?

I redid the start page and it picked up a lot more. But oddly, it skipped 2 repos because I had more than 300 commits in each. (The full text of the stated reason was truncated.) That seems quite odd and of course counterproductive if the purpose is to understand our coding behavior.

Ah, so the full message is +300 commits. Use App. FYI. It really doesn't make sense to me to exclude my two most active repos from display on my public profile.

guys we are looking into it. will keep you updated.

mcg1969, the commit limit is now raised to 10,000.

Also, as I don't use Python, "I like list comprehensions in python" this feels out of place tbh :)

It is getting better! Alas it still does not see my most "Important" org https://github.com/cvxr, and it confuses MATLAB for objective-c.

Ah, restarting picked up the CVXR repos.

@rogeralsing acknowledging the bug. looking into it.


[AutoPR eventhub/resource-manager] Event Hubs 2018-01-01: correction in the type of List API of VirtualNetwork

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3767

(message created by the CI based on PR content)

Codecov Report

Merging #3232 into restapi_auto_eventhub/resource-manager will increase coverage by <.01%.
The diff coverage is 62.83%.

@@                            Coverage Diff                             @@
##           restapi_auto_eventhub/resource-manager    #3232      +/-   ##
==========================================================================
+ Coverage                                    55.2%    55.2%   +<.01%
==========================================================================
  Files                                        9198     9198
  Lines                                      184876   184879       +3
==========================================================================
+ Hits                                       102067   102070       +3
  Misses                                      82809    82809

Impacted Files
Coverage Δ

...ure/mgmt/eventhub/operations/regions_operations.py
39.39% <0%> (ø)
:arrow_up:

...operations/disaster_recovery_configs_operations.py
15.65% <0%> (+0.07%)
:arrow_up:

...nthub/azure/mgmt/eventhub/operations/operations.py
41.93% <0%> (ø)
:arrow_up:

azure-mgmt-eventhub/azure/mgmt/eventhub/version.py
100% <100%> (ø)
:arrow_up:

.../mgmt/eventhub/operations/event_hubs_operations.py
97.12% <100%> (+0.01%)
:arrow_up:

.../eventhub/operations/consumer_groups_operations.py
73.91% <72.72%> (ø)
:arrow_up:

.../mgmt/eventhub/operations/namespaces_operations.py
83.8% <85%> (-0.17%)
:arrow_down:

Continue to review full report at Codecov.

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

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/3240


Increase the size of libapplauncher.so

Describe the bug
Hi.
Hello
I used to get the output of the Release Deb package, the size of libapplauncher.so was 1.6Mb, but now it is 2.3Mb.
I have not made any changes in the project and the compose version.
I just went from ubuntu 20 to 22.
Why is the size increased ?
Of course, I think this is related to jdk (jpackage), right ?
Affected platforms

Desktop

Versions

Kotlin version*: 1.9.10
Compose Multiplatform version*: 1.5.2
OS version(s)* (required for Desktop and iOS issues): Ubuntu 22.04
OS architecture (x86 or arm64): x86
JDK (for desktop issues): 18

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Import numpy before a call to sys.setdlopenflags

Must import numpy before a call to sys.setdlopenflags, or there may be a segfault in certain situations. Fixes #2034.
The situation occurs when users have compiled TensorFlow, but use binary distributions of SciPy. Alternatives to this patch would be documentation stating that users who compile TensorFlow from source should also recompile SciPy if they need it.

Can one of the admins verify this patch?

Nice investigation...how did you find it/any idea why this happens? Adding @keveman who added this part of code (although he slow to respond due to ICLR)

Jenkins, test this please.
Nice catch. Very obscure, much wow.

Yeah so this is just like some horrible edge case and part of the reason for the UNIQUE_SYMBOL mess in NumPy I guess.
Anyway when you include NumPy, you have to call import_array to populate some function pointers. Each C module needs to do this separately, and is supposed to have its own housekeeping for these function pointers. With the GLOBAL flag on though, there is overlap in some of these shared symbols (the UNIQUE_SYMBOL is the same in NumPy and Scipy:  PyArray_API). The result is the initialization code generated by any f2py module will clobber the housekeeping and dereference a bad pointer. Whoops! But if you do the import before the GLOBAL flag is set, then there is no shared symbol conflict, so the modules happily do their own housekeeping like they expect. O_o
@yaroslavvb I have just really, really wanted to use TensorFlow on the Texas supercomputing clusters, which use too old GLIBC to use the binary distribution, so I've been stuck compiling TF myself :stuck_out_tongue:

Hm... timeout, probably infrastructure failure. Will try again later.

@tensorflow-jenkins test this please


[SPARK-26820][SQL] Prints log messages when hints ignored in the analyzer

What changes were proposed in this pull request?
This pr added code to log ignored hints in RemoveAllHints.
How was this patch tested?
Pass Jenkins.

@gatorsmile Does this pr matches your intention in the Jira?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8763/
Test PASSed.

Test build #103324 has started for PR 24055 at commit ddc4f85.

Test build #103324 has finished for PR 24055 at commit ddc4f85.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103324/
Test PASSed.

Retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8971/
Test PASSed.

Test build #103560 has started for PR 24055 at commit ddc4f85.

Test build #103560 has finished for PR 24055 at commit ddc4f85.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103560/
Test PASSed.

ping @gatorsmile @maryannxue

Test build #103698 has started for PR 24055 at commit 82979d8.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9090/
Test PASSed.

Test build #103698 has finished for PR 24055 at commit 82979d8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Retest this please

Test build #103706 has started for PR 24055 at commit 82979d8.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9095/
Test PASSed.

Test build #103706 has finished for PR 24055 at commit 82979d8.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103706/
Test FAILed.

Retest this please

Test build #103714 has started for PR 24055 at commit 82979d8.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9102/
Test PASSed.

Test build #103717 has started for PR 24055 at commit 82979d8.

Test build #103717 has finished for PR 24055 at commit 82979d8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #103714 has finished for PR 24055 at commit 82979d8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103717/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/103714/
Test PASSed.

Build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15709/
Test PASSed.

cc @maryannxue @maropu @cloud-fan


CI/CD: add GH action bots

### Tasks
- [x] https://github.com/robvanderleek/create-issue-branch
- [x] https://github.com/philschatz/project-bot

project-bot seems no longer maintain, give up


Interactive window doesn't auto scroll to the bottom

Testing #127378
Run a cell like so:
import time
while (True):
    time.sleep(.1)
    print('*')

It should be scrolling the whole time.

The interactive window doesn't scroll automatically at all for me. Even when I execute cells one after another, the interactive window doesn't scroll.

Fixed in latest build of Jupyter extension for VS Code Insiders and duplicate of https://github.com/microsoft/vscode-jupyter/issues/6603.
@repied, this issue tracks the new native interactive window UI, which I'm assuming you explicitly opted into using jupyter.enableNativeInteractiveWindow? If so you should get the fix in the next stable VS Code (~3 weeks from now), or you can install VS Code Insiders to get the fix tonight.
If the interactive window is not scrolling for you and you have not configured the setting I mentioned above, that is a bug, please open an issue at https://github.com/microsoft/vscode-jupyter. Thank you!


rename actions "SWS: Goto/select {next,previous} marker/region" to "SWS: Go to/time-select {next,previous} marker/region"

rename actions: Goto -> Go to, select region -> time-select region  (to not confuse with 'select region for rendering' which was added to REAPER later)

rename "SWS: Goto/select {next,previous} marker/region" to "SWS: Go to/time-select {next,previous} marker/region"
report: https://forum.cockos.com/showpost.php?p=2479136&postcount=2956

rename "SWS: Select {next,previous} region" to "SWS: Time-select {next,previous} region"

rename "SWS/S&M: Go to/select region %d (obeys smooth seek)" to "SWS/S&M: Go to/time-select region %d (obeys smooth seek)"

While at it, maybe these should be renamed to "Go to/**time-**select...",
because selecting region has become a different meaning meanwhile:
https://forum.cockos.com/showthread.php?t=235259

Makes sense to me. There are also more "select region" actions at SnM.cpp:747 and MarkerList.cpp:690-691.

I was trying to fix the indention but I don't get this tab indention thing. :/
Here it looks ok, but on GitHub it's screwed up:
https://github.com/reaper-oss/sws/pull/1555/files#diff-1b218784a31e9326a04c19f5f8ce29e4a16e658a0c9027206e0de57bab464770R690
How about we using spaces rather then tabs? :P

In MarkerList.cpp's g_commandTable? Yeah, it looks horrible. +1 for spaces!

Actually I was wondering if we could maybe do away with the "We prefer: Hard tabs, equal to 4 spaces" coding guideline in the Wiki (or even turn it around for preferring spaces).
What do you think?

Didn't know about that hard tabs guideline. cfillion.cpp is violating it...  :scream:  I've abided by the "match the style of surrounding code" unspoken rule to far. (For my own code I'm a fan of 2-space indentation and spaces for alignment.)
The coding style is widely inconsistent throughout. There are tools (eg. clang-tidy) to enforce a specific style (there are a few widely-used ones), but considering how messy the code is I'm not sure it's worth rewriting all files at this point (and making git blame useless)...

So - would you be ok with removing that rule?
I think the issue is that if we're using hard tab = 4 spaces but GitHub's default tab size = 8 spaces (afaik), so things can look messy (when e.g. used for alignment).

Yes. It's already not respected in many files (or parts of files!). A better guideline would be to match nearby code for indentation and general style. For alignment, spaces are objectively better as they aren't subject to the editor's settings.

Just edited the Coding guidelines wikipage accordingly. Feel free to add/change. :)


Remove the deprecated RunTransaction function

the file_format_check bot is failing with
  - Requires reformatting: "firestore/src/common/transaction_function.h"
------ BEGIN FORMATTING DIFF OF firestore/src/common/transaction_function.h
---
+++
@@ -35,4 +35,3 @@
 }  // namespace firebase

 #endif  // FIREBASE_FIRESTORE_SRC_COMMON_TRANSACTION_FUNCTION_H_
-
------ END FORMATTING DIFF OF firestore/src/common/transaction_function.h
  > Done. 1 file(s) need formatting.


A.01.IR13.1.crs

This issue has been extracted from the issue list on:https://ies-svn.jrc.ec.europa.eu/issues/2685
Comment
Ambiguity: the hyperlink of 'RS_Identifierá'does not point to the right source
In addition 'It is suggested ...' or "the test can grab ...' is not an unambiguous description of a test.
Testability: Also, it is unclear how to 'validate the coordinate reference system against the advertised system'?
The data specifications provide http URIs for all default CRSs. Is this relevant for this test (note that http://www.opengis.net/def/crs/EPSG/0/4326 is not one of them)?

The list of allowed CRSs depends on the theme.
It seems unreasonable to download the full dataset every time the metadata is validated.
It is also practically valueless since by the time all the dataset has been validated, it might have changed and become invalid.
However, it is feasible and in general useful to "probe" and test only the first bytes/kilobytes/megabytes of a dataset. For xml this is enough to determine if the character encoding, the spatial object type, the encoding, the transfer rate and so on, correspond to what is declared and/or required.

Note that the requirement for the CRS metadata element is only that it has to be present (spec target: metadata record). There is no requirement on the value it should have. The requirements on the CRS of the data set are different (spec target: data set).


cmd/compile: missed trivial bound check elimination

Go version
go version go1.22.2 darwin/arm64
Output of go env in your module/workspace:
GO111MODULE=''
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/filippo/Library/Caches/go-build'
GOENV='/Users/filippo/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/filippo/pkg/mod'
GONOPROXY='github.com/FiloSottile/*,filippo.io/*'
GONOSUMDB=''
GOOS='darwin'
GOPATH='/Users/filippo'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org'
<EMAIL_ADDRESS>GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
<EMAIL_ADDRESS>GOVCS=''
GOVERSION='go1.22.2'
GCCGO='gccgo'
AR='ar'
CC='clang'
CXX='clang++'
CGO_ENABLED='1'
GOMOD='/Users/filippo/src/filippo.io/mlkem768/go.mod'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/_j/hq4ytn1n4b94fhrpvvb9tktr0000gn/T/go-build3556564162=/tmp/go-build -gno-record-gcc-switches -fno-common'

What did you do?
I have this function in a fairly hot loop.
const n = 256
type fieldElement uint16
type nttElement [n]fieldElement

func nttMul(f, g nttElement) nttElement {
	var h nttElement
	for i := 0; i < 128; i++ {
		a0, a1 := f[2*i], f[2*i+1]
		b0, b1 := g[2*i], g[2*i+1]
		h[2*i] = fieldAdd(fieldMul(a0, b0), fieldMul(fieldMul(a1, b1), gammas[i]))
		h[2*i+1] = fieldAdd(fieldMul(a0, b1), fieldMul(a1, b0))
	}
	return h
}

What did you see happen?
Both f[2*i] and f[2*i+1] get a isInBounds(). g[2*i] and g[2*i+1] don't.
What did you expect to see?
One or zero bounds checks.
One if the compiler realized that if f[2*i+1] is in bounds with i positive and small (0 to 127), then f[2*i] is clearly in bounds.
Zero if the compiler figured out that i is 0 to 127, so 2*i+1 is 0 to 255, and the size of f is a constant 256.

It looks there is a workaround now:
func nttMul(f, g nttElement) nttElement {
	var h nttElement
	for i := 0; i < len(f); i += 2 {
		a0, a1 := f[i],
		f[i+1]
		b0, b1 := g[i],
		g[i+1]
		_, _, _, _ = a0, a1, b0, b1
	}
	return h
}

Not benchmark it yet.

That does indeed remove the bounds checks on f, but we get one on gammas[i/2], which also feels avoidable.
Slices should in theory add overhead compared to const size arrays and remove information from the type system

Yes, there are still several cases BCE fails to handle.
Now, a workaround is to double the length of gammas and only uses its elements at 2N indexes.

It is possible. BCE will not always get positive effects.
Another wan to remove all bound checks is declare fieldElement as [2]uint16. I'm not sure about the final effect.

The test shows that double length of gammas is the most performant: https://gotipplay.golang.org/p/VKJDqUPuhKV (maybe the implementation of fieldMul and fieldAdd will affect the results).

Simple reproducer:
func f(a [256]byte) {
	for i := 0; i < 128; i++ {
		_ = a[2*i]
	}
}

We should be able to get rid of the bounds check. Currently I think it fails because the prove pass can't tell that the math 2*i doesn't overflow and become negative. It does seem to grok that 2*i < 256.
Should be fixable. That code is delicate though, it is not immediately obvious to me where that fix would go.


Repo didn't roll back

When running the Azure DevOps generator, when an error occurs, the repo fails to roll back. So is because of the below issue:
    TypeError: Cannot read property 'rmdir' of undefined
    at RepoGenerator.rollback (C:\Users\***\Documents\Projects\alm-labs\powerapps-project-template\generators\azuredevops\generator\RepoGenerator.js:32:29)
    at process._tickCallback (internal/process/next_tick.js:178:7)

Fixed by #58


Conflict between Julia ArgParse and RainFarm versions

Installation via the Conda method on a RHEL 6.6 system failed with the following error:
ERROR: LoadError: Unsatisfiable requirements detected for package ArgParse [c7e460c6]:
ArgParse [c7e460c6] log:
├─possible versions are: [0.6.1-0.6.2, 1.0.0-1.0.1] or uninstalled
├─restricted to versions 1.0.1 by an explicit requirement, leaving only versions 1.0.1
└─restricted by compatibility requirements with RainFARM [e9a4e08f] to versions: 0.6.1-0.6.2 — no versions left
  └─RainFARM [e9a4e08f] log:
  ├─possible versions are: 1.0.1-1.0.2 or uninstalled
  └─restricted to versions * by an explicit requirement, leaving only versions 1.0.1-1.0.2

I can upload the entire console output if more information is needed

Solved by #1529.


Homepage management

Hello, love Linkding! I'm just starting to migrate my Pinboard stuff onto here :)
Apart from singular bookmarks, I was wondering, does Linkding also allow collecting homepages of blogs / writers I love?
Lets take an example - say http://paulgraham.com. I'd like to store this as a favourite blog of mine, but don't want any RSS-like capability to notify when a new post arrives. Is this possible currently in Linkding?

I'm not sure what you mean. Do you mean that the app should check for new blog articles and automatically add them as bookmarks? Or group articles of the same blog together somehow?

My comment didn't clarify that properly, but I meant the latter - grouping articles of the same blog together. Looking at the groups with higher n(item) would indicate them as one's favourite blogs.

There is an existing issue to group bookmarks by domain, which is pretty much what you want: https://github.com/sissbruecker/linkding/issues/415
There is also another issue to add more capabilities for manual grouping of bookmarks: https://github.com/sissbruecker/linkding/issues/285
I think I'll close this one in favor of these.


Keywords are missing from completion list at the beginning of razor code block

VS Version: 17.9.0 Preview 1.1
Reproduce:
Start typing inside Razor code block:
https://github.com/dotnet/razor/assets/9047283/49ac39bf-94e5-453c-8c0d-154ea6f6b5dc

Sorry, it was a mistake, I've used the wrong type of code block.


Added Bratajit-03 to README.md

I have made my contribution and have added my name in the README.md

@eddiejaoude sir, kindly review my PR and merge it.


iPad issue: Ticket Snap does not work.

Snap to Grid does not work.
////////CONTRIBOARD TESTING, 24.7.2015////////
Contriboard (SUT) versions:
Client version v1.2.100
API version v0.4.38
IMG version v0.1.10
Testing environment:
Heli (1 kpl iPad iOS 8.3 (12F69) Flowdock -> Apple web-view)
////////CONTRIBOARD TESTING, 24.7.2015////////

I've noticed some funky snap behaviour on both Chrome and Safari

Cannot reproduce on said iPad? Might be related to some performance issues...

Closed now
Client versio	v1.2.118
API versio	v0.4.42
IMG versio	v0.1.10


Add ARC support for Linux and Windows

Would be great if you can add ARC support for Linux and Windows servers.

Hi @alpbabayigit
could you please elaborate on what support you're looking for in detail? In general, Azure Defender offers support for Arc-enabled machines in multi-cloud and hybrid scenarios for both, Windows and Linux. For example. we just GA'ed support for the integrated Qualys VA solution on ARC-enabled machines. You can find information about how to connect your non-Azure machines to Azure Security Center/Azure Defender in our documentation.

Hi @alpbabayigit
could you please elaborate on what support you're looking for in detail? In general, Azure Defender offers support for Arc-enabled machines in multi-cloud and hybrid scenarios for both, Windows and Linux. For example. we just GA'ed support for the integrated Qualys VA solution on ARC-enabled machines. You can find information about how to connect your non-Azure machines to Azure Security Center/Azure Defender in our documentation.

Closing as solved


browserify: getting super tired of updating test/fixtures/secureBundling/lavamoat/node/policy.json

This file changes almost every time package-lock.json changes. It fails the dirty check often.  As @naugtur put it, it's a "commit detector."

[ ] We need to remove it from VCS.
[ ] If this policy file is used as a snapshot: It should not be used as a snapshot. We can make assertions about some subset of the policy, if necessary.
[ ] If this policy file is used only in the context of execution, then great--we don't need to do anything else except remove it from VCS.

I'll look at how exactly it's being consumed, then do 1 and (2 or 3).

Ref: #987


Add JSON tags for Report struct

Hello,
Thank you for this nice package. I use it for a Discord bot. While I was trying to get the information about the last reports I noticed that I couldn't get the unixtime as described on the warcraftlogs API documentation. I tried to convert it and was running into multiple errors.
After I looked into the API call to see if it is in the JSON I looked into your package.
After adding the JSON tags for the Report struct the Object contains the right values and I can convert it to UTC.
This is at the same time an issue report and and a solving PR.
Regards Peuserik
Add JSON tags for Report struct

to fix empty and/or nil object

Sorry for a year-long delay, this has totally slipped my mind. So this got noticed only once i was going through some old things.


Add gallery of example sample maps

Add a link to the wonderful work others have created so we can see all the power...
Currently the only examples are http://geojson.io/#id=github:benbalter/dc-wifi-social/blob/master/bars.geojson&map=14/38.9140/-77.0302 and one more, found deep in the right panel, and just as examples for remote calls, not general examples, as seen on the "get inspired" section of http://umap.openstreetmap.fr/ .

Oh cool, you found  http://umap.openstreetmap.fr/  already, which is what you're looking for.


Clicking on the button to buy advances too quickly breaks the exp calculation

In this case was buying 5 advances for perception by clicking with the mouse on a decentish pace and after that was wondering why there was too much exp left. The log snippet above revealed that one of the purchases didn't use any exp at all.
System version 3.5.1

I doubt the log is the issue here, and would break regardless


Post can be closed before end of cashout window

Impact: post can be closed early (by Golos leaders or Golos.io)
STR:

create post
execute closemssg action before end of cashout window

Actual result:
Post closed
Expected result:
Error that cashout window not closed yet

Fixed by PR https://github.com/GolosChain/golos.contracts/pull/783.
Currently there are no ability to close message before end of cashout window.


Simulating 16 colors on 8 color terminals

Would it be possible to simulate 16 colors on 8-color terminal by using 8 colors + bold flag on terminals that support bold attribute and otherwise keep the existing behavior? Essentially Color[8-15] = Color[0-7] + bold, i.e. ColorYellow=ColorOlive+Bold`.
BTW, It may be possible to to achieve the same by Color[0-7] = Color[8-15] + Dim but I haven
We work on an application that uses rivo/tview package that written on top of gdamore/tcell and our initial version looked different on linux console vs. ssh terminal even though terminal was set to linux for both. We figured out the workaround above to make application look consistent but it looks more like a hack and it is not obvious for developers who will inherit this code.  It would be much better if tcell package would handle it through the API.

To be honest, I'm not sure this is desirable.  We tell the application how many colors the terminal can support;  it isn't clear that conflating this with bold is universally desirable.   If "bold" is something other than just "bright" (for example by making characters thicker) then this is not the same thing.  On your terminal it appears that bold is achieved by simply increasing the intensity.  (That's typical of how 16-color is supported on other platforms.)
Having said that, you can change the terminal description to express 16-colors with the escape sequences for those colors setting the physical bold attribute.  This can be done without actually modifying code.
If we know which terminals suffer this defect (using bold for intensity and not supporting the real 16-color palette) we can fix this in the tcell distribution with a special hacked up terminal entry for it.

I agree that applying this rule universally is probably not a good idea, there are too many weird things in terminal world. But we know for sure that at least linux and xterm behave this way. For linux console it is in kernel spec (I've seen it but can't find the the reference anymore) and xterm behaves the same way. So, would it be possible to do it at least for these two?

I still think that the idea of an override to "fix" a broken terminfo entry (which seems to be the case for Linux -- I wouldn't do this for xterm because there are waaaay too many xterm clones out there, and "real" xterm supports the full color palette, including 24-bit color) would not be a bad approach.

I'm going to close this for now.

:+1:


Error 500

[13-12-2015 23:09] System.Exception: Downloading stash for Talisman, details logged to DebugInfo.log, please open a ticket at https://github.com/Stickymaddness/Procurement/issues
a következő helyen: POEApi.Model.POEModel.GetStash(String league, String accountName)
a következő helyen: Procurement.ViewModel.LoginWindowViewModel.LoadStashItems(Character character)
a következő helyen: Procurement.ViewModel.LoginWindowViewModel.d__6.MoveNext()
a következő helyen: System.Collections.Generic.List1..ctor(IEnumerable1 collection)
a következő helyen: System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
a következő helyen: Procurement.ViewModel.LoginWindowViewModel.<>c__DisplayClass2.b__1()
a következő helyen: System.Threading.Tasks.Task.Execute()

GGG have changed the API without bothering to let us know, as usual.
I'll release a fix as soon as I can!

Procurement 1.9.7 has been released which fixes this issue!
https://github.com/Stickymaddness/Procurement/releases/tag/1.9.7

500 Internal Error since 09 Dec 2018.. is tehre any thing i need to fix?

Hi Charles,
i was going to open a new ticket but the moment i wrote "500 Internal
Error" the previous post popped up... I'll make a new ticket if you like.
The Debug file is attached here for your reference. it is still giving the
same Internal error as of yesterday.
Cheers

Tayyab Farooq
------ A Humble Servant of Allah

<https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail>
Virus-free.
www.avast.com
<https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail>
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>

On Mon, Dec 10, 2018 at 11:26 PM Charles Macanka<EMAIL_ADDRESS>wrote:

> Please create a new issue with any updates, instead of commenting on a
> years-closed issue. 😺
>
> A 500 error suggests there is an issue on GGG's server. The servers were
> down earlier today, which certainly could have been the cause. When the
> game is back up, please try again. Are you still having the issue? Are you
> using the latest version of Procurement? What is the context of the issue?
> Also attach your DebugLog.info file if you're still having problems.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/Stickymaddness/Procurement/issues/284#issuecomment-445940659>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ArpRY74X1sZyXk-ZeBE8-FLP7q31fBXMks5u3rV-gaJpZM4G0Zwo>
> .
>


[WFLY-19638] Upgrade com.google.guava:failureaccess from 1.0.1 to 1.0.2

Issue: https://issues.redhat.com/browse/WFLY-19638
previously https://github.com/wildfly/wildfly/pull/17363 was closed, but this is behinds downstream now.

@ehsavoie @liweinan @jimma @jasondlee Please review

This upgrade removes the guava support of JBeret:

https://github.com/wildfly/wildfly/pull/18121

Thanks @soul2zimate


TYP: fix a Mypy issue and pin Mypy 0.981

Mypy is extremely fragile, and random CI issues are going to crop up unless it is pinned.

Okay that is all green, and fixed the issue that showed up on gh-148 (see this CI log):
 mesonpy/__init__.py:769: error: Unused "type: ignore" comment

This is a trivial fix and is helpful for other PRs, given that it's a bit tricky to get CI green on them. I'll go ahead and merge this. Please feel free to post-review @FFY00


Dependency submission action should handle non-executable gradlew script

Gradle 8.6
JDK 17
I'm using the dependency-submission action and just used the simplest example.
name: Dependency Submission

on: [ push ]

permissions:
  contents: write

jobs:
  dependency-submission:
    runs-on: ubuntu-latest
    steps:
    - name: Checkout sources
      uses: actions/checkout@v4
    - name: Generate and submit dependency graph
      uses: gradle/actions/dependency-submission@v3

However, its action is failing and can't execute gradlew which I never changed and IDE generated on project initialization. Here I'm providing links to the full raw pipeline error and my gradlew file.
2024-02-08T09:24:03.7482952Z ##[error]Error: Gradle script '/home/runner/work/freeDictionaryApp/freeDictionaryApp/gradlew' is not executable.
2024-02-08T09:24:03.7536635Z Error: Gradle script '/home/runner/work/freeDictionaryApp/freeDictionaryApp/gradlew' is not executable.
2024-02-08T09:24:03.7538765Z     at verifyIsExecutableScript (/home/runner/work/_actions/gradle/actions/v3.0.0/dist/setup-gradle/main/index.js:140758:15)
2024-02-08T09:24:03.7541933Z     at validateGradleWrapper (/home/runner/work/_actions/gradle/actions/v3.0.0/dist/setup-gradle/main/index.js:140744:5)
2024-02-08T09:24:03.7546118Z     at Object.gradleWrapperScript (/home/runner/work/_actions/gradle/actions/v3.0.0/dist/setup-gradle/main/index.js:140737:5)
2024-02-08T09:24:03.7548440Z     at Object.<anonymous> (/home/runner/work/_actions/gradle/actions/v3.0.0/dist/setup-gradle/main/index.js:140677:95)
2024-02-08T09:24:03.7549679Z     at Generator.next (<anonymous>)
2024-02-08T09:24:03.7551015Z     at /home/runner/work/_actions/gradle/actions/v3.0.0/dist/setup-gradle/main/index.js:140667:71
2024-02-08T09:24:03.7552046Z     at new Promise (<anonymous>)
2024-02-08T09:24:03.7553530Z     at __webpack_modules__.23584.__awaiter (/home/runner/work/_actions/gradle/actions/v3.0.0/dist/setup-gradle/main/index.js:140663:12)
2024-02-08T09:24:03.7555832Z     at Object.executeGradleBuild (/home/runner/work/_actions/gradle/actions/v3.0.0/dist/setup-gradle/main/index.js:140676:12)
2024-02-08T09:24:03.7556935Z     at /home/runner/work/_actions/gradle/actions/v3.0.0/dist/setup-gradle/main/index.js:141589:33

When I clone your repository, the gradlew script isn't executable, as mentioned in the error. This means that either:

The IDE you used to generate the project didn't make this file executable, OR
The executable bit wasn't retained when you committed the project to GitHub

I think the error message is pretty clear, but the I think the action could be improved to handle a non-executable gradlew script, even if it makes no sense to have a project like that.

When I clone your repository, the gradlew script isn't executable, as mentioned in the error. This means that either:
* The IDE you used to generate the project didn't make this file executable, OR

* The executable bit wasn't retained when you committed the project to GitHub

I think the error message is pretty clear, but the I think the action could be improved to handle a non-executable gradlew script, even if it makes no sense to have a project like that.

Thanks.

When I clone your repository, the gradlew script isn't executable, as mentioned in the error. This means that either:
* The IDE you used to generate the project didn't make this file executable, OR

* The executable bit wasn't retained when you committed the project to GitHub

I think the error message is pretty clear, but the I think the action could be improved to handle a non-executable gradlew script, even if it makes no sense to have a project like that.

I don't know about that, however, somehow changing my workflow file to this fixed the problem. Modern Android apps need JDK 17 but I don't know why we need to grant permission for the gradlew file explicitly.
name: Dependency Submission

on: [ push ]

permissions:
  contents: write

jobs:
  dependency-submission:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout sources
        uses: actions/checkout@v4
      - name: Set up JDK 17
        uses: actions/setup-java@v4
        with:
          java-version: '17'
          distribution: 'temurin'
          cache: gradle
      - name: Grant execute permission for gradlew
        run: chmod +x gradlew
      - name: Generate and submit dependency graph
        uses: gradle/actions/dependency-submission@v3

If you build from the command-line, you will execute ./gradlew build. This requires the executable bit to be set on the script.
If you are only building from the IDE, then you may never notice this being broken.
I've pushed a fix for this issue, so with the next release you'll just see a warning in your workflow run, not an error.


Execute TM1py script via TI

I'm calling a TM1py script via TI as suggested here on Cubewise site.
If I execute it via command line, logged on to the Windows Server with my user credentials, it works.
If I execute it via TI nothing happens. TM1 service account has rights on Python folder.
In the TM1py script I'm currently testing with my user credentials (normally I do not provide user/password since we're using security mode 5).
creds = {
'address':'xxxxx.xxx.xxx.xxx',
'port':'12346',
'user':'domain\user',
'password':'MyPassword',
'namespace':'XXX',
'gateway':'https://xxxxx.xxx.xxx.xxx/ibmcognos/bi/v1/disp',
'ssl':'True'
}
Am I missing something?

There are few questions here:

What is the command you have in TI process?
What do you mean when you say nothing happens? What's the expectation?
Is python added to the environment PATH?
Do you have python installed in C drive?

Try these:

Logoutput the command
Copy the command written in the tm1server.log and open cmd using the service account credentials in your windows server and run the command.

Also note that security mode 5 doesn't mean SSO is turned ON.

Hi @rkvinoth,

What is the command you have in TI process?

What do you mean when you say nothing happens? What's the expectation?
That start time of a chore that is maintained in a helper cube is written to chore

Is python added to the environment PATH?
Yes

Do you have python installed in C drive?
No, on D-drive

Logoutput the command
Done

Copy the command written in the tm1server.log and open cmd using the service account credentials in your windows server and run the command.
Then I get this: #TE_Test2 is the name of the chore

It's complaining that #TE_Test2 can not be found. Does this mean that I'll have to add this service account to TM1 as admin?

@hermie64
When you call a python script from the TI process, the Python process will log into your TM1 application using the Service account's credentials (this is the service account that is running your TM1 service). So, if you have some ADMIN activity provide ADMIN access to the Service account.

After adding ADMIN access to the service account it works.
Thanks guys!


stdlib: clean up some whitespace issues

Clean up incorrect whitespace.  NFC.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test and merge


Add dash to category title or page title

I am having trouble adding a dash to category and page titles. Is there an escape character that I need to use or is this a small bug?

Seems like this could be related to #65 too.


Trino won't start

Trying on minikube, both trino-coordinator and trino-worker pods crash at startup, with log error :
cat: etc/node.properties.template: No such file or directory
cat: etc/config.properties.template: No such file or directory
ERROR: JVM config file is missing: /trino-server-360/etc/jvm.config

@ghpu, this is likely due to a version mismatch -- the Docker files currently have version 363, whereas the K8s config uses version 360. Should be an easy fix -- just grep for 360 and replace with 363. Will try to get to this myself today.

@ghpu, managed to fix this, trino has now been updated to 368 and I have tested the example in the README -- works now!


Killed clustering

Hello,
So I tried to run cluster.split. It took a lot of time for it to calculate the distances. Also, the dist files were big (+250 GB big). When it was doing the clustering, I got this message:
cluster.split(fasta=final.fasta, count=final.count_table, taxonomy=final.taxonomy, taxlevel=4, cutoff=0.03)
/******************************************/
/******************************************/
Selecting sequences for group Sphingomonadales (2 of 60)
Number of unique sequences: 36453

...

Output File Names:
final.10.dist

Clustering final.3.disttemp

tp	tn	fp	fn	sensitivity	specificity	ppv	npv	fdr	accuracy	mcc	f1score
6.60748e+07	1.47227e+08	8.81038e+06	5.46799e+06	0.92357	0.943537	0.882348	0.96419	0.882348	0.93726	0.856761	0.902489

Killed

I tried it the "trad" way using dist.seqs and then cluster. Again, the dist file was very big and I got the following message after running cluster :
mothur > dist.seqs(fasta=final.fasta, cutoff=0.03)

...

It took 8414 secs to find distances for 164436 sequences.<PHONE_NUMBER> distances below cutoff 0.03.

Output File Names:
final.dist

mothur > cluster(column=final.dist, count=final.count_table, cutoff = 0.03)

Using 12 processors.

Clustering final.dist
iter	time	label	num_otus	cutoff	tp	tn	fp	fn	sensitivity	specificity	ppv	npv	fdr	accuracy	mcc	f1score

0.03
Killed

This is the summary.seqs before clustering:

		Start	End	NBases	Ambigs	Polymer	NumSeqs
Minimum:	1	472	227	0	3	1
2.5%-tile:	1	472	252	0	4	9546
25%-tile:	1	472	253	0	5	95457
Median: 	1	472	253	0	5	190914
75%-tile:	1	472	253	0	5	286370
97.5%-tile:	1	472	253	0	5	372281
Maximum:	1	472	259	0	8	381826
Mean:	1	472	252	0	4
# of unique seqs:	164436
total # of seqs:	381826

I have never had this problem with the training files from the workshop and with real data from a labmate (which had many much more sequences). I don't know if there are too many unique sequences, or if there is a problem with my sequences.
My computer has 24 GB of RAM (I know, I need to get another 16GB stick) and was using 12 cores at 2.0 GHz speed.
In advance, thank you!

Hi - do you have 2x150 or 2x250 nt reads? If it's 2x150 then I suspect you have a high error rate that is inflating the number of unique sequences and causing problems.
You might try cluster.split with taxlevel=5 or 6.
You might also try cluster.split in two steps where you use 12 processors on the splitting and distance calculation step and 1 processor on the clustering step.
These suggestions should help with the memory limitation issues.
Pat

Hello,
I have 2x250.
I ran cluster.splits with a taxlevel=6, but I had the same problem. The group that is causing so many problems is Rhizobials, almost all sequences are from that group (67,000 sequences just for that group? which doesn't make sense for my samples anyway). I think I just got a bad sequencing run.
Is there a way to run summary.qual in all the qual files made with fastq.info?


Update ReadRetrieveReadChatSErvice to use AzureOpenAIChatCompletionService

Consider removing since updates from #137 already provide a chat completion service.
Consider turning the ReadRetrieveReadChatService into an Azure Function.

Fixed by #163


Update for 1.8.7/1.8.8

Thank you! I have added you as a contributor so you can just merge your commits directly.

Np! I also released your plugin on Spigot :) Check out the Spigot page if you like!

I actually did that before, but md5 thought it was "offensive" and had it removed.  Still getting downloads and I still use this plugin on my RPG server to keep the animal populations up.


Caching problems

Every two, three days I get an error that there was a problem with Caching and I have to sign out and reload the app.

Next time you get an error message, can you open the console using f12 and paste the output here?

Of course when it happens again

The problem is back

I'll try to add source-map-support so we can identify the exact line of code where it is happening. It will take some days though @eBourgess . :)


Apply styling to password reset modal

A centered version of the modal that looks more uniform and centers vertically as the screen height changes.

Automated message from Dropbox CLA bot
@brockwhittaker, it looks like you've already signed the Dropbox CLA. Thanks!

Thanks for working on this @brockwhittaker!  A couple notes:
(1) There's a lint failure
(2) I would do this as 2 commits -- first one that adds password_reset_container (and reindents the HTML) and then a second that does the actual changes; that would make it a lot easier to see what you actually changed inside the code.

The new commits now separate the addition of the container divs from the content changes in the password reset modal. It also passes linter tests.

Looks like you still have linter failures from trailing whitespace; you should fix your editor to not do those.
(I'm increasingly thinking we should add an atom config to the repo once we get this right).

Merged (after fixing the trailing whitespace error).  Thanks @brockwhittaker!


namespaces: enforce a character set for namespaces

Signed-off-by: Stephen J Day<EMAIL_ADDRESS>Closes #1046

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1059   +/-   ##
=======================================
  Coverage   59.28%   59.28%
=======================================
  Files           5        5
  Lines         781      781
=======================================
  Hits          463      463
  Misses        204      204
  Partials      114      114

Continue to review full report at Codecov.

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

LGTM


Copy text content for each field

Same button as train

All record fields will have this button (it's not optional)

Ref : #2877


Date error in b4 on Sonoma v14.0

**** S.U.P.E.R.M.A.N. 4.0.0-beta4 - SUPER STARTUP WORKFLOW **** Failed conversion of ``12 Oct 14:41:00'' using format ``%b %d %H:%M:%S'' date: illegal time format usage: date [-jnRu] [-I[date|hours|minutes|seconds]] [-f input_fmt]             [-r filename|seconds] [-v[+|-]val[y|m|w|d|H|M|S]]             [[[[mm]dd]HH]MM[[cc]yy][.SS] | new_date] [+output_fmt] Status: Mac computer with Intel running: macOS Sonoma 14.0-23A344

The output of last -1 reboot reboot time                                Thu 12 Oct 14:41
UK format seems to break the code

Error is at line 3416
mac_last_startup=$(date -j -f "%b %d %H:%M:%S" "${last_reboot_response}:00" +"%Y-%m-%d:%H:%M:%S") # Expected output: 2023-08-25:00:16:00

ugh... what version of macOS are you on? I thought I had this sorted.
Different versions of macOS return different values.

Ok... I'm gonna try a different approach... can you try this line please:
last -1 reboot | sed -e 's/  */ /g' | awk '{print $4" "$5" "$6;}'
it should return a format like this: Sep 28 21:21

I get 12 Oct 14:41 on UK format

ugh.... that makes this so much more complex

I know, I tried fixing it myself quickly and failed.

hrm... I think I'm gonna detect if the first character is a number or letter and then assume there are "only" two formats.

@ofirgalcon Can you try these two commands please?
last_reboot_response=$(last -1 reboot | sed -e 's/  */ /g' | awk '{print $4" "$5" "$6;}')
date -j -f "%d %b %H:%M:%S" "${last_reboot_response}:00" +"%Y-%m-%d:%H:%M:%S"
Output should look like this: 2023-08-25:00:16:00

Failed conversion of ``o o t:00'' using format ``%d %b %H:%M:%S''
date: illegal time format
usage: date [-jnRu] [-I[date|hours|minutes|seconds]] [-f input_fmt]
            [-r filename|seconds] [-v[+|-]val[y|m|w|d|H|M|S]]
            [[[[mm]dd]HH]MM[[cc]yy][.SS] | new_date] [+output_fmt]

when I add echo "${last_reboot_response}"
I get
o o t

yuk... what does just last -1 reboot put out?

reboot time                                Thu 12 Oct 14:41

Can you test this please?
last -1 reboot | sed -e 's/  */ /g' | awk '{print $4" "$5" "$6;}'
To be clear... running as root/sudo.

last -1 reboot | sed -e 's/ */ /g' | awk '{print $4" "$5" "$6;}'

o o t

OK... so this is a github problem... I can use that same snippet of code directly from BBedit and it works... but the code I just copied and pasted from github doesn't work. They must be sneaking in some invisibles in there.

https://github.com/Macjutsu/super/releases/tag/v4.0.0-beta6

I get this error when I run b6
Error: Unregonized last reboot date format: 12 Oct 14:41

Of course... b/c why can't any of these methods have the same freaking time format?
Anyhow... this is clearly a localization issue... so If you could please replace the following if statement starting on line 3541 and lemme know how that goes... thanks again!
if [[ "${last_reboot_response}" =~ ^[a-zA-Z]{3}\ [0-9]{2}\ [0-9]{2}:[0-9]{2}$ ]]; then
	mac_last_startup=$(date -j -f "%b %d %H:%M:%S" "${last_reboot_response}:00" +"%Y-%m-%d:%H:%M:%S") # Expected output: 2023-08-25:00:16:00
elif [[ "${last_reboot_response}" =~ ^[0-9]{2}\ [a-zA-Z]{3}\ [0-9]{2}:[0-9]{2}$ ]]; then
	mac_last_startup=$(date -j -f "%d %b %H:%M:%S" "${last_reboot_response}:00" +"%Y-%m-%d:%H:%M:%S") # Expected output: 2023-08-25:00:16:00
else # Unknown format.
	log_super "Error: Unregonized last reboot date format: ${last_reboot_response}"; check_error="TRUE"
fi

I think it's working now.

@bibliocommons-github
If you find your way back here please try this latest version... replace the if statement starting on line 3541 with this...
if [[ "${last_reboot_response}" =~ ^[a-zA-Z]{3}\ [0-9]+\ [0-9]{2}:[0-9]{2}$ ]]; then
	mac_last_startup=$(date -j -f "%b %d %H:%M:%S" "${last_reboot_response}:00" +"%Y-%m-%d:%H:%M:%S") # Expected output: 2023-08-25:00:16:00
elif [[ "${last_reboot_response}" =~ ^[0-9]+\ [a-zA-Z]{3}\ [0-9]{2}:[0-9]{2}$ ]]; then
	mac_last_startup=$(date -j -f "%d %b %H:%M:%S" "${last_reboot_response}:00" +"%Y-%m-%d:%H:%M:%S") # Expected output: 2023-08-25:00:16:00
else # Unknown format.
	log_super "Error: Unrecognized last reboot date format: ${last_reboot_response}"; check_error="TRUE"
fi


WAD - (hehe, I meant WIP for the branch name, but oh well)

pretty rudimentary right now.  However, it works! Main points:

its in commonJS now, and requires a node version that supports ES2015 features such as Class and Promise.
options is very rudimentary at the moment. I'm not sure what the best way to parse options natively (i.e. not writing in strings).
I want to make it isomorphic, without too much overhead. Bable perhaps? or just write two versions?
I also need to add tests, which is something I've never done. I'm thinking https://facebook.github.io/jest/ over mocha or something else.
I've not settled on a style guide. What styleguide do you guys use for openAQ normally?

Anyhow, first published module, if I've gone about this in the wrong way completely, please feel free to rip it to shreds. 👍

💃

I want to make it isomorphic, without too much overhead. Bable perhaps? or just write two versions?

babel can help with this and it'll also let you make it more generic if you use it to transpile the code to older ES standards.

options is very rudimentary at the moment. I'm not sure what the best way to parse options natively (i.e. not writing in strings).

Allowing the user to specify an object might be more JS'y?
const options = { country: "US", page: 2 }

Maybe call them parameters or something other than options since I think options usually refers to the request itself (like timeout) and not the data that goes along with it.

I also need to add tests, which is something I've never done. I'm thinking https://facebook.github.io/jest/ over mocha or something else.

openaq-api uses chai/mocha, but I think any framework is fine, don't overthink it. One thing to watch out for is you should probably mock all the requests. You don't want your code to be hitting the API every time the tests are run, you're testing the network then and the API and not your code.

I've not settled on a style guide. What styleguide do you guys use for openAQ normally?

We use eslint and a slightly modified version of https://github.com/feross/eslint-config-standard (gotta have those semicolons!). https://github.com/openaq/openaq-api/blob/develop/.eslintrc

babel can help with this and it'll also let you make it more generic if you use it to transpile the code to older ES standards.
yeah, I'll start digging into how to do this with babel. The most ideal would be to only include the parts of babel that I need. Not sure how that works with babel specifically. anyhow.

Allowing the user to specify an object might be more JS'y?
const options = { country: "US", page: 2 }

Yeah, that bit makes perfect sense. I think what I was struggling with in my mind was:

How to write a compact helper function to loop through an object of options, if any, and concatenate them into a valid string to be appended.  The basic version of this shouldn't be hard but the next issue could impact its design and possibly (?) make it much more complex.

should I enforce checking for valid options for each individual method? Or should I simply parse any and all params given in a parameter object and call it good? The former seems like it would greatly increase the complexity of each, but maybe i'm overthinking it?

One thing to watch out for is you should probably mock all the requests.
yep, which jest obviously supports. After a cursory reading of this though, its not clear to me how generic mocks should be. I'll need to dig in further and probably take a swing at it and then ask more pointed questions.

(gotta have those semicolons!)
bleh. I hate semi's for two reasons. It's very unclear from a newbie perpective where you actually need them. And if you dont actually need them, then why the hell keep them at all? Proper whitespace formatting does a much better job at indicating where logic begins and ends. Anyhow, thats a big can o' worms.

Thanks for the feedback, I'll merge what I've got now, and work on the rest later. Eventually, if you want, we can move this into the openaq org.

yeah, I'll start digging into how to do this with babel. The most ideal would be to only include the parts of babel that I need. Not sure how that works with babel specifically. anyhow.

Honestly for babel, I've just figured out a pattern to do what I need to do and that's pretty much all I know how to do. You can see how we use it for transpiling at https://github.com/openaq/openaq-fetch/blob/develop/index.js

How to write a compact helper function to loop through an object of options, if any, and concatenate them into a valid string to be appended. The basic version of this shouldn't be hard but the next issue could impact its design and possibly (?) make it much more complex.

should I enforce checking for valid options for each individual method? Or should I simply parse any and all params given in a parameter object and call it good? The former seems like it would greatly increase the complexity of each, but maybe i'm overthinking it?

If it were me, I'd just write a loop over each key and concatenate it appropriately. I wouldn't worry about whether they're valid parameters. That'd make the package very dependent on the API and force updates whenever there are API updates. I'm also a bit lazy... :)

Thanks for the feedback, I'll merge what I've got now, and work on the rest later. Eventually, if you want, we can move this into the openaq org.

:boom: ❤ 💃 Whatever works best for you is cool! It can either live here and we can point to it or it can live on the openaq org and you can have write access to it there. Either way, probably easier if only one copy exists between you and OpenAQ so it doesn't get confusing with forks.


Header style improperly affects default page numbering

Description
If the first page has a header but no other content, the page number takes on the styling of the header.
Example:
#set page(height: 100pt, width: 100pt)
#set page(numbering: "1")
= Header no text
#pagebreak()
= Header with text
#lorem(5)
= Header no text

Produces:

Reproduction URL
No response
Operating system
macOS
Typst version

[X] I am using the latest version of Typst

Duplicate of #241


Error check function after failed rebuild function

After a failed rebuild (snasphots were not find), when check function is called, here is the response :
[05/13/18 12:51:01] Node is rebuilding...
[05/13/18 12:52:01] Node is rebuilding...
[05/13/18 12:53:01] Node is rebuilding...

The node is working perfectly

Can you provide the rebuild.log file ?
Thanks


After deployment site is not working

The problem is that when the data is not delivered from the API call and you are using it already.

in line number 18 try giving length to by default 10 and see what it be the output.

Bro your API is giving ERROR; you can't use it. It states that requests from the browser are not allowed on the Developer plan of the News API, except when the request is coming from localhost.

In other words, the News API you are using has CORS restrictions, and it only allows requests from localhost

Bro your API is giving ERROR; you can't use it. It states that requests from the browser are not allowed on the Developer plan of the News API, except when the request is coming from localhost.

In other words, the News API you are using has CORS restrictions, and it only allows requests from localhost

OK


avs4you watermark

Describe the request
Command that adds the avs4you watermark,
just put it in the center with this image

Is your feature request related to a problem? Please describe.
No response
Additional context
No response

Added with #243


Unit tests

Closes #11
Closes #2
Add unit tests using the pytest framework.
The tests are run through a new CI job, which also uploads a code coverage report to codecov.io, see app.codecov.io/gh/SINTEF/soft7.
As a minimum, before making this PR ready for review, I want some basic unit tests for:

[ ] factories/dataspace_factory.py
[ ] factories/datasource_factory.py

When tests for dataspace_factory.py have been put in place, issue #15 can be addressed, and it should be immediately apparent if any changes introduce a degradation of the expected behaviour.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@fa5ac36). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #17   +/-   ##
=======================================
  Coverage        ?   10.87%
=======================================
  Files           ?       12
  Lines           ?      377
  Branches        ?        0
=======================================
  Hits            ?       41
  Misses          ?      336
  Partials        ?        0

Flag
Coverage Δ

pytest
10.87% <0.00%> (?)

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

The work on this has opened my eyes to the fact that the various parts of this repository really should be looked at again, possibly with a new approach to what it should actually contain - and do.

Closing as these changes are either already implemented through other PRs or are outdated.


Eager code with tf.function decoration is >2x slower than without it.

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

TensorFlow version:
TF 2.0.0
Python version:
Python 3.7.6
CUDA/cuDNN version:
10.0.130
GPU model and memory:
GTX2080

Describe the current behavior
I walk through the @tf.function usage examples from Tensorflow Tutorial. There is a example shows that the eager code version of  conv_layer with @tf.function decorator could run faster than without it. However, when I imitate it and write eager code version  of embedding_layer, the decorator one is >2x slower than the not decorator one. See the code below
Code to reproduce the issue
# check time it for embedding layer
from tensorflow.keras.initializers import RandomUniform
import numpy as np
import tensorflow as tf

input_dim = 1000
output_dim = 300
emb_layer = tf.keras.layers.Embedding(input_dim, output_dim, RandomUniform(-1,1))

@tf.function
def emb_fn(indices):
    return emb_layer(indices)

indices = np.random.randint(0, input_dim-1, size=64)
indices = tf.constant(indices)
# warm up
emb_layer(indices); emb_fn(indices)

print("Eager embed: ", timeit.timeit(lambda: emb_layer(indices), number=10000))
print("Function embed: ", timeit.timeit(lambda: emb_fn(indices), number=10000))

The result
Eager embed:  1.895560858771205
Function embed:  4.15901411511004

@HansiZeng,
I tried to reproduce the issue but could not find much difference in the execution time. Please find the Gist here.


Shard chain sanity test

content
Adding basic shard chain sanity tests to pass through the common flows and build out the basic test tooling/helpers to build such tests.
substantive changes/bug fixes

initialize latest_block_header for genesis ShardState similar to phase 0 genesis
shard block parent root should be signing_root rather than hash_tree_root
handle case in process_shard_attestations when no participants
fix verification of beacon_block_root within process_shard_block_header. Utilize current shard_state epoch instead of parent

cannot use shard parent epoch because you are not guaranteed to be able to look back arbitrarily far into the beacon state for state root verification.
current method is also not fully correct. It assumed at slot 0 of an epoch that the shard block proposer already has the beacon block from slot 0 of that same epoch which is not a reasonable assumption as the shard proposer is supposed to create a block at the start of the slot (before the beacon block could propogate). We could consider the beacon state root epoch to come from shard_state.slot - 1. I think this works, but is a bit hacky. Open to discussion/alternatives

notes
Currently is open against #1383 but will swap if that gets merged first.
As discussed with @protolambda, I realized that the @always_bls decorator is order dependent with @spec_state_test and the basic shard block tests were being skipped due to this. This deserves an investigation to (1) see if any other tests are affected by this and (2) check into removing this order dependency (or at least throw a warning if done incorrectly)

@JustinDrake, I specifically want input on how to handle beacon_block_root wrt the shard slot

current method is also not fully correct. It assumed at slot 0 of an epoch that the shard block proposer already has the beacon block from slot 0 of that same epoch which is not a reasonable assumption as the shard proposer is supposed to create a block at the start of the slot (before the beacon block could propogate). We could consider the beacon state root epoch to come from shard_state.slot - 1. I think this works, but is a bit hacky. Open to discussion/alternatives

Need to discuss this note I left in the header comment. Still not resolved as is

I'm of the mind to get this and #1383 merged asap. Then continue conversations about the shape of phase 1 offline


Yozh is no longer recognized as a Drive

Hey, this is David R from Sigma.
Before, when I would plug the robot into the computer, it would get recognized as a storage drive and pop up in files. Now it just draws power but doesn't connect. How should I fix this?
Contact me at<EMAIL_ADDRESS>

Hi, David!
Yes, we had multiple problems with CIRCUITPY drive disappearing, not sure why.
I wrote some ways to fix this in this google doc:
https://docs.google.com/document/d/1C4rf-fTO2_A6d-Higqi1a8Bx7qhjb13ecvsjiudTvfw/edit?usp=sharing
(see page 2)

closed


Add support to highlight critical paths

This PR introduces a feature to highlight critical paths in an attack graph. Aggregtation functions can set a hidden attribute _marking with the ID of the child node to mark the edge from the node to the specified child node. Therefore, the collection object passed to the aggregation functions now includes the ID of all child nodes and the ID of the node itself.
Markings are only shown if a node is selected. Additionally, it only shows the critical path from the selected node "downwards". The following shows how an examplary marking looks when selecting a node:

The aggregation functions for the TS 50701 template were updated to include the _marking attribute in aggregations. For the AND function all outgoing edges are marked because it doesn't make sense to mark a specific child node as critical for the worst likelihood/risk. Hence, all outgoing edges are marked to show also the influence of critical nodes below an AND node:

Note: The aggregation functions for the RKL template shall be updated before merging this PR.
Closes #57.

@tobi18991 Thank you for the PR!
To check the backwards compatibility, I created the following table.
Did I catch the behaviour correctly? Am I missing some cases?

Plugin updated
Functions updated in AttackGraph
Behaviour

yes
yes
new behaviour as described in PR

yes
no
old behaviour as _marking is never written and the plugin ignores the missing attribute

no
yes
old behaviour. Functions write _marking but it is never evaluated by the (old) plugin

no
no
old behaviour

@Mq89 Yes, that's right. I would only add that when the plugin is not updated but the functions are updated that the value of _marking is undefined because the id attribute in the collection as well as in the child nodes does not exist. Nevertheless, as this does not lead to an error (only undefined during local tests) it should be okay.


Add optional third parameter to route: name

This name will be sent along with SET_PAGE requests.
Useful for things like page tracking requests; we send different data based on the type of page, which may have several different routes (for example, the event data's target is a link if you're on a comment page, or a subreddit if you're on a subreddit listing page, etc.)
👓 @nramadas @schwers

nuking in favor of exposing getHandler(url).


Gameplay optimizations. Part 1

A small but large list of improvements:

Slabs. Now the player jumps on them automatically. (Fix #4)
You can now put another half-block on the half-block and then they will be replaced with an alternative block.
The raycast now also returns normal. This added a lot of opportunities and saved from unpleasant consequences. However, the raycast may still work somewhat incorrectly on the edges of the blocks.
The raycast additionally checks whether it has hit the block.
Now the raycast has the ignoreLiquid parameter, which allows you to ignore liquid blocks. Now this is quite enough, but in the future you will most likely have to use something like a mask.
In fact, this is a continuation of 2. Now you can choose the block installation logic for each block. As an example, now flowers can only be placed on the ground, grass or sand.
Now, after placed block, there is a check for a collision of the block with the player.
The player can no longer jump while in the air.
Player movement is now normalized. Previously, the player could move faster by holding down 2 keys.
Fixed the size of the player's collider. (Fix #5)
The maximum shadow is now not so dark. (Part #2)
Lava now has colliderType = BlockColliderType_Liquid

Also, some things that worked well before my changes were made were corrected. In addition, I had to copy some of the code from the raylib master branch, since they should appear only in a future version (I needed the Vector3ClampValue() function, which is not in raylib 4.0.0).

https://user-images.githubusercontent.com/5556081/179912794-a7540868-f210-4bb6-8abd-e711fae7e692.mp4

Awesome, thanks! 🙂


Issue in Deploy Azure Infrastructure

When we try to deploy the webapi Application it throws an error HTTP Error 500.30 - ASP.NET Core app failed to start
https://github.com/microsoft/chat-copilot/blob/main/scripts/deploy/README.md#powershell-1

in the deployment terminal, I was able to see the fallowing error : UserWarning: You are using cryptography on a 32-bit Python on a 64-bit Windows Operating System. Cryptography will be significantly faster if you switch to using 64-bit Python.

please help in resolving this issue

Hello @shashankms9 , check to see if the App Service is set to 64-bit.
The .NET Core app is most likely built as 64-bit and needs to be in an App Service that is configured with 64-bit.
You can also run the EXE manually using the KUDU console and see any startup exception.

In the deployment log it showing as below

EXE manually using the KUDU console it is throwing some error

in the Configuration stack has not been Configured with .net, i have tried after Configuring it to .net 8 still same issue

Thanks for running the command line test. It shows us the error.
The error is then in your appsettings.json or App Settings Environment Variables.
"AzureQueue" has been deprecated.
It is now "AzureQueues"
Change this link in the main.bicep
{
name: 'KernelMemory:DataIngestion:DistributedOrchestration:QueueType'
value: 'AzureQueue'
}
to
{
name: 'KernelMemory:DataIngestion:DistributedOrchestration:QueueType'
value: 'AzureQueues'
}
And redeploy.
And we should fix this in the repo as well.


Add support for Python 3.10

Python 3.10 was released on 2021-10-04:

https://discuss.python.org/t/python-3-10-0-is-now-available/10955

Also replace testing Windows on AppVeyor with Windows and macOS on GitHub Actions.

GHA doesn't support testing Python 3.4
3.4 is EOL and will be removed in https://github.com/pypa/pep517/pull/131
3.4 is failing on AppVeyor (on my fork)
AppVeyor isn't enabled for this repo
AppVeyor has a single sequential job, taking nearly 17 minutes for just Windows
GHA has 20 jobs in parallel, taking less than 2 minutes for all three operating systems

That looks good, thanks.


Make file loader optional

Adding boolean explorer.config.fileLoader.

Test Notes
Confirm that the fileLoader control isn't shown when you use the settings input at the bottom explorer test page to change the fileLoader setting from true to false

Working as expected. When true is changed to false, the File Loader control no longer appears.


Fix issues when using map or set types in structs in python

Thanks for your patch @lucasmarshall, in order for us to accept a pull request we need to have a jira ticket associated with it. Please see our contributing guide at http://thrift.apache.org/docs/HowToContribute


Download failed for certain courses, for example with /draft/:id/ in the URL)

Draft courses like the one below
./udemy-dl.py https://www.udemy.com/draft/368340/learn/ -k cookies.txt
would fail to download with "[-] : Downloading course information, course id not found .. (failed)". The issue lied in the comparison made in the _extract_course() method.
Also, the shebang should get the correct Python interpreter from the user environment not a hard-coded path. I typically run Python scripts in a virtualenv and the original code would fail.

@hknust how can i reproduce this error i don't see any issue with using.
def __extract_course(self, response, course_name):
        _temp = {}
        if response:
            for entry in response:
                course_id = entry.get('id')
                published_title = entry.get('published_title')
                if course_name in (published_title, course_id):
                    _temp = entry
        return _temp


Batchsize for test

Hello, thank you very much for your excellent work!
When I tried to reproduce your results: for the same test data, if the batch size is different, different test miou accuracies will be obtained. What is the reason for this? In theory, should batch size not affect the accuracy of testing?
For example, when batchsize is 96, my test miou is 45.78, but when I turn it to 1, my test miou is 35.63?

Hi, thanks for your interest. Yes, the batch-size during inference should not affect the results. But if you also change the batch-size during training, the results may be different.

发件人: Zheng Yuanlong @.>
发送时间: 2024年9月17日 2:02
收件人: niejiahao1998/IFA @.>
抄送: Subscribed @.***>
主题: [niejiahao1998/IFA] Batchsize for test (Issue #6)
Hello, thank you very much for your excellent work!
When I tried to reproduce your results: for the same test data, if the batch size is different, different test miou accuracies will be obtained. What is the reason for this? In theory, should batch size not affect the accuracy of testing?
For example, when batchsize is 96, my test miou is 45.78, but when I turn it to 1, my test miou is 35.63?
―
Reply to this email directly, view it on GitHubhttps://github.com/niejiahao1998/IFA/issues/6, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AON6SS2Y2CTALYMSX5ABVODZW4MM5AVCNFSM6AAAAABOJ2GVJ6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGUZDSMJSGE3TENQ.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hello, I have downloaded the model weights you provided and tested them according to the method you provided. I found that the accuracy of the test varies greatly with different batch sizes. Specifically, as follows:
-Dataset: Deepglobe
-Sh file: test.sh
-Python file: test. py (all source code, unmodified)

batchsize: 96, 1
-Usage model: resnet50_1shot_avg-50.63pth
-Scenario: One shot test
-Result: for bs 96: 50.63 for bs 1: 44.76

The batch-size for inference should not affect the performances. I did not encounter the same problem before.

发件人: Zheng Yuanlong @.>
发送时间: Wednesday, September 18, 2024 2:39:18 PM
收件人: niejiahao1998/IFA @.>
抄送: #NIE JIAHAO# @.>; State change @.>
主题: Re: [niejiahao1998/IFA] Batchsize for test (Issue #6)
Hello, I have downloaded the model weights you provided and tested them according to the method you provided. I found that the accuracy of the test varies greatly with different batch sizes. Specifically, as follows:
-Dataset: Deepglobe
-Sh file: test.sh
-Python file: test. py (all source code, unmodified)

batchsize: 96, 1
-Usage model: resnet50_1shot_avg-50.63pth
-Scenario: One shot test
-Result: for bs 96: 50.63 for bs 1: 44.76

image.png (view on web)https://github.com/user-attachments/assets/126e2433-e376-4e0c-be0c-ebd0a9482935
image.png (view on web)https://github.com/user-attachments/assets/073fdb0c-3fa4-41f7-888c-426433619e73
―
Reply to this email directly, view it on GitHubhttps://github.com/niejiahao1998/IFA/issues/6#issuecomment-2357626330, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AON6SSZC7KGLYTN4YFYIPIDZXENZNAVCNFSM6AAAAABOJ2GVJ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJXGYZDMMZTGA.
You are receiving this because you modified the open/close state.Message ID: @.***>


[IoT Hub] - X509CA support

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1723

We will need to change the swagger spec again. Please don't merge.


Stress test the crateDB cluster using HTTP and JDBC There is a large difference in response speed between the two, with JDBC's response time approximately HTTP Ten times the way. Why is that?

CrateDB version:
3.1.0
Environment description:

Stress test the crateDB cluster using HTTP and JDBC
There is a large difference in response speed between the two, with JDBC's response time approximately HTTP
Ten times the way. Why is that?

Steps to reproduce:

We'll need more information to have any clue what's going on.
Please describe exactly how you're running your stress tests and how you take measurements.


Default jsconfig target

vscode 1.52.1
Code 1:
[].includes; // function
Object.entries; // any

Code 2:
import fs from "fs";

Object.entries; // function
/_/.dotAll; // boolean
[].flat; // any

Apparently, default target in absence of a jsconfig/tsconfig is es2016. Adding a node import bumps it up to es2018.
I'd expect it to be esnext in both cases.

It doesn’t change the default, it triple-slash references the libs it needs. https://github.com/DefinitelyTyped/DefinitelyTyped/blob/866931d58a831e163ef40f7830864270c281a20f/types/node/base.d.ts#L10

@andrewbranch
The issue is not node types changing the lib, but what it's set to by default to begin with. Why es2016?

Default is ES3. See #36180.

but where'd the definitions for ES2015-16 stuff come from then? Is this something specific to tsserver/intellisense in vscode?

What else is in your project / what does your jsconfig.json look like? The default lib file brings in es5, dom, webworker.importscripts, and scripthost. It appears as expected for me.

I'm getting this in a project with no jsconfig (same when no project is open at all):

Oddly enough, creating an empty jsconfig.json changes the behavior to what you described.
I'm guessing there's a set of default values used for intellisense in absense of a jsconfig.json different from defaults for the jsconfig itself?

When there’s no config file, we create what’s called an “inferred project,” and we actually let the editors set whatever options they want for those. So the default you observed actually comes from VS Code, here: https://github.com/microsoft/vscode/blob/b84858babef576c432154199a8cb7b03a3a0d914/extensions/typescript-language-features/src/utils/tsconfig.ts#L25

That makes sense. Thanks for the explanation.


Provide an example/tools on how to compile source code without emscripten.

emscripten is basically llvm + a runtime lib for browser
if this project already provides the runtime then there is no need for emscripten in the toolchain
furthermore, the latest llvm already provides some improvements in this regard
I was able to do this some time ago (compile a minimal .wasm program with only llvm) but it was a very tricky endeavour so if other developers want to achieve the same thing they would have to walk through stones to accomplish it
so, a working an example and pipeline for compiling without emscripten would be a nice addition to this repository

Thanks for the comments, @almosnow
You should be right that a compiler like clang should be able to do the work. We will do some tests and update the README accordingly.

The issue is solved in PR #37


include fasth2

Hi @wapu , I included my implementation for a "fast" Householder product. You will find the final project report under https://github.com/paulhfu/Faster-Orthogonal-Parameterization-with-Householder-Matrices/blob/master/report/report.pdf
All my implementations are in https://github.com/paulhfu/Faster-Orthogonal-Parameterization-with-Householder-Matrices/blob/master/FastH/fast_h.py
Some tests are in https://github.com/paulhfu/Faster-Orthogonal-Parameterization-with-Householder-Matrices/blob/master/test/test_fast_h.py
I also altered the tests in this repo to include Householder perms. Let me know if I should revert that.

Thank you very much Paul, sorry for being slow with the merge!