Improvements on the donation system

A "checkout screen" after downloading an app, informing the user of why donations are nice.
On top will be a donation button for the app the user most recently (/just) installed. And below is a list of other apps the user has installed from ßstore
Then the user can decide to donate to apps after they have tested them easily.
This "checkout screen" would also be available at any time, perhaps as one of the app categories

Because the title of this thread is just "improvements" I would like to say what is in my mind, too.
There should be an easy and quick way from the device itself so that user can appreciate the developer(s) of an app, either donations or a simple thank you message.

Because the title of this thread is just "improvements" I would like to say what is in my mind, too.
There should be an easy and quick way from the device itself so that user can appreciate the developer(s) of an app, either donations or a simple thank you message.

I think this can be simply achieved by a good rating system. We already got kind of rating system but it's not mature yet.

Because the title of this thread is just "improvements" I would like to say what is in my mind, too.
There should be an easy and quick way from the device itself so that user can appreciate the developer(s) of an app, either donations or a simple thank you message.

I think this can be simply achieved by a good rating system. We already got kind of rating system but it's not mature yet.


Build tests during buildPhase although they are not run

Projects may share unit tests by bundling them into "associated test
JARs". Setting -Dmaven.test.skip prevents even building the tests, so the
associated JARs are not built, and other projects complain of missing
dependencies. Instead set -Dmaven.test.skip.exec so that the unit tests are
compiled, but not run, during buildPhase.
Fixes: #32

Makes sense, this is what is done in mvnix-update so should be same in buildPhase.
Thank you for this!


error occurred while decoding column "created_at": mismatched types; Rust type core::option::Option<chrono::datetime::DateTime<chrono::offset::utc::Utc>> (as SQL type TIMESTAMPTZ) is not compatible with SQL type TIMESTAMP

Description

create table with col defined:

.col(ColumnDef::new(AccountBalance::CreatedAt).timestamp().extra("default now()".into()).not_null())

generate entity by using ‵sea-orm-cli`:

#[derive(Clone, Debug, PartialEq, DeriveEntityModel)]
#[sea_orm(table_name = "account_balance")]
pub struct Model {
    ...
    pub created_at: DateTimeUtc,
    ...
}

get data from database:

MyModel::find_by_id(id).one(&state.db).await?;

will get error:
error occurred while decoding column \"created_at\": mismatched types; Rust type `core::option::Option<chrono::datetime::DateTime<chrono::offset::utc::Utc>>` (as SQL type `TIMESTAMPTZ`) is not compatible with SQL type `TIMESTAMP`

Versions
sea-orm-cli: 0.8.2
sea-orm-migration = { version = "^0.8.0", features = [ "sqlx-postgres", "runtime-tokio-rustls" ] }

data can be inserted but will failed to get from database

I think this is related to: https://github.com/SeaQL/sea-orm/issues/586


[SPARK-13658][SQL] BooleanSimplification rule is slow with large boolean expressions

JIRA: https://issues.apache.org/jira/browse/SPARK-13658
What changes were proposed in this pull request?
Quoted from JIRA description: When run TPCDS Q3 [1] with lots predicates to filter out the partitions, the optimizer rule BooleanSimplification take about 2 seconds (it use lots of sematicsEqual, which require copy the whole tree).
It will great if we could speedup it.
[1] https://github.com/cloudera/impala-tpcds-kit/blob/master/queries/q3.sql
How to speed up it:
When we ask the canonicalized expression in Expression, it calls Canonicalize.execute on itself. Canonicalize.execute basically transforms up all expressions included in this expression. However, we don't keep the canonicalized versions for these children expressions. So in next time we ask the canonicalized expressions for the children expressions (e.g., BooleanSimplification), we will rerun Canonicalize.execute on each of them. It wastes much time.
By forcing the children expressions to get and keep their canonicalized versions first, we can avoid re-canonicalize these expressions.
I simply benchmark it with an expression which is part of the where clause in TPCDS Q3:
val testRelation = LocalRelation('ss_sold_date_sk.int, 'd_moy.int, 'i_manufact_id.int, 'ss_item_sk.string, 'i_item_sk.string, 'd_date_sk.int)

val input = ('d_date_sk === 'ss_sold_date_sk) && ('ss_item_sk === 'i_item_sk) && ('i_manufact_id === 436) && ('d_moy === 12) && (('ss_sold_date_sk > 2415355 && 'ss_sold_date_sk < 2415385) || ('ss_sold_date_sk > 2415720 && 'ss_sold_date_sk < 2415750) || ('ss_sold_date_sk > 2416085 && 'ss_sold_date_sk < 2416115) || ('ss_sold_date_sk > 2416450 && 'ss_sold_date_sk < 2416480) || ('ss_sold_date_sk > 2416816 && 'ss_sold_date_sk < 2416846) || ('ss_sold_date_sk > 2417181 && 'ss_sold_date_sk < 2417211) || ('ss_sold_date_sk > 2417546 && 'ss_sold_date_sk < 2417576) || ('ss_sold_date_sk > 2417911 && 'ss_sold_date_sk < 2417941) || ('ss_sold_date_sk > 2418277 && 'ss_sold_date_sk < 2418307) || ('ss_sold_date_sk > 2418642 && 'ss_sold_date_sk < 2418672) || ('ss_sold_date_sk > 2419007 && 'ss_sold_date_sk < 2419037) || ('ss_sold_date_sk > 2419372 && 'ss_sold_date_sk < 2419402) || ('ss_sold_date_sk > 2419738 && 'ss_sold_date_sk < 2419768) || ('ss_sold_date_sk > 2420103 && 'ss_sold_date_sk < 2420133) || ('ss_sold_date_sk > 2420468 && 'ss_sold_date_sk < 2420498) || ('ss_sold_date_sk > 2420833 && 'ss_sold_date_sk < 2420863) || ('ss_sold_date_sk > 2421199 && 'ss_sold_date_sk < 2421229) || ('ss_sold_date_sk > 2421564 && 'ss_sold_date_sk < 2421594) || ('ss_sold_date_sk > 2421929 && 'ss_sold_date_sk < 2421959) || ('ss_sold_date_sk > 2422294 && 'ss_sold_date_sk < 2422324) || ('ss_sold_date_sk > 2422660 && 'ss_sold_date_sk < 2422690) || ('ss_sold_date_sk > 2423025 && 'ss_sold_date_sk < 2423055) || ('ss_sold_date_sk > 2423390 && 'ss_sold_date_sk < 2423420) || ('ss_sold_date_sk > 2423755 && 'ss_sold_date_sk < 2423785) || ('ss_sold_date_sk > 2424121 && 'ss_sold_date_sk < 2424151) || ('ss_sold_date_sk > 2424486 && 'ss_sold_date_sk < 2424516) || ('ss_sold_date_sk > 2424851 && 'ss_sold_date_sk < 2424881) || ('ss_sold_date_sk > 2425216 && 'ss_sold_date_sk < 2425246) || ('ss_sold_date_sk > 2425582 && 'ss_sold_date_sk < 2425612) || ('ss_sold_date_sk > 2425947 && 'ss_sold_date_sk < 2425977) || ('ss_sold_date_sk > 2426312 && 'ss_sold_date_sk < 2426342) || ('ss_sold_date_sk > 2426677 && 'ss_sold_date_sk < 2426707) || ('ss_sold_date_sk > 2427043 && 'ss_sold_date_sk < 2427073) || ('ss_sold_date_sk > 2427408 && 'ss_sold_date_sk < 2427438) || ('ss_sold_date_sk > 2427773 && 'ss_sold_date_sk < 2427803) || ('ss_sold_date_sk > 2428138 && 'ss_sold_date_sk < 2428168) || ('ss_sold_date_sk > 2428504 && 'ss_sold_date_sk < 2428534) || ('ss_sold_date_sk > 2428869 && 'ss_sold_date_sk < 2428899) || ('ss_sold_date_sk > 2429234 && 'ss_sold_date_sk < 2429264) || ('ss_sold_date_sk > 2429599 && 'ss_sold_date_sk < 2429629) || ('ss_sold_date_sk > 2429965 && 'ss_sold_date_sk < 2429995) || ('ss_sold_date_sk > 2430330 && 'ss_sold_date_sk < 2430360) || ('ss_sold_date_sk > 2430695 && 'ss_sold_date_sk < 2430725) || ('ss_sold_date_sk > 2431060 && 'ss_sold_date_sk < 2431090) || ('ss_sold_date_sk > 2431426 && 'ss_sold_date_sk < 2431456) || ('ss_sold_date_sk > 2431791 && 'ss_sold_date_sk < 2431821) || ('ss_sold_date_sk > 2432156 && 'ss_sold_date_sk < 2432186) || ('ss_sold_date_sk > 2432521 && 'ss_sold_date_sk < 2432551) || ('ss_sold_date_sk > 2432887 && 'ss_sold_date_sk < 2432917) || ('ss_sold_date_sk > 2433252 && 'ss_sold_date_sk < 2433282) || ('ss_sold_date_sk > 2433617 && 'ss_sold_date_sk < 2433647) || ('ss_sold_date_sk > 2433982 && 'ss_sold_date_sk < 2434012) || ('ss_sold_date_sk > 2434348 && 'ss_sold_date_sk < 2434378) || ('ss_sold_date_sk > 2434713 && 'ss_sold_date_sk < 2434743)))

val plan = testRelation.where(input).analyze
val actual = Optimize.execute(plan)

With this patch:
352 milliseconds
346 milliseconds
340 milliseconds

Without this patch:
585 milliseconds
880 milliseconds
677 milliseconds

How was this patch tested?
Existing tests should pass.

cc @davies @marmbrus

Test build #52902 has started for PR 11647 at commit c6e8415.

Test build #52902 has finished for PR 11647 at commit c6e8415.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52902/
Test FAILed.

retest this please.

It still seem pretty slow. Any other way to speed this up by an order of magnitude?
cc @marmbrus

Test build #52905 has started for PR 11647 at commit c6e8415.

With latest change:
233 milliseconds
207 milliseconds
220 milliseconds

BTW, by completely removing the comparison of canonicalized expression in Expression.semanticEquals, it can't be faster anymore.
In other words, the logic in Canonicalize after this change should be unrelated to the time spent on BooleanSimplification now. If I am wrong about this, please let me know.

Test build #52909 has started for PR 11647 at commit c75e602.

What are the time spent beyond that?

I think it is the time needed to go through BooleanSimplification itself? It is a big rule. A few pattern cases in it is complicated as I see.

For example, this case:
// Common factor elimination for conjunction
      case and @ (left And right) =>
        val lhs = splitDisjunctivePredicates(left)
        val rhs = splitDisjunctivePredicates(right)
        ....

Because the where clause in TPCDS Q3 has a long pattern like this: (...) || (...) || (...) ..........., the recursively running splitDisjunctivePredicates should spend significant time.

Should I update the title of this PR, as I simplify Canonicalize instead of BooleanSimplification?

One way I can think of which is a bit ugly is to special case a sequence of ors and optimize it without recursion.

Test build #52905 has finished for PR 11647 at commit c6e8415.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52905/
Test PASSed.

Merged build finished. Test PASSed.

Hmm, like this?
protected def splitDisjunctivePredicates(condition: Expression): Seq[Expression] = {
  condition match {
    case Or(Or(Or(Or(Or(Or(Or(Or(l, r))))))), cond2) =>
      splitDisjunctivePredicates(l) ++ splitDisjunctivePredicates(r) ++
        splitDisjunctivePredicates(cond2)
    case Or(cond1, Or(Or(Or(Or(Or(Or(Or(l, r)))))))) =>
      splitDisjunctivePredicates(cond1) ++ splitDisjunctivePredicates(l) ++
        splitDisjunctivePredicates(r)
    case Or(Or(Or(Or(Or(Or(Or(l, r)))))), cond2) =>
      splitDisjunctivePredicates(l) ++ splitDisjunctivePredicates(r) ++
        splitDisjunctivePredicates(cond2)
    case Or(cond1, Or(Or(Or(Or(Or(Or(l, r))))))) =>
      splitDisjunctivePredicates(cond1) ++ splitDisjunctivePredicates(l) ++
        splitDisjunctivePredicates(r)
    case Or(Or(Or(Or(Or(Or(l, r))))), cond2) =>
      splitDisjunctivePredicates(l) ++ splitDisjunctivePredicates(r) ++
        splitDisjunctivePredicates(cond2)
    case Or(cond1, Or(Or(Or(Or(Or(l, r)))))) =>
      splitDisjunctivePredicates(cond1) ++ splitDisjunctivePredicates(l) ++
        splitDisjunctivePredicates(r)
    case Or(Or(Or(Or(Or(l, r)))), cond2) =>
      splitDisjunctivePredicates(l) ++ splitDisjunctivePredicates(r) ++
        splitDisjunctivePredicates(cond2)
    case Or(cond1, Or(Or(Or(Or(l, r))))) =>
      splitDisjunctivePredicates(cond1) ++ splitDisjunctivePredicates(l) ++
        splitDisjunctivePredicates(r)
    case Or(Or(Or(Or(l, r))), cond2) =>
      splitDisjunctivePredicates(l) ++ splitDisjunctivePredicates(r) ++
        splitDisjunctivePredicates(cond2)
    case Or(cond1, Or(Or(Or(l, r)))) =>
      splitDisjunctivePredicates(cond1) ++ splitDisjunctivePredicates(l) ++
        splitDisjunctivePredicates(r)
    case Or(Or(Or(l, r)), cond2) =>
      splitDisjunctivePredicates(l) ++ splitDisjunctivePredicates(r) ++
        splitDisjunctivePredicates(cond2)
    case Or(cond1, Or(Or(l, r))) =>
      splitDisjunctivePredicates(cond1) ++ splitDisjunctivePredicates(l) ++
        splitDisjunctivePredicates(r)
    case Or(Or(l, r), cond2) =>
      splitDisjunctivePredicates(l) ++ splitDisjunctivePredicates(r) ++
        splitDisjunctivePredicates(cond2)
    case Or(cond1, Or(l, r)) =>
      splitDisjunctivePredicates(cond1) ++ splitDisjunctivePredicates(l) ++
        splitDisjunctivePredicates(r)
    case Or(cond1, cond2) =>
      splitDisjunctivePredicates(cond1) ++ splitDisjunctivePredicates(cond2)

    case other => other :: Nil
  }
}

Ah, the above is wrong. I forget to consider other side of each Or.

Test build #52909 has finished for PR 11647 at commit c75e602.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52909/
Test FAILed.

Test build #52918 has started for PR 11647 at commit cbab017.

I was measuring the time to finish the code snippet shown in pr description. But I think I should only measure the time spent on val actual = Optimize.execute(plan).
Let me update it:
With this patch:
84362 microseconds
84075 microseconds
86192 microseconds
Without this patch:
492139 microseconds
503386 microseconds
463162 microseconds

Test build #52918 has finished for PR 11647 at commit cbab017.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52918/
Test PASSed.

Test build #52986 has started for PR 11647 at commit 04b3ce4.

Test build #52986 has finished for PR 11647 at commit 04b3ce4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52986/
Test PASSed.

@rxin @marmbrus I've addressed comments. Any other change I should do for this? Thanks!

Thanks - I will let @marmbrus do the final review & merge.

Test build #53050 has started for PR 11647 at commit 4d3a310.

Thanks for reviewing this.

Test build #53050 has finished for PR 11647 at commit 4d3a310.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53050/
Test PASSed.

LGTM, merging to master.


Spaces replaced by TAB in file editor

Perform all steps below and tick them with [x]

[x] Check the related part of the Documentation
[x] Update openHASP to the latest version
[x] Reproduce the issue and describe all steps

Describe the bug
When I edit a file on an OpenHasp device using the file editor and enter multiple spaces, they are replaced by TAB character(s)
To Reproduce
Just edit a file, enter  multiple spaces and the use the arrow keys to back go left. The cursor will jump rather than take one space-step at a time.
Expected behavior
No TABs stored
Screenshots or video
First I erase the spaces, then enter new spaces and lastly walks left with arrow keys. And the cursor jumps.
https://github.com/user-attachments/assets/91cd0b04-968a-4973-b0a4-c7877d12586c

Just noticed that I can type Oprion-Space on a Mac keyboard to enter hard spaces and they are marked with red background and dots.

In the editor field, press F1.
Then select Show Settings Menu ctrl-,

Uncheck "Soft Tabs" and see if this fixes the tab problem.

Editor field?

The field where you type your jsonl code.

Ok found it and problem solved. Is it possible to make soft tabs disabled by default?


Fonduer

Support Fonduer in Snorkel.

Ping me w @ajratner if anything I should look at here btw
On Thu, Sep 21, 2017 at 2:19 PM Braden Hancock<EMAIL_ADDRESS>wrote:

@bhancock8 commented on this pull request.
In snorkel/matchers.py
https://github.com/HazyResearch/snorkel/pull/596#discussion_r140361955:

@@ -225,8 +242,8 @@ def init(self):

     # Compile regex matcher
     # NOTE: Enforce full span matching by ensuring that regex ends with $!

   self.rgx = self.rgx if self.rgx.endswith('$') else self.rgx + r'$'

   self.r = re.compile(self.rgx, flags=re.I if self.ignore_case else 0)

   self.rgx = self.rgx if self.rgx.endswith('$') else ('(?:' + self.rgx + ')' + '$')

Added a comment. We group self.rgx first so that the $ applies to all
components of an 'OR' expression, and not just the last one. (e.g., we want
r'(a|b)$' instead of r'a|b$')
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/HazyResearch/snorkel/pull/596#discussion_r140361955,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABgw_bbvHDui-vq843ocUV34RJ89eShpks5sktLdgaJpZM4NCDEo
.

Closing this because Fonduer has been released as a separate repo: https://github.com/HazyResearch/fonduer


Update doc for .krmignore behavior

We should update the doc website to document the .krmignore file behavior in current kpt commands.

cc @mikebz @frankfarzan

There are still some questions around whether the .krmignorefile is the correct way to handle this. Ref https://github.com/GoogleContainerTools/kpt/issues/1664

@mortent Yes but we should document it for current behavior. If you think it's unnecessary then we can close this.


typos in README

Amazing work! Already using it for a project. Found a couple of typos. In the below part of the README median should be aggregate_median and plot_aggregate_metrics should be plot_interval_estimates`.
algorithms = ['DQN (Nature)', 'DQN (Adam)', 'C51', 'REM', 'Rainbow',
              'IQN', 'M-IQN', 'DreamerV2']
