Small visual patch

Added the following:

Added procps-ng package to env. This is required for making nextflow reports and timelines.
Added the rich python library to adda progress bar to the consecutive gaussian blur process to aid users in estimating how long the procedure might take.

@ViriatoII If in the future, you are ok with me merging PRs that mostly change reporting and add packages like this by myself, let me know. For now I still wanted to you to check.
I am considering adding argparse options for BOXSIZE and LOOPNUM as currently its a bit annoying to catch and add these in nextflow processes.
Would you be ok with that in a next PR?

Hey @FloWuenne,
Yes, I'm fine with you accepting PRs. I'm just wandering if it makes sense splitting Mindagap into 2 branches, so that people without nextflow might also directly use the script, or are the nextflow additions invisible for a user calling the script with python mindagap.py input.tiff ?

Ok great! I will merge PRs like this myself in the future if not changing major things.
No worries, this PR and any other PRs geared for making use in nextflow won't break the usage as a script. In fact nextflow just uses the Mindagap script from within a conda environment, docker or singularity container. I always test the script from within conda before making a PR!


Slides do not reorder correctly

When I drag slides left and right to reorder them a few things happen
(1) The Your Slides window hangs
(2) The slides do reorder, but not in "Your Slides", even if the kernel is restarted.  In fact, if I flip through the slides, the orange box jumps around in "Your Slides".  This means I cannot change the order a second time!

@carolynlphillips thanks again for the report, and for helping us build this boat as we sail it, as they say!
As we discussed, no problem if you can't share the notebook in question... but perhaps:

what other players might be involved? any other funky extensions that need to be considered for integration?
what browser/os/etc. are you seeing this on?
is anything going on in the javascript console?

If all else fails, perhaps we can jump on gitter and hammer out the step-by-step, see if it's reproducible... would really help build the test case.
I've wanted to get closer to the reveal model (#24, with "down" subslides, #42 as a maximized view), or just go for it and let authors make whatever kind of graph they want... either way, would probably want to use something for the heavy lifting like:
http://marvl.infotech.monash.edu/webcola/examples/gridifiedSmallGroups.html

I am seeing this behavior on OSX/Chrome. I have livereveal and dashboard extensions installed too, fwiw.
Also, if I unlink and relink all the slides to cells in proper order, it seems to preserve some old slide order.

Same observations here. If I start making slides in a fresh notebook, without manual reordering, the slide order in the editor gets quasi-randomized as soon as the 11th slide is added. The actual presentation order of the slides is still correct, but it makes any reordering from this point onward nearly impossible.

I also have this problem. I've added 10+ slides and am not able to rearrange them.

I also have the same problem, every browser on OS X, and with any number of slides greater than 4 or 5

Same issue here on OS X.  Tested on chrome and safari.

