How to set Api Key for private Feature Layer

Hello,
Currently I run into an issue when trying to add a private Feature Layer to Gis map, error:
An ArcGIS authentication error occurred when accessing "[feature_layer_url]". An API key or credential may be required. For more information see (https://developers.arcgis.com/documentation/mapping-apis-and-services/security/).
Since there's no information or any example that mentions about setting api key to feature layer, I wonder how that can be done?
I've tried this but not work
ArcGISEnvironment.apiKey = [apiKey]; // <- Already set api key for ArcGIS environment here

// Define service feature table
final serviceFeatureTable = ServiceFeatureTable.fromUri(
   Uri.parse(
      '[feature_layer_url]',
   ),
);

// Define feature layer with service feature table above
final featureLayer = FeatureLayer.withFeatureTable(
   serviceFeatureTable..apiKey = widget.apiKey,
);

// Add feature layer to current map
map.operationalLayers.add(featureLayer); // <- Get error for API key here
mapViewController.arcGISMap = map;

Hi there! Thanks for reaching out. You are correct to set your API key via ArcGISEnvironment.apiKey. You don't need to set it specifically on, e.g. the feature layer. So that part is good.
An initial thing to check - are you calling ArcGISEnvironment.apiKey = [apiKey]; in the main() method before you call runApp?
i.e. in your main.dart file:
void main() {
  ArcGISEnvironment.apiKey = [apiKey];
  runApp(MyApp());
}

Closing as resolved. If you have further questions regarding setting the API key, feel free to reopen this issue.


CMA fails at Runtime when OS doesn't provide permissions

Original issue: https://charm.cs.illinois.edu/redmine/issues/1749

Ronak saw this issue on his laptop with a netlrts-linux-x86_64 build.
Although the issue is being identified and handled, there's nothing much the user can do to fix it. A check for permissions during configure can help disable CMA on machines that support it but do not provide permissions for it.
------------- Processor 0 Exiting: Called CmiAbort ------------
Reason: Can't use CMA for SHM! Permissions Denied

Original date: 2017-12-06 17:03:35

Related: the print statement that says if CMA is enabled should only be printed once on PE 0, but currently it gets printed multiple times:
$ ./charmrun +p4 ./pgm +vp4 ++local
Charmrun> scalable start enabled.
Charmrun> started all node programs in 0.006 seconds.
Charm++> Running in non-SMP mode: numPes 4
Converse/Charm++ Commit ID: v6.8.2-138-g529f8574d
Charm++> cma enabled for within node transfers using the zerocopy API
Charm++> cma enabled for within node transfers using the zerocopy API
Charm++> cma enabled for within node transfers using the zerocopy API
Charm++> cma enabled for within node transfers using the zerocopy API
Charm++> scheduler running in netpoll mode.
CharmLB> Load balancer assumes all CPUs are same.
Charm++> Running on 1 unique compute nodes (8-way SMP).
Charm++> cpu topology info is gathered in 0.001 seconds.
[Partition 0][Node 0] End of program

Also we should capitalize "CMA"

Original date: 2017-12-06 20:56:29

From Core meeting: we'd like to add a runtime check for CMA support. This should be fixed with high priority, since it breaks all Charm++ programs on some machines.

Original date: 2017-12-07 15:18:59

Fix: ~~https://charm.cs.illinois.edu/gerrit/#/c/3381/~~ https://github.com/UIUC-PPL/charm/commit/9f5778579c07774ceeed06773f81f27478305763


Bump version to 2.0

The versioning is very confusing right now.
With the recent release 1.1.3 https://github.com/microsoft/WSL/releases/tag/1.1.3 it was really confusing to for people who do not follow the project on a day to day basis, just see news/marketing pieces about WSL, WSL2, and releases.
The .Net project finally cleared up its versioning mess. I think the situation for WSL is far simpler, and it would make communication more straightforward, if version 2.x+ would simply mean WSL2.

Agreed. We will be bumping our version number to 2 fairly soon.


[Feature]: Standalone time picker

Describe the feature
I was looking for a Vue 3 time picker and didn't find much. Then found Radix Vue and thought that this framework could need a standalone time picker with a nice dropdown.
Additional question: Is it possible to add a time picker to a date picker?

Additional question: Is it possible to add a time picker to a date picker

Yes, it can be done with @internationalized/date pkg, I think we can even create Time Picker with it but, I need to check the docs to find out the way
Meanwhile, you can check this source
https://github.com/damianricobelli/shadcn-cal-com/
https://shadcn-cal-com.vercel.app

Or perhaps can be fixed by https://github.com/radix-vue/radix-vue/issues/555

Drop-down time picker like vue2-timepicker would be great.

Seeing there's multiple request for this component, perhaps we can create one with https://react-spectrum.adobe.com/react-spectrum/TimeField.html as reference 😁


chore: update shared-lib readme to clarify purpose

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

docs update

What is the new behavior (if this is a feature change)?

clarify the purpose and intended use of shared-lib

Other information:

also some minor tidying of removing the unnecessary dependency on bluerpints-integration from playout-gateway
Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report
Base: 79.56% // Head: 79.56% // No change to project coverage :thumbsup:

Coverage data is based on head (232ca7b) compared to base (859d035).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           release49     #822   +/-   ##
==========================================
  Coverage      79.56%   79.56%
==========================================
  Files             99       99
  Lines          25054    25054
  Branches        1960     1960
==========================================
  Hits           19933    19933
  Misses          5115     5115
  Partials           6        6

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Support include Kconv

Marked Kconv methods as module_function
See https://github.com/ruby/ruby/blob/master/ext/nkf/lib/kconv.rb for original Ruby methods
(Ctrl + F → module_function)
Motivation

Support include Kconv
Test plan

None

Merged with #3891

Merged with #3891


A question about USB

Dude, I have the same configuration as you, same mobo same CPU same vga.
I already did the usbports and Im using the 0.76 version of usbinjectall.kext, everything is working flawsly but
i would like to know if there is any possibility to power-off the computer's USB/USB-c ports as soon as it goes into sleep mode, I have an nvme drive that heats up a lot even when the computer is in sleep mode, others devices such as keyboard or mouse is fine, but this drive isnt :(
Thanks in advance

You might try checking “Put drives to sleep” or something like that in the Power settings

You might try checking “Put drives to sleep” or something like that in the Power settings

This is my default setting for this option, didnt dit the trick , sad :(

My USB (all of them at the same time) get randomly shut off after a week of uptime, sometimes a few days. This problem persists very long across all versions.

My USB (all of them at the same time) get randomly shut off after a week of uptime, sometimes a few days. This problem persists very long across all versions.

Did you try to use the NVMEfix.kext ?

@tucomel Not sure if you've solved this? I know it is recommended to replace USBInjectall.kext after installation. Most would suggest you complete port mapping b/f installing the OS now as it has become harder to port map since macOS 11.3.
I suggest you try the USB mapping kext provided or create your own following the guide. Hopefully that will solve the issue for you. Let me know how you make out.


[automated] Update Lotus version in API tests

Changes

Updates Lotus version in the JSON-RPC API tests to the latest release.

Parity tests for Filecoin.StateMinerPreCommitDepositForPower is broken
2024-10-09T07:17:37.3037734Z Request dump: Request { method_name: "Filecoin.StateMinerPreCommitDepositForPower", params: Array [String("f0138097"), Object {"DealIDs": Null, "Expiration": Number(3055733), "SealProof": Number(8), "SealRandEpoch": Number(2004533), "SealedCID": Object {"/": String("bagboea4b5abcbp5d7ycbulzvf6ic73ab2zkeebeiasiyndbvpcl5zeqkk776uclm")}, "SectorNumber": Number(1990), "UnsealedCid": Null}, Array [Object {"/": String("bafy2bzaceaybqytxxdounajnsa7jgjtr4nkrzrlkdawoux2mgacpn6emeamas")}, Object {"/": String("bafy2bzacecqhuvzpreas6i5liaacfkp3wvrdcoqv4y5d4oggwxhhov4aq6cwe")}, Object {"/": String("bafy2bzaceaoeazbwlzzj55rzqm24chajk4x2wheksneshikm6yg4nqge56fmi")}, Object {"/": String("bafy2bzacecok4b4nj5mqwi2kgkrxfaetb7pdpdkliwqooy5ejcpdeaib557ua")}]], result_type: PhantomData<serde_json::value::Value>, api_paths: V1, timeout: 120s }
2024-10-09T07:17:37.3043281Z Request params JSON: ["f0138097",{"DealIDs":null,"Expiration":3055733,"SealProof":8,"SealRandEpoch":2004533,"SealedCID":{"/":"bagboea4b5abcbp5d7ycbulzvf6ic73ab2zkeebeiasiyndbvpcl5zeqkk776uclm"},"SectorNumber":1990,"UnsealedCid":null},[{"/":"bafy2bzaceaybqytxxdounajnsa7jgjtr4nkrzrlkdawoux2mgacpn6emeamas"},{"/":"bafy2bzacecqhuvzpreas6i5liaacfkp3wvrdcoqv4y5d4oggwxhhov4aq6cwe"},{"/":"bafy2bzaceaoeazbwlzzj55rzqm24chajk4x2wheksneshikm6yg4nqge56fmi"},{"/":"bafy2bzacecok4b4nj5mqwi2kgkrxfaetb7pdpdkliwqooy5ejcpdeaib557ua"}]]
2024-10-09T07:17:37.3047779Z Forest response: "343502843503823804794"
2024-10-09T07:17:37.3048942Z Lotus response: "34350284350382380479"
2024-10-09T07:17:37.3049691Z Diff: -"343502843503823804794"
2024-10-09T07:17:37.3050043Z
2024-10-09T07:17:37.3050224Z +"34350284350382380479"
2024-10-09T07:17:37.3050514Z
2024-10-09T07:17:37.3050521Z
2024-10-09T07:17:37.3056918Z Request dump: Request { method_name: "Filecoin.StateMinerPreCommitDepositForPower", params: Array [String("f0138097"), Object {"DealIDs": Null, "Expiration": Number(3055733), "SealProof": Number(8), "SealRandEpoch": Number(2004533), "SealedCID": Object {"/": String("bagboea4b5abcbun7tqgnaiyq3sup5bb33zebqjlynkgrcbtoxg3sfn37ysc66bcd")}, "SectorNumber": Number(1989), "UnsealedCid": Null}, Array [Object {"/": String("bafy2bzacebba4hqc72yuuyziut43omsivdgzrfrvflmjyixyflzlknkkwibei")}, Object {"/": String("bafy2bzaceabdnjyoqte374ebwkfgyoq67yssfku26v6rmn5eipdtkuxcoslyg")}, Object {"/": String("bafy2bzacec3esagt4x4manbrvpczem67ssc5aqdojne5qzpfszoemme2hswkc")}]], result_type: PhantomData<serde_json::value::Value>, api_paths: V1, timeout: 120s }
2024-10-09T07:17:37.3066247Z Request params JSON: ["f0138097",{"DealIDs":null,"Expiration":3055733,"SealProof":8,"SealRandEpoch":2004533,"SealedCID":{"/":"bagboea4b5abcbun7tqgnaiyq3sup5bb33zebqjlynkgrcbtoxg3sfn37ysc66bcd"},"SectorNumber":1989,"UnsealedCid":null},[{"/":"bafy2bzacebba4hqc72yuuyziut43omsivdgzrfrvflmjyixyflzlknkkwibei"},{"/":"bafy2bzaceabdnjyoqte374ebwkfgyoq67yssfku26v6rmn5eipdtkuxcoslyg"},{"/":"bafy2bzacec3esagt4x4manbrvpczem67ssc5aqdojne5qzpfszoemme2hswkc"}]]
2024-10-09T07:17:37.3069781Z Forest response: "343502431473351305643"
2024-10-09T07:17:37.3070368Z Lotus response: "34350243147335130563"
2024-10-09T07:17:37.3071026Z Diff: -"343502431473351305643"
2024-10-09T07:17:37.3071354Z
2024-10-09T07:17:37.3071508Z +"34350243147335130563"
2024-10-09T07:17:37.3071780Z
2024-10-09T07:17:37.3071786Z

Oof. Either there's a regression in Lotus, or they fixed a bug.

@lemmih But you approved it! This means you volunteered to investigate it!

@lemmih It seems @LeshyBot "fixed" it. It's probably a particular event on the blockchain that triggered the issue and it would happen anyway. I believe the best course of action now is to upload the snapshot that fails the test and create an issue to investigate this behaviour.

Tracking issue: https://github.com/ChainSafe/forest/issues/4879
I will update the issue once the snapshot upload has finished.


Use standard (not ssl) mongodb version

The current manifest points to the 2008plus-ssl distribution, which doesn't work for me. I get this error when I try to execute mongodb.exe: "The ordinal 341 could not be located in the dynamic link library C:\Users\William\AppData\Local\scoop\apps\mongodb\3.2.3\bin\mongod.exe". (The error can be solve by downloading some ssl libraries as described here. But then I might as well install mongodb manually).
Using the normal distribution fixes this problem.
Of course this is no good for people who need the ssl version. So maybe there should be a separate manifest for that?

@william-ellis Perhaps PR this into https://github.com/scoopinstaller/versions under the name mongodb-nossl
@lukesampson

The problem was solved in 32015c5


.NET Core

Hi,
Is it possible to install Kudu on Linux using .NET Core? Would be sweet to do git push deploy to a Raspberry Pi / Arduino :)

I'm a bit ignorant about .NET core, but doesn't that only run the new ASP.NET 5 (vNext)? If so, it definitely wouldn't work, as Kudu is a 'standard' ASP.NET app. Eventually, we'll move to vNext, but I don't think we're ready for that now.

Yeah, ASP.NET 5. Okay, thanks David.


Allow non-reentrant process for RT-target debugging

https://decibel.ni.com/content/message/144491

Feature supported as of version 1.1.9
Look for SMO.Example.lvproj for a demo.
[Feature: 22] Modified the Process Sync methods to allow usign them into subVIs of the Process.vi dynamic dispatch. This allows to encapsulate a process loop into a subVI and change its reentrancy settings at will.
[Feature] SMO.Example process demonstrates how to launch JKI State Machine process VI as a standalone tester without call for parent node.


Update samples in docs, spring-cli projects, and AI workshop code to use version 0.8.0

There are various places with sample code that needs to be updated.  Known places are

[ ] reference documentation
[ ] spring-cli project catalogs
[ ] Azure AI Workshop
[ ] Azure Acme application
[ ] Sample Carnia RAG search application

Since I tried to use it, failed and had to adapt it anyway, I can provide an update to the example in:
Open AI - Getting Started
If you want that, which format is best? Pull request?

Hello,
I tried to use this section of the documentation OpenAI Manual Configuration to manual configure the OpenAI embedding client by creating an instance of OpenAiEmbeddingClient but it is not working.
The ".withDefaultOptions" method is not present on the OpenAiEmbeddingClient class.
I ended up doing the following to make it work:
        return new OpenAiEmbeddingClient(
                openAiApi,
                MetadataMode.EMBED,
                OpenAiEmbeddingOptions.builder().withModel(OpenAiApi.DEFAULT_EMBEDDING_MODEL).build(),
                RetryUtils.DEFAULT_RETRY_TEMPLATE);```

https://github.com/spring-projects/spring-ai/issues/447   and thanks @emileastih1


Temporary fix for serde issue

This is a temporary fix until  https://github.com/serde-rs/serde/issues/1454 gets fixed.
Some GenesisConfig could not be defined due to
the trait `_IMPL_SERIALIZE_FOR_Log::_serde::Deserialize<'_>` is not implemented for `<T as balances::Trait>::Balance`

from a generated genesis config looking like this:
    #[cfg(feature = "std")]
    #[serde(rename_all = "camelCase")]
    #[serde(deny_unknown_fields)]
    pub struct GenesisConfig<T: Trait> {
        pub claims: Vec<(EthereumAddress, T::Balance)>,
    }

This PR just get types starting with the trait bound and write serde bounds for them statically through serde 'bound' pragma.

does this still need to be backported ? I can't reproduce on v1.0

Isn't it already in substrate 1.0? (I think, by looking at the date, that the patch_this refers to something else)


clone() looses owner

Really weird problem.
When you clone the model, the owner of the cloned model becomes null.
While it seems to be right a the first glance, in case if this model was a part of the store and has references to other models, it breaks store lookups as they traverse ownership tree upward.
The situation itself is very non-intuitive and weird. So far, the proposal is to add clone option keepOwner. Will investigate it further, it looks like conceptual problem.

Right way to fix it is to override defaultStore. Will see how it works in 2.0 alpha.

Right way is model.otherModel.clone({ pinStore : true }). Commited to develop branch.


non-constant format string in call to cosmossdk.io/errors.Wrapf

Problem
On v2.x
$ make lint
--> Running golangci-lint
WARN The linter 'exportloopref' is deprecated (since v1.60.2) due to: Since Go1.22 (loopvar) this linter is no longer relevant. Replaced by copyloopvar.
x/blobstream/keeper/keeper_data_commitment.go:112:78: printf: non-constant format string in call to cosmossdk.io/errors.Wrapf (govet)
			return types.DataCommitment{}, errors.Wrapf(types.ErrAttestationNotFound, fmt.Sprintf("nonce %d", i))
			                                                                          ^
x/blobstream/keeper/keeper_data_commitment.go:139:61: printf: non-constant format string in call to cosmossdk.io/errors.Wrapf (govet)
			return false, errors.Wrapf(types.ErrAttestationNotFound, fmt.Sprintf("nonce %d", i))

Proposal
Address the lint complaints

I've done this issue.
https://github.com/celestiaorg/celestia-app/pull/3924


feat: Kubernetes service discovery

Surprising to see that this has not been implemented or mentioned anywhere yet (unless I'm blind) considering this project is all about being cloud-native.
https://github.com/itzg/mc-router#kubernetes-usage is a great example for forced hosts, but it'd be nice to expand this to Pod support aswell.

Using Kubernetes's KRM can be a good configuration source for the Gate Lite mode.
Currently Gate Lite only sources its configuration from the local config file.
To add this feature we need to provide support for live config change that an adapter can use to sprinkle in external configuration from Kubernetes resources.

To anyone who wants to implement this: I've just built something similar for one of my projects:
everbuild-org/prismcloud#images/ingress-gate/gate/plugin/service_discovery

@Bloeckchengrafik you should post your tool in https://discord.com/channels/633708750032863232/1042460518595690526
and add it to https://github.com/minekube/awesome

Thanks a lot for the suggestion! I'll do that when it becomes somewhat production ready!


GatlingTestRunner: add cli to run with custom parameters

Currently we run load testing for Kibana build via FTR and custom config. It solves the basic needs, but does not allow to pass custom arguments to maven command.
We need to have a cli for load testing that allows to specify scenario, path to load testing project, etc.
node scripts/load_testing_runner \
  --relativePath "../kibana-load-testing" \
  --scenario org.kibanaLoadTest.simulation.DemoJourney

I ended up with GATLING_SIMULATIONS =DemoJourney node scripts/functional_tests --config test/load/config.ts

I ended up with GATLING_SIMULATIONS =DemoJourney node scripts/functional_tests --config test/load/config.ts


Handle leading and trailing whitespace for comments.

This PR closes #616. Just trimming whitespace from issue comments before seeing if it is a /check-enforcer [action] command. Makes it more resilient to folks copying the command from somewhere with a leading or trailing space.

I'm fine with the change, anything else that should be trimmed?

That is the only place we process any input like that so it should be all that is required.


[FEATURE] Login button

Description
getting a login button
Screenshots
No response
Additional information
No response

@akshaya-hub we have a login button


Added emscripten makefile

This is really neat, thank you!
I'd appreciate if you could move the index.html to examples/javascript.html or similar, and add a small section into the README.md (maybe a section after "In Python") also linking to index, possibly using http://rawgit.com/.

@lucasb-eyer Will try ... ;-)

Thanks! In case you can't find time for that, I'd rather you let me know than let this PR die.

Ok, just close it.
On Sun, Apr 26, 2015 at 1:09 PM, Lucas Beyer<EMAIL_ADDRESS>wrote:

Thanks! In case you can't find time for that, I'd rather you let me know
than let this PR die.
—
Reply to this email directly or view it on GitHub
https://github.com/lucasb-eyer/heatmap/pull/4#issuecomment-96365613.

--

This e-mail is intended for the named recipient(s). It may contain
privileged and/or confidential information. If you are not one of the
intended recipients, please notify the sender immediately and destroy this
e-mail and attachment(s): you must not copy, distribute, retain or take any
action in reliance upon the email or attachment(s). While all reasonable
efforts are made to safeguard inbound and outbound e-mails, Tomas Korcak
cannot guarantee that attachments are virus-free or are compatible with
your systems, and does not accept liability in respect of viruses or
computer problems experienced. Thank you.

After having stalled this for so long, I'm looking into running it, then documenting it myself and finally merging. A few questions:

Why do you save and restore the stack in index.html.
Why malloc four bytes into offset and never use them?
'object' is not a known type to cwrap. 'number' should be used for C-pointers and, even worse, two numbers for 64-bit pointers. Since I never used emscripten before, I assume I'm missing something and just opening the file in firefox/chromium is not how it's supposed to be used?


Announce to discussions on similar issues.

I had the same problem, so I solved the problem by creating this plugin next-image-loader.
It transparently applies the custom loader defined in next.config.js to the props in next/image.
// next.config.js
const withImageLoader = require('next-image-loader')(
  // (resolverProps: { src: string; width: number; quality?: number }) => string
  ({ src, width, quality }) => `${src}?width=${width}&quality=${quality || 75}`
)

module.export = withImageLoader({

})

https://github.com/vercel/next.js/discussions/24736
https://github.com/vercel/next.js/discussions/22980


[4.0.0-rc.0] swipe to go back

Bug Report
Ionic version:

[x] 4.x
Current behavior:
Since beta.18, we can no longer swipe to go back to the previous page. It opens the menu. It seems it behaves like that when there is a side menu.
Steps to reproduce:

Start a new project with sidemenu starter template.
Add a child route/component to the list
Navigate to it
Try to go back by swiping to the left.

Ionic info:

Ionic:

   ionic (Ionic CLI)             : 4.6.0 (/usr/local/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.0.0-rc.0
   @angular-devkit/build-angular : 0.11.4
   @angular-devkit/schematics    : 7.1.4
   @angular/cli                  : 7.1.4
   @ionic/angular-toolkit        : 1.2.2

System:

   NodeJS : v8.12.0 (/usr/local/bin/node)
   npm    : 6.5.0
   OS     : macOS Mojave

Twitter or amazon app for exemple.  Swiping again opens the menu.

Alright! I will check it out and implement a fix, thanks for the research work!

Thanks

Apps that want to use swipe-to-go-back but retain access to the menu in nested pages usually have both a back arrow and a hamburger on nested pages. I think it's clunky, but it works.
Here's the header on Amazon's root page:

And here's one level in where STGB still works:

An Google Play Store always shows the menu on swipe ;)

Yes it depends which device you're using, I forgot to mention, this is about iOS not android


Where is the correct place for specialization

Hi,
I hope you can help me with this, I am diffing 2 objects which have been unmarshalled using jaxb.
I am having problem with types of
https://docs.oracle.com/javase/7/docs/api/javax/xml/datatype/Duration.html
which need to be instantiated with https://docs.oracle.com/javase/7/docs/api/javax/xml/datatype/DatatypeFactory.html
When running
node.canonicalSet( merged, node.canonicalGet( newInformation ) );
I get :
de.danielbechler.diff.instantiation.TypeInstantiationException thrown due to : Failed to create instance of type 'class com.sun.org.apache.xerces.internal.jaxp.datatype.DurationImpl'. Reason: Type doesn't have a public non-arg constructur
How do I get around this?

Hi,
you can provide a custom InstanceFactory like so:
InstanceFactory instanceFactory = new InstanceFactory() {
	@Override
	Object newInstanceOfType(Class<?> type) {
		if (Duration.isAssignableFrom(type)) {
			return DatatypeFactory.newInstance().newDuration(0);
		}
		return null
	}
}

ObjectDiffer objectDiffer = ObjectDifferBuilder.startBuilding()
	.introspection().setInstanceFactory(instanceFactory).and()
	.build()

However, I don't think it will help in your case, because the library is trying to instantiate the Duration because you are trying to assign a new value to one of its properties. This will ultimately fail, because Duration is an immutable data structure.
I think what you really want to do is to mark Duration as an equals-only type, so that the ObjectDiffer won't look at it's properties and only use its equals method to determine whether it has changed. This way canonicalSet will simply replace the entire Duration instance, instead of trying to modify it.
Hope this helps! Keep me posted. :wink:


small updates to pig examples

change s3n to s3 in paths
small typo in airline script

cc @ddaniels888

:+1:


refactor: use string literals

Comparing results between 'main' and 'Current_run'
Host 'Brown-rhinoceros-beetle' with 8 'aarch64' processors with 7 GB memory, running:
#1 SMP PREEMPT_DYNAMIC Mon May 27 16:05:41 UTC 2024

MyBenchmark
lipsum metrics
Time (wall clock): results within specified thresholds, fold down for details.

Time (wall clock) (μs) *
p0
p25
p50
p75
p90
p99
p100
Samples

main
16
16
16
16
17
17
18
100

Current_run
15
15
15
15
15
16
16
100

Δ
-1
-1
-1
-1
-2
-1
-2
0

Improvement %
6
6
6
6
12
6
11
0

lipsum-zh metrics
Time (wall clock): results within specified thresholds, fold down for details.

Time (wall clock) (ns) *
p0
p25
p50
p75
p90
p99
p100
Samples

main
1570
1577
1581
1590
1638
1775
1972
100

Current_run
1557
1563
1567
1582
1623
1940
1942
100

Δ
-13
-14
-14
-8
-15
165
-30
0

Improvement %
1
1
1
1
1
-9
2
0

medium-fragment metrics
Time (wall clock): results within specified thresholds, fold down for details.

Time (wall clock) (μs) *
p0
p25
p50
p75
p90
p99
p100
Samples

main
44
44
44
44
44
47
47
100

Current_run
44
44
44
44
44
47
47
100

Δ
0
0
0
0
0
0
0
0

Improvement %
0
0
0
0
0
0
0
0

small-fragment metrics
Time (wall clock): results within specified thresholds, fold down for details.

Time (wall clock) (ns) *
p0
p25
p50
p75
p90
p99
p100
Samples

main
4660
4665
4669
4674
4678
4824
4824
100

Current_run
4637
4641
4645
4649
4653
4805
4806
100

Δ
-23
-24
-24
-25
-25
-19
-18
0

Improvement %
0
1
1
1
1
0
0
0

strong metrics
Time (wall clock): results within specified thresholds, fold down for details.

Time (wall clock) (μs) *
p0
p25
p50
p75
p90
p99
p100
Samples

main
20
20
20
20
20
21
21
100

Current_run
20
20
20
20
20
22
26
100

Δ
0
0
0
0
0
1
5
0

Improvement %
0
0
0
0
0
-5
-24
0

tiny-fragment metrics
Time (wall clock): results within specified thresholds, fold down for details.

Time (wall clock) (ns) *
p0
p25
p50
p75
p90
p99
p100
Samples

main
459
465
472
497
535
537
538
100

Current_run
469
475
492
509
585
636
642
100

Δ
10
10
20
12
50
99
104
0

Improvement %
-2
-2
-4
-2
-9
-18
-19
0


🧑💻:  Add the glowing cursor animation feature

:red_circle: Title : Adding glowing cursor
:red_circle: Enhancement Aim : To give beautiful glowing effcet on moving in the screen
:red_circle: Changes :  Where ever the user takes their cursor the beautiful 3d bubble or something will glows off which make website looks more beautiful
Something like this :

:white_check_mark: To be Mentioned while taking the issue :

Full name :  Naman Parashar
What is your participant role? (Mention the Open Source Program name. Eg. GSSOC, SSOC, JWOC, etc.)   :  UNDER GSSOC 24

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@namandude go ahead.


Update French translation

please dont merge into staging, merge into master, master may have newer translations that are not merged into staging yet


The binding issue when using "run as Spring Boot App" of Spring Tool Suite 4

[Symptom]
When I use the latest Spring Tool Suite 4.01 to run the sample application of spring-cloud-stream,
I found that when I use the "run as Spring Boot App" instead of "run as Java application", the application
will not connect the rabbitmq server to create exchange and queue.
It appears it's a issue of Spring Tool Suite. Because there is no any exception log, I spent some time to find
if there is anyone encounter the same issue but no finding.
Luckily, I found when I comment the following dependency in build.gradle, it works!
//testImplementation('org.springframework.cloud:spring-cloud-stream-test-support')
The exchange and queue will be created when run as Spring Boot App.
Because the project is created by spring initializr, the dependency testImplementation('org.springframework.cloud:spring-cloud-stream-test-support') is automatically
added by default, I guess the compatablity is confirmed.
Could you help take a look if it's a issue of 'org.springframework.cloud:spring-cloud-stream-test-support'
library? Because there is no any exception log, it's hard to find the root cause from user.
[sample code for reproduce]
https://github.com/hyderai/TestSpringCloudStream

That is most likely because the test binder is picked up as default (instead of the Rabbit). . . but we'll take a look if here is anything that can be done

@aclement: When you get a chance, could you or someone from the STS team have a look at this?

Today I tried the  latest nightly build (<IP_ADDRESS>901302038-CI-B2145) and the problem still exists.

@bboeck If you tried to update STS from a update site, I wouldn't necessarily trust that it will update everything correctly. Please try a full distribution build. You can download one here:  http://dist.springsource.com/snapshot/STS4/nightly-distributions.html (pick one under "Eclipse-based distribution builds" section).
Also I should mention that the fix I referred above will only take effect on new launch configurations. If you are using an old workspace you may be using old launch configuration, these old launch configurations will still behave the same way as before the bugifx.
So to sumarize, the latest STS "Run As >> Spring Boot App" functionality should use the exact same classpath as when using BuildShip directly (i.e. "Run As >> Java Application"),  but only for new launch configurations.
It is still possible the bugfix is buggy. So, if all of the following are true:
a) you are using a full nightly distribution build
b) you are using newly created launch configurations
c) what you are trying to do is working correctly when running via "Run As >> Java Application"
d) but is not working correctly when running via "Run As >> Spring Boo App"
If all of these are true, then the conclusing must be that either our bugfix is not working properly, or the bug you are experiencing is a different bug then we assumed. Then, please raise a ticket with STS 4 (https://github.com/spring-projects/sts4) and please make sure to include enough details that we can reproduce and debug the issue. The best would be to attach a sample project that we can run, along with a short description what the expected correct and observed incorrect behavior is.
For the moment, I think the most likely reason why you are still seeing the bug is because you are probably still using old launch configuration(s) with incorrect classpath(s).

@kdvolder I have just installed spring-tool-suite-4-4.1.2.CI-B1528-e4.11.0-win32.win32.x86_64.zip and the problem seems to be solved. Everything is working fine now.
Thx for your help!

@bboeck Glad to hear that, thanks for letting us know!


SQLiteException when I try read a SQLite file generated out of application

I'm using SQLite PCL with Xamarin and, in my application, I had to download a SQLite file as a byte array (byte[]) from a web service and use this file in my app as a SQLite DataBase.
I'm trying to do this, but I'm had SQLiteException: file is encrypted or is not a database
That's the class that I'm tryng to create a SQLite:
public abstract class BaseLocalDao<POCO, Entity> : ILocalDao<POCO, Entity> where POCO : class, IPOCO where Entity : class, IEntity, new()
    {
        protected IConverter<POCO, Entity> _converter;
        protected readonly SQLiteConnection _dataBase;

        public BaseLocalDao()
        {
            var controleArquivoBanco = Mvx.Resolve<IControleArquivoBanco>();

            _dataBase = new SQLiteConnection(controleArquivoBanco.GetPath());
            _dataBase.CreateTable<Entity>();
        }
}

The connection is create, but when CreateTable is execute, the app throw the SQLiteExcpetion that I mentioned.

It sounds to me like the DB is encrypted or corrupted.
Try opening on the command: sqlite3 my.db
That should let you know of any troubles.

I used SQLite Studio to check the file generated by web service and there aren't error's on file.

where I has to execute this command?

I find the error. You can delete or close this issue. I was converting the string of SQLite file to byte[] for the wrong way.


"$" expression outside quote around...

I'm attempting to wrap <sys/mman.h> which runs successfully but generators the following line:
const __ptr_t = $(Expr(:incomplete, "incomplete: premature end of input"))

resulting in the error:
LoadError: syntax: "$" expression outside quote around /home/colinxs/workspace/crypto/obs/gen/mman.jl:313

The corresponding c code is in <sys/cdefs.h>:
/* This is not a typedef so `const __ptr_t' does the right thing.  */
#define __ptr_t void *

I can blacklist it for now, since it's not used directly in the generated code, but figured I'd file an issue to put it on the radar.

Thanks for reporting this!


sqlite: 3.31.1 -> 3.32.0

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/sqlite/versions.
meta.description for sqlite is: "A self-contained, serverless, zero-configuration, transactional SQL database engine"
meta.homepage for sqlite is: "https://www.sqlite.org/"
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.32.0 with grep in /nix/store/p63y49bryg0vd356m77l7gs4qm64s78r-sqlite-3.32.0-bin

directory tree listing: https://gist.github.com/414254b0e14e969cdb028db0f0244ca2

du listing: https://gist.github.com/779d6112968a3f03f32172055eff4694

Rebuild report (if merged into master) (click to expand)

95064 total rebuild path(s)

27027 package rebuild(s)

26802 x86_64-linux rebuild(s)
24735 i686-linux rebuild(s)
19330 x86_64-darwin rebuild(s)
24197 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
AusweisApp2
DisnixWebService
EBTKS
EmptyEpsilon
Fabric
LASzip
Literate
MMA
NSPlist
OVMF
OVMF-CSM
OVMF-secureBoot
PlistCpp
QuadProgpp
R
SDL
SDL2
SDL2_gfx
SDL2_image
SDL2_mixer
SDL2_net
SDL2_ttf
SDL_Pango
SDL_gfx
SDL_gpu
SDL_image
SDL_mixer
SDL_net
SDL_sound
SDL_stretch
SDL_ttf
SPAdes
SkypeExport
Sylk
Xaw3d
_0x0
_1password
_20kly
_2bwm
_389-ds-base
_3llo
_3mux
_90secondportraits
_9pfs
a2jmidid
abbaye-des-morts
abc-verifier
abcde
abcl
abcm2ps

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/p63y49bryg0vd356m77l7gs4qm64s78r-sqlite-3.32.0-bin \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A sqlite https://github.com/r-ryantm/nixpkgs/archive/c5f7bda79f1134b1017bcbd11528a5e6ff7da6bc.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/p63y49bryg0vd356m77l7gs4qm64s78r-sqlite-3.32.0-bin
ls -la /nix/store/p63y49bryg0vd356m77l7gs4qm64s78r-sqlite-3.32.0-bin/bin

Security vulnerability report

Security report (click to expand)

CVEs resolved by this update:

CVE-2020-11655
CVE-2020-11656
CVE-2020-9327

CVEs introduced by this update:
none
CVEs present in both versions:

CVE-2020-13434
CVE-2020-13435

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @edolstra @np for testing.

@ofborg eval

Changes seem to be non-breaking https://www.sqlite.org/releaselog/3_32_1.html

bumped it to 3.32.1 patch

Upstream claims more CVE fixes in these bumps than found by the bot: https://www.sqlite.org/cves.html#status_of_recent_sqlite_cves   Also /cc roundup https://github.com/NixOS/nixpkgs/issues/88403 and maintainer @pesterhazy.

@GrahamcOfBorg test grafana
@GrahamcOfBorg test matrix-synapse

tests are failing from dependent package being unable to build.... not sure where to go from here

3.32.2 has been released.

Sorry, I didn't realize other people had added commits to this PR before I closed it. I reopened it.

@zowoq added 3.32.2

I see sqlite-analyzer very often gets forgotten...

Any opinions on how security problems for sqlite should get handled on 20.03?  Just update it the same?

Any opinions on how security problems for sqlite should get handled on 20.03? Just update it the same?

This would be my assumption.


Use non-SSL links for roadmap due to warnings

Currently viewing https://roadmap.iojs.org/results in a security warning. I didn't look into the root cause because I'm pretty sure I'd be unable to fix it. This update just makes the roadmap links load without warnings. If someone else would rather deal with the issue causing the SSL warning then this PR could be closed.

why is that warning popping up? do we have a ticket for this in the build repo?

oh wait, i think we're just using gh-page for roadmap, that's why SSL isn't on.

@mikeal right, gh-pages, this is related #254


[FEATURE] Add a new Page for each repository

Is this a unique feature?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Is your feature request related to a problem/unavailable functionality? Please describe.
Yes. I would like to create a new page for each repository. Which can be sharable and will result in people coming to our website to rate the repository. Now we can just ask them to share the link of their GitHub repository. But there is now a way they can share the link through which user can give their ratings on the repo rater.
Proposed Solution

Separate Page for Each Repo Listed.
Can be accessed by /repository_name

Screenshots
No response
Do you want to work on this issue?
Yes
If "yes" to above, please explain how you would technically implement this
I will create a separate Page in which the slug having the component will be fetched. And we will create the slug using the repository and username in some unique manner.

Great idea, I think it needs some planning, but something we can do in the future


/read-only ignores .gitignore and .aiderignore when adding files or directories.

Issue
/read-only does not follow .gitignore, verifying expected behavior.

Here is a working copy of a fix that applies .gitignore, .aiderignore, and determines if file is in repo. I'm not sure if this breaks other requirements of /read-only. If so a new command or command plus option to allow for ignoring git or another type of ignore?
While working through this, another concept came to mind of allowing for /code or /architect chat to ask to modify /read-only files to further encourage cache use with larger repo context.
command.py.md

I noticed this too, if you run /read-only * it will add the entire .git directory

Thanks for trying aider and filing this issue.
Yes, /read is allowed to add any files from anywhere in the file system.

You'll have to use /read carefully if you wish to use it with broad wildcards to pick up huge parts of your repo but not all of it.
This is usually not advised, since it will likely confuse the LLM. As such, it's not in scope for aider to make this "easy".

Not even the .git repo? How about if the wildcard includes more than 10 documents the program warns you "Add 1000 files?"

Thanks for update.
I brought this up as I have had some luck retuning /read-only to follow .gitignore and .aiderignore and have a larger set of files in cache. There is a sweet spot on the number of tokens that work well, but it helps get large cache hits vs. writes on supported models for low cost requests.
Separately, I'm interested if there has been any research to determine the optimal performance by token size of the major models.

I'm labeling this issue as stale because it has been open for 2 weeks with no activity. If there are no additional comments, it will be closed in 7 days.

I'm closing this issue because it has been stalled for 3 weeks with no activity. Feel free to add a comment here and we can re-open it. Or feel free to file a new issue at any time.Note: A bot script made these updates to the issue.


☂ Tasks for removing traffic_manager

These are the open tasks for removing

[ ] Handle traffic_manager tasks in traffic_server threads. #1372

So How do you manage sockets after server crash if we remove the traffic manager?

We will manage sockets in traffic_server only.

I mean, as @shukitchan point in last summit, if the server crash, then every connection request will be reseted. But if we have the manager, the socket will be hold until the server restart successfully.

If traffic_server crashes, instead of lying to the client using some intermediary program isn't it better just to send a real status like a 503 Service Unavailable and let the client deal with it?  If traffic_server crashes it's kind of not available.

Damien, there's still a few mentions of traffic_manager in the master branch now, can we clean that up please? It's not longer deprecated it's completely removed.

Damien, there's still a few mentions of traffic_manager in the master branch now, can we clean that up please? It's not longer deprecated it's completely removed.

@zwoop  Do we want to even wipe out the "deprecated" mentions of it?


Bug in ascii command

Overview
ascii command with no arguments gives "``````" also when called with @ mentions it looks aweful. Maybe we could just disable it when there is a mention.
Affected commands
ascii
Expected output
Preferably error messages
Actual output
"``````" for no output and a ton of garbage for mentions.

Steps to reproduce
use command with a mention, or use a command with no args

@mrb25
Want to take care of this?

@JessWalters already in the works on my fork. Just doing a lot more bugtesting before making the merge.

Ok sounds good

Fixed by merge #35


⚠ Bofrost.be has degraded performance

In cfd9c0c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1264 ms

Resolved: Bofrost.be performance has improved in 1964a49.


tikv panic:918 unexpected state: Applying(0)

Bug Report

What version of TiKV are you using?
[2022/02/23 22:50:32.535 +00:00] [INFO] [lib.rs:80] ["Welcome to TiKV"]
[2022/02/23 22:50:32.535 +00:00] [INFO] [lib.rs:85] ["Release Version:   5.4.0-alpha"]
[2022/02/23 22:50:32.535 +00:00] [INFO] [lib.rs:85] ["Edition:           Community"]
[2022/02/23 22:50:32.535 +00:00] [INFO] [lib.rs:85] ["Git Commit Hash:   a7d1595f5486616be34e0cf2bbf372edb5f0e85a"]
[2022/02/23 22:50:32.535 +00:00] [INFO] [lib.rs:85] ["Git Commit Branch: heads/refs/tags/v5.5.0-nightly"]
[2022/02/23 22:50:32.535 +00:00] [INFO] [lib.rs:85] ["UTC Build Time:    Unknown (env var does not exist when building)"]
[2022/02/23 22:50:32.535 +00:00] [INFO] [lib.rs:85] ["Rust Version:      rustc 1.60.0-nightly (1e12aef3f 2022-02-13)"]
[2022/02/23 22:50:32.535 +00:00] [INFO] [lib.rs:85] ["Enable Features:   jemalloc mem-profiling portable sse test-engines-rocksdb cloud-aws cloud-gcp cloud-azure"]
[2022/02/23 22:50:32.535 +00:00] [INFO] [lib.rs:85] ["Profile:           dist_release"]
What operating system and CPU are you using?
not involving
Steps to reproduce
run tipocket case: bank2 -run-time=6h -tikv-replicas=4 -tidb-replica-read=leader-and-follower -nemesis=random_kill,kill_tikv_1node_5min,shuffle-leader-scheduler,shuffle-region-scheduler,random-merge-scheduler,subcritical_skews
http://<IP_ADDRESS>:31714/archived-workflows/test-store/36631d5a-80ce-43a6-ba1d-857f7bde7bf4
What did you expect?
the cluster status normal;
What did happened?
tikv panic:
[2022/02/23 22:50:16.958 +00:00] [FATAL] [lib.rs:466] ["[region 50] 918 unexpected state: Applying(0)"] [backtrace="   0: tikv_util::set_panic_hook::{{closure}}\n             at home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tikv/components/tikv_util/src/lib.rs:465:18\n   1: std::panicking::rust_panic_with_hook\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/std/src/panicking.rs:702:17\n   2: std::panicking::begin_panic_handler::{{closure}}\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/std/src/panicking.rs:588:13\n   3: std::sys_common::backtrace::__rust_end_short_backtrace\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/std/src/sys_common/backtrace.rs:138:18\n   4: rust_begin_unwind\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/std/src/panicking.rs:584:5\n   5: core::panicking::panic_fmt\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/core/src/panicking.rs:143:14\n   6: raftstore::store::peer_storage::PeerStorage<EK,ER>::snapshot\n             at home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tikv/components/raftstore/src/store/peer_storage.rs:1283:13\n      <raftstore::store::peer_storage::PeerStorage<EK,ER> as raft::storage::Storage>::snapshot\n             at home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tikv/components/raftstore/src/store/peer_storage.rs:736:9\n      raft::raft_log::RaftLog::snapshot\n             at rust/git/checkouts/raft-rs-42b8049ef2e3af07/2a3d7b6/src/raft_log.rs:467:9\n      raft::raft::RaftCore::prepare_send_snapshot\n             at rust/git/checkouts/raft-rs-42b8049ef2e3af07/2a3d7b6/src/raft.rs:670:26\n   7: raft::raft::RaftCore::maybe_send_append\n             at rust/git/checkouts/raft-rs-42b8049ef2e3af07/2a3d7b6/src/raft.rs:823:25\n   8: raft::raft::RaftCore::send_append\n             at rust/git/checkouts/raft-rs-42b8049ef2e3af07/2a3d7b6/src/raft.rs:760:9\n      raft::raft::Raft::send_append\n             at rust/git/checkouts/raft-rs-42b8049ef2e3af07/2a3d7b6/src/raft.rs:873:9\n      raft::raw_node::RawNode::on_entries_fetched\n             at rust/git/checkouts/raft-rs-42b8049ef2e3af07/2a3d7b6/src/raw_node.rs:430:21\n      raftstore::store::fsm::peer::PeerFsmDelegate<EK,ER,T>::on_significant_msg\n             at home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tikv/components/raftstore/src/store/fsm/peer.rs:1198:17\n      raftstore::store::fsm::peer::PeerFsmDelegate<EK,ER,T>::handle_msgs\n             at home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tikv/components/raftstore/src/store/fsm/peer.rs:639:49\n   9: <raftstore::store::fsm::store::RaftPoller<EK,ER,T> as batch_system::batch::PollHandler<raftstore::store::fsm::peer::PeerFsm<EK,ER>,raftstore::store::fsm::store::StoreFsm>>::handle_normal\n             at home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tikv/components/raftstore/src/store/fsm/store.rs:789:9\n  10: batch_system::batch::Poller<N,C,Handler>::poll\n             at home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tikv/components/batch-system/src/batch.rs:406:27\n  11: raftstore::store::worker::refresh_config::PoolController<N,C,H>::increase_by::{{closure}}\n             at home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tikv/components/raftstore/src/store/worker/refresh_config.rs:71:21\n      std::sys_common::backtrace::rust_begin_short_backtrace\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/std/src/sys_common/backtrace.rs:122:18\n  12: std::thread::Builder::spawn_unchecked::{{closure}}::{{closure}}\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/std/src/thread/mod.rs:498:17\n      <core::panic::unwind_safe::AssertUnwindSafe as core::ops::function::FnOnce<()>>::call_once\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/core/src/panic/unwind_safe.rs:271:9\n      std::panicking::try::do_call\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/std/src/panicking.rs:492:40\n      std::panicking::try\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/std/src/panicking.rs:456:19\n      std::panic::catch_unwind\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/std/src/panic.rs:137:14\n      std::thread::Builder::spawn_unchecked::{{closure}}\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/std/src/thread/mod.rs:497:30\n      core::ops::function::FnOnce::call_once{{vtable.shim}}\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/core/src/ops/function.rs:227:5\n  13: <alloc::boxed::Box<F,A> as core::ops::function::FnOnce>::call_once\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/alloc/src/boxed.rs:1854:9\n      <alloc::boxed::Box<F,A> as core::ops::function::FnOnce>::call_once\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/alloc/src/boxed.rs:1854:9\n      std::sys::unix::thread::Thread::new::thread_start\n             at rustc/1e12aef3fab243407f9d71ba9956cb2a1bf105d5/library/std/src/sys/unix/thread.rs:108:17\n  14: \n  15: clone\n"] [location=/home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/tikv/components/raftstore/src/store/peer_storage.rs:1283] [thread_name=raftstore-1-1]
[2022/02/23 22:50:16.958 +00:00] [INFO] [apply.rs:1763] ["exec ConfChange"] [epoch="conf_ver: 89 version: 41"] [type=AddLearner] [peer_id=700] [region_id=207]
[2022/02/23 22:50:16.958 +00:00] [INFO] [apply.rs:1903] ["add learner successfully"] [region="id: 207 start_key:<PHONE_NUMBER>000000FF455F698000000000FF000001016163636FFF756E7420FF343232FF3637390000FD0000FD end_key:<PHONE_NUMBER>000000FF455F698000000000FF000001016163636FFF756E7420FF353135FF3437340000FD0000FD region_epoch { conf_ver: 89 version: 41 } peers { id: 493 store_id: 6 } peers { id: 693 store_id: 4 } peers { id: 700 store_id: 1 }"] [peer="id: 955 store_id: 5 role: Learner"] [peer_id=700] [region_id=207]
[2022/02/23 22:50:32.535 +00:00] [INFO] [lib.rs:80] ["Welcome to TiKV"]

/type bug
/severity critical

TiKV panicked because the peer try to generate a snapshot for other peers but found that it itself is in applying snapshot state.
This panic may introduce by https://github.com/tikv/tikv/pull/11409, when a peer fetch raft logs asynchronously finished, it will still try to send append msg to other peers but without checking whether the peer is still leader (in this case it is even applying snapshot). And if the requested raft log is compacted
https://github.com/tikv/tikv/blob/bf1db1fd3ce847e0fb67b91ebe7ebe518d0a49eb/components/raftstore/src/store/peer_storage.rs#L1006
the above line will return StorageError::Compacted before looking up the async_fetch_results, thus the peer will try to get a snapshot instead, and panic if it is in applying snapshot state. /cc @Connor1996 @BusyJay

Also, I find that async_fetch_results won't be freed after a leader is stepped down.

So term change is considered while leadership changes is not. :(

Also, I found that async_fetch_results won't be freed after a leader is stepped down.

Why? Isn't it cleared immediately at
https://github.com/tikv/tikv/blob/bf1db1fd3ce847e0fb67b91ebe7ebe518d0a49eb/components/raftstore/src/store/fsm/peer.rs#L1259

This may change other follower's leader internal field, and may provide false information for observers. Perhaps raft-rs should also check whether leader id changes without term changes.

Why? Isn't it cleared immediately at
https://github.com/tikv/tikv/blob/bf1db1fd3ce847e0fb67b91ebe7ebe518d0a49eb/components/raftstore/src/store/fsm/peer.rs#L1259

I see

Perhaps raft-rs should also check whether leader id changes without term changes.

Simply checking self.state == StateRole::Leader before sending MsgAppend may be enough. Besides, I think we should also do the term check in raft-rs otherwise it could be a pitfall for the user of raft-rs.

Agree. Both the term check and leadership check should be done in raft-rs.

In which case, the leadership changes while the term doesn't? @NingLin-P

term will always changed. But on_fetched_result bypass term change.

In which case, the leadership changes while the term doesn't? @NingLin-P

Term will also change, but the current term check simply remove the async_fetch_res, and then calling on_entries_fetched can still trigger the abovementioned case.


Optimize the frontend

I want to work on it. Assign it to me.

@dogee123

I would like to work upon it . please assign me ;)

I would like to work upon it . please assign me :)

