[HOLD https://github.com/Expensify/Auth/pull/6962] Server side report search

This needs the following PRs:

Auth PR: https://github.com/Expensify/Auth/pull/6962
App PR: https://github.com/Expensify/App/pull/10342

Details
Add server side chat search:

Needs new command SearchForChats in Web-E: https://github.com/Expensify/Web-Expensify/pull/34559
Input text change will trigger sending a request to SearchForChats to fetch additional chats from the server, this is debounced by 300ms
Reports obtained from SearchForChats are stored in the same report collection (COLLECTION.REPORT). Then, when options are created, there is no difference from reports fetched during the app start or during the search.
Added componentDidUpdate to SearchPage to update options if props.reports change (possibly because we got reports from the server chat search)

Fixed Issues

$ https://github.com/Expensify/Expensify/issues/221789 (+ server side search)
Tests
Checkout:

Auth PR: https://github.com/Expensify/Auth/pull/6962
App PR: https://github.com/Expensify/App/pull/10342

Log in New Dot
Go to your preferences (Click Avatar -> Preferences) and change the Priority mode to #focus
Logout
Open chrome dev tools, network tab
Login again and search for the request "OpenApp", the last element in the array under onyxData in the response should contain only the reports that you see in the LHN in #focus mode:

Open the search menu to open a new chat, test searching some chat that is not in the bar currently and a room (i.e. #announce), it may take a bit because they are fetched from the server but they should appear.
Test that you can open a room like #announce
Check that while you type in the search bar a request is sent with command SearchForChats:

this request pull chats based on the searched text

[ ] Verify that no errors appear in the JS console

PR Review Checklist

Contributor (PR Author) Checklist

[ ] I linked the correct issue in the ### Fixed Issues section above
[ ] I wrote clear testing steps that cover the changes made in this PR

[ ] I added steps for local testing in the Tests section
[ ] I added steps for Staging and/or Production testing in the QA steps section
[ ] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I included screenshots or videos for tests on all platforms
[ ] I ran the tests on all platforms & verified they passed on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[ ] I followed proper code patterns (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I followed the guidelines as stated in the Review Guidelines
[ ] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

PR Reviewer Checklist
The Contributor+ will copy/paste it into a new comment and complete it after the author checklist is completed

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] iOS / native
[ ] Android / native
[ ] iOS / Safari
[ ] Android / Chrome
[ ] MacOS / Chrome
[ ] MacOS / Desktop

[ ] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] Any functional components have the displayName property
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

QA Steps
The QA should wait for the PR in Auth and App to be deployed, otherwise it doesn't make much sense

Auth PR: https://github.com/Expensify/Auth/pull/6962
App PR: https://github.com/Expensify/App/pull/10342

Log in New Dot
Go to your preferences (Click Avatar -> Preferences) and change the Priority mode to #focus
Logout
Open chrome dev tools, network tab
Login again and search for the request "OpenApp", the last element in the array under onyxData in the response should contain only the reports that you see in the LHN in #focus mode:

Open the search menu to open a new chat, test searching some chat that is not in the bar currently and a room (i.e. #announce), it may take a bit because they are fetched from the server but they should appear.
Test that you can open a room like #announce
Check that while you type in the search bar a request is sent with command SearchForChats:

this request pull chats based on the searched text

[ ] Verify that no errors appear in the JS console

Screenshots

Web

Mobile Web

Desktop

iOS

Android

Testing in Android...

I noticed a small bug/problem:

Log in when priority mode is #focus, it won't load all your reports
Change the priority mode to Most recent, your reports are still not there :(

My normal thought would be, well lets chain a request to fetch the missing reports, but that is not very New Dot Api 1:1:N... do you have a suggestion on this @marcaaron ?
One option is that the API command UpdateChatPriorityMode returns the reports as onyx updates when you change to Most recent, does that sound wrong or good? When we change to #focus, there is no need to send the reports from the API.

One option is that the API command UpdateChatPriorityMode returns the reports as onyx updates when you change to Most recent, does that sound wrong or good? When we change to #focus, there is no need to send the reports from the API.

Agree 100%, this sounds like exactly what we should do to me.

Closing this for now as we no longer require a quick temporary fix


host import function declaration order causes incompatible import types

Hi there! I'm just starting to explore using wasm to create a plugin framework.
I was starting to play with host import functions, and noticed i was unable to have more than one.
Host
let log_str = Func::wrap(&mut store, exports::log_str);
let instance = Instance::new(&mut store, &module, &[ log_str.into() ])?;

Wasm
extern "C" {
    fn log_str(ptr: i32, len: i32);
}

Output
[17:50:14.104] Hello from wasm!
If i now add a second function:
Host
let get_assembly_count = Func::wrap(&mut store, exports::get_assembly_count);
let log_str = Func::wrap(&mut store, exports::log_str);

let instance = Instance::new(&mut store, &module, &[get_assembly_count.into(), log_str.into() ])?;

Wasm
extern "C" {
    fn get_assembly_count() -> i32;
    fn log_str(ptr: i32, len: i32);
}

Output
[17:56:14.791] [ERROR] Failed to load mod: incompatible import type for `env::log_str`
I've looked through the documentation/examples/issues and i've never seen this mentioned, so i'm a bit lost

Even more perplexing behavior, if i switch
let instance = Instance::new(&mut store, &module, &[get_assembly_count.into(), log_str.into() ])?;

to
let instance = Instance::new(&mut store, &module, &[ log_str.into(), get_assembly_count.into() ])?;

it now works...
[18:02:18.073] Hello from wasm!
[18:02:18.073] There are 97 assemblies

Thanks for the report! As the documentation for Instance::new indicates:

The entries in the list of imports are intended to correspond 1:1 with the list of imports returned by Module::imports

You can't rely on the order in the source being the exact same order in the output module. When using Instance::new you'd instead need to use Module::imports to learn what order to pass things in.
Instead, though, as Instance::new indicates, it's recommended to use the Linker type which does name-based resolution which is generally easier to work with.

Ahh thank you, the Linker does indeed solve that problem!


blob jar file

Hi,
I see there's a blob jar file in gradle/wrapper/gradle-wrapper.jar
Prebuilt binaries are not welcomed in Debian and have to be removed from source tarball. While I can do this downstream, it's always better to not include them from the upstream (and make the repo clean). Please consider removing it from repo (or source tarball).

Hi,
I don't like it either, but this is "common practice" for Android projects.

The scripts generated by this task are intended to be committed to your version control system. This task also generates a small gradle-wrapper.jar bootstrap JAR file and properties file which should also be committed to your VCS. The scripts delegates to this JAR.

https://docs.gradle.org/current/dsl/org.gradle.api.tasks.wrapper.Wrapper.html
https://stackoverflow.com/questions/20348451/why-should-the-gradle-wrapper-be-committed-to-vcs
Removing it would probably annoy some users. We can still not use it if we don't want to:
https://github.com/Genymobile/scrcpy/blob/31bd95022bc525be42ca273d59a3211d964d278b/release.sh#L12
Keep the issue open, though. I would like other opinions about this.


Store tip metadata through set_metadata instead of a remark

Remark is not possible at the moment and breaks the referenda.
(But might be possible after https://github.com/paritytech/substrate/pull/14260). Still, metadata is preferred.
### Tasks
- [x] Fix & Deploy referenda
- [x] Reapply https://kusama.polkassembly.io/referenda/205 -> https://github.com/paritytech/substrate/pull/13993,
- [x] Reapply https://kusama.polkassembly.io/referenda/206 -> https://github.com/paritytech/substrate/pull/13027,
- [x] Reapply https://kusama.polkassembly.io/referenda/207 -> https://github.com/paritytech/cumulus/pull/2294
- [x] In each new referenda, on polkassembly, leave a comment that it's re-submitting a tip for according PR due to this issue

New referenda:

https://kusama.polkassembly.io/referenda/216
https://kusama.polkassembly.io/referenda/217
https://kusama.polkassembly.io/referenda/218

cc @mordamax

Solved by https://github.com/paritytech/substrate-tip-bot/pull/79


Add the variant identity to the seenArtifacts key

In ResolvedArtifactCollectingVisitor, the artifacts are identified by ResolvableArtifact.getId(), so in case of ComponentFileArtifactIdentifier the artifact is identified by the componentId and fileName which could be the same for artifacts in the same module (the same ProjectComponentIdentifier) and producing the same fileName (but the path is different).
The problem exists only for artifacts with different capabilities as a node in the dependency graph has to have all the artifacts attributes matching, but they could have different capabilites, and so it's important to use the capability when identifying the artifact.
This change adds the variant attributes and capabilites as part of the key in ResolvedArtifactCollectingVisitor to solve this case.
Fixes #18458

This solution does not work - see comment on #18458


call socket.setKeepAlive after the connect event

https://github.com/nodejs/node/issues/31663

Codecov Report

Merging #2074 (5638850) into master (329885b) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master    #2074   +/-   ##
=======================================
  Coverage   95.25%   95.25%
=======================================
  Files         387      387
  Lines        3516     3516
  Branches      423      423
=======================================
  Hits         3349     3349
  Misses         83       83
  Partials       84       84

Impacted Files
Coverage Δ

packages/client/lib/client/socket.ts
80.90% <0.00%> (ø)

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 329885b...5638850. Read the comment docs.


#427 - Crash on mxd open redo

Additional fix/workaround for issue where: addin compiled on earlier version still crashes on versions 10.5.1+
Had no idea why it was crashing but just added exception handler. You can't see the diff for designer.cs file in Github but here is the change:

Why is there a code change in an 'auto-gernerated' file?

@jmccausland - Good question - it was that way before I touched: https://github.com/Esri/coordinate-conversion-addin-dotnet/blame/dev/source/CoordinateConversion/ArcMapAddinCoordinateConversion/Config.Designer.cs

This is very alarming.  There's a lot of .NET NoNo's merged in the code in the last year or so.


Path to config file inside OTIO file?

Could we put the path to the config file inside the OTIO file?
That way if we need to re-run a test, we know which config file was used.

Good idea. I'll add that


Switch to fhir values for route to facility

Proposed Changes

Switch to fhir values for route to facility
closes https://github.com/ohcnetwork/care_fe/issues/9511

Merge Checklist

[ ] Tests added/fixed
[ ] Update docs in /docs
[ ] Linting Complete
[ ] Any other necessary step

Only PR's with test cases included and passing lint and test pipelines will be reviewed
@ohcnetwork/care-backend-maintainers @ohcnetwork/care-backend-admins

Summary by CodeRabbit
Release Notes

New Features

Introduced a route_to_facility field in patient consultations to enhance routing information.
Added new facility records and updated existing ones in the facility data structure.
Enhanced patient registration with a link to the last consultation.

Bug Fixes

Improved validation logic for the route_to_facility field to ensure data integrity.

Tests

Updated test cases to reflect changes in the route_to_facility handling and improved error messaging in user modification tests.

Chores

Refined the database reset process for improved efficiency.

@coderabbitai review


AsyncAPI spec v3 support in .NET Nats template

Reason/Context
This Issue is used to track changes needed to support AsyncAPI v3. As a code owner, please edit this list of TODO tasks in order to properly track the progress 🙂 Once this issue is closed it means that v3 is now fully supported in this library.
Remaining tasks:

[ ] Update template to use parser v2, this is already supported by the generator by changing the apiVersion
[ ] Update Modelina dependency once released - https://github.com/asyncapi/modelina/issues/1376

still relevant


v5 updates

@adowling2

v5c patched to apply sine test GP to step test
v5d created to train GP on all dT data as function of T, Q, Qdot; applied to both tests

Results in all tests did not outperform benchmark. May want to maintain two-state benchmark model and find way to use GP for modeling residual, etc.

@jakekrell I agree, this is interesting.
I agree that the next reasonable step is to try training a GP for the residuals of the benchmark model. I think you can do it with just a minor modification to your code, i.e., add an extra step to calculate the residuals.

Added a couple high-level notes for implementing the GP in Pyomo -- though still need to figure out the time indexing in detail.
@adowling2 I opened this PR to track progress of this time indexing development, so that ideally with another FoKL release we can simply call fokl_to_pyomo(GP, ..., m=m, ..., ode=True) when embedding the GP model in Pyomo.

@jakekrell A few more ideas based on our discussion today:

In a new conda environment, install idaes-pse and then idaes get-extensions. This will install Pyomo and Ipopt with HSL linear algebra.
Extract the values for u1 we get from the optimal Pyomo solution and then make predictions using the native FoKL and benchmark models.
I think I know why the Pyomo simulator is not working... Try adding a new variable du and then write a differential equation that du/dt = du. We created a DerivativeVar for dudt to use with the FoKL model. I suspect the Simulator in Pyomo does not know what to do if thee is a DerivativeVar that is not the LHS of a differential equation.


Do not implement ObserverSystem for fn(In<Trigger<...>>, ...)

Objective

Fix #14924
The implementation of SystemParamFunction for fn(In<T>, ...) does not constrain the lifetime of T. Allowing fn(In<T>, ...) to be called by bevy as a fn(T, ...) system would enable users to arbitrarily specify the lifetime of the T value provided by bevy

Solution

Change the associated type In from Input to In<Input> in the implementation of SystemParamFunction for fn(In<Input>, ...), to prevent automatically implementing the XXXSystem trait for fn(In<T>, ...) when implementing the trait for fn(T, ...)

Testing

The code provided in #14924 can no longer compile

Migration Guide

fn(In<Trigger<...>>, ...) no longer implements ObserverSystem

This is a (pretty big) breaking change, as every use of System<In = T> now needs to become System<In = In<T>>. This is also likely why CI is failing.

@SkiFire13 Yes, one solution I thought of is to add a helper trait:
trait HelperTrait {
    type InnerType;
}

Then implement this trait for In<T>, (), and Trigger<E, B>. For example:
impl<T> HelperTrait for In<T> {
    type InnerType = T;
}

impl<E, B> HelperTrait for Trigger<E, B> {
    type InnerType = Trigger<E, B>;
}

It's a bit suspicious to me that we're fixing UB without touching any unsafe code or adding any safety invariants.

I don't think this is a good fix, I'm going to close this PR.


Update CA image for k8s 1.30

How to categorize this PR?

/area control-plane
/kind enhancement
What this PR does / why we need it:
Update CA image for k8s 1.30. (Refer https://github.com/gardener/autoscaler/releases/tag/v1.30.0 for release notes of gardener CA 1.30.0)
Which issue(s) this PR fixes:
Fixes part of #9508
Special notes for your reviewer:
As the CI release job didn't create an update PR so filing the manual one here.
Please check if the release notes are fine.
Release note:

Shoot clusters with Kubernetes version `>= v1.30` will use cluster-autoscaler `v1.30.0`.

Synced changes till v1.30.1 of upstream autoscaler

Added machine cpu architecture support for scale-from-zero scenarios.

Fixed a bug where some default NodeGroupAutoscalingOptions were not considered

@ashwani2k Please fix the release note syntax so that the latter two are accounted for autoscaler component and not for gardener.

Updated kindly check.

Now you removed them 🙈 I think you could still keep them - just the syntax was wrong such that it was treated as release note for g/g (and not for g/a). Pls fix :)

I think I get the intent now.
I've updated again to segregate g/g vs a/g release notes. Can you recheck.

/lgtm

@ashwani2k You're right, that's better. Thank you 😄
/approve


from_many() fails to return Array[Signal]

The method from_many() does not return Array[Signal] because it cannot convert a generic array into a typed array. This is a known issue in Godot, and it may be resolved in a future update.
A recommended workaround is to use Array.assign() to convert the array to the correct type.
https://github.com/godotengine/godot/issues/72566

static func from_many(input_signals: Array[Signal]) -> Array[Promise]:
	var typed_input_signals: Array[Promise]
	typed_input_signals.assign(input_signals.map(
	    func(input_signal: Signal):
	        return Promise.From(input_signal)
		    )
		)
	return typed_input_signals

Ugh, I need to fix the GdUnit setup to catch those issues...
Nevertheless, thanks for letting me know!

No worries! 😁
I've enjoyed how useful this plugin is and could see it becoming a mainstay for all of my future projects.


[CAM-10258] EC2 Instance Role feature for appliance

@gsanchezu @davisein

@gsanchezu I've edited all mentions to Cloud Application Manager Data Center Edition and Cloud Application Manager.

Let's wait to merge it once the feature is pushed to Prod


Add a way for sections to add custom extra methods in the YAML

This makes it possible to define custom extra methods on a generated section class.
For example:
section:
  name: My section
  [...]
  generator:
    entity:
  extraMethods: |-
    public function getFooBar(): string
    {
        return $this->getFoo()->getBar();
    }

(|- is just YAML syntax for multi-line strings)

This is a great idea, but I feel a bit reluctant to code in a yml file. Maybe we can:

Extend from an entity to add custom code. Something like:
class Bla extends GeneratedEntity {
public function getFooBar(): string
{
return $this->getFoo()->getBar();
}
}
Have the code that needs to be added in a separate file another way. Maybe we could use the magic method: __call and have it pass on the call to a custom file.
Maybe we can include a trait into the entity.

Using __call is probably a bad idea because it makes static analysis harder. IDEs can't autocomplete the extra methods or detect non-existing methods that way.
Putting the methods in the YAMLs has the advantage of keeping the YAMLs self-contained, but I don't know if that's really useful.
I like both the trait and the extension idea. I think traits would be best, because none of the existing generated code would have to move around.
It could create a sub-namespace and put empty traits in it that it uses in the classes. So if it generates AppBundle\Entity\Foo, then it also generates AppBundle\Entity\ExtraMethods\Foo, without any methods, and adds use ExtraMethods\Foo to the body of Foo. But it only generates the file for the trait if it doesn't exist, so if you add your own code to the trait then that won't be overwritten.
That has the disadvantage of creating a lot of empty traits, but it has the advantage that you don't need to change the configuration or regenerate anything if you want to start adding extra methods to a section.

Empty traits are generated and used now. They're not overwritten, so it's safe to put custom code in them.
Depends on dionsnoeijen/sexy-field#100.


Kentik synthetics metrics: expose as gauge, not as counter

I think that all metrics which are exposed by Ksynth should be exposed as Gauge (as that is the nature of those metrics), while ktranslate is exposing them as Counters. This makes an issue, since the metrics such as "packets sent" or "packets loss" are increasing over time and the output really acts like a counter.

I think this is good in my testing now. LMK when you can have a chance to validate.

Working now.


Error 400: invalid request

Error 400: invalid_request
The out-of-band (OOB) flow has been blocked in order to keep users secure. Follow the Out-of-Band (OOB) flow Migration Guide linked in the developer docs below to migrate your app to an alternative method.
Request details: redirect_uri=urn:ietf:wg:oauth:2.0:oob

Thank you for reporting. This should now be fixed with this change.


genuine plea for help

please I'm a new beginner and still figuring out how to send out tweets with my automated twitter account. It seems twitter is asking me to apply for elevated access. Is there any way I can access endpoints that allow me to tweet without applying for elevated access?

Yes, you can just use Free access.
https://developer.twitter.com/en/docs/twitter-api/getting-started/about-twitter-api
https://twittercommunity.com/t/announcing-new-access-tiers-for-the-twitter-api/188728


Default directive hooks' binding.modifiers to an empty object

To avoid having to do something like binding.modifiers && binding.modiers.foo to check for the existence of a modifier, should binding.modifiers be an empty object if none are provided?

Implemented.


Unknown module Data.Functor Control.Category, Data.Unit, Prelude

So I've been using Prelude and such for some time and now after installing Purescript-Rx which I suppose depends on Purescript-Control I get these errors. What baffles me most is that I do not understand why it cant see Prelude because minutes ago I successfully imported it with other libs.
Errors:
[1/4 UnknownModule] bower_components\purescript-control\src\Control\Alt.purs:6:1

  6  import Data.Functor (class Functor, map, void, ($>), (<#>), (<$), (<$>))
     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

  Unknown module Data.Functor

[2/4 UnknownModule] bower_components\purescript-control\src\Control\Extend.purs:9:1

      v
   9  import Control.Category (id)
  10
      ^

  Unknown module Control.Category

[3/4 UnknownModule] bower_components\purescript-control\src\Control\Lazy.purs:3:1

     v
  3  import Data.Unit (Unit)
  4
     ^

  Unknown module Data.Unit

My bower:
{
  "name": "front-ps",
  "version": "1.0.0",
  "moduleType": [
    "node"
  ],
  "ignore": [
    "**/.*",
    "node_modules",
    "bower_components",
    "output"
  ],
  "dependencies": {
    "purescript-console": "^0.1.1",
    "purescript-eff": "^1.0.0-rc.1",
    "purescript-prelude": "^1.0.0-rc.3",
    "purescript-functions": "^1.0.0-rc.1",
    "purescript-maybe": "^1.0.0-rc.2",
    "purescript-rx": "^0.8.2"
  },
  "resolutions": {
    "purescript-prelude": "^1.0.0-rc.3",
    "purescript-maybe": "^1.0.0-rc.2",
    "purescript-monoid": "^1.0.0-rc.2",
    "purescript-control": "^1.0.0-rc.1",
    "purescript-invariant": "^1.0.0-rc.1",
    "purescript-console": "^1.0.0-rc.1",
    "purescript-eff": "^1.0.0-rc.1",
    "purescript-functions": "^1.0.0-rc.1"
  }
}

Try clearing out your bower_components and then reinstalling. It looks like you have the old version of the prelude installed alongside the release candidate for purescript-control. I'm not sure this generally is going to work out though, as I don't think purescript-rx has been updated for the release candidate libraries.

I've tried deleting bower_components and only having

  "dependencies": {
    "purescript-prelude": "1.0.0-rc.6",
    "purescript-rx": "^0.8.2"
  },

but now I get
[1/1 DuplicateModuleName]

  Module Control.Apply has been defined multiple times.

           Src   Lib   All
Warnings   0     0     0
Errors     0     0     1
* ERROR: Subcommand terminated with exit code 1

What if you drop the explicit dependency on the release candidate prelude and allow purescript-rx to install its dependencies transitively? As I mentioned before, I don't think the two are compatible, so you'll have to use all pre-release-candidate libraries for now.

Ill try with pre rc and then later try and make a pull request on rx. Thanks


Access Media Stream without Compliance Policy

Hello Team,
I am trying to access media streams.
I wanted to inquire if its possible to access participant media streams without creating creating/assigning any compliance policy request.

@aarushi-mittal
I don't think it is possible to get the media streams of a participant in a call without assigning a compliance recording policy.
But still, @ksikorsk  can provide you with the exact case.


initializing ledger store: open migrates\0-init-schema: file does not exist

After i Download and Install the the pre-compiled binary "numary_x.y.z_Windows-64bit.zip",
I fellow the tutos :
numary config init
numary storage init
numary version
Version: 1.7.1
Date: 2022-08-02T10:37:19Z
Commit: 22b2d2c
numary server start --debug
ok
but when, i start testing
intro.num :
send [COIN 100] (
source = @world
destination = @centralbank
)
numary exec dunshire intro.num
i get :
time="2022-08-10T16:07:33+01:00" level=fatal msg=EOF
in the client side :
numary exec quickstart example.num
time="2022-08-10T05:44:44+01:00" level=fatal msg=EOF
and this error in the server side:
time="2022-08-10T16:18:02+01:00" level=debug msg="QueryRowContext: SELECT ledger FROM ledgers WHERE ledger = ? [dunshire]"
time="2022-08-10T16:18:02+01:00" level=debug msg="ExecContext: INSERT INTO ledgers (ledger, addedAt) VALUES (?, ?) ON CONFLICT DO NOTHING [dunshire 2022-08-10 16:18:02.8947982 +0100 CET m=+7.719215001]"
time="2022-08-10T16:18:02+01:00" level=debug msg="Initialize store"
time="2022-08-10T16:18:02+01:00" level=info msg=Request ip=<IP_ADDRESS> latency=2.3679ms method=POST path=/dunshire/script status=500 user_agent=Go-http-client/1.1
time="2022-08-10T16:18:02+01:00" level=error msg="initializing ledger store: open migrates\0-init-schema: file does not exist"
ps: I use sqlite as Driver
...
server:
http:
basic_auth: ""
bind_address: localhost:3068
storage:
cache: true
dir: c:/.numary/data
driver: sqlite
postgres:
conn_string: postgresql://localhost/postgres
sqlite:
db_name: numary
ui:
http:
bind_address: localhost:3068
version: 1.7.1

@ZinZingoo Hello! Thks for the report.
The bug is on Windows version.
The PR which fix it is https://github.com/numary/ledger/pull/312
Will be reviewed and merged probably tomorrow.


add menu of browse options

(see current yamz.net): high score | recent | volatile | stable | alphabetical

Visit yamz.net, click "Browse", then the UI in question appears.

As a counterpoint to @jkunze's original message, there is a UI/UX (anti-)pattern relating the number of options affecting whether an explicit list is better/worse than a dropdown. @jkunze Do you anticipate many more browsing options? If not, hiding these other options, while it would clear up the display, also means "one additional click" for sorting.

I don't know. The change I'm suggesting would essentially restore on yamz.link missing functionality that's present on yamz.net. I'm certain that original functionality could benefit from further UX review for.

Ok, I misunderstood.

Here is yamz.link missing the above pictures horizontal list of links. We might also consider adding a "Sort By" label to make it clear that it is a different view on the same data rather than incurring some processing, as implied by them being naked, floating hyperlinks. Some indicator, even if not a label, would be helpful and more accessible.

High Score is just a sort by Score descending- the same thing as if you were to click on the column header. Do we want to make it a top 10 or something so there aren't two different labels for the same thing? Stable will return for every term right now because no one is editing terms. Recent is again sort by date descending- do we make it within some target period?
Alphabetical- we might consider using clear filter or something like that since the default view is a paged sort alphabetical.


Raise when trying to acquire in R/O or missing folder

Signed-off-by: Bernát Gábor<EMAIL_ADDRESS>

I'm not sure we can make the soft file lock deadlock safe 🤦 but will keep trying 😟


PLT-6896 Add paging to system console log viewer

Summary

Add paging to system console log viewer
Set default value of per_paging for logs to 1000

Ticket Link
PLT-6896
#6724
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Ran make check-style to check for style errors (required for all pull requests)
[ ] Ran make test to ensure unit and component tests passed
[ ] Added or updated unit tests (required for all new features)
[x] Has server changes (please link)

PLT-6896 per-paging for logs by kaakaa · Pull Request #7903 · mattermost/mattermost-server

[x] Has redux changes (please link)

PLT-6896 Fix request parameters for `/logs` by kaakaa · Pull Request #336 · mattermost/mattermost-redux

[x] Has UI changes
[x] Includes text changes and localization file (.../i18n/en.json) updates

When testing this change in local, plugins/PostMessageView test failed.
https://gist.github.com/kaakaa/2604e194f05d32a1d74fc6e9ec84e9fc#file-test-log-L6887
And I don't know why CI is success.
The failure in local testing is not problem?

@kaakaa can you try rebasing at some point - I expect that will fix your local test failure.

Thank you for reviewing @grundleborg.
Rebasing is done and test failures is fixed.

Hi @jasonblais, @jwilander.
Is there anything I can do for this PR?
With option1, I'll fix server side code to change the order of logs in a page.
With option2, I'll fix server side code to change the order of reading logs.
With option3, I'll fix few lines of webapp code.

@kaakaa Chatted with Joram. Option 3 sounds good, let's go with that

Thank you @jasonblais!
Implementing option3 is done.

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-0be77206495f66330.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1

Thanks @kaakaa! Although, I'm getting a blank screen on the test server with the attached JS error. I can't reproduce the error on master.
Maybe try rebasing your branch to see if that helps resolve it, since I doubt it's related to this PR.

Nevermind, this might be an issue on master. Investigating.

I encountered the same error about missing setSystemEmojis.
In my local, it was solved with cleaning environment by make clean, updating matter-redux, and re-running .
Since setSystemEmojis funcition was exists in ed1d042, I didn't commit the following changes.
yarn.lock
-  resolved "https://codeload.github.com/mattermost/mattermost-redux/tar.gz/ed1d042770aa5fe805949da62a508f607452a746"
+  resolved "https://codeload.github.com/mattermost/mattermost-redux/tar.gz/76b62e1683a307fe52c52ef3cdae06d021c28d79"

Should I commit this updating?

@kaakaa Yeah, let's try that. Might be related to it

@jasonblais Done 👍

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-07581a16e991b3dc3.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1

@jwilander Ready for review.
@kaakaa Huge thanks for this. Just needs to resolve merge conflicts for yarn.lock

Looks good to me, I'll merge right after we cut the release branch. Nice work on this @kaakaa !

@kaakaa can give you this a rebase? Then I'll merge it in

@jwilander rebasing is done!

Merged! Great work on this @kaakaa :)

Agree! Big thanks @kaakaa! :)


"connect" returns a value

This is not an issue, but a remark.
"connect" is a macro which returns a value, not different, I think, of the values returned by the miscellaneous connect functions of GTK.
But, for some reason, the compiler doesn’t check that the value returned by "connect" is assigned or discarded. This may, in some cases, cause errors difficult to find (and bugs if, unfortunately, the types match which is unlikely as "connect" returns as "culong").
Here is an example:
proc p(): SomeType =
  result = someValue
  # Body of the procedure.
  # ...
  # ...
  # The following line is the last line of the function, so the value returned by "connect"
  # is used as return value for the procedure.
  connect(someObject, "signal", someHandler)

Not sure we can do something to avoid this and it is not a frequent case, but I encountered it.

That is an interesting remark indeed.
The Nim connect macro/template is marked with the {.discardable.} pragma by intend so that the user is not forced to write "discard connect(...)". Same as in C code. connect() can return an clong ID, which may be needed in rare cases, but generally is not needed. (The chess board uses that ID to block the handler when computer is thinking for example). But that issue would still exist without the discardable pragma when connect is the last statement in an proc that returns an culong. Well, maybe the compiler would warn for that case. So this is more an issue for the Nim bug tracker or the Nim forum.

I have tried to provide a test case, but it seems to work fine with V 0.17.2

import macros

macro mymacro(i: int): untyped =

  let x = """
proc rint(i: int): int {.discardable.} =
  7
rint(64)
"""
  result = parseStmt(x)

template myTemp(i: int): untyped =
  mymacro(i)

proc rint2(i: int): int =
  result = 13
  myTemp(0)

echo rint2(0)

Output is 13 as expected. So there must be something very special to my connect() macro?

Can not reproduce with latest Nim v 0.17.2
This code always outputs 13 as expected for me.
# nim c button.nim
import gintro/[gtk, glib, gobject, gio]

proc buttonClicked (button: Button) =
  button.label = utf8Strreverse(button.label, -1)

proc p1(b: Button): culong =
  result = 13
  #b.connect("clicked",  buttonClicked)
  connect(b, "clicked",  buttonClicked)

proc activate (app: Application) =
  let window = newApplicationWindow(app)
  window.title = "GNOME Button"
  window.defaultSize = (250, 50)
  let button = newButton("Click Me")
  window.add(button)
  #button.connect("clicked",  buttonClicked)
  echo p1(button)
  window.showAll

proc main =
  let app = newApplication("org.gtk.example")
  connect(app, "activate", activate)
  discard app.run

main()

Closing as I can not reproduce the issue with latest Nim v0.17.2.
Please reopen if you can give an example with issue for latest Nim compiler.


Fix changing the compression codec

See test_recompression() in asdf/tests/test_compression.py
So we should distinguish the input compression type from the output one. Otherwise, loading a file with zlib-ed arrays and saving it with bzip2-ed arrays fails, as the compression is overridden.
This involves careful observation of every Block.compression usage and deciding if it should be "input" or "output". Besides, i removed the ambiguous property is_compressed. So yeah, this breaks the public API.
Another notice is that opening a zlib file and saving it with write_to without specifying all_array_compression changed it's behavior. Before: file is saved with the same output compression codec as the input one. After: file is saved without any compression.

Coverage decreased (-0.06%) to 93.979% when pulling aedeb55ebfa22d12a4d81d0debed3caddb5a8d8d on vmarkovtsev:fix-recompression into 89643aa528ae92c75c110169e65877f88662153c on spacetelescope:master.

Can you please try to add and/or modify a test so that test coverage does not decrease with this merge?

Coverage decreased (-0.2%) to 93.914% when pulling 8223fb299ce51d6abeceeb590da717e5e7b81887 on vmarkovtsev:fix-recompression into ab99df124c987c852635a25238a516c1dc14f011 on spacetelescope:master.

Coverage increased (+0.01%) to 94.173% when pulling ab0fe6e0ef18a4a278851bbb4252582e5f195133 on vmarkovtsev:fix-recompression into ab99df124c987c852635a25238a516c1dc14f011 on spacetelescope:master.

Coverage increased (+0.1%) to 94.267% when pulling f70a10bc4d0e85acfcc86943a27522d5ef280f98 on vmarkovtsev:fix-recompression into ab99df124c987c852635a25238a516c1dc14f011 on spacetelescope:master.

@drdavella Done. Had to add some extra tests to make the coverage happy.
I added "input" compression type, which is the default. It means, the same compression for output as in input. Thus None all_array_compression forces the decompressed output.


Change default classifier empty.

Motivation:
Change default classifier empty.  Then users do not need specify classifier to import dependency.

Pull Request Test Coverage Report for Build 349

1 of 8 (12.5%)  changed or added relevant lines in 3 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+7.1%) to 44.91%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sofa-ark-parent/core/common/src/main/java/com/alipay/sofa/ark/common/util/EnvironmentUtils.java
0
1
0.0%

sofa-ark-parent/support/ark-plugin-maven-plugin/src/main/java/com/alipay/sofa/ark/plugin/mojo/ArkPluginMojo.java
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

sofa-ark-parent/support/ark-support-starter/src/main/java/com/alipay/sofa/ark/support/runner/JUnitExecutionListener.java
2
100.0%

Totals

Change from base Build 329:
7.1%

Covered Lines:
1103

Relevant Lines:
2456

💛  - Coveralls


Document minimum platform support for ses-shim

Would be nice to have a clearly documented minimum platform support versions (may differ from dev tools / build system requirements)
I currently use node v10
node v12 is the current latest LTS
https://github.com/nodejs/Release#release-schedule
one potential issue to wider platform support is get-anonymous-intrinsics use of
new RegExp()[SymbolMatchAll]();

https://github.com/Agoric/SES-shim/blob/7e69a42086154d91fe0d58406cd4fa83c437e971/packages/ses/src/get-anonymous-intrinsics.js#L35
here is my quick-and-dirty monkey patch to re-add node v10 support (also used babel' plugin to convert import -> commonjs)
https://github.com/kumavis/cytoplasm/commit/d928d43f66d21c4847ac30028114c5193658ab05

Hi @kumavis do you need support earlier than v12?
If we agree on v12, what is needed? Could you extract just the changes needed for that into a SES-shim PR? Thanks.

I don't think support for earlier than v12 is really reasonable due to the release schedule
so nothing is needed, except documenting and testing the supported versions

Tracking on #22. Thank you.


cli: move client-only code out of the server flow

earlier, some of the client exclusive code was being run on zot server
instance too. This PR moves that code out of the way

lgtm thanks


Looking to understand missing data

Hoping I can have someone validate my thinking on this one or give me pointers on where to look.
I'm looking at Moloch DAOs on xDai and Polygon and seeing some with no members.moloch.title. Here is an example query for a particular wallet:
{
  members(where: {memberAddress: "0xb53b0255895c4f9e3a185e484e5b674bccfbc076"}) {
    moloch { title }
    molochAddress
  }
}

Running that query on the xDai subgraph page, these results come up (snipped for size, 12 of 14 results are title: null):
{
  "data": {
    "members": [
      {
        "moloch": {
          "title": null
        },
        "molochAddress": "0x0079255ac5861e109fe80cd105fdc5de819897fa"
      },
      {
        "moloch": {
          "title": "HausDAO"
        },
        "molochAddress": "0x283bdc900b6ec9397abb721c5bbff5ace46e0f50"
      }
    ]
  }
}

On the xDai blockexplorer:

null title
HausDao

Both seem to be Moloch based, which is good in that the subgraph is pulling the correct data based on my understanding.
My understanding
I have read through the Moloch.sol contract and the HausDao Summoner. I'm no Solidity expert but it seems that the only way to have a DAO title (with Moloch) is via the HausDao registerDao function which writes an event. With that understanding, would it be safe to assume that the DAOs in the results are Moloch based but were not registered? They may have been summoned or deployed by hand but probably not registered. Otherwise there would be a bug.
Is my understanding correct? Is there a way I could verify or help to verify?
Related issue: https://github.com/MetaFam/TheGame/issues/549

Good catch!
Dao titles are no longer saved in the subgraph. That was an old feature of moloch V1 and early V2 daos - added through registering in the factory. New factories for the latets v2 daos do not have that title field so we don't map anything to title.
DAO titles can now be changed by daos - and it's stored in dao metadata outside the subgraph.
I think the fix here is actually to remove the title field completely to avoid confusion.


Could not get unknown property 'mergeResourcesProvider' when running on Android

Steps to reproduce:
I got this error while running on Android: yarn android (react-native run-android).

Where:
Script '/Users/theoyoga/Documents/react-native-elements-storybook/node_modules/react-native/react.gradle' line: 95
What went wrong:
A problem occurred configuring project ':app'.
> Could not get unknown property 'mergeResourcesProvider' for object of type com.android.build.gradle.internal.api.ApplicationVariantImpl.

Temporary solution:
I found the issue and a quick workaround from this link: facebook/react-native/24022#474717616. I solved it by upgrading gradle version and SDK versions, etc in android/build.gradle.
@haruelrovix  I also submitted a PR related to this error. Please take a look.
Thanks.

@haruelrovix Hmm never mind, it turned out I got stuck with the old version of master. I don't know why I still had the old version that was 6 months old :(
Closing this issue.

@theodorusyoga 😄
When working on forked repo, make sure to pull the upstream first.
git pull upstream master

An example:
➜  react-native-elements git:(test/increase-coverage-for-socialicon) git remote -v
origin	https://github.com/haruelrovix/react-native-elements.git (fetch)
origin	https://github.com/haruelrovix/react-native-elements.git (push)
upstream	https://github.com/react-native-elements/react-native-elements (fetch)
upstream	https://github.com/react-native-elements/react-native-elements (push)

If you still want to contribute to this repo, please check this issue: https://github.com/react-native-elements/react-native-elements-storybook/issues/17.

@haruelrovix sure 😄 realized I forgot to add upstream on my forked repo. Will look into that issue


Unexpected Results with Concurrent RedisPipeline and RedisConnection Usage

When using a RedisPipeline created from a RedisConnection, both have access to the same Channel instance.
The former will write then flush in two separate steps, while the latter writeAndFlush in a single step.
When using both concurrently - say on separate threads - or even if you for some reason need to break up the pipeline with other non-pipeline calls, the results are unexpected as the connection's command is written in between the pipeline's.
Failing Unit Test
    func test_concurrentChannelUsage() throws {
        let pipeline = connection.makePipeline()
            .enqueue { $0.get("key") }
            .enqueue { $0.increment("key") }
            .enqueue { $0.increment("key", by: 30) }

        _ = try connection.increment("key").wait()

        pipeline.enqueue { $0.decrement("key") }

        let results = try pipeline.execute().wait()

        XCTAssertEqual(results.count, 4)
        XCTAssertTrue(results[0].isNull)
        XCTAssertEqual(results[1].int, 1)
        XCTAssertEqual(results[2].int, 31)
        XCTAssertEqual(results[3].int, 30)
    }

As part of PR #36 RedisPipeline has been removed from NIORedis so this bug no longer is valid.


CB-365 avoid too long resource name in ImageCatalogTest

Didn't work

Didn't work

I got the original issue back


Non-idempotency in braced literals

Braced literal expressions involving function calls duplicate them; this is problematic in the case that those functions do IO or observe or mutate state, as seen below:
https://civet.dev/playground/?code=aSAuPSAwCmYgOj0gKCkgPT4gaSsrCnQgOj0gWyJhIiwgImIiXQpjb25zb2xlLmxvZyB7dFtmKCldfQ%3D%3D

A similar problem occurs for multi-assignment syntax: https://civet.dev/playground/?code=KGNvdW50KGtleSkgPz0gMCkrKwooY291bnQoa2V5KSA%2FPSAwKSArPSAxCisrY291bnQgKj0gMg%3D%3D

Indeed, this is a "need refs" issue, currently tracked here: https://github.com/DanielXMoore/Civet/issues/50#issuecomment-1402394139
The next step here is #406, which we discussed recently on #compiler in Discord. Once that's complete, these should be easy to fix.


⚠ ClamAV has degraded performance

In 2ad94d3, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 245 ms

Resolved: ClamAV performance has improved in 97a68e9 after 14 minutes.


Q: Are rx/tx characteristics guarantied to be provided for nRF8001-board?

Hi, I've read https://learn.adafruit.com/getting-started-with-the-nrf8001-bluefruit-le-breakout/adding-app-support
UART Service UUID: 6E400001-B5A3-F393-E0A9-E50E24DCCA9E
TX Characteristic UUID: 6E400002-B5A3-F393-E0A9-E50E24DCCA9E
RX Characteristic UUID: 6E400003-B5A3-F393-E0A9-E50E24DCCA9E

Does it mean you've created custom BLE service with custom characteristics and the firmware is uploaded to your nRF8001 boards that are in stock (https://www.adafruit.com/products/1697)?
Is it nRF8001-specific service (and developed not by Adafruti) or it's created and uploaded by Adafruit?
I'm going to purchase that board and use for my iOS app.
One more question (not sure it's for your but for hardware engineer likely but probably you can know the answer):
I can't see Rx/Tx pin at https://www.adafruit.com/images/970x728/1697-04.jpg, so how can i wire it to my Arduino serial Rx/Tx pins?

Can you please post this up in the Adafruit support forums since that's the right place for this? The support staff generally don't monitor the Github issues or receive the email alerts here.
But yes those are custom services that are defined by Nordic (i.e. not officially adopted services/chars accepted by the Bluetooth SIG), and are optional and are generated with the Software the accompanies the nRF8001 on Arduino (etc.).  These services don't exist in the 8001 itself ... no services do ... you always have to put your service definitions onto the 8001 via config information in SW at startup.

Thanks for the reply, sorry about posting in wrong place.


slideTo bug with loop: true

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 4.3.3

Platform/Target and Browser Versions: Windows 10, Chrome

Live Link or JSFiddle/Codepen or website with isssue:

What you did
I am initializing Swiper with loop: true option. I then want to use API method slideTo to go to certain slides.
Expected Behavior
Whenever I call slideTo(n), I expect the slider to show me n-th slide, regardless of underlying implementation. Good example of this is bxSlider - it also uses cloned nodes for infinite loop functionality, but bxSlider handles API transitions to n-th slide properly.
Actual Behavior
When I initialize Swiper with loop: true, the slideTo(0) method shows last slide instead of first. slideTo(1) shows first slide instead of second etc. I believe this error happens because when loop: true option is set, the slider creates clones of first and last slides, and the slideTo method doesn't take that into account.

P. S. I have just checked and this bug hasn't been fixed in latest release version 4.3.3, I will edit issue description (I have discovered this bug on version 4.2.2)

There is .slideToLoop method that does exactly what you need

Thank you for your answer. This is of course still one of the dumbest things that I have encountered in my life. Why stop at 2 methods instead of 1? The possibilities here are truly endless:

slideToHorizontal / slideToVertical
slideToFixedHeight / slideToAutoHeight
slideToFade / slideToCube / slideToFlip
etc.


More docs updates & twine check workflow

update the sphinx theme, some typo fixes

added a new workflow that adds builds wheels and runs twine check.

and added a check-manifest


Question: How do I remove the default rounded corner of Buttons and TextBoxes?

After upgrading to the latest WinUI library, the Buttons and TextBoxes have rounded corners. But I want the previous sharp corners.
Manually setting CornerRadius to 0 fro all the controls is a pain. Is there a ThemeResource which I can override to set the default corner radius to 0 for Buttons and TextBoxes?

@MuziburRahman There is a global control corner radius theme resource which you can use: ControlCornerRadius
Example:
<Page.Resources>
     <CornerRadius x:Key="ControlCornerRadius">0</CornerRadius>
</Page.Resources>

This sets the corner radius for every control on that page to 0 (excluding overlay controls such as Popup and Flyout). If you only want to change the control corner radius for select controls, I'm afraid you currently have to use styles to achieve this.
Note that if you also want to restore the previous border thickness, you have per-control resources to achieve this:
<Page.Resources>
     <Thickness x:Key="TextControlBorderThemeThickness">0</Thickness>
</Page.Resources>

Additional resources to check out:

Preview API documentation about Corner Radius
Fluent XAML Theme Editor to easily visualize border thickness/control corner radius changes and code generation to include in your app

@Felix-Dev thanks a lot. :D

As @Felix-Dev mentioned there is an open issue regarding the setting of this value in application resources. However maybe this issue was partially resolved, since I was able to remove the corner radius by adding the following
<CornerRadius x:Key="ControlCornerRadius">0,0,0,0</CornerRadius>
to the application resources.


🐛 [Bug] Cannot convert any detection model from torchvision nor detectron

Bug Description
Hi,
I have hard times to export retinanet or mask-rcnn. I successfully traced models both from torchvision and detectron,
but it fails to compile. Since there are others who has problems with torchvision models, I would be happy if you
could resolve this problem once and for all. The only model for object detection which works is SSD from nvidia torch hub, but
supporting standart torchvision models would be a huge win.
To Reproduce
Steps to reproduce the behavior:
Note that traced_model works well, no problem here
import torch
import torchvision
import torch_tensorrt

model = torchvision.models.detection.retinanet_resnet50_fpn(num_classes=1, pretrained=False)
model = model.eval().to("cuda")

traced_model = torch.jit.trace(model, torch.randn((1, 3, 640, 640)).to("cuda"), strict=False)
torch.jit.save(traced_model, "resnet_50_traced.jit.pt")

trt_model_fp32 = torch_tensorrt.compile(traced_model, **{
    "inputs": [torch.ones((1, 3, 640, 640), dtype=torch.float32).cuda()],
    "enabled_precisions": {torch.float32}, # Run with FP32
    "workspace_size": 1 << 22
})

Error:
""" Only tensors, lists, tuples of tensors, or dictionary of tensors can be output from traced functions """
Expected behavior
I expect it to work, like magic :) !
Environment
Standart docker build from your dockerfile, base pytorch image is 21.10
Additional context
Do you think you could provide us with the tutorial how to deploy at least one detection model from torchvision or detectron? end to end? Thank you!

This seems like an error coming from torch.jit.trace, can you try using torch.jit.script

If I try this code for scripting:
import torch
import torchvision
import torch_tensorrt

model = torchvision.models.detection.retinanet_resnet50_fpn(num_classes=1, pretrained=False)
model = model.eval().to("cuda")

scripted_model = torch.jit.script(model, torch.randn((1, 3, 640, 640)).cuda())
torch.jit.save(scripted_model, "resnet_50_scripted.jit.pt")

out = scripted_model([torch.randn((3, 640, 640), dtype=torch.float32).cuda()])
print(out)

trt_model_fp32 = torch_tensorrt.compile(scripted_model, **{
    "inputs": [torch.ones((3, 640, 640), dtype=torch.float32)],
    "enabled_precisions": {torch.float32}, # Run with FP32
    "workspace_size": 1 << 22
})

I am getting an error:
"""
temporary: the only valid use of a module is looking up an attribute but found  = prim::SetAttr[name="_has_warned"](%self, %self.backbone.fpn.extra_blocks.use_P5)
"""
Scripted model itself looks to work well, print(out) gets me
"""
({}, [{'boxes': tensor([], device='cuda:0', size=(0, 4), grad_fn=), 'scores': tensor([], device='cuda:0', grad_fn=), 'labels': tensor([], device='cuda:0', dtype=torch.int64)}])
"""
Do you think your team could look into the support/tutorial for torchvision or detectron models? I am literally trying to compile retinanet and maskrcnn for over a year now. This would help a lot of people I think

Yes, we are working on an end to end demo for detection models

I look forward to. thanks a lot!

Hi @narendasan , just asking... do you have any timetable for '''  end to end demo for detection models ''' ? is it possible that you can get into that this year?

There isnt a hard timeline but we are actively working on it

@narendasan any news on this?

We are in the process of supporting Detectron2 OD networks. We will updated when there is a concrete timeline.

Same problem, I have try FasterRCNN in torchvision.models.detection
install torch-tensorrt by pip3 install torch-tensorrt -f https://github.com/NVIDIA/Torch-TensorRT/releases
if use torch.jit.trace, when compile,   """ Only tensors, lists, tuples of tensors, or dictionary of tensors can be output from traced functions """
if use torch.jit.script, when compile,  """
temporary: the only valid use of a module is looking up an attribute but found = prim::SetAttr[name="****"]
"""

This issue has not seen activity for 90 days, Remove stale label or comment or this will be closed in 10 days

Any news?

@joshuafc  We had our focus on transformers and dynamic shape models in the recent times. We couldn't get to finish an example based on detectron
@bowang007  Can you share the status of retinanet or mask-rcnn if you have worked on either of those ?

This issue has not seen activity for 90 days, Remove stale label or comment or this will be closed in 10 days

Any news?

So we are all playing with LLMs and ChatGPT now?

There are technical limitations upstream (torchscript, torchdynamo) that we are working with PyTorch on being resolved so nothing about detectron support has changed that much but it is still on our radar.


chore(06-solomachine, 07-tendermint): make client state methods moved to lcm private

Description
surfaced during alpha audit of v9.
closes: #XXXX

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against the correct branch (see CONTRIBUTING.md).
[ ] Linked to GitHub issue with discussion and accepted design, OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/).
[ ] Added relevant godoc comments.
[ ] Provide a conventional commit message to follow the repository standards.
[ ] Include a descriptive changelog entry when appropriate. This may be left to the discretion of the PR reviewers. (e.g. chores should be omitted from changelog)
[ ] Re-reviewed Files changed in the GitHub PR explorer.
[ ] Review SonarCloud Report in the comment section below once CI passes.

I guess we could add the api removals in the migration docs


Releases 23 apr 2024

Fix alignment issue in toastr
Fix css prop overriding in toastr

@akzhysparrow  the toastr dismissal behaviour, is that solved ?

@akzhysparrow the toastr dismissal behaviour, is that solved ?

Not yet @justin-ss, this one is for UI issues, will check the other one


Usability with git rebase, amend, etc

I have been using atom as my default editor when using git. One productivity issue I have is with the --wait command where you have to close the entire atom instance for waiting to be done. I know this sounds normal but when rebasing the atom instance needs to opened and closed many times and the load times on windows leave some time for thumb twiddling. My window load time is around 3-4 seconds.
It would be hot if atom could trigger the --wait completion when the tab is closed. I believe this is the behavior in sublime text.
My cursory search for this topic didn't show any results.

#461?


README: Add badges for GitHub Actions and secp256k1-jdk-developers Matrix room

Approved, though I'm a bit uneasy with the thought that we're perhaps leaking access log data to shields.io.

I agree. There are at least a few possible alternatives:

Make an image and commit to the repo.
Make an SVG image and commit to repo (might allow easier changes/updates to image text)
Use a text-only link


Solving command not found: compdef

antidote requires compinit for completions to work.
Add the following to the top of your ~/.zshrc if it's missing:
autoload -Uz compinit && compinit

Is this something you're suggesting be added to the antidote docs?

@mattmc3 I think so! I wasn't too sure where to add it exactly so I eventually just created an issue so that it's indexed by search engines.
Also it would be great if antidote load could warn if autoload -Uz compinit && compinit wasn't run.

While that feature would be easy to implement, and not a bad idea overall, the more I think on it the more I'm convinced antidote needs to stay in its lane and stick to providing great plugin-management features and not extras that people then need to override if they want to go a different way. I know other projects like Znap take a different approach and have prompt features, etc.
My goal with antidote is to let people compose their ideal Zsh experience through plugins, so my recommendation for completions would be to use things like zephyr or prezto.
However, I did update the docs to show how to add completions. Hope that will suffice: https://getantidote.github.io/completions


Implement support for deferred functions (resolves #987)

resolves #987
related to #922
Jobs can now register deferred functions -- functions that are run when the
job finishes, when the job crashes and is restarted or upon batch system
shutdown.
Every job now tests the state of other jobs on the node and can handle
cleanup of other jobs that have failed.
Added 4 tests for the deferred functions (it works, it works on restart,
jobs can handle other dead jobs, worker cleanup handles dead jobs)

@hannes-ucsc this is ready to look at. The tests are running because one test failed and I've implemented a fix for it that is orthogonal to the deferred function logic.

@hannes-ucsc this is ready! I'll implement all your comments (including the one about the docstring) in one go.

@hannes-ucsc i implemented the suggested changes

Jenkins, test this please.

@hannes-ucsc this is ready!

IT failure:
<Response><Errors><Error>InsufficientInstanceCapacity<Message>We currently do not have sufficient m3.large capacity in the Availability Zone you requested (us-west-2a). Our system will be working on provisioning additional capacity. You can currently get m3.large capacity by not specifying an Availability Zone in your request or choosing us-west-2c, us-west-2b.</Message></Error></Errors><RequestID>b56b1347-2ea9-4307-a99a-86d99f3574c8</RequestID></Response>
If this happens more often (I've seen it once before) we should think about a solution.


eslint-plugin: Require eslint ^8.40.0 as peerDependency

Commit 6da1cb1bfb3e787122536ff14fd09194c8b55cbe started to use context.sourceCode property, which is introduced in ESlint v8.40.0.^eslint-8.40.0
This patch updates the peerDependency for eslint to require that version or later. The change will help users using @definitelytyped/eslint-plugin outside of DefinitelyTyped properly manage their dependencies.

@microsoft-github-policy-service agree

I've failed to commit pnpm-lock.yaml and revised patch.

I'm sorry to be late to reply, and thank you for merging this patch.
My story is that I used to use dtslint to lint and type-check the type definitions for a Node.js library I'm maintaining. dtslint was a perfect fit for my need because it could type-check type definitions against multiple versions of TypeScript, and had a good ESlint config, although I knew it was created and maintained as an internal tool for DefinitelyTyped project.
As a result of the recent restructuring in DT repository, dtslint finally stopped working on external projects. Hence I decided to migrate to another test runner (which might not exist yet) and direct use of ESlint. I found, however, that some of the ESlint rules implemented in this repository (e.g. no-any-union, no-const-enum, redundant-undefined) were still useful for TS type definitions outside DT.
As I understand the packages in this repository are for internal use of the DT project and guarantee no stable versioning/interface, it should be most desirable for me to fork this library and stop depending on DT-tools. But, tracking changes in the TypeScript language features and ESlint internals is equally burdensome for me as tracking DT-tools (or asking to fix DT-tools like this one 🙇). Another option may be to propose typescript-eslint project include these rules, but I guess the rules reside here because they are preferences of the DT project and too opinionated to be there (actually, I don't know how they decide to accept a new rule in their default ruleset).
I also think the ESLint config from DT/eslint-plugin can be valuable for library authors maintaining their type definitions in their repository because their users and contributors are likely familiar with the DT coding convention and it can provide a consistent experience.

There's definitely still a wider question of if we expect people to be able to run this repo's code outside of DT, though I will note that no-const-enum could be replaced with no-restricted-syntax. no-any-union and redundant-undefined are certainly ones that would be useful in ts-eslint, IMO.

Ah, ts-eslint already has https://typescript-eslint.io/rules/no-redundant-type-constituents/ which covers no-any-union though does so with type info (and checks for more badness). I'm going to check to see if we could just use this.

@jakebailey Thank you for your advice on ESlint rules.


.app bundles in archives mounted with mountarchive do not work

.app bundles in archives mounted with mountarchive do not work yet, the user has to copy out the application first.
We should make it work.
Example: https://github.com/helloSystem/Utilities/archive/master.zip
Why?
Execute permission is there, application can be executed from the terminal.
-r-xr-xr-x  0 root  wheel    12K 16 Feb. 18:49 /var/run/user/1001/_home_user_Desktop_Utilities-master.zip/Utilities-master/3D Printing/FreeCAD.app/FreeCAD

How do you use mountarchive?

When it works, then just download the zip above, double-click it, and be able to execute the applications therein. If that doesn't work yet, then we have a bug ;-)

I think it mounts it, but doesn't open the mounted folder.

More importantly (but not for this issue), how do you associate a file with a .app bundle?

My Fonts.tar.gz works fine, I think it's something about that particular zip file, because running unzip from the command line didn't actually extract anything.

I double-clicked the zip archive in Filer and it said 'The path is not valid,' so it's either something about .zip files in general or that particular one.

archivemount -f -o readonly,nonempty,allow_root Utilities-master(1).zip /var/run/user/1001/Utilities-master(1).zip
2
launch Filer /var/run/user/1001/Utilities-master(1).zip
# Descending into "/Applications/Office"
# Descending into "/Applications/Utilities"
# Descending into "/Applications/Video"
# Descending into "/Applications/3D Printing"
# Descending into "/Applications/Developer Preview"
# Descending into "/Applications/Graphics"
# Descending into "/Applications/Preferences"
# Descending into "/Applications/Developer"
# Descending into "/Applications/Audio"
# Descending into "/Applications/Autostart"
################### Checking "/System/Filer.AppDir/AppRun"
# Found "/System/Filer.AppDir/AppRun"
Took 8 milliseconds to find candidates via the filesystem
Candidates: (QFileInfo(/System/Filer.AppDir/AppRun))
Selected candidate: "/System/Filer.AppDir/AppRun"
QProcess: Destroyed while process ("archivemount") is still running.

The user with the ID 1001 should have write access to /var/run/user/1001/; are you saying this is not the case @linuxkettle?

No, my UID is 1000, so no 1000 directory was created.
Also, I can't mount any ZIP archives even after fixing that problem. Tarballs work fine, but when I run mountarchive on a ZIP folder, it opens Filer but doesn't mount the archive, so I'm left with a Filer window showing a folder in it, but seconds afterwards it says that the folder doesn't exist.
Does this mean the archive is getting unmounted?

No, my UID is 1000, so no 1000 directory was created.

Interesting!
On which system (exact version) does this happen?

It's because I created a new user, the live one was 1001.
I installed it differently because I needed to keep some of the files on my HDD, and the installer stalls when I try to install to a USB (because of issue #200). Although the Users app should create a runtime directory for the new user.

Good point. What piece of software is normally resposible for creating those directories?

I don't know, in Linux systemd does it, but on FreeBSD it seems to be a do-it-yourself-or-else thing. The odd thing is, tar archives sometimes work, and every archive format except zip mounts on /var/run/user/1001 anyway.
Usually though, tar archives just say "Device not configured." and any attempt to do anything in them results in that error message being shown again.

in Linux systemd does it

Interesting. So is this whole /var/run/user/$UID thing a Fedora-ish thing rather than a Unix-ish thing? Then we may want to use something else if there is a more Unix-native way to have a per-user temporary location.


align.doctest errors

The changes to nltk.align have resulted in some doctest errors, e.g.:
% cd nltk/nltk/test
% make align.errs
File "/Users/sb/git/nltk/nltk/test/align.doctest", line 86, in align.doctest
Failed example:
    print(round(em_ibm1.probabilities['the']['das'], 1))
Exception raised:
    Traceback (most recent call last):
      File "/opt/local/Library/Frameworks/Python.framework/Versions/3.4/lib/python3.4/doctest.py", line 1318, in __run
        compileflags, 1), test.globs)
      File "<doctest align.doctest[15]>", line 1, in <module>
        print(round(em_ibm1.probabilities['the']['das'], 1))
    AttributeError: 'IBMModel1' object has no attribute 'probabilities'

@hoontw, would you please take a look at this?

Sure. Sorry for the mistake. I didn't realize that there were other doctests hiding in the /test folder.

Thanks


Rally

https://forum.rally.io/t/guide-how-to-submit-a-funding-proposal/1407
Primary DAO Activities: Funding projects, companies or individuals via grants to build on top of the $RLY Ecosystem, which includes:
Specific Projects: Build on $RLY, support existing projects for specific purposes, etc.
Community (Education): Get grants to build elements of the $RLY ecosystem that will help current and future members.
Product Launches: Applications of each launch must have prototyped a feature-full product. We would provide a ~$50K equity-free contribution based on specific milestones for your application.
Events and Hackathons: Sponsoring events and hackathons for people to learn about $RLY and build in the $RLY ecosystem.
Bounties: Funding $RLY ecosystem specific bounties.

Starting to catch some chatter that Rally is building a web3 "spaces" so could be something interesting there.


make the ui like wallaby.js

it would be cool if the // error display is remove,         because it alters the code. better to make the UI like Wallaby.js which shows passed and failed test on the number pane. and does not show errors written to file in comments.

Wallaby does the same thing as this, using the text annotation API, I just added "//" before it.


Render b with escaped format

It works, but

it uses string concat on format
the rebuild of format is ugly

Current coverage is 96.79% (diff: 94.44%)

Merging #25 into render will decrease coverage by 3.20%

@@           render        #25   diff @@
========================================
  Files           3          3
  Lines         276        312    +36
  Methods        35         36     +1
  Messages        0          0
  Branches       40         44     +4
========================================
+ Hits          276        302    +26
- Misses          0          8     +8
- Partials        0          2     +2

Powered by Codecov. Last update 3c070c6...ed80ee8

The code could be a little simpler but I think the approach is good.
Escaped braces in formats are something I've never seen. So your code basically copies a single string and it's done. The main case is as efficient as before.
Escaped formats require concats and stuff but let's be honest: (a) this is very unusual; (b) given the length of a format string, an additional copy won't hurt much.

Regarding rebuild the code is simple and clear, so that's good enough for me. Rebuild is only test code anyway.
BTW: note that rebuild can't have 100% fidelity because spaces around alignments are lost in our internal representation of the template.

@jods4 I agree with your feedback. Too bad it's really busy at my side, at work and support at NLog.
Feel free to apply your changes to the branch

@304NotModified I'll try to pick this up somehow... sorry but you probably figured out that I'm quite busy at work at the moment as well.

That would be great!
FYI
I'm back from holiday at 10 Sept and I will first focus on NLog 4.3.8

working on this

started on changes after review, but not fully done

All done :)
@jods4 if you could quickly double check, that would be nice :)


flutter_windowmanager has been deprecated, need to update as per latest flutter version

Getting below error in console panel:
The plugins `flutter_windowmanager` use a deprecated version of the Android embedding.
To avoid unexpected runtime failures, or future build failures, try to see if these plugins support the Android V2 embedding. Otherwise, consider removing them since a future release of Flutter will remove these deprecated APIs.

Need to update flutter_windowmanager plugin as per latest flutter version. Thanks.

The embedding was updated in version 0.2.0 of the plugin. Can you update the dependency version and try again?

@pmundt confirmed, by updating to 0.2.0, the warning is gone.
Thanks @pmundt

This library records audio when playing video... tested on android 10 and 11.. I am using 0.2.0 version

@adispatil I''m not sure how that would happen given that the library contains no logic beyond toggling application flags, but if you've encountered something like this, please open a separate issue with more information and steps to reproduce and I'd be happy to have a look at it.

Thanks for the confirmation @afandiyusuf, I'm now closing this as resolved.


Expanded schemes and filetype registrations

Problem description
University of Richmond is a Linux/Python/Oracle shop, with MicroStrategy as the data warehouse. Our staff have written an ETL system entirely in Python 3, and that makes us an obvious consumer of smart_open. The ETL system executes a few more than 1000 integrations per day, and is considered Tier 1 business software. My impressions of the smart_open package are uniformly positive and I plan to incorporate it into our ETL software.
Additional schemes
We would like to take on the task of expanding the existing RFC 3986 schemes to encompass the things we are using now in other, less regular forms:

box://
sharefile://
azure://
https:// read and write

Additional filetypes
We also deal with a number of other file formats that smart_open could handle by expanding the register_compressor() hook. We are interested in tackling msgpackand pgp/gpg first.
Diagnosing files of unknown type
As well as having a few file extensions of our own, we frequently receive files from our vendors that are of unknown provenance. Files are sometimes bzipped instead of gzipped, and are sometimes encrypted without being named .pgp or .gpg. We would like to add inspection for the well known file signatures of various file types.
Versions
Linux-3.10.0-1062.4.1.el7.x86_64-x86_64-with-redhat-7.7-Maipo
Python 3.6.9 (default, Sep 11 2019, 16:40:19)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-16)]
smart_open 1.9
Question
Does the smart_open community think the above sounds like a "good idea," and something of wide interest, or should we fork the code and continue along our own vector? Given the work above and the existing commitments, we are not likely to have a deliverable before 30 June 2020.
George Flanagin
Computer Scientist
Puryear Hall, 118 UR Drive
Office G28
University of Richmond, VA 23173
+1.804.287.6392 (+1.80I.BURN.EXAms)

Checklist
Before you create the issue, please make sure you have:

[x] Described the problem clearly
[x] Provided the version numbers of the relevant software

Hi George,
Thank you for your interest in the smart_open library. The below is my personal opinion. @piskvorky , please let us know what you think about this, too.
I think adding the additional schemes would be a great idea. BTW, AFAIK https reading works now, but writing doesn't.
I'm not so sure about the other functionality: additional file types and file type diagnosis. I think this sort of stuff belongs on a level above what smart_open does. For example, look at our recommended way of dealing with gzip files. I think you could do the same thing with the proposed file types, couldn't you?
Similarly, there are already libraries out there for diagnosing file types. For example, see python-magic. Integrating it with smart_open is trivial:
from smart_open import open
import python-magic

with open(..., 'rb') as fin:
    # recommend using at least the first 2048 bytes, as less can produce incorrect identification
    detected_type = magic.from_buffer(open(fin.read(2048))

I should have made it clear by an italic or something that with https, I meant to have emphasis on the write.
What you said sounds fine to me. When I get back to my office on Monday, I will suggest that we proceed as you described. We use the magic package currently; I see that one of the goals may be to have as few entanglements with other packages as possible/practical. My vision is warped by building products/systems instead of packages/libraries. They are different skills.
In re: the zip files (or gpg files in our case), I see that you are referencing the same package independence. Our crew probably wants to be able to open('abc.gpg', 'rb'), but we will wrap our code externally, and figure out a way to pass in the list of recipients for open('abc.gpg', 'wb')
I have to think a bit more about the schemes. In our ETL software, we exploit the ~/.ssh/config file's ability to define aliases for the host names, and the connection methods. That's at the application level, though. The University's policies prohibit use of passwords for credentials, and as a consequence we tend to see everything through the lens of key-based authentication.

Yes, that's exactly right. The tradeoffs when building a product and a library are different. That said, we can probably find significant common ground. For example, I'm not opposed to using ~/.ssh/config for the ssh submodule. The existing library we use for SSH (paramiko) probably has support for interpreting that config file already.

paramiko definitely has that capability, and we use it now.


Crash when payment should happen

objectservice_1 | Caused by: java.lang.NullPointerException
objectservice_1 |       at edu.kit.ipd.crowdcontrol.objectservice.payment.PaymentCalculator.estimatePayment(PaymentCalculator.java:67)
objectservice_1 |       at edu.kit.ipd.crowdcontrol.objectservice.payment.PaymentDispatcher.dispatchPayment(PaymentDispatcher.java:64)
objectservice_1 |       at edu.kit.ipd.crowdcontrol.objectservice.payment.PaymentDispatcher.lambda$new$0(PaymentDispatcher.java:48)
objectservice_1 |       at rx.Observable$27.onNext(Observable.java:7928)
objectservice_1 |       at rx.observers.SafeSubscriber.onNext(SafeSubscriber.java:139)
objectservice_1 |       ... 47 more

Integer in line 67 is NULL.

NUUUUUULLLLLL

NUUUUUUUUUUULLLLL !!!
ноль !!!!


[Postgres-XL ] Unable to retrieve data from table

Using DBeaver trying to connect to Postgres-XL, able to view table columns details but unable to retrieve data. Error thrown is as shown

When click on the "Test connection" button, it is successful as shown

Please post error stacktrace (from Error Log view or from log file: https://github.com/serge-rider/dbeaver/wiki/Log-files).
Also you could try to switch to auto-commit mode (in main toolbar/menu)

I am suspecting this maybe the issue of Postgres-XL, but my colleague who
is using Valentina Studio 7 is able to access without issue
On Thu, Jul 13, 2017 at 1:34 PM, Serge Rider<EMAIL_ADDRESS>wrote:

Please post error stacktrace (from Error Log view or from log file:
https://github.com/serge-rider/dbeaver/wiki/Log-files).
Also you could try to switch to auto-commit mode (in main toolbar/menu)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/serge-rider/dbeaver/issues/1851#issuecomment-314975713,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGg70P8yMEp5WmT6928ID15edsw7Xcr6ks5sNayDgaJpZM4OOWT2
.

This is possible. DBeaver wasn't tested with Postgres-XL.
But if you will post error stacktrace then maybe it will be possible to fix the issue.

I have already attached the log file in previous email too. I will attach
them again here
On Thu, Jul 13, 2017 at 8:58 PM, Serge Rider<EMAIL_ADDRESS>wrote:

This is possible. DBeaver wasn't tested with Postgres-XL.
But if you will post error stacktrace then maybe it will be possible to
fix the issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/serge-rider/dbeaver/issues/1851#issuecomment-315069997,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGg70G2Tw02IOs9kdfAqTWkrmecMXkWxks5sNhRfgaJpZM4OOWT2
.

Please add logs in GitHub comment (by drag-n-drop) or send it to me on<EMAIL_ADDRESS>You can't attach files when you reply to GitHub by email.

Sorry, not notice about that.
Attached are the log files involved
log.txt
dbeaver-debug.txt

FYI: I work with a few Postgres-XL instances regularly. The workaround I use is to use the PostgreSQL (Generic) driver. You lose some functionality for doing things like database backups/restores, but I've been using it for 6+ months without other issues.

@kyleflan thanks a lot, it is able to work as you said, using the PostgreSQL (Generic) driver able to work properly. Thanks a lot.
But the description of PostgreSQL (Generic) driver of "PostgreSQL JDBC driver. Can be used for old PostgreSQL server versions (before 8.x)." maybe can change a bit, usable for Postgres-XL

According to log this error happens when DBeaver executes query like select * from table. I doubt that using generic driver will resolve this.
@csyoong28 did you try to execute SQL queries in SQL editor?
Also this may be related to Postgres XL config (https://stackoverflow.com/questions/38037716/issue-with-datanodes-on-postgres-xl-cluster).

As for now, using generic driver option do works well, able to execute SELECT statement in editor without problem.
I do go through the stackoverflow issue, but I am not sure if that is the solution since Valentina Studio is working correctly.

@serge-rider I think I've tracked down the cause for this and the reason for the generic driver working but not the latest driver. See issue #1896.

Great


Update from GCM to FCM

Since Twilio has updated their docs and android quickstart. Also, the current TwilioSDK version is at 2.0.0-beta4 which will be soon deprecated.
Following changes have been made :

Update from GCM to FCM
Removed GCMRegistrationService that extends IntentService because Android O doesn't allow IntentService anymore as per twilio android quickstart and android docs
Changed name from com.phonegap.plugins.twiliovoice to more conventional cordova naming cordova-plugin-twiliovoice
Fixed the version on plugin.xml to match package.json
Removed disconnect sound since wasn't used at all on android

Nice work! Hopefully this will get merged soon :)
..My OCD thank you for the naming change!

Wow, great work!

@jefflinwood, Thanks for merging this PR!
I installed the plugin locally to make sure it's working, however after a quick look at plugin.xml I noticed that plugin.xml L88 should also be removed since it can produce errors when cordova cli doesn't find that resource on sounds folder.
I can't explain why cordova didn't complain for this resource...

Strange! Ok, if you push a new commit, and make a new PR, I can merge it pretty quick.

@jefflinwood, done, look at #43


[Payment Request] Crypto

Please consider a method to donate to the project via Bitcoin etc
Thank you for your work.

I will look into a good way to add crypto as an option for donations. In the meantime, if you would like to donate via crypto, you can use any of the following addresses (others available if desired). Be sure to send only the listed asset type to each address!
Bitcoin (BTC): 3P92LR9wgy1r13WwxqhGJL7UUdtA1THxZk
Ethereum (ETH): 0x2CC76dDCC9fD7e89687740d6b0D7235A0f8eaA2E
Shiba Inu (SHIB): 0x6557c990e55d87C6b35093f7fd1167323930e66f
Dogecoin (DOGE): DGMxaFU7nv6hfQ4974Ep1eLXag5LKvmnFm
USD Coin (USDC): 0x0d4826D4D4F8d65F9A2e58E2CD32b70b8DAC59F5
Thank you for the support!

Added the addresses to a wiki page for now: https://github.com/toddrob99/searcharr/wiki/Donations-via-Cryptocurrency


Two Elevator Contacts being powered simultaneously

Describe the Bug
In an elevator built using the new Elevator Pulley and Elevator Contact blocks, the contact below the one the elevator is at is also activated. For example, if the elevator is at floor 7, the contact at floor 7 will produce a redstone signal as it should, but the contact at floor 6 will also produce a signal.
Reproduction Steps
Place an elevator pulley above a superglued elevator cab, powered with a creative motor. Elevator contacts are placed in a single point on the X/Z axis, 4 blocks apart on the Y axis. When the elevator is moved to a different floor, either by powering a contact or by using Contraption Controls, all elevator contacts stop producing a signal, but when the elevator stops, the contact at its floor and the contact at the floor below it will be producing power, and visibly illuminated.
Expected Result
Only the elevator contact at the floor the elevator is at should be powered, no other ones.
Screenshots and Videos

Crash Report or Log
No response
Operating System
Windows 10
Mod Version
0.5.1b
Minecraft Version
1.19.2
Other Mods
Accurate Block Placement, Ambient Sounds, Block Meter, Carpet Mod, Create, Create Track Map, Create Train Trapdoor Disabler, Create: Steam N Rails, Do A Barrel Roll, Extended Block Shapes, Immersive Aircraft, Indium, Jade, JEI, LambDynamicLights, LazyDFU, Light Emitting Diode, Lithium, Logicates, Mod Menu, Mouse Tweaks, Red Bits, Redstone Bits, Redstone Pen, Reese's Sodium Options, Replay Mod, Simple Chunkloader, Sodium, Sodium Extra, Sound Physics Remastered, Starlight, Supplementaries, Xaero's MiniMap/WorldMap, Zoomify, and all necessary libraries/API's
Additional Context
The issue persists after logging out, restarting PC, disassembling and reassembling the elevator, or breaking and replacing all elevator-related blocks. The issue occurs with different elevators in different worlds. All elevator contacts stop producing a signal while the elevator is in motion, but when the elevator stops at a floor, both the current floor's contact and the contact of the floor below start producing a signal.

I can confirm this issue exists on both 0.5.1b Patch 1 and Patch 2

It looks like it's an upstream issue

This has been fixed in newer versions


Minimize network transfer due to fonts

As a USER in a low resource setting, I want web pages to load quickly and use as little of my data package as possible.
As a USER I want fonts on pages to look good no matter what language is on the page.
I think requirements are

Even if I am reading an /en/ page, if there are Arabic characters I still want them to be readable and look good in context
If I never access a page with Arabic characters, I don't want to download them
I want to avoid downloading character sets multiple times, for example if the character in "T" is used in English and the same character is used in Turkish I don't want to download that "T" character twice

@ChrisMarsh82 @sheralim012 per our convo today


Add DatetimeAccessor for accessing datetime fields via .dt attribute

[x] Partially closes #358
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff
[x] whatsnew entry

This uses the register_dataarray_accessor to add attributes similar to those in pandastimeseries which let the users quickly access datetime fields from an underlying array of datetime-like values. The referenced issue (#358) also asks about adding similar accessors for.str`, but this is a more complex topic - I think a compelling use-case would help in figuring out what the critical functionality is
Virtual time fields
Presumably this could be used to augment Dataset._get_virtual_variable(). A season field would need to be added as a special field to the accessor.

@shoyer I corrected things based on your comments. The last commit is an attempt to refactor things to match the way that methods like rolling/groupby functions are injected into the class; this might be totally superfluous here, but I thought it was worth trying.

Updated with support for multi-dimensional time data stored as dask array.

Finished clean-up, added some documentation, etc. I mangled resolving a merge conflict with my update to whats-new.rst (5ae4e08) in terms of the commit text, but other than that I think we're getting closer to finishing this.
wrt to the virtual variables, I think some more thinking is necessary so we can come up with a plan of approach. Do we want to deprecate this feature entirely? Do we just want to wrap the datetime component virtual variables to the .dt accessor if they're datetime-like? We could very easily do the latter for 0.9.3, but maybe we should target a future major release to deprecate the virtual variables and instead encourage adding a few specialized (but commonly-used) accessors to xarray?

wrt to the virtual variables, I think some more thinking is necessary so we can come up with a plan of approach. Do we want to deprecate this feature entirely?

This is a good question, possibly worth raising in a separate issue for discussion. I think they are a convenient shortcut, but they shouldn't be the primary interface.

Do we just want to wrap the datetime component virtual variables to the .dt accessor if they're datetime-like?

Yes, certainly. We definitely don't want to have two separate implementations of datetime components in the code base.

There's a test-case relating to #367 (test_virtual_variable_same_name) which is causing me a bit of grief as I re-factor the virtual variable logic. Should we really be able to access variables like ds['time.time']? This seems to break the logic of what a virtual variable does, and was implemented to help out with time GroupBys and resampling (something I'll eventually get around to finishing up a refactor for - #1272).
Two options for fixing:

add a "time" field to DateTimeAccessor.
add some additional if-then-else logic to _get_virtual_variable.

add a "time" field to DateTimeAccessor.

This would be my preferred fix.
I agree that ds['time.time'] is a little confusing but think it's unambiguous: it says get the "time" component (an array with datetime.time elements) from the "time" variable (an array of dtype=datetime64), i.e., equivalent to df['time'].dt.time in pandas.

Turns out it was easy enough to add an accessor for ds['time.time']; that's already provided via pandas.

Alrighty, patched and ready for a final look-over! I appreciate the help and patience, @shoyer!

Thanks @darothen! This is really nice


[OFFICIAL] Generated Issue: ParserException

Please fill out the details below.
Summary:
Steps to reproduce:
Fetch https://api.jikan.moe/v4/anime/32268/relations
Additional Details
Jikan Parser Version: 3.0.0.x-dev@b619665b0dc73908b794d5c464055a4c786c7b34
PHP: 8.0.15
Redis: Connected
Exception: Jikan\Exception\ParserException
Code: 0
Message: Failed to parse 'https://myanimelist.net/anime/32268/'
Trace: /var/www/jikan-rest/vendor/jikan-me/jikan/src/Exception/ParserException.php on line 24
Request: GET /v4/anime/32268

Seems to be resolved. Feel free to re-open the issue if it occurs again.