Get hw feature

Implement get hospital wing feature
Current command word: get/hw

Codecov Report
Base: 72.42% // Head: 71.01% // Decreases project coverage by -1.41% :warning:

Coverage data is based on head (beabd27) compared to base (3ba9893).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #48      +/-   ##
============================================
- Coverage     72.42%   71.01%   -1.42%
  Complexity      409      409
============================================
  Files            71       74       +3
  Lines          1273     1297      +24
  Branches        132      135       +3
============================================
- Hits            922      921       -1
- Misses          314      338      +24
- Partials         37       38       +1

Impacted Files
Coverage Δ

...address/logic/commands/GetHospitalWingCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
88.88% <0.00%> (-11.12%)
:arrow_down:

...ess/logic/parser/GetHospitalWingCommandParser.java
0.00% <0.00%> (ø)

.../person/HospitalWingContainsKeywordsPredicate.java
0.00% <0.00%> (ø)

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

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


The json for management level deployment is incorrect

Unable to deploy using https://raw.githubusercontent.com/Azure/azure-docs-json-samples/master/management-level-deployment/azuredeploy.json
Parsing error exists

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

ID: 24ff2a94-83fa-c921-b156-c0e6056d1f7f
Version Independent ID: bec70adc-7931-e08c-3559-b345601b24ce
Content: Deploy resources to management group - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/deploy-to-management-group.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@kanika1894 Thanks for your comment. We will review the issue and get back to you shortly

Also, I tried correcting the syntax issue (by removing the extra comma) to do the deployment
But even then it shows an issue
So, probably some issue in the template which needs correction for successful deployment

@kanika1894 Thanks for your feedback. JSON has been updated. We will close the issue for now. If there are further questions , please revert and we will be glad to assist you.


The driver does not work

It prints
Session ... is locked by a concurrent client.

I'm wondering how anyone can use it.
See https://github.com/ClickHouse/ClickHouse/issues/54952

@alexey-milovidov IIRC, you mentioned it was an incorrect usage. Could you provide more details?

same issue here using the odbc driver from SQL Server (CLICKHOUSEODBCW.DLL <IP_ADDRESS>817 2022-09-09)
Code: 373. DB::Exception: Session clickhouse_odbc_6468917993997271918 is locked by a concurrent client. (SESSION_IS_LOCKED) (version <IP_ADDRESS> (official build))

the error is intermittent/random but seems related to server load

seems related to server load

@gfody could you elaborate please?

while testing we noticed that the longer it takes for the queries to execute the more likely we were to hit this exception
eg
select * from openrowset('MSDASQL',
'Driver={ClickHouse ODBC Driver (Unicode)};username=...;password=...;url=http://...',
'select .... limit 100000')

testing with different numbers for limit 100000 increasing it seems to increase the odds that we'll hit the "locked by concurrent client" exception, although it really is very random. we also tested various querystring parameters in url as elsewhere it was mentioned to try setting wait_end_of_query=1 and different values for buffer_size .. the rate of this exception seemed mostly dependent on the limit or execution duration.
if we try setting session_id to some random value then we always immediately get the exception 100% of the time, eg:
select * from openrowset('MSDASQL',
'Driver={ClickHouse ODBC Driver (Unicode)};username=...;password=...;url=http://...?session_id=123456789',
'....')

OLE DB provider "MSDASQL" for linked server "(null)" returned message "HTTP status code: 500
Received error:
Code: 373. DB::Exception: Session 123456789 is locked by a concurrent client. (SESSION_IS_LOCKED) (version <IP_ADDRESS> (official build))

@gfody  could you test this version? https://github.com/ClickHouse/clickhouse-odbc/releases/tag/v<IP_ADDRESS>41018

closed in favour of https://github.com/ClickHouse/clickhouse-odbc/issues/423

@gfody could you test this version? https://github.com/ClickHouse/clickhouse-odbc/releases/tag/v<IP_ADDRESS>41018

confirmed I haven't been able to repro the issue w/this version - thanks!


perf: Fixing double deserialization issue - 30% performance improvment

Description of changes
Closes #3309
Summarize the changes made by this PR.

Improvements & Bug fixes

Removes body parameters from API routes, adds OpenAPI extras to preserve API docs

Test plan
How are these changes tested?

[x] Tests pass locally with pytest for python, yarn test for js, cargo test for rust

Documentation Changes
N/A

#3310  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


More fixes for re-instating old EIGEN build methods

configure.ac is now the same as it was prior to 7bfe3f6528f61c7f813e1a06388ab2fa80b6034c.

The same fix has now been applied as part of #122.
Congratulations on ignoring my build fix pull request for three years 😛


Suggestion: Adding Unique Hash

Is there a way to add a column that gives a unique hash. Call it "DataHash" or "IntegrityHash".
Purpose: When copying over data to another SQL table I want to do a compare and not duplicate any records. And a clean way of doing that is by having a unique hash for every row.

A random twist here, can you use the timestamp as the unique? You can create a time series out of it. There will never be a duplicate on timestamp

It is guaranteed that every entry will have a unique time stamp ?

Yeah, this is based on buffer exit, so you won't have 2 entries of the same time and type and file. Potentially if you have to make sure that the row is unique, the constraint should be time and file. That ensures that you can have uniqueness across your dbs (diff devices)


Enabling Black formatter removes syntax highlighting

Hi, whenever I enable Black formatter (v2023.6.0), my Python syntax highlighting gets removed. The only other extensions I have enabled are Python v2023.20.0 and Pylance v2023.11.10. See video for demonstration.
https://github.com/microsoft/vscode-black-formatter/assets/24358775/56ccbb41-627e-4d51-98aa-f11ec0b2ad9a
Version: 1.84.2
Commit: 1a5daa3a0231a0fbba4f14db7ec463cf99d7768e
Date: 2023-11-09T10:50:47.800Z
Electron: 25.9.2
ElectronBuildId: 24603566
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.18.3-051803-generic snap

Moving this to Pylance. All we do is start another language server that handles formatting. This should not change the behavior of other LS.

This might be an issue due to LS-client vserion 8, will try updating to LS-client version 9.


Make it compatible with systemjs 0.19.18

@ludohenin @NathanWalker would you review and merge these changes?

Fixes https://github.com/mgechev/angular2-seed/issues/431.

Hi @mgechev do we need to set System.config 2x ?

Ups, nope. We just need to move the config above the injected libs.

I was actually unable to test this due to a typings issue. Once this PR https://github.com/mgechev/angular2-seed/pull/437 is merged, then we can likely test this change out. zone was renamed to zone.js on typings. And jasmine needed a commit update.

@mgechev Now that https://github.com/mgechev/angular2-seed/pull/437 is merged, you might rebase master here ;) I can verify after that!

I rebased master into my local checkout of this and all looked good! merging!


Smoke test failure: Could not load type 'Microsoft.Build.Logging.ProfilerLogger'

starting language C#, type console
    running new
The template "Console Application" was created successfully.

Processing post-creation actions...
Running 'dotnet restore' on /home/chris/dotnet_source-build/testing-smoke/C#_console/C#_console.csproj...
Unhandled Exception: System.TypeLoadException: Could not load type 'Microsoft.Build.Logging.ProfilerLogger' from assembly 'Microsoft.Build, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
   at Microsoft.Build.CommandLine.MSBuildApp.Execute(String[] commandLine)
   at Microsoft.Build.CommandLine.MSBuildApp.Main(String[] args)

Restore failed.
Post action failed.
Description: Restore NuGet packages required by this project.
Manual instructions: Run 'dotnet restore'

    new succeeded
    running restore

Unhandled Exception: System.TypeLoadException: Could not load type 'Microsoft.Build.Logging.ProfilerLogger' from assembly 'Microsoft.Build, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
   at Microsoft.Build.CommandLine.MSBuildApp.Execute(String[] commandLine)
   at Microsoft.Build.CommandLine.MSBuildApp.Main(String[] args)

Confirmed that reverting fe4ae4bd94f332a2970f00d4bf98f4f4c1f5829d. no longer repros the issue.  Tried reverting the output path changes here on a hunch with no luck.  Attempting to bisect the msbuild changes now.


Patch posts

chord.html hasn't been updated because of:
> install.packages('chorddiag')
Warning message:
package ‘chorddiag’ is not available for this version of R

A version of this package for your version of R might be available elsewhere,
see the ideas at
https://cran.r-project.org/doc/manuals/r-patched/R-admin.html#Installing-packages

EDIT: solved

same for choropleth.html:
> install.packages('maptools')
--- Please select a CRAN mirror for use in this session ---
Warning message:
package ‘maptools’ is not available for this version of R

A version of this package for your version of R might be available elsewhere,
see the ideas at
https://cran.r-project.org/doc/manuals/r-patched/R-admin.html#Installing-packages

same for circularpacking.html:
> install.packages('circlepackeR')
Warning message:
package ‘circlepackeR’ is not available for this version of R

A version of this package for your version of R might be available elsewhere,
see the ideas at
https://cran.r-project.org/doc/manuals/r-patched/R-admin.html#Installing-packages

this post is broken:
https://www.data-to-viz.com/graph/hexbinmap.html

I don't update too many post at a time just in case


[Feature request] ability to adjust alarm volume

It's a gentle alarm sound (on Windows), but regardless I (and my colleagues) would like the ability to adjust its volume.
**thanks for your work. It's my favourite pomodoro timer I've come across.

Thanks! Volume control is now available with v0.6.0.


Simplified MockSupportTest fixture in preparation for splitting

Now it'll be easier to split. Needs to be shrinked more still.

Looks good. If you want to merge it, I'll work from there.

Now it'll be easier to split. Needs to be shrinked more still.

Take it to a shrink, perhaps ~.~ ?


Root detection not working as expected, delayed hook detection

Describe the bug
I have integrated the freeRasp SDK in my flutter app. I am mainly interested in the root detection and hook detection capabilities. On testing I have observed that I am able to run on my rooted android emulator (no callback), and when I try to dump the memory there is a delay in getting a onHooks callback.
To Reproduce

Here's how I've integrated the SDK

class MyApp extends StatefulWidget {
  const MyApp(this.environment, {super.key});
  final Environment environment;

  @override
  State<StatefulWidget> createState() => _MyAppState();
}

class _MyAppState extends State<MyApp> {

  final _callback = ThreatCallback(
    onHooks: () => _exitApp(),
    onPrivilegedAccess: () => _exitApp(),
  );

  @override
  void initState() {
    super.initState();
    _initializeRasp(widget.environment);
  }