Also having this problem. :( What a pity!

Also seeing this, or at least the order of the slides in the slide editor is not the same as the order of the slides as they appear in the actual presentation (which makes it impossible to do any further reordering).
Using OS X and chrome.


endpointAuthentication property name is confusing

This has caused confusion on the job a couple times.  It is not about the TrustManager authenticating the server's cert, or the KeyManager allowing the server to verify us, but is about checking that the subject name on the cert matches the hostname of the request.  This is called endpoint identification in the relevant RFC.  Turning off endpointAuthentication does not affect trust management.
Furthermore: is this a property of the client, or the request?  Perhaps a default on the client, but we might want to turn it on or off depending on hosts known to have misidentified certs.

Do you have a suggestion for an alternate name?

It's called "endpoint identification" in the RFC and the [Java API](https://docs.oracle.com/javase/7/docs/api/javax/net/ssl/SSLParameters.html#setEndpointIdentificationAlgorithm(java.lang.String).
People seem to be confusing the existing name either for mutual authentication (2-way SSL) or cert validation (i.e., what the TrustManager does).

So endpointIdentification or checkEndpointIdentification? The latter is getting pretty wordy but the former is kind of meaningless. I do agree that it currently suggests mutual authentication which is a no-go.

I like checkEndpointIdentification.

:+1: Sounds good to me. When it comes to security, I prefer wordy over ambiguous.

:+1:  Descriptive is good if it's causing problems. We should make sure to reference RFC and the original name, so people have no trouble finding what they are looking for if they come from the RFC/java side.


accessing user/save creates empty campaign

Steps to reproduce

log in as user
access titled URL
navigate to user home

Expected results
no change in campaign numbers
Actual results
empty campaign is created
Suggest
Add validation to campaign class

fixed by https://github.com/sleepbusorg/sleepbus.org/commit/5ac264742c1125aa80b96fae4ac1386cec942080

not a great fix, will be cleaning up classes, entity creation and validation and re-address then


A fast way to generate a random point ?

Hi,
Is DotSpatial support a fast way to generate an set of random points on some geometry ? Or what is the better way to generate random point inside specific geometry using DotSpatial ?
Thank you.

https://github.com/sindizzy/DSW/wiki/RandomPoints


wrong import path?

hello
when I do:
go get github.com/bugfender/bugfender-integration-elasticsearch
I get the following error on windows:
unrecognized import path "bugfender-integration-elasticsearch/pkg/dummy": import path does not begin with hostname
unrecognized import path "bugfender-integration-elasticsearch/pkg/elasticsearch": import path does not begin with hostname
unrecognized import path "bugfender-integration-elasticsearch/pkg/integration": import path does not begin with hostname
can this be fixed?

Which go version are you using? You should use at least version 1.15

Ah I see what you mean. Will fix.

Fixed in #797d9e2


Add frontend side pagination.

Changes

How to test these changes?

Related issues
Resolves https://github.com/bugout-dev/moonstream/issues/257

Loading heavily loaded address like uniswap contract
0x7a250d5630B4cF539739dF2C5dAcb4c659F2488D loads indefinitely and returns 500

Moved unresolved issues in to https://github.com/bugout-dev/moonstream/issues/264


Add logger dependency

Ruby 3.3.5 started to issue a warning if logger is loaded from standard library.

How can I reproduce this warning? Ruby 3.3.5 and bundle exec rspec gives nothing about that.

Not sure why you don't see it.
I do not have anything special setup, and my Ruby versions are managed with rbenv.
This single file (no need for Gemfile) reproduces the problem:
require 'bundler/inline'

gemfile do
  gem 'filewatcher', '2.1.0'
end

$ ruby test.rb
/home/vagrant/.rbenv/versions/3.3.5/lib/ruby/3.3.0/bundled_gems.rb:75: warning: logger was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.5.0.
You can add logger to your Gemfile or gemspec to silence this warning.

$ ruby -v
ruby 3.3.5 (2024-09-03 revision ef084cc8f4) [x86_64-linux]

And just to confirm, this eliminates the warning:
require 'bundler/inline'

gemfile do
  gem 'filewatcher', '2.1.0'
  gem 'logger', '1.6.1'
end

This single file (no need for Gemfile) reproduces the problem:

That's my file inside this repo:
require 'bundler/inline'

gemfile do
  gem 'filewatcher', path: './'
end

And there is an output:
~/Projects/ruby/filewatcher/filewatcher (main|✔)
> ruby .test.rb

~/Projects/ruby/filewatcher/filewatcher (main|✔)
> ruby -v
ruby 3.3.5 (2024-09-03 revision ef084cc8f4) [x86_64-linux]

Here is a clean, objective reproduction:
Files
Dockerfile
FROM ruby:3.3.5-alpine
WORKDIR /app
RUN gem install filewatcher
COPY test.rb .
RUN chmod +x test.rb
CMD ["/app/test.rb"]

test.rb
#!/usr/bin/env ruby

require 'bundler/inline'

gemfile do
  gem 'filewatcher', '2.1.0'
  # gem 'logger', '1.6.1'
end

Commands
$ docker build -t temp .
$  docker run --rm -it temp

Output
/usr/local/lib/ruby/3.3.0/bundled_gems.rb:75: warning: logger was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.5.0.
You can add logger to your Gemfile or gemspec to silence this warning.


[TODO] Add new ops like those of handshake dialect in CIRCT

IDEA: fork, join, sink ...

I notice that there are two primitive KPN nodes that I need to implement this HLS DF here that are not regular SDF operators (and don't have regular multiplicities). They have data dependent multiplicities, and I hope (?) they might be sufficient to cover all remaining HLS scenarios.

Diverging execution y = diverge(c, x_0, ..., x_N) with y = x_c
First, a conditional token is received from channel c. Then, the data token is received from channel x_c, and pushed to channel y.
The wait state machine for this node would have to initialize in the wait c state, but then transition to the correct wait x_c state depending on the data recieved. That means the edges in this FSM depend on valid and data.
A more general algorithm to implement these FSMs might determine that the wait states following wait c are a set (just like unordered_pull), but would have to predicate these edges with a control-flow witness (here thats the token received from c) to avoid consuming from an input that must stay untouched.

Converging execution y_0, ..., y_N = converge(c, x) with y_c = x
First, a conditional token is received from channel c. Then, the data token is received from channel x, and pushed to channel y_c.
This seems trivial, as the resulting conditional should already be correctly synthesized to an FSM, and pushing to a channel is not priviledged like pulling.

Regardless, this type of DF needs to be supported by Olympus for this to make sense. Since Olympus does not consider the bodies of the processes, a stream channel should probably result in the right interface for the IP block. The questions is whether the multiplicity can be omitted.


Flight enchantment on Elytra

Just a random idea. But what would you say to a flight enchantment (rare) which can only be placed on an elytra, granting you creative flight when wearing it. It sort of is in keeping with vanilla (requiring an elytra to fly).
Not sure if it fits your mod or if it is possible (it might mean creating an otherwise non-functioning elytra because of the way it works? Not sure)
Feel free to dismiss/close this suggestion.

Custom elytra enchants are a good idea.
But i dont know if it makes sense to do it as creative flight.  I already have a food that does that, and other mods have all sorts of ways to fly.  Maybe a way to increase speed or some other way to improve them.

Im not the biggest fan of creative flight myself, but if I do add it in future ill have to find some new different way.
In one of the recent updates [1.10.3] I changed the Multi-Jump enchantment so that it can go on more gear, not just boots.  So you can put it on elytra.  Try it out, it works really well, basically you do not need fireworks. You can jump a few times from the ground, start flying, then while gliding you can do more mid-air jumps to keep your height going.
For this I am not going to add other elytra enchants.


CASM-3638 Add examples for global parameters

Description

Checklist Before Merging

[ ] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[ ] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[ ] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

This time, without spaces in filenames.


Monerodocs.org -> GetMonero.dev

The MoneroDocs.org website is now defunct. Please let me know if I missed anything!

the unforeseen scenario of an mkdocs fork inside the monero-project repo (https://github.com/monero-project/monero-docs)  happened and is now hosted @ docs.getmonero.org , we can leave this open as an example of how its done for convenience until a replacement PR is made or close it before then


fix: alert processing crash in get_route

In c4914b5 we removed a seemingly unused clause of get_route that allowed passing nil as the route ID. Though this was not covered by any tests, it turned out to be used when an uncommon set of conditions occurred, and this caused processing of the "oldest" group of alerts to always crash in production.
This adds two tests that both fail in the presence of this bug: one for ServiceInfoCache that explicitly tests passing nil as the route ID, and one for InformedEntityFilter replicating the conditions that cause nil to be passed as the route ID.
This also fixes an incorrect pattern in InformedEntityFilter that never would have matched, because it was matching a subscription type (which only has atom values) against a string.

"if I have a subscription to stop X without a route specified, does that mean that it should match an alert for a specific route serving my stop (i.e. match any route)?"

In this case the alert is for a different stop than the one your subscription is for, so it should not match (note in the test the subscription is for "some stop" but the informed entity applies to "some other stop").
The specific condition that triggers the bug is:

An alert exists with an informed entity that:
a. applies to a specific route and stop
b. applies to the RIDE activity at that stop
A "stop subscription" exists for a stop that is not the specific stop the entity applies to

The informed entity matching goes like this:

Trip matching doesn't apply since the entity does not specify a trip
Route type matching doesn't apply since neither the entity nor the subscription has a specific route type
Direction ID matching doesn't apply since neither the entity nor the subscription has a specific direction ID
Route matching doesn't apply since the route of a "stop subscription" is nil (this is considered to match all routes)
For stop matching:
a. Since neither the origin of the subscription nor the stop of the entity are nil, we are not in any of the special cases of stop_match?
b. Since neither the subscription's origin nor destination are the same as the informed entity's stop, we call into in_between_stop_match? to determine whether the entity's stop is between the subscription's origin and destination
c. This is not an accessibility subscription *⃣, so we are not in the special case of in_between_stop_match?
d. We continue into stops_in_between, where:

RIDE is among the entity's activities, so that match succeeds
We construct a "route filter" that includes the subscription's route, which is nil ⚠
We pass that filter to ServiceInfoCache.get_route, which crashes 🛑

*⃣ It might seem weird that a "stop subscription" could exist outside the context of accessibility subscriptions, since I had thought that was the only way to create one. What might have actually happened on production is that the offending subscription was an accessibility one, but was not caught by the special case of in_between_stop_match? due to the string/atom confusion that this PR also fixes. However, the existing tests seem to assume a non-accessibility stop subscription can exist. This could simply be a case of the test data being "unrealistic" and the validations not forcing it to be realistic. In any case, our bases should now be covered.

My guess is that the number is zero, the tests are just not using 100% realistic data, and the production crash was a result of the string/atom mixup in in_between_stop_match?. But yeah, might be a good thing to follow up on.

@digitalcora Ok, yeah I guess I just made an assumption that maybe existing data was causing the crashes.


chore: updating localnet config update log message for --no-update case

Proposed Changes

Minor patch to change wording when for log message informing on update of the localnet

@neilcampbell reverted the changes, the latest is just tweak in the message


Key error while updating fees

Traceback (most recent call last):
  File "/root/lndmanage/lndmanage.py", line 5, in <module>
    main()
  File "/root/lndmanage/lndmanage/lndmanage.py", line 663, in main
    asyncio.run(_main())
  File "/usr/local/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/local/lib/python3.10/asyncio/base_events.py", line 646, in run_until_complete
    return future.result()
  File "/root/lndmanage/lndmanage/lndmanage.py", line 603, in _main
    await parser.run_commands(lndnode, args)
  File "/root/lndmanage/lndmanage/lndmanage.py", line 584, in run_commands
    feesetter.set_fees(
  File "/root/lndmanage/lndmanage/lib/fee_setting.py", line 152, in set_fees
    channel_fee_policies, stats = self.new_fee_policies(init)
  File "/root/lndmanage/lndmanage/lib/fee_setting.py", line 223, in new_fee_policies
    cumul_fee_rate += self.channel_fee_policies[
KeyError:'xxxxxxxxxxxxxxxxxxxxxxxxxxx:1'

I already tried clearing the cache, but it made no difference.

It looks like an not all open channels appear in lncli feereport. Hope the error solved itself, but if it happens again, this would be the first I would check.

Could you check whether the unknown channel appears in lncli feereport?

It looks like that the channel is not operational because your peer has forgotten about the funding process (usually after 2 weeks of an unconfirmed opening). With the latest mempool situation, this happens quite often, see for instance this issue https://github.com/lightningnetwork/lnd/issues/7426 and how to fix it. I'll nevertheless supply a fix for that.

This is not my lucky day.. First, I lost 5M sats by blindly following the instructions in that issue.

Sorry this happend.

And to make it even worse, it did not even solve my problem with managelnd, because now I simply get a different error:

This is an issue in computing mean/median values for forwardings. #146 fixes the key error (it adds a better error message) and removes the warnings.


Images not Exporting?

When I export a comp, bodymovin doesn't create an images directory. Is this something I need to do manually? Is it going off from the AE comp's links?
My animation doesn't have any of the images visible (I assume because the links are broken), but I can tell they're still animating correctly by inspecting the demo.

hi, can you share the .aep and the demo.html file?

BM-Image_Test-folder.zip

I also just noticed a problem with images exporting – it had been working fine yesterday morning, and when I exported new updates this afternoon, the image files export into a folder but are not showing up when I look at the animation in the browser.

reference.zip

also including .aep and demo file for reference, thanks!

Found the issue, it will be fixed on the next release.

Is there a way to link these up manually within the json? I've unminified it in order to find where it calls my images, but I don't know the syntax for hooking up a path. I'm just hoping to troubleshoot before a deadline!

Wondering also if this bug is linked to the most recent version of creative
cloud; I noticed the image exporting broke the morning after my creative
cloud auto-updated. Also curious if you have an idea of when the next
release will be – aiming to launch a project this coming Monday.
thanks much!!!
R
On Tue, Jun 13, 2017 at 9:09 AM, snfisch<EMAIL_ADDRESS>wrote:

Is there a way to link these up manually within the json? I've unminified
it in order to find where it calls my images, but I don't know the syntax
for hooking up a path. I'm just hoping to troubleshoot before a deadline!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bodymovin/bodymovin/issues/475#issuecomment-308110103,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AabfJVeMrd8__E8WkP3GLCFd9uZ8rh1Nks5sDooPgaJpZM4N0qak
.

--

Fixed and uploaded.
sorry about the delay but I've been away this week.

Hi, I also have the same issues with broken images when I try to export a simple composition with a simple animation. :(

@jcruz95 if you're still having trouble can you open a new issue with the project?


validate row: Oracle NUMBER with scale doesn't match PostgreSQL DECIMAL with scale

Fractional NUMBER column values do not match equivalent values in PostgreSQL due to differences in CAST behaviour.
For example, in Oracle:
SELECT CAST(CAST(100 AS NUMBER(5,2)) AS VARCHAR2(10)) FROM dual;

CAST(CAST(
----------
100

PostgreSQL equivalent:
# SELECT CAST(CAST(100 AS DECIMAL(5,2)) AS VARCHAR(10));
 varchar
---------
 100.00

The upshot of this is row data mismatches when using default validation type cast behaviour.
A suggested workaround for this specific Oracle to PostgreSQL project is to edit the config YAML file and for each Oracle NUMBER column with scale > 0:

Change type cast to to_char
Add default_to_char_fmt of "99999999.99" (matching p and s with the number of 9s)

The project is to migrate 1000s of database therefore a manual process is not viable. Also the workaround only works for this specific combination of source/target engines because Oracle and PostgreSQL share number formatting masks. An MSSQL to PostgreSQL project could still be a problem.
This is very similar to issue 680 (https://github.com/GoogleCloudPlatform/professional-services-data-validator/issues/680) but I've raised it as an independent issue because there is no common alternative already coded. Issue 680 can utilise existing strftime  functionality. This issue either needs a new engine agnostic alternative to cast or engine specific code to say "if the source and target are Oracle/PostgreSQL then use to_char and build a suitable mask".

This would also require us to track the column data type since we only want to replace cast with to_char if the column is a decimal/number column. An alternative here is to run column validation (SUM, AVG, etc) instead of row level where we need to hash a string value. Column validation would provide the same accuracy.
Otherwise we would need Oracle to Postgres specific code.

The TRIM_SCALE() function was introduced in PostgreSQL 13. Version 11 and 12 are still supported so I need to rethink this strategy.

Could we use the TO_CHAR() for decimals with a specific format? We also support the round() function.
Another option is to add the calculated field in the orchestration of DVT - i.e. in the YAML generation process add calculated fields for to_char() or round() for decimal columns instead of in the DVT code

Yes, I've used TO_CHAR() for decimals with a specific format as you suggest. We can use FM in the format to remove any unnecessary padding.


FIX: mesh operations 5568

Description
Close #5568.
Issue linked
Please mention the issue number or describe the problem this pull request addresses.
Checklist

[x] I have tested my changes locally.
[x] I have added necessary documentation or updated existing documentation.
[x] I have followed the coding style guidelines of this project.
[x] I have added appropriate tests (unit, integration, system).
[x] I have reviewed my changes before submitting this pull request.
[x] I have linked the issue or issues that are solved by the PR if any.
[x] I have agreed with the Contributor License Agreement (CLA).

Hi @Devin-Crawford , I identified the issue, I will use this PR to fix the issue.

Close this PR because branch case (FIX->fix) made a duplicated branch.
https://github.com/ansys/pyaedt/pull/5595


Merge pull request #2 from PVCabrera/main

adding post page edits

updating post page changes from main


Toolkit Reports using income as part of Spending Calculations

Name of Defective Feature
System Settings (Required)
ToolKit Version: v2.1.0
Operating System (Mac/Windows/Etc): Windows 10 Home
Browser (Chrome/Firefox/Safari): Chrome
Browser Version: 65.0.3325.146 (Official Build) (64-bit)
Other Extensions:
Export of Toolkit Settings (Bottom Left of Settings Page)

Settings Export

[{"key":"AccountsDisplayDensity","value":"0"},{"key":"AccountsEmphasizedOutflows","value":true},{"key":"AccountsStripedRows","value":true},{"key":"AdditionalColumns","value":true},{"key":"AdjustableColumnWidths","value":true},{"key":"AutoDistributeSplits","value":false},{"key":"BetterScrollbars","value":"0"},{"key":"BudgetProgressBars","value":"both"},{"key":"BudgetQuickSwitch","value":false},{"key":"CalendarFirstDay","value":"0"},{"key":"CategoryActivityCopy","value":false},{"key":"CategoryActivityPopupWidth","value":"0"},{"key":"ChangeEnterBehavior","value":true},{"key":"CheckCreditBalances","value":true},{"key":"CheckNumbers","value":false},{"key":"ClearSelection","value":false},{"key":"CollapseSideMenu","value":true},{"key":"ColourBlindMode","value":false},{"key":"CompactIncomeVsExpense","value":false},{"key":"CreditCardEmoji","value":false},{"key":"CurrentMonthIndicator","value":false},{"key":"CustomFlagNames","value":false},{"key":"DaysOfBuffering","value":true},{"key":"DaysOfBufferingHistoryLookup","value":"0"},{"key":"DisableToolkit","value":false},{"key":"DisplayTargetGoalAmount","value":"0"},{"key":"EasyTransactionApproval","value":true},{"key":"EditAccountButton","value":"0"},{"key":"EnableRetroCalculator","value":false},{"key":"EnlargeCategoriesDropdown","value":true},{"key":"EnterToMove","value":false},{"key":"GoalWarningColor","value":true},{"key":"GoogleFontsSelector","value":"0"},{"key":"HideAccountBalancesType","value":"0"},{"key":"HideAgeOfMoney","value":false},{"key":"HideHelp","value":true},{"key":"HideReferralBanner","value":true},{"key":"ImportNotification","value":"1"},{"key":"IncomeFromLastMonth","value":"0"},{"key":"LargerClickableIcons","value":true},{"key":"MonthlyNotesPopupWidth","value":"0"},{"key":"NavDisplayDensity","value":"0"},{"key":"PrintingImprovements","value":true},{"key":"PrivacyMode","value":"0"},{"key":"QuickBudgetWarning","value":false},{"key":"ReconciledTextColor","value":"4"},{"key":"RemovePositiveHighlight","value":false},{"key":"RemoveZeroCategories","value":false},{"key":"ResizeInspector","value":false},{"key":"RightClickToEdit","value":true},{"key":"RowHeight","value":"0"},{"key":"RowsHeight","value":"0"},{"key":"RunningBalance","value":"1"},{"key":"SeamlessBudgetHeader","value":false},{"key":"ShowCategoryBalance","value":true},{"key":"ShowIntercom","value":true},{"key":"SpareChange","value":false},{"key":"SplitKeyboardShortcut","value":true},{"key":"SplitTransactionAutoAdjust","value":true},{"key":"SquareNegativeMode","value":true},{"key":"StealingFromFuture","value":true},{"key":"SwapClearedFlagged","value":false},{"key":"TargetBalanceWarning","value":true},{"key":"ToBeBudgetedWarning","value":true},{"key":"ToggleMasterCategories","value":false},{"key":"ToggleSplits","value":true},{"key":"ToggleTransactionFilters","value":"1"},{"key":"TransactionGridFeatures","value":true},{"key":"goalIndicator","value":true},{"key":"highlightNegativesNegative","value":false},{"key":"l10n","value":"0"},{"key":"options.dark-mode","value":true},{"key":"pacing","value":"0"},{"key":"popupCalculator","value":true},{"key":"reports","value":true}]

Steps to Reproduce the Behavior

Open nYNAB
Click Toolkit Reports
Click Spending by Category or Spending by Payee
Look at the tables and see a negative amount of spending

Expected Behavior

I expected to see the amount of money I've spent over a given time period.
Actual Behavior

My total spending shows as negative.  When looking at the lists on the right side with all my "Spending" activity, at the bottom of the list includes all the Payees who I've received income from.  All the income values next to those Payees show as negative amounts.  Since I've saved more money than I've spent, my total spending is a negative number when it should be a much larger positive number.

@axnjackson11 thanks for the report -- this one's on me -- I've opened a PR to fix this in our next release.

Hey @axnjackson11 the fix for this should be in our current release. If you notice the issue still exists and you're definitely running the newest version of the toolkit, please reopen! Thanks!


stream plot images not linking to code pages

Some of the images on the landing page don't link to code. I think only the stream plot examples suffer from this:

after click gives

some more I noticed

also, just to mention, I think the latex rendering bug for x/y is fixed after an update.


Add jambalaya wine pairing

Add turkovich estate white blend to jambalaya recipe as a wine pairing

No longer have access to Turkovich wines :(


RFC: InfiniteHits should load previous pages

When reloading a page with urlSync and page > 0, the infinite hits doesn't load the previous hence it doesn't reproduce the exact same state. We decided to close https://github.com/algolia/instantsearch.js/issues/2750 but we've had other users asking for it.
Proposed API
const widget = instantsearch.widgets.infiniteHits({
  container: string|HTMLElement,
  templates: [InfiniteHitsTemplates],
  showMoreLabel: [string],
  transformData: [InfiniteHitsTransforms],
  cssClasses: [InfiniteHitsCSSClasses],
  escapeHits: [boolean],
+  loadPreviousPagesOnReload: [boolean=false]
}: InfiniteHitsWidgetOptions);

Behaviour
The widget is not aware of the nature of the rendering. Therefore it should check at each rendering if the current page matches the cache of results. If it doesn't it should reload the missing pages.
Cool side effect of that, is that it could handle the changes of hits per page.
Raise your voice ;)
Because we might have a lot of pages,  we should probably add anchors, and go directly to the current page?
The name is long but explicit 🤔
Please comment if you see a better name or you see any other issue.

I prefer to have a button to load previous pages one by one rather than possibly breaking any InstantSearch site just by going to a very far a way page

I prefer to have a button to load previous pages one by one rather than possibly breaking any InstantSearch site just by going to a very far a way page

That's interesting. Where would the results of the previous page go?

This is not completely a reusable example, but my idea is this: https://codesandbox.io/s/q7mp2zyonw (toggle the defaultPage prop in <App /> to see the behaviour I mean)

Met with Sasha and she helped us define the problem and a plan
There is not one problem but 2:

When the user go to a product page and go back to the search
When you share your URL

Based on that we have two proposals:

when the browser load the URL, load n previous results and display a button on top to be able to go back on the first page "There are xxx products, see all of them"
when the browser load the URL, load n previous results and display a button on top to load also the previous page

Next steps:

[ ] build PoC's
[ ] user testing
[ ] pick one

For the second proposal, does that mean that if you're use case 1 (product page then go back), your navigator will put you at a place that will not be the one you were on previously (because the previous results are not displayed?)
Not sure to get this: the two above without loading previous pages at all

We're going to test both and see what works best.
On Tue, Mar 27, 2018 at 4:53 PM, Marie-Laure Thuret <
<EMAIL_ADDRESS>wrote:

For the second proposal, does that mean that if you're use case 1 (product
page then go back), your navigator will put you at a place that will not be
the one you were on previously (because the previous results are not
displayed?)
Not sure to get this: the two above without loading previous pages at all
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/algolia/instantsearch.js/issues/2835#issuecomment-376555221,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjYq-PSAPtC01j8NVM9i4PhfFRvu6qFTks5tilJtgaJpZM4S8ehc
.

--
https://www.algolia.com/
Alexandra (Sasha) Prokhorova
Product Designer
07 69 75 89 79  <07+69+75+89+79>
https://www.linkedin.com/in/alexandraprokhorova
https://www.algolia.com/


Wrong result of JsDiff.diffLines when there is an empty line before a change

var oldStr = "Line1\n" + "\n" + "Line2\n";
var newStr = "Line1\n" + "\n" + "ChangedLine\n";
JsDiff.diffLines(oldStr, newStr);

outputs
[
  {
    "count": 1, // Should be 2
    "value": "Line1\r\n" // Should be "Line1\n\n"
  },
  {
    "count": 1,
    "added": true,
    "value": "ChangedLine\n"
  },
  {
    "count": 1,
    "removed": true,
    "value": "Line2\n"
  }
]

In fact it seems successive \n are replaced by \r and ignored in counts

CLosing in favor of #52


[ENH] Got sdc_unwarp to work with FUGUE

This is a PR to make more public the update of the fieldmaps development branch.

New version of ds054_downsampled has been uploaded (new json files including TotalReadoutTime).
Running circle without cache to fetch the new version


Add jsdoc as a valid demo link in the schema

See this example: https://github.com/carbon-design-system/carbon-for-ibm-dotcom/pull/8500#discussion_r828087569
The Carbon for IBM.com services and utilities have functions that are demoed / documented in JSDoc. Let's add that as a valid value.
I get it that these links are actually documentation not demonstrations... but given how we're primarily indexing patterns and components, I think this is okay.

Closing this. Not needed anymore.


fix(legacy-plugin-chart-paired-t-test): fix paired t-test table chart

This PR fixes several bugs that were breaking Paired t-test table chart.

Chart was using a metric name as a to transform data, which worked only with saved metrics, since backend returns a saved metric as a string. If an adhoc metric was used, chart was still trying to use it as a key, even though it's an object, which led to an 'undefined' error.
reactable-arc was not rendering table body. I replaced it with the latest version of reactable.
CSS for scrollable table content was buggy, caused chart to have 0px height.

What is left to decide is how to handle a case when no group by column was specified. The chart does render when group by is not specified, but it doesn't make much sense. What we can do is either display an error message (see video) or leave it as it is (see screenshot). If we go for the first option, we need to figure out a proper, descriptive error message - the one in the video is just an example.
https://user-images.githubusercontent.com/15073128/108093700-4aea6e80-707e-11eb-8cda-a83ef14df964.mov

UPDATE:
After adding a validator for groupby, user cannot run query without specifying at least 1 group by column.

CC: @villebro @ktmud @rusackas @junlincc

UPDATE:
After adding a validator for groupby, user cannot run query without specifying at least 1 group by column.

<img width="501" alt="Screen Shot 2021-02-25 at 9 37 38 P@kgabryje src="https://user-images.githubusercontent.com/67837651/109259761-bbe20280-77b1-11eb-9564-4d5581dbd0c0.png">
sorry for the late sign-off, @kgabryje when you have a chance, let's remove the second group-by. thanks 🙏
we also need to find a dataset to properly test it.

@junlincc Ugh, what an embarassing bug 🤦 Fixed in https://github.com/apache-superset/superset-ui/pull/979


showAppsToView not working? (Android 9)

Steps to reproduce
Share.open({"showAppsToView": true, "type": "application/pdf", "url": "file:///storage/emulated/0/Download/8e447932-1411-444a-bfa4-aa6845f6a507.pdf"})
Expected behaviour
It should show at least one of the following:

Actual behaviour
Share shows up but it doesn't show any pdf related

Environment

React Native version: 0.61.5
React Native platform + platform version: Android 9, Emui 9.1.0

react-native-share
Version: " 3.2.0
I haven't tested this with an emulator yet and I can confirm that the file exists, not corrupted, and that the app has permissions needed

I installed Adobe PDF and "Combine to Adobe PDF" shows.
Is this an intended behavior or am I doing something wrong? Could it possible be a problem with the other apps?

I ended up using another package called react-native-file-viewer. I checked its code and wondered if I should just copy paste stuff from there to here but I don't really know much native coding ;D
I think this was the last commit for the functionality https://github.com/react-native-community/react-native-share/pull/215/commits/3c76ef2b8e64f8480bd52d57bffbd76439474fee
This is the implementation in react-native-file-viewer
https://github.com/vinzscam/react-native-file-viewer/blob/master/android/src/main/java/com/vinzscam/reactnativefileviewer/RNFileViewerModule.java
I skimmed it and they look similar

@marqroldan I had a similar problems, try to apply this change https://github.com/react-native-community/react-native-share/pull/770 inside your node_modules to check would it work for you as well or not.


Bump build(s) to Go 1.10.4

Bump build(s) to Go 1.10.4

Coverage remained the same at 92.437% when pulling 9f9300552d5f52cb5e47e3d2fba4fc175a6ac0b7 on cs/go-1.10.4 into 19b29f086337b9b3d74cb47bf52dc0c5b18b1179 on master.


Video playing support

Why doesn't videos or gifs play in Andstatus timeline?. Is there something I need to turn on?.
Apart from this. Amazing project! Love it so much. I will be contributing to maybe documentation and stuff soon. Personally I think it can be way better than twidere.

Hi @karnotcycle,
Thank you for the AndStatus project review,

Regarding animated GIFs support: it works in a timeline, but animated GIFs, as it appears, are mostly not supported by social networks,  please see: https://github.com/andstatus/andstatus/issues/330
Video preview is shown in AndStatus. You have to click on it to launch external video player: the one that you have to choose and install on  your device. Does this work for you?

I am extremely sorry, I had the notion carried on from an older version. It's been a while since I used the new one. I just went through everything last night. Most of the problems are fixed plus there are some new features too.
Sorry, I didn't mean to be mean. Pardon my ignorance.

@karnotcycle Thank you for feedback. Glad that you like the app. Maybe you could also share your experience in Google Play or the Net... ;-)

Hey, sorry but I didn't know where else to reach you. I want to ask what is the motivation behind naming the app andstatus? What's it mean?  Feel free to ignore. But I'd like to know if you don't mind it... :)

@karnotcycle  Thank you for asking :-)
Back in 2009 I joined development of "AndTweet" app.  The app supported only Twitter at that time hence the name, I guess.
In 2010 I became the main developer of the app.
In 2011 Twitter announced a list of its own words, forbidden to be used in names of other developers' apps... and the "tweet" word was among them.
This is why I had to rename the app. I made minimal change: "Status" is used in the new name as a synonym of "Tweet".
Since that time AndStatus was literally re-written several times, and now it supports several social networks, Twitter not being the most popular by AndStatus users anymore. So the new name fits it much better.

Thank you for the answer. I don't really use playstore, but I have got most of my friends over to andstatus by showing them how amazing it is. Thank you for such a great app. I love the concept of it.
I would like to discuss about the UI and help make it even better, if you could point me to its open issue it'd be great.
I am coming from tusky. I never thought I will like andstatus, seriously the recent installation was just to see where the project is going. But for some unfathomable reason I am in love with it. I am finding it hard to go back to tusky or twidere. :)

@karnotcycle I'm open to UI improvements, thank you for offering your help in identifying and making such improvements.
In order to be achievable, such improvements should be as small as it is practical.
Please note that the app has a lot of UI tweaks in its Settings. Play with them, and maybe something is already done, but hidden or not advertised properly...
Next please select improvement that you feel would be the most important for you, and create an issue here in GitHub (start with searching of an existing one...), so we could discuss it.
Waiting for your feedback!


Cannot read property 'nameState' of null

Description
After performing the NewRelic Apollo integration I've started noticing the following in our logs.  This doesn't mean a whole lot but did notice this reported in the koa integration back in June - wondering if maybe it is related here.  Apologies for the poor format:
2021-01-29T10:44:44.636Z [WARN] Cannot read property 'nameState' of null Context: {"error":{"message":"Cannot read property 'nameState' of null","extensions":{"code":"INTERNAL_SERVER_ERROR","exception":{"stacktrace":["TypeError: Cannot read property 'nameState' of null"," at setTransactionName (/usr/cocina/node_modules/@newrelic/apollo-server-plugin/lib/create-plugin.js:314:33)"," at Object.willSendResponse (/usr/cocina/node_modules/@newrelic/apollo-server-plugin/lib/create-plugin.js:224:11)"," at /usr/cocina/node_modules/apollo-server-core/dist/utils/dispatcher.js:21:31"," at Array.map (<anonymous>)"," at Dispatcher.callTargets (/usr/cocina/node_modules/apollo-server-core/dist/utils/dispatcher.js:18:24)"," at Dispatcher.<anonymous> (/usr/cocina/node_modules/apollo-server-core/dist/utils/dispatcher.js:27:43)"," at Generator.next (<anonymous>)"," at /usr/cocina/node_modules/apollo-server-core/dist/utils/dispatcher.js:8:71"," at new Promise (<anonymous>)"," at __awaiter (/usr/cocina/node_modules/apollo-server-core/dist/utils/dispatcher.js:4:12)"]}},"name":"TypeError"}}
Expected Behavior
I'd like to not have this error appear in the logs either by changing my integration somehow or updating the package if necessary.
Steps to Reproduce
Not clear what triggers this error.
Your Environment
Node v12.14.1
Apollo Server Express 2.17.0
Additional context
We're also instrumented with AWS Xray as well.

Interesting. Really surprised there would be state loss introduced mid-plugin execution.
Do you by chance have any other custom or 3rd-party plugins in use?
We can handle this symptom, though. We can grab the transaction off of the operation segment.
Thank you for letting us know and sorry for the inconvenience!

Thanks for looking at this, much appreciated!
We're also instrumenting with AWS Xray at the same time which is operating as custom Express middleware but not an Apollo plugin per-se.  I'm somewhat surprised the two coexist as well as they do and generally speaking I get good information, it is just that there are certain scenarios that throw this error above that I can't nail down.  If it can be masked at least, that would really be awesome.


Results are not sorted under View user accounts page.(Agent Panel)

Description of the bug
After entered all the details and by  clicking on search button the searched result is not sorted.  User Name, Account Name, Expiry Date has no effect during Search. These are not implemented. Only Company name is working
To Reproduce
Steps to reproduce the behavior:

Go to 'https://www.pulseptt.com/agent.html'
Login with valid credential:- 1. User name: john 2. Password: 12345678.
Navigate to User Management.
Click on View User Accounts.
Enter company name:- titan pvt. ltd.
Enter User name:- pttqazxsasqwe
Enter account name:- pttqaa
Enter License Expiry After:- 8.5.22
Click on Search Button.
See error.

Expected behavior
Searched result must sorted and display on the screen and Search by Company Name (List with ALL), Agent Name (List with ALL), Account Type (List with ALL), Status (List with ALL) should be implemented.
Screenshots
After clicking on search button, the output is shown in below screenshot:-

Desktop (please complete the following information):

OS: Windows 10
Browser :Version 100.0.4896.60 (Official Build) (64-bit)
Version [0.1.1]

Now Searched result is sorted and display on the screen but "Search by Company Name (List with ALL), Agent Name (List with ALL), Account Type (List with ALL), Status (List with ALL)  and Expiry Date should auto-fill with current dates" is not working.


Use training IR in torchchat export

Export is moving to training IR in the coming weeks, as a result export_for_training will be alias to export in the future. After export switches to training IR, i will replace this call with torch.export.export again.

Awaiting vision pin to realign, then we'll test if segfault is fixed on pytorch/pytorch

Absorbing into https://github.com/pytorch/torchchat/pull/1367


DifferenceInWeekendDays excludes weekend days

That's what it says in the readme :open_mouth:

Thanks! It's fixed now :tada:


[DO NOT MERGE] Study refactor

Not ready for merge yet. Details can be found here: #378

Some miscellaneous, I'm looking at it on my phone so haven't been able to try it out yet:
The structure of the computers is a little weird, they look configurable but aren't rebound.
The name computers doesn't suggest the functionality to me.
Should the renderers allow a root accessor to be specified in case you've stashed the result in a nested object? It seems a likely use-case for multi-valued indicators.

Wow - big text!

The structure of the computers is a little weird, they look configurable but aren't rebound.

Yeah, I haven't looked at the component API yet, I just wanted to give people a chance to see what it looks like from an end-user perspective when these components are split out.

The name computers doesn't suggest the functionality to me

Totally agree - I've been struggling with that one. Any ideas?

Should the renderers allow a root accessor to be specified in case you've stashed the result in a nested object? It seems a likely use-case for multi-valued indicators.

I'm not sure I follow you there! It's also worth noting that some indicators, those that output a single series, would not have dedicated renderers, you'd just plug the output straight into a series:
var ma = fc.indicators.computers.movingAverage();
ma(data);

var line = fc.series.line()
        .xScale(dateScale)
        .yScale(priceScale);

container.append('g')
        .datum(data)
        .call(line);

Opps thought that would give a numbered list... Apparently not!
Transformers, mutators, combinators... They still don't sound right though.
I'll have a go at creating an example tomorrow. But, in the last example I
don't understand how the line series knows the y value?
On Tue, 26 May 2015 20:36 Colin Eberhardt<EMAIL_ADDRESS>wrote:

Wow - big text!
The structure of the computers is a little weird, they look configurable
but aren't rebound.
Yeah, I haven't looked at the component API yet, I just wanted to give
people a chance to see what it looks like from an end-user perspective when
these components are split out.
The name computers doesn't suggest the functionality to me
Totally agree - I've been struggling with that one. Any ideas?
Should the renderers allow a root accessor to be specified in case you've
stashed the result in a nested object? It seems a likely use-case for
multi-valued indicators.
I'm not sure I follow you there! It's also worth noting that some
indicators, those that output a single series, would not have dedicated
renderers, you'd just plug the output straight into a series:
var ma = fc.indicators.computers.movingAverage();
ma(data);
var line = fc.series.line()
.xScale(dateScale)
.yScale(priceScale);
container.append('g')
.datum(data)
.call(line);
—
Reply to this email directly or view it on GitHub
https://github.com/ScottLogic/d3-financial-components/pull/382#issuecomment-105643561
.

Probably a mistake in the example ;-)
On Tue, May 26, 2015 at 9:04 pm, ScottLogic/d3-financial-components
<EMAIL_ADDRESS>wrote:
Opps thought that would give a numbered list... Apparently not!
Transformers, mutators, combinators... They still don't sound right though.
I'll have a go at creating an example tomorrow. But, in the last example I
don't understand how the line series knows the y value?
On Tue, 26 May 2015 20:36 Colin Eberhardt<EMAIL_ADDRESS>wrote:

Wow - big text!
The structure of the computers is a little weird, they look configurable
but aren't rebound.
Yeah, I haven't looked at the component API yet, I just wanted to give
people a chance to see what it looks like from an end-user perspective when
these components are split out.
The name computers doesn't suggest the functionality to me
Totally agree - I've been struggling with that one. Any ideas?
Should the renderers allow a root accessor to be specified in case you've
stashed the result in a nested object? It seems a likely use-case for
multi-valued indicators.
I'm not sure I follow you there! It's also worth noting that some
indicators, those that output a single series, would not have dedicated
renderers, you'd just plug the output straight into a series:
var ma = fc.indicators.computers.movingAverage();
ma(data);
var line = fc.series.line()
.xScale(dateScale)
.yScale(priceScale);
container.append('g')
.datum(data)
.call(line);
—
Reply to this email directly or view it on GitHub

https://github.com/ScottLogic/d3-financial-components/pull/382#issuecomment-105643561

.

—
Reply to this email directly or view it on GitHub
[https://github.com/ScottLogic/d3-financial-components/pull/382#issuecomment-105650096] .[https://github.com/notifications/beacon/ABDBft4Dtewld-zudt6uVq_R2ZuTsiWKks5oNMk1gaJpZM4EqZcH.gif]

I've rolled-out the pattern to all of the indicators, and re-bound properties where appropriate.

[ ] add series to a renderers namespace, computers to a algorithms namespace and algorithms to algorithms.calculators namescape.
[ ] By default, merge calculator output to a single property, rather than flattening
[ ] algorithms should return the merged output, for more flexibility
[ ] add design guidelines

@chrisprice - I think that covers everything - I'm happy with the result :-)


AppSync delta table is missing the base table index

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v19.6.0
Amplify CLI Version
12.1.1
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No
Describe the bug
In my GraphQL schema, I have this model with an indexed attribute zone. all AppSync queries/Mutations/subscriptions are generated by the cli.
when I run the query to get data by zone all the queries work as expected, except when I try to get the delta data ( lastSync on or after current - DeltaSyncTTL) I get this error The table does not have the specified index byZone.
after checking the delta table, I found that the CLI didn't create the index automatically. I had to create it manually.
type Collections @model  @auth(rules: [{ allow: private }, { allow: public }]) {
  id: ID!
  *****
  zone: ID! @index(name: "byZone", sortKeyFields: ["timestamp"])
  *****
  *****
  *****
  *****
  *****
  timestamp: AWSDateTime!
}

Expected behavior
amplify cli should create the index in the delta table
Reproduction steps

create graphql api using the cli, with a model like

type Collections @model  @auth(rules: [{ allow: private }, { allow: public }]) {
  id: ID!
  *****
  zone: ID! @index(name: "byZone", sortKeyFields: ["timestamp"])
  *****
  *****
  *****
  *****
  *****
  timestamp: AWSDateTime!
}

run this query

query SyncCollections(
    $filter: ModelCollectionsFilterInput,
    $lastSync: AWSTimestamp,
    $limit: Int)
    {
        syncCollections(filter: $filter, lastSync: $lastSync, limit: $limit) {    items {
                __typename
                _deleted
                _lastChangedAt
                _version
                createdAt
                id
                zone
                timestamp
                updatedAt
            }
            nextToken
            startedAt
        }
    }

VARIABLES
{
   "filter": {
     "and": [
       {
         "zone": {
           "eq": "zoneID"
         }
       }
     ]
   },
   "limit": 500,
   "lastSync":<PHONE_NUMBER>600
}

If the lastSync is null or is before current - DeltaSyncTTL the the query works because it runs the query on the Base table
If the lastSync on or after current - DeltaSyncTTL the query returns an error The table does not have the specified index: byZone because it runs the query on the Delta table and the delta table is missing the index

Project Identifier
No response
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Amplify do not create index on the custom fields on the delta table. LastSync argument is meant to specify the time the data was last synced on the device, so it is expected to be less than (or before) the current timestamp. Is there a reason why lastSync is greater (or after) the current timestamp in your use case?

lastSync is after current - DeltaSyncTTL. it could never be greater then current.
if Amplify does not create the index on the delta table, how can we do that? manually? this index is essential, as without it, the synchronization query won't function properly.
please checkout the original bug
https://github.com/aws-amplify/amplify-flutter/issues/3658#issuecomment-1710462091

@sundersc
this is not a datastore bug, it's an Amplify cli issue. as the cli didn't create the index in the delta table. creating the index manually solved the issue.

This is also happening with the sync expressions queries against the delta table.
amplify cli version 12.2.3 and  higher..  It works on amplify cli version 12.1.1 . I'm not sure when this started to happen, but somewhere between 12.1.1. and 12.2.3.
we have sync expressions against all our models ( user_id) so we sync against that instead of full base table scans.
we have around 10ish models at the moment.

Can you share your feature flags? amplify/cli.json.

{
  "features": {
    "graphqltransformer": {
      "addmissingownerfields": true,
      "improvepluralization": false,
      "validatetypenamereservedwords": true,
      "useexperimentalpipelinedtransformer": true,
      "enableiterativegsiupdates": true,
      "secondarykeyasgsi": true,
      "skipoverridemutationinputtypes": true,
      "transformerversion": 2,
      "suppressschemamigrationprompt": true,
      "securityenhancementnotification": false,
      "showfieldauthnotification": false,
      "usesubusernamefordefaultidentityclaim": true,
      "usefieldnameforprimarykeyconnectionfield": false,
      "enableautoindexquerynames": true,
      "respectprimarykeyattributesonconnectionfield": true,
      "shoulddeepmergedirectiveconfigdefaults": false,
      "populateownerfieldforstaticgroupauth": true
    },
    "frontend-ios": {
      "enablexcodeintegration": true
    },
    "auth": {
      "enablecaseinsensitivity": true,
      "useinclusiveterminology": true,
      "breakcirculardependency": true,
      "forcealiasattributes": false,
      "useenabledmfas": true
    },
    "codegen": {
      "useappsyncmodelgenplugin": true,
      "usedocsgeneratorplugin": true,
      "usetypesgeneratorplugin": true,
      "cleangeneratedmodelsdirectory": true,
      "retaincasestyle": true,
      "addtimestampfields": true,
      "handlelistnullabilitytransparently": true,
      "emitauthprovider": true,
      "generateindexrules": true,
      "enabledartnullsafety": true
    },
    "appsync": {
      "generategraphqlpermissions": true
    },
    "latestregionsupport": {
      "pinpoint": 1,
      "translate": 1,
      "transcribe": 1,
      "rekognition": 1,
      "textract": 1,
      "comprehend": 1
    },
    "project": {
      "overrides": true
    }
  },
  "debug": {
    "shareProjectConfig": true
  }
}

{
  "features": {
    "graphqltransformer": {
      "addmissingownerfields": true,
      "improvepluralization": false,
      "validatetypenamereservedwords": true,
      "useexperimentalpipelinedtransformer": true,
      "enableiterativegsiupdates": true,
      "secondarykeyasgsi": true,
      "skipoverridemutationinputtypes": true,
      "transformerversion": 2,
      "suppressschemamigrationprompt": true,
      "securityenhancementnotification": false,
      "showfieldauthnotification": false,
      "usesubusernamefordefaultidentityclaim": true,
      "usefieldnameforprimarykeyconnectionfield": false,
      "enableautoindexquerynames": true,
      "respectprimarykeyattributesonconnectionfield": true,
      "shoulddeepmergedirectiveconfigdefaults": false,
      "populateownerfieldforstaticgroupauth": true
    },
    "frontend-ios": {
      "enablexcodeintegration": true
    },
    "auth": {
      "enablecaseinsensitivity": true,
      "useinclusiveterminology": true,
      "breakcirculardependency": true,
      "forcealiasattributes": false,
      "useenabledmfas": true
    },
    "codegen": {
      "useappsyncmodelgenplugin": true,
      "usedocsgeneratorplugin": true,
      "usetypesgeneratorplugin": true,
      "cleangeneratedmodelsdirectory": true,
      "retaincasestyle": true,
      "addtimestampfields": true,
      "handlelistnullabilitytransparently": true,
      "emitauthprovider": true,
      "generateindexrules": true,
      "enabledartnullsafety": true,
      "generatemodelsforlazyloadandcustomselectionset": false
    },
    "appsync": {
      "generategraphqlpermissions": true
    },
    "latestregionsupport": {
      "pinpoint": 1,
      "translate": 1,
      "transcribe": 1,
      "rekognition": 1,
      "textract": 1,
      "comprehend": 1
    },
    "project": {
      "overrides": true
    }
  },
  "debug": {
    "shareProjectConfig": true
  }
}

Hi, any updates? It's critical issue on my side.

I tried to repro the issue in the AppSync console but cannot succeed. Do you have the same error returned if running the query in AppSync? Or is it only thrown in the datastore app?

Only on the android side, on ios it seems to work well.

Hello @AaronZyLee
To reproduce, just checkout amplify example https://docs.amplify.aws/lib/datastore/sync/q/platform/android/#advanced-use-case---query-instead-of-scan
And set lastSync to now - 30 minutes to query delta table
I open this bug since one month on it https://github.com/aws-amplify/amplify-android/issues/2564 with all details to reproduce
Thanks

I also found that on Android in the AmplifyDatabase.db if 2 models have the same index name then only one index will be added and second will be ignored.

@ridakaddir Could you share the code block that computes the value of current - DeltaSyncTTL?

@sundersc
Android datastore prepares lastSync and pass it to GraphQLRequest

after that
GraphQLResponse data is null - thats why we are getting IrRecoverableException

Hi, any updates?

@AaronZyLee @sundersc Do you have any updates?

@wojciechzahradnikdeviniti Could you send a report by running the following command and provide your project identifier? It would be helpful to examine the backend config directly to repro the issue.
amplify diagnose --send-report

We have identified the root cause for this issue. While we fix this, a workaround is to override the Sync Query preAuth resolvers to properly specify the deltaSyncTableTtl. For example, if you have a model named Users,

Copy the contents of Query.SyncUsers.preAuth.1.req.vtl from amplify/backend/api/<api_name>/build/resolvers to the resolver overrides directory which is amplify/backend/api/<api_name>/resolvers.
Update the resolver override to include the following statement at the beginning: #set( $ctx.stash.deltaSyncTableTtl = <my-delta-sync-ttl> ). The value of <my-delta-sync-ttl> defaults to 30. This value is in minutes.
Push your changes using amplify push.

hi, @phani-srikar
Thanks! Will amplify-cli lib be also updated?

hi @phani-srikar
I didn't do the workaround but I updated amplify-cli to 12.5.2 with the fix you provided.
I pulled all using amplify pull and I still have that issue with IrRecoverableException.
I can't see any changes in preAuth file.

Could you please help me?

You will probably need to amplify push to update the AppSync API. If you see No changes detected when pushing you can add a newline to your schema.graphql and then push again to force an update.

Ok after next pull it seems to working fine, thanks :)


fix: able to use openai-compatible llms/embeddings in ragas_once

What type of PR is this?

What this PR does / why we need it
Which issue(s) this PR fixes

Fixes #
Special notes for your reviewer

@0xff-dev We have changed the arguments:

--metrics  has been changed to a string which seperates different metrics with comma
--model -> --llm
--embedding : this is a new argument

should add license header to ragas like data-processing


Readme formatting

Formatting of readme looks incorrect. For example, see point 2 there -- the users who would copy-paste this onto their command line will be surprised.
This came from #945

Thanks for raising the issue. The typo has been fixed.

Very similar issue with point 2 in different iteration (install instructions for ROCm v3.0), so using same thread.
https://rocm.github.io/ROCmInstall.html
The resulting rocm.list file looks more like a manual for gpg then anything meaningful, and it produces the error on next step (sudo apt update):
"""
waltp@waltp-HP-Pavilion-Laptop-15z-cw100:~$ sudo apt update
E: Type 'gpg' is not known on line 1 in source list /etc/apt/sources.list.d/rocm.list
E: The list of sources could not be read.
"""
More detailed explanation (including rocm.list file) in text:
ROCm_install_difficulty.txt

Oh, I see.  The typo was fixed for the GitHub Readme text but not the website-style presentation at https://rocm.github.io/ROCmInstall.html
I deleted rocm.list and then followed the corrected Readme text, and now it's fine. :)

Duplicate of #957

Actually, I think the https://rocm.github.io/ROCmInstall.html page still needs some finessing. In the "Ubuntu" section, the "|" pipe command is formatted into a table for whatever reason, so it isn't evident that the two commands are to be given on the same line. Also, a copy paste of the command provided (echo ‘deb [arch=amd64] http://repo.radeon.com/rocm/apt/debian/ xenial main’) (ie. with single quotes) put those quotes into rocm.list as well, which obviously doesn't work. Replacing them with double quotes worked. My system details are:
NAME="Ubuntu"
VERSION="18.04.3 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04.3 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

More nits:
https://rocm.github.io/ROCmInstall.html:

The below needs to be separated out into two commands:
sudo apt update sudo apt install rocm-dkms
The commands in the By default, add any future users to the video group section are missing the "|" operator again. This also needs to be corrected in the wiki doc.

In the intervening months, our installation directions have been centralized in the readthedocs site. I believe the issues raised here are all fixed in the current documentation. Thanks!


fixed modules references

Using relative module references. Allows importing of the project in a subfolder.

I made a pip package of the repo in https://github.com/Spijkervet/SimCLR/commit/b844b0121271d37d7c372bdc16ad46bcccd1bddd :)


Setting prompt to None causes empty output for audio URLs

Setting prompt=None here when there is no vocab causes empty transcript output with some presigned AWS URLs.
Not sure why. Downloading the audio URL, as well as the converted audio URL, and then uploading either of these files to the audio file endpoint causes no issues. Confirmed on two different audio URLs.
Setting vocab=[" "] fixes the issue when calling the audio URL endpoint, as does setting prompt="" on line 405 here.
Setting prompt="" as the default fallback.

Reopening as the change has an effect on other low-resource languages such as cs, where setting prompt="" results in an empty transcript list. As an aside, have noticed that turning vad_filter off has a positive effect in these cases. As the wide-ranging effects appear to be unpredictable, reverting this.

Reverting in https://github.com/Wordcab/wordcab-transcribe/pull/171


Add EIPs to CFI, based on ACD150

Decided on https://github.com/ethereum/pm/issues/662

Codecov Report
Base: 68.75% // Head: 72.04% // Increases project coverage by +3.29% :tada:

Coverage data is based on head (b6ca5c1) compared to base (333baf1).
Patch coverage: 97.97% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #656      +/-   ##
==========================================
+ Coverage   68.75%   72.04%   +3.29%
==========================================
  Files         481      518      +37
  Lines       26836    29051    +2215
==========================================
+ Hits        18451    20930    +2479
+ Misses       8385     8121     -264

Flag
Coverage Δ

unittests
72.04% <97.97%> (+3.29%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/ethereum/arrow_glacier/spec.py
0.00% <0.00%> (ø)

src/ethereum/gray_glacier/spec.py
0.00% <0.00%> (ø)

src/ethereum/muir_glacier/spec.py
0.00% <0.00%> (ø)

src/ethereum/paris/utils/hexadecimal.py
87.50% <87.50%> (ø)

src/ethereum/paris/spec.py
92.61% <92.61%> (ø)

src/ethereum/paris/utils/message.py
96.00% <96.00%> (ø)

src/ethereum/paris/vm/gas.py
96.34% <96.34%> (ø)

src/ethereum/paris/vm/interpreter.py
96.63% <96.63%> (ø)

src/ethereum/paris/state.py
97.34% <97.34%> (ø)

src/ethereum/paris/trie.py
97.79% <97.79%> (ø)

... and 43 more

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

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


Feat/new stuff

:tada: This PR is included in version 1.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
v1.2.0
GitHub release

Your semantic-release bot :package::rocket:


Update Deploying an application using odo Section and Fixing Broken Links

What is the purpose of this change? What does it change?
This pull request updates odo's REAMDME to fix steps in the Deploying an application using odo section of the documentation. These updates include the following:

Including Minishift documentation for Step 1 in case users don't have Minishift installed.
Removing step 3 odo app create, which is no longer used in v1.0.0-beta1.
Adding a step to delete the component created as part of the tutorial.

This change also addresses broken links in the README as referenced in #1704.
Was the change discussed in an issue?
fixes #1699
fixes #1704
How to test changes?
Go through the new steps listed and verify you are able to create a component, expose its url, access the nodejs application, and delete the component.

/ok-to-test

/lgtm
e2e-scenario is failing on login issues again :-(
/retest
error: You must be logged in to the server (Unauthorized)
error: You must be logged in to the server (the server has asked for the client to provide credentials)

/retest
[odo]  ✗  Please log in to the cluster


Moves global scope out of shared/util.

Follow up on #8761
Using this in modules is tricky. (It will be replaced by babel to undefined). Moving globalScope out of shared/utils into separate file -- this will allow to create single object for compatibility and util usages.

Strangely enough, when testing this locally using gulp server, I see the following printed in the console
Warning: Setting up fake worker.   util.js:282:11
ReferenceError: require is not defined   compatibility.js:24:5

Not all compatibility.js references where removed  in https://github.com/mozilla/pdf.js/pull/8778/files#diff-a72ef320c3225c98b2163b89a3e8ff62L48

/botio test


Issue#419 Implementation for InstanceAdmin::GetInstance with unit tests for no errors

PR Created for following:

Implementing the function itself, Implementing the sample snippet and Implementing the unit tests for "no errors"
Following will be part of Next PR:
Implementing the integration test, and Implementing the unit tests for "unrecoverable error", "retry with recoverable errors".

I have implemented GetInstance API with another way also as below, but in that I was facing problem, that how to get Instance based on instance ID.
btproto::GetInstanceRequest request;
request.set_name(instance_id);
//Call RPC call to get response
auto response = ClientUtils::MakeCall(
*client_, *rpc_policy, *backoff_policy, metadata_update_policy_,
&InstanceAdminClient::GetInstance, request,
"InstanceAdmin::GetInstance", status, true);
if (status.ok()) {
result = response;
}

Fixing build issue, due to rebase merge conflict issue


Missing content-type application/json;charset=utf-8 for status code 404 on /wallets

Context
I'm building a library for Elixir that talks to the cardano-wallet and I think I found a bug.
The content-type header seems to be missing when calling http://localhost:8090/v2/wallets/<wallet_id> with an non existing id which returns with status code 404.
This is the headers I get back:
{"date", "Thu, 01 Apr 2021 21:12:48 GMT"},
{"server", "Warp/3.3.5"},
{"content-length", "122"}
If I call the same endpoint but with a badly formatted id which returns a status code 400 I get these headers:
{"date", "Thu, 01 Apr 2021 21:12:48 GMT"},
{"server", "Warp/3.3.5"},
{"content-length", "92"},
{"content-type", "application/json;charset=utf-8"}
The problem with this is that the content type is used by many http clients to automatically decode the json response to a struct so the implementation will be cluttered with workarounds.
Essential Information

Version:      master
Platform:     Linux
Installation: Built from source
Network: testnet

Good catch thanks. It may actually be the case for other endpoints too :thinking: , we should test this generically using the API data types to make sure not to miss any.

I also remember seeing some non-JSON plain text error responses from the API - which would be annoying for API consumers which unconditionally expect JSON. But I forget which error it was...
Anyway, WIP PR #2597.


Document various flavours of testing

Add a section to our documentation:
Testing

Flavours (01, move other pages further)

Unit testing (01)
Integration testing (02)
E2E testing (03)

In each page describe the theory of each approach and then showcase on an example how to do it in Protostar.
The flavours page should simply be an intro of what's going on here and put this there:
    ```mdx-code-block
    import DocCardList from '@theme/DocCardList';
    import {useCurrentSidebarCategory} from '@docusaurus/theme-common';

    <DocCardList items={useCurrentSidebarCategory().items}/>
    ```

Closing in favour of #1802


chore(devdeps<EMAIL_ADDRESS>

Security-related maintenance update.

@SocketSecurity ignore<EMAIL_ADDRESS>@SocketSecurity ignore<EMAIL_ADDRESS>@SocketSecurity ignore npm/<EMAIL_ADDRESS>These are all intended to have network access

@SocketSecurity ignore<EMAIL_ADDRESS>@SocketSecurity ignore<EMAIL_ADDRESS>New authors look OK; one appears to be on the npm CLI team, the other is a bot operated by that team.


asn1_item_embed_new(): don't free an embedded item

An embedded item wasn't allocated separately on the heap, so don't
free it as if it was.

Done

(does this require a reconfirm, formally speaking?)

Well the approval was intended to apply either way. But, for the avoidance of doubt: reconfirm.

Merged.
master: 590bbdfdf43b97abf8817f506f8ab46687d1eadd
1.1.0: ca80ba81e243c30873cd11870422a3873a64be73


npm ERR! request to https://registry.cnpmjs.org/tailwindcss failed

npm ERR! code ERR_TLS_CERT_ALTNAME_INVALID
npm ERR! errno ERR_TLS_CERT_ALTNAME_INVALID
npm ERR! request to https://registry.cnpmjs.org/tailwindcss failed, reason: Hostname/IP does not match certificate's altnames: Host: registry.cnpmjs.org. i
s not in the cert's altnames: DNS:r.cnpmjs.org
npm ERR!      C:\Users\Eddie\AppData\Roaming\npm-cache\_logs\2020-06-11T13_06_30_594Z-debug.log

If you'd like to open an issue that actually describes your problem we can definitely help, but just copy and pasting an error isn't enough information for us to actually help.


(Mino) Use native git with commit-id plugin

Since JGit doesn't support git worktrees, the commit-id plugin for Maven also doesn't support them. This is a small tweak to tell it to use native git instead. There wasn't any significant performance degradation from what I saw locally.
See issue 215 on that plugin's GH repo for more details.

Is there any concern about having different git versions in our build environments? Maybe that is something we should check and normalize. Otherwise it looks good to me.

I don't think so - this really just affects local development when you are trying to build within a worktree. Since we're only really pulling basic info from git, a native instance of any version should suffice.


ST: Fix for consuming messages

Type of change

Bugfix

Description

Added json-path-matchers as a dependency for using JSON matchers instead of working with objects
Increased timeout for consuming messages. 20 seconds is not enough for consuming messages
Moved logic with parsing JSON to matchers (it will be helpful to have good assertion output in a case with failure)
Fixed JSON from kafka-verifiable-consumer.sh output

The previous state of the object:
{...} {...}
after fix we have(Added brackets and commas):
[{...}, {...}]
Checklist

[ ] Write tests
[ ] Make sure all tests pass
[ ] Update documentation

@tombentley I completely agree,

Increasing timeout to 30 secs is not a good fix, but I don't know how to execute kafka-verifiable-consumer.sh more correct. Maybe @asorokhtey can suggest any solution to remove this timeout at all?
TBH I executed this tests many times only on my local PC and this test was stable, but you are right I need to execute this test also on CI many times before merging this PR.


error logs and kong exited

Summary
error logs
2017/02/01 22:00:04 [notice] 9419#0: exit
2017/02/01 22:00:04 [notice] 9417#0: signal 17 (SIGCHLD) received
2017/02/01 22:00:04 [notice] 9417#0: worker process 9419 exited with code 0
2017/02/01 22:00:04 [notice] 9417#0: signal 29 (SIGIO) received
2017/02/01 22:00:04 [notice] 9422#0: exit
2017/02/01 22:00:04 [notice] 9417#0: signal 17 (SIGCHLD) received
2017/02/01 22:00:04 [notice] 9417#0: worker process 9422 exited with code 0
2017/02/01 22:00:04 [notice] 9417#0: signal 29 (SIGIO) received
2017/02/01 22:00:04 [notice] 9421#0: exit
2017/02/01 22:00:04 [notice] 9417#0: signal 17 (SIGCHLD) received
2017/02/01 22:00:04 [notice] 9417#0: worker process 9421 exited with code 0
2017/02/01 22:00:04 [notice] 9417#0: exit

Steps To Reproduce
Additional Details & Logs

Kong version (0.9.5)
Operating System centos 6

@koolay please provide some more info. What commands did you execute, with what result?

Closing due to lack of updates.


Adding Game of Thrones Board Game

Submitting this PR means you have:

[x] searched the readme file and searched the open pull requests (no duplicates please)
[x] and will follow the desired format

Do you have further questions? Check the contribution guidelines or open an issue.
When you're ready, check off the above acknowledgements and continue with your Pull Request... and thanks!

Thanks for contributing.

How can i get a Hacktoberfest tag?

Well, as far as this PR goes (internal to this repo), we can tag the issue so labelled with #10. As for the tracking for Hacktoberfest, provided you have signed up at the official Hacktoberfest page, it will track this public Pull Request and count it towards your total. You can log in with your GitHub account there and view your progress as well.
https://hacktoberfest.digitalocean.com/


Attempt to correct linked file regEx in branchUrl

Hi, I /think/ this is correct:
(.+) is changed to (\w+) and the slash is moved inside the "blob" bit. I tested and it correctly addresses the issue in #189. It had been grabbing all of blob/branch-name/path/to/file.rb and replacing, instead of just blob/branch-name.
...but am not sure of branch name conventions. Will (\w+) correctly grab any branch name?

I fear that’s not enough, branch names can include - and even / ... maybe we can get the branch name entirely from a different property? 🤔

a wait, no we are fine, because the blob_url contains a commit sha in the response, not a branch name, so yeah, we should be fine.
Could you try to add a test covering the change, so we don’t break it in future?

Whoa, but does GitHub generate URLs that include / in branch names? Or does it convert them? Is the limitation GitHub or Git?
We could also compare it to the filename, so basically snip out the filename then subtract the relative path from the blob_url to get the branchUrl?

commit sha - yes, that's what I thought. cool!

Let me look at the tests, hang on a sec!

Sorry, i'm making a real mess of my PRs.
I'm trying to open a PR with just the test to let it fail first.

OK, so for what it's worth, in my PR mess, i now have all three states :-/ -- well, don't we all love empiricism:

this PR has the fix but not the test
https://github.com/hoodiehq/first-timers-bot/pull/192 demonstrates the test (failing) without the fix
https://github.com/hoodiehq/first-timers-bot/pull/191 demonstrates both the test and the fix (passing)

We can move to #191 or I can merge it into here too?

Ideally, to make my life easiest :D you do the following

close all open PRs
Create a new PR with one commit that only has the test, it will show the error status on that commit.
Then push the fix, which will change it to success

see https://twitter.com/gr2m/status/963879699385495552 :)

OK, I'm closing this, and moving to https://github.com/hoodiehq/first-timers-bot/pull/192 which demonstrates the test failing. I'll add the fix and then will see the green ✔  get added there. Thanks @gr2m !


Test Failure: test runs_tasks_from_multiple_project

I am working on packaging a nix flake for moon since I wanted to use it on my nixos machine.
I can run 'cargo build' and get a working target/debug/moon but when I run cargo check I get a test failure:
     Running tests/check_test.rs (target/debug/deps/check_test-a1f4c3bab8ea4b14)

running 7 tests
test runs_on_all_projects_from_root_directory ... ok
test runs_tasks_in_project ... ok
test runs_for_all_projects_even_when_not_in_root_dir ... ok
test forces_cache_to_write_only ... ok
test runs_tasks_in_project_using_cwd ... ok
test creates_run_report ... ok
test runs_tasks_from_multiple_project ... FAILED

failures:

---- runs_tasks_from_multiple_project stdout ----
thread 'runs_tasks_from_multiple_project' panicked at 'assertion failed: predicate::str::contains(\"base:standard\").eval(&output)', crates/cli/tests/check_test.rs:72:5

Is this just on my machine?
Thanks!

My nix flake is over at https://github.com/darkone23/moon/blob/nix-flake/flake.nix
In a checkout of that branch:
"nix develop" gets me all the build dependencies and I can run "cargo make test" but I still get failures like:
Summary [   4.133s] 143/1172 tests run: 135 passed, 8 failed, 0 skipped
        FAIL [   2.481s] moon_cli::check_test runs_tasks_from_multiple_project
        FAIL [   1.512s] moon_cli::docker_test prune_node::focuses_for_npm
        FAIL [   1.379s] moon_cli::docker_test prune_node::focuses_for_pnpm
        FAIL [   1.238s] moon_cli::docker_test prune_node::focuses_for_yarn
        FAIL [   1.199s] moon_cli::docker_test prune_node::focuses_for_yarn1
        FAIL [   1.521s] moon_cli::docker_test setup_node::installs_npm
        FAIL [   0.536s] moon_cli::docker_test setup_node::installs_pnpm
        FAIL [   0.305s] moon_cli::docker_test setup_node::installs_yarn

I'm wondering if I should skip the tests in my case or if there is something wrong with my build or what...
Any guidance is appreciated - cheers!

@darkone23 To debug these, you can add a assert.debug() after this call: https://github.com/moonrepo/moon/blob/0b0163586ea6bb57a10de2a7c02d59b53dde0791/crates/cli/tests/check_test.rs#L66
It will dump a bunch of information.

Thank you. That helps and I can see that this is an issue on my end.
$ /home/xxxxxx/.proto/tools/node/18.0.0/bin/node --version
bash: /home/xxxxxx/.proto/tools/node/18.0.0/bin/node: cannot execute: required file not found

This is probably because that install of node is expecting system libraries that nixos is not providing.
I will close this issue and do more digging on my journey to get moon working on nixos.
Thanks again for your help.


Int2IntHashmap incorrect size() after rehash

   @Test
    public void correctSizeAfterRehash() throws Exception
    {

        Long2LongHashMap map = new Long2LongHashMap(16, 0.6D, -1);

        IntStream.range(1, 17).forEach(i -> map.put(i, i));

        assertEquals(16, map.size());
        ArrayList<Long> keys = new ArrayList<>();

        map.forEach((k, v) -> keys.add(k));

        keys.forEach(map::remove);

        assertTrue(map.isEmpty());
    }

P.S There is still typo in the IAE in the constructor.

if (loadFactor >= 1.0)
        {
            throw new IllegalArgumentException("Load factor must be <= 1.0");
        }

Since you are asserting for >= , the message should state < only

Hmm, I've just added and committed this testcase for the Int2IntHashMap and it passes fine. Perhaps I've misread something, but I'm closing this issue for now. If there's a testcase that fails feel free to re-open the issue.

@RichardWarburton  I suggest you check your code generator then.

I've just run this exact same test against agrona head and it passes fine. So I conclude that its already been fixed in the current snapshot. There should be a new release out soon.

Do you have a snapshots repo somewhere ?

We don't publish snapshots anywhere at the moment I don't think.

I've build the snapshot and it's working locally. :+1:


Add SAML IDP logout url (+DefaultRelayState)

Related to https://github.com/descope/etc/issues/7014

force authentication :)
plz make sure all of them are support in the management service as well.

Yea I added the force-auth and also the defaultRelayState params (added also in mgmt service).


#18 add eslint and prettier

code refactoring with prettier and lint(base with Next.js) settings

task: https://github.com/gitkoshelew/CC.ui/issues/27


Replace babili --> minify

 Replace babili —> minify

Coverage remained the same at 100.0% when pulling cd4c18a0004074375b3dbb6d9de890732f91ebcd on feature/babili-to-minify into d000183f1e5ca109d195750a94c97457d3fd12b9 on master.


Make the launch background drawable compatible with older Android API levels

The https://github.com/flutter/flutter/pull/65182 change introduced a
drawable with a reference to a theme attribute.  Loading this requires
a Resources.getDrawable API introduced in API level 21.

Thanks Jason


Helper should allow prefixing

Note: for support questions, please use stackoverflow. This repository's issues are reserved for feature requests and bug reports.

**I'm submitting a ... **

[x] feature request
[ ] bug report
[ ] support request => Please do not submit support request here, see note at the top of this template.

Do you want to request a feature or report a bug?
Most of the grid classes can be customized with the exception of the helper clases
https://github.com/lucasgruwez/waffle-grid/wiki/Using-the-grid#helper-classes

What is the current behavior?
Currently you cannot prefix these classes which creates conflicts with other packages.

What is the expected behavior?
Allow to have a scss variable you can set to prefix helper classes/

Ah yes, I can see how that would conflict with other libraries...
I like the idea of adding an scss variable to prefix the helper classes and will implement it ASAP.

Done! v1.3.6, use $helper-prefix to set the prefix for the helper classes, for example, if its set to "waffle-", the classes would be .waffle-center, .waffle-m-hide, etc...

@lucasgruwez Awesome! thank you!


Private extractor support

Adds optional support for an owner in the extractor heartbeat.
To test this, run an extractor with the following ENV VARs:
export<EMAIL_ADDRESS>export EXTRACTOR_KEY=secret_test

When the extractor starts, the email will be assigned as owner. That user should be able to see the extractor in lists as normal, but other users cannot. There are endpoints to grant other permissions:
POST/DELETE<EMAIL_ADDRESS>POST/DELETE http://localhost:8000/api/v2/listeners/123/groups/456
POST/DELETE http://localhost:8000/api/v2/listeners/123/datasets/456

...where 123 is extractor ID, and 456 is group ID. Only owner has permission to do this currently.
BASIC RULES.
If you are looking at a file or dataset Analysis tab, you will see extractor listed...

if extractor's access field is null
if you are access.owner, listed in access.users, or in a group listed in access.groups
if the dataset or the file's dataset are listed in access.datasets

If you are looking at Extraction History, same rules but dataset rule is ignored. NOTE: if the user has submitted a job to an extractor, they should be able to see it here... need to work out permissions on that.
Changed the logic slightly for the process filter so if the extractor has process set to {} for file or dataset (i.e. no rule specified) it will be included in those lists.

I'm running this but getting an error in the heartbeat:
    raise exc
  File "/Users/helium/ncsa/clowder2/backend/heartbeat_listener.py", line 34, in callback
    owner = msg["owner"]
KeyError: 'owner'

I checked in the extractor I am running and the environment variables are set.
Do I need a specific version of pyclowder? Here's my environment variables as I've set th

em.

Does the pyclowder branch need to be merged before this one?

Sorry can you resolve conflicts, thanks!

I think the issue above is also reflected in the failing pytest:

FAILED app/tests/test_feeds.py::test_feeds - assert 422 == 200

where 422 = <Response [422 Unprocessable Entity]>.status_code

I think the issue above is also reflected in the failing pytest:

FAILED app/tests/test_feeds.py::test_feeds - assert 422 == 200

where 422 = <Response [422 Unprocessable Entity]>.status_code

This should be resolved. It was due to the ListenerAuthorization dependency injection I added using listener_id in most places, but in this particular call we are POSTing a small object with listener_id and automatic so the validation was failing. I separated the call to ListenerAuthorization outside of dependency on this call only, for lack of a better idea

Something is not quite right here. On Extractor page, users should be able to see all the extractors(probably not the private extractor) irrespective of admin mode.
Here on this branch, I can see them in admin mode. When I click on 'Drop admin mode', the page doesn't refresh on it's own. After explicitly refreshing the page, it shows empty list.

@max-zilla
I am seeing this error
INFO:     <IP_ADDRESS>:54745 - "GET /api/v2/listeners?skip=0&limit=5&heartbeat_interval=0&category=&label=&alive_only=false&dataset_id=true&all=false&enable_admin=false HTTP/1.1" 500 Internal Server Error ERROR:    Exception in ASGI application Traceback (most recent call last):   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/uvicorn/protocols/http/h11_impl.py", line 429, in run_asgi     result = await app(  # type: ignore[func-returns-value]   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/uvicorn/middleware/proxy_headers.py", line 78, in __call__     return await self.app(scope, receive, send)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/fastapi/applications.py", line 276, in __call__     await super().__call__(scope, receive, send)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/starlette/applications.py", line 122, in __call__     await self.middleware_stack(scope, receive, send)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/starlette/middleware/errors.py", line 184, in __call__     raise exc   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/starlette/middleware/errors.py", line 162, in __call__     await self.app(scope, receive, _send)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/starlette/middleware/cors.py", line 92, in __call__     await self.simple_response(scope, receive, send, request_headers=headers)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/starlette/middleware/cors.py", line 147, in simple_response     await self.app(scope, receive, send)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/starlette/middleware/exceptions.py", line 79, in __call__     raise exc   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/starlette/middleware/exceptions.py", line 68, in __call__     await self.app(scope, receive, sender)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in __call__     raise e   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in __call__     await self.app(scope, receive, send)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/starlette/routing.py", line 718, in __call__     await route.handle(scope, receive, send)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/starlette/routing.py", line 276, in handle     await self.app(scope, receive, send)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/starlette/routing.py", line 66, in app     response = await func(request)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/fastapi/routing.py", line 237, in app     raw_response = await run_endpoint_function(   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/fastapi/routing.py", line 163, in run_endpoint_function     return await dependant.call(**values)   File "/Users/helium/ncsa/clowder2/backend/app/routers/listeners.py", line 437, in get_listeners     EventListenerDB.access.datasets == PydanticObjectId(dataset_id),   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/bson/objectid.py", line 103, in __init__     self.__validate(oid)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/bson/objectid.py", line 201, in __validate     _raise_invalid_id(oid)   File "/Users/helium/.virtualenvs/backend-qaHHe5b_/lib/python3.9/site-packages/bson/objectid.py", line 35, in _raise_invalid_id     raise InvalidId( bson.errors.InvalidId: 'true' is not a valid ObjectId, it must be a 12-byte input or a 24-character hex string

@max-zilla I see that error when I am on the page and am not in superAdmin mode. And no extractors are visible at all until I reload the page with superAdmin.
I was thinking that they should show up, and we just need superAdmin mode to enable/disable?

I think I fixed the error that was causing no extractors to be visible. The component was sending in 'true' as a dataset_id as a default value. It should be 'null' and then the next one for 'all' should be 'true.' Committed this.

Something is not quite right here. On Extractor page, users should be able to see all the extractors(probably not the private extractor) irrespective of admin mode.
Here on this branch, I can see them in admin mode.

When I click on 'Drop admin mode', the page doesn't refresh on it's own.
After explicitly refreshing the page, it shows empty list.

  3. When I am a diff regular user (other than the owner of private extractor), I can see no extracator.
I think all these might be related to the ListnerAuth class.

This might be fixed by the commit I made.
I would like someone else to check that it is working as intended.
See comment at end.

With the commit I made, I think things work as intended for the admin user, but I don't think it is working for other users.
I registered the wordcount extractor NOT as a private one, and even if it's running and enabled, it still only shows up for my admin user, and not for any others.

I found what was going wrong here.
When extractors are picked up by the heartbeat, the owner is "" but the script checks if they are None. I added that and now extractors that are not private are showing up like they should.
I did create an issue and a branch, but since this is such a tiny fix I just put it here.
To test, kill the heartbeat_listener docker container, and run the script in pycharm. Access will be None for public extractors registered.

thanks for the fix, @tcnichol !


 - is there a way not to have a default selected?

- is there a way not to have a default selected for <Input type="select" multiple>?
Versions:

bootstrap: 4.0.0
reactstrap: 5.0.0-beta

The problem

In the sample code in github, the demo shows that no option is selected as default
I copied the code and the first option is selected. Any way to disable the selection? Something like this.

here is the code that I tried:
 <Input type="select" name="selectMulti" id="exampleSelectMulti" multiple>
            <option>1</option>
            <option>2</option>
            <option>3</option>
            <option>4</option>
            <option>5</option>
 </Input>

That is being done by the browser per the HTML spec. A workaround is explicitly setting the defaultValue (or value if you plan on managing the input yourself) to an option which doesn't exist (like an empty string).
Demo: https://stackblitz.com/edit/reactstrap-v5beta-vsibj3?file=Example.js

@TheSharpieOne thanks for this! Can we add this defaultValue example in the documentation? https://reactstrap.github.io/components/form/


add installation into readme in chinese

Codecov Report

Merging #40 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #40   +/-   ##
=======================================
  Coverage   76.05%   76.05%
=======================================
  Files          75       75
  Lines        5229     5229
=======================================
  Hits         3977     3977
  Misses        862      862
  Partials      390      390

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 e2b34d0...ed29cf8. Read the comment docs.


Search by accession number is broken

We have an Apache rule that converts the URL to lowercase letters. Therefore the accession number also gets converted into lower case, causing there to be no hits in the database.
I've added fix to convert the accession to upper case before querying into the database.
I'm also checking to see if there is a way to do case insensitive queries

In retrospect I think having the Apache rule was a bit too broad of a stroke, given that it was implemented specifically to accommodate CoVizu and covizu URLs.  This is also breaking other things on the server such as https://filogeneti.ca/ApplBioinf

Reverted change made in #500. Search by accession number works now.


read challenge navigation

path planning for writes

frame carosel
record frame diffs
rose button for directions
paint button for painting (select type of paint)
select position on grid for moving bot
next step button for adding frames
play back of frames marks frames that are not correct
record state diffs so minor errors can be corrected

carousel has prev step, next step, add step


Keras SavedModel: Ignore custom metrics failure when compile=False

Keras currently doesn't support serialising and loading custom metrics in the saved model format.
However, when loading models with compile=False these metrics are not needed but still prevent the model from being loaded correctly.
This PR will ignore such errors when used with tf.keras.models.load_model(path, compile=False) allowing to reload models without code even if they have been saved with custom metrics. This is important since "SavedModels are [supposed to be] able to save custom objects like subclassed models and custom layers without requiring the original code".
This PR fixes #43478 which was introduced in TensorFlow 2.2. It is a workaround until proper support for serialising and deserialising custom metrics is implemented. /cc @k-w-w, @yhliang2018, @qlzh727
@mihaimaruseac Would you still accept a cherry-pick of this PR onto the 2.4 release branch, since it fixes a bug that prevents loading of saved models with custom metrics that have been saved with TensorFlow 2.2 or newer? To me it looks like the risk of this PR breaking any current users is pretty low, but I might be missing something here.

Apologies for the delay. Unfortunately this is too late for a cherry-pick as it will need another RC and then push the final release in the next year :(

Unfortunately this is too late for a cherry-pick as it will need another RC and then push the final release in the next year :(

That makes sense, I only pinged you because #43478 was labeled as a regression issue for 2.3 so I thought it might still have a chance to make it :)

In any case, after merging, let's make a cherrypick on the 2.4 branch. In case there is a patch release we can get it fixed in 2.4.1.