Major syntax updates, PEP8 Compliance, enhancement suggestions, and proposed changes!

Suggestions

The modules.py file in each package is very redundant, users can simply do the following:

from pygorithm import string
help(string)

# Outputs
"""
Help on package pygorithm.string in pygorithm:

NAME
   pygorithm.string - Collection of string methods and functions

PACKAGE CONTENTS
   anagram
   isogram
   modules
   palindrome
   pangram

DATA
   __all__ = ['anagram', 'pangram', 'isogram', 'palindrome']

FILE
   c:\users\doarni\dev\pygorithm\pygorithm\string\__init__.py
"""

The get_code() functions can be removed from all files, python is not compiled so people can easily open the files up and view their source code.
It is extremely redundant importing inspect every time, instead the user can import that themselves and it will still act the same, it is not necessary to be in each .py file.

Example:
from pygorithm.string import palindrome
import inspect

print(inspect.getsource(palindrome))

The time_complexities() function can also be removed, instead this can be included in each module/.py docstrings or in the functions docstrings. When the user calls help() on the function, it will explain how they work, and give them the time complexities.

A sort of function to time some algorithms? We could incorperate a decorator package, users could decorate their functions with them, and see how effeciant each are!

Use function annotations to annotate inputs, and use a custom decorator so when a user calls methods, we can make sure their input is correct, as in checking that the input is a str, int, float, to make python act more like other languages since by default python does not check input parameters. Though this could make the source code for the functions more confusing for newcommers to python! More information: https://code.tutsplus.com/tutorials/python-3-function-annotations--cms-25689

package string should be changed to strings. string is a default python module, having a package shadow builtin/default modules can cause problems.

Decide whether or not to support Python 2.7. heap.py uses super() which is a Python 3+ builtin. Something can be added to the main init.py to check for the version thats being run. I've included from __future__ import division in some of the modules since I see that // has been used and Python 3+ you only need /. If you want to support Python 2.7 you or someone will need to rewrite heao.py!

Updates

data_structures

Updated init.py
graph.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Removed redundant import statments in methods
Updated syntaxing for PEP8 compliance
Updated conditional sytaxing
Removed camel casing from variable and parameter names
Updated _union() and _set_of() in WeightedGraph() class to make them private
Added self.forest to init() or WeightedGraph() class
Updated _topological_sort_rec() in TopologicalSort() class to make it private
Updated _check_cycle_rec() in CheckCycleDirectedGraph() class to make it private
Updated _check_cycle_rec() in CheckCycleUndirectedGraph() class to make it private

queue.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Removed redundant import statments in methods

linked_list.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Removed redundant import statments in methods
Updated conditionals and syntaxing
Removed redundant parenthesis around conditionals
Removed camel casing on variables

heap.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Removed redundant import statments in methods
Updated parent_idx, left_child_idx, right_child_idx, time_complexities to be static
removed unused variable 'ROOT' from heapify_down()

stack.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Removed redundant import statments in methods
Removed camel casing on methods and variables
Updated get_code() in Stack(object) to be static
Updated _isOperand(), _precedence() in InfixToPostfix() to be static and private
Updated get_code() to be static
Updated method _isOperand()
Removed redundant parenthesis around conditionals

tree.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Removed redundant import statments in methods
Removed redundant parenthesis around conditionals
Updated min_val_bst_node(), get_code()
Removed camel casing on variables and parameter names

string

anagram.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Fixed indentations in is_anagram()
Updated syntaxing for PEP8 compliance

palindrome.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Updated syntaxing for PEP8 compliance
Optimized code of is_palindrome

isogram.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Updated syntaxing for PEP8 compliance
Optimized code of is_isogram

pangram.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Updated syntaxing for PEP8 compliance
Optimized code in is_pangram

fibonacci

generator.py

Updated syntaxing for PEP8 compliance
Updated parameter name on nested function sequence(), it was shadowing outer functions parameter name

goldenratio.py

Updated syntaxing for PEP8 compliance
Functions no longer nested

math

lcm.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Updated syntaxing for PEP8 compliance
Updated code to prevent variables from shadowing method names

lcm_using_gcd.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Updated syntaxing for PEP8 compliance
Updated code to remove camel casing on parameter names

sieve_of_eratosthenesd.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Updated syntaxing for PEP8 compliance
Removed redundant parenthesis in conditionals
Optimized conditionals

searching

binary_search.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Updated syntaxing for PEP8 compliance
Updated code in binary_search

breadth_first_search.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Updated syntaxing for PEP8 compliance

depth_fist_seach.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Updated syntaxing for PEP8 compliance
Updated code in depth_first_search

linear_search.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Updated syntaxing for PEP8 compliance
Removed camel casing from variable and parameter names

quick_select.py

Updated docstrings at top of file
Updated docstrings for functions
Optimized import statements
Updated syntaxing for PEP8 compliance
moved nested functions out of search()

After #38 issue resolution All unit tests pass. Thanks MrDupin!!!
test_data_structures
   .
----------------------------------------------------------------------
Ran 1 test in 0.002s

OK

test_fibonacci
.
----------------------------------------------------------------------
Ran 1 test in 0.000s

OK

test_math
...
----------------------------------------------------------------------
Ran 3 tests in 0.002s

OK

test_searching
....
----------------------------------------------------------------------
Ran 4 tests in 0.004s

OK

test_sorting
.........
----------------------------------------------------------------------
Ran 9 tests in 0.009s

OK


chore(DTFS2-6996): remove moment from dtfs-central-api

Introduction :pencil2:
Moment js is deprecated, it should be replaced.
Resolution :heavy_check_mark:

Replace moment with date-fns to convert a search string into a date when querying dateReceived

This covers code originally written under DTFS2-3065. We've had to agree new ACs for this behaviour but have tried to maintain, within reason, the same behaviour

Codecov Report
Attention: Patch coverage is 77.27273% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 21.58%. Comparing base (fbe7ec4) to head (569ed84).
Report is 80 commits behind head on main.

Files
Patch %
Lines

...1/controllers/tfm/deal/tfm-get-deals.controller.js
0.00%
4 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2805      +/-   ##
==========================================
- Coverage   22.57%   21.58%   -0.99%
==========================================
  Files        1136      726     -410
  Lines       27255    16547   -10708
  Branches     4486     2332    -2154
==========================================
- Hits         6153     3572    -2581
+ Misses      18388    11663    -6725
+ Partials     2714     1312    -1402

Flag
Coverage Δ

api-test
0.48% <ø> (ø)

ui-test
?

unit-test
23.61% <77.27%> (+18.97%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Symbol name collision with pkcs11.h identifier CKM_RSA_AES_KEY_WRAP

Describe the bug
Building an application that includes both sdk/keyvault/azure-security-keyvault-keys/inc/azure/keyvault/keys/key_client_models.hpp and OASIS pkcs11.h fails due to both defining a symbol named CKM_RSA_AES_KEY_WRAP. Namespacing is irrelevant here, as pkcs11.h defines a macro.
Exception or Stack Trace
gcc compiler error:
In file included from /workspaces/azure_kms_pkcs11/pkcs11_compat.h:12,
                 from /workspaces/azure_kms_pkcs11/attributes.h:1,
                 from /workspaces/azure_kms_pkcs11/aws_kms_pkcs11.cpp:11:
/usr/include/p11-kit-1/p11-kit/pkcs11.h:802:42: error: expected unqualified-id before numeric constant
  802 | #define CKM_RSA_AES_KEY_WRAP            (0x1054UL)
      |                                          ^~~~~~~~
/usr/include/p11-kit-1/p11-kit/pkcs11.h:802:42: error: expected ')' before numeric constant
  802 | #define CKM_RSA_AES_KEY_WRAP            (0x1054UL)
      |                                         ~^~~~~~~~

To Reproduce
apt-get install gcc g++ p11-kit libp11-kit-dev
g++ -c snippet.cpp -I/usr/include/p11-kit-1/p11-kit -I<path to azure-sdk-for-cpp>

Code Snippet
Add the code snippet that causes the issue.
#include <pkcs11.h>
#include <azure/keyvault/keys/key_client_models.hpp>

Expected behavior
pkcs11 and Azure sdk can be used in the same compilation unit.
Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):

OS: Ubuntu 22.04
IDE : no
Version of the Library used: main bd08d226e1726825ee43cff219b3cba45126910d

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

Thank you for submitting this issue. Looking into it .

as a temporary workaround reverse the includes
#include <azure/keyvault/keys/key_client_models.hpp>
#include <pkcs11.h>
Also you might need to call the macro as such KeyEncryptionAlgorithm::(CKM_RSA_AES_KEY_WRAP) – taking macro in braces disables macro expansion.


Add installation instructions to README

The library title on Github is big_O, but one needs pip install big-O to install it. This ambiguity can be easily solved by including a short installation section to the README. I will try to do it later but in case someone wants to do it before then.

can I be assigned this?

https://github.com/pberkes/big_O/pull/53


Namespace omitted for comma-joined selectors in some scenarios

Given the following:
div {
  display: flex;

  &.foo,
  p:not(:last-child) {
    background: red;
  }
}

The output is currently:
[namespace] div {
	display: -webkit-box;
	display: -webkit-flex;
	display: -ms-flexbox;
	display: flex;
}

[namespace] div.foo,p:not(:last-child) {
	background: red;
}

But it should be:
[namespace] div {
	display: -webkit-box;
	display: -webkit-flex;
	display: -ms-flexbox;
	display: flex;
}

[namespace] div.foo,[namespace] p:not(:last-child) {
	background: red;
}

Note the lack of [namespace] for the second selector in the comma chain.

This is a regression from: https://github.com/thysultan/stylis/commit/ccb001641bead49b660344956252d1a0dd4d38f1

I think the bug is at least as old as 4.x just based on testing with this example repo https://github.com/arackaf/styled-components-repro (I'm using package.json overrides to force particular versions.)


fix isLoading issue when updating records - fix #39