# Load ALE scores as a dictionary mapping algorithms to their human normalized
# score matrices, each of which is of size `(num_runs x num_games)`.
atari_200m_normalized_score_dict = ...
aggregate_func = lambda x: np.array([
  metrics.median(x),
  metrics.aggregate_iqm(x),
  metrics.aggregate_mean(x),
  metrics.aggregate_optimality_gap(x)])
aggregate_scores, aggregate_score_cis = rly.get_interval_estimates(
  atari_200m_normalized_score_dict, aggregate_func, reps=50000)
fig, axes = plot_utils.plot_aggregate_metrics(
  aggregate_scores, aggregate_score_cis,
  metric_names=['Median', 'IQM', 'Mean', 'Optimality Gap'],
  algorithms=algorithms, xlabel='Human Normalized Score')

Thanks, @MishaLaskin for pointing out the typos! Fixed them in<PHONE_NUMBER>1f88ac12f27cc83af02484e3bc499f. Also, it's awesome that you already are using the library :) Also, the colab at bit.ly/statistical_precipice_colab may also be useful regarding how to use the library.


Teacher mode

i have to check whether lessons where the teacher is absent are shown as blank.

for that i can probably use the style -> visibility: hidden attribute


Settings file permissions.

Hello,
during build of container, settings.php file and sites/default folder gets wrong permissions and Drupal shows error on status report page.
sites/default gets 775
settings.php gets 664
Thank you.

@yariklutsiuk add the following to your Dockerfile --
# Config directory should be non-writable.
RUN chmod 755 /app/docroot/sites/default

Just needs to be run after the code has been added to the image.


Add box2d new binding

This new binding:

Does not have GWT/iOS support yet, bc I dunno how to
Fixes https://github.com/libgdx/libgdx/issues/5948 and https://github.com/libgdx/libgdx/issues/6711
Is not as tested as the official one, but I've tested on Windows 11 and it works so far.

I feel like that's better suited as a PR to the libGDX repository itself than a third-party library, but even then a certain level of library maturity and stability would be required to make it to this list, as users expect all resources here to be battle-tested and ready to rock.
New Box2D bindings are always exciting so I'll keep the PR open for discussion, but I recommend an official libGDX PR is opened with it.

Oh I just realized setDestructionListener is neither implemented in the offical box2d. So this library is 100% drop-in replacement.

This PR has been moved to https://github.com/libgdx/libgdx/pull/7144


Portable Executable error: Invalid bitcast %379 = bitcast i32 %378 to i16

I've installed on my linux host portable executable, tried to run it and received that error:
quantum0@test-host:~/realesrgan$ ./realesrgan-ncnn-vulkan -i input.jpg -o output.png -n realesrgan-x4plus
[0 llvmpipe (LLVM 12.0.0, 256 bits)]  queueC=0[1]  queueG=0[1]  queueT=0[1]
[0 llvmpipe (LLVM 12.0.0, 256 bits)]  bugsbn1=0  bugbilz=0  bugcopc=0  bugihfa=0
[0 llvmpipe (LLVM 12.0.0, 256 bits)]  fp16-p/s/a=1/1/0  int8-p/s/a=1/1/0
[0 llvmpipe (LLVM 12.0.0, 256 bits)]  subgroup=8  basic=1  vote=1  ballot=1  shuffle=0
WARNING: lavapipe is not a conformant vulkan implementation, testing use only.
Invalid bitcast
  %379 = bitcast i32 %378 to i16