 @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: 'Demo app',
      home: SplashPage(),
    );
  }

  Future<void> _initializeRasp(Environment environment) async {
    final config = TalsecConfig(
      androidConfig: AndroidConfig(
        packageName: AppConfig.getPackageName,
        signingCertHashes: [
          hashConverter.fromSha256toBase64(AppConfig.androidSha256)
        ],
      ),
      iosConfig: IOSConfig(
        bundleIds: [AppConfig.getPackageName],
        teamId: AppConfig.iosTeamId,
      ),
      watcherMail<EMAIL_ADDRESS>// TODO: Update this email address
      isProd: true,
    );

    Talsec.instance.attachListener(_callback);

    await Talsec.instance.start(config);
  }

  static void _exitApp() async {
    await SystemChannels.platform.invokeMethod<void>('SystemNavigator.pop');
  }
}

For rooting the emulator, I have used adb root command

> adb root
adbd is already running as root

For heap dump, I am using fridadump and have followed the steps mentioned here. When I pull the dump, it takes a few seconds for the onHooks callback to be called.

Expected behavior

Application should exit as soon as its launched on a rooted device
onHooks should be called immediately (or during execution of) when python3 fridump.py -U -s MyApp is executed

Please complete the following information:

Device: Pixel 6 Pro (emulator, without playstore)
OS version: API 31
Version of freeRASP: 6.6.0

Hello @shahmharsh,
thank you for reporting the issue, we will look at it.
Kind regards,
Talsec team

I am also facing this issue in my production application and because of that it has affected my new deployment as well. Is there a solution for this issue? I have tried adding a delay after starting the talsec engine but still issue is coming and hooking/root detection can be bypassed using Magisk/Frida etc. If a solution can be provided as soon as possible it will be really helpful.

Hello @shahmharsh,
Thank you for reporting this issue.
Root Issue:
After further investigation, I've been able to reproduce the issue with adb root rooting. We will work on a fix for this. Please note that other forms of root detection are not impacted by this bug and will continue to function correctly.
Hooking Issue:
I've also reproduced the hooking issue. The challenge with this detection is conceptual. freeRASP relies on periodic checks rather than proactive protection due to the nature of mobile platforms. Mobile platforms have limited computing power and energy resources mean that proactive detection could make app slower and/or significantly drain the battery.
freeRASP performs security checks at startup and then periodically after a set interval. This creates a time window that could be exploited for an attack. If you start the app directly with frida, it will be detected. However, if you run the app and then attach frida within this time window, frida won't be detected until the next periodic check.
BusinessRASP+ offers the option to adjust the length of this period, allowing for more or less frequent checks.
We will add this issue to our backlog and work on improvements for the future.
Best regards,
Jaroslav from Talsec

Closing this issue and dividing it into bug issue and enhancement issue.


Hyperlink for RSA

There is a hyperlink for third party integrations with Duo Security and Trusona but there is no hyperlink for RSA.
Can you please add a link to the Azure AD integration for RSA using custom controls?

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

ID: e03d33c6-c60b-153c-bf76-a127be502170
Version Independent ID: 7bbf1dc5-bd91-d03a-7939-439bfbee85c9
Content: Access controls in Azure Active Directory conditional access
Content Source: articles/active-directory/active-directory-conditional-access-controls.md
Service: active-directory
Product: unspecified
GitHub Login: @MarkusVi
Microsoft Alias: markvi

@ejazr83 Thanks for the feedback. We are actively investigating and will get back to you soon.

@ejazr83 Thanks for reporting! I have assigned the issue to the author to investigate and update as appropriate.

Hi @ejazr83, thanks for your contribution, I have created a backlog item to track this.
I will update the article with the link as soon as I have one.
#please-close

Thanks @MarkusVi , Do you have an approximate time when this will be done? weeks or months?

@ejazr83 We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


RAM consumption is 3 - 4 times the cached data

Hi team,
When I am trying to render a data of size 300mb using volume viewport the RAM consumption in the activity monitor seems to be 1.2gb which is 3 -4 times.
Troubleshooting on volumeBasic Example:

Commented out the code from createAndCacheVolume function call. Found that the RAM consumption was minimal as volume is not yet loaded in cache.

Uncomment createAndCacheVolume function and kept below code commented which is having load and setVolumes function. Found that the RAM consumption was as per the volume size as the volume is now loaded in the cache.

Now uncommented load function and found that the RAM consumption incremented by data size

Now uncommented setVolumes function and found that the RAM consumption incremented by data size

Steps to Reproduce

Use a data-set of size 300mb.
Use the volumeBasic example and run on safari/chrome.
Monitor the RAM usage in activity monitor.

The current behavior
RAM consumption seems to be very high than the actual data when monitored.
The expected behavior
RAM consumption should be as per the data size.
OS
Windows, Mac OS
Node version
v16.15.0
Browser
Chrome: 117.0.5938.92, Safari: 17.0

Yes, I have also encountered this issue. The JavaScript heap occupies 500m, but the Chrome task manager shows that it occupies over two gigabytes of memory. @sedghi

size

Thank you very much for your answer, solved my memory problems some questions, I have another question, MPR how to support more than one thousand, more than two thousand images in series, when I try to view the MPR of this order of magnitude sequence, it will prompt that the array cannot be allocated, this need to view a large number of pictures  MPR scene how to deal with?

It is not a matter of number of slices, it is the size of eventual monitor
If you can't fit it regularly you can have two options in the configuration of init
preferSizeOverAccuracy: as the name suggest it will make the size smaller using halfFloat (11 bit, not 16 bit) but as a matter of reducing accuracy
Integers between 0 and 2048 can be exactly represented (and also between −2048 and 0)
Integers between 2048 and 4096 round to a multiple of 2 (even number)
Integers between 4096 and 8192 round to a multiple of 4
Integers between 8192 and 16384 round to a multiple of 8
Integers between 16384 and 32768 round to a multiple of 16
Integers between 32768 and 65519 round to a multiple of 32
useNorm16Texture: reduces the size (in half) with full accuracy, but has some issues in some devices

It is not a matter of number of slices, it is the size of eventual monitor
If you can't fit it regularly you can have two options in the configuration of init
preferSizeOverAccuracy: as the name suggest it will make the size smaller using halfFloat (11 bit, not 16 bit) but as a matter of reducing accuracy
Integers between 0 and 2048 can be exactly represented (and also between −2048 and 0)
Integers between 2048 and 4096 round to a multiple of 2 (even number)
Integers between 4096 and 8192 round to a multiple of 4
Integers between 8192 and 16384 round to a multiple of 8
Integers between 16384 and 32768 round to a multiple of 16
Integers between 32768 and 65519 round to a multiple of 32
useNorm16Texture: reduces the size (in half) with full accuracy, but has some issues in some devices

Thank you very much for the two solutions you provided, which gave me a better understanding of this problem. I will try to use these two methods later and see the results. That is to say, the high usage of memory space is due to a bug in using 16 bit groups to store data. Therefore, using 32-bit arrays to store 16 bit arrays results in more memory usage, so this is not a problem with the number of sequences. If my understanding is correct

Hey @sedghi
When tried with unit8 data we found the same RAM problem.
On further troubleshooting with you in the OHIF weekly meeting we found that the web workers are causing the extra RAM consumption.
Can you please let us know what can be a solid solution to resolve this issue?


Update cats-core to 2.6.0

Updates org.typelevel:cats-core from 2.4.2 to 2.6.0.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (2.4.2).
You might want to review and update them manually.
project/plugins.sbt

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "cats-core" } ]

labels: library-update, semver-minor, old-version-remains

Superseded by #133.


Fixed Prototype Pollution Bug by ignoring dangerous keys

:bar_chart: Metadata *
join-assign is vulnerable to Prototype Pollution.
Bounty URL: https://www.huntr.dev/bounties/1-npm-join-assign
:gear: Description *
Prototype Pollution refers to the ability to inject properties into existing JavaScript language construct prototypes, such as objects.
JavaScript allows all Object attributes to be altered, including their magical attributes such as __proto__, constructor and prototype. An attacker manipulates these attributes to overwrite, or pollute, a JavaScript application object prototype of the base object by injecting other values. Properties on the Object.prototype are then inherited by all the JavaScript objects through the prototype chain.
:computer: Technical Description *
Fix implemented by not allowing to modify object prototype.
:bug: Proof of Concept (PoC) *

Create the following PoC file:

// poc.js
const { assign } = require('./src')

console.log('Before: ' + {}.polluted)
assign({}, JSON.parse('{"__proto__": {"polluted": true}}'))
console.log('After: ' + {}.polluted)

Execute the following commands in terminal:

npm i join-assign # Install affected module
node poc.js #  Run the PoC

Check the Output:

Before : undefined
After : true

:fire: Proof of Fix (PoF) *

+1 User Acceptance Testing (UAT)

I've executed unit tests.
After fix the functionality is unaffected.

👋 Hello, @diogoeichert - @arjunshibu has opened a PR to us with a fix for a potential vulnerability in your repository. To view the vulnerability, please refer to the bounty URL in the first comment, above.
Ultimately, you get to decide if the fix is 👍 or 👎. If you are happy with the fix, please write a new comment (@huntr-helper - LGTM) and we will open a PR to your repository with the fix. All remaining PRs for this vulnerability will be automatically closed.
If you have any questions or need support, come and join us on our community Discord!
@diogoeichert & @arjunshibu - thank you for your efforts in securing the world’s open source code! 🎉

Hi! Thank you very much for this. While I truly appreciate the collaboration, I must say I don't agree with the solution, specially because I don't see this as a problem this package should try to solve. If the prototype has been polluted already by the time join-assign performs an assign or clone operation, it means the problem should have been prevented at an earlier point, most likely avoiding random code from other dependencies to be executed.
That being said, this change would also break some important functionality currently available, like the creation of multi-inheritance instances, e.g.
		function A() {
			this.a = 1;
		}

		function B() {}

		B.prototype.b = function () {
			return this.a;
		};

		const a = new A();
		const b = new B();
		join.assign(a, b);
		a.b(); // should be able to call this method and return 1

I have extended the unit tests to cover this case specifically. To run the tests, you can rebase your branch from the main branch and run npm test. Thanks again.


Standard Transformer UI

Does this PR introduce a user-facing change?:

- Update model deployment page to include the settings of standard transformer deployment
- Update model details page to include standard transformer details

Screenshots

Model deployment page

Model version details page

Question:

How user configure cpu / memory request from UI?
I think in sdk user is able to configure those value, will it be override if user redeploy using UI?

