[Feature] SDK generators should create a user agent

Is your feature request related to a problem? Please describe.
Today a Fern generated SDK does not create a user-agent.
Each generated SDK should send a User-Agent header that is formatted with something like
<package>/<language>/<version>

In practice, this may look like
cohere-ai/typescript/7.2.0
com.cohere/java/1.0.0

Describe the solution you'd like
Step 1: Introduce the user agent field into the IR
Step 2: Update the code generators to use the user agent field to send headers
Additional Context
MDN: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/User-Agent
gRPC: https://grpc.github.io/grpc/core/md_doc__p_r_o_t_o_c_o_l-_h_t_t_p2.html

hey can I work on this?

@kisharnath sure:
Steps you'll need to take:

Add a userAgent field to the intermediate representation (https://github.com/fern-api/fern/tree/main/fern/ir-types-v29)
Populate the field in generateIntermediateRepresentation.ts
After that merges, new SDKs for the Intermediate Representation will be published
Upgrade the Python/Java/Go/TypeScript generators to read this new field and send a user agent

Note: this was closed as stale and is not completed.


'System.OutOfMemoryException' was thrown on a Large Table

The script has been very helpful to my project. Thank you very much.
I have run into one problem - When I run the script on a very large table, I get the following error:

Exception of type 'System.OutOfMemoryException' was thrown.

The particular table has 4 columns - 2 Int32 and 2 String type with about 1.7 million rows of data. Is it something that we can solve - or is it a inherent system limitation?

The tool was originally designed for synchronising smaller data sets between environments via source control, so typically this meant that a few thousand rows of static data would be the max. Others have used it for slightly different purposes, such as for performing simple ETL operations between databases.
In order to allow it to be used for datasets of this size, and have the data fully synchronised (e.g. all rows in the target inserted/updated/deleted to reflect the source data set), then a fundamentally different approach is needed in order to merge data in a reliable and efficient way.
The @include_values bit = 1 parameter, added recently (see #37, #47 & #34), might be a better option for working with larger datasets. It drops the VALUES clause containing all the data, in favour of simply joining onto the @table_name at merge time. This would allow you to populate the source table before running the merge, in whichever way is appropriate for your dataset (e.g. a single BULK INSERT statement containing all data, batches of INSERT statements containing thousands of rows at a time, etc).
For example:
EXEC [AdventureWorks2017].dbo.sp_generate_merge
  @table_name = 'CurrencyRate',
  @schema = 'Sales',
  @include_values = 0

Generates the following statement:
MERGE INTO [Sales].[CurrencyRate] AS [Target]
USING [Sales].[CurrencyRate] AS [Source]
ON ([Target].[CurrencyRateID] = [Source].[CurrencyRateID])
WHEN MATCHED AND (
	NULLIF([Source].[CurrencyRateDate], [Target].[CurrencyRateDate]) IS NOT NULL OR NULLIF([Target].[CurrencyRateDate], [Source].[CurrencyRateDate]) IS NOT NULL OR
	NULLIF([Source].[FromCurrencyCode], [Target].[FromCurrencyCode]) IS NOT NULL OR NULLIF([Target].[FromCurrencyCode], [Source].[FromCurrencyCode]) IS NOT NULL OR
	NULLIF([Source].[ToCurrencyCode], [Target].[ToCurrencyCode]) IS NOT NULL OR NULLIF([Target].[ToCurrencyCode], [Source].[ToCurrencyCode]) IS NOT NULL OR
	NULLIF([Source].[AverageRate], [Target].[AverageRate]) IS NOT NULL OR NULLIF([Target].[AverageRate], [Source].[AverageRate]) IS NOT NULL OR
	NULLIF([Source].[EndOfDayRate], [Target].[EndOfDayRate]) IS NOT NULL OR NULLIF([Target].[EndOfDayRate], [Source].[EndOfDayRate]) IS NOT NULL OR
	NULLIF([Source].[ModifiedDate], [Target].[ModifiedDate]) IS NOT NULL OR NULLIF([Target].[ModifiedDate], [Source].[ModifiedDate]) IS NOT NULL) THEN
 UPDATE SET
  [CurrencyRateDate] = [Source].[CurrencyRateDate],
  [FromCurrencyCode] = [Source].[FromCurrencyCode],
  [ToCurrencyCode] = [Source].[ToCurrencyCode],
  [AverageRate] = [Source].[AverageRate],
  [EndOfDayRate] = [Source].[EndOfDayRate],
  [ModifiedDate] = [Source].[ModifiedDate]
WHEN NOT MATCHED BY TARGET THEN
 INSERT([CurrencyRateID],[CurrencyRateDate],[FromCurrencyCode],[ToCurrencyCode],[AverageRate],[EndOfDayRate],[ModifiedDate])
 VALUES([Source].[CurrencyRateID],[Source].[CurrencyRateDate],[Source].[FromCurrencyCode],[Source].[ToCurrencyCode],[Source].[AverageRate],[Source].[EndOfDayRate],[Source].[ModifiedDate])
WHEN NOT MATCHED BY SOURCE THEN
 DELETE
;

The downside is that you'd need to manually edit the statement to use the pre-populated table object, rather than the original table object:
USING [Sales].[CurrencyRate] AS [Source]

to:
USING [#sourceTempTable/@sourceTableVar/etc] AS [Source]

I'm curious to know if this kind of general approach would suit your needs? If so then there's always the possibility of adding a new parameter to override the source table name.

"The downside is that you'd need to manually edit the generated statement to use the pre-populated table object (rather than the original table object), "
A way to make it more dynamic would be add an additional parameter that would take in a string - custom table name (e.g. @custom_source_name = '#currencyRateBulkInserted') - and the procedure would use this parameter's value instead if NOT NULL (a validation can be added at the top of the merge to throw an error if an object with that name doesn't exist). Just a thought off the top of my head. It's possible that I might be missing something.

An update on this: @EitanBlumin has very helpfully implemented a new parameter that allows you to split source rows into multiple MERGE statements. To use,  use @max_rows_per_batch=1000 or whatever batch size you need and be sure to also include the @delete_if_not_matched=0 param.
This should avoid the out-of-memory exception. If it recurs in spite of this, please comment here and I will re-open the issue.


Connecting three trackers makes stream extremely slow

The service and configuration utility works well with two controllers. However, using three controllers (And I assume more), seems to make the stream extremely slow. In some cases, the config tool gets stuck at "Connecting to PSMove Service". I wonder if this is the case of my computer, since I have a rather modest PC.

Update: This seems to happen at random! I managed to get three trackers smoothly connected at some point. However, when I changed the USB position, it went back to working slowly.

This is a well-known issue. It's in the wiki but it might be a bit hard to find. See number 2 here.

This is a well-known issue. It's in the wiki but it might be a bit hard to find. See number 2 here.

Thanks for the reply. I did some shuffling around and managed to get it working.


Hotfix/fixing device role retrieval

Codecov Report

Merging #29 into master will not change coverage.
The diff coverage is 0%.

@@         Coverage Diff          @@
##           master   #29   +/-   ##
====================================
  Coverage       0%    0%
====================================
  Files           6     6
  Lines         369   387   +18
====================================
- Misses        369   387   +18

Impacted Files
Coverage Δ

accessors/device.go
0% <0%> (ø)
:white_check_mark:

Continue to review full report at Codecov.

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


How Create A Invoice with PHP API

Hello, how I create a invoice (like this https://www.authorize.net/solutions/merchantsolutions/merchantservices/invoicing/) with API in PHP?.
Thanks,

Hi @lucasdevsquad
We are actively developing REST APIs  with native JSON support.
If you are interested in being a beta tester of our REST APIs, please email<EMAIL_ADDRESS>You should be able to use the invoicing REST  APIs in it for managing invoicing  .
Thanks
Anurag


Compile error under Centos 6.7

The latest commit on the dev branch is failing to build under Centos 6.7, but is building fine under OSX. (commit e160d1fb718aa1242cde3d62d65cfc0d5b7e9d2e).
I'm not sure which exact commit broke it as my previous build is quite old.
Running gcc (GCC) 4.4.7 20120313 (Red Hat 4.4.7-16)
Build output:
Making all in hashkit
make[3]: Entering directory `/home/vagrant/netflix/dynomite/src/hashkit'
gcc -DHAVE_CONFIG_H -I. -I../..  -I ../../src -I ../../contrib/murmur3  -Wall -Wshadow -Wpointer-arith -Winline -Wunused-function -Wunused-variable -Wunused-value -Wno-unused-parameter -Wno-unused-value -Wconversion -Wsign-compare -Wstrict-prototypes -Wmissing-prototypes -Wredundant-decls -Wmissing-declarations -Werror=implicit-function-declaration -g -O2 -MT dyn_crc16.o -MD -MP -MF .deps/dyn_crc16.Tpo -c -o dyn_crc16.o dyn_crc16.c
In file included from ../../src/dyn_core.h:129,
                 from ../../src/dyn_token.h:7,
                 from dyn_crc16.c:23:
../../src/dyn_util.h:352: error: redefinition of typedef ‘msec_t’
../../src/dyn_types.h:2: note: previous declaration of ‘msec_t’ was here
../../src/dyn_util.h:353: error: redefinition of typedef ‘usec_t’
../../src/dyn_types.h:3: note: previous declaration of ‘usec_t’ was here
In file included from ../../src/dyn_message.h:29,
                 from ../../src/dyn_core.h:132,
                 from ../../src/dyn_token.h:7,
                 from dyn_crc16.c:23:
../../src/dyn_types.h:1: error: redefinition of typedef ‘msgid_t’
../../src/dyn_types.h:1: note: previous declaration of ‘msgid_t’ was here
../../src/dyn_types.h:2: error: redefinition of typedef ‘msec_t’
../../src/dyn_util.h:352: note: previous declaration of ‘msec_t’ was here
../../src/dyn_types.h:3: error: redefinition of typedef ‘usec_t’
../../src/dyn_util.h:353: note: previous declaration of ‘usec_t’ was here
dyn_crc16.c:62: warning: no previous prototype for ‘hash_crc16’
dyn_crc16.c: In function ‘hash_crc16’:
dyn_crc16.c:68: warning: conversion to ‘uint32_t’ from ‘char’ may change the sign of the result
make[3]: *** [dyn_crc16.o] Error 1
make[3]: Leaving directory `/home/vagrant/netflix/dynomite/src/hashkit'

Hi, I got the same here but then I realized I was not using the master branch, are you ?
[root@host dynomite]# git branch

master

try cloning with: git clone https://github.com/Netflix/dynomite.git --branch master

That could be the case. Please try on the master branch and let us know.
On Wed, May 4, 2016 at 10:40 AM, Flavio Torres<EMAIL_ADDRESS>wrote:

Hi, I got the same here but then I realized I was not using the master
branch, are you ?
[root@host dynomite]# git branch

master

try cloning with: git clone https://github.com/Netflix/dynomite.git
--branch master
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/Netflix/dynomite/issues/290#issuecomment-216943508

@shailesh33 I think @mansoor-s talks that the dev branch. I doubt though that the latest commit is the cause of the issue. Some of the recent redefinition?

Hey I do not have a setup to test this out, can you refresh latest dev and try again. Thanks.

I can try this again with the latest dev branch commit a bit later today. Will let you know how it goes.

Sorry for the long delay.
The master branch is building successfully, however, the dev branch is still failing with the following error:
Making all in src make[2]: Entering directory/home/vagrant/dynomite/src'
Making all in hashkit
make[3]: Entering directory /home/vagrant/dynomite/src/hashkit' gcc -DHAVE_CONFIG_H -I. -I../..  -I ../../src -I ../../contrib/murmur3  -Wall -Wshadow -Wpointer-arith -Winline -Wunused-function -Wunused-variable -Wunused-value -Wno-unused-parameter -Wno-unused-value -Wconversion -Wsign-compare -Wstrict-prototypes -Wmissing-prototypes -Wredundant-decls -Wmissing-declarations -Werror=implicit-function-declaration -g -O2 -MT dyn_crc16.o -MD -MP -MF .deps/dyn_crc16.Tpo -c -o dyn_crc16.o dyn_crc16.c In file included from ../../src/dyn_core.h:130,                  from ../../src/dyn_token.h:7,                  from dyn_crc16.c:23: ../../src/dyn_util.h:352: error: redefinition of typedef ‘msec_t’ ../../src/dyn_types.h:3: note: previous declaration of ‘msec_t’ was here ../../src/dyn_util.h:353: error: redefinition of typedef ‘usec_t’ ../../src/dyn_types.h:4: note: previous declaration of ‘usec_t’ was here dyn_crc16.c:62: warning: no previous prototype for ‘hash_crc16’ dyn_crc16.c: In function ‘hash_crc16’: dyn_crc16.c:68: warning: conversion to ‘uint32_t’ from ‘char’ may change the sign of the result make[3]: *** [dyn_crc16.o] Error 1 make[3]: Leaving directory /home/vagrant/dynomite/src/hashkit'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory /home/vagrant/dynomite/src' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory /home/vagrant/dynomite'
make: *** [all] Error 2
`

you are right there is a redefinition. Let me see how to resolve this. I have been moving code and refactoring a bit lately.

@mansoor-s @ipapapa  @shailesh33
I build Dynomite in Amazon Linux - I never had issues - Here is a simple example how to build it.
https://gist.github.com/diegopacheco/37eee1c48c3cdce17b1fdbd7b5f23319
Cheers,
Diego Pacheco

@mansoor-s  @flaviotorres  We haven't heard back from you regarding this question. At this time, we're marking this case as resolved. If you want to follow up let us know and we can either reopen the issue, or please file a new issue.


automatically removing the LRM character

If possible, the iD editor should automatically remove the LRM character [1]. (an invisible unicode character used in some languages)
This character can be accidentally introduced by editors that copy and paste information from other sources. Fortunately it seems this character only appears in the end (and maybe at the beginning) of an URL.
Usually this character is no big deal, but it can be nasty in tags like website=*, contact:website=* and similar ones, because it can corrupt their address.
Examples of such tags can be seen in [2].
Note that recently I removed this character from URLs across the globe (around 1200), and yet it is growing again (around 60 right now).

I implemented these simple cleanups today because I was in this code anyway for #2301.
This code does not do complete validation of emails and websites, it only strips out troublesome characters likely caused by users pasting in bad data.


Atom 1.29.0

Microsoft Reviewers: Open in CodeFlow

/azp run

Duplicate of #11591

Duplicate of #11591


CM-464: Quicksort Algorithm in C# - Fixes

Build is getting stuck both on Jenkins and for me locally. I've canceled the build so please check out what's happening.

Let me check

retest this please

Hey @martinchege324 any updates on this?

Hey @CodeMazeBlog, testing out a few fixes then I will push another version in the course of the day

Sounds good.

Hey @CodeMazeBlog, I have pushed an update that passes all those tests


Visual Studio App Missing In Teams

Per the "Configuring Azure DevOps Services Tabs in Microsoft Teams" section: We do not have a "Visual Studio" or "VSTS" app in Teams. My Azure DevOps org uses the same tenant as my Teams account. There is an "Azure DevOps" app that seems to provide the same functionality -- I've added a Kanban board & dashboard tab using this app. Has the "Visual Studio" app been rebranded?

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

ID: 1cddc0ae-93af-297c-2e71-334143ebee1e
Version Independent ID: cb76e516-8471-c3e1-45f4-0a20a223f32a
Content: Create a service hook for Azure DevOps Services and TFS with Microsoft Teams
Content Source: docs/service-hooks/services/teams.md
Product: devops
GitHub Login: @elbatk
Microsoft Alias: elbatk

same issue. any plans to get this fixed soon?

why remove "Visual Studio" or "VSTS" app  T.T

VSTS/DevOps is not available in Microsoft Team, I tried to add connect but it is not available

Why this app does not available in Microsoft Team?

Thanks for your question. It looks like your question isn't an issue with the documentation, but an issue or question specific to your implementation or a bug in the product itself.
For help with a specific task, here are a couple of options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot

If you've encountered an issue with the product itself, to make sure it gets in front of the right people, please submit your bug here.
If you have an idea for improving the product, please submit your idea here.


Delete Galaxy-Workflow-Tutorial_training_regionalGAM_complete_workflo…

…w.ga

already updated in https://github.com/galaxyproject/training-material/pull/1463

Thank you!


os: Rename("oldDir", "newExistingDir") doesn't work on Windows

What version of Go are you using (go version)?
1.5.3

What operating system and processor architecture are you using (go env)?
windows - amd64

What did you do?
Rename a folder to new name that is already a folder. It works fine on linux-amd64 but cause error on windows-amd64

http://play.golang.org/p/m__tRp4mZh

What did you expect to see?
Replacement of newDir on windows as it does on linux.

What did you see instead?
Error. Access denied. Even running as administrator!

/cc @alexbrainman

Thanks

Note that on GNU/Linux the os.Rename will only succeed if the destination directory is empty.  If the destination directory has any files it will fail with ENOTEMPTY.  I think we should consider checking for and rejecting rename to an existing directory on non-Windows.

@bradfitz, latest change (CL 6140) of windows os.Rename introduced use of MoveFileEx. Our code uses MOVEFILE_REPLACE_EXISTING flag, I guess, to make "overwriting of destination file" work. But according to the doco https://msdn.microsoft.com/en-us/library/windows/desktop/aa365240(v=vs.85).aspx:
MOVEFILE_REPLACE_EXISTING ... This value cannot be used if lpNewFileName or lpExistingFileName names a directory.

That is why, I guess, MoveFileEx return ERROR_ACCESS_DENIED in that scenario. (Running example above I see MoveFileEx return ERROR_ACCESS_DENIED).
I will let you decide what to do here.
Alex

@alexbrainman is there anything blocking this? can I be of any help?

is there anything blocking this?

No, not really. Someone needs to come up with solution to this problem. I am not looking at this.

can I be of any help?

Feel free to propose solution to this.
Alex


Remove notebook dependencies from requirements-dev.txt

Summary
This removes the notebook dependencies from requirements-dev.txt, as otherwise they are specified in two places.  This PR supersedes #37.
Details and comments

I've experienced the following error when running tox -elint on this branch on one machine, but not on my primary workstation:
************* Module tests.test_backends
tests/test_backends.py:20:0: E0401: Unable to import 'qiskit.providers.ibmq' (import-error)
tests/test_backends.py:20:0: E0611: No name 'ibmq' in module 'qiskit.providers' (no-name-in-module)

------------------------------------------------------------------
Your code has been rated at 9.91/10 (previous run: 9.91/10, +0.00)

ERROR: InvocationError for command /tmp/prototype-qrao/.tox/lint/bin/pylint -rn --py-version=3.6 --disable=fixme,invalid-name,missing-function-docstring,protected-access,duplicate-code qrao tests (exited with code 14)
___________________________________ summary ____________________________________
ERROR:   lint: commands failed

All other tox environments work, even on the machine where this one fails.
I think I'll merge this soon anyway, as it will presumably be resolved once #15 is merged, in case it ever crops up again.


Fix issue when tabBar pushed up with keyboard

This pull request fixes android Issue by adding keyboard listeners.

hello! I think this approach is generally what we want to do but there are a few of issues:

it'd be better to hide the tab bar rather than remove it entirely
this option should be configurable. we probably want to default to it being enabled
code needs to be formatted using prettier (run yarn lint --fix to format it)

closing this because an approach like https://github.com/callstack/react-native-paper/pull/680 is preferred


Refactor manifest hex detection

Refactor this into it's own function so process isn't repeated across both NPM and GH selection types.

Closed out as part of v2.0.0 refactoring.


Compile time warning on Cray XE6

Original author: Nikhil Jain
Original issue: https://charm.cs.illinois.edu/redmine/issues/303

Charm++ and subsequent application compilation is complaining about  "not finding modules cray-pmi or cray-alpslli".

Original author: Nikhil Jain
Original date: 2015-02-09 15:54:02

Resolved by updating the pmi config.


DragDropObjects and DragDropTextOps readme files are swapped

The readme files for DragDropObjects and DragDropTextOps are swapped.
DragDropObjects shows drag and drop functionality with adorners and DragDropTextOps shows how to create a data object from a text selection.

@carpenterx Thank you for pointing this out. Can you go ahead and create a PR for the same ??


Searching a query with Google's search techniques

Thanks for the great work. I would like to be able to use Google's search techniques as described here.
I tried to achieve this as follows:
googlenews = GoogleNews(lang='en', encode='utf-8')
# googlenews.search('coronavirus')  # with this, I saw some articles about Scotland. So I changed it to ->
googlenews.search('coronavirus -scotland')
news = googlenews.results()

But the news are still related to Scotland.
Is it possible to have this feature? It would be very good to have Google's search operators.

It seems google doesn't have exclude feature for the news at the moment, we might not able to create this feature at current stage.

Oh this is sad :( Okay then. Thıs issue can be stay open as "feature request" if you would like to. It's a good feature IMO. Thanks for the work.

I am attempting to do the following search:
googlenews.get_news('"big bank" -lender -national')
I didn't get any results, so I decided to look into this. As it turns out, Google doesn't like the url this produces, which includes pluses in between terms. This is a result of line #256:
key = "+".join(key.split(" "))
If you change "+" here to "%20", the url resolves correctly and you get results. Since changing this single character will enable this additional functionality, might want to consider altering it.


Missing staking ledger recovery

This is actually the best way to implement the staking ledger updates, also. So, up-prioritizing.

Related #1312


Thoughts on a "donate" button?

I've been thinking about adding a donate button, a la PayPal, to Fuse, mainly so I can justify spending a good chunk of time doing the following:

Create new features
Bug fixes
Better project page and documentation
Tech support
More examples
Visually driven customization (think of wizards) for non-developers

Do you think such a button would be seen as a personal affront to the Open Source community?

I think it's perfectly moral to get paid for your work. Example: https://twitter.com/dan_abramov/status/618466568213778432

Took the plunge. Added the lovely button:

Please do that. This is a good strategy from www.vuejs.org . He is doing great too.

http://fusejs.io is up, with a live demo and more examples. I've added the support/donation at the bottom 😅

This is such a great tool ! I am a summer intern, and I find Fuse really
cool so I am implementing it in my current project. I sincerely like it and
once I am on the track of using it, I am thinking of asking my boss for
donation for you! great job
On Fri, Aug 5, 2016 at 4:12 PM, Kirollos Risk<EMAIL_ADDRESS>wrote:

http://fusejs.io is up, with a live demo and more examples. I've added
the support/donation at the bottom 😅
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/krisk/Fuse/issues/96#issuecomment-237951121, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFmAn9mzqR0rpEFaDr9kkLMAxnJ1bSzqks5qc5k-gaJpZM4Ja4jn
.

Wow @krisk http://fusejs.io looks Cool ❤
Good domain name, easy to remember. more over an exclusive site shows your dedication towards FuseJs.
Nice to see that you provided Patreon (recurring pledge) in addition to Paypal.
Good move.


chore: slightly improve README.md and CLI description

edit: after a bit of thinking I realised that this could be interpreted in a somewhat offensive/disrespectful manner and I prematurely appologise if it does come as so and do understand if it gets merged without any prior warning.
Thanks for the great work and gl with all your projects!

well frick

should be alright now

Thanks your contribution!!


React TSX: Default Props Ignored with Index Signature

TypeScript Version:

3.7.2
Nightly

Search Terms:
defaultprops index signature
Code
import * as React from "react";

export interface FooOptions {
  x: string;
  y?: string;
  [key: string]: any; // remove this and everything is ok
}

interface FooProps extends FooOptions {
  a: boolean;
}

class Foo extends React.Component<FooProps> {
  public static defaultProps = {
    a: true,
  };

  public render() {
    return
Remove index signature from FooOptions to make this work

  }
}

class App extends React.Component {
  public render() {
    return (
      <Foo x={""} />
    );
  }
}

Expected behavior:
Default property recognized as present
Actual behavior:
Error that parameter a is missing: Property 'a' is missing in type '{ x: string; }' but required in type 'Readonly<FooProps>'
Playground Link:

vNightly
3.7.2

Related Issues:
https://github.com/microsoft/TypeScript/issues/23812

Interestingly, as I recently mentioned elsewhere, we don't actually implement any of the logic for checking defaultProps in the compiler - that's entirely in the domain of the react.d.ts over on DefinitelyTyped. I'd re-file this over there, where one of the .d.ts file maintainers can take a look at it.

Specifically, this is probably a known behavior for defaultProps types, but don't consider me the authority here, I'm just reading the source~

Thanks for looking into it! I've opened https://github.com/DefinitelyTyped/DefinitelyTyped/issues/40367


Zero division error - cannot figure out source.

Hello,
I have some sparse hierarchical data that I am running through pyaf. None of my individual timeseries are entirely 0, yet I'm getting a divide by zero error when trying to run lEngine.train on my dataset.
---------------------------------------------------------------------------
ZeroDivisionError                         Traceback (most recent call last)
~/anaconda3/envs/tf-gpu/lib/python3.6/site-packages/pyaf/HierarchicalForecastEngine.py in train(self, iInputDS, iTime, iSignal, iHorizon, iHierarchy, iExogenousData)
     22         try:
---> 23             self.train_HierarchicalModel(iInputDS, iTime, iSignal, iHorizon, iHierarchy, iExogenousData);
     24         except tsutil.PyAF_Error as error:

~/anaconda3/envs/tf-gpu/lib/python3.6/site-packages/pyaf/HierarchicalForecastEngine.py in train_HierarchicalModel(self, iInputDS, iTime, iSignal, iHorizon, iHierarchy, iExogenousData)
     93         self.mSignalHierarchy = lSignalHierarchy;
---> 94         self.mSignalHierarchy.fit();
     95

~/anaconda3/envs/tf-gpu/lib/python3.6/site-packages/pyaf/TS/SignalHierarchy.py in fit(self)
    186         self.create_all_levels_models(lAllLevelsDataset, self.mHorizon, self.mDateColumn);
--> 187         self.computeTopDownHistoricalProportions(lAllLevelsDataset);
    188         lForecast_DF = self.internal_forecast(self.mTrainingDataset , self.mHorizon)

~/anaconda3/envs/tf-gpu/lib/python3.6/site-packages/pyaf/TS/SignalHierarchy.py in computeTopDownHistoricalProportions(self, iAllLevelsDataset)
    273                         self.mAvgHistProp[col][col1] = (lEstim[col1] / lEstim[col]).mean();
--> 274                         self.mPropHistAvg[col][col1] = lEstim[col1].mean() / lEstim[col].mean();
    275         # print("AvgHitProp\n", self.mAvgHistProp);

ZeroDivisionError: float division by zero

Any ideas as to what this might be or how to debug the issue in my dataset or the code itself?

Hi @andmib
Thanks for using pyaf. You are having an issue with computing  "Proportions of the historical averages" in a top-down approach for a hierarchical model.
Is it possible to share your model, with anonymized data ? This will allow me to solve the issue for you and all other pyaf users. This is my preferred way (creating an issue pointed you to this).
Otherwise, you can still try to solve the problem on your own , by protecting the division by zero. You can, for example,  dump the pandas dataframe lEstim in a csv file and print (col1 , col) to see f the mean is really zero.

@antoinecarme Thanks for the quick response! Unfortunately I wouldn't feel comfortable trying to anonymize the data and send it over, so I'm going to try your latter suggestion of dumping IEstim into a CSV and seeing where the mean is zero. I will keep you updated!

Thanks. Your feedback is welcome. I am interested if you can give a sketch of the diagnosis to be able to reproduce it on an artificial dataset/hierarchy.

@antoinecarme Looks like it is simply a sparsity issue - I have many groups, when split into train/test, are very sparse. Looks like I just need to drop those groups out.

@andmib
Good point. Did not think of sparsity, especially when the groups are generated by pyaf.

What type of hierarchy are you using ?
Does this happen on intermediate hierarchical  levels/signals ?

Pyaf can probably take care of discarding such groups automatically.

@andmib
Good point. Did not think of sparsity, especially when the groups are generated by pyaf.
1. What type of hierarchy are you using ?

2. Does this happen on intermediate hierarchical  levels/signals ?

Grouped, based on the grouped example ipython notebook.
Only the most granular level - at higher levels of the hierarchy, the mean is not 0 (but close on some).

Very interesting. Can you please share the pickled model (not the data) ?

Pyaf can probably take care of discarding such groups automatically.

But let's say you have two groups - wine-red-america and wine-red-europe, where wine-red-europe is super sparse or close to 0. You may still want a (probably bad) forecast for wine-red-europe based off the wine-red grouping?

Yes. You are right. But needs to be investigated.

Very interesting. Can you please share the pickled model (not the data) ?

I haven't gotten to a successfully trained model yet ;) Keeps failing with zero division error, so I'm dropping groups in my original data based on some threshold of having data prior to using pyaf.

you can pickle the model before training. That's already good for me (and safe for you ;).

you can pickle the model before training. That's already good for me (and safe for you ;).

Can you show me at what point I pickle the model in the example "grouped" model ipynb?

Just before the last line here :

Just before the last line here :

So joblib.dump(lEngine, 'pickedfile.pkl')? It's failing right at the .train line.

Did you add
joblib.dump(lEngine, 'pickledfile.pkl')
before
lSignalHierarchy = lEngine.train(train_df , lDateColumn, lSignalVar, 1, lHierarchy, None)
??

Did you add
joblib.dump(lEngine, 'pickledfile.pkl')
before
lSignalHierarchy = lEngine.train(train_df , lDateColumn, lSignalVar, 1, lHierarchy, None)
??

No, that's what I'm asking - which object do you want me to pickle?

lEngine

Do you have an email address/box I can shoot this over to you?

Sorry, will need also the pickle of another object : lHierarchy.
My public email is on the top of this file :
https://github.com/antoinecarme/pyaf/blob/master/TS/Signal_Grouping.py

Sorry, will need also the pickle of another object : lHierarchy.
My public email is on the top of this file :
https://github.com/antoinecarme/pyaf/blob/master/TS/Signal_Grouping.py

Sent. But I did get this working. I just needed to drop groups with too few observations - in your example, that would be columns in train_df that are almost all 0.

received your message.
Thanks. Will keep this page updated when something changes. Leaving.

Nice to hear that's OK for you. You can close this issue if you want.

Nice to hear that's OK for you. You can close this issue if you want.

Will do - this is a very helpful package by the way, and I will keep you updated.


Height Problem when only using a few word

Here's the screenshot for it

Please let me know, if you need more information :)) I really loves to contribute for this project of yours, to help others to acommplish their design of Adaptive Text :))
Really appreciate of what were you doing!!!

I've got same problem , try add textVeiw from java code using the class :
http://www.andreamaglie.com/android-auto-fit-textview/


Incorrect typings for getBacklinksForFile and CustomArrayDict

Many thanks for all the PR's, this one looks good to me!
As a aside: apologies for all the errors I've made, I added most of these typings in a single evening - in a half sleep-deprived state. Currently I'm a bit low on time, but I'll try going over all of them as soon as I'm able (plus adding in the missing types on a lot of the workspace and editor-related components).


import tensorflow (v2.1.0) fails

System information

Windows 10
TensorFlow installed from (pip install tensorflow):
TensorFlow version: 2.1.0
Python version: 3.7.1
Installed using virtualenv:

Describe the problem
import tensorflow as tf fails
(tf2) D:\@Temp\@Issues\2020-02-04-tf\tf2>python -c "import tensorflow as tf;print(tf.reduce_sum(tf.random.normal([1000, 1000])))"
Traceback (most recent call last):
  File "D:\<EMAIL_ADDRESS>line 58, in <module>
    from tensorflow.python.pywrap_tensorflow_internal import *
  File "D:\<EMAIL_ADDRESS>line 28, in <module>
    _pywrap_tensorflow_internal = swig_import_helper()
  File "D:\<EMAIL_ADDRESS>line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
  File "C:\Users\igor-z\AppData\Local\Continuum\anaconda3\lib\imp.py", line 242, in load_module
    return load_dynamic(name, filename, file)
  File "C:\Users\igor-z\AppData\Local\Continuum\anaconda3\lib\imp.py", line 342, in load_dynamic
    return _load(spec)
ImportError: DLL load failed: The specified module could not be found.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "D:\<EMAIL_ADDRESS>line 101, in <module>
    from tensorflow_core import *
  File "D:\<EMAIL_ADDRESS>line 40, in <module>
    from tensorflow.python.tools import module_util as _module_util
  File "D:\<EMAIL_ADDRESS>line 50, in __getattr__
    module = self._load()
  File "D:\<EMAIL_ADDRESS>line 44, in _load
    module = _importlib.import_module(self.__name__)
  File "C:\Users\igor-z\AppData\Local\Continuum\anaconda3\lib\importlib\__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "D:\<EMAIL_ADDRESS>line 49, in <module>
    from tensorflow.python import pywrap_tensorflow
  File "D:\<EMAIL_ADDRESS>line 74, in <module>
    raise ImportError(msg)
ImportError: Traceback (most recent call last):
  File "D:\<EMAIL_ADDRESS>line 58, in <module>
    from tensorflow.python.pywrap_tensorflow_internal import *
  File "D:\<EMAIL_ADDRESS>line 28, in <module>
    _pywrap_tensorflow_internal = swig_import_helper()
  File "D:\<EMAIL_ADDRESS>line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
  File "C:\Users\igor-z\AppData\Local\Continuum\anaconda3\lib\imp.py", line 242, in load_module
    return load_dynamic(name, filename, file)
  File "C:\Users\igor-z\AppData\Local\Continuum\anaconda3\lib\imp.py", line 342, in load_dynamic
    return _load(spec)
ImportError: DLL load failed: The specified module could not be found.

Failed to load the native TensorFlow runtime.

See https://www.tensorflow.org/install/errors

for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

tf (v1.15.0) in other virtual environment shows next:
(tf1) D:\@Temp\@Issues\2020-02-04-tf\tf1>python
Python 3.7.1 (default, Dec 10 2018, 22:54:23) [MSC v.1915 64 bit (AMD64)] :: Anaconda, Inc. on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import tensorflow as tf
2020-02-05 15:31:28.406892: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'cudart64_100.dll'; dlerror: cudart64_100.dll not found
2020-02-05 15:31:28.411004: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine.
>>> tf.__version__
'1.15.0'
>>>

@constructor-igor, Suspecting that you have installed tensorflow-gpu 2.1 version. Please check and install pip install tensorflow==2.1. Thanks!

@gadagashwini
I checked and found next installed "tensorflow" packages:
six                  1.14.0
tensorboard          2.1.0
tensorflow           2.1.0
tensorflow-estimator 2.1.0
termcolor            1.1.0

@constructor-igor,
Open ...\Lib\site-packages\tensorflow\python_pywrap_tensorflow_internal.pyd use Dependency Walker, it will show you the
DLL dependency tree, you will find which DLL cause the problem. You can then install the needed version,
or add some path to system variable.
Dependency Walker link : http://www.dependencywalker.com/.
Thanks!

@gadagashwini
folder ...\Lib\site-packages\tensorflow doesn't contain any .pyd file.

I found .pyd files in ...\Lib\site-packages\tensorflow_core\python folder only.

Dependency Walker shows next (see below).
Please, could you explain what should be installed for TF2?

Closing as duplicate.
https://github.com/tensorflow/tensorflow/issues/36167#issuecomment-577886156
#36167 #36151 #36138 #36054 #36045 #36020 #36003 #35988 #35903 #35880 #35865 #35805 #35789 #35773 #35772 #35767 #35766 #35749 #35721 #35618 #35204

solution:
download and install Microsoft Visual C++ Redistributable for Visual Studio 2015, 2017 and 2019 package from
https://support.microsoft.com/en-us/help/2977003/the-latest-supported-visual-c-downloads
Windows 10 x64 requires vc_redist.x64.exe.


Upgrade GitHub runners to at least 2.297.0 by 2022-10-26

https://github.com/actions/runner/releases/tag/v2.297.0
New release is out, which means we have 30 days to upgrade
Same deal as https://github.com/iree-org/iree/issues/10246, except I have now made it much easier to upgrade (just a config push).
I'll wait at least a week so that we can make sure no one reports issues with the new version (and we may pick up some patch releases).

Well now 2.298.0 is out. Going to wait another weekish to ensure that one is stable and jump straight to that :-)

Lol 2.298.1 already out

@pzread could you please upgrade these to the latest version while I'm out? I won't have time to manage the rollout before I go OOO. Instructions are at https://github.com/iree-org/iree/tree/main/build_tools/github_actions/runner#updating-the-runners. The version is set here: https://github.com/iree-org/iree/blob/656cbbbf332630975bdcff8529b686460e33f1fc/build_tools/github_actions/runner/gcp/create_templates.sh#L46

Rolled out to canary


Updated translation.

Coverage remained the same at 39.849% when pulling 3d569e1c5f3e66cf97e172a75afd627a9074045b on Myrmece:patch-7 into<PHONE_NUMBER>a29ae6529d3283941c978068b8c8c2 on NyaaPantsu:dev.


CompilationDatabase tool emits paths into variant directories for source files when not using absolute paths.

Describe the bug
When using the CompilationDatabase tool with COMPILATIONDB_USE_ABSPATH == False, the file field appears to refer into the variant directory, not the source directory:
"file": "build/cached/third_party/s2/base/int128.cc",
Instead of
"file": "src/third_party/s2/base/int128.cc"
Required information

Link to SCons Users thread discussing your issue.
Discussed directly with @bdbaddog

Version of SCons
master

Version of Python
3.7

How you installed SCons
source

What Platform are you on? (Linux/Windows and which version)
macOS, but shouldn't matter.

@acmorrow  - So you'd expect the source file to point to source of variantdir, but target to point to variant dir itself?

@bdbaddog - I'm not following you there. The error is in the file field of the JSON result. It is referring to a source file by a path into the variant directory. That would only be true if you were building with duplicate=1 or similar. The file field should have the path to the source file where it lives on disk. I'm not sure what you mean by "target to point to variant dir itself".

@acmorrow like this
 % cat compile_commands.json
[
    {
        "command": "/Users/bdbaddog/.virtualenvs/scons37/bin/python mygcc.py cc -o build/test_main.o -c build/test_main.c",
        "directory": "/private/var/folders/ng/48pttrpj239fw5rmm3x65pxr0000gn/T/testcmd.32413.myyo97s5",
        "file": "src/test_main.c",
        "output": "build/test_main.o"
    }
]%

Yes, that looks like what I would expect for command, directory, and file. If you provide a fix that adjusts that, I'll re-run my experiment to A/B compare against the MongoDB one and see that the output is now aligned.
Regarding the output field: I wonder. I'm not sure if it is actually supposed to be a path or not. The "spec" doesn't really say: it says it should be "The name of the output created by this compilation step". Does that include the path? Who knows?

I'm sure once we release it we'll get feedback.. ;)
Adding a test for variant_dir with duplicate = True and False..

@bdbaddog - I pulled this and redid the A/B test vs the MongoDB version. Looks great! As discussed elsewhere, the output field is new and I'm not sure the spec really tells us enough to know what ought to go there: a filename or a path. Either way, if there is someone out there for whom it matters, I think they will let us know if we have it wrong.


Proposal: support explicit implementation relation

In other words, support Java-like interfaces, maybe like this:
import iface

iface explicit Animal: # Define an interface which demands explicit implementation relations
  proc say(): string

type
  Dog {.implements: Animal.} = ref object of RootRef

# or as an alternative to the pragma:
Dog implements Animal

proc say(d: Dog): string = "bark"
proc doSmth(a: Animal): string = a.say()
assert(doSmth(Dog()) == "bark") # This would throw an error if the implements-clause was missing

This would help whenever the intention of the implementer is relevant and it would enable #3.
Could probably be implemented by storing implements relations with macrocache.

Unfortunately I can't see a way to declare interface relation "upfront", as there's no "hook" to be called at the end of semantic pass. However this can be easily achieved with an explicit statement after the object has been implemented with something along the lines:
template assertImplements(T: typedesc, I: typedesc[Interface]) =
  when not compiles(to(default(T), I)):
    {.error: $T & " does not implement " & $I .}

# ... iface decl
# ... Dog decl
# ... Dog impl

assertImplements(Dog, Animal)


Align formatDate DateStyle formats with shopify-i18n

Overview
#397 is adding some basic date format options from the legacy i18n solution, but we ultimately want to align with the pre-existing work in shopify-i18n.
Type

[ ] New feature
[x] Changes to existing features

Motivation

What inspired this feature request? What problems were you facing,
or what else makes you think this should be in quilt?

The work in #397 was a first pass at bringing in similar functionality, with an expected second pass to fill in remaining formats.
Area

[x] Add any relevant Area: <area> labels to this issue

Scope

Is this issue related to a specific package?

[x] Tag it with the Package: <package_name> label.

Is it related to a new package?

[ ] Tag it with the New package Label

Checklist

[x] Please delete the labels section before submitting your issue
[x] I have described this issue in a way that is actionable (if possible)

Hey @patsissons - I'm looking to make use of some of the humanized date formats that Polaris recommends. Things like 13 minutes ago, or Yesterday at 10:30.
Was this ticket meant to address this?
The full list of recommended formats is here: https://polaris.shopify.com/content/grammar-and-mechanics#section-dates-numbers-and-addresses

oof, this one is old 😂. I seem to recall this issue was just to add the initial DateStyle enum to react-i18n and should have probably been closed many moons ago. feel free to close it with any new efforst you plan on addressing 👍

haha, true. I'll start a new ticket with the Polaris specific formats.


Misprocessed URL in Cli Dashboard

In my environnement, where Reposilite is behind a reverse proxy with a context root path, Cli Dashboard didn't work because there is a small bug:
    if (origin.endsWith('/')) {
      origin = origin.substring(1)
    }

In my case, orign ends with a '/' so the substring is invalid and lead to break the websocket url.

By saying context root path you mean the base path? :) Could you provide your custom base path, it might be useful during debug.

Yes, you are right, I mean base path :-)
My configuration starts with:
hostname: <IP_ADDRESS>
port: 8081
basePath: /reposilite/

My basePath ends with a '/'   I've tried to remove it but when I restart the configuration is automatically modified and the ending slash come back again

Yes, you are right, I mean base path :-)
My configuration starts with:
hostname: <IP_ADDRESS>
port: 8081
basePath: /reposilite/

My basePath ends with a '/' I've tried to remove it but when I restart the configuration is automatically modified and the ending slash come back again

I've made some fixes related to custom base path, check out the 2.9.14.


Inet::IPAddress::fromString should support parsing the scope ID as well

Problem
Currently, providing a scoped IPv6 address (e.g. fe80::3e61:5ff:fe0c:89f8%47) where the network interface index is encoded in the IPv6 address, fails to parse.
https://github.com/project-chip/connectedhomeip/blob/master/src/inet/IPAddress-StringFuncts.cpp#L85
Proposed Solution
This method should be updated to parse and retain the scope ID, if present.
Maybe relevant discussion here? http://savannah.nongnu.org/task/?13106

@g-coppock It's probably not practical to add scope support to IPAddress parser directly because the IPAddress struct doesn't include the scope ID definition - It does make sense as scope is not part of IPv6 address but associated with the context where it is being used:

The zone to which a particular non-global address pertains is not
encoded in the address itself but determined by context, such as the
interface from which it is sent or received.  Thus, addresses of a
given (non-global) scope may be re-used in different zones of that
scope.  For example, two different physical links may each contain a
node with the link-local address fe80::1.

https://www.rfc-editor.org/rfc/rfc4007#section-5
So both the Lwip ip6_addr_t and linux in6_addr doesn't include scope but linux  sockaddr_in6  does (https://man7.org/linux/man-pages/man3/inet_ntop.3.html).
While the CHIP PeerAddress is something similar to sockaddr_in6, it should be easier to just add a string parser for PeerAddress and use it against IPAddress::fromString where a PeerAddress is required. Thoughts?
(Note: it seems mostly an IPAddress is used together with an additional InterfaceId parameter, for example https://github.com/project-chip/connectedhomeip/blob/b9d8ef17c54467d67d0bd86847f6dbbfd3d98cb3/src/inet/UDPEndPoint.cpp#L133

That makes sense -- more specifically yeah I think we just need a way to go from a String to a scoped address. If PeerAddress makes more sense there I think that's fine.


Remap setSectionResizeMode to Qt 4

Goal
Enable Qt 4 bindings the use of setSectionResizeMode.
# PySide
import sys
from Qt import QtWidgets
app = QtWidgets.QApplication(sys.argv)
widget = QtWidgets.QTreeView()
qheaderview = widget.header()
qheaderview.setSectionResizeMode(qheaderview.Fixed)
widget.show()
app.exec_()

Reference

Rationale
Originally offered here but through misunderstandings it never saw the light of day.
This enables a PyQt5/PySide2 implementation to run on PyQt4/PySide, without altering the behavior of those bindings.
Implementation
A simple remapping. See the test and implementation for details!
@fredrikaverpil let me know what you think.

This works now, but there is a problem.
Importing PySide before Qt.py under Python 3 has side-effects which prevents the remapping from taking place. This may be due to some of the magic happening upon importing PySide and I'm expecting this to be outside of our control.
The exposing lines are here, remapping is here.
This makes monkey-patching classes a non-starter.

Here's an alternative, to subclass rather than monkey-patch.
The problem in this case is that the function QTreeView.header() isn't returning our subclass, but rather the original class; as the call is to C++ and C++ knows nothing of what we've done externally.
A fact easily proven via:
id(qheaderview) != id(QtWidgets.QHeaderView)

Stack trace

This makes subclassing for filling in missing functionality in older bindings a non-starter as well.
Due to these findings, and unless anyone has an alternative implementation to share, this cannot be merged. :(

Here's a third alternative from qtpy; also monkey-patching the class.

Important lessons learned here are:

We cannot replace members of QtWidgets et. al. with our own subclasses due to classes returned via internal functionality, such as QTreeView.header(), does not know nor care about this.
We cannot monkey-patch a class because (1) it modifies the original binding and/or (2) it sometimes doesn't work; as seen here with Python 3.5-only (more).

Closing this, and will refer back here whenever any of these two solutions crop up in pull-requests.

Due to the PR coming from a master branch, rather than a feature branch, I've force-pushed it back to it's original state and stored these changes for future reference in an alternative branch.

https://github.com/abstractfactory/Qt.py/tree/setSectionResizeMode


[CMake] Use PUBLIC/PRIVATE were relevant + remove unused file.

@nim65s any comment on this ?

Thanks for cleaning the forgotten src/CMakeLists :)
I don't have other comments.


Stuck on waitlist page

I recieve an invite from my friend and I still stuck to the waitlist page. I tried restart and reinstalled the application.

This issue is not with the app.


Making originalAction an array to trace effect history

Shall we provide originalActions as an array instead of just the previous action to trace the history of side effect? Also it will provide a way for redux-logger to nicely log all the history side effects.

are you referring to this file? https://github.com/redux-loop/redux-loop/blob/master/modules/errors.js

@bdwain yep but more about the install.js file. I've made a rough implementation here
https://github.com/Marswang92/redux-loop/commit/5e1fbfa58db6a2b4b0925a5f8820ede86ae265a7

i'm not sure I follow. Each side effect is triggered by a single action anyway. Where does the array of actions come from?

So in my use case there're side effects which are caused by an action which is emitted by another side effects. Use the this repo's example, Start short increment is dispatched from the Cmd.batch from Start both increment and  Finish short increment is caused by the side effect from Start short increment. It would be quite informative to be able to trace back this whole behavior.

I think I see. I'm a little confused about the implemntation you linked though. You changed the signature of store.dispatch in there to take an array of actions. That seems like it would be an issue. Wouldn't redux just ignore that action array?

So the way I'm doing this is taking the advantage of liftReducer. Since we don't want to overwrite the reducer controlled by redux, what we can do is asking liftReducer to keep on tracking originalAction. When you dispatch the action to liftReducer basically it will check if originalAction exist or not. If so, just simply append the current action into the array.

So the way I'm currently use is when you dispatch an action with cmd, you'll either create an array of originalAction which contains it self or just append it to the array of its originalActions. The dispatch method inside the redux-loop middleware will handle the originalActions parameter properly.

hi @Marswang92 . I'm very sorry I haven't responded in so long. I haven't had a ton of time lately.
If you can get this to work in a branch, I'd be happy to merge it in. I think I see where your rough implementation is going but I haven't been able to think through it completely yet.
One thing I did notice is that I'm pretty sure when you pass the originalActions array into store.dispatch, it will get ignored. store.dispatch is the underlying dispatch method and not the one that we expose. Other than that, it seems like what you're going for would work.

Yeah I'll try to spend some time on that maybe this weekend, thx!

Also, just so you're aware, #151 seems like it might have a significant effect on the implementation of this. With Cmd.list, not all actions get dispatched from the install.js file. Unless you have the batch option enabled, the promise from the list cmd is resolved with an empty array.

Possibly related to #131


The warning message for the new pyright version makes --output-json not valid JSON.

Running
pyright --outputjson | jq

used to work,
but now I get
parse error: Invalid numeric literal at line 926, column 8.
pyright --outputjson >> output.txt

shows that std out includes
WARNING: there is a new pyright version available (v1.1.224.post0).
Please install the new version or set PYRIGHT_PYTHON_FORCE_VERSION to `latest`

at the end (after the JSON output), which is messing up jq.
As a side note, I get this message even though I am on the latest (1.1.224.post0), so the only workaround at the moment seems to be to set that env variable.

I hadn't considered that, I'll disable the warning if --outputjson is present and add an env variable to disable the warning.
Also are you sure that the pyright command actually points to the correct pyright installation? I've encountered a path mismatch before.

When I do pyright --version
pyright --version
WARNING: there is a new pyright version available (v1.1.224.post0).
Please install the new version or set PYRIGHT_PYTHON_FORCE_VERSION to `latest`

npx: installed 1 in 4.462s
pyright 1.1.224

so it seems like the version returned is missing the .post0.

@jthompson6 can you run this please?
python -c 'import pyright; print(pyright.__version__)'

16:44 $ python -c 'import pyright; print(pyright.__version__)'
1.1.224.post0

This happens for me in circle ci too, where I
pip install pyright on an image without pyright, then run
pyright --outputjson | jq .summary and get the same error.

@jthompson6 I've pushed a new version that should disable this warning. Please try it out and let me know if it works.

Yes, it works now. Thanks a lot.


error install

@TDL77 According to what you provided, this is unobvious why the error occured, but it is definetly the environment issue. You can check the following:

Ensure that notebook kernel is set to the correct python path. Compare command line which python output and Notebook output of

import sys
print(sys.executable)

Ensure that you are running on GPU available machine. Check the nvidia-smi output

Ensure that there is single cupy installed inside the environment. Check the pip freeze | grep cupy output. Since, for example, conda cupy and pypi cupy-cuda110 or cupy-cuda11x are all the different packages! And import conflicts can occur. If cupy was already installed inside conda/mamba env, just pip uninstall cupy-cuda110 and try again

Check that you installed the correct cupy for your cuda. cupy-cuda110 is the package for CUDA11.0. If you are running under another CUDA just uninstall cupy-cuda110 and install the version you need

If nonthing helps, just let me know again


Easier FTL Controller Database Migrations

Right now when managing a deployed FTL cluster, managing database migrations of the controller is a bit inconvenient. It involves copying the database migration files from the FTL repo and then running the migrations using dbmate.
It would be nice if there was some way to perform these migrations without having to do this as it becomes tricky when trying to manage multiple deployed environments with different versions.
I believe there is a tool called ftl-initdb that is used for local dev that has the migration files baked in for the given version. Perhaps the solution here is to allow that tool to take connection strings (it may do already), and have the binaries available somewhere for downloading, but I am not sure if this is necessarily the best path forward, I am open to other suggestions.

As discussed, aim for a ftl migrate command which runs embedded migrations locally.


Overwriting existing SIP always results in error at first try (but then works on 2nd)

Example:
E:\testiromlab>c:\python36\python f:\johan\pythonCode\omSipCreator\omSipCreator\
omSipCreator.py write kb-c91bbfda-147a-11e7-93d2-00237d497a29 test6
This will overwrite existing directory 'test6' and remove its contents!
Do you really want to proceed (Y/N)? > y
CRITICAL - cannot remove 'test6'
INFO - Batch verification yielded 1 errors and 0 warnings

Then on 2nd try:
E:\testiromlab>c:\python36\python f:\johan\pythonCode\omSipCreator\omSipCreator\
omSipCreator.py write kb-c91bbfda-147a-11e7-93d2-00237d497a29 test6
This will overwrite existing directory 'test6' and remove its contents!
Do you really want to proceed (Y/N)? > y
INFO - Processing PPN 155658050
INFO - creating SIP directory
INFO - creating carrier directory
INFO - copying files to carrier directory
INFO - creating carrier directory
INFO - copying files to carrier directory
INFO - writing METS file
INFO - Verify / write resulted in 0 errors and 0 warnings

Doesn't seem to happen anymore so closing this issue.


refactor: Qdrant async improvements

Description:
This PR intends to simplify the async implementation of Qdrant vectorstore using the AsyncQdrantClient interface introduced in<EMAIL_ADDRESS>

@eyurtsev, review please.

@efriis, review please.


Improve about page

I have added more content for the improvement of the about section.

Share the link of the changes you made.

How i conflict with tha basw branch

Read this:
https://github.com/skpandey885/HealhUb/blob/main/Documentation/Publish-your-changes.md

hey @codervivek5. You have to add content in all sections of the "About" Page. You have only added  content in one section. Please make the necessary changes and add to this PR.

This is the link:
we have added some contact regarding the about us page
https://codervivek5.github.io/HealhUb/

hey @codervivek5. You have to add content in all sections of the "About" Page. You have only added content in one section. Please make the necessary changes and additions to this PR.

one section means without changing paragraph like this

@skpandey885 I will add FAQ Questions and content to the privacy policy section.


Setting up integration tests with jest and write some basic tests for Code-Editor

Checklist

[ ] I run yarn run webpack:build:main: the project builds without errors.
[ ] I run yarn lint: the project builds without code style warnings.
[ ] I updated the documentation and models.
[ ] I tested the changes and all related features on the test server https://artemistest.ase.in.tum.de.
[ ] I added (end-to-end) test cases for the new functionality.

Motivation and Context

Description

Steps for Testing

Log in to ArTEMiS
Navigate to Course Administration
...

Screenshots

Closing this as the bamboo build is not triggered on this PR.


Error "Failed to download manifest"

When trying to install on a ESP Muse Luxe I received a error message reading "Failed to download manifest".
From looking into the browser console a request to https://sle118.github.io/squeezelite-esp32-installer/base_manifest.json fails with 404.
Before that there are also other errors, especially "main.js:159 Unable to set manifest for platform SqueezeAmp: TypeError: Cannot read properties of undefined (reading 'manifest')".
Looking into the artifacts repository it looks to me like there are only files for SqueezeAmp while those for I2S and Muse were deleted with the most recent commit.

Looks like some later update of the images resolved the issue.

@krmr thank you for reporting this. We had to remove ones of the releases from the main project since it had introduced an issue which caused console and GUI to malfunction


Unrolling U3 gate to new basis set of devices

Information

Qiskit Terra version: 0.16.1
Python version: 3.7
Operating system: Big Sur mac

What is the current behavior?
Under transpile, unroller seems to fail to convert U3 gate to the new basis set ['id', 'rz', 'sx', 'x', 'cx', 'reset'] used on commercial devices, and gives error:
QiskitError: "Error decomposing node u3: 'NoneType' object has no attribute 'data'"
Unclear to me if bug of intended. Perhaps very closely related to https://github.com/Qiskit/qiskit-terra/issues/5840
Steps to reproduce the problem
from qiskit.transpiler.passes import Unroller
from qiskit.transpiler.passmanager import PassManager

qc = QuantumCircuit(2,2)
qc.u3(0,0,0,[0,1])
qc.u3(0,pi,0,[0])
qc.u3(0,pi,0,[1])
qc.measure_all()
display(qc.draw())

basis_gates = ['id', 'rz', 'sx', 'x', 'cx', 'reset'] # backend.configuration().basis_gates
pm = PassManager()
pm.append(Unroller(basis_gates))
transpile(qc, pass_manager=pm)

Results in
QiskitError: "Error decomposing node u3: 'NoneType' object has no attribute 'data'"

What is the expected behavior?
I would imagine that the single qubit basis gates are universal and the U3 would be unrolled onto them, but perhaps that's wrong?
Suggested solutions
I wonder if the EquivalenceLibrary might be useful here to substitute all U3 gates in a circuit with their 𝑈3(𝜃,𝜙,𝜆)=𝑅𝑍(𝜙)𝑅𝑋(−𝜋/2)𝑅𝑍(𝜃)𝑅𝑋(𝜋/2)𝑅𝑍(𝜆) decomposition? Perhaps along the lines of https://github.com/Qiskit/qiskit-terra/blob/54c8870623e28cb3f9f4eb92828fb8fd5f9dde49/test/python/transpiler/test_unroll_custom_definitions.py

Seems to be solved with transpile(qc, backend, optimization_level=0), but not sure which pass is responsible

See: https://github.com/Qiskit/qiskit-terra/issues/5886#issuecomment-782715490 where I went into more detail on this when someone opened a similar issue before (which was also a duplicate of #5840). The unroller pass doesn't work with an arbitrary universal basis it just descends through the definitions of the gate and in the case of u3 is defined in terms of u: https://github.com/Qiskit/qiskit-terra/blob/master/qiskit/circuit/library/standard_gates/u3.py#L94 which won't find a match in the new ibmq basis. You want to use the BasisTranslator pass (which is the default in all the preset passmanagers do now) to do the conversion which uses the equivalence library as you pointed to.
If you change your example above but swap the Unroller pass for the BasisTranslator you get:
global phase: -π
        ┌───────┐┌────┐┌───────┐┌────┐┌────────┐┌───────┐┌────┐┌───────┐┌────┐┌────────┐ ░ ┌─┐
   q_0: ┤ RZ(0) ├┤ √X ├┤ RZ(π) ├┤ √X ├┤ RZ(3π) ├┤ RZ(0) ├┤ √X ├┤ RZ(π) ├┤ √X ├┤ RZ(4π) ├─░─┤M├───
        ├───────┤├────┤├───────┤├────┤├────────┤├───────┤├────┤├───────┤├────┤├────────┤ ░ └╥┘┌─┐
   q_1: ┤ RZ(0) ├┤ √X ├┤ RZ(π) ├┤ √X ├┤ RZ(3π) ├┤ RZ(0) ├┤ √X ├┤ RZ(π) ├┤ √X ├┤ RZ(4π) ├─░──╫─┤M├
        └───────┘└────┘└───────┘└────┘└────────┘└───────┘└────┘└───────┘└────┘└────────┘ ░  ║ └╥┘
   c: 2/════════════════════════════════════════════════════════════════════════════════════╬══╬═
                                                                                ║  ║
meas: 2/════════════════════════════════════════════════════════════════════════════════════╩══╩═
                                                                                0  1

The other option you can use is the unitary synthesis pass for basis translation: https://github.com/Qiskit/qiskit-terra/blob/master/qiskit/transpiler/preset_passmanagers/level1.py#L154-L160 which involves a few more steps but should also work.

Thank you, @mtreinish ! I think that solves it


Fixup SoC FW authentication flows section

Need to synchronize with runtime spec

Varun, thanks for filing. Let's complete by 9/8, and hand off to steph-morton


could not acquire a semaphore for execution

it is my zuul config:
 hystrix.command.default.execution.timeout.enabled=false
hystrix.command.default.fallback.enabled = false
hystrix.command.default.circuitBreaker.enabled = false

zuul.semaphore.max-semaphores=50

hystrix.command.default.execution.isolation.semaphore.maxConcurrentRequests=50
zuul.ribbonIsolationStrategy= SEMAPHORE
hystrix.command.default.execution.isolation.strategy=SEMAPHORE

when I restart the tomcat,and the tps is 50, sometimes it reports:
Caused by: java.lang.RuntimeException: could not acquire a semaphore for execution
        at com.netflix.hystrix.AbstractCommand.handleSemaphoreRejectionViaFallback(AbstractCommand.java:966)

then it runs without the error log,it is ok.
why?

Hi @xiaominger, this could explain why
https://github.com/Netflix/Hystrix/wiki/How-it-Works#5-is-the-thread-poolqueuesemaphore-full

Hi @xiaominger, this could explain why
https://github.com/Netflix/Hystrix/wiki/How-it-Works#5-is-the-thread-poolqueuesemaphore-full

but why it is ok soon


Queue does not receive any jobs

Hi there,
first of all thank you for the project. I did the installation as written in the doku. I can log in with Jon DOE but, if I register a new user I see in the DB but no email is triggered. And yes, I am running php artisan queue:work.
Do you have any idea, why this might be?

Check your smtp configuration in your env file.
On Mon, 29 Jan, 2024 at 12:29, Christoph Goering @.***>
wrote:

Hi there,
first of all thank you for the project. I did the installation as written
in the doku. I can log in with Jon DOE but, if I register a new user I see
in the DB but no email is triggered. And yes, I am running php artisan
queue:work.
Do you have any idea, why this might be?
—
Reply to this email directly, view it on GitHub
https://github.com/devaslanphp/project-management/issues/105, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A57XVW3VXYXEQKFIEKPYVNLYQ52ZTAVCNFSM6AAAAABCPFS3RKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEYDKMJWGY4TCOA
.
 Im sure it will work.
On Mon, Jan 29, 2024 at 1:30 PM Christoph Goering @.***>
wrote:

I think the problem is, it is not even creating a job. I create a user but
nothing happens in the jobs table in the DB. The failed_jobs are also
empty.
—
Reply to this email directly, view it on GitHub
https://github.com/devaslanphp/project-management/issues/105#issuecomment-1914502307,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A57XVW7KEEQZNLEYOCYXK4DYQ6B6BAVCNFSM6AAAAABCPFS3RKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJUGUYDEMZQG4
.
You are receiving this because you commented.Message ID:
@.***>

Make sure you have this QUEUE_CONNECTION=database into the .env file.

@ExtremOPS i just have installed and can confirm the same

@mihaisolomon

add settongs to env file
create docker image
deploy the project
Accces the project from browser (with predefined user)
add a new user

the result shoud be no email sent (to newly added ) when you adding a user on the admin panel

It is worth mentioning that  the "php artisan queue:work" command is not running automatically after docker container starts, either way if this service is started. Even after starting this manually adding the user from admin panel will not send the email.

From what I see it's not configured to run the job from the start, I created a PR https://github.com/devaslanphp/project-management/pull/112 with the supervisor that should fix one of the problems.
On the rest I'll investigate and update this PR


converting to eks 1.22 kubectl-convert installation issue - docs

Hey guys, was a pleasure talking with you on AWS summit TLV
Context
I stumbled across your upgrade to eks 1.22 tutorial and your guide to install kubectl-convert is specific to linux kernel
#installing kubectl convert
curl -LO "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/linux/amd64/kubectl-convert" && sudo install -o root -g root -m 0755 kubectl-convert /usr/local/bin/kubectl-convert

If you want to make it more agnostic I suggest adding OS mentioning like
#installing kubectl convert on linux
curl -LO "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/linux/amd64/kubectl-convert" && sudo install -o root -g root -m 0755 kubectl-convert /usr/local/bin/kubectl-convert

#installing kubectl convert on macOS
# Intel
curl -LO "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/darwin/amd64/kubectl-convert" && chmod +x ./kubectl-convert
# Apple Sillicon
curl -LO "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/darwin/arm64/kubectl-convert" && chmod +x ./kubectl-convert

sudo mv ./kubectl-convert /usr/local/bin/kubectl-convert && sudo chown root: /usr/local/bin/kubectl-convert

or something like it, would do a PR but didn't find it, just my suggestion :)

@Itamar-BY Could you please implement the fix?

Fixed:)


sample test case #3

sample test case to get started with unit tests.
Don't forget to add PHPUnit as dev dependency in composer.json file

More details for Unit testing in php using PHPUnit https://phpunit.de/manual/current/en/writing-tests-for-phpunit.html

@developersanjog Also consider the template I provided in comment


Custom cache configurations not working

Node Version: v18.17.1
NPM Version: 10.2.0
Prisma Client: 5.17.0
prisma-extension-redis: 1.0.0
ioredis: 5.4.1
Redis Server: 7.4.0
Hi, when I try to cache using cache: true it works but when I try to cache using a custom cache configurations I get an error. Here is the code I used:
try {
      const product = await prisma.product.findMany({
            where: {
                name: {
                      contains: search
                }
            },
            take: 5,
            cache: { ttl: 60 * 5, key: getCacheKey([ { prisma: 'Product' }, { search } ]) }
        });
} catch (error) {
    console.error(error)
}

I get this error:
ERROR  EXECABORT Transaction discarded because of previous errors.

 at parseError (node_modules\redis-parser\lib\parser.js:179:12)
 at parseType (node_modules\redis-parser\lib\parser.js:302:14)

My prisma client looks like this:
import { PrismaClient } from '@prisma/client'
import { Redis } from 'ioredis'
import { getCacheKey, getCacheKeyPattern, PrismaExtensionRedis, type CacheConfig } from 'prisma-extension-redis'

const redis = new Redis()

const cache: CacheConfig = {
  ttl: 60,
  storage: {
    type: 'redis',
    options: {
      client: redis,
      invalidation: { referencesTTL: 60 },
      log: console
    }
  }
}

const prismaClientSingleton = () => {
  const client = new PrismaClient()
  return client.$extends(
    PrismaExtensionRedis({ cache, redis })
  )
}

declare const globalThis: {
  prismaGlobal: ReturnType<typeof prismaClientSingleton>;
} & typeof global;

const prisma = globalThis.prismaGlobal ?? prismaClientSingleton()

export { prisma, getCacheKey, getCacheKeyPattern }

if (process.env.NODE_ENV !== 'production') globalThis.prismaGlobal = prisma

So I have been doing more tests and if I say cache: true, it works however its when I try to do a custom cache configuration I get the error I spoke of above. Here is the custom configuration that I am using:
cache: { ttl: 60, key: getCacheKey([ { prisma: 'Product' }, { search } ]) }

Hi @jeky1950 ,
Could you provide an example project that I can use to replicate this bug?

Hi @yxx4c
Here is a repo with the bug reproduced

Hi @jeky1950,
Thanks for sharing the repo. I ran some tests and didn't encounter the error you mentioned. Could you please try deleting the node_modules folder and any other relevant cache? If you haven't done this already, it could help resolve the issue.
The only difference in our environments is that I'm using DragonflyDB instead of Redis, but I don't think that should lead to the error you're experiencing. I suspect it might be related to the cached files, but please keep me updated if that's not the case. Additionally, if you're on Discord, feel free to reach out to me there (same ID).

Hi @yxx4c
Thank you for your suggestion. Unfortunately it wasn't the problem. You see I am still new to Redis and after some time trying to find a solution to this error message that I was getting on the repo I had sent you:
ReplyError: EXECABORT Transaction discarded because of previous errors.
    at parseError (C:\Users\Antony\Documents\customized-cache-error\node_modules\redis-parser\lib\parser.js:179:12)
    at parseType (C:\Users\Antony\Documents\customized-cache-error\node_modules\redis-parser\lib\parser.js:302:14) {
  command: { name: 'exec', args: [] },
  previousErrors: [
    ReplyError: ERR unknown command 'JSON.GET', with args beginning with: 'prisma:user'
        at parseError (C:\Users\Antony\Documents\customized-cache-error\node_modules\redis-parser\lib\parser.js:179:12)
        at parseType (C:\Users\Antony\Documents\customized-cache-error\node_modules\redis-parser\lib\parser.js:302:14) {
      command: [Object]
    }
  ]
}

I was able to determine that the error was because I did have the RedisJSON module installed on my redis server. And this ultimately fixed my problem. Thanks again for all your help. You have create an amazing tool and I appreciate all the work that you have put into it. I hope more people get to use it.
Sincerely from Zimbabwe


[test] Adapt helloworld tests to build systems

Adapt to new test syntax.

Reformat to be consistent with the coding style of ReFrame.

Fixes #395

@jenkins-cscs retry dom


Fix typo with wildfly in create command and docs

What is the purpose of this change? What does it change?

Fixes a typo assuming we are not asking user to create a component named wildly
Was the change discussed in an issue?
fixes #???

How to test changes?

/ok-to-test

/retest

/lgtm


New Rule: Empty Args

With Sass, if a function or mixin doesn't have any arguments, the parenthesis can be left off of the definition and, in the case of mixins, the invocation as well

Started to look into this one. Experience is slightly different. Parenthesis can be left off of the definition of mixins but not functions. When I try I get a Sass error;
@function baz ": expected "(", was "{"
While I can drop the parenthesis off both the invocation of functions and mixins.
// Causes an error
@function baz {
  @return 'baz';
}

// Is ok
.baz {
  content: baz;
}

// Is ok
@mixin qux {
  content: 'baz';
}

// Is ok
.qux {
  @include qux;
}

Check out this Sass Meister:
http://sassmeister.com/gist/17649e207a82c4efb946
The function isn't being invoked if it doesn't include (), you just happen to have named the function and its return the same, making it appear to be OK.
There is, however, a discrepancy between Libsass and Ruby Sass as to whether a function can be defined w/o (). Libsass allows it, Ruby Sass does not.

Haha oh dear. Amateur mistake!

I trusted you man! I trusted you! :stuck_out_tongue_closed_eyes:

Hahaha :grin: - Thoughts on above?

Yah, only for mixins makes sense to me


[Bug] Empty Search Attributes

What are you really trying to do?
I would like to view the status of all current workflow executions, but the status is empty.
Describe the bug
I started a new workflow and then listed the running workflows, but the status and email are empty.
Minimal Reproduction
$ cd background-checks
$ ./start
$ ./run-cli bgc-company --email<EMAIL_ADDRESS>Created check
$ ./run-cli bgc-company list
Background Checks:
ID: <RUN_ID> Email:  Status:

Environment/Versions
MacOS 13.5 Ventura
Intel i9
commit: 0425e4069dffad892061058a090bc4ad8d41e371

I think this is fixed by #47 because this is working for me now. Please retry once that is merged and confirm.


[$500] Android - Room - Unable to refocus on the room name field after defocusing it

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Issue found when executing: PR https://github.com/Expensify/App/pull/29771
Version Number: v1.3.92-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation: @
Action Performed:

Launch New Expensify app.
Go to Start chat > Room.
Enter a room name.
Remove focus from the room name field by tapping on the blank space.
Tap on the room name field.

Expected Result:
User is able to focus on the room name field with no issue.
Actual Result:
User is unable to focus on the room name field.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/115492554/fc7e4f03-d377-40a3-9057-e3299b125fe6

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01cdcda8d4b7dff0de
Upwork Job ID:<PHONE_NUMBER>922177024
Last Price Increase: 2023-10-27

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

I think team is planned to revert the issue PR - https://github.com/Expensify/App/issues/28877#issuecomment-1782952190

Thank you for letting me know! I am going to keep an eye to see if we revert it

We're reverting the offending PR

We can close this issue since we're CPing the revert and it'll be tested again in staging.


links to linter docs should remove "en-us" from the link

Also I'm considering adding some minimal testing:

link doesn't include en-us (not sure if that's possible, need to get redirect target)
link appears to point to appropriate page (as opposed to being a 404 or something similar)

@StephenWeatherford do you mean aka.ms links? Where are these links in our extension found? That can likely be managed by our content developers manually as it is low overhead.

Yes, the aka.ms links.  Won't take long, just needs to be done.  Would really like to have testing in place as part of it, though, if possible, since right now testing has to be done manually.

Okay, so would run through all of the bicep aka links, then shave off the en-us in all of them? Or you're saying, any time a link is put into the bicep core code base, we run tests to detect en-us?

Getting too complex.  I went ahead and updated all the aka links - now I get the correct localized pages.  I've changed this work item to add tests that whenever they are run will look through and try to find links that incorrect point to en-us.

Dup of #5445


⚠ Samsung has degraded performance

In 9491173, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2532 ms

Resolved: Samsung performance has improved in 140a31a.


Update nav logo to match models.accordproject.org

The Accord Project nav logo on the template library page is inconsistent with the nav logo on models.accordproject.org , which uses the correct logo.

@Michael-Grover Can i take up this issue?

sure, thanks

@jeromesimeon Do i have to file a PR for this against the branch cicero-0.20.0 ?

@arteevraina That one can go directly against master! (the logo was only on the branch, but this one is general styling).

@jeromesimeon Thanks, Sending a PR shortly :)


Read Snowflake views definitions with the Snowflake-specific query

Impact

[x] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Hello again! More enhancements for the Snowflake database, yes.
Now I would like to introduce you minor enhancement about the correct Snowflake view definitions reading.
Why did we do it? Because one of our users reported about NPE during the schema comparison in the Snowflake.
The error was here: https://github.com/liquibase/liquibase/blob/master/liquibase-core/src/main/java/liquibase/snapshot/jvm/ViewSnapshotGenerator.java#L86
A definition can be null in some side cases. AbstractJdbcDatabase#getViewDefinition can return null explicitly.
Unfortunately, we could not reproduce this error. And decided to improve reading definitions for the Snowflake database at least. To increase the chances of the fact that the NPE will not happen again.

Example for view:
CREATE OR REPLACE TABLE employees (title VARCHAR, employee_ID INTEGER, manager_ID INTEGER);

INSERT INTO employees (title, employee_ID, manager_ID) VALUES
    ('President', 1, NULL),  -- The President has no manager.
        ('Vice President Engineering', 10, 1),
            ('Programmer', 100, 10),
            ('QA Engineer', 101, 10),
        ('Vice President HR', 20, 1),
            ('Health Insurance Analyst', 200, 20);

CREATE VIEW employee_hierarchy (title, employee_ID, manager_ID, "MGR_EMP_ID (SHOULD BE SAME)", "MGR TITLE") AS (
   WITH RECURSIVE employee_hierarchy_cte (title, employee_ID, manager_ID, "MGR_EMP_ID (SHOULD BE SAME)", "MGR TITLE") AS (
      -- Start at the top of the hierarchy ...
      SELECT title, employee_ID, manager_ID, NULL AS "MGR_EMP_ID (SHOULD BE SAME)", 'President' AS "MGR TITLE"
        FROM employees
        WHERE title = 'President'
      UNION ALL
      -- ... and work our way down one level at a time.
      SELECT employees.title,
             employees.employee_ID,
             employees.manager_ID,
             employee_hierarchy_cte.employee_id AS "MGR_EMP_ID (SHOULD BE SAME)",
             employee_hierarchy_cte.title AS "MGR TITLE"
        FROM employees INNER JOIN employee_hierarchy_cte
       WHERE employee_hierarchy_cte.employee_ID = employees.manager_ID
   )
   SELECT *
      FROM employee_hierarchy_cte
);

Things to worry about
CREATE_VIEW_AS_PATTERN has private status in the original class. Maybe getter will be better. But I just stole it.
And trimming of the ";" symbol looks not good, but this is what the definition from Snowflake looks like.

Thanks for the fix and the good PR description. Mostly the PR is:

Changing to a different query to get the view definition which should be better than in the generic information_schema query
Ensuring the trailing ; gets removed

Correct? The NPE is being fixed by the fact that your new query should not be returning null definitions anymore?
I'm wondering if a better structure to the changse are:

Instead of overriding getVewDefinition() so much, if you move your new query to a GetViewDefinitionGeneratorSnowflake class that query will get plugged into the exiting logic, just using a different query. Less duplication and also taking advantage of any bugfixes/feature-improvements in the base getVewDefinition method in the future.
If you need to remove the trailing ; you may still have to override getViewDefinition(), but only with a call to super.getViewDefinition() and removing a trailing ; if the value is not null.
Add a "if not null" check in ViewSnapshotGenerator, since that code needs to be handling null values coming back from getViewDefinition and is not. That is not just a problem with snowflake, and would be good to fix regardless.

Think those changes would make it better, @LonwoLonwo ?

Did you get a chance to see my comment above, @LonwoLonwo ?

Oh, sorry, @nvoxland for the late response. I missed the answer. Thanks for the reviewing and tips.
Do I need to add any logging in case like this?:
if (definition == null || definition.isEmpty()) {
            return null;
}

Do I need to add any logging in case like this?:

I don't have a strong opinion one way or the other.
It probably wouldn't hurt to add debug-level logging there. It can help people troubleshoot what's going on, and that's always good. Just as long as it's not such a common scenario that it's not adding value for troubleshooting.

Ok, logging added. I hope that the "info" level is enough.

Initial/Pre-Review Thoughts
Changes are looking good, thanks for the updates.
Questions I have:

None

Potential risks:

None

What could make the full review difficult:

Nothing

@filipelautert and @LonwoLonwo, hello!

Is it possible to add an integration test for this fix?
Can you please give me a bit more description on how to reproduce the bug? Was the diff between an empty SnowflakeDB and one poulated with the objects in the example SQL you provided?

Also: nice to see you again, @LonwoLonwo! 👋🏻
Internal Functional Snowflake Test Suite Started Here --> [Snowflake functional test suite started here: https://github.com/liquibase/liquibase-pro-tests/actions/runs/4622799974.

Nice to see you again @XDelphiGrl also
Unfortunately, we could not reproduce the issue of our customer. Perhaps there were problems with the presence of permissions from the user or broken data. Therefore, it turns out that I added an explicit view definition reading for Snowflake and the processing of potential NPE handling. I apologize that I could not help anymore.

@LonwoLonwo, I guessed that would be your answer. Your PRs always have excellent repro steps! I understand (and have experienced) being unable to replicate a bug that happens only in a customer environment. I'd be interested in hearing if this fixes their problem.
I'm running our (internal) Snowflake functional test suite. This should at least tell us if there is something impacted by this change. I see some failures but I am 93% certain they are not due to this PR. We're having a week of strange build issues...
I hope you have a lovely day!


Composition with different step sizes and different right-hand-sides

A potential use case involves finding B-series for operator splitting methods.  To do this, we would need composition to work with different step sizes (this should be easy) and different generic right-hand sides; e.g. abstract symbolic functions f and g.
The substeps of the splitting method might be solved exactly or with a RK method.
@ranocha do you have any suggestions about how to implement this?

We could generalize compose to colored trees and their associated series to deal with this. Right now, we only have substitute for series based on colored trees. To implement compose, we need to implement splittings of colored trees in RootedTrees.jl, in particular the SplittingIterator.


[Support v4] Fix unqueue first then publish

Could this be merged to a v4 support branch?

IMO a good implementation would commit events without any commit() call, just setting rules about refresh frequency provided from the configuration.
On the implementation side this would be a simple setInterval calling the publishers on a regular basis if uncommitted events remain.

We also should deprecate the auto commit feature sooner or later.


CPUParticles2D not working on AMAZON TATE Device

Godot version:
3.1 beta (all of them)
OS/device including version:
2012 Amazon Kindle Fire HD 7” (GOOGLE AMAZON TATE) running CyanogenMod version 11
PowerVR SGX 540
ANDROID 4.4.2
Issue description:

For some reason CPUParticles2D don’t seem to work at all, I havent seen this problem on any other device I've tried so far. I know its a bit old but it works nicely with 2.15
Minimal reproduction project:

OpenGL-ES Info.txt
logcat.txt
ParticleTest.zip

The hardware is just too low-end to support the features needed for CPUParticles2D. Your device only supports a maximum of 8 Vertex attributes. It is the same problem faced in https://github.com/godotengine/godot/issues/27407
There are no plans to create a renderer that supports such low-end devices. Sorry.


Bump @marp-team/marp-core from 3.9.0 to 4.0

This issue only asks to marp-vscode bump version of @marp-team/marp-core from 3.9.0 to 4.0 @marp-team/marp-core's releases.

It's in progress at #462. The blocker for bumping version is that an audit for vulnerability check is failing. I'm going to work for that in this year, but currently I'm busy for my other projects. We are welcome to a pull request to fix audit on depending libraries from contributors.


Add --fail-level-output flag

Intended for having CI run across a codebase, and only fail at a certain level
And provide output for only the cops causing CI failure, without a spam of e.g.
refactor offenses.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] 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 rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

I'm fine with the suggested change in general, but I don't like much the name of the flag, as it doesn't seem to me it accurately conveys what it does. Perhaps --display-only-fail-level-output? Or maybe others will have a better idea about this.

Perhaps --display-only-fail-level-offenses or --show-only-fail-level?

Or we can just make this the default behavior if --fail-level is supplied. Might save us from thinking about the extra command-line flag. :-) Can't imagine how we can actually make this short, I like @sunny's first suggestion the most.

I wasn’t sure about the name either, I don’t mind verboseness though, I can change it to —display-only-fail-level-offenses. If we want to make it the only way —fail-level works, we’d need to think about autocorrect. Ignore level & output every correction? Output no corrections? Output only those corrections that would cause that fail-level to fail? (I lean toward the first)

When you put it like this - let's just go with —display-only-fail-level-offenses and simplify our lives. :-)

Thanks!


Creating database layer.

Task:

create a database layer.
setting up the connection.
connection pooling.

Note: Please assign the task to yourself and start working on it.

Done


How can I use .py file to do style-transfer instead of CLI command?

Dear Crowsonkb.
How can I use .py file to do style-transfer instead of CLI command?  could you please provide such python code?  thanks very much!

thanks lot!


Fix client_ssl_test and server_ssl_test when built against OpenSSL 1.0.2.

This should hopefully fix the remaining bugs we found in #24960.

@veblush FYI: still getting one weird failure, will look into it this week. Sorry for the delay.

@matthewstevenson88 No worry and please take your time!


Wrong constraint verification

Constraints (eg ValueSizeConstraint) ignores multibyte encodings
How to reproduce:
from pyasn1.type import char, constraint

char.UTF8String("ф").subtype(subtypeSpec=constraint.ValueSizeConstraint(1, 1))

>>> 'ф'.encode('utf-8')
b'\xd1\x84'

Thank you for your report!
Here's a quick fix for you to try out. I'm still working on adding tests and looking for possible corner cases.

It turned out that it's better to have character types holding unicode contents (as opposed to serialized octet stream) so that all duck-typing features would work out-of-the-box for character types. I've updated the branch I'm tackling this problem in.

Great!
Thank you.

Merged to master. Please raise an issue in case of any troubles.


Add GoogleSheetsQuery command

Add GoogleSheetsQuery command for querying a Google Sheet for data to a tab delimited file.

Added command


Support react 0.14 prereleases

This fix will let you use any of the react 0.14 betas and react 0.14.0-rc1.
Semver won't allow prereleases by default.

Interesting, is this just npm? Bower doesn't seem to mind including betas in the range.

AFAIK Bower doesn't use peer dependencies.
But this behavior is in accordance to semver. You can try it out here, just put in react and >= 0.13.0 < 0.15.0, and then try >= 0.13.0 < 0.15.0 || ^0.14.0-alpha.

Right on, LGTM.

Released in 0.6.5

thanks!