LLVM ERROR: Broken function
Aborted

What I'm doing wrong?
Maybe this will be useful:
quantum0@test-host:~/realesrgan$ uname -a
Linux test-host 5.4.0-96-generic #109-Ubuntu SMP Wed Jan 12 16:49:16 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Btw it's not enough just to download executable and run, i had to install these dependencies:
sudo apt-get install libvulkan-dev
sudo apt-get install libgomp1

Try purging vulkan and trying this: https://github.com/lutris/docs/blob/master/InstallingDrivers.md#amd--intel

Yes, that helps! Thank you very much!

Must this work on GPU? It doesn't work on Mac

@t-k-c I am also looking for a solution. Found any? I wanted to run it on a GCP VM.

I'm also running into this issue. Any help would be appreciated!


Angular-Storybook: a11y addOn does not show violations when it should do

Describe the bug
When using the a11y storybook addon, scenarios which should return violations but are not doing so.
To Reproduce
Steps to reproduce the behavior:

Visit Storybook official netlify link
Note how there are  No a11y violations found with the Invalid contrast example
Attempting this myself within my own project I'm seeing similar results:
Configure angular project to use a11y addon by following the getting started doc:

npm install -D @storybook/addon-a11y
import '@storybook/addon-a11y/register';

Configure stories with a11y addon:

import { checkA11y, configureA11y } from '@storybook/addon-a11y';

Specify stories file with a11y configurations
Add a11y decorator to stories file
Set the following scss attributes of our button to the color white: background-color, background-color-hover and background-color-active.
Set the display attribute of our component to display: none

Expected behavior
Should see violations as component is not meeting the a11y configurations ['wcag2a', 'wcag2aa', 'wcag21aa']
Screenshots
Storybook official netlify link

My own project

Code snippets
button.stories.ts
import { storiesOf } from '@storybook/angular';
import { withNotes } from '@storybook/addon-notes';
import { action } from '@storybook/addon-actions';
import { checkA11y, configureA11y } from '@storybook/addon-a11y';

import { ButtonComponent } from '../projects/component-library/src/lib/buttons/button/button.component';

// a11y Configurations
const a11yConfigs = {
  runOnly: ['wcag2a', 'wcag2aa', 'wcag21aa'],
};
configureA11y(a11yConfigs);

storiesOf('Buttons/Button', module)
  .addDecorator(checkA11y)
  .addDecorator(withNotes)
  .add(
    'Primary',
    () => ({
      component: ButtonComponent,
      props: {
        btnClick: action('Button was clicked'),
        text: 'Prescribe',
      },
    }),
    { notes: 'Primary style button' }
  )
  .add(
    'Primary disabled',
    () => ({
      component: ButtonComponent,
      props: {
        text: 'Prescribe',
        disabled: true,
      },
    }),
    { notes: 'Primary style disabled button' }
  );

button.component.scss
$primary-color: #0033a0;
$primary-hover: #00277c;
$primary-disabled: #cac8c5;

button {
  display: flex;
  align-items: center;
  justify-content: center;
  cursor: pointer;
  height: 44px;
  letter-spacing: 0.01em;
  font-size: 14px;
  border-radius: 100px;
  min-width: 196px;
  padding: 0 16px;
  font-family: Helvetica;
  color: #fff;

  &:focus {
    outline: none;
  }

  &.primary {
    background-color: #fff;

    &:hover {
      background-color: #fff;
    }

    &:active {
      background-color: #fff;
    }

    &:disabled {
      cursor: default;
      background-color: $primary-disabled;
    }
  }
}

System:

OS: [MacOS]
Device: [Macbook Pro 2017]
Browser: [chrome]
Framework: [angular]
Addons: [a11y]
Version: [5.0.0-rc.5]

Additional context
n/a

Storybook is not displaying any violations because in this case (BaseButton | Invalid contrast) axe-core, the library we are using for a11y, doesn't return any violations.
In fact this color-contrast rule is not in "passing" rules array neither but in incomplete.

As per doc:

The incomplete array (also referred to as the "review items") indicates which nodes could neither be determined to definitively pass or definitively fail. They are separated out in order that a user interface can display these to the user for manual review (hence the term "review items").

BUT for now only passes and violations array are displayed in SB ui. Maybe we should think about adding something to display incomplete rules array?

Thanks @gaetanmaisse . Opened a new feature request for this and closing this issue for now.


Update serde_yaml to v0.8.24

Automated changes by create-pull-request GitHub action

Codecov Report

Merging #19 (04227ca) into main (7792a14) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main     #19   +/-   ##
=====================================
  Coverage   1.95%   1.95%
=====================================
  Files        230     230
  Lines      11772   11772
=====================================
  Hits         230     230
  Misses     11542   11542

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 7792a14...04227ca. Read the comment docs.


Issue with GitHub Action and GPT Model Selection

Summary
I am currently using a GitHub Action in my workflow, but I'm encountering an unexpected response. Despite having a balance in my account, the action defaults to using gpt-3.5-turbo-16k instead of gpt-4.0.
Screenshot of the Issue
(Full error log is available in the attached screenshot.)

GitHub Action Configuration
Here is the relevant part of my GitHub Action configuration:
name: PR Agent

on:
  pull_request:
  issue_comment:

permissions:
  issues: write
  pull-requests: write
  contents: write

jobs:
  pr_agent_job:
    runs-on: ubuntu-latest
    name: Run on every pull request, and respond to user comments
    steps:
      - name: PR Agent action step
        id: pragent
        uses: Codium-ai/pr-agent@main
        env:
          OPENAI_KEY: ${{ secrets.OPENAI_KEY }}
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          GITHUB_ACTION.AUTO_REVIEW: true
          GITHUB_ACTION.AUTO_IMPROVE: true

Question
Is there a specific configuration or step required to ensure the GitHub Action utilizes gpt-4.0 instead of defaulting to gpt-3.5-turbo-16k? Do I need to use gpt-4.0 specifically?
Any guidance or suggestions would be greatly appreciated.

Somehow it started working.

Similar Issues

Unable to use Github Action anymore (score=0.87)

Unable to get PR-Agent GitHub Action to use Azure OpenAI (score=0.86)

GPT-4 prediction failure (score=0.86)

Github actions error (score=0.85)

Inline comments aren't working (score=0.84)

Similar Issues

Unable to use Github Action anymore (score=0.73)

Unable to get PR-Agent GitHub Action to use Azure OpenAI (score=0.72)

GPT-4 prediction failure (score=0.72)

Github actions error (score=0.69)

Inline comments aren't working (score=0.69)

Nothing happens? What am I missing? (score=0.67)

Adding Descriptive Comments or Reviews Directly Upon PR Creation (score=0.67)

Unrecognized request argument supplied: azure on GitHub Actions (score=0.66)

Similar Issues

Unable to use Github Action anymore (score=0.73)

Unable to get PR-Agent GitHub Action to use Azure OpenAI (score=0.72)

GPT-4 prediction failure (score=0.72)

Github actions error (score=0.69)

Inline comments aren't working (score=0.69)

Nothing happens? What am I missing? (score=0.67)

Adding Descriptive Comments or Reviews Directly Upon PR Creation (score=0.67)

Unrecognized request argument supplied: azure on GitHub Actions (score=0.66)

Similar Issues

Unable to use Github Action anymore (score=0.73)

Unable to get PR-Agent GitHub Action to use Azure OpenAI (score=0.72)

GPT-4 prediction failure (score=0.72)

Github actions error (score=0.69)

Inline comments aren't working (score=0.69)

Nothing happens? What am I missing? (score=0.67)

Adding Descriptive Comments or Reviews Directly Upon PR Creation (score=0.67)

Unrecognized request argument supplied: azure on GitHub Actions (score=0.66)

Similar Issues

Unable to use Github Action anymore (score=0.73)

Unable to get PR-Agent GitHub Action to use Azure OpenAI (score=0.72)

GPT-4 prediction failure (score=0.72)

Github actions error (score=0.69)

Inline comments aren't working (score=0.69)

Nothing happens? What am I missing? (score=0.67)

Adding Descriptive Comments or Reviews Directly Upon PR Creation (score=0.67)

Unrecognized request argument supplied: azure on GitHub Actions (score=0.66)


District hover in the table changes the map and map stats

@yogigachinmath Thanks for this! UX is awesome, though there seems to be a synchronization issue somewhere. It might be unrelated to your code.

@yogigachinmath Thanks for this! UX is awesome, though there seems to be a synchronization issue somewhere. It might be unrelated to your code.

I will look at it

Hi, I have fixed the bug please check it out. There is one more bug, Initial hover is not working

Just checked it out locally, looks good! Could you squash your commits and rebase over master?

Just checked it out locally, looks good! Could you squash your commits and rebase over master?

Hi, I have already merged it, sorry i did not squash them.

There seems to be some merge conflicts, take your time to get them resolved and push them :)
This looks good so looking forward to it!

@yogigachinmath Thanks! It's coming together nicely. I found another small bug. If you stop hovering on a district in the table, it's still highlighted on the map. Then, if you go hover over the map, multiple districts are highlighted.

Plus, some small merge artifacts need to be cleaned up.


change +[RLMSyncUser allUsers] to return a dictionary

As described in realm/realm-cocoa-private#376. /cc @austinzheng

Now rebased on top of #4322 for simplicity, as I intend to merge that once it passes CI, then this.


There is no way to go back once you opened the link

When I tried to open a link to gitlab, it rendered inside of the app, not opening new browser. So it's stucked there and can't comeback to Mattermost.
It made the app unsusable, because whenever you open the link, you need to force quit and restart the app to go back.
Tested on iPhone7, iOS 10.1.1
App version 3.5
Maybe it's related with other issue in desktop app.
Thank you.

+1 we're currently not able to use the app anymore since the GitLab login button is opening the GitLab login URL in the browser app Instead of staying in the Mattermost app.

@AimenJoe would you have a sample link for us to try? Does this reproduce for all GitLab links, or only a few?

@qkdreyer I think yours might be caused by sessions expiring, we have a ticket here to look into the issue: https://mattermost.atlassian.net/browse/PLT-4349
Can you try uninstalling and then reinstalling the app to see if the problem is resolved?

@jasonblais Maybe it caused we are using mattermost in the same domain as gitlab instance. The external links working fine, only the gitlab links doesn't work.
Gitlab installed on : http://abcd.ab:8000
Mattermost installed on : http://abcd.ab:9000
Sorry that I can't give you the real link since its private..
Thank you.

@jasonblais you were totally right, reinstalling fixed the problem. Thanks :)

Hmm, thanks @AimenJoe, let me check with our devs and get back to you

Hey @AimenJoe, really sorry for the delay..
Can you help upgrade to Mattermost 3.6, available on our download page, and see if the issue persists?
If it does, please let me know

@jasonblais It's still same. When I open the link from Gitlab which is installed same domain with different port, it opens inside of webview.

@AimenJoe we're having some trouble reproducing the issue and we suspect it might be a problem on the GitLab side.
Would you like to post this in the GitLab troubleshooting forums instead?

