mixed types in array don't work

[config]
list = [
  { key = "value"},
  "foo",
]

this way file will be parsed but "foo" will be ignored: {config: {list: [{key: value}]}}

[config]
list = [
  "foo",
  { key = "value"},
]

After parsing toml file above parser will throw an error: IndexError: list index out of range.
Traceback:
  File "/path/to/lib/python3.7/site-packages/toml/decoder.py", line 456, in loads
    multibackslash)
  File "/path/to/lib/python3.7/site-packages/toml/decoder.py", line 725, in load_line
    value, vtype = self.load_value(pair[1], strictly_valid)
  File "/path/to/lib/python3.7/site-packages/toml/decoder.py", line 802, in load_value
    return (self.load_array(v), "array")
  File "/path/to/lib/python3.7/site-packages/toml/decoder.py", line 914, in load_array
    a[b] = a[b] + ',' + a[b + 1]
IndexError: list index out of range

This was only recently added to toml itsel: https://github.com/toml-lang/toml/pull/676/
However, we should add this.

I can still reproduce this error and, indeed, the result is rather inconsisten: it can dump arrays with mixed types, but it fails when loading that same output.
To reproduce:

Python Python 3.8.3
toml 0.10.1

import toml
data = {"a": [1, "cat"]}
data_encoded = toml.dumps(data)

The above works and results in data_encoded equal to 'a = [ 1, "cat",]\n' . Then:
data_back = toml.loads(data_encoded)

results in:
Traceback (most recent call last):
  File "XXXXXXX/python3.8/site-packages/toml/decoder.py", line 510, in loads
    ret = decoder.load_line(line, currentlevel, multikey,
  File "XXXXXXX/python3.8/site-packages/toml/decoder.py", line 777, in load_line
    value, vtype = self.load_value(pair[1], strictly_valid)
  File "XXXXXXX/python3.8/site-packages/toml/decoder.py", line 875, in load_value
    return (self.load_array(v), "array")
  File "XXXXXXX/python3.8/site-packages/toml/decoder.py", line 1024, in load_array
    raise ValueError("Not a homogeneous array")
ValueError: Not a homogeneous array
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "XXXXXXX/python3.8/site-packages/IPython/core/interactiveshell.py", line 3343, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-7-8f12f46e5bec>", line 1, in <module>
    data_back = toml.loads(data_encoded)
  File "XXXXXXX/python3.8/site-packages/toml/decoder.py", line 513, in loads
    raise TomlDecodeError(str(err), original, pos)
toml.decoder.TomlDecodeError: Not a homogeneous array (line 1 column 1 char 0)

The same applies when loading/dumping to a file rather than a string.

Reproduced this error with mixed integer/float array
Dump
import toml

mixed_types = [0, 1.1, 2.2, 3.3] # mixed integer and floats

toml.dumps({'array': mixed_types})

'array = [ 0, 1.1, 2.2, 3.3,]\n'

Load
toml.loads(toml.dumps({'array': mixed_types}))

ValueError                                Traceback (most recent call last)
~\Anaconda3\envs\modeling\lib\site-packages\toml\decoder.py in loads(s, _dict, decoder)
455                 ret = decoder.load_line(line, currentlevel, multikey,
--> 456                                         multibackslash)
457             except ValueError as err:
~\Anaconda3\envs\modeling\lib\site-packages\toml\decoder.py in load_line(self, line, currentlevel, multikey, multibackslash)
724         else:
--> 725             value, vtype = self.load_value(pair[1], strictly_valid)
726         try:
~\Anaconda3\envs\modeling\lib\site-packages\toml\decoder.py in load_value(self, v, strictly_valid)
801         elif v[0] == '[':
--> 802             return (self.load_array(v), "array")
803         elif v[0] == '{':
~\Anaconda3\envs\modeling\lib\site-packages\toml\decoder.py in load_array(self, a)
940                     if ntype != atype:
--> 941                         raise ValueError("Not a homogeneous array")
942                 else:
ValueError: Not a homogeneous array
During handling of the above exception, another exception occurred:
TomlDecodeError                           Traceback (most recent call last)
 in
----> 1 toml.loads(toml.dumps({'array': mixed_types}))
~\Anaconda3\envs\modeling\lib\site-packages\toml\decoder.py in loads(s, _dict, decoder)
456                                         multibackslash)
457             except ValueError as err:
--> 458                 raise TomlDecodeError(str(err), original, pos)
459             if ret is not None:
460                 multikey, multilinestr, multibackslash = ret
TomlDecodeError: Not a homogeneous array (line 1 column 1 char 0)

Spec v1.0.0-rc.2 says:

Array

Arrays are square brackets with values inside. Whitespace is ignored. Elements are separated by commas. Arrays can contain > values of the same data types as allowed in key/value pairs. Values of different types may be mixed.

# Mixed-type arrays are allowed
numbers = [ 0.1, 0.2, 0.5, 1, 2, 5 ]
contributors = [
  "Foo Bar<EMAIL_ADDRESS>  { name = "Baz Qux", email =<EMAIL_ADDRESS>url = "https://example.com/bazqux" }
]

Still a problem on Python 3.9.9 with toml 0.10.2:
arr = [ 'a', 0 ]

Error:
toml.decoder.TomlDecodeError: Not a homogeneous array ...