I would like to work on it . please assign me :)

@dogee123  I have given one pr, I have fixed navbar, card, and go to top icon, check them out


[Outlook] (quickstart) update unified manifest quickstart

These changes make the following assumptions:

The changes won't be published until GA of unified manifest for Outlook, currently expected in late February.
At the time of GA, The Yo Office option for task pane will also provide a manifest choice.

The quick start has been edited so it will work for either manifest.
The special quick start for the unified manifest, that used TypeScript, has been moved to the "/excludes" folder in case we ever want to use it again. It has been removed from the ToC.

Approving with a question. The other drafts include a link to the unified manifest quick start. Will these PRs be updated before they're merged or will the instructions be updated in a separate PR?

I'm redoing the PR for the quick starts. Because we're adding support for JavaScript with the unified manifest (that development is probably an email that I scheduled-sent for later this week), we'll have just one quick start and the user will be instructed to choose JavaScript and then their choice of XML or unified manifest. The existing unified manifest quick start is moving the /excludes folder. See this PR: https://github.com/OfficeDev/office-js-docs-pr/pull/4341


How to run Windows container

Please could you add sample how to run Windows container ? There is statement about support on this page "You can use virtual nodes to schedule Windows Server containers without the need for Windows Server nodes in an AKS cluster."

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 916ac90f-8a94-5f3d-eb99-fe06c39eb6cf
Version Independent ID: 2aad2e47-7fde-8d24-f824-236a26e5d887
Content: Create virtual nodes using the Azure CLI in Azure Kubernetes Services (AKS)
Content Source: articles/aks/virtual-nodes-cli.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@jjindrich
Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