Closing this issue due to inactivity. Please reopen if the issue is still unresolved.


add 0.3.0 change list

What this PR does:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #210 (58deeaa) into develop (3fb9d1b) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 58deeaa differs from pull request most recent head 43c2b05. Consider uploading reports for the commit 43c2b05 to get more accurate results

@@           Coverage Diff            @@
##           develop     #210   +/-   ##
========================================
  Coverage    39.71%   39.71%
========================================
  Files           40       40
  Lines         2397     2397
========================================
  Hits           952      952
  Misses        1343     1343
  Partials       102      102

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 3fb9d1b...43c2b05. Read the comment docs.


fix(npm): Updating deps. 0 vulns

:tada: This PR is included in version 1.2.2 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Responsive Navbar

Hi @VanshKing30, I've noticed following problems with navbar:

Currently there is no navbar styling to indicate the current page user is navigating.
Navbar is not responsive on smaller screen sizes.
Nav elements text(not centered) styling is quite not desirable:

I would like to work on this for GSSoC'23. Can you please assign this to me?

What changes can you make , please share some screenshots

Hey @VanshKing30 , these are some issues which I can work on a fix:
Hover color on login button doesn't look as good in my opinion:

Text in nav elements is not center aligned:

Extra padding or left of navbar on small size screen

And also a small pop-like framer motion animation would look good on nav elements, and some highlight inidicating which page is active.

@The-Phoenics How long will it take you to solve this issue ?

@The-Phoenics How long will it take you to solve this issue ?

I can do these fixes by tomorrow.


choose audio encoding settings

with ffmpeg -i i can see the audio format used in recordings is vorbis, which i assume is just the default that ffmpeg uses. i'd prefer it if i could record with libopus at a custom bitrate, or sometimes flac.

I recently added --audio-codec (set codec, let ffmpeg decide encoder) and --ffmpeg-audio-encoder (hardcode which encoder to use) flags in master. Agree that a --audio-bitrate flag would be a good addition still.

added --audio-bitrate. Open new issues for other feature requests


Add prev/next panel item actions. Fix #44625

@isidorn Not done yet. I found panelService is not returning the panels in correct order when some of them were hidden / move-around.
Any idea how can I retrieve the order and visibility of the panels? IPanel and IPanelIdentifier don't seem to have it.

I also wanted to suggest having Ctrl-Tab and Ctrl-Shift-Tab bound to them using panelFocus. But it seems we don't have that when condition and have to do terminal/output/problems/debug-console focus. But when I look even closer there seems to be no output focus or debug-console focus...
Just want to know if you are pro or against adding those two shortcuts behind the when clause.

Against since it will break muscle memory of users. Currently ctrl-tab changes the editor no matter the focus. I would not just change this.

@isidorn How about cmd + ] and [? They are bound to indentLine or outdentLine by default on editorTextFocus. I'm thinking about making them go to next/prev Viewlet or Panel Item when Sidebar or Panel has focus.

@octref let's start without a default shortcut and if we see that this action is popular we can easily add one later. This is a better strategy than adding first since it is later hard to remove

@octref thanks for your PR. However this week we are in endgame and on Thu I go on vacation so I will review this next milestone. Hope that makes sense.

Sure, see you next month :)

@octref thanks again for your PR. However I find it strange that we add next / previous panel and we do not add next / previous viewlet.
How does this go together with the Views > Open View action.
From the implementation perspecitve everything looks great. Except that I would try to do more code reuese between the two actions. Since they are doing everything the same just the +1 or -1 on to which panel to navigate.
fyi @sandy081 since he is tackling various views this milestone

@isidorn

However I find it strange that we add next / previous panel and we do not add next / previous viewlet.

I think it's better to do that as another PR. I can do it too.

How does this go together with the Views > Open View action.

I never used it. I believe that action serves a different purpose than this one. Ideally in each group (editor, viewlets, panels) I just want an action to go to the prev/next thing visually, not to recall names of each parts of VS Code and jump to it.
What I asked for in #15262 (UI part aside) is contextual ctrl+tab. Ideally I hope we can have these commands / when clauses
Ctrl-Shift-Tab:

workbench.action.previousPanelItem when panelFocus
workbench.action.previousViewlet when viewletFocus (or sidebarFocus)
workbench.action.previousEditor when editorFocus

Ctrl-Tab:

workbench.action.nextPanelItem when panelFocus
workbench.action.nextViewlet when viewletFocus (or sidebarFocus)
workbench.action.nextEditor when editorFocus

@octref thanks. The PR looks good to me know. And these actions make sense.
Let's see what @bpasero and @sandy081 think about introducing these actions and if they are ok we can merge it in.

I think having similar action to navigate viewlets is good. How about navigating views inside viewlet?

@isidorn if you want to merge for April, feel free to review. otherwise running out of time, moving to May.

@bpasero I reviewed this and the code is good. I just need your blessing to add the "Next Panel" / "Previosu Panel" actions. You do not have to look at the code

@isidorn oh OK. We can have the actions but I am with @sandy081 and think for consistency we could have the same actions for viewlets as well.
@octref I think commands like "Previous Panel item" should user uppercase all the way (e.g. Previous Panel Item)

@bpasero great, thanks for feedback
@octref then this PR is good, just update the label and once you provide the viewlet actions PR I will merge both these things in. In the meantime let's keep this on deck.

@isidorn @bpasero I've cleaned up the API a bit, including

getOrderedPanels(): string[] -> getPinnedPanels(): IPanelIdentifier[] to follow more of our terminology and align with getPanels
Clarified a bit on that getPanels return panels following the visual order

I have a question though. In compositeBar.ts, now we have this:
	private getVisibleComposites(): string[] {
		return Object.keys(this.compositeIdToActions);
	}

	public getPinnedComposites() {
		return this.pinnedComposites;
	}

They seem to be doing the same thing.

Why is getVisibleComposites using Object.keys(this.compositeIdToActions) instead of pinnedComposites? Are those two different in some scenarios?
Can I remove getPinnedComposites and make getVisibleComposites public?

@octref sorry for not responding I was on vacation when you asked the question.
And now @sandy081 has refactored the whole rold of the compositeBar so you would have to check with him.
So for us to continue here can you please resolve conflicts possible with the help of Sandeep and once we have taht we can merge that in if @sandy081 agrees

A lot has changed on master.. I'll open a new PR.


Update Overview figure on Simularium homepage

What needs to happen?
Update the overview image on the homepage. I've split that image to 2, we can stack them (simularium-overview.png first then simularium-agent-rep.png) or sandwich a chunk of text with them (the chunk that starts with "Current object representation options..." and ends before "Planned for future").
Why should we do this?
This image has been updated in the submitted paper and we want to match that.
When does this need to get done?
Before the paper is published.
Assets
Simularium overview

Agent rep

Confirmed with @blairlyons we don't need to sandwich text between the 2 new images, will keep layout as is.


Preserve stacktraces for async/await

We use self.ifError(err) instead of self.fail()
which logs the original stacktrace instead of creating
a new Error object.
Fixes #536

Addressed feedback.

Addressed feedback.

Cleaned up the test, this looks good to merge to me.

Cleaned up the test, this looks good to merge to me.

I’m going to land #538 first and rebase this, so i can be sure tests pass in all relevant node versions, and then I’ll land this one.

I’m going to land #538 first and rebase this, so i can be sure tests pass in all relevant node versions, and then I’ll land this one.

Codecov Report

Merging #537 (cb5d7fd) into master (82c3904) will decrease coverage by 0.29%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master     #537      +/-   ##
==========================================
- Coverage   73.87%   73.57%   -0.30%
==========================================
  Files          19       19
  Lines         754      757       +3
  Branches      145      146       +1
==========================================
  Hits          557      557
- Misses        197      200       +3

Impacted Files
Coverage Δ

lib/test.js
95.66% <75.00%> (-0.59%)
:arrow_down:

lib/results.js
99.30% <0.00%> (-0.70%)
:arrow_down:

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 82c3904...cb5d7fd. Read the comment docs.

Codecov Report

Merging #537 (cb5d7fd) into master (82c3904) will decrease coverage by 0.29%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master     #537      +/-   ##
==========================================
- Coverage   73.87%   73.57%   -0.30%
==========================================
  Files          19       19
  Lines         754      757       +3
  Branches      145      146       +1
==========================================
  Hits          557      557
- Misses        197      200       +3

Impacted Files
Coverage Δ

lib/test.js
95.66% <75.00%> (-0.59%)
:arrow_down:

lib/results.js
99.30% <0.00%> (-0.70%)
:arrow_down:

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 82c3904...cb5d7fd. Read the comment docs.

I'll fix these last failing tests, and land this tonight.

I'll fix these last failing tests, and land this tonight.


[enhancement] horizontal portals

I'm sure you've thought of this, but if you haven't, here's the suggestion. Other mods do this but only using crying obsidian. Your mod changes most of the things datapacks solved a few years ago, but in mod form and in one file. Horizontal portals would be awesome for many base compaction reasons but also because I don't want to find a huge up/down space to put a fancy portal in my base rn, but I do have a very convenient large tube that'd fit a portal at the bottom.
Anyway, great mod.

I second this, along with making the portal that spawns in the Nether match the shape of the one you made in the overworld (at least as a config option given for the potential to break blocks)


Update: Upgrade node to 8 in travis build

Cuts the build time by a few minutes for each package
The difference:

This comes with the upgrade. But as long as nothing breaks, i guess we could use this.


Any way to Undo the last applied filter?

Tried by removing the last value from filterConf but it's not resetting in the UI. Any ideas

in my tables, I use a separated search input and I am binding a variable to the input
when I want to reset the UI i change the variable and call the onSearch method


1344 model comparison

Okay... so based on discussions with @ScottKolmarWFI I think we've pushed various pieces into main and the rest of the code should be handled by the new plugin manager, so we'll close this PR


Background Color

How can I change the background color?

Configuration
You can find all the configuration in /etc/ly/config.ini. The file is commented, and includes the default values.

Can I Use Hex Values?
I would also like to know. The configuration is commented, but I'm not certain what the numbers for the background colour correspond to?

Same Question

You may find this link helpful: https://askubuntu.com/questions/147462/how-can-i-change-the-tty-colors

@xenbeta
I noted a workaround which works for me to use hex colors in #146.

Well, to be honest config.ini isn't so much commented. The text background color id is quite ambiguous. It could, at least, include a range of id's to be used.
In my case, changing this value does nothing. My screen keeps black/white.

You can find a list of color codes in termbox_next/termbox.h:
// Colors (see struct tb_cell's fg and bg fields).
#define TB_DEFAULT 0x00
#define TB_BLACK   0x01
#define TB_RED     0x02
#define TB_GREEN   0x03
#define TB_YELLOW  0x04
#define TB_BLUE    0x05
#define TB_MAGENTA 0x06
#define TB_CYAN    0x07
#define TB_WHITE   0x08