Doc (https://toml.io/en/v1.0.0#array):

Values of different types may be mixed.

Still a problem 3 years later... See https://github.com/python-poetry/poetry/issues/7094

Yup like cykerway said, this should be allowed: https://toml.io/en/v1.0.0#array:~:text=%23 Mixed-type arrays are allowed
numbers %3D [ 0.1%2C 0.2%2C 0.5%2C 1%2C 2%2C 5 ]
Still a problem 5 years later.


[TB] other Typography atoms don't reflect Font Settings atom changes

Problem/Concern
It looks like if you change the default fonts for your design system in ThemeBuilder in the Typography->Font Settings atom, the other Typography atoms don't pick up the change and still show "Open Sans" as their selected value.
To recreate:

I loaded the Google Fonts "Roboto" and "Merriweather" onto my macbook using these instructions: https://flourishonline.com/download-google-fonts-mac-pc/
I load the pre-defined "Sample" design system into Theme Builder
I click on Typography->Font Settings from the left nav bar
I change the primary front to "Roboto" and the secondary font to "Merriweather"
I click on Typography->Body Styles from the left nav bar

The entry for "Body 1" still says that the font is Open Sans.  However, the description in the Font Settings page says, "The primary font is used for the body and small font styles."  If I go to edit the font family for that entry, the new primary and secondary names are not in the list of selectable values.
We will also need to decide:

what font family values should be in the dropdown in the Edit modal dialogs.  Only primary and secondary or also any previously selected values for primary and secondary?
what should the behavior be when new fonts are selected in  Font Settings?  Should everything just switch?  Or should they keep their currently selected value and the user has to go in and manually change each one?  Or do all of the values get overwritten?

Proposed Solution
The behavior in the system should match what is described in the text.

The font family should use either --primaryFont or --secondaryFont instead of being hard coded by the SDK.  The user is selecting either the var(--primaryFont) or var(--secondaryFont) font variable for all of the typography properties.  The select dropdown should show "Primary Font: Open Sans", "Secondary Font: Roboto" so they know which font name is currently assigned to primary and secondary font variables.

My changes are available and ready:
https://github.com/discoverfinancial/a11y-theme-builder/pull/322

PLEASE NOTE my comment in the PR stating that SDK changes are needed:

This requires changes to the SDK. The SDK needs default values for all typography styles. This amounts to primaryFont for everything, except header/displays which use secondaryFont. Currently the SDK uses Open Sans for each, no matter what the Primary or Secondary Font is.


restore-from fails

Help us track down and resolve parity problems faster with this template.
What command did you execute?
(barberscore-api)bash-3.2$ development restore-from production
What did you expect to happen?
Restore from backup
What actually happened?
(barberscore-api)bash-3.2$ development restore-from production
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 59.1M  100 59.1M    0     0  5412k      0  0:00:11  0:00:11 --:--:-- 9602k
pg_restore: [archiver] unsupported version (1.13) in file header
(barberscore-api)bash-3.2$

Some information about your installation

What's your operating system?
macOS Sierra 10.13.3

What's the output of which development, which staging, which production?
Parity has had multiple installation channels, and it's not uncommon for an
old version to be somewhere else in your path.

(barberscore-api)bash-3.2$ which development
/usr/local/bin/development
(barberscore-api)bash-3.2$ which staging
/usr/local/bin/staging
(barberscore-api)bash-3.2$ which production
/usr/local/bin/production

If installed via Homebrew, what does brew list parity output?

(barberscore-api)bash-3.2$ brew list parity
/usr/local/Cellar/parity/2.4.0/bin/development
/usr/local/Cellar/parity/2.4.0/bin/production
/usr/local/Cellar/parity/2.4.0/bin/staging
/usr/local/Cellar/parity/2.4.0/lib/parity/ (5 files)
/usr/local/Cellar/parity/2.4.0/lib/parity.rb

If installed via Rubygems, what's the gem version?

N/A

I should also mention it has been working flawlessly; I've only been updating application code so I'm not sure what's causing it to fail.  Local Postgres and Heroku Postgres both 10.2

me too

I'm seeing this as well. We saw some automated backups outside of Heroku (not using Parity) fail for a similar reason. I'm not sure what the issue is, but I suspect this is a bigger problem. I'm going to try executing the commands for download and restore separately, if you could all do the same that would be helpful.

I can't download the files with curl using the command we currently use in the code (macOS, High Sierra 10.13.3) due to an error with libcurl. I am now curious if Heroku used to expose these DBs using HTTP and now they are HTTPS. @yun313350095 are you also on macOS?

I am able to download using wget, so that may be the easiest way to fix this beyond having people do a customized installation of curl with Homebrew.

Hi, the issue seems to be that Heroku did an upgrade for a vulnerable patch version of PG 9.6.x. You need to update your local PG version to 9.6.8 or 10.3.x to resolve the issue.
https://www.postgresql.org/docs/9.6/static/release-9-6-8.html

Not for me -- on current versions both local and prod, and have been
throughout.  was the first thing I checked.
On Mon, Mar 5, 2018 at 6:05 AM, Geoff Harcourt<EMAIL_ADDRESS>wrote:

Hi, the issue seems to be that Heroku did an upgrade for a vulnerable
patch version of PG 9.6.x. You need to update your local PG version to
9.6.8 or 10.3.x to resolve the issue.
https://www.postgresql.org/docs/9.6/static/release-9-6-8.html
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/thoughtbot/parity/issues/146#issuecomment-370429144,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJ3urGnXNaZTc1CEBrP1pt25xqiS-Niks5tbUZEgaJpZM4SZZaP
.

now functional again.  not sure what changed, but perhaps was on the Heroku side.  regardless, working and closing.


Materialized Views not accessible

System Info
When specifying a materialized view in include_tables, the following error occurs:
include_tables {'table_name'} not found in database
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[X] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
include_tables=['table1', 'table2']
db = SQLDatabase.from_uri("postgresql://xxx:xxx@xxx:5432/xxx", include_tables=include_tables)
Expected behavior
Materialized views should be accessible.

Hi!.
I noticed this as well.
It looks like there is a change in the sql-alchemy
Changed in version 2.0: For those dialects that previously included the names of materialized views in this list (currently PostgreSQL), this method no longer returns the names of materialized views. the [Inspector.get_materialized_view_names()](https://docs.sqlalchemy.org/en/20/core/reflection.html#sqlalchemy.engine.reflection.Inspector.get_materialized_view_names)method should be used instead.
So maybe near here, one could make a change like this:
diff --git a/langchain/sql_database.py b/langchain/sql_database.py
index fc0a5098..f193cf0c 100644
--- a/langchain/sql_database.py
+++ b/langchain/sql_database.py
@@ -53,6 +53,7 @@ class SQLDatabase:
         self._all_tables = set(
             self._inspector.get_table_names(schema=schema)
             + (self._inspector.get_view_names(schema=schema) if view_support else [])
+            + (self._inspector.get_materialized_view_names(schema=schema) if view_support else [])
         )

         self._include_tables = set(include_tables) if include_tables else set()

Hi,
I faced the same issue initially, it seems you need to send a flag view_support as True (defaults to False) , if you want it to check for materialized views.

Try adding one more parameter (view_support = True) and see if this solves the issue in your case as it did in mine.
db = SQLDatabase.from_uri( "postgresql://xxx:xxx@xxx:5432/xxx", include_tables=include_tables, view_support = True)`

Hi,
I have a similar issue. 'one_all' is a materialized view. When I run:
db = SQLDatabase(engine, schema='one',  include_tables=['one_all'], view_support = True)
I get no errors, yet it actually brings all the tables, and even excludes the materialized view one_all, when I am actually telling to bring only the materialized view. Any suggestion¿?

anyone here?

Hi @micaelarg ,
in db = SQLDatabase(engine, schema='one', include_tables=['one_all'], view_support = True)
what is the schema= 'one' ? and are we giving the view name in include_tables?

psycopg2.errors.UndefinedTable

Any changes this will be merged anytime soon?


Support latest es/os versions

Support latest versions of each engine.
es: https://www.elastic.co/guide/en/elasticsearch/reference/current/es-release-notes.html
os: https://opensearch.org/versions/

Also want to add a workflow to detect new es/os release automatically.

opensearch 2.18.0 supported: #154


[WIP] [#62] improve MinHashLSH-based deduplication for Japanese

 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.


pull from bottom is too hard to refresh，can you tell me how to edit it  thanks

I has used SwipyRefreshLayout in my project, and find a little problem .It's hard to start the refresh when i pull from bottom. Do you have some issues for me ?thanks anyway.

Hello @heiyeliuying
I did not notice it. I guess you can play with the indices (the first numbers in the class) to see if that improves your scroll?
Cheers

@olivierg13 thanks for this useful library.
Is there any way to change REFRESH_TRIGGER_DISTANCE variable?


appliance: fix progress version in Gemfile

Fixes veewee build issue: https://github.com/jedi4ever/veewee/issues/1103

LGTM.  found the same issue this weekend

Additional note: when the issue is hit, gem uninstall progressbar and remove older versions.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-893

Merging this based on separate systemvmtemplate builder job result.


build checkstyle_java faild

ERROR: heron/tools/java/src/com/twitter/bazel/checkstyle/BUILD:12:1: Java compilation in rule '//tools/java/src/com/twitter/bazel/checkstyle:checkstyle_java' failed: java failed: error executing command external/local_jdk/bin/java -Xbootclasspath/p:external/bazel_tools/third_party/java/jdk/langtools/javac.jar -jar external/bazel_tools/tools/jdk/JavaBuilder_deploy.jar ... (remaining 1 argument(s) skipped): com.google.devtools.build.lib.shell.BadExitStatusException: Process exited with status 1.
tools/java/src/com/twitter/bazel/checkstyle/JavaCheckstyle.java:110: error: incompatible types: Predicate cannot be converted to Predicate
return getSourceFiles(extraActionFile, Predicates.not(Predicates.or(
^
tools/java/src/com/twitter/bazel/checkstyle/JavaCheckstyle.java:117: error: incompatible types: Predicate cannot be converted to Predicate
return getSourceFiles(extraActionFile, Predicates.or(
^
Note: Some messages have been simplified; recompile with -Xdiags:verbose to get full output
2 errors

@mtunique - just wondering what platform you are building with?

java version "1.8.0_101"
Java(TM) SE Runtime Environment (build 1.8.0_101-b13)
Java HotSpot(TM) 64-Bit Server VM (build 25.101-b13, mixed mode)
OS: macOS 10.12 (16A320)

@mtunique - perhaps I was not clear, whether you are building with centos, ubuntu or mac? If so, can you specify the version?

ubuntu is also failed.
@kramasamy
java version "1.8.0_101"
Java(TM) SE Runtime Environment (build 1.8.0_101-b13)
Java HotSpot(TM) 64-Bit Server VM (build 25.101-b13, mixed mode)
Linux 4.4.0-36-generic #55~14.04.1-Ubuntu SMP Fri Aug 12 11:49:30 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

@mtunique - I just compiled in mac and everything seems to be ok. I was wondering if you have docker installed in your machine. If so, you can try compiling it in docker and see if that passes. For docker compilation, do the following
docker/build-artifacts.sh ubuntu14.04 master ~/heron-output

and see if it works?

@mtunique - also wondering what command did you give while compiling?

@kramasamy
I run ./scripts/setup-intellij.sh.

@mtunique - oh! ok. Let us try to compile in command line and see if that works in your machine. At the root directory of heron, do the following:
./bazel_configure.py
bazel build --config=ubuntu heron/...

I change   (JavaCheckstyle.java)
  private static String[] getSourceFiles(String extraActionFile,
                                         Predicate<String> predicate) {

to
private static String[] getSourceFiles(String extraActionFile,
                                         Predicate<CharSequence> predicate) {

and
  private static void checkStyle(String[] files, String config) throws IOException {
    if (files.length == 0) {
      LOG.fine("No java files found by checkstyle");
      return ;
    }

to
  private static void checkStyle(String[] files, String config) throws IOException {
    if (files.length == 0) {
      LOG.fine("No java files found by checkstyle");
      return;
    }
```.
It is passed on mac.

@mtunique - ok cool. Can you send a pull request?

@kramasamy ok


CB-18037: Added E2E test for SDX resize recovery from provisioning failure.

Jira: https://jira.cloudera.com/browse/CB-18037
Objective: Add an E2E resize test which performs recovery after a provisioning failure.
For this I needed to make some changes in order to correctly cause a failure during the provisioning step.
Tested this locally along with the other E2E test and verified that everything is working as expected.

Marked this as DO NOT MERGE as the following needs to happen before this can be merged:

SDX resize E2E tests on 2.61.0 need to be passing, this effort is tracked here: https://jira.cloudera.com/browse/CB-18100
#13192 needs to be merged in as this is built on those changes
Need to verify that the changes in #13192 are successful in the real E2E tests

We don't need the above anymore so this can now be merged.


Fix handling of cancelled writes after changes introduced in e286b24e…

…803e645449a10894a706be242388b092
Motivation:
e286b24e803e645449a10894a706be242388b092 changed how Promise.setUncancellabe() behaves but did not take this into acount in the ChannelOutboundBuffer. This could lead to the situation that the internal datastructure of ChannelOutboundBuffer became inconsistent.
Modifications:

Only increment flushed count if the promise was not cancelled before and correctly unlink if it was.
Add unit tests

Result:
No more infinite loop possible while in the write loop and a cancel happens in between.

Let me merge this so I can make progress. @chrisvest let me know if there is any follow needed

@normanmaurer Looks good. Nice to get test coverage for this scenario.


Style: don't do even/odd row coloring with <= 2 parameters

This looks goofy.

Maybe same for projects list?

Actually this is fine, for now at least. Trimming.


Task to lint the whole workspace does not show up

I'm using ESLint 2.1.13 with VSC 1.51.1, I've enabled the option in settings:

but the the only tasks shown are the ones I have defined in my tasks.json (plus the one I recently deleted, which seems to be working as intended):

That works for me.

Can you please provide me with a GitHub repository I can clone that demos what you are experiencing.


scipy.special.binom loss of precision due to floating point

When using scipy.special.binom for moderately large inputs loss of precision develops due to floating point error. I'm not aware of any situation in which floating point values for binomial coefficients are actually needed. Shouldn't there at least be an option to calculate integer binomial coefficients to arbitrary precision in scipy?

Use scipy.special.comb if you want exact integer calculations.
Binomial coefficients occur frequently in special functions and there
floating point implementation is important also for non-integer values
are needed.


Object reference not set to an instance of an object error in application template

When I create new project from Modern UI WPF Application template I get the error in design time (it's build and run ok).
How I can fix it?

VS2013

Rebuild project and reopen file in designer. This should help

Sure, I tried it many time without effect.
ReSharper can't find Heading1, Heading2 etc StaticResources too. Could you check this "bug" on you PC, please?

In my case i don't use ReSharper and XAML UI Designer. I use Source Editor to code and Blend to visualization and i haven't this bug.

ps.
sorry for my poor english

I have the same issue on VS 2015.

Did anyone find a solution for this issue. I am experiencing the same thing using VS2015.

Try: DynamicResource instead StaticResource

Thank you for the suggestion. Yes I tried that, actually, that was one of the first things that I tried, but still got the same results. Unfortunately, the app resource code for that isn't available to me just using the templates. Removing the style reference altogether and running the application gives you a completely blank window so I know the xaml engine can "see" the resource I just think that there are some bugs in the resource itself that cannot be resolved. :(

I just found out that the issue only happens with the XAML Designer. I am guessing it happens as the designer tried to use the resources to render the preview window (which I don't use btw).
If I open the XAML with the Source Code Text Editor as mjasin pointed before the error goes away.
Thanks mjasin, I have been hitting my head on the wall because of this.

I just moved the Style setting to the code behind. I put this line after InitializeComponent();
Style = (Style)App.Current.Resources["BlankWindow"];
Then just removed the style setting from the xaml.

I had the same problem with ModernUI extention. My solution was to edit the App.Xaml Resources:
<Application.Resources>
        <ResourceDictionary>
            <ResourceDictionary.MergedDictionaries>
                <ResourceDictionary Source="/FirstFloor.ModernUI;component/Assets/ModernUI.xaml" />
                <ResourceDictionary Source="/FirstFloor.ModernUI;component/Assets/ModernUI.Light.xaml"/>
            </ResourceDictionary.MergedDictionaries>
        </ResourceDictionary>
    </Application.Resources>
</Application>

Yes. It seems like it is just a designer issue for rendering the controls.


User Interface Design

Abstract: Design the layout and flow of the login page, ensuring it displays all necessary information clearly.
Use a program such as Figma to create mockups for the layout of the login page, as well as submenus such as password reset, etc.
Definition of Done: Figures exist for each state of the login page, and can be easily accessed by stakeholders (on Github, on the CMS website, or otherwise)

After discussion with group D, decided to use preexisting Figma artifacts from ticket 7 as a basis for login page design. Figma mockups can be seen here: https://canvas.ou.edu/courses/314619/files/folder/Artifact/Ticket 7?preview=96830757


Add Authentication via Azure Active Directory

Hi Mark,
this is just the starter project that I have been looking for - an angular 2 client with a .net core server that supports server side rendering.
However, I was wondering whether it was possible to add authentication support via the azure active directory.
So far, I have created an app entry for my server API and configured this via my appsettings.json file and added the following into the startup.cs file.
  app.UseJwtBearerAuthentication(new JwtBearerOptions
  {
    AutomaticAuthenticate = true,
    AutomaticChallenge = true,
    Authority = String.Format(Configuration["AzureAd:AadInstance"], Configuration["AzureAd:Tenant"]),
    Audience = Configuration["AzureAd:Audience"],
  });

When I add the [Authorize] attribute to the controller in the server, it is stopping any browsers calling the API for the data which is wonderful.  However, the call in the api.ts file
    return this._http.get(url, options)
        .map(res => res.json())
        .do(json => { if (isNode) { this._cache.set(key, json); } })
        .share();

is still getting through the authentication regardless of the [Authorize] attribute.  Is there any reason why this is the case?  I would have thought that I would have to integrate some AAD javascript (e.g. ADAL) to authenticate the client with the active directory prior to the call being successful to the server.
Any ideas?
Al

Is this done with the  [ValidateAntiForgeryToken] attribute in the server controllers?  Does this the same effect without the need of linking to active directory?

Let me get back to you soon, apologies swamped this weekend!

Are you providing options to the call there? Headers or anything?

Hi Mark,  Sorry for the delay in getting back to you... The day job is quite hectic at the moment.  I will have another look at this over the next few days and I will let you know if I can get it working correctly.  I have found some other templates that show me how to do it so I will try and integrate it into this template.

I have finally had some time to give this some more thought... and I have been reading around and there may be a better solution to what I need to do .... IdentityServer4.   Does this make any sense and would it be of any interest to you?

Is IdentityServer4 just for ActiveDirectory or other login ID situations?

It can be used with other external login providers...
https://blogs.msdn.microsoft.com/webdev/2016/09/19/introducing-identityserver4-for-authentication-and-access-control-in-asp-net-core/
I was going to develop my application to use ActiveDirectory but I am beginning to consider the benefit of linking in with other identity providers e.g. google/facebook etc.. so if you are interested in adding that functionality into this core system then I would be happy to develop/integrate it so you can get the benefit too

I think having Google/FB/Twitter integration would be huge for most people, plus they could just as easily remove it if they don't need it!

I'm actually going to revert a few recent changes do you want to wait and I'll ping you when we're all set and you can try to add those features? @albrookesplowman

No probs - I have done a clone of a couple of days ago so I will try some bits and pieces in that clone and then, once ready, add it in when I have it working.... fingers crossed... :)

in startup.cs
 public static void Main(string[] args)
 {
   // FOR USING KESTREL
   // var host = new WebHostBuilder()
   //     .UseKestrel()
   //     .UseContentRoot(Directory.GetCurrentDirectory())
   //     .UseIISIntegration()
   //     .UseStartup<Startup>()
   //     .Build();
  //FOR USING WebListener
  var host = new WebHostBuilder()
    .UseContentRoot(Directory.GetCurrentDirectory())
    .UseStartup<Startup>()
    .UseWebListener(options =>
    {
      options.ListenerSettings.Authentication.Schemes = AuthenticationSchemes.NTLM;
    })
    .Build();

  host.Run();
}

this should work, if not make sure the web.config looks like: (I'm pretty sure this is for if you are hosting with IIS) This will not work on anything but windows, this does not need to be hosted on IIS. Not even sure if you CAN host this with IIS atm.
<?xml version="1.0" encoding="utf-8"?>
<configuration>
  <system.webServer>
    <handlers>
     <add name="aspNetCore" path="*" verb="*" modules="AspNetCoreModule"
      resourceType="Unspecified"/>
   </handlers>
   <aspNetCore processPath="%LAUNCHER_PATH%" arguments="%LAUNCHER_ARGS%"
  stdoutLogEnabled="false" stdoutLogFile=".\logs\stdout" forwardWindowsAuthToken="true"/>
 </system.webServer>
 </configuration>

I know you probably already moved on, but there are some adal.js wrappers for ng4 out there that have auth API call wrappers. Here is an example
https://github.com/ranveeraggarwal/ng2-adal-QuickStart

@MarkPieszak I am closing this one since it seems to be dead. We should add auth (AAD, FB, Google+) to the list of add-on modules


Package request: sqlitestudio

SQLiteStudio
A free, open source, multi-platform SQLite database manager written in C++, with use of Qt framework.
Metadata

homepage URL: https://sqlitestudio.pl/
source URL: https://github.com/pawelsalawa/sqlitestudio
More info about running on NixOS: Not able to run the portable version on NixOS #4838
license: GPL v3
platforms: Windows x64, Windows i386, MacOS X x64, Linux x64

Duplicated #126536


Add my name to readme.md file

Fixes or introduces:
Proposed Changes

add my profile

@dwikychandra21 - Please resubmit without the whitespace changes. Thank you!


Show the type of destructuring parameter in case it has type defined

This is a follow-up PR for pull request on main repo - https://github.com/TypeStrong/typedoc/pull/621. This changes the displayed type of destructured parameter:
/**
 * A function with a destructuring argument that implements interface.
 */
export function functionWithTypedDestr({name}: classes.INameInterface) { }

Before change:

After change:

No other test cases are affected by this change.

I was just looking for this. Could it be merged please?


per tenant authentication

The initial concerns of #2 and #3 have been adressed but not revised nor merged.  I'm trasitioning to focus on this issue.


cantata: pin to ffmpeg_6

Things done
ZHF:  https://github.com/NixOS/nixpkgs/issues/352882

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build cantata

@ofborg build cantata

@ofborg build cantata

I'll kick off darwin builds real quick just to double check if it even compiles so we don't have to wait for ofborg.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 353982

x86_64-darwin

aarch64-darwin

@khaneliman Maybe you forget to pull. I have move it to lib.platforms.unix.

Just looks like some dependencies might need to get shifted around to get around their being marked badPlatforms on darwin.

@khaneliman Could you check if this dependecies is only for linux?

@khaneliman Could you check if this dependecies is only for linux by removing it? Or we just leave it as this. If anyone is interested in darwin, they will do such thing.

I think we should just mark it badPlatform until it gets properly setup with all the dependencies.

@khaneliman I just see udisk2 is only an optional dependecy.

@khaneliman I just see udisk2 is only an optional dependecy. I think it may only work on linux.

Upstream notes:

Supports Linux, macOS, Windows, and Haiku. NOTE: Only Linux actively supported as of 2.3.3

They are also an archived repo, so I guess I wouldn't worry about it. Sorry for the hassle.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 353982

x86_64-darwin

:x: 1 package failed to build:

cantata

aarch64-darwin

:x: 1 package failed to build:

cantata

Looks like that worked for triggering the build, but it indeed has a qt build error on darwin:
/tmp/nix-build-cantata-2.5.0.drv-0/source/3rdparty/solid-lite/ifaces/device.cpp:23:10: fatal error: 'QDBusConnection' file not found
       > #include <QDBusConnection>

@khaneliman Done.

@Bot-wxt1221 Thanks, could you split those meta changes into a separate commit, too?

@khaneliman Done.

Thanks, just running a nixpkgs-review for linux real quick. Appreciate your patience.

Without ofborg, nixpkgs-review is too slow. So I just build it manully.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 353982

x86_64-linux

:white_check_mark: 1 package built:

cantata

aarch64-linux

:white_check_mark: 1 package built:

cantata

x86_64-darwin

aarch64-darwin


Remove unused framebuffer code

Hi,
this pull request is a proposal for removing some currently unused code to make the plugin easier to maintain. During my last debugging (searching for the controller latency problem) of the godot_oculus_mobile plugin I noticed that the opengl framebuffer setup in framebuffer.cpp is not used and also not needed. It will just use gpu resources for the unused MSAA zbuffers that are created.
From what I can tell so far godot has an internal setup of the target gl framebuffer for rendering and uses only the target color texture from the swap chain that is created via the VrApi.
This is also the reason why setting MSAA inside the godot_oculus_mobile plugin has no effect.
I also think multiviewsupport will need some changes to the ARVR api; this is the reason why I  removed this setup code from the framebuffer.cpp as it is currently not functional.
The final commit is some small cleanup and renaming of the functions inside framebuffer.h to be more consistent with ovr_mobile_session.h

Looks good! Can you squash the commits then it should be good to merge.
Thanks for the cleanup!

Squashed them. Thanks for the fast response.


Exclude :graalvm-native-image-app from being published

Tested by manually doing the following:

rm -r ~/.m2/repository/org/mongodb
rm -r ./build/docs
./gradlew clean docs publishToMavenLocal -x signMavenJavaPublication

check that ./build/docs/bson exists and ./build/docs/graalvm-native-image-app does not.

check that ~/.m2/repository/org/mongodb/bson exists and ~/.m2/repository/org/mongodb/graalvm-native-image-app does not.

JAVA-5414

Thanks! I agree about javaCodeCheckedProjects, as all the javadocs get generated in the same place, not in build subdirectories of each project. However, I don't think javaCodeCheckedProjects needs to be updated - there is nothing wrong with :graalvm-native-image-app being statically checked. The checks still seem valuable, despite the code not being intended for production.


increase gpg timeout value for in-toto

What does this PR do?
Increase the timeout value for any GPG subprocess to run from in-toto.
Motivation
Developers find the current timeout value (3 seconds) to short to successfully find and enter their PINs.
Review checklist

[ ] PR has a meaningful title or PR has the no-changelog label attached
[ ] Feature or bugfix has tests
[ ] Git history is clean
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

Additional Notes
@zippolyte and @ofek should test this to make sure that the PR is satisfactory.

@zippolyte Thanks! Could you run a quick test on your machine?

Yup I did, it's way better 💯

Codecov Report

Merging #2613 into master will decrease coverage by 6.64%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2613      +/-   ##
==========================================
- Coverage   83.85%   77.21%   -6.65%
==========================================
  Files         644       21     -623
  Lines       36425      610   -35815
  Branches     4426       65    -4361
==========================================
- Hits        30544      471   -30073
+ Misses       4563      113    -4450
+ Partials     1318       26    -1292


Add unused variable highlighting

I think having a highlighting feature for unused variables would be a great addition for better user experience

Thank you for your contribution! It appears you have a regex \\b_([\\w]+[?!]?) for matching unused variables, would \\b_\\w* work or is there an edge case I'm missing?
https://crystal-lang.org/reference/1.8/syntax_and_semantics/assignment.html#underscore


The Baidu map personalization style does not work.

Version
4.9.0
Reproduction link
https://echarts.apache.org/examples/zh/editor.html?c=map-polygon
Steps to reproduce
The Baidu map in the example now has the same problem.
What is expected?
The map background should be blue and so on.
What is actually happening?
The map is no style.

Thanks for your reporting. This is a bug brought by a typo. You can use the bmap extension of 4.8.0 or modify the variable name newMapStyle2 in line 105 to newMapStyle.


Invalid balance and tx list on Chrome

Using Marina commit 108eeed7
Added Marina extension to Chrome and Firefox (via load unpacked)
Created two different wallets on each browser (different mnemonics)
Got 2 L-BTC on Chrome through faucet and sent 1 L-BTC from Chrome to Firefox.
On the Chrome side, balance is showing 0 (should be 1) and the transaction list shows a value of -2 L-BTC (should be -1).
On the Firefox side everything is showing properly (balance = 1 and transaction list shows an inbound of 1 L-BTC)
Esplora shows one of the outputs as non confidential

Left is Chrome, right is Firefox

Steps taken to replicate bug:

Create new wallet on Chrome
Create new wallet on Firefox
nigiri stop --delete
nigiri start --liquid
generate receiving address on Chrome
nigiri faucet --liquid <address from 5>
generate receiving address on Chrome
nigiri faucet --liquid <address from 7>
generate receiving address from Firefox
send 1 L-BTC from Chrome to Firefox (to address from 9)
ERROR: on Chrome, balance is now 0 (should be 1)
ERROR: on Chrome, Tx list shows -2 L-BTC outbound (should be -1)
On firefox things appear ok (balance and tx list)
Esplora shows one output as non confidential

cc/ @louisinger

It's very weird that the second output in the transactions is "unblinded" ie. not confidential.
By default Marina blinds all outputs of the receivers and his own for the change (which heuristically is vout#1 most of the time)

@tiero @bordalix I'm able to reproduce the "blinding error", however I get some weird balances result (only the first "send"). Note that, using the settings/deepRestorer on both wallets I'm able to "recover" the right balance value. I really think there are some regression on the updater which causes some weird behavior during the first send.
@bordalix are u able to "recover" the balances using deepRestorer ? And what happens if u try to send another tx after the buggish one?
PS: The same scenario works well on restricted asset PR where I refactored the updater using redux-saga.

@louisinger after deepRestorer I was able to recover the balance on Chrome.
But the Tx list stills shows the -2 L-BTC Tx (when it was -1 L-BTC).
After a new Tx from Chrome to Firefox of 0.5 L-BTC:
Chrome on the left, Firefox on the right

But the Tx list stills shows the -2 L-BTC Tx (when it was -1 L-BTC).

This is totally normal. The tx data is computed only one time (when fetched) - so unfortunately deepRestorer does not re-compute the "inbound/outbound" values of txs (we may change this but not the problem here I think)

After a new Tx from Chrome to Firefox of 0.5 L-BTC:

So it confirms that the bug only happens during the first send. I think the latest changes on master should fix at least your blinding error. see there: https://github.com/vulpemventures/marina/blob/master/src/application/redux/reducers/wallet-reducer.ts#L54. (-1 becomes 0). I think we should investigate there :thinking:

Can we cherrypick the updater with redux-saga from restricted PR? @louisinger
Currently I tagged new version 0.3.4 with Testnet, let's figure out a way to fix this for a new patch release

If doable to cherrypick (cc @louisinger) I would assign to you @bordalix to try to propose a fix

As agreed offline with @louisinger, we think he would be more effective in fixing this trying to cherry-picking the updater refactor

@louisinger just to let you know, if I change the network to Liquid and then back to Regtest, I get the correct balance. Without the need to do deep restore.

@louisinger just to let you know, if I change the network to Liquid and then back to Regtest, I get the correct balance. Without the need to do deep restore.

Yeah, each time you switch the network type: a deep restorer is launched in order to discover new addresses.

Fixed by #264 ?

Well, the problem still exists if I send some L-BTC to myself:

In this case, I faucet 2 L-BTC and then sent 1 L-BTC to myself.
The balance is 1.9999964 L-BTC (2 - fees Tx), but if I add up all Tx on the list, the result is 2.
@louisinger told me that to show self transfers as a "zero" withdraw it's a voluntary decision, so I think is ok to close it.


Unordered array content on PATCH

Maybe it's a problem on layer 8, but it seems that order of array-content is non deterministic after patch, even when using the same index multiple times.
To Reproduce
Create a new resource like so:
curl 'http://localhost:8090/api/v1/dataresources/' -i -X POST \
    -H 'Content-Type: application/json' \
    -d '{
            "creators" : [{
            "familyName" : "Doe",
            "givenName" : "John",
            "affiliations" : [ "test1", "test2" ]
        }],
        "titles" : [{
            "value" : "Most basic resource for testing"
        }],
        "resourceType" : {
            "value" : "testingSample",
            "typeGeneral" : "DATASET"
        }
    }'

This results in first inconsistency as affiliations will be ["test2","test1"].
Now add another entry to the array (replace ID and ETAG with values of the step before):
curl 'http://<IP_ADDRESS>:8090/api/v1/dataresources/<ID>' -i -X PATCH \
    -H 'Content-Type: application/json-patch+json' \
    -H 'If-Match: <ETAG>' \
    -d '[{
        "op" : "add",
        "path" : "/creators/0/affiliations/0",
        "value" : "test3"
    }]'

Fetch the changed resource:
curl 'http://<IP_ADDRESS>:8090/api/v1/dataresources/<ID>' -i -X GET

Now affiliations is ["test2","test3","test1"], the insertion happened between elements 0 and 1.
Repeating the previous steps (patch, get), results in:

["test4","test2","test3","test1"], this is the only result that is as expected (at least for my understanding).
["test4","test5","test2","test3","test1"], again between elements 0 and 1.
["test4","test5","test2","test3","test6","test1"], insertion between elements 3 and 4.

Expected behavior
Adding [ "test1", "test2" ] leads to [ "test1", "test2" ] added to resource.
Adding an entry respects the path given, for example
{ "affiliations": [ "test1", "test2" ] }
with patch:
[{ "op": "add", "path": "/affiliations/0", "value": "test3" }]
leads to:
{ "affiliations": [ "test3", "test1", "test2" ] }
whereas patch:
[{ "op": "add", "path": "/affiliations/1", "value": "test3" }]
leads to:
{ "affiliations": [ "test1", "test3", "test2" ] }
and patch:
[{ "op": "add", "path": "/affiliations/2", "value": "test3" }]
as well as patch:
[{ "op": "add", "path": "/affiliations/-", "value": "test3" }]
leads to:
{ "affiliations": [ "test1", "test2", "test3" ] }
according to Specification:

An element to add to an existing array - whereupon the supplied value is added to the array at the indicated location. Any elements at or above the specified index are shifted one position to the right.

Version
Tested version 1.4.1-SNAPSHOT as well as 1.5.4-SNAPSHOT with no obvious difference.

I made more tests and it seems that ordering is always the same for same inputs, but differs for other strings, but ordering is not alphabetically.
I also found out that there is no difference in the output when inserting strings at a different position (other index in path), so either index in path is ignored completely or some sorting is applied afterwards.

While this issue has impact in base-repo, it actually relates to kit-data-manager/service-base and src/main/java/edu/kit/datamanager/util/PatchUtil.java. However, also there only a third party library is used where it comes to the following line of code at the end: https://github.com/java-json-tools/json-patch/blob/55b27b96045baa4ef356b2b183304551f64a18e4/src/main/java/com/github/fge/jsonpatch/AddOperation.java#L130
As one can see,  the index is properly evaluated and the new element should be added at the correct position, but there is a "but". Afterwards, the modified resource is written to the basebase and what you see as an "array" is actually represented as Set of strings, which has no index/primary key that can be utilized for sorting as it is the case for sets of complex objects. For strings, only uniqueness of entries, but no specific order is guaranteed from the database side. This indeed leads to unpredictable behaviour in terms of ordering, but I'm afraid it is working as intended. ;-)

Indeed, it's also not possible to add the same value twice (never tried that before), which also points to a set rather than an array.
Of course I would prefer that the order is retained and arrays behave like arrays (there are no sets in JSON specification) and JSON-Patch works like it should according to specification, but this would probably be more work than just adding an additional order/index column to the database.
So at least it should be clearly communicated everywhere in the documentation where these "set-arrays" occur, I used this documentation to dive into this topic and would suggest that you add a prominent notice at the top of the page.

Thanks for the pointer, I've added a note.


[BUG] MA crops and Farmer w/bonemeal

you can use bonemeal in the farmer to grow MA crops. should only be able to use fertilizer on MA crops.

Fixed as of 1.9.15 which will be released shortly, thanks for reporting!


0.1.2 release - publish and point CLI to latest install yaml for Istio, Knative, eventing, bus

The riff cli depends on published copies of the install yaml for Knative components and for Istio. Currently these are manually prepared and published prior to each release.
Let's please capture how this process is done for others on the team to learn or automate in future.

We now point to the files published nightly by the Knative project. The only file we host ourselves is previous/riff-build/riff-build-0.1.0.yaml containing the build-template we use. It is stored in the GCS bucket https://storage.googleapis.com/riff-releases and it doesn't change that frequently.

I kinda hope riff-build is never updated :)

A detailed list of all steps is now in #697


Feature/88 vda name registry multi sign on mumbai

I can't test this due to this error:
Error HH8: There's one or more errors in your config file:

  * Invalid account: #0 for network: polygonmainnet - Expected string, received undefined
  * Invalid account: #0 for network: polygontestnet - Expected string, received undefined

Please update this PR description with the issue it close, more information about the change and how to test.

I can't test this due to this error:
Error HH8: There's one or more errors in your config file:

  * Invalid account: #0 for network: polygonmainnet - Expected string, received undefined
  * Invalid account: #0 for network: polygontestnet - Expected string, received undefined

Please update this PR description with the issue it close, more information about the change and how to test.

This is now working with a .env file.
@ITStar10 will add information in the README.md on how to configure the .env with a sample .env file.

Ok to merge these changes.
@ryank000 and @tahpot will review the multi sig documentation and then the three of us can jump on a call and walkthrough together.


Ramalama Run Doesnt have an Output

A came across an interesting issue where I wasn't getting an output anymore for ramalamam run. I found that in Commit f180b13 model.py line 230 was changed from using exec_cmd to run_cmd. What happens is run_cmd uses subprocess.run which doesn't pipe the output to the terminal (as its piped to stdout) while os.execvp does.
The line can be switched back to return the output back to terminal.
Im wondering if this was changed to fix a bug. If so maybe we can unify exec_cmd and run_cmd? Subprocess.run's output can be on the terminal. I know that there are a few differences to pick one over the other as subprocess creates a child instance while os.execvp overrides the python script.

The reason we used exec in the past, is it simplifies things, so we replace the whole process with a "podman run --rm" or the I containerised version. There's no need to fork another process and it's simpler not to.

Makes sense! Then passing stdout=None will fix it if we don't want to use exec again.
run_cmd(conman_args, debug=args.debug, stdout=None)

I was suggesting we do use exec again, there's no need to fork here and manage two processes, but if we want to do via run_cmd I guess it's fine.

Ah, I misunderstood. I agree we should use exec again instead of run_cmd for running exec_model_in_container. I'm with you, at that stage there is no reason to fork another process once we execute the main command.

If you can change it to exec and it does not break any tests then fine. The issue I was trying to catch is when the llama.cpp or vLLM does not exists in the container, to give a decent error to the user.
Try running a ramalama with a --image fedora and see the error you get if the llama.cpp file is not in the image, if the error looks ok to a naive user, then we can stay with exec.  My goal was to tell them that the image had to have llama.cpp installed in it for ramalama to work with it.

I would take usability over worrying about another fork/exec anyday.

Ah... The lack of errors is probably because we have to redirect stderr to null because of all the llama-cli verboseness. When we move to the recently merged llama-run alternative, we can remove that stderr to null hack. llama-run is also prettier, colorised and has less bugs.

Please open a PR to do this ASAP and then we can get a new release out.

I started playing with Ramalama yesterday and hit this issue as well:) Glad it is already reported and being looked into!