Question:

How can user configure cpu / memory request from UI?

I thought that we were going to hide the complexity of configuring standard transformer resources from the UI and just use the environment target's default resources configuration. If you think user need it in the UI, I think we can add an accordion component that could display and hide the resources config.

I think in sdk user is able to configure those value, will it be override if user redeploy using UI?

Yes. Both of them can override each other config.


Get this library featured in https://www.droidcon.com/content/

Apparently, this has already been listed there since Nov 23, 2021 😮 THIS IS AWESOME!!! ❤ ❤ ❤ ❤ ❤
https://www.droidcon.com/2021/11/23/android-contacts-reborn/
🔥 🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥


runtime error: unsigned integer overflow: 0 - 1 cannot be represented in type 'unsigned int' (crypto/ec/curve25519.c)

Similar to the previously reported overflows, built on a fresh Debian 9 x64 machine per the fuzzing instructions with the addition of -fsanitize=integer -fsanitize-address-use-after-scope -fsanitize=float-divide-by-zero -fsanitize=float-cast-overflow. This one only triggers with the client fuzzer and the client corpora.
UBSAN_OPTIONS=print_stacktrace=1 fuzz/client fuzz/corpora/client
INFO: Seed: 726001437
INFO: Loaded 1 modules (54706 guards): [0x2220ee0, 0x22565a8),
Loading corpus dir: fuzz/corpora/client
Loaded 1024/3127 files from fuzz/corpora/client
Loaded 2048/3127 files from fuzz/corpora/client
INFO: -max_len is not provided; libFuzzer will not generate inputs larger than 16645 bytes
#0      READ units: 3062
crypto/ec/curve25519.c:1246:5: runtime error: unsigned integer overflow: 0 - 1 cannot be represented in type 'unsigned int'
    #0 0x15144ca in equal /root/openssl/crypto/ec/curve25519.c:1246:5
    #1 0x150ff3a in table_select /root/openssl/crypto/ec/curve25519.c:3385:35
    #2 0x14dea07 in ge_scalarmult_base /root/openssl/crypto/ec/curve25519.c:3432:5
    #3 0x14ff837 in X25519_public_from_private /root/openssl/crypto/ec/curve25519.c:4707:3
    #4 0xc94cc9 in ecx_key_op /root/openssl/crypto/ec/ecx_meth.c:90:13
    #5 0xc91b8f in pkey_ecx_keygen /root/openssl/crypto/ec/ecx_meth.c:418:12
    #6 0xddba52 in EVP_PKEY_keygen /root/openssl/crypto/evp/pmeth_gn.c:107:11
    #7 0x5dda1c in ssl_generate_pkey_curve /root/openssl/ssl/s3_lib.c:4140:9
    #8 0x6f7fbe in add_key_share /root/openssl/ssl/statem/extensions_clnt.c:551:25
    #9 0x6f750d in tls_construct_ctos_key_share /root/openssl/ssl/statem/extensions_clnt.c:637:10
    #10 0x6d5448 in tls_construct_extensions /root/openssl/ssl/statem/extensions.c:747:15
    #11 0x76f9ce in tls_construct_client_hello /root/openssl/ssl/statem/statem_clnt.c:1147:10
    #12 0x7548b6 in write_state_machine /root/openssl/ssl/statem/statem.c:792:45
    #13 0x74cf15 in state_machine /root/openssl/ssl/statem/statem.c:401:21
    #14 0x749efd in ossl_statem_connect /root/openssl/ssl/statem/statem.c:229:12
    #15 0x651cd5 in SSL_do_handshake /root/openssl/ssl/ssl_lib.c:3327:19
    #16 0x4edb2e in FuzzerTestOneInput /root/openssl/fuzz/client.c:91:9
    #17 0x4eddae in LLVMFuzzerTestOneInput /root/openssl/fuzz/driver.c:25:12
    #18 0x4f968a in fuzzer::Fuzzer::ExecuteCallback(unsigned char const*, unsigned long) /root/test/Fuzzer/FuzzerLoop.cpp:460:13
    #19 0x4f98f1 in fuzzer::Fuzzer::RunOne(unsigned char const*, unsigned long) /root/test/Fuzzer/FuzzerLoop.cpp:399:3
    #20 0x4f94e1 in fuzzer::Fuzzer::RunOne(std::vector<unsigned char, std::allocator<unsigned char> > const&) /root/test/Fuzzer/./FuzzerInternal.h:98:41
    #21 0x4f94e1 in fuzzer::Fuzzer::ShuffleAndMinimize(std::vector<std::vector<unsigned char, std::allocator<unsigned char> >, std::allocator<std::vector<unsigned char, std::allocator<unsigned char> > > >*) /root/test/Fuzzer/FuzzerLoop.cpp:380
    #22 0x4f24b1 in fuzzer::FuzzerDriver(int*, char***, int (*)(unsigned char const*, unsigned long)) /root/test/Fuzzer/FuzzerDriver.cpp:742:6
    #23 0x4eddf0 in main /root/test/Fuzzer/FuzzerMain.cpp:20:10
    #24 0x7fa6758da2b0 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x202b0)
    #25 0x420259 in _start (/root/openssl/fuzz/client+0x420259)

SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior crypto/ec/curve25519.c:1246:5 in
MS: 0 ; base unit:<PHONE_NUMBER>000000000000000000000000000000
0x50,0x4f,0x53,0x54,0x20,
POST
artifact_prefix='./'; Test unit written to ./crash-1d92f5bfe0270ab938b478dcc770f6f9843cc92a
Base64: UE9TVCA=

Like the other bugs this issue is only visible if you compile OpenSSL with UBSan and -fsanitize=integer.