TB_DEFAULT is analogous to TB_BLACK for background colors and TB_WHITE for foreground colors. All other termbox colors are equivalent to their system color codes (TB_BLACK represents color0, TB_RED represents color1, etc...)
As a side note, the bg option only affects the background color for the login box and the shutdown/reboot options.

 Why not use normal strings or integers as normal people, like "red", "blue" or 2, 3, 4. I'm afraid not everyone understands this "colour table code" (not even me, I'm not a programmer!).

How did you get the whole background that dark blue/greyish color?

I'm also curious as to how you changed the background color entirely.

I had this same problem and suffered a little to solve it. I will post here a summary of my research as a reference for the future:
The option bg in the configuration file is misleading. It don't change the color of the background but only the dialog box and the 'menu bar' at the top. That is, the actual background will always be pure black.
If you are still interested it takes values between 0 and 8, some comments suggest using '0x00' to '0x08' but at least for me that don't work. The colors are as described by https://github.com/nullgemm/ly/issues/207#issuecomment-662795683, you can see the actual colors by opening the TTY (CTRL + ALT + F3) and sending the command msgcat --color=test. Changing the TTY colors as in this link won't work though, .bashrc or .zshrc are only loaded after login so any change won't affect ly.
The working solution  is to change the colors of the TTY using systemd like was done here: https://github.com/nullgemm/ly/issues/146.
You should add this line:
ExecStartPre=/usr/bin/printf '%%b' '\e]P0{background-color}\e]P7{foreground-color}\ec'
On file /lib/systemd/system/ly.service, line 7 (under [Service]). Change background-color and foreground-color to the RGB code of the desired color, don't use "#", see https://github.com/nullgemm/ly/issues/146 for a working example. Note that this is changing the value of color 0 and color 7 so remember to keep the bg and fg as the default values in order for this to work.

I use FreeBSD, so no systemd here.

Guys, checkout my idea to use half blocks "quadrants" for the Doom Fire effect to make the fire a better resolution https://github.com/nullgemm/ly/issues/236

Guys, checkout my idea to use half blocks "quadrants" for the Doom Fire effect to make the fire a better resolution https://github.com/nullgemm/ly/issues/236

I've never seen someone hijack a github issue like this before.
In all honestly though, that's not related.

Changing the background color is not possible in Ly, and I don't plan on supporting it. However you can change the default TTY color palette with setcolors and even automate this with mkinitcpio-colors, this is what I do.

Changing the background color is not possible in Ly, and I don't plan on supporting it. However you can change the default TTY color palette with setcolors and even automate this with mkinitcpio-colors, this is what I do.

Do you mind sharing how you got mkinitcpio-colors working? I can't seem to get the colors hook to work.


Request: release a library

The idea is a Java (or Kotlin) library (jar) that is decoupled from the Android gui. This way we could write our own gui in Java or Kotlin for Android. The library would take prompts as input (String, JSON or what ever) and would return the answer after computing (as Strings or better as Streams).

+1

Moving to #361


Enable configuration of AI service method parameters allowed annotations #888

Looks good!
Can you squash the commits and also add the test?
Thanks!

Looks good!
Can you squash the commits and also add the test?
Thanks!

Sure, it will be time to rebase/push when the test is ready :+1:

Finger crossed for ci :crossed_fingers:
Also, do you think this kind of changes could be candidate for a potential 0.18.1 release, if any ? Is there a need to backport on maintenance branch or so ?

We can add it to 0.18.1 as it's a small change and this repository is expected to move fast anyway


Change MatrixCache.F to allow pickling solved problems.

Previously we couldn't pickle problems that are already solved, as MatrixCache.F is a function that cannot be pickled. I changed MatrixCache.F to be a function object (thanks to functools.partial()) to fix this. This should allow pickling and sending solved problems through multiprocessing.

looks good.


fix: slate errors in action creator editor when switching actions types

Video: http://recordit.co/fs733thtK0
The first time you switch an action type to an API or CARD you also have to select a Callback or Template. Changing either of these options call their respective onChange* function and which save the option and setting up the SlateValuesMap to match the associated callback arguments or template variables.
There is a related behave where the SlateValuesMap is always one stop behind. For instance when you change to API, it will still be in the format for a TEXT payload until you select a callback option which forces it to be re-written to match the callback arguments.
The bug is that the SlateValues will still be in the incorrect format when switching. Normally this is ok because we haven't selected a Callback or Template but since and won't try to use/render the SlateValues but since the option is already saved it will try to render the values even though they are wrong.
Similarly to trying to preserver entities I tried to preserve the dropdown value, but I was still showing errors. I think to due the sequence of setStates that occur.  We have to re-use the logic for setting up the slate value but only do it under the right conditions and it was getting difficult.
Again defaulted to safe path of resetting the values.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Feature/k8s dedicated workers

[X] Installs cleanly on a fresh build of most recent master branch
[X] Upgrades cleanly from the most recent release
[X] Updates documentation relevant to the changes

No docs to update for this change, it just adds @Zogg's dedicated k8s roles to lay groundwork for future updates in 1.1. We'll be using this branch as a jumping off point for splitting #1133.

Only one problem on this still: the sample files need to be updated. GCE most especially, since the sample file uses the new syntax.

@BrianHicks Is this ready for testing? How will we know that it works properly, just that there are additional worker nodes with the listed properties?

yes, and yes.

@BrianHicks What do you think about adding them to the terraform configurations in testing/?

Yes, definitely.

On Apr 5, 2016, at 4:59 PM, Langston Barrett<EMAIL_ADDRESS>wrote:
@BrianHicks What do you think about adding them to the terraform configurations in testing/? Should they be tested on every Travis build?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

It also doesn't look like this updates do.sample.tf.

This should probably update ALL the *.sample.tf files.

The workers are created as expected.


Library deprecation support

Deprecation is an important part of the open source software lifecycle. Construct library vendors should have a way to mark a library as deprecated on the Construct Hub.
On NPM, authors can specifically mark their packages as deprecated, in a process which allows them to include a deprecation reason. See https://docs.npmjs.com/deprecating-and-undeprecating-packages-or-package-versions
Requiring authors to specify some kind of extra metadata in their package.json isn't really appropriate here because after an author has deprecated a library, it's likely they do not want to publish a new version (with the updated package.json).
A more appropriate solution here is to check against externally to see if the package has been deprecated. It's possible to check if the package has been deprecated on npm from the output of npm view <package> --json. (We could also (optionally) check if the package's repository is on GitHub, in which case if the repository has been archived then we could assume it has been deleted.)
If a package is deprecated, we still want to show it and its API reference, but we probably don't want to show it in search results (?) and we should clearly indicate that it is deprecated on the package's detail page

In addition authors can mark specific versions of a package as deprecated. By default they are hidden on npmjs.com and viewing a deprecate version does prominently display a deprecation notice.

Upvoting this on behalf of HashiCorp -- we have a prebuilt provider that we decided to no longer maintain: https://github.com/hashicorp/cdktf-provider-yandex and archived the GitHub repo, but it is still appearing as though it is actively being supported in Construct Hub: https://constructs.dev/packages/@cdktf/provider-yandex/v/2.0.32?lang=typescript which is quite problematic for us.

Shortly checked the situation, roughly 100 packages listed which are either deprecated or gone from npm registry
#/bin/bash
# needs jq and npm install -g check-is-deprecated
echo '["'$(check-is-deprecated $(curl -s "https://constructs.dev/catalog.json"| jq -c '.packages[] | .name' | sort -u | jq -scr ' join(" ")') | tr '\n' '--')'"]' \
  | jq '.[0] | split("--")[] | split(":") | {package: .[0], verdict: (if .[1] == "-✖ npm" then "deprecared" else "gone" end)}' \
  | jq -s 'group_by(.verdict)[] | {(.[0].verdict): [.[] | .package]}' | jq -s

[
  {
    "deprecared": [
      "@aws-cdk/app-delivery",
      "@aws-cdk/assets-docker",
      "@aws-cdk/aws-dynamodb-global",
      "@aws-cdk/aws-eks-legacy",
      "@aws-cdk/aws-quickstarts",
      "@aws-cdk/cdk",
      "@aws-cdk/cdk-assets-schema",
      "@aws-cdk/runtime-values",
      "@aws-cdk/yaml-cfn",
      "@cdk-cloudformation/alexa-ask-skill",
      "@cdk-cloudformation/registry-test-resource1-module",
      "@moia-dev/bastion-host-forward",
      "@moia-dev/bastion-host-rds-forward",
      "@pepperize-testing/cdk-autoscaling-gitlab-runner",
      "@pepperize/cdk-serverless-cluster-from-snapshot",
      "@quirion/cdk-constructs",
      "@stackspot/cdk-core",
      "@stackspot/cdk-event-bridge",
      "@stackspot/cdk-sns",
      "@stefanfreitag/aws-budget-notifier",
      "cdk-automated-waf",
      "cdk-cloudfront-deploy",
      "cdk-datalake-constructs",
      "cdk-lambda-token-authorizer-jwt",
      "cdk-secrets",
      "cdk-ssm-document",
      "cdk-ssm-documents",
      "cdk-triggers",
      "hiii-construct-utils",
      "monocdk-experiment",
      "monocdk-nag",
      "monocdk-serverless-clamscan",
      "pgstac-cdk-construct",
      "udondan-test-package"
    ]
  },
  {
    "gone": [
      "@cloudxs/cdk-iot-lorawan",
      "@jayhuss/npm-zip-lambda",
      "@jayhuss/serverless-nextjs",
      "@jhuss/code-zip",
      "@kikoda/constructs",
      "@pepperize-testing/cdk-ses-smtp-credentials",
      "@renovosolutions/cdk-managed-instance-role",
      "@richicoder/cdk-ecs-s3-files",
      "@rocktemplates/aws-cdk-templates",
      "@stackspot/cdk-api-gateway",
      "@stackspot/cdk-app-task-definition",
      "@stackspot/cdk-ecs",
      "@stackspot/cdk-env-container",
      "@stackspot/cdk-env-eventbridge",
      "@stackspot/cdk-env-load-balancer",
      "@stackspot/cdk-env-sns",
      "@stackspot/cdk-env-vpc",
      "@stackspot/cdk-load-balancer",
      "@stackspot/cdk-vpc",
      "cdk-chime-phone-numbers",
      "cdk-ec2-imagebuilder",
      "cdk-ecs-service-extensions",
      "cdk-ez-vpc",
      "cdk-k0s-cluster",
      "cdk-monitoring",
      "cdk-rstudio-sagemaker",
      "cdk-rstudio-workflow",
      "cdk-vue3",
      "cdkfoldingathome",
      "cdktf-dbt-snowflake",
      "cdktf-metaflow",
      "cdktf-snowflake",
      "cloudadder-cdk-constructs",
      "custom-cloud9-ssm",
      "django-cdk",
      "dynamodb-dashboard-example",
      "dynamodb-projen",
      "easy-cerver",
      "jacobitegboje",
      "my-dummy-construct",
      "my-second-projen-construct",
      "s3-bucket-alpha",
      "s3-bucket-open",
      "s3-bucket-tt",
      "schadem-cdk-construct-csv-to-aurora",
      "schadem-cdk-construct-sfn-comprehend-classifier",
      "schadem-cdk-construct-sfn-generate-csv",
      "schadem-cdk-construct-sfn-idp-decider",
      "schadem-cdk-construct-sfn-test",
      "schadem-cdk-construct-sfn-textract-classifiction-configurator",
      "schadem-cdk-construct-sfn-textract-output-config-to-json",
      "schadem-cdk-construct-sfn-token-bucket",
      "schadem-cdk-construct-test",
      "schadem-cdk-construct-test1",
      "schadem-cdk-construct-textract-generic-async",
      "schadem-cdk-construct-textract-generic-sync",
      "serverless-spy2",
      "sns-app-jsii-component",
      "test-cdk-construct",
      "test-cdk-constructs-v2",
      "test1-cdk-constructs",
      "test1-cdk-constructs-v2",
      "test3-cdk-constructs-v2",
      "test5-cdk-constructs",
      "test6-cdk-constructs-v2",
      "user-pool-identity-provider-github"
    ]
  }
]