You will have to manually install the open source Virtual Kubelet ACI provider (instructions can be found here: https://github.com/virtual-kubelet/azure-aci) to provision a Windows server virtual node.  Please also visit ACI's documentation to learn more about the preview limitations for Windows containers (https://docs.microsoft.com/en-us/azure/container-instances/container-instances-overview#linux-and-windows-containers) since they will also apply to your Windows server pods.

But Virtual Kubelet creates container with public IP. I need same design as have linux, it makes private ip for container in kubernetes cluster

Thank you @jjindrich for the feedback.
Virtual nodes are not available for Windows nodes at this time.
cc @keikhara
#please-close


Service appProtocol supported values deviate from documented Kubernetes recommendations

Bug Description
Istio explicit protocol selection documentation specifies that the appProtocol property on a Kubernetes Service can be one of approximately ten supported values, e.g. http, https, tcp, when used with Istio for traffic management.
However, the Kubernetes documentation for the same appProtocol property states:

Values should either be IANA standard service names or domain prefixed names such as mycompany.com/my-custom-protocol.

While several of the Istio-supported protocol values are IANA standard service names, e.g. http, redis, and mysql, several others are not:

http2: not an IANA standard service name
grpc and grpc-web: not IANA names
mongo: the IANA name is mongodb
tcp: not an IANA service name
tls: not an IANA name

To mitigate future compatibility issues, it may be prudent to align Istio's use of the appProtocol property with the Kubernetes documentation. For example, tcp could become istio.io/tcp.
I imagine the current values would continue to be honored by Istio for backward compatibility, but new naming conventions would be encouraged.
Version
istio 1.16

Additional Information
No response

There has been some discussion in Kubernetes about this as well, as ~everyone is doing this. cc @robscott


Scribble hub extension

Steps to reproduce
0 chapters in all novels
Whenever i enter a novel it has 0 chapters however whenever i use webview there are chapters but it doesn't show in LNreader
Expected behavior
Example: World Keeper(novel)
In webview there are 940 chapters
However in LNreader there are 0
Actual behavior
0 chapters
LNReader version
1.1.15
Android version
Android 12
Device
Redmi 8a(custom rom)
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with a source, I should be opening an issue in the sources repository.
[X] I have updated the app to version 1.1.15.
[x] I will fill out all of the requested information in this form.

If this is an issue with a source, I should be opening an issue in the sources repository.


bug: different pdf text extraction output from unstructured and unstructed-inference libarary

Describe the bug
PDF text extraction by pdfminer works differently in unstructured repo and unstructured-inference repo.
To Reproduce
PDF: algebra-graph-level1-1.pdf

unstructured-inference repo

# unstructured-inference repo

layout = process_file_with_model(
    filename="algebra-graph-level1-1.pdf",
    model_name=None,
)

elements = layout.pages[0].elements

print("\n\n".join([str(el) for el in elements]))
print(f"n_of_elements: {len(elements)}")

unstructured repo

# unstructured repo

elements = partition_pdf(
    filename="algebra-graph-level1-1.pdf",
    strategy="fast",
)

print("\n\n".join([str(el) for el in elements]))
print(f"n_of_elements: {len(elements)}")

Expected behavior
The PDF text extraction output should be the same.
Additional context
The logic for extracting PDF text using pdfminer was implemented separately in both repositories.

completed via #2158

Completed via #2158


Option to log duplicate keys #1101

Hello, thanks for the great work on snakeyaml project.
I’ve been using either direcly on maby of the projects I work and indirecly (for instance smallrye-config).
In some of the teams I partecipate, sometimes some not very disciplined developer entered some duplicate keys in yaml configurations.
As far as I understand, right now SnakeYAML has an all or nothing approach to duplicate keys (using LoadOpttions.isAllowDuplicateKeys()): either an exception is thrown or they are totally ignored.
What about adding something like a isWarnOnDuplicateKeys() option to log duplicate keys?
https://bitbucket.org/snakeyaml/snakeyaml/issues/1101

Thanks @maslovale, any other action is required from me on the original issue or will you take care of it?


Support Proxy a in FFI imports

In cases where you make heavy use of generics, you need something to determine a type parameter.
Take for example:
foreign import java unsafe "@interface addAll" addAll ::
  (Extends a Object, Extends b (Collection a)) => b -> Java b Bool

(from https://gist.github.com/amir/0782874fb5bfe40fddc56a7e37735877#file-mycollections-hs-L40)
Note that when you use addAll a for some a that only determines the type of b but there's nothing to determine the type of a causing the Extends' type family to get stuck with a compilation error (see the Gist above). The goal is to support something like:
foreign import java unsafe "@interface addAll" addAll ::
  (Extends a Object, Extends b (Collection a)) => Proxy a -> b -> Java b Bool

So that the user can specify addAll (Proxy :: Proxy a) a1 to determine the a type variable. Note that the Proxy a type has no runtime representation and contributes nothing to determine the Java method signature of the imported method.

Closing this because implementing Type Applications will solve this. Updates on Type Applications are tracked on #799


Django 5.0.x warnings with examples from documentation

When I just adopted the examples given in the README like
    logo = PictureField(
        upload_to='logo',
        blank=True,
        null=True)

I got thiese warnings:
`WARNINGS:
techtree.Item.logo: (fields.E101) width_field and height_field attributes are missing
	HINT: Please add two positive integer fields to 'techtree.Item' and add their field names as the 'width_field' and 'height_field' attribute for your picture field. Otherwise Django will not be able to cache the image aspect size causing disk IO and potential response time increases.

Inside example app I see there is some more code – but would be nice if the simplest examples in the README doesn't produce a warning ;)

Hi @frlan 👋
Thanks for reaching out. That is an excellent point, the "tutorial" (of sorts) should be exemplary. Would you care to extend the documentation yourself?
Cheers!
Joe

Closed due to inactivity.


Update rules_scala to v6.1

Update rules Scala to the latest version https://github.com/bazelbuild/rules_scala/releases/tag/v6.1.0 which correctly register the java toolchain

lgtm


update(falco/values.yaml): use the new grpc address introduced in falco 0.33

Signed-off-by: Aldo Lacuku<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-chart

/area falco-exporter-chart

/area falcosidekick-chart

/area event-generator-chart

What this PR does / why we need it:
This change the grpc socket path from unix:///var/run/falco/falco.soc to unix:///run/falco/falco.sock. Please note that this change is potentially a breaking change if deploying the event generator alongside Falco < 0.33.0.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Checklist

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: alacuku
Once this PR has been reviewed and has the lgtm label, please assign cpanato for approval by writing /assign @cpanato in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold

LGTM label has been added.  Git tree hash: 8e9feb13cdcb75744b140e60f96faf1879d5876b

/unhold


Add failing test for keeping the event data after a delayed transition

See #1155.

It should be pretty trivial to put the original event on the special xstate.after event.

Based on the comment, I tried to fix the test myself for a while but I'm not yet deep enough in the codebase to understand all the things that are going on inside the StateNode.ts file. 😅 I'm afraid, the best thing I can do at the moment is:

Closing based on https://github.com/davidkpiano/xstate/issues/1155#issuecomment-778832957


Release v5.4.0

Added Declarations metric
Replaced json2csv to json-2-csv module.

Coverage increased (+0.01%) to 97.62% when pulling 732c82093ac735afb72dab1e95afd3f004ea4ca1 on develop into 5166c693931dfcbd0e0a02edb8444daa26b9abcd on master.


zsh: command not found: draw

Describe the bug
Was hoping to play with draw but I cannot seem to access it post-install.
To Reproduce
Steps to reproduce the behavior:

Install go (using instructions on https://go.dev/doc/install)
Install draw (using instructions provided in README.md)
Restart the terminal, check to make sure the go command is in the path
Execute draw and see the error

Expected behavior
The draw command executes the program
Screenshots

Desktop (please complete the following information):

OS: Ubuntu 22.04

Additional context
I am not used to running go applications and I am unsure of where they are installed. I feel like I am just missing something but, if I am, I am raising the issue because the missing "something" is undocumented.
Thanks! Looking forward to playing with draw

Super late but you need to add go's bin directory to your path. By default it should be in $HOME/go/bin
In your .zshrc add
export PATH="$HOME/go/bin:$PATH"


Enable nullable for FileAppender, RollingFileAppender, and unit tests

For #118 .
And fix null issues. Also includes adding friend reference for unit tests to remove reflection code across unit tests for type safety, removing ArrayList usage in favor or strongly typed lists, and a few opportunistic auto-properties and other modernizations. Doing this required restoring strong name for log4net.Tests, which then led to converting 3 NExpect uses to Assert and removing the NExpect reference, as it is not strong-named and net462 requires strong named assemblies to use only strong named assemblies.

@erikmav I've created a new branch and PR because I couldn't resolve the conflicts in the old one.
#119


useQuery not updated after local resolves

version & exchanges:
latest
Steps to reproduce

in graphcache, create a local resolver that returns undefined
create a query that will use that local resolver , a request is made, but after response (saw it in network ), useQuery didn't update react state and got an fetch successful , the data field is null which is weird

Expected behavior
if undefined is returned , and request is made, state should have that response but not
Actual behavior
request made, response received, but use Query no got update

Sorry, but we have examples and templates including example APIs for creating reproductions, so the minimum we ask for is either more investigation work or a small reproduction, since that really just takes a few minutes in the examples we have 😅 We can reopen this issue once confirmed or answer related usage questions here


Implement 'cancel' method for result of runner.run() (closes #270)

\cc @AlexanderMoskovkin @kirovboris
\cc @VasilyStrelyaev  - please, check comments and strings

/r-

FPR

\r-

@testcafe-build-bot \retest

lgtm

lgtm

lgtm
@VasilyStrelyaev is it all ok now?

yep, lgtm


Move the current (source|destination).address to (source|destination).socket.address?

and add a new (source|destination).address that is equivalent to (client|server).address.
originally from https://github.com/open-telemetry/semantic-conventions/issues/206#issuecomment-1651174945

Closing, I believe this is addressed now with network.(peer|local).address


Spotlight storybook docs and code snippets

What
Add spotlight storybook docs and code snippets
Why
https://skype.visualstudio.com/SPOOL/_workitems/edit/3552733
https://skype.visualstudio.com/SPOOL/_workitems/edit/3552749
How Tested
Tested locally deployed storybook
Process & policy checklist

[ ] I have updated the project documentation to reflect my changes if necessary.
[ ] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

Holding off merging this until I am certain it will be part of our next beta release


Security Bug found when attaching supporting documentation on Purchase Invoice

We have identified a security bug when creating new purchase invoice and attaching supporting documents via attachments.
Where multiple users are entering data the attachments from the other open purchase invoice are visible from any other purchase invoice.
Eg
User A creates an Open Invoice A and Attachment A. Attachment A is visible to user B who is working on Open Invoice B

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0fb5508f-038b-e7e3-e18d-f34a24b9304b
Version Independent ID: 63f68f86-591b-44c5-def4-cfd1eed57a88
Content: Process Incoming Documents - Business Central
Content Source: business-central/across-process-income-documents.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: sgroespe

Hello,
Thank you for your feedback! We can't recognize this problem in the base application (and we also see it as a problem with confidentiality rather than security), so you might want to reach out to your support contact to find out why you are seeing this behaviour.
I'm closing the issue because it's not related to the docs.

this is definitely a security issue. What happens if the ceo is place a confidential purchase order relating to a take over bid and someone in the business sees this document that would be a massive breach of sec regulation.
I would suggest strongly that this issue be escalated to the necessary channels.
Thank Eva
Kind Regards
Ben Opit
+61411707070

On 31 Mar 2020, at 7:53 pm, Eva Dupont<EMAIL_ADDRESS>wrote:

Hello,
Thank you for your feedback! We can't recognize this problem in the base application (and we also see it as a problem with confidentiality rather than security), so you might want to reach out to your support contact to find out why you are seeing this behaviour.
I'm closing the issue because it's not related to the docs.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hello again, Ben,
Again, I urge you to contact support about this issue. My team and I cannot help you. If you are not sure of how to do that, then please check here: https://docs.microsoft.com/en-au/dynamics365/business-central/product-help-and-support#report-a-problem
BR
Eva Dupont


API：Get all available stamps for this node    404

bee version:0.5.3

Are you still facing this after the upgrade? What version are you running?

I did not upgrade，the current version is 0.5.3

This API endpoint is only available on v0.6.*


[jest-plugin-fs] How do I unmock files which should be read?

Hi @ms88privat, sorry for the lack of docs, I actually just added that feature. Here's an example, hope it helps:
https://github.com/negativetwelve/jest-plugins/blob/master/packages/jest-plugin-fs/src/__tests__/fs.spec.js#L80-L90
fs.unmock(arrayOfPathsToFilesToUnmock);

where arrayOfPathsToFilesToUnmock is an array of file paths. This reads in the actual file and inserts it into the virtual filesystem

Thank you. I got this error now: Invalid file signature. I'm trying to load .png files. Any idea?

Ah interesting, it only works with text based files. Can you help me understand the usecase for unmocking a png file? I would love to support your use case but it'll require some changes

I'm also not so sure about my use case, I just tried to find a solution to my problem, that my jest tests are extremely slow when it comes to the file system. So I'm trying to mock it as much as possible. The real use case would be image comparison.
This line Jimp.read(path); takes about 4 seconds to execute inside a jest test. If I invoke the same code via node it is almost instant.

Interesting, is that line run in a beforeEach block? if so, it's executing once per test, not just once. To manually mock, try this (pseudo code):
const image = Jimp.read(path);

describe('my tests', () => {
  beforeEach(() => fs.mock({[path]: image}));

  it('should do something', () => {

  });
});

basically read the image using your preferred reader outside the tests (only once), and then mocking the path with the contents of the file in the beforeEach block

I only got one test. Just reading the images takes ages. So mocking the rest afterwards will also not help. I couldn't test loading the image outside the test because I need it inside and it is also a promise.

Going to close this issue since I think it's specific to your test. If you think otherwise, please feel free to re-open and update us on where things are at!


Suggestion: Mono Noteblock support

just like mono jukebox, is it possible to add an option to make noteblock sound to be mono? ty in advance! ^^

yeah i'd totally add that, tho i thought that it was already in mixtape as i made it from the music type and they're in the same music type. Ill look into it next time i work on mixtape.

alrighty, I'm looking forward for it! (including the 1.21 port hehe) :)


It'd be great to have a gosop compiled with x/crypto/openpgp

I don't know how much the API diverged, and how much work that would be.  If we had a gosop to test x/crypto/openpgp with, we could put it into the test suite.  Maybe that'd even encourage some users to switch to GopenPGP.

It'd essentially entail a rewrite (except for the CLI part, of course). Note that GopenPGP is a wrapper around ProtonMail/go-crypto, which is in turn derived from x/crypto/openpgp - so GopenPGP itself shares no API with x/crypto. GopenPGP also provides a much more high-level API than {x/,ProtonMail/go-}crypto, so a SOP implementation using the latter would need to reimplement a bunch of logic from GopenPGP.
It might be easier to build a version of GopenPGP (v2) that uses x/crypto/openpgp. That should not be too much work since ProtonMail/go-crypto is largely backwards compatible with it, though x/crypto/openpgp might be missing some things that GopenPGP uses - but I'm not sure if those parts are critical to the functionality needed by SOP; that might be worth investigating.

Thanks for providing the details.  I cannot judge how much work that is and what the benefit might be, really, so feel free to close the issue, your call.


sails.js and react-router server side rendering

I want to render my router on server side in sails.but i dont understand how...i tried to create a policy and i added it to all actions of my site controller.and it didnt work.i found some code of old react-router's server side rendering.but what about new router?can someone explain step by step how can i use router on server side?

Thanks for your question!
We want to make sure that the GitHub issue tracker remains the best place to track bug reports and feature requests that affect the development of React Router.
Questions like yours deserve a purpose-built Q&A forum. Could you post this question to Stack Overflow with the tag #react-router? https://stackoverflow.com/questions/ask?tags=react-router.
We also have an active and helpful React Router community on Reactiflux, which is a great place to get fast help with React Router and with the rest of the React ecosystem. You can join at https://discord.gg/0ZcbPKXt5bYaNQ46.


Salesforcerest Connector: Update view does not display field values after connection creation

Description:
After creating a Salesforcerest Connector connection, when navigating to the update view to modify a previously added field value, the update form appears empty. None of the previously added values are displayed in the update form. However, the source view of the connection correctly shows the previously added values.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
v0.9.24071508
OS, DB, other environment details and versions:
Steps to reproduce:

Create a new project.
Goto Local Entries -> Connections. Create a new connection.
Again goto Local Entries -> Connections -> [your created connection].
Compare the appeared form with the source file of the connection.

Related Issues:

The latest release v1.0.4 has resolved the issue. Please reopen if the problem persists.


generateUnreferencedSchema doesn't work for some cases

In my case I can't find a POJO generated for my exception scenarios. For example:

ErrorDto: !include schemas/ErrorDto.raml
...
500:
body:
application/json:
type: ErrorDto`

ErrorDto is not generated.
@stojsavljevic , wht do you think about it? Am I missing something in config? I have:
<generateUnreferencedSchemas>true</generateUnreferencedSchemas>

@stojsavljevic , can you help? Is it a proper bug or some limitation we can circumvent?

@stojsavljevic , please assist.

Hi,
so there are few issues with this:

property (generateUnreferencedSchemas) works only for schemas and not for data types since it will try to generate POJO using Jackson
we're not supporting schemas with 1.0 since RJP10V2RamlRoot returns types in getSchemas() method

We can fix both issues but first one looks easy since we need to check raml version and generate POJOs same way we do that when type is actually used.
Please consider a PR :)

@stojsavljevic, your activity on the 25th made me google Religion in Serbia, just to make sure you are not a crazy workaholic :stuck_out_tongue_closed_eyes:

This ticket will be my next priority.


From zero to flashed, complete instructions for Ubuntu

@dlech , as someone may get stuck, here and there, I've prepared a bit more detailed walk-through for your convenience to flash a Control+ Hub with a freshly built firmware, the complete workflow in a single place.
Enjoy!
sudo apt-get install -y make build-essential libssl-dev zlib1g-dev libbz2-dev \
libreadline-dev libsqlite3-dev wget curl llvm libncurses5-dev libncursesw5-dev \
xz-utils tk-dev libffi-dev liblzma-dev python-openssl

mkdir PyBricks (or your choice)
cd PyBricks
curl https://pyenv.run | bash

edit  ~/.bashrc so that last lines are:
# pyenv & poetry paths you'll need later on
export PATH="/home/<user>/.pyenv/bin:/home/<user>/.poetry/bin:$PATH"
eval "$(pyenv init -)"
eval "$(pyenv virtualenv-init -)"

Then restart your terminal or, just issue:
exec "$SHELL"
Clone the code base to build the firmware you'll upload, if you don't have a firmware.zip:
git clone https://github.com/pybricks/pybricks-micropython
cd pybricks-micropython/
make
make cplushub

Clone pybricksdev as per instructions, install Python 3.8.2 and check it, then install poetry:
git clone https://github.com/pybricks/pybricksdev.git
cd pybricksdev
pyenv install 3.8.2
python -V
curl -sSL https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py | python
poetry install

Skip this if you're fine with Bluetooth on Linux.
Check you have a working bluetooth dongle with the command lsusb , you should see a line similar to this:
Bus 001 Device 002: ID 0a5c:21e8 Broadcom Corp. BCM20702A0 Bluetooth 4.0

Power-on your LEGO hub, launch the command line tool bluetoothctl, then issue a scan on:
[NEW] Controller 00:02:72:3F:BC:F3 <yourpc> [default]
Agent registered

[bluetooth]# scan on
Discovery started
[CHG] Controller 00:02:72:3F:BC:F3 Discovering: yes
[NEW] Device 90:84:**:**:**:3E Pybricks Hub
[NEW] Device DB:A9:**:**:**:19 Mi Smart Band 4
[CHG] Device DB:A9:**:**:**:19 UUIDs: 0000fee0-0000-1000-8000-0080********

(it is always a good idea to have other BLE gears if avaialble in the area, just to double-check everything is fine)
To flash, first power off your hub.
Prepare the command below (and don't hit Enter) in your shell as you're supposed to have just two hand.
poetry run pybricksdev flash ../pybricks-micropython/bricks/cplushub/build/firmware.zip -d 15
Press and keep pressed the green button until it flashes multicolor (red green blue).
Mind the batteries, of course you checked there's plenty of charge in them.
Now run the command above with Enter.
It is safe to remove your finger from the green button once you see the progress bar moving after the update, now it is uploading the firmware (or when your finger becomes blue and cold, whichever comes first).
Creating firmware
Searching for LEGO Bootloader
Found: 90:84:**:**:**:3E
Connecting to 90:84:**:**:**:3E
Connected successfully!
Erasing flash and starting update
100%|███████████████████████████████████████████████████| 157k/157k [02:41<00:00, 971B/s]

You're done!

This is documented in the README now.


add immutable feature

How to categorize this PR?

/kind enhancement
/area os
/os garden-linux
What this PR does / why we need it:
add immutable feature to gardenlinux
Which issue(s) this PR fixes:
Fixes #1353
Special notes for your reviewer:
Release note:

replaced by #1435


Fix Future Creation and Memory Management

Switch to Factory Method for Future Creation
Postpone Message Deletion until Release
Update Docs and Example

Creating futures in the default constructor led to spurious future creation when pup'ing. Instead, the default constructor is now a no-op, so we may want to hide it and make it accessible only when pup'ing. Future creation is now explicit via a factory method.
Deleting the message after the first get led to failures when calling get multiple times. Instead, message deletion is now delayed until release which checks to see if there is a msg and deletes it.
Also added a sensible error message for currently unsupported use-cases (i.e. non-local value retrieval).

Instead of interfering with the default constructor, would it work to leave it unchanged and instead define an empty future(PUP::reconstruct) { }?

Instead of interfering with the default constructor, would it work to leave it unchanged and instead define an empty future(PUP::reconstruct) { }?

Oho! I wasn't aware of this method for enabling PUP recreation. This is much better ^_^


adding alt to slide

adding attribute to the link and pass them on the slide.
So now, adding alt will be passed, enabling using pinterest pin it extension, sharing the image, etc
Other attributes can also be added, not sure it is needed

The alt-attribute is not a valid attribute for anchors:
http://stackoverflow.com/questions/14850187/is-it-correct-to-use-alt-tag-for-an-anchor-link


Impossible to run LUIS model from a non 'westus' region

Describe the bug
It is impossible to run LUIS model from a non "westus" region.
Even changing it on the settings results in a non working model (publishing works, running it doesn't).
Version
v0.0.3
Browser

[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge
[X] Electron

OS

[ ] macOS
[X] Windows
[ ] Ubuntu

Expected behavior
Need to run LUIS model from a non "westus" region (e.g.: "westeurope").

Fix in #3029 for different luis region publish but can not specify with other language.

could I close it as more comments in #3035

Yes, let's close this. Thank you! :)


Cleanup cpp_linter source code starting from v2

Since we have already abstracted python sources from cpp-linter-action to cpp-linter repo, and we'll change action from using docker to composite steps, it's ready to clean up cpp_linter related code starting from v2 release.

The docs should be altered (or removed), so that the only the README (& possibly the new contributing guidelines) are rendered. The API reference pages will no longer be needed if the python source code is removed. In fact, removing the python sources without altering the docs may actually break docs' build.

The dev-test.yml CI and the coverage badge should go too.

If https://github.com/cpp-linter/cpp-linter-action/pull/83 is ready to merge, it should be simple to merge it and then clean up the source code to avoid conflicts.

Submitting a review shortly


Manual / user-defined intrinsic & extrinsic calibration?

Thanks for creating this SDK (mostly) open source. It's very useful already.
One thing I noticed today: It's not possible to replace the factory camera calibration by another calibration that is created manually.
I haven't yet tested how good the factory calibration is. It seems to be pretty good, judging only visually, but maybe not perfect. Overlaying depth and color shows some slight misalignments towards the edges of the image frame (where effects of distortion are the strongest).
Do you plan to add any functionality for the user to supply a custom calibration?
Looking at other products like stereolab's ZED camera, you can see that they provide a calibration tool with user feedback, which handles storage and calculation of a device's intrinsics & stereo extrinsics automatically.
Storing the calibration could be done in a few ways:

Provide a file format and function with which you can load from / store to the disk. This would replace getting a calibration from a device handle in such applications
Provide setter-functions so that the custom calibration can be stored on the device. Don't know if this is supported by the hardware. The factory calibration would remain available at all times, should the user screw up the calibration somehow.

Calculation could be done by:

Creating functions which can calculate intrinsics & extrinsics from calibration target & measured  points. There could be an example / tool built around these functions which display a checkerboard on screen, uses OpenCV to detect the checkerboard, and can calibrate automatically (with some user feedback)
Using OpenCV, such as in other examples, to detect a calibration target (checkerboard), but also to estimate distortion parameters. The distortion model (Brown-Conrady) used seems to be compatible with OpenCV currently anyway. IIRC OpenCV would also support a rational model, but I don't know if it aligns with Azure Kinect SDK's model.

@fishcu  thank you for the feedback. I can add few comments on your observation of the depth color overlay at edges.

There are some doc here describe the functions and also the reference code for these transformation functions are open sources in the Github.
Recommend to use the depth to color camera transformation since it handles occlusion. Due to performance reason, the color to depth transformation does not handle occlusion.
If the slight misalignment on the edge you are talking about is not from occlusion, then please keep in mind there are multiple reasons can lead to this slightly misalignment. e.g. the resolution of the color and depth are quite different, therefore, during transform, due to resolution difference, slightly imperfect edge in depth can be amplified in color; calibration accuracy is one possible reason too; other than that, depth accuracy, even it is quite small, but can contribute to this as well since we are using depth value to transform.

Currently there is no plan to add support of overriding the factory calibration. You still can do your own calibration and just create the k4a_calibration_t object from your own calibration file format. The object type is defined in https://microsoft.github.io/Azure-Kinect-Sensor-SDK/master/structk4a__calibration__t.html. One example, around L230 ish, since it create a new extrinsic calibration between two devices, and to help the transformation, it create a new "virtual" calibration object by using the manual calibrated extrinsics and the factory intrinsics, then use it later in the code.
Please let us know whether this can unblock you.

Thank you for the useful summary.
I have one more question: Is it possible that not all distortion models are implemented yet? I could only find the relevant code for the standard Brown and RATIONAL_6KT models. I have not tried undistorting the WFOV depth yet, but I can imagine that these models might not be the best fit because of the fisheye lens. I'm guessing that K4A_CALIBRATION_LENS_DISTORTION_MODEL_THETA would correspond better to the fisheye model.

@fishcu Documentation quote "Azure Kinect devices are calibrated with Brown Conrady." The lens is not a fisheye lens, it is a normal lens with wide field of view. The team did the calibration have verified rational6kt or brownconrady is the best fit for this camera. To be compatible with opencv, browncontrady was chosen for the final product factory calibration.
Yes, there are some enum value in the calibration model type (theta/polynomial) was not implemented (they are legacy code we should clean up, I will send a PR for that)

I think it will be a good idea to have the functionality to override factory calibration because of #803.

@rabbitdaxi
In your comment of  9 Sep 2019, you referenced an example. The link is incorrect. I would like to review this example. Could you please provide the link you intended?

@interwebjill corrected the link inline in the comment I made. Hope it helps


Change canvas axes for image placement.

Motivation
When running displaying images on the canvas, for example running: sct_label_utils -i t2.nii -create-viewer 3,4 . The output image was overlapping the instructions making it a bit uncomfortable to read without zooming out.
Description
Fixes image positioning on canvas to avoid overlapping with instructions.
Screenshots

Fixes: #2357

@charleygros Thank you!
@charleygros, @jcohenadad Are we good to merge it?

@gmotzespina Good for me :)


option to set directory

Problem Description
I would like to set directory before running task.  For example, if I have sub directory in the project, I want to use that as directory for my task.   The directory should be relative directory from project.
Fore example, if I have subdirectory "sub1", then
Code Example
[tasks.do_something]
cd = "sub1"
script = [
  "ls"
]

Error Stack/Info (if any)
This should cd to sub1 in the my project and run "ls".

You can change cwd as part of the cli. Or you need it per task?
You can do it as part of the script itself
This is the real question: do we revert back after task ends? How it impacts sub tasks and dependencies? What if a dependency already ran not part of this task so it wont run again? I guess to be as clear as possible this directory should only impact this specific task action and no other task (not dependencies and not sub tasks called via run_task)

What do you think?

For my need, I just need for this task.  I can do this as part of script but is bit of clumsy to do multi-line in TOML file.

this doesn't look clumsy to me, but anyway i'll implement
script = [
'''
cd mydir
ls
'''
]

done. I'll be releasing it soon.

new 0.5.3 version released


docs/operators: fix a typo

s/are preserved the output/are preserved in the output
Signed-off-by: Julian Wiedmann<EMAIL_ADDRESS>

Thanks!


installation instructions insufficient

Tried pulling this down to play with it but webpack wouldn't load. It looks like your webpack.config is in es6 and node doesn't support es6 modules yet. I guess your environment is doing something fancy to get it to work. That's not typical though.
I transpiled the webpack config locally to try to get it working but even then the public directories aren't getting created and filled with the appropriate files.

@jethrolarson yeah, sorry. This was a draft I started half-year ago and didn't have time to finish.
I plan to come back to this project later but can't promise.
Relative imports and ES6...
I usually use bin/install to update webpack and tests to run through babel-node.
So to run this you need

to have Unix env
Run npm install
make sure you use Babel5 (npm install babel@5.?? -g) or issue source bin/env to enter local shell.

Babel6 has introduced a way to alias folders so you can get away without symlinks.
But I didn't try that yet so for now the approach I described is an approach I use for all my projects.


Add bclib to build local

Codecov Report

Merging #284 (94bec62) into main (5a3caf5) will decrease coverage by 4.48%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #284      +/-   ##
==========================================
- Coverage   78.66%   74.17%   -4.49%
==========================================
  Files          16       16
  Lines        3224     3187      -37
==========================================
- Hits         2536     2364     -172
- Misses        688      823     +135

Impacted Files
Coverage Δ

src/compiler/orcv2.jl
0.00% <0.00%> (-72.39%)
:arrow_down:

src/compiler/interpreter.jl
31.11% <0.00%> (-60.38%)
:arrow_down:

src/compiler/utils.jl
44.82% <0.00%> (-37.94%)
:arrow_down:

src/logic.jl
75.00% <0.00%> (-5.00%)
:arrow_down:

src/compiler/validation.jl
60.07% <0.00%> (-2.52%)
:arrow_down:

src/api.jl
48.00% <0.00%> (-1.62%)
:arrow_down:

src/pmap.jl
79.23% <0.00%> (-1.10%)
:arrow_down:

src/Enzyme.jl
85.85% <0.00%> (-0.49%)
:arrow_down:

src/compiler.jl
79.08% <0.00%> (-0.44%)
:arrow_down:

src/typetree.jl
92.52% <0.00%> (-0.34%)
:arrow_down:

... and 2 more

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 5a3caf5...94bec62. Read the comment docs.


Unknown extension type "DS_Store"

It's a hidden file created by Mac OS in many directories. I can be safely ignored for the purpose of this library.

Thanks for submitting this.
I'd be happy to accept a PR for this.

It would be nice to expose to the user some sort of functionality for ignoring arbitrary files.
This might be somewhat similar to how .gitignore files work.


Listener system based on a persistent state

Problem
Right now when an application wants to listen for an event (or result) we create a stream and everytime we get events we go through all the different streams open and check if they match the event. This option is fine but if we need to scale to 100+, 1000+ listeners this will be really hard to have good performances and also with the introduction of the workflow file this is not really relevant. Another thing is that it will be kind of hard to syncronize this over a network.
Solution
We could store all the listeners with their filter in database and based on that for every events/result query the database to find all the listeners that need this event and then send them. The logic will move from the listener part (that filter the event) to the event part (that find the matching listeners).
This way we can be closer to what the workflow file will look like and have pseudo code like this when we receive event/result
func receiveEvent(event) {
  listeners := db.findListenersFromEvent(event)
  for listener := range listeners {

    // To manage the case of an application that listen the api stream
    if hasStream(listener) {
      stream(listener).send(event)
    }

    // To manage the case of a workflow file
    if hasExecution(listener) {
      execution(listener).execute(event)
    }
  }
}

This would be helpful for the following issues https://github.com/mesg-foundation/core/issues/378, https://github.com/mesg-foundation/core/issues/195

pinging myself

Moving to the forum https://forum.mesg.com/t/listener-system-based-on-a-persistent-state/17