Does this patch help?
diff --git a/crypto/ec/curve25519.c b/crypto/ec/curve25519.c
index e467eef7d3..dd588c3907 100644
--- a/crypto/ec/curve25519.c
+++ b/crypto/ec/curve25519.c
@@ -1242,10 +1242,7 @@ static uint8_t equal(signed char b, signed char c) {
   uint8_t ub = b;
   uint8_t uc = c;
   uint8_t x = ub ^ uc; /* 0: yes; 1..255: no */
-  uint32_t y = x;      /* 0: yes; 1..255: no */
-  y -= 1;              /*<PHONE_NUMBER>: yes; 0..254: no */
-  y >>= 31;            /* 1: yes; 0: no */
-  return y;
+  return x == 0;
 }

 static void cmov(ge_precomp *t, const ge_precomp *u, uint8_t b) {

That's funny, because unsigned integer underflow is not supposed to be undefined behaviour.
Is this GCC and which version do you use?

clang version 5.0.0 (trunk 305735)
'''
git clone https://chromium.googlesource.com/chromium/src/tools/clang
clang/scripts/update.py
'''

While signed integer overflow is clearly undefined behaviour, unsigned integer overflow is not.
If there a clang option to get warnings for signed but not for unsigned integer overflow?

yeah, don't make this change; i worry about the constant-time lossage.

me too.


Fix spectral range calculation for GradGenerator

The routine to convert GradGenerator to a dense matrix was missing, causing specrange with method=:diag to fail.

Codecov Report

Merging #26 (4fc14a5) into master (bdb4d83) will increase coverage by 0.54%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
+ Coverage   88.82%   89.37%   +0.54%
==========================================
  Files          14       14
  Lines         868      875       +7
==========================================
+ Hits          771      782      +11
+ Misses         97       93       -4

Impacted Files
Coverage Δ

src/gradgen.jl
94.02% <100.00%> (+3.47%)
:arrow_up:

Continue to review full report at Codecov.

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


Conflicting namespace between SoundWave and Animation (A_)

Sound Wave and Animation have the same prefix of A_. It seems that the Sound Wave's "A_" was supposed to stand for Audio, but I think since the suffix "SW_" is not used anywhere, it's better to use "SW_" than "A_".
I would like to know what's the reason behind this convention?

This is so that Cues + Waves who up together side by side, which is admittedly an old practice because it was incredibly hard to find stuff back in the days of the old UE3 content browser. Most companies from this time period still kind of adopted this practice, so there isn't any modern reason if you do not want to preserve the side-by-side ness of this convention.
This is one of those things that a modern fork might want to consider, but I desire to keep here for legacy capability and I'm simply used to have these files listed side by side.

Where does that A_ prefix for sound cues come from? I'm failing to find any other sources that would use it except this style guide. Looking at random audio assets from UE marketplace, I only see _Cue suffix, without any prefixes.


Circular Imports in SQLAlchemy Db creation and routes.

Routes are now filled on init.
SQLAlchemy now creates the file correctly.


Hide private methods in API reference (MkDocs)

In https://deel-ai.github.io/oodeel/api/methods/ for instance, the API is a bit messy and hard to read because of methods like _deviation or _stats which are useless for the user. Is it possible to filter the doc content we want to show?

I think private methods can be removed from generated API by updating filters parameter in docs/api/methods.md
By the way, the default Python handler in mkdocstrings was recently changed and new options are available for a better rendering of the API documentation. We might work on it.


Client side control of SSH.

Can we please make SSH support something that can be disabled from the client side?
I quite like the way tailscale handles it where if you want to use their SSH implementation you have to connect the client with an argument passed so it tells the control plane SSH is available before you can switch on tailscale SSH.

This is hugely important, otherwise all my Tailscale boxes basically have root on each other :/

This is a big issue for me.
I don't want the cloud hosted (either managed or on a vm) management for the overlay network to be able to bypass access control for ssh.
I already have my own ssh management in place

hello, any news here?

Any progress on this topic? Maybe @braginini (as you posted on #852)?
As highlighted in https://github.com/netbirdio/netbird/issues/683#issuecomment-1806477634, this SSH feature (among others) is really a no-go as the netbird server/network operator is currently too powerful. ACLs etc. are not a mitigation for this threat.

+1

+1


Do not require strobealign to be in PATH when running tests

This just removes a minor inconvenience.

approved


Feature request: Focal point

I've worked in a closed-source version of a software that does pretty much the same as this. One very useful feature that it had was the ability to use an arbitrary point to crop an image to. In flyimg this is sort of achieved with the gravity parameter but it is not very precise. In the one I worked with you could specify an x and y focal point float that ranged from 0 to 1.
In the context of my use case, this served two main purposes:

The user would upload an image and choose the focal point. No matter how we cropped the image, the "important" part (aka focal point) would always be in the center.
We ran some images through face detection and set the focal point automatically to the center of the face. Then, cropped profile pictures would always look great even if a user uploaded an image with a weird aspect ratio.

Yes! We are very aware of the limitations of gravity. We are reworking the whole cropping+resizing module to have more granular control and easier extensibility. I was precisely thinking about setting the focal point with 0 to 1 ranges for X and Y axis. I didn't have a name for it though. So expect this feature soon. Not next release, but probably the one after that.
Face detection integration will be much easier after we refactor the geometry module. And since we have face detection we can get not only profiles pictures generated, we can get group pictures "auto" cropped too, using the concentration of cages as gravity area. Although we don't have a timeline for that yet.

Hello everyone, there's some update about this issue? Any timing info about 1.2 versione and it's milestone? Thank you very much

This is a very interesting Feature request, sorry it took us a while before reply to you.
To summarize it'll be great if we can add a smart-crop option which will focus on the most important part of an image for cropping.
For the face-detection part we can call it a smart-face-crop option.
@poisa Thanks again for this feature, we gonna work on it soon.
@bedoz Soon :)

I've had a play around with this library and the feature-set that you offer is absolutely amazing considering it is open source. Unfortunately without being able to do a crop with the face as the focal point it can't be considered in the same league as Cloudinary 😞 Are there still plans to add this feature in?

Hi @joshbrw,
Thank you for your interest in our project and your kind words about it.
Unfortunately, we didn't find a proper time to get around this issue and implement it.
I'll try my best to implement it asap.
Thank you again for your support!

+1 It will be awesome

Yeah i want to renew my request to this feature...

Thank you for your message, I'm sorry regarding this delay.
This feature request will be a high priority and hopefully, we'll release it soon.

I have used the https://imagekit.io/ they have a good "Smart Crop"

We're happy to announce the release of the smart cropping feature: https://github.com/flyimg/flyimg#smart-crop
It would be great if you could test it out and provide feedback about it!
@fhferreira , @bedoz , @joshbrw
Thank you All for your support and help!

We're happy to announce the release of the smart cropping feature: https://github.com/flyimg/flyimg#smart-crop
It would be great if you could test it out and provide feedback about it!
@fhferreira , @bedoz , @joshbrw
Thank you All for your support and help!


[BUG] Invalid reset packet (incorrect ack sequence number) when connection backlog is full

Describe the bug
If you set connection backlog to a low limit, and connection count starts to reach the limit so new connections cannot be handled, it is expected that FreeRTOS+TCP will send RST packet upon receiving a SYN request it cannot handle (see FreeRTOS_TCP_State_Handling.c#L963).
However, the RST packets sent here have a wrong ACK value, as you can see in the following screenshot. <IP_ADDRESS> (client, Linux) is trying to connect to <IP_ADDRESS> (server running FreeRTOS+TCP). The client sent a SYN packet with sequence number 615815458, and server replied the same ACK number.

This behavior will make TCP stack of the client ignore this RST packet, because as per RFC793, Page 69, when a segment arrives for a connection in a state other than LISTEN, it is at first checked whether the sequence number falls into the receiver window, and if it doesn't, it's dropped (ref). When client sent SYN with seq=n, it is expected that the RST packet comes back with ack=n+1 (you can check this behavior on any normal Linux PC). In the screenshot below, <IP_ADDRESS> is a normal Linux server:

Target

Development board: rp2040+w5500
Instruction Set Architecture: ARM Cortex m0+
IDE and version: CLion
Toolchain and version: arm-none-eabi-gcc (GNU Arm Embedded Toolchain 10.3-2021.10) 10.3.1

Host

Host OS: Linux
Version: Debian 11

To Reproduce

Create a simple TCP server with FreeRTOS+TCP that listens but never accepts connections. Set the backlog to 3.
Connect to this server with TCP clients on your PC. Make 4 connections simultaneously.

Expected behavior
An RST packet with correct ACK number is sent from FreeRTOS+TCP. The fourth TCP client reports Connection refused.
** Actual behaviour **
The RST packet with wrong ACK number is ignored, and the fourth TCP client keeps waiting.
Screenshots
See above.

Hello @t123yh , I was aware of the rule in RFC793 ( page 69 ), but I had never realised that the when replying to a SYN, the seq must be incremented with 1 also. I will prepare and test a PR for this.
Thank you very much for taking the effort to report this.

Hello @t123yh, I just made the required change in PR #724.
I tested it on an embedded device:

Now the DUT responds with Seq=1, in stead of Seq=0.
Thanks

Thank you for bringing this bug to our attention @t123yh and thank you for fixing the said bug @htibosch.
Since the PR is merged, I will go ahead and close the issue now.


Mouse wheel scrolling

It would be nice to support scrolling using the mouse wheel in the main view, similar to less's --mouse flag.

I am not planning to support mouse input in any form in igrep. I think that would unnecessarily complicate current behavior.
Sorry for the very late response.


Refactor Package Manifest Parsing into a Reader Interface

Currently different operations are performed to read package manifests from:

Directory
Lockfiles
JSON Dump

In future, we may need to be able to read from SBOM (SPDX, CycloneDX). To be able to ensure separation of concerns, we should

Define a reader interface for reading package manifests
Have implementation for different sources of package manifest

** Work in Progress **
Overview
vet has multiple source of input for packages to be analysed. It can

Scan a directory to read supported package manifests (e.g. pom.xml)
Parse a list of package manifests provided as input (using --lockfile option)
Scan a directory for JSON files (generated with --json-dump-dir option)

It also supports the undocumented feature of reading declared dependencies from a python binary wheel file. In future, it will support binary artifacts such as jar, gem, docker image etc. as source of input.
The current implementation for this package manifest reading capability is strongly coupled with the scanner package and is treated like utility functions. This causes the unnecessary need for scanner package to change whenever we add a new package manifest source. In reality, the package reading capability is large enough to be an independent concern to be used by vet scanner as well as by other tools needing such capability.
Requirements

Define an interface to standardize the contract for a package manifest reader
Refactor existing manifest reader functions as implementation of the standard interface
Apply dependency inversion principle to decouple scanner from details of package manifest readers


Repeat executions use cached module

I'm calling generate_typescript_defs repeatedly within a Python script in response to file changes (via watchdog). The goal is to have any changes to my Pydantic models automatically propagated to their TypeScript counterparts, which will then cause a "watch" build (e.g. Angular's ng serve) to highlight all TypeScript references that require update.
Watchdog is correctly executing generate_typescript_defs in response to each Pydantic model change, but pydantic2ts appears to use a cache of the module acquired during its first run. No matter how I change the Pydantic model the generated TypeScript is always the same, until I kill the process and restart. Is there any way I can force pydantic2ts to reload the module each time generate_typescript_defs is called, without starting a new Python process each time?
My module references are in the dot-separated format, not file paths, and looking at the source for import_module I had hoped to simply call importlib.reload(module) prior to each call to generate_typescript_defs, but this did not make a difference. Any suggestions would be very much appreciated.

Ahh this is a really cool use case! I like it. I'll write some tests to try and figure out if I can figure out a clean way to optionally reload modules each time
That said, one solution that might work now would be to have generate_typescript_defs use a fresh interpreter each time.  I know you said you'd like to avoid starting a new python process each time, but I'm assuming that was in reference to the watchdog process? (correct me if I'm wrong)
If you'd like to try that, I think calling this instead of generate_typescript_defs may work:
from multiprocessing import Process

def run_pydantic2ts(module, output):
    p = Process(target=generate_typescript_defs, args=(module, output))
    p.start()
    p.join()


Build cobertura report only on master for successful builds

Building the report takes time, and when someone submits a pull request,
we should not show it as failure if the build timeouts for the report.
We only need the report when we build master, and only if the build is
successful.

LGTM +1


Repo sync for protected CLA branch

The pull request is created from master637713852882224197 to master to fix git push error for protected CLA branch

@opbld15 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Fixing multiple video renderers on the same HTML page for Flutter Web

This commit fixes the case where multiple RTCVideoRenderer objects present on the same Flutter Web HTML page and where another object was using ui.platformViewRegistry.registerViewFactory() method.
This PR is supposed to fix issue #259 .

LGTM


Unhelpful error message when using a metadata plugin credential

When I make a credential object using grpc_metadata_credentials_create_from_plugin and then use it to make a channel, and then make a call on that channel, I get the status message "Failed to create security connector." I didn't know what that meant at first; it would help to have something there that indicates that I needed an SSL or similar credential.

This seems to be obsolete with the new auth plugin API.


Exit button of some apps (XNAT, Gaussian, etc.) doesn't work correctly in Mac OS

@KanishkaS ' comment:
@JSousa-UoL: The exit button ('red cross') still has the issue. I did not attempt to login but only exited using this option rather than cancel and it throws the following error.
[CRITICAL] Login attempt failed for https://xnat.vital-it.ch, please make sure your credentials for user  are correct!
Traceback (most recent call last):
File "/Users/kanishkasharma/Documents/GitHub/WEASEL/CoreModules/WEASEL/Menus.py", line 62, in
self.menuItem.triggered.connect(lambda : objFunction(self))
File "/Users/kanishkasharma/Documents/GitHub/WEASEL/Pipelines/iBEAt/XNATappiBEAt.py", line 22, in download
with xnat.connect(url, user=username, password=password) as session:
File "/Users/kanishkasharma/.local/lib/python3.8/site-packages/xnat/init.py", line 402, in connect
logged_in_user = check_auth(requests_session, server=server, user=user, logger=logger)
File "/Users/kanishkasharma/.local/lib/python3.8/site-packages/xnat/init.py", line 66, in check_auth
raise exceptions.XNATLoginFailedError(message)
xnat.exceptions.XNATLoginFailedError: Login attempt failed for https://xnat.vital-it.ch, please make sure your credentials for user  are correct!
Abort trap: 6

This specific window close button issue seems to be Mac-only. This exit ('red cross') button is disabled in Windows but it seems that for Mac it's active and the action triggered is the same as the OK button.
Unfortunately this cannot be solved now as we're developing Weasel for Weasel. You'll have to use the cancel button in every Weasel application and please avoid the exit button in Mac for now. It seems this isn't only on XNAT application, it also happens on Pipelines' applications such as Threshold, Square, etc.
I'm going to close this issue and open a new one with these last two comments.

@KanishkaS I made a new commit when this was resolved. This change means that for input windows for Gaussian, Threshold and XNAT, the close button should be either hidden or disabled.
I tested on my Mac VM, but could you try this on yours to check if it's working? Feel free to close if it's all good.

@JSousa-UoL: the button is disabled now but the pop-up window looks strange (see - 'ok' and 'cancel' button). Not sure if this is again a resolution issue.

It is related to the resolution issue - linking #55

Solution in #55


How to stop what we have started

After "sudo ./setup foo.bar ipaddress" a deamon getting started, by which we are getting an continuous update on named.log.
Because after this comman : sudo tail -f /var/log/named/named.log
This what i am getting
21-May-2020 19:09:45.725 notify: info: zone localhost/IN: sending notifies (serial 2)
21-May-2020 19:09:45.758 general: info: managed-keys-zone: Key 20326 for zone . acceptance timer complete: key now trusted
21-May-2020 19:09:45.759 resolver: info: resolver priming query complete
21-May-2020 19:30:55.724 general: info: received control channel command 'stop'
21-May-2020 19:30:55.724 general: info: shutting down: flushing changes
21-May-2020 19:30:55.724 general: notice: stopping command channel on <IP_ADDRESS>#953
21-May-2020 19:30:55.725 network: info: no longer listening on <IP_ADDRESS>#53
21-May-2020 19:30:55.725 network: info: no longer listening on <IP_ADDRESS>#53
21-May-2020 19:30:55.725 network: info: no longer listening on <IP_ADDRESS>#53
21-May-2020 19:30:55.728 general: notice: exiting

What if I want to end this process, whats the command for it.
How to completely remove this from the server/system where ever it is installed, to start it freshly

I have solved the issue,
While I was checking the status of Bind9, it was showing stopped and thrown some error about duplicate entries
When I checked,

On /etc/bind/named.conf
got multiple entries of : include "/etc/bind/named.conf.log";

On /etc/bind/named.conf.default-zones
got multiple entries of :
 zone "foo.bar" {        type master;        file "/etc/bind/db.local";  };

This may have happened because I run 'setup' file for multiple times, which generated the duplicate entries in these files.
Isn't it suppose to replace the entries on rerun of the setup file ?

Bind9 is running as a service, so you may stop it like any other service really. You do not need to re-run the setup script, just restart the service using systemctl or any other equivalent.

Isn't it suppose to replace the entries on rerun of the setup file ?

Not really, at least that wasn't the plan. This is just there to spin up a machine, run the script and start throwing out payloads that call to it. Hope this answers your question. :-)


Add menuAttributes to fix #361

Add the menuAttributes option to allow for attributes and their values to be added to the menu part of the component.
This fixes #361 as it allows for aria-labelledby to be set to the id of the existing label which in turn allows the code to pass the AXE rule "aria-input-field-name" (ARIA input fields have an accessible name).
No new tests have been added yet. While this PR adds the ability to add aria-label or aria-labelledby, by default these are not set, and so the default examples on the demo pages will still not pass the aXe tests.
What do you think we should do about this?

Set a default aria-label value (But what would it say by default?)
Forcibly set an id on the label and use aria-labelledby to associate the two
Update all the examples to include manually setting an aria-label

Thanks for this update @mchughbri (and your patience with us finding the time to review it 😬 ). The code update is looking good 🙌🏻
For the examples, it would be neat to update them now we have that new option. I think having an id and using aria-labelledby would nudge people towards the use intended for menuAttributes, so that would be my prefered way to go. If you have a moment to update them and add a little testing for the new option, that'd be ace! Thanks 😄

Updated examples and added test for the new menuAttributes option

@romaricpascal now done thanks.

Oh sorry for my unclear instructions, I meant a force push with a different commit hash to try and trigger whatever Github's do to relaunch the checks.

Hey @romaricpascal just wondered if we were any closer to getting another approval on this? Thanks.

@mchughbri Sorry for the second delay on this! Looking good, merged :)


WebDAV SSL implementation is unintelligible

Expected behaviour
Proper validation of SSL certificates similarly to IMAP
Actual behaviour
One of the two socket factories used disables hostname verification, essentially breaking SSL.
mSchemeSocketFactory.setHostnameVerifier(org.apache.http.conn.ssl.SSLSocketFactory.ALLOW_ALL_HOSTNAME_VERIFIER);
 * TODO: find out what's going on here and document it.
 * Using two socket factories looks suspicious.

This is terrible. Quoting the Javadoc:

The ALLOW_ALL HostnameVerifier essentially turns hostname verification off. This implementation is a no-op, and never throws the SSLException.

The code was committed in : https://github.com/k9mail/k-9/commit/8f45d76\
I'm not sure why we stopped using the TrustedSocketFactory, which seems to verify certificates correctly. Possibly the use of ALLOW_ALL_HOSTNAME_VERIFIER was a mistake (the SSL libraries are terrible for making opaque code that subtly breaks - see:

The Most Dangerous Code In The World

Steps to reproduce
Untested. I suspect:

Create an WebDAV account
Point to a server with a bad certificate
No error thrown

Environment
K-9 Mail version: master
Android version: N/A
Account type (IMAP, POP3, WebDAV/Exchange): WebDAV
Mitigation factors
In previous versions WebDAV is broken. Specifically, due to #1143 it will fail to sync at all often.
Intended resolution
Short term: Replace argument with STRICT_HOSTNAME_VERIFIER
This may cause current WebDAV usage to fail due a badly signed certificate that is never the less expected.
Long term: To resolve this I intend to rip out the WebDAVSocketFactory and replace it with non-deprecated APIs, with tests for SSL errors, following best practice as outlined in: http://developer.android.com/training/articles/security-ssl.html

WebDavSocketFactory doesn't look like code I wrote. I probably copied it from somewhere. Sadly, I can't remember doing that and have no idea what my motivation was :(

It looks like you copied it from the TrustedManagerSocketFactory. I just can't see the reasoning behind using separate factories yet.

See PR #1182 for the resolution to this.

Obsolete because of #4251


Fix RSM types.

Fixes parsing RSM numerical types as actual numbers instead of strings.

@legastero Please merge this. :)

@legastero please merge this.

@legastero are you maintaining this project? ;)


Hex density scaling

initial PR for hex density scaling, same content as pdf I shared titled "Method for geographic based reward scaling_1006.pdf"

Rendered view https://github.com/helium/HIP/blob/465611dcfa6a5036c5bce30b0f3e5223aaf064d6/0099-hex-density-based-transmit-reward-scaling.md

Numbered and merged as HIP17. To add feedback or contribute changes, please visit the tracking issue: https://github.com/helium/HIP/issues/60


NativeBase as components

So I'm getting into React and was looking around for resources.
Found this app which seems nice. I also stumbled upon NativeBase (http://nativebase.io/) and, well it would be really nice if those could be added as components to Deco.
Tried manually (npm) installing it and using them inside Deco, worked like a charm! :)

+1

+1

+1 Yep, we're doing this.

+1 This would be great, any idea when it'll be ready?

+100

+120

Are you still doing this?


tooltipster not working in IE8 or IE11 - RESOLVED

Hello,
I can get this to work with a hover and content attribute in Chrome or Firefox, but not at all in IE.
Caleb's hover sample on the Tooltipster page works in IE - the one that shows HTML text and an image of chocolate soufflé, but not for me.
I am trying to accomplish that same type of thing, but have patterned my attributes very similarly to no avail. If anyone can shed light on this, it would be nice. On the Tooltipster home page, one of the list items shows Fixed Position and Width. It's this type of sample I'm trying to replicate.
Thank you!

Hello,
I noticed incompatibilities the other day when you use jQuery 1.8 in IE, is that your case ? Since jQuery 1.9 was also a little buggy, I suggest you upgrade to jQuery 1.10.
If it's still not working, check the console to see if there are errors, it may help you find what's wrong. Copy-paste the errors here if you think that the error does not come from your own code.
Thank you.

Thanks....I am including these, though.

And that's the odd thing. I'm not getting an error at the console level, so I'm not too sure how to properly debug it. Going to keep trying though.
Thanks.

I would not open a docx file, please paste its content here.

Were you able to discern anything from my screen shots?
From: Louis Ameline<EMAIL_ADDRESS>Sent: Thursday, November 12, 2015 12:41 PM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
I would not open a docx file, please paste its content here.
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D156196789&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=8wBSqONNAVYbBuYOmgFPC1TeSMfCUVB0ejnhBIDBTyo&s=63c6rwyg8CiItU3QsULieVwIuAqtIT1_qHln-5lxM24&e=.

I do not wish to open docx files, please post your images here.

Yes, I did….did you receive them?
From: Louis Ameline<EMAIL_ADDRESS>Sent: Friday, November 13, 2015 3:08 PM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
I do not wish to open docx files, please post your images here.
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D156557162&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=7cpwiJ3_k3l3ME64MZps8lrCMy-YjxARVQpG6WBME5A&s=BfM3_uveXE4aEgB73RpnFH1UlRemqB8Gz5Sz34sGLMg&e=.

I don't see any screenshots on this page.

Trying again.
Done in IE 8 or 11
[tooltipster_html]
Broken HTML
In Chrome
[tooltipster_chrome]
Please confirm they get through on this one.
Thanks,
Barry
From: Louis Ameline<EMAIL_ADDRESS>Sent: Friday, November 13, 2015 3:21 PM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
I don't see any screenshots on this page.
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D156559662&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=x0pkGNQEP65zSaMl9sb2c1p5fpzuOaFylFL323nrYMU&s=8j9be5m_DsF6jUmXt3f1u7_w1ub9nkwQO_odjyPcfbc&e=.

Are you sending the pics by mail? Come to this page instead and include them here.

I had been, yes, and the web page here suggested uploading docx files among others.
Let me see if I can upload the pics in question
From: Louis Ameline<EMAIL_ADDRESS>Sent: Friday, November 13, 2015 5:01 PM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
Are you sending the pics by mail? Come to this page instead and include them here.
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D156582872&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=x90TDcatx3DLk8r8fMyngLoPeakuehlHvM6fwZoS61I&s=58TzoUEOeNCp3BQd0apOslSSAFVWAyM9efDgi4pnd1U&e=.

There they are! ^^^ First one is in Chrome OR Firefox (or even safari will replicate this).
Bottom one is in IE -- either 8 or 11 behave this way.

Can I have the url of the page please?

It’s on an intranet, closed off.
From: Louis Ameline<EMAIL_ADDRESS>Sent: Friday, November 13, 2015 5:08 PM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
Can I have the url of the page please?
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D156583970&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=pKAscMIwTtZUI4CNU22n54cNs4HV60fr6rI0j4WqyFM&s=RHoj2-aZRwI14Xnvu9GxM7OuCTy4xHzOcpZef2g4DpI&e=.