https://gist.github.com/cfuerst/8b9853dbd9072b16cf6f2a31fe75ff5f


duplicate SST in svg_elements.csv

Just a heads up, not a real "issue". I modified svg_elements.csv as there were two entries for sea surface temperature. Also the ordering of the svg elements is the way Jennifer Brown requested.

Ah, yeah sorry I didn't see that line was already in there. :+1:


How to disable highlight in the  high contrast theme?

you cannot see the words with the high contrast theme,i try to use this:

"ep_comments_page": {"highlightSelectedText": false}

It did not work.

Duplicate of #217


GH-43796: [C++] Indent preprocessor directives

Rationale for this change
This is for easy to read.
FYI: Google C++ style guide doesn't require indent in preprocessor directives nor deny it:
https://google.github.io/styleguide/cppguide.html#Preprocessor_Directives
// Good - directives at beginning of line
  if (lopsided_score) {
#if DISASTER_PENDING      // Correct -- Starts at beginning of line
    DropEverything();
# if NOTIFY               // OK but not required -- Spaces after #
    NotifyClient();
# endif
#endif
    BackToNormal();
  }

What changes are included in this PR?

Add clang-format configurations for preprocessor directives indentation
Apply these configurations

Are these changes tested?
Yes.
Are there any user-facing changes?
No.

GitHub Issue: #43796

(I'll fix the lint failure later.)

The discussion thread: https://lists.apache.org/thread/gb2dpphhwz358zfb03k9ly18gs92rzdr

No objection in the discussion thread.

I'll merge this.


Support for ??? "Empty Coalesce" operator

Hi,
I'm using this with Craft CMS Twig templating and am curious how I'd go about circumventing a syntax error with the ??? operator.
I use this plugin, which it itself is based on a PR pending on the Twig repo, quite a bit.
When compiling with Prettier I get Error: ERROR: Unexpected token "?" of value "?" which occurs as an example, within a conditional such as {% if foo ??? bar %}
Obviously, it's right to throw an error as it isn't official Twig markup, but was wondering if there was a way to get past that.
Thanks a bunch for your work on this!

Hey, have you found a good solution? I only found out that I could use <!-- {% set id = element.myId ??? null %} --> around it to get past it, but these results in a lot of <!-- --> comments on page 🤔


Remote self certs test fails on Mac and Windows

Mac times out
Windows gives this error:
Failed to connect to remote Jupyter notebook. Check that the Jupyter Server URI setting has a valid running server specified. https://fv-az42:8888/ TypeError: request to https://fv-az42:8888/api/contents/?1584437624293 failed, reason: write EPROTO 796:error:1425F102:SSL routines:ssl_choose_client_version:unsupported protocol:c:\ws\deps\openssl\openssl\ssl\statem\statem_lib.c:1922:

Test is disabled until these can be fixed

No longer failing

Did you re activate the tests? They were disabled

And if you could look at this bug first before the nice to haves:
https://github.com/microsoft/vscode-python/issues/11156

And if you could look at this bug first before the nice to haves

Oops missed that. sorry.

They were disabled

🤦

THis is not working for me. I get a 500 error.

Hard to say if this is working or not as the flake tests aren't passing

going to close this for now. Works on linux and mac.


Update tests to use test data created on dev stack.

What
A number of fixes were added and functions created to clean the data so that comparisons between actual and expected results can be made.
The README has been updated so that users will know what steps in docker ckan stack are needed to set it up for the tests to pass.
Reference
https://trello.com/c/D0RJy2Q3/178-improve-stability-and-reliability-of-functional-tests-on-dev-stacks

Can we at least have some more comments/docstrings/more granular commit messages that explain some of what's going on with the new code?

I'll add something in the code


Fix the decryption so that it uses the default proxy

The computePubShares and the setup of the nodes use the same endpoints but the computePubShares does not need to be redirected to a proxy address explicitly (it can use the default)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.08%) to 62.533%

Files with Coverage Reduction
New Missed Lines
%

contracts/evoting/evoting.go
4
69.2%

Totals

Change from base Build<PHONE_NUMBER>:
0.08%

Covered Lines:
3101

Relevant Lines:
4959

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.08%) to 62.533%

Files with Coverage Reduction
New Missed Lines
%

contracts/evoting/evoting.go
4
69.2%

Totals

Change from base Build<PHONE_NUMBER>:
0.08%

Covered Lines:
3101

Relevant Lines:
4959

💛  - Coveralls


Error when reading a xls file ("more columns than column names")

Hi everyone,
I'm having a problem reading a xls file in R Studio due to the following error message (using the "gdata" package and the command "read.xls"):
"Error in read.table(file = file, header = header, sep = sep, quote = quote,  :
more columns than column names"
In particular, I'm trying to import a xls file (http://www.pcr.uu.se/digitalAssets/159/159834_1external_support_compact_dataset_1.00_20110325-1.xls) in order to merge it together with two other datasets and to create one single data frame.
I was trying some things and looked into online forums discussing possible solutions, but I just couldn't fix it on my end... I'm assuming it's not a huge issue, but if someone has had a similar problem or knows how to fix this I would be very happy!
Here's the link to the R Script file including the code:
https://github.com/KatrinHeger/CollaborativeResearchProject/blob/master/Importing_UCDP_ES.R
Thank's a lot!
Best,
Benedikt

Dear Benedikt,
Daniel and I had a similar problem. It related to the fact, that the .xls file was not nicely formated: containing a "header" with information regarding the file, not all variables were named and some copyright info at the end of the file.
We used the options "skip" to get rid of the header including the variable names, used "col.names" to manually name the variables and "nrows" to limit the number of rows loaded (and thereby getting rid of the additional info at the end of the document).
For us it worked then. Hope it helps,
Lars

Dear Lars,
Thank you very much for your quick reply and help! Did you use a specific package to perform those commands or are they already part of the basic R Studio suite?
Thanks a lot!
Best,
Benedikt

Dear Benedikt,
These options are part of the read.table command from the utils package (usually attached to R).
Here is an example: read.table(file = "", sep="", na.strings = "", nrows = , skip = , header=FALSE, col.names=c(...))
Hope this helps,
Lars

Thank's a lot and have a nice Saturday evening! :-)
Best,
Benedikt

From: LarsMehwald <notifications<EMAIL_ADDRESS>Sent: Saturday, November 7, 2015 8:57 PM
Subject: Re: [SyllabusAndLectures] Error when reading a xls file ("more columns than column names") (#58)
To: HertieDataScience/SyllabusAndLectures <syllabusandlectures<EMAIL_ADDRESS>Cc: mr-r0b0t <benediktabendroth<EMAIL_ADDRESS>Dear Benedikt,
These options are part of the read.table command from the utils package (usually attached to R).
Here is an example: read.table(file = "", sep="", na.strings = "", nrows = , skip = , header=FALSE, col.names=c(...))
Hope this helps,
Lars

Reply to this email directly or view it on GitHubhttps://github.com/HertieDataScience/SyllabusAndLectures/issues/58#issuecomment-154743334.

Also rio should work well:
library(rio)

main <- import('159834_1external_support_compact_dataset_1.00_20110325-1.xls')


Style Guide

Requirements

Filling out the template is required.

All new code requires tests to ensure against regressions.

However, if your PR contains zero code changes, feel free to select the checkmark below to indicate so.

[ ] Have you ran tests against this code?

[X] This PR contains zero code changes.

Description of the Change
This PR adds our Coding Style Guide.
That is the documentation of it, as well as the configurations within eslint and prettier to enforce it.
The majority of items that are here is based on the style that has naturally evolved for the codebase as it's been worked on.
The rules are simple, and non-exhaustive, ensuring not to get in the way of development, and not stopping a PR from being merged if broken.
Since @Digitalone1 is the major one that will be affected by this change I wanted to check in with you to see what you thought.
I tried to only include rules we are already following, but thought it'd be a good idea to define them someplace. So if this looks good to you awesome, otherwise please feel free to provide feedback on what should be changed, I don't want this to get in the way of anyone's development.

Very nice, I like this guides.
At the moment I have installed prettier outside the editor from AUR because I need also the plugin for PHP which I was not able to run as package inside pulsar. But I suppose there's no need to parse the existing code manually through prettier since it's done by GH Actions after the commit, right?

Very nice, I like this guides.
At the moment I have installed prettier outside the editor from AUR because I need also the plugin for PHP which I was not able to run as package inside pulsar. But I suppose there's no need to parse the existing code manually through prettier since it's done by GH Actions after the commit, right?

Yeah you shouldn't need to worry about prettier setup at all.
In fact prettier has been running for over 4 months now on the repo, just bundles it up into the GitHub Standards Action so that you or I never have to worry about it

@Daeraxa Added the requested changes


Create phar file for more comfortable

Create phar file for more comfortable

This is a good idea and there is already a WIP pull request for this in #28.

Nice info


revise charm report

Revising the comment on mechanism and the comment and score on additionality for the Charm Industrial project.

reviewed offline with @jhamman , gonna merge in


Allow optional balancer in config

Refs: https://github.com/metarhia/metacom/issues/199

[x] tests and linter show no problems (npm t)
[x] code is properly formatted (npm run fmt)
[x] description of changes is added in CHANGELOG.md

Landed in https://github.com/metarhia/impress/commit/02c414424087dbe09d458351daee9208d5719d9e


Offline group members on join

Every time I invited someone to party they appeared with "thunder" aka offline mark on their avatar(class icon in my case). I did /reload and then it shows correctly.

Thanks for all your reports. I'll look into this one.

Hopefully fixed with https://github.com/Sticklord/oUF_Abu/commit/be2e99bc3bfdfce60fd3bb94ae1bec6fe790efae


Protocol Mapper Issue

When working with the protocol mappers I encounter a severe bug in the JSON implementation.
This class Keycloak.Net.Models.ProtocolMappers.Config should be changed to Dictionary<string,object>
Per the docs https://www.keycloak.org/docs-api/5.0/rest-api/index.html#_protocolmapperrepresentation this attribute it's type map meaning that there's not defined structure so a named based structure like the one you have is not working and will not retrieve all the information.
please change it to Dictionary class:
see also
https://docs.microsoft.com/en-us/dotnet/api/system.text.json.serialization.jsonextensiondataattribute?view=net-5.0
https://stackoverflow.com/questions/14893614/how-to-serialize-a-dictionary-as-part-of-its-parent-object-using-json-net

Suggested change:
Keycloak.Net.Models.ProtocolMappers.ProtocolMapper
[JsonProperty("config")]
public Config Config { get; set; }
should be replaced to:
[JsonProperty("config"), JsonExtensionData]
public Dictionary<string, object> Config { get; set; }

I have created a new nuget package that has this fix https://www.nuget.org/packages/Keycloak.Net.Core/


[Feature] Get Original File Name

When I add a file to the gallery in IOS, the file name changes. But the original file name must be stored somewhere, because Google Photos shows the original file name.
Is it possible to retrieve the original file name in iOS and Android?
This way it would be possible to retrieve the same image on both Android and iOS devices after the image has been synchronized from one device to the other through Google Photos.

Use next code to include the library.
MaterialApp(
  builder: (context, widget) {
    return LoadingProvider(
      child: widget,
    );
  },
);

In iOS, the file name is not a public field, using this field to search for unknown risks (system updates may cause app crash).

Although PHAsset has a kvo attribute to get "originalFilename", it cannot be used as a predicate condition.

I have a similar issue. I save some data about the photo in my app database. And we have to find a way to keep this data when the user changes his phone model for example and download all pics again. I saw there is a field on iOS EXIF Dictionary called: kCGImagePropertyExifImageUniqueID. This field should not change even when redownloading the photo, right? Is there a way we could retrieve this value?

After some testing I saw that the "filename" (maybe the same as the ImageUniqueID?) is still kept when downloading a photo twice. I tested by saving a photo to the Files app in iOS than saving it back again on the Photo Library, I ended up with two images with the same "filename"...... But if just sharing and saving the photo directly in the gallery the filename changes.... I was thinking.... If we could have access to another field like the "keywords" for example, we could put a generated id there.... And in my tests this will be kept when saving the photo twice or sharing and saving it again......
Is this a feature we could implement?


Making About us part Responsive

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I would like to add the links and hover effects to make it Responsive.
Use Case
This feature will benefit all the Career Zunction users
Benefits
Who are using this website can go through that links and search the internship they like
Add ScreenShots

Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

I would like to work on this issue . so please assign this task to me under the label gssoc24 and level 2,or 3

Under GSSOC 24, Please assign the work to me!

what links are you willing to add @n14rishitha

what links are you willing to add @n14rishitha

Like when they click on those. A page with all different types of Internships will open So they can choose from them.


f.inputWrangling() error

Issue
When I am running the fb_robyn_exec Script and run the
dt_mod <- f.inputWrangling()
I get  this error:
Error in if (any(train_all0)) { : missing value where TRUE/FALSE needed
Steps to Reproduce
I can provide the csv and my code if needed. If there are any common fixes it would be appreciated.
Expected Behavior
I expect the code to run without error
Environment
R version (4.1.0)

I had some blank values in the dataset. I once I fixed that it resolved the issue.


SAML assertion encryption is not working for the sub orgs due to missing registry initialisation for the sub org

Describe the issue:
Core cannot find the private key of the sub organization since the registry is not initialized properly at the time of private key retrieval.
How to reproduce:

Create a sub organization in the IS
Add a standard based SAML authenticator with encryption enabled for the SAML assertion
Try to login via the IDP
IDP authentication failed since the system cannot find the private key for the sub organization

NOTE : When the registry is being initialized via retrieving the meta file in the browser, the registry get initialized.
i.e: https://localhost:9443/t/daa61cbb-28ca-4918-89e0-d40a9a205c4e/identity/metadata/saml2
Expected behavior:
SAML login should work without any error when the assertion is encrypted.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: [e.g., IS 5.10.0, IS 5.9.0]
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

Have decided to use the public cert of the root organization. Hence not initialising the sub-org registry to load the decryption keys can be neglected as the root organization's key can be used to decrypt.


Add ?next= param to /logout

Also to other routes if relevant.

This needs to be added to /auth/<provider>/callback as well.


Tensorflow mystique issue with MirroredStrategy freeze on particular local GPU combo

https://stackoverflow.com/questions/52425485/tensorflow-mirroredstrategy-freeze-on-particular-gpu-combo
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04 host Linux Ubuntu 16.04 inside Docker container
TensorFlow installed from (source or binary): source and binary
TensorFlow version (use command below): 1.10.1 - 1.11.rc1
Python version: 3.5 - 3.6
Bazel version (if compiling from source): 0.17.1
GCC/Compiler version (if compiling from source): 5
CUDA/cuDNN version: 9.0, 9,2/7.2
GPU model and memory:
(0) Nvidia Titan x 980 12gb
(1) Nvidia Titan x 980 12gb (same)
(2) Nvidia 1080ti 11gb
(3) Nvidia 1080ti 11gb (same)
Exact command to reproduce: example source below

Describe the problem
Four GPUs on the same desktop:
(0) Nvidia Titan x 980 12gb
(1) Nvidia Titan x 980 12gb (same)
(2) Nvidia 1080ti 11gb
(3) Nvidia 1080ti 11gb (same)
MirroredStrategy (say from the obvious example code below) goes ok in combinations of the above (1,2), (1,3) but freezes on (2,3), (1,2,3).
(0 - not used at distribution as causes another sort of CUDA_WAIT_TIMEOUT error due to guessed kernel timeouts for being used to render display)
Have no idea where to go further. Checked with original binary docker images both latest and nightly. Checked with compiled from source TensorFlow r1.10-r1.11 within original nvidia/cuda binary docker image. All the same behaviour.
Source code / logs
import os
os.environ["CUDA_DEVICE_ORDER"]="PCI_BUS_ID"
os.environ['CUDA_VISIBLE_DEVICES']="2,3"

import tensorflow as tf

def model_fn(features, labels, mode):
  layer = tf.layers.Dense(1)
  logits = layer(features)

  if mode == tf.estimator.ModeKeys.PREDICT:
    predictions = {"logits": logits}
    return tf.estimator.EstimatorSpec(mode, predictions=predictions)

  loss = tf.losses.mean_squared_error(
               labels=labels, predictions=tf.reshape(logits, []))

  if mode == tf.estimator.ModeKeys.EVAL:
    return tf.estimator.EstimatorSpec(mode, loss=loss)

  if mode == tf.estimator.ModeKeys.TRAIN:
    train_op = tf.train.GradientDescentOptimizer(0.2).minimize(loss)
    return tf.estimator.EstimatorSpec(mode, loss=loss, train_op=train_op)

def input_fn():
  features = tf.data.Dataset.from_tensors([[1.]]).repeat(100)
  labels = tf.data.Dataset.from_tensors(1.).repeat(100)
  return tf.data.Dataset.zip((features, labels))

distribution = tf.contrib.distribute.MirroredStrategy()
config = tf.estimator.RunConfig(train_distribute=distribution)
classifier = tf.estimator.Estimator(model_fn=model_fn, config=config)

classifier.train(input_fn=input_fn)

classifier.evaluate(input_fn=input_fn)

freeze_output_r1.10.txt
freeze_output_r1.11.txt
normal_completion_output_r1.10.1.txt
normal_completion_output_r1.11.txt
tf_env_r1.10.1.txt
tf_env_r1.11.txt

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Mobile device

There was mentioned a lot ...:, ...:, ...:
ASUS plx chip and bios related issue.
Kernel parameter pci=nommconf has fixed everything.
iommu off - should as well but as a last measure.


Even with specific MS package source there is no package named "mdatp" for Ubuntu 18.04 in "prod" channel

Repro

Install Ubuntu 18.04 32-bit (in XenServer).  The architecture will show as "i686"
Follow all steps about setting up Microsoft's package repo for the "prod" channel.
Confirm that Microsoft's package repo is present by running cat /etc/apt/sources.list.d/* and seeing this line: deb [arch=amd64] https://packages.microsoft.com/ubuntu/18.04/prod bionic main
Note that it's for https://packages.microsoft.com/ubuntu/18.04/prod unlike the example given in this page which states deb [arch=amd64] https://packages.microsoft.com/config/ubuntu/18.04/prod bionic main (the example has an extra "config").  That's because the "config" URL is not in the "prod.list" at https://packages.microsoft.com/config/ubuntu/18.04/prod.list
Run sudo apt -t bionic install mdatp as suggested in the article.

Expected

That it install the MDATP.

Actual

You get an error message that states E: Unable to locate package mdatp

Notes
Once I tried with the amd64 architecture install of Ubuntu then everything worked flawlessly.  So it seems that this issue is just limited to trying to install MDATP on a 32-bit Ubuntu.  If that is not supported (is it?), then we can close this ticket as "not supported".

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

ID: bab0b894-0199-7515-b060-a6724e2c9b9e
Version Independent ID: 988ff83c-c4b5-d2f3-023e-2d8dd191b7ae
Content: Deploy Microsoft Defender for Endpoint on Linux manually
Content Source: microsoft-365/security/defender-endpoint/linux-install-manually.md
Product: m365-security
Technology: mde
GitHub Login: @Dansimp
Microsoft Alias: dansimp

Notes
Once I tried with the amd64 architecture install of Ubuntu then everything worked flawlessly.  So it seems that this issue is just limited to trying to install MDATP on a 32-bit Ubuntu.  If that is not supported (is it?), then we can close this ticket as "not supported".

@Dansimp Please help us in resolving this issue. Thanks

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Dedicated page for principal propagation in SAP Help site for SAP Task Center.

Issue description

Hello
Principal propagation is a great value offered by SAP Task Center (wherever applicable) and can make it a preferable option for consolidating and managing tasks across several applications. It can be a distinguishing feature against other options like creating a custom app.
Therefore, I believe a dedicated page explaining principal propagation, and how SAP Task Center provides and handles it, would be a great addition to the SAP Help site for SAP Task Center.
Kind Regards
Serdar
Feedback Type (Optional)
content gaps
Page Title on SAP Help Portal (prefilled)
Security
Page URL on SAP Help Portal (prefilled)
https://help.sap.com/docs/task-center/sap-task-center/security?locale=en-US

Hi, @ssimsekler, thanks for your feedback! We’ll look into it and come back to you if we have any questions.

Hi Serdar, thank you very much for your suggestion. Once SAP Task Center supports 3rd party task provider integration, there would be additional documentation provided to help implementation and integration efforts including coverage for principal propagation flows.


Attempt at running integration tests in a goal

We need to simplify our goals in machine.ts/goals.ts as they are getting out of hand! I've not taken on that work now, as I want to see if it works first.
We might choose enable integration tests for every service, but just for the moment it is conditional on a file in the repo existing so we can try it out on one or two repos first.

@whostolebenfrog  no services yet. will probably start with org service.


Azure Active Directory Identity Protection

Change(s):
- New Schema for Data connector: template_AzureActiveDirectoryIdentityProtection
Version Updated:

2.0.3
Resolved PR #7215

Hi @v-prasadboke, data connector not visible in manage section also check count and number of contents.

Hi @v-prasadboke, please check comments above.

Hi @v-prasadboke, please respond on comments above. Thanks.

Hello @v-atulyadav as these solution are 1p conenctors there is an issue with the links which are present in the data connector description

Looking into this


Migrate to CSS Modules

Plan:

Setup CSS Modules on project and migrate one component
Bring css modules to storybook
Check build and services integration
Migrate other components to css-modules
Remove styled-components from build and storybook


Add atomic_long::get_and_decrement method

Added atomic_long::get_and_decrement method.
Also added a test for atomic_long::increment_and_get
backport from https://github.com/hazelcast/hazelcast/pull/17699

Linux test PASSed.

Windows test FAILed.

verify-windows

Windows test FAILed.

verify-windows

Linux test FAILed.

Windows test PASSed.

Windows test FAILed.

Linux test FAILed.

Windows test PASSed.

verify-linux

Linux test PASSed.


As a user with a hearing impairment, I want to not be dependent on sound, so that I can successfully experience all elements of the site.

As a user with a hearing impairment, I want to not be dependent on sound, so that I can successfully experience all elements of the site.
Story
This issue was given a size of: Medium
This issue is important one but can be held off until the later stages as it isn't as vital to the core build.

As of right now no code on the site is using sound dependent notifications.


Cannot upload jpeg image

When I want to upload jpeg image it is always failed, while the png file manage to be uploaded.
Then after add new user which has profile picture uploaded the next add user still has the png file stuck in it. Is this a bug? is this bug also what makes the jpg files failed to be uploaded?

for the first issue I need to run a debug to make sure what is the content of the array $_FILES(["newPict"]["type"]) really is.
as for the second bug that is because I set the png pict processing as last resort for else. Since the pict is already set any value even if the file is blank will be processed as png image.

done and fixed. I add another else option in case the system failed to detect the photo file type. Looks like the $_FILES['newPict"]["type"] is not really consistent. Glitches do happen with this command which I failed to understand why.


Add QR code scanner for withdrawal invoices

Close #598
Added camera icon:

Clicking on camera icon opens QR code scanner:

Picture is just blurry because my webcam is bad. Scanning QR codes works well with mobile camera.

This is awesome!


question in calculating the norm factor for haar feature

https://github.com/Itseez/opencv/blob/master/apps/traincascade/features.cpp#L12
E.g., the sample is 20x20 size, this line shows that the rect used to calculate norm factor is the center rect with size of 18x18 which just excluding one pixel border.
This will help to improve the result? or, do I miss something?
@StevenPuttemans @vpisarev
As I found in some matlab implementation of VJ framework, the rect used to calculating norm factor is exactly the same size of sample.

Problem is that I am still after 2 years into the topic sometimes wondering why some of the implementation details have been done. Like you say, this is different to the matlab equivalent and I have yet to discover why. I guess it might have to do with the default step of the sliding window?

I have done some experiments and the result shows that the same size as the sample for norm rect have slightly better performance, thus I suggest to change this parameter in the default implementation.

I suggest you to make a PR, add test results before and after changing it and it will surely get in!


CMake: Update tensorflow and llvm submodules.

This is a adds changes to get the CMake build working again. It includes the commits of #744 and #762, which can be removed from this PR after those made it to the master, followed by a rebase.

The main change applies to iree_opt_library. It is necessary to register the transforms libraries. Further some additional passes were missing. We could consider if we really want to link anything into iree_opt_library or if we leave this to the depending targets, e.g. check-opt. This is the way MLIR does it. The target MLIRMlirOptMain has only few dependencies, whereas the whole list it added to the tool mlir-opt. That's also the reason why whole_archive_link() does not has to be used anymore. In MLIR's CMake configuration whole_archive_link() is only used to build mlir-translate and called for
  MLIRSPIRVSerialization
  MLIRTargetLLVMIR
  MLIRTargetNVVMIR
  MLIRTargetROCDLIR

For IREE, it is sufficient to just link MLIRSPIRVSerialization as a whole archive into iree-translate or rather into iree_translate_library.
Anyway, these changes allow to use CMake again :)

Awesome, thanks @marbre!  Yeah we should hand pick the dialects/passes only useful to us. Less dependencies is better. :) I think it's reasonable to drop  MLIRTargetNVVMIR and MLIRTargetROCDLIR given we've no use of them. For MLIRTargetLLVMIR, I'm not sure about the current progress on LLVM CPU side. Maybe it will be used later. But if it does not fail to remove at the moment, we can also remove it and add later when it's needed.

(Just checking the build status, haven't looked into the details of these updates)
Awesome, linux_bazel and linux_cmake Kokoro builds are passing :)
The clang-format "error" on the style check is coming from https://github.com/google/iree/pull/762, but we should be able to ignore it - those files should be auto-formatted on import.

I've re-exported this as https://github.com/google/iree/pull/767, since I can't rebase this PR


deployer version upgrade

Deployer Version upgraded to : 4.1

Thank you for PR, but Deployer 4.x have some break BC. I need check deploy.php file again and make it working with new version of Deployer.
Can you do it? Welcome ! :clap: :tada:

@anmolnagpal
I made update for Deployer 4.1
:beers: for Vietnamese Tết Holiday :beers:


[fix] escape some strings in query builders

Escape comments and other string fields that are not currently escaped in our query builders.
Test Plan

[ ] acceptance tests

References

Fixes #127

Codecov Report

Merging #231 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #231   +/-   ##
=======================================
  Coverage   65.81%   65.81%
=======================================
  Files          50       50
  Lines        3698     3698
=======================================
  Hits         2434     2434
  Misses        937      937
  Partials      327      327

Impacted Files
Coverage Δ

pkg/snowflake/schema.go
94.28% <100.00%> (ø)

pkg/snowflake/stage.go
67.59% <100.00%> (ø)

pkg/snowflake/view.go
86.15% <100.00%> (ø)

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 049eaae...cc5462d. Read the comment docs.


Add metrics of ellapsed time in different stages when scanning

Description
We have different stages when scanning a project (e.g. file gathering, parsing, conversion, query execution). The goal is to track times between these different stages and present them in our logs

Closed in #2961


faster besselj0 for x<25

this is about 2x faster on my computer where applicable. This also should be done for besselj1 and bessely0, bessely1

Here's a plot of relative errors.

Though it's another branch. It might be best to keep the power series for very small arguments to ensure that besselj0(0.0)= 1.0 (test that fails currently).

Fixed.  I used 1 to few terms, and didn't force the 1st coefficient to be 1.0.

Codecov Report

Merging #15 (73179d2) into master (f772ccf) will decrease coverage by 0.90%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
- Coverage   99.56%   98.65%   -0.91%
==========================================
  Files          12       11       -1
  Lines         683      670      -13
==========================================
- Hits          680      661      -19
- Misses          3        9       +6

Impacted Files
Coverage Δ

src/besselj.jl
100.00% <100.00%> (ø)

src/constants.jl
90.62% <0.00%> (-9.38%)
:arrow_down:

src/Bessels.jl

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 f772ccf...73179d2. Read the comment docs.

Also, for future reference, here is the code used to generate the polynomials
using ArbNumerics
function r(x,root)
    abs(x)<eps(Float64) && return -besselj(1,ArbFloat(x)+root[1]+root[2])
    return besselj(0,ArbFloat(x)+root[1]+root[2])/x
end
function r2(x,root)
    return besselj(0,ArbFloat(x)+root[1]+root[2])
end
polys = []
for i in 2:2:16
    poly = Float64.(Tuple(ratfn_minimax(x->r(x,pts[i]), ((pts[i-1][1]-pts[i][1]-.1)/2, (pts[i+1][1]-pts[i][1]+.1)/2), 12, 0)[1]))
    push!(polys, (0.0, poly...))
    if i!=16
        poly = Float64.(Tuple(ratfn_minimax(x->r2(x,pts[i+1]), ((pts[i][1]-pts[i+1][1]-.1)/2, (pts[i+2][1]-pts[i+1][1]+.1)/2), 13, 0)[1]))
        push!(polys, poly)
    end
end
Tuple(polys)

for the 0-pi/2 range, the invocation was (1.0, Float64.(Tuple(ratfn_minimax(x->(besselj(0,sqrt(x))-1)/x, (1e-20,nextfloat(pi/2)^2), 7, 0)[1]))...)

We may also want to extend this range just a bit more to blend with the asymptotic expansion better. I had chosen that cutoff to blend well with the previous method but we may want to use this for x < 26.0 now ?

Makes sense. Do you want to do the honors, or should I?

If you have it set up that might be easiest! Otherwise, I can look more at this tomorrow and also for J1, Y0, and Y1. But this is awesome. Hoping to finally finish these special cases! (at least for Float64)

updated (the post with computation details and error plots are also updated).

The last 2 points are wrong. They should be
(25.903672087618382, 4.894530726419825e-16) and (26.1,0.0). (you want the last point to be the reflection of the last root around your cutoff value).

It looks like I also misspoke were pts contains the roots of besselj0 and besselj1? Unsure what the first point is though.

The derivative of besselj0 is -besselj1, so extrema of besselj0 are roots of besselj1. The first point is a reflection of the second point around x=pi/2 (which is the lower cutoff)

This now is implemented for besselj1 although something with my polynomials is slightly broken, which is causing spikes in error at the extrema of the function which really shouldn't be happening.

This is now failing tests because SpecialFunctions is too inaccurate near 3.8317059702075125. The tests probably need to use Arb.

I agree that we should probably switch the tests over to Arb for all this. I actually had a draft of this early on but ran into https://github.com/JeffreySarnoff/ArbNumerics.jl/issues/48 causing some issues. Now that it is fixed we should move over.
Though, for this test besselj1 is computed in higher precision I believe using SpecialFunctions.jl as they provide BigFloat implementations. I'll have to check this some more.

I added a fix to the sign error for besselj1 and updated spacing, moved polys to only file. The sign error could be done in a different way perhaps. This is ready to merge on my end.


add Telegram::Client#disconnect

Hi @ssut,
I would love too see an explicit #disconnect method or is there any other graceful disconnect possibility at the moment?
Thank you very much

Hello there,
No way right now, I've been trigging keyboard interrupt when I need to stop it. If you are going to make a pull request it would be appreciated.

I'm also triggering a keyboard interrupt but I'm using a trap context) where I'm triggering cleaning up (closing files, closing sockets etc. like trap('INT') { telegram_client.disconnect }).
Anyway I saw that @GeNiuS69 implemented the trigger for a graceful disconnect and I asked him about making a PR already (this is why you should see the comment reference about your answer).

Ahh I see, I thought it was yours. I therefore, wait for a PR of it otherwise I'll merge it after I make a PR by myself if it doesn't matter.

Awesome! Thank you! :tada: