Daq Viewer "initialize detector" crashes viewer / dashboard

Clicking on "initialize detector" of the 2d or 0d viewer crashes the viewer (and dashboard).
There is no error message in the log, but in the console is written something like "QThread: Destroyed while thread is still running".
Viewer are the mock viewers or an application of the generic camera viewer.
Pymodaq version is 4.0.8
Pyqtgraph is the most recent version
Python 3.11
Windows 10
How to reproduce:

Start the dashboard in cli
Create a preset with a mockdetector
Load preset
Click on initialize detector

In fact, the preset has already initialized the control modules (see the check box for that in the presets). It means that in fact you are uninitializing the modules by clicking on them. However this is true that there is an issue related to the thread when deinitializing.

if you know a proper way to stop a thread, i'd like to hear it !!
So far I do:
try:
    self._hardware_thread.quit()
    self._hardware_thread.wait()
    finished = self._hardware_thread.isFinished()
    if finished:
        pass
    else:
        print('Thread still running')
        self._hardware_thread.terminate()
        self.update_status('thread is locked?!', 'log')
except Exception as e:
    self.logger.exception(str(e))

while the thread is initialized as:
hardware = DAQ_Detector(self._title, self.settings, self.detector)
self._hardware_thread = QThread()
if config('viewer', 'viewer_in_thread'):
    hardware.moveToThread(self._hardware_thread)

self._hardware_thread.hardware = hardware
self._hardware_thread.start()

I narrowed it down:
For the Move (which is working), the sequence at preset loading is: "Move_ui.send_init" and then "Move.init_hardware".
For the Viewer (which is not working) the "Viewer_ui.send_init" is never called, just "Viewer.init_hardware".
If I load another preset (a mock mover and a mock camera) I reproduce above results. If I do not initialize the detectors at preset loading, I can connect/disconnect at will.
If I do not initialize the camera in the beam_steering_mock preset at loading time, the preset never loads. It hangs at the first move (the first piezo) after the camera.
Conclusion: Somewhere the preset loading of viewers has a bug regarding viewer initialization.

this should be solved by PR #199 into pymodaq_dev

Should have been closed by #174


Fixes grammar ERRs in Speak Up!

After some testing, I think Speak Up! was raising grammar errors because RimThreaded created ThreadStatic copies of GrammarResolver.rules. This PR fixes that with a global static GrammarResolver.rules.
This is probably not thread safe; I think we may need to synchronize calls to GrammarResolver.Resolve() so that GrammarResolver.AddRule()s are not interleaved with the rules.Clear() at the start of ResolveUnsafe.

Related to https://github.com/cseelhoff/RimThreaded/issues/546

You are right, the clear in the beginning must have tricked Hoff into thinking that this is one of those cases where Threadsafing is fine. Now the rulepool can remain threadstatic it is just an object pool it doesn't matter if it is syncronized or not. Infact the other solution would be to make a concurrent simple linear pool(that would use interlocked increments for its internal index) or use one of the millions cuncurrent object pools I made like the PulsePool. But again is useless just leave it threadstatic.
Now for the rules syncronization I don't think is necessary here. You should simply replace "rules.Clear();" in ResolveUnsafe, with "rules = new Dictionary<string, List>()>" or even "rules =  PulsePool<Dictionary<string, List>>.Pulse(rules);" that should provide enough protection from cuncurrency while still recycling the grammar rules.
So Again keep the rulesPool threadstatic and try swap that rules.clear() with "rules =  PulsePool<Dictionary<string, List>>.Pulse(rules);" and see if it enough protection from cuncurrency.

Thanks!
On Thu, Jan 13, 2022, 07:40 Caleb Seelhoff @.***> wrote:

fix for SpeakUp implemented. This helped find the root cause. However the
implementation was slightly different so this PR will not be merged in as
is.
—
Reply to this email directly, view it on GitHub
https://github.com/cseelhoff/RimThreaded/pull/703#issuecomment-1012253761,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHRW5NRUVWYVOYGTKRUPTDUV3W5NANCNFSM5HC7P7YA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Support InstantiationService in extension host

In the VS Code frontend the ExternalTerminalService is registered via registerSingleton here:
https://github.com/microsoft/vscode/blob/7c88e07cc69633729be10d25e8408707fce48bf4/src/vs/workbench/contrib/externalTerminal/node/externalTerminalService.ts#L389-L395
I'd like to use the same service in the backend (ext host) and I would like to share the approach how it is initialised (registered). Currently I'm doing this:
https://github.com/microsoft/vscode/blob/7c88e07cc69633729be10d25e8408707fce48bf4/src/vs/workbench/contrib/debug/node/terminals.ts#L17-L25
But I would rather like to use the InstantiationService and the registerSingleton calls for this.

@weinand there is now an instantiation service in the extension host and ExtHostDebug... is being created with that. That should give you access to those terminal services and/or the instantiation service itself.
https://github.com/microsoft/vscode/blob/3a47abace18bd464e12f30ebb731519384f51f77/src/vs/workbench/api/node/extHost.api.impl.ts#L126

@jrieken great, I'll give it a try!


Delete checkboxs

Hi, i'm trying to hide checkboxs from some nodes but I can't access to the 'i' element with class 'tree-checkbox' because it's generated by the component.
The only thing I'v got is to set a class inside 'tree-anchor' named 'hideCheckbox'
Is there anyone with the same problem?
Thanks!

yes... but no solution.. I'm interested about the css trick :)

Manu no success for me.. can you share your solution<EMAIL_ADDRESS>


Can't connect to bucket using fstab

this will connect to the bucket with no issues /usr/local/go/bin/goofys -o allow_other --uid=48 --gid=48 --dir-mode=0777 --file-mode=0777 bucket-name /var/www/html/site/upload_folder
but doing this in fstab fails with the errors below the instance has access to s3 using a role assigned to the ec2 instance
/usr/local/go/bin/goofys#bucket-name /var/www/html/site/upload_folder fuse _netdev,allow_other,--uid=48,--gid=48,--dir-mode=0777,--file-mode=0777 0 0
/usr/local/go/bin/goofys[1957]: main.ERROR Unable to access
'bucket-name': permission denied
Jul 28 21:16:02 stqa /usr/local/go/bin/goofys[1957]: main.FATAL Mounting file sy
stem: Mount: initialization failed
FYI this use to work and then stopped more recently

Root user had aws credentials set up which was cases the issue. /root/.aws/credentials.  Once I cleared those out the issue went away.


handler return value in example

Updating the hello example in readme to reflect latest slim-develop changes.
Route handlers must return a \Psr\Http\Message\ResponseInterface since they are part of the middleware stack.

I think this was already done in #1187?

@mnapoli
oh right, thanks. i totally missed that. I pulled the latest dev about 30-40 minutes ago..


Update AbstractDAO to use JPA CriteriaQuery API

Hibernate's Criteria API (org.hibernate.Criteria) was deprecated and the dropwizard's AbstractDAO class was using that API. It also causes logs to be flooded with this warning.
WARN org.hibernate.orm.deprecation: HHH90000022: Hibernate's legacy org.hibernate.Criteria API is deprecated; use the JPA javax.persistence.criteria.CriteriaQuery instead
Is there any plans to refactor AbstractDAO to use JPA Criteria API?

You can use the criteriaQuery method on AbstractDao to access the JPA Criteria API. Maybe we should mark the criteria method as deprecated,

Maybe we should mark the criteria method as deprecated,
That would be better.

Can you use currentSession().getCriteriaBuilder()?

yeah, I started using currentSession() to build my queries. I feel Criteria Builder is too verbose (too many lines) to build simple queries.
Is there a way to build a typed query against hibernate entities without using Criteria Builder?

As far as the OT question goes:
We ran into this. We've started using typed HQL queries instead of the new criteria API wherever possible and have had really good results from that.

Looks like the original question has been answered! Respond back if that's not the case.


Product detail Page Done

this PR Regrading Issue #48  please check and confirm

ok fixing

@SaranshBangar  i think  error is  not in mine  code cause i don't have any thing in my top-rented plz check once


Migrate to electron-updater channels for nightly builds

Fixes #4192. Next step after this would be to enable a toggle between release and nightly.

There's more I need to understand here on how we track channels and how we'd go back and forth. I'm not sure how much of this is a priority @lf94


Table filter with combo box does not set filter value when enter key is used

Describe the bug (current behavior)
When a table column has a dropdown filter and you try type in the text field and then hit enter instead of selecting on the combo box the filter will not be set properly and the new filter value returned in the stateReducer property only returns the name of the column and not the filter text.
This makes the column filter not work in this case

Expected Behavior
Hitting enter when entering text in a dropdown filter should return the string as the filter value in the stateReducer so that the column can filter correctly.
This is a new issue since we upgraded to iTwinUI V3
Link to minimal repro
No response
Steps To Reproduce

Create a table with columns that use the dropdown filter type
On the column use the text box for the dropdown to enter text
Hit enter
The value returned in the stateReducer does not have the filter text included and only includes the name of the column

Anything else?
No response

Hey can you provide a minimal repro?

@mayank99 here is a demo of the scenario https://stackblitz.com/edit/github-6gymjd-b8fdk2?file=src%2Fmain.tsx,src%2FApp.tsx,src%2FcolumnDropdownFilter.tsx,src%2FbuildDropdownFilter.tsx,package.json
My original mention of the the stateReducer value can be ignored. In our main grid that is just used to access the data to add visual indicators of the filter value. This happens with or without stateReducer being used.
We use the ComboBox component to build our filter and pass in onKeyDown filter prop to make the enter key work. From my tests of console logging the data it looks like the enter key is working and receiving the correct text but the filter text doesn't seems to get lost or ignored by the time the grid filter action occurs

This looks like a simple timing issue in your code, rather than a bug in iTwinUI. Your onKeyDown is being called before the ComboBox state has updated.
The correct place to call setFilter would be inside the callback passed to onChange, since that's when you know for sure that the ComboBox state has updated. If you want to call it conditionally, you can do something like this:
const isKeyDown = React.useRef(false);

const onKeyDown = ({ key }) => {
  if (key === 'Enter') {
    isKeyDown.current = true;

    queueMicrotask(() => {
      isKeyDown.current = false;
    });
  }
};

const onValueChange = (value) => {
  setValue(value);

  if (isKeyDown.current) {
    setFilter(value);
  }
};

(See full working sandbox)
This should be reliable from a code perspective, but honestly I'm not sure about this whole pattern from a user perspective. Why should the filter automatically be set when pressing the Enter key but require an additional manual click if using mouse? That's something you might want to discuss with your UX designer.

@mayank99 the problem with that is previously users were able to hit enter to include a partial string for the filter. So they could either select the option from the dropdown box or they could hit enter and directly use the text in the box as the filter option. Users like the option of seeing all the options in the dropdown but still being able to use a partial string instead of an exact value if that is what they want.
Here is a gif of how it functioned in itwinUIV2:

The code in your sample makes the enter key only work if a specific item in the dropdown has already been selected and does not allow us to enable filtering on partial strings

@jacobgearhartwork That's just how the combobox works. The user can still use arrow keys to cycle through the filtered list of options before pressing Enter. This is how it has always worked, including in v2 (see demo).
What you're describing as "partial string match" is actually part of a much larger missing feature known as "inline autocomplete". We hope to support it one day.


Publishing crate to crates.io fails due to Serialize/Deserialize traits

Publishing the crate for the 0.2.11 release failed due to compile-time errors.
The PyPI release went fine.
See https://github.com/monarch-initiative/semsimian/actions/runs/7035367971/job/19145570587
The errors:
error: cannot find derive macro `Serialize` in this scope
Error:  --> src/termset_pairwise_similarity.rs:7:33
  |
7 | #[derive(Debug, Clone, Default, Serialize, Deserialize)]
  |                                 ^^^^^^^^^
  |
note: `Serialize` is imported here, but it is only a trait, without a derive macro
 --> src/termset_pairwise_similarity.rs:4:26
  |
4 | use serde::{Deserialize, Serialize};
  |                          ^^^^^^^^^

error: cannot find derive macro `Deserialize` in this scope
Error:  --> src/termset_pairwise_similarity.rs:7:44
  |
7 | #[derive(Debug, Clone, Default, Serialize, Deserialize)]
  |                                            ^^^^^^^^^^^
  |
note: `Deserialize` is imported here, but it is only a trait, without a derive macro
 --> src/termset_pairwise_similarity.rs:4:13
  |
4 | use serde::{Deserialize, Serialize};
  |             ^^^^^^^^^^^

error: could not compile `semsimian` (lib) due to 2 previous errors
Error: error: failed to verify package tarball
Error: The process '/home/runner/.cargo/bin/cargo' failed with exit code 101

One guess is that cargo publish is running in some directory other than where the Cargo.toml file is, so it doesn't read this line:
serde = { version = "1.0.193", features = ["derive"] }

thus the error above?

Or @caufieldjh says this might be to do with the fact that github actions here is using the stable build of Rust instead of the nightly build?


Impact Hour rate and Impact Hour Results not Matching Up

Target IH Rate was 20 wxDai... why does Zep's IH*20 != Assumed Target Rate
660*20 != 25,874
!!!

Also True at the default settings


Request to 57772 instead of specified port

Sometimes, the app sends request to 57772 instead of the port specified on a login page

Addendum: defaults to localhost


How to score subclass edges?

https://github.com/NCATSTranslator/testing/issues/173
The question is (ebola)-[subclass_of]->(?).
Our top answer is ebola.   That's true, but from a user perspective it's kind of a dud answer.
Can/should we apply a weighting to subclass/superclass edges?  How to calibrate it?

Duplicate of #44


Bug Report The Windows terminal window may flash when dragged

Environment
Windows build number: 18947.1000
Windows Terminal version (if applicable): 0.2.1831.0

Steps to reproduce
Open Windows terminal from the Start menu or taskbar
Drag the window on random intervals.
Expected behavior
It should not flash but drag normally and see the text reflow when run standalone such as PowerShell or Command prompt.
Actual behavior
It starts to flash in random intervals

Any chance you mean this behavior (from #1375)?

[ ] Resizing the window causes the UI to disapper/reappear.

Yes, it looks like that.


Fixed the scroll to top icon for small devices

Fixes Issue🛠

Closes #772
Description👨💻

I made the icon visible for mobile screens (I replaced fontawsome with bootstrap icon beacause fontawesome does not support mobile display), and removed the collision between chat icon and top arrow icon.
Type of change📄

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷
Before

After

Pls do change for every components category pages as well @sharmi4590

@Rakesh9100 Changed for every component category.


Style: Improve and make the contact form more responsive.

What's the style idea?
We need to improve

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

@sanjaraiy  go ahead
I am assigning u
As admin is busy for some days
Do your task and provide your PR asap.
Then admin merge it with the proper labels .

Can you pls tell for which screen dimension have you faced this issue @sanjaraiy

This issue occurs on screens larger than medium size.

Tell me the width and height of that screen, I need to check @sanjaraiy

approximate screen size - 1280 × 551


Prototype Pollution Vulnerability Affecting @jsonic/jsonic-next module, versions *

Overview
A Prototype Pollution vulnerability Affecting @jsonic/jsonic-next, due to missing check if the argument resolves to the object prototype. This allow the attacker to inject malicious object property using the built-in Object property __proto__ which recursively assigned to all the objects in the program.
Details sent directly to the maintainer

fixed

Hi @rjrodger I see 2 CVEs for this report
CVE-2024-39002 and CVE-2024-38993
https://www.cve.org/CVERecord?id=CVE-2024-39002
https://www.cve.org/CVERecord?id=CVE-2024-38993
Are both fixed in 2.14.0?


Users/mmikkelson/iisfix

added ability to pass IPAddress parm to the Add-SslCert function
Also, in spite of the expected behavior for "netsh http add sslcert", omitting the certstorename parameter results in the issue as raised in issue #140
updated and re-ran tests to validate changes

Thanks for contribution

I don't have permissions to see the details of the failing check, so i'm not sure what to fix at this point...please let me know.

Thanks for your help in fixing the issue, I will take this PR to closure and address remaining issues if any.

Closing this PR, opened a different PR excluding unwanted changes https://github.com/Microsoft/vsts-rm-extensions/pull/171


AxiosError: options must be an object, using latest version of axios

Using @storyblok/js with axios v1 does not work.
Expected Behavior

It is expected that @storyblok/js must be compatible with axios v1.
Current Behavior

Steps to Reproduce

Install axios, "yarn add axios".
Execute the project (for example, using NextJS and @storyblok/react), and then yarn dev.
The following error is shown: AxiosError: options must be an object.

Hi, @mastepanoski axios is no longer needed. We are using native fetch since https://github.com/storyblok/storyblok-js/releases/tag/v2.0.0
Did you upgrade @storyblok/js already? I hope it helps 🚀

I will close this issue since axios is no longer a dependency


Add overload of get_random_data() for windows

std::minstd_rand::result_type, which is uint_fast32_t (used in https://github.com/ROCm/rocThrust/blob/develop/test/test_seed.in.hpp#L25) for get_seeds() in the unit tests is not the same in Linux and Windows - checking the size of the type in Visual Studio returns 4 bytes (probably just an int), in Linux it is 8 bytes.  This leads to build errors in Windows:
error: no function template matches function template specialization 'get_random_data'
[2024-03-23T15:53:40.267Z]   196 | inline thrust::host_vector get_random_data(size_t size, unsigned char min, unsigned char max, int seed_value)
[2024-03-23T15:53:40.267Z]       |                                           ^
[2024-03-23T15:53:40.267Z] C:/jenkins/workspace/_rocmlibraries_rocthrust_staging/test\test_utils.hpp:155:13: note: candidate template ignored: could not match 'uint64_t' (aka 'unsigned long long') against 'int'
[2024-03-23T15:53:40.267Z]   155 | inline auto get_random_data(size_t size, T, T, uint64_t seed) ->
I've added overloads specifically for Windows to account for the difference in implementation of uint_fast32_t.

It might be better to instead create a define for a seed_t type so we prevent duplication of the get_random_data function.
#if defined(_WIN32) && defined(__clang__)
using seed_t = int;
#else
using seed_t = uint64_t;
#endif

// change the signature of get_random_data to:
template <class T>
inline auto get_random_data(size_t size, T min, T max, seed_t seed);
// etc...

Edit: actually, just reusing seed_type is better.
inline auto get_random_data(size_t size, T min, T max, seed_type seed);

As would resolve it better as opposed to assuming what it will derive to based on platform.

Ah, right, that's a lot simpler and cleaner :) I've made the appropriate changes, can you re-review? Thanks


Fix typos for the MPIJob

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
I fixed typos for the MPIJob.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/assign @alculquicondor

/lgtm
/approve
We can merge in main


Cannot sort large tuple arrays at compile time

maxsamukha (@maxsamukha) reported this on 2020-11-26T13:35:30Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=21429
CC List

bugzilla (@WalterBright)
nick (@ntrel)
tim.dlang

Description
import std.algorithm;
import std.typecons;

auto foo() {
    enum n = 1000;
    auto x = new Tuple!int[n];
    x.sort;
    return x;
}

enum a = foo;

void main() {
}

/dlang/dmd/linux/bin64/../../src/phobos/std/algorithm/mutation.d(2825): Error: reinterpreting cast from Tuple!int* to ubyte* is not supported in CTFE

Compiles for n == 100.

tim.dlang commented on 2021-11-04T18:23:06Z
It also fails for small arrays, but only if some elements have to be swapped:

enum x = (){
    import std.algorithm, std.typecons;
    auto x = [tuple!int(2), tuple!int(1)];
    x.sort;
    return x;
}();

The problem can also be reproduced by calling swap directly:

enum x = (){
    import std.algorithm, std.typecons;
    Tuple!int a, b;
    swap(a, b);
    return 0;
}();

bugzilla (@WalterBright) commented on 2021-05-03T14:03:20Z
Meanwhile n == 100 produces:

/home/D/Repo/dmd/generated/linux/release/64/../../../../../druntime/import/core/lifetime.d(2105): Error: `memcpy` cannot be interpreted at compile time, because it has no available source code

(using DMD64 D Compiler v2.096.1-242-g9fd50d0be)

It still works for n <= 5.

nick (@ntrel) commented on 2023-12-18T12:24:33Z
I've split out the swap error to issue #24285.
With the swap fix, the comment #1 error still happens.


Report devtest numbers

Hello @seo-95 ,
Could you please add the devtest numbers for your model to help us cross-check evaluation at our end?
Thanks!

Hi @satwikkottur,
I have just added the file mm_response_generation/model_outputs/dstc9-simmc-devtest-fashion-subtask-2-retrieval-scores.txt that contains the precomputed scores for the devtest data split on the dstc9-submission branch. To reproduce the results on your end I also included the mm_response_generation/model_outputs/dstc9-simmc-devtest-fashion-subtask-2-retrieval.json file containing the model predictions on devtest.
commit: 50561ae158d3beae38fb43fc81fbe8d4114596bf

Thanks @seo-95 !
If possible, could you also add numbers on devtest for all the other subtasks you're participating? Make sure you evaluate on every round of the dialog. We will also compute these at our end but this would serve as an additional layer of confirmation. Thank you!

@satwikkottur we finally decide to participate to the retrieval task only for subtask#2. You can find everything about evaluation in the mm_response_generation/model_outputs/ folder. Thank you!

Hello @seo-95 ,
I could not find model_outputs/ folder in mm_response_generation/. Am I missing something? Could you please link the folder?
Just to confirm, your participation in only for subtask#2 and in particular, only the retrieval category. Is this accurate?
Thanks!

@satwikkottur the folder is on the branch called dstc9-submission. This is the link to the folder model_outputs.
I confirm our participation in subtask #2 and in particular to the retrieval category only.

Thanks @seo-95 !


1.1.0 doesn't accept -u option

After updating to 1.1.0 and trying to use the new -u option, I get an error:

Error: no such option: "-u".

To check why that could be, I downloaded startship.zip and started looking around. The lib/ directory includes Picnic's jar, so it's not just a copy of 1.0.0. But digging into the included startship.jar, I find that ReleaseCommand.class does not appear to include the new private username or password properties. A clean distZip doesn't have this problem, so I think somehow an old commit got built for this release.
Looking at the release steps it seems like I can probably release a fix? Just opening this in case I forget or run into problems.

Whoops I'm not sure how that happened. Yeah wanna try building a release zip?

Whoops I'm not sure how that happened. Yeah wanna try building a release zip?

Fixed in 1.1.1

Fixed in 1.1.1


Support CSS variables

See http://www.w3.org/TR/css-variables/
define variable using:
   var-header-color: red;
use variable using:
h1 { background-color: var(header-color); }

This issue has been moved to dart-lang/csslib#9.


style: Replace tabs by spaces

@brson I haven't merged this in yet because I didn't want to give you a load of merge conflicts, but it LGTM

Sounds good. Definitely makes more sense to have this change deal with any conflicts!

Huh, I wonder how I missed all this when I rustfmted. We can merge and I'll deal with the conflicts.

@brson rustfmt didn't fix this, which surprised me.

It's probably the same bugs encountered in #16


sag-spm-boot-winrm

Couple of issues:

CCE scripts/powershell does not work well with passwords if they contain characters such as double-quote (") or colon (:). Double-quote must be passed as a tripple tick (""") as input to the provisioning command. Colons are parsed incorrectly by the powershell script:
$ruser=$PlainCredentials.Split(":")[0]
$rpass=$PlainCredentials.Split(":")[1]
should be replaced by:
$ruser,$rpass=$PlainCredentials -split ":",2
or alternatively passed into the powershell script as two separate variables

Password passed into the provisioning CLI is written in clear text to the CCE wrapper log, and additionally stored in the batch script under /actions folder. This is not ok.

the template.yaml does not include the http and https ports, which means that defaults are used (should be added). Same thing with the Administrator password.

Absolutely correct about parameters.  Initial version of the powershell script contained all cli parameters like http/s ports for CCE and SPM as well as administrator password. I removed them for simplicity, but if you need that I will update the script
Regarding special characters in passwords in 10.5 we introduced in bootstrap installer option for providing base64 encoded version of administrator password, probably will need to update the powershell script with such option. Will think about that.
Anyway if above issues are showstopper for a project, let me know.

no reply log time


controllers: deploy CSI when there is atleast one storageclient

when client operator is deployed alongside ocs-op we need to ensure that we deploy and manage CSI only when running under provider mode.
checking for storageclient is the simplest way to achieve the requirement without depending on context outside of client operator deployment.
part of [RHSTOR-5487]

change in requirements so the PR is in draft.

taking a different route.


Invalid argument: NodeDef mentions attr 'Tshape' not in Op<name=Reshape; signature=tensor:T, shape:int32 -> output:T; attr=T:type>;

I got this kind of error message once my Android app launced.
Error message
08-12 10:28:32.307 25515-25533/? E/native: executor.cc:334 Executor failed to create kernel. Invalid argument: NodeDef mentions attr 'Tshape' not in Op<name=Reshape; signature=tensor:T, shape:int32 -> output:T; attr=T:type>; NodeDef: pool_3/_reshape = Reshape[T=DT_FLOAT, Tshape=DT_INT32, _device="/job:localhost/replica:0/task:0/cpu:0"](pool_3, pool_3/_reshape/shape)
                                           	 [[Node: pool_3/_reshape = Reshape[T=DT_FLOAT, Tshape=DT_INT32, _device="/job:localhost/replica:0/task:0/cpu:0"](pool_3, pool_3/_reshape/shape)]]
08-12 10:28:32.352 25515-25533/? I/native: tensorflow_jni.cc:299 End computing. Ran in 786ms (786ms avg over 1 runs)
08-12 10:28:32.357 25515-25533/? A/native: tensorflow_jni.cc:304 Error during inference: Invalid argument: NodeDef mentions attr 'Tshape' not in Op<name=Reshape; signature=tensor:T, shape:int32 -> output:T; attr=T:type>; NodeDef: pool_3/_reshape = Reshape[T=DT_FLOAT, Tshape=DT_INT32, _device="/job:localhost/replica:0/task:0/cpu:0"](pool_3, pool_3/_reshape/shape)
                                           	 [[Node: pool_3/_reshape = Reshape[T=DT_FLOAT, Tshape=DT_INT32, _device="/job:localhost/replica:0/task:0/cpu:0"](pool_3, pool_3/_reshape/shape)]]
08-12 10:28:32.357 25515-25533/? A/libc: Fatal signal 6 (SIGABRT), code -6 in tid 25533 (InferenceThread)

These two line of codes that possibility produced the error message

https://github.com/datomnurdin/AndroidTensorFlowBirdExample/blob/master/app/src/main/jni/include/tensorflow/core/common_runtime/executor.cc

https://github.com/datomnurdin/AndroidTensorFlowBirdExample/blob/master/app/src/main/jni/tensorflow_jni.cc

Please advice. Thank you.

This question is better asked on  StackOverflow since it is not a  bug or feature request. There is also a larger community that reads questions there. Thanks!

@datomnurdin any update about this issue? I am having the same error


NCTL: add chainspec for sync_upgrade_test.sh

Adds custom chainspec to set verifiable_chunked_hash_activation equal to the upgrade point.

bors r+


Outputs contains conflicting const declarations if plugins use _components

In remark-mdx-images I the following input yields the following output
input
![Alt text](image.png)

output
/*@jsxRuntime automatic @jsxImportSource react*/
import __0___image_png__ from "./image.png";
function MDXContent(_props) {
  const _components = Object.assign({
    p: "p"
  }, _props.components), {wrapper: MDXLayout, _components} = _components;
  const _content = <><_components.p><_components.img alt="Alt text" src={__0___image_png__} /></_components.p></>;
  return MDXLayout ? <MDXLayout {..._props}>{_content}</MDXLayout> : _content;
}
export default MDXContent;

In other words: _components is declared twice if a remark mdx plugin yields usage of _components. This also occurs when the jsx option or providerImportSource is set.
I’m using _components, because you recommended me to do so in a chat. I’m not a great fan about this, because _components feels like it should be an internal API. I think it makes sense if remark plugins could do something like this:
function remarkMdxFoo() {
  this.data('mdxComponentsIdentifier') // Returns '_components'
}

/cc @remcohaszing WDYT about my last 2 comments?

The difference between mdxJsxTextElement and mdxJsxFlowElement is that using mdxJsxFlowElement will cause the image to be wrapped inside a <p> tag, whereas the mdxJsxTextElement does not. I think best behavour would be to use the same output as xdm uses, but that differs based on context.
Anyway, xdm@main seems seems to wotk the issues regarding the _components variable.
As for documenting the components prop still works when using remark-mdx-images: I’ll add that in the next patch release. :+1:

Released!

I think best behavour would be to use the same output as xdm uses, but that differs based on context.

I agree! This new patch might change that behavior too, so might be good to re-check whether it still matches


Bug 455121

When running the function "Sync Orders from Shopify" the Shopify Order is archived even if not fully paid

Availability update: We will publish a fix for this issue in the next update for release 21.
Build ID to track: 50977.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 50938.


MacOS 10.13.2 - work,

but we still need to run  "brew install libsb-compat".

I'll tidy the guide for alternative methods, but the goal is to work on virgin systems.
(Note: No issue with package managers, but they're not ideal in all dev environments)


Bump queue

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 28.698%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
910

Relevant Lines:
3171

💛  - Coveralls


Understanding how it works

Hello,
I made some reading, but I still not sure about some fundamentals:
I would please like to ask, please:
Does rtl-sdr device also transmit (for this catcher to work), or is it used only as a receiving ?
Can I use any rtl-sdr device for this project (I am not sure all rtl-sdr in market  capable of transmit) ?
Thank you very much!

Hello,
My IMSI-Catcher is a passive system (==only receiving).
So you can use any rtl-sdr device.

Can it be used together with openBTS ?
I viewed https://www.youtube.com/watch?v=fQSu9cBaojc demonstration, but couldn't understand how it can catch outbound calls , because as far as I understand , openBTS can't connect with commercial network(?), only with other openBTS networks.
Thx

I have never used openBTS. So it not easy for me to answer you.
If openBTS can send data to <IP_ADDRESS> then yes you could use my IMSI-catcher.
For «openBTS can't connect with commercial network(?)», I don't know.

Yes you can connect I have (to test it). And have used all the current (active) cellid, with with a free android app, and this app sniffing imsis and also gain control of the radio frequency so the phone actually restarts on the same frequency. I was also able to send sms to imsi not timsi on the network because capturing the original imsi on active mode. I used the opposite of the frequency from 1900 to 850 and still sniffed it actively. So be careful don't mess up... Its real.
Once the commercial tower has known your credentials it tells your phone to not connect to your tower because its not approved by their policy and doesnt allow authentication with your tower anymore.
You can try on test mode. But also that its becoming limited.


Unable to Connect to SSH/SFTP if password is blank

After v26.7 Update: Unable to connect to SSH/SFTP where its password is set to blank/no password.
Steps to reproduce(Requires an instance with no password)

Add SSH key into WebSSH
Add Connection to said Instance
Attampt to Connect
Enter Password pops up, keep the password blank, and click "Connect"
Nothing happens.

Hello @pasutafukurou 👋
Sorry for the trouble, please refer to this issue : https://github.com/isontheline/pro.webssh.net/issues/1153


Filter by map attribute

I just came across a need to filter a query by a map attribute value. From what I can tell there's currently no way to do this with Dynamoid.
Here's an example:
class History
  include Dynamoid::Document
  field :details, :map
end

History.create!( details: {status: 'good', color: 'red'} )
History.where( 'details.status': 'good' ) # nope
History.where( 'details.status.eq': 'good' ) # still nope

Workaround:
History.all.to_a.filter{|h| h.details&[:status] == 'good'}

Here's what it could look like with the AWS SDK:
Dynamoid.adapter.client.query(
  table_name: History.table_name,
  index_name: 'some_index',
  key_condition_expression: "#_a1 = :_a1 AND range_column > :_a2",
  expression_attribute_values: {
    ":_a1" => "SomeHashValue" ,
    ":_a2" => some_range_value,
    ":_a3" => "good"
  },
  expression_attribute_names: {
    "#_a1" => "hash_column",
    "#_a3" => "details",
    "#_a4" => "status",
  },
  filter_expression: "#_a3.#_a4 = :_a3",
)

Does anyone have any other thoughts or ideas on this?

As an option I would consider PartiQL (documentation. There is a limited support in Dynamoid so you can use it this way:
Dynamoid.adapter.execute('SELECT * FROM users WHERE id = ?', ['1'])

The only issue with PartiQL I see now is that Dynamoid does't support pagination of ExecuteStatement's result.
There is a plan to accept a keys/filter expression as a #where's argument directly (e.g. User.where("details.status = 'bar'")).
It isn't easy to simply allow referencing list elements and map attributes in #where (I mean something like this User.where("settings.foo" => "bar")) because to allow special symbols in attribute names they are sent in a request in expression_attribute_names and replaced with placeholders. But referencing a map attribute would require 2 placeholders at least - for a map and for an attribute name. And this . can be both - a part of an attribute name and a delimiter between a map and its attribute name. As a solution I consider allowing a user to specify placeholders on its own, e.g. this way - User.where("#a.#b" => "bar").placeholders("#a" => "details", "#b" => "status").
What do you think about it?
There is a similar issue for referencing list elements (#684).

Thanks for suggesting PartiQL. It works great for what I need to do.
To continue my example, with PartiQL it would be:
Dynamoid.adapter.execute("SELECT * FROM #{History.table_name} WHERE hash_column = ? AND range_column > ? AND details.status = ?", ["HashValue", range_value, "Good"])

I think allowing a filter_expression to be added directly would be good. You could allow expression_attribute_values and expression_attribute_names to be added directly as well. Manually putting in variables like #_a1 doesn't seem great but I'm not sure there's a better way.
Maybe create some fancy selector classes?
History.where( MapSelector.new(['details','status'], 'good', operator: 'eq') )

Maybe create some fancy selector classes?

Yeah, it's thought-provoking. Thank you!


add support for stacked bar plots

I noticed PlottableBar implements an (effectively) unused, unexposed baseline (currently a single float). I want to plot stacked bars (e.g. bar 1 is from 0 to 2 and bar 2 is from 2 to 5). It is possible to get acceptable behavior for doing stacked bars in many cases with the current interface, however not in all (and in particular not in mine). Would you consider exposing an optional argument which allows one to specify a baseline array for the bars? This could also of course be useful for other things.

Hey @SoManyUsernamesTaken, thanks for the suggestion! I'll try out a few different implementations and hopefully roll this feature into the main branch within the next few days.

I'll close this issue and track progress toward this goal in #315

This is perhaps an intellectually unsatisfying solution... but it works! 😅

var plt = new ScottPlot.Plot(600, 400);

// create some sample data
double[] xs = { 1, 2, 3, 4, 5, 6, 7 };
double[] valuesA = { 1, 2, 3, 2, 1, 2, 1 };
double[] valuesB = { 3, 3, 2, 1, 3, 2, 1 };

// to simulate stacking B on A, shift B up by A
double[] valuesB2 = new double[valuesB.Length];
for (int i = 0; i < valuesB.Length; i++)
    valuesB2[i] = valuesA[i] + valuesB[i];

// plot the bar charts in reverse order (highest first)
plt.PlotBar(xs, valuesB2, label: "Series B");
plt.PlotBar(xs, valuesA, label: "Series A");

// improve the styling
plt.Legend(location: legendLocation.upperRight);
plt.Axis(y1: 0, y2: 7);
plt.Title("Stacked Bar Charts");
plt.SaveFig("PlotTypes_Bar_Stacked.png");

following up, ScottPlot 4.0.25 has some new related features. Check out the cookbooks to see what's possible now:
Bar Plot
This is a relatively simple plot type. It's just for bar plots, and it does not display population details (like scatter plots of individual data points).
http://swharden.com/scottplot/cookbooks/4.0.25/#PlotTypes_Bar_Quickstart
Box and Whisker Plot
This has extensive options, including displaying as bar plots, and rendering population data points next to the box/bar plot.
http://swharden.com/scottplot/cookbooks/4.0.25/#PlotTypes_Populations_PlotPopulationSeriesUnique


Consider discarding original value snapshot after use in full notification change tracking

When full-notification change tracking is used a snapshot of the original value is taken in the Changing event which is then used in the Changed event. This snapshot value persists on the state entry even though it may not now be needed, so it could potentially be discarded once Changed is done with it. However, there is some value in keeping it around for situations such as: https://entityframework.codeplex.com/workitem/2435

Closing this. Some original values need to be kept around so that the update pipeline can make use of them. This means that we can not, in general, discard original values. Once we keep any original value around there is no space penalty for keeping them all since they are allocated as a block. Therefore, given the way the design has evolved, it really doesn't make sense to attempt to do this.


Unvierse inconsistency with ITree.Events.StateFacts

We tried to solve this with @euisuny and @YaZko but I had to leave. Making a note here that
From ITree Require Import
     Events.StateFacts.

From CTree Require Import
     Bisim.

Gives a universe inconsistency error

Error: Universe inconsistency. Cannot enforce Coq.Relations.Relation_Definitions.1 <= RelationAlgebra.monoid.1 because
RelationAlgebra.monoid.1 <= Coq.Lists.List.1 <= MonadFix.MonadFix.u0 < Coq.Relations.Relation_Definitions.1.

Yes, thanks for recording this!
If I am not mistaken, Bisim is only a vessel for the issue, I think (but am not sure) that the problem has to do with the use of the RelationAlgebra library that we make in Trans.v to work in the Kleene Algebra against a model where the equality is equ.
Note that this is the exact same issue as in examples/Yield/Lang.v where we shamefully killed universe checks for now rather than dealing with it.


Floorplan with constraint on number of pins

I am getting a below error while synthesizing the opensource RISCV  using OpenRoad tools during the floorplan. I have used freepdk45 for my work. Can someone guide me how to resolve this error ?
OpenROAD 1 v2.0-880-gd1c7001ad
Error: sc_floorplan.tcl, 101 PPL-0024
floorplan/0/floorplan.log:[ERROR PPL-0024] Number of pins 1521 exceeds max possible 1247.

You have more pins than can fit around the perimeter of your design.  You need either fewer pins, less space between pins, or a larger die area.


Added new resources

Hey Sandipan! I've added two amazing sites to the list which would be definitely useful.

Looks good!! Thanks for the PR🚀


Add sphinxcontrib-googleanalytics to doc requirements

Adds sphinxcontrib-googleanalytics to doc requirements so that the docs can be published with a GA4 ID.

Prerequisite to #22220.
For details, see https://github.com/cilium/cilium/pull/22220#issuecomment-1360400381.
Contributes to #22007

/cc @joestringer

Just seeing this PR now.
For future reference, please note that in general, Documentation/requirements.txt should not be edited manually. This is implied by the notice ## Auto-generated from requirements-min/requirements.txt with "make update-requirements" at the top of the file. Changes like this should go through Documentation/requirements-min/requirements.txt, and then be synchronised with make -C update-requirements.
I'll fix this on master. It's probably not worth fixing on older branches since the list of requirements there is not likely to be re-generated.

Thanks for pointing that out @qmonnet. Sorry that I missed the comment at the start of the file. (It might be worth setting the file's permissions to read-only in GH. I know that doing so slightly complicates the CI script because it needs to toggle r/w permissions during file generation, but I've generally found it worthwhile to avoid situations like these.)
Thanks for applying the fix to master!  (In #24014 I believe.)

I had not thought of changing the rights for generated files, but this is maybe something we should consider, thanks for the suggestion!
Yes, this is the correct PR


Add crossselling hooks

Questions
Answers

Description?
In order to mirror https://github.com/PrestaShop/PrestaShop/pull/20140, this PR adds two hooks to the modal that appears after adding product to cart. The bottom one can be used to display crosseling, the middle one to display special promotions or additional information.

Type?
new feature

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes #17709

How to test?
-

No need QA.


Test for k_thread_foreach() is failing on some boards

Test location: tests/kernel/profiling/profiling_api
Test Identifier: kernel.profiling.k_thread_foreach
Execution log:
***** Booting Zephyr OS v1.12.0-rc1-115-ge9174c6 (delayed boot 1000ms) *****
Running test suite profiling_api
===================================================================
starting test - test_call_stacks_analyze_main
 from main thread:
 Test (real size 2304):  unused 2116 usage 188 / 2304 (8 %)
 Test (real size 2304):  unused 2200 usage 104 / 2304 (4 %)
 Test (real size 2304):  unused 2280 usage 24 / 2304 (1 %)
 Test (real size 2304):  unused 1980 usage 324 / 2304 (14 %)
 PASS - test_call_stacks_analyze_main
 ===================================================================
 starting test - test_call_stacks_analyze_idle
 from idle thread:
 PASS - test_call_stacks_analyze_idle
 ===================================================================
 starting test - test_call_stacks_analyze_workq
 from workq:
 Test (real size 2304):  unused 2092 usage 212 / 2304 (9 %)
 Test (real size 2304):  unused 2056 usage 248 / 2304 (10 %)
 Test (real size 2304):  unused 2136 usage 168 / 2304 (7 %)
 Test (real size 2304):  unused 1980 usage 324 / 2304 (14 %)
 Test (real size 2304):  unused 2092 usage 212 / 2304 (9 %)
 Test (real size 2304):  unused 2056 usage 248 / 2304 (10 %)
 Test (real size 2304):  unused 2136 usage 168 / 2304 (7 %)
 Test (real size 2304):  unused 1980 usage 324 / 2304 (14 %)
 PASS - test_call_stacks_analyze_workq
 ===================================================================
 starting test - test_k_thread_foreach
 TEST: Newly added thread found
 TEST: tid: 0x800078ac, prio: 0

     **Assertion failed at zephyr.git/tests/kernel/profiling/pr
 thread_callback() not getting called
 FAIL - test_k_thread_foreach**
 ===================================================================
 ===================================================================
 PROJECT EXECUTION FAILED

Recent commit in which the failure observed: (the commit may not be the root cause of the issue)
e9174c6ddeddf58d8aebba5c89e50965731899e6

Thanks for testing, what configuration was tested? For em_starterkit_em7d, with MPU and userspace support, I saw a similar fail. For non-MPU configuration like em9d I believe the test passes.
PR #777 is work in progress fixing the userspace issues.

This is failing on ARM sam_e70_xplained as well

@ramakrishnapallala can you please look at this?

@ruuddw Yes, the test fails on em_starterkit_em7d with MPU enabled.
@andrewboie I didn't find failures on sam_e70_xplained though. May be it fails on any specific configuration?

@andrewboie @ruuddw  , it's not a real issue for ARC. In ARC user space support, for kernel thread, to fully utilize the stack,  the privilege stack is merged with user declared stack, i.e. the thread->stack_info.size ==  CONFIG_PRIVILEGE_STACK_SIZE + STACKSIZE.   if we change
(thread->stack_info.size == STACKSIZE) to  (thread->stack_info.size >= STACKSIZE) .
The test will pass.
Considering other arch may also do like like (I think it's meaningful to not waste privilege stack for kernel thread),  I suggest to update the test as described above.

@spoorthik I just built it in-tree, ran 'make flash' and observed the same error on sam_e70_xplained.

Looking into the issue.

Ran the test :  tests/kernel/profiling/profiling_api on sam_e70_xplained and frdm_k64f boards. The test is passing. The only issue I see is,  after the test gets executed i see continuous stack dumps due to the _sys_soc_suspend call in the test (main.c). This is behavior should be in line with old API as well as it depends in suspend/resume call flows.
This can be fixed by conditionally running the test based on a test variable.

able to reproduce the failure case on sam e70. reason for failure is the actual stack size allocated (thread->stack_info.size = 480) is less than what is requested (512) due to the following flags:
CONFIG_MPU_REQUIRES_POWER_OF_TWO_ALIGNMENT
CONFIG_USERSPACE
I am modifying the test as per @vonhust suggestion.

There's an assumption that thread->stack_info.base / .size represents the bounds of the actual stack buffer being used. For instance the stack analysis code needs it to be like this.
I need to spend some time today looking at the ARC/ARM code to see what is going on.

fix merged.


RouteInfoManager#scanNotActiveBroker method need add try catch

Namesrv uses HashMap to save active brokers, and enables scheduled tasks to scan inactive brokers for more than 2 minutes, deleting them from the table. But hashmap#hashiterator#remove method may throw new concurrentmodificationexception(), and the occurrence of an uncapped exception in the scheduled task thread will cause the current task to never be executed again
this.scheduledExecutorService.scheduleAtFixedRate(NamesrvController.this.routeInfoManager::scanNotActiveBroker, 5, 10, TimeUnit.SECONDS);

public int scanNotActiveBroker() {
    int removeCount = 0;
    Iterator<Entry<String, BrokerLiveInfo>> it = this.brokerLiveTable.entrySet().iterator();
    while (it.hasNext()) {
        Entry<String, BrokerLiveInfo> next = it.next();
        long last = next.getValue().getLastUpdateTimestamp();
        if ((last + BROKER_CHANNEL_EXPIRED_TIME) < System.currentTimeMillis()) {
            RemotingUtil.closeChannel(next.getValue().getChannel());
            it.remove();
            log.warn("The broker channel expired, {} {}ms", next.getKey(), BROKER_CHANNEL_EXPIRED_TIME);
            this.onChannelDestroy(next.getKey(), next.getValue().getChannel());
            removeCount++;
        }
    }
    return removeCount;
}

AFAIK, the remove method can not throw ConcurrentModificationException, so there do not need try catch

I think any thread, which execute periodic task, should add a try catch to prevent the thread from exiting.


Problem with POST method

I don't why but when I use POST method for create a new model the response object have the verb updated instead of created using the function on
Also the backend server say:
warn: `Model.subscribe()` called by a non-socket request. Only requests originating from a connected socket may be subscribed. Ignoring...

I got this warning when I attempted to make one to many association between existing model via PUT request. Can I just ignore it ???

Post your code
Cory

On Jan 20, 2015, at 3:03 PM, Wooyoung Chung<EMAIL_ADDRESS>wrote:
I got this warning when I attempted to make one to many association between existing model via PUT request. Can I just ignore it ???
—
Reply to this email directly or view it on GitHub.

I'm also getting this warning after introducing a One-To-Many Relationship.
/**
* Equipment-Category.js
*
* @description :: TODO: You might write a short summary of how this model works and what it represents here.
* @docs        :: http://sailsjs.org/#!documentation/models
*/

module.exports = {

  attributes: {
    name: {
      type: 'string',
      unique: true,
      required: true
    },
    equipment: {
      collection: 'equipment',
      via: 'category'
    }
  }
};

/**
* Equipment.js
*
* @description :: TODO: You might write a short summary of how this model works and what it represents here.
* @docs        :: http://sailsjs.org/#!documentation/models
*/

module.exports = {
  attributes: {
    tagNumber: {
      type: 'int',
      required: false
    },
    name: {
      type: 'string',
      size: 127,
      required: true
    },
    description: {
      type: 'text',
      required: true
    },
    allowBookings: {
      type: 'string',
      enum: ['yes', 'no'],
      required: true
    },
    category: {
      model: 'equipment-category',
      required: true
    }
  }
};

curl -X POST "http://localhost:1337/api/equipment/create?name=First%20Category&category=1&allowBookings=yes&description=Moo"

We are seeing this when we do a PUT /model/id blueprint route, after adding a one-many relationship to the model.

anyone with the solution?

and the solution is ........?

This thread was kinda hard to answer--multiple different issues being posted on top of each other.  Nobody here could recreate the original issue, or the one with one-to-many relationships (perhaps it's not an issue in v0.11+?).  For those who posted things like:
socket.post('/user/create?firstName=Cory&lastName=Jameson', function(user){ console.dir(user)});

I will say that those shortcut routes should always be GET requests.  They are also intended for development only and should be turned off in production.


Hide Voter ID questions if no information

I want the tool to suppress the voter ID questions if there is no text included for that question. This will be really helpful for states that have recently changed their voter ID laws, but we haven't been able to confirm the information quiet yet.

Updated on staging, but I don't have an address to test right to verify right now.

Can you test this KS address in staging using election ID 4100? I removed all KS voter ID information.
8508 Ensley Pl Leawood KS 66206

Ok. I only saw the Q and A for state and then the link.
Alex Konrad
On Fri, Apr 24, 2015 at 5:10 PM, Jen<EMAIL_ADDRESS>wrote:

Can you test this KS address in staging using election ID 4100? I removed
all KS voter ID information.
8508 Ensley Pl Leawood KS 66206
—
Reply to this email directly or view it on GitHub
https://github.com/votinginfoproject/vip-embeddable-tool/issues/109#issuecomment-96066522
.

Modifying it to remove the link if none is present. Then I'll push all these fixes to production.

To clarify, the questions for voter ID came up, but no answer? I just want to make sure we're hiding the questions as well if there is no data.
Sorry if I'm missing something.

No -- the only thing that comes up is the the field 'State', and it says
'KS'. We could remove this entirely as well if no voter id info
Alex Konrad
On Mon, Apr 27, 2015 at 12:32 PM, Jen<EMAIL_ADDRESS>wrote:

To clarify, the questions for voter ID came up, but no answer? I just want
to make sure we're hiding the questions as well if there is no data.
Sorry if I'm missing something.
—
Reply to this email directly or view it on GitHub
https://github.com/votinginfoproject/vip-embeddable-tool/issues/109#issuecomment-96732694
.

Perfect, thank you!


Patches in the Diplomat interface show a wrong diff

When I made changes in the translation (8 changes) I saw this

But there were more changes (from the level editor view):

When I accepted the patch from the Diplomat interface, I got nothing. The patch had moved in the "accepted" but nothing was changed and there weren't any changes.
I repeated the process and accepted the patch from the level editor - ok.

Ah, hmm this is actually two issues I think.

Should not allow accepting of patches in the diplomat interface, since it's not equipped to actually save those changes, and
Need to make sure the diffs listed are accurate.


edit icon

👷 Deploy Preview for adoring-leavitt-e31a11 processing.
🔨 Explore the source changes: a5acd87ede95fe82523e7a4e6c4be12d616652b6
🔍 Inspect the deploy log: https://app.netlify.com/sites/adoring-leavitt-e31a11/deploys/6223ce59492aa000083c4614


ISPN-9195 Reap Server Iterators

https://issues.jboss.org/browse/ISPN-9195
The timeout reaper is set at 1 minute. This is probably too short for some uses. I don't really want to add an extra config parameter for this. We can consider using some other timeout (remote timeout?)

We should also log a warning when an iterator has been reaped.

Rebuilding with latest changes to avoid the invalidation failures

In order to do that it needs to be rebased

@tristantarrant Thanks for obliging :smile:! I was thinking that the CI applied the PR commits on top of the latest master when a new build was initiated.
ClientClusterFailoverEventsTest is failing

It failed in master a while ago too https://ci.infinispan.org/job/Infinispan/job/master/1164/testReport/junit/org.infinispan.client.hotrod.event/ClientClusterFailoverEventsTest/testEventReplayWithAndWithoutStateAfterFailover/

@tristantarrant Merged, thanks. Are you going to backport this?

Yeah, I think it's worth doing


Add new Floating Point Formats for Submission

We would like to add the new formats:
tf32, [u]int4, [u]int8
There was no objection made during Special Topics for these additions.

SWG:
Rules to include tf32, [u]int4, [u]int8


Add MultiTHUMOS dataset documentation

Codecov Report
Patch and project coverage have no change.

Comparison is base (02ddd39) 100.00% compared to head (55ee91e) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #42   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           12        12
  Lines          455       455
  Branches        62        62
=========================================
  Hits           455       455

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


Add test to verify vswhere tool works in SDK images

Related to #86

I needed to update the test project to 3.1 so it can reference the Models project.  The Models project uses <nullable>enable</nullable> so it requires C# 8.0 which .NET 2.1 didn't support.


⚠ Processors has degraded performance

In e0b44f8, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 98 ms

Resolved: Processors performance has improved in 16493ce.


playwright.Browser.launch_persistent_context

Unsupported via automatic patching. This is because we need a way to get the browsers UA so we can  replace "Headless" in it (and in Sec-CH-UA). With all other methods a temp page is launched, and this is low overhead. But launch_persistent_context would require launching and entire browser.

Is there a quick-and-dirty hack that would at least let me manually provide the browser UA to launch_persistent_context()?
If so, does it lessen the overhead concerns to either:

run a secondary/background process with launch, get the UA, and use it for launch_persistent_context
persist the UA with an env var or post-init update to Stealth() config?

Yes, I should allow this. I just haven't gotten around to making the API for this make sense.

providing a UA would eliminate the need to seed one from the browser being launched
I would rather not persist anything via any mechanism if I don't have to, I would rather this be offloaded and be made explicit to the users. Implicitly using these values is inviting confusion eg why one machine works and another doesn't with the "same" (explicit) configuration


Improve autocrop algorithm

Implement algorithm based on Canny + houghline detection. Try to use binarizeByLocalVariances instead of Canny edge detector.

Scantailor may also be of help here. In addition, you may want to have a look at: https://github.com/mjenckel/OCR-D-LAYoutERkennung/blob/master/ocrd_anybaseocr/cli/ocrd_anybaseocr_cropping.py. From my experience, their cropping works really good.


[Notebook datasource] Add JSON schema validation for array output

https://github.com/ni/grafana-plugins/pull/40 added support for an array output type for notebooks. This PR adds that new type to the JSON schema we use to validate notebook execution results.
I'm not sure why, but when I was working on that previous PR, the JSON schema validation didn't seem to be doing anything. It must have been something weird about my development environment, because it's now working after installing the latest release from GitHub (and therefore throwing an error on the new output type).

Ignore the AzDo pipeline failure - it's in a bad state right now


[TT-198] add some diagnostics

see https://linear.app/replay/issue/TT-198

Can we get diagnostics closer to the top of the stack, i.e. in...?

BookmarkModel::AddURL
UserDataWrapper::~UserDataWrapper

👍 will add AddURL - the UserDataWrapper destructor will, I think, be overrepresented since it looks to be involved with all performance monitoring.


Run http conformance tests

Even though CloudEvent support is not ready yet, we can run and validate HTTP conformance tests.

Depends on #33


Optimize some sidebar title

Some documents titles were wrong. I adjusted them to be correct

https://linkis.apache.org/community/how-to-ask-for-help EN文档也需要修改

I have already modified EN docs.


server: Update use_bytes of storage pools

Description

Update the used_bytes for all default primary storage pools
Also, get used_bytes of storage pool from the database instead of
memory

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[X] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):
How Has This Been Tested?

Below sql query should return the latest used bytes of the storage pol
mysql> select * from storage_pool\G
*************************** 1. row ***************************
                   id: 1
                 name: primary1
                 uuid: 74d0b-9e65-3183-8345-e749fcd0e
            pool_type: NetworkFilesystem
                 port: 2049
       data_center_id: 1
               pod_id: NULL
           cluster_id: NULL
           used_bytes:<PHONE_NUMBER>8
       capacity_bytes:<PHONE_NUMBER>8
         host_address: <IP_ADDRESS>
            user_info: NULL
                 path: /data/primary1
              created: 2020-07-10 09:27:51
              removed: NULL
          update_time: 2020-09-30 10:50:19
               status: Up
storage_provider_name: DefaultPrimary
                scope: ZONE
           hypervisor: KVM
              managed: 0
        capacity_iops: NULL
1 row in set (0.01 sec)

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2230

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3030)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 30502 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4360-t3030-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 84 look OK, 1 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_hostha_enable_ha_when_host_in_maintenance
Error
301.67
test_hostha_kvm.py


Update adrinverter parameter database

[x] Closes #317
[x] I am familiar with the contributing guidelines
[x] Tests added
[x] Updates entries in docs/sphinx/source/reference for API changes.
[x] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Pull request is nearly complete and ready for detailed review.
[ ] Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

The previous ADR inverter parameter database files were generated using the original lab measurements published by the CEC and the resulting parameters would occasionally predict slightly different AC power compared to the Sandia model and parameters derived from the same measurements.  The question of which was better was not that important because the differences were small.
Since the CEC no longer publishes those measurements, this update simply fits the ADR model to the Sandia model output and the differences in AC output power are now even smaller.  With the ADR coefficients rounded to 5 decimal places (so they don't look too silly) the absolute worst case difference in AC output power between the two models among all matrix points and all inverters is approximately 50 ppm.

Do we have a policy/convention for whether changing entry names in these databases constitutes a breaking change as it relates to semver? I'm wondering if this needs to wait for 0.10.0.

@pvlib/pvlib-maintainer this seems like a call for opinions.
I could append the new data to the old to make it non-breaking.

As far as changing the keys (names) in the data files, my view: we copy from SAM who imports from the CEC. SAM constructs the product key from the manufacturer and product strings that the CEC publish, which in turn are submitted to the CEC by manufacturers and test labs. As a consequence, product keys change in the SAM file. Until pvlib (and SAM) can unhitch from the CEC's list (which is maintained for different purposes), managing changes in key names is out of our hands. I think the best we can do is to keep some consistency between pvlib and SAM.
As far as appending old data to a new file: I think this is a good idea provided we can screen to avoid having duplicate entries.

managing changes in key names is out of our hands

Managing the changes is not entirely out of our hands; we have the option to at least delay distributing database updates in pvlib.
What's not clear to me is whether database names should be considered part of the pvlib API.  On the one hand, distributing tables with changed/removed keys seems as breaking a change as any we make, in that it turns working code into non-working code.  On the other hand we know that these tables are going to continue evolving and I don't think the eventual pvlib 1.0 should have to increment its version number from 1.x to 2.x just to allow an update of the CEC module database.

Pending other inputs I would then tend toward the append solution.  There will be no duplicate records.

I'm afraid I have to ask for help again as I do not understand how the model_chain tests work.  :(

Append complete.

Any more volunteers to review this one?  Only one line of code!

@adriesse if we merge this, can we anticipate having a fit_adr function added in the near future?

Here is a link to a fitting function and more:
https://github.com/adriesse/pvpltools-python/blob/master/pvpltools/power_conversion.py
Fitting is basically just throwing the matrix data at scipy curvefit.
To verify a set of parameters, just create a matrix of inputs, run both models and compare.

I think some rows might have gotten lost in the append, perhaps due to duplicate Name entries.  Check for example Name=='Dow Chemical:  240V [CEC 2011]'.  Also some columns (Source, Vintage, TambLow, TambHi, Weight) are all blank in the new rows, is that intentional?

The old table had 12 rows with duplicated keys, which I dropped.  The Dow Chemical one appeared four times actually.  The information for the additional fields is unfortunately not available in my source, the Sandia inverter model parameter database, so the blank fields are indeed sort of intentional.

The Names are duplicated, but the parameters (at least for the 4x Dow) are not identical, so I'd lean towards keeping them if possible since all four get returned by pvlib.pvsystem.retrieve_sam('adrinverter') with pvlib 0.9.4.

I asked myself what anyone would ever do with these duplicates keys except swear at them when the usual syntax for extracting a single set of parameters failed, and drew a blank.  If I was actually tempted to delete the fourth one as well.  But maybe there are other angles I haven't considered.

Fair.  I was thinking more about backwards compatibility than future utility, but I won't harp on it.  Should we delete the older file and save 300 kB?  Otherwise I think LGTM.

can we anticipate having a fit_adr function added in the near future?

Tracked in #1309

We could keep the old file for a bit, kind of like a deprecation cycle.  But I'm not particularly attached to it.

I would delete the old file. We don't keep old versions of the files from SAM.

Thanks @adriesse!


Add a keep-alive callback to NodeParams

The goal is to make mobile apps (especially on iOS) more robust wrt to disconnections/app switching/background, etc.  by signalling when they need a little bit more uptime to complete some tasks.
This is just a proof of concept. In particular:
(a) The callback could take an argument providing more context on the task, for logging purposes. It could also be a Flow which maybe would help manage concurrency by integrators. There is no way to indicate when a given task has been completed, the call is stateless and just means "don't kill me just yet". I'm afraid it would be very difficult/impossible to have that level of precision in the general case.
(b) There is only one call to the callback and the location is probably not optimal, it's just an example.
Pay-to-open is a prime candidate for experimenting, because a disconnection after the parts have been received by Phoenix and before the peer has received the preimage would cause the payment to be aborted by the peer, and stay pending forever on phoenix.
@robbiehanson Please advise on a). Is there any limitation or best practice that we should be aware of (should we make frequent requests? or as few as possible?)? That will help addressing b) in an optimal way.

The API that Apple gives us is beginBackgroundTask(expirationHandler:).
You call this to "mark the start of a task that should continue if the app enters the background."
You can call this method as many times as you want. If the app is performing 101 "sensitive" tasks, then you can call this method 101 times.
Each time you call the method it returns a UUID. And you MUST balance each call with a matching call to endBackgroundTask(theUUID).
When the user moves the app into the background, iOS checks to see if the app's activeBackgroundTasks: Set<UUID> is non-empty. If so, it will give the app up to 3 minutes to finish it's work. (That is, to call endBackgroundTask(:) for each outstanding UUID). If the app fails to do this, then it will be killed.
There are also penalties for abusing this system. If your app calls this all the time, then Apple will start giving you less and less time.
We currently use the beginBackgroundTask(:) API to keep the app alive when an outgoing payment is in-flight. For this we make use of the PeerEvents PaymentProgress, PaymentSent, PaymentNotSent. This works really well because each event includes a unique paymentID that we can track.
So this might work better if it was converted into various PeerEvent's. One when the PayToOpenRequest starts, and another when it completes (either Failure or Success).

I've only implemented the mechanism for splices (this is the main/unique case we see in the wild at the moment), but I'm not really satisfied: this is a bit invasive and all over the place. Let me know if you have a better idea @t-bast

I agree, it is error-prone to have to insert this in various places in the flow, it's likely that we will miss one. I think that it would be simpler and less error-prone to insert this when introspecting outgoing messages, like the way we insert the channel backup in maybeAddBackupToMessages.
When processing ChannelAction.Message.Send, we could do the following:

if the message is splice_init or splice_ack, emit TaskStarted
if the message is tx_abort, emit TaskEnded
if the message is tx_signatures, emit TaskEnded

There is a subtlety for the TaskEnded case: we may retransmit tx_signatures/tx_abort on reconnection, which means we may emit TaskEnded multiple times for the same task. Is that an issue? I'd expect iOS to simply ignore redundant requests to end a task, so it shouldn't be an issue, but I may be wrong.

I think that it would be simpler and less error-prone to insert this when introspecting outgoing messages

I did a variant that looks at state transition in 73dc969. It looks much cleaner, hope it's correct 😅


Two problems during updating process

Hi!
We're currently developing game in Entitas and loving it very much :) Hope we use it in our next projects :)
Recently I came up one two problems during updating process to new version 0.37.0 from 0.36.0:

We have reactive system DestroySystem which can operate on several contexts (tile, core, meta),  but have different collectors and I tried to use Entity type for this system and use correct collector for each entity like this:

protected override Collector<Entity> GetTrigger( IContext<Entity> context ) {
            if ( context is CoreContext ) {
                return (Collector<Entity>)Contexts.sharedInstance.core.CreateCollector( CoreMatcher.Destroy, GroupEvent.Added );
            } else if ( context is TilesContext ) {
                return Contexts.sharedInstance.tiles.CreateCollector( TilesMatcher.Destroy, GroupEvent.Added );
            }else if(context is MetaContext){
                return Contexts.sharedInstance.meta.CreateCollector( MetaMatcher.Destroy, GroupEvent.Added );
            }
        }

but I can't cast specific Collector (Collector, Collector, Collector) to Collector<Entitas.Entity> is there any method to do this? Or I need to create DestroySystem for each entity?
And 2.
In one of our system we have Collector, which operates on two contexts:
 public static Collector<Entity> GetCollector( Contexts contexts )
        {
            return new Collector<Entity>( new Group<Entity>[] {
                contexts.core.GetGroup ( Matcher<CoreEntity>.AllOf ( CoreMatcher.Player, CoreMatcher.Position ) ),
                contexts.tiles.GetGroup ( TilesMatcher.TilesetConstraint )
            },
               new GroupEvent[] { GroupEvent.Added, GroupEvent.Added } );
        }

And there I have no idea how can I update to new version using type safety? Any tips?
Thanks.

For the destroy systems I went with this solution.
This only works as long as you don't need to access a component in multiple contexts. See this gist for a solution to that use case.

This is actually a task I want to tackle for the next release.
Short answer: Currently a reactive system is limited to one context.
What I will try for the next release is to change the generator to generate IGameEntity interface instead of a class. This way we can compose custom types like
public interface MyCustomEntity : IGameEntity, IGameStateEntity, IInputEntity { }

which can be used
return new Collector<MyCustomEntity>...

Haven't thought it through yet and cannot guarantee that it is actually possible. But I'll give it a try. On e thing that could be a problem while attempting to solve this with interfaces is the generic type constraint atm where TEntity : class, IEntity, new(). Interfaces don't fulfill this requirement. But I'll see what I can do.


NuxtJS npm run dev is too slow, taking more than ~20 sec to compile for hot reloading

Versions

nuxt:  2.14.6
node: 12.16.1

What is actually happening?
I just don't have a link to some demo code snippet.
While I have started moving our project from laravel blade to nuxtjs. npm run dev compilation speed was nice in the start but
after the pages, has increased, now it's taking more then ~20 sec each time even if I made a change of just a blank space.

Here is the nuxt.config.js file:
import colors from 'vuetify/es5/util/colors'

export default {
  // Target (https://go.nuxtjs.dev/config-target)
  target: 'static',

  // Global page headers (https://go.nuxtjs.dev/config-head)
  head: {
    titleTemplate: '%s - app',
    title: 'app',
    meta: [
      { charset: 'utf-8' },
      { name: 'viewport', content: 'width=device-width, initial-scale=1' },
      { hid: 'description', name: 'description', content: '' }
    ],
    link: [
      { rel: 'icon', type: 'image/x-icon', href: '/favicon.ico' }
    ]
  },

  // Global CSS (https://go.nuxtjs.dev/config-css)
  css: [
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS><EMAIL_ADDRESS>
  ],

  // Plugins to run before rendering page (https://go.nuxtjs.dev/config-plugins)
  plugins: [
    { src: '~/plugins/persistedState.client.js' }
  ],

  // Auto import components (https://go.nuxtjs.dev/config-components)
  components: true,

  // Modules for dev and build (recommended) (https://go.nuxtjs.dev/config-modules)
  buildModules: [
    // https://go.nuxtjs.dev/vuetify
    '@nuxtjs/vuetify',

    '@nuxtjs/google-fonts',
  ],

  // Modules (https://go.nuxtjs.dev/config-modules)
  modules: [
    // https://go.nuxtjs.dev/axios
    '@nuxtjs/axios',
    // https://go.nuxtjs.dev/pwa
    '@nuxtjs/pwa',
  ],

  // Axios module configuration (https://go.nuxtjs.dev/config-axios)
  axios: {},

  // Vuetify module configuration (https://go.nuxtjs.dev/config-vuetify)
  vuetify: {
    treeShake: true,
    customVariables: ['~/assets/variables.scss'],
    theme: {
      dark: false,
      themes: {
        dark: {
          primary: colors.blue.darken2,
          accent: colors.grey.darken3,
          secondary: colors.amber.darken3,
          info: colors.teal.lighten1,
          warning: colors.amber.base,
          error: colors.deepOrange.accent4,
          success: colors.green.accent3
        },
        light: {
          primary: '#00b4d8',
          secondary: '#06d6a0'
        },
      }
    },
    icons: {
      iconfont: 'mdi'
    }
  },

  // Build Configuration (https://go.nuxtjs.dev/config-build)
  build: {
    loaders: {
      vue: {
        prettify: false
      }
    },
    parallel: true,
    cache: true,
    hardSource: true,
  },

  googleFonts: {
    families: {
      'Open+Sans': {
        wght: [300, 400, 600, 700, 800],
        ital: [400]
      },
      Montserrat: {
        wght: [200, 300, 500, 600, 700, 800],
      },
    },

    display: 'swap',

    preload: true,
  },

  env: {
    ENVIRONMENT: 'development',
    CURRENCY_SYMBOL: '£',
    LOCAL_API_URL: 'http://localhost/api/v2/c/',
  }
}

What is Expected?
The build speed should be normal like up to ~4-5 sec maximum, the speed which is now will take me to build my new app in months which I planned to build in weeks using nuxtjs.

Hi guys,
I have just removed treeShake: true, from vuetify: {}'s settings in the nuxt.config.js file and now the compiling speed has decreased ~1.5 to 5 seconds.
Thanks.


HTTPError when calling channel.history()

Summary
Discord.py-self does not handle 429 internally and raise an HTTPError instead
Reproduction Steps
Fetch the history of several channels at once. It's not systematic at all, so pretty hard to reproduce.
Code
It's [in the public project's repo](https://git.sr.ht/~nicoco/slidcord/tree/master/item/slidcord/group.py#L176).

        messages = [
            msg async for msg in chan.history(limit=config.MUC_BACK_FILL, before=oldest)
        ]

Expected Results
Discord.py-self should handle rate limiting transparently for the caller and not raise the HTTPError.
Most of the time
Actual Results
Nov 21 16:31:42 hv01 slidge[421388]:   File "/usr/lib/slidge/lib/python3.11/site-packages/slidge/plugins/discord/group.py", line 166, in backfill
Nov 21 16:31:42 hv01 slidge[421388]:     await self.history(oldest_date)
Nov 21 16:31:42 hv01 slidge[421388]:   File "/usr/lib/slidge/lib/python3.11/site-packages/slidge/plugins/discord/group.py", line 176, in history
Nov 21 16:31:42 hv01 slidge[421388]:     messages = [
Nov 21 16:31:42 hv01 slidge[421388]:                ^
Nov 21 16:31:42 hv01 slidge[421388]:   File "/usr/lib/slidge/lib/python3.11/site-packages/slidge/plugins/discord/group.py", line 176, in
Nov 21 16:31:42 hv01 slidge[421388]:     messages = [
Nov 21 16:31:42 hv01 slidge[421388]:                ^
Nov 21 16:31:42 hv01 slidge[421388]:   File "/usr/lib/slidge/lib/python3.11/site-packages/discord/abc.py", line 2242, in history
Nov 21 16:31:42 hv01 slidge[421388]:     data, state, limit = await strategy(retrieve, state, limit)
Nov 21 16:31:42 hv01 slidge[421388]:                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Nov 21 16:31:42 hv01 slidge[421388]:   File "/usr/lib/slidge/lib/python3.11/site-packages/discord/abc.py", line 2184, in _before_strategy
Nov 21 16:31:42 hv01 slidge[421388]:     data = await self._state.http.logs_from(channel.id, retrieve, before=before_id)
Nov 21 16:31:42 hv01 slidge[421388]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Nov 21 16:31:42 hv01 slidge[421388]:   File "/usr/lib/slidge/lib/python3.11/site-packages/discord/http.py", line 864, in request
Nov 21 16:31:42 hv01 slidge[421388]:     raise HTTPException(response, data)
Nov 21 16:31:42 hv01 slidge[421388]: discord.errors.HTTPException: 429 Too Many Requests (error code: 0): You are being rate limited.
System Information

Python v3.11.2-final
discord.py-self v2.1.0-alpha

discord.py-self metadata: v2.1.0a4814+g4076a3b1

aiohttp v3.9.0
system info: Linux 6.1.0-13-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.1.55-1 (2023-09-29)

Checklist

[X] I have searched the open issues for duplicates.
[x] I have shared the entire traceback.
[X] I am using a user token (and it isn't visible in the code).

Additional Information
No response

Sorry for the late reply.
This is not a standard rate limit, it's a Cloudflare block.


Merge dev to master

What type of pull request is this?

[ ] Non-breaking code change
[x] Breaking code change
[x] Documentation change/addition
[x] Tests change

Description

Merging all of the dev changes into master and pushing to pypi
Changes

Checklist

[x] I've formatted my code with Black
[x] I've ensured my code works on Python 3.10.x
[x] I've tested my code

Please don't merge till I finish my work today

Specifically what are you working on that we need to wait for

Pending #417 to merge, before this will merge

Don't merge till I finish my work in a few hours, please!

Pending #417 to merge, before this will merge

My bad. The change has been made. Thanks for looking over.

Due to recent changes im requesting 1 (re)review


Add support for URI templates

http://tools.ietf.org/html/draft-gregorio-uritemplate-08
Would like support for URI templates in requests sent through a client and in service descriptions.

Complete


Round out functionality for setting multiple settings at once

From Jonathan Swafford:

I see a getProjectSettings method to get all project settings. Would it be possible to add a method getSystemSettings to get all system settings?
I see a getSubSettings method that gets Project sub settings. Do you think support will be added to get System sub settings?
Also, is there a way to set sub settings (Project or System)?

I think these could all be easily implemented.

@swaffoja Does 'getProjectSettingsAsArray' work for the read part?
In some of our more 'complex' external modules, I'm currently using an ACE-mediated JSON editor to set the config.  It isn't as user friendly, but works nicely for development with a lot of flexibility:

Happy to share if it helps...

I think for the first item in my list the AbstractExternalModule class just needs a new method called getSystemSettings that calls the existing ExternalModules->getSystemSettingsAsArray(...). That should only take 2 minutes and I'll probably submit a PR this weekend.
Getting and setting project/system subsettings seems a bit more complex. The functionality exists in the REDCap UI, it just needs to be exposed to developers via the AbstractExternalModule class.
I'd be curious to see how you are using an ACE-mediated JSON editor to set the config. I haven't heard about ACE before. It sounds like you might have a page with an inline JSON editor that saves project or system settings on save?

Hi @swaffoja - I made a dummy EM that has the ace editor.
https://github.com/susom/redcap-em-ace-config-editor-template
It would be cook for mycap to have a new action-tag called @JSON or something that converts a textarea into an ACE editor with json syntax.  It would allow people to view the data stored in REDCap from mycap in json format in a much more intuitive format...  I'm happy to take a shot at it too...

@123andy I tried it out. Looks sweet!
I think an ACE editor for a JSON textarea would be beneficial to many different projects, not just MyCap. I think you could make a separate EM called "Extended Action Tags" (or whatever) that implements your editor. People could contribute additional action tags to the repo. MyCap would recommend that Extended Action Tags is installed. I'd make sure all the JSON fields that MyCap creates utilize your @json (or @eat-json?) action tag.

Hey guys, I'm not sure if this helps, but I'm sharing below a code that I've used in a few modules already, which is pretty useful for me as a developer. It basically displays all the configuration hierarchically - so the sub-settings are nested to their parents.
ExternalModules.php of a given module:
    /**
     * Formats settings into a hierarchical key-value pair array.
     *
     * @param int $project_id
     *   Enter a project ID to get project settings.
     *   Leave blank to get system settings.
     *
     * @return array $formmated
     *   The formatted settings.
     */
    function getFormattedSettings($project_id = null) {
        $config = $this->getConfig();

        if ($project_id) {
            $type = 'project';
            $settings = ExternalModules::getProjectSettingsAsArray($this->PREFIX, $project_id);
        }
        else {
            $type = 'system';
            $settings = ExternalModules::getSystemSettingsAsArray($this->PREFIX);
        }

        $formatted = array();
        foreach ($config[$type . '-settings'] as $field) {
            $key = $field['key'];

            if ($field['type'] == 'sub_settings') {
                // Handling sub settings.
                $formatted[$key] = array();

                if ($field['repeatable']) {
                    // Handling repeating sub settings.
                    foreach (array_keys($settings[$key]['value']) as $delta) {
                        foreach ($field['sub_settings'] as $sub_setting) {
                            $sub_key = $sub_setting['key'];
                            $formatted[$key][$delta][$sub_key] = $settings[$sub_key]['value'][$delta];
                        }
                    }
                }
                else {
                    foreach ($field['sub_settings'] as $sub_setting) {
                        $sub_key = $sub_setting['key'];
                        $formatted[$key][$sub_key] = reset($settings[$sub_key]['value']);
                    }
                }
            }
            else {
                $formatted[$key] = $settings[$key]['value'];
            }
        }

        return $formatted;
    }

Return example:
Array
(
    [helper_text] => Helper text
    [email_contents] => Array
        (
            [recipient] =><EMAIL_ADDRESS>            [subject] => Test
            [body] => This is a test
        )
    [contacts] => Array
        (
            [0] => Array
                (
                    [firstname] => Joe
                    [lastname] => Doe
                )
            [1] => Array
                (
                    [firstname] => Janie
                    [lastname] => Doe
                )
        )
)

If you guys think that's useful, I'll adapt it to the core, and create a pull request.

Heh, Travis (another developer on my team) just made something very similar to this.  We were thinking of replacing the getSubSettings() implementation with it.

This feature seems important enough that it should probably be the default behavior of getSubSettings().  Off the top of your head, do you happen to know if it would be backward compatible to replace the getSubSettings() implementation with this one?

@mmcev106 I think that won't be a problem.

I still haven't found the time to assess the risks involved in changing this implementation out, and I'm not sure if/when that will happen...  I did recently write a simple unit test for getSubSettings() in order to fix another issue.  I would need more time to make sure it covers all the cases involved in this change though.

Any thoughts on setSubSettings or how that should work?  I'm in a use case where I want to be able to do this...
Alternates would be  'addSubSetting', 'removeSubSetting' or 'updateSubSetting'
What happens if you have a key in your data that isn't defined in your config.json - should that issue a failure?

I could see setSubSettings() working very similarly to getSubSettings() when it comes to looping through the config.  Currently getSubSettings() is dependent on settings being in config.json for detecting the appropriate array structure, but we could potentially pass in a PHP defined configuration structure as an argument.
Now that I think it through, I think supporting setSubSettings() for settings not in config.json is a bad idea.  I think it might be more appropriate just to store a PHP array (which is currently supported, and serialized to JSON under the hood).


SPARQL: Item label does not seem to follow configured language, "en" hard-coded

https://github.com/commons-app/apps-android-commons/blob/main/app/src/main/resources/queries/query_for_item.rq#L21
Problem: "en" is hard-coded for item label
Replacing with "${LANG}" might be enough to fix the issue.

Could I have a go at this ?

After replacing, please test with various languages and post screenshots showing Nearby with a famous Wikidata item selected, for instance https://www.wikidata.org/wiki/Q243
Thanks!

I have been able to implement the required changes. But I would like to clarify the expected behaviour for testing.  My understanding of the test is  to select a famous Wikidata item in the nearby page and ensure that the tags with the native language is given preference. Is this correct ?

Couple more tests were done with the Bust of Gustave Eiffel link, The langages tested were, French, Armenian, Arabic & Turkish. Before each screenshot was taken, language preference was updated and then the item was located in Nearby.
P.S: I tried real hard to locate and retrieve info about Eiffel Tower however, despite mocking the exact co-ordinates of its location, I was unable to locate / retrieve its details.

Looks great!
Would you mind sending a pull request?


2571 auto aggregate

Pull Request Checklist:

[x] Positive Test Case Written by Dev

[x] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

Navigate to create a new resource, add a title and drag/drop a folder with a file in it that can be set as an aggregation (e.g. file.tif).  Click Create Resource.  When the new resource page opens, ensure the folder was created in the resource with the file inside the folder.  Also ensure an aggregation was set for that file or folder (if it was a .tif file, the aggregation would be set at the folder level)

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.

@pkdash I've addressed all of your comments and merged in your latest commits for 2502.

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.

Build finished.


Update to Version 3.3.3

Are you planing to update MJML to the latest version?

Hi @Skinwalker
We're using ^ char in package json so it should pickup the latest version available yet. You can try to reinstall it to update dependencies

Tried that too - but all i got is:
Couldn't find any versions for "gulp-mjml" that matches "^"
? Please choose a version of "gulp-mjml" from this list: (Use arrow keys)
❯ 2.2.0
2.1.1
2.0.0
1.0.1
1.0.2
1.0.0
In the npm repo the latest version is 2.2.0

You don't need to type^, in the package json mjml is listed as a dependency with ^ so it will pull the latest version available. Just remove all gulp-mjml versions and reinstall the latest

I removed gulp-mjml ...
What do i add now?
gulp-mjml or mjml?

Just install gulp-mjml back

alright - that worked - thank you!


updated ptloma.edu to pointloma.edu

Official URL: http://www.pointloma.edu/
wikipedia entry to document the name change: https://en.wikipedia.org/wiki/Point_Loma_Nazarene_University
MICS faculty page showing pointloma.edu email domain: http://www.pointloma.edu/experience/academics/schools-departments/department-mathematical-information-computer-sciences/contact-us
Computer Science degree program: http://www.pointloma.edu/experience/academics/schools-departments/department-mathematical-information-computer-sciences/mics-programs/computer-science-major
and a pdf
http://www.pointloma.edu/sites/default/files/filemanager/Mathematical_Information_Computer_Sciences/Major_Requirements/Bachelor_of_Science_in_Computer_Science-Software_Engineering_2016-2017_N.pdf

@maplebar Thank you very much for the update and the details! Pull request merged.


Add description for doc pages.

This PR adds the required description parameter for the docs pages.

Cool! Thanks!

Just pushed the commit to my branch


fxas21002c move to PX4Gyroscope and cleanup

@PX4/testflights could you quick this is a quick flightest on an NXPhlite (nxp_fmuk66-v3)?

Tested on NXP FMUK66 v3
Flight Card 1
Modes Tested:

Position Mode: Good.
Altitude Mode: Good.
Stabilized Mode: Good.

Procedure:
Arm in position mode, check take off behavior (Vehicle should stay on the ground as long as the throttle stick is at the middle or below, and start to ascend smoothly as long as throttle stick as above the middle.), the land and disarm.
Same procedure was repeated with Altitude and Stabilized mode.
Notes:
No issues were noted, good flight in general.
Flight Card 2
Modes Tested

Mission Plan Mode (Automated): Good.
RTL (Return To Land): Good.

Procedure
Arm and Take off in mission plan mode then make sure that vehicle follows all waypoints as shown in QGC, once completed all waypoint see landing behavior.
Note:
No issues were noted, good flight in general.
Flight Card 3
Modes Tested

Position Mode: Good.
Mission Plan Mode (Automated): Good.
RTL (Return To Land): Good.

Procedure
Arm and Take off in position mode, after flying for approximately one minute, switched to mission plan mode then make sure that vehicle follows all waypoints as shown in QGC, once completed all waypoint see landing behavior.
Note:
No issues were noted, good flight in general.
Flight Card 4
Modes Tested

Position Mode: Good.
Failsafe RTL (Return To Land): Not returning after RC shutdown.

Note:
No issues were noted, good flight in general.
Logs:

Flight Card 1: https://review.px4.io/plot_app?log=e4acf049-9f83-4d0d-b092-19a74f299680
Flight Card 2: https://review.px4.io/plot_app?log=75112b12-75ca-4640-90f9-46127734e008
Flight Card 3: https://review.px4.io/plot_app?log=a13a92ff-dbf7-456b-a340-a0323d6571da
Flight Card 4: https://review.px4.io/plot_app?log=bec11055-a422-43c6-9a80-94d26daedf84
Full Flight: https://review.px4.io/plot_app?log=3f9dea93-f353-4ddb-8446-d4384dcfa3dd

Thanks @Junkim3DR


Need auto convert from customer's currency to THB

Prestashop allows you to have multiple currencies for the convenience of the customer, but Omise allows to pay in Thai baht only. The function of automatic conversion of customer's currency into Thai baht at the payment process is very necessary. Otherwise, payment does not pass. Previous versions of the module at least reported that currency is not allowed for this method of payment. Now Omise is simply not displayed.

We’re going to close this since there hasn’t been any activity on this issue for a while. If you think it's still relevant, please feel free to reopen it and we’ll take another look.


SyntaxError: invalid syntax for print(*args, **kwargs)

I've got an error as a traceback below:
Traceback (most recent call last):
  File "setup_rouge.py", line 7, in <module>
    from files2rouge import settings
  File "/home/cerdas/files2rouge/files2rouge/__init__.py", line 2, in <module>
    from files2rouge.files2rouge import main
  File "/home/cerdas/files2rouge/files2rouge/files2rouge.py", line 17, in <module>
    from files2rouge import utils
  File "/home/cerdas/files2rouge/files2rouge/utils.py", line 14
    print(*args, **kwargs)
          ^
SyntaxError: invalid syntax

the error occured when i'm trying to run this code:
def tee(saveto, *args, **kwargs):
    """Mimic the tee command, write on both stdout and file
    """
    print(*args, **kwargs)
    if saveto is not None:
        print(file=saveto, *args, **kwargs)

i've read and searching for similar problem, but i still can't solved the error. Thanks

add the line from __future__ import print_function at the very beginning fix the problem


Reply box hides a comment on the bottom

When replying, the box hides the bottom comment, which usually I want to reply.

Browser: Chrome
Operating system: Win10

Closing this issue, it's been solved when we changed how comment box appear.