Then I can't help you, sorry. Try to debug yourself with breakpoints.

I can provide the source code if needed.
From: Louis Ameline<EMAIL_ADDRESS>Sent: Friday, November 13, 2015 5:08 PM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
Can I have the url of the page please?
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D156583970&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=pKAscMIwTtZUI4CNU22n54cNs4HV60fr6rI0j4WqyFM&s=RHoj2-aZRwI14Xnvu9GxM7OuCTy4xHzOcpZef2g4DpI&e=.

I don't have time to play around with files, sorry. If you can manage to create a public page it's great, otherwise I'll close this issue.

OK, very good.
I’ll see if I can push out to a public site and replicate there, and then provide that IRL.
Thanks.
From: Louis Ameline<EMAIL_ADDRESS>Sent: Saturday, November 14, 2015 8:59 AM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
I don't have time to play around with files, sorry. If you can manage to create a public page it's great, otherwise I'll close this issue.
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D156709246&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=Sw39Rlt2tJtLiIVn6zGd6N9G2f3GXRmTLM-6wzDU5n0&s=VtseRCKOG2qYCRSpelljS4_0dVNpE_JqaOZMNo1b5uQ&e=.

See if you can access this link, Louis.
This is public.
http://bpropes20.readyhosting.com/barry/tooltip_test_public.html
In it I’m seeing a broken image…not sure if that’s indicative of something else.
Regards,
Barry
From: Louis Ameline<EMAIL_ADDRESS>Sent: Saturday, November 14, 2015 8:59 AM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
I don't have time to play around with files, sorry. If you can manage to create a public page it's great, otherwise I'll close this issue.
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D156709246&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=Sw39Rlt2tJtLiIVn6zGd6N9G2f3GXRmTLM-6wzDU5n0&s=VtseRCKOG2qYCRSpelljS4_0dVNpE_JqaOZMNo1b5uQ&e=.

Or you can try this one.
http://www.barrypropes.com/barry/tooltip_test_public.html
Thanks,
Barry
From: Louis Ameline<EMAIL_ADDRESS>Sent: Saturday, November 14, 2015 8:59 AM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
I don't have time to play around with files, sorry. If you can manage to create a public page it's great, otherwise I'll close this issue.
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D156709246&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=Sw39Rlt2tJtLiIVn6zGd6N9G2f3GXRmTLM-6wzDU5n0&s=VtseRCKOG2qYCRSpelljS4_0dVNpE_JqaOZMNo1b5uQ&e=.

Make it work in Chrome at least, then we'll see in IE...

I actually got it now.
In IE11. Let me check 8.
Kind of odd though, only one link will display…the others don’t like the idea of it.
From: Louis Ameline<EMAIL_ADDRESS>Sent: Monday, November 16, 2015 9:21 AM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
Make it work in Chrome at least, then we'll see in IE...
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D157064856&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=ErEKUA9205UhRb6oaj6TSkbDVcNkSpIciPd5gmgjg5Y&s=QKymj-YR66ylOb7LAoDQ3CQSw5p7FKnA0Ok4V43ZJ88&e=.

Actually, I got one working, but another div doesn’t, and I was thinking should.
Take a look at the 2nd div, demo-html.
That one is one that is not working, but I was thinking should.
Thanks,
Barry
From: Louis Ameline<EMAIL_ADDRESS>Sent: Saturday, November 14, 2015 8:59 AM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
I don't have time to play around with files, sorry. If you can manage to create a public page it's great, otherwise I'll close this issue.
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D156709246&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=Sw39Rlt2tJtLiIVn6zGd6N9G2f3GXRmTLM-6wzDU5n0&s=VtseRCKOG2qYCRSpelljS4_0dVNpE_JqaOZMNo1b5uQ&e=.

Look, I'm sorry but I do not have the time to go through this if you cannot set up a clear test use case. Try to get help on stackoverflow on how to use Tooltipster. Thank you.

Ok, thanks.
From: Louis Ameline<EMAIL_ADDRESS>Sent: Monday, November 16, 2015 9:37 AM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
Closed #486https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=OgAT-KKv8K7vaT-pFXhor-KqssbRKwu6jnx0OLIz3TA&s=aK3wbFe_nYDrwoOJisbaLlEUAx2ykjg_XoCHf3PU0kc&e=.
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23event-2D465555060&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=OgAT-KKv8K7vaT-pFXhor-KqssbRKwu6jnx0OLIz3TA&s=8RmAxg9ooh8qRukT2HAJLWxb9i8glIf7ZjqkQjtJPTQ&e=.

I got this working….thanks for your, help?
From: Louis Ameline<EMAIL_ADDRESS>Sent: Monday, November 16, 2015 9:21 AM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
Make it work in Chrome at least, then we'll see in IE...
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D157064856&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=ErEKUA9205UhRb6oaj6TSkbDVcNkSpIciPd5gmgjg5Y&s=QKymj-YR66ylOb7LAoDQ3CQSw5p7FKnA0Ok4V43ZJ88&e=.

Maybe not for my help, but at least for my time and the attention I paid as long as I could.

Thank you!
It is always good for an extra set of eyes, though. It does help when another can look and see something quickly! : )
From: Louis Ameline<EMAIL_ADDRESS>Sent: Monday, November 16, 2015 12:44 PM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
Maybe not for my help, but at least for my time and the attention I paid as long as I could.
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23issuecomment-2D157131973&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=hS0GvHFM4FUlYY7WSZe-92R79Xqj_XpVV7sqUCbmfK8&s=hUwa-hzt9apdy9ivIK2uyERBWdMjVjbw9mYng7A64PM&e=.

Not sure if you can add to this or not, but I thought I would let you know I did find a workaround or solution to this while exploring something with Angular JS.
It merely entails adding THIS line of code right after the head tag, as the first META element.

This allowed it to work!
Regards,
Barry Propes
Citimortgage - Workflow Enhancements
6400 Las Colinas
(972)653-4561
Follow us on:  [Twitter] http://twitter.com/Citibank/   [Facebook] http://www.facebook.com/citibank   [YouTube] http://www.youtube.com/citi
From: Propes, Barry L [GCB-OT]
Sent: Monday, November 16, 2015 9:37 AM
To: 'iamceege/tooltipster'; iamceege/tooltipster
Subject: RE: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
Ok, thanks.
From: Louis Ameline<EMAIL_ADDRESS>Sent: Monday, November 16, 2015 9:37 AM
To: iamceege/tooltipster
Cc: Propes, Barry L [GCB-OT]
Subject: Re: [tooltipster] tooltipster not working in IE8 or IE11 (#486)
Closed #486https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=OgAT-KKv8K7vaT-pFXhor-KqssbRKwu6jnx0OLIz3TA&s=aK3wbFe_nYDrwoOJisbaLlEUAx2ykjg_XoCHf3PU0kc&e=.
—
Reply to this email directly or view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_iamceege_tooltipster_issues_486-23event-2D465555060&d=BQMCaQ&c=j-EkbjBYwkAB4f8ZbVn1Fw&r=GrngNI9-Y1vwka9eiSQJvBjwJtXdM9U9QfF-SCO-iqM&m=OgAT-KKv8K7vaT-pFXhor-KqssbRKwu6jnx0OLIz3TA&s=8RmAxg9ooh8qRukT2HAJLWxb9i8glIf7ZjqkQjtJPTQ&e=.

For future reference i managed this to work by using this:
,
functionPosition: function(instance, helper, position){
/console.log("---------------------------");
console.log(instance);
console.log("---------------------------");
console.log(helper);
console.log(helper.origin);
console.log($(helper.origin).offset());
console.log("---------------------------");
console.log(position);/
					var pos = $(helper.origin).offset();
					if(position.coord.left == 0){
						position.coord.left = pos.left;
					}

			        return position;
			    }

Ok, WhitecastlePT, did that manage to hold it in place, and across different browsers? Do you have a screen shot or demo URL of this?

Well in my case is not ideal solution but this is only for IE When rendered in IE8. Helper is the element of origin so i figured, ok if i know the position of the element i can add it to the tooltip if its 0.
In chrome or firefox position.coord.left is never 0 so i can say this is crossbrowser. However its not ideal solution.


Write documentation for unit testing with XQSuite

Your to-do
Convert experimentation with testing in separate djbpitt/exist-sandbox repo to testing tutorial
Issues you may encounter
None
Additional context

[] How to make the script to create unit test HTML output accessible in eXist-db


rename kur to kur_ara

@stweil Please review.

@Shreeshrii  Kur is correct, does not need change.

@BisarOmer, would kmr be fine?

Please see https://en.m.wikipedia.org/wiki/Kurdish_languages
kmr is kurmanji written in Latin script
ckb is Sorani written in Arabic script
Tesseract 3 kur was in Arabic script.
Tesseract 4 kur was in Latin script and was renamed to kmr.

@stweil  kmr is fine for the Kurmanji dialect which is written in the Latin alphabet and in tessdata should remain as kmr, but for the Kurdish central which is written in the  Arabic script. kur or  kur_cen will be fine and so far tesseract is not trained for Kurdish central so not available at tessdata.  and also on langdata kurd_ara needs change

@stweil  kmr is fine for the Kurmanji dialect which is written in the Latin alphabet and in tessdata should remain as kmr, but for the Kurdish central which is written in the  Arabic script. kur or  kur_cen will be fine and so far tesseract is not trained for Kurdish central so not available at tessdata.  and also on langdata kurd_ara needs change


Get rid of do_queue?

The Operation class has in its __init__() the slightly obscure do_queue=True parameter. It not only clutters the docs (unless manual docstrings are given) but it is also not very well documented and the feature should be irrelevant (and thus hidden, imo) from the user.
As far as I can tell, this option is actually set to False only in two places:
The first one, in default_gaussian.py is actually even marked as a hack:
    # HACK, we need access to the Poly instance in order to expand the matrix!
    op = qml.expval.PolyXP(Q, wires=wires, do_queue=False)
    Q = op.heisenberg_obs(N)

Only Q is used in the following code, so it would be better to obtain Q here without having to first instanciate an PolyXP() object. In a similar situation it was recently argued that we should accept code duplication in order to keep the plugin code self-contained. I would argue that the same rational could/should be applied here.
The second is in qnode.py in _pd_analytic(). Here, PolyXP() is called to compute the observable to measure via the "2nd order method" and do_queue=False is used to prevent the computed observables from adding themselves to the queue of the qnode, as they are only to be passed on to the device via execute().
In conclusion, I would suggest to remove the do_queue property from Operation, create a private sub-class of PolyXP with an overwritten queue() method that does nothing and use that instead of the regular, user facing, PolyXP in these to places. To make the plugin code fully self-contained, one could further duplicate the code from PolyXP.heisenberg_obs() in the default_gaussian plugin.
Here all appearances of do_queue in /pennylane/:
grep --color -nH -r -e  'do_queue' *
operation.py:190:        do_queue (bool): Indicates whether the operation should be
operation.py:253:    def __init__(self, *args, wires=None, do_queue=True):
operation.py:300:        if do_queue:
operation.py:433:        do_queue (bool): Indicates whether the operation should be immediately
Binary file plugins/__pycache__/default_gaussian.cpython-35.pyc matches
plugins/default_gaussian.py:636:    op = qml.expval.PolyXP(Q, wires=wires, do_queue=False)
Binary file __pycache__/qnode.cpython-35.pyc matches
Binary file __pycache__/operation.cpython-35.pyc matches
qnode.py:767:                    return pennylane.expval.PolyXP(qp, wires=range(w), do_queue=False)

I'm in favour of patching Operation.queue in this case. I think it is suboptimal to have parameters that exist purely for test-purposes in the codebase, especially when it is easily achievable otherwise.
I don't know exactly how to do it, but I'm pretty sure that one could implement a suppress_queuing fixture or, even better, a context manager that supresses queuing in its context by patching Operation.queue.

@johannesjmeyer, since last discussion on this issue, we have noticed use cases for do_queue outside of unit tests. For example, there are some optimizers that may need to modify the QNode queue dynamically.
One solution is to remove the exception that is raised if an operation is used outside of a QNode.
Queueing now becomes implicit if inside a QNode, and the operation does nothing outside of a QNode (which I think makes sense)

@johannesjmeyer, since last discussion on this issue, we have noticed use cases for do_queue outside of unit tests. For example, there are some optimizers that may need to modify the QNode queue dynamically.

Could you point me to an example?

Queueing now becomes implicit if inside a QNode, and the operation does nothing outside of a QNode (which I think makes sense)

Yeah, that definitely makes sense!

Queueing now becomes implicit if inside a QNode, and the operation does nothing outside of a QNode (which I think makes sense)

If we can make sure that the queuing happens only when called inside the QNode and not inside expval(PauliX(0)) then we can get rid of the need to QNode._current_context.queue.remove(ops) inside the expval.__call__ method. I do not know if there can be a selective check like that to stop queuing if called inside the expval function.
Also makes sense to make it implicit.

Suggested simple fix:

If an op is created outside of context managers, then it is not added to any queue (similar to what happens when do_queue=False)
If an op is created inside a relevant context manager, it is automatically added in the corresponding circuit queue (similar to current default behaviour)
If an op is created inside a relevant context manager, but the user does not want to add it to the queue, they specify this with an extra keyword argument, e.g., _queue=False

In addition, it feels hacky to pull ops out of the queue just to update return_type, then put them right back. There are three potential solutions, which differ in scope:

Re-think the queuing behaviour across the board (probably too hard)

Remove queuing from just observables (since it is not needed), and have the expval/sample/var functions perform the queuing

See if it's possible to find and update the observable in the queue, without removing it?

I'm thinking option number (2) is preferable.

Closed by #359


Write VCD file

Is it possible to write in vcd file ?
Could be useful to annotate vcd traces.

No, not at the moment. Although you can add new signals to a trace like here, and I have an experimental (still private) branch that allows Surfer to connect to WAL to show these signals.
So this is something we are working quite actively on.
I will have a bit more time starting next week and will try to implement writing back vcds, though.


Adding an overview of the fragments system

This is to address #203
This is a first pass at documenting our thinking on the fragment system. I wanted to get something in the readme to provide some guidance. It could definitely use more content on how to author an extension/fragment.

Definately good to have this description.
I think it could be a good idea to separate "Developer" things and the API specification into separtate documents. Not sure whether the average user is interested in the more internal technical stuff for development of the spec itself.

Oh and we should somehow add the generation process to the CI. I just ran the commands and one of the generated files doesn't seem to be up-to-date. We should ensure they are always up-to-date.


PEP 3111 -- Simple input built-in in Python 3000

http://www.python.org/dev/peps/pep-3111/

Covered in https://github.com/IronLanguages/ironpython3/pull/118.


ngx-datatable not resizing correctly inside a flex div wrapper

I am building an Angular 10 app that is using flex to layout various panels.
I have a ngx-datatable wrapped by a flex based div and I have a problem where the ngx-datatable that is notionally set to take 100% width and 100% height is not resizing as the flex wrapper resizes.
The wrapping div has a final vertical resize that happens after the ngx-datatable has rendered and shows data.   The table does  not resize after that final vertical resize leaving a vertical gap between the bottom of the table and bottom of the wrapper div, as shown below

Any suggestions?

.eis-vbox {
  height: 100%;
  width: auto;
  box-sizing: border-box;
  display: flex;
  flex: auto;
  flex-direction: column;
  align-items: stretch;
  padding: 0px;
  margin: 0px;
}

.full-area-grid {
  height: 100% !important;
  width: 99% !important;
}

<div id="simpleListWrapper" #wrapper class="eis-vbox" >

  <ngx-datatable #simpleListTable id="simpleList"
    class="material striped full-area-grid"
    [rows] = "rows$ | async"
    [columns]="columns"
    [sorts]="sorts"
    [scrollbarV]="true"
    [footerHeight]="50"
    [headerHeight]="50"
    [rowHeight]="40"
    selectionType="checkbox"
    [loadingIndicator]="isloading > 0"
    [scrollbarV]="true"
    [externalPaging]="true"
    [externalSorting]="true"
    [count]="totalRows$ | async"
    [offset]="pageNumber$ | async"
    (page)="setPage($event)"
    (sort)="onSort($event)"
    (select)="onSelect($event)"
  >
  </ngx-datatable>

</div>

@davoutuk I noticed your container has a display: flex; flex-direction: column;style, however, you're setting the datatable height to 100%. Have you tried flex: 1instead?


[Feature Request] Indefinite delays

Is your feature request related to a problem? Please describe.
Finite delays, even portMAX_DELAY, can be a source of subtle, hard to find bugs.
Describe the solution you'd like
An INDEFINITE_DELAY macro that is a special value or NaN (it could for example be the maximum possible integer value, such that portMAX_DELAY's value is decreased by 1, and the macro renamed to portMAX_FINITE_DELAY), such that any function facing this, will automatically internally check if the max delay has been reached/exceeded, and automatically then reloop/retry.
Describe alternatives you've considered
Currently, I must place calls to blocking functions (that I pass portMAX_DELAY as a parameter) within an empty while loop, which is error-prone due to repetition, and ugly.
Additional context
Add any other context or screenshots about the feature request here.

Thanks for the feature request. I'll bring it up with the rest of the development team.

@moefear85 if you define INCLUDE_vTaskSuspend to 1, then FreeRTOS treats portMAX_DELAY as being infinite.

thx... was this a preexisting feature? I wasn't aware of it

Yes it was preexisting.  Maybe easy to miss though!

@moefear85, did @jefftenney's comment resolve your issue?

i haven't tried it yet, i'll get around to it later. btw the point of it was that it's generally safer and avoids bugs by having an explicit portINFINITE_DELAY macro that always works as such without needing to be "turned on".


Use AbsoluteUri in FakeNavigationManager to preserve escpaed uri's

Pull request description
Fixes #458
In theory this is a API breaking change as Uri is different before than after the change.

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

Thanks for this. I don't see a reason this should affect existing users. Do you?

Only if someone was relying on unescaped url's from the FakeNavigationManager :D

Oh yeah, remember to update the changelog.md file accordingly.

In favor of your latest change. It has less side effects.

Ahh, a can you fix the test? Then ill merge.

Fixed @egil
The test also just tests for the "raw" input aka Uri.OriginalString

Merged. Thank you for the contribution!


Airflow Webserver fails to Start

After running docker-compose up, all images install correctly but the web server never successfully launches. In the output, this appears to be the error:
webserver_1  | KeyError: 'Variable example_variables_config does not exist'
I'm unsure how to fix this.

@EvanKaeding https://github.com/tuanavu/airflow-tutorial/issues/14#issuecomment-467301310

Thanks for reporting this error, I have commented out the example_variables.py to avoid the missing variable error.


Add disk caching

WIP. Currently the test tree sequence has no UUID so trying to find a way to get one.

LGTM - does it work?
We'll want to integrate with the CLI, have you thought about how to configure the cache dir? (I was think appdirs might be a good approach, ultimately?)

Easiest way to get a uuid is to save to tempfile and read back in

I've modified this to use appdirs and to cache based on the hash of files of functions in the calling tree of the cached function. This means that changes to pages code won't trigger an invalidation, which will help a lot with dev.

diskcache is really quite complex.  We really don't need this, and it also means we'd have to be much more careful about managing opening and closing the cache than we are here. We could just store the files (as CSVs, even) based on their keys and it would be much simpler and easier to maintain.

From the diskcache docs: "Cache objects are thread-safe and may be shared between threads. Two Cache objects may also reference the same directory from separate threads or processes. In this way, they are also process-safe and support cross-process communication." So I don't think we need to worry about opening and closing

The key generation policy is clever, but expensive to run and hard to test. Just incrementing a number when we change the dataframe format isn't that hard, and it'll make key generation fast.

Yeah, fair point. I replaced it with a version string.

Did you push the changes here @benjeffery?

Whoops, should be there now!

From the diskcache docs: "Cache objects are thread-safe and may be shared between threads. Two Cache objects may also reference the same directory from separate threads or processes. In this way, they are also process-safe and support cross-process communication." So I don't think we need to worry about opening and closing

In principle - in practise, sqlite hasn't worked well in my experience without carefully handling the context managers. Let's try it out and see anyway.


Different timezone used with  dateparse.RetryAmbiguousDateWithSwap(true) option.

When running the below test time zone is not consistent:
func TestDateParseOptions(t *testing.T) {
	t0, err := dateparse.ParseAny("21/09/2010", dateparse.RetryAmbiguousDateWithSwap(true))
	require.NoError(t, err)

	t1, err := dateparse.ParseAny("09/21/2010", dateparse.RetryAmbiguousDateWithSwap(true))
	require.NoError(t, err)

	require.Equal(t, t0, t1)
}

   -(time.Time) 2010-09-21 00:00:00 +0200 CEST
   +(time.Time) 2010-09-21 00:00:00 +0000 UTC

I guess this is due to this line (below) using time.Local instead of the loc variable. If anyone can guide me if this is somehow intentional or can be changed (I can create a merge request).
https://github.com/araddon/dateparse/blob/6b43995a97dee4b2c7fc0bdff8e124da9f31a57e/parseany.go#L253

@jozuenoon -- this bug is fixed in the fork of this package https://pkg.go.dev/github.com/itlightning/dateparse (we're trying to maintain this package) -- I verified that your example test case already passes in the forked package.


Add info about manual installation on Windows in docs

Rn we don't explicitly mention that the installation script won't work out of the box on windows, we should mention that manual installation is required.
We can get inspiration from Scarb https://docs.swmansion.com/scarb/download.html#windows

Is this still open for contributions?

@Etimfon007 yup, willing to work on that?

@Etimfon007 assigned you!

Hey @Etimfon007, are you already in progress of creating the PR for this one? If not, let me temporarily un-assign you until #710 is done, so we're not blocking this issue for others.

Hello @cptartur I'm on progress. Prolly before the end of this week I'll be merging a PR.

Hello @cptartur can you temporarily un-assign me to this? I'm currently working on another issue. Just so that someone else can pick this one up to fast-track the development of StarkNet


Add step to recreate all VMs on deploy of new CA

This PR adds a step to recreate all VMs when deploying a new CA and when activating that new CA.

This is necessary in order to force BOSH to propagate your new CA onto all VMs via the IaaS metadata.
Skipping this step would result in downtime when regenerating certificates because some VMs would not trust the new CA while others would have certificates signed by that new CA.
This should also be applied to all versions of OpsManager (PRs will be created).

@nickjameswebb thanks! The changes have been merged and cherry-picked to the 2.0 - 2.3 branches


pull request for e2e test at 01/02/2017 13:56:15

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


Brew: Add greedy preference; upgrade and doctor commands.

Description

Add "greedy" upgrade preference.
Add upgrade and doctor commands.

Type of change

Bug fix / improvement for my extension

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing

Review done. @nhojb One issue that I had when running brew doctor: It reports "Your system is ready to brew" when I run it in the terminal but shows some errors when running it through the extension command. I guess that's because the PATH variable is not set?

Yes, I get the same problem. I think launchd (and hence Raycast) pick up a different environment than a shell you might run from your Terminal. I almost didn't include the doctor command for this reason. Perhaps it's better just to remove it (I originally wrote it to try and debug issues with the brew process running in the extension process)?

On second thoughts I think I'll just remove the doctor command.

Yeah, the env is different since it's a different process, launched outside of the shell – the shell usually picks the customized PATH up from one of the known config files (~.zshrc, etc.) so brew doctor would yield different output when launched from there.


Update CONTRIBUTING.md

Added Steps to create a virtual environment and to activate it

Hi @Ashwin95525  , we will be merging this PR!
We appreciate your contribution!
Great work

@sai-sahith7 Thank you so much for your time and for merging the PR.
I really appreciate it!!


Can't load MIcrosoft.extensions.Logging.LoggerFactoryOptions when running publish

Hi!  Interesting tool!
Tried it on a  project and got a loading exception.  The project is .net core.

Makes any sense ?

Seems like something is wrong with assembly resolve.
(I had a lot of issues in that area due to the fact that I need to actually run the web project assembly to get Api Explorer info).
I'll try to reproduce that issue. Would you be able to provide any repro project?
Even .csproj file content might be enough...

You have ASP.Net Core 3.1 project, right?
Seems like I've broken 3.1 compatibility in latest release in some cases.
Could you try using 0.0.5 release to check if it works?
( dotnet tool update --global GenerateAspNetCoreClient.Refit --version 0.0.5 )

@OsirisTerje
I've added (quite an ugly) workaround to support 3.1 projects, so I hope to have it fixed in 0.3 version.
Would you be able to check it?

I expect this issue to be fixed, so I'm closing it.
Please let me know if it still happens for you.


Organization List Page - Delete an Organization

Web Path
Home Page -> Nonprofits

Requirement
When Delete button is clicked, and user confirms "YES", delete this organization.

Rest API
You should call this endpoint to delete an organization: DELETE /api/organization/delete/{id}
Rest API document is located at: http://localhost:8080/swagger-ui.html

This issue was moved to Code4SocialGood/c4sg-web#23


Fix form field dragging bug

Currently when dragging form fields, the field jumps about 100px above where it should be.

This has gone


Add docs for correlation metrics

Linked issue(s)
What change does this PR introduce and why?
Adds documentation for Pearson's Correlation Coefficient, and Spearman's Correlation Coefficient.
Please check if the PR fulfills these requirements

[ ] Include reference to internal ticket and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Relevant tests for the changes have been added
[ ] Relevant docs have been added / updated

These pages look really great, @brian386. 👍
A couple of questions I have are:

Does "monotonic relationship" need any further explanation? I'm thinking a small callout next to the first paragraph of metrics/spearman-correlation could be nice.
As I was reading both pages, I assumed that the example data were exactly the same but it seems like one of the predicted temperatures is different. Was that intentional? I think it would be nice to have a matching example so you can kind of compare how these metrics can differ.

Thanks! For 1, I explain monotonic relationship in the second part of the sentence: "where as one variable increases, the other also increases." Should this be explained further or worded differently? For 2, I changed the numbers to match - I wasn't paying too much attention to that before.


Fresh start from Template v.1.9

Many thanks to contributing to nf-core/diaproteomics!
Please fill in the appropriate checklist below (delete whatever is not relevant). These are the most common things requested on pull requests (PRs).
PR checklist

[ ] This comment contains a description of changes (with reason)
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If necessary, also make a PR on the nf-core/diaproteomics branch on the nf-core/test-datasets repo
[ ] Ensure the test suite passes (nextflow run . -profile test,docker).
[ ] Make sure your code lints (nf-core lint .).
[ ] Documentation in docs is updated
[ ] CHANGELOG.md is updated
[ ] README.md is updated

Learn more about contributing: https://github.com/nf-core/diaproteomics/tree/master/.github/CONTRIBUTING.md

Looks like you still have .travis.yml there - needs deleting

Great stuff!


Importing lib from npm gives an object instead of function

Library we are importing:
https://github.com/defunctzombie/num/blob/master/num.js
Import code:
import num from 'npm:num'

Returns an object:
{ default: function }
Is there a way to import this where num is a function and not an object?

Is it because the package.json main's property?
https://github.com/defunctzombie/num/blob/master/package.json#L17

This is the expected behavior. It keeps you compatible with the ES6 import syntax used everywhere in ember-cli. You'll notice that every other dependency behaves the same way (try require('ember') in the console).
Instead of writing this:
var myThing = require('npm:my-thing');

write this:
import myThing from 'npm:my-thing';

And the ES6 transpiler will do the right thing for you.

Ah ok, I was debugging with a debugger statement and using the imported library.
I see what you are saying now. Thank you.


DFCLoader failed to reuse already existing bucket

@alex-aizman @VladimirMarkelov @liangdrew @sasanap
this fixes Sachin's dfcloader problem
thanks


table on container image view with multi arch on smal screens do not scale

tested against gitea versions: 1.21.0+dev-559-g3b129aaa80, 1.20.2
the hash should have text-overflow: ellipsis; or something like that ....

QQ. How to push the docker image?
PS already gone though the docs.

@6543 Please tell me how can I reproduce this issue?

can you assign this issue to me

QQ. How to push the docker image? PS already gone though the docs.

Just push a multiarch image to the build-in registry

Hello, I am trying to reproduce this issue but I am not able to push to the built-in registry.
I have setup gitea on my local using the installation guide.
I am following this guide to push a docker image, for which, I have to login to my private registry first.
app.ini
[server]
SSH_DOMAIN = localhost
DOMAIN = gitea.example.com
HTTP_PORT = 3000
ROOT_URL = http://gitea.example.com/

Steps I took:
$ docker login -u mihir -p 12345678 gitea.example.com
Login Succeeded

$ docker image tag hello-world:latest gitea.example.com:3000/mihir/hello-world:latest

$ docker push gitea.example.com:3000/mihir/hello-world:latest
The push refers to repository [gitea.example.com:3000/mihir/hello-world]
12660636fe55: Retrying in 8 seconds

The docker push command keeps on retrying and eventually fails.
I changed the log level to TRACE, and I found these logs whenever I try to push my image:
2023/12/26 22:57:48 ...s/process/manager.go:188:Add() [T] Start 658b0d14: GET: /v2/ (request)
2023/12/26 22:57:48 ...eb/routing/logger.go:47:func1() [T] router: started   GET /v2/ for <IP_ADDRESS>:64403
2023/12/26 22:57:48 ...eb/routing/logger.go:102:func1() [I] router: completed GET /v2/ for <IP_ADDRESS>:64403, 401 Unauthorized in 0.1ms @ container/container.go:118(container.ReqContainerAccess)
2023/12/26 22:57:48 ...s/process/manager.go:231:remove() [T] Done 658b0d14: GET: /v2/

I have also added the following line at the bottom of my /etc/hosts:
<IP_ADDRESS>       gitea.example.com

Version Info:
Gitea version: 1.22.0+dev-467-g907c97aab built with GNU Make 3.81, go1.21.0
System Version: macOS 14.1.1 (23B81)

It should have been fixed because now it only shows truncated digest


Update mix.exs

clean up dependency, unlock calendar 0.17 (previously locked to 0.16.x)
close #33

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


[backport] Move CI requirements to Cupy repository

Backport of #2533

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 06c6bcd510a26291f0a347b726d9d3f3bc9ec26e, target branch v6) succeeded!

CI is unrelated


Propose to add italian language

Added the italian traslation file .json

Thanks!

Yw, and if u need any other traslation write to me!


Fixes #4783 - preserve content_type for unauthenticated test responses

In Rails 4, for unauthenticated controller tests which trigger the failure app, ensure that the simulated failure response includes a content_type.
This works in Rails5, which parses the content-type header on-demand, but not in Rails4 which requires setting the response's content_type explicitly.
Fixes #4783

It seems like something might be missing from this commit. Making an unauthenticated controller request now results in NoMethodError: undefined method rails5?' for Devise::Test:Module`
Where is the rails5? method defined? I can't find it.
Thanks!

@sbleon Yeah, this method is defined in our test suite's rails app and it shouldn't be used outside of it. I've already opened a PR to fix it: https://github.com/plataformatec/devise/pull/4809

Thanks for the explanation and the link, Leonardo!

Leon