@lATAl could you check this over for me and see if it makes sense/has no errors?
I suppose this is a breaking change, but perhaps in another sense it's a revision of one of the breaking changes for 5.0.0. I think it would be best to backport this and release it as 5.0.2 and 5.1.1

@devvmh I recently update to 5.1.1 and everything's going right, found no error.


[decorators] Removed big command/function dictionary

Removed big command/function dictionary, and instead added command decorator.

Forgot to mention - I changed !help to use python built-in documentation functions.
If commands function begins with """!command : help text", it's added to !help command

added missing readme, fixed biki and f3wiki I broke. Another PR coming.


Expose buildUserAgent as protected

fix #7549

@weidongxu-microsoft @ChenTanyi
Following up to get a resolution on this change proposal. Will this logic be copied to where it is needed or should we begin discussing another solution?

@alzimmermsft yeah, we have copied it to where we need it. If you think it is better than use it from base class, I'll close this PR. Is it possible for the function to be public?


CC app is not working

it's been a while company communicator app is not working & I tried the client & browser same result

hi @peddivyshnavi
no it an old deployment
FYI

@i7asko ,
Could you please share the call stack of error from application insights resource from the deployed resource group.
Go to resource group where CC app is deployed --> Go to Application insight type resource --> Go to Failures section from Left hand side menu --> share the screenshots and stack trace for Top 3 response codes.

hi @v-jaygupta
FYI

Hi @i7asko
By checking screenshots provided we see its pointing to different repo. Please note, that we are only able to troubleshoot versions that are from our main repo. Even though they may be using out of the box features, the fact that they are indeed running a customized version means there are other variables present and the team is unable to dive into all the potential differences between our version vs custom changes. We will be unable to further troubleshoot unless it is the main repo / non customized version.
If we can be of any help for future issues, please do continue and engage. Hope you understand out limitations as it is with our inability to support customized versions (not just for this app template, but for any of the app templates).

@i7asko
Could you please update us on the status of the issue?

hi @peddivyshnavi
please allow me some im checking with Microsoft to figure out , our current version pulled from which repo & I'll update you ASAP

Hi @i7asko
Could you please update us on the status of the issue?

hi @peddivyshnavi , thanks for your follow up
i still have communication & logs going on with Microsoft  as of now  i don't know what the next action plan

@i7asko
Could you please update us on the status of the issue?

@i7asko
Could you please update us on the status of the issue?


Question about dense re-feeding.

In dense re-feeding what do we show as output to model. For example let's say B is our training batch. For model input X and label y both will be equal to B. Let's say model gives y_hat as output for our batch B. Now, in dense re-feeding we give this y_hat as input to model. Now my question is what will be the labels in dense re-feeding? Will it be y_hat or is it B?

"For model input X and label y both will be equal to B"

to be more precise, B=[X,y]
"Let's say model gives y_hat as output for our batch B"
the model (f) consumes only X from B, so y_hat=f(x) and we compute loss between y_hat and y
"Now my question is what will be the labels in dense re-feeding? Will it be y_hat or is it B?"
In dense re-feeding we feed y_hat again and get y_hat2=f(y_hat), the labels remain the same - y and we compute loss between y_hat2 and y.

Thanks @okuchaiev

@okuchaiev Why do we do Iterative Dense refeeding?


adds the navbar fixed to the bottom n navstate

also adds the homepage element for the safari white screen


consider allowing to specify the exact file instead of a folder that has a tsconfig.json

seriously, each tsconfig.json has to have its own folder in order to be referenced with --project parameter

What if I need 2 projects based on the same set of files?

This would be really nice.

Behold, ask and ye shall receive a ~2 month old pull request from @SaschaNaz which incredibly has absolutely no merge conflicts right now: #4883
Also that sort of makes this a duplicate of #2869, though I'm leaving this open for the sake of discussion.

Since --project is taken already I suggest the --configuration parameter

this is a duplicate of  #2869. we should revive  #2869 as well as #4883


create new branch for pull request.

message to remind contributors to create   branch for pull request

resolves #issueNumber
Checklist for profile addition

[ ] Name the PR fix: add <your-username>.
[ ] You've added file named your-username.json in the profiles directory.
[ ] You've linked the PR to the correct issue.

Other additions/changes

[ ] Changes made in this PR are mentioned below
[ ] Search previous suggestions before making a new one, as yours may be a duplicate.
[ ] I acknowledge that all my contributions will be made under the project's license.
[ ] I've read the contributing guidelines

This PR adds/fixes

Notes

Hey, the docs are clear. We don't need these changes. Thank you

Thanks for reviewing 👍


Chart improvements

Chart

[x] baseline support

Value guaranteed to be visible in domain (x/y).  Useful with optional negative values since xDomain={[0, null]} would ignore negative values

[x] TransformContext integration
[x] GeoContext integration
[x] TooltipContext integration
[ ] Bounds integration

Modify xScale/yScale along with using motionScale with tweening
Useful with Brush

[ ] Better group/stack integration

see use of createDimensionGetter()
Would be nice if could have derived scales (x1scale, y1scale) needed for group scales

[ ] Dedicated color scale instead of using rScale (which is also useful for radius, etc)

Layout components

Canvas

[x] Full width/height of container (like Svg)
[x] scaleCanvas() integration
[x] TransformContext integration
[x] class support

Svg

[x] TransformContext integration
[x] class support

Html

[x] class support

@mhkeller gathering some changes I've made (or plan to) within LayerChart, some of which might be worth pushing upstream to LayerCake (such as full width Canvas).  Integration with LayerChart contexts like TransformContext, GeoContext, and TooltipContext makes some of these changes impractical to move ATM.

Thanks I'll keep an eye on it. What does full width Canvas do?

If you applied a padding (such as <Chart padding={{ top: 60 }}>) and zoom or translate the canvas, the chart would be "clipped" since the <canvas> is absolute positioned and inset based on the padding.  The change uses context.translate() and the full container size (which aligns the results with  <svg transform="...">)
The Choropleth example demonstrated this (compare to an older commit preview).

Before
After

You have to scale/zoom or drag/translate to see the issue (not really an issue with most static charts).  This would be a problem with more chart types, but I've mostly only focused on Canvas support for geo (since it typically benefits the most with complex geojson).   Also hoping to find a fix for the infinite loop issue before I expand out too much (might wait til Svelte 5).


add bench_gen_env, and use it internally in bench_env

The only thing I see here that you might object to is that I changed bench_env to use the new bench_gen_env internally.  This reduces the amount of new code, so I think it's a good thing.  The compiler should be able to optimize the closure such that it's as fast as the old version, and in any case the generation of the input is not timed.

That’s exactly what I had in mind. Will merge when I get home.

Merged and released in 1.0.0.  Thanks!


Identify content rules that can be checked automatically

It is a huge waste of time getting authors and approvers to check for particular words, spellings and spaces around punctuation when a machine can do it seconds. Our dictionary could also be made available to other agencies to use in their checkers.

Part of GOV.AU's work.


Gamlerhart Last Update: Disabling GC

Check List:

[x] You have run ./mvnw verify and the project builds successfully
[x] Tests pass (./test.sh <username> shows no differences between expected and actual outputs)
[x] All formatting changes by the build are committed
[x] Your launch script is named calculate_average_<username>.sh (make sure to match casing of your GH user name) and is executable
[x Output matches that of calculate_average_baseline.sh
[x For new entries, or after substantial changes: When implementing custom hash structures, please point to where you deal with hash collisions (line number)

Execution time: < 5 seconds
Execution time before: 5 + seconds

Disabling the GC gave another small performance boost.
I experimented with other things, but failed to make it faster. I now would have to study more from other solutions =).

I stop myself now =).
Thanks again for the learning experience. I've summarized my leanings here: https://www.gamlor.info/posts-output/2024-01-12-one-billion-row-challenge/en/

Runs out of heap space for the 10K keyset on 32 cores (see create_measurements3.sh):
Terminating due to java.lang.OutOfMemoryError: Java heap space

Aha...Tried to simulate at 32/64thread machine. Should pass the 10K set now =).

Have you tried with 10K keys? It shows differences:
+ timeout -v 300 ./test.sh gamlerhart measurements_10K_1B.txt
Validating calculate_average_gamlerhart.sh -- measurements_10K_1B.txt
WARNING: Using incubator modules: jdk.incubator.vector
227c227
< Burl;-20.6;9.0;48.2
---
> Burl;-20.6;8.6;37.9
376c376
< Edmo;-17.5;10.7;40.4
---
> Edmo;-17.5;10.6;40.4
700c700
< Lian;-17.4;12.6;44.2
---
> Lian;-15.3;13.1;44.2
871c871
< Noga;-17.8;18.6;49.1
---
> Noga;-10.1;19.9;49.1
1306c1306
< Y;-14.2;15.4;44.6
---
> Y;-14.2;15.4;44.7
1315c1315
< Yogy;-16.5;14.8;47.4
---
> Yogy;-16.5;14.7;47.4
1326c1326
< Zagr;-15.7;14.2;45.8
---
> Zagr;-15.7;14.0;43.2
1769,1770c1769,1770
< afang;-19.7;10.3;38.1
< afangd;-17.2;13.5;48.1
---
> afang;-19.7;10.3;41.5
> afangd;-12.7;15.5;48.1
1890c1890
< akaJ;-18.3;14.4;47.8
---
> akaJ;-13.3;15.6;47.8
2011c2011
< am;-9.5;19.1;47.8
---
> am;-9.5;19.1;50.1
2021c2021
< amPūnchLaurelTavernySui;-19.8;12.6;50.1
---
> amPūnchLaurelTavernySui;-19.8;11.0;41.1
2129c2129
< anSlidellBayan HotGreenburghShibat;-17.4;15.5;47.5
---
> anSlidellBayan HotGreenburghShibat;-17.4;15.3;47.5
2688c2688
< b;-14.8;13.5;42.4
---
> b;-16.5;13.5;43.4
2791c2791
< bovyanColcapirhuaBagneuxClondalkinNorth BethesdaAzemmourSuttonGbadoliteWest Bridgfor;-16.5;18.7;49.7
---
> bovyanColcapirhuaBagneuxClondalkinNorth BethesdaAzemmourSuttonGbadoliteWest Bridgfor;-11.2;19.2;49.7
3128c3128
< dmaRagusaLebanonSomer;-19.7;16.8;47.3
---
> dmaRagusaLebanonSomer;-12.9;17.1;47.3
3311c3311
< eMasrakhBiarritzR;-14.9;13.4;43.5
---
> eMasrakhBiarritzR;-14.9;13.6;43.5
3324c3324
< ePes;-21.6;9.2;46.2
---
> ePes;-21.6;8.9;36.5
3334c3334
< eRangsitBeledw;-14.7;15.2;43.2
---
> eRangsitBeledw;-14.7;15.3;43.2
3752c3752
< esJa;-12.7;17.0;46.7
---
> esJa;-12.7;16.8;45.5
4175c4175
< hHng;-19.4;13.6;50.6
---
> hHng;-19.4;12.6;41.5
4455c4455
< i;-13.7;16.6;46.7
---
> i;-14.4;16.6;46.7
4838c4838
< inUm;-18.0;15.0;44.0
---
> inUm;-18.0;14.7;41.9
5177c5177
< k;-9.3;19.1;47.7
---
> k;-9.3;19.1;49.9
5247c5247
< kbyC;-19.0;12.8;49.9
---
> kbyC;-19.0;12.2;45.5
5338c5338
< l;-9.3;19.8;49.4
---
> l;-9.3;19.8;49.8
5462c5462
< landsIsioloAnte;-21.4;8.8;41.2
---
> landsIsioloAnte;-21.4;8.7;41.2
5718,5719c5718,5719
< lpr;-20.1;14.0;49.8
< lpu;-14.2;17.2;49.0
---
> lpr;-20.1;11.6;40.2
> lpu;-14.2;17.0;45.7
6090,6091c6090,6091
< nMach;-17.4;14.6;45.4
< nMachengAkur;-14.3;18.6;49.5
---
> nMach;-17.4;14.6;48.7
> nMachengAkur;-9.1;19.8;49.5
6168c6168
< na;-13.9;15.8;44.4
---
> na;-14.0;15.8;44.4
6181c6181
< naSo;-22.9;8.5;39.0
---
> naSo;-22.9;8.3;37.5
6295c6295
< neSoissonsMirganjAïn;-17.0;14.5;48.8
---
> neSoissonsMirganjAïn;-14.8;14.7;48.8
6884c6884
< olSarıyerSumqayıtŌts;-20.4;9.9;41.3
---
> olSarıyerSumqayıtŌts;-20.4;9.6;41.3
7083,7084c7083,7084
< org;-9.6;20.0;48.5
< orge;-13.3;19.3;49.1
---
> org;-9.6;20.0;48.6
> orge;-13.3;19.1;49.1
7186c7186
< ouTây Ni;-14.5;19.0;48.3
---
> ouTây Ni;-13.7;19.6;48.3
7511c7511
< raS;-18.6;11.4;41.1
---
> raS;-18.6;11.3;41.1
7598c7598
< rdh;-21.1;12.7;44.1
---
> rdh;-21.1;12.6;42.6
7600c7600
< rdu;-23.9;10.3;44.9
---
> rdu;-23.9;9.8;38.3
7603c7603
< re;-8.7;19.9;47.4
---
> re;-10.3;19.9;50.1
7616c7616
< ree;-18.9;14.7;46.3
---
> ree;-18.9;14.2;43.3
7622c7622
< ren;-12.9;19.2;51.7
---
> ren;-12.9;19.0;51.7
7629c7629
< rer;-16.0;14.4;44.9
---
> rer;-16.0;14.3;42.2
7667c7667
< rha;-23.5;7.4;43.6
---
> rha;-23.5;7.0;33.7
7797c7797
< roPointe-à-PitreKuala TerengganuCampecheByatarayanpurMaw;-11.7;17.2;47.6
---
> roPointe-à-PitreKuala TerengganuCampecheByatarayanpurMaw;-11.7;17.4;47.6
7998c7998
< sCa;-15.0;19.5;51.7
---
> sCa;-9.6;19.9;51.7
8009c8009
< sFa;-13.0;19.1;49.6
---
> sFa;-13.0;19.0;49.6
8105c8105
< seLikasiDengtacunLembokZhijiangChengjiaoBeipiaoSuoluntunStaten IslandKota BharuCiudad LópezM;-15.3;17.9;47.8
---
> seLikasiDengtacunLembokZhijiangChengjiaoBeipiaoSuoluntunStaten IslandKota BharuCiudad LópezM;-9.6;18.0;47.8
8858c8858
< uenR;-15.9;14.2;47.1
---
> uenR;-15.9;14.3;47.1
9007c9007
< upaLji;-15.2;16.6;51.3
---
> upaLji;-15.2;16.9;51.3
9047c9047
< uraCiudad del Car;-14.6;14.7;45.8
---
> uraCiudad del Car;-14.6;14.2;44.0
9051c9051
< uramCa;-24.7;9.0;43.3
---
> uramCa;-24.7;8.5;43.3
9542c9542
< zinoAcámbaroMaisons-AlfortLechangBolgatangaMa;-10.6;17.6;47.5
---
> zinoAcámbaroMaisons-AlfortLechangBolgatangaMa;-10.6;17.7;47.5
9732c9732
< ābādBa;-11.9;18.8;49.0
---
> ābādBa;-11.9;19.1;49.0

FAILURE: ./test.sh gamlerhart measurements_10K_1B.txt failed

Thanks for the command. I wasn't aware of the new 10k test set.  I hand a bug indeed. Should be finally fixed.
The performance on the 10k 1billion set is bad =). Mostly likely because I was lazy in the comparison function and did not vectorize it for larger sets. Or my horrible hash function produces to many collisions on large keys.
I probably will leave it be at this anyway.
Thanks for all the effort.

Thanks for the command. I wasn't aware of the new 10k test set. I hand a bug indeed. Should be finally fixed.
The performance on the 10k 1billion set is bad =). 50+ seconds on my machine. Mostly likely because I was lazy in the comparison function and did not vectorize it for larger key. And my horrible hash may produce to many collisions with more keys as well =)
Anyway, I don't plan to fix that and stop myself here.
Thanks for all the effort.

#
Result (m:s.ms)
Implementation
JDK
Submitter
Notes

00:04.474
link
21.0.1-open
Roman Stoffel

Nice!

Passes the 10K test too.


Don't try to add extras to null objects.

Surprisingly, typeof null === 'object'.  I modified CesiumTexturedBoxTestAddExtras.gltf to trigger the bug fixed by this PR.  I realize it's kinda dodgy glTF, but I ran into it in a real model.

Does the model pass validation?  (Note that is written against glTF 2.0, not 1.0, so it might not pass for other reasons).
If it doesn't pass, we don't want to put workarounds in gltf-pipeline unless it is to workaround a super popular exporter.  It will hurt the overall glTF ecosystem.
If this change is still needed, the only tweak I suggest is:

I modified CesiumTexturedBoxTestAddExtras.gltf to trigger the bug fixed by this PR...

Instead of changing the file, add a new test that adds the object with null properties.

It was created by obj2gltf.  I don't know if there's something weird about my OBJ that caused it.
I think the change is good one even the situation is unlikely.  It's pretty clear the author of the code here just didn't realize that typeof null === 'object' (cause that's totally weird).  I'll put it in a separate .gltf, but it's worth mentioning that that gltf is already invalid.  It's missing accessors, for one thing.

Can you submit a bug or pull request to obj2gltf?
I still need to see a case where this matters in practice on valid glTF; otherwise, I'm not sure that this is a good change, it is the kind of "fix" that creates unreliable ecosystems where workarounds are required to conform to the unwritten spec.

it is the kind of "fix" that creates unreliable ecosystems where workarounds are required to conform to the unwritten spec.

Yeah, maybe.  Though notice that in that function there are three places where a new object is pushed onto objectStack.  In two of the three, there's a defined check in addition to the typeof x === 'object' check.  My PR adds it to the third.  It's not clear to me why the extra check is justified in two cases but not the third.  Also, this model loaded fine prior to https://github.com/AnalyticalGraphicsInc/gltf-pipeline/commit/68d3849b143f3687b7ee53cc05e2b0f7ab2361ef.  Was this "tightening" really intended in a cleanup PR?
Either way, I agree it's worth a minute to figure out why obj2gltf is generating this and I'll do so when I get a chance.

Let me state it another way.  As currently written, before this PR, a null as a value of property is fine, whether the spec allows nulls there or not.  The model will load just fine.  But if there's a null inside an array, attempting to load the model will crash with a null reference exception.  You're welcome to close this PR if you disagree, but to me that behavior is pretty hard to defend.

I found the source of the NaNs, in case it matters.  My OBJ has a few normals that are the zero vector.  Maybe they're not used so they don't matter, maybe it's a problem with the original model, I'm not sure.  In any case, obj2gltf normalizes the normals during the conversion process (why? I don't see anything in the glTF spec that requires that normals be normalized), which produces NaN.  That causes the min and max range of the normal attribute to be NaN, and JSON.stringify serializes NaN as null.

This change seems fine to me and just seems like an oversight as all the other areas that check typeof item === 'object' in addPipelineExtras and removePipelineExtras first check that its defined.
Also thanks for tracking down the root cause in obj2gltf.

Sorry folks, have had bigger fish to fry.  If you guys think this is inline with the spec, then go for it.  If the spec needs work, then please submit an issue to https://github.com/KhronosGroup/glTF/.
As for requiring normals to be normalized, see https://github.com/KhronosGroup/glTF/issues/812.  In practice, most people would want to do this to save runtime processing.


Python 3 compatibility: basestring, unicode(), xrange()

Fixes for 30 flake8 errors on Python 3...
flake8 testing of https://github.com/kensho-technologies/graphql-compiler on Python 3.6.2
$ flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics
./graphql_compiler/debugging_utils.py:19:18: F821 undefined name 'xrange'
        for i in xrange(0, len(too_many_parts) - 1, 2)
                 ^

./graphql_compiler/debugging_utils.py:24:14: F821 undefined name 'xrange'
    for i in xrange(1, len(parts)):
             ^

./graphql_compiler/debugging_utils.py:87:26: F821 undefined name 'xrange'
                for i in xrange(1, len(separate_keywords) - 1, 2):
                         ^

./graphql_compiler/compiler/blocks.py:55:35: F821 undefined name 'basestring'
                all(isinstance(x, basestring) for x in self.start_class)):
                                  ^

./graphql_compiler/compiler/blocks.py:98:35: F821 undefined name 'basestring'
                all(isinstance(x, basestring) for x in self.target_class)):
                                  ^

./graphql_compiler/compiler/blocks.py:256:43: F821 undefined name 'basestring'
        if not isinstance(self.direction, basestring):
                                          ^

./graphql_compiler/compiler/blocks.py:320:43: F821 undefined name 'basestring'
        if not isinstance(self.direction, basestring):
                                          ^

./graphql_compiler/compiler/blocks.py:347:22: F821 undefined name 'xrange'
            for i in xrange(self.depth + 1)
                     ^

./graphql_compiler/compiler/expressions.py:78:35: F821 undefined name 'basestring'
        if isinstance(self.value, basestring):
                                  ^

./graphql_compiler/compiler/expressions.py:102:37: F821 undefined name 'basestring'
        elif isinstance(self.value, basestring):
                                    ^

./graphql_compiler/compiler/expressions.py:107:36: F821 undefined name 'basestring'
            elif all(isinstance(x, basestring) for x in self.value):
                                   ^

./graphql_compiler/compiler/expressions.py:182:41: F821 undefined name 'unicode'
        match_variable_name = '{%s}' % (unicode(variable_with_no_dollar_sign),)
                                        ^

./graphql_compiler/compiler/expressions.py:206:20: F821 undefined name 'unicode'
            return unicode(self.variable_name)
                   ^

./graphql_compiler/compiler/expressions.py:237:16: F821 undefined name 'unicode'
        return unicode(self.field_name)
               ^

./graphql_compiler/compiler/expressions.py:619:42: F821 undefined name 'unicode'
        if not isinstance(self.operator, unicode):
                                         ^

./graphql_compiler/compiler/helpers.py:10:36: F821 undefined name 'unicode'
VARIABLE_ALLOWED_CHARS = frozenset(unicode(string.ascii_letters + string.digits + '_'))
                                   ^

./graphql_compiler/compiler/helpers.py:60:30: F821 undefined name 'basestring'
    if not isinstance(value, basestring):
                             ^

./graphql_compiler/compiler/helpers.py:62:12: F821 undefined name 'unicode'
    return unicode(value)
           ^

./graphql_compiler/compiler/helpers.py:100:30: F821 undefined name 'basestring'
    if not isinstance(value, basestring):
                             ^

./graphql_compiler/compiler/helpers.py:158:44: F821 undefined name 'basestring'
        if field and not isinstance(field, basestring):
                                           ^

./graphql_compiler/compiler/helpers.py:185:34: F821 undefined name 'basestring'
        if not isinstance(child, basestring):
                                 ^

./graphql_compiler/compiler/helpers.py:199:60: F821 undefined name 'unicode'
        mark_name = u'__'.join(self.query_path) + u'___' + unicode(self.visit_counter)
                                                           ^

./graphql_compiler/query_formatting/gremlin_formatting.py:19:30: F821 undefined name 'basestring'
    if not isinstance(value, basestring):
                             ^

./graphql_compiler/query_formatting/gremlin_formatting.py:89:43: F821 undefined name 'basestring'
        if not isinstance(argument_value, basestring):
                                          ^

./graphql_compiler/query_formatting/gremlin_formatting.py:90:30: F821 undefined name 'unicode'
            argument_value = unicode(argument_value)
                             ^

./graphql_compiler/query_formatting/match_formatting.py:18:30: F821 undefined name 'basestring'
    if not isinstance(value, basestring):
                             ^

./graphql_compiler/query_formatting/match_formatting.py:75:43: F821 undefined name 'basestring'
        if not isinstance(argument_value, basestring):
                                          ^

./graphql_compiler/query_formatting/match_formatting.py:76:30: F821 undefined name 'unicode'
            argument_value = unicode(argument_value)
                             ^

./graphql_compiler/tests/test_helpers.py:37:14: F821 undefined name 'xrange'
    for i in xrange(len(expected_blocks)):
             ^

./graphql_compiler/tests/test_ir_generation_errors.py:441:48: F821 undefined name 'xrange'
            args = (variable_names[i] for i in xrange(num_args))
                                               ^

Hey @cclauss, thank you for the PR and for helping make this project Python 3 compatible!
To avoid duplicating the try-except-NameError block at the top of many files, what do you think about having a single top-level compat.py file that has all the Python version compatibility things like that, and then import it wherever they are necessary?
Otherwise, looks good to me. Reminder to please sign the CLA so I can merge this when it's ready -- the link is the Details part of the clahub merge check.

Might be more maintainable to use
from six.moves import xrange
from six import string_types as basestring

instead of special casing!

@ColCarroll I assume there's also something in six that works around the unicode vs str problems too? I'm open to adding six as a dependency if it helps make this easier.

Ah yeah, looks like from six import text_type as unicode.  I like the idea of putting these in compat.py in case it needs to change, or there is more logic to be added later.  flask actually calls it _compat so it does not make the top level namespace any busier.  Good comments there:
https://github.com/pallets/flask/blob/master/flask/_compat.py

From flask:
Some py2/py3 compatibility support based on a stripped down
version of six so we don't have to depend on a specific version of it.

Since we're only using like ~3 things, I think we can afford to write our own tiny compat.py file instead of depending on six. Thoughts?

Yah -- seems like it would be like ~8 lines.  What are your thoughts on using text_type and string_types instead of unicode and basestring?

I think that's fine and probably preferable -- it'll also silence pylint. I'm also willing to help if @cclauss wants some help.

Closed in favor of #17


Creating a new projects on ubuntu 18.04 takes about 10 hours

Version
3.9.2
Environment info
vue create some-project

Steps to reproduce
vue create some-project
What is expected?
project creates
What is actually happening?
the project creates in 10 hours

Could you please provide more context on this issue?
Which options did you choose? At what phase was the command stuck? Any console outputs?

I found out that the reason of this issue was in npm. When I tried to use yarn everything became good. Thank you


Multiplatform setup

This change tries to make the WelcomeBot setup possible on more than one platform.
Tasks:

[x] improve setup.sh

[x] fix typo: git subodule -> git submodule
[x] get rid of sudo

[x] create setup.bat
[x] add a mention about setup to README.md

@Kubo2 I'll wait until you are finished with everything to merge.

@michaelpri10 Okay, if you have any comments about the patch, feel free to let them here.

Hey guys, I have just pushed a new version of setup.bat: Kubo2/WelcomeBot@5aff5c9c7293b2fb3165ebcdbed7df9709926ccf
I will mark it as finished in my tasklist above, but if you have any comments, please feel free to let them!
I am going to supplement README.md and then this can be finally merged.

This is now ready to be merged.
/cc @michaelpri10 @Jacob-Gray @ProgramFOX Please review this.

ping @michaelpri10

Hey guys, is here any problem with this PR? Please let me know if so, I wil try to fix it.

I don't speak Batch or Shell so I cannot tell anything about it, sorry.


Error processing request

**Is your feature request related to a problem? problem showing "There was an error processing your request: No details were returned
"
in the console its showing "hook.js:608
Warning: Encountered two children with the same key, cohere/command. Keys should be unique so that components maintain their identity across updates. Non-unique keys may cause children to be duplicated and/or omitted — the behavior is unsupported and could change in a future version. Error Component Stack

i added a .env file and then ran it but still...
Describe the solution you'd like:

Describe alternatives you've considered:

Additional context:

same issue

I'm also experiencing the same issue. I performed a clean install on the machine, and even installed a brand new instance of the app on another computer!  Any help would be gratefully appreciated. Thanks.

Hello.
Personnaly I had to create a new model with different name :
Ex:  ollama create -f ./Modelfile qwen2.5-coder:7b-32k
And use this model to get it work qwen2.5-coder:7b-32k
Hope it could help

Thanks, but I have the same error.

Just by the way you must keep the two models (original and 32k version)
What is your error code from browser?

I have a 500 error and I have kept the previous models

I solved the problem by changing the model

Where is running your ollama stack ? In docker ? Same VM PC MAC ?

I'm having the same issue, tried using OpenRouter and ChatGPT as providers  but the same error occurs

Please try with the latest vanilla version of this fork


Prelude for custom widgets

Writing a custom widget involves a lot of boilerplate. It would be cool if there was a prelude for the imports that the Widget trait needs.
For reference, this is the snippet I'm using to create custom widgets.
use druid::{
	BoxConstraints, Env, Event, EventCtx, LayoutCtx, LifeCycle, LifeCycleCtx, PaintCtx,
	RenderContext, Size, UpdateCtx, Widget,
};

pub struct ${1:WidgetStruct} {
	${2}
}

impl Widget<${3:Data}> for $1 {
	fn event(&mut self, _ctx: &mut EventCtx, _event: &Event, _data: &mut $3, _env: &Env) {
	}

	fn lifecycle(&mut self, _ctx: &mut LifeCycleCtx, _event: &LifeCycle, _data: &$3, _env: &Env) {
	}

	fn update(&mut self, _ctx: &mut UpdateCtx, _old_data: &$3, _data: &$3, _env: &Env) {
	}

	fn layout(&mut self, _ctx: &mut LayoutCtx, bc: &BoxConstraints, _data: &$3, _env: &Env) -> Size {
		bc.max()
	}

	fn paint(&mut self, _ctx: &mut PaintCtx, _data: &$3, _env: &Env) {

	}
}

@emigr2k1 okay, added a widget prelude. If you wanted to update some of our examples to make use of it, those PRs would be welcome!


⚠ Jenkins Plugins Artifactory has degraded performance

In 7ed6189, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1098 ms

Resolved: Jenkins Plugins Artifactory performance has improved in a71bb30 after 12 minutes.


tpr-backed storage needs unit tests for watch / watch list

Whoops... #700 was closed a bit pre-maturely. Here's a new, more specific issue.
Watch / watch list tests are missing from the tpr-backed storage unit tests.

I am taking this


[IPv6] Add support for dual-stack when using kube-proxy for Service

Add a config item for IPv6 Serivce CIDR if using kube-proxy to
provide Service functions.
Output IPv6 traffic from host gateway if its destination is a
Service address.
Use ct_mark to identify Service traffic and output the reply
packet to the host gateway to ensure the DNAT processing in iptables.

Codecov Report

Merging #1200 into ipv6 will decrease coverage by 0.16%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             ipv6    #1200      +/-   ##
==========================================
- Coverage   41.09%   40.92%   -0.17%
==========================================
  Files          74       74
  Lines       10805    10839      +34
==========================================
- Hits         4440     4436       -4
- Misses       5999     6035      +36
- Partials      366      368       +2

Flag
Coverage Δ

#unit-tests
40.92% <0.00%> (-0.17%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/agent/agent.go
13.56% <0.00%> (-0.21%)
:arrow_down:

pkg/agent/openflow/client.go
10.54% <0.00%> (ø)

pkg/agent/openflow/pipeline.go
19.79% <0.00%> (-0.50%)
:arrow_down:

pkg/agent/util/net.go
30.64% <0.00%> (-1.03%)
:arrow_down:

pkg/apiserver/storage/ram/watch.go
85.71% <0.00%> (-3.18%)
:arrow_down:

...ntroller/networkpolicy/networkpolicy_controller.go
73.43% <0.00%> (-0.56%)
:arrow_down:

pkg/agent/querier/querier.go
65.90% <0.00%> (+0.79%)
:arrow_up:

/test-all

/test-all

/test-all

/test-ipv6-conformance

/test-ipv6-conformance

/test-ipv6-conformance

/test-all

/test-all

/test-all

/test-all

/test-windows-networkpolicy
/test-e2e

/test-all


Build script is failing

Build step is failing to generate the js files and the output folder. It simply exits with this log. Am I missing something here?? I did install all @abaplint packages in global as well. Installed all the required ABAP extensions for VS Code.

FYI. Thought that this might an issue with my system setup. Tried replicating with a Github Codespace and encountered the same error.

try again, I've updated the dependencies to latest, https://github.com/open-abap/azure-functions-v4-abap-template/pull/7

Thanks @larshp . The flow seems to work now.
But I have one last question. The build folder contains all the global types, data elements, exception classes etc...Is there a way we can generate only the necessary global artifacts? I am don't have a full understanding of the transpiler here, but can we use the syntax trees during the parse phase to restrict the list of generated artifacts ??

https://github.com/open-abap/azure-functions-v4-abap-template/blob/main/abap_transpile.json#L5 defines which stuff to pull in, you can define any repository there
there is no "tree shaking" implemented as of now, but would be a nice feature

also see https://open-abap.org, I will be reducing the number of objects in "core", just takes time...

Awesome....BTW I genuinely appreciate the work you and others from ABAP community are doing.
I wish to contribute to these topics....I can see lot of topics related to open abap, abaplint, etc.. scattered over different repos...I will scour them for beginner issues first and start contributing....Any beginner guide/links would be very helpful 👍👍👍


DO NOT MERGE: Make Spiff a library.

This is for discussion, only.
We found that we need a dynamic version of YAML for our own purposes, but we cannot statically compute everything required for a configuration without interacting with other systems in our case.  This change does the following:

[ ] Refactors Spiff to be a library, not a binary.  In this model, builtins would be provided by consumers of this library.
[ ] Refactors Spiff to work in phases.  Use some of the config, do some stuff, have builtins provide other parts of the config as a result...
[ ] Brings PEG up to date and cleans up dependencies only required by the binary.

This is what would be needed to complete this work:

[ ] Import paths would need to be rewritten so the merge still refers to upstream.
[ ] Better test coverage is required...

[ ] Phased flows fail in reasonable ways; the errors are understandable.
[ ] Dependencies propagate properly across builtins and delay argument evaluation.
[ ] Cyclic dependencies between phases are detected.  (I don't think they are right now!)

[ ] A proper strategy for the existing binaries and builtins would be required.  My suggestion is to move that stuff into a separate repo...

Thoughts?

Hi @shutej,
spiff is not currently under active development at the moment.  I will close this PR for now.
Thanks,
Amit


Payment issues

Payment issues: no BitPay on Join, free memberships (price 0) still get processed through a payment provider.
Related issue - #139

@AntonLV  please post notification here as soon as this problem is fixed -<EMAIL_ADDRESS>

(1) My Bitpay doesnt checkout in the join proces and (2) my free level members are being forced to through the paypal process. In relaton to the membership issue: MEMBERSHIPS
Levels	Add Level	Settings
System Levels
Non-member Actions
Standard Actions
Promotion Actions..
ad interim, to resolve the issue of my free membership levels being forced to paypal process; is there anyway to just extend the system level free membership levels (ad interim) so that i could direct my visitors who are only interested in free membership to the system's free levels memberships. and how do visitors reach the "standard" level

RE 1: BitPay is not available for "Pay Before Join" mode, because it has delayed payment processing. Therefore only PayPal and 2Checkout from default Payment module are available for "Pay Before Join" mode.

this is not totally responsive to my last support inquries. Right now I
think its I need to ask about refund and charge back policies questions.
On Fri, Nov 13, 2015 at 4:03 AM, Anton<EMAIL_ADDRESS>wrote:

RE 1: BitPay is not available for "Pay Before Join" mode, because it has
delayed payment processing. Therefore only PayPal and 2Checkout from
default Payment module are available for "Pay Before Join" mode.
—
Reply to this email directly or view it on GitHub
https://github.com/boonex/dolphin.pro/issues/143#issuecomment-156368085.

Oh this is not boonex its github. Pardon me. Ive been waiting for hours etc
for boonex responses
On Fri, Nov 13, 2015 at 4:08 AM, jomo williams<EMAIL_ADDRESS>wrote:

this is not totally responsive to my last support inquries. Right now I
think its I need to ask about refund and charge back policies questions.
On Fri, Nov 13, 2015 at 4:03 AM, Anton<EMAIL_ADDRESS>wrote:

RE 1: BitPay is not available for "Pay Before Join" mode, because it has
delayed payment processing. Therefore only PayPal and 2Checkout from
default Payment module are available for "Pay Before Join" mode.
—
Reply to this email directly or view it on GitHub
https://github.com/boonex/dolphin.pro/issues/143#issuecomment-156368085
.

RE 2: The enhancement was done. You may check it in Ticket #172 (Payment before join: need a way to join without payment for free membership)


chore(coral-web): consolidate strings into one file

Description:
Consolidates all of the strings/labels used in coral_web/ into one file so that users may easier translate the app into other languages.
AI Description

This PR introduces a new feature to the application: the ability to customize the UI in a different language. To achieve this, the labels in the src/interfaces/coral_web/src/constants/strings.tsx file have been translated.
The following changes have been made:

The AgentCard component now uses the STRINGS object for labels and titles, ensuring consistency across the application.
The AgentForm component has been updated to use the STRINGS object for placeholders, labels, and tool names, providing a more localized experience.
The AgentToolFilePicker component now utilizes the STRINGS object for button labels, making it easier to understand for non-English speakers.
The AgentsList component has been modified to use the STRINGS object for displaying the list of assistants, enhancing the user experience for non-English users.
The AgentsSidePanel component now employs the STRINGS object for button labels, improving the overall localization of the application.
The CreateAgent component has been updated to use the STRINGS object for titles, descriptions, and button labels, allowing for a more seamless localization process.
The DeleteAgent component now leverages the STRINGS object for error messages and button labels, ensuring a consistent user experience across different languages.
The DiscoverAgentCard component has been modified to use the STRINGS object for button labels and titles, making it more accessible to users who prefer a different language.
The DiscoverAgents component now utilizes the STRINGS object for titles and descriptions, providing a more localized experience for users.
The UpdateAgent component has been updated to use the STRINGS object for error messages, button labels, and titles, allowing for a more seamless localization process.
The AuthLink component now employs the STRINGS object for button labels, ensuring a consistent user experience across different languages.
The Citation component has been modified to use the STRINGS object for displaying the number of references, making it more accessible to users who prefer a different language.
The CitationDocumentHeader component now utilizes the STRINGS object for displaying file and tool information, providing a more localized experience for users.
The CitationDocumentSnippet component has been updated to use the STRINGS object for button labels, ensuring a consistent user experience across different languages.
The CitationTextHighlighter component now employs the STRINGS object for displaying output, making it more accessible to users who prefer a different language.
The ConnectDataModal component has been modified to use the STRINGS object for descriptions and button labels, improving the overall localization of the application.
The ComposerError component now leverages the STRINGS object for error messages and button labels, ensuring a consistent user experience across different languages.
The ComposerToolbar component has been updated to use the STRINGS object for file attachment labels, allowing for a more seamless localization process.
The DataSourceMenu component now utilizes the STRINGS object for listbox titles, providing a more localized experience for users.
The DragDropFileUploadOverlay component has been modified to use the STRINGS object for file upload labels, making it more accessible to users who prefer a different language.
The WebSearchConfigurationModal component now employs the STRINGS object for button labels and placeholder text, ensuring a consistent user experience across different languages.
The Composer component has been updated to use the STRINGS object for message input placeholders, allowing for a more seamless localization process.
The Header component now leverages the STRINGS object for button labels and tooltips, ensuring a consistent user experience across different languages.
The MessagingContainer component has been modified to use the STRINGS object for button labels, making it more accessible to users who prefer a different language.
The ConversationError component now utilizes the STRINGS object for error messages, providing a more localized experience for users.
The ConversationCard component has been updated to use the STRINGS object for button labels and titles, allowing for a more seamless localization process.
The ConversationListHeader component now employs the STRINGS object for displaying the list of chats, ensuring a consistent user experience across different languages.
The ConversationListPanel component has been modified to use the STRINGS object for error messages and search placeholders, making it more accessible to users who prefer a different language.
The DataTable component now utilizes the STRINGS object for error messages and button labels, providing a more localized experience for users.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.23%. Comparing base (ca5ab66) to head (d6d0eaf).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #764   +/-   ##
=======================================
  Coverage   75.23%   75.23%
=======================================
  Files         265      265
  Lines       11862    11862
=======================================
  Hits         8924     8924
  Misses       2938     2938

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Don't change split header for whisper

Prevent normal channel split headers from changing colours when receiving a whisper

Related Issue: #691


OpenCV 4.X Support - Apps & Samples

./apps/mivisionx_winml_classifier/classifier.cpp:#include <opencv/cv.h>
./apps/mivisionx_winml_classifier/classifier.cpp:#include <opencv/highgui.h>
./apps/image_augmentation/image_augmentation.cpp:#include <opencv/highgui.h>
./apps/mivisionx_openvx_classifier/source/classifier.cpp:#include <opencv/cv.h>
./apps/mivisionx_openvx_classifier/source/classifier.cpp:#include <opencv/highgui.h>

./utilities/inference_generator/src/nnef2openvx.cpp:             << "#include <opencv/cv.h>" << std::endl
./utilities/inference_generator/src/nnef2openvx.cpp:             << "#include <opencv/highgui.h>" << std::endl
./utilities/rali/rali_performance_tests_with_depth/rali_performance_tests_with_depth.cpp:#include <opencv/highgui.h>
./utilities/rali/rali_basic_test/rali_basic_test.cpp:#include <opencv/highgui.h>
./utilities/rali/rali_dataloader_tf/rali_dataloader_tf.cpp:#include <opencv/highgui.h>
./utilities/rali/rocal_unittests/rocal_unittests.cpp:#include <opencv/highgui.h>
./utilities/rali/rali_dataloader/rali_dataloader.cpp:#include <opencv/highgui.h>
./utilities/rali/rali_performance_tests/rali_performance_tests.cpp:#include <opencv/highgui.h>
./utilities/rali/rali_unittests/rali_unittests.cpp:#include <opencv/highgui.h>
./utilities/rali/rali_video_unittests/rali_video_unittests.cpp:#include <opencv/highgui.h>
./utilities/rali/rali_dataloader_mt/rali_dataloader_mt.cpp:#include <opencv/highgui.h>

./samples/model_compiler_samples/include/classification.h:#include <opencv/cv.h>
./samples/model_compiler_samples/include/classification.h:#include <opencv/highgui.h>
./samples/inference/mv_objdetect/visualize.h:#include <opencv/cv.h>
./samples/inference/mv_objdetect/visualize.h:#include <opencv/highgui.h>

PR #854 -- Add OpenCV Support for utilities/rali

with updates to develop, this issue is resolved.


Project 2: Function to visualize paths in the original form

Besides the plots for key features of paths (see the pull request #162) we need a function to plot paths in their original form, which would look like:

This issue was originally suggested in #122 but is separated since we should discuss a bit more about additional functionalities, such as

adding some rectangle to illustrate the area for home, water, and food (by @jJasonWang)
showing some key features at the same time (e.g. speed) by changing the color of the path (by @aksam-ahmad)
controlling some plot parameters like color, types of plots (e.g. points or a line), the size of points, the width of line, etc.

@aksam-ahmad, @alannaiverson, @jcpalumbo3, @jJasonWang, @sicihuang, any comments are welcomed!

Have you tried to color the path based on the speed? I think it would be hard to find the pattern by it. While we can change the width of the line, I am not sure how its effect will be.
For my idea, I found that it would be hard to set the exact area for home, water, and food.

Your 2 points are fair enough @jJasonWang: I haven't tried using gradient, and the purpose of such functionality might be a bit unclear.
Right now we can modify the width of the line by controling alpha. So do you think we are ready to merge?

I think the alpha control the transparency of the line, right? We can add an argument for linewidth. Then it should be ready to merge.

Good point. I just modified.
Just clarification, we don't have to test the input of this function, do we?

We don't have to test the input of the plot function. See the reason in #143.

Thanks @jJasonWang! I missed this pull request but it makes sense now.

I close this issue since the pull request #172 was merged.


Documentation update TECHOPS-559

Current coverage is 51.98% (diff: 100%)

Merging #138 into master will not change coverage

@@             master       #138   diff @@
==========================================
  Files            98         98
  Lines          4940       4940
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2568       2568
  Misses         2131       2131
  Partials        241        241

Powered by Codecov. Last update 60020df...918ffb8

@samsalisbury For the life of me, I cannot remember where we left this. Can you do a counter-PR?

@nyarly I've opened a PR continuing from this one at https://github.com/opentable/sous/pull/143


WIQL query only returns first 50 rows of results

Describe the bug
Have created a query using WIQL Playground but the results appear to be limited to the first 50 rows.
Message at foot of report "Showing first 50 results" without any obvious sign/link to show additional results.
When we click "Open in queries" CTA then the vsts UI displays 100 results.
We need to use WIQL because the asof functionality isn't available in vsts.
Environment:

Chrome
Hosted

Screenshots
No errors in the dev console - returns 200 response
Expected behavior
I expect to see all work items that match the query to be displayed when I press "Run"
Additional context
N/A

In order to limit the scope of the extension the playground was intentionally written to be a very limited environment for testing queries.
If you use the open in queries and save as a new query the saved query should still be using the asof clause.

Thanks for confirming Otto


Wrong syntax highlight when using @ in css

When using @ in the css like in the following screenshot:

the syntax highlight gets messed up in the rest of the document:

Thank you for posting this. I do remember seeing something similar to this in the past. I was just doing some testing on my own code that is similar to what you posted, I noticed that unless you place some type of ending tag it will mess up all following highlighting. Just out of curiosity on line 29, can you place a ';' after ocLogo?
Also, more of reviewing your code, not related to this, why are you mixing string html with ejs? You can build the function with all function parts inside ejs tags and have the html stand by itself outside ejs tags.  If there is a reason you chose this way, I am genuinely curious as to why you this way is better in your mind. I could very well be wrong in my thinking. I love to learn other and different ways.

I will make this a separate issue, but when I "commented out" a single line with double slashes with an ending ejs tag on the same line, this also caused errors.

I think it’s partly because of line 19, where you’re using EJS inside CSS, which is untested and probably doesn’t work.
In either case, you really should avoid using inline <style> blocks.

Also, can you upload the file so that I can inspect this on my end.

This seems to have simply disappeared with the fix for #31.
Also, I couldn’t reproduce this with the latest stable release.

Also, can you upload the file so that I can inspect this on my end.

Sorry, I did not see the comment. Good thing it got solved, thank you!


add unsigned to the new migrations

resolves #9443

It is not support pgsql database.

@vaseninm how can I get database type?
I can add expression for mysql

@berezuev namespace yii\db\mysql; For example: framework/db/oci/ColumnSchemaBuilder.php

Can't be applied because incompatible with at least PostgreSQL.


RuntimeError: CUDA error: an illegal memory access was encountered

Traceback (most recent call last):
File "/root/autodl-tmp/depth-aware-3DGS/train.py", line 322, in
training(lp.extract(args), op.extract(args), pp.extract(args), args.test_iterations, args.save_iterations, args.checkpoint_iterations, args.start_checkpoint, args.debug_from)
File "/root/autodl-tmp/depth-aware-3DGS/train.py", line 101, in training
loss = (1.0 - opt.lambda_dssim) * Ll1 + opt.lambda_dssim * (1.0 - ssim(image, gt_image))
File "/root/autodl-tmp/depth-aware-3DGS/utils/loss_utils.py", line 38, in ssim
window = window.cuda(img1.get_device())
RuntimeError: CUDA error: an illegal memory access was encountered
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1
Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.
Training progress:   5%|███████▎                                                                                | 480/10000 [00:54<17:54,  8.86it/s, Loss=0.3098132]
(depth_aware_3dgs) root@autodl-container-1ca84e8b33-f925896c:~/autodl-tmp/depth-aware-3DGS#

I used a 32G gpu, but out of memory

maybe you can improve the version of the  cudatookit


XBox Game Pass on PC does not work with porn filter

When I open xbox game pass for PC application, my game library is empty, and even searching for games I already own shows no results, so Xbox is not working properly with porn filter.
If I disable the porn filter, xbox app starts working again.
Using master filter instead fixes the problem.

Adding exceptions for
v10.events.data.microsoft.com
v10.vortex-win.data.microsoft.com
Seems to work.

I have narrowed down the filter list until I found these:
<IP_ADDRESS> azurefd.net
<IP_ADDRESS> www.azurefd.net
It seems the porn filter (https://raw.githubusercontent.com/StevenBlack/hosts/master/alternates/porn/hosts) is blocking Microsoft Azure Front Door domain entirely.
Info on domain here:
https://www.netify.ai/resources/domains/azurefd.net

Thank you for this Federico @enlightdm.
Ping @sinfonietta this is probably a Type I error in your list.

@StevenBlack I just fixed this

Thank you @Sinfonietta and Federico @enlightdm.
This is live and online now in the latest release.
Closing.


Catalog fails to load in UI (time out)

Rancher Versions:
Server: 1.5.4
scheduler:
cattle
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
On-premise HA rancher with RDS Aurora database in AWS (12 network hops between server and DB)
Steps to Reproduce:
a) In the UI click on Catalog -> All
b) In the UI click on Catalog -> Community
Results:
a) 504 gateway timeout page appears after 30 seconds
b) Sometimes catalog is displayed after a load time short of 30 seconds
Even after setting logback.root.level to TRACE and replicating the issue no log entries relating to the catalog timeout were printed.

That is from the SSL termination device or proxy in between.  Probably Apache from the message.  There is no timeout in Rancher.

@vincent99 Correct, haproxy timeout is set to 30 sec. The issue is that the catalog page should not take 30+ seconds to load in the first place.

Verified that this is working significantly faster and no longer an issue with this on-premise setup.


suggestion to add "You know nothing" meme

A discussion could be opened to add "you know nothing" meme
http://knowyourmeme.com/memes/you-know-nothing-jon-snow
The text on the gif would be the best, but different approaches are possible.
either
 you know nothing,
or
 you know nothing   // you think you can restart server over lunchtime? you know nothing
or
you know nothing

Oh, sorry, this project only deals with Ingress-specific commands, such as badges, checking septicycle, and other things related to Ingress.
Please open an issue at hubot-scripts to request this feature (suggested title: add meme to meme-captain.coffee).

 Thanks for the info then :)

No worries! :)


AttributeError: exit

Hi,
My project was working fine untill today afternoon...
Since then, the following code is reporting an "AttributeError: exit"
from pandasdmx import Request
esta = Request('ESTAT')
une_resp = esta.get('data','demo_gind')

I attached a printscreen of the error:

Do you have an idea of the source of the issue ?
Thank you

Hi,
I will gladly look at your problem once you send me a code example
copy-pasted into the message rather than a screenshot.
Thanks.
Leo
Am 13.03.2017 um 21:53 schrieb Vincent Matton:

Hi,
My project was working fine untill today afternoon...
Since then, the following code is reporting an "AttributeError: exit"
|from pandasdmx import Request esta = Request('ESTAT') une_resp =
esta.get('data','demo_gind') |
I attached a printscreen of the error:
capture-python
https://cloud.githubusercontent.com/assets/12695531/23874808/4785edae-0837-11e7-961f-84ee550f8881.PNG
Do you have an idea of the source of the issue ?
Thank you
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dr-leo/pandaSDMX/issues/48, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADA58vAU7pnifAFSgu4lC0Ty4JLxJj2Gks5rlazUgaJpZM4MbzXe.

Hi Leo,
Thank you for your quick answer.
The code is very simple (as in my previous message) :
from pandasdmx import Request
esta = Request('ESTAT')
une_resp = esta.get('data','demo_gind')

Thanks,
Vincent

Pls send the traceback. Thanks.

Am 14.03.2017 um 08:42 schrieb Vincent Matton<EMAIL_ADDRESS>Hi Leo,
Thank you for your quick answer.
The code is very simple (as in my previous message) :
from pandasdmx import Request
esta = Request('ESTAT')
une_resp = esta.get('data','demo_gind')
Thanks,
Vincent
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I retry to run the script right now and it seems to work...
It was maybe an issue with the Eurostat server as I noticed that I wasn't able to connect to it with other Python scripts this morning...
Anyway, thank you for your support !!


mod_access: support CLF format

mod_access: support CLF format

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

Codecov Report

Merging #214 into develop will decrease coverage by 0.07%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           develop     #214      +/-   ##
===========================================
- Coverage    53.01%   52.94%   -0.08%
===========================================
  Files          219      219
  Lines        21724    21750      +26
===========================================
- Hits         11518    11515       -3
- Misses        8899     8928      +29
  Partials      1307     1307

Impacted Files
Coverage Δ

bfe_modules/mod_access/request_log.go
0% <0%> (ø)
:arrow_up:

bfe_websocket/server_conn.go
72.72% <0%> (-3.79%)
:arrow_down:

bfe_balance/bal_gslb/sub_cluster.go
55.55% <0%> (+7.4%)
: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 c029144...20fa790. Read the comment docs.


Filter notifiable users in reminder notifications

Ensure reminder notifications are sent only to notifiable users.
The user affected by the bug @nshoes fixed in #523 received a reminder notification around noon today. This should fix that!

Would be worth validating with a spec.


Fix iOS memory leak

Add content of incorrectly removed tearDownGL method call
Use weak references for TGMapViewController

Fixes #1690

👍


[DO NOT MERGE] Composer support for Debug Adapter Protocol

Description
Bot Builder supports Dialog Debugging through an implementation of the server side of the Debug Adapter Protocol.  This implementation can be used with the Visual Studio Code Debugger which implements the client side of the Debug Adapter Protocol with some light integration provided by the Dialog Debugger Extension.
This PR adds Composer support for the client side of the Debug Adapter Protocol.
TODO ideas:

initial review (e.g. dependencies - redux, redux sagas, debug adapter protocol)
update Composer Bot to latest version of bot builder SDK
register debugger server in Composer Bot
merge Composer master branch into this branch
move to Composer extension
improve shell API to navigate by $designer.id
support for more debugger features (e.g. expression evaluation)

depends on https://github.com/microsoft/botbuilder-dotnet/pull/3637 and earlier PRs


346/weight end point

Post request to receive all facts in order of importance per outcome

Please add an entry to README.md describing how to use this endpoint and its output.


fix(*): Code coverage checks block features from merging

When attempting to improve code coverage using the firestore-send-email module it is proving difficult to add coverage without breaking the CI. This PR can be seen here.
Separate investigations have been attempted to improve the coverage of modules such as the Firebase Cli, it is considered the lack of coverage on dependent modules such as the Cli are causing the coverage to drop.
Proposed solution and steps to update:

Remove code coverage check in the CI to enable modules to merge without having to hit the current threshold. (fixes https://github.com/firebase/extensions/pull/586).

Merge required modules for coverage (fixes https://github.com/firebase/extensions/pull/527)

@jhuleatt has fixed this


Can glint support discrete feature id?

I have millions of discrete 64-byte features, remapping features to continuous ids is expensive. Can glint support that?

Although remapping is expensive, it's currently the only feasible thing to do with Glint since all matrices and vectors are stored as dense arrays.
One of my goals right now is to write a roadmap that outlines several things we want to implement in the near future. It includes sparse matrices and vectors. With sparsity it will be possible to create very large feature spaces with little/no memory overhead. However, because everything is indexed with Long values, it would still be limited to a size of 2^63.

How long will it take? I am eager to intergrate glint with YARN with Long key features. I have a preliminary version, but it takes long time to work on bigger data.

I can't give an accurate time estimate at the moment. In terms of importance, I prioritize implementing fault tolerance over other features right now, so it could be on the order of months before I get to sparsity.
And even then, I'm not sure how well this will work... Sparse data structures typically come with considerable memory overhead (one or more objects per key/value pair), which the JVM garbage collector unfortunately does not like. I'm considering using something like debox or scala-offheap to bypass this garbage collection problem, but both are rather experimental.

I have use debox in my algorithm, and it helps a lot. Now I transform ids to continuous ones, and I look forward to get your help to intergration glint with spark. Thanks a lot.

I implemented a key-value based partitioner to avoid remapping feature id. Hash-based version may be more scalable to big data. I am glad to send a pull request if there is a necessary.

@cstur4 I'd like to see that PR - for sparse models I'm looking at it would be very important for scalability.


fix(typerror): fix unhandled possible undefined of ref #62

@emilkowalski Fixes #62

Looks great, thank you! Could you remove changes to other files than src/index.tsx?

@emilkowalski Done 👀


Dashmate throws SyntaxError on docker 500

Expected Behavior

Dashmate shows readable error message
Current Behavior

When there is a major issues with Docker (f.e. out of disk memory), dashmate wrongfully parses this response, and failing with SyntaxError: Unexpected token < in JSON at position 0 error.
[STARTED] Force nodes to sync
[DATA] 0.00% complete
SyntaxError: Unexpected token < in JSON at position 0
    at JSON.parse (<anonymous>)
    at IncomingMessage.<anonymous> (/Users/ivanshumkov/Projects/dashevo/platform/.yarn/cache/@dashevo-dashd-rpc-npm-2.4.2-60538862a8-fbbb49ed2c.zip/node_modules/@dashevo/dashd-rpc/lib/index.js:128:26)
    at IncomingMessage.emit (node:events:525:35)
    at endReadableNT (node:internal/streams/readable:1358:12)
    at processTicksAndRejections (node:internal/process/task_queues:83:21)

<TITLE>503 Service Unavailable</TITLE>

Service Unavailable

HTTP Status code:503

node:internal/process/promises:279
            triggerUncaughtException(err, true /* fromPromise */);
            ^
OperationalError: Dash JSON-RPC: Error Parsing JSON: Unexpected token < in JSON at position 0
    at IncomingMessage.<anonymous> (/Users/ivanshumkov/Projects/dashevo/platform/.yarn/cache/@dashevo-dashd-rpc-npm-2.4.2-60538862a8-fbbb49ed2c.zip/node_modules/@dashevo/dashd-rpc/lib/index.js:133:21)
    ... 2 lines matching cause stack trace ...
    at processTicksAndRejections (node:internal/process/task_queues:83:21) {
  cause: Error: Dash JSON-RPC: Error Parsing JSON: Unexpected token < in JSON at position 0
      at IncomingMessage.<anonymous> (/Users/ivanshumkov/Projects/dashevo/platform/.yarn/cache/@dashevo-dashd-rpc-npm-2.4.2-60538862a8-fbbb49ed2c.zip/node_modules/@dashevo/dashd-rpc/lib/index.js:133:21)
      at IncomingMessage.emit (node:events:525:35)
      at endReadableNT (node:internal/streams/readable:1358:12)
      at processTicksAndRejections (node:internal/process/task_queues:83:21),
  isOperational: true
}

Possible Solution

Catch such errors and provide readable message for the user about the issue
Steps to Reproduce (for bugs)

Make your docker run out of disk memory
dashmate start

Context

Your Environment

Version used:
Environment name and version (e.g. Chrome 39, node.js 5.4):
Operating System and version (desktop, server, or mobile):
Link to your project:

I haven't seen this for a while. @pshenmic and you? I feel like Docker changed the response format. Please reopen if it still the case.


Add support for C++20 & 2b

In order to support C++20, cxx_standard::__STD_CXX dictionary in utils/utils.py  should be fixed to contain:
...
        '-std=c++2a': 202002,
        '-std=gnu++2a': 202002,
        '-std=c++20': 202002,
        '-std=gnu++20': 202002,
        '-std=c++2b': float('inf'),
        '-std=gnu++2b': float('inf'),

But I'm not sure we should limit the list of valid values as we kind of know we have a new version every 3 years, and at most 3 (draft) versions per decade.

I made the fix for the next release. Sadly I don't see a different solution right now. Better than nothing I guess.


Issue #977: Added validations of login node variables for Rocky MS with IPA server installed and Leap Login Node

Issues Resolved by this Pull Request
Fixes #977
Description of the Solution

Added validations of login node variables for Rocky MS with IPA server installed and Leap Login Node
Added rescue block for the kubeadm join command to inform user has to run kubeadm reset manually in case of any failure of the command.

Suggested Reviewers
@sujit-jadhav

Closing the PR due to workflow jobs not running


encode labels for single-value, and for all levels of nestings of containing lists

Now, with multi-labeling, references become a "deep" (!) list.
EncodeLabel tried to chase the depth via specifying * and set_multiple  features of dict_set.
Suggested here is a simple EncodeLabel that needs no *, and does not use the (suspicious, in my view) set_multiple, and is ready for the new references, no matter how deep their lists are.

Dangerous if value is a list of scalars and lists

closing for now. Perhaps in the future, when more levels of list nesting will show in references.


Where is config/oidc-googleCloud.json

Hi!
I try to connect whit Google Cloud Healthcare.
I read this docs https://docs.ohif.org/connecting-to-image-archives/google-cloud-healthcare.html#
But where is config/oidc-googleCloud.json?

There is a note at the top of that page:

ATTENTION: The original documentation for this integration lives in the legacy version 1 Meteor documentation. You can find it here. These docs will mirror the Meteor documentation until our React implementation has been updated to work with Google Cloud Healthcare.

Clicking find it here takes you to: https://docs.ohif.org/history/v1/connecting-to-image-archives/google-cloud-healthcare.html
All code for v1 (the Meteor version) of the OHIF Viewer lives on this branch:
https://github.com/OHIF/Viewers/tree/v1.x
The file you're looking for lives here:
https://github.com/OHIF/Viewers/blob/v1.x/config/oidc-googleCloud.json


Add IRA-like calculations to economics module

Add a simple way to estimate the impact of IRA-like tax incentives on a geothermal project:

Investment Tax Credit - applied to reduce CAPEX?
Production Tax Credit - applied to reduce OPEX and thereby reduce LCOE?

Need to account for "adders" like 10% for local content, 10% for Energy Community, etc.
Also need to account for time limits (e.g., PTC expires 10 years after it is started

@malcolm-dsider is this related to and/or duplicative of https://github.com/NREL/GEOPHIRES-X/issues/169?title=Additional+economic+calculations?

No. Related and perhaps executed at the same time, but different calculations.

Completed via PR #177


partial search or multi search

hi, is there a way to make partial search or multiple search in the node window, i mean like looking for ore and log but nothing else?

No. Why?

No. Why?
To do something like this:

It was just for example, it would be nice if it was possible to generate a list of only the necessary nods.


{Core} Provide error recommendation for validate_file_or_dict

Description
This PR is created according to the feedback on az ad app federated-credential (https://github.com/Azure/azure-cli/pull/22727).
validate_file_or_dict is used to parse argument that is either a JSON file or JSON string.
If the file exists, but contains invalid JSON string:
> az ad app federated-credential create --id 233dd73b-72e3-424a-9367-7588d957267e --parameters 'D:\test\fic-error.json'
Failed to parse D:\test\fic-error.json with exception:
    Failed to parse JSON: {
  "audiences": [
    "api://AzureADTokenExchange"
  ],
  "description": "Updated description"
    "id": "f684b731-198c-406e-8c8d-f2dd23d93cdf",
  "issuer": "https://token.actions.githubusercontent.com/",
  "name": "Testing",
  "subject": "repo:octo-org/octo-repo:environment:Production"
}
Error detail: Expecting ',' delimiter: line 6 column 5 (char 105)

If a non-existing file or invalid JSON string is provided, CLI fails with
> az ad app federated-credential create --id 233dd73b-72e3-424a-9367-7588d957267e --parameters 'non-existing.json'
Failed to parse JSON: non-existing.json
Error detail: Expecting value: line 1 column 1 (char 0)
The JSON may have been parsed by the shell. See https://docs.microsoft.com/cli/azure/use-cli-effectively#use-quotation-marks-in-arguments
PowerShell requires additional quoting rules. See https://github.com/Azure/azure-cli/blob/dev/doc/quoting-issues-with-powershell.md

The error may be misleading, because we allow passing an in-line JSON string. We first checks if the string is an existing file path. If not, we fall back to in-line JSON. There is no way for us to know if the provided string is a non-existing JSON file or invalid JSON string.
After this PR, for existing file with invalid JSON string, we show:
> az ad app federated-credential create --id 233dd73b-72e3-424a-9367-7588d957267e --parameters 'D:\test\fic-error.json'
Failed to parse file 'D:\test\fic-error.json' with exception:
Failed to parse string as JSON:
{
  "audiences": [
    "api://AzureADTokenExchange"
  ],
  "description": "Updated description"
    "id": "f684b731-198c-406e-8c8d-f2dd23d93cdf",
  "issuer": "https://token.actions.githubusercontent.com/",
  "name": "Testing",
  "subject": "repo:octo-org/octo-repo:environment:Production"
}
Error detail: Expecting ',' delimiter: line 6 column 5 (char 105)
Please provide a valid JSON file path or JSON string.

For non-existing file or invalid JSON string, we show:
> az ad app federated-credential create --id 233dd73b-72e3-424a-9367-7588d957267e --parameters 'non-existing.json'
Failed to parse string as JSON:
non-existing.json
Error detail: Expecting value: line 1 column 1 (char 0)
Please provide a valid JSON file path or JSON string.
The provided JSON string may have been parsed by the shell. See https://docs.microsoft.com/cli/azure/use-cli-effectively#use-quotation-marks-in-arguments
PowerShell requires additional quoting rules. See https://github.com/Azure/azure-cli/blob/dev/doc/quoting-issues-with-powershell.md

Core


chore: Add support for useDefineForClassFields: true

Add support for useDefineForClassFields: true.
Closes SAP/cloud-sdk-backlog#1136.

@deekshas8 Do we have a test that checks what requestConfig.payload is when calling new CreateRequestBuilder(...)?

We do kinda test it here . When I ran this test with ES2022 setup (I generated the TestEntity with ES2022), the test failed because the postBody no longer matched (had all other properties with undefined ). I will check if we can somehow add an explicit test for this.


Can I use the automatic schema generator for non-referenced classes?

Background is: We use jsonapi.org in our play project, which auto-generates its routes, so we don't have them in the routes file. Obviously I have to write the specs for that completely on my own, but at least the schemas could be auto generated. Maybe by another variable in the build.sbt file like:
swaggerAdditionalSchemas := Seq("jsonapiorg.resources.MyRestDTO")

I'd imagine it not too hard to be implemented. What do you think?

Thank you for getting in touch. @aronschmidt
Are you using scala-jsonapi ?
I don't have knowledge about it, so I would be grateful if you could provide comments on more specific usage for sample projects or proposed methods.

No, we're using qvantel/jsonapi-scala. But tbh we want to replace it by normal routes bc it adds much more overhead to our api compared to what we really use. I just wanted a little help for writing our openapi specs, bc I have to do that now and it will take some time to fully replace jsonapi.
Oh, I didn't see this repo is a fork of the iheartradio one. Maybe I should ask my question there?

@aronschmidt
Thank you for your reply.

Oh, I didn't see this repo is a fork of the iheartradio one. Maybe I should ask my question there?
No, this repository is the master now. I have taken over from the iheartradio maintainer and am now developing it. I believe if you comment on the upstream repository, you might not receive a response.

I still don't fully understand the details of qvantel/jsonapi-scala. I'd like to see a sample project using these in the Play Framework, or a more detailed explanation of the desired functionality.
So, you want to generate openapi json from jsonapi, is that correct?

So, you want to generate openapi json from jsonapi, is that correct?

That'd be a dream, yes. But for now I'd be happy if I could simply define a list of case class references, so the schemas for them get auto generated and inserted into the swagger.json file.


functional-tester: limit lease creation in lease stresser

too many leases created during leases stressing phase is causing compaction to timeout.
bound leases creation solves compacting issue.

related the compaction issue in https://github.com/coreos/etcd/issues/6685

This should probably be done with rate.Limiter

@fanminshi Probably we can try limiter to limit this to a relatively low rate. like 1 round per 5 seconds or so.

All fixed. PTAL @xiang90 @gyuho

let's make two pr. one for the limit thing; one for the lease update. thanks!

Maybe another PR for log level configuration

all fixed. PTAL @xiang90 @gyuho @heyitsanthony

lgtm


Timer and tests

Introduce timer - through the prompt box for now
rework the game tests a bit

My bad I pushed to main accidentally - more conflicts 😜