SQL Trigger for Java not supported in Microsoft.Azure.Functions.ExtensionBundle 4.x

Hi,
When using Microsoft.Azure.Functions.ExtensionBundle
The SQL Trigger is does not work for Java applications.
The function doesn't get triggered. The Microsoft.Azure.Functions.ExtensionBundle.Preview supports this.
However the feature was released: https://learn.microsoft.com/en-us/azure/azure-functions/functions-bindings-azure-sql-trigger?tabs=isolated-process%2Cportal&pivots=programming-language-java

hello do you have an expected release time for the @SqlTrigger. I have cobbled together an example for playing with it, but am running into this error:
[2023-12-19T22:35:07.527Z] System.Private.CoreLib: Exception while executing function: Functions.EmployeeTrigger. System.Private.CoreLib: Result: Failure
Exception: ClassCastException: class com.microsoft.azure.functions.worker.binding.ExecutionContextDataSource cannot be cast to class java.lang.String (com.microsoft.azure.functions.worker.binding.ExecutionContex
tDataSource is in unnamed module of loader 'app'; java.lang.String is in module java.base of loader 'bootstrap')
Stack: java.lang.ClassCastException: class com.microsoft.azure.functions.worker.binding.ExecutionContextDataSource cannot be cast to class java.lang.String (com.microsoft.azure.functions.worker.binding.Execution
ContextDataSource is in unnamed module of loader 'app'; java.lang.String is in module java.base of loader 'bootstrap')

Are you using the Preview bundle? That should still work to at least verify that your function is written correctly. The error you're getting isn't one I immediately recognize, but it seems like a separate issue at first glance. Maybe related to https://github.com/Azure/azure-functions-java-worker/issues/315? @lucyzhang929 Is this an error you recognize?
If you're still unable to figure it out, please open a new issue for investigation. And if you could include your function code that would help track down what's going on here.

I haven't seen this error before. @niravassar-hitrust As Charles suggested, please open a new issue and share your function code if you continue to see the error.

Thanks for the attention. Yes we have been looking into this issue, and from what i can tell its an infrastructure issue throwing an exception with the java-worker library, probably something you all will have to look into. I will create another issue here in github, and post detailed instructions on my setup of local db with Azure Sql Server (local), and also the commands i ran, along with a zip of my java function app project. I hope this will help. Expect a post today.

filed it in https://github.com/Azure/azure-functions-extension-bundles/issues/335


Good to know when on *nix

https://github.com/fsharp/FSharp.Compiler.Service/blob/08daf6d5a0abb62c01a475ea442c2996b0e79335/FSharpBuild.Directory.Build.props#L39-L54

https://github.com/dotnet/sdk/issues/335#issuecomment-530293809


Update for AKS deployment

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/microservices/api-design.md
:white_check_mark:Succeeded
View

docs/microservices/ci-cd.md
:white_check_mark:Succeeded
View

docs/microservices/data-considerations.md
:white_check_mark:Succeeded
View

docs/microservices/domain-analysis.md
:white_check_mark:Succeeded
View

docs/microservices/gateway.md
:white_check_mark:Succeeded
View

docs/microservices/index.md
:white_check_mark:Succeeded
View

docs/microservices/ingestion-workflow.md
:white_check_mark:Succeeded
View

docs/microservices/interservice-communication.md
:white_check_mark:Succeeded
View

docs/microservices/logging-monitoring.md
:white_check_mark:Succeeded
View

docs/microservices/microservice-boundaries.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/microservices/api-design.md
:white_check_mark:Succeeded
View

docs/microservices/ci-cd.md
:white_check_mark:Succeeded
View

docs/microservices/data-considerations.md
:white_check_mark:Succeeded
View

docs/microservices/domain-analysis.md
:white_check_mark:Succeeded
View

docs/microservices/gateway.md
:white_check_mark:Succeeded
View

docs/microservices/index.md
:white_check_mark:Succeeded
View

docs/microservices/ingestion-workflow.md
:white_check_mark:Succeeded
View

docs/microservices/interservice-communication.md
:white_check_mark:Succeeded
View

docs/microservices/logging-monitoring.md
:white_check_mark:Succeeded
View

docs/microservices/microservice-boundaries.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[V2] Using anchor with react-router break styled

I have found a bug in v2.0.0-5
When clicking on url that use an anchor, the <ThemeProvider /> has no more effect.
Reverting to v1.4.3 solve the issue.
Version
2.0.0-5
Reproduction
http://www.webpackbin.com/NJGs7ebqG
https://webpackbin.herokuapp.com/NJGs7ebqG
Steps to reproduce
Use the anchroScroll method from here in your react-router : https://github.com/ReactTraining/react-router/issues/394#issuecomment-220221604
Expected Behavior
To scroll to the anchor in question
Actual Behavior
It scroll but It also break the <ThemeProvider /> and use the theme from defaultProps instead.

@mxstbr This bug is a bit major. It does not only concern the react-router...
I also encounter issue ussing <Provider /> and react-redux. On my LoginForm when I type in the <input name="username" />, the store get updated and the <ThemeProvider /> also lose the theme
Almost all our apps use styled-components v2 and are impacted by this bug.

Seems to be the same issue : #574

Corrected and released in 2.0.0-9.


Update free-programming-books-langs.md

added think stats,book by allen b downey on EDA
What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Duplicate of #10879


vac error?

plz fix fasttttt

not sure if you realized this project has been dead for a good while now

It's not "dead"
There is just no need to update it?
The VAC error is probably a "you" problem.
could be fixed with (vmprotect, etc)

Simple solution, take your dll, take your injector (I recommend rando injector), get VMProtect, go ahead and VMProtect both dll and the injector and fixed no more VAC errors

duplicate of #27 / out of scope

duplicate of #27 / out of scope


Stats of the memory optimised

Understanding the metrics is crucial, especially when there is a noticeable enhancement. This request is for the inclusion of a feature that would add details regarding the percentage or quantity of memory optimized after the execution of the betteralign ./... command. Such information would contribute to a more comprehensive assessment of the impact and effectiveness of the optimization process.

Partially addressed in https://github.com/dkorunic/betteralign/pull/13.

I think we can close this for now.


Fix chat event typos

@ProbablePrime, thanks for your PR! By analyzing the blame information on this pull request, we identified @connor4312 to be a potential reviewer

lgtm


Improve text filtering

Emails are currently filtered and some headers are removed manually
Improve the removal of headers and email metadata either through better parsing when reading the input or through use of LLMs

test emails and see how accurate models perform


Cannot capture the process output

I'm thinking of adding these 2 to the method signature:
IObserver<string> outputObserver = null, IObserver<string> errorObserver = null

Maybe just use the CliWrap project instead? I'd rather not scope creep when
there's already an alternative that covers that use case, imho. That
project covers many more use cases.  😊
https://github.com/Tyrrrz/CliWrap
https://www.nuget.org/packages/CliWrap/
On Wed, Jul 4, 2018, 3:34 AM José Manuel Nieto<EMAIL_ADDRESS>wrote:

I'm thinking of adding these 2 to the method signature:
IObserver outputObserver = null, IObserver errorObserver =
null
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jamesmanning/RunProcessAsTask/issues/21, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AANxWtfF7rHrTVmbx0xC1dmjapoQ_KFPks5uDG_ngaJpZM4VCF2y
.

To really wrap a cli, you'll want to preserve the interleaving of stderr and stdout - even though that's an intrinsic race-condition.  After all, many CLI apps write to both stderr and stdout, and in an order that may well matter.
I don't believe cliWrap actually does that, and even if it did - it's not using IObservable, so actually registering code to execute just-in-time when there's output available is a pain.

I'm going to add a comment into the README about it but I think there's enough available alternatives (CliWrap, Process directly) that I'd rather keep handling/observing the stdout/stderr outside the scope of this particular project.


Add aa option for runtime register

Why?
If we use extend_runtime_measurement to measure policy or init-data we need to specify in which register it is measuring into, since the default might not work in each situation.
There is an option to specify it at the caller site, which would be kata-agent. it would be an attestation-related parameter that we would have to pipe through via kata to the attester, which is impractical.
How
Currently the attester is being initialized in each RPC call that is interacting with the TEE, this change moves the TEE detection into the initialization of the AttestationAgent instance.
A default_register_index field is now added to the AA configuration, if this per-guest default is not set it will fall back to a hardcoded default.

Oh. I think there would be some overlap thing for pcr index here . Besides this, I am ok with make attester a struct member of AA

@Xynnn007 there are some overlaps with

Oh. I think there would be some overlap thing for pcr index here . Besides this, I am ok with make attester a struct member of AA

yes, I just noticed the redundancy. let's get the other PR merged first, or maybe you can cherry-pick ad78ee2c80ba9444265c40cddc785a9422faa79e?

@mkulke Yea, I cherry picked the commit in that PR

superseded by #548


fix(navigation-plan): add query string from redirect instruction

See issue: https://github.com/aurelia/router/issues/639

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@MartyBoi It doesn't seem we can simply append the redirect location. There're cases where things would override each other, and there is an issue with redirect with dynamic part in the URL too.
Would need @davismj to chime in

This seems like a reasonable and non breaking change. Could it be merged?

test

I would really appreciate if a maintainer would look at this PR. It seems really simple and I have used it for a long time now in a big app without any issues.
ping @bigopon

@rmja there' concerns stated in my comments above https://github.com/aurelia/router/pull/643#issuecomment-507198131
I don't think we can simply merge this yet. And for the very least, we should have a failing test, can you help with that? I can then help proceed this forward faster.

@bigopon Thanks for getting back. There is a very simple example in the issue https://github.com/aurelia/router/issues/667.
Basically, if a route is configured like { route: "does-not-work", redirect: "/dashboard?autoselect=true"; }, then the parameter autoselect is not passed on as a parameter to dashboard.

How do we proceed with this?

@bigopon ping..

Thanks @rmja , it's been a bit difficult to proceed with this for I haven't had time to focus on v1 repos (there been a few queued items), and this repo specifically needs special setup work refactoring to make all future work fast/easy. Will keep this in mind and act as soon as I can.


Raspberrypi false positive

Hello, it seems, that domain www3.raspberrypi.org is being blocked by this block list. It impacts the Raspberry Pi update process and renders the system outdated. Please, consider removing this domain from the list.

I don't see it on any of AG's own lists.  Maybe another you use?  It does come up in StevenBlack's.


[kubespray] api server was refused

Where to config the api-server of k8s deployed with kubespray? It reports:
 $ kubectl get namespace

The connection to the server localhost:8080 was refused - did you specify the right host or port?

There seems not an item to config the api-server :(

check ~/.kube/config
https://github.com/microsoft/pai/blob/a8585bfe8648bae8d06ea2429c9a8811f9a4dc01/contrib/kubespray/script/service-boot.sh#L31


Fix Ruby 2 CI builds

Setup-ruby installs bundler, so I think there's no need to do it manually. Trying to run these locally with act now, see if it helps.

It passed! Fantastic! Love a Big Green Button 😁


Suggestion: Support gif images with gifsicle

Gifsicle is a good terminal based tool that can optimize and compress gif images. It would be awesome if trimage supported it.

gifsicle is nice, but the compression it does it actually not lossless, whilst in Trimage I only want to do lossless compression. There are, AFAIK, no real lossless gif compression techniques.

@Kilian Oh right! I didn't know. Nevermind then.


sendRequest and sendNotification unclear

So I'm writing a languageserver for a hobby-compiler of mine, im writing the server itself in Java, using the lsp4j framework / library, and I'm using the @JsonNotification thing, code is as follows
public interface ExtendedLanguageClient extends LanguageClient {

    @JsonNotification("exampleLang/endpoint")
    void endpoint(HTMLString htmlString);
}

And I want to invoke this endpoint from the JS side of things (in VSCode)
I do this as follows:
	client.onReady().then(() => {
		client.onNotification(new NotificationType(`exampleLang/endpoint`), (event: any) => {
			console.log(event);
		});
		console.log("Sending notif");
		client.sendNotification(new NotificationType(`exampleLang/endpoint`));
		console.log("Sent notif");
	});

But the onNotification event never gets called, any idea why? It's kind of urgent and the documentation on this specific function is very lacking.
I of course also tried to use sendRequest but it takes a method, and the rest isn't really that well documented.
Is there a proper documentation page I could look at?
Or maybe one of you could guide me in the right direction?
Thanks in advance.

@Yuhanun sendNotification and sendRequest on the client API send a notification or a request to the server. The server API will send them to the client.
You might want to have a look at https://github.com/microsoft/vscode-extension-samples/tree/master/lsp-sample for an example implementation of a server in node and an LSP client triggered from withing VS Code.
If you only want to implement your language as a pure VS Code extension please refer to the VS Code language API here: https://code.visualstudio.com/api/references/vscode-api#languages


Mocked dynamic response content type not defined in OAS spec

Describe the bug
Prism mocks dynamic XML responses from the browser. XML is not defined in my spec.
To Reproduce

Given this OpenAPI document'

openapi: 3.0.0
info:
  title: Example API
  version: '1.0'
  contact:
    name: Bear
    url: 'https://bear.dojo'
    email<EMAIL_ADDRESS>  termsOfService: 'https://bear.dojo/terms'
  description: The Bear Dojo API
paths:
  /honey:
    get:
      summary: Get Raw Honey
      responses:
        '200':
          description: 'Success, found the bears can have some honey.'
          content:
            application/json:
              schema:
                type: object
                properties:
                  ok:
                    type: boolean
                    example: true
                  honey:
                    type: object
                    properties:
                       name:
                         type: string
                       location:
                         type: string
                       species:
                         type: string
                         example: killer bees
                required:
                  - ok
                  - honey
      description: Get some raw honey.
      operationId: getRawHoney

Run this CLI command

prism mock https://gist.githubusercontent.com/pytlesk4/ddbd7c6f791e68b2b34b0cbd6b11f250/raw/5170c01992bc43c8f8f335ec64ae5ca73257e6e8/bear.dojo.v1.yaml

Open any browser and go to http://localhost:4010/honey.

See error

Expected behavior
Dynamic JSON response.
Environment (remove any that are not applicable):

Library version: 3.1.0
OS: OSX
Browser: Firefox

Note to self: the serialized is not correctly checking that the Accept header and the reproducible format are in sync


Removed space root support

Relates to #44

Coverage decreased (-0.6%) to 88.861% when pulling ed75671508be7750f546050d46edf65dfc209d5e on bugfix/remove-space-root-support into 91a1bbd27d64ce28a0ab362bb5ff4971e00ab57a on master.

Coverage decreased (-0.6%) to 88.861% when pulling 6fc1fc4ec678133a88be3bebc0d5d441c152aa80 on bugfix/remove-space-root-support into 91a1bbd27d64ce28a0ab362bb5ff4971e00ab57a on master.


Support Scala 2.13.11 and 2.12.18

Update acyclic to 0.3.8

master seems already support Scala 2.12.18 and Scala 2.13.11

master seems already support Scala 2.12.18 and Scala 2.13.11

This is targeting the 2.x branch which is used by Mill 0.10.x

Thank you so much!

Hi @lolgab, thank you for adding support for Scala 2.13.11 and 2.12.18. Is it possible to create a new release of amm (2.5.9), which includes the bumped minor version of Scala?

I think @lolgab already tagged a release, but for some reasons the version was not picked up by CI. Probably because the tag was pushed after the CI jobs ran. Let me try to re-run them. In case it's urgent, you can use version 2.5.8-2-159ff8f6 already.


Does PuppetX::Jenkins::Provider::Cli need to handle more exception types?

At present, PX::J::P::Cli only retries the CLI jar if the exception script matches one of the known authentication error strings.  However, I occasionally see other exception messages which I suspect are caused by the jenkins daemon not having enough time to "settle" after being [re]started.
Eg.
==> master: Error: Failed to apply catalog: Execution of '/bin/java -jar /usr/lib/jenkins/jenkins-cli.jar -s http://localhost:8080 groovy /usr/lib/jenkins/puppet_helper.groovy user_info_all' returned 255: Apr 11, 2016 1:38:45 PM hudson.remoting.SynchronousCommandTransport$ReaderThread run
==> master: SEVERE: I/O error in channel Chunked connection to http://localhost:8080/cli
==> master: java.io.StreamCorruptedException: invalid stream header: 0A0A0A0A

This error keeps coming up in the acceptance tests under travis:
  Error: Failed to apply catalog: Execution of '/usr/bin/java -jar /usr/lib/jenkins/jenkins-cli.jar -s http://localhost:8080 groovy /usr/lib/jenkins/puppet_helper.groovy job_list_json' returned 255: java.io.IOException: No X-Jenkins-CLI2-Port among [X-Jenkins, null, X-Hudson, X-Hudson-Theme, Content-Length, Expires, X-Jenkins-Session, Set-Cookie, Content-Type, Server, Cache-Control, X-Content-Type-Options]

Also frequently seeing this error message for several types and the error message appears to always have valid json. (passes validation).
Error: Failed to apply catalog: unable to parse as JSON: [ ...


Creation method moves

Description:
Adds modules to export creation methods in a more consistent/conventional fashion.
Basically anything that was available with Rx.Observable.nameHere is now available from rxjs/observable/nameHere. At least that's the idea.
Related issue (if exists):
#1554

in other words, you can write in Babel:
import { timer } from 'rxjs/observable/timer';
import { map } from 'rxjs/operator/map';

timer(1000)::map(x => x + x).subscribe(::console.log);

Cool, this looks good to me. :)

Just came to thought - maybe need to update README to includes guidance of how-to, I'd expect this'll be de-facto recommendation of import?

Let me check this in late today to tomorrow, if there isn't additional suggestions around.

I've got it, thanks for the review @kwonoj

:+1:


Bootstrap Schematic Done

Bootstrap Driver & Track-and-Hold switch for SAR ADC. Schematic & testbench done.
Initial commit of most SAR ADC schematics. Cleanup & testbench is needed

Ok, will be there in the next pull request.


refactor: change CLI mode option setting to a reasonable location. And delete GlobalCLIOptions in order of declaration

Description

Just code style, no relevant issue to solve.
Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

@underfin just to check here, vite preview didn't explicitly list mode before. Is it ok that now every command will have this option?

@patak-js It seems that --mode is useless for vite preview. The effect it does is determine whether call sirv with dev true or false.

Enable "dev" mode, which disables/skips caching. Instead, sirv will traverse the file system on every request.
Additionally, dev mode will ignore maxAge and immutable as these options generate a production-oriented Cache-Control header value. https://github.com/lukeed/sirv/tree/master/packages/sirv#optsdev

https://github.com/vitejs/vite/blob/690b35e857cea833ed4f1782e5c87b2b7064d072/packages/vite/src/node/preview.ts#L44-L51
config.isProduction is determined by --mode.
https://github.com/vitejs/vite/blob/690b35e857cea833ed4f1782e5c87b2b7064d072/packages/vite/src/node/config.ts#L323
The other commands serve, build, optimize are OK. For --mode will at least affects on tree shaking. It will be helpful when users debug some hard problems.


Design System > Build Verification > [Version <IP_ADDRESS>(s)] received on 23rd Sept 2024

Design System > Build Verification > [Version <IP_ADDRESS>(s)] received on 23rd Sept 2024
<IP_ADDRESS>(s) 🌈 Pre-release
🚀 What's Changed
Bug 671: Storybook > Component > Forms basic > Required message is not display. by @harshitsaini-waiin in https://github.com/Wai-Technologies/raaghu-react/pull/674
Bug 659 : Storybook > Components > Contact Us > Default > Displaying Multiple Error Messages At the One Time by @harshitsaini-waiin in https://github.com/Wai-Technologies/raaghu-react/pull/664
Bug 489: Storybook > Elements >Product Image >Column Layout> Alignment Issue. by @harshitsaini-waiin in https://github.com/Wai-Technologies/raaghu-react/pull/665
Enhancement 529 : Design System > Implement Contributor Component in Storybook by @hemalatagaikwad in https://github.com/Wai-Technologies/raaghu-react/pull/530
Resolved Bug 625 : Design System Issues Of Elements and Charts by @prakashmhaske in https://github.com/Wai-Technologies/raaghu-react/pull/627
Resolved Bug 655 : Billing Address After save clear by @suchithravipin1991 in https://github.com/Wai-Technologies/raaghu-react/pull/672
Resolved Bug 522 : Storybook > Charts > Scatter Chart > Docs > Scatter Chart Icon Does Not Display Proper by @suchithravipin1991 in https://github.com/Wai-Technologies/raaghu-react/pull/677
Added a Contributors File on Root Directory by @BhakareAbhishek in https://github.com/Wai-Technologies/raaghu-react/pull/683
Resolved Bug : Storybook >Components >Claims > Default > Dropdown values are hidden not display proper. by @dhirajlaha in https://github.com/Wai-Technologies/raaghu-react/pull/666
Staging : Merge Changes from Development to Staging Branch by @Ketan-Khalate in https://github.com/Wai-Technologies/raaghu-react/pull/709

Sr.No
Bug Title
Bug /Task ID
Product
Status
Worked on

1
Storybook   > Component > Forms basic > Required message is not display. · Issue   #671 · Wai-Technologies/raaghu-react (github.com)
671
Storybook
Reopen
Harshit

2
Storybook   > Components > Contact Us > Default > Displaying multiple error   messages at the one time. · Issue #659 · Wai-Technologies/raaghu-react   (github.com)
659
Storybook
Reopen
Harshit

3
Storybook   > Elements >Product Image >Column Layout> Alignment Issue. ·   Issue #489 · Wai-Technologies/raaghu-react (github.com)
489
Storybook
Closed
Harshit

4
Storybook   >Component > Billing Address >Default > After submitting field is   not clear. · Issue #655 · Wai-Technologies/raaghu-react (github.com)
655
Storybook
Closed
Suchithra

5
Storybook   >Charts>Scatter Chart>Docs> Scatter Chart icon does not display   proper. · Issue #522 · Wai-Technologies/raaghu-react (github.com)
522
Storybook
Reopen
Suchithra

6
Storybook   >Components >Claims > Default > Dropdown values are hidden not   display proper. · Issue #656 · Wai-Technologies/raaghu-react (github.com)
656
Storybook
Closed
Dhiraj

7
Design   System 7.6.20 > Issues of Elements, chart · Issue #625 ·   Wai-Technologies/raaghu-react (github.com)
625
Storybook
Closed
Prakash

8
Design   System > Implement Contributor Component in Storybook · Issue #529 ·   Wai-Technologies/raaghu-react (github.com)
529
Storybook
Closed
Akshay

Closed = 5
Reopen = 3


Allow custom HTTP methods to be used

Possible solution to #3.
Please suggest if we should be white-listing the methods that the plugin will accept. I'm not sure if that could be abused, but upon testing Firefox simply specified any text i entered as the method (not PUT, DELETE, etc), regardless if it existed or not.

Thanks for the pull request!


Use better analysis_options

include: package:flutter_lints/flutter.yaml

analyzer:
  errors:
    missing_required_param: error
    missing_return: error
    todo: ignore
    sdk_version_async_exported_from_core: ignore
  strong-mode:
    implicit-casts: false
    implicit-dynamic: false
  exclude:
    - '**.mocks.dart'
    - 'lib/util/license.dart'
    - 'test/coverage_helper_test.dart'

linter:
  rules:
    - unawaited_futures
    - always_declare_return_types
    - unsafe_html
    - use_full_hex_values_for_flutter_colors
    - prefer_final_in_for_each
    - prefer_final_fields
    - prefer_final_locals

Fixed in #4


build issue (master 3.05.01)

# apt-get install -y --force-yes autoconf automake libtool
# cd /var/bin && git clone https://github.com/tesseract-ocr/tesseract.git
# cd tesseract && ./autogen.sh

Running autoconf
Missing autoconf-archive. Check the build requirements.

  Something went wrong, bailing out!

Please install autoconf-archive and then rerun ./autogen again


Bayesian global lateralisation

This pull request integrates Bayesian posterior and SS data queries with global lateralisation and multiple semiology combinations, with 3 options to combine semiologies as instigated in pull Bayesian-correction #222
It also performs softmax on the proportions not raw values
That is, it addresses #224 #225 #221 and softmax #223


Should external C functions check their external object pointer for non-null?

There was an open discussion if the pointer of the external object passed to external C functions should be checked for non-null. This is not consistently considered troughout the library. We already figured out the three options

Do not check pointers.

Advantage: Short and clear code
Disadvantage: Apllication may crash with null-pointer access violation.

Silently accept if the pointer is non-null.

Advantage: Application no longer crashes with null-pointer access violation.
Disadvantage: There might be some kind of undefined behavior in case of a null-pointer.

Do error handling if pointer is null.

Advantage: Application no longer crashes with null-pointer access violation and no undefined behavior.
Disadvantage: Long code and not clear yet, if really necessary.

I am not sure if this actually is an MA issue. See mo:#1436 for a similar issue on character pointer arguments. The question on external object pointers was not yet asked and answered there.

I opt for 1.
One needs to trust that the pointer that the Modelica tool passes into the external C actually points to the expected external object. In general, if the Modelica tools passes in invalid pointers one is out of luck. Only in the special case, that a null pointer is passed, one has a chance to detect the error.  Therefore, I see a valid pointer as a precondition which needs to be guaranteed/checked by the Modelica tool, rather than by the external C code. Hence, my preference for trade-off 1.

Since it is different for other external objectes provided by the MA, I am going to ask at the Modelica Trac.

Seems we now have a mix of 1. and 2. at different places in the code.  Anyway, there are pros and cons for the different options and depending on what the code does one might lean towards a different option. For example, if one has a tool which (wrongly) calls a function with a null-pointer in some cases, but these cases are generally save to ignore, it is a practical approach to simply ignore it. Hence, I don't mind if this is not handled 100% the same way throughout the library.


sniffer-sql and localfile protocol

sniffer-sql is a variant of sniffer which dumps the messages into a SQL database using pyodbc (required python dependency).
Additionally, there is a new protocol to get the logs localfile:// which pulls messages out of a raw binary file. It should be a byte for byte copy of a serial stream.

Thanks for this contribution. Is FileStreamRange defined somewhere?

It isn't defined in this PR. One of the problems with processing one raw file at a time (besides performance loss due to opening and closing the database connection) is that we lose at least one message that is caught straddling 2 raw files. I implemented a FileStreamRange that takes a file pattern like NNNN where NNNN is an integer. The FileStreamRange takes a pattern like [n1,n2] and process a range of files from n1 to n2. It handles messages straddling 2 files and attempts to open the file n1-1 to synchronize and will pass the message straddling n1-1 and n1
I didn't include this FileStreamRange in the PR because it isn't absolutely required and I wanted your opinion about it.

From: Daniel @.>
Sent: Friday, July 29, 2022 7:25 AM
To: dulitz/finitude @.>
Cc: phdussud @.>; Author @.>
Subject: Re: [dulitz/finitude] sniffer-sql and localfile protocol (PR #6)
Thanks for this contribution. Is FileStreamRange defined somewhere?
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdulitz%2Ffinitude%2Fpull%2F6%23issuecomment-1199385415&data=05|01||53bfe2db558b465b593308da716e2a3d|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637947015210659236|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=MV4JmyyXMRj27g6987AGVfxRF2Ew%2F7JkARkhvvLHGiU%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABVQKXYYXY3Q4VDEK2QPNCDVWPSU7ANCNFSM54MIICLA&data=05|01||53bfe2db558b465b593308da716e2a3d|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637947015210659236|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=pSjIWhagw%2BztKuQc2CJfvF4rXbqrx3ZYRjK92a%2FkPkg%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

It isn't defined in this PR. One of the problems with processing one raw file at a time (besides performance loss due to opening and closing the database connection) is that we lose at least one message that is caught straddling 2 raw files. I implemented a FileStreamRange that takes a file pattern like NNNN where NNNN is an integer. The FileStreamRange takes a pattern like [n1,n2] and process a range of files from n1 to n2. It handles messages straddling 2 files and attempts to open the file n1-1 to synchronize and will pass the message straddling n1-1 and n1 I didn't include this FileStreamRange in the PR because it isn't absolutely required, and I wanted your opinion about it.

From: Daniel @.>
Sent: Friday, July 29, 2022 7:25 AM
To: dulitz/finitude @.>
Cc: phdussud @.>; Author @.>
Subject: Re: [dulitz/finitude] sniffer-sql and localfile protocol (PR #6)
Thanks for this contribution. Is FileStreamRange defined somewhere?
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdulitz%2Ffinitude%2Fpull%2F6%23issuecomment-1199385415&data=05|01||53bfe2db558b465b593308da716e2a3d|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637947015210659236|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=MV4JmyyXMRj27g6987AGVfxRF2Ew%2F7JkARkhvvLHGiU%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABVQKXYYXY3Q4VDEK2QPNCDVWPSU7ANCNFSM54MIICLA&data=05|01||53bfe2db558b465b593308da716e2a3d|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637947015210659236|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=pSjIWhagw%2BztKuQc2CJfvF4rXbqrx3ZYRjK92a%2FkPkg%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>


firmware: pkg upgrade before release type transition

When restoring a config with a development release type and pkg requiring an upgrade the underlying scripts seem to start a release type shift instead of updating pkg first. A similar issue exists when flavour and release type are out of sync. Release type should be switched last when the transition is safe.
Note to self: in this particular situation both release type and flavour were out of sync and there were stable updates to install (20.7 to 20.7.4/20.7.5).

Probably related to #4500

Merged into parent issue.


[Feature request] letsencrypt-companion

A companion service to the swarm-listener that would create certificates automatically would be great.  Something like:
https://github.com/JrCs/docker-letsencrypt-nginx-proxy-companion

See #56


refactor pyo3-ffi example to an example project

I HATE C'S INTEGER HIERARCHY
...but done 👍

@davidhewitt I did rework the exception handling, please merge if you approve 😄


[Fix-13248][UI] Ironed out the issue with failing to query task instances by some state of task.

Purpose of the pull request
This PR will close #13248 .

Brief change log
I found that the list of task state on the frontend is not aligned with those on the backend. So when querying the task instances by a wrong task state the system will notify the message that is "query task list paging error".

Verify this pull request
This change added tests and can be verified as follows:

Manually verified the change by testing locally.

The test result is as follows:

Codecov Report

Merging #13251 (81e52b2) into dev (3ecb4db) will increase coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #13251      +/-   ##
============================================
+ Coverage     39.40%   39.47%   +0.07%
+ Complexity     4316     4283      -33
============================================
  Files          1077     1072       -5
  Lines         40820    40506     -314
  Branches       4684     4654      -30
============================================
- Hits          16086    15991      -95
+ Misses        22940    22734     -206
+ Partials       1794     1781      -13

Impacted Files
Coverage Δ

...er/worker/config/TaskExecuteThreadsFullPolicy.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...lphinscheduler/plugin/task/api/utils/LogUtils.java
26.96% <0.00%> (-34.58%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...erver/master/processor/queue/TaskEventService.java
75.00% <0.00%> (-5.36%)
:arrow_down:

...worker/registry/WorkerConnectionStateListener.java
26.66% <0.00%> (-2.75%)
:arrow_down:

...eduler/api/dto/schedule/ScheduleCreateRequest.java
83.33% <0.00%> (-2.39%)
:arrow_down:

...org/apache/dolphinscheduler/remote/utils/Host.java
42.55% <0.00%> (-2.13%)
:arrow_down:

...e/dolphinscheduler/service/utils/ProcessUtils.java
25.60% <0.00%> (-2.05%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
51.38% <0.00%> (-1.39%)
:arrow_down:

...erver/worker/runner/WorkerTaskExecuteRunnable.java
37.25% <0.00%> (-1.26%)
:arrow_down:

... and 59 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Future

hey all, are there any plans to make the plugin fit for Java 9+ and to develop it further?
Best, Patric

Unfortunately no, work on the plugin is on hold.


Add new dependencies

rioxarray
psutil

Pretty sure you don't actually need psutil.  Can get cpu count from multiprocessing.  I think my branch had that modification.

PR #29 adds rioxarray to the environment.yml, which resolves this issue.
I removed the psutil dependency, as it isn't necessary.
Note psutil lets you easily switch between psutil.cpu_count(logical=True) (use virtual cores)  and psutil.cpu_count(logical=False) (use only physical cores) , which is convenient on shared resources. I primarily use it for this purpose, but there might be other/better ways that don't require additional dependancies. In any case, with nproc in main_experiment2.py the user has higher level control and can decide upstream.


GH-34333: [Python] Test run_query with a registered scalar UDF

Rationale for this change
Currently Acero has a way to execute a registered UDF via substrait however there are no tests for it.

What changes are included in this PR?
This PR adds a test for passing a registered UDF via a substrait plan.

Are these changes tested?
N/A

Are there any user-facing changes?
No

Closes: #34333

@westonpace I am not sure what this error is about:
==================================== ERRORS ====================================
_ ERROR collecting opt/conda/envs/arrow/lib/python3.7/site-packages/pyarrow/tests/test_udf.py _
ImportError while importing test module '/opt/conda/envs/arrow/lib/python3.7/site-packages/pyarrow/tests/test_udf.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
opt/conda/envs/arrow/lib/python3.7/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
opt/conda/envs/arrow/lib/python3.7/site-packages/pyarrow/tests/test_udf.py:22: in <module>
    import pyarrow._substrait
E   ModuleNotFoundError: No module named 'pyarrow._substrait'

from this:
https://github.com/apache/arrow/actions/runs/4295311322/jobs/7485550126
Any idea?

@westonpace Ok build is green now let me know if any changes you would like me make


DefaultLogTest.java:42-44: Handle NullPointerException...

The puzzle 36-6ed53b15 from #36 has to be resolved:
https://github.com/llorllale/loggit-maven-plugin/blob/bb2ac45dc2c0159fef324697fa8b76e1ee38a384/src/test/java/org/llorllale/mvn/plgn/loggit/DefaultLogTest.java#L42-L44
The puzzle was created by George Aristy on 26-Feb-18.
Estimate: 30 minutes, role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

Dup of #38


Fix JTC segfault

@swiz23 noticed a segfault in the JTC specifically when aborting a motion, followed by deactivating, followed by activating where the segfault would occur. After looking at this PR, I think it very much has to do with the line
output_state = trajectory_msgs::msg::JointTrajectoryPoint().
I think the flow of events occur as follows:
When aborting, set_hold_position() is called. That essentially adds and writes an empty trajectory message in order to disregard the current one.
When sample() is called in the update() loop, state_desired_ gets overwritten with that empty trajectory point meaning it has an empty position, velocity, and acceleration array.
The controllers get deactivated
The controllers get reactivated. When this happens, read_state_frome_hardware is called on state_desired_. Within that function, we try to index the position array within state_deisred_ as if it has the same size as dof_ when it in fact no longer does. This causes the segault.
The solution is to simply move the line defining the output_state within the sample() function in trajectory.cpp to after the last possible place that the function can return false -> here. I tested this out at least for the case that @swiz23 had issues with and verified that there was no longer a segfault.
This probably fixes #423
@JafarAbdi

Just double checked this and also verified that it fixes my issue.

@mergifyio backport to humble


Unable to load profile

Hi,
When I try to access my profile https://www.opendota.com/players/124964145 , I get stuck with a loading screen (screenshot below).

I am able to load other profiles normally, and have tried to load the profile when both when logged in and out from opendota. I have checked the box (and saved the setting) to expose public match data in the dota 2 client. For what it's worth, I've tried accessing this on both Chrome (version 60.0.3112.90) and Safari (version 10.1.1) on macOS Sierra, as well as on Chrome for Android.
Thanks

Console output

Thanks, this looks like an issue with a change to the mmr estimate data format.  Will push a fix on the API

#1123

I rolled out an API fix to prevent the crash, looks like the header is loading properly now.
@mbyakow also submitted a fix that handles this case and better defaults for players who no matches recorded.
I'll try to ship that to prod this weekend.


Getting the value of a pixel of an image

Hi, is there a way to get the value of a pixel by its coordinates?
I want to implement thresholding for annotations, and in order to compare nearby pixels against a threshold I need to probe their value in the image.
Thanks!

This is timely, as we have been working on new features. Please try out the drawing user interface demo.

Note that when you move the mouse cursor over a voxel, the voxel intensity is reported at the bottom of the screen.
You can try the Draw/Otsu feature that will that shows JavaScript for multi-otsu segmentation

Thank you! I am able to get the intensity of the current pixel this way.
Is there a way to access arbitrary pixel values by ijk coordinates? I want to compare neighboring pixels to the current one for implementing thresholding, but as far as I can tell this method only reports on the current pixel. Thanks again!

You can access all the image data directly. Regardless of how an image is stored on disk, the voxel data is loaded to memory as RAS. The code below also shows how to apply the NIfTI data scaling slope and intercept.
paste the code below into the scripting demo and press run script.
function voxelIJK(r,a,s) {
	//voxel intensity for given column (l->r), row (p->a) and slice (i->s)
	let hdr = nv1.volumes[0].hdr;
	let img = nv1.volumes[0].img;
	let dimsRAS = nv1.volumes[0].dimsRAS; //image always oriented as RAS
	//convert 0-indexed 3D ras to 1D array address
	let vx = r + a * dimsRAS[1] + s * dimsRAS[1] * dimsRAS[2];
	//raw intensity value (assumes scalar image, beware of rgba images)
	let raw = img[vx];
	//convert raw value to scaled value
	let cal = (raw * hdr.scl_slope) + hdr.scl_inter;
	console.log('dimensions', dimsRAS , 'intensity', cal);
	alert('intensity: '+ cal);
}
voxelIJK(103, 128, 112);

@neurolabusc, what are your thoughts on adding a new helper function to get the value at an arbitrary voxel location? Essentially, adding your example voxelIJK function to NiiVue. I can implement if you're onboard.

@hanayik I am fine with this, but this is a computationally inefficient if the developer wants to read a whole array of voxels. One might also want a variation (or have an overloaded argument) to choose between returning the raw value saved to disk versus the scaled calibrated value.

Does nv.volumes[0].getValue(x,y,z) offer this functionality?

@haehn yes. getValue() will provide the same result. In addition, getValue() can handle 4D data and provide a scalar intensity value for RGB images. I suspect that getValue()` is not very efficient if you want to read a large array of voxels.


Build from fct table

While running basically the same model in parallel to smooth the transition between models, I noticed that building the models from a fct_ga4__event_page_view model that I created was 40 times more efficient than from stg_ga4__event_page_view.

We build from stg_ga4__event_page_view in a number of places which are all opportunities for a performance improvement. This doesn't take into account caching, but I haven't noticed much of a cache effect from model to model (cache clearly helps when re-using fields within a single model).
However, I only have anecdotal evidence of the little cache re-use between models, so I'd like to see evidence in either respect.
I see us having two options here:

We make fct_ga4__event_page_view a core package model
We conditionally build from fct_ga4__event_page_view if it is present; otherwise falling back to stg_ga4__event_page_view

The first option seems simpler. It has the advantage that enabling and disabling package models is a pattern that we've used elsewhere, so it's not a big leap to expect users to disable the fct_ga4__event_page_view model. However, I find myself customizing this model a lot for each client so we'd be creating a model that almost always gets disabled.
The second option requires greater complexity in the package models. I suspect many people will create fct_ga4__event_page_view models that don't error without consulting any documentation and the errors messages should make it fairly clear what is missing from any fct_ga4__event_page_view model created by users.
Is this something that we should pursue?
Is caching more effective than I give it credit?
Do you have any preferences for either of these methods?

I think that our perspective on a lot of these things will change when materialized view support finally lands in dbt-bigquery (currently in the experimental plugin, but now being looked at for the core support plugin).
Right now we are kind of hamstrung by the half-implemented idea of supporting streaming data, right?

@dgitis sorry, I'm not following. Can you elaborate on your design of fct_ga4__event_page_view? Is it just a table materialization of stg_ga4__event_page_view?
And what was the SQL you were running where you saw an improvement? Without seeing that, it's hard to tell whether the improvement was due to the SQL itself (use of select *, etc) vs. the source of data.


how xr_quantile computes percentiles with overlap from different times and NaNs

Hi again! This is the continued discussion from https://github.com/opendatacube/odc-stac/issues/177. As mentioned in the other issue, I'm using HLS data (both HLSL30.v2.0 and HLSS30.v2.0 collections) for a short time frame (5-16-2024 through 5-18-2024) over a small region of interest in Washington state. I am currently using odc-geo==0.4.6, odc-stac==0.3.9, and odc-algo=0.2.2. We're generating composites for the 15th, 50th, and 90th percentiles. I've attached a jupyter notebook with a reproducible example.
As mentioned in the odc-stac issue, using a slightly different method, we anticipate the percentile composites to look something like this (the 50th percentile is shown here).

However, in using xr_quantile, we're getting different outputs. This is what the RGB of cloud masked input data looks like, one time slice per day.

This is what the 15th, 50th, and 90th percentiles look like when generated with xr_quantile. Given how much coverage there is with the input imagery over the three days, we would expect each percentile image to have lots of coverage with at least one valid value per pixel (as opposed to the widespread NaN values, which are mostly present in the 90th percentile).

Based on these results, I have several questions about the algorithm. Starting with a simple example, if there are 3 scenes that overlap for a particular pixel, each from a different day, a pixel could have up to 3 values. Let's say those three values are 100, 500, and 1000. With the way we compute percentiles, the 15th, 50th, and 90th percentiles would be 220, 500, and 900, respectively.

How does xr_quantile handle NaN values in the pixel value arrays? For the three day example, (nan, 500, 1000), np.percentile() would return all NaN, but np.nanpercentile() would return 15th=575, 50th=750, 90th=950. How would xr_quantile handle the (nan, 500, 1000) array?

Related to the first question, is this value what the nodata argument in xr_quantile is designed as an input for? For example, if I use a nodata=0, the array (nan, 500, 1000)  would become (0, 500, 1000), and the outputs would be 15th=150, 50th=500, 90th=900.

Do we have any idea what might be going on with the overlap at the different quantiles? As I mentioned, we would expect all three days worth of pixel values to be accounted for in each quantile calculation so that each image has coverage like quantile=0.15 does.

replication-code-odc-algo.zip
Question -- what's the best way to provide you with the sample dataset? It's too big to attach here.

Have you tried calculating the quantiles using Xarray's methods instead of the implementation of this package? Just make sure to also install the package Numbagg in order to speed things up. Otherwise you will run into issues that have been around for some while and probably led to this package's implementation in the first place...

Hey @ithaca-oyler, I'll try and have a look at this this week! Just to clarify, have you seen different results with respect to NaN handling from xr_quantile vs. the native .quantile() functionality from xarray? It would be great to do a side-by-side comparison to verify that the results you're seeing are different from that existing functionality (although .quantile() is extremely slow for large datasets given it loops through each pixel one-by-one).
(Just pinging the original algorithm developer @kieranricardo as well as @omad and @alexgleith as I believe they have been working with this code base recently too and may know if there's a more recent implementation that might avoid some of these issues)

Hey folks
There were a few nan bugs in the geomedian code, which I think have been resolved. That's where I'd look. Maybe it's sorting including nan, and nan is up the top, so the 90th percentile is nan.
I guess dropping nans and taking a percentile of that is the right approach, unless they're all nan... not sure though.
Some ref to changes: https://github.com/auspatious/datacube-compute/commit/4e032582557321c957237451dae35923d833a4e2#diff-f5e0ecb72dacb41ee8b0a629b23efb91d1a88c95b2c39272b48ade4830ec0938L61

Hi everyone! Thanks for taking a look at this issue. I have some more insights about my issue in particular.
It turns out by looking at three days, I made an edge case! I plotted the valid_counts data array created in np_percentile() and as shown here, the yellow stripe shows where there is an overlap in data across all three days. With the valid_counts input into keep_good_np() set to >=3, we were only getting outputs in the yellow stripe.

By setting valid_counts >=0, I got the expected outputs (the 15th percentile is shown here). I confirmed there is indeed a difference between the rasters by doing a raster subtraction between the 90th and 15th percentiles (the second image shown here). Where the image is brighter is also where the input imagery overlaps the most. For my use case, this likely won't be a big issue at scale, but it is an issue for small time frames and in areas with sparse imagery (<=3 scenes).

Now, curiously enough, the issue persists when the nodata value in xr_quantile is set to 0. I can understand how it would result in lower quantile values, but not 0s where valid_counts < 3.

In response to the comments above:

@maawoo Thanks for pointing me to Numbagg and the Xarray methods! I'll have to give them a shot to see if it speeds things up or changes things
@robbibt I haven't checked the NaN handling between xr_quantile and .quantile() with xarray yet, but I'll be sure to update you when I do the comparison.
@alexgleith Do you think this is also what could be happening when the nodata value is set to 0 in xr_quantile?


Incorrect coloring after complex var block

VSCode colors the following incorrectly:
var (
	vertexData = gblob.LittleEndianBlock(make([]byte, len(geometry.vertices)*int(stride)))
)

if 5 > 10 {

}

The if keyword is light blue instead of the standard purple.
This is a redirect of an issue opened originally to gopls. Please check issue for more detailed information if needed:
https://github.com/golang/vscode-go/issues/3370

Thanks for the details! This bug should be fixed in v0.6.9.
v0.6.9

Can you please install the latest version of go-syntax and try it?

Yes, with the latest version, after a manual install, it is no longer reproduced.
However, I am confused. I don't have this plugin installed by default. Does this mean that it ships with VSCode or with the Go plugin? Do I need to wait for those plugins to pull the latest version of this one for it to work by default (without manual install)?

Go Syntax became the default Go grammar in the official VS Code in v1.86 (release note).
I've published a new version (v0.6.9) to fix this issue today. VS Code updates the grammars once a month, and most likely this fix will be available in the stable VS Code next month.
Think about this extension as a pre-release, nightly version of the default VS Code Go grammar. You can either use this extension to have the quick fix, or you can wait until VS Code updates the grammars.

Thanks for the quick update. It makes sense now.
I am closing this issue. Thanks, again!


Can i build static library with 'go build' for android platform?

When i tried to build a static library with 'go build' tools, i got this error:

command-line-arguments
invalid value "c-archive" for flag -buildmode: buildmode c-archive not supported on android/arm
.....
it`s a cross-compile Project, my environment is :
export GOOS=android
export GOARCH=arm
export GOARM=7
export CGO_ENABLED=1
export CC=/app/go-hoofoo/android-ndk-r20/toolchains/llvm/prebuilt/linux-x86_64/bin/armv7a-linux-androideabi28-clang
export CXX=/app/go-hoofoo/android-ndk-r20/toolchains/llvm/prebuilt/linux-x86_64/bin/armv7a-linux-androideabi28-clang++

the compile command was: go build -buildmode=c-archive -o libfoo.a  *.go

Can any body help me ?

Please see https://github.com/golang/go/wiki/Questions.


Topic name length

Currently topic names for annotators are limited to 32 characters (see here)
Kafka allows topic names up to 255 characters

And here it is limited to 16. We should either set the type to bigstring and enforce a limit of 80 chars or set the type to text and the limit to 255.


add fog action

Thanks for the feedback!
On Mon, Jul 1, 2024 at 5:47 PM vercel[bot] @.***> wrote:

The latest updates on your projects. Learn more about Vercel for Git ↗
https://vercel.link/github-learn-more
Name Status Preview Comments Updated (UTC)
three-js-journey ✅ Ready (Inspect
https://vercel.com/gitrexdevs-projects/three-js-journey/7TJGJDitoV7EYQDb8PTNdvau4rM6
) Visit Preview
https://vercel.live/open-feedback/three-js-journey-git-dev-gitrexdevs-projects.vercel.app?via=pr-comment-visit-preview-link&passThrough=1 💬
Add feedback
https://vercel.live/open-feedback/three-js-journey-git-dev-gitrexdevs-projects.vercel.app?via=pr-comment-feedback-link Jul
1, 2024 3:46pm
—
Reply to this email directly, view it on GitHub
https://github.com/winer328/threeJs-journey/pull/2#issuecomment-2200500260,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHNCKMGFAVEXCTAPXQSXAETZKF2X5AVCNFSM6AAAAABKF26OWKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBQGUYDAMRWGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Investigate lowering of GENX_FpToFpOp without GenISA

There is this code:
tatic Value convertFp32ToFp16(Location loc,
                                 ConversionPatternRewriter &rewriter,
                                 const Value &v, const RoundingMode rounding,
                                 const Target &target) {
    switch (target) {
    case mlir::triton::NVVM: {
      PTXBuilder builder;
      StringRef ptx;
      switch (rounding) {
      case RoundingMode::RTNE:
        ptx = "cvt.rn.f16.f32";
        break;
      case RoundingMode::RTZ:
        ptx = "cvt.rz.f16.f32";
        break;
      default:
        llvm::errs() << "WARNING: unsupported rounding mode for f32->f16 "
                        "conversion: "
                     << stringifyRoundingMode(rounding) << "\n";
        llvm_unreachable("");
      }

Which we aren't lowering correctly now because we fail to use the same rounding mode as NVIDIA.  To port this code faithfully we will need to set the rounding mode, which is not possible via llvm::FPTruncOp.

Which we aren't lowering correctly now because we fail to use the same rounding mode as NVIDIA.

Fixed in https://github.com/intel/intel-xpu-backend-for-triton/pull/397.

Next experiment is to use https://llvm.org/docs/LangRef.html#constrained-floating-point-intrinsics.

Next experiment is to use https://llvm.org/docs/LangRef.html#constrained-floating-point-intrinsics.

Created https://github.com/intel/llvm/pull/12588 to use https://llvm.org/docs/LangRef.html#constrained-floating-point-intrinsics.

The PR is merged, we can no longer generating GenISA for fp to fp conversion.


9_Combined_AspId_and_EFStorage - Build Error

Please only use the issue tracker for bug reports and/or feature requests. For general security questions, or free or commercial support options do not use the issue tracker and instead see here for more details.
For bug reports,  include the relevant log files related to your issue. See here how to enable logging. Delete this line once you have.
Finally, please keep the issue concise and to the point. If you paste in more code than the text for the issue you are reporting then we will most likely not read it.
Issue / Steps to reproduce the problem

Cloned the repository
Opened IdentityServer4\samples\Quickstarts\9_Combined_AspId_and_EFStorage\Combined.sln in Visual Studio 2019.
Re-built the solution. Got build errors.
Tried also NuGet package restore. Got build errors.

See image:

Relevant parts of the log file
<log goes here>

perhaps a clone issue. it builds for me.

I tried cloning again, and got build issue again, so I tried to see what could be the possible causes.
I also tried to open up another solution, IdentityServer4.Core.sln, and I get the same error as for the sample project, so it seems I cannot build any solution from IdentityServer4...
When trying to do nuget restore: Error occurred while restoring NuGet packages: The local source 'D:\GitHub\IdentityServer\IdentityServer4\nuget' doesn't exist.

UPDATE: The issue was caused by NuGet localsource, I found the solution here: https://stackoverflow.com/questions/49446394/unable-to-find-nuget-local-source-error-occurred-while-restoring-nuget-packages
So I did the following, in case it occurs to someone else in the future:

Tools > Options > NuGet Package Manager
I saw that inside Available sources "IdentityServer Local" was referencing \IdentityServer4./nuget
So I changed it to \IdentityServer4

Now the nuget packages are able to be restored and now the build works.


Ping block explorer APIs as a CI step

Or even better, (re)verify a contract to check if contract verification works, and omit the API information if it doesn't. This would prevent these kind of errors.

Might need some ideas for how to do this reliably for all chains.
As a first attempt, I tried to simply ping the chain.explorer.api.url with a HEAD request (aka a GET but only returns the headers) and then check that the status value is < 400. This works for maybe 50% for the existing chains, but for the rest, they return a 400 error. I think most are some kind of Etherscan API fork and seem to require module and action parameters with the request. So I then tried adding these basic query string params to all requests:
{
  module: 'account',
  action: 'eth_get_balance',
  address: '0x0000000000000000000000000000000000000000'
}

This seems to work for all chains except boba-bnb and boba-ethereum. I don't yet know if there is some kind of parameter combination that would work for all chain APIs.
I'm happy to try the contract (re)verification, but not quite sure what I would need to add to each chain object to allow this and how each chain would be different. e.g. what contract are we verifying?
From the Etherscan docs, it seems like there is quite a lot of data required to do this


[Spanner] requestTimeout option not working as expected

We expected requestTimeout to terminate the request when it exceeds the set time but when we tried to get thousands of rows at once, the request did not timeout and waited for the results to return.
Environment details

OS: Alpine Linux 3.10.1
PHP version: 7.2.20
Package name and version: googleapis/google-cloud-php-spanner 1.17.0

Steps to reproduce

create a table and insert millions of rows.
set $client = new SpannerClient(['requestTimeout' => 1]);.
send SQL statement which returns thousands of rows.
php times out after 30 seconds. (seems like it was waiting for a response on Google\ApiCore\ServerStream->readAll())

Code example
$client = new SpannerClient(['requestTimeout' => 1]);
$db = $client->connect($instanceId, $databaseId);
$db->execute('SELECT * from LargeTable')->rows()->current();

gRPC version we are using is v1.20.
We will try to reproduce this with the latest version.

@dwsupplee Hi, I investigated more deeply. As a result, it turned out that \Google\ApiCore\Serializer::encodeMessage was spending a lot of time, not gRPC communication.
For example, if running LIMIT 1000 query, \Google\ApiCore\Serializer::encodeElement is called more than 1000 times, even if only the first row is fetched by calling Generator::current().
Code example
require __DIR__.'/vendor/autoload.php';

use Google\Cloud\Spanner\SpannerClient;
use OpenCensus\Trace\Tracer;

opencensus_trace_method(\Google\ApiCore\ServerStream::class, 'readAll');
opencensus_trace_method(\Grpc\Call::class, 'startBatch');
opencensus_trace_method(\Generator::class, 'current');
opencensus_trace_method(\Google\ApiCore\Serializer::class, 'encodeMessage');
opencensus_trace_method(\Google\ApiCore\Serializer::class, 'encodeElement');

$exporter = new \OpenCensus\Trace\Exporter\OneLineEchoExporter();
Tracer::start($exporter);

$instanceId = 'xxx';
$databaseId = 'xxx';

$client = new SpannerClient(['requestTimeout' => 1]);
$db = $client->connect($instanceId, $databaseId);

$row = $db->execute('SELECT * from LargeTable LIMIT 1000')->rows()->current();

Output
[  5409.30 ms] /test.php
  [http.status_code] 200
  [http.host] localhost:8080
  [http.port] 80
  [http.method] GET
  [http.path] /test.php
  [http.user_agent] curl/7.54.0
[   258.49 ms] Grpc\Call::startBatch
[   744.52 ms] Grpc\Call::startBatch
[    12.41 ms] Google\ApiCore\Serializer::encodeMessage
[     2.96 ms] Google\ApiCore\Serializer::encodeElement
[     0.81 ms] Google\ApiCore\Serializer::encodeElement
[     0.04 ms] Google\ApiCore\Serializer::encodeElement
[     0.04 ms] Google\ApiCore\Serializer::encodeElement
[     0.71 ms] Google\ApiCore\Serializer::encodeElement
[     0.04 ms] Google\ApiCore\Serializer::encodeElement
[     0.04 ms] Google\ApiCore\Serializer::encodeElement
[  3461.13 ms] Generator::current
[     0.39 ms] Grpc\Call::startBatch
[     0.01 ms] Google\ApiCore\ServerStream::readAll
[    24.62 ms] Google\ApiCore\ServerStream::readAll
[    22.09 ms] Grpc\Call::startBatch
[  3398.86 ms] Google\ApiCore\Serializer::encodeMessage
[     5.23 ms] Google\ApiCore\Serializer::encodeElement
[     3.82 ms] Google\ApiCore\Serializer::encodeElement
[     1.70 ms] Google\ApiCore\Serializer::encodeElement
[     0.04 ms] Google\ApiCore\Serializer::encodeElement
[     0.87 ms] Google\ApiCore\Serializer::encodeElement
[     0.05 ms] Google\ApiCore\Serializer::encodeElement
[     1.49 ms] Google\ApiCore\Serializer::encodeElement
[     0.04 ms] Google\ApiCore\Serializer::encodeElement

... ( more than 1000 times)

[     0.01 ms] Generator::current
[     6.32 ms] Google\ApiCore\ServerStream::readAll
[     0.03 ms] Grpc\Call::startBatch
[     0.02 ms] Grpc\Call::startBatch

@taka-oyama & @castaneai,
I'm still having trouble replicating this issue as described - would it be possible to share a sample set of data in a similar shape to your use case?
In regards to the large number of encodeElement calls - the first response the library receives from the streaming call is a PartialResultSet. This response can contain a large number of rows (in some cases even the entire expected result set), depending on how large each row is. We have a centralized point in our request/response cycle which encodes responses from the API into a common format and then passes this response off to the client library code. It is at this point the client library code takes that response (yielded to us from the generator provided by the gRPC call) and processes each individual row, then yields them up to the user. This means there's a good chance all expected rows will actually be encoded up front as opposed to one by one as you yield them in user land code. This is something we have plans to improve upon, most likely as part of a 2.0 release for Spanner (as there is a good chance we would need to include some breaking changes to improve the workflow). In the meantime, is it possible to limit the scope of your query to information you plan to utilize as part of the PHP request?

@dwsupplee Thank you for your investigating.
We found this problem while investigating other different problems.
In a real use case, it is rare to execute such a large table SELECT.
Anyway, it is desirable to be able to set a timeout that includes an encodeElement in our use case.

I watched how the requestTimeout parameter is passed.
The requestTimeout value specified when creating the SpannerClient is stored in the $requestTimeout field of the RequestWrapperTrait trait of the GrpcRequestWrapper() object
The SpannerClient constructor creates a connection using the Grpc class in which GrpcRequestWrapper is created. The $config parameter containing requestTimeout is successfully passed along this chain.
In the process Database::execute('SELECT * from LargeTable'), GrpcRequestWrapper->send is called in which requestTimeout is converted to milliseconds and, under the name noRetriesRpcTimeoutMillis, participates in the request stack through the gRPC module generated in GapicClientTrait->createCallStack

In my opinion, noRetriesRpcTimeoutMillis comes to calls implemented in the gRPC module, but it ignores this parameter.
Configuration: Ubuntu 18.10; php 7.2.19; gRPC version: 1.22.1

@dwsupplee I saw that when sending a request an array of $optionalArgs is created and filled, but it is not involved anywhere. Maybe this is precisely the reason for the problem that you need to pass this array as a parameter to $backoff->execute (..)?
On the other hand, when I execute code similar to the one in the first post, I get an Exception with status = "DEADLINE_EXCEEDED"

If I execute with a big restriction, for example 'requestTimeout' => 7
then I get the expected response (see PHP-code on screen):

===========
Hello World

Need to check on any large table if the problem is resolved?


clang-tidy-sarif: 0.6.6 -> 0.7.0

Changelog: https://github.com/psastras/sarif-rs/releases/tag/clippy-sarif-v0.7.0
Diff: https://github.com/psastras/sarif-rs/compare/clippy-sarif-v0.6.6...clippy-sarif-v0.7.0

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build clang-tidy-sarif clang-tidy-sarif.passthru.tests


chore: Use published docker image

I have locally built and published docker image to docker hub. So now locally and in CI we can pull it from there.
For now, I have decided to remove build action from workflow. If we want to automate some of this process, we can do that later, but now it seems we are updating dockerfile pretty rarely. So when that happens, we can manually publish it again (need acc to do that, so @FlyersPh9 might need to learn 3 commands to run).

You should have just ignored your local header test failures.

I think we need two versions of this image pushed to docker. It fails on mac

We can publish a multi-arch image! In bcb2e5a (#571), I changed the approach to detect the platform in a script. This was done to reduce the time of build. But if we are publishing the image, we should use --platform.


use fixed version of proto instead of master branch (#843)

cherry-pick to 2.1 branch.

/run-all-tests

Codecov Report

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

@@          Coverage Diff           @@
##             2.1     #850   +/-   ##
======================================
  Coverage       ?   53.43%
  Complexity     ?     1105
======================================
  Files          ?      150
  Lines          ?     6964
  Branches       ?      812
======================================
  Hits           ?     3721
  Misses         ?     2879
  Partials       ?      364

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 4b7a0a6...8bd16dc. Read the comment docs.


SampledProperty.setInterpolationOptions does not properly ignore undefined

The implementation does not match the documentation.  This makes it impossible to load CZML that relies on the documented defaults for algorithm / degree (LINEAR / 1)
failing test case:
var property = new Cesium.SampledProperty(Number);

var interpolationAlgorithm = property.interpolationAlgorithm;
var interpolationDegree = property.interpolationDegree;

property.setInterpolationOptions({});

expect(property.interpolationAlgorithm).toEqual(interpolationAlgorithm);
expect(property.interpolationDegree).toEqual(interpolationDegree);

Other issues in this class:

setInterpolationOptions should treat options as optional, as indicated in the doc for the function.
The example code incorrectly uses backticks for strings instead of single quotes.


Assertion is not Verification

May as well hard code a final result, rather than pretend to do a calculation based on a supposed supply from the yellow pages.
https://github.com/lastmjs/eth-total-supply/blob/10fa8a52dce617548553d9653a4b56062bf4ba38/calculate-block-reward-issuance.ts#L101

That would be 3 ether in big number motion? Or am I missing something?

@androolloyd the values are in WEI, which is 10^18 ETH.
@JoelBondurant The base block rewards are hard-coded into each client, there is no other way to determine issuance (determining supply by summing account balances is another method of determining supply).
That function has documented multiple locations where you can verify the numbers that each client is using to issue ETH. Here they are:
/**
 * Returns the base block reward for a block based on block number.
 * The rewards started at 5 ETH per block, and have been changed twice with hard forks.
 * These rewards can be verified in the Ethereum client implementations.
 * Some of these resources are included below:
 * Geth block reward definitions: https://github.com/ethereum/go-ethereum/blob/master/consensus/ethash/consensus.go#L40
 * Geth hard fork block number definitions: https://github.com/ethereum/go-ethereum/blob/master/params/config.go#L55
 * Open Ethereum block reward and hard fork block number definitions: https://github.com/openethereum/openethereum/blob/master/ethcore/res/ethereum/foundation.json, look at engine.Ethash.params.blockReward
 * Nethermind block reward and hard fork block number definitions: https://github.com/NethermindEth/nethermind/blob/master/src/Nethermind/Chains/foundation.json, look at engine.Ethash.params.blockReward
 */

I'm not sure how this could be improved, do you have any suggestions?

As far as I can tell, the Yellow Paper does not mention the fact that the baseReward changes depending on the fork. It's totally correct to say it does, and the code clearly works this way, but the Yellow Paper doesn't say that. See Equation 116. It would probably be beneficial to submit an issue report to the Yellow Paper. Unless I'm missing something. I'll submit it, if someone here can confirm my suspicion.

@JoelBondurant Totally agree that Assertion is not Verification. See https://github.com/Great-Hill-Corporation/trueblocks-core/blob/develop/src/other/issuance/README.md. This issue should be closed.

Closing, the scope of Ethereum is limited to assertions and not validations such as summing account balances.

Obviously, you've not been reading the work, as we show that we do actually verify the balances of every miner and every uncle miner at every block, and in every case it reconciles to the expected behavior as per the Yellow Paper. Happy to see this closed, though.

Plenty to follow here, empty values as expected per scope:
https://github.com/lastmjs/eth-total-supply#total-eth-account-balances-1

We've had our first values added: https://github.com/lastmjs/eth-total-supply#total-eth-supply


Every track liked on last.FM when listening to a playlist on YouTube Music with some liked songs

OS: Win10 1909 18363
GPMDP Version: 4.7.1 (date modified: 01/08/20)
Issue Descriptions:
Listening to one of my own playlists on YouTube Music now. Every song scrobbled to Last.FM is being marked as liked. See the below screenshots.
The playlist:

The songs on YT Music:

LastFM scrobbles:

Steps to Reproduce:
It seems to happen after a liked song is played. The liked song is correctly reported as liked to last.fm, but every song after that is also reported as liked, even though it's not.

I also had this problem on Fedora 32


openai version

May I know which version of openai is used for training? I wonder if which version provides openai_object since the latest one does not support it.
Thank you!

0.27.1 works


Drop support for Swift 5.2 and 5.3

As outlined in a Swift forums post in November ’21, SwiftNIO will only support the latest non-patch Swift release and the 2 immediately prior non-patch versions.
In this commit we drop support for Swift 5.2 and 5.3. We update CI for Swift 5.4 to run on bionic instead of focal to ensure that we still test bionic. Also we use the new SPM API break checker.

@yim-lee Could you please disable the CI for 5.2 and 5.3 and change the CI file for 5.4 from docker/docker-compose.2004.54.yaml to docker/docker-compose.1804.54.yaml.

CI and branch rules updated.
@swift-server-bot test this please

Nightly and 5.7 fail because of Sendable requirements.


feat(AlgebraicGeometry/ResidueField): classification of Spec K ⟶ X with K a field

This PR adds more API for residue fields and in particular the classification of morphisms Spec K ⟶ X with K a field.
From the valuative criterion project.
Co-authored-by: Qi Ge
Co-authored-by: Andrew Yang

[x] depends on: #15240

And maybe wait for #17403 before merging this

I agree. In the meantime, I have untangled the imports because of the large-import label and also because I think it is reasonable that ResidueField does not import ClosedImmersion but rather the other way round.

Please merge master and incorporate the new changes. Thanks!

LGTM other than these minor comments
But I think I wrote most of the statements so I'll refrain from maintainer-merging.

Thanks!
bors merge


Create userthemes error?

Hello
I tried to create a user theme from "theme creator".
After creating it, I used it from "Manage themes", but "Warnning" was displayed.
Warning: filemtime(): stat failed for plugins/Customizer/Assets/css/userthemes/test.css in /var/www/html/kanboard/app/Helper/AssetHelper.php on line 37
And that was not reflected.
Also, the user theme did not exist in the "plugins/Customizer/Assets/css/userthemes/" directory.
Is there any solution?
Thanks a lot
Best regards

Most likely, permission issues. Try manually creating the folder plugins/Customizer/Assets/css/userthemes
And see if that works

As you said, it was a permission issue.
It was solved by making the following changes.
Thank you so, so much!
[before]
drwxr-xr-x 2 root root ... **userthemes**

[after]
drwxrwxrwx 2 root root ... **userthemes**

👍


Build failure: python312Packages.torchrl

Fixed by https://github.com/NixOS/nixpkgs/issues/310608
Steps To Reproduce
Steps to reproduce the behavior:

nix build .#python312Packages.torchrl, where . contains the current nixpkgs master.

Build log

Build Log
error:
       … while evaluating the attribute 'drvPath'
         at /nix/store/bbinibbz7s91wnyldl9vda2wc1p5sbjx-source/lib/customisation.nix:365:7:
          364|     in commonAttrs // {
          365|       drvPath = assert condition; drv.drvPath;
             |       ^
          366|       outPath = assert condition; drv.outPath;

       … while evaluating the attribute 'drvPath'
         at /nix/store/bbinibbz7s91wnyldl9vda2wc1p5sbjx-source/lib/customisation.nix:365:7:
          364|     in commonAttrs // {
          365|       drvPath = assert condition; drv.drvPath;
             |       ^
          366|       outPath = assert condition; drv.outPath;

       … while calling the 'derivationStrict' builtin
         at <nix/derivation-internal.nix>:34:12:
           33|
           34|   strict = derivationStrict drvAttrs;
             |            ^
           35|

       (stack trace truncated; use '--show-trace' to show the full, detailed trace)

       error: tensorflow-2.13.0 not supported for interpreter python3.12

Additional context

See https://github.com/NixOS/nixpkgs/issues/345475 and https://github.com/NixOS/nixpkgs/issues/327140 for similar issues.
Metadata

system: "x86_64-linux"
host os: Linux 6.11.7, NixOS, 24.11 (Vicuna), 24.11.19700101.dirty
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.24.10
nixpkgs: /nix/store/1b3nl1m5dgb1866c6nvwvwk2i5kjs27z-source

Notify maintainers

@GaetanLepage

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Looks like tensorflow was updated to point to tensorflow-bin (https://github.com/NixOS/nixpkgs/pull/345658). This no longer has a build failure.


The timezone at the Linux Pop_OS is not respected

I am from Brazil and my system time is 5:50 AM and the time at the messages shows 8:48 AM.
It's a little issue and I love de app.

Hi, what language do you see whatsapp in? Is it the same you see on the web? (if it's English, most likely there is no whatsapp version in your language)
Sorry for the delay :(

Duplicate #13


vim plugin support in https://github.com/pandysong/ghost-text.vim

I have written a plugin in python3 and vim's channel feature. Please consider to add to the list in the front-page:
https://github.com/pandysong/ghost-text.vim

There’s already a vim script, does this differ?

sure. PR created:
https://github.com/GhostText/GhostText/pull/144

Thanks.


Tabular: Added automatic boolean conversion

Issue #, if available:
Description of changes:

Automatically convert features with only 2 unique values to boolean
Drop boolean features for KNN, speeds up KNN by 100x+ without loss of quality on datasets such as CoverType.

TODO:

[x] Benchmark

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-1286-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1286/1/index.html

Benchmark results show improvement (errors are server-side and can be ignored)
                                  framework  > AutoGluon_bestquality_1h_2021_08_22_bool  < AutoGluon_bestquality_1h_2021_08_22_bool  = AutoGluon_bestquality_1h_2021_08_22_bool  % Less Avg. Errors Than AutoGluon_bestquality_1h_2021_08_22_bool  time_train_s  metric_error  time_infer_s  loss_rescaled  time_train_s_rescaled  time_infer_s_rescaled      rank  rank=1_count  rank=2_count  rank=3_count  rank>3_count  error_count
0  AutoGluon_bestquality_1h_2021_08_22_bool                                           0                                           0                                        1032                                           0.000000                  3379.353981  61601.164201     40.616505       0.477670               1.014899               1.165075  1.492233           497           535             0             0            8
1       AutoGluon_bestquality_1h_2021_08_14                                         479                                         495                                          56                                          -0.750196                  3382.340874  62252.525237     39.666117       0.486408               1.016602               1.112381  1.507767           487           551             0             0            2

Job PR-1286-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1286/2/index.html


Uncaught TypeError: The "original" argument must be of type Function

Environment
Knex version:0.95.4
Database + version:pg+
OS:win10 + chrome  89.0.4389.114
Bug

Can't initialize knex. In fact, when you open site http://knexjs.org, the console will print this error too.

Error message

util.js?3022:602 Uncaught TypeError: The "original" argument must be of type Function
    at promisify (util.js?3022:602)
    at eval (fs.js?eba2:8)
    at Object../node_modules/knex/lib/migrations/util/fs.js (chunk-vendors.js:6026)
    at __webpack_require__ (main.js:849)
    at fn (main.js:151)
    at eval (template.js?8f9c:3)
    at Object../node_modules/knex/lib/migrations/util/template.js (chunk-vendors.js:6059)
    at __webpack_require__ (main.js:849)
    at fn (main.js:151)
    at Object.eval (MigrationGenerator.js?0437:2)

Just simple code:

const knex = require('knex')({
  client: 'pg',
  version: '7.2',
  connection: {
    host : '<IP_ADDRESS>',
    user : 'your_database_user',
    password : 'your_database_password',
    database : 'myapp_test'
  }
});

I doubt it is webpack related because when I run node test.js, everything is OK.

Did you ever resolve this? I'm currently experiencing this. I'm wondering if it's because I've empty module resolved util in my webpack config.

I have a repro of this issue here: https://github.com/dwjohnston/knex-webpack-issue/tree/main/take3


Support 0<data_fraction<1 for CustomDatasetWithoutLabels

This pull request provides a fix to issue  #327.

@kaland313 thanks for spotting this. Looking at the fix, it seems fine.


Why use 10000 as the base of the log function?

https://github.com/lucidrains/denoising-diffusion-pytorch/blob/f264a1eca1ee5a2a28a2859e3dc871d275e9c602/denoising_diffusion_pytorch/denoising_diffusion_pytorch.py#L119
Hi, thanks for your great works.
While I'm looking into your codes, I found the base of the math.log function is 10000. May I ask why 10000 is used instead of other numbers?

@bugsuse that was faithfully transcribed from Ho's original implementation
10000 probably because that is what is used in language modeling

Got it. Thanks a lot!


Topic: Strong password validation

Describe the types of topics you'd add in our Website
The password check in your website is only of 6 character check which is not a strong password validation. Please assign me this.
Could you share some screenshots or a video of your previous work?
I implemented the generate password feature in this project.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC-24-extd contributor
[X] Soon, I'll complete it and create a Pull Request.

Hi @aryadevesh! Thanks for opening this issue on Ratna Supermarket. We appreciate your contribution to this open-source project. We aim to respond or assign your issue as soon as possible.


Retain type of collection

Retain the type of the collection passed to deepHydrate().

Great job. Thank you


Added parseFloat in goog.string.format

toFixed doesn't work without parseFloat

@lomonosv You need to sign a CLA before we can merge this PR.

Thanks for your answer. I signed a CLA.

@lomonosv This PR now has 12 commits instead of 1. I think you may need to rebase onto HEAD?

Please update this PR. Right now it is not mergeable.

merged with minor edits


Possibly split out /app (default tenant) into a separate namespace called pooled_tenant - and treat it like a normal tenant in terms of application (re-)deployment

I like this idea. We won't implement it yet, but would accept your PR sometime if you're interesting in contributing ;)


Red Pictures

Hello,
This is a really cool project.   I have a few ideas in my that this will be very helpful for.   I was playing around with the project the other day and have run into a problem.    I understand that you are by no means required to help with troubleshooting and it is most likely user error on my part.  I was trying to use the StreetViewImageLoader and all the images that it was pulling where just red.
Thank you

Turns out I have no idea what I am doing and my API had just expired.
Thank you again for the cool tool.


Adaptation to use embbeded JRE in bundled applications made with Max

I adapted the a little the code from Peter, in order to be able to merge on the latest commit in master.
I adapted the code so now, JRE 8+ are accepted, by setting proper libjli.dylib path.
This enables:
Max 8 beta runtime to find and use JRE 8+ stored in max-mxj package directory
(Max 8 doesn't search for java outside of mxj, so its fine to detect JRE)

This doesn't enable:
Use of old JRE 6 as embedded JRE. But it doesn't change the behaviour of mxj when nothing is embedded.

Comments from Peter Slack (I used his merge request as a base to start mine), still valid:
I've added an option to embedd a jre directly in the max-mxj packages file for OSX
the directory jre should be placed in max-mxj package directory
it should point to the base of an installed jre or copy the jre into this directory
the option looks for jre/Contents/Home to exist
this only works for 64 bit mode, there is no 32 bit embedded option for OSX
I would also like to merge in the changes from the latest mxj and windows embedd option will follow
It has been tested on OSX and Windows, with Java 8 and various OpenJDK / OpenJRE up to 15.

This seems to be working on my local install to the extent that I can test it. I'll need to do some additional testing on Windows. If there is no embedded JRE, where does the object search exactly?

I'm unsure about removing 32 bit version from Windows if that was a thing Cycling has done, it makes sense to remove 32 bit but not sure if it makes sense to remove it here unless it was removed from production.

I'm not concerned about 32-bit support. Since 32-bit versions apparently don't support embedding at all, users can grab old versions of the package if they need 32-bit binaries. I'm much more concerned about making real Cmake projects and getting arm64/universal 64-bit compiles working (and solving any arm64 problems) at this point.

I feel there are top level directions from Cycling engineering that are missing or need to be updated : what is the minimum target (ie win 8 or what OSX version etc), what are the supported platforms (x86 x86_64, arm64)? With this many changes I feel there also needs to be an explicit test matrix so we can cover all the permutations besides what are the minimum production specs from Cycling. In the current state we covered 32 and 64 bit platforms for both OSX and Windows and only JDK, today those specifications are different but I feel it isn't up to contributors like myself to decide. There are some specific tests that were done, for example use with clojure to ensure it meets requirements.

Oh sorry, that was unclear from me. I don't expect that this pull request satisfies any of my engineering requirements, merely that it implements the embedding feature which has been a pain point for users, especially as Java is removed from the OS on the macOS side.
I'm planning to do the leg work on what is to follow (real Cmake project, universal/arm64 support, etc.) and I'm totally happy with the state of the work to this point.

OK, I've kicked the tires and am comfortable with these changes, which appear to be working properly. I'm prepared to merge this to master once we clear up the usage of -XstartOnFirstThread. I have a branch based on this work which adds true Cmake support, and then a branch based on that branch which adds arm64 support, so merging this is a first step to that bit of progress.


Add enableReinitialize to the context

In some component down the tree in my app I tried to check if enableReinitialize  was turned on. However, its value is always undefined. I think it should be added to the formik bag because there could be scenarios where it's needed. Nevertheless, the typings would make you believe it's accessible but it's actually not.

Oh a custom reinitialization function? That'd be cool! Well, if this might set the wrong precedent, then I understand you won't add it in. I've found another way to know if the enableReinitialize has been set, so it's not much of a problem. The only thing that got my mind a little twisted is that the types would make you believe it's available. Also, I do think it could be handy to see whether enableReinitialize has been set to true, or in the future: the reinitialization function has been set.


Add test for failed app install

This test ensures that an invalid helm install fails correctly
and does not result in tiller continually running.
Validates https://github.com/rancher/rancher/issues/18571

Wait to merge until https://github.com/rancher/rancher/pull/19079

LGTM from me
@sangeethah please review - per @daxmc99 this test requires a functioning cluster so was done in validation tests.

Added flake8 formatting suggestions


Issue Setting up a cache

The API works great!!! Unfortunately while setting up caching, I consistently get this error:
File "C:\Users\{USER}\anaconda3\envs\apps\lib\site-packages\beckett\clients.py", line 194, in __init__
    super(BaseClient, self).__init__(*args, **kwargs)
TypeError: object.__init__() takes exactly one argument (the instance to initialize)

I am running python: 3.8.12 on a windows 10 machine, with Pokepy version 0.4.
Another small issue is for whatever reason everytime I get a return from the client it is in a list instead of the pokepy resource class (as shown in the documentation). Any ideas?

Hey @dsambrano,
latest version of pokepy is 0.6.2. Try using the latest version and see if your issues are addressed

Thanks you the advice. Unfortunately, I get a Resolution Error when trying to install that specific version (or anything about .0.6 for that matter)
Namely because the beckett 0.8 dependency requires verions 2.10.0 of requests and pokepy (0.6.2) requires version 2.21 of requests.
ERROR: Cannot install beckett==0.8.0 and pokepy==0.6.0 because these package versions have conflicting dependencies.

The conflict is caused by:
    pokepy 0.6.2 depends on requests==2.21.*
    beckett 0.8.0 depends on requests==2.10.0

Damn... this wasn't a problem some versions of pip ago...
Well, I guess pokepy is broken until I can manage to deprecate beckett...
You could checkout pokebase

Maybe @Kronopt you could just submit a PR over at beckett updating the requirements.

I could, but It would probably just go unmerged.
Phalt has already mentioned that he is no longer working on beckett, so I'd rather not propose any new changes

One could fork it, update the deps and push a new package. I can do it, or you can do it (if you feel maintaining the new package).

I don't feel like maintaining that new package as I'm already working on another one to replace it.
Feel free to do it if you want to, though

Did this ever get fixed? Still running into same issue

Not yet

Has this been fixed yet?


Explain folder creation

Folder management is mentioned in the tutorial, but folder creation is never addressed. As a new user, it was not clear to me how folders are created, so perhaps an explanation is in order: Either in the tutorial or in the documentation.

I'll remark that in my opinion folder management should be changed into drag-and-drop altogether (part of the problem was that the location for the folder button isn't intuitive to me), then folders would be created by dragging one feed onto another (as in Android, for example). But this is a big feature request.


Adding new Azure-related articles

Resolved via https://github.com/spring-guides/gs-spring-boot-for-azure/commit/1a674d8e57da015c428d387047f33160ef7c8a58


storagecluster: Increase mons db allocated space

recent cases of mons db require more space then 10G. Increase the space
to 50G.
BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1944580
Signed-off-by: Nitin Goyal<EMAIL_ADDRESS>

this looks good in general.
But are we agreed on 50Gi?
The BZ seems to mention at least 50Gi ...

Yes, I believe so.

The code change is simple enough and valid. However, I worry about merging this without any consideration for upgrade scenarios of existing clusters. I believe this should go in Rook, since it handles the creation of the storage volume. @travisn @leseb Thoughts?

Rook just creates the PVC based on what's provided by the template. If the CSI driver supports volume resize (block + fs) then we are covered. I don't think the mon store needs to be resized, but let me double check. If that's the case, an init container would do the trick!

Rook just creates the PVC based on what's provided by the template. If the CSI driver supports volume resize (block + fs) then we are covered. I don't think the mon store needs to be resized, but let me double check. If that's the case, an init container would do the trick!
Edit: nothing to resize on the mon data store if the fs is resized! Nothing to do on the Rook side.

So changing the CephCluster spec should lead to a modification of the Mon's PVC and thus trigger a volume expansion from the CSI driver?

Rook just creates the PVC based on what's provided by the template. If the CSI driver supports volume resize (block + fs) then we are covered. I don't think the mon store needs to be resized, but let me double check. If that's the case, an init container would do the trick!
Edit: nothing to resize on the mon data store if the fs is resized! Nothing to do on the Rook side.

So changing the CephCluster spec should lead to a modification of the Mon's PVC and thus trigger a volume expansion from the CSI driver?

Correct.


[BugFix-AppConfig] Assign default endpoint to default pipeline builder

Description
Fixed the bugs with missing endpoint assign when TokenCrendeitla is provided.
All SDK Contribution checklist:

[] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

/azp run java - appconfiguration - tests