Copy-to-clipboard fix

target.setSelectionRange(0, target.value.length); does not always seem to be
working. It doesn't work once the text is unselected. Using textarea.select()
function: https://developer.mozilla.org/en-US/docs/Web/API/HTMLInputElement/select
Signed-off-by: Ahmet Alp Balkan<EMAIL_ADDRESS>Fixes #3965.

This change is

I am seeing some weird stuff with github today. The source branches and the pull requests are not syncing commits when I update a commit with --amend and push with -f. Anyone else seeing this?

Yeah, @gnufied mentioned a similar problem this morning.

Looks like the commits are updated now. PTAL again.
/assign @chenopis.

/lgtm I tested it on https://deploy-preview-4190--kubernetes-io-vnext-staging.netlify.com/docs/tasks/run-application/run-single-instance-stateful-application/

Thank you for fixing this!


AttributeError: type object 'NewBase' has no attribute 'is_abstract'

System information

-OS Platform and Distribution  =Ubuntu 14.04

**TensorFlow installed =
i mtrying to install tensorflow using following link

https://storage.googleapis.com/tensorflow/linux/cpu/tensorflow-1.1.0-cp27-none-linux_x86_64.whl
Describe the problem
when i am trying to import tensorflow its not working what to do you can check following logs
Source code / logs
root@ubuntu:/home/vivek# python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/tensorflow/init.py", line 24, in
from tensorflow.python import *
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/init.py", line 104, in
from tensorflow.python.platform import test
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/test.py", line 38, in
from tensorflow.python.framework import test_util as _test_util
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/test_util.py", line 45, in
from tensorflow.python.platform import googletest
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/googletest.py", line 33, in
from tensorflow.python.platform import benchmark  # pylint: disable=unused-import
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/benchmark.py", line 119, in
class Benchmark(six.with_metaclass(_BenchmarkRegistrar, object)):
File "/usr/lib/python2.7/dist-packages/six.py", line 617, in with_metaclass
return meta("NewBase", bases, {})
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/benchmark.py", line 114, in new
if not newclass.is_abstract():
AttributeError: type object 'NewBase' has no attribute 'is_abstract'
root@ubuntu:/home/vivek#

This question is better asked on  StackOverflow since it sounds like a setup problem. There is also a larger community that reads questions there. Thanks!


Feature request: yo team project types for SSDT(SQL Server database projects), SSRS, SSAS, SSIS

Seeing as VSTS and TFS are so versatile at dpeloying SQL Server project artefacts, it would be awesome to create these project types for yo team to simplify and get started with SQL Server DevOps pipelines using K8S/AKS, Azure SQL Database and onprem SQL Server, SSIS, SSRS, SSAS instances?

K8s/AKS is being worked on by a group and they will be submitting a PR when they are done. As for SQL do you just want the ARM template to write up the SQL and the app so an empty DB? Or did you want a sample scheme?  The reporting services is not currently ben looked up at. But if you wanted to take a crack at it and submit a PR let me know.

Hi @DarqueWarrior , thanks for the feedback. This is great news!
For SQL, I am up for both
ARM templates will work great for setting up an Azure SQL Database environment if it doesn't exist. This is great for standing up the IaaC, CaaC part.
But additional to this I'm more thinking of deployments to existing SQL Server, SSIS, SSAS and SSRS environments using the project deployment model (.dacpac, .ispac, .rspac, .aspac). So If we could create a yo team template for these it would be awesome as they could leverage off existing extensions in VSTS/TFS or deploying SSIS and DACPAC
I'd love to take a crack at it. This will kinda force me to learn how to create a generator.
Can I just create my own branch or do I have to fork?
Best Regards from South Africa!

Just fork my repo and create a branch in your fork for the SQL stuff. When you are ready you submit a PR from your fork back to my repo. Good luck!

Will do. Thanks a lot!


feat(iac): github action should now use the new CI command

Depends on #628 to be merged for the action-unstable, and deployed for the action to work.
This was not tested yet.
Updates the IaC Github Action to use the new ggshield iac scan ci command instead of ggshield iac scan all.

This changes the behavior of the action: it will now only block if new vulnerabilities are introduced.
Is it ok?

Codecov Report

Merging #646 (7398172) into main (dbf6068) will decrease coverage by 0.24%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #646      +/-   ##
==========================================
- Coverage   93.80%   93.56%   -0.24%
==========================================
  Files         106      108       +2
  Lines        4872     4912      +40
==========================================
+ Hits         4570     4596      +26
- Misses        302      316      +14

Flag
Coverage Δ

unittests
93.56% <ø> (-0.24%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 5 files with indirect coverage changes

LGTM with the change on changelogs.

@Paul-GitGuardian we should update the action files as well, the description will lack some options offered by the CI scan.


[BUG] DeprecationLogger - Unbounded memory usage

Describe the bug
DeprecationLogger dedupes messages that has been already logged based on value of X-Opaque-Id provided in request.
Keys set is maintained in DeprecatedMessage. The entries in set are unbounded and are appended based on the user provided header value X-Opaque-Id. In case the user ends up passing unique Opaque-Id per request it can result in OOM on node.
Related component
Other
To Reproduce

Create a system index in cluster say system-test-index-1
curl -H "X-Opaque-Id: <timestamp-in-milliis>" "https://<endpoint>/_all/_alias/test-alias"
Repeat step 2 for 10K times and you will notice increased heap memory usage

Expected behavior
DeprecatdMessage#keys should be memory bounded and not result in out-of-memory.
Additional Details
Plugins
Please list all plugins currently enabled.
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):

OS: [e.g. iOS]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@rajiv-kv Thanks for raising this, definitely an issue. Want to contribute a fix? Maybe a circuit breaker?

@rajiv-kv Thanks for raising this, definitely an issue. Want to contribute a fix? Maybe a circuit breaker?

Sure @dblock - can explore.

This was introduced in https://github.com/opensearch-project/OpenSearch/pull/1660, @andrross we both missed this. I think the map should both have an expiration and be capped in size (in bytes) + be adjustable.

I think the map should both have an expiration for its items and be capped in size (in bytes) + be adjustable.

@dblock I think a fixed-sized LRU cache with no expiration would probably be fine? If you happen to fill up the cache then you start evicting the least recently accessed items, potentially leading to duplicated log messages for those keys. I'd definitely like to avoid the complexity of making it adjustable. We probably don't need to overthink this because the goal is to de-dupe most messages to avoid spamming the log.

Actually, I wonder if this is a good application for a bloom filter? A small amount of false positives are probably tolerable, because they just lead to duplicate log messages.

@andrross  Yes, the first thing is that it should be bounded so that it doesn't keep on growing. But, I would like to understand the logic behind opaque-id to be appended, if all requests have unique id it doesn't help at all.

Is there a way for user to see the X-Opaque-Id in logs ? I am trying to understand if there is any downside if we track the messages at per feature (Key) rather than per request (KeyWithXOpaqueId).

@dblock @andrross Thanks for updating the documentation and stating the usage of X-Opaque-Id header. But this may not be enough, we need to audit all the official clients like OpenSearch-hadoop to ensure it is used correctly. In case, a user gets into trouble simply because the opensearch clients are mis-using the header, that's not good experience.


Fix a false positive for Layout/SpaceAroundOperators

This PR fixes a false positive for Layout/SpaceAroundOperators when upward alignment. Two false positive examples are shown.

@output and @logger assignments are aligned with the same margin

@integer_message = 12345
@output  = StringIO.new
@logger  = Logger.new(@output)

expected and tagging assignments are aligned with a blank line in between

expected = posts(:welcome)

tagging  = Tagging.all.merge!(includes: :taggable).find(taggings(:welcome_general).id)
assert_no_queries { assert_equal expected, tagging.taggable }

I got this issue feedback from @kamipo. Thank you!
cf: https://github.com/rails/rails/pull/36943

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

I think the docs should be expanded to cover this. In particular I'm not quite sure that alignment should stretch over blank lines, as they are typically used to separate logical blocks, although on the other hand it's hard to imagine there are two logical blocks if the end of the previous and the begining of the next block are assignments. Tricky case, but I'm willing to accept this change if we explain the behaviour clearly.

Thank you for your feedback. I've added the doc for this change.
Let me add a little explanation. This was a potential (breaking) change vertical alignment consisting of one or more whitespace around the operator at https://github.com/rubocop-hq/rubocop/pull/7211, for part of users it was an unexpected change.
e.g. https://github.com/rails/rails/pull/36943#issuecomment-521659529
On the other hand, I agree that there are cases where https://github.com/rubocop-hq/rubocop/pull/7211 behavior is preferred rather than vertical alignment. So, It may be better to make it configurable in future.

We should also add good examples with vertical alignment but this can be done later. Thanks for working on this!


Adds PhyloreferenceUsingApomorphy class and apomorphy property

See #39 and #40 for proposal and discussion threads. Class and property are folded into the same changeset here because invariably one depends (at least semantically) on the other.
Closes #39. Closes #40.

Note to self: remember to change base branch to master before merging.

How about now, @gaurav?

Perfect!


Support UVINDEX

Is your feature request related to a problem? Please describe.
I like to have a forecast of the UVINDEX from current day.
Describe the solution you'd like
Provide sensors to show UVINDEX like.

Current UV
Max UV of this day
Min UV of this day

I found some UV index data from DWD. However, these are only available as a single value for the whole day

Thanks, yes. I looked into the app. Seems one value per day is correct.

It is now implemented.

Hey, I am happy with the forecast. I am using the UV-Index to notify for sun blocker. But today the home assistant integration shows an index of 3 but the dwd android app displays just max 2. Is there a way to evaluate this differences?

@panhans Yes, you could evaluate the raw data here: https://opendata.dwd.de/climate_environment/health/alerts/uvi.json
The integration automatically picks the closest location based on your configuration.


New+ - can we input a variable when creating folders?

Description of the new feature / enhancement
Rather than just duplicating the folder names in the templates area, can New+ be enhanced to allow a variable name to be added at the beginning or end of each folder? So for example, if I have a folder called "Project", it would allow me to enter a variable name called "Holiday in Turkey" and the resultant folder would be "Holiday in Turkey Project". This variable should also propogate to any sub folders underneath "Project".  e.g. "Project has sub folders called JPG, RAW and Finished, each of them would have the variable appended or prepended to them to make them unique.
Scenario when this would be used?
I would otherwise end up with a lot of folders with the same name and would need to edit them to make them unique.
Supporting information
See above.

+1
I am not sure if current New+ can do this, though It would be a great feature to have.
I have my own small bat file to do same thing, doing it from New+ would be amazing!

Maybe things like this, can be implemented through autorunning a macro aptly named inside the folder.
Also, some auto-integers could be added, like a sortable date. So if I have a "yyyymmdd - Project" folder, it could become "20241007 - Turkish Project"

I like that idea too.
At least we've given the developers something to think about.

Also, some auto-integers could be added, like a sortable date. So if I have a "yyyymmdd - Project" folder, it could become "20241007 - Turkish Project"

I think this feature is partly implemented already by PowerRename after 0.73 (see #3814 for reference), where such system works with a Regex form-factor.
Maybe it is simple enough to port/abstract from?

Thank you for the feedback and idea! Currently working on this. This might never ship but hope to have this included after New+ Windows 10 support potentially lands in an upcoming PowerToys release.
Working on adding support for:

Date and time component variables, such as $YYYY, $YY, $MM, $DD,  - using the same logic as in PowerRename
$PARENT_FOLDER_NAME - will replace with the name of the parent folder
%environment-variable% support - will replace with the value of the corresponding environment variable

Example:
Template filename:
$YYYY-$MM-%DD, $hh - $PARENT_FOLDER_NAME by %USERNAME%
Filename would become:
2024-11-22, 12 - PowerShell project by cgaarden
If you have further ideas/feedback, please share!
Thank you again!


[BUG] TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'

/Users/athena/Desktop/athena/sam_altman.py
athena@Athenas-MacBook-Pro athena % /usr/bin/python3 /Users/athena/Desktop/athena/sam_altman.py
/Users/athena/Library/Python/3.9/lib/python/site-packages/urllib3/__init__.py:35: NotOpenSSLWarning: urllib3 v2 only supports OpenSSL 1.1.1+, currently the 'ssl' module is compiled with 'LibreSSL 2.8.3'. See: https://github.com/urllib3/urllib3/issues/3020
  warnings.warn(
swarms is up to date!
Traceback (most recent call last):
  File "/Users/athena/Desktop/athena/sam_altman.py", line 6, in <module>
    from swarms import Agent, ChromaDB, OpenAIChat
  File "/Users/athena/Library/Python/3.9/lib/python/site-packages/swarms/__init__.py", line 7, in <module>
    from swarms.agents import *  # noqa: E402, F403
  File "/Users/athena/Library/Python/3.9/lib/python/site-packages/swarms/agents/__init__.py", line 1, in <module>
    from swarms.agents.agent_wrapper import agent_wrapper
  File "/Users/athena/Library/Python/3.9/lib/python/site-packages/swarms/agents/agent_wrapper.py", line 1, in <module>
    from swarms.structs.agent import Agent
  File "/Users/athena/Library/Python/3.9/lib/python/site-packages/swarms/structs/__init__.py", line 1, in <module>
    from swarms.structs.agent import Agent
  File "/Users/athena/Library/Python/3.9/lib/python/site-packages/swarms/structs/agent.py", line 15, in <module>
    from swarms.memory.base_vectordb import AbstractVectorDatabase
  File "/Users/athena/Library/Python/3.9/lib/python/site-packages/swarms/memory/__init__.py", line 7, in <module>
    from swarms.memory.lanchain_chroma import LangchainChromaVectorMemory
  File "/Users/athena/Library/Python/3.9/lib/python/site-packages/swarms/memory/lanchain_chroma.py", line 10, in <module>
    from swarms.models.openai_models import OpenAIChat
  File "/Users/athena/Library/Python/3.9/lib/python/site-packages/swarms/models/__init__.py", line 32, in <module>
    from swarms.models.openai_models import (
  File "/Users/athena/Library/Python/3.9/lib/python/site-packages/swarms/models/openai_models.py", line 228, in <module>
    class BaseOpenAI(BaseLLM):
  File "pydantic/main.py", line 178, in pydantic.main.ModelMetaclass.__new__
  File "pydantic/typing.py", line 400, in pydantic.typing.resolve_annotations
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/typing.py", line 290, in _eval_type
    return t._evaluate(globalns, localns, recursive_guard)
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/typing.py", line 546, in _evaluate
    eval(self.__forward_code__, globalns, localns),
  File "<string>", line 1, in <module>
TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'
athena@Athenas-MacBook-Pro athena %

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Hi starshipvc,
This appears to be coming from "/Users/athena/Desktop/athena/sam_altman.py".
Would you be able to share a minimal reproducable example?
https://en.wikipedia.org/wiki/Minimal_reproducible_example
Please ask any questions you may have,
Evelyn Mitchell


Implementing Uart read

Implements embedded_hal::serial::Read for UART

CI error should be fixed. The code should work, I tested it on a Basys3. I hope the code also helps others ^^

Thanks a bunch, glad my funny project continues to be useful to people :)


Synonyms that equal a label should be removed

See here
prefix owl: <http://www.w3.org/2002/07/owl#>
prefix oboInOwl: <http://www.geneontology.org/formats/oboInOwl#>
prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#>

SELECT ?term ?related
WHERE
{
  {
    ?term oboInOwl:hasRelatedSynonym ?related ;
      rdfs:label ?related ;
      a owl:Class .
      [
         a owl:Axiom ;
           owl:annotatedSource ?term ;
           owl:annotatedProperty oboInOwl:hasRelatedSynonym ;
           owl:annotatedTarget ?related ;
           oboInOwl:hasDbXref ?xref ;
      ] .

    FILTER (isIRI(?term) && regex(str(?term), "^http://purl.obolibrary.org/obo/MONDO_"))
  }
}

There are about 500 such cases. The solution should be easy:
Just delete the synonym! (Right @cmungall?)
Thank you @kshefchek for reporting.

we should make them EXCLUDE
Synonym to be removed from public release but maintained in edit version as record of external usage

per discussion w/ @matentzn, we will flag these all for exclusion

exact/label is not a major problem.. Lets ignore that. Any other one however is a problem.


Suggested front page

Added missing line on files. How do we push to the live instance?
So sorry to ask again. I will ensure to document properly this time.

This is a broken pull request. Closing and reopening a new one.


Pressing Enter changes columns after 2/3 rows

In the Demo Press F2 on Row 3, Col 3 and go on pressing Enter
at some point of time , cursor moves to col4 and pressing Enter again it moves to Col5 and so on.
Can you please check and let us know whether it is intended behaviour based on some condition
or is it a bug.

Cursor moving to col4 is intentional, since col3 has been set uneditable in the demo. Atleast in Vaadin 8 version the enter key goes down by pressing Enter on col4, so it is working correctly since the extension was instantiated as new FastNavigation<>(this, false, true); where the changeColumnOnEnter = false. I also tested another setting FastNavigation<>(this, true, true); and then enter changed the column as it should.

Cannot reproduce


⚠ Bofrost.fr has degraded performance

In 46ff2e0, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1991 ms

Resolved: Bofrost.fr performance has improved in 6005bac after 4 minutes.


Add set up of tenv shell completion to Homebrew formula

As of now, tenv has completion support built in. tenv completion can be used to generate completion scripts for various shells.
There is also a formula for tenv in Homebrew. See here.
Formulae can be used to set up completion. This is currently not done for tenv.
This is the idea: Add set up of tenv shell completion to Homebrew formula
How could it be done?
The install method in the formula definition needs to be edited.
For the awscli I find for example this:
rm bin.glob("{aws.cmd,aws_bash_completer,aws_zsh_completer.sh}")
bash_completion.install "bin/aws_bash_completer"
zsh_completion.install "bin/aws_zsh_completer.sh"
(zsh_completion/"_aws").write <<~EOS
  #compdef aws
  _aws () {
    local e
    e=$(dirname ${funcsourcetrace[1]%:*})/aws_zsh_completer.sh
    if [[ -f $e ]]; then source $e; fi
  }
EOS

source

https://github.com/Homebrew/homebrew-core/pull/172289

@trallnag good idea! Thank you for MR to homebrew core!


New release?

I was wondering if there would be a new release of com-rs to crates.io, that includes the fixes from #207 and #209?

Hi there! We're likely to release a new version in the coming weeks. However, we're currently trying to better figure out how com-rs and windows-rs fit together. Thanks for having patience while we figure that out!


HTML builtin favicon behavior inconsistent with html-webpack-plugin

System Info
System:
OS: Windows 10 10.0.19045
CPU: (16) x64 11th Gen Intel(R) Core(TM) i9-11900H @ 2.50GHz
Memory: 16.57 GB / 31.73 GB
Binaries:
Node: 18.15.0 - C:\Program Files\nodejs\node.EXE
npm: 9.5.0 - C:\Program Files\nodejs\npm.CMD
Browsers:
Edge: Spartan (44.19041.1266.0), Chromium (112.0.1722.48)
Internet Explorer: 11.0.19041.1566
Details
When using the html-webpack-plugin and supplying a path to a favicon property, the file at that path is copied into the output folder and the resulting html file has the favicon link pointing to the path of where the file was copied into the output folder.  This works whether providing a relative or an absolute path.
When using builtins.html, the behavior is different.  If providing an absolute path, the final html output will reference that full absolute path which is likely invalid since that path might not be available in the final runtime.  If providing a relative path, the folder structure to obtain that relative path will be copied into the output directory prepended with a forward slash.  However, the path separator character remains a backslash (likely due to Windows path separators).
For comparison purposes:
Webpack Absolute Path
Input: favicon: path.resolve(process.cwd(), 'static', 'favicon.ico'),
Output: <link rel="icon" href="/favicon.ico">
Rspack Absolute Path
Input: favicon: path.resolve(process.cwd(), 'static', 'favicon.ico'),
Output: <link rel="icon" href="/C:\Users\<hidden>\<hidden>\<hidden>\<hidden>\<hidden>\static\favicon.ico"
Note: All the path segments marked as <hidden> are to avoid leaking my folder structure into this issue log.  Each one replaces a subdirectory that was present in the output path.
Webpack Relative Path
Input: favicon: path.join('static', 'favicon.ico'),
Output: <link rel="icon" href="/favicon.ico">
Rspack Relative Path
Input: favicon: path.join('static', 'favicon.ico'),
Output: <link rel="icon" href="/static\favicon.ico" />
Although the Rspack relative path behavior should work, I would argue that the webpack behavior is preferable as your input folder structure doesn't necessarily need to align with the output folder structure.  With respect to Rspack's absolute path behavior, the current behavior doesn't work at all since the icon is not in the output folder and the path it uses is unlikely to be available in the runtime environment.
Reproduce link
https://github.com/shawnmcknight/test-rspack-favicon
Reproduce Steps

Set favicon property in builtins.html to an absolute path
Run build process
Inspect build output folder's index.html file and look at the favicon <link> which has been generated.

Thanks for reporting. We would look into it. If you have time to create a reproduction repo using https://github.com/web-infra-dev/rspack-repro, that would be more helpful for us to investigate the issue.

Did you try using @rspack/plugin-html as a workaround?

Did you try using @rspack/plugin-html as a workaround?

I just tried it and it does seem to work properly with respect to the favicon behavior, although I think the builtin should probably work too if it intends to accept a favicon property.  It would be preferable to avoid adding plugins if I can avoid them.
Some things feel a bit off in that package though in terms of usability:

The package has a direct dependency on @rspack/core.  I would think this would be preferable as a peer dependency, but maybe there's a reason why it needs to be a direct dependency?

I happen to be using the @rspack/cli nightly build, but with this being a direct dependency, I now end up with 2 versions of @rspack/core instead of just reusing the existing version.  It's giving me type errors that I think are a result of two different versions of @rspack/core being resolved.

The package is built as cjs, but only provides a simulated default export for the main class.

This causes issues with using it via both require and import.  If I use it via require that means I need to do:
const { default: HtmlRspackPlugin } = require('@rspack/plugin-html');

That doesn't seem like that's the intended way to consume it.
If I'm using it via import in an esm module, I need to:
import HtmlRspackPlugin from '@rspack/plugin-html';
...
	const plugins = [
		new HtmlRspackPlugin.default({
...

which also doesn't seem like the intended way to consume it.
This problem can be illustrated by the arethetypeswrong tool.
It probably makes sense to log separate issues for those, so I will do so.

I've added a reproduction repo at https://github.com/shawnmcknight/test-rspack-favicon. It creates output with both rspack and webpack with the favicon using both relative and absolute paths.

Thanks for reporting. i will fix it recently.

builtins.html was replaced with rspack.HtmlRspackPlugin, which unfortunately suffers from this same issue.
I was able to work around this by following the recommendations in this discussion.  However, @rspack/plugin-html is now deprecated with a recommendation for using html-webpack-plugin which has optional peer dependencies for @rspack/core or webpack.  Using html-webpack-plugin works.


Websockets frontend

Made websockets in the frontend.
I wish I had another week to make this a bit more abstract, as it's a bit ugly now, but it is what it is. We chose for flexible & efficient sockets over hard-refreshing the whole page (like another group that I won't name does), which comes with a more complex implementation behind the scenes.
To test, you can simply open two windows next to each other (can be on the same account, doesn't matter) and make some changes to students & projects. Just make sure you're on the right edition, because #524 hasn't been merged yet as of now.

Very satisfying!
Some small issues:

Suggestions field (with motivations) doesn't change when a coach makes a suggestion
When the confirmed filter is set (to yes) and another admin makes a final decision (yes), the new "yes"-student doesn't show up in the filtered studentslist

I'll take a look!

Very satisfying!
Some small issues:

Suggestions field (with motivations) doesn't change when a coach makes a suggestion
When the confirmed filter is set (to yes) and another admin makes a final decision (yes), the new "yes"-student doesn't show up in the filtered studentslist

I have returned.
The first one should work, can't reproduce the issue. It's possible that it takes a second, or that you have to be focused on the tab. This is what I get:
https://user-images.githubusercontent.com/60451863/169592090-f18c35b1-31c9-466f-8bec-3ba325f7ef8d.mp4
The second one is a bit harder, because I'd have to match the student against all of the current filters. This also includes things like only students you've suggested for, which means I'd have to send an API call. Do we want to do this for every single websocket event?

I think this would give conflict with the infinite scrolling? If not then it might be worth it

Codecov Report

Merging #534 (77daf73) into develop (74bcefc) will decrease coverage by 0.51699%.
The diff coverage is n/a.

:exclamation: Current head 77daf73 differs from pull request most recent head c54c678. Consider uploading reports for the commit c54c678 to get more accurate results

@@                 Coverage Diff                 @@
##             develop        #534         +/-   ##
===================================================
- Coverage   97.00655%   96.48956%   -0.51700%
===================================================
  Files             86          86
  Lines           2138        2108         -30
===================================================
- Hits            2074        2034         -40
- Misses            64          74         +10

Impacted Files
Coverage Δ

backend/src/database/crud/projects.py
89.39394% <0.00000%> (-10.60606%)
:arrow_down:

backend/src/app/routers/users/users.py
92.68293% <0.00000%> (-7.31708%)
:arrow_down:

backend/src/app/logic/projects.py
82.50000% <0.00000%> (-4.16667%)
:arrow_down:

...kend/src/app/routers/editions/projects/projects.py
97.77778% <0.00000%> (-2.22222%)
:arrow_down:

backend/src/app/routers/editions/editions.py
80.85106% <0.00000%> (-1.14894%)
:arrow_down:

backend/src/app/routers/login/login.py
84.21053% <0.00000%> (-0.78947%)
:arrow_down:

backend/src/database/models.py
100.00000% <0.00000%> (ø)

backend/src/app/schemas/login.py
100.00000% <0.00000%> (ø)

backend/src/app/logic/editions.py
100.00000% <0.00000%> (ø)

backend/src/database/crud/users.py
100.00000% <0.00000%> (ø)

... and 7 more


Change where we cache the package finder to alleviate issue collecting hashes with multiple indexes.

Fixes #6249
The fix
My theory is that caching the package finder property was wrong, because it uses other attributes which may yield multiple finders in the case of multiple indexes.   So I moved the point at which the caching takes place.
The checklist

[X] Associated issue
[ ] A news fragment in the news/ directory to describe this fix with the extension .bugfix.rst, .feature.rst, .behavior.rst, .doc.rst. .vendor.rst. or .trivial.rst (this will appear in the release changelog). Use semantic line breaks and name the file after the issue number or the PR #.

If this truly fixes it -- well cache was introduced in 3.9 but in October (~8 days away) we can actually drop Python 3.8 support because it goes EoL.

After reviewing the old index compatibility finder patch, I see I missed porting over part of the old patch.   First order of business is seeing if this fixes the issue --if so I can backport this patch when I have  to redo the patching for 24.2

closed in favor of:  https://github.com/pypa/pipenv/pull/6276


CLI command to delete disconnected services

Jira Ticket: https://chefio.atlassian.net/browse/A2-910

Just like https://github.com/chef/automate/issues/858 but delete the disconnected services from the database.


How to upgrade capacitor version  3?

I need to upgrade capacitor v3 from v2 .any docs for that?

we can update manually in package.json ,
i know,i need to configure twice times to update v2 to v3 however
i want to run vue add capacitor
then i reinstall latest packages again .
i feel better .if package option given in vue add capacitor while run (choose v2, choose v3)

i am facing issue  after i updated v3
it working =>npx add run android
but when i run npm run capacitor:serve
it launching android studio but showing error
Unable to find method 'org.gradle.api.artifacts.result.ComponentSelectionReason.getDescription()Ljava/lang/String;'.
full error

Unable to find method 'org.gradle.api.artifacts.result.ComponentSelectionReason.getDescription()Ljava/lang/String;'.
Possible causes for this unexpected error include:
Gradle's dependency cache may be corrupt (this sometimes occurs after a network connection timeout.)
Re-download dependencies and sync project (requires network)

The state of a Gradle build process (daemon) may be corrupt. Stopping all Gradle daemons may solve this problem.
Stop Gradle build processes (requires restart)

Your project may be using a third-party plugin which is not compatible with the other plugins in the project or the version of Gradle requested by the project.

In the case of corrupt Gradle processes, you can also try closing the IDE and then killing all Java processes.


Ability to let response pass through for a given location

We got a setup where the main site uses a reverse proxy to serve a different site under a specific location. This sub-site has it's own 404-pages that should be shown, and in a previous version of the 404-handler this wasn't a problem as we set httpErrors errorMode="Custom" existingResponse="PassThrough" under the specific <location> tag in web.config.
We noticed however that this doesn't work anymore as we get an empty 404-response instead of the original response from the sub-site, and we suspect it's due to SetStatusCodeAndShow404 now indirectly running Response.Clear() by calling HttpContextBase.SetStatusCode(StatusCode) which didn't happen previously.
Commit with the change
Is there anyway that we can accomplish the desired of behaviour of letting the 404-handler handle redirects etc for the entire site, but disable it (or pass through the original response) in some part of it?

The mentioned change was just refactoring that was doing the same as before but using custom extension methods for better readability.
Is this still an issue? How did it work before then? Could it be that your setup was different?
Now there is no way to disable the 404 handler for specific URLs or sections.


On add one quarter or month lossing one hour

Description of the Issue and Steps to Reproduce:
If you add 1 quarter or 1 month to your time, then you lose 1 hour.
Environment:
Mac OS,  node v6.3.1, moment v2.17.1
Example:
var moment = require('moment');
console.log(moment(new Date('2017-03-01')).add(1, 'Q').toDate());
console.log(moment(new Date('2017-03-01')).add(1, 'M').toDate());

2017-05-31T23:00:00.000Z
2017-03-31T23:00:00.000Z

But if you use utc time then it add correctly.
var moment = require('moment');
console.log(moment.utc(new Date('2017-03-01')).add(1, 'Q').toDate());
console.log(moment.utc(new Date('2017-03-01')).add(1, 'M').toDate());

2017-06-01T00:00:00.000Z
2017-04-01T00:00:00.000Z

With other keys y,w,d it works correctly too:
console.log(moment(new Date('2017-03-01')).add(1, 'w').toDate());
console.log(moment(new Date('2017-03-01')).add(1, 'd').toDate());
console.log(moment(new Date('2017-03-01')).add(1, 'y').toDate());

console.log(moment.utc(new Date('2017-03-01')).add(1, 'w').toDate());
console.log(moment.utc(new Date('2017-03-01')).add(1, 'd').toDate());
console.log(moment.utc(new Date('2017-03-01')).add(1, 'y').toDate());

2017-03-08T00:00:00.000Z
2017-03-02T00:00:00.000Z
2018-03-01T00:00:00.000Z
2017-03-08T00:00:00.000Z
2017-03-02T00:00:00.000Z
2018-03-01T00:00:00.000Z

It's not losing an hour; your timezone is. Here's a closer look at what's happening:
> date = new Date('2017-03-01') //this creates a time using this UTC date
'2017-03-01T00:00:00.000Z'
> var start = moment(date);
> start.format() //which locally is some other time
'2017-02-28T19:00:00-05:00'
> start.add(1, 'M').format(); //we add a month and get back that same time (19:00), but note the offset change!
'2017-03-28T19:00:00-04:00'
> start.add(1, 'M').toISOString(); //here is back in UTC, one hour earlier
'2017-04-28T23:00:00.000Z'

So you took a UTC data, added a month to it in local time, and translated it back to UTC. In the process, Moment accounted for a DST because it's trying to make sure that the local times stay consistent across the local date addition. That dropped an hour.
The lesson here is to do the calculation in the timezone you actually want the answer in. (Also, since you're using Node, you should run the whole process in UTC).


Blocked google icons

Despite the fact that  googleadapis.l.google.com  is included in the whitelist, it is still blocked and is present in the host.
How to fix it? This happened after yesterday's update.

You mean after upgrading hBlock to version 3.0.0?
This version introduces some breaking changes, in this particular case the file /etc/hblock.d/whitelist.list is now /etc/hblock/allow.list.

The rest of the lists should also be moved to /etc/hblock/ ?
Do they have the same names or do they also need renaming?
Where can I read about new locations and correct file names?

Yes, the other paths have also changed.
You can check out the new paths by running hblock --help or by looking at the documentation in GitHub.

Да, изменились и другие пути.
Вы можете проверить новые пути, запустив hblock --helpили просмотрев документацию на GitHub .

Thanks for the quick response!
This solved my problem - now everything works as it should)
Thanks!


[FSDE-1071] Add interactive documentation examples of chi vue pagination

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-115/1/. :white_check_mark:

You can check this PRs instance in https://ed68404c93cb9a3b2cef26385bd3104bad82b0c1-chi-pr.enterprise.ctl.io

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-115/2/. :x:

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-115/3/. :x:

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-115/4/. :white_check_mark:

You can check this PRs instance in https://126fd77a8ecced5940b9b3b6285190bf01d0c797-chi-pr.enterprise.ctl.io

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-115/5/. :white_check_mark:

You can check this PRs instance in https://d262f6f25440a50d2ed18401879344c69b9a42b0-chi-pr.enterprise.ctl.io


235 develop primary blazor components

@TheHadiAhmadi The first page is:

and the kitchen-sink is:

Is this what you see?

I see

and

are you sure css file is not cached?
in changed files I can see f-alert, ... classes which are not applied in KitchenSink page yet

Home page is correct and I didn't worked on it yet


[GO] array.Binary and array.String should use int64 offsets.

Describe the bug, including details regarding any error messages, version, and platform.
LargeBinary and LargeString use int64 offsets, however Binary and String types use int32 offsets, this makes them susceptible to slice index out of bounds errors when the column/array is larger than ~2GB ~= 2^31 bytes.
To reproduce try deserializing a parquet file that is greater than 2.2 GB.
A workaround is to force the go library to deserialize the field/column as LargeBinary instead of Binary:

explicitly store the arrow schema during write. see store_schema https://arrow.apache.org/docs/cpp/parquet.html#roundtripping-arrow-types-and-schema
and schema explicitly uses the large_binary or large_string type when defining the schema that is used to write the parquet files.

Error looks like:
panic: runtime error: slice bounds out of range [:-2147483014]

goroutine 95 [running]:
github.com/apache/arrow/go/v17/arrow/array.(*Binary).Value(...)
	/go/pkg/mod/github.com/apache/arrow/go/v17@v17.0.0/arrow/array/binary.go:59
github.com/apache/arrow/go/v17/arrow/array.(*Binary).ValueStr(0xc000178d20?, 0xc091402a00?)
	/go/pkg/mod/github.com/apache/arrow/go/v17@v17.0.0/arrow/array/binary.go:67 +0xfa
extractorvalidator/data.BootstrapRecordsFromParquet({0x1de1a40, 0xcc6a9775f0}, 0x0)
	/.../data/records.go:78 +0x582
main.validationWorker({0x1dccd90, 0x2c31840}, 0x0?, {0x0?}, 0xc0000315e0, 0xc000001de0, 0xc0000fe9c0)
	/.../command.go:428 +0x125
created by main.RunValidateCmd in goroutine 1
	/.../command.go:174 +0xb90

version and platform
Arrow Version: github.com/apache/arrow/go/v17 v17.0.0
Platform: Linux 20.04.1-Ubuntu  x86_64 x86_64 x86_64 GNU/Linux

Component(s)
Parquet, Other

There are a couple ways we can adjust to do this that I can think of, depending on your use case:

We could add an option to the ArrowReaderProperties to explicitly always use LargeString/LargeBinary for strings (eliminating the need for the workaround). This requires a user to know ahead of time that they need to use LargeString/LargeBinary, which may not be feasible or the best route.
Are you using ReadTable? Or are you streaming the records? We could check the size of the column data ahead of time and force it to split records based on the column size, that would avoid this problem. This would require a bit of extra up-front work to do the checking, but allows seamless record reading without the user needing to know ahead of time.

Thoughts?

Thanks for the response @zeroshade, I am using arrow/go/v17's pqarrow.ReadTable to read the records from parquet files.
Wouldn't using int64 offsets instead of int32 be the simplest approach?

I don't want to change the current default as it's possible users may be relying on the current behavior that defaults to using String and Binary instead of their Large variants.
If both approaches are reasonable, then I think I'll first simply add the option to force LargeString/LargeBinary as that is much easier to do. I'll wait on attempting the automatic splitting until someone specifically requests that. I'll try to get to this in the next week or two.

Okay, this sounds good to me; thanks!


Enable primary and multiple KB configuration (static)

See https://grips.semantic-web.at/x/cYVlB for details.

Please do not forget to also adjust the sample config files. I cannot see any adjustments to https://github.com/odalic/sti/blob/develop/config/kbsearch.properties

Please do not forget to also adjust the sample config files. I cannot see any adjustments to https://github.com/odalic/sti/blob/develop/config/kbsearch.properties

Maybe I was not clear enough. I have checked out the newst branch with modifications by @JanVana from https://github.com/odalic/sti/pull/107 . It seems fine to me and does contain the mentioned adjustments. But I have to yet accept the pull request to develop and I will do so as soon as possible (after today Skype meeting and maybe some additional tests).

Postponed until 2016-10-08 to allow some polishing of the code.


Added missing German translations and replaced some terms with more common ones

Now in sync with current master.

Coverage remained the same at 98.9% when pulling 0e43013124dea0a4b3dce3750b9b5dfec4ba78e5 on schneikai:master into c2f1d1e6c17a1bd082c2b9772019aaad366d2ee9 on comfy:master.

Coverage remained the same at 98.9% when pulling 0e43013124dea0a4b3dce3750b9b5dfec4ba78e5 on schneikai:master into c2f1d1e6c17a1bd082c2b9772019aaad366d2ee9 on comfy:master.